I know @strager has done some SIMD magic, but I think this is beyond my reach:

https://github.com/quick-lint/quick-lint-js/blob/master/src/quick-lint-js/fe/keyword-lexer.cpp
https://github.com/quick-lint/quick-lint-js/blob/master/src/quick-lint-js/fe/keyword-lexer.h

But we should at least get LLVM to hand us some finely tuned assembly code, or have @strager point us in the right direction 😁

Are you doing this for fun or you think it matters?
Lookups don't seem substantial in profiles.

I've noticed 2 things so far:
1.
The existing match statement's assembly is comparison after comparison:
pub fn match_keyword(s: &str) -> (Self, Atom) {
 push    r15
 push    r14
 push    r12
 push    rsi
 push    rdi
 push    rbx
 sub     rsp, 72
 mov     r12, r8
 mov     r15, rdx
 mov     r14, rcx
 if len == 1 || len >= 12 {
 cmp     r8, 1
 je      .LBB2359_4
 cmp     r12, 11
 ja      .LBB2359_4
     lea     rax, [r12, -, 2]
     cmp     rax, 9
     ja      .LBB2359_124
     lea     rcx, [rip, +, .LJTI2359_0]
     movsxd  rax, dword, ptr, [rcx, +, 4*rax]
     add     rax, rcx
     jmp     rax
.LBB2359_16:
     movzx   eax, word, ptr, [r15]
     cmp     eax, 29537
 "as" => (As, Atom::new_inline("as")),
 je      .LBB2359_127
     movzx   eax, word, ptr, [r15]
     cmp     eax, 28516
 "do" => (Do, Atom::new_inline("do")),
 je      .LBB2359_128
     movzx   eax, word, ptr, [r15]
     cmp     eax, 26217
 "if" => (If, Atom::new_inline("if")),
 je      .LBB2359_129
     movzx   eax, word, ptr, [r15]
     cmp     eax, 28265
 "in" => (In, Atom::new_inline("in")),
 je      .LBB2359_130
     movzx   eax, word, ptr, [r15]
     cmp     eax, 29545
 "is" => (Is, Atom::new_inline("is")),
 je      .LBB2359_23
     movzx   eax, word, ptr, [r15]
     cmp     eax, 26223
 "of" => (Of, Atom::new_inline("of")),
 jne     .LBB2359_124

Which gets more expensive the longer the string (i.e. once it's >4 chars, >8 chars etc)
The jumps are to blocks that look like this
.LBB2359_132:
     buffer[i - 1] = text[i - 1]; (C:\Users\yfeig\.cargo\registry\src\github.com-1ecc6299db9ec823\compact_str-0.7.0\src\repr\inline.rs:63)
     xorps   xmm0, xmm0
     movups  xmmword, ptr, [rsp, +, 51], xmm0
     mov     dword, ptr, [rsp, +, 67], 0
     mov     word, ptr, [rsp, +, 48], 28518
     mov     byte, ptr, [rsp, +, 50], 114
     InlineBuffer(buffer) (C:\Users\yfeig\.cargo\registry\src\github.com-1ecc6299db9ec823\compact_str-0.7.0\src\repr\inline.rs:67)
     mov     rax, qword, ptr, [rsp, +, 63]
     mov     qword, ptr, [r14, +, 23], rax
     movzx   eax, word, ptr, [rsp, +, 48]
     mov     word, ptr, [r14, +, 8], ax
     movzx   eax, byte, ptr, [rsp, +, 50]
     mov     byte, ptr, [r14, +, 10], al
     mov     rax, qword, ptr, [rsp, +, 51]
     mov     qword, ptr, [r14, +, 11], rax
     mov     eax, dword, ptr, [rsp, +, 59]
     mov     dword, ptr, [r14, +, 19], eax
     movzx   eax, byte, ptr, [rsp, +, 63]
     mov     byte, ptr, [r14, +, 23], al

This doesn't seem right, because that's the code for new_const.
pub const fn new_const(text: &str) -> Self {
        if text.len() > MAX_SIZE {
            panic!("Provided string has a length greater than our MAX_SIZE");
        }

        let len = text.len();
        let mut buffer = [0u8; MAX_SIZE];

        // set the length
        buffer[MAX_SIZE - 1] = len as u8 | LENGTH_MASK;

        // Note: for loops aren't allowed in `const fn`, hence the while.
        // Note: Iterating forward results in badly optimized code, because the compiler tries to
        //       unroll the loop.
        let text = text.as_bytes();
        let mut i = len;
        while i > 0 {
            buffer[i - 1] = text[i - 1];
            i -= 1;
        }

        InlineBuffer(buffer)
    }

It's as if the Atom::new_inline() aren't const.
(Note that in mov     word, ptr, [rsp, +, 48], 28518, the number 28518 is for UTF-8 "of". These code blocks aren't for the case where we didn't have a match. There are dozens of such blocks, they each an inline of new_inline for a specific hardcoded value.)
I didn't test, but I wonder if moving the match to its own function that returns a &'static Atom would fix this.
I cloned your branch with the phf_map, and this isn't happening. I think it might thanks to KEYWORDS being static (maybe in combination with the value tuples being explicitly stored in an array?).
I'm surprised there's no noticeable improvement. I guess it doesn't outweigh #2.
2.
phf_map! is interesting.
It creates a (general) perfect hashing map for us with no effort, which is amazing:
phf::Map {
    key:<PHONE_NUMBER>5322966823u64,
    disps: &[
        (1u32, 45u32),
        (0u32, 0u32),
        (1u32, 37u32),
        (1u32, 22u32),
        (4u32, 54u32),
        (0u32, 1u32),
        (1u32, 50u32),
        (9u32, 76u32),
        (0u32, 29u32),
        (5u32, 43u32),
        (0u32, 0u32),
        (0u32, 1u32),
        (3u32, 5u32),
        (7u32, 64u32),
        (3u32, 5u32),
        (83u32, 20u32),
        (3u32, 69u32),
    ],
    entries: &[
        ("abstract", (Abstract, Atom::new_inline("abstract"))),
        ("implements", (Implements, Atom::new_inline("implements"))),
        ("false", (False, Atom::new_inline("false"))),
        ("do", (Do, Atom::new_inline("do"))),
        ("get", (Get, Atom::new_inline("get"))),
        ...
    ],

The hashing function used is SipHash which is the default in Rust (SipHasher13).
It has cryptographic benefits that aren't relevant for our use case (and, I imagine, for the majority of phf_map uses..), so using other hash functions could result in a speedup (however, the phf crate doesn't seem to have support for this).
A hand-written PHF map and hashing function tailored to this use case (and known assumptions: keys are finite (small) set of strings, length between 2 to 12) could definitely be faster.
Like I said initially - it doesn't seem like the perf gains would be meaningful, but it could be done "just for fun".
I'm still a little baffled at how the PR performs worse.
On one hand I'd expect most of the shorter keywords to be way more common (in which case, a handful of comparisons is less work than the hash), but on the other hand I'd expect most identifiers to be variable names that fall between 2 to 12 chars (and have to go through all of the comparisons, which I'm guessing is much more work than the hash?).

Are you doing this for fun or you think it matters? Lookups don't seem substantial in profiles.

This is the hottest path in our lexer and its dominant on the profiler, around 2-5%.
I always have a gut feeling this can be improved, and @strager mentioned SIMD so I looked at ago, and showed me how to look at the assembly code.
So here are my findings:

Why does the const Atom::new_inline not compiling down to a compact representation? Is it because there's a loop inside the code? This part is beyond me.

If I want to fine tune the string matching part:

Why is the matching compared string by string? Should I explicitly do:

branch by string length?
order the keywords by frequency they appear in JavaScript / TypeScript?
add an explicit check for TypeScript to match less keywords?

If I want to go the phf route:

I guess we need to find a better phf hashing alternative?

The best we can do is study and port the SIMD algorithm from @strager, but again, this is beyond my knowledge.

Why does the const Atom::new_inline not compiling down to a compact representation? Is it because there's a loop inside the code? This part is beyond me.

What do you mean?
As far as I can tell, the strings are compact - they live on the stack.

Changing to &static seems to have a better performance, https://github.com/Boshen/oxc/pull/143, but the assembly code look almost identical?

You still see the bazillion inlined new_const?
Based on the improvement, I'm inclined to believe they aren't there (can take a look if needed).

Why is the matching compared string by string? Should I explicitly do:
branch by string length?
order the keywords by frequency they appear in JavaScript / TypeScript?
add an explicit check for TypeScript to match less keywords?

I'd expect it to give an improvement, yeah. Worth checking out.

I guess we need to find a better phf hashing alternative?

I searched around and didn't find anything.
Since I became vested, I might try to make something as a side project.
A basic one would just use a faster hash. A more advanced one would enable using a custom hash and key comparison, which would enable creating super-optimal solutions like @strager's.

The best we can do is study and port the SIMD algorithm

The only SIMD involved is for the string key comparison (after you hash and get an index you still need to compare the lookup key with the found key, because inputs can be arbitrary right? identifier names are arbitrary). That's keyword_lexer::key_strings_equal.
The real voodoo is static selection_type select(const char* key, std::size_t key_length), which efficiently takes the first and last two chars, which are unique in the keyword set, and uses those bits for the "dumb" hash algorithm.
It's brilliant.

Since I became vested, I might try to make something as a side project.

There's a PR sitting there for two years, and given the amount of "Rewrite it in Rust" projects going on, this would be a great and valuable project to work on.

The real voodoo is static selection_type select(const char* key, std::size_t key_length), which efficiently takes the first and last two chars, which are unique in the keyword set, and uses those bits for the "dumb" hash algorithm.
It's brilliant.

This technique can be found from Wojciech Muła's article on SIMD-friendly algorithms for substring searching.

So given all the things we can improve on, I'll walk up the ladder and tune this step by step.
Thank you so much for all the inputs, @YoniFeng and @strager, I have learned so much from you.


Add example for Toggle Component action

After talking this over with Huyen, this is more a demo of existing Skuid functionality than a new project, so I'm closing this PR. However, the sample page around it could make an interesting experiment in the future!


Script replaces seemingly random existing images with each run

With one run, the script will create new device types, but then on subsequent runs it seems to overwrite device images with a new name, then the next sometimes it'll do the same again, and again.
Example: device type EMAT09-10
First run:
Difference
{
"front_image": ""
}
{
"front_image": "devicetype-images/eaton-emat09-10.front.png"
}
Second run:
Difference
{
"front_image": "devicetype-images/eaton-emat09-10.front.png"
}
{
"front_image": "devicetype-images/eaton-emat09-10.front_8zE6qHf.png"
}
Third run:
Difference
{
"front_image": "devicetype-images/eaton-emat09-10.front.png"
}
{
"front_image": "devicetype-images/eaton-emat09-10.front_KCjLwwE.png"
}
Of course, this is completely unnecessary and no doubt is filling up storage with random new images. I'm just wondering - why does the script even overwrite images in the first place? Surely it's enough to upload them once.
I pulled the version of the script down yesterday, so it's a current issue.

Can confirm, seemingly pointless image updates at random.

This is done automatically by the Netbox API. You can manually recreate this by uploading an image file with the same name as the existing image file for a device type image. It will automatically append something to make the name unique.
Perhaps the solution here is to have a flag that forces overwriting existing images instead of always overwriting.


Error concerning calico.yaml

When I try to deploy the environment, I get error:
k8s-head: unable to recognize "https://raw.githubusercontent.com/ecomm-integration-ballerina/kubernetes-cluster/master/calico/calico.yaml": no matches for kind "Deployment" in version "apps/v1beta1"
k8s-head: unable to recognize "https://raw.githubusercontent.com/ecomm-integration-ballerina/kubernetes-cluster/master/calico/calico.yaml": no matches for kind "DaemonSet" in version "extensions/v1beta1"
Does anyone else also encounter this issue ?

In addition to that, when sshing into the master node and running
kubectl describe pod coredns-5644d7b6d9-2tmzt,
I get following warning:
Warning  NetworkNotReady   113s (x152 over 6m54s)  kubelet, k8s-head  network is not ready: runtime network not ready: NetworkReady=false reason:NetworkPluginNotReady message:docker: network plugin is not ready: cni config uninitialized

The same issue on my side, a month ago I managed to deploy cluster locally, now having those issues.

Also, I`m not able to list resources(node, pods, etc) from worker nodes.

Also, I`m not able to list resources(node, pods, etc) from worker nodes.
vagrant@k8s-node-1:~$ kubectl get nodes The connection to the server localhost:8080 was refused - did you specify the right host or port?

This is because by default, the nodes are not configured to run such operations. If you run kubectl config view from the Master node, you'll see that there are configurations for a cluster, a server name, an IP address, etc. if you run the same command from either node, however, you'll see that these fields are left empty.
In order to enable such operations from the nodes, you can run sudo cat /etc/kubernetes/admin.conf from the Master node, and paste the output into a file of the same name and directory in each of your worker nodes.
Once this is done, run export KUBECONFIG=/etc/kubernetes/admin.conf and you should now be able to list resources from your worker nodes, as well as creating Deployments, Pods, Services, etc.

I faced the same issue today. Looks to be specification difference for latest kubernetes version.
fixed the same by using older kubernetes version (1.15.0). Changes in below statements in Vagrantfile. Added k8s version. And it fixed my issues. Ideally calico.yaml should be updated to latest k8s version though.
"apt-get install -y kubelet=1.15.0-00 kubectl=1.15.0-00 kubeadm=1.15.0-00"
"kubeadm init --kubernetes-version="1.15.0" --apiserver-advertise-address=$IP_ADDR --apiserver-cert-extra-sans=$IP_ADDR  --node-name $HOST_NAME --pod-network-cidr=<IP_ADDRESS>/16"
Thanks,
Deepak


Bump yaml from 2.4.1 to 2.4.2

Bumps yaml from 2.4.1 to 2.4.2.

Release notes
Commits

updated-dependencies:

dependency-name: yaml dependency-type: direct:development update-type: version-update:semver-patch ...

dup of #3762


MINOR: Update year in NOTICE

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks @rajinisivaram! Merging it without waiting for the build to complete as the build do not verify the NOTICE file.


[sync] crypto/x509: add new CRL parser, deprecate old one

https://github.com/golang/go/commit/2de2f6df64a8fff36fe9752f893a7cfff4590762
Adds a new, cryptobyte based, CRL parser, which returns a
x509.RevocaitonList, rather than a pkix.CertificateList. This allows us
to return much more detailed information, as well as leaving open the
option of adding further information since RevocationList is not a
direct ASN.1 representation like pkix.CertificateList. Additionally
a new method is added to RevocationList, CheckSignatureFrom, which is
analogous to the method with the same name on Certificate, which
properly checks that the signature is from an issuing certiifcate.
This change also deprecates a number of older CRL related functions and
types, which have been replaced with the new functionality introduced
in this change:

crypto/x509.ParseCRL
crypto/x509.ParseDERCRL
crypto/x509.CheckCRLSignature
crypto/x509/pkix.CertificateList
crypto/x509/pkix.TBSCertificateList

The structure Certificate's change will be in golang 1.18.


How do you transpose a matrix?

Does the library have an equivalent to Numpy's way of transposing matrices (matrix.T)?

Math.switchdim.
Or it might be math.transpose now.. I'm not sure which version the name change kicks in

It will be math.transpose after this PR is in:
https://github.com/PAIR-code/deeplearnjs/pull/167
Before this PR, you can use Math.switchDim, but this is only supported on a NDArrayMathCPU object. Once that PR is in, it will be available on the GPU.

math.transpose is now in. Feel free to close this issue if this is sufficient!


Please hide repository

It seems to me that this repo violates the Honor Code of Coursera.
They explicitly ask ppl to not engage in any activity that may dishonestly improve the result of other students. That's in their forum and elsewhere. This page is on the first page when googling for the course, ppl find this repo whether they want or not.
Yes, I've seen #5, #13, #22, #25. It makes me sad. If you enjoyed the course you could thank them by honouring the code.

just kindly open issues on all the repo with the solution ( just about 300 repos )
then I will hide it :)
Sent from my iPhone

On 14 Dec 2017, at 3:04 AM, Lennart Kolmodin<EMAIL_ADDRESS>wrote:
It seems to me that this repo violates the Honor Code of Coursera.
They explicitly ask ppl to not engage in any activity that may dishonestly improve the result of other students. That's in their forum and elsewhere. This page is on the first page when googling for the course, ppl find this repo whether they want or not.
Yes, I've seen #5, #13, #22, #25. It makes me sad. If you enjoyed the course you could thank them by honouring the code.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Other ppl breaking the Honor Code shouldn't mean that you also do it. Two wrongs don't make a right.
Also, your repo shows up on the first page when searching for the course. That is, even without searching for solutions. The "other 300" don't show up.

You should hidden your repo, I just realize you no only put the content but also put your answer.
These answer itself is not very difficult, but will lead us(who not very good at deep learning) to read the documents and API to find the answers.
If you are good at this area, you should write on how to solve these problems, write more content but please don't just put the answers.


template with v-if attribute doesn't render on Safari [vuejs version v0.12.9]

I ran into an issue while using vue on the mobile.I'm using template tag to create a three layer nested structure with v-if attribute.But the innermost layer wasn't able to render correctly on Safari,while it ran just well on chrome(android).
Here is the jsfiddle link.
http://jsfiddle.net/vq4utpd1/
You could try running it at safari & chrome to see the different renderings.
Does anybody ran into this issue before?
Any help will be appreciated!
Screenshot on Chrome:

Screenshot on Safari:

Thanks in advance, guys.

Duplicate of https://github.com/yyx990803/vue/issues/1118 (already fixed)

@yyx990803 右大大，我很确定用了最新的12.9版本呀
jsfiddle link:
http://jsfiddle.net/qinbinhua601/vq4utpd1/1/

新版本刚发布，升级到 0.12.10 就有了


Issue with game speed and fullscreen mode [R6 Black ops 2.0] (Windows 11)

Hi!
I'm currently trying to play R6 Black ops 2.0 on windows 11 with DDrawCompat.
I got two major problems :

For some reason I can't force the game to be in window/borderless. I did try with the fullscreen optimization option to off however, the OS windows seem to force the game to be in full screen.
The game run really too fast. I did try to limit fps to 30 even changing all fps limiter options, vsync off and on, changing CPU affinity to 1, and forcing a low refresh rate than my screen (60). None is working

I had the same problem in windows 10 however, I was able to make it work by running the game in window mode with voodoo which I also try with no luck on windows 11
I attached a debug session of the game
My spec is gtx3070 WITHOUT GSYNC with 144hz monitor 1440p running windows 11.
Thank you so much!
DDrawCompat-R6BO.zip

How do you know if the game is running in exclusive or borderless fullscreen mode? Why does it matter? Or did you mean real windowed mode, where the rest of the desktop is visible?
Anyway, based on your logs it looks like borderless fullscreen to me, but I'm not sure because you have GPU scheduling enabled, and I don't know what exclusive fullscreen mode is supposed to look like in that case. I don't have hardware to test that.
As for the game speed, it doesn't depend on FPS at all, so any FPS controls probably won't help. I guess the game uses timers for that, so it's probably some timer related issue, not a DirectX problem. Have you tried restarting your PC? Some timers are known to be problematic due to overflows in old games if your PC hasn't been properly restarted in a while.

@narzoul Alt-tabbing makes all my screen flash like when I'm in exclusive. I don't think it should do that in borderless

The game still changes display modes when alt-tabbing, unless your desktop resolution and refresh rate matches the one chosen by the application (or overridden by DDrawCompat). Based on the config in your logs, the "flash" is probably caused by the DisplayRefreshRate=60 setting. Change it back to app or desktop and see if it helps.
I have no idea about the game speed issue. It works fine on Windows 11 for me, so I doubt that it's caused by that, unless you are on some Insider Preview build. I might try to mess with the timer APIs later to see if I can get the game to run faster or slower that way, though a clean system reboot (not a shutdown) should have fixed any issues related to that as far as I know.

The game still changes display modes when alt-tabbing, unless your desktop resolution and refresh rate matches the one chosen by the application (or overridden by DDrawCompat). Based on the config in your logs, the "flash" is probably caused by the DisplayRefreshRate=60 setting. Change it back to app or desktop and see if it helps.
I have no idea about the game speed issue. It works fine on Windows 11 for me, so I doubt that it's caused by that, unless you are on some Insider Preview build. I might try to mess with the timer APIs later to see if I can get the game to run faster or slower that way, though a clean system reboot (not a shutdown) should have fixed any issues related to that as far as I know.

Yeah, the flash is my display "flick" like it's been turned on or changed its source.
Changing DisplayRefreshRate changes nothing, I will try on my second display instead.
I "fixed" the game speed by using the in-game feature with the button "F6" and "F8".
I really don't know what it's doing behind the scene however, after pressing like 5 times it seems to run at the native speed.

Changing DisplayRefreshRate changes nothing, I will try on my second display instead.

Any luck with that? It really should have worked, but you can try again with v0.5.0. Send me debug logs after reproducing the issue if it still doesn't work. Maybe a video recording would also help.


Error serializing TimeSpan.Zero

The TimeSpanSerializer doesn't correctly serialize 0.
This is needed for alarm triggers at event start time for example which, should look like
TRIGGER:P0D
Instead the Serializer only generates:
TRIGGER:P
Bug was also already present in DDay.iCal

Hi Alex, please reference your request to the specific part in the RFC 5545 spec. I can not find the default null value description for duration (3.3.6 in RFC5545).

`dur-value  = (["+"] / "-") "P" (dur-date / dur-time / dur-week)
And every possible value has
1*DIGIT
Thats why it can't be just P but P0D

I wonder if TimeSpanSerializer.SerializeToString can be delegated to NodaTime, which supports ISO-8601 formatting OOTB:
http://nodatime.org/1.2.x/userguide/period-patterns.html
For some reason, an ISO-8601 TimeSpan.ToString("o") isn't a thing, which is weird, because I'd expect the date and time types to be symmetrical where possible.

it's not exactly ISO-8601

Note that unlike
[ISO.8601.2004], this value type doesn't support the "Y" and "M"
designators to specify durations in terms of years and months.

Fixed in a12a089731abd846576780ced1d82b4cd1763de6
Available in nuget version 2.2.10


chore: change out from stderr to stdout

Coverage increased (+0.04%) to 73.906% when pulling 5a6144a947d6050700424caeb7a6b7694ea24dd0 on howieyuen:log-to-stdout into 13781cae112ec1f30af31c7eca9e7bba796fea1c on KusionStack:main.


iterate over lxc containers

If I try
lxc stop a b c
and a b and c are all three running it works as expected.
However if b for whatever reason is already stopped, a will be stopped then a message will be returned stating "error: The container is already stopped"
and c is kept running.
I feel it would be more logical that either:
upfront it is being checked that all containers are running (and can be stopped) and issue an error if not.
or:
the command just proceeds with the next container (and so in the above example both a and c will be stopped).
Similar for other commands (destroy, start, ...).
(I bumped on this because I wanted to stop a few containers on the command line and one of them was already stopped).

So I think that code needs some tweaking to actually do all those actions in parallel and just report errors as they come (if any). So in your case, all running containers would be stopped and the one which is already stopped will cause an error to be printed (and error code to be returned) but without affecting the stop operation for the others.
Would that work for you?
(It would also have the side effect of being much quicker as all containers would stop at the same time instead of in sequence)

Sounds great to me!

stgraber@dakara:~/data/code/lxc/lxd (stgraber/master)$ lxc start c1 nesting c5
c5: error: not found
nesting: error: The container is already running
nesting: Try `lxc info --show-log nesting` for more info
c1: error: The container is already running
c1: Try `lxc info --show-log c1` for more info

error: Some containers failed to start

So I've got that working pretty well. Just need to make the solution a bit more generic so I can apply it everywhere we take multiple arguments.


ImportError: cannot import name 'IndexMixin'

Hi,
When I ran
import scanpy.api as sc

I met this error:
     19 from scipy import sparse
     20 from scipy.sparse import issparse
---> 21 from scipy.sparse.sputils import IndexMixin
     22 from natsort import natsorted
     23

ImportError: cannot import name 'IndexMixin'

Is there any requirements for the version of scipy?
Thanks in advance,
BP

Hi, I just tried install a higher version of scipy, and it works now.
pip install scipy==1.2.1

Refer to: https://www.scipy.org

I also ran into this issue when using scanpy==1.4.2 and scipy==1.3.0 (which are the versions installed when I install using conda). Enforcing scipy==1.2.1 in my conda environment file fixed it.

This problem also exists in 1.4.3.

duplicate of theislab/anndata#146

Hi. I'm struggling with the same issue that is not resolved with the suggestions above (using other scipy version==1.2.1). I have scanpy==1.6.0 and scipy==1.5.2 in my environment. I am using Ubuntu in Windows.
And when I import scanpy in jupyter notebook, I get the error:
ImportError: cannot import name 'IndexMixin' from 'scipy.sparse.sputils' (/home/levinbioinformatics/anaconda3/envs/scgen-env/lib/python3.7/site-packages/scipy/sparse/sputils.py)
Please advise!

I have the same issue as @levinhein!

Same issue here. Downgrading scipy from 1.5.4 to 1.2.1 helps.


Close unused project sets in Forge and Assembly

Describe the Issue
There is a substantial amount of non-production instances that can be cleaned up and deleted. Try to get closer to the baseline cost
This issue might require additional effort with closing accounts due to permission issue identified previously.
Acceptance Criteria

[ ] Keep one project set for Forge and Assembly

PL - Nov 20 -
Forge: Keep 2 projects
Assembly: 2 projects

Main assignee: Max


No signature of method: hash.android() is applicable for argument types

Hello,
Environment:
Android: API 31
BuildTools: 31.0.0
GradleWrapper: 6.7.1
Android Gradle Plugin: 3.6.4
Kotlin: 1.6.21
During the build of version 1.1.7 this error below is occurring.
> Configure project :flutter_contacts

WARNING: The option 'android.enableR8' is deprecated and should not be used anymore.

It will be removed in a future version of the Android Gradle plugin, and will no longer allow you to disable R8.

FAILURE: Build failed with an exception.

* Where:

Build file '/Users/vagrant/.pub-cache/hosted/[pub.dev/flutter_contacts-1.1.7/android/build.gradle](http://pub.dev/flutter_contacts-1.1.7/android/build.gradle)' line: 30

* What went wrong:

A problem occurred evaluating project ':flutter_contacts'.

> No signature of method: build_9udfh814bypf1noppqbc8eqkz.android() is applicable for argument types: (build_9udfh814bypf1noppqbc8eqkz$_run_closure2) values: [build_9udfh814bypf1noppqbc8eqkz$_run_closure2@27da07e5]

* Try:

Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Thanks for reporting and fixing @trfiladelfo !

Hello, you don't want to make your contact list eternal / The version from a few years ago


Correct typo in setting of PR upload base ref

Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.

Great integration test! I see that my typo was just missing the curly brackets.

Great integration test! I see that my typo was just missing the curly brackets.


go_1_20: 1.20.2 -> 1.20.4

Backport of https://github.com/NixOS/nixpkgs/pull/224714 and https://github.com/NixOS/nixpkgs/pull/229564
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Oh crud. Looks like I messed something up. Sorry for the noise!


Options for 2024.2 and 2025.0 releases

Thank you for providing this easy-to-use GitHub action to setup oneAPI resources.
Since your latest update in May, I noticed that Intel oneAPI has had two new releases, i.e., 2024.2 and 2025.0.
I wanted to kindly ask if it is possible to add these new releases to your GitHub action. Let me know if I can somehow support you. I will be happy to help with the right instructions.

I pubished an update. Let me know if it works. Please look over the README. I added some directions on how to do an update. You have to cut/paste installer URLS from spack.

Thank you for the update. So far it seems to work fine in the projects I'm using. Next time, I will prepare a patch myself following your README and #48.
I noticed that one of your CI jobs is failing: compilers/Test MPI
That is because ifort has been discontinued. I will prepare a patch.

Thanks for the PR's and offering to help with the updates!


PRO-941 Set up

Requirements and project set up. I kept this much simpler than our 2.x (and 3.x alpha) pages.
I link to the different requirement software download pages (Node has it on the home page). If we had a nice tab component to go through each individual system it might be worth spelling it out, but I looked at our peers and that's not something I see anyone doing. It mostly seems to make it look more involved to set up Apostrophe.

Thank you much! It's almost a CMS.


Add init to compose

Address rancher/rancher#8608

test failures indicate init wasnt picked up by docker. We'll have to commit the agent PR, release, and update this PR with the new agent version. I'll do that at some point

Closing in favor of https://github.com/rancher/cattle/pull/2823


Create Dropdown Searchable and should be like combobox

We need some feature like this in bootstrap 5 quickly.
check this https://developer.snapappointments.com/bootstrap-select/
we need similar to this inbuild in bootstrap 5.
Before opening:

Search for duplicate or closed issues
Read the contributing guidelines

Feature requests must include:

As much detail as possible for what we should add and why it's important to Bootstrap
Relevant links to prior art, screenshots, or live demos whenever possible

This is extra 60KB of minified code. I don't think this is a good idea, better to keep it separate.

In BS 5 they deprecated the Jumbotron and some other stuff too.
I think they should add and then give dev to choose which js function. If we need it in our project then we will add into it.


Failure creating Event Hub in existing namespace: Forbidden

I'm having trouble creating a new Event Hub in an existing Service Bus namespace using SBE.
I have an existing ASB namespace “mrtl-rma-test-01” that contains a Queue and a Topic. I've connected successfully to it using the default RootManageSharedAccessKey.
I tried to create an Event Hub by right-clicking on the Event Hubs node in the navigation tree, selecting Create Event Hub, putting “mrtl-rma-test-eventhub-01” as the Relative URI, and clicking the Create button in the lower left.
SBE froze for a bit over 30 seconds, and then displayed some error messages in the log.
<12:29:56> The files C:\Users\NY510094\AppData\Roaming\Service Bus Explorer\UserSettings.config and C:\Users\NY510094\local\repos\ServiceBusExplorer\src\ServiceBusExplorer\bin\Debug\ServiceBusExplorer.exe.Config are used for the configuration settings.
<12:30:00> The application is now connected to the sb://mrtl-rma-test-01.servicebus.windows.net/ service bus namespace.
<12:30:00> MessagingFactory successfully created
<12:30:02> The queue rma-pricing-requests has been successfully retrieved.
<12:30:03> The topic rma-pricing-notifications has been successfully retrieved.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:823b965b-c281-40e6-9ca4-cb8e8b221c1d, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:36:55 PM. Method <CreateEventHub>b__0: retry 1 of 10.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:4cdb9b33-5321-47a4-b8cd-8a4a5f40816e, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:36:58 PM. Method <CreateEventHub>b__0: retry 2 of 10.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:b3a2076d-5c38-4092-a66f-411e934f7b42, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:37:00 PM. Method <CreateEventHub>b__0: retry 3 of 10.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:34bb3a36-b1d7-4af8-8489-cb1ab0a3d5e7, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:37:03 PM. Method <CreateEventHub>b__0: retry 4 of 10.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:92475651-b846-490c-ac79-655dcb90cadb, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:37:06 PM. Method <CreateEventHub>b__0: retry 5 of 10.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:b3988ad8-5367-4761-ac74-42371aaf4bcc, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:37:08 PM. Method <CreateEventHub>b__0: retry 6 of 10.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:12e87477-0bbd-4760-a551-8d9013e96786, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:37:11 PM. Method <CreateEventHub>b__0: retry 7 of 10.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:472ea391-446c-4e83-86a1-fbaf9627cadd, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:37:13 PM. Method <CreateEventHub>b__0: retry 8 of 10.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:cb02c886-f73a-416b-8c89-4bd0d97b5c54, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:37:16 PM. Method <CreateEventHub>b__0: retry 9 of 10.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:00eeb0aa-7970-4ffe-89ad-dc8abe6d69bc, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:37:19 PM. Method <CreateEventHub>b__0: retry 10 of 10.
<12:37:21> Exception: The remote server returned an error: (403) Forbidden. TrackingId:aebb2b0f-f2b9-4873-ae64-98b9a6d31c28, SystemTracker:mrtl-rma-test-01.servicebus.windows.net:mrtl-rma-test-eventhub-01, Timestamp:3/14/2019 4:37:21 PM
<12:37:21> InnerException: The remote server returned an error: (403) Forbidden.

Is this expected to work? When I look at Event Hubs in the Azure Portal web page, it looks like they exist in separate namespaces from the ASB Queues and Topics. For example, they're a separate "service" from the main Service Bus service. And when I create an Event Hub in the Portal, it won't allow me to reuse the same namespace that's already in use for a Queues and Topics namespace.
A couple usability issues besides the failure itself:

The UI froze for over half a minute, with no indication that it was working on anything.
While the exception timestamps indicate they came in over the course of 30 seconds, the log entries for them weren't written until the end of the process.
A 403 Forbidden error isn't typically expected to go away over time, so should SBE really be doing those retries?

Yeah, I think the issue is that Namespaces for Queues/Topics, Event Hubs, and Relays are different kinds of namespaces, and can only contain their appropriate type of entity.
I created a new Event Hub namespace in Azure Portal, connected to it with SBE, and created a new Event Hub there, and it worked. Then I tried to create a Queue in that namespace, and got basically the same error as before:

<13:01:34> The files C:\Users\NY510094\AppData\Roaming\Service Bus Explorer\UserSettings.config and C:\Users\NY510094\local\repos\ServiceBusExplorer\src\ServiceBusExplorer\bin\Debug\ServiceBusExplorer.exe.Config are used for the configuration settings.
<13:02:10> The application is now connected to the sb://mrtl-rma-eventhub-test-01.servicebus.windows.net/ service bus namespace.
<13:02:10> MessagingFactory successfully created
<13:02:53> The event hub test-hub-01 has been successfully retrieved.
<13:03:04> 2 partitions for the event hub test-hub-01 have been successfully retrieved.
<13:03:04> 1 consumer groups for the event hub test-hub-01 have been successfully retrieved.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:1194f199-badb-46f4-a288-21aa83183b99, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:13 PM. Method <CreateQueue>b__0: retry 1 of 10.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:2ddef03c-3847-49f9-be1b-ed507c07543d, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:16 PM. Method <CreateQueue>b__0: retry 2 of 10.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:168d1621-a2dd-4e03-9345-599fd16deb74, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:18 PM. Method <CreateQueue>b__0: retry 3 of 10.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:25d17f74-419e-4482-8381-7da466fc2c05, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:21 PM. Method <CreateQueue>b__0: retry 4 of 10.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:2c949c0b-1c65-4447-abcf-231b0322590b, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:24 PM. Method <CreateQueue>b__0: retry 5 of 10.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:26c14729-9bd0-45bf-a695-678d95fca85f, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:26 PM. Method <CreateQueue>b__0: retry 6 of 10.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:c866f718-fb76-4e01-9541-74f2b133fcba, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:29 PM. Method <CreateQueue>b__0: retry 7 of 10.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:67018b24-5785-4f0d-9ab1-621b2015ce4d, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:32 PM. Method <CreateQueue>b__0: retry 8 of 10.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:0cc9b025-25a5-4d3c-b3eb-3aae8a9f6d20, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:34 PM. Method <CreateQueue>b__0: retry 9 of 10.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:62c55004-8888-4b70-955a-b78988d37680, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:37 PM. Method <CreateQueue>b__0: retry 10 of 10.
<13:06:39> Exception: The remote server returned an error: (403) Forbidden. TrackingId:f6923809-c55e-46e1-8a4e-f9ac7bd62e18, SystemTracker:mrtl-rma-eventhub-test-01.servicebus.windows.net:my-stupid-test-queue, Timestamp:3/14/2019 5:06:39 PM
<13:06:39> InnerException: The remote server returned an error: (403) Forbidden.

Is there a way SBE could detect what kind of Namespace it's connecting to, and only enable the creation controls for the appropriate entity types for that namespace?

Microsoft changed the way namespaces work about two years ago. Namespaces created before that may contain all four kinds of entities, but newer namespaces are limited to one kind as you noticed,
I do not think it possible to determine if is an old or new namespace. In the new Service Bus client there is support for determining which kind of entity a namespace is using but I don't know how it reacts when querying an old, mixed namespace. Service Bus Explorer is still using the old Service Bus Client. We do want to move to new one.

The UI froze for over half a minute, with no indication that it was working on anything.
While the exception timestamps indicate they came in over the course of 30 seconds, the log entries for them weren't written until the end of the process.

That is something that would be great if it was fixed.,,

A 403 Forbidden error isn't typically expected to go away over time, so should SBE really be doing those retries?

You are most likely correct. The new Service  Bus Client has more structured exceptions which makes it easier to determine the right kind of action.

Okay, that makes sense.

That is something that would be great if it was fixed.

I'll have a look at the code and see if it's something I could help with.
The new Service Bus Client – is that the Microsoft.Azure.ServiceBus client discussed in #284? Is it mature enough to start using now/soon?

I don't know how it reacts when querying an old, mixed namespace

Looks like there's a NamespaceType.Mixed enum to indicate those.

The new Service Bus Client has more structured exceptions which makes it easier to determine the right kind of action.

Ah, yep. I see an IsTransient indicator there.

Thankful for your help.
Yes, that is the one in #284
Great to see that there is a Mixed enum at least.

I'm looking at MainForm.cs in the Design view in Visual Studio 2017, and I don't see a MenuStrip for Event Hubs, even though I see one for everything else in the navigation tree. Am I missing something?

Okay. I think there's a quick partial fix for this. The retry logic in RetryHelper already has some logic to distinguish transient from non-transient exceptions. It checks MessagingExceptions to see if they're transient, and treats all MessagingEntityAlreadyExistsExceptions and CommunicationExceptions as non-transient.
                try
                {
                    action();
                    return;
                }
                catch (MessagingEntityAlreadyExistsException)
                {
                    throw;
                }
                catch (CommunicationException)
                {
                    throw;
                }
                catch (MessagingException ex)
                {
                    if (numRetries == 0 || (!ex.IsTransient))
                    {
                        throw;
                    }
                    writeToLog([...]);
                    Thread.Sleep(retryTimeout);
                }

The problem happening with this Event Hub creation is that the exception thrown by the failed creation in the wrong type of Namespace is a Microsoft.ServiceBus.Messaging.QuotaExceededException. And there's no specific catch clause for QuotaExceededException, so it falls down to the generic catch (Exception ex) clause. And the generic clause treats all unknown exceptions as transient.
                catch (Exception ex)
                {
                    if (numRetries == 0)
                    {
                        throw;
                    }
                    writeToLog([...]);
                    Thread.Sleep(retryTimeout);
                }

I'm writing a fix that will specifically catch QuotaExceededException and treat it as non-transient. I think that's fair, since that error is unlikely to go away unless you use a different management function to remove an existing item, or change your Pricing Tier, or something that will require action besides just waiting. (The case of other admins concurrently changing quota-affecting stuff can be ignored here, I think.)
That will cut the freeze time down from 30 seconds to just a couple seconds, which is a lot more acceptable.
I think you could go one further, though: Maybe the generic Exception catch clause should consider unknown exceptions to be non-transient, and not retry on them. What do you think?

I did a PR #330 that partially fixes this by special-casing QuotaExceededException to be considered non-transient. It doesn't go all the way and change generic Exception to be non-transient, though. I'm being conservative.
That only partially fixes this problem, though. The UI still freezes for a few seconds with no indication of what it's doing. And Relay creation still freezes for 30 seconds. I think the next step is to put up a progress bar to let the user know some work is actually happening, and the UI didn't encounter an internal error.


vm/adb: add 60 sec delay to allow Android boot to finish

vm/adb: add 60 seconds delay after reboot to wait until system finishes initialization
executor: removed condition around 'initialize_tun'
Often after reboot Android devices experience a race condition. When a phone is still on the screen with 'G' logo and a progress bar syz-fuzzer already starts execution. Sometimes the phone doesn't go past this screen and can't start all services. In this case fuzzing continues however it doesn't produce results. I can assume that due to dozens per second syscalls Android can't finalize initialization in time. I do not know what exactly causes this race condition however it doesn't happen if I comment out 'initialize_tun'.

@a-nogikh is this infrastructure issue or something wrong with the code?
https://github.com/google/syzkaller/runs/7437040275?check_suite_focus=true


T619 Nested window functions

You have an error in your SQL syntax: SELECT CUST_CODE CUST_NAME row_number() over (order by opening_amt) FROM CUSTOMER order by opening_amt no viable alternative at input '(' at line 1 position 48 near @948:48='('<29>1:48

SELECT CUST_CODE, CUST_NAME, row_number() over (order by opening_amt) FROM CUSTOMER order by opening_amt


Liver ontology expansion for colouring of anatomogram shapes/structures

cell type shapes/parent structures not active (grey) for matching cell types and theIr parent structure (liver lobule):
https://wwwdev.ebi.ac.uk/gxa/sc/experiments/E-HCAD-9/results/anatomogram
cholangiocyte, Kupffer cell, hepatic stellate cell, both the sinusoidal endothelial cell types and the portal endothelial cell all should have a corresponding shape in the anatomogram highlighted as well as the parent structure (liver lobule) in the level-up (main image)
e.g. https://www.ebi.ac.uk/ols/ontologies/efo/terms?iri=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FCL_1000488

Duplicate of https://github.com/ebi-gene-expression-group/atlas-web-single-cell/issues/128.

hi! - so I had a look at dev again and we're still not seeing all the cell types we should see highlighted:
cholangiocyte, Kupffer cell, hepatic stellate cell, both the sinusoidal endothelial cell types and the portal endothelial cell all should have a corresponding shape in the anatomogram highlighted
this is not just a duplicate of #128 - which is mainly about retrieval of parent terms  (although we have that problem here too) - but here we do not have even the direct matches highlighted - e.g. we have data annotated as 'Kupffer cell' and there is 'Kupffer cell' shape in the anatomogram (tagged with the relevant ontology ID and label), but it's not highlighted. Same for all the other cell types I mentioned above too
https://wwwdev.ebi.ac.uk/gxa/sc/experiments/E-HCAD-9/results/anatomogram

@sfexova Could you please let us know if this is the case in the test environment?
https://www-test.ebi.ac.uk/gxa/sc/experiments/E-HCAD-9/results/anatomogram

@alfonsomunozpomer Apologies! works great in test, all cell types highlighted as they should be. (Colouring of the parent structure - liver lobule, the tiny hegagons - in the level-up image structure doesn't but that's a different story, same as 128, like you said).


Add a better error message when a user runs the compilot in a nested directory

If you're supposed to run the copilot in the parent directory but run it any number of levels in, you'll see a message along the lines of:
AssertionError: Unexpected mypy exit code 2. Process info: CompletedProcess(args=['mypy', '--config-file', '/var/folders/q5/xdhsj6997hl8qghm3bx6v74r0000gn/T/mypy-copilot-kcxuh4uf/mypy.ini', '--show-error-codes', '--error-summary', '.'], returncode=2, stdout='', stderr='. is not a valid Python package name\n'

You will see . is not a valid Python package name whether you're in correct_dir/x or correct_dir/x/y.
As a user of typing-copilot, I first activated poetry shell, installed typing copilot, and then tried running typing-copilot init, which failed with the above error. It's easy to miss the step of changing dirs, so it would be beneficial to wrap this error with a hint / suggestion.

Thanks! Definitely a good place to add some polish.


routing to cant juggle page

Hey :) I added a page for can't juggle and did the routing so that someone who doesn't know how to juggle now gets sent to a can't juggle page. Those who can juggle 1-5 remain the same.

this looks fabbbb!!


Adding footsteps

Hey there! I'm really enjoying using this asset package! However, one drawback I have is I have no idea how to add footsteps.
I'm currently working on a horror game and there's a section where the ambience just stops and I want there to be footsteps to make it more spooky.
Please let me know what script I will need to edit to add footsteps, and what I will have to write down to make footsteps efficient! :)

There are free footstep systems in Unity Assets Store. Maybe you can try to use one of them?


Merge the v2 branch into master

Curious if there are plans to merge the v2 branch into master as it's been a while since the latest update to the v2 branch? Having v2 API support in the SDK would be very helpful!
I'm happy to help out with some PRs as well if desired!

cc @r0adkll

Ah, yes, sorry. Let me get this merged and figure out a new deployment since jcenter is no more

Thanks!


bug: 404 inference not found

Here's the list info detail
NAME       ENDPOINT                                             STATUS  INVOCATIONS  REPLICAS
nginx-ing  http://nginx-ing-yjmz0eabd9c97h08.modelz.live        Ready             0  1/1
           http://localhost:80/inference/nginx-ing.test

Then when I access the deployment by using endpoint, the error message has been raised
curl  http://nginx-ing-yjmz0eabd9c97h08.modelz.live
{"http_status_code":404,"message":"inference not found","request":"GET /inference/nginx-ing.default/","op":"inference-proxy","error":{}}

Hi,
It's weird and seems the ip address is missing. The link should looks like http://nginx-ing-yjmz0eabd9c97h08.<ip-address>.modelz.live
Could you try initialize mdz with server public ip?
# Provide the public IP as an argument
$ mdz server start <IP_ADDRESS>


Improve Development/Release Workflow

Development:

The CLI expects the promdump .tar.gz file to be found in the /tmp directory. This is inconvenient during development.

[ ] Update the core target to bundle up the promdump .tar.gz
[x] Add a CLI option to specify where to read the dev promdump .tar.gz file from (2b463c3f83ab7ddcfb1e214c6f6ba7661d6b09fa)

Release:

Auto-update the kubectl plugin manifest during a release. See krew doc here.

Fix in #7.


fix 360afchrome freeze bug

using the previous thing it froze my 360AFChrome cuz the lines made it glitch out

All good


Feature for default customization in action menu

Action menu html files were edited to be render in a list tag or just
a button to leverage existing code. Default button is rendered by
action_menu because the toggled list is rendered in same pattern.
Looking for alternative ways of writing documentation. Preferred
separate code block with instructions, but syntax would not allow.
Thanks for contributing to Wagtail! 🎉
Before submitting, please review the contributor guidelines http://docs.wagtail.io/en/latest/contributing/index.html and check the following:

Do the tests still pass? (http://docs.wagtail.io/en/latest/contributing/developing.html#testing) yes
Does the code comply with the style guide? (Run make lint from the Wagtail root) yes
For Python changes: Have you added tests to cover the new/fixed behaviour? yes
For front-end changes: Did you test on all of Wagtail’s supported browsers? Please list the exact versions you tested. chrome
For new features: Has the documentation been updated accordingly? Yes, but I would like to make it look a little better. Could use help on where to read up on writing documentation.

https://github.com/wagtail/wagtail/issues/5438

Just a thought - rather than having a default flag and requiring exactly one menu item to have it set, could we simply make a rule that the menu item with the lowest (or highest) order value is made the default? That way, we avoid having to redefine the whole list to set a new default, and it's easier to resolve clashes if these hooks are used in more than one place (for example, there are multiple third-party extensions installed that add new menu items).

I am not sure why the test is breaking. I read the documentation, and it seems like valid syntax. https://docs.djangoproject.com/en/2.2/topics/testing/tools/#django.test.SimpleTestCase.assertHTMLEqual. Does anyone have in suggestions?

I used assertTemplateUsed instead of assertContains because templates are rendered by the database of the project. Each database renders html a little differently, so that why my test work only on sqlite. I can hardcoded each possible response for each database, but I believe this method is more maintainable.
The only downside is that my test do not proves the last item is default. It only checks to if the appropriate templates were used. if the item is default, it must be displayed first.
I am still researching different ideas, but what do you think is best?

templates are rendered by the database of the project

I can promise that they aren't... :-)
We need to get to the bottom of why these differences are occurring. On Slack I suggested printing response.content (NOT response.context), as this would allow us to see the generated HTML, and therefore how it differs from the expected output. We can then decide on the best way to account for this in the tests - for example, if the only differences are in whitespace, we can easily set the tests to ignore that (although even then, I'd still want to understand why that's affected by the choice of database).

Hi @rjpruitt16,
The assertContains was failing because you had the page ID 6 hard-coded in the Unpublish and Delete URLs:

            Unpublish

            Delete

The single_event_page is created at the start of the test (in the setUp method), so it'll only be assigned an ID of 6 if the ID sequence is reset after every test, and apparently Postgres and MySQL don't do that. If you change this to pass in the actual ID:
    menu_items_html = '''

            Unpublish

            Delete

            <input type="submit" name="action-submit" value="Submit for moderation" class="button" />

            <button type="submit" class="button action-save button-longrunning " data-clicked-text="Saving…" >Save draft</button>

'''.format(id=self.single_event_page.id)

then the test should pass on all databases.

I saw this too. My apologies for the delay, I just started school again. I will make the change and push now.

Is there something else to be built? @gasman @kaedroho

@gasman @kaedroho

Thanks again @rjpruitt16!

Have rebased on current master here https://github.com/gasman/wagtail/tree/rjpruitt16-SaveDraft - I'll apply the final fixes so that we can hopefully get this merged for 2.7.

Change the behaviour to make the lowest order the default, and set SaveDraftMenuItem to 0. (It's unlikely that any existing code has used 0 or something negative as the order.)

This approach is now implemented in https://github.com/gasman/wagtail/commit/187b78849ffaf29f4050a16c589357a76d358a9a

Have now opened an updated PR to replace this one: #5612


Model's maximum context length error with ChatGPT plugins

I am testing out the newly released support for ChatGPT plugins with langchain. Below I am doing a sample test with Wolfram Alpha but it throws the below error
"openai.error.InvalidRequestError: This model's maximum context length is 4097 tokens. However, your messages resulted in 127344 tokens. Please reduce the length of the messages."
What is the best way to deal with this ? Attaching below the code I executed
`import os
from langchain.chat_models import ChatOpenAI
from langchain.agents import load_tools, initialize_agent
from langchain.tools import AIPluginTool
tool = AIPluginTool.from_plugin_url("https://www.wolframalpha.com/.well-known/ai-plugin.json")
llm = ChatOpenAI(temperature=0,)
tools = load_tools(["requests"] )
tools += [tool]
agent_chain = initialize_agent(tools, llm, agent="zero-shot-react-description", verbose=True)
agent_chain.run("How many calories are in Chickpea Salad?")`

The message you are pushing into the OPenai model has len more then it context length (4097) tokens , chunks your request into small length lesser then 4096 tokens

@sergerdn The error is caused as the plugin response is larger than context window of GPT-3. GPT-4 can probably overcome it but that would be just a workaround. Looking for a better solution which can work with GPT-3 too


Add TEMP keyword to CREATE TABLE statement

@alexfu, in relation to #12, add the optional TEMP keyword to CREATE TABLE statement.
Users were unable to create tables in the temp database.

Replace 2 existing classes CreateTableBuilder and CreateTableBuilder with one new class CreateTableSegmentBuilder.
Rename the existing enum DropType to ObjectType. It can also be used in creating/dropping view, trigger and index.

Cool! :v:


Add optional support for .webm movies ( libvpx library )

To do so, FFmpeg needs to be compiled with the --enable-libvpx option.  This optonal BSD codec/format is very important as it is specifically designed to work at 8K.  Most 8K movies use it.
Currently, albeit I added code to turn on the codec (trivial code change), tlRender reads the movies partially and then shows the frames black.  Just like what happened with the TopGun clip I showed you once.

Most 8K movies use it

In what context? For example the latest production test footage from the ASC uses mostly QuickTime ProRes:
https://dpel.aswf.io/asc-stem2/

reads the movies partially and then shows the frames black

Is this for a specific webm movie or all webm movies?

Most 8K movies use it

In what context? For example the latest production test footage from the ASC uses mostly QuickTime ProRes: https://dpel.aswf.io/asc-stem2/

Google and youtube seem to use it for their 8K movies.

reads the movies partially and then shows the frames black

Is this for a specific webm movie or all webm movies?

Fot most webm movies.  I only have an 8K movie that plays fine.

Here's a .webm movie that on my Mac it starts playing fine and then turns black;
https://mega.nz/file/2fhEUSza#6vrzJlNVAhkT8-VC3aRMPCEhr-91Pv4zIiO2rySTENM

Note:  I added a PR with the changes needed in tlRender to build FFmpeg with libvpx support, albeit only dynamically.


Find way to remove sleeps from CI pipeline

It adds a lot of time to the time it takes to run tests, and I think it should be possible to use some tricks to check whether we can continue instead of using sleeps. Not sure how yet...

I guess the naive way would be to poll a relevant end point - when it responds, it is ready to for testing.


Allow EmbedBuilder to accept an Object field value

Pull Request Etiquette
There are several guidelines you should follow in order for your Pull Request to be merged.

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the [contributing guidelines][contributing].

It is sometimes better to include more changes in a single commit.
If you find yourself having an overwhelming amount of commits, you
can rebase your branch.

Description
Allow the value type to be an Object because it won't break existing calls to this method and only add an extra method call (toString()) on any strings. An example of where this is useful is a field that displays an int like "Guilds" with value "9001". The developer can easily type: embedBuilder.addField("Guilds", String.valueOf(getGuildCount()), false);
But it's a lot cleaner for the developer to just type: embedBuilder.addField("Guilds", getGuildCount(), false)

Rejected, this is not a good design pattern.


Question about examples

The spec states that "all bound functions are exercised in the examples, to ensure the bindings and necessary wrapping, unwrapping, encoding, and decoding has been performed correctly".
My current bindings contain 942 exported functions. Creating a demo for each of these is going to be a massive task - can we simplify this requirement a little?
If ensuring the bindings are correct is the main goal, then I can simply call each one in turn with dummy arguments, but it would be part of a test suite, not necessarily the examples. I also already have a bit more confidence in the correctnesss of my bindings since they're based on an existing set of bindings for TypeScript.

Yes, simply calling each one and ensuring no exceptions are thrown (e.g. in a test suite) would be fine. It's just to provide a semi-easy method of making sure the bindings were defined correctly.

^^^ Also let me know if the above is too onerous.

Sounds good.
The TypeScript definition files contain some things which are not documented in the API docs, which I have been removing, and some things which use things like union types or any, so I need to spend a little more time to get working examples for those, but I'm almost done.

Awesome, look forward to seeing it.


Solving problem of new version of firebase.Promise

Solving error Namespace 'firebase' has no exported member 'Promise'.

Any update on when this will be merged?


Set UTF-8 as default encoding for peep

This was necessary for me to make the build, each time I used peep, work. Maybe will help someone too, and hopefully not break anything.

Thank you for the PR, unfortunately it's failing on Python 3.

I found that for Python 3 the default encoding is already UTF-8 and even cannot be changed. r?

I defer to @erikrose for review :-)

@erikrose r?

Sorry, I stopped working with peep since the times and created the patch. Feel free to close this PR.

Thanks. Peep being deprecated anyway, I think we'll close this until somebody else stumbles upon a similar problem.


ENH Hyperparameter search

This is a big one, but hyperparameter search could be interesting for how to combine layers/extractors into a pipeline.

scikit-learn may be working on this.... closing for now


Use debian as base image

This will reduce the size of the image.

As an official image has been released, this is no more needed.


HDFS-17160. Metrics fsyncCount should be computed correctly in flushOrSync.

Description of PR
Currently in method flushOrSync, it will only increase fsyncCount once when isSync == true.
But actually we fsync twice when checksumOut != null and streams.getDataOut() != null. So we should increase fsync count twice.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 28s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
32m  8s

trunk passed

+1 :green_heart:
compile
0m 53s

trunk passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
0m 50s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
checkstyle
0m 42s

trunk passed

+1 :green_heart:
mvnsite
0m 55s

trunk passed

+1 :green_heart:
javadoc
0m 51s

trunk passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
1m 16s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
1m 57s

trunk passed

+1 :green_heart:
shadedclient
21m 54s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 45s

the patch passed

+1 :green_heart:
compile
0m 48s

the patch passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
0m 48s

the patch passed

+1 :green_heart:
compile
0m 42s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
javac
0m 42s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 33s

the patch passed

+1 :green_heart:
mvnsite
0m 49s

the patch passed

+1 :green_heart:
javadoc
0m 38s

the patch passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
1m 10s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
1m 51s

the patch passed

+1 :green_heart:
shadedclient
22m 14s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
189m 24s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 40s

The patch does not generate ASF License warnings.

283m 52s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestFileChecksum

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5952/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5952

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 1b6b7c3a2bd5 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / f06cc7cfdf35ec94faf5702205b4bde035e1fc42

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5952/1/testReport/

Max. process+thread count
3721 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5952/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


Logo and background bug

Hello/GoodNight
I am writing to ask you how to change the logo and the image of me when I make changes because the bind her makes me a big black and purple square
ps: sorry I'm french I use google translation so it can have a fault
cordially
droopi29

Do you have Resource Loader installed to add your custom textures?
Also post your json file & your folder structure please.

hello
yes i have Resource loader

I need a screenshot of the problem pls.

Read the Resource Loader page again, assuming your image is in resources/test/textures/becube.png the resource location woud be "test:textures/becube.png"

Thank you for your help but I have another request is it possible teamspeak opens with a button and it logged in! Reponder me quickly got a server
cordially
Droopi29


I am also facing similar issue with Ehcache 3.x version and getting below exception with JDk11. So any idea to fix this.

I am also facing similar issue with Ehcache 3.x version and getting below exception with JDk11. So any idea to fix this.
EXCEPTION STACK TRACE = Unable to make field private jdk.internal.reflect.ConstructorAccessorImpl jdk.internal.reflect.DelegatingConstructorAccessorImpl.delegate accessible: module java.base does not "opens jdk.internal.reflect" to unnamed module @1e6a3214
ERROR STACK TRACE = [java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:340), java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:280),
Originally posted by @ripunjDev in https://github.com/ehcache/ehcache3/issues/2671#issuecomment-543531248

If you want to use byte-based sizing in Ehcache 3 in a modular JVM you will need to break open any module that contains cached classes to allow the Ehcache 3 code to access them. This is because Ehcache does byte-based sizing by walking the object graph using reflection and sizing the actual objects stored in heap. That should mean a sequence of:
--add-opens ${module}/${package}=ALL-UNNAMED

The exact sequence obviously depends on what types you are storing in the cache.
FYI: The reason this works in Hazelcast is because Hazelcast isn't really sizing objects on heap. Instead it's serializing the object and then sizing that byte array.


Fit to Screen Option

Hi @jerosoler,
Thanks man, great project, congratulations. 🥇
Is there any way to resize and center align all drawflow? Fit to screen option
Example:

Hi! Thanks!
The red box. You can make it any size you want. It's css.
If you mean moving the canvas view:

#88


Could not find "ModInstallPath (either in regedit or registry.vdf) [97]

System Specs:
OS: Garuda Linux
Kernel: 6.4.4-zen1-zen
DE: KDE Plasma 5.27.6

Clicking the "APPLY WITH BOOP" button on any given cover the menu to select the correct game properly appears. I then select the game and click on "OK". Then nothing happens. Restarting steam doesn't show me any new artworks or icons.
Looking into ~/.local/state/sgdboop_error.log shows me:
Could not find "ModInstallPath (either in regedit or registry.vdf) [97]
with a timestamp of everytime i tried to add artwork.
I installed it via the AUR but also tried the flatpak version, both of them give me the exact same error.
I have tried to start Steam as the Native Version or the Runtime version.
I saw that this error got posted here before but no one ever provided a solution. This is my try aswell. I'd love to have a solution for this issue.
Thanks and have a nice day!

I found that maybe your resource is relatively large so that the backend is still downloading the resource.

I found that maybe your resource is relatively large so that the backend is still downloading the resource.

you can test some smaller image. If it works, your can just wait for downloading.Maybe this plugin can support downloading info in the future

On a fresh Windows 11 and Steam install, it seems the ModInstallPath registry key is no longer created.
You can create this key as a workaround but the source should be fixed here.
To use the workaround, create the key string value, or paste the following into a reg file and run it:
ModInstallPath.reg:
Windows Registry Editor Version 5.00

[HKEY_CURRENT_USER\Software\Valve\Steam]
"ModInstallPath"="C:/Games/Steam/steamapps/common"

This resolves the error and symlink moaning in the logs and the tool works again.


Subscriptions inside composer

What is the best way to do multiple subscriptions inside to the composer function. I do in this way up until now:
export const composer = ({context, companyId}, onData) => {
  const {Meteor, Collections} = context();
  const handle1 = Meteor.subscribe('vehicles.list', companyId);
  const handle2 = Meteor.subscribe('companies.single', companyId);
  if (handle1.ready() && handle2.ready()) {
    const vehicles = Collections.Vehicles.find().fetch();
    const company = Collections.Companies.findOne();
    onData(null, {vehicles, company});
  }
};

Or do I have to do something like multiple composers for subscriptions. multiple composers

I think you've the correct way to handle it. Using multiple composer for this purpose is not a good idea. Use multiple composers  when dealing with multiple data sources.

Thank you


Bug: Invalid prices when discounting prices with active Avatax

What are you trying to achieve?
The uni prices returned from Saleor should not in increase when I change the quantity or something inside the order.
Steps to reproduce the problem

Activate Avatax
Create order in the channel where avatax is active
add adresses
add lines
add order discount - 100 %
add shipping
change the number of quantity for one of the lines.
The prices will change from 0.00 to a different value. It is not true as the discount - 100% is still applied.

What did you expect to happen?
I expect that the price of lines will change only when I add the discount. When I change the quantity the unit price should be the same.
Logs

I tested this locally and found that the issue occurs when Avatax is enabled, as it reduces the unit price (see the first screenshot). Additionally, the prices increase when the quantity for a specific line is increased. Even if I increase the quantity for line A, the prices for lines B and C also increase (see the first and second screenshots).
Link to internal discussion: https://saleorcommerce.slack.com/archives/C039MGJDWU8/p1677847853144589
Environment
Saleor version: 3.9.28
OS and version: OSX

Reported in: https://saleorcommerce.slack.com/archives/C017L12TRJS/p1677843900295369


Adding icon (or html) to submenu title

Version
2.6.5
Steps to reproduce
<ContextMenu id="id">
  <MenuItem onClick={onClick}>
    <Icon /> Do something
  </MenuItem>
  <SubMenu title="### how to show an icon? ###" >
    <MenuItem onClick={onClick}>
      <Icon /> Do something
    </MenuItem>
  </SubMenu>
</ContextMenu>

<ContextMenuTrigger id="id">
  click
</ContextMenuTrigger>

Expected Behavior
I want to add an Icon to the title of a submenu. Is it possiblie? I can't find something about it in the API docs.
Some Ideas on how it could look like:
<SubMenu>
  <SubMenu.title><Icon /> text with icon</SubMenu.title>
  <MenuItem>...</MenuItem>
</SubMenu

or
<SubMenu title="<Icon /> text with icon">
  <MenuItem>...</MenuItem>
</SubMenu

I would prefere version one as it looks cleaner to me.
Actual Behavior
I can't put html in the title attribute of the SubMenu:
<SubMenu title="No HTML here!">
  ...
</SubMenu>

You can use this
<SubMenu title={<Icon />}>
  <MenuItem>...</MenuItem>
</SubMenu

Yes of course. I haven't thought about that...


Is this package supported if I use Django 1.8?

Want to know this ASAP, as we are choosing a notification package for exactly same features provided by this app. Please let me know

Yes, this package supports 1.8
https://django-mail-queue.readthedocs.org/en/latest/installation.html#requirements

Thank you! That was too quick :-)


Issue with rhoReactingCentralFoam

Followed the manual for setting up the updated rhoReactingCentralFoam solver. We are having issues when starting rhoReactingCentralFoam on the demo case.
What we did:

Copied "applications" and "src" folders to our user folder.
Did ./Allwmake on rhoReactingCentralFoam
Did wmake on src/basic, reactionThermo, then specie.
Ran blockMesh, setFields on demo. rhoReactingCentralFoam gives following error.

We found the issue in the "Make/files". There's a case type. reactions should be Reactions

We found the issue in the "Make/files". There's a case type. reactions should be Reactions

Hi Jomarchris
First of all, thank you for your cooperation.
I am not sure on which version you have tried to compile these codes, but on OpenFOAM v7, the directories were exactly as I wrote in the Make/files. If you are interested, you can check that out at the following link:

Kind regards,
Keivan

Hi @jomarchris  I am very fortunate to see your comment. I have the same issue. May I ask how you solved it?
Best wishes,
Wang


[Rust] Clippy lints

There's some clippy lints that are returning errors, where they haven't before. Not sure what's happening, as we've pinned versions.

Thank you @nevi-me  -- looks like there are still a few more lints being hit

Looks to me like Rust 1.50 was released today: https://forge.rust-lang.org/

So clippy got pickier

darn it, I was hoping it isn't a new stable release, because we have a lot of unnecessary Result<()>. This will take me longer

@nevi-me  -- I am working on a fix for the other lint errors based on this PR. My plan is to just disable the lints that are still failing -- because as you say changing Result<..> signatures is a much larger change

Thanks @alamb, it's inefficient use of time to try pursue all the lints now. Thanks, I'm closing this.

https://github.com/apache/arrow/pull/9476


Cannot get 4.0.15 working with VS 2017

The default installation location for 4.0.15 seems to be C:\Program Files (x86)\MSBuildExtensionPack\4.0
If I use that and try to get the tasks from my project file like this:

I get this error, see the way Visual Studio wants to use it's private copy of MsBuildExtensionPack

Microsoft Visual Studio
The imported project "C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\MsBuildExtensionPack\4.0\MSBuild.ExtensionPack.tasks" was not found. Also, tried to find "MsBuildExtensionPack\4.0\MSBuild.ExtensionPack.tasks" in the fallback search path(s) for $(MSBuildExtensionsPath) - "C:\Program Files (x86)\MSBuild" . These search paths are defined in "C:\Users\aphilpotts\AppData\Local\Microsoft\VisualStudio\15.0_de9a5e6f\devenv.exe.config". Confirm that the path in the  declaration is correct, and that the file exists on disk in one of the search paths.  C:\dev\ink\working\logical ink\server\database2\LogicalInk.sqlproj
OK
To try to remedy this I COPIED the install to C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\MsBuildExtensionPack and it got further, but then fails when running the Task, like this:
C:\dev\ink\working\logical ink\server\database2\LogicalInk.sqlproj(216,7): Error:  MSB4062: The "MSBuild.ExtensionPack.FileSystem.File" task could not be loaded from the assembly C:\Program Files (x86)\MSBuildExtensionPack\4.0\MSBuild.ExtensionPack.dll. Could not load file or assembly 'file:///C:\Program Files (x86)\MSBuildExtensionPack\4.0\MSBuild.ExtensionPack.dll' or one of its dependencies. The system cannot find the file specified. Confirm that the  declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask.
any ideas?
This is how I run the task

You have to install to the default install dir (MsBuildExtensionPack, not MSBuild/ExtensionPack). You can adjust the import statements in your build files to look like this to compensate:

This will allow them to work with either version installed.


Hive Cluster Relocation is stuck on kube-root-ca.crt and openshift-service-ca.crt already exists

Hive Cluster Relocation is stuck on kube-root-ca.crt and openshift-service-ca.crt already exists . These configmaps are created by default when namespace created and hive clusterdeployment fails with error
    - lastProbeTime: '2022-12-20T11:03:48Z'
      lastTransitionTime: '2022-12-20T10:08:51Z'
      message: >-
        failed to copy *v1.ConfigMapList: could not copy *v1.ConfigMap resource
        "kube-root-ca.crt": failed to sync existing resource: configmaps
        "kube-root-ca.crt" already exists
      reason: MoveFailed
      status: 'True'
      type: RelocationFailed

Is there a way to ignore these configmaps ? We are stuck in a migration , looking forward to any help !

Howdy! I'm looking into this, but lots of people have lots of vacation until January, so it may be a little bit.
In the meantime, can you please tell me how you are installing hive, and at what version?

Couple of things I've discovered/surmised:

This is a timing problem. The code path that attempts to copy configmaps has logic like: Create() => if error == AlreadyExists, Delete() and Create() again => if error again, bail. The kube-controller-manager reconciles these configmaps; so the error only happens if it manages to do so between that Delete()/Create(). I was able to do a relocate in my test bed, so it doesn't always happen. My logs show that it did indeed replace the objects, but the configmaps contain the proper (destination) data so the k-c-m did revert them after.
Why hasn't this come up before? Well, the relocate feature was merged in June 2020 when supported OpenShift versions were 4.2 to 4.4 (source). The k-c-m code that creates these configmaps was introduced in k8s 1.21 (citation needed -- this is hearsay from a slack thread @wking found for me). The first OpenShift version to use k8s 1.21 was 4.8 (source) which GAed a in July 2021 (same source as above). I'm not going to assert that y'all are the first to try using relocation in the past 1.5y and/or 4-5 releases, but it's not impossible :) Also possible others just didn't hit the timing problem.

I'm afraid I don't have a workaround for you. I'm looking into code changes that would help. In the meantime, the below would really help me out:

How are you installing hive? From OperatorHub or $other? (asked above, copied here for completeness)
What version of hive is running? Output of oc logs -n hive deploy/hive-operator | head -1 (s/-n hive/appropriate namespace/ if needed). On both sides of the relocate, please. (ditto)
OpenShift/k8s versions on both sides. Output of oc version.
Logs from the clusterrelocate controller on both sides. Output of oc logs -n hive deploy/hive-controllers | grep controller=clusterRelocate (s/// namespace again  if necessary).
Does the ClusterDeployment exist on the destination? Long shot; but if it does, it may just mean that we have a code path that does the right things, but doesn't clean up status properly.

We track bugs in Jira, so I've created https://issues.redhat.com/browse/HIVE-2080 for this one. Please feel free to follow along there.

We had hive installed via rhacm multiclusterhub and rhacm (version 2.4) was installed via operatorhub  on source cluster and rhacm version 2.6 on target cluster via operatorhub
The source cluster is turned off but there is no hive operator in hive namespace ( there are hive controllers and hive admission deployments are present)
Openshift Version of both cluster is 4.9
We turned off the source cluster so don't have logs anymore
ClusterDeployment did not move and the relocation stopped at the configmap error

The source cluster where we are moving from we successfully moved 4 openshift clusters (ClusterDeployments/configmaps/secrets) all to new hive but one got stuck in it , I tried it many times but last one stayed stuck because it kept saying the configmaps already exists. I tried to update the configmap or tried deleting the namespace to beat the race condition but nothing helped.
Yes , I know the configmap creation feature in namespace came in 1.20 But I strongly feel it should be handled. As We had to turn off the old hub we just imported the cluster in rhacm for now and turned it off  without moving clusterdeployment and machinepools.
Great that you created a bug and please can you give us updates  on it so we can follow up here.

Tracking via https://issues.redhat.com/browse/HIVE-2080


[ubuntu 24.04] install-powershell.sh script fails with 404

Description
Hi there,
Currently the script images/ubuntu/scripts/build/install-powershell.sh fails because of http return 404 - file not found.
It tries to download https://mirrors.edge.kernel.org/ubuntu/pool/main/i/icu/libicu72_72.1-3ubuntu2_amd64.deb
Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[X] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] macOS 15
[ ] macOS 15 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
_
Is it regression?
no
Expected behavior
Working url
Actual behavior
failing with http 404
Repro steps
Start the build workflow

Hi @habandy , Thank you for bringing this issue to our attention. We are looking into this issue and will update you on this issue after investigating

Updated the issue. This only affects ubuntu 24.04 as it seems, because of the if condition in the script. older versions like 20.04 and 22.04 should not be affected.

Hi @habandy This PR will fix your issue and it has been merged. Could you please retry by forking the latest code from the main branch. We are closing this issue, thanks.


feat: add builder

use versioning of containerbase/buildpack
no short versions
build only latest version

Should we switch the user to ubuntu ?


uCLibc may not have complete fenv.h support even if header present

This is related to bug #279971 in WebKit.
I am not 100% sure how things work when building the uClibc C library, but my understanding is that the only a few architectures provide all the functions from fenv.h, and even on architectures where support is partial (or missing) the configuration program will allow setting UCLIBC_HAS_FENV=y regardless.
So it seems that when building against uClibc more smarts than only checking __has_include(<fenv.h>) may be needed.

Hello @aperezdc and thank you for the report. Is webkit using uClibc or https://uclibc-ng.org/ ? Which version(s)?

A PR to add a better check would be very welcome, of course

Hello @aperezdc and thank you for the report. Is webkit using uClibc or https://uclibc-ng.org/ ? Which version(s)?

The uClibc was forked in 2014, see https://uclibc-ng.org/. So I don't think that this one is very relevant. The compile issue was seen by using buildroot which ships with uClibc-ng v1.0.50. See https://github.com/buildroot/buildroot/blob/2024.08.1/package/uclibc/uclibc.mk.


"Kind" Property for SecurityInsightsAlertRuleResource?

Hi. I am using the GetAllAlertRules method and the SecurityInsightsAlertRuleResource objects that are being returned do not seem to have a property of "Kind" associated with them (e.g., Fusion, Scheduled, etc.) is there plans to integrate this or is there a way to do this already?

Also no property for "IsEnabled"

Resolved


countResponse and booleanResponse not splitted per ResultSet

Now that we are implementing beacons in different projects, we are facing the problem that count and boolean responses are always a result of the aggregation of the resultSets, but they never show the results splitted by resultSet. This makes impossible to know, for a client requesting a beacon that only allows boolean/count granularity, which are the datasets that are responding positively or negatively. As we can see in the resultSet response from the standards:
        "ResultsetInstance": {
            "additionalProperties": true,
            "properties": {
                "exists": {
                    "type": "boolean"
                },
                "id": {
                    "description": "id of the resultset",
                    "example": "datasetA",
                    "type": "string"
                },
                "info": {
                    "description": "Additional details that could be of interest about the Resultset. Provided to clearly enclose any attribute that is not part of the Beacon specification.",
                    "type": "object"
                },
                "results": {
                    "items": {
                        "type": "object"
                    },
                    "minItems": 0,
                    "type": "array"
                },
                "resultsCount": {
                    "description": "Number of results in this Resultset.",
                    "type": "integer"
                },
                "resultsHandovers": {
                    "$ref": "../../common/beaconCommonComponents.json#/definitions/ListOfHandovers",
                    "description": "List of handovers that apply to this resultset, not to the whole Beacon or to a result in particular."
                },
                "setType": {
                    "default": "dataset",
                    "description": "Entry type of resultSet. It SHOULD MATCH an entry type declared as collection in the Beacon configuration.",
                    "type": "string"
                }
            },

We believe this resultSet response should be naturally returned by any type of granularity, with results being exclusively for the record granularity and resultsCount for the count and record granularity.
I will work on a solution in the next days to have this implemented unless the opinion on this is mostly the contrary and I wanted to know what do you think.
Thanks,
Oriol

This has been in discussion for a while & also is in line with the need to add an optional beacon identifier to each ResultsetInstance - the following from the presentation at ELIXIR AHM 2023 (this was not "formal" and the resultsCount etc. just demonstrate what needs to be implemented):
resultSets:
  - id
    beaconId
    infoUrl
    setType
    exists
    resultsCount
    returnedGranularity
    ...
    results []
  - id
    beaconId
    infoUrl
    exists
    ...

So, yes; and I had also called before for (especially in the network context but in general, too) for ant response being a ResultSetRespones (w/ an optional response part) to allow "single Beacon, different dataset permissions" responses.

Back to this statement:

We believe this resultSet response should be naturally returned by any type of granularity, with results being exclusively for the record granularity and resultsCount for the count and record granularity.

Yes, this has come up before - granularity does not explicitly say one should not report per dataset (resultSet); however, currently AFAIK the results element is required per resultSet. I had somewhere documented that sjust the removal of this requirement would enable boolean or count resulsetResponses.

HI @mbaudis,
you are right that the removal of the results plus the removal of resultsCount required condition would be enough (if we think in boolean resultSets, for example).
My solution is aligned with what you are saying @mbaudis. In addition, I would have resultSets splitted by count and boolean, take a look at my work here https://github.com/ga4gh-beacon/beacon-v2/commit/a6295684c9c8597aea885ed8e9513261fb9ecf8e. These new two responses would be like a resultSets but in the case of countSets there would be no results element, and in the case of booleanSets, no results and no resultsCount element. This would allow to still have the three types of responses differentiated.
Let me know what you think.
Thanks,
Oriol

@costero-e Well, this looks per se correct - but

it doesn't solve mixed granularity responses since always the response type specific parameters are required elements
I don't know how those responses can be requested, in contrast to the non-resultset count/boolean responses.
a booleanResultsetResponse still is different from a basic one regarding it security aspects (not necessarily relevant IMO but still...)

So since we have to take it as a given that we need an option for mixed responses, at least for larger network queries, we either need then another response type for those which is more permissive. And that can realistically be used for any response...
I understand that it is good for explanations to have tightly defined schemas; "this beacon only provides up to beaconCountResultsetResponse and does not implement record level delivery" etc. IMO the only argument.

Hi @mbaudis, ok, I have just added the YAML solution, here 3f9edbbb4b87b0c3d714d120cc9576a1060210aa. A bit more correct now!
About your answer, I see what you mean, but I think a Beacon Network should decide which response it is returning (record, count or boolean). A Beacon Network can't respond with a mixed granularity, can it? Maybe this has its own limitation, for example, a response will be always limited by the max granularity accepted by each beacon and dataset, but IMO then a user should refine the query per beacon/dataset (knowing what is the max granularity the user has per dataset/beacon) if the user wants more complete responses.

A Beacon Network can't respond with a mixed granularity, can it? Maybe this has its own limitation, for example, a response will be always limited by the max granularity accepted by each beacon and dataset, but IMO then a user should refine the query per beacon/dataset (knowing what is the max granularity the user has per dataset/beacon) if the user wants more complete responses.

Network is not the same as Aggregator
In reality you want to create aggregators with many different beacons, w/ different status of maintenance and content etc. And there are many explorative query scenarios where one just wants to fire off at least count queries etc. Yes, one could iterate through query-response-requery ... scenarios but that's e.g. not really nice over a UI and certainly against the "discovery" aspect. So if every beacon not responding with the requested resultSet type just gives an error this is not helpful, even for an overview.

Hi @mbaudis, ok, I have just added the YAML solution, here 3f9edbb.

+1 for this or a similar solution, I've been writing network code for our own count beacons, but I don't have a way to serve a standard count response that shows a breakdown of counts for each beacon. So this is still an issue even without mixing granularities.
We are also working on beacons with multiple datasets where a count-per-dataset response would probably be useful.


Interfaces without a host

Skydive 0.11.0
Sometimes in the UI, interfaces appear without being part of a host.

It is a known issue of the 0.11, It should be fixed in master, so will be available in the next release (soon)

Great! :)

I close it, feel free to re-open it if the issue is still present in the master branch


Filed "TotalSum" should be replaced

As you can see big gap under the "Total sum" field. This field should be at the bottom of the region

Fixed here - 904efd499f26fcab54e1ea367aacbf4dff940be1
No looks like this:

Closed with previous comment