added kedro-dolt plugin

Description

resolves #916
Development notes

Checklist

[x] Read the contributing guidelines
[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[x] Added a description of this change and added my name to the list of supporting contributions in the RELEASE.md file
[ ] Added tests to cover my changes

Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

You may just need to run make lint on this PR and then it will pass the linters and we can merge it 😄

make lint

make lint

Thanks! @himanshu007-creator  we should also add a link to this page on the Dolt docs
https://www.dolthub.com/blog/2021-06-16-kedro-dolt-plugin/

Can you update the PR title to "Add kedro-dolt plugin to list of Community Developed Plugins" or something similar? :) (imperative mood for the first verb, being explicit around the fact that it's adding the plugin to the list rather than creating the plugin as part of Kedro)

I got the following output process_begin: CreateProcess(NULL, pre-commit run -a --hook-stage manual, ...) failed. make (e=2): The system cannot find the file specified. make: *** [Makefile:19: lint] Error 2

Unfortunately, it looks like you're on Windows, so you have to jump through a few more hoops. https://kedro.readthedocs.io/en/stable/14_contribution/02_developer_contributor_guidelines.html#before-you-start-development-set-up suggests looking at https://github.com/quantumblacklabs/kedro/blob/0.17.5/.circleci/config.yml#L128-L179 to understand how to set up your environment, but I appreciate that this isn't the most friendly. In short, you'll see that it uses Chocolatey to install Make on Windows.

I have make and Chocolaty installed already✔


Cast database engines to array.

Prevents passing null to the foreach loop in case no database engines defined.
Fix #2202

Released on v9.7.2, thanks!


Add pdm lockfile support

Add support for parsing package information from  pdm.lock -files used by pdm, package and dependency manager for Python (https://pdm-project.org/latest/)

Looks good, just got a few changes - would you mind also adding an "empty.lock" (which is "no packages", not "completely empty file") and "two-packages.lock" fixtures as well? (I know technically your other tests cover this, but it's consistent with the standard set of fixtures we have for our other lockfiles)

Added requested tests. Amended fixes to the existing commit.

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

Comparison is base (cbdceae) 79.85% compared to head (3b6e492) 79.68%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #776      +/-   ##
==========================================
- Coverage   79.85%   79.68%   -0.17%
==========================================
  Files          90       91       +1
  Lines        6174     6213      +39
==========================================
+ Hits         4930     4951      +21
- Misses       1040     1057      +17
- Partials      204      205       +1

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


chenge wsRouter match type

#73

Hi @SaltFish001, thanks for contributing.
I have extended the prefix matching strategy to use https://www.npmjs.com/package/micromatch
From v3.3.0+ you can use: /rtp/*.flv as prefix value.
I am closing this PR as it is now a duplicate.
Best Regards

Thanks a lot


import Layer typo.

What does this PR do?
Fix a minor issue in the documentation.
My first pull request ever, it was about time. Please let me know if I could improve in any way.
Thanks.

Thanks for your contribution @januszdotnl and welcome to the team


[ROCm] Fix for ROCm CSB Breakage - 200630

The following commit (which switched G's internal CI to use ROCm 3.5) breaks the ROCm CSB build (which still uses ROCm 3.3)
https://github.com/tensorflow/tensorflow/commit/22def20bae7be6d5b790b360abed5919385b16c2
This PR/commit simply puts back a couple of codes that were removed the the previous commit, and makes them conditional on ROCm 3.5.
Note that the ROCm CSB build will be switching to ROCm 3.5 or higher in the near future, at which point all codes within the true block for #if TENSORFLOW_COMPILER_IS_HIP_CLANG will become default, and those in the false / #else block will be removed.

/cc @chsigg @cheshire @nvining-work

@chsigg gentle ping

@chsigg gentle ping
Need this PR to fix the broken ROCm CSB

@chsigg gentle ping

@chsigg I do not havbe visibility into the results for the internal ROCm build at your end. Let me know if it passed and whether we can take this PR

@chsigg gentle ping

@chsigg gentle ping


Occasional Error While Playing Track

Some tracks (from yt) are skipped in the middle or at the beginning. This is what I get in my logs:
https://pastebin.com/2vgB0icz
I'm using D4J and almost the example integration with some minor changes.

Caused by: java.lang.RuntimeException: java.net.SocketTimeoutException: Read timed out
Looks like bad net to me.

The issue might be mitigated by newer versions which also retry on socket timeouts. But probably mostly caused by network issues. An exception is live streams where it may be caused by Lavaplayer being too impatient when waiting for next segment which is not yet available.


Need root LICENSE files in hal_stm32 module

Describe the bug
The https://github.com/zephyrproject-rtos/hal_stm32 repository does not have a root LICENSE file. Most, but not all of the source files within the repository have license information.
Impact
This is blocker to locally replicating this repository for builds.

@jettr, Txs for reporting this issue.
I'm not sure how we can fix this, more explicitly I'm not sure what should a root LICENSE file contain in case of a module like hal_stm32.
Here is the status of the package:
Most of the files (the ones coming from cube packages) are BSD-3 Licensed in this module, when this is not the case, they are:
-CMake related files (some of them being Apache 2.0, few not licensed)
-2 python scripts (ST Copyright, missing Apache 2.0 that should be added)
-README (not licensed)
What can be done easily is to update the CMake and python scripts to Apache 2.0.
I don't think there is a need to do anything on README.
Once this is done:

is the root License file still required ?
What could it mention knowing that the module contains mix of BSD3 and few Apache 2.0  ?

cc @MaureenHelm, as it seems more modules could be impacted

I am talking with our internal teams on which potential solution would work best. I will let you know when I hear back. Thank you for responding so quickly!

cc @MaureenHelm, as it seems more modules could be impacted

@ioannisg Can you add this to the list of items to consider for module repos?

@MaureenHelm I think we already have a discussion item for license files and license checks.
Will make sure we discuss this soon.

Is there any update on this? This is blocking our forward progress on replicating our needed Zephyr dependencies and building it in an official way.
Do we have an estimate on when we think we can resolve this for the stm32 repository?

@jettr sorry for the delay, we'll try to get this fixed by next week.

Thank you! I really appreciate it!

Do we have an idea of how we are going to fix this with a potential timeline? This really is blocking us.

I can't add my feedback on that pull request (I don't think I have permissions to leave a comment). It looks like it will be okay, but I am getting it reviewed internally just to make sure I am not missing anything. I will let you know soon. Thank you!


Redirect mails to maildrop based on ENV variables

Currently, staging sends real emails.  This changes the seeder such that
it writes a maildrop.dsd.io address for each prison if the
SMTP_HOSTNAME environment variable is set to maildrop.dsd.io.  I
decided to do it this way as it prevents us from having to change the
smoketests to account for different environments. This would have been
the case with any of the other mail interceptor gems.

If we set SMTP_HOSTNAME to maildrop.dsd.io, does Maildrop show emails there regardless of domain? If so, then we don't need the address mangling.
Does sending all email to Maildrop (via SMTP_HOSTNAME) stop the smoke tests from running on staging?
Having maildrop.dsd.io hard-coded in the middle of SeedEntry makes me feel unhappy.
You can send an email to a mailbox.dsd.io address through the internet, and it will be delivered. This presents an alternative course, which is to have another variable – PRISON_EMAIL_DOMAIN, say – and use that to generate prison email addresses.


added version as an option along with test

This pull request allows for version to be passed as a segment in the url.

Hey @cbhoweth!
I never saw this PR. My apologies!! Merging in now :) Thank you!!!


Flag for disabling colored output

There are some terminals who do not display colored ouput properly. For such cases kubescape needs to support a flag (so as --plain-output) that when used, the output (stdout/stderr) will be without colors at all.
If you wish to take this upon yourself, please contact one of the maintainers for more information 😀

@dwertent Can we please close this since #445 is merged? Thanks.

@Akasurde I think it's better we close it when we release a new version :)


Support for CachingConnectionFactory

It would be nice if it support CachingConnectionFactory or something pooling mehanism.
Now I can not wrap ConnectionFactory to CachingConnectionFactory in simple @Configuration file becouse circular dependency and encounter other jmsAutoConfig healt etc.
Simular like this https://github.com/spring-projects/spring-boot/issues/12161
Thank you!

What kind of circular dependency are you referring to?
Would you mind to provide an failing application example?
I'm looking into the possibility of creating a PR.
Thanks

I've made a PR with the pooled connection factory:
https://github.com/ibm-messaging/mq-jms-spring/pull/12

Thank you very much!


SSE for watchOS

Hey there! I apologize for bringing up this issue, but there isn’t a discussion tab available in the repository.
I wanted to express my appreciation for your amazing library, which seamlessly integrates SSE with SwiftUI. It’s been working flawlessly on watchOS and I’m thoroughly impressed!
However, I’ve encountered a question. Apple has implemented restrictions on certain types of connections, such as websockets, starting from watchOS 9. Based on my initial testing, it appears that SSE connections are permitted. Everything functions perfectly when I test the application on a real device in debug mode (built from Xcode), as I continuously receive events.
Yet, when I switch to using TestFlight, it seems there is some form of limitation in place, and not all events are being received.
Interestingly, when I connect the watch to a charger, it operates as expected. Could you provide any insights on whether this is the correct approach to utilizing DataStream for watchOS? Have I overlooked something? Thanks!

Hi,
I don't have much experience with watchOS, and haven't tested EventSource on it yet.
Your message suggests the issue might be in how watchOS manages long-lived SSE connections to conserve battery life. I suggest to check Apple's developer documentation on watchOS power management and network requests for guidance.

Yeah, thanks for the response. After the tests it seems that SSE are not reliable on Apple Watch because of the battery management.


api information added

There's a whole section on AJAX.

How do you link an API in your code?

Even though there is a whole other topic on AJAX, I think it would still be helpful to have a little example here to illustrate accessing an API.

Added an AJAX call example.

Hey Brendon. Can you please correct the spelling error in one of your titles so I can merge this doc to the curriculum?


Feature update posts display

Update layout structure for mobile responsive

Add SCREENS media query helper, using styled-breakpoints package.
Revamp Main.tsx
Add initial Search component

Mobile Views
Jobs listing

Jobs detail

@SheldonCheng are you able to create a branch on this repo without forking? That way build will be able to copy the secrets and pass.

Replaced by https://github.com/jobsgowhere/jobsgowhere/pull/60


Exclude junit/hamcrest from spring-cloud-function-adapter-gcp classpath

Using spring-cloud-function-adapter-gcp transitively includes Junit and Hamcrest on the classpath because the junit5-capture-system-output-extension is included with compile scope.

@stefnoten Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@stefnoten Thank you for signing the Contributor License Agreement!


Test categorical confidence intervals

Added a test for the LinearRegressionEstimator which tests confidence intervals as a pandas series for a categorical data type. Ideally, the estimate_* methods should all return pandas series for confidence intervals.

Merging now so I can make use of in #264


Increase Zwave light update delay

Description:
Increases the timer for a refresh after zwave light state changes.  The current 2s is often too short to allow a dimmer to complete it's transition from on to off or off to on so HA updates with the wrong value.
Related issue (if applicable): fixes #
#3389
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If code communicates with devices, web services, or a:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

I don't like this. Now we optimize for dimmers but make it worse for non dimmers (slower updates). Can you detect if a light is dimmer based on a command class ?

I had thought that in terms of zwave light was only used for dimmers, while the switch component was used for non-dimmers.  So I think we are already confined to only dimmers within the light component.

If a device is using COMMAND_CLASS_SWITCH_MULTILEVEL it is a dimmer.

It looks like we're supporting two different command classes: https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/zwave.py#L120-L121
Also for future proofing it would be good to make sure we check against the current command class.

Are you still planning on addressing our comments?

Looking at this some more, we are supporting multiple device types (GENERIC_TYPE_SWITCH_MULTILEVEL, GENERIC_TYPE_SWITCH_REMOTE, SPECIFIC_TYPE_POWER_SWITCH_MULTILEVEL, SPECIFIC_TYPE_SCENE_SWITCH_MULTILEVEL, SPECIFIC_TYPE_NOT_USE), but only one command class (COMMAND_CLASS_SWITCH_MULTILEVEL) so no matter the device type the functionality of it being a dimmer would be the same.
This also only effects the refresh to verify the state change after the action occurs, so the only negative end-user effect would be a slightly longer delay to report back whether the action happened correctly, not the action itself.

Like I said, for future proofing it is nice if we lock it to only apply for the multilevel switch command class. A person adding a new light command class will not know to check in the light platform to make sure the values are correct.
I believe 5 seconds is resulting in a very poor user experience. The frontend will wait 2 seconds to assume the command ran correctly before resetting the switch to the current known value. This fix will cause the switch to go back and forth, causing confusion.

I understand your concerns and am testing changes now that implement the longer delay as a workaround for specific devices only.  I'll update the PR for others to test once I put it through some functional tests this evening.

With the recent changes to the light command classes i'm going to close this PR and open a new one when ready to simplify the merge.


Regenerated from OpenAPI spec

Changes

Modification to template to regenerate the change made in #172
Use fully qualified type names instead of aliasing them on import (e.g. compute.ComputeSpec)
Update integration tests to accommodate type name changes in the spec.

Spec: 8687781fe17d83694a0c67396e04c70830b0d398 (2023-06-23)
Type renames:

compute.BaseClusterInfo -> compute.ClusterSpec
compute.ClusterInfo -> compute.ClusterDetails
jobs.JobTaskSettings -> jobs.Task
jobs.RunSubmitTaskSettings -> jobs.SubmitTask
jobs.JobWebhookNotifications -> jobs.WebhookNotifications
jobs.CreateJobFormat -> jobs.Format
jobs.GitSourceGitProvider -> jobs.GitProvider
jobs.FileArrivalTriggerSettings -> jobs.FileArrivalTriggerConfiguration

Tests

[x] make test run locally
[x] make fmt applied
[x] relevant integration tests applied

Codecov Report
Patch coverage: 72.28% and project coverage change: +0.03 :tada:

Comparison is base (41f17a8) 53.36% compared to head (2eba6e0) 53.40%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #199      +/-   ##
==========================================
+ Coverage   53.36%   53.40%   +0.03%
==========================================
  Files          29       29
  Lines       18331    18441     +110
==========================================
+ Hits         9783     9848      +65
- Misses       8548     8593      +45

Impacted Files
Coverage Δ

databricks/sdk/service/billing.py
52.72% <ø> (ø)

databricks/sdk/service/provisioning.py
50.55% <ø> (ø)

databricks/sdk/service/sql.py
56.06% <ø> (ø)

databricks/sdk/service/workspace.py
51.59% <25.00%> (ø)

databricks/sdk/service/serving.py
53.43% <40.00%> (-0.04%)
:arrow_down:

databricks/sdk/service/settings.py
50.10% <47.27%> (-0.47%)
:arrow_down:

databricks/sdk/service/jobs.py
55.55% <73.14%> (+0.08%)
:arrow_up:

databricks/sdk/service/catalog.py
53.00% <76.92%> (+0.22%)
:arrow_up:

databricks/sdk/service/compute.py
53.85% <81.98%> (+0.05%)
:arrow_up:

databricks/sdk/service/sharing.py
53.92% <83.33%> (+0.06%)
:arrow_up:

... and 1 more

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

@nfx The docs update worked after all (had to reinstall the SDK package a second time for some reason...)
Included a list of type renames in the PR summary.


Lock absurd version, remove unused dependencies.

@src-code

CLA is valid!

LGTM

Coverage remained the same at 100.0% when pulling 3eb370e9ced164e0080dec3290177127f7220d75 on absurd-lock into ede28f17f9fda39ea0769d8a8039dfdd41f55bd4 on master.

Coverage remained the same at 100.0% when pulling 3eb370e9ced164e0080dec3290177127f7220d75 on absurd-lock into ede28f17f9fda39ea0769d8a8039dfdd41f55bd4 on master.


[Feature] Provide an undoAll method in FUniver to recover to the state before all editing operations.

Initial checklist

[X] Is this really a problem?
[X] I have searched the Github Issues for similar issues, but did not find anything.

Problem
[Feature] Provide an undoAll method in FUniver to recover to the state before all editing operations.

In a purely display scenario, a series of temporary draft operations are made in a sheet. After switching to another sheet, it's necessary to restore the original state. This can be achieved with the undoAll method, or is there a better approach?

There is no plan to support this requirement at present. Currently you can achieve your goal by caching the previous snapshot and then reloading the current snapshot data when needed.

There is no plan to support this requirement at present. Currently you can achieve your goal by caching the previous snapshot and then reloading the current snapshot data when needed.

How to reloading the current snapshot?

https://stackblitz.com/edit/awesome-univer-sheets-vite-demo-sw3ee3?file=src%2Fmain.ts
reload the current snapshot data in this demo only work once.

https://stackblitz.com/edit/awesome-univer-sheets-vite-demo-sw3ee3?file=src%2Fmain.ts
reload the current snapshot data in this demo only work once.

You need deepClone the snapshot when you use it, then it will work. @wang2lang


fix yq usage to support fake yq

Issue #, if available:
Description of changes:
We have run into an issue a couple times now where on Ubuntu 24.04 a version of yq is included but it is not the correct yq.  The noticable issue is that return values are double quoted.  The -r tells yq to remove the double quotes, which works with both the correct and incorrect version of yq.
We probably should add a more complete fix here where we validate the source of yq, but I think this will avoid the current pain during the image-builder flow since we dont use yq too much.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.20

/test all

/retest

/approve
/lgtm


Add Pre-Processing Techniques of Deep Learning

Image Resize
Description: Adjusting the dimensions of an image.
Benefit: Standardizes input size for machine learning models, reduces computation, and can help speed up processing.
Application: Preparing images for neural networks, reducing image storage size.
Image Crop
Description: Extracting a specific region from an image.
Benefit: Focuses on the region of interest, removes unwanted parts, and can help in improving model accuracy by providing only relevant information.
Application: Object detection, facial recognition.
Edge Detection
Sobel, Prewitt, Canny
Description: Techniques to highlight the edges within an image.
Benefit: Essential for feature extraction, helps in identifying object boundaries, and is useful in various image analysis tasks.
Application: Object detection, image segmentation, pattern recognition.
Noise Removal
Description: Reducing or eliminating noise from an image.
Benefit: Enhances image quality, improves feature extraction, and can lead to better model performance.
Application: Medical imaging, photography, and any application where image clarity is crucial.
Image Conversion (Grayscale, Binary)
Description: Converting an image to grayscale (single channel) or binary (black and white).
Benefit: Simplifies the image, reduces computational complexity, and is often required for certain algorithms.
Application: OCR (Optical Character Recognition), thresholding, and preparatory steps for edge detection.
Histogram Equalization
Description: Adjusting the contrast of an image using its histogram.
Benefit: Enhances the contrast, improves the visibility of features, and can make details in dark or bright regions more discernible.
Application: Enhancing medical images, satellite imagery, and improving the visual quality of photographs.
Use Case
Preprocessing on single and multiple images using python.
this will help users understanding pre processing techniques and increase accuracy of the model
Benefits and Applications Summary
Image Resize: Standardizes size, reduces computation.
Image Crop: Focuses on important regions.
Edge Detection: Extracts structural information.
Noise Removal: Enhances image quality.
Image Conversion: Simplifies processing, reduces complexity.
Histogram Equalization: Improves contrast and visibility of features.

@sanjay-kv please assign this issue to me

assign me this issue.

@SHRAVANIc01 its already assign to different user. sorry abt that, you can raise a new issue.


Feature/tp 6571) Shared filters

allows the user to search for devices shared by them shared: ('by-me'), or shared with them ('to-me') or everyone ('all'). And also the owner of the device.

Looks good to me.
For future new usages we should use the new extended version of searchDevices, so need to keep in mind to port the new parameters to the new function when needed


Create mercury.txt

Add a notes file for the planet Mercury

Thank @mlbeller for your contribution.


Start with telemetry popup

When being initialized Sentry makes an API request to the Sentry server which is unwanted before the user gives their consent. Solution: Before showing the onboarding flow show the Telemetry popup and ask the user for their consent; when the user changes permissions close the current Sentry client or reinitialize it depending on the choice. Also, pass the release version to Sentry when initializing it.
Minor bug fix: copying text to the clipboard doesn't work if the user doesn't click on a button, so this function call is removed now and shouldn't throw an error in the console.

@anastasiya1155 Do we need to accompany this with any changes to the Tauri logic?

@anastasiya1155 Do we need to accompany this with any changes to the Tauri logic?

I don't think so, looks like BE does not make this initial request to sentry


TypeError:  is not JSON serializable

TypeError: <django.utils.functional.__proxy__ object at 0x030B1D30> is not JSON serializable in a Djrill message (perhaps it's a merge var?). Try converting i  t to a string or number first.

What does it mean?

The "not JSON serializable" error means one of the options you've set on your EmailMessage is an unexpected type, and Djrill doesn't know how to convert it to something Mandrill will understand. So you'll need to convert it to a string yourself, when you set the option.
This commonly occurs with merge variables, but it could be any option you're setting. (We'd need to see the part of your code that's creating and sending the message to say more.)
django.utils.functional.__proxy__ probably means the issue is with an option being set to a lazy string translation (ugettext_lazy). If that's the problem, you can probably fix it by switching to the non-lazy version of ugettext.

Yes that was it, thanks.


HTTP(s)-Server crashes in start() if used with a "stopped" io_service.

Hi,
I came across this bug, when trying to run both Simple-Websocket-Server and Simple-Webserver through the same io_service. Following code should reproduce the error:
auto io_service = std::make_shared<boost::asio::io_service>();

HttpServer server(80, 0);
server.io_service = io_service;
server.start();

The reason seems to be the call to io_service.reset() (instead of io_service->reset()) in start(). https://github.com/eidheim/Simple-Web-Server/blob/master/server_http.hpp#L151

Thank you for seeing this!


[djangoflow_fcm] After permission is granted getToken throws Subscription failed - no active Service Worker

I have done a temporary workaround for now using https://github.com/firebase/flutterfire/issues/11798#issuecomment-1826360250
This is a bug due to firebase_messaging pacakge. It's being tracked here. Will close this ticket once firebase team addresses the issue.
https://github.com/firebase/flutterfire/issues/11954

Fixed on firebase_messaging

iin which update can you please mention the release version.


Cannot stop a running transaction as admin if the charger is offline.

I think I already fixed it, maybe not deployed in prod...

I think I already fixed it, maybe not deployed in prod...

Ok: https://github.com/LucasBrazi06/ev-dashboard/commit/71c59e573d0f4e1ec175eeb1eea019b3ec5fa0b6
There's no similar checks in the backend?

There's no similar checks in the backend?
Nope.

I think I've found another similar issue: I cannot delete a charger that have a non existent (or at least not visible in the dashboard) session running on testcharger tenant in QA: ABE_Electricite_SAR_Cp3 ;)

I think I've found another similar issue: I cannot delete a charger that have a non existent (or at least not visible in the dashboard) session running on testcharger tenant in QA: ABE_Electricite_SAR_Cp3 ;)

If that's reproducible and can happen, yes, this has to be fixed :)

I think I've found another similar issue: I cannot delete a charger that have a non existent (or at least not visible in the dashboard) session running on testcharger tenant in QA: ABE_Electricite_SAR_Cp3 ;)

If that's reproducible and can happen, yes, this has to be fixed :)

A second charger have the same issue: TUD093003B so I say it's reproducible. The culprit transaction id must be found now ...

Need to tested to reproducibility.


Enemy setup

Spawn enemies on the map

Ticket is blocked by the lack of a map and a camera for the player


Years/Months only

Hi there, it would be nice to have options to display a calendar with years only or months only.

I think your best bet is to probably roll your own component -- you don't need the overhead of moment.js and the additional functionality that this component provides.


az vmss update-instances doesn't reflect ssh keys in VM, reimage has to be it.

As titled, the update-instances won't update ssh key. according to portal notice: "Once the scale set is updated, all new VMs have the new credentials. Existing VMs only have the new credentials if they are updated to the latest model and then reimaged ."

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

ID: 3b81a147-db1c-b92f-88d1-6dd0991a72fe
Version Independent ID: f5a2c949-498f-5848-d89c-031d1a757120
Content: SSH into Azure Kubernetes Service (AKS) cluster nodes
Content Source: articles/aks/ssh.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Close as tested again the instruction works.

The instruction provided doesn't set VMSS public key in the VMSS spec, so the impl is different from portal.

@tdihp Thanks for the feedback.


OLCNE default resources are insufficient for Istio

OLCNE with 2 worker nodes is not enough to run all pods of Istio. When observing the pods, some of them are in the Pending state and the cause is:

Warning  FailedScheduling  2m11s (x2 over 2m11s)  default-scheduler  0/3 nodes are available: 1 node(s) had taints that the pod didn't tolerate, 2 Insufficient memory.
Warning  FailedScheduling  55s (x2 over 55s)      default-scheduler  0/3 nodes are available: 1 Insufficient cpu, 1 Insufficient memory, 1 node(s) had taints that the pod didn't tolerate.

Some possible solutions:

Set the minimum number of Worker Nodes as 3 if ISTIO variable is true
Document the minimum number of worker node for Istio
Remove the taint of master
Increase the vCPUs to 3 by default

I'm running Istio with 2 worker nodes. The bump to 3GB of RAM and 2 vCPUs should've covered the requirements. Let me see if I can reproduce this locally.
Note that we're not going to untaint the master, but we may increase the vCPUs (if necessary). I don't want to increase the minimum worker node count unless we really have to.

Another possible solution would be running a single replica for each pod in the Istio control plane. I noticed that there are two replicas for each one of them. Is that something possible to do during the module installation?

We need at least two replicas for the module in order to support a zero-impact upgrade process. I'll check what (if any) options are available when we create the module, but I suspect this isn't (currently) possible).

I've closed the PR that increases the resources. We may need to look at modiying the Istio configuration prior to deployment to lower the resource requirements.

Maybe OCLNE should allow the user to choose an Istio configuration profile

That would make a good ER for OLCNE, yes. I'll open it internally.

Do we need to keep this issue open?
There isn't anything directly actionable for this project at this point in time...

No, this can be closed. There is an internal enhancement request to track this.


Fix typo in server connection object interface name

Changed ISeverConnectionOptions to IServerConnectionOptions and used the type in test file to prove it is working.

@locnguyen Thanks mate!


Do not Limit zoom in ZoomHelper

@charlenni I now removed the limiting in ZoomHelper. The only place there is limiting is in the ViewportLimiter. It is of course better to have it in one place. There was already a problem with the current solution, the ViewportLimiter allowed one level more than the ZoomHelper. I now removed this extra level in the ViewportLimiter to keep the behavior for mouse wheel the same but I am not sure I changed it for some other use case. Either way, the current logic is straight forward and if users want an extra level this is possible through another ViewportLimiter implementation.

So, if I understand it correct, I have to create a new ViewportLimiter to get, what I want (set a range of zoom levels that could be displayed independent from the zoom levels the TileSource has. Is this correct?

You can set ViewportLimited.ZoomMode = ZoomMode.Unrestricted to allow zooming outside of the zoom levels in the layers. I think this is rarely what users want.
You can set ViewportLimited.ZoomMode = ZoomMode.KeepWithinResolutions and set ViewportLimiter.ZoomLimits to extend (or restrict) the range outside of the list of Resolutions. This might be sufficient for your use case.
Further, you could implement your own IViewportLimiter for alternative use cases. And there are currently already two other IViewportLimiter implementations that might be useful.
It is not possible to override the full list of Resolutions.  It would make sense to add this possibility. It is easy to understand. The ZoomHelper logic would have to change to allow this.

Thank you for your help. I will try the second or the first version.


Guidance for new projects - should LTS be used in preference to this repository?

Is your feature request related to a problem? Please describe.
The recommended path forward for use of AWS Services (Shadow, OTA etc) with FreeRTOS is very unclear. It appears that this repository will soon become deprecated and https://github.com/FreeRTOS/FreeRTOS-LTS should be used going forward. This (LTS) repository does not yet contain the OTA library, and there is no clear example of using the OTA library with Shadow/direct MQTT access.
Describe the solution you would like.
A clear and concise demo/example of using AWS Shadow/OTA and direct MQTT with (specifically) the ESP32 platform.
Additional context
Our major reason for wanting to upgrade AWS libraries/FreeRTOS is to allow a single MQTT connection to be used for Shadow/ OTA and direct MQTT publishing. This is not supported with the version of FreeRTOS we are currently using (1.4.7). Our device is battery powered and only periodically within wifi range, we would like to reduce the time required to synchronise shadow and check for OTA jobs, and our intended approach is to reduce the connection time by sharing a connection for these services.

@thealastair,
Thanks for bring these questions and concerns to our attention. Please find answers to your questions below.

First of all, this repository is not going to be deprecated and will serve as a channel for the FreeRTOS reference integrations of the FreeRTOS and AWS libraries. Please find the FreeRTOS page about reference integrations here.
OTA library doesn't have a LTS release yet. However, we are working on a LTS release for OTA library currently.
About the connection sharing, we are working on MQTT Agent APIs to enable MQTT connection sharing between multiple tasks. Please refer to the Lab-Project-coreMQTT-Agent for the MQTT Agent APIs and the demo applications for connection sharing. Note that this is a work in progress, but we are not expecting a drastic change in the MQTT Agent APIs. We are currently working on a plan to release these APIs.

Please let us know if you have any further questions about this.

Hello @thealastair ,
Just wanted to share that OTA library is now part LTS release and it available to download from here or from GitHub repository
Thanks!

Thanks for the response, we'll check it out.

I'll close this issue now.  Please feel free to open another if you help.


encoding/gob: Encoder fails to encode maps with keys of type url.URL

What version of Go are you using (go version)?

$ go version
go version go1.12.5 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/xxx/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/xxx/gocode"
GOPROXY=""
GORACE=""
GOROOT="/home/xxx/go"
GOTMPDIR=""
GOTOOLDIR="/home/xxx/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build478246212=/tmp/go-build -gno-record-gcc-switches"

What did you do?
https://play.golang.org/p/8-8Qws7ogyH
What did you expect to see?
Gob encoder successfully encodes a value of type map[url.URL]something.
What did you see instead?
There's an error: gob: unaddressable value of type *url.URL, despite the fact that the map keys are of type url.URL, not *url.URL.

Perhaps this is just the error being confusing. One cannot take the address of map keys or values directly, so that's probably the error that the encoder is hitting. Perhaps it could copy the key value to then take the copy's address, but I wonder if that could be unexpected behavior. For example, what if the key type is large and copying is a non-trivial amount of work?
Funnily enough, the spec makes no mention of map key/values not being addressable. It seems to be a limitation of the current Go implementation.

To clarify my previous comment - the String method uses a pointer receiver, so it can only be called on a non-pointer value if it's addressable. See https://golang.org/pkg/net/url/#URL.String

With a custom type: https://play.golang.org/p/jZJr0eYPal5
The problem seems to be the pointer receiver in MarshalBinary. With a non-pointer receiver encoding/gob works fine, even if I add a String method with a pointer receiver: https://play.golang.org/p/dK7SsqahtAd


Support control flow in Unroller transpiler pass

Add support for ControlFlow instructions to Unroller transpiler pass.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 17 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 83.46%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
52407

Relevant Lines:
62793

💛  - Coveralls

@Mergifyio refresh


Moved "file-uri-to-path" dep to app package.json (fix #566)

Moved "file-uri-to-path" dependency from root package.json to app/package.json, otherwise an error is thrown by the packaged application. This fix issue #566.

Perfect, cheers!


Create Help Ishaan.java

The solution for the Problem of the day | 09-10-2022.
Please accept my PR and help me in my first Open source contribution. Also, tag it with Hacktoberfest 2022.
Please fill the details before contributing.

⚠Important!!

[x] Done reading CONTRIBUTING.md?

Adding Code PRs?🔥
⚠Do not add multiple files for the same solution!!!

[ ] Added problem statement(Readme.md) with solution?

[x] Contributing new file?
[ ] Appending your code in already existing file?

From Platform:

[ ] Leetcode
[x] GFG
[ ] Interviewbit
[ ] Others(only acceptable after DISCUSSIONS)

Migration PRs?🤝

[ ] Done deleteing file from home directory?
[ ] Added file with correct name and correct directory?

So, can I just rename my file as Help Ishaan 2.java and create another PR?

There is no problem that exists like Help Ishaan 2, will recommend you check the already existing file and contribute your code there. (append at the end of that file)


Redesign fixes

These changes (along with a small bug fix in the main ArchiveBox repo) let the extension successfully make a request to a running ArchiveBox server.
This was tested with v0.8.5rc51
The request to the REST API is made from the background script, instead of the popup script, to avoid having to deal with CORS issues.
I'm not doing anything with the response from the fetch request for now, but we could probably have an indicator that updates to show that the snapshot was successful.
It seems like the content_scripts section of the manifest.json was superfluous, and led to errors loading the extension for me, so I removed it.

LGTM. thanks!


Auth0 - access_denied: Invalid state

Observed behavior
Hello,
After following the latest guide  for deployment I am still not able to login with Auth0 even after the recent update of the server. I am simply trying to deploy the application for testing purposes but even after I tried with both s3 bucket and localhost I am getting this error:
Web output:

Console log:

Debug output:

After login button is pressed Winchan opens a different browser window (as it should) and shuts down after 1-2 seconds. The URL that I managed to capture (edited for clarification):

https://<auth0_domain>.eu.auth0.com/sso_dbconnection_popup/<auth0_client_id>

which seems correct based on the auth0-js file

The procedure was done very carefully so I don't believe I missed something in the process but something is breaking the state value that auth0-lock.js sees. State is only passed as argument in
Auth0Manager.js:85:22 with value the URL of the current window that called the login.
Any help would be greatly appreciated
Expected behavior
Simple login
Steps to reproduce the problem
Even the guide is outdated on some parts (regarding Auth0 setup) you just have to follow the process described in Read The Docs and just add 2 Auth0 applications (one REGULAR WEB APPLICATION and one MACHINE-TO-MACHINE)
Any special notes on configuration used
Updated version of env.yml (server) contains 2 new entries which refer to the MTM application:

AUTH0_API_CLIENT
AUTH0_API_SECRET

The rest are filled as described in the latest documentation.

Unfortunately, Auth0 issues can be difficult to diagnose. I don't believe this is a bug, but looks to be a misconfigured application. Elaborating on which other Auth0 config variables you have set in both the UI and server may help us diagnose.
However, as a first crack, you may want to try logging in with an updated version of Chrome -- some browsers can experience issues with logging in.


Error: spawn prisma-generator-nestjs-dto ENOENT

Hi,
I just tried out your tool and get following Error on my windows machine after running prisma generate:
Error: spawn prisma-generator-nestjs-dto ENOENT
I tried running it in git bash, IntelliJ Terminal, CMD, Windows Terminal. Is it maybe an missing permission (I have a lokal Admin account..)? Or is my schema code wrong?
generator client {
  provider        = "prisma-client-js"
  output          = "../generated/prisma-client"
}

generator nestjsDto {
  provider                        = "prisma-generator-nestjs-dto"
  output                          = "../libs/shared-dto/src/test"
  outputToNestJsResourceStructure = "false"
  exportRelationModifierClasses   = "true"
  reExport                        = "false"
  createDtoPrefix                 = "Create"
  updateDtoPrefix                 = "Update"
  dtoSuffix                       = "Dto"
  entityPrefix                    = ""
  entitySuffix                    = ""
}

datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

[...]

UPDATE:
I also tried it after doing this:
rimraf node_modules
npm cache clean --force
npm ci

Do you have any idea?
Regards

installing the package should create a symlink in node_modules/.bin/prisma-generator-nestjs-dto pointing to node_modules/prisma-generator-nestjs-dto/dist/cli.js.
Can you pls verify this exists?
The provider field in your schema tells the prisma cli which command to execute to run this generator - so basically what the error is telling you is that the cli can not span the generator process because the file doesn't exist.

Hi @benkroeger ,
I had a look, and there are 3 files:

I read about running the "spawn" function with {shell: true} on windows, maybe this is not happening?

idk. I'm just providing the command - the call (spawn etc) is in the prisma cli.
Have you tried running this in a privileged mode?
I know there are other users running this on windows - so in general it seems to work. I don't have a windows machine at hand.

what happens when you call the .cmd file directly (not expecting the generator to actually run then because all the inputs are missing - but maybe the process starts)

I can call it directly. So I try to check, if<EMAIL_ADDRESS>is the problem and get back here later.

I just tried another generator to see if it' a general issue:
generator jsonSchema {
  provider = "prisma-json-schema-generator"
}

And it worked... 🤔
Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma

✔ Generated Prisma Client (3.4.2) to .\generated\prisma-client in 1.34s

✔ Generated Prisma JSON Schema Generator to .\prisma\json-schema in 173ms
You can now start using Prisma Client in your code. Reference: https://pris.ly/d/client

import { PrismaClient } from './generated\prisma-client'
const prisma = new PrismaClient()

Good morning,
found the "problem", @benkroeger ! I had a look into the package.json of this project and saw, that node >= 14 is needed. So I just tried it with 14.18.1 and everything works. The errer message was totally misleading.
Have a reat day! Thank your for this great tool!


Find a different syntax for inline snapshot so that cargo fmt can work

Currently, @r###"..."### the inline snapshot syntax is invalid during cargo fmt analyze and cannot be formatted.
Perhaps a different syntax (e.g. i=r"") can be used for helping in this case.

Check out https://github.com/mitsuhiko/insta/issues/456, both this:

Would be great for inline snapshots to work with rustfmt, but don't think there's a way — if the inline snapshot is a normal expression without special syntax (such as @), it's not possible to discriminate between (name, value) and (value, snapshot)

...and the broader discussion.
I agree that it would be great to allow rustfmt to work, though lots of other tradeoffs...

Sharing my workaround:

before cargo fmt, run sg scan --rule <path to yaml> (need install ast-grep)

# Replace `@` with `-` in macros
id: pre-format
language: Rust
rule:
  pattern: '@'
  precedes:
    kind: raw_string_literal
  inside:
    kind: macro_invocation
    stopBy: end
fix: '-'

run cargo fmt

run sg scan --rule <path to yaml>

# Replace `-` with `@` in macros
id: post-format-0
language: Rust
rule:
  pattern: '-'
  precedes:
    kind: raw_string_literal
  inside:
    kind: macro_invocation
    stopBy: end
fix: '@'

from https://github.com/mitsuhiko/insta/issues/605#issuecomment-2366916072

rustfmt is requiring a syntactical valid expression, but not semantically valid expression. Thus there is room to find such patterns. For example, my using -r###""### utilize the fact that unary minus to a string is valid in syntax but invalid in type. Symbols like *, !, ?(postfix) are also possible.

OK, that's quite interesting.
@mitsuhiko do you have thoughts? We could offer an option of something like assert_snapshot!("foo", *"foo");?

@andylokandy, on further reflection, I'm not sure there's a syntax which would actually work here?
We need to identify the type of macro in the macro_rules!. Currently we do that with @literal and:

If we use something syntactically valid like !"Testing", then the macro has ambiguities
If we use something like +"Testing", then cargo fmt no longer works

Am I right in thinking this is the full set of options? Or is there some other way around it?

If we use something syntactically valid like !"Testing", then the macro has ambiguities

Could you provide a example of the ambiguities? I may not fully understand it. (and what if *?)

Try my branch — the rust compiler will complain about ambiguities in the macro...

Got it. It's a limitation of the compiler (lack of backtrack when parsing macro), but there is a nasty way to get through: instead of directly defining the pattern arms, consume the tokens one by one. In detail:

receive input as $($input:tt)*
match one token a time, until the pattern is determined
call sub-macro for each pattern

Here is a demo of this technique.

Ah OK cool!
If you want to make a branch / draft PR with this on, I think that would be quite helpful. Assuming the interface for users is still nice and internal stuff is well-documented, I would be +1 on this.
(...though I'm not the sole decider, to set expectations)


Update example links: replace run locally by see code for more clarity

Relative to https://github.com/evancz/elm-architecture-tutorial/issues/39.
Just replacing the 'run locally' by 'see code' for accessing to the codebase of each example, more clear for me :)

Great! I was having a hard time finding where everyone was talking about in the issue. I agree this adds clarity.

Cool, thank you :) Yeah, I had intended for this to be "if you want to run it locally, here's the code" but I think it stopped making sense as the post got longer. Or I just was thinking weird! In any case, thank you!


add poom.dev to webring

Link location: topmost on both desktop & mobile views.

@heypoom At last, welcome to the webring! Sorry for the noise — I was testing some automation to make reviewing PRs easier.


Add the "OperatorGraph" class to generate query plan.

This PR adds the OperatorGraph class to generate a the query plan.
The OperatorGraph class provides two interfaces for constructing the query plan:
addOperator: adds an operator with a unique operatorID, an operator type, and the operator's properties.
addLink: adds a link from an operator to another operator, as specified by operatorID.
The OperatorGraph checks if the graph is a valid Directed Acyclic Graph (DAG), and then invokes the corresponding functions to build IOperator Java Objects and link them together.

Current coverage is 63.89% (diff: 85.92%)

Merging #240 into master will increase coverage by 1.76%

@@             master       #240   diff @@
==========================================
  Files           129        130     +1
  Lines          3288       3490   +202
  Methods           0          0
  Messages          0          0
  Branches        529        568    +39
==========================================
+ Hits           2043       2230   +187
- Misses         1111       1114     +3
- Partials        134        146    +12

Powered by Codecov. Last update b050bfb...1da7479

Add the issue number in the title.

@zuozhiw let us merge this PR if it's done.

One of the tasks in #238 is finished.

In my opinion, query plan looks more like a tree than a graph. There are three reasons that I think of.

Considering execution of a plan, if a plan has a tree structure, it can be run by post order traversal, but if it has a graph structure then it gets much complicated.

There is a plan explained in the comments
                -> RegexMatcher -->
KeywordSource --<                     >-- Join --> FileSink
                  -> NlpExtractor -->

Suppose a plan looks like below, then neither DFS or BFS will work properly, maybe we should come up with something different
KeywordSource          -> RegexMatcher -->
                                                                                >-- Join --> FileSink
                                   -> RegexMatcher -->
KeywordSource --<                                          >-- Join -->
                                  -> NlpExtractor -->

Plus according to our current implementation, the first plan works more like below.

KeywordSource --<   -> RegexMatcher -->
                                                                             >-- Join --> FileSink
KeywordSource --<   -> NlpExtractor -->

Two operators inside Join operator must have different source operator instance even though they are basically the same, otherwise cursor will get corrupted.

It seems not intuitive that in JSON query there should be links connecting operators. I think we can just convert such JSON into a tree, if we further develop our JSON spec.


Magic Guard should block crash damage

I wasn't really sure what a more appropriate way to do this would have been, since just using recoil would make Rock Head block the damage, I believe.

We should probably split damage and moveDamage functions, kind of like how directDamage is split.


support specifying source Port when connect

can we add support specifying source port in below API:
e.g.
redisAsyncConnectBind(const char *ip, int port,
const char *source_addr, +++int source_port+++)
redisAsyncConnectBindWithReuse(const char *ip, int port,
const char *source_addr, +++int source_port+++)
Thanks!

any update?

Adding yet another argument would sadly mean a breaking change and I hesitate to do so at the moment.
What would be the use case for setting the source port as well? How frequently is this even necessary?


Behavior of system view in system-less containers is unintuitive

The system creation form gets its own view, which isn't true of any other creation form. I think this results in unintuitive behavior.
To observe the confusion:

Switch to a container with no systems.
Click "System" on the sidebar. This takes you to /core/system/create. A system creation form is shown, there's no system list, the page title is "Systems", and the "Systems" item on the sidebar is white. This matches the appearance of an empty list view.
Switch to a container with at least one system. The URL remains /core/system/create, so (again) a system creation form is shown, there's no system list, the page title is "Systems", and the "Systems" item on the sidebar is white. This matches the appearance of an empty list view, which may lead to the false assumption that the container has no systems.

Either the system creation form should be shown on the system list view (rather than its own special view) or we should abandon the idea of automatically opening creation forms when there are no objects. I'm in favor of the latter idea.

Resolved by PR #804.


Add support for fractions in the Recipe View and Recipe Form

When a recipe calls for a fraction (IE: 1/3 cup Flour) that needs to be expressed as a decimal (IE 0.333 cup Flour). We should support users entering in fractions and display them properly as well.

How do you plan on displaying 1/3 rather than 0.333? Assuming we want to keep the numerical value of each quantity.

Yes we would keep the numerical value of 0.333 in the DB
There are some good libraries that we can use to get the fractions displaying better on the recipe view.(http://mathjs.org/docs/datatypes/fractions.html) But allowing users to create recipes with fractions will be another challenge.

Im going to unassign myself for this right now. Not 100% how I want to go about doing this.


fix news code on mobile

resolve https://github.com/codeigniter4projects/website/issues/226

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.161%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
246

Relevant Lines:
323

💛  - Coveralls


New filterPosition prop causes react "unique key" error in console

Describe the bug
When the newly introduced filterPosition prop is set to "top", I get the "Each child in a list should have a unique key prop",  in the console. I haven't tested with "bottom", but it is probably the same.
To Reproduce
Steps to reproduce the behavior:

Just set filterPosition to "top" and check console

Screenshots

The new FilterCell row is missing key

codesandbox
I will make a codesandbox if you need one. This should be very simple to reproduce

@elmoro666 I will fix it in next version, thanks for issue report

fixed in<EMAIL_ADDRESS>

Thanks Allen!

fijo<EMAIL_ADDRESS>
@AllenFang  We are in version 4.0.3 and the problem continues


Aesthetical aspect of the User Interface (UI)

The whole page need to be modified so that the controls are laid on the page properly. Currently the controls are not well lined up in the page.

@mekoki Having a look at http://mamase.org/mamase/chart, do you think they are well lined now? If so, please close the issue

I assume this is fixed. If not please reopen @mekoki


Uniform distribution instead of random permutation

Hi,
First of all thank you for publishing the paper and for providing the code with it for reproducibility. While going through the code, I noticed a slight discrepancy (maybe intended) between the description of the algorithm and the actual routine.
In page 12, it says that :

The algorithm is divided into two phases, a preliminary and optional
phase and the main component. In the first preprocessing phase, a random
variable is introduced into the dataset, specifically, a permutation of the most
influential variable according to the mean of the absolute values of the Shapley
values between the original variables.

However, in the function introduce_random_variable, a uniform distribution is sampled from the most influential variable:
sample_train = np.random.uniform(low=random_variable_train.min(), high=random_variable_train.max(), size=(len(X_train),))
sample_val = np.random.uniform(low=random_variable_val.min(), high=random_variable_val.max(), size=(len(X_val),))

Following the paper definition, it should be:
sample_train = np.random.permutation(random_variable_train)
sample_val = np.random.uniform(random_variable_val)

I doubt that it has a big impact on results but I wanted to check with you nonethless whether it is intended or not. Happy to hear your thoughts.
Thanks,
Thomas

Hi,
First of all, thank you for noticing. Indeed, what the paper says is not being done and should be changed (and will be changed) as you propose. Still, the effect of this modification should be minimal. The intention of that whole fragment is to introduce a random feature that has no relationship with the target variable. That way you can measure the importance of a feature that you know in advance that has no real relationship with the target to be predicted. The construction of this feature as indicated in the paper or with any other criteria should not have any impact, since all procedures would result in an unnecessary explanatory variable.
As I say, we will change the code and the results of the paper if they change considerably, which would surprise us.
Carlos

hi Carlos, thanks for your reply. Indeed I would assume it's not material at all. Thanks for bringing an answer nonetheless


Parallel request routing for Blue Ocean

This issue has its origin in Jenkins Blue Ocean project. In BO, we wanted to expose all Actions of a Build to the URL space of BO, including its sub-objects, in more REST-ful manner. Such code needs to be able to deal with arbitrary Action subtypes that plugins define.
The way we bind those objects to the URL space should mimic the classic UI, thus it should follow regular Stapler routing rules. But BO also need to be able to tweak some aspects of it. For example, index.jelly should be replaced by the state transfer of the object. By extension, we also want to disable all other Jelly/Groovy views that are defined on those objects. In the future a need might arise to do some additional tweaking like this.
I thought about various ways of doing it, such as doing this entirely in BO, but I eventually came to believe that such capability should live in Stapler. I think of this capability as a way of wrapping & filtering existing routing rules and making adjustments, and there's really nothing Jenkins specific about it.
This issue captures this task, so that the solution I propose can be discussed in the context of the requirement.

I'm developing this in #77

Resolved


episode 10: Considerations when porting an OpenMP-CPU code to OpenMP-GPU

things to consider etc

Stub for this lesson at https://github.com/ENCCS/openmp-gpu/blob/main/content/porting.rst


Extension to RGBA as well as RGB

Hi there, I've been using LIIF on emoji glyphs and got some great results, however I'd like to recover the transparency, which I had to remove* by simple alpha compositing (i.e. flattening the image) before passing the PNG inputs to LIIF.

* flattening onto a grayscale background after calculating the grayscale tone not present in any semitransparent pixels, with greatest Euclidean distance from the median of the pixel mean in the image

I tried to "supervise" the estimation of transparency but it was only a rough estimate, and the results it gives are not satisfactory (despite the high quality obtained from LIIF)

This subsection of an emoji glyph was flattened against a black background then run through LIIF.
The bottom right plot shows the recovered transparency (RGBA image) flattened against a different background colour (white)

I was wondering if you think the code could be modified in some way for this, to supervise an estimate of the alpha channel?
It seems like it should be possible but it's unclear to me how I might implement it, any advice would be appreciated

Hi, thanks for your interest in our work! LIIF on emoji is really an interesting project.
I am not very familiar with the processing of alpha channel. Since the model is only trained for RGB images, the alpha channel will make it an out-of-distribution task. If there are a large amount of LR-HR pairs of images with alpha channel, a straight-forward method is to modify the code to work on 4-channel images (since all the code assume 3-channel, there can be many necessary modifications such as the encoder model part and the data normalization part) and train a 4-channel SR model.

Hi, thanks for your interest in our work! LIIF on emoji is really an interesting project.
I am not very familiar with the processing of alpha channel. Since the model is only trained for RGB images, the alpha channel will make it an out-of-distribution task. If there are a large amount of LR-HR pairs of images with alpha channel, a straight-forward method is to modify the code to work on 4-channel images (since all the code assume 3-channel, there can be many necessary modifications such as the encoder model part and the data normalization part) and train a 4-channel SR model.

No problem, thanks for the response! 😄

No problem, thanks for the response! 😄

No problem, thanks for the response! 😄

Have you tried this later? I am curious about the result.


move message-id counter into the session-class

it is not necessary to have a global counter in the session-handler class, because together with the session id it results in a unique identifier for each message

message-id-counter will remain in the session-handler, because its a singleton and so its accessible from the classless send methods.


[BUG]  - Allsky map postToMap.sh error.

Camera model
RPi HQ Camera
Operating System
Bookworm
Allsky version
v2024.12.06
Pi and amount of memory (512 MB, 1, 2, 4, or 8 GB)
Pi 5, 4GB
Bug Description
Seeing this error in my notification:
The website is accessible using the URL for both the image and the website mentioned in the error message so not sure if it's the port number causing this.
postToMap.sh: ERROR: Image URL 'https://allsky.challa.co:1982/current/resized/image.jpg' does not appear to be a valid image.ERROR: Website URL 'https://allsky.challa.co:1982/allsky/index.php/' does not appear to be a valid web site.   (December 27, 06:39:22 AM)

allsky.log
settings.json.txt
configuration.json.txt

@chvvkumar, Kumar, change your WebUI login and password ASAP.
I am on your WebUI page and if I was hacker I could control your Pi.

@chvvkumar, Kumar, change your WebUI login and password ASAP. I am on your WebUI page and if I was hacker I could control your Pi.

Thanks for the head's up!  I changed it to test some things but changed it back now.

For the Website URL error, remove index.php from the URL in the WebUI.   Note in the red message it appended a / to the URL because it assumes the URL points to a directory.  I will look at changing this.
After making the change, please run:
postToMap.sh  --debug --force

I will look at the image URL error when I am on my PC.  In the meantime, I am curious why you have resized in the URL?  I assume you created an allsky/tmp/resized/ directory and are somehow copying the current image there?

You can lower your Debug Level to 3.  Level 4 is rarely needed anymore and makes the log harder to read.
Also, you can disable Use dark frames because they aren't implemented yet on RPi cameras.  If you made some changes to get them to work in the prior release, note these messages in the log file:
*** darkSubtract.sh: WARNING: 'AS_TEMPERATURE_C' not set; continuing without dark subtraction.

That seems to have fixed it!

Executing:
curl --silent --show-error -i -H 'Accept: application/json' -H 'Content-Type:application/json' --data '{
"location": "OFallon, MO",
"owner": "Kumar",
"latitude": "38.758331N",
"longitude": "90.689438W",
"website_url": "https://allsky.challa.co:1982/allsky/",
"image_url": "https://allsky.challa.co:1982/current/resized/image.jpg",
"camera": "RPi HQ",
"lens": "Fisheye",
"computer": "RPi 5 Model B Rev 1, 4 GB",
"allsky_version": "v2024.12.06",
"machine_id": "bf352f9be0e04948b56f17b00a27d47e"
}' https://www.thomasjacquin.com/allsky-map/postToMap.php

Returned:
HTTP/2 200
date: Sat, 28 Dec 2024 01:05:43 GMT
content-type: text/html; charset=UTF-8
content-length: 12
server: o2switch-PowerBoost-v3

UPDATED 1 5.

postToMap.sh: Map data UPDATED.  You can make 4 more updates today.

The resized image is what I normally use everywhere other than the webui. I use it on:
The desktop display I made:
https://github.com/AllskyTeam/allsky/discussions/3630
As an input to my ML cloud detection project I made:
https://github.com/chvvkumar/simpleCloudDetect
Both of those pull a new image every 30s from the AllSky Pi and sending a large ~2.5MB image is not necessary. So there is a morgify cron task that takes the latest image, resizes it to 750x750 and saves it to /resized/ folder.
As for the debug level-4, I am checking my periodic log for the module I developed for allsky-modules:
https://github.com/AllskyTeam/allsky-modules/tree/dev/allsky_mqttsubscribe
And finally, I will go ahead and disable the dark frame subtraction, I managed to get it working a while ago but I don't think I need to enable it any more.

Is everything fixed now, or just the directory URL?

Everything is fixed now but I am actually not sure why that message popped up today. I have been running with the same /resized/ image URL for quite a while now.
This is an issue that still happens though, which is why I thought it might be the port number in the URL causing it. The current image doesn't show up on the map if the address has a host:port in the URL.
https://github.com/AllskyTeam/allsky/discussions/3967

I am closing this Issue.
I created Issue #4128 for the problem you had with index.php in the URL, and the Map not working with URLs with port numbers is already documented.


Featre Request: Switch from log4net to something more generalized (like Serilog)

It would be very helpful if there wasn't a hard dependency on a specific logging framework (I do not really use log4net).  Any thought to switching over to something like Serilog that allows for multiple different logging frameworks to be "plugged in"?

+1

Non-issue as of PR #107.


Update ni2dpulse.py

Option for crude normalization by the effective area.
Any suggestion for a different approach ?

Thanks for doing this!
You might add a check for CALDB and use the arf there if it is set? Or have "CALDB" as an option for the --arf command?
ALso, you might check with Zaven to find the normalization used in the plot he showed. In addition to just scaling by effective area I think he was scaling so each row had the same RMS or something like that?


Should test for #define CM0PLUS not be CORE_CM0PLUS?

Hi there!
See here:
https://github.com/STMicroelectronics/stm32wlxx_hal_driver/blob/main/Src/stm32wlxx_hal_subghz.c#L269
https://github.com/STMicroelectronics/stm32wlxx_hal_driver/blob/main/Src/stm32wlxx_hal_subghz.c#L303
https://github.com/STMicroelectronics/stm32wlxx_hal_driver/blob/main/Src/stm32wlxx_hal_subghz.c#L369
These line test if "CM0PLUS" is defined but the STM32CubeIDE generated dual core project defines "CORE_CM0PLUS". Is this a mistake?
thanks in advance,
Pieter

Hi @pieterconradie,
Thank you for this question. This looks like a bug needing to be fixed. Another look to other drivers clearly shows the macro to check is CORE_CM0PLUS, not CM0PLUS.
https://github.com/STMicroelectronics/stm32wlxx_hal_driver/blob/42e788df1344d9db4fcc99a81c6d2c3a5c4ba9bf/Inc/stm32wlxx_hal_comp.h#L384
https://github.com/STMicroelectronics/stm32wlxx_hal_driver/blob/42e788df1344d9db4fcc99a81c6d2c3a5c4ba9bf/Inc/stm32wlxx_hal_rcc.h#L780
https://github.com/STMicroelectronics/stm32wlxx_hal_driver/blob/42e788df1344d9db4fcc99a81c6d2c3a5c4ba9bf/Src/stm32wlxx_ll_exti.c#L107
This has been reported to our development teams. I'll get back to you as soon as I have their feedback.
With regards,

ST Internal Reference: 173061

Fixed in c3f558c56e6a34881cad7a735fd5347d451fcd94


Join issue

Alright, so I did everything however when I get to inviting it no servers op up, I am signed in and own 3 discords and a bot testing discord and none are showing, any reason why?

Are you sure you changed the client_id part of the url to your own Client ID?


Find all references fails on latest nightly

This is a serious regression in nightly and it's important to fix it before the next release.
@matklad, please take a look.
https://github.com/rust-ethereum/akula
branch ra-repro
src/downloader/sentry_client_reactor.rs:55 - cannot find references to new, also !!MISSING: command!! above signature.
src/downloader/sentry_client.rs:30 - cannot find references to, or implementations of trait.

thread '<unnamed>' panicked at 'overflow depth reached'

It's a chalk issue, and probably not a regression.

CC https://github.com/rust-analyzer/rust-analyzer/issues/7817. Something like this might help:
"rust-analyzer.server.extraEnv": {
      "CHALK_OVERFLOW_DEPTH": 250
}

@lnicola Can confirm that setting Chalk overflow depth works for first case.
I did report this as regression, because I didn't see any RA issues with this codebase before. Nevertheless, apologies if it's not a regression per se.

The second issue is new (unreported), I suppose, but #7817 has plenty of people running into this with tonic.
We should at least increase the default overflow depth.

Unassigning myself: I've pinged chalk folks about the chalk specific issue. I can't repro the missing command one, let's wait if someone else hits it (the relevant code was changed in this release).
Thanks for tagging with nightly regression -- its better to over-tag then under-tag!


Use bootstrap tokens to add nodes to cluster

Rather than using X 509 certs to authenticate nodes use bootstrap tokens to add nodes to the cluster.
Links:

TLS bootstrapping
Authenticating with Bootstrap Tokens
Add new node to k8s with bootstrap token (Shameless self plug)

Given that I need to create certificates already I prefer to use certificates for everything.


Invoicing details section

Details

The PR introduces the API implementation for changing the invoice company name and website.
Fixed Issues

$ https://github.com/Expensify/App/issues/45176
PROPOSAL: N/A
Tests

Preconditions:

Enable invoices of a workspace
Add the details section (WorkspaceInvoicingDetailsSection) to the invoice page (WorkspaceInvoicesPage). I am adding the section in the upcoming PR (https://github.com/Expensify/App/pull/50565).

+import WorkspaceInvoicingDetailsSection from './WorkspaceInvoicingDetailsSection';
...
{(hasVBA?: boolean, policyID?: string) => (
  <View style={[styles.mt3, shouldUseNarrowLayout ? styles.workspaceSectionMobile : styles.workspaceSection]}>
+    {policyID && <WorkspaceInvoiceBalanceSection policyID={policyID} />}

Open the workspace’s invoices page
Click on the company name route.
Update the value.
Verify that the company name has been updated successfully.
Click on the company website route.
Update the value.
Verify that the company website has been updated successfully.

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/19b70fe7-133b-423d-9bc4-21779092ce2d

Android: mWeb Chrome

https://github.com/user-attachments/assets/3bcb0631-8e6a-4de5-ade1-6c44b9cb31a0

iOS: Native

https://github.com/user-attachments/assets/c6e97f41-caa1-4491-82b2-cbde142f0b2d

iOS: mWeb Safari

https://github.com/user-attachments/assets/f1e9fb23-815d-4596-af59-f909868b45b5

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/4a79c221-3301-4f80-9114-1b63282ad13d

MacOS: Desktop

https://github.com/user-attachments/assets/e2d7395c-881b-44dc-b811-6def4744564f

@alitoshmatov bump

@rezkiy37 Everything seems to be working fine.
One question though, why fullscreen loading is happening after updating company data? I see that data is being applied optimistically, what is causing loading then?

@rezkiy37 Everything seems to be working fine.
One question though, why fullscreen loading is happening after updating company data? I see that data is being applied optimistically, what is causing loading then?

That's a reasonable question. AFAIK, this behavior is expected because was added 7 months ago.
https://github.com/Expensify/App/blob/186c1a976acbc65faf8b739e34bb0c7936fbd1f2/src/pages/workspace/WorkspacePageWithSections.tsx#L147-L151
cc @madmax330

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/dc643a2c-516d-4ed9-98a6-a26c82d44f3b

Android: mWeb Chrome

iOS: Native

https://github.com/user-attachments/assets/e344ab4e-1831-4892-8dc7-e7a2912adfdf

iOS: mWeb Safari
https://github.com/user-attachments/assets/3a50b68f-c262-442e-892d-44706ce1c1e0

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/77761354-8197-4e3e-a98d-d34d620cac57

MacOS: Desktop

https://github.com/user-attachments/assets/0aeb788f-28f9-427c-9d89-8423d330d8cd

It does seem like it is not related to current PR, but we should definitely follow up on this issue.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@rezkiy37 FYI, QA was not run on this PR due to the QA test section being empty. I'm not sure if that was intended or not. The feature was not working and was reported as a deploy blocker in https://github.com/Expensify/App/issues/51871.
In the future, please add specific QA tests to your PRs or be more clear about why there are no QA tests. If you desire to have QA run the same tests as the "Tests" section, then please indicate so by adding "Same as tests" in the QA section.
Thank you!


Evaluating code inline with Python

When using Python to evaluate code inline, Light Table seams to evaluate the whole file upon connect. This causes the evaluator to spend a lot of time if there are time consuming functions or if there are errors in the file. (with a spinning working indicator that doesn't stop).
As a workaround, I have to cut everything but the first 10 lines of code, save the file, evaluate the file, paste the cut part again, save the file again; and then I can evaluate the code inline normally.
Is there a better way for this?
I have tested it with OSX Lion and Mountain Lion,
Versions 0.4 and 0.5 of Light Table (0.5.2)
Python 2.7.1
pyzmq==13.1.0
ipython==1.0.0

I created LightTable/Python#38 for this in the Python plugin repo.
Note that we really need someone to step up and handle Python support as neither none of the members of the core team or any frequent contributors use Python.


TOAST validation

It is possible to do validation TOAST with Selendroid ?

We have a working e2e test covering this. https://github.com/selendroid/selendroid/blob/85b1f6dc95e41c721261e5b969bbd6a95cfe12a5/selendroid-test-app/src/test/java/io/selendroid/nativetests/WaitForProgressBarGoneAwayTest.java#L87-L93


Enter amount on the receive screen is always in sats

The enter amount option on the receive screen is always in sats, even when the wallet is in BTC, or even if it shows that the amount is in fiat.
It should allow to at least switch between the sats/ btc and fiat, and convert the right amount in fiat. It should also have the decimal point when fiat/ btc amounts are used.

Verified this issue on dev app v 1.2.18(410)


Too many response codes results in element overlap

Example

200,400,401,429,500
Perhaps it could / should be a drop down if there are more than 4?

I've made it configurable via useTheme composable. As always, it can be configured per page, or globally in VitePress Theme config. Can you please check it in #51?
Another great improvement would be to auto switch to Select or Tabs based in the space available in viewport (when it overflows the x space, auto-switch to Select alternative; otherwise use Tabs). But I think this would be a little more complex and could be a future feature.


feat(reactivity): remove redundant code

When I was studying the source code from vue3  reactivity
I found that on computed this section there was bloated code
So I thought we could remove it, and the test passed successfully

This PR is currently marked as a feat. I wonder whether refactor might be more appropriate?
The line public dep?: Dep = undefined features in several of the ref implementations. e.g. RefImpl and CustomRefImpl. If we're going to remove it from one then maybe it should be removed from all of them?
The property itself comes from the RefBase type, which marks it as optional. That property is then used in trackRefValue and triggerRefValue. As the property is optional, nothing complains if the initial value isn't explicitly set to undefined, but nevertheless the property will be added once the ref is used elsewhere.
I'm unsure what the reasoning was behind explicitly declaring these properties with undefined values. Perhaps it was just for greater explicitness, or maybe it was to ensure that the 'shape' of the objects won't change at runtime post-creation (for better performance).

This PR is currently marked as a feat. I wonder whether refactor might be more appropriate?
The line public dep?: Dep = undefined features in several of the ref implementations. e.g. RefImpl and CustomRefImpl. If we're going to remove it from one then maybe it should be removed from all of them?
The property itself comes from the RefBase type, which marks it as optional. That property is then used in trackRefValue and triggerRefValue. As the property is optional, nothing complains if the initial value isn't explicitly set to undefined, but nevertheless the property will be added once the ref is used elsewhere.
I'm unsure what the reasoning was behind explicitly declaring these properties with undefined values. Perhaps it was just for greater explicitness, or maybe it was to ensure that the 'shape' of the objects won't change at runtime post-creation (for better performance).

This PR is currently marked as a feat. I wonder whether refactor might be more appropriate?
The line public dep?: Dep = undefined features in several of the ref implementations. e.g. RefImpl and CustomRefImpl. If we're going to remove it from one then maybe it should be removed from all of them?
The property itself comes from the RefBase type, which marks it as optional. That property is then used in trackRefValue and triggerRefValue. As the property is optional, nothing complains if the initial value isn't explicitly set to undefined, but nevertheless the property will be added once the ref is used elsewhere.
I'm unsure what the reasoning was behind explicitly declaring these properties with undefined values. Perhaps it was just for greater explicitness, or maybe it was to ensure that the 'shape' of the objects won't change at runtime post-creation (for better performance).

Oh, you're right. I made a mistake

This PR is currently marked as a feat. I wonder whether refactor might be more appropriate?
The line public dep?: Dep = undefined features in several of the ref implementations. e.g. RefImpl and CustomRefImpl. If we're going to remove it from one then maybe it should be removed from all of them?
The property itself comes from the RefBase type, which marks it as optional. That property is then used in trackRefValue and triggerRefValue. As the property is optional, nothing complains if the initial value isn't explicitly set to undefined, but nevertheless the property will be added once the ref is used elsewhere.
I'm unsure what the reasoning was behind explicitly declaring these properties with undefined values. Perhaps it was just for greater explicitness, or maybe it was to ensure that the 'shape' of the objects won't change at runtime post-creation (for better performance).

Oh, you're right. I made a mistake


invoke V3

Time spent on this PR:
Resolves: #773
Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Testing

What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

It seems for rust tests, the katana accounts are not funded in STRK?


[Samples] Rename schemas in samples to be unique to avoid collisions

The schema names in samples which use schemas should be unique to avoid collisions.

Fixed with: https://github.com/SAP/xsk/pull/839


LB-SAC implementation

Implementation of Q-ensemble for Offline RL: Don't Scale the Ensemble, Scale the Batch Size.
TODO:

[ ] rerun full D4RL Gym benchmark (as in paper)
[ ] update tables in the README
[ ] create report based on wandb logs

original implementation for reference: https://github.com/tinkoff-ai/lb-sac

wandb report: https://wandb.ai/tlab/CORL/reports/LB-SAC-D4RL-Results--VmlldzozNjIxMDY1


''Get all Float'' Function removed

Hi Devs,
The ''Get all Float'' function is gone after the recent update. Please fix it thanks <3 it's more convenient when i want to have a quick glance.

This has been resolved in v2.0.1 and published to the Chrome web store. You'll be able to update in roughly an hour.


Add June 2023 GDK to the new unified GDK winget ID.

Add June 2023 GDK to the new unified GDK winget ID per #186890.
Microsoft Reviewers: Open in CodeFlow

@microsoft/xbox-developer-experience-toolbox for review.


Fail to load master.c

/adm/obj/master.c line 40: syntax error before  crash(string error,
No error handler for error: *Error in loading object '/adm/obj/master'
program: (none), object: (none), file: (none)

same question

please use fluffos:
https://github.com/mudchina/fluffos

yeah I compile fluffos in my OSX but still this question. :(

Please modify the config file: bin/config.ES2:
# absolute pathname of mudlib
mudlib directory : /Users/liming/workspace/mud/es2-utf8/mudlib

# absolute pathname of driver/config dir
binary directory : /Users/liming/workspace/mud/es2-utf8/bin

Change to your actual path.

Just now, I just git clone the sam code and run it on my MacOSX, it works fine.

It's working now.
With the original driver in this repo, it works well!
Thank you!


Homepage not syncing under "blocks" and "transactions" after doing it once

Hey. I have a problem with homepage of the explorer. It's not adding the blocks or transactions, even if i press the refresh button next to the "blocks" or "transactions" parts. I can however explore the blocks just fine, but they are not updated on the home page. This is the output on the backend, when pressing refresh:
"[1] POST /data 200 3.714 ms - -"
Could someone point me if there is something wrong with my configuration, or do i need to setup some kind of cronjob for the homepage to be updated?

I did setup a cronjob for the sync.js to run every 15 seconds, but isn't there any better way to do this?


CSN model training does not converge

Thanks for providing this comprehensive code base.
When I tried to train the newly updated model "CSN", loss has been maintained at around 0.7, although a dozen epochs have been trained.

@gfjiangly Do you know what is CSN? I try to search paper of CSN but failed, could you tell me CSN's full name? What kind of network does it represent?

We haven't released training config for CSN yet, you can refer to the original paper for the training setting. You can also test the trained model to see if accuracy is OK.


Enable unsafe_block_in_unsafe_fn feature

See https://github.com/rust-lang/rust/issues/71668

We will have to wait for that lint to stabilize, right? Otherwise it seems like it would be hard to keep things warning free on both stable and nightly.

We will have to wait for that lint to stabilize, right? Otherwise it seems like it would be hard to keep things warning free on both stable and nightly.

We could allow the unused_unsafe lint on stable to avoid the warnings.

We could allow the unused_unsafe lint on stable to avoid the warnings.

#![deny(unsafe_op_in_unsafe_fn)] is stable, see https://github.com/rust-lang/rust/pull/79208!
So we can just add this whenever


Indicated that client certificates for REST are available since 7.2

Ready for merge upon release of 7.2.0.