Alex Beattie

Results 28 comments of Alex Beattie

Yes that PR would fix the problem for anyone who will update to the version of tensorflow which will eventually contain the cpuinfo fix when it is merged. The problem...

As noted by your [comment](https://github.com/bazelbuild/bazel/pull/17547#issuecomment-1442077924), there are still 3 projects with open PRs that rely on the `darwin` means `x86_64` convention. Additionally [ruy](https://github.com/google/ruy/blob/master/third_party/cpuinfo.BUILD) is not listed but is another dependency...

I think it would be beneficial to define the behavior that should be supported (or expected). I can see two cases: 1. OpenSim should read the data file based on...

I agree that in research the commas are not used for numerics generally but the system locale is important for things like date and calendar display settings within the system...

Also are you able to reopen this issue? It looks like the closing of the linked PR also closed the issue inadvertently.

I think that is a great solution! I am still relatively new with this code base so I didn't get to that level of digging yet. Hopefully the maintainers could...

It looks like @adamkewley has used [this solution](https://github.com/ComputationalBiomechanicsLab/opensim-creator/blob/main/src/OpenSimCreator/Platform/OpenSimCreatorApp.cpp#L86) in OpenSimCreator if you also want to check that for reference.

> iirc, I wrote the comment because we had a problem in prod where dutch students in TU Delft were ending up with incompatible data files when they were shared...

@nickbianco @ksomml I have created a minimum reproduction of the issue [here](https://github.com/gateway240/opensim-core-examples). The example sets the locale locally in the code (to Finnish but any locale which uses commas for...

Sure thing. @nickbianco Here's the full backtrace from gdb: ```gdb #0 __pthread_kill_implementation (threadid=, signo=signo@entry=6, no_tid=no_tid@entry=0) at pthread_kill.c:44 #1 0x00007fffec4a5463 in __pthread_kill_internal (threadid=, signo=6) at pthread_kill.c:78 #2 0x00007fffec44c120 in __GI_raise (sig=sig@entry=6)...