✨ Add support for hybrid_property

[van51]

Attempt to solve #299 and add support for hybrid_property properties.

[github-actions]

📝 Docs preview for commit 55c9526 at: https://635fb362686fae1e8384a5e3--sqlmodel.netlify.app

[github-actions]

📝 Docs preview for commit f5a474f at: https://635fb41dc6090b1e80805996--sqlmodel.netlify.app

[van51]

The proposed addition was working correctly for sqlalchemy==1.4.23, but on the latest sqlalchemy==1.4.41 is not working anymore. The error is shown on the checks above. I'll try to pinpoint what has changed.

[github-actions]

📝 Docs preview for commit a7ec9ff at: https://6362568c806a400cb061aacd--sqlmodel.netlify.app

[codecov]

Codecov Report

Base: 98.49% // Head: 98.50% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (1359098) compared to base (ea79c47).
Patch coverage: 100.00% of modified lines in pull request are covered.

❗ Current head 1359098 differs from pull request most recent head 93509eb. Consider uploading reports for the commit 93509eb to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #482   +/-   ##
=======================================
  Coverage   98.49%   98.50%           
=======================================
  Files         185      186    +1     
  Lines        5856     5890   +34     
=======================================
+ Hits         5768     5802   +34     
  Misses         88       88           

Impacted Files	Coverage Δ
tests/conftest.py	100.00% <100.00%> (ø)
tests/test_sqlalchemy_properties.py	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[github-actions]

📝 Docs preview for commit 32b125c at: https://6362577741a93d01dcd99ece--sqlmodel.netlify.app

[github-actions]

📝 Docs preview for commit 1359098 at: https://636258b54b0bfa040aa9faf9--sqlmodel.netlify.app

[github-actions]

📝 Docs preview for commit 93509eb at: https://639ce098ecf1fd03689b400e--sqlmodel.netlify.app

[shepilov-vladislav]

Any plans to solve it?

[van51]

Hi, it has been fixed in the current PR implementation. I had forgotten to add a comment about it.

[shepilov-vladislav]

@van51 Any plans to merge it? I use your commit pip install git+https://github.com/van51/sqlmodel.git#93509eb and can confirm that it works as expected last 2 months. But I can't understand why it's still not merged to trunk 😅

[van51]

@van51 Any plans to merge it? I use your commit pip install git+https://github.com/van51/sqlmodel.git#93509eb and can confirm that it works as expected last 2 months. But I can't understand why it's still not merged to trunk 😅

Hi @shepilov-vladislav , unfortunately I don't have write access to this repository. We have to ping/mention someone from the maintainers.

I am glad to know that it works for you though :)

[shepilov-vladislav]

I see 😢 @tiangolo Can you help with this? What can we do to force this?

[linpan]

I see 😢 @tiangolo Can you help with this? What can we do to force this?

[1219295581]

This hybird_property is cool ,it solved my pain,hope to merge

[felimuno]

works great. hope to merge too.

[kyhorne]

Would love to have this

[Islati]

Bumping this as it really should be pulled & merged.

[PipeKnight]

@tiangolo after new releases with sqlalchemy 2.0 support, maybe it's time to merge this?

[lesleslie]

Willy Wonka: But Charlie, don’t forget what happened to the man who suddenly got everything he always wanted.
Charlie Bucket: What happened?
Willy Wonka: He lived happily ever after.

Please sir, merge and release.

[50Bytes-dev]

Use my PR #801 if you need these features

[marhoy]

@tiangolo Any ETA on this? Would be really appreciated 😊