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

Add date to menu #933

Closed
wants to merge 36 commits into from
Closed

Add date to menu #933

wants to merge 36 commits into from

Conversation

mo-sanamahmood
Copy link

@mo-sanamahmood mo-sanamahmood commented Nov 12, 2024

I've added the CYLC_TASK_CYCLE_POINT to the menu description on the webpage so you can see the case date. This involved minor modification of lines 35-41 of /src/CSET/_workflow_utils/finish_website.py

Contribution checklist

Fixes #932

Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

  • Documentation has been updated to reflect change.
  • New code has tests, and affected old tests have been updated.
  • All tests and CI checks pass.
  • Ensured the pull request title is descriptive.
  • Conda lock files have been updated if dependencies have changed.
  • Attributed any Generative AI, such as GitHub Copilot, used in this PR.
  • Marked the PR as ready to review.

jfrost-mo and others added 30 commits November 8, 2024 10:29
This restores compatibility with the iBoss TLS interception certificate
that the Met Office is using. There is an upstream ticket open to get
this certificate well formed, after which we can remove this line.

Fixes #903
…itude point and the corrected longitude co-ordinate.
…ithin the domain after the longitude convention has been applied. Also, boundary_margin has been updated to 1 in all relevant tests as this should be the minimum value. This change only affects tests and is not used as a CSET input.
… the longitude rotation of the grid from the west.
… the longitude rotation of the grid from the east.
It can't be measured outside of an editable install, and doesn't
contain any logic.
This avoids potential issues with licencing on the defaults channels.
This makes reviewing the changes in conda lock files easier, as
packages will no longer move around.

Fixes #864
Explicitly use conda-forge conda channel for updating conda and resolving lock files
Exclude __main__.py from test coverage measurement
Remove HTTP status check as urlopen already does it
Test templating variables into recipe parser
Test recipe listing and detailing functions
jfrost-mo and others added 5 commits November 11, 2024 15:57
…comments added; np.mean function replaced with np.median.
Changes to fix longitude convention to be in the range -180 degrees to 180 degrees where possible
Copy link
Contributor

github-actions bot commented Nov 12, 2024

Coverage

…cking

Disable strict checking that TLS certificates are well formed
@jfrost-mo
Copy link
Member

Ah, this pull request has been made the wrong way around, and is trying to merge the main branch into yours, rather than your branch into main.

Looking at the branch it doesn't look like there are any changes. Did you make sure to commit before pushing? main...932_add_date_to_menu

@jfrost-mo
Copy link
Member

Closing as the wrong way around.

@jfrost-mo jfrost-mo closed this Nov 12, 2024
@jfrost-mo jfrost-mo removed their request for review November 12, 2024 16:46
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