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

test_inventories_compact() fails with Sqlite on arm Mac #205

Open
bennibbelink opened this issue Nov 22, 2024 · 1 comment · May be fixed by cyclus/cyclus#1837
Open

test_inventories_compact() fails with Sqlite on arm Mac #205

bennibbelink opened this issue Nov 22, 2024 · 1 comment · May be fixed by cyclus/cyclus#1837

Comments

@bennibbelink
Copy link
Contributor

bennibbelink commented Nov 22, 2024

Tried to add arm support to cymetric conda package and stumbled across this issue. Can reproduce with these steps (on arm mac):

  1. conda create -n test-env python==3.12
  2. conda activate test-env
  3. conda install conda-forge/label/cycamore_dev::cycamore conda-forge/label/cyclus_dev::cyclus pytest pytables
  4. (in cymetric dir)python -m pip install --target $(python -m site --user-site) --no-deps .
  5. cd tests && python -m pytest

If i comment out test_root_metrics.py:135-140 everything else passes, or if I remove the sqlite tuple from DBS in tools.py it also passes, so it seems to have something to do with sqlite. Have not been able to replicate on any other platforms. I also am not sure if the issue is in cymetric or upstream

@bennibbelink
Copy link
Contributor Author

bennibbelink commented Dec 9, 2024

Doing a sqldiff of the two DBs produced on arm mac and x86 ubuntu, the only changes I'm seeing besides SimId are the dependency versions.

ubuntu -> arm mac:
UPDATE Info SET SimId=x'7c6e538c705944a499085ef1b684808b', Hdf5Version='1.14.3-', BoostVersion='1_84', LibXML2Version='2.12.7' WHERE rowid=1;

arm mac -> ubuntu:
UPDATE Info SET SimId=x'810bb492740f4a4292f1c52a75e48c49', Hdf5Version='1.14.4-3', BoostVersion='1_86', LibXML2Version='2.13.4' WHERE rowid=1;


Backtrace:

* thread #1, queue = 'com.apple.main-thread', stop reason = signal SIGABRT
  * frame #0: 0x000000018e30e600 libsystem_kernel.dylib`__pthread_kill + 8
    frame #1: 0x000000018e346f70 libsystem_pthread.dylib`pthread_kill + 288
    frame #2: 0x000000018e253908 libsystem_c.dylib`abort + 128
    frame #3: 0x000000018e15d524 libsystem_malloc.dylib`malloc_vreport + 896
    frame #4: 0x000000018e1610a8 libsystem_malloc.dylib`malloc_report + 64
    frame #5: 0x000000018e17f1a4 libsystem_malloc.dylib`find_zone_and_free + 528
    frame #6: 0x0000000101fb020c libc++.1.0.dylib`std::__1::locale::~locale() + 72
    frame #7: 0x00000001026306a4 libboost_serialization.dylib`boost::archive::basic_text_iprimitive<std::__1::basic_istream<char, std::__1::char_traits<char>>>::basic_text_iprimitive(std::__1::basic_istream<char, std::__1::char_traits<char>>&, bool) + 348
    frame #8: 0x00000001026440cc libboost_serialization.dylib`boost::archive::xml_iarchive_impl<boost::archive::xml_iarchive>::xml_iarchive_impl(std::__1::basic_istream<char, std::__1::char_traits<char>>&, unsigned int) + 40
    frame #9: 0x000000010c956578 libcyclus.dylib`cyclus::SqliteBack::ColAsVal(boost::shared_ptr<cyclus::SqlStatement>, int, cyclus::DbTypes) + 4232
    frame #10: 0x000000010c94fbb8 libcyclus.dylib`cyclus::SqliteBack::Query(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>>, std::__1::vector<cyclus::Cond, std::__1::allocator<cyclus::Cond>>*) + 1544
    frame #11: 0x0000000101dfa26c lib.so`__pyx_pw_6cyclus_3lib_12_FullBackend_5query(_object*, _object* const*, long, _object*) + 4892
    frame #12: 0x0000000100068a54 python`PyObject_Vectorcall + 88

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 a pull request may close this issue.

1 participant