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

No definition of plot_transitions in network.py #18

Closed
aymeric75 opened this issue May 19, 2022 · 2 comments
Closed

No definition of plot_transitions in network.py #18

aymeric75 opened this issue May 19, 2022 · 2 comments

Comments

@aymeric75
Copy link

aymeric75 commented May 19, 2022

In latplan\latplan\network.py there is this piece of code:

              on_epoch_end = lambda epoch,logs: \
                    self.plot_transitions(
                        plot_val_data,
                        self.path+"/",
                        epoch=epoch),

But no definition of plot_transitions function in network.py

So for instance, when I run

latplan learn puzzle mnist 3 3 40000 StateAE

I get this error:

  File latplan/network.py line 450 function <lambda> : self.plot_transitions(
                   epoch = 50
                    logs = {'t_loss': 237913.65086805556, 'v_loss': 236870.45}
           plot_val_data = '<numpy.ndarray float32  (1, 2, 48, 48, 1)>'
                    self = <latplan.model.StateAE object at 0x14b2840da4c0>

                     AttributeError: 'StateAE' object has no attribute 'plot_transitions'

Best regards

@guicho271828
Copy link
Owner

Yeah, it is an older model, please consider cubespaceae etc.

@aymeric75
Copy link
Author

aymeric75 commented May 22, 2022

Hello,

Thanks, now I do
latplan learn puzzle mnist 3 3 40000 CubeSpaceAE_AMA4Plus instead

But, it's strange because once in the "train" function (in latplan\network.py) then nothing happens. This function does not call any Keras training function (e.g fit)

The only training function that I see is in plot_transitions function (also in network.py) but this function does not seems to be called with 'latplan learn..'

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

No branches or pull requests

2 participants