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

Use AniDB's Title Dump/Fuzzy Name Searching #11

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

Conversation

XVicarious
Copy link
Owner

Utilize the title dump from AniDB instead of the 3rd party website for finding the ID. Also use fuzzy searching for the title so we can get good results.

Brian Maurer added 30 commits November 6, 2018 15:17
that used the id field
use 'lxml-xml' instead of 'lxml' for parser in get_soup
md5digest initialized to reduce complexity of the line
increase timed limiter to 3 seconds
xml_cache_path is now a class variable
in __load_xml_to_database change == to > to actually skip already added anime
start to rewrite __load_xml_to_database, its shit
rewrite by_name to use the database, still not up to my standards
linted some stuff
updated tox.ini
Oppai420 added 19 commits November 27, 2018 19:48
Initialize AnidbParser with a session and an anime
add _get_anime to set the parser's anime
remove session from parse
use self.series
remove future imports (see README.md)

anidb_parsing_interface.py:
remove unneeded imports
make anime_seasons a list
fix _get_anime_season to actually return a season
use self.session
add _get_list_tag to easily parse certain AniDB tags

anidb_search.py:
fix stupid typo of 'language'
calculate if cache is expired and use that in the if to reduce complexity
add last_lookup to potentially speed up searches
remove by_name as it is no longer used anywhere

api_anidb.py:
remove future imports
minor changes to Anime.expired and Anime.__repr__

anidb_cache.py:
remove future imports
…he anime, it is closed on __del__, which only works on Python 3.4+
lookup has a session again
fix order of variables in AnidbParser.is_banned
wrap most of AnidbParser.parse in a session.no_flush to prevent
autoflushing
muss up api_anidb and the schema, does it work? probably
@XVicarious
Copy link
Owner Author

I'm losing steam. My motivation has plummeted since I was banned from the API. Not sure if I will get unbanned. I went on the AniDB IRC channel and was warmly greeted by their saint of an admin.. Apparently AnidbParser was requesting the same resource multiple times in quick succession. My logs don't show that, but apparently theirs do. I'm not entirely sure that it is possible that the script actually did this, as I check for the existance of the cache file before I request anything. I think this was about a month ago. I added some checks to my personal config file for Flexget to log any and all requests that it made, and adjusted some code. I have been banned from the API for at least a week. I'll wait a few weeks and see what happens.

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.

None yet

1 participant