search

OSUKED / ElexonDataPortal

Python wrapper for the Elexon/BMRS API
https://osuked.github.io/ElexonDataPortal
MIT License
52 stars 14 forks source link

Derived system data #13

Closed peterdudfield closed 2 years ago

peterdudfield commented 2 years ago
  1. Added Derived system data - this is much quicker than B1770, but also has some other nice things in it
  2. Add pytests for
    • B1770
    • DERSYSDATA
    • Of course there can be a lot more added, but I thought I wouldat leastt start with these, and then others can be slowly added.
  3. add pytest workflow -haven'tt tested this properly yet, but hopefully with the PR is might run?
peterdudfield commented 2 years ago

I now see that maybe the way to do it, is to add something into the endpoint.csv?

AyrtonB commented 2 years ago

Thanks Peter, those are really helpful. I've been meaning to add some tests for a while!

Yes, the ideal way to add a new endpoint is through the endpoint.csv. If can add them there that would be great, if not let me know and I'd be happy to do them myself.

peterdudfield commented 2 years ago

Thanks Peter, those are really helpful. I've been meaning to add some tests for a while!

Yes, the ideal way to add a new endpoint is through the endpoint.csv. If can add them there that would be great, if not let me know and I'd be happy to do them myself.

Thanks - I added it to endoint.csv, and it seemed to work, apart from this bug - https://github.com/OSUKED/ElexonDataPortal/issues/15