httpimport icon indicating copy to clipboard operation
httpimport copied to clipboard

Load a python script that contains a main function

Open dvmorris opened this issue 2 years ago • 14 comments

I want to load this file and call the main function: https://raw.githubusercontent.com/dmlc/xgboost/d3f06467797acd61e80e87be38fb9812a016a64e/demo/guide-python/categorical.py

How do I do this? I see your example that loads a file from a gist that contains a single function, but I can't get this technique to work with the file above.

dvmorris avatar Nov 01 '23 14:11 dvmorris

Hello @dvmorris Can you please respond with what you've tried so far?

operatorequals avatar Nov 01 '23 16:11 operatorequals

This example works:

import httpimport

url = "https://gist.githubusercontent.com/operatorequals/ee5049677e7bbc97af2941d1d3f04ace/raw/e55fa867d3fb350f70b2897bb415f410027dd7e4"

with httpimport.remote_repo(url):
  import hello

hello.hello()
# Hello world

This works:

$ pip install xgboost_ray

from xgboost_ray.examples import simple
simple.main(1, 4)

This does not work:

import httpimport

url = "https://raw.githubusercontent.com/ray-project/xgboost_ray/master/xgboost_ray/examples/simple.py"

with httpimport.remote_repo(url):
  import simple

simple.main()

ModuleNotFoundError: No module named 'simple'

dvmorris avatar Nov 01 '23 18:11 dvmorris

I believe that removing the filename from the url of the non-working example will fix it as well. The import statement looks for the file itself.

operatorequals avatar Nov 01 '23 18:11 operatorequals

import httpimport

url = "https://raw.githubusercontent.com/ray-project/xgboost_ray/master/xgboost_ray/examples/"

with httpimport.remote_repo(url):
  import simple

simple.main(1, 4)
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
[<ipython-input-3-c2511d889ed5>](https://localhost:8080/#) in <cell line: 17>()
     15   import simple
     16 
---> 17 simple.main(1, 4)

AttributeError: module 'simple' has no attribute 'main'

dvmorris avatar Nov 02 '23 15:11 dvmorris

Does this test assume that xboost_ray is installed on the system, and you're just trying to import it via a remote script?

rkbennett avatar Nov 07 '23 20:11 rkbennett

Yes, the issue with this particular use case is that this pip package doesn't include the examples in it.

On Tue, Nov 7, 2023, 2:08 PM rkbennett @.***> wrote:

Does this test assume that xboost_ray is installed on the system, and you're just trying to import it via a remote script?

— Reply to this email directly, view it on GitHub https://github.com/operatorequals/httpimport/issues/59#issuecomment-1799840808, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABNPDLLEYCOM3YWLOWPOFDYDKIK3AVCNFSM6AAAAAA6ZNB36GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJZHA2DAOBQHA . You are receiving this because you were mentioned.Message ID: @.***>

dvmorris avatar Nov 08 '23 13:11 dvmorris

So I was looking into this a bit, and I think xboost_ray uses some odd python files. Saw a bunch of pyx in my testing. Haven't had time to look into it more though.

rkbennett avatar Nov 08 '23 19:11 rkbennett

Traceback (most recent call last):
  File "c:\Users\franc\OneDrive\Escritorio\ti.py", line 6, in <module>
    import hello
ModuleNotFoundError: No module named 'hello'

boludoz avatar Nov 13 '23 04:11 boludoz

Traceback (most recent call last):
  File "c:\Users\franc\OneDrive\Escritorio\ti.py", line 6, in <module>
    import hello
ModuleNotFoundError: No module named 'hello'

Can you paste your code in here?

rkbennett avatar Nov 13 '23 13:11 rkbennett

Yes, the issue with this particular use case is that this pip package doesn't include the examples in it. On Tue, Nov 7, 2023, 2:08 PM rkbennett @.> wrote: Does this test assume that xboost_ray is installed on the system, and you're just trying to import it via a remote script? — Reply to this email directly, view it on GitHub <#59 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABNPDLLEYCOM3YWLOWPOFDYDKIK3AVCNFSM6AAAAAA6ZNB36GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJZHA2DAOBQHA . You are receiving this because you were mentioned.Message ID: @.>

Are you running this on linux? I suppose I should have started with that question

rkbennett avatar Nov 14 '23 16:11 rkbennett

@dvmorris I did some testing with another tool and I don't think there is actually an issue with being unable to import packages with main functions. I think it's likely a different issue which is that when the import happens it's missing a dependency (likely ray[train] or sklearn) which is causing a failure in the import, this then cascades back to an actual issue in this project, which is that when an error happens during import with httpimport it silently continues and you end up with broken packages showing imported, but they are never fully imported. If you really want to test this theory you could try my code from my PR. Otherwise, you can try it with my od_import project. Testing there it definitely has a main function

import od_import
config = {'type':'pastebin', 'visibility': 'unlisted', 'paste_key': 'wgkj6xLj', "module": "simple"}
with od_import.remote_source('https://pastebin.com', config=config):
  import simple

Having said that, in my testing, while I do get simple to import and it has a main function, when you attempt to run simple.main(1,4) like you do in your example I get an error that xgboost-ray training isn't supported on Windows. So if you're trying it on Windows it's not going to work.

For reference: image

rkbennett avatar Nov 14 '23 16:11 rkbennett

Traceback (most recent call last):
  File "c:\Users\franc\OneDrive\Escritorio\ti.py", line 6, in <module>
    import hello
ModuleNotFoundError: No module named 'hello'

Can you paste your code in here?

The problem seems to be python 3.12, it doesn't happen with od_import.

boludoz avatar Nov 14 '23 17:11 boludoz

Traceback (most recent call last):
  File "c:\Users\franc\OneDrive\Escritorio\ti.py", line 6, in <module>
    import hello
ModuleNotFoundError: No module named 'hello'

Can you paste your code in here?

The problem seems to be python 3.12, it doesn't happen with od_import.

Yeah, httpimport doesn't currently support 3.12, there's an open issue for adding that support, but in 3.12 they depricated the load_module function and completely removed the find_module function, which has broken compatibility with httpimport.

rkbennett avatar Nov 14 '23 17:11 rkbennett