search

astro-datalab / notebooks-latest

Default set of Data Lab notebooks, by DL team and contributed by users
BSD 3-Clause "New" or "Revised" License
60 stars 49 forks source link

Remove %%time and related comments for Group 2 (10 notebooks) #199

Closed kareninysimba closed 1 year ago

kareninysimba commented 1 year ago

Notebooks in group 2 to be edited. These NBs got their %%time stamp and related "This may/will take XX seconds/mins" comments removed and their version updated:

./03_ScienceExamples/MagellanicClouds/ExploringStellarPopulationsAroundMCsWithVHSDR5.ipynb ./03_ScienceExamples/TimeSeriesAnalysisRrLyraeStar/TimeSeriesAnalysisOfRrLyraeStar.ipynb ./03_ScienceExamples/SpectralEnergyDistributions/MidInfraredPhotometry_unWISEdr1_AllWISE.ipynb ./03_ScienceExamples/SpectralEnergyDistributions/SpectralEnergyDistributionsInSplusDr1.ipynb ./03_ScienceExamples/DESI/01_Intro_to_DESI_EDR.ipynb ./03_ScienceExamples/DESI/02_DESI_EDR_SDSS_Comparison.ipynb ./03_ScienceExamples/LargeScaleStructure/LargeScaleStructureSdssLs.ipynb ./03_ScienceExamples/WhiteDwarfs/HypervelocityWhiteDwarfCandidatesInGaiaDR3.ipynb ./01_GettingStartedWithDataLab/01_JupyterPythonSQL101.ipynb ./01_GettingStartedWithDataLab/02_GettingStartedWithDataLab.ipynb

Assigning @rnikutta and @jacquesalice to review

kareninysimba commented 1 year ago

I believe I resolved that last night. I don't see it in the PR or in the status output any more. It seems like someone else was also modifying that nb? It was in my revised group 2 though.

rnikutta commented 1 year ago

Hi @kareninysimba it still shows up for me (see attached screenshot).But if it's ok with you, I think I can resolve this when I start merging our three PRs. diff-conflict