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

Issues with plotting, date due to insufficient codes #48

Open
dhamalakamal opened this issue Nov 7, 2024 · 7 comments
Open

Issues with plotting, date due to insufficient codes #48

dhamalakamal opened this issue Nov 7, 2024 · 7 comments
Labels
help wanted Extra attention is needed question Further information is requested

Comments

@dhamalakamal
Copy link
Contributor

dhamalakamal commented Nov 7, 2024

  1. As I see that the code for ifft, padding /unpadding/ winwowing, removing noise is not being merged in "preparation.py" , the third part of HW5 couldnot be started. Is there any chance of those part being merged today?

  2. Is there any one who has already created jupyter notebook and started working for the third part? However, I have started working on it already.

  3. Also, I tried to work on that file as of the present form. I got the graph as follows:

image

image

I got the same plot for combined and raw data. I may not be correct. I hope of getting suggestions for this part too.

@laserlab
Copy link
Contributor

laserlab commented Nov 7, 2024

How about reviewing #46 so it can be merged

@laserlab
Copy link
Contributor

laserlab commented Nov 7, 2024

inverse fft is merged now @SchrodingersStruggle

@ojha-aditya
Copy link
Contributor

@dhamalakamal what is the name of your Jupyter notebook? Maybe we can name it data_presentation.ipynb right away so that there are no conflicts with differently named notebooks later. Could we take all data presentation members here @laserlab ?

@ojha-aditya
Copy link
Contributor

Also, @dhamalakamal which .json file did you use for generating these plots from the Mount Loa folders? I am little confused which one to use for my task that's why?

@dhamalakamal
Copy link
Contributor Author

@ojha-aditya, I named it as 'combined_plot_mount_lua.ipynb', But I havent sent a pull request.

And the json file I used is inside a folder name: 'mauna-loa-data/flask_monthly.json'

@dhamalakamal
Copy link
Contributor Author

@laserlab , I have got mid-term exam exam tommorow, So, I think I will not be able to review it today. But, I will try , if I got a time by midnight. Also, I would like to make a request for the extension of deadline of this HW till saturday midnight.

@dhamalakamal dhamalakamal added help wanted Extra attention is needed question Further information is requested labels Nov 7, 2024
@dhamalakamal
Copy link
Contributor Author

How about reviewing #46 so it can be merged

I have reviewed and left my comments on #46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants