Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dataset export entry point #4857

Merged
merged 10 commits into from
Dec 20, 2022

Conversation

jenshnielsen
Copy link
Collaborator

@jenshnielsen jenshnielsen commented Dec 6, 2022

This is useful for kicking of postprocessing, backup, upload etc.

  • Document
  • Test using a entry point that logs the export event

@codecov
Copy link

codecov bot commented Dec 6, 2022

Codecov Report

Merging #4857 (5526303) into master (c220bf7) will increase coverage by 0.01%.
The diff coverage is 88.00%.

@@            Coverage Diff             @@
##           master    #4857      +/-   ##
==========================================
+ Coverage   68.30%   68.31%   +0.01%     
==========================================
  Files         339      340       +1     
  Lines       32128    32150      +22     
==========================================
+ Hits        21944    21964      +20     
- Misses      10184    10186       +2     

@jenshnielsen jenshnielsen marked this pull request as ready for review December 13, 2022 14:17
Copy link
Contributor

@Dominik-Vogel Dominik-Vogel left a comment

Choose a reason for hiding this comment

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

This is beautiful!
I have tested it and it works ✔️

@jenshnielsen jenshnielsen merged commit 3f78c23 into microsoft:master Dec 20, 2022
@jenshnielsen jenshnielsen deleted the add_export_entry_point branch December 20, 2022 12:52
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.

None yet

2 participants