Skip to content

SDSS cones#30

Open
GeorgySk wants to merge 18 commits intomasterfrom
sloan_cones
Open

SDSS cones#30
GeorgySk wants to merge 18 commits intomasterfrom
sloan_cones

Conversation

@GeorgySk
Copy link
Owner

  • implement new color cuts for Sloan Digital Sky Survey (SDSS) simulations
  • implement halo WDs (calculate normalization cone mass)
  • add ellipses for Toomre diagram
  • compare results with an article by Anguiano (?)

@GeorgySk GeorgySk force-pushed the sloan_cones branch 2 times, most recently from 24c56e7 to ee95725 Compare September 29, 2017 21:14
@GeorgySk GeorgySk mentioned this pull request Oct 9, 2017
@GeorgySk GeorgySk changed the base branch from classification_task to master October 9, 2017 10:49
@GeorgySk GeorgySk mentioned this pull request Oct 9, 2017
effective_temperature = Column(Float(),
nullable=True)
surface_gravity = Column(Float(),
nullable=True)
Copy link
Collaborator

Choose a reason for hiding this comment

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

звучит круто


common /enanas/ luminosityOfWD,massOfWD,metallicityOfWD,
& effTempOfWD
& effTempOfWD, log_g
Copy link
Collaborator

Choose a reason for hiding this comment

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

почему здесь пробел, а строкой выше их нет?

Copy link
Owner Author

Choose a reason for hiding this comment

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

добро пожаловать в удивительный мир фортрана
насколько я понимаю, тут можно вообще на одной строке все пробелы опустить
даже end function и endfunction - одно и то же

Copy link
Owner Author

Choose a reason for hiding this comment

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

я старался везде пробелы расставлять. где не расставлены - там чужие следы старые

@GeorgySk GeorgySk self-assigned this Oct 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants