-
Notifications
You must be signed in to change notification settings - Fork 122
Pull requests: google/gin-config
Author
Label
Projects
Milestones
Reviews
Assignee
Sort
Pull requests list
Register dynamically imported configurables under their real module
ParseContext._register built the configurable's module name from
ImportStatement.partial_path(), which for an aliased import substitutes
the alias into the path (import gin.testdata.dynamic_registration as dr
produced 'gin.testdata.dr.Class'). The registry selector then didn't
match the real module, so gin.get_bindings()/get_configurable() failed
for modules that were only ever imported under an alias, and the same
configurable could be registered twice under different names across
files. Use module_path() so the registered name always reflects the
real module.
#221
opened Sep 12, 2026 by
rootkiller6788
Loading…
Add unit tests for StrEnum constants registration in constants_from_enum
#218
opened Aug 31, 2026 by
AbhiPra24
Loading…
Initial checkin of
gin.Config class and required utilities.
cla: yes
#146
opened Jul 30, 2021 by
copybara-service
Bot
Loading…
Fix comments about whitelist of configurable()
cla: yes
#65
opened Apr 9, 2020 by
emailweixu
Loading…
Add module for sklearn external configurables
cla: yes
#49
opened Jan 26, 2020 by
gmrukwa
Contributor
Loading…
ProTip!
Mix and match filters to narrow down what you’re looking for.