8000 Global code refactor by zawadzkim · Pull Request #47 · zawadzkim/pySWAP · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Global code refactor #47

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 143 commits into from
Feb 17, 2025
Merged

Global code refactor #47

merged 143 commits into from
Feb 17, 2025

Conversation

zawadzkim
Copy link
Owner

A large refactoring of the code base has been completed. Many new features have been added and some have been improved. Enhancements, among others:

  • more generic validation
  • modularized Model class
  • improved accessibility of model components and better managed imports
  • improved docstrings
  • Many new documentation pages and jupyter notebooks with examples
  • function parallelizing model runs

This PR implements all necessary features to run the models. More features will be added once the package gains traction and gets user feadback.

Mateusz and others added 30 commits July 5, 2024 23:21
… of git repo by default and a .gitignore file.
Mateusz and others added 29 commits January 13, 2025 18:04
Majority of work here is realted to cleaning up the imports. I want the user to be able to intuitively search for classes to define using IDEs, so the imports have to have a good structure.
Added a new plugin to control the order of items in the navigation.
Added some index.md files, gor rid of excess files in the docs, controlling the order of elements via the awesome pages plugin.

Improved some descriptions.
Changes will be described elswhere.
next step is to catch up with the docs and make sure other ways of loading the model still work.
In the next step I need to run tests and eliminate the remaining bugs
…ckage.

The restructurization is to fight off the circular imports issue. There are still some remaining after this commit.
It will be the default fix and the aim for the next releases to move all type-checking imports into TYPE_CHECKING guard.
It's a bit of a dirty commit before the weekend and leaving for home. Changes will be elaborated on in the PR description.
@zawadzkim zawadzkim merged commit 9bdd1ad into main Feb 17, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0