Code indexing in gitaly is broken and leads to code not being visible to the user. We work on the issue with highest priority.

Skip to content
Snippets Groups Projects

more general sddsVariableAt and statVariableAt

Merged frey_m requested to merge 318-optimizer-generalizing-objectives-with-sddsvariableat into master

Finally managed to find proper way of creating this merge request. :smile:

I haven't yet updated the documentation. -- Done.

Closes #318 (closed)

Edited by frey_m

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • frey_m added 1 commit

    added 1 commit

    Compare with previous version

  • frey_m added 1 commit

    added 1 commit

    Compare with previous version

  • snuverink_j resolved all discussions

    resolved all discussions

  • snuverink_j
  • frey_m resolved all discussions

    resolved all discussions

  • snuverink_j approved this merge request

    approved this merge request

  • snuverink_j resolved all discussions

    resolved all discussions

  • frey_m changed the description

    changed the description

  • Developer

    What I don't like about these changes is that they're breaking the existing input files of the users (the next version of OPAL won't be a new major version). We could change the code such that the users have to provide either 3 or 4 arguments. If there are only 3 arguments then s is the fourth.

  • I agree strongly with @kraus

  • Author Maintainer

    In this case s would need to be the default value.

  • Yes, we can not break existing code in a minor revision

  • frey_m added 1 commit

    added 1 commit

    • e63e8082 - reorder arguments for backward compatibility

    Compare with previous version

  • Author Maintainer

    @kraus, @adelmann I reordered the arguments for backward compatibility.

  • Author Maintainer

    The manual has also been updated in this regard.

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading