Warning from BeautifulSoup

[srghma]

/usr/lib/python3.4/site-packages/bs4/__init__.py:166: UserWarning: No parser was explicitly specified, so I'm using the best available HTML parser for this system ("lxml"). This usually isn't a problem, but if you run this code on another system, or in a different virtual environment, it may use a different parser and behave differently.

To get rid of this warning, change this:

 BeautifulSoup([your markup])

to this:

 BeautifulSoup([your markup], "lxml")

  markup_type=markup_type))```

[kkyaw88]

I also have the same problem. Not find out yet the solution.

[wenbowang]

I just came across the same error and here is my solution on MacOS:
edit the file '/Library/Python/2.7/site-packages/PyDictionary/utils.py'
change 'return BeautifulSoup(requests.get(url).text)'
to 'return BeautifulSoup(requests.get(url).text, 'html.parser')'.

Then, it will work. I got inspired by this link.

[yang1992]

BeautifulSoup(html_doc,"lxml")

[geekpradd]

Can anyone open up a pull request with the above fixed?
I'll merge it. I'm currently not participating in open source development (doing algorithmic competitive programming instead) so any help would be appreciated.

[bunchesofdonald]

I created a PR to fix this here: #3

[srghma]

@geekpradd , as maintainer can you upload fixed version to pip and I'll close issue?

[AndreiRegiani]

@geekpradd , as maintainer can you upload fixed version to pip and I'll close issue?

+1 would be nice to have pip version updated, currently has the annoying warning...

[renweibo]

+1 for this fix

[srghma]

@renweibo , you can use @bunchesofdonald fork with pip directly

[renweibo]

@BjornMelgaard Thanks for the reply. Installing directly from github branch failed for there is no setup.py as a normal pypi package has.

It's better to update pypi also.

Even I manually update the util.py with the patch, it still failed in python 3.6 environment.

[srghma]

@renweibo lol

[timabbott]

We had the same problem with a missing setup.py. @geekpradd can you do a release in pypi with this fix? It should be trivial.