-
Notifications
You must be signed in to change notification settings - Fork 43
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
Wrong kind of problem #435
Comments
Hi @Shi-Raida. We decided to remove the Ineed, those problems you mentioned don't have the A small example: you want to write the duration of an action So, if a planner can handle this particular situation, but not numeric fluents in general, only the |
I think this is where we have a problem in this case: the problem kind should apply to the problem as it is currently, not as it may appear after an hypothetical transformation. In particular, one should not assume that the problem will be grounded. |
I get that. The idea behind the decision is that The grouding was just to make an example |
The motivation under this choice is more that with When we have only We followed in some way the semantic of the PDDL requirement @arbimo Do you see any cases where an engine cannot correctly specify its specific supported kind? |
Comparison with PDDL is a bit misleading I believe since there is no notion of integer types there. So any cost, duration or numeric effect implies continuous numbers. For instance, in a development branch of Aries, we currently support integer expressions (with static or non-static fluents) in durations, effects, and action costs. We do not support real expressions anywhere. Currently, the only option that we have to have the problem kind faithfully capture this absence of support for continuous number is to remove the features:
Then the name is very misleading. There are continuous numbers in this problem. (this is also what we stated in the report and documentation for problem kind)
I get what you mean and this is I believe accurately captured by the I don't think we would loose anything by giving |
Ok I get your point! |
Hi @arbimo! We discussed this issue with @mikand and we propose to replace
What do you think? We will try to keep the backward compatibility using the |
I think that sounds great! Just a quick question: would you coalesce INT to REAL if both are present? Both answers are correct I believe as from my understanding, the By only deprecating the previous features I think it should be possible to maintain backward compatibility. But I am struggling to see an elegant way where the user does not have to opt-in 😨 |
I would set both the |
The following problems are using fluent with
RealType
but the problem does not have the kindCONTINUOUS_NUMBERS
. Is it not a bug?The text was updated successfully, but these errors were encountered: