[Compiler] Rename compiler ROCM target to HIP

As a follow-up to https://github.com/iree-org/iree/pull/17710, this patch renames the compiler target backend from ROCM to HIP to be consistent with driver.
User-facing changes:
IREE_TARGET_BACKEND_ROCM to IREE_TARGET_BACKEND_HIP
iree-hal-target-backends=rocm to iree-hal-target-backends=hip

IIRC we didn't want to do this? (I think Lei told me ROCM and HIP were independent? did someone change their mind? :)

They are indeed independent. We wanted to be consistent with the backend names and thus I started working on it. But happy to hear thoughts on moving forward
cc: @antiagainst @ScottTodd

Mostly just thinking about HSA: it will be using the same codegen pipeline but not targeting HIP, so if there's a better name for what it is that'd be ideal ("ROCDL" may have been what Lei mentioned before). There will be a HIP IREE::HAL::TargetDevice and HSA IREE::HAL::TargetDevice that share the same codegen backend.

I'm not sure there's a better answer. Both rocm and hip names refer to software stacked above. If we were naming from scratch, we would name like llvm: "amdgpu". From that perspective, I would rename the runtime driver "amdgpu-hip". That's what I call it in shortfin fwiw.

Ok you all type too fast. +1 to amdgpu-hip, or maybe amdgpu-llvm. Either way it would be nice to switch away from "rocm."

FWIW, the bespoke clang is called hipcc too

AMDGPU may work, but ROCm as the umbrella also makes sense (HIP and ROCr/HSA are both under the ROCm umbrella) and is equivalent to CUDA.

(basically, I don't want inconsistency and I don't want the compiler side that targets multiple runtime implementations named after a particular runtime implementation)

AMDGPU alone is also the ultimate backend for spirv/vulkan with amdvlk. If they ever implement offline shader compilation, we will have a name collision with that.

IIUC AMDGPU is the name of the target in llvm right? https://llvm.org/docs/AMDGPUUsage.html
So maybe amdgpu-llvm for clarity? I kind of like the idea of being consistent with the llvm backend we're targeting and also differntiating from spir-v.

IIUC AMDGPU is the name of the target in llvm right? https://llvm.org/docs/AMDGPUUsage.html
So maybe amdgpu-llvm for clarity? I kind of like the idea of being consistent with the llvm backend we're targeting and also differntiating from spir-v.

amdgpu is the name of the backend, I the two targets are amdgcn-amdpal (graphics/vulkan) and amdgcn-amd-hsa (hip/rocm)

IMO hip and amdgpu-hip are the two least problematic names

reminder: hip != hsa, and we'll be using the same thing for hsa - in my dreams we end up deleting our hip HAL - hip is problematic :)

OK, I give up, naming proves hard again 🙃

yeah, we're in the ROC* name playground that is certifiably inscrutable - there's no good names :P

What if we go with amdgpu-rocm for the compiler target thereby encompassing both hip and hsa, and also skirting the potential name clash with vulkan

Also tangential question, could we have aliased names for target backends? e.g. if I put in hip or hsa as the compiler backend it would alias to amdgpu-rocm or whatever we choose to call it. If so I'd prefer to lean into a more llvm focused name because it makes the C++ very clear that it's going to hand the executables off to llvm.

Also tangential question, could we have aliased names for target backends? e.g. if I put in hip or hsa as the compiler backend it would alias to amdgpu-rocm or whatever we choose to call it. If so I'd prefer to lean into a more llvm focused name because it makes the C++ very clear that it's going to hand the executables off to llvm.

My only problem with rocm is that I've never really understood what it is :P

rocm == cuda (effectively) - an umbrella term for the entire suite of libraries/tools/languages/etc

I think the stem can defensively be amdgpu or rocm. I prefer the first as it is more descriptive of a physical thing that will always be that thing.

Also tangential question, could we have aliased names for target backends? e.g. if I put in hip or hsa as the compiler backend it would alias to amdgpu-rocm or whatever we choose to call it. If so I'd prefer to lean into a more llvm focused name because it makes the C++ very clear that it's going to hand the executables off to llvm.

the llvm command line flag library supports aliases

https://iree.dev/guides/deployment-configurations/gpu-rocm/#build-the-compiler-from-source
Says Please make sure you have followed the [Getting started](https://iree.dev/building-from-source/getting-started/) page to build the IREE compiler, **then enable the ROCm compiler target with the IREE_TARGET_BACKEND_ROCM option.**

Could you write - how? I tried to rebuild with this flag, but it is not recognized

Please file issues for support questions to help keep pull request code reviews focused on the code. To your question, configure CMake with -DIREE_TARGET_BACKEND_ROCM=ON in the list of options. We would need to see what you tried and the logs to debug further.


AttributeError: module 'signal' has no attribute 'SIGALRM'

Thanks for your great work.
When running the run.sh or run.py, I got an attribute error while it was rescaling the expression: refined_candidate[0]["predicted_tree"] =  scaler.rescale_function(env, refined_candidate[0]["predicted_tree"], *scale_params[0])
Traceback (most recent call last):
  File "D:\xx\Anaconda\envs\deeplearning39\lib\site-packages\IPython\core\interactiveshell.py", line 3526, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-1-60ea49bf5158>", line 2, in <module>
    scaler.rescale_function(env, refined_candidate[0]["predicted_tree"], *scale_params[0])
  File "D:\xx\TPSR\timeout.py", line 16, in wrapper
    signal.signal(signal.SIGALRM, _handle_timeout)
AttributeError: module 'signal' has no attribute 'SIGALRM'

I would need assistance with fixing an error. Can you please guide me on how to resolve this issue?

Hi @Ohay-Angah ,
Thank you for your interest in TPSR!
It seems like the AttributeError you're encountering is related to the signal.SIGALRM attribute, which is not available on Windows platforms. If you're using Windows, this could be the cause of the problem. I couldn't replicate this error on the Unix-based system.
If the operating system is Windows, probably need to consider an alternative approach for the functionality. If you have access to a Unix-based system, you might want to try running the script there and let us know if the issue persists.

It works on Unix-based systems like a charm. I can also run your code on Windows if commenting timeout(10) property on the top of rescale_function.
How do you understand the eval_results, 'cause running the symbolicregression skeleton seems to give me very different expression outcomes from the expectations?
python run.py --eval_mcts_on_pmlb True \
                   --pmlb_data_type feynman \
                   --target_noise 0.0 \
                   --lam 0.1 \
                   --horizon 200 \
                   --width 3 \
                   --num_beams 1 \
                   --rollout 3 \
                   --no_seq_cache False \
                   --no_prefix_cache True \
                   --max_input_points 200 \
                   --max_number_bags 10 \
                   --save_results True

I put a snippet of my current results here.

@Ohay-Angah Happy to hear that!
TPSR is building on the Transformer symbolic regression models. As these models are pretrained with millions of synthetic math functions, their power is in learning strong priors for this task (numeric-to-symbolic mapping) rather than the exact recovery of true expressions. We noticed that in most cases predicted expressions with pretrained priors can generalize well to OOD data which signifies that the learned mathematical relations might be good approximation of true math relations. I hope this explanation helps!

The explanation helps, @parshinsh! I close this issue as it's been resolved!


Remove default value for image.tag

In charts/uptrace/values.yaml the field image.tag is set as seen below.
image:
  tag: '1.7.0'

this causes the issue of not using the appVersion field from the charts/uptrace/Chart.yaml file.
I would suggest setting the image.tag = "" to use the charts appVersion by default.
As of chart version "1.7.4" with an appVersion of "1.7.4" the uptrace image that is being used is "1.7.0" instead of "1.7.4"

I've removed the tag in the master branch.


Unbind is not sending query string parameters

According to the OSB API spec on unbind, service_id and plan_id must be sent in the query string. We are not sending those.

cc/ @ljnelson

Thanks much.


Windows version of desktop app

When we can expect a Windows version?

Please follow milestone #38 to track the progress of Windows dvpn app.


After file is uploaded the Create drop-down is not closing

Overview
Steps to reproduce:

click "Create"
select "Upload File"
upload a file
see that the drop-down is still visible not closed

@shashigharti
Can we just close the dialog when Upload File button is clicked?

Thank you! I think I have tried that and other ways but every change that I made impacted other functionality (dropdown menu). I will try again and share the details here.


Removed defer - resolves #28

Changes Unknown when pulling be334aeb1edd35903196bb379579abe26af13c96 on bluebird-defer into ** on master**.


Allow three-way merge when last applied configuration is missing

At this moment, work controller will refuse to process objects if no last applied configuration is present (code). If the last applied configuration annotation is missing from objects for any reason (e.g. user tampering), the system will be trapped in an error loop.
During on-duty session, we have identified two clusters so far which are suffering from this pattern; since Kubernetes three-way merge algorithm itself does accept objects w/o last applied configuration (code), we could solve this pattern by dropping the requirement from our end as well.
See the discussion in on-duty channel for impact clusters.

https://github.com/Azure/fleet/pull/353


[SPARK-45105][DOCS] Make hyperlinks in documents clickable

What changes were proposed in this pull request?
The pr aims to make hyperlinks in documents clickable, include: running-on-mesos.html & running-on-yarn.html
Why are the changes needed?
Improve the convenience of using Spark documents.
Before:

After:

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manually test.
Was this patch authored or co-authored using generative AI tooling?
No.

Similar issues with Spark website have also been modified: https://github.com/apache/spark-website/pull/475

Merged to master


Group Echo Submission

Hi Prof. Goodman,
Here's group echo. We would like to make a submission of our Assignment Candidate Generators.
Thanks,
Echo

Hi Prof. Goodman,
We have updated the codes. Please help to test in production. Thanks!

Hi Prof. Goodman,
is there any other issue with our codes? Thanks!

Please look at the "files changed" tab


Add stress test scripts

Add a stress test script script for each mass storage endpoint. Each script tests all attached DAPLink boards at the same time. All three scripts can be running at the same time. To determine that a CI setup is stable these three tests should be run at the same time with multiple boards attached for 1+ hour (ideally 24+ hours).

@mbed-bot: TEST

CC: @sg-

@mbed-bot: TEST


name emojis!!

inspired by ghosty ghomoji or wtvr it is
depends on #325
maybe related to #351?

aw sh i forgot i still have to finish execute


Let kts precompiled scripts synthetic builds respect --offline

Fixes https://github.com/gradle/gradle/issues/23564

@bot-gradle test AFT

I've triggered the following builds for you:

AllFunctionalTestsPullRequestFeedback build

@bot-gradle test AFT

I've triggered the following builds for you:

AllFunctionalTestsPullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

I've triggered a build for you.


Add updateable field to plan object

per #582

What is the semantic relationship between the service level updatable field and plan level updatable field? Does a service updatable effect the plan updatable. For example is it valid to have service updatable false and plan updatable true?

I'm leaning towards saying that the plan can be "false" if the service is "true", but the plan can not be "true" if the service is "false".  ie. a plan can be more restrictive than the service, but not less.

I take that back... Russell just pointed out to me that "bindable" for services and plans allows for a plan to override the service regardless of the service's value.  So we should probably be consistent with that.

So leave it as is then? Independent of the service offering's field?

@jberkhahn @duglin I suggest adding in the default/precedence clauses like are done in the services.bindable and the plan.bindable fields. For services.bindable:
This specifies the default for all plans of this service. Plans can override this field (see Plan Object).

For plan.bindable (changing bindable to plan_updateable):
This field is OPTIONAL. If specified, this takes precedence over the `bindable` attribute of the service. If not specified, the default is derived from the service.

borrowed the wording from bindable per Russel's suggestion

Fixed

I wonder if having plan_updateable on the service offering and updateable on the service plan is confusing?
Perhaps we should just use plan_updateable in both places for consistency?

There is some text in the Updating a Service Instance section which talks about enabling support for updating the service instance(To enable support for the update of the plan, a Service Broker MUST declare support per service by including "plan_updateable": true in its catalog endpoint.) I think that needs to be enhanced with the implications of the updateable flag on the plan.

@jberkhahn to address comments and then we will re-review (hopefully asynchronously before next week).

LGTM

Added the link.


help parameter

What is the recommended way to add a documentation or a description or a help string to an attribute?  I am thinking of something similar to the argparse help keyword.  I have not been able to find anything like this in the documentation or source.
I can always annotate each attr.ib with a comment where they are defined in the source code, but it seems like a field that could naturally be bundled into the attr itself.

What is the recommended way to add a documentation or a description or a help string to an attribute?

You want to generate a comment in the source code?

I think they want some way of exposing __doc__...?

There a ticket on this but I’m too much on a vacation to find it.


Cannot play M3U8 file ("No TS fragments found")

Using the latest version, I'm trying to play an M3U8 file, the files being served from python -m SimpleHTTPServer.
I get this error:
FlasHLS Clappr (version: <IP_ADDRESS> - flashls: <IP_ADDRESS>+(58f08b4), id: o5)
VM3344:1 WARN:No TS fragments found in http://localhost:8000/example_html/hls_stream_sintel/ts/480x270.m3u8
VM3345:1 WARN:empty level Playlist, retry in 1000 ms
ERROR:HLSError(code/url/msg)=2/http://localhost:8000/example_html/hls_stream_sintel/sintel-trailer.m3u8/no fragments in playlist

The player is set up with:
var player = new Clappr.Player({
    source          : "http://localhost:8000/example_html/hls_stream_sintel/sintel-trailer.m3u8",
    parentId        : "#player",
    baseUrl         : "http://localhost:8000/clappr/dist/",
    autoPlay        : true,
})

The M3U8 file exists, so do the TS segments:
$ curl http://localhost:8000/example_html/hls_stream_sintel/sintel-trailer.m3u8
#EXTM3U
#EXT-X-STREAM-INF:PROGRAM-ID=1,BANDWIDTH=500000,RESOLUTION=480x270
ts/480x270.m3u8
#EXT-X-STREAM-INF:PROGRAM-ID=1,BANDWIDTH=1000000,RESOLUTION=640x360
ts/640x360.m3u8
#EXT-X-STREAM-INF:PROGRAM-ID=1,BANDWIDTH=2000000,RESOLUTION=1280x720
ts/1280x720.m3u8

$ curl http://localhost:8000/example_html/hls_stream_sintel/ts/480x270.m3u8 | head
#EXTM3U
#EXT-X-VERSION:3
#EXT-X-TARGETDURATION:2
#EXT-X-MEDIA-SEQUENCE:0
#EXTINF:1.023222,
480x2700.ts
#EXTINF:1.000000,
480x2701.ts
#EXTINF:1.000000,
480x2702.ts

This happens on Windows 7 in Chrome (Version 44.0.2403.89 m), but not on Mac OS X.

@slhck how are you generating this hls? So we can try to replicate this behavior.
Can you try to serve this files from a nginx? (with proper mime-type)
http {
    server {
        listen 80;

        location /example_html {
            types {
                application/vnd.apple.mpegurl m3u8;
                video/mp2t ts;
            }
            root /path/example_html/;
            add_header Cache-Control no-cache;
            add_header Access-Control-Allow-Origin *;
        }

        location /crossdomain.xml {
            default_type text/xml;
            return 200 '<?xml version="1.0"?>
                <!DOCTYPE cross-domain-policy SYSTEM "http://www.adobe.com/xml/dtds/cross-domain-policy.dtd">
                <cross-domain-policy>
                    <site-control permitted-cross-domain-policies="all"/>
                    <allow-access-from domain="*" secure="false"/>
                    <allow-http-request-headers-from domain="*" headers="*" secure="false"/>
                </cross-domain-policy>';
            expires 24h;
        }
    }
}

I'm using these scripts to generate the .TS files. (I just changed the resolution/bitrate, I think).
Let me try with nginx later, but I'm wondering how that could have an effect, since it works on OS X with the plain SimpleHTTPServer from Python.

Thanks @slhck !
I also don't know, it was just a guess, I think maybe windows and macos works different with flash requiring proper header (mime-type). In the end it might be a problem with flash hls implementation.


chore(main): release faro-shop 0.47.0

:robot: I have created a release beep boop
0.47.0 (2024-09-10)
Features

Bump app to 0.38.8 (#778) (1899ab9)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

faro-shop-0.47.0
:sunflower:


Error with updating plugins

[x ] I am on the latest Hyper.app version
[x ] I have searched the issues of this repo and believe that this is not a duplicate

ElementaryOS 0.4.1:
Hyper.app 1.3.3:
https://gist.github.com/Hunteerq/fe01910bb2b42670fc29b999ab405784 link to hyper.js:
https://gist.github.com/Hunteerq/c349038bd0445b6de1e378a315620cac  link to error debug.log
is Vanilla:

Issue
I have edited hyper.js file and after one save I encountered plugin error, I tried both, undoing changes, and reinstalling hyper, but it didn't work. As a result none of a plugins works and every time I start Hyper I get  error log about plugins..

Update to Hyper 1.4 repaired everything!


Titlepage: Missing field for Processing Time

As required by the DHBW, the titlepage should include information about the processing time ("Bearbeitungszeitraum").

The date attribute can take an array of two datetime objects to show a date range. So if you want to display a range, just provide the start and end date like this:
date: (start-date, end-date)


Add check to oref0-meal to make sure there's enough glucose history to match carb history

After watching COB stay way to high for a long time I finally found the issue was caused by using my short glucose.json with 10 entries instead of the 24h glucose.  The result was that carbs older than 50 minutes were never getting decayed.

Looks like that fixed it.  Can you please test to confirm?

Looks good.
I saw both the "Could not parse input data" and "Optional feature meal assist disabled: not enough glucose data to calculate carb absorption" while upgrading one of my rigs.


Kotlin latest version compatibility

Prerequisites
N/A
Describe the bug
The Compatibility section of the README specifies Kotlin support up to 1.5.x. However, given the publically known vulnorabilities that are fixed in version 1.7.x, support for newer versions of kotlin would be beneficial, otherwise to avoid these issues people would need to use a different package.
I have done some very basic testing. All versions of 1.6.x are working fine, but I'm seeing issues on 1.7.x. This was nowhere near as comprehensive or thourough as any maintainer could do, hence it's probably pointless to provide details.
Expected behavior
Documented compatibility with the latest version of Kotlin.

It looks like you're correct that version 1.6.x should work, but 1.7.x is broken.  It looks like in 1.7.x, the Kotlin plugin decided to stop extending SourceTask.  Unfortunately, that would mean that to integrate, the Avro plugin would need to either depend on a specific version of the Kotlin plugin to compile, or use version-specific reflection, in order to access the right method to declare new sources.  I don't think that either of those are the right direction for this plugin.  Thus, in the next version of this plugin, I'll be dropping the Kotlin plugin integration logic, making it so that build authors can wire up the tasks as they see fit without this plugin getting in the way.  It would be possible to encapsulate that logic in a derived plugin, if desired.

Version 1.4.0 has been released.


[WIP] Add SYSTEM_ALERT_WINDOW and WRITE_SETTINGS support

see #50
TODO

[x] Add SYSTEM_ALERT_WINDOW support
[x] Add WRITE_SETTINGS support
[x] Refactor addWithCheckBody
[x] Refactor addResultCaseBody
[x] Add validation that does not mix annotations for @NeedsPermission
[x] Add onActivityResult
[x] Add test cases

I will remove [WIP] after I implement this pull request.

Here we go, @hotchemi
There are some points you should review very carefully.

PERMISSION_UTILS, addWithCheckBody and addResultCaseBody are moved to BaseProcessorUnit
To make it consist, local variable activity is now gone
Create mock android.provider.Setting for testing purpose.

Ah sorry for the late. Your change is so huge so I'm gonna dive into the detail this night.

@shiraji Your PR is almost awesome!

To make it consist, local variable activity is now gone

You said it, but what do u mean?

Thanks

You said it, but what do u mean?

SupportFragmentProcessorUnit used local variable "activity", on the other hand, NativeFragmentProcessorUnit used target.getActivity(). This difference was really hard to keep when I moved addWithCheckBody() to BaseProcessorUnit. Since, ActivityProcessorUnit doesn't use local variable, to make it consist, I removed local variable "activity".

ic.
I'll merge it after next version's release.
Thanks a lot!

Thanks for reviewing this. And I would like to thanks for giving me a chance to learn Kotlin!

@shiraji It was the work of @aurae. I haven't done anything haha.


[Bug]: Test Container locks up testhost after Windows Update August update

Testcontainers version
3.3.0 and 3.5.0
Using the latest Testcontainers version?
Yes
Host OS
Windows 11
Host arch
x64
.NET version
7.0.400
Docker version
Client:
 Cloud integration: v1.0.35-desktop+001
 Version:           24.0.5
 API version:       1.43
 Go version:        go1.20.6
 Git commit:        ced0996
 Built:             Fri Jul 21 20:36:24 2023
 OS/Arch:           windows/amd64
 Context:           default

Server: Docker Desktop 4.22.1 (118664)
 Engine:
  Version:          24.0.5
  API version:      1.43 (minimum version 1.12)
  Go version:       go1.20.6
  Git commit:       a61e2b4
  Built:            Fri Jul 21 20:35:45 2023
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.21
  GitCommit:        3dce8eb055cbb6872793272b4f20ed16117344f8
 runc:
  Version:          1.1.7
  GitCommit:        v1.1.7-0-g860f061
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Docker info
Client:
 Version:    24.0.5
 Context:    default
 Debug Mode: false
 Plugins:
  buildx: Docker Buildx (Docker Inc.)
    Version:  v0.11.2-desktop.1
    Path:     C:\Program Files\Docker\cli-plugins\docker-buildx.exe
  compose: Docker Compose (Docker Inc.)
    Version:  v2.20.2-desktop.1
    Path:     C:\Program Files\Docker\cli-plugins\docker-compose.exe
  dev: Docker Dev Environments (Docker Inc.)
    Version:  v0.1.0
    Path:     C:\Program Files\Docker\cli-plugins\docker-dev.exe
  extension: Manages Docker extensions (Docker Inc.)
    Version:  v0.2.20
    Path:     C:\Program Files\Docker\cli-plugins\docker-extension.exe
  init: Creates Docker-related starter files for your project (Docker Inc.)
    Version:  v0.1.0-beta.6
    Path:     C:\Program Files\Docker\cli-plugins\docker-init.exe
  sbom: View the packaged-based Software Bill Of Materials (SBOM) for an image (Anchore Inc.)
    Version:  0.6.0
    Path:     C:\Program Files\Docker\cli-plugins\docker-sbom.exe
  scan: Docker Scan (Docker Inc.)
    Version:  v0.26.0
    Path:     C:\Program Files\Docker\cli-plugins\docker-scan.exe
  scout: Command line tool for Docker Scout (Docker Inc.)
    Version:  0.20.0
    Path:     C:\Program Files\Docker\cli-plugins\docker-scout.exe

Server:
 Containers: 39
  Running: 2
  Paused: 0
  Stopped: 37
 Images: 54
 Server Version: 24.0.5
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Using metacopy: false
  Native Overlay Diff: true
  userxattr: false
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Cgroup Version: 1
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: io.containerd.runc.v2 runc
 Default Runtime: runc
 Init Binary: docker-init
 containerd version: 3dce8eb055cbb6872793272b4f20ed16117344f8
 runc version: v1.1.7-0-g860f061
 init version: de40ad0
 Security Options:
  seccomp
   Profile: unconfined
 Kernel Version: <IP_ADDRESS>-microsoft-standard-WSL2
 Operating System: Docker Desktop
 OSType: linux
 Architecture: x86_64
 CPUs: 20
 Total Memory: 15.47GiB
 Name: docker-desktop
 ID: 1a6d71d1-9038-4cf4-b6ae-7a1a745742f9
 Docker Root Dir: /var/lib/docker
 Debug Mode: false
 HTTP Proxy: http.docker.internal:3128
 HTTPS Proxy: http.docker.internal:3128
 No Proxy: hubproxy.docker.internal
 Experimental: false
 Insecure Registries:
  hubproxy.docker.internal:5555
  <IP_ADDRESS>/8
 Live Restore Enabled: false

WARNING: No blkio throttle.read_bps_device support
WARNING: No blkio throttle.write_bps_device support
WARNING: No blkio throttle.read_iops_device support
WARNING: No blkio throttle.write_iops_device support
WARNING: daemon is not using the default seccomp profile

What happened?
run a test in visual studio or dotnet test
1-2 tests would pass fine
test runner tries to continue running test, but is currently locked up.
test running is locked up due to WerFault.exe is catching an error and reporting problem to Windows

also happening on coworker's machine.
can provide video if requested
Relevant log output
from dotnet logs,

logs did say testcontainer stopped before 2 or 3rd test starts.

Additional information
No response

I have not noticed any issues on my Windows test environment.

test runner tries to continue running test, but is currently locked up.
test running is locked up due to WerFault.exe is catching an error and reporting problem to Windows

Does this imply that the test keeps running infinite, or is there something causing the test process to fail (can you share the WerFault.exe report)? Could you please look into the output of the docker ps and docker inspect command? My initial assumption is that the dependent container may not be initializing correctly, causing Testcontainers to wait for the readiness check confirmation (can you share the container builder configuration).
Occasionally, resetting Docker Desktop to its default factory settings can also be helpful 😅.

looks like testhost is the program that is using werfault.

at the start of the all of the test run, the test container starts up fine.

but when the 2nd or 3rd test of the whole test run, the container i am starting up with testcontainer is no longer shown in docker desktop.

I am sorry, I do not think I can help much with this information. Have you tried to reset Docker Desktop? Are you possibly exceeding the resources allocated to your test host?

but not seeing logging info

Run the tests with Debug configuration and check the test / debug window in Visual Studio (View > Output).
In addition to that, try to run dotnet test --verbosity detailed to get a verbose output.

i have not reset docker desktop.  i want to preserve my existing containers.  trying code modifications
will provide a dotnet test -v:diag shortly

Microsoft (R) Test Execution Command Line Tool Version 17.7.1 (x64)
Copyright (c) Microsoft Corporation.  All rights reserved.
Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
[testcontainers.org 00:00:00.05] Connected to Docker:
Host: npipe://./pipe/docker_engine
Server Version: 24.0.5
Kernel Version: <IP_ADDRESS>-microsoft-standard-WSL2
API Version: 1.43
Operating System: Docker Desktop
Total Memory: 15.47 GB
InitializeAsync Called
[testcontainers.org 00:00:00.17] Docker container 8e84a8c7fd52 created
[testcontainers.org 00:00:00.23] Start Docker container 8e84a8c7fd52
[testcontainers.org 00:00:01.53] Wait for Docker container 8e84a8c7fd52 to complete readiness checks
[testcontainers.org 00:00:01.54] Docker container 8e84a8c7fd52 ready
[testcontainers.org 00:00:01.59] Docker container d5e4bb66c8f0 created
[testcontainers.org 00:00:01.60] Start Docker container d5e4bb66c8f0
[testcontainers.org 00:00:01.92] Wait for Docker container d5e4bb66c8f0 to complete readiness checks
[testcontainers.org 00:00:01.93] Execute "/bin/sh -c true && (grep -i ':01538' /proc/net/tcp || nc -vz -w 1 localhost 5432 || /bin/bash -c '</dev/tcp/localhost/5432')" at Docker container d5e4bb66c8f0
[testcontainers.org 00:00:03.05] Execute "/bin/sh -c true && (grep -i ':01538' /proc/net/tcp || nc -vz -w 1 localhost 5432 || /bin/bash -c '</dev/tcp/localhost/5432')" at Docker container d5e4bb66c8f0
[testcontainers.org 00:00:04.19] Execute "/bin/sh -c true && (grep -i ':01538' /proc/net/tcp || nc -vz -w 1 localhost 5432 || /bin/bash -c '</dev/tcp/localhost/5432')" at Docker container d5e4bb66c8f0
[testcontainers.org 00:00:04.32] Docker container d5e4bb66c8f0 ready
Fi=000,Fa=000,Pe=000 #   > FEATURE: GetRecentlyAccessedForUserHandlerTests
Before Start
Fi=000,Fa=000,Pe=001 #  1> SCENARIO: Validates QueryResults RecentlyAccessedWorkflow ReturnsCreatedWorkflow
Fi=000,Fa=000,Pe=001 #  1>   STEP 1/6: GIVEN There Is A Person...
Fi=000,Fa=000,Pe=001 #  1>   STEP 1/6: GIVEN There Is A Person (Passed after 867ms)
Fi=000,Fa=000,Pe=001 #  1>   STEP 2/6: AND A Workflow Exists...
Fi=000,Fa=000,Pe=001 #  1>   STEP 2/6: AND A Workflow Exists (Passed after 409ms)
Fi=000,Fa=000,Pe=001 #  1>   STEP 3/6: AND The Workflow Has A Version...
Fi=000,Fa=000,Pe=001 #  1>   STEP 3/6: AND The Workflow Has A Version (Passed after 1s 158ms)
Fi=000,Fa=000,Pe=001 #  1>   STEP 4/6: AND A Recently Accessed Exists...
Fi=000,Fa=000,Pe=001 #  1>   STEP 4/6: AND A Recently Accessed Exists (Passed after 92ms)
Fi=000,Fa=000,Pe=001 #  1>   STEP 5/6: WHEN Query Is Invoked...
Before End
Fi=000,Fa=000,Pe=001 #  1>   STEP 5/6: WHEN Query Is Invoked (Passed after 572ms)
Fi=000,Fa=000,Pe=001 #  1>   STEP 6/6: THEN QueryResult IncludesRecentlyCreatedWorkflow...
Fi=000,Fa=000,Pe=001 #  1>   STEP 6/6: THEN QueryResult IncludesRecentlyCreatedWorkflow (Passed after 13ms)
Fi=001,Fa=000,Pe=000 #  1>   SCENARIO RESULT: Passed after 3s 211ms
After Start
[testcontainers.org 00:00:24.40] Delete Docker container d5e4bb66c8f0  <!--
After End

interesting.
removing [assembly: CollectionBehavior(DisableTestParallelization = true, MaxParallelThreads = 2)] got me working again.  i thought i saw something to ensure 1 process is running database connections.

I'm having the same issue. Sadly I don't get any logs to post here.
The Ryuk container does start (logs exist) but it fails to start my custom container (Typesense). It takes about 60s until the StartAsync is finished and then I get an exception:
 Unhandled exception. System.InvalidOperationException: Could not find resource 'TypesenseContainer'. Please create the resource by calling StartAsync(CancellationToken) or CreateAsync(CancellationToken).
brickhub-backend-1  |    at DotNet.Testcontainers.Guard.ThrowIf[TType](ArgumentInfo`1& argument, Func`2 condition, Func`2 ifClause)
brickhub-backend-1  |    at DotNet.Testcontainers.Resource.ThrowIfResourceNotFound()
brickhub-backend-1  |    at DotNet.Testcontainers.Containers.DockerContainer.GetMappedPublicPort(String containerPort)
brickhub-backend-1  |    at DotNet.Testcontainers.Containers.DockerContainer.GetMappedPublicPort(Int32 containerPort)

My setup is a little more complex, I try to start the testcontainer from within another container in a docker compose. Yes I did mount the volume for docker:
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock

Funnily enough this issue does not occur when running the app natively (not from within the docker container).
I also noticed that the container is never created - so this is not an issue with the WaitStrategy
Any way you can help me or for me to debug this behavior?

This looks like a Compose configuration. Did you set the environment variable mentioned here: https://dotnet.testcontainers.org/examples/dind/#compose? The Wormhole configuration varies a bit for each environment. I need more information about your setup if it is still not working.

@HofmeisterAn thank you! I missed that part...


Npm Run Ios does not work clearly

🐛 Bug Report
Im using Expo . After i update my macOs and xCode to latest versions. I can not run my project with npm run ios.
To Reproduce
Open my project in terminal. Write npm run ios
Expected Behavior
Before update latest versions. When i write npm run ios in terminal , simulator open quickly now i got error;
Simulator is installed but is identified as 'com.apple.CoreSimulator.SimulatorTrampoline'; don't know what that is.
Simulator is not installed.  If you already have the latest version of Xcode installed, you may have to run the command sudo xcode-select -s /Applications/Xcode.app.
Code Example
I do not have any code to share for this issue
Environment
info
React Native Environment Info:
OS: macOS 10.14.4
npm: 6.8.0 - /usr/local/opt/node/bin/npm
SDKs:
iOS SDK:
Platforms: iOS 12.2, macOS 10.14, tvOS 12.2, watchOS 5.2
Android SDK:
API Levels: 26, 28
Build Tools: 28.0.3
System Images: android-26 | Google APIs Intel x86 Atom, android-27 | Google Play Intel x86 Atom, android-28 | Google APIs Intel x86 Atom
IDEs:
Android Studio: 3.3 AI-182.5<IP_ADDRESS>64788
Xcode: 10.2/10E125 - /usr/bin/xcodebuild
npmPackages:
react: 16.5.0 => 16.5.0
react-native: https://github.com/expo/react-native/archive/sdk-31.0.0.tar.gz => 0.57.1
npmGlobalPackages:
create-react-native-app: 2.0.2
react-native-cli: 2.0.1
react-native-create-library: 3.1.2

It might be related to expo, please report it to them. For my plain React Native project, I don't see any issue after macOS and XCode update.


Fix rotateArray function

There is currently a method called rotateArray that lives in /lib/rotateArray.js.
It is incomplete and needs to be fixed!
Acceptance Criteria:

Running yarn test rotateArray results in tests passing.
You have written a skeleton method for someone else to work on.
You have written tests surrounding your skeleton method.
Running yarn lint does not print any errors to the console!
Optional: write one or two more tests surrounding the method you are solving to account for potential edge cases.

Please include the skeleton of a new method + an accompanying test for someone else to work on at the time of creating a pull request! A pull request will most likely be denied if it does not contain a skeleton method for someone else to work on! For more information, please read the Contributing Guide.
Thank you so much for your contribution!

Sorry for doing this slightly out of order, but
@BlakeGuilloud I would like to work on this :)

you got it! I'll review.

Hey @domosapien ! Looks like your pull requests are missing Skeleton Methods for other people to work on. If you don't mind following the CONTRIBUTING.md, we require all pull requests to contain a problem for someone else to work on. This ensures the project continues to grow!  Thanks for your contributions- looking forward to getting your work in!

Ok, I misunderstood what that meant. I'll add them now.

Nice work! Thanks


Consider using central package management.

We could try to make dependencies management easier via central package management as @gfoidl suggested in https://github.com/JasperFx/marten/pull/2789#discussion_r1400711573.

I'm voting no on this one. We don't have that many projects, and that central package thingies doesn't have any equivalent to Paket's lock file, so there's no reproducible builds.

@oskardudycz I can pick up this one. I looked at MSBuild stuff and there is room for improvement. CPM is only one feature to consider based on @gfoidl's comment.

Just not happening, and I really need the issue list slimmed down just to find things easier. Would be way more helpful for Wolverine though.


Fixed issue with invisible fields in element modals

Matrix fields aren’t getting “pimped” in element modals, but their fields were still getting hidden. Made a change to show fields if they’re within one of these modal windows.
Not sure if you're taking pull requests but thought I'd open one anyway!

Also fixed the same issue with matrix fields inside Super Table fields.

This is really helpful, thanks! I'll take a look soon and merge it.

This will be merged in the next release, thanks!


Add list of branches to ntuplehelper

As suggested by @YongyiBWu, it would be good if ntuplehelper could list all branches. At the moment, you need to know the name of the branch so that you can get all the leaves from it.

Now added in #221. There is also a page with the full list of branches in the repository here


ERROR in ./src/pageConfig.json

Running npm build output with successful build:
ERROR in ./src/pageConfig.json
Module build failed: SyntaxError: Unexpected token { in JSON at position 2318
at JSON.parse ()
at Object.module.exports (C:\Users\blablabl_user_name\Desktop\git-playbook-master\node_modules\json-loader\index.js:3:49)
@ ./src/services/configManager.js 3:0-44
@ ./src/router/index.js
@ ./src/main.js
In the file:
module.exports = function (source) {
if (this.cacheable) this.cacheable();
var value = typeof source === "string" ? JSON.parse(source) : source;
value = JSON.stringify(value)
.replace(/\u2028/g, '\u2028')
.replace(/\u2029/g, '\u2029');
return module.exports = ${value};
}
Build complete.

Resolved mine by reinstalling. Thank you so much.

you'll typically see this error when pageConfig.json isnt a valid JSON. the reason re-install resolved it was that pageConfig.json was reset by that. Next time make sure pageConfig.json is valid and you should be good too. Closing

Thank you so much. Great work on this.


[Issue report] Tablet gestures not working with official Raspberry Pi Touch Display

Describe the bug or issue
Tablet Mode gestures (except swipe from left to go back) not working.
Describe your setup

Raspberry Pi 4B with 2GB RAM
Chromium OS r96 r1
Pi connected via DSI + 5V + GND to an official Raspberry Pi Touch Display

This required switching from KMS to FKMS in config.txt: dtoverlay=vc4-fkms-v3d

To Reproduce

Enter Tablet Mode

Enable "Debugging keyboard shortcuts" flag
Click the Restart button
Press Ctrl + Alt + Shift+ T to switch to Tablet Mode

Open an app
Swipe up from the bottom -- tried long, short, swipe + hold

Expected behaviour
OS should show Home or Pinned apps or All open app windows, respectively.
Actual behavior
A regular scroll happens
Note swiping in from the left edge of the screen to go back works fine, so I don't think it's a hardware issue.

Also any pointers on booting into Tablet Mode would be great since I don't intend to have a physical keyboard connected once set up, thanks!

This is a known limitation to Chromium OS, as the Chromium OS we released believes you are having a "Chromebox", not a Chrometablet. At this moment we do not have any intention to change this behaviour and it is thus outside the scope of this project.


Download Tracker/Download Stats

Track downloads of the app.

we can access stats through http://www.somsubhra.com/github-release-stats/?username=datproject&repository=dat-desktop :sparkles: - good enough for now I reckon


'NoneType' object has no attribute 'close_button'

I can't launch kivy designer on ubuntu 15.10.
in terminal:
mnrl@mnrl-K54HR:~/Masaüstü/kivy-designer-master$ ./main.py
[INFO   ] [Logger      ] Record log in /home/mnrl/.kivy/logs/kivy_16-03-06_7.txt
[INFO   ] [Kivy        ] v1.9.2-dev0
[INFO   ] [Python      ] v2.7.10 (default, Oct 14 2015, 16:09:02)
[GCC 5.2.1 20151010]
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_gif, img_sdl2, img_pil (img_ffpyplayer ignored)
[INFO   ] [Factory     ] 182 symbols loaded
[INFO   ] [Text        ] Provider: sdl2
[INFO   ] [OSC         ] using  for socket
[INFO   ] [Window      ] Provider: sdl2(['window_egl_rpi'] ignored)
[INFO   ] [GL          ] OpenGL version <3.0 Mesa 11.0.2>
[INFO   ] [GL          ] OpenGL vendor <X.Org>
[INFO   ] [GL          ] OpenGL renderer <Gallium 0.4 on AMD CAICOS (DRM 2.40.0, LLVM 3.6.2)>
[INFO   ] [GL          ] OpenGL parsed version: 3, 0
[INFO   ] [GL          ] Shading version <1.30>
[INFO   ] [GL          ] Texture max size <16384>
[INFO   ] [GL          ] Texture max units <16>
[INFO   ] [Window      ] auto add sdl2 input provider
[INFO   ] [Window      ] virtual keyboard not allowed, single mode, not docked
[INFO   ] [Clipboard   ] Provider: gtk3(['clipboard_dbusklipper'] ignored)
xclip version 0.12
Copyright (C) 2001-2008 Kim Saunders et al.
Distributed under the terms of the GNU GPL
[INFO   ] [Cutbuffer   ] Provider: xclip
[INFO   ] [CutBuffer   ] cut buffer support enabled
Traceback (most recent call last):
File "./main.py", line 12, in
DesignerApp().run()
File "/usr/lib/python2.7/dist-packages/kivy/app.py", line 802, in run
root = self.build()
File "/home/mnrl/Masaüstü/kivy-designer-master/designer/app.py", line 1708, in build
self.root = Designer()
File "/home/mnrl/Masaüstü/kivy-designer-master/designer/app.py", line 204, in init
self.designer_settings = DesignerSettings()
File "/home/mnrl/Masaüstü/kivy-designer-master/designer/designer_settings.py", line 50, in init
super(DesignerSettings, self).init(*kwargs)
File "/usr/lib/python2.7/dist-packages/kivy/uix/settings.py", line 905, in init
self.add_interface()
File "/usr/lib/python2.7/dist-packages/kivy/uix/settings.py", line 935, in add_interface
interface = cls()
File "/usr/lib/python2.7/dist-packages/kivy/uix/settings.py", line 687, in init
self.menu.close_button.bind(
AttributeError: 'NoneType' object has no attribute 'close_button'
all requirements are installed and up to date.

I cant launch too on ubuntu 16.04 and 14.04,
''[INFO   ] [Logger      ] Record log in /home/yavuz/.kivy/logs/kivy_16-06-18_0.txt
[INFO   ] [Kivy        ] v1.9.1
[INFO   ] [Python      ] v2.7.11+ (default, Apr 17 2016, 14:00:29)
[GCC 5.3.1 20160413]
[INFO   ] [Factory     ] 179 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_gif, img_sdl2, img_pil (img_ffpyplayer ignored)
[INFO   ] [Text        ] Provider: sdl2
[INFO   ] [OSC         ] using  for socket
[INFO   ] [Window      ] Provider: sdl2(['window_egl_rpi'] ignored)
[INFO   ] [GL          ] OpenGL version <3.0 Mesa 11.2.0>
[INFO   ] [GL          ] OpenGL vendor
[INFO   ] [GL          ] OpenGL renderer <Mesa DRI Intel(R) Haswell Mobile >
[INFO   ] [GL          ] OpenGL parsed version: 3, 0
[INFO   ] [GL          ] Shading version <1.30>
[INFO   ] [GL          ] Texture max size <8192>
[INFO   ] [GL          ] Texture max units <32>
[INFO   ] [Window      ] auto add sdl2 input provider
[INFO   ] [Window      ] virtual keyboard not allowed, single mode, not docked
/usr/local/lib/python2.7/dist-packages/kivy/core/clipboard/clipboard_gtk3.py:14: PyGIWarning: Gtk was imported without specifying a version first. Use gi.require_version('Gtk', '3.0') before import to ensure that the right version gets loaded.
from gi.repository import Gtk, Gdk
[INFO   ] [Clipboard   ] Provider: gtk3(['clipboard_dbusklipper'] ignored)
xclip version 0.12
Copyright (C) 2001-2008 Kim Saunders et al.
Distributed under the terms of the GNU GPL
[INFO   ] [Cutbuffer   ] Provider: xclip
[INFO   ] [CutBuffer   ] cut buffer support enabled
Traceback (most recent call last):
File "main.py", line 12, in
DesignerApp().run()
File "/usr/local/lib/python2.7/dist-packages/kivy/app.py", line 802, in run
root = self.build()
File "/home/yavuz/kivy-designer/designer/app.py", line 1713, in build
self.root = Designer()
File "/home/yavuz/kivy-designer/designer/app.py", line 206, in init
self.designer_settings = DesignerSettings()
File "/home/yavuz/kivy-designer/designer/designer_settings.py", line 49, in init
super(DesignerSettings, self).init(*kwargs)
File "/usr/local/lib/python2.7/dist-packages/kivy/uix/settings.py", line 906, in init
self.add_interface()
File "/usr/local/lib/python2.7/dist-packages/kivy/uix/settings.py", line 936, in add_interface
''

same error on me

same error:(((


Client Status Filter Ability

When performing a request to gather system_status data using the following:
kwargs = {}
kwargs["objtype"] = u'client'
kwargs["status"] = u'Leader'

response = handler_object.get(**kwargs)

When dealing with this data I would like to be able to review only the data from the systems that have reported into the server in the last X number of registration periods (default is 4 minutes but I should be able to filter upon any set amount of time).
I found a way around this by doing the processing locally using the following but this is not optimal as it forces me to do post-processing of data:
day_ago = datetime.datetime.utcnow() - datetime.timedelta(days=1)
day_ago.isoformat()[:19]

client_list = []
for each in out['client_status']:
	client_time = datetime.datetime.strptime( each['last_registration'], "%Y-%m-%dT%H:%M:%S" )
	if day_ago < client_time:
		client_list.append(each['host_name'])

So there is an example of this in the repository:
https://github.com/tanium/pytan/blob/master/EXAMPLES/POC/get_client_filter.py
Please look through that and if you have any questions let me know. This script uses a server side feature called cache_filters to restrict what results come back. It's very powerful, but not documented anywhere nor really used at all in the current version of pytan (other than this proof script).


Hardware keyboards don't work properly, pressing return doesn't send message

Describe the bug
Hardware keyboards (bluetooth keyboards or a built-in phone hw keyboard, tested both) don't work properly, pressing return doesn't send message. That is very annoying because it requires reaching out to the screen just to send, which involves taking off the hands of the keyboard and greatly interrupts typing flow
To Reproduce

Open rocketchat android app and open a chat
Type a message on a hardware keyboard(!) and press the return/enter key

Expected behavior
Message is sent off. Instead, it starts a newline (which is really not what happens on desktop either! I would understand for shift+return, but as a default for return...? also, shift+return doesn't send either, I haven't actually found any way to send a message using the hardware keyboard on android)
Devices and Versions

Your Rocket.Chat.Android version: 3.1.1 (f-droid)
Your Rocket.Chat Server version: no idea, but I think it affects all servers

Mobile device model and OS version: gemini pda - android 7.1.1
Additional context

I have problem with rocketchat on mobile app. my problem is first user login mobile app cann't send any message to other user until that user login on web or desktop app to active user then we can send message normal on the mobile app.
Rocketchat  version : 0.72.2


Automated cherry pick of #9405: Hotfix/webconsole trim url

Cherry pick of #9405 on release/3.4.
#9405: Hotfix/webconsole trim url

Codecov Report

Merging #9408 (5364ba5) into release/3.4 (fc16c37) will decrease coverage by 2.63%.
The diff coverage is 0.57%.

@@              Coverage Diff               @@
##           release/3.4   #9408      +/-   ##
==============================================
- Coverage         6.19%   3.55%   -2.64%
==============================================
  Files              761     306     -455
  Lines           118405   62896   -55509
==============================================
- Hits              7330    2235    -5095
+ Misses          110412   60507   -49905
+ Partials           663     154     -509

Flag
Coverage Δ

aFlag
3.55% <0.57%> (-2.64%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/cloudcommon/notifyclient/notify.go
1.78% <0.00%> (-0.03%)
:arrow_down:

pkg/compute/models/cloudaccounts.go
0.47% <0.00%> (-0.02%)
:arrow_down:

pkg/compute/models/cloudregions.go
1.52% <0.00%> (-0.02%)
:arrow_down:

pkg/compute/models/cloudsync.go
0.00% <0.00%> (ø)

pkg/compute/models/dbinstance_skus.go
2.79% <0.00%> (-0.04%)
:arrow_down:

pkg/compute/models/domainquota.go
23.80% <0.00%> (ø)

pkg/compute/models/elasticcache_skus.go
3.13% <0.00%> (-0.05%)
:arrow_down:

pkg/compute/models/elasticipresource.go
0.00% <0.00%> (ø)

pkg/compute/models/elasticips.go
1.07% <0.00%> (-0.02%)
:arrow_down:

pkg/compute/models/guest_actions.go
0.00% <0.00%> (ø)

... and 465 more

Continue to review full report at Codecov.

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


Training transcoder Generates same token at the beginning of training

When I initiate the transcoder training, for the BT step it shows generated source sent: with the same repeated token =. Just wanted to check if this is an expected behaviour or am I wrong somewhere? Also, if you could share a dummy train.log file of an already trained model here for my reference it would be helpful.
INFO - 06/02/22 02:18:54 - 0:00:31 - Generated source sent: </s> = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = = =  </s>
INFO - 06/02/22 02:18:54 - 0:00:31 - Target (x1) sent: </s> class var4_class { int var5 = 5 ; double var6 = 4.62 ; } public class java_4_20_23 { public static void main ( String [ ] args ) { int var0 = 3 ; double var1 = 4.57 ; String var2 = " vahjj " ; double var3 = 1.37 ; var4_class var4 = new var4_class ( ) ; int var7 = 1 ; int var8 = 4 ; var8 = var7 * var7 ; var1 = var3 ; System . out . println ( " var0 = " + var0 ) ; System . out . println ( " var1 = " + var1 ) ; System . out . println ( " var2 = " + var2 ) ; System . out . println ( " var3 = " + var3 ) ; System . out . println ( " var5 = " + var4 . var5 ) ; System . out . println ( " var6 = " + var4 . var6 ) ; System . out . println ( " var7 = " + var7 ) ; System . out . println ( " var8 = " + var8 ) ; } } </s>

Hi,
Yes it's normal at the beginning of training. The model hasn't been trained to generate correct sentences yet and tends to generate bad sentences. After more updates, it should start generating correct-looking programs and improve afterwards.


Turn off hydro cost curves

The current cost curves go up very steeply for large hydro. That is, while the starting CAPEX for Indonesia is 3300 $ /kW, the start of the simulation already has a 10,000 $/kW CAPEX due to the cost curves, making the price unrealistically high (>$200 / MWh)  and at odds with published estimates of large hydro LCOE (typically under $100 / MWh). I see two options:

We turn it off now and redesign later.
We wait till we have time and redesign it.

Given the long list of tasks, I propose we go for option 1.

Agreed in meeting that we are are turning this off.


Errors in selective_scan_cuda_core.fwd()

When running python vmamba.py, the following errors are raised (ubuntu18.04+torch2.1+cuda12.1):

for check_vssblock() and check_vssm1_ssoflex_equals_mambassm():

for print(VSSM(forward_type("v0")).flops()):

for print(VSSM(forward_type("v2")).flops()) and print(VSSM(forward_type("v2nozact")).flops()):

Is the gcc in your computer support c++14 or c++17? try update your gcc and build selective_scan again.


make every thing be better

as programmer & developer, should have B Tree in heart every moment.

hi,what can i do for u?

as everyone knows, the HTML is best programming language in this world, so i suggest rewrite this use HTML, i will commit a patch for this and hope you can merge :)


WIP: Interfaces

Moved from Pull Request #81
See work-in-progress/interface.gr for some ideas on syntax/semantics to be implemented.
Todo
Typechecking

[ ] ensure kinds of interface-constrained type-variables match with instance head
e.g., interface {Foo a} => Bar (a : Type -> Type), where interface Foo (a : Type) should not kind-check

Evaluating
Syntax

[x] interfaces interface NAME a where
[x] instances instance IFACE NAME where
[x] constraints

[x] in functions: f : forall (a : Type) . {IFACE a} => a -> a
[x] in interfaces: interface (IFACE a) => NAME a where
[x] in instances: instance (IFACE a) => IFACE (NAME a) where
[x] multiple: e.g., f : forall (a : Type) . {IFACE0 a, IFACE1 a} => a -> a

Possible Future Features

[ ] named instances

[ ] instance IFACE DAT named NAME syntax
[ ] instance IFACE DAT using NAME syntax
[ ] instance IFACE DAT named NAME using NAME syntax

@dorchard I'm closing this so it isn't just hanging around.


Fixing issue #53 - added Head component to Header.js

I've added the head component to the Header.js file, passed the title as props and have removed the imports of Head wherever not used in the pages directory.

Thanks @rkoalsi . I will merge PR once https://github.com/prixite/prixite.github.io/pull/56 is merged.

@rkoalsi we have merged https://github.com/prixite/prixite.github.io/pull/56. Can you please get the latest changes from upstream/master?

@umairwaheed task complete from my end.


segfault running partitioning example

Running the 07_partitioning example I am getting a segfault. Here is a back trace:
eduroam-255-150:07_partitioning nwatkins$ lldb ./partitioning
Current executable set to './partitioning' (x86_64).
(lldb) run
Process 14962 launched: './partitioning' (x86_64)
Running daxpy for 1024 elements...
Partitioning data into 4 sub-regions...
Initializing field 0 for block 2...
Initializing field 0 for block 1...
Initializing field 0 for block 3...
Initializing field 0 for block 0...
Initializing field 1 for block 0...
Initializing field 1 for block 1...
Process 14962 stopped
* thread #1: tid = 0xf81c8, 0x000000010001a000 partitioning`LegionRuntime::LowLevel::Runtime::get_reservation_impl(this=0x0000000101001200, r=Reservation at 0x00007fff5fbfdae8) + 352 at shared_lowlevel.cc:4683, queue = 'com.apple.main-thread, stop reason = EXC_BAD_ACCESS (code=EXC_I386_GPFLT)
    frame #0: 0x000000010001a000 partitioning`LegionRuntime::LowLevel::Runtime::get_reservation_impl(this=0x0000000101001200, r=Reservation at 0x00007fff5fbfdae8) + 352 at shared_lowlevel.cc:4683
   4680         assert(r.id != 0);
   4681         assert(r.id < reservations.size());
   4682 #endif
-> 4683         ReservationImpl *result = reservations[r.id];
   4684         PTHREAD_SAFE_CALL(pthread_rwlock_unlock(&reservation_lock));
   4685         return result;
   4686     }
  thread #3: tid = 0xf82b0, 0x000000010001a000 partitioning`LegionRuntime::LowLevel::Runtime::get_reservation_impl(this=0x0000000101001200, r=Reservation at 0x0000000100e00948) + 352 at shared_lowlevel.cc:4683, stop reason = EXC_BAD_ACCESS (code=EXC_I386_GPFLT)
    frame #0: 0x000000010001a000 partitioning`LegionRuntime::LowLevel::Runtime::get_reservation_impl(this=0x0000000101001200, r=Reservation at 0x0000000100e00948) + 352 at shared_lowlevel.cc:4683
   4680         assert(r.id != 0);
   4681         assert(r.id < reservations.size());
   4682 #endif
-> 4683         ReservationImpl *result = reservations[r.id];
   4684         PTHREAD_SAFE_CALL(pthread_rwlock_unlock(&reservation_lock));
   4685         return result;
   4686     }
(lldb)

Could you elaborate the system you were using when you hit this bug? I'm not able to reproduce this bug. Also, was the SHARED_LOWLEVEL variable set to 1 in Makefile?

One more thing to ask. What was the commit id you checked out?

Hi @magnatelee this almost certainly was using the shared low-level run-time, and the commit would have been from around the time of the comment, but this was from about 6 months ago so I'm not exactly sure. I'll close this out and report anything new I find.


Missing authority checks

The reports zamp_metrics_provider and zamp_clear_zamp_store should have an authority check.

"should have" --> must have :-)


memory leak with unstable connections from peers

Hi there,
As previous issue #2336, we have witnessed memory leaks eventually as result of memory exhaustion. This issue is the case with current version v0.14.3
However, the connection from the peers in our testing environment is not so stable, and here is the log on this circumstances:
ERROR[2020-02-27|16:28:17.855+0800|] p2p/server.go:975                         Failed RLPx handshake                    addr=<IP_ADDRESS>:32901                      conn=dyndial err="AcceptStream net io.reader timeout"
INFO [2020-02-27|16:28:17.855+0800|] p2p/server.go:951                         Setting up connection failed             addr=<IP_ADDRESS>:32901                      err="AcceptStream net io.reader timeout"
INFO [2020-02-27|16:28:17.855+0800|] p2p/dial.go:310                           Dial error                               task="dyndial 70d8c54d2dcc4d49 <IP_ADDRESS>:32901" err="AcceptStream net io.reader timeout"
INFO [2020-02-27|16:28:30.546+0800|] consensus/ucon/timer_mgr.go:376           Timer: time out.                         Round=194587 RoundIndex=682  MaxRoundIndex=682
INFO [2020-02-27|16:28:57.585+0800|] consensus/ucon/timer_mgr.go:376           Timer: time out.                         Round=194587 RoundIndex=683  MaxRoundIndex=683
INFO [2020-02-27|16:29:24.566+0800|] consensus/ucon/timer_mgr.go:376           Timer: time out.                         Round=194587 RoundIndex=684  MaxRoundIndex=684
ERROR[2020-02-27|16:29:47.568+0800|] p2p/server.go:975                         Failed RLPx handshake                    addr=<IP_ADDRESS>:32901                      conn=dyndial err="AcceptStream net io.reader timeout"
INFO [2020-02-27|16:29:47.568+0800|] p2p/server.go:951                         Setting up connection failed             addr=<IP_ADDRESS>:32901                      err="AcceptStream net io.reader timeout"
INFO [2020-02-27|16:29:47.568+0800|] p2p/dial.go:310                           Dial error                               task="dyndial 70d8c54d2dcc4d49 <IP_ADDRESS>:32901" err="AcceptStream net io.reader timeout"

The memory used ramp up quickly and only one day later the total memory consumption is merely 3G.
Here is the pprof heap statistics:
Showing top 10 nodes out of 212
      flat  flat%   sum%        cum   cum%
     388MB 13.74% 13.74%      388MB 13.74%  github.com/syndtr/goleveldb/leveldb/memdb.New
  312.33MB 11.06% 24.79%   329.33MB 11.66%  github.com/lucas-clemente/quic-go.(*session).postSetup
  249.36MB  8.83% 33.62%   249.36MB  8.83%  github.com/lucas-clemente/quic-go.init.0.func1
  238.95MB  8.46% 42.08%   238.95MB  8.46%  github.com/syndtr/goleveldb/leveldb/util.(*BufferPool).Get
  141.77MB  5.02% 47.10%   167.78MB  5.94%  github.com/lucas-clemente/quic-go/internal/handshake.newCryptoSetup
  138.52MB  4.90% 52.01%   138.52MB  4.90%  crypto/aes.newCipher
  127.52MB  4.51% 56.52%   127.52MB  4.51%  reflect.New
  125.01MB  4.43% 60.95%   125.01MB  4.43%  math/big.nat.make
  122.50MB  4.34% 65.28%   261.01MB  9.24%  gitlab.iyouchain.com/youchain/go-youchain/rlp.decodeBigInt
   92.03MB  3.26% 68.54%    92.03MB  3.26%  crypto/aes.(*aesCipherGCM).NewGCM

Below is the heap profile on the testing node:
pprof.you-linux-amd64.alloc_objects.alloc_space.inuse_objects.inuse_space.033.pb.gz
This is quite similar to the situation of previous issue on version v0.11.1. So could you please help to watch on this scenario:  what is the performance of quic when handling unstable connection from neighbored peers?

When such an error occurs, do you release all resources associated with this session? It's important that you don't hold any pointer to the Session, as well as to any of the streams you might have opened on or accepted from that session.

Hi @marten-seemann, thanks for your instant feedback. This case reoccurrence was based on the scenarios when some peers with poor connection, i.e. this poor peers were in the DHT table after node discovery, however, the handshake between peers were not established at all.
Per the implementation of quic-go to the session, is there any best practice on it? We have obeyed the original guide on it, but not know whether it is updated since then. The code path could be seen from the pprof profile.

The code path could be seen from the pprof profile.

Which pprof command are you running to see that?

This case reoccurrence was based on the scenarios when some peers with poor connection, i.e. these poor peers were in the DHT table after node discovery, however, the handshake between peers were not established at all.
Per the implementation of quic-go to the session, is there any best practice on it?

quic.Dial won't return before the QUIC handshake completes. So I guess the problem occurs with connections that have been establish and then fail for some reason (maybe a connection timeout).

Hi @marten-seemann : please first unzip the .gz file attached above. Then you can use the pprof tool as below example:
go tool pprof -http ":8848" pprof.you-linux-amd64.alloc_objects.alloc_space.inuse_objects.inuse_space.032.pb

Thank you @ewagmig.
I'm wondering, which version of quic-go are you using. session.postSetup was removed in https://github.com/lucas-clemente/quic-go/pull/2185, some time between v0.12.1 and v0.13.0, and not introduced since. So it looks like you're running v0.12.1 or earlier.

Hi @marten-seemann, thanks for your feedback. We will double check the quic version we are using.
Will come to you later~

This case reoccurrence was based on the scenarios when some peers with poor connection, i.e. these poor peers were in the DHT table after node discovery, however, the handshake between peers were not established at all.
Per the implementation of quic-go to the session, is there any best practice on it?

quic.Dial won't return before the QUIC handshake completes. So I guess the problem occurs with connections that have been establish and then fail for some reason (maybe a connection timeout).

Hi, @marten-seemann this clue is quite important to us with current scenario:

the enchandshake has already been established
the following protocol handshake failed due to some reason, just like this log from the testing node:

So I'm wondering how quic handle on this situation, or what step should be emphasized to prevent this connection from continuing?

I don’t see any quic-go error message in this screenshot.

Yes, there were no error messages from this screenshot printing the proto handshake phase error. There is no relation with the quic. I was just  recalled with what you mentioned on the quic.Dial return mechanism.
We have deployed testing node with older version by mistake and replaced the one with version v0.14.3. The testing scenario keeps the same and until now everything goes well.
We will keep watching the new quic performance against such poor connection.

Hi @marten-seemann We have replaced the quic version with latest one, i.e. v0.14.4 and run the testing nodes under previous connection environment. 2 days have passed and there was no outstanding proof on quic.
Here is the pprof from latest quic version, F.Y.I.
pprof.you-linux-amd64.alloc_objects.alloc_space.inuse_objects.inuse_space.042.pb.gz
Hope this could give positive feedback on quic handling connection.

Happy to hear that the memory leak has apparently been fixed! Thanks for reporting!

@marten-seemann ,  I would like to reopen this issue.
I would like to mention that I am still experiencing the same issue described in this closed issue.
Despite the closure of the issue, I have encountered the memory leak problem consistently. I have not been able to find a resolution or workaround for it yet.
I kindly request reopening the issue or providing further guidance on how to address this persistent memory leak. Any assistance or insight into this matter would be greatly appreciated.
Thank you for your attention and support.
I have used version 0.35.1 of the codebase.
File: webService.run
Build ID: 046cd5ef5de1f12c5d4b70dbb96ebafb3be85a33
Type: inuse_space
Time: Jun 3, 2023 at 4:22pm (+0330)
Entering interactive mode (type "help" for commands, "o" for options)
(pprof) web
(pprof) top
Showing nodes accounting for 388.27MB, 86.67% of 447.98MB total
Dropped 194 nodes (cum <= 2.24MB)
Showing top 10 nodes out of 116
      flat  flat%   sum%        cum   cum%
  201.79MB 45.04% 45.04%   201.79MB 45.04%  github.com/quic-go/quic-go.init.0.func1
   58.38MB 13.03% 58.08%    58.38MB 13.03%  github.com/webservice/utility.ReadHTTPDataBodyResponse
   32.08MB  7.16% 65.24%    32.08MB  7.16%  github.com/smallnest/goframe.(*lengthFieldBasedFrameConn).ReadFrame
   29.01MB  6.48% 71.71%    30.01MB  6.70%  github.com/quic-go/quic-go.newConn
   15.77MB  3.52% 75.23%    15.77MB  3.52%  regexp.(*Regexp).get
   13.22MB  2.95% 78.18%    15.36MB  3.43%  compress/flate.NewWriter
   11.50MB  2.57% 80.75%    11.50MB  2.57%  runtime.malg
   11.50MB  2.57% 83.32%    11.50MB  2.57%  golang.org/x/net/internal/socket.(*mmsghdrsPacker).prepare
    8.01MB  1.79% 85.11%     8.01MB  1.79%  runtime.allocm
       7MB  1.56% 86.67%    41.27MB  9.21%  github.com/quic-go/quic-go.(*Transport).init.func1

Please open a new issue if you think you’ve found a bug. We’d need a few more infos though to actually debug this. It’s expected that a sync.Pool would hold some memory, that’s that it’s there for.


Feature:

@Abrar-2510 what type of features do you want to add to this website. I am good at front-end so, I can help with this issue.

@Abrar-2510 can you please teel the issue.


Push emulator stuff back to Adam's repo.

Pulled into my emu branch instead, for now
See https://github.com/AEFeinstein/Super-2021-Swadge-FW-Sandbox/pull/9


Extract file info for packages

Motivation
See issue #781
Proposed changes
This is a continuation of @PrajwalM2212 's pr #573. I tried to address the latest comments and change requests made by @nishakm on that pull request.
Signed-off-by: Yann Jorelle<EMAIL_ADDRESS>

I was thinking of merging the file data obtained when running tern with scancode here with the package file data. But then I noticed that when you run tern with scancode, it doesn't seem to run the scripts in the command library that get the installed packages.
This is a bit confusing to me because in this case scancode isn't really an extension, as it doesn't extend the metadata obtained from running the command library scripts. In my opinion it would make more sense if when running tern with scancode, it would first run the command library scripts and then run scancode to fill in the file metadata.
Does this make any sense, or am I missing something :blush: ?

This is a bit confusing to me because in this case scancode isn't really an extension, as it doesn't extend the metadata obtained from running the command library scripts. In my opinion it would make more sense if when running tern with scancode, it would first run the command library scripts and then run scancode to fill in the file metadata.
Does this make any sense, or am I missing something blush ?

Hi @yannjor, sorry for getting back to you on this so late. Tern extensions are just a way to run any analyzer that is not "container aware" on a container image and to collate the results in any of the supported formats. I designed it this way because a lot of file analyzers are written in other languages and support different environments. For example, scancode cannot be installed with python 3.7 or 3.8 right now, so it cannot be used as a library in Tern itself. However, if python 3.6 was installed with tern and scancode, invoking scancode is not a problem. There is an issue to incorporate the package data that scancode produces #790. I am not sure if scancode can actually map what files came with what package installation though. This is why we treat each layer as a package with files in them.
The second problem is how do we resolve data pulled in from scancode (or any other extension) and what is already available in the cache. For example, the script method finds license MIT associated with a file, but scancode finds BSD. Another easier case is the script method was not able to find the license for a file associated with a package but scancode was able to find it for the same file. The metadata needs to be consolidated, not duplicated.

@yannjor I will take a look at this, and your other PR this week. Again, apologies for the delay 😅

@nishakm Ok, no worries :blush:

Right now, there are FileData object that automatically generated at the ImageLayer level. I wonder if you could also add some functionality to remove the duplicate FileData object at the ImageLayer level that is found in the package level.

I added a function for this in common.py that gets called by add_base_packages. Do you think this is a good place to do this?

@yannjor I tested your changes and it looks like the files that get collected don't get into the report.
Eg:
1. Run tern -r report -f html -i debian:buster -o report.html
2. Open report.html with a browser

Look at layers -> packages -> files you will see all files are 0.

@nishakm

I tested your changes and it looks like the files that get collected don't get into the report.

I think the probem is with the base.yml scripts, but I can't really figure out how to fix it. It used to work before my latest changes to it.
Previously I had this (what @PrajwalM2212 wrote):
  files:
    invoke:
      1:
        container:
        - "pkgs=`dpkg --get-selections | cut -f1 -d':' | awk '{print $1}'`"
        - 'for p in $pkgs; do files=`dpkg-query -L $p`; for file in $files; do if [ -f $file ]; then echo $file; fi; done; echo LICF; done'
    delimiter: 'LICF'

I then changed the delimiters to "\n" and removed the echo LICF, because otherwise the output would be a string and I would have to manually split it into a list in package.py:
        raw_file_list = package_dict.get('files', False)
        if raw_file_list:
            file_list = []
            for file_path in raw_file_list.split('\n'): # <--- Here
            ...
            package_dict['files'] = file_list

I noticed that most of the other scripts use awk to print stuff. Maybe it could be used here too? I'm don't have much experience in bash scripting so I would appreciate some help :blush: I am also fine with you working on top of this if you prefer.

@nishakm

I tested your changes and it looks like the files that get collected don't get into the report.

That problem came from a change I had made inbase.yml in an attempt to make the package file extraction part more readable. I reverted the change so it should work now. The lines that fetch the files for the packages are still really long (about 150 characters) and not very readable in my opinion. Maybe this YAML multi-line string syntax could be used?
I noticed another problem with the deletion of duplicate files that are found both at the layer and package level. The file paths for layer files seem to be relative paths from the root directory (eg: etc/file), while the package files use absolute paths right now. Should we change the layer files to use absolute paths instead or change the package files to use relative ones? Here is the code in image_layer.py that creates the layer files.

@yannjor Sorry again for the delay. These changes looks good so I will merge the PR.


feat: allow ability to lock fee module in presence of severe bug

Description
ref #821 #860 #1031

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards.
[x] Wrote unit and integration tests
[x] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[x] Added relevant godoc comments.
[x] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[x] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #1239 (49588fa) into main (c7209a5) will decrease coverage by 0.00%.
The diff coverage is 81.48%.

@@            Coverage Diff             @@
##             main    #1239      +/-   ##
==========================================
- Coverage   80.01%   80.01%   -0.01%
==========================================
  Files         166      166
  Lines       11916    11941      +25
==========================================
+ Hits         9535     9555      +20
- Misses       1920     1925       +5
  Partials      461      461

Impacted Files
Coverage Δ

modules/apps/29-fee/types/keys.go
96.66% <0.00%> (-3.34%)
:arrow_down:

modules/apps/29-fee/keeper/keeper.go
91.84% <50.00%> (-1.42%)
:arrow_down:

modules/apps/29-fee/ibc_module.go
92.10% <100.00%> (+0.67%)
:arrow_up:

modules/apps/29-fee/keeper/msg_server.go
78.04% <100.00%> (+3.04%)
:arrow_up:

I guess the only slightly confusing thing, is the subtle difference between locked vs disabled.

I guess the only slightly confusing thing, is the subtle difference between locked vs disabled.

Good point, although hopefully the locked functionality is never referred to outside of by core developers. Do you have any suggestions? At the very least I can reference my ADR in the code documentation


ci: unified crates.io release workflow using release-plz

Overview
Use a common unified CI for Rust release using release-plz.

Unified workflow source code

Details

The workflow is similar to the release-please workflow as it creates a release PR with expected changes, monitors next semver, updates changelogs etc. but it's more specialized for Rust packages because it supports crates.io publishing out of the box, additional breaking API checks using cargo-semver-checks, easier to configure and support (for example, only one workflow file instead of two).
The behavior is configurable through release-plz.toml file in the workspace root. Comments added to each line to explain what each setting does.

Current restrictions
As release-plz does not provide the same output context as release-please, notifications for successful releases are not sent yet in the same format. It would require refactoring of the format-release-please-for-slack-action or simplier, just use GitHub release notifications in Slack that will not require any set up.

@popzxc , I also checked that the bot user has the correct permissions for this repository, so we can proceed with the merge and the final stage of testing.

Some more fixes are in progress.


Validate userns container config is consistent with sandbox userns config

This is a follow-up to address one item of this comment by @fuweid: https://github.com/containerd/containerd/pull/7679#pullrequestreview-1231373767
The first commit fixes an issue running the tests (assert vs require, so the function result can be nil by mistake instead of failing early).
The second commit is the follow-up itself. I didn't use reflect, as it might be too slow to use on the container creation path. But this means we need to update sameUsernsConfig() if the runtime.UserNamespace type changes. Let me know if you prefer to use reflect here.
The third commit just removes a variable that is not really helpful. It is a cosmetic change, no functional change.

CI failures are unrelated, but pushing again to trigger a retest

/ok-to-test


Field types & validation

Users should be able to set the type of each column, but they should be more user-friendly types than database terms. For instance:

Friendly type
Maps to
Maps from

text
text
character, character varying, text

number
numeric
integer, smallint, bigint, numeric, decimal, real, double precision, money

checkbox
boolean
boolean

list
enum
enum

email
text w/validation
text w/validation

url
text w/validation
text w/validation

money
numeric(48,3) w/formatting
?

percent
numeric(3?,2) w/formatting?
?

I'm thinking that postgres domains might be helpful to ensure validation of custom types like email and url are enforced server (database)-side. But would that cover everything? It would be nice to use "pure postgres features" whenever possible for portability.

To get a list of types from postgres:
select format_type(oid,null) from pg_type where typnamespace=(select oid from pg_namespace where nspname='pg_catalog') and typelem=0 and typrelid=0;

thx @rhodiumtoad!

Type mapping is now abstracted by the back-end, and it's ready for the client to start modifying it and using it.

Now that the column-types branch is basically done, I'm asking myself what the real value-add is? I think the only real "feature" from an application standpoint is the validation that's inherently provided.. but there's certainly other ways to go about that. There is value in using the underlying database from another application, but I wonder if that's outweighed by the difficulty in changing a column's type (if invalid values are present).


Codegen is still broken for associative tables

Thank you for the changes on Issue #50 , but the changes expose a new issue. usersRoles[i] is nil, so assigning RoleID and UserID cause a panic due to the nil pointer dereference. I have to add usersRoles[i] = &UsersRoleSetter{} as the first line in the for loop to fix the nil assignment issue.
	usersRoles := make([]*UsersRoleSetter, len(related))

	for i, rel := range related {
		usersRole := usersRoles[i]
		usersRole.UserID = omit.From(o.ID)
		usersRole.RoleID = omit.From(rel.ID)
	}

Try 8b1005 and see if it is fixed there

Released in v0.21.0


[New Feature]: Allow type inclusion of readonly records

Description
Currently, the following is not allowed:
type A record {
    string a;
} & readonly;

type B record {
    *A;
} & readonly;

The following compiletime error is thrown:
incompatible types: 'A' is not a record(BCE2091)

It would be great to allow type inclusion of records that are also readonly.
Describe your problem(s)
No response
Describe your solution(s)
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

We plan on allowing this as suggested in https://github.com/ballerina-platform/ballerina-spec/issues/1224.

Closing this issue since the corresponding spec issue is still open. We will introduce changes to the implementation once the spec change is done.


test: add uninstall_pipelines_operator function to CI script

Introduced the uninstall_pipelines_operator function to remove the OpenShift Pipelines operator and associated resources. This ensures that any existing pipeline resources are cleaned up before new installations, preventing potential conflicts and ensuring a clean testing environment.
Description
Please explain the changes you made here.
Which issue(s) does this PR fix

Fixes #?

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Close in favor of https://github.com/janus-idp/backstage-showcase/pull/1861


Don't wire through manufacturer labels in schematic

This requires moving the manufacturer labels as a schematic_text core output, rather than inserting inside of circuit-to-svg

/bounty $30

@seveibar
I've created a function that handles the chip manufacturer's label and chip name, and converts them into a schematic-text element. How can I modify the trace so it does not get passed to the schematic-text?

Inside Trace.ts schematic_text will need to be added as an obstacle

@seveibar
this is after added schematic_text to obstacle

nice that looks correct, the trace is no longer going through the schematic trace

we should also make it not go through the symbol 🤔

we should also make it not go through the symbol 🤔 (probably a separate PR i guess)

It's already added to obstacle


Image::blend_rect might not blend as expected

Godot 3.1.1
tl;dr: Image::blend_rect() may cause transparent regions to darken, contrary to what you would get with a drawing app.
Explanation:
When trying to work on a painting app in which you can paint on transparent images using shaders and a transparent viewport, I got to deal with alpha blending. I tried blending a diagonal white-to-transparent-white gradient with the Godot icon, and quickly stumbled on the following problem:
Expected (Paint.NET):

Obtained (Godot):

Notice the darkened corner. From a pure math standpoint, it's just the result of averaging colors from the "invisible" pixels of the destination image, ending up darkening the gradient. But it may not be the expected result, at least to me it wasn't.
So I went bulldozer mode and blended myself in shaders, coming up with this:
vec4 blend_alpha(vec4 a, vec4 b) {
	vec4 res;
	res.rgb = b.a * b.rgb + a.a * a.rgb * (1.0 - b.a);
	res.a = b.a + a.a * (1.0 - b.a);
	res.rgb /= res.a; // THIS
	return res;
}

That last division did the trick.
But then, looking at Image::blend_rect, I noticed it does the same thing, except the division. So I tested it, and it has the same issue.
So is this really expected or should it be patched?

Maybe an additional parameter could determine what to do with alpha.

Image::blend_rect Works for me (opening this image on github will show the transparent areas as black color)

Sorry I should add a reproduction project because it really didn't work as expected for me.

ImageAlphaBlend.zip
Run main.tscn, then see the result in the file explorer: you will notice it will have dark corners, while the expected result doesn't.

pos(62,2) shows #132e3f A7 on expected.png (tested with aseprite)
pos(62,2) shows #000101 A7 on icon.png (tested with aseprite)
looks like the expected.png also has an error
Which leads to this solution
https://en.wikipedia.org/wiki/Alpha_compositing#Alpha_blending
https://github.com/godotengine/godot/blob/7e9c1041ac1d30c7620713635a76ba4caf29d673/core/image.cpp#L2207
double out_a = (double)(sc.a + dc.a * (1.0 - sc.a));
dc.r = (double)(sc.a * sc.r + dc.r * dc.a * (1.0 - sc.a))/out_a;
dc.g = (double)(sc.a * sc.g + dc.g * dc.a * (1.0 - sc.a))/out_a;
dc.b = (double)(sc.a * sc.b + dc.b * dc.a * (1.0 - sc.a))/out_a;
dc.a = (double)(sc.a + dc.a * (1.0 - sc.a));

pos(62,2) shows # 132e3f A7 on expected.png (tested with aseprite)
pos(62,2) shows # 000101 A7 on icon.png (tested with aseprite)

I think that difference is meaningless since that pixel is located in a transparent area. Godot saves PNGs without compressing the transparent areas, while my painting software does.

Yes it looks like it is not working properly.

I've tried the solution above, testing with pixelorama to see results, and indeed it works better.

Photoshop style blending is a bit different to game style blending, as you need to deal with situations like blending onto a transparent background, whereas usual game blending is designed to be fast. There are also other tradeoffs with speed and accuracy. So if you want to support this you might be best off with a separate routine / code path for the photoshop style blending.
Of course if you wanted to make photoshop like app you would probably need other blending modes too. And consider the colour spaces, doing your blending in linear for instance.
In practice for anything except simple apps I think you'd need to write some custom code in c++ for such an app.

So, this could be addressed adding a 4th param, something like a bool fix_alpha or something, default to false. And if that parameter is ture, use the formula from above... Would that change have a chance to get merged in master?

@azagaya Sounds good to me. Personally, I'd name the parameter accurate_alpha or something like that to imply it results in more "correct" visuals at the cost of performance.

@Calinou And should the same be implemented in blend_rect_mask()?

@azagaya Probably, it makes sense to implement it there if it can be done easily enough.

Also, now that i'm there, if the source pixel is fully transparent, we could avoid blending the colors, as it should result in the destination color anyways... similar to how mask works in blend_rect_mask() we could avoid calculations if sc.a == 0

Myself I would like to advocate replacing the old behaviour completely. My reasoning being:

for opaque destination it works the same
for transparent destination blending white transparent source repeatedly will produce black image instead of slowly fading to white and who would ever want that? That is borked premultipled alpha behaviour where conversion is applied every time! It does nothing right, imho very clearly bugged.
someone said the additional division is cost. Well, the original algorithm already does quite a bit of work, like constructing new objects for every pixel in the image for whatever reason, so that is not a meaningful factor
for proper blending mode choice you would probably want to implement multiple blend modes in Color class, which btw has blend() function which properly blends alpha and should probably be used in Image blend functions to cut down unnecessary code

Overall it seems to be a very simple bug and until a proper rich blending library is maybe made someday, replacing the current behaviour with Color.blend() is the most sane and correct solution which adds less lines than it removes.

That makes sense to me. If some core contributor is ok with that, i can replace the PR to do exactly what you say.
I didn't know about the Color.blend() function.

I've tryed and seems to work perfectly fine with Color.blend()


Add rsa keypairs with correct (ans.1 der) encoding

basically add a sane way to generate RSA keys and encrypt/decrypt based on those keys

Should be fixed and errors handled now

ill merge into the current code for now and we can move it over when the rewrite is complete

nevermind i didn't read the code right lol, i though it was for the old code

conflicts in
src/net/encryption/Cargo.toml
src/net/encryption/src/lib.rs
@en1gma713
fix these and I'll merge it

the rewrite's entire file structure updated before i could get this in. closing this and submitting a new one with the correct structure.


List all missing controls on XML deserialisation

Each time we type a wrong value in the XML and the engine bug instead of say what's wrong, we have to report it to create a control if possible.

Hi,
it's possible to translate? :bowtie:

done !

hi, can you explain it in more detail? I dont understand what is to do here. What do you mean with control?

When a user write an XML definition, he may make a lot of "mistakes", for exemple :

Text case problem
a value out of range
a value unknown by the enigne
etc.
From typing errors or else. The code that read the definitions and create the bluprints (Builders) has  too few checks and may let errors pass throught.

For exemple, if you type an unknown element name, the code won't use its value and the defaut value will be used instead, leading to hard-to-source bugs.
We should found a way to solve that problem. The code must detect errors and throw warnings, or maybe there is another solution?

I prepare a EntityEditor (see here https://github.com/wuendsch/OpenRTS-EntityEditor).
One of the last steps are now, to integrate it into our current system

The EntityEnditor is now released for a first beta version. It's possible that we integrate it directly into OpenRTS repository. I close this issue now.


When adding a layer via the add layer button, allow the user to enter a name for the layer

This is useful because currently if you add a layer, and check to use it as a basemap it then appears in the basemap layer list, as just "layer" ... if you have added multiple layers as basemaps this way, this is no longer useful

Backlog item - https://app.asana.com/0/23019535413944/23192032216679


Is WikipediaQueryRun working properly?

Hello, I am trying to get results from wikipedia with WikipediaQueryRun but I am not getting any results.
Here is my code
const { WikipediaQueryRun } = require('langchain/tools');

 const tools = [
      new Calculator(),
      new WikipediaQueryRun(),
    ];

      const executor = await initializeAgentExecutorWithOptions(tools, model, {
      agentType: 'chat-zero-shot-react-description',
      verbose: true,
    });

     const input = 'What is langchain?';
     const response = await executor.call({
        input: input,
      });

Here is output:
[chain/end] [1:chain:AgentExecutor] [10.41s] Exiting Chain run with output: {   "output": "{\"lc\":2,\"type\":\"constructor\",\"id\":[\"langchain\",\"schema\",\"SystemMessage\"],\"kwargs\":{\"content\":\"Thought: The user is asking for information about \\\"langchain\\\" from Wikipedia. I should use the wikipedia-api tool to fetch this information.\\n\\nAction: \\n```\\n{\\n  \\\"action\\\": \\\"wikipedia-api\\\",\\n  \\\"action_input\\\": \\\"langchain\\\"\\n}\\n```\\n\",\"additional_kwargs\":{}}}"
Is there a documentation with every use case for the Wikipedia Agent?
Thank you

Solved with:

 const executor = await initializeAgentExecutorWithOptions(tools, model, {
      agentType: 'zero-shot-react-description', // i changed to agent type and works fine
      verbose: true,

@gkhngyk  is it possible to add memory with this agent

@gkhngyk is it possible to add memory with this agent

You can add memory to agent executor.
For example:
const executor = await initializeAgentExecutorWithOptions(
      tools,
      model,
     {
    agentType: 'zero-shot-react-description',
    memory: memory // <-- this is the memory you created above
     }

@gkhngyk  when we try to add buffermemory it shows error

Can you try buffermemory with agentType: 'chat-conversational-react-description'
-> https://js.langchain.com/docs/modules/agents/agent_types/chat_conversation_agent

[llm/error] [1:chain:AgentExecutor > 4:chain:LLMChain > 5:llm:ChatOpenAI] [338ms] LLM run errored with error: "Request failed with status code 400"
[chain/error] [1:chain:AgentExecutor > 4:chain:LLMChain] [339ms] Chain run errored with error: "Request failed with status code 400"
[chain/error] [1:chain:AgentExecutor] [4.81s] Chain run errored with error: "Request failed with status code 400"
Trace: Error: Request failed with status code 400
By using the agentType: 'chat-conversational-react-description'    getting the above error @gkhngyk


Rebuild for Python 3.7, GCC 7, R 3.5.1, openBLAS 0.3.2 #7

Fixes #7

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


clean up

This pull request includes numerous changes, none of them affecting the binary output. I'll explain them because some of them may be questionable:

5ec4f35c966d280079bb3392aacef90916b55953: there was a missing space which pissed me off enough to fix it.
142b3cb2295d5507da1ce91ddf2bc993cf8b6123: remove redundant ignores. some items should be in a local ignore, and some have no reason to be there.
8fb94c0dcf19966f8479cdbeb53b5cfe2fe510a0: license looks better as an MD file, even if not in markdown. also changed copyright because multiple contributors.
29f14ab6e693a667b7df7846c7b0a9034e1908bb: move the config example to a file without spaces. generally not a good idea to include a space in a file name, especially for a cross-platform project.
fa0148fbe1a3ae3eb344d3fe2cfa8b2389e68da9: files that should've been ignored or non-existent in the first place were removed.

@Cormanz reverted

I'm almost ready to merge, but you should remove the old LICENSE now that you added LICENSE.md.

@Cormanz im really intelligent i swear

LGTM


Get bus / queue details through the rest API

For getting the status of the queue, memory usage, and statistics something like 'GET /queues/myqueue' or similar would be helpful.

Definitely something I would like to take a look at.  Right now the default queue names are MD5 hashes of the handler type names in order to ensure uniqueness and persistence across restarts, which isn't intuitive or discoverable.  I'm going to have to rethink that so that the queue names are a little more user friendly and can survive a type rename -- possibly with a [Queue] attribute or some such.  Once that's in place then we will have much friendlier queue names that would make such an API much more useful.


Bundle_queue takes targets as [[klass, id]]

We need to pass targets to InventorySync.bundle_queue as [klass, id]
pairs not just an array of ids.

curl -X POST --user admin:smartvm http://localhost:3000/api/red_hat_migration_analytics -d '{"action": "bundle", "provider_ids": [2]}' 2>/dev/null | python -m json.tool
{
    "message": "Bundling providers ids: 2",
    "success": true,
    "task_href": "http://localhost:3000/api/tasks/5",
    "task_id": "5"
}

Ivanchuk backport details:
$ git log -1
commit e9e312293f60468c4205b018878454c618c578c9
Author: Mike Turley<EMAIL_ADDRESS>Date:   Thu Aug 15 14:40:44 2019 -0400

    Merge pull request #15 from agrare/pass_correct_targets_to_bundle_queue

    Bundle_queue takes targets as [[klass, id]]

    (cherry picked from commit 51b40d2f9da1e78d26ae594eb3687097b73533f8)


Scale testing improvements

Various improvements for scale testing, updated instructions and observations. Details in individual commits.

Updated.


[WFCORE-4559] Incorrectly formatted result produced by LocalPatchOperationTarget.info(stream)

https://issues.jboss.org/browse/WFCORE-4559

@aloubyansky thanks!


Support to disable/enable browser reload

There are some cases where a change on the project files doesn't have to refresh the current browser page. This PR aims to add the ability to disable/enable the reload easily on the same browser.
To achieve that this PR adds a small container:

Which the user can move around:
https://github.com/adamchainz/django-browser-reload/assets/75857767/ed261994-d6fc-4d88-ba13-636d3a6c3c40
And which the user can click to disable the browser reload:

Or enable it:

Thanks for the idea, but I don't think this is the best approach. Injecting a floating button for this use case seems quite overbearing: the button can get in the way, extra CSS and JS are needed, and most users likely don't care.
If you have a few URLs that you don't want to reload, you can subclass the middleware to skip those. We could document how to do that.
Alternatively, we could add a JS API to disable reloading for the current browser session. Something like "open the dev console and type djangoBrowserReload.disable()". What do you think?

Thanks for the idea, but I don't think this is the best approach. Injecting a floating button for this use case seems quite overbearing: the button can get in the way, extra CSS and JS are needed, and most users likely don't care.
If you have a few URLs that you don't want to reload, you can subclass the middleware to skip those. We could document how to do that.
Alternatively, we could add a JS API to disable reloading for the current browser session. Something like "open the dev console and type djangoBrowserReload.disable()". What do you think?

Yeah, that seems ok to me. Am closing this PR then.


Unable to initialise existing Java project

Environment

Extension Version: 1.16.1
OS: Win 11
Project Type: Java 21, Maven

Problem
When 'Azure Functions: Initialize Project for use with VS Code...' is used on an existing Java project exists and does not have a 'host.json' file at it's root, then it fails with:
11:35:32 AM: Error: Error: ENOENT: no such file or directory, open 'c:\Users\UserName\test-function\target\azure-functions\test-function-20241206112659480\pom.xml'

I believe this is because it scans the project for a host.json file and when it finds it, it wrongly assumes that's the root. As the com.microsoft.azure:azure-functions-maven-plugin generates this file it then finds it in the target/ directory.
Expected
The plugin scans for pom.xml files rather than host.json to find project root. This would be somewhat complicated by multi-module projects, but pom.xml is a safer approach than host.json.
Workaround
Just create a host.json file, even though it should be auto-generated.

Hi @dansiviter
Thanks for your suggestion. I have a follow up question though.  We assumed that if we were looking for the pom.xml file instead of the host.json, that would be searching for the root of any Java project. However, we are specifically looking for the root of the Functions app (which may be in a sub-folder of your Java project since people may have it nested within their Java app).
Is this a wrong assumption on our part?