-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update dev/main #276
Open
rhoadesScholar
wants to merge
327
commits into
dev/main
Choose a base branch
from
main
base: dev/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
Update dev/main #276
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
"validate_run" must now be called to do validation from run name. The Run must be preloaded and passed to call "validate" directly.
…to validate_patch
…acapo into validate_patch
There appear to be some python formatting errors in f604f40. This pull request uses the [psf/black](https://github.com/psf/black) formatter to fix these issues.
…to validate_patch
There appear to be some python formatting errors in cb300f5. This pull request uses the [psf/black](https://github.com/psf/black) formatter to fix these issues.
There appear to be some python formatting errors in 81ec675. This pull request uses the [psf/black](https://github.com/psf/black) formatter to fix these issues.
… class prediction tasks
If the kernel size of the task is set to 3, then this will cause errors if you are trying to use a model with input shape `(1, y, x)`. The head should just be mapping the penultimate layer embeddings into the appropriate dimensions for the task so it desn't need a size larger than 1.
…used anywhere else
Simplified the parameterized train test, and added validation. Fixed bugs that were found
There appear to be some python formatting errors in 85a6dda. This pull request uses the [psf/black](https://github.com/psf/black) formatter to fix these issues.
multiple unitteststs + fixes
`get_paths` gets the paths per dataset by searching for "raw", but should return the path to the dataset containing raw
`get_paths` gets the paths per dataset by searching for "raw", but should return the path to the dataset containing raw
Potential solutions to #349
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.
No description provided.