Code Monkey home page Code Monkey logo

Comments (7)

pvelesko avatar pvelesko commented on June 14, 2024 1

I do not know if we can make it more graceful, as long as the user sees it's because of missing capabilities and not just crash, I think there's not much else we can do.

Ok so sounds like we don't have an issue in that regard then since we already print the exact error.

Should we add a separate test suite for the double tests since they are not supposed to work on OpenCL/LZ devices which do not provide double support?

Sounds like good idea and something we can consider for the next release. I haven't come across a device that doesn't support double precision (so no way of testing) so it doesn't seem like a high-priority issue at this time.

from chipstar.

pvelesko avatar pvelesko commented on June 14, 2024

from chipstar.

pvelesko avatar pvelesko commented on June 14, 2024

The "Double type is not supported on this platform." is a HW/driver issue

I haven't seen this error before. Are you using an outdated driver? Trying to understand what could cause a failure to emulate.

OpenCL/SPIR-V doesn't require double support or its SW emulation

I thought we enable double support by using #pragma OPENCL EXTENSION cl_khr_fp64 : enable?

Can we fail more gracefully at runtime?

If we fail to compile we throw an error and print the compilation log:

    CHIPERR_CHECK_LOG_AND_THROW(Status, ZE_RESULT_SUCCESS, hipErrorTbd);
    logError("ZE Build Log: {}", std::string(LogStr).c_str());

besides changing the error from hipErrorTbd to something more specific, how do you propose we make JIT failure more graceful?

from chipstar.

pjaaskel avatar pjaaskel commented on June 14, 2024

This is an extension, meaning it's not mandatory to support it. I do not know if we can make it more graceful, as long as the user sees it's because of missing capabilities and not just crash, I think there's not much else we can do.

from chipstar.

pjaaskel avatar pjaaskel commented on June 14, 2024

All of these cases except the one reported in Issue #134 seem to be caused by the double precision issue. I'll change this ticket to fix it for 0.9 via the double precision test tag and another test suite for tests that are not supposed to work with non-double capable HW, and let's leave Issue #137 open for later consideration.

from chipstar.

pjaaskel avatar pjaaskel commented on June 14, 2024

Not sure if we really need this one as there's a workaround for the only known target where the issue appears.

from chipstar.

pvelesko avatar pvelesko commented on June 14, 2024

from chipstar.

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.