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.

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

Code style black #1201

Merged
merged 16 commits into from
Jun 26, 2020
Merged

Code style black #1201

merged 16 commits into from
Jun 26, 2020

Conversation

wkerzendorf
Copy link
Member

Switching code-style to black - one subpackage at a time.

@wkerzendorf wkerzendorf force-pushed the code_style_black_main branch from 695035d to 8d3510c Compare June 24, 2020 21:30
@wkerzendorf wkerzendorf marked this pull request as draft June 24, 2020 21:30
@wkerzendorf wkerzendorf force-pushed the code_style_black_main branch from 8d3510c to 9adb5b4 Compare June 24, 2020 21:40
@codecov
Copy link

codecov bot commented Jun 24, 2020

Codecov Report

Merging #1201 into master will decrease coverage by 0.01%.
The diff coverage is 70.03%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1201      +/-   ##
==========================================
- Coverage   77.73%   77.71%   -0.02%     
==========================================
  Files          92       91       -1     
  Lines        5731     5727       -4     
==========================================
- Hits         4455     4451       -4     
  Misses       1276     1276              
Impacted Files Coverage Δ
tardis/analysis.py 0.00% <0.00%> (ø)
tardis/constants.py 100.00% <ø> (ø)
tardis/io/atom_data/__init__.py 100.00% <ø> (ø)
tardis/io/atom_data/atom_web_download.py 42.10% <0.00%> (ø)
tardis/io/parsers/stella.py 0.00% <0.00%> (ø)
tardis/io/setup_package.py 0.00% <0.00%> (ø)
tardis/plasma/exceptions.py 87.50% <0.00%> (ø)
tardis/plasma/setup_package.py 0.00% <0.00%> (ø)
tardis/scripts/cmfgen2tardis.py 0.00% <0.00%> (ø)
tardis/simulation/__init__.py 100.00% <ø> (ø)
... and 68 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cb98efd...c624b47. Read the comment docs.

@wkerzendorf wkerzendorf force-pushed the code_style_black_main branch from 88d487d to 6c2f12f Compare June 26, 2020 17:41
tardis/io/decay.py Outdated Show resolved Hide resolved
tardis/io/util.py Outdated Show resolved Hide resolved
Co-authored-by: Jaladh Singhal <[email protected]>
)
)
"ionization data for the ion ({}, {})".format(
str(counts.index[counts.index != counts]),
Copy link
Member Author

Choose a reason for hiding this comment

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

need to change that.

@wkerzendorf wkerzendorf marked this pull request as ready for review June 26, 2020 19:01
@wkerzendorf wkerzendorf merged commit 3297d26 into master Jun 26, 2020
@wkerzendorf wkerzendorf deleted the code_style_black_main branch June 26, 2020 19:14
atharva-2001 pushed a commit to atharva-2001/tardis that referenced this pull request Oct 1, 2021
* add pyproject with black settings

* black codestyle for all except model, montecarlo

* Update tardis/io/atom_data/util.py

Co-authored-by: Jaladh Singhal <[email protected]>

* Update tardis/io/atom_data/util.py

Co-authored-by: Jaladh Singhal <[email protected]>

* Update tardis/io/config_reader.py

* Update tardis/io/config_reader.py

* Update tardis/io/decay.py

* Update tardis/io/util.py

Co-authored-by: Jaladh Singhal <[email protected]>

* Update tardis/plasma/properties/general.py

Co-authored-by: Wolfgang Kerzendorf <[email protected]>

* Update tardis/plasma/properties/general.py

Co-authored-by: Jaladh Singhal <[email protected]>

* Update tardis/plasma/properties/nlte.py

Co-authored-by: Jaladh Singhal <[email protected]>

* Update tardis/plasma/properties/partition_function.py

Co-authored-by: Jaladh Singhal <[email protected]>

* Update tardis/plasma/properties/partition_function.py

Co-authored-by: Wolfgang Kerzendorf <[email protected]>

* Delete properties.py

* Update tardis/plasma/standard_plasmas.py

Co-authored-by: Wolfgang Kerzendorf <[email protected]>

* Delete test_montecarlo.py

Co-authored-by: Jaladh Singhal <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants