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.

Sign up for GitHub

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

Jump to bottom

added check-allocate to Min, Max and Requantization_Range Ops' outputs #68

Open
wants to merge 115 commits into
base: develop
Choose a base branch
from
Open

added check-allocate to Min, Max and Requantization_Range Ops' outputs #68

wants to merge 115 commits into from

Conversation

neil-tan
Copy link
Member

@neil-tan neil-tan commented Dec 9, 2017

hot fix for code
added check-allocate to Min, Max and Requantization_Range Ops' outputs

Knight-X and others added 30 commits October 28, 2017 15:22
@Knight-X
Merge pull request #21 from neil-tan/integTest
c430949 
Integ test
@neil-tan
Merge pull request #22 from neil-tan/integTest
18c18fa 
  fix include name NNOps to NnOps
@neil-tan
context draft
91cf419
@neil-tan
re-vised draft
e2aaf98
@neil-tan
wip
7b2a5f9
@Knight-X
tensor extend first commit
a65a8a9 
  1. extend different type tensor for sd, memory
  2. inherit super class for polymorphism
@neil-tan
draft for merge
d219be5
@Knight-X
Merge pull request #25 from neil-tan/context_dev
9b47169 
Context dev
@Knight-X
Draft for tensor refactor
c09b38e 
  1. test idea quickly
  2. sync idea
  3. take type from tensor
  4. make type system in ramtensor
@Knight-X
add feature to ram tensor class
f669f69 
  1. implement add function
  2. implement customized ram tensor constructor
@mbartling
Add python requirements for SD preparation
e5b67bd
@Knight-X @neil-tan
wip
4d9269c
@neil-tan
suggesting tensor ref counter
96d3186
@neil-tan
typo
cd61f37
@neil-tan
Merge branch 'featuretensor_refactor' of ssh://github.com/neil-tan/uT…
de63c85 
…ensor into featuretensor_refactor
@neil-tan
suggesting tensor ref counter
12d77c8
@neil-tan
typo
60e0439
@neil-tan
Merge branch 'feature_tensor_ref' of ssh://github.com/neil-tan/uTenso…
4de1ce8 
…r into feature_tensor_ref
@Knight-X
make idxTest pass firstly
68a905f
@Knight-X
replace tensor<T> to tensor declaration in other functions such as T…
6b4349a 
…ensorConstant
@Knight-X
fix coding style
83509d1
@Knight-X
Merge pull request #28 from neil-tan/feature_tensor_ref
0fb021a 
Feature tensor ref initial merge commit
@Knight-X
1. change read function syntax according to interface
2d9ca3a
@Knight-X
make syntax of write function correct
038631d
@neil-tan
Merge pull request #27 from mbartling/b/getting-started
2cc354f 
Add python requirements for SD preparation
@neil-tan
Merge pull request #29 from neil-tan/featuretensor_refactor
d21c4ee 
Featuretensor refactor
@neil-tan
context ops compile sucessful
34c5e30
@neil-tan
DType WIP; added context, ops, tesnor-ref-count
bd3e49a
@Knight-X
revise main for test idx and matrixops
1b163e0
@Knight-X
Merge branch 'featuretensor_refactor' of https://github.com/neil-tan/…
af90dce 
…uTensor into featuretensor_refactor
Knight-X and others added 15 commits November 19, 2017 14:02
@Knight-X
Merge branch 'F/52' of https://github.com/neil-tan/uTensor into F/52
860cf9f
@neil-tan
Math, Matrix, Context passed
13071ba
@neil-tan
MathTests.hpp: make variable naming more consistent
b7530f5
@Knight-X
1. pass run deep mlp demo for name lookup
13872d7 
  2. modify tensorCast for name lookup
@Knight-X
Merge branch 'F/52' of https://github.com/neil-tan/uTensor into F/52
eacb822
@neil-tan
updated readme
1e58e90
@neil-tan
release 0.1.0
147f990
@neil-tan
Merge pull request #54 from neil-tan/F/52
a91e2eb 
F/52
@neil-tan
context stateful wip
0b884b6
@neil-tan
codeGenTemplate test passed
5209e03
@neil-tan
context lambda wip
66468d8
@neil-tan
context: add_static, addCached, push_static; context internal gc wip
1f96bc2
@neil-tan
updated comment in context.cpp
ae3a7a4
@Knight-X
Merge pull request #60 from neil-tan/F/context_cg_ref
b5c1702 
F/context cg ref
@neil-tan
added check-allocate to Min, Max and Requantization_Range Ops' outputs
f0e3c35
@neil-tan neil-tan added the bug label Dec 9, 2017
@dboyliao
Copy link
Member

dboyliao commented Dec 9, 2017

This does not behave as tf.reduce_max if the axis argument is not None, you're reducing entire tensor.
You are prohibiting all users for uTensor not to use axis in tf.reduce_max, tf.reduce_min...etc.

@mbartling
Copy link
Member

@neil-tan is this still relevant?

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

@Knight-X Knight-X Awaiting requested review from Knight-X

@dboyliao dboyliao Awaiting requested review from dboyliao

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@neil-tan @dboyliao @mbartling @Knight-X @BitYog