Support loading certificates installed locally

I've been using the rustls crate in iamb, but it doesn't currently load locally installed certificates. I can use the rustls_native_certs crate to take care of that.
Additionally, I should probably add a feature to support compiling with OpenSSL instead, since I assume it probably handles locally installed certs in a way people are more used to.

Additionally, I should probably add a feature to support compiling with OpenSSL instead

Would you mind pinging me if you do this? It's not an issue  but, it requires changes to our package, as I will need to link base openssl at compile time.
TIA

@0323pin I've added support for building linking against the system libraries. I've put some instruction in a PACKAGING.md file. Let me know if you have any questions!

@ulyssa Thank you for the heads-up.
Since the default features didn't change, I don't think I'll do any changes to our package. I try to keep defaults as far as possible for my packages.
I'll start a build from git-HEAD later today and will let you know if there're issues

@ulyssa I think you forgot to update Cargo.lock
===> Building for iamb-0.0.9
error: no matching package found
searched package name: `hyper-tls`
perhaps you meant:      hyper-rustls
location searched: registry `crates-io`
required by package `reqwest v0.11.22`
    ... which satisfies dependency `reqwest = "^0.11.10"` (locked to 0.11.22) of package `matrix-sdk v0.7.1`
    ... which satisfies dependency `matrix-sdk = "^0.7.1"` (locked to 0.7.1) of package `iamb v0.0.9-alpha.1 (/usr/pkgsrc/wip/iamb-dev/work/iamb-9c6ff58b968f1b5bcfe995b8d65761160b98c3e9)`
As a reminder, you're using offline mode (--offline) which can sometimes cause surprising resolution failures, if this error is too confusing you may wish to retry without the offline flag.
*** Error code 101

Stop.

Thank you for catching that! I've fixed it in #219.

@ulyssa All good with both features. I've, both built and tested the resulting binaries :+1:


about train code

I found it seems there is mix between the train and test dataset
self.file_path = '/user/guoxia11/cvlshare/cvl-guoxia11/IMDL/REAL

self.file_path_fake = '/user/guoxia11/cvlshare/cvl-guoxia11/IMDL/FAKE

as the TrainData and ValData are using the same data root and the img_retreive function are the same
Also there is no model.train() called after the validation step the loss doesn't  reduce since the first epoch.
And a lot of bugs in this code, I strongly suspect that there is data fraud in this paper

I strongly agree with your opinion


Ability to edit object on info page

It would be great to have the ability to edit an object once you are on the object's info page. At present, you can only edit an object on the table view (e.g. <IP_ADDRESS>:8888/networks), not on info view (e.g. <IP_ADDRESS>:8888/network/11/info).
This would be useful as the longer the list of indicators grows, the harder it becomes to edit the required object. Also, when accessing an object through the Campaign group view, it navigates directly to the info page, therefore cannot edit an object

:+1:

This issue should be fixed in 3e9e09ee2be3c65a205f5a375f615bb7c467a671.
Let me know if there are any issues!


Design data flow for RecordForm

Old version data flow draft

Can i use state machine modeling https://www.youtube.com/watch?v=QFwJKeS2bn0&t=3479s for designing complex component?
How to describe different states, which are basically different data and can have infinite states? Is it described as context rather than as state?

https://github.com/skorphil/nextjs-form/blob/main/docs/data-flow.md


.NET template

Don't know if you're interested in having more templates, but this one has served me well for (not-very-serious) .NET stuff.

Merging as-is due to it adding value. I removed the lock per PR 72. Sorry for the late response/merge.


added two Schoo PWCs downstream of the PS for target scans

I'm going to ask you to do a few minor things before recommending a merge:

in Geometry/Beam_Scan_PWCs.txt can you document this a little bit more from a high level?  What's a Schoo PWC?  Why'd you put them here?  Just a brief description for the next poor grad student I put on this problem, with names/emails of the relevant AD people, or links to docdb to one or more of your talks.  Just some minor orientation for the next person
do we have MW943NU in the simulation already?

Re: MW943NU: I've been using one of the virtual detectors in the beamline ("IC943_flange_up_tracer") to represent that multiwire. I should probably update this.


support customize user & group key in jwt

Token is fixed,  maybe need direct the user key when use the create_jwt_token.
thanks.
#40

Run black --check --diff .
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.9.9/x64/bin/black", line 5, in <module>
    from black import patched_main
  File "/opt/hostedtoolcache/Python/3.9.9/x64/lib/python3.9/site-packages/black/__init__.py", line 52, in <module>
    from typed_ast import ast3, ast27
  File "/opt/hostedtoolcache/Python/3.9.9/x64/lib/python3.9/site-packages/typed_ast/ast3.py", line 40, in <module>
    from typed_ast import _ast3
ImportError: /opt/hostedtoolcache/Python/3.9.9/x64/lib/python3.9/site-packages/typed_ast/_ast3.cpython-39-x86_64-linux-gnu.so: undefined symbol: _PyUnicode_DecodeUnicodeEscape
Error: Process completed with exit code 1.

can retry ?

Very strange actions problem, I will try to fix it later.

Found it. https://github.com/python/typeshed/pull/6278
Can you execute poetry update and submit the new poetry lock to this PR?

OK, I push the new poetry.lock

Thank you for your efforts!


Use my own progress indicator

And some other tweaks.

@LarsJK I'm merging this right away as I have other changes I need to do, but I would still appreciate a review if/when you have time :)


Section::fetchAssociatedSections vs Section::fetchChildAssociations

https://github.com/symphonycms/symphony-2/blob/2.6.x/symphony/lib/toolkit/class.section.php#L166
vs.
https://github.com/symphonycms/symphony-2/blob/2.6.x/symphony/lib/toolkit/class.section.php#L187
I think fetchAssociatedSections should be marked as deprecated now...

Thanks mate!


Change the buttons to use interrupts to update the UI

At the moment we're polling the buttons waiting for someone to click them - it would be nicer if this was driven by interrupts.

Nice idea. As a reference I also adapted a component to read the L58 touch of this epaper, it also uses events (callback functions) you can find it here:
https://github.com/martinberlin/FT6X36-IDF
and also some implementation examples here:
https://github.com/martinberlin/cale-idf/tree/master/main/demos/parallel

I'll take a look - I didn't support touch initially as I was using the SD Card which is on the same pins as the touch controller and there's no way to coordinate the two.
But when using SPIFFS the touch controller is available. Waking from deep sleep might still need to be done from the buttons as the touch controller is switched off when the EPD is powered down.

One important to reminder is that I've read in GPIO & RTC documentation the following:

Please do not use the interrupt of GPIO36 and GPIO39 (Used by DOWN button) when using ADC or Wi-Fi with sleep mode enabled. Please refer to the comments of adc1_get_raw. Please refer to section 3.11 of ‘ECO_and_Workarounds_for_Bugs_in_ESP32’ for the description of this issue. As a workaround, call adc_power_acquire() in the app. This will result in higher power consumption (by ~1mA), but will remove the glitches on GPIO36 and GPIO39.

No idea if it will affect something but is good to keep it in mind.


Translate Error :

Hi
When I want to Translate Java codes to Objective-C with Command inc Mac Termnal >>
java -jar juniversal/juniversal-translator/build/libs/juniversal-translator-0.8.3.jar -o $HOME/javaoutput/objectivec/testbuild -l objectivec -classpath  jsimple/libraries/jsimple-util/build/libs/jsimple-util-0.9.3.jar  -sourcepath $HOME/androidstudioprojects/testbuild/app/src/main/java  $HOME/androidstudioprojects/testbuild/app/src/test/java

Shows me error >>
Error: 'objectivec' is not a valid target language

I think it's Opening Ceremony Level :+1:

The JUniversal translator doesn't support Objective-C output.  My plan to support iOS is via Swift and C++ output, though both are still work in progress, not complete enough yet to use.   At first, my thinking was support iOS via Objective C+, and the C++ support is mostly there, but since Swift 2 introduced exception handling (at least a form of it), that tilted the scales back to Swift.  I've been busy unfortunately & haven't gotten to spend much time on JUniversal recently, though really do want to get back to it.
As for Objectice-C output, Google's j2objc translator already does that, so probably best to stick with that.  You're welcome to use my Gradle wrapper to invoke j2objc (see directions at http://juniversal.org/getting-started) or the standalone Gradle wrapper for j2objc (see https://github.com/j2objc-contrib/j2objc-gradle) or invoke it directly.

I Have send Some Mail to you but no answer Recieved
If it Doesn't Work So Please Note Into Website About That


Volume of speech is wrong relative to the volume of music

There are two related issues here:

The volume of the speech is generally very low even if the volume of music is acceptable.
The music is not stopped or lowered when the agent is speaking.

In theory, the latter vould be accomplished with pactl load-module module-role-ducking ducking_roles=voice-assistant volume=20%, but this has no effect because the music is playing from an ALSA-to-PulseAudio stream that lacks the requisite role (should be "music" or "video").
This issue occurs on Home Assistant OS, using the add-on.

Turns out one can set the Spotify volume from inside the Spotify app, including for the spotifyd client, without affecting the PulseAudio volume, so this is less pressing of an issue.


Preloading Fonts

Maybe this isn't the right place for this, but I'm just trying to preload my font files, but can't seem to find a way to do it. Is there a way with this plugin to inject the path into the head so they can be preloaded?

Hello - I'm sorry to say that this plugin is no longer being maintained.  I have tried to close all issues ahead of doing this but instead have found that the issue list just keeps growing.  I simply do not have the time to keep up.  Sorry.
Please keep an eye on the README where I will record anyone who forks this repo with the intention to supporting it in the future.


Fix typo of delete-yatai.sh

Fix typo
kubectl delete namesapce yatai-components
-> kubectl delete namespace yatai-components

thank you @heartcored98!


grpcproxy: efficiently handle millions of unique watches

The current watch proxy will scale for many clients watching one key / one key range by coalescing similar watch streams. However, there are cases where it may be a single client per key with millions of clients. The current grpcproxy would help a little by using a single grpcstream, but it will still open many substreams on an etcd server. This wastes server resources and makes disconnects expensive since the proxy must (serially) reregister all the watches.
Two approaches to scaling:

Extend the coalescing code to merge similar nearby key intervals, either being exact about the intervals or by throwing away unneeded data. (probably too complicated to be useful)
Open a full keyspace watch and stream the data into an mvcc backend. Watches would be handled with the same code that handles watches on the etcd server.

The backend watcher proxy won't replace the current stream coalescing proxy; the two should be able to work together or independently.

@heyitsanthony
Does the current stream coalescing model work like the following?
            +-------------+
            |     mvcc    |
            +------+------+
                   ^ watch key A
                   |
            +-------------+
            | etcd server |
            +------+------+
                   ^ watch key A (s-watcher)
                   |
           +-------+-----+
           | gRPC proxy  |  -------+
           |             |         |
           ++-----+------+         |watch key A (c-watcher)
watch key A ^     ^ watch key A    |
(c-watcher) |     | (c-watcher)    |
    +-------+-+  ++--------+  +----+----+
    |  client |  |  client |  |  client |
    |         |  |         |  |         |
    +---------+  +---------+  +---------+

Open a full keyspace watch and stream the data into an mvcc backend. Watches would be handled with the same code that handles watches on the etcd server.

Would what your proposing work like this, assuming that the clients want to watch millions of keys which happen to be mostly unique? If that's the case, are we essentially removing load from the etcd server by transmitting all events to the grpc proxy without discriminating depending on whether any client actually wanted to receiving updates on that specific key. Then allowing the grpc proxy to handle the discrimination part?
            +-------------+
            |     mvcc    |
            +------+------+
            |______^______| Watch all keys?
                   |
            +-------------+
            | etcd server |
            +------+------+
                   ^ watch key A-C (s-watcher)
                   |
           +-------+-----+
           | gRPC proxy  |  -------+
           |             |         |
           ++-----+------+         |watch key C (c-watcher)
watch key A ^     ^ watch key B    |
(c-watcher) |     | (c-watcher)    |
    +-------+-+  ++--------+  +----+----+
    |  client |  |  client |  |  client |
    |         |  |         |  |         |
    +---------+  +---------+  +---------+

Yes. The proxy would stream events from the server into its own mvcc backend to service watches. The diagram should have an mvcc hanging off the gRPC proxy, though.

moving to 3.4


New worker pool

We need to implement a new worker pool in which a slow shard won't affect other shards when there are idle workers. This new worker pool will be used as both the event pool and the apply pool.
We can use dragonboat's snapshot worker pool for creating/applying snapshots, code available here.
https://github.com/lni/dragonboat/blob/ea6f310feeb6fa61677857eb00099f5d34851987/engine.go#L314
@zhangxu19830126 and I discussed a pipelined design. Jobs to be processed by apply workers will be initiated by some event workers when processing raft Ready. Snapshot related jobs will be initiated by apply workers. Apply workers will update/maintain peer replica snapshot state (whether doing snapshots), snapshot workers just reactively do whatever requested to do. Dragonboat's state transition code is here -
https://github.com/lni/dragonboat/blob/ea6f310feeb6fa61677857eb00099f5d34851987/engine.go#L1188
We also talked about the potential requirement for using a separate pool of workers to shutdown replicas, but this is of low priority in our current implementation. Just for reference, dragonboat's implementation is here -
https://github.com/lni/dragonboat/blob/ea6f310feeb6fa61677857eb00099f5d34851987/engine.go#L786

dup of #241


Fixes for new semi-sync handling.

@michael-berlin
Fixes https://github.com/youtube/vitess/pull/1521#issuecomment-189566221

LGTM
Thanks! :-)


Govuk elements

:see_no_evil: integrated @robmckinnon gem for GDS assets.
:speak_no_evil: cleaned up some UI
:hear_no_evil: better place to start to update remainder of app.

@himal-mandalia @jsugarman


tls: move out of extensions

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@phlax is there a way to exempt testdata from glint checks?

Ok I think this is finally passing, modulo windows which has been having CI issues.

/retest


ESP IDF(Arduino as component)not identifying BluetoothSerial.h  (IDFGH-10958)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
4.4.2
Operating System used.
Windows
How did you build your project?
Eclipse IDE
If you are using Windows, please specify command line type.
CMD
What is the expected behavior?
it should be able to identify Arduino library "BluetoothSerial.h" and should be able to complie the code
What is the actual behavior?
error: 'BluetoothSerial' does not name a type
BluetoothSerial SerialBT;
Steps to reproduce.
i had followed documentation by espressif and added arduino as component
in sdkconfig file i have changed CONFIG_FREERTOS_HZ to 1000
i had run the command idf.py menuconfig and selected "Start with loop and serup function"
After that i had wrote a simple classic bluetooth code and its not at all identifying it
Build or installation Logs.
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp:7:1: error: 'BluetoothSerial' does not name a type
 BluetoothSerial SerialBT;
 ^~~~~~~~~~~~~~~
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp: In function 'void connectToBluetooth()':
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp:19:9: error: 'SerialBT' was not declared in this scope
     if (SerialBT.available())
         ^~~~~~~~
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp:19:9: note: suggested alternative: 'Serial2'
     if (SerialBT.available())
         ^~~~~~~~
         Serial2
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp:47:7: error: 'connectToWiFi' was not declared in this scope
       connectToWiFi(ssid.c_str(), password.c_str());
       ^~~~~~~~~~~~~
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp: In function 'void setup()':
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp:89:1: error: 'SerialBT' was not declared in this scope
 SerialBT.begin("ESP32_BT_1");
 ^~~~~~~~
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp:89:1: note: suggested alternative: 'Serial2'
 SerialBT.begin("ESP32_BT_1");
 ^~~~~~~~
 Serial2
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp: In function 'void loop()':
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp:98:5: error: 'SerialBT' was not declared in this scope
     SerialBT.println("Enter credentials");
     ^~~~~~~~
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp:98:5: note: suggested alternative: 'Serial2'
     SerialBT.println("Enter credentials");
     ^~~~~~~~
     Serial2
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp:105:5: error: 'SerialBT' was not declared in this scope
     SerialBT.println("Connected successfully");
     ^~~~~~~~
C:/Espressif/frameworks/esp-idf-v4.4.5/workspace/GAZZELE_MINI/main/main.cpp:105:5: note: suggested alternative: 'Serial2'
     SerialBT.println("Connected successfully");
     ^~~~~~~~
     Serial2

More Information.
please help me through it as soon as possible

i would like to add one more point when i use "wifi.h" header file its working fine and its getting compiled so whats wrong with BluetoothSerail and BLE?


cloud_to_cloud:python:http2_server:rst_after_data failure

possibly related to https://github.com/grpc/grpc/issues/10624, cc @ericgribkoff again
Error Message

Failure
Standard Output

Traceback (most recent call last):
  File "src/python/grpcio_tests/tests/http2/negative_http2_client.py", line 177, in <module>
    main()
  File "src/python/grpcio_tests/tests/http2/negative_http2_client.py", line 173, in main
    _run_test_case(args.test_case, stub)
  File "src/python/grpcio_tests/tests/http2/negative_http2_client.py", line 134, in _run_test_case
    _rst_after_data(stub)
  File "src/python/grpcio_tests/tests/http2/negative_http2_client.py", line 100, in _rst_after_data
    next(resp_future), messages_pb2.COMPRESSABLE, _RESPONSE_SIZE)
  File "/var/local/git/grpc/py27/local/lib/python2.7/site-packages/grpc/_channel.py", line 366, in next
    return self._next()
  File "/var/local/git/grpc/py27/local/lib/python2.7/site-packages/grpc/_channel.py", line 357, in _next
    raise self
grpc._channel._Rendezvous: <_Rendezvous of RPC that terminated with (StatusCode.INTERNAL, Received RST_STREAM with error code 0)>

log https://grpc-testing.appspot.com/job/gRPC_interop_pull_requests/13226/#showFailuresLink
seen in pr https://github.com/grpc/grpc/pull/10616

Seems obsolete.


Support Discord's new replies

They're fancy and neat looking.

Worse yet, if they are used on the Discord side, they appear as regular messages on the Matrix side without any reference to what they are a reply or even so much as an indication that they are a reply. This is quite confusing.

d --> m implemented in https://github.com/matrix-discord/mx-puppet-discord/commit/0ef32fa593e24bfbbf572f302be8acf3a5c76550

I think this issue can be closed?

it only handles incoming repliesy not yet outgoing ones. they are still quotes

2021 9:47:14 Timo Kösters @.***>:

I think this issue can be closed?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub[https://github.com/matrix-discord/mx-puppet-discord/issues/112#issuecomment-808044587], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AASSEVDTD23FYZHXUHE2FZDTFRCZBANCNFSM4T6ZTJOQ].
[data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEgAAABICAYAAABV7bNHAAAAAXNSR0IArs4c6QAAAARzQklUCAgICHwIZIgAAAArSURBVHic7cEBDQAAAMKg909tDjegAAAAAAAAAAAAAAAAAAAAAAAAAAA+DFFIAAEctgHwAAAAAElFTkSuQmCC###24x24:true###][Sledovací obrázek][https://github.com/notifications/beacon/AASSEVGQ3AUIKASOQ3OSCB3TFRCZBA5CNFSM4T6ZTJO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGAU4QKY.gif]

I'd love to have a crack at closing this issue so here's a rundown of what I think needs to be done:
Unless I am mistaken I believe the better-discord.js library needs to be updated to the upstream commit https://github.com/discordjs/discord.js/commit/60e5a0e46f57cf297b66f1a940d24a20f46b5319 or later.
Also I think there would need to be a new database table storing matrix message ID to discord message ID, which I do not think the bot is currently doing, so you can tell discord which message you are replying to.
parseMatrixMessage would also need to be updated to not quote the message before sending to discord (or whichever function adds the quotation). I'm guessing it's line 374 that does this:
https://github.com/matrix-discord/mx-puppet-discord/blob/c17384a6a12a42a528e0b1259f8073e8db89b8f4/src/matrix/MatrixUtil.ts#L374
However, reading the code of FormatMessage, it doesn't seem to create a block quote when replying to a message?
https://github.com/matrix-discord/matrix-discord-parser/blob/c1f3046002367cbe61252e336f2767f18d2ac6b9/src/matrixmessageparser.ts#L376-L377
Finally you would send the message with chan.messages.resolveID(<id of corresponding discord message>).reply(<message ontent>) if the message on matrix a reply.
Is there anything I've missed or gotten wrong?

the problem with updating better-discord.js is that the newer discord.js requires newer than node12, but better-discord.js should still be compatible with node12. So, we might need to implement our own thing there, sadly.

Also I think there would need to be a new database table storing matrix message ID to discord message ID, which I do not think the bot is currently doing, so you can tell discord which message you are replying to.

it is already doing that, as soon as mx-puppet-discord enables the replies feature in mx-puppet-bridge it auto-resolves and gives mx-puppet-bridge the needed discord message IDs etc.

However, reading the code of FormatMessage, it doesn't seem to create a block quote when replying to a message?

the mx-reply tag is short-circuited to not render anything

2021 2:59:27 Alvie Rahman @.***>:

I'd love to have a crack at closing this issue so here's a rundown of what I think needs to be done:
Unless I am mistaken I believe the better-discord.js library needs to be updated to the upstream commit @.***[https://github.com/discordjs/discord.js/commit/60e5a0e46f57cf297b66f1a940d24a20f46b5319] or later.
Also I think there would need to be a new database table storing matrix message ID to discord message ID, which I do not think the bot is currently doing, so you can tell discord which message you are replying to.
parseMatrixMessage would also need to be updated to not quote the message before sending to discord (or whichever function adds the quotation). I'm guessing it's line 374 that does this:
https://github.com/matrix-discord/mx-puppet-discord/blob/c17384a6a12a42a528e0b1259f8073e8db89b8f4/src/matrix/MatrixUtil.ts#L374
However, reading the code of FormatMessage, it doesn't seem to create a block quote when replying to a message?
https://github.com/matrix-discord/matrix-discord-parser/blob/c1f3046002367cbe61252e336f2767f18d2ac6b9/src/matrixmessageparser.ts#L376-L377
Finally you would send the message with chan.messages.resolveID().reply() if the message on matrix a reply.
Is there anything I've missed or gotten wrong?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub[https://github.com/matrix-discord/mx-puppet-discord/issues/112#issuecomment-842743218], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AASSEVDYNPPV3QPPFXJQ2STTOG3WXANCNFSM4T6ZTJOQ].
[data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEgAAABICAYAAABV7bNHAAAAAXNSR0IArs4c6QAAAARzQklUCAgICHwIZIgAAAArSURBVHic7cEBDQAAAMKg909tDjegAAAAAAAAAAAAAAAAAAAAAAAAAAA+DFFIAAEctgHwAAAAAElFTkSuQmCC###24x24:true###][Sledovací obrázek][https://github.com/notifications/beacon/AASSEVGZL2VPNZBSEC3236TTOG3WXA5CNFSM4T6ZTJO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGI5T3MQ.gif]

also there is the discord issue of webhooks not being able to reply to webhook messages or thelike, soru heard

2021 2:59:27 Alvie Rahman @.***>:

I'd love to have a crack at closing this issue so here's a rundown of what I think needs to be done:
Unless I am mistaken I believe the better-discord.js library needs to be updated to the upstream commit @.***[https://github.com/discordjs/discord.js/commit/60e5a0e46f57cf297b66f1a940d24a20f46b5319] or later.
Also I think there would need to be a new database table storing matrix message ID to discord message ID, which I do not think the bot is currently doing, so you can tell discord which message you are replying to.
parseMatrixMessage would also need to be updated to not quote the message before sending to discord (or whichever function adds the quotation). I'm guessing it's line 374 that does this:
https://github.com/matrix-discord/mx-puppet-discord/blob/c17384a6a12a42a528e0b1259f8073e8db89b8f4/src/matrix/MatrixUtil.ts#L374
However, reading the code of FormatMessage, it doesn't seem to create a block quote when replying to a message?
https://github.com/matrix-discord/matrix-discord-parser/blob/c1f3046002367cbe61252e336f2767f18d2ac6b9/src/matrixmessageparser.ts#L376-L377
Finally you would send the message with chan.messages.resolveID().reply() if the message on matrix a reply.
Is there anything I've missed or gotten wrong?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub[https://github.com/matrix-discord/mx-puppet-discord/issues/112#issuecomment-842743218], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AASSEVDYNPPV3QPPFXJQ2STTOG3WXANCNFSM4T6ZTJOQ].
[data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEgAAABICAYAAABV7bNHAAAAAXNSR0IArs4c6QAAAARzQklUCAgICHwIZIgAAAArSURBVHic7cEBDQAAAMKg909tDjegAAAAAAAAAAAAAAAAAAAAAAAAAAA+DFFIAAEctgHwAAAAAElFTkSuQmCC###24x24:true###][Sledovací obrázek][https://github.com/notifications/beacon/AASSEVGZL2VPNZBSEC3236TTOG3WXA5CNFSM4T6ZTJO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGI5T3MQ.gif]

Is there any progress on this? I would love to send replies as discord replies rather then quote replies.


MRTK3 Themeing not working with Canvas Objects

Describe the bug
Hey guys, not sure if someone can help me but I currently play around with the experimantal Themeing Option in MRTK3.
Thanks to the Databind Example I got Non Canvas UI to work but not matter what I do I can't get Canvas Elements to change color. I used the 3 Themes from Zappys Playground but I can't use the scripts there as they all depend on so many other scripts and stuff that I don't want to redo all the scripts.
I have the Theme Selector and Theme Provider on an Empty Object that does not get destroyed on Scene change and I have Consumer Helper on my Canvas Object in the current scene. Anyone has an Idea/ and example where it works? I appreciate all the help I can get.
To reproduce
I have the Theme Selector and Theme Provider on an Empty Object that does not get destroyed on Scene change and I have Consumer Helper on my Canvas Object in the current scene.
Expected behavior
Canvas Object changes color
Screenshots

(the small non canvas Button changes color, just not my big Login Canvas)
Your setup (please complete the following information)

Unity Version 2021.3.15f1
MRTK Version [e.g. v2.0]

Target platform (please complete the following information)

HoloLens 2

Additional context
Could only find Non Canvas Examples in this Repo. The only color changing canvas example I found is Zappys Playground. Saldy since I did not use Zappys Playground as the base for my project I can't use the scripts 1:1 thanks to all the other stuff that happens there. Might ReDo my Project in the Future with Zappy as the base for translations and stuff but for now I can't

Hey there, I wanted to give a quick heads up to save you some time hunting.
We were in the process of publishing the revised theming system that is compatible with our canvas UI tooling when the MRTK team was eliminated. The existing theming system does not work with Canvas UI.
Sorry I don't have better news.

Hey @Zee2 thanks for the anwers. Still mad at Microsoft that they killed this project literally a month before GA. Still thanks for the detailed answer, guess I will use some time in the future to dig into it.
Good luck at your new? job tho!

Got the Themes of Zappys Playground working in my project. Here are all the Steps I did:
Copy this Folder in my Project: https://github.com/microsoft/ZappysPlayground/tree/8693f527a6a30fd8db0c4c7c7bbce98459331c2f/Assets/MSPlayground/Core/Data
Import com.unity.localization through the package manager
Attach the Theme Selector, Data Source Provider and Data Source Theme Provider to an empty Object (my Setting Object)
Attach the "Data Consumer Maskable Graphic Material" Script to your Themeable Object.
Set "Maskable Graphics To Modify" to the Backplate Canvas Element(s) in your Themeable Object
Set "Data Binding Profile" a new Profile with the "Data Key Path" to "MatsCanvas.InnerPlateMaterial"
With that my object got the proper Material. Right now I need to manually do these last steps for all UI Elements but hey as long as it works I'm happy. I'm still open to improvements/ other strategies to make it work.


Navigation Timeout Exceeded on unreachable addresses such as http://<IP_ADDRESS>

Steps to reproduce
Tell us about your environment:

Puppeteer version: 0.10.2
Platform / OS version: CentOS 7.3
URLs (if applicable): http://<IP_ADDRESS>

What steps will reproduce the problem?
const puppeteer = require('puppeteer');

(async() => {
  const browser = await puppeteer.launch();
  const page = await browser.newPage();
  page.on('console', (...args) => {
    for (let i = 0; i < args.length; ++i)
    console.log(`${i}: ${args[i]}`);
  });
  page.on('error', err => {
    console.log(err)
  })
  await page.goto('http://<IP_ADDRESS>/');
  console.log(page.url());
  browser.close();
})()

What is the expected result?
A soon-ish response with the console message:
http://<IP_ADDRESS>/
or an exception similar to the non-headless Chrome error of ERR_ADDRESS_UNREACHABLE
What happens instead?
Error: Navigation Timeout Exceeded: 30000ms exceeded
This issue is near identical to #782, but his issue was a heavy page, this issue relates to an unreachable location. I've included the final IO logs from Puppeteer when dumpio is set. That last event received is a Page.lifecycleEvent: networkIdle message... then some 29s later the exception is raised.
...
Fri, 06 Oct 2017 22:31:19 GMT puppeteer:page page.goto("http://<IP_ADDRESS>/")
Fri, 06 Oct 2017 22:31:19 GMT puppeteer:session SEND ► {"id":8,"method":"Page.navigate","params":{"url":"http://<IP_ADDRESS>/"}}
Fri, 06 Oct 2017 22:31:19 GMT puppeteer:protocol SEND ► {"id":10,"method":"Target.sendMessageToTarget","params":{"sessionId":"81c771bf-d189-47cf-b1eb-5045489d5e9c:1","message":"{\"id\":8,\"method\":\"Page.navigate\",\"params\":{\"url\":\"http://<IP_ADDRESS>/\"}}"}}
Fri, 06 Oct 2017 22:31:19 GMT puppeteer:protocol ◀ RECV {"id":10,"result":{}}
Fri, 06 Oct 2017 22:31:19 GMT puppeteer:protocol ◀ RECV {"method":"Target.receivedMessageFromTarget","params":{"sessionId":"81c771bf-d189-47cf-b1eb-5045489d5e9c:1","message":"{\"method\":\"Page.frameStartedLoading\",\"params\":{\"frameId\":\"58574.1\"}}","targetId":"81c771bf-d189-47cf-b1eb-5045489d5e9c"}}
Fri, 06 Oct 2017 22:31:19 GMT puppeteer:session ◀ RECV {"method":"Page.frameStartedLoading","params":{"frameId":"58574.1"}}
Fri, 06 Oct 2017 22:31:19 GMT puppeteer:protocol ◀ RECV {"method":"Target.receivedMessageFromTarget","params":{"sessionId":"81c771bf-d189-47cf-b1eb-5045489d5e9c:1","message":"{\"method\":\"Network.requestWillBeSent\",\"params\":{\"requestId\":\"58574.1\",\"loaderId\":\"58574.2\",\"documentURL\":\"http://<IP_ADDRESS>/\",\"request\":{\"url\":\"http://<IP_ADDRESS>/\",\"method\":\"GET\",\"headers\":{\"Upgrade-Insecure-Requests\":\"1\",\"User-Agent\":\"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36\"},\"mixedContentType\":\"none\",\"initialPriority\":\"VeryHigh\",\"referrerPolicy\":\"no-referrer-when-downgrade\"},\"timestamp\":1384375.928498,\"wallTime\":1507329079.577,\"initiator\":{\"type\":\"other\"},\"type\":\"Document\",\"frameId\":\"58574.1\"}}","targetId":"81c771bf-d189-47cf-b1eb-5045489d5e9c"}}
Fri, 06 Oct 2017 22:31:19 GMT puppeteer:session ◀ RECV {"method":"Network.requestWillBeSent","params":{"requestId":"58574.1","loaderId":"58574.2","documentURL":"http://<IP_ADDRESS>/","request":{"url":"http://<IP_ADDRESS>/","method":"GET","headers":{"Upgrade-Insecure-Requests":"1","User-Agent":"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36"},"mixedContentType":"none","initialPriority":"VeryHigh","referrerPolicy":"no-referrer-when-downgrade"},"timestamp":1384375.928498,"wallTime":1507329079.577,"initiator":{"type":"other"},"type":"Document","frameId":"58574.1"}}
Fri, 06 Oct 2017 22:31:19 GMT puppeteer:protocol ◀ RECV {"method":"Target.receivedMessageFromTarget","params":{"sessionId":"81c771bf-d189-47cf-b1eb-5045489d5e9c:1","message":"{\"id\":8,\"result\":{\"frameId\":\"58574.1\"}}","targetId":"81c771bf-d189-47cf-b1eb-5045489d5e9c"}}
Fri, 06 Oct 2017 22:31:19 GMT puppeteer:session ◀ RECV {"id":8,"result":{"frameId":"58574.1"}}
Fri, 06 Oct 2017 22:31:20 GMT puppeteer:protocol ◀ RECV {"method":"Target.receivedMessageFromTarget","params":{"sessionId":"81c771bf-d189-47cf-b1eb-5045489d5e9c:1","message":"{\"method\":\"Page.lifecycleEvent\",\"params\":{\"name\":\"networkAlmostIdle\",\"timestamp\":1384376.919345}}","targetId":"81c771bf-d189-47cf-b1eb-5045489d5e9c"}}
Fri, 06 Oct 2017 22:31:20 GMT puppeteer:session ◀ RECV {"method":"Page.lifecycleEvent","params":{"name":"networkAlmostIdle","timestamp":1384376.919345}}
Fri, 06 Oct 2017 22:31:20 GMT puppeteer:protocol ◀ RECV {"method":"Target.receivedMessageFromTarget","params":{"sessionId":"81c771bf-d189-47cf-b1eb-5045489d5e9c:1","message":"{\"method\":\"Page.lifecycleEvent\",\"params\":{\"name\":\"networkIdle\",\"timestamp\":1384376.919345}}","targetId":"81c771bf-d189-47cf-b1eb-5045489d5e9c"}}
Fri, 06 Oct 2017 22:31:20 GMT puppeteer:session ◀ RECV {"method":"Page.lifecycleEvent","params":{"name":"networkIdle","timestamp":1384376.919345}}

It's an edge case, but Chrome might wait forever if the timeouts are set to 0.

@ericdraken well, my non-headless chrome behaves the same way: tries to load the page for infinite amount of time.
I don't think we're doing anything wrong here.


Regenerated Android vector drawables

Recreated all existing android vector drawables and added missing ones that already existed in SVG formats within repository. Fixes #299. I used this tool, which eliminates problems introduced when converting with current tool. It would be a good practice to require this procedure for all newly created icons in the future.

Also fixes #189.

Some icons still seem to be broken. I will try exporting them manually.

OK. Let me know when they're ready to merge.

Broken icons:

[ ] sentry
[ ] gojek
[ ] vivino
[ ] elementaryos
[ ] rust
[ ] lastpass
[ ] calendar
[ ] sentry
[ ] redhat
[ ] open_benches
[ ] opencores

This looks great. There are 3 SVGs which look like they've been changed by mistake. If you can revert them, I'll merge the drawables.
Thank you so much for your efforts on this.

I will look at them, but to generate some android drawables I had to edit couple SVGs, they were so "optimized", that they didn't even work in Inkscape. I will not push them since they seem to work on GitHub, but maybe I will create an issue and a branch with edited files, since I don't feel like messing with optimizing SVGs, they are full size as exported from Inkscape ... your call.
I will add some new icons from master and give you a heads up to merge. Also it would be a good practice to require people at least include XML versions in future PRs. I can maybe write a wiki on this.

Ok, @edent you can merge this.

Brilliant stuff - thank you so much. Hopefully this can be useful in andOTP and others.

@vangyyy could you please work your magic again? We have lots of new icons :-)


customElement decorator should use type string for tagName

This is currently keyof HTMLElementTagNameMap but this doesn't work nicely in TypeScript right now and the result is you have to cast to any. Instead we should just use string for now.

Seconding this. I just ran into this issue while testing out lit element on an existing component.

@nrdobie That's because that long list is what keyof HTMLElementTagNameMap is equivalent to. That's not a wrong setting at all; lit-element currently requires you to manually extend the interface. I don't think that should be necessary, and switching to string would suffice (rather than casting to any).

Just ran into this, would be a nice fix. @nrdobie how would changing to string change the behavior?

Functionally it won't but it doesn't require the HTML tag list to be
updated for TypeScript. I know I'm splitting hairs and I don't really have
a strong opinion. I can union this with string and get the best of both.
On Fri, Dec 7, 2018, 3:22 PM Duncan<EMAIL_ADDRESS>wrote:

Just ran into this, would be a nice fix. @nrdobie
https://github.com/nrdobie how would changing to string change the
behavior?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Polymer/lit-element/issues/291#issuecomment-445369485,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAq41T08B9bQBnclEJtUBmCfmAl-RDzoks5u2twzgaJpZM4YPcps
.

Can someone explain the reasoning for having keyof there? I don 't fully understand why this wasn't a string in the first place.

Fixed via https://github.com/Polymer/lit-element/pull/292.


Add luxon and setup context

This PR closes #5

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Hide leftnav on overlay click

I'm able to open and close the leftnav using the state.open property but clicking on the overlay to dismiss the nav menu is not working. Should this work out of the box or do I have to do something? If so, how would I go about doing this?

Same too

+1

That does not work out of the box, have a look at the second example of http://www.material-ui.com/#/components/left-nav, you need to handle onRequestClose.

To spare future visitors the trip to the docs, it's as simple as adding e.g.
open={this.state.open}
onRequestChange={open => this.setState({open})}

... if you're using this.state.open to keep track of the open/closed state.

I believe you have to set the docked property to false for it to work. onRequestChange won't get fired


Cover case where input tag has no type specified

An input tag with no type attribute is considered to be of type="text",
but was not being picked up by Materialize.

Thank you @timtribers .


function retracing in tensorflow 2.3 with forward nufft

First, thank you so much for this implementation. I have been getting the following warning with the forward nufft in tf 2.3:
WARNING:tensorflow:11 out of the last 11 calls to <tensorflow.python.ops.custom_gradient.Bind object at 0x7fb3044ab450> triggered tf.function retracing. Tracing is expensive and the excessive number of tracings could be due to (1) creating @tf.function repeatedly in a loop, (2) passing tensors with different shapes, (3) passing Python objects instead of tensors. For (1), please define your @tf.function outside of the loop. For (2), @tf.function has experimental_relax_shapes=True option that relaxes argument shapes that can avoid unnecessary retracing. For (3), please refer to https://www.tensorflow.org/tutorials/customization/performance#python_or_tensor_args and https://www.tensorflow.org/api_docs/python/tf/function for  more details.

Have you been getting this warning when you run this? I am happy to try and debug it myself, but I need to figure out where in the code the function call is happening.

Hi @zacman2400 ,
Sorry for getting back to you so late, for some reason my watch setting was not on by default.
By any chance could you give me a link to a google colab where I can reproduce this error?
Also don't forget to post the entire stack trace, as well as use code formatting for your next issues to allow for further understanding.
However, I have been getting this error for some other reasons in other contexts.
See for example this and the linked issues.
It seems that up until recently, some people were dealing with this issue (see).
I don't have a test for the integration of the NUFFT in a Keras model unfortunately so it wouldn't show up in the CI logs.

For future reference, with TensorFlow version 2.3, a reproducible example is the following (based on the basic example notebook):
import numpy as np
import tensorflow as tf
import matplotlib.pyplot as plt

from skimage.data import shepp_logan_phantom
from tfkbnufft import kbnufft_forward, kbnufft_adjoint
from tfkbnufft.kbnufft import KbNufftModule
from tfkbnufft.mri.dcomp_calc import calculate_radial_dcomp_tf

dtype = tf.float32

image = shepp_logan_phantom().astype(np.complex)
im_size = image.shape
image = tf.convert_to_tensor(image)[None, None, ...]
nufft_ob = KbNufftModule(im_size=im_size, grid_size=grid_size, norm='ortho')

def create_traj(nspokes=40):
    spokelength = image.shape[-1] * 2
    grid_size = (spokelength, spokelength)

    ga = np.deg2rad(180 / ((1 + np.sqrt(5)) / 2))
    kx = np.zeros(shape=(spokelength, nspokes))
    ky = np.zeros(shape=(spokelength, nspokes))
    ky[:, 0] = np.linspace(-np.pi, np.pi, spokelength)
    for i in range(1, nspokes):
        kx[:, i] = np.cos(ga) * kx[:, i - 1] - np.sin(ga) * ky[:, i - 1]
        ky[:, i] = np.sin(ga) * kx[:, i - 1] + np.cos(ga) * ky[:, i - 1]

    ky = np.transpose(ky)
    kx = np.transpose(kx)

    ktraj = np.stack((ky.flatten(), kx.flatten()), axis=0)
    ktraj = tf.convert_to_tensor(ktraj)[None, ...]
    return ktraj

for nspokes in range(10, 200, 10):
    ktraj = create_traj(nspokes)
    kdata = kbnufft_forward(nufft_ob._extract_nufft_interpob())(image, ktraj)

The fix for this example is pretty simple actually:
forw = kbnufft_forward(nufft_ob._extract_nufft_interpob())
for nspokes in range(10, 200, 10):
    ktraj = create_traj(nspokes)
    kdata = forw(image, ktraj)

You need to define the forward operation for a given interpolation object once, otherwise it defines a tf.function in a loop which triggers the warning.
I am going to consider this issue closed, but @zacman2400 if this was not your context, please feel free to re-open.


UI/2: Detail screens

As a user, I want to be presented with more detailed information when clicking / tapping on search results (given by list) or locations / pins on the map. This information can be presented through a popover or full screen view depending on my device's screen size. On this view, I want to see all the information the database has to offer about the given entry in a well-organized and easy to comprehend fashion to be able to find what I need quickly and efficiently. Should geo-information be included, I want to have the option to go straight to the map and invoke navigation there.

Key aspects

UX & visual design
modularity and extensibility
(collaboration with Map)

Closing since this is team BR's focus for their 3rd sprint


[Bug] jsforce.Connection is undefined (resulting in "a.Connection is not a constructor")

I ran into this issue when deploying a NextJS project to a Hetzner VPS through Coolify (default nixpack build package).
I was also able to reproduce it just once locally when using next build and next start (non-deterministic).


jQuery $.ajax requests missing parameters when using poltergeist

Hi team,
I'm trying to test a piece of basic ajax functionality using RSpec feature test.
Here's the test:
require 'spec_helper'

feature 'Users can fill in their Objectives Tracker' do

  sign_in_as(:member)

  background do
    report = FactoryGirl.create(:report, club: current_user.club, current_step: "summary")
    @new_assignee = FactoryGirl.create(:user, club: report.club)

    # ReportPillars
    first = FactoryGirl.create(:report_pillar, pillar_name: "A First Pillar", report: report)
    second = FactoryGirl.create(:report_pillar, pillar_name: "B Second Pillar", report: report)

    # Objectives
    @report_objective = FactoryGirl.create(:report_objective, enabled: true, accepted: true, report_pillar: first, objective_statement: "First Objective")

    # Strategies
    FactoryGirl.create(:report_strategy, accepted: true, report_objective: @report_objective, strategy_statement: "First Objective's Accepted Strategy")

    open_main_menu_option("Objectives Tracker")
  end

  scenario 'User fills in details for an Objective', :ajax, :js do
    within(".objective-tracker tbody") do
      # Select a new assignee
<EMAIL_ADDRESS>    end

    # Member fills in due date using calendar (this will select current month)
    find(".ui-datepicker-trigger").click
    click_button("Done")

    wait_for_ajax!

    expect(@report_objective.reload.assignee).to eq(@new_assignee)
    expect(@report_objective.due_date).to eq(Date.today.beginning_of_month)
  end
end

So here's the important parts:

The functionality works in browser
The same test passes using selenium as a web driver

After a little investigation, I found that when using poltergeist the issue is that the request parameters don't come through. In the controller actions, the params hash doesn't include them.
I think it's something to do with the headers. My rough theory is that without a certain type of header Rails doesn't know what to do with the data and ignores it.
I've done some googling and can't find anything to do with poltergeist and ajax requests failing so I figured I'd raise the issue here.
I would have thought AJAX requests would be pretty common, so perhaps I'm missing something obvious.
If it helps, here's the coffeescript that makes the request:
# Initialize an empty namespace
TopClub.ObjectivesTracker = {} if !TopClub.ObjectivesTracker

class TopClub.ObjectivesTracker.ObjectiveRow

  constructor: (objectiveRow) ->
    @_objectiveRow = $(objectiveRow)
    @_idInput = @_objectiveRow.find("input[name*=id][type=hidden]")
    @_assigneeSelect = @_objectiveRow.find("select")
    @_dateInput = @_objectiveRow.find("input.month_year_date_picker")
    @_doneInput = @_objectiveRow.find("input[name*=complete][type=checkbox]")

    @_assigneeSelect.change(@_updateObjectiveOnAPI)
    @_dateInput.change(@_updateObjectiveOnAPI)
    @_doneInput.change(@_updateObjectiveOnAPI)

  _asQueryParams: =>
    {
      objective: {
        id: @_idInput.val(),
        assignee_id: @_assigneeSelect.val(),
        due_date: @_dateInput.val(),
        complete: @_doneInput[0].checked
      }
    }

  _updateDidFail: =>
    $cells = @_objectiveRow.find("td")
    $cells.removeClass("error")

    addErrorClass = ->
      $cells.addClass("error")
    # Need to have timeout or animation will not be replayable
    setTimeout(addErrorClass, 1)

  _updateDidSucceed: =>
    # no-op

  _updateObjectiveOnAPI: =>
    $.ajax(@_updateURL(), {
      data: @_asQueryParams(),
      type: "PATCH"
    }).fail(@_updateDidFail).done(@_updateDidSucceed)

  _updateURL: ->
    $("form")[0].action

As a workaround, I ended up using an :ajax tag on my test so I could use another web driver for ajax requests. EG:
# in spec_helper.rb

  config.around(:each, ajax: true) do |ex|
    original_driver = Capybara.javascript_driver
    Capybara.javascript_driver = :selenium
    ex.run
    Capybara.javascript_driver = original_driver
  end

This still happens.
That way:
  $.ajax(
    url: "/profiles"
    type: "DELETE"
    data:
      some_param: 1

does not send any params, while:
  $.ajax(
    url: "/profiles"
    type: "POST"
    data:
      _method: "DELETE"
      some_param: 1

works.

@pokonski what version of phantomjs are you using? It's a limitation of QtWebKit (and a bunch of older versions of browsers - not sure about current versions). This is because the HTTP spec was/is pretty ambiguous about whether or not delete requests should pass entity bodies or not, and whether or not servers should accept or ignore entity bodies from delete requests.  This may be supported in newer QtWebKit so maybe try phantomjs 2, but if you really insist on doing delete requests with params you're going to need to test it on all the browsers you want to support (especially mobile).  There's nothing poltergeist can do to "fix" this issue

@twalpole thanks for the answer, I checked the RFC to realize the behavior is undefined.
I'm on 1.9.0 version of PhantomJS.

Hi @jordanmaguire
I got same problem.

I found that when using poltergeist the issue is that the request parameters don't come through. In the controller actions, the params hash doesn't include them.

could you please tell me how to workaround the problem? my project is stable in production so I can't update code.

I included a workaround at the bottom of my original post - switch driver for that one test.

@jordanmaguire
I see your comment. I want to workaround by a way which don't need to switch driver. I tried to research but seem it can't be.
finally I have to use your way  switch from poltergeist to selenium driver.
thanks you so much.

You gotta be kidding 1.9.0


Connection refused on connecting MQTT-SN gateway

I am getting below error message while connecting client to MQTT-SN gateway which is connecting to MQTT Mosquitto Broker.
ubuntu@ip-172-31-85-25:~/projects$ ./mqtt-sn-tools/mqtt-sn-pub -p 1888 -h <IP_ADDRESS>  -t SH -m "1" -q 1 -d
2020-05-07 11:44:05 DEBUG Debug level is: 1.
2020-05-07 11:44:05 DEBUG Network timeout is: 5 seconds.
2020-05-07 11:44:05 DEBUG Trying <IP_ADDRESS>...
2020-05-07 11:44:05 DEBUG Connecting...
2020-05-07 11:44:05 DEBUG Sending CONNECT packet...
2020-05-07 11:44:05 DEBUG waiting for packet...
recv failed: Connection refused

Which MQTT-SN gateway are you using?
Are you sure it is listening on port 1888?

@njh  You are right in that port i missed to specify 1883. Now i am getting different error.
20200510 154812.941   CONNACK           <===  mqtt-sn-tools-23621                 20 02 00 00
     PacketHandleTask gets CONNACK      from the broker.
20200510 154812.942   CONNACK           --->  mqtt-sn-tools-23621                 03 05 00
errno == 22 in UDPPort::sendto
sendto <IP_ADDRESS>:0 length = -1
Error: ClientSendTask can't send a packet to the client mqtt-sn-tools-23621. Error=22
recved from <IP_ADDRESS>:49378 length = 25

I had to make sure that the multicast and unicast ports were different:
# UDP
GatewayPortNo=1883
MulticastIP=<IP_ADDRESS>
MulticastPortNo=10000

Other than that, I am not sure what else to suggest. I did my initial inoperability testing using rsmb.

@njh you rock. Looks like document of paho mqtt has reversed the port nos as below.
# UDP
GatewayPortNo=10000
MulticastIP=<IP_ADDRESS>
MulticastPortNo=1883


StatusBarItem - add support for specifying a list of commands in the hover

It would be nice if extensions would be able to specify a list of commands which would then be passed on to the HoverService so that they are rendered in the hover status bar.

This needs to set the related actions options when creating the hover:
https://github.com/microsoft/vscode/blob/fc03f2bf3d5041009ad246efbf6973922f65728f/src/vs/workbench/browser/parts/statusbar/statusbarItem.ts#L123


Wrong camera permission

Hi  the camera permission is out of date . the right one is blow
import AVFoundation

internal extension Permission {
    var statusCamera: PermissionStatus {
        let status = AVCaptureDevice.authorizationStatus(for: AVMediaType.video)

        switch status {
        case .authorized:          return .authorized
        case .restricted, .denied: return .denied
        case .notDetermined:       return .notDetermined
        }
    }

    func requestCamera(_ callback: @escaping Callback) {
        guard let _ = Bundle.main.object(forInfoDictionaryKey: .cameraUsageDescription) else {
            print("WARNING: \(String.cameraUsageDescription) not found in Info.plist")
            return
        }

        AVCaptureDevice.requestAccess(for: AVMediaType.video) { _ in
            callback(self.statusCamera)
        }
    }
}

Hi @Album1993, could you please update to the latest version of Permission ?
Also I'm not sure what you mean by "the right one is blow" ?


Issue#3 Improve loading time for images by Compressing

Summary

Used  Github Image Actions to optimize images

Added  .github/workflows/calibreapp-image-actions.yml in the repository

Image Actions will compress images so that they’re smaller, every time we push an image in the main branch,

We can control the quality of the images, we have kept it to default as of now

Image Actions adds optimized images to the current Pull Request and posts a summary comment.
Image Actions will run in the following scenarios:

on Pull Requests containing images (not including forks)

on pushing of images to main (for forks)

on schedule at 11 PM every Sunday(we can change the days and timing) if anything gets missed with any of the above scenarios

Screenshots
Demo Test: In a forked repository, we created a separate branch, added an image, and pushed it to the main branch of the forked repo. So while making the PR the images are optimized.

Author
@KshitijDarekar @psahithireddy

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

References and Resources

https://github.com/calibreapp/image-actions
https://github.com/marketplace/actions/image-actions
https://github.com/marketplace/actions/image-optimizer

Additional Info
This PR closes issue#3

Great work. I have a question though - will Github Action compress an already-compressed image? If it's going to compress all images every Sunday at 11 pm then the image quality might be really bad after a few compressions.

I supposed it should not compress an already compressed image as when we made a PR (in the forked repo)it mentioned that the 2 images did not require optimization. But I will verify it again.

Should we have cron at all? Maybe it's enough to have it run on pushes and PRs.

Perhaps we would be fine without a cron job as well. Although in the case of push and PR the changes should include an update in the imagery (i.e addition or deletion of an image file), for Image Actions to optimize.Open to suggestions.

Image-actions does not store state information about images, they're always compressed — if there's a significant improvement (more than -1%), the image will be committed, otherwise ignored.
That way, if subsequent commits update the same image, it'll be reevaluated and committed if need be.
Source :
https://github.com/calibreapp/image-actions/issues/23

I don't think we'll need cron. With cron, the image will be repeatedly compressed until its quality is so bad that compressing it again doesn't make much of a difference (i.e. <1% improvement). I might be wrong though but at least that's my understanding.
Compressing the images on push to main and (PR | merge) will be sufficient as there's no other way we can introduce an image to our repo. After deploying this, we might want to manually trigger the action once to compress existing images in our repo.

I don't think we'll need cron. With cron, the image will be repeatedly compressed until its quality is so bad that compressing it again doesn't make much of a difference (i.e. <1% improvement). I might be wrong though but at least that's my understanding.
Compressing the images on push to main and (PR | merge) will be sufficient as there's no other way we can introduce an image to our repo. After deploying this, we might want to manually trigger the action once to compress existing images in our repo.
@ditsuke @KshitijDarekar What do you think?
I'm going to bed now - just merge it if you agree with me, otherwise we can discuss when I wake up

Alright, @ditsuke  what do u suggest ?Should we remove the cron job or not ?

I think lets remove it.If any PRs have images it will automatically work on them, so I don't think we need a corn job, especially when we are not very sure of it

I don't think we'll need cron. With cron, the image will be repeatedly compressed until its quality is so bad that compressing it again doesn't make much of a difference (i.e. <1% improvement). I might be wrong though but at least that's my understanding.
Compressing the images on push to main and (PR | merge) will be sufficient as there's no other way we can introduce an image to our repo. After deploying this, we might want to manually trigger the action once to compress existing images in our repo.
@ditsuke @KshitijDarekar What do you think?
I'm going to bed now - just merge it if you agree with me, otherwise we can discuss when I wake up

Alright, @ditsuke  what do u suggest ?Should we remove the cron job or not ?

I think we should remove it!

Let's remove it and get it merged


Fix a bug where the unreferenced files would still be in the manifest

Since the extraneous variable was set only in the apply method and reused afterwards, the unreferenced files wouldn't be removed until webpack was restarted. This is a potential problem when used with dev server or watch option.

Thanks! 🎉 Will be released v1.0.0 on Monday.


Modify Device Report test to fit Schedule Event removal

Since Schedule and Schedule Event are removed from data model according to https://github.com/edgexfoundry/go-mod-core-contracts/pull/28, Device Report will associate to Interval Action instead of Schedule Event.
The related test script should be updated accordingly.

@cherrycl please look into this issue
thanks

Be advised that the DeviceReport is probably also going to be removed. If this isn't causing an actual error in the test run, I wouldn't spend any cycles on it.

@tsconn23 edgexfoundry/go-mod-core-contracts#28 will cause some error about DeviceReport, so we need to fix.  The test data of DeviceReport contains ScheduleEvent

I will fix this issue after edgex-go/go.mod is updated to latest.

@cherrycl PR was merged


Server coordinates still load when the server is not listed in the config

Server coordinates still load when the server is not listed in the config.

Fixed in commit 4cdc14490230fdf3c321eba5a33b2c52fb38a952


add laravel-ui tabler.io

@PierreLebedel thanks for the contribution! I will schedule to review it next week, now busy with Laravel 9 related work.

@PovilasKorop There is no rush, I suspect that the release of Laravel 9 involves a lot of work (and a lot of daily content for us ;) !

@PierreLebedel please make some fixes

Remove search input

Remove bell icon

On users page add info message like on other templates Sample table page

On about page remove buttons

When done ping Povilas :wink:

It's done! My bad, I thought putting some unnecessary components was a good idea. I removed the indicated elements and a few others to simplify the templates files. Thank you. Ping @PovilasKorop

@PierreLebedel thanks a lot for your work, I've just released the v1.4 with Tabler :)


Add ClosureContextProcessor

Hi! That processor generates a context by a closure if the closure is set as the only value in the context.
I use it a lot to reduce performance impact by debug code.
e.g.
function register () {
    // this:
    $log->debug('User registration in progress: {uuid}, {date}, {profiled}, {more}', [
        // expensive context generation !!!
    ]);
    // changes to:
    $log->debug('User registration in progress: {uuid}, {date}, {profiled}, {more}', [fn() => [
        // expensive context generation in closure will run only if debug level is set
    ]]);

    $log->info('User {name} registered.' ['name' => $user->name]);
}

I wonder if this is the best way to do this.. I don't really like that it makes the context dependent on the configuration of a processor, and you need to configure the processor on the right handler to ensure it does not call your closure too early.
How about this alternative:
class LazyContext {
    public function __construct(
        private \Closure $contextGenerator
    ) {}

    public function jsonSerialize() {
        return ($this->contextGenerator)();
    }
}

// then use it as:
$log->debug('foo', [new LazyContext(function () {
    return 'anything';
})]);

That would then be normalized by NormalizerFormatter as [0 => ['LazyContext' => 'anything']] and by JsonFormatter as ['anything'] I believe.
So your solution produces cleaner output, but carries a bit more configuration risk.
Anyway I do wonder if this really belongs in Monolog itself.. Is it so common to create expensive-to-init context data? I can't recall ever having that problem at least.

It's an option that I've shared. And yes, it has the same issues as the PsrLogMessageProcessor. I don't like it too. But loosing 60% of perfomance by logging I hate more :smiling_face_with_tear:

Is it so common to create expensive-to-init context data?

An array of five strings with calls of get_debug_type(), get_class() and few getters is enough to slow down an app. Closures work faster... for 2x, 3x times faster.
Simple example:
        $log->debug('debug', [
            fn() => [
                'one' => \get_class($object),
                'two' => \get_debug_type($object),
                'three' => $object->getName(),
                'four' => $count,
                'five' => $count,
            ],
        ]);

and
        $log->debug('debug', [
                'one' => \get_class($object),
                'two' => \get_debug_type($object),
                'three' => $object->getName(),
                'four' => $count,
                'five' => $count,
        ]);

Creation of closure work 3 times faster: 22925691ms vs 67377785ms vs in 100000 loop in my laptop.
When logging is needed in some hot loop, it matters. Symfony's Messenger does it. When we just started to use it, a worker consumed about 2000 messages per second without logging and only 10 with it. So, logs can slow down to 200 times. Thanks to blackfire >.< we have found the problem.

How about this alternative

Two creations of objects are twice as expensive. It's not good in the fight for performance.

Sorry for the delay here, looks reasonable to me, thanks!


Bug fix/sorted collect reported null as undefined

Scope & Purpose
Fixed a bug where a COLLECT group, that should return null, would return undefined instead.

[X] Bug-Fix for devel
[ ] Bug-Fix for a released version (TODO: Create backports, probably 3.5-3.7)
[X] The behavior in this PR can be (and was) manually tested
[X] The behavior change can be verified via automatic tests

Testing & Verification
This PR adds tests that were used to verify all changes:

[X] Added Regression Tests

Documentation

[ ] Added a Changelog Entry

http://jenkins.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/11403/

Changelog entry is still missing, don't merge until that's added.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/11411/


lost ssh connection after firmware upgrade in ODI sfp in firmware M114_sfp_ODI_hybrid_221209.tar

i upgraded my ODI stick in M114_sfp_ODI_hybrid_221209.tar. But i try to connect via SSH in port 22 and i can't. I tried to add rule in firewall but nothing. I can't connect from Kitty anymore. The firmware doesn't let me change OMCIInfo so i need ssh to pass the flash set commands.
Does anyone knows how to connect with ssh?
thank you

I believe ODI use SHA1 SSH
ssh -oKexAlgorithms=+diffie-hellman-group1-sha1 -oCiphers=+3des-cbc<EMAIL_ADDRESS>
I recommend connect ODI via Telnet instead

i do connect if previous firmware. Only this firmware refuses/ I get the message connection refused. Something like port been closed. I tried firewall settings but again nothing. I ssh from windows with Kitty but yes from linux the ssh needs:
ssh -oKexAlgorithms=+diffie-hellman-group-sha1 -o HostKeyAlgorithms=+ssh-rsa -o PubkeyAcceptedAlgorithms=+ssh-rsa -c 3des-cbc<EMAIL_ADDRESS>anyway i flashed the firmware [M110_sfp_ODI_220923.tar] and i had access again with ssh.

ok Telnet works fine...ssh doesnt in this firmware version

THANK YOU ANIME4000.


test functorch under a debug build of Python

Suggested by @albanD (we want to make sure we're not leaking memory or doing something weird with the pybind API) who volunteered to help with this.
Here are the instructions on how to build functorch and run tests:
# Step 1: Build latest PyTorch, somehow. (NB: not in debug mode, some of our tests fail under that)
...

# Step 2: Build functorch
git clone https://github.com/pytorch/functorch.git
cd functorch
python setup.py develop

# Step 3: run tests

# Testing dependencies. Hopefully these are python-only?
pip install expecttest

# basically, all the tests aside from test_functorch_lagging_op_db are supposed to pass
# assuming our CI is green
# you might get some failures from "networkx is not available" but installing networkx requires having a build of numpy...
pytest -v --ignore=test/test_functorch_lagging_op_db.py test/ -v

Just for the record, building pytorch at 79a216ce576521aebb913d7c5e72e1f36fb04a86 with python 3.8 with pydebug at 66dd5338a1ca98921c8e6c51228541ef8ed8076a
Summary is 15 failed, 5431 passed, 5018 skipped, 376 xfailed, 493 warnings.
One of the error is due to networkx.
The 14 others are RuntimeError: DispatchKeys not yet supported: FuncTorchDynamicLayerBackMode FuncTorchDynamicLayerFrontMode
So all seems good here!

Thanks @albanD !


add insertion sort in python3

insertion sort in python3

Please edit the README file accordingly :)

README updated

README.md updated

Please resolve the merge conflicts :)

Removed merge conflicts please review


Implement a clear_queue button

This PR implements a "Clear Queue" button. Any comments/suggestions are highly appreciated.

Thanks, I will merge this now.

Thanks for your review and helpful suggestions, @cryos!


Media player view size and design size issues

axmol version:
devices test on: iPhone/Windows
developing environments

NDK version: r23c
Xcode version: 14.2+
Visual Studio:

VS version: 2022 (17.9+)
MSVC version: 19.39+
Windows SDK version: 10.0.22621.0+

cmake version:
Steps to Reproduce:

Add the 1280x720.mp4 video from cpp-tests to a fresh project.
Add this code to MainScene.cpp init() method:
auto* videoPlayer = ui::VideoPlayer::create();
videoPlayer->setPosition(visibleSize / 2);
videoPlayer->setAnchorPoint(Vec2::ANCHOR_MIDDLE);
videoPlayer->setContentSize(visibleSize / 2);
videoPlayer->setFileName("1280x720.mp4");
videoPlayer->setLooping(true);
videoPlayer->play();
videoPlayer->setKeepAspectRatioEnabled(true);
videoPlayer->setUserInputEnabled(true);
addChild(videoPlayer);

In AppDelegate.cpp, replace the applicationDidFinishLaunching method with the code here:
bool AppDelegate::applicationDidFinishLaunching()
{
    // initialize director
    auto director = Director::getInstance();
    auto glView   = director->getGLView();
    if (!glView)
    {
#if (AX_TARGET_PLATFORM == AX_PLATFORM_WIN32) || (AX_TARGET_PLATFORM == AX_PLATFORM_MAC) || \
    (AX_TARGET_PLATFORM == AX_PLATFORM_LINUX)
        glView = GLViewImpl::createWithRect("rendertest", ax::Rect(0, 0, 1080, 1920));
#else
        glView = GLViewImpl::create("rendertest");
#endif
        director->setGLView(glView);
    }

    // turn on display FPS
    director->setStatsDisplay(true);

    // set FPS. the default value is 1.0/60 if you don't call this
    director->setAnimationInterval(1.0f / 60);

    // Set the design resolution
    glView->setDesignResolutionSize(720, 1280, ResolutionPolicy::SHOW_ALL);

    // create a scene. it's an autorelease object
    auto scene = utils::createInstance<MainScene>();

    // run
    director->runWithScene(scene);

    return true;
}

Now, with videoPlayer->setKeepAspectRatioEnabled(true);, this happens when you run the app:

No touch input works, since it is all out of alignment, and touching other areas of the screen triggers the media controls.
If I minimize the window, and bring it back up, it changes to this:

All touch works correctly, since the control images are exactly aligned with the touch locations.  Full-screen switching also works.
With videoPlayer->setKeepAspectRatioEnabled(false);:

Same problem with touch input, controls cannot be clicked on, since the actual touch positions are not aligned with the controls.
If you minimize the window, and bring it back up, this happens:

All controls and touch positions are aligned now, and full-screen mode works as expected, for example:

So, what happens when the window is minimized and then brought back to the foreground. Whatever is happening there is fixing the problem with the video player.
Now, the issue of the controls being out of the video player window when videoPlayer->setKeepAspectRatioEnabled(true) is in PR #1851, so it now looks like this (after window is minimized then brought back to foreground):

@halx99 Sorry, I think I confused things a bit.  PR #1851 does not address the problem described in this issue, it's just for adjusting the visual position of the media controls to cater for the aspect ratio.
The issue described here isn't related to the media controls, but rather to something being incorrect with the glview when view size is greater than design size, which is also affecting the touch input locations.  It only fixes itself if the window is minimized (put into background) then brought back (moved to foreground).

@halx99 Possible fix (only tested on Windows so far), adding the following code to bool GLViewImpl::initWithRect(std::string_view viewName, const ax::Rect& rect, float frameZoomFactor, bool resizable) after the call to glfwCreateWindow:
int actualWidth, actualHeight;
glfwGetWindowSize(_mainWindow, &actualWidth, &actualHeight);
if (static_cast<int>(windowSize.width) != actualWidth || static_cast<int>(windowSize.height) != actualHeight)
{
    windowSize.set(static_cast<float>(actualWidth), static_cast<float>(actualHeight));
}

In the example above, attempting to set 1080,1920 actually resulted in a real window size of 1080,1421, so overwriting windowSize with the true size returned by glfwGetWindowSize fixed the problem. I have only tested this on Windows so far.

ok, could create pr?
获取Outlook for Androidhttps://aka.ms/AAb9ysg

From: RH @.>
Sent: Friday, April 26, 2024 11:30:13 PM
To: axmolengine/axmol @.>
Cc: Deal @.>; Mention @.>
Subject: Re: [axmolengine/axmol] Media player view size and design size issues (Issue #1850)
@halx99https://github.com/halx99 Possible fix (only tested on Windows so far), adding the following code to bool GLViewImpl::initWithRect(std::string_view viewName, const ax::Rect& rect, float frameZoomFactor, bool resizable) after the call to glfwCreateWindow:
int actualWidth, actualHeight;
glfwGetWindowSize(_mainWindow, &actualWidth, &actualHeight);
if (static_cast(windowSize.width) != actualWidth || static_cast(windowSize.height) != actualHeight)
{
windowSize.set(static_cast(actualWidth), static_cast(actualHeight));
}
image.png (view on web)https://github.com/axmolengine/axmol/assets/8603230/d4b49c6c-3edb-45e7-9f9b-d5b66597f4b9
In the example above, attempting to set 1080,1920 actually resulted in a real window size of 1080,1421, so overwriting windowSize with the true size returned by glfwGetWindowSize fixed the problem. I have only tested this on Windows so far.
―
Reply to this email directly, view it on GitHubhttps://github.com/axmolengine/axmol/issues/1850#issuecomment-2079619702, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABVHOJ5O4GAJQ6ICLCKORL3Y7JXILAVCNFSM6AAAAABGZE7BB6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZZGYYTSNZQGI.
You are receiving this because you were mentioned.Message ID: @.***>

Unless anyone has any objections, this issue seems to be fixed, so closing it.


Resized image wrong size.

Provide a description of requested docs changes
Hello. So I have an image without a background that is 2000x2000 px and I wanted to resize it to 270x200. When I use Fit mode in my custom size it creates a 200x200 size instead. If I use Fill mode it sizes comes out correct but parts of the item in the image gets cropped.
Is there a way to get the 270x200 format correct from my 2000x2000 image without getting the item on my image cropped off?

Can you confirm this issue is still relevant in v0.73? /needinfo


Alert within Popover won't close

Short description of the problem:
An Alert launched within a Popover stays on the screen, does not close in response to Dismiss button.
What behavior are you expecting?
Alert should close.
Steps to reproduce:

Launch an alert from within a Popover
Click on Dismiss button

       let alert = this.alertCtrl.create( {
        title: "Failed",
        subTitle: "Message",
        buttons: [{ text:'Dismiss' }]
      });
      alert.present();*/

Other information:
Happens because Alert's enabled property is never getting set, is waiting to do so on a lifecycle event that never happens.
Which Ionic Version? 2.0 beta11
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 6.3.1
Gulp version:  CLI version 3.9.1
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-beta.11
Ionic CLI Version: 2.0.0-beta.37
Ionic App Lib Version: 2.0.0-beta.20
OS: Windows 7 SP1
Node Version: v4.4.5

Hello! Thanks for opening an issue with us! Are you opening your alert in this manner:
popover.dismiss().then(() => {
  // create alert
  alert.present();
});

Because all of our navigation in Ionic 2 is async, you have to use promises to wait for one transition to end before firing another one. You can read more about that here. Thanks for using Ionic!

No, I'm presenting the Alert from the Popover object's code on TOP of the unclosed Popover. Nested. Once the user dismisses the Alert (a rare error message), I want the user to resume interacting with the Popover. One could make UX-convention arguments that I shouldn't be using one popup on top of another, but is that flat-out prohibited by Ionic?

Hello! At this time it is prohibited by Ionic with alerts. Our Alerts work very similar to the way the native iOS sdk handles alerts in that when they are open they are the only overlay component that should be open at that time. Sorry for any hassle this causes! However, closing the popover and then opening an alert will work, and in terms of UX may be a more clear indication to the user that there was an issue with the action they initiated from the popover. Hope this explains things, and thanks for using Ionic!

Will this be added to documentation for Alert, etc? Enforcement would also be useful.


3.0.2-beta.1

Breaking Changes
N/A
Changes

feat: Add EIP6963Connector to defaults

Associated Issues
N/A

Closing in favor of https://github.com/WalletConnect/web3modal/pull/1365


IDE tasks should be provided by a separate plugin

At the moment IDE tasks are provided by default when applying the plugin as such:
plugins {
    id 'com.github.blindpirate.gogradle' version '0.6.5'
}

Some users might never want to generate IDE-specific files and therefore do not want to use those tasks or see them in the task list.
It's far more flexible to split this functionality into a separate plugin that can ship with the same JAR. As a user I'd apply the plugin if needed e.g. like this:
plugins {
    id 'com.github.blindpirate.gogradle-ide' version '0.6.5'
}

You can read more about the implementation approach here:

https://guides.gradle.org/designing-gradle-plugins/#capabilities-vs-conventions
https://guides.gradle.org/implementing-gradle-plugins/#assigning_appropriate_plugin_identifiers

I believe this task is not an IDE-related task and therefore should not be grouped with tasks like vscode and the like?
showGopathGoroot - Show GOPATH and GOROOT in build.

IDE support is removed in 0.8.0 because it's a bit hackery. Now users need to set GOPATH and GOROOT which can be displayed by gradle showGopathGoroot.


Adopting support to set the source for a diagnostic

Notice this requires<EMAIL_ADDRESS>

Thanks @egamma !

@egamma I have published 0.9.1 which embed your feature. I have implemented 'source' inside typescript.java and it works well! Thanks!
But it seems that TypeScript 2.3.1 doesn't fill 'source' with 'ts' for diagnostic. Is it normal?

Thanks!

But it seems that TypeScript 2.3.1 doesn't fill 'source' with 'ts' for diagnostic. Is it normal?

yes it is 'normal', we do it this in VS Code on the client side, but it would be better when the tsserver sets the source field, then it is consistent across the clients.

Thnaks @egamma for your answer. Could you fill an issue about this problem in TypeScript issues?


Not able to control the picker with external button

Hi, I'm trying to control the Flatpickr expanded state with external button like this:
import { useRef } from 'react';
import Flatpickr from 'react-flatpickr';
import flatpickr from 'flatpickr';

flatpickr.l10ns.default.firstDayOfWeek = 1;

export const DatePicker = (): JSX.Element => {
  const pickerRef = useRef<Flatpickr | null>(null);
  const calendar = pickerRef?.current?.flatpickr;
  const isOpen = calendar?.isOpen;

  return (
    <div>
      <Flatpickr
        ref={pickerRef}
        options={{
          minDate: new Date(),
          dateFormat: 'd/m/Y',
        }}
      />
      <button
        type="button"
        onClick={!isOpen ? () => calendar?.open() : () => calendar?.close()}
        // onClick={() => calendar?.toggle()} <== it doesn't work as well
      >
        Click
      </button>
    </div>
  );
};

Unfortunately, only the first click on the "Click" button works as expected. It's opening the calendar view.
When I try to close it, then it closes and opens up again the calendar. I'm not sure how to control it properly. Any help would be appreciated.
version: 3.10.7

Clicking outside of the popup closes it automatically, so it's never actually open by the time the click handler is processed.
You can rely on this to handle closing it for you, or you can use something like the inline config option to refactor your input.

So there's no way to add element detection at the current state, right?

I don't know what you mean by that, sorry.
Using event.stopPropagation() in your click handler will probably make it behave as you expect, though as I said you might as well just rely on the auto-close behavior.
Anyway, closing this because it's not a bug, or related to react-flatpickr at all I don't think.


collapseable sorting/grouping hierarchy

can take up quite some space which is not used if just one criteria is defined

won't fix for now


dbb-ucd-packaging.groovy is packaging outputs with a deploy type of null when they should be ignored

In my testing dbb-ucd-packaging.groovy is packaging outputs with a deploy type of null when they should be ignored.  I believe it is because my DBB output has other members with a valid deploy type so the packaging code is incorrectly treating all members as having a valid deploy type.

There is a case not handled by the script. When in the outputs we have more than one entry. If for example we have 2 entries and one of the 2 entries is not null then the 2 entries will be added.
A temporary workaround is to surround this with this test if ( output.deployType != null):

@dennis-behm  This fix is in this PR https://github.com/IBM/dbb/pull/75 (with another one).
@peterceck  FYI

Well, just briefly.
Why should we not allow an empty deployType? According to the UCD docs, this is optional:
https://www.ibm.com/docs/en/urbancode-deploy/7.1.1?topic=platform-ship-list-files
Peter, what is the issue you are facing? Does the packaging or the deployment fails?

Yes, the UCD deployment fails if there is no mapping rule for the object type.  See my UCD output below.  I could deploy the object to a dummy dataset because it is not a executable that needs to be deployed.  But this is wasteful. It is better to exclude it from the package like we always did with RTC.   If a type does need to be deployed then the DBB build scripts can be updated to set a deploy type for it.
/usr/lpp/java/J8.0_64/bin/java -Djava.library.path=/var/ucd/v7/lib/zos/ -Dfile.encoding=IBM-1047 -Dconsole.encoding=IBM-1047 -Djdk.tls.client.protocols=TLSv1.2 -Xmx128m -cp /var/ucd/v7/opt/apache-ant-1.9.15/lib/ant.jar:/var/ucd/v7/opt/apache-ant-1.9.15/lib/ant-launcher.jar:/var/ucd/v7/var/plugins/com.ibm.ucd.plugin.zos_54_2e7ccba3dca88967d25b4c928292464af064f26c4687210cee57ed248c067a1f/lib/ant-contrib.jar:/var/ucd/v7/var/plugins/com.ibm.ucd.plugin.zos_54_2e7ccba3dca88967d25b4c928292464af064f26c4687210cee57ed248c067a1f/lib/zos-deploy-common.jar:/var/ucd/v7/var/plugins/com.ibm.ucd.plugin.zos_54_2e7ccba3dca88967d25b4c928292464af064f26c4687210cee57ed248c067a1f/lib/uDeployRestClient.jar:/var/ucd/v7/lib/zos/zostoolkit.jar:/var/ucd/v7/opt/zoslib/slpp.jar:/var/ucd/v7/var/plugins/com.ibm.ucd.plugin.zos_54_2e7ccba3dca88967d25b4c928292464af064f26c4687210cee57ed248c067a1f/classes:/var/ucd/v7/var/plugins/com.ibm.ucd.plugin.zos_54_2e7ccba3dca88967d25b4c928292464af064f26c4687210cee57ed248c067a1f/lib/securedata.jar:/var/ucd/v7/var/plugins/com.ibm.ucd.plugin.zos_54_2e7ccba3dca88967d25b4c928292464af064f26c4687210cee57ed248c067a1f/lib/jettison-1.1.jar:/var/ucd/v7/var/plugins/com.ibm.ucd.plugin.zos_54_2e7ccba3dca88967d25b4c928292464af064f26c4687210cee57ed248c067a1f/lib/commons-codec.jar:/var/ucd/v7/var/plugins/com.ibm.ucd.plugin.zos_54_2e7ccba3dca88967d25b4c928292464af064f26c4687210cee57ed248c067a1f/lib/zos-deploy-native.jar:/var/ucd/v7/var/plugins/com.ibm.ucd.plugin.zos_54_2e7ccba3dca88967d25b4c928292464af064f26c4687210cee57ed248c067a1f/lib/CommonsUtil.jar:/var/ucd/v7/opt/groovy-2.4.15/lib/* deploy_package /var/ucd/v7/var/temp/logs3682168535997635658/input.props /var/ucd/v7/var/temp/logs3682168535997635658/output.props
Missing PDS mapping rules for:  PCECK.TUTO.GITLAB.MORTGAGE.OBJ

Ok, it fails, because there is no PDS mapping in your case. So, isn't the problem located earlier in the process?
Isn't it an issue on zAppBuild, that it reports PCECK.TUTO.GITLAB.MORTGAGE.OBJ as an output, while we don't would like to have it as an output?

@dennis-behm  The current code in  dbb-ucd-packaging.groovy filter the cases when the deployType is null but not all cases see; https://github.com/IBM/dbb/issues/76#issuecomment-826927925. I think this case arrived with OBJ and COPYBOOK. Now @drbruce-git may have a good reason to make zAppBuild adding  OBJ and COPYBOOK as outputs.

And this  fix here: https://github.com/jbyibm/dbb/commit/a2ef28f815a5d0b767121e6c3db219ef2a125094

While it is true that the deployType field requires the output field to be set, the reverse is not true.  You can have an output field without a deploy type.  This was done by design (and is similar to RTC-EE) and allows you to record the fact in your build report that a build step resulted in an output for use by another language build without it necessarily meant to be deployed like an OBJ deck or BMS Copybook

Closing while it has been resolved


WolfLink Disappearing!

Hello there! I noticed about the next update, that whenever you load the game, the arrow/ tool you edited to be the summoner of wolflink is gone. Is there anything I can do, so that, I can summon WolfLink or any other enemy whenever I am playing without opening my pc to edit it again? Thanks!

Hmm, not sure what you mean? The wolf link health code is re-enabled but nothing has been removed?

What I mean is:
I am using an weapon like sword to summon wolflink. after editing it'll be an unknown item of-course. So Just equip it using the key combo on the controller and the wolflink appears. My problem is, when I quit the game then load again, the item I edited is gone. So I need to have another weapon to edit to become the summoner of wolflink. I just want to know if it is possible to not keep on editing an item to become a summoner, just it will be permanent and no need to open a laptop to edit again.?

No there is no way to do that. After you change the weapon to an invalid ID and save and load it'll delete said item due to it not being a weapon any longer. There is no permanent way to keep an item that allows summoning of certain NPC's.

Oh I see! But last time, before the update 1.2. It's good, it keeps.


switch global null to come from Nullable module and not Null

This changes the globally available null value to originate from Nullable rather than Null. Makes sense imo given that global null and undefined now originate from the same module.
It also makes working with domRefs in React more natural, allowing you to write null directly instead of Nullable.null:
@react.component
let make = () => {
  let divRef = React.useRef(null)
  <div ref={divRef->ReactDOM.Ref.domRef}> {"Hello"->React.string} </div>
}

(side note - would be cool to get completion for ReactDOM.Ref.* in the scenario below. Might be able to work that into the editor tooling)
Thoughts? cc @cknitt @glennsl

I like it! 🙂

Ditto! :)

What about typical bindings, is the idea that one most often encounters null | undefined | T, rather than null | T?
Asking as the few binding I looked at to add genType support to Core were using null | T.
Otherwise one has o bind with Null.t and will have most often to reach for Null.null.

Good question. I seems to recall the decision was made long ago to prefer Nullable.t over Null.t and Undefined.t, just for simplicity. I don't think React refs have ever been set to undefined, so I believe that is the reason why it's typed as it is.
I wonder if it would be practical to make Null.t a subtype of Nullable.t...


Next start code simd

Fixed a pointer type in the load instructions

Thank you for the change. Merged to main. Closing ...


Can you access methods from objects Added to a ScriptObject, rather than Imported?

I'm adding several .NET objects to a ScriptObject. The properties of these objects are accessible, but the methods are not.
I set a MemberFilterDelegate like  this:
templateContext.MemberFilter = m => true;

In theory, that should allow every member in, but the methods still are not available.
In reading the doc, it seems that the default MemberFilterDelegate only applies to objects brought into a ScriptObject through Import rather than Add. Is this correct?
In a larger sense, what is the difference between Import and Add?  Why would I import something to a ScriptObject, rather than adding it?
Is there any way to expose methods of objects added to a ScriptObject?

Is there any way to expose methods of objects added to a ScriptObject?

There is no way. First because it is a safety feature of scriban (you can't accidentally release a method that a user could use) and second because "methods" in scriban don't really exist, there is no concept of "this" receiver for example. They are only "static functions" attached to an object but nothing else.

Good to know.
What is the difference between Add and Import?  Describe it to me, and I'd be happy to submit a PR for the doc clarification.
(I write a bit...)

Import is making a copy of what the object is exposing (public fields, properties, static functions...) to the ScriptObject.
Otherwise ScriptObject is in the end just a dictionary, so if you add a .NET object as a value of a key, it will be accessible and its properties as well (but only its properties/fields, not methods), in that case they are not imported, you are actually manipulating the same .NET object.

The documentation for Import .NET object is here it is probably not clear, but the difference is at least stated 😅

I'll be honest, I still don't understand why Add and Import are two different methods. I spent some time playing around in LinqPAD -- I added and imported all sort of stuff, so I have the specifics down. But in terms of language semantics, I just think the mental model of both is "make this data available to the template," and so it's confusing that there are two different methods for this.

Add adds to a dictionary directly, Import takes things from one object and copy them to the object you import to.


Unable to use laravel-model-settings with laraval-spatie-multitenancy

Describe the bug
I'm unable to use laravel-model-settings with laraval-spatie-multitenancy due to collision  between traits
To Reproduce
Steps to reproduce the behavior:

Create a model that use both "HasSettingsField" and "UsesLandlordConnection" or "UsesTenantConnection"
Execute seeder
php artisan tenants:artisan "migrate --path=database/seeders/tenant --database=tenant --seed

Expected behavior
Seed database without exception
** Actual behavior**
Trait method Glorand\Model\Settings\Traits\HasSettingsField::getConnectionName has not been applied as App\Models\Tenant::getConnectionName, because of collision with Spatie\Multitenancy\Models\Concerns\UsesLandlordConnection::getConnectionName
Screenshots

Additional context
https://spatie.be/docs/laravel-multitenancy/v1/introduction

You can omit a method's inclusion by excluding it from one trait in favor of the other and doing the exact same thing in the reverse way.
trait A {
    public function sayHello()
    {
        echo 'Hello from A';
    }

    public function sayWorld()
    {
        echo 'World from A';
    }
}

trait B {
    public function sayHello()
    {
        echo 'Hello from B';
    }

    public function sayWorld()
    {
        echo 'World from B';
    }
}

class Talker {
    use A, B {
        A::sayHello insteadof B;
        A::sayWorld insteadof B;
        B::sayWorld insteadof A;
    }
}

$talker = new Talker();
$talker->sayHello();
$talker->sayWorld();

The method sayHello is imported, but the method sayWorld is simply excluded.

Overriding & Extending a PHP Trait Method
trait FooTrait
{
	public function bar()
	{
		return 'A';
	}
}

class Example
{
	use FooTrait {
		bar as traitBar;
	}

	public function bar()
	{
		return $this->traitBar() . 'B';
	}
}

echo (new Example())->bar(); // Outputs 'AB'

Reference: https://andy-carter.com/blog/overriding-extending-a-php-trait-method


FR: table with all significant symbols defined

E.g.

Great suggestion, will try to fix this soon (hopefully along with expanded set of notes)


Merging tables

Codecov Report

Merging #2399 into dev will increase coverage by <.01%.
The diff coverage is 95.83%.

@@            Coverage Diff             @@
##              dev    #2399      +/-   ##
==========================================
+ Coverage   93.77%   93.77%   +<.01%
==========================================
  Files         161      161
  Lines       18687    18702      +15
==========================================
+ Hits        17523    17538      +15
  Misses       1164     1164

Impacted Files
Coverage Δ

...iita_pet/handlers/api_proxy/tests/test_artifact.py
98.22% <ø> (ø)
:arrow_up:

...pet/handlers/study_handlers/tests/test_artifact.py
97.19% <ø> (ø)
:arrow_up:

qiita_db/test/test_analysis.py
98.12% <100%> (+0.07%)
:arrow_up:

qiita_db/test/test_util.py
99.63% <100%> (ø)
:arrow_up:

qiita_db/analysis.py
96.46% <100%> (+0.49%)
:arrow_up:

qiita_db/util.py
96.53% <90%> (-0.4%)
:arrow_down:

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 d8aa061...6617c10. Read the comment docs.


Do not enter _ready twice

There are two commits in this PR -- the first guards only against a special case of add_child inside _enter_tree, while the second one guards against all cases.
Because of this, this PR is going to break some projects (and maybe AnimationPlayer/Tween/Joint2D as well, who knows..).
But, the first commit fixes a buggy behavior, so it can (and maybe should) be backported to 2.x.

@reduz WDYT about the change?

Reverted b6eab00 as discussed in https://github.com/godotengine/godot/issues/3290#issuecomment-263388003


community guidelines

This PR adds some community guidelines and templates for issues and PRs for #4 .
I'll do another pass when the repository is public because some of this only works for public repositories.

Does CONTRIBUTING.md constitute a CLA? Should we add some boilerplate text at the bottom of README.md?

Good point. I added a CLA.md and mention it in CONTRIBUTING.md.
The PR template also requests to tick the box for heaving read the agreements.
Do you think that's enough?

@karthikbhargavan ?

Looks good to me.


fix process time count

Issues associated with this PR
Your PR should present related issues you want to solve.
Sign the CLA
Make sure you have signed the CLA
Solutions
You should show your solutions about the issues in your PR, including the overall solutions,
details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases.
And you need to show the ut result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #846 into stable will increase coverage by <.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           stable     #846      +/-   ##
==========================================
+ Coverage   55.68%   55.69%   +<.01%
==========================================
  Files         233      233
  Lines       23755    23757       +2
==========================================
+ Hits        13228    13231       +3
+ Misses       9316     9313       -3
- Partials     1211     1213       +2

Impacted Files
Coverage Δ

pkg/proxy/downstream.go
33.98% <50%> (+0.74%)
:arrow_up:

pkg/module/http2/transport.go
78.3% <0%> (-0.3%)
:arrow_down:

pkg/module/http2/server.go
81.77% <0%> (+0.07%)
:arrow_up:

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 cfffdd8...c301b5e. Read the comment docs.


some question about result

why your second baseline's all class result is lower compared to openpcdet's result, all class ap lower more than 2%, especially for ped( l1ap lower 7%)?  small batch size will get better result, but don't have so much different, even I use 8 v100 32g to train, I don't have so much difference with openpcdet's result, did you use different config like bigger voxel size?
why not compared with centerpoint, centerpoint just use centerhead as anchorhead, it don't have too much difference with Second, but it did give very much improvement for cyc and ped, does patch is still useful?

Thanks for the questions.

We start this work in January, 2021. Our code is based on a much older version (v0.4.0) of OpenPCDet (please refer to this git commit https://github.com/open-mmlab/OpenPCDet/tree/a7cf5368d9cbc3969b4613c9e61ba4dcaf217517).  OpenPCDet is being continually updated and has come to v0.5.2 with improved implementation. The SECOND baseline of OpenPCDet v0.4.0  is lower than that of OpenPCDet v0.5.2.     And it's worth noting that, even though AziNorm-based SECOND is based on a poorer version of OpenPCDet, it still significantly outperforms SECOND based on the up-to-date OpenPCDet. We will provide the code and models of both SECOND and AziNorm-based SECOND for comparison.

For the same reason, at the time we conducted experiments, CenterPoint was not supported by OpenPCDet yet. (OpenPCDet v0.5.0 starts to support CenterPoint).  We validated AziNorm based on available SECOND and PV-RCNN.
The improvement of AziNorm comes from normalizing point clouds along the radial direction, instead of the design of patch (refer to Tab.6 of the paper).   It’s intuitive that leveraging the radial symmetry is beneficial, no matter whether centerhead or anchorhead is adopted. We think AziNorm can also improve CenterPoint. If time permits, we will consider migrating the code into the newest version of OpenPCDet and including experiments about CenterPoint.

1.openpcdet v0.5.2's improvement come from use gtaug from all data, rather than 10% data in v0.4.0, so v0.4.0 and v0.5.2 don't have too much different.
2.I know the AziNorm's improvement come from normalize along the radial direction, sorry for only saying patch。I think the implementation of operation that patch and then normalize is not hard, so why not use centerpoint's official repo to compare centerpoint and centerpoint with AziNorm, I just think compare with sota is more convincing.

Our experiments are based on the same augmentation of v0.4.0. Thus the baseline is lower.
Thanks for the advice. It's OK to integrate AziNorm into centerpoint's official rep. Experiments of SECOND and PV-RCNN are both based on OpenPCDet. Working on a unified and well-developed codebase can reduce the workload.

Maybe I can help to answer this question, OpenPCDet change to use two return data of Waymo in v0.5.0. The performance difference of 1 return and 2 return results can be found in the PVRCNN++ paper. @AndyYuan96
@outsidercsy Looking forward to see your code, and I think it would be much better to also provide results with OpenPCDet >= v0.5.0 since the 2return results of Waymo is more commonly used in recent works. I also worry about the performance mismatch before see this issue.