Quick Help: Sharp resize function is not working related with the width and height

Hello Sharp community,
I have a bug with a Sharp resize function.
The original image is:

The resized image is:

I used Sharp resize function like below:
sharp(img.buffer)
      .resize({ width: 2500, height: 3333 })
      .withMetadata()
      .toBuffer();

But the result image's width is 3333 and height is 2500. The image and height were exchanged when resize the image.
Please help me quickly.

The code and image provided here works as-expected for me, but GitHub can remove/alter EXIF metadata when uploading images.
Any kind of mismatched dimensions/rotation are usually the result of EXIF orientation, so please check the input image. You can auto-orient based on EXIF data by adding rotate() to your pipeline.
https://sharp.pixelplumbing.com/api-operation#rotate

Thank you lovell,
That works for me. I was impressed by using this module.


Transaction hangs in receiving for 2+ days

Description
After the weekend I noticed that one of the transactions is hanging in "receiving" status. The app has not been opened during the entire weekend. Looks like this transaction must have been failed in 720 blocks (approx. 12 hours). Logs attached.
logs.zip
How to reproduce
Pre-conditions
Wallet A (desktop app, sender) is installed and running.
Wallet A has some UTXO to send.
Wallet B (mobile app, receiver) is installed and running.
Steps to reproduce

Send some UTXO from Wallet A to Wallet B.
Act fast now: When transaction in Wallet B appears in "receiving" status - cancel the transaction in Wallet A while it's still in "waiting for receiver".
Check the transaction status in Wallet B in some time.

Actual: Hangs in "receiving" forever.
Expected: Appears in "expired" status in 720 blocks.

wallet.db.zip
password: 1

@anatolse @DenisDemyanko what a status of this investigation please?

fixed on master

please test on masternet build.

checked


[Docs][Connector-V2][Slack]Refactor connector-v2 docs using unified format Slack

Purpose of this pull request
Related issue: #4572
Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Hi, please use mvn spotless:apply reformat the doc.

Thanks for your review, done.


zh_CN.utf8 cannot show

hi admin
i am a new atom user, i use atom as a node/less editor.
My system $LANG=zh_CN.UTF-8
i have set the file encoding and atom perferences encoding to utf8.
when i write an html test file(include chinese charactors), the atom pane show a square, but in the browser is right.
I guess maybe atom lack fonts to show chinese charactors.
thank you for your help!

@playaround88 This problem has been around for a long time but has not been resolved


[Feature] Expose response headers, specifically response status, in the API action

Users would like more control of how to handle their API responses on the basis of their response status codes. We could expose this information in a separate field inside the action object.

The quick way to do this would be to add a third argument to the function.
onError = function (responseBody, params, responseMeta): void

If we add this in the first argument, that would be a breaking change.
The better experience would probably be to be an object structure from which you can deconstruct the items you need

I think we can achieve both elements from what you have suggested above if we keep the response body object as is and use the new paramter as the encapsulated object for all other information. May be something we can follow across the board.

+1 for this feature request. Have an api that only returns authentication session information in response headers. Looking to use this specifically https://demo.microstrategy.com/MicroStrategyLibrary/api-docs/index.html#/Authentication/postLogin


Improvement: Added Embedding Section to README.md

Fixes issue #606

[x] I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested
Updated README
Additional context & links
More context is described in issue #606

Thanks for the contribution! Examples like this are available in the OpenAI API reference documentation, e.g., here: https://platform.openai.com/docs/api-reference/embeddings/create
If we were to add content like this to the README for every method, the README would grow too long, unfortunately. We're exploring other alternatives for this.
Accordingly, I'll go ahead and close this. Thanks again!


Add cares glue for libuv event loop

I don't know how much code coverage this will get in this repo's tests, but I got this building on a branch of grpc-node and all of the tests pass. Those tests mainly go through the "localhost" codepath but I had to fix problems in the grpc_ares_ev_driver_libuv.cc file to get them to work so it is exercising that codepath.

clang-format is required.

I'll let @apolcyn do the review on this.

@apolcyn Can you take another look? I addressed all of the review comments.

I have made all of the requested changes.

How does loopback6.unittest.io.grpc resolve to anything? It's not a real DNS name.

How does loopback6.unittest.io.grpc resolve to anything? It's not a real DNS name.

Whoops, I meant loopback6.unittest.grpc.io
This resolves to [::1]
There are also a few more names like this used in the dualstack_sockets_test.

OK, I can add that when I integrate these changes into the node repo. Otherwise I think this is complete.

The ObjC failure is #18542. The Windows failure is #18637. The Bazel RBE ASAN failure is #18939.


Logging information for indiviual cost functions

In GitLab by @cmastalli on Feb 15, 2019, 15:19
I found that it's only possible to get the cost data for the set of cost functions using in an OC problem. However it's very convenient, for debugging and analysing, to get access of each individual derivatives and cost value.
We should figure out a pattern that allows us to do this. Please see a similar topic here: #107.

In GitLab by @nmansard on Feb 15, 2019, 15:45
You already can: define a cost model, create a data by giving the pinocchio data you like (for example the one of the corresponding DAM), and it will work.

In GitLab by @nmansard on Feb 18, 2019, 11:18
I will close this issue as I believe the solution works. Re-open it if need be.

In GitLab by @nmansard on Feb 18, 2019, 11:18
closed


replaceAddressesAndBigNumberify does not allow controllable order

https://github.com/statechannels/statechannels/blob/68eb445a35e9be52a06b65c19427505c84163f9e/packages/nitro-protocol/test/test-helpers.ts#L219
If I pass in {B: 10, A: 10} it might return:

[{destination: B, amount: 10}, {destination: A, amount: 10}]
[{destination: A, amount: 10}, {destination: B, amount: 10}]

I should be able to control the order of the destinations.

Actually, this just modifies an outcome shorthand. It's on the user to properly construct the outcome from the shorthand.


VS Code: Retry as Sudo doesn't work since 1.70

Describe the bug
It seems that since VS Code 1.70, #49643 and #76526 are back.
This time, the error message when clicking "Retry as Sudo..." is
Failed to save 'filename': User did not grant permission.

Steps To Reproduce
Steps to reproduce the behavior:

Open VS Code;
Try to save a root-owned file;
Click "Retry as Sudo...";
See error.

Expected behavior
A prompt appears to ask your password.
Notify maintainers
@eadwu @Synthetica9 @maxeaubrey @bobby285271
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.18.16-zen1, NixOS, 22.11 (Raccoon), 22.11.20220807.4bdf416`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.10.3`
 - channels(root): `"nixos"`
 - channels(user): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@SuperSamus I'm not able to reproduce the error message: are you running a polkit authentication agent?
With VS Code version 1.69.2, without polkit authentication agent installed, I had a different error message:

To fix this, I installed packages polkit and polkit_gnome.
Since I use i3, I also needed to run the polkit authentication agent , for example running the desktop file /run/current-system/sw/etc/xdg/autostart/polkit-gnome-authentication-agent-1.desktop with dex (Gnome runs it automatically but i3 does not).
With polkit authentication agent running, after "Retry with Sudo..." you will have a dialog asking the password for sudo.
With this setup working, I then updated VS Code to the unstable channel, v. 1.70.0, and it's working, no error messages.
[emiliano@galactica:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.59, NixOS, 22.05 (Quokka), 22.05.2322.92fe622fdfe`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.1`
 - channels(root): `"nixos-22.05, nixos-unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Trying again, even I can't reproduce it anymore.
Must have been a fluke or something... Sorry for the noise.
Closing. (Using Plasma, by the way)

Searching for "User did not grant permission" brings this: microsoft/vscode#80516.
So I was able to reproduce the error message  entering, three times, a wrong password in the polkit authentication dialog, then in VS Code I got the "user did not grant permission" message:

Retry as sudo doesn't work for me on vscode, using nixos-unstable as system channel


Settings Page: Add Contributors

After issue #190 is completed using GitHub contributors API add all the contributors below the admins.
with their GitHub ids.
if they want their LinkedIn to be added then wish to drop them to here.

My LinkedIn profile :)

Here's mine.

@anushbhatia Has of issue #190 is completed, I would like to work upon this under Gssoc'21,
Please assign it to me.


feat(aws-lambda-sagemaker): New aws-lambda-sagemaker pattern implementation

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: codebuildgithubautobuildPro-1r1mBrMNRii0
Commit ID: 9e15fcdd3786e259c19e017a220858950bf74817
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: codebuildgithubautobuildPro-1r1mBrMNRii0
Commit ID: bf0ab5d61f40a63760c9e27df826c4f42a8e0dd3
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: codebuildgithubautobuildPro-1r1mBrMNRii0
Commit ID: 82204fdc30506e0e5967e56cc60a93512ba2c149
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


examples

Thanks, this seems to fix quite a few mistakes... any source for which fields are optional in the server.dat format? I can't seem to find anything about it on the minecraft wiki...

no, i haven't found a source either, but i think the name and ip will always be there. i have seen situations where the other two fields are not

From my own experience you are correct, sad that there isn't a good source...
But thanks for the pr :)


Fix broken build when +beval is disabled and +textprop is enabled

I found that build was broken when FEAT_BEVAL is not defined but FEAT_TEXTPROP is defined (for popup).
This is because popup implementation is depending on beval.c to use find_word_under_cursor() but proto.h and beval.h don't consider that. I updated them to consider FEAT_TEXTPROP with this patch.

Thanks.  It also works with just a slightly different change to proto.


Reelsteady Joiner Error

Hi guys,
Fresh install on my Win10 x64 machine.
I can't make reelsteady joiner to work. All the files are good as it work on my laptop but not on my desktop one...
Any idea what wrong ?
Here is the error :
NonError: { error: 'No stream maps found' }
    at module.exports (I:\Program Files\ReelSteady Joiner\resources\app.asar\node_modules\ensure-error\index.js:14:10)
    at handleError (I:\Program Files\ReelSteady Joiner\resources\app.asar\node_modules\electron-unhandled\index.js:50:10)
    at process.<anonymous> (I:\Program Files\ReelSteady Joiner\resources\app.asar\node_modules\electron-unhandled\index.js:130:4)
    at process.emit (node:events:527:28)
    at emit (node:internal/process/promises:140:20)
    at processPromiseRejections (node:internal/process/promises:269:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:97:32)

reelsteady-joiner 1.3.1
Electron 20.0.2
win32 10.0.19045
Locale: fr
Thanks for your help

Do you mean that on your laptop ReelSteady Joiner does not give you any errors and on another computer it does? Anyway, without the files giving you the problem I can't offer you any more help.
The error indicates that the files do not have stream maps available, so they lack the necessary information to carry out the entire process.

Could you share a screenshot of the execution of this command?
"I:\Program Files\ReelSteady Joiner\node_modules\ffmpeg-static\ffmpeg.exe" -i <mp4_path>
A screenshot of each of the chapters would be helpful, as well as from a computer where it works and others where it doesn't, at the same time I insist again that if it is possible to share the files that cause the problem, it would be quite helpful to debug the problem

The execution doesn't work as the path is uncorrect.
I tried with ;
I:\Program Files\ReelSteady Joiner*resources\app.asar.unpacked*\node_modules\ffmpeg-static\ffmpeg.exe" -i <mp4_path>
I get a blue cmd page for half a second blank and it disapear, same on both machine.
I can send you the files, but I get the same messages with any files I want to join, so not shure if it will help as its not files related but something wrong with my window.
I installed the last ffmpeg, but didn't help

Correct, the route that I had indicated is wrong although the one you provided should work, expected output:

Could it be due to some ffmpeg execution problem on your machine?
Something like that coud help (replace to fit your system):
ffmpeg -i GX010150.MP4 && pause
The cmd should freeze after executing the ffmpeg command

Thanks for the fast reply.
ffmpeg problem I don't know. I update the system variable (path : and add
ffmpeg/bin in the C root with last updated ffmpeg files) but doesn't make
any progress.
Still the same error message from joiner even with a fresh new install.
I am ulploading the files and wil share it to you once its done (4go)
When trying your command with the second part of the video :
Le mer. 27 sept. 2023 à 13:45, Rubén @.***> a écrit :

Correct, the route that I had indicated is wrong although the one you
provided should work, expected output:
[image: image]
https://user-images.githubusercontent.com/29512781/271110119-a228dd5a-ab60-4707-a227-b10f75f7535e.png
Could it be due to some ffmpeg execution problem on your machine?
Something like that coud help (replace to fit your system):
ffmpeg -i GX010150.MP4 && pause
The cmd should freeze after executing the ffmpeg command
—
Reply to this email directly, view it on GitHub
https://github.com/rubegartor/ReelSteady-Joiner/issues/64#issuecomment-1738053760,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BC2G534Z5DCS2JZZPBZRIBDX4SF7VANCNFSM6AAAAAA5G4EMKQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

And if you try: I:\Program Files\ReelSteady Joiner\resources\app.asar.unpacked\node_modules\ffmpeg-static\ffmpeg.exe" -i <mp4_path>? Instead of your own installation of ffmpeg.
Reelsteady Joiner doesn't use your ffpmeg instead it uses its own executable included in ReelSteady Joiner, so maybe is due to some error in ReelSteady Joiner ffmpeg.
I'll take a look at the files you attached anyway.

And if you try: I:\Program Files\ReelSteady Joiner\resources\app.asar.unpacked\node_modules\ffmpeg-static\ffmpeg.exe" -i <mp4_path>? Instead of your own installation of ffmpeg.
Reelsteady Joiner doesn't use your ffpmeg instead it uses its own executable included in ReelSteady Joiner, so maybe is due to some error in ReelSteady Joiner ffmpeg.
I'll take a look at the files you attached anyway.

This command line just pop up a blue blank window for half a second and that all and disapear forever.
It does the exact same on my Laptop where RSGO joiner works well.
I installed RSGO joiner on a usb stick. It work when plugged to my laptops but I still have the error message while running it with my desktop computer..

I have generated a debug version that stores some more information in the logs to be able to continue investigating the problem since I have tried to replicate your system and installation but I have not been able to replicate the problem.
If it's not a problem, install the next version and try again, if it still gives you problems, share the log with me again and I will review it for errors.
Debug version: https://we.tl/t-ciAE73QAdO

Heya,
Thanks for your help but unfortunately no improvement.
here is the log :
[2023-09-30 20:41:42.058] [info]  OS: win32 - 10.0.19045
[2023-09-30 20:41:42.060] [info]  version: 1.3.2-debug
[2023-09-30 20:41:42.060] [info]  isDev?: false
[2023-09-30 20:41:42.061] [info]  config: {
"savePath": "I:\test joiner",
"exportOption": 0,
"concurrentProjects": 5,
"fileModifyDates": 1,
"preservePCMAudio": 0
}
[2023-09-30 20:41:42.061] [info]  exePath: C:\Program Files\ReelSteady Joiner
[2023-09-30 20:41:42.061] [info]  macExePath: C:\Program Files\ReelSteady Joiner
[2023-09-30 20:41:42.062] [info]  ffmpegPath: C:\Program Files\ReelSteady Joiner\resources\app.asar.unpacked\node_modules\ffmpeg-static\ffmpeg.exe
[2023-09-30 20:41:42.062] [info]  gyroProcessPath: C:\Program Files\ReelSteady Joiner\app\bin\win\udtacopy.exe
[2023-09-30 20:41:42.070] [info]  exiftool: C:\Program Files\ReelSteady Joiner\app\bin\win\exiftool.exe
[2023-09-30 20:41:42.071] [info]  exiftool config: C:\Program Files\ReelSteady Joiner\app\bin\exiftool.config
[2023-09-30 20:41:42.072] [debug] concat.txt content:
file 'I:\D R O N E S\F P V\Tré la tete JOINER\GX013096.MP4'
file 'I:\D R O N E S\F P V\Tré la tete JOINER\GX023096.MP4'
[2023-09-30 20:41:42.072] [info]  Project: {
"_id": "34548952-a009-4228-9e14-10d43e5711bf",
"_type": "mp4",
"_projectPath": "I:\test joiner\GX013096",
"_dirPath": "I:\D R O N E S\F P V\Tré la tete JOINER",
"_files": [
"GX013096.MP4",
"GX023096.MP4"
],
"_log": {
"functions": {},
"hooks": [],
"isDev": false,
"levels": [
"error",
"warn",
"info",
"verbose",
"debug",
"silly"
],
"logId": "34548952-a009-4228-9e14-10d43e5711bf",
"variables": {
"processType": "browser"
},
"transports": {}
},
"_name": "GX013096",
"_outputName": "GX013096_joined.mp4",
"_duration": 381,
"_modifiedDate": "2023:04:18 18:28:32-07:00",
"_failed": false,
"_available": true,
"_completed": false
}
[2023-09-30 20:41:42.073] [info]  outputName: GX013096_joined.mp4
[2023-09-30 20:41:42.073] [info]  projectPath: I:\test joiner\GX013096
[2023-09-30 20:41:42.073] [info]  concatFilePath: C:\Users\Alexandre W10\AppData\Local\Temp\concat_34548952-a009-4228-9e14-10d43e5711bf.txt
[2023-09-30 20:41:42.073] [info]  outputFilePath: I:\test joiner\GX013096\GX013096_joined.mp4
[2023-09-30 20:41:42.074] [info]  File sizes:
[2023-09-30 20:41:42.074] [info]  I:\D R O N E S\F P V\Tré la tete JOINER\GX013096.MP4 size:<PHONE_NUMBER>B
[2023-09-30 20:41:42.074] [info]  I:\D R O N E S\F P V\Tré la tete JOINER\GX023096.MP4 size: 771231814B
[2023-09-30 20:41:42.106] [debug] Stream-maps:
[2023-09-30 20:41:42.107] [debug] error: Error: Command failed: "C:\Program Files\ReelSteady Joiner\resources\app.asar.unpacked\node_modules\ffmpeg-static\ffmpeg.exe" -f concat -safe 0 -i C:\Users\Alexandre W10\AppData\Local\Temp\concat_34548952-a009-4228-9e14-10d43e5711bf.txt -y
ffmpeg version 6.0-essentials_build-www.gyan.dev Copyright (c) 2000-2023 the FFmpeg developers
built with gcc 12.2.0 (Rev10, Built by MSYS2 project)
configuration: --enable-gpl --enable-version3 --enable-static --disable-w32threads --disable-autodetect --enable-fontconfig --enable-iconv --enable-gnutls --enable-libxml2 --enable-gmp --enable-bzlib --enable-lzma --enable-zlib --enable-libsrt --enable-libssh --enable-libzmq --enable-avisynth --enable-sdl2 --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxvid --enable-libaom --enable-libopenjpeg --enable-libvpx --enable-mediafoundation --enable-libass --enable-libfreetype --enable-libfribidi --enable-libvidstab --enable-libvmaf --enable-libzimg --enable-amf --enable-cuda-llvm --enable-cuvid --enable-ffnvcodec --enable-nvdec --enable-nvenc --enable-d3d11va --enable-dxva2 --enable-libvpl --enable-libgme --enable-libopenmpt --enable-libopencore-amrwb --enable-libmp3lame --enable-libtheora --enable-libvo-amrwbenc --enable-libgsm --enable-libopencore-amrnb --enable-libopus --enable-libspeex --enable-libvorbis --enable-librubberband
libavutil      58.  2.100 / 58.  2.100
libavcodec     60.  3.100 / 60.  3.100
libavformat    60.  3.100 / 60.  3.100
libavdevice    60.  1.100 / 60.  1.100
libavfilter     9.  3.100 /  9.  3.100
libswscale      7.  1.100 /  7.  1.100
libswresample   4. 10.100 /  4. 10.100
libpostproc    57.  1.100 / 57.  1.100
C:\Users\Alexandre: No such file or directory
[2023-09-30 20:41:42.107] [debug] stdout:
[2023-09-30 20:41:42.107] [debug] stdout: ffmpeg version 6.0-essentials_build-www.gyan.dev Copyright (c) 2000-2023 the FFmpeg developers
built with gcc 12.2.0 (Rev10, Built by MSYS2 project)
configuration: --enable-gpl --enable-version3 --enable-static --disable-w32threads --disable-autodetect --enable-fontconfig --enable-iconv --enable-gnutls --enable-libxml2 --enable-gmp --enable-bzlib --enable-lzma --enable-zlib --enable-libsrt --enable-libssh --enable-libzmq --enable-avisynth --enable-sdl2 --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxvid --enable-libaom --enable-libopenjpeg --enable-libvpx --enable-mediafoundation --enable-libass --enable-libfreetype --enable-libfribidi --enable-libvidstab --enable-libvmaf --enable-libzimg --enable-amf --enable-cuda-llvm --enable-cuvid --enable-ffnvcodec --enable-nvdec --enable-nvenc --enable-d3d11va --enable-dxva2 --enable-libvpl --enable-libgme --enable-libopenmpt --enable-libopencore-amrwb --enable-libmp3lame --enable-libtheora --enable-libvo-amrwbenc --enable-libgsm --enable-libopencore-amrnb --enable-libopus --enable-libspeex --enable-libvorbis --enable-librubberband
libavutil      58.  2.100 / 58.  2.100
libavcodec     60.  3.100 / 60.  3.100
libavformat    60.  3.100 / 60.  3.100
libavdevice    60.  1.100 / 60.  1.100
libavfilter     9.  3.100 /  9.  3.100
libswscale      7.  1.100 /  7.  1.100
libswresample   4. 10.100 /  4. 10.100
libpostproc    57.  1.100 / 57.  1.100
C:\Users\Alexandre: No such file or directory
[2023-09-30 20:41:42.108] [debug] Extracted maps:

Great, now with these new logs I have managed to locate the error, I will fix it for the next version. Thanks for helping!

Great, now with these new logs I have managed to locate the error, I will fix it for the next version. Thanks for helping!

Great, thanks for your help.
Any idea when it will be released ? possibility to send me an beta updated version with the issue fixed ?

Probably over the next week, I'm doing the last tests to verify that everything is working correctly.

Fixed in version 1.3.2 now available


Add tiered storage support for Pulsar SQL

Motivation
Add support for Pulsar SQL to be able to query offloaded ledgers

rerun java8 tests
rerun integration tests

rerun java8 tests

rerun java8 tests

rerun cpp tests

rerun integration tests


RedirectDb: Show existing url owner if other user

This can be helpful in negotiating with someone to update or transfer ownership of a link.

Coverage increased (+0.05%) to 97.794% when pulling 40f81740090669e66a79b4fabb6c3da7931d38c1 on show-existing-owner into 9f9906ad58d2ec117e6df717c09c5ad0624be7b2 on master.


⚠ Bofrost.at has degraded performance

In 0b2e5f6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2094 ms

Resolved: Bofrost.at performance has improved in ff6a96a.


ng:repeat-ed radio buttons are broken

when radio buttons are nested in ng:repeat, they are broken because angular renames name="foo" to name="@foo". In ng:repeat scopeid is different for each button, which messes up the grouping of radio buttons by name.

    <input type="radio" name="choice" value="{{item}}"
           id="item_{{$index}}">

      {{item}}

choice: {{choice}}

this is still broken in 0.10.3: https://gist.github.com/1293065
Proposed solution: instead of using current scope id in name attribute, use the scope id to which the ng:model is attached to.


implementing on vision tranformers

hey @lucidrains , newbie here and wanted to implement rotary embeddings on ViT for a research project. I'm not able to figure out how I can modify queries and keys if they're not in the correct tensor shape. Could you explain what the problem might be? Thanks in advance!
Code for ViT + rotary embeddings:
import torch
from torch import nn

from einops import rearrange, repeat
from einops.layers.torch import Rearrange
from rotary_embedding_torch import apply_rotary_emb, RotaryEmbedding, broadcat
# helpers

def pair(t):
    return t if isinstance(t, tuple) else (t, t)

# classes

class FeedForward(nn.Module):
    def __init__(self, dim, hidden_dim, dropout = 0.):
        super().__init__()
        self.net = nn.Sequential(
            nn.LayerNorm(dim),
            nn.Linear(dim, hidden_dim),
            nn.GELU(),
            nn.Dropout(dropout),
            nn.Linear(hidden_dim, dim),
            nn.Dropout(dropout)
        )

    def forward(self, x):
        return self.net(x)

class Attention(nn.Module):
    def __init__(self, dim, heads = 8, dim_head = 64, dropout = 0.):
        super().__init__()
        inner_dim = dim_head *  heads
        project_out = not (heads == 1 and dim_head == dim)

        self.heads = heads
        self.scale = dim_head ** -0.5

        self.norm = nn.LayerNorm(dim)

        self.attend = nn.Softmax(dim = -1)
        self.dropout = nn.Dropout(dropout)

        self.to_qkv = nn.Linear(dim, inner_dim * 3, bias = False)

        self.to_out = nn.Sequential(
            nn.Linear(inner_dim, dim),
            nn.Dropout(dropout)
        ) if project_out else nn.Identity()

    def forward(self, x):
        x = self.norm(x)

        qkv = self.to_qkv(x).chunk(3, dim = -1)
        q, k, v = map(lambda t: rearrange(t, 'b n (h d) -> b h n d', h = self.heads), qkv)

        pos_emb = RotaryEmbedding(
            dim = 32,
            freqs_for = 'pixel',
            max_freq = 256
        )

        freqs_h = pos_emb(torch.linspace(-1, 1, steps = 256), cache_key = 256)
        freqs_w = pos_emb(torch.linspace(-1, 1, steps = 256), cache_key = 256)

        # concat the frequencies along each axial
        # broadcat function makes this easy without a bunch of expands

        freqs = broadcat((freqs_h[:, None, :], freqs_w[None, :, :]), dim = -1)

        # rotate in frequencies

        q = apply_rotary_emb(freqs, q)
        k = apply_rotary_emb(freqs, k)

        dots = torch.matmul(q, k.transpose(-1, -2)) * self.scale

        attn = self.attend(dots)
        attn = self.dropout(attn)

        out = torch.matmul(attn, v)
        out = rearrange(out, 'b h n d -> b n (h d)')
        return self.to_out(out)

class Transformer(nn.Module):
    def __init__(self, dim, depth, heads, dim_head, mlp_dim, dropout = 0.):
        super().__init__()
        self.norm = nn.LayerNorm(dim)
        self.layers = nn.ModuleList([])
        for _ in range(depth):
            self.layers.append(nn.ModuleList([
                Attention(dim, heads = heads, dim_head = dim_head, dropout = dropout),
                FeedForward(dim, mlp_dim, dropout = dropout)
            ]))

    def forward(self, x):
        for attn, ff in self.layers:
            x = attn(x) + x
            x = ff(x) + x

        return self.norm(x)

class ViT(nn.Module):
    def __init__(self, *, image_size, patch_size, num_classes, dim, depth, heads, mlp_dim, pool = 'cls', channels = 3, dim_head = 64, dropout = 0., emb_dropout = 0.):
        super().__init__()
        image_height, image_width = pair(image_size)
        patch_height, patch_width = pair(patch_size)

        assert image_height % patch_height == 0 and image_width % patch_width == 0, 'Image dimensions must be divisible by the patch size.'

        num_patches = (image_height // patch_height) * (image_width // patch_width)
        patch_dim = channels * patch_height * patch_width
        assert pool in {'cls', 'mean'}, 'pool type must be either cls (cls token) or mean (mean pooling)'

        self.to_patch_embedding = nn.Sequential(
            Rearrange('b c (h p1) (w p2) -> b (h w) (p1 p2 c)', p1 = patch_height, p2 = patch_width),
            nn.LayerNorm(patch_dim),
            nn.Linear(patch_dim, dim),
            nn.LayerNorm(dim),
        )

        self.pos_embedding = nn.Parameter(torch.randn(1, num_patches + 1, dim))

        self.cls_token = nn.Parameter(torch.randn(1, 1, dim))
        self.dropout = nn.Dropout(emb_dropout)

        self.transformer = Transformer(dim, depth, heads, dim_head, mlp_dim, dropout)

        self.pool = pool
        self.to_latent = nn.Identity()

        self.mlp_head = nn.Linear(dim, num_classes)

    def forward(self, img):
        x = self.to_patch_embedding(img)
        b, n, _ = x.shape

        cls_tokens = repeat(self.cls_token, '1 1 d -> b 1 d', b = b)
        x = torch.cat((cls_tokens, x), dim=1)
        x += self.pos_embedding[:, :(n + 1)]
        x = self.dropout(x)

        x = self.transformer(x)

        x = x.mean(dim = 1) if self.pool == 'mean' else x[:, 0]

        x = self.to_latent(x)
        return self.mlp_head(x)

Error:
RuntimeError: The size of tensor a (16) must match the size of tensor b (65) at non-singleton dimension 1

@mukvnd i have an example here

Thank you very much!


ERROR: UndefVarError: y not defined

I am trying to run a mixed-effects model in julia (R is too slow for my data), but I keep getting this error.
I have installed DataArrays, DataFrames , MixedModels, and RDatasets packages and am following the tutorial here --> http://dmbates.github.io/MixedModels.jl/latest/man/fitting/#Fitting-linear-mixed-effects-models-1
These are my steps:
using DataArrays, DataFrames , MixedModels, RDatasets

I get these warnings

WARNING: Method definition ==(Base.Nullable{S}, Base.Nullable{T}) in
module Base at nullable.jl:238 overwritten in module NullableArrays at
/home/home/.julia/v0.6/NullableArrays/src/operators.jl:128. WARNING:
Method definition model_response(DataFrames.ModelFrame) in module
DataFrames at
/home/home/.julia/v0.6/DataFrames/src/statsmodels/formula.jl:352
overwritten in module MixedModels at
/home/home/.julia/v0.6/MixedModels/src/pls.jl:65. WARNING: Method
definition ==(Base.Nullable{S}, Base.Nullable{T}) in module Base at
nullable.jl:238 overwritten in module NullableArrays at
/home/home/.julia/v0.6/NullableArrays/src/operators.jl:128. WARNING:
Method definition ==(Base.Nullable{S}, Base.Nullable{T}) in module
Base at nullable.jl:238 overwritten in module NullableArrays at
/home/home/.julia/v0.6/NullableArrays/src/operators.jl:128. WARNING:
Method definition model_response(DataFrames.ModelFrame) in module
DataFrames at
/home/home/.julia/v0.6/DataFrames/src/statsmodels/formula.jl:352
overwritten in module MixedModels at
/home/home/.julia/v0.6/MixedModels/src/pls.jl:65. WARNING: Method
definition model_response(DataFrames.ModelFrame) in module DataFrames
at /home/home/.julia/v0.6/DataFrames/src/statsmodels/formula.jl:352
overwritten in module MixedModels at
/home/home/.julia/v0.6/MixedModels/src/pls.jl:65.

I get an R dataset from lme4 package (used in the tutorial)
 inst = dataset("lme4", "InstEval")

 julia> head(inst)
 6×7 DataFrames.DataFrame
│ Row │ S   │ D      │ Studage │ Lectage │ Service │ Dept │ Y │
│ 1   │ "1" │ "1002" │ "2"     │ "2"     │ "0"     │ "2"  │ 5 │
│ 2   │ "1" │ "1050" │ "2"     │ "1"     │ "1"     │ "6"  │ 2 │
│ 3   │ "1" │ "1582" │ "2"     │ "2"     │ "0"     │ "2"  │ 5 │
│ 4   │ "1" │ "2050" │ "2"     │ "2"     │ "1"     │ "3"  │ 3 │
│ 5   │ "2" │ "115"  │ "2"     │ "1"     │ "0"     │ "5"  │ 2 │
│ 6   │ "2" │ "756"  │ "2"     │ "1"     │ "0"     │ "5"  │ 4 │

I run the model as shown in the tutorial
 m2 = fit!(lmm(Y ~ 1 + Dept*Service + (1|S) + (1|D), inst))

and get
ERROR: UndefVarError: Y not defined
  Stacktrace:
  [1] macro expansion at ./REPL.jl:97 [inlined]
  [2] (::Base.REPL.##1#2{Base.REPL.REPLBackend})() at ./event.jl:73

The same thing happens when I try it with my own data loaded using "readtable" from DataFrames package
I am running julia 0.6.0 and all packages are freshly installed. My system is arch linux 4.11.7-1 with all the latest packages. Julia installs without a problem, but some packages give warnings (see above).
Also,  just installed julia on friend's mac via brew, and it gives the same warnings and error.

On 0.6, you'd have to use the write @formule Y ~ 1 + Dept*Service + (1|S) + (1|D). The example needs to be updated.

Like so? m2 = fit!(lmm(@formule(Y ~ 1 + Dept*Service + (1|S) + (1|D), inst)))
This still doesn't work. I am new to julia, can provide full syntax?

Almost …
m2 = fit!(lmm(@formula( Y ~ 1 + Dept*Service + (1|S) + (1|D)), inst))

Am 06.07.2017 um 19:05 schrieb hikea<EMAIL_ADDRESS>Like so? m2 = fit!(lmm(@formule(Y ~ 1 + Dept*Service + (1|S) + (1|D), inst)))
This still doesn't work. I am new to julia, can provide full syntax?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/dmbates/MixedModels.jl/issues/91#issuecomment-313458336, or mute the thread https://github.com/notifications/unsubscribe-auth/AA2VGMyhxRdM0RjLd426-8kJ3stAPX8Yks5sLRPzgaJpZM4OP6Ur.

Great! It worked. Thank you very much!

Thanks for answering @andreasnoack and @kliegl. I will fix the documentation when I get back home on Sunday.


Creep is rendered behind the glass surface it should be over

Creep is rendered behind the glass surface it should be over:

This affects any impact:

It's possible that this is the same underlying problem as https://github.com/DaemonEngine/Daemon/issues/906. Either one might be caused by the mark polys being generated insufficiently far in front of the surface.

This is supposed to be worked around by using polygonOffset, but maybe the factor and offset that we use are wrong.

Creep has the SS_DECAL sort order so it is drawn before almost all translucent surfaces. And it is treated as a translucent surface, so depth doesn't even matter since the dueling surfaces are both translucent.
So the renderer is working as designed here... but it's an old design based on heuristics that favor batching surfaces with a shared texture to improve speed at the cost of correctness.
With the material system, everything renders as we would like, yay! I think it might ignore fine-grained shader sorting distinctions?
Until very recently, the creep texture had only 0 or 1 alpha values. So we could have fixed it without changing the sort order by using depthWrite in the shader. Now it has alpha that fades out at the edges so we couldn't do that.

I think it might ignore fine-grained shader sorting distinctions?

Yeah, it essentially skips this sort when adding surfaces since it just does R_AddWorldSurfaces(), and then they either end up in the same material in the order they were received in or implicitly correctly sorted by material.

This might be caused by the same issue as #1079.

This might be caused by the same issue as #1079.

Sort of. Both are caused by a farther back surface being drawn after one in front. The difference is that in this case the renderer is "required" to draw the closer surface first, according to the shaderSort_t ordering. In #1079 the surfaces have the same sort number, so which one is drawn first is decided by some other arbitrary rules.

This might be caused by the same issue as #1079.

Sort of. Both are caused by a farther back surface being drawn after one in front. The difference is that in this case the renderer is "required" to draw the closer surface first, according to the shaderSort_t ordering. In #1079 the surfaces have the same sort number, so which one is drawn first is decided by some other arbitrary rules.

Perhaps this explicit sort should just be ignored? I guess it was used to be able to sort transparents by shader while generally avoiding issues like this by explicitly assigning some sort to the shader.

I've thought about adding a cvar for sorting the translucent surfaces by distance (assuming we actually have the information to do that?) and ignoring all the other sorting criteria. This would probably be off for lower presets since it makes the renderer slower by preventing merging.

Heh, that I had the same thought. I'm pretty sure that R_RecursiveWorldNode() inherently creates the list of BSP surfaces sorted by distance.


fix: default variation size

If we use an icon like this, without setting up global icon preferences
<AcademicCap variation="mini" />

you would expect size to be defaulted to the variation size, like viewBox, but it is not.
<svg xmlns="http://www.w3.org/2000/svg" role="img" width="24" height="24" fill="none" aria-label="academic cap" viewBox="0 0 20 20" stroke-width="1.5">
    ...
</svg>

This PR fixes the above issue, assuming this is not intended behaviour.
Issue Details

The issue lies in these size default assignments for the different variations
https://github.com/shinokada/svelte-heros-v2/blob/b1a805748a81954bc305e5996c0b8c908c593bd1/src/lib/AcademicCap.svelte#L24-L25
since size is never falsy as it is defaulted to '24' when declaring props, size is never asigned '20'
https://github.com/shinokada/svelte-heros-v2/blob/b1a805748a81954bc305e5996c0b8c908c593bd1/src/lib/AcademicCap.svelte#L7-L8

Proposal Details

This patch does a few things:

removes the size prop default assignments to '24'
https://github.com/shinokada/svelte-heros-v2/blob/7a60c02bfe63ee7132e50461fe38ef65d33f643c/src/lib/AcademicCap.svelte#L7-L8

instead of using an $effect, derive viewBox and variationSize from the variation prop
https://github.com/shinokada/svelte-heros-v2/blob/7a60c02bfe63ee7132e50461fe38ef65d33f643c/src/lib/AcademicCap.svelte#L22-L31

default width and height to variationSize if size isn't set (either by the consumer component or global context)
https://github.com/shinokada/svelte-heros-v2/blob/7a60c02bfe63ee7132e50461fe38ef65d33f643c/src/lib/AcademicCap.svelte#L38-L39

Thanks for the PR. v2.0.0-next.7 is released with this fix.


[Android Demo] question answering

Hi, @YUNQIUGUO looks like the script prepare_model.py can be shared between Android and IOS.


Various warnings compiling for Arch Release

In file included from /build/sway/src/sway-0.5/include/layout.h:5:0,
                 from /build/sway/src/sway-0.5/include/config.h:11,
                 from /build/sway/src/sway-0.5/sway/container.c:6:
/build/sway/src/sway-0.5/sway/container.c: In function 'new_output':
/build/sway/src/sway-0.5/sway/container.c:83:23: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
     sway_log(L_DEBUG, "restoring output %lu:%s", handle, op_name);
                       ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
/build/sway/src/sway-0.5/sway/container.c:89:20: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  sway_log(L_DEBUG, "New output %lu:%s", handle, name);
                    ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
/build/sway/src/sway-0.5/sway/container.c: In function 'new_view':
/build/sway/src/sway-0.5/sway/container.c:257:20: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  sway_log(L_DEBUG, "Adding new view %lu:%s to container %p %d",
                    ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
/build/sway/src/sway-0.5/sway/container.c: In function 'new_floating_view':
/build/sway/src/sway-0.5/sway/container.c:295:20: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  sway_log(L_DEBUG, "Adding new view %lu:%x:%s as a floating view",
                    ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
/build/sway/src/sway-0.5/sway/container.c: In function 'destroy_output':
/build/sway/src/sway-0.5/sway/container.c:357:20: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  sway_log(L_DEBUG, "OUTPUT: Destroying output '%lu'", output->handle);
                    ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
[ 76%] Building C object swaybar/CMakeFiles/swaybar.dir/ipc.c.o
[ 78%] Linking C executable ../bin/swaybar
[ 79%] Building C object sway/CMakeFiles/sway.dir/criteria.c.o
[ 79%] Built target swaybar
[ 81%] Building C object sway/CMakeFiles/sway.dir/debug_log.c.o
/build/sway/src/sway-0.5/sway/debug_log.c: In function 'container_log':
/build/sway/src/sway-0.5/sway/debug_log.c:29:17: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'wlc_handle {aka const unsigned int}' [-Wformat=]
  fprintf(stderr,"(h:%2ld)",c->handle);
                 ^
[ 82%] Building C object sway/CMakeFiles/sway.dir/extensions.c.o
[ 84%] Building C object sway/CMakeFiles/sway.dir/focus.c.o
/build/sway/src/sway-0.5/sway/focus.c: In function 'set_focused_container':
/build/sway/src/sway-0.5/sway/focus.c:108:24: warning: format '%ld' expects argument of type 'long int', but argument 5 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  swayc_log(L_DEBUG, c, "Setting focus to %p:%ld", c, c->handle);
                        ^
In file included from /build/sway/src/sway-0.5/include/layout.h:5:0,
                 from /build/sway/src/sway-0.5/include/container.h:7,
                 from /build/sway/src/sway-0.5/include/focus.h:11,
                 from /build/sway/src/sway-0.5/sway/focus.c:3:
/build/sway/src/sway-0.5/sway/focus.c: In function 'set_focused_container_for':
/build/sway/src/sway-0.5/sway/focus.c:199:20: warning: format '%ld' expects argument of type 'long int', but argument 4 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  sway_log(L_DEBUG, "Setting focus for %p:%ld to %p:%ld",
                    ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
/build/sway/src/sway-0.5/sway/focus.c:199:20: warning: format '%ld' expects argument of type 'long int', but argument 6 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  sway_log(L_DEBUG, "Setting focus for %p:%ld to %p:%ld",
                    ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
[ 85%] Building C object sway/CMakeFiles/sway.dir/handlers.c.o
[ 86%] Building C object sway/CMakeFiles/sway.dir/input.c.o
In file included from /build/sway/src/sway-0.5/include/layout.h:5:0,
                 from /build/sway/src/sway-0.5/include/container.h:7,
                 from /build/sway/src/sway-0.5/include/handlers.h:3,
                 from /build/sway/src/sway-0.5/sway/handlers.c:11:
/build/sway/src/sway-0.5/sway/handlers.c: In function 'handle_view_created':
/build/sway/src/sway-0.5/sway/handlers.c:203:20: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  sway_log(L_DEBUG, "handle:%ld type:%x state:%x parent:%ld "
                    ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
/build/sway/src/sway-0.5/sway/handlers.c:203:20: warning: format '%ld' expects argument of type 'long int', but argument 6 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  sway_log(L_DEBUG, "handle:%ld type:%x state:%x parent:%ld "
                    ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
/build/sway/src/sway-0.5/sway/handlers.c: In function 'handle_view_destroyed':
/build/sway/src/sway-0.5/sway/handlers.c:280:20: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  sway_log(L_DEBUG, "Destroying window %lu", handle);
                    ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
/build/sway/src/sway-0.5/sway/handlers.c: In function 'handle_view_geometry_request':
/build/sway/src/sway-0.5/sway/handlers.c:342:20: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
  sway_log(L_DEBUG, "geometry request for %ld %dx%d @ %d,%d", handle,
                    ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
/build/sway/src/sway-0.5/sway/handlers.c: In function 'handle_view_state_request':
/build/sway/src/sway-0.5/sway/handlers.c:369:22: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'wlc_handle {aka unsigned int}' [-Wformat=]
    sway_log(L_DEBUG, "setting view %ld %s, fullscreen %d", view, c->name, toggle);
                      ^
/build/sway/src/sway-0.5/include/log.h:32:23: note: in definition of macro 'sway_log'
  _sway_log(VERBOSITY, FMT, ##__VA_ARGS__)
                       ^
[ 88%] Building C object sway/CMakeFiles/sway.dir/input_state.c.o
[ 89%] Building C object sway/CMakeFiles/sway.dir/ipc-server.c.o
/build/sway/src/sway-0.5/sway/ipc-server.c: In function 'ipc_client_handle_readable':
/build/sway/src/sway-0.5/sway/ipc-server.c:208:32: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  if (read_available - received >= client->payload_length) {
                                ^

What flags are you using to produce these warnings? If the flags are defined in a custom makepkg.conf, maybe you can post that?

The usual arch flags, but I believe this is i686.


GOG Galaxy error message on first install

when trying to run the latest version of the installation script on a clean steamdeck oled, i'm getting this error

log file attached
NonSteamLaunchers-install.log

@ivan-aws yeah that's a new error from gog can't do much about it

ok, so probably then something that might resolve itself after a bit, right?

@ivan-aws hopefully, if the error continues. I'll try to fix it myself I'll give it some time, currently if you're using GOG with nonsteamlaunchers whether decky or desktop, I would make sure the compatability is set to Steams Proton 9.0.3 for now. This will get rid of most errors.

@ivan-aws ive fixed the installtion of it on both plugin and desktop.  but running the games and finishing the installs of each game may require a different proton


[Algorithm failure]  serious Memory leak on middle size point cloud, need help.

Technically, It should not consider as a bug,  more like an algorithm failure.
I compare Teaser++ with ICP kind registration like the T-RO.
ICP kind method can handle 50k points (nearest point correspondence ) with no problem.
However, Teaser++ was crushed with a huge memory leak.
I trace it to
registration.cc
   computeTIMs(const Eigen::Matrix<double, 3, Eigen::Dynamic>& v,
                                          Eigen::Matrix<int, 2, Eigen::Dynamic>* map)

and find
map->resize(2, N * (N - 1) / 2);
Eigen::Matrix<double, 3, Eigen::Dynamic> vtilde(3, N * (N - 1) / 2);
and in loop
// populate the index map
Eigen::Matrix<int, 2, Eigen::Dynamic> map_addition(2, N);
for (size_t j = 0; j < N; ++j) {
  map_addition(0, j) = i;
  map_addition(1, j) = j;
}

if N = 50k, vtilde,map will be 500000^2 / 2.
In the following loop, the memory reached its limit.
I know it does not really need 50k to do the registration.
But I want Teaser to compute the max_clique_; I think the PMC process is better than RANSAC, especially with many outliers.
To compute
max_clique_ = clique_solver.findMaxClique(inlier_graph_);

It needs inlier_graph_, and inlier_graph_  needs  src_tims_map_,right?
Any advice will be helpful.
Thanks.

I realize it needs a lot of memory to create the graph,  a lot of memory to store the edge information.
    // Calculate some important indices
    // For each measurement, we compute the TIMs between itself and all the measurements after it.
    // For example:
    // i=0: add N-1 TIMs
    // i=1: add N-2 TIMs
    // etc..
    // i=k: add N-1-k TIMs
    // And by arithmatic series, we can get the starting index of each segment be:
    // k*N - k*(k+1)/2

Hi @cdb0y511, thank you for your interest and feedback. By design, TEASER++ is unfortunately not very memory efficient. As you noticed, in order to build the graph we check for all possible combinations of the input correspondences.
I will instead suggest significantly reduce the size of the inputs. You should view inputs to TEASER++ as 3D point correspondences, not raw 3D points. You should use some feature extraction algorithms to first extract features, then use TEASER++ to solve the registration.
In the paper, we demonstrated the possibility of using TEASER++ without any feature matching: essentially we generate all possible correspondences, and input those correspondences to TEASER++ (and we compare it with ICP-based methods). However, due to the large memory requirement, such all-to-all correspondences are only possible for small point clouds (< 200 points on my machine).
Let me know if you have more questions.


B-21375 UB Allowance Calculation - INT

Agility ticket
Upstream Dependencies:
https://github.com/transcom/mymove/pull/13977
Downstream Dependencies:
https://github.com/transcom/mymove/pull/14102
Summary
We are adding the UB allowance to the entitlements for an order. The UB allowance is calculated based off a combination of branch, grade, orders type, dependents authorized, whether or not it's an accompanied tour, and in the case of civilian pay grades - the number of dependents and whether they are under 12, or 12 and over.
Work in this PR:

New table called ub_allowances
Took data from the excel sheet attached to B-21375, pulled out the unique rows, and inserted them into ub_allowances to allow lookups based on branch, grade, orders type, dependents authorized, whether or not it's an accompanied tour
Calculated the UB allowance in the cases of:

customer order creation
customer order amendment
SC creating orders on behalf of the customer
SC editing variables that would change the UB allowance (either in the Allowances section or the Orders section)
TOO editing variables that would change the UB allowance (either in the Allowances section or the Orders section)

The meat and potatoes of the logic is in pkg/models/entitlements.go, in the GetUBWeightAllowance() function.
How to test

Set FEATURE_FLAG_UNACCOMPANIED_BAGGAGE to true.
Please download a copy of the Uniformed Service Member Unaccompanied Baggage Lookup Table
Pick out a few random values and verify that they exist in the ub_allowances table and have the correct UB allowance value.
Closely read the AC on the story, especially the part about how to calculate civilian allowances. Please do the math for the civilian pay grade and the number of dependents and double check that your math matches mine.
Using the AC from Cameron's B-21374, go through every situation where the orders or allowances can be edited, and continue to recheck the orders.entitlement.ub_allowance value. Sometimes when it's edited, a new entitlement is added so you will have to retrace from the actual order down to the entitlement each time
Edit one field at a time - branch, order type, grade, dependents authorized, accompanied tour, etc. Try hard to break it - using combos from the spreadsheet, keep checking for accurate UB allowance values.
In the case of a combination that isn't valid to receive a UB entitlement, that's fine - just verify that the order.entitlement.ub_allowance is 0, as that's a valid value. You'll notice that the GetUBWeightAllowance() function doesn't return an error in most cases, as 0 is fine, it's just a situation where someone isn't entitled to a UB allowance.

Edge cases to pay attention to:

Grade of E9 special senior enlisted should get the same UB allowance as E9s
Space Force branch should get the same UB allowance as Air Force
Any orders type other than Temporary Duty will be considered PCS, in regards to the spreadsheet values (per the AC on the feature)
Local moves should get 0 for their UB allowance
Max UB allowance for a family is 2000 lbs

Not sure if this is correct. The ubAllowance says 0 but dependents of over and under age is set > 0 and accompanied is set to true and false in the other. I was editing the dependents as a SC and then when I toggled Accompanied that's when I noticed the ubAllowance set to 0 no matter what. the allowance was initially according to the excel sheet.

What's the combo of branch, grade, orders type, depedents authorized, and whether it's accompanied? I'm not 100% following, we can huddle if you want

Not sure if this is correct. The ubAllowance says 0 but dependents of over and under age is set > 0 and accompanied is set to true and false in the other. I was editing the dependents as a SC and then when I toggled Accompanied that's when I noticed the ubAllowance set to 0 no matter what. the allowance was initially according to the excel sheet.

What's the combo of branch, grade, orders type, depedents authorized, and whether it's accompanied? I'm not 100% following, we can huddle if you want

Disregard. I spent some time today testing this and didnt find any issues.

Not sure if this is correct. The ubAllowance says 0 but dependents of over and under age is set > 0 and accompanied is set to true and false in the other. I was editing the dependents as a SC and then when I toggled Accompanied that's when I noticed the ubAllowance set to 0 no matter what. the allowance was initially according to the excel sheet.

What's the combo of branch, grade, orders type, depedents authorized, and whether it's accompanied? I'm not 100% following, we can huddle if you want

Disregard. I spent some time today testing this and didnt find any issues.

Sounds good, let me know if you run into anything weird


translation of missing stuff just added for persian language.

Hi!
Many things were left without translating and we are going to use django-cms as our main cms from now on, so we had to translate django-cms completly.
I'll work more on django-cms and I'll help translation and bug-fix (and maybe adding features) in future.
I'm not sure if I was supposed to commit to the develop branch or not, actually it's my first contribute to any public github project. Correct me if I was wrong on anything!

@pouya-abbassi
Hello there,
Thank you for your contribution.
And congratulations on your first contribution (to a public repo) :)
We use transifex to manage all translations for djangoCMS.
You can read more here.
Once your translations have been approved on transifex, we'll take care of merging them in the core prior to releasing.

T

Coverage decreased (-4.0%) to 84.817% when pulling a6bc640ae2bbe776d5e227a0626bf728a27356fd on pouya-abbassi:develop into 8c9f2af2760cbe4c23ff2adb3e6585dd701c563d on divio:develop.

Should I do it again using transifex or is it fine the way it is?
Plus, I found some of these translations (that has been done before) are confusing due to miss-matching situations.
You know! some words have different meaning on different situations!!!
Should I re-translate them like what I did right now or should I do it in transifex???

ps:
I need more time and more playing around to find out where should be edited so I can't do it right now.

@pouya-abbassi
Yes, please use transifex.

Coverage decreased (-4.0%) to 84.817% when pulling a6bc640ae2bbe776d5e227a0626bf728a27356fd on pouya-abbassi:develop into 8c9f2af2760cbe4c23ff2adb3e6585dd701c563d on divio:develop.


Use battery status provided by electron

There is a problem with this approach as well as using systeminformation library to get the battery percent, both use the battery's maxcapacity and currentcapacity values to calculate the current load percent but these values are not always the same when the battery is at full charge.
This is because the battery will have some degradation with time and the current capacity will be lower than the max capacity, showing a misleading battery percent as you can see in the screenshot.

I'm working on a fix to this issue for systeminformation so maybe we can use it later.
Opinions, suggestions?

I would stick with the provided battery info from electron. It might just be that the batterymanager did not trigger the change event. @NickTikhonov what do you think?

Forget about my previos comment, the Battery Status API doesn't have that issue.

That's nice 👍 😄


Introduce Ident::parse_any for parsing identifiers with rust keywords

For your use case is it better to treat _ as a raw ident or not? Why?

My usecase is a GraphQL parser. The spec allows single _ as names.

Seems reasonable, I am on board with accepting _ here.
I would prefer to avoid introducing a RawIdent type because this is just an alternative way to parse an Ident. Maybe you could do what we have for Field where we expose call!(Field::parse_named) and call!(Field::parse_unnamed) for struct fields and tuple struct fields respectively. Here we would have syn!(Ident) for the default behavior and call!(Ident::parse_any) to accept any ident including keywords.

Seems more than reasonable! Updated the PR

I released 0.13.10 containing this function.


Create Managing-Employees-and-Reports->-Approval-Workflows.md

Details

Fixed Issues
$
PROPOSAL: https://github.com/Expensify/Expensify/issues/311424
Tests

[ ] Verify that no errors appear in the JS console

Offline tests

QA Steps

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

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR
[x] I ran the tests on all platforms & verified they passed on:

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

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

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

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

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Hi @zanyrenney, I think the article should replace the Approval-Workflows.md, but I already made that change in the PR :) I just noticed the link for the instructions in How to set an approval workflow is not set correctly and also the images are broken:

You can see the live changes of this PR here https://zanyrenney-patch-3.helpdot.pages.dev/articles/expensify-classic/manage-employees-and-report-approvals/Approval-Workflows#gsc.tab=0

Hey @marcochavezf thank you for making the change - the images are placeholders because I had to submit a separate PR for them (as per the instructions!)

Oh I see, thanks for the clarification! I will merge this PR then

Updated PR author checklist to pass the checks, since this only applies to App code changes

Hmm, something's up with the check. I will merge and remove Emergency label since it's only a markdown addition

Ah no emergency label, this is new

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

🚀 Deployed to staging by https://github.com/marcochavezf in version: 1.3.87-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/marcochavezf in version: 1.3.87-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.87-12 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/marcochavezf in version: 1.3.88-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/chiragsalian in version: 1.3.88-11 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Add metrics generation to track QoR

Description
OpenROAD supports metrics generation to track QoR.
Command: ./openroad -metric file.json
Metrics.tcl available here
Implement metrics generation for OpenLane flow as well.
Proposal
Sample metrics.json for floorplan stage:
{
	"run__flow__platform__time_units": "1ns",
	"run__flow__platform__capacitance_units": "1pF",
	"run__flow__platform__resistance_units": "1kohm",
	"run__flow__platform__voltage_units": "1v",
	"run__flow__platform__current_units": "1mA",
	"run__flow__platform__power_units": "1nW",
	"run__flow__platform__distance_units": "1um",
	"floorplan__timing__setup__tns": 0.0000,
	"floorplan__timing__setup__ws": 1.5600,
	"floorplan__power__internal__total": 0.0005,
	"floorplan__power__switching__total": 0.0001,
	"floorplan__power__leakage__total": 0.0000,
	"floorplan__power__total": 0.0006,
	"floorplan__design__io": 54,
	"floorplan__design__instance__count": 270,
	"floorplan__design__instance__area": 2284.6912,
	"floorplan__design__instance__count__stdcell": 270,
	"floorplan__design__instance__area__stdcell": 2284.6912,
	"floorplan__design__instance__count__macros": 0,
	"floorplan__design__instance__area__macros": 0.0000,
	"floorplan__design__instance__utilization": 0.0341,
	"floorplan__design__instance__utilization__stdcell": 0.0341,
	"floorplan__design__io__pre_restruct": 54,
	"floorplan__design__instance__count__pre_restruct": 270,
	"floorplan__design__instance__area__pre_restruct": 2284.6912,
	"floorplan__design__instance__count__stdcell__pre_restruct": 270,
	"floorplan__design__instance__area__stdcell__pre_restruct": 2284.6912,
	"floorplan__design__instance__count__macros__pre_restruct": 0,
	"floorplan__design__instance__area__macros__pre_restruct": 0.0000,
	"floorplan__design__instance__utilization__pre_restruct": 0.0341,
	"floorplan__design__instance__utilization__stdcell__pre_restruct": 0.0341,
	"floorplan__design__io__post_restruct": 54,
	"floorplan__design__instance__count__post_restruct": 266,
	"floorplan__design__instance__area__post_restruct": 1993.1616,
	"floorplan__design__instance__count__stdcell__post_restruct": 266,
	"floorplan__design__instance__area__stdcell__post_restruct": 1993.1616,
	"floorplan__design__instance__count__macros__post_restruct": 0,
	"floorplan__design__instance__area__macros__post_restruct": 0.0000,
	"floorplan__design__instance__utilization__post_restruct": 0.0297,
	"floorplan__design__instance__utilization__stdcell__post_restruct": 0.0297
}

Working on it.

@donn any updates ? It will be great to sync up on METRICS support as in OR.

Nothing I can share publicly, no.

Now that OpenLane 2 is public- we do that there. We add -metrics metrics.json to every OpenROAD invocation and extract the results.
This issue, like many others, will remain open until OpenLane 2 is in this repository.


I would like to submit a Chinese translation for this project. What should I do?

How do I create the lang file (if any)?

Hi, this was the least prioritized task. I kept thinking about taking the translation out of the program into separate xml file packages, but never got around to it. So far everything is embedded in the application Resources.resx.
I'm working on a big update right now, and I think within a couple weeks I'll get to that task as well. I would be very grateful for the translation. You can start translating now, and then just drop me the file later (what a mess...). And as soon as I get to it and rework everything, I'll add your translation to the ADE and list you in the credits.
Thank you 🙏

@DigitalE4rth I have finished the translation and placed the file here
Resources.zip

Thank you 🥰🙏
You've helped a lot


Decide what to do about undocumented Witness::push panics

These panics are caused by us storing indices as u32 rather than usize and we forgot to document it.
Possible solutions:

store usize indices
keep panics and document them, also make them more explicit with track_caller etc.
make push fallible
require PushBytes instead of [u8]

See https://github.com/rust-bitcoin/rust-bitcoin/pull/3259#issuecomment-2329605961

Storing usize indices just pushes the problem further out -- if you go out of range you'll then have Witnesses that nodes will not parse. (Of course, if you have 4GB+ witnesses you'll likely have many other problems so maybe this is an academic concern..)
I would suggest making push fallible, and providing another method to push objects that are less than MAX_SIZE large. (I'm not sure if PushBytes is the right type? I feel that Script should be, and we should also have a MAX_SIZE limit on Script...though that would undermine our ability to convert to/from Vecs...)

Is the API complexity somehow worth it here?

Yes, because this API is less-used and the required logic is way simpler (you literally just check the length on each push and reject the push if it's over). It's the same complexity as with PushBytes except arguably less disruptive.
With Transaction you have multiple fields per input and per output and can't even compute the total length without reserializing. You'd basically need an opaque type, a Transaction::try_update method which took a closure that was passed a copy of the transaction to modify, and after the closure finished the copy was reserialized and accepted/rejected.

Maybe the type should be renamed to Max32bLenBytes (or a better name) and used for both.

Yeah, we would want a better name (and the actual limit, if we are taking compact-size decodability seriously is 32 megs). TBH we could just keep the name PushBytes and reduce its size limit. It looks like script::Builder::push already takes an AsRef<PushBytes> so there'd be no change there; we'd just start using the same bound for Witness::push.
As an aside, in the docs for these functions we should mention that the docs for PushBytes have a whole list of AsRef<PushBytes> implementors. (I didn't know this until just now when I checked.)

That means all push operations [on script::Builder] become fallible...

Yeah, I see. For scripts you're right. With Witness we are only checking the length of individua pushes, meaning that we can have infallible push_signature, push_pushbytes, etc., methods. But if users had to unwrap every .push_opcode on Script that would be extremely annoying (and useless, because as you say, everyone would just unwrap).

The issue with Witness is the same as txins - when pushing you may still exceed the total limit on transaction

This is true, but there's still value in checking that the Witness itself is in limit. (Specifically, the value is that we don't need to think about how to handle out-of-range compact sizes within the Witness type itself.)

If the script is required be within some size then it makes sense to restrict PushBytes to that size instead of 2^32B. Re-adds branches on 32-bit architectures but it's fine since we never promised there won't be any.

Yep -- scripts are always serialized with a compact-size prefix, so they are subject to the 32M limit.

There's still the possibility that one pushes too many items into the Witness. I'd say use PushBytes as a best-effort protection and give up on other things.

Agreed. I am a little tempted to add a cfg flag that tries to panic when it detects that the total size of Witnesses or Scripts exceeds the limit, which may help some people during debugging. But for normal development I think we would need dependent types to do this "ergonomically".

Specifically, the value is that we don't need to think about how to handle out-of-range compact sizes within the Witness type itself.

We could just use usize but surely it'd waste a bit of space if such witnesses are invalid to begin with. So I'd argue the real value of requiring PushBytes is a memory/cache optimization. :)

I am a little tempted to add a cfg flag that tries to panic when it detects that the total size of Witnesses or Scripts exceeds the limit

Why not just debug_assert? I think this is quite in-line with with Rust does with integers.

we would need dependent types to do this "ergonomically".

I suspect even that wouldn't really help that much.
Anyway, I plan for Witness to have a bunch of standard constructors so unless you're doing custom scripts you don't have to deal with any of this. (And even then I want appending the control block & co to be easier.)

(and the actual limit, if we are taking compact-size decodability seriously is 32 megs)

When you say 32 megs, are you referring to the  uint64_t MAX_SIZE = 0x02000000 or some other limit?

When you say 32 megs, are you referring to the  uint64_t MAX_SIZE = 0x02000000 or some other limit?

Yes, that is what I'm referring to.

Why not just debug_assert? I think this is quite in-line with with Rust does with integers.

I worry that this could still turn an "object that can't be validly serialized' bug into a DoS bug. But yeah, I wouldn't be opposed to doing this.

Well, debug_assert is configurable via a canonical cfg and most people don't use debug asserts in production.

Hmm, ok, yeah, I'm warming up to the idea to just using debug_assert.


Question: Minimum API Level Android

Just curious what the minimum API level supported is? via: https://github.com/Azure/azure-mobile-apps-net-client/issues/227

I don't know for certain.  I would have to check and test it.  But I would expect that it's compatible a long way back.  Probably as far back as Xamarin.Android itself can go.  Like I said, this needs to be verified.
However, it's not clear in the azure-mobile-apps-net-client issue what source of compatibility is in play.  I assume their library may have API level restrictions that SQLitePCL.raw does not.  For example, if they don't support API level 14, it is probably not because of SQLitePCL.raw.

Thanks @ericsink, yeah they could be using something odd under the hood I guess. From reading that they support 19+.
I was reading in your Readme about Android and SQLite failures with < 19 devices do to the version that it is on, so maybe it has to do with that.

Ah, I see.
Compatibility is to some extent determined by which SQLite functions you are using and which instance of the SQLite library you are using.  So for example, if you are using a SQLite feature that was added in SQLite 3.7.14, then, well, you get the idea.
And if you are using the Android system SQLite, then this could be mapped on to the notion of Android API level compatibility by knowing which versions of Android shipped with which versions of SQLite.
That's the issue I mentioned in the README about sqlite3_close_v2().
And of course, this is different now that using the Android system SQLite is prohibited from Android N and up.
OTOH, if you are using a SQLite that is bundled with the app, then the question of Android API level compatibility becomes different.  It is in that context that I was thinking when I said that I believe SQLitePCL.raw is compatibility a long way back.
But the SQLite features you are calling are still determined by the specific build of SQLite you are using.

Interesting, this all make sense. I actually am now looking into this and the bug was report against the old version, but their 3.0 is now using bundle_green, so it should be good.


JavaScript Capstone Project!.. 🎯

Hi Reviewer, 👋
This project: The JavaScript capstone project, is about building a web application based on an external API.
We implemented the following:

A home page showing a list of items displayed from API.
A popup window showing item details plus comments is also displayed from API.
Like button from the items updates the like count.
The number of items and the one for comments is tracked and updated on the page.

This project:

Uses GitFlow
Uses Webpack
Uses Modular programming

Hello @ christianbiring1, thanks for the code review and the feedback but ill urge you to check on this problem you had requested for change:
Kinldy make sure all your test are passing ✔ , as you may notice how some are failing.

It's working correctly on our end, and all the test is passing. i think you should install :
npm install -g jest;
npm install --save -dev jest  and more important  npm install jest -environment-jsdom . Thanks ill be waiting for feedback.


Git diff view shows "File has no contents" for modified files

Firstly, thank you for such an amazing project - I've just swapped to Atom and other than this issue it's been a great experience :)
Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
When using Atom and viewing a modified file in Git, the editor always shows "File has no contents". It correctly shows for new files but not for deleted or modified files.
I have tried with Atom Beta and Atom both in and out of safe mode. The project I am working in has a .git folder in the root.
Steps to Reproduce

Open a git based project in Atom.
Edit an existing file.
Bring up the git pane.
Click on a modified file to view it.

Expected behavior: The editor shows the git diff.
Actual behavior: The editor shows "File has no contents" (see screenshot)
Reproduces how often: 100% of the time, across two machines. I have never been able to get this to work.

Versions
atom --version
  Atom    : 1.20.0
  Electron: 1.6.9
  Chrome  : 56.0.2924.87
  Node    : 7.4.0

apm --version
  apm  1.18.4
  npm  3.10.10
  node 6.9.5 x64
  python 2.7.13
  git 2.13.4

I am on OS X 10.12.6. I have also tried a completely fresh install.

I also opened the dev tools and noticed that when I click on a modified file I get this error:
Expected "* Unmerged path ", "diff ", "diff --cc ", or end of input but "\x1B" found."
name
:
"SyntaxError"

"SyntaxError: Expected "* Unmerged path ", "diff ", "diff --cc ", or end of input but "\x1B" found.
    at peg$buildStructuredError (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/what-the-diff/lib/parser.js:381:18)
    at Object.peg$parse [as parse] (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/what-the-diff/lib/parser.js:1942:17)
    at parse (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/what-the-diff/index.js:6:27)
    at /Applications/Atom Beta.app/Contents/Resources/app/node_modules/github/lib/git-shell-out-strategy.js:615:49
    at Generator.next (<anonymous>)
    at step (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/github/lib/git-shell-out-strategy.js:92:197)
    at /Applications/Atom Beta.app/Contents/Resources/app/node_modules/github/lib/git-shell-out-strategy.js:92:367
    at process._tickCallback (internal/process/next_tick.js:103:7)"

I solved it! I found this issue in atom/github: https://github.com/atom/github/issues/977
I had ui = always under the color part of my gitconfig. Deleting that fixes it. I hope this saves someone else time if they have this problem.

Isn't this still an issue, even though you found a solution? Does not sound like intended behavior.

I closed it because I think the issue that is open and tracking it is the one in atom/github (which in hindsight is where I should have opened this in the first place). That's why I closed this - for people searching for the same issue they will still find the solution + a link to the canonical issue.

I just realized that the reason for my Atom showing "File has no contents" is not due to the same error. When I open the console, what I'm seeing is:
"Unable to find item at path [path/of/file] with staging status unstaged"
Does anybody have an idea what is causing it?

@jgn-epp I'm having the same issue :(

I dont get why this bug issue is closed when its not fixed and its still broken.

@the-j0k3r I think it's because there was also another issue open with this cause of the error. The reason I'm getting "File has no contents" in the git diff pane, is because the repo I'm working in contains spaces in the path.

Yes, I made a report there too but people are still commenting here and the open bug is ignored.

Hi Epp ❤
https://github.com/atom/github/pull/1217 will fix the spaces in the path issue.

@Ben3eeE Excellent, thank you for the heads up, and I hope it will be merged and backported asap.


UI Refactor

[ ] improve navbar

[ ] language picker https://codepen.io/ScottWindon/pen/ExPqvyX?editors=1010
[ ] light/dark mode -> change slelect with toggle (mostly will only support 2 themes) ex. https://astro-shadcn-ui-template.vercel.app/
[ ] Improve link layout

[ ] Improve blog

[ ] add navigation
[ ] add base style to elements

[ ] Improve scrollbars and base style

Decided to make a new template for that


Multiple hashtags not sorted chronologically

Describe the bug
It appears that in a column displaying multiple hashtags these are not interlaced to show posts chronologically as they were posted but rather one hashtag after another. Regardless of what is actually happening, I can only see posts containing one of the hashtags of interest but I checked that posts containing the others are available int he given timeframe.

Which site: phanpy.social
Which instance: alpha.polymaths.social

To Reproduce
Steps to reproduce the behavior:
Add a column with multiple hashtags
Expected behavior
I expect to see posts with the hashtags of interest to be displayed in chronological order.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Fedora 38
Browser: Firefox
Version 119

Additional context
Add any other context about the problem here.

Is the order of posts like very distant apart? From the client-side, it literally renders whatever the API returns (there's also no parameter to sort it too). From what I understand, sometimes the order is skewed by the time they are federated, which also happens on Mastodon for the Home timeline.
Phanpy does have the logic to sort on the client-side only for Home timeline, but it's not perfect as it won't be re-sorted again when paginated, to avoid jumpy post positions.


playbook does not exist on this guest

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 2.1.2
VirtualBox 5.2.12r122591
-bash: ansible: command not found

Your OS

macOS (version)

Full console output

https://gist.github.com/monsterreh2b/d62520cabbe88a50bd1feb4d9e4a0451
Summary

vagrant up attempted twice,
vagrant provision attempted twice
outcome currently is
playbook does not exist on the guest: /vagrant/provisioning/playbook.yml

I have the same problem and can not get running my Drupal VM.

What was the solution to your issue?


Alert sorting in 0.7 is not consistent.

When testing the RC, issues are rendered in different order on each refresh. Can be reproduced simply by reloading the page, or toggling the silence switch.
Not a big issue (more an annoyance), but a bit confusing for the end-user. Suggest sorting by start-date, perhaps with a per-alertname toggle so the end user can sort by newest or oldest.

Agree! I wonder if the sorting (most recent first) can be done in the api.

This will be addressed in #872

This issue still persists.

closed by #969


Affective task: server timestamp events

This includes when the participants see a picture, when they see the rating, when they click on any of the rating box, and when they hit submit

Related issue: https://github.com/ml4ai/tomcat-baseline-tasks/issues/44

Another related issue: https://github.com/ml4ai/tomcat-baseline-tasks/issues/45

Duplicate of https://github.com/ml4ai/tomcat-baseline-tasks/issues/45

https://github.com/ml4ai/tomcat-baseline-tasks/issues/45


FR: Apply color to multiple items at once

Feature Request:
When multi-selecting notes, you cannot apply a file color
Expected result:
Able to multi-select notes and apply the same file color to all selected items or folders

+1

Would be very handy...

+1 fellow gm. been a year since the request has been opened (to my surprise). i also need this feature

+1


General information and about page

Add some text explaining what this is about?
See #7

Replaced by #16


Concurrency issue on specs build

I'm trying to work with crystal-i18n lib but I'm facing a strange error.
The original lib has broken dependencies and deprecated API calls, so, in order to make it easy to reproduce this issue, @nuxlli has created a fork and a new branch.
You can reproduce it by running the following commands:
$ git clone https://github.com/nuxlli/crystal-i18n && cd crystal-i18n
$ git checkout feature/updating_crystal
$ crystal spec
Error in ./spec/i18n_spec.cr:27: instantiating 'I18n::I18n#currency(String)'

    obj.currency(123.123.to_s).should(eq("€123,123"))
        ^~~~~~~~

in ./src/i18n/i18n.cr:111: Bug: already had input observer

      value = self.number(value, locale)
                   ^~~~~~

If I comment other number tests, leaving only the currency one to be run, it passes.
I'm not quite sure, but this sounds like a concurrency issue on specs build (i.e. a bug on Crystal language -- that's why I'm submitting this issue here)

Reduced:
def foo(x : String = "")
  /#{1}/
end

foo
foo("")

Fixed!
@fearenales As a workaround, until the next version that will contain this fix, you can replace this:
/(\d+)#{dec_separator}?(\d+)?/i

with this:
Regex.new("(\\d+)#{dec_separator}?(\\d+)?", Regex::Options::IGNORE_CASE)

Thank you @asterite ! That workaround worked like a charm! :)


add torchdr in the dimension reduction section

TorchDR is a dimensionality reduction (DR) library designed to offer a unified interface for a range of widely used DR methods, including t-SNE, UMAP, LargeVis, and others. The library achieves high computational efficiency by leveraging GPU acceleration, making it especially well-suited for handling large, high-dimensional data such as single-cell datasets.

Thanks, @huguesva. And welcome to the contributor club!


docs: clarify requirements documentation

PR Type
[x] Documentation content changes
What Is the Current Behavior?

What Is the New Behavior?
Does this PR Introduce a Breaking Change?
[x] No
Other Information
AB#92712

I'll take over @intershop/c3po-documentation-l10n

@shauke I reviewed it and I think it's understandable. However, I suggest not writing everything in bold, but putting it in a node section. While testing that, I realized that there seems to be a problem with the note boxes used in the PWA; they aren't rendered anymore. Is that a known issue?

Added Note box formatting and used the same extended information in the release notes.
https://github.com/intershop/intershop-pwa/releases


[Bugs] Component demo examples

This issue is to track what needs to be updated with the component demo windows.

[ ] Button missing the feature to add an icon.

[ ] Accordion bx--accordion__content is not using the correct font size in the demo but it is correct in the react story book. The title and content should be the same size, the content is appearing larger.

Checkbox

[ ] Enhancement: Can we add at least one more in the list to show how they would stack.

Related: https://github.com/carbon-design-system/carbon/issues/4988

[ ] Add full page feature
The option to expand the demo window to take up the full screen is missing.
https://github.com/carbon-design-system/carbon-website/issues/357#issuecomment-538528000

@aagonzales there's need for a new section for "alternates" or "variants". This would be the Storybook equivalent of 'stories', although we don't need to be nearly exhaustive with the variants we expose.

These would be for essentially swapping out the playground code for entirely new code. This would be useful for compound components and components with related purposes but distinct markup:

Datatable
Dropdown/Multiselect/Combobox
DatePicker
Skeletons

I know we wanted to avoid dropdowns in this component, but I feel like it'd be useful here to have a meaningful ux difference between a "knob" that toggles a single prop and a "variant" which swaps out the code entirely.

Accordion

[ ]  Bug: bx--accordion__content is not using the correct font size in the demo but it is correct in the react story book. The title and content should be the same size, the content is appearing larger.

@aagonzales regarding the content type: it seems like the actual type styles are only being applied to p inside of the content. Which is not something we give any guidance on. We either need to add some docs about how anything inside the accordion item should be wrapped in a p tag, or we should apply that style to the accordion content selector and not the paragraph inside it (this would be something resolved in the monorepo, not the carbon website, if that's the case). For now, I'll just wrap the text in a p.
https://github.com/carbon-design-system/carbon/blob/f1bfad63dbd4ca88fa3a285dee34d3e32fc69949/packages/components/src/components/accordion/_accordion.scss#L96-L115