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.

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

projects3.0 evolution from projects2.0 #216

Open
atherdon opened this issue Oct 21, 2019 · 38 comments
Open

projects3.0 evolution from projects2.0 #216

atherdon opened this issue Oct 21, 2019 · 38 comments

Comments

@atherdon
Copy link
Collaborator

atherdon commented Oct 21, 2019

Main purpose is to go away from old and stupid es5 version ~~ projects2.0
because we keeping all projects in one folder just for now, so our team
members were less confused.
I think one of our goals is to create a microservice with CSV parser that
will run on Heroku so anyone can parse their stuff when he wants.
And clean up and code upgrade will really help and our project will go to a
next evolutionary step

You'll easily get used to it. I tried to reduce every problem that we have
with projects2.0, so you'll get into it pretty quick I guess.
Because we just cleaning up - not creating from scratch :)

Originally posted by @atherdon in #208 (comment)

@atherdon
Copy link
Collaborator Author

Now I'm updating generator module at our csv-parser module and will release a new version on npm.
then we'll be able to upgrade it on projects3.0

@atherdon
Copy link
Collaborator Author

@EbrahimKreem @sibasish14 @saphal1998 I think this video will be pretty useful for you.
https://www.loom.com/share/03c161e2231e45469075bad1e83cdf07

Btw, @vadim9999 do you interested to jump into it? if yes - please buzz me and we'll see what we need to do

@atherdon
Copy link
Collaborator Author

Hi everyone, i have one idea about what simple move we can do.
as we have csv datasets at projects3.0 would it be wise to remove the same data from projects2.0?
it can be a problem for fixing our small bugs, but it can move slowly as forward ;)

@atherdon
Copy link
Collaborator Author

@sibasish14 did you watch that video, right?
i made some changes recently, can you sync your fork with changes from master and check how projects3.0 works inside?
I can give you a proper task, but want you to explore it and try to find something that you can do - any simple thing is ok

@atherdon atherdon pinned this issue Oct 28, 2019
@sibasish14
Copy link
Contributor

@atherdon Yes, I went through the video. I think it would be nice if you assigned me a task. That way I would proceed as per the goals. Also, I think I will understand the codebase better that way.

@atherdon
Copy link
Collaborator Author

atherdon commented Oct 29, 2019

@sibasish14 As I think @EbrahimKreem is working with FAO dataset right now - the easiest way will be to separate projects between few people. l think you should jump into projects3.0/FoodComposition and try to launch it. i'm sure it will generate some errors. and our goal will be to move FoodComposition to work from proj2.0 to project3.0 structure. tell me if this make sence

@sibasish14
Copy link
Contributor

Sorry I am a bit late.
Can you please explain how can I launch it?

@atherdon
Copy link
Collaborator Author

no problem at all. if i get your question correctly, you should go into projects3.0/FoodCom..
then run yarn install - it will install fresh packages and start a build process first time. I assume that build went wrong and show you some errors. you can put them here with screenshot or try to solve it by yourself by comparing code inside of that project with code related to other 3.0 projects

@atherdon
Copy link
Collaborator Author

as we made all datasets code at 2.0 working in the same logic/way - now our goal to make it work in more proper, but similar way. don't worry - it's easy

@sibasish14
Copy link
Contributor

I am getting the following output after doing npm install in projects3.0/FoodComposition

Screenshot from 2019-11-02 01-58-23
Screenshot from 2019-11-02 01-58-25

@atherdon
Copy link
Collaborator Author

atherdon commented Nov 1, 2019 via email

@sibasish14
Copy link
Contributor

I did yarn build:
Screenshot from 2019-11-03 12-19-45

@atherdon
Copy link
Collaborator Author

atherdon commented Nov 3, 2019

ok, then go to fao project and see what Ebrahim did there in order to make parser work, i think there is no huge chnages will be necessary @sibasish14

@atherdon
Copy link
Collaborator Author

atherdon commented Nov 3, 2019

@sibasish14 please check this video https://www.loom.com/share/606346ba1b814243a5a1ae589e4468e9

@sibasish14
Copy link
Contributor

I was solving as per the changes made in the FAO. I copied the contents of the index.js file from the FAO folder to the FoodComposition folder. When I tried to commit, I received a lot of linting errors:
Screenshot from 2019-11-08 09-52-13
I'll make a PR once I fix these errors.

@atherdon
Copy link
Collaborator Author

atherdon commented Nov 8, 2019

cool - @sibasish14 - for constants - you can just comment them. i want to improve our system while ago with this contants, but don't have time to perfect it.
Btw, I think we'll have similar issues with other projects as well

@sibasish14
Copy link
Contributor

I fixed most of them. Can you help me fix the rest? I don't understand how to solve the rest.
Screenshot from 2019-11-08 23-19-46

@atherdon
Copy link
Collaborator Author

atherdon commented Nov 8, 2019

can you compare code at src folder with code from other projects as well?

@atherdon
Copy link
Collaborator Author

atherdon commented Nov 8, 2019

maybe it can help a bit.

what do you think about utils.js file inside of this project? are we using metohds from it? if no - just comment them. all of them

@atherdon
Copy link
Collaborator Author

atherdon commented Nov 8, 2019

about file csv_parser - i think it's just an old rudiment - we don't need it to make everything work - because our separated module published at npm and can be installed in a correct way

@sibasish14
Copy link
Contributor

sibasish14 commented Nov 10, 2019

Now there is only one error left:
/home/sibasish/GroceriStar/food-datasets-csv-parser/projects3.0/FoodComposition/src/index.js

44:13 error Unable to resolve path to module '../../../package/dist/index.cjs' import/no-unresolved

I figured that there's no dist folder in the package directory. How do I fix it?

@ebrahim-2
Copy link
Contributor

ebrahim-2 commented Nov 10, 2019

rather than referencing the dist folder you can use the npm module ex:
const { findCsvFiles, parseCsv, csvToJson } = require("@groceristar/food-dataset-csv-parser")

@ebrahim-2
Copy link
Contributor

I figured that there's no dist folder in the package directory. How do I fix it?

if you want to make a dist folder for development purposes:
1- cd package
2- npm run dev

@sibasish14 sibasish14 mentioned this issue Nov 10, 2019
7 tasks
@atherdon
Copy link
Collaborator Author

atherdon commented Nov 10, 2019 via email

@hanroro
Copy link
Contributor

hanroro commented Nov 11, 2019

@atherdon Should I work on Restaurants folder for the same thing?

@atherdon
Copy link
Collaborator Author

@hanroro good idea - pursue it!

@hanroro
Copy link
Contributor

hanroro commented Nov 14, 2019

@atherdon Just submitted two PR for USFA and Restaurants. Please have a check.

@atherdon
Copy link
Collaborator Author

atherdon commented Nov 14, 2019 via email

@hanroro
Copy link
Contributor

hanroro commented Nov 14, 2019

does restaurants dataset parsed and generating json under projects3.0 folder? if yes- then we can kill restaurants folder at projects2.0 - correct? if yes - please do - let's clean it up a bit

From the codes I added. I think it do parse, but I do not know where the output is located and it seems there is not any file created. So how can i check that?

@atherdon
Copy link
Collaborator Author

atherdon commented Nov 14, 2019 via email

@ebrahim-2
Copy link
Contributor

ebrahim-2 commented Nov 14, 2019

that's why I opened #262 @hanroro we will fix it soon at the mean time you can test it manually ex:

cd projects3.0/folderName/src
node index.js

@atherdon
Copy link
Collaborator Author

guys, i made some changes in our repo - please sync your forks with master branch

@atherdon
Copy link
Collaborator Author

@sainiyash2903 jump here- into the action :)

@yashaswisaini
Copy link

@atherdon Now a days, my examinations are going on. So, I'm little bit busy . I will be in action from 9 December.

@atherdon
Copy link
Collaborator Author

sure sure

@atherdon
Copy link
Collaborator Author

atherdon commented Dec 8, 2019

@sainiyash2903 jump here. we'll clean up this module with you

@yashaswisaini
Copy link

@atherdon I need your guidance, in my work . I don't know what to do . Everything is new for me . I'm learning React side by side.

@atherdon
Copy link
Collaborator Author

atherdon commented Dec 17, 2019

@sainiyash2903 sure, but part of my goal here is to see how you "act" in the open ocean. I mean yes I can guide you - but did you go through our conversations and explore it on your own?
Again - this code is simple - you just need to break this first wall :)

I always have this example: most of the peoples cant do 30 pull-ups - because it's hard and they not trained well. But what if someone will stand behind you with a rifle. and if you don't do it - they will shoot. I assure you - people will have better results in the second case.
So I'm waiting for a more detailed question from you. Or just go at code and try to break something and show it to us :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants