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

Added save function (UNTESTED) #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pandalion98
Copy link

Utilizes JSON. Might have to double-check the dependencies. Might even need to re-do the load/save function. Should check if file exists and is writable.

Utilizes JSON. Might have to double-check the dependencies. Might even need to re-do the load/save function. Should check if file exists and is writable.
@parmentf
Copy link
Owner

I planned to use file-concept-network in order to adapt the save/load functions to the kind of concept network used.

Unfortunately I did not yet modify node-ector to make it asynchronous (as file-concept-network and mongo-concept-network are asynchronous).
See #2.

Although the solution you propose works with the current ector, I prefer make ector able to use different kinds of concept networks...

Thanks for sharing your work (and interest), it's often the only way for me to be sufficient motivated to code again ;)

@pandalion98
Copy link
Author

Yeah. A DB-based solution would really work better.

@Unbr8kbal
Copy link

Hi guys,

Currently i'm hooking up express with mongodb, should i pull request that in?

Thanks

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

Successfully merging this pull request may close these issues.

3 participants