Skip to content

Remove Visualizer/ prefix from downloaded profiles maybe? #31

@miharekar

Description

@miharekar

I used to prepend Visualizer/ prefix to downloaded profiles, but then a while back removed it. So I was surprised when I pulled a certain shot and it was still prefixed. I triple checked my code and was so confused what was going on. And then I found this:

de1app_plugin_DYE/DYE.tcl

Lines 2345 to 2350 in 87676c5

set pparts [split $profile(profile_title) "/"]
if { [llength $pparts] == 1 } {
set profile(profile_title) "Visualizer/$profile(profile_title)"
} elseif { [lindex $pparts 1] ne "Visualizer" } {
set profile(profile_title) "Visualizer/[lindex $pparts end]"
}

The reason I removed it was this discussion: https://3.basecamp.com/3671212/buckets/7351439/messages/7534772989#__recording_7836790146

I see that your code is now preventing multiple nesting, but still might be causing D-Flow issues. I'm not sure, since I don't use it, but maybe there are other issues?

Feel free to close this issue, if you don't think it's an issue. I just wanted to let you know that I've removed the prefixing and that you might follow suite. Or not. As you wish 😅

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions