[Good First Issue][NNCF]: dump the ignored scope more gracefully
Context
NNCF saves parameters of the compression to OpenVINO IR.
It's done in dump_parameters method using set_rt_info API.
But the problem is that ignored scope is saved not gracefully - it prints all fields even if they are empty.
What needs to be done?
- Dump ignored scope in more compact way.
- not dump
<validate value="True" /> - not dump fields
names,patternsortypeswhen they are empty. - when all fields empty - dump in one line:
<ignored_scope value=[]>
- Add very lightweight unit test checking the content of <rt_info> for a dummy synthetic model, for example via get_rt_info API.
Example Pull Requests
No response
Resources
Contact points
@ljaljushkin
Ticket
129593
.take
Thank you for looking into this issue! Please let us know if you have any questions or require any help.
Hello @ljaljushkin and @alexsu52 ,
I have corrected it to,
- not dump
<validate value="True" /> - not dump
names,patternsortypeswhen they are empty.
In the case when,
when all fields empty
The dump is looking like <ignored_scope value="None" /> is this fine? If not then is there any other way other than modifying the C++ code of set_rt_info API?
Hello, never mind, I figured it out!
Now it's dumping <ignored_scope value="[]" /> when all fields are empty ^^
Will create a PR soon after adding unit tests.