Skip to content

Conversation

@kevinmessiaen
Copy link
Member

Description

Issue

By default we try to serialise models by value using cloudpickle in order to be able to execute them in ML Worker without having to install the referenced dependency. However cloudpickle is not able to serialise everything causing the serialisation to fail.

Solution

For complex code that aren't pickable, it's always possible to save the code in a module and install it into the ML worker. Therefore when cloudpickle fail to save objects by value, we fallback into saving by reference with a warning.

Related Issue

Type of Change

  • 📚 Examples / docs / tutorials / dependencies update
  • 🔧 Bug fix (non-breaking change which fixes an issue)
  • 🥂 Improvement (non-breaking change which improves an existing feature)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🔐 Security fix

@linear
Copy link

linear bot commented Feb 20, 2024

@sonarqubecloud
Copy link

@kevinmessiaen kevinmessiaen merged commit 28e8178 into main Feb 21, 2024
@kevinmessiaen kevinmessiaen deleted the GSK-2819 branch February 21, 2024 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants