search

ValeevGroup / SeQuant

SeQuant: Symbolic Algebra of Tensors over Operators and Scalars
GNU General Public License v3.0
15 stars 5 forks source link

Improve clang format script #215

Closed Krzmbrzl closed 2 months ago

Krzmbrzl commented 3 months ago

@evaleev do the scripts (still) work on macOS?

evaleev commented 2 months ago

@evaleev do the scripts (still) work on macOS?

appears to. What was the issue on "your" platform?

Krzmbrzl commented 2 months ago

What was the issue on "your" platform?

Turning paths into relative ones failed because I have ~/Documents/ symlinked to a directory on a different drive. And since SeQuant lives at ~/Documents/Git/SeQuant its path expanded to something like /other/drive/SeQuant whereas my home still is /home/robert. Thus, the used algorithm for turning paths inside the SeQuant dir to paths relative to the home directory failed. In the new script I instead make paths relative to the SeQuant root dir itself.

And while I was doing that I found out that we don't have to code the "making a path relative to another" thing ourselfs.

For completeness sake, I am working on Linux (KDE Neon to be precice)

evaleev commented 2 months ago

Thanks!