hydra-python-core icon indicating copy to clipboard operation
hydra-python-core copied to clipboard

Datatype argument for Doc writer.

Open chrizandr opened this issue 3 years ago • 3 comments

I'm submitting a

  • [ ] bug report.
  • [x] feature request.

Current Behaviour:

We are using the keyword arguments to add range to supported property, which is fine. But users may not always use xsd:float/integer, while defiining properties.

Expected Behaviour:

It might be useful to have a separate datatype argument in docwriter while creating a new supportedProperty, which we map directly to xsd:float/integere instead of using the keyword arguments to specify the datatype. We should give them an option define the datatype using python datatypes which we add to range as xsd:float/integer and let that behaviour be overridden with kwargs instead of relying on kwargs only.

I'd like to hear thoughts on this, it should be a small addition.

Steps to reproduce:

Do you want to work on this issue?

chrizandr avatar Jul 06 '21 14:07 chrizandr

@Mec-iS @chrizandr I'm a little confused about this as I couldn't get a clear picture of what exactly we want here.

It might be useful to have a separate datatype argument in docwriter while creating a new supportedProperty

Do you mean that we can have another datatype argument in HydraClassProp ? like :

HydraClassProp(prop_uri, prop_title, required=True, read=True, write=True, datatype="float")

and then use it to set range to xsd:float ?

farazkhanfk7 avatar Jul 08 '21 07:07 farazkhanfk7

I didnt understand the issue completely but I think we can leave this for later and go on with the content of the proposal for now.

Mec-iS avatar Jul 08 '21 08:07 Mec-iS

Yes, that is what I meant @farazkhanfk7 . Sorry if the wording has not made it clear.

chrizandr avatar Jul 09 '21 07:07 chrizandr