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

Separate orbiter analyze Command, add input file to dag dict post-filter, trim log output #16

Merged
merged 7 commits into from
Sep 3, 2024

Conversation

fritz-astronomer
Copy link
Contributor

  • feat: add trim_dict to reduce log output of large objects
  • refactor: separate orbiter analyze into it's own command
  • fix: allow adding DAGs together (if parts were in multiple files)
  • feat: orbiter analyze - write to file & get unknown task type from doc_md
  • feat: add __file key to dag dict after filter. Make orbiter_kwargs[file_path] relative to input_dir
  • fix: continue (w/ error) for failure to load a file of type
  • fix: fix import path

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 42.04545% with 51 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@49f6139). Learn more about missing BASE report.

Files with missing lines Patch % Lines
orbiter/objects/dag.py 4.54% 21 Missing ⚠️
orbiter/__main__.py 40.62% 19 Missing ⚠️
orbiter/rules/rulesets.py 22.22% 7 Missing ⚠️
orbiter/objects/project.py 84.61% 2 Missing ⚠️
orbiter/rules/__init__.py 77.77% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #16   +/-   ##
=======================================
  Coverage        ?   79.14%           
=======================================
  Files           ?       35           
  Lines           ?     1362           
  Branches        ?        0           
=======================================
  Hits            ?     1078           
  Misses          ?      284           
  Partials        ?        0           

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

@fritz-astronomer fritz-astronomer merged commit 4472792 into main Sep 3, 2024
6 checks passed
@fritz-astronomer fritz-astronomer deleted the analysis branch September 3, 2024 21:08
@fritz-astronomer fritz-astronomer mentioned this pull request Sep 4, 2024
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.

2 participants