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

what library of manifest is evaporate suppose to be using? #26

Open
brando90 opened this issue Jul 25, 2023 · 2 comments
Open

what library of manifest is evaporate suppose to be using? #26

brando90 opened this issue Jul 25, 2023 · 2 comments

Comments

@brando90
Copy link

I think this is not present in either the src install of manifest or manifest-ml from pypi

Traceback (most recent call last):
  File "/lfs/ampere1/0/brando9/evaporate/evaporate/run_profiler.py", line 477, in <module>
    main()
  File "/lfs/ampere1/0/brando9/evaporate/evaporate/run_profiler.py", line 473, in main
    run_experiment(profiler_args)
  File "/lfs/ampere1/0/brando9/evaporate/evaporate/run_profiler.py", line 237, in run_experiment
    file2chunks, file2contents, manifest_sessions = prepare_data(
  File "/lfs/ampere1/0/brando9/evaporate/evaporate/run_profiler.py", line 64, in prepare_data
    manifest_sessions = get_manifest_sessions(profiler_args.MODELS, MODEL2URL=profiler_args.MODEL2URL, KEYS=profiler_args.KEYS)
  File "/afs/cs.stanford.edu/u/brando9/evaporate/evaporate/utils.py", line 149, in get_manifest_sessions
    manifest, model_name = get_manifest_session(
  File "/afs/cs.stanford.edu/u/brando9/evaporate/evaporate/utils.py", line 211, in get_manifest_session
    params = manifest.client_pool.get_client().get_model_params()
AttributeError: 'ClientConnectionPool' object has no attribute 'get_client'. Did you mean: 'num_clients'?

trying to debug it right now.

@brando90
Copy link
Author

seems that manifest-ml not source worked.

I recommend not putting the manifest src instructions in the current way, it looks like install from src is a req.

@xiaohaoxing
Copy link

I debugged and replaced this function with get_current_client().

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

No branches or pull requests

2 participants