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

[WIP] Testing and debugging #6

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

Olshansk
Copy link
Member

@Olshansk Olshansk commented Dec 28, 2024

There are a handful of things to resolve before this is merged in.

  1. Several TODO_IN_THIS_... that we have to identify and see if it's an actual issue
  2. Option (1) in the installation process doesn't work due to the following error (i.e. missing .dylib file)
    Traceback (most recent call last):
      File "/Users/olshansky/workspace/pocket/poktroll-clients-py/example.py", line 3, in <module>
        from poktroll_clients.proto.poktroll.application.tx_pb2 import *
      File "/Users/olshansky/workspace/pocket/poktroll-clients-py/venv/lib/python3.11/site-packages/poktroll_clients/__init__.py", line 4, in <module>
        from .go_memory import ffi, libpoktroll_clients, go_ref, check_err, check_ref, GoManagedMem
      File "/Users/olshansky/workspace/pocket/poktroll-clients-py/venv/lib/python3.11/site-packages/poktroll_clients/go_memory.py", line 5, in <module>
        from poktroll_clients.ffi import ffi, libpoktroll_clients
      File "/Users/olshansky/workspace/pocket/poktroll-clients-py/venv/lib/python3.11/site-packages/poktroll_clients/ffi.py", line 261, in <module>
        lib_load_path = get_packaged_library_path()
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^
      File "/Users/olshansky/workspace/pocket/poktroll-clients-py/venv/lib/python3.11/site-packages/poktroll_clients/ffi.py", line 236, in get_packaged_library_path
        raise OSError(
    OSError: Native library not found for Darwin arm64
    Expected path: /Users/olshansky/workspace/pocket/poktroll-clients-py/venv/lib/python3.11/site-packages/poktroll_clients/lib/libpoktroll_clients-arm64.dylib
    Expected filename: libpoktroll_clients-arm64.dylib
  3. Use black to format all .py files in the repo.
  4. When I try to build the .whl myself the source files aren't there:
    $ pip show -f poktroll-clients
    Name: poktroll-clients
    Version: 0.2.0a0.dev1
    Summary: Python bindings for the Pocket Network protocol client library (libpoktroll-clients)
    Home-page:
    Author:
    Author-email: Bryan White <[email protected]>
    License:
    Location: /Users/olshansky/.local/share/virtualenvs/poktroll-clients-py--Q_wU4NY/lib/python3.11/site-packages
    Requires: atomics, cffi, protobuf
    Required-by:
    Files:
      poktroll_clients-0.2.0a0.dev1.dist-info/INSTALLER
      poktroll_clients-0.2.0a0.dev1.dist-info/METADATA
      poktroll_clients-0.2.0a0.dev1.dist-info/RECORD
      poktroll_clients-0.2.0a0.dev1.dist-info/REQUESTED
      poktroll_clients-0.2.0a0.dev1.dist-info/WHEEL
      poktroll_clients-0.2.0a0.dev1.dist-info/direct_url.json
      poktroll_clients-0.2.0a0.dev1.dist-info/licenses/LICENSE

@Olshansk Olshansk force-pushed the testing_and_debugging branch from f209777 to f786186 Compare December 29, 2024 01:36
@Olshansk Olshansk force-pushed the testing_and_debugging branch from f786186 to 66f19ce Compare December 29, 2024 01:53
@Olshansk Olshansk force-pushed the testing_and_debugging branch from ca5d45c to 79596cc Compare December 29, 2024 02:17
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.

1 participant