Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Fix support for open last workfile with AVALON_SCENEDIR #5127

Closed

Conversation

BigRoy
Copy link
Collaborator

@BigRoy BigRoy commented Jun 12, 2023

Changelog Description

Fix support for open last workfile with AVALON_SCENEDIR + cosmetics

Additional info

Resolves #5126

Testing notes:

  1. Enable launch last workfile (= enabled by default)
  2. Add "AVALON_SCENEDIR": "scenes" to e.g. the houdini or maya environment
  3. Save a workfile with the Workfiles tool.
  4. Restart the application, it should launch the existing workfile directly.

@BigRoy BigRoy added type: bug Something isn't working community contribution labels Jun 12, 2023
@BigRoy BigRoy requested a review from iLLiCiTiT June 12, 2023 09:11
@ynbot ynbot added the size/XS Denotes a PR changes 0-99 lines, ignoring general files label Jun 12, 2023
@iLLiCiTiT
Copy link
Member

Like I've mentioned in discord communication. We didn't want to support scenedir, only for host integrations that require it. In that case it is handled by the host addon. That discussion happend few months ago, so maybe we can reopen the discussion @antirotor @mkolar @jakubjezek001 . But with that in mind, I don't know if this will be approved.

@BigRoy
Copy link
Collaborator Author

BigRoy commented Jun 12, 2023

Link to the 'discord communication'

Thanks - just for context: we've been using AVALON_SCENEDIR all the time for Houdini, Fusion and Maya. So being able to have a subfolder for our workfiles for those applications will be a requirement for us for backwards compatibility and also a forward preference to still allow that behavior as it's preferred by our artists too.

As described in #5126 I'm fine with dropping AVALON_SCENEDIR if there's a replacement built-in feature through which we can define the workfiles subdirectory in the work root per application.

We also have our workdir template include the {app_name} so we have e.g. work/{task}/{app}/{scenedir} resulting in work/animation/maya/scenes. The maya workspace.mel still resides in the work root (maya/ folder) but the workfiles themselves end up in scenes subfolder. This is the behavior we're looking to preserve (and with this PR also support at least with last workfile open on launch)

@BigRoy
Copy link
Collaborator Author

BigRoy commented Jun 30, 2023

We are currently using this in production 👍

@mkolar
Copy link
Member

mkolar commented Jul 7, 2023

I agree with the functionality, but I think we need a slightly different approach. We have some requests to make the workfile handling a little more robust, and this should be included in there. We'll prep some proposal soon hopefully.

@mkolar
Copy link
Member

mkolar commented Feb 6, 2024

Because we're splitting OpenPype into ayon-core and individual host addons, this PR would have to be re-created to target one of those. The PR has merit, the solution, not so much. Let's try to revisit in AYON

We're closing it down, but we'll he happy for a new PR to ynput/ayon-core repository once it's up.

@mkolar mkolar closed this Feb 6, 2024
@ynbot ynbot added this to the next-patch milestone Feb 6, 2024
@jakubjezek001 jakubjezek001 removed this from the next-patch milestone Feb 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
community contribution size/XS Denotes a PR changes 0-99 lines, ignoring general files type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Bug: Open last workfile does not work with AVALON_SCENEDIR
6 participants