mlfoundations/open_clip

Pattern Findings

487 matches across 10 categories. Click a row to expand file-level details.

Hyper-Verbose Identifiers234 hits · 266 pts

Severity	File	Line	Snippet
LOW	tests/test_audio_zero_shot.py	87	def test_hf_audio_dataset_wrapper_and_classnames():
LOW	tests/test_audio_zero_shot.py	101	def test_hf_audio_dataset_wrapper_accepts_decoder_like_audio():
LOW	tests/test_audio_zero_shot.py	116	def test_build_hf_audio_zero_shot_dataset_defaults_to_no_workers(monkeypatch):
LOW	tests/test_audio_zero_shot.py	152	def test_build_hf_audio_zero_shot_dataset_sets_worker_context(monkeypatch):
LOW	tests/test_audio_zero_shot.py	189	def test_run_audio_zero_shot_classifier():
LOW	tests/test_audio_zero_shot.py	219	def test_audio_zero_shot_template_validation():
LOW	tests/test_audio_zero_shot.py	229	def test_run_audio_zero_shot_classifier_uses_bare_model_dummy_audio(monkeypatch):
LOW	tests/test_hf_model.py	22	def test_pretrained_text_encoder(model_id):
LOW	tests/test_naflex_timm_conversion.py	46	def _tiny_native_vit_clip_config():
LOW	tests/test_naflex_timm_conversion.py	67	def _tiny_native_vit_quickgelu_clip_config():
LOW	tests/test_naflex_timm_conversion.py	100	def _write_tiny_big_vision_npz(path):
LOW	tests/test_naflex_timm_conversion.py	167	def test_force_naflex_vision_passes_use_naflex_to_timm(monkeypatch):
LOW	tests/test_naflex_timm_conversion.py	186	def test_force_naflex_vision_accepts_timm_models_by_module_membership():
LOW	tests/test_naflex_timm_conversion.py	193	def test_force_naflex_vision_configures_pe_core_timm_tower():
LOW	tests/test_naflex_timm_conversion.py	206	def test_force_naflex_vision_converts_native_vit_config():
LOW	tests/test_naflex_timm_conversion.py	222	def test_force_naflex_vision_rejects_non_vit_model():
LOW	tests/test_naflex_timm_conversion.py	231	def test_parse_use_naflex_enables_timm_naflex_aug_cfg():
LOW	tests/test_naflex_timm_conversion.py	239	def test_parse_force_naflex_vision_does_not_enable_naflex_data_pipeline():
LOW	tests/test_naflex_timm_conversion.py	248	def test_naflex_data_config_defaults_to_model_patch_size():
LOW	tests/test_naflex_timm_conversion.py	317	def test_convert_naflex_timm_state_dict_maps_patch_embed_weight():
LOW	tests/test_naflex_timm_conversion.py	346	def test_convert_naflex_native_vit_state_dict_folds_class_pos_embed():
LOW	tests/test_naflex_timm_conversion.py	386	def test_force_naflex_native_vit_dense_output_matches_native(monkeypatch, config_fn):
LOW	tests/test_task_specific_unit.py	84	def test_clip_task_loss_aggregation():
LOW	tests/test_task_specific_unit.py	94	def test_clip_task_logit_scale_in_output():
LOW	tests/test_task_specific_unit.py	108	def test_coca_build_loss_inputs_autoregressive_shift():
LOW	tests/test_task_specific_unit.py	121	def test_coca_training_forward_produces_loss():
LOW	tests/test_task_specific_unit.py	131	def test_coca_compute_accum_loss_concatenates_batches():
LOW	tests/test_task_specific_unit.py	164	def test_distill_teacher_frozen():
LOW	tests/test_task_specific_unit.py	173	def test_distill_teacher_stays_eval():
LOW	tests/test_task_specific_unit.py	182	def test_distill_training_forward_prefixes_teacher_outputs():
LOW	tests/test_task_specific_unit.py	204	def test_distill_teacher_no_grad():
LOW	tests/test_audio_model_configs.py	10	def test_clap_htsat_tiny_config_instantiates_with_audio_deps():
LOW	tests/test_clap_task.py	78	def test_clap_task_training_forward_returns_loss():
LOW	tests/test_clap_task.py	86	def test_clap_task_eval_forward_keeps_audio_features_key():
LOW	tests/test_clap_task.py	94	def test_clap_task_eval_forward_accepts_text_kwargs():
LOW	tests/test_clap_task.py	102	def test_clap_task_dummy_batch_and_batch_size():
LOW	tests/test_clap_task.py	113	def test_clap_task_accum_loss_maps_audio_features_to_clip_loss():
LOW	tests/test_clap_task.py	123	def test_create_task_dispatches_clap(monkeypatch):
LOW	tests/test_clap_task.py	145	def test_factory_dispatches_audio_config_to_clap(tmp_path, monkeypatch):
LOW	tests/test_clap_task.py	185	def test_evaluate_uses_task_primary_key_for_audio():
LOW	tests/test_clap_task.py	215	def test_audio_checkpoint_retries_only_weights_only_payload_errors(monkeypatch):
LOW	tests/test_clap_task.py	231	def test_audio_checkpoint_does_not_retry_unrelated_unpickle_errors(monkeypatch):
LOW	tests/test_zero_shot_eval.py	26	def test_zero_shot_eval_handles_already_unwrapped_model(monkeypatch):
LOW	tests/test_zero_shot_eval.py	74	def test_zero_shot_eval_unwraps_wrapped_model_once(monkeypatch):
LOW	tests/test_zero_shot_eval.py	122	def test_zero_shot_eval_rejects_non_image_model():
LOW	tests/test_zero_shot_eval.py	143	def test_accuracy_returns_python_floats():
LOW	tests/test_zero_shot_eval.py	160	def test_run_zero_shot_classifier_accepts_naflex_image_dict():
LOW	tests/test_task_compile.py	51	def test_task_compile_keeps_task_methods_and_compiles_train_eval_forward():
LOW	tests/test_task_compile.py	73	def test_task_compile_model_compiles_trainable_module_only():
LOW	tests/test_task_compile.py	83	def test_compiled_train_step_runs_forward_backward_and_optimizer_step():
LOW	tests/test_task_compile.py	102	def test_compiled_train_step_handles_grad_clip():
LOW	tests/test_task_compile.py	123	def test_train_state_counter_restore_prefers_checkpoint_metadata():
LOW	tests/test_task_compile.py	146	def test_train_state_counter_restore_estimates_legacy_checkpoint_values():
LOW	tests/test_task_compile.py	164	def test_tensor_learning_rate_updates_in_place():
LOW	tests/test_audio_checkpointing.py	5	def test_whisper_encoder_uses_gradient_checkpointing(monkeypatch):
LOW	tests/test_audio_checkpointing.py	42	def test_htsat_basic_layer_checkpointing_handles_tuple_output():
LOW	tests/test_eval_task.py	58	def test_get_model_from_raw_task(label, TaskCls, model_name, task_kw):
LOW	tests/test_eval_task.py	67	def test_get_model_from_compiled_task(label, TaskCls, model_name, task_kw):
LOW	tests/test_eval_task.py	77	def test_get_model_from_plain_model(label, TaskCls, model_name, task_kw):
LOW	tests/test_eval_task.py	85	def test_get_model_from_distill_task():
174 more matches not shown…

Decorative Section Separators60 hits · 237 pts

Severity	File	Line	Snippet
MEDIUM	tests/test_task_specific_unit.py	14	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_specific_unit.py	16	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_specific_unit.py	79	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_specific_unit.py	81	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_specific_unit.py	103	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_specific_unit.py	105	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_specific_unit.py	159	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_specific_unit.py	161	# ---------------------------------------------------------------------------
MEDIUM	tests/test_eval_task.py	46	# ──────────────────────────────────────────────────────────────────────
MEDIUM	tests/test_eval_task.py	48	# ──────────────────────────────────────────────────────────────────────
MEDIUM	tests/test_eval_task.py	105	# ──────────────────────────────────────────────────────────────────────
MEDIUM	tests/test_eval_task.py	107	# ──────────────────────────────────────────────────────────────────────
MEDIUM	tests/test_data_csv.py	28	# ---------------------------------------------------------------------------
MEDIUM	tests/test_data_csv.py	30	# ---------------------------------------------------------------------------
MEDIUM	tests/test_data_csv.py	78	# ---------------------------------------------------------------------------
MEDIUM	tests/test_data_csv.py	80	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	18	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	20	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	51	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	53	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	78	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	80	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	110	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	112	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	183	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	185	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	207	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_fsdp_unit.py	209	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_checkpoint.py	226	# ──────────────────────────────────────────────────────────────────────
MEDIUM	tests/test_task_checkpoint.py	228	# ──────────────────────────────────────────────────────────────────────
MEDIUM	tests/test_task_base_unit.py	61	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	63	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	107	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	109	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	150	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	152	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	215	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	217	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	225	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	227	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	249	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	251	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	18	# ---------------------------------------------------------------------------
MEDIUM	tests/test_task_base_unit.py	20	# ---------------------------------------------------------------------------
MEDIUM	src/open_clip/pos_embed.py	6	# --------------------------------------------------------
MEDIUM	src/open_clip/pos_embed.py	8	# --------------------------------------------------------
MEDIUM	src/open_clip/pos_embed.py	14	# --------------------------------------------------------
MEDIUM	src/open_clip/pos_embed.py	19	# --------------------------------------------------------
MEDIUM	src/open_clip/pos_embed.py	70	# --------------------------------------------------------
MEDIUM	src/open_clip/pos_embed.py	74	# --------------------------------------------------------
MEDIUM	src/open_clip/audio/whisper.py	52	# ---------------------------------------------------------------------------
MEDIUM	src/open_clip/audio/whisper.py	54	# ---------------------------------------------------------------------------
MEDIUM	src/open_clip/audio/whisper.py	111	# ---------------------------------------------------------------------------
MEDIUM	src/open_clip/audio/whisper.py	113	# ---------------------------------------------------------------------------
MEDIUM	src/open_clip/audio/whisper.py	148	# ---------------------------------------------------------------------------
MEDIUM	src/open_clip/audio/whisper.py	150	# ---------------------------------------------------------------------------
MEDIUM	src/open_clip/audio/whisper.py	242	# ---------------------------------------------------------------------------
MEDIUM	src/open_clip/audio/whisper.py	244	# ---------------------------------------------------------------------------
MEDIUM	src/open_clip/audio/whisper.py	374	# ---------------------------------------------------------------------------
MEDIUM	src/open_clip/audio/whisper.py	376	# ---------------------------------------------------------------------------

Unused Imports107 hits · 86 pts

Severity	File	Line
LOW	tests/test_factory_task.py	8
LOW	tests/test_factory_task.py	12
LOW	tests/test_task_checkpoint.py	14
LOW	tests/test_task_checkpoint.py	22
LOW	tests/test_audio_data.py	1
LOW	tests/test_training_simple.py	5
LOW	tests/test_wds.py	9
LOW	src/open_clip/zero_shot_classifier.py	3
LOW	src/open_clip/pretrained.py	7
LOW	src/open_clip/pretrained.py	13
LOW	src/open_clip/hf_model.py	13
LOW	src/open_clip/__init__.py	1
LOW	src/open_clip/__init__.py	3
LOW	src/open_clip/__init__.py	4
LOW	src/open_clip/__init__.py	4
LOW	src/open_clip/__init__.py	5
LOW	src/open_clip/__init__.py	5
LOW	src/open_clip/__init__.py	5
LOW	src/open_clip/__init__.py	5
LOW	src/open_clip/__init__.py	5
LOW	src/open_clip/__init__.py	5
LOW	src/open_clip/__init__.py	6
LOW	src/open_clip/__init__.py	6
LOW	src/open_clip/__init__.py	6
LOW	src/open_clip/__init__.py	6
LOW	src/open_clip/__init__.py	7
LOW	src/open_clip/__init__.py	7
LOW	src/open_clip/__init__.py	7
LOW	src/open_clip/__init__.py	8
LOW	src/open_clip/__init__.py	8
LOW	src/open_clip/__init__.py	8
LOW	src/open_clip/__init__.py	8
LOW	src/open_clip/__init__.py	8
LOW	src/open_clip/__init__.py	8
LOW	src/open_clip/__init__.py	8
LOW	src/open_clip/__init__.py	9
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	10
LOW	src/open_clip/__init__.py	13
LOW	src/open_clip/__init__.py	13
LOW	src/open_clip/__init__.py	13
LOW	src/open_clip/__init__.py	13
LOW	src/open_clip/__init__.py	13
LOW	src/open_clip/__init__.py	13
LOW	src/open_clip/__init__.py	13
LOW	src/open_clip/__init__.py	13
LOW	src/open_clip/__init__.py	15
LOW	src/open_clip/__init__.py	15
LOW	src/open_clip/__init__.py	16
LOW	src/open_clip/__init__.py	16
LOW	src/open_clip/__init__.py	16
47 more matches not shown…

Deep Nesting42 hits · 40 pts

Severity	File	Line
LOW	src/open_clip/coca_model.py	247
LOW	src/open_clip/zero_shot_classifier.py	76
LOW	src/open_clip/pretrained.py	812
LOW	src/open_clip/hf_model.py	99
LOW	src/open_clip/convert.py	13
LOW	src/open_clip/convert.py	21
LOW	src/open_clip/push_to_hf_hub.py	209
LOW	src/open_clip/factory.py	44
LOW	src/open_clip/factory.py	259
LOW	src/open_clip/factory.py	684
LOW	src/open_clip/factory.py	899
LOW	src/open_clip/model.py	367
LOW	src/open_clip/tokenizer.py	390
LOW	src/open_clip/loss.py	405
LOW	src/open_clip/utils.py	67
LOW	src/open_clip/transformer.py	958
LOW	src/open_clip/transformer.py	1396
LOW	src/open_clip/transformer.py	188
LOW	src/open_clip/transformer.py	749
LOW	src/open_clip/transformer.py	1478
LOW	src/open_clip/transform.py	367
LOW	src/open_clip/audio/convert.py	76
LOW	src/open_clip/audio/whisper.py	379
LOW	src/open_clip/audio/htsat.py	1158
LOW	src/open_clip/audio/htsat.py	104
LOW	src/open_clip/audio/htsat.py	741
LOW	src/open_clip/audio/htsat.py	1032
LOW	src/open_clip/audio/transform.py	94
LOW	src/open_clip/task/base_task.py	377
LOW	src/open_clip_train/legacy_train.py	79
LOW	src/open_clip_train/legacy_train.py	261
LOW	src/open_clip_train/legacy_main.py	71
LOW	src/open_clip_train/distributed.py	23
LOW	src/open_clip_train/audio_zero_shot.py	266
LOW	src/open_clip_train/train.py	179
LOW	src/open_clip_train/train.py	296
LOW	src/open_clip_train/train.py	449
LOW	src/open_clip_train/zero_shot.py	54
LOW	src/open_clip_train/main.py	90
LOW	src/open_clip_train/profiler.py	131
LOW	src/open_clip_train/data.py	381
LOW	src/open_clip_train/data.py	699

Excessive Try-Catch Wrapping20 hits · 22 pts

Severity	File	Line	Snippet
LOW	src/open_clip/pretrained.py	900	except Exception:
LOW	src/open_clip/pretrained.py	912	except Exception as e:
LOW	src/open_clip/push_to_hf_hub.py	118	except Exception as e:
LOW	src/open_clip/factory.py	370	except Exception as e:
LOW	src/open_clip/factory.py	394	except Exception as e_weights:
LOW	src/open_clip/factory.py	397	except Exception as e_config:
LOW	src/open_clip/factory.py	425	except Exception as e:
LOW	src/open_clip/factory.py	586	except Exception as e:
LOW	src/open_clip/factory.py	614	except Exception as e:
LOW	src/open_clip/factory.py	636	except Exception as e:
LOW	src/open_clip/factory.py	723	except Exception as e:
LOW	src/open_clip/factory.py	739	except Exception as e:
LOW	src/open_clip/tokenizer.py	193	except Exception:
LOW	src/open_clip/timm_model.py	150	except Exception as e:
LOW	src/open_clip/audio/whisper.py	430	except Exception as e:
LOW	src/open_clip/audio/htsat.py	1205	except Exception as exc:
MEDIUM	src/open_clip/audio/htsat.py	1158	def create_htsat_model(audio_cfg, enable_fusion=False, fusion_type="None"):
LOW	src/open_clip_train/file_utils.py	40	except Exception as e:
LOW	src/open_clip_train/profiler.py	228	except Exception as e:
MEDIUM	src/open_clip_train/profiler.py	229	print(f'Error profiling {m}: {e}')

Cross-File Repetition4 hits · 20 pts

Severity	File	Snippet
HIGH	src/open_clip/transformer.py	prune layers not required for specified intermediates.
HIGH	src/open_clip/transformer.py	prune layers not required for specified intermediates.
HIGH	src/open_clip/transformer.py	prune layers not required for specified intermediates.
HIGH	src/open_clip/transformer.py	prune layers not required for specified intermediates.

Redundant / Tautological Comments9 hits · 14 pts

Severity	File	Line	Snippet
LOW	src/open_clip/push_to_hf_hub.py	111	# Check if repo already exists and determine what needs updating
LOW	src/open_clip/factory.py	172	# Check if safetensors or not and load weights accordingly
LOW	src/open_clip/factory.py	355	# Check if the required 'model_cfg' key is present
LOW	src/open_clip/factory.py	417	# Check if `pretrained` is a known tag
LOW	src/open_clip/factory.py	575	# Check if model has the 'visual' attribute
LOW	src/open_clip/factory.py	753	# Check if config determination failed completely (should only be possible if initial schema parsing failed badly)
LOW	src/open_clip/transformer.py	1442	# Check if we have transformer blocks to work with
LOW	src/open_clip/timm_model.py	215	# Check if the underlying timm model has set_input_size method
LOW	src/open_clip_train/distributed.py	134	# Check if init_process_group supports device_id (added in PyTorch ~2.4).

Docstring Block Structure2 hits · 10 pts

Severity	File	Line	Snippet
HIGH	src/open_clip/factory.py	88	Parses a model name string to identify a schema and the remaining identifier. Args: model_name: The mo
HIGH	src/open_clip/factory.py	1142	Creates a contrastive vision-language model from pretrained weights with optional preprocessing transform. Thi

AI Slop Vocabulary3 hits · 9 pts

Severity	File	Line	Snippet
MEDIUM	src/open_clip/factory.py	657	# Absolutely no weights were loaded from any source
MEDIUM	src/open_clip/tokenizer.py	470	# adding lower (for case-sensitive tokenizers) will make it more robust but less sensitive to nuance
MEDIUM	src/open_clip/tokenizer.py	606	# adding lower (for case-sensitive tokenizers) will make it more robust but less sensitive to nuance

Over-Commented Block6 hits · 6 pts

Severity	File	Line	Snippet
LOW	src/open_clip/pos_embed.py	1	# Copyright (c) Meta Platforms, Inc. and affiliates.
LOW	src/open_clip/tokenizer.py	421	self.clean_fn = get_clean_fn(clean)
LOW	src/open_clip/transformer.py	781
LOW	src/open_clip/audio/htsat.py	1121	# output_dict = self.forward_features(x)
LOW	src/open_clip/audio/htsat.py	1141	# framewise_output += d["framewise_output"]
LOW	src/open_clip_train/data.py	541	persistent_workers=args.workers > 0,

Score History

Severity Breakdown

Pattern Findings