-
Notifications
You must be signed in to change notification settings - Fork 1
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
5 lisa new independent recovery analysis #6
base: develop
Are you sure you want to change the base?
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.
For test_ independentrecovery.py, please refer to the tutorial section Create test for new analysis
.
It should be under tests > pyincore_incubator > analyses. The related files should be in the same folder as well.
Also, the test doesn't pass. Please modify it accordingly.
# and is available at https://www.mozilla.org/en-US/MPL/2.0/ | ||
|
||
|
||
from pyincore.analyses.independentrecovery.independentrecovery import IndependentRecovery |
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.
This line should be from pyincore_incubator.analyses.independentrecovery.independentrecovery import IndependentRecovery
, because independentrecovery belongs to pyincore_incubator
@@ -0,0 +1,37 @@ | |||
from pyincore import IncoreClient, RepairService, MappingSet, Dataset, DataService | |||
from pyincore.analyses.independentrecovery.independentrecovery import IndependentRecovery |
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.
This line should be from pyincore_incubator.analyses.independentrecovery import IndependentRecovery
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.
Tests pass. Approve.
@IN-CORE/core-dev For testing, please install this pyincore PR IN-CORE/pyincore#606 |
No description provided.