Pull to refresh on desktop

Hi Hein
In the FlexScrollView Pull to refresh does not seem to work with a normal 2 finger gesture in any desktop browser.
Down press and drag works in Chrome dev tools in device mode.
A quick guess, I have not really dug into it, is that you are only listening to touch events and not scroll events.  Any idea if there is a quick hack I can use to fix this?

Hey Theo,
In order to enable that you need to set mouseMove to true.
Unfortunately, pull to refresh is not possible using mousewheel events, which are fired when you use the (Mac) trackpad. That's because of the nature of 'mousewheel' events, which don't have a 'start' and 'end' phase like mousedown/up and touchstart/touchend has.

ping @trobbertze
Can I close this?

Sure thing
On Tue, Jun 9, 2015 at 6:47 PM Hein Rutjes<EMAIL_ADDRESS>wrote:

ping @trobbertze https://github.com/trobbertze
Can I close this?
—
Reply to this email directly or view it on GitHub
https://github.com/IjzerenHein/famous-flex/issues/107#issuecomment-110428529
.


Korean translation for terminal.integated.shell.windows removes slashes from path names

    // 터미널이 Windows에서 사용하는 셸의 경로입니다. Windows와 함께 제공되는 셸을 사용하는 경우(cmd, PowerShell 또는 Ubuntu의 Bash) 64비트 버전을 사용하려면 C:WindowsSystem32보다 C:Windowssysnative가 더 좋습니다.
    "terminal.integrated.shell.windows": "cmd.exe",

"C:WindowsSystem32보다" should be "C:\Windows\System32보다"
"C:Windowssysnative보다" should be "C:\Windows\sysnative보다"

Japanese is the same:
// 端末が Windows で使用するシェルのパス。Windows に付属のシェル (cmd、PowerShell、または Bash on Ubuntu) を使用する場合、64 ビット バージョンを使用するには、C:WindowsSystem32 ではなく、C:Windowssysnative を選びます。

I'm pretty sure these locales need at least some separator in Windows paths if not \

@Tyriar I quickly checked the translations for the 3 languages and I see correct path separator in loc strings, no difference from other Latin languages. Can you elaborate repro environment as I could remember path separator on Windows OS varies on fareast languages? I guess system locale (codepage) may impact. @dbaeumer, Can you share your opinion?

I consumed translation on Friday morning. Could it be that this didn't make its way yet into MonacoLocalization repository. @Tyriar in which file are these strings. Then I can have a look.

This was on a recent build of Insiders (v1.7) and the strings are in settings.json.

loc file location, e.g. KOR - terminal.contribution.nls.resx.lcl
  <Item ItemId=";terminal.integrated.shell.windows" ItemType="0;.resx" PsrId="211" Leaf="true">
    <Str Cat="Text">
      <Val><![CDATA[The path of the shell that the terminal uses on Windows. When using shells shipped with Windows (cmd, PowerShell or Bash on Ubuntu), prefer C:\Windows\sysnative over C:\Windows\System32 to use the 64-bit versions.]]></Val>
      <Tgt Cat="Text" Stat="Loc" Orig="New">
        <Val><![CDATA[터미널이 Windows에서 사용하는 셸의 경로입니다. Windows와 함께 제공되는 셸을 사용하는 경우(cmd, PowerShell 또는 Ubuntu의 Bash) 64비트 버전을 사용하려면 C:\Windows\System32보다 C:\Windows\sysnative가 더 좋습니다.]]></Val>
      </Tgt>
      <Prev Cat="Text">
        <Val><![CDATA[The path of the shell that the terminal uses on Windows.]]></Val>
      </Prev>
    </Str>
    <Disp Icon="Str" />
  </Item>

Actually this was a tooling issue. I fixed it in the MonacoLocalization repository.


fix: resolving rate limit issue

Fixed two issues:

429 rate limit
Docker pull limit per hour

Reduced api calls from around ~100 to 1.
Redesigned the splunk_matrix_update.py file and added doc strings for better understanding.

Incorporates https://github.com/splunk/addonfactory-test-matrix-action/pull/175.

:tada: This issue has been resolved in version 2.1.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Expose '-b' flag in help messages

This PR unmarks the '-b' (broker name) flag as experimental, and adds help text on its use to the relevant commands.
cc @waterlink @AartiKriplani @nmaslarski

⚠ Not to be merged before #1572.


[PLACEHOLDER] Moving away from ether.js

[PLACEHOLDER]

We should open an issue for each package-specific like #258 #276


Cache bazel flags proto

It takes around 65ms to execute the bazel help commands in the new workspace to grab the flags. We can cache the flags per-version to avoid that.
Type of change

Performance (a code change that improves performance)

➜  aspect-cli git:(zbarsky/version) hyperfine -i 'bazel build //:no-such-target'                                                                                ~/aspect-cli
Benchmark 1: bazel build //:no-such-target
  Time (mean ± σ):     285.6 ms ±   4.2 ms    [User: 56.3 ms, System: 31.2 ms]
  Range (min … max):   280.3 ms … 292.4 ms    10 runs

➜  aspect-cli git:(zbarsky/version) ✗ hyperfine -i 'bazel-bin/cmd/aspect/aspect_/aspect build //:no-such-target'                                                                                ~/aspect-cli
Benchmark 1: bazel-bin/cmd/aspect/aspect_/aspect build //:no-such-target
  Time (mean ± σ):     244.4 ms ±   4.4 ms    [User: 49.1 ms, System: 17.9 ms]
  Range (min … max):   238.2 ms … 251.1 ms    12 runs

Test plan
The first commit adds timings so you can see how the cache helps. The second commit removes them to keep the code clean. (We should squash before merging)

Replaced by https://github.com/aspect-build/aspect-cli/commit/540b9ff8d2731f6246cdcb0b1ba1cadb2de45db9


DMR: NGINX Spike

AS A engineer
I WANT to understand better Rainer's vision for how NGINX can be used for DMR's dumb pipe
SO THAT I can see how DMR fits into the overall architecture
A working spike (quick throw-away code/config, no tests, no PR etc) to showcase how NGINX can be used to do the routing required for DMR in the context of the overall Burokratt solution.
Goal is that team is comfortable moving forward with NGINX.
Acceptance Criteria

[ ] A working demonstration of NGINX DMR routing a non-classified message to Classifer if it does not have a ministry
[ ] Work out mapping between ministry and bot forward URL
[ ] A demo on how the mapping table gets updated
[ ] How DMR handles when classifier service is not available. Switch classifier off and ensure messages are not dropped (see @tonistuulNG comment below)
[ ] Create and deploy  mock endpoints for classifier and two bots

What happens when:

Bot sends message to DMR
DMR responds with 202
DMR tries to send message to Classifier, but network is now failing / Classifier is not working
Do we lose the message?

To be split for this sprint and next sprint

https://sites.psu.edu/jasonheffner/2015/06/19/nginx-use-different-backend-based-on-http-header/

By default, Nginx accepts GET requests. A quick hack to allow POST requests as well is as following:
server {
    # ...

    error_page  405     =200 $uri;

    # ...
}

In case we have 5k participants and mapping them within 1 Nginx conf is not an option, load balancing comes to play. See https://docs.nginx.com/nginx/admin-guide/load-balancer/http-load-balancer/ for more.

More from https://docs.nginx.com/nginx/admin-guide/load-balancer/http-load-balancer/
Down for maintainance
"If one of the servers needs to be temporarily removed from the load‑balancing rotation, it can be marked with the down parameter in order to preserve the current hashing of client IP addresses. Requests that were to be processed by this server are automatically sent to the next server in the group:"
upstream backend {
    server backend1.example.com;
    server backend2.example.com;
    server backend3.example.com down;
}

Using server weights
"By default, NGINX distributes requests among the servers in the group according to their weights using the Round Robin method. The weight parameter to the server directive sets the weight of a server; the default is 1:"
upstream backend {
    server backend1.example.com weight=5;
    server backend2.example.com;
    server <IP_ADDRESS> backup;
}

Define a fallback server

Report errors
Respond to participant that made the initial request that

https://serverfault.com/questions/765483/how-to-add-a-fallback-to-my-proxy-in-nginx

Define max_fails and fail_timeout on fallbacks
http://nginx.org/en/docs/http/ngx_http_upstream_module.html#upstream
upstream backend {
    server backend1.example.com weight=5;
    server <IP_ADDRESS>:8080       max_fails=3 fail_timeout=30s;
    server unix:/tmp/backend3;

    server backup1.example.com  backup;
}

"By default, requests are distributed between the servers using a weighted round-robin balancing method. In the above example, each 7 requests will be distributed as follows: 5 requests go to backend1.example.com and one request to each of the second and third servers. If an error occurs during communication with a server, the request will be passed to the next server, and so on until all of the functioning servers will be tried. If a successful response could not be obtained from any of the servers, the client will receive the result of the communication with the last server."

Spike is done here: https://github.com/buerokratt/DMR-Nginx


[pro code] Undo/Redo stack

Summary
Similar to https://github.com/mui/mui-x/issues/9339 but with more components.
Examples

https://developer.mescius.com/wijmo/demos/Undo/UndoStack/purejs

Motivation
No response
Search keywords: undo/redo

hi just checking in here! is there an estimated at which this will be worked on?

@jjram19 What's your use case?

@oliviertassinari im using DataGridPremium right now for editable table ui and a big customer request is Cmd+z undo to undo cell edits. on unfocused (non-editing state) was wondering if we could get cmd+z to undo the previous committed change to the table

So you are looking for https://github.com/mui/mui-x/issues/9339, not this issue.

I think we can categorize this feature request as part of improved form support:

scoping the undo/redo stack to a single form
resetting the stack on successful form submission

Something I'm surprised about is that it seems to be missing in react-hook-form


cutoff in time view

it seems only a subrange of time values are plotted in the feature views for some clusters. for example the cluster below, before and after splitting:

before splitting, there are no points after a certain time, after splitting you can see that there were in fact spikes across the whole recording.

definitely weird! can I have a look at this dataset? (I'll also need to know what are these cluster numbers)
I'm wondering if it's some GPU bug...

sure. i saw this before pr/397 as well, just fyi.
actually - i see it in the other dataset that i put in my drive previously:
https://drive.google.com/open?id=0B0mC-hj9DiJMfk54TE1LU1BUWUVYcDBsOFBVR2hCb05jak9COWhwUW9iU3pjOF9hLXZ3MVE
here's an example from that set. cluster 144:

and then after splitting [through a random part of the cluster]:

[also, if you'd downloaded the data before, i updated the .kwik after my manual clustering so the numbering might be different than the one you downloaded before...]

thanks! I've found and fixed the bug in PR #418 (it was not GPU-related at all, just a stupid bug in the way spikes were subselected do match the maximum number of allowed spikes in the view)..


Add a common regression test driver.

Update the Regression framework so it has a trivial common driver that we can use to list tests and run individual tests.

cc @jacksontj


Using swagger:strfmt on a []byte type (UUID) infers the type as an array of strings

Problem statement
Using // swagger:strfmt on a []byte type (UUID) infers the type as an array of strings.
I have a local version of uuid.UUID which is [16]byte under the hood. Using // swagger:strfmt uuid4 gets the type close but it is reflecting a slice/array of strings instead of a string value.
I've tried setting // swagger:type string on both the type and the property but neither have had an effect.
Swagger specification
definitions:
  Base:
    properties:
      _id:
        items:
          format: uuid4
          type: string
        type: array
        x-go-name: Identifier

Steps to reproduce
import "github.com/google/uuid"

// ID is a UUIDv4
// swagger:strfmt uuid4
// can not set swagger type here either
type ID uuid.UUID

//swagger:model base
type Base struct {
         // ↓ has no effect
	// swagger:type string
	Identifier    ID `json:"_id" bson:"_id"`
}

swagger generate spec -o swagger.yaml --scan-models

you can use strfmt.ObjectId for strfmt.UUID as the type and it should work as expected. These are TextMarshal/Unmarshal implementations

Being able to define the swagger type would be nice, and not just for UUIDs and such. It doesn't seem like I'll be able to use go-swagger so I'll close this out.
Thank you anyway.

you can use strfmt.ObjectId for strfmt.UUID as the type and it should work as expected. These are TextMarshal/Unmarshal implementations

strfmt.ObjectId behaves exactly the same, is there any workaround for this?


Fix C pointer highlighting in the hover widget

The existing regex pattern ("\\[*c\\]") for C pointers is wrong as the asterisk is not escaped. This worked fine in the editor somehow, but shows weird highlighting in the hover widget.
This PR fixes it to highlight *c inside [] correctly.
Before:

After:

Thanks!


Cannot find module: 'moralis' while Netlify deploy

I'm submitting a ...
[x] bug report
[x] feature request

Hi, I am trying to deploy the app to Netlify, but it shows error:
1:23:45 PM: > react-scripts build
1:23:47 PM: Creating an optimized production build...
1:23:53 PM: Failed to compile.
1:23:53 PM:
1:23:53 PM: ./node_modules/react-moralis/lib/index.esm.js
1:23:53 PM: Cannot find module: 'moralis'. Make sure this package is installed.

I will install moralis to my project now, but it is possible to fix it for new users?

I'm submitting a ...
[x] bug report
[x] feature request

Hi, I am trying to deploy the app to Netlify, but it shows error:
1:23:45 PM: > react-scripts build
1:23:47 PM: Creating an optimized production build...
1:23:53 PM: Failed to compile.
1:23:53 PM:
1:23:53 PM: ./node_modules/react-moralis/lib/index.esm.js
1:23:53 PM: Cannot find module: 'moralis'. Make sure this package is installed.

I will install moralis to my project now, but it is possible to fix it for new users?

This package requires react, react-dom and moralis to be installed dependencies (to keep the bundle-size of this package small). Please let me know if you still have issues after adding the dependencies.

No, it worked.
But maybe then need to update instructions? Usually, people copy, and locally it worked without moralis install.


Proxy Scrape improvement + Awesome Readme

Hi! I made a better version of your program; Now, the program doesn't stop when the procies are invalid, but it rescrapes new ones
And i also made a better readme ;)

All looks fine, thanks for this pull request.


Is there a possibility not to have height correction?

https://github.com/antoniolago/react-gauge-component/blob/b47d07d38dd110d95cbc725a25efab416be70443/src/lib/GaugeComponent/hooks/chart.ts#L80C11-L80C11

Hello, yes, it is possible but I'm curious about your use case, since removing height correction will cut the gauge:

maybe tell us about your issue so we can think about a solution


[liblzma] Build error on arm64-android when trying to build GDAL for arm64-android

Hi, when attempting to install GDAL (and exactly liblzma), I received the following error.  I installed this using the NDK 21.3.6528147 toolchain, but also faced difficulty installing GDAL with another NDK's toolchain (25.1.8937393 in that case).
The command(s) I used to try and install gdal for arm64-android. Please let me know if there is a workaround.
git pull
./vcpkg update
./vcpkg upgrade
export ANDROID_NDK_HOME=~/Android/Sdk/ndk/25.1.8937393/
./vcpkg install gdal:arm64-android

In file included from /home/auterion/repos/auterion-qgroundcontrol/libs/libgdal/vcpkg/buildtrees/liblzma/src/v5.6.2-7484848813.clean/src/liblzma/check/crc32_fast.c:21:
/home/auterion/repos/auterion-qgroundcontrol/libs/libgdal/vcpkg/buildtrees/liblzma/src/v5.6.2-7484848813.clean/src/liblzma/check/crc32_arm64.h:57:9: warning: implicit declaration of function '__crc32b' is invalid in C99 [-Wimplicit-function-declaration]
crc = __crc32b(crc, *buf);
^
/home/auterion/repos/auterion-qgroundcontrol/libs/libgdal/vcpkg/buildtrees/liblzma/src/v5.6.2-7484848813.clean/src/liblzma/check/crc32_arm64.h:67:9: warning: implicit declaration of function '__crc32d' is invalid in C99 [-Wimplicit-function-declaration]
crc = __crc32d(crc, aligned_read64le(buf));
^
/home/auterion/repos/auterion-qgroundcontrol/libs/libgdal/vcpkg/buildtrees/liblzma/src/v5.6.2-7484848813.clean/src/liblzma/check/crc32_arm64.h:82:32: error: use of undeclared identifier 'HWCAP_CRC32'
return (getauxval(AT_HWCAP) & HWCAP_CRC32) != 0;
^

Thanks for the quick response. I will pull down r26 and give that a shot!
On Wed, Sep 25, 2024 at 10:13 PM Kai Pastor @.***>
wrote:

vcpkg CI uses NDK r26.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/issues/41178#issuecomment-2375768227,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHVVHEE3R654VDKQ2OTVXP3ZYN3XFAVCNFSM6AAAAABO355BY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZVG43DQMRSG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

So, I was indeed able to build using the default static entry for the VCPKG_LIBRARY_LINKAGE, but my project needs to compile GDAL to be dynamically-linked, so I tried updating the triplets/arm64-android.cmake to reflect this and unfortunately that lead to more compilation error.
So, to summarize:

I am now using NDK r26
I must generate dynamically-linked libraries
I switched to using Windows

Here is the general output:
-- Using msys root at C:/Users/josh/Downloads/vcpkg-master/vcpkg-master/downloads/tools/msys2/212eb2bc95cd8a82
-- Using cached msys2-mingw-w64-x86_64-pkgconf-1~2.2.0-1-any.pkg.tar.zst.
-- Using cached msys2-msys2-runtime-3.5.3-3-x86_64.pkg.tar.zst.
-- Using msys root at C:/Users/josh/Downloads/vcpkg-master/vcpkg-master/downloads/tools/msys2/1e74ca60daa10104
-- Configuring arm64-android-dbg
-- Configuring arm64-android-rel
-- Building arm64-android-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:134 (message):
    Command failed: C:/Users/josh/Downloads/vcpkg-master/vcpkg-master/downloads/tools/msys2/212eb2bc95cd8a82/usr/bin/make.exe -j 29 --trace -f Makefile all
    Working Directory: C:/Users/josh/Downloads/vcpkg-master/vcpkg-master/buildtrees/libiconv/arm64-android-dbg/
    See logs for more information:
      C:\Users\josh\Downloads\vcpkg-master\vcpkg-master\buildtrees\libiconv\build-arm64-android-dbg-out.log
      C:\Users\josh\Downloads\vcpkg-master\vcpkg-master\buildtrees\libiconv\build-arm64-android-dbg-err.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_build_make.cmake:136 (vcpkg_execute_build_process)
  scripts/cmake/vcpkg_install_make.cmake:2 (vcpkg_build_make)
  ports/libiconv/portfile.cmake:39 (vcpkg_install_make)
  scripts/ports.cmake:192 (include)

Here is my log file:
libtool: compile:  C:/Users/josh/Downloads/android-ndk-r26d-windows/android-ndk-r26d/toolchains/llvm/prebuilt/windows-x86_64/bin/clang.exe -I. -I../.././../src/v1.17-84a471678e.clean/libcharset/lib -I.. -I../.././../src/v1.17-84a471678e.clean/libcharset/lib/.. -I../include -g -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -Wformat -Werror=format-security -fPIC -fno-limit-debug-info -fvisibility=hidden --target=aarch64-none-linux-android21 -DANDROID -D_FORTIFY_SOURCE=2 --sysroot=C:/Users/josh/Downloads/android-ndk-r26d-windows/android-ndk-r26d/toolchains/llvm/prebuilt/windows-x86_64/sysroot -DBUILDING_LIBCHARSET -DHAVE_CONFIG_H -c ../.././../src/v1.17-84a471678e.clean/libcharset/lib/relocatable-stub.c  -fPIC -DPIC -o .libs/relocatable-stub.o
Makefile:55: update target 'libcharset.la' due to: target does not exist
/bin/sh ../libtool --mode=link C:/Users/josh/Downloads/android-ndk-r26d-windows/android-ndk-r26d/toolchains/llvm/prebuilt/windows-x86_64/bin/clang.exe -LC:/Users/josh/Downloads/vcpkg-master/vcpkg-master/installed/arm64-android/debug/lib --target=aarch64-none-linux-android21 -Wl,--build-id=sha1 -Wl,--no-rosegment -Wl,--no-undefined-version -Wl,--fatal-warnings -Wl,--no-undefined -Qunused-arguments --sysroot=C:/Users/josh/Downloads/android-ndk-r26d-windows/android-ndk-r26d/toolchains/llvm/prebuilt/windows-x86_64/sysroot -g -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -Wformat -Werror=format-security -fPIC -fno-limit-debug-info -fvisibility=hidden -o libcharset.la -rpath /C/Users/josh/Downloads/vcpkg-master/vcpkg-master/installed/arm64-android/debug/lib -version-info 1:0:0 -no-undefined localcharset.lo relocatable-stub.lo
libtool: link: C:/Users/josh/Downloads/android-ndk-r26d-windows/android-ndk-r26d/toolchains/llvm/prebuilt/windows-x86_64/bin/clang.exe -shared  -fPIC -DPIC  .libs/localcharset.o .libs/relocatable-stub.o   -LC:/Users/josh/Downloads/vcpkg-master/vcpkg-master/installed/arm64-android/debug/lib -lc  -Wl,--build-id=sha1 -Wl,--no-rosegment -Wl,--no-undefined-version -Wl,--fatal-warnings -Wl,--no-undefined --sysroot=C:/Users/josh/Downloads/android-ndk-r26d-windows/android-ndk-r26d/toolchains/llvm/prebuilt/windows-x86_64/sysroot -g -fstack-protector-strong   -Wl,-soname -Wl,libcharset.so -o .libs/libcharset.so
make[2]: Leaving directory '/c/Users/josh/Downloads/vcpkg-master/vcpkg-master/buildtrees/libiconv/arm64-android-dbg/libcharset/lib'
make[1]: Leaving directory '/c/Users/josh/Downloads/vcpkg-master/vcpkg-master/buildtrees/libiconv/arm64-android-dbg/libcharset'

There is no error logged. The -lc looks unusual, but IDK.
vcpkg CI build Android on Linux. This is targeting POSIX on POSIX.
Windows hosts might work to some extent, but with GDAL at the end of a long pipeline, you are very likely to run into problems. This is targeting POSIX on WINDOWS partially using a POSIX runtime (MSYS2). Many ways to fail. For example host tools built by icu are known to get path separators wrong.

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


Module missing attribute

Fix for AttributeError: 'module' object has no attribute 'TYPES' error.

Thanks, merging.


Crashing in character creation.

When I goto characters -> new, sometimes it crashes on the race menu when picking a race, and sometimes it crashes immediately after going to the class choice (they never even appear).
Game Version: 1.5.74
UB Version: 1.5.74

Not this mod. Check your game files.


Enums not modeled as strings

Looks like a regression?
Enums that were formerly generated as strings appear to be coming out as enum types.
Swagger file
https://github.com/Microsoft/botbuilder-dotnet/blob/master/libraries/Swagger/ConnectorAPI.json
"DeliveryModes": {
      "description": "Values for deliveryMode field",
      "enum": [
        "normal",
        "notification"
      ],
      "type": "string",
      "properties": {},
      "x-ms-enum": {
        "name": "DeliveryModes",
        "modelAsString": true
      }
    },

Note: Model as string is true..
Generated Code
    /**
     * Get a delivery hint to signal to the recipient alternate delivery paths for the activity.
     The default delivery mode is "default". Possible values include: 'normal', 'notification'.
     *
     * @return the deliveryMode value
     */
    public DeliveryModes deliveryMode() {
        return this.deliveryMode;
    }

    /**
     * Set a delivery hint to signal to the recipient alternate delivery paths for the activity.
     The default delivery mode is "default". Possible values include: 'normal', 'notification'.
     *
     * @param deliveryMode the deliveryMode value to set
     * @return the Activity object itself.
     */
    public Activity withDeliveryMode(DeliveryModes deliveryMode) {
        this.deliveryMode = deliveryMode;
        return this;
    }

Reference
http://azure.github.io/autorest/extensions/

This is by design - these types are what we called ExpandableStringEnum. They appear like enums and have all the benefits of enums, but you can instantiate a new one as if they are "expandable".


Update django.rst

Easier copy paste

I don't see the need for this change, people shouldn't just copy-and-paste code.
Thanks!


az acr repository show-manifests --detail

This is autogenerated. Please review and update as needed.
Describe the bug
az acr repository show-manifests --detail does not return the same manifest data as what is shown in portal.
Command Name
``  az acr repository show-manifests -n weblogixtest --repository nginx --detail
Errors:
To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
``  az acr repository show-manifests -n weblogixtest --repository nginx --detail

Expected Behavior
Would expect to see the detailed manifest for the image/tag as what is shown in the portal.
{
"schemaVersion": 2,
"mediaType": "application/vnd.docker.distribution.manifest.v2+json",
"config": {
"mediaType": "application/vnd.docker.container.image.v1+json",
"size": 7463,
"digest": "sha256:b9e1dc12387ae52eee5da783128deb8bdee0dc3b29fc3874cd81cf2190f71099"
},
"layers": [
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 27108069,
"digest": "sha256:a076a628af6f7dcabc536bee373c0d9b48d9f0516788e64080c4e841746e6ce6"
},
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 26468819,
"digest": "sha256:45d7b5d3927df5b2939f5caad6115d3f3bcb914846255bc4800c7baceb500691"
},
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 601,
"digest": "sha256:5e326fece82e18449f9f3b56f38dbcbaa1f92dffd7b88d2d560e6208d62bb063"
},
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 903,
"digest": "sha256:30c386181b689e0ec7e2bbde4d312decd39b1e28158f12b6d1c02a730a8f6c24"
},
{
"mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
"size": 666,
"digest": "sha256:b15158e9ebbec109b3c4541fcfb2c3fe3ce7f9efba6ca98abb9708349ed9b63f"
}
]
}
Environment Summary
Linux-4.19.128-microsoft-standard-x86_64-with-glibc2.29
Python 3.8.5
Shell: bash

azure-cli 2.0.81

Extensions:
azure-devops 0.17.0

Additional Context

From the API I can call out to
curl -X GET -H "Authorization: Bearer $TOKEN" -H "Content-Type:application/json" -H "Accept:application/vnd.oci.image.manifest.v1+json;q=0.4, application/vnd.docker.distribution.manifest.v2+json;q=0.5, application/vnd.docker.distribution.manifest.list.v2+json;q=0.6" https://weblogixtest.azurecr.io/v2/nginx/manifests/stable
And this allows me to fetch the info, but would be much easier if it was exposed via the az acr command space

acr

For this we might need to expose --detail from az acr repository show. Let us wait for for votes from community

+1 from me. It is definitely surprising that az acr repository show-manifests does not return Docker manifests.

I think adding --detail to az acr repository show makes sense, more detailed info about an --image would include the actual Image manifest, not just the manifest digest.

This is probably out of scope, but adding actual image manifest data like labels could be really helpful too!
My principal use case is looking up an image by matching labels applied by build pipeline on it, without having to pull each images.

Same, I was trying to figure out how to get access to the "layers" in a repository in my ACR. Except in my case it's a helm chart not a docker image, though I still want to see the layers. Looking at the requests made in the portal, it seems it requests the /v2/{repository}/manifests/{tag} endpoint to get this data, but I don't see any az acr command that can get the same.

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @toddysm, @yugangw-msft.

Could you kindly let us know whether this issue is fixed with "az acr repository show-manifest" command?

@yugangw-msft

Changing the assignment to @northtyphoon and @terencet-dev


Cell testing does not work when instance name is not provided

Description:
Cell testing requires the use of -n <cell name> option. When we does not provide a cell name, cellery creates a random one. If a cell name is not given, cell testing Job incorrectly tries to call a non existing URL as seen in the logs:
Log: Request executed: POST http://--gateway-service:80/controller/orders HTTP/1.1
Log: java.net.UnknownHostException: --gateway-service: Name or service not known
Steps to reproduce:
Run cellery testing command without specifying a name:
cellery test test/petbetest:0.0.4

Fixed in https://github.com/wso2-cellery/samples/pull/105.


Cannot read property 'apply' of undefined

L.polyline is ok
but L.cicrle has errors

it's ok, I must add the random,but why?
because the position is always same ,so it must add the random key ?

"leaflet": "1.6.0",
"leaflet-ant-path": "^1.3.0",
"react": "^16.13.1",
"react-leaflet-ant-path": "^1.1.0",


[fealib] allow script and language statements usage within a named lookup

copied from https://github.com/adobe-type-tools/afdko/issues/101#issuecomment-178309158
"The current (and intended) case in makeotf is that script and language statements affect only the enclosing feature, and are not attached to lookups defined within a feature. As a result, script and language statements used within a named lookup have no effect on other features which reference the named lookup. I find this confusing and difficult to explain, but am reluctant to break existing font data. After thinking a bit more, I propose leaving the wording as is, and just add a warning to makeotf when script and language statements are used within a named lookup."
Could fealib also allow the same processing for the code of the type:
languagesystem DFLT dflt;
feature test {
lookup L {
script grek;
pos T 100;
} L;
} test;

For an unrelated reason (https://github.com/behdad/fonttools/issues/505), feaLib’s output currently does not match makeotf’s output for this snippet.

After https://github.com/behdad/fonttools/commit/aae350c84a323ad3426b1c0f181ca877e95cc95c, the outputs of feaLib and makeotf are now identical for the above snippet.


Introduce destroy_note_unsafe

destroy_note currently computes a note hash, which it needs to pass to the kernel so that it can properly squash the note if it was a transient note. This is wasteful, because we typically have already computed the same note hash: e.g. in most cases we read before deleting.
We could have a version of destroy_note that also takes the note hash as a second parameter, where the caller needs to make certain that the hash corresponds to the note. We'd then make e.g. get_notes return both the notes and the hashes, so that they can be passed to destroy_note_unsafe.

With https://github.com/AztecProtocol/aztec-packages/pull/7834 merged, we should use this new function in pop_notes.


Methods that have the same name will share the same map/counter in selective logging

Describe the bug
If there are 2 methods in the same class with the same name but different jvm signatures, they will share the same map/counter in the context of selective logging.
Example :
Process counter is set to "2".
Class A has methods m1() and m1(int) [same names but different signatures]
DirectInvoking m1() will yield a candidate, calling m1(int) right after that will not.
Reproduction steps

Run unlogged-spring-maven-demo with a counter of your choice
DirectInvoke generateReferralCode from org.unlogged.demo.utils.ReferralUtils -> generates candidate
DirectInvoke generateReferralCode(int customLimit) next, there will be no candidate

Expected behavior
For methods with different signatures to have different counters/maps.
Additional context
No response

This is fixed with SDK v0.6.102 and can be tested. Video Demo is here.

This is no longer happening on sdk 0.6.102. Closing.


Make initial release of this repo

@szakeri
Issues: Issue #56
Problem: Need to move the three supported templates from unsupported to
f5_supported
Analysis: Moved the 2_nic and 3_nic standalone templates to
f5_supported, along with the patch_upload_ve_image template.
Tests: Ran all functional tests for each template and remapped the tests
to look in the supported directory, instead of unsupported

@jputrino
I would like you to checkout the brief documentation I made for running the functional tests.


can't work on node 4+

my mac's environment is

node v4.2.1
node-gyp v3.0.3

Looks like this might be because tchannel needs to bump farmhash version @Raynos

We also suspect that bufrw depends details of the node 0.10 era Buffer, but have not yet investigated.


Add metric PMAD

🚀 Feature Request
Add metric class to calculate PMAD (see page 15)
Proposal

In functional_metrics.py create method pmad to calculate the metric
In metrics.py create class PMAD(Metric)
Define greater_is_better property for this metric

Test cases

Add pmad function to all the tests in test_functional_metrics.py
Add PMAD class to all the tests in test_metrics.py

Additional context
No response

The reference in commit is accidental, the commit itself shouldn't be reviewed.

As seen here and in etna.metrics.functional_metrics.py, WAPE follows the same formula as PMAD, but is more widespread.
Should the issue thereby be closed, or should PMAD be implemented (regardless of the fact that it is a duplicate of WAPE)?

Not relevant because of present WAPE.


IgnoreMaxNS leads to false computingcolaboration maxNs

I am moving the issues from our audit collaboration repo to the nmt repo, because we decided to report issues here. It would be easier to follow the progress.
Here is the description of the issue about computing maxNs with n.ignoreMaxNS set to true that we discussed about.
We are not sure what is the expected behavior when it comes to calculating maxNs with n.ignoreMaxNS set to true.
As we understand the expected behavior under these conditions is to choose a maxNs that is less than n.precomputedMaxNs if possible. If this is the case there might be an issue with computing maxNs here, which can lead to setting it to n.precomputedMaxNs  even if it could be set to a namespaceId that is smaller than n.precomputedMaxNs .
If we assume the following:

n.ignoreMaxNS = true
leftMinNs != n.precomputedMaxNs
rightMaxNs = n.precomputedMaxNs
rightMinNs < n.precomputedMaxNs
rightMinNs>leftMaxNs .

Than the maxNs will be set to n.precomputedMaxNs even if it should be set to rightMinNs because it is the largest namespaceId smaller than n.precomputedMaxNs.
Example with numbers (presented also in the image at the end):
Lets assume that following:

precomputedMaxNs = 10
ignoreMaxNS = true
leftNode:  left{minNs: 7; maxNs:8 }
rightNode:  right{minNs: 9; maxNs:10 }

Based on this logic in HashNode the  minNs and `maxNs' for the node that is being calculated will take the following values:

minNs = 7 (because it is the lowest value)
maxNs = 10 or the precomputedMaxNs (because the else branch of the if statement will be executed and there 10 will obviously be chosen even if the  right.minNs is larger then left.maxNs but not equal to precomputedMaxNs ).

This numerical example with the current behavior and the behavior we expect is graphically  presented in the following image.

Here is the link to the issue on the audit repo.

This issue has been resolved through the following pull request: link. The PR incorporates specific unit tests that focus on the calculation of namespace ranges for nodes. These calculations are based on the namespaces of their respective left and right children, specifically when the IgnoreMaxNamespace flag is enabled. These additions to the codebase aim to provide a clear understanding of the expected behavior when this flag is turned on.


[Bug] exec: "./startup.sh": permission denied: unknown.

After building, I try to run the container and no matter the permissions or if I run docker as sudo I get the following error:
docker: Error response from daemon: OCI runtime create failed: container_linux.go:370: starting container process caused: exec: "./startup.sh": permission denied: unknown.
I did a git clone to /opt and then cd'd in to youtube-dl-server and ran the build command. I did a chmod +x startup.sh and then tried to run the container but no luck. Any suggestions would be appreciated.
My run command is as follows:
sudo docker run -dit --name yds --rm -p 8010:8080 \
 -v /opt/youtube-dl-server/downloads:/app/downloads \
 -v /opt/youtube-dl-server/db:/app/db \
 -e APPNAME=YDS \
 -e ADMINUSER=$USER \
 -e PASSWORD=$PW \
 -e TZ=$TZ \
 katznboyz1/youtube-dl-server:latest

Hiya, unfortunately I am not familiar with docker, and I am relying on contributors for docker support, so I can't be sure what is causing this. I'm sure you already googled, and may have even found the same link, but here was a link that seems to describe your problem https://stackoverflow.com/questions/44687685/getting-permission-denied-in-docker-run. I will leave this issue open with the hopes of other people chiming in.

Hey thanks! I added the following to the Dockerfile:
RUN chmod +x startup.sh
right above
ENTRYPOINT ["/startup.sh"]
Fired right up. I have been using images for awhile but as is probably apparent, I have never touched the actual dockerfiles before,
Thanks again.


pyautogui.FAILSAFE

Describe the bug
If the bot goes ingame, the error comes and the game closes
Version
3.0.1
ERROR CODE:
PyAutoGUI fail-safe triggered from mouse moving to a corner of the screen. To disable this fail-safe, set pyautogui.FAILSAFE to False. DISABLING FAIL-SAFE IS NOT RECOMMENDED.
[20:23:22] [ERROR  ] Traceback (most recent call last):
File "C:\Users\test\Desktop\lolbot\lolbot\bot\bot.py", line 67, in run
launcher.launch_league(
File "C:\Users\test\Desktop\lolbot\lolbot\bot\launcher.py", line 60, in launch_league
manual_login(username, password)
File "C:\Users\test\Desktop\lolbot\lolbot\bot\launcher.py", line 76, in manual_login
controller.write(username)
File "C:\Users\test\Desktop\lolbot\lolbot\bot\controller.py", line 26, in write
pyautogui.write(keys, interval=0.11)
File "C:\Users\test\AppData\Local\Programs\Python\Python311\Lib\site-packages\pyautogui_init_.py", line 593, in wrapper
failSafeCheck()
File "C:\Users\test\AppData\Local\Programs\Python\Python311\Lib\site-packages\pyautogui_init_.py", line 1734, in failSafeCheck
raise FailSafeException(
pyautogui.FailSafeException: PyAutoGUI fail-safe triggered from mouse moving to a corner of the screen. To disable this fail-safe, set pyautogui.FAILSAFE to False. DISABLING FAIL-SAFE IS NOT RECOMMENDED.
[20:23:22] [ERROR  ] Unknown Error. Exiting

Interesting. Does this happen every time the bot logs in?

Traceback (most recent call last):
File "urllib3\connectionpool.py", line 466, in _make_request
File "urllib3\connectionpool.py", line 1095, in validate_conn
File "urllib3\connection.py", line 730, in connect
File "urllib3\connection.py", line 909, in ssl_wrap_socket_and_match_hostname
File "urllib3\util\ssl.py", line 469, in ssl_wrap_socket
File "urllib3\util\ssl.py", line 513, in _ssl_wrap_socket_impl
File "ssl.py", line 455, in wrap_socket
File "ssl.py", line 1041, in _create
File "ssl.py", line 1319, in do_handshake
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "urllib3\connectionpool.py", line 789, in urlopen
File "urllib3\connectionpool.py", line 490, in _make_request
urllib3.exceptions.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000)
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "requests\adapters.py", line 667, in send
File "urllib3\connectionpool.py", line 843, in urlopen
File "urllib3\util\retry.py", line 519, in increment
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='api.github.com', port=443): Max retries exceeded with url: /repos/iholston/lol-bot/releases/latest (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000)')))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "main.pyw", line 8, in
File "lolbot\view\main_window.py", line 35, in init
File "lolbot\view\about_tab.py", line 17, in init
File "requests\api.py", line 73, in get
File "requests\api.py", line 59, in request
File "requests\sessions.py", line 589, in request
File "requests\sessions.py", line 703, in send
File "requests\adapters.py", line 698, in send
requests.exceptions.SSLError: HTTPSConnectionPool(host='api.github.com', port=443): Max retries exceeded with url: /repos/iholston/lol-bot/releases/latest (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000)')))
this is mine

I'll have to look into that one as well.

I believe both of these issues should be fixed in the latest release.


net.connect issue with Node.js 7.10.0

A commit recently landed in Node core that was released as v7.10.0 two days ago.
It breaks async-listener because net.connect no longer calls Socket.prototype.connect. Furthermore, it's not possible to simply monkey-patch net.connect because it's being called internally in node by simply referencing the connect function and not the exports.connect function.
I'm working on fixing this together with @cjihrig and @jasnell. See https://github.com/nodejs/node/pull/12852 for details.
It will most likely be released as v7.10.1.

The work to fix this have now been moved to https://github.com/nodejs/node/pull/12861


remove deploy documentation to /documentation

Summary
copy of https://github.com/qiskit-community/qiskit-nature/pull/1222
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4448

Relevant Lines:
4813

💛  - Coveralls


Linting rule for license header + adds missing headers.

R: @philipwalton @addyosmani @gauntface
Fixes #867
I'm just enforcing it for packages/**/*.{mjs,js} and I'm only checking for the Copyright portion (since we've used copyright headers that vary in formatting subtly), but that might be sufficient. Let me know if you think otherwise and I'll adjust the PR.

Thanks for taking the time to do this.


Unexpected CFBundleExecutable Key. The bundle at 'Payload/MyApp.app/FirebaseAuthUI.bundle' does not contain a bundle executable

Xcode version: 10.2
Firebase SDK version: 5.20.1
Firebase Component: FirebaseAuthUI
Component version: 6.2.1

I am trying to build and submit my app to Testflight. It builds fine and then when it goes to upload it to testflight, it comes back with the error:

ERROR ITMS-90535: "Unexpected CFBundleExecutable Key. The bundle at 'Payload/MyApp.app/FirebaseAuthUI.bundle' does not contain a bundle executable. If this bundle intentionally does not contain an executable, consider removing the CFBundleExecutable key from its Info.plist and using a CFBundlePackageType of BNDL. If this bundle is part of a third-party framework, consider contacting the developer of the framework for an update to address this issue."

Why? I haven't had this issue before.

The FirebaseUI Auth bundle should contain only assets, so you're safe removing the bundle executable key as the error says.
I'm not sure what could have caused this issue. Are you using CocoaPods to integrate FirebaseUI?

@morganchen12 I am using Cocoapods. Weirdly this error was caused on CircleCI when building and submitting my app. However when I built it on my computer and submitted it (without changing anything), no error occurred.

Strange. Can you file this bug on CircleCI's bug tracker? We'll keep this issue open for now in case anyone else runs into it.

@morganchen12 Will do. I just tried uploading a new build and CircleCI gives me the same error. I notice they're not using the release version of XCode 10.2, could that be an issue?

@yusuftor Did you find fix?

@mefilt Its to do with Xcode 10.2 not being supported. They are working on releasing it very soon apparently: https://discuss.circleci.com/t/when-can-we-expect-the-final-xcode-10-2-image-to-be-released/29261/33

Closing this bug, since it's not actionable here. Please feel free to continue commenting if this issue resurfaces.


Fallback to Vibration API for < iOS 10

Is it possible for a built in fallback if < iOS 10?

Hi Noitidart, thanks for the suggestion. I need to find some time to update the lib, but I will keep that in mind. Thanks!

Thank you for being so open! :)

Haha not a problem, I appreciate the feedback

I've added this conditional to our app for now:
Platform.OS === "ios" && parseInt(Platform.Version, 10) >= 10 ? ReactNativeHaptic.generate('selection') : Vibration.vibrate()

Any plans to add Android haptics?  This package is unstable - https://github.com/milk-and-cookies-io/react-native-haptic-feedback
Would love to get Android haptics up and running and I'd be willing to help build it out.

Hello Wesley! I would love that, feel free to make a PR and I will merge it in. Thank you!

Charles,
I'm currently writing custom Android vibration patterns to match the iOS haptic methods.  I'm planning on submitting a PR in the near future.

Amazing @konjoinfinity, thank you for your contribution!

"Near future" means today haha

@konjoinfinity That's great!


docs: BugHerd Tasks 88, 89, 90, 91

SUMMARY
Address BugHerd Tasks
88 - Document https://github.com/apache/superset/discussions/23745#discussioncomment-5801457
89 - Document AUTH_ROLES_MAPPING https://apache-superset.slack.com/archives/C0170U650CQ/p1696514613463559
90 - Document about SUPERSET_CONFIG_PATH variable
91 - Specifying correct python version in docs for official docker image
cc: @sfirke
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thanks @mdeshmu for the change. Would you mind providing links to the Bugherd tasks and additionally include a synopsis of the task so that anyone without access to Bugherd can grok what the underlying issues being resolved are.

@john-bodley added links of the tasks and synopsis in the description

@sfirke @rusackas please review.

Thanks for flagging, I will review this.

This are great additions @mdeshmu !  Only other thing: I noticed while reviewing this that the page title for https://superset.apache.org/docs/security/ is "Role based Access" - it should be "Security" probably.  It's confusing to see that on the search engine results page.  I'd also like to remove to the top heading that says "Roles", as that first summary paragraph should stand alone with no heading.
Are you up for making those fixes while you are editing this page?

@sfirke Those are great suggestions.
This is gonna improve my English comprehension a lot. :)
I will incorporate them soon.

@sfirke All changes are done.

Thanks @mdeshmu , this was a great addition!


'Rename array() function to make_array(), extend array[]

~Draft until~:

[x] builds on https://github.com/apache/arrow-datafusion/pull/3121
[x] Needs improved type errors

Which issue does this PR close?
Closes  https://github.com/apache/arrow-datafusion/issues/3115
Closes https://github.com/apache/arrow-datafusion/issues/3123
Rationale for this change
The core rationale is that this change is needed to support the next release of sqlparser-rs (which includes https://github.com/sqlparser-rs/sqlparser-rs/pull/566) which makes it impossible to use array as a function
More details here: https://github.com/sqlparser-rs/sqlparser-rs/pull/566#discussion_r941921187
The array() function is Spark syntax.
Postgres syntax is array[] and postgres actually treats the array keyword specially (it can not be used as a function)
alamb=# select array(1,2);
ERROR:  syntax error at or near "1"
LINE 1: select array(1,2);

What changes are included in this PR?

Rename array() to make_array() (so it doesn't conflict with postgres)
Update SQL planner to support array[] call make_array for array[] syntax
Require all arguments to make_array and array[] be the same type (as arrays have a single type for all elements)
Tests for same

Are there any user-facing changes?

array() is now written make_array()
array[] supports columns rather than just constants (e.g. array[column, column, ..] now works)
make_array() will error if the types are not exactly the same

@alamb I believe this is the next PR in line to be merged into sqlparser-0.21

@alamb I believe this is the next PR in line to be merged into sqlparser-0.21

Never mind. These changes are not required in order to upgrade.

Update on this PR: I think I may have gotten too ambitious in this PR: array() somewhat confusingly returns a FixedSizedList and array[] returns a List.
Maybe we can just settle for updating the name of the array function to make_array 🤔  Will sleep on it

Update on this PR: I think I may have gotten too ambitious in this PR: array() somewhat confusingly returns a FixedSizedList and array[] returns a List.
Maybe we can just settle for updating the name of the array function to make_array thinking Will sleep on it

I went ahead and created a PR just to do the rename: https://github.com/apache/arrow-datafusion/pull/3199

Thanks @andygrove  -- I am going to abandon this PR for now. I learned a lot but I don't have time to figure out the implementation details of ListArrays more ;)


FIRRTL is happy to rename main when deduping

Type of issue: bug report
The main module should never be a candidate for renaming.
The following FIRRTL:
circuit main:
  module dupe:
    input in: UInt<8>
    output out: UInt<8>
    out <= in
  module main:
    input in: UInt<8>
    output out: UInt<8>
    out <= in

will die with the following:
Exception in thread "main" firrtl.FIRRTLException: Internal Error! Please file an issue at https://github.com/ucb-bar/firrtl/issues
	at firrtl.Utils$.error(Utils.scala:440)
	at firrtl.Utils$.throwInternalError(Utils.scala:181)
	at firrtl.Driver$.$anonfun$execute$1(Driver.scala:252)
	at logger.Logger$.$anonfun$makeScope$2(Logger.scala:138)
	at scala.util.DynamicVariable.withValue(DynamicVariable.scala:58)
	at logger.Logger$.makeScope(Logger.scala:136)
	at firrtl.Driver$.execute(Driver.scala:224)
	at firrtl.Driver$.execute(Driver.scala:302)
	at firrtl.Driver$.main(Driver.scala:318)
	at firrtl.Driver.main(Driver.scala)
Caused by: java.util.NoSuchElementException: key not found: main
	at scala.collection.MapLike.default(MapLike.scala:231)
	at scala.collection.MapLike.default$(MapLike.scala:230)
	at scala.collection.AbstractMap.default(Map.scala:59)
	at scala.collection.mutable.HashMap.apply(HashMap.scala:65)
	at firrtl.transforms.CheckCombLoops.run(CheckCombLoops.scala:258)
 ...

What is the current behavior?
If main is the last of a set of duplicate modules, it will be removed and FIRRTL references to it replaced with its surviving duplicate. However, the FIRRTL compiler has squirreled away references to main and it will be upset when it disappears.

What is the expected behavior?
main should win any deduplication battles.

Please tell us about your environment:

Darwin dhcp-35-82.EECS.Berkeley.EDU 18.2.0 Darwin Kernel Version 18.2.0: Thu Dec 20 20:46:53 PST 2018; root:xnu-4903.241.1~1/RELEASE_X86_64 x86_64
FIRRTL afdb780aeca80c1fed94dd20fd22037490169472

What is the use case for changing the behavior?
Preserve the main!
Impact: no functional change
Development Phase: proposal

I think the use case you are looking for is "Remember the Main!"

Missed it by that much ...

Fixed by #1023


A few minor bug fixes

Fixed a small bug in font changes through translation files

It's not a bug?

Nvm i misread it


MongoDB in Kubernetes Fails to Run as Root Use

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
I've encountered an issue when trying to deploy a MongoDB instance within a Kubernetes cluster where MongoDB fails to run as the root user, and I'm seeking assistance in resolving this.
Environment Details:
Kubernetes Cluster Version: v1.25.6 (AKS)
MongoDB Image: docker.io/vbergbauer/examples-bookinfo-mongodb:1.16.4
Kubernetes Deployment Configuration:
When attempting to deploy MongoDB within our Kubernetes cluster, the deployment is failing the main reason is permission I enabled istio_cni in istio control plane istio requires to deploy as root user but mongodb only (not other deployment) can not deploy as root user.
apiVersion: apps/v1
kind: Deployment
metadata:
  name: mongodb-v1
  namespace: bookinfo
  labels:
    app: mongodb-v1
    version: v1
spec:
  replicas: 2
  selector:
    matchLabels:
      app: mongodb-v1
      version: v1
  template:
    metadata:
      labels:
        app: mongodb-v1
        version: v1
    spec:
      serviceAccountName: mongodb-database
      securityContext:
        seccompProfile:
          type: RuntimeDefault
      containers:
      - name: mongodb-v1
        image: docker.io/vbergbauer/examples-bookinfo-mongodb:1.16.4
        imagePullPolicy: IfNotPresent
        ports:
        - containerPort: 27017
        volumeMounts:
        - name: data-db
          mountPath: /data/db
        securityContext:
          allowPrivilegeEscalation: false
          runAsUser: 1000
          runAsGroup: 3000
          fsGroup: 2000
          capabilities:
            drop: ["ALL"]
          runAsNonRoot: true
      volumes:
      - name: data-db
        persistentVolumeClaim:
          claimName: bookinfo-mongodb

Including the error you see may be useful.


Error: Key "plugins": This appears to be in eslintrc format (array of strings) rather than flat config format (object).

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Describe the bug
This isn't strictly speaking an issue with eslint-stylistic and may also require a bug report to dbaeumer.vscode-eslint but I think it would be worthwhile to have this documented since it may influence the content in the Migration Guide.
This config:
// eslint.config.js
import eslint from "@eslint/js";
import typescriptParser from "@typescript-eslint/parser";
import typescriptPlugin from "@typescript-eslint/eslint-plugin";

export default [
	{
		"plugins": [
			"@typescript-eslint"
		],

Throws this error in the Output > ESLint tab in Visual Studio Code:
Error: Key "plugins": This appears to be in eslintrc format (array of strings) rather than flat config format (object).

In remediating this, I had to play a bit of guess and check with how the individual package guides are currently written, since the guides suggest:
// .eslintrc.js
module.exports = {
  plugins: [
    '@stylistic'
  ]

But what I ultimately needed was:
import eslint from "@eslint/js";
import typescriptParser from "@typescript-eslint/parser";
import typescriptPlugin from "@typescript-eslint/eslint-plugin";
import stylisticPlugin from "@stylistic/eslint-plugin";

export default [
	{
		"plugins": {
			"@typescript-eslint": typescriptPlugin,
			"@stylistic": stylisticPlugin
		},

But since I'm only planning on linting *.ts files, I'm not sure if it would be better to import @stylistic/eslint-plugin-ts instead.
Generally speaking, I'm not sure when it makes sense to specify rules with @stylistic/ts/ vs @stylistic/.
Reproduction
N/A
Contributes

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests

In legacy config format, plugins takes an array of string; In Flat config it takes an object: https://eslint.org/docs/latest/use/configure/configuration-files-new#configuration-objects
The document is not wrong, as we indicated it's // .eslintrc.js. While indeed we could improve the docs to also mentions about Flat config usage

Oh, I incorrectly assumed that the package docs were using the flat config format.


Parallel file operations

[x] change over to using our own implementation of symlink_status and is_symlink on windows, since the originals (in std::experimental::filesystem, anyways) are broken
[x] generic parallel work queue implementation
[x] delete files in parallel
[ ] copy files in parallel?
[ ] download files in parallel?

It seems to be related to #6989.

Please see #7405 -- this broke compiling vcpkg using VS2015


missing Windows IoT Enterprise policy csp reference

The page "Policies in Policy CSP supported by Windows 10 IoT Enterprise" is visible in the search/filter on the left side, but the link does not lead to Windows IoT Enterprise, where has the content gone? I very much desire to read that page.

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

ID: 33e7455a-e798-e89e-a9bf-bdff51d00ba6
Version Independent ID: 7dc81b34-4cc2-2ecc-bb5b-9900b85e3817
Content: Policies in Policy CSP supported by Windows 10 IoT Core - Windows Client Management
Content Source: windows/client-management/mdm/policies-in-policy-csp-supported-by-iot-core.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Hey @sveinungre, we're working on automating the CSP docs, and we don't have a supportability indicator for IoT Enterprise to automate this information so I'm closing this issue at this time.


Check value-from-file before committing change.

When setting a key/value in a state with a value from a file, test the
value to see if it's already set.

+1 thx @joejulian


broken with 1.4.3

i tried to create a new container and the page looks very strange. going back to 1.3.7 works

The latest tag on Docker hub is version 1.3.8, which works OK. Otherwise, this image/repo is not actively maintained anymore.

The latest tag on Docker hub is version 1.3.8, which works OK. Otherwise, this image/repo is not actively maintained anymore.


feat: add testing to create-fuels

Closes #2744

Summary
Added vitest to the create fuels template with program type testing via launchTestNode. In code documentation to support #2461.
Checklist

[x] I added — tests to prove my changes
[x] I updated — all the necessary docs
[x] I reviewed — the entire PR myself, using the GitHub UI
[ ] I described — all breaking changes and the Migration Guide

I'd suggest syncing the changes from the template into apps/create-fuels-counter-guide.

@petertonysmith94 done 👍🏻


Question about logs & pows in base 10

Why when a Number is converted to a Fraction the library internally uses powers and logarithms in base 10? Shouldn't that potentially reduce floating-point precision? Why not base 2?

As long as you're in the MAX_SAFE_INTEGER range, the pow function has no numerical problems. For the log, there might be a problem, but as it is floor'ed, the problem vanishes. When you already use the bigfraction.js, shipped with fraction.js (and eventually becoming the new fraction.js), the problem with MAX_SAFE_INTEGER also vanishes. Do you have examples where this brings in problems?

Currently, the only possible problem might be performance, but that's only a concern when dealing with BigInts

Yep, performance is also a concern to me when it comes to switching to BigInt. The code is refactored maybe a million times to come up with a performant implementation of any aspect in the lib. When you see something to improve upon performance, you're more than welcome the bring these things in! :)

Speaking of that, I've heard that encapsulation/localization/snapshotting/whatever_name of global objects can increase performance because the engine doesn't need to change the behavior of a library if a piece of code replaces a global property. But I heard in other places that it actually slows down the execution because browsers aren't prepared for that.
What I mean is doing something like:
(function(abs){
  'use strict';
  console.log(abs(-1))
})(Math.abs)

Or maybe:
(function(){
'use strict';
const {abs} = Math;
console.log(abs(-1))
})()
Instead of this:
(function(){
  'use strict';
  console.log(Math.abs(-1))
})()

That's just an example, there's no difference there, but in a library with functions and methods that may be called repeatedly during runtime it makes a difference, because it becomes impossible for external code to modify the internals of the library.
Even MDN recommends it for polyfills. And yes, I'm aware library != polyfill, but the idea can still be worth trying.
I don't recommend doing it until some benchmarks are made on multiple runtime environments, to avoid unnecessary work.
I'm in the process of doing that transition for one of my libs, but I did it because I don't like side-effects lol, not just for speed.
It can also reduce the size of the source when minified, although that's not always the case for prototype methods:
(function(){
  'use strict';
  const {map} = Array.prototype;
  console.log( map.call([1, 2, 3], x => x + 1) )
})()

If you don't want to implement it, it's fine. Thank you for your time and reading

As far as I know, in some engines Math.abs is faster since they mark Math.abs as not changed (by a polyfill for example) and link the symbol "Math.abs" directly to improve performance. Thats why those symbols are not shortened in Fraction.js. Besides that repeating symbols over and over again also improves gzip compression ratio to speed up data transfer. The size of the actual code does not matter, since most connections support gzip. That's also why closure compiler optimizes for that. But those are micro optimizations, algorithmic optimizations might improve performance much more

Interesting, thank you for the info. And I too agree that algorithmic optimizations are much more important because those are usually the responsibility of devs instead of compilers, and because they are higher-level

I'll close the ticket for now, feel free to bring up other optimizations on Fraction.js :)


[🐛 Bug]: Allow NextResponse.rewrite() to proxy a different host in middleware.

Description
I am not sure this is even within the scope of next-on-pages or a fundamental Workers security choice. If it is in scope, I'm not sure if it is a feature request or a bug in how Pages interprets Next.js middleware.
For context, we have a marketing site, docs site, and web app. Each are Next.js projects, but it is required for multiple reasons that they be separate sites.
On Vercel we hosted the app and docs on subdomains. The marketing site, on the root domain, used Next.js middleware to rewrite requests. For example, /dashboard to app.site.com/app/dashboard and /docs/welcome to docs.site.com/docs/welcome. By setting app and docs as the basePath for those sites, this worked well and resulted in no routing conflicts.
On Cloudflare Pages, redirects and same-host rewrites seem to work the same, but rewrites that change the host (to another Pages project on a subdomain) fail silently at runtime. The middleware runs, and console.log() shows that the logic is functioning fine, but when the middleware returns the result of NextResponse.rewrite(), Pages returns a 404 rather than rewriting.
Because Vercel uses Workers as their 'edge runtime', I had assumed the behavior would match between Vercel and CF Pages.
Additional Information
I first submitted this bug but realized that the actual difference in behavior was more narrow and possibly not a bug at all.
Would you like to help?

[ ] Would you like to help implement this feature?

Hi @khalil-omer, thanks for the issue 🙂
Sorry it took me a bit to get around to it, I'm having a look at it now
In the meantime I have created this minimal reproduction of your issue, where we can see that the difference in behaviour Pages and Vercel have in regards to external rewrites:

https://next-on-pages-issue-551-reproduction.pages.dev/
https://next-on-pages-issue-551-reproduction.vercel.app/

(Please have a look and let me know if the reproduction is not correct 🙂)

@khalil-omer I have implemented a fix for the issue in #568, it sees to work as expected based on my testing
See the pages example mentioned above:

Before: https://97f409b8.next-on-pages-issue-551-reproduction.pages.dev/
and
After: https://c60293e7.next-on-pages-issue-551-reproduction.pages.dev/

If you want please try to PR's prerelease and let me know if it solves the issue on your end as well 🙂 🙏


Add New Project ( Quiz App )

Description of project :-
Quiz App
A quiz App which is created using Html , CSS and  Javascript in which there are questions which has four options and you have to mark the correct answer and at last you can check you score.
Expected Behaviour :-

It is a very Interesting and enjoyable game.
In this a person get knowledge about Html , CSS , Javascript and python basics.

Screenshots

Is is a front UI of Quiz game .

It is the final score page.

It is very Interesting game .
So , please assign it to me under SSOC ' 23 .

Okk Thanks @pranjay-poddar


Fix link to issue creation

Hey team,
I just spotted an unintended link that refers to the issue creation tab of the Porsche Open Source Platform. While I was here, I left a PR for you :)
Best regards,
Eduard
✅ Checks

[X] I have read and accept the Contributor License Agreement

thanks @itrich


program stuck  with css classes cached

VSCode Version: 1.5.2 ,1.4
OS Version: Win7 x64

Steps to Reproduce:

open a project include more .css or .less files

click in the bottom of vscode gui   ⚡  'css classes cached(click the cache again)'

the state show the message 'Looking for CSS classes on the workspace...'

waiting for a while, then the program stuck( IDE freezes) and alert 'The window is no longer responding '

ps:
open Windows Task Manager ,i can see many process named Code.exe

Reading your description it seems that the issue originates from an extension and not VS Code itself. Thereby I will close this issue and kindly ask you to report this issue against the extension repository.
Most extensions are on GitHub as well so it should be easy to file it against the extension. If you open an extension on the market place, click on the Support link to find out more.

Thanks again for your feedback!


Scrolling on a mobile devices

Problem: The sticky menu only shows up when the user stops scrolling.
On mobile devices the user is scrolling for a long distance so the sticky menu shows up really late and it looks like an error.
thx

Noticed on Google Chrome (Apple IOS).

+1

+1

This is a bug in ios related to position:fixed.
However sticky.js sets position:fixed dynamically.
If you use this way, you never avoid the issue.
In order to avoid this, you should use the way to be changed by scrollTop
for example in css:
header {position:fixed;width:100%;top:44px;}
.is-sticky header {top:0px;}

for example in js:
        $(window).on('scroll', () => {
            let scrollTop = $(window).scrollTop();

            if(scrollTop > 44) {
                $(document.body).addClass('is-sticky');
            } else {
                $(document.body).removeClass('is-sticky');
            }
        })

Adding transform: translate3d(0, 0, 0); to the sticky element fixed the issue for me.
It would be great to add s.stickyElement.css('transform', 'translate3d(0px, 0px, 0px)'); after 	s.stickyWrapper.css('height', s.stickyElement.outerHeight()) in the plugin.

god bless you, @xrochoa

@xrochoa bug still present for me on iPad ios10


chore(apps/prod/tiup-publisher): bump docker image to v20240228-86-gc1947d9

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


[docs]: add arch-based installation instructions, node 19 info.

added  instructions for installing mplayer on arch based systems.
informed potential users about incompatibility of better-sqlite3 with node.js version 19 and above.

Thanks for your contribution :heart:


Recirculation links in footer

It was a neat gimmick but I should just factor it out as a bl.ock for later use elsewhere.

Won't fix, closed by https://github.com/BloombergMedia/whatiscode/commit/992ca47ab5b83490d33250081e45458563709961. Cf. http://bl.ocks.org/tophtucker/593964bb8e6984b9f3af for demo.


feat(build): Utilise in-built 6to5 support in SystemJS

Instead of utilising the watch task to fire off the associated build-html and build-system tasks, utilise JSPM 0.11.0's built-in 6to5 transpiler support, which will do it automagically for us.

A note: the watch task in this pull still calls the build-system and build-html steps. Perhaps a refactor that only calls serve without any of the build steps would be better here?

Going to punt on this for now. May revisit it after the 6to5 integration stabilizes a bit.


Upgrade go-piv to v2.2.0

Support for AES management keys through https://github.com/go-piv/piv-go/releases/tag/v2.0.0
Support for YubiKey 5.7.x firmware capabilities through https://github.com/go-piv/piv-go/releases/tag/v2.2.0 thanks to @maraino 🎉

Fixes #114
Replaces #575

I had this change almost done, but I also added support for 3072/4096 bit RSA keys.


[nitter] wrong uploader

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running yt-dlp version 2021.04.03
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log
youtube-dl -J https://nitter.pussthecat.org/manential/status/1379007759811772419#m -v
[debug] Command-line config: ['-J', 'https://nitter.pussthecat.org/manential/status/1379007759811772419#m', '-v']
[debug] Loading archive file None
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] yt-dlp version 2021.04.03 (zip)
[debug] Plugin Extractors: ['SamplePlugin']
[debug] Python version 3.8.6 (CPython 64bit) - Linux-5.11.0-7612-generic-x86_64-with-glibc2.32
[debug] exe versions: ffmpeg 4.3.1, ffprobe 4.3.1, rtmpdump 2.4
[debug] Proxy map: {}
{"id": "1379007759811772419", "title": "franceinfo - #OnVeutLesNoms #chalencon #MangeonsLesRiches #restaurantclandestin ", "description": "#OnVeutLesNoms #chalencon #MangeonsLesRiches #restaurantclandestin ", "uploader": "franceinfo", "timestamp":<PHONE_NUMBER>, "uploader_id": "manential", "uploader_url": "https://nitter.pussthecat.org/manential", "view_count": 883, "like_count": 41, "repost_count": 21, "comment_count": 40, "formats": [{"format_id": "256", "format_index": null, "url": "https://nitter.pussthecat.org/video/5B85E8AC863C4/https%3A%2F%2Fvideo.twimg.com%2Fext_tw_video%2F1080208155735400450%2Fpu%2Fpl%2F320x180%2FrHf8P7IawktnLB4H.m3u8", "manifest_url": "https://nitter.pussthecat.org/video/F766930E58019/https%3A%2F%2Fvideo.twimg.com%2Fext_tw_video%2F1080208155735400450%2Fpu%2Fpl%2FFaGFrWPuuNDv7k2f.m3u8%3Ftag%3D6", "tbr": 256.0, "ext": "mp4", "fps": null, "protocol": "m3u8", "preference": null, "quality": null, "width": 320, "height": 180, "vcodec": "avc1.4d0015", "acodec": "mp4a.40.2", "format": "256 - 320x180", "http_headers": {"User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3649.1 Safari/537.36", "Accept-Charset": "ISO-8859-1,utf-8;q=0.7,*;q=0.7", "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8", "Accept-Encoding": "gzip, deflate", "Accept-Language": "en-us,en;q=0.5", "Cookie": "hlsPlayback=on"}}, {"format_id": "832", "format_index": null, "url": "https://nitter.pussthecat.org/video/622EA0CE58406/https%3A%2F%2Fvideo.twimg.com%2Fext_tw_video%2F1080208155735400450%2Fpu%2Fpl%2F640x360%2FYwCMb4BqAeCcCcLG.m3u8", "manifest_url": "https://nitter.pussthecat.org/video/F766930E58019/https%3A%2F%2Fvideo.twimg.com%2Fext_tw_video%2F1080208155735400450%2Fpu%2Fpl%2FFaGFrWPuuNDv7k2f.m3u8%3Ftag%3D6", "tbr": 832.0, "ext": "mp4", "fps": null, "protocol": "m3u8", "preference": null, "quality": null, "width": 640, "height": 360, "vcodec": "avc1.4d001f", "acodec": "mp4a.40.2", "format": "832 - 640x360", "http_headers": {"User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3649.1 Safari/537.36", "Accept-Charset": "ISO-8859-1,utf-8;q=0.7,*;q=0.7", "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8", "Accept-Encoding": "gzip, deflate", "Accept-Language": "en-us,en;q=0.5", "Cookie": "hlsPlayback=on"}}, {"format_id": "2176", "format_index": null, "url": "https://nitter.pussthecat.org/video/AAEEBA690FA91/https%3A%2F%2Fvideo.twimg.com%2Fext_tw_video%2F1080208155735400450%2Fpu%2Fpl%2F1280x720%2Ft-IdaHQfnAVVvmJw.m3u8", "manifest_url": "https://nitter.pussthecat.org/video/F766930E58019/https%3A%2F%2Fvideo.twimg.com%2Fext_tw_video%2F1080208155735400450%2Fpu%2Fpl%2FFaGFrWPuuNDv7k2f.m3u8%3Ftag%3D6", "tbr": 2176.0, "ext": "mp4", "fps": null, "protocol": "m3u8", "preference": null, "quality": null, "width": 1280, "height": 720, "vcodec": "avc1.640020", "acodec": "mp4a.40.2", "format": "2176 - 1280x720", "http_headers": {"User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3649.1 Safari/537.36", "Accept-Charset": "ISO-8859-1,utf-8;q=0.7,*;q=0.7", "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8", "Accept-Encoding": "gzip, deflate", "Accept-Language": "en-us,en;q=0.5", "Cookie": "hlsPlayback=on"}}], "thumbnails": [{"id": "large", "url": "https://nitter.pussthecat.orghttps://nitter.pussthecat.org/pic/ext_tw_video_thumb%2F1080208155735400450%2Fpu%2Fimg%2Fhm6uLLKu6P1Bm29A.jpg%3Alarge"}, {"id": "medium", "url": "https://nitter.pussthecat.orghttps://nitter.pussthecat.org/pic/ext_tw_video_thumb%2F1080208155735400450%2Fpu%2Fimg%2Fhm6uLLKu6P1Bm29A.jpg%3Amedium"}, {"id": "orig", "url": "https://nitter.pussthecat.orghttps://nitter.pussthecat.org/pic/ext_tw_video_thumb%2F1080208155735400450%2Fpu%2Fimg%2Fhm6uLLKu6P1Bm29A.jpg%3Aorig"}, {"id": "small", "url": "https://nitter.pussthecat.orghttps://nitter.pussthecat.org/pic/ext_tw_video_thumb%2F1080208155735400450%2Fpu%2Fimg%2Fhm6uLLKu6P1Bm29A.jpg%3Asmall"}, {"id": "thumb", "url": "https://nitter.pussthecat.orghttps://nitter.pussthecat.org/pic/ext_tw_video_thumb%2F1080208155735400450%2Fpu%2Fimg%2Fhm6uLLKu6P1Bm29A.jpg%3Athumb"}], "thumbnail": "https://nitter.pussthecat.orghttps://nitter.pussthecat.org/pic/ext_tw_video_thumb%2F1080208155735400450%2Fpu%2Fimg%2Fhm6uLLKu6P1Bm29A.jpg", "upload_date": "20210405", "extractor": "Nitter", "webpage_url": "https://nitter.pussthecat.org/manential/status/1379007759811772419#m", "webpage_url_basename": "1379007759811772419", "extractor_key": "Nitter", "playlist": null, "playlist_index": null, "display_id": "1379007759811772419", "requested_subtitles": null, "format_id": "2176", "format_index": null, "url": "https://nitter.pussthecat.org/video/AAEEBA690FA91/https%3A%2F%2Fvideo.twimg.com%2Fext_tw_video%2F1080208155735400450%2Fpu%2Fpl%2F1280x720%2Ft-IdaHQfnAVVvmJw.m3u8", "manifest_url": "https://nitter.pussthecat.org/video/F766930E58019/https%3A%2F%2Fvideo.twimg.com%2Fext_tw_video%2F1080208155735400450%2Fpu%2Fpl%2FFaGFrWPuuNDv7k2f.m3u8%3Ftag%3D6", "tbr": 2176.0, "ext": "mp4", "fps": null, "protocol": "m3u8", "preference": null, "quality": null, "width": 1280, "height": 720, "vcodec": "avc1.640020", "acodec": "mp4a.40.2", "format": "2176 - 1280x720", "http_headers": {"User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3649.1 Safari/537.36", "Accept-Charset": "ISO-8859-1,utf-8;q=0.7,*;q=0.7", "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8", "Accept-Encoding": "gzip, deflate", "Accept-Language": "en-us,en;q=0.5", "Cookie": "hlsPlayback=on"}}

Description
With this video, there is a bug: it says author is franceinfo while the real author is @manential
this is because it picks the first author with in fullname

also wrong like, view, repost, comment count for the same reason: it picks first occurrence but not the good one if the tweet in second in thread, timeline…

I think a similar issue would happen if there are two videos and we want the same.
I know we can look after id="m" or class="main-tweet" to get info about the tweet in url, not the first in thread


No coverage for cdylib crate called from C/C++ executables

Describe the bug
I have a project (https://github.com/Luthaf/rascaline/, coverage branch, PR with CI is https://github.com/Luthaf/rascaline/pull/126) which is built as both a rust crate (tarpaulin works great here, thanks a lot!) and exports a C API through a cdylib crate (rascaline-c-api). The C API is then used to create a C++ (and Python) interface for the code.
The C and C++ API are tested as a normal C/C++ project, using CMake to first build the rust code, and then build C and C++ test executables. I made sure to pass the right flags to rustc (setting RUSTFLAGS="-Cdebuginfo=2 --cfg=tarpaulin -Cinstrument-coverage -Clink-dead-code") when building the rust shared library.
The LLVM-based coverage instrumentation seems to work fine, generating .profraw files (I added a small patch in https://github.com/Luthaf/tarpaulin/commit/c67eb17884510e06fe7fca0ce61fc15805367537 since the profraw files are generated inside a different directory), but then no coverage is reported for the c-api crate at all. I added a handful of dbg! inside cargo-tarpaulin and llvm-profparser, and as far as I understand the code it seems that the profraw files contain at least some coverage data.
To Reproduce
git clone https://github.com/Luthaf/rascaline
cd rascaline
git checkout coverage
cargo tarpaulin --all-features --workspace --engine=llvm --test=c-api-tests --skip-clean --verbose

Here is the output I get: tarpaulin-rascaline.txt
I'm wondering if there could be an issue with file name parsing or something like this, since all the path shown are relative to the rascaline-c-api crate, and not the root of the repository/root of the cargo workspace.
If you have a couple of pointer/thinks I could look for, I'm happy to continue debugging this and send a PR once it is fixed.
Expected behavior
Using an instrumented shared library from C or C++ should be able to collect and report coverage on the C API implementation in rust.

Ok, I think I found the issue!
Using the actual test executable as an object file makes the code miss a lot of executed functions:
../llvm-profparser/target/release/cov show --instr-profile target/tarpaulin/profraws/tests/c-api/c_api_tests-468943f98a4ac45c_16259556021706463993_0-2295843.profraw --object target/debug/deps/c_api_tests-468943f98a4ac45c

But forcing the code to use the shared library as an object file, and it it able to assign the coverage to the expected functionc
../llvm-profparser/target/release/cov show --instr-profile target/tarpaulin/profraws/tests/c-api/c_api_tests-468943f98a4ac45c_16259556021706463993_0-2295843.profraw --object target/debug/librascaline.so | les
s

I guess this makes sense since the test executable never loads this shared library. Instead it execute ctest, which then executes the actual C/C++ executables, which load this shared library.

I guess the simplest thing to do might be to use the tools from llvm-profparser manually to extract coverage information, and use that in CI.
Do you think it would make sense to add an option to cargo tarpaulin to manually add other object files to the list that ends up loaded in the CoverageMapping? https://github.com/xd009642/tarpaulin/blob/57f2a75ea2b85334e6f0980ea5f0b84475c15698/src/statemachine/instrumented.rs#L134

sorry for late response, I do think that feature makes sense and I'd be happy to add it or accept a PR to do so :smile:

Okay I've implemented this and I can see it works (output at bottom). I'll just be making a PR and merging it once everything is shown to pass
All the increases in coverage show difference before me doing it and now with the new --objects arg:
|| Tested/Total Lines:
|| rascaline/src/calculator.rs: 73/113 +64.60%
|| rascaline/src/calculators/descriptors_by_systems.rs: 106/118 +89.83%
|| rascaline/src/calculators/dummy_calculator.rs: 38/41
|| rascaline/src/calculators/lode/radial_integral/gto.rs: 0/58
|| rascaline/src/calculators/lode/radial_integral/mod.rs: 0/33
|| rascaline/src/calculators/lode/radial_integral/spline.rs: 0/9
|| rascaline/src/calculators/lode/spherical_expansion.rs: 0/384
|| rascaline/src/calculators/neighbor_list.rs: 0/211
|| rascaline/src/calculators/radial_basis/gto.rs: 37/39
|| rascaline/src/calculators/radial_basis/mod.rs: 2/8
|| rascaline/src/calculators/soap/cutoff.rs: 31/59
|| rascaline/src/calculators/soap/power_spectrum.rs: 213/278 +76.62%
|| rascaline/src/calculators/soap/radial_integral/gto.rs: 24/30
|| rascaline/src/calculators/soap/radial_integral/mod.rs: 28/29
|| rascaline/src/calculators/soap/radial_integral/spline.rs: 7/7
|| rascaline/src/calculators/soap/radial_spectrum.rs: 0/80
|| rascaline/src/calculators/soap/spherical_expansion.rs: 261/326
|| rascaline/src/calculators/sorted_distances.rs: 0/41
|| rascaline/src/errors.rs: 7/34
|| rascaline/src/labels/keys.rs: 52/62
|| rascaline/src/labels/samples/atom_centered.rs: 58/72
|| rascaline/src/labels/samples/long_range.rs: 0/27
|| rascaline/src/labels/samples/mod.rs: 5/6
|| rascaline/src/math/double_regularized_1f1.rs: 57/65 +87.69%
|| rascaline/src/math/eigen.rs: 162/167
|| rascaline/src/math/erf.rs: 0/98
|| rascaline/src/math/exp.rs: 0/40
|| rascaline/src/math/gamma.rs: 19/165
|| rascaline/src/math/hyp1f1.rs: 91/142
|| rascaline/src/math/hyp2f1.rs: 0/170
|| rascaline/src/math/k_vectors.rs: 0/42
|| rascaline/src/math/spherical_harmonics.rs: 100/127 +78.74%
|| rascaline/src/math/splines.rs: 101/115 +87.83%
|| rascaline/src/systems/cell.rs: 23/107
|| rascaline/src/systems/chemfiles.rs: 21/25
|| rascaline/src/systems/neighbors.rs: 117/133
|| rascaline/src/systems/simple_system.rs: 25/32
|| rascaline/src/types/matrix.rs: 36/66 +54.55%
|| rascaline/src/types/mod.rs: 8/36
|| rascaline/src/types/vectors.rs: 12/37 +32.43%
|| rascaline-c-api/src/calculator.rs: 65/72 +90.28%
|| rascaline-c-api/src/logging.rs: 17/19
|| rascaline-c-api/src/profiling.rs: 19/23
|| rascaline-c-api/src/status.rs: 29/37 +78.38%
|| rascaline-c-api/src/system.rs: 121/172 +70.35%
|| rascaline-c-api/src/utils.rs: 10/10
|| rascaline-c-api/tests/utils/mod.rs: 26/28 +1.19%
||
50.11% coverage, 2001/3993 lines covered, +44.34% change in coverage

Thanks a lot, that's great!!

Any reason for the call to canonicalize here: https://github.com/xd009642/tarpaulin/blob/f92c1bb02c43a3415efd33676e220bbcdc1e4f25/src/config/parse.rs#L39
It fails on CI, since the shared library does not exist yet when the Config is created from the ArgMatches.

Generally just to get an absolute path, because they're needed in various replaces (reports, coverage collection) and relative paths are pernicious with workspaces and hard to keep track of for normalising at the end. However, that canonicalise is for the --manifest-path arg. I guess you meant this one https://github.com/xd009642/tarpaulin/blob/f92c1bb02c43a3415efd33676e220bbcdc1e4f25/src/config/parse.rs#L129
It might be possible to remove it some how, or maybe moving the join to the end after the unwrap would fix it.

@Luthaf if you try out the fix/canonicalise-object branch it might fix the issue for me (let me know please :pray: )

Thank you so much for the quick fix! I needed another small change, which I put in #1225. With that, I'm able to collect coverage for my project.

Cool I'll look at the PR and merge etc and look to get a release out once I'm done with work today :+1:

And this is released now


Actor revalidation

Description
Actor validation made required and also improved.
Checklist

[x] I have performed a self-review of the changes
[x] Run lint
[ ] Check all the tests + files for artifacts
[x] Revise test actor (and maybe test pipeline)
[x] Add docs  to functions

To Consider

Add tests (if functionality is changed)
Update API reference / tutorials / guides
Update CONTRIBUTING.md (if devel workflow is changed)
Update .ignore files, scripts (such as lint), distribution manifest (if files are added/deleted)
Search for references to changed entities in the codebase

That's fair.
We can validate function signatures at Node level and transition labels at Script level.
(i.e. validators validate_response_signature, validate_label_signature, validate_condition_signature, validate_processing_signatures in Node and check_labels_exist_in_script for Script)
Node validation is easier to understand and debug: you define a function that checks a specific aspect of a node; and pydantic handles all the logic of validating it on its own -- we don't have to write our own loops.
This allows us to validate Node arguments even if they are not defined at the second level of the script. (which is not a common use case tbh)

@pseusys
It doesn't seem like test_script is correct. There is indeed no "node" node from the GLOBAL node's perspective.
I assume that this test is supposed to test script creation.
Could you replace test_script (and remove the script_test function) with two tests (or one parametrized) that attempt to create a script object from TOY_SCRIPT and MULTIFLOW_SCRIPT objects of dff.utils.testing.toy_script?

Yep, seems good
Will likely approve soon (potentially with minor corrections).

Wait, no.
You forgot to add tests:
https://github.com/deeppavlov/dialog_flow_framework/pull/289#discussion_r1542659199

In case there was a misunderstanding:
I approve the current version of actor validation and would like to merge this PR once the tests are added.


Update telegram to 0.9.17

Thank you for the contribution. It needed some fixes, so they were made in c9c3122b1727723649b6efff159da6a43f839aa1. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Notes on plots

⚠PyPlot: should be ok, may have to explicitly suppress STDOUT; probably best to let Conda do the whole thing and add .Julia/conda to the cache though annoying that it indicates it has to install matplotlib every time...
⚠ CairoMakie: seems ok too, also needs to suppress code output (see https://tlienart.github.io/xranklin-cairomakie/)
✅ Plots with GR backend works, no stdout, require installing QT-related stuff, takes 22s which is fine.
Note: could try a heuristic that if the result is showable as image, stdout has a different class added to it which can then be suppressed everywhere. (https://github.com/tlienart/xranklin-plots/blob/a43cab396abb970040c19e36796559dded5af75f/.github/workflows/deploy.yml#L38-L42)
❓ PGFPlotsX , might try to do the kind of gymnastics done for TikzCD which installs a bunch of stuff see https://github.com/tlienart/Franklin.jl/blob/9705c13d39c69ca92d7b37e7453f2394d6e21513/.github/workflows/deploy.yml#L20-L23
❓ PlotlyJS should just work since it's a JS lib (...)

Can probably stop there and indicate how people can do the rest...
NOTE;

might be able to spot that something is SVG/PNG showable, assume it's an image, and add a class to STDOUT so that it can be globally suppressed.

all those are now ok; could add GLMakie, and other stuff like Gaston etc but maybe for future


Add a blog post for KubeCon NA 2024 in SLC

This adds a small blog post for what to see at KubeCon + CloudNativeCon NA 2024 in regards to kcp.

/cc @mjudeikis @sttts @scheeles

(I can also add someone else as author so my face isn't the only one on all those posts on the blog, feel free to volunteer yourself :D)

/hold
until publishing date of the post.

Looks great!

/lgtm
/approve

/lgtm
/approve

/hold cancel


Update rel from main

@qarni Assuming this branch is resolving merge conflicts from merging main into rel, should be good. Approving.


PAL-657 added javadoc to compile phase (warning only)

Need to validate -Pquick is working correctly


Provide integration tests to be run as part of CI

We need integration tests to verify the CC works.
Assumptions:

kubernetes cluster is already deployed

The tests should consists of:

building cc
running cc with various settings (pod without admissions, pod with admissions, ...)
running cc as a pod in the cluster
...

Magic command to run:
make integration-test

Points 2 and 3 to me seem like end to end tests in cluster capacity context, isn't it? Any tests that require spawning a cluster would be end-to-end? Or what would be a end-to-end test?


GSAP Text Animations on Homepage

Objective : I propose enhancing the homepage with text animations using GSAP (GreenSock Animation Platform) to improve user engagement and make the interface more dynamic.
Details:

Animate heading and subheading text on page load.
Include subtle fade-in or slide-in animations for improved visual appeal.
Ensure animations are responsive and do not affect performance on lower-end devices.
Will add comments to code to ensure clarity for future contributors.

Please assign this issue to me, as I would love to work on it. I am also a contributor in GGSOCExt 24.
Looking forward to your feedback!

thank you for raising the issue. i'll assign this issue after you complete the issue #11

Hello Scribbie-Notes Team,
My name is Ishan, and I am a 4th-year BTech student with experience in full-stack development. I would love to contribute to your project. Here is my LinkedIn profile: https://www.linkedin.com/in/ishanguptaofficial/.
Could you please assign me to this issue? Looking forward to your response.
Best regards,
Ishan

Dear @yashmandi,
The work on issue #11 has been completed. As per our earlier discussion, could you please assign it to me? It would be great to have this in my task list as you promised.


resnet50 btf file drive link - unavailable for download

Per the instructions here:
https://github.com/tensorflow/runtime/blob/master/documents/resnet.md
the resnet50_graph_inference_tensors.btf is to be downloaded via the Google drive link. However, the file at the link isn't available for download. Info on the file shows:
Download permission
Viewers cannot download

Sorry for the inconvenience, we updated the link to the BTF file. Please try again.


joining main with varan

-Enhancement of the main & remove all the apps from it.
-Integrate the first two modules correctly into main & ensure they run well in the terminal as long within Varan API.
-Correct the url for module 2 & return the response valid in json format.
Sanitising the user input in the API (remove the white space and lower case the input)

PR #69

This has conflicts outstanding do we still need to merge this pull request?


Player control buttons blank in light mode of Obsidian

Hi
The title summarizes, it. In dark mode it is ok, but in light mode, the buttons of the player are all white/blank. I am using 2.11.1 in last Obsidian version 1.1.9. Here you can see an screenshot:

I have tried and disabled all plugins, restarted Obsidian, but no change. It is always like that. Could it be fixed, please?
Tx

Apologies for the long period of inactivity on this project. I wanted to inform you that the plugin has been fully rewritten for the v3 release. I'm currently preparing for the beta release. Once it's available, if you encounter similar issues, please don't hesitate to open a new issue. Thank you for your understanding.


Blank link and white space in makecode's blog

On the blog, there is a blank clickable link.
Also I think that the "Published by Microsoft" should stay at the bottom of the screen and the side bar must  be dark until the bottom of the page.

Want to look into this @veler. I can't repro the clickable link, but I agree it should be black all the way. I'm okay with the published by Microsoft being at the bottom of the sidebar list (wherever that ends). There's already a Powered by Microsoft, and Microsoft at the bottom of the page anyway.

Alright I will take a look


fmt: prints "" for %#v when type implements GoStringer

Go version
1.22, 1.23, dev branch
Output of go env in your module/workspace:
N/A

What did you do?
https://go.dev/play/p/2eq4QlnXX9e
What did you see happen?
%#v outputs <nil> which is not correct Go syntax
What did you expect to see?
I expected to get (*time.Time)(nil) instead of <nil>.
It looks like this happens because we have a Time.GoString function with non-pointer receiver; I verified that using the A and B types.
One library that uses %#v and expects it to produce correct Go syntax is github.com/gogo/protobuf

I think this is working as intended?

The fmt.GoStringer interface is called when the formatting verb is %#v
The method set for *time.Time includes that of time.Time
But it's a nil pointer dereference panic to dereference the pointer to call the GoString
fmt normalizes all nil pointer derefs to <nil>

It's wrong for the library to expect that %#v will not be overwritten by custom GoString implementations.

%#v is documented as "a Go-syntax representation of the value". <nil> is not that. The caller isn't doing anything wrong by passing a nil value.

Can't fmt use reflection to detect when GoString has a non-pointer receiver and the value is a nil pointer?

That's just the general rule, more specifically, lower down:

Except when printed using the verbs %T and %p, special formatting considerations apply for operands that implement certain interfaces. In order of application:
...
3. If the %v verb is used with the # flag (%#v) and the operand implements the GoStringer interface, that will be invoked.

and

Format errors
...
If the panic is caused by a nil receiver to an Error or String method, however, the output is the undecorated string, "".

So <nil> is consistent with the rest of fmt.

It is debatable if the nil receiver wording applies here since the method has a non-pointer receiver. It is the fmt code that chooses to dereferenciate the pointer, it is not the function panicing.
In any case, wording aside, why do you think this outcome is preferable? IMO the fact that by adding a correct GoString method you can break existing usages of nil seems very problematic.

It's not clear that we can change this behavior at this point.  It's been this way for many releases.  Changing it now seems likely to do more harm than good.

I see. Could we at least call this out specifically in the documentation? Maybe: "If the panic is caused by a nil receiver to an Error, String, or GoString method (including the case where the method is defined on a non-pointer receiver), however, the output is the undecorated string"


The ES5 code from Traceur 0.0.110 doesn't work in web browser.

I upgrade the Google Traceur to the latest 0.0.110. However, the compilation-output ES5 JavaScript program can't work in the web browser.

The transpiling command for Traceur:

node_modules/.bin/traceur
--script=www/js/widget-nest-thermostat.bbjs
--out=www/js/widget-nest-thermostat.js
--require=true
--arrow-functions=true --block-binding=true --classes=true
--computed-property-names=true --default-parameters=true
--destructuring=true --for-of=true --generators=true --numeric-literals=true
--property-methods=true --property-name-shorthand=true --rest-parameters=true
--spread=true --symbols=true --template-literals=true --unicode-escape-sequences=true
--unicode-expressions=true --proper-tail-calls=true --annotations=true
--array-comprehension=true --async-functions=true --async-generators=true
--exponentiation=true --export-from-extended=true --for-on=true
--generator-comprehension=true --member-variables=true --spread-properties=true
--types=true --experimental --source-maps=file

The traceur runtime has been introduced in the HTML page.

The file "traceur-runtime.js" is copied from the folder "node_modules/traceur/bin".

The Error is thrown from the web browser, when I visit the web page.
TypeError: Cannot read property 'default' of undefined

By scanning the compilatio-output ES5 JS program, I discovered the error point is
var $__initTailRecursiveFunction = $traceurRuntime.getModule($traceurRuntime.normalizeModuleName("traceur/dist/commonjs/runtime/modules/initTailRecursiveFunction.js", "widget-nest-thermostat.bbjs")).default;
That's to say, $traceurRuntime.getModule($traceurRuntime.normalizeModuleName("traceur/dist/commonjs/runtime/modules/initTailRecursiveFunction.js", "widget-nest-thermostat.bbjs")) returns null.
Memo: in my project, ES6 JS file is suffixed with ".bbjs". After they are compiled, their corresponding ES5 file is ".js".

It's worth saying that, as for the<EMAIL_ADDRESS>the compilation-output ES5 JavaScript program works in the web browser.
Is it a regression bug?

Please look in to your output for the $traceurRuntime.registerModule() with the name from your initTailRecursiveFunction.js.  Also compare these with the the old and new versions of the compiler.
The compiler generates $traceurRuntime.registerModule() to store the modules and $traceurRuntime.getModule() to retrieve them.  The register call must be missing or out of order.
Once you have a hint perhaps you can create a small test case. We did make some changes in the runtime.

Given my investigation, the<EMAIL_ADDRESS>ES5 code is very different from<EMAIL_ADDRESS>The below JS statement is inexistent in the<EMAIL_ADDRESS>compilation result.
var $__continuation = $traceurRuntime.getModule($traceurRuntime.normalizeModuleName("traceur/dist/commonjs/runtime/modules/continuation.js", "widget-nest-thermostat.bbjs")).default;
However, in the web browser, the above JS statement brings about a NullPointer error, because $traceurRuntime.getModule($traceurRuntime.normalizeModuleName("traceur/dist/commonjs/runtime/modules/continuation.js", "widget-nest-thermostat.bbjs")) return null.

Looks like continuation.js did not make it into the commonjs runtime.

I see dist/commonjs/runtime/modules/continuation.js in the npm release.
I suspect there is cyclic dependency with the require that causes this to be undefined. I'll try to create a minimal test case.

Hey folks, thanks for the continued work on the project! 💃
Just wanted to leave a note here for anyone else currently experiencing issues with this - turning off properTailCalls didn't seem to do the trick for me, but turning off importRuntime did. ¯\(ツ)/¯
ref: https://github.com/richgilbank/Scratch-JS/commit/d0f2e7b51e6f207dc3940f756491500ae320f81a#diff-402dfb8c015ea27de73101a0b8d9fc32R17

Cloud you please teach me what the feature "importRuntime" takes on during the transformation from ES 6 to ES 5?