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

jason first commit #19

Open

haoyunl2 wants to merge 3 commits into main from jason

Collaborator

haoyunl2 commented Mar 24, 2023

Jason first SEG commit.

haoyunl2 and others added 2 commits

March 23, 2023 20:23


          jason first commit

8cd3b37


          francis suggestion

70013db

Member

mloubout commented Mar 24, 2023

Few main comments

you can't have math in the text
you need to add the math to the figure
remove the (a), (b) these are way too big, same for start/middle/...
what the total injected amount
Axes/labels if figures aren't readable


          Change with Francis Idea

93c8da3

Member

ziyiyin97 commented Mar 24, 2023

Need to undo your last commit since it adds all unnecessary files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet