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 Houdini default executable path examples for Mac and Linux #15

Merged

Conversation

BigRoy
Copy link
Contributor

@BigRoy BigRoy commented Aug 2, 2024

Changelog Description

Set up draft as defaults for Houdini executables on mac and linux

Additional info

These paths may need to be confirmed to be correct by Linux/Mac users.

Based on e.g. https://www.sidefx.com/faq/question/install-linux/

Testing notes:

  1. Confirm the defaults
  2. The new defaults should apply
  3. Settings should work when loading new applications package

@BigRoy BigRoy added the type: enhancement Improvement of existing functionality or minor addition label Aug 2, 2024
@BigRoy BigRoy requested a review from m-u-r-p-h-y August 2, 2024 12:06
@BigRoy BigRoy self-assigned this Aug 2, 2024
server/applications.json Outdated Show resolved Hide resolved
Copy link

@Lypsolon Lypsolon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Linux defaults are reasonable for Houdini. but you should not point to a specific Houdini patch version. there is usually a symlink called hfs20.0 or hfs20.5, etc; I would say it is more likely that the symlink is set (via the Houdini launcher) than that the exact Houdini version is installed.

ps: If a user changes the default install location, they can change the settings, I say.

Also not sure if it should be part of this pr, but Houdini 20 and maybe even 20.5 would be nice

@BigRoy
Copy link
Contributor Author

BigRoy commented Aug 28, 2024

The Linux defaults are reasonable for Houdini. but you should not point to a specific Houdini patch version. there is usually a symlink called hfs20.0 or hfs20.5, etc; I would say it is more likely that the symlink is set (via the Houdini launcher) than that the exact Houdini version is installed.

ps: If a user changes the default install location, they can change the settings, I say.

Also not sure if it should be part of this pr, but Houdini 20 and maybe even 20.5 would be nice

Could you provide me the full default executable paths (and preferably even "suggest" them so we can finalize this PR?)

@Lypsolon
Copy link

executable

/opt/hfs20.0 this is an path example.
also there is an almalinux 9 machine in the ynput office if you want to test on Linux there is infrastructure for that. (its limited and i hope we get to a way better point but there is a baseline)

Copy link

@Lypsolon Lypsolon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a few cosmetics.
the Houdini versions are out of order
19.5 19 18.5 20.5
thats a bit meh. can we have it 18.5 19 19.5 20 20.5 ?

aaaand you are missing Hou20.0 no ?

@BigRoy
Copy link
Contributor Author

BigRoy commented Sep 18, 2024

thats a bit meh. can we have it 18.5 19 19.5 20 20.5 ?

aaaand you are missing Hou20.0 no ?

Yes, and yes. But this PR just adds paths for Linux and Mac - I did not touch the order.
I can shuffle it around and add Hou 20.0 of course - not sure why it was lacking upstream.

@Lypsolon
Copy link

thats a bit meh. can we have it 18.5 19 19.5 20 20.5 ?
aaaand you are missing Hou20.0 no ?

Yes, and yes. But this PR just adds paths for Linux and Mac - I did not touch the order. I can shuffle it around and add Hou 20.0 of course - not sure why it was lacking upstream.

understandable. i think the shuffle and adding 20.0 would be great.
if we dont want it in this PR thats fine.

rest looks good.

@BigRoy
Copy link
Contributor Author

BigRoy commented Sep 19, 2024

@Lypsolon can you approve this with a review?

I can look into shuffling the order and add houdini 20.0 in a separate PR.

@MustafaJafar
Copy link
Contributor

MustafaJafar commented Sep 19, 2024

Hey,

I followed along the discussion and checked the installation guides as well as some YT videos.
So, there are installation directory and symlink to that directory (which will be most likely overridden) .
e.g.

  • Install directory: /opt/hfs20.5.332/
  • Symlink: /opt/hfs/20.5/

I wonder what do people in various studios mostly use?

@BigRoy
Copy link
Contributor Author

BigRoy commented Sep 19, 2024

e.g.

* Install directory: `/opt/hfs20.5.332/`

* Symlink:  `/opt/hfs/20.5/`

I wonder what do people in various studios mostly use?

Just for context - I actually changed this to the symlink one based on comments from @Lypsolon but I had the other one originally. Input of course is greatly appreciated!

@Lypsolon
Copy link

e.g.

* Install directory: `/opt/hfs20.5.332/`

* Symlink:  `/opt/hfs/20.5/`

I wonder what do people in various studios mostly use?

Just for context - I actually changed this to the symlink one based on comments from @Lypsolon but I had the other one originally. Input of course is greatly appreciated!

the symlinc is /opt/hfs20.5/ not hfs/20.5
other than that i believe that the symlinc is the best as its updates when you update via launcher etc.

@MustafaJafar
Copy link
Contributor

other than that i believe that the symlinc is the best as its updates when you update via launcher etc.

yeah, but, if they are using Major.minor.patch version, it may fail, correct?

@Lypsolon
Copy link

other than that i believe that the symlinc is the best as its updates when you update via launcher etc.

yeah, but, if they are using Major.minor.patch version, it may fail, correct?

what are you talking about ?

when you install a version of Houdini on linux a symlinc fouler is created to point to the newest (i belive prod) version of Houdini you have installed. if you now update hou 20.5.01 to hou 20.5.02 the symlinc will be updated by the Houdini launcher.

thats the idea behind the system that they implemented.
it is posible that this is a houdini launcher installer funciton as i did not install houdini by hand for a long while.

also its a lot more possible that some one installed a version of Houdini that we did not have in the defaults than that they dont have the symlinc

@BigRoy
Copy link
Contributor Author

BigRoy commented Sep 19, 2024

when you install a version of Houdini on linux a symlinc fouler is created to point to the newest (i belive prod) version of Houdini you have installed. if you now update hou 20.5.01 to hou 20.5.02 the symlinc will be updated by the Houdini launcher.

thats the idea behind the system that they implemented. it is posible that this is a houdini launcher installer funciton as i did not install houdini by hand for a long while.

also its a lot more possible that some one installed a version of Houdini that we did not have in the defaults than that they dont have the symlinc

Correct - but from a studio POV that may be less preferred. For context, we have multiple Houdini 19.5 and 20.0 versions in different productions. If using the symlink path, I'd then need to remap them manually.

Also, I dislike it a tiny bit because it mismatches what Houdini and Mac does.

@Lypsolon
Copy link

Correct - but from a studio POV that may be less preferred. For context, we have multiple Houdini 19.5 and 20.0 versions in different productions. If using the symlink path, I'd then need to remap them manually.

if you use different versions with prjA and prjB you will need to remap / change it in the settings in all cases no meter what we put there.
i believe question should be what is the most likely to be the default, if you run "none default" or multiple versions we will need to touch the settings in every case.

Also, I dislike it a tiny bit because it mismatches what Houdini and Mac does.

Houdini and Mac ? you mean windows and mac right ?
with that one i get the point but would say a default should be as close to the default on the given system, i mean a mac user dosnt care what windows dose because why would he and vise versa.
i believe the setting for linux should feel natural for the linux user, the same for windows and mac.

@BigRoy
Copy link
Contributor Author

BigRoy commented Sep 20, 2024

if you use different versions with prjA and prjB you will need to remap / change it in the settings in all cases no meter what we put there.

No you don't ;) you'd have a default template showing you how to set it to a specific version.
Then you just add one entry for the other project. And assign the respective version to the projects.

It's better to be explicit than implicit.

If the application in settings state "Houdini 20.5.332" then I'd be quite pissed as admin that it'd secretly run another version on some machine - I want that version!

i believe the setting for linux should feel natural for the linux user, the same for windows and mac.

Do note that, it's not for the user - but for the admin.


@m-u-r-p-h-y @antirotor any thoughts?

So that is to decide to point to WHICH of these two:

  • Install directory: /opt/hfs20.5.332/
  • Symlink: /opt/hfs/20.5/

Then we can pick the one we want for Linux here and merge this.

@antirotor
Copy link
Member

The more specific one. When you configure as an admin version of DCC, it is usually the specific version because it is tested and your plug-in works with it. There might be reason to use different version on Windows than Linux because of some particular compatibility issue but that should be always considered some edge case and that change must be done willingly by the admin himself.

Copy link

@Lypsolon Lypsolon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is open for a long time so Im just gonna say.
Im Good with the changes and i think its good to go.

@BigRoy
Copy link
Contributor Author

BigRoy commented Oct 8, 2024

@antirotor with 5204637 I've now made it the exact paths again on Linux.

@MustafaJafar or @Lypsolon can you just confirm for me once more that's a valid default path on linux - if so, smash the MERGE button please.

@BigRoy BigRoy merged commit 82f900b into ynput:develop Oct 8, 2024
1 check passed
@BigRoy BigRoy deleted the enhancement/houdini_add_platform_defaults branch October 9, 2024 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants