Issue/datamodel 1.0.7

[peterdudfield]

Pull Request

Description

use pvsite-datamodel 1.0.7
removed some function from get_data, had to adjust some tests

#89

How Has This Been Tested?

CI tests

• 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

[codecov]

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (a5026bb) 36.45% compared to head (5e03080) 30.48%.

Files	Patch %	Lines
src/sites_toolbox.py	50.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
- Coverage   36.45%   30.48%   -5.97%     
==========================================
  Files          15       15              
  Lines         864      797      -67     
==========================================
- Hits          315      243      -72     
- Misses        549      554       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rachel-labri-tipton]

Hi @peterdudfield, I'm having trouble running the code locally. I updated the pvsite-datamodel but when I try to run the code, I keep getting this error:

.

Did you happen to have this issue?

[peterdudfield]

I dont, might have to install https://pypi.org/project/h5py/, but seems weird. Is this a new bug for you?

[rachel-labri-tipton]

yeah, it's a new bug. otherwise, the code looks good. glad those functions are moved into the data model. i went ahead and approved.