s4753993-GCN - #477

Open
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom
arshupadhyaya:master
Open

s4753993-GCN#477
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom
arshupadhyaya:master

Conversation

@arshupadhyaya

@arshupadhyayaarshupadhyaya commented Oct 23, 2022•
edited
Loading

Copy link
Copy Markdown

Hello Shakes,
This PR, to the best of my knowledge contains files required to run a 2 layer GCN along with TSNE mapping on the facebook.npz dataset. All the work was done on google colab, and I tried my best to create individual modules for the various functions including functions to load data, the model itself as well as training and testing function.
However I was not able to successfully implement a driver function in python, hence I have submitted the jupyter file containing all the functions together.
To successfully run the code please open GCN_final.ipynb, and run all commands in sequential order. Please just ensure file_path to load data matches local environment.
Thank you
Arsh Upadhyaya
(s4753993)

arshupadhyayaand others added 30 commits October 21, 2022 16:34
@arshupadhyaya
Create s47539934-GCN
6e19f4d
@arshupadhyaya
Delete s47539934-GCN
bc99d1a
@arshupadhyaya
Added new folder and README
9f93239
@arshupadhyaya
Added Preprocessed dataset
1d7eb06
@arshupadhyaya
Create Model.py having a 2 layer GCN
2205c15
@arshupadhyaya
added header block and comments in model.py
9d5c1a2
@arshupadhyaya
add header block and comments in data.py
d3fcd49
@arshupadhyaya
adding bias and dropout, along with minor changes
52ad3d7
@arshupadhyaya
Change structure of data.py, to better fit test and train set
c808c20
The previous datatypes for adjacency matrix, and especially features were creating problems during testing phase.
@arshupadhyaya
created training and testing function, and stored accuracy and losses
e8891bb
@arshupadhyaya
better method to store accuracy and loss
c775efe
Had trouble returning all four(validation_accuracies,validation_losses,train_accuracies,train_losses) together from one function, so decided to save in np arrays and load the arrays during driver code.
@arshupadhyaya
Added screenshots of accuracies and losses for both training and vali…
0d47b5d
…dation sets
@arshupadhyaya
accuracy and loss for test set
94585ca
@arshupadhyaya
mistakenly added test result in wrong directory, so deleted
2327c1e
@arshupadhyaya
accuracy and loss values after 200 epochs on test set
5624049
@arshupadhyaya
creating a predictive model to train model in real time and plot rele…
ba4ceb1
…vant graphs
@arshupadhyaya
removed data split and doing it in predict.py instead
d22ffe3
@arshupadhyaya
Added header block to train.py
61c3787
@arshupadhyaya
tsne map for GCN with 200 epochs
da34c7a
@arshupadhyaya
Added tsne map with 50 epochs
315fc58
@arshupadhyaya
updated predict model to include plotting tsne
789ebab
@arshupadhyaya
add tsne map with 100 epochs
4072e1a
@arshupadhyaya
create the tsne mapping function along with header block
1ab79b1
@arshupadhyaya
adding final working jupyter file with all functions and results incl…
47ea171
…uded
@arshupadhyaya
mistakenly put in wrong directory so removed
f74b4cf
@arshupadhyaya
final jupyter file containing working version of GCN model
7f78321
@arshupadhyaya
fixing directory
2696430
@arshupadhyaya
again changing directory
81dc37e
@arshupadhyaya
deleted an irrelevant file
8021030
@arshupadhyaya
Added initial readme file
e6bf0ec
@gayankugayanku added the GCN label Nov 4, 2022
@gayanku

gayanku commented Nov 9, 2022

Copy link
Copy Markdown
Collaborator

Looks good. TSNE also fine.

@shakes76

shakes76 commented Nov 20, 2022

Copy link
Copy Markdown
Owner

Good Practice (Design/Commenting, TF/Torch Usage)

Adequate use and implementation
Good spacing and comments
Header blocks

Recognition Problem

Solves problem
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting
No Data leakage
Difficulty: Normal -5

Commit Log

Meaningful commit messages, could be more informative -1
Progressive commits used

Documentation

ReadMe acceptable, background/architecture missing -2
Good Description and Comments
Markdown used PDF submitted

Pull Request

Successful Pull Request (Working Algorithm Delivered on Time in Wrong Branch) -1
Feedback required, must be pushed to topic-recognition branch not master, please fix -2
Request Description good

@arshupadhyaya
arshupadhyaya changed the base branch from master to topic-recognitionNovember 20, 2022 23:20
@arshupadhyaya

arshupadhyaya commented Nov 20, 2022

Copy link
Copy Markdown
Author

Hey Shakes, I believe I have fixed the branch issue, also just for knowledge could I know what you mean by README missing background/architecture? Does it mean architecture of the GCN model itself?

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Feedback_NeededGCN

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@arshupadhyaya@gayanku@shakes76
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
s4753993-GCN - #477
Open
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Open
s4753993-GCN#477
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Conversation
@arshupadhyaya
@arshupadhyayaarshupadhyaya
commented
Oct 23, 2022•

edited

Loading

Copy link


Copy Markdown

Hello Shakes,

This PR, to the best of my knowledge contains files required to run a 2 layer GCN along with TSNE mapping on the facebook.npz dataset. All the work was done on google colab, and I tried my best to create individual modules for the various functions including functions to load data, the model itself as well as training and testing function.

However I was not able to successfully implement a driver function in python, hence I have submitted the jupyter file containing all the functions together.

To successfully run the code please open GCN_final.ipynb, and run all commands in sequential order. Please just ensure file_path to load data matches local environment.

Thank you

Arsh Upadhyaya

(s4753993)
arshupadhyayaand others
added 30 commits
October 21, 2022 16:34
@arshupadhyaya
Create s47539934-GCN
6e19f4d
@arshupadhyaya
Delete s47539934-GCN
bc99d1a
@arshupadhyaya
Added new folder and README
9f93239
@arshupadhyaya
 Added Preprocessed dataset
1d7eb06
@arshupadhyaya
Create Model.py having a 2 layer GCN
2205c15
@arshupadhyaya
added header block and comments in model.py
9d5c1a2
@arshupadhyaya
add header block and comments in data.py
d3fcd49
@arshupadhyaya
adding bias and dropout, along with minor changes
52ad3d7
@arshupadhyaya
Change structure of data.py, to better fit test and train set
c808c20
The previous datatypes for adjacency matrix, and especially features were creating problems during testing phase.
@arshupadhyaya
created training and testing function, and stored accuracy and losses
e8891bb
@arshupadhyaya
better method to store accuracy and loss
c775efe
Had trouble returning all four(validation_accuracies,validation_losses,train_accuracies,train_losses) together from one function, so decided to save in np arrays and load the arrays during driver code.
@arshupadhyaya
Added screenshots of accuracies and losses for both training and vali…
0d47b5d
…dation sets
@arshupadhyaya
accuracy and loss for test set
94585ca
@arshupadhyaya
mistakenly added test result in wrong directory, so deleted
2327c1e
@arshupadhyaya
accuracy and loss values after 200 epochs on test set
5624049
@arshupadhyaya
creating a predictive model to train model in real time and plot rele…
ba4ceb1
…vant graphs
@arshupadhyaya
removed data split and doing it in predict.py instead
d22ffe3
@arshupadhyaya
Added header block to train.py
61c3787
@arshupadhyaya
tsne map for GCN with 200 epochs
da34c7a
@arshupadhyaya
Added tsne map with 50 epochs
315fc58
@arshupadhyaya
updated predict model to include plotting tsne
789ebab
@arshupadhyaya
add tsne map with 100 epochs
4072e1a
@arshupadhyaya
create the tsne mapping function along with header block
1ab79b1
@arshupadhyaya
adding final working jupyter file with all functions and results incl…
47ea171
…uded
@arshupadhyaya
mistakenly put in wrong directory so removed
f74b4cf
@arshupadhyaya
final jupyter file containing working version of GCN model
7f78321
@arshupadhyaya
fixing directory
2696430
@arshupadhyaya
again changing directory
81dc37e
@arshupadhyaya
deleted an irrelevant file
8021030
@arshupadhyaya
Added initial readme file
e6bf0ec
@gayankugayanku
added
the 
GCN

label
Nov 4, 2022
@gayanku
gayanku
commented
Nov 9, 2022

Copy link


Copy Markdown

Collaborator
Looks good. TSNE also fine.
@shakes76
shakes76
commented
Nov 20, 2022

Copy link


Copy Markdown

Owner
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate use and implementation

Good spacing and comments

Header blocks
Recognition Problem
Solves problem

Driver Script present

File structure present

Shows Usage & Demo & Visualisation & Data usage

Module present

Commenting

No Data leakage

Difficulty: Normal -5
Commit Log
Meaningful commit messages, could be more informative -1

Progressive commits used
Documentation
ReadMe acceptable, background/architecture missing -2

Good Description and Comments

Markdown used PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Wrong Branch) -1

Feedback required, must be pushed to topic-recognition branch not master, please fix -2

Request Description good
@arshupadhyaya
arshupadhyaya
changed the base branch from
master
to
topic-recognitionNovember 20, 2022 23:20
@arshupadhyaya
arshupadhyaya
commented
Nov 20, 2022

Copy link


Copy Markdown

Author
Hey Shakes, I believe I have fixed the branch issue, also just for knowledge could I know what you mean by README missing background/architecture? Does it mean architecture of the GCN model itself?
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels

Feedback_NeededGCN

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@arshupadhyaya@gayanku@shakes76

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
s4753993-GCN - #477
Open
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Open
s4753993-GCN#477
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Conversation
@arshupadhyaya
@arshupadhyayaarshupadhyaya
commented
Oct 23, 2022•

edited

Loading

Copy link


Copy Markdown

Hello Shakes,

This PR, to the best of my knowledge contains files required to run a 2 layer GCN along with TSNE mapping on the facebook.npz dataset. All the work was done on google colab, and I tried my best to create individual modules for the various functions including functions to load data, the model itself as well as training and testing function.

However I was not able to successfully implement a driver function in python, hence I have submitted the jupyter file containing all the functions together.

To successfully run the code please open GCN_final.ipynb, and run all commands in sequential order. Please just ensure file_path to load data matches local environment.

Thank you

Arsh Upadhyaya

(s4753993)
arshupadhyayaand others
added 30 commits
October 21, 2022 16:34
@arshupadhyaya
Create s47539934-GCN
6e19f4d
@arshupadhyaya
Delete s47539934-GCN
bc99d1a
@arshupadhyaya
Added new folder and README
9f93239
@arshupadhyaya
 Added Preprocessed dataset
1d7eb06
@arshupadhyaya
Create Model.py having a 2 layer GCN
2205c15
@arshupadhyaya
added header block and comments in model.py
9d5c1a2
@arshupadhyaya
add header block and comments in data.py
d3fcd49
@arshupadhyaya
adding bias and dropout, along with minor changes
52ad3d7
@arshupadhyaya
Change structure of data.py, to better fit test and train set
c808c20
The previous datatypes for adjacency matrix, and especially features were creating problems during testing phase.
@arshupadhyaya
created training and testing function, and stored accuracy and losses
e8891bb
@arshupadhyaya
better method to store accuracy and loss
c775efe
Had trouble returning all four(validation_accuracies,validation_losses,train_accuracies,train_losses) together from one function, so decided to save in np arrays and load the arrays during driver code.
@arshupadhyaya
Added screenshots of accuracies and losses for both training and vali…
0d47b5d
…dation sets
@arshupadhyaya
accuracy and loss for test set
94585ca
@arshupadhyaya
mistakenly added test result in wrong directory, so deleted
2327c1e
@arshupadhyaya
accuracy and loss values after 200 epochs on test set
5624049
@arshupadhyaya
creating a predictive model to train model in real time and plot rele…
ba4ceb1
…vant graphs
@arshupadhyaya
removed data split and doing it in predict.py instead
d22ffe3
@arshupadhyaya
Added header block to train.py
61c3787
@arshupadhyaya
tsne map for GCN with 200 epochs
da34c7a
@arshupadhyaya
Added tsne map with 50 epochs
315fc58
@arshupadhyaya
updated predict model to include plotting tsne
789ebab
@arshupadhyaya
add tsne map with 100 epochs
4072e1a
@arshupadhyaya
create the tsne mapping function along with header block
1ab79b1
@arshupadhyaya
adding final working jupyter file with all functions and results incl…
47ea171
…uded
@arshupadhyaya
mistakenly put in wrong directory so removed
f74b4cf
@arshupadhyaya
final jupyter file containing working version of GCN model
7f78321
@arshupadhyaya
fixing directory
2696430
@arshupadhyaya
again changing directory
81dc37e
@arshupadhyaya
deleted an irrelevant file
8021030
@arshupadhyaya
Added initial readme file
e6bf0ec
@gayankugayanku
added
the 
GCN

label
Nov 4, 2022
@gayanku
gayanku
commented
Nov 9, 2022

Copy link


Copy Markdown

Collaborator
Looks good. TSNE also fine.
@shakes76
shakes76
commented
Nov 20, 2022

Copy link


Copy Markdown

Owner
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate use and implementation

Good spacing and comments

Header blocks
Recognition Problem
Solves problem

Driver Script present

File structure present

Shows Usage & Demo & Visualisation & Data usage

Module present

Commenting

No Data leakage

Difficulty: Normal -5
Commit Log
Meaningful commit messages, could be more informative -1

Progressive commits used
Documentation
ReadMe acceptable, background/architecture missing -2

Good Description and Comments

Markdown used PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Wrong Branch) -1

Feedback required, must be pushed to topic-recognition branch not master, please fix -2

Request Description good
@arshupadhyaya
arshupadhyaya
changed the base branch from
master
to
topic-recognitionNovember 20, 2022 23:20
@arshupadhyaya
arshupadhyaya
commented
Nov 20, 2022

Copy link


Copy Markdown

Author
Hey Shakes, I believe I have fixed the branch issue, also just for knowledge could I know what you mean by README missing background/architecture? Does it mean architecture of the GCN model itself?
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels

Feedback_NeededGCN

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@arshupadhyaya@gayanku@shakes76

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
s4753993-GCN - #477
Open
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Open
s4753993-GCN#477
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Conversation
@arshupadhyaya
@arshupadhyayaarshupadhyaya
commented
Oct 23, 2022•

edited

Loading

Copy link


Copy Markdown

Hello Shakes,

This PR, to the best of my knowledge contains files required to run a 2 layer GCN along with TSNE mapping on the facebook.npz dataset. All the work was done on google colab, and I tried my best to create individual modules for the various functions including functions to load data, the model itself as well as training and testing function.

However I was not able to successfully implement a driver function in python, hence I have submitted the jupyter file containing all the functions together.

To successfully run the code please open GCN_final.ipynb, and run all commands in sequential order. Please just ensure file_path to load data matches local environment.

Thank you

Arsh Upadhyaya

(s4753993)
arshupadhyayaand others
added 30 commits
October 21, 2022 16:34
@arshupadhyaya
Create s47539934-GCN
6e19f4d
@arshupadhyaya
Delete s47539934-GCN
bc99d1a
@arshupadhyaya
Added new folder and README
9f93239
@arshupadhyaya
 Added Preprocessed dataset
1d7eb06
@arshupadhyaya
Create Model.py having a 2 layer GCN
2205c15
@arshupadhyaya
added header block and comments in model.py
9d5c1a2
@arshupadhyaya
add header block and comments in data.py
d3fcd49
@arshupadhyaya
adding bias and dropout, along with minor changes
52ad3d7
@arshupadhyaya
Change structure of data.py, to better fit test and train set
c808c20
The previous datatypes for adjacency matrix, and especially features were creating problems during testing phase.
@arshupadhyaya
created training and testing function, and stored accuracy and losses
e8891bb
@arshupadhyaya
better method to store accuracy and loss
c775efe
Had trouble returning all four(validation_accuracies,validation_losses,train_accuracies,train_losses) together from one function, so decided to save in np arrays and load the arrays during driver code.
@arshupadhyaya
Added screenshots of accuracies and losses for both training and vali…
0d47b5d
…dation sets
@arshupadhyaya
accuracy and loss for test set
94585ca
@arshupadhyaya
mistakenly added test result in wrong directory, so deleted
2327c1e
@arshupadhyaya
accuracy and loss values after 200 epochs on test set
5624049
@arshupadhyaya
creating a predictive model to train model in real time and plot rele…
ba4ceb1
…vant graphs
@arshupadhyaya
removed data split and doing it in predict.py instead
d22ffe3
@arshupadhyaya
Added header block to train.py
61c3787
@arshupadhyaya
tsne map for GCN with 200 epochs
da34c7a
@arshupadhyaya
Added tsne map with 50 epochs
315fc58
@arshupadhyaya
updated predict model to include plotting tsne
789ebab
@arshupadhyaya
add tsne map with 100 epochs
4072e1a
@arshupadhyaya
create the tsne mapping function along with header block
1ab79b1
@arshupadhyaya
adding final working jupyter file with all functions and results incl…
47ea171
…uded
@arshupadhyaya
mistakenly put in wrong directory so removed
f74b4cf
@arshupadhyaya
final jupyter file containing working version of GCN model
7f78321
@arshupadhyaya
fixing directory
2696430
@arshupadhyaya
again changing directory
81dc37e
@arshupadhyaya
deleted an irrelevant file
8021030
@arshupadhyaya
Added initial readme file
e6bf0ec
@gayankugayanku
added
the 
GCN

label
Nov 4, 2022
@gayanku
gayanku
commented
Nov 9, 2022

Copy link


Copy Markdown

Collaborator
Looks good. TSNE also fine.
@shakes76
shakes76
commented
Nov 20, 2022

Copy link


Copy Markdown

Owner
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate use and implementation

Good spacing and comments

Header blocks
Recognition Problem
Solves problem

Driver Script present

File structure present

Shows Usage & Demo & Visualisation & Data usage

Module present

Commenting

No Data leakage

Difficulty: Normal -5
Commit Log
Meaningful commit messages, could be more informative -1

Progressive commits used
Documentation
ReadMe acceptable, background/architecture missing -2

Good Description and Comments

Markdown used PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Wrong Branch) -1

Feedback required, must be pushed to topic-recognition branch not master, please fix -2

Request Description good
@arshupadhyaya
arshupadhyaya
changed the base branch from
master
to
topic-recognitionNovember 20, 2022 23:20
@arshupadhyaya
arshupadhyaya
commented
Nov 20, 2022

Copy link


Copy Markdown

Author
Hey Shakes, I believe I have fixed the branch issue, also just for knowledge could I know what you mean by README missing background/architecture? Does it mean architecture of the GCN model itself?
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels

Feedback_NeededGCN

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@arshupadhyaya@gayanku@shakes76

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
s4753993-GCN - #477
Open
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Open
s4753993-GCN#477
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Conversation
@arshupadhyaya
@arshupadhyayaarshupadhyaya
commented
Oct 23, 2022•

edited

Loading

Copy link


Copy Markdown

Hello Shakes,

This PR, to the best of my knowledge contains files required to run a 2 layer GCN along with TSNE mapping on the facebook.npz dataset. All the work was done on google colab, and I tried my best to create individual modules for the various functions including functions to load data, the model itself as well as training and testing function.

However I was not able to successfully implement a driver function in python, hence I have submitted the jupyter file containing all the functions together.

To successfully run the code please open GCN_final.ipynb, and run all commands in sequential order. Please just ensure file_path to load data matches local environment.

Thank you

Arsh Upadhyaya

(s4753993)
arshupadhyayaand others
added 30 commits
October 21, 2022 16:34
@arshupadhyaya
Create s47539934-GCN
6e19f4d
@arshupadhyaya
Delete s47539934-GCN
bc99d1a
@arshupadhyaya
Added new folder and README
9f93239
@arshupadhyaya
 Added Preprocessed dataset
1d7eb06
@arshupadhyaya
Create Model.py having a 2 layer GCN
2205c15
@arshupadhyaya
added header block and comments in model.py
9d5c1a2
@arshupadhyaya
add header block and comments in data.py
d3fcd49
@arshupadhyaya
adding bias and dropout, along with minor changes
52ad3d7
@arshupadhyaya
Change structure of data.py, to better fit test and train set
c808c20
The previous datatypes for adjacency matrix, and especially features were creating problems during testing phase.
@arshupadhyaya
created training and testing function, and stored accuracy and losses
e8891bb
@arshupadhyaya
better method to store accuracy and loss
c775efe
Had trouble returning all four(validation_accuracies,validation_losses,train_accuracies,train_losses) together from one function, so decided to save in np arrays and load the arrays during driver code.
@arshupadhyaya
Added screenshots of accuracies and losses for both training and vali…
0d47b5d
…dation sets
@arshupadhyaya
accuracy and loss for test set
94585ca
@arshupadhyaya
mistakenly added test result in wrong directory, so deleted
2327c1e
@arshupadhyaya
accuracy and loss values after 200 epochs on test set
5624049
@arshupadhyaya
creating a predictive model to train model in real time and plot rele…
ba4ceb1
…vant graphs
@arshupadhyaya
removed data split and doing it in predict.py instead
d22ffe3
@arshupadhyaya
Added header block to train.py
61c3787
@arshupadhyaya
tsne map for GCN with 200 epochs
da34c7a
@arshupadhyaya
Added tsne map with 50 epochs
315fc58
@arshupadhyaya
updated predict model to include plotting tsne
789ebab
@arshupadhyaya
add tsne map with 100 epochs
4072e1a
@arshupadhyaya
create the tsne mapping function along with header block
1ab79b1
@arshupadhyaya
adding final working jupyter file with all functions and results incl…
47ea171
…uded
@arshupadhyaya
mistakenly put in wrong directory so removed
f74b4cf
@arshupadhyaya
final jupyter file containing working version of GCN model
7f78321
@arshupadhyaya
fixing directory
2696430
@arshupadhyaya
again changing directory
81dc37e
@arshupadhyaya
deleted an irrelevant file
8021030
@arshupadhyaya
Added initial readme file
e6bf0ec
@gayankugayanku
added
the 
GCN

label
Nov 4, 2022
@gayanku
gayanku
commented
Nov 9, 2022

Copy link


Copy Markdown

Collaborator
Looks good. TSNE also fine.
@shakes76
shakes76
commented
Nov 20, 2022

Copy link


Copy Markdown

Owner
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate use and implementation

Good spacing and comments

Header blocks
Recognition Problem
Solves problem

Driver Script present

File structure present

Shows Usage & Demo & Visualisation & Data usage

Module present

Commenting

No Data leakage

Difficulty: Normal -5
Commit Log
Meaningful commit messages, could be more informative -1

Progressive commits used
Documentation
ReadMe acceptable, background/architecture missing -2

Good Description and Comments

Markdown used PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Wrong Branch) -1

Feedback required, must be pushed to topic-recognition branch not master, please fix -2

Request Description good
@arshupadhyaya
arshupadhyaya
changed the base branch from
master
to
topic-recognitionNovember 20, 2022 23:20
@arshupadhyaya
arshupadhyaya
commented
Nov 20, 2022

Copy link


Copy Markdown

Author
Hey Shakes, I believe I have fixed the branch issue, also just for knowledge could I know what you mean by README missing background/architecture? Does it mean architecture of the GCN model itself?
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels

Feedback_NeededGCN

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@arshupadhyaya@gayanku@shakes76

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
s4753993-GCN - #477
Open
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Open
s4753993-GCN#477
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Conversation
@arshupadhyaya
@arshupadhyayaarshupadhyaya
commented
Oct 23, 2022•

edited

Loading

Copy link


Copy Markdown

Hello Shakes,

This PR, to the best of my knowledge contains files required to run a 2 layer GCN along with TSNE mapping on the facebook.npz dataset. All the work was done on google colab, and I tried my best to create individual modules for the various functions including functions to load data, the model itself as well as training and testing function.

However I was not able to successfully implement a driver function in python, hence I have submitted the jupyter file containing all the functions together.

To successfully run the code please open GCN_final.ipynb, and run all commands in sequential order. Please just ensure file_path to load data matches local environment.

Thank you

Arsh Upadhyaya

(s4753993)
arshupadhyayaand others
added 30 commits
October 21, 2022 16:34
@arshupadhyaya
Create s47539934-GCN
6e19f4d
@arshupadhyaya
Delete s47539934-GCN
bc99d1a
@arshupadhyaya
Added new folder and README
9f93239
@arshupadhyaya
 Added Preprocessed dataset
1d7eb06
@arshupadhyaya
Create Model.py having a 2 layer GCN
2205c15
@arshupadhyaya
added header block and comments in model.py
9d5c1a2
@arshupadhyaya
add header block and comments in data.py
d3fcd49
@arshupadhyaya
adding bias and dropout, along with minor changes
52ad3d7
@arshupadhyaya
Change structure of data.py, to better fit test and train set
c808c20
The previous datatypes for adjacency matrix, and especially features were creating problems during testing phase.
@arshupadhyaya
created training and testing function, and stored accuracy and losses
e8891bb
@arshupadhyaya
better method to store accuracy and loss
c775efe
Had trouble returning all four(validation_accuracies,validation_losses,train_accuracies,train_losses) together from one function, so decided to save in np arrays and load the arrays during driver code.
@arshupadhyaya
Added screenshots of accuracies and losses for both training and vali…
0d47b5d
…dation sets
@arshupadhyaya
accuracy and loss for test set
94585ca
@arshupadhyaya
mistakenly added test result in wrong directory, so deleted
2327c1e
@arshupadhyaya
accuracy and loss values after 200 epochs on test set
5624049
@arshupadhyaya
creating a predictive model to train model in real time and plot rele…
ba4ceb1
…vant graphs
@arshupadhyaya
removed data split and doing it in predict.py instead
d22ffe3
@arshupadhyaya
Added header block to train.py
61c3787
@arshupadhyaya
tsne map for GCN with 200 epochs
da34c7a
@arshupadhyaya
Added tsne map with 50 epochs
315fc58
@arshupadhyaya
updated predict model to include plotting tsne
789ebab
@arshupadhyaya
add tsne map with 100 epochs
4072e1a
@arshupadhyaya
create the tsne mapping function along with header block
1ab79b1
@arshupadhyaya
adding final working jupyter file with all functions and results incl…
47ea171
…uded
@arshupadhyaya
mistakenly put in wrong directory so removed
f74b4cf
@arshupadhyaya
final jupyter file containing working version of GCN model
7f78321
@arshupadhyaya
fixing directory
2696430
@arshupadhyaya
again changing directory
81dc37e
@arshupadhyaya
deleted an irrelevant file
8021030
@arshupadhyaya
Added initial readme file
e6bf0ec
@gayankugayanku
added
the 
GCN

label
Nov 4, 2022
@gayanku
gayanku
commented
Nov 9, 2022

Copy link


Copy Markdown

Collaborator
Looks good. TSNE also fine.
@shakes76
shakes76
commented
Nov 20, 2022

Copy link


Copy Markdown

Owner
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate use and implementation

Good spacing and comments

Header blocks
Recognition Problem
Solves problem

Driver Script present

File structure present

Shows Usage & Demo & Visualisation & Data usage

Module present

Commenting

No Data leakage

Difficulty: Normal -5
Commit Log
Meaningful commit messages, could be more informative -1

Progressive commits used
Documentation
ReadMe acceptable, background/architecture missing -2

Good Description and Comments

Markdown used PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Wrong Branch) -1

Feedback required, must be pushed to topic-recognition branch not master, please fix -2

Request Description good
@arshupadhyaya
arshupadhyaya
changed the base branch from
master
to
topic-recognitionNovember 20, 2022 23:20
@arshupadhyaya
arshupadhyaya
commented
Nov 20, 2022

Copy link


Copy Markdown

Author
Hey Shakes, I believe I have fixed the branch issue, also just for knowledge could I know what you mean by README missing background/architecture? Does it mean architecture of the GCN model itself?
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels

Feedback_NeededGCN

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@arshupadhyaya@gayanku@shakes76

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
s4753993-GCN - #477
Open
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Open
s4753993-GCN#477
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Conversation
@arshupadhyaya
@arshupadhyayaarshupadhyaya
commented
Oct 23, 2022•

edited

Loading

Copy link


Copy Markdown

Hello Shakes,

This PR, to the best of my knowledge contains files required to run a 2 layer GCN along with TSNE mapping on the facebook.npz dataset. All the work was done on google colab, and I tried my best to create individual modules for the various functions including functions to load data, the model itself as well as training and testing function.

However I was not able to successfully implement a driver function in python, hence I have submitted the jupyter file containing all the functions together.

To successfully run the code please open GCN_final.ipynb, and run all commands in sequential order. Please just ensure file_path to load data matches local environment.

Thank you

Arsh Upadhyaya

(s4753993)
arshupadhyayaand others
added 30 commits
October 21, 2022 16:34
@arshupadhyaya
Create s47539934-GCN
6e19f4d
@arshupadhyaya
Delete s47539934-GCN
bc99d1a
@arshupadhyaya
Added new folder and README
9f93239
@arshupadhyaya
 Added Preprocessed dataset
1d7eb06
@arshupadhyaya
Create Model.py having a 2 layer GCN
2205c15
@arshupadhyaya
added header block and comments in model.py
9d5c1a2
@arshupadhyaya
add header block and comments in data.py
d3fcd49
@arshupadhyaya
adding bias and dropout, along with minor changes
52ad3d7
@arshupadhyaya
Change structure of data.py, to better fit test and train set
c808c20
The previous datatypes for adjacency matrix, and especially features were creating problems during testing phase.
@arshupadhyaya
created training and testing function, and stored accuracy and losses
e8891bb
@arshupadhyaya
better method to store accuracy and loss
c775efe
Had trouble returning all four(validation_accuracies,validation_losses,train_accuracies,train_losses) together from one function, so decided to save in np arrays and load the arrays during driver code.
@arshupadhyaya
Added screenshots of accuracies and losses for both training and vali…
0d47b5d
…dation sets
@arshupadhyaya
accuracy and loss for test set
94585ca
@arshupadhyaya
mistakenly added test result in wrong directory, so deleted
2327c1e
@arshupadhyaya
accuracy and loss values after 200 epochs on test set
5624049
@arshupadhyaya
creating a predictive model to train model in real time and plot rele…
ba4ceb1
…vant graphs
@arshupadhyaya
removed data split and doing it in predict.py instead
d22ffe3
@arshupadhyaya
Added header block to train.py
61c3787
@arshupadhyaya
tsne map for GCN with 200 epochs
da34c7a
@arshupadhyaya
Added tsne map with 50 epochs
315fc58
@arshupadhyaya
updated predict model to include plotting tsne
789ebab
@arshupadhyaya
add tsne map with 100 epochs
4072e1a
@arshupadhyaya
create the tsne mapping function along with header block
1ab79b1
@arshupadhyaya
adding final working jupyter file with all functions and results incl…
47ea171
…uded
@arshupadhyaya
mistakenly put in wrong directory so removed
f74b4cf
@arshupadhyaya
final jupyter file containing working version of GCN model
7f78321
@arshupadhyaya
fixing directory
2696430
@arshupadhyaya
again changing directory
81dc37e
@arshupadhyaya
deleted an irrelevant file
8021030
@arshupadhyaya
Added initial readme file
e6bf0ec
@gayankugayanku
added
the 
GCN

label
Nov 4, 2022
@gayanku
gayanku
commented
Nov 9, 2022

Copy link


Copy Markdown

Collaborator
Looks good. TSNE also fine.
@shakes76
shakes76
commented
Nov 20, 2022

Copy link


Copy Markdown

Owner
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate use and implementation

Good spacing and comments

Header blocks
Recognition Problem
Solves problem

Driver Script present

File structure present

Shows Usage & Demo & Visualisation & Data usage

Module present

Commenting

No Data leakage

Difficulty: Normal -5
Commit Log
Meaningful commit messages, could be more informative -1

Progressive commits used
Documentation
ReadMe acceptable, background/architecture missing -2

Good Description and Comments

Markdown used PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Wrong Branch) -1

Feedback required, must be pushed to topic-recognition branch not master, please fix -2

Request Description good
@arshupadhyaya
arshupadhyaya
changed the base branch from
master
to
topic-recognitionNovember 20, 2022 23:20
@arshupadhyaya
arshupadhyaya
commented
Nov 20, 2022

Copy link


Copy Markdown

Author
Hey Shakes, I believe I have fixed the branch issue, also just for knowledge could I know what you mean by README missing background/architecture? Does it mean architecture of the GCN model itself?
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels

Feedback_NeededGCN

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@arshupadhyaya@gayanku@shakes76

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
s4753993-GCN - #477
Open
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Open
s4753993-GCN#477
arshupadhyaya wants to merge 41 commits into
shakes76:topic-recognitionfrom 
arshupadhyaya:master
Conversation
@arshupadhyaya
@arshupadhyayaarshupadhyaya
commented
Oct 23, 2022•

edited

Loading

Copy link


Copy Markdown

Hello Shakes,

This PR, to the best of my knowledge contains files required to run a 2 layer GCN along with TSNE mapping on the facebook.npz dataset. All the work was done on google colab, and I tried my best to create individual modules for the various functions including functions to load data, the model itself as well as training and testing function.

However I was not able to successfully implement a driver function in python, hence I have submitted the jupyter file containing all the functions together.

To successfully run the code please open GCN_final.ipynb, and run all commands in sequential order. Please just ensure file_path to load data matches local environment.

Thank you

Arsh Upadhyaya

(s4753993)
arshupadhyayaand others
added 30 commits
October 21, 2022 16:34
@arshupadhyaya
Create s47539934-GCN
6e19f4d
@arshupadhyaya
Delete s47539934-GCN
bc99d1a
@arshupadhyaya
Added new folder and README
9f93239
@arshupadhyaya
 Added Preprocessed dataset
1d7eb06
@arshupadhyaya
Create Model.py having a 2 layer GCN
2205c15
@arshupadhyaya
added header block and comments in model.py
9d5c1a2
@arshupadhyaya
add header block and comments in data.py
d3fcd49
@arshupadhyaya
adding bias and dropout, along with minor changes
52ad3d7
@arshupadhyaya
Change structure of data.py, to better fit test and train set
c808c20
The previous datatypes for adjacency matrix, and especially features were creating problems during testing phase.
@arshupadhyaya
created training and testing function, and stored accuracy and losses
e8891bb
@arshupadhyaya
better method to store accuracy and loss
c775efe
Had trouble returning all four(validation_accuracies,validation_losses,train_accuracies,train_losses) together from one function, so decided to save in np arrays and load the arrays during driver code.
@arshupadhyaya
Added screenshots of accuracies and losses for both training and vali…
0d47b5d
…dation sets
@arshupadhyaya
accuracy and loss for test set
94585ca
@arshupadhyaya
mistakenly added test result in wrong directory, so deleted
2327c1e
@arshupadhyaya
accuracy and loss values after 200 epochs on test set
5624049
@arshupadhyaya
creating a predictive model to train model in real time and plot rele…
ba4ceb1
…vant graphs
@arshupadhyaya
removed data split and doing it in predict.py instead
d22ffe3
@arshupadhyaya
Added header block to train.py
61c3787
@arshupadhyaya
tsne map for GCN with 200 epochs
da34c7a
@arshupadhyaya
Added tsne map with 50 epochs
315fc58
@arshupadhyaya
updated predict model to include plotting tsne
789ebab
@arshupadhyaya
add tsne map with 100 epochs
4072e1a
@arshupadhyaya
create the tsne mapping function along with header block
1ab79b1
@arshupadhyaya
adding final working jupyter file with all functions and results incl…
47ea171
…uded
@arshupadhyaya
mistakenly put in wrong directory so removed
f74b4cf
@arshupadhyaya
final jupyter file containing working version of GCN model
7f78321
@arshupadhyaya
fixing directory
2696430
@arshupadhyaya
again changing directory
81dc37e
@arshupadhyaya
deleted an irrelevant file
8021030
@arshupadhyaya
Added initial readme file
e6bf0ec
@gayankugayanku
added
the 
GCN

label
Nov 4, 2022
@gayanku
gayanku
commented
Nov 9, 2022

Copy link


Copy Markdown

Collaborator
Looks good. TSNE also fine.
@shakes76
shakes76
commented
Nov 20, 2022

Copy link


Copy Markdown

Owner
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate use and implementation

Good spacing and comments

Header blocks
Recognition Problem
Solves problem

Driver Script present

File structure present

Shows Usage & Demo & Visualisation & Data usage

Module present

Commenting

No Data leakage

Difficulty: Normal -5
Commit Log
Meaningful commit messages, could be more informative -1

Progressive commits used
Documentation
ReadMe acceptable, background/architecture missing -2

Good Description and Comments

Markdown used PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Wrong Branch) -1

Feedback required, must be pushed to topic-recognition branch not master, please fix -2

Request Description good
@arshupadhyaya
arshupadhyaya
changed the base branch from
master
to
topic-recognitionNovember 20, 2022 23:20
@arshupadhyaya
arshupadhyaya
commented
Nov 20, 2022

Copy link


Copy Markdown

Author
Hey Shakes, I believe I have fixed the branch issue, also just for knowledge could I know what you mean by README missing background/architecture? Does it mean architecture of the GCN model itself?
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


No reviews


Assignees


No one assigned


Labels

Feedback_NeededGCN

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@arshupadhyaya@gayanku@shakes76