Skip to content

changes - #16

Open
jizongFox wants to merge 65 commits into
origin_masterfrom
master
Open

changes#16
jizongFox wants to merge 65 commits into
origin_masterfrom
master

Conversation

@jizongFox

Copy link
Copy Markdown
Owner

Short Description

Please give a short summary of the main points of this PR

PR Checklist

PR Implementer

This is a small checklist for the implementation details of this PR.
If you submit a PR, please look at these points (don't worry about the RisingTeam
and Reviewer workflows, the only purpose of those is to have a compact view of
the steps)

If there are any questions regarding code style or other conventions check out our
summary.

  • Implementation
  • Docstrings & Typing
  • Check __all__ sections and __init__
  • Unittests (look at the line coverage of your tests, the goal is 100%!)
  • Update notebooks & documentation if necessary
  • Pass all tests
  • Add the checksum of the last implementation commit to the Changelog

RisingTeam

RisingTeam workflow
  • Add pull request to project (optionally delete corresponding project note)
  • Assign correct label (if you don't have permission to do this, someone will do it for you.
    Please make sure to communicate the current status of the pr.)
  • Does this PR close an Issue? (add closes #IssueNumber at the bottom if
    not already in description)

Reviewer

Reviewer workflow
  • Do all tests pass? (Unittests, NotebookTests, Documentation)
  • Does the implementation follow rising design conventions?
  • Are the tests useful? (!!!) Are additional tests needed?
    Can you think of critical points which should be covered in an additional test?
  • Optional: Check coverage locally / Check tests locally if GPU is necessary to execute

pre-commit-ci Bot and others added 30 commits October 4, 2021 19:51
updates:
- [github.com/psf/black: 21.7b0 → 21.9b0](psf/black@21.7b0...21.9b0)
- [github.com/executablebooks/mdformat: 0.7.7 → 0.7.10](hukkin/mdformat@0.7.7...0.7.10)
resolved partially imported errors by extracting ntuples to utils fol…
…config

[pre-commit.ci] pre-commit suggestions
… and fixed a bug on Random Crop transformation
adding padding as a feature. fixed a bug on percentile normalization,…
bug fixed: Oneof api has no attibute keys
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