UnityTimeOutException: The Unity environment took too long to respond.

I am using obstacle tower 1.2 binary and mlagents-envs-0.6.2, under a Linux environment.
In [1]: from obstacle_tower_env import ObstacleTowerEnv

In [2]: ObstacleTowerEnv.ALLOWED_VERSIONS
Out[2]: ['1', '1.1', '1.2']

In [3]: env = ObstacleTowerEnv('./ObstacleTower/obstacletower.x86_64', retro=False, realtime_mode=False)
Found path: ./ObstacleTower/obstacletower.x86_64
Mono path[0] = './ObstacleTower/obstacletower_Data/Managed'
Mono config path = './ObstacleTower/obstacletower_Data/MonoBleedingEdge/etc'
Preloaded 'ScreenSelector.so'
Preloaded 'libgrpc_csharp_ext.x64.so'
Logging to $HOME/.config/unity3d/Unity Technologies/ObstacleTower/Player.log

I get the following error:
UnityTimeOutException: The Unity environment took too long to respond. Make sure that :
         The environment does not need user interaction to launch
         The Academy and the External Brain(s) are attached to objects in the Scene
         The environment and the Python interface have compatible versions.

I am correctly using the obstacletower 1.2 binary, and it was a head-less mode.
2f6c7ca247c98cdb27776bfde28037a36d1280a4  obstacletower.x86_64

Related Issue?
I checked #44 #46 but they were about Windows or before 1.2 release. I had tried out v1.0 long time ago, and I remember there was no problem at that time.
Executing the binary from the console (with strace) -- nothing follows after some system calls.
... (many lines) ...

munmap(0x7f6f1ac54000, 92100)           = 0
munmap(0x7f6ee85f5000, 2109768)         = 0
munmap(0x7f6ee83ef000, 2118024)         = 0
open("./ObstacleTower/obstacletower_Data/Resources/UnityPlayer.png", O_RDONLY) = 4
fstat(4, {st_mode=S_IFREG|0644, st_size=24711, ...}) = 0
brk(0x415c000)                          = 0x415c000
read(4, "\211PNG\r\n\32\n\0\0\0\rIHDR\0\0\0\200\0\0\0\200\10\2\0\0\0L\\\366"..., 1048576) = 24711

What should I check for this issue?

Hi @wookayin
Are you running this in a headless environment? Or does your linux system have a display?

Hi @awjuliani
I also have the same problem.
It works fine on the local machine. But when I run it on a server I get the same issue.
Do I understand correctly that I need to set up Xvfb server first?
I did it and set option docker_training=True and False, but still have this problem.

It happened in a head-less mode, but previously I was able to run the environments in head-less mode if I remember correctly, and shouldn't it be the case?. I will try having display servers running.

Hi @wookayin and @GrachevArtem
In order to run Obstacle Tower on a headless Linux machine you will either need to run xvfb on that machine (and you can use our obstacle-tower-challenge repo/docker image to do that https://github.com/Unity-Technologies/obstacle-tower-challenge), or run a virtual x-server. We have instructions for installing a virtual x server as part of our GCP/Dopamine guide: https://github.com/Unity-Technologies/obstacle-tower-env/blob/master/examples/gcp_training.md

Hi I have the same issue here, I'm trying to run it in docker and getting same exception both with xvfb and with virtualgl.
(base) root@dl0:~/root/obstacle-tower-challenge# xvfb-run python run.py
Found path: /root/root/obstacle-tower-challenge/./ObstacleTower/obstacletower.x86_64
Mono path[0] = '/root/root/obstacle-tower-challenge/./ObstacleTower/obstacletower_Data/Managed'
Mono config path = '/root/root/obstacle-tower-challenge/./ObstacleTower/obstacletower_Data/MonoBleedingEdge/etc'
Preloaded 'ScreenSelector.so'
Preloaded 'libgrpc_csharp_ext.x64.so'
Display 0 'screen': 640x480 (primary device).
Logging to /root/.config/unity3d/Unity Technologies/ObstacleTower/Player.log
Traceback (most recent call last):
  File "run.py", line 26, in <module>
    env = ObstacleTowerEnv(args.environment_filename, docker_training=args.docker_training)
  File "/opt/conda/lib/python3.7/site-packages/obstacle_tower_env.py", line 45, in __init__
    timeout_wait=timeout_wait)
  File "/opt/conda/lib/python3.7/site-packages/mlagents_envs/environment.py", line 69, in __init__
    aca_params = self.send_academy_parameters(rl_init_parameters_in)
  File "/opt/conda/lib/python3.7/site-packages/mlagents_envs/environment.py", line 491, in send_academy_parameters
    return self.communicator.initialize(inputs).rl_initialization_output
  File "/opt/conda/lib/python3.7/site-packages/mlagents_envs/rpc_communicator.py", line 80, in initialize
    "The Unity environment took too long to respond. Make sure that :\n"
mlagents_envs.exception.UnityTimeOutException: The Unity environment took too long to respond. Make sure that :
	 The environment does not need user interaction to launch
	 The Academy and the External Brain(s) are attached to objects in the Scene
	 The environment and the Python interface have compatible versions.

Hi @twoleggedeye
One potential reason for this may be that it takes longer than 30 seconds for the Unity environment to initialize and handshake with python. Our python api comes with a timeout_wait parameter which can be increased. Could try try with a larger value for this variable, and let us know if that resolves the issue?

@awjuliani I increased timeout_wait and now able to run it with xvfb, thank you! However it doesn't run with virtualgl. Seems it tries to initialize audio driver which is obviously missing on the headless server.
Found path: /root/root/obstacle-tower-challenge/./ObstacleTower/obstacletower.x86_64
Mono path[0] = '/root/root/obstacle-tower-challenge/./ObstacleTower/obstacletower_Data/Managed'
Mono config path = '/root/root/obstacle-tower-challenge/./ObstacleTower/obstacletower_Data/MonoBleedingEdge/etc'
Preloaded 'ScreenSelector.so'
Preloaded 'libgrpc_csharp_ext.x64.so'
Display 0 'TurboVNC': 1240x900 (primary device).
Logging to /root/.config/unity3d/Unity Technologies/ObstacleTower/Player.log
ALSA lib confmisc.c:768:(parse_card) cannot find card '0'
ALSA lib conf.c:4292:(_snd_config_evaluate) function snd_func_card_driver returned error: No such file or directory
ALSA lib confmisc.c:392:(snd_func_concat) error evaluating strings
ALSA lib conf.c:4292:(_snd_config_evaluate) function snd_func_concat returned error: No such file or directory
ALSA lib confmisc.c:1251:(snd_func_refer) error evaluating name
ALSA lib conf.c:4292:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:4771:(snd_config_expand) Evaluate error: No such file or directory
ALSA lib pcm.c:2266:(snd_pcm_open_noupdate) Unknown PCM default
ALSA lib confmisc.c:768:(parse_card) cannot find card '0'
ALSA lib conf.c:4292:(_snd_config_evaluate) function snd_func_card_driver returned error: No such file or directory
ALSA lib confmisc.c:392:(snd_func_concat) error evaluating strings
ALSA lib conf.c:4292:(_snd_config_evaluate) function snd_func_concat returned error: No such file or directory
ALSA lib confmisc.c:1251:(snd_func_refer) error evaluating name
ALSA lib conf.c:4292:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:4771:(snd_config_expand) Evaluate error: No such file or directory
ALSA lib pcm.c:2266:(snd_pcm_open_noupdate) Unknown PCM default
X Error of failed request:  BadMatch (invalid parameter attributes)
  Major opcode of failed request:  72 (X_PutImage)
  Serial number of failed request:  112
  Current serial number in output stream:  113

Hi @twoleggedeye,
Unfortunately we don't support virtualGL right now with Unity / Obstacle Tower. If you'd like to use a GPU rendering solution, we recommend running a virtual x-server. https://github.com/Unity-Technologies/obstacle-tower-env/blob/master/examples/gcp_training.md#set-up-xserver

Ok, thanks!


fs: support VFS-based named event group

Summary
This extension for the event group, to establish a connection between named event groups and a task. The task may reference the event group associated with a VFS-based name. Then the event group can be used in subsequent calls to nxevent_wait() or nxevent_post().
This is an internal OS interface and should not be used by applications.
Impact
none, add OS interface
Testing
vela

CI failed. Waiting for a dependent patch .


openSUSE \ SLES 12 Package is missing

Hey,
I have seen that there is a RPM package and was wondering if it was tested on them?
Was the RPM built for usage on CentOS\RH\Fedora or also for openSUSE\SLES?

It works on openSUSE, but you can't download the package with zypper

We don't regularly test every release on every flavor of Linux, there are too many of them to do that for the resources we have available.
Also, we're trying to control the amount of Issues that we have to triage. For general questions, the Atom message board is where you'll probably be able to find answers faster.


Publish rate limiting for topic

Motivation
Producers and consumers can interact with broker on high volumes of data. This can monopolize broker resources and cause network saturation, which can have adverse effects on other topics owned by that broker. Throttling based on resource consumption can protect against these issues and it can play an important role for large multi-tenant clusters where a small set of clients using high volumes of data can easily degrade performance of other clients.
State of the art
Right now, broker has mechanism to restrict the number of concurrent publish messages on each connection. So, broker serves only certain (1K) number of concurrent publish per connection.
However, client can create multiple producers for the topic by creating separate connection for each producer. So, client can aggregately publish large number of messages on the topic using multiple connections and it can have adverse impacts on broker n/w bandwidth, CPU and main-memory which can ultimately affect other topics/tenants on the same broker.
Publish Rate limiting  on the topic
Publish rate-limiting on a topic should be able to throttle publish-rate on a topic. So, once topic reaches per-second publish rate-limiting quota, broker will not read messages from the topic until quota will be refreshed at every one second.
So, broker can maintain a count of the current number of published messages and once this counter reaches to rate-limiting threshold, broker can start throttling new messages until counter will be reset at the next second.
There are multiple mechanism in which broker can throttle and restrict new incoming messages  on the topic.
One of the approaches: Once topic exceeded publish quota, broker will stop reading from all the connections on which any producer of that topic which has exceeded publish-quota is connected. And broker will start reading again once publish quota will be refreshed (at next second).
Disadvantage:
- Client can use one connection to serve multiple producers from different topics. So, client  can publish messages on all different topics using the same shared connection. So, broker will not read messages from other topics which are sharing the same connection with the topic that has exceeded the publish-quota. So, client will face high publish latency for other topics which are sharing the same connection but has not exceeded publish quota.
Alternative approach

Once topic exceeded publish quota, broker will fail incoming publish and send publish-failure to client.
Disadvantage:

In this scenario, client will receive publish-failure once it reaches to publish-quota so, client producer will close the connection and try to resend the same messages again which will unnecessarily increases the traffic and wastes n/w bandwidth at client and server side.
also, it also causes client to close and recreate connection and it will increase the number of lookup requests at broker which we want to avoid.

Once topic exceeded publish quota, broker will fail incoming publish and send publish-failure to client. Client can handle this publish-failure and resend those failed messages after certain backoff without closing connection.
Disadvantage:

it requires client change and client needs to upgrade library
it avoids reconnection and extra lookup but it still requires client to re-send the same messages again which could waste the n/w bandwidth at client and server side.

Broker changes
Throttling limit per topic
After configuring publish rate-limit, each topic can publish with in that rate-limit . If a topic is configured with rate-limiting of 10 messages per second then broker serves only 10 publish messages per second for that topic.
Throttling limit: Message-rate Vs Byte-rate
Throttling limit can be defined in two ways:
Message-rate: Publish can be throttled by total number of messages per second. If message-rate is configured 100/second then broker serves only 100 messages publish per second.
Byte-rate: Publish can be throttled by total number of bytes per duration. If byte-rate is configured 1MB/second then broker serves publish with total size of 1 MB per second.
If both the limits are configured for the topic then broker applies both the limit and throttles based on whichever threshold reaches first.
Throttling configuration
User can configure publish-throttling using pulsar-admin api. Broker stores this namespace configuration as part of namespace policies. Following configuration forces broker to serve only 1000 messages publish per second for each topic under the given namespace.
pulsar-admin namespaces set-publish-rate <property/cluster/namespace> -m 1000

Following configuration forces broker to dispatch only 1000 bytes per second for each topic under the given namespace.
pulsar-admin namespaces set-publish-rate <property/cluster/namespace> -b 1000

By default, the value of both the configurations is 0 which disables publish-throttling for the namespace.

This has been fixed by #3986


Make function TLS configuration independent of function worker TLS configuration

Make function TLS configuration independent of function worker TLS configuration

I will fix this issue


Initial eleventy build - WIP

For reference, we gave some feedback on these changes in Basecamp:
https://3.basecamp.com/3091560/buckets/10590409/messages/5831546187#__recording_5870938075
Ideally this would be worked on soon so that the changes can be merged in.
cc @simonrjones

Just to update this PR. I have created some tasks in a GitHub projects board that need to be actioned before this PR can be reviewed again:
https://github.com/orgs/studio24/projects/6/views/1
Jamie has some time to look at it today and tomorrow. But we will likely need additional time to get this completed and merged in. Perhaps this can be done via Friday afternoon time, but if not we will need extra scheduled time for this.


Manual summary breaks Markdown formatting

Manual summary breaks Markdown formatting by inserting extra newlines. This causes the Summary and rest of content to be placed in separate
 elements even if they should not have been. This treats the  separator differently from any other  comment and does not appear to be documented.
What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.118.2-da7983ac4b94d97d776d7c2405040de97e95c03d+extended darwin/amd64 BuildDate=2023-08-31T11:23:51Z VendorInfo=brew

Does this issue reproduce with the latest release?
Yes

For example...
markdown
a  b c

.Summary is rendered to:

a

.Content is rendered to:

a

b c


fix: LIVE-7659 fix non rendering lotties in CLS flows

📝 Description
For some reason (sadly I dont understand why) we lost the lottie animation rendering on the CLS flow.
This patches that bug but we'd still need to understand the underlying regression cause.
We also need to check that it doesn't break the firmware update restoration flow, I have no way of testing this currently.
❓ Context

Impacted projects: ledger-live-mobile
Linked resource(s): LIVE-7659

✅ Checklist

[ ] Test coverage
[ ] Atomic delivery
[x] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

converting to draft because I still didn't achieve pixel perfection 😤


Expected behavior of first() on a model

If I am searching for a record in a DB and I call first() like so...
 return $query->where('email', $email)
            ->orWhere('username', $email)
            ->orWhere('home_email', $email)
            ->orWhere('other_email', $email)
            ->first();

... and there is not a record, I am returned back in instance of the Query builder opposed to null. I looked at the API and it does say that it can return null but that is not what I am getting back: Laravel API
Why am I getting the builder instance returned? It seems like I should get null when there are no records so that I can easily check to see if something was returned.
My question is, why is it created this way? What are the cons to returning null instead of a builder instance? I may be missing something, but it seems like that would be a better idea.

I can't replicate this issue, do you have a demo repo I could check out?

I tried the same and I'm getting null. Not sure what you have missed

@dericcain what version of Laravel are you on?

Sorry but please ask on the forums, first() seems to work fine for all of us so it's most likely that you have something different in your setup, please include more details on the forums so that you can get help faster.


When support devcontainer stop/down features

When support devcontainer stop/down features, any plain yet?

Let's use https://github.com/devcontainers/cli/issues/386 for tracking!
Given the number of upvptes ^, I'd open this topic for discussion with other maintainers.
// cc @bamurtaugh


hack/check-links.sh: improve check invocation

Description of the change:

hack/check-links.sh: ignore github.com/operator-framework/operator-sdk/edit/master/
hack/*: move test_lib.sh functions into common.sh and import common.sh where applicable

Motivation for the change: link checker fails to find newly added markdown files
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

/cc @asmacdo @camilamacedo86


kn-quickstart is blocked by MacOS

Hi Team.
I'm trying to setup knative under MacOS  using quickstart plugin and this manual https://knative.dev/docs/getting-started/quickstart-install/#install-the-knative-quickstart-plugin
Unfortunately, when I try to run next command

kn quickstart --help

MacOS blocks execution, please see screenshot below.
Any ideas how this issue can be fixed.
Thanks in advance.

What version of the quickstart plugin are you using? And what version of MacOS?
I don't use a Mac, but based on a quick web search it looks like you may need to authorize the app in the System Preferences menu.

Hi @VladyslavKurmaz. When I have "App store and identified developers" selected in my security settings, I am given the option to open the file or not.

Can you try this to see if it helps you out?


Multiplatform docker build broken in CI

Since today the docker CI build is failing.
--------------------
   7 |
   8 |     # We only need a small runtime for this step, but make sure glibc is installed
   9 | >>> FROM frolvlad/alpine-glibc as runtime
  10 |     COPY --chmod=0755 --from=builder /app/target/release/atomic-server /atomic-server-bin
  11 |     ENV ATOMIC_STORE_PATH="/atomic-storage/db"
--------------------
ERROR: failed to solve: frolvlad/alpine-glibc: no match for platform in manifest sha256:44a54734fa33b549162845a9839fa4c745c5132a1e59f5d871c681050a9dd6bb: not found
Error: buildx failed with: ERROR: failed to solve: frolvlad/alpine-glibc: no match for platform in manifest sha256:44a54734fa33b549162845a9839fa4c745c5132a1e59f5d871c681050a9dd6bb: not found

I'm pretty confident that it's not because of a change in our code.

Setting the image to the prev version, frolvlad/alpine-glibc:alpine-3.16_glibc-2.34, worked

Issue upstream: https://github.com/Docker-Hub-frolvlad/docker-alpine-glibc/issues/62


Correct intent error

Description
This patch fixes an intent(out) vs intent(inout) error that resulted in glaciers melting.
In SFCDIF1_GLACIER, the variable FV was declared as intent(out). Technically, this means that FV was uninitialized.  FV was initialized for the first iteration (ITER=1), but it was not initialized for subsequent calls (iterations 2 through 5).

In the re-factored code, "FrictionVelBare" (which is the re-named variable FV) is indeed listed as "inout":
https://github.com/NCAR/noahmp/blob/master/src/ResistanceBareGroundMostMod.F90
Note that Noah-MP-5.0's structure avoids having to pass variables into and out of routines.
However, the developers note that this variable is both an input and an output for this routine.


[$250] Search - Saved search area is partially overlapped by url field

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

Version Number: 9.0.47
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Navigate to staging.new.expensify.com
Open Search tab
Open any saved search
Click 3-dot menu next to it > Rename
Enter an emoji or name with X:Y format and save it
Tap "v" at the top of a Search page to open the menu
Observe the saved search area.

Expected Result:
Saved search area is fully visible
Actual Result:
Saved search area is partially covered by the url field when opened after renaming it or right after saving the search
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/de3d2df7-2069-472d-a1f8-16dc837ae62a
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021846258880159780544
Upwork Job ID:<PHONE_NUMBER>159780544
Last Price Increase: 2024-10-15

Issue OwnerCurrent Issue Owner: @ikevin127

@strepanier03  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Proposal
Please re-state the problem that we are trying to solve in this issue.
Saved search area is partially covered by the url field when opened after renaming it or right after saving the search
What is the root cause of that problem?
The issue happens if we exceed the maxHeight of the ScrollView in PopoverMenu. The wrap component of the PopoverMenu modal has paddingVertical is 16px so the item inside ScrollView here can be overlapped
https://github.com/Expensify/App/blob/39393b794b983e3164b213052a922aec69689c9b/src/components/PopoverMenu.tsx#L268
https://github.com/Expensify/App/blob/39393b794b983e3164b213052a922aec69689c9b/src/pages/Search/SearchTypeMenuNarrow.tsx#L193-L197
What changes do you think we should make in order to solve the problem?
For this case, we should move the padding-bottom from the wrap component of PopoverMenu to the ScrollView.
https://github.com/Expensify/App/blob/39393b794b983e3164b213052a922aec69689c9b/src/pages/Search/SearchTypeMenuNarrow.tsx#L193-L197
To do that we need some steps like this

Introduce two new props in PopoverMenu: innerContainerStyle is the innerContainerStyle prop we will pass to the PopoverWithMeasuredContent in PopoverMenu, scrollViewContainerStyle is the container style we will apply to the ScrollView here

https://github.com/Expensify/App/blob/39393b794b983e3164b213052a922aec69689c9b/src/components/PopoverMenu.tsx#L268

We will pass innerContainerStyle as styles.pb0 and scrollViewContainerStyle as styles.pb4 to PopoverMenu here

https://github.com/Expensify/App/blob/39393b794b983e3164b213052a922aec69689c9b/src/pages/Search/SearchTypeMenuNarrow.tsx#L193
What alternative solutions did you explore? (Optional)
NA

Working on this now.

I don't have an iOS device to repro this on, but I think this is a very minor visual issue.

♻ Reviewing today...

@strepanier03 I tried reproducing this on iOS Simulator and had no luck.

I reviewed a PR recently (https://github.com/Expensify/App/pull/50244) which fixed the scrolling of the list on large layout devices where it wasn't working, and I think it's expected that when opening the popover menu, the list scroll is at the top when the layout has scrolling space expanded due to one or multiple saved search items, depending on device height.
Therefore I propose to close this as a non-issue. Otherwise I'd need more information on the device this was reproduced on and more detailed steps on how exactly - if I can reproduce then I'll also be able to test whether the existing proposal fixes the issue.

@ikevin127 You can try to save many saved searches and open the first saved search. Besides, I think the only problem here is that we have a space at the bottom.

Thanks for the info! I noticed the space at the bottom on my side as well even though it was not going over any of the saved search options.

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

I think there might be a mistake regarding the platforms on which the issue is occuring as the ✅ iOS: Native is checked but all videos where this was reproduced are on ✅ iOS: mWeb Safari from what I can see.
Before removing the white space from the bottom as suggested by the proposal, I want to ask whether the white space at the bottom was put there on purpose. If nobody knows I guess we can move forward with removing it.
This is how things look on my side (dark theme) and with Safari tab bar both on the bottom and on the top:
https://github.com/user-attachments/assets/284ce84c-0528-458f-b59b-e6ebf6f0c8cd
cc @luacmartins @Kicu @Expensify/design Tagging all which either worked on SearchFiltersNarrow.tsx or might want have a say in whether we want that bottom space removed or anything else changed when it comes to this issue's Expected result.

[!note]
My take
I think that the bottom spacing referenced in https://github.com/Expensify/App/issues/50592#issuecomment-2415674664 is there for a reason as it matches the popover's side / top spacing (see below) and back to this issue: when we have multiple saved search items in the list or even if there's just one and the height of the device is small -> it's expected we would have overlapping while still being able to scroll -> therefore not an issue.
It would only be an issue if the overlapping would not allow scrolling and the option item would be cut by that spacing without being able to scroll to see it fully.
But maybe some of you might see this from a different perspective, hence I tagged you and opened this up for discussion.

Top / Bottom Spacing
Side Spacing

I think the bottom space is fine to be there, but it should scroll with the entire content of the modal. So basically we want these kinds of modals to have 16px padding on the top and bottom, but that padding should be considered as part of the scrollable area that is kept within the overall modal height, if that makes sense.

@shawnborton I think we should only move the padding bottom to the scrollable area. Moving the padding-top to the scrollable area the content will be close to the top edge when scrolling.

I don't think I agree with you, I think both top and bottom should be in the scrollable area. But let's see some videos to compare :)

@shawnborton Here is the video

Only the bottom padding is in the scrollable area

https://github.com/user-attachments/assets/2175da75-e09f-4f1d-80bb-112d2000cf61

Both top and bottom padding are in the scrollable area

https://github.com/user-attachments/assets/dcb37670-bf67-49b9-95fb-e8eab7571eae

I think the second video (both top and bottom can scroll) looks much better to me. Curious what the rest of the @Expensify/design thinks though!

I agree—I think that feels more natural/expected to me.

Updated proposal.
@ikevin127 The design is confirmed. Please help to take a look at my proposal again.

100% with the second video 👍

cc @luacmartins for assignment here based on proposal review from https://github.com/Expensify/App/issues/50592#issuecomment-2420474775.

@mkzie2 Please let us know when we can expect a PR to be ready for review 🧑💻

[x]  [@ikevin127] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/40419.
[x]  [@ikevin127] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/40419/files#r1819876695.
[x]  [@ikevin127] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A.
[x]  [@ikevin127] Determine if we should create a regression test for this bug.
[x]  [@ikevin127] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Go to Search.

Create several saved search items if none (to make the popover scrollable).

Press the header's dropdown button.

Verify there's top padding in the popover.

Scroll the list.

Verify there's NO top AND bottom padding while scrolling.

Scroll to the bottom.

Verify there's bottom padding.
In other words, the padding is scrolled with the list instead of being fixed at the top or bottom.

Do we agree 👍 or 👎.

Reg test submitted. Handling payments now.

Payment Summary

[x] $250 - @ikevin127 via Upwork (Reviewer)
[x] $250 - @mkzie2 via Upwork (Contributor)


Liquid tag 'uri'  is not working correctly

I am using URL rewriting policy with 'value' evaluated as liquid. The value is not returned correctly. It works with liquid {{ service.id }}, or {{ host }}, but not with {{ uri }}.
policy chain:
"policy_chain": [
      {
        "name": "apicast",
        "version": "builtin",
        "configuration": {

        }
      },
      {
        "name": "url_rewriting",
        "version": "builtin",
        "configuration": {
          "query_args_commands": [
            {
              "op": "set",
              "arg": "arg",
              "value_type": "liquid",
              "value": "{{ uri }}"
            }
          ]
        }
      }
    ],

request response:
{
  "args": {
    "arg": "",
    "user_key": "rate-limit-app-1-jsmadis-wjodncie"
  },
  "headers": {
    "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8",
    "Accept-Encoding": "gzip, deflate, br",
    "Accept-Language": "en-US,en;q=0.9",
    "Connection": "close",
    "Cookie": "pl_authenticated=..."
    "Forwarded": "for=<IP_ADDRESS>;host=rate-limit-api-jsmadis-obwvarye-3scale-staging.<3scale_url>;proto=https",
    "Host": "httpbin.org",
    "Upgrade-Insecure-Requests": "1",
    "User-Agent": "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.81 Safari/537.36",
    "X-3Scale-Proxy-Secret-Token": "Shared_secret_sent_from_proxy_to_API_backend_78a28d5a37aa56ca",
    "X-Forwarded-Host": "rate-limit-api-jsmadis-obwvarye-3scale-staging.<3scale_url>"
  },
  "origin": "<IP_ADDRESS>, <IP_ADDRESS>",
  "url": "https://rate-limit-api-jsmadis-obwvarye-3scale-staging.<3scale_url>/get?user_key=rate-limit-app-1-jsmadis-wjodncie&arg="
}

I can confirm this is a bug.
The problem is that when rendering the template strings, the URL rewriting policy uses the "policies context" without merging it with what the ngx_variable module exposes. This is a bug that might happen with other policies too. I think that merging the policies context with the ngx_variable one should be done in the TemplateString module to avoid doing it in every policy.

Fixed in #891


108-barnsley-fern-rainbow

trying to correct the name from Barney to barnsley

it does not updated in the list of the site however. I wish everyone could see it.


update env to fix issues with numpy nan handling in linear algebra computations when using conda

i dug up this solution somewhere in stackoverflow in a link that i've now lost.  The main fix here is nomkl.  but switching to python 3.11 is probably a good idea.

...@bsweger I've moved lightgbm up to the conda section....


nodejs loop problem  not synchronize

con.query('SELECT *  from table',async function(err, rows, fields){
rows.forEach(async function(data, index, arr) {
const result = await cal_funct(json);
console.log(result);
});
});
function cal_funct(){
//some--action
retrun 'a';
}
//============================================
it print table data after 'a'
i want result like this
table data--1
'a'
table data --2
'a'
please help me...................................................................... :)

@krishnaraj1993 Sorry for my lack of response until now. It doesn't look like anything related to ChatterBot was included in what you posted. I'm going to close this ticket off.


NIST Vulnerability Disclosure Reports (VDR)

https://nvlpubs.nist.gov/nistpubs/SpecialPublications/NIST.SP.800-161r1.pdf

See https://github.com/ietf-scitt/use-cases/issues/26 for more details


⚠ Bofrost.fr has degraded performance

In b980f83, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 988 ms

Resolved: Bofrost.fr performance has improved in 79aa9b0 after 7 minutes.


proposal: encoding/base32: add Encoding.Strict

base64.Encoding.Strict was added in #15656, but there was no equivalent method added to base32.Encoding.
We should add the same method to be consistent:
package base32

// Strict creates a new encoding identical to enc except with
// strict decoding enabled. In this mode, the decoder requires that
// trailing padding bits are zero, as described in RFC 4648 section 3.5.
func (enc Encoding) Strict() *Encoding

I should note that Strict is not a good name as it doesn't describe which dimension we are being more strict on.
We could take this time to rename it as RejectNonZeroPadding and add the method to both base32 and base64
and then deprecate the existing base64.Encoding.Strict method.

This feature combined with #53845 makes it possible to implement a truly bijective mapping between baseXX and binary data. This would allow the use of base32 and base64 to produce a truly canonical encoding per RFC 4648, section 3.5.


Add data bindings to gs-grid component.

https://github.com/ganesh-vellanki/gs-grid/projects/1#card-45027726

LGTM


adding some markers inside OpenCV

Adding Matlab like markers in OpenCV since I needed them for an application I made.

Could you provide this for 2.4 branch? It would be awesome :)

Yeah I will backport it to 2.4 also once this one works fine.

Thank you! I have two questions:
Why didn't you make just one function and an enumeration with available markers?
Parameter shift should  be added, shouldn't it?

@mshabunin so you are suggesting 1 function and then a marker_type enumerator? That could work also, changes will not be that difficult. Secondly, I will look into the shift thing, not sure what it does right now and not sure if it is necessary for markers yet :D

Hmm screwed it up :dancers: lets try to fix this commit history

@mshabunin is this what you had in mind?

@StevenPuttemans, yes, thank you.

:+1:

Completely forgot this :D will have a look at it today!

Still some misalignment in the enum documentation. Seems like it needs to go on top of each line. Will fix this first!

@alalek fixed the docs!
http://pullrequest.opencv.org/buildbot/export/pr/5386/docs/d6/d6e/group__imgproc__draw.html

@sturkmen72 applied changes suggested!

@StevenPuttemans, thanks! I will replicate my comment in #5390:
this is useful function! May I suggest to change the order of parameters? i.e. move markerSize before the thickness, since the latter two parameters (thickness and especially lineType) are less important, in my opinion

@vpisarev done in both PRs!

Still some misalignment in the enum documentation. Seems like it needs to go on top of each line.

It is not necessary, just use //!< prefix

Actually now documentation is aligned in the table. If i put it after the expression then the explanations all sink down one leven

Level*

I mean to comment object on the same line you should use //!< instead //! (that is used to put comment before object):
https://www.stack.nl/~dimitri/doxygen/manual/docblocks.html#memberdoc

Okay thank you for the info!

Adapted as you suggested, but it seems the HAL test is failing, which has nothing to do with my input :)

Can this be merged? Cleaning up repos and would be usefull for me if this is in OpenCV :)

sorry for delay. :+1:

Thank you very much!


New webhook page supports quick switching type

New webhook page supports quick switching type

Sometimes user select the wrong type when creating a new webhook.  User need to return to the list page and click the Add button again to select it.
This PR is in the add page, user can click the type-icon in the right-top-corner to quickly switch different types of webhooks.

Please don't do this.  Easy for maintainers to review the PR commit by commit.

Here is the Guideline.
https://github.com/go-gitea/gitea/blob/main/CONTRIBUTING.md

Please don't do this.  Easy for maintainers to review the PR commit by commit.

Here is the Guideline. https://github.com/go-gitea/gitea/blob/main/CONTRIBUTING.md

Sorry, that is a temp log, accidentally submitted up; I've re-edited it now.

Please don't do this.  Easy for maintainers to review the PR commit by commit.

Here is the Guideline. https://github.com/go-gitea/gitea/blob/main/CONTRIBUTING.md

Sorry, that is a temp log, accidentally submitted up; I've re-edited it now.

Thanks @kerwin612 for considering this feedback and making maintainers life easy for future. Appreciated your efforts in this PR.
Kudos 💯

This has multiple issues. On overview page, it renders a empty button:

On view page, it renders the icon partially outside the header:

Overall, the UX with that button seems strange. I could not guess this icon is clickable by looking at the UI. I feel like webhook type should be a dropdown element in the form itself, and switching should ideally also preserve any previously entered values for fields that are in both old and new form.

This has multiple issues. On overview page, it renders a empty button:

On view page, it renders the icon partially outside the header (because english font is not as large as chinese):

Also, the UX with that button seems strange. I could not guess this icon is clickable by looking at the UI.
I feel like webhook type should be a <select> style dropdown element in the form itself, and switching it should ideally also preserve any previously entered values for fields that are in both old and new form.

When I submit PR, the style issues you mentioned does not exist. It may be that the modification of master during the period brings compatibility problems. I will re-adapt it. I will also consider the solution that you mentioned to take webhook type as part of form and replace it with <select>.

<select> dropdown would be ideal, you can take a hint from user management where depending on the value of the auth source dropdown, the form's fields change:

<select> dropdown would be ideal, you can take a hint from user management where depending on the value of the auth source dropdown, the form's fields change:

I tried what you said and it didn't look good because the webhook type with icon;

You wouldn't show a icon in this case, just a dropdown with the type name like in the screenshot.

You wouldn't show a icon in this case, just a dropdown with the type name like in the screenshot.

I completely understand; however, what I considered at the beginning was that the entire form was divided into different templates according to different types, and It is not appropriate to put the switch type in each form template, so I thought of using the icon in the upper right corner to switch;

The current PR UX is as follows:

Another implementation is as follows:

@silverwind @lunny

Another implementation is as follows:
@silverwind @lunny

I would like the second, some benefits we will gain from the design.

It's straight forward
It's possible to change the following options according the first selection
An search box could be added in webhook type selection box

Another implementation is as follows:
@silverwind @lunny

I would like the second, some benefits we will gain from the design.

It's straight forward
It's possible to change the following options according the first selection
An search box could be added in webhook type selection box

I merged the second implementation into the current branch.
I don't think there's a need to add a search box to the current (fewer) options.
Teams discuss it? I look forward to hearing from you!

Maybe I'm wrong, but I think https://github.com/go-gitea/gitea/pull/29114 has now implemented what this PR set out to do, right?

Maybe I'm wrong, but I think #29114 has now implemented what this PR set out to do, right?

I don't understand very much, I submitted this PR in 23 years, and also made changes according to the audit suggestions, why there has been no administrator response to this PR, and the PR you mentioned was submitted in 24 years, and it can be merged very quickly;

Maybe I'm wrong, but I think #29114 has now implemented what this PR set out to do, right?

I don't understand very much, I submitted this PR in 23 years, and also made changes according to the audit suggestions, why there has been no administrator response to this PR, and the PR you mentioned was submitted in 24 years, and it can be merged very quickly;

Sorry for the slow reviews. Can #29114 satisfy your requirements?


Attempt to Link in SceneComposer results in "NullPointerException"

Steps to reproduce:

Create a new scene based on "empty jME3 scene" template.
Hit "Edit in Scene Composer". It opens an empty view with coordination axis
Hit "File -> Link in Scene Composer"

Actual Result: NullPointerException thrown:

Notes:
After step 1 I'm unable to open a scene. E.g. when I click on scene nothing happens, hence I try "Edit in Scene Composer"
My environment details as per jmonkey about window:
Product Version: jMonkeyEngine SDK v3.3.0-stable
Java: 11.0.6; OpenJDK 64-Bit Server VM 11.0.6+10
Runtime: OpenJDK Runtime Environment 11.0.6+10
System: Mac OS X version 10.16 running on x86_64; UTF-8; en_US (jmonkeyplatform)

Also I'm using openjdk11 that I installed via brew (brew install openjdk@11).

Also when I open a project in the SDK's editor, I'm getting a similar error.
Stacktrace looks similar to the above:
Loaded asset CookScene
java.lang.NullPointerException
	at com.jme3.gde.scenecomposer.SceneEditorController$17.run(SceneEditorController.java:721)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:313)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:740)
	at org.netbeans.core.TimableEventQueue.dispatchEvent(TimableEventQueue.java:136)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)
java.lang.NullPointerException
	at com.jme3.gde.scenecomposer.SceneEditorController$17.run(SceneEditorController.java:721)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:313)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:740)
	at org.netbeans.core.TimableEventQueue.dispatchEvent(TimableEventQueue.java:136)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

This highlights another issue with the scenecomposer/j3o's; that there's no way to recover a file once something has gone wrong with it.

I'm a bit unsure of the use case here. Are you trying to link the same scene you just created? Because choosing "Link in Scene Composer" from the File menu won't give you a dialog, but use the currently selected scene in the Project window. So it's equivalent to right clicking a scene and selecting "Link in Scene Composer".


Question - how to configure Azure Firewall with existing public IP resource

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions
terraform:
Terraform v1.3.5
on linux_amd64
azure provider:
hashicorp/azurerm v3.34.0
module:
source = "Azure/caf-enterprise-scale/azurerm"
version = "3.0.0"
Description
Describe the bug
Hi.
This is not a bug more of a question of a setting. We have our public ip  that we want to use instead of a public ip that gett created by the module while we deploy the hub firewall.
We cant find anywhere in the wiki guide how to do that. The question is how to we use our own static public IP (created outside the module) for the hub firewall with the module ? This is because we want to have same public IP for the FW even if we redploy the FW due to FW öppnings in onprem firewalls.
BR
Hung
Steps to Reproduce
Hi.
This is not a bug more of a question of a setting. We have our public ip that we want to use instead of a public ip thats got created by the module while we deploy the hub firewall.
We cant find anywhere in the wiki guide how to do that. The question is how to we use our own static public IP for the hub firewall with the module ?
BR
Hung
Screenshots

Additional context

Thank you for the question @hungbui78
Assuming you are using hub_networks (this isn't possible with Virtual WAN), you should be able to do this using the advanced input as follows:
  configure_connectivity_resources = {
    # additional settings removed for brevity
    advanced = {
      custom_settings_by_resource_type = {
        azurerm_firewall = {
          connectivity = {
            eastus = { # replace `eastus` with the location you are deploying to
              ip_configuration = [
                {
                  name                 = "" # set as per https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/firewall#name
                  public_ip_address_id = "" # set as per https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/firewall#public_ip_address_id
                  subnet_id            = "" # set as per https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/firewall#subnet_id
                }
              ]
            }
          }
        }
      }
    }
  }

The presence of this setting should stop the module from creating the corresponding public IP resource, and attach the specified public IP to the Azure Firewall.
For more information on these settings, please refer to the azurerm_firewall documentation.

Thank you for the question @hungbui78
Assuming you are using hub_networks (this isn't possible with Virtual WAN), you should be able to do this using the advanced input as follows:
  configure_connectivity_resources = {
    # additional settings removed for brevity
    advanced = {
      custom_settings_by_resource_type = {
        azurerm_firewall = {
          connectivity = {
            eastus = { # replace `eastus` with the location you are deploying to
              ip_configuration = [
                {
                  name                 = "" # set as per https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/firewall#name
                  public_ip_address_id = "" # set as per https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/firewall#public_ip_address_id
                  subnet_id            = "" # set as per https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/firewall#subnet_id
                }
              ]
            }
          }
        }
      }
    }
  }

The presence of this setting should stop the module from creating the corresponding public IP resource, and attach the specified public IP to the Azure Firewall.
For more information on these settings, please refer to the azurerm_firewall documentation.

Hi Kevin.
Thx for the help and it works like a charm :)
BR
Hung


[$500] Web - Workspace - The initial selection applies to the new name too

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Navigate to https://staging.new.expensify.com/
Log in with an expensifail account
Create a workspace
Create a room named "asd"
Rename the room by selecting "asd" and replacing it with this text from the clipboard "the user is able to select"

Expected Result:
The previous selection shouldn't be applied to the pasted text.
Actual Result:
The initial selection applies to the new name too.
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/1dea7fba-2982-4000-886b-82664cbaad9b
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
What is the root cause of that problem?
This is happening because of the following code:
https://github.com/Expensify/App/blob/054eb86c638dc35d4a28bf71ad7788adf8a2256a/src/components/RoomNameInput/index.js#L35-L42
What changes do you think we should make in order to solve the problem?
We should check if there was an active selection. If there was – we should move the cursor to the end of the replaced text:
            let newSelection;
            if (selection.start === selection.end) {
                newSelection = {
                    start: selection.start + offset,
                    end: selection.end + offset,
                };
            } else {
                newSelection = {
                    start: selection.end + offset,
                    end: selection.end + offset,
                };
            }

            setSelection(newSelection);

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Workspace - The initial selection applies to the new name too
What is the root cause of that problem?
This is happening because of the following code:
https://github.com/Expensify/App/blob/054eb86c638dc35d4a28bf71ad7788adf8a2256a/src/components/RoomNameInput/index.js#L35-L42
What changes do you think we should make in order to solve the problem?
We need to remove this logic.
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
When replacing the text in the room name, the selection remains.
What is the root cause of that problem?
This is happening because of the following code:
https://github.com/Expensify/App/blob/054eb86c638dc35d4a28bf71ad7788adf8a2256a/src/components/RoomNameInput/index.js#L35-L42
What changes do you think we should make in order to solve the problem?
We need this logic and add new.
if (modifiedRoomName !== newRoomNameWithHash) {
      const newSelection = {
          start: modifiedRoomName.length,
          end: modifiedRoomName.length,
      };
      setSelection(newSelection);
  }

What alternative solutions did you explore? (Optional)

@paultsimura your RCA makes sense, but i am not sure why this is happening on texts with space between words. i am trying to test the solution.

i am not sure why this is happening on texts with space between words

This is because, in the room names, the spaces (and only spaces) get replaced with dashes:
https://github.com/Expensify/App/blob/5c5c862a10b75214899da32333a7f3a694e38be3/src/libs/RoomNameInputUtils.ts#L6-L14
And because of this, only for the names with spaces, this condition becomes true:
https://github.com/Expensify/App/blob/5c5c862a10b75214899da32333a7f3a694e38be3/src/components/RoomNameInput/index.js#L35-L42

Testing proposals.

@getusha does this explanation make sense to you though? https://github.com/Expensify/App/issues/33652#issuecomment-1871961619

And because of this, only for the names with spaces, this condition becomes true:

@paultsimura Can't we  instead compare the length of the strings like  if (modifiedRoomName.length !== newRoomNameWithHash.length) then?

Can't we instead compare the length of the strings

We must compare the string values here. If we enter "some text", it will be transformed into "some-text", so the length will be the same and the logic will not be executed. As a result, the cursor will jump to the end of the input every time a space is added, and this logic was added to fix exactly this:
https://github.com/Expensify/App/assets/12595293/d014c54b-d606-4b92-b5c9-7d988652d174
The whole logic is correct, we just need to handle the selection start as I've described in the proposal.

Discussing proposals

The proposal from @paultsimura looks good to me and works well, thanks for the patience!
🎀 👀 🎀
C+ Reviewed!

Proposal looks ok. The bug feels very minor to me. I am surprised we have such complex logic for pasting text into an input.

Thanks. The PR is ready for review: https://github.com/Expensify/App/pull/34076

@getusha Mind helping get us kicked off with the BZ checklist?

Payment summary:

Issue reporter: QA
Contributor: @paultsimura $500
Contributor+: @getusha $500

Offers sent to both contributors!

Thanks, accepted

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
[@getusha] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/
[@getusha] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/12899/files#r1467873981
[@getusha] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/a
[@getusha] Determine if we should create a regression test for this bug. No, this is a very minor bug
[@getusha] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Thanks @getusha. Those make sense to me. I've processed your offer in Upwork so I'll close this issue.


Qwen2-VL support detection

This will help you. In fact, qwen2-vl support object detection!

https://github.com/QwenLM/Qwen2-VL/issues/9
https://github.com/QwenLM/Qwen2-VL/issues/105
https://github.com/QwenLM/Qwen2-VL/issues/190

谢谢 Rongsheng提供的几个链接， 我快速看了一下里面的讨论， 发现 qwen-vl 和 qwen2-vl确实有考虑支持 object detection，但貌似qwen2-vl在object detection 性能上很差...  这个可能也是qwen2-vl发布的时候，没有刻意提 他们支持object detection的原因。 看起来 他们说 后续会提供一个方案。

谢谢 上面的几个链接， 很有帮助。

Maybe this pr had solved this issue.
https://github.com/huggingface/transformers/pull/33487/files

Thank you Rongsheng. https://github.com/huggingface/transformers/pull/33487/files looks a reasonable fix. Hope it will be merged into main branch of transformers lib.
In this repo, I may cherry pick that PR first and have another try.

look forward to it.


[TC-DM-2.2], [TC-DM-2.4] - Basic Information Cluster Events - Leave event - Need information for verification steps

Problem - Need information for verification steps for reading LEAVE event
App used - all-clusters-app
Platform - Chip-tool - RPI-4, 8GB RAM
DUT - RPI-4, 8GB RAM
Network - Ble-wifi
Commit - c1e1d018820c0fdd51f124f607b9207f31fae2ff
Sequence of steps I tried with :

Commission DUT with TH using below commands :
DUT -  sudo ./all-clusters-app --wifi
Controller -  ./chip-tool pairing ble-wifi 1 GRLPrivate_EXT matter123 20202021 3840
Uncommission  the DUT with TH using below :
./chip-tool operationalcredentials remove-fabric 1 1 0
Subscribe the event using below command :
./chip-tool basic subscribe-event leave 1 100  1 0

PFA Logs:
leave_dut.txt
leave_cntrl.txt
Note : This is not SDK issue but need information how to verify the leave event

You need to subscribe before the fabric removal and have the subscription live during the fabric removal.  Then you may receive a message on the subscription with the event.
So for example:
chip-tool interactive start

followed by the following commands in interactive mode:
    basic subscribe-event leave 1 100 1 0
    operationalcredentials remove-fabric 1 1 0

When I do that I see:
>>> operationalcredentials remove-fabric 1 1 0
...
[1655304788672] [818:12068413] CHIP: [TOO] Endpoint: 0 Cluster: 0x0000_0028 Event 0x0000_0002
[1655304788672] [818:12068413] CHIP: [TOO] 	 Event number: 196609
[1655304788672] [818:12068413] CHIP: [TOO] 	 Priority: Info
[1655304788672] [818:12068413] CHIP: [TOO] 	 Timestamp:<PHONE_NUMBER>
[1655304788672] [818:12068413] CHIP: [TOO]   Leave: {
[1655304788672] [818:12068413] CHIP: [TOO]    }
...
[1655304788674] [818:12068413] CHIP: [DMG] Received Command Response Data, Endpoint=0 Cluster=0x0000_003E Command=0x0000_0008
[1655304788674] [818:12068413] CHIP: [TOO] Endpoint: 0 Cluster: 0x0000_003E Command 0x0000_0008
[1655304788674] [818:12068413] CHIP: [TOO]   NOCResponse: {
[1655304788674] [818:12068413] CHIP: [TOO]     statusCode: 0
[1655304788674] [818:12068413] CHIP: [TOO]     fabricIndex: 1
[1655304788674] [818:12068413] CHIP: [TOO]    }


Add support for observers for macOS

Currently, the Mac version of LottieView do not support observation of the current progress/frame rate of an animation. For the iOS counterpart, I'm using CADisplayLink to observe the animation, but unfortunately it's not available on AppKit.
If anyone has any suggestion on how I could keep track of the animation's progress on macOS, feel free to let me know! You're also welcome to contribute with a pull request with your implementation if you're feeling extra generous.
Thanks for your help!

CVDisplayLink ?

Thanks for the suggestions @Psli! I'll give them a read later and see if I can work something out. Meanwhile, if you managed to get something working on your end with these, feel free to contribute with a pull request if you're feeling generous!

I've got something working with CVDisplayLink, it's currently on the dev branch:
https://github.com/tfmart/LottieUI/blob/55d7ce4e2258484063d3d7cc2af2a926353537e8/Sources/LottieUI/macOS/AnimationProgressObserver%2BmacOS.swift
However, it seems that both this new solution for macOS and the current one for iOS with CADisplayLink doesn't seem to work with the new Core Animation engine introduced in Lottie 4.0. I'll attempt to take a second look at this issue at a later time, it might also be interesting to open a discussion or issue at Lottie's repo as well regarding this.
If you must use either the .onProgress or onFrame modifiers, it's required to use the main thread rendering engine, which can be done with the .renderingEngine modifier:
LottieView("MyAnimation")
    .renderingEngine(.coreAnimation)
    .onProgress { _ in
        // ...
    }

I've just released v1.3.3 which includes the framerate/progress observers for macOS! However, there's an issue when using Lottie's new Core Animation rendering engine, for both iOS and macOS.
I'll be closing this issue since the observers are now available, and for further discussion about the rendering engine issues, I've opened issue #10 to do so


Buffer overflow when opening ARM64 binary written in golang

This issue was seen on 2.2.2592-dev (Build ID c5691f77) in linux.
$ python3
Python 3.8.5 (default, Jul 28 2020, 12:59:40)
[GCC 9.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import binaryninja as bn
>>> bv = bn.BinaryViewType.get_view_of_file("/home/user/conway")
*** buffer overflow detected ***: terminated
Aborted (core dumped)

The UI also crashes immediately after opening the conway binary.
The source for the conway binary can be found in the golang tutorial playground here:
https://play.golang.org/p/jibIOKi7wqW
I compiled it on a Rpi 4 running Ubuntu.
Here is the compiled binary (delete the .txt extension):
conway.txt

PM'd you on slack. Please update to latest dev builds. This should be fixed in https://github.com/Vector35/arch-arm64/commit/0c4a5b8750bcf71bb27e9351c97dc0a3b5de09be arch-armv7 but binja hadn't been updated to point to it yet.

PM'd you on slack. Please update to latest dev builds. This should be fixed in https://github.com/Vector35/arch-arm64/commit/0c4a5b8750bcf71bb27e9351c97dc0a3b5de09be arch-armv7 but binja hadn't been updated to point to it yet.

I can verify this is fixed in latest dev. I am closing this now. Feel free to re-open it if there is still any issue with it

I can verify this is fixed in latest dev. I am closing this now. Feel free to re-open it if there is still any issue with it


Create feature-request.yml

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 77.917%

Files with Coverage Reduction
New Missed Lines
%

src/Tempest/Database/src/GenericDatabase.php
1
90.91%

Totals

Change from base Build<PHONE_NUMBER>7:
-0.02%

Covered Lines:
4340

Relevant Lines:
5570

💛  - Coveralls


Suddenly getting NET::ERR_CERT_COMMON_NAME_INVALID on Chrome for SSL sites

Plese fill the template when you reporting a new issue, thanks!

Which platform are you running AnyProxy
Mac
The version of the AnyProxy
3.10.4
So anyproxy was working fine until Chrome 58. Now suddenly I get this error when browsing HTTPS sites.
I think it is related to this: https://www.chromestatus.com/feature/4981025180483584 https://alexanderzeitler.com/articles/Fixing-Chrome-missing_subjectAltName-selfsigned-cert-openssl/
Is there any known way around this for anyproxy?
Thanks!

@Jonovono
Can you try to remove all certs in ~/.anyproxy/certifications and try again?

Hmm, same issue. Should I update to anyproxy 4.0?

@Jonovono
Yes, please have a try:)

Exactly same issue here. I should rewrite my code entirely to test version 4.x... any solution for 3.x ??
Temporal solution: defaults write com.google.Chrome EnableCommonNameFallbackForLocalAnchors -bool true

Ya I don't think version for fixes it either, or did you have sucess @mt-sergio


[BUG] Some queries fail when cost-based optimizations are enabled

Describe the bug
With the experimental cost-based optimizer enabled, 23 of the NDS queries fail due to inconsistent joins (incompatible mix of CPU/GPU operators).
The queries that fail are q7, q9, q26, q27, q28, q30, q32, q36, q44, q59, q81, q92, q1, q6, q10, q54, q85, q94, q11, q13, q16, q23a, q35

@andygrove is this resolved with #1910 ?

@andygrove is this resolved with #1910 ?

@sameerz No, but it is resolved by https://github.com/NVIDIA/spark-rapids/pull/1954


u/myuser should redirect to the profile

Update the Auth context to store User too, this shouldn't be a security issue, as only the profile redirect will use it (check the URL and redirect to profile page, where the /users/profile endpoint is used, NOT the user from context).

or not


Error about loading finetuned model

Hi @hyn2028,
I meet this error when I load the finetuned model.
Traceback (most recent call last):
File "test.py", line 347, in
main()
File "test.py", line 197, in main
model, tokenizer = load_model_tokenizer_for_generate_separate(args.config_path, args.model_path)
File "./llm-cxr/training/generate.py", line 57, in load_model_tokenizer_for_generate_separate
model = AutoModelForCausalLM.from_pretrained(
File "../transformers/models/auto/auto_factory.py", line 471, in from_pretrained
return model_class.from_pretrained(
File "../transformers/modeling_utils.py", line 2405, in from_pretrained
raise EnvironmentError(
OSError: Error no file named pytorch_model.bin, tf_model.h5, model.ckpt.index or flax_model.msgpack found in directory
./checkpoints/llmcxr_origin_report__2023-06-19_00-13-51.
This are my saved files. Did the training code save the model?

Thank you in advance.

I found that I can use zero_to_fp32.py to generate the model 'pytorch_model.bin'


Correct the Kubernetes example for Request Transformer Kong plugin

Where is the problem?
https://docs.konghq.com/hub/kong-inc/request-transformer/
What happened?
The Example plugin configuration given for Kubernetes in Request Transformer plugin is little incorrect -
Docs link - https://docs.konghq.com/hub/kong-inc/request-transformer/
Field names like remove, rename, add etc should not repeat in yaml for headers, querystring or body
Current example -
apiVersion: configuration.konghq.com/v1
kind: KongPlugin
metadata:
  name: request-transformer-example
config:
  remove:
    headers:
    - x-toremove
    - x-another-one
  remove:
    querystring:
    - qs-old-name:qs-new-name
    - qs2-old-name:qs2-new-name
  remove:
    body:
    - formparam-toremove
    - formparam-another-one
  replace:
    body:
    - body-param1:new-value-1
    - body-param2:new-value-2
  rename:
    headers:
    - header-old-name:header-new-name
    - another-old-name:another-new-name
  rename:
    querystring:
    - qs-old-name:qs-new-name
    - qs2-old-name:qs2-new-name
  rename:
    body:
    - param-old:param-new
    - param2-old:param2-new
  add:
    headers:
    - x-new-header:value
    - x-another-header:something
  add:
    querystring:
    - new-param:some_value
    - another-param:some_value
  add:
    body:
    - new-form-param:some_value
    - another-form-param:some_value
plugin: request-transformer

Expected example -
apiVersion: configuration.konghq.com/v1
kind: KongPlugin
metadata:
  name: request-transformer-example
config:
  remove:
    headers:
    - x-toremove
    - x-another-one
    querystring:
    - qs-old-name:qs-new-name
    - qs2-old-name:qs2-new-name
    body:
    - formparam-toremove
    - formparam-another-one
  replace:
    body:
    - body-param1:new-value-1
    - body-param2:new-value-2
  rename:
    headers:
    - header-old-name:header-new-name
    - another-old-name:another-new-name
    querystring:
    - qs-old-name:qs-new-name
    - qs2-old-name:qs2-new-name
    body:
    - param-old:param-new
    - param2-old:param2-new
  add:
    headers:
    - x-new-header:value
    - x-another-header:something
    querystring:
    - new-param:some_value
    - another-param:some_value
    body:
    - new-form-param:some_value
    - another-form-param:some_value
plugin: request-transformer

Please let me know where can I update this change. I am not able to find it as these are picked from value_in_examples field under config.
What did you expect to happen?
The Kubernetes example needs to be corrected for this plugin.
Code of Conduct and Community Expectations

[X] I agree to follow this project's Code of Conduct
[X] I agree to abide by the Community Expectations

Hey, the ability to do this was added in this PR
https://github.com/Kong/docs.konghq.com/pull/4493/files
I can provide an example of what the syntax would look like tomorrow if you still want to tackle this task.

This was fixed in a rework of the plugin docs: https://docs.konghq.com/hub/kong-inc/request-transformer/how-to/basic-example/


Copyright  2015 --> 2016 update

A lot of references to 2015 need to be updated now that its 2016. I have posted a gist of all of them, found using ag a faster version of ack. If you download the file in the gist linked below, you can open it in Emacs and then M-x compilation-mode to go visit each file and line mentioned. YMMV with other editors.
https://gist.github.com/zbeekman/a09504020f6689e5a43d

Thanks for doing this.  I suppose we'll need to do a refresh annually.  I don't use emacs, but I did take a look at the file online.

@rouson Who said I was doing it?! :stuck_out_tongue_closed_eyes:
However, I would be happy to do it, it should be pretty quick. A question for you though:
Should I update the copyright year for source code, etc. files that have not been edited yet in the new year? Or should I just update everything?

@zbeekman  Thanks for letting me volunteer you!  I honestly don't have a clue what the appropriate practice is.  I generally assume that it's best to update the date on files only if the file has an internal copyright date range that does not extend through the year of the file's time stamp.  My theory is that it matches the practice for copyrighting books, which I think only get updated when a new edition is published.  But this is guesswork for me.  Maybe there's a good reference online the describes the best practice.  I suppose this could be another "git hook" -- assuming that's the right terminology and it's feasible to implement.

I'll ponder the git-hook issue... And try to google/ask around
On Wed, Feb 3, 2016 at 3:45 PM Damian Rouson<EMAIL_ADDRESS>wrote:

@zbeekman https://github.com/zbeekman Thanks for letting me volunteer
you! I honestly don't have a clue what the appropriate practice is. I
generally assume that it's best to update the date on files only if the
file has an internal copyright date range that does not extend through the
year of the file's time stamp. My theory is that it matches the practice
for copyrighting books, which I think only get updated when a new edition
is published. But this is guesswork for me. Maybe there's a good reference
online the describes the best practice. I suppose this could be another
"git hook" -- assuming that's the right terminology and it's feasible to
implement.
—
Reply to this email directly or view it on GitHub
https://github.com/sourceryinstitute/opencoarrays/issues/130#issuecomment-179456086
.

@zbeekman please, share such a git-hook if you find any :smile:

It seems that the convention is to update only when the file in question is edited. The following command will give a list of files and lines containing the string '2015' that have changed since 2015-12-31:
git grep --name-only 2015 $(git log --pretty=format:%H --since="12/31/2015") | cut -d : -f 2 | sort | uniq | while read line; do grep -Hn 2015 $line; done

A number of these have already been taken care of.

Nice.  Thanks!


Feature/sync tuple

This PR adds an implementation of tuple for the Sync effect type that does not rely on Prelude.tupleF.

Any specific reason why not to use tupleF?

Very good point. My initial reasoning was to make it more closely resemble Effect, but I didn't stop to think about why Effect didn't use tupleF and how it is inherently different. I will withdraw the PR.


SuSiE fine-mapping

The scripts and steps followed to run susie using in-sample LD on the meta-analysis GWAS results:

Computing LD and converting to matrix format for running SUSIE (Fri, 23:37, 21-Jun-24).
Susie was successfully run on the example protein and finally three causal variants were identified (Sunday, 16:46, 24-Jun-24).

If there is a misalignment of alleles in the PLINK2 LD file and GWAS results, then the credible set variants will be infinity,
To solve this, we computed the LD matrix with Pearson correlation in R using the dosage exported from PGEN file. The PGEN-based LD looks aligned with GWAS summary file (checked the diagnostic plot of eigenvalues susieR::kriging_rss()). As a result, the susie_rss() model identified one credible set pertaining 6 variants at this locus: seq.12593.33/19_54219624_54427869.
Next, I test it for the other locus with 34 signals!
Thu, 19:18, 11-Jul-24.
Dariush


Variable Explorer crashes when scrolling

Description
What steps will reproduce the problem?

Opened up a dataFrame in Spyder (70851,19)
Did a fast scroll though the data

What is the expected output? What do you see instead?
Was expecting to scroll though the data.  Instead, had a crash.
Please provide any additional information below
File "C:\ProgramData\Anaconda3\lib\site-packages\spyder\widgets\variableexplorer\dataframeeditor.py", line 260, in get_value
value = self.df.iat[row, column]
File "C:\ProgramData\Anaconda3\lib\site-packages\pandas\core\indexing.py", line 1869, in getitem
return self.obj._get_value(*key, takeable=self._takeable)
File "C:\ProgramData\Anaconda3\lib\site-packages\pandas\core\frame.py", line 1979, in _get_value
return _maybe_box_datetimelike(series._values[index])
File "C:\ProgramData\Anaconda3\lib\site-packages\pandas\core\common.py", line 174, in _maybe_box_datetimelike
value = tslib.Timestamp(value)
File "pandas/_libs/tslib.pyx", line 390, in pandas._libs.tslib.Timestamp.new
File "pandas/_libs/tslib.pyx", line 1575, in pandas._libs.tslib.convert_to_tsobject
File "pandas/_libs/tslib.pyx", line 1669, in pandas._libs.tslib.convert_datetime_to_tsobject
File "pandas/_libs/tslib.pyx", line 1848, in pandas._libs.tslib._check_dts_bounds
pandas._libs.tslib.OutOfBoundsDatetime: Out of bounds nanosecond timestamp: 9999-01-01 00:00:00
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\site-packages\spyder\widgets\variableexplorer\dataframeeditor.py", line 279, in data
value = self.get_value(row, column-1)
F

This looks like an error in Pandas. Please post the code you use to generate the Dataframe you're trying to visualize in Spyder.

Closed due to lack of response. Please re-open if you can provide an example to reproduce this, thanks.


Goodbye 👋


workflow engine stuck waiting for already-finished jobs

This happened when running https://github.com/reanahub/reana-workflow-engine-snakemake/pull/42#discussion_r859837148
Only run-batch-... is running, all the run-job-... pods have finished:
$ kubectl get pods | grep run-
reana-run-batch-7c0ebe80-6cf3-44df-9899-105ffa5ab062-f2j8k   3/3     Running   0          57m

job-controller has cleaned up all the jobs (175):
$ kubectl logs reana-run-batch-7c0ebe80-6cf3-44df-9899-105ffa5ab062-f2j8k -c job-controller | grep 'Cleaning Kubernetes job' | wc -l
175

According to job-controller, all the job have finished:
$ kubectl exec reana-run-batch-7c0ebe80-6cf3-44df-9899-105ffa5ab062-f2j8k -c job-controller --  curl localhost:5000/jobs > jobs.json
$ cat jobs.json | jq '.jobs[] | values[].status' | wc -l
175
$ cat jobs.json | jq '.jobs[] | values[].status' | grep finished | wc -l
175

r-w-e-snakemake confirms that 175 jobs were submitted, however only 171 have finished:
$ kubectl logs reana-run-batch-7c0ebe80-6cf3-44df-9899-105ffa5ab062-f2j8k -c workflow-engine | grep 'submitted job:' | wc -l
175
$ kubectl logs reana-run-batch-7c0ebe80-6cf3-44df-9899-105ffa5ab062-f2j8k -c workflow-engine | grep 'job is finished.' | wc -l
171
$ kubectl logs reana-run-batch-7c0ebe80-6cf3-44df-9899-105ffa5ab062-f2j8k -c workflow-engine | tail
2023-07-17 13:29:58,697 | snakemake.logging | Thread-1 | INFO | Finished job 124.
2023-07-17 13:29:58,697 | snakemake.logging | Thread-1 | INFO | 169 of 176 steps (96%) done
2023-07-17 13:29:58,701 | reana-workflow-engine-snakemake | Thread-1 | INFO | make_data job is finished. job_id: 76807136-32ca-4349-ab4d-1b32d7df8bb8
2023-07-17 13:29:58,702 | snakemake.logging | Thread-1 | INFO | [Mon Jul 17 13:29:58 2023]
2023-07-17 13:29:58,702 | snakemake.logging | Thread-1 | INFO | Finished job 139.
2023-07-17 13:29:58,702 | snakemake.logging | Thread-1 | INFO | 170 of 176 steps (97%) done
2023-07-17 13:30:08,720 | reana-workflow-engine-snakemake | Thread-1 | INFO | make_data job is finished. job_id: 9d02883d-c7da-4f67-909a-72fd53db0dbe
2023-07-17 13:30:08,721 | snakemake.logging | Thread-1 | INFO | [Mon Jul 17 13:30:08 2023]
2023-07-17 13:30:08,721 | snakemake.logging | Thread-1 | INFO | Finished job 154.
2023-07-17 13:30:08,721 | snakemake.logging | Thread-1 | INFO | 171 of 176 steps (97%) done

In the database four jobs are still reported as running:
reana=# select id_, backend_job_id, status from __reana.job where status != 'finished';
                 id_                  |                   backend_job_id                   | status
--------------------------------------+----------------------------------------------------+---------
 607da740-1199-4cba-9ab7-2cb9ac5772a8 | reana-run-job-160bec2f-86bf-47bc-a208-f0635c3632e4 | running
 b5ffb667-6910-4e3f-be6e-21c13ca49161 | reana-run-job-f410a704-794b-4cac-8ad3-687644107acb | running
 985e6a86-3bc2-4fee-86b0-9e513e80b3f6 | reana-run-job-16380469-6e75-4f9f-98b0-cfd4f4f03d5e | running
 49f6e328-00ca-4ca9-a6ed-dc7602e6b9fd | reana-run-job-291c18c3-ccff-4b16-a300-0cdb37671f1c | running
(4 rows)

Additional note: reana.yaml
inputs:
  files:
    - Snakefile
workflow:
  type: snakemake
  file: Snakefile
  resources:
    kerberos: true
outputs:
  files:
  - myoutput.png


The 31,000 USDC deal went into a slow path and has been in place for over 1.5 hours!

Transaction hash:
https://connextscan.io/tx/0x4fa4f9ab01ae1f1932809b3c35af08a37578a80e6eec747fa350ef95bd1212ee?src=search
Additional context
Add any other context about the problem here.

completed

transaction has been completed


Suggestion:  enable discussions on this repo

I've noticed that many issues (including some of my own) are not signaling problems with the Wazero codebase, but instead questions for the community to the effect of "how do I do X?" or "is feature Y supported?".
I think it would be helpful to enable the discussions feature on this repository for such interactions.  In addition to keeping code issues and developer support separate, the discussions feature has the benefit of facilitating search.

We currently do a lot of chat on #wazero and #wazero-dev channels on gophers slack, which is searchable, but not as linkable or searchable here. Knowing that, are you still interested in discussions? Regardless, I think we need to update our issue template.
Note: You may need an invite to join gophers.

Oh, brilliant.  See you there!

We could update the issue template and put the GopherSlack info there

now that we have provided the gopher slack info in the website https://wazero.io/community/#keeping-up-to-date I think we could close this.


Loading module with SystemJS

Is it possible to the load the module with SystemJS? If so how should it be configured? I have it configured as below
 paths: {
      'npm:': 'node_modules/'
  },
   map: {
     ..... ,
     'ng2-bootstrap': 'npm:ng2-bootstrap/bundles/ng2-bootstrap.umd.js',
    'ng2-color-picker': 'npm:ng2-color-picker'
  },
  packages: {
    ....., ,
   'ng2-color-picker': { main: 'index.js', defaultExtension: 'js' }
}
It's looking for a file that doesn't exist
Error: Uncaught (in promise): Error: (SystemJS) XHR error (404 Not Found) loading http://localhost:3000/node_modules/ng2-color-picker/src/components.js
Instead of  http://localhost:3000/node_modules/ng2-color-picker/src/components/index.js

Hi neilpimley,
I'm not too familiar with SystemJS, however it looks like the index path is incorrect, try this:
{
'ng2-color-picker': { main: 'lib/index.js', defaultExtension: 'js' }
}

Hope this helps, let me know either way!
Regards,
Andy

Hi @neilpimley,
I had the same issue. This worked for me:
var map = {
    …
    'ng2-color-picker': 'app/third-party/ng2-color-picker'
};

var packages = {
    …
    'ng2-color-picker': { main: 'lib/index.js', defaultExtension: 'js', map: {
            './lib/src/components': './lib/src/components/index.js',
            './lib/src/models': './lib/src/models/index.js'
     }}
};

Regards

Closing as a solution has been confirmed, thanks drihm.


[Bug Report] Image is only built for linux/arm64?

Hi everyone,
as I was working on the helm chart for a PR I wanted to run in linux/arm64/v8 and encountered
Failed to pull image "ghcr.io/opencost/opencost-parquet-exporter:latest": rpc error: code = Unknown desc = no matching manifest for linux/arm64/v8 in the manifest list entries on my M1/minikube.
I've take a look at the workflow and also checked the responsible action. Its docs point out the platform property: "Fixed platforms for current node. If not empty, values take priority over the detected ones.". It seems to me that we could extend the build with

- name: Build and push Docker image with cache
        id: build-and-push
        if: github.event_name != 'schedule' &&  startsWith(github.ref, 'refs/tags/v') != true
        uses: docker/build-push-action@0565240e2d4ab88bba5387d719585280857ece09 # v5.0.0
        with:
          ...
         platforms: linux/amd64, linux/arm64, linux/arm/v8, linux/arm/v7, linux/arm/v6

Specific values are debatable I guess. Same goes for the build-and-push-nocache block.
What do you think? Id raise the PR if you like :)
All the best
Christoph

I'm sure we'd definitely appreciate Arm support. FWIW arm64/v8 is the same as arm64 (as opposed to a Raspberry Pi or somesuch), so this should be relatively straightforward.

Fixed in https://github.com/opencost/opencost-parquet-exporter/pull/9


Inconsistencies with initializing nested arrays between slang in glsl mode and glslc

https://godbolt.org/z/eModn8d5d
https://godbolt.org/z/Tb4cTTMh9
MRE godbolts are attached.
const int TOP_FACE_POINTS[4] =
    int[4](0, 0, 1, 0);

const int BOTTOM_FACE_POINTS[4] =
    int[4](0, 1, 0, 1);

const int FACE_LOOKUP_TABLE[2][4] = int[2][4](
    TOP_FACE_POINTS,
    BOTTOM_FACE_POINTS);

const int FACE_LOOKUP_TABLE_2[2][4] = int[2][4]( // NOTE the reversing of the order from slang!
    int[4](0, 0, 1, 0),
    int[4](0, 1, 0, 1));

const int TOP_FACE_POINTS[4] =
    int[4](0, 0, 1);

const int BOTTOM_FACE_POINTS[4] =
    int[4](0, 1, 0);

const int FACE_LOOKUP_TABLE[2][4] = int[2][4](
    TOP_FACE_POINTS,
    BOTTOM_FACE_POINTS);

const int FACE_LOOKUP_TABLE_2[2][4] = int[4][2](
    int[4](0, 0, 1),
    int[4](0, 1, 0));

[shader("compute")]
void computeMain(){}

The above glsl is accepted, while the slang is rejected even when operating in glsl mode.

This is not valid Slang syntax and we have not yet added compatiblity support for glsl mode.
The correct Slang syntax for initializing global constant array is to use initializer list:
static const int arr[4][2] = { {1,2,3,4}, {5,6,7,8}};

I think this is a bug in parsing type int [x][y], it should result in type Array<Array<int, y>, x>.

To workaround this for now before the issue is fixed, either write:
const int FACE_LOOKUP_TABLE[2][4] = Array<Array<int, 4>, 2>(
    TOP_FACE_POINTS,
    BOTTOM_FACE_POINTS);

or:
const int FACE_LOOKUP_TABLE[2][4] = {
    TOP_FACE_POINTS,
    BOTTOM_FACE_POINTS };

@9291Sam is the alternative syntax sufficient to unblock you? This is a problem specific to GLSL style front loaded array type syntax. If you use the HLSL style syntax with the initializer list you should get correct results. We probably won't be able to get to a proper fix in the near-term, but welcome contribution.

@bmillsNV No, this was not a blocking issue for me in the first place, I just wanted to put this on the radar.

Thanks for confirming and for filing this issue. This is worth resolving, but we don’t have current available bandwidth – moving to the backlog for now. Please do feel welcome to propose a solution if you’re able to before it makes it out of the backlog!


Idea: Embed google group somewhere in iNat site

The Google Group is a little hard to find. There are 1004 members of the Google group which is a small fraction of the 70K+ who have added observations (I'm counting unverifiable there).
Here's another site that displays their google group posts within the site: http://symbiota.org/docs/google-group/

What's the advantage of doing this vs. linking directly to it from the footer? If we embed it that won't make it any easier for people to actually participate int he Google Group.

I think the advantage is visibility and findability. I wouldn't click on feedback in the footer unless I really had something to say, but if I were poking around for more information about iNaturalist and saw that there are several posts a day in a forum-like space I might be interested in exploring. Since you don't have to be logged in to read the posts, I think people are more likely to stumble across them if it's visible on a page within iNaturalist, maybe somewhere in the help or about sections.

This is clearly no longer relevant considering the awesome integration of the Discourse forum.


Added Algebra Algorithms

Description
Added:
2. Algebra
2.1. Sieve of Eratosthenes
2.2. Fibonacci Sequence
2.3. Linear Diophantine Equations
2.4. Euler's Totient Function
2.5. Modular Multiplicative Inverse
Fixes issue #4
Type of change
Please delete options that are not relevant.

[ ] Added Algorithm
[ ] Folder Structure

Added to SUMMARY.md (delete section if not needed)?
Have you added the page to SUMMARY.md if this is a new page and not a fix?

[ ] Yes

Perfect :fire:

@allcontributors add @iammanish17 for contributing to documentation


render of "taxonomy" failed when changing default logo

Render fails when I assign any value to the logo key in data/navbar.toml.
Even if it is the default "img/logo.svg".
hugo: collected modules in 715 ms
Start building sites …
hugo v0.133.0-c9777473d1369f812d727a6c07dc57ad7be7bf62+extended linux/amd64 BuildDate=2024-08-17T19:57:41Z VendorInfo=gohugoio
ERROR render of "taxonomy" failed: "/tmp/hugo_cache/modules/filecache/modules/pkg/mod/github.com/hugo-porto/theme@v0.1.7/layouts/_default/baseof.html:10:9": execute of template failed: template: _default/list.html:10:9: executing "_default/list.html" at <partial "navbar.html" .>: error calling partial: "/tmp/hugo_cache/modules/filecache/modules/pkg/mod/github.com/hugo-porto/theme@v0.1.7/layouts/partials/navbar.html:42:24": execute of template failed: template: partials/navbar.html:42:24: executing "partials/navbar.html" at <$logo.RelPermalink>: nil pointer evaluating resource.Resource.RelPermalink

I've tried to debug the issue. I could see that $logo is null, as indicated in the error, but I couldn't figure out why.


Update configuring.rst

As there are no steps ,mentioned above !!, so it should either be steps below or more accurate can be "the default configuration file present at /conf is enough"

Thanks for the patch! I've modified it very slightly, hope you don't mind too much.


Add dictionary support to dbt-clickhouse

To resolve #553 in a way that will work with dev/local/k8s/Altinity/CH Cloud we will need to add dictionary support to ClickHouse. @SoryRawyer has written a proposal for this that I'd like to roll into our next SoW, this task is just to track the necessity for this work for v1.

@SoryRawyer has gotten this merged here: https://github.com/ClickHouse/dbt-clickhouse/pull/255 and it was released in v1.7.4 so we just need to update aspects-dbt and tutor-contrib-aspects to use those changes.

I believe we can call this done. @SoryRawyer I'm closing this but if there are any bug fixes or changes we need to push back upstream lmk and we can reopen it.


.xlsx file that opens w/ other applications is silently imported without content

Hi there,
What reasons would there be for an excel file that (say) Apple Numbers can import without issue, failing silently with js-xlsx? By failing I mean no sheets are recorded, and the non-existent sheets are then not populated with data (obviously). No errors are sent to console.
test.xlsx
Thanks!

In general it tries not to blow up if certain things look weird.  Passing the option WTF:1 to read or readFile will suppress that behavior.  In the node command that's the --dev option.
In this file, there were a few XML inconsistencies.  For certain tags like numFmt, the spec asserts that the content must be empty.  Excel and Numbers usually write these cells with an empty body but this file has a zero-length string body.
For example, here's the first number format (rename the XLSX file with a .zip extension, extract, and look at xl/styles.xml):
<numFmt numFmtId="82" formatCode="[$-010409]m/d/yyyy"></numFmt>

The expected form is
<numFmt numFmtId="82" formatCode="[$-010409]m/d/yyyy"/>

We will roll out a fix soon.


Hard limit on dvr channels

Describe the bug
Since the last plex update I’m hitting a hard limit of 77 channels on the dvr. At first I thought I had mapped them wrong and went around hunting for issues. After a while I noticed a pattern and doesn’t matter what tuner it’s coming from or the EPG source there is a max of 77 channels that can be added to the plex DVR.
Could someone please check this and let me know if there is a workaround for it?
Thanks in advance
Server (please complete the following information):

OS: Ubuntu 18.04
Intel x64
xTeVe Version: 2.2.0 running on docker

To Reproduce
Steps to reproduce the behavior:

Add live TV DVR
Select Channels
No channels will be added until only 77 are selected, otherwise you get the error "There was a problem saving your DVR. Please try again."

Expected behavior
All channels to be added
Additional context
I am not certain if this is an xteve or plex issue. However if anyone could help me understand that would be great!

I just updated Plex on my Synology to version Version <IP_ADDRESS>02.  I deleted my xTeVe DVR and re-added it and did not encounter this hard limit issue.  What platform are you running Plex on and are you using the Plex Pass version or the public version?  I have a Plex Pass but I only installed the public version update.
Thanks!

Mine is still a version behind so will update it and let you know :)

Yeah no dice. Same version has yours and for some reason I can't add all channels... Very frustrating!

Guy's, This is not Plex. Please refer all Plex questions to Plex.


Detect GKE platform with Windows containers

Currently Platform.Instance() fails to correctly detect GKE on Windows.

We should devote some time to this, possibly in the next fixit.

I now have a Windows cluster (that I was going to remove soon...) so I'm in a good position to retest this and see what is available.

(But reduced the priority to p3 as it's not very important.)

Actively working on this now...

KUBERNETES_SERVICE_HOST is defined
KUBERNETES_SERVICE_PORT is defined
HOSTNAME is not defined, but COMPUTERNAME is - but that's only the first part of the pod name; Dns.GetHostName() returns the complete name.
Reading the certificate fails with "Internal.Cryptography.CryptoThrowHelper+WindowsCryptographicException: Unspecified error"; likewise reading the mountinfo file fails
We'll need to test separately under .NET Framework; it might be simplest to just report pod name and namespace

Okay - I can't get the namespace or pod JSON even without the certificate check (URL returns a 403). So let's make the code just do what it can, and return at least the right platform type and minimal information.
I believe we already fail in many cases anyway without legacy authentication.

Ah - without any metadata, we can't return a GkePlatformDetails. I think we might need to close this as infeasible...


黑屏然后闪退

运行ossas.exe后黑屏，一段时间后显示以下文字：

c:\python37\lib\site-packages\PyInstaller\loader\pyimod03_importers.py:489: MatplotlibDeprecationWarning:
The MATPLOTLIBDATA environment variable was deprecated in Matplotlib 3.1 and will be removed in 3.3.
Traceback (most recent call last):
File "tensorflow\python\pywrap_tensorflow_internal.py", line 18, in swig_import_helper
File "imp.py", line 296, in find_module
ImportError: No module named 'pywrap_tensorflow_internal'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "tensorflow\python\pywrap_tensorflow.py", line 58, in
File "c:\python37\lib\site-packages\PyInstaller\loader\pyimod03_importers.py", line 489, in exec_module
File "tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
File "tensorflow\python\pywrap_tensorflow_internal.py", line 20, in swig_import_helper
ImportError: DLL load failed: 动态链接库(DLL)初始化例程失败。
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "main.py", line 5, in
File "c:\python37\lib\site-packages\PyInstaller\loader\pyimod03_importers.py", line 489, in exec_module
File "Seq2Seq.py", line 8, in
File "c:\python37\lib\site-packages\PyInstaller\loader\pyimod03_importers.py", line 489, in exec_module
File "tensorflow_init.py", line 40, in
File "c:\python37\lib\site-packages\PyInstaller\loader\pyimod03_importers.py", line 489, in exec_module
File "tensorflow\python_init_.py", line 49, in
File "c:\python37\lib\site-packages\PyInstaller\loader\pyimod03_importers.py", line 489, in exec_module
File "tensorflow\python\pywrap_tensorflow.py", line 74, in
ImportError: Traceback (most recent call last):
File "tensorflow\python\pywrap_tensorflow_internal.py", line 18, in swig_import_helper
File "imp.py", line 296, in find_module
ImportError: No module named '_pywrap_tensorflow_internal'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "tensorflow\python\pywrap_tensorflow.py", line 58, in
File "c:\python37\lib\site-packages\PyInstaller\loader\pyimod03_importers.py", line 489, in exec_module
File "tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
File "tensorflow\python\pywrap_tensorflow_internal.py", line 20, in swig_import_helper
ImportError: DLL load failed: 动态链接库(DLL)初始化例程失败。
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.
[11552] Failed to execute script main

运行main.py的反馈是：

Traceback (most recent call last):
File "F:\Ossas_ChatBox\Ossas_ChatBot-master\main.py", line 5, in
import Seq2Seq as rnn
File "F:\Ossas_ChatBox\Ossas_ChatBot-master\Seq2Seq.py", line 8, in
from tensorflow.keras.preprocessing.text import Tokenizer
ModuleNotFoundError: No module named 'tensorflow'

requirements.txt里的东西都装了

对于tensorflow初始化失败问题：
https://github.com/Dimsmary/Ossas_ChatBot/issues/4
对于运行main.py后的问题：
ModuleNotFoundError: No module named 'tensorflow'
你并没有安装tensorflow


Support for om client credentials

In order to support opsmanagers configured w/ SAML, LDAP, etc., which don't use basic auth & only support client credentials, it would be real nifty if hammer could support --client-id (OM_CLIENT_ID) and --client-secret (OM_CLIENT_SECRET) in lieu of username/password
Thanks for the consideration 👍

This is an interesting area that I hadn't really considered, what kind of config change do you think would make the most sense?
The two main versions of this I can see would be
{
  "ops_manager": {
    "url": "OPSMAN-FQDN",
    "username": "OPSMAN-CLIENT-ID",
    "password": "OPSMAN-CLIENT-PASSWORD",
    "client_credentials": true
  },
  ...
}

Where ops_manager.client_credentials would default to false or
{
  "ops_manager": {
    "url": "OPSMAN-FQDN",
    "client_id": "OPSMAN-CLIENT-ID",
    "client_password": "OPSMAN-CLIENT-PASSWORD"
  },
  ...
}

where it would default to using ops_manager.username and ops_manager.password if they were present.
I'll run this past a couple of people, do you have any particular preference or other thoughts?

besides opsmgrs which don't have basic auth creds, the secondary use-case I'm thinking of (but haven't actively experimented with yet) for client creds is using hammer in the context of automation, where hammer would be used as a cleaner way to delegate commands to pks, bosh, cf, etc. within a reusable concourse task

that's a good question, I think I'm down for option 2, where you can specify either client_id/client_secret or username/password, or if you specify both, it takes one or the other as default & ignores the other?
For example it looks like pivotal-cf/om will accept client-id/secret and username/password and when it creates the oauth client it takes client-id/secret as precedence if provided, and username/password as secondary
I feel the same thing applies with hammer, but instead hammer use username/password as the "primary" preferable choice in case both client-id/secret and username/password are provided
... so tl;dr, second choice feels right; if you can run it by some other people and get their opinions that'd probably be real helpful 👍

I think the second choice would be my preferred option as well.  Although I'd argue for the default to be the client_id/client_secret to be the default so that it is consistent with om.  I think hammer is early enough in its life to make this switch.