Reads ping, hangs on speed test, Kodi 20 RC2

Hi !
I used AE-PCx86-11.0.18 Kodi 20 beta ( from AlexElec.tv repo, a fork of CoreElec) and  the Speedtest addon worked fine.
However with the AE 11.0.19 version with Kodi 20 RC2, the addon shows ping but hangs at the speed test part.
See Kodi error log pic attached.

Same issue on LibreELEC Nexus final, and Kodi final Nexus for both Android and OSX.  Ping only, but does not calculate speed .. No error log, as one is not produced.

Same issue. LibreELEC 20.1-Nexus on RPI4. Ping only but no speedtest.
`2023-05-03 12:29:28.012 T:805      info : Loading skin file: /storage/.kodi/addons/script.speedtester/resources/skins/Default/720p/script-speedtester_main.xml, load type: LOAD_ON_GUI_INIT
2023-05-03 12:29:33.365 T:1284    error : Exception in thread
2023-05-03 12:29:33.366 T:1284    error : Thread-2 (consumer)
2023-05-03 12:29:33.366 T:1284    error : :
2023-05-03 12:29:33.366 T:1284    error :
2023-05-03 12:29:33.366 T:1284    error : Traceback (most recent call last):
2023-05-03 12:29:33.366 T:1284    error :
2023-05-03 12:29:33.366 T:1284    error :   File "/usr/lib/python3.11/threading.py", line 1038, in _bootstrap_inner
2023-05-03 12:29:33.367 T:1284    error :
2023-05-03 12:29:33.367 T:1284    error :   File "/usr/lib/python3.11/threading.py", line 975, in run
2023-05-03 12:29:33.374 T:1284    error :
2023-05-03 12:29:33.374 T:1284    error :   File "/storage/.kodi/addons/script.speedtester/resources/lib/addon.py", line 738, in consumer
2023-05-03 12:29:33.389 T:1284    error :
2023-05-03 12:29:33.389 T:1284    error :
2023-05-03 12:29:33.389 T:1284    error : while thread.isAlive():
2023-05-03 12:29:33.389 T:1284    error :
2023-05-03 12:29:33.389 T:1284    error :
2023-05-03 12:29:33.389 T:1284    error :
2023-05-03 12:29:33.389 T:1284     info : Skipped 9 duplicate messages..
2023-05-03 12:29:33.389 T:1284    error : ^
2023-05-03 12:29:33.390 T:1284     info : Skipped 13 duplicate messages..
2023-05-03 12:29:33.390 T:1284    error :
2023-05-03 12:29:33.390 T:1284    error :
2023-05-03 12:29:33.390 T:1284    error : AttributeError
2023-05-03 12:29:33.390 T:1284    error : :
2023-05-03 12:29:33.390 T:1284    error : 'FileGetter' object has no attribute 'isAlive'
2023-05-03 12:29:33.390 T:1284    error : . Did you mean: '
2023-05-03 12:29:33.390 T:1284    error : is_alive
2023-05-03 12:29:33.390 T:1284    error : '?
2023-05-03 12:29:33.390 T:1284    error :
2023-05-03 12:29:33.443 T:1279    error : EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
- NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
Error Type: <class 'AttributeError'>
Error Contents: 'Thread' object has no attribute 'isAlive'
Traceback (most recent call last):
File "/storage/.kodi/addons/script.speedtester/resources/lib/addon.py", line 658, in onClick
self.speedtest(share=True, simple=True)
File "/storage/.kodi/addons/script.speedtester/resources/lib/addon.py", line 860, in speedtest
dlspeed = self.download_speed(urls, simple)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/storage/.kodi/addons/script.speedtester/resources/lib/addon.py", line 752, in download_speed
while prod_thread.isAlive():
^^^^^^^^^^^^^^^^^^^
AttributeError: 'Thread' object has no attribute 'isAlive'
-->End of Python script error report<--
`

The above solition worked for me too. It should be merged.


Planning Adjustments on Coastal Buildings and Blocked Area's

After Docklands and some extra information i have got, i will note the following
I gone try to fix all Coastal Buildings Measurements and i will add the same building with the so called Blocked Area's
I have seen the Parser use only the first numbers it see and this is wrong.
This will mean that, if i succeed the old coastal buildings used in a Layout stay as is, but when selecting the same building in the new preset (i hope i can do this as soon as possible) update.
For the Blocked area copies of it, i need maybe more time
But please give me some time to fix this
For the ones who pointed me on this flaw i say : Sorry but i did not see it then, but after Docklands and the extra info i got, i was digging deeper into the Buildblocker Files and then i noticed the whole Buildblocker was build into 4 different measurements. so, that you only see the "ground" measurement of coastal buildings has to do that the parser took only the first it saw.
My deepest apologizes.
Regards
Sting McRay

Update :
Since today i fixed the wrong sizes of coastal Buildings, Here some examples

Upper is old presets, lower on is on the new presets (still busy to update all things)

This post need to be kept open, as i still want to try the Blocked Area part of this post


organize imports on save expands minimized imports

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

VS Code Version: 1.69.2
OS Version: windows

Steps to Reproduce:

add this to settings.json
"editor.codeActionsOnSave": {
"source.organizeImports": true
},
minimize your imports
save
imports get expanded automatically

it's a bit annoying to manually minimize imports every time I save. and I also like having my imports get organized on save, it helps remove unused imports.

Also partially caused by https://github.com/microsoft/TypeScript/issues/39823 but that's only for cases where the imports are already sorted


Upgrade dependencies

Codecov Report

Merging #89 (bac76c1) into main (123d27b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #89   +/-   ##
=======================================
  Coverage   79.17%   79.17%
=======================================
  Files          16       16
  Lines        1426     1426
=======================================
  Hits         1129     1129
  Misses        231      231
  Partials       66       66

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 123d27b...bac76c1. Read the comment docs.


config option changed from name to database

The ISequelizeConfig interface was changed recently.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #136   +/-   ##
=======================================
  Coverage   96.84%   96.84%
=======================================
  Files         100      100
  Lines         918      918
  Branches      124      124
=======================================
  Hits          889      889
  Misses          9        9
  Partials       20       20

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 ec5bc97...302b381. Read the comment docs.


Improve message timing consistency for bare metal feedclient

Updating an existing client won't have any effect but new installs or when the interactive part is rerun, it'll update.

Thank you


commbench crashes in the ctxt test on netlrts-linux-x86_64-clang

This failure was seen in autobuild: http://charm.cs.illinois.edu/autobuild/old.2020_04_23__01_02/netlrts-linux-x86_64-clang.txt
make[3]: Entering directory '/scratch/autobuild/netlrts-linux-x86_86-clang/charm/netlrts-linux-x86_64-clang/benchmarks/converse/commbench'
../../../bin/charmc -optimize -production  -optimize -production  memoryAccess.c
../../../bin/charmc -optimize -production  -optimize -production  commbench.c
../../../bin/charmc -optimize -production  -optimize -production  overhead.c
../../../bin/charmc -optimize -production  -optimize -production  timer.c
../../../bin/charmc -optimize -production  -optimize -production  proc.c
../../../bin/charmc -optimize -production  -optimize -production  smputil.c
../../../bin/charmc -optimize -production  -optimize -production  pingpong.c
../../../bin/charmc -optimize -production  -optimize -production  flood.c
../../../bin/charmc -optimize -production  -optimize -production  broadcast.c
../../../bin/charmc -optimize -production  -optimize -production  reduction.c
../../../bin/charmc -optimize -production  -optimize -production  ctxt.c
../../../bin/charmc -optimize -production  -optimize -production  pingpong-cachemiss.c
../../../bin/charmc -optimize -production  -optimize -production  -o pgm memoryAccess.o commbench.o overhead.o timer.o proc.o smputil.o pingpong.o flood.o broadcast.o reduction.o ctxt.o pingpong-cachemiss.o -language converse++
../../../bin/testrun  ./pgm +p4  ++local
Charmrun> scalable start enabled.
Charmrun> started all node programs in 0.027 seconds.
Charm++> Running in non-SMP mode: 4 processes (PEs)
Converse/Charm++ Commit ID: f76a4fb
Isomalloc> Synchronized global address space.
Charm++> scheduler running in netpoll mode.
proc module init
proc module init
proc module init
proc module init
Charm++> Running on 1 hosts (1 sockets x 4 cores x 2 PUs = 8-way SMP)
Charm++> cpu topology info is gathered in 0.032 seconds.
...
...
...
[broadcast] (CmiReduce) 2.706548e-03 seconds per 4 bytes
[broadcast] (CmiReduce) 3.912032e-03 seconds per 16 bytes
[broadcast] (CmiReduce) 3.791792e-03 seconds per 64 bytes
[broadcast] (CmiReduce) 3.507205e-03 seconds per 256 bytes
[broadcast] (CmiReduce) 5.090213e-03 seconds per 1024 bytes
[broadcast] (CmiReduce) 9.981157e-03 seconds per 4096 bytes
[broadcast] (CmiReduce) 3.016170e-03 seconds per 16384 bytes
[broadcast] (CmiReduce) 1.182148e-02 seconds per 65536 bytes
[broadcast] (CmiReduce) 2.210896e-02 seconds per 262144 bytes
[broadcast] (CmiReduce) 5.062893e-02 seconds per 1048576 bytes
[ctxt] initiated
[ctxt] Thread Context Switching Overhead = 3.624036e-06 seconds
------------- Processor 0 Exiting: Called CmiAbort ------------
Reason: Msg handler does not exist, possible race condition during init

[0] Stack Traceback:
  [0:0] pgm 0x43e3a2
  [0:1] pgm 0x423ddb
  [0:2] pgm 0x423b92 CsdScheduler
  [0:3] pgm 0x4404de ConverseInit
  [0:4] pgm 0x41d540 main
  [0:5] libc.so.6 0x7ffff6d6e830 __libc_start_main
  [0:6] pgm 0x41c749 _start
Fatal error on PE 0> Msg handler does not exist, possible race condition during init

Makefile:13: recipe for target 'test' failed
make[3]: Leaving directory '/scratch/autobuild/netlrts-linux-x86_86-clang/charm/netlrts-linux-x86_64-clang/benchmarks/converse/commbench'
Makefile:21: recipe for target 'test-commbench' failed
make -C cthtest test OPTS='-optimize -production  -optimize -production ' TESTOPTS='++local'

real	4m15.376s
user	0m0.019s
sys	0m0.000s
make[3]: *** [test] Error 1
make[2]: *** [test-commbench] Error 2

This only happens during exit after executing the ctxt test.
The fixes applied to deal with destructors during exit in the lbMerge-5 branch appear effective in resolving this issue for non-smp, but the bug is still present in netlrts-linux-x86_64-smp-clang.

I can reproduce the issue locally, but it does not happen when run with ASan.

This happened on UCX too:
make[3]: Entering directory `/scratch/users/nbhat4/autobuild/ucx/charm/ucx-linux-x86_64/benchmarks/converse/commbench'
../../../bin/testrun  ./pgm +p4

Running on 4 processors:  ./pgm
./charmrun: line 152: mvapich2-start-mpd: command not found
Charm++> Running in non-SMP mode: 4 processes (PEs)
Converse/Charm++ Commit ID: e3a2c6d
Charm++ built with internal error checking enabled.
Do not use for performance benchmarking (build without --enable-error-checking to do so).
Isomalloc> Synchronized global address space.
proc module initproc module init
proc module init

proc module init
Charm++> Running on 2 hosts (1 sockets x 3 cores x 1 PUs = 3-way SMP)
Charm++> cpu topology info is gathered in 0.002 seconds.
[broadcast] initiated
[broadcast] (Consecutive CmiSyncBroadcastAllAndFree) 6.425332e-07 seconds per 4 bytes
[broadcast] (Consecutive CmiSyncBroadcastAllAndFree) 6.735703e-07 seconds per 16 bytes
[broadcast] (Consecutive CmiSyncBroadcastAllAndFree) 8.018340e-07 seconds per 64 bytes
[broadcast] (Consecutive CmiSyncBroadcastAllAndFree) 1.094582e-06 seconds per 256 bytes
[broadcast] (Consecutive CmiSyncBroadcastAllAndFree) 1.309877e-06 seconds per 1024 bytes
[broadcast] (Consecutive CmiSyncBroadcastAllAndFree) 2.036258e-06 seconds per 4096 bytes
[broadcast] (Consecutive CmiSyncBroadcastAllAndFree) 6.046979e-06 seconds per 16384 bytes
[broadcast] (Consecutive CmiSyncBroadcastAllAndFree) 2.099982e-05 seconds per 65536 bytes
[broadcast] (Consecutive CmiSyncBroadcastAllAndFree) 9.356099e-05 seconds per 262144 bytes
[broadcast] (Consecutive CmiSyncBroadcastAllAndFree) 3.792557e-04 seconds per 1048576 bytes
[broadcast] (CmiSyncBroadcastAllAndFree) 1.580113e-06 seconds per 4 bytes
[broadcast] (CmiSyncBroadcastAllAndFree) 1.627361e-06 seconds per 16 bytes
[broadcast] (CmiSyncBroadcastAllAndFree) 2.162432e-06 seconds per 64 bytes
[broadcast] (CmiSyncBroadcastAllAndFree) 2.542303e-06 seconds per 256 bytes
[broadcast] (CmiSyncBroadcastAllAndFree) 3.320748e-06 seconds per 1024 bytes
[broadcast] (CmiSyncBroadcastAllAndFree) 5.224684e-06 seconds per 4096 bytes
[broadcast] (CmiSyncBroadcastAllAndFree) 1.322270e-05 seconds per 16384 bytes
[broadcast] (CmiSyncBroadcastAllAndFree) 3.226037e-05 seconds per 65536 bytes
[broadcast] (CmiSyncBroadcastAllAndFree) 1.404162e-04 seconds per 262144 bytes
[broadcast] (CmiSyncBroadcastAllAndFree) 5.378389e-04 seconds per 1048576 bytes
[reduction] initiated
[broadcast] (CmiReduce) 3.716635e-06 seconds per 4 bytes
[broadcast] (CmiReduce) 3.728117e-06 seconds per 16 bytes
[broadcast] (CmiReduce) 4.248643e-06 seconds per 64 bytes
[broadcast] (CmiReduce) 4.437857e-06 seconds per 256 bytes
[broadcast] (CmiReduce) 4.887824e-06 seconds per 1024 bytes
[broadcast] (CmiReduce) 6.169404e-06 seconds per 4096 bytes
[broadcast] (CmiReduce) 9.423370e-06 seconds per 16384 bytes
[broadcast] (CmiReduce) 2.696743e-05 seconds per 65536 bytes
[broadcast] (CmiReduce) 1.002547e-04 seconds per 262144 bytes
[broadcast] (CmiReduce) 3.611678e-04 seconds per 1048576 bytes
[ctxt] initiated
[ctxt] Thread Context Switching Overhead = 1.763191e-07 seconds
------------- Processor 0 Exiting: Called CmiAbort ------------
Reason: Converse zero handler executed-- was a message corrupted?

[0] Stack Traceback:
  [0:0] pgm 0x4441e5 CmiAbortHelper(char const*, char const*, char const*, int, int)
  [0:1] pgm 0x4442ed
  [0:2] pgm 0x425090
  [0:3] pgm 0x425ec5 CsdScheduleForever
  [0:4] pgm 0x42616d CsdScheduler
  [0:5] pgm 0x44962a ConverseInit
  [0:6] pgm 0x41eece main
  [0:7] libc.so.6 0x2ab4e7c57555 __libc_start_main
  [0:8] pgm 0x41db6f
[mpiexec@golub291] HYDU_sock_write (utils/sock/sock.c:294): write error (Bad file descriptor)
[mpiexec@golub291] HYD_pmcd_pmiserv_send_signal (pm/pmiserv/pmiserv_cb.c:177): unable to write data to proxy
[mpiexec@golub291] ui_cmd_cb (pm/pmiserv/pmiserv_pmci.c:79): unable to send signal downstream
[mpiexec@golub291] HYDT_dmxu_poll_wait_for_event (tools/demux/demux_poll.c:76): callback returned error status
[mpiexec@golub291] HYD_pmci_wait_for_completion (pm/pmiserv/pmiserv_pmci.c:198): error waiting for event
[mpiexec@golub291] main (ui/mpich/mpiexec.c:340): process manager error waiting for completion
make[3]: *** [test] Terminated


sync-rdoc: Allow users to specify the RDoc store path

Motivation
Because all my rdoc are generated in a custom directory, my default RDoc store was empty and rdoc-sync did not find any documentation for my RBIs:
X has existing doc but can't find it with ri; not clobbering

This PR allows the user to specify from which directory the store should be loaded:
rbi/tools/sync-rdoc.rb rbi/ --rdoc-store /path/to/my/store

Test plan
Are there tests for this tool?

Thank you!


#845 Return a status 404 for core-command calls with bad device ID

#845
Possibly a regression from commit: https://github.com/edgexfoundry/edgex-go/commit/40337b3b7d1dcbb47afc1e140e71d89914688f67#diff-2311174e4ebc3d4be67d0cf62dc5e1d2R72
It appears that other code (possibly just test code) depends on this case being handled by an ErrNotFound instead of an ErrServiceClient, so I've just updated the caller to handle this error appropriately.

Closing to rework.


FREEMARKER-55: Add default tld sources

Shouldn't these be the initial values inside TemplateFactory.Builder? Then as a user of the Builder API you don't need to set the properties to them explicitly. They are the default values after all.

@ddekany Thanks for the comment! Indeed it's better to put those defaults in Builder. I've updated PR with a new commit: 73a53e70a5c838af451605380fc446e55b75762c


Add StocksBulkUpdate mutation

I want to merge this change because it adds StocksBulkUpdate mutation.
Resolves #11963
New types:
input StockBulkUpdateInput {
  variantId: ID
  variantExternalReference: String
  warehouseId: ID
  warehouseExternalReference: String
  quantity: Int!
}

type StocksBulkUpdate {
  count: Int!
  results: [StockBulkResult!]!
}

type StockBulkResult {
  stock: Stock
  errors: [StockBulkUpdateError!]
}

type Mutation {
    stocksBulkUpdate(
      errorPolicy: ErrorPolicyEnum
      stocks: [StockBulkUpdateInput!]!
    ): StocksBulkUpdate
}

Impact

[ ] New migrations
[x] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Could you add a benchmark for this new mutation? Please use different inputs for them.

Added.


Unicode highlight: Non-breaking space not highlighted

Non-breaking space is not highlighted when it should've been.
The character 'å' is a common letter in the Norwegian alphabet and can also be used as a word. The phrase "for å gi et eksempel" (which means "to give an example") is an example of this. A non-breaking space after the word 'å' is not highlighted by vscode as can be seen in the screenshot where all spaces are non-breaking spaces (U+00A0):

I'm guessing this is a regression introduced by #140960. I see the reason for why it's behaving like this, but it would be nice if it was possible to make it work in both scenarios.
Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.73.1
OS Version: macOS 13.0.1

Steps to Reproduce:

Configure vscode with "editor.unicodeHighlight.allowedCharacters": { "å": true }
Test with this example text: "For å gi et eksempel"
Verify that a non-breaking space after 'å' is not highlighted

Thanks! The bugfix should be available soon in VS Code Insiders.
It would be awesome if you could verify the bugfix!

@hediet It's working fine now in vscode-insiders, sweet! Thanks again 🙂


Please see my problem

http://www.newmediacampaigns.com/blog/add-text-to-an-image-with-php?cache=off#comment-ea2b8178874541329300ef05286581a6

I am new to php, i installed the library with the composer as described in your post everything was successful, now when run the php code from your example,  get this error:
[Fri Jul 21 15:50:57.525530 2017] [:error] [pid 9355] [client <IP_ADDRESS>:33024] PHP Warning:  imagettftext(): Could not find/open font in /var/www/me.usa.cc/comp/vendor/intervention/image/src/Intervention/Image/Image.php on line 1300

there is a error fetal error ! in
$image = new \NMC\ImageWithText\Image('source.jpg');
Fatal error: Uncaught Error: Class 'NMC\ImageWithText\Image' not found


[Bounty] Candid builder for dart class

Description
To build a candid builder for agent dart
Context
Candid is used to describe IDL for canister functions.
Candid builder is used to transform candid files to usable functions/classes for agent clients in different languages.
For dart language, all interfaces are classes. To make a workable builder, like we do json decoding classes, candid builder should also transform all those interfaces into classes.
Candid files are trans-compiled into typescript and javascript using didc, link is here
There are repos maintained by community for other languages:

Kotlin
Haskell
AssemblyScript
elm

Since agent_dart is ported agains agent-js, all IDL interfaces and styles are very similar to javascript/typescript.
for example , in generarted javascript:
const Purpose = IDL.Variant({
    'authentication' : IDL.Null,
    'recovery' : IDL.Null,
});
const CredentialId = IDL.Vec(IDL.Nat8);

in agent_dart, we can almost simply copy paste
final Purpose = IDL.Variant({
    'authentication' : IDL.Null,
    'recovery' : IDL.Null,
});
final CredentialId = IDL.Vec(IDL.Nat8);

So we might be able to :

modify and use didc to generate those idl,
THEN write classes using dart builder

To see working classes in dart, please refer to wallet/ledger.dart
Acceptance Criteria

[ ] modify and combine didc to builder to generate idls
[ ] use dart builder to generate artifacts in dart classes
[ ] tests and working examples, published to other repo.

Reward
30 ICPs

i'll deal with it @neeboo

i'll deal with it @neeboo

cool, let's discuss it

According discussion, please update targets ETA:

modify and combine didc to builder to generate idls
ETA:

use dart builder to generate artifacts in dart classes
ETA:

tests and working examples, published to other repo.
ETA:

hi @neeboo ,I'm tring to make a didc demo with dart support（just a very simple demo to check my idea）,here's what I have done:

Add  dart support for subcommand bind of didc("didc bind -t dart" )
write the dart parsing code according to the javascript.rs and ledger.dart in agent_dart
output artifacts in dart code

Here's the input candid file(copy from the candid guide docs)
 type address = record {
  street : text;
  city : text;
  zip_code : nat;
  country : text;
};
service address_book : {
  set_address: (name : text, addr : address) -> ();
  get_address: (name : text) -> (opt address) query;
}

Here's the artifacts generated by didc demo:
class address{
   BigInt zip_code;
   String street;
   String country;
   String city;
  address({ this.zip_code,  this.street,  this.country,  this.city})
  factory adress.fromMap(Map map)({
    return address (
      zip_code:map["zip_code"],
      street:map["street"],
      country:map["country"],
      city:map["city"],
    )
  })
  Map<String,dynamic> toJson()=>{
    return address (
      "zip_code":zip_code,
      "street":street,
      "country":country,
      "city":city,
    )
  }..removeWhere((dynamic key,dynamic value)=> key==null||value==null);
}
Service addressIdl =  IDL.Service({
  'get_address' : IDL.Func([IDL.Text], [IDL.Opt(address)], ['query']),
  'set_address' : IDL.Func([IDL.Text, address], [], []),
});

Here's the demo didc and candid file:
demo.zip
I'm not quite sure if the artifact meets your requirement, perhaps we could have more discussion based on this demo(including the ETA).

Sorry been taking me a while to look back your demo
I think the major purpose is fitting the criteria, we just need more types def and to build with the real case.
To see if you can generate the Ledger.candid classes and types like we did in wallet/ledger package
So please continue and keep up the good work, if you need any discussion , just ping me.

hi @neeboo, i've finished most part of the work(still have a little problem,need discuss with you).
This zip file includes the didc full demo, ledger.did and generated dart code:
demo.zip
I diffed the output.dart and ledger.dart in agent_dart project,and I think most of the generated code is correct.
But as i mentioned above,there is still a problem not easy to fix:
Check the  definition of AccountIdentifier  in  output.dart and ledger.dart,you'll find it.
I tried  to fix it but failed ,because the  original candid module(depended by didc) doesn't provide enough  support to solve it .
May be we can discuss about the detail later when you are free.

AccountIdentifier

Hi @zbyzhlsp , thanks for the hard work, I have seen you made great progress here.  To complete the whole parser, I think we can try the following paths:

For the AccountDentitfier , in the Candid, we have Text, in agent_dart, we can use primitive types to migrate these types to actual dart classes/types, we shall list all the primitive types, such as Nat64, we use BigInt in agent_dart.

I will update the README in lib/candid folder to list the primitive types out.

Most of the codes are generated by your modified didc however it sometimes does not quite fit the dart-lang specification such as required and ; stuff.  Since we might need lates dart-lang support for generated code. We can use source_gen to migrate and reformat the code again, then we can manipulate the didc output to compatible with latest dart-lang specification. About this part, I had some experience with source_gen, we can work together

hi @neeboo , I fixed the "AccountDentitfier" like bugs by the approach we discussed yet, and it looks like the output is correct.
About the demo, I added a typedemo.did file besides the ledger.did, to cover the cases of the definition I can come up with.
I pushed the code to the repository I forked from dfinity/candid: https://github.com/zbyzhlsp/candid/tree/dart_bind_support
Here is the compiled demo(including the new added typedemo.did )
demo3.zip

Sorry for replying late. Great work from your last commit. There are some requirements from my side.
Practically, we need xxx.fromMap() in factory method and yyy.toJson() in toJson() method. But types in fileds are hard to recognized by rust because they are in dart.
So maybe we can find an alternative way to workaround

Having an offline chat with @neeboo and @zbyzhlsp, the conclusion is that @zbyzhlsp is OOO and unable to continue with the builder. Unassigning him and assigned myself for further investigations.

Now we have candid_dart. Closing as the bounty has been claimed.


[Bug]: Expiration unlock condition 'slot' field

Issue description
The field in the deserialized EUC is slot while the typescript models define slotIndex
{"type":3,"returnAddress":{"type":8,"accountId":"0x21c983484baec0435205ba1a4c31ef0e66565fd84c93b086ca4c91ec8b9a65e5"},"slot":17724}

VS
declare class ExpirationUnlockCondition extends UnlockCondition {
    readonly returnAddress: Address;
    readonly slotIndex: SlotIndex;
    constructor(returnAddress: Address, slotIndex: SlotIndex);
}

Version
2.0
Expected behaviour
N/A
Actual behaviour
N/A
Can the issue reliably be reproduced?
Yes
Steps to reproduce the issue
No response
Errors
No response
Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

https://github.com/iotaledger/iota-sdk/pull/2215 ?

Can/Should this be closed?


Do not require password for truststore

Any of the checked boxes below indicate that I took action:

[X] Reviewed the Contribution Guidelines.
[X] Signed the Gradle CLA.
[X] Ensured that basic checks pass: ./gradlew quickCheck

For all non-trivial changes that modify the behavior or public API:

[X] Before submitting a pull request, I started a discussion on the Gradle developer list,
the forum or can reference a JIRA issue.
https://discuss.gradle.org/t/truststorepassword-is-required-because-of-a-bug-in-org-apache-httpcomponents-httpclient/17621
[X] I considered writing a design document. A design document can be
brief but explains the use case or problem you are trying to solve,
touches on the planned implementation approach as well as the test cases
that verify the behavior. Optimally, design documents should be submitted
as a separate pull request. Samples
can be found in the Gradle GitHub repository. Please let us know if you need help with
creating the design document. We are happy to help!
[N/A] The pull request contains an appropriate level of unit and integration
test coverage to verify the behavior. Before submitting the pull request
I ran a build on your local machine via the command
./gradlew quickCheck <impacted-subproject>:check.
(Technically it'd be nice to have something create client keystores/truststores and server
keystores/truststores to do the comparison as part of a dynamic test, but that would be
significantly more difficult to implement right now)
[N/A] The pull request updates the Gradle documentation like user guide,
DSL reference and Javadocs where applicable.
(Shouldn't need to change anything with this, as it will now function as other Java
applications)

Because it throws an apache http SSLInitializationException it was
misleading as to what was really the cause of a password being
required for the use of a custom truststore.

Thanks for the pull request, Spencer. It's interesting that this still fails for us as it was supposed to be fixed with HttpClient 4.2.4. We are currently using 4.4.1. Could you please also provide an integration test that demonstrates that the issue doesn't occur anymore?

The issue is no longer also with HttpClient, as it is in gradle internal code that was copied years ago from HttpClient that is still evidencing the problem.
I figured I was going to get a response to add an integration test ;-)  I'll see when I can scrape up enough time to do that properly.
-Spencer
  From: Benjamin Muschko<EMAIL_ADDRESS>
To: gradle/gradle<EMAIL_ADDRESS>Cc: Spencer Allain<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Sent: Wednesday, September 14, 2016 7:54 AM
Subject: Re: [gradle/gradle] Do not require password for truststore (#697)
Thanks for the pull request, Spencer. It's interesting that this still fails for us as it was supposed to be fixed with HttpClient 4.2.4. We are currently using 4.4.1. Could you please also provide an integration test that demonstrates that the issue doesn't occur anymore?—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Added tests to verify desired behavior.  One test would have failed without change.

Internal review: https://code-review.gradle.org/cru/REVIEW-6256

Thanks @merscwog! For the fix and the extensive test. Your contribution will be integrated in Gradle 3.2.

LGTM 👍


csrf: Handle opaque origins correctly ("Origin: null")

Treat requests that have an opaque origin like requests that have no origin information.
Additional Description: For more information about "Origin: null", see https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Origin
Risk Level: Low
Testing: Unit tests
Release Notes: in changelogs/current.yaml

Sorry please merge main.
/wait

Sorry please merge main.

done

Sorry please merge main one more time to fix the CI error.
/wait

Sorry please merge main one more time to fix the CI error.

Done, let's see if CI passes


Using the method RxBleConnection#setupNotification to open the notification is invalid

I have a device that I try to use #setupNotification to open the notification,but the Throwable in method onError is null.I use Google API to open this Characteristic notification is successful.How can I get the BluetoothGatt，I would like to use google api while using this open source library。

Hello @RiverZhong
Could you paste your code, logs, anything that can be useful to debug the issue?
Currently you cannot obtain the BluetoothGatt from the RxBleConnection in other way than force casting to RxBleConnectionImpl and getting the BluetoothGatt by reflection.

@RiverZhong Any update?

Thank you for your help, but now is China's Spring Festival, I'm on vacation, and had no device.

Google example requires UUID corresponding descriptors, in my test equipment is empty，but using setNotification() is effective

My colleague @uKL is working currently on this issue. Judging on the Bluetooth Specification the Client Characteristic Config descriptor is optional but it shall be set to appropriate value to make the notifications / indications work. It is not clearly described what should be the state in both scenarios in the Specification.
Thing is that in my understanding - if a characteristic supports notifications / indications the CCC descriptor must be present. Unfortunately some device vendors think different.
We will somehow allow to setup a notification without the CCC descriptor but the user will have to specifically set it this way.

Version 1.2.0-SNAPSHOT has an alternative way of setting the notifications / indications that do not need the descriptor. Check them out.

Thanks very much


MasterDB Close by multi sync

Hello,
as long as the slave entry is enabled the master DB is closed, when I sync it with an copy in an other path via Trigger "Synchronize active db with file/url"
As soon as I disable the entry the master DB stays open.
Any idea
kind regards

Remember this plugin is executed every time you save the database. Whether is saved by a trigger or another plugin.
Anyway, I didn't really understand your problem, is the master DB being closed by the plugin? Could you elaborate a little more so I can reproduce? How is the trigger set?

I believe I am experiencing the same issue that tnug384 is having.
For testing I downloaded a portable version of keepass and launched it from scratch without any customization except your plugin.  I open my Master DB fine, but when I press save it closes the master DB and then reopens.
this happens so fast, and the only reason I would have noticed is my DB is set to warn me if my master password is over 90 days old, and that pops up each time after saving.
Let me know if this clarifies or if you need more information.

I also have the same problem => I have a master DB open that should synchronize some folders with other slave DBs.
If I change something in the master DB and save the changes the sync job starts and closes the master DB and reopens an empty KeePass session - no DB is already open then.
Because of that I am not able to access my standard KeePass DB from the minimized tray during the sync jobs of the plugin are configured / online.
If I set the sync jobs to "expired" everything works as expected again => after saving changes in the master DB KeePass minimizes to tray with the locked master DB and I am able to work with it the normal way (e.g. for using auto-type).
Does anybody know a solution for that?

I'm going to look into this further. I use the plugin on a daily basis and always get de masterDB to open right after the sync job.

I can also reproduce this problem, although it seems to be slightly different.
As with the others, I have a KeePass trigger that executes on saving the DB to synchronise it with another DB remotely.
If I save my main (master) DB, it gets closed and reopened at least for a moment, since KeeAutoExec triggers to open some other DBs unaffected by the Master/Slave sync.
I still have the tab with my master DB visible in KeePass, but the programm crashes when I click it:
System.ArgumentException: The value is outside the expected range.
   at KeePass.UI.DocumentManagerEx.set_ActiveDocument(PwDocument value)
   at KeePass.Forms.MainForm.UpdateUI(Boolean bRecreateTabBar, PwDocument dsSelect, Boolean bUpdateGroupList, PwGroup pgSelect, Boolean bUpdateEntryList, PwGroup pgEntrySource, Boolean bSetModified, Control cOptFocus)
   at KeePass.Forms.MainForm.MakeDocumentActive(PwDocument ds)
   at KeePass.Forms.MainForm.OnTabMainSelectedIndexChanged(Object sender, EventArgs e)
   at System.Windows.Forms.TabControl.OnSelectedIndexChanged(EventArgs e)
   at System.Windows.Forms.TabControl.WmSelChange()
   at System.Windows.Forms.TabControl.WndProc(Message& m)
   at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

It seems like the reopening of the DB goes wrong, maybe because the synchronisation has not completed.
I use the officially recommended way for synchronising with triggers without any plugins, as described here: https://keepass.info/help/kb/trigger_examples.html#dbsync

I believe I am having this issue.  In my case I have some triggers enabled that do an automatic save on the master database after a set idle time.  When these are enabled, if I do a manual save on the master it does a synchronisation to the slave database OK but it closes the master database and I'm left with Keepass running but no database open.
When I disable the trigger system and do the same save, it works fine (ie. synchronisation runs and the Master Database is left open).


chore: dev limit

change line limit in sandbox (loki capped at 5000)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (7abf751) 71.33% compared to head (7ce841b) 71.33%.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #1152   +/-   ##
=======================================
  Coverage   71.33%   71.33%
=======================================
  Files         296      296
  Lines       10045    10045
  Branches     2319     2320    +1
=======================================
  Hits         7166     7166
  Misses       2879     2879

Flag
Coverage Δ

unitests
71.33% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Quantize the model

Hi @TropComplique ,
Have you ever tried to quantize the model by Quantization-aware training? If so, how is the result?
I am trying to create_training_graph in model.py at line 72 like this.
  assert mode == tf.estimator.ModeKeys.TRAIN tf.contrib.quantize.create_training_graph(quant_delay=100000)
Since I am not familiar with tf.estimator. I don't quite know where to add
tf.contrib.quantize.create_eval_graph()
to export a quantized graph before a savedmodel.
Or do I need to rewrite the project without tf.estimator.
If you get any ideas, please let me know.
Thanks a lot.

Hi, Did you figure this out? I am facing the same issue.


Check boxes shape in RoIPool / Align

Fix https://github.com/pytorch/vision/issues/1957
The boxes should be Tensor[K, 5] or List[Tensor[L, 4]]

Thanks for the improvements! Tests are failing, can you have a look?

Most of the test failures are unrelated to this PR (it's a breakage in PyTorch JIT I believe), but there is still one relevant error
self = <test_ops.DeformConvTester testMethod=test_boxes_shape>


Improve synthetic_size json output:

Describe your changes
Improve json that is used for visualization.

add new fields 'parent_lsn' and 'wal_from_parent' to segments;
output 'timeline_inputs' lsns as u64 to be consistent with other places

This pseudocode describes how to interpret the json to understand how final synthetic_size was calculated:
arr is the array of `SegmentSizeResult` records

for i in  0..arr.len() {
    tenant_synthetic_size += match arr[i].method
    {
        SnapshotHere -> segments[i].size,
        Wal -> segments[i].wal_from_parent.   //can be null, that's ok
        SnapshotLater | Skipped -> 0,
    }
}

@duskpoet , FYI
Issue ticket number and link
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Not sure I totally follow what wal_from_parent  is. Is it currentSegment.lsn - parentSegment.lsn?

@duskpoet

Not sure I totally follow what wal_from_parent is. Is it currentSegment.lsn - parentSegment.lsn?

yes, it is

@duskpoet

Not sure I totally follow what wal_from_parent is. Is it currentSegment.lsn - parentSegment.lsn?

yes, it is

It's easily calculated then, does it make sense to include them?

Opinions split here, so I'm closing this PR.
We agreed with Ed that existing output format is good enough to proceed with visualization.


Document how to create a new manged zone and delegate it

What
Add end user documentation on using the managed zone to delegate DNS to the multi-cluster gateway controller

duplicate of #202


Possible data gap for 2024-03-14T10:00:00Z in EBOCF

Describe the issue
A time 2024-03-14T10:00:00Z data is missing from "Indicative period BMU cashflows by settlement date (EBOCF)" and "Indicative period BMU cashflows by settlement period (EBOCF)" report, settlementPeriod=21
Steps to Reproduce
Steps to reproduce the behavior:

Go to https://bmrs.elexon.co.uk/api-documentation/endpoint/balancing/settlement/indicative/cashflows/all/{bidOffer}/{settlementDate}
Fields: "settlementDate" = "2024-03-14", "bidOffer" = "bid" (or "bidOffer" = "offer", data is missing for both values)
Click Execute
In data returned look for 2024-03-14T10:00:00Z. Data is missing for 2024-03-14T10:00:00Z settlementPeriod 21

Expected behaviour
I expect there should be at least one record for either bid or offer for 2024-03-14T10:00:00Z settlementPeriod 21
HTTP requests/responses
https://data.elexon.co.uk/bmrs/api/v1/balancing/settlement/indicative/cashflows/all/bid/2024-03-14
https://data.elexon.co.uk/bmrs/api/v1/balancing/settlement/indicative/cashflows/all/offer/2024-03-14
https://data.elexon.co.uk/bmrs/api/v1/balancing/settlement/indicative/cashflows/all/bid/2024-03-14/21
https://data.elexon.co.uk/bmrs/api/v1/balancing/settlement/indicative/cashflows/all/offer/2024-03-14/21
Additional context
Looks like this is related to https://github.com/elexon-data/insights-issues/issues/35 and https://github.com/elexon-data/insights-issues/issues/37
Checks
Please answer the following questions for yourself before submitting an issue.

[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed
[x] I'm reporting the issue to the correct repository
[x] I'm reporting a problem and not a feature request
[x] I have added the correct labels
[x] I have added screenshots or other relevant evidence (if applicable)

At present this is a genuine gap in data which is being investigated.


Running HaaS in Dev-mode with postgresql as database backend

We need to update HACKING.rst with instructions to run HaaS in dev mode with postgresql database backend.
Current instructions are incomplete and needs elaboration.
After following the given instruction I had to troubleshoot it further.

I have this documentation. Will double check then make a pull request so people can edit it.

Did you end up with the error command 'gcc' failed with exit status 1 while trying to do
pip install psycopg2 in dev-mode

you can create the PR, I will take a look at it.

@Izhmash you run HaaS in devmode with postgresql? Can you see if there's something in the documentation that you can add that would be helpful for others?

I'm using HIL with postgres locally, yeah.  I'll try to remember how I set it up and add any comments.

The process is pretty much the same as for production -- install postgress and psycopg2, create a postgres db, and put the url in hil.cfg. you can put it in testsuite.cfg to run the tests against postgres.
The error @SahilTikale mentioned above looks like the c toolchain isn't installed, which will cause problems with any Python library that calls C.
I see the value in having extra docs around this, but I also don't want to clutter docs/ with stuff where there is nothing hil-specific going on.
Maybe just the high level I wrote above, plus a link to a wiki article?

The only thing extra I noticed when configuring for Postgresql testing was that I needed a database called 'hil_tests'.  I remember one of the tests specifically was trying to access a db with that name, and when I tried to point testsuite.cfg at my database named 'hil' it didn't work.  That part got me hung up a bit.

Huh, I haven't seen that -- my local postgres db is called hil. Which test was it?

I can't remember which test, but it might have something to do with how Travis installs the databases (https://github.com/CCI-MOC/hil/search?utf8=✓&q=hil_tests&type=).
But anyway I was trying to figure out if it was at all possible to run the tests with a different DB name, and it sounds like you can.

Yeah, it should work with any name -- hil_tests is just what we happen to use on travis.

The process is pretty much the same as for production -- install postgress and psycopg2, create a postgres db, and put the url in hil.cfg. you can put it in testsuite.cfg to run the tests against postgres

@SahilTikale Does this still warrant a documentation update?
I am gonna close this issue because the consensus here seems like we don't need an update in the docs.
if you still think this is pertinent, please re-open it.


glew-targets.cmake points to wrong OpenGL location on OSX

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[X] ran brew update and can still reproduce the problem?
[X] ran brew doctor, fixed all issues and can still reproduce the problem?
[X] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)
I am trying to build my application, which links to GLEW. I use CMake for the build generator.
What happened (include command output)
When linking, make fails complaining it cannot find OpenGL in a path which it shouldn't be looking. This happens because glew-targets.cmake points to the wrong location.

Command output

make[2]: *** No rule to make target `/Applications/Xcode-beta.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/System/Library/Frameworks/OpenGL.framework', needed by `main'.  Stop.
make[1]: *** [CMakeFiles/main.dir/all] Error 2
make: *** [all] Error 2

What you expected to happen
I expected make to be linking to the actual location of OpenGL.
Step-by-step reproduction instructions (by running brew commands)
Create CMakeLists.txt containing

cmake_minimum_required(VERSION 3.8.2)
project(test_glew)

set(SOURCE_FILES main.cpp)

find_package(OpenGL REQUIRED)
find_package(GLEW REQUIRED)

add_executable(main ${SOURCE_FILES})

Create cpp file containing just a main function.

int main(int argc, char* argv[]){return 0;}

Run cmake . followed by make, to reproduce error.
Output of brew config and brew doctor commands

HOMEBREW_VERSION: 2.2.1
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: 2fe99bd9823fadea1cf918b1ed8366de7f40d336
Last commit: 4 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 32520d21e717f105e5ab9d170306d15616098977
Core tap last commit: 4 hours ago
HOMEBREW_PREFIX: /usr/local
CPU: quad-core 64-bit haswell
Homebrew Ruby: 2.6.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3/bin/ruby
Clang: 10.0 build 1001
Git: 2.24.0 => /usr/local/bin/git
Curl: 7.54.0 => /usr/bin/curl
macOS: 10.14.6-x86_64
CLT: <IP_ADDRESS>.1.1562985497
Xcode: N/A
CLT headers: <IP_ADDRESS>.1.1562985497
XQuartz: 2.7.11 => /opt/X11

Your system is ready to brew.

For people that are looking for a temporary solution, you can just do
brew install --build-from-source glew

Can you please open a pull request bumping its revision? That should hopefully fix the problem for everyone.


can't load package 'regex'

To reproduce
git clone https://github.com/stereobooster/pony-lisp.git
cd pony-lisp
make docker-build-image
make docker-build

Result
stable env ponyc  lisp/test -o build/release
...
Error:
regex: couldn't locate this path
Error:
/src/main/lisp/parser.pony:1:1: can't load package 'regex'
use "regex"
^
Error:
/src/main/lisp/test/tokenizer.pony:2:1: can't load package '..'
use ".."
^
Makefile:37: recipe for target 'build/release/test' failed
make: *** [build/release/test] Error 1

Hi @stereobooster,
you need to do stable fetch to fetch the dependencies before you do stable env.

Hi,
It is done in Dockerfile:
FROM ponylang/ponyc

...

COPY bundle.json ./
RUN stable fetch

but I can move it to makefile 🤔

@stereobooster move it to the Makefile and let me know what happens.

Yes it helped. I have no idea what is the difference, though.
Thanks for help.

@stereobooster stable fetch needs to be executed from the root of your project. I would guess that it isn't being executed from there when the Docker image is building. Just a guess. Good luck!


Confirm Action -keyword

Originally submitted to Google Code by @yanne on 10 Nov 2008

Add new keyword Get Confirmation, based on this:
def get_confirmation(self):
"""
By default, the confirm function will return true, having the
same effect
as manually clicking OK."""
return self._selenium.get_confirmation()

Originally submitted to Google Code by @yanne on 3 Dec 2008

Get Confirmation is somewhat confusing as a keyword name.
Implemented keywords Confirm Action and Choose Cancel On Next Confirmation in r93

Originally submitted to Google Code by @pekkaklarck on 5 Dec 2008

Is there some technical reason for using a combination of 'Choose Cancel On Next
Confirmation' and 'Confirm Action' keywords for canceling an action? Having just
'Cancel Action' feels quite a bit more logical, but I don't know is it possible to
implement such a keyword.


SSR support: Don't import popper if window undefined

Please makes sure these boxes are checked before submitting your PR, thank you!

[X] Make sure you are merging your commits to master branch.
[X] Add some descriptions and refer relative issues for you PR.
[X] Rebase your commits to make your pull request meaningful.
[X] Make sure that your changes pass npm test, npm run lint and npm run build.

Changes in this pull request

Checking that window is defined before requiring and exporting popper-mixins

Hi @winton I've tried to add this fix to my nextjs project but am still receiving error popper window error
{ ReferenceError: window is not defined
    at /Users/phil/Sites/git/clients/brandcrush/front/node_modules/element-react-ssr/dist/npm/es5/libs/utils/popper.js:57:16
    at root (/Users/phil/Sites/git/clients/brandcrush/front/node_modules/element-react-ssr/dist/npm/es5/libs/utils/popper.js:48:26)
    at Object.<anonymous> (/Users/phil/Sites/git/clients/brandcrush/front/node_modules/element-react-ssr/dist/npm/es5/libs/utils/popper.js:53:3)

Have you attempted to use this with nextjs?

Can someone work on this fix ?


[ERROR][BertEncoderTransformer] int8 mode only works with sm >= 75.

could encoders be run on V100, P100, P40 in int8 mode？

These GPUs do not contain the INT8 tensor cores. They may be runnable, but may be slower.

These GPUs do not contain the INT8 tensor cores. They may be runnable, but may be slower.

I had a try, but gemm test failed. I think this maybe result from that int8 weights are stored on tensor cores that don't support int8 calculations.

FasterTransformer does not provide the feature to run INT8 on cuda core (Namely, on GPU before Turing).

FasterTransformer does not provide the feature to run INT8 on cuda core (Namely, on GPU before Turing).

It was proved to be true after I tried to comment codes "exit(-1)" behind the sm judgement.

what's the means of SM,  and have you resolved it ?

No. It is a limitation due to hardware.

Close this bug because it is inactivated. Feel free to re-open this issue if you still have any problem.


Documentation updates needed for Splunk Infrastructure Monitoring

On the Splunk Requirements Page, in the Splunk Infrastructure Monitoring section, you almost miss the instructions because it's so short. Perhaps better to state:
Obtain the following from your Splunk Observability Cloud environment:

Realm
Token

On the Splunk Connect for Kubernetes page, this is very confusing:
"To learn more about SCK visit. Including configuration with Splunk Observability Cloud."
On that same page, there should be a box for O11y that contains
microk8s helm3 upgrade --install sck
--set="clusterName=rakuten_test_cluster"
--set="splunkObservability.realm="
--set="splunkObservability.accessToken="
--set="splunkObservability.ingestUrl=https://ingest..signalfx.com"
--set="splunkObservability.apiUrl=https://api..signalfx.com"
--set="splunkObservability.metricsEnabled=true"
--set="splunkObservability.tracesEnabled=false"
--set="splunkObservability.logsEnabled=false"
splunk-otel-collector-chart/splunk-otel-collector
On that same page, in the variables description section, a variables description for O11y should be added that has the realm and token
Also, In the Poller section of the yaml, if using O11y, poller usernameSecrets needs to be uncommented and set to blank
poller.usernameSecrets: {}

Hello! Thank you so much for the updates to the docs. I don't see this update, however:
Also, In the Poller section of the yaml, if using O11y, poller usernameSecrets needs to be uncommented and set to blank
poller.usernameSecrets: {}
On this page.

Hi!
I didn't include poller.usernameSecrets: {} in the documentation, because we fixed it from inside of Helm chart in commit https://github.com/splunk/splunk-connect-for-snmp/commit/6207c6fe271365d9fc647342fa9d60fd9d763f54. It should work fine now even without uncommenting that line.
Btw, thank you for your tips about the documentation, we appreciate it very much 😃

Awesome! Thank you!


Ubuntu 22.04 support

Ubuntu 22.04 is out, but it seems like it has a PREEMPT_RT kernel already precompiled? See https://ubuntu.com/engage/an-introduction-to-real-time-linux-part-i
cc: @carlossvg @LanderU @razr

It seems like the RT kernel is only available if you subscribe to Canonical's "Ubuntu Advantage" (with an exemption for personal use). So it seems like we probably still have to build our own kernel for this.

Even if it is precompiled it is probably a generic ARM64 kernel and we need to rebuild it for RPI4. The only advantage I see is that it is integrated with an RT_PREEMPT patch. I couldn't find any git reference to it https://git.launchpad.net/~ubuntu-kernel/ubuntu/+source/linux/+git/jammy/

Yeah I think we can probably use the existing system to patch the kernel for 22.04 and build our own Debian package.

I have built the RPI4 kernel for Ubuntu 22.04:
docker build --build-arg UBUNTU_VERSION=jammy --build-arg KERNEL_VERSION=5.15.0 --build-arg LTTNG_VERSION=2.13 -t rtwg-image-22.04 .
docker run -ti rtwg-image-22.04
cd $HOME/linux_build/linux-raspi
make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- LOCALVERSION=-raspi -j `nproc` bindeb-pkg

then I have updated my RPI4 to ubuntu 22.04:
sudo apt-get update
sudo apt-get upgrade

# 20.04 -> 22.04
sudo sed -i 's/focal/jammy/g' /etc/apt/sources.list

sudo apt-get update
sudo apt-get upgrade
sudo apt-get dist-upgrade
sudo apt-get autoremove
sudo apt-get clean

copied and installed deb
ubuntu@ubuntu:~$ ls
linux-headers-5.15.30-rt38-raspi_5.15.30-rt38-raspi-1_arm64.deb  linux-libc-dev_5.15.30-rt38-raspi-1_arm64.deb
linux-image-5.15.30-rt38-raspi_5.15.30-rt38-raspi-1_arm64.deb
ubuntu@ubuntu:~$ sudo dpkg -i *.deb

ubuntu@ubuntu:~$ sudo reboot
ubuntu@ubuntu:~$ uname -r
5.15.30-rt38-raspi

@LanderU if you can review and merge my updated PR, I can add a job to build a kernel for 22.04

@shuhaowu @razr @LanderU https://github.com/ros-realtime/linux-real-time-kernel-builder/pull/35 has been merged. This issue should be unblocked now. This would allow us to add support for humble and rolling now.

@razr @LanderU the kernel build was failed. Is there is an update on this? I really appreciate the great work.

@nexus1203, sorry, I've re-run the jobs, if they fail I need to review why.

@nexus1203, https://github.com/ros-realtime/linux-real-time-kernel-builder/actions/runs/2681210749 would you check if the release from this job works for you? Thanks!


Update VideoGrant.kt

In VideoGrant.kt, there seems to be a mistake in description of CanSubscibe and CanPublushData class. Description of CanSubscribe is supposed to be allow participants to subscrive to the room and similar for CanPublishData class.

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


Trying to understand the delay between relocating replicas and graceful master takeover

Hi there! We're evaluating orchestrator and whether it would be a good fit for our team.
Right now there's some confusion on how to trigger a controlled failover event. For example if we need to restart the writer/master server. So this wouldn't be an unexpected failure event.
Right now we trigger that via:
orchestrator -c relocate-replicas -i $(orchestrator -c which-cluster-master) -d $(orchestrator -c get-best-master-replacement)
orchestrator -c graceful-master-takeover

My main question is that we can't run the second command graceful-master-takeover immediately. We have to wait for over a minute otherwise orchestrator fails with:

FATAL GracefulMasterTakeover: master <IP_ADDRESS>:3306 should only have one replica (making the takeover safe and simple), but has 3. Aborting

What is the reason for this delay? If it's connected to a poll, is there a specific config we can configure to reduce the time or better yet, a command that we could run to trigger an update so that we don't have to wait for the next cycle in the poll?

@zmoazeni perfect timing. This was just merged a couple days ago. Details inside.

Ahhh that makes a lot of sense now. Thanks!!

I hope to make a release next week.

fixed and released in https://github.com/github/orchestrator/releases/tag/v3.0.3


Update http4s-blaze-client, ... to 0.21.21

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-dsl

from 0.21.9 to 0.21.21.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Superseded by #488.


VSCode's rust-analyzer extension crashes because of "Multi-lines is disabled by default by displaydoc."

This crate's UdsCommand causes crash of VSCode's rust-analyzer extension.

@sirius-ccwu thx for reporting, but this sounds like a bug in the rust-analyzer?  Is there a relevant issue in that project? I would love to help to solve it.

I don't think is a bug of rust-analyzer. Rus-analyzer does not use displaydoc.

It's resulted by the comment of UdsCommand::RoutineControl. More detailed error,
thread '<unnamed>' panicked at 'Multi-line comments are disabled by default by displaydoc. Please consider using block doc comments (/** */) or adding the #[ignore_extra_doc_attributes] attribute to your type next to the derive.', /home/ccwu/.cargo/registry/src/github.com-1ecc6299db9ec823/displaydoc-0.2.4/src/attr.rs:78:13 stack backtrace:    0: std::panicking::begin_panic    1: displaydoc::attr::AttrsHelper::display

I've fixed it by using a /** */, but test_display_doc failed.

I've also opened a bug report on rust-analyzer at https://github.com/rust-lang/rust-analyzer/issues/15840.

@sirius-ccwu I cannot reproduce your issue.  My vscode is v1.84.0 and rust analyzer v0.3.1722
I simply opened the project (dir) with this repository, and tried running and step-by-step debugging of the spot_test in lib.rs.  I also tried typing code, and viewing most of the UDS module's files - no issues.
version info
vscode
Version: 1.84.0
Commit: d037ac076cee195194f93ce6fe2bdfe2969cc82d
Date: 2023-11-01T11:28:21.782Z
Electron: 25.9.2
ElectronBuildId: 24603566
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.15.0-88-generic

rust-analyzer
Name: rust-analyzer
Id: rust-lang.rust-analyzer
Description: Rust language support for Visual Studio Code
Version: 0.3.1722
Publisher: The Rust Programming Language
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=rust-lang.rust-analyzer

I'm also using vscode 1.84.0 and rust-analyzer extension 0.3.1722 on WSL. I run code in the automotive_diag, open the Output pane, select output of Rust Analyzer Language Server. A thread handling procedure macro crashes.

I released another version that has only one line per doc. Not ideal but oh well.


Implement TimeStampMicroTZVector for parquet isAdjustedToUTC timestam…

#926 #576

@fb64 might interest you ;)

@fb64 might interest you ;)

Yes, it's part of missing mapping for some Arrow Types.
It would be nice to extend it to the others missing TimeStamp*TZVector Types : TimeStampSecTZVector, TimeStampNanoVector, TimeStampMilliTZVector
Also you can add those types to this test


Use /environments/{environment_api_key}/edge-identities/{edge_identity_identity_uuid}/edge-featurestates/ for feature override

Instead of /api/v1/environments/HALuMcoNiqFCVwYtJdddyv/edge-identities/{edge_identity)uuid}/featurestates/
when overriding from

I've done this however I think we've deemed that we can't actually show this tab for edge because https://api-staging.flagsmith.com/api/v1/environments/R8Pkw76haSiBZz7yQiECwc/featurestates/?anyIdentity=1&feature=3614&page=1 isn't possible @gagantrivedi , though maybe reopen if you reckon this isn't the case

Yeah, that's true, but I think we are planning to deprecate anyIdentity anyway, right @matthewelwell?

However, I can make /environments/{environment_api_key}/edge-identities/{edge_identity_identity_uuid}/edge-featurestates/ support query param feature that I think front-end can use to filter?

Added here: https://github.com/Flagsmith/flagsmith/pull/1056

Yeah, the problem here @gagantrivedi is that the anyIdentity parameter basically shows all user overrides for all identities (not just a single identity) in a given environment. We just can't achieve that with the data structures in dynamo (without massive pain) so we'll just have to hide the tab for the edge identity viewset.

Ah yeah, that makes sense


Update using DifferentialUpdate.

Do you have any example code on how to implement the IChangeTracking interface?  I am having difficulties getting an update to work that will only update properties that have changes.
Thanks.

IChangeTracking is a simple interface that's deceptively difficult to implement, especially if you want to support relationships and rollback. Your POCO needs to keep track of all properties that have changed since the last time AcceptChanges was called and provide those changes (usually in the form of a IDictionary<string, object>) when GetChanges is called.
Here's a primitive implementation:
public class Account : IChangeTracking
{
    private readonly IDictionary<string, object> _changes = new Dictionary<string, object>();

    private string _accountName;
    private string _type;

    public string AccountName
    {
        get { return _accountName; }
        set { Set(nameof(AccountName), out _accountName, value); }
    }

    public string Type
    {
        get { return _type; }
        set { Set(nameof(Type), out _type, value); }
    }

    private void Set<T>(string name, out T field, T value)
    {
        field = value;
        _changes[name] = value;
    }

    public bool IsChanged => _changes.Count > 0;

    public object GetChanges()
    {
        return _changes;
    }

    public void AcceptChanges()
    {
        _changes.Clear();
    }
}

Note that properties will still be included in the change set even if the consumer sets them back to their original value. If you want to support rollback via IRevertibleChangeTracking then you'll need to keep track of your POCO's original state also.
I recommend you study the SDataResource code for a complete implementation.

Thank you for the response and example code.

Based on your example, I have been able to successfully implement IChangeTracking, except for nested objects.  I have a contact class and address class.  Both implement IChangeTracking.  Updating each class individually works and only updates modified properties.  When I Update a contact with an address in the same Put (using SDataPayloadOptions), the contact is updated correctly (only properties flagged as changed are updated), but the address is not (all fields are updated in the database).
Suggestions?
Thanks.

Your GetChanges method needs to also check relationship properties that happen to implement IChangeTracking.
A very simple implementation might look like this:
public object GetChanges()
{
    if (Address != null && Address.HasChanges)
    {
        _changes["Address"] = Address.GetChanges();
    }
    else
    {
        _changes.Remove("Address");
    }

    return _changes;
}

As you can probably see by now, change tracking certainly complicates your model/DTOs. If I had more time to work on this library then I'd probably implement a "session" type object that maintains the original snapshot and automatically generates change sets for any kind of object (just like NHibernate sessions).

Thank you.  I implemented the code above (and it makes sense), but the address is still not being updated correctly.  When debugging though, I noticed that the address record is not being updated, but a new address record is being created.  I determined this by looking at the address key that was pulled before the update and then looking at the address key after.  using a browser to go to the old key, domain/sdata/slx/dynamic/-/addresses('a8KD5A000C7W'), and the new key, /sdata/slx/dynamic/-/addresses('a8KD5A000C7X'), shows that they both still exist, but the contact is associated with the new key.
Maybe this is how updates work in Infor in order to keep a history, but I would have thought the record would be updated.  Assuming the record should be updated, then I am doing something wrong in my other code.
Do you know if the record should be updated or a new one created?
I'm using this to pull the existing record (in this case, I don't have an id, just first and last name):
Here is my code:
`var checkContact = client.Query()
.Where(c => c.FirstName == FirstName && c.LastName == LastName)
.OrderBy(c => c.LastName)
.ThenBy(c => c.FirstName)
.Fetch(c => c.Address)
.FirstOrDefault();
            if (checkContact != null)
            {
                this.Id = checkContact.Id;
                this.Address.Key = checkContact.Address.Key;
                client.DifferentialUpdate = true;
                var contactOptions = new SDataPayloadOptions { Include = "Address" };
                var contact = client.Put(this, null, contactOptions);
            }`

That would indicate that the nested entity didn't have a key specified, which makes sense based on the sample code I sent. You'll need to change the internal _changes dictionary into a SDataResource on all your entities (or a common base class) and make sure you set the Key property before returning it:
private readonly SDataResource _changes = new SDataResource();

[SDataProtocolProperty]
public string Key { get; set; }

public object GetChanges()
{
    _changes.Key = Key;

    //TODO: relationships

    return _changes;
}

That worked.  Thank you for your time and patience.


Use flatfile logging for docker on all distributions

In order to make log-forwarding as easy as possible, we should use json-file logging for Docker on all distributions. This implies that we have to setup logrotate to avoid filled disks.
Acceptance criteria:

Docker logs to json-files on all distributions we support
Docker uses the same folder to log to on all distrbutions
Logrotate is set up on all distributions to rotate docker logs
Logrotate uses a size-based approach to determine weather to rotate the logs

We can do it for v2.11.1

This is done.


How to encrypt ARO nodes with custom key like AKS

I am trying to find a way to encrypt openshift nodes running in Azure using customer managed key . Same feature is supported in AKS while creating clusters. Please provide any documents from Azure or Redhat .
Thanks

@saswatmohanty01 this is on the roadmap and coming. We are in the process of enabling Openshift to do this, and once this is done ARO will follow soon after.


Inconsistent Provider URL for GitHub OIDC Integration

Describe the bug
When I'm using the web user interface and create a new OIDC Integration via "Administration" -> "Platform Management" -> "Manage Integrations" -> "New Integration" -> "OpenID Connect" and select "GitHub" as the Provider Type, the default Provider URL is https://token.actions.githubusercontent.com

I tried the new platform_oidc_configuration resource from Terraform but here I'm getting an error for the exact same URL. Please the trailing slash in the error message.
╷
│ Error: Invalid Attribute Configuration
│
│   with platform_oidc_configuration.some_test_terraform,
│   on platform_oidc_configuration_devops_training.tf line 4, in resource "platform_oidc_configuration" "some_test_terraform":
│    4:   issuer_url    = "https://token.actions.githubusercontent.com"
│
│ issuer_url must be set to https://token.actions.githubusercontent.com/ when provider_type is set to 'GitHub'.
╵

Here is my terraform resource
resource "platform_oidc_configuration" "some_test_terraform" {
  name          = "some-test"
  description   = "Test team integration"
  issuer_url    = "https://token.actions.githubusercontent.com"
  provider_type = "GitHub"
  audience      = "jfrog-github"
}

Here is my terraform version
Terraform v1.8.2
on linux_amd64
+ provider registry.terraform.io/hashicorp/azuread v2.48.0
+ provider registry.terraform.io/jfrog/artifactory v10.7.4
+ provider registry.terraform.io/jfrog/platform v1.7.2
+ provider registry.terraform.io/jfrog/project v1.5.2

Requirements for and issue

[x] A description of the bug
[x] A fully functioning terraform snippet that can be copy&pasted (no outside files or ENV vars unless that's part of the issue). If this is not supplied, this issue will likely be closed without any effort expended.
[x] Your version of artifactory (you can curl it at $host/artifactory/api/system/version
[x] Your version of terraform
[x] Your version of terraform provider

Expected behavior
The error message should not appear and the URL should work without a trailing slash, i.e. https://token.actions.githubusercontent.com

In the spec they also use the URL without trailing slash
https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/about-security-hardening-with-openid-connect

@zemirco Thanks for the report. I've added this to our plan to fix.

Awesome 🥳 Thank you very much!
Can you release a new version? The registry still shows version 1.7.2 but in the Changelog https://github.com/jfrog/terraform-provider-platform/blob/main/CHANGELOG.md#173-may-6-2024 you've added 1.7.3.

@zemirco I'm bundling #73 in the same release so it shouldn't be long.


Feature Request: Allow manual append to config files

For example, It would be nice If you could block certain log messages directly through puppet/hiera.

Can you explain your requirements a little bit more? It's hard for me to understand, what you want.

I think he means something like this


More eclipse functionality

Missing tests

Coverage decreased (-2.008%) to 91.73% when pulling 217785f9b5713c75564e84e5d4ea3feb0df9ce9a on Juanlu001:more-eclipse-functionality into 23e356139d3a730d9f1fb73ab619618726d552c0 on satellogic:master.

Coverage decreased (-2.008%) to 91.73% when pulling 217785f9b5713c75564e84e5d4ea3feb0df9ce9a on Juanlu001:more-eclipse-functionality into 23e356139d3a730d9f1fb73ab619618726d552c0 on satellogic:master.


Making each level in the hierarchy "slice-like"

This will allow semantics that looks more idiomatic for some people, such as:
for chain in &mut model { // instead of model.chains_mut()
    for residue in &mut chain { // instead of chain.residues_mut()
        ....
    }
}

and
chain[256] = some_residue

[The answers to this stackoverflow question] seem to help to achieve this.

I like the proposal, but I also really like the possibility of iterating over all levels of organisation (so directly over Atoms from a PDB for example). So I think some extra thought on how to achieve that would be great. I will try and see if a struct can implement IntoIterator multiple times with different Item types.
And in the best case it would be possible to keep the DoubleEndedIterator, because I needed that one to increase the speed of the parser. As I needed to traverse all atoms to find the right one to extend with an ANISOU (when parsing an ANISOU) but it nearly always is the very last atom.

I like the proposal, but I also really like the possibility of iterating over all levels of organisation (so directly over Atoms from a PDB for example). So I think some extra thought on how to achieve that would be great. I will try and see if a struct can implement IntoIterator multiple times with different Item types.
And in the best case it would be possible to keep the DoubleEndedIterator, because I needed that one to increase the speed of the parser. As I needed to traverse all atoms to find the right one to extend with an ANISOU (when parsing an ANISOU) but it nearly always is the very last atom.

I do not think multiple IntoIterator definitions are possible. So the way it is done now is the idiomatic way of having multiple 'view's over the structs (as done in String for example). We could think of having the immediate level below the struct available as an iterator (eg Iterator for chains) but I think that does not provide a clear indication of what is happening.
The same is true for the indexing, it is provided now by a function which specifies the struct result which I think is clear for readers of the code.
But if you think differently please discuss.

I do not think multiple IntoIterator definitions are possible. So the way it is done now is the idiomatic way of having multiple 'view's over the structs (as done in String for example). We could think of having the immediate level below the struct available as an iterator (eg Iterator for chains) but I think that does not provide a clear indication of what is happening.
The same is true for the indexing, it is provided now by a function which specifies the struct result which I think is clear for readers of the code.
But if you think differently please discuss.


Autograd y = x and y = 1*x gives different gradient

Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for essential information to most of the technical issues and bug reports. For non-technical issues and feature requests, feel free to present the information in what you believe is the best form.
For Q & A and discussion, please start a discussion thread at https://discuss.mxnet.io
Description
Autograd y = x and y = 1*x gives different gradient
Environment info (Required)
----------Python Info----------
Version      : 3.5.2
Compiler     : GCC 4.4.7 20120313 (Red Hat 4.4.7-1)
Build        : ('default', 'Jul  2 2016 17:53:06')
Arch         : ('64bit', 'ELF')
------------Pip Info-----------
Version      : 9.0.1
Directory    : /home/haoxli/anaconda3/lib/python3.5/site-packages/pip
----------MXNet Info-----------
Version      : 0.12.0
Directory    : /home/haoxli/anaconda3/lib/python3.5/site-packages/mxnet
Commit Hash   : 4f2af2d2e5216ab3a1faadcc117709b6836029dc
----------System Info----------
Platform     : Linux-4.2.0-27-generic-x86_64-with-debian-jessie-sid
system       : Linux
node         : haoxli-PC
release      : 4.2.0-27-generic
version      : #32~14.04.1-Ubuntu SMP Fri Jan 22 15:32:26 UTC 2016
----------Hardware Info----------
machine      : x86_64
processor    : x86_64
Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                12
On-line CPU(s) list:   0-11
Thread(s) per core:    2
Core(s) per socket:    6
Socket(s):             1
NUMA node(s):          1
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 63
Stepping:              2
CPU MHz:               1698.730
BogoMIPS:              6996.11
Virtualization:        VT-x
L1d cache:             32K
L1i cache:             32K
L2 cache:              256K
L3 cache:              15360K
NUMA node0 CPU(s):     0-11
----------Network Test----------
Setting timeout: 10
Error open Gluon Tutorial(cn): https://zh.gluon.ai, <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:645)>, DNS finished in 0.23898673057556152 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.0454 sec, LOAD: 0.1997 sec.
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0007 sec, LOAD: 0.5545 sec.
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.0038 sec, LOAD: 0.0354 sec.
Timing for FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0229 sec, LOAD: 0.1169 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0006 sec, LOAD: 0.0936 sec.

I'm using Python
Build info (Required if built from source)
N/A mxnet-cu80 (0.12.0)
Error Message:
Minimum reproducible example
import mxnet.autograd as ag
import mxnet.ndarray as nd

x1 = nd.array([[1,2]])
x1.attach_grad()

x2 = nd.array([[1,2]])
x2.attach_grad()

with ag.record():
    y1 = x1
    y2 = 1*x2
y1.backward()
y2.backward()
print (x1.grad)
print (x2.grad)

Steps to reproduce

run the above code
get output as following:

[[ 0.  0.]]
<NDArray 1x2 @cpu(0)>

[[ 1.  1.]]
<NDArray 1x2 @cpu(0)>

What have you tried to solve it?
N/A

Seems like if y1 is just a copy of x1, so the computation is not recorded, thus returning 0 as gradient.
@piiswrong

https://github.com/apache/incubator-mxnet/pull/8701
Here is the fix.

Fixed now


UTF8 words are not shown correctly

It would be really nice to fix showing of utf8 words. I've attached a picture of this issue:

The problem is mapping bytes to characters, which doesn't work for multi-byte characters. The UTF8 inspector solves this.

Yup, UTF-8 is a PITA  when it comes to use visible "characters". look up "Graphemes" for more info.


HAWQ-716. Support code coverage build and report in hawq Makefile

You might want to avoid using the absolute path of lcov, etc. (users could install them anywhere).
Does the code check whether lcov packages have been installed at the configure stage?

@paul-guo-, please find my comments below:

it now uses lcov, gcov, genhtml directory instead of using the absolute path, assuming they are system search path
lcov, gcov, genhtml are checked in configure already

+1

LGTM


mWeb - Chat -By pressing the back arrow on PDF protected file, transition is not smooth

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to https://staging.new.expensify.com/
Log in
Upload one Pdf protected file twice
Open in view mode but don't enter password
Press forward and back arrows

Expected Result:
Transition between all attachments is smooth.
Actual Result:
By pressing the back arrow on PDF protected file, transition is not smooth. The file twitches and it is not clear whether the back arrow works or not.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/235213900-d743e55d-1a7a-478e-b7b8-788c6d758430.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @jliexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Tested on v<IP_ADDRESS> and did not see this issue. Works great.


Additional response ext tests

Just bringing the coverage for response-extensions to 100%.
Also removed a what I am assuming is a redundant parseInt, similar to the one removed in the previous pr.
Also, I noticed in the options interface, there is an r missing in prior, is that intentional?
I'm guessing fixing it now would be breaking?

Hi @edreesjalili, amazing, thanks for contributing!
I will just merge and release as is!
Regarding your second question, prioRequestsProcessing stands for prioritise requests processing?, and yes, is just a shortcut.
Best Regards,
Rolando


Route Combinators

I was thinking quite a long time on combinators for Route abstraction. The idea is to replace the partial function route in  Endpoint by special type Route, whose instance may be build to plenty of predefined combinators. Thus the particular Endpoint is actually something like this:
type Endpoint[Req, Rep] = (Route[R], Service[Req, Rep])

So, we still will be able to compose a bunch of endpoints by orElse operator.
The Route[R] abstraction is tactually a function String => Option[R], . It takes a path and fetches some R from it or None if the given path doesn't correspond the given Route. Thus we some how should be able to flatMap (i.e., using the ~> operator) the Route result to underlying service. Like this:
def service(i: Int): Service[A, B]
val r: Route[Int] = ??? // route that fetches Int value
val endpoint1: Endpoint[A, B] = r ~>  service(_)
val endpoint2: Endpoint[A, B] = r ~> { i => service(i) }

We might have plenty of predefined combinators like Route[Int], Route[String], ect and use them build an entire route. Considering that int() and string() are combinators the API might looks as follows:
val r: Route[Int, String] = Get / "v" / int("version") / "users" / string("name")

Now we can define an Endpoint:
val e: Endpoint[A, B] = r ~> { (version, name) => service(version, name)}

This approach solves a lot of issues:

We no longer need a type alias for route: #114
We may decouple HttpRequest from Route
We would be able to share the sub-route with other routes via composition
We finally can define a proper reverse routes
We can fetch the path of concrete route which was asked recently on Gitter.

I believe this feature, as well as RequestReader will be cornerstones of Finch 1.0.0.
Let me know what do you think @pasviegas, @benjumanji, @rpless, @rodrigopr.

I like the direction. One of the potential benefits is that we'll be able to inspect routing and users could extend it with meta data (as desired in #73). The other issue with routing currently is we can't really define it independently of the service it maps to. I had a lot of issues with this when I was experimenting with #79. Right now all of our routing is built on a set of extractors that are defined in Finagle. Its fine for simple use cases, but becomes a problem when you want more complicated things (i.e. composition and path variables).
The only thing I would suggest at the moment is a shorthand for type combinators in addition to longer form suggested above. Something like:
val r: Route[Int, String] = Get / "v" / int / "users" / string

Looks like with such change we may close at least the following tickets: #114, #79, #122 and also make a good progress towards #73. There is also an old issue that hasn't been properly resolved: #54. We can support this having a route value.
And regarding the combinators name. Ryan @rpless is right: we don't need string labels for each combinator. So, it might be just int, double, etc. The one thing I forgot about here is naming convention we used for RequestReader, which is also sort of combinator. Thus, we may consider having names like IntRoute, DoubleRoute, etc in order to make things looks uniform and similar to other Finch code.
val r: Route[Int, String] = Get / "v" / IntRoute / "users" / StringRoute

What an exciting change! Can't wait to start working on it.
PS: Still waiting for the feedback from other contributors.

Nice, I like the idea a lot.
How the proposed api would handle endpoints with multiple routes? Or do you plan to use endpoint composition for that?

Thanks for the feedback @rodrigopr!
What I want to do at the high level is have a Route with is a function:
trait Route[A] {
  def apply(path: List[String]): Option[A]
}

So, it's just responsible for fetching the path params. There is also an Endpoint[Req, Rep] that has the same meaning as previously but now it's more strict. Instead of partial function route, there is should be a simple function:
trait Endpoint[Req, Rep] { self =>
  def route(req: Req)(implicit ev: Req => HttpRequest): Option[Service[Req, Rep]]
  def |(that: Endpoint[Req, Rep]): Endpoint[Req, Rep] = new Endpoint[Req, Rep] {
     def route(req: Req)(implicit ev: Req => HttpRequest): Option[Service[Req, Rep]] =
        self.route(req) orElse that.route(req)
    }
}

It takes a request and tries to route it to the underlying service. Thus, the orElse operator (i.e, |) is just a composition of two options. While the operator ~> in Route might be defined as:
def ~>[Req, Rep](fn: A => Service[Req, Rep]): Endpoint[Req, Rep] = new Endpoint[Req, Rep] {
    def route(req: Req)(implicit ev: Req => HttRequest): Option[Service[Req, Rep]] =
      apply(req) map fn
}

So, the API looks like:
val userService[Req, Rep] = ???
val userEndpoint[Req, Rep] = Get / "users" / StringRoute ~> useService(_) |
                                                Get / "users2" / StringRoute ~> userService(_) | NotFound

The nice thing about it is that we might be able to define or in the route itself (having an operator | in Route):
val userEndpoint[Req, Rep] = Get / ("users" | "user" ) / StringRoute ~> useService(_) |

Just one question I've got so far. How can we unflat tuple to avoid problems like this:
val r: Route[(A, B), C), D)] = Get / RouteA / RouteB / RouteC / RouteD

We can use something like shapeless HList or provide a bunch (up to 22) Route[A, B, ..., Z] which is probably the better idea if we want to avoid heavy dependencies.

A third option would be a mechanism like in Scala's parser combinators where based on a simple case class case class ~[A,B](a:A,b:B) you can then write Route[A ~ B ~ C ~ D] instead.
This avoids both, heavy dependencies as well as having Route0 to Route22 in the type system.

Really like the idea!!
Would be nice to have something in a branch to actually test it out.
I also would like StringRoute or IntRoute to be substituted by case classes
case class User(name: String)
val userEndpoint[Req, Rep] = Get / "user" / User ~> useService(_)

@jenshalm this is a really nice idea!
I just experimented in REPL a bit:
scala> case class Route[R](r: R)
defined class Route

scala> case class /[A, B](a: A, b: B)
defined class $div

scala> Route[Int / Int](/(10, 10)).r match { case a / b => a + b }
res8: Int = 20

We can use this approach for sure. The code of Parser is [here|https://github.com/scala/scala/blob/v2.10.2/src/library/scala/util/parsing/combinator/Parsers.scala].
Thus, the API would look like:
val route: Route[Int / String] = Get / "version" IntRoute / "users" / StringRoute ~> {
  case version / name => service(version, name)
}

BTW, @pasviegas thanks for the feedback!
I finally just got something in mind. I'm really happy with how it looks like. I'm planing to publish an experimental branch with package io.finch.route this weekend.


Android SEGV started happening on new Chrome with Emojis

It looks like there is another crash inside of the Android Webview in Chrome that is related to Emojis. I have created a pretty simple repro case here. This crash doesn't happen every time, but it happens pretty reliably if you close the app and reopen it about 5 or six times. I am testing on a Pixel 2 running Android 9, Security Patch Level April 5, 2019.
I think it is tied to emojis because the exact same page without emojis can be loaded successfully several times.
This is what the crash looks like:
2019-04-03 12:47:53.523 29820-29909/com.webviewbug I/chromium: [INFO:SkTypeface_remote.cpp(107)] GlyphCacheMiss generateFontMetrics: Rec
      textsize 162 prescale 1 preskew 0 post [1 0 0 1]
      frame 0 miter 0 format 1 join 0 cap 0 flags 0x114
      lum bits 0, device gamma 64, paint gamma 64 contrast 0
2019-04-03 12:47:53.523 29820-29909/com.webviewbug I/chromium: [INFO:SkTypeface_remote.cpp(47)] GlyphCacheMiss generateMetrics: Rec
      textsize 162 prescale 1 preskew 0 post [1 0 0 1]
      frame 0 miter 0 format 1 join 0 cap 0 flags 0x114
      lum bits 0, device gamma 64, paint gamma 64 contrast 0
2019-04-03 12:47:53.523 29820-29909/com.webviewbug A/libc: Fatal signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x878 in tid 29909 (Chrome_InProcGp), pid 29820 (com.webviewbug)
2019-04-03 12:47:53.580 30012-30012/? I/crash_dump64: obtaining output fd from tombstoned, type: kDebuggerdTombstone
2019-04-03 12:47:53.581 926-926/? I//system/bin/tombstoned: received crash request for pid 29909
2019-04-03 12:47:53.582 30012-30012/? I/crash_dump64: performing dump of process 29820 (target tid = 29909)
2019-04-03 12:47:53.608 30012-30012/?  A/DEBUG: Build fingerprint: 'google/walleye/walleye:9/PQ2A.190405.003/5310204:user/release-keys'
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG: Revision: 'MP1'
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG: ABI: 'arm64'
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG: pid: 29820, tid: 29909, name: Chrome_InProcGp  >>> com.webviewbug <<<
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG: signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x878
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG: Cause: null pointer dereference
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG:     x0<PHONE_NUMBER>000000  x1  00000000ffffffff  x2  0000007e295fc688  x3  0000007e295fc7b0
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG:     x4  0000007e295fd980  x5<PHONE_NUMBER>000004  x6<PHONE_NUMBER>000000  x7<PHONE_NUMBER>008000
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG:     x8  0000007e3c73c000  x9<PHONE_NUMBER>000001  x10 0000007e27a9c690  x11<PHONE_NUMBER>000002
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG:     x12<PHONE_NUMBER>000000  x13<PHONE_NUMBER>000000  x14 ffffffffffffffff  x15<PHONE_NUMBER>000018
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG:     x16 0000007e3c6c6870  x17 0000007ece9a1cf0  x18<PHONE_NUMBER>83c2c2  x19 0000007e295fc7b0
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG:     x20 0000007e295ff588  x21 0000007e3c392cb8  x22 0000007e3c719000  x23 0000007e27a57160
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG:     x24 0000007e295fe350  x25<PHONE_NUMBER>000002  x26 0000007e43f10800  x27 0000007e472ab898
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG:     x28 0000007e27db5b20  x29 0000007e295fc670
2019-04-03 12:47:53.608 30012-30012/? A/DEBUG:     sp  0000007e295fc640  lr  0000007e3a2d6c54  pc  0000007e3a2d6c8c
2019-04-03 12:47:53.646 30012-30012/? A/DEBUG: backtrace:

This doesn't appear to be #430 or #429 since I have already upgraded to 73.03683.90 and there are still crashes. I checked the User Agent, and I assume the Webview is also using the latest version.
As this appears to be a Chrome bug, I plan to file a bug with Google as well. Just wanted to put this here in case anyone else was seeing this. This is currently causing a lot of crashes for us.

Here is a link to the chromium bug: https://bugs.chromium.org/p/chromium/issues/detail?id=949295


npm link with cs binary script

Hello, there
I can't get get coffe binary script to resolve npm installed modules
I put up a small project to demonstrate that https://github.com/pgherveou/cs_link_test
basically I can do npm link from the project  and get this binary script to run globally

#!/usr/bin/env node
var moment  = require('moment');

console.log(moment().toString());

but the same won't work for a coffee binary
#!/usr/bin/env coffee

moment  = require 'moment'
console.log moment().toString()

I am not sure this is a bug, but one of you surely can tell me why !

 Run console.log(module.paths) to see the difference this makes.
Since our rule of thumb is that coffee should behave like node whenever possible, I'm filing this as a bug and investigating.

Easy fix: Just had to add an fs.realpathSync call to the line where we set module.paths. Install from master and let me know if you still have any problems, alright? Thanks for reporting.

Thanks Trevor, Ill give it a try
I can't really ask other people to work from the master, so I ll keep my binary script in js until next release
Thks for your help


crc: 2.13.1 -> 2.14.0

Description of changes
https://github.com/crc-org/crc/releases/tag/v2.14.0
On another note, the automatic update script doesn't update the sha256 for the git tag.  This causes problems in two ways, actually building the wrong version and breaking the crc setup command with an obscure expected crc_libvirt_4.11.13_amd64.crcbundle but found crc_libvirt_4.12.1_amd64.crcbundle in Makefile error, due to the mismatching versions.  I don't know what the solution to this is, but it is something that should likely be fixed as it causes issues if people are unaware of the problem and update the package.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@shikanime ping because you are the author of the update script.


add java options to empty.proto

Add java options to empty.proto for consistency.
In grpc-java, empty.proto has some java options. https://github.com/grpc/grpc-java/blob/master/interop-testing/src/main/proto/io/grpc/testing/integration/empty.proto
ref:
https://github.com/grpc/grpc-java/issues/4241
https://github.com/grpc/grpc-java/commit/156cc44c5f6ad64ff9b6706e4b03f24abadb4c99
cc: @ejona86

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

@nathanielmanistaatgoogle, I don't know the merge process for this repo.

@carl-mastrangelo  is this PR still useful? I think you've mentioned that you've updated the .protos in our repo recently. Let's resolve/close.

It is not.


borken when ModelSpeedup

borken when ModelSpeedup:
Ubuntu20:

NNI version:2.7
Training service (local|remote|pai|aml|etc):local
Client OS:
Server OS (for remote mode only):
Python version:3.7
PyTorch/TensorFlow version:PyTorch 1.11.0
Is conda/virtualenv/venv used?:conda
Is running in Docker?:No

Configuration:

Experiment config (remember to remove secrets!):
Search space:

Log message:
nnimanager.log:

Traceback (most recent call last):
File "lib/prune_gssl.py", line 137, in
ModelSpeedup(model, torch.rand(3, 3, cfg.input_size, cfg.input_size).to(device), "./prune_model/mask.pth").speedup_model()
File "/home/inspur/Projects/nni/nni/compression/pytorch/speedup/compressor.py", line 512, in speedup_model
self.infer_modules_masks()
File "/home/inspur/Projects/nni/nni/compression/pytorch/speedup/compressor.py", line 355, in infer_modules_masks
self.update_direct_sparsity(curnode)
File "/home/inspur/Projects/nni/nni/compression/pytorch/speedup/compressor.py", line 237, in update_direct_sparsity
node.outputs) == 1, 'The number of the output should be one after the Tuple unpacked manually'
AssertionError: The number of the output should be one after the Tuple unpacked manually

dispatcher.log:
nnictl stdout and stderr:

How to reproduce it?: below is my forward code

def forward(self, x):
x = self.conv1(x)
x = self.bn1(x)
x = F.relu(x)
x = self.maxpool(x)
x = self.layer1(x)
x = self.layer2(x)
x = self.layer3(x)
x = self.layer4(x)
cls1 = self.cls_layer(x)
offset_x = self.x_layer(x)
offset_y = self.y_layer(x)
nb_x = self.nb_x_layer(x)
nb_y = self.nb_y_layer(x)
x = self.my_maxpool(x)
cls2 = self.cls_layer(x)
x = self.my_maxpool(x)
cls3 = self.cls_layer(x)
return cls1, cls2, cls3, offset_x, offset_y, nb_x, nb_y

"cls1, cls2, cls3" are produced by the same layer(cls_layer), when return cls1, cls2, cls3, error comes up, when return one of them, error is gone

below is self.my_maxpool

self.my_maxpool = nn.MaxPool2d(kernel_size=2, stride=2, padding=0)

reproduced. It's an error in our graph-representation tool that it assumes each node has only one output node. we are going to resolve it in 2.9.1 or 3.0.

Hi, I have install nni version 2.10 and I am facing the same problem


ci(actions): Change actions/checkout version

Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/
Relates to https://github.com/ScoopInstaller/GithubActions/issues/21

[x] I have read the Contributing Guide.

Tests and BucketTemplate repos also need this.


bug: fix mismatched ids

Fixes AvatarBeam and AvatarPixel circular mask not applying
Fixes AvatarSunset not displaying

Before:

After

Thanks for this @f11xter!