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

4.1.12 release branch #2042

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

4.1.12 release branch #2042

wants to merge 11 commits into from

Conversation

dbeltrankyl
Copy link
Contributor

@dbeltrankyl dbeltrankyl commented Dec 18, 2024

@dbeltrankyl dbeltrankyl added this to the Autosubmit 4.1.12 milestone Dec 18, 2024
@dbeltrankyl dbeltrankyl force-pushed the 4.1.12-release-branch branch from db71101 to 0231bce Compare December 18, 2024 14:00
@codecov-commenter
Copy link

codecov-commenter commented Dec 18, 2024

Codecov Report

Attention: Patch coverage is 93.03483% with 14 lines in your changes missing coverage. Please review.

Please upload report for BASE (master@3706cb9). Learn more about missing BASE report.
Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
autosubmit/notifications/mail_notifier.py 91.22% 2 Missing and 3 partials ⚠️
autosubmit/autosubmit.py 84.21% 1 Missing and 2 partials ⚠️
autosubmit/job/job.py 96.38% 2 Missing and 1 partial ⚠️
autosubmit/platforms/paramiko_platform.py 88.88% 2 Missing ⚠️
autosubmit/job/job_packages.py 66.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master    #2042   +/-   ##
=========================================
  Coverage          ?   49.80%           
=========================================
  Files             ?       72           
  Lines             ?    17569           
  Branches          ?     3418           
=========================================
  Hits              ?     8751           
  Misses            ?     8007           
  Partials          ?      811           
Flag Coverage Δ
fast-tests 49.80% <93.03%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

LuiggiTenorioK and others added 3 commits December 27, 2024 13:20
…t a fix with the configparser

* Changelog

* Fixes an issue with custom_directives being added letter by letter

* undo a commmit that shouldn't be there

* Added test

* Fix other tests due the new return

* The test_str_list wasn't really a str_list fixed
@kinow
Copy link
Member

kinow commented Jan 9, 2025

This branch has been deployed today to the ClimateDT VM as 4.1.12-dev-2b553f0, from commit 2b553f0. That commit is also pinning the config parser to the main branch, commit BSC-ES/autosubmit-config-parser@169b1eb.

kinow and others added 3 commits January 20, 2025 08:11
… to emails (#1997)

* Add all expid_dir_path functions to autosubmit.py and fix tests !509

All previous definitions all expid_path, tmp_path, log_path and
aslogs_path (and similar) have been removed and their calls have been
switched for BasicConfig.foo() calls. New tests added

* Add compression of log files and one test

* Default to last log file and two more test cases

* Add more tests

* Add suggestions

* Small corrections

* Add last suggestions

* Linting

---------

Co-authored-by: Irene Simo Munoz <irene.simomunoz@bsc.es>
@dbeltrankyl
Copy link
Contributor Author

Pending changelog to be done before the final release

dbeltrankyl and others added 4 commits January 23, 2025 15:51
* overwallclock fix

* Fix current tests

* Fix wrapper wallclock

* Fix wrapper wallclock

* changed time.time() x datetime.now()

* Applied Bruno patch

* Moved comment

* feedback

* fix change feedback

* fix change feedback

* fix change feedback

* test

* Checking coverage

* fix older tests to take into account the wallclock

* more test

* another test

* another test

* More tests

* More tests

* More tests

* More tests

* More tests

* More tests

* More tests

* patch from bruno and added an additional assert

* feedback

* Changes from Bruno

* feedback

* feedback

* pep 8

* removed unused job_status

* unused import removed

* Wallclock added to adjusted parameters

* Wallclock added to adjusted parameters
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.

4 participants