Skip to content

Conversation

@sbalandi
Copy link
Collaborator

No description provided.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@sbalandi sbalandi force-pushed the ace branch 8 times, most recently from d6328bf to f10fb06 Compare August 19, 2025 21:49
@sbalandi sbalandi force-pushed the ace branch 8 times, most recently from 196b7a5 to 92c58be Compare August 20, 2025 15:31
@sbalandi sbalandi marked this pull request as ready for review August 20, 2025 17:05
@review-notebook-app
Copy link

review-notebook-app bot commented Aug 25, 2025

View / edit / reply to this conversation on ReviewNB

aleksandr-mokrov commented on 2025-08-25T11:52:45Z
----------------------------------------------------------------

Line #25.    # pip_install(

Remove it


sbalandi commented on 2025-08-25T19:01:01Z
----------------------------------------------------------------

removed

@review-notebook-app
Copy link

review-notebook-app bot commented Aug 25, 2025

View / edit / reply to this conversation on ReviewNB

aleksandr-mokrov commented on 2025-08-25T11:52:46Z
----------------------------------------------------------------

Fix link


sbalandi commented on 2025-08-25T19:01:32Z
----------------------------------------------------------------

done

@review-notebook-app
Copy link

review-notebook-app bot commented Aug 25, 2025

View / edit / reply to this conversation on ReviewNB

aleksandr-mokrov commented on 2025-08-25T11:59:13Z
----------------------------------------------------------------

Line #12.    convert_models(pipeline, model_dir=ov_converted_model_dir, orig_checkpoint_path=checkpoint_dir, quantization_config=weights_compression_config)

Could you separate model_dir for fp16, int8 and int4?


sbalandi commented on 2025-08-25T19:01:17Z
----------------------------------------------------------------

added

Copy link
Collaborator Author

removed


View entire conversation on ReviewNB

Copy link
Collaborator Author

added


View entire conversation on ReviewNB

Copy link
Collaborator Author

done


View entire conversation on ReviewNB

@aleksandr-mokrov aleksandr-mokrov merged commit 86fec7d into openvinotoolkit:latest Aug 26, 2025
15 of 16 checks passed
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

Successfully merging this pull request may close these issues.

2 participants