carpedm20 / carpedm20/MemN2N-tensorflow

Global gradients clipping should be used in stead of clipping each matrix separately.

Open
#19 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
824
Forks
245
PR merge metrics
No merged PRs in 30d

Description

I found your code measuring norms and clipping gradients for each parameter separately. But in the paper, the authors said "the l2 norm of the whole gradient of all parameters..." and your method was used in QA tasks.

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the code that measures gradient norms and clips gradients for each parameter separately. Compare that behavior with the paper's requirement to use the L2 norm across all parameters, then verify that the implementation applies global clipping and preserves the expected QA-task behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
python, tensorflow
Domain
machine-learning
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.