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

Draft simplerene integration #189

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

guillep
Copy link

@guillep guillep commented Jul 10, 2024

No description provided.

@labordep labordep added the enhancement New feature or request label Jul 17, 2024
@labordep
Copy link
Member

@Nyan11 have you test it?

@Nyan11
Copy link
Collaborator

Nyan11 commented Jul 31, 2024

Yes,
The idea is to create a meta-model of Bloc using the simpleRene description language.
And then use this meta-model to automatically generate the Pyramid properties.

For example:
We can define the concept of "font-size" as a number between 1 and infinity.
We can defnine the concept of "BlTextElement" as an object link with a "font-size" by the font-size accessor.
Therefor we can use this information to generate the "font-size" property inside Pyramid for the BlTextElement.

The pros:

  • This project is interesting because anyone could add new properties to the meta-model and Pyramid could understand them.
  • Therefor a Pyramid developper do not need to know anything about Bloc or Toplo.
  • Anything defined by SimpleRene could be editable by Pyramid, not only Bloc or Toplo.
  • The definition of Bloc or Toplo could be reuse to create a serializer or to check the precondition/postconditions of the methods.

The cons:

  • SimpleRene is "complex", and implementing the different features in Pyramid will be time-consuming.
  • We would still need to write all the definitions of Bloc and Toplo using simpleRene, this is also time-consuming.

I dont think anyone except us would need the meta-model of Bloc.
I dont think we should invest the time to do this change.

But if we have the time, this proposition should be done.
It will enable Pyramid to edit anything that has been described with SimpleRene.

@labordep
Copy link
Member

labordep commented Aug 3, 2024

Ok, it's looks like an architectural feature for Pyramidion, isn't it?
I like principles.

@labordep labordep requested a review from Nyan11 August 28, 2024 13:56
@labordep
Copy link
Member

labordep commented Sep 3, 2024

Thanks @guillep, we will use it!

@Nyan11 Nyan11 mentioned this pull request Sep 3, 2024
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 this pull request may close these issues.

3 participants