Andrew Truong

Results 54 comments of Andrew Truong

My initial thought is this can be supported without changing the core factories by inheriting from `Use` and having a count there. Something like ``` class Sequence(Use): def __init__(self) ->...

This would be great to support. This is probably a limitation of Python typing as does not have the ability to extract partial args in this case

Hi @MRigal , This isn't explicitly supported currently. Probably worth considering with `Traits` or `Params` feature which was mentioned elsewhere. A workaround could be overriding `process_kwargs` to avoid multiple calls...

Sorry for the slow reply! I can't find for conversation for these terms now but believe it comes from `factoryboy`. Based on their docs, this is probably closer to [Param](https://factoryboy.readthedocs.io/en/stable/reference.html#parameters)...

Great. Agree naming should be consistent with `factoryboy` on this. Main parts need to change are list [here](https://github.com/litestar-org/polyfactory/issues/604#issuecomment-2496097135) for traits/params but happy to provide any more info or guidance :)

Hi @leverage-analytics , I've pushed a fix for this. Does this issue still occur after syncing with main?

I think this is hard to support as alias may be anything. I think want to push on with v3 and support this more robustly with https://github.com/litestar-org/polyfactory/issues/717

Can the above functionality be achieved by overriding `build` and `process_kwargs` directly?

@iloveitaly changes and tests look good. Happy to merge once conflicts are resolved and documentation is added

Can `should_use_default_value` be overriden at the factory level instead? If always use a default for this type then should handle this case https://github.com/litestar-org/polyfactory/blob/main/polyfactory/factories/base.py#L951-L963