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

[FEATURE] Restructure Transcribe.jl #47

Closed
pulsipher opened this issue Apr 8, 2020 · 0 comments · Fixed by #58
Closed

[FEATURE] Restructure Transcribe.jl #47

pulsipher opened this issue Apr 8, 2020 · 0 comments · Fixed by #58
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@pulsipher
Copy link
Collaborator

Describe the feature you'd like
transcribe.jl needs to be rewritten to address the following concerns:

  • Needs to adopt a more straightforward paradigm like measure expansion
  • Should make a support/variable look up table for all variables to quicken variable searches
  • Should build expressions in-place using the look-up table
  • Reduced variables need to be mapped to transcription variables to allow partial transcription of vector parameters
  • Reduced variables and measures should reference transcription equivalents in order to query their values
  • Perhaps measure expansions should be located in connected equality constraints in order to query value
  • The parameter references present in each expression should be found at creation not in TranscriptionOpt
  • Parameter references should be ordered at creation
  • Make a way for PointVariableRefs generated by measure expansion to have names.

All of the above should improve performance and stability by an appreciable margin and make it much more straightforward.

Does this pertain to a particular class of problems/techniques? If so explain.
Everything

Describe what you currently do about this
We use a hacky methodology with a lot of unnecessary searches and data type conversions.

@pulsipher pulsipher added the enhancement New feature or request label Apr 8, 2020
@pulsipher pulsipher added this to the v0.2.0 milestone Apr 8, 2020
@pulsipher pulsipher self-assigned this Apr 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant