Skip to content
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

Restructure project #19

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Restructure project #19

wants to merge 6 commits into from

Conversation

cpaniaguam
Copy link
Collaborator

This pull request includes significant updates to the pyproject.toml file and the removal of the setup.py file. The most important changes involve updating project metadata, dependencies, and optional dependencies.

Metadata updates:

  • Updated the URLs for the "Homepage" and "Bug Tracker" to new GitHub repository links.
  • Added a license field specifying the license file.
  • Added keywords field to include relevant tags such as "LANs", "generative models", "cognitive science", "neuroscience", and "sbi".

Dependency updates:

  • Updated the versions of existing dependencies and added new dependencies to ensure compatibility with the latest versions.

Optional dependencies and development setup:

  • Added a new optional dependency group viz with wandb.
  • Defined a dev dependency group with various development tools such as coverage, ipykernel, jupyterlab, pytest, and others.

File removal:

  • Removed the setup.py file as it is no longer needed with the updated pyproject.toml configuration.

Copy link
Collaborator

@AlexanderFengler AlexanderFengler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good so far.
Is this still a draft?

@cpaniaguam
Copy link
Collaborator Author

This looks good so far. Is this still a draft?

Yes. I'd like to keep it this way until we have a test suite in place. #20

Copy link
Collaborator

@AlexanderFengler AlexanderFengler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@AlexanderFengler AlexanderFengler marked this pull request as ready for review February 28, 2025 01:05
@AlexanderFengler AlexanderFengler marked this pull request as draft February 28, 2025 01:06
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.

2 participants