tutorials icon indicating copy to clipboard operation
tutorials copied to clipboard

dianna package: eXplainable AI for ONNX models (XAI4ONNX)

Open elboyran opened this issue 1 year ago • 1 comments

Description

  • Added an overview tutorial for the DIANNA (Deep Insight and Neural Network Analysis) package and a link in the README

Motivation and Context

  • This informs the ONNX community about an eXplainable AI (XAI) tool for their ONNX models

elboyran avatar Jun 21 '24 09:06 elboyran

Indicating Ramakrishnan Sivakumar and Javier Martinez as reviewers as advised in the Slack thread.

elboyran avatar Jun 21 '24 09:06 elboyran

Looks good to me. Please wait for @javier-intel to approve as well.

ramkrishna2910 avatar Jul 12 '24 21:07 ramkrishna2910

@ramkrishna2910 and @javier-intel thank you for the approval. I do not have write access to be able to merge the PR.

elboyran avatar Jul 15 '24 07:07 elboyran

I wonder if the urls should be improved? At least some checks fail.

andife avatar Jul 15 '24 09:07 andife

I will try to fix those. (All checks were passing when I first submitted the PR.)

elboyran avatar Jul 15 '24 09:07 elboyran

@andife, @ramkrishna2910, @javier-intel I see now the previously failing workflow is expecting approval.

elboyran avatar Jul 15 '24 10:07 elboyran

Hi @elboyran there still appear to be a couple of failures in the last log. Were this the same or are they new ones?

javier-intel avatar Jul 15 '24 12:07 javier-intel

Hi @elboyran there still appear to be a couple of failures in the last log. Were this the same or are they new ones?

Some are the same, but it's hard for me to judge which are considered insecure URLs. I removed some I thought they might be. Some URL checks still fail without pointing to specific ones. Shall I remove all then (they are pointing to data and papers to explainers)?

Please, note that some checks fail not from my tutorial, but from the repo's README.md (which I modified only with a link to my tutorial).

elboyran avatar Jul 15 '24 12:07 elboyran

Hi @javier-intel , I have removed the links. If it fails again, I hope it is not due to my notebook. [Some links failed before in the README, not from my edits.]

elboyran avatar Jul 15 '24 12:07 elboyran

There are 3 issues flagged by the validation tool. Those are in the main branch and unrelated to your changes so they shouldn't block the PR. I'll merge the changes and fix the 404 flagged by the tool separately at a later time.

javier-intel avatar Jul 15 '24 13:07 javier-intel

[celebrate] Elena Ranguelova reacted to your message:


From: Javier Martinez @.> Sent: Monday, July 15, 2024 1:13:11 PM To: onnx/tutorials @.> Cc: Elena Ranguelova @.>; Mention @.> Subject: Re: [onnx/tutorials] dianna package: eXplainable AI for ONNX models (XAI4ONNX) (PR #291)

Merged #291https://github.com/onnx/tutorials/pull/291 into main.

— Reply to this email directly, view it on GitHubhttps://github.com/onnx/tutorials/pull/291#event-13505419005, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAYYBWPRIIR4PNEJRK2YP6DZMPDGPAVCNFSM6AAAAABJVRF7FKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGUYDKNBRHEYDANI. You are receiving this because you were mentioned.Message ID: @.***>

elboyran avatar Jul 15 '24 17:07 elboyran