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

updates to support dashboard #21

Merged
merged 33 commits into from
Jul 11, 2024
Merged

updates to support dashboard #21

merged 33 commits into from
Jul 11, 2024

Conversation

EmnaTay
Copy link
Collaborator

@EmnaTay EmnaTay commented Jul 5, 2024

I added the status table like you suggested, i will change the dashboard to not be in system (1) but to go in a folder number 9. That s until i get data to create the dashboard

@hzaatiti-NYU
Copy link
Contributor

Hi Emna, amazing work!

can I ask you to resync your fork and update your branch with the most recent commits, please resolve the conflicts that you may get.
My most recent change is that I added to the conf.py an automation to generate the plotly HTML files. So now to change the data of the simple graph I made, we just need to change the data in docs/source/dashboards/generate_dashboard.py

In order for me not to disturb you much with resyncing every hour, I will stop (from now on) commiting directly to the main branch (Sorry about this), I will make my own pull requests and once their dense will merge them.

@Hzaatiti Hzaatiti marked this pull request as draft July 7, 2024 06:34
@Hzaatiti
Copy link
Owner

Hzaatiti commented Jul 7, 2024

Hi Emna, I set your PR to draft until you are done rebasing your branch with the latest commits from main.
Once your PR is ready for review please change its status to "Ready for Review"
Thank you :)

@hzaatiti-NYU
Copy link
Contributor

hzaatiti-NYU commented Jul 10, 2024

Im getting this error when trying to compute the SNR from a .fif data file, i made another branch on my repo for it if you wana check it out. Not sure how to solve this :) let me know if you have any idea
ValueError: file '/home/docs/checkouts/readthedocs.org/user_builds/meg-pipeline/checkouts/28/docs/source/dashboards/data/test-raw.fif' does not start with a file id tag

EDIT: Ok i think I have solved the issue, so I avoided using a .fif file which was causing a problem on the ubuntu build, instead I use the .con file type. The .con file is directly generated by the MEG-KIT system, the file is then red by the python script, a special function in MNE (MEG python library) gets the data from it called read_raw_kit

@hzaatiti-NYU hzaatiti-NYU marked this pull request as ready for review July 11, 2024 12:11
@hzaatiti-NYU hzaatiti-NYU merged commit 0c0dd7e into Hzaatiti:main Jul 11, 2024
2 checks passed
@EmnaTay EmnaTay deleted the newint branch July 12, 2024 13:10
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