Initial Merge Pull Request#2
Initial Merge Pull Request#2MishaSavchenko wants to merge 33 commits intoPhys767-Spring17:masterfrom
Conversation
drewjrosen
left a comment
drewjrosen
left a comment
There was a problem hiding this comment.
Basically you need to comment more on everything.
Log_of_planck_function.py
Outdated
There was a problem hiding this comment.
You should comment on everything below. Explain more what is going on.
Spectral_Radiance.py
Outdated
…ion from the MCMC calculation
…ons with an arbitrary and a calculated burn point
…esult int results.dat, Spectral_radiance uses results.dat to plot last estimate, spectral_radiance.py does the same with some issues
kelle
left a comment
kelle
left a comment
There was a problem hiding this comment.
I made several comments in line.
also, files in pycache should not be here. Several other students have had all these extra files in their repos as well. Consider asking David for help.
.cache/v/cache/lastfailed
Outdated
There was a problem hiding this comment.
this file should not be tracked by github. (Nothing in the .cache directory should be.)
There was a problem hiding this comment.
Pretty sure this file shouldn't be tracked either.
mcmc_test/Log_of_planck_function.py
Outdated
mcmc_test/Log_of_planck_function.py
Outdated
There was a problem hiding this comment.
delete "just". logflux is the log of the Planck function.
mcmc_test/Log_of_planck_function.py
Outdated
mcmc_test/Log_of_planck_function.py
Outdated
There was a problem hiding this comment.
I dont know what the code is doing here. what does Y1_B mean?
mcmc_test/Log_of_planck_function.py
Outdated
There was a problem hiding this comment.
where do these numbers come from?
3 participants
No description provided.