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

Add information about Sandia Cubit compatibility #139

Open
wants to merge 10 commits into
base: rc-0.2.4
Choose a base branch
from

Conversation

gonuke
Copy link
Member

@gonuke gonuke commented Sep 29, 2022

Based on info from Coreform, adding info about which Sandia Cubit version works.

Also anticipates next release...???

See here for updated README

@pshriwise
Copy link
Member

Thanks for this update, @gonuke! Nice to have some more info on the connection between these Cubit flavors.

@jbae11 has volunteered to try the plugin packages out for Sandia version 15.8 if you haven’t already.

@jbae11
Copy link

jbae11 commented Sep 29, 2022

I've requested 15.8 and will probably get it next week, the poc is in travel.

@jbae11
Copy link

jbae11 commented Oct 4, 2022

@pshriwise I might be understanding it wrong, but it looks like all the ubuntu version (except 18.04) have been tested with 15.8? So is it just the mac that I need to test?

@pshriwise
Copy link
Member

@jbae11 I think that's right but I don't see a link to a Mac version for v0.2.4. In fact, I don't see any of those artifacts on the release page for v0.2.4. I can download them using the links in the README though.

@gonuke any thoughts here?

@jbae11
Copy link

jbae11 commented Oct 6, 2022

@pshriwise I can only find the versions up to v0.2.3 in the release page, is there another link in the README that has v0.2.4?

@pshriwise
Copy link
Member

@pshriwise I can only find the versions up to v0.2.3 in the release page, is there another link in the README that has v0.2.4?

Not for Mac that I could see. I'm not sure why though. IIRC we should just need to enable it in the automate builds.

@jbae11
Copy link

jbae11 commented Oct 7, 2022

okay if there isn't one pre-built, I can try to build it myself (although not sure how long that's going to take)? Let me know how you want me to proceed.

@gonuke
Copy link
Member Author

gonuke commented Oct 7, 2022

This is a draft PR waiting for the 0.2.4 release to happen!

@gonuke
Copy link
Member Author

gonuke commented Oct 7, 2022

There were issues when trying to build for this version of Cubit on MacOS. @bam241 can perhaps remember what they are. Also some discussion in #135

@gonuke
Copy link
Member Author

gonuke commented Oct 7, 2022

Actually - @bam241 fixed the MacOS problem 🎉

You can find a build tarball here: https://github.com/svalinn/Cubit-plugin/actions/runs/2883295348

@pshriwise
Copy link
Member

pshriwise commented Oct 7, 2022

This is a draft PR waiting for the 0.2.4 release to happen!

Welp, that would explain it. 😆Sorry, I'm a little out of date with respect to the development cycle here.

@jbae11 if you can give the link @gonuke provided a try that would be great!

@jbae11
Copy link

jbae11 commented Oct 7, 2022

Thanks @gonuke @pshriwise @bam241

I untared, moved the files, but it seems like Cubit is not recognizing the plugin (i.e. help dagmc errors). Is there an environment variable I should be setting or something?

image

@gonuke gonuke changed the base branch from develop to rc-0.2.4 October 10, 2022 14:53
@gonuke gonuke marked this pull request as ready for review October 10, 2022 14:54
@gonuke
Copy link
Member Author

gonuke commented Oct 10, 2022

Thanks @jbae11 - I am still not entirely clear where files need to be placed for the Sandia version of Cubit. I'll have to get that info next.

@gonuke
Copy link
Member Author

gonuke commented Oct 10, 2022

Also, perhaps someone with Coreform MacOS experience can advise? @bam241 ?

@pshriwise
Copy link
Member

Lookin at @jbae11's screenshot again... we ship a .so file with the tarball for OSX?

@bam241
Copy link
Member

bam241 commented Oct 15, 2022

Lookin at @jbae11's screenshot again... we ship a .so file with the tarball for OSX?

we do ship both, .dylib for either static or shared (can't recall which is which), and .so for the other.

I think the problem here is likely a path problem.

We build the plugin to be install in Coreform-Cubit-X.Y.app/Contents/MacOS/ the path is not ready for the Sandia version as it does not match exactly the Coreform one.

Sadly one did not figure out out to deal with relative path rather than absolute path....

sorry it took me so long to see the notification....

@bam241
Copy link
Member

bam241 commented Oct 15, 2022

one solution would be to build a sandia specific version with the proper sandia path.

this might only require to build a different copy using the correct Sandia path when setting them before packing in mac_build_plugin_pkg(){

@bam241
Copy link
Member

bam241 commented Oct 15, 2022

looking at it more carefully, we actually only correct the path for 17.1.

But I believe the same kind of "touch" would be required to build a proper Sandia version from the one we build

@jbae11
Copy link

jbae11 commented Oct 17, 2022

so should I wait for another build or is there a way for me to alter that path on my end?

@jbae11
Copy link

jbae11 commented Oct 17, 2022

Also I'm not sure if this is a useful datapoint, but I tried the same thing with Cubit15.8 on Ubuntu 20.04, and it didn't work initially, but worked when I added the ..../Cubit-15.8/bin and ..../Cubit-15.8/bin/plugins/svalinn to the LD_LIBRARY_PATH. I tried the same thing with the mac, but still no luck.

@gonuke
Copy link
Member Author

gonuke commented Nov 20, 2022

While we're still working out the installation details.... does it makes sense to merge this since the release page includes this info?

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.

4 participants