Skip to content

refactor: remove legacy nowcasting_datamodel dependencies and support#216

Open
msaqibatifj wants to merge 1 commit into
openclimatefix:mainfrom
msaqibatifj:nowcasting_datamodel
Open

refactor: remove legacy nowcasting_datamodel dependencies and support#216
msaqibatifj wants to merge 1 commit into
openclimatefix:mainfrom
msaqibatifj:nowcasting_datamodel

Conversation

@msaqibatifj
Copy link
Copy Markdown

Pull Request

Description

Please delete the italicised instruction text!
Please include a summary of the change and which issue is fixed.
Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration

  • Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

  • Yes

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@msaqibatifj
Copy link
Copy Markdown
Author

Hi, I have made the required changes. if you need anything else do tell.

@katyalmohit
Copy link
Copy Markdown
Contributor

katyalmohit commented May 29, 2026

@msaqibatifj there are some lint issues. You can resolve them first. You can check it with uv run ruff check.
Also, could you please update the PR description and mention the issue?

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