Skip to content

Conversation

dcolinmorgan
Copy link
Contributor

No description provided.

@lmeyerov
Copy link
Contributor

lmeyerov commented Nov 4, 2023

CI failing?

@lmeyerov
Copy link
Contributor

I think the latest master has that fixed, so merge it in?

"metadata": {},
"output_type": "execute_result"
}
],
"source": [
"url1 = 'https://s3-us-west-1.amazonaws.com/graphistry.demo.data/BIOGRID-ALL-3.3.123.tab2.txt.gz'\n",
"rawdata = pandas.read_table(url1, na_values=['-'], engine='c', compression='gzip')\n",
"url1 = 'http://thebiogrid.org/downloads/archives/Release%20Archive/BIOGRID-3.3.123/BIOGRID-ALL-3.3.123.tab2.zip'\n",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https

@lmeyerov
Copy link
Contributor

"id": "6lK_VrzZT1n7"
},
"source": [
"#### get a free api-key at https://www.graphistry.com/"
Copy link
Contributor

@lmeyerov lmeyerov Jul 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • For raw data, get a free Kaggle account...
  • For the GPU-cloud-accelerated visualization step, get a free API key at https://hub.graphistry.com

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

put the kaggle account step next to the graphistry account step so clear during setup

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove collab dependency for kaggle

Copy link
Contributor

@lmeyerov lmeyerov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments

Copy link
Contributor

@lmeyerov lmeyerov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See requests for removing unncessary (?) umap post-processing + some cleaner intros

"name": "stdout",
"output_type": "stream",
"text": [
"--2024-07-09 07:23:29-- https://raw.githubusercontent.com/dcolinmorgan/grph/main/PRJNA544527-meta_inf.txt\n",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like still need to update this one

can now use: https://gist.github.com/lmeyerov/b650f1ef9e56c3f1888ebb009bc5ed46

"source": [
"from google.colab import userdata\n",
"g_user=userdata.get('g_user')\n",
"g_pass=userdata.get('g_pass')"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in all notebooks, remove colab runtime dependency

Copy link
Contributor

@lmeyerov lmeyerov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See new comments and unresolved old ones

@dcolinmorgan Can we also add a link to something like nb, or nb1+nb2, to the readme.md gallery area with a biogrid link? please keep short / test render to ensure layout doesn't blow out

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.

2 participants