48 make it possible to give multiple checkpoints in analysis - #49
Merged
ClementRomac merged 14 commits intoSep 11, 2026
Merged
ClementRomac merged 14 commits into
ClementRomac merged 14 commits into
Conversation
…m_addr for simulator configuration
…ling and checkpoint integration - Updated plotting logic to utilize a color map for series and checkpoints in space coverage analysis. - Enhanced the runner module to support multiple discovery inputs, including checkpoints, and improved data structure for progression payloads. - Modified the visualization UI to replace comparison paths with dataset paths, allowing for more flexible analysis inputs. - Improved JavaScript functions for managing dataset rows and saving inputs in local storage. - Updated documentation to reflect changes in analysis input handling and visualization features. - Added validation for checkpoint manifests and improved error handling in discovery loading functions.
…enhance feature handling
…le colors for datasets, including UI updates for ancestor display options.
…ulation for improved performance and clarity
Roirtur
commented
Sep 8, 2026
Contributor
Author
There was a problem hiding this comment.
This was made to be able to change the configuration from a checkpoint. So if original checkpoint had knn = 3 and now i want knn = 2 from this checkpoint, it now works
Roirtur
commented
Sep 8, 2026
| with (directory / filename).open("w") as file: | ||
| json.dump(config, file, indent=2, sort_keys=True, default=str) | ||
| file.write("\n") | ||
| return filename |
Contributor
Author
There was a problem hiding this comment.
This adds the config.json file into checkpoint to trace what was the actual configuration of this checkpoint
Roirtur
commented
Sep 8, 2026
| raise ValueError(f"Checkpoint manifest must contain a JSON object: {manifest_path}") | ||
| checkpoint = Checkpoint(checkpoint_path, manifest) | ||
| _validate_checkpoint_manifest(checkpoint) | ||
| return checkpoint |
Contributor
Author
There was a problem hiding this comment.
Added to let user add a checkpoint via absolute path, same as random runs.
ClementRomac
deleted the
48-make-it-possible-to-give-multiple-checkpoints-in-analysis
branch
September 11, 2026 08:52
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.