Re: [lucene] branch main updated: Prevent NPEs while still handling the polar case for horizontal planes right off the pole

2022-11-24 Thread Dawid Weiss
I have multiple Windows machines, including laptops. Never had this issue (but I stay away from WSL and such). Performance is, I'd say 25% slower than comparable Linux machines. Something is wrong with your rig. Dawid On Thu, Nov 24, 2022 at 12:26 PM Karl Wright wrote: > My entire tool set and

Re: [lucene] branch main updated: Prevent NPEs while still handling the polar case for horizontal planes right off the pole

2022-11-24 Thread Karl Wright
My entire tool set and work environment is inside WSL. I've determined that the issue for me is the performance of the file system. I had to remove the (bundled) antivirus software to get even where I am now. But I have no evidence that even doing windows-native operations with this disk are

Re: [lucene] branch main updated: Prevent NPEs while still handling the polar case for horizontal planes right off the pole

2022-11-24 Thread Jan Høydahl
I’m not on Windows myself, but I think the trick is doing the git clone to the WSL file system. So you may have one checkout for use with windows and another for use within wsl. And if you’re a CLI person, there’s a GitHub cli tool ‘hub’ that is handy: https://hub.github.com/ Jan Høydahl >

Re: [lucene] branch main updated: Prevent NPEs while still handling the polar case for horizontal planes right off the pole

2022-11-17 Thread Dawid Weiss
I never used WSL but it does seem like the problem there: "As you can tell from the comparison table above, the WSL 2 architecture outperforms WSL 1 in several ways, with the exception of performance across OS file systems, which can be addressed by storing your project files on the same

Re: [lucene] branch main updated: Prevent NPEs while still handling the polar case for horizontal planes right off the pole

2022-11-17 Thread Robert Muir
if your machine is really 12 cores and 64GB ram but is that slow, then uninstall that windows shit immediately, that's horrible. On Thu, Nov 17, 2022 at 5:46 AM Karl Wright wrote: > > Thanks - the target I was using was the complete "build" target on the whole > project. This will be a

Re: [lucene] branch main updated: Prevent NPEs while still handling the polar case for horizontal planes right off the pole

2022-11-17 Thread Dawid Weiss
> I have slow network here so it is possible that the entire build was slow > for that reason. The machine is a new Dell laptop, 12 cores, 64GB memory, > but I am running under Windows Subsystem for Linux which is a bit slower > than native Ubuntu. Still, the gradlew command you gave takes many

Re: [lucene] branch main updated: Prevent NPEs while still handling the polar case for horizontal planes right off the pole

2022-11-17 Thread Karl Wright
Thanks - the target I was using was the complete "build" target on the whole project. This will be a valuable improvement. ;-) I have slow network here so it is possible that the entire build was slow for that reason. The machine is a new Dell laptop, 12 cores, 64GB memory, but I am running

Re: [lucene] branch main updated: Prevent NPEs while still handling the polar case for horizontal planes right off the pole

2022-11-17 Thread Dawid Weiss
> Thank you for the comment. > Sorry if it came out the wrong way - I certainly didn't mean it to be unkind. > It took me several days just to get things set up so I was able to commit > again, and I did this through command-line not github. > These things are not mutually exclusive - I work

Re: [lucene] branch main updated: Prevent NPEs while still handling the polar case for horizontal planes right off the pole

2022-11-17 Thread Karl Wright
Thank you for the comment. It took me several days just to get things set up so I was able to commit again, and I did this through command-line not github. The full gradlew script takes over 2 hours to run now so if there's a faster target I can use to determine these things in advance I'd love

Re: [lucene] branch main updated: Prevent NPEs while still handling the polar case for horizontal planes right off the pole

2022-11-16 Thread Dawid Weiss
Hi Karl, This commit broke the build because code formatting was off (this was fixed in a subsequent, unrelated commit). I spent some time looking for the issue to check what happened and couldn't find it anywhere. Github's PR infrastructure makes it quite convenient to ensure everything passes