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

Avoid null in nearnormalHemisphericalVisibleTransmittances #319

Closed
christoph-maurer opened this issue Feb 29, 2024 · 2 comments
Closed

Avoid null in nearnormalHemisphericalVisibleTransmittances #319

christoph-maurer opened this issue Feb 29, 2024 · 2 comments
Assignees
Labels
ISE Fraunhofer Institute for Solar Energy Systems ISE LBNL Lawrence Berkeley National Laboratory

Comments

@christoph-maurer
Copy link
Member

When I query https://igsdb-v2.herokuapp.com/graphql/ for

{
  allOpticalData(
    where: {and: [{nearnormalHemisphericalVisibleTransmittances: {some: {greaterThanOrEqualTo: 0.9}}}, {nearnormalHemisphericalVisibleReflectances: {some: {lessThanOrEqualTo: 0.01}}}, {resources: {some: {dataFormatId: {equalTo: "9ca9e8f5-94bf-4fdd-81e3-31a58d7ca708"}}}}]}
  ) {
    edges {
      node {
        componentId
        nearnormalHemisphericalSolarReflectances
        nearnormalHemisphericalSolarTransmittances
        nearnormalHemisphericalVisibleReflectances
        nearnormalHemisphericalVisibleTransmittances
        infraredEmittances
        resourceTree {
          root {
            value {
              locator
            }
          }
        }
      }
    }
  }
}

I receive

            "nearnormalHemisphericalSolarTransmittances": [
              0.8163923,
              0.8163923
            ],
            "nearnormalHemisphericalVisibleReflectances": [
              0.004783743,
              0.004784637
            ],
            "nearnormalHemisphericalVisibleTransmittances": [
              0.9774387,
              null
            ],

It would be better to see the nearnormalHemisphericalVisibleTransmittances twice as in nearnormalHemisphericalSolarTransmittances.

@christoph-maurer christoph-maurer self-assigned this Feb 29, 2024
@github-project-automation github-project-automation bot moved this to 🆕 New in EQWIN-P Feb 29, 2024
@github-actions github-actions bot added the triage Review issue to make sure it is relevant, actionable, and has all the necessary information label Feb 29, 2024
@christoph-maurer christoph-maurer added ISE Fraunhofer Institute for Solar Energy Systems ISE LBNL Lawrence Berkeley National Laboratory and removed triage Review issue to make sure it is relevant, actionable, and has all the necessary information labels Feb 29, 2024
@danielmcquillen
Copy link
Collaborator

danielmcquillen commented Mar 1, 2024

@christoph-maurer This issue was due to an issue in our importer. It's since been fixed, if you try again you should not see null values in nearnormalHemisphericalVisibleTransmittances

@RDmitchell FYI

@christoph-maurer
Copy link
Member Author

@danielmcquillen It works well, thank you!

@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in EQWIN-P Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ISE Fraunhofer Institute for Solar Energy Systems ISE LBNL Lawrence Berkeley National Laboratory
Projects
Status: Done
Development

No branches or pull requests

2 participants