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

HTN issues #101

Closed
pmdoll opened this issue May 1, 2017 · 4 comments
Closed

HTN issues #101

pmdoll opened this issue May 1, 2017 · 4 comments
Assignees
Milestone

Comments

@pmdoll
Copy link
Collaborator

pmdoll commented May 1, 2017

For first.pamela, When using htn task,

  • We should be able to create HTN from pamela class first-tpn instead of first-tpn-htn
  • in between constraints are not created
  • Bounds are not created for plant activities
  • Appearance of labels is unexpected and misleading.

TPN created using cfm method

first cfm

TPN created using htn task

first htn

See pamela file for individual commands for compiling first.pamela file

@pmdoll pmdoll added this to the May 2017 milestone May 1, 2017
@dcerys
Copy link
Member

dcerys commented May 2, 2017

Addressing the 4 points above:

  • A documented current limitation of the HTN generation is that pmethods cannot have a combination of more than one parallel/sequence/choice. If a 2nd-level parallel/sequence/choice, it's trivial to create a sub-pmethod with the same effect.
  • It is true that between constraints are not created. Let's create that as a separate issue, so it can be addressed.
  • The bounds of primitive method declarations is addressed by Issue HTN/TPN generation doesn't include bounds (or other metadata) declared for primitive methods #98
  • The appearance of the labels is certainly different. The full display of labels is dependent on Issue Harmonize TPN and HTN schemas #43

Using the HTN-generated version, with the bounds specified at the calling site, we can see (missing the between constraint):

screenshot 2017-05-01 22 06 22

@pmdoll
Copy link
Collaborator Author

pmdoll commented May 2, 2017

Capitalized labels with dashes removed should not be related to pamela. It is a planviz bug or warrants an explanation.

@tmarble
Copy link
Collaborator

tmarble commented May 2, 2017

Addressing each of these points, in turn:

@dcerys
Copy link
Member

dcerys commented Jun 22, 2017

All of the items in this compound issue have been included in separate, individual issues

@dcerys dcerys closed this as completed Jun 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants