Coder Social home page Coder Social logo

Excluding files from merger about buildingspy HOT 5 OPEN

lbl-srg avatar lbl-srg commented on July 28, 2024
Excluding files from merger

from buildingspy.

Comments (5)

mwetter avatar mwetter commented on July 28, 2024

@Mathadon , @rubenbaetens
We could add a function set_excluded_files to flag files that should not be merged, and then it would be up to the IDEAS developers to set the files you don't want. I however think this is not a clean solution because then what is now the Annex 60 core may diverge from one library to another.
Would it work in your case to extend from the weather bus and add the functionality that IDEAS needs?
I also don't see why IDEAS.Fluid.Sources.Outside_CpLowRise requires a different connector. Do you add functionality that needs additional information?

from buildingspy.

rubenbaetens avatar rubenbaetens commented on July 28, 2024

@mwetter @Mathadon There is indeed no need for another connector for IDEAS.Fluid.Sources.Outside_CpLowRise as we do not add any additional functionality. Though we created other models that required additional information which we originally included in our weaBus, thus making the bus no longer compatible with A60 bus.

Is extending from the A60 weabus and adding parameters a solution here ? As you remain attempting to connect two busses with different parameters set (though with a common base).

from buildingspy.

Mathadon avatar Mathadon commented on July 28, 2024

Thank you for the suggestion. I already tried using the extends clause but this does not work (dymola error message: incompatible connectors).

Our weather bus connector contains several additional Real and Bus variables that do not exist in the A60 version. Still, it would be convenient to use only one bus within IDEAS. I prefer having incompatible connectors between IDEAS-AixLib rather than having incompatible connectors within IDEAS. The solution I proposed seemed to be the cleanest, although I'm open for other suggestions!

from buildingspy.

mwetter avatar mwetter commented on July 28, 2024

What are these additional variables? Are they of general interest to the other libraries as well?

from buildingspy.

Mathadon avatar Mathadon commented on July 28, 2024

No, they are mostly variables such as light incidence angles and intensity for windows.

from buildingspy.

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.