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

Difficult to Determine Direction of Access Feature Instances #758

Closed
philip-alldredge opened this issue Dec 9, 2016 · 3 comments · Fixed by #2098
Closed

Difficult to Determine Direction of Access Feature Instances #758

philip-alldredge opened this issue Dec 9, 2016 · 3 comments · Fixed by #2098

Comments

@philip-alldredge
Copy link
Collaborator

The direction of all feature instances for Access features is In Out in the instance model. I believe it should be set to In or Out based on whether it is a provides or requires feature or another field should be provided. It is difficult to reliably determine the kind of the feature instance using the underlying feature because there are cases such as inverse feature groups which would cause the direction to be opposite of the underlying feature.

@sprocter
Copy link
Contributor

Hi Philip,

I discussed this issue with @lwrage this morning, and we were wondering if you could provide a little more context as to your planned usage of the direction of the access feature instance. That is, is your primary interest determining the shape of a symbol in a graphical view of the instance model? Or in some new analysis? or something else entirely?

I don't think provides / requires can be cleanly mapped to in / out, because -- as I understand it -- a component that provides access could be providing access to read and / or write data. Similarly, a component requiring access could be requiring read or write access.

@philip-alldredge
Copy link
Collaborator Author

The primary interest is for determining the shape of a symbol in the graphical view of the instance model. However, this issue could effect analysis. Currently there isn't an easy way to determine if a feature instance is a provides or a requires.

I have to agree that depending on how you think about the in/out concept vs provides/requires that perhaps it isn't an appropriate mapping. I typically think of a requires as an input because access to the data/bus is being given(input) to the component.

However, my main desire is a way of easily determining whether a feature instance is a provides/requires in the same way that we can determine if port feature instances are inputs or outputs.

@lwrage lwrage added the backlog label Jan 16, 2018
@lwrage lwrage added this to the 2.3.3 milestone Mar 3, 2018
@lwrage lwrage added next and removed backlog labels Mar 5, 2018
@lwrage lwrage removed this from the 2.3.3 milestone Apr 10, 2018
@lwrage lwrage added backlog and removed next labels Jul 2, 2018
@lwrage lwrage removed the backlog label Jul 9, 2019
@lwrage
Copy link
Contributor

lwrage commented Dec 5, 2019

Change as follows:

  • provides access feature instances will have direction out
  • requires access feature instances will have direction in

Code currently calls getDirection() to determine allowed data flow, which is always in out for access features => Create new method getFlowDirection() that returns in out for access features and calls getDirection() for all other features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants