⚗️🎯 Fix norm limit regularizers hpo defaults

[mberr]

Fix #1273

[cthoyt]

Maybe there’s a more automated way to check all of these components have HPO defaults? Do we have a class resolved for it?

[mberr]

Maybe there’s a more automated way to check all of these components have HPO defaults? Do we have a class resolved for it?

I guess we have, at least for some of them 🤔

These ones also look like we may need to test some things:

pykeen/src/pykeen/hpo/hpo.py

Line 213 in cb226fc

default_kwargs_ranges=lr_schedulers_hpo_defaults[self.lr_scheduler],

pykeen/src/pykeen/hpo/hpo.py

Line 203 in cb226fc

default_kwargs_ranges=optimizers_hpo_defaults[self.optimizer],

[mberr]

It should also suffice to only have one of kwargs_ranges and default_kwargs_ranges be non-empty

pykeen/src/pykeen/hpo/hpo.py

Lines 879 to 895 in cb226fc

	def _get_kwargs(
	trial: Trial,
	prefix: str,
	*,
	default_kwargs_ranges: Mapping[str, Any],
	kwargs: Optional[Mapping[str, Any]] = None,
	kwargs_ranges: Optional[Mapping[str, Any]] = None,
	) -> Mapping[str, Any]:
	_kwargs_ranges = dict(default_kwargs_ranges)
	if kwargs_ranges is not None:
	_kwargs_ranges.update(kwargs_ranges)
	return suggest_kwargs(
	trial=trial,
	prefix=prefix,
	kwargs_ranges=_kwargs_ranges,
	kwargs=kwargs,
	)