Cover with customized device_class door rendered as window

Home Assistant release with the issue:

0.92.1
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Home Assistant running on Linux python virtual environment
Sonoff SV switches
Component/platform:

https://www.home-assistant.io/components/cover.mqtt/
Description of problem:
device_class of cover in customize.yaml is set to door, which is valid based on https://www.home-assistant.io/components/cover/, but rendered as a window. The front-end code only caters for device_class garage
/frontend_es5/chunk.92725aec273936c40a46.js (formatted) line 2939 has the function, it only caters for device_class garage, everything else gets rendered as the default cover
cover: function(e) {
var t = "closed" !== e.state;
switch (e.attributes.device_class) {
case "garage":
return t ? "hass:garage-open" : "hass:garage";
default:
return Object(o.a)("cover", e.state)
}
},
The above switch is missing a case for all the other valid device_class entries
Below is the default render for cover:
/frontend_es5/chunk.92725aec273936c40a46.js (formatted) line 1963:
case "cover":
return "closed" === t ? "hass:window-closed" : "hass:window-open";
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
customize.yaml:
cover.sonoff_sv_gate:
  device_class: door
  friendly_name: Gate

configuration.yaml:
cover:
  - platform: mqtt
    name: "sonoff_sv_garage"
    state_topic: "cmnd/sonoffgarage/POWER2"
    command_topic: "cmnd/sonoffgarage/POWER1"
    payload_open: "ON"
    payload_close: "ON"
    payload_stop: "ON"
    state_open: "OFF"
    state_closed: "ON"
    optimistic: false
  - platform: mqtt
    name: "sonoff_sv_gate"
    state_topic: "cmnd/sonoffgate/POWER2"
    command_topic: "cmnd/sonoffgate/POWER1"
    payload_open: "ON"
    payload_close: "ON"
    payload_stop: "ON"
    state_open: "OFF"
    state_closed: "ON"
    optimistic: false

groups.yaml:
Gate:
    view: no
    control: hidden
    icon: mdi:gate
    entities:
      - cover.sonoff_sv_gate

Traceback (if applicable):

Additional information:

Contribution welcome.

Source is here: https://github.com/home-assistant/home-assistant-polymer/blob/dev/src/common/entity/cover_icon.ts
Icons are here: http://materialdesignicons.com/

I subsequently migrated to hass.io running on a pi, this issue is stil present. I see there was a commit (https://github.com/home-assistant/home-assistant-polymer/commit/9fa8544972385caac792a8a01839308fb866a023#diff-554688b76a5dde9f1e734b5f787e0494) in the last 7 days that looks like it could address the issue, I'll upgrade in the future to confirm whether it has been fixed.


[RAC] Table typeahead captures all keyboard input, leaving none for nested Inputs

Provide a general summary of the issue here
I've got a table like this, where the inputs can't capture a single keystroke, because the table hungrily swallows them all:
import { Cell, Column, Input, Row, Table, TableBody, TableHeader } from "react-aria-components";

const names = "ABCDEFGHIJKLMNOPQRSTUVWXYZ".split("").map((letter) => ({id:letter}));

export default function Home() {
  return (
    <Table>
      <TableHeader>
        <Column isRowHeader>Name</Column>
        <Column>Value</Column>
      </TableHeader>
      <TableBody items={names}>
        {(row) => (
          <Row>
            <Cell>{row.id}</Cell>
            <Cell><Input type="text" /></Cell>
          </Row>
        )}
      </TableBody>
    </Table>
  );
}

🤔 Expected Behavior?
I'd love for the inputs to Just Work™ in a situation like this.
😯 Current Behavior
https://github.com/adobe/react-spectrum/assets/13100/c28fa89b-ba9d-4e3f-a881-ee5a7a367389
💁 Possible Solution

Is there a good reason for useTypeSelect to be using the capture phase? If it used the bubble phase, the inputs themselves could choose if/when to stop propagation.
This is a bit more blunt, but why not allow typeahead to be disabled on tables? useSelectableCollection already takes a disallowTypeAhead prop, but there's not currently a way to drill that all the way down through <Table> → useTable → useGrid → useSelectableCollection.

🔦 Context
#1214 is a 3-year-old ticket that mentions that tables swallow spaces, but somehow fails to mention that tables can also swallow all keystrokes as long as the input is matching the value of a row header.
🖥 Steps to Reproduce
https://codesandbox.io/p/sandbox/old-voice-k5n9p4
Version
"react-aria-components": "^1.1.1"
What browsers are you seeing the problem on?
Chrome
If other, please specify.
No response
What operating system are you using?
macOS
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

Supporting editable text inputs inside Table cells is a larger issue under https://github.com/adobe/react-spectrum/issues/2328
I do think useTypeSelect using a capture listener may need to be revisited, as it was originally used so that spacebar in fields like picker would be used for typeahead before being used by other handlers. (via #869)
I think we would need a better reason to allow passing disallowTypeAhead into Table, since it's a feature keyboard users may expect.

Of course tables are better with typeahead, and a grid edit mode would be amazing, but I don't see a timeline (or even a single response) on #2328, presumably because it's a big project and there's a lot to get right.
I want to benefit from the many accessibility improvements y'all have baked into tables, but when I run into a deal-breaker like this one, it forces my hand to use different tech to power my table, at which point I'll be parting ways not only with the typeahead, but also with all the other accessibility features baked into RAC tables. I think the coupling makes this decision unnecessarily all-or-nothing.
If there's one thing disallowTypeAhead has in its favor is that it's relatively cheap to implement, and it would allow me to build using react-aria now while edit mode is still baking. Even though I'd be turning one feature off, it would be a huge accessibility win for my users.

Even with disabling typeahead now, you'll still run into keyboard accessibility issues with having text fields inside grid cells, so I don't think this is a good use case for RAC Table at the moment unfortunately.


Add basic auth support to HttpZipkinSpanReporter

I would like to use Basic Auth to make my Zipkin server a little more secure and needed to implement this into the SpanReporter. This is definitely a simple solution and I see some potential for refactoring the Auth element into a class so that things like " Basic b64token" string only has to be created once. I'll update the PR if I do that.

@adriancole - any feedback on this?

sure.. here's my feedback :)
If we consider this, we should also for a sec consider what is the more spring cloud way of addressing outbound HTTP auth. This doesn't mean we don't special-case BASIC, just we should think a little.
For starters, user+password combos change, so it might be cleaner to do something like this with an interceptor. If there are other implicit http clients in the classpath of sleuth, I'd bet there's a less hard-coded, and more spring way to accomplish this.
For example, zipkin doesn't have basic auth, and if it were me, I'd probably use something like expirable API keys or OAuth token as opposed to passwords. Even feign doesn't hard-code basic auth, rather bundles an interceptor to do that.. Is there another option we could use?

ex. one option could be to make a Feign, OkHttp or otherwise reporter for those who want to customize things, using their interceptors. It could be auto-configured.
This would allow us to leave the basic (no pun intended) one simple, and not run down a slope of adding little configurations here and there for the somewhat knee-capped interface of HttpUrlConnection.

I'm not well versed in much of the Spring ecosystem. What looks like the best way to do this from an integration perspective with the Sleuth libs? Spring-cloud-sleuth-zipkin-feign or work in the sleuth-zipkin project?

@brockovercash I took another look at the problem and did sth like this: https://github.com/spring-cloud/spring-cloud-sleuth/pull/316 . I've abstracted the call to Zipkin via a RestTemplate.

Closing this one in favour of #316 . Thanks @brockovercash for the idea and feedback!!


Review readme.md and source files. Issue #5

This closes Lemoncode/redux-by-sample#5

Good stuff, proceeding to merge, pleas ask @vicmator or Rocío to update the issue and kanban status


add support for TLS 1.3

add support for TLS 1.3, this is a finalised RFC (RFC 8446) and is now supported within Firefox (Release blog)

Thanks for the PR!
I'll wait for an official Mozilla or OWASP configuration guide for supporting TLS v1.3 (with the corresponding ciphers).
Related: #27


[DEMO/DISCUSSION] possible way of solving double-scrollbar in kommander

allow style-attr to be passed in order to let consumers decide on how/where to embed a table.
i abused a story to reproduce the kommander-issue and demo it. those changed would need to be removed again of course.
i have no idea on how to do idiomatic ui-kit-stuff, so please excuse this potentially very unsophisticated approach.
⚠ this might be totally opposing the philosophy of ui-kit and everyone should use the flex-helpers to solve problems like we currently have in kommander? 🤔
before

after

after reading https://github.com/bvaughn/react-virtualized/blob/master/docs/usingAutoSizer.md#can-i-use-autosizer-within-a-flex-container i think that's an utterly bad approach.


Current visual block

How do I get the current visual block?
I was looking at https://github.com/neovim/python-client/blob/master/neovim/api/nvim.py#L394, but it does not have anything related to visual blocks.
I've grepped - nothing.
Is it possible to do this over Neovim RPC api?

Is it possible to do this over Neovim RPC api?

@povilasb yes, you can use nvim_eval("getpos(\"'<\")") and nvim_eval("getpos(\"'>\")") to get the visual marks, then get the text using those positions.
https://github.com/neovim/python-client/pull/269 will make this easier I guess.

Works like a charm! Thanks :)

< and > will get the last selection, not current.
For current v and . can be used.
Credit: https://stackoverflow.com/a/47051271/11756613


Bug: It hangs at installing Xcode command line tools

When installing on fresh M2 Mac Mini:
Installing Xcode Command Line Tools (xcode):
xcode-select: error: Unable to get active developer directory. Use `sudo xcode-select --switch path/to/Xcode.app` to set one (or see `man xcode-select`)

It seems to hang indefinitely at the Xcode step, need to disable as workaround I think.

@corneliusroemer can you share the macOS Version you are using when this issue occurred?

Latest stable: 14.6.1


Fix getFormQuestionsData undefined

Resolves #57

@blcham
I resolved the issue with 'undefined' when trying to read 'getFormQuestionsData.' However, I'm curious about how it worked previously and what update caused the SForms component to lose context.
Additionally, after successfully executing the hello-world example, the script is not appearing in the table. This might be an issue with the repository.


integration week 2

Status
PRODUCTION / DEVELOPMENT
Description
PR from integration week (Yusuke/Aki).
Related PRs
List related PRs against other branches:

branch
PR

other_pr_production
link

other_pr_master
link

Todos

[ ] Tests
[ ] Documentation

Steps to Test or Reproduce
Outline the steps to test or reproduce the PR here.
roslaunch pkg_A executable_A

The car should move.

@yk-fujii please remove this file:  ros/src/actuation/vehicles/packages/as/nodes/pacmod_interface/.pacmod_interface.cpp.swp

Sorry, I mistaked merge operation... i'll revert

@yk-fujii just out of curiosity - how did you revert the merge?


(bug) You don't pass the $notifiable to the shouldInterrupt() method ?

If I understand the docs, I should be able to add a public function shouldInterrupt($notifiable) to my class MyCoolNotification extends Notification, something like:
<?php

namespace App\Notifications;

use Illuminate\Bus\Queueable;
use Illuminate\Notifications\Notification;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Notifications\Messages\MailMessage;

class MyCoolNotification extends Notification
{
    use Queueable;

    public function shouldInterrupt($notifiable) {
        return $notifiable->my_bool_value;
    }
}

but I get an exception:  Too few arguments to function App\Notifications\MyCoolNotification::shouldInterrupt(), 0 passed in ..............\vendor\thomasjohnkane\snooze\src\Models\ScheduledNotification.php on line 95 and exactly 1 expected
in that line 95 you call: return (bool) $notification->shouldInterrupt();
I think you are missing passing the $notifiable ??

Good catch!
You're right, it's currently not passing the $notifiable object.
I'll PR a fix shortly, thanks for the report!


ADC user documentation

Draft of adc user doc

Otherwise this looks great! Good job

Thanks for your comments @susilehtola. I restructured the introduction and added a sentence about how different propagators can lead to different ADC variants. Spec factors stand for spectroscopic factors described in Section 4.8.3.
All changes are incorporated via 609dd1f

I have added a few examples of different propagators and also specified that the adc module at present just deals with the one-electron Green's function to obtain IP's and EA's.
Changes done via 71d2318https://github.com/pyscf/pyscf-doc/pull/18/commits/71d231881ca8e2a14a464e6558e0c2c542110af7

Other than that, this is starting to look good!

I made the suggested changes via f69245d


Upload Unix Binaries to Releases Page

Please upload the release binaries for Linux systems to the releases page, here.   Blog posts are referring to this, yet it's Windows-only here.  Please fix it up.  Thanks.

Sorry? Terminal is a windows project. Are you asking about WSL? That’s at https://github.com/Microsoft/WSL.


More operator overloads

Enable expressions such as 1 + df
For dotnet/spark#277
Should be a straightforward review

Closing this for now. It might require a bigger change, and I'd rather do it after I address the first round of API review feedback


flutter_ume_kit_ui 和系统的SearchBar冲突

##flutter和dart环境
Flutter 3.10.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 9cd3d0d9ff (9 days ago) • 2023-05-23 20:57:28 -0700
Engine • revision 90fa3ae28f
Tools • Dart 3.0.2 • DevTools 2.23.1
##dependencies
flutter_ume: ^1.0.1
flutter_ume_kit_ui: ^1.0.0
flutter_ume_kit_device: ^1.0.0
flutter_ume_kit_perf: ^1.0.0
flutter_ume_kit_show_code: ^1.0.0
flutter_ume_kit_console: ^1.0.0
flutter_ume_kit_dio: ^1.0.0
##错误日志
Error (Xcode): ../../.pub-cache/hosted/pub.dev/flutter_ume_kit_ui-1.1.0/lib/components/widget_detail_inspector/widget_detail_inspector.dart:202:24: Error: 'SearchBar' is imported from both 'package:flutter/src/material/search_anchor.dart' and 'package:flutter_ume_kit_ui/components/widget_detail_inspector/search_bar.dart'.

同样问题，希望尽快得到修复

同样问题，希望尽快得到修复

有解决吗？ 如何处理呢？

现在flutter_ume_kit_ui用了1.1.0也不行.


fix: Cross-module calls should execute in the context of the callee

What this PR does / why we need it

Notes for your reviewers

Mind filling out the desc for why on this change for the future?
I'll need to look at this one more in-depth, we pass around template in various places to make it easier to know what's going on. The downside there is that I'm very sure that there's something missed in this.

Ok, I checked it out. I forgot that I made sure this would be safe by having the Tpl<Type> structs which did the comixing. Because of this and NewFuncMap this should be safe.

module.Call's function comment needs to be updated to reflect this as well as a doc regen with that.


Align to gitops structure, update versions in dockerfile

With the OKD community showing interest in operators/catalogs, I wanted to align all the pipeline structures to our best practices layout, update the tools dockerfile to ensure the latest versions of golang, golangci-lint, operator-sdk, opm and kustomize.
The next step is to add an example operator ("simple-operator") to enable community users to get to terms with operator development and showcase this pipeline.

/assign @aguidirh @sherine-k @LorbusChris

@LorbusChris - thanks for the feedback, I'll make the changes as you pointed out.


system_tests_skstat: Process 4759 (name=test-router) error: did not terminate properly, required SIGABORT

https://github.com/jiridanek/skupper-router/runs/6628435597?check_suite_focus=true#step:9:3112
24: ::SkstatSslTestSslPasswordFile::test_ssl_trustfile_to_unsecured ERROR

24: E           RuntimeError: Errors during teardown:
24: E
24: E           Process 4759 (name=test-router) error: did not terminate properly, required SIGABORT
24: E           skrouterd -c test-router.conf -I /home/runner/work/skupper-router/skupper-router/skupper-router/python
24: E           /home/runner/work/skupper-router/skupper-router/skupper-router/build/tests/system_test.dir/tests/system_tests_skstat/SkstatSslTestSslPasswordFile/setUpClass/test-router-6.cmd
24: E           >>>>
24: E           <<<<

Report coredump stacktraces (if any tests failed)
Thread 3 (Thread 0x7f2f87cca700 (LWP 4761)):
#0  futex_wait_cancelable (private=<optimized out>, expected=0, futex_word=0x60a000000d6c) at ../sysdeps/nptl/futex-internal.h:183
#1  __pthread_cond_wait_common (abstime=0x0, clockid=0, mutex=0x60a000000e00, cond=0x60a000000d40) at pthread_cond_wait.c:508
#2  __pthread_cond_wait (cond=0x60a000000d40, mutex=0x60a000000e00) at pthread_cond_wait.c:647
#3  0x000055908ba1746d in sys_cond_wait (cond=0x60a000000d40, held_mutex=0x60a000000e00) at ../src/posix/threading.c:92
#4  0x000055908baea3d8 in router_core_thread (arg=0x619000198280) at ../src/router_core/router_core_thread.c:222
#5  0x000055908ba17924 in _thread_init (arg=0x603000036d00) at ../src/posix/threading.c:172
#6  0x00007f2f8f9db609 in start_thread (arg=<optimized out>) at pthread_create.c:477
#7  0x00007f2f8ebbd133 in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

Thread 2 (Thread 0x7f2f897dc700 (LWP 4760)):
#0  futex_wait_cancelable (private=<optimized out>, expected=0, futex_word=0x60a0000015a8) at ../sysdeps/nptl/futex-internal.h:183
#1  __pthread_cond_wait_common (abstime=0x0, clockid=0, mutex=0x60a000001[44](https://github.com/jiridanek/skupper-router/runs/6628435597?check_suite_focus=true#step:10:45)0, cond=0x60a000001580) at pthread_cond_wait.c:508
#2  __pthread_cond_wait (cond=0x60a000001580, mutex=0x60a000001440) at pthread_cond_wait.c:647
#3  0x000055908ba1746d in sys_cond_wait (cond=0x60a000001580, held_mutex=0x60a000001440) at ../src/posix/threading.c:92
#4  0x000055908ba3652d in _plog_thread (context=0x619000198280) at ../src/protocol_log.c:1064
#5  0x000055908ba17924 in _thread_init (arg=0x603000036cd0) at ../src/posix/threading.c:172
#6  0x00007f2f8f9db609 in start_thread (arg=<optimized out>) at pthread_create.c:477
#7  0x00007f2f8ebbd133 in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

Thread 1 (Thread 0x7f2f8e372280 (LWP 4759)):
#0  __pthread_clockjoin_ex (threadid=139842118526720, thread_return=0x0, clockid=<optimized out>, abstime=<optimized out>, block=<optimized out>) at pthread_join_common.c:1[45](https://github.com/jiridanek/skupper-router/runs/6628435597?check_suite_focus=true#step:10:46)
#1  0x000055908ba17b9c in sys_thread_join (thread=0x603000036d00) at ../src/posix/threading.c:202
#2  0x000055908bacbaa2 in qdr_core_free (core=0x619000198280) at ../src/router_core/router_core.c:141
#3  0x000055908bb580db in qd_router_free (router=0x608000000f20) at ../src/router_node.c:2096
#4  0x000055908b9a9fad in qd_dispatch_free (qd=0x60c000000040) at ../src/dispatch.c:361
#5  0x000055908bbdd64a in main_process (config_path=0x7fffe5e556af "test-router.conf", python_pkgdir=0x7fffe5e556c3 "/home/runner/work/skupper-router/skupper-router/skupper-router/python", test_hooks=false, fd=2) at ../router/src/main.c:113
#6  0x000055908bbdf62c in main (argc=5, argv=0x7fffe5e5

https://github.com/skupperproject/skupper-router/runs/6627967692?check_suite_focus=true#step:29:95
Core was generated by `skrouterd -c test_router_1.conf -I /home/runner/work/skupper-router/skupper-rou'.
Program terminated with signal SIGABRT, Aborted.
#0  __futex_abstimed_wait_common64 (private=128, cancel=true, abstime=0x0, op=265, expected=2433, futex_word=0x7f332bcc9910) at futex-internal.c:57
Downloading 0.00 MB source file /usr/src/debug/glibc-2.34-34.fc35.x86_64/nptl/futex-internal.c...
57	    return INTERNAL_SYSCALL_CANCEL (futex_time64, futex_word, op, expected,
[Current thread is 1 (Thread 0x7f3331fbdf00 (LWP 2430))]

Thread 4 (Thread 0x7f332bcc9640 (LWP 2433)):
#0  __futex_abstimed_wait_common64 (private=0, cancel=true, abstime=0x0, op=393, expected=0, futex_word=0x7b1000003ea8) at futex-internal.c:57
#1  __futex_abstimed_wait_common (futex_word=futex_word@entry=0x7b1000003ea8, expected=expected@entry=0, clockid=clockid@entry=0, abstime=abstime@entry=0x0, private=private@entry=0, cancel=cancel@entry=true) at futex-internal.c:87
#2  0x00007f33329bc9af in __GI___futex_abstimed_wait_cancelable64 (futex_word=futex_word@entry=0x7b1000003ea8, expected=expected@entry=0, clockid=clockid@entry=0, abstime=abstime@entry=0x0, private=private@entry=0) at futex-internal.c:139
#3  0x00007f33329bf140 in __pthread_cond_wait_common (abstime=0x0, clockid=0, mutex=0x7b1000003ec0, cond=0x7b1000003e80) at pthread_cond_wait.c:504
#4  ___pthread_cond_wait (cond=cond@entry=0x7b1000003e80, mutex=mutex@entry=0x7b1000003ec0) at pthread_cond_wait.c:619
#5  0x00007f3333059451 in __tsan::call_pthread_cancel_with_cleanup (fn=fn@entry=0x7f33329bef50 <___pthread_cond_wait>, c=c@entry=0x7b1000003e80, m=m@entry=0x7b1000003ec0, abstime=abstime@entry=0x0, cleanup=cleanup@entry=0x7f333300[88](https://github.com/skupperproject/skupper-router/runs/6627967692?check_suite_focus=true#step:29:89)50 <cond_mutex_unlock(CondMutexUnlockCtx*)>, arg=arg@entry=0x7f332bc884c0) at ../../../../libsanitizer/tsan/tsan_platform_linux.cpp:453
#6  0x00007f333302f4dd in cond_wait (thr=0x7f332bc[89](https://github.com/skupperproject/skupper-router/runs/6627967692?check_suite_focus=true#step:29:90)000, pc=139857875891755, si=<optimized out>, fn=0x7f33329bef50 <___pthread_cond_wait>, c=0x7b1000003e80, m=0x7b1000003ec0, t=0x0) at ../../../../libsanitizer/tsan/tsan_interceptors_posix.cpp:1166
#7  0x00007f333302f6a5 in __interceptor_pthread_cond_wait (c=c@entry=0x7b1000003e80, m=m@entry=0x7b1000003ec0) at ../../../../libsanitizer/tsan/tsan_interceptors_posix.cpp:1177
#8  0x000000000047e212 in sys_cond_wait (cond=0x7b1000003e80, held_mutex=held_mutex@entry=0x7b1000003ec0) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/posix/threading.c:92
#9  0x00000000004abb3c in router_core_thread (arg=0x7b6000090800) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/router_core/router_core_thread.c:222
#10 0x000000000047e01e in _thread_init (arg=0x7b080001daa0) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/posix/threading.c:172
#11 0x00007f3332ffe550 in __tsan_thread_start_func (arg=0x7ffc9030d4c0) at ../../../../libsanitizer/tsan/tsan_interceptors_posix.cpp:959
#12 0x00007f33329bfda2 in start_thread (arg=<optimized out>) at pthread_create.c:443
#13 0x00007f333295f9e0 in clone3 () at ../sysdeps/unix/sysv/linux/x86_64/clone3.S:81

Thread 3 (Thread 0x7f332ccca640 (LWP 2432)):
#0  __futex_abstimed_wait_common64 (private=0, cancel=true, abstime=0x0, op=393, expected=0, futex_word=0x7b1000008328) at futex-internal.c:57
#1  __futex_abstimed_wait_common (futex_word=futex_word@entry=0x7b1000008328, expected=expected@entry=0, clockid=clockid@entry=0, abstime=abstime@entry=0x0, private=private@entry=0, cancel=cancel@entry=true) at futex-internal.c:87
#2  0x00007f33329bc9af in __GI___futex_abstimed_wait_cancelable64 (futex_word=futex_word@entry=0x7b1000008328, expected=expected@entry=0, clockid=clockid@entry=0, abstime=abstime@entry=0x0, private=private@entry=0) at futex-internal.c:139
#3  0x00007f33329bf140 in __pthread_cond_wait_common (abstime=0x0, clockid=0, mutex=0x7b1000008280, cond=0x7b1000008300) at pthread_cond_wait.c:504
#4  ___pthread_cond_wait (cond=cond@entry=0x7b1000008300, mutex=mutex@entry=0x7b1000008280) at pthread_cond_wait.c:619
#5  0x00007f3333059451 in __tsan::call_pthread_cancel_with_cleanup (fn=fn@entry=0x7f33329bef50 <___pthread_cond_wait>, c=c@entry=0x7b1000008300, m=m@entry=0x7b1000008280, abstime=abstime@entry=0x0, cleanup=cleanup@entry=0x7f3333008850 <cond_mutex_unlock(CondMutexUnlockCtx*)>, arg=arg@entry=0x7f332cc89500) at ../../../../libsanitizer/tsan/tsan_platform_linux.cpp:453
#6  0x00007f333302f4dd in cond_wait (thr=0x7f332cc8a000, pc=139857875891755, si=<optimized out>, fn=0x7f33329bef50 <___pthread_cond_wait>, c=0x7b1000008300, m=0x7b1000008280, t=0x0) at ../../../../libsanitizer/tsan/tsan_interceptors_posix.cpp:1166
#7  0x00007f333302f6a5 in __interceptor_pthread_cond_wait (c=c@entry=0x7b1000008300, m=m@entry=0x7b1000008280) at ../../../../libsanitizer/tsan/tsan_interceptors_posix.cpp:1177
#8  0x000000000047e212 in sys_cond_wait (cond=0x7b1000008300, held_mutex=held_mutex@entry=0x7b1000008280) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/posix/threading.c:92
#9  0x000000000048478b in _plog_thread (context=0x7b6000090800) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/protocol_log.c:1064
#10 0x000000000047e01e in _thread_init (arg=0x7b080001da40) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/posix/threading.c:172
#11 0x00007f3332ffe550 in __tsan_thread_start_func (arg=0x7ffc9030d430) at ../../../../libsanitizer/tsan/tsan_interceptors_posix.cpp:959
#12 0x00007f33329bfda2 in start_thread (arg=<optimized out>) at pthread_create.c:443
#13 0x00007f333295f9e0 in clone3 () at ../sysdeps/unix/sysv/linux/x86_64/clone3.S:81

Thread 2 (Thread 0x7f332dccb640 (LWP 2431)):
#0  0x00007f3332a33605 in __GI___clock_nanosleep (clock_id=clock_id@entry=0, flags=flags@entry=0, req=req@entry=0x7f332dc8a5a0, rem=rem@entry=0x0) at ../sysdeps/unix/sysv/linux/clock_nanosleep.c:48
#1  0x00007f3332a381a7 in __GI___nanosleep (req=req@entry=0x7f332dc8a5a0, rem=rem@entry=0x0) at ../sysdeps/unix/sysv/linux/nanosleep.c:25
#2  0x00007f3332a65189 in usleep (useconds=<optimized out>) at ../sysdeps/posix/usleep.c:32
#3  0x00007f333303053b in __interceptor_usleep (usec=100000) at ../../../../libsanitizer/tsan/tsan_interceptors_posix.cpp:359
#4  0x00007f333305b529 in __tsan::BackgroundThread (arg=<optimized out>) at ../../../../libsanitizer/tsan/tsan_rtl.cpp:180
#5  0x00007f33329bfda2 in start_thread (arg=<optimized out>) at pthread_create.c:443
#6  0x00007f333295f9e0 in clone3 () at ../sysdeps/unix/sysv/linux/x86_64/clone3.S:81

Thread 1 (Thread 0x7f3331fbdf00 (LWP 2430)):
#0  __futex_abstimed_wait_common64 (private=128, cancel=true, abstime=0x0, op=265, expected=2433, futex_word=0x7f332bcc9910) at futex-internal.c:57
#1  __futex_abstimed_wait_common (futex_word=futex_word@entry=0x7f332bcc9910, expected=2433, clockid=clockid@entry=0, abstime=abstime@entry=0x0, private=private@entry=128, cancel=cancel@entry=true) at futex-internal.c:87
#2  0x00007f33329bc9af in __GI___futex_abstimed_wait_cancelable64 (futex_word=futex_word@entry=0x7f332bcc9910, expected=<optimized out>, clockid=clockid@entry=0, abstime=abstime@entry=0x0, private=private@entry=128) at futex-internal.c:139
#3  0x00007f33329c1873 in __pthread_clockjoin_ex (threadid=139857754887744, thread_return=0x0, clockid=0, abstime=0x0, block=<optimized out>) at pthread_join_common.c:102
#4  0x00007f333302f945 in __interceptor_pthread_join (th=0x7f332bcc9640, ret=ret@entry=0x0) at ../../../../libsanitizer/tsan/tsan_interceptors_posix.cpp:1028
#5  0x000000000047e5[90](https://github.com/skupperproject/skupper-router/runs/6627967692?check_suite_focus=true#step:29:91) in sys_thread_join (thread=0x7b080001daa0) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/posix/threading.c:202
#6  0x00000000004a90f3 in qdr_core_free (core=0x7b6000090800) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/router_core/router_core.c:141
#7  0x00000000004c4d4b in qd_router_free (router=0x7b1800000a80) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/router_node.c:20[96](https://github.com/skupperproject/skupper-router/runs/6627967692?check_suite_focus=true#step:29:97)
#8  0x0000000000460ae4 in qd_dispatch_free (qd=qd@entry=0x7b2000000000) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/dispatch.c:361
#9  0x0000000000461a73 in qd_dispatch_free (qd=qd@entry=0x7b2000000000) at /home/runner/work/skupper-router/skupper-router/skupper-router/src/dispatch.c:351
#10 0x000000000042719b in main_process (config_path=config_path@entry=0x7ffc9030f056 "test_router_1.conf", python_pkgdir=python_pkgdir@entry=0x7ffc9030f06c "/home/runner/work/skupper-router/skupper-router/skupper-router/python", test_hooks=<optimized out>, fd=fd@entry=2) at /home/runner/work/skupper-router/skupper-router/skupper-router/router/src/main.c:[113](https://github.com/skupperproject/skupper-router/runs/6627967692?check_suite_focus=true#step:29:114)
#11 0x000000000042656d in main (argc=5, argv=0x7ffc9030df38) at /home/runner/work/skupper-router/skupper-router/skupper-router/router/src/main.c:363

https://github.com/skupperproject/skupper-router/runs/6639274898?#step:29:95

Not fixed, sigaborted again on branch containing the fix above, https://github.com/skupperproject/skupper-router/runs/6698510074?#step:29:84


Finish testing for new release

Finish all of the testing for the next Beta release on June 1st. The neural network and weather API with voice implementation should be tested as needed.
Note:
Add a new function ot the testing script for manual NN startup

Ready to release files into github, creating new issue and possibly new branch to pull from


Tree: move delta

Description
Move delta to tree. This will enable changing the forest editing API to just take a delta.

I think it would make sense to move visit.ts under tree as well (and move the matching test file).

I think it would make sense to move visit.ts under tree as well (and move the matching test file).

Thats a good point. It won't block the forest change though (only the implementations of forest will need it), so I'll leave it for a separate change.


Expected results?

I get the following when evaluating on MAPS after training the model over 100k iterations.
These metrics appear to be quite low, especially the frame metrics which are 0.65/0.65/0.64 whereas the Maestro paper reports 0.90/0.95/0.81.
Is this expected?
Thanks!
                            note precision                : 0.795 ± 0.096
                            note recall                   : 0.756 ± 0.109
                            note f1                       : 0.773 ± 0.096
                            note overlap                  : 0.541 ± 0.101
               note-with-offsets precision                : 0.362 ± 0.127
               note-with-offsets recall                   : 0.345 ± 0.126
               note-with-offsets f1                       : 0.352 ± 0.125
               note-with-offsets overlap                  : 0.808 ± 0.092
              note-with-velocity precision                : 0.739 ± 0.093
              note-with-velocity recall                   : 0.704 ± 0.110
              note-with-velocity f1                       : 0.719 ± 0.096
              note-with-velocity overlap                  : 0.543 ± 0.102
  note-with-offsets-and-velocity precision                : 0.341 ± 0.123
  note-with-offsets-and-velocity recall                   : 0.325 ± 0.124
  note-with-offsets-and-velocity f1                       : 0.332 ± 0.122
  note-with-offsets-and-velocity overlap                  : 0.807 ± 0.092
                           frame f1                       : 0.636 ± 0.108
                           frame precision                : 0.649 ± 0.163
                           frame recall                   : 0.654 ± 0.102
                           frame accuracy                 : 0.475 ± 0.115
                           frame substitution_error       : 0.106 ± 0.058
                           frame miss_error               : 0.240 ± 0.108
                           frame false_alarm_error        : 0.337 ± 0.338
                           frame total_error              : 0.683 ± 0.337
                           frame chroma_precision         : 0.686 ± 0.155
                           frame chroma_recall            : 0.696 ± 0.102
                           frame chroma_accuracy          : 0.516 ± 0.106
                           frame chroma_substitution_error: 0.064 ± 0.033
                           frame chroma_miss_error        : 0.240 ± 0.108
                           frame chroma_false_alarm_error : 0.337 ± 0.338
                           frame chroma_total_error       : 0.641 ± 0.315

I have also noticed low performance on MAPS. But the performance in the MAESTRO paper to compare fairly is the second row in Table 6, the one tested on MAPS without data augmentation: 0.82/0.83/0.61.
I'm suspecting that this is partly related to #3 , although I haven't had bandwidth to verify that.
I'll be able to get back on this month before ISMIR starts.

@jongwook I'm going to try the proposed fix in #3 . Wouldn't the fairer comparison the one trained on MAESTRO not MAPS (fourth row of Table 6)?

Thanks! As far as I understand, all of their experiments are trained on MAESTRO, and Table 6 (row 1-2) shows how the model generalizes to the MAPS dataset - I wasn't able to reproduce the same level of generalizability with my implementation. When trained & tested on MAESTRO, this implementation can achieve similar performance to the row 4 of table 6.

I have also trained the model for 100k, but I got the following result. Should I predict on earlier iteration checkpoints? Or maybe something went wrong?
/home/hanju/miniconda3/envs/hanju/lib/python3.6/site-packages/mir_eval/transcription.py:167: UserWarning: Estimated notes are empty.
  warnings.warn("Estimated notes are empty.")
/home/hanju/miniconda3/envs/hanju/lib/python3.6/site-packages/mir_eval/multipitch.py:275: UserWarning: Estimate frequencies are all empty.
  warnings.warn("Estimate frequencies are all empty.")
100%|██████████████████████████████████████████████████████████████████████████████████████████████████| 60/60 [02:55<00:00,  2.24s/it]
                            note precision                : 0.000 ± 0.000
                            note recall                   : 0.000 ± 0.000
                            note f1                       : 0.000 ± 0.000
                            note overlap                  : 0.000 ± 0.000
               note-with-offsets precision                : 0.000 ± 0.000
               note-with-offsets recall                   : 0.000 ± 0.000
               note-with-offsets f1                       : 0.000 ± 0.000
               note-with-offsets overlap                  : 0.000 ± 0.000
              note-with-velocity precision                : 0.000 ± 0.000
              note-with-velocity recall                   : 0.000 ± 0.000
              note-with-velocity f1                       : 0.000 ± 0.000
              note-with-velocity overlap                  : 0.000 ± 0.000
                           frame f1                       : 0.000 ± 0.000
                           frame precision                : 0.000 ± 0.000
                           frame recall                   : 0.000 ± 0.000
                           frame accuracy                 : 0.000 ± 0.000
                           frame substitution_error       : 0.000 ± 0.000
                           frame miss_error               : 1.000 ± 0.000
                           frame false_alarm_error        : 0.000 ± 0.000
                           frame total_error              : 1.000 ± 0.000
                           frame chroma_precision         : 0.000 ± 0.000
                           frame chroma_recall            : 0.000 ± 0.000
                           frame chroma_accuracy          : 0.000 ± 0.000
                           frame chroma_substitution_error: 0.000 ± 0.000
                           frame chroma_miss_error        : 1.000 ± 0.000
                           frame chroma_false_alarm_error : 0.000 ± 0.000
                           frame chroma_total_error       : 1.000 ± 0.000

@hanjuTsai see #1

Thanks @brianc118 !
@hanjuTsai I've updated requirements.txt to install the latest mir_eval commit directly from Github.

Hi @jongwook, I have been trying to train on MAPS since yesterday, however, I am still facing the Userwarning shown above. Additionally, the metrics.json file in my case is shown to be empty. Could you kindly point out what I might be doing wrong?
The command I am using for running is:
python3 train.py with train_on="MAPS" logdir=runs/baseline iterations=10000 validation_interval=100 checkpoint_interval=100

I checked it till after 1100 iterations, but there was no change.

@justachetan Is your loss decreasing? You'll need ~100,000 iterations (ideally more) to see sensible results

I am not able to even see the loss. The metrics.json file generates is completely empty. As per my understanding, on running the above command, the metrics should get logged after every 100 iterations, right? I saw in another issue that they were able to see some results after 500 iterations, as the Userwarning about empty reference frames went away.
Hence, I was confused as to why this is happening.

train.py writes tensorboard logs from which you should be able to see the loss curves.

This is what the loss plot looks like. The metrics.json file is still empty though.

I think you'll need to let the training run for a lot more, until at least 100k. The loss will stay at ~0.2 for a short while, and decrease steadily:

BTW I don't remember saving metrics.json during training. Are you sure that file is generated by my code?

Yep, I have not made any changes to your code as of now. I assumed that it was getting generated from there.

I think you'll need to let the training run for a lot more, until at least 100k. The loss will stay at ~0.2 for a short while, and decrease steadily:

BTW I don't remember saving metrics.json during training. Are you sure that file is generated by my code?

Your loss plot does not seem to have as many fluctuations as mine. Is this while training on MAPS only?

FYI your plot contains lines from multiple tensorboard log files, hence looking messy. Also note that my plots are smoothed significantly; the dim curves in the background are the actual data points. If you train until ~100k it'll look similar.

So even in your case, while training on MAPS, the Accuracy/Recall plots for notes and frames are not available till 100k iterations?

I don't have the numbers for MAPS at hand, but it'll be generally similar. See Figure 6 of https://arxiv.org/pdf/1906.08512.pdf ; the blue baseline curve is for the MAESTRO dataset.

The plot seems to indicate that you were not getting any values on Frame F1 ot Note F1 till about 100k iterations. Could you kindly confirm if this is correct?

You'll get sensible frame/note F1 values after around 100k, as said earlier. mir_eval will stop complaining once the predictions start having some notes.

Hi @jongwook, I have read your paper ADVERSARIAL LEARNING FOR IMPROVED ONSETS AND FRAMES MUSIC TRANSCRIPTION. In your paper, you reported good experiment results in both onsets and frames and your proposed method.
I think you might solve this problem already. If so, I would like you to teach us how to solve this problem.
Thanks!


TIR JOUEUR

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Certaint types de tir du joueur pourrait detruire des morceau de decor (on récupére le(s) collider(s) touché, et on les envoi dans unne function de ChunkManager pour les detruire, et detruire en même temp la Tile et le Vector4 associé


Is there a way to avoid model parallelism in multi-gpu environment

I have a model whose embedding table takes less than 2 gb in total. I have a 4 gpu system with me. When I try to run model in multi-gpu environment, it takes 20-30 percent more time per iteration compared to when I run model on a single gpu. This should not have happened.
I am guessing that the architecture divides the embedding tables across multiple gpu, and the increased time is due to communication between these machines during butterfly shuffle. Am I right?
If yes, is there a way to copy all tables across all GPU's and make model data-parallel only.
Am I looking at this increased iteration time correctly.

In your case it would probably be better to use a single GPU. You can do so by adding --use-gpu flag to the command line and setting environment variable

CUDA_VISIBLE_DEVICES=0

With respect to using data parallelism only,
(i) The combination of model and data parallelism is designed to handle a typical scenario where large embedding tables can not be replicated due to memory constraints.
(ii) In your case, you could indeed replicate everything (including the embeddings) and avoid the butterfly shuffle, but you would still have the overhead of performing the allreduce operation, that would be required to synchronize the weights across GPUs.
(iii) We do not implement the approach (ii) in the current code. If you try it yourself, one difficulty to watch for is the splitting of the input mini-batch into sub-batches for the embedding layers. Notice that this input consists of offsets and indices corresponding to embedding lookups, rather than a tensor with first dimension being the mini-batch size (as is typical for convolution or fully connected layers).

@mnaumovfb  Thanks, I will try to implement this, will make a PR if it works.


Trying to access hex in plugin

Hi - I'm trying to access the value of the colour swatch in a custom plugin but everything in the ColorData Object seems to be private so I can't get the value. Is there any way I can do this?

There's no ColorData object in this plugin.  So sorry!


Update Fileversion for adapter and framework dlls

https://github.com/Microsoft/testfx/issues/268

Verified on TestFx.CI.Real build definition.


Fix/179

@baffelli Are these fixes about Multithreading/processing? I see unrelated commits, e.g., matplotlib. Perhaps adjust the title.

I'll rebase it. I think I forked off the wrong branch.

From: Edoardo Baldi @.>
Sent: Thursday, December 14, 2023 8:59:06 PM
To: empa-scientific-it/python-tutorial @.>
Cc: Simone Baffelli @.>; Mention @.>
Subject: Re: [empa-scientific-it/python-tutorial] Fix/179 (PR #188)
@baffellihttps://github.com/baffelli Are these fixes about Multithreading/processing? I see unrelated commits, e.g., matplotlib. Perhaps adjust the title.
—
Reply to this email directly, view it on GitHubhttps://github.com/empa-scientific-it/python-tutorial/pull/188#issuecomment-1856497805, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF7KCDJK2R4L3Q6T5VL5E33YJNLAVAVCNFSM6AAAAABAU556V2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJWGQ4TOOBQGU.
You are receiving this because you were mentioned.Message ID: @.***>


The theme got changed

There is no toggle theme button  #144  and everything is changed to light theme which is missing some elements.

Can you explain further with screenshots

Can you explain further with screenshots

there is no toggle button,

The toggle button seems to be there...

Its not present in the deployed link!  may be i am wrong,sorry

No problem, try disabling some extensions on your browser


feat:340 add invoke transaction

RPC endpoint for adding INVOKE transactions
Pull Request type
Please add the labels corresponding to the type of changes your PR introduces:

Bugfix
Feature [x]
Code style update (formatting, renaming)
Refactoring (no functional changes, no API changes)
Build-related changes
Documentation content changes
Testing
Other (please describe):

What is the current behavior?
Can only invoke transactions via extrinsic
Resolves: #340
What is the new behavior?

starknet_addInvokeTransaction

Does this introduce a breaking change?
No

LGTM apart from the small change

Oh and please update docs/starknet_features_compatibility.md too


Considering revisiting the inscription content wire format

The current content type approach is expensive and limiting the kind of content that can be done through Inscriptions.
We sometime see the content type of inscriptions being mis-qualified, because the content type is not available in the wire-format.
It'd be great to revisit this inscription content wire format, taking into accounts feedbacks from this community (ie metadata for all these JPEGs), and leveraging the work and thoughts that went into other projects.
There are a lot of fascinating ideas behind IPLD that were battle tested through IPFS and that would add a new dimension to ordinals.
Inscription content type officially supported:
https://github.com/ordinals/ord/blob/03f6867755c246cf4ea64b9d85134558ce06b9b5/src/media.rs#L19-L40

Erratum, the content type is present in the inscription text/plain;charset=utf-8.
The issue is coming from something else.

@lgalabru, I like the concepts. What do you have in mind specifically?
I think issue https://github.com/ordinals/ord/issues/1523 might have been similar in terms of both content type and metadata ideas, which @casey shot down before, as you can see in the comments.
I agree with you that both things are quite desirable. I already support arbitrary metadata in arb and would like to do something with the content types. Let's work on these.

This issue is too general.


unit test code snipped

[ ] for facts
[ ] for scenarios

https://www.codeproject.com/Articles/838842/Deploying-snippets-within-VSIX-package
https://docs.microsoft.com/en-us/visualstudio/ide/how-to-distribute-code-snippets?view=vs-2017
https://marketplace.visualstudio.com/items?itemName=MadsKristensen.SnippetPackProjectTemplate


Add plotting submodule and px2radec function

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 44.88%. Comparing base (b95ad45) to head (55c2d3a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #11   +/-   ##
=======================================
  Coverage   44.88%   44.88%
=======================================
  Files          10       10
  Lines         479      479
  Branches       54       54
=======================================
  Hits          215      215
  Misses        253      253
  Partials       11       11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix awkward formatting in Planning the Test

https://github.com/axisgroup/evaluation-toolkit/tree/master/3. Planning the Test#the-complexity-in-our-stars

This is the brain dump section rnow

did you have some feedback on the Checklist as well?

Repeat for each task uses the wrong header (should be same as others)

Opening paragraph doesn't need to be quoted, since it's original and you aren't citing anything.

Could also benefit from #23 and #24


Improve the UX for resolving

Resolved conversations weren't interactive but the text was still blue 🤔
This fixes that and makes the UI much clearer 👍

 Resolved/Unresolved states


Grey vs. Gray

I noticed the default colours include Grey, rather than Gray.
I'm British so it's nice to see (what I would call) the correct spelling of this colour, but as the spelling of the CSS default would be gray, I'm confused why this spelling isn't used. Obviously a minor issue, but I think the colours should be consistent with CSS spellings for clarity's sake.
Thoughts?

Agreed, however, this is limited by the CSS which is not managed here.  See https://github.com/Semantic-Org/Semantic-UI/issues/4779.

Thanks for clarifying @levithomason

https://github.com/Semantic-Org/Semantic-UI/issues/4779


Refactor shadowdom

This refactors tab container to make extensive use of the shadowdom.
Each commit describes the benefits and features of this refactor so it's best to read each commit.

Also, thanks for breaking up the PR so nicely!


Specify required versions of Node and npm at beginning of Guide

As an initiate to the DoneJS Quick Start Guide, I previously wrote:

I notice at https://donejs.com/SettingUp.html that specific versions of Node and npm are required by DoneJS. If still applicable, I strongly recommend that information be added to the Guide.

I now notice that the Quick Start Guide is a subsequent step to the Setting Up DoneJS guide, which does include required versions of Node and npm.  So it is a matter of ensuring the user encounters these guides in the correct order.
For that purpose (and because I stumbled over it myself), I hereby request a reference to Setting Up DoneJS as its prerequisite at the beginning of the Quick Start Guide.  Thanks.

Fixed in #799


Support for bunker with local relays

I'm trying to use bunker login using citrine but nothing happens after pasting the bunker uri
Using a public relay works
Probably related to tls
Can you add support for local relays?

Also worth noting, it won't function with nak bunker either.


Runtime out of memory error

Bug description
Cloud points are not viewed in Body tracking viewer
Steps to reproduce the behavior:

Go to body tracking viewer
Infra image appears
Run time error is printed in the command window
C:\repos\onnxruntime\onnxruntime\core\providers\cuda\cuda_call.cc:97 onnxruntime:cudacall c:\repose\onnxruntime\onnxruntime\core\providers\cuda\cuda_call.cc:91 onnxruntime::cudacall CUDS failure 2: out of memory ; GPU=9; hostname= ; expr=cudamalloc((void**)&p, size);
No cloud points of the skeleton

Screenshot:

Desktop

OS with Version: Windows 10 Enterprise 1903 18362.239
SDK Version:  0.9.0
Firmware version:
== Azure Kinect DK Firmware Tool ==
Device Serial Number:<PHONE_NUMBER>12
Current Firmware Versions:
RGB camera firmware:      1.6.98
Depth camera firmware:    1.6.70
Depth config file:        6109.7
Audio firmware:           1.6.14
Build Config:             Production
Certificate Type:         Microsoft

@Shri-Humrudha it sounds like your GPU does not have enough free memory to run body tracking. Do you have other applications open on your machine?

Hi, I got some problems like this too! I wanna to run my programs for 4 times to get joints information from four different .mvk files, however when processing the fourth video, the cmd line gave me an error information like yours and I have no idea how to solve it. If you found a solution and can share it with me, I'll be appreciate that!

@YZR821 and @Shri-Humrudha it sounds like your GPUs are running out of available GPU memory for body tracking. This could be due to opening multiple trackers or other programs that are running.

I have only one tracker application opened. It was never working for me
though.
On Thu, Aug 15, 2019, 8:12 AM Chris Edmonds<EMAIL_ADDRESS>wrote:

@YZR821 https://github.com/YZR821 and @Shri-Humrudha
https://github.com/Shri-Humrudha it sounds like your GPUs are running
out of available GPU memory for body tracking. This could be due to opening
multiple trackers or other programs that are running.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues/576?email_source=notifications&email_token=AMSR4TEHE4QX3EWUMJFF6M3QEVW7DA5CNFSM4II7EV3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4MCVZY#issuecomment-521677543,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMSR4THNCIL3ZPFSLAEFLFLQEVW7DANCNFSM4II7EV3A
.

Can you please comment on what GPU you are using?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Shri-Humrudha<EMAIL_ADDRESS>Sent: Thursday, August 15, 2019 8:47:30 AM
To: microsoft/Azure-Kinect-Sensor-SDK<EMAIL_ADDRESS>Cc: Chris Edmonds<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [microsoft/Azure-Kinect-Sensor-SDK] Runtime out of memory error (#576)
I have only one tracker application opened. It was never working for me
though.
On Thu, Aug 15, 2019, 8:12 AM Chris Edmonds<EMAIL_ADDRESS>wrote:

@YZR821 https://github.com/YZR821 and @Shri-Humrudha
https://github.com/Shri-Humrudha it sounds like your GPUs are running
out of available GPU memory for body tracking. This could be due to opening
multiple trackers or other programs that are running.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues/576?email_source=notifications&email_token=AMSR4TEHE4QX3EWUMJFF6M3QEVW7DA5CNFSM4II7EV3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4MCVZY#issuecomment-521677543,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMSR4THNCIL3ZPFSLAEFLFLQEVW7DANCNFSM4II7EV3A
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAzure-Kinect-Sensor-SDK%2Fissues%2F576%3Femail_source%3Dnotifications%26email_token%3DAJSZLA7BXI45V4UAZIUOCPTQEV3BFA5CNFSM4II7EV3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4MFZMQ%23issuecomment-521690290&data=02|01|Christopher.Edmonds%40microsoft.com|87538db8631349fdb1b208d72197e25e|72f988bf86f141af91ab2d7cd011db47|1|0|637014808536079052&sdata=7Wu2iOA5bJ6K2VkyE7wN9bLTe0fxzQZdaImxPEpn3%2BI%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJSZLA5SMTMDXNXNIT5IM2TQEV3BFANCNFSM4II7EV3A&data=02|01|Christopher.Edmonds%40microsoft.com|87538db8631349fdb1b208d72197e25e|72f988bf86f141af91ab2d7cd011db47|1|0|637014808536079052&sdata=5Z6rXbrwjYx0KlP0NanlXPTJFrz41gkW%2BsD8LpaL%2ByY%3D&reserved=0.

@cdedmonds First, let me explain my project to you: I run a piece of code for four time, and each time I create one tracker to get the joints information from a .mkv file. After finishing one .mkv file, I shutdown and destroyed the tracker, then I open another .mkv file and do the same things. However, I noticed that my  GPU memory didn't release after I shutdown and destroy the tracker, and I didn't find any other functions in the SDK to solve this problem. I'm not sure whether I explained my situation clearly, but thanks for your help!


🐛 fix(vpn): hides spinner when asked for password

Description

:tada: This PR is included in version 5.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Can't find device

Hi!
Nice tool, just what I need!
Unfortunately I can't get it to work. Trying UDIDs from the XCode "Devices and simulators" window, the only feedback I get from xcmonkey is  Can't find device 62291923-EE3D-40AF-B1B2-C99831BA6142.
The app I want to stress test is running on the simulator with the given UDID. What am I doing wrong?

Hi @JBergsee,
Thanks for reporting this!
The tool uses this command to fetch all simulators: idb list-targets --json
Could you please run it locally and share the output.
Best wishes,
Alexey

Hi,
Thank you, it lists my AppleDisplay and my connected iPad, but not any of the simulators in XCode. Why aren't they recognized?

@JBergsee, could you please share output of these commands:

xcode-select -p
gcc --version

And confirm if xcrun simctl list does show your iOS Simulators?

Thank you for helping!
Here's the output:
johan@MBP-M2-Max ~ % xcode-select -p
/Library/Developer/CommandLineTools
johan@MBP-M2-Max ~ % gcc --version
Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1)
Target: arm64-apple-darwin22.4.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin
johan@MBP-M2-Max ~ % xcrun simctl list
xcrun: error: unable to find utility "simctl", not a developer tool or in PATH
johan@MBP-M2-Max ~ % echo $PATH
/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/System/Cryptexes/App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Library/Apple/usr/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/local/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/appleinternal/bin
johan@MBP-M2-Max ~ % export PATH=$PATH:/Library/Developer/CommandLineTools/usr/bin
johan@MBP-M2-Max ~ % echo $PATH
/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/System/Cryptexes/App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Library/Apple/usr/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/local/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/bin:/var/run/com.apple.security.cryptexd/codex.system/bootstrap/usr/appleinternal/bin:/Library/Developer/CommandLineTools/usr/bin
johan@MBP-M2-Max ~ % xcrun simctl list
xcrun: error: unable to find utility "simctl", not a developer tool or in PATH

Awesome, I'm glad it is fixed now for you!
And I hope you'll enjoy playing with xcmonkey 🙂


Cannot add the ListenerOAuth2Handler connector from the architecture diagram

Description:
Cannot add the ListenerOAuth2Handler connector from the architecture diagram.
Steps to reproduce:

Open workspace with an Architecture diagram
Click more icon and add an external connector
Search "ListenerOAuth2Handler" on the marketplace
Click the "Add Connector" button

Client initialization code generation doesn't generate

RCA
The issue was the implementation didn't handle code generation for connectors that don't have a return type in the metadata.


Implement advanced search functionality

Description
Advanced search can be explained as affording search across actors and sub-actors in addition to deeply nested search in the inbox.
Additional Information

Relevant prior work from Studio


Intra-statement comment interferes with function return value (again)

swc version
$ grep swc package.json
    "@swc/cli": "0.1.18",
    "@swc/core": "1.0.49,
    "swc-loader": "0.1.6",

Input
() => {
  return (
    // Interfering comment
    foo.bar || false
  );
};

Output
'use strict';
(function() {
    return // Interfering comment
foo.bar || false;
});

Problem
The comment and newline is inserted between the return statement and the body of the return expression.

I'll fix this in a fundamental way.
The cause

Wrong transformation changes start position of span of member expression to not match with it of object of member expression


v5 extension detection (dashboard)

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR adds support for new contract features and refactors contract UI components.
Detailed summary

Added @shazow/whatsabi dependency
Updated function selectors in multiple files
Refactored contract UI components for improved feature support

The following files were skipped due to too many changes: apps/dashboard/src/contract-ui/hooks/useRouteConfig.tsx

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4303  👈
#4302
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jnsdls and the rest of your teammates on  Graphite


Use react-native-config to read the REACT_NAV_LOGGING environment variable when running in React Native

Issue
Navigation state changes aren't logged when running on native.
Cause
The conditional for enabling logging was using process.env.REACT_NAV_LOGGING which can't be set on native platforms. process.env will only ever contain NODE_ENV.
Fix
The community accepted method of using environment variables in React Native is to use react-native-config. This pull request adds react-native-config as a dependency and uses it only on native platforms, web will still use process.env.
I've also updated the docs to give some indication that this environment variable exists and needs to be set in order to enable logging.

I'm not sure this is a dependency that we want to add to react-navigation. This lib aims to be a JS-only solution to navigation for React Native. react-native-config introduces native dependencies and installation instructions that increase the barrier to entry. This native module dependency also breaks any examples that we have working with Expo.
You can pass an onNavigationStateChange prop to that navigator instead of modifying the source directly. This should resolve your problem.

Thanks for the response @matthamil, I can completely understand the reasons for not adding a native dependency like this.
Unfortunately passing an onNavigationStateChange prop isn't particularly feasible with nested navigators as you'd need to pass that prop to every navigator -- not impossible, but certainly not ideal.
Given the docs say that logging should be the default action, I think it should be enabled by default and be an opt-out (in development). Would we be able to swap process.env.REACT_NAV_LOGGING for process.env.NODE_ENV === 'development' so that logging is enabled by default.
I recall seeing some issues around some people complaining that this slows down native devices, so I could possibly simplify/reduce the data that is logged to some simple strings rather than the entire state and action objects.
Thoughts?

@levibuzolic - I think you might be misunderstanding the environment that this JS is executed in: it runs inside JavaScript Core on your phone. So, there is no process.env.NODE_ENV because there is no node. Environment variables don't make much sense in React Native because the execution environment is the user's phone -- think of the React Native environment more like a web browser tab than a node process.
As for logging being default, in my opinion, I don't think it's a good idea -- imagine that every library did this, your logs would be a waterfall of spam.

Unfortunately passing an onNavigationStateChange prop isn't particularly feasible with nested navigators as you'd need to pass that prop to every navigator -- not impossible, but certainly not ideal.

Make a wrapper for the navigators that you use and pass through the prop automatically, then use the wrapper everywhere :) You could alternatively just expose some function on react-navigation to set the logging state and perhaps the maintainers would be more likely to accept such a PR. Maybe something like this:
import { configureLogs } from 'react-navigation';

configureLogs({
  transitions: true,
  logFunction: () => {},
});

@brentvatne: I think you might be misunderstanding the environment that this JS is executed in: it runs inside JavaScript Core on your phone. So, there is no process.env.NODE_ENV because there is no node. Environment variables don't make much sense in React Native because the execution environment is the user's phone -- think of the React Native environment more like a web browser tab than a node process.

NODE_ENV does exist actually, it's the only ENV that React Native sets, this allows invariant to work correctly in development.
Looking over the issues about logging -- #360, #1466 and 1919bff719908a885610e5005a9a594e473ce7fd there seems to be significant demand for it to be off by default. Which I can completely understand.
I think @brentvatne's suggestion for a global state/configuration is worthwhile, I'll take a look at implementing something along those lines.

Thinking a little more, how do you feel about having a global onNavigationStateChange rather than something specific to configuring logging?
I feel like it's likely that users of libraries like Redux might appreciate being able to globally set onNavigationStateChange so they can also handle nested navigators.

sorry my bad, I forgot it was polyfilled here: https://github.com/facebook/react-native/blob/3a031cc93a87834c3b63f96f12a8b3cdb94d3eb6/Libraries/Core/InitializeCore.js#L85-L90
not to be confused however with the environment being anything like node.


Rename global data collection switch.

The repo tag needs to be moved so that FirebaseCore.podspec gets the updated version to fix the travis issue.

The travis issue is just a lint error because the tag hasn't been updated - I'll land this and then push the updated tags.


Download from feature group returns nothing

I can't remember if this was supposed to be fixed, is a new issue, or what but here goes.  I went to workspace and found the feature group I was interested in. Check boxes appear to be missing, btw.  I selected them all and hit download, chose txt, and got el zippo!

#1164

I'm making this "fixed" since the download button in the green action bar should work now.


Implementing HTTP/1.1 keep-alive feature on websocketpp

Regarding #10, I have implemented keep-alive feature on websocketpp.
Plus I've modified Connection:close behavior, too.

For HTTP/1.0 request or the request with Connection:close, add Connection:close header to response and terminate the connection
For error responses(4xx and 5xx), add Connection:close header and terminate the connection

http_plugin of eosio should be adjusted to set cutting timeout for idle HTTP connections using websocketpp::endpoint::set_open_handshake_timeout().

Does this defect still exist in the upstream version of websocketpp?

Is there any upstream version? I thonght it’s idle.

There’s some more patches for this. I haven’t committed on my branch yet.


wire: test flakes on Travis with "failed to cache compiled Go files"

Describe the bug
The Wire tests fail intermittently on Travis with an error like:
        wire_test.go:96: go [list -e -json -compiled -test=false -export=false -deps=true -tags=wireinject -- example.com/foo]: exit status 1: go: finding github.com/google/go-cloud v0.2.0
            go build runtime/internal/atomic: failed to cache compiled Go files

To Reproduce
This is an intermittent failure, but I've only seen it on Travis. Uncertain what the triggers are.
Expected behavior
Wire tests should pass.
/cc @vangent, who I believe hit this failure before, but I can't find evidence of it.

For anyone who might still be running into this, I eventually compiled from the tip of master - where this commit - that  includes details in errors from cache operations exists.
Afterwards, I got a new error:
go build internal/race: loading compiled Go files from cache: reading srcfiles list: cache entry not found: open /Users/username/Library/Caches/go-build/d7/d7ef4361930d6b215edf518790736afc3772a7c331e4f6e310d5a8022a736a8d-a: no such file or directory

Which suggested that the issue might be related to file permissions associated with the Cache directory at /Users/username/Library/Caches/go-build/.
After running:
chown -R $(whoami):admin /Users/username/Library/Caches/go-build

I was able to run both wire and go list -compiled=true... without any errors.

@zombiezen the Go bug was fixed for Go 1.13. This shouldn't be happening anymore. (I checked that the repro no longer fails.)

Interesting. @cespare I ran into the issue when using both Go 1.12 and Go 1.13. Perhaps the cause of the error in my case was different from what others were observing.

Excellent, thank you! I'll take a look at getting the tests running in parallel again.


support stop() with libcurl <= 7.31.0 on RedHat 7

Hi Joe,
Thanks for the PR. It looks fine but please clean it up as it now contains all the commits from your previous PR. I guess you need to update your fork master branch and resend the PR so it contains only one essential commit.


Added WebSocket Example

#442

Thanks for this! I wonder if it would be possible to transfer more of the logic to Rust though? Right now it looks like the workhorse of the example's logic is all in JS, and to show off Rust usage it might be cool to have the majority of the logic live in Rust

I'm gonna go ahead and close this, but feel free to resubmit with my coment addressed!


InvalidAuthenticityToken

This is quite strange. We have one test client computer - a Windows 64 bit machine - that cannot log in using any browser! (IE, FF, Chrome). We are getting ActionController::InvalidAuthenticityToken.
Our session_store looks like this:
MyApp::Application.config.session_store :dalli_store,
:memcache_server => ['xx'],
:namespace => 'sessions',
:key => '_myapp_r3_session', :expire_after => 30.minutes
If I remove Dalli, and just use:
MyApp::Application.config.session_store :cookie_store, :key => '_xxx-xxx-devise_session', :domain => :all
We can log in without issue.
The login page is not cached. I can see it is submitting a different authenticity token each time.
What can we do to debug this further and resolve this? For now we have had to switch off dalli
environment is Rails 3 and yes we are using Devise

i have configured dalli gem version 2.7.4
added, Rails.application.config.session_store  ActionDispatch::Session::CacheStore, :expire_after => 5.minutes, :domain => :all, tld_length: 2
Once the user logged in and  session expires(20 seconds got  over), i am receiving the below issue
ActionController::InvalidAuthenticityToken (ActionController::InvalidAuthenticityToken)
After that i cant able to login again. always it throws the same error.


Receive notifications

Should i see "Receive notifications for new episodes" option in Stremio Windows app? I see it in web.stremio.com but in Windows app i see it only for Cinemeta, not for Tmdb addon.

Stremio does not support calendar from third party addons only for Cinemeta, @jaruba told me that this will be possible in the future but it is not possible now.

Stremio doesn't support calendar functions for catalogues besides Cinemeta. Stremio devs have said that they won't implement that feature shortly.

i see, maybe the are testing it now with the web version of Stremio


Can this sniff a standard RS485 Modbus bus i.e in parallel with other devices on the bus ?

Hi i am trying to decipher the communication between a Solar Inverter and a Power Energy meter.
Can i just connect this into the bus with a (RS485 to USB connector) both of these devices and it will sniff all communication between them or is this for sniffing an application on my PC sending on to the RS485 bus ?
Craig

Hi, yes you can use a RS485 to USB adapter. This program just opens the serial port (/dev/tty*) and saves the captured data to a .pcap file, so it can be analyzed with Wireshark, splitting the packet with a timeout, i.e. it doesn't get into the logic of the protocol, because that would have meant interpreting the full protocol (and maybe the machine uses custom function codes), but it checks the CRC16 so you can see if it's capturing correctly, even if even packets with non valid CRC are included in the capture (they can be filtered with WireShark later anyway).
It works fairly well, if not you have to tweak the timeout between packet in the source code, when you get it right it should work. Also pay attention to the adapter you use, I suggest to connect only the A and B data lines to the converter, not the signal ground, since that could introduce ground loops. If you see a lot of zero in the capture you probably have to had/remove the pullup/pulldown resistors on the line (the converter may or may not have them inside).
By the way, if you want to sniff the protocol to know how the machine works, without wanting to generate pcap files, you can also use the Modscan32 program for Windows. With that program you can monitor the communication and build dashboards that monitor parameters in real time. That program works only for function codes to read/write coils and registers, not with all the function codes of the program (but this is what most Modbus devices will use anyway)

Hi @alerighi I'm trying to find the communication with a ventilation system and its controller. I tried your sniffer but my results are:
captured packet 1: length = 2, CRC: FFFF = FFFFFFFE00 [FAIL]
captured packet 2: length = 1, CRC: FFFF = FFFFFFFE30 [FAIL]
captured packet 3: length = 4, CRC: 9403 = FFFFFFBCFFFFFFA6 [FAIL]
captured packet 4: length = 11, CRC: 41BA = FFFFFFB406 [FAIL]
captured packet 5: length = 5, CRC: 33F4 = 3639 [FAIL]
captured packet 6: length = 2, CRC: FFFF = 06FFFFFFB0 [FAIL]
captured packet 7: length = 1, CRC: FFFF = 0630 [FAIL]
captured packet 8: length = 3, CRC: B23E = 0630 [FAIL]

Can you please give me some hints?

Hi, sorry if I respond late, I didn't see the message. It seems you are reading a lot of 1s: this may be caused by wrong levels on the RS485 line that 485 transreceiver logic.
You can try the following things:

double check the positive/negative polarity (don't rely that A is positive and B negative, I've seen inverted multiple times, verity with a multimeter if in doubt)
try to connect only the A and B data lines, not the ground (even indirectly trough the ground of the power cable, separate the ground of the PC than the one of the equipment, for example using a laptop)
try to change the transreceiver with a better chip
try with a transreceiver without the autosense logic (of course putting it always in receive mode)
try to add a pull up/pull down resistor on the line to 3.3V/5V and down to GND
keep your wires as short as possible and use shielded cable

You can verify with a simple terminal application (such as picocom on Linux or Putty on Windows) that you receive data that makes sense on the serial port. If you see a lot of FF there is something wrong at the physical level.

@nagyrobi This is a longshot, but did you have success with your ventilation system/controller? I'm trying to do the same thing with my Broan ERV where I'm pretty sure the communication method is Modbus, but also struggling to get useful data off the bus. Thanks!
More details here: https://community.home-assistant.io/t/venmar-vanee-broan-erv-hvac-controller-output/339770

Sniffing never succeeded properly. I gave up this and instead started discusdions with the manufacturer who in the end shared with me the modbus registers documentation.

@nagyrobi @kevin-david
Hello, I have found the timing can be pretty hard to set it right.
personally at 115200 bps i had to set the -t (--interval) at 1000 to have the right amount of data (ie cutting right at the end of each modbus frames)
I will do more testing at different speed to be sure of the timing calculation.

@Ekristoffe thanks for the info - were you sniffing a ventilator or a different device?

@Ekristoffe thanks for the info - were you sniffing a ventilator/ERV/etc. or a different device?

I have used it to sniff a lot of different devices and protocols.
As long as you get the communication parameter right and the right time too it should be fine.
I am using a Rpi2b

Closed since there is no further activity.


sui-header center aligned has no effect

sui-header center aligned doesn't seem to work (though explicitly specifying the class does). Feature Status says it's 100% supported but maybe that doesn't include "common attributes"?
Example on codesandbox:
  <sui-container>
    <sui-header as="h1" center aligned>sui-header as h1 center aligned</sui-header>
    <sui-header as="h1" class="center aligned">sui-header as h1 class="center aligned"</sui-header>
  </sui-container>

I am using the semantic-ui npm package version 2.4.2.

https://semantic-ui-vue.github.io/#/elements/header#text-alignment
It should be <h1 is="sui-header" text-align="center">

Thanks!


[improve][broker] Add optional parameters for getPartitionedStats

PIP: https://github.com/apache/pulsar/pull/21228

Verifying this change

[x] Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:
https://github.com/crossoverJie/pulsar/blob/c97cdf3d535e90cb725cbf6016cc8ecc2570ce9a/pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApiTest.java#L1050-L1130
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[x] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[x] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/crossoverJie/pulsar/pull/17

Codecov Report

Merging #21611 (c97cdf3) into master (98bf9dd) will increase coverage by 36.55%.
Report is 13 commits behind head on master.
The diff coverage is 84.70%.

Additional details and impacted files

@@              Coverage Diff              @@
##             master   #21611       +/-   ##
=============================================
+ Coverage     36.75%   73.31%   +36.55%
- Complexity    12215    32744    +20529
=============================================
  Files          1715     1893      +178
  Lines        131076   140732     +9656
  Branches      14314    15497     +1183
=============================================
+ Hits          48182   103177    +54995
+ Misses        76498    29446    -47052
- Partials       6396     8109     +1713

Flag
Coverage Δ

inttests
24.12% <34.11%> (-0.06%)
:arrow_down:

systests
24.68% <32.94%> (-0.24%)
:arrow_down:

unittests
72.65% <84.70%> (+40.90%)
:arrow_up:

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

Files
Coverage Δ

...pulsar/broker/admin/impl/PersistentTopicsBase.java
64.32% <100.00%> (+54.17%)
:arrow_up:

...pache/pulsar/broker/admin/v1/PersistentTopics.java
51.40% <100.00%> (+41.23%)
:arrow_up:

...he/pulsar/broker/admin/v2/NonPersistentTopics.java
62.67% <100.00%> (+34.89%)
:arrow_up:

...pache/pulsar/broker/admin/v2/PersistentTopics.java
80.72% <100.00%> (+67.60%)
:arrow_up:

...n/java/org/apache/pulsar/broker/service/Topic.java
36.36% <ø> (+27.27%)
:arrow_up:

...rvice/nonpersistent/NonPersistentSubscription.java
52.76% <100.00%> (+24.48%)
:arrow_up:

...ker/service/persistent/PersistentSubscription.java
76.85% <100.00%> (+26.46%)
:arrow_up:

...rg/apache/pulsar/client/admin/GetStatsOptions.java
87.50% <100.00%> (+7.50%)
:arrow_up:

...in/java/org/apache/pulsar/client/admin/Topics.java
77.50% <100.00%> (+45.00%)
:arrow_up:

.../apache/pulsar/broker/service/GetStatsOptions.java
75.00% <75.00%> (ø)

... and 3 more

... and 1454 files with indirect coverage changes


Bug in iOS 16 DevBeta1

https://github.com/google/promises/blob/46c1e6b5ac09d8f82c991061c659f67e573d425d/Sources/Promises/Promise%2BAsync.swift#L28
String can convert to NSError in iOS 16.
This most likely is a bug in iOS 16.
Hope Apple could fix this soon.

My app also worked strangely on iOS16, so I tried debugging it and the reason was the code above as well.
However, this problem is not only a problem with iOS 16, but potentially also on iOS 15 and lower versions.
Just add this code to your project and the problem will be reproduced regardless of iOS version.
extension String: Error {}
It is assumed that the problem occurred in iOS16 because the above code exists in 'SafariServices.framework' of iOS16.
I think this is an issue that needs to be resolved quickly.

@ykjchen Can we get someone to look at this? It's a pretty massive issue that breaks a ton of use cases for Promises

Thanks for reporting. It looks like this results in success being handled like an error, in the case where the promise type is String.
Is that the correct read on this?

Yes that is correct.

Thanks for reporting. It looks like this results in success being handled like an error, in the case where the promise type is String.
Is that the correct read on this?

Yes that is correct.
However, currently there is only a problem with the String type in iOS16,
If it extends to Int: Error {}, etc., it may be a problem for other types as well.
For details, refer to the information submitted to the forums.
https://developer.apple.com/forums/thread/708025

Does changing the check from:
if let error = value as? NSError {

to something like:
if type(of: error) is NSError.Type

seem like a possible solution?
(from https://stackoverflow.com/a/44698046)

I believe that does work yes.

I opened a PR for this here: https://github.com/google/promises/pull/188

Is there anything else I need to do in order to get this merged? We are holding off on pushing the next build of our app until this has been resolved

Apologies for the delay, thanks for your patience, just added a review.

No worries, just got those changes in so hopefully all is done now

Can you make a new release so we can target it in spm? @ykjchen

Sure, it's done now.


SP2-608 Agree Plan page

Adds the web page and its validation. Backend endpoint does not yet exist so some tests are left as TODO.

Seems this is not working as expected

Given I am on the View Plan page
AND Given there are Goals for Now in the Plan without Steps
When I click Agree Plan
Then the View Plan page is displayed with appropriate Errors

Is this expected to work as part of this ticket?

No, the task I'm doing at the moment will validate that there are goals and that they have steps - you shouldn't be able to get to the Agree Plan page at all without those things being true.


Bug: instance of charge_state_analysis should populate the remaining two input parameter

Add reporting of nuclides and mass_to_charge_range parameters for to the NeXus/HDF5 I/O

Read from ion. in add_standardized


Updated User Guide

Added skeleton for User Guide, hyperlinks, changed some descriptions and wrote my own relevant parts.

Codecov Report
Base: 45.17% // Head: 45.17% // No change to project coverage :thumbsup:

Coverage data is based on head (5edc64e) compared to base (79aad8a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #91   +/-   ##
=========================================
  Coverage     45.17%   45.17%
  Complexity      570      570
=========================================
  Files           145      145
  Lines          3059     3059
  Branches        396      396
=========================================
  Hits           1382     1382
  Misses         1537     1537
  Partials        140      140

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

LGT to the team!


Incorrect ScrollView behavior when setting scroll_x

After manually setting scroll_x, subsequent drag-scroll uses the wrong position.
To reproduce:

Click > button
Scroll back to the left side with drag-scroll

Code: https://gist.github.com/bionoid/0c16de49f79e8e6d3714
(I did not verify if this is the case for scroll_y also)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/1455409-incorrect-scrollview-behavior-when-setting-scroll_x?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github).

Let's track this at https://github.com/kivy/kivy/issues/1078.


Create ergocub-design-lowerbody.yml

adding ergocub-design-lowerbody repo to let AMI have access to https://github.com/icub-tech-iit/ergocub-design-lowerbody/issues/16#issuecomment-1029551294

That repo was archived and thus read-only.
I've unarchived it.


Throw errors on model deletion failures

Description
Throws errors on model deletion. Previously we were indicating failure, but returning 200 response codes. This changes that to throw exceptions with the correct response code.
Deprecates constructor for DeleteModelResponse that takes an error/result, as this result should only be returned on success.
Issues Resolved
#831
Check List

[X] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Do you also want to refactor the BWC tests for model Deletion?
https://github.com/opensearch-project/k-NN/blob/main/qa/restart-upgrade/src/test/java/org/opensearch/knn/bwc/ModelIT.java

@naveentatikonda yes, I missed that one. Will update.

Codecov Report

Merging #834 (7b00ec7) into main (f6d3d40) will decrease coverage by 0.21%.
The diff coverage is 79.16%.

:exclamation: Current head 7b00ec7 differs from pull request most recent head 87367cb. Consider uploading reports for the commit 87367cb to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #834      +/-   ##
============================================
- Coverage     85.14%   84.94%   -0.21%
+ Complexity     1079     1077       -2
============================================
  Files           151      152       +1
  Lines          4370     4383      +13
  Branches        390      389       -1
============================================
+ Hits           3721     3723       +2
- Misses          472      483      +11
  Partials        177      177

Impacted Files
Coverage Δ

...main/java/org/opensearch/knn/indices/ModelDao.java
81.01% <68.75%> (-0.73%)
:arrow_down:

...xception/DeleteModelWhenInTrainStateException.java
100.00% <100.00%> (ø)

...arch/knn/plugin/transport/DeleteModelResponse.java
81.48% <100.00%> (-18.52%)
:arrow_down:

... and 3 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Failure on windows related to: https://github.com/opensearch-project/OpenSearch/pull/7080.
Please review @naveentatikonda @martin-gaievski @navneet1v


Possible way to move file/folder

Hi,
It would be nice if we get the ability of moving file/folder to another folder

Technically its possible - you need to go to source file or folder and press "Download" for it. Then in destination simply type same-remote-name:/destination/folder. Destination must be a folder. If source is a file, then it will will be placed in destination folder with same file name (I am not yet using new rclone copyto and moveto commands). Destination folder will be created if it doesn't exist.
Not a very nice way, but it works. I couldn't figure out nice way to allow this in GUI.

You could add Cut and Paste options to the right click popup, or move files via mouse drag

+1000
It would be great to have the ability to easily move files and folders, especially for our encrypted remotes because we can't figure out which file is which so we can't move them from Amazons graphical interface.

Well actual moving will be responsibility of rclone. I'll provide only easy way to start this from GUI.
Afaik currently rclone cannot do move inside crypt remotes. It will download and upload file. Not sure if that is fixed yet or not.

Thanks for the reply @mmozeiko !

I'll provide only easy way to start this from GUI.

Yes! That's what I meant.
I haven't tested the move command on an ecnrypted remote, but I don't see any limitation for encrypted remotes in the documentation.
http://rclone.org/commands/rclone_move/

@mmozeiko
the moveto command works on crypt remotes without downloading/uploading the files

Oh right. So probably using this command on the GUI will work :)

@dewa710, excellent. I'll switch to use moveto instead of move.

You will plan to add "right click" menu to move files with same remote?

This is just to say that I too would welcome the ability to move folders (and let me add: thank you very much for rclone-browser; it is something of a godsend).

Yes, would be nice to able to move folders via the GUI on the remote.

Any update of this?

Did implementing the moveTo command on right click prove tricky or is there a backlog of other features in front of this one?

Any update on this?


Notebook fail to load on windows10 using Reinforcement Learning Dino Run.ipynb

The error was:
TypeError: Cannot read property 'length' of undefined
See the error console for details.

There was an illegal cell format in the notebook. It is fixed now. Thanks


[BUG] Reloading edit view of vocabulary

The toolbar for zoom in and zoom out is doubled. Moving  the toolbar displays another one in the background of the mx-graph component.

Why ? : The issue is due to angular life cycle hook being called twice which initilizes toolbar twice and overrides reference where only one is movable. Similarly on component destroy hook it destroys the last overriden referenced component

This piece of code was added a while ago for debugging in the app.module. This is responsible for bootsrapping the application  twice in the browser which was redunantly added hence the whole app was being instantiated twice.
platformBrowserDynamic().bootstrapModule(AppModule).then(moduleRef => {     const applicationRef = moduleRef.injector.get(ApplicationRef);     const componentRef = applicationRef.components[0];     // allows to run ng.profiler.timeChangeDetection();    enableDebugTools(componentRef); }).catch(err => window['console'].error(err));


Create Weapon Management UI for Weapon Manager

Description
As a player I want the UI to show me what weapons I have and what weapons I have equipped.
The user can equip up to 2 weapons at a time. Furthermore, they can preview minor states about each weapon that they hold. What I want is when players view their weapons on the bottom left corner of the screen ... they can view what primary weapon they have equipped. However when they press "Tab" the UI will reveal what weapons they have equipped and the states of the weapons.
Acceptance Criteria

[ ] The weapon management UI exists
[ ] The current equipped weapon is visible
[ ] The tab expansion of weapon information exists
[ ] The player can view the basic stats for the weapon they have equipped.

Implementation

Create a widget for the active weapons
When pressing tab, the weapon UI should be invoked to display the weapons basic stats
Have the UI contain its own views
Have the views be given data that it requires and not the weapon data itself

# Update to Scope
The HUD might be known as the 'inventory HUD' as this might accommodate future inventory items. Instead there will be a temporary menu called the "Quick Equip HUD" which shows all the essential information on the player's HUD without break the loop with the game and ultimately pausing the experience.
The quick display will instead only reveal essential information:

A calculation for the power of the weapon based on the different stats of the weapon (calculates the DPS off each weapon)
A configuration of 4 to 6 weapon slots (6 being probable maximum)
An organisation of UI that encircles the Player's view


Implement zeroize for RawKey

The u256_struct_zeroize is too much code duplication. Let's wait until the FFI layer is gone and then let's change RawKey to not use the u256_struct macro


Make local-to-scene resources behavior consistent in child scenes

See #64526 for background. ~Also this supersedes it.~ UPDATE: This PR complements it.
~Like the other PR, this very likely fixes- #62014.~

Needs rebase.

Thanks!


Adapt the training informant to FeAI

By @s314cy from FeAI
Bring the following changes to the training informant:
Modify the current statistics to account for the centralized way of training
Display more statistics thanks to the centralized server

Being resolved in a new issue


Inline snapshot testing output fails when URL request body expected contains carriage returns

Describe the bug
I'm converting our assertion helper that tests our API client routes generate the correct URL requests over to the new inline snapshot tool. It works well except for one test that uses my swift-url-routing-multipart extension to create a multipart request body. This intentionally contains \r\n line breaks, but these get converted to \n when written into the source file using the .raw strategy, causing the test to fail when re-run.
func testCarriageReturnInlineSnapshot() {
    let string = "This is a line\r\nAnd this is a line\r\n"
    var request = URLRequest(url: URL(string: "https://www.example.com")!)
    request.httpMethod = "POST"
    request.httpBody = string.data(using: .utf8)!
    assertInlineSnapshot(of: request, as: .raw) {
        """
        POST https://www.example.com

        This is a line
        And this is a line

        """
    }
}

Expected behavior
I'd expect the original line breaks to be preserved.
Environment

swift-snapshot-testing 1.13.0
Xcode 14.3
OS: iOS 16

Strange! The \r is definitely preserved in the syntax we're writing, so this is probably an issue of Xcode's line ending settings. Either way carriage returns are probably an important thing to verify, so I've opened #772 to address.

I've tested out the above fix and it has resolved my issue.


About new Multi GPU abstraction layer talked at Polaris tech day..

according to PCPER:
"Microsoft’s Chas Boyd was on-hand at AMD’s editor’s day for Polaris and previewed ideas that MS is implementing to help improve multi-GPU scaling. The best news surrounded a new abstraction layer for game developers to utilize for multiple GPU support that MS would be releasing on GitHub very shortly. According to Microsoft, with only “very little” code adjustment, DX12 titles should be able to implement basic multi-GPU support."
would this layer be avaiable on this repo?
also would be nice to hear also about:
"previewed ideas that MS is implementing to help improve multi-GPU scaling"
thanks..

Yes, the layer will be available in this repo.  We will also be publishing two mGPU sample projects along with it - one using the layer and one not using the layer.  We're making some final tweaks and doing additional testing and plan to post it all soon.

Will this layer be used to implement Multi-GPU support in games using UWP and the Windows Store (e.g. Gears of War: Ultimate Edition and Quantum Break)?

What will this layer provide actually? In what way?

@pritster5, the layer is shippable in UWP applications, so yes, titles in the Windows Store will be able to take advantage of the layer.  I can't say whether they will all use it or not though.  It's up to the studios producing the titles.  We believe that studios that write their own mGPU solutions tailored to their engines can outperform our general purpose library, but for studios lacking the resources to do so, this library should be a very effective resource.
@tjukic, this first iteration of the layer will focus on providing a solution for alternate frame rendering (AFR). We will be publishing some documentation with the layer explaining what it does and does not provide and how to weave it into your project. It's more information than would fit nicely into an issue thread such as this, so I'll hold off on that for now.

I have committed the MultiGPU abstraction layer and sample to the develop branch and merged to master today.  Take a look at the samples and the abstraction layer and let us know what you think.  There are additional details about the layer in the readme.md files for both the sample and the library


Locked products

Hi everyone,
Some products are locked to regular users.
Here, we have a complaint about it: https://forum.openfoodfacts.org/t/some-products-cannot-be-modified-vendor-lock/594
Do we have a way to prevent edits on such products?
Tracking issue: https://github.com/openfoodfacts/openfoodfacts-server/issues/8282

cf. https://github.com/openfoodfacts/openfoodfacts-dart/issues/727

cf. https://github.com/openfoodfacts/smooth-app/issues/5335

We could add a general warning if we have a data source from the platform (when entering an editing section ? on the product page ?)

We could add a general warning if we have a data source from the platform (when entering an editing section ? on the product page ?)

How do they deal with that on the website?

@stephanegigandet It looks like we don't have access to the "is that field provided by the producer" data through owner_fields or sources when we're dealing with images.
Are there other specific product fields to query for producer provided images?
cf. https://world.openfoodfacts.org/api/v3/product/3560070800292?fields=owner_fields,sources

@stephanegigandet For the record I've managed to "unselect" a locked image: https://fr.openfoodfacts.org/produit/7300400481588/fibres-wasa

25 novembre 2024, 11:38:56 CET - monsieurtanuki Images sélectionnées (Suppression : front_fr) - unselected image front_fr

@g123k @teolemon From off-dart 3.18 we're able to say if an image is "locked", with bool? product.isImageLocked(ImageField, OpenFoodFactsLanguage)
How are we supposed to display that information?

As it's a contributor-only feature, I suggest adding it:

The gallery, as an indicator (same as the outdated icon)

In the bottom sheet with the date (cf #5949)

In the details page, on the AppBar (the only place we have)


Using what aviram said (for reference):

I strace'd a Go binary - it simply does socket, connect, then sends data and receives data
it uses annoying primitives such as sendmsg and such but we shouldn't care if we do it like we do in tcp
socket gets created, socket calls connect -> instead of connecting to real socket, connect it to our endpoint then relay the data
need to hook recvfrom though so the address will make sense


Use enum intead of strings

Hey, nice project.
I want to suggest to use the enum values in the Parser instead of comparing strings.
You have everything in place, but you .toString() and compare it to string.
See Parser.cs line 34:
if (tok.TokenName.ToString() == "Import")

This is slow and fragile. If you have a typo in the string the compiler will not cover your back. You will only see the parser failing.
As you want to place this project as a learning experience, I suggest to promote good practices.
if (tok.TokenName == Lexer.Token.Import)

It is not even longer, has less brackets and all around a bit better 😎

Thanks for the tip, I've just updated my parser to use enums instead of strings :)

It looks much better now. 👍

Left a comment in the commit, but RightBrace got turned into RightParan and broke compilation of functions.