Install from VSIX... does not install from VSIX if extension has been uninstalled

Issue Type: Bug
We've had cases where installing a new version of our beta extension doesn't seem to actually install it.  This is one scenario where I can definitely repro, although I'm not positive it's the only one.

Find a VSIX to experiment with (I can e-mail mine two versions if you want)
Make a copy of the VSIX, rename both to be clear (e.g. vsix1.vsix, vsix2.vsix)
Rename them to .zip files
Add a version1.txt file to vsix1 and a version2.txt file to vsix2 (in the extensions folder)
Rename back to .vsix
Use Install from VSIX...  to install vsix1, then vsix2, then vsix1.  After each install, verify on disk that the expected version{1,2}.txt file is in the extension folder (home/.vscode-insiders/extensions/{extensionfolder}
Install vsix1
disable the extension from vscode's extensions pane
Use Install from VSIX... again to install vsix2
!) If you check on disk, you'll see that it's actually vsix1 that is installed
EXPECTED: For Install from VSIX..., it should always install from the actual VSIX, and not simply restore what is currently on disk.

VS Code version: Code - Insiders 1.40.0-insider (37d34176a6ecf1d09b16a0cad11bb619f3b1e48f, 2019-11-01T10:19:19.506Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Xeon(R) W-2133 CPU @ 3.60GHz (12 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.73GB (14.16GB free)

Process Argv
--folder-uri file:///c%3A/Users/stephwe/Repos/vscode-azurearmtools

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

vscode-typescript-tslint-plugin
ms-
1.2.2

azurerm-vscode-tools
msa
0.8.1-alpha

Can you please provide the vsixs you used, so that it will be easy for me to reproduce what you are seeing. Guess the steps remain same with the vsixs you provided.

Simplified the repro.  However, the bug seems to have been fixed (and in particular, rather than simply enabling the extension disabled extension when vsix2 is installed and vsix1 is disabled,  it installs vsix2 but keeps it disabled).  Thanks.


marvin: fix exception logging

Description
Fixes #5203
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[x] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

Tested marvin logs with and without changes while making ACS throw exception for an async API
[root@pr5201-t1929-kvm-centos7-marvin marvin]# nosetests --with-xunit --xunit-file=results.xml --with-marvin --marvin-config=./pr5201-t1929-kvm-centos7-advanced-cfg -s -a tags=advanced --hypervisor=KVM tests/smoke/test_deploy_vm_iso.py

==== Marvin Init Started ====

=== Marvin Parse Config Successful ===

=== Marvin Setting TestData Successful===

==== Log Folder Path: /marvin/MarvinLogs/Sep_02_2021_08_56_13_EU0IU4 All logs will be available here ====

=== Marvin Init Logging Successful===

==== Marvin Init Successful ====
=== TestName: test_deploy_vm_from_iso | Status : EXCEPTION ===

=== Final results are now copied to: /marvin//MarvinLogs/test_deploy_vm_iso_AWJQLB ===
[root@pr5201-t1929-kvm-centos7-marvin marvin]# cat /marvin//MarvinLogs/test_deploy_vm_iso_AWJQLB/failed_plus_exceptions.txt
2021-09-02 08:56:26,082 - CRITICAL - EXCEPTION: test_deploy_vm_from_iso: ['NoneType: None\n']
[root@pr5201-t1929-kvm-centos7-marvin marvin]# rm -f /usr/local/lib/python3.6/site-packages/marvin/cloudstackException.py
[root@pr5201-t1929-kvm-centos7-marvin marvin]# vi /usr/local/lib/python3.6/site-packages/marvin/cloudstackException.py
[root@pr5201-t1929-kvm-centos7-marvin marvin]# python3 -m py_compile /usr/local/lib/python3.6/site-packages/marvin/cloudstackException.py
[root@pr5201-t1929-kvm-centos7-marvin marvin]# nosetests --with-xunit --xunit-file=results.xml --with-marvin --marvin-config=./pr5201-t1929-kvm-centos7-advanced-cfg -s -a tags=advanced --hypervisor=KVM tests/smoke/test_deploy_vm_iso.py

==== Marvin Init Started ====

=== Marvin Parse Config Successful ===

=== Marvin Setting TestData Successful===

==== Log Folder Path: /marvin/MarvinLogs/Sep_02_2021_08_57_49_DQLC8J All logs will be available here ====

=== Marvin Init Logging Successful===

==== Marvin Init Successful ====
=== TestName: test_deploy_vm_from_iso | Status : EXCEPTION ===

=== Final results are now copied to: /marvin//MarvinLogs/test_deploy_vm_iso_1X0ZMN ===
[root@pr5201-t1929-kvm-centos7-marvin marvin]# cat /marvin//MarvinLogs/test_deploy_vm_iso_1X0ZMN/failed_plus_exceptions.txt
2021-09-02 08:58:02,358 - CRITICAL - EXCEPTION: test_deploy_vm_from_iso: ['Traceback (most recent call last):\n', '  File "/usr/lib64/python3.6/unittest/case.py", line 60, in testPartExecutor\n    yield\n', '  File "/usr/lib64/python3.6/unittest/case.py", line 622, in run\n    testMethod()\n', '  File "/marvin/tests/smoke/test_deploy_vm_iso.py", line 150, in test_deploy_vm_from_iso\n    hypervisor=self.hypervisor\n', '  File "/usr/local/lib/python3.6/site-packages/marvin/lib/base.py", line 672, in create\n    virtual_machine = apiclient.deployVirtualMachine(cmd, method=method)\n', '  File "/usr/local/lib/python3.6/site-packages/marvin/cloudstackAPI/cloudstackAPIClient.py", line 2625, in deployVirtualMachine\n    response = self.connection.marvinRequest(command, response_type=response, method=method)\n', '  File "/usr/local/lib/python3.6/site-packages/marvin/cloudstackConnection.py", line 381, in marvinRequest\n    raise e\n', '  File "/usr/local/lib/python3.6/site-packages/marvin/cloudstackConnection.py", line 376, in marvinRequest\n    raise self.__lastError\n', '  File "/usr/local/lib/python3.6/site-packages/marvin/cloudstackConnection.py", line 105, in __poll\n    % async_response)\n', 'Exception: Job failed: {accountid : \'f91ae009-0bb0-11ec-85de-1e0050000361\', userid : \'f91c23af-0bb0-11ec-85de-1e0050000361\', cmd : \'org.apache.cloudstack.api.command.admin.vm.DeployVMCmdByAdmin\', jobstatus : 2, jobprocstatus : 0, jobresultcode : 530, jobresulttype : \'object\', jobresult : {errorcode : 533, errortext : \'Unable to create a deployment for VM instance {id: "33", name: "i-35-33-VM", uuid: "6a8e9431-8a31-4a4e-be22-b4a9f97283b9", type="User"}\'}, jobinstancetype : \'VirtualMachine\', jobinstanceid : \'6a8e9431-8a31-4a4e-be22-b4a9f97283b9\', created : \'2021-09-02T08:57:55+0000\', completed : \'2021-09-02T08:57:55+0000\', jobid : \'9fcc8cf6-6433-423d-8577-c8972b6235b9\'}\n']

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 1098

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 1127

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-1976)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 41511 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr5401-t1976-kvm-centos7.zip
Smoke tests completed. 89 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

Ping for review @weizhouapache @rhtyd @davidjumani @sureshanaparti


warn: nuki-extended.0 (81304) Failed retrieving /list from Nuki Bridge with name nuki-bridge (forcePlainToken: false)!

v2.6.0 gives this Warning:
2022-06-05 21:26:47.960  - warn: nuki-extended.0 (81304) Failed retrieving /list from Nuki Bridge with name nuki-bridge (forcePlainToken: false)!

Changed PlainToken setting, but the Warning persists:
2022-06-05 21:28:21.773  - info: nuki-extended.0 (81338) starting. Version 2.6.0 in /opt/iobroker/node_modules/iobroker.nuki-extended, node: v18.3.0, js-controller: 4.0.23
2022-06-05 21:28:21.897  - info: nuki-extended.0 (81338) Polling Nuki Web API with a frequency of 20s.
2022-06-05 21:28:22.165  - warn: nuki-extended.0 (81338) Failed retrieving /list from Nuki Bridge with name nuki-bridge (forcePlainToken: true)!

Here the same, with Node 14 and NPM 6. Adapter starts, but don't react with the bridge any more. No command will be proceed to the devices. ( one bridge, 4 Locks ).
First Adapter is green, after a while it change to yellow without an error.  Same issue was some versions before..
Version 2.5.0 works perfect.

Same here!

States for the devices (Lock and Opener) are not updated and not recreated when once deleted. Only the bridge comes up.

Same problems with 2.6.0. No actions are possible. after the downgrade to 2.5.0 it works again.

Debug log please ...

@theimo1221 thank you for checking

Morning @w0nk1, @Grothesk242, @Azathoth88 and @Ilovegym66,
Please change the adapter logging to debug, and paste the message after the Failed retrieving... warning here, to help us identifying the issue.
Thanks and best regards
Thiemo

Habe den Adapter jetzt auf nochmal auf die neue Version gezogen. Debug-Mode ist an.
Was erstmal auffiel: Der Status: Verbunden mit Gerät oder Dienst war nach dem Update gelb.
Nachdem ich diesen nochmals neu gestartet habe, grün. "Mehr" Log wird aber auch nicht geliefert.
Ich habe Probeweise die ACTION zum Tür öffnen verwendet - keine Reaktion und kein Eintrag im Log.

nuki-extended.0
2022-06-09 08:49:01.428
info
Listening for Nuki events on port 51989.

nuki-extended.0
2022-06-09 08:49:01.131
info
State value to set for "nuki-extended.0.bridges.zuhausebridgewhite.scanResults.1.rssi" has to be type "string" but received type "number"

nuki-extended.0
2022-06-09 08:49:01.130
info
State value to set for "nuki-extended.0.bridges.zuhausebridgewhite.scanResults.0.rssi" has to be type "string" but received type "number"

nuki-extended.0
2022-06-09 08:49:01.090
info
State value to set for "nuki-extended.0.bridges.zuhausebridgewhite.uptime" has to be type "string" but received type "number"

nuki-extended.0
2022-06-09 08:49:00.688
warn
Failed retrieving /list from Nuki Bridge with name ZuhauseBridgeWhite (forcePlainToken: true)!

nuki-extended.0
2022-06-09 08:49:00.359
info
Polling Nuki Web API with a frequency of 300s.

nuki-extended.0
2022-06-09 08:49:00.251
info
starting. Version 2.6.0 in /opt/iobroker/node_modules/iobroker.nuki-extended, node: v14.19.3, js-controller: 4.0.23

nuki-extended.0
2022-06-09 08:48:55.648
info
Terminated (ADAPTER_REQUESTED_TERMINATION): Without reason

nuki-extended.0
2022-06-09 08:48:55.647
info
terminating

nuki-extended.0
2022-06-09 08:48:55.647
info
Adapter stopped und unloaded.

nuki-extended.0
2022-06-09 08:48:55.646
info
Got terminate signal TERMINATE_YOURSELF

nuki-extended.0
2022-06-09 08:48:03.240
info
Listening for Nuki events on port 51989.

nuki-extended.0
2022-06-09 08:48:02.941
info
State value to set for "nuki-extended.0.bridges.zuhausebridgewhite.scanResults.1.rssi" has to be type "string" but received type "number"

nuki-extended.0
2022-06-09 08:48:02.940
info
State value to set for "nuki-extended.0.bridges.zuhausebridgewhite.scanResults.0.rssi" has to be type "string" but received type "number"

nuki-extended.0
2022-06-09 08:48:02.900
info
State value to set for "nuki-extended.0.bridges.zuhausebridgewhite.uptime" has to be type "string" but received type "number"

nuki-extended.0
2022-06-09 08:48:02.509
warn
Failed retrieving /list from Nuki Bridge with name ZuhauseBridgeWhite (forcePlainToken: true)!

nuki-extended.0
2022-06-09 08:48:02.179
info
Polling Nuki Web API with a frequency of 300s.

nuki-extended.0
2022-06-09 08:48:02.079
info
starting. Version 2.6.0 in /opt/iobroker/node_modules/iobroker.nuki-extended, node: v14.19.3, js-controller: 4.0.23

nuki-extended.0
2022-06-09 08:47:57.382
info
Terminated (ADAPTER_REQUESTED_TERMINATION): Without reason

nuki-extended.0
2022-06-09 08:47:57.381
info
terminating

nuki-extended.0
2022-06-09 08:47:57.381
info
Adapter stopped und unloaded.

nuki-extended.0
2022-06-09 08:47:57.380
info
Got terminate signal TERMINATE_YOURSELF

Please Try GitHub version. Versiuon number wil not change so please manually restart after update.
Note: With npm 6 GitHub installs can take 10+mins, just wait)

Ok I iterate with @arteck rigjt now ... so better wait :-)


GUI Desktop App for uv

At some point, Astral will want revenues to help fund everything.
One thing that business users like is GUI desktop apps. They're willing to pay for them, even though they're arguably "just" wrappers around functionality that's available in the CLI. See, for example:

GitKraken (for git)
JetBrains DataGrip (for PostgreSQL)
Docker Desktop (for Docker); you only have to pay if your company is big

FYI in my mind the most comporable GUI tool to something as "uv but with a GUI" would be the Anaconda Navigator. And indeed, it does require payment for companies with more than 200 employeers.
I have no idea what astral's future plans are, but there's certainly room for disruption. I used to maintain an Anaconda distrubition with ~5000 active installs across the company, I was surprised I'd often get questions from data scientists and analysists whose primary method to interact with Python environments was via the GUI, as it was not a pleasent tool to use.


About Loss Function

Hi, Thanks for your great work first!
I'm just wondering why you use nll_loss  instead of cross_entropy in your code https://github.com/TorchSSL/TorchSSL/blob/c4d272202b254efd85e78c9f72c8842edc752e04/train_utils.py#L295
BTW, I see cross_entropy was commented and said it is unstable, can you share your experience about why it is unstable?
For my understanding, CrossEntropyLoss = softmax + log + nll_Loss.
If I'm missing something, please forgive me. Thanks!

I think NLL loss is more numerical steady than cross entropy cause it calculate log softmax instead of softmax first. For more information, you can check https://stats.stackexchange.com/questions/436766/cross-entropy-with-log-softmax-activation

Thanks for your reply.
I'll check it in my experiment.


Content-Disposition header is ignored with OkHttpBackend

When the following code is executed the Content-Disposition header is missing.
val request = sttp.          get(uri"$requestUri").readTimeout(25.minutes)
implicit val backend = OkHttpSyncBackend()
val response = request.response(asByteArray).send()
val contentDispositionHeader = response.headers.toMap.getOrElse("Content-Disposition",null)

I assume it is sent back by the server? Does only OkHttpSyncBackend have this behavior? What happens if you use HttpURLConnectionBackend ?

Yes it sure does, checked it with Postman.
With HttpURLConnectionBackend it gives a connection timeout exception, with the same amount of readTimeout. So I've no way to check the response here.

Hm well that's weird but might happen :) Maybe you'd like to attempt creating a test case?

Closing due to lack of activity


chore(dependencies): update dependencies

📝 Changed routes:

/community
/wallet-sdk

(view full size)
(view full size)

Commit f7f0cf6fc242df75e469dcc5fc2cd2f943772632 (https://thirdweb-pn8gun1wb.thirdweb-preview.com).


feat: docker compose for dagger quickstart

fixes #207
Acceptance Criteria:

GIVEN
WHEN
THEN

Jack is a new user who wants to try out Dagger locally quickly, AND,Jack doesn’t want to do any complex installation of dependencies like Kafka or Apache Flink locally
Jack runs the Docker Compose  file by executing docker compose up in the directory containing the docker compose file
Run Dagger jar in a containerA local kafka cluster should be setup with just one broker containerA kafka producer container should be spawned which continuosly produces messages to some source Kafka topicA bare minimum dagger job with just a 30 sec aggregateSQL query should get submitted to the local Flink cluster with both source and sink as KafkaThe dagger should continue to consume messages from the source topic and publish messages to the sink topic until Jack explicitly executes docker compose down

Jack is a new user who wants to try out Dagger locally quickly, AND,Jack doesn’t want to do any complex installation of dependencies like Kafka or Apache Flink locally
Jack runs the Docker Compose  file by executing docker compose down in the directory containing the docker compose file
All the spawned containers: Dagger, Kafka, etc should be closed down gracefully.

@sumitaich1998 Let's name the topmost folder as quickstart instead of quickstart-docker.
This folder will house other resources as well in the future stories and not just the docker compose file


BIO_get_ktls_recv(): fix return check

Return checks like if (BIO_get_ktls_recv()) are changed to if (BIO_get_ktls_recv() > 0) to capture more accurate semantic. Most of return checks in if (!BIO_get_ktls_recv()) format are reserved except it leads to error processing stage, where negative return values should be classfied too.
Thanks for taking time for reviewing.

CI failure is relevant.


Warnings about locale from dev-c7

bash: warning: setlocale: LC_ALL: cannot change locale (en_GB.UTF-8): No such file or directory
bash: warning: setlocale: LC_ALL: cannot change locale (en_GB.UTF-8)
/bin/sh: warning: setlocale: LC_ALL: cannot change locale (en_GB.UTF-8)

In ubuntu this can be fixed with
apt install locale
I have yet to work out how to do it on centos (And have spent a bit of time trying)

If you
unset LC_ALL
Then the messages go away. However I'm not sure what the side effects of this would be.

Another approach to preserve the en_GB setting. I was able to make it work by adding the following lines to the build
RUN sed -i "s/override_install_langs=en_US/override_install_langs=en_GB/" /etc/yum.conf
ENV LC_ALL=en_GB.utf8
RUN yum reinstall -y glibc-common

This is based on  https://stackoverflow.com/questions/52401710/change-system-locale-inside-a-centos-rhel-without-using-localectl and other results from furious googling.
Not sure why the inconsitency in the capitalisation of "utf8" requiring me to change the env var.
Any thoughts on these approaches?

Andy, I have also seen the yum reinstall approach and think that is the way to go.

I found that I have LC_ALL set in my .bashrc_local.
It does not seem to be required for me elsewhere any more. So as long as nobody else is getting this issue then I think we can take no further action on the Dockerfile. If it does come up again then I think the glibc-common change above will work.

@aawdls I'm not sure this is resolved. Michael Abbott has seen this and it is because some apps he wants to use are trying to set locale.


Issue 7 - Posting message doesnt work in build version:

Updated the post message method to call chat.postMessage instead of
channel.info.  Regenerated a minified version as well.

Hey @J-Graham, just wondering what you used to generate the minified version?

In this case it was easy to just run it through a JS compression tool since it is all in 1 file.  However, if you are looking into minifying larger angular applications I would recommend this method along with GRUNT http://chrislarson.me/blog/how-minify-angularjs-scripts.html

@J-Graham Great thanks. Would be great if we could integrate grunt or gulp into this project so that anyone can update the build folder as this needs to be done everytime the src is updated.


Error: Model 'Yanki - Basic (and reversed card)' not found

I'm finally able to reproduce an issue previously reported by @dscherdi in #1. Looking into it now, it might have to do with the sequence in which notes are synced.
Also seems to affect "Yanki - Basic (type in the answer)" note types.

Should be fixed in 1.2.0.


Clock drifts constantly on macOS because fedora's pool is unavailable

Describe the bug
The clock in the underlying VM on macOS drifts. Even over this weekend, it drifted nearly 2 mins.
Steps to reproduce
Start finch on macOS, run a long running container. Wait a couple of days. Run date and see it is no longer accurate
Expected behavior
I expect sub-second accuracy of the VM's clock.
Screenshots or logs
Sample logs on the VM:
Oct 21 10:07:18 lima-finch chronyd[665]: Source <IP_ADDRESS> replaced with <IP_ADDRESS> (2.fedora.pool.ntp.org)
Oct 21 10:08:50 lima-finch chronyd[665]: Detected falseticker <IP_ADDRESS> (2.fedora.pool.ntp.org)
Oct 21 10:08:50 lima-finch chronyd[665]: Selected source <IP_ADDRESS> (2.fedora.pool.ntp.org)
Oct 21 10:08:54 lima-finch chronyd[665]: Can't synchronise: no majority
Oct 21 10:09:54 lima-finch chronyd[665]: Detected falseticker <IP_ADDRESS> (2.fedora.pool.ntp.org)
Oct 21 10:09:54 lima-finch chronyd[665]: Selected source <IP_ADDRESS> (2.fedora.pool.ntp.org)
Oct 21 10:10:00 lima-finch chronyd[665]: Can't synchronise: no majority
Oct 21 10:10:59 lima-finch chronyd[665]: Detected falseticker <IP_ADDRESS> (2.fedora.pool.ntp.org)
Oct 21 10:10:59 lima-finch chronyd[665]: Selected source <IP_ADDRESS> (2.fedora.pool.ntp.org)
Oct 21 10:11:04 lima-finch chronyd[665]: Can't synchronise: no majority

Additional context
Replacing the fedora pool in /etc/chrony.conf with: pool time.aws.com iburst and running sudo service chronyd force-reload might fix the problem (it certainly synced the clock this morning, but only time will tell).

(it certainly synced the clock this morning, but only time will tell)
😄

Interesting, it looks like there is some history with this issue reported before. re: https://github.com/runfinch/finch/issues/248
@nicdoye, can you confirm the virtual machine type the issue was observed?

Yep, I have vmType: vz

Thanks, looking to create a repro on my side to see if can get any new information on this issue.

Running container for 72 hours with no drift between host, VM, and container clock.
Details:

finch v1.2.3
VM type: vz
macOS 14.6.1
2.6 GHz 6-core Intel Core i7

@nicdoye, just to verify when updating the VM type from qemu to vz, it would require a VM re-initialization. i.e. finch vm stop && finch vm remove && finch vm init
Otherwise, perhaps can you share your Finch, macOS, and architecture you are experiencing the issue?

I don't think I've ever bothered with the qemu type. I did remove and init the vm a couple of weeks ago. I first reported this back in February (https://github.com/runfinch/finch/issues/837) on 1.1.2 and now I've seen it on 1.4.0 and 1.4.1

finch v1.4.1
VM type: vz
macOS 15.0.1
M1 Pro 16" 2021

A second M1 Mac runs Podman, which is also based on fedora (39 not 40) on Apple Virtualisation, and while it's not used as much as my work Mac, I've not seen the same issue there. It seems to have the same/similar chronyd setup as the finch one, here.
I'll fire finch up there, and see if I can get it to happen. Maybe our MDM, or using cap SYS_ADMIN or something is causing the issue. 🤷🏻♂


BCDB: OperationalError, no such table

I'm not exactly sure how this happened, but It seems anytime I'm trying to save something in BDB I get this error now.
I didn't do anything special. just logged into my jsx container today and failed to start the 3bot server with this error.
*****Remote Exception*****
--SOURCE: GEDIS SERVER <IP_ADDRESS>:8901 SOURCE METHOD: default.package_manager.package_add-20--
Fri 15 09:56:27 nts/peewee/peewee.py -3069 - execute_sql                        : EXCEPTION:
    OperationalError('no such table: index_jumpscale_threebot_package_1_26',)
--TRACEBACK------------------
/sandbox/lib/jumpscale/Jumpscale/servers/gedis/handlers.py in _handle_request
    407    self._log_debug("params cmd %s %s" % (params_list, params_dict))
/sandbox/lib/jumpscale/Jumpscale/servers/threebot/base_actors/package_manager.py in package_add
    70    package.prepare()
/sandbox/lib/jumpscale/Jumpscale/core/BASECLASSES/JSConfigBCDB.py in save
    123    self.save_()
/sandbox/lib/jumpscale/Jumpscale/core/BASECLASSES/JSConfigBCDB.py in save_
    135    self._data.save()
/sandbox/lib/jumpscale/Jumpscale/data/schema/JSXObject.py in save
    176    r = self._model.find(**args_search)
/sandbox/lib/jumpscale/Jumpscale/data/bcdb/BCDBModel.py in find
    629    for id in self.find_ids(nid=nid, **kwargs):
/sandbox/lib/jumpscale/Jumpscale/data/bcdb/BCDBModel.py in find_ids
    602    cursor = self._find_query(nid, **kwargs)
/sandbox/lib/jumpscale/Jumpscale/data/bcdb/BCDBModel.py in _find_query
    584    return self.query_model([field], whereclause, values)
/sandbox/lib/jumpscale/Jumpscale/data/bcdb/BCDBModel.py in query_model
    591    return self.index.db.execute_sql(query, values)
/sandbox/lib/jumpscale/Jumpscale/clients/peewee/peewee.py in execute_sql
    3076    self.commit()
/sandbox/lib/jumpscale/Jumpscale/clients/peewee/peewee.py in __exit__
    2831    reraise(new_type, new_type(*exc_args), traceback)
/sandbox/lib/jumpscale/Jumpscale/clients/peewee/peewee.py in reraise
    196    raise value.with_traceback(tb)
/sandbox/lib/jumpscale/Jumpscale/clients/peewee/peewee.py in execute_sql
    3069    cursor.execute(sql, params or ())
-----------------------------

Fri 15 09:56:27 t_package_manager.py - 274 - handle_error                       : EXCEPTION:
    GEDIS SERVER <IP_ADDRESS>:8901 SOURCE METHOD: default.package_manager.package_add
--TRACEBACK------------------
<string> in <module>
    1
/sandbox/lib/jumpscale/Jumpscale/servers/threebot/ThreeBotServersFactory.py in local_start_default
    91    path="/sandbox/code/github/threefoldtech/jumpscaleX_threebot/ThreeBotPackages/zerobot/webplatform/"
/sandbox/var/codegen/gedis/threebot/client/default_package_manager.py in package_add
    44    self.handle_error(e,1,cmd_name=cmd_name)
/sandbox/var/codegen/gedis/threebot/client/default_package_manager.py in handle_error
    274    raise j.exceptions.RemoteException(message=msg, data=logdict, exception=e)
-----------------------------

will be resolved when doing our fixes in
threefoldtech/home#398


Unable to see Azure Cache for Redis servers in Azure US Government

After I log into my Azure US Government account, I'm able to see the following resource types without any issue:

Azure Functions.
Azure Resource Groups (and inner resources).

For Azure Cache for Redis, under all the subscriptions I have access to, I see the following message:

Error: Entry not found in cache.

Hi @nturinski, I tried to reload the window and it didn't solve this issue.
Here is a preview of what I see:

I forgot to mention it in the initial message, but I did do the whole sign out/sign in approach without any success.

Okay, thanks for troubleshooting with us.  We'll have to investigate exactly what's going on here.
Last thing, did this subscription ever work or is this a new issue?

It never worked for Redis resources, but I'm able to see other resource types for the same subscriptions. I have access to roughly 4 different Azure subscriptions in that cloud as well with an identical buggy behavior.

Actually, I just noticed that your view pane says "Caches" on it. This extension is managed on this repo https://github.com/microsoft/vscode-azurecache.
Could you file the issue there?  Though unfortunately, it doesn't seem like it's very active 😞

Closing as external bug.


gerrit-174827-34

Current coverage is 62.31% (diff: 90.16%)

Merging #113 into master will decrease coverage by 0.38%

@@             master       #113   diff @@
==========================================
  Files           265        272      +7
  Lines         50105      52144   +2039
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          31416      32494   +1078
- Misses        18689      19650    +961
  Partials          0          0

Powered by Codecov. Last update f8f8814...6a5c99a


Update as upstream, add new translation and some fixes

Update as upstream
Standardize docs
Improve grammar
Fix typos
Add BrowserWindow translation (not yet completed)

Thanks!


Server Notify - VPSSignature mismatch

I think I've found a strange bug with SagePay\Server - it is consistently failing to process the transaction for me even though all of the address/CVV checks are passing without issue.
I have followed the documentation to the letter but am getting different values returned from the getSignature() and buildSignature() methods even though I am setting the SecurityKey properly, so the isValid() call is always failing.
I've consulted the Sage Pay documentation and examined the code, and spent a long time checking through the ordering of the $signatureData array in the buildSignature() method in ServerNotifyTrait.  I am completely at a loss as to why these signatures aren't matching, and I've tried disabling CSRF protection as well but it's made no difference.
Example request data:
{"VPSProtocol":"3.00","TxType":"PAYMENT","VendorTxCode":"98","VPSTxId":"{1D6F2F07-5467-66CD-2DD0-B4832F0D5C1B}","Status":"OK","StatusDetail":"0000 : The Authorisation was Successful.","TxAuthNo":"2878376","AVSCV2":"ALL MATCH","AddressResult":"MATCHED","PostCodeResult":"MATCHED","CV2Result":"MATCHED","GiftAid":"0","3DSecureStatus":"ERROR","CardType":"AMEX","Last4Digits":"0004","VPSSignature":"CABA1A286D831D13A909CE548F7EDA90","DeclineCode":"00","ExpiryDate":"1222","BankAuthCode":"99972"}
Calls to functions and their results:
getSignature(): caba1a286d831d13a909ce548f7eda90
buildSignature(): 6d1b3f53a2d708b6ac512b290e2f6eff
I would really appreciate any advice anyone can provide, as I am losing the will to live with this now.

Just a note about VendorTxCode/transactionId - they must be unique for all time. If you are using a simple incrementing integer, there is a danger that you will hit the same IDs again, especially if you are rebuilding databases when testing. I would recommend using a UUID.

The notification handler is the hardest part of Sagepay Server, so we feel your frustration. The problem is, you can't see what Sagepay sees. One thing it is worth doing, is trying Postman or something similar to send a POST to your endpoint. Often that can thrown up some extra characters or errors that you could not see any other way.
On the plus side, it looks like Sagepay is sending you the result, and the authorisation is being accepted, so that is a good start. I suspect it is something simple in the notification handler. Here are the steps:
Set up your notification handler:
$gateway = OmniPay::create('SagePay\Server')->initialize([
    'vendor' => 'foobar',
    'testMode' => true,
]);

$serverRequest = $gateway->acceptNotification();

Get the transaction ID that the notification was sent on behalf of:
$transactionId = $serverRequest->getTransactionId();

Now you need to look up the transactionReference that you previously saved in the database (must be stored on the server, since this server request has no access to your front-end session). It will be a JSON string and look something like this:

{"SecurityKey":"YHCGXTXF6U","VPSTxId":"{5E293E3D-E9C6-6906-0841-822B1266256A}","VendorTxCode":"phpne-demo-39952230"}

This along with everything that is POSTed to you is enough to check the signature is valid. Set that up:
$serverRequest->setTransactionReference($transactionReference);

This will then tell you whether the signature is valid:
$serverRequest->isValid();

Are you getting that far?  You probably want to be in more control of the way the application exits, and there are details in the documentation, but this crude method will get things working quickly for you while developing.

On that transactionReference, you happened to save all the parts of it individually, and want to pass them all in one-by-one, you can do that.
$serverRequest->setSecurityKey("YHCGXTXF6U");
$serverRequest->setVPSTxId("{5E293E3D-E9C6-6906-0841-822B1266256A}"); // Possibly without curly barackets.

That's an option that may help you debug.

Thank you for the quick responses @judgej - I have been doing more testing today but still haven't managed to resolve it.  I'm using version 3.2.3 by the way.
I've improved the VendorTxCode to ensure it's always unique now, so thank you for the heads up on that potential future issue!
So this is now what's being sent (sensitive info replaced):
Array (
    [VPSProtocol] => 3.00
    [TxType] => PAYMENT
    [Vendor] => vendor1
    [AccountType] => E
    [Description] => Order 9582
    [Amount] => 65.45
    [Currency] => GBP
    [VendorData] =>
    [VendorTxCode] => 9582-108-191122012240
    [ClientIPAddress] =>
    [ApplyAVSCV2] => 0
    [Apply3DSecure] => 0
    [BillingFirstnames] => John
    [BillingSurname] => Doe
    [BillingAddress1] => 88
    [BillingAddress2] =>
    [BillingCity] => Conwy
    [BillingPostCode] => 412
    [BillingState] =>
    [BillingCountry] => GB
    [BillingPhone] => 01234 567890
    [DeliveryFirstnames] => John
    [DeliverySurname] => Doe
    [DeliveryAddress1] => 88
    [DeliveryAddress2] =>
    [DeliveryCity] => Conwy
    [DeliveryPostCode] => 412
    [DeliveryState] =>
    [DeliveryCountry] => GB
    [DeliveryPhone] => 01234 567890
    [CustomerEMail] =><EMAIL_ADDRESS>    [AllowGiftAid] => 0
    [NotificationURL] => https://website.com/basket/payment/notify
)

All looks pretty good to me, apart from maybe the ClientIPAddress being missing. I'm specifying that as part of the Card object but it doesn't seem to transfer into the request here.
What's coming back:
{"VPSProtocol":"3.00","TxType":"PAYMENT","VendorTxCode":"9582-108-191122012240",
"VPSTxId":"{24B7369E-F6AD-CD00-0B2F-8ED465CA5E85}","Status":"OK",
"StatusDetail":"0000 : The Authorisation was Successful.","TxAuthNo":"2891347",
"AVSCV2":"ALL MATCH","AddressResult":"MATCHED","PostCodeResult":"MATCHED",
"CV2Result":"MATCHED","GiftAid":"0","3DSecureStatus":"OK",
"CAVV":"bXFTYm9tbXF4bm42Z2hKbm9aRWo=","CardType":"VISA","Last4Digits":"0006",
"VPSSignature":"6574F9F97BEED87460CC54F57365A785","DeclineCode":"00",
"ExpiryDate":"1222","BankAuthCode":"999777"}

I'm storing the json TransactionReference in the database before redirecting to Sage Pay:
{"SecurityKey":"PXKZZHIPEZ","VPSTxId":"{24B7369E-F6AD-CD00-0B2F-8ED465CA5E85}","VendorTxCode":"9582-108-191122012240"}

Then in the notifcation handler doing:
$notifyRequest = $gateway->acceptNotification();
$notifyRequest->setTransactionReference($transactionRefFromDb);

And still, when I run $notifyRequest->isValid() it's still returning false because apparently the signatures don't match.  It is successfully returning me to the failUrl, but obviously should be taking me to success...
if (! $notifyRequest->isValid()) {
    $notifyRequest->invalid($omnipay->getConfigUrl('fail'), 'Signature not valid');
} else {
    $notifyRequest->confirm($omnipay->getConfigUrl('success'));
}

Sorry for the huge essay, but I am completely at a loss as to what is going on here.  Is the Token part required (from the bottom of the readme)?  I don't need to store the card details for more than one transaction so didn't think I'd need this.

I've finally figured it out thanks to Postman, a tool which I'd heard of but never used before.  Firstly, I apologise for any wasted time, and secondly thank you very much for taking the time to help me out.  You've helped me to improve the integration and also given me a new tool in my debugging arsenal.
For anyone else experiencing the same problem, my issue was that within the notification handler the vendor field hadn't been set properly within the gateway's parameters before calling isValid().  So the SecurityKey was correct, but the vendor field was null, so the signature wasn't matching.  I just did a var_dump() on the notification and almost immediately saw that the vendor was missing so figured it was probably that.
Thank you @judgej for building and maintaining this repo.  I have to say this integration has been one of the more bizarre I've come across as far as payment gateways go, so it can't have been/be easy.  I hope you have a great weekend.

Don't apologise - nobody's time has been wasted :-)
Thank you for following up and providing details of the source of the problem. If there is anything that missing - even it just a snippet or two - from the documentation, then anything you have to add would be much appreciated. For me, I tend not to understand what is going on until I have a clear picture of the architecture in my head. I think sometimes it takes that key phrase to crystallise the image. But if not, don't worry, we have documentation in this issue.

To be honest the documentation already covers what my issue was, and had I followed it properly it would have worked from the start. My problem was I'd created a helper class to build and configure the gateways, then not called the initialize() function within that helper class to set up the proper parameters. I was so covinced that it was an issue with the SecurityKey that I wasn't looking elsewhere. I'd like to say lesson learned, but I'm sure something very similar will happen again next week 😆 Maybe I'm just in the wrong profession!

Hi everyone,
I have same problem where the signature returned from sagepay is not matching with md5 hash im generating in c#.
this is the format of my hash string my secrete key which i have stored is okay, status is OK, vendorTxCode is unique, vendor name in lower case. im stuck in this issue for last 5 days . your help will be highly appreciated.
String format:
<EMAIL_ADDRESS>String Result:
{79334006-A2A8-9B15-CA1D-A5B59262661B}4FA92744-DD26-47B6-B759-7D111A94DD99OK11416492huddersfieldtaALL MATCHH7RVJYGRFIMATCHEDMATCHEDMATCHED0VISA0006001225999777


Run continuous batch size LLM benchmark using the python script

Have you run this at least once manually to confirm it works for experiment 3?

Yes


Will project be maintained?

Will project be maintained? Can i use it in production?

Looks like no. Why create a project simply to abandon it without at least doing the community the courtesy of taking it down? MEAN stack has been idle for 2 years now. I wish I knew why these projects keep getting abandoned.

Talk is cheap, commits speak volumes. Use in production if you're willing to maintain it yourself.

@pgrekovich I wouldn't recommend using this boilerplate for a new project. This boilerplate is outdated and contains many issues that could only be fixed with a deep rewrite.

@jgautheron thanks mate!


[WIP][Parser][SR-1052] Added @discardableResult attribute parsing.

What's in this pull request?
Part 1 for SR-1052: Add parsing of new @discardableResult attribute.
Future commits will add use in diagnostic generation, stdlib, etc.
Resolved bug number: Part 1 of SR-1052

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@lattner It's been a week. Can this PR get reviewed? I'd like to finish SR-1052, but everything else builds on this PR.

This looks great, sorry for the delay!


feat(linter): add more "ban-ts-comment" test cases.

https://github.com/oxc-project/oxc/issues/2934

name
passed
failed

ts-expect-error
14
20

ts-ignore
15
20

ts-nocheck
11
9

ts-check
11
8

total
51
57

I have removed test cases related emoji. Please review the code again, Thank you. @Boshen @mysteryven


Delete temp files after copying them while shuffling

Hi,
this is just a proposal, during shuffling with bullet utils, to delete each temp file immediately after copying it into the target output file.
In this way half of the disk space will be necessary.
I faced this issue, because my disk is not big enough for the data I am processing.
As a solution for me I concatenate the files into the tmp directory manually after deleting the input file.
I think this could be a nice improvement.
Best regards,
Krasimir

@bagaturchess this is not how the shuffling utility works, the files are not concatenated sequentially, but interleaved, so are all in use right up to the end.

Thanks for the answer. Than my proposal is not valid. :-)
I am closing the issue.
Best regards, Krasimir


Convert docstrings to numpydoc

Is your feature request related to a problem? Please describe.
Since in the near future a large number of new people will begin to work on the project, it is necessary to agree on a standard for docstrings. And introduce some kind of automatic checks.
For my part, I would like to suggest nympydoc package that we can use in github actions CI.

So far, the approach has been:
def some_method(self, arg1, arg2):
	"""The one-line description of the method.

	Args:
		arg1: Description
		arg2: Description

	Returns:
		What this returns (if anything)
	"""
	# Begin implementation
	return something

I am open to new suggestions as long as we are standardized. numpydoc seems reasonable, interested to hear other opinions.

Looks like numpydoc has a standard extension in sphinx, which will make the cross-reference easier.

numpydoc is a good testable standard. Let's go with that. We can install pydocstyle and test from bash with
pydocstyle --convention=numpy modin/pandas

@anmyachev is this still relevant?

@mvashishtha It is necessary to make an analysis of which files are not yet checked in the CI for compliance with numpydoc  standard. If there are such files, then we need to check them too.

Most of the code uses this format; when we find files where the format is different, we will create separate tasks and convert the corresponding files to the new format.


[OSS-Fuzz] Introspector build failures since using new PM

Since merge of https://github.com/google/oss-fuzz/pull/7788, around 59 projects fail to build.
As I checked some including json, valijson, wabt, wolfssl, and znc, the error message is:
/usr/bin/ld.gold: fatal error: LLVM gold plugin: <unknown>:0: Undefined temporary symbol .LtmpXXXXX

This is a duplicate of https://github.com/ossf/fuzz-introspector/issues/48

When the issue was posted the build was based on the new pass manager: Marco built the project using the patches as they were in oss_fuzz_integration/oss-fuzz-patches.diff at the time, which did not include -flegacy-... flags (https://github.com/ossf/fuzz-introspector/blob/b9e2761b2533bea544b540e55953a3900ff1a887/oss_fuzz_integration/oss-fuzz-patches.diff#L9), meaning the new PM was used at that point. We only switched back to the old pass manager in the local fuzz-introspector set up around this PR https://github.com/ossf/fuzz-introspector/pull/141

Thanks,

meaning the new PM was used at that point.

So it means we ack some projects will have such issue with new PM.

+1 on sticking to new PM.
Projects can enable old PM on demand if they find it necessary to build Fuzz Introspector.

Should be fixed once https://github.com/google/oss-fuzz/pull/7828 is merged.

Should be fixed once google/oss-fuzz#7828 is merged.

This fixed the majority. Still 18 are failing, will open another issue as the error is not about this issue anymore.


rename api.proto files in pkg/hooks to api_*.proto due to protobuf wa…z

What this PR does / why we need it:
Simple rename to avoid future errors
Which issue(s) this PR fixes
fixes #4881
Special notes for your reviewer:
Wanted to rename it directly to v1alpha1.proto, info.proto etc, but name info.proto is already taken. I decided to use api_*.proto convention.
Release note:

NONE

/ok-to-test

/lgtm
/approve
Ran make generate && make, everything in sync and the errors are gone. Tanks :+1:

/retest


[DO NOT MERGE][ACR] az acr repository metadata show/update/delete: Add repository metadata API exposure.

Description
Enables listing of metadata keys, downloading metadata by key to file, updating metadata by key from file, and deleting metadata by key.
Testing Guide
Assuming your ACR registry is named registryName and it contains a repository named repositoryName:

List current metadata:

az acr repository metadata show --name registryName --repository repositoryName

Create a text file testFile.in in the current directory with some content. Upload it as metadata under the key testKey:

echo "test file contents" > testFile.in
az acr repository metadata update --name registryName --repository repositoryName --key testKey --file testFile.in

See that the repository now has metadata with key testKey:

az acr repository metadata show --name registryName --repository repositoryName

Download the metadata under key testKey to file testFile.out in the current directory; it should have the same content as testFile.in:

az acr repository metadata show --name registryName --repository repositoryName --key testKey --file testFile.out
cat testFile.out

Delete metadata under key testKey:

az acr repository metadata delete --name registryName --repository repositoryName --key testKey

See that the repository no longer has metadata with key testKey:

az acr repository metadata show --name registryName --repository repositoryName

History Notes
[ACR] az acr repository metadata show/update/delete: Add repository metadata API exposure.

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR title and description has followed the guideline in Submitting Pull Requests.

[X] I adhere to the Command Guidelines.

ACR

ACR

Moved to https://github.com/AzureCR/azure-cli/pull/320.


Autoware on 32bit or ARM systems

We are trying to launch Autoware on Nvidia Jetson TK1, which is 32bit and Arm architecture.
Most of drivers and libs are installed pretty good, however we stuck on "glviewer" missing while ./catkin_make_release
We found out this gist: "Autoware patch for building on Ubuntu 14.04 32bit " https://gist.github.com/syohex/dbbe8b11f8340125e678 , but were unable to patch latest build files. Reason might be,that patch is 6 month old, but cmake files are newer.
Any workarounds on that?

Most of drivers and libs are installed pretty good, however we stuck on "glviewer" missing while ./catkin_make_release

glviewer is built on only x86 architecture now(Its condition branch is here). I tried to build it on Jetson TK1 one and half years ago but I could not built it.

It could be good if somebody could find any workaround on that.
All Nvidia automotive prototyping hardware like Jetson TK1, TX1 and Drive PX are based on ARM. Could be great hardware to test and run Autoware.

After look into the build process, there I found the issue in CMakeLists.txt
within Autoware/ros/src/util/packages/RobotSDK/glviewer/CMakeLists.txt
if("${ARCHITECTURE}" STREQUAL "x86_64" OR "${ARCHITECTURE}" STREQUAL "i686")
include_directories(
...
change into
if("${ARCHITECTURE}" STREQUAL "x86_64" OR "${ARCHITECTURE}" STREQUAL "i686" OR "${ARCHITECTURE}" MATCHES "^arm")
But I still encounter some other issue.

Is there any resolution to this? I am stuck on the same thing, with an ARM board.

when i try to run this project in nvidia jetson TX2,I also meet this question.how to solve this problem?

I send the problem to autoware@googlegroups.com,and I got the answer" "
`It is necessary to change CMakeFiles.txt of glviewer package.
Please refer to the URL below:
https://github.com/CPFL/Autoware/blob/a0bbcad24f6b648380f63fbe8e9d76b9b2dc0798/ros/src/util/packages/RobotSDK/glviewer/CMakeLists.txt#L40
Thanks.`

hi can anybody help me regarding installation steps to be followed to install Autoware on NVIDIA Jetson TX1


 Thanks. Cheers.

Why is the formatting inconsistent within the same file?

This is not even intended to impose an opinion on the style of whitespace (which is also pretty bad).
But rather an attempt to contribute by fixing the formatting errors caused by negligent contributors/moderators. Are you really asking me to "send my suggestions to the forum" for that?
Regards


Update bower to support angular 1.6.

This may be a little preemptive, but I was getting a head start updating our app to the 1.6 release candidate and I haven't had any issues yet with this library supporting it.
Is there a chance we can get the bower file updated to support this?

Good idea, @joewolschon. I set the dependencies so that all future 1.x releases will be supported. Available in the latest release, 0.4.8.

Sweet, @cvn I meant to suggest that.
Thanks so much for the quick update and release!


LNK1104 for uWindowCapture.def

I believe this is me not knowing how to properly configure my project, but I am receiving this error when trying to compile from main branch. Can anyone help me?
Project path doesn't contain spaces.
uWindowCapture.def is on uWindowsCapture project root structure.

Drag and drop to visual studio creates a symbolic link, which was not resolving correctly. Copying the actual file solved the problem.

just wondering, why not just place this file under the project folder in this case?


Move feature matrix calculation helper functions to separate file

calculate_feature_matrix.py has almost 800 lines of code now. This pull requests keeps calculate_feature_matrix, calculate_chunk, approximate_features, linear_calculate_chunks, and parallel_calculate_chunks in the file but moves the other functions into a separate file.  This PR also moves the distributed client and cluster creation into a new helper function and adds a test case for the new helper function.

Codecov Report

Merging #177 into master will increase coverage by 0.09%.
The diff coverage is 95.03%.

@@            Coverage Diff             @@
##           master     #177      +/-   ##
==========================================
+ Coverage   93.13%   93.23%   +0.09%
==========================================
  Files          69       70       +1
  Lines        7688     7712      +24
==========================================
+ Hits         7160     7190      +30
+ Misses        528      522       -6

Impacted Files
Coverage Δ

...computational_backends/calculate_feature_matrix.py
96.98% <100%> (+2.78%)
:arrow_up:

featuretools/computational_backends/api.py
100% <100%> (ø)
:arrow_up:

...utational_backend/test_calculate_feature_matrix.py
99.62% <100%> (ø)
:arrow_up:

featuretools/computational_backends/utils.py
94.36% <94.36%> (ø)

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 f224e22...8ca5dda. Read the comment docs.

Looks good!


bug: title/description of toast hardly visible in darkmode

Description
When using the default values after running npx shadcn-ui@latest init and npx shadcn-ui@latest add toast, the title and description of a toast are barely visible in darkmode:

Only after manually adding dark:text-slate-100 (in my case at least with the default values) to both ToastTitle and ToastDescription is it possible to read the toast:

Expected behavior
The text should be visible in darkmode without any additional changes
I'd be happy to help with a PR!

Good @DerTimonius day how is it going ? just want seek help the toast in mine are not working how did you install that properly?
i follow this unfortunately not work in my machine https://ui.shadcn.com/docs/components/toast
thanks

@1danii did you use CSS variables? in the project where I encountered the issue I don't. I only followed the installation (for Astro in the case above) without making additional changes to the tw config

@1danii  do you have any reference for that installation ? unfortunately not work on me
i use also version 13.4.12
here's the issue i landed when i put this components in my layout
Error: Toast must be used within ToastProvider next js
i follow both cli and manual way of installation but no luck
https://ui.shadcn.com/docs/components/toast

what i did was i create a components i follow the docs and import it in my layout

then in my toast sample component i try to trigger that onclick

and did not appear any toast message

Still not working


Fix inner loop solution test runs

The test infra change did not account for the fact that the test app has a different name when it is build from the inner loop solution (MuxControlsTestApp vs MuxControlsInnerLoopTestApp). This change updates the code to check which solutions we are building from.

@Felix-Dev FYI.

/azp run

@StephenLPeters Just checked out this PR on my local machine and I could successfully run an interaction test in the innerloop again. Thanks for the fix 🎉

/azp run

/azp run

/azp run

/azp run

I pulled this PR and it did work, but it still installs the MuxControlsTestApp and uninstalls the MuxControlInnerLoopTestApp. I need to manually install the app everytime before running the inner loop tests.

/azp run

/azp run


Latest ElasticSearch 7.3.1 Version

Hi,
Thanks for creating this appender it is exactly what I am looking for. However, I downloaded and built the source and tried to use this and I get a null pointer exception when the appender tries to initialize the logger. I noticed that I was trying to use the appender with the latest ElasticSearch and saw that the RestClient was slightly different in its implementation. I updated the code locally to test if I could get it to work and updated the version of ElasticSearch in the POM but still, I get the same issue. The error I get is below:
Caused by: java.lang.NullPointerException
	at org.apache.commons.logging.LogFactory$Log4jLog.<init>(LogFactory.java:204)
	at org.apache.commons.logging.LogFactory$Log4jDelegate.createLog(LogFactory.java:166)
	at org.apache.commons.logging.LogFactory.getLog(LogFactory.java:109)
	at org.apache.commons.logging.LogFactory.getLog(LogFactory.java:99)
	at org.elasticsearch.client.RestClient.<clinit>(RestClient.java:97)

My appender settings are as follows:
<?xml version="1.0" encoding="UTF-8"?>
<Configuration status="DEBUG" strict="true" name="ElasticSearchAppender"
               packages="com.github.magrossi.log4j2.elasticsearch">
    <Appenders>
        <!-- Sends logs immediately -->
        <Appender type="ElasticSearch"
        name="ELASTIC"
        maxBulkSize="0"
        maxDelayTime="0"
        esIndex="process-api-log-"
        esType="logs"
        dateFormat="yyyyMMdd">
        <!-- ElasticSearch credentials, if required
        user="${env:myEsUser}"
        password="${env:myEsPassword}">-->
        <!-- List of nodes in your ElasticSearch cluster -->
            <Host type="HttpAddress" scheme="http" host="localhost" port="9200"/>
        </Appender>
        <RollingFile name="LogToRollingFile" fileName="logs/process-api.log"
                     filePattern="logs/$${date:yyyy-MM}/process-api-%d{MM-dd-yyyy}-%i.log.gz">
            <PatternLayout>
                <Pattern>%d %p %c{1.} [%t] %m%n</Pattern>
            </PatternLayout>
            <Policies>
                <TimeBasedTriggeringPolicy />
                <SizeBasedTriggeringPolicy size="10 MB"/>
            </Policies>
        </RollingFile>
    </Appenders>
    <Loggers>
        <Logger name="PROCESS_API_LOGGER" level="debug" additivity="false">
            <AppenderRef ref="LogToRollingFile"/>
            <AppenderRef ref="ELASTIC"/>
        </Logger>
        <Root level="ERROR">
            <AppenderRef ref="LogToRollingFile"/>
        </Root>
    </Loggers>
</Configuration>

Should the current version work with the latest ES? Its also worth noting that I do not have a password on my local ES instance I am testing with. Maybe that makes a difference or not I am not sure.
Thanks,

It seems like they have removed type in ElasticSearch 7, so I don't think this is compatible. You would need to update the Gradle ES client dependency and remove the type from the code. I'm happy to accept a PR if you'd like to implement this.


Apache Avro file format not supported

the Avro file format is not supported.
"
elif self.input_format in ["json", "csv", "tsv", "tsv.gz", "parquet"]:
"

Do you know of any image url dataset that in avro ?
If yes we could definitely bring support for this format


iOS: not working

Hi,
if I'm using the iOS export, the parsing takes 1 second and reports: "Bei Begegnungsüberprüfungen wurden keine Treffer mit positiv getester Person gefunden." (the CWA reports 1 encounter). If I'm reformatting the file manually (deleting everything before "[{"Timestamp"" and after ""}]") the processing takes for a long time without any result). Am I missing a step?
TIA

Hi,
are you running the latest iOS version (update from this week)? This sounds like an iOS Bug that has been fixed in the latest version, see https://github.com/corona-warn-app/cwa-app-ios/issues/1106.

iOS 14.0.1, will update to 14.1

Unfortunately no difference after updating to iOS 14.1, CWA 1.5.1(1)

I think the update only takes effect after the next check of CWA, so you need to wait for eventually another 24 hours. Please note that there is another bug in parsing the log file of iOS 14.1 since apple changed the way file hashes are calculated. I intend to fix it today/tomrrow, see also #5.

A workaround for #5 is implemented and everything should work as supposed to work.

A workaround for #5 is implemented and everything should work as supposed to work.

It's working! Excellent. Thanks for your weekend time 👍


integrations: fastai speed up tests

[ ] ❗ I have followed the Contributing to DVCLive guide.

[ ] 📖 If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.

Thank you for the contribution - we'll try to review it as soon as possible. 🙏
If we provide simpler task, we can save some time on tests.
In this case, on my personal machine - from 88 s to 5 s.

Codecov Report
Base: 54.35% // Head: 54.44% // Increases project coverage by +0.09% :tada:

Coverage data is based on head (36939d4) compared to base (167fe68).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #301      +/-   ##
==========================================
+ Coverage   54.35%   54.44%   +0.09%
==========================================
  Files          33       33
  Lines        1597     1598       +1
  Branches      172      152      -20
==========================================
+ Hits          868      870       +2
  Misses        724      724
+ Partials        5        4       -1

Impacted Files
Coverage Δ

tests/test_fastai.py
100.00% <100.00%> (ø)

src/dvclive/dvc.py
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Make http-sync optional

Could we get http-sync changed to an optionalDependency? This will simplify and speed up the setup process for those of us who don't need the github hooks. The github hook could detect the missing package and print a helpful message.
Noticed that there may be some changes to http-sync in store for io.js/0.12 compatibility and if that will solve this problem then great.

http-sync will likely be removed when we switch to io.js

execSync will also be removed, further speeding up installs.

@TheAxeMan301 I'm working on converting our CLI to use all async utilities, so this should be able to be removed before switching to iojs


Simplify code-gen?

Little usability request here...
For those of us that have to integrate SPIRV-Tools into different build systems (really common problem on bigger projects!), having complicated code-gen steps is a tricky and fragile thing to re-create and maintain support for.
I managed to get things working, but I had to call like 4 different python scripts with a ton of arguments without clear distinction between which ones are inputs vs outputs. And setting up the tracking of dependencies was also pretty hairy since some commands can generate multiple outputs.
Other Khronos repositories (e.g. OpenGL-Registry, SPIRV-Headers, etc) pre-generate said headers to make life easier for the common end-user case, so in an ideal world since none of these files appear to be machine-dependent they would just be committed and updated only when necessary.
But possibly at least make the code-gen a bit easier to do outside of the cmake environment (simple clear syntax for inputs vs single-output, etc, e.g. gen_header.py -o some.inc some.json).

You can get the generated SPIR-V headers from https://github.com/KhronosGroup/Spirv-headers is that what you're looking for?

I would love if SPIRV-Headers included everything, but it doesn't.
build-version.inc
extension_enum.inc
enum_string_mapping.inc
DebugInfo.h
core.insts-{SpirVConfigName}.inc
glsl.std.450.insts.inc
opencl.std.insts.inc
operand.kinds-{SpirVConfigName}.inc
generators.inc

Right, but those .inc files reflect data representations that are internal to SPIRV-Tools.
I've rev'd those multiple times in the past and actually considered yet another representation change last week.  Those python scripts are.... dense and confusing as well.

We won't be doing anything about this.


Do I have to run NEURON to send animation data?

I have a large scale network simulation and I'd like to visually represent a small subsection; There is not really any documentation so I'm struggling. From the videos it seems that I should actually be simulating in NEURON for BlenderNEURON to pick up on the signal. This online approach is completely unfeasible; I have the recorded simulation data in an HDF5 file. Is it possible to feed this to BlenderNEURON in an offline fashion?

Hi again,
Interesting use case. BN was initially designed for the "coupled to NEURON" use case, but I wanted to leave it relatively easy to add adapters for other simulators, or in your use case, pre-recorded simulation data.
You should be able to hack the methods of the BlenderNode class
In Blender's Python console, it's instantiated in the bpy.types.Object.BlenderNEURON_node variable.

Most likely, you could modify the import_groups_from_neuron method to read the HDF5 file instead of making an RCP call to NEURON.
In the method, the call to from_full_NEURON_group expects a JSON formatted version of morphology/activity for the sections.
If you convert your HDF5 data to the JSON format that method expects using Blender's python, then you should be able to perform the task all within Blender.
An alternative approach could be to replace the NEURON RCP node code (the methods listed here ) with code that reads your HDF5 file and formats the morphology/animation data in the same JSON format.
I can document what the JSON format should look like, but I would suggest placing a few print statements in the above import_groups_from_neuron method (with a few-compartment cell + few ms of activity) to get a quick sense.

Hi again,
Interesting use case. BN was initially designed for the "coupled to NEURON" use case, but I wanted to leave it relatively easy to add adapters for other simulators, or in your use case, pre-recorded simulation data.
You should be able to hack the methods of the BlenderNode class
In Blender's Python console, it's instantiated in the bpy.types.Object.BlenderNEURON_node variable.

Most likely, you could modify the import_groups_from_neuron method to read the HDF5 file instead of making an RCP call to NEURON.
In the method, the call to from_full_NEURON_group expects a JSON formatted version of morphology/activity for the sections.
If you convert your HDF5 data to the JSON format that method expects using Blender's python, then you should be able to perform the task all within Blender.
An alternative approach could be to replace the NEURON RCP node code (the methods listed here ) with code that reads your HDF5 file and formats the morphology/animation data in the same JSON format.
I can document what the JSON format should look like, but I would suggest placing a few print statements in the above import_groups_from_neuron method (with a few-compartment cell + few ms of activity) to get a quick sense.


Make command line flags for check.sh and simple_check.sh actually work

For instance, bash continuous-integration/simple_check.sh --only=incremental-coverage does not perform the correct behavior of running pytest and then performing incremental coverage. This is also an issue at OpenFermion-Cirq.

Obsoleted by the creation of individual check scripts.


Maven Latest Spring-Boot dependencie breaks Spring Social

My app uses Spring Boot  and I am trying to use Spring Social to do the facebook log-in.
But I was only able to run the Getting Started · Accessing Facebook Data - Spring tutorial of yours using this
    <parent>
    <groupId>org.springframework.boot</groupId>
    <artifactId>spring-boot-starter-parent</artifactId>
    <version>1.2.3.RELEASE</version>
    </parent>

as confirmed in this SO question
but my app runs with this, which is the same version you guys mention in the tutorial.
    <parent>
    <groupId>org.springframework.boot</groupId>
    <artifactId>spring-boot-starter-parent</artifactId>
    <version>1.2.7.RELEASE</version>
    </parent>

What should I do?

You can see the error I am getting on this SO Question

So have you found a way around to bypass this error?
I dont mean downgrading to 1.2.3 (that is absurd, if they have a RELEASE version of boot, make it be compatible with everything else...) as I am using some stuff from the newer releases.

@AndreevskiAleksandar I have not. But I just checked the Spring Boot github page and it seems that there is a 1.3.0 RELEASE out there, Again, have not tested, but they might have overcome this, hence I also posted this issue there, but it seems that cross posting is not polite.

@cescoferraro I am on 1.3.0 RELEASE and still the same problem..

@AndreevskiAleksandar sad. I wish I could help, but kinda left this project aside for a while.

Doesn't look like this library is actively maintained.

I just tried with spring-social-facebook-2.0.3.RELEASE and spring-boot 1.3.1.BUILD-SNAPSHOT and Facebook.isAuthorized() still throws NullPointerException. It seems like it has something to do with this bean from FacebookAutoConfiguration in spring-boot-autoconfigure-1.3.1.BUILD.SNAPSHOT:
@Bean
		@ConditionalOnMissingBean(Facebook.class)
		@Scope(value = "request", proxyMode = ScopedProxyMode.INTERFACES)
		public Facebook facebook(ConnectionRepository repository) {
			Connection<Facebook> connection = repository
					.findPrimaryConnection(Facebook.class);
			return connection != null ? connection.getApi() : null;
		}

I'm not familiar with this type of bean scoping. For whatever reason the dynamic proxy target for the injected Facebook bean instance is null, causing the NullPointerException.


⚠ LPDM Handbook (hosted on Divio) has degraded performance

In 041ce6a, LPDM Handbook (hosted on Divio) (https://repohubdev.tk) experienced degraded performance:

HTTP code: 500
Response time: 5462 ms

Resolved: LPDM Handbook (hosted on Divio) performance has improved in fe749a3.


Context names containing "()" makes zsh autocompletion stop working

Having parentheses in context name may mess with autocompletion. Simple workaround, though,  just skip the parentheses. I haven't tried any other funky characters.
MacOS
kubectx 0.9.4 installed with brew
clean zsh
To reproduce:
> kubectx "(mycontext)"=.
Context "mycontext" renamed to "(mycontext)".

> kubectx <Tab>
(eval):1: missing end of string
(eval):1: missing end of string
(eval):1: missing end of string

> kubectx "mycontext(1)"=.
Context "(mycontext)" renamed to "mycontext(1)".

> kubectx <Tab>
eval):1: unknown file attribute: 1
eval):1: unknown file attribute: 1
eval):1: unknown file attribute: 1

> kubectx "()"=.
Context "mycontext(1)" renamed to "()".

> kubectx <Tab>
(eval):1: parse error near `()'
(eval):1: parse error near `()'
(eval):1: parse error near `()'

> kubectx mycontext=.
Context "()" renamed to "mycontext".

> kubectx <Tab>
-     mycontext

Not surprised. I suspect most people don't exercise the context names to this extent.
bash/zsh completion scripts are tough to work with. If you can find a fix, patches are appreciated!


Enable node version for specific tag

This commit:

allows for getting the node version of a tag by appending /<tag> to the url.
allows for getting the node version for scoped packages.
adds node version examples

Builds on #926. Resolves #723.

Hi, just wanted to follow up! Are you up for adding those tests?

I added some service tests for npm version in #1128. When that's merged I'll pick this up, add the tests, and merge it.

I wrote tests for this. Unfortunately the new code isn't working. It tries to hit https://registry.npmjs.org/%40stdlib%2Fstdlib which doesn't do anything.
Reopened as #1144.

Something must have changed with how the registry handles URL encoding. The following URL works for me:
https://registry.npmjs.org/@stdlib%2Fstdlib
which suggests to me that decoding on their end may we incomplete. When testing locally prior to submitting the PR, the URL you posted did return the desired package meta data.


TwitchAPI.Settings cannot be accessed

Hi, I'm trying to integrate the library into a Unity program and somehow I am unable to set the TwitchAPI.Settings as it is returning me an error, stating that an object reference is required for the non-static field, method, or property.
Could you let me know if there is a fix for this?

The TwitchApi isnt a static class anymore. it now needs an instance to be created to be used.
This was made to make it DI capable for the new .net core work flow.

I see. So how do I pass my client ID and oauth token in?

you can do that in the constructor like TwitchApi api = new TwitchApi(clientId, oauth);

or you can do it like you would do it in the old version once you have an instance

I see. I need to try it, because I did that, and I'm getting some errors. It may be unrelated. Thanks for the help!

You're welcome ;)

Sorry, one last question. Then how do you pass the access token over to the client instance?

Thank for your help Syzuna! I'm closing the thread.

Did you find an answer on the client question? wasnt home and didnt have time to answer on phone. Wanted to do it now but it looks like you found an answer already ^^

Yeah, the guide I was following was slightly outdated. So I explored a little and managed to get it to connect.

Glad you got it figured out :)


[BUG] Failed to decompress 'deflate' encoding: Error: incorrect header check

System info

Playwright Version: [v1.XX]
Operating System: [All, Windows 11, Ubuntu 20, macOS 13.2, etc.]
Browser: [All, Chromium, Firefox, WebKit]
Other info:

Source code

[ ] I provided exact source code that allows reproducing the issue locally.

Link to the GitHub repository with the repro
[https://github.com/your_profile/playwright_issue_title]
or
Test file (self-contained)
 RequestOptions requestOptions = buildCommonRequestOptions()
                .setHeader(HttpHeaders.CONTENT_TYPE, APPLICATION_JSON_UTF8_VALUE)
                //.setHeader(HttpHeaders.ACCEPT_ENCODING, "*")
                .setHeader(HttpHeaders.REFERER, irseBrowserConfig.getReferer())
                .setData(JacksonConverter.write(parameDO));

        APIResponse apiResponse = apiRequestContext.post(irseBrowserConfig.getDetailUrl(), requestOptions);

Steps

[Run the test]
[...]

Expected
[Describe expected behavior]
Actual
Failed to decompress 'deflate' encoding: Error: incorrect header check
[Describe actual behavior]

We need a reproducible example that we could run locally to make this report actionable. Please file a new issue with a self-contained repro and link it to this one.


SkyBlock Levels Display?

Is your suggestion related to a problem? Please describe in detail.

No

Describe your suggestion
A clear and concise description of what you want to happen.

Allow users to see the skyblock level (as of 0.15 SkyBlock Update) progression/display above the player skills tab

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Ex. [...] could be a problem, but [...]

No?

Additional context
Add any other context or screenshots about the suggestion here.

No?

Hey! Skyblock levels aren't currently in the Hypixel API. Levels are actually planned to be added to the Hypixel API in the future. We will do our best to incorporate it into SkyCrypt as soon as it becomes available. In the meantime, we appreciate your patience and understanding.


Transferring changes from master.

This update is to pull in selected changes from master that were not applied to devel.

I added back some of the tests from main.  Discovered a minor bug which was resolved.


Improve number fields behavior when formatted with a unit

The new NumberField component is awesome, but while using it with style: unit format options specified, I realized it's behavior diverges pretty significantly from the lovely behavior in in the react spectrum component it's based on. This PR brings it closer into line, though it's still not quite perfect.
To see the issue:

create a NumberField
specify the following format options:
attempt to type into the field:

a decimal number into the field
typos like alphabetical characters
anything into the unit segment

const formatOptions = {
    minimumFractionDigits: 0,
    maximumFractionDigits: 1,
    style: "unit",
    unit: "second",
    unitDisplay: "long"
};

Expected behavior:  The NumberField should normalize the user's input into a valid format
Actual behavior: The displayed value is polluted until submission, at which point it'll do it's best to parse what's there and renormalize.
This PR confirms that the user's changes are still parsable before committing them, without getting in the way of deleting the contents of the field entirely. It generally makes the unit behave like ineditable text.
Note that this may be able to completely replace the allowedInput regex prop (and related machinery) which doesn't have an analog in the react-spectrum NumberField, but I didn't want to make that assumption without checking in first.

Sorry, just noticed the lint issues. Hopefully all addressed, though I couldn't figure out how to run the reporter locally to test (pnpx biome ci didn't report the found issues)

You can run pnpm lint && pnpm format to format code. Then pnpm test to build and test.
If test works locally but not on here, it's falky and needs to rerun on github.

Thanks for the clarification! Not sure how I missed it first time around.

Following up to see if there's anything else I can do to help get this across the line.

Looks like after a re-run everything's green. Any remaining work for me here?

Hey all good! Almost done with #381 then I'll be able to merge.

Thanks for the PR! I'll try to replace allowedInput and only use this.

Awesome news, and congrats on the refactor! If there's anything I can help with feel free to @ping, Kobalte's been a huge help to me! :)


Added notice about dropdown printing

Adds a noticed about the dropdown tag printing the value when a template is rendered and suggests using the variable tag if needed.

Closing in favour of https://github.com/wintercms/docs/commit/a2f95e76000e3cd4a750f88248ebf4fded700efe


Created protocol factories and unit tests

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Adds a factory method which takes a  Settings.ctl type definition as an input parameter, and return a  object with the provided settings.
It also adds unit tests which verifies the functionality of these factory methods.
Why should this Pull Request be merged?
The protocol factory methods are needed by the Instrument Addon Scripting API.
The tests are necessary to validate the functionality of the factory methods.
What testing has been done?
Wrote unit tests, run them locally and they all passed.

I want us to strive for consistency, therefore all typedef inputs to Factories should be called using the same pattern "protocol_name Settings". I saw some of them being called "* PortSettings", please correct that.

Typo in UDP Factory -> name of typedef input is "UPD *"

For Convbox, typedef input must have the word "settings" starting with capital "S".

Current Command Code should not be part of the user visible settings of AK protocols.

For AK RS-xxx, I still see a "PortSettings" cluster, are we keeping that?

For AK RS-xxx, I still see a "PortSettings" cluster, are we keeping that?

It's the same in the current implementation of "RS-xxx" and "AK RS-xxx". If we change that than we would need to update all other VIs that access this property.

For the test VIs icons, could you please erase the "Example" text and maybe add some text specific for each protocol?

For the test VIs icons, could you please erase the "Example" text and maybe add some text specific for each protocol?

Changed as suggested


Catches invalid cursor errors

continues as if there was no pos file

If you're new, read the [guidelines]
(https://github.com/reevoo/guidelines/blob/master/pull_requests.md).
Good code is tested, designed for change, easy to understand
and able to cope when things go wrong.
Good pull requests are descriptive, small and short-lived.
For the Author

[ ] Description explains what you have done and why.
[ ] Tests have been written (or you have explained why you don't need
tests).
[ ] Documentation/comments have been added/updated.

For the Reviewer

[ ] The code is well-designed and able to cope with change.
[ ] The code is easy to understand.
[ ] Tests cover happy and unhappy paths in the code.
[ ] Tests are green on CI (or pulled/tested on your machine).

I don't see any comments on your Pull Request.
Are you too good for code reviews now?
No. No you aren't.
Maybe you forgot to say who reviewed it. That's fine, but let me know.


Printing all paths of a binary tree that leads to a particular sum with any node as root

Algorithm - To print all the paths of a binary tree that leads to a given sum with any node taken as root.
Languages Supported

C

C++

Java

Python

I would like to contribute to this issue in C, C++. Please assign.

I'm up for this to implement in python. @ujjawal-1999 please assign the task.

I would like to work on this issue in C++ as a part of Hackin Codes 2020

@pradeep98 Go ahead with Python.
Please follow the contribution guidelines when submitting your contribution.

@ujjawal-1999 Go ahead with C. @shankhanil007 can proceed with C++.
Please follow the contribution guidelines when submitting your contribution.


Provide an event based mechanism for retrieving transaction receipts on transactions

i think currently, most write methods return TransactionReceipt if void.
for a reason that people can keep track of the hashes offchain or do further processing.
in general, i think we need to have a better system in the SDK for TransactionReceipt, i feel like we should almost have a .on('transactionSent', (transactionReceipt) => void) event in the SDK so people can subscribe for these tx receipt. (for future notes)
Originally posted by @jakeloo in https://github.com/nftlabs/nftlabs-sdk-ts/pull/19#discussion_r747009111

@jakeloo  so I don't think this should block the royalty module unless we feel like we absolutely need to return this info in the current iteration.

nope. it's not blocking.


add better error reporting for cyclic dependencies

Not the best possible solution, but at least it's something :)

Related to https://github.com/alcuadrado/truffle-flattener/issues/14


Swift 3 @objc inference in Swift 4 mode is deprecated

Xcode shows me this warning:
The use of Swift 3 @objc inference in Swift 4 mode is deprecated. Please address deprecated @objc inference warnings, test your code with “Use of deprecated Swift 3 @objc inference” logging enabled, and then disable inference by changing the "Swift 3 @objc Inference" build setting to "Default" for the "FAPanel" target.
Can you fix it?

@theMoe
Issue is resolved. Please have a look into updated pod.
https://cocoapods.org/pods/FAPanels

Perfect! Thanks! I will try it.