-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/packaging #4
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few comments
.bumpversion.cfg
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't want it anymore?
README.md
Outdated
2. Install pre-commit hooks `pre-commit install` | ||
3. Eventually build documentation. | ||
3. Eventually build documentation by calling. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
By calling?
README.md
Outdated
@@ -207,8 +221,8 @@ We rely on a combination of `Black`, `Pylint`, `Mypy`, `Ruff` and `Isort` to enf | |||
## Building Documentations | |||
|
|||
1. Activate your virtual environment. | |||
2. install the `quadra` package with at least `doc` version or `dev` version. | |||
3. To run the webserver for realtime rendering and editing run `mkdocs serve` and visit `http://localhost:8000/`. | |||
2. install the `quadra` package with at least `doc` version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
install -> Install
pyproject.toml
Outdated
{name = "Refik Can Malli", email = "refikcan.malli@orobix.com"}, | ||
{name = "Silvia Bianchetti", email = "silvia.bianchetti@orobix.com"}, | ||
] | ||
keywords = ["deep", "learning", "experiment", "lightning", "hydra-core"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It makes sense to separate deep and learning?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope spaces are allowed. For me it is okay to add as deep learning
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If they are not allowed probably deep-learning is better for search
repository = "https://github.com/orobix/quadra" | ||
|
||
# Adapted from https://realpython.com/pypi-publish-python-package/#version-your-package | ||
[tool.bumpver] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
O:
Summary
pyproject.yaml
file for library information. It is a step taken to migrate fromsetup.py
.README.md
and fix documentation errors.Type of Change
Checklist
Please confirm that the following tasks have been completed:
Note: We need to retag after the merge.