Skip to content

added backtracking/backreference#10

Open
dekiesel wants to merge 3 commits intorhasspy:masterfrom
dekiesel:backtracking
Open

added backtracking/backreference#10
dekiesel wants to merge 3 commits intorhasspy:masterfrom
dekiesel:backtracking

Conversation

@dekiesel
Copy link

@dekiesel dekiesel commented Oct 28, 2024

This PR should close #4 .

It definitely takes away from the beauty of the recursion, but it works

Please let me know if/how I can improve this PR and thank you for all your fantastic work!

What it does: A dict is used to note what has been changed in an input template. After a sentence has been created the substitutions of the input sentence are used to change the output sentence accordingly.

@synesthesiam
Copy link
Member

Thanks! I'll take a look this week.

@synesthesiam synesthesiam self-assigned this Oct 28, 2024
@synesthesiam synesthesiam added the enhancement New feature or request label Oct 28, 2024
@dekiesel
Copy link
Author

Is there anything I can do to help speed things up

@synesthesiam
Copy link
Member

I have this added in a separate code base along with some extra features. I'll probably pull in that code instead, but I want to give you credit for the idea and your work 🙂

@dekiesel
Copy link
Author

Thank you very much for your work! Giving me (part) of the credit is a nice thought but really not necessary. I am just happy to help.

@dekiesel
Copy link
Author

Any news on this? I am afraid my sentences will stop working once you pull in your code, so I've hold off on using these changes in "production".

@dekiesel
Copy link
Author

This repo hasn't seen any development, is there another project we should use now? I read that some parts of wyoming are getting deprecated? But I am not sure if this exact part of the project is impacted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature request: In/out replacement backreference

2 participants