Menu

FGData Merge Request #293: Add option to read upper winds from simbrief (merged)

Merging...

Merged

Something went wrong. Please, merge manually

Checking if merge is possible...

Something went wrong. Please, merge manually

Lars Toenning wants to merge 1 commit from /u/ltoenning/fgdata/ to next, 2022-08-16

Commit Date  
[741d15] (feature/simbrief_upper_winds) by Lars Toenning Lars Toenning

Add option to read upper winds from simbrief

2022-05-22 12:54:23 Tree

Discussion

  • James Turner

    James Turner - 2022-08-08

    Have you check with Thorsten R about this change? I don't know to what degree he still has ownership of the local weather code, or worries about this kind of extension. (For me it seems okay)

     
  • legoboyvdlp

    legoboyvdlp - 2022-08-11

    Responding to Thorsten's query - there's no conflict at all with my weather addon (well, unless they are used together - I've never tried that). In fact - this one is probably better, because it skips the step of having to parse wgrib files locally using a C# utility.

     
  • James Turner

    James Turner - 2022-08-16
    • Status: open --> merged
     
  • Thorsten Renk

    Thorsten Renk - 2022-08-16

    For the record, I'm entirely fine with this, James just beat me by 10 minutes merging :-)

     
  • James Turner

    James Turner - 2022-08-16

    Apologies Thorsten, I took Johnathan's reply above to mean he had spoken to you. Glad it worked out.

     

Log in to post a comment.

MongoDB Logo MongoDB