-
Notifications
You must be signed in to change notification settings - Fork 39
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 support to TILs in the PDDLWriter #543
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #543 +/- ##
==========================================
+ Coverage 84.96% 85.00% +0.03%
==========================================
Files 200 200
Lines 26433 26469 +36
==========================================
+ Hits 22458 22499 +41
+ Misses 3975 3970 -5 ☔ View full report in Codecov by Sentry. |
c279224
to
2c623a1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I just suggested using timed-initial-effect
has feature name to make it less ambiguous and avoid potential clashes.
Right now there is no place for it in the documentation but it should be mentioned in a future "advanced temporal modeling features".
No description provided.