Z2-Converter API Mainnet : GetSmartContractState

Issue Summary:
Calling GetSmartContractState on the ZQ1 and ZQ2 has the following diff.
Here are the URLs for ZQ1 and ZQ2.
ZQ2 : http://<IP_ADDRESS>:4201/
ZQ1 : http://<IP_ADDRESS>:4201/
Result:
Missing keys:
--- Viewing difference for GetSmartContractState ---

Keys present in ZQ1 but missing in ZQ2:
  root['result']['verifier_receiving_addr']
  root['result']['buff_deposit_deleg']['0x9737708a0b47ff38f29b7dce795f76aaf357e7e0']
  root['result']['buff_deposit_deleg']['0xa97c340b0ddbf2680215a046b54a13d4c411fcc0']
  root['result']['buff_deposit_deleg']['0x2352540547907dae0c7529cb0d132ee68a658b9e']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1571']
  root['result']['buff_deposit_deleg']['0x5769845058770c2358b06ff8954e386ba7ddf07c']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1571']
  root['result']['buff_deposit_deleg']['0xb65d3600d6f46a27e89eac465925ac77213e6530']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1571']
  root['result']['buff_deposit_deleg']['0xd265e252084f73ed2d7c17a68a6d31e0d48cc27b']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['1571']
  root['result']['deleg_stake_per_cycle']['0x2352540547907dae0c7529cb0d132ee68a658b9e']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1571']
  root['result']['deleg_stake_per_cycle']['0x5769845058770c2358b06ff8954e386ba7ddf07c']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1571']
  root['result']['deleg_stake_per_cycle']['0x7192b32e733d11f9f57e6d4c4f874684fcbc717b']['0x82b82c65213e0b2b206492d3d8a2a679e7fe52e0']['1571']
  root['result']['deleg_stake_per_cycle']['0x82b4c4a99cfc97b1497b51619694a69c77eac4a2']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['1571']
  root['result']['deleg_stake_per_cycle']['0x9b8b821e4e0a73d119fb2a7cb42f91b6bf72e036']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1571']
  root['result']['deleg_stake_per_cycle']['0xb14e071b62b1f9951b0d0ee23346f72ec51bc4a6']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['1571']
  root['result']['deleg_stake_per_cycle']['0xb43bebf029e04234b41b00feeb4582c24ac3816e']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1571']
  root['result']['deleg_stake_per_cycle']['0xb65d3600d6f46a27e89eac465925ac77213e6530']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1571']
  root['result']['deleg_stake_per_cycle']['0xd265e252084f73ed2d7c17a68a6d31e0d48cc27b']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['1571']
  root['result']['deposit_amt_deleg']['0x11017aa1733259fdcb65ad1db744beb2879fc40b']['0xc3ed69338765424f4771dd636a5d3bfa0a776a35']
  root['result']['direct_deposit_deleg']['0x11017aa1733259fdcb65ad1db744beb2879fc40b']
  root['result']['direct_deposit_deleg']['0x18556311c005f8f5bad9d41c83e62d71e41bae12']
  root['result']['last_withdraw_cycle_deleg']['0x11017aa1733259fdcb65ad1db744beb2879fc40b']['0xc3ed69338765424f4771dd636a5d3bfa0a776a35']
  root['result']['ssn_deleg_amt']['0xc3ed69338765424f4771dd636a5d3bfa0a776a35']['0x11017aa1733259fdcb65ad1db744beb2879fc40b']
  root['result']['withdrawal_pending']['0x18556311c005f8f5bad9d41c83e62d71e41bae12']

Keys present in ZQ2 but missing in ZQ1:
  root['result']['buff_deposit_deleg']['0x11017aa1733259fdcb65ad1db744beb2879fc40b']
  root['result']['buff_deposit_deleg']['0x18556311c005f8f5bad9d41c83e62d71e41bae12']
  root['result']['buff_deposit_deleg']['0xb14e071b62b1f9951b0d0ee23346f72ec51bc4a6']
  root['result']['buff_deposit_deleg']['0x2352540547907dae0c7529cb0d132ee68a658b9e']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1555']
  root['result']['buff_deposit_deleg']['0x5769845058770c2358b06ff8954e386ba7ddf07c']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1558']
  root['result']['buff_deposit_deleg']['0xb65d3600d6f46a27e89eac465925ac77213e6530']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1539']
  root['result']['buff_deposit_deleg']['0xd265e252084f73ed2d7c17a68a6d31e0d48cc27b']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['1522']
  root['result']['deleg_stake_per_cycle']['0x2352540547907dae0c7529cb0d132ee68a658b9e']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1555']
  root['result']['deleg_stake_per_cycle']['0x5769845058770c2358b06ff8954e386ba7ddf07c']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1558']
  root['result']['deleg_stake_per_cycle']['0x7192b32e733d11f9f57e6d4c4f874684fcbc717b']['0x82b82c65213e0b2b206492d3d8a2a679e7fe52e0']['1423']
  root['result']['deleg_stake_per_cycle']['0x82b4c4a99cfc97b1497b51619694a69c77eac4a2']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['1330']
  root['result']['deleg_stake_per_cycle']['0x9b8b821e4e0a73d119fb2a7cb42f91b6bf72e036']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1569']
  root['result']['deleg_stake_per_cycle']['0xb14e071b62b1f9951b0d0ee23346f72ec51bc4a6']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['1567']
  root['result']['deleg_stake_per_cycle']['0xb43bebf029e04234b41b00feeb4582c24ac3816e']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1562']
  root['result']['deleg_stake_per_cycle']['0xb65d3600d6f46a27e89eac465925ac77213e6530']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['1539']
  root['result']['deleg_stake_per_cycle']['0xd265e252084f73ed2d7c17a68a6d31e0d48cc27b']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['1522']
  root['result']['deposit_amt_deleg']['0x11017aa1733259fdcb65ad1db744beb2879fc40b']['0x122219cceab410901e96c3a0e55e46231480341b']
  root['result']['last_buf_deposit_cycle_deleg']['0x11017aa1733259fdcb65ad1db744beb2879fc40b']
  root['result']['last_buf_deposit_cycle_deleg']['0x18556311c005f8f5bad9d41c83e62d71e41bae12']
  root['result']['last_withdraw_cycle_deleg']['0x11017aa1733259fdcb65ad1db744beb2879fc40b']['0x122219cceab410901e96c3a0e55e46231480341b']
  root['result']['ssn_deleg_amt']['0x122219cceab410901e96c3a0e55e46231480341b']['0x11017aa1733259fdcb65ad1db744beb2879fc40b']

Different Keys:
Values that differ between ZQ2 and ZQ1:
  root['result']['_balance']:
    ZQ2:<PHONE_NUMBER>836755835157
    ZQ1:<PHONE_NUMBER>927233584237
  root['result']['cycle_rewards_deleg']:
    ZQ2:<PHONE_NUMBER>10029
    ZQ1:<PHONE_NUMBER>52173
  root['result']['deposit_amt_deleg']['0x18556311c005f8f5bad9d41c83e62d71e41bae12']['0x90d3dbd71c54c38341a6f5682c607e8a17023c28']:
    ZQ2:<PHONE_NUMBER>000000
    ZQ1:<PHONE_NUMBER>0000
  root['result']['deposit_amt_deleg']['0x2352540547907dae0c7529cb0d132ee68a658b9e']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2:<PHONE_NUMBER>91750411
    ZQ1:<PHONE_NUMBER>90697708
  root['result']['deposit_amt_deleg']['0x5769845058770c2358b06ff8954e386ba7ddf07c']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2:<PHONE_NUMBER>15443434
    ZQ1:<PHONE_NUMBER>90805137
  root['result']['deposit_amt_deleg']['0x9737708a0b47ff38f29b7dce795f76aaf357e7e0']['0x635eff625a147c7ca0397445eee436129ee6ca0b']:
    ZQ2:<PHONE_NUMBER>9758184
    ZQ1:<PHONE_NUMBER>9834676
  root['result']['deposit_amt_deleg']['0xa97c340b0ddbf2680215a046b54a13d4c411fcc0']['0x90d3dbd71c54c38341a6f5682c607e8a17023c28']:
    ZQ2:<PHONE_NUMBER>834440747
    ZQ1:<PHONE_NUMBER>868313022
  root['result']['deposit_amt_deleg']['0xb65d3600d6f46a27e89eac465925ac77213e6530']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2:<PHONE_NUMBER>73653127
    ZQ1:<PHONE_NUMBER>41734099
  root['result']['deposit_amt_deleg']['0xd265e252084f73ed2d7c17a68a6d31e0d48cc27b']['0x635eff625a147c7ca0397445eee436129ee6ca0b']:
    ZQ2:<PHONE_NUMBER>9476856
    ZQ1:<PHONE_NUMBER>9476856
  root['result']['last_buf_deposit_cycle_deleg']['0x2352540547907dae0c7529cb0d132ee68a658b9e']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2: 1555
    ZQ1: 1571
  root['result']['last_buf_deposit_cycle_deleg']['0x5769845058770c2358b06ff8954e386ba7ddf07c']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2: 1558
    ZQ1: 1571
  root['result']['last_buf_deposit_cycle_deleg']['0x9737708a0b47ff38f29b7dce795f76aaf357e7e0']['0x635eff625a147c7ca0397445eee436129ee6ca0b']:
    ZQ2: 1534
    ZQ1: 1571
  root['result']['last_buf_deposit_cycle_deleg']['0xa97c340b0ddbf2680215a046b54a13d4c411fcc0']['0x90d3dbd71c54c38341a6f5682c607e8a17023c28']:
    ZQ2: 1568
    ZQ1: 1571
  root['result']['last_buf_deposit_cycle_deleg']['0xb65d3600d6f46a27e89eac465925ac77213e6530']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2: 1539
    ZQ1: 1571
  root['result']['last_buf_deposit_cycle_deleg']['0xd265e252084f73ed2d7c17a68a6d31e0d48cc27b']['0x635eff625a147c7ca0397445eee436129ee6ca0b']:
    ZQ2: 1522
    ZQ1: 1571
  root['result']['last_withdraw_cycle_deleg']['0x18556311c005f8f5bad9d41c83e62d71e41bae12']['0x90d3dbd71c54c38341a6f5682c607e8a17023c28']:
    ZQ2: 1368
    ZQ1: 1571
  root['result']['last_withdraw_cycle_deleg']['0x2352540547907dae0c7529cb0d132ee68a658b9e']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2: 1555
    ZQ1: 1571
  root['result']['last_withdraw_cycle_deleg']['0x5769845058770c2358b06ff8954e386ba7ddf07c']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2: 1558
    ZQ1: 1571
  root['result']['last_withdraw_cycle_deleg']['0x7192b32e733d11f9f57e6d4c4f874684fcbc717b']['0x82b82c65213e0b2b206492d3d8a2a679e7fe52e0']:
    ZQ2: 1423
    ZQ1: 1571
  root['result']['last_withdraw_cycle_deleg']['0x82b4c4a99cfc97b1497b51619694a69c77eac4a2']['0x635eff625a147c7ca0397445eee436129ee6ca0b']:
    ZQ2: 1330
    ZQ1: 1571
  root['result']['last_withdraw_cycle_deleg']['0x9b8b821e4e0a73d119fb2a7cb42f91b6bf72e036']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2: 1569
    ZQ1: 1571
  root['result']['last_withdraw_cycle_deleg']['0xb14e071b62b1f9951b0d0ee23346f72ec51bc4a6']['0x635eff625a147c7ca0397445eee436129ee6ca0b']:
    ZQ2: 1567
    ZQ1: 1571
  root['result']['last_withdraw_cycle_deleg']['0xb43bebf029e04234b41b00feeb4582c24ac3816e']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2: 1562
    ZQ1: 1571
  root['result']['last_withdraw_cycle_deleg']['0xb65d3600d6f46a27e89eac465925ac77213e6530']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']:
    ZQ2: 1539
    ZQ1: 1571
  root['result']['last_withdraw_cycle_deleg']['0xd265e252084f73ed2d7c17a68a6d31e0d48cc27b']['0x635eff625a147c7ca0397445eee436129ee6ca0b']:
    ZQ2: 1522
    ZQ1: 1571
  root['result']['ssn_deleg_amt']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['0x9737708a0b47ff38f29b7dce795f76aaf357e7e0']:
    ZQ2:<PHONE_NUMBER>9758184
    ZQ1:<PHONE_NUMBER>9834676
  root['result']['ssn_deleg_amt']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['0xd265e252084f73ed2d7c17a68a6d31e0d48cc27b']:
    ZQ2:<PHONE_NUMBER>9476856
    ZQ1:<PHONE_NUMBER>9476856
  root['result']['ssn_deleg_amt']['0x90d3dbd71c54c38341a6f5682c607e8a17023c28']['0x18556311c005f8f5bad9d41c83e62d71e41bae12']:
    ZQ2:<PHONE_NUMBER>000000
    ZQ1:<PHONE_NUMBER>0000
  root['result']['ssn_deleg_amt']['0x90d3dbd71c54c38341a6f5682c607e8a17023c28']['0xa97c340b0ddbf2680215a046b54a13d4c411fcc0']:
    ZQ2:<PHONE_NUMBER>834440747
    ZQ1:<PHONE_NUMBER>868313022
  root['result']['ssn_deleg_amt']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['0x2352540547907dae0c7529cb0d132ee68a658b9e']:
    ZQ2:<PHONE_NUMBER>91750411
    ZQ1:<PHONE_NUMBER>90697708
  root['result']['ssn_deleg_amt']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['0x5769845058770c2358b06ff8954e386ba7ddf07c']:
    ZQ2:<PHONE_NUMBER>15443434
    ZQ1:<PHONE_NUMBER>90805137
  root['result']['ssn_deleg_amt']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['0xb65d3600d6f46a27e89eac465925ac77213e6530']:
    ZQ2:<PHONE_NUMBER>73653127
    ZQ1:<PHONE_NUMBER>41734099
  root['result']['totalstakeamount']:
    ZQ2:<PHONE_NUMBER>547963978211
    ZQ1:<PHONE_NUMBER>520363978211
  root['result']['ssnlist']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['arguments'][2]:
    ZQ2:<PHONE_NUMBER>7446324204
    ZQ1:<PHONE_NUMBER>4041194481
  root['result']['ssnlist']['0x82b82c65213e0b2b206492d3d8a2a679e7fe52e0']['arguments'][2]:
    ZQ2:<PHONE_NUMBER>130127357
    ZQ1:<PHONE_NUMBER>247292400
  root['result']['ssnlist']['0xc3ed69338765424f4771dd636a5d3bfa0a776a35']['arguments'][1]:
    ZQ2:<PHONE_NUMBER>80634911
    ZQ1:<PHONE_NUMBER>80634911
  root['result']['ssnlist']['0x122219cceab410901e96c3a0e55e46231480341b']['arguments'][2]:
    ZQ2:<PHONE_NUMBER>4518012915
    ZQ1:<PHONE_NUMBER>0850916515
  root['result']['ssnlist']['0x122219cceab410901e96c3a0e55e46231480341b']['arguments'][1]:
    ZQ2:<PHONE_NUMBER>62698703371
    ZQ1:<PHONE_NUMBER>62698703371
  root['result']['ssnlist']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['arguments'][2]:
    ZQ2:<PHONE_NUMBER>4248801792
    ZQ1:<PHONE_NUMBER>8413289455
  root['result']['ssnlist']['0xb83fc2c72c44b6b869c64384375c979dc3f7cf05']['arguments'][6]:
    ZQ2:<PHONE_NUMBER>7970079
    ZQ1:<PHONE_NUMBER>0360051
  root['result']['ssnlist']['0x90d3dbd71c54c38341a6f5682c607e8a17023c28']['arguments'][6]:
    ZQ2:<PHONE_NUMBER>43947288
    ZQ1:<PHONE_NUMBER>77819563
  root['result']['ssnlist']['0x635eff625a147c7ca0397445eee436129ee6ca0b']['arguments'][6]:
    ZQ2:<PHONE_NUMBER>21625105
    ZQ1:<PHONE_NUMBER>11701597
  root['result']['ssnlist']['0x90d3dbd71c54c38341a6f5682c607e8a17023c28']['arguments'][2]:
    ZQ2:<PHONE_NUMBER>9877025531
    ZQ1:<PHONE_NUMBER>0779009289
  root['result']['ssnlist']['0x90d3dbd71c54c38341a6f5682c607e8a17023c28']['arguments'][1]:
    ZQ2:<PHONE_NUMBER>33099275517
    ZQ1:<PHONE_NUMBER>05499275517

Observations:

Although the diff is small for a contract like staking, but not sure of the reason. I used the same persistence tarball 4162888 for both ZQ1 and ZQ2.
The return response takes 320 seconds on ZQ2 while it takes 9.5 seconds on ZQ1.

Conclusion:
If the above differences are expected and align with the migration to ZQ2, no further action is required. However, if these differences are unexpected, they may indicate issues that must be addressed.

This happened because stateDelta folder was not migrated to ZQ2, so some state deltas of 0-88 blocks remained as they were not moved to the state. they are moved only after the 100th block. cc @bzawisto @JamesHinshelwood @DrZoltanFazekas

To address this, we need to ensure that we use or generate the backup after the MoveUpdateToDisk operation at the 99th block, with transactions paused. This approach will guarantee that no state deltas remain in ZQ1's persistence. is that correct understanding?

same as #1673 , please proceed with ZQ1 persistence after block 99 of the respective epoch to ensure there are no deltas that weren't applied

@chetan-zilliqa Is this solved with the new conversion or do we need to look into it more?

The issue is resolved. Sorry, i had kept it open because we wanna add some reminder steps to ensure the above. I will move it to confluence. For now closing the issue.


Embedded python install

I am having difficulty embedding python install to the plugin. Is it just a matter of dumping the py install folder inside the binary folder? Every time I do that UEditor crashes. I am not seeing a guide on how to build the plugin with embedded python? Thanks!

Hi, ensure the python version of the embeddable distribution is the same for which you have built the plugin. Then yes, just copy it to the Binary folder.
I suggest you to download a binary embedded release of the plugin and just substitute its 3 plugin-related DLLs

I suggest you to download a binary embedded release of the plugin and just substitute its 3 plugin-related DLLs

This approach worked really well for me. Thank you!


Separate out and fix CustomSelect

Thanks for pulling this out into a separate file, much easier to read. Perfect fix works great in demo & app.


Error generating PR description 'str' object does not support item assignment

Can I help you this error?

you are using a bad codel model, that does not obey the prompt instructions.
Use a better model

I followed the instructions and used the ollama/llama3.2 model listed here. Is this a better model?

Use claude sonnet, gpt-4o, or google gemini.
All models in the world are supported. It doesn't mean they are good enough for code tasks


README: Add godoc/travis/coveralls badges.

Signed-off-by: Andrea Luzzardi<EMAIL_ADDRESS>

LGTM


Updated BlogCard

Updated Twitter icon in the Blog Card.
Screenshots :
before

after

Summary by CodeRabbit

Refactor

Updated the social media icon in the Blog Card component from a Twitter icon to an alternative Twitter icon design.

Hi @FrancescoXX , review check the PR :-)


Bdd connection

Added a few basic tests for the Connection class
Also added information about how to execute the bdd suite on the readme

Coverage remained the same at 85.263% when pulling 4ed2ea369c027ad3b051a519997d888f2be5feb8 on bdd-connection into d4da55d3f027cfb89a1383ba37d973f0f71282a9 on develop.


Remove ability to pass tokens

For the sake of best practice & uniform actions across the MDAnalysis ecosystem, the ability to pass API token is being removed.
All future uses of this action will only work with OpenID connect trusted publisher workflows.

Let's see what happens with CI on merge.


gb is installed using go, but when i hit gb command, "command not found" error coming

Hello,
I have installed the gb using the go tool. However when I hit the gb command, "command not found" error coming. Should i set any PATH variables ?
Here are the commands, i have executed.
root@DAEDALUS-STUD-U:~# go get github.com/constabulary/gb/...
root@DAEDALUS-STUD-U:~# ls
Bilder     Downloads  Musik       Schreibtisch  Vorlagen
Dokumente  go         Öffentlich  Videos
root@DAEDALUS-STUD-U:~# gb
gb: Befehl nicht gefunden.
root@DAEDALUS-STUD-U:~#

Is $GOPATH/bin in your $PATH?

Specially add /root/go/bin to your $PATH

On 10 Apr 2017, at 21:13, jibinpjohn<EMAIL_ADDRESS>wrote:
Hello,
I have installed the gb using the go tool. However when I hit the gb command, "command not found" error coming. Should i set any PATH variables ?
Here are the commands, i have executed.
root@DAEDALUS-STUD-U:# go get github.com/constabulary/gb/...
root@DAEDALUS-STUD-U:# ls
Bilder Downloads Musik Schreibtisch Vorlagen
Dokumente go Öffentlich Videos
root@DAEDALUS-STUD-U:# gb
gb: Befehl nicht gefunden.
root@DAEDALUS-STUD-U:#
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Many thanks,It worked.
root@DAEDALUS-STUD-U:~/go/bin# export PATH=$PATH:/root/go/bin
root@DAEDALUS-STUD-U:~/go/bin# echo $PATH
/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/go/bin:/root/go/bin
root@DAEDALUS-STUD-U:~/go/bin# gb
gb, a project based build tool for the Go programming language.
Usage:
    gb command [arguments]

Can you close the issue ?


Building Key Indices

While doing some performance investigation using pacer with titan, I noticed what I think is a bug in the Key Indices module. blueprints_graph.getIndexedKeys allows query for indexed keys on edges or vertices. The Pacer key_indices method calls the getIndexedKeys twice for vertices when the type is not specified - taking the unique set of the result. This seems unnecessary at best.
I found this issue while investigating some pathologic behavior in pacer with Titan as a backend.
main profile results:
Total time: 115.15

     total        self    children       calls  method
----------------------------------------------------------------
    110.34        0.09      110.25      326120  Array#each
    104.25        0.00      104.25           1  TicketInserter#insert
    104.07        0.00      104.07           1  CrmKinesis::EventHandlers::Ticket#process
     65.53        0.00       65.53           6  Range#each_with_time_with_zone
     65.53        0.00       65.53           6  Range#each
     65.53        0.00       65.53           1  CrmKinesis::EventHandlers::Ticket#insert_store_day
     63.67        0.10       63.57       59710  Array#map
     61.09        0.02       61.07         433  CrmKinesis::EventHandlers::Ticket#create_vertices
     39.42        0.09       39.33       18420  Proc#call
     36.20        0.00       36.20           1  Guestly.close_graph
     36.20        0.00       36.20           1  Pacer::PacerGraph#shutdown
     36.20       36.20        0.00           1  Java::ComThinkaureliusTitanGraphdbBlueprints::TitanBlueprintsGraph#shutdown
     33.64        0.08       33.56        3038  Pacer::Core::Graph::VerticesRoute.add_edges_to
     29.34        0.06       29.27        3471  Pacer::Titan::Graph#indexed_route
     28.46        0.03       28.43        4337  Pacer::Titan::Graph#key_indices
     28.43        0.05       28.38        4337  Pacer::PacerGraph::KeyIndices.key_indices
     27.87       27.87        0.00        4337  Java::ComThinkaureliusTitanGraphdbBlueprints::TitanBlueprintsGraph#getIndexedKeys
     21.42        0.17       21.25       11143  Pacer::Core::Route.each
     20.30        0.03       20.27        3038  Pacer::Core::Graph::GraphIndexRoute.e
     15.55        0.19       15.36       55683  Java::ComTinkerpopPipes::AbstractPipe#next
     15.36        0.11       15.25       50054  Pacer::Pipes::WrappingPipe#processNextStart
     11.64        0.07       11.57        3038  Pacer::GraphTransactionsMixin.transaction
     11.17       11.04        0.13        9810  Java::ComTinkerpopPipesFilter::PropertyFilterPipe#next
     10.82        0.08       10.74        3730  Kernel.require
      9.31        0.01        9.31         433  Pacer::Core::Graph::GraphIndexRoute.v
      8.69        0.33        8.36      289956  Class#new
      8.23        0.04        8.20       30858  Kernel.tap

This method key_indicies is called 4337 times taking about 30% of the cpu time to build the same set over an over again. In titan this is an expensive operation to build the set which should be done at most once per transaction.
I have a naive patch implementation we are using in testing right now to memoize this method in pacer, but I am not sure how to clear the cached values when a transaction is closed? I can extend this PR with a proposed solution for the memoizing this method or open a new PR for that work if you think it belongs here. I am curious whether this is is an issue unique to titan or if this is a slow operation across all blue prints implementations. That might help determine where the caching mechanism belongs.

Hi David,
Good catch!
It definitely should not be getting the indices twice. At the same time, I don't think it would make any sense to produce a list of both vertex and edge indices. They probably wouldn't even be distinguishable, at least not reliably across graphs...
Probably makes more sense to just default the type argument of that method to :vertex.
As for memoization, that's probably best to make Titan-specific. The best way to do that is probably to override the method in the Titan adapter. Perhaps a TTL based cache with invalidation on local index operations?
Thanks,
Darrick

Okay - will modify the PR tonight as suggested and tag you on changes in pacer-titan as well.
Thanks
David

@pangloss Does this look about right to fix the bug for nil type?

That looks good, thanks!


Create mime parser super class. Add streamable mime parser.

Thanks! I'll need to test this out but it looks promising and I like the abstraction that you added.


Create model for Bootcamp Applicant Status report

User Story

Bootcamps applicant status report is currently on BI. It's worth making it to our data lake.
Description/Context

The underlying query is https://bi.odl.mit.edu/queries/969/source
Source tables:
applications_bootcampapplication
applications_applicationstep
applications_bootcamprunapplicationstep
applications_applicationstepsubmission
auth_user
profiles_profile
profiles_legaladdress
klasses_bootcamp
klasses_bootcamprun
ecommerce_line
ecommerce_order
Acceptance Criteria

[ ] Stage applications_* tables first
[ ] Make a model for Bootcamps applicant status report

Related issue https://github.com/mitodl/hq/issues/847

Can this be closed?

Can this be closed?

These tables are not staged in our data platform. I think we still need these tables to replace report like https://bi.odl.mit.edu/queries/969/source unless you disagree?

@KatelynGit FYI, int__bootcamps__applications in https://github.com/mitodl/ol-data-platform/pull/1018 should have all the non user profile related fields for replacing report https://bi.odl.mit.edu/queries/969/source. There are a lot of user profile fields in that report but you can pull from int__bootcamps__users


Fix PHPUnit configuration

Pull Request Test Coverage Report for Build 1120

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 89.089%

Totals

Change from base Build 1117:
0.7%

Covered Lines:
5152

Relevant Lines:
5783

💛  - Coveralls


Potential changes

I'm not super familiar with linux commands so I ran into a few issues when setting this up. I added a few things to my local version of the script that I thought may be help to those who aren't as tech savvy. Hope you consider these.

Hi @blakeb21, thanks for your advice and changes


Implemented Max-Rects packing

Major improvement:

Implemented Max-Rects packing, which is more efficient than multi-bin-pack.
Add Decimal round option for json simplification and image pixel snapping.

Bug fixed:

Font edge rendering artifacts on msdf output
Glyph bounding-box and x/y offset issue.
Alpha channel for monochrome sdf output.

Code review fixed.
Also find the cause of msdf font edge artifacts, fixed. 🍺

@soimy thanks for this terrific PR. The fonts are packed much more efficiently, and the edge artifacts are gone. Fantastic work! I did have one minor issue with running this on my 10.11.6 MacOS computer. Specifically, the executable generated the error Reason: Incompatible library version: msdfgen.osx requires version 21.0.0 or later, but libfreetype.6.dylib provides version 19.0.0. This was not a problem for me, as it was easy to recompile msdfgen. However, to make it easier for others, I would recommend building this on an older machine. I am happy to share my executable. I am very impressed by your work.

@neurolabusc msdfgen 1.4 osx binary in this repo is built on freetype v2.8.
In your case, u just need to run brew update && brew upgrade freetype to update your freetype to the newest version.
But you are right, for more user-friendly approach, we need to either:

use private dynamic library with msdf binary, just like the windows version.
use msdfgen as a nodejs native add-on ( built with node-gyp ) as mentioned #3

@neurolabusc Can you help me test the private dynamic library version of the msdf binary?
Just made a commit in my repo soimy/msdf-bmfont-xml@f154f92dbe60ef730f7e683066856f2303bafad2

@soimy your inclusion of the additional libraries works. Thanks again. Future versions of my MRIcroGL and Surfice will benefit from your work.

@neurolabusc Happy to hear that! Keep an eye on my dev branch for more feature and fixes.

@arilotter Sorry about that, it was leaked from another branch (Export xml format)


Multithread the emulator!

From<EMAIL_ADDRESS>on July 29, 2009 23:27:16
Describe the New Feature: 1) Devise a work queue approach where the executive class spawns one thread
per CPU core and assigns CTAs to threads as they complete.
2) For atomic ops, rather than locking, asynchronously push data into a
local queue, when it overflows then lock and do a bulk update.  Also do a
bulk update when the CTA completes to eliminate stragglers and when a fence
instruction is called. Which milestone does the feature belong to? 1.0.0 Which branch does the new feature go in? Branch
Original issue: http://code.google.com/p/gpuocelot/issues/detail?id=16

From<EMAIL_ADDRESS>on February 02, 2012 10:01:54
Labels: -Priority-Medium Priority-Low


Error probing device: map::at

SoapySDR 0.7.1, SoapySDRPlay 0.2.0 on Ubuntu 14.04 lts, x86-64.
I get "Error probing device: map::at" with SoapySDRUtil --probe every time, I can't use Soapy at all.
after uncommenting try and catch @guruofquality mentioned in #50
Probe device
terminate called after throwing an instance of 'std::out_of_range'
  what():  map::at
Aborted (core dumped)

despite what it says about a core dump, I do not have a core dump file because ubuntu. I will post it if I find it or succeed in producing it.
(gdb) run --probe
Starting program: /usr/local/bin/SoapySDRUtil --probe
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
######################################################
##     Soapy SDR -- the SDR abstraction library     ##
######################################################

Probe device
[New Thread 0x7ffff3e03700 (LWP 25327)]
[New Thread 0x7ffff3602700 (LWP 25328)]
[Thread 0x7ffff3e03700 (LWP 25327) exited]
[New Thread 0x7ffff3e03700 (LWP 25329)]
[Thread 0x7ffff3e03700 (LWP 25329) exited]
[Thread 0x7ffff3602700 (LWP 25328) exited]
terminate called after throwing an instance of 'std::out_of_range'
  what():  map::at

Program received signal SIGABRT, Aborted.
0x00007ffff6fbdc37 in raise () from /lib/x86_64-linux-gnu/libc.so.6
(gdb) bt
#0  0x00007ffff6fbdc37 in raise () from /lib/x86_64-linux-gnu/libc.so.6
#1  0x00007ffff6fc1028 in abort () from /lib/x86_64-linux-gnu/libc.so.6
#2  0x00007ffff77e4535 in __gnu_cxx::__verbose_terminate_handler() () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#3  0x00007ffff77e26d6 in ?? () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#4  0x00007ffff77e2703 in std::terminate() () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#5  0x00007ffff77e26c7 in std::rethrow_exception(std::__exception_ptr::exception_ptr) () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#6  0x00007ffff7b6ba97 in std::__basic_future<SoapySDR::Device*>::_M_get_result (this=0x7fffffffd6b0) at /usr/include/c++/4.8/future:598
#7  0x00007ffff7b683cc in std::shared_future<SoapySDR::Device*>::get (this=0x7fffffffd6b0) at /usr/include/c++/4.8/future:804
#8  0x00007ffff7b5d90f in SoapySDR::Device::make (inputArgs=...) at /home/vrm/sw/src/SoapySDR-soapy-sdr-0.7.1/lib/Factory.cpp:192
#9  0x00007ffff7b5db56 in SoapySDR::Device::make (args=...) at /home/vrm/sw/src/SoapySDR-soapy-sdr-0.7.1/lib/Factory.cpp:201
#10 0x0000000000413571 in probeDevice (argStr=...) at /home/vrm/sw/src/SoapySDR-soapy-sdr-0.7.1/apps/SoapySDRUtil.cpp:200
#11 0x0000000000413a1c in main (argc=2, argv=0x7fffffffda08) at /home/vrm/sw/src/SoapySDR-soapy-sdr-0.7.1/apps/SoapySDRUtil.cpp:320
(gdb)

I think its throwing in the factory function. Since its in another thread, maybe there is no trace from that thread.
Can you put a few prints here, see if it gets to the constructor and how far it gets: https://github.com/pothosware/SoapySDRPlay/blob/master/Settings.cpp#L33
If that doesnt give you any prints, I would also try to check the find function: https://github.com/pothosware/SoapySDRPlay/blob/master/Registration.cpp#L34

in the meantime, is SoapySDRPlay 0.2.0 compatible with SoapySDR 0.6.1?
Gave it a try and it finds no device, and there is no "Error probing device: map::at"

in the meantime, is SoapySDRPlay 0.2.0 compatible with SoapySDR 0.6.1?
Gave it a try and it finds no device, and there is no "Error probing device: map::at"

I cant see why 0.6.1 release would be any different in terms of compatibility. It should be fine assuming the support module for SDRPlay is properly installed.

findSDRPlay entered, but SoapySDRPlay::SoapySDRPlay was not entered

Cool, did findSDRPlay throw the error, or did it complete and return nicely?

|      85    std::cerr << "findSDRPlay is returning\n";
|      86
|      87    return results;
|      88 }

well this print did happen

Quick question, are you using SoapySDRPlay release tag "soapy-sdrplay-0.2.0" or master branch (latest commit 31a3a361f77c9ac451f8722bb71df9c6a443614f)?

|||    72         dev["label"] = lblstr;
|||    73         results.push_back(dev);
|||    74         std::cout << "in results: " << lblstr << "\n";
|||    75         _cachedResults[rspDevs[i].SerNo] = dev;

and this in findSDRPlay is not happenning

started with release tag "soapy-sdrplay-0.2.0" then switched to master

nDevs: 1
.devAvail:
teh plot thickens

You said that the constructor was not entered, here is a map::at in the initializer list, so this could be the culprit if for some reason the serial was not passed in: https://github.com/pothosware/SoapySDRPlay/blob/master/Settings.cpp#L34
Also, if it helps, you can find which line in the factory throws, if that helps: https://github.com/pothosware/SoapySDR/blob/master/lib/Factory.cpp#L130

are you sure RSP1 devices have serial numbers?

Not 100 % sure how the manufacturer does it. However, every handle gets a "serial" field written dev["serial"] = rspDevs[i].SerNo;, so this key should exist, even if it was an empty string. I'm curious if for some reason its not present.

||     56      if (rspDevs[i].devAvail)
rspDevs[i].devAvail is 0 and this if block does not run. I have no idea why, since linrad runs fine.
maybe i should try a reboot, but first let me tell the current installation of sdrplay api:
I had 1.8.x API installed and linrad running fine with the very old sdrplay extio dll ported to linux by me (uses the first api version). I wanted to keep the old api for linrad thus skpped sdrplay installer and only installed the .so file and made the symlinks manually. I put the include file in a non-system dir so I compile SoapySDRPlay with this:
cmake .. -DCMAKE_INCLUDE_PATH=/home/vrm/sdrplay/include
I removed the 1.8.x version of the .so file temporary (it's still not present in /usr/local/lib) to avoid problems while trying Soapy. Funny thing linrad still works. ldd libExtIO_SDRPlay.so verifies it's linking against /usr/local/lib/libmirsdrapi-rsp.so which is symlink to the 2.13 API.
I still have a 32bit .so file in /usr/local/lib/i386-linux-gnu, but right now I tried removing it, ldconfig and no effect on linrad ans Soapy still gives this:
Probe device
findSDRPlay entered
nDevs: 1
.devAvail: 0
findSDRPlay is returning
terminate called after throwing an instance of 'std::out_of_range'
  what():  map::at
Aborted (core dumped)

let me try a rebot now :)

I've also verified that nothing comes out of SoapySDRPlay_getClaimedSerials() in findSDRPlay()

after reboot devAvail is still 0, linrad still works.
but I have many of this in dmesg:
[  164.413553] msi2500 3-1:1.0: Registered as swradio0
[  164.413559] msi2500 3-1:1.0: SDR API is still slightly experimental and functionality changes may follow
Maybe something in the kernel has the device, but linrad worked fine so I did not bother with it. Let me check the udev rules.

rmmod msi2500;rmod msi001 fixes my problem

So 'Error probing device: map::at' happens when findSDRPlay() does not find any available devices. Pretty poor error message if you ask me :)

I've just seen this thread. The following modules need to be blacklisted for the RSP1 to work correctly with SoapySDRPlay...
sdr_msi3101
msi001
msi2500
are you saying that there is a problem after you have blacklisted these modules?

are you saying that there is a problem after you have blacklisted these modules?

I have not blacklisted them yet, and your ExtIO ported to linux works okay without blacklisting or rmmoding, that's why I haven't bothered with it yet. After rmmoding SoapySDRUtil --probe works fine, haven't tries any SoapySDR app yet. Also udev rules from 1.8.x are in place and I did not see any notes on blacklisting modules in you API package.

So 'Error probing device: map::at' happens when findSDRPlay() does not find any available devices. Pretty poor error message if you ask me :)

That would have been good to know that the map at error is only when you dont see any devices.
What are are being passed to --probe? For example --probe="driver=sdrplay".
My guess is that it doesnt find any devices, and tries to make it anyway. Even when you dont specify the driver=sdrplay, if sdrplay is the only module available it still probably tries to call its constructor. So its probably happening in the line suggested.
@jazzkutya can you confirm its that its this line: https://github.com/pothosware/SoapySDRPlay/blob/master/Settings.cpp#L34
If its that line, we can just throw an exception like "no sdrplays specified to be opened" and the bad error message goes away. :-)

That would have been good to know that the map at error is only when you dont see any devices.
It's hard to notice this when the only device that could show up gives a cryptic error.

What are are being passed to --probe? For example --probe="driver=sdrplay".
nothing. SPEC in the man page is undocumented.

@jazzkutya can you confirm its that its this line: https://github.com/pothosware/SoapySDRPlay/blob/master/Settings.cpp#L34
If its that line, we can just throw an exception like "no sdrplays specified to be opened" and the bad error message goes away. :-)

How can I confirm this? I'm no good in C++ and don't know how to put a debug print on a chained method call or what it is called :)

        34 SoapySDRPlay::SoapySDRPlay(const SoapySDR::Kwargs &args)
-       35 {
|       36     std::cerr << "SoapySDRPlay::SoapySDRPlay entered\n";
|       37     serNo=args.at("serial");
|       38     std::cerr << "SoapySDRPlay::SoapySDRPlay after serNo()\n";
|       39     // retreive hwVer and device index by API

result:
Probe device
findSDRPlay entered
nDevs: 0
findSDRPlay is returning
SoapySDRPlay::SoapySDRPlay entered
terminate called after throwing an instance of 'std::out_of_range'
  what():  map::at
Aborted (core dumped)

Hopefully this confirms it. It also confirms I'm no good at C++ - turns out it's not chained method call but property initialisation :)

the API docs specifies that DevNm is a unique reference [identifying one device?], possibly implying this should be used instead of the serial number. I'm not sure this really means the constructor and findSDRPlay() should be changed to use DevNm.

I pushed what should be a fix to get a better error message: https://github.com/pothosware/SoapySDRPlay/commit/4e0c67d1e381e64597cfb5463965ef33f129ba35

the API docs specifies that DevNm is a unique reference [identifying one device?], possibly implying this should be used instead of the serial number. I'm not sure this really means the constructor and findSDRPlay() should be changed to use DevNm.

The device number is used once it finds a match for the serial number.
Most SoapySDR modules use serial numbers in the handle to support multiple devices of the same type. Device index numbers can be ambiguous and change. So in practice most device modules actually work the way this one does (at least USB ones do).

this is how it looks with your commit, and reenabled try/catch is SoapySDRUtil:
vrm@zivatar:~$ SoapySDRUtil --probe
######################################################
##     Soapy SDR -- the SDR abstraction library     ##
######################################################

Probe device
Error probing device: no sdrplay device found
vrm@zivatar:~$

many thanks for your time and commit.


xgboost plot tree generates with wrong comparison operator

Thanks for participating in the XGBoost community! We use https://discuss.xgboost.ai for any general usage questions and discussions. The issue tracker is used for actionable items such as feature proposals discussion, roadmaps, and bug tracking.  You are always welcomed to post on the forum first :)

Issues that are inactive for a period of time may get closed. We adopt this policy so that we won't lose track of actionable issues that may fall at the bottom of the pile. Feel free to reopen a new one if you feel there is an additional problem that needs attention when an old one gets closed.
For bug reports, to help the developer act on the issues, please include a description of your environment, preferably a minimum script to reproduce the problem.
For feature proposals, list clear, small actionable items so we can track the progress of the change.

xgboost_plot_tree_issue.zip

Please do not file an issue with an empty description.

Please do not file an issue with an empty description.

I added a pdf file with all the descriptions, think that is sufficient. I'm really sorry about the confusion.
By the above flow chart, the model first checking the Total stat and if the Total stat value is greater than 574 it says it's a legendary pokemon else check's the other parameter. After checking the Total stat's it checks the Egg_Group_1_Undiscoverd which is created after using pandas get_dummies function and qualifies a pokemon legendary if it's value greater than 1. Then it checks the Attack stat and classifies legendary pokemon for when their attack value is greater than 83.
Note: The comparison symbol given in the flow chart is of opposite nature. It's a bug Xgboost library.
The code used to create the model and plot tree is below:
from xgboost import XGBClassifier
mdl = XGBClassifier(max_depth=5,n_estimators=100,learning_rate=0.1)
mdl.fit(X_train,y_train)
predictions_xgb = mdl.predict(X_test)
import xgboost as xgb
import matplotlib
xgb.plot_tree(mdl,num_trees=2)
fig = matplotlib.pyplot.gcf()
fig.set_size_inches(20, 10)

I fail to see any bug. XGBoost always uses the same comparison operator "<". So all test nodes in the plot should be of form [feature] < [threshold]

if the Total stat value is greater than 574 it says it's a legendary pokemon

Not true. That's not what the model does. If the total stat is greater than 574, then the model predicts a negative score -0.166972861, which indicates a low probability for the legendary pokemon. (That is, it is more likely that the pokemon is not legendary.)

it checks the Egg_Group_1_Undiscovered ... qualifies a pokemon legendary if it's value greater than 1.

Also not true. If Egg_Group_1_Undiscovered < 1, then the output is -0.112729825. This is a negative score, so it indicates a low probability for the legendary pokemon. (That is, it is more likely that the pokemon is not legendary.)

I fail to see any bug. XGBoost always uses the same comparison operator "<". So all test nodes in the plot should be of form [feature] < [threshold]

In the present case, Total > 574 required, simillary in other case i.e., [feature] > [threshold].
You can check the dataset below:
https://www.kaggle.com/abcsds/pokemon

No, it is not possible to customize the comparison operator in XGBoost. XGBoost always uses the less than ("<") operator.

if the Total stat value is greater than 574 it says it's a legendary pokemon

Not true. That's not what the model does. If the total stat is less than 574, then the model predicts a negative score -0.166972861, which indicates a low probability for the legendary pokemon. (That is, it is more likely that the pokemon is not legendary.) On the other hand, the model asks other questions if the total stat is greater than or equal to 574.

it checks the Egg_Group_1_Undiscovered ... qualifies a pokemon legendary if it's value greater than 1.

Also not true. If Egg_Group_1_Undiscovered < 1, then the output is -0.112729825. This is a negative score, so it indicates a low probability for the legendary pokemon. (That is, it is more likely that the pokemon is not legendary.)

Thanks a lot for the explanation. It helps a lot.

Good luck with your competition! Keep in mind that each tree in XGBoost produces either a positive or a negative number. A positive number signifies a high probability for the positive class, whereas a negative number signifies a high probability for the negative class.


[GPU-Plugin]

Add fast histogram algorithm
Fix Linux build
Add 'gpu_id' parameter

Also fixes #2218 & #2224

This is great! Please fix the codestyle (mainly namespace indentation issues),

Fixed. Linter doesn't see .cuh files :)


fix(package.xml): install ROS package xml file

Required follow up of #47 so that the package can be used from an install space.

@Rayman FYI


fzf-tmux not installed by conda install

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce
Running conda install -c conda-forge fzf can install only fzf and fzf-tmux is missing in the python bin directory in CentOS 7.
Conda is used as sudo is not available.

Thanks, you might want to report the issue to https://github.com/conda-forge/fzf-feedstock instead. It wouldn't be too hard to change the recipe.

Ok, I reported the issue at https://github.com/conda-forge/fzf-feedstock/issues/52 .
Thanks for an awesome tool!

FYI, fzf 0.53.0 has native --tmux option that can replace fzf-tmux.
https://github.com/junegunn/fzf/blob/master/ADVANCED.md#--tmux


Add more fonts

Can we add more fonts? It only supports Cascadia and Virgil. Is there any specific reason? I would love to add more.

Shall we add the ability to use system fonts?

@Shwetago hi, you can try my fork version: https://excalidraw-zh.com/

We've recently introduced more fonts with https://github.com/excalidraw/excalidraw/pull/8012! Even more will be coming in the future.


Assets hash change

Describe the bug
The furo_asset_hash variable changes hashes for the same resource in the same build.
To Reproduce
Steps to reproduce the behavior:

build any docs
grep furo.css docs/_build/html/*.html | head

docs/_build/html/adaptation.html:      <link rel="stylesheet" href="_static/styles/furo.css?state=cee7d01cd83c5f569d94bbb7">
docs/_build/html/connection.html:      <link rel="stylesheet" href="_static/styles/furo.css?state=11c4ff532c780a855f77bef8">
docs/_build/html/cursor.html:      <link rel="stylesheet" href="_static/styles/furo.css?state=b71df5d38afc2bca89011307">
docs/_build/html/errors.html:      <link rel="stylesheet" href="_static/styles/furo.css?state=898b0ed01020906787d65ca4">
docs/_build/html/from_pg2.html:      <link rel="stylesheet" href="_static/styles/furo.css?state=dce1277c6407b08d176e7d16">
docs/_build/html/genindex.html:      <link rel="stylesheet" href="_static/styles/furo.css?state=c09859bfc41dac5b6e5709be">

Expected behavior
The hash of the resource shouldn't change.
Additional context
pip freeze | grep furo
furo==2020.11.19b18

I've played with this code so much. 😂
I think the asset hash needs to be generated on import and set into a global, instead of generating a new one on each call from html-context.
PRs welcome if you can get to this before I can!

Fixing


Going monorepo

This merges the following repos into riff and changes so that the bare minimum still works:
function-controller
function-proto
function-sidecar
helm-charts
http-gateway
kubernetes-crds
message-transport
riff-cli
topic-controller

The first ~220 commits (up until 261cb68) are preserved history of each repo.
Then, on top of that:

rewrote go import paths
removed vendor/ dirs inside each project
created vendor/ dir at root of repo (converting riff-cli to glide along the way)

Each make default target seems to be working and produces a running executable.

For reference, this is the helper script I used:
#!/bin/bash

# See https://stackoverflow.com/questions/1425892/how-do-you-merge-two-git-repositories/10548919#10548919

REPOS="function-controller function-proto function-sidecar helm-charts http-gateway kubernetes-crds message-transport riff-cli topic-controller"

function rewrite_paths() {
	git filter-branch -f --tree-filter "zsh -c 'setopt extended_glob && setopt glob_dots && mkdir -p $1 && (mv ^(.git|$1) $1 || true)'" -- --all
}

function merge_repos() {

	for repo in riff $REPOS ; do
		git clone https://github.com/projectriff/$repo
		pushd $repo
			echo $repo >> /tmp/counts
			git rev-list --count HEAD >> /tmp/counts
			echo "" >> /tmp/counts
			rewrite_paths $repo
		popd
	done

	pushd riff
		for repo in $REPOS ; do
			git remote add  --fetch $repo ../$repo
			git merge --allow-unrelated-histories --no-edit $repo/master
		done

		echo "---------------" >> /tmp/counts
		git rev-list --count HEAD >> /tmp/counts
	popd

}

function rewrite_imports() {
	pushd riff

		for repo in $REPOS ; do
			pushd $repo
				GOFILES=$(find pkg cmd -type f -name '*.go')
				for f in $GOFILES ; do
					sed -i '' 's|"github.com/projectriff/\(.*\)"|"github.com/projectriff/riff/\1"|g' $f
				done
			popd
		done
	popd
}

rm /tmp/counts
mkdir work
pushd work
popd work

#merge_repos()
rewrite_imports

Does this show as branch-per-repo or are the commits interleaved by time? I can't tell from Github's UI, but it looks like interleaved.

I believe it's branch-per-repo. I don't expect you'll be checking those commits anyway ;)
I left them with 2 parents because it's easier to spot that there is something special happening then

the riff-cli is not included in the top-level build

the riff-cli is not included in the top-level build

Added that. Wanted to give it special treatment so we can go get github.com/projectriff/riff


Voice Calling turns off screen erroneously

The voice call feature is intended to blacken the screen when the user is in-call via use of the ambient light sensor (e.g. if the phone is against the user's ear, the screen won't needlessly be on).
However it does this erroneously when the user has a call going on in the background, and may incidentally cover their ambient light sensor. This behavior continues after the user has left the call for a short amount of time.
This should be fixed so that the screen is only automatically turned off when the user has the in-call screen open.

waiyd
Which device do you have ?

Galaxy S7 Edge on Marshmallow
Version 0.6.8


Fixes issue #4469 selecting text doesn't always work in upload wizard

Description (required)
Fixes #4469
What changes did you make and why?

Removed the initial setText calls for the caption and description editText fields, since they seem to unneeded and set later on lines 148 and 152. This seems to have fixed the issue, where text was not selectable (and therefore copyable) when a user click the 'next' button and then returned to the screen with the caption and description.

Tests performed (required)

Opened app and started upload wizard, after taking a picture and adding a caption and description
Tested that selecting the text still works before pressing 'next'
Clicked 'next' and then 'previous' and then confirmed that the text could now selected as expected
Tested progressing to and then back from multiple steps during upload process (after adding 'depicts' and 'categories' etc.)
Tested an upload with the change, the caption and description working as expected
Tested ProdBeta on Google Pixel 4a (5G) with API level 30 (Android Version 11)

And thanks for nicely written PR too :)

@benryder1988 This commit unfortunately broke the "Copy to subsequent media" feature. Would you mind checking? Thanks! :-)

This commit also unfortunately breaks the feature that auto-fills the caption of pictures uploaded via Nearby.
I reverted it for now, but looking forward to a new pull request as fixing the test selection bug would be really nice :-) Thanks!


Issue with Loading Environment Variables in Docker Container

When I run my app using Docker, I get errors like "Invalid environment" for all my environment variables in the browser console. However, I'm certain that my environment variables are loaded into the container, as I have verified using the docker exec bash command.
The issue I noticed is that when I build my image with the .env file included, everything works fine. However, when I add .env to .dockerignore, I encounter this problem. It seems that the library only works with .env files and cannot get environment variables from the system or the Docker container.

Experiencing the same issue after upgrading to Docker 4.30.0. I'm not sure if it's related to the upgrade.

Are you using turborepo to build/run ?
https://turbo.build/repo/docs/reference/run#--env-mode-option
Turbo  --env-mode is strict by default (only env variables specified in the config are passed to the container)
Using
$ turbo run build --env-mode loose

fixes this

Are you using turborepo to build/run ?
https://turbo.build/repo/docs/reference/run#--env-mode-option
Turbo --env-mode is strict by default (only env variables specified in the config are passed to the container)
Using
$ turbo run build --env-mode loose

fixes this

No I'm using create-t3-app

Are you using turborepo to build/run ?
https://turbo.build/repo/docs/reference/run#--env-mode-option
Turbo --env-mode is strict by default (only env variables specified in the config are passed to the container)
Using
$ turbo run build --env-mode loose

fixes this

No I'm using create-t3-app

that has nothing to do with deployment ?
Check your Dockerfile / Docker commands if it contains "turbo"..

I don't think this is related to this library.
You need to:

for client side variables, the values have to be known at build-time as these are (most likely) statically replaced. So your Dockerfile must provide these (either from an env file or build args)
for server variables, you don't want to inject these when you build the image. you instead want to provide these when starting the container. To make the build not fail when the variables aren't there, you can set the skipValidation option to something that identifies your build pipeline.


Oauth integration

Would be great to see here oauth integration :)

I'm open for pr :) But yes would be nice :)


Support both Vue2 & Vue3

Are you going to have a version that will support both Vue2 & Vue3 not just as a separate library? Currently installing @next throws Cannot call a class as a function error in Vue2.

Sorry, it's not possible.

@adamberecz I also have trouble using this library with nuxt. Did you resolve the error message? Or did you switch to another library? If yes, which one? Thanks!

No I didn't. I've decided to create a custom Vue wrapper for nouislider on my own.


Hyperlink to zulip-devel-announce added.

This way it is a bit simpler for users to find the page to subscribe.

Merged after tweaking the commit message to have a docs: prefix, thanks @guettli!
I also added f6376d57e46d8ff549628e6ced0e0419a8b3fed5 on top to further indicate that these forums are really low traffic.


mosca or aedes, which one is better?

mosca or aedes, which one is better in production?

Mosca is more battle tested and more stable. Aedes is more lightweight, fast, but as you have seen it requires more hardening against edge cases.

@mcollina is Mosca gonna be merged/based on Aedes eventually?

Big question. In theory yes, but it might also stay two separate things.
Il giorno ven 5 mag 2017 alle 19:56 Jyotman Singh<EMAIL_ADDRESS>ha scritto:

@mcollina https://github.com/mcollina is Mosca gonna be merged/based on
Aedes eventually?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mcollina/aedes/issues/101#issuecomment-299532744, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADL41w8xLo8yRyGrweZwV9hPsxPny7Rks5r22K0gaJpZM4M1LuQ
.


Fix and un-grey any iterator tests.

Scope & Purpose
Stabilize the "any" tests and un-greylist them. Current approach is just to relax the distribution constraints a bit. It's an arbitrary threshold anyway, and the current algorithms are working correctly for the design, and the design is about the best we can without significant work (and probably significant slowdown).

[x] Bug-Fix for devel-branch (i.e. no need for backports?)

Related Information

[x] There is an internal planning ticket: https://github.com/arangodb/planning/issues/4682

Testing & Verification
This change is already covered by existing tests.
Jenkins: http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr/7393/

Further question: IMHO this PR just increases the chances of the test passing, but it doesn't guarantee it. Is there any way we can improve on this?
IMHO the any algorithm is quite ok for MMFiles, but for RocksDB we want to avoid expensive seeks, so we will often return the same documents.
We could lower the test expectations to take this into account, and for example be satisfied if 100 calls to any() produce at least 2 different documents for RocksDB or such.

Additionally, if we are sure the test passes under almost any (pun intended) circumstances, we should rename it and remove the "nondeterministic" tag from its name.

Interestingly enough, the RocksDB test with the old threshold managed to run about 140 times on my machine before failing once, while the MMFiles test usually failed within 10 iterations. I think our thresholds were simply too tight to account for any single run with the pseudo-random behavior.
We could potentially increase the sample size/space to tighten the variance more, in particular for MMFiles (not RocksDB since it is so slow). At that point if it fails there may be something more seriously wrong, so we could remove the non-deterministic flag. What do you think, @jsteemann?

We can definitely increase the sample size (provided it does not blow up test times) and even lower the test assumptions so that we expect the distribution to be less random than we expect now.
I am also happy with something like "if we call any() 1000 times we want to see at least 5 different documents or such". That is good enough, given that we cannot guarantee any level of randomness.

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr/7424/

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr/7427/

https://<IP_ADDRESS>/view/PR/job/arangodb-matrix-pr/7457/

https://<IP_ADDRESS>/view/PR/job/arangodb-matrix-pr/7466/

Tests blue


Integration restart manager

What type of PR is this?
/kind feature

What this PR does / why we need it:

Make the integration test framework able to stop the manager running in an envtest instance.
Reuse the envtest instances in the test suites.
Reorganize some suites for better performance while running in parallel.

Which issue(s) this PR fixes:

Relates to #2993
Special notes for your reviewer:
Does this PR introduce a user-facing change?

None

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/test pull-kueue-test-integration-main

/cc @alculquicondor

Make the integration test framework able to stop the manager running in an envtest instance.
Reuse the envtest instances in the test suites.
Reorganize some suites for better performance while running in parallel.

Would you mind splitting this into dedicated PRs for the ease of reviewing?

Make the integration test framework able to stop the manager running in an envtest instance.
Reuse the envtest instances in the test suites.
Reorganize some suites for better performance while running in parallel.

Would you mind splitting this into dedicated PRs for the ease of reviewing?

I'd prefer not to, the only file that contains "substantial" changes is test/integration/framework/framework.go, for the rest of the , it's just about changing the place where fwk.Init() is called (BeforeSuite instead of BeforeAll or similar) and for the webhook related tests moved in one package.

I'd prefer not to, the only file that contains "substantial" changes is test/integration/framework/framework.go, for the rest of them , it's just about changing the place where fwk.Init() is called (BeforeSuite instead of BeforeAll or similar) and for the webhook related tests moved in one package.

You can generally expect much faster reviews for PRs which are small and separate concerns, please read:

https://www.kubernetes.dev/docs/guide/pull-requests/#smaller-is-better-small-commits-small-pull-requests
https://google.github.io/eng-practices/review/reviewer/speed.html#large

In particular, moving the webhooks feels like separate PR, we would also maybe be able to see what is the performance benefit of various improvements separately. Otherwise it is hard to tell if one optimization helps and how much, if there is another optimization biasing the stats.
Anyway, if you want I can try to review the PR in this form, but seeing that the process may not be as effective as it could.

I'd prefer not to, the only file that contains "substantial" changes is test/integration/framework/framework.go, for the rest of them , it's just about changing the place where fwk.Init() is called (BeforeSuite instead of BeforeAll or similar) and for the webhook related tests moved in one package.

You can generally expect much faster reviews for PRs which are small and separate concerns, please read:

https://www.kubernetes.dev/docs/guide/pull-requests/#smaller-is-better-small-commits-small-pull-requests
https://google.github.io/eng-practices/review/reviewer/speed.html#large

In particular, moving the webhooks feels like separate PR, we would also maybe be able to see what is the performance benefit of various improvements separately. Otherwise it is hard to tell if one optimization helps and how much, if there is another optimization biasing the stats.
Anyway, if you want I can try to review the PR in this form, but seeing that the process may not be as effective as it could.

Most of the changes are trivial. Also the webhook moving needs to be in the same PR as the Init call changes, otherwise we'll have a broken intermediate state.

I would prefer some of the drive-by fixes/cleanups to be in dedicated PRs so that they could cherry-pick if needed. Also, to reduce distractions and minimize the cognitive load. As mentioned in https://github.com/kubernetes-sigs/kueue/pull/3054#issuecomment-2360285542, I believe it makes the review process easier and faster overall.
In any case, I think I understand the changes now, and are ok to go with them.
/lgtm
/approve

/hold

1/5
/test pull-kueue-test-integration-main

2/5
/test pull-kueue-test-integration-main

3/5
/test pull-kueue-test-integration-main

4/5
/test pull-kueue-test-integration-main

5/5
/test pull-kueue-test-integration-main

5 out of 5 ok, @mimowo please have another look

/unhold

/lgtm


LaTeX/Overview: Add it with macros & examples

It brings LaTeX to the list! With macros support.

I don't think we really need a  javascript loop to define five macros, but that probably doesn't hurt.

I don't think we really need a javascript loop to define five macros, but that probably doesn't hurt.
Except the fact that it's might be indeed longer than to just type five times the same macros. Size/performance issues are negligible and I was a bit lazy :c (okay that's not an excuse).

Conflicts resolved.

Conflicts resolved.


Use with Storyboard?

Thanks for this great library.
I am having trouble using it with Storyboard. If I do the following:
@IBOutlet var input: PinCodeInputView?

Xcode complains about using a generic type, whereas if I do something like:
@IBOutlet var input: PinCodeInputView<ItemView>?

it doesn't work either (@IBOutlet property cannot have non-'@objc' class type 'PinCodeInputView).
Is there any way to use the library with Storyboard? Thanks!

Ditto


Add email utility in support of #33

Add an email utility to support #33 using the hover email SMTP relay.

Added in release 2021.03.17


Release with rig

I don't imagine this will work the first time, but let's try it.

So there's still manual work in preparing a release:

Change version.sbt.  rig will trim the -SNAPSHOT suffix, so this isn't strictly necessary.
Make versions.md match.  We could automate this, but haven't.
Update the changelog.  Automating this is hard unless we get a lot crankier about pull request comments.  We could maybe automate timestamping it.  Maybe we could cut our changelog over to GitHub releases, but we've been hesitant to do that because the source of truth for that is not in git.
Tag the release, which ought to match version.sbt, at least up to snapshot.  Rig is good at turning the version into a tag, but then we don't have a trigger for which builds get released to sonatype and which ones are just snapshots.

Does appveyor use a wild default platform encoding?
https://ci.appveyor.com/project/ChristopherDavenport/http4s-m0hqu/build/1.0.13/job/3802rum34ot9xts0#L4059

A half-baked proposal:

Make versions.md match. We could automate this, but haven't.

We have a build step that generates toml.  We have git.  We could spit out the latest tag from each minor release, and generate that into versions.md.

Update the changelog. Automating this is hard unless we get a lot crankier about pull request comments. We could maybe automate timestamping it. Maybe we could cut our changelog over to GitHub releases, but we've been hesitant to do that because the source of truth for that is not in git.

We could go back to a convention of a notes/*.md file per release.  We own our own data.  And we could have a release step that puts this in the GitHub releases.  And then GitHub Releases is our changelog.

Rig is good at turning the version into a tag, but then we don't have a trigger for which builds get released to sonatype and which ones are just snapshots.

We trigger a release build if version.sbt does not end in -SNAPSHOT and (as a sanity check) we have release notes.
Now cutting a release is as simple as providing notes/X.Y.Z.md and setting versions.sbt.  Travis/rig will take care of updating versions.md, publishing the release notes, publishing the release to sonatype, creating a tag, and bumping the version number on the branch.
🤔

I solved the versions list.  We would just need to be sure to tag before making the site on a release build.
I tried GitHub Releases and deem them unfit for use.  They don't read the description from an annotated tag's body, and any attempt to change the description through the API updates the date, which destroys the timeline.  I guess our sanity check could be to grep the changelog for release notes.

Going to take a different approach on this.


feat: add deleteAll function

:tada: This issue has been resolved in version 2.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bug: Shouldn't close video tabs

Any pinned tab will never be removed.  Will that solve your problem?
Typically, if I care about a video, it will be in the foreground anyway and not be removed.  Also, if minimum tabs apply to each window, then a video playing in a second window shouldn't be closed either.

What about podcasts? or music in a tab? I have been using this since yesterday. I normally use Tab wrangler in Chrome and it's just closing podcast windows mid sentence as well as videos. Very frustrating.

@bonafideduck I was mentioning Tab wrangler because it is a model of how yours should work. I have had to switch back to Tab Wrangler because it doesn't close mid video on an active tab.

@cantonalex  I'm considering a different route.  I'm going to look into replacing CloseTabs for Safari with a port of TabWrangler.  Were you trying CloseTabs because it was the only thing available on Safari (DuckTabs on Chrome) or was there a capability in it that would be nice in tab wrangler?


Double Audio

Script version
5.5
Describe the bug
Double audio occurs if the video has a watch history, and in all cases, skipping a section through text causes double audio.
Expected behavior
Screenshots
If applicable, add screenshots or a video to help explain your problem.
Desktop (please complete the following information):

OS: Windows 11
chrome
Version 125.0.6422.114

Additional context

Windows 11, Chrome
I have an issue similar to this: When I click on a video, an ad plays in the background. And when I press K or SPACE bar, either the ad pauses or the video pauses. Not the mention when both of them play at the same time, it drives me absolutely insane because it sounds like 5 tabs playing at once. Very weird!

same issue bruh
Windows 10
Chrome Version 125.0.6422.142 (Official Build) (64-bit)
Version 5.5

I'm getting the same thing, it's like playing an add in background.

Youtube shorts are just black video.

same issue and double audio please fix it after update

Same issue here, opening any video will play two audios and even if I pause the video there will still be the audio of the video playing in the background

Same Issue. Two audios over one video. When the video first loads up it's just the two audios and a blurred screen for the video until I refresh. Windows 11 on YouTube, and the browser is Chrome.

Brave browser here, just update to 5.5 and got the same issue.

Same here. At least double audio stops when i pause the playback through youtube controls.
Brave: Version 1.66.118 Chromium: 125.0.6422.147 (Official Build) (64-bit)
OS: Manjaro KDE

Same Issue

Youtube shorts are just black video.

The same things happening on my end.

I am facing a double sound in Firefox on windows 11. After pausing the video, the audio is playing automatically.

Fix that worked for me:
https://github.com/TheRealJoelmatic/RemoveAdblockThing/issues/460#issuecomment-2146297048


[+] Ignore /plugins from Nuxt.js

Fix #3

This PR indeed solves the problem, tested with hexgis/skyviewer. Approved :+1:


how to get out all dot from the address?

Failed at this way:
// totalAmount
let info = await api.tx.balances.transferKeepAlive(toAddr, amount).paymentInfo(pair);
// fee
let fee = info.partialFee.toNumber();
// sendAmount - totalAmount - fee
let transfer = api.tx.balances.transferKeepAlive(toAddr, amount-fee);
let txhash = await transfer.signAndSend(pair);

sendAmount  = totalAmount - fee.
why tx on-chain is always failed...

Substrate has 2 transfer methods -

balances.transfer
balances.transferKeepAlive

the difference is the following: the keep alive version will fail when the remaining balance falls below the existential deposit. The transfer will allow the chain to reap the account if the balance falls below existential.
So in the example above, since you are using keep alive, the account will never be reaped and will always be kept to at least existential.

Substrate has 2 transfer methods -

balances.transfer
balances.transferKeepAlive

the difference is the following: the keep alive version will fail when the remaining balance falls below the existential deposit. The transfer will allow the chain to reap the account if the balance falls below existential.
So in the example above, since you are using keep alive, the account will never be reaped and will always be kept to at least existential.

transfer will allow the chain to reap t

tks, I tried to reap the account with transfer and succeed,
could you give me some API-docs or link about the difference above so that I can learn more

transfer will allow the chain to reap t

tks, I tried to reap the account with transfer and succeed,
could you give me some API-docs or link about the difference above so that I can learn more

In Substrate, all the available calls are as configured on the runtime. (Some chains don't have the balances module for instance). So the API received the metadata from the chain and decorates based on that.
The metadata has some documentation attached, see for transfer here - https://polkadot.js.org/docs/substrate/extrinsics#transferdest-lookupsource-value-compactbalance
(The above is just the documentation supplied by the runtime, with the actual source being https://github.com/paritytech/substrate/blob/master/frame/balances/src/lib.rs#L437)

In Substrate, all the available calls are as configured on the runtime. (Some chains don't have the balances module for instance). So the API received the metadata from the chain and decorates based on that.
The metadata has some documentation attached, see for transfer here - https://polkadot.js.org/docs/substrate/extrinsics#transferdest-lookupsource-value-compactbalance
(The above is just the documentation supplied by the runtime, with the actual source being https://github.com/paritytech/substrate/blob/master/frame/balances/src/lib.rs#L437)


docker compose resource url 404

Describe the bug (must be provided)
https://oss-cdn.nebula-graph.com.cn/nebula-graph-studio/3.2.2/nebula-graph-studio-v3.2.2.tar.gz 404 in https://docs.nebula-graph.com.cn/3.0.0/nebula-studio/deploy-connect/st-ug-deploy/#docker_studio step 1

Thanks @zhiwen95
We will fix this soon, please use https://github.com/vesoft-inc/nebula-studio/archive/refs/tags/v3.2.2.tar.gz instead first before it's detachable first ;)

fixed, you can download now


LDAP Authentication with '@' in Usernames

Hi,

Gogs version (or commit ref):
v0.9.71
Operating system:
RHEL7, Openldap

Description
For some reason we have '@' in our usernames (uid), like a.b@c
For a testuser without '@' LDAP authentication/login works.
For users with '@'  I'm getting an login error.
So I guess if the username contains an '@' it is regarded as an email?
Users are located within an ou=users,...
and filter is (&(objectClass=posixAccount)(uid=%s))
Thanks
Markus

We have the same problem in our organization.
+1

I can confirm that. We can't authenticate using the krb5PrincipalName LDAP attribute (in addition to uid) for that reason (Kerberos principal names use "@" as the realm separator).
According to the log the resulting LDAP filter looks correct and the authentication is successful ("Bound successfully with userDN..."). But I get the login form again.

This might be silly question, but why would you prefer krb5PrincipalName over uid / cn? I think the real problem might not be ldap authentication, but authentication in general. If you check login form it allows you to login with either username or with email. Username with @ might get interpreted as an email during login

@skorhone I don't prefer it it over uid/cn, but I usually allow several attributes ORed in an LDAP filter (e.g. uid, kr5PrincipalName, mail etc.). That way the user is free to choose what to use as login. But the real advantage is that it's possible to use multiple authentication methods with a single user database, if the applications supports this. To give a practical example: With applications supporting this (Dovecot, Grafana and many others) we use Kerberos/GSSAPI authentication (secure and convenient, very common in enterprises) but sometimes allow other authentication methods as fallback. By using multiple attributes ORed in an LDAP filter, we are still able to map this to the correct identity, regardless of the authentication method used (in which case an application might see different IDs).
I think it's wrong to assume a different attribute just because a certain character appears in a login. If someone wants to allow either uid or email to be used as a login name, than this should be done with an LDAP filter (with ORed attributes).

Merge this into https://github.com/gogits/gogs/issues/4827.


scheduled_messages_overlay_ui: Convert module to TypeScript.

I manually tested the module by scheduling some messages (both type "stream" and "private"), testing keyboard events like delete to remove the scheduled msg, enter  to edit the msg. Also tested if the messages are sorted in correct order based on their scheduled time.

Merged, thanks @LaPulgaaa!


Tree in Nav &Table (like Sencha)

If you cross the accordion with the navigation controls or table can be a new element - a tree.
Example:
http://docs.sencha.com/ext-js/4-0/#!/example/tree/treegrid.html
Sorry for my English. Used Google Translator.

Interesting. +1 :D

Sorry, we won't be going down this route for our navs. It's a bit heavy handed and a paradigm that I've not seen used well on the web.

Google uses a tree-like menus in products such as Reader and Documents. ))


Replace Attachment Not Working

Describe the bug in current situation
When trying to replace and attachment in PROD (using 'Replace Attachment'), error is thrown (and you subsequently can't replace the attachment). This doesn't appear to be occurring in TEST? This appear to occur on all requests in PROD.
Private Zenhub Image
PSA-2023-33480 is example.
Link bug to the User Story
Impact of this bug
Describe the impact, i.e. what the impact is, and number of users impacted.
Chance of Occurring (high/medium/low/very low)
Pre Conditions: which Env, any pre-requesites or assumptions to execute steps?
PROD
Steps to Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Actual/ observed behaviour/ results
Expected behaviour
A clear and concise description of what you expected to happen. Use the gherking language.
Screenshots/ Visual Reference/ Source
If applicable, add screenshots to help explain your problem. You an use screengrab.

working as expected, so closing


Update/new blockstreamgreen guide

This fork is unmaintained, you'll want to open this in https://github.com/start9labs/electrs-startos instead :)


ClassCastException on IMAP sync #811

Hi, I see the same problem of issue #811 classcastexception: com.fsc on my Inbox folder after a mail server was changed. Other folders look fine but I cannot receive new mail now.
K-9 Mail version: 5.203
Device Model: Samsung Galaxy S4
Android version: 5.0.1
I also found this problem mentioned at
http://grokbase.com/t/gg/k-9-mail/142prrhagm/classcastexception-com-fsc-error-on-some-folders

You should be able to see the full stack trace in the k9mail-errors folder for that account. It would be really useful if you could paste that here :)

I just had a look, but my K9mail-errors folder is empty.

:-1:  We probably need a debug log then - if you're able to do it.

It seems like the problem disappeared.
I still have the same version of K-9 Mail and didn't change anything but now I see all the latest mail!

I currently have this problem. Here is the output that got dumped into my k9mail-errors folder:
ClassCastException: com.fsck.k9.mail.store.imap.ImapList cannot be cast to java.lang.String

K9-Mail version: 5.206
Device make: HUAWEI
Device model: HUAWEI GRA-L09
Android version: 5.0

java.lang.ClassCastException: com.fsck.k9.mail.store.imap.ImapList cannot be cast to java.lang.String
	at com.fsck.k9.mail.store.imap.ImapList.getString(ImapList.java:36)
	at com.fsck.k9.mail.store.imap.ImapFolder.handleFetchResponse(ImapFolder.java:870)
	at com.fsck.k9.mail.store.imap.ImapFolder.fetch(ImapFolder.java:754)
	at com.fsck.k9.controller.MessagingController.fetchUnsyncedMessages(MessagingController.java:1317)
	at com.fsck.k9.controller.MessagingController.downloadMessages(MessagingController.java:1162)
	at com.fsck.k9.controller.MessagingController.synchronizeMailboxSynchronous(MessagingController.java:951)
	at com.fsck.k9.controller.MessagingController$9.run(MessagingController.java:739)
	at com.fsck.k9.controller.MessagingController.runInBackground(MessagingController.java:204)
	at com.fsck.k9.controller.MessagingController.access$000(MessagingController.java:115)
	at com.fsck.k9.controller.MessagingController$1.run(MessagingController.java:173)
	at java.lang.Thread.run(Thread.java:831)

Some notes:

This only happens on INBOX. All other folders can be refreshed without trouble.
I had this problem a few months ago (around early March I believe, but I don't remember what version I was running). I believe at the time I re-installed the app and the problem went away.
The problem crosses versions. I updated three days ago to 5.206. Whatever version I had before that also had the problem. I updated hoping it would solve the problem, but obviously it didn't.
I can provide more debug output if needed.

It would be useful to see the verbose output showing the IMAP response received from the server. From the error it's not clear whether it's our parsing or the server's bad implementation (for example as seen in https://github.com/k9mail/k-9/issues/2474 )


Fix time picker to update period based on selected date

Requires https://github.com/openstatusHQ/time-picker/pull/21 to be merged

Thanks @zammitjohn! 🙏


magic_number = pickle_module.load(f, **pickle_load_args) EOFError: Ran out of input

Hi,
I'v been trying to use this model, but I came up with bugs showed below:
magic_number = pickle_module.load(f, **pickle_load_args)
EOFError: Ran out of input
how can I solve this problem?
thank u so much!

I think the pickled data file has incorrect file size (e.g., incomplete/failed download of the files). Could you check if PATH_TO_CG2ALL/model/*.ckpt files have non-zero file sizes?

thank u so much for replying! I've already solve this problem.It is exactly because when I download the code, I missed the Calpha.ckpt file. So the procedure went wrong.
It is an excellent job, we use this model to transform the 28058 idrs sequences from cg2all. Thank u for your work and hope all the best!
--
发自我的网易邮箱
在 2024-06-23 00:17:45，Lim Heo @.***> 写道：
I think the pickled data file has incorrect file size (e.g., incomplete/failed download of the files). Could you check if PATH_TO_CG2ALL/model/.ckpt files have non-zero file sizes?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.**>


Add support for excluding words

Resolved in https://github.com/hrai/auto-capitalise-sentence/pull/82


Diraccess has problems with accessing files when exported

Tested versions

Reproducible Godot v4.3.stable

System information
Godot v4.3.stable - Windows 10.0.22631 - GLES3 (Compatibility) - NVIDIA GeForce RTX 4070 Ti (NVIDIA; <IP_ADDRESS>94) - 13th Gen Intel(R) Core(TM) i7-13700K (24 Threads)
Issue description
Diraccess seems to have a few problems with accessing files once exported, this has been observed to happen when trying to get the name of files within a directory. In my main project I can use get_files_at() to get the names of all the files in a directory, however once trying to do that in the export it only exports part of 3 file names.
In the minimum reproducible project on 4.3 stable, the folder is not found at all

Steps to reproduce
Diraccess get files at --> folder
then export
either the folder won't be found or the file names listed will be incomplete
Minimal reproduction project (MRP)
newGameProject.zip

This is what it looks like in my current project in the editor:

and this is what it looks like in the windows exported console

Note how the file name is just cutoff for opendyslexic-regular.otf.imp

the res:// filesystem is imported/converted into godot internal format during the export process.
.import files contain the mapping information of the source file path to the imported internal file (imported files are .scn for scenes, .ctex for textures).
You should be able to load data from the .import files using ConfigFiles, reading the [deps]: source_file string, which you can then pass to load(). Though, more easily being able to explore/dynamically load items in folders at runtime is a very useful feature.
I know there are at least a couple other issues and proposals relating to the exported project filesystem, but I can't seem to find them.

Thanks for the clarification o7 I think a warning being shot out when using diraccess get_files and etc might be a good idea then if the prefix is res


Add getter and setter for HTML element style attributes

Is this a good fit for purescript-simple-dom?
Unfortunately, unsafeGetStyleAttr and unsafeSetStyleAttr must misbehave somewhat to meet their goals, because the range of attributes that may be set include— beyond the specification— methods to get/set browser-specific CSS extensions. For that reason, I thought it best to match the respective getAttribute and setAttribute type signatures.
One difference: calling getAttribute on a nonexistent attribute name returns null, while calling getStyleAttr on a nonexistent style attribute returns undefined. I'm unsure if this is a problem.

Just realized that styleProperty would be a better name.

I think so, thank you!
(I might change the name to styleProperty in a later commit, but looks okay to me :+1:)


fail to switch python version

mac installed from brew
➜  ~ pyenv versions
  system
* 3.4.3 (set by /Users/broven/.pyenv/version)
➜  ~ python --version
Python 2.7.10

I switched python3.4.3 version in pyenv, but when I type python --version still the old one. why?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/32534379-fail-to-switch-python-version?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github).

i fix it


Revert MDC Web imports to /dist

This PR is a clean up PR if https://github.com/material-components/material-components-web/pull/3245 is merged in. PR https://github.com/material-components/material-components-web-react/pull/182 changed components to point to /dist/mdc.foo.js files in the interim.

blocked by https://github.com/material-components/material-components-web-react/pull/424

moving to v0.9.0 since we're moving select to a later release.

Fixed in #697


WIP: Work with SSM via wrapper

Added list functionality
Added push functionality
delete functionality should be fixed via PR to the opensource project (I've already checked the code - it should be easy)

Testing:

 Implementation ssm-tool for parameter store

 Implementation ssm-tool for parameter store

Do we really need to have a separate command for this? I'm curious.

@AutomationD no, it's just to show and for testing - if you like it - I will update with deleting old command

Do we really need to have a separate command for this? I'm curious.

@AutomationD no, it's just to show and for testing - if you like it - I will update with deleting old command

Do we really need to have a separate command for this? I'm curious.

@AutomationD no, it's just to show and for testing - if you like it - I will update with deleting old command

Ah, makes sense.
@igorkotof could you please give more overview of which tool are we using, maybe post a link in the PR text?

Do we really need to have a separate command for this? I'm curious.

@AutomationD no, it's just to show and for testing - if you like it - I will update with deleting old command

Ah, makes sense.
@igorkotof could you please give more overview of which tool are we using, maybe post a link in the PR text?

Do we really need to have a separate command for this? I'm curious.

@AutomationD no, it's just to show and for testing - if you like it - I will update with deleting old command

Ah, makes sense.
@igorkotof could you please give more overview of which tool are we using, maybe post a link in the PR text?

@AutomationD I've added

Do we really need to have a separate command for this? I'm curious.

@AutomationD no, it's just to show and for testing - if you like it - I will update with deleting old command

Ah, makes sense.
@igorkotof could you please give more overview of which tool are we using, maybe post a link in the PR text?

@AutomationD I've added

Do we really need to have a separate command for this? I'm curious.

@AutomationD no, it's just to show and for testing - if you like it - I will update with deleting old command

@igorkotof So, what's the plan with global secrets?

Do we really need to have a separate command for this? I'm curious.

@AutomationD no, it's just to show and for testing - if you like it - I will update with deleting old command

@igorkotof So, what's the plan with global secrets?

Do we really need to have a separate command for this? I'm curious.

@AutomationD no, it's just to show and for testing - if you like it - I will update with deleting old command

@igorkotof So, what's the plan with global secrets?

@AutomationD keep like $(INFRA_DIR)/env/$(ENV)/secrets/global.json

Do we really need to have a separate command for this? I'm curious.

@AutomationD no, it's just to show and for testing - if you like it - I will update with deleting old command

@igorkotof So, what's the plan with global secrets?

@AutomationD keep like $(INFRA_DIR)/env/$(ENV)/secrets/global.json


On-demand loading

Hello,
Is there a way to load the plugin on-demand? It's an option that is provided by a few plugin managers, e.g. vim-plug, allowing you to load a plugin that way:
Plug 'junegunn/vim-github-dashboard', { 'on': ['GHDashboard', 'GHActivity'] }

The on parameter specifies which function of the plugin triggers the loading.
However, I can't seem to succeed in applying this method to suda, which is a relatively slow plugin to load (compared to the other plugins that I use, I mean; suda takes about 30ms to load according to vim --startuptime).
So the idea would be to only load suda when needed (that is, when a file is saved using this plugin)... Any idea?
Any help would be greatly appreciated!
Best regards,
Dupond

The problem is that you don't directly call a command (like :GHDashboard) that would trigger autoloading.
You could possibly install this as an optional plugin using the native package manager, then map something to packadd! suda.
Other than that I don't think 'on' and 'for' are useable because this plugin isn't just used for a specific trigger command or filetype.

OK, thanx for the explanation.


Nicer warnings for missing clauses

missing after clause should be pointed out
OK.for do
  a <- foo()
end

solved in 1.9.4


Fails to display updated code

We have written the needed data into your clipboard because it was too large to send. Please paste.

Please fill out the issue template and follow our issue reporting guideline: https://github.com/Microsoft/vscode/blob/master/CONTRIBUTING.md#reporting-issues.


Error on reconnect

When reconnecting the send() method checks to see if _ws has a value and sends if it does. However, if you send during reconnect with _ws in CONNECTING readyState it throws an error. I think send should check for the value, and the readyState.

Hi, thanks for reporting.
Should be fixed in latest release (v4.1.3)


Use --depth 1 once issue related to it in Github CLI get resolve

Description
To clone the repository in any script we don’t want to pull the complete history by using git clone REPO_NAME. There is a option in git command to avoid this by using git clone --depth 1 but there is a issue in Github cli for this that with --depth 1 gh pr create give error. Due to which can’t use this option for now in scripts mentioned below.

hack/choco/update-choco-package.sh
hack/homebrew/update-homebrew-package.sh

The motive of this issue is to track the issue and use --depth 1 once the other issue get resolved.

FYI. I had this issue lately and we decided to use treeless clone instead for our CI for a custom need.
We found this article https://github.blog/2020-12-21-get-up-to-speed-with-partial-clone-and-shallow-clone/ that explains a different way to clone a repo.
Blobless and treeless works great to use gh pr create right after :) You can pick the one that works best for your need.

Awesome, thanks for the pointer @vschoener !!

This looks like it's in the code base already. Closing.


enhanced comment on config file

blank lines are not a problem.

@gdams : for your fears about bad editing  :)


End-to-End Testing React App with Jest

BUG w/: https://github.com/English3000/crdwk/blob/master/frontend/pages/home/__tests__/AuthForm-test.js
Expected behavior: tests should pass
Current behavior: tests fail
The issue is in this part:

afterEach(done => {
    function callback() {
      console.log(testStore.getState());
      expect(testStore.getState().errors.length).toBeGreaterThan(0);
      done();
    }
    console.log(testStore.getState());

    setTimeout(callback, 0);
    // makes 2 AJAX requests
    // errors are dispatched, but never reach reducer

    //Timeout - Async callback was not invoked
    // within the 20000ms timeout specified by jest.setTimeout.

    // it's an issue w/ setTimeout (commented out the clicks [see via link at top] )

    // jest.runAllTimers();
    // jest.advanceTimersByTime(1000);
    // now the tests run but the AJAX request doesn't have time to reach the server
});

Jest config:

"jest": {
    "testURL": "http://localhost:3000"
  },

OS: Mac

"engines": {
    "node": "6.10.1",
    "npm": "3.10.10"
  },

"jest": "^22.1.4",

I am trying to build intuitive end-to-end tests (which I could ideally also use for React Native), e.g. click this button and these errors should appear.

Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions. Thank you :)


fixing bugs in load_rocstories_dataset in run_openai_gpt.py

The csv reader requires a delimiter argument to read .tsv file in the given example dataset. I've also added link for the dataset and provided a sample eval results in comments. Also, the eval dataset needs to be different from the training dataset, which I've also fixed in the given command to run this script.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #724   +/-   ##
=======================================
  Coverage   62.22%   62.22%
=======================================
  Files          18       18
  Lines        3979     3979
=======================================
  Hits         2476     2476
  Misses       1503     1503

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 98dc30b...0c18366. Read the comment docs.

Hi, what is the run_openai_gpt2_custom.py file for?

Hi Thomas, I customized the run_openai_gpt.py file to add support for gpt-2. That's why the name might be a bit confusing (run_openai_gpt2_custom.py). I'll better create a separate PR for this file, with a better name.Any suggestion?


wingetcreate can't create a reference when the forked repo has >= 254 branches

Brief description of your issue
wingetcreate (latest) can no longer create PR's successfully. The token which is used is still valid.
Looking at the error, my assumption is that this call fails.
Telemetry Settings
------------------
The Windows Package Manager Manifest Creator collects usage data in order to improve your experience.
The data is anonymous and collected only by Microsoft.
By default, telemetry is enabled but can be disabled by running `wingetcreate settings` and editing your settings file.

Manifest validation succeeded: True

Submitting pull request for manifest...

Octokit.NotFoundException: Not Found
   at Octokit.Connection.HandleErrors(IResponse response)
   at Octokit.Connection.RunRequest(IRequest request, CancellationToken cancellationToken)
   at Octokit.Connection.Run[T](IRequest request, CancellationToken cancellationToken)
   at Octokit.ApiConnection.Post[T](Uri uri, Object data, String accepts, String contentType, CancellationToken cancellationToken)
   at Microsoft.WingetCreateCore.Common.GitHub.<>c__DisplayClass[19](https://github.com/JanDeDobbeleer/oh-my-posh/runs/5403344265?check_suite_focus=true#step:3:19)_0.<<SubmitPRAsync>b__1>d.MoveNext()
--- End of stack trace from previous location ---
   at Polly.AsyncPolicy.<>c__DisplayClass40_0.<<ImplementationAsync>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Polly.Retry.AsyncRetryEngine.ImplementationAsync[TResult](Func`3 action, Context context, CancellationToken cancellationToken, ExceptionPredicates shouldRetryExceptionPredicates, ResultPredicates`1 shouldRetryResultPredicates, Func`5 onRetryAsync, Int[32](https://github.com/JanDeDobbeleer/oh-my-posh/runs/5403344265?check_suite_focus=true#step:3:32) permittedRetryCount, IEnumerable`1 sleepDurationsEnumerable, Func`4 sleepDurationProvider, Boolean continueOnCapturedContext)
   at Polly.AsyncPolicy.ExecuteAsync(Func`3 action, Context context, CancellationToken cancellationToken, Boolean continueOnCapturedContext)
   at Microsoft.WingetCreateCore.Common.GitHub.SubmitPRAsync(String packageId, String version, Dictionary`2 contents, Boolean submitToFork)
   at Microsoft.WingetCreateCore.Common.GitHub.SubmitPRAsync(String packageId, String version, Dictionary`2 contents, Boolean submitToFork)
   at Microsoft.WingetCreateCLI.Commands.BaseCommand.GitHubSubmitManifests(Manifests manifests)
   at Microsoft.WingetCreateCLI.Commands.SubmitCommand.SubmitManifest()
   at Microsoft.WingetCreateCLI.Commands.SubmitCommand.Execute()
   at Microsoft.WingetCreateCLI.Program.Main(String[] args)
{"message":"Not Found","documentation_url":"https://docs.github.com/rest/reference/git#create-a-reference"}

Error: Process completed with exit code 1.

Steps to reproduce
Invoke-WebRequest 'https://aka.ms/wingetcreate/latest/self-contained' -OutFile wingetcreate.exe
# Create the PR
./wingetcreate.exe submit --token $Token $Version

Expected behavior
It creates a PR sucessfully
Actual behavior
It errors
Environment
GitHub Actions windows-latest

I discovered what's causing the issue and a way to fix it, but not yet where exactly it then originates from. wingetcreate is unable to CREATE a reference when the forked repo has 254 branches.
Possible causes:

We validate if a branch was already created but do not paginate correctly and bail out somehow
The GitHub API can't create a ref when this is the case.

I'll try to investigate a bit further, but my time is limited. But hey, I found the root cause :-)

@JanDeDobbeleer thanks for digging into this! I'll ask the team to investigate today and see if we can get more traction. I think I've got one other report that I just moved into this repository.

Thank you @JanDeDobbeleer for making my life easier :smile: I will work on adding an additional check for this and showing a friendly error message so that it is clearer as to what action the user should take to resolve this issue.

@ryfu-msft do you know if it's related to this codebase/Oktokit or GitHub's API?

@ryfu-msft do you know if it's related to this codebase/Oktokit or GitHub's API?

I believe this is related to GitHub's API. We simply call on the Octokit client to get the reference to the repository, which just invokes GitHub's REST API.

@ryfu-msft that's indeed how I interpreted the code path. It's strange though. In my case the issue was resolved as soon as I deleted all remote autogenerated branches (and I then assumed that was the root cause), but that's not what people in #234 experience. Same error, different conditions. I'm actually really intrigued to debug this one.

@JanDeDobbeleer, I believe the issue is caused when your forked branch is behind the upstream reference "microsoft:master".
Once I fetched the new commits and merged it into the fork's master branch, I was able to submit successfully. Could you try that and verify that it resolved the issue? If so, I'll need to work on adding functionality to have winget-create check that the fork's master branch is up to date before creating a reference.

@ryfu-msft I actually did also reset my branch which fixed the issue. Will close this one, I think you're right.