Merged
Conversation
…thon versions only
Streamlined Setup Process, New Citation Format
…nerate regardless of existing
…r each simulation
…e of floating point disrepancies
Reimplement Testing, Refactored ASTRO-DF
Performance Enhancements
* move ruff config into seperate file * update ruff configuration: increase line length to 88 and expand selected rules for enhanced linting * updated docstrings and formatting * fixed formatting and comments to meet new guidelines * Added docstring for file * fixed formatting and docs in directory, utils, and testing * removed unused x arg from deterministic_stochastic_constraints_and_gradients * fixed formatting, switched eval to literal_eval for security * fixed all formatting/doc issues outside of ARG002, D102, C901 * reorganized and reformatted dev utils to fix warnings * fixed classproperty decorators in child classes being incorrectly flagged as missing docstrings * fixed unused variable warnings * fixed majority of formatting errors for demos * fixed remaining formatting warnings in demos * ruff autofixes for core and gui * fixed demo formatting, ensured each was using a main() function * fixed broken gradient generation * converted os.path to Pathlib, general formatting fixes * fixed non-docstring line length warnings * removed security warnings * fixed all non-complexity warnings in exp base * switched from changing dir to adding to path, new existing results check * standardized file_path instead of filepath, improved handling of path vs file name args * updated demos to use PlotType enum * fixed path append order to allow for local MRG installs for dev utils * implemented custom override decorator, made check factor functions private * updated workshop notebook to use PlotType * commended out unused check_simulatable_factors methods for problems * fixed Path and formatting for sphinx config * fixed commented out line being too long * ruff autofixes for GUI classes * excluded GUI and mccabe issues from Ruff Checking * removed csv_filename arg from create_design * fixed possible unbound access if GUI isn't configured properly * moved docs config out of source folder * regenerated documentation modules * modified conf.py to work with new structure, seperated out API update into seperate script * added clean and apidocs options to makefile, moved index out of source folder * added docs-specific conda config * modified api-docs to list module at top of docs * removed old docs, moved conf.py/index.rst back into source * added speedup + ignored dirs to makefile, fixed path resolution for conf.py * fixed sphinx config path * refactored docs to use AutoAPI, reorganized file structure * added workaround for accessing simopt.directory without instantiation * changed references to subheaders instead of main headers * replaced amusement park grid png with rst table * added leave column to transition probabilities * changed docstrings to Google style * added AutoAPI folder to gitignore * added override to vector_to_factor_dict * added override to factor_dict_to_vector methods * added override to response_dict_to_objectives methods * fixed inconsistent formatting and broken links * removed colons on subheaders * removed descriptions for cleaner sidebar * mass swap to google docstrings * fixed formatting, disabled documentation of GUI * fixed formatting, changed headers for improved structure * added module links for API documentation to model/solver descriptions * fixed spacing issues * updated templates, standardized 4-space tab * moved templates, added logo to docs * reorganized conf, fixed broken links in index * fixed workshop notebook formatting
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
1 participant
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.
Changelog
Code
mean_priceValue #154Curveclass and related utils into separate filesutils.pyandutils.pyifiles to store common SimOpt utility functionsexperiment_window.pyGUI filemainmethodsTesting
test/expected_results/.yamlfiles were used to allow for quick manual comparisons against the expected valuestest_experiments.pyandexperiment_test_core.pyto reduce code duplication while still allowing for unittest and VS Code test runner compatibility.Environment
setup_simopt.shandsetup_simopt.batto streamline the setup processruff.tomlDocumentation