Skip to content

I added multiline input feature. #68

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 38 commits into
base: master
Choose a base branch
from
Open

I added multiline input feature. #68

wants to merge 38 commits into from

Conversation

LilKeyboard
Copy link

I didn't have much time to test it well but after few checks it look promising

radekrepo and others added 30 commits February 11, 2025 10:10
@radekrepo
upgrade
682e9c0
@radekrepo
Merge remote-tracking branch 'remotes/upstream/dev' into generalize_r…
8dff529 
…ag_creation
@radekrepo
custom exception message
9834545
@radekrepo
response to comments
aae7de3
@radekrepo
remove examples from docstrings. Remove 'if not workdir' from if __na…
838c88f 
…me__ == main
@radekrepo
fix for 'if not work_dir
41b73dc
@radekrepo
rename file_extension_constraint to code_extensions
d2f8a67
@radekrepo
make default file extensions as the baseline option
62b86c4
@radekrepo
improve function names
881f300
@radekrepo
debug
d02ea95
@radekrepo
remove use of work_dir checks and missing environment variable error.
5630559
@Grigorij-Dudnik
Merge pull request Grigorij-Dudnik#56 from radekrepo/generalize_rag_c…
ab9fdaf 
…reation

upgrade of write_descriptions.py
@Grigorij-Dudnik
numpy in req
d0d6093
@Grigorij-Dudnik
code splitter added
5660f79
@Grigorij-Dudnik
code splitter added
b65614a
@Grigorij-Dudnik
Merge pull request Grigorij-Dudnik#64 from Grigorij-Dudnik/master
53e6de3 
fix merge
@Grigorij-Dudnik
PR tempalte
13f5ed1
@Grigorij-Dudnik
Merge branch 'dev' of https://github.com/Grigorij-Dudnik/Clean-Coder-AI…
804f448 
… into dev
@radekrepo
restore previous prompt
c17adcb
@radekrepo
Merge remote-tracking branch 'remotes/upstream/dev' into generalize_r…
fa54328 
…ag_creation
@Grigorij-Dudnik
Merge pull request Grigorij-Dudnik#65 from radekrepo/generalize_rag_c…
1e9938c 
…reation

restore earlier prompt version for write_descriptions.py
@Grigorij-Dudnik
write descriptions - old code restored
b8352ac
@Grigorij-Dudnik
write chunk descriptions started to work
1e0c448
@Grigorij-Dudnik
write chunk descriptions started to work
17dbdb3
@Grigorij-Dudnik
repairing chunk describing
05d5b86
@Grigorij-Dudnik
researcher improved
d522b67
@Grigorij-Dudnik
updates
28ff399
@Grigorij-Dudnik
updating semantic retrieval
7ea944e
@Grigorij-Dudnik
working on binary ranker
c4dca44
@Grigorij-Dudnik
binary ranker for semantic retrieval done
de89d42
@Grigorij-Dudnik
Copy link
Owner

Hey @LilKeyboard. Thank you for your PR! Appreciate your work on that feature, witch is needed for Clean Coder's user.

I have few coments for the PR:

  • How do you think, maybe it is possible multiline input to be active by default? Just to make every input multiline.
  • On windows, it exiting from multiline input does not work as intended. Ctrl+Z does not exit it. Have you access to windows computer to test it?
  • Please put a time in test. In Clean Coder, we intending to have only the best-quality code. It's possible, you can found and solve issues faster.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@LilKeyboard @Grigorij-Dudnik @radekrepo