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

deprecated test case (NeuralFactory) + important bugfix! (v3) #298

Merged
merged 5 commits into from
Jan 28, 2020

Conversation

tkornuta-nvidia
Copy link
Contributor

  • bugfix: decodator now also returns the result of the function (!)
  • several NeuralFactory methods indicated as deprecated
  • added init to decorators

Signed-off-by: nvidia <tkornuta@nvidia.com>
Signed-off-by: nvidia <tkornuta@nvidia.com>
Signed-off-by: nvidia <tkornuta@nvidia.com>
Signed-off-by: nvidia <tkornuta@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Jan 27, 2020

This pull request fixes 3 alerts when merging c0f53b8 into d9fe6eb - view on LGTM.com

fixed alerts:

  • 2 for Wrong name for an argument in a call
  • 1 for Non-callable called

Signed-off-by: nvidia <tkornuta@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Jan 27, 2020

This pull request fixes 3 alerts when merging 068d9d9 into d9fe6eb - view on LGTM.com

fixed alerts:

  • 2 for Wrong name for an argument in a call
  • 1 for Non-callable called

@okuchaiev okuchaiev merged commit 69ae6fc into master Jan 28, 2020
@tkornuta-nvidia tkornuta-nvidia deleted the deprecated_neural_factory_v3 branch January 28, 2020 18:33
dcurran90 pushed a commit to dcurran90/NeMo that referenced this pull request Oct 15, 2024
* README: Lab requires python3 devel headers/library

The python3 development headers and library are required, or else
llama-cpp-python fails to build.

    × Building wheel for llama-cpp-python (pyproject.toml) did not run successfully.
    │ exit code: 1
    ╰─> [23 lines of output]
        *** scikit-build-core 0.8.2 using CMake 3.28.3 (wheel)
        *** Configuring CMake...
        2024-03-05 11:42:14,297 - scikit_build_core - WARNING - libdir/ldlibrary: /usr/lib64/libpython3.12.so is not a real file!
        2024-03-05 11:42:14,297 - scikit_build_core - WARNING - Can't find a Python library, got libdir=/usr/lib64, ldlibrary=libpython3.12.so, multiarch=x86_64-linux-gnu, masd=None

Signed-off-by: Stef Walter <stefw@redhat.com>

* README: Add gh tool install command on Fedora Linux

This is required to download the model.

Signed-off-by: Stef Walter <stefw@redhat.com>

* README: Command to install the g++ compiler on Fedora Linux

Add the command to install the g++ compiler on Fedora Linux. Otherwise
we get an error similar to the following when building llama-cpp-python

    × Building wheel for llama-cpp-python (pyproject.toml) did not run successfully.
    │ exit code: 1
    ╰─> [21 lines of output]
        *** scikit-build-core 0.8.2 using CMake 3.28.3 (wheel)
        *** Configuring CMake...
        loading initial cache file /tmp/tmppdxr1vom/build/CMakeInit.txt
        -- The C compiler identification is GNU 14.0.1
        -- The CXX compiler identification is unknown
        -- Detecting C compiler ABI info
        -- Detecting C compiler ABI info - done
        -- Check for working C compiler: /usr/bin/cc - skipped
        -- Detecting C compile features
        -- Detecting C compile features - done
        CMake Error at CMakeLists.txt:3 (project):
          No CMAKE_CXX_COMPILER could be found.

Signed-off-by: Stef Walter <stefw@redhat.com>

---------

Signed-off-by: Stef Walter <stefw@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants