Move a sphere on trimesh like a character

Hello,
This is not an issue, just a question.
I have a 3D terrain (using trimesh) generated by a noise, and I want to control a sphere with my keyboard like a character walking on the terrain.

In a plane, this is simple, we just change the x and y position of the sphere, but in a 3D terrain, we need to adjust the Z position according to the trimesh, and I have no idea how to do that.
I only want to use cannon.js, not three.js because I only use Cannon to calculate collisions on server side without rendering.
Any suggestions ?

If you have a simple movements then yes, @schteppe 's methods should work perfectly for you.
If you wanted a bit more complex behavior which accounts for velocities, I could also help you with that.

The boxes represent all the different vectors I'm working with.
Although my method requires cross product of vectors and rotating a vector around a specific axis, for which I'm using THREE functions. Anyways, let me know if you want me to elaborate further... 🙂

@schteppe
Yeah, but is not that simple because my terrain isn't a 2D heigthmap projected in 3D, it's a fully 3D shape with caves and floating islands (I use Trimesh, not Heightfield), like this :

So at a Z position, there can be multiple possibilities, so the raycast doesn't solve my problem.
@swift502
What a nice demo !
Yes, I'm very interested by your method because I need my character to be able to jump, rotate and fall down (like yours).
Do you think that your method will work in a complex terrain like mine ?
Is your source code in public access somewhere?

It should definitely work on any terrain.
The project is here. I'm just afraid that my implementation is way too complex for what you need. You'll need to pick out very specific things.
Raycasting is in character's physics pre step callback.
Sticking to ground is in character's physics post step callback.
The logic is divided into callbacks because of frame skipping when rendering is too slow, which is again, something you probably don't care about.
If you were confused by my setup, I could definitely give you simplified overview of what I do, with an extract of the relevant code. In a few days. I'm a bit busy right now...

This was definitively the help I needed.
I understand the basics, and I'll try to implement this in my code.
I'll let you know about my progress, thank you for taking the time to help me.

Hey @Zbluu,
I tackled the "transforming local velocity into global according to surface's tilt" problem two more times, because each time it turned out there were problems. If you had problems with that, I think I finally found the correct and quite elegant solution.
The only problem is, it's using even more THREE. And also it's using a 4D matrix, which is, as far as I'm concerned, basically magic. 😄
Just wanted to let you know, because the solution I had few days ago wasn't really sufficient for me.
 // If we're hitting the ground, stick to ground
        if(this.character.rayHasHit) {

            // Flatten velocity
            newVelocity.y = 0;

            // Measure the rotation from direct "up" vector to the surface normal
            let up = new THREE.Vector3(0, 1, 0);
            let normal = new THREE.Vector3().copy(this.character.rayResult.hitNormalWorld);
            let q = new THREE.Quaternion().setFromUnitVectors(up, normal);

            // Rotate an identity matrix
            let matrix = new THREE.Matrix4().makeRotationFromQuaternion(q);

            // Apply the matrix to appropriately rotate the velocity vector
            newVelocity.applyMatrix4(m);

            // Apply velocity
            this.velocity.copy(newVelocity);
        }

Wow man, you rock.
Before this, I used to apply a constant velocity on X and Z axis to move my character, which gave the impression that he was moving faster on a slope than on the flat. But this code resolve my problem, tanks ! 😄
And have you ever had any detection problems with the Raycast ? Like I said in my previous post, when the triangle is too small, my character passes through the ground. I don't understand why you're setting up the Raycast length to 0.61, maybe I missing something ?
It seems to me that it's more a Trimesh collision bug than a Raycast bug, but you don't use Trimesh so you probably didn't have it.
@schteppe, do you know anything about the bug that I described in my previous comment ?
Maybe it would be better if I open a new issue with some reproduct cases ?

@Zbluu I think I’ve seen this before... could you double check if it’s fixed in the most recent version of cannon? (Run grunt to rebuild the repo)

@zbluu yeah I haven't used trimesh yet, only primitives.

@Zbluu I finally got some geometry to test, and everything seems to work fine. Although trimesh seems to generally have a quite big performance impact. (Not that I didn't expect it)
Demo is here:
jblaha.art
The movement angle still looks a bit off when moving at a large slope in a diagonal direction (relative to the slope), but you don't normally notice it...

Off topic but awesome demo, congrats.

@swift502 Nice demo again.
I had issues only with very small triangles that is not the case for you, but I implemented my own raycast and it's seems to work correctly now.


Tensorflowsharp community/fourm

Is there a forum or community for asking questions re Tensorflowsharp?
I like StackOverflow, but does that get enough attention from Tensorflowsharp users?

For now, the GitHub issues look fine, but if a community develops elsewhere, I would be happy to point there.

We can use Gitter for live chat:
https://gitter.im/TensorFlowSharp/Lobby#


Update 0058-length-of-last-word.py

If I am not mistaken [::-1] creates a new (reversed) string, which would give that solution a space complexity of O(n).
Instead we should just use a backwards iterator.

File(s) Modified: 0058-length-of-last-word.py
Language(s) Used: python3
Submission URL: https://leetcode.com/problems/length-of-last-word/submissions/1277515120/

@Zirafnik Thanks for your contribution 👍


Release-2.2 rbac fixes

https://github.com/open-cluster-management/multicloudhub-operator/pull/548
https://github.com/open-cluster-management/multicloudhub-operator/pull/547

/approve

/hold

/unhold

/unhold


Payment: New Payment module

Payment module

Ability to create pay button and accept payment
Ability to create pay page with content managed line items
Pay using credit card or Wallet
Support for workflow events

Available in current latest preview


Fix LocalJumpError in Rails controllers

There were a few messups here:

The Rails controller tests never tested a "happy case" (that is, no exceptions)
The Rails controller tests didn't check to see the exception they were actually catching - so the LocalJumpError went unnoticed!
You can't yield from the block form of around_action
I'm not sure why I thought class_eval was necessary here - it isn't.

Fix #773

Can confirm this fixes the issue for me 🎉


Accessories not loading from the config.json file

Hi all
My system does not seem to recognise the Accessories that are listed in the config.json file. The plugins register OK. I have two accessories in the config file and one platform. The milights all work and controll fine. I even tried a fake accessory but no luck.
Help appreciated (running on Rpi with Jessy)
" Loaded config.json with 0 accessories and 1 platforms."

I have the same problem today.  I have homebridge running on my Windows10 PC and also running on Debian, and on RaspberryPI, but today I went to install a fresh copy on debian, and it doesn't work :(  I changed the PIN number and the username, but it had zero effect.  How can we see what config.json file is actually being loaded and any errors during load?

Just wondering, why not post the json instead of an image?
It looks like you have an extra }] on the second line from the bottom.

We really need to move away from these silly json configs. can someone write a UI5 frontend for them :)

Others have had this issue lately too.  there is too little error handling.  we need more error handling.  in my case it was this, once I displayed the real error that was being supressed by homebridge/lib/server.js
ERROR: Couldn't find a config.json file at '/root/.homebridge/config.json'. Look at config-sample.json for examples of how to format your config.js and add your home accessories.

ERROR: Couldn't find a config.json file at '/root/.homebridge/config.json'. Look at config-sample.json for examples of how to format your config.js and add your home accessories.

Looks adequate to me, glad it worked out for you.
Probably time to close the issue.

its actually a bug in homebridge.  I traced it back to this, NPM installed homebridge elsewhere, and then homebridge struggled to locate it properly. the problem homebridge code is this:
if (customStoragePath) return customStoragePath;
var home = process.env.HOME || process.env.HOMEPATH || process.env.USERPROFILE;


Add Class field to Job

So that one can distinguish between different types of Jobs.
@msneeden

👍 💯


[Evaluated] Issues in test_foreach

🐛 Describe the bug
Re-triage it by https://github.com/intel/torch-xpu-ops/commit/cbb4ab17a781c77108443f12f7ce254a345f1a14. Old issue is https://github.com/intel/torch-xpu-ops/issues/469
Unexpected success
CPU fallback fails. Implementation difference between CPU and CUDA. Expect success on CPU and expect fail on CUDA. When we use CPU fallback and align expected fail list with CUDA, these cases fail.
"test_parity__foreach_ceil_fastpath_inplace_xpu_complex128",
"test_parity__foreach_ceil_fastpath_inplace_xpu_complex64",
"test_parity__foreach_ceil_fastpath_outplace_xpu_complex128",
"test_parity__foreach_ceil_fastpath_outplace_xpu_complex64",
"test_parity__foreach_clamp_max_fastpath_inplace_xpu_complex128",
"test_parity__foreach_clamp_max_fastpath_inplace_xpu_complex64",
"test_parity__foreach_clamp_max_fastpath_outplace_xpu_complex128",
"test_parity__foreach_clamp_max_fastpath_outplace_xpu_complex64",
"test_parity__foreach_clamp_min_fastpath_inplace_xpu_complex128",
"test_parity__foreach_clamp_min_fastpath_inplace_xpu_complex64",
"test_parity__foreach_clamp_min_fastpath_outplace_xpu_complex128",
"test_parity__foreach_clamp_min_fastpath_outplace_xpu_complex64",
"test_parity__foreach_erf_fastpath_inplace_xpu_complex128",
"test_parity__foreach_erf_fastpath_inplace_xpu_complex64",
"test_parity__foreach_erf_fastpath_outplace_xpu_complex128",
"test_parity__foreach_erf_fastpath_outplace_xpu_complex64",
"test_parity__foreach_erfc_fastpath_inplace_xpu_complex128",
"test_parity__foreach_erfc_fastpath_inplace_xpu_complex64",
"test_parity__foreach_erfc_fastpath_outplace_xpu_complex128",
"test_parity__foreach_erfc_fastpath_outplace_xpu_complex64",
"test_parity__foreach_floor_fastpath_inplace_xpu_complex128",
"test_parity__foreach_floor_fastpath_inplace_xpu_complex64",
"test_parity__foreach_floor_fastpath_outplace_xpu_complex128",
"test_parity__foreach_floor_fastpath_outplace_xpu_complex64",
"test_parity__foreach_frac_fastpath_inplace_xpu_complex128",
"test_parity__foreach_frac_fastpath_inplace_xpu_complex64",
"test_parity__foreach_frac_fastpath_outplace_xpu_complex128",
"test_parity__foreach_frac_fastpath_outplace_xpu_complex64",
"test_parity__foreach_lgamma_fastpath_inplace_xpu_bfloat16",
"test_parity__foreach_lgamma_fastpath_inplace_xpu_complex128",
"test_parity__foreach_lgamma_fastpath_inplace_xpu_complex64",
"test_parity__foreach_lgamma_fastpath_outplace_xpu_bfloat16",
"test_parity__foreach_lgamma_fastpath_outplace_xpu_complex128",
"test_parity__foreach_lgamma_fastpath_outplace_xpu_complex64",
"test_parity__foreach_maximum_fastpath_inplace_xpu_complex128",
"test_parity__foreach_maximum_fastpath_inplace_xpu_complex64",
"test_parity__foreach_maximum_fastpath_outplace_xpu_complex128",
"test_parity__foreach_maximum_fastpath_outplace_xpu_complex64",
"test_parity__foreach_minimum_fastpath_inplace_xpu_complex128",
"test_parity__foreach_minimum_fastpath_inplace_xpu_complex64",
"test_parity__foreach_minimum_fastpath_outplace_xpu_complex128",
"test_parity__foreach_minimum_fastpath_outplace_xpu_complex64",
"test_parity__foreach_round_fastpath_inplace_xpu_complex128",
"test_parity__foreach_round_fastpath_inplace_xpu_complex64",
"test_parity__foreach_round_fastpath_outplace_xpu_complex128",
"test_parity__foreach_round_fastpath_outplace_xpu_complex64",
"test_parity__foreach_sigmoid_fastpath_inplace_xpu_complex128",
"test_parity__foreach_sigmoid_fastpath_inplace_xpu_complex64",
"test_parity__foreach_sigmoid_fastpath_outplace_xpu_complex128",
"test_parity__foreach_sigmoid_fastpath_outplace_xpu_complex64",
"test_parity__foreach_sign_fastpath_inplace_xpu_complex128",
"test_parity__foreach_sign_fastpath_inplace_xpu_complex64",
"test_parity__foreach_sign_fastpath_outplace_xpu_complex128",
"test_parity__foreach_sign_fastpath_outplace_xpu_complex64",
"test_parity__foreach_trunc_fastpath_inplace_xpu_complex128",
"test_parity__foreach_trunc_fastpath_inplace_xpu_complex64",
"test_parity__foreach_trunc_fastpath_outplace_xpu_complex128",
"test_parity__foreach_trunc_fastpath_outplace_xpu_complex64",
"test_autodiff__foreach_sigmoid_inplace_xpu_complex128",
"test_autodiff__foreach_sigmoid_outplace_xpu_complex128",
"test_binary_op_with_scalar_self_support__foreach_pow_is_fastpath_True_xpu_bool",
AssertionError: RuntimeError not raised
"test_0dim_tensor_overload_exception_xpu",

RuntimeError: Tried to instantiate dummy base class CUDAGraph
"test_big_num_tensors__foreach_max_use_cuda_graph_True_xpu_float32",
"test_big_num_tensors__foreach_max_use_cuda_graph_True_xpu_float64",
"test_big_num_tensors__foreach_norm_use_cuda_graph_True_xpu_float32",
"test_big_num_tensors__foreach_norm_use_cuda_graph_True_xpu_float64",

AssertionError: Tensor-likes are not close!
"test_pointwise_op_with_tensor_of_scalarlist_overload__foreach_addcdiv_is_fastpath_True_xpu_float16",

Versions
Collecting environment information...
PyTorch version: N/A
Is debug build: N/A
CUDA used to build PyTorch: N/A
ROCM used to build PyTorch: N/A
OS: Ubuntu 22.04.3 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: 14.0.0-1ubuntu1.1
CMake version: version 3.22.1
Libc version: glibc-2.35
Python version: 3.9.19 (main, Mar 21 2024, 17:11:28) [GCC 11.2.0] (64-bit runtime)
Python platform: Linux-5.15.0-106-generic-x86_64-with-glibc2.35
Is CUDA available: N/A
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: N/A
GPU models and configuration: Could not collect
Nvidia driver version: Could not collect
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: N/A
CPU:
Architecture: x86_64
CPU op-mode(s): 32-bit, 64-bit
Address sizes: 52 bits physical, 57 bits virtual
Byte Order: Little Endian
CPU(s): 224
On-line CPU(s) list: 0-223
Vendor ID: GenuineIntel
Model name: Intel(R) Xeon(R) Platinum 8480+
CPU family: 6
Model: 143
Thread(s) per core: 2
Core(s) per socket: 56
Socket(s): 2
Stepping: 8
CPU max MHz: 3800.0000
CPU min MHz: 800.0000
BogoMIPS: 4000.00
Flags: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf tsc_known_freq pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb cat_l3 cat_l2 cdp_l3 invpcid_single intel_ppin cdp_l2 ssbd mba ibrs ibpb stibp ibrs_enhanced tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid cqm rdt_a avx512f avx512dq rdseed adx smap avx512ifma clflushopt clwb intel_pt avx512cd sha_ni avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local split_lock_detect avx_vnni avx512_bf16 wbnoinvd dtherm ida arat pln pts hwp hwp_act_window hwp_epp hwp_pkg_req avx512vbmi umip pku ospke waitpkg avx512_vbmi2 gfni vaes vpclmulqdq avx512_vnni avx512_bitalg tme avx512_vpopcntdq la57 rdpid bus_lock_detect cldemote movdiri movdir64b enqcmd fsrm md_clear serialize tsxldtrk pconfig arch_lbr amx_bf16 avx512_fp16 amx_tile amx_int8 flush_l1d arch_capabilities
Virtualization: VT-x
L1d cache: 5.3 MiB (112 instances)
L1i cache: 3.5 MiB (112 instances)
L2 cache: 224 MiB (112 instances)
L3 cache: 210 MiB (2 instances)
NUMA node(s): 2
NUMA node0 CPU(s): 0-55,112-167
NUMA node1 CPU(s): 56-111,168-223
Vulnerability Gather data sampling: Not affected
Vulnerability Itlb multihit: Not affected
Vulnerability L1tf: Not affected
Vulnerability Mds: Not affected
Vulnerability Meltdown: Not affected
Vulnerability Mmio stale data: Not affected
Vulnerability Retbleed: Not affected
Vulnerability Spec rstack overflow: Not affected
Vulnerability Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1: Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2: Mitigation; Enhanced IBRS; IBPB conditional; RSB filling; PBRSB-eIBRS SW sequence; BHI BHI_DIS_S
Vulnerability Srbds: Not affected
Vulnerability Tsx async abort: Not affected
Versions of relevant libraries:
[pip3] intel-extension-for-pytorch==2.1.30+git9e27c62
[pip3] numpy==1.26.4
[pip3] torch==2.4.0a0+gitff98731
[conda] intel-extension-for-pytorch 2.1.30+git9e27c62 dev_0
[conda] numpy 1.26.4 pypi_0 pypi
[conda] torch 2.4.0a0+gitff98731 pypi_0 pypi

Caused by those OP fallback to cpu at current stage

"test_pointwise_op_with_tensor_of_scalarlist_overload__foreach_addcdiv_is_fastpath_True_xpu_float16" passed now

Close the issue according to latest skip list.


cmd/compile: racy writes to global var not observed

What version of Go are you using (go version)?

$ go version
1.13, 1.15, 1.17, 1.20

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
darwin, centos
go env Output
$ go env

What did you do?
Directly run the following code, you will see that the program keeps printing 0, which is not expected.
package main

import (
    "fmt"
    "time"
)

type X struct {
    X0 int
    X1 int
    X2 int
    X3 int
    X4 int
    X5 int
    X6 int
    X7 int
    X8 int
    X9 int
}

func (x *X) set(a int) {
    x.X0 = a
    x.X1 = a
    x.X2 = a
    x.X3 = a // [tag 2]
    //x.X4 = a // [tag A]
    //x.X5 = a
    //x.X6 = a
    //x.X7 = a
    //x.X8 = a
    //x.X9 = a
}

var global X

func main() {
    go change()
    go display()
    select {}
}

func change() {
    count := 0
    for {
        count += 1
        newX := X{} // [tag 1]
        newX.set(count)
        global = newX
        //time.Sleep(time.Microsecond) // [tag B]
    }
}

func display() {
    for {
        time.Sleep(time.Second)
        g := global
        fmt.Println(g.X0)
    }
}

But if we uncomment the line with [tag A], or uncomment the line with [tag B], or run the program with gcflag='-N -l', then the program can print results greater than 0, which is as expected.
Another experiment: If we change the line with [tag 1] as newX := X{X0:1}, then we have to further comment the line with [tag 2] to reproduce the 'all zero output' result. It seems to be related with the amount of instructions nearby.
By comparing the assembly codes generated by go tool compile (shown in the figure below), we find that the entire for loop in function change() is wiped out, probably because the optimizer infers that those lines of code do not make any influences.

The above test code was originally written to test the consistency during struct copy. And a real-world project might not be easy to trigger this bug. But it indeed reveals a defect in go compiler, which needs to be reviewed.
This issue has been testified by my colleagues.

I don't think this is a bug. There is no synchronization in your code. A program with a data race in it is not a valid Go program and it could do anything.
See #45826.

@wdvxdr1123 It is not allowed to do "anything", it still must adhere to the Go memory model. But in this case, it is allowed to not observe the racy write to global and as a result, simply spin indefinitely.

I believe this is valid behavior according to the memory model

@seankhliao Please remind me how does the memory model make the output results different, between go run and go run -gcflags='-N -l'.

@seankhliao Please remind me how does the memory model make the output results different, between go run and go run -gcflags='-N -l'.

@seankhliao run the above code with go run -gcflags='-N -l'  and  go run , the results are different, it means the Go  compiler makes unreasonable optimization

In the presence of data races:

Otherwise, a read r of a memory location x that is not larger than a machine word must observe some write w such that r does not happen before w and there is no write w' such that w happens before w' and w' happens before r. That is, each read must observe a value written by a preceding or concurrent write.

In both instances, the program observes some valid value (initial write, write by other goroutine).
As it contains a data race, we make no guarantees about which value is observed.

@seankhliao I am afraid we are talking about different things.

As for the data race, I know there could be unpredictable outcome in one read. But since the program has been running for seconds, minutes or hours (with large number of none-zero write operations), how can the display function still keep reading the initial value, zero?
From the assembly code, we can see that the write operation in for loop is completely removed. Therefore, there is actually no data race when the program is running. The program keeps printing 0-s simply because write operations are not included in the binary executable file.

Let me clear up some things:
1, The memory model specifies a set of allowed behaviours, and a compiled program is deemed valid if its behaviour aligns with one of the allowed ones. As a result, 2 programs can express different behaviour and be both valid. In this case, the Go memory model says that the result of a racy load from a larger-than-word variable global is unspecified, which means that any value observed is legal, and both the unoptimised and the optimised compiled version is valid.
2, The memory model does not have a notion of wall clock, or real time, in the memory model, 1 action either happens before, at the same time or after another action. This happen-before relationship is a mathematical relationship defined and described in the memory model and has nothing to do with actual before-ness in terms of real time. In this program, the load g := global and the store global = newX do not establish a happen-before relationship. As a result, the load does not have to observe the store and it can return an arbitrary value which in this case it reads the original one.
3, Raciness, defined by the Go memory model is a property of a Go program, not the property of the compiled code. The raciness of which is defined by the memory model of the hardware architecture. As a result, multithreading reasoning must be performed on the Go source code, and in this case, analysis shows raciness in the global variable.

@merykitty Am I right to restate your explanation as follows:
There is no happen-before relationship between racy write and read. So the compiler does not know whether write makes any observable differences. In this context, it is allowed to believe that read only relies on the initial value and write does not make actual influences. Therefore the compiler can choose to delete the 'useless' write without taking any responsibility, even if there is a possibility that write does have some influences to read in real-world.
(1) When the number of instructions near write increases, the complier tends to keep write, although the added instructions contribute none to determine the usefulness of write. Why? One should be concerned if a real-world code with deliberate none-synchronized variable (performance reason) works fine only because the number of instructions around it is above some threshold but not because the compiler has the responsibility to keep it.
(2) Assume that a mutex is added around read and write. The race is avoided, but the compiler still cannot determine wether read is before or after write. Can I explain that the compiler keeps write in this case just because lock and unlock add up many instructions around write, as it does in (1)?
(3) In C/C++ compiler, only O3+ is risky to change the logic of the code. But go compiler provides a risky default optimization. It is reasonable, from the view of a normal programmer?
In my opinion, the default compiler should be designed to translate the code and make it run as it looks like. Not in the other way around: the relation of write and read cannot be mathematically determined, so the compiler can freely delete the uncertain codes.

@huzhiwen93 It seems to me that you are not used to working with concurrency at this low level, so I think that my attempt to explain the rationales behind the memory model would be futile. As a result, I will kindly refer to you the excellent blogs from Russ about this subject, which may clear up much of your confusion.

In my opinion, the default compiler should be designed to translate the code and make it run as it looks like.

History shows that that this really really hard to define.  Different people have completely different intuitions about what it means to "run as it looks like."
That said, I am surprised that in this case the change function is compiled into an infinite loop that does nothing.  @randall77 what compiler optimization is causing it to drop the assignment to the package scope variable?

The memory state of that loop is never used. No breaks, no panics, nothing. The compiler elides memory modifications that are known not to lead to any synchronization point (or any read within the goroutine).

Thanks.


Suggestions: Limiting results to increase performance and caching discussion

Hello,
it's me again😄. I'd like to suggest or discuss some things

How do you plan to implement caching? Any repository which the exporter retrieves is caches for an amount x? Also the findings etc? What about cache saving then?
Performance: To increase performance I would like to suggest to only retrieve the latest x images of a repo. This would lead to both an increase of performance as well decreasing the chance to hit rate limits. Make this configurable with an environment variable -> ever better.

I was planning to implement just a simple in-memory cache for the security findings as those are only updated once every twenty-four hours anyway. There's no need to call the API unless that cache is expired; so on start it will call the API for all security findings and then only once every twenty-four hours (unless your process restarts, in which case it will then start hitting the API again).
The cache doesn't really need to store the results of the call, just that the call was made as the results are effectively stored via the Prometheus metrics that are exported.
This could be improved with the use of an external cache like Redis, but I'd like to avoid adding a dependency for a simple Prometheus exporter unless we really need it. An external cache would avoid calling the API again on process restart.
Even with the cache this still hits rate-limiting issues, even with client-side rate limiting. So the only other improvement I can think of is, as you mention, limiting the number of images we grab security findings/metrics for. This would of course be configurable with maybe a sane default (not sure if we should have a default limit, or leave it unlimited by default).
In my use case I'd configure it to be unlimited as I want to know the security findings of all images, as in my environment all images in ECR represent a use in our cluster somewhere (i.e. old images that are no longer used are deleted after some time).

There's also the option of disabling collection of certain metrics if a user isn't interested in them. If the security findings metric calls are too heavy for a user and they're not particularly interested in them they could have the option to disable that collection entirely.

I think you're right with the cache, there is no need for external dependencies like Redis. If we can keep this simple then why not.
About the number of images I did some quick maths. We currently have about 3500 different images and scraping every single one would take like forever. If the rate limit allows one call per second this would be almost an hour to fetch all of them. Like you said, even then it's likely to hit rate limits, so maybe a default value in between 10 and unlimited might be useful. For example our average image count per repo is 34. The chance is high other organizations also have a few repos with a high number of images and more repos with less images. I prefer keeping this setting lower like 10 or 25 so you don't run in this rate limiting issue when using default parameters.
Another thing to keep in mind is the more images you scrape, the more metrics you export and this can also result in more cost because of increased traffic when scraping in case you're using a cloud provider like AWS.
Two more things came to my mind when I wrote this wall of text:

I'd love a metric to see how long the scraping took, maybe a gauge?
If scraping all metrics can take up to several hours, we should publish the metric as soon as the image and findings was parsed. 👍

If I can you help you with development, please tell me.

@pgrunm-DrKlein Sorry, I was out of town at a conference last week. I released v0.3.0 during that time that adds in limiting the number of tags scanned per image: https://github.com/celestialorb/aws-ecr-prometheus-exporter/releases/tag/v0.3.0.
The additional configuration is in the README, but it comes with a limit setting as well as a toggle though it is disabled by default so you'll need to explicitly enable it.

Yes, I saw that. :) I just wanted to assist you with development, because I really appreciate your work. I think this project is really useful and surely takes so time to develop. 👍


Transaction hash is not displayed on EVM block validation errors in the pipeline

Describe the bug
We set the transaction hash to default https://github.com/paradigmxyz/reth/blob/8cfa5efe62e79923b317d9685fc9f8205d5a92b5/crates/storage/provider/src/providers/database/provider.rs#L1352-L1353, which is why it's zeroed when we report the error https://github.com/paradigmxyz/reth/blob/8cfa5efe62e79923b317d9685fc9f8205d5a92b5/crates/revm/src/processor.rs#L288
We need to calculate the hash for the provided transaction when such an error occurs.
Node logs
Feb 10 11:29:44 high reth[10996]: 2024-02-10T18:29:44.107025Z ERROR sync::pipeline: Stage encountered an execution error: EVM reported invalid transaction (0x0000000000000000000000000000000000000000000000000000000000000000): Transaction error: NonceTooHigh { tx: 1, state: 0 } stage=Execution bad_block=15703508
Feb 10 11:29:44 high reth[10996]: Stage encountered an execution error: EVM reported invalid transaction (0x0000000000000000000000000000000000000000000000000000000000000000): Transaction error: NonceTooHigh { tx: 1, state: 0 }

will take a look on this

@justcode740 assigned! Let me know if you have any questions.

it seems pretty trivial one, lmk if i miss anything here...


app successful run but i cant access from local address

I am trying to run the vue-hackernews-2.0
npm run dev

> vue-hackernews-2.0@ dev /root/Projects/vue-hackernews-2.0
> node server

server started at localhost:8080

 DONE  Compiled successfully in 3575ms                                  18:39:35

 DONE  Compiled successfully in 4483ms                                  18:39:36

webpack built e137da6148c57546e735 in 4483ms

app successful run but i can access from local address
from browser load nothing only stay in loading
my npm and node version
npm -v
6.4.1

node -v
v8.12.0

i have the same problem


Make emoji keyboard display the last used skin tone

When selecting a skin tone variant in the emoji keyboard, the recent list shows that skin tone, but the keyboard itself shows the neutral skin tone.
Make the keyboard properly register the last used skin tone.

I will try to get this implemented in 9.0, which will be able to refactor the entire emoji keyboard since it will target iOS 15.

This will be implemented in a 9.x minor update.


Feature additional aqs data

https://github.com/matthias-bs/BresserWeatherSensorReceiver/issues/97

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
215

Relevant Lines:
215

💛  - Coveralls


Missing Link?

"Service accounts: Internal Azure DevOps accounts used to support a specific service, such as Agent Pool Service, PipelinesSDK. For descriptions of service accounts, see [Security groups, service accounts, and permissions](permissions.md#collection-level groups). "
Where do I find Permissions.md?

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

ID: 851595a3-32b8-c0e4-ff4f-aa84cf436175
Version Independent ID: c2cb9c6c-de29-a7a9-9728-fbbc1d223a4f
Content: About security, authentication, authorization, and security policies - Azure DevOps
Content Source: docs/organizations/security/about-security-identity.md
Product: devops
Technology: devops-security
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@D-Moonesinghe - Thank you for your inquiry. The link you want is Security groups, service accounts, and permissions. I will update the article to fix the broken link.


[flash_ctrl] RMA burst ratio and flash endurance

During rma entry, the flash controller currently writes one flash word at a time.
Depending on the flash technology, this level of writing may bring flash close to its endurance limit before an erase.
We should slightly increase the program burstiness to ensure this does not happen.
Ideally this should be a templated parameter, but an easier solution is probably better for now.

@jdonjdon


Improve grammar of new feature status page.

./_site/docs/reference/feature-stages.html

internally linking to /community.html, which does not exist (line 1)
community

That's what I get for writing stuff at the airport :-)
Fixed
On Thu, Nov 30, 2017 at 8:32 PM, Laurent Demailly<EMAIL_ADDRESS>wrote:

@ldemailly commented on this pull request.
In _docs/reference/feature-stages.md
https://github.com/istio/istio.github.io/pull/750#discussion_r154266246:

@@ -106,4 +110,4 @@ Below is our list of existing features and their current phases. This informatio
| Reproducible oss characterization tool (Fortio)    | Alpha
| Mixer bottom up perf    | Alpha

-Please get in touch if there are features you'd like to see in our future releases!
+Please get in touch by joining our community if there are features you'd like to see in our future releases!
the link is https://istio.io/community/ (no .html)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio.github.io/pull/750#pullrequestreview-80403817,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVucHRcAvA3gHKVDLeAQCFgCCbmFc4TRks5s74FJgaJpZM4Qx0UK
.


Addition of Agree to Terms and Conditions checkbox in Apply PopUp (GDPR Requirement)

Is there an option for adding checkboxes according to GDPR laws that candidates need to read and agree to our terms and conditions with a link to it.
Is it possible to add this to the popup

Is there any alternate ways that Bullhorn can help German companies be GDPR law compliant on this regard?

This is important for us as well!
Until the Career Portal has a checkbox to accept our Privacy Policy we can't use the portal for legal reasons. Please be aware of two requirements for the check box to be GDPR compliant:

It needs to link to our Privacy Policy (admin needs to be able to add a custom link to the check box description text - or configure the text freely; latter is preferred)
It cannot be pre-checked

If you would add this functionality in the near future you would help us a lot!

Hi All
If you are using WordPress or can use. Matador Jobs has this as a standard feature.
Paul

Hi Paul,
thanks for following up! Matador is rather expensive, and adding this check box functionality is pretty important on an international level for privacy policy compliance. If you would include it in a future roll-out you'd improve our Bullhorn experience significantly.

Hello,
The customization made may have resulted in an invalid JSON file.  To check what is wrong you can use a json validator website such as this: https://jsonformatter.curiousconcept.com
As far as the feature, I am currently working on a simple way to implement this as well as creating a simple privacy policy tooltip/excerpt.  I will be updating github on the over the next week.

Hi Charles,
thanks a lot for working on it! Much appreciated :)!


build: misc/cgo/fortran fails with ld: library not found for -lgfortran

At tip:
##### ../misc/cgo/fortran
# _/Volumes/Data/src/go.tip/misc/cgo/fortran
ld: library not found for -lgfortran
clang: error: linker command failed with exit code 1 (use -v to see invocation)
FAIL	_/Volumes/Data/src/go.tip/misc/cgo/fortran [build failed]
2016/02/24 14:28:34 Failed: exit status 2

This is fixable (for me) by setting LIBRARY_PATH=/usr/local/Cellar/gcc/5.3.0/lib/gcc/5, but given the length of the discussion at https://github.com/Homebrew/homebrew/issues/8539, I think that I'm not going to be the only person to hit this. It'd be nice if this failed gracefully if libgfortan isn't on the library path.
/cc @kortschak

I have a crude fix for this that I will send tonight.


Use LinkedBlockingQueue to process results - based on thomaswue

/**

Solution based on thomaswue @thomaswue  solution, commit:
commit d0a28599c293d3afe3291fc3cf169a7b25ae9ae6
Author: Thomas Wuerthinger<EMAIL_ADDRESS>Date:   Sun Jan 21 20:13:48 2024 +0100

Changes:

Use LinkedBlockingQueue to store partial results, that

will then be merged into the final map later.
As different chunks finish at different times, this allows
to process them as they finish, instead of joining the
threads sequentially.

This change seems more useful for the 10k dataset, as the

runtime difference of each chunk is greater.

Use only 4 threads if the file is >= 14GB.

This showed much better results on my local test, but I only
run with 200 million rows (because of limited RAM), and I have
no idea how it will perform on the 1brc HW.
*/

Check List:

[x] Tests pass (./test.sh <username> shows no differences between expected and actual outputs)
[x] All formatting changes by the build are committed
[x] Your launch script is named calculate_average_<username>.sh (make sure to match casing of your GH user name) and is executable
[x] Output matches that of calculate_average_baseline.sh
[x] For new entries, or after substantial changes: When implementing custom hash structures, please point to where you deal with hash collisions (line number): same as thomaswue existing solution

Execution time:
Execution time of reference implementation:

Time for 200 million rows 10k dataset:
time java --enable-preview --class-path target/average-1.0.0-SNAPSHOT.jar dev.morling.onebrc.CalculateAverage_tivrfoa > me.out

real	0m4,288s
user	0m0,380s
sys	0m0,175s

time java --enable-preview --class-path target/average-1.0.0-SNAPSHOT.jar dev.morling.onebrc.CalculateAverage_thomaswue > thomas.out

real	0m9,242s
user	0m0,341s
sys	0m0,183s

Output differs from the expected one for the 1B file:
Validating calculate_average_tivrfoa.sh -- measurements_1B.txt
1c1
< Abha;-30.9;18.0;68.0
---
> Abha;-31.9;18.0;68.0

FAILURE: ./test.sh tivrfoa measurements_1B.txt failed

00:04.255, nice.

00:04.255, nice.

Thank you!
Do you have the time for the 10k, please?

Yepp, here you go:

Validating calculate_average_tivrfoa.sh -- measurements_10K_1B.txt

+ rm -f measurements.txt
+ ln -s measurements_10K_1B.txt measurements.txt
Benchmark 1: timeout -v 300 ./calculate_average_tivrfoa.sh 2>&1
  Time (mean ± σ):      9.300 s ±  0.031 s    [User: 0.002 s, System: 0.004 s]
  Range (min … max):    9.266 s …  9.338 s    5 runs

Summary
  tivrfoa: trimmed mean 9.29848737266, raw times 9.294628507659999,9.27442990966,9.32640370066,9.266159457659999,9.33784911966

Leaderboard

| # | Result (m:s.ms) | Implementation     | JDK | Submitter     | Notes     |
|---|-----------------|--------------------|-----|---------------|-----------|
|   | 00:09.298 | [link](https://github.com/gunnarmorling/1brc/blob/main/src/main/java/dev/morling/onebrc/CalculateAverage_tivrfoa.java)| 21.0.2-graal | [tivrfoa](https://github.com/tivrfoa) | GraalVM native binary, uses Unsafe |

Yepp, here you go:
Validating calculate_average_tivrfoa.sh -- measurements_10K_1B.txt

+ rm -f measurements.txt
+ ln -s measurements_10K_1B.txt measurements.txt
Benchmark 1: timeout -v 300 ./calculate_average_tivrfoa.sh 2>&1
  Time (mean ± σ):      9.300 s ±  0.031 s    [User: 0.002 s, System: 0.004 s]
  Range (min … max):    9.266 s …  9.338 s    5 runs

Summary
  tivrfoa: trimmed mean 9.29848737266, raw times 9.294628507659999,9.27442990966,9.32640370066,9.266159457659999,9.33784911966

Leaderboard

| # | Result (m:s.ms) | Implementation     | JDK | Submitter     | Notes     |
|---|-----------------|--------------------|-----|---------------|-----------|
|   | 00:09.298 | [link](https://github.com/gunnarmorling/1brc/blob/main/src/main/java/dev/morling/onebrc/CalculateAverage_tivrfoa.java)| 21.0.2-graal | [tivrfoa](https://github.com/tivrfoa) | GraalVM native binary, uses Unsafe |

Thank you!
My idea completely failed.
Trying to optimize using a different HW does not work ... :sweat_smile:

Haha, yeah, at the top it's a bit like walking in the dark if you're on a
different ISA.

Message ID: @.***>


fix(quickfix list): avoid clobbering existing lists

I've found a small issue with the quickfix list integration.
Setting the replace flag when sending grug-far results into the quickfix list truncates existing results. That's a problem because the quickfix list keeps a history of all the lists generated during the session, and you can navigate the history with :cprev and :cnext. Which means that populating the list from grug overtakes whatever list is currently displaying.
For example, imagine you have the lists in :chistory

order
title

1
LSP references

2
Some other plugin list

*
3
LSP references

Then you navigate back to see the second list with :cprev.

order
title

1
LSP references

*
2
Some other plugin list

3
LSP references

Then you use grug-far and send the results to the list:

order
title

1
LSP references

*
2
Grug FAR results for: someString

3
LSP references

Now I've lost Some other plugin list.
Inserting a brand new entry seems like the desired behavior, not only prevents clobbering existing lists, but also helps you navigate previous grug-far results without opening the grug-far ui first.

Yeah, you're right, I went by (faulty) memory as I have already mapped those commands to < and > in the quickfix buffer 😅


space in filename replaced with period

When I run this
github-release upload --user retorquere --repo $npm_package_name --tag "v$npm_package_version" --name "The Brain.js" --file "docs/The Brain.js"

The file name in the release ends up being The.Brain.js. Can I prevent this?

Yeah, I wish it didn't convert spaces to periods/dots…


Fix namespace resolver not replacing property type

Coverage decreased (-9.05%) to 90.639% when pulling 9a19016221b3d316140e7d90237e369b847dc820 on Fix-namespace-resolver-not-replacing-property-type into 85faf1d206584ccfc5321b6f0da0b6fe5f1904bc on develop.


Navbar

[x] Structure of Navbar
[x] CSS of Navbar
[x] Responsiveness of Navbar.
[x] functionality of Navbar.

All task are completed


typings field in package.json is invalid

"typings": "./dist/index.d.ts",

./dist/index.d.ts does not exists.
It seems like it should be ./dist/tosource.d.ts.

good catch, would you like to submit a PR?

good catch, would you like to submit a PR?

Yes, I will.

Yes, I will.


The method 'call' was called on null.

Hi,
First of all, thanks for the cool plugin!
I am facing a weird issue not sure if you can help me. I have implemented the plugin in my code and each time just press to zoom it throws this error message:
The method 'call' was called on null.
Here is my code:
 Scaffold(
          // (....)
          body: PinchZoom(
            image: Image.file(imagePath),
            zoomedBackgroundColor: Colors.black.withOpacity(0.5),
            resetDuration: const Duration(milliseconds: 100),
            maxScale: 2.5,
          ),
          // (....)

Error Log:
════════════════════════════════════════════════════════════════════════════════
E/flutter (12662): [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: NoSuchMethodError: The method 'call' was called on null.
E/flutter (12662): Receiver: null
E/flutter (12662): Tried calling: call()
E/flutter (12662): #0      Object.noSuchMethod  (dart:core-patch/object_patch.dart:51:5)
E/flutter (12662): #1      _PinchZoomState._show
package:pinch_zoom/src/pinch_zoom_widget.dart:181
E/flutter (12662): #2      _PinchZoomState.build.<anonymous closure>.<anonymous closure>
package:pinch_zoom/src/pinch_zoom_widget.dart:63
E/flutter (12662): #3      new Future.delayed.<anonymous closure>  (dart:async/future.dart:326:39)
E/flutter (12662): #4      _rootRun  (dart:async/zone.dart:1182:47)
E/flutter (12662): #5      _CustomZone.run  (dart:async/zone.dart:1093:19)
E/flutter (12662): #6      _CustomZone.runGuarded  (dart:async/zone.dart:997:7)
E/flutter (12662): #7      _CustomZone.bindCallbackGuarded.<anonymous closure>  (dart:async/zone.dart:1037:23)
E/flutter (12662): #8      _rootRun  (dart:async/zone.dart:1190:13)
E/flutter (12662): #9      _CustomZone.run  (dart:async/zone.dart:1093:19)
E/flutter (12662): #10     _CustomZone.bindCallback.<anonymous closure>  (dart:async/zone.dart:1021:23)
E/flutter (12662): #11     Timer._createTimer.<anonymous closure>  (dart:async-patch/timer_patch.dart:18:15)
E/flutter (12662): #12     _Timer._runTimers  (dart:isolate-patch/timer_impl.dart:397:19)
E/flutter (12662): #13     _Timer._handleMessage  (dart:isolate-patch/timer_impl.dart:428:5)
E/flutter (12662): #14     _RawReceivePortImpl._handleMessage  (dart:isolate-patch/isolate_patch.dart:168:12)
E/flutter (12662):

════════ Exception caught by widgets library ═══════════════════════════════════

Thank you for reporting the issue. This bug is now fixed in the 0.0.3 release


Partners: Fixed streaming in MistralAI with ainvoke and callbacks

Fix streaming in mistral with ainvoke

[x] PR title
[x] PR message
[x] Add tests and docs:

[x] Added a test for the fixed integration.
[x] An example notebook showing its use. It lives in docs/docs/integrations directory.

[x] Lint and test: Ran make format, make lint and make test from the root of the package(s) I've modified.

Hello

I Identified an issue in the mistral package where the callback streaming (see on_llm_new_token) was not functioning correctly when the streaming parameter was set to True and call with ainvoke.
The root cause of the problem was the streaming not taking into account. ( I think it's an oversight )
To resolve the issue, I added the streaming attribut.
Now, the callback with streaming works as expected when the streaming parameter is set to True.

How to reproduce
from langchain_mistralai.chat_models import ChatMistralAI
chain = ChatMistralAI(streaming=True)
# Add a callback
chain.ainvoke(..)

# Oberve on_llm_new_token
# Now, the callback is given as streaming tokens, before it was in grouped format.

I've noticed an interesting detail in the _generate function.
It includes the line should_stream = stream if stream is not None else self.streaming.
However, this line is absent in the async def _agenerate function should_stream = stream if stream is not None else None.

@efriis  Up ?

Up @efriis


[Outlook] (online meeting) Supported on iOS

Docs Build status updates of commit 3434de4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/online-meeting.md
:white_check_mark:Succeeded
View

docs/outlook/outlook-mobile-addins.md
:white_check_mark:Succeeded
View

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded
View

docs/reference/manifest/extensionpoint.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 9e6c05e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/online-meeting.md
:white_check_mark:Succeeded
View

docs/outlook/outlook-mobile-addins.md
:white_check_mark:Succeeded
View

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded
View

docs/reference/manifest/extensionpoint.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


So what rmskin does?

Just realise that there is rmskin on the release page. what function does it provide?

@Droyk Have a look here: https://www.rainmeter.net/


shooting fixed

What does this PR do?
This PR will fix the non responsive nature of shooting button and space bar and even added hold and click click response on space bar
Fixes #204

https://github.com/user-attachments/assets/2934c1fd-aa03-40c5-b4ae-c06ed26dbcf2
Type of change

Bug fix (non-breaking change which fixes an issue)

How should this be tested?

[ hold space bar for fire] Test A
[ single click space bar for fire ] Test B

in both the test you can see the responsiveness of space bar
Mandatory Tasks

[yes] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

@YadavAkhileshh kindly merge this PR
#hacktoberfest-accepted #gssoc-ext #level3

@YadavAkhileshh kindly merge this PR
#hacktoberfest-accepted #gssoc-ext #level3

@YadavAkhileshh kindly merge this PR soon so that I can work on other issue
Kindly attach the appropriate labels


Stuck on "Listening for Discord IPC Calls"

I successfully built and ran your project, but for some reasons its not working
This is the output from the Windows guest:
Connecting to <IP_ADDRESS>:8765 Successfully connected to <IP_ADDRESS>:8765 Listening for Discord IPC Calls
This does not change, even if i restart the game and/or discord

Hello, good to see that someone is using this program haha
Can you please also send the logs of the host program ?
Also, does the game your trying supports Discord Rich Presence ?

Hi, thanks for answering so fast :D
Im happy your program exists, I was really amazed when I did not found any other program that achieves this.
Heres the Client log on Windows:
PS C:\Users\Leon\Desktop> .\guest.exe --address <IP_ADDRESS> Connecting to <IP_ADDRESS>:8765 Successfully connected to <IP_ADDRESS>:8765 Listening for Discord IPC Calls
And heres the Host log (on Manjaro):
Listening on: <IP_ADDRESS>:8765 Accepted connection <IP_ADDRESS>:49746
Im running the host by just executing ./host, I assume this is supposed to work like this, considering the Connection itself is working?
The game I was initially trying out was Genshin Impact, which normally shows in Discord that I am palying it and for how long i was playing it.
While writing this I just tried out League of Legends, which from what I know, offers the same Info to Discord.
League of Legends Logs:
Client:
PS C:\Users\Leon\Desktop> .\guest.exe --address <IP_ADDRESS> Connecting to <IP_ADDRESS>:8765 Successfully connected to <IP_ADDRESS>:8765 Listening for Discord IPC Calls Accepted connection 25e04b96-512f-4f61-9c54-e7ef65225599
Host:
Listening on: <IP_ADDRESS>:8765 Accepted connection <IP_ADDRESS>:50171 Attempting to connect to /run/user/1000/discord-ipc-0 Connected to /run/user/1000/discord-ipc-0
When Playing League of Legends, its showing either "In Lobby" or "In Game", instead of just "Playing" when playing Genshin Impact.
Is it possible that "Playing Game XXX" does not count as a Rich Presence?
Thanks

I checked and unfortunately Genshin Impact doesn't support Rich Presence so yeah it couldn't work :/
Maybe there is a program that hooks into Genshin Impact and adds support for Rich Presence for the game.
I will close the issue since there is no real problem then 😅

Oh, then sorry this issue, I misinterpreted what rich presence meant.
And sorry again for re-opening this issue, but I thought opening another issue just for this question would be a bit overkill 😅
Would it be possible for your program to broadcast these non rich presence Infos like in genshin impact (Playing Genshin Impact etc...) to discord on the Host? Didn't find any other program that does this either. Or is it strictly for rich presence?stuff?
Thanks

Hey, no worry, discussing does not reopen the issue :)
Unfortunately discord doesn't use RPC for detecting simple programs/games, It just scans through active processes and displays infos if it recognizes one.
Maybe there's a way to trick this and display those infos but this a bit out of this project's scope and I don't really know how we can do this.
Though I'd be happy to implement such a thing if someone finds a way to achieve this or maybe an alternative solution.

Oh okay, thats a whole different thing then :D
I'll try some things out, maybe I get it to work.
But im still happy I found your project, its still really helpful


Attempt to cache bazel output on travis

This may cause problems, so before merging I should add a comment (in a separate commit) to restart the build (to ensure that the cache isn't breaking anything and actually does speed things up).

Thanks @cromwellian seems that there are still some problems with this and it looks like Alex is working on it too.
I'll leave this for now but can resume work on it or pass it on to @alxrsngrtn if there's problems with alternative approaches.

Note: This decreased my build and test time from 32minutes 27seconds to 23 minutes and 52 seconds. Not a huge improvement, but moving in the right direction.


Modify error message banner for 403 Forbidden

Is your feature request related to a problem? Please describe.
Request to change the error message that appears when the user receives a 403. The error message tells the user to consent additional permissions for Graph Explorer (the client app), but many 403s are the result of failed checks on the signed-in user's privileges, so no consent can solve the issue.
Describe the solution you'd like
Modify the error message text as follows:
"You need to consent to the permissions on the Modify permissions (preview) tab" --> "Either the signed-in user does not have sufficient privileges, or you need to consent to one of the permissions on the Modify permissions (preview) tab"
Describe alternatives you've considered
N/A
Additional context

@thewahome @darrelmiller can you please explain to me how's the error message implemented here? I'd like to know whether @Jackson-Woods suggested fix would be enough or if more work is needed

Yes, changing that message should be sufficient.

@adhiambovivian  just want to confirm that we are working on this ticket? I see it's still assigned to me


Eran/install

Tests looks OK.
I'll add new ones once the code get's to the right place
Ran 12 of 12 Specs in 30.948 seconds
SUCCESS! -- 12 Passed | 0 Failed | 0 Pending | 0 Skipped

@eranco74 where is the generation of the ignition files?

@eranco74 where is the generation of the ignition files?

Not in this commit


Restart

When the victim restarts their computer the malware goes away, is this a bug or something yall havent added yet?

or maybe av detected it lmfao. check startup ?!?

do you still have that issue?

I'm having this issue.


Add gpt-4o mini

https://openai.com/index/gpt-4o-mini-advancing-cost-efficient-intelligence/

This is already addressed here: https://github.com/mckaywrigley/chatbot-ui/pull/1814
Working correctly. Waiting for merge.

As a temporary workaround, you can also add new model with the New Model functionality.


Rufus.Rufus version 3.14.1788

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Not supported - yet


multi-cluster-management flag available in fresh installation

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Deploy rancher/rancher:master-head 51d074b on RKE single-node cluster by helm
Go to Global Settings in Explorer and switch to Feature Flags pane

Result:

MCM feature flag is present but according to https://github.com/rancher/rancher/issues/33264#issuecomment-863780746 it shouldn't be there:

Other details that may be helpful:

It couldn't be Deactivated once it is Active so I guess MCM flag shouldn't be present on fresh cluster
Also I didn't find a way how to disable the MCM feature flag - in 2.5.8's Ember UI there were two radio-buttons in initial wizard for it like:I want to create or manage multiple clusters & I'm only going to use the cluster Rancher was installed on but these are missing in Explorer.
I checked helm inspect values rancher-latest/rancher output but I couldn't find anything related to MCM there

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): rancher/rancher:master-head 51d074b
Installation option (single install/HA): any

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): any
Machine type (cloud/VM/metal) and specifications (CPU/memory): any
Kubernetes version (use kubectl version): v1.19.6

We're taking away the disable action, not hiding it entirely (in https://github.com/rancher/dashboard/pull/3314)

Tested on rancher/rancher:master-head c7f3cce The action for disabling MCM is hidden now:

Closing.


Add remaining data type support in dictionary

add data type support in dictionary - blob, bufferId, fieldKey, objectId and phoneno
update XML file loader to support iso-8859-1 encoding

working on the failure tests


I was still using it normally until one day it reported an error like that

If I use the package, it will report an error as shown in the debug screen, and if I don't use it, it will work normally
Debug Console
Launching lib\main.dart on Windows in debug mode...
main.dart:1
/C:/Developer/flutter/packages/flutter_localizations/lib/src/l10n/generated_material_localizations.dart(41545,41): error G020FDE51: A method declaration needs an explicit list of parameters. [D:\Workspace\Coding\DevUP_LMS\devuplms-flutter\build\windows\flutter\flutter_assemble.vcxproj]
/C:/Developer/flutter/packages/flutter_localizations/lib/src/l10n/generated_material_localizations.dart(41545,41): error G90CBB506: The name of a constructor must match the name of the enclosing class. [D:\Workspace\Coding\DevUP_LMS\devuplms-flutter\build\windows\flutter\flutter_assemble.vcxproj]
/C:/Developer/flutter/packages/flutter_localizations/lib/src/l10n/generated_material_localizations.dart(41545,40): error G67247B7E: Expected ';' after this. [D:\Workspace\Coding\DevUP_LMS\devuplms-flutter\build\windows\flutter\flutter_assemble.vcxproj]
/C:/Developer/flutter/packages/flutter_localizations/lib/src/l10n/generated_material_localizations.dart(41545,40): error G44692867: A value of type 'int' can't be returned from a function with return type 'ScriptCategory'. [D:\Workspace\Coding\DevUP_LMS\devuplms-flutter\build\windows\flutter\flutter_assemble.vcxproj]
/C:/Developer/flutter/packages/flutter_localizations/lib/src/l10n/generated_material_localizations.dart(41545,41): error G6814800A: Expected a class member, but got 'ScriptCategory'. [D:\Workspace\Coding\DevUP_LMS\devuplms-flutter\build\windows\flutter\flutter_assemble.vcxproj]
/C:/Developer/flutter/packages/flutter_localizations/lib/src/l10n/generated_material_localizations.dart(41545,41): error GB23DA612: The superclass, 'GlobalMaterialLocalizations', has no unnamed constructor that takes no arguments. [D:\Workspace\Coding\DevUP_LMS\devuplms-flutter\build\windows\flutter\flutter_assemble.vcxproj]
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(249,5): error MSB8066: Custom build for 'D:\Workspace\Coding\DevUP_LMS\devuplms-flutter\build\windows\CMakeFiles\601b17c202b30b5f36d32fa72406aba7\flutter_windows.dll.rule;D:\Workspace\Coding\DevUP_LMS\devuplms-flutter\build\windows\CMakeFiles\5b79815e23bc1c7fe6ea2d6e6308cfa7\flutter_assemble.rule' exited with code 1. [D:\Workspace\Coding\DevUP_LMS\devuplms-flutter\build\windows\flutter\flutter_assemble.vcxproj]
Exception: Build process failed.
Exited (sigterm)

Hello. I can see the issue came from Flutter SDK flutter_localizations (a bit confused with my package where it does not have "s" at the end). Have you tried to update Flutter to see if the issue is solved? Or have you faced the issue after updating Flutter?


Resolve the BIO_ctrl() / OSSL_FUNC_BIO_ctrl_fn declaration clash [TAKE 2]

This is resolved by pretending that it isn't a clash, by making an internal
function in core that simply casts the result of ossl_core_bio_ctrl() to
int and hope for the best.  Since OSSL_FUNC_BIO_ctrl_fn declared an
int return value, that cast was done implicitly anyway.  Now it's done in
a more controlled manner.

This is a followup on #18198

As noted by @t8m here, this solution has its limitations, on platforms where int and long are different sizes

Rebase, now that #18198 is merged

@openssl/otc, please consider this vs #18583

OTC will have to decide between this and #18583

OTC: TAKE 2 is the preferred option with added error checking to ensure the result is <= INT_MAX. We would also be interested in seeing a version of TAKE 1 which is backwards and forwards compatible.

This seems to have stalled awaiting updates requested by OTC?


update get cashout provider

This PR fixes #530
Changes
Refactor getCashoutProvider service
Tests

You know, actually, one thing that could be done is adding them on a seeders script 🤔


markdown: Add support for inline video thumbnails.

Inspired from #13350.
Somewhat broader issue: #9081
Tested on videos from
https://file-examples.com/index.php/sample-video-files/.
discussion: https://chat.zulip.org/#narrow/stream/378-api-design/topic/Video.20markdown.20support.20.2320449

End to end tested successfully.

Thinking about this a bit harder, something that gives me a little pause here is that this means that we'll be proxying all video traffic (not just the preview) through camo.  So if someone posts a 5GB video to Zulip, every user that presses play on it is going to cause camo to download that file and send it back out -- so 10GB of traffic per user that clicks play, which can add up fast for folks that may be paying for bandwidth to the Zulip instance.
There's clearly a privacy tradeoff here, in that the whole reason we're proxying through camo is so that the hosting website doesn't get any information about the Zulip users in a stream that posts a link to it.  The question is a little how much users expect that privacy to extend to the embedded video if they press play.  It's possible that the middle ground here is that we will want a way to make the preview load from Camo, as it does here, and that clicking the play button swaps the element out for one that pulls from the original source, so that those play from the origin, and don't generate a lot of bandwidth usage from Zulip.
I'm fine to merge this as-is and revisit later if this is causing undue bandwidth use for us or others, but flagging for @timabbott.

Nice!
It feels odd to me that the interactions with an uploaded video look totally different from what you can do with a photo. But it certainly seems better than the current state.

It feels odd to me that the interactions with an uploaded video look totally different from what you can do with a photo. But it certainly seems better than the current state.

Sorry, I don't think I follow.  What interactions look different?

It's possible that the middle ground here is that we will want a way to make the preview load from Camo, as it does here, and that clicking the play button swaps the element out for one that pulls from the original source, so that those play from the origin, and don't generate a lot of bandwidth usage from Zulip.

I think that's probably a reasonable strategy; it'd be in line with how we handle Youtube links as well, which is that once you click it to play the video, we load the Youtube player.  We should update our security/privacy model documentation when doing this, but I think that'd make me much more comfortable with merging a change of this form.

Sounds good.
@amanagr, does that sound like something you can tackle?

Yup, on it.

It feels odd to me that the interactions with an uploaded video look totally different from what you can do with a photo. But it certainly seems better than the current state.

Sorry, I don't think I follow. What interactions look different?

The video has a unique set of buttons and menu options:

There's nothing analogous to the view we offer for pictures:

There are of course difference in functionality, but for example we're offering totally different UIs for downloading an image vs. a video.

Sorry for the late push on this one. Got buried amongst other work.

If we want the same behaviour as youtube or images on this, we have get thumbnail image for video and display image which is then replaced by video on click. So, we first have to work towards getting a thumbnail engine working (which is another project).

If we want the same behaviour as youtube or images on this, we have get thumbnail image for video and display image which is then replaced by video on click. So, we first have to work towards getting a thumbnail engine working (which is another project).

This comment confuses me a bit.  We have a thumbnailer for videos, in a sense -- camo with the new flag on.
The behaviour I'm seeing with this commit is close, but I suspect we need more JS trickery.  If I click on the middle of the video, I see it make a request to the upstream, and it pretty seamlessly swaps out the element and starts playing -- which is great!
If I click the ▶ icon, or scrub around the timeline by dragging the slider at the bottom, it makes more requests to camo to pull the data, instead of going to the upstream.  I suspect that we should start out with an element with <video controls="false", which may mostly solve this issue.  It's possible that in order to more fully prevent folks from interacting with the element from camo we'll have to put a transparent div on top of the <video> but I think that's probably not going to be necessary -- especially since the "failure case" is we just play it through Camo (which will work fine!) and nothing stops dedicated folks from pulling the camo URL and playing it directly if they wanted to.

@alexmv updated based on your feedback!

@timabbott I'm not totally sure of the status here, but marking for "integration review" so that you can consider.

Wasn't there an "#api design" thread about the precise CSS classes to use for this? I don't see it obviously linked, and also am not sure if this is current for those proposed changes; a general update on our understanding of remaining questions would be very helpful.

https://chat.zulip.org/#narrow/stream/378-api-design/topic/Video.20markdown.20support.20.2320449/near/1453814 was the thread in question.
I think this needs a rebase, and a trivial update to the layout proposed there:
<div class="message_inline_image message_inline_video">
    <video data-video-original-url="{url}" preload="metadata" src="{camo_url}">
</div>

This seems worth trying to include in 7.0 given the small amount of work we think is left; @amanagr please ping me when a rebase is completed and I can try to get it in.

Very interested in this feature as we trial different chat solutions for integration in our environment. Think this will be merged any time soon?

Looking also forward to see this merged :)

We are having some final discussion over here before this gets updated and merged.

This is looking pretty sweet, Aman! I left some comments mostly about video aspect-ratios, which I think we can account for a little more deliberately. Happy to discuss any of this stuff :)

Merged a couple prep commits as the series ending with f7b633f5111f3152f94745736d587824dc9ac6a6 to shrink this a bit.

I think this needs a commit for https://github.com/zulip/zulip/pull/20449#discussion_r1340386429 but otherwise I think this is ready to merge.

Merged after a small tweak to the final commit message. Thanks @amanagr and @alexmv for all the work pushing to get this finished!

Awesome! Been waiting for this!


Terminal not runing python scripts

Environment data

VS Code version: 1.44.2
Extension version (available under the Extensions sidebar): 3.8.2
Python version (& distribution if applicable, e.g. Anaconda): 3.8

Expected behaviour
Script should run in terminal.
Actual behaviour
I run a simple script as print("Hello") and it doesn't run. Istead, it open the command prompt for a second and then it closes.
I have already uninstall visual studio code several times, deleting code folder also. I didn't change any setting.

@manun92 Sorry for the delayed response. What is the version of the extension you are using? Also, when you run the command directly in a terminal does it work?

Closing this due to lack of activity.


Old screenshots for using .NET Core CLI with VSTS

https://docs.microsoft.com/en-us/dotnet/core/tools/using-ci-with-cli#visual-studio-team-services-vsts
The VSTS build definition screenshots and instructions are dated.

We might just wanna link to some existing content from VSTS instead?
https://docs.microsoft.com/en-us/vsts/build-release/apps/aspnet/build-aspnet-core
https://docs.microsoft.com/en-us/vsts/build-release/tasks/build/dotnet-core
https://www.visualstudio.com/team-services/support/build-deploy-asp-net-core-app-azure/

Also, if we wanted to make the docs topics to show up in our own TOC and not just link to the content, I know there are options, I just never tried.


PE-6147: feat(empty state): implement empty state

--- Releases ---
Android release: https://appdistribution.firebase.google.com/testerapps/1:305132849030:android:6cf0cd5ec064fad3ffce07/releases/2at02b7sdfj98

General note: tested on Firebase URL and found on mobile I could scroll to see empty state cards, but not on other breakpoints.


WordPress.com Image Proxy Images 404

The last update that introduced const ENABLE_REWRITE_DOMAINS = true;   doesn't seem to be working correctly atleast when I tested it on a website with wordpress.com's cdn proxying images.
When I had the domain being proxied all images were returning a 404 and with a CF-Cache-Status: ByPass.
I attempted reloading the page a few times to no avail so I ended up commenting out the line for wp.com images for now. The rewriting of the urls in the HTML worked as expected but it was an issue with the response returning a 404.

Do you have a sample site I can take a look at? Doesn't need the worker on it, just something to test against that shows the issue.

From: Scott Hartley<EMAIL_ADDRESS>Sent: Tuesday, March 19, 2019 7:44 PM
To: pmeenan/cf-workers
Cc: Subscribed
Subject: [pmeenan/cf-workers] WordPress.com Image Proxy Images 404 (#2)
The last update that introduced const ENABLE_REWRITE_DOMAINS = true; doesn't seem to be working correctly atleast when I tested it on a website with wordpress.com's cdn proxying images.
When I had the domain being proxied all images were returning a 404 and with a CF-Cache-Status: ByPass.
I attempted reloading the page a few times to no avail so I ended up commenting out the line for wp.com images for now. The rewriting of the urls in the HTML worked as expected but it was an issue with the response returning a 404.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/pmeenan/cf-workers/issues/2, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAbHBcoA_mw4RoKIcB6Wvo9m7hcV8CVnks5vYXZrgaJpZM4b9Vlo.

https://www.sertmedia.com/ I will flip the worker switch back on for now for the images

It's actually easier if you leave it switched off if you don't mind. Then I can test the rewriting in the worker sandbox.

From: Scott Hartley<EMAIL_ADDRESS>Sent: Tuesday, March 19, 2019 10:29 PM
To: pmeenan/cf-workers
Cc: Patrick Meenan; Comment
Subject: Re: [pmeenan/cf-workers] WordPress.com Image Proxy Images 404 (#2)
https://www.sertmedia.com/ I will flip the worker switch back on for now for the images
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/pmeenan/cf-workers/issues/2#issuecomment-474659537, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAbHBV2siQCs40qo0hcyGckcXiayo3eWks5vYZ0GgaJpZM4b9Vlo.

Argh. Should be fixed now. There was a fast-path exit for image requests to bypass the worker logic but it was also bypassing for proxied requests, not just rewriting.

Hmm, I updated the script and I flushed NGINX Cahe + Purged the entire CloudFlare cache to no avail. I will try flipping the rewrites off for those images for now.


feat: create kubernetes events in case of insufficient capacity errors

Description
We need to have improved visibility of ICE events when karpenter encounters them.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

9 of 9 (100.0%)  changed or added relevant lines in 2 files are covered.
14 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.07%) to 81.293%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/provisioning/scheduling/topologygroup.go
2
96.75%

pkg/test/environment.go
5
89.71%

pkg/controllers/node/controller.go
7
70.59%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
6931

Relevant Lines:
8526

💛  - Coveralls


api token not found

I had gh-slack working previously. Maybe because of the upgrade to 0.0.16, I'm not sure, I'm now getting:
gh slack -d -i https://github.com/znull/znull/issues/1 https://github.slack.com/archives/...
2023/05/05 11:41:45 net/http: invalid byte '\x15' in Cookie.Value; dropping invalid bytes
api token not found

I'm not sure why, but deleting all slack entries from macos keychain and logging in again fixed this.

For others who find this, upgrading the extension solved this for me:
gh extension upgrade gh-slack


About passing globals (how to "make awesome")

This code:
h::table(function(){
        # sadly, i'm not sure how to get around this at the moment :(  help me make this awesome
        global $table_data;
//...

possibly it could have overloaded variant? Such as
h::table(function($context_class_object_containing_globals) {
//...

or
h::table($context, function() {
///...

??

And many thanks to you and yours htmlgen! It's a charm! :)

That's an interesting idea. I didn't think I'd ever work on this project again, but now that I'm doing more PHP work lately, I might pick it back up.

@AndruxaSazonov OK, I suppose your request prompted me to rewrite the whole damned thing. I think you'll be very pleased with the results.
Check out 2.0.0-pre.
Here's a sort of chaneglog... of sorts...

All the magic is gone
It's just pure PHP with no funny business.
Templating via native PHP require (see the provided example)
Syntax is roughly the same, but much nicer
Code base is 60% smaller
Output buffering is gone
View helpers like h::cycle are gone
Data store is no longer necessary. set_variable and get_variable are gone
Pretty printing is out (at least for now)

I'll let you know once 2.0 is live and stable. I still want to add some unit tests. I'll get to it over the next day or two.

Surpsingly exciting!.. Thanks for your rework!

@AndruxaSazonov, @avsazonov
Version 2.0 released just now
Any feedback is appreciated. If you find issues, please create a new issue.


Release v6.46.0 for windows 64-bit contains incorrect version of CLI

Please fill out the issue checklist below and provide ALL the requested information.

[X] I reviewed open and closed github issues that may be related to my problem.
[X] I tried updating to the latest version of the CF CLI to see if it fixed my problem.
[X] I attempted to run the command with CF_TRACE=1 to help debug the issue.
[X] I am reporting a bug that others will be able to reproduce.

Describe the bug and the command you saw an issue with
I have downloaded the release 6.46 of the CLI for Windows 64-bit and the CLI contained in both the binaries zip and delivered by the installer has the version 7.0.0-beta, which seems to have experimental features in the push command.
cf push my-app
This command is in EXPERIMENTAL stage and may change without notice

Updating app my-app...
Pushing app my-app to org *** / space dev as ************...
Getting app info...
Mapping routes...
API endpoint not found at 'https://******************/v3/routes?domain_guids=**********&hosts=my-app&paths='
FAILED

cf -v
7.0.0-beta.24+15564551a.2019-07-16

What happened
The version of CLI contained in the release 6.46 for Windows 64-bit is not correct.
Expected behavior
The version of CLI delivered by release 6.46 for Windows 64-bit should be 6.46 and the push should not be experimental.
I have checked the release 6.46 for Linux and the version of the CLI is correct.
To Reproduce
Download the release 6.46 for windows 64-bit (binaries or installer)
Check the version of the CLI using cf -v.
Provide more context

platform: Windows 64-bit
version of the CLI you are running: Downloaded 6.46, but the packages deliver 7.0.0-beta

Hello @aparolin ,
Thank you very much for opening this issue.
We confirmed the incorrect behavior on our end. We believe it is a result of the v7 CLI team beginning their release process, and accidentally interfering with the v6 releases.
We will take the necessary steps to fix this now.
Thank you,
Brendan and @acrmp

@aparolin This is fixed. The problems seemed to only be with the Windows related releases. We have tested the 64 bit Windows installer and binary.
If you continue to see problems, please feel free to reopen this issue.


VertexModelGardenLlama using publisher model?

llm = VertexModelGardenLlama(
  model=MODEL,
  credentials=credentials,
  location=REGION,
  project=PROJECT_ID
)

I try to use llama3.1 model like above, and it throws the following error
HTTPStatusError: Error response 400 while fetching
  "error": {
    "code": 400,
    "message": "Project `XXXXXX` is not allowed to use Publisher Model `projects/XXXXXX/locations/REGION/publishers/meta/models/MODEL`",
    "status": "FAILED_PRECONDITION"
  }

The curl command using the same settings (model, credentials, locations, project_id) works. So I don't think it is the server issue.

could you share a full snippet, please?
it's working fine on my side.

@lkuligin
llm = VertexModelGardenLlama(model="meta/llama-3.1-405b-instruct-maas")

ValidationError: 1 validation error for VertexModelGardenLlama
  Value error, Model meta/llama-3.1-405b-instruct-maas is not supported yet! [type=value_error, input_value={'model_name': 'meta/llam... 'default_metadata': ()}, input_type=dict]

langchain-google-vertexai==2.0.7

Could someone please add meta/llama-3.1-8b-instruct-maas, meta/llama-3.1-70b-instruct-maas, and meta/llama-3.1-405b-instruct-maas to _LLAMA_MODELS? Thanks.
cc: @lkuligin


Rank results following rules

Create a RankedStreamBuilder with settings and stuff to customise the sorting algorithms settings.
This builder produce a RankedStream which streams the results using a StreamWithState (with the levenshtein state) and sort the results with some possible optimisations.
The internal StreamWithState returns each matching word (with the right levenshtein distance) associated with the document id, word index in the query, the attribute where the word is located and position of the word in this attribute.
This is raw data and that must be sorted to match the following rules:

Typo: Will rank higher a word that doesn’t contain a typing mistake.
Words: Will rank higher an object matching more of the words typed by the user.
Proximity: Will rank higher words close to each other (George Clooney is better than George word Clooney).
Attribute: Will rank higher a match in a more important attribute (Title, Description).
Words position: Will rank higher words that are at the beginning of an attribute.
Exact: Will rank higher words that match exactly without any suffix (if prefix search is supported).

First step to follow the rules is to collect results (document id, word index in the query, attribute and position) in a collection to help sorting them.
But these rules must be applied using bucket sorting: if, after sorting results using the first rule, there is matches that are equal so use the next rule to disassociate them and so on if equallity are detected.
The typo rule is not the more complex to implement, sort documents according to the levenshtein distance only. If there is multiple words take the maximum of the minimum query words distances.
The words rule is straightforward: the index of the word in the query is returned in the result of the internal StreamWithState stream, this information can be used to know which word the result talk of and so simply count the number of different matching words ([1, 2, 3] is better than [1, 2] or [1, 1, 1]).
The proximity rule can be implemented by sorting the results by the attribute first, by the word index in the query after and finally by the index in the attribute. Once the results for each document are sorted then sort documents by the smallest sum of distances between each query word.
The attribute rule has already been fulfilled in the previous rule application.
The words position rule has already been granted by the previous sorting, select the different word index in the query and get the one that is the nearest from the start of the attribute, rank document based on these values (lower is better).
The exact rule is not defined for the moment.
Conclusion
The best way to implement all of these rules is to create a collection for each document that match something. Documents can be added as the research progresses, the collections contain structures with the index of the word in the query followed by the distance of the match, the attribute in which the word has matched and finally the index of this word in the attribute.

https://github.com/meilisearch/MeiliDB/blob/62fe6a82637b9b9cbb4f81cf912a96c5b45dd612/meilidb-core/src/query_builder.rs#L373-L507


Fix typo '--omit-snaphot'

The Problem/Issue/Bug:
When executing commands on a project whose files have gone missing, an error message instructs you to return the files or delete the project. The "delete the project" part of that message has a typo.
How this PR Solves The Problem:
This PR fixes the typo.
Manual Testing Instructions:
Automated Testing Overview:
Related Issue Link(s):
Release/Deployment notes:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I changed the usage to ddev delete and now this is ready to go when tests complete.


VMware PowerCLI step.

I think it would be prudent to mention the version of the PowerCLI step from where it currently resides to just before the user starts the Azure Migrate collector.  The reason for this is that the web app that starts does do a verification for it, but if it's a pre requisite, why not mention it before the collector starts?  This way it's configured and ready to go.
I tried to make this edit actually in the document and submit it for review, but GitHub kept giving me 500 error.  Not sure if we're allowed to submit those edits anymore.

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

ID: 2369d635-5ccf-92f1-b534-e2dce47113a1
Version Independent ID: e3567682-a600-4e20-6d72-f03e025c1e4f
Content: Discover and assess on-premises VMware VMs for migration to Azure with Azure Migrate
Content Source: articles/migrate/tutorial-assessment-vmware.md
Service: azure-migrate
Product: unspecified
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@johnwildes Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

Thanks for your comment. It's not a prerequisite since setting up PowerCLI can only occur after the collector VM is created. Rather, it's a task when setting up the collector VM.

#please-close

@johnwildes We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


I don't like the html import!

I want to manage polymer component with npm and webpack.
Although  said polymer use the HTML import to deal with dependence is to emulate the webcomponents standard, but I think it actually hinder the development of polymer. The official standard is not necessarily the standard of people's recognition, web's modular standard is still based on npm. Webpack is good, Polymer is expected to officially support a NPM and webpack can be used to regulate the components, so that to promote the ecological construction of polymer components. See how many of the components of the react-xxx are now.

I think you mean Bower? HTML Imports have nothing to do with npm. The former provides a client-side loading solution for web components, the latter provides package management. As of now, Imports are the way to load components and do dep management.
There's discussion here on switching to npm.


--rm and -d are conflicting options, so let's remove -d for now

This seems to only be the case if you're running an old version of docker. Closing.


Added customizable background. Updated deprecated headers

Thanks for your contribution. Can you split this into two pull requests? One addressing just the deprecated headers and one making the background color customizable? That would be great :slightly_smiling_face:.


Avoid AIX compiler issue with macro argument name change

Fixes #12756
The IBM XLC compiler on AIX has problems handling macro substitution when argument names appear in strings. Renaming the arguments so they don't match any substring fixes this.
Tested on AIX 7.2 using IBM XLC v13.

This pull request is ready to merge

Thanks. Merged to master.


Create user documentation

Documentation was created and is available on readthedocs.org. Doc build is automated using Github webhooks.


ReferenceError: navigator is not defined in angular universal

hi
when run project in ssr
with command : npm run dev:ssr
get same error:
return new DeviceUUID(navigator.userAgent);
                        ^
ReferenceError: navigator is not defined

im using a mock browser , in server.ts file :
const MockBrowser = require('mock-browser').mocks.MockBrowser;
const mock = new MockBrowser();
global['navigator'] = mock.getNavigator();
global['window'] = mock.getWindow();
global['document'] = mock.getDocument();

thanks

SSR runs on the server, where navigator and other objects (like window) aren't available. When coding for SSR you have to make sure to check the platform before executing code that use these objects.

@Parziphal
i worked based on your tips
like this:
 if (isPlatformBrowser(this.platformId)) {
        import('device-uuid').then( module => {
             const deviceId = new module.DeviceUUID().get() ;
         });
  }

and worked properly
thanks :)


Show the name of the signed in user on the Disconnect action menu item.

Show the signed in user given  name + family name in the Disconnect action bar menu item.

Looks good.


Extraneous data written to package.json while debugging an extension that writes to package.json

Related to vscode extension test run modifies package.json after 1.67.0
I have a couple of extensions that I wrote that can take user settings and create commands from them.  In order to have these commands available in the Command Palette it is necessary to write to some fields in the package.json for that extension.
This is happening when debugging in the ExtensionHost.  When the package.json is written to, other information is being added:
 "identifier": {
  "value": "ArturoDent.find-and-transform",
  "_lower": "arturodent.find-and-transform"
 },
 "isBuiltin": false,
 "isUserBuiltin": false,
 "isUnderDevelopment": true,
 "extensionLocation": {
  "$mid": 1,
  "fsPath": "c:\\Users\\Mark\\find-and-transform",
  "_sep": 1,
  "external": "file:///c%3A/Users/Mark/find-and-transform",
  "path": "/c:/Users/Mark/find-and-transform",
  "scheme": "file"
 },
 "targetPlatform": "undefined",

There is no warning to me when this is done - sometimes this extra info is added near the top and sometimes near the bottom of the package.json.  If the extension gets published with this extra info, the extension fails.  It not only fails, it takes all of vscode down with it.
It is not clear to me what value that extra info adds to me as extension author but it is irritating to have to check the package.json and delete them.
Steps to Reproduce:
This is how I am writing to package.json:
const findSettings = await extensionCommands.getSettings("findInCurrentFile");
const findSettingsCommands = await _makePackageCommandsFromFindSettings(findSettings);

thisExtension.packageJSON.contributes.commands = builtins.concat(findSettingsCommands);

fs.writeFileSync(path.join(context.extensionPath, 'package.json'), JSON.stringify(thisExtension.packageJSON, null, 1));

Debug an extension that itself writes to the package.json
Modify the user settings and reload.
The extra data noted above is added to package.json by vscode.

The simpler extension which exhibits this behaviour is Command Alias.
@sandy081

bump @sandy081 as I think, judging from the issue mentioned at the top, that this should be assigned to @sandy081?

I think this is being added by the write you are doing. JSON.stringify(thisExtension.packageJSON, null, 1) contains this additional details. Please check it.

I see that that extra data is added to the Extension Host version of the package.json immediately on start-up (of the Extension Host).
So when I later write JSON.stringify(thisExtension.packageJSON, null, 1) it is getting incorporated.
The question is, how do I test this without writing to the thisExtension.packageJSON?  To enable the ability to create commands in the user's settings that are then added to the package.json and so in the Command Palette I have to write to the package.json and reload and the commands from the settings are now available as hoped.
I suppose I could, in testing, first strip out those fields (identifier, isBuiltIn, etc.)  before I write back to the package.json.  But that seems unfortunate since they will never be there in regular use.  And would that then screw up functioning of the Extension Host - maybe not since there has to be a reload anyway and they get added back in again?
Is that extra data necessary for the Extension Host?

It is recommended to read the package.json from the extension location to get the original contents.

It is recommended to read the package.json from the extension location to get the original contents.

:+1: The best is to read the file from disk, parse it, modify the JSON and then write it back to disk. As you have discovered, the packageJSON exposed via the API contains these additional properties. We exposed them by accident since the first time we supported extensions and now it is not safe to remove them, as some existing extensions might have taken a dependency on these properties being there.

I am using this code now:
exports.getPackageJSON = async function () {

  const extensionPath = extensions.getExtension('ArturoDent.find-and-transform').extensionPath;

  const packageJSONUri = Uri.file(path.join(extensionPath, 'package.json'));
  const packageContents = (await workspace.fs.readFile(packageJSONUri)).toString();
  const packageJSON = JSON.parse(packageContents);

  return packageJSON;
}

It works around the issue.  Thanks for looking into this.


feat(SetTheory/Game/PGame): not_lt

Adds not_lt in complement of not_lf and not_le.

a) This should not be a simp lemma
b) Once the definition of LF is unfolded, this just gives ¬ x < y ↔ ¬ y < x ∨ x ≤ y. You should be able to prove this for preorders in general, can you do that?

b) Once the definition of LF is unfolded, this just gives ¬ x < y ↔ ¬ x ≤ y ∨ y ≤ x. You should be able to prove this for preorders in general, can you do that?

If so, would it be best to keep this not_lt theorem?

You can keep it for now. But I do plan in the future to remove LF, or at least keep it only as a notation for ¬ y ≤ x.

It is a stronger relation, and that's the issue. Mathematically, the games {0, 0 | } and {0 | } are entirely the same, because their left and right sets are the same. But in the way we set PGame up, these two are non-equal, and in fact, they're not even relabellings of each other, even though they are identical.
The distinction between these two games is an artifact of our type theoretic foundations, and isn't really mathematical, which is why we (at the very least I) want to get rid of the relabelling notion which does take it into account.


[Creator Starter Kit] Documentation: the 'discussions' link on Github home page leads to a 404 page

On the Github home page for the PowerCAT Creator Kit, the 'discussions' link in paragraph 'Asking and Answering Questions' leads to a 404 page.

Thanks for reporting this. I removed that link as we are going to use Issue to better track and respond to requests/questions.


Maya: Light family - OP-1430

Changelog Description
Introduces new light family in Maya for publishing lights.
Testing notes:

Add any light types into project_settings/maya/publish/ValidateLightRequired/light_types that needs to be considered as a light family.
Add any required attribute values into project_settings/maya/publish/ValidateLightRequired/attribute_values. First field is {node_type}.{attribute_name}, second field is the value expected.
Add any optional attribute values into project_settings/maya/publish/ValidateLightOptional/attribute_values. First field is {node_type}.{attribute_name}, second field is the value expected.
Publish and load.

we should also support light blocker and such from arnold (but i think they are transform)

@tokejepsen  @BigRoy User could also add any transfom nodes I guess ...this should be possible even now when set via OP Settings to the Validate Light settings list. Or not?

we should also support light blocker and such from arnold (but i think they are transform)

@tokejepsen @BigRoy User could also add any transfom nodes I guess ...this should be possible even now when set via OP Settings to the Validate Light settings list. Or not?

Yeah, havent tried but you could have transform as valid light type.

Yeah, havent tried but you could have transform as valid light type.

Nope, seems like only shapes are the only valid types? Did try to add lights_GRP for my transform node and no luck.
@tokejepsen Can you confirm?

Nope, seems like only shapes are the only valid types? Did try to add lights_GRP for my transform node and no luck.
@tokejepsen Can you confirm?

My bad. Yeah, only the shape types are being validated against atm.
Will have a think for next time I get back to this.

@BigRoy how about this PR? what u think? shouldnt we just merge it as is?? It seems quite ok to me but nothing happening here...

PR is in draft waiting for client funding.

@antirotor @BigRoy guys how about this kiddo? almost ready but on hold for such a long time. Would be cool to get it out from the doors!

What is the stage of this PR? It seems to be left off for some time....

@moonyuet I think that this PR is functional at its core functionality (just for lights) however there been some proposal to broaden its function for other type of object classes like lightblocker etc.
It really depends if these should be included within this PR or not. Maybe just review the code and test it as is should do....

Closing due to lack of client support.


Firebase app crashes on startup (EXC_BREAKPOINT)

I recently updated my project and all of my pods were updated aswell. I always keep a copy of my previous app version.
My podfile:
pod 'Firebase/Core'
pod 'Firebase/Messaging'
pod 'Firebase/Firestore'
pod 'Firebase/Performance'
pod 'Firebase/Database'
pod 'Firebase/InAppMessagingDisplay'
pod 'Firebase/RemoteConfig'
pod 'Firebase/Analytics'

My AppDelegate:
override init() {
    FirebaseApp.configure()
}

This crash happens everytime i run the app on the simulator/physical device. And i get:

There is no error. Nothing to fix on my end, However it is stopping us from developing another version of the app.
**Stackoverflow  link`

We'll likely need at least a backtrace and ideally a reproducible example to help.
From this information, it looks like it might be a duplicate of #3270

Hi @paulb777. I cannot show a reproducible example for this app. As i just installed the pods and ran the app in the simulator/physical device. No logic in the app, just a simple view with a black background.

In all my apps this bug
only in iOS 13 Beta 2
Crash in this line.

Only crashes on iOS 13 for me as well.

Thanks for the reports - I am able to reproduce in beta 2 and will try to track down the issue.
@rawandahmad698 as a side note, FirebaseApp.configure() should go in application(_:didFinishLaunchingWithOptions:) and AppDelegate should not have an overridden init method.

@ryanwilson How soon can we wait for the update?

@ivanvorobei for a status update, as soon as I discover the root issue and fix. I believe I have an idea as to what's going on (locking on a serial thread) but need to find the best solution to get around this, I'm not sure why it's only crashing on iOS 13.
For a patch to fix this, it depends on how long it takes but we're working hard on it and will report as soon as we know.

Closing in favor of the original #3270. Please follow conversation there.