search

wvsu-cict-code / thesis-package

College's guidelines in packaging your thesis application.
2 stars 4 forks source link

Carbonilla et. al. Package Evaluation #27

Closed cictapps closed 1 year ago

cictapps commented 1 year ago

Title: Syrene: Chatbot Assisting Mental Health Professionals in Dealing with Patients/Clients Using Machine Learning

Members: Gene Caleb C. Carbonilla, Xyphrus Von Keith V. Caguan, Jirah Kate C. Solano, Joshreen D. Reyes

geniusqwerty12 commented 1 year ago

Checked the project (client, admin and server) and tried the functionalities of the system, all is working with little to no issues.

As for the user manual, here are my feedbacks:

  1. On the System requirements section, add the software needed to run the system, so far, I installed: node python visual studio code

  2. Installation Step 2, missing venv activate, and instead of running the command stated on the manual, you can just run: pip install -r requirements.txt

  3. Installation on the client and admin, please clarify the steps for each app.

  4. Add a little bit of content on the back cover.

  5. Please put the server, client and admin on the server on a folder called "src" as stated on the thesis package template for web apps. Once done, update the steps that navigates the user to each directory on the manual.

@cictapps

joshierin commented 1 year ago

Hi Sir @geniusqwerty12 . Good evening. Thank you for the feedback.

Have already implemented revisions in the user manual:

  1. Added the software that needed to run the system in the "System Requirements".
  2. Added a line of command in the 1st step of "Installation" in order to activate the virtual environment. 
    $ . venv/Scripts/activate

    and have changed the line of command in the 2nd step to:

    $ (venv) pip install –r requirements.txt
  3. Have clarified the steps of each app for the installation of client and admin.
  4. Added a little bit of design/content in the back cover. (Note: have also changed some design in our front cover.)
  5. Had created src folder and put the server, admin and client inside the src folder. Lastly, have updated the steps that navigates the user to each directory on the manual.

Uploaded both .pub and .pdf of the user manual in the doc folder. Thank you!

geniusqwerty12 commented 1 year ago

All of the changes above are present in the manuscript. One minor issue on the Deployment section, before running the command "npm start", please include the command "npm install" for installing the npm dependencies. image

joshierin commented 1 year ago

Hello Sir @geniusqwerty12.

We have added the "INSTALLATION OF DEPENDENCIES IN FRONTEND" before the "DEPLOYMENT OF FRONTEND" section and added a line of command of:

npm install

in our user manual. Thank you!

geniusqwerty12 commented 1 year ago

Good day @joshierin, the changes are present in the user manual. Please upload the soft copy of your manuscript once it is ready for hardbinding. @cictapps changing the status to for certification to process their certificate in advance.

joshierin commented 1 year ago

Thank you Sirs @geniusqwerty12 and @cictapps for checking and certifying in advance in our thesis package. God bless!

cictapps commented 1 year ago

We have sent the completion certificate. Congratulations!