how to use full range of 12 bit pixel color depth?

hi,
how can i use the full range of 12 bit pixel color depth?
i am able to set the pixel color depth of the panel driver up to 12 bit,
but i only can draw text in colors at 565 (5-6 bits per color pixel).
the other function of the driver uses max. 8 bit per color pixel.
the background is i try to dimm my LED panel depending on the ambient light - ambient light range 0lx...10000lx. so the panel text will be readable at every ambient brightness without glaring or being illegible.
the setBrightness(uint8_t) function is not that precise as i need it will distort colors at the darker colors.
also trying to keep the panel brightness at maximum and dimming the color values in software does not the trick.
because the text functions still uses the 565 pixel color depth and makes not use of the full possible 12 bit per color pixel.
that results in color distortions and at lower values the text colpletely disapears sooner or later depending on its color values abter dimming.
what i have to do to draw text and other stuff in full 12 bit per color pixel?
my hardware:

ESP32-dev board
2x 32x32 LED matrix panels in a chain

my software:

Arduino-IDE; 1.8.19
Arduino-ESP32: v2.0.11 (Esp Sdk: v4.4.5)
ESP32 HUB75 LED MATRIX PANEL DMA Display: v3.0.9
Adafruit GFX Library: v1.11.17
Adafruit BusIO: v1.14.1

This hardware library simply uses AdafruitGFX for all drawing functions, which seems to limit text to 565 (16 bit unsigned).
I did try hacking AdafruitGFX with my GFX_Root library in another github repository of mine, I'm not sure if I change the text color functions however.


[ic-date-input/picker]: Add 'hideHelperText' prop

Summary
By default, the date input/picker displays helper text'. This ticket is to create a prop which removes the helper text.
💬 Description
By default, the date input/picker displays helper text'. Some designs may not require help text as the placeholders for picker/input present the required format. If custom helper text is required, there is no need to remove the helper text. The solution is to create a simple 'hideHelperText' prop to hide it.
In terms of a11y, the format is announced by screen readers so there should not be an issue. The announcement is made via labelledby so it may be better to add 'sr-only' class to the helper text instead of removing the element from the DOM.
💰 Use value
Add extra flexibility to the visual for input/picker as some designs may not use helper text for form controla.
📝 Acceptance Criteria
If relevant, describe in full detail the different interactions and edge cases that the component or patterns needs to fulfil.
Given as a developer
When I set 'hideHelperText' prop to true
Then the helper text should disappear visually but still be read out by screen readers
✏ Designs
N/A
## 🧾 Guidance
N/A
Additional info
The helper text within the doc returns 'undefined' as the custom helper text value. This needs to be changed to 'Use format <date_format>'. This is a small update which can be added to this ticket.

Community member asked for this to be added


[5.4] Improve the dd helper function

This PR does two things:

If app.debug is set to false (which will be default in production), a warning is logged with a message and the calling file and line, adding the supposed dd output to the logging context.
Display the calling file and line in the start of the regular output, making it easier to know where your dd was called from, in case you accidentally left it in there

Hmm with this it'll still call a die(1) and will result into a blank page, maybe you should also execute die() only in the debug mode, so at least it's fully disabled...

@fernandobandeira the idea is to have the behaviour unaltered. The idea is not to alter the outcome, as I don't think dd should be disabled, the idea is rather to not print sensitive information. I personally would be in favour of having it be a proxy for logging, similar to the warning log I added, and not dieing, but I think that is quite a large change to make, much larger than this.

Rather than doing this I think I would rather just let people define their own dump helpers if they need this behavior.

I was just handling a security situation last night because of a stray dd being left behind and never caught by tests, accidentally leaking key information that resulted in a severe breach. And I know for certain that "I left a dd in by accident" is a very common mistake that can also be frustrating to track down. It seems to me that not disclosing internal data in production would be a sensible security default, similar to how exceptions and their stack traces don't get shown on production settings.
Granted, it's trivial to extend the function or replace it, but good defaults and all, eh. It's sort of breaking, though, for people actually using dd for "spit out JSON and exit request".

With regards to it being breaking @rizqidjamaluddin that was something I considered too. My reasoning for targeting 5.4 was "I might as well try to get a security thing in as early as possible", with the worst case scenario being moving it to 5.5. That said, I would like @taylorotwell to reconsider this, at least for 5.5 since, as mentioned, good defaults and all.

Should be possible to just override VarDumper::$handler from service provider, and have the logic executed there.
https://github.com/symfony/var-dumper/blob/master/VarDumper.php#L41

@crynobone the current dd function does not resolve the Dumper from the IoC container, it executes (new Dumper)->dump


Open issue to latest comment

Sometimes I'm only interested in the latest comment. What if the comment number is linked to the latest comment...

Test:
https://github.com/sindresorhus/refined-github/issues for issues with multiple comments.

"Latest comment" can just be the footer, so no API calls are required

PR welcome.

For anyone that wants this, please also send this feature request to GitHub<EMAIL_ADDRESS>


What if we do not have the analytical solution for differential eqn??

Hello Ben,
For the given differential equation, what if we do not have the analytical solution...
Then we do not have those data points (x_data, y_data) by which we have got loss_1, and if we train our model only by physics-based loss "loss_2", we won't get accurate results.
In  Raissi et. al. paper on PINNs, they have taken the loss_1 term corresponding to the boundary and initial conditions.
So can we get an accurate result for the given differential equation(1 SDOF) if we incorporate the two initial conditions given in the question in the loss
and not the data points we have taken from the analytical solution of the differential equation?

Yes, in the blog post I included multiple data points of the solution, just to make it easy to understand.
You can instead just use data points along the boundaries/initial conditions; this will ensure the BC/ICs are obeyed and the solution the PINN learns is unique. In this case you do not need to know the true solution, only the BC/ICs are required.


Don't filter external links by protocol

@eanders-MS,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Thank you!


aws sdk dependencies moved

Hi,
AWS seems to be moving some packages to another repository, and it's making warnings during installation. Also, some messages indicate these libs are built for internal use.

Thank you for reporting this. I've already updated the dependencies to the latest version (see #8), the PR is currently stuck here due to another issue: #13
I may create a second PR just for the dependencies to release them a bit earlier.
Regarding your second question: these libraries are not meant to be used by end users as they are a bit more complicated. But that's why I created this library, so only I have to dig through the docs and not everyone else :-D
Translated with www.DeepL.com/Translator (free version)

Hi @clementAC
my PR (https://github.com/smithy-lang/smithy-typescript/pull/1226) into the smithy/* package got merged yesterday and I just released a new version of this package with the updated dependencies.
So the newest version v3.1.0 should not print any deprecation warning any longer.


Customize CSS

Could you please give me an example of changing the default styles?
Thanks in advance.

Is there anyone who could help me?
for example, how can I change the calendar's primary color?

hello,
I have this problem too. i mean this is an amazing component but i can't even change the dafault grey border from styles. i don't know if i'm really noob or this is a very complicated project to customize. really appreciate if someone can help.
thanks in advance

hello,
I have this problem too. i mean this is an amazing component but i can't even change the dafault grey border from styles. i don't know if i'm really noob or this is a very complicated project to customize. really appreciate if someone can help.
thanks in advance

I have no idea that this is the best way or not, but I did change the style by overriding them like this.
You can also add some custom classes to the DatePicker component and change those as well.
@omidazm94
.tether-element {
z-index: 1;
.calendarContainer {
width: 280px;
padding: 5px;
background-color: #242428;
}
}


Trying scrcpy v1.19 - Segmentation fault (core dumped)

Hello, I'm trying to update your docker file from scrpy v1.16 to v1.19, but unfortunately I'm getting the following problem (tried with all three drivers, but my host is intel)...
docker run --rm -i -t --privileged -v /dev/bus/usb:/dev/bus/usb -v /tmp/.X11-unix:/tmp/.X11-unix -e DISPLAY=$DISPLAY scrcpy:amd
/ # adb devices -l
* daemon not running; starting now at tcp:5037
* daemon started successfully
List of devices attached
0A171JEC219692         unauthorized usb:1-4 transport_id:1

/ # adb devices -l
List of devices attached
0A171JEC219692         device usb:1-4 product:sunfish model:Pixel_4a device:sunfish transport_id:1

/ # scrcpy
INFO: scrcpy 1.19 <https://github.com/Genymobile/scrcpy>
/usr/local/share/scrcpy/scrcpy-server: 1 file pushed, 0 skipped. 173.8 MB/s (37330 bytes in 0.000s)
[server] INFO: Device: Google Pixel 4a (Android 12)
INFO: Renderer: opengl
INFO: OpenGL version: 4.6 (Compatibility Profile) Mesa 21.2.5 (git-53b2b224dc)
INFO: Trilinear filtering enabled
INFO: Initial texture: 1080x2336
Segmentation fault (core dumped)
/ #

Or (just to show it doesn't make a difference)...
docker run --rm -i -t --privileged -v /dev/bus/usb:/dev/bus/usb -v /tmp/.X11-unix:/tmp/.X11-unix -e DISPLAY=$DISPLAY scrcpy:intel
/ # adb devices -l
* daemon not running; starting now at tcp:5037
* daemon started successfully
List of devices attached
0A171JEC219692         unauthorized usb:1-4 transport_id:1

/ # adb devices -l
List of devices attached
0A171JEC219692         device usb:1-4 product:sunfish model:Pixel_4a device:sunfish transport_id:1

/ # scrcpy
INFO: scrcpy 1.19 <https://github.com/Genymobile/scrcpy>
/usr/local/share/scrcpy/scrcpy-server: 1 file pushed, 0 skipped. 181.4 MB/s (37330 bytes in 0.000s)
[server] INFO: Device: Google Pixel 4a (Android 12)
INFO: Renderer: opengl
INFO: OpenGL version: 4.6 (Compatibility Profile) Mesa 21.2.5 (git-53b2b224dc)
INFO: Trilinear filtering enabled
INFO: Initial texture: 1080x2336
Segmentation fault (core dumped)
/ #

I do see my mobile's screen for a split second on my desktop, so it's almost working ;-)
Please let me know if this is a scrcpy issue!
Many thanks,
Dan.

I'm suddenly not sure, so I logged a bug with scrcpy directly:
https://github.com/Genymobile/scrcpy/issues/2763


Error related to log4j2

I'm trying to run the pipeline on a PBS cluster and during the TBBWA step, I'm getting this very odd error:
ERROR StatusLogger Unable to create class org.apache.logging.log4j.core.impl.Log4jContextFactory specified in jar:file:/laboratorio/sabmi/karla.lima/sabmi_sra_marilia/mtbseq_nf_test/mtbseq-nf/conda_envs/mtbseq-nf-env/opt/gatk-3.8/GenomeAnalysisTK.jar!/META-INF/log4j-provider.properties
ERROR StatusLogger Log4j2 could not find a logging implementation. Please add log4j-core to the classpath. Using SimpleLogger to log to the console...
INFO  16:18:38,339 GenomeAnalysisEngine - Deflater: JdkDeflater
INFO  16:18:38,340 GenomeAnalysisEngine - Inflater: JdkInflater
INFO  16:18:38,341 GenomeAnalysisEngine - Strictness is SILENT
INFO  16:18:38,499 GenomeAnalysisEngine - Downsampling Settings: Method: BY_SAMPLE, Target Coverage: 10000
INFO  16:18:38,510 SAMDataSource$SAMReaders - Initializing SAMRecords in serial
INFO  16:18:38,587 SAMDataSource$SAMReaders - Done initializing BAM readers: total time 0.07
##### ERROR ------------------------------------------------------------------------------------------

I'm using a new conda environtment provided by the recipe on this repository, I'm also using the correct GATK jar.

This seems related https://github.com/ngs-fzb/MTBseq_source/issues/23 and reported for a conda_env, which I haven't been able to reproduce on my setup yet.
Note that I haven't been able to reproduce either, this issue using the docker container

I'm testing the GATKjar version 3.8.1 (GenomeAnalysisTK-3.8-1-0-gf15c1c3ef), avaible on this link: https://storage.googleapis.com/gatk-software/package-archive/gatk/GenomeAnalysisTK-3.8-1-0-gf15c1c3ef.tar.bz2
As suggested on this issue, so we can see if updating the jar solves the problem with mtbseq

After a long test on IEC Cluster the issue seems to be fixed, If it's okay to you @abhi18av, I'll close this issue and keep it as a possible troubleshooting for future use.

Brilliant!
@Mxrcon , would you mind adding a general outline of the solution please?

Sure!

[ ] Download a new GATK tar

wget https://storage.googleapis.com/gatk-software/package-archive/gatk/GenomeAnalysisTK-3.8-1-0-gf15c1c3ef.tar.bz2

[ ] Untar the GATK.jar

 tar -xvf GenomeAnalysisTK-3.8-1-0-gf15c1c3ef.tar.bz2

[ ] Register the new GATK and use on the pipeline as usual

 gatk-register GenomeAnalysisTK-3.8-1-0-gf15c1c3ef/GenomeAnalysisTK.jar

Hmm, I see.
In that case, do you think it's worth upgrading the default gatk version within the pipeline?
I have updated the title of this issue to help anyone discover the cause of this issue earlier.


Cannot Detect Preload Model (i.e. ros_hydro, ros_fuerte)

The simtrack runs fine and /simtrack/image can be seen, but no green tracking or anything.
Also /simtrack/DepthCloud display nothing.
I followed the README.md on the first page strictly, except
1.) My system

Ubuntu 14.04
ROS Indiago
GeForce GTX 1080 (one GPU system)
Kinect2 (libfreekinect2)

2.) roslaunch kinect2_bridge is run with depth_device:=0 parameters instead of depth_device:=1
3.) models_to_print.pdf is measured 9.5cm instead of 10cm
4.) The attach patch is applied to make it compiled with my system.
simtrack1080.patch.zip
Thank you in advanced.

Is the kinect2 calibrated ?
Try to print again the models without any margin to make sure the model has the right size.
Why did you change the CMakelist.txt s ?

Thanks for your quick response.
The kinect2 is calibrated.
I will try to reprint the model again  (maybe 2moro)
CMakelists.txt are changed for

path if cub library
GeForce 1080 is seemed to be compute 60 (it compiles without this change, but cannot find the device and finally got seg fault)

Tried with the printed_model with perfect 10cm,
but still got nothing in /simtrack/image
(only a monochrome image of the scene without any overlay-ed color).

Found the bugs. Sorry for the troubling.
the compute_arch for GeForce GTX 1080 is -gencode=arch=compute_61,code=sm_61.
All works fine !


Partial payments are not allowed, amount should be equal checkout's total.

when i added 1 item or first item into the cart this message appear. Also when running sale on item also display this message and can not process the checkout process. When multi items added into the cart then checkout process works good.  this problem came with avalara tax plugin activation.

I was able to reproduce this bug on version 3.0 for product on sale. Thanks for reporting! Added to our backlog :).


CypherpunkPay

Git repo
Deb repo for apt
Demo
Docs
This is more like a lightweight BTCPaysServer (STORE) that is a Linux Daemon.

Just to collect the idea or any commitment to implement?

just collecting the idea for now, but this one seems easy as it is already a daemon if someone wants to take it.


When GPS is within Chicago, use the Chicago Open Data portal.

This is the data I was talking about: https://data.cityofchicago.org/Facilities-Geographic-Boundaries/Boundaries-Neighborhoods/bbvz-uum9

@pjsier where's the api?

It's a Socrata site, so they've got the endpoints set up according to this standard https://dev.socrata.com/

Actually looking at it now, not sure if they support geospatial queries. May have to either download the GeoJSON and put it on the site or somewhere else to query it

Correction: sorry for the spam, but this might work: https://dev.socrata.com/docs/functions/within_circle.html it's a bad workaround, but they weirdly don't have a geospatial within, but this should work the same way

If anything, it would be https://dev.socrata.com/docs/functions/within_polygon.html

That's what I thought at first, but that's having the user create  a polygon boundary as opposed to finding the polygon boundary. I figured using within_circle would be the easiest proxy for a point

So I was playing around with it, and the neighborhoods map isn't actually available through the API at all. It's obviously not ideal, but I converted it to GeoJSON, and while it'd be too much to render it on the site, maybe we could put it in CartoDB or convert it to PostGIS?

Turfjs is used with a static GeoJSON file (only about 150kb) to loop through all neighborhoods and print which one the point is within, otherwise returns "outside Chicago"


Addition to using-on-macos.md

Add steps to add vs code to the PATH so it can be launched from the terminal
Summary
Added details on how to open Visual Studio Code from the terminal by typing code .

My latest commit shows OpenPublishing.Build - Validation status: errors but it doesn't tell me what the errors are. Any thoughts?

@ricardodsanchez I'll close and reopen to start a new build. That usually happens when an exception occurs or internal error.

@jzeferino Can this be approved? Or are there any other steps needed in order to approve it? Thanks.

One of us from the .NET documentation team will review your changes shortly.

@ricardodsanchez, @mairaw answered :)

@ricardodsanchez please let me know if you'd like to make the changes I'm proposing. Thanks!

@mairaw Thank you for the reply and the reminder about it. What you propose is a good idea in order to keep relevant documentation linked to each other. However, as a developer, I prefer to get the information (examples, documentation, etc) I need right when I need it, without jumping to another resource. This document Getting started with .NET Core on macOS is targeted to people wanting to use .NET Core on a Mac, and my suggestion is about giving them the option to launch Visual Studio Code from the terminal, which is something a developer using a Mac will want to do often. My two cents. What are your thoughts about that?

@ricardodsanchez I still think that this piece of information is a general information on how to work with VS Code. Even though it might be common for developers to do that, I think it distracts from the goal of the article. And we try to avoid to duplicate content.
The other thing about this particular topic that always bugged me a little is that the tutorial is not a VS Code one or particular to macOS, it's mostly a CLI tutorial with an entry about debugging in the end, which we show how to do already on this topic: https://docs.microsoft.com/en-us/dotnet/core/tutorials/with-visual-studio-code#debug.
Adding my team here to see what they think @JRAlexander @BillWagner @rpetrusha.

I think @mairaw is correct. While it is helpful to know that for VS Code, I'd agree that it distracts from the goal of the article.

I agree with @mairaw. This is a getting started tutorial, which is aimed at an entry-level developer. Information on starting Visual Studio Code from the command line is well outside the scope of the topic. It's useful information, but it properly belongs in a Visual Studio Code topic, not in a topic on getting started with .NET Core using Visual Studio Code. And of course, if we add it here, information on starting Visual Studio Code from the command line has to be changed in two places if the information changes -- and duplicated information is almost very difficult to correct.

@ricardodsanchez please let us know if you'd like to link to that VS Code article instead. Otherwise, I'll be closing this PR. Thanks so much!

Closing for now. Feel free to reopen if you'd like to address our feedback. Thank you again @ricardodsanchez for your interest in contributing to our documentation.


Small doc fixes

Several small fixes to the docs - see the commits.

What are the text changes to doc/instances.md?
Github doesn't show a useful diff here.

What are the text changes to doc/instances.md?
Github doesn't show a useful diff here.

Only removing spaces at the end of the line (default configuration in my editor). So Git actually displays it correctly. ;)

Ah, Github Mobile didn't show them, Github Web does, fun...
Can you move those whitespace fixes into a separate commit?

(They likely weren't there as an accident, they were there because we needed actual line breaks at some point with some markdown parsers but that shouldn't be needed with Sphinx)

Updated.


Fix comment NOTE about SetVLOGLevel

Fix the comment about SetVLOGLevel according to https://github.com/google/glog/pull/650 which added support for updating vmodule levels after vmodule level has been cached.
Fixes https://github.com/google/glog/issues/858

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

Project coverage is 67.22%. Comparing base (45f99f5) to head (7e258ca).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1109      +/-   ##
==========================================
+ Coverage   63.57%   67.22%   +3.64%
==========================================
  Files          20       14       -6
  Lines        2578     1907     -671
  Branches      894      705     -189
==========================================
- Hits         1639     1282     -357
+ Misses        671      447     -224
+ Partials      268      178      -90

see 17 files with indirect coverage changes

Thanks for the PR.
Doesn't it make more sense to supply the updated information in the user guide, for instance, by updating this section?

I have put a basic description of SetVLOGLevel in the doc and more detailed information in the .h file. Let me know if this looks good enough.


Use  as the filename for SILLocation-less functions

Use  as the filename for SILLocation-less functions to avoid misleading source locations in backtraces.
(Also) fixes rdar://problem/39116765

@swift-ci test and merge

@swift-ci test and merge


Added manual vm creation script.

Added my manual script (which I know works).  I will be updating it for our new DNS entries.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


The chat is not showing Unread messages when replied to a chat via email

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

Version Number: v1.4.83-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @VictoriaExpensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1718319503018869
Action Performed:

Go to staging.new.expensify.com
Initiate a DM to B
As B reply by email

Expected Result:
The reply displayed as unread message in LHN
Actual Result:
The reply not displayed in LHN
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
https://github.com/Expensify/App/assets/38435837/3a791359-811e-46f5-a430-c8226c9f9ca2

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0190fc8ada72fd4d14
Upwork Job ID:<PHONE_NUMBER>363871252
Last Price Increase: 2024-06-26

Issue OwnerCurrent Issue Owner: @

@thesahindia was testing cuz Internal wasn't assigning a C+ for some reason

Related, because when the C+ is assigned via Internal, I believe an Upwork job is created for them (hm... that might only be if they're not paid via NewDot?)

https://github.com/Expensify/Expensify/issues/405038

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Trying to test, awaiting email from test user. @thesahindia , can you also try to test? In case you're able to before me.

How do you get the email to send to you?  I tried multiple times yesterday and checked that notifications were set to immediate.
Do you think this can be worked on externally @thesahindia ?

Thanks @thesahindia , marked External, I checked a couple times today, never got the email/s for some reason.

Checking on internally https://expensify.slack.com/archives/C01SKUP7QR0/p1719001928198269?thread_ts=1719001866.214179&cid=C01SKUP7QR0

Bumped to $500 to get some eyes.  Also trying to reproduce again with sending messages to two accounts so I can reply.

This seems like a BE problem. I could see the message in the payload if reload the App.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The chat is not showing Unread messages when replied to a chat via email
What is the root cause of that problem?
Websocket doesn't send the onyx message to update.
What changes do you think we should make in order to solve the problem?
Send the message to FE with following format
  "onyxMethod": "merge",
            "key": "reportActions_",

What alternative solutions did you explore? (Optional)
NA

Checking in #expensify-open-source about here

The chat is not showing Unread messages when replied to a chat via email appears to need to be backend. It seems important to fix but it's in #vip-vsb which is on hold. Please comment if you have feedback or if an internal engineer is willing to take it on.

Checking in #engineering-chat (thought it was #engineering when I posted :ohnohthing)
https://expensify.slack.com/archives/C03TQ48KC/p1719964100569379

I looked into the backend code and it seems we are intentionally marking the message as read. I wanted to confirm this behavior. If a user has replied via email, they have seen the message sent by the other user, correct? In this case, do we still need to show the message as unread in the LHN?

Yeah, from the steps in OP, I think so (I tried to reproduce multiple times without luck though).
Steps

Go to staging.new.expensify.com
Initiate a DM to B
As B reply by email

Expected Result:
The reply displayed as unread message in LHN
so.. user A sends message to B, B replies via email, chat is not showing bold for user A.
This would mean there's no other way for user A to have seen the message from user B, right?

Investigating this, here is a request logs https://www.expensify.com/_devportal/tools/logSearch/?#query=request_id:("8a015512ad7ec58a-SEA")+AND+timestamp:[2024-07-08T15:08:19.672Z+TO+2024-07-08T17:08:19.672Z]&index=logs_expensify-029291, it is using the addComment command so unsure why we are not queuing updates for the receiver.

We are queuing onyx update for the report but this is not getting pushed to the receiver.

I am unable to reproduce this issue locally. I tried mimicking the process of sending a response via email, and the comment was added to the chat in real-time. I will check if I can reproduce this on the production or staging environment and might ask for a retest. This functionality is expected to work correctly since we are using the same method that the NewDot client uses when users respond to a chat, so everything should be the same.

Ok, finally figured out the root cause here. Since replying through email is a different flow we need to ensure we create a createSendFetchableJob to push the updates to the receiver here https://github.com/Expensify/Web-Expensify/blob/fe0db5837a4825162ea4b3a79901a25748191a3f/lib/Mailgun/API.php#L1505

This was fixed internally here: https://github.com/Expensify/Web-Expensify/pull/42742, so I am going to close this out now.


Update google-cloud-bigquery to 1.137.1

Updates com.google.cloud:google-cloud-bigquery from 1.122.2 to 1.137.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-bigquery" } ]

labels: library-update, semver-minor

Superseded by #251.


bump machine to v0.15.0-rancher93

Issue: https://github.com/rancher/rancher/issues/38422#issuecomment-1198729324
This PR bumps rancher machine to v0.15.0-rancher93 which includes a fix for error message reporting on the UI

Closed due to drone build not triggering

Good to go!


lsp-plugins: 1.2.15 -> 1.2.16

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/lsp-plugins/versions.
meta.description for lsp-plugins is: Collection of open-source audio plugins
meta.homepage for lsp-plugins is: https://lsp-plug.in
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.2.16 with grep in /nix/store/a6rz85p5n7qncvpj41b5n2w0j69ijmd7-lsp-plugins-1.2.16
found 1.2.16 in filename of file in /nix/store/a6rz85p5n7qncvpj41b5n2w0j69ijmd7-lsp-plugins-1.2.16

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath

easyeffects
gnomeExtensions.easyeffects-preset-selector
lsp-plugins
pulseeffects-legacy

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/a6rz85p5n7qncvpj41b5n2w0j69ijmd7-lsp-plugins-1.2.16 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A lsp-plugins https://github.com/r-ryantm/nixpkgs/archive/2ae9dea1d92ad6966df0c282e9629a4cefd4001b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/2ae9dea1d92ad6966df0c282e9629a4cefd4001b#lsp-plugins

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/a6rz85p5n7qncvpj41b5n2w0j69ijmd7-lsp-plugins-1.2.16
ls -la /nix/store/a6rz85p5n7qncvpj41b5n2w0j69ijmd7-lsp-plugins-1.2.16/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

7 packages built:

easyeffects
easyeffects.debug
gnomeExtensions.easyeffects-preset-selector
lsp-plugins
lsp-plugins.dev
lsp-plugins.doc
pulseeffects-legacy

Maintainer pings
cc @magnetophon @PowerUser64 for testing.

nixpkgs-review:
7 packages built:
easyeffects easyeffects.debug gnomeExtensions.easyeffects-preset-selector lsp-plugins lsp-plugins.dev lsp-plugins.doc pulseeffects-legacy

Looks good to me!

Result of nixpkgs-review pr 313849 run on x86_64-linux 1

7 packages built:

easyeffects
easyeffects.debug
gnomeExtensions.easyeffects-preset-selector
lsp-plugins
lsp-plugins.dev
lsp-plugins.doc
pulseeffects-legacy


feat(Algebra/Algebra/Basic): prove algebraMap_eq_zero_iff for NoZeroSMulDivisors

Can you fix the downstream failures, and add the eq_one_iff version too?

bors d+

bors r+

bors r+


Codacy is not working

It does not analyze code and gets stuck here:

we need to enable codacy checks for the base branch to prevent codacy failures


No console logs to be found in chrome running npm run dev

Summary
I expect there to be console.logs visible in the console in terminal that is being spawned after running npm run dev.
First i thought this might been a issue with the svelte-ts template but the vue-ts does not seem to work either.
Reproduction
npm create vite-plugin-web-extension
And choose svelte-ts or vue-ts (have not tried the others)
Environment

  System:
    OS: macOS 14.4.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 314.72 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.9.0 - ~/Library/Caches/fnm_multishells/5827_1715270057097/bin/node
    npm: 10.1.0 - ~/Library/Caches/fnm_multishells/5827_1715270057097/bin/npm
    pnpm: 8.15.4 - /opt/homebrew/bin/pnpm
    bun: 1.0.30 - /opt/homebrew/bin/bun
  Browsers:
    Chrome: 124.0.6367.119
    Safari: 17.4.1
  npmPackages:
    vite: ^5.0.0 => 5.2.11
    vite-plugin-web-extension: ^4.0.0 => 4.1.3

Runtime logs show up in the browser's devtools, not the terminal used to run WXT. That terminal only shows build logs/errors.

I'm sorry for the confusion. But they dont show in the console devtools where the plugin is running in. The chrome devtools that is being spawned after running npm run dev

I don't really understand what you mean.... console logs work fine for me. If you go to the popup page's console you should see Hello from the popup! and if you go to the service worker/background script's console you should see Hello from the background!

popup right click inspect

@aklinker1 IMHO this can be closed


Use port 9200 for 1.3.2 branch

Signed-off-by: Joshua Li<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #390 (bf647b5) into aos-<IP_ADDRESS> (5b7c1e9) will increase coverage by 4.35%.
The diff coverage is n/a.

@@                Coverage Diff                @@
##             aos-<IP_ADDRESS>     #390      +/-   ##
=================================================
+ Coverage          72.86%   77.21%   +4.35%
=================================================
  Files                 94       30      -64
  Lines               3928     1870    -2058
  Branches             609      375     -234
=================================================
- Hits                2862     1444    -1418
+ Misses               891      423     -468
+ Partials             175        3     -172

Flag
Coverage Δ

dashboards-reports
77.21% <ø> (ø)

reports-scheduler
?

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

Impacted Files
Coverage Δ

.../reportsscheduler/model/ReportDefinitionDetails.kt

...rtsscheduler/action/GetAllReportInstancesAction.kt

...rtsscheduler/resthandler/ReportStatsRestHandler.kt

...tsscheduler/model/GetAllReportInstancesResponse.kt

...search/reportsscheduler/action/PluginBaseAction.kt

...opensearch/reportsscheduler/model/SearchResults.kt

...rtsscheduler/model/InContextReportCreateRequest.kt

...eduler/model/UpdateReportInstanceStatusResponse.kt

...ortsscheduler/model/GetReportDefinitionResponse.kt

...search/reportsscheduler/settings/PluginSettings.kt

... and 54 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b7c1e9...bf647b5. Read the comment docs.


use sanic.response.json|text

Hello.
it does not seem that it is possible to use sanic.response.json or .text. Look like everything should be json. Is it possible to autorize text return value for some api ?

It's totally fine to return anything that is serializiable.


tile.py: changed get_instance_sites function

This PR allows get_isntance_sites not to break with the https://github.com/SymbiFlow/prjxray/pull/644 changes, this because the find_origin_coordinate in lib.py has changed.
the quick_test still does not work with the upstream db version because the 074-dump-all fuzzer should be run with the fixes.
Signed-off-by: Alessandro Comodi<EMAIL_ADDRESS>

Fixes #134


Percentile response time anomalies at 100%

With locust 0.7.2 there seems to be a calculation problem with the 100% column. It's often reported a few ms lower than the preceding columns, which should be impossible. In the example below it's especially glaring when there was only one request for a particular endpoint: 99%: 580; 100%: 575. But I see a similar anomaly in the last line, which has 53 requests. In my data the anomalies seem to be common with less that 100 requests, but rare or at least invisible after that.
 Let me know if this is not the case.


Hide video codec info

You hided audio codec info which is nice. It would looks more polished if you hide also video codec info. Averege user does not need these codecs information and who does need it, it is available from the menu.

Thanks, i added it and it works.
And the movies should not be watched in the browser but rather in media player :smiley: . Browsers are generally not good for playing media files. Anyway HEVC should be supported in the Chromium-based browsers, so probably only Firefox users are affected but that is minority of all users. And this minority can still check the codec from the menu :wink:


Changed MixedRealityRaycastHit to use collider transform instead of RaycastHit.transform

changed MixedRealityRaycastHit to use collider transform instead of Unity.RaycastHit.transform because this one potentially returns the transform of the gameobject that has a rigidbody attached instead of the actually collision transform.
This was the cause for not being able to interact with any handles of bounding box whenever a rigidbody component was attached.
Unitys documentation on RaycastHit.transform is not clear on what's happening. It only states that: "The Transform of the rigidbody or collider that was hit."
While digging around I found other ppl running into this as well:
https://answers.unity.com/questions/190482/raycasthittranslatecollider.html
My observation while debugging confirms what's mentioned in that thread. Whenever we've no rigidbody component attached the transform will be set to the hit colliders transform. Whenever there's a rigidbody component somewhere in hierarchy the transform will be populated with the rigidbody gameobject transform even though the hit result collider will point to a gameobject lower in the hierarchy.
Changes

Fixes: #4901
Removed obsolete comment I found in BoundingBoxHelper

Verification
tested modifying bounding box with and without rigid body component attached
run tests

/azp run

/azp run

This feels potentially related to https://github.com/microsoft/MixedRealityToolkit-Unity/pull/5617

@keveleigh , @ritijain is there a reason why the default value for focusIndividualCompoundCollider is false? I feel like this is something ppl can easily run into with bounding box and I would like to enable this per default.

I removed my previous changes and replaced them instead with an inspector change for bounding box that will notify users if their setup doesn't work

@keveleigh , @ritijain is there a reason why the default value for focusIndividualCompoundCollider is false? I feel like this is something ppl can easily run into with bounding box and I would like to enable this per default.

This was set to false by default so as not to make any functional change in default MRTK functionality. If there is enough of a case here to set it to true, i am not opposed to it.


Python: combined stream and non-stream HF tests

Motivation and Context

The Hugging Face tests sometimes failed, this combines the streaming and non-streaming tests, loading the model once, instead of twice.
Description

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

Python Test Coverage Report • FileStmtsMissCoverMissingTOTAL4871104079% report-only-changed-files is enabled. No files were changed during this commit :)
Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

993
11 :zzz:
0 :x:
0 :fire:
20.580s :stopwatch:

Python Test Coverage Report • FileStmtsMissCoverMissingTOTAL4871104079% report-only-changed-files is enabled. No files were changed during this commit :)
Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

993
11 :zzz:
0 :x:
0 :fire:
23.424s :stopwatch:

Python Test Coverage Report • FileStmtsMissCoverMissingTOTAL4871104079% report-only-changed-files is enabled. No files were changed during this commit :)
Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

993
11 :zzz:
0 :x:
0 :fire:
20.968s :stopwatch:

Python Test Coverage Report • FileStmtsMissCoverMissingTOTAL4871104079% report-only-changed-files is enabled. No files were changed during this commit :)
Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

993
11 :zzz:
0 :x:
0 :fire:
20.966s :stopwatch:

Python Test Coverage Report • FileStmtsMissCoverMissingTOTAL4871104079% report-only-changed-files is enabled. No files were changed during this commit :)
Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

993
11 :zzz:
0 :x:
0 :fire:
21.538s :stopwatch:

Python Test Coverage Report • FileStmtsMissCoverMissingTOTAL4871104079% report-only-changed-files is enabled. No files were changed during this commit :)
Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

993
11 :zzz:
0 :x:
0 :fire:
25.384s :stopwatch:


Make create_embed_qs.sh support Elixir

I have the same problem when I try on my side, I can think of two reasons:
1 - The setup fails
2 - Something's wrong on Dagger playground when using Elixir
@crjm Do you know if Dagger Playground can struggle with a new setup? Is there any changes we may need to do on our side?

@wingyplus it looks like you want to create an embeddable Elixir snippet which is not related to the quickstart? If so, then the changes should be applied to create_embed.sh instead of create_embed_qs.sh, as the latter is intended only for use with the Dagger quickstart at https://docs.dagger.io/quickstart/ .
This is also mentioned in the style guide:
Note that the command differs depending on whether the embed is for use in the Quickstart or elsewhere, as the Quickstart has special requirements.

cd docs
# for embeds apart from quickstart embeds
TOKEN=$TOKEN ./create_embed.sh YOUR-SNIPPET-FILE
# for quickstart embeds
TOKEN=$TOKEN ./create_embed_qs.sh YOUR-SNIPPET-FILE

I suggest transferring the changes you have made in the PR to create_embed.sh and then trying to create the embeddable snippet again.

I suggest transferring the changes you have made in the PR to create_embed.sh and then trying to create the embeddable snippet again.

I updated create_embed.sh and I hit the same issue :/
TOKEN=$TOKEN ./create_embed.sh ./test.exs
...

https://play.dagger.cloud/embed/INOiQrplCK1

The link is still broken: https://play.dagger.cloud/embed/INOiQrplCK1

I am pretty sure that Elixir is not available on the Playground. cc @marcosnils
Also, https://play.dagger.cloud/playground/9m4N7SgFN-J doesn't work since the container image does not have git installed:

@gerhard Thank you for the guidance! I forgot that elixir image didn't bundle git in the image.
This is a new version upload by the latest query https://play.dagger.cloud/embed/27uUMLP7l8t. It can open but cannot execute. I'm not sure why. :/
cc @TomChv @vikram-dagger

I think it's because as @gerhard  said, Elixir isn't supported by the Playground for now :/

I think it's because as @gerhard said, Elixir isn't supported by the Playground for now.

So I will retest once it support.


Registry: Additional Properties Throws MAPI error

Need to remove MAPI from this dialog if it's gonna work in registry editor, or suppress it

fixed in #448


Remove unused shared, unify bridge crates

Summary

RFCs: Link to RFC, Link to RFC, or $\emptyset$.
Categories: any of protocol-units, networks, scripts, util, cicd, or misc.

Changelog

Testing

Outstanding issues

Here's what's left to clean up:

In harness.rs:

set_initiator_contract, set_signer_address and set_weth_contract don't appear to exist

In movement.rs

getting a lot of ``selfparameter is only allowed in associated functions associated functions are those inimplortrait definition  . Also getting this in setup.rs
Should Address be BridgeAddress?

In utils.rs, initiator_set_timelock doesn't appear to exist

Throughout, we should check our time locks, to make sure they're correct. I added default timelocks to make things pass so anywhere there's args.time_lock should be reviewed.

Here's what's left to clean up to make tests run:

In harness.rs:

set_initiator_contract, set_signer_address and set_weth_contract don't appear to exist

In movement.rs

getting a lot of ``selfparameter is only allowed in associated functions associated functions are those inimplortrait definition  . Also getting this in setup.rs
Should Address be BridgeAddress?

In utils.rs, initiator_set_timelock doesn't appear to exist

Throughout, we should check our time locks, to make sure they're correct. I added default timelocks to make things pass so anywhere there's args.time_lock should be reviewed.

Update on progress:
got all tests passing except test_eth_client_should_deploy_initiator_contract. Seems to be related to how Config::build_for_test is working
On main there's Option for the three addresses:
pub struct Config {
	pub rpc_url: Url,
	pub ws_url: Url,
	#[serde_as(as = "serde_with::DisplayFromStr")]
	pub signer_private_key: PrivateKeySigner,
	pub initiator_contract: Option<Address>,
	pub counterparty_contract: Option<Address>,
	pub weth_contract: Option<Address>,
	pub gas_limit: u64,
}

impl Config {
	pub fn build_for_test() -> Self {
		Config {
			rpc_url: "http://localhost:8545/".parse().unwrap(),
			ws_url: "ws://localhost:8545/".parse().unwrap(),
			signer_private_key: PrivateKeySigner::random(),
			initiator_contract: None,
			counterparty_contract: None,
			weth_contract: None,
			gas_limit: 10_000_000_000,
		}
	}
}

But in this PR's branch, they're strings:
pub struct Config {
	pub rpc_url: Url,
	pub ws_url: Url,
	#[serde_as(as = "serde_with::DisplayFromStr")]
	pub signer_private_key: PrivateKeySigner,
	pub initiator_contract: String,
	pub counterparty_contract: String,
	pub weth_contract: String,
	pub gas_limit: u64,
}

impl Config {
	pub fn build_for_test() -> Self {
		Config {
			rpc_url: "http://localhost:8545/".parse().unwrap(),
			ws_url: "ws://localhost:8545/".parse().unwrap(),
			signer_private_key: PrivateKeySigner::random(),
			initiator_contract: "0x1234567890abcdef1234567890abcdef12345678".to_string(),
			counterparty_contract: "0x1234567890abcdef1234567890abcdef12345678".to_string(),
			weth_contract: "0x1234567890abcdef1234567890abcdef12345678".to_string(),
			gas_limit: 10_000_000_000,
		}
	}
}

Thinking maybe the String part is fundamentally incorrect, and they should be Option<Address> instead?
I added dummy values, the 0x123... address
but not sure whether those fields should go back to being Option<Address> (edited)
When I try it with Option and run rust_backtrace=1 cargo test --test client_eth_movement test_eth_client_should_deploy_initiator_contract  -- --nocapture --test-threads=1 it fails with
thread 'test_eth_client_should_deploy_initiator_contract' panicked at protocol-units/bridge/service/tests/harness.rs:119:14:
Failed to create EthClient: Missing initiator contract address


Audible book does not play, voice response saying "i am having trouble to play the book"

My app is based on SDK v1.7.1, with media player ported to gstreamer v1.10.
The SampleApp works well with functions of authentication, playing music and displaying display cards.
But it encounters trouble playing audible book, there's voice directive telling "i am having trouble to play the book".
I got console output as attached, can't explain why,

the status changing from  IDLE -> LISTENING -> THINKING -> SPEAKING -> IDLE -> SPEAKING -> IDLE -> SPEAKING -> IDLE ..
when would alexa response " having trouble to play the book " ? any media format handling capability missed ? (MP4)

I'm using device with Linux 4.4, gstreamer 1.10, plugins installed.
The device is ARM based, with audio output via HDMI connecting to TV.
audible book.txt

Hi @georgeyao310012,
When attempting to access the Audible content, there was an error related to SSL certificates:
2018-08-14 08:29:24.288 [ 1a] E LibCurlHttpContentFetcher:curlEasyPerformFailed:error=Peer certificate cannot be authenticated with given CA certificates
It is interesting that there was no authentication failure. Do you see this with other music providers? Can you check your SSL certificates, and perhaps try using a different one?

Thank you for the reply,
Yes, it's strange that no failure with authentication or any other things. Tried other audio services, like music and radio works well, but failed with Audible book.
We tried different certificates, as described in tickets of:
https://github.com/alexa/avs-device-sdk/issues/452,
with ticket downloaded from url of https://curl.haxx.se/docs/caextract.html.
Things does not change.
Any other possible suspects, or test I can do to further investigate on the issue ?

Thanks for the help.
It's because we use the right CA for authorization, but that CA is not correctly used while setting up HTTPS connection while trying to access audible services.
It works now.
So this ticket could be closed.


Rich Presence does not show up for anyone other than the client

... and invites are also broken (the plus next to the send message)
The code can be found here: https://github.com/RowanHall/desktop-youtube-music/
I have absolutely no idea why this is happening

Can you confirm if this is still happening? We had some Rich Presence issues over the weekend and a fix was deployed Sunday afternoon PST!

Can confirm it is fixed, thanks msciotti!


unresolved macro metrics::histogram!

hello, i'm new to rust and ra, when i use macro 'metrics::histogram!',i could go to the defination of the macro,but it alway report problem with unresolved macro metrics::histogram!,i want to figure out why, should i download the metric crate by rustup?

Which version of the metrics crate are you using?

We don't support #[proc_macro_hack], but somewhere between 0.13.0-alpha.11 and 0.13.0-alpha.13 metrics stopped using that.

i use the 0.13.0-alpha.8, so i should update the version of the metric??

Any version later than 0.13.0-alpha.13 should work. I tested the latest one (0.16). Attribute proc macros will be implemented soon(ish), so it might start working for that one too, but unless you have good reasons you probably shouldn't use an -alpha build anyway.

i got it ， thank you

Well, let's close this then.


Worst 2015 passwords broken link

Hi team,
at the bottom of the page the link to the 2015 worst passwords is broken

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

ID: 523758d0-17ea-7061-d9ce-8f5a31a76b4c
Version Independent ID: 545fd950-8ef0-837b-8839-8aa21da0c6eb
Content: Password policy recommendations - Microsoft 365 admin
Content Source: microsoft-365/admin/misc/password-policy-recommendations.md
Service: microsoft-365-business
GitHub Login: @KwekuA
Microsoft Alias: kwekua

@phenixita Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/10342
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @phenixita Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[WIP][MARXAN-1122]adds-ACL-scenarios-controller

-Adds ACL to every endpoint in the scenarios-controller.
-Fixes some missing ImplementsACL decorator not present in projects-listing-controller.
-Various additions to ACL-utils.

thanks @rubvalave. I am not too convinced with the new findAllPaginatedAcl() method, as this would leave the base class' own findAllPaginated() open to accidental use without access control checks.
One possible way to handle this would be what I sketched quickly in https://github.com/Vizzuality/marxan-cloud/tree/feat/MARXAN-1122-scenario-acl-application%2Bandrea-suggestions.

so quickly in fact that it's obviously incomplete and doesn't even transpile, but just to give a rough idea 😇


Will this crate's MSRV bump past Debian Stable?

This is a nice crate and it may be useful in the async-executor crate. However async-executor follows a strict Debian Stable-based MSRV policy. Do you foresee a future where the MSRV for this crate is bumped past Debian Stable? (Currently 1.63)

This crate doesn't get many updates, it is mostly a "finished product". With that said, I don't want to guarantee that the MSRV will never change in the future. If you need something to work on an ancient version of Rust then you should explicitly pin the version.

then you should explicitly pin the version.

(And this ought to be via Cargo.lock, not via an = in `Cargo.toml.)

Thank you for clarifying. Will an MSRV bump be considered a minor version break? If so I can just pin to the minor version

Yes, I can at least guarantee that.

Yes, I can at least guarantee that.

Thanks!


Removing distinct on from get so that sort works

A clean replacement for #75

Thanks. I'm sure the line was there for a reason - hopefully to fix a bug that no longer occurs.


custom training for yolact to onnx conversion

@Ma-Dan Thanks for the script, I am trying to convert the yolact model to onnx model, i referred your script and i setup all the dependency as you mentioned, while i started the training on my custom data-set, it through error.

/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/cuda/init.py:134: UserWarning:
Found GPU0 GRID K520 which is of cuda capability 3.0.
PyTorch no longer supports this GPU because it is too old.
The minimum cuda capability that we support is 3.5.

warnings.warn(old_gpu_warn % (d, name, major, capability[1]))
/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/cuda/init.py:134: UserWarning:
Found GPU1 GRID K520 which is of cuda capability 3.0.
PyTorch no longer supports this GPU because it is too old.
The minimum cuda capability that we support is 3.5.

warnings.warn(old_gpu_warn % (d, name, major, capability[1]))
/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/cuda/init.py:134: UserWarning:
Found GPU2 GRID K520 which is of cuda capability 3.0.
PyTorch no longer supports this GPU because it is too old.
The minimum cuda capability that we support is 3.5.

warnings.warn(old_gpu_warn % (d, name, major, capability[1]))
/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/cuda/init.py:134: UserWarning:
Found GPU3 GRID K520 which is of cuda capability 3.0.
PyTorch no longer supports this GPU because it is too old.
The minimum cuda capability that we support is 3.5.

warnings.warn(old_gpu_warn % (d, name, major, capability[1]))
Traceback (most recent call last):
File "train.py", line 382, in
train()
File "train.py", line 143, in train
yolact_net = Yolact()
File "/home/ubuntu/efs_model/models/YOLACT/Modified_Yolact/yolact.py", line 395, in init
self.backbone = construct_backbone(cfg.backbone)
File "/home/ubuntu/efs_model/models/YOLACT/Modified_Yolact/backbone.py", line 437, in construct_backbone
backbone = cfg.type(*cfg.args)
File "/home/ubuntu/efs_model/models/YOLACT/Modified_Yolact/backbone.py", line 64, in init
self.conv1 = nn.Conv2d(3, 64, kernel_size=7, stride=2, padding=3, bias=False)
File "/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 332, in init
False, pair(0), groups, bias, padding_mode)
File "/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 46, in init
self.reset_parameters()
File "/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 49, in reset_parameters
init.kaiming_uniform(self.weight, a=math.sqrt(5))
File "/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/nn/init.py", line 315, in kaiming_uniform_
return tensor.uniform_(-bound, bound)
RuntimeError: CUDA error: no kernel image is available for execution on the device

can any one faced this issue, tell any suggestion for this error. @dbolya @abhigoku10

@saisubramani  sorry for delay in response , no i did not face this issue!!i shall re-check

@saisubramani sorry for delay in response , no i did not face this issue!!i shall re-check
@abhigoku10 hi @Ma-Dan had replied and he said that we can use the source code to start the training, and after getting the model file which is in .pth format i want to convert that into onnx model . i started custom training and i got the .pth now i am trying to convert it into onnx, i am using the code which was suggested by De-Man in his code , i am using eval.py script, while running the script it shows an error, can you able to sort out this error

home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/cuda/init.py:134: UserWarning:
Found GPU0 GRID K520 which is of cuda capability 3.0.
PyTorch no longer supports this GPU because it is too old.
The minimum cuda capability that we support is 3.5.

warnings.warn(old_gpu_warn % (d, name, major, capability[1]))
/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/cuda/init.py:134: UserWarning:
Found GPU1 GRID K520 which is of cuda capability 3.0.
PyTorch no longer supports this GPU because it is too old.
The minimum cuda capability that we support is 3.5.

warnings.warn(old_gpu_warn % (d, name, major, capability[1]))
/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/cuda/init.py:134: UserWarning:
Found GPU2 GRID K520 which is of cuda capability 3.0.
PyTorch no longer supports this GPU because it is too old.
The minimum cuda capability that we support is 3.5.

warnings.warn(old_gpu_warn % (d, name, major, capability[1]))
/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/cuda/init.py:134: UserWarning:
Found GPU3 GRID K520 which is of cuda capability 3.0.
PyTorch no longer supports this GPU because it is too old.
The minimum cuda capability that we support is 3.5.

warnings.warn(old_gpu_warn % (d, name, major, capability[1]))
Traceback (most recent call last):
File "train.py", line 382, inhow can
train()
File "train.py", line 143, in train
yolact_net = Yolact()
File "/home/ubuntu/efs_model/models/YOLACT/Modified_Yolact/yolact.py", line 395, in init
self.backbone = construct_backbone(cfg.backbone)
File "/home/ubuntu/efs_model/models/YOLACT/Modified_Yolact/backbone.py", line 437, in construct_backbone
backbone = cfg.type(*cfg.args)
File "/home/ubuntu/efs_model/models/YOLACT/Modified_Yolact/backbone.py", line 64, in init
self.conv1 = nn.Conv2d(3, 64, kernel_size=7, stride=2, padding=3, bias=False)
File "/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 332, in init
False, pair(0), groups, bias, padding_mode)
File "/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 46, in init
self.reset_parameters()
File "/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 49, in reset_parameters
init.kaiming_uniform(self.weight, a=math.sqrt(5))
File "/home/ubuntu/anaconda3/lib/python3.6/site-packages/torch/nn/init.py", line 315, in kaiming_uniform_
return tensor.uniform_(-bound, bound)
RuntimeError: CUDA error: no kernel image is available for execution on the device

Modification made by me :
hi i had founded that you are using eval.py script for converting the yolact model to onnx model i having a doubt
pred_outs = net(batch)
This give a list which having an size of 1, how you are using the index in ,
preds = detect({'loc': pred_outs[0], 'conf': pred_outs[1], 'mask':pred_outs[2], 'priors': pred_outs[3], 'proto': pred_outs[4]})
it showing
IndexError: list index out of range

so what i did is , i just added few lines
pred_outs = dict(pred_outs[0]) pred_outs=pred_outs['detection']
now its in dictionary format by using the key value i can take the values of detection, but when i cross checked the detection which is in (dictionary format) it having a key values of
('mask','class','score','proto','net')

what value can i assign for the
pred_out[0],pred_outs[1],pred_outs[2],pred_outs[3],pred_outs[4]

In my understanding 'conf' ':mean score,'mask':mask,'proto': means proto what about 'loc' and 'priors'
preds = detect({'loc': pred_outs[0], 'conf': pred_outs[1],  'mask':pred_outs[2], 'priors': pred_outs[3], 'proto': pred_outs[4]})
i tried like this
preds = detect({'loc': pred_outs['box'], 'conf': pred_outs['score'], 'mask':pred_outs['mask'], 'priors': pred_outs['class'], 'proto': pred_outs['proto']})
It showing Error:
TypeError: call() missing 1 required positional argument: 'net'
can you help me to sort this issue ? if i am wrong please tell me.
Thanks for the reply @abhigoku10

@saisubramani  have you solved the issue? I have similar issue when converting onnx to coreml for yolact(madan's coreml branch)

@saisubramani have you solved the issue? I have similar issue when converting onnx to coreml for yolact(madan's coreml branch)

hi, @dzyjjpy can i know what is the error?  i did'nt solve that one i want to work on that.
at what place you are facing the error.give the details so that i can check it!

https://github.com/dbolya/yolact/issues/374
@saisubramani


[beta] Unclear instructions for "Coding Interview Data Structure Questions: Create a Doubly Linked List"

Challenge create-a-doubly-linked-list has some minor issues with the instructions:

Some paragraphs are split in the middle of sentences.
"Instructions" needs to be replaced with a
 (see other sections for examples)
 tags to highlight key words like method names.
<dfn></dfn> tag around the first occurrence of "Doubly Linked List".
Unclear description of what the input to the methods will be:

The add method should accept integers or strings and add them to the list and remove should accept an integer or string and remove all copies of this integer or string that exist in the list.

I think the above could be changed to something like:

The add method should add an element to the list, while the remove method removes all occurrences of a given element.

@Greenheart I can take this up too...

@Manish-Giri Thanks!
I just found some issues with the tests that prevents the challenge from being completed at all. I'll fix this right away, but feel free to fix the points of the original post! :blush:

@Greenheart Ah okay, understood!

@Manish-Giri Seems like I missed these lines.
My solution uses ES6 classes, which explains why the ES5-way of adding these hidden utility methods doesn't work.
@Manish-Giri Should we rewrite the tail to make sure people with modern browsers who want to use modern JavaScript can do so and still pass the challenge?
I think one approach would be to simplify the existing print() and printReverse() to a singel method called values() that is visible in the challengeSeed. That way, people who want to use classes can do so and see what they need to rewrite to make the challenge work. What do you think? :smile:

@Greenheart

Should we rewrite the tail to make sure people with modern browsers who want to use modern JavaScript can do so and still pass the challenge?

Couldn't agree more. Always good to have the option of being able to use the latest versions.
As for simplifying the print() and printReverse() methods to values(), were you thinking of something on the lines of iterating the linked list both ways and storing the node values in two different arrays in values() itself, and then printing it? Or, iterate in one direction and just use .reverse() on the list? Either way, I second the idea of simplification!
And lastly, for making values() visible in challengeSeed, I just have the feeling that it might simplify the challenge a bit too much!
But you're the best judge! 💯

@Manish-Giri

iterate in one direction and just use .reverse() on the array

Exactly :blush:

And lastly, for making values() visible in challengeSeed, I just have the feeling that it might simplify the challenge a bit too much!

The values() method would replace print() and printReverse(), two utility functions that only are used in the tests. The challenge still asks campers to implement add() and remove() so I don't think the difficulty would be affected. Rather, this would make it easier to use an ES6 class as campers can see what code they need to rewrite to allow their new methods add() and remove() to be tested.

But you're the best judge!
No, I'm not! The more that give their opinions, the better :blush:

@Manish-Giri I created a solution that meets the requirements of this challenge.
class Node {
  constructor (data, prev) {
    this.data = data
    this.prev = prev
    this.next = null
  }
}

class DoublyLinkedList {
  constructor () {
    this.head = null
    this.tail = null
    this._length = 0
  }

  values () {
    const result = []

    let node = this.head

    while (node !== null) {
      result.push(node.data)
      node = node.next
    }

    return result
  }

  add (element) {
    if (this.head === null) {
      const node = new Node(element, null)
      this.head = node
      this.tail = node
    } else {
      const node = new Node(element, this.tail)
      this.tail.next = node
      this.tail = node
      ++this._length
    }
  }

  remove (element) {
    if (this.head === null) return null

    let current = this.head
    const removed = []

    while (current !== null) {
      if (current.data === element) {
        removed.push(current.data)
        --this.length

        if (current.prev === null) {
          this.head = current.next
        } else {
          current.prev.next = current.next
        }

        if (current.next === null) {
          this.tail = current.prev
        } else {
          current.next.prev = current.prev
        }
      }

      current = current.next
    }

    return removed
  }
}

If this values() method is used with .reverse(), it can effectively replace the print() and printReverse() methods.
If we see value in traversing the list from the other way around, I also experimented with combining both use cases into a single method that has a parameter.
values (reverse = false) {
  const start = reverse ? 'tail' : 'head'
  const next = reverse ? 'prev' : 'next'
  const result = []

  let node = this[start]

  while (node !== null) {
    result.push(node.data)
    node = node[next]
  }

  return result
}


Item validation fails if unitPrice is 0

The OpenAPI specification states for item.unitPrice:
    Item:
      type: object
      required:
        - unitPrice
        - units
        - vatPercentage
        - productCode
      properties:
        unitPrice:
          type: integer
          format: int64
          example: 1590
          minimum: 0
          maximum: 99999999
          description: Unit price of an item in currency minor unit, eg. EUR cents. VAT should be included in amount unless `usePricesWithoutVat` is set to true.

The API accepts items with unitPrice = 0, but Paytrail\SDK\Model\Item validation fails:
https://github.com/paytrail/paytrail-php-sdk/blob/c1df05e968abf8daae5d2159dcd9a44df7e817ba/src/Model/Item.php#L379-L381
This was changed in SDK version 2.5.1 (https://github.com/paytrail/paytrail-php-sdk/pull/74)
Which behaviour is correct, SDK or API?

API is correct, validation should be done against only null value. Will fix. For refunds, amount should be positive not zero, since zero amount refund is not refund at all.


Deploy to demo

Deploy to demo env. version of ehealth that is going to be deployed to prod.
according to the release notes

[x] define required versions of all the components @AlekzandrV
[x] deploy @vsirius

deployed


issue #1, Seed identification API for Nachet frontend

Goal of the Pull Request:
This pull request aims to introduce a function that extracts the content from all the seeds within the HTML content and converts them into a usable JSON format.
Wanted result:
[
    {
      "family": "Malvaceae",

      "common_name": "Velvetleaf",

      "duration_of_life_cycle": "Annual",

      "seed_or_fruit_type": "Seed",

      "identification_features":
       {
        "seed_length": "3.0 - 4.0 mm",

        "seed_width": "2.3 - 3.6 mm",

        "shape": "Heart-shaped seed with unequal lobes and a conspicuous notch at hilum end",

        "surface_texture": "Seed dull with minute reticulations and scattered tubercles,concentrated at the hilum",

        "colour": "Seed greyish brown with a lighter-coloured pubescence",

        "other_features": "Flap-like tissue covering the hilum may be present"
       },

       "images": [
        "https://inspection.canada.ca/DAM/DAM-plants-vegetaux/STAGING/images-images/invasive_plants_seed_factsheet_abutilon_theophrasti_01cnsh_1476382781946_eng.jpg",
        "https://inspection.canada.ca/DAM/DAM-plants-vegetaux/STAGING/images-images/invasive_plants_seed_factsheet_abutilon_theophrasti_01cnsh_1476382781946_eng.jpg"
      ],

       "source": "https://inspection.canada.ca/plant-health/seeds/seed-testing-and-grading/seeds-identification/abutilon-theophrasti/eng/1472604373914/1472604374428"
    }
  ]

Related Issue:

https://github.com/ai-cfia/louis-db/issues/1

The website from which information is being crawled

Missing data:

Similar species
General Information
Distribution

Currently, we have a function that searches for a seed based on its common name. We intend to modify this function to search for seeds using their scientific names (e.g., Abutilon theophrasti for Velvetleaf) and retrieve information for all seeds, not just one.

Start creating a Python test with relevant test cases based on the comments you've received and fixes you've done during development.

The objective here is to:

Retrieve all HTML pages about the seeds.
Establish a connection to the OpenAI Azure Chat.
Request ChatGPT to parse the files and provide JSON output.
Transform the JSON data into a format suitable for the database.
Establish a normalized database structure to store this data.

Related commit:

Creation of the file (9db48f9)

The code is making good progress. The next steps involve adding the source and image information to the JSON. However, a challenge I'm facing is that I don't have access to the source in the document I'm parsing, and it appears that the image links are local and therefore not useful.
Once I address this issue, my next task will be to parse all the seeds and store them in another location.

The text will undergo changes until the pull request is concluded.
To summarize this Pull Request https://github.com/ai-cfia/louis-db/pull/21 :
The objective was to obtain all the seeds from the official government website in a usable data format, specifically JSON. I explored various approaches, and while there is always room for code improvement, it appears that the main issue has been resolved. I successfully retrieved the information from the web and converted it into JSON format.
In the subsequent paragraph, I will provide a summary of the actions I have taken, explaining both their successes and limitations. It's important to note that the challenge I faced was primarily due to the website's quality. If the website had a properly formatted document, such as XML, certain aspects of the task would likely have been easier.

Here are the approaches I attempted:

My initial attempt was to accomplish everything solely using PostgreSQL. However, I encountered numerous challenges in terms of time, data quality, complexity, adaptability, and the difficulty of reusability and maintenance. This approach might have worked if the website had been perfectly structured, but there were too many exceptions and irregularities that made parsing the website data effectively impractical (75d8cd9).
I addressed the website's formatting issue by employing ChatGPT to generate JSON. Initially, I utilized SQL commands to retrieve a list of all seeds, then matched each seed with its respective page using the scientific name. After stripping away all HTML tags, I provided GPT with a JSON template containing the relevant text and requested it to populate it with the accurate data. My main challenge revolved around execution time and difficulty in obtaining image links. (9db48f9, aec1731, 059f0ae, 1ff7f2d).
I switched from ChatGPT version 4 to version 3.5 turbo. This adjustment resulted in a reduction of the time required per seed by a factor of 2 and also decreased the number of tokens used per seed. Remarkably, the accuracy of the results remained unchanged.
To incorporate images into the JSON, I initially attempted to achieve this solely using SQL and Python. However, the website's complexities made it exceedingly challenging to execute this task. Consequently, I introduced a section in which I sent the data to ChatGPT, along with images and their corresponding descriptions in the prompt. With this information, GPT was able to discern which images corresponded to the specific seed and which were related to similar species, thus effectively handling the image inclusion process. (e621c5e).
I modified the process of sending seeds by adopting an on-demand approach. Rather than retrieving information for all seeds from the beginning, we now fetch data only when necessary. This adjustment has significantly reduced execution time when parsing seeds that have already been seen. However, I encountered some issues during seed parsing, including instances where some seeds were missing and others were added when they shouldn't have been. (e621c5e).
I've updated the process as per your latest changes. Now, we're no longer searching for seed pages within the HTML content; instead, we're using the URLs. However, the challenge persists, as some URLs on the website are still not correctly formatted, preventing us from accessing the correct page for every seed. To address this issue, you've introduced a text file to manage problematic seeds separately, allowing for the implementation of an alternative method at a later stage. These adjustments should help improve the accuracy and effectiveness of the data retrieval process. (e621c5e).

Statistics following the latest commit: (e621c5e)

Total seeds: 124
Newly added seed: 2
Skipped because already existing seed: 106
Problematic seed found during search: 16

Needed changes:

I will alter the method for obtaining seed pages. We will retrieve them through URLs found on the main page where all the seeds are listed.

Related Issue:

https://github.com/ai-cfia/louis-db/issues/1

@JolanThomassin it fails linting with ruff. Please ensure it passes before reviews.

@JolanThomassin also please address reviewers previous comments by answering them ("done" or asking for clarification)

@rngadam
Lint should be working now. Some code had too many comments, but that's okay now. I have replied to or reacted to every comment, so I'm unsure what's wrong. I have added new comments to make sure that everything requested has been corrected or explained, so it should be fine now 👍

With the last changes the code don't work anymore, I need to make some tests

I still see my comments as "Pending", I hope you can see them cause I can't find how to send them

@JolanThomassin I suggest you resolve your own conversations you initiated in this PR, by commenting on what changed, or why you decided not to continue or creating follow up issues.


Farcry 4 crashes on title screen (Windows 10)

On windows 10 farcry 4 crashes upon trying to load the main menu. What makes this odd is when using the apitrace dlls provided from the bug report page the game loads just fine so I'm not sure whats wrong with it exactly
(launch options used: -fullscreen -skipintro)
Software information
Far Cry 4
System information

GPU: NVIDIA RTX 2080 TI
Driver: 552.12
DXVK version: 2.3.1
OS: Windows 10 22H2 (OS Build 19045.5170)

Apitrace file(s)
https://drive.google.com/file/d/1-fwtZVTqv74AkXL2U580zyLHL7KeDEUp/view?usp=drive_link
Log files
FarCry4_d3d11.log
FarCry4_dxgi.log

Something spam-creates a huge number of Vulkan devices and at some point this just fails. Don't know why this would happen (any third-party / nvidia stuff active that might interact with Vulkan and DXGI?), but this seems firmly outside of our control.

aw thats unfortunate. and to answer your question: nothing as far as i know. thanks for the quick response

not sure if this is relevant but i know farcry 4 uses nvidia gameworks so maybe that might be causing the issue?

oh okay, thank you


Chefstyle updates

These are new proposed cops

@chef/client-core thoughts and feels?

I dislike EmptyCaseCondition but don't really care.
I find MultilineArrayBraceLayout a step backwards in terms of readability.

so @thommay the default for MultilineArrayBraceLayout is 'symmetrical' and supports either of these:
# symmetrical: good
# new_line: bad
# same_line: good
[ :a,
  :b ]

# symmetrical: good
# new_line: good
# same_line: bad
[
  :a,
  :b
]

i think the problem you're seeing is that you don't like that the opening brace did not start on its own line in those cases.

ok, I mean the first example is awful (which is what you had in this PR)

right but that's a function of the first line being written incorrectly, MultilineArrayBraceLayout is behaving well, we just need to get around to the cop that will fix the first line.  if you look at the code it is fixing it is worse:
[ :a,
  :b,
]

that is something nobody wants.

and I think I wrote both of the code fragments that EmptyCaseCondition caught.  i'm real tempted to drop that one.

I'm not sure we want to auto-fix the InheritException ones. In that case I'm okay with enabling it and using the magic comments to disable for specific cases where we know it's what we want.

EmptyCaseCondition gone
MultilineArrayBraceLayout gone (too much trouble)

@chef/client-core okay fixes applied i think

Looks ok to me now


whitelisting admin IP

after adding a second IP in site/configuration/serverconfig file I get a white page and the main daemon does not start even after reboot
 Main:   stopped       Daemon Directory: /home/yiimp-data/wallets

define('YAAMP_ADMIN_IP', 'x.x.x.x, x.x.x.x');

sudo apt install php7.3-memcache php7.3-memcached memcached  try this


MM-15057: Adding guest badge to identify guest users

Summary
Adding guest badge to identify guest users
Ticket Link
MM-15057
Checklist

[x] Has UI changes
[x] Includes text changes and localization file updates

@michaelgamble @jespino any updates on this one?

@enahum @hmhealey Now this is ready to get reviewed.

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@enahum I'm not sure what is stopping this to get it.

@jespino I believe the spec has changed but not the PR

@enahum Ok, I'll review the whole thing, and ping you :)

@enahum now matches the new design

@jespino is the feature branch updated against master? the navigation library was updated and the API changes

@enahum yes it is

@enahum Done! PTAL.


VS Code freezes when printing unicode characters in Powershell

Testing #208456
For testing, I tried running this script in a powershell integrated terminal:
for ($i=0; $i -le 0xFFFF; $i++) {
    Write-Host -NoNewline ([char]$i)
}

This gets a few hundred characters in before the entire window stops  responding

The same script works in an external powershell terminal

Interestingly can also happen on MacOS using:
for i in {0..65535}; do
  printf "\\u$(printf "%04x" $i)"
done

This works fine for me on Windows with Windows PowerShell and pwsh 7 🤔

I can also reproduce the renderer hang on macOS


syslogng: 3.17.2 -> 3.18.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/syslog-ng/versions.
meta.description for syslogng is: '"Next-generation syslogd with advanced networking and filtering capabilities"'.

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/bin/pdbtool had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/bin/dqtool had a zero exit code or showed the expected version
/nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/bin/loggen passed the binary check.
Warning: no invocation of /nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/bin/update-patterndb had a zero exit code or showed the expected version
/nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/bin/syslog-ng passed the binary check.
/nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/bin/syslog-ng-ctl passed the binary check.
/nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/bin/syslog-ng-debun passed the binary check.
4 of 7 passed binary check by having a zero exit code.
0 of 7 passed binary check by having the new version present in output.
found 3.18.1 with grep in /nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1
directory tree listing: https://gist.github.com/659b47a84891370ccad152c4473dec6a
du listing: https://gist.github.com/c944bcd5fe2499741ab5b0485d90a1fb

Outpath report (click to expand)

Outpath difference report
6 total rebuild paths
2 package rebuilds
2 x86_64-linux rebuilds
2 i686-linux rebuidls
0 x86_64-darwin rebuilds
2 aarch64-linux rebuilds
First ten rebuilds by attrpath
syslogng
syslogng_incubator

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A syslogng https://github.com/r-ryantm/nixpkgs/archive/44a568435957ae4779dfd543843bf24c0efc1c85.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1
ls -la /nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/bin

cc @rickynils @fpletz for testing.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: syslogng
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: syslogng
Partial log (click to expand)
/nix/store/jm3wq4j75fwjl9n8i9gjl347j2azdmjf-syslog-ng-3.18.1/sbin/syslog-ng-debun: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
/nix/store/jm3wq4j75fwjl9n8i9gjl347j2azdmjf-syslog-ng-3.18.1/bin/update-patterndb: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/jm3wq4j75fwjl9n8i9gjl347j2azdmjf-syslog-ng-3.18.1...
moving /nix/store/jm3wq4j75fwjl9n8i9gjl347j2azdmjf-syslog-ng-3.18.1/sbin/* to /nix/store/jm3wq4j75fwjl9n8i9gjl347j2azdmjf-syslog-ng-3.18.1/bin
shrinking RPATHs of ELF executables and libraries in /nix/store/d1gi2cqi1nnbz9zpd1kxcjlaj7kqrj9j-syslog-ng-3.18.1-man
gzipping man pages under /nix/store/d1gi2cqi1nnbz9zpd1kxcjlaj7kqrj9j-syslog-ng-3.18.1-man/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/d1gi2cqi1nnbz9zpd1kxcjlaj7kqrj9j-syslog-ng-3.18.1-man
checking for references to /build in /nix/store/d1gi2cqi1nnbz9zpd1kxcjlaj7kqrj9j-syslog-ng-3.18.1-man...
/nix/store/jm3wq4j75fwjl9n8i9gjl347j2azdmjf-syslog-ng-3.18.1

Success on x86_64-linux (full log)
Attempted: syslogng
Partial log (click to expand)
/nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/bin/update-patterndb: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
/nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/sbin/syslog-ng-debun: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1...
moving /nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/sbin/* to /nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1/bin
shrinking RPATHs of ELF executables and libraries in /nix/store/1fyphmbd07f86dj6p69wqwadiysdwcva-syslog-ng-3.18.1-man
gzipping man pages under /nix/store/1fyphmbd07f86dj6p69wqwadiysdwcva-syslog-ng-3.18.1-man/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/1fyphmbd07f86dj6p69wqwadiysdwcva-syslog-ng-3.18.1-man
checking for references to /build in /nix/store/1fyphmbd07f86dj6p69wqwadiysdwcva-syslog-ng-3.18.1-man...
/nix/store/arfadi0hg5sqvgk47z3wcbrc07jkqkpq-syslog-ng-3.18.1


Clicking on the action to turn on lights doesn't work

Description

Observed on master, hotkeys still work.
Reproduction

Try to turn on a light using the action button
Fail

i'm not able to replicate this. Do you have any further steps?

I also have this issue.
My steps are:
spawn as sec, pick up a seclite out of the lockers and place it in my pocket, try to click on the action to turn the seclite on that appears in my action bar. It does not turn on.
As a work around, I can press the action bar hotkey (1-0) and it works fine.

Seems very repoducable on my machine
Most any light action (including pda light) will double act instantly, both sounds played.
PDA light UI works totally fine, just the action.

This also happens with masks and suit pieces, or any action that doesn't have a cooldown.

for me its been very inconsistent, jetpacks work one round and then the next I cannot toggle the button


Request to have the Angular2 subreddit added to the Support page (angular.io/support.html)

There is a forum on reddit that is specifically targeted at Angular 2 that now has 1,850+ subscribers. In reddit parlance it is the /r/Angular2 subreddit.
This subreddit is two months shy of being 1 year old and has steadily attracted a larger following. The forum's growth stats can be seen here.
The forum is dedicated to helping Angular 2 enthusiasts learn the ropes, participate in discussions, share resource links and announce updates.
With 1,850+ current subscribers together with the accelerating subscriber growth rate, it would be great if the forum could be listed with the other Support websites listed on this page.
/r/Angular2 is a great resource for disseminating information to enthusiasts and is worthy of consideration for inclusion on the Support page.

CLA signed and submitted.

thanks! it's live now!


add NodeConf Argentina CFP date

Thanks :)

Hoping to apply to this, too! @yoshuawuyts @watson any chance I could hit you up for help with my submission?


feat: spin up from oci registry

~WIP-ish~
~NOTE: this PR hides bindle commands / related flags on up~

Not sure why the integration test is failing - the help output looks okay to me... 🤔


feat: [cinder-csi-plugin] add leader election

What this PR does / why we need it:
Adding support for running HA CSI controller.
Which issue this PR fixes(if applicable):
fixes #702
Special notes for reviewers:

Increase replicas to 3, all operations should work as expected (attach, detach, provision, deprovision, resize, snapshot).
I tested it on k8s 1.20, all operations are completing without errors. Also I can see successful leader election change events:
29m         Normal    LeaderElection      lease/csi-cinderplugin                               csi-cinder-controllerplugin-0 became leader
3m30s       Normal    LeaderElection      lease/csi-cinderplugin                               csi-cinder-controllerplugin-0 became leader
29m         Normal    LeaderElection      lease/external-attacher-leader-csi-cinderplugin      csi-cinder-controllerplugin-0 became leader
3m30s       Normal    LeaderElection      lease/external-attacher-leader-csi-cinderplugin      csi-cinder-controllerplugin-0 became leader
29m         Normal    LeaderElection      lease/external-resizer-csi-cinderplugin              csi-cinder-controllerplugin-0 became leader
3m30s       Normal    LeaderElection      lease/external-resizer-csi-cinderplugin              csi-cinder-controllerplugin-0 became leader
29m         Normal    LeaderElection      lease/external-snapshotter-leader-csi-cinderplugin   csi-cinder-controllerplugin-0 became leader
3m30s       Normal    LeaderElection      lease/external-snapshotter-leader-csi-cinderplugin   csi-cinder-controllerplugin-0 became leader

Cordon all nodes, deletetd csi-cinder-controllerplugin-0 (so it won't be scheduled again),  leader changed to csi-cinder-controllerplugin-2 successfully and all CSI operations still working.
Release note:

cinder-csi-plugin: add leader election flag (for HA CSI controller) and switch from statefulSet to Deployment

/assign @ramineni

@ramineni do we need to also have e2e tests for an HA setup?

I believe the use of StatefulSet with spec.replicas: 1 was to ensure a single replica at all times. With leader election, it can become a Deployment instead.

I believe the use of StatefulSet with spec.replicas: 1 was to ensure a single replica at all times. With leader election, it can become a Deployment instead.

Agree. now with leader election, even with replicas: 1 users should be able to have a rolling update strategy with no down time rollouts. Also I switched to HA 3 replicas by default, which is also a sane default value used in other csi helm charts by default (such as ceph-csi)

/ok-to-test

Its not the update of only charts now, so please remove the same from title and release notes

@ramineni all your suggestions applied.

/lgtm
@m-yosefpor I edited the release note, please double check.

/lgtm
@m-yosefpor I edited the release note, please double check.

SGTM. Thanks.

@m-yosefpor Thanks. Looks good.
/lgtm

/approve

@lingxiankong any ETA on when the next release for openstack-cinder-csi helm chart will be?

@lingxiankong any ETA on when the next release for openstack-cinder-csi helm chart will be?

Hi @m-yosefpor, the current chart release policy is we only do the official release together with the project, so for this specific one, we have to wait until v1.23 release.
I know that may sound weird, I will talk with the team and see what we could do.


feat: 🎸 update the HF webhook content

We don't use the new fields for now.

Codecov Report
Base: 87.92% // Head: 89.91% // Increases project coverage by +1.98% :tada:

Coverage data is based on head (7387538) compared to base (e31129e).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #671      +/-   ##
==========================================
+ Coverage   87.92%   89.91%   +1.98%
==========================================
  Files          48       17      -31
  Lines        1623      565    -1058
==========================================
- Hits         1427      508     -919
+ Misses        196       57     -139

Flag
Coverage Δ

jobs_mongodb_migration
?

services_admin
?

services_api
89.91% <ø> (ø)

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

Impacted Files
Coverage Δ

services/api/src/api/routes/webhook.py
66.12% <ø> (ø)

services/admin/src/admin/utils.py

...ation/src/mongodb_migration/database_migrations.py

services/admin/src/admin/app.py

services/admin/tests/test_prometheus.py

services/admin/src/admin/routes/force_refresh.py

.../migrations/_20221116133500_queue_job_add_force.py

services/admin/src/admin/routes/cache_reports.py

...ngodb_migration/src/mongodb_migration/migration.py

services/admin/tests/fixtures/hub.py

... and 22 more

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

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


pull request for e2e test at 01/26/2017 22:45:10

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


import error with swiper

Describe the bug
Running vite with import swiper (context inside an SFC component, see repro) fails to resolve the import to swiper (log below)
  import { Swiper, SwiperSlide } from 'swiper/vue';

Reproduction
https://codesandbox.io/s/charming-water-itnc5
System Info

required vite version: 1.0.0-rc.4
required Operating System: NixOS (linux)
required Node version: 14.9.0
Optional:

yarn version: 1.22.5
Installed vue version (from yarn.lock or package-lock.json): 3.0.0-rc.12
Installed @vue/compiler-sfc version: 3.0.0-rc.12
Installed swiper version: 6.2.0

Logs (Optional if provided reproduction)
$ vite

[vite] Error: module imports rewrite failed for<EMAIL_ADDRESS> Error: [vite] can't find package.json for a node_module file: "/@modules/swiper/core". something is wrong.
    at Object.normalizePublicPath (/sandbox/node_modules/vite/dist/node/resolver.js:223:27)
    at Object.exports.resolveImport (/sandbox/node_modules/vite/dist/node/server/serverPluginModuleRewrite.js:186:29)
    at rewriteImports (/sandbox/node_modules/vite/dist/node/server/serverPluginModuleRewrite.js:114:46)
    at app.use (/sandbox/node_modules/vite/dist/node/server/serverPluginModuleRewrite.js:61:28)
[vite] Error: module imports rewrite failed for<EMAIL_ADDRESS> Error: [vite] can't find package.json for a node_module file: "/@modules/swiper/core". something is wrong.
    at Object.normalizePublicPath (/sandbox/node_modules/vite/dist/node/resolver.js:223:27)
    at Object.exports.resolveImport (/sandbox/node_modules/vite/dist/node/server/serverPluginModuleRewrite.js:186:29)
    at rewriteImports (/sandbox/node_modules/vite/dist/node/server/serverPluginModuleRewrite.js:114:46)
    at app.use (/sandbox/node_modules/vite/dist/node/server/serverPluginModuleRewrite.js:61:28)
[vite] Error: module imports rewrite failed for<EMAIL_ADDRESS> Error: [vite] can't find package.json for a node_module file: "/@modules/swiper/core". something is wrong.
    at Object.normalizePublicPath (/sandbox/node_modules/vite/dist/node/resolver.js:223:27)
    at Object.exports.resolveImport (/sandbox/node_modules/vite/dist/node/server/serverPluginModuleRewrite.js:186:29)
    at rewriteImports (/sandbox/node_modules/vite/dist/node/server/serverPluginModuleRewrite.js:114:46)
    at app.use (/sandbox/node_modules/vite/dist/node/server/serverPluginModuleRewrite.js:61:28)

  Error: [vite] can't find package.json for a node_module file: "/@modules/swiper/core". something is wrong.
      at Object.normalizePublicPath (/sandbox/node_modules/vite/dist/node/resolver.js:223:27)
      at app.use (/sandbox/node_modules/vite/dist/node/server/serverPluginModuleRewrite.js:60:75)

Adding this module to optimizeDeps including list should solve this problem:
// vite.config.js
module.exports = {
  optimizeDeps: {
    include: ["swiper/vue"],
  },
}

It is don't work for me too, when I add optimizeDeps, it can run now, but in console of chrome, the error occured: runtime-dom.esm-bundler-4e13d33b.js:1524 Uncaught (in promise) Error: Maximum recursive updates exceeded. This means you have a reactive effect that is mutating its own dependencies and thus recursively triggering itself. Possible sources include component template, render function, updated hook or watcher source function.
at checkRecursiveUpdates (runtime-dom.esm-bundler-4e13d33b.js:1524)
at flushJobs (runtime-dom.esm-bundler-4e13d33b.js:1498)