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

[Q] some fault of svs #27

Closed
asas21212aa opened this issue Apr 12, 2023 · 8 comments
Closed

[Q] some fault of svs #27

asas21212aa opened this issue Apr 12, 2023 · 8 comments
Assignees

Comments

@asas21212aa
Copy link

sorry, i found some fault when using the method='svs'

@JHoelli
Copy link
Contributor

JHoelli commented Apr 13, 2023

Hi @asas21212aa,

would you mind sharing some more information on the occurring error so that we can try to locate the issue?
(e.g., Error Message ideally with Line Number, Version of TsInterpret, Function Calls, Parameters used or in the best Case a Minimal Sample ).

This is relevant, e.g. SVS is currently not implemented for TensorFlow Models and the Method is supposed to be called in the current Version of TSInterpret (0.1.3) with Upper Case: method='SVS'.

@JHoelli JHoelli self-assigned this Apr 13, 2023
@asas21212aa
Copy link
Author

asas21212aa commented Apr 13, 2023 via email

@JHoelli
Copy link
Contributor

JHoelli commented Apr 13, 2023

Hi @asas21212aa ,

I just rechecked the notebook locally and for me it seems to work. Therefore I would need a bit more information:
(The easiest would be if you share the exact code you are using.)

  • What is your version of TSInterpret? (pip show TSInterpret)
  • What are the input/Output shapes?
    int_mod=TSR(model, train_x.shape[-1],train_x.shape[-2], method='SVS', \ mode='feat')
    When using mode='feat' with 'Basic_Motion' train_x.shape[-1] is supposed to be 100 and train_x.shape[-2] is supposed to be 6. For mode='time' it is the other way around.
    item = np.array([test_x[0,:,:]]) label = int(np.argmax(test_y[0]))
    exp=int_mod.explain(item,labels=label,TSR = False)
    item has shape (1,6,100) for feat or (1,100,6) if you chose time

@asas21212aa
Copy link
Author

asas21212aa commented Apr 13, 2023 via email

@JHoelli
Copy link
Contributor

JHoelli commented Apr 13, 2023

Hi @asas21212aa,

Would you mind updating the TSInterpret version to 0.1.3?
pip install --upgrade pip
pip install --upgrade TSInterpret

Your version is quite old (its from August 2022). Multiple updated version are in-between and we did some major changes in the implementation. Therefore the current notebook implementations will not work with the older package versions.

If you still have issues, let me know.

@asas21212aa
Copy link
Author

asas21212aa commented Apr 17, 2023 via email

@JHoelli
Copy link
Contributor

JHoelli commented Apr 18, 2023

Hi @asas21212aa ,

I am sorry but I do not understand what you mean by "Selection Edition".

The version is there https://pypi.org/project/TSInterpret/. Please make sure you use a python version >= 3.9.

Maybe you need to update pip before installing TsInterpret: pip install --upgrade pip or python -m pip install --upgrade pip. As I am not sure what you mean by Selection Edition I am unsure how to do that there.

@JHoelli
Copy link
Contributor

JHoelli commented Sep 18, 2023

Closed due to inactivity

@JHoelli JHoelli closed this as completed Sep 18, 2023
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