Skip to content

Fluent workflows #228

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 36 commits into
base: main
Choose a base branch
from
Open

Fluent workflows #228

wants to merge 36 commits into from

Conversation

hpohekar
Copy link

@hpohekar hpohekar commented Sep 2, 2025

@AzeezIsh

Creating PR as discussed with @gyeole

AzeezIsh and others added 30 commits July 28, 2025 11:31
@AzeezIsh
added embedding changes
5fcbe39
@AzeezIsh
functional Go changes
fe9c478
@AzeezIsh
collection changes
c4039da
@AzeezIsh
checkstyle
0211a13
@AzeezIsh
unnecessary logging
76b9739
@AzeezIsh
Merge branch 'main' into hybrid_embedding
328402a
@AzeezIsh
removed style comments, keyword fixing
2073980
@AzeezIsh
Merge branch 'hybrid_embedding' of https://github.com/ansys/aali-flowkit
e6e7c7c 
 into hybrid_embedding
@AzeezIsh
removed internal functions joined with param
c2a47b0
@AzeezIsh
shifted test case to match new function output
8be07da
@AzeezIsh
introduced variadic params for backward compatibility, unified to a s…
f14bf28 
…ingular function
@AzeezIsh
checkstyle
f1940bc
@AzeezIsh
sample one point to determine collection
5513ead
@AzeezIsh
generalist search rather than keyword. too specific for varying colle…
925cfc2 
…ctions
@AzeezIsh
handling set collection types and detect dynamically
4b271a1
@AzeezIsh
added helper functions for dbresponse collection types
cffa8c4
@AzeezIsh
fixed mapping and introduced nil check instead of variadic param
2fe0517
@AzeezIsh
pre-commit files
47a68f3
@AzeezIsh
Merge branch 'main' of https://github.com/ansys/aali-flowkit into hyb…
984c2c4 
…rid_embedding
@AzeezIsh
grpc handled functions incorperation
0dae870
@AzeezIsh
better empty case handling
51dcdc8
@AzeezIsh
include func testing changes
e3ce1ca
@AzeezIsh
direct go function calls don't include grpc layer
f6d3685
@AzeezIsh
improved summary fields focus and text refinement
7872370
@AzeezIsh
more checkstyle
3168211
@AzeezIsh
moved mapping logic and dbresponses to a seperate PR
806991c
@AzeezIsh
Merge branch 'main' of https://github.com/ansys/aali-flowkit into hyb…
9b21693 
…rid_embedding
@AzeezIsh
code_gen elements changes
a48fd0c
@AzeezIsh
added fluent wrapper in flowkit
e684cfe
@AzeezIsh
Merge branch 'main' of https://github.com/ansys/aali-flowkit into flu…
40d0097 
…ent_workflows
@hpohekar hpohekar requested a review from a team as a code owner September 2, 2025 11:54
@hpohekar hpohekar requested a review from AzeezIsh September 2, 2025 11:54
rcrisanti added a commit that referenced this pull request Sep 2, 2025
@rcrisanti
feat: add fluent workflow
25e7d93 
This is equivalent to #228 but
with the data extraction parts removed. We will revisit that in a
separate PR.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AzeezIsh AzeezIsh Awaiting requested review from AzeezIsh

@FelixKuhnAnsys FelixKuhnAnsys Awaiting requested review from FelixKuhnAnsys

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hpohekar @AzeezIsh @gyeole