[petri-foo] Doesn't save and restore properly in NSM

Post by rosea.grammostola
Using Petri-foo from git, critical bugs ...
The sample banks aren't restored. When I save the session file, I
can't find the file in the session folder... big troubles ...

Been away; only just got the chance to investigate. Can't see the
problem.

I did this to install:

rm -rf Petri-Foo
git clone https://github.com/jwm-art-net/Petri-Foo.git
cd Petri-Foo
cmake .
make
sudo make install

Confirmed I am running the freshly-installed petri-foo

which petri-foo
/usr/local/bin/petri-foo

And then:

* I started non-session-manager
* Created a new session
* added client petri-foo
* (petri-foo starts with the 'default' patch)
* replaced the 'default' sample
* closed session
* re-opened session
* confirmed petri-foo loads the selected sample
* added empty patch
* selected sample for empty patch
* selected trim play mode for empty patch
* closed session
* re-opened session
* confirmed petri-foo loads both samples

Does the view->message_log window tell you anything?

If you follow the steps above to install and problems persist please
try rebuilding with debug info:

cd Petri-Foo
cmake . -DBuildForDebug=1
make
sudo make install

And tell me what the debug info says. If possible
subscribe and post to petri-foo-devel.

Thanks,
James.

James Morris

2012-08-21 17:30:12 UTC

Post by James Morris
cd Petri-Foo
cmake . -DBuildForDebug=1
make
sudo make install
And tell me what the debug info says. If possible
subscribe and post to petri-foo-devel.

PS run non-session-manager from a terminal to see petri-foo debug info
under session management.

Post by James Morris
Thanks,
James.

--
http://jwm-art.net/
image/audio/text/code/

rosea.grammostola

2012-12-15 10:11:21 UTC

Post by James Morris
cd Petri-Foo
cmake . -DBuildForDebug=1
make
sudo make install
And tell me what the debug info says. If possible
subscribe and post to petri-foo-devel.

PS run non-session-manager from a terminal to see petri-foo debug info
under session management.

[Petri-Foo.nAYAH] 11:10:07.222 ERROR: failed to create dir 'samples'
within '/home/derick/NSM Sessions/petri/Petri-Foo.nAYAH/'
[nsmd] Client "Petri-Foo" replied with: OK in 7.311000ms
.....[nsmd] Done.
/nsm/server/save says Saved.

I hope you test your own software too.

\r

rosea.grammostola

2012-12-15 10:16:45 UTC

Post by James Morris
cd Petri-Foo
cmake . -DBuildForDebug=1
make
sudo make install
And tell me what the debug info says. If possible
subscribe and post to petri-foo-devel.

PS run non-session-manager from a terminal to see petri-foo debug info
under session management.

Also if I connect the port in patchage it says

Jack: Connected Petri-Foo.nTNIG:out_left => system:playback_1
Unable to find port `jack:12' to connect

James Morris

2012-12-15 11:03:19 UTC

Post by James Morris
cd Petri-Foo
cmake . -DBuildForDebug=1
make
sudo make install
And tell me what the debug info says. If possible
subscribe and post to petri-foo-devel.

PS run non-session-manager from a terminal to see petri-foo debug
info under session management.

no i don't fucking bother. what's the point when there's users falling
over themselves to help out?

Post by rosea.grammostola
\r

--
http://jwm-art.net/
image/audio/text/code/

rosea.grammostola

2012-12-15 11:11:04 UTC

Post by James Morris
cd Petri-Foo
cmake . -DBuildForDebug=1
make
sudo make install
And tell me what the debug info says. If possible
subscribe and post to petri-foo-devel.

PS run non-session-manager from a terminal to see petri-foo debug
info under session management.

no i don't fucking bother. what's the point when there's users falling
over themselves to help out?

Ok, I was irritated that it didn't save the state. Users can be
irritating, but it can be irritating too when you can't make music but
hit another bug, I hope you can understand that too.

I've it build with debug enabled now ...

James Morris

2012-12-24 15:22:34 UTC

Post by James Morris
cd Petri-Foo
cmake . -DBuildForDebug=1
make
sudo make install
And tell me what the debug info says. If possible
subscribe and post to petri-foo-devel.

PS run non-session-manager from a terminal to see petri-foo debug
info under session management.

Sorry for my previous response to this.

It's quite straightforward to fix this so Petri-Foo responds with
ERR_UNSAVED_CHANGES rather than ERR_OK.

Is it acceptable for a client to bring up a dialog window to provide
the user with detailed information about what went wrong?

Thanks,
James.

Happy holidays... Will soon be torn away from the PC...

J. Liles

2012-12-24 19:07:39 UTC

Post by James Morris
cd Petri-Foo
cmake . -DBuildForDebug=1
make
sudo make install
And tell me what the debug info says. If possible
subscribe and post to petri-foo-devel.

PS run non-session-manager from a terminal to see petri-foo debug
info under session management.

Sorry for my previous response to this.
It's quite straightforward to fix this so Petri-Foo responds with
ERR_UNSAVED_CHANGES rather than ERR_OK.
Is it acceptable for a client to bring up a dialog window to provide
the user with detailed information about what went wrong?
Thanks,
James.
Happy holidays... Will soon be torn away from the PC...

I think I missed something. What message are we talking about here? 'open'
with ':switch;' capability?

If so, since 'save' would have been called immediately before 'open',
something must have gone wrong at that point to end up in a situation where
ERR_UNSAVED_CHANGES would make any sense. What's going wrong with 'save'?

J. Liles

2012-12-24 19:10:48 UTC

Post by J. Liles