-
Notifications
You must be signed in to change notification settings - Fork 65
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
Quick fixes that need to be done #105
Comments
I think my implementation can be used, with regards to point 3. |
@gastonche great! Do you have time to make a PR for that? |
I am not yet done with the issue i was working on but sure i can make time for this. I just need directions on how to go about it here |
Ok, work on WebMemex is more important. This is the file where the capturing process starts: https://github.com/WorldBrain/Research-Engine/blob/master/src/js/content.js |
Okay I'll set it up and quickly make a PR since i will be faster and have implemented it already. |
@oliversauter In which js file exactly are you handling the downloads? |
@Chaitya62 Here is the file and the function that starts it all: https://github.com/WorldBrain/Research-Engine/blob/master/src/js-browserify/download_urls.js#L169 |
@oliversauter Hey I have figured out a different approach wanted to ask you before I implement it |
Thanks @Chaitya62 for doing this research. This will be helpful for the WebMemex implementation of the download process. But we need a more general approach for this, because there are many cases where the download is interrupted or stops. |
@oliversauter Okay I ll find someother approach can you suggest any method so that I can reproduce the error? and also the master branch has the latest code right ? |
Yes, the master branch has the latest code. I could not reproduce the error so far, but heard from a lot of users that it happened. |
thanks to @gastonche and his PR #111 we could get rid of the webapp problem |
So there are few things that need to be done in order to have a more stable version:
Sorry @arpitgogia that we have to undo your work here, but we experienced PouchDB to be way to slow. Be aware that there are also changes I made
The download process oftentimes stops at pages that have Virus warnings or other kind of interruptions. We need to make sure that there is a time out that skips a page in case it takes more than, say 1 minute to process. (@Chaitya62)
Fix the problem that we also experience with the WebMemex, that with web apps, only the first visited page is really indexed.
Maybe we can take parts of your implementation @gastonche ?
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: