Skip to content

Task unaware xor vs rxor random forest bootstrap experiment #544

Open
kfenggg wants to merge 33 commits into
neurodata:stagingfrom
kfenggg:staging
Open

Task unaware xor vs rxor random forest bootstrap experiment #544
kfenggg wants to merge 33 commits into
neurodata:stagingfrom
kfenggg:staging

Conversation

@kfenggg

@kfenggg kfenggg commented May 7, 2022

Copy link
Copy Markdown
Contributor

Reference issue

Experiment for task unaware learning.

Type of change

Add jupyter notebook containing experiment and functions file to run the experiment.

What does this implement/fix?

Adds an experiment that explores when xor and rxor are different enough to train a new random forest.

Additional information

@PSSF23 PSSF23 requested a review from jdey4 May 9, 2022 15:19

@PSSF23 PSSF23 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

black formatting needed. Also the file name is quite long. You can make it shorter.

@jdey4

jdey4 commented May 9, 2022

Copy link
Copy Markdown
Collaborator

Please do not print 10,20,30... in the notebook.

@kfenggg

kfenggg commented May 9, 2022

Copy link
Copy Markdown
Contributor Author

black formatting needed. Also the file name is quite long. You can make it shorter.

Fixed

@kfenggg

kfenggg commented May 9, 2022

Copy link
Copy Markdown
Contributor Author

Please do not print 10,20,30... in the notebook.

Fixed

@kfenggg kfenggg requested a review from PSSF23 May 9, 2022 16:37
@PSSF23

PSSF23 commented May 9, 2022

Copy link
Copy Markdown
Member

black formatting needed. Also the file name is quite long. You can make it shorter.

Fixed

Nope~

@kfenggg

kfenggg commented May 9, 2022

Copy link
Copy Markdown
Contributor Author

black formatting needed. Also the file name is quite long. You can make it shorter.

Fixed

Nope~

Are both files still formatted incorrectly? I ran black on the functions file and jblack on the notebook.

Edit: I just reran both, I think there were some changes made to the notebook so I reuploaded that. Please, let me know if there is still a problem.

@PSSF23 PSSF23 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kfenggg Check not passed. You can take a look yourself. black works for jupyter notebooks natively. Other packages will not pass.

@codecov

codecov Bot commented May 10, 2022

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.09%. Comparing base (acec608) to head (58d71e1).
⚠️ Report is 25 commits behind head on staging.

Additional details and impacted files
@@           Coverage Diff            @@
##           staging     #544   +/-   ##
========================================
  Coverage    90.09%   90.09%           
========================================
  Files            7        7           
  Lines          404      404           
========================================
  Hits           364      364           
  Misses          40       40           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kfenggg kfenggg requested a review from PSSF23 May 10, 2022 00:20

@PSSF23 PSSF23 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM @kfenggg Actually you can make the title shorter and more consistent as other XOR experiments: https://proglearn.neurodata.io/experiments/xor_rxor_exp.html. You should also add subsections to make the tutorial more readable.

@jdey4 your thoughts?

kfenggg added 4 commits May 9, 2022 20:33
@kfenggg kfenggg requested a review from PSSF23 May 10, 2022 00:41
@jdey4

jdey4 commented May 17, 2022

Copy link
Copy Markdown
Collaborator

I think, this PR needs more work to publish it in the main or staging branch. But this is good for doing the grading. We can save the PR in a separate branch.

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.

3 participants