Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add interoperability chapter #168

Merged
merged 16 commits into from
Nov 15, 2023
Merged

Add interoperability chapter #168

merged 16 commits into from
Nov 15, 2023

Conversation

lazappi
Copy link
Member

@lazappi lazappi commented Apr 3, 2023 •
edited
Loading

Add the interoperability covering how to move data between R and Python (and other languages)

TODO:

  • Finish code that requires rpy2 (I currently can't get this to work in the notebook, would love some help)
  • Get comments from external people
  • Adjust based on reviewer comments

Reviewer comments:

  • Spell/typo check
  • Add a summary paragraph/box
  • Add MuData section
    • Mention altExps

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@github-actions github-actions bot added chore enhancement New feature or request labels Apr 3, 2023
@Zethson Zethson linked an issue Apr 4, 2023 that may be closed by this pull request
Interoperability #26
Closed
@Zethson Zethson changed the base branch from development to master April 24, 2023 08:20
@timtreis timtreis self-requested a review August 8, 2023 11:44
@alitinet alitinet marked this pull request as ready for review August 8, 2023 13:52
@alitinet alitinet assigned alitinet and unassigned alitinet Aug 8, 2023
@alitinet alitinet self-requested a review August 8, 2023 13:53
@lazappi lazappi marked this pull request as draft August 8, 2023 14:37
@Zethson Zethson marked this pull request as ready for review August 8, 2023 20:19
@lazappi
Copy link
Member Author

lazappi commented Nov 15, 2023

@Zethson I think I'm happy with this now if you want to give it a final check

@lazappi lazappi requested a review from Zethson November 15, 2023 15:49
@Zethson
pre-commit
b0fbd8f 
Signed-off-by: zethson <[email protected]>
Zethson
Zethson approved these changes Nov 15, 2023
View reviewed changes
Copy link
Member

@Zethson Zethson left a comment

Choose a reason for hiding this comment

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

Exceptional work - as always.

Thank you so much!

@Zethson Zethson merged commit def4a1a into master Nov 15, 2023
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alitinet alitinet alitinet left review comments

@Zethson Zethson Zethson approved these changes

@timtreis timtreis Awaiting requested review from timtreis

Assignees
No one assigned
Labels
chore enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Interoperability
3 participants
@lazappi @alitinet @Zethson