-
Notifications
You must be signed in to change notification settings - Fork 2
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
New release from develop #139
Open
dstalzjohn
wants to merge
30
commits into
main
Choose a base branch
from
develop
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains 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
## 📥 Pull Request Description Changed all possible versions to >= ## 📝 Checklist Please make sure you've completed the following tasks before submitting this pull request: - [x] Pre-commit hooks were executed - [x] Changes have been reviewed by at least one other developer - [ ] Tests have been added or updated to cover the changes (only necessary if the changes affect the executable code) - [x] All tests ran successfully - [x] All merge conflicts are resolved - [ ] Documentation has been updated to reflect the changes - [ ] Any necessary migrations have been run
Automatically generated by python-semantic-release
…a `ResultAnalyzer` (#134) ## 📥 Pull Request Description This PR adds the following features and changes: - feat: Add possibility to pass additional arguments when initializing a `ResultAnalyzer` - refactor: Add an experiment context to the `initialize` function of the `DataframeAnalyzer` - refactor: Pass the experiment context of a train or eval pipeline run to the result analyzer. ## 👀 Affected Areas - Result analyzers - Analyse op - Train and eval job
Automatically generated by python-semantic-release
Automatically generated by python-semantic-release
Automatically generated by python-semantic-release
Automatically generated by python-semantic-release
…given datetime object The timestmap format used in niceml has a special foramt. If it is necessary to create this, it can now be created based on a given datetime object.
Automatically generated by python-semantic-release
…ult (#140) ## 📥 Pull Request Description Currently you cannot use `read_image` and `write_image` without specifying a FileSystem. The default (`LocalFileSystem`) was not used correctly. This is now fixed in this PR. ## 👀 Affected Areas - ioutils ## 📝 Checklist Please make sure you've completed the following tasks before submitting this pull request: - [X] Pre-commit hooks were executed - [ ] Changes have been reviewed by at least one other developer - [ ] Tests have been added or updated to cover the changes (only necessary if the changes affect the executable code) - [X] All tests ran successfully - [X] All merge conflicts are resolved - [X] Documentation has been updated to reflect the changes - [X] Any necessary migrations have been run
Automatically generated by python-semantic-release
## 📥 Pull Request Description Using listdir from FSSpec multiple times for the same path with a FileSystem other than LocalFileSystem (e.g. S3FileSystem or AzureFileSystem) resulted in outdated results, as listdir uses an internal caching. To fix this, the niceml function list_dir, which used fsspecs listdir, was exchanged with fsspecs function with the refresh option. Additionally, some typos were fixed. Github test pipeline was updated to use macos-13 instead of macos-12 ## 👀 Affected Areas Everywhere where the list_dir function of niceml is used. ## 📝 Checklist Please make sure you've completed the following tasks before submitting this pull request: - [x] Pre-commit hooks were executed - [x] Changes have been reviewed by at least one other developer - [ ] Tests have been added or updated to cover the changes (only necessary if the changes affect the executable code) - [ ] All tests ran successfully - [x] All merge conflicts are resolved - [ ] Documentation has been updated to reflect the changes - [ ] Any necessary migrations have been run --------- Co-authored-by: Nils Uhrberg <[email protected]>
Automatically generated by python-semantic-release
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.
📥 Pull Request Description
📝 Checklist
Please make sure you've completed the following tasks before submitting this pull request: