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

[POC][ABSTRACTION]: Initial #1260

Closed
wants to merge 8 commits into from
Closed

[POC][ABSTRACTION]: Initial #1260

wants to merge 8 commits into from

Conversation

amadolid
Copy link
Collaborator

@amadolid amadolid commented Sep 9, 2024

CHANGES

Abstraction (NOT YET INTEGRATED)

  • jaclang/runtimelib/interface.py
    • Base Classes that will be used in JacFeatureSpec and JacFeature
    • Will be used by plugin as base class
  • jaclang/runtimelib/implementation.py
    • Default implementation based on interface classes
    • Will be used on default JacFeatureImpl
  • __serialize__ / __deserialize__
    • Common models (Anchors, Architypes, Access) now have overridable way of serialization/deserialization

Anchor/Architype logical process migration

  • All common logical process will now be on plugin, segregated by class (JacAccessValidation, JacNode, JacEdge, JacWalker) extended by JacFeature
  • Anchors / Architypes only holds attributes and __serialize__ / __deserialize__ method by default.

@amadolid amadolid force-pushed the major/abstractions branch 8 times, most recently from 382f6cd to 8740430 Compare September 16, 2024 16:47
@amadolid amadolid changed the title [INPROGRESS][POC][ABSTRACTION]: Initial [POC][ABSTRACTION]: Initial Sep 16, 2024
@amadolid amadolid marked this pull request as ready for review September 17, 2024 17:25
@amadolid amadolid force-pushed the major/abstractions branch 3 times, most recently from 743effb to 3cf26f3 Compare September 24, 2024 13:38
@amadolid
Copy link
Collaborator Author

moved to #1317

@amadolid amadolid closed this Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant