Reflecting current version of the deployed Tenant Control Plane version rather then the specified one in the spec

With the current state, the additional printer columns provided with kubectl get tenantcontrolplanes is reporting the specified version in the spec using the following JSON path.
https://github.com/clastix/kamaji/blob/e8bf1e4066705143ec50fc73ad1683567fb209e6/api/v1alpha1/tenantcontrolplane_types.go#L270
During an upgrade upon version bump-up, it represents the desired state rather than the current one that isn't yet the reconciled one.
Current output
$: kubectl get tcp -w
NAME   VERSION   STATUS   CONTROL-PLANE-ENDPOINT   KUBECONFIG              AGE
test   v1.23.3   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   5m53s
test   v1.23.4   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   5m56s
test   v1.23.4   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   5m58s
test   v1.23.4   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   5m59s
test   v1.23.4   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   6m
test   v1.23.4   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   6m1s
test   v1.23.4   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   6m1s
test   v1.23.4   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   6m2s
test   v1.23.4   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   6m3s
test   v1.23.4   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   6m4s
test   v1.23.4   Upgrading   <IP_ADDRESS>:6443      test-admin-kubeconfig   6m4s
test   v1.23.4   Upgrading   <IP_ADDRESS>:6443      test-admin-kubeconfig   6m4s
test   v1.23.4   Upgrading   <IP_ADDRESS>:6443      test-admin-kubeconfig   6m16s
test   v1.23.4   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   6m17s
test   v1.23.4   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   6m19s
test   v1.23.4   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   6m19s
test   v1.23.4   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   6m20s
test   v1.23.4   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   6m20s
test   v1.23.4   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   6m21s

Proposed solution
A change in the additional printer columns as follows can fix the styling issue.
// +kubebuilder:printcolumn:name="Version",type="string",JSONPath=".status.kubernetesResources.version.version",description="Kubernetes version.version"

Desired result
$: kubectl get tcp -w
NAME   VERSION   STATUS   CONTROL-PLANE-ENDPOINT   KUBECONFIG              AGE
test   v1.23.1   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   3m47s
test   v1.23.1   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   3m58s
test   v1.23.1   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   3m58s
test   v1.23.1   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   3m58s
test   v1.23.1   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   3m59s
test   v1.23.1   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   4m
test   v1.23.1   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   4m
test   v1.23.1   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   4m1s
test   v1.23.1   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   4m2s
test   v1.23.1   Ready    <IP_ADDRESS>:6443      test-admin-kubeconfig   4m3s
test   v1.23.1   Upgrading   <IP_ADDRESS>:6443      test-admin-kubeconfig   4m3s
test   v1.23.1   Upgrading   <IP_ADDRESS>:6443      test-admin-kubeconfig   4m4s
test   v1.23.1   Upgrading   <IP_ADDRESS>:6443      test-admin-kubeconfig   4m15s
test   v1.23.2   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   4m16s
test   v1.23.2   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   4m16s
test   v1.23.2   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   4m16s
test   v1.23.2   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   4m17s
test   v1.23.2   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   4m17s
test   v1.23.2   Ready       <IP_ADDRESS>:6443      test-admin-kubeconfig   4m18s

@bsctl @mendrugory

@prometherion
I am not sure if version during upgrading phase should be the new one (upgrading to or upgrading from). It is a matter of taste.

Ok, let's go instead for changing the order of the resource handlers, putting the upgrade phase as first item in the slice.


Diego/proto 2657 keep reward emissions from mint

Purpose of Changes and their Description
Fix over rewards emissions to cache current block rewards.
Link(s) to Ticket(s) or Issue(s) resolved by this PR
Are these changes tested and documented?

[ ] If tested, please describe how. If not, why tests are not needed.
[ ] If documented, please describe where. If not, describe why docs are not needed.
[ ] Added to Unreleased section of CHANGELOG.md?

Still Left Todo
Fill this out if this is a Draft PR so others can help.

Merging to the base proto-2657 without reviews


v3 documentation resolves to 404s

What is the type of issue?
No response
What is the issue?
The main page links to:
OpenAPI/Swagger
Typescript
Tanstack query
etc..
All of them resolve to 404 at the moment :(
The same links without /docs work all right
Where did you find it?
No response

@luixo This should be resolved again, be aware that v3 docs are not finished yet so still some changes will come in the coming weeks.


Use "main" as the protobuf head branch name

As title.
https://fusion2.corp.google.com/invocations/0ce41ab0-bc28-42ec-bd8f-d4a65628d452/targets

The error link if wrong (point to an unrelated build?)
I think you meant this link:
https://source.cloud.google.com/results/invocations/630da787-1f66-479d-9a6d-d0bca653601b/log (which seem to be the first failure of this kind for the job).
+ git fetch origin
+ git checkout origin/master
error: pathspec 'origin/master' did

context: Looks like protobuf has recently renamed their master branch to main

Merging. There are other build problems toward ProtoBuf at head with Python. I will try to dig into that.


react 16 React.Fragment Tabs bug

React.Fragment on Tabs don't work

if i return react 16 React.Fragment array of Tab to Tabs they shown but don't work :(

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Current Behavior

Steps to Reproduce (for bugs)

 <Tabs
                        value={value}
                        onChange={this.handleChange}
                        scrollable
                        scrollButtons="on"
                        indicatorColor="primary"
                        textColor="primary"
                    >
                        <MyTabs />
                    </Tabs>

MyTabs.js
export default class  extends Component{
    render(){
            return(
                <React.Fragment>
                    <Tab key="0" label="Live" icon={<LiveTv />} />
                    <Tab key="1" label="Info" icon={<Assessment />} />
                </React.Fragment>
            )
    }
}

Context

Your Environment

Tech
Version

Material-UI

React

browser

etc

Please provide a full reproduction test case. This would help a lot 👷 .
A live example would be perfect. This codesandbox.io template may be a good starting point. Thank you!

You can get around this by just returning an array with the fragment.
instead of
return (
    <React.Fragment>
        <Tab key="0" label="Live" icon={<LiveTv />} />
        <Tab key="1" label="Info" icon={<Assessment />} />
    </React.Fragment>
)

you can do
return [
    <Tab key="0" label="Live" icon={<LiveTv />} />
    <Tab key="1" label="Info" icon={<Assessment />} />
]

hope this helps!


Spring Security ACL

Hi there! I followed the instructions on https://www.baeldung.com/spring-security-acl, but the MethodSecurityExpressionHandler configuration didn't work on Spring Boot 2.2.6. What worked was creating a bean of AclPermissionEvaluator. So, instead of:
@Configuration
@EnableGlobalMethodSecurity(prePostEnabled = true, securedEnabled = true)
public class AclMethodSecurityConfiguration
  extends GlobalMethodSecurityConfiguration {

    @Autowired
    MethodSecurityExpressionHandler
      defaultMethodSecurityExpressionHandler;

    @Override
    protected MethodSecurityExpressionHandler createExpressionHandler() {
        return defaultMethodSecurityExpressionHandler;
    }
}

@Bean
public MethodSecurityExpressionHandler
  defaultMethodSecurityExpressionHandler() {
    DefaultMethodSecurityExpressionHandler expressionHandler
      = new DefaultMethodSecurityExpressionHandler();
    AclPermissionEvaluator permissionEvaluator
      = new AclPermissionEvaluator(aclService());
    expressionHandler.setPermissionEvaluator(permissionEvaluator);
    return expressionHandler;
}

I did:
@Configuration
@EnableGlobalMethodSecurity(prePostEnabled = true, securedEnabled = true)
public class MethodSecurityConfiguration extends GlobalMethodSecurityConfiguration {
}

@Configuration
public class AclConfiguration {

    @Bean
    public AclPermissionEvaluator aclPermissionEvaluator() {
        return new AclPermissionEvaluator(aclService());
    }

    /* Other ACL configurations */
}

Hi @rafaelrenanpacheco,
Thanks for the feedback! I'm sorry to hear you face issues with this.
We'll take a look into this. But can you please clarify when does the problem occurs? Is it during startup? Is any exception thrown? Or does some functionality doesn't work as expected?
I'd appreciate your input here.

Hello!
With the first configuration the ACL was just ignored, with the second configuration it worked as expected, and ACL policy was applied when calling secured methods.
I'm no longer using Spring ACL, so if this issue doesn't happen on your application, I think this issue may be closed.
Best regards,
Rafael Pacheco.

Thanks for the reply @rafaelrenanpacheco!
Well, I've just checked our example project and the ACL is working fine.
Just to clarify, have you observed the issue mentioned above in our sample application? Or was it one that you've created yourself?

It was on an application I created myself. I followed the ACL tutorial and at the end, when calling a HTTP endpoint that calls a secured method, the ACL was completely ignored. After removing the MethodSecurityExpressionHandler bean and creating a bean for AclPermissionEvaluator it started to work just fine. In this project I'm also using Spring Cloud Security, so it may have caused some configuration conflict. Also, I just saw that the spring project used for this tutorial is using the current version of Spring Boot, and, if it is working as you said, I think we can close this issue.
Best regards,
Rafael Pacheco.

Thanks for clarifying @rafaelrenanpacheco!


h2c: failing to compile C code

We are seeing this failure with i2c:
Building h2c-1.0.0...
Preprocessing library h2c-1.0.0...
In file included from dist/build/System/IO/I2C_hsc_make.c:1:0:
I2C.hsc: In function ‘main’:
I2C.hsc:117:56: error: ‘I2C_M_TEN’ undeclared (first use in this function)
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:96:10: note: in definition of macro ‘hsc_enum’
     if ((x) < 0)                                        \
          ^
I2C.hsc:117:56: note: each undeclared identifier is reported only once for each function it appears in
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:96:10: note: in definition of macro ‘hsc_enum’
     if ((x) < 0)                                        \
          ^
I2C.hsc:118:55: error: ‘I2C_M_RD’ undeclared (first use in this function)
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:96:10: note: in definition of macro ‘hsc_enum’
     if ((x) < 0)                                        \
          ^
I2C.hsc:119:60: error: ‘I2C_M_NOSTART’ undeclared (first use in this function)
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:96:10: note: in definition of macro ‘hsc_enum’
     if ((x) < 0)                                        \
          ^
I2C.hsc:120:65: error: ‘I2C_M_REV_DIR_ADDR’ undeclared (first use in this function)
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:96:10: note: in definition of macro ‘hsc_enum’
     if ((x) < 0)                                        \
          ^
I2C.hsc:121:63: error: ‘I2C_M_IGNORE_NAK’ undeclared (first use in this function)
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:96:10: note: in definition of macro ‘hsc_enum’
     if ((x) < 0)                                        \
          ^
I2C.hsc:122:63: error: ‘I2C_M_NO_RD_ACK’ undeclared (first use in this function)
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:96:10: note: in definition of macro ‘hsc_enum’
     if ((x) < 0)                                        \
          ^
I2C.hsc:120:75: error: ‘I2C_FUNC_10BIT_ADDR’ undeclared (first use in this function)
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:96:10: note: in definition of macro ‘hsc_enum’
     if ((x) < 0)                                        \
          ^
I2C.hsc:121:74: error: ‘I2C_FUNC_SMBUS_PEC’ undeclared (first use in this function)
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:96:10: note: in definition of macro ‘hsc_enum’
     if ((x) < 0)                                        \
          ^
I2C.hsc:122:68: error: ‘I2C_FUNC_I2C’ undeclared (first use in this function)
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:96:10: note: in definition of macro ‘hsc_enum’
     if ((x) < 0)                                        \
          ^
I2C.hsc:162:15: error: invalid application of ‘sizeof’ to incomplete type ‘i2c_msg_t {aka struct i2c_msg}’
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:86:39: note: in definition of macro ‘hsc_size’
     hsc_printf("(%ld)", (long) sizeof(t));
                                       ^
I2C.hsc:170:15: error: invalid application of ‘sizeof’ to incomplete type ‘i2c_msg_t {aka struct i2c_msg}’
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:86:39: note: in definition of macro ‘hsc_size’
     hsc_printf("(%ld)", (long) sizeof(t));
                                       ^
I2C.hsc:178:5: error: invalid use of incomplete typedef ‘i2c_msg_t’
I2C.hsc:179:5: error: invalid use of incomplete typedef ‘i2c_msg_t’
I2C.hsc:180:5: error: invalid use of incomplete typedef ‘i2c_msg_t’
I2C.hsc:181:5: error: invalid use of incomplete typedef ‘i2c_msg_t’
In file included from dist/build/System/IO/I2C_hsc_make.c:1:0:
I2C.hsc:185:15: error: invalid application of ‘sizeof’ to incomplete type ‘i2c_msg_t {aka struct i2c_msg}’
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:86:39: note: in definition of macro ‘hsc_size’
     hsc_printf("(%ld)", (long) sizeof(t));
                                       ^
I2C.hsc:187:5: error: invalid use of incomplete typedef ‘i2c_msg_t’
I2C.hsc:188:5: error: invalid use of incomplete typedef ‘i2c_msg_t’
I2C.hsc:189:5: error: invalid use of incomplete typedef ‘i2c_msg_t’
I2C.hsc:190:5: error: invalid use of incomplete typedef ‘i2c_msg_t’
In file included from dist/build/System/IO/I2C_hsc_make.c:1:0:
I2C.hsc:218:15: error: invalid application of ‘sizeof’ to incomplete type ‘i2c_msg_t {aka struct i2c_msg}’
/opt/ghc/8.0.2/lib/ghc-8.0.2/template-hsc.h:86:39: note: in definition of macro ‘hsc_size’
     hsc_printf("(%ld)", (long) sizeof(t));
                                       ^
I2C.hsc:219:5: error: invalid use of incomplete typedef ‘i2c_msg_t’
I2C.hsc:220:5: error: invalid use of incomplete typedef ‘i2c_msg_t’
compiling dist/build/System/IO/I2C_hsc_make.c failed (exit code 1)
command was: /usr/bin/gcc -c dist/build/System/IO/I2C_hsc_make.c -o dist/build/System/IO/I2C_hsc_make.o -fno-stack-protector -fno-stack-protector -D__GLASGOW_HASKELL__=800 -Dlinux_BUILD_OS=1 -Dx86_64_BUILD_ARCH=1 -Dlinux_HOST_OS=1 -Dx86_64_HOST_ARCH=1 -Icfiles -Idist/build/autogen -include dist/build/autogen/cabal_macros.h -I/opt/ghc/8.0.2/lib/ghc-8.0.2/bytestring-<IP_ADDRESS>/include -I/opt/ghc/8.0.2/lib/ghc-8.0.2/base-<IP_ADDRESS>/include -I/opt/ghc/8.0.2/lib/ghc-8.0.2/integer-gmp-<IP_ADDRESS>/include -I/opt/ghc/8.0.2/lib/ghc-8.0.2/include -I/opt/ghc/8.0.2/lib/ghc-8.0.2/include/

@eamsden I have disabled h2c (and bno055-haskell) as it is breaking the build -- it looks like we are missing some development headers from our build environment, presumably because we are missing some development packages. Can you please advise.

h2c needs the appropriate i2c-dev.h header file to compile. That's usually available on a Linux system but has architecture-specific constants, so including it with the package doesn't seem to make sense.
Is it possible to have stack build h2c with the i2c-dev.h header present?

Sorry @eamsden I wasn't very clear. You would usually add the headers by installing a development package in the Linux build environment but I am not familiar with this library. Do you know which Debian development package we should be installing, or the package installed on your own development system?

It should be i2c-tools (there's no -dev package which is weird for a Debian packaging setup.)

I'm on ubuntu, so I'm not sure about Debian.
For me, apt-file find shows the file in both

linux-libc-dev
libi2c-dev

Both of these packages are installed on my system and both install a header file of that name to that place, so I'm not sure which one I have, but my wild guess would be linux-libc-dev would be correct.

Note that it's not actually linking against any library (other than libc). It's calling ioctl but with some constants and structs defined in this header file. Thus my guess that the linux-libc-dev package is the correct one.

I took a look at the debian container that I use to build this for ARM. It is indeed linux-libc-dev @cdornan

Yeah I would have guessed the smaller libi2c-dev too.
If you want to open a PR to re-enable h2c and add this dev package to debian-bootstrap.sh...

@juhp done: #2651


ASE Standard Logic App creation at Portal unable to select existing App Service Plan.

Describe the Bug with repro steps

Try create ASE Standard Logic App in WestEurope region.

Review existing Standard LA in same region, note down their ASP name.

When creating the Standard Logic App at portal view. Select the same ASE, try find those ASP name in dropdown which were previously noted.

Observe that the these ASPs are not there in the dropdown. Only option is to create a new ASP.

As a workaround, we had to place the ASP name manually and deploy via ARM template.

What type of Logic App Is this happening in?
Standard (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
NA

Screenshots or Videos

Browser
Chrome
Additional context
No response
AB#25191028

This should now be fixed with a hotfix that went out today


Use after free in FreeDesktopScreenSaver::inhibit()

Godot version
4.0.dev.custom_build. bb409efa1
System information
Ubuntu 21.04
Issue description
Address sanitizer log
==5116==ERROR: AddressSanitizer: heap-use-after-free on address 0x60300018adf0 at pc 0x7fc378521ab7 bp 0x7fff2dcb6590 sp 0x7fff2dcb5d38
READ of size 9 at 0x60300018adf0 thread T0
    #0 0x7fc378521ab6 in __interceptor_strlen ../../../../src/libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc:389
    #1 0x7fc378331d08 in _dbus_string_init_const (/lib/x86_64-linux-gnu/libdbus-1.so.3+0x31d08)
    #2 0x7fc37831d9ab  (/lib/x86_64-linux-gnu/libdbus-1.so.3+0x1d9ab)
    #3 0x7fc3783200fd in dbus_message_iter_append_basic (/lib/x86_64-linux-gnu/libdbus-1.so.3+0x200fd)
    #4 0x7fc378320d95 in dbus_message_append_args_valist (/lib/x86_64-linux-gnu/libdbus-1.so.3+0x20d95)
    #5 0x7fc37832103c in dbus_message_append_args (/lib/x86_64-linux-gnu/libdbus-1.so.3+0x2103c)
    #6 0x20b459c in FreeDesktopScreenSaver::inhibit() platform/linuxbsd/freedesktop_screensaver.cpp:65
    #7 0x20cc9f0 in DisplayServerX11::screen_set_keep_on(bool) platform/linuxbsd/display_server_x11.cpp:835
    #8 0x2142d4c in DisplayServerX11::DisplayServerX11(String const&, DisplayServer::WindowMode, unsigned int, Vector2i const&, Error&) platform/linuxbsd/display_server_x11.cpp:4298
    #9 0x212da23 in DisplayServerX11::create_func(String const&, DisplayServer::WindowMode, unsigned int, Vector2i const&, Error&) platform/linuxbsd/display_server_x11.cpp:3658
    #10 0x12a8a89d in DisplayServer::create(int, String const&, DisplayServer::WindowMode, unsigned int, Vector2i const&, Error&) servers/display_server.cpp:592
    #11 0x21d37a6 in Main::setup2(unsigned long) main/main.cpp:1553
    #12 0x21ce35a in Main::setup(char const*, int, char**, bool) main/main.cpp:1405
    #13 0x2088046 in main platform/linuxbsd/godot_linuxbsd.cpp:51
    #14 0x7fc3775c9564 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x28564)
    #15 0x2087c8d in _start (/usr/bin/godot4s+0x2087c8d)

0x60300018adf0 is located 16 bytes inside of 32-byte region [0x60300018ade0,0x60300018ae00)
freed by thread T0 here:
    #0 0x7fc3785988f7 in __interceptor_free ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:127
    #1 0x15754d7e in Memory::free_static(void*, bool) core/os/memory.cpp:168
    #2 0x208dd6c in CowData<char>::_unref(void*) core/templates/cowdata.h:217
    #3 0x208c59a in CowData<char>::~CowData() core/templates/cowdata.h:380
    #4 0x208c501 in CharString::~CharString() core/string/ustring.h:128
    #5 0x20b44dc in FreeDesktopScreenSaver::inhibit() platform/linuxbsd/freedesktop_screensaver.cpp:58
    #6 0x20cc9f0 in DisplayServerX11::screen_set_keep_on(bool) platform/linuxbsd/display_server_x11.cpp:835
    #7 0x2142d4c in DisplayServerX11::DisplayServerX11(String const&, DisplayServer::WindowMode, unsigned int, Vector2i const&, Error&) platform/linuxbsd/display_server_x11.cpp:4298
    #8 0x212da23 in DisplayServerX11::create_func(String const&, DisplayServer::WindowMode, unsigned int, Vector2i const&, Error&) platform/linuxbsd/display_server_x11.cpp:3658
    #9 0x12a8a89d in DisplayServer::create(int, String const&, DisplayServer::WindowMode, unsigned int, Vector2i const&, Error&) servers/display_server.cpp:592
    #10 0x21d37a6 in Main::setup2(unsigned long) main/main.cpp:1553
    #11 0x21ce35a in Main::setup(char const*, int, char**, bool) main/main.cpp:1405
    #12 0x2088046 in main platform/linuxbsd/godot_linuxbsd.cpp:51
    #13 0x7fc3775c9564 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x28564)

previously allocated by thread T0 here:
    #0 0x7fc378598c47 in __interceptor_malloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:145
    #1 0x15753d3f in Memory::alloc_static(unsigned long, bool) core/os/memory.cpp:75
    #2 0x401fe8f in CowData<char>::resize(int) core/templates/cowdata.h:286
    #3 0x15a21867 in CharString::resize(int) core/string/ustring.h:136
    #4 0x16b86d6f in String::utf8() const core/string/ustring.cpp:1955
    #5 0x20b445b in FreeDesktopScreenSaver::inhibit() platform/linuxbsd/freedesktop_screensaver.cpp:58
    #6 0x20cc9f0 in DisplayServerX11::screen_set_keep_on(bool) platform/linuxbsd/display_server_x11.cpp:835
    #7 0x2142d4c in DisplayServerX11::DisplayServerX11(String const&, DisplayServer::WindowMode, unsigned int, Vector2i const&, Error&) platform/linuxbsd/display_server_x11.cpp:4298
    #8 0x212da23 in DisplayServerX11::create_func(String const&, DisplayServer::WindowMode, unsigned int, Vector2i const&, Error&) platform/linuxbsd/display_server_x11.cpp:3658
    #9 0x12a8a89d in DisplayServer::create(int, String const&, DisplayServer::WindowMode, unsigned int, Vector2i const&, Error&) servers/display_server.cpp:592
    #10 0x21d37a6 in Main::setup2(unsigned long) main/main.cpp:1553
    #11 0x21ce35a in Main::setup(char const*, int, char**, bool) main/main.cpp:1405
    #12 0x2088046 in main platform/linuxbsd/godot_linuxbsd.cpp:51
    #13 0x7fc3775c9564 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x28564)

SUMMARY: AddressSanitizer: heap-use-after-free ../../../../src/libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc:389 in __interceptor_strlen
Shadow bytes around the buggy address:

Problematic line
https://github.com/godotengine/godot/blob/a738ddb9a40ed829d64b380f95ecb6cbf30004c2/platform/linuxbsd/freedesktop_screensaver.cpp#L58
This should be fixed in this way
https://github.com/godotengine/godot/pull/45657/files#diff-3c95ffea6d81c05adea361b3206db7c6f12ff45f6876e25395eab944afcef2f9R1596-R1597
Steps to reproduce
Above
Minimal reproduction project
No response

CC @shroudedcode

I have to admit that I barely understand what's happening here, so I'd be fine with someone else fixing this. 😅

@shroudedcode .utf8() returns a CharString which holds the UTF-8 data, and .get_data() returns a pointer to that data.
In app_name_string.utf8().get_data(), the CharString is an temporary object which will be destroyed after the statement. So the app_name pointer will be pointing to something that's freed. Making the CharString a variable can make sure app_name points to valid data.

This should be fixed in this way
https://github.com/godotengine/godot/pull/45657/files#diff-3c95ffea6d81c05adea361b3206db7c6f12ff45f6876e25395eab944afcef2f9R1596-R1597

Sorry, wat the magic?

305 line

It's calling the String operator of the Variant class.

Why not use an explicit method or template?
For example as in QT v.value<T>();


Add mapper for org.springframework.web.server.ResponseStatusException

Overriding this one:
io.quarkus.spring.web.runtime.ResponseStatusExceptionMapper implements ExceptionMapper<ResponseStatusException>

@lwitkowski i am a newbie to open source contribution, can i start working on this issue.

@SivaTharun: Feel free to pick this topic (or other similar Exception). Please first read CONTRIBUTING.md. Also please note that we strive for good coding quality thus you may expect - more than usual - in-depth review process. Don't be discouraged by that, we're more then glad to help you with it.
We also try to keep test coverage for all work we do so I'd suggest putting effort in delivering test with initial version of PullReq.

Thanks @pazkooda for the info about contributions and the PR review process.


Kcw grunt/reconfig fb

Reconfigured Firebase and Crashlytics  removed mixpanel

No more red dots

@kcw-grunt looks good to me.


Add FSPoll backend

Added a test for diff. Let me know if there are other things you'd like to test.

Bump for another look?


bug: Drizzle Studio requires a new version of Drizzle Kit to run locally

Provide environment information
System:

OS: macOS 14.4.1
CPU: (14) arm64 Apple M3 Max
Memory: 203.14 MB / 36.00 GB
Shell: 5.9 - /bin/zsh

Binaries:

Node: 22.1.0 - /opt/homebrew/bin/node
npm: 10.7.0 - /opt/homebrew/bin/npm
pnpm: 9.1.0 - /opt/homebrew/bin/pnpm

Describe the bug
The version of drizzle-kit that's shipped with T3 (version 0.20.14) is out of date as of ~11:45 CST today, Friday, May 5th.
Neither the database or drizzle themselves are broken, it's just that drizzle-studio needs the newest version to run locally.
When running pnpm run db:studio using version 0.20.14, the drizzle-studio page gives me this:

The newest drizzle-kit version is 0.21.1 - https://www.npmjs.com/package/drizzle-kit
Drizzle has a guide for migrating to the new version - https://orm.drizzle.team/kit-docs/upgrade-21#how-to-migrate-to-0210
Reproduction repo
N/A
To reproduce
I couldn't provide an example repo since my database connections are local-only and this is a problem that only pops up in a local environment.
In a repo with a working DB connection (i.e. drizzle studio would load fully as of yesterday) that's using drizzle-kit version 0.20.14, run pnpm run db:studio and open the studio page to see the update prompt.
Additional information
I'm opening this with the intention of fixing the bug myself, but if someone gets to it before I do that's all good.

Updated to latest kit


Selenium Gird Scaler | Edge active sessions not being properly counted

Report
Been working on getting a selenium grid 4 deployment working in AKS using Keda to scale. Chrome and Firefox have been no problem but scaling Edge browser pods seems to have a bug.
Currently using the following scaled-object:
apiVersion: keda.sh/v1alpha1
kind: ScaledObject
metadata:
  name: selenium-edge-scaledobject
  namespace: legion-selenium-grid
  labels:
    deploymentName: selenium-edge-node-deployment
spec:
  minReplicaCount: 0
  maxReplicaCount: 80
  scaleTargetRef:
    name: selenium-edge-node-deployment
  triggers:
    - type: selenium-grid
      metadata:
        url: 'https://selenium-grid-qe-tools.accruentsystems.com/graphql'
        browserName: 'MicrosoftEdge'

New sessions in the queue appear as 'MicrosoftEdge' and scale up just fine:

However, taking a look at the active sessions I noticed that the browserName on the active sessions becomes 'msedge':

Thus when it comes time for Keda to check for a scale-down event it sees that there are no active sessions and proceeds scale to 0.
Expected Behavior
Keda detects that Edge pods are still in use and scales down gradually as needed.
Actual Behavior
When scaling down Keda will remove all Edge pods regardless of current active sessions.
Steps to Reproduce the Problem

Deploy selenium grid and Edge deployments
Apply scaled-object for edge (see description)
Can be done in a variety of ways but an easy way: Run a single long (10 minute +) test on selenium grid. Grid will scale up to allow your test to run but will eventually scale to 0 before your test can finish.

Logs from KEDA operator
example

KEDA Version
2.6.1
Kubernetes Version
1.21
Platform
Microsoft Azure
Scaler Details
Selenium Grid
Anything else?
I am not in any way familiar with Go and am having issues deploying a local change to AKS but I believe all that really needs done here is a specific check for "MicrosoftEdge" as a browserName and then simply use the session-specific naming of 'msedge' in that situation.
Probably something like this (though im sure there is a more elegant way to put it). Around line 225 in selenium_grid_scaler:
var sessions = seleniumResponse.Data.SessionsInfo.Sessions
	for _, session := range sessions {
		var capability = capability{}
		var browserCapName = ""
		if browserName == "MicrosoftEdge" {
			browserCapName = "msedge"
		} else {
			browserCapName = browserName
		}
		if err := json.Unmarshal([]byte(session.Capabilities), &capability); err == nil {
			if capability.BrowserName == browserCapName {
				if strings.HasPrefix(capability.BrowserVersion, browserVersion) {
					count++
				} else if browserVersion == DefaultBrowserVersion {
					count++
				}
			}
		} else {
			seleniumGridLog.Error(err, fmt.Sprintf("Error when unmarshaling sessions info: %s", err))
		}
	}

Testing this a bit further and it seems that this limits the scale-up as well.
For example, take this testing scenario:

Scaled object allows up to 20 edge pods at once, normal scale-up behavior.
I have 10 Edge pods, each currently running a test.
I have 5 sessions for edge waiting in the session queue.

What we would expect to happen here is that we would scale up to 15 pods in order to allow the 5 tests in the queue to start. In actuality, nothing happens. I believe this is also because of the edge sessions having a different browserName when they are in an active session. Keda knows there are 10 pods up but does not see any active sessions under the name "MicrosoftEdge" so it assumes the sessions in the queue will be picked up soon and thus does not scale.

Hi,
I'm not an expert but this seems an error in the upstream, I mean. KEDA recovers those values and applies the logic in base of them, but I guess that the consistency between them is part of the upstream responsibility.
Maybe I'm missing something but I think that if we apply that patch and then the upstream uses the same name in both cases, KEDA will fail again.
thoughts @kedacore/keda-core-contributors ?

@JorTurFer After talking with some contributors for selenium it seems that the edgeDriver accepts a few values as valid (having trouble finding any sort of a list though) when starting but will in the end self identify as 'msedge' every time:

It would seem in this case that the driver session would never report anything other than "msedge".
Modifying my session call to appear as "msedge" gets it in the session queue but my current hurdle is getting the Edge node to accept the "msedge" browserName stereotype as something to pick up on the node.
Still working on this so there may be a way to get this working as-is. Even though this is an upstream responsibility I think a small modification in one/two file(s) here would be much simpler than large code changes across multiple projects and tools.

Perhaps this could be an optional parameter such as sessionBrowserName that if set is used for session identification, otherwise ignored and browserName is used as normal? This would allow flexibility outside of this case for edge as well.

I agree with that optional parameter. Maybe explaining it well in the docs we could fill the gap and don't be attached to future changes in their side because it's just remove the value for the optional parameter (but this edge case should be documented IMO)

@JorTurFer Is there a tag needed for this to be discussed or have someone chime in?

An optional parameter seems like the best solution for this, together with a proper documentation.

@zroubalik I have some time so taking a stab at this now.

I have a working solution for this but ran into a bit of a snag: #3061
Going to see about fixing that as well in this as it should be fairly simple.


Pull request history "merged" problem.

It seems that the "merged" actions are not associated with the correct user. Sometimes it is associated with the issue opener sometimes with the closer. Using the github /user/repo/issues api the correct author login could be acquired.
For example the following issues are all merged by "juditacs" user (based on the github api), and in the sql dump:

https://api.github.com/repos/juditacs/wordcount/pulls/15:

opened: kundralaci
merged: juditacs
closed: juditacs

https://api.github.com/repos/juditacs/wordcount/pulls/18

opened: juditacs
merged: juditacs
closed: juditacs

https://api.github.com/repos/juditacs/wordcount/pulls/22

opened: juditacs
closed: juditacs

I have run the following query on the the MySQL query interface
select prh.*, u1.login
    from pull_requests pr, pull_request_history prh, projects p, users u, users u1
    where u.login = 'juditacs'
    and p.name = 'wordcount'
    and pr.pullreq_id = '15'
    and p.owner_id = u.id
    and pr.id = prh.pull_request_id
    and pr.base_repo_id = p.id
    and u1.id = prh.actor_id

It returns the following:

This corresponds to the series of events that I see here: https://github.com/juditacs/wordcount/pull/15
I do not see what the problem is.

That one is correct, but:
select prh.*, u1.login
    from pull_requests pr, pull_request_history prh, projects p, users u, users u1
    where u.login = 'juditacs'
    and p.name = 'wordcount'
    and pr.pullreq_id = '22'
    and p.owner_id = u.id
    and pr.id = prh.pull_request_id
    and pr.base_repo_id = p.id
    and u1.id = prh.actor_id

gives:

Which is is different than: juditacs/wordcount#22
I included the 3 examples to demonstrate the different cases.

I see. You are missing the merged event for PR 22. This means that the GitHub event associated with the merge was not received or processed by GHTorrent. I 've done some research in the Mongo database and it seems this is indeed the case. I am not sure what the reason is but I think it is because the merged happened while GHTorrent was moving to the cloud, so some events might have been lost.

Unfortunately the query interface doesn't give me any results as guest user now, so I work with the downloaded dumps.
I checked another pull request from another repository: https://github.com/zseder/hunvec/pull/94, and I see the same problem even though this PR's activity was in last july, not when GHTorrent was movint to the cloud.
Can you check it? Thanks!

Hi @gousiosg, just adding to that. We found some other cases like this.
Just another example (bitcoin/bitcoin, issue #6221)
https://api.github.com/repos/bitcoin/bitcoin/pulls/6221
It was, in fact, merged. But the pull_request_history shows only two actions (opened and closed).
SELECT prh.* FROM pull_requests pr, pull_request_history prh
WHERE pr.id = prh.pull_request_id AND pr.base_repo_id=864 and pr.pullreq_id=6221;
What you suggest? Moving to Mongo to avoid these cases? :-)
Thanks a lot


Prepare release candidate v0.9.0-RC2

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8337/
Test FAILed.

Jenkins, test this please

LGTM

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8338/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/8340/
Test PASSed.


New method AttachmentsList.add(attachment)

I added a new function to the AttachmentsList class to manipulate attachments without decode/encode and rewriting id's etc.
We are using the Mail gem for a message router and need everytime to copy the original attachments.
Instead of re-creating that attachments like
mail_in = Mail.read_from_string(mail_source)
mail_out = Mail.new
# copy attachments
mail_in.attachments.each do |a|
  mail_out.attachments[a.filename] = {
    :mime_type => a.mime_type,
    :encoding => a.content_transfer_encoding,
    :content_id => a.content_id,
    :content_disposition => a.content_disposition,
    :content => a.decoded
  }
end

now you just can append the already processed attachment from original message.
mail_in = Mail.read_from_string(mail_source)
mail_out = Mail.new
# copy attachments
mail_in.attachments.each { |a| mail_out.attachments.add(a) }

Hopefully this change makes sense for you.
Looking for your feedback
Tom

Can these be added using the existing parts API?

Hi @jeremy
what does that mean: "Can these be added using the existing parts API?"
Should I check it with latest sources?
Thanks for some more info
Tom

Give this a shot:
mail_in = Mail.new(mail_source)
mail_out = Mail.new
mail_in.attachments.each { |part| mail_out.body << part }


Move "SDK Towards 1.0" post to docs/understand

docs/posts does not get published automatically.  This way, recommended by
@nopcoder, we keep it with other technical documentation and easy to find.

Thanks!


happy new year，how i cross domain use the bibireader

**e.g:**http://www.lsjxw.cn/res/txb/js/epubreader/bibi/index.html?book=http://kod.ls123.site/txbmicloudWP4rvk20200829213816.epub
the console:
Access to XMLHttpRequest at 'http://kod.ls123.site/txbmicloudWP4rvk20200829213816.epub/META-INF/container.xml' from origin 'http://www.lsjxw.cn' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
bibi.js:11 Failed to open the book (as a file: Data Invalid / as a folder: Probably CORS Blocked)
log @ bibi.js:11
U.error @ bibi.js:11
Promise.catch (async)
x.hello @ bibi.js:11
(anonymous) @ bibi.js:11
document.addEventListener.e @ bibi.js:11
(anonymous) @ bibi.js:11
bibi.js:11 GET http://kod.ls123.site/txbmicloudWP4rvk20200829213816.epub/META-INF/container.xml net::ERR_FAILED
(anonymous) @ bibi.js:11
U.download @ bibi.js:11
s @ bibi.js:11
(anonymous) @ bibi.js:11
Promise.catch (async)
or @ bibi.js:11
(anonymous) @ bibi.js:11
initializeBook @ bibi.js:11
(anonymous) @ bibi.js:11
Promise.then (async)
x.loadBook @ bibi.js:11
Promise.then (async)
x.hello @ bibi.js:11
(anonymous) @ bibi.js:11
document.addEventListener.e @ bibi.js:11
(anonymous) @ bibi.js:11
the curl：
curl -I http://kod.ls123.site/txbmicloudWP4rvk20200829213816.epub
HTTP/1.1 200 OK
Server: nginx/1.16.1
Date: Fri, 01 Jan 2021 02:47:24 GMT
Content-Type: application/epub+zip
Content-Length: 149803
Last-Modified: Sat, 26 Dec 2020 13:58:17 GMT
Connection: keep-alive
ETag: "5fe74179-2492b"
Access-Control-Allow-Origin: *
Accept-Ranges: bytes
but the same file if not in other domain display well
http://www.lsjxw.cn/res/txb/js/epubreader/bibi/index.html?book=http://www.lsjxw.cn/txbmicloudWP4rvk20200829213816.epub
the console
Bibi: 📕 Hello! (v1.2.0)
the curl
curl -I http://www.lsjxw.cn/txbmicloudWP4rvk20200829213816.epub
HTTP/1.1 200 OK
Server: nginx/1.16.1
Date: Fri, 01 Jan 2021 03:07:30 GMT
Content-Type: application/epub+zip
Content-Length: 149803
Last-Modified: Thu, 31 Dec 2020 17:52:26 GMT
Connection: keep-alive
ETag: "5fee0fda-2492b"
Access-Control-Allow-Origin: *
Accept-Ranges: bytes
i want to get  the auther 's help!!!!!!!thanks

why the "META-INF/container.xml" add to the bottom!?????

why the "META-INF/container.xml" add to the bottom!?????

who can give me a success e.g of the cross domain

who can give me a success e.g of the cross domain

why the "META-INF/container.xml" add to the bottom!?????

I have set the CORS settings on my DigitalOcean Spaces and yet it didn't work for me. please help me. why request adding /META-INF/container.xml to the end of epub file url?


[BUG] [REGRESSION] Old larger slider ui

Describe the bug
One of the previous versions had a large slider input that was easily draggable. This is no longer the case
To Reproduce
Steps to reproduce the bug:

Create a form with a Slider type component (I used 1 to 99 as range)
Run the form (with QuickAdd for example)
View on mobile

Actual behavior
A small slider is shown (similarly on desktop)
Expected behavior
A large slider filling the width of the screen is shown
Additional context
Some time during one of the previous updates of either Obsidian or Modalform, the slider changed from a large ui to the tiny one that's currently shown on Android and Windows. The new one is extremely hard to use.
Don't know if this was due to a change in Obsidian (Electron update?) or Modal forms

Alternatively, I need a better way to (very quickly!) input numbers in the range from 1 to 99. I suppose changing to a number field means I'd need to do 4 clicks (touches) instead of one. That may be better than the current imprecise way, but not as fast

Can you provide an screenshot of how it looks? I didn't noticed any change.
This is how it looks in my mobile phone

@danielo515 that is how it looks like now. But it wasn't how it looked like in the past.
I don't have a screenshot for what that used to look like unfortunately, only a description from my memory. It filled the whole width, instead of just a small part, and I think the knob was green and square.
This made me realize there's yet another possibility: the theme that I was using (minimalist theme) might have changed. This probably happened in a major Obsidian UI change. Possibly Obsidian 1.6 or 1.5. Sorry that I'd been so late in reporting this!

Nah, don't worry. I also noticed the regression, but I was not sure if it
was going to be a problem.
Will it be better if (on mobile) the bar is pushed to the next "line" and
takes the full  width?
On Thu, Aug 22, 2024 at 10:37 PM seomwan @.***> wrote:

@danielo515 https://github.com/danielo515 that is how it looks like
now. But it wasn't how it looked like in the past.
I don't have a screenshot for what that used to look like unfortunately,
only a description from my memory. It filled the whole width, instead of
just a small part, and I think the knob was green and square.
This made me realize there's yet another possibility: the theme that I was
using (minimalist theme) might have changed. This probably happened in a
major Obsidian UI change. Possibly Obsidian 1.6 or 1.5. Sorry that I'd been
so late in reporting this!
—
Reply to this email directly, view it on GitHub
https://github.com/danielo515/obsidian-modal-form/issues/314#issuecomment-2305586873,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARKJWMRNJIRJ4JSCXJFH6TZSZDXVAVCNFSM6AAAAABM5IRXIOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBVGU4DMOBXGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
https://danielorodriguez.com

Will it be better if (on mobile) the bar is pushed to the next "line" and takes the full width?

Btw, in case it wasn't obvious from the reaction, I meant yes, it would help. Ultimately, my goal is inputting 2 digit numbers as fast as possible on mobile, so if there's an alternative, that works too!


Best Way to run multiple wrappers

I'm not even sure that using multiple wrappers is what I need to do but I think it is - I need to run a system that takes several camera inputs (non overlapping FOV's) and detect objects in each of the camera frames. I can get it working with only a single wrapper but surely as the object detection on a given camera is un-related to any other camera (as far as yolo is concerned) there must be a way to use the built in parallel programming systems in c# to accelerate the process.
Currently when I attempt execution I get a memory access error the error occurs in the block of the yolo wrapper that copies the image array to unmanaged memory.
With GPU : Exception thrown at 0x00007FFD8EBB31E9 (yolo_cpp_dll_gpu.dll) in ImageStitching.exe: 0xC0000005: Access violation writing location 0x0000000000000000.

I think the best option for you is use just one wrapper. Add the frames to a queue and process image one by one

Ah okay, fair enough. I guess I need to find an alternative, if everything has to go through the single wrapper that's going to be a mega bottleneck. I'm guessing it's a case of me not managing memory correctly.

Do you use the graphics card?

Yeah, I'm running the pre-trained yolo-v3 model on a GTX 1050 feeding it  grey scale frames at 300x300 resolution (Yes I know it's a random resolution)

I had the same challenge: Processing images with three different models. This doesn't work because If I try to use more then one wrapper I detect nothing.
I worked around: I build a single exe file which act as a WCF server. I start the exe file with three different configurations for the model (and for the port number for the service). With this separation, I reach my goal. Not beautiful but usable as a workaround.


How to Build a Query like this?

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[x] elastic.v6 (for Elasticsearch 6.x)
I have this source found at https://blog.trifork.com/2014/03/26/server-side-clustering-of-geo-points-on-a-map-using-elasticsearch-continued/
{
    "query": {
        "match_all": {}
    },
    "size": 0,
    "aggs": {
        "filtered_cells": {
            "filter": {
                "geo_bounding_box": {
                    "location": {
                        "top_left": "50.1234, 4.0000",
                        "bottom_right": "52.1234, 4.5555"
                    }
                }
            },
            "aggs": {
                "cells": {
                    "geohash_grid": {
                        "field": "location",
                        "precision": "precision"
                    },
                    "aggs": {
                        "center_lat": {
                            "avg": {
                                "script": "doc['location'].lat"
                            }
                        },
                        "center_lon": {
                            "avg": {
                                "script": "doc['location'].lon"
                            }
                        }
                    }
                }
            }
        }
    }
}

How can i build this? i know that i can use geocentroid but I don't know how to build the querys with elastic.
Can you please guide me. Thank you.

I understand that documentation is not as good as it should be. But please understand that I also simply don't have enough time to implement each individual request for converting the JSON API to the Go API.
Having said that, here's a few hints.
First, the tests really are helpful. They not only serve to ensure we have a solid basis when making changes, but also as a way to illustrate usage. E.g. search_aggs_test.go illustrates using most aggregation types in an integration test.
Your query should basically look like this (without having tested it in any way):
// Create a client
client, err := elastic.NewClient(...)
...
// Create the query
query := elastic.NewMatchAllQuery()
...
// Create the aggregations (best to setup from inside to outside)
cellsAgg := ...
filteredCellsAgg := elastic. NewFilterAggregation().Filter(
    elastic.NewGeoBoundingBoxQuery("location").
      TopLeft(50.1234, 4.0000).
      BottomRight(52.1234, 4.5555)
    ).
    SubAggregation("cells", cellsAgg))
...
// Execute the search
resp, err := client.Search().
  Index(...).Type(...).
  Query(query).
  Aggregation("filtered_cells", filteredCellsAgg).
  Do(context.TODO())
...
// Check the search response
filteredCellsAggRes, found := agg.Filter("cells")
if !found {
  ...
}
if filteredCellsAggRes == nil {
  ...
}
if filteredCellsAggRes.DocCount != 2 {
  ...
}
...

HTH.

@olivere thank you for you example, now I got the result as expect.
Thank you!


Illegal instruction

Hi!
I just tried this example repository, when I try to send a push notification, my server crashes with "Illegal instruction: 4"
Tested on a macOS 10.12.6 with PHP7.3.8
Any ideas ?

OK, I just figured out what happens. This is related with my php bin. Installing the same version with brew solved the issue.


DEPRECATION: spglib depreciating ASE Atoms-like input.

IFermi currently supports spglib version 2.2.0. To take VASP-like input and put it into spglib, IFermi currently converts to an ASE Atoms-like format, such as in the following code.
https://github.com/fermisurfaces/IFermi/blob/a681674bdb5af86758e97fe7eaeec9f62a7e900e/src/ifermi/interpolate.py#L128-L131
However, using ASE Atoms-like input for spglib is deprecated, and is completely absent in spglib 2.3.0.
At present, installing the latest version of spglib causes IFermi to fail.
The current work-around is to downgrade spglib to 2.2.0 or less.
A fix would involve converting the structures to spglib cell format manually, before passing to spglib, rather than relying on ASEAtomsAdaptor. A fix would be required before further upgrading spglib to 2.3.0 or greater.

Thanks for the bug report. I've pushed a fix in version 0.3.4


Revert "fix(apps/prod/tekton/configs/triggers): disable tiflash mac target builds"

Reverts PingCAP-QE/ee-ops#1168

/approve


[RBF] Nonce used twice after replacing a tx

I have 5 pending txs with nonce 10, 11, 12, 13, 14.
I create a new tx to replace tx 10 by changing the nonce of the new tx to 10 and by adjusting the fees. The new tx is broadcasted.
Next time I create a tx the suggested nonce is 10, it should be 15.

After using the "Edit nonce" feature, the extension has overwritten some of my txs without me noticing it. To me this is more than "P3".

@timstackblock do we have a related PR we can link here?

@friedger We made a fix to improve nonce calculation that was released after the day you posted this issue. I tested the above scenario and it's working fine now.
Can you try again ? It should work as expected now.


Visualizations failed to render after adding dimension and measures

Kudos to the team.. Its worthy effort..
Can you please help me getting it executed..
I added d3.min.js file at the root. I have changed this in graphics-deltaviz-delta.js file - define(["jquery", "text!./graphics-deltaviz-delta.css", "./d3.min"
to
define(["jquery", "text!./graphics-deltaviz-delta.css", "./d3.min.js"
this made object to work and allowed to enter dimension and measures.
then i have added dimension and measures in QS but after this an error is appearing stating "visualizations failed to render".
Demo app is showing same message.. I have 64 bit machine and 1.0.2 version of QSense.

I got the solution .....thanks a lot....

Hi @shiv
What error do you have ? You can check error messages i.e. in Chrome Console using CTRL+SHIFT+J

Thanks blake for the Reply.. Sorry for being late, its working now....


Bump tests on master branch to Blender 2.83 Alpha.

The blender-2.82-release branch will continue testing against 2.82 Beta, and master will migrate to testing against 2.83 Alpha.
Fixes #873.
See https://github.com/KhronosGroup/glTF-Blender-IO/issues/873#issuecomment-573449980

/cc @scurest

Yeah, like that (not sure if it will ever be useful but...)


Preferred method for accessing file contents?

Some of the endpoints have URLs to files that contain useful data.
Example:
https://qa-reports.linaro.org/api/testruns/2553981/metadata_file/
What is the preferred method of using the squad-client library to access the contents of that file?

I'm using the SquadApi in the following manner currently:
ep = 'api/testruns/%d/tests_file' % (tr.id)
 response = SquadApi.get(ep)
tests = response.json()

I swapped to using the following, which is similar:
ep = 'api/testruns/%d/tests_file' % (tr.id)
response = SquadApi.get(ep)
tests = json.loads(response.text, strict=False)

I think a helper that can provide this would be useful. What do you think?

It looks like we won't be using the *_file options, so closing this.


Search results type for PR

Repository search results are currently categorized in 4 types: Code, Commits, Issues and Wikis (if available).
Where issues is a combination of issues and PR. My suggestion is that we make another type for all PR's and separate them out of the issues type.

These are the URLs to make it happen:
https://github.com/sindresorhus/refined-github/search?q=diff+is%3Aissue&type=Issues
https://github.com/sindresorhus/refined-github/search?q=diff+is%3Apr&type=Issues

I understand this is under discussion, but as a learning project I build this feature: https://github.com/sindresorhus/refined-github/compare/master...jerone:features/jvw-1635-search-results-filter
If interested I can create an PR...


advanced_server.cpp intermittence no output

Compile with sample program, sometimes there is no output, you have to click the link again, does anyone know why?

Hi @iams13 ,
It's impossible to draw any conclusions from the information available.
It would be helpful if you could submit another issue, containing detailed repeatable steps to reproduce the issue locally.


[Suggestion] Manganese compatibility with Metallurgy 4

Crushed Manganese Oxide should be craftable by oxidizing Manganese Dust from Metallurgy 4

Recipe added for 1.8a ;)


Bad Links

Some of your links aren't working...possible it's intentional, unsure though.

You mean the Open-Source Model link ? It works perfectly though


Speed up Appveyor builds

We're building all crates, then rebuilding them immediately after for testing. This removes the first build.

Good call, TravisCI got this but Appveyor missed it.


Reduced ops for grpc_chttp2_stream_map_find().

Cached grpc_chttp2_parsing_lookup_stream() results.
Inlined grpc_chttp2_parsing_lookup_stream() - it's a direct jump to
grpc_chttp2_stream_map_find().
Removed a few branches in grpc_chttp2_stream_map_find().

This is purely internal to chttp2 with no external API implication, so I'll let Yash review.

https://github.com/grpc/grpc/pull/19672 for debug asserts to simplify this PR.


Three categories of precision are missing in quantitative comparison results on the sscbench-kitti-360 dataset

Hello! Thank you so much for such excellent work! This is a great contribution to community research!
However, I have a question for you. I noticed that the quantitative comparison on SSCBench-KITTI-360 in the article lacks accuracy reports for three classes, which are as follows: parking(2.31%), other-grnd(2.05%), other-struct.(4.33%). The accuracy calculation of these three categories is also missing in the evaluation code. Can you explain why?

Thanks for your reply! I get it!
All the best！
Pute


Await the adapt call of a dask cluster instance if it's an asynchronous method

This PR will await the adapt call when the underlying dask cluster implementation returns a future for that method.
As discussed in the issue, it seemed like an update that should be done upstream in the dask-kubernetes package but according to its maintainer, the new operator.KubeCluster is doing I/O during adapt so it should in theory be awaited when run asynchronously.
Closes #57
Checklist

[x] References any related issue by including "Closes #" or "Closes ".

If no issue exists and your change is not a small fix, please create an issue first.

[ ] Includes tests or only affects documentation.
[x] Passes pre-commit checks.

Run pre-commit install && pre-commit run --all locally for formatting and linting.

[ ] Includes screenshots of documentation updates.

Run mkdocs serve view documentation locally.

[ ] Summarizes PR's changes in CHANGELOG.md

Thanks for contributing this! Would you be able to help add a CHANGELOG entry and also a test with a mock cluster that has an awaitable   adapt method?

@ahuang11 I added a test to check if the adapt method was called properly and updated the CHANGELOG as requested. Since the cluster class instantiation happens inside the DaskTaskRunner class, I am not sure how to use a unittest.Mock instance in that case so I just added dummy Mocks in the test itself. Please let me know if there's a more elegant way of doing this.


Failed to connect to DynamoDB Local on systems with a non-basic default credentials profile

If I have a default credentials profile on my system that is not the access key + secret key type, when I try to connect to DynamoDB Local in the Toolkit, I get a "Failed to connect to DynamoDB Local" error:

Some examples of when this happens:

there is no default credentials profile on my system
my default profile leverages AWS SSO

We've been able to reproduce this situation, and need to investigate a fix in the Toolkit. As a workaround, set up a default profile on your system that has a valid access key and secret key, prior to launching Visual Studio. The Toolkit should be able to successfully connect to DynamoDB Local.

A fix for this issue has been released in version <IP_ADDRESS>


Updated libraries list with those that support Vapor 4.0.

README.md
I updated all those packages that already support 4.0. I also deleted the two dead vapor services and the dead VaporNation newsletter.
I also request information on whether packages were still under development from a number of people and will update as I hear.

Woo, thank you for your work, that’s an amazing help for sure!
Can I ask for two things first, though?

Instead of submitting this as an issue, could you fork the repository, commit your changes on a new branch and submit a pull request against this repository, please? That way, we can discuss the individual code-level details better.
I think that in order to not lose the filtered for Vapor 3 compatibility list (and at the same time not to have many entries duplicated twice in the main list), the libraries, frameworks and entries that support (or at some point supported) Vapor 3 should have both Vapor 3’s and Vapor 4’s icon on a single line. Kind of like this in the past, when we had a lot of libraries with both Vapor 2 & Vapor 3 compatibility: https://github.com/Cellane/awesome-vapor/commit/516f5713726376d9b87a8e603befae16eba1f12b#diff-45c6dc1ce947996409b06d7f3e7d8125fb44d29b39ae969d268928157c3306bbR33-R38

Thank you again and I’m looking forward to the pull request!

Happy to make those changes and have a few that are no longer support that can be deleted. I don’t know how to create a pull request. Happy to learn, just no idea what to do.
/carmi

On Sep 30, 2021, at 5:45 PM, Milan Vit @.***> wrote:
Woo, thank you for your work, that’s an amazing help for sure!
Can I ask for two things first, though?
Instead of submitting this as an issue, could you fork the repository, commit your changes on a new branch and submit a pull request against this repository, please? That way, we can discuss the individual code-level details better.
I think that in order to not lose the filtered for Vapor 3 compatibility list (and at the same time not to have many entries duplicated twice in the main list), the libraries, frameworks and entries that support (or at some point supported) Vapor 3 should have both Vapor 3’s and Vapor 4’s icon on a single line. Kind of like this in the past, when we had a lot of libraries with both Vapor 2 & Vapor 3 compatibility: 516f571#diff-45c6dc1ce947996409b06d7f3e7d8125fb44d29b39ae969d268928157c3306bbR33-R38 https://github.com/Cellane/awesome-vapor/commit/516f5713726376d9b87a8e603befae16eba1f12b#diff-45c6dc1ce947996409b06d7f3e7d8125fb44d29b39ae969d268928157c3306bbR33-R38
Thank you again and I’m looking forward to the pull request!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Cellane/awesome-vapor/issues/86#issuecomment-931805316, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABJK5BLFVKTSTN6CHUDOJQDUEUABDANCNFSM5FBKOOPQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thank you so much, and sorry for the lack of updates from my side! Things have been quite hectic recently… will try to review/change/approve by the end of the week! 🤞


The js_module_root flag doesn't work as documented

Currently, the ES6 module loader interprets import paths that start with / as absolute filesystem paths.  e.g., you can have:
import { X } from '/home/hearnden/projects/myproj/app/x';

Surely it would be more useful, and probably closer in line with https://html.spec.whatwg.org/multipage/webappapis.html#integration-with-the-javascript-module-system, for the path to be interpreted with respect to (one of) the module root URIs.
e.g., with a module root URI of /home/hearnden/projects/myproj/app, you could have:
import { X } from '/x';

which would do the right thing, regardless of the path of the importing script.
This appears trivial to make happen:
diff --git a/src/com/google/javascript/jscomp/ES6ModuleLoader.java b/src/com/google/javascript/jscomp/ES6ModuleLoader.java
index a36ff64..83e423a 100644
--- a/src/com/google/javascript/jscomp/ES6ModuleLoader.java
+++ b/src/com/google/javascript/jscomp/ES6ModuleLoader.java
@@ -102,9 +102,11 @@ public final class ES6ModuleLoader {
     URI uri = createUri(name);
     if (isRelativeIdentifier(name)) {
       URI referrerUri = normalizeInputAddress(referrer);
-      uri = referrerUri.resolve(uri);
+      uri = normalizeAddress(referrerUri.resolve(uri));
+    } else if (isAbsoluteIdentifier(name)) {
+      // Strip leading slash, to match the normalized entries in moduleUris.
+      uri = createUri(name.substring(1));
     }
-    URI normalized = normalizeAddress(uri);
     if (moduleUris.contains(normalized)) {
       return normalized;
     }

However, that change isn't backwards compatible, because it would break the use of absolute filesystem paths.  But surely that would be a good thing?

The compiler is actually pretty agnostic when it comes to where the absolute root actually is. It's based off the file path provided to it by the source flags --js=/path/to/foo.js. For instance, the grunt/gulp plugins anchor the root to the current working directory of the compiler.
Also, make sure you look at the --js_module_root flag.
The change you suggest above would break active projects.
I'm closing this issue for now. If the --js_module_root flag doesn't do what you want, then we can re-open.

Bare import paths work fine with --js_module_root, and this issue report doesn't affect them. Please note I'm talking about absolute import paths, not bare import paths.  i.e.,
import { X } from '/x/x';

not
import { X } from 'x/x';

Absolute import paths only get interpreted as a path from the filesystem root.  They don't get interpreted as a path from a module root.
Failing example:
/tmp/x/x.js:
export const X = 1;

/tmp/y/y.js:
import { X } from '/x/x';

With --js_module_root /tmp --js /tmp/x/x.js /tmp/y/y.js, the import fails:
/tmp/y/y.js:1: ERROR - Failed to load module "/x/x"
import { X } from '/x/x';
^

1 error(s), 0 warning(s)

The import works fine with the bare path x/x, which is the --js_module_root behavior you're describing.
All these invocations fail with the same error when attempting to use an absolute import path.

--js_module_root /tmp --js /tmp/x/x.js /tmp/y/y.js
--js_module_root /tmp --js x/x.js y/y.js

Using --js /x/x.js fails with ERROR - Cannot read: /x/x.js.

Ah - I think I see what you meant.
--js_module_root / --js x/x.js y/y.js does the trick, allowing an import of '/x/x' to get resolved as x/x relative to the the compiler's working directory.
So --js_module_root isn't a module root in the normal sense (i.e., a location from which to resolve module paths, like a classpath or an include path).  Rather, it's an arbitrary string prefix to strip off import paths before resolving the remainder against the current working directory of the compiler.  i.e., more like a mount point.
Thanks for your help.

I'm not sure I fully understand why this doesn't work:
--js_module_root /tmp --js /tmp/x/x.js /tmp/y/y.js

From my understanding of the flag, it should have. I may need to look into that more.

Apologies if this is hijacking this issue but I can't find any other issue or wiki page more relevant.
I am struggling to figure out how to get node_modules and commonjs requires to work with Closure Compiler using --js_module_root.
It would be really helpful if there was a node_modules example added here, showing the command line, directory structure, working dir (if relevant) and require line:
https://github.com/google/closure-compiler/wiki/JS-Modules
Also the command line usage could be more verbose. "Path prefixes to be removed from ES6 & CommonJS modules." I'm struggling to see why you'd want to put long prefixes in your require statements.

@supersteves The reason there is no documentation is it doesn't really work. You have to pre-process node_modules files so that the imports are paths instead of named modules.
There are changes to the compiler I have planned to address this, but nothing has been actually started yet.

Thanks @ChadKillingsworth ... saw your response only now after posting https://github.com/google/closure-compiler/issues/1773

I believe this is working as documented/intended.


Use AtomicIntegerFieldUpdater to mutate volatile int totalAvailablePermits in PersistentDispatcherMultipleConsumers class

Motivation
AtomicIntegerFieldUpdater should be used to update volatile int fields when the value is incremented or decremented. This prevents "lost update" issues.
Modifications
Use existing TOTAL_AVAILABLE_PERMITS_UPDATER to mutate totalAvailablePermits field.
Co-authored-by: Devin Bost<EMAIL_ADDRESS>

This PR was co-authored with @devinbost . Thanks for @devinbost for pinpointing the code location around totalAvailablePermits in PersistentDispatcherMultipleConsumers class. This fix wouldn't have happened without @devinbost 's persistent investigation work. This change might fix issues such as #6054.
@codelipenghui @merlimat @eolivelli @rdhabalia Please review this change.

btw. This problem was only in the PersistentDispatcherMultipleConsumers class. Similar code in NonPersistentDispatcherMultipleConsumers didn't have the issue: https://github.com/apache/pulsar/blob/31f831574c3a65bae0e76801facaf2deb0b17fbb/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/nonpersistent/NonPersistentDispatcherMultipleConsumers.java#L94-L148

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

I'm closing this PR since as @merlimat pointed out, all mutations are already safe since they are done in synchronized methods in PersistentDispatcherMultipleConsumers class.

Currently PersistentStreamingDispatcherMultipleConsumers.readMoreEntries method isn't synchronized:
https://github.com/apache/pulsar/blob/f2d72c9fc13a33df584ec1bd96a4c147774b858d/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentStreamingDispatcherMultipleConsumers.java#L137-L140
Is there a reason why it's not a synchronized method? In one location the method is called within a synchronized block like this:
https://github.com/apache/pulsar/blob/31f831574c3a65bae0e76801facaf2deb0b17fbb/pulsar-broker/src/main/java/org/apache/pulsar/broker/delayed/InMemoryDelayedDeliveryTracker.java#L191-L205
Synchronization is missing in this location:
https://github.com/apache/pulsar/blob/a1cebdb3e7cc3b8ee3ce567058182e7d31c762d2/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java#L2439-L2451

Another location where the readEntries call is explicitly wrapped in a synchronized block
https://github.com/apache/pulsar/blob/875262a0ae2fba82a6dd7d46dd2467d675b0d9c3/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java#L612-L621
I'm running an experiment to see if tests pass when readEntries method is made synchronized.

I opened a separate PR #10413 to make the readMoreEntries method synchronized.

Not needed since #10413 was merged.


Integrate DESI spec catalog

Two questions for @jrastinejad:

Is the DESI database table already populated?
If so, have you tested this on a target that you know has a DESI host galaxy match? Can you post the name of that target here?

I fixed one typo and the code runs now. But I tried testing it on a random source in the DESI table, and I found out that that source has 0 flux in all bands:
sassy=# select targetid, target_ra, target_dec, z, flux_g, flux_r, flux_z, fiberflux_g, fiberflux_r, fiberflux_z from desi_spec_q3c limit 1;
      targetid      |     target_ra      |     target_dec     |         z          | flux_g | flux_r | flux_z |      fiberflux_g      |     fiberflux_r      |     fiberflux_z
--------------------+--------------------+--------------------+--------------------+--------+--------+--------+-----------------------+----------------------+----------------------
<PHONE_NUMBER>57593610 | 23.914121939862518 | 29.974787585945496 | 1.0791710884059862 |      0 |      0 |      0 | -0.003990191034972668 | 0.026369713246822357 | 0.002765043405815959
(1 row)

We should figure out what this means and decide how to treat matches like this. Right now the code just crashes.

Hey Griffin, I pushed new changes so now the code:

only runs if z, Gaia or flux_r value are greater than zero
uses a Gaia mag if flux_R does not exist.
I looked at the first 20 rows of the DESI table, and only one object did not have a flux_r nor Gaia value. It's spec-z was high (1.2) which leads me to believe we won't miss bright nearby hosts.


Verification

Hello 17/1/22 currently, and i was wondering if anyone using it has a verification problem. It tries to verify with 2captcha and the other option but it always fails. Any suggestions/help?
Thanks.

Too me. Hixx


Creating completely custom shapes

I am trying to make a custom shape which cannot be created using TLBoxUtil (like in the example). More specifically, I am trying to make a B-Spline shape that takes a list of points as props. So, I am trying to use TLShapeUtil, which I assume is the correct type to extend. However, the Box2d class is not exported, so I can't type the getBounds return type correctly or initialize a Box2d. This is the first issue I have run into, so I am not sure whether there are more issues with defining shapes this way.
Also, custom shapes need to implement a getOutline method. What does this actually do? What is the "outline" of a shape. This is not specified in its documentation. It should return an array of Vec2dModel (which are just points?), so I think that these should be the corners of the shape. But, I feel like this could be implemented for all shapes by using its getBounds method. So, I do not really see the point of having to implement this method for each shape, or am I wrong about what this function should do?
Also, how would I go about making a tool for this shape. What class can I extend? I do not think a TLShapeTool is exported (or something like that).

I just found that the @tldraw/primitives package exports Box2d, so I can use this I think. Is this package mentioned anywhere in the examples or on docs.tldraw.dev?

I've added another example in my fork to demonstrate how to make a custom shape from scratch (without relying on the Box util and tool). Might be useful as a starting point or skeleton: https://github.com/TodePond/tldraw-examples/blob/main/src/examples/6-custom-config-from-scratch/CustomConfigFromScratchExample.tsx

Nice, I've been able to figure everything out with your example. For now at least :).


ALL Streets are Broken

toontown-220706_031941.log

In toontown.hood.StreetSign.py, replace 'http://www.toontownfantasy.com/assets/phase_3/sign/street-signs/img/' with 'http://www.toontownfantasy.com/assets/phase_3/sign/'.


OptionButton dropdown popup doesn't scale

Godot version:

v3.1.dev.custom_build.6f32842
OS/device including version:

Issue description:

Steps to reproduce:

Create OptionButton node
Change scale in Rect->Scale e.g. x=2, y=2
Play Scene
Press button and dropdown don't scale

Minimal reproduction project:

I see the same problem in 3.0.5


Feature/email notification

Status page
-- Participant gets emailed their status page QR code and link upon joining the queue.
-- (BUGFIX) Fixed a bug where the creation of Notification object is not fetching the correct message data.
-- Participants will now get notification messages from staff.
-- added default value to messages "Your turn is here!"
-- Status page has notification modal that will flashes red when there are new unread notification.
-- Added wait time and estimated wait time in status page.
-- Leave Queue have has a confirmation prompt.
-- Participant now have auto generated numbers (A001 - A999 - B001)
-- Plays a sound when a new notification comes in.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 27.53623% with 150 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

manager/views.py
5.55%
85 Missing :warning:

participant/views.py
28.26%
33 Missing :warning:

manager/forms.py
0.00%
13 Missing :warning:

manager/utils/send_email.py
42.10%
11 Missing :warning:

manager/utils/code_generator.py
71.42%
3 Missing and 3 partials :warning:

participant/forms.py
0.00%
1 Missing :warning:

participant/models.py
92.30%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Flag
Coverage Δ

unittests
37.16% <27.53%> (+3.48%)
:arrow_up:

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

Files with missing lines
Coverage Δ

manager/models.py
38.61% <100.00%> (+2.56%)
:arrow_up:

manager/utils/category_handler.py
26.92% <ø> (ø)

participant/urls.py
100.00% <100.00%> (ø)

participant/forms.py
36.84% <0.00%> (-2.05%)
:arrow_down:

participant/models.py
79.59% <92.30%> (+20.40%)
:arrow_up:

manager/utils/code_generator.py
71.42% <71.42%> (ø)

manager/utils/send_email.py
42.10% <42.10%> (ø)

manager/forms.py
59.61% <0.00%> (-7.06%)
:arrow_down:

participant/views.py
35.02% <28.26%> (+0.25%)
:arrow_up:

manager/views.py
19.43% <5.55%> (-0.47%)
:arrow_down:

... and 2 files with indirect coverage changes

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests

Good job, Tora


Update to version 1.0.2: Exclude directories starting with a period

Issue: tested with a Nuxt3 project and realised we need to exclude .nuxt/ and .output/ folders.
Fix: exclude all dirs beginning with a period.

Approved


MLIL optimizes out assignments that might be better left in

Analyzing this function:
mov rax, 0x42
test rax, rax
jz L1
call rax
L1:
ret

Results in the following MLIL:
+-------------------------------------------------------+
|main:                                                  |
|   0 @ 00000604  if (false) then 1 @ L1 else 2 @ 0x606 |
+-------------------------------------------------------+
    ,............' :
    :              :
    : +------------v-------------------+
    : |   2 @ 00000606  rax = sub_42() |
    : |   3 @ 00000606  goto 1 @ L1    |
    : +--------------------------------+
    `...........,  :
                :  :
         +------v--v-----------------+
         |   1 @ 00000608 return rax |
         +---------------------------+

The rax = 0x42 appears in the Mapped MLIL but has been optimized out when generating the MLIL. The value of rax at 1 can only come from 2 because the condition at 0 is false. When going to SSA the value from that impossible branch becomes explicit:
        +-----------------------------------------------------------------+
        |main:                                                            |
        |   0 @ 00000604  if (false) then 1 @ __elf_header else 2 @ 0x606 |
        +-----------------------------------------------------------------+
            ,.........................' `................,
            :                                            :
+-----------v-----------------+   +----------------------v--------------------------+
|   1 @ 00000000  goto 4 @ L1 |   |   2 @ 00000606  rax#1, mem#1 = sub_42() @ mem#0 |
+-----------------------------+   |   3 @ 00000606  goto 4 @ L1                     |
            :                     +-------------------------------------------------+
            :                                            :
            `.......................,  ,.................'
                                    :  :
                 +------------------v--v---------------------+
                 |   4 @ 00000608  rax#2 = phi(rax#0, rax#1) |
                 |   5 @ 00000608  mem#2 = phi(mem#0, mem#1) |
                 |   6 @ 00000608  return rax#2              |
                 +-------------------------------------------+

Now there's a phi node that refers to rax#0 which is undefined. Is it better to leave the rax = 0x42 in so there aren't unnecessarily undefined variables or better to optimize it out? Is there a easy way for someone analyzing that phi node to discover that rax#0 is an impossible option and should be ignored?

This behavior is currently working as expected. RAX#0 is not undefined, it's just the entry value. That said, there is a separate related issue we're keeping up though it refers to HLIL, specifically #2219


User can choose when to be woken up

User can choose a time slot

Created /config/wake_up_hour endpoint


refactor: introduce UPNode as common interface for UPF and gNB nodes

I worked on refactoring the SMF code, and this is a change I found very helpful to improve code reuse and readability.
For simpler management of UPNodes in the UserPlaneTopology, a new struct is added:
type UPNode struct {
	Name   string
	Type   UPNodeType
	ID     uuid.UUID
	NodeID pfcpType.NodeID
	Links  UPPath
}

The gNB and UPF structs "inherit" these fields using Golang's struct embedding (similar to inheritance).
The *UPNode pointer is given to the constructors of gNB and UPF structs (e.g., the NewUPF method).
E.g.,
type GNB struct {
	UPNode
	ANIP net.IP
}

and
type UPF struct {
	*UPNode
...
}

Additionally, both gNB and UPF structs implement the UPNodeInterface:
type UPNodeInterface interface {
	String() string
	GetName() string
	GetID() uuid.UUID
	GetType() UPNodeType
	GetLinks() UPPath
	AddLink(link UPNodeInterface) bool
	RemoveLink(link UPNodeInterface) bool
	GetNodeID() pfcpType.NodeID
	GetNodeIDString() string
}

This makes the UUID of each UPF a uuid.UUID instead of a string.
Now, the UPPath consists of UPNodes that implement the interface:
type UPPath []UPNodeInterface

This makes the code more concise and removes unnecessary empty fields from objects.
Additional smaller changes:

The MatchedSelection method is moved to the upf.go file for clarity.
Add method configToNodeIDto convert the single NodeID string from the SMF config to either IPv4, IPv6, or FQDN NodeID type.
Use existing LinksFromConfiguration method in NewUserPlaneInformation method instead of code duplication.
The UPF UUID is now a uuid.UUID and not a string.
Reformat UPFSelectionParams.String() output for better readability.
Added a bit of logging here and there, mainly traces.
Replace every occurence of UPF.NodeID.ResolveNodeIdToIp().String() and similar with a method.

Hi @LaumiH,
Do you test the functionalities with this PR? And which scenarios did you take?
I'm going to fix the linter and test this PR, and I'm wondering about the progress you made.
Thanks!

Hi @andy89923, I tested only session establishment with 1 gNB and 1 UE so far. In addition, I have tested the heartbeat procedure and UPF failure detection. I have also checked the unit tests.
Are there any test scenarios/cases that you typically do? I can help with testing.


Conversions from Rep to Shamir sharing for proving.

Pull request open in #121

Pull request open in #121

If you write fixes #62 somewhere in the other pull request (comment or commit message) it will be automagically linked


using AutoBuild with androidBuild* results in duplicated sourcegenerators (and other things)

for now, workaround by doing
sourceGenerators in Compile <<= (sourceGenerators in Compile) (g => Seq(g.last))
other things get multiply added, but are less of an issue, including:
unmanagedClasspath
managedClasspath
resolvers (for android sdk local repos)
cleanFiles

The same with 0.13.x syntax:
    sourceGenerators in Compile :=Seq((sourceGenerators in Compile).value.last),

1.4.10 changes sourceGenerators so that it properly uses each individual task. This workaround no longer applies. The proper fix is to remove the conjunction of android.AutoBuild and individual androidBuildXXX mechanisms.

AutoBuild is deprecated, not really an issue anymore


Refactoring fake data files

updated the commits from the last "update readme" pull request (had an issue with pushing)

Made a fix to the structure of the code (added parameters to connect() so that it can have a single functionality and be tested more easily)


[Question] read issue

would this work for functions without a symbol, such as those in roblox.
i am trying to hook the print function to see if it is possible, and it keeps crashing whenever i call print from c++, but not when anything else prints

ex, some way of me providing a uintptr_t where the function is at (i can add the slide before)
then i provide what i want it to be replaced with and i somehow get the original

Can you show me your code?

would excerpts of my code be fine? i intend to sell my finished product as a competitor to the existing mac roblox cheats (im making mine for arm instead of intel, so performance isnt gonna suck)

Yeah, only the part of calling tinyhook is enough.
Or you can create another simple sample code of how you want to use tinyhook, so I can find the problem.

im sorry, my c++ compiler has self destructed, how can i fix this
whatever [main●●] cmake -DCMAKE_OSX_ARCHITECTURES=arm64 .

CMake Warning at /opt/homebrew/Cellar/cmake/3.30.5/share/cmake/Modules/Platform/Darwin-Initialize.cmake:311 (message):
  Ignoring CMAKE_OSX_SYSROOT value:

   /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX14.5.sdk

  because the directory does not exist.
Call Stack (most recent call first):
  /opt/homebrew/Cellar/cmake/3.30.5/share/cmake/Modules/CMakeSystemSpecificInitialize.cmake:34 (include)
  CMakeLists.txt:3 (project)

-- The CXX compiler identification is AppleClang <IP_ADDRESS>00026
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - failed
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ - broken
CMake Error at /opt/homebrew/Cellar/cmake/3.30.5/share/cmake/Modules/CMakeTestCXXCompiler.cmake:73 (message):
  The C++ compiler

    "/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: '/Volumes/Dock/LThanine/CMakeFiles/CMakeScratch/TryCompile-W6dj7m'

    Run Build Command(s): /opt/homebrew/Cellar/cmake/3.30.5/bin/cmake -E env VERBOSE=1 /usr/bin/make -f Makefile cmTC_797cb/fast
    /Applications/Xcode.app/Contents/Developer/usr/bin/make  -f CMakeFiles/cmTC_797cb.dir/build.make CMakeFiles/cmTC_797cb.dir/build
    Building CXX object CMakeFiles/cmTC_797cb.dir/testCXXCompiler.cxx.o
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++   -arch arm64 -MD -MT CMakeFiles/cmTC_797cb.dir/testCXXCompiler.cxx.o -MF CMakeFiles/cmTC_797cb.dir/testCXXCompiler.cxx.o.d -o CMakeFiles/cmTC_797cb.dir/testCXXCompiler.cxx.o -c /Volumes/Dock/whatever/CMakeFiles/CMakeScratch/TryCompile-W6dj7m/testCXXCompiler.cxx
    Linking CXX executable cmTC_797cb
    /opt/homebrew/Cellar/cmake/3.30.5/bin/cmake -E cmake_link_script CMakeFiles/cmTC_797cb.dir/link.txt --verbose=1
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++  -arch arm64 -Wl,-search_paths_first -Wl,-headerpad_max_install_names CMakeFiles/cmTC_797cb.dir/testCXXCompiler.cxx.o -o cmTC_797cb
    ld: library 'c++' not found
    c++: error: linker command failed with exit code 1 (use -v to see invocation)
    make[1]: *** [cmTC_797cb] Error 1
    make: *** [cmTC_797cb/fast] Error 2

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:3 (project)

i fixed it, after reinstalling xcode, and uninstalling cmake via brew, then installing the official cmake application

So, is there any issue with tinyhook?

yes, i can probably send code tomorrow, but not right now. im having issues with my gaming pc
all of my tech is giving me trouble all at once it seems

files
offsets.hpp
not my actual offsets.hpp, but should work. i removed everything except print (btw its a live offset, works on current roblox)
#include <mach-o/dyld.h>
#include <cstdint>

#define extract_func(addr, sig) reinterpret_cast<sig>(_dyld_get_image_vmaddr_slide(0) + (uintptr_t)addr)
#define num uintptr_t
namespace addresses {
  num f_print = 0x10120db1c;
}
namespace signatures {
  typedef int (*f_print)(int lvl, const char* text);
}
// #define rbx_function(pname, sig, addr) sig*  ## pname = (sig*)addr;
namespace rbxf {
  // rbx_function(print, signatures::f_print, addresses::f_print);
  auto print = extract_func(addresses::f_print,signatures::f_print);
}

main.cpp
just an example of what i did
#include <tinyhook.h>
#include "./offsets.hpp"

int (*orig_print)(int lvl, const char* text);
int my_print(int lvl, const char* text) {
  printf("print hooked, text: %s\n", text);
  return orig_print(lvl,text) ;
}
void install_print_hook() {
  tiny_hook(rbxf::print,my_print,reinterpret_cast<void**>(&orig_print));
  // test
  rbxf::print(0,"is it hooked?"); // crashes
}

// probably some constructor or whatever here that causes it to load from a inserted dylib

Can you give me a link of current roblox? I don't know which one you're talking about.

just go on the roblox website and download installer (on arm mac)
that print offset i provided is outdated, here is a better one: 0x1012306b0
(when u compile the code to a dylib u have to use insert_dylib because roblox disabled dylib loading with enviroment varibles)

emm, I see.
I'm not an experienced cpp programmer, but I think it's not the issue of tinyhook.
I tried your code, and got a zero when I printed the value of rbxf::print before calling tiny_hook.
If I modified the code like this:
void install_print_hook() {
  auto print = extract_func(addresses::f_print,signatures::f_print);
  tiny_hook((void*)print,(void*)my_print,reinterpret_cast<void**>(&orig_print));
  // test
  print(0,"is it hooked?"); // crashes
}

it worked.

it printed "print hooked, text: "? or did it just compile
i tried what you did and it does not work

look

are you testing roblox wih it inserted? or are you just compiling it. it compiles fine, but crashes on runtime. (compilation was never an issue, except when cmake broke when it updated)

ohhhhh, i seeeeee. this might be a threading issue

it is now segfaulting

did you see the screenshot in my post above?
well, it could be the offset changed.
I don't think the segfaulting is related to tinyhook, maybe you should check the argument before calling tinyhook.

i can verify that the offset is correct because it prints correctly before it gets hooked
roblox only updates on wendsdays to i have to find a new offset for tomorrow

did you see the screenshot in my post above?
well, it could be the offset changed.
I don't think the segfaulting is related to tinyhook, maybe you should check the argument before calling tinyhook.

how are you inserting the dylib? are you using insert_dylib or are you resigning roblox than using enviroment varibles?

I used insert_dylib and resigned the main executable

well then, my mac is brik then

i disabled and re-enabled sip on my mac, but i kept unrestricted debugging and dtrace
then i restarted my mac
now my code works flawlessly
maybe my mac just needed a restart

yes, it works, my Mac is with sip on.

i figured it out, something is causing crashes. i dont know how to fix it other than trying random things. so far trying random things has worked


Can 61 frames pretrained weight be released?

Thank you for your work!
I find train a 61 frames weight is too long ,can you release a 61 frames weight? Thank you very much!

It take time for preparation….

I want to train for 15000 steps for 61 frames, but stopped at step 10000(err occerrd), the checkpoint-10000 folder have be saved, but unet and controlnet folder didn't be saved,  how can i run a test.py only with checkpoint-10000 folder? Or can't run a test.py without unet and controlnet folder?

see FAQ

Released! see here.


Weather station - Ultraviolet light sensor

Ultraviolet light sensor

MP8511 UV Sensor
ESPHome
Adafruit

Originally posted by @jcallaghan in https://github.com/jcallaghan/home-assistant-config/issues/13#issuecomment-624076582

Wiring with ESP32.

Using a filter to convert the ADC voltage to the UV Index.
    filters:
      - lambda: return (x/0.1);

Ever slightly impressed with this. I don't have a UV lamp to calibrate this but found the University in Manchester has a weather station and provides information to Defra.
They are reporting 2.7 UV Index while I am reporting 2.36 UV Index. This makes me feel great! 😎

[ ] I need to understand if the UV Index is reported by a daily average or the high point of the day.

UV Index sources

Manchester University weather station
The Whitworth Meteorological Observatory - Current Data
The Weather Network
Weather Online
Met Office

[ ] Check this data is available in Grafana.

Something like this in Lovelace would be pretty cool.
https://rurulalatec.com/smart lab/fuv001/UV-6.png

While I have been testing the pins have come loose a few times and skewed my data 🙁 but never mind as it will be fitted to a PCB board soon.

[ ] I need to find out the best way to ensure the sensor has visibility while also maintaining it from the elements and weatherproofing it. Right now I have it sat in a test-tube like piece of plastic.

Waterproofing UV Sensors - Fused silica

Since these sensors are incredibly cheap, what about spraying some conformal coating on the breakout board and see what happens?
Maybe that's enough to weatherproof it for a few years


Remove unused terraform variable from kubo-lbs.tf

This variable is not used within the template.
With this change, step 3 in "Step 2: Configure Load Balancers" in cfcf docs needs to be updated to no longer supply this variable to the terraform apply command (https://docs-cfcr.cfapps.io/installing/gcp/routing-gcp/#step-2-configure-load-balancers).

Thank you for catching this and making a PR @ishustava!


tests/CI fail for double mut borrow

While working on #476: after submitting the PR and then later force pushing some small changes, I noticed one of the build tests failed for a BorrowMutError. However upon my next push it re-ran and passed fine. Since my changes didn't affect anything that would be running in the tests, I figured it was unlikely to be the cause.
Now I see CI failed on latest push for the same issue.
Can't reproduce it locally, as I ran the relevant test for 10 mins in a loop locally and not 1 failure. Both times it was the Ubuntu runner.

This should be fixed here https://github.com/tursodatabase/limbo/pull/468


Kafka Kraft Mode Helm Charts

Hi Team,
We could see the latest update from the "Apache Kafka" official site as we can use "Kafka's latest version 3.3.1 Kraft mode is Production Ready."
From Confluent Kafka charts we can not able to deploy the Kubernetes cluster without zookeeper mode, Could you please share what are the configurations to be changed for cp helm charts to make it for Kraft mode.
Please let us know when will be the supported helm charts will be available?
Thanks,

These charts are no longer maintained, so there will unlikely be an update to Confluent 7.x where Kraft is available with Kafka 3.3+.


TENANT_HOST_PATTERN is not correct in the mta.yaml

Should be:
TENANT_HOST_PATTERN: "^(.*)--mt-hw-ar-app."

@doublezhuang can you explain the reason for the"--mt-hw-ar-app" change?

Removed the landscape part of TENANT_HOST_PATTERN.


Issue 617

@homu r+

:pushpin: Commit 2e99e22 has been approved by kbknapp

:hourglass: Testing commit 2e99e22 with merge 345e0e3...

:sunny: Test successful - status


Use raw_window_handle from Bevy

Having this implementation of a UIViewController which is loading and showing an ad in full screen.
@interface ViewController : UIViewController
@property(nonatomic, strong) GADInterstitialAd *interstitial;
@end

@implementation ViewController

- (void)viewDidLoad {
    [super viewDidLoad];

    GADRequest *request = [GADRequest request];
    [GADInterstitialAd loadWithAdUnitID:@"APP_ID"
                                request:request
                      completionHandler:^(GADInterstitialAd *ad, NSError *error) {
        if (error) {
            NSLog(@"Failed to load interstitial ad with error: %@", [error localizedDescription]);
            return;
        }
        NSLog(@"Interstitial initialised");
        self.interstitial = ad;
        self.interstitial.fullScreenContentDelegate = self;
        [self.interstitial presentFromRootViewController:self];
    }];
}

I need to use the UIWindow from winit to being able to present this UIViewController until is closed
 ViewController *mainViewController = [[ViewController alloc] init];
    window.rootViewController = mainViewController;
    [window makeKeyAndVisible];

where window is the UIWindow obtain from winit
Once the add is closed in the callback I can set the old UIViewController that I had created initially by winit.
- (void)adDidDismissFullScreenContent:(GADInterstitialAd *)ad {
    NSLog(@"Interstitial was dismissed.");
   window.rootViewController = oldWinitViewController;
}

Currently, my challenge stems from my lack of knowledge about winit, and I'm uncertain if pursuing this direction is practical.
Any practical example of documentation how to obtain the UIWindow and UIViewController created by winit?
So far checking raw-window-handler I implement this code in bevy
    let entity = window_query.single();
    let raw_window:&winit::window::Window = windows.get_window(entity).unwrap();
    unsafe {
        let handler = raw_window.raw_window_handle();
        match handler {
            RawWindowHandle::UiKit(ios_handler) => {
                /// Logic here
            }
           _=>()
        }
    };

But for some reason I cannot understand, even those classes are in the project because are imported, my winit::window::Window it cannot find  raw_window_handle in compilation time, even the IDE can. The same happens with
UiKit enum type, it cannot resolve it.
I don't know if there is any limit scope from Bevy or what happens there.
Regards

raw_window_handle() is a trait method. Do you have the HasRawWindowHandle trait in scope?
The RawWindowHandle enum is from a different crate. You might need to depend on it directly if winit/bevy does not re-export it.


Update patch version

Purpose

Update patch version to v0.5.1.
To-do

Content

[ ] I have read and checked the items on the review checklist.

bors r+


Get-Help -online is broken on Azure VMs

Steps to reproduce
Get an Azure VM - Server 2016
PS C:\> get-help -Online gps

Expected behavior
Opens default browser with help content
Actual behavior
get-help : Starting a browser to display online Help failed. No program or browser is associated to open the URI
https://go.microsoft.com/fwlink/?LinkID=113324.
At line:1 char:1
+ get-help -Online gps
+ ~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Get-Help], PSInvalidOperationException
    + FullyQualifiedErrorId : InvalidOperation,Microsoft.PowerShell.Commands.GetHelpCommand

Environment data

PS C:\> $PSVersionTable

Name                           Value
----                           -----
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0
PSVersion                      6.0.0-alpha
PSEdition                      Core
CLRVersion
PSRemotingProtocolVersion      2.3
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   <IP_ADDRESS>
GitCommitId                    v6.0.0-alpha.14-59-g14311052514b2626a00eafeae14978c7dac7867c

Important: PR #3051 was merged as a temporary workaround to avoid the test failure. When fixing this issue, please make sure the changes in PR #3051 is reverted.

#3051 introduced the if((-not ($skipTest)) -and $IsWindows) block in test/powershell/engine/Help/HelpSystem.OnlineHelp.Tests.ps1 because our default-browser-discovery code, which was used in Get-Help -Online, didn't work correctly on the Azure code-coverage VM.
After #3281, that code was replaced by calling into ShellExecuteEx. We need to verify if Get-Help -Online works fine now on the Azure code-coverage VM, and if it does then this if block should be removed completely.

Just created a WS2016 VM in Azure and get-help -Online gps works as expected although you do get a bunch of warning dialog boxes due to Internet Explorer Enhanced Security Configuration (which is expected).  However, if we use Server Core VMs, then I expect there to be no web browser (or GUI) available, so it depends on the type of VM we use for our nightly runs.
Closing this issue as the original issue doesn't repro.

@SteveL-MSFT do you think the if((-not ($skipTest)) -and $IsWindows) block should be removed?

cc @JamesWTruher are we using Server Core for CC?

@SteveL-MSFT that makes sense.


Update RegistryCI.jl by updating the .ci/Manifest.toml file (1.5.3)

This pull request updates RegistryCI.jl by updating the .ci/Manifest.toml file.

This is wrong.

This is wrong.

This should have updated the Manifest.1.5.3.toml file.

This should have updated the Manifest.1.5.3.toml file.


Add function to demonstrate serde integration

This function converts a message to JSON and back.

Thank you for the quick review!