Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates default parameter values for the refiner component and improves documentation consistency. The key changes are: (1) increasing the default n_restarts from 1 to 8 for better optimization reliability, (2) switching the default noise distribution from uniform to normal with corresponding kwargs, and (3) enhancing project metadata and documentation.
- Changed default
n_restartsfrom 1 to 8 across refiner and model classes - Updated default
refiner_p0_noise_kwargsfrom{'low': -5, 'high': 5}(uniform) to{'loc': 0.0, 'scale': 5.0}(normal) - Bumped version to 0.4.5 and expanded pyproject.toml metadata with keywords, classifiers, and URLs
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/flash_ansr/refine.py | Updated default n_restarts parameter from 1 to 8 in the fit method |
| src/flash_ansr/flash_ansr.py | Updated docstrings and default refiner_p0_noise_kwargs to use normal distribution parameters; updated load method default n_restarts to 8 |
| src/flash_ansr/baselines/skeleton_pool_model.py | Updated default refiner_p0_noise_kwargs to use normal distribution parameters |
| src/flash_ansr/baselines/brute_force_model.py | Updated default refiner_p0_noise_kwargs to use normal distribution parameters |
| pyproject.toml | Bumped version to 0.4.5; added keywords, classifiers, and additional project URLs; improved project description |
| docs/training.md | Added helper scripts section with brief documentation |
| docs/index.md | Updated example code to use explicit n_restarts=8 and improved code formatting |
| docs/getting_started.md | Updated example code to use explicit n_restarts=8 and improved code formatting |
| docs/evaluation.md | Streamlined evaluation documentation, reorganized content into clearer sections |
| README.md | Updated project title to "Fast Amortized Neural Symbolic Regression" and bumped version to 0.4.5 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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
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.