Skip to content

Conversation

@peterdudfield
Copy link
Contributor

@peterdudfield peterdudfield commented Mar 28, 2022

Pull Request

Description

add a read function to read 'InputDataLastUpdatedSQL'
Fixes #20

How Has This Been Tested?

  • normal tests

  • added 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

@peterdudfield peterdudfield self-assigned this Mar 28, 2022
Read last input data last udpated
:param session: database session
:param session: database session
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update

@codecov-commenter
Copy link

Codecov Report

Merging #22 (e181379) into main (c49c9b9) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #22      +/-   ##
==========================================
+ Coverage   97.34%   97.37%   +0.02%     
==========================================
  Files          16       16              
  Lines         528      534       +6     
==========================================
+ Hits          514      520       +6     
  Misses         14       14              
Impacted Files Coverage Δ
nowcasting_datamodel/read/read.py 98.88% <100.00%> (+0.07%) ⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@peterdudfield peterdudfield merged commit e4db78f into main Mar 30, 2022
@peterdudfield peterdudfield deleted the feature/20-read-latest-input branch March 30, 2022 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Add read latest data table

3 participants