Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use model_name_suffix in _get_job_name() #209

Merged
merged 1 commit into from
Aug 27, 2019

Conversation

drachenbach
Copy link
Contributor

Use model_name_suffix to be able to customize the job name. Very useful to distinguish similar runs.

@codecov
Copy link

codecov bot commented Aug 27, 2019

Codecov Report

Merging #209 into master will decrease coverage by 0.27%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
- Coverage   89.32%   89.05%   -0.28%     
==========================================
  Files          12       12              
  Lines         731      731              
==========================================
- Hits          653      651       -2     
- Misses         78       80       +2
Impacted Files Coverage Δ
spotify_tensorflow/luigi/tensorflow_task.py 90.62% <100%> (ø) ⬆️
spotify_tensorflow/example_decoders.py 89.28% <0%> (-7.15%) ⬇️

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 af1faf8...a5d30c4. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Aug 27, 2019

Codecov Report

Merging #209 into master will decrease coverage by 0.27%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
- Coverage   89.32%   89.05%   -0.28%     
==========================================
  Files          12       12              
  Lines         731      731              
==========================================
- Hits          653      651       -2     
- Misses         78       80       +2
Impacted Files Coverage Δ
spotify_tensorflow/luigi/tensorflow_task.py 90.62% <100%> (ø) ⬆️
spotify_tensorflow/example_decoders.py 89.28% <0%> (-7.15%) ⬇️

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 af1faf8...a5d30c4. Read the comment docs.

@andrewsmartin andrewsmartin merged commit cbaa6b8 into spotify:master Aug 27, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants