add RequestRateLimiter demo

Please fill up those following information before send Pull request

Pull request NO:
Agent repository URL:
Testcases repository URL:

@zhaoyuguang @xiaoy00 We are setting the test case in main repo PR process, like this plugin. https://github.com/apache/skywalking/pull/3575
@zhaoyuguang I hope you could work with @dmsolr and me, to ship the gateway scenario to main repo, then this PR should land there too. What do you think?
My plan is to freeze this repo, after all immigrations finish, this repo will be achieved forever.

This repo is closing, and moving into Apache repo.

This repo is closing, and moving into Apache repo.

copy that


Issue 686/update isomorphic example from cyc

[ ] I added new tests for the issue I fixed/built
[ ] I ran make test FOO for the package FOO I'm modifying
[x] I used make commit instead of git commit

Can you rebase and remove the first commit please?

Hi @BrianDGLS , thank you for taking your time to send us this PR. We appreciate the effort, however this comes as a surprise, it's a complete rewrite of the example. I would recommend it to be a standalone example in some other repository, then published to this list https://github.com/cyclejs-community/awesome-cyclejs/ . When it comes to including it in this repo, I would recommend first discussing it in an issue. Hope you understand why. :)


op-node: log namespaced data details

Overview
This PR adds some additional logging around retrieving data from celestia, to help in debugging.
Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Full logs: https://github.com/jcstein/logs/blob/main/v0.10.0-logs-2.log
raw: https://media.githubusercontent.com/media/jcstein/logs/main/v0.10.0-logs-2.log
Relevant logs for error:
op-node_1      | t=2023-05-23T22:41:14+0000 lvl=warn msg="requesting celestia namespaced data"    origin=0xfba9093e09e45ddd6c478f770fbd5bf6750e1cec0fa95e28cdbd801ecb13adb9:4 namespace="[232 229 246 121 191 113 22 203]" height=558,982
op-node_1      | t=2023-05-23T22:41:14+0000 lvl=warn msg="unable to retrieve data from da"        origin=0xfba9093e09e45ddd6c478f770fbd5bf6750e1cec0fa95e28cdbd801ecb13adb9:4 err="share sequence has 1 shares but needed 0 shares"
op-node_1      | t=2023-05-23T22:41:14+0000 lvl=warn msg="Failed to parse frames"                 origin=0xfba9093e09e45ddd6c478f770fbd5bf6750e1cec0fa95e28cdbd801ecb13adb9:4 err="parsing frame 0: reading channel_id: unexpected EOF"

Safe block: 0
Latest block: 386+

:shipit:


Instructions are incorrect

The instructions are incorrect. I have copied the example for loading Child Features into an EPIC, and ADO says the upload has failed as it must have a Title field. In the example there is a Title 1 and a Title 2 field (no Title field)
Please advise
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 4296d1cc-f211-9d17-d97f-2b0858fdea7d
Version Independent ID: cc66dfee-adde-cdfb-b3f5-5d58d89d3c81
Content: Import update bulk work items with CSV files - Azure Boards
Content Source: docs/boards/queries/import-work-items-from-csv.md
Service: azure-devops-boards
GitHub Login: @chcomley
Microsoft Alias: chcomley

Please ignore - Despite doing nothing different, it is working now after trying again


KDP Author Account Subtype support and Sponsored Products ASIN Validation for KDP Authors are now available in the Amazon Advertising API

Support for a new Kindle Direct Publishing (KDP) author identifier is now available as a vendor subtype in the Amazon Advertising API. With this release, it is possible to identify Authors programmatically, providing developers with the ability to control access to features or differentiate the customer experience. We also added KDP Author ASIN Validation for Sponsored Products, already enforced in the Amazon Advertising Console, which prevents KDP Authors from promoting ASINs outside of their KDP catalogs. This implementation moves us closer to achieving parity between the Advertising Console UI and API for books advertisers. For more information, see the reference documentation.
https://advertising.amazon.com/API/docs/en-us/info/release-notes?order=40
https://advertising.amazon.com/API/docs/en-us/info/release-notes

@nguyentoanit please add this part
See docs: https://advertising.amazon.com/API/docs/en-us/reference/2/profiles#/Profiles/listProfiles

Specifically the subType prop from the AccountInfo.


errcheck without any errors is reported as '0%'

Hi,
for my project restic the errcheck check does not return anything, but it is reported as 0%: https://goreportcard.com/report/github.com/restic/restic#errcheck

I think that's wrong, the percentage should be the other way around...

Strange; this normally happens when a check fails to run. Whatever the reason was for the failure before though, I refreshed it just now and there were no errors:

The errcheck test is reasonably new and is giving some issues on certain repos (can be slow, or not report missing error checks), so I'm disabling it for now. We can re-enable it once the issues are sorted out.
To better address this in the future, from now on if a test fails because of a timeout or other error, the error will be reported in the UI:

Thanks for taking a look! Great project by the way :smiley:


[WIP] tridiagonal_eigh

This PR covers the mods required to implement the computation of eigenvectors and eigenvalues of a (symmetrical) tridiagonal matrix, such as the one emerging from a reduction to Hessenberg form.

At commit c0e27ff, ipu_tridiag_solve() runs a Vertex implementing Thomas algorithm.
One can run python examples/tridiag_solver_example.py 1472 1472 to solve 1472 symmetrical tridiagonal linear systems with random coefficients, each with 1472 unknowns, concurrently.
Profiling indicates this takes a total of 176,743 cycles. Memory usage was 46.7MiB (memory excluding gaps).
Note that at present, the algorithm is implemented as a Vertex, therefore using just one of the 6 threads.

With commit 85b4ebd, I added Alex's code for the computation of eigenvalues of a tridiagonal matrix and added an example in which eigenvectors of a real symmetric matrix are computed using the inverse iteration method.
The chart below shows the execution trace for the computation of the 1472 eigenvalues and eigenvectors of a 1472x1472 symmetric matrix. Only two inverse iterations were performed.
The vast majority of the 46.7M cycles are taken by the Hessenberg tridiagonalization.

There was a small but non-negligible "reconstruction" error.
Compiling module jit_real_symmetric_eigh.0:
[##################################################] 100% Compilation Finished [Elapsed: 00:04:44.4]
ic| np.max(np.abs(mat @ x.T - eig * x.T)): 0.00011253357

Memory usage is reported below. Note the huge component due to Internal Exchange Message Buffers (276MiB) and Internal Exchange Code (115MiB).


VM id and UUID are not the same

This was found when I tried to get vm id with python sdk. The vm_id and system uuid are not the same. see as below
using python sdk
>>> vm
<azure.mgmt.compute.v2018_10_01.models.virtual_machine.VirtualMachine object at 0x7ff5c9abcf10>
>>> vm.vm_id
u'7cdc1f83-c56c-4e5e-9f1f-60b0072ec8e6'
using dmidecode
root@vm:~# dmidecode -s system-uuid
831FDC7C-6CC5-5E4E-9F1F-60B0072EC8E6
This issue might be related to a bug: [SRU] Handle changing UUID endian-ness on Azure in cloud-init

Thank you for opening this issue! We are routing it to the appropriate team for follow up.

Hi @shawnswlin
I would suggest to open a support ticket on this one, since the Python SDK is just a messenger from RestAPI to Python: we don't post-process in anyways the vm_id and you get straight from RestAPI what was presented (and I have no context at the Python level to know what is right or wrong). Your command dmidecode is inside the VM, and out of scope of what I know.
Closing this issue out of scope of this SDK,
Thank you,


Adding an area that can apply physics forces to objects

Use cases:

Low gravity areas
Launchers for players / objects
Water or air currents, and conveyor belts

You could also add for com.interrupt.dungeoneer.entities.areas.PhysicsForceArea a JSON schema in jsonschema/current/dungeoneer/entities/area/PhysicsForceArea.schema.json and add a reference to it in jsonschema/current/Interfaces.schema.json:AreaLike.


Remove logger crate in Chronicle-Common and just adopt the Bee logger

Need to remove the #[derive(Debug)] for any types which use the logger

Current LoggerConfigBuilder in bee-common has not Clone trait. There will be a problem by just adopting it, because Rust does not allow deriving external traits on external structs.
Either we need to refactory the usage of LoggerConfigBuilder in Chronicle, or the LoggerConfigBuilder should have the derived Clone trait.

Closing this for now as we already discussed to settle on the current implementation


cannot use L515 in Unity

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
L515

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC

SDK Version
v2.50.0

Language
unity 2020.1.8f1

Segment
VR/AR

Issue Description
I downloaded unity package latest version and the others im using are latest version. I played on start scene but i couldnt use L515.. error message like below

what can i do currently??

Hi @heojun990322  The L515 camera model has been compatible with the RealSense Unity wrapper since SDK version 2.37.0 and the Unity wrapper is also compatible with Unity 2020.
May I first confirm that you are able to use the camera normally outside of Unity in the RealSense Viewer tool, please?
If the camera does work correctly in the Viewer, have you followed the Unity wrapper setup method below.

Download the Unity package file Intel.RealSense.unitypackage from the 'Assets' file list for version 2.50.0 of the SDK.

https://github.com/IntelRealSense/librealsense/releases/tag/v2.50.0

Launch Unity and open a project.  A clean new project will be best for testing purposes.

Go to Assets > Import Package > Custom Package and select the Intel.RealSense.unitypackage file from the location on your computer that it was downloaded to (likely to be the Downloads directory of your Windows user account folder).  The RealSense Unity wrapper should then import into your project.

Go to the Assets > RealSenseSDK2.0 > Scenes folder in your Unity 'Assets' panel and double-leftclick on the scene file called StartHere.  When the scene has loaded, click on the small triangular 'Play' button at the top of your Unity screen  (not the File > Build and Run method) to run the scene in the Unity editor.

Thnks for reply @MartyG-RealSense
I confirmed my L515 works correctly in the RealSense Viewer tool.
Then I followed steps you suggested from 1 to 4 accurately.
But I got same result.
When I clicked play button is

And when I clicked start is

Below is console..

The warning Couldn't resolve requests indicates that a resolution or FPS speed was being requested that the L515 camera does not support.
Could you provide an image of profile details listed in the RsDevice component for the 'StartHere' scene please?
https://github.com/IntelRealSense/librealsense/tree/master/wrappers/unity#realsensedevice

Profile listed in my RsDevice component

The camera profile looks fine for an L515 that has firmware <IP_ADDRESS> installed (to provide 640x480 RGB support).
I ran my own tests in Unity 2021.1.16f1 with an L515 using the 4-step procedure I described earlier.  I didn't experience the ExternalException red error that you did.  Furthermore, my StartHere scene had a full list of samples provided when run.

The sample programs were also able to run correctly on my L515.

I had downloaded Unity 2021.1.16f1 and tesed L515 following steps again. But I experienced same problems.
And I had tried it on my labtop too, not my PC. Strangely, the result was same..

You have said that your L515 camera works correctly in the Viewer, which suggests that it is a problem that is not related to the camera hardware or the firmware driver.
Is there any difference if you unplug the micro-sized end of the USB cable from the base of the L515 camera, turn the connector around the other way and re-insert it into the camera base (USB-C cables are two-way insertion at the micro-sized end).

Hi @heojun990322  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


Latest version throwing syntax errors on startup

user@ubuntu:~/isp-data-pollution$ sudo python3 ./isp_data_pollution.py
  File "./isp_data_pollution.py", line 218
    print(f'This is ISP Data Pollution 🐙💨, Version {__version__}')

SyntaxError: invalid syntax

Is python3.6 needed to run string-formatting paradigms?

Yes, and possibly more importantly for chromedriver compatibility, though I haven't checked.
On other fronts I'm being pushed to move to 3.7.

Thanks, after installing 3.6 I was up and running!
Quick question: What does "timeout" and "short_timeout" change? Also, is there a way to randomize the request interval? e.g. Make a HTTP request, waits 2 min, requests, waits 5 min, etc?

timeout and short_timeout` control how long the main process waits for a browser child process response before killing its child and moving on. I wouldn't mess with these unless you understand the specifics or like to watch a zillion headless Chrome browsers launched in the background.
The time differences between the requests are already randomized. Grep for the .chi2_mean_std method. The current parameters are hardcoded in specific calls roughly based on average browsing habits.

Outstanding, thank you sir.


autodisconnect after few seconds

Hi!
I have just installed your amazing app and I am having a little problem.
When I log into a machine via RDP, session disconnects automatically after a few seconds.
If i log in again, session is when i left it.
Could you help me?

Hi, it's possible that (non exhaustively) you reached the maximum number of connections on the remote machine or the maximum number of connections for the user, or a Windows update was pending, or that someone else disconnected you by connecting the machine, etc.
It's a matter of configuration. You should enable the ability for a user to open multiple sessions on the remote machine by disabling the single user session restriction and the maximum number of connections  as explained there: https://portal.databasemart.com/kb/a220/how-to-enable-multiple-single-remote-desktop-sessions-in-windows-server-2012.aspx
If it's not enough, you could enable the wfreerdp logs (https://github.com/cedrozor/myrtille/blob/master/Myrtille.Services/app.config#L190), restart Myrtille.Services (windows service) and check the logs (into the <myrtille install path\log> folder)

Hi,
I am sure the maximum number of connections has not been reached.
In log file, I see the following:
2019-05-27 17:16:34,932 [30] INFO  System.Diagnostics redirection [(null)] - Connecting remote session de410bec-d13a-4c9d-89cf-a9c8cfd4e333, type RDP, security AUTO, server (:port) <IP_ADDRESS>:25222, vm (none), domain AXIUM, user david, program (none)
2019-05-27 17:17:21,342 [30] INFO  System.Diagnostics redirection [(null)] - Disconnected remote session de410bec-d13a-4c9d-89cf-a9c8cfd4e333, exit code 0
2019-05-27 17:17:21,358 [30] INFO  System.Diagnostics redirection [(null)] - Requesting session host list
<EMAIL_ADDRESS>Nacho
Escalada
Sistemas y Comunicaciones
<EMAIL_ADDRESS>948985582tel:948985582
<EMAIL_ADDRESS>nacho<EMAIL_ADDRESS><EMAIL_ADDRESS>grupoaxium.comhttps://grupoaxium.com/
Blanca de Navarra, 3 ‑ Bajo
31500
‑
Tudela
(Navarra)
Este mensaje y sus archivos adjuntos van dirigidos exclusivamente a su destinatario, pudiendo contener información confidencial sometida a secreto profesional. No está permitida su reproducción o distribución sin nuestra autorización expresa. Si usted no es el destinatario final por favor elimínelo e infórmenos por esta vía.
De acuerdo con lo establecido por el REGLAMENTO (UE) 2016/679 DEL PARLAMENTO EUROPEO Y DEL CONSEJO de 27 de abril de 2016, le informamos que sus datos están incorporados al sistema de tratamiento titularidad de Axium Soluciones S.L con la finalidad de prestarle los servicios que nos ha solicitado, atender sus solicitudes de información y poder enviarle comunicaciones comerciales que le puedan interesar, legitimados por la relación comercial que nos une, por su solicitud o bien por nuestro interés legítimo. Los datos proporcionados se conservarán mientras se mantenga la relación comercial o durante los años necesarios para cumplir con las obligaciones legales. Los datos no se cederán a terceros salvo en los casos en que exista una obligación legal. Los datos han sido obtenidos del propio interesado. Atenderemos sus derechos en<EMAIL_ADDRESS>.Puede consultar información detallada sobre sus derechos y nuestra política en www. grupoaxium.com/política-de-privacidad.
De: cedrozor<EMAIL_ADDRESS>Enviado el: lunes, 27 de mayo de 2019 15:07
Para: cedrozor/myrtille<EMAIL_ADDRESS>CC: Nacho Escalada - Grupo Axium<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Asunto: Re: [cedrozor/myrtille] autodisconnect after few seconds (#155)
Hi, it's possible that (non exhaustively) you reached the maximum number of connections on the remote machine or the maximum number of connections for the user, or a Windows update was pending, or that someone else disconnected you by connecting the machine, etc.
It's a matter of configuration. You should enable the ability for a user to open multiple sessions on the remote machine by disabling the single user session restriction and the maximum number of connections as explained there: https://portal.databasemart.com/kb/a220/how-to-enable-multiple-single-remote-desktop-sessions-in-windows-server-2012.aspx
If it's not enough, you could enable the wfreerdp logs (https://github.com/cedrozor/myrtille/blob/master/Myrtille.Services/app.config#L190), restart Myrtille.Services (windows service) and check the logs (into the <myrtille install path\log> folder)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/cedrozor/myrtille/issues/155?email_source=notifications&email_token=AMFIE7XWBFVR4UOQ3PE7DQ3PXPMIZA5CNFSM4HPUKBT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWJYIVQ#issuecomment-496206934, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMFIE7TNDQOECBFTYSWUNF3PXPMIZANCNFSM4HPUKBTQ.

Thanks for the logs. From there, I can see the disconnect is not a few seconds after but almost 50 secs. Once connected, can you interact with the session? if you can't, Myrtille will assume a network issue and disconnect the user after 1mn (https://github.com/cedrozor/myrtille/blob/master/Myrtille.Web/Web.Base.config#L154)
I also see en exit code 0, meaning a "normal" disconnect.
Can you paste the content of wfreerdp_.log when the problem occurs?

Hi,
Yes, I can interact with the session.
Where wfreerdp log is located?
I think it is a problema in the rdp server, not myrtille, but I am not sure. This server has Remote Desktop Services enabled. Could it be the problem?
Regards,
<EMAIL_ADDRESS>Nacho
Escalada
Sistemas y Comunicaciones
<EMAIL_ADDRESS>948985582tel:948985582
<EMAIL_ADDRESS>nacho<EMAIL_ADDRESS><EMAIL_ADDRESS>grupoaxium.comhttps://grupoaxium.com/
Blanca de Navarra, 3 ‑ Bajo
31500
‑
Tudela
(Navarra)
Este mensaje y sus archivos adjuntos van dirigidos exclusivamente a su destinatario, pudiendo contener información confidencial sometida a secreto profesional. No está permitida su reproducción o distribución sin nuestra autorización expresa. Si usted no es el destinatario final por favor elimínelo e infórmenos por esta vía.
De acuerdo con lo establecido por el REGLAMENTO (UE) 2016/679 DEL PARLAMENTO EUROPEO Y DEL CONSEJO de 27 de abril de 2016, le informamos que sus datos están incorporados al sistema de tratamiento titularidad de Axium Soluciones S.L con la finalidad de prestarle los servicios que nos ha solicitado, atender sus solicitudes de información y poder enviarle comunicaciones comerciales que le puedan interesar, legitimados por la relación comercial que nos une, por su solicitud o bien por nuestro interés legítimo. Los datos proporcionados se conservarán mientras se mantenga la relación comercial o durante los años necesarios para cumplir con las obligaciones legales. Los datos no se cederán a terceros salvo en los casos en que exista una obligación legal. Los datos han sido obtenidos del propio interesado. Atenderemos sus derechos en<EMAIL_ADDRESS>.Puede consultar información detallada sobre sus derechos y nuestra política en www. grupoaxium.com/política-de-privacidad.
De: cedrozor<EMAIL_ADDRESS>Enviado el: martes, 28 de mayo de 2019 0:03
Para: cedrozor/myrtille<EMAIL_ADDRESS>CC: Nacho Escalada - Grupo Axium<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Asunto: Re: [cedrozor/myrtille] autodisconnect after few seconds (#155)
Thanks for the logs. From there, I can see the disconnect is not a few seconds after but almost 50 secs. Once connected, can you interact with the session? if you can't, Myrtille will assume a network issue and disconnect the user after 1mn (https://github.com/cedrozor/myrtille/blob/master/Myrtille.Web/Web.Base.config#L154)
I also see en exit code 0, meaning a "normal" disconnect.
Can you paste the content of wfreerdp_.log when the problem occurs?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/cedrozor/myrtille/issues/155?email_source=notifications&email_token=AMFIE7U2CXSBEPOXJAJIRHDPXRLANA5CNFSM4HPUKBT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWKSAHY#issuecomment-496312351, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMFIE7RORQK63EL2O2B3TO3PXRLANANCNFSM4HPUKBTQ.

wfreerdp logs are located into the myrtille log folder, if you enabled them: https://github.com/cedrozor/myrtille/blob/master/Myrtille.Services/app.config#L190. Myrtille.Services need to be restarted.
The server need to have Remote Desktop Services enabled, so it's not the problem.

Hi, any news ?

Hey,
we also encounter this behavior, "all of the servers disconnect the user from themselves within seconds of no activity being detected. This means that I need to get back and then reconnect to the server. This means that I cannot watch processes (such as the installation or configuration) that last more than 20 seconds" as a user reported.
What keeps the connection/session alive even when the tab is not on focus ?

Hi Zak,
The myrtille "ClientIdleTimeout", which disconnect the client after a given inactivity timeout, is disabled by default (https://github.com/cedrozor/myrtille/blob/master/Myrtille.Web/Web.Base.config#L181). Can you check that param to see if it's enabled (and which value)?
If enabled (value > 0, in ms), it overrides but doesn't remove the RDS idle timeout (default 20mn). You should check the various timeouts for your RDS session collection (https://serverfault.com/questions/873399/rds-2016-session-limits).
Good question if the window/tab doesn't have focus... In my previous tests, the periodical FSU (fullscreen update) was correctly sent every 30 secs. If this "heartbeat" is not received and "ClientIdleTimeout" is set, myrtille will indeed disconnect the remote session(s) after that time.
Which browser are you using? I know Chrome changed many times such "out of focus" behavior. I will re-check it.
Meanwhile, you should set "ClientIdleTimeout" to 0 if it isn't.

@MrZakos do you share the session? guests are disconnected after 1mn of inactivity (no "heartbeat") regardless of the "ClientIdleTimeout" param (which only applies to the owner). So, if the window/tab is not focused and the browser stops executing the javascript code (!), this explains your issue.
I will confirm that asap. If it's not reliable, I will have to either remove this feature or have a clear warning if the window/tab loose focus.

Hey,
ClientIdleTimeout is set on default value 60000 (I can check value 0 on other environment later on)
there are no 2 users on the same session so no worry on this angle,
I checked on Chrome and I think most users are using chrome/firefox,
I think warning is nice but not a solution ,
thanks :)

I will have the "heartbeat" into a (background) web worker so that it isn't throttled by the tab being inactive.
For now, you can set ClientIdleTimeout to 0 (to disable the feature).

Hey,
I changed ClientIdleTimeout to 0 but still same  beahavior , a client response :
"I just checked and nothing has changed.

As planned, I had the heartbeat into a web worker so that it should tick even if the tab isn't focused.  I will try to release it this month.

great, thanks!

Hey , I just saw Version 2.7.0  released - does this fix the issue ?

Hi, yes it should. Tell me if it doesn't.

@MrZakos, please also tell me if it does, so I can close this issue.

hey ,
it seems it's working , it doesn't disconnect when tab is not focused.
but I found something else , when I resize the browser (chrome) , it reconnect with correct size - but
it's freeze - I see the machine screen but can't do anything

Hi Zak, thanks for your input. Indeed, there is something that escaped the tests :/ I identified the problem and will make an hotfix asap for the 2.7.0.

everything should be fixed now (hotfixed the 2.7.0), please reopen as needed


Model application name and system tag checks

Closes #422.

There's no Haskell implementation of these two things. They shouldn't be hard to add.

This PR concerns the formal spec only.

This PR concerns the formal spec only.

To avoid further drifting apart of the LaTeX and Haskell counterparts of the spec, I'd suggest doing this in a lockstep fashion.


Fail to analyze GRDB

Hello!
swift-package-info --for https://github.com/groue/GRDB.swift -v 5.4 --product GRDB
->
Cannot initialize Kind from invalid String value system

Hey @MontakOleg , thanks for the report :).
I'll look into that, most likely an unmapped scenario on Package.swift decoding. Since there's no contract/specification, we might find one or other cases to fix.


Switching IL Views in Linear View Has Wrong Line Selection

Switching IL views in linear view works most of the time except in LLIL <--> MLIL, in which case the wrong line ends up being selected, even though bv.offset produces the correct result.

This is now fixed in 2.1.2440-dev. Selections of multiple lines are maintained as well on a best effort basis.

It turns out this is better, however not 100% fixed.

Fixed in 2.2.2614-dev.

Fixed in 2.2.2614-dev.


Run commands without a GPU

The problem
I'm working on a GitHub Actions workflow to cog push a model to Replicate. Part of the release process includes running a script to download weights:
cog run script/download-weights

Unfortunately that script can't be run in the Actions workflow, because GitHub's hosted runners don't (yet?) have GPUs, and this model has gpu: true set in it's cog.yaml file. But there isn't anything in the download-weights script that specifically requires a GPU to run. The current workaround is to approximate the Cog environment by manually installing Python, installing Python dependencies, etc, but that's a pain.
A possible solution
It would be nice to be able to run one-off cog commands and somehow specify that the GPU is not needed:
cog run script/download-weights --no-gpu-needed-thanks

Builds work without a gpu. There's logic that can be reused there.

being able to build and test on a mac seems like it is an issue hitting some folks on discord

Dupe https://github.com/replicate/cog/issues/590


Data and model license clarification

The CC-BY license is very explicit about permitting commercial use, contrary to the description in the README:
https://github.com/datamol-io/safe/blob/main/README.md?plain=1#L163
Can you clarify what the actual license is? Is it actually CC-BY, or did you intend to apply a non-commercial license?
Thanks.

Hi @rkern, I have updated the README.  Thanks for noticing this.

The training dataset is licensed under CC BY 4.0. See DATA_LICENSE for details. This code base is licensed under the Apache-2.0 license. See LICENSE for details.
Note that the model weights of SAFE-GPT are exclusively licensed for research purposes (CC BY-NC 4.0).

It's only the current model weight of SAFE-GPT that is CC BY-NC.


Update test alerts

I suspect that if we merge https://github.com/infrawatch/functional-tests/pull/45 and then rebase this on master, the errors will go away

Since @lnatapov tested this and approves, I'll merge the PR


Cyclomatic Complexity

Currently Documentation::getDescription has a cycolmatic complexity of 12. The mess detector rules say this must be below 10
    /**
     * Gets a methods description.
     * This is an example of a description. In PSR-5 it follows the summary.
     * @param string[] $lines
     * @return string
     */
    protected function getDescription(array $lines)
    {
        $description = '';
        $summaryFound = false;
        $summaryPassed = false;
        foreach ($lines as $line) {
            if ($line && !$summaryPassed) {
                $summaryFound = true;
                if (substr(trim($line), -1) == '.') {
                    $summaryPassed = true;
                }
                continue;
            }
            if (!$line && $summaryFound && !$summaryPassed) {
                $summaryPassed = true;
                continue;
            }
            if ($line && $line[0] == '@') {
                break;
            }
            if ($line && $summaryPassed) {
                $description .= $line."\n";
            }
        }
        return trim($description);
    }

This can be reduced by moving some of the complex comparisons into seperate methods, eg:
    private function isEndOfSummary($line, $summaryFound, $summaryPassed)
    {
        return (!$line || substr(trim($line), -1) == '.') && $summaryFound && !$summaryPassed;
    }

    private function hasDescriptionEnded($line)
    {
        return $line && $line[0] == '@';
    }

    /**
     * Gets a methods description.
     * This is an example of a description. In PSR-5 it follows the summary.
     * @param string[] $lines
     * @return string
     */
    protected function getDescription(array $lines)
    {
        $description = '';
        $summaryFound = false;
        $summaryPassed = false;

        foreach ($lines as $line) {
            if ($line && !$summaryPassed) {
                $summaryFound = true;
            }
            if ($this->isEndOfSummary($line, $summaryFound, $summaryPassed)) {
                $summaryPassed = true;
                continue;
            }
            if ($this->hasDescriptionEnded($line)) {
                break;
            }
            if ($line && $summaryPassed) {
                $description .= $line."\n";
            }
        }
        return trim($description);
    }

This takes the complexity below 10 and still passes the tests, though could be reduced further by moving the other if's out.
The question is, is it worth it, and do there methods belong here in the Documentation class?

We can break this class up into multiple sub classes or traits. For example:
AyeAye\Api\Documentation\Summary
AyeAye\Api\Documentation\Description
AyeAye\Api\Documentation\Parameters
AyeAye\Api\Documentation\ReturnType

This would allow us to tidy away the numerous one line methods that can be extracted from these larger more complicated methods, and could allow for easier dependency injection in the event somone wants to change the way any part of it works.


[Feature request] Support for tRPC v11

We're releasing a new major of tRPC! 🚀
It looks like you're not importing anything that should break stuff, but would be nice to add || ^11.0.0-next-alpha.175 as peer dependency and updating your deps to use @next & see if it works
https://trpc.io/docs/v11/migrate-from-v10-to-v11

If there are issues when you just bump your deps to @next, please open an issue on the tRPC repo! 🙏

There might be some missing deps, if you just get us a list we'll add them

I'm working on fixing this 👀


As a CIF administrator, I want to describe the relationships between funding stream phases so that I can define how a project can enter each phase

Description:
As a CIF administrator, I want to describe the relationships between funding stream phases
Acceptance Criteria:
Given
When
Then
Development Checklist:

[Other AC]
[Other AC]
[Other AC]
Meets the DOD

Personas:

Program Development Team
Operations Team

Notes:

Adding applicable Personas to the ticket
Program Development Team - CIF Persona
Operations Team - Persona

Let's close this until there's an actual need to add constraints for project statuses transitions

@matthieu-foucault - should we close this or just add it to release #2?

Well we haven't established that this is a need yet. We'll want users to play with the application first, to see if they need more constraints. This may be more relevant once we open the application to external users


Add "geom" framework

Add the geom framework (and directory).
Description

The framework will be middleware for MAPL3. Also, it will a MAPL geometric type.

How Has This Been Tested?

This is new functionality. It has been tested internally, though some tests are disabled because some of the framework has not been implemented.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[ ] I have tested this change with a run of GEOSgcm (if non-trivial)
[x] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[x] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

There are some tests that I am working on. Will post updates to branch as I finish the tests.

Disabled the @assertEqual in test_FieldAXPY_R8 temporarily.

Intel build/test works. GNU build/tests fails on test_FieldCOPY_R8R4. The conversion from REAL64 to REAL32 generates a difference below threshold for Intel, but the difference is above threshold for GNU.
The test formula is:
$\frac{\textrm{abs}(x-y)}{\textrm{abs}(y)} < \textrm{epsilon}(\textrm{real}(1.0, \textrm{kind}=\verb|ESMF_KIND_R4|))$
$x$ is a REAL64 ESMF_Field and $y$ is a REAL32 ESMF_Field. $x$ is copied to $y$, which should cause a loss of precision, but the loss of precision should be less be less than the precision of the REAL32 ESMF_Field.
I'll look at this over the weekend.

@darianboggs   The test does not need to be so strict.   The failure you sight is not a failure in your implementation but in how the individual compiler is doing its arithmetic, which is beyond the scope of a unit test.   If I see a simple relaxation, I'll make the change and approve.

OK - I'm flumoxed.   The values in the test look like they should be integers.  Roundoff should not be happening and you should be getting exact equality.
Side note.   pFUnit has tolerance options (absolute and relative) for assertEqual.     (This is part of why I encouraged you to work with me once you had sketched out a few tests.    You've done some good work, but ... could have saved yourself some difficulty.)

Note: I updated the CHANGELOG.md in this branch because it looked like a merge got a bit...wonky.

Also: Lots of the modules have mapl3g in them but the PR is to develop. Is that right?

Only one module is activated. The others should not have been added, but were due to insufficient instructions on my part.
Not harming anything for now though.

GNU seems to have failed one of the new unit tests.


Workspace wrong population - files renamed and moved out of folder

Hello and thank you for the awesome extension!
I have the following issue while using it:
We have this configuration:

{
"sourceDirectory": "src",
"sharePointSiteUrl": "https://domain/subsiteName",
"publishingScope": "None",
"authenticationType": "Digest",
"remoteFolders": [
"/SiteAssets/**/."
]
}

The real "Site Assets" structure on the server is:
SiteAssets/js/subsiteName111.xxx.js
SiteAssets/js/subsiteName222.yyy.js
SiteAssets/js/subsiteName333.zzz.js
File names start with the same name as the subsiteName.
After populating the workspace the js files are shown outside the js folder and are renamed. The populated structure is the following:
SiteAssets/js111.xxx.js
SiteAssets/js222.yyy.js
SiteAssets/js333.zzz.js
If the file names do not contain the subsite name everything is fine.

@tockata I can confirm this is a bug. I will resolve this in the next version of SPGo!

@tockata - I am working this issue right now with @koltyakov over here https://github.com/koltyakov/sppull/issues/27. Please feel free to add any additional details I may have missed.

This is the same problem that i have now.

All- I've just posted SPGo 1.3.5 to the VSCode Marketplace which will resolve this issue. Please reopen this issue if you are still experiencing issues.


logs_contain can't find target logs

Related to https://github.com/dbrgn/tracing-test/issues/2
Confirming that tracing-test doesn't support target right now.

@mooreniemi thanks for the reproducer! I moved it into the issue at #4!


spring cloud gateway memory leak

Please see https://github.com/reactor/reactor-netty/issues/2384

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Do not subtract list item indent if it has more than one blank line

In this situation, the item cannot contain any nested blocks, so it should not be subtracted from the line start.
Fixes #751

Thanks!


Add GPTQ Quantization

What does this PR do?
This PR adds the possibility to perform GTPQ quantization. I tried to be as generic as possible to support any kind of models ( whether it is Transformer models or not). The backend relies on auto_gptq library where we use GTPQ class and QuantLinear class. With this API you can do conversion, saving and loading quantized weights. We have a dependence on accelerate to save the weights and load the quantized weights efficiently without allocating more memory than needed.
Quantize model:
model_name = "bigscience/bloom-1b7"
from transformers import AutoModelForCausalLM, AutoTokenizer
from optimum.gptq import GPTQQuantizer, load_quantized_model
from transformers import AutoTokenizer

tokenizer = AutoTokenizer.from_pretrained(model_name, use_fast=True)
model = AutoModelForCausalLM.from_pretrained(model_name, torch_dtype = torch.float16, device_map = "auto")
quantizer = GPTQQuantizer(bits=4, group_size=128, desc_act=False)
quantized_model = quantizer.quantize_model(quantized_model, tokenizer, dataset="c4")

Save model
save_folder = 'bloom-1b7-quantized_optimum'
quantizer.save(model,save_folder)

Convert model and Load quantized weights
from accelerate import init_empty_weights
with init_empty_weights():
    empty_model = AutoModelForCausalLM.from_pretrained(model_name, torch_dtype=torch.float16)
empty_model.tie_weights()
model_from_saved = load_quantized_model(empty_model, save_folder=save_folder, device_map="auto")

PS: In a follow-up PR, I will use this API to integrate GTPQ quantization into transformers + modify from_pretrained and  save_pretrained so that we take into account GPTQ quantized model, just like what we did for bitsandbytes
TODO:

[ ] Doc
[ ] CI
[ ] script to calculate the perplexity
[ ] cpu offload (for now, if some modules are offloaded to the cpu (using device_map), the quantization will not work )

I still think that using exllama kernel would be very interesting (for ref https://github.com/fxmarty/q4f16-gemm-gemv-benchmark & huggingface/text-generation-inference#553 (comment), second one is with Triton, that I see is not used here), but integrating with AutoGPTQ is already a nice step (making sure that cuda-old is used when possible).

Yes, adding exllama kernel would be awesome. The best would to be add it on auto_gtpq library.

Is there a hard requirement on accelerate? If so, why?

For the quantization, no hard requirement on accelerate. However, there is a hard requirement for saving and loading the quantized weights. I did that to mimic the behavior that we will have in transformers integration (empty model + loading weights + dispatch according to the device_map). To be clear, the saving and loading function in GPTQQuantizer are not mean to be used for HF transformers model. It is just a way for non Transformer model to save and load their model.

It is also not clear to me what should go in accelerate and optimum, if this PR puts a hard requirement on accelerate. Why not put it in accelerate directly? Why bitsandbytes in accelerate & this in optimum? I think it can be quite confusing to users.

To me, the hard requirement on accelerate is not a big problem as transformers has accelerate as core depedency and optimum is dependent on transformers. However, it might be confusing to users since bnb is on accelerate.
wdyt @younesbelkada ?

Checking in from a transformers API perspective: looks good to me, assuming the from_pretrained and save_pretrained utilities are going to be implemented as described in the plan 🙌
BTW, on script to calculate the perplexity -> this will be VERY helpful to the community, we should definitely publicize it when it is ready!

Can you run make style?


Instance migration refinements

This PR addresses some of the comments made on PR #1226.  It renames some variables, factorizes some code, adds comment to some confusing parts of launcher and fixes a nasty bug in scheduler that could have lead to restarted instances being deleted if the attempt to restart them failed in scheduler.

Coverage increased (+0.01%) to 65.64% when pulling 12c3de9d5c3ddf2326286f30b60906426219fe36 on markdryan:migration into 8de4c246f3667cb556acd916efb18d086f93aeab on 01org:master.

Coverage increased (+0.01%) to 65.64% when pulling 12c3de9d5c3ddf2326286f30b60906426219fe36 on markdryan:migration into 8de4c246f3667cb556acd916efb18d086f93aeab on 01org:master.

@tpepper Could you check the scheduler changes?


Walkthrough - Documentation - Login step duplicated across sections of a task

Description
While on a single task of a walkthrough, the first step of each section asks the user to login to a Red Hat application to perform the steps. For example, task 1 of walkthrough 1 asks the user to login to Red Hat Fuse Online for both section 1.1 and 1.2. In my opinion, since the user is already logged in from section 1.1, the step to log in on 1.2 is not necessary.
Expected Behavior
Reduce the number of steps by removing duplicate login steps that have already been performed.
Actual Behavior
Login steps duplicated on walkthrough tasks
Steps to Reproduce

Select the Integrating event-driven and API-driven applications (AMQ) walkthrough
Click Get Started
Step 1 from both section 1.1 and 1.2 require the user to log into the Red Hat Fuse Online console.

Specifications

Browser: Chrome
Browser version: 70.0.3538.77
OS: MacOS Mojave

Additional Notes

@stacymcauliffe We discussed this and decide to leave as-is for now, is that correct?

We either want to repeat this step consistently at the beginning of each task, or we want to say it only once and assume the user will understand that they are staying in the application.
For users who go through an entire walkthrough end to end in one sitting, the latter approach is preferable. The risk we take with that approach is the user who stops in the middle of a walkthrough and then comes back to the web app will have no obvious way to log in to their console again.
The simplest solution IMO is to repeat the log in info at the beginning of each task. We could also spend some time considering a design solution for this story if the team thinks it's very disruptive to repeat that log in step (as a user, I want to be able to return to a walkthrough I've partially completed and get the correct log in information).

closing, might need to be reopened in the context of reviewing  partially completed walkthroughs


Use vcpython27 archive URL as a fix for fakenet-ng.python.fireeye

The vcpython27 installer was removed by Microsoft (as it's now unsupported). However, in the latest available Flare VM release fakenet-ng.python.fireeye still requires vcpython27 to properly install.

Archive.org has a copy of the original installer available at https://web.archive.org/web/20210106040222/download.microsoft.com/download/7/9/6/796EF2E4-801B-4FC4-AB28-B59FBF6D907B/VCForPython27.msi
I was able to install fakenet-ng.python.fireeye by updating the URL in a local copy of the vcpython27 chocolateyinstall.ps1 file and then installing it using choco pack followed by choco install vcpython27 -s "$pwd" -f (in PowerShell). Once vcpython27 was installed, fakenet-ng.python.fireeye installed fine as well.
Can Fireeye create and host a vcpython27.fireeye version of the Chocolatey vcpython27 package using the archive.org URL noted above as a workaround? Chocolatey comments for the vcpython27 package indicate that they will also be removing the vcpython27 package soon, potentially making fakenet-ng.python.fireeye uninstallable.

@crahan
Thanks for finding a link for this! However, instead of relying on that I've added a new package called fakenet-ng.fireeye that uses the FakeNet-NG releases. I've also updated the flare vm installers to use fakenet-ng.fireeye rather than fakenet-ng.python.fireeye.
Please give it a whirl and let me know if you have any issues!

Thanks!
I uninstalled vcpython27, uninstalled fakenet-ng.python.fireeye, installed fakenet-ng.fireeye and did not run into any issues (except for having to manually remove the original fakenet-ng shortcut and fakenet_logs desktop folder).


promqltest: run each eval command in a subtest

This PR modifies the behaviour of the promqltest package to run each eval command in a subtest.
This means we gain all the quality of life features that come with subtests such as filtering and granular reporting of failures.

Thanks. I think this is quite complex and we could probably achieve it more easily since we pass down t already?

I've changed to a different approach in a0843de based on this.
It's still a bit awkward as we can't easily mock t, so I've had to add a method that the test for runTest can override to capture the error.
Let me know what you think.

What do you think of #14638 ?

Looks good to me, I'm happy with that if you're happy with that. I'll leave some minor comments on that PR.

Closing in favour of https://github.com/prometheus/prometheus/pull/14638


How to highlight the specific text？

How to highlight the specific text like README picture shows (yellow)
typora version : 0.9.93
OS : windows10 1909
I doubt the version don't support this feature as picture shows blew

Check the Markdown in the preferences Settings to see if the highlight is on.

@MarMomento It's worked. thx~


support NODE_OPTIONS

currently you need to pass --loader=esmock as an argument to node because esmock checks only process.argv.
It would be good to support NODE_OPTIONS env var too.

@cjroebuck would you show how esmock should be started with NODE_OPTIONS? maybe support could be added.

It would be something like:
NODE_OPTIONS='--loader=esmock' node src/whatever.js
Thank you very much for esmock

related https://github.com/TypeStrong/ts-node/discussions/1748


Update the Registration dialog form's design

💡 Summary
Update the Registration dialog form's design
Motivation and context
This would be useful because currently there are spacing issues and the code does not align with MUI standards. Also to add a feature that will allow a user know they have filled out the registration form and to expect an email.
Implementation notes
Functionality would stay the same.
Acceptance criteria
How do we know when this work is done?

[ ] Appearance of the dialog and its form has changed.
[ ] A popup that let's you know you will receive an email.

On hold. Blocked by #2364


[BUG] grpc: error while marshaling: string field contains invalid UTF-8

Describe the bug
Unable to restore from backup
To Reproduce
backup some volumes with 1.6.1 version, restore them on a fresh installed 1.6.1 version.
In longhorn-manager-* logs I found:
2024-09-25T16:40:32+02:00 time="2024-09-25T14:40:32Z" level=info msg="Restoring backup" func="controller.(*EngineMonitor).restoreBackup" file="engine_controller.go:1498" backupTarget="cifs://<IP_ADDRESS>/NAS/NAS/Backup/longhorn/" backupVolume=pvc-ccfc0070-821a-4776-86ff-59ea7ff0a08e controller=longhorn-engine engine=pvc-ccfc0070-821a-4776-86ff-59ea7ff0a08e-e-0 lastRestoredBackupName= node=k0s-05 requestedRestoredBackupName=backup-0b0ad915e4c64427
2024-09-25T16:40:32+02:00 E0925 14:40:32.925383       1 engine_controller.go:842] failed to update status for engine pvc-ccfc0070-821a-4776-86ff-59ea7ff0a08e-e-0: failed to restore backup backup-0b0ad915e4c64427 in engine monitor, will retry the restore later: proxyServer=<IP_ADDRESS>:8501 destination=<IP_ADDRESS>:10478: failed to restore backup cifs://<IP_ADDRESS>/NAS/NAS/Backup/longhorn/?backup=backup-0b0ad915e4c64427&volume=pvc-ccfc0070-821a-4776-86ff-59ea7ff0a08e to volume pvc-ccfc0070-821a-4776-86ff-59ea7ff0a08e: rpc error: code = Internal desc = grpc: error while marshaling: string field contains invalid UTF-8

Expected behavior
The restore should start
Support bundle for troubleshooting
I'll attach the support bundle after the issue creation
Environment

Longhorn version:  1.6.1
Impacted volume (PV): all but you can refer to pvc-ccfc0070-821a-4776-86ff-59ea7ff0a08e
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Helm
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: K0s v1.30.4

Number of control plane nodes in the cluster: 3
Number of worker nodes in the cluster: 4

Node config

OS type and version: Ubuntu 24.04
Kernel version: 5.15.0
CPU per node: 1
Memory per node: 8/16
Disk type (e.g. SSD/NVMe/HDD): SSD/HDD
Network bandwidth between the nodes (Gbps): 1Gbps

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetal
Number of Longhorn volumes in the cluster: more or less 50

I'll attach the support bundle after the issue creation

The file generated is too big. I uploaded and shared on google drive:
https://drive.google.com/file/d/1_9mtQ19KISeKQtNoGzIa61VA0KKHEyLo/view?usp=sharing

After more or less 16 hours, some restore were completed (for example pvc-66ef9037-8c87-48d1-b8ac-ad59fe608e9a).
I generated and uploaded a new support bundle, if useful: supportbundle_b30b025b-2c9e-4ebe-af97-29be9622d7f3_2024-09-26T07-31-29Z.zip

Just checked the log. There are tons of such errors for multiple volumes in the longhorn manager pods, while there is no restore-related log in all instance manager pods at all. One typical reason that leads to this case is, the gRPC clients/servers (proxy gRPC server in Longhorn) somehow messed up the API definition in the proto file: https://stackoverflow.com/questions/70489810/i-get-grpc-invalid-utf-8-when-i-make-a-request-to-grpc-endpoint-from-flutter-app
Then I checked the image for the related Longhorn workload pods. The result is that everyone is using v1.6.1 version hence there should not be a gRPC client/server version mismatching issue...
BTW, the valid volume pvc-66ef9037-8c87-48d1-b8ac-ad59fe608e9a you mentioned did experience a restore. I haven't found any restore success log in both support bundle.


[SMALLFIX] Small fixes in several classes

Fixed link in documentation in FileSystemWorker
Collapsed multiple catch statements into one in
FileWorkerMasterSyncExecutor
Simplified assert statements in IdUtilsTest
Removed not needed variable in IndexedSetTest
Used <> generic in InodeDirectory

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11209/
Test PASSed.

LGTM

LGTM.
thanks for improving the code health.


archive ingest: winslow/staging - unauthorized when talking to api-stage

Failed to ingest bag, status codes was 401 response body was {"@context":"https://api.wellcomecollection.org/storage/v1/context.json","errorType":"http","httpStatus":"401","label":"Unauthorized","description":"Unauthorized","type":"Error"}

I removed the scope from the token request, Matthias will check if this helps


New Feature: Array.uniq

Closes #372
Hi there, thanks for the great library!
Saw an opportunity for an easy contribution and thought I'd pitch in.
I tried to hazard a guess to the @since tag, but let me know if you'd like me to change it (or anything else).
I was going to use toArray from Set, but that function orders the elements which I personally find counter-intuitive.

Edit:
Seems using assert.deepStrictEqual(uniq([NaN, NaN], setoidNumber), [NaN]) fails on Travis despite succeeding locally. I've changed to check for array length instead.

@gcanti Made the changes. Not sure what the stance is here on using new for side-effects, but can assign it to a const if you'd like.

Took tests for Array.member from Set.member. Let me know if you’d like me to expand on them, or have more feedback.

Thanks @alex-ketch

Has this been released yet? Maybe the name could be changed to the full one unique?

@raveclassic not yet, FWIW both lodash and rambda use uniq though

Some people are used to 'distinct'


[Feature]Collect DB Keywords Links

Is your feature request related to a problem? Please describe.
We Don't know Information about DB Keyword Now
Describe the solution you'd like
Find A Links about it and Comment in this Issue
Describe alternatives you've considered
like this

mysql8.x

Mysql

5.6
5.7
8.0

Oracle

8
8
8
7
10.1
8.1.5
8.1.6
9.2
7.3
10.2
20
19
18
12.2
-12.1
11.2

Maria

Maria

H2

H2

SQLITE

SQLite

Microsoft SQL Server

2016
2017
2019

#Cassandra
3.X

Dynamo

Dynamo

Elasticsearch

6.8
7.X

Hive

Hive

Microsoft Access

Access

DB2
-[5.3]
-[5.3]
-[5.3]
-[5.3]


Missing border line when there are no blocks, a few CSS fixes

Minor CSS issues for when there are no blocks on a page.

Missing border below message/block area
The placeholder area should be the height of a collapsed block so that when a block is added the page doesn't jump.
There should be a 20px space below the block area and page meta description accordion.

Refer to designs for dimensions https://projects.invisionapp.com/share/N5MVKR392DS#/318369132_Basics
More space is required at the bottom of the ElementalArea so that there's a larger drop area for a block  at the bottom of the list. Currently you can't drop if the curser is beyond the bottom of the last block
Can't drop (no blue line) looks like this:

Excuse the screenshots, it's hard to capture.

I've increased the minimum height of elements by a couple of pixels to 8rem and also applied that to the empty list element so they'll be the same size. It's easier to do that probably than to reduce the existing height of the elements. Is that OK? If not I can look at it again so it matches the designs more closely.

PR at #546

For reference they're currently about 10px larger vertically than in the designs. If we adjust that we'll need to check with the banner block to make sure that still looks right, that's the one that will probably need more tweaking than the content block which mostly looks like the bottom padding which needs to be adjusted

All of the collapsed content blocks should have the same height, even the banner block. If the banner block is different we can look at that as a separate issue. They should be 98px including the borders according to the designs

Related/duplicate: #484
Another reason for adding space below the block area is to assist with drag and drop. @clarkepaul if you're happy with 20px at the bottom being enough for this as well, we'll close #484

#546 fixes most of these points, but we should leave this open because we need to reduce the height of the elements to match the designs. I'll update the issue title to reflect this.


[REQUEST] Market OreDict option

Add an option to sell OreDict items
For example:
"ingotCopper": {
"stackObj": "thermalfoundation:material", (Thermal Exp Copper ingot)
"amount": 128,
"value": 10,
"isBuy": false,
"oreDict": true (Will accept all oredicted ingots, work only for "isBuy": false)
}

Actually any other wayt o implement this will do.

I see what you're saying here, but it might be difficult to explain to players. I might use "stackObj" as the name for both ore dictionary ("oreBlock") and regular items ("minecraft:") and the mod can just look for any ore dictionaries matching. I don't see a problem with this method as there will be no ore dicts with "modid:item" so it shouldn't interfere. Having it only work on selling makes sense as well. Thanks for this idea!


Playlist: adding a local thumbnail makes playlist crash

Open local edit page for some playlist
Update thumbnail there
Opening playlist crashes page

Might be related to the recent emoji changes, hmm. What do you mean by local? Localhost?


Audio not coming out jack 3.5mm on RPi4

Describe the problem
Using a RaspberryPi4 and setting the env variable AUDIO_OUTPUT to either RPI_AUTO or RPI_HEADPHONES. Does not make the audio coming out of the RPi4's 3.5mm jack.
Using the latest commit e3098287cb9b0b866e528251b424e388e3fbbb6a
To Reproduce
Steps to reproduce the behavior:

Set the device as a STANDALONE and set the env variable AUDIO_OUTPUT to either of the two options above.
Try to reproduce music
Check result

Expected behavior
I would expect the audio to work since I've tested it with RaspberryPi OS and the jack works so it's not a hardware issue.
Additional info

Audio service logs
audio  --- Audio ---
audio  Starting audio service with settings:
audio  - pulseaudio 15.0
audio  - Pulse log level: NOTICE
audio  - Default output: RPI_HEADPHONES
audio  - Default volume: 75%
audio
audio  Detected audio cards:
audio  0 vc4hdmi0 vc4-hdmi-vc4-hdmi-0
audio  1 vc4hdmi1 vc4-hdmi-vc4-hdmi-1
audio
audio
audio  WARNING: BCM2835 audio card not found, are you sure you are running on a Raspberry Pi?
audio  Setting audio routing rules...
audio  Routing 'balena-sound.input' to 'balena-sound.output'.
audio  Routing 'balena-sound.output' to '0'.
audio  W: [pulseaudio] main.c: This program is not intended to be run as root (unless --system is specified).
audio  E: [pulseaudio] module-alsa-card.c: Failed to find a working profile.
audio  E: [pulseaudio] module.c: Failed to load module "module-alsa-card" (argument: "device_id="0" name="platform-fef00700.hdmi" card_name="alsa_card.platform-fef00700.hdmi" namereg_fail=false tsched=yes fixed_latency_range=no ignore_dB=no deferred_volume=yes use_ucm=yes avoid_resampling=yes card_properties="module-udev-detect.discovered=1""): initialization failed.
audio  E: [pulseaudio] module-alsa-card.c: Failed to find a working profile.
audio  E: [pulseaudio] module.c: Failed to load module "module-alsa-card" (argument: "device_id="1" name="platform-fef05700.hdmi" card_name="alsa_card.platform-fef05700.hdmi" namereg_fail=false tsched=yes fixed_latency_range=no ignore_dB=no deferred_volume=yes use_ucm=yes avoid_resampling=yes card_properties="module-udev-detect.discovered=1""): initialization failed.
audio  W: [pulseaudio] server-lookup.c: Unable to contact D-Bus: org.freedesktop.DBus.Error.NotSupported: Unable to autolaunch a dbus-daemon without a $DISPLAY for X11
audio  W: [pulseaudio] main.c: Unable to contact D-Bus: org.freedesktop.DBus.Error.NotSupported: Unable to autolaunch a dbus-daemon without a $DISPLAY for X11
audio  W: [pulseaudio] module-loopback.c: Too many underruns, increasing latency to 205.00 ms

Device information

Raspberry Pi 4 64bit
Raspberry Pi 4 (using 64bit OS)
balenaOS 2.107.10+rev3

I'm experiencing the same issue - everything seems to work, except the audio. Other than the above, I note the following in the log info shown in the Balena console.
28.12.22 19:11:21 (+1300)  audio  E: [pulseaudio] module-alsa-card.c: Failed to find a working profile.
28.12.22 19:11:21 (+1300)  audio  E: [pulseaudio] module.c: Failed to load module "module-alsa-card" (argument: "device_id="0" name="platform-fef00700.hdmi" card_name="alsa_card.platform-fef00700.hdmi" namereg_fail=false tsched=yes fixed_latency_range=no ignore_dB=no deferred_volume=yes use_ucm=yes avoid_resampling=yes card_properties="module-udev-detect.discovered=1""): initialization failed.

Update, I have cleared the "vc4-kms-v3d" value from the "Define DT overlays" option in the Device Configuration list. After a reboot, and restarting the airplay service for some reason, the audio from the 3.5mm jack is working.

@AnnoyedPlatypus just got the opportunity to test this but nope, still not working!


Ban Appeal - Yaporeo

i was banned for simply saying  "sex"
it was an insta-ban
i didn't know i couldn't say sex. i thought swearing was okay, because i saw someone say the F word. my friend on the other hand [joshiewoshiepoo] said the n word so he would be banned too. hes black, dont worry.
i hope i can be let back onto the sever because i love the map and the community. i understand if i am still banned. thank you.

I was considering the unban in the first half of your appeal, but considering the second half is a poor joke, you won't be unbanned.


UTC hours auto-incrementing on date change

When switching a date while using utc="preserve", any subsequent date selection will increment the hour by 1, even if you select the exact same date. Reproducable in StackBlitz and locally with the same code. This shouldn't increment at all when the confirm button is clicked.
Happens with and without range prop as well.
<template>
  <div>
    <Datepicker v-model="date" placeholder="blah" utc="preserve" />
  </div>
</template>

<script setup>
import { ref } from 'vue';
import Datepicker from '@vuepic/vue-datepicker';
import<EMAIL_ADDRESS>
const date = ref(new Date()); // also happens with all versions of Date()
</script>

Further, this seems to only happen when confirming. In the internal-model-change emitter the times are correct.

PR with fix: https://github.com/Vuepic/vue-datepicker/pull/184

Will close when the version is published 😄

Will close when the version is published 😄

My bad! Thanks for the review and merge!

Fix is available under 3.6.0-alpha.1


Typos change in contributing.md

Typos changes
forusm -> forums
quality -> the quality
open source -> open-source
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

“forusm” wasn’t doing it for you? :)

@MGatner why don't you make the PR of "Better debug routes #2568" and submit it


Fixed RJSF input tooltip disappearing on hover

Notes for Reviewers
This PR fixes #
https://user-images.githubusercontent.com/51183483/222763572-3bc7e588-300a-4367-81d5-8a272f84ea7e.mov
Signed commits

[x] Yes, I signed my commits.

Task linked: CU-863g6cyka Ensure configuration tooltips don't disappear while in focus


WIP

Summary of the changes / Why this improves CrateDB
Fixes:
cr> show create table t;
+-----------------------------------------------------+
| SHOW CREATE TABLE doc.t                             |
+-----------------------------------------------------+
| CREATE TABLE IF NOT EXISTS "doc"."t" (              |
|    "a" ARRAY(INTEGER)                               |
| )                                                   |
| CLUSTERED INTO 4 SHARDS                             |

cr> select * from t;
+--------+
| a      |
+--------+
| [null] |
| NULL   |
| [1]    |
| []     |
+--------+

cr> select * from t where a != [];
+-----+
| a   |
+-----+
| [1] | -- missing [null]
+-----+

cr> show create table t5;
+-----------------------------------------------------+
| SHOW CREATE TABLE doc.t5                            |
+-----------------------------------------------------+
| CREATE TABLE IF NOT EXISTS "doc"."t5" (             |
|    "a" ARRAY(INTEGER) STORAGE WITH (                |
|       columnstore = false                           |
|    )                                                |
| )                                                   |
| CLUSTERED INTO 4 SHARDS                             |

cr> select * from t5;
+--------+
| a      |
+--------+
| []     |
| NULL   |
| [1]    |
| [null] |
+--------+

cr> select * from t5 where a != [1];
+--------+
| a      |
+--------+
| []     |
| NULL   | -- unfiltered
| [null] |
+--------+

Checklist

[ ] Added an entry in the latest docs/appendices/release-notes/<x.y.0>.rst for user facing changes
[ ] Updated documentation & sql_features table for user facing changes
[ ] Touched code is covered by tests
[ ] CLA is signed
[ ] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in the latest docs/appendices/release-notes/<x.y.0>.rst
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

The performance impact of the fix:
# Results (server side duration in ms)
V1: 5.9.0-d6cc2a6ac021cfcd2a0460e6325e7eda31abb4bf
V2: 5.9.0-39876d7270893b6cc1fe9ee820dd6ee4989ccf0e

// without doc-values

Q: select count(*) from t_arrays_biased_empty where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       33.620 ±  116.033 |     22.729 |     23.859 |     24.952 |   2304.439 |
|   V2    |      180.239 ±  226.668 |    155.052 |    159.129 |    162.733 |   3573.573 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 137.12%                           + 147.85%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 137.12%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |      389.643 ±  312.787 |    347.844 |    356.771 |    367.616 |   4260.982 |
|   V2    |      586.921 ±  639.280 |    515.331 |    524.726 |    533.100 |   8040.258 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               +  40.40%                           +  38.11%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 40.40%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        1.445 ±    2.217 |      0.878 |      1.241 |      1.382 |     37.635 |
|   V2    |      142.905 ±    4.687 |    138.615 |    141.455 |    143.867 |    182.915 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 196.00%                           + 196.52%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 196.00%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        1.052 ±    1.297 |      0.573 |      0.761 |      0.830 |     12.115 |
|   V2    |      274.850 ±    9.905 |    264.713 |    270.930 |    278.053 |    347.478 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 198.47%                           + 198.88%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 198.47%
The test has statistical significance

// with doc-values

Q: select count(*) from t_arrays_biased_empty_2 where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        3.925 ±   27.227 |      1.575 |      2.516 |      2.639 |    608.619 |
|   V2    |      387.925 ±  182.108 |    357.766 |    366.800 |    373.558 |   2393.549 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 195.99%                           + 197.27%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 195.99%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements_2 where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       15.350 ±  163.171 |      5.353 |      8.620 |      8.856 |   3656.509 |
|   V2    |       70.407 ±  248.260 |     50.109 |     51.425 |     52.483 |   4046.039 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 128.40%                           + 142.57%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 128.40%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty_2 where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |      114.144 ±   73.317 |    101.786 |    105.049 |    108.884 |    987.184 |
|   V2    |      183.272 ±    6.710 |    176.456 |    180.837 |    185.450 |    214.075 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               +  46.49%                           +  53.02%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 46.49%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements_2 where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       48.799 ±  159.281 |     35.002 |     36.880 |     44.718 |   3584.004 |
|   V2    |       26.248 ±    2.330 |     24.629 |     25.381 |     26.433 |     40.712 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               -  60.10%                           -  36.94%
There is a 99.84% probability that the observed difference is not random, and the best estimate of that difference is 60.10%
The test has statistical significance

I have two approaches and below are their perf measurements:
Approach1:
# Results (server side duration in ms)
V1: 5.9.0-d6cc2a6ac021cfcd2a0460e6325e7eda31abb4bf
V2: 5.9.0-859142fafd057b09cc61f09027549c2257347d2a

Q: select count(*) from t_arrays_biased_empty where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       13.434 ±   95.979 |      4.293 |      6.710 |      7.191 |   1932.328 |
|   V2    |      249.330 ±  275.741 |    221.754 |    226.644 |    230.290 |   5702.057 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 179.55%                           + 188.50%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 179.55%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |      247.619 ±  335.247 |    206.722 |    212.908 |    222.905 |   4137.374 |
|   V2    |     1059.174 ±  645.585 |    972.165 |    992.189 |   1007.619 |   8392.924 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 124.21%                           + 129.33%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 124.21%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        1.114 ±    3.588 |      0.607 |      0.865 |      0.978 |     66.742 |
|   V2    |      207.322 ±    7.741 |    201.521 |    205.061 |    208.058 |    300.646 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 197.86%                           + 198.32%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 197.86%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        0.794 ±    0.198 |      0.582 |      0.767 |      0.851 |      4.176 |
|   V2    |      517.162 ±   14.905 |    501.747 |    511.269 |    523.892 |    628.503 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 199.39%                           + 199.40%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 199.39%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty_2 where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        4.245 ±   28.307 |      1.521 |      2.378 |      2.506 |    616.207 |
|   V2    |      188.579 ±  161.278 |    158.054 |    169.356 |    180.555 |   2014.583 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 191.19%                           + 194.46%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 191.19%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements_2 where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       16.195 ±  176.333 |      5.135 |      8.982 |      9.193 |   3950.995 |
|   V2    |       69.942 ±  253.817 |     45.130 |     48.043 |     56.061 |   4094.966 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 124.79%                           + 137.00%
There is a 99.99% probability that the observed difference is not random, and the best estimate of that difference is 124.79%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty_2 where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       87.991 ±   80.607 |     74.927 |     77.488 |     80.961 |   1068.084 |
|   V2    |       87.135 ±   11.168 |     76.380 |     83.166 |     92.073 |    134.550 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               -   0.98%                           +   7.07%
There is a 18.60% probability that the observed difference is not random, and the best estimate of that difference is 0.98%
The test has no statistical significance

Q: select count(*) from t_arrays_biased_5_elements_2 where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       38.341 ±  175.285 |     25.855 |     27.067 |     30.419 |   3931.494 |
|   V2    |       27.287 ±    5.267 |     21.693 |     25.780 |     33.508 |     42.161 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               -  33.69%                           -   4.87%
There is a 84.10% probability that the observed difference is not random, and the best estimate of that difference is 33.69%
The test has no statistical significance

Approach 2:
# Results (server side duration in ms)
V1: 5.9.0-d6cc2a6ac021cfcd2a0460e6325e7eda31abb4bf
V2: 5.9.0-2bab39117d09db9f3019913cbd879ebe9f73584a

Q: select count(*) from t_arrays_biased_empty where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       38.313 ±  210.645 |     21.158 |     22.822 |     24.236 |   4343.798 |
|   V2    |      174.004 ±  260.946 |    151.017 |    154.447 |    156.433 |   5512.171 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 127.82%                           + 148.50%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 127.82%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |      256.795 ±  308.878 |    212.163 |    225.241 |    235.129 |   3794.734 |
|   V2    |      679.418 ±  634.339 |    599.955 |    616.950 |    629.550 |   7888.278 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               +  90.28%                           +  93.02%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 90.28%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        1.444 ±    1.805 |      0.946 |      1.269 |      1.422 |     39.462 |
|   V2    |      148.090 ±   92.623 |    136.121 |    138.918 |    140.652 |   1199.875 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 196.14%                           + 196.38%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 196.14%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        0.942 ±    0.234 |      0.641 |      0.887 |      0.960 |      3.130 |
|   V2    |      354.786 ±  334.388 |    309.660 |    319.657 |    331.844 |   4249.585 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 198.94%                           + 198.89%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 198.94%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty_2 where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        4.004 ±   24.538 |      1.646 |      2.796 |      2.923 |    549.435 |
|   V2    |      273.175 ±  139.509 |    249.392 |    256.698 |    262.980 |   1810.408 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 194.22%                           + 195.69%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 194.22%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements_2 where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       17.641 ±  204.902 |      5.239 |      9.040 |      9.239 |   4589.995 |
|   V2    |       71.988 ±  249.109 |     50.276 |     52.238 |     54.591 |   4074.390 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 121.27%                           + 140.99%
There is a 99.98% probability that the observed difference is not random, and the best estimate of that difference is 121.27%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty_2 where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       76.746 ±   73.682 |     61.316 |     65.927 |     74.907 |    947.853 |
|   V2    |      129.058 ±    7.665 |    122.624 |    126.253 |    129.935 |    167.362 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               +  50.84%                           +  62.78%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 50.84%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements_2 where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       27.837 ±  202.686 |     15.749 |     16.840 |     17.277 |   4532.838 |
|   V2    |       25.397 ±    2.583 |     24.205 |     24.662 |     25.062 |     40.813 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               -   9.17%                           +  37.70%
There is a 21.22% probability that the observed difference is not random, and the best estimate of that difference is 9.17%
The test has no statistical significance

Looks like we cannot choose one over the other because each approach performs better at different queries, will look into it further.

Perf measurement for the fix:
# Results (server side duration in ms)
V1: 5.9.0-1d994dcc418ed978d9b17ee2cc9b1842c8fd54db
V2: 5.9.0-a1dde366139463d4d250752f7a846a4b9ae38f0d

Q: select count(*) from t_arrays_biased_empty where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       48.808 ±  233.309 |     21.410 |     22.754 |     24.266 |   2949.321 |
|   V2    |      223.029 ±  444.034 |    180.128 |    183.865 |    187.267 |   7579.623 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 128.18%                           + 155.95%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 128.18%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |      524.761 ±  388.940 |    463.848 |    476.268 |    486.280 |   3962.422 |
|   V2    |      870.051 ±  788.972 |    753.444 |    772.927 |    787.405 |   8095.180 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               +  49.51%                           +  47.50%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 49.51%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        1.707 ±    2.670 |      1.091 |      1.440 |      1.635 |     36.870 |
|   V2    |      183.062 ±  127.838 |    162.883 |    166.920 |    169.586 |   1286.753 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 196.30%                           + 196.58%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 196.30%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        1.148 ±    0.252 |      0.895 |      1.110 |      1.204 |      4.347 |
|   V2    |      450.000 ±  389.190 |    389.282 |    401.117 |    412.761 |   3870.868 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 198.98%                           + 198.90%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 198.98%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty_2 where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |        6.442 ±   55.166 |      1.923 |      2.790 |      2.912 |    956.744 |
|   V2    |      255.662 ±  183.588 |    222.433 |    229.446 |    236.636 |   1871.761 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 190.17%                           + 195.19%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 190.17%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements_2 where xs != []
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       21.112 ±  219.995 |      4.971 |      8.816 |      9.013 |   3818.698 |
|   V2    |       72.517 ±  297.858 |     40.386 |     41.857 |     45.587 |   3750.963 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               + 109.81%                           + 130.41%
There is a 98.35% probability that the observed difference is not random, and the best estimate of that difference is 109.81%
The test has statistical significance

Q: select count(*) from t_arrays_biased_empty_2 where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |      143.816 ±   94.161 |    124.651 |    129.390 |    137.933 |   1005.982 |
|   V2    |      116.640 ±    9.739 |    109.445 |    112.912 |    117.268 |    175.906 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               -  20.87%                           -  13.60%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 20.87%
The test has statistical significance

Q: select count(*) from t_arrays_biased_5_elements_2 where xs != [1]
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |       29.305 ±  220.733 |     12.681 |     13.303 |     15.766 |   3822.364 |
|   V2    |       20.773 ±    2.326 |     19.038 |     19.697 |     20.342 |     27.398 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               -  34.07%                           +  38.75%
There is a 49.65% probability that the observed difference is not random, and the best estimate of that difference is 34.07%
The test has no statistical significance


Remove uses of CreateScratchAsync from test fixtures

I missed these cases in the previous change.

:shipit:


Transaction Alerts

Is there a way to have the transactions alerts happen real time (within a few minutes at least)? A player was added and it didn't alert in the discord channel until 10 hours later.

The alerts should happen almost instantly. Interesting. Have you checked the fronted dyno to make sure it is off? If that is on, the bot will sleep. Please make sure it is off. I haven't heard of transactions taking that long. I def need more info to work on if you have it?

Awesome, thanks for the help! I missed the step where I need to turn back
off the web dyno. Just did that now.
On Sun, Aug 29, 2021 at 9:47 PM Landon Patmore @.***>
wrote:

The alerts should happen almost instantly. Interesting. Have you checked
the fronted dyno to make sure it is off? If that is on, the bot will sleep.
Please make sure it is off. I haven't heard of transactions taking that
long. I def need more info to work on if you have it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LandonPatmore/yahoo-fantasy-bot/issues/94#issuecomment-908025786,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVLUXIMISVJ4IBJXBDN62R3T7MEN5ANCNFSM5DAOFL2A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

No problem, if the issue persists, reopen this. Glad to hep!


Separate UsbEventWatcher construction from start of enumerating devices

Library version
10.0.1
OS & OS version
Ubuntu 18.04 LTS
Describe the bug
This is kind of a bug or more like a race condition.
Since we have to new up UsbEventWatcher before we can subscribe to UsbDeviceAdded event. It is possible that some device will be missed in the event subscriber because UsbEventWatcher constructor already starts enumerating devices in a background thread.
Suggestion
Could you please separate construction of UsbEventWatcher from start of enumerating devices.
Constructor could just not call the Start method and you could simply make the Start method public and let user call it after they finished subscribing to all the desired events.
An even better option would be to separate EnumerateDevices from StartMonitoring into separate public methods.
And make EnumerateDevices populate the UsbDevicesList instead of firing UsbDeviceAdded event. This way it is more natural to use the library.
You can new up UsbEventWatcher, then EnumerateDevices to get a list of all currently known devices, then StartMonitor to monitor for changes to devices current and new.

Thank you for noticing this.
I agree that constructor shouldn't call Start() before you have the chance to subscribe to events.
Unfortunately removing Start() from constructor is a breaking change for all who use my library - if they don't call Start() it will just stop working.
I can put another Action callback parameter in constructor, to let you subscribe to events in constructor.
Or perhaps I will add bool startWatcher = true to constructor and it will work for all other users, and you can set it to false.
About separating EnumerateDevices from StartMonitoring - I agree that your suggestions would make the library better :)
I would also have to do that for Windows and for MacOS - the library has to be consistent - but unfortunately I don't have the time to do that now.

Thank you, i understand. Let's go withbool startWatcher = true for now as a constructor arg. This will be a pretty simple change

Ok, thanks :) I published a new NuGet .


fix(audioTrackRender):  reset audioTrack positon to zero avoid track …

…position overflow

Codecov Report

Merging #197 into develop will decrease coverage by 0.08%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #197      +/-   ##
===========================================
- Coverage    70.46%   70.38%   -0.09%
===========================================
  Files          203      203
  Lines        10964    10964
===========================================
- Hits          7726     7717       -9
- Misses        3238     3247       +9

Impacted Files
Coverage Δ

mediaPlayer/system_refer_clock.cpp
91.66% <0.00%> (-4.17%)
:arrow_down:

mediaPlayer/media_packet_queue.cpp
76.00% <0.00%> (-4.00%)
:arrow_down:

mediaPlayer/buffer_controller.cpp
56.77% <0.00%> (-2.55%)
:arrow_down:

framework/demuxer/play_list/SegmentTracker.cpp
83.06% <0.00%> (-1.10%)
:arrow_down:

mediaPlayer/player_msg_control.cpp
81.45% <0.00%> (-0.81%)
:arrow_down:

framework/demuxer/play_list/HLSStream.cpp
67.86% <0.00%> (-0.67%)
:arrow_down:

mediaPlayer/player_notifier.cpp
82.88% <0.00%> (-0.54%)
:arrow_down:

framework/codec/ActiveDecoder.cpp
87.81% <0.00%> (-0.51%)
:arrow_down:

mediaPlayer/SuperMediaPlayer.cpp
77.26% <0.00%> (-0.19%)
:arrow_down:

framework/demuxer/avFormatDemuxer.cpp
80.12% <0.00%> (+0.31%)
:arrow_up:

... and 3 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 8c5628f...820abcd. Read the comment docs.


Setting tile as ‘impassable’ requires 4 mouse clicks

One common scenario for editing tileset data is setting the passability to ‘impassable’. Most tiles don’t require directional passability: if it’s a rock or a bush, I can’t enter it from any side. So, in RM2k[3], there are two modes: for setting normal passability and for setting directional passability.
But R48 forces me to click on all the directions just to set the rock impassable.
So, R48 workflow to set a few tiles as impassable is:

Hover the mouse over the tile to change,
Click on it,
Hover the mouse over the flags to the right,
Click on 1:down,
Click on 2:left,
Click on 3:right,
Click on 4:up.

The RM2k[3] behaviour is:

(Few mouse movements are neccessary because the mouse doesn’t travel away from the tile list)
Click on the tile.

It would be nice if this was simplified in R48. Possible solutions I can think of:

Add some action like right/middle-clicking on the tile that would toggle the 1+2+3+4 flags of that tile (and won’t require moving the mouse from the tile list).
Add a button to set/unset all 1:down, 2:left, 3:right, 4:up with one click — this would reduce the number of clicks to just 2 (1 to set the tile, 1 to change the passability — still more actions then RM2k’s 1 click, though).
Maybe something else?

...GitHub deciding it should close issues before the solution is actually double-checked again sigh


Move content_panels inside BookPage definition.

Other than the above quibble, looks good to me!

That's true, and exactly what I did in my code!

Merged in ce57281f6af3ce6024b95211972c4da4fd8749b0 + parents. Thanks!


Update Ukraine holidays

Proposed change

There is no holidays in Ukraine during the period of martial law: Law No. 2136-IX.
Also removed Soviet-era holidays (before 1991).
Type of change

[ ] New country holidays support (thank you!)
[x] Supported country holidays update (calendar discrepancy fix, localization)
[ ] Existing code/test/process improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency upgrade (version update)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts and has clean commit history
[x] The code style looks good (make pre-commit)
[x] I've added tests to verify that the new code works and all tests pass locally (make test, make tox)
[ ] The related documentation has been added/updated (check off the box for free if no updates is required)

Coverage: 100.0%. Remained the same when pulling 25f70a67591dad44352426aeb4d6bd945be89070 on KJhellico:upd-ukraine into d34e0b3eee6720ca1be64066bec7159c5624e482 on dr-prodigy:beta.


Dashboard Modified for stage

Modified Dashboard, Added logout

Changes Unknown when pulling b0697b4442488dfb35fadfe0464b5027633e0725 on kittdotech:master into ** on frictionlessdata:master**.


Parallelize e2e tests using matrix strategy

Part of AN-462, "Investigate how to improve the CI reliability/speed"
This PR splits the longest step yarn run test:e2e into 3 parallel matrix jobs, one for each of the packages that have e2e tests. I re-ran the Action 3 times (1, 2, 3) and compared the total time to the previous 10 Action runs on master [minutes]:

That is a genuine potential downside, but if someone is adding e2e tests for a new package I find it reasonable to expect they'll check the CI to confirm their added tests work. In that case they'll see the e2e matrix job doesn't include their new tests and should add the new package to the build matrix array.
I suppose there could be another test that looks in packages for test:e2e and confirms that all packages that have it are accounted for in the build matrix, but that sounds like a fair amount of work for this hypothetical.

It is hard to spot sometimes (https://github.com/api3dao/airnode/pull/749/commits/f36c1cc3c16de01548c7d0cc4f418f897e15485a#diff-0fab668cedab8990ae29c18bfe48e39224faa6071f154cb0d3ba8c28e8643961R28) but I guess someone would notice this at some point...
Feel free to merge.