Coder Social home page Coder Social logo

Comments (7)

aothms avatar aothms commented on July 24, 2024

I agree. I brought up the same point originally.

from ids.

berlotti avatar berlotti commented on July 24, 2024

This indeed keeps on coming back.
How would we put that in the specification?
Bringing in @CBenghi as a potential feature for the IDS diagnostics tool.

from ids.

CBenghi avatar CBenghi commented on July 24, 2024

We probably have to have a conversation as to what we mean when specifying the measure.

I see two aspects:

  1. a constraint on the type of value, but also
  2. information for converting the unit for the quantitative measures to the agreed SI unit

IfcLabel and IfcText probably don't need the conversion part and constraint on type can be approximated by the basetype on the value as xs:string.

That said I'm not necessarily opposed to adding them to the measures, it's just food for thoughts.

from ids.

MatthiasWeise avatar MatthiasWeise commented on July 24, 2024

My understanding was that it should be a constraint (option 1).
Conversion does not make sense to me.

from ids.

berlotti avatar berlotti commented on July 24, 2024

Options:

  1. keep it as is (just require 'ifcText' for example)
  2. turn measure into an 'ids:Value' so you can use enumerations and patterns
    2b) turn measure into a 'xs:list' to allow multiple values for measure
  3. add STRING, NUMBER, REAL (and maybe some others to the list of allowed values for measure

from ids.

aothms avatar aothms commented on July 24, 2024

And maybe equally important, the documentation is somewhat inconsistent:

Attribute name:

  • Schema: measure
  • Doc table: Data Type

Possible measure values:

  • For your convenience, a list of common data types are listed here: ... IfcLabel, ...
  • A valid data type name, taken from the IDS units table (does not include label)

Required:

  • Table says: Data Type, required YES
  • Description says "If not specified ..."

My take on these things was that we would use the full list of IFC defined types (which includes some types you would not or cannot actually use) instead of just the measures because it's the simplest way to define the superset of measures + sensible non-numeric and unitless value types. Hence, I think it would make sense to rename the attribute measure -> dataType. I'm a bit uncertain on whether the attribute should be required?


The documentation should also be extended to show how to specify e.g boolean and binary values (is that following xsd grammar or express grammar).

from ids.

berlotti avatar berlotti commented on July 24, 2024
  • renamed to 'data type' instead of 'measure'.
  • made required (but can be empty)
  • clarified it can be any ifcDefinedType; with additional 'base' types like STRING, UNDEFINED, COUNT, etc. --> needs documentation

This is a significant impact on the implementations.....

from ids.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.