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

Merging #544 (58d71e1) into staging (acec608) will not change coverage.
The diff coverage is n/a.

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

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update acec608...58d71e1. Read the comment docs.

@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