Skip to content

25237 needs a follow up work #25597

@stas00

Description

@stas00

Looks like the modified check_config_can_be_init_without_params in #25237 broke the yet unmerged #24796

I patched it to skip these tests here so that we could merge it here 906485b
but most likely this needs to be fixed yet.

FWIW, this part doesn't fail:

    def check_config_can_be_init_without_params(self):
        config = self.config_class()
        self.parent.assertIsNotNone(config)

The updated test in this PR expects the config class to fail if it's a composition, but in the case of Idefics it succeeds (not sure why it should fail).

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions