Skip to content
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

Spiking out force modern docker compose instead of docker-compose #100

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Spiking out force modern docker compose instead of docker-compose #100

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
0e0a4c0
why do we have a magic git line just hanging out?
epugh Aug 26, 2024
9084dee
when this line is hit, at least for me on osx, it just sits there in …
epugh Aug 26, 2024
06e5896
if we are using "python3" then let's use "python3"!
epugh Aug 26, 2024
e144663
sklearn is gone, so remove, and bump a few dependenciest to make scik…
epugh Aug 26, 2024
7d9d8e7
Merge branch 'main' into force_modern_docker_compose
epugh Aug 26, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ This will run jupyter and all search engines in Docker containers. Check that ea
- Jupyter: [localhost:8888](localhost:8888)

## You want to build your own LTR notebooks
git
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that is acting as a section header in this context.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
maybe some other formatting needed then?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, that's weird, it doesn't belong


Follow these steps if you want to do more serious work with the notebooks. For example, if you want to build a demo with your work's data or something you want to preserve later.

### Run your search engine with Docker
Expand Down
5 changes: 2 additions & 3 deletions requirements.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ ipython-genutils==0.2.0
ipywidgets==7.6.5
jedi==0.18.1
Jinja2==3.0.3
joblib==1.1.1
joblib==1.2.0
jsonschema==4.4.0
jupyter
jupyter-client==7.4.4
Expand Down Expand Up @@ -60,12 +60,11 @@ pyzmq==25.1.1
qtconsole==5.2.2
requests==2.27.1
retrying==1.3.3
scikit-learn==1.3.0
scikit-learn==1.5.1
scipy==1.10.1
seaborn==0.11.2
Send2Trash==1.8.0
six==1.16.0
#sklearn==1.3.0
SQLAlchemy==1.3.24
terminado==0.13.1
testpath==0.5.0
Expand Down
6 changes: 4 additions & 2 deletions tests/test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,7 @@ test_http_service 8983 Solr
# Rebuild venv
python3 -m venv tests_venv
source tests_venv/bin/activate
python -m pip install -U pip wheel setuptools
python3 -m pip install -U pip wheel setuptools
pip install -r requirements.txt

echo "================================================"
Expand All @@ -179,7 +179,9 @@ else
echo "================================================"
echo "> POOP! Tests Failed 💩 For:"
fi
git log -n 1
git_log=$(git log -n 1)
echo "Latest commit:"
echo "$git_log"
echo "================================================"
echo " ==============================================="
echo " HELLO-LTR TEST DETAILS"
Expand Down