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.

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

Copy Button still exists At the Bottom of the reproduce python code block #1254

Closed
0x23d11 opened this issue Jan 23, 2024 · 3 comments · Fixed by #1256
Closed

Copy Button still exists At the Bottom of the reproduce python code block #1254

0x23d11 opened this issue Jan 23, 2024 · 3 comments · Fixed by #1256

Comments

@0x23d11
Copy link
Contributor

0x23d11 commented Jan 23, 2024

Screen.Recording.2024-01-23.at.5.31.06.AM.mov

I have recently worked on adding a copy button on the top right corner of the codeblock and its working for both household and policy and after that I removed the copy button from PolicyReproducibility.jsx file. But this only removed the button from policy page and not in the household page.

@0x23d11
Copy link
Contributor Author

0x23d11 commented Jan 23, 2024

I would love to work on it, but please explain does the text placement in the end "Run the code below into a Python notebook to reproduce the microsimulation results." is correct?

@MaxGhenis
Copy link
Contributor

I think we should remove the notebook note. Let's keep it clean and potentially move toward #263

@MaxGhenis
Copy link
Contributor

MaxGhenis commented Jan 23, 2024

Actually saw the notebook prepopulates some helpful code to install the package so we can keep it. Just remove microsimulation from the household snippet (or both household and microsim if easier) and change into to to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Closed
Development

Successfully merging a pull request may close this issue.

2 participants