o-smirnov / fundamentals_of_interferometry

Fundamentals of Radio Interferometry
GNU General Public License v2.0
4 stars 15 forks source link

Filename (FN) Errors #28

Open jfunction opened 7 years ago

jfunction commented 7 years ago

So, I wrote a shell command to catch some of the filename errors.

The command finds all .ipynb files, then finds the ones that don't match the pattern "some digits, an underscore, some more digits or x's, another underscore, some lowercase letters or underscores or digits, then the text .ipynb" and finally sorts the results. I admit it's a hack and could be improved upon.

find . -type f | grep \.ipynb | grep -Ev \[0-9\]\+_\[x0-9\]\+_\[_a-z0-9\\-\]\+\.ipynb | sort

Here are the current results:

0_Introduction/0_introduction.ipynb 0_Introduction/1_glossary.ipynb 0_Introduction/2_Appendix.ipynb 0_Introduction/editing_guide.ipynb 0_Introduction/.ipynb_checkpoints/editing_guide-checkpoint.ipynb 2_Mathematical_Groundwork/fft_implementation_assignment.ipynb 3_Positional_Astronomy/3_0_Introduction.ipynb 3_Positional_Astronomy/3_1_Equatorial_Coordinates.ipynb 3_Positional_Astronomy/3_2_Hour_Angle.ipynb 3_Positional_Astronomy/3_3_Horizontal_Coordinates.ipynb 3_Positional_Astronomy/3_4_Direction_Cosine_Coordinates.ipynb 4_Visibility_Space/4_1_Periodic_Functions_and_Fourier_Analysis.ipynb 4_Visibility_Space/4_2_The_Baseline.ipynb 4_Visibility_Space/4_3_The_2-element_Interferometer.ipynb 4_Visibility_Space/4_4_The_Visibility_Function.ipynb 4_Visibility_Space/4_5_1_UV_Coverage_UV_Tracks.ipynb 4_Visibility_Space/4_5_2_UV_Coverage_Improving_Your_Coverage.ipynb 4_Visibility_Space/4_6_The_Fourier_Approximation_VanCittert-Zernike_Theorem.ipynb 4_Visibility_Space/4_problem_set.ipynb 6_Deconvolution/clark_clean_assignment.ipynb 6_Deconvolution/hogbom_clean.ipynb 8_Calibration/8_0_Introduction.ipynb 8_Calibration/8_1_Calibration_Least_Squares_Problem.ipynb 8_Calibration/8_2_1GC.ipynb 8_Calibration/8_3_2GC.ipynb 8_Calibration/8_4_3GC.ipynb 8_Calibration/8_problem_set.ipynb 9_Practical/9_3_Observing_smearing.ipynb

jfunction commented 7 years ago

This is in reference to the editing guide

jfunction commented 7 years ago

Also all of 1 (Radio Science) - the numbering should be single digits where possible (so 01 should just be 1)

griffinfoster commented 7 years ago

It is tedious, but we should probably update all the chapters to fit the chapter 1 style (e.g. 01_04_radio_regime.ipynb) so that chapters with more than 9 sections are in the correct order when displayed in the juypter tree.

landmanbester commented 7 years ago

Hey man, I changed the numbering in chapter 1 because of the editing guide. General consensus here is to leave it as it is. Maybe we can write a script to do this write at the end. Ideas anyone?

On Tue, Feb 7, 2017 at 12:22 PM, Griffin Foster notifications@github.com wrote:

It is tedious, but we should probably update all the chapters to fit the chapter 1 style (e.g. 01_04_radio_regime.ipynb) so that chapters with more than 9 sections are in the correct order when displayed in the juypter tree.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ratt-ru/fundamentals_of_interferometry/issues/28#issuecomment-277957816, or mute the thread https://github.com/notifications/unsubscribe-auth/AQDondWOBG1mp2P8e7GULPchBIFkw6J8ks5raEXvgaJpZM4L4GhT .

-- Landman Bester (Ph.D) - Junior Algorithms Scientist

Square Kilometre Array (SKA) South Africa The Park, 4th floor, Pinelands 7405 Cape Town, South Africa

griffinfoster commented 7 years ago

Yeah, lets save this for later, it is a detail.