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.

Sign up for GitHub

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

Jump to bottom

Prospector #156

Merged
merged 56 commits into from
Jul 7, 2023
Merged

Prospector #156

merged 56 commits into from
Jul 7, 2023

Conversation

muethela
Copy link
Collaborator

PR to add Gordon+23 results to main branch. Note HG20211127I JSON needs updating before merge occurs.

profxj and others added 30 commits June 1, 2022 05:58
runnin
b3c88e6
file
4f8532c
@muethela
adding new transmission curves
7e36f86
@profxj
Merge remote-tracking branch 'origin/RM' into prospector
295094c
@profxj
minor edit
d9cb48f
@profxj
Merge remote-tracking branch 'origin/RM' into prospector
45ebce7
@profxj
PATH on 20190520B
0a04cb6
@profxj
minor
9e879dc
Merge remote-tracking branch 'origin/main' into prospector
9261997
@profxj
Merge branch 'prospector' of https://github.com/FRBs/FRB into prospector
1206453
@muethela
adding more transmission curves
5cecbd5
@profxj
return avg DM halos
1aee222
Merge remote-tracking branch 'origin/main' into prospector
62e9329
new FRBs
a41802b
PATH wip
a3bf6f5
update z for FRB 20200906A
d943c9d
@profxj
20220610A
c35f3d5
@profxj
Merge branch 'prospector' of https://github.com/FRBs/FRB into prospector
8a1c847
@profxj
prepping for dervied
c97224f
@profxj
mo
1f29466
@profxj
Merge branch 'prospector' of https://github.com/FRBs/FRB into prospector
afe74db
@muethela
Merge branch 'prospector' of https://github.com/FRBs/FRB into prospector
b95c57a
@muethela
adding new defs for prospector derived
5529170
@muethela
adding sSFR def
19164c9
FRB added
e32493f
@shivi071089
Update FRB20210117A.json
ec58593 
DM value is updated.
@Astronomer-Mohit
dervied properties of FRB 20181030A
bdec78a 
From https://iopscience.iop.org/article/10.3847/2041-8213/ac223b
@profxj
minor
bced45e
@profxj
Merge remote-tracking branch 'origin/main' into prospector
7055b97
@muethela
clarifying derived property defs
e836256
profxj
profxj approved these changes Jul 7, 2023
View reviewed changes
Copy link
Contributor

@profxj profxj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great stuff!

You need to pull in main before we can merge it

@profxj profxj merged commit 92dde01 into main Jul 7, 2023
11 checks passed
@profxj profxj deleted the prospector branch July 7, 2023 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@almannin almannin almannin left review comments

@SunilSimha SunilSimha SunilSimha requested changes

@profxj profxj profxj approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@muethela @profxj @SunilSimha @almannin @shivi071089 @Astronomer-Mohit