Skip to content

Comments

[DRAFT] Update MultipleSources notebook to include projection to lonlat#96

Closed
nikeethr wants to merge 3 commits intoACCESS-Community-Hub:developfrom
nikeethr:tutorial/radar-lonlatproj-multiple-sources
Closed

[DRAFT] Update MultipleSources notebook to include projection to lonlat#96
nikeethr wants to merge 3 commits intoACCESS-Community-Hub:developfrom
nikeethr:tutorial/radar-lonlatproj-multiple-sources

Conversation

@nikeethr
Copy link
Collaborator

@nikeethr nikeethr commented Apr 11, 2025

resolves #95

may need @tennlee to run through this with me and edit some parts. Converting to a draft until we've gone through it.

TODO:

  • fix graphviz issues (probably on my end)
  • merge radar data source with other sources using pipeline (in contrast to branched/split data-sources)
  • clean up documentation (currently quite wordy)
  • finish "advanced" configuration section OR move it to "Future Work"

@nikeethr nikeethr requested a review from tennlee April 11, 2025 07:56
@nikeethr nikeethr marked this pull request as draft April 11, 2025 07:59
@nikeethr
Copy link
Collaborator Author

nikeethr commented May 5, 2025

Should rebase on top of #106 once that's merged in.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 14896363186

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 59.975%

Totals Coverage Status
Change from base Build 14896137038: 0.0%
Covered Lines: 9660
Relevant Lines: 15646

💛 - Coveralls

@nikeethr nikeethr changed the title Update MultipleSources notebook to include projection to lonlat [DRAFT] Update MultipleSources notebook to include projection to lonlat Jun 10, 2025
@nikeethr
Copy link
Collaborator Author

NOTE: this notebook should now be updated to use #152 and/or if this PR is no longer relevant we can discard it.

@tennlee
Copy link
Collaborator

tennlee commented Aug 1, 2025

We have gone through projection again and have now implemented an alternative approach which was just merged. This PR is now outdated, so closing it.

@tennlee tennlee closed this Aug 1, 2025
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.

Update MultipleSources tutorial with lonlat projection for Rainfields3 (oldname: RadarDemo)

4 participants