Implemented new decaying formula

This is a solution for issue #174, reverting the rewards calculation from being based on current SEEDS exchange price to an exponential decaying formula.

The unit test currently doesn't verify the formula results, for the reward to actually start to decrease we need about 60 residents/citizens, any idea how can I add this many residents without making the test slow to a crawl? Maybe preset the test database?

Sorry this took so long didn't realize it was done

Testet deploy log
❯ ./scripts/seeds.js run accounts history settings
Local

compiler command: eosio-cpp -abigen -I ./include -contract accounts -o ./artifacts/accounts.wasm ./src/seeds.accounts.cpp
accounts compiled
deploy accounts
creating accts.seeds

account accts.seeds already created

createAccount
abi deployed
accounts code was already deployed
compiler command: eosio-cpp -abigen -I ./include -contract history -o ./artifacts/history.wasm ./src/seeds.history.cpp
history compiled
deploy history
creating histry.seeds

account histry.seeds already created

createAccount
abi deployed
history code was already deployed
compiler command: eosio-cpp -abigen -I ./include -contract settings -o ./artifacts/settings.wasm ./src/seeds.settings.cpp
settings compiled
deploy settings
creating settgs.seeds

account settgs.seeds already created

createAccount
abi deployed
settings code was already deployed

~/work/seeds/seeds-contracts master 27s
❯ cleos push action settgs.seeds reset '{}' -p settgs.seeds@active
executed transaction: a869deecde63cbef312fe0d862551bc12e5665f14348004d7a540b9476f449c9  96 bytes  4313 us
settgs.seeds <= settgs.seeds::reset          ""
warning: transaction executed locally, but may not be confirmed by the network yet         ]
~/work/seeds/seeds-contracts master
❯ unlockcleoswallet.sh
Error 3120007: Already unlocked
Error Details:
Wallet is already unlocked: default
~/work/seeds/seeds-contracts master
❯ cleos push action histry.seeds migrate '{}' -p histry.seeds@active
executed transaction: 8683988e9904e2cfbee98d4735534f6ae2c0742427f3ba967340657799ac9351  96 bytes  340 us
histry.seeds <= histry.seeds::migrate        ""
warning: transaction executed locally, but may not be confirmed by the network yet         ]
~/work/seeds/seeds-contracts master
❯ ./scripts/seeds.js test accounts
Local
Local

accounts
reset proposals
reset accounts
reset token stats
reset settings
reset exchange
reset escrow
add users
filling account with Seedds for bonuses [Change this]
plant 50 seeds
plant 100 seeds
add referral firstuser is referrer for seconduser
update reputation
update user data
make resident
user not ready to become citizen{"code":500,"message":"Internal Service Error","error":{"code":3050003,"name":"eosio_assert_message_exception","what":"eosio_assert_message assertion failure","details":[{"message":"assertion failure with message: resident: user has less than required transactions number has: 0 needed: 1","file":"wasm_interface.cpp","line_number":1075,"method":"eosio_assert"}]}}
test citizen    ✔  Given not citizen: should cant vote
✔  Given citizen: should can vote

test resident    ✔  Given resident: should cant vote

test citizen again
test resident
test citizen second user
test testremove    ✔  Given referred became resudent: should reveive Seeds in escrow
✔  Given referred became citizen: should reveive Seeds in escrow
✔  Given changed reputation: should have correct values
✔  Given invited became resudent: should have community building score
✔  Given invited became citizen: should have community building score
✔  Given created user: should have correct timestamp
✔  Given invited user: should have row in table
✔  Given users table: should show joined users
✔  Given test-removed user: should have 1 fewer users than before

vouching
✖  AssertionError [ERR_ASSERTION]: Missing ABI for account: harvst.seeds
-------------------------------------------------------------------------
operator: deepEqual
diff:
source: at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:93:5)
vouching with reputation
✖  AssertionError [ERR_ASSERTION]: Missing ABI for account: harvst.seeds
-------------------------------------------------------------------------
operator: deepEqual
diff:
source: at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:93:5)
Ambassador and Org rewards
Ambassador and Org rewards =====    ✖  AssertionError [ERR_ASSERTION]: Missing ABI for account: orgs.seeds
-----------------------------------------------------------------------
    operator: deepEqual
    diff:
    source: at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

Proportional rewards
✖  AssertionError [ERR_ASSERTION]: Missing ABI for account: orgs.seeds
-----------------------------------------------------------------------
operator: deepEqual
diff:
source: at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:93:5)
make resident
reset accounts
reset history
reset exchange
reset token stats
add users
make resident - fail
can resident - fail    ✔  Given does not fulfill criteria for resident: should be visitor
✔  Given does not fulfill criteria for resident - canresident is false: should be visitor

plant 50 seeds
make 10 transactions
add referral
update reputation
can resident
make resident    ✔  Given fulfills criteria for resident: should is resident

make citizen
✖  AssertionError [ERR_ASSERTION]: Missing ABI for account: harvst.seeds
-------------------------------------------------------------------------
operator: deepEqual
diff:
source: at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:93:5)
reputation & cbs ranking
reset accounts
add users    ✔  Given 3 users with rep: should have entries in rep table
✔  Given 4 users with cbs: should have entries in cbs table
✔  Given cbs ranked in one go: should have same order
✔  Given 3 users with rep: should have number in sizes table
✔  Given 4 users with cbs: should have number in sizes table
✔  Given 4 users total: should have 4 in sizes table
✔  Given removed rep: should have entries in rep table
✔  Given removed rep from first user: should had 100, minus 2 is 98
✔  Given removed rep: should have number in sizes table

passed: 24,  failed: 5  of 29 tests  (17.6s)

~/work/seeds/seeds-contracts master 18s
❯ ./scripts/seeds.js run harvest
Local

compiler command: eosio-cpp -abigen -I ./include -contract harvest -o ./artifacts/harvest.wasm ./src/seeds.harvest.cpp
harvest compiled
deploy harvest
creating harvst.seeds

account harvst.seeds already created

createAccount
abi deployed
harvest code was already deployed

~/work/seeds/seeds-contracts master 12s
❯ ./scripts/seeds.js test accounts
Local
Local

accounts
reset proposals
reset accounts
reset token stats
reset settings
reset exchange
reset escrow
add users
filling account with Seedds for bonuses [Change this]
plant 50 seeds
plant 100 seeds
add referral firstuser is referrer for seconduser
update reputation
update user data
make resident
make citizen
user not ready to become citizen{"code":500,"message":"Internal Service Error","error":{"code":3050003,"name":"eosio_assert_message_exception","what":"eosio_assert_message assertion failure","details":[{"message":"assertion failure with message: user has not referred enough residents or citizens: 0","file":"wasm_interface.cpp","line_number":1075,"method":"eosio_assert"}]}}
test citizen    ✔  Given not citizen: should cant vote
✔  Given citizen: should can vote

test resident    ✔  Given resident: should cant vote

test citizen again
test resident
test citizen second user
test testremove    ✔  Given referred became resudent: should reveive Seeds in escrow
✔  Given referred became citizen: should reveive Seeds in escrow
✔  Given changed reputation: should have correct values
✔  Given invited became resudent: should have community building score
✔  Given invited became citizen: should have community building score
✔  Given created user: should have correct timestamp
✔  Given invited user: should have row in table
✔  Given users table: should show joined users
✔  Given test-removed user: should have 1 fewer users than before

vouching
reset accounts
reset harvest
reset token stats
reset settings
reset exchange
add users
test citizen
request vouch from user    ✔  Given init: should be empty XX

vouch for user    ✔  Given after vouching: should get rep bonus for being vouched

vouch for user but not resident or citizen
test resident    ✔  Given after user is resident: should sponsor gets rep bonus for sponsoring resident
✔  Given resident vouch: should rep bonus
✔  Given after user is resident: should all sponsors gets rep bonus
✔  Given after user is citizen: should all sponsors gets rep bonus

set max vouch to 3    ✔  Given max vouch reached: should can still vouch

err expected {"code":500,"message":"Internal Service Error","error":{"code":3050003,"name":"eosio_assert_message_exception","what":"eosio_assert_message assertion failure","details":[{"message":"assertion failure with message: user is already fully vouched!","file":"wasm_interface.cpp","line_number":1075,"method":"eosio_assert"}]}}    ✔  Given vouch sponsor is not resident or citizen: should not be able to vouch
✔  Given vouch a second time: should not be able to vouch
✔  Given vouch past max points: should not be able to vouch

vouching with reputation
reset accounts
reset harvest
reset settings
reset exchange
add users
test citizen    ✔  Given init: should be empty

vouch for user    ✔  Given after vouching: should get rep bonus for being vouched

Ambassador and Org rewards
Ambassador and Org rewards =====    ✖  AssertionError [ERR_ASSERTION]: Missing ABI for account: orgs.seeds
-----------------------------------------------------------------------
    operator: deepEqual
    diff:
    source: at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

Proportional rewards
✖  AssertionError [ERR_ASSERTION]: Missing ABI for account: orgs.seeds
-----------------------------------------------------------------------
operator: deepEqual
diff:
source: at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:93:5)
make resident
reset accounts
reset history
reset exchange
reset token stats
add users
make resident - fail
can resident - fail    ✔  Given does not fulfill criteria for resident: should be visitor
✔  Given does not fulfill criteria for resident - canresident is false: should be visitor

plant 50 seeds
make 10 transactions
add referral
update reputation
can resident
make resident    ✔  Given fulfills criteria for resident: should is resident

make citizen
reset settings
reset harvest
reset accounts
reset token stats
add users
make citizen - fail
can citizen - fail    ✔  Given does not fulfill criteria for citizen: should be visitor
✔  Given does not fulfill criteria for citizen - cancitizen false : should be false

make citizen - fail    ✔  Given does not fulfill criteria for citizen: should be resident

plant 200 seeds
make 50 transaction
add 3 referrals
update reputation SCORE    ✔  Given does not fulfill criteria for citizen: should is resident still
✔  Given does not fulfill criteria for citizen: should is resident still

can citizen - should succeed
make citizen    ✔  Given does fulfill criteria for citizen: should is citizen

reputation & cbs ranking
reset accounts
add users    ✔  Given 3 users with rep: should have entries in rep table
✔  Given 4 users with cbs: should have entries in cbs table
✔  Given cbs ranked in one go: should have same order
✔  Given 3 users with rep: should have number in sizes table
✔  Given 4 users with cbs: should have number in sizes table
✔  Given 4 users total: should have 4 in sizes table
✔  Given removed rep: should have entries in rep table
✔  Given removed rep from first user: should had 100, minus 2 is 98
✔  Given removed rep: should have number in sizes table

passed: 42,  failed: 2  of 44 tests  (40s)

~/work/seeds/seeds-contracts master 40s
❯ ./scripts/seeds.js run organizations
Local

compile failed for organizations error: Error: Contract not found: organizations No source file: ./src/seeds.organizations.cpp
error deploying
TypeError: Cannot read property 'name' of undefined
at deploy (/Users/nik/work/seeds/seeds-contracts/scripts/deploy.command.js:10:35)
at async deployAction (/Users/nik/work/seeds/seeds-contracts/scripts/seeds.js:44:7)
at async runAction (/Users/nik/work/seeds/seeds-contracts/scripts/seeds.js:85:3)
at async batchCallFunc (/Users/nik/work/seeds/seeds-contracts/scripts/seeds.js:95:5)
at async Command. (/Users/nik/work/seeds/seeds-contracts/scripts/seeds.js:138:5)

~/work/seeds/seeds-contracts master
❯ ./scripts/seeds.js run organization
Local

compiler command: eosio-cpp -abigen -I ./include -contract organization -o ./artifacts/organization.wasm ./src/seeds.organization.cpp
organization compiled
deploy organization
creating orgs.seeds

account orgs.seeds already created

createAccount
abi deployed
organization code was already deployed

~/work/seeds/seeds-contracts master 14s
❯ ./scripts/seeds.js test accounts
Local
Local

accounts
reset proposals
reset accounts
reset token stats
reset settings
reset exchange
reset escrow
add users
filling account with Seedds for bonuses [Change this]
plant 50 seeds
plant 100 seeds
add referral firstuser is referrer for seconduser
update reputation
update user data
make resident
make citizen
user not ready to become citizen{"code":500,"message":"Internal Service Error","error":{"code":3050003,"name":"eosio_assert_message_exception","what":"eosio_assert_message assertion failure","details":[{"message":"assertion failure with message: user has not referred enough residents or citizens: 0","file":"wasm_interface.cpp","line_number":1075,"method":"eosio_assert"}]}}
test citizen    ✔  Given not citizen: should cant vote
✔  Given citizen: should can vote

test resident    ✔  Given resident: should cant vote

test citizen again
test resident
test citizen second user
test testremove    ✔  Given referred became resudent: should reveive Seeds in escrow
✔  Given referred became citizen: should reveive Seeds in escrow
✔  Given changed reputation: should have correct values
✔  Given invited became resudent: should have community building score
✔  Given invited became citizen: should have community building score
✔  Given created user: should have correct timestamp
✔  Given invited user: should have row in table
✔  Given users table: should show joined users
✔  Given test-removed user: should have 1 fewer users than before

vouching
reset accounts
reset harvest
reset token stats
reset settings
reset exchange
WARN: 10 attempts to find canonical signature
add users
test citizen
request vouch from user    ✔  Given init: should be empty XX

vouch for user    ✔  Given after vouching: should get rep bonus for being vouched

vouch for user but not resident or citizen
test resident    ✔  Given after user is resident: should sponsor gets rep bonus for sponsoring resident
✔  Given resident vouch: should rep bonus
✔  Given after user is resident: should all sponsors gets rep bonus
✔  Given after user is citizen: should all sponsors gets rep bonus

set max vouch to 3    ✔  Given max vouch reached: should can still vouch

err expected {"code":500,"message":"Internal Service Error","error":{"code":3050003,"name":"eosio_assert_message_exception","what":"eosio_assert_message assertion failure","details":[{"message":"assertion failure with message: user is already fully vouched!","file":"wasm_interface.cpp","line_number":1075,"method":"eosio_assert"}]}}    ✔  Given vouch sponsor is not resident or citizen: should not be able to vouch
✔  Given vouch a second time: should not be able to vouch
✔  Given vouch past max points: should not be able to vouch

vouching with reputation
reset accounts
reset harvest
reset settings
reset exchange
add users
test citizen    ✔  Given init: should be empty

vouch for user    ✔  Given after vouching: should get rep bonus for being vouched

Ambassador and Org rewards
Ambassador and Org rewards =====
reset contracts
add user
ambassador invites a user named testorgowner
     Invite Secret: 354a41544c76474d674b6a4d6e4c4c48424864626a61707952376d595873594a
     Secret hash: a60bda0a4a8d0286b3a7d6ceea004a3e41e5a31c917a3d60af2851112cfce7dd
token.seeds.transfer from seedsuseraaa to join.seeds (800.0000 SEEDS)
join.seeds.invite from seedsuseraaa
user accepts the invite and becomes a Seeds user
join.seeds.accept invite for testorgowner
accept success!
user sends funds to orgs.seeds contract
user creates org
user sends funds to newly created org
org signs up people
org invites 2 users
     Invite Secret: 354b5470634a717a5757455938724c77476638705a5366464a574d3337775657
     Secret hash: fd0bb87177253851a6eebf7b3e817ac83e685c4615fc24ee3d8d351f226b9690
token.seeds.transfer from testorg11111 to join.seeds (50.0000 SEEDS)
join.seeds.invite from testorg11111
     Invite Secret: 354b6863507759373658464d484b796b58536b3555596b44444373525931386b
     Secret hash: 086f2827ee1006e1188a302575e774ed64ab74b104b940a01a9a16f7123b11de
token.seeds.transfer from testorg11111 to join.seeds (100.0000 SEEDS)
join.seeds.invite from testorg11111
join.seeds.accept invite for seedsorgusr1
accept success!
join.seeds.accept invite for seedsorgusr2
accept success!
user 1 becomes resident    ✔  Given after resident: should reveive Seeds in escrow

user 2 becomes citizen    ✔  Given after citizen 1: should reveive Seeds in escrow

user 1 becomes citizen    ✔  Given after citizen 2: should reveive Seeds in escrow

Proportional rewards
reset contracts
set exchange price
add user
individual invites a user named invited
     Invite Secret: 354a6b6f434b36324e6a4a35574c6473684456374b766e4b334e37476b6d4a6a
     Secret hash: 4944b67801c2cbbb4225c12964f81b125e4591c88f0680c294ad95314f6eb2b6
token.seeds.transfer from seedsuseraaa to join.seeds (800.0000 SEEDS)
join.seeds.invite from seedsuseraaa
user accepts the invite and becomes a Seeds user
join.seeds.accept invite for invited
accept success!
change settings
user becomes resident    ✔  Given after resident: should receive Seeds in escrow

user becomes citizen    ✔  Given after citizen: should receive Seeds in escrow

make resident
reset accounts
reset history
reset exchange
reset token stats
add users
make resident - fail
can resident - fail    ✔  Given does not fulfill criteria for resident: should be visitor
✔  Given does not fulfill criteria for resident - canresident is false: should be visitor

plant 50 seeds
make 10 transactions
add referral
update reputation
can resident
make resident    ✔  Given fulfills criteria for resident: should is resident

make citizen
reset settings
reset harvest
reset accounts
reset token stats
add users
make citizen - fail
can citizen - fail    ✔  Given does not fulfill criteria for citizen: should be visitor
✔  Given does not fulfill criteria for citizen - cancitizen false : should be false

make citizen - fail    ✔  Given does not fulfill criteria for citizen: should be resident

plant 200 seeds
make 50 transaction
add 3 referrals
update reputation SCORE    ✔  Given does not fulfill criteria for citizen: should is resident still
✔  Given does not fulfill criteria for citizen: should is resident still

can citizen - should succeed
make citizen    ✔  Given does fulfill criteria for citizen: should is citizen

reputation & cbs ranking
reset accounts
add users    ✔  Given 3 users with rep: should have entries in rep table
✔  Given 4 users with cbs: should have entries in cbs table
✔  Given cbs ranked in one go: should have same order
✔  Given 3 users with rep: should have number in sizes table
✔  Given 4 users with cbs: should have number in sizes table
✔  Given 4 users total: should have 4 in sizes table
✔  Given removed rep: should have entries in rep table
✔  Given removed rep from first user: should had 100, minus 2 is 98
✔  Given removed rep: should have number in sizes table

passed: 47,  failed: 0  of 47 tests  (56s)

~/work/seeds/seeds-contracts master 57s
❯ ./scripts/seeds.js test history
Local
Local

make a transaction entry
history reset
accounts reset
update status
add transaction entry
transactions result [
  {
    "id": 0,
    "to": "seedsuserbbb",
    "quantity": "10.0000 SEEDS",
    "timestamp":<PHONE_NUMBER>
  }
]    ✔  Given transactions table: should have transaction entry

make a history entry
history reset
history make entry
now time 16074116
check that history table has the entry
timestamp<PHONE_NUMBER>    ✔  Given action was tracked: should have table entry
✔  Given action was tracked: should timestamp is kinda close

add resident
add citizen    ✔  Given add resident entry: should have resident row
✔  Given add citizen entry: should have resident row

org transaction entry
history reset
accounts reset
settings reset
org reset
token reset weekly
create org, users
create organization
add transaction entry from user to org    ✔  Given transactions table: should no change when org transact
✔  Given org tx table: should have 2 tx entry
✔  Given transactions table: should have transaction entry
✔  Given org transactions table: should have transaction entry
✔  Given org transactions table 2: should have transaction entries for both in and out

passed: 10,  failed: 0  of 10 tests  (7s)

~/work/seeds/seeds-contracts master 8s
❯ ./scripts/seeds.js test settings
Local
Local

settings
✔  Given reset settings: should have config items with params
✔  Given reset settings: should have contract items with accounts
✔  Given set a value: should have value
passed: 3,  failed: 0  of 3 tests  (812ms)

~/work/seeds/seeds-contracts master
❯ EOSIO_NETWORK=telosTestnet ./scripts/seeds.js run accounts history settings
Telos Testnet

compiler command: eosio-cpp -abigen -I ./include -contract accounts -o ./artifacts/accounts.wasm ./src/seeds.accounts.cpp
accounts compiled
deploy accounts
creating accts.seeds

account accts.seeds already created

createAccount
abi deployed
Success: accounts deployed to accts.seeds
accounts deployed
compiler command: eosio-cpp -abigen -I ./include -contract history -o ./artifacts/history.wasm ./src/seeds.history.cpp
history compiled
deploy history
creating histry.seeds

account histry.seeds already created

createAccount
abi deployed
error deploying
{"code":500,"message":"Internal Service Error","error":{"code":3080001,"name":"ram_usage_exceeded","what":"Account using more than allotted RAM usage","details":[{"message":"account histry.seeds has insufficient ram; needs 752020 bytes has 701398 bytes","file":"resource_limits.cpp","line_number":215,"method":"verify_account_ram_usage"}]}}
compiler command: eosio-cpp -abigen -I ./include -contract settings -o ./artifacts/settings.wasm ./src/seeds.settings.cpp
settings compiled
deploy settings
creating settgs.seeds

account settgs.seeds already created

createAccount
abi deployed
Success: settings deployed to settgs.seeds
settings deployed

~/work/seeds/seeds-contracts master 1m 11s
❯ cleos -u https://test.hypha.earth/ system buyram -k illumination histry.seeds 1000
Error 3120003: Locked wallet
Ensure that your wallet is unlocked before using it!
Error Details:
You don't have any unlocked wallet!
~/work/seeds/seeds-contracts master 6s
❯ unlockcleoswallet.sh
Unlocked: default
~/work/seeds/seeds-contracts master
❯ cleos -u https://test.hypha.earth/ system buyram -k s33dst3stn3t histry.seeds 1000
Error 3050003: eosio_assert_message assertion failure
Error Details:
assertion failure with message: overdrawn balance
pending console output:
~/work/seeds/seeds-contracts master 7s
❯ cleos -u https://test.hypha.earth/ system buyram -k seeds histry.seeds 1000
executed transaction: 43566f22527fdc34d17f8be2af214824fb5b7772915a4c45239b605b4ed06b96  112 bytes  1596 us
eosio <= eosio::buyrambytes           {"payer":"seeds","receiver":"histry.seeds","bytes":1024000}
eosio.token <= eosio.token::transfer        {"from":"seeds","to":"eosio.ram","quantity":"294.8069 TLOS","memo":"buy ram"}
eosio.token <= eosio.token::transfer        {"from":"seeds","to":"eosio.ramfee","quantity":"1.4815 TLOS","memo":"ram fee"}
eosio.token <= eosio.token::transfer        {"from":"eosio.ramfee","to":"eosio.rex","quantity":"1.4815 TLOS","memo":"transfer from eosio.ramfee ...
seeds <= eosio.token::transfer        {"from":"seeds","to":"eosio.ram","quantity":"294.8069 TLOS","memo":"buy ram"}
eosio.ram <= eosio.token::transfer        {"from":"seeds","to":"eosio.ram","quantity":"294.8069 TLOS","memo":"buy ram"}
eosio.trail <= eosio.token::transfer        {"from":"seeds","to":"eosio.ram","quantity":"294.8069 TLOS","memo":"buy ram"}
seeds <= eosio.token::transfer        {"from":"seeds","to":"eosio.ramfee","quantity":"1.4815 TLOS","memo":"ram fee"}
eosio.ramfee <= eosio.token::transfer        {"from":"seeds","to":"eosio.ramfee","quantity":"1.4815 TLOS","memo":"ram fee"}
eosio.trail <= eosio.token::transfer        {"from":"seeds","to":"eosio.ramfee","quantity":"1.4815 TLOS","memo":"ram fee"}
eosio.ramfee <= eosio.token::transfer        {"from":"eosio.ramfee","to":"eosio.rex","quantity":"1.4815 TLOS","memo":"transfer from eosio.ramfee ...
eosio.rex <= eosio.token::transfer        {"from":"eosio.ramfee","to":"eosio.rex","quantity":"1.4815 TLOS","memo":"transfer from eosio.ramfee ...
eosio.trail <= eosio.token::transfer        {"from":"eosio.ramfee","to":"eosio.rex","quantity":"1.4815 TLOS","memo":"transfer from eosio.ramfee ...
warning: transaction executed locally, but may not be confirmed by the network yet         ]
~/work/seeds/seeds-contracts master 6s
❯ EOSIO_NETWORK=telosTestnet ./scripts/seeds.js run history
Telos Testnet

compiler command: eosio-cpp -abigen -I ./include -contract history -o ./artifacts/history.wasm ./src/seeds.history.cpp
history compiled
deploy history
creating histry.seeds

account histry.seeds already created

createAccount
abi deployed
Success: history deployed to histry.seeds
history deployed

~/work/seeds/seeds-contracts master 28s
❯ cleos -u https://test.hypha.earth push action settgs.seeds reset '{}' -p settgs.seeds@active
executed transaction: e00304b14b09e2dbe4326ee8ce35546abd857d094d275a6dc9d0e6f1ae6d9e19  96 bytes  5475 us
settgs.seeds <= settgs.seeds::reset          ""
warning: transaction executed locally, but may not be confirmed by the network yet         ]
~/work/seeds/seeds-contracts master 6s
❯ cleos -u https://test.hypha.earth get table histry.seeds histry.seeds sizes
{
"rows": [],
"more": false,
"next_key": ""
}
~/work/seeds/seeds-contracts master
❯ cleos -u https://test.hypha.earth push action histry.seeds migrate '{}' -p histry.seeds@active
executed transaction: d38661b7f81a65cb934c2ea1999b4de2d94b70c473ac974074df72c22125da80  96 bytes  1189 us
histry.seeds <= histry.seeds::migrate        ""
warning: transaction executed locally, but may not be confirmed by the network yet         ]
~/work/seeds/seeds-contracts master 8s
❯ cleos -u https://test.hypha.earth get table histry.seeds histry.seeds sizes
{
"rows": [{
"id": "citizens.sz",
"size": 8
},{
"id": "regens.sz",
"size": 0
},{
"id": "reptables.sz",
"size": 0
},{
"id": "residents.sz",
"size": 4
}
],
"more": false,
"next_key": ""
}
~/work/seeds/seeds-contracts master
❯

Mainnet deploy

❯ EOSIO_NETWORK=telosMainnet ./scripts/seeds.js deploy settings

Telos Mainnet
+  deploy settings
+  creating settgs.seeds
-  account settgs.seeds already created
+  createAccount
+  abi deployed
+  Success: settings deployed to settgs.seeds
+  settings deployed

~/work/seeds/seeds-contracts master 19s
❯ cleos -u https://node.hypha.earth push action settgs.seeds reset '{}' -p settgs.seeds@active

executed transaction: c7ff30a0eda2a6fbd9182205aa7eb2740b277e4825c0951c46024fae147c20fc  96 bytes  5544 us
#  settgs.seeds <= settgs.seeds::reset          ""
warning: transaction executed locally, but may not be confirmed by the network yet         ]

~/work/seeds/seeds-contracts master
❯ EOSIO_NETWORK=telosMainnet ./scripts/seeds.js deploy accounts history

Telos Mainnet
+  deploy accounts
+  creating accts.seeds
-  account accts.seeds already created
+  createAccount
+  abi deployed
+  Success: accounts deployed to accts.seeds
+  accounts deployed
+  deploy history
+  creating histry.seeds
-  account histry.seeds already created
+  createAccount
+  abi deployed
+  Success: history deployed to histry.seeds
+  history deployed

~/work/seeds/seeds-contracts master 36s
❯ cleos -u https://node.hypha.earth get table histry.seeds histry.seeds sizes

{
  "rows": [],
  "more": false,
  "next_key": ""
}

~/work/seeds/seeds-contracts master
❯ cleos -u https://node.hypha.earth push action histry.seeds migrate '{}' -p histry.seeds@active

executed transaction: 846426f622fc59f28bf048a8ace7a38ab86403a517bc7e0e01342735dd4dccbe  96 bytes  4279 us
#  histry.seeds <= histry.seeds::migrate        ""
warning: transaction executed locally, but may not be confirmed by the network yet         ]

~/work/seeds/seeds-contracts master 6s
❯ cleos -u https://node.hypha.earth get table histry.seeds histry.seeds sizes

{
  "rows": [{
      "id": "citizens.sz",
      "size": 70
    },{
      "id": "regens.sz",
      "size": 0
    },{
      "id": "reptables.sz",
      "size": 0
    },{
      "id": "residents.sz",
      "size": 143
    }
  ],
  "more": false,
  "next_key": ""
}

~/work/seeds/seeds-contracts master
❯

@juliolrmonteiro I think the number of citizens is incorrect here - maybe history tables don't have a complete history of citizens (genesis citizens?)
Or were some demoted?

Yes I think it's just because initial citizens aren't in that table
cleos -u https://node.hypha.earth get table histry.seeds histry.seeds citizens --lower illumination
{
"rows": [],
"more": false,
"next_key": ""
}

In the long run, I don't think this discrepancy of 30 odd people makes a difference
What about demoted citizens @juliolrmonteiro ? Since we now demote them... I guess we should ignore that... which this table does, it doesn't update when citizens are demoted, these tables are meant to simply store the date someone became a citizen


Github Eviction Policy

I have a question regarding the retention of the benchmark result.
As far as I understood the code correctly then benchmark results are stored in ./cache/benchmark-data.json and then loaded from the github cache on the next run. I think this is a tricky idea.
But a github cache is only kept for 7 days when it is not touched anymore (see here).
So benchmark result are gone when the benchmark is not executed within these 7 days?

Using gh-page branch could address this concern.

Closing as the question was raised 3 years ago. Feel free to reopen,


conv2d invalid shape error

Follow on to issues #14140 and #14557.
Git tag: v0.52.0
For some combinations of input sizes, a Conv weight matrix shape is invalid error is thrown.  The code below will fail.  Note that it is simply size changes from issue #14557.  What should the kernel shape be to apply the same kernel to all input channels yielding the same number of putput channels?
import torch
import ttnn

device_params = {"l1_small_size": 24576}
device = ttnn.open_device(device_id=0, **device_params)

BATCH_SIZE   = 3
IN_CHANNELS  = 32
OUT_CHANNELS = 32
MAT_SIZE     = 32
KERN_SIZE    = 3
PAD_SIZE     = 1
a = torch.ones((BATCH_SIZE, IN_CHANNELS, MAT_SIZE, MAT_SIZE), dtype=torch.float32)
b = torch.ones((OUT_CHANNELS, IN_CHANNELS, KERN_SIZE, KERN_SIZE), dtype=torch.float32)
input_tensor  = ttnn.from_torch(a, layout=ttnn.TILE_LAYOUT, device=device)
weight_tensor = ttnn.from_torch(b, layout=ttnn.TILE_LAYOUT, device=device)

res = ttnn.conv2d(input_tensor=input_tensor,
                  weight_tensor=weight_tensor,
                  device=device,
                  in_channels=IN_CHANNELS,
                  out_channels=OUT_CHANNELS,
                  batch_size=BATCH_SIZE,
                  input_height=MAT_SIZE,
                  input_width=MAT_SIZE,
                  kernel_size=(KERN_SIZE,KERN_SIZE),
                  padding=(PAD_SIZE,PAD_SIZE),
                  stride=(1,1),
                  dilation=(1,1),
                  groups=1)
out, out_height, out_width, conv_weight_tensor, conv_bias_tensor = res

print(out_height)
print(out_width)
print(out)
print(conv_weight_tensor)
print(conv_bias_tensor)
print("")

out_torch = ttnn.to_torch(out)
print(out_torch.shape)

ttnn.close_device(device)

Hi @twist-vector thanks for filing this issue.
Unfortunately our conv2d op is not as easy to use as it should be, but we are working on it :).
I've tweaked your test-case above to inline with current guidelines how conv2d should be used.
import torch
import ttnn

device_params = {"l1_small_size": 24576}
device = ttnn.open_device(device_id=0, **device_params)

BATCH_SIZE   = 3
IN_CHANNELS  = 32
OUT_CHANNELS = 32
MAT_SIZE     = 32
KERN_SIZE    = 3
PAD_SIZE     = 1
a = torch.ones((BATCH_SIZE, IN_CHANNELS, MAT_SIZE, MAT_SIZE), dtype=torch.float32)
b = torch.ones((OUT_CHANNELS, IN_CHANNELS, KERN_SIZE, KERN_SIZE), dtype=torch.bfloat16)

#input_tensor  = ttnn.from_torch(a, layout=ttnn.TILE_LAYOUT, device=device)
#weight_tensor = ttnn.from_torch(b, layout=ttnn.TILE_LAYOUT, device=device)

# torch tensor is in nchw layout, and ttnn expects nhwc layout
a_nhwc = torch.permute(a, (0, 2, 3, 1))

input_tensor = ttnn.from_torch(a_nhwc)
weight_tensor = ttnn.from_torch(b)

res = ttnn.conv2d(input_tensor=input_tensor,
                  weight_tensor=weight_tensor,
                  device=device,
                  in_channels=IN_CHANNELS,
                  out_channels=OUT_CHANNELS,
                  batch_size=BATCH_SIZE,
                  input_height=MAT_SIZE,
                  input_width=MAT_SIZE,
                  kernel_size=(KERN_SIZE,KERN_SIZE),
                  padding=(PAD_SIZE,PAD_SIZE),
                  stride=(1,1),
                  dilation=(1,1),
                  groups=1)
out, out_height, out_width, conv_weight_tensor, conv_bias_tensor = res

tt_output_tensor = ttnn.from_device(out)
torch_output_tensor = ttnn.to_torch(tt_output_tensor)

# torch_output_tensor is in row major layout and NHWC shape
# NHWC to NCHW
torch_output_tensor = torch_output_tensor.reshape(BATCH_SIZE, out_height, out_width, torch_output_tensor.shape[-1])
torch_output_tensor = torch_output_tensor[:, :, :, :OUT_CHANNELS]

torch_output_tensor = torch.permute(torch_output_tensor, (0, 3, 1, 2))

print(out_height)
print(out_width)
print(out)
print(conv_weight_tensor)
print(conv_bias_tensor)
print("")

print(torch_output_tensor.shape)

ttnn.close_device(device)

Some things to call out here compared to your code.

Float32 is not supported data type for weights on conv2d, so I've used float16.
ttnn.conv2d expects activations (input_tensor) to be in NHWC format (batch, height, width, input_channels)
Output follows the same format as input (NHWC)
Weights need to be prepared in a special way to be properly consumed by the conv2d op. In case that input tensor is on host (this is the option I'm using in the code above), conv2d will assume weights are not prepared and op will do it. If weights are on device op assumes they are prepared and doesn't do anything which can lead to bad outcomes if they are not prepared.
To get weights in "right" form, the easiest way is to run conv2d once with weights on host and op will output prepared weights along with the output tensor.

@twist-vector can we close this one?

I'm closing this one same as #14557, @twist-vector  if you disagree pls reopen the issue.


How to pass clang options to be applied to generated object file?

For example, I can compile my binary using clang -target riscv64-unknown-elf -fno-builtin-printf -fno-lto -O2 -c example.c -o example.o, how may I pass down the additional options through onnx-mlir? This might be a dumb question as I'm new to LLVM toolsets.
I tried $ onnx-mlir -mtriple=riscv64-unknown-elf -mllvm="-fno-builtin-printf -fno-lto -O2" --EmitObj ./nn.onnx:
[1/6] Mon Sep 23 15:21:19 2024 (0s) Importing ONNX Model to MLIR Module from "nn.onnx"
[2/6] Mon Sep 23 15:21:19 2024 (0s) Compiling and Optimizing MLIR Module
[3/6] Mon Sep 23 15:21:19 2024 (0s) Translating MLIR Module to LLVM and Generating LLVM Optimized Bitcode
opt: Unknown command line argument '-fno-builtin-printf -fno-lto -O2'.  Try: 'opt --help'
opt: Did you mean '--rotation-prepare-for-lto'?
opt -O0 --mtriple=riscv64-unknown-elf --code-model small -fno-builtin-printf -fno-lto -O2 -o ./nn.bc ./nn.unoptimized.bc
Error message:
Program path: ~/git_repo/llvm-project/build/bin/opt
Command execution failed.

Seems only a single option is allowed/supported.
$~/experimental/onnx-mlir-example$ onnx-mlir -mtriple=riscv64-unknown-elf -mllvm="--disable-simplify-libcalls -O2" --EmitObj ./nn.onnx
[1/6] Mon Sep 23 15:35:49 2024 (0s) Importing ONNX Model to MLIR Module from "nn.onnx"
[2/6] Mon Sep 23 15:35:49 2024 (0s) Compiling and Optimizing MLIR Module
[3/6] Mon Sep 23 15:35:49 2024 (0s) Translating MLIR Module to LLVM and Generating LLVM Optimized Bitcode
opt: Unknown command line argument '--disable-simplify-libcalls -O2'.  Try: 'opt --help'
opt: Did you mean '--disable-simplify-libcalls'?
opt -O0 --mtriple=riscv64-unknown-elf --code-model small --disable-simplify-libcalls -O2 -o ./nn.bc ./nn.unoptimized.bc
Error message:
Program path: ~/git_repo/llvm-project/build/bin/opt
Command execution failed.
$:~/experimental/onnx-mlir-example$ onnx-mlir -mtriple=riscv64-unknown-elf -mllvm="--disable-simplify-libcalls" --EmitObj ./nn.onnx
[1/6] Mon Sep 23 15:36:40 2024 (0s) Importing ONNX Model to MLIR Module from "nn.onnx"
[2/6] Mon Sep 23 15:36:40 2024 (0s) Compiling and Optimizing MLIR Module
[3/6] Mon Sep 23 15:36:40 2024 (0s) Translating MLIR Module to LLVM and Generating LLVM Optimized Bitcode
[4/6] Mon Sep 23 15:36:40 2024 (0s) Generating Object from LLVM Bitcode
[5/6] Mon Sep 23 15:36:40 2024 (0s) Compilation completed

I have seen the system being fragile with multiple options in a string, not entirely sure why.
One workaround is to add support for multiple instance of -mllvm-.
Do you want me to add this feature?

I have seen the system being fragile with multiple options in a string, not entirely sure why. One workaround is to add support for multiple instances of -mllvm=<single option>.
Do you want me to add this feature?

That would be nice. Thanks.

The reason is that strings have to be passed separately in the argv[][] for the command. There is another small issue for the example here: -O0 is created by default. If -O2 is passed the mllvm, I got error for "multiple -O#".
I am going create a PR to fix this issue of multiple options in mllvm.

Thanks @chentong319

@jinchenglee Which step do you intend to use these options, opt or llc? The current implementation has another issue is that the -mllvm is used in both opt and llc. But they may support different options. I need to introduce new options for onnx-mlir.

@jinchenglee There are three options in onnx-mlir to pass options to opt and/or llc

mllvm: only one is used. passed to both opt and llc
Xopt: multiple is allowed. passed to only opt
Xllc: multiple is allowed. passed to only llc

I merged a new PR to handle the issue with mllvm. I added example of how to use Xopt and Xllc. Hope this PR2955 solved your issue.

@jinchenglee can we close this issue?

@jinchenglee can we close this issue?

Sorry, I don't yet have time to try these yet. But for what this ticket is asking for, it's safe to close. If any issues, I or anyone can reopen it. Thanks.


New package: qdsim v0.5.0

Registering package: qdsim
Repository: https://github.com/amartyabose/qdsim
Created by: @amartyabose
Version: v0.5.0
Commit: 88e215246abb802dbc918f14b95a6d6e431d262c
Reviewed by: @amartyabose
Reference: https://github.com/amartyabose/qdsim/commit/88e215246abb802dbc918f14b95a6d6e431d262c#commitcomment-147809733
Description: Simulations using QuantumDynamics.jl made a breeze

@goerz --- Yes, sorry... While I have released Julia packages, I got confused because this uses the Comonicon.jl package to give a command line utility.  I'll upload the code to another repository with proper naming, address the other issues and open a registration request.

I assume this was opened by mistake

No problem… But I think it should be possible to use Comonicon to make this utility an optional part of your existing QuantumDynamics package, see https://github.com/JuliaRegistries/General/pull/117132#issuecomment-2408564756. That would be a much cleaner solution. Otherwise, I think you should make use of the QuantumDynamics "namespace" (I consider an author of Package to "own" the names PackageSomething) instead of using a new package name that will only confuse your users, and pollute the General namespace


Permission checking

From the feedback:

permission checking things should occur in controllers, not models.  (Some validation etc. is good, but permissions are probably too much)

Removed checking from models and claimList


gulp.dest() does not close files when using from watch()

I needed to create new issue since @phated closed my previous issue
What were you expecting to happen?
I'm running gulp.watch() locally to recompile static files in public/dist directory.
I also watch public/dist directory with Skaffold to propagate changes to containers in my Kubernetes cluster.
What actually happened?
File changes does not get propagated when files are changed by gulp, but they propagated when I change them by some text redactor.
I traced gulp process and looks like gulp does not close files when writes to them.
I searched for some option in gulp.dest() but did not find anything.
Also interesting, that tasks, which are triggered from watch(), work good and close files when they are triggered not inside watch().
Please post a sample of your gulpfile (preferably reduced to just the bit that's not working)
This task is running from .gulp.watch() task
gulp.task('myapp.js', () => {
  return gulp.src(SRC_PATH + '/myapp.app.js')
    .pipe(plumber())
    .pipe(bro({
      transform: [
        babelify,
        [browserifyCss, { global: true }]
      ]
    }))
    .pipe(rename('myapp.min.js'))
    .pipe(base64())
    .pipe(development(sourcemaps.init({ loadMaps: true })))
    .pipe(production(ngAnnotate()))
    .pipe(production(uglify()))
    .pipe(development(sourcemaps.write('.')))
    .pipe(production(rev()))
    .pipe(assetManifest({
      bundleName: 'myapp-scripts',
      log: development() && !isWatching,
      manifestFile: MANIFEST_FILE,
      pathPrepend: MANIFEST_PREPEND_PATH
    }))
    .pipe(gulp.dest(getTenantAssetsPath('myapp')))
});

What version of gulp are you using?
4.0.2
What versions of npm and node are you using?
NPM: 6.11.3
Node: v10.17.0
OS: MacOs Catalina

Depending on what watcher you are using, you might need to update your fs.Stats on the files you are creating. https://gulpjs.com/docs/en/api/dest#metadata-updates

If nothing in your pipeline updates fs.Stats, we will create a file with the same timestamp as the source inputs.

@phated after some research, I realized that skaffold uses golang lib fsnotify, which tries to open file in exclusive mode before notifying about file change.
Is there some possibility in gulp to not keep files open during watch?

Hi,
I have a similar issue that I'm not able to solve (only on Gulp 4, Gulp 3 works correctly). My use case is pretty similar: I have one watcher that is doing some processing, and copying the result file into a different folder. Then I have another watcher that watches for changes in this directory. If I manually change the file in text editor, the wathcer works. However if the file is changed by the first watch task, then the second watch does not detect the change. It looks like this:
function css() {
  return src('css/theme.css')
    // Do stuff and copy result to another folder
    .pipe(dest('dist/assets'));
}

exports.default = () => {
  watch('css/**/*.css', css);
  watch('dist/assets/theme.css', () => {
    // Only outputted if I manually edit the "dist/assets/theme.css" file, but never when the file is changed by the first task
    console.log('foo');
  });
};

What are the options (beside downgrading to Gulp 3) to achieve that @phated ?


Shif particular bank holidays

Hello Marco I need to add "special" days that are not banks holidays (Mother's Day, Grandmother's Day, ...).
Most of time these days fall on Sundays.
The problem is when we have to deal with exceptions.
In France, Mother's Day is celebrated on the last Sunday of May
{ "FR", 6, 0, -7 ,1, 0, "Mothers Day", 0, 100, 0, 0 }
Unless this day is the same as the Whit sunday then we have to shoft to the first sunday of June !!!
Have you any hint or solution ? Many thanks

You can avoid two holidays on the same day with the Conflict Priority parameter.
However, once you have Whit Sunday on the last Sunday of May, the big issue is shifting Mother's Day to the following Sunday. The current system doesn't have a way to manage that - and I think that it's too complex to implement a system that can manage that logic in a flexible way.
Probably it's better to add exceptions as single dates using FIrstYear and LastYear with the same year for the Mother's Day holidays that are in June.


Fix scope info for AddColumns / Search functions

The ScopeInfo declaration for the Search and AddColumn functions was incorrectly being applied to all arguments; it should only be applied to the ones that represent column names.

Also RenameColumns (only existing names are in scope)

✅ No public API change.

✅ No public API change.

✅ No public API change.


fix for sporadic porch crash during new package revision

Current code in package variant controller was assuming that "status.upstreamLock.Git.Ref" in PackageRevision resource will always having proper value. But status.upstreamLock field in PackageRevision could be nil in initial transition stage. Due to which Porch was crashing. Issue is fixed as part of this PR

/ok-to-test

Please also check the dates on the copyright notices on the top of the go files.

/approve

/test presubmit-nephio-go-test

/lgtm


dummy rename for action ci run

for some reasons, last action CI did not deploy latest HEAD, still need another dummy commit (and dummy PR)


2nd latency of llama3-8B-Instruct with int4 & all-in-one tool issue

the 2nd latency of llama3-8b-instruct with int4 and bs=1 is larger than bs=2, ipex-llm=2.5.0b20240504

Already reproduce the issue, and will fix it later. We recommend you use fp16 for non-linear layer, please refer to benchmark scripts all-in-one, and select transformer_int4_fp16_gpu API.


Replacing entire version tag instead of a specific matched version number

Modified the replacing part of the task so it now replaces entire tags and not just the matching version number.
It should solve Issue #176 - but it might break other things.

I've done some testing with this and it works pretty well other than one small use case, which is if I specify the field I want to update and it already exists. If you can resolve the merge conflict we're seeing by rebasing onto master (I made some other changes while this PR was around) then I'll merge this in and deal with the small use case mentioned above as I've got a fix for that working in a test branch.

Ill try solving the conflict - not entirely sure on how to go about rebasing across repos though.

Note 100% sure I did it correctly (given that all sorts of comments suddenly appeared here).

That's a little odd that it's caused all those commits to appear. I'll merge it into a test branch and see what happens.

I've done some testing and it looks good to me. I'll merge this and try to get a new release out later today.


L1 fitting issue on GS with TTNN RN50 Batch = 8, BFLOAT16, HiFi4

RN50 with TTNN runs e2e when tensors are in DRAM. With fully sharded tensors, Downsample fails with L1 fitting issue. There seems to be fragmented memory. Using reallocate/sharded_move does not help.
Moving output of conv before downsample to DRAM bypasses the issue for now.
Branch: asarje/shwetankTT/resnet
Test: pytest tests/ttnn/integration_tests/resnet/test_ttnn_functional_resnet.py::test_resnet

so far @mywoodstock has ResNet50 fps = 1000.
This reallocate is the blocker for improving perf.
help?

My take on this issue is that need to track the allocations of the tensors on the device and keep their offsets as the model is running.  With this information it will become obvious where we need to reallocate/deallocate.   At the moment folks are having to guess at where this needs to be done.  Can we flip this bug to a feature request and get this going on the next sprint?

@abhullar-tt , is it possible to easily track the allocations and deallocations here?

we have tt_lib.device.EnableMemoryReports() that generates reports of mem allocation in .reports/tt_metal dir
three reports get generated:

program_memory_usage_summary
program_l1_usage_summary
program_detailed_memory_usage

L1 usage summary would probably be helpful here

I'll try and enable that and take a look.
Also,
Is there an easy way to hook into the allocation and deallocation and throw an exception when something happens.  I think this would only be applicable as a debugging mechanism.   I was imagining using something like this class below and then you just hook into the calls that are happening...
class AllocatorTracker {
   private:
    std::map<size_t, size_t> allocations;

   public:
    void allocate(size_t offset, size_t length) {
        if (length == 0) {
            throw std::runtime_error("Cannot allocate zero length.");
        }
        auto it = allocations.lower_bound(offset);
        if (it != allocations.begin()) {
            auto prevIt = std::prev(it);
            size_t prevEnd = prevIt->first + prevIt->second;
            if (prevEnd > offset) {
                throw std::runtime_error("Allocation overlaps with an existing allocation.");
            }
        }
        if (it != allocations.end() && offset + length > it->first) {
            throw std::runtime_error("Allocation overlaps with an existing allocation.");
        }
        allocations[offset] = length;
        // std::cout << "Allocated " << length << " units at offset " << offset << std::endl;
    }
    void deallocate(size_t offset) {
        auto it = allocations.find(offset);
        if (it == allocations.end()) {
            throw std::runtime_error("No allocation found at the specified offset.");
        }
        // std::cout << "Deallocated " << it->second << " units from offset " << offset << std::endl;
        allocations.erase(it);
    }
    void printAllocations() const {
        std::cout << "Current Allocations (offset: length):" << std::endl;
        for (const auto &alloc : allocations) {
            std::cout << alloc.first << ": " << alloc.second << std::endl;
        }
    }
} track_allocations;

there are already checks to ensure that allocations don't overlap and it will throw if it hits OOM

@mywoodstock this is a P0, appears it's just sitting :)
are you working on fixing it, or someone else?
is it no longer blocking? should we re-prioritize?

yes im actively working on it

There are several issues encountered to get the full model working seamlessly with the sharding configs, @tt-nshanker , @nsmithtt and i are actively working on resolving the short comings with ttnn.

Replacing this issue with the actual bigger issue: https://github.com/tenstorrent-metal/tt-metal/issues/6035.

Replacing this issue with the actual bigger issue: #6035.

Issue #6035 is for cleaning up TTNN API. Not critical to functionality.

yeah, we are doing workarounds to bypass preprocessor

yeah, we are doing workarounds to bypass preprocessor

Should we have another issue to represent model changes to bypass preprocessor

Should we have another issue to represent model changes to bypass preprocessor

I'll just use this issue for that - will update name etc

I
Glad to here that we have a workaround.  Given the workaround, I think we can bump this down to a P1.

checkpointing: TTNN is at 60% of TTLIB now.

this checkpoint is now in main

@mywoodstock, based on our conversation, the last remaining items are:

There are L1 clash with batch 16 and 20, so need to go through op by op and move tensors around to make space
The last part of the graph is using l1 interleaved, and needs to be resharded to use width sharding like in ttlib version
Upload to demos folder

This was already achieved. Closing.


Refactor iteration over object containers

Just flying a kite here. The following are still missing:

doc comments
libtests
methods allOf, anyOf, noneOf
dictionary methods
? C-API ?

meh, this is okay, but I generally prefer for loops to higher order functions. It's a debate we have at work...the engineers who are more front-end oriented and use frameworks like Angular really like functional programming, and those of us who are more low-level, prefer explicit loops and conditionals. I'm definitely not against functional programming, especially when there can be heavy optimization under the covers, and things like map functions are pretty handy, but I don't know that forEach offers any real value. It makes a more functional style approach available to people who prefer that style, but it also adds even more stuff to qpdf's already very crowded API.
If you really felt like this would help, you could probably convince me to take it, but it's not something I feel like qpdf really needs. Just my $0.02.

That said, if (based on the comment you made to me by email), an approach like this would make possible optimizations without having to expose more internals, I would be in favor.
In other words, if this is a vehicle for clean optimization, I have no objection to this. If it's just about making a more functional style available, then I'm luke warm but would be willing to accept if you think it would be an improvement.

I am looking at this purely from a point of how we can provide the fastest way to loop over qpdf containers.
I keep looking at aitems and forEach to see which one is going to be cleaner and faster.

I have now found a version of aitems that outperforms getArrayAsVector without exposing QPDF_Array. This removes the reason for investigating the forEach approach.


error while training in multiple GPUs

hello ,
I try to train faster-RCNN model on my dataset using the following script (which is the same on your repository):
for FOLD in 1 2 3 4 5;
do
  bash tools/dist_train_partially.sh semi ${FOLD} 10 8
done

but i got the following error during the training:

any help to understand this error and solve it , please ?

请问您解决了吗？

yes, I did it . the problem was in two side ,
one related to my dataset which does not annotate by segmentation (mask) , and i used the code with segmentation by mistake.
the other side was related to connect my code with wandb. you shall have your account there in case you want to use wandb other wise make it unused in the code.


MONDO_0018866 'Aicardi-Goutieres syndrome' is incorrectly classified as a subclass of 'Moyamoya syndrome'

this is possibly a case where the identifier was maintained but the label was changed
From workshop

I think the issue is with all the children of moyamoya syndrome - Moyamoya can be a feature of some of these children but they should not be classified this way


Add sveltekit-starter-blog by Scott Spence

Add starter that was made by Scott Spence, it's a really good starter that I enjoy using. Hope that's OK.

This is great! 👍  Could you please also add the starter to sites.yml?

Done!

I noticed there's a typo in your post url: https://scottspence.com/2021/05/06/graphcms-svelte-strter. Missing an 'a' in starter. Do you want to fix that before I merge?

Wow good spot! I've asked Scott about it, Ill let you know

Yeah, the typo isn't intentional but I didn't spot it until I'd polished it already 😂
It's not had a great deal of traffic so can change it if that's preferred

He is going to leave the URL like that apparently, I'm happy to merge it it you are.

Thanks @endymion1818 for this contribution and @spences10 for fixing the URL!


reuse closure when creating and testing

HHVM makes different closures objects that don't compare to the original functions.
1) ValidationFactoryTest::testMakeMethodCreatesValidValidator
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    'foo' => Closure$ValidationFactoryTest::testMakeMethodCreatesValidValidator#3 Object ()
-    'implicit' => Closure$ValidationFactoryTest::testMakeMethodCreatesValidValidator#4 Object ()
+    'foo' => Closure$ValidationFactoryTest::testMakeMethodCreatesValidValidator Object ()
+    'implicit' => Closure$ValidationFactoryTest::testMakeMethodCreatesValidValidator#2 Object ()
 )

Instead I think this test really wants to make sure the same closure went in and came out again.

Changes Unknown when pulling 6a69b89896ee7d33ffbce9056f93e44057afeef8 on ptarjan:reuse_closure into ** on laravel:master**.


ASP.NET Web apps discovering on Hyper-V

Is it still true this statement?
Currently the discovery of ASP.NET web apps is only available with appliance used for discovery of servers running in your VMware environment.
I have tested in my lab environment using Hyper-V and I have got a discovering of my Web Apps on IIS.
Thanks
Ric
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 0cc3b734-55b6-23ac-9608-7d1acc90b0c5
Version Independent ID: 84f39499-c34d-6d48-63ad-f0dfccc0313a
Content: Discover software inventory on on-premises servers with Azure Migrate - Azure Migrate
Content Source: articles/migrate/how-to-discover-applications.md
Service: azure-migrate
GitHub Login: @Vikram1988
Microsoft Alias: vibansa

@ripom
Thanks for your feedback! We will investigate and update as appropriate.

@ripom,
Thank you for pointing it out. The note has been removed and the updated doc is available at https://learn.microsoft.com/en-gb/azure/migrate/how-to-discover-applications


render-web-deploy

Current dependencies on/for this PR:

master

PR #1   👈

This comment was auto-generated by Graphite.


Feature/new init syntax

Coverage increased (+6.6%) to 91.882% when pulling d4d86631e750a7f587c9743e11d5e9882f2b99c2 on feature/new-init-syntax into c0619fd1d7610aaee482166b9b49bb8a5f1a81b7 on master.

Coverage increased (+6.6%) to 91.882% when pulling d4d86631e750a7f587c9743e11d5e9882f2b99c2 on feature/new-init-syntax into c0619fd1d7610aaee482166b9b49bb8a5f1a81b7 on master.


HIP27: Support CBRS 5G on the Helium network

Author(s): @zer0tweets (Boris Renski, FreedomFi)
Initial PR: #133
Start Date: 2020-02-23
Category: Technical, Economic
Rendered view:
https://github.com/helium/HIP/blob/master/0027-cbrs-5g-support.md
Summary:
This proposal suggests a technical and economic mechanisms to support higher new wireless protocols on Helium network, starting with LTE and 5G in CBRS spectrum band

super exciting, unless I'm misunderstanding are there concerns about the pricing proposal costing 5x that of comcast?
"Comcast charges $10 for each $50GB"
"we initially set the price of passing 1GB of CBRS data through Helium network at $1 per GB"

$10 per 50GB is retail price for Comcast fiber/cable backhaul, not for cellular service; we just put that data point for reference to see what backhaul costs could look like and make sure they won't be more than cost Helium price for cellular

It would amazing to achieve this!!!!!! I can help with the product management / research side (I know you have your team but I'd be happy to share my knowledge).

On behalf of the @dewi-alliance I am pleased to memorialize community support for this proposal, as evidenced by supportive comments here on GitHub, on Discord,
popular support in informal polling, and support during the most recent community call. I'll be submitting a separate PR to mark this proposal approved.
I believe FreedomFi is already hard at work on the code necessary to actually impement this concept. At that point the HIP would actually become real, and the code changes go through further review, testing, and deployment.

It would be great to get some clarity on the requirements for LTE EPC's for use with this HIP. I'm aware that FreedomFI is planning on producing a product around this proposal, but can other companies or individuals also participate?

I'm interested in learning how sharing consumer grade Internet connection bandwidth via 5G Helium hotspots does not create legal problems by violating the Terms of Service. The vast majority of consumer grade Internet broadband products specifically prohibit the customer from 'reselling' the service.


Update Documentation Link

A summary of what changes this PR introduces and why they were made.
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactor
[ ] Performance
[ ] Tests
[ ] Build
[x] Documentation
[ ] Code style update
[ ] Continuous Integration
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Does this PR release a new version?

[ ] Yes
[x] No

Thanks for submitting this pull request! A maintainer will review this in the next few days and explicitly select labels so you know what's going on.
If no reviewer appears after a week, a reminder will be sent out.

Your pull request doesn't follow our contribution guidelines. Please review and correct it.

Codecov Report

Merging #13 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             master   #13   +/-   ##
======================================
  Coverage         0%    0%
  Complexity       19    19
======================================
  Files             4     4
  Lines            55    55
======================================
  Misses           55    55

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40512cc...41aa59e. Read the comment docs.

A contributor has approved this PR. A maintainer will merge this PR shortly. If it shouldn't be merged yet, please leave a comment saying so and we'll wait.
Thank you for your contribution!

Your pull request has been merged but was not assigned a bounty tier. A maintainer will assign a bounty tier to this pull request in the next few days.

Your pull request has been merged and marked as tier 6. It will earn you $0 USD.

Your pull request has been merged and marked as tier 5. It will earn you $0 USD.


Will the project support AMD GPUs on windows soon?

I have a 5700XT card and I'm forced to use my weak 1060TI card. The progress is good, but very slow.
I also have 5900X CPU.
Is there a version that can utilize my CPU better?
Thanks.

I have RX 6700 XT and I hope that RVC project will support amd cards in the windows os.


Remove beta from logo on official website

Is your feature request related to a problem?
I am kinda confused if I am on the official site or the beta version of it
Describe the solution you would like
I would like to have the pure logo of Koda without the beta sign and keep the beta sign only on the beta webpage
Describe alternatives you have considered
???
Screeenshots

Are there opened related issues?
No response

Hey, @exezbcz wdyt about this issue?

hey, thanks for reporting!
I guess hence we have a lot of moving parts even in production, I would stick with the beta labels everywhere.
It's, for example, same as with Nuxt RC versions, ppl use them in production, so there might still be some dragons, which we have from time to time, so ppl can might sense something might go wrong.
Of course we are trying our best and when we will have, let's say at least 10+ internal devs, I would be confident removing, bc in case some major fuckup, we will be able quickly react and dispatch a fix. Most of time we did even with little help, but we have still some fragile parts 😅
Plus we can think of something whenever we will be adjusting logo to have it different way, because greek symbol is probably not way to descriptive for end-user/retail audience :)


allowDropdown prop not working

Hello,
I'm trying to set the allowDropdown to false as the doc says, it should hide the dropdown arrow and the selected flag won't be clickable. However, the flag is still clickable, it hides the arrow, but I can  still click the flag and it will show the countries menu.
Any thoughts how can I accomplish this?
Thanks in advance

I'm also having this issue and I submitted a PR.

Steps to reproduce the issue:

Set allowDropdown prop to false
Flag is set on the right side (with no arrow) but I'm still able to click the flag and show country dropdown.


Higher / variable number of iterations for equity calculation

Great work first of all!
The montecarlo simulation works fine and very fast for me.
However, I do not understand why the number of iterations for equity calculation is set to 10 in the main environment (unlike e.g. the montecarlo tests with 5000 iterations):

it results in a very high variance
it can be set higher without high cost (especially for the c++ implementation)
it should be an argument imho
especially the agent that considers equity should have a good estimate (and maybe recalculate it)

Hope I did not overlook something, otherwise it should be an easy fix

Can you post a link to the code you're referring to? tx

sure:
https://github.com/dickreuter/neuron_poker/blob/master/gym_env/env.py#L269

Yes I agree, it should be set to around 5000.


Ancient libraries

On OS X, this is organized with external third party libraries.  The SDL2 is ancient and won't compile properly on OS X.  Probably other libraries are old too.

Honestly, the sdl2 library should use autoconf to build.  The cmake build is busted or at least totally not documented.  I can't get it to compile properly.

https://github.com/Homebrew/homebrew-core/blob/a38d7e99bc103e9564208f2d155701d70001c8cc/Formula/s/sdl2.rb


Exercises-w1

Exercises
Name
Karin Cheng
Class
Tech 4
GitHub username
https://github.com/KarinCheng
Bootcamp exercises
https://github.com/KarinCheng/Tech/blob/master/Week1/Lesson1.js
Bootcamp notes

👋 Thanks, we'll look at the exercises from exercises-w2 for your final grade!


Replace function refs with reffer refs

I totally forgot that I had published reffer and had intended to get it in here, since string refs aren’t around anymore, and throwing weird functions around always bugged me!

Interesting!
Looking at this code change made me wonder… is the real problem that we're using overly long arg names?
ref={(buildMessageTextField) => this.buildMessageTextField = buildMessageTextField}

vs just:
ref={(el) => this.buildMessageTextField = el}

vs referrer:
ref={this::reffer("buildMessageTextField")}

Reffer is less characters, but it's also less clear that it sets this.buildMessageTextField, which is what you need if you want to access it. And what you need to make sure you don't clobber/shadow with some other function.
Hrmm…

Doesn’t seem like there was much enthusiasm here, and that’s fair. Going to close this one!


Adaptor: Static site (.html files) doesn't work

Qwik Version
0.12.1
Operating System (or Browser)
MacOS
Node Version (if applicable)
16.13.1
Which component is affected?
Starters / CLI
Expected Behaviour
npm run ssg works like documented in README.md
Actual Behaviour
 ~/E/qwik-app  main  npm run ssg
npm ERR! Missing script: "ssg"
npm ERR!
npm ERR! To see a list of scripts, run:
npm ERR!   npm run

npm ERR!      /Users/solidevolution/.npm/_logs/2022-11-01T15_02_04_124Z-debug-0.log

Additional Information
npm run build.server doesn't work, too
 ~/E/qwik-app  main  npm run build.server

> build.server
> vite build -c adaptors/static/vite.config.ts

vite v3.2.1 building SSR bundle for production...
✓ 15 modules transformed.
<EMAIL_ADDRESS>            9.58 KiB
server/entry.ssr.mjs                    14.68 KiB
server/assets/index.qwik.34c73f2f.mjs   5.36 KiB
[Error: ENOENT: no such file or directory, open '/Users/solidevolution/Entwicklung/qwik-app/dist/service-worker.js'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'open',
  path: '/Users/solidevolution/Entwicklung/qwik-app/dist/service-worker.js'
}

SSG results
Generated: 2 pages
Duration: 92.0 ms
Average: 46.0 ms per page

You need to run build before running build.server.
Works fine.

Thanks for opening the issue. Like @JeanMeche pointed out, running build, which will also run build.client will ensure the service-worker file is built. However, running build.server is valid too, and should not error like this. Updated it to ensure this doesn't crash in that scenario. Thanks


Some ARIA parent roles must contain specific child roles to perform their intended accessibility functions.

Lighthouse reports new error:
Some ARIA parent roles must contain specific child roles to perform their intended accessibility functions.
What are the steps to reproduce this issue?

Go to https://mandrasch.github.io/accessible-menu-bootstrap-examples/bs5-examples/
Run Lighthouse
See error

.dropdown-menu should probably be role="group" instead of a menubar within a menubar, but I'm not expert.
I believe the issue is that there is a menubar within a menubar when using a dropdown menu

It's probably also worth looking into automated lighthouse tests so prevent these kinds of bugs in the future. I'll need to look into that.

Thanks! Love the script.

@danderson421 Glad to hear it!
This change is now captured in v1.0.3 (and also in the latest releases of the beta, main accessible-menu project, and the bs4 version).


glutils: fix glsl error parsing for amd

Related/fixes #73 (i think)
Amd gl outputs errors from glgetShaderInfoLog in a different format than what the previous implementation expected, which would cause a crash calling stoi on an invalid number string
Example amd output:
Vertex shader failed to compile with the following errors:
ERROR: 0:10: error(#132) Syntax error: "rawr" parse error
ERROR: error(#273) 1 compilation errors.  No code generated

This fixes crash to parse the second line correctly, the last line will be ignored, but im not sure if theres any value in it, this also may want to be tested on nvidia, but the code path should be the same as before in that case

Thank you for fixing this. I agree, this PR probably fixed #73 . I've also added additional checks to see if the string actually contains a number just to be sure that it doesn't crash there anymore.


Enable Pubternal API checks

Addresses https://github.com/aspnet/AspNetCore/issues/6975

I'll add annotations and follow up items as appropriate.

Thanks for doing this, John!

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=162552&view=logs

@anurse I see you have moved this to preview6 milestone, do you want me to hold off merging to master even after fixing the tests?

This comment was made automatically. If there is a problem contact<EMAIL_ADDRESS>I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/2181

If all PR checks pass, I'm okay if this merges today.


The second time the modal is dismissed with the keyboard or backdrop the onDismiss event is not being fired

Thanks for reporting this. Taking a look now.

Actually, the onDismissed is being called always for the first action if it is a close or dismiss.
Thank you :)

@okoleilat this should be fixed in v0.4.6. Try it out and let me know how it goes.

perfect :) It is indeed fixed. Keep up the good work 👍


Corrected date format for Portuguese (both locales)

Short date formats in Portuguese (both in Portugal and Brazil) are expressed as dd/MM/yyyy (http://en.wikipedia.org/wiki/Date_format_by_country). I've updated the symbols to reflect that.

This file is auto-generated based on data from http://cldr.unicode.org/

Great. It seems everybody auto-genereates this data based on somewhere else... lol
Okay, I'll try and propose a change there.
But thanks, anyway.


Added Scroll to top button

Related Issue #37

I have added Scroll to top button to the main webpage itself

Closes: #37
Describe the changes you've made
Added scroll to top button which should be visible at every part of the page(not only at the bottom).
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[ ] New and existing unit tests pass locally with my changes.

Screenshots

Hey @urvashi-code1255  This is the new PR made by me after deleting my old forked repo and doing the required changes in new one


git.io/fisher-up-me 404 not-found

http://git.io/fisher-up-me

@aristidesfl Unfortunately the link is not available anymore. I think it may be somewhere buried in the fisherman's Slack history.
It's been more than half a year since it was relevant though. What problem are you having?

$ fisher --version
fisher version 1.3.1

@aristidesfl Current version is 2.10.0! Before you try to update, remove fisherman $fisher_home and $fisher_config, and remove whatever mention to fisherman from your config.fish.
Then install using the procedure in the README and you should be okay.

It works, thanks!


Syntax error in doc

Components -> Typography -> Page Header

Example page header

Must be page-header

Already resolved in 2.0.2-wip.


Remove mtp

What this PR does / why we need it
Towards https://github.com/giantswarm/roadmap/issues/3726
This PR switches from mtp to loki ingress for authentication
It also changes the way we configure the alloy secrets to be able to change the values when we update them on the clusters as the config reloader does not handle evn variable changes
Checklist

[x] Update changelog in CHANGELOG.md.
[x] Follow deployment test procedure in the tests/manual_e2e directory and have a working branch.

@TheoBrigitte it should be fine now


[Feature Request] Add cancel keybinding to cut operations in half

Sometimes some operations take too long it would be really nice to be able to cancel them

You should be able to press the Esc key any time there is a "loading..." type window in the screen. Have you tried that, or are there certain cases where that doesn't work?

Closing as resolved, please reopen if you have any further questions about this behavior.


External Alert Manager secret type certificate fix

How to categorize this PR?
/area monitoring
/kind bug
What this PR does / why we need it:
If insecure_skip_verify key is not explicitly set in secret for external Alert Manager config, gardener won't use the secret for configuring the prometheus
-> https://github.com/gardener/gardener/blob/master/pkg/operation/botanist/monitoring.go#L398
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you @donCappo for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.

/squash
/invite @wyb1


Connections should be reused between different publish calls.

I see you are not reusing connections when publishing events. The RabbitMQ documentation states that connections should be long-lived objects. Creating a connection has a lot of overhead attached to it. See also https://www.rabbitmq.com/dotnet-api-guide.html#connection-and-channel-lifspan
Also, make sure you enable automatic connection failure recovery. It will save you some major headaches. Details about it are listed in the documentation: https://www.rabbitmq.com/dotnet-api-guide.html#recovery

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8f590167-a388-94ac-e39b-463978a761b0
Version Independent ID: cf81b170-d5eb-ca3d-ef50-fff0bea82673
Content: Implementing an event bus with RabbitMQ for the development or test environment
Content Source: docs/standard/microservices-architecture/multi-container-microservice-net-applications/rabbitmq-event-bus-development-test-environment.md
Product: dotnet
GitHub Login: @CESARDELATORRE
Microsoft Alias: wiwagn

Hi @wmeints, thanks for the heads up!
I think perhaps it should be emphasized more that the code shown is just a simplified version, to make it easier to grasp the whole concept. There are two links to the actual code, both right before and right after the code fragment. The actual code considers what you mention and also includes adding the retries policy with Polly, but I think the actual code adds too much "noise" to the core concept.
You can check that in the connection is a singleton in the actual code:
https://github.com/dotnet-architecture/eShopOnContainers/blob/fe0e41d0a6375664913b21bbb8a6f5abe43bc49d/src/Services/Ordering/Ordering.API/Startup.cs#L155
Comments?

Hi Miguel,
Sorry for the late reply. I'm in the middle of writing the last two
chapters of my book. So that takes most of my time.
It certainly looks a lot better in the github repo.
And I totally understand that you can't add every aspect of the code in the
description on the website. But as you mentioned, it should be clear that
it is for demonstration purposes.
Is the actual code linked to from the article? I couldn't find it back when
I submitted the comment.
Kind regards,
Willem Meints
On Mon, 11 Feb 2019 at 11:50, Miguel Veloso<EMAIL_ADDRESS>wrote:

Hi @wmeints https://github.com/wmeints, thanks for the heads up!
I think perhaps it should be emphasized more that the code shown is just a
simplified version, to make it easier to grasp the whole concept. There are
two links to the actual code, both right before and right after the code
fragment. The actual code considers what you mention and also includes
adding the retries policy with Polly, but I think the actual code adds too
much "noise" to the core concept.
You can check that in the connection is a singleton in the actual code:
https://github.com/dotnet-architecture/eShopOnContainers/blob/fe0e41d0a6375664913b21bbb8a6f5abe43bc49d/src/Services/Ordering/Ordering.API/Startup.cs#L155
Comments?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dotnet/docs/issues/9439#issuecomment-462285892, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABepq6-quEqtOTO7KWUwVIQ7K6lINOkkks5vMUr6gaJpZM4ZG_53
.

Hi Willem,
Yep, the actual code is linked from the article, will emphasize that it's a simplified version what's included in the article!
Cheers.


edit a file only

I use Zed main code editor. But sometimes I just need to edit a file on my machine. Is it possible to do without importing a directory and if not, is this something that could be done?

This is not currently possible, but wouldn't be very hard to add. What version of Zed are you using? The Chrome app or standalone? Also, what would expect the flow to be. Should there be a "Open a File" option in the project picker which pops up a file picker or something?

I use the Chrome app. For me, a simple "Open a File" option in the project picker would be perfect! Thank you for the speed with which you answer questions. It is much appreciated.

Implemented, will be part of 1.1. Only available in the Chrome App for now, if there's interest in this for standalone I can implement that as well.


new login page

Not vertically aligned for every screen size

@ShreyaLaheri you need to resolve the merge conflicts.


Changes required in Resource page

Change theme according to the home page.

Add navbar
Add Syllabus, Notes, Books, Previous Year Papers under each department using different cards or do something creative(For eg: under CSE, there should be some cards of notes, syllabus, books etc. )
Add footer

i would like to contribute

I would like to contribute


Add Ferrari2018 libs

Pull Request
This PR is for a

[X] New Publication(s)
[ ] Correction

For the following list(s):

[ ] ancientmetagenome-anthropogenic (README)
[ ] ancientmetagenome-environmental (README, ask @thedir-team-dirt for advice)
[X] ancientmetagenome-hostassociated (README, ask @thedir-team-bugparty for advice)
[ ] ancientsinglegenome-hostassociated (README, ask @thedir-team-pathopeeps for advice)

New Publication
Publication Information
This pull request is to add samples from the following publication(s): 10.3390/genes9090436
This is to close #532
Checklist

[ ] Publication is published (preprints currently not accepted)?
[ ] Checked the publication is not already in the database?
[ ] Checked samples in this publication are not previously published data (newly re-sequenced metagenomes are OK!)?
[ ] Samples are shotgun metagenomes (hostassociated-singlegenome may also contain whole-genome enriched data)?
[ ] Checked the list follows conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)?
[ ] Changelog is updated to include the publication under 'Added'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from corresponding team (see list of lists above for each team)?
[ ] (If applicable) Made a separate PR to add new categories to controlled lists (stored under /assets/enums, e.g. material or archive)? If so the corresponding PR is here:

If you do not know how to check errors in failed validation checks, expand here

Correction
This PR is for

[ ] ancientmetagenome-hostassociated
[ ] ancientsinglegenome-hostassociated
[ ] ancientmetagenome-environmental
[ ] ancientmetagenome-anthropogenic

Reference
This pull request is to correct samples from the following publication(s):
This is to close
Description
The issue is:

Checklist

[ ] Checked the corrected entries follow conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)
[ ] Changelog is updated to include the publication under 'Changed'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from corresponding team (see list of lists above for each team)?

As per our discussion on gather: replace Zurich Functional Genomics Center with University of Zurich


SharePoint Modern Design understanding permissions levels

Hi there, thanks for this information, is there any plan to update this page or create a new one to describe permissions level for SharePoint Modern Design? there are many settings on this document that are not available on the new SharePoint, thanks

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2f072d4a-23e6-d73f-397e-4ffc000570a9
Version Independent ID: 1ee08317-82cd-9f23-a1e9-d2380fb7ff78
Content: Understanding permission levels in SharePoint - SharePoint Online
Content Source: SharePoint/SharePointOnline/understanding-permission-levels.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@GamalielCorona Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @GamalielCorona, thank you for your feedback.
Maybe it's the article you're looking for:
https://docs.microsoft.com/sharepoint/modern-experience-sharing-permissions

@GamalielCorona Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


docs: warn against shared objects

Commit Message: Add warning to documentation about using filter state "shared with upstream". Rename the enum to explicitly warn.
Risk Level: low
Testing: none
Docs Changes: yes
Release Notes: none

/docs

probably also needs a filled out PR description. Thank you!
/wait

/retest


Tests are still the generic cookiecutter tests

Need to write actual tests 🤣

I think with https://github.com/jdeschamps/napari-sklearn-decomposition/pull/8 merged we can close this.


Support envFrom in rayclusters deployed with Helm

Why are these changes needed?
It is often useful to populate the container environment with values specified as Kubernetes secrets or configmaps, using the envFrom key of the pod spec. For example, we use this technique to provide database credentials to our pods.
This PR implements such functionality for Ray clusters deployed with Helm.
Checks

[x] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] This PR is not tested :(

Thanks for the contribution!


allow setting "default search path"

Related to #8.

@vzsg schema name?

Sorry, I was being imprecise. What I actually meant is setting a default search path after the connection is made, some database drivers expose it. The search path is important in (typ. enterprisish) scenarios where schemas are used as an extra level of grouping/access control over tables.
This has been a pretty common feature request for the libpq-based postgresql package and the -provider.

Ah okay, will re-open to track that issue then.

@fwgreen something like this:
public func boot(_ app: Application) throws {
    try app.withNewConnection(to: .psql) { conn in
        return conn.simpleQuery("SET search_path TO myschema").transform(to: ())
    }.wait()
}

I think that would only change that setting for that single (ephemeral) connection. You could prepend it with ALTER DATABASE xyz, I think.

I can confirm that prepending with ALTER DATABASE does work!
public func boot(_ app: Application) throws {
    try app.withNewConnection(to: .psql) { conn in
        return conn.simpleQuery("ALTER DATABASE my_db SET search_path TO my_schema").transform(to: ())
    }.wait()
}

Maybe one day users could have something as simple as
static let entity = "my_schema.my_table"

But this gets us up and running, so I'll take it.  Thank you all!

Ah, didn't realize that was ephemeral to the connection. You can use DatabasesConfig to add connection configuration closures:
var databases = DatabasesConfig()
// register your dbs and what have you
databases.appendConfigurationHandler(on: .psql) { conn in
    return conn.simpleQuery("SET search_path TO myschema")
}
services.register(databases)

@tanner0101 Will this workaround continue to work in the next major release?

This workaround is not perfect, at least not in Vapor 3. PostgreSQLConnection contains a TableNameCache, which maps OIDs <-> table names using a pair of dictionaries. Unfortunately, this isn't a 1:1 relation in the general case, and breaks whenever multiple schemas use the same table names. I wanted to create new schemas per testcase (eg. 'testing_UserTests_20190718_202247'), which would allow my tests to execute in parallel, but the corrupted TableNameCache cause them to not find their own tables.
I have not tried this in Vapor 4. It seems like the Postgres driver has been completely redone, so it's possible that this works now.  However, without explicit support and the tests to back it up, it's possible that this accidentally breaks even if it is currently working. I'm willing to help in this regard. Let me know what your plans are for this.


george / add top alignment for message icon

Wrong (current) behaviour:

Proper alignment according Figma design:

@heorhi-deriv please resolve the conflict 🙏

Looks good @heorhi-deriv but what happens if the message is single line?  it should be aligned in the center. Having it at the top works but for single-line it might look abit off. I suggest to add a prop for iconPosition which the client can set either at the top or the center.
Regardless LGTM.

Looks good @heorhi-deriv but what happens if the message is single line? it should be aligned in the center. Having it at the top works but for single-line it might look abit off. I suggest to add a prop for iconPosition which the client can set either at the top or the center.
Regardless LGTM.

Ok, @aizad-deriv i see your concern. Ok let me pass the prop


chore: release 1.2.190

:robot: I have created a release *beep* *boop*
1.2.190 (2022-11-17)
Features

staged-dockerfile: support ONBUILD instructions (part 1, preparations) (8a813b5), closes #2215

Bug Fixes

build: inconsistent report path when final-repo used (5924702)
staged-dockerfile: fix meta args always expands to empty strings (8f6b562)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/werf/werf/releases/tag/v1.2.190 :sunflower:


e-signature flow

Implements e-signature flow as per:
https://trello.com/c/p8qUfLUA/230-3-ui-to-sign-the-fw-agreement-on-the-supplier-side-functional-tests
https://dmp-prototype-sign-agreement.herokuapp.com/

[x] add legal authority question
[x] add form validation
[x] add no response page
[x] add yes response page
[x] fix breadcrumbs
[x] add legal text
[x] handle form submission
[x] confirmation page
[x] confirmation email
check details page framework-signed
[x] unit tests
[x] fix supplier dashboard

@katstevens do you know if the 'yes' response page i.e. https://dmp-prototype-sign-agreement.herokuapp.com/agreement should replace an existing route, be a new route, or be tied to a specific framework?

@katstevens do you know if the 'yes' response page i.e. https://dmp-prototype-sign-agreement.herokuapp.com/agreement should replace an existing route, be a new route, or be tied to a specific framework?

I think we agreed to do new routes, so we can easily switch back to the old process if necessary.
The new process should be for iterations of G-Cloud >= 12, and DOS >= 5.

@katstevens Do we also want to restrict this flow to a specific framework status?

Also one thing we'll need to update is the link on the _framework_actions.html template, to get to the new flow from the supplier dashboard

Thanks for the guidance on this @katstevens. I have updated this in 7452ee9.

I've had a go at squashing some of the commits but have ran out of steam 😹 Will merge this now so we can try out on Preview.


[accelerator][FeatBugFix] Improve manual optimization API

What does this PR do?

This PR implements the proposal from 5769.
Todos
[] Support sync_ddp_block for manual_optimization.
Fixes #5769 <- this links related issue to this PR
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified
[ ] Check that target branch and milestone match!

Did you have fun?
Make sure you had fun coding 🙃

Hello @tchaton! Thanks for updating this PR.

In the file pytorch_lightning/core/optimizer.py:

Line 105:121: E501 line too long (129 > 120 characters)
Line 147:18: E203 whitespace before ':'
Line 148:25: E203 whitespace before ':'

Great improvement @tchaton. Love it

Does this change the current behaviour? If anyone expected zero grad to be done for them within the manual backwards will this change the behaviour?

Does this change the current behaviour? If anyone expected zero grad to be done for them within the manual backwards will this change the behaviour?

Yes, I added a note in the doc for it. It is breaking BC, but enable full control on optimization for the users. We had 2 design meeting and decided to make this choice before people really start to adopt it.


Print file and line number for anonymous functions

Coverage increased (+0.5%) to 96.61% when pulling 0514ac058101fd18903d2e1658144fa1968c2b9e on jlopezr:master into 3944bbcb777b1d71a1d04b6c92ffbdfd3bf2cdd2 on analog-nico:master.


Removing duplicate test cases and fixing typo in Text.Primitives.Tests.

Resolves this discrepancy between VS test discovery/run and command line:
------ Discover test started ------
[xUnit.net 00:00:00.5287380]   Discovering: System.Text.Primitives.Tests
[xUnit.net 00:00:02.2739751]   Discovered:  System.Text.Primitives.Tests
A test with the same name 'System.Text.Primitives.Tests.Utf8StringTests.StringEqualsWithThreeArgs (7b20df66ebc54d70aa2f6290cb0e75c08e639f33)' already exists. This test is not added to the test window.
A test with the same name 'System.Text.Primitives.Tests.Utf8StringTests.StringEqualsWithThreeArgs (7b20df66ebc54d70aa2f6290cb0e75c08e639f33)' already exists. This test is not added to the test window.
A test with the same name 'System.Text.Primitives.Tests.Utf8StringTests.TryComputeEncodedBytesShouldMatchEncoding_Utf8 (fae5145b9049eaa872e7b3fb901cf7ee166bfa6a)' already exists. This test is not added to the test window.
A test with the same name 'System.Text.Primitives.Tests.Utf8StringTests.TryComputeEncodedBytesShouldMatchEncoding_Utf16 (36396a74f5c5bb934ec1d1dbf0d03865b66434c5)' already exists. This test is not added to the test window.
========== Discover test finished: 572 found (0:00:02.9093646) ==========
------ Run test started ------
[xUnit.net 00:00:00.7018542]   Starting:    System.Text.Primitives.Tests
[xUnit.net 00:00:05.7034494]   Finished:    System.Text.Primitives.Tests
========== Run test finished: 568 run (0:00:07.9177629) ==========
cc @shiftylogic, @KrzysztofCwalina

This is a small enough change, will merge shortly.


pip installation of a local wheel into an asv conda environment?

Trying to benchmark a project that has some installation requirements that are private files that can't go up onto pypi...
I didn't see a way in the configuration file to easily change the installation source from pypi to a local wheel or egg file?  Does such a method exist?  Thanks.

If they're available as conda packages in some directory, you can try adding it as a conda channel in .condarc.
If they're available only as pip wheels, and given as install_requires in setup.py, you can try setting pip environment variables e.g. maybe export PIP_FIND_LINKS=file:///path/to/wheels; asv run ...


Permission to use the bot features or whitelist feature

I will like to know if there is any possible way to change the commands on the configuration to make the bot accept the commands from everyone in the server. Or at least, to have a code to add people in the whitelist, since the bot only have blacklist functions. Thank you.

add all the commands in the default tab


Resources should be able to be categorized using non-hirarchical tags

This is an undertaking that will be required to complete #33
What "buckets" should be used for categorization?

What about topics that the resource might already be associated with?
Should categories actually be a new object (just like resources, topics and goals). And actually be "higher level" then goals.
For example "Robotics" could be a category that a "introduction to robotics programming" goal, "assembling the codebot hardware" goal, and "PID programming" goals all fall into. In addition, perhaps the "PID programming" goal has a second category of algorithms?
something else?
How ever it falls I think flexibility is key to the system.

Another method might be to group by tags.. which would allow goals, topics and resources to be attached to multiple tags.
Then the UI could offer tags grouped possibly by the most used / most things attached.
Searching tags??

This is on hold.. should be evaluated as part of the merge of outfox because outfox had a tagging mechanism


Failed to run code

File "/home/xxx/anaconda3/envs/LIIR/lib/python3.6/site-packages/smac/env/starcraft2/starcraft2.py", line 292, in _launch
self._run_config = run_configs.get(version=self.game_version)
TypeError: get() got an unexpected keyword argument 'version'
TypeError: get() got an unexpected keyword argument 'version'
TypeError: get() got an unexpected keyword argument 'version'

Are you using the latest version of PySC2 on the master branch?

No,I'm using pysc2==2.0.2

SMAC requires that you use pysc2>=3.0.0. Please update it!

Should  I use the latest version of PySC2 ？Such as 3.0?

OK,thanks,I'll try.


modified README and added requirements.txt

I have added more details to the README file, and also included a requirements.txt file which shows the libraries to be installed in virtual environments, in order to be able to install and run a local instance of the application.
Please review and accept for the changes to the pushed to the main branch.

I approve the changes you made to the file

Changes looks good.

thank you such an elaborate pull request ....

thank you

Reviewed, looks good.


pull request for e2e test at 12/11/2016 14:45:04

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Allow to use markdown in notifications

Version: 1.32.0-exploration (user setup)
Commit: 7ecc511083c881fa4319f35f743f5ea43e01bf49
Date: 2019-02-22T08:53:52.866Z
Electron: 4.0.5
Chrome: 69.0.3497.106
Node.js: 10.11.0
V8: 6.9.427.24-electron.0
OS: Windows_NT x64 10.0.17763

Steps to Reproduce:

Get yourself a notification with markdown code, in my case I used the token notification
Look at markdown code not renderedm in this case it is the backticks not making editor.maxTokenizationLineLength not monospace.

Does this issue occur when all extensions are disabled?: Yes

Is there any recommendation of showing code snippet in VSCode Plugin?


feat: monitor embedding

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.