Question about "visualize event argument"

Volar 1.1.4 added a new setting, volar.inlayHints.eventArgumentInInlineHandlers, enabled by default. Here's the commit:
https://github.com/vuejs/language-tools/commit/274b027cf30482b95fd20ff822b66b40698cec87
The commit doesn't reference an issue; we're very interested to know more about the intent and how to leverage it best!

It was added for new users to understand these two points.

The presence of the $event variable in Inline handlers
Inline handlers are actually a callback, so it is correct that v-if type narrowing does not work

However, depending on user experience this may also be a nuisance, so just disable it in user space settings as needed.

Thanks, John; your explanation is much appreciated.

You're welcome, I'm Johnson btw. :)

For those who need more context:
Volar now proposes to add $event => to event handlers via Intellisense:

You can disable this feature in Volar’s settings or with volar.inlayHints.eventArgumentInInlineHandlers.
Johnson mentions two reasons for the addition:

Vue already exposes the variable $event in event handlers. Volar proposes adding an arrow function to make it more obvious. See https://vuejs.org/guide/essentials/event-handling.html#accessing-event-argument-in-inline-handlers

Both styles below are correct, but the first one might surprise a new user because $event doesn't exist on the component instance, only in event callbacks:
  <button @click="warn('Form cannot be submitted yet.', $event)">
    Submit
  </button>

  <button @click="$event => warn('Form cannot be submitted yet.', $event)">
    Submit
  </button>

Note that Vue compiles both styles to the same result, so you’re not losing any performance by adding the $event => wrapper:
    _createElementVNode("button", {
      onClick: _cache[0] || (_cache[0] = $event => (_ctx.warn('Form cannot be submitted yet.', $event)))
    }, " Submit "),
    _createElementVNode("button", {
      onClick: _cache[1] || (_cache[1] = $event => _ctx.warn('Form cannot be submitted yet.', $event))
    }, " Submit ")

Check for yourself in this playground (click on the JS tab to see how the template is compiled): https://sfc.vuejs.org/#eNqtkD0OwjAMha9iWUiABMleFQQLF2DN0hYDBeJEiUuFqt6dpkVMjGz+ef6s9[…]SeoNXER8zreO5SjndonLhoodKhYaltqQo2nUZXBspDGCDCdEb7rF/A3K8dfo=

Event handlers are callbacks, and types won’t be narrowed inside event handlers by a v-if. See https://github.com/vuejs/language-tools/issues/1249 and https://github.com/vuejs/language-tools/issues/269 and many, many other discussions on the subject.

In other words, assume that foo is a data property of type object or undefined:

    Test 1: {{ foo.bar }}

    Test 2 nested

    <button @click="alert(foo.bar)">
      Submit
    </button>

    <button @click="$event => alert(foo.bar)">
      Submit
    </button>

Why is this happening, and why the wrapper makes it more obvious?
The v-if and the @click handler execute at different times, with foo taking different values. In other words, the type narrowing provided by the v-if is lost by the time the callback is involved. Adding an arrow function makes it clearer that a callback is present.
Many people stumble on this one, which has nothing to do with Volar or Vue. This can be reproduced in pure TypeScript:
let foo: string | undefined;
if (foo) {
  ($event: any) => onSelect(foo) // Argument of type 'string | undefined' is not assignable to parameter of type 'string'.ts(2345)
}
function onSelect(foo: string) { }

Check for yourself in this playground:
https://www.typescriptlang.org/play?#code/DYUwLgBAZg9jBcEDOYBOBLAdgcwgHwgFdMATEKLEEgbgCh0oI[…]GxnADowJCYAJgBmABYAVjZaAF9aKGItdGUlVXUQLR0EZCKcDk4IFqA

Thanks! Just link your explanation to the hint.


GLIBC 2.38 linking error with fsanitize with nixpkgs/nixos-23.11

Describe the bug
With nixpkgs 23.11, when building a C project with automake using -fsanitize=address flags, a build error occurs involving libstdc++.
Steps To Reproduce
Steps to reproduce the behavior:

Create a C project with automake
Compile and link with -fsanitize=address

Observed behavior
...
/nix/store/1fn92b0783crypjcxvdv6ycmvi27by0j-binutils-2.40/bin/ld:
/nix/store/myw67gkgayf3s2mniij7zwd79lxy8v0k-gcc-12.3.0-lib/lib/libstdc++.so.6:
undefined reference to<EMAIL_ADDRESS>collect2: error: ld returned 1 exit status

Expected behavior
Proper linking not involving libstdc++.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.45, NixOS, 23.05 (Stoat), 23.05.20230815.bfd953b`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.5`
 - channels(user): `"home-manager-23.05.tar.gz"`
 - nixpkgs: `/nix/store/zq7ldyawbgkjhj0k0fphrds5dphm4ksy-source`

PS: This has been run using a flake running nixpkgs/nixos-23.11, despite the environment, and works properly in nixpkgs/nixos-23.05.

I'm experiencing this bug with NixOS 23.11, using Rust and libcpuid via cpuid-rs.
Currently working on using the NixOS 23.05 Flake input, so I can still work on my project, but this should be bumped for examination in prep for next NixOS release.


Dialog Editor action added to MIQ product features

Adding new button actions into miq_product_features for the dialog editor
Links

https://github.com/ManageIQ/manageiq-ui-classic/pull/1398
https://www.pivotaltracker.com/n/projects/1613907/stories/144481043

@romanblanco : why can't we reuse existing features? You are not introducing new functionality, just replacing/rewriting. Right?
Please, explain.
Othe that that, it looks good ;-)

@martinpovolny If I understand correctly, we want to keep the old dialog editor together with the new one for a while, until it's covered with tests, etc.
Once the old dialog is removed, I can remove the new features and reuse the ones we use now for the old dialog editor.
/cc @dclarizio

@martinpovolny In addition, once this is in, we will need to look at all of the fixes/enhancements that were made to the existing editor in the FINE release.  We have a separate story for that work, so we will need the old/new editors side by side for now.

Ok, cool.


Moe sync

@ronshapiro looks like travis failed in the throwingproviders :-(

Do you want to take a look? I'm not familiar at all, just the messenger :stuck_out_tongue_winking_eye:


Add Stream::poll_next

Adding poll_next to the Stream trait will simplify #125.
After a discussion with @yoshuawuyts and @withoutboats, we became confident that the Stream trait of the future will never solely rely on async fn next() and will always have to rely on fn poll_next().
This PR now makes our Stream trait implementable by end users.
I also made a few adjustments around pinning to all() and any() combinators since they take a &mut self, which implies Self: Unpin. A rule of thumb is that if a method takes a &mut self and then pins self, we have to require Self: Unpin.

bors r+


Explore using block or flex layout rather than absolute positioning

Pros:

Resize handle wouldn't need to be nested inside of a Panel

Cons:

PanelGroup would need to somehow account for resize handle size when assigning panel size

If flex layout was used, this could be delegated to the browser with flex-grow

A second nice aspect of this is that it might enable us to drop the explicit size prop requirement for PanelGroup too!


chore: upgrade charts

updated chart jx3/grafana-dashboard from 0.1.2 to 0.1.3

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign abayer
You can assign the PR to them by writing /assign @abayer in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm-kapp
df0ddb5ae5af228a9e0ed8bd54d87d4400583118
link
/test gsm-kapp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gsm-kapp
df0ddb5ae5af228a9e0ed8bd54d87d4400583118
link
/test gsm-kapp

tls
df0ddb5ae5af228a9e0ed8bd54d87d4400583118
link
/test tls

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest


Add user PlacementRule as DRPCs owner reference, to enable auto deletion of DRPC if userPl is deleted

Currently we watch and have a finalizer on the user PlacementRule and act on deletion of the user placement rule to clean up DRPC. This however does not garbage collect the DRPC as it is not deleted.
This issue is to track and improve this behavior, by adding the user PlacementRule as the owner of DRPC, such that deletion propagation can occur.

This is already done.


ESPHome 2023.12.9 "Component inaccessible due to ambiguity"

Dear Developer !
Since i would like to Test and find out to most suitable display for specific project, i went to "troubles" wih upper mentioned ST7567s models, due to lack of support in ESPHome. Im using most recent 2023.12.9 PROD version, and during compiling ill get following errors (please see below). Do we have please the option to solve it, or i have to wait for official release where should be regarding to your words this component included ?
Thank you very much !
`HARDWARE: ESP32 240MHz, 320KB RAM, 4MB Flash

toolchain-xtensa-esp32 @ 8.4.0+2021r2-patch5
Dependency Graph
|-- AsyncTCP-esphome @ 2.0.1
|-- WiFi @ 2.0.0
|-- FS @ 2.0.0
|-- Update @ 2.0.0
|-- ESPAsyncWebServer-esphome @ 3.1.0
|-- ESPmDNS @ 2.0.0
|-- noise-c @ 0.1.4
|-- Wire @ 2.0.0
|-- ArduinoJson @ 6.18.5
Compiling .pioenvs/esphome-web-95f614/src/esphome/components/st7567s/st7567s.cpp.o
Compiling .pioenvs/esphome-web-95f614/src/main.cpp.o
In file included from src/esphome/components/st7567s/st7567s.cpp:1:
src/esphome/components/st7567s/st7567s.h:15:7: warning: direct base 'esphome::PollingComponent' inaccessible in 'esphome::st7567s::ST7567S' due to ambiguity
class ST7567S : public PollingComponent, public display::DisplayBuffer, public i2c::I2CDevice {
^~~~~~~
src/esphome/components/st7567s/st7567s.cpp: In member function 'virtual void esphome::st7567s::ST7567S::setup()':
src/esphome/components/st7567s/st7567s.cpp:58:15: error: request for member 'mark_failed' is ambiguous
this->mark_failed();
^~~~~~~~~~~
In file included from src/esphome/core/color.h:3,
from src/esphome/components/display/display.h:8,
from src/esphome/components/display/display_buffer.h:6,
from src/esphome/components/st7567s/st7567s.h:3,
from src/esphome/components/st7567s/st7567s.cpp:1:
src/esphome/core/component.h:118:16: note: candidates are: 'virtual void esphome::Component::mark_failed()'
virtual void mark_failed();
^~~~~~~~~~~
src/esphome/core/component.h:118:16: note:                 'virtual void esphome::Component::mark_failed()'
src/esphome/components/st7567s/st7567s.cpp: In member function 'virtual void esphome::st7567s::ST7567S::dump_config()':
src/esphome/core/component.h:47:13: error: request for member 'get_update_interval' is ambiguous
if (this->get_update_interval() == SCHEDULER_DONT_RUN) {
^~~~~~~~~~~~~~~~~~~
src/esphome/components/st7567s/st7567s.cpp:72:5: note: in expansion of macro 'LOG_UPDATE_INTERVAL'
LOG_UPDATE_INTERVAL(this);
^~~~~~~~~~~~~~~~~~~
src/esphome/core/component.h:309:20: note: candidates are: 'virtual uint32_t esphome::PollingComponent::get_update_interval() const'
virtual uint32_t get_update_interval() const;
^~~~~~~~~~~~~~~~~~~
src/esphome/core/component.h:309:20: note:                 'virtual uint32_t esphome::PollingComponent::get_update_interval() const'
src/esphome/core/component.h:49:20: error: request for member 'get_update_interval' is ambiguous
} else if (this->get_update_interval() < 100) {
^~~~~~~~~~~~~~~~~~~
src/esphome/components/st7567s/st7567s.cpp:72:5: note: in expansion of macro 'LOG_UPDATE_INTERVAL'
LOG_UPDATE_INTERVAL(this);
^~~~~~~~~~~~~~~~~~~
src/esphome/core/component.h:309:20: note: candidates are: 'virtual uint32_t esphome::PollingComponent::get_update_interval() const'
virtual uint32_t get_update_interval() const;
^~~~~~~~~~~~~~~~~~~
src/esphome/core/component.h:309:20: note:                 'virtual uint32_t esphome::PollingComponent::get_update_interval() const'
In file included from src/esphome/components/sensor/sensor.h:3,
from src/esphome/core/application.h:16,
from src/esphome/components/st7567s/st7567s.cpp:3:
src/esphome/core/component.h:50:58: error: request for member 'get_update_interval' is ambiguous
ESP_LOGCONFIG(TAG, "  Update Interval: %.3fs", this->get_update_interval() / 1000.0f);
^~~~~~~~~~~~~~~~~~~
src/esphome/core/log.h:97:90: note: in definition of macro 'esph_log_config'
esp_log_printf_(ESPHOME_LOG_LEVEL_CONFIG, tag, LINE, ESPHOME_LOG_FORMAT(format), ##VA_ARGS)
^~~~~~~~~~~
src/esphome/core/component.h:50:5: note: in expansion of macro 'ESP_LOGCONFIG'
ESP_LOGCONFIG(TAG, "  Update Interval: %.3fs", this->get_update_interval() / 1000.0f);
^~~~~~~~~~~~~
src/esphome/components/st7567s/st7567s.cpp:72:5: note: in expansion of macro 'LOG_UPDATE_INTERVAL'
LOG_UPDATE_INTERVAL(this);
^~~~~~~~~~~~~~~~~~~
In file included from src/esphome/core/color.h:3,
from src/esphome/components/display/display.h:8,
from src/esphome/components/display/display_buffer.h:6,
from src/esphome/components/st7567s/st7567s.h:3,
from src/esphome/components/st7567s/st7567s.cpp:1:
src/esphome/core/component.h:309:20: note: candidates are: 'virtual uint32_t esphome::PollingComponent::get_update_interval() const'
virtual uint32_t get_update_interval() const;
^~~~~~~~~~~~~~~~~~~
src/esphome/core/component.h:309:20: note:                 'virtual uint32_t esphome::PollingComponent::get_update_interval() const'
In file included from src/esphome/components/sensor/sensor.h:3,
from src/esphome/core/application.h:16,
from src/esphome/components/st7567s/st7567s.cpp:3:
src/esphome/core/component.h:52:58: error: request for member 'get_update_interval' is ambiguous
ESP_LOGCONFIG(TAG, "  Update Interval: %.1fs", this->get_update_interval() / 1000.0f);
^~~~~~~~~~~~~~~~~~~
src/esphome/core/log.h:97:90: note: in definition of macro 'esph_log_config'
esp_log_printf_(ESPHOME_LOG_LEVEL_CONFIG, tag, LINE, ESPHOME_LOG_FORMAT(format), ##VA_ARGS)
^~~~~~~~~~~
src/esphome/core/component.h:52:5: note: in expansion of macro 'ESP_LOGCONFIG'
ESP_LOGCONFIG(TAG, "  Update Interval: %.1fs", this->get_update_interval() / 1000.0f);
^~~~~~~~~~~~~
src/esphome/components/st7567s/st7567s.cpp:72:5: note: in expansion of macro 'LOG_UPDATE_INTERVAL'
LOG_UPDATE_INTERVAL(this);
^~~~~~~~~~~~~~~~~~~
In file included from src/esphome/core/color.h:3,
from src/esphome/components/display/display.h:8,
from src/esphome/components/display/display_buffer.h:6,
from src/esphome/components/st7567s/st7567s.h:3,
from src/esphome/components/st7567s/st7567s.cpp:1:
src/esphome/core/component.h:309:20: note: candidates are: 'virtual uint32_t esphome::PollingComponent::get_update_interval() const'
virtual uint32_t get_update_interval() const;
^~~~~~~~~~~~~~~~~~~
src/esphome/core/component.h:309:20: note:                 'virtual uint32_t esphome::PollingComponent::get_update_interval() const'
*** [.pioenvs/esphome-web-95f614/src/esphome/components/st7567s/st7567s.cpp.o] Error 1
In file included from src/esphome.h:59,
from src/main.cpp:3:
src/esphome/components/st7567s/st7567s.h:15:7: warning: direct base 'esphome::PollingComponent' inaccessible in 'esphome::st7567s::ST7567S' due to ambiguity
class ST7567S : public PollingComponent, public display::DisplayBuffer, public i2c::I2CDevice {
^~~~~~~
: In function 'void setup()':
:121:23: error: request for member 'set_update_interval' is ambiguous
In file included from src/esphome/components/api/proto.h:3,
from src/esphome/components/api/api_pb2.h:5,
from src/esphome/components/api/api_connection.h:4,
from src/esphome.h:3,
from src/main.cpp:3:
src/esphome/core/component.h:298:16: note: candidates are: 'virtual void esphome::PollingComponent::set_update_interval(uint32_t)'
virtual void set_update_interval(uint32_t update_interval);
^~~~~~~~~~~~~~~~~~~
src/esphome/core/component.h:298:16: note:                 'virtual void esphome::PollingComponent::set_update_interval(uint32_t)'
:122:23: error: request for member 'set_component_source' is ambiguous
In file included from src/esphome/components/api/proto.h:3,
from src/esphome/components/api/api_pb2.h:5,
from src/esphome/components/api/api_connection.h:4,
from src/esphome.h:3,
from src/main.cpp:3:
src/esphome/core/component.h:148:8: note: candidates are: 'void esphome::Component::set_component_source(const char*)'
void set_component_source(const char source) { component_source_ = source; }
^~~~~~~~~~~~~~~~~~~~
src/esphome/core/component.h:148:8: note:                 'void esphome::Component::set_component_source(const char)'
In file included from src/esphome/components/api/api_connection.h:7,
from src/esphome.h:3,
from src/main.cpp:3:
src/esphome/core/application.h: In instantiation of 'C* esphome::Application::register_component(C*) [with C = esphome::st7567s::ST7567S]':
:123:44:   required from here
src/esphome/core/application.h:149:31: error: 'esphome::Component' is an ambiguous base of 'esphome::st7567s::ST7567S'
this->register_component_((Component *) c);
^~~~~~~~~~~~~~~
*** [.pioenvs/esphome-web-95f614/src/main.cpp.o] Error 1
========================== [FAILED] Took 6.31 seconds ==========================`

please share your config

this component might have been included into your esphome version, so please check if its there. remove external_component section

Thank you. Config here 👍
`esphome:
name: esphome-web-95f614
friendly_name: ESP32 Test Board
external_components:
source: github://latonita/esphome-st7567s-display@main
components: [st7567s]
esp32:
board: esp32dev
framework:
type: arduino
Enable logging
logger:
Enable Home Assistant API
api:
encryption:
key: "b1BUVx6kGwYXjO11xhnVW6Favs3OFnX5V86cNhIbaYg="
ota:
wifi:
ssid: !secret wifi_ssid
password: !secret wifi_password
web_server:
dallas:

pin: GPIO19
update_interval: 5s

sensor:

platform: dallas
address: 0xfc3c77e381c59e28
name: "ESP32_Test_Board_Temperature"
id: internal_temperature

switch:

platform: gpio
pin: GPIO18
id: internal_heating_relay
name: "ESP32_Test_Board_Relay"
inverted: true

binary_sensor:

platform: gpio
pin:
number: GPIO23
mode:
input: true
pullup: true
name: "ESP32_Test_Board_PIR"
id: internal_pir
on_state:
then:
- if:
condition:
binary_sensor.is_on: internal_pir
then:
- display.page.show: page2
else:
- display.page.show: page1

i2c:

sda: GPIO21
scl: GPIO22
scan: true
frequency: 800kHz
id: bus_1

font:

file: "gfonts://Calibri"
id: calibri
size: 15

display:

platform: st7567s
i2c_id: bus_1
id: internal_display
address: 0x3F
pages:

id: page3
lambda: |-
id: page4
lambda: |-
it.printf(8, 3, id(calibri), "Temperature: %.0f°C", id(internal_temperature).state);`

this component might have been included into your esphome version, so please check if its there. remove external_component section

Thank you for your reply !
Ive tested

platform: st7567s
ill get "Platform not found" message sadly. Do you have maybe idea in which platform should be this display included ? I checked some changelogs for latest releases, but found nothing :(

apologies for confusion. it is in Development branch for some time already.

No need to apologies, thank you for your response ! But does it mean please, that i have to switch to deve, no change to use your code under Prod as custom component ? No workaround for that please ?
Thank you !

I will check later today with current public/non-dev brach
BR, Anton

From: ronnieSVK @.>
Sent: Monday, February 5, 2024 1:35:54 PM
To: latonita/esphome-st7567s-display @.>
Cc: Anton Viktorov @.>; Comment @.>
Subject: Re: [latonita/esphome-st7567s-display] ESPHome 2023.12.9 "Component inaccessible due to ambiguity" (Issue #1)
No need to apologies, thank you for your response ! But does it mean please, that i have to switch to deve, no change to use your code under Prod as custom component ? No workaround for that please ?
Thank you !
—
Reply to this email directly, view it on GitHubhttps://github.com/latonita/esphome-st7567s-display/issues/1#issuecomment-1926899489, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA6O66SLU2LISGGXZPM5NLLYSDG2VAVCNFSM6AAAAABCYIB7ZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRWHA4TSNBYHE.
You are receiving this because you commented.Message ID: @.***>

@ronnieSVK okkaay, i backported dev version to be compatible with current esphome release.
This repo is updated.
Quick way to happiness:
external_components:
  - source: github://latonita/esphome-st7567s-display@main
    components: [st7567_base, st7567_i2c]

i2c:
#...

display:
  - platform: st7567_i2c
    id: lcd
    address: 0x3f
    update_interval: 500ms
    lambda: |-
      it.line(0,0,10,0); it.line(0,0,0,10);
      it.line(127,63,117,63); it.line(127,63,127,53);

Dear Anton,
Thank you very much for your work, time and effort ! Your custom component works now also in Prod without any issues ! The code was immediately valid and the display working. Time to continue testing :)
Issue solved !
Thank you very much once again,
Greetings
Ronnie


Dependencies for Python 3.7 on Debian 10 Buster

I am using a Debian 10 Buster system (on an ARM embedded system), which has Python 3.7 installed.
I am also using attrs and cattrs from PyPi.
$ pip3 freeze | grep attr
attrs==22.1.0
cattrs==22.1.0

When I install cattrs-extras, version 0.1.1 is installed instead of the latest 0.2.0.  I presume that's due to the Python 3.8 dependency for 0.2.0.  Correct?
That in turn causes my cattrs to be downgraded to 1.10.0 !!
$ pip freeze | grep attr
attrs==22.1.0
cattrs==1.10.0
cattrs-extras==0.1.1

According to PyPi, cattrs-extras requires cattrs<2.0.0, but cattrs has moved to the CalVer version numbering scheme, which is now at 22.1.0
https://libraries.io/pypi/cattrs-extras
Is there a problem with the dependencies for cattrs ?

When I install cattrs-extras, version 0.1.1 is installed instead of the latest 0.2.0. I presume that's due to the Python 3.8 dependency for 0.2.0. Correct?
Exactly. I decided to drop 3.7 support since it's close to EOL and missing lots of cool language features. I'll check if 0.2.0 has everything ok with dependencies tho.

Mmmm, not my decision, but 10 months until EOL is not really that "close".  It will live on years after EOL (at least in major distributions).
Don't get me wrong.  I am a big fan of using the latest and greatest.  I wish Python 3.11 was here now - though it's not far off 😉
However, some older code bases still need to be maintained.  Would you believe I'm actually in the process of migrating a Python 2.7 code base to Python 3.7 🤣  It's a big step.  I wish I could go straight to Python 3.11 (Debian 12 Bookworm - not released) or Python 3.9 (Debian 11 Bullseye).
Anyway, it is what it is, and it looks like using updated cattrs (from what I was initially using with Python 2.7) is behaving nicely that I might not need the magic code I borrowed from Stack Overflow, which was a minimal port of the cattrs-extras Converter class to Python 2.7.
Cheers 😄


Cannot allocate memory mmap failed for size<PHONE_NUMBER> at offset 0

when i run it on raspberry pi (ram 4gb），i got this:
Loading model...
Traceback (most recent call last):
File "examples/demo-server.py", line 10, in
import beamdecode
File "/home/pi/masr/examples/../beamdecode.py", line 29, in
blank_index,
File "/home/pi/.local/lib/python3.7/site-packages/ctcdecode/init.py", line 18, in init
self._num_labels)
RuntimeError: third_party/kenlm/util/mmap.cc:122 in void* util::MapOrThrow(std::size_t, bool, int, bool, int, uint64_t) threw ErrnoException because `(ret = mmap(__null, size, protect, flags, fd, offset)) == ((void *) -1)'.
Cannot allocate memory mmap failed for size<PHONE_NUMBER> at offset 0

Have a same problem! Hoping a solution


Added support for Arch Linux and refined Unknown distro option

Added support for Arch Linux and rejigged the Unknown distro option to first, check for apt-get in case its a Debian variant but then failing if apt-get cannot be found.

@kurobeats  Can you re-submit this to the (preferred) 2.0_beta branch or dev branch?


DO NOT MERGE testing CI

@vvbandeira @maliberty Should fail...

The integration test from ORFS would have caught this...


Version change to 0.3.0

This is so we can publish the commit that removes the time crate.

This change is

r? @Fraser999
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @dirvine

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Hide buttons in discord (like the gift button)

Is your feature request related to a problem? Please describe.
Can we add an option to hide the gift button for example ?
Describe the solution you'd like
A CSS modification or a button to tick, in order to hide the gift button
Describe alternatives you've considered
CSS modifications.

I've made a personal CSS for WebCord to actually implement this for myself. Moreover, some buttons should hide when WebCord detects it takes too much space in order to improve the readability of smaller window sizes. But since there's no UI for CSS modifications, you would have to find a special directory (its documented in Features.md) to actually be able to inject the CSS. This feature is considered as stable and is quite a complex way to inject CSS, replace Discord stylesheet and do it without any risk of being detected by Discord's internal scripts (e.g. via MutationObserver) unlike it is done with the other solutions (in most Discord mods or browser extensions, CSS is injected using <style> HTML element and that could trigger the MutationObserver if Discord would develop a code using this native class to prevent a specific modifications done to its site.

Nice ! Could you share that file please ? Because i tried to follow the Features.md instructions, yesterday, but it didn't work...

/**
 * @name Nitro removal theme
 * @description Removes all "Nitro" references from Discord.
 * @author SpacingBat3<EMAIL_ADDRESS> * @version 1.0.0
 */

div[aria-controls="discord-nitro-tab"],
a[href="/store"],
div[class|="buttons"] > button {
  display:none
}

Save it as *.theme.css (where * means any file name). Also remember that if you're on Windows, a file extension might be hidden and creating a new file in notepad might in reality result in creating a .txt file (i.e. *.theme.css.txt) instead (which is ignored by WebCord).

Closed as it is not planed and everyone can fix it with just custom CSS injection.


Can not type Korean in Linux

OS: Ubuntu Linux 16.04.1 LTS 64-bit
Hyper: hyper-1.3.1-x86_64-linux.AppImage
Issue
(I didn't install this for test.)
My issue is that i can not type korean. Here is a image.

Check right-top, as you can see i was typing korean. But i can't type any koreans whereas bash terminal can.
How to solve this issue? If Hyper doesn't support Korean would please do that?

let's track korean in #410


Flaky test: TestAdam

https://jenkins.preferred.jp/job/chainer/job/chainer_pr/1490/TEST=chainer-py3,label=mn1-p100/
20:51:03 _ TestAdam_param_3_{adabound=False, amsgrad=False, dtype=float16, loss_scaling='static', use_placeholder=True}.test_linear_model_multi_gpu _
20:51:03 chainer/testing/parameterized.py:89: in new_method
20:51:03     utils._raise_from(e.__class__, s.getvalue(), e)
20:51:03 chainer/utils/__init__.py:106: in _raise_from
20:51:03     six.raise_from(new_exc.with_traceback(orig_exc.__traceback__), None)
20:51:03 chainer/testing/parameterized.py:78: in new_method
20:51:03     return base_method(self, *args, **kwargs)
20:51:03 chainer/testing/condition.py:80: in wrapper
20:51:03     fail()
20:51:03 chainer/testing/condition.py:51: in fail
20:51:03     instance.fail(msg)
20:51:03 E   AssertionError: Parameterized test failed.
20:51:03 E
20:51:03 E   Base test method: TestAdam.test_linear_model_multi_gpu
20:51:03 E   Test parameters:
20:51:03 E     adabound: False
20:51:03 E     amsgrad: False
20:51:03 E     dtype: <class 'numpy.float16'>
20:51:03 E     loss_scaling: static
20:51:03 E     use_placeholder: True
20:51:03 E
20:51:03 E
20:51:03 E   (caused by)
20:51:03 E   AssertionError:
20:51:03 E   Fail: 10, Success: 0
20:51:03 E
20:51:03 E   The first error message:
20:51:03 E   Traceback (most recent call last):
20:51:03 E     File "/work/chainer/chainer/testing/condition.py", line 60, in <lambda>
20:51:03 E       lambda: f(ins, *args[1:], **kwargs),
20:51:03 E     File "/work/chainer/tests/chainer_tests/optimizers_tests/test_optimizers_by_linear_model.py", line 168, in test_linear_model_multi_gpu
20:51:03 E       self.assertGreater(cuda.to_cpu(accuracy.data), 0.9)
20:51:03 E   AssertionError: array(0.5, dtype=float16) not greater than 0.9

Seems that the error is related to float16, I changed the @condition.retry(10) to `@condition.retry(1). This allowed me to reproduce the failures in my environment and it only fails when float16 is used.
Maybe there is the need to use mixed-precission inside the classifier?

FYI, I observed this error yesterday twice in a row.
Before that I have never seen it. It could be related to some recent change.


fix(Global Nav): Fixed missing / in various links for center

this is specific to IE for some reason

This fixes #109.
:+1:

@arw180, you have been selected :smiling_imp:

Didn't do a full test but :+1:


Investigate Leela for Javascript

6 years old, but maybe... https://github.com/frpays/lc0-js

The JS port is old and slow. Integration with the local memory and GPU is spotty in browsers. It makes a lot more sense to focus on external engines and use Leela there.


Invalid token please try again

I installed the codeium.vim , and the service has been downloaded normally. In neovim, I executed the Codeium Auth command to open the window to get the token. Paste the token, but it prompts Invalid token, please try again

Same here
After paste the token to nvim, it shows error:
Error detected while processing function codeium#command#Command[6]..4:
line   48:
E474: Attempt to decode a blank string
E474: Failed to parse

@LintaoAmons I solved this problem manually, first open the codeium.vim installation directory, open the autoload/codeium/command.vim file, locate line 95, find the request interface, , use postman Or other interface debugging tools, as shown in the figure below, get the api_key and name, paste it into command.vim

@LintaoAmons I solved this problem manually, first open the codeium.vim installation directory, open the autoload/codeium/command.vim file, locate line 95, find the request interface, (image removed), use postman Or other interface debugging tools, as shown in the figure below, get the api_key and name, paste it into command.vim (image removed)

@zmy20220416 apologies but I've removed your comment since it seems to leak some of your info (API key, etc.). I've preserved the text above.

@pqn Thanks

@zmy20220416 I've added some logging in 64c886dcd6dfaf64685b383fe40cd567c7be3dac. If you temporarily remove ~/.codeium/config.json and try the Auth command again, do you see any message?
@LintaoAmons which commit are you on? Do you know precisely which line is failing with that error?

@pqn

Navigating to https://www.codeium.com/profile?response_type=token&redirect_uri=vim-show-auth-token&state=6D8D4F0F-1DDA-41F3-ABDD-28D1*****6&scope=openid%20profile%20email&redirect_parameters_type=query
Error detected while processing function codeium#command#Command[6]..4:
line   43:
E474: Attempt to decode a blank string
E474: Failed to parse
line   44:
E896: Argument of get() must be a List, Dictionary or Blob
Unexpected response:
line   43:
E474: Attempt to decode a blank string
E474: Failed to parse
line   44:
E896: Argument of get() must be a List, Dictionary or Blob
Unexpected response:
line   43:
E474: Attempt to decode a blank string
E474: Failed to parse
line   44:
E896: Argument of get() must be a List, Dictionary or Blob
Unexpected response:

@LintaoAmons thank you, it looks like we aren't logging the error that curl is probably having due to a missing flag. I've updated it in the latest commit (aa2fb0032a340cc2ffb121a18ede4b880f9c9f35), let me know if you see an error message now.

I too am struggling with similar issues. When i pate my Auth Token i am getting an error:
*Error detected while processing function codeium#command#Command[6]..7:
line   42:
E730: Using a List as a String

@korgoth I've merged<PHONE_NUMBER>695b0b0fbd0d3f0cf0d0fe115ca658 so try it again and let me know what you see.

Seems fine right now. But it seem not recognise the filetype for now? When I was in bash file, it gives me some js code.

Seems fine right now. But it seem not recognise the filetype for now? When I was in bash file, it gives me some js code.

Great! The issue you're describing should be fixed soon by an upcoming update that is currently in an experiment.

Closing this as it seems to be resolved. Please reopen if you are still encountering this problem.

Installed codeium.vim with packer in Neovim. After running :Codeium Auth and pasting the token I get:
Paste your token here: *********** ...  ************Error detected while processing function codeium#command#Command[6]..4: line   43: E474: Unidentified byte: stty: 'standard input': Inappropriate ioctl for device^@{"api_key":"********-****-****-****-******************","name":"First Last"} E474: Failed to parse stty: 'standard input': Inappropriate ioctl for device^@{"api_key":"********-****-****-****-******************","name":"First Last"}
My Neovim config uses NvChad if relevant.

I got this kind of error after I pasted the codeium auth token following the cmd :Codeium Auth:
Error detected while processing function codeium#server#Start[36]..<SNR>35_ActuallyStart:
line   13:
E903: Process failed to start: unknown error: "C:\Users\[USERNAME]/.codeium/bin/84ec5f0e149bb382127d0625f179a49fed435091/language_server_windows_x64.exe"

May I ask how to solve this issue? Thanks.

Can you try manually executing that file? There might be an antivirus or permissions error that is preventing us from launching it.

I solved the issue by following this reply.
Completey removing $HOME/.codeium/bin folder and letting the plugin reinstall language server solve the issue

I have the same issue: installed using packer and I get a "failed to parse" error

Same issue even after delete .codeium directory, when I use :Codeium Auth
E474: Unidentified byte: Enter host password for user
E474: Failed to parse Enter host password for user
I'm currently on WSL and I use Packer on Neovim

@gustaveturrell my guess is when you run curl it's asking for a password for some reason.

I am getting this error in the current release on Xcode with a MacBookAir M2

@bryandebourbon please report this at the Xcode repository: https://github.com/intitni/CodeiumForXcode

Installed codeium.vim with packer in Neovim. After running :Codeium Auth and pasting the token I get:
Paste your token here: *********** ...  ************Error detected while processing function codeium#command#Command[6]..4:
line   43:
E474: Unidentified byte: stty: 'standard input': Inappropriate ioctl for device^@{"api_key":"********-****-****-****-******************","name":"First Last"}
E474: Failed to parse stty: 'standard input': Inappropriate ioctl for device^@{"api_key":"********-****-****-****-******************","name":"First Last"}

My Neovim config uses NvChad if relevant.

I had the same error, tried both Fish and Bash shells, and it turned out the error is caused by my personal shell configurations. To solve it, I did the following:

Temporally disable .bashrc mv .bashrc .bashrc.bak
Temporally disable config.fish mv config.fish  config.fish.bak
Then open neovim (or any neovim distor) and run :Codeium Auth.
Copy/paste the key and hit enter.
Enable your Fish or Bash config (or both) again by mv .bashrc.bak .bashrc and mv config.fish.bak config.fish

well i am on windows using nv chad and i am getting this error
````Error` detected while processing function codeium#command#Command[6]..1:                             line   66:                                                                                E474: Unidentified byte: An unknown element "" was received. This can happen if the remote process closed or ended abnormally.^@                                                                                E474: Failed to parse An unknown element "" was received. This can happen if the remote process closed or ended abnormally.^@```1


NavController.popTo() not working

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Current behavior:
APage push to BPage, BPage push to CPage. When CPage use popTo(PageA) function, instead of pop to Page A, it just pop to BPage.
Expected behavior:
It should've pop to APage
Steps to reproduce:

PageA.navCtrl.push(PageB);
PageB.navCtrl.push(PageC);
PageC.navCtrl.popTo(PageA);

Related code:
A sample application via GitHub
https://github.com/ifrankwang/ionic-popto-issue.git
Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (/Users/mac_dev/.config/yarn/global/node_modules)

    @ionic/cli-utils  : 1.19.0
    ionic (Ionic CLI) : 3.19.0

global packages:

    cordova (Cordova CLI) : 7.1.0

local packages:

    @ionic/app-scripts : 3.1.5
    Cordova Platforms  : none
    Ionic Framework    : ionic-angular 3.9.2

System:

    Android SDK Tools : 26.1.1
    Node              : v8.9.3
    npm               : 5.6.0
    OS                : macOS High Sierra

The popTo() method is not within the documentation:
https://ionicframework.com/docs/api/navigation/NavController/
This means it is not intended for use outside of the internal framework.

Even though it says it accepts a Page type I don't think the implementation actually supports it. I myself do use it with a viewcontroller reference.  Although you have to be careful relying on undocumented implementation functions as they can change at any time without any notice. I would much prefer there was some documented way to do it, but there isn't unless you only want to popToRoot.

@herbzi Using ViewController reference as param of popTo works for me! @jayordway popToRoot doesn't sounds logical, since PageA in my example project might not be the root page in a real project.


Move VM Image List off of GitHub

Notice in the cloud.py, we have a hardcoded path to a file hosted in github. It means every time users execute az vm image list, the command will fetch the file from raw.githubusercontent.com. This render a few issues:

Why do we rely on a third party to host files for runtime execution?
Github is not a file share service. It is a source control service. That means the url we relies on is not 100% reliable.
Github can throttle the access to this file just like they throttle the release files, which will break our command.
Github can be blocked in some subnet or behind a firewall, which will make this command unusable.
Github won’t guarantee the performance of this file host.

I understand that the logic behind this is that this fire will be updated after CLI is released so as to keep to the latest image types list. However, we should have this file publish to a storage account under our control to guarantee the user experience.

endpoints=CloudEndpoints(
  | management='https://management.core.windows.net/',
  | resource_manager='https://management.azure.com/',
  | sql_management='https://management.core.windows.net:8443/',
  | batch_resource_id='https://batch.core.windows.net/',
  | gallery='https://gallery.azure.com/',
  | active_directory='https://login.microsoftonline.com',
  | active_directory_resource_id='https://management.core.windows.net/',
  | active_directory_graph_resource_id='https://graph.windows.net/',
  | active_directory_data_lake_resource_id='https://datalake.azure.net/',
  | vm_image_alias_doc='https://raw.githubusercontent.com/Azure/azure-rest-api-specs/master/arm-compute/quickstart-templates/aliases.json'),    # pylint: disable=line-too-long
  |

@JasonRShaver I agree. It's better to use Azure Storage for file hosting.
@jseely Have you run az cloud set --name AzureUSGovernment? What does "no public internet access" mean? Can you access Azure public cloud resource?

This is for a government client. They have their environment locked down. I would assume that would mean they would not have access to Azure Public Cloud resources either. I can try to get more details...
Is it not possible to have the CLI pull the data required from the Resource Provider?

@jseely
This doc contains the alias for a few commonly used images.  It is co-maintained by CLI and Compute service team, you can bypass the query by providing the urn. here are the examples to pull from the RP
(env) C:\sdk\azure-cli>az vm image list -otable --publisher canonical --offer ubuntu --sku 18.04 --all --query "[].urn"
Result
------------------------------------------------------
Canonical:UbuntuServer:18.04-DAILY-LTS:18.04.201907180
Canonical:UbuntuServer:18.04-DAILY-LTS:18.04.201908011
Canonical:UbuntuServer:18.04-DAILY-LTS:18.04.201908070
Canonical:UbuntuServer:18.04-DAILY-LTS:18.04.201908131
Canonical:UbuntuServer:18.04-DAILY-LTS:18.04.201908140
Canonical:UbuntuServer:18.04-LTS:18.04.201804262
...

(env) C:\sdk\azure-cli>az vm create --image Canonical:UbuntuServer:18.04-LTS:18.04.201804262 -g myRG -n myVM

@yugangw-msft in our usage the --image value that we provide is a Managed Image. Are you saying it shouldn't be looking up the githubusercontent.com content in this case?

No, it shouldn't be.
For managed images which were captured/built by customers, they have 2 kinds of residences:

As a standalone images, which can be retrieved through az imge list
In a shared image gallery which can be retrieved through  az sig image-version list

Close issue for it's inactive for a long time.

This is not inactive for a long time. This is an active issue we still have.

This is not inactive for a "long time." This is an active issue we still have. Two weeks when the last comment is "yeah it shouldn't be doing this" doesn't seem like a closable issue to me.

OK. I don't think Github is a file server.

I have opened this PR #10658 which should fix the issue we were running into.
I also noticed a remark in https://github.com/Azure/azure-powershell/issues/8027 from @BorisB2015 that a solution for this may be to add the aliases doc, that's currently on githubusercontent.com, into the build process for both the Powershell module and the Azure CLI. I think that may be a better long term solution as the image aliases will then also be made available to uses in Azure Environments without public internet access. (Note: I have two large customers in the Defense and Banking industries where this is the case)

https://github.com/Azure/azure-rest-api-specs/blob/master/arm-compute/quickstart-templates/aliases.json
This is the file that we use to convert alias to urn.
@jseely Thanks for creating the PR.
I agree. Let the CI task copy the latest file to project as a resource file. Then CLI/Powerver no longer need to visit github.

https://github.com/Azure/azure-rest-api-specs/blob/master/arm-compute/quickstart-templates/aliases.json
This is the file that we use to convert alias to urn.
@jseely Thanks for creating the PR.
I agree. Let the CI copy the latest file to project as a resource file. Then CLI/Powershell can read from it if there is no internet access.

I merged the PR. It will be available in Azure 2.0.76. Official Release: 11/04/2019.
PR: [Compute] vm create: Add a local copy of image alias file so that it can be accessed in a restricted network environment
Environment like Azure GOV can't access Internet other than servers of Azure. When creating VM, Azure CLI visit a image alias file in Github to translate image alias to full urn. This patch intends to add a local copy of this file as a backup in case of network issue.


Date validator

Given a date field, if I apply the following validation:
'fecha': {'format': '%d/%m/%Y'}
It works well. When I edit the record from a Model.form the date is displayed and required in the format indicated.
Now, if the validation is the following:
'fecha': {'format': '%d/%m/%Y', 'allow': 'empty'},
In the Model.form, the validation works fine, but the date is displayed in this format: YYYY-MM-DD, which, before submitting, forces me to modify the value to DD/MM/YYYY

@josejachuf will inspect this shortly


Hide / Show edition features according to user permissions

On every page, Hide / Show edition features according to user permissions

Done in #80


Error [ERR_SERVER_ALREADY_LISTEN]: Listen method has been called more than once without closing.

i was setup project but i got some error
Error [ERR_SERVER_ALREADY_LISTEN]: Listen method has been called more than once without closing.
at Server.listen (net.js:1432:11)
at Object. (C:\Users\PC\Desktop\stream server\easyrtc\server_example\server.js:57:11)
at Module._compile (module.js:662:30)
at Object.Module._extensions..js (module.js:673:10)
at Module.load (module.js:575:32)
at tryModuleLoad (module.js:515:12)
at Function.Module._load (module.js:507:3)
at Function.Module.runMain (module.js:703:10)
at startup (bootstrap_node.js:193:16)
at bootstrap_node.js:665:3
PS C:\Users\PC\Desktop\stream server\easyrtc\server_example> node server.js
info    - EasyRTC: Starting EasyRTC Server (v1.1.0) on Node (v9.6.1)
debug   - EasyRTC: Emitting event 'startup'
debug   - EasyRTC: Running func 'onStartup'
net.js:1432
throw new errors.Error('ERR_SERVER_ALREADY_LISTEN');
^

Simply check that you don't call listen() twice in you code this is a typical error when you copy paste without understanding.

May be you have something on 8080 of used port already

No . on window it got error . i remoted end code listen port and it's runing . but on ubuntu it ok

The Problem is that the .listen Function is indeed called twice by accident.  See here:
https://github.com/priologic/easyrtc/blob/master/server_example/server.js#L16
https://github.com/priologic/easyrtc/blob/master/server_example/server.js#L57
Removing it from Line 16 fixes the error and keeps the console log that the Server started successfully

thank you

@kurodenjiro @TheClonker
See #374 for fix in master and beta.
Sorry, I missed it, feel free to PR next time ;)


Support for objects?

Feature request:
I see references to things like convertObjCObjectToJSIValue and the reverse but mmkv.mm doesn't seem to be exposing this functionality.
Would it be possible to get native object/dict support so one could avoid using JSON.stringify() (and rely on the native counter-part)? Or have you found that it's more trouble than it's worth?

Hmm, on iOS there is support for storing id (basically any), I'll have to see how that will work.
What do you think the return type should be?
I'm thinking unknown:
getObject: (key: string) => unknown

I'll see what's possible in https://github.com/mrousavy/react-native-mmkv/pull/7

A way to do it is already explain on documentation :
const user = {
  username: 'Marc',
  age: 21
}

storage.set('user', JSON.stringify(user))

const jsonUser = storage.getString('user') // { 'username': 'Marc', 'age': 21 }
const userObject = JSON.parse(jsonUser)

May this Issues should be closed ?

Yeah I think closing this issue makes sense. JSON is fast and easy, otherwise I would have to create a custom implementation and binary serializer (protobuf/flatbuf/flexbuf?) and that's maintainable native code.


add zendframework/zftool

it seems that zftool is not installed in composer.lock, and is required for development-mode (provisioned via docker-compose). please let me know if i have misconfigured something. i have tried 1.4.1 and master.

this request will explicitly add zendframework/zftool as a dev dependency. i ran into a conflict with zendframework/zend-json, and had to allow for a downgraded zend-json version to resolve it.

apologies, i believe i had an old development.config.php in place which referenced ZFTool instead of ZendDeveloperTools which i see now is in the dist file.


Compute_Accuracy

Reminder

[X] I have read the README and searched the existing issues.

System Info

I'm curious about this metrics for and how could i use this? and when? ( ComputeAccuracy )

and I saw llama-factory paper's metrics ( multi-choice ) and I  wonder if this metrics are match with ComputeAccuracy
anyone can answer me ?
please tell me how can i use this metrics, give me some example commands
thank you!
Reproduction
.
Expected behavior
No response
Others
No response

please refer https://github.com/hiyouga/LLaMA-Factory/blob/main/examples/train_lora/llama3_lora_eval.yaml


Learn how Go unit tests work, and add them to your code

For reference: https://golang.org/pkg/testing/
Add as much as time allows, and preferably you would move your development style to test driven development (TDD): https://en.wikipedia.org/wiki/Test-driven_development

I'll mark this as done, but this becomes an ongoing task now whenever it makes sense and when you add new code/functionality.


teamtalk SDK license

Hello,
Do I as an end user of this application need to purchase a teamtalk SDK license?

No you don't. The SDK runs fine without it, and I wouldn't anyway. TT has horrible security and the developer doesn't seem to care, so I'm not supporting him financially. Encryption should be a lot better, it shouldn't store user information in a plain text XML file, etc.


Exception: Default path for queries not found

Hi,
I installed tract_querier and downloaded the wmql_example_dataset. Inside of the wmql_example_dataset folder I ran tract_querier -a MNI152_wmparc_1mm_small.nii.gz -t IIT2mean.trk -q wmql_1_cst.qry -o result1 as described in the readme.txt. This resulted in the error Exception: Default path for queries not found. This happens on OSX and on Ubuntu with and without VirtualEnvironment. I am using python 2.7.
I would appreciate any help.

Hello @wasserth, how did you install the package? python setup.py install?

Exactly

tract_querier was available globally afterwards. So I ran it in the example folder.

could you please check if the folder /usr/lib/python2.7/site-packages/tract_querier/queries exists? (assuming python2.7 is in /usr/lib. If not could you try and find the *.qry files under the /usr/lib/python2.7 folder?

On OSX (using miniconda) I have a folder ~/miniconda3/lib/python2.7/site-packages/tract_querier-0.1-py2.7.egg/tract_querier/queries. This folder contains several .qry files. On Ubuntu in the VirtualEnvironment I have the same folder with this path: ~/tract_querier_virtual_env/lib/python2.7/site-packages/tract_querier-0.1-py2.7.egg/tract_querier/queries

I got it running using a dirty quick fix: hard code the path of ...site-packages/tract_querier-0.1-py2.7.egg/tract_querier/queries into tract_querier.__init__.

While I'm finding the problem you also can add -I  ~/tract_querier_virtual_env/lib/python2.7/site-packages/tract_querier-0.1-py2.7.egg/tract_querier/queries to the command line if you don't want to live with the dirty fix

Hi @wasserth I think I have fixed the bug. Could you please check if it works now?

Working now. Thanks.

thanks to you!

@wasserth  would you mind to tell what you have done to fix that ? I'm encountering the same issue..
thx !

Hi @gst,
Could you please describe your python environment? In the meantime you can use the -I option as I mentioned previously in this thread to get things running.

Using the latest version of tract_querier it works for me without any modification. Demianw fixed some things a few days ago.

hi, sorry for noise / no worry.. I found my problem.
thx for taking care to answer !


Support Zstd compression to improve send performance

Compared to Zlib, ZSTD offers a higher compression ratio and throughput, refer to: https://github.com/facebook/zstd

A test case
Test data set: 4MB random string
Compression level: 5
Compression ration:

SendMessage throughput:

I think this is a very good attempt

Performance is much better.


First release

[x] Add documentation, mkdocs implementation, docstrings
[X] Simplify schema design to depend on Element Event
[X] Blackify

Please add the following to requirements.txt:
datajoint>=0.13.0

Please add the subject.Subject table to the DataJoint diagram.

Please add the subject.Subject table to the DataJoint diagram.

Please add the subject.Subject table to the DataJoint diagram.

@CBroz1 Please incorporate this suggestion.

Please add the following to requirements.txt:
datajoint>=0.13.0

Please add the subject.Subject table to the DataJoint diagram.

@CBroz1 Please incorporate the above suggestions.

Hi @kabilar - Sorry I missed your comments earlier regarding the diagram and requirements. I've incorporated those suggestions. Thanks!


Add registry-specific credential helper support

Implemented https://github.com/docker/docker/pull/23496
Started the implementation PR as per @dmcgowan 's advice during distribution meeting, so that developers can comment on something concrete.
This change adds a "credentialHelpers" member to the config file where users can specify credential helpers on a per-registry basis. This does not over-load the "credsStore" field, which now acts as the 'default' credential store in the event that one is not explicitly specified for the registry in question.
Signed-off-by: Jake Sanders<EMAIL_ADDRESS>

Here's GCR's credential helper implementation which was the motivation for this PR:
https://github.com/GoogleCloudPlatform/docker-credential-gcr

ping @dmcgowan @aaronlehmann PTAL

I am OK with the design, there are a few relevant issues which effect its usefulness though. These aren't blockers but should be kept in mind and hopefully find someone interested in taking them on.

There is no config subcommand, this setting needs to be done by manually editing a JSON file or an external tool
Having the scheme in registry names still seems to mess up the credential store, only effects docker login https://something, see https://github.com/docker/docker/issues/22910
docker build sends all the credentials up to the engine! This really needs to be fixed to only send up the credentials need to pull the base image. However as is this makes doing docker build with credential store enabled much slower.

whoops :x

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "registry-credential-helpers" git@github.com:KingEmet/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

continued in https://github.com/docker/docker/pull/25851


[DOCK-2053] Change "Cancel" to "OK" in workflow version modal

Description
When viewing the version of a workflow, closing the modal should say "OK" instead of "Cancel".
Issue
JIRA
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that your code compiles by running npm run build
[x] If this is the first time you're submitting a PR or even if you just need a refresher, consider reviewing our style guide
[x] Do not bypass Angular sanitization (bypassSecurityTrustHtml, etc.), or justify why you need to do so
[x] If displaying markdown, use the markdown-wrapper component, which does extra sanitization
[x] Do not use cookies, although this may change in the future
[x] Run npm audit and ensure you are not introducing new vulnerabilities
[x] Do due diligence on new 3rd party libraries, checking for CVEs
[x] Don't allow user-uploaded images to be served from the Dockstore domain

The two states:


[FLINK-8227] Optimize the performance of SharedBufferSerializer

What is the purpose of the change
This pull request optimize the performance of SharedBufferSerializer
Verifying this change
This change is a performance improvement without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@kl0u or @dawidwys I think this could use your opinion.
General questions:

Is there any reason to use a boxed integer over a primitive one? Primitives should be the default choice.
Is is correct that this the id is transient?

I think it is a very good and needed change.
As for @StephanEwen questions:
Ad 1. I see no problem with this entryId being primitive int.
Ad 2. I think this field should be transient it is used only during the serialization process to reflect the order in which we serialize the SharedBufferEntries to restore the links between them. It does not play any role outside of serialization.
Ad 3. I agree it should be at least long. It will limit  the number of intermittent state of pattern graph.
After changing int to long I think it is ok to be merged.

@dawidwys  @StephanEwen Sorry for late response. For question 1 and 2, I have the same thought with @dawidwys and have updated the PR accordingly. For question 3,  I think int is enough as we currently store SharedBufferEntry in a HashMap for each SharedBufferPage, and the size of  HashMap is int. If we want to support long, we should also change HashMap to something else. What's your thought?

@dianfu Don't we assign unique id across all pages, not just a single one? The maximum number of entries is MAX_INT * MAX_INT then, isn't it? If I am right, I think we should use long then.

The page number is the number of patterns and so it is usually very small.  The long is only needed in theory just as you said MAX_INT * MAX_INT resulting to MAX_LONG. If this makes sense, then the HashMap may be should also be replaced as the entry number in one page may be MAX_LONG in theory too.
Changing int to long will introduce state incompatible issue and we need to take care of that when performing serialization/deserialization. I just wonder if it is worth making this change.

You are right. I see now this int is not the only reason for the limit of SharedBufferEntry. Increasing this limit would require way more changes. Let's keep int then.
Will merge it soon.

Merged it. Thanks for the contribution!


Fix/pm 487 control network error eth

Coverage decreased (-0.9%) to 88.96% when pulling f4e60717cb34143f979328f3086b34bf1218d839 on fix/PM-487-Control-network-error-eth into ab80b10ed98ab4bd59c2fd32c1946808663bc95f on master.


Implement comms for widgets support

[x] Add support of widgets created from the back-end
[x] Add support of widgets created from the front-end

Noice.


Add pylint checking to GitHub workflow

See https://github.com/Senzing/stream-loader/issues/303

Pull requests:

https://github.com/Senzing/governor-postgresql-transaction-id/pull/30


Sonar / fix sonar warning multiple tsconfig.json

fix https://github.com/jhipster/jhipster-lite/issues/677

Thanks @Bolo89 !
I know it's only 1 line of code but don't hesitate to claim the bounty, as I know we can spend time to reproduce, analyze and search the solution
So it's well deserved


LESS style updates are not propagating (Preact)

Hey Matt,
Super awesome repo, really digging the setup. I was wondering if you've run into issues with updating styles on the preact branch? I can see that the live reload is coming through but it doesn't apply the updated styles on save.
Right now, I've got it working by changing this portion of bundler.js:
if (app) {
    attachShaderReload(app)
    app.on('watch', (ev, file) => {
      if (/\.(less)$/i.test(file)) {
        style.write() // <----- Added this line!
        app.reload(style.getCSSURL())
      }
    })
  }

Can submit a PR if this is the solution.

This is odd — I just tested on my machine and LESS updates are working fine without any changes.
The way it works is that LESS file changes trigger app.reload, which re-requests style.css. As a result, instead of just serving a static style.css file, the middleware responds with a less-ified version of main.less.
I'm on OSX 10.12.6<EMAIL_ADDRESS>and<EMAIL_ADDRESS>What platform/versions are you using?

Super strange. I'm on macOS 10.12.6<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I just did a fresh clone and yarn installed, and tried some style changes, and no dice.
I can see that the stylesheet updates looking at the style inspector, but the actual changes don't propagate. Posting a lo-fi gif below:


Validate friendly name

Why?
Friendly name validation
How?
Declared friendly name regex in order to have max length 24 and no spaces at the beginning and the end
Testing
Make sure the friendly name is validated properly
Additional context
fixes fe-985

Panteli, there is the same issue I had as well, it does not accept 1 character, when it should. I used the regex "^\\S.{0,24}$" in case that helps.
Everything else (only white spaces, more than 24 chars, empty string etc) works like a charm.

Good catch! Now I use the same with you.
Keep in mind this regex allows spaces at the end, minor since we trim the text before validation and submit


If image metadata is empty, add fallback to get dimensions

If height/width is empty, fetch from native image. Done. Closed.

If height/width is empty, fetch from native image. Done. Closed.


Literal narrowing lost without explicit const assertion

TypeScript Version:  3.9.2

Search Terms:
const assertion, literal narrowing
Code
const a1 = 'A1'
const a2 = 'A2'

const actionCreators = {
    a1: () => ({type: a1}),
    a2: (b: number) => ({type: a2, b}),
}
type Action = ReturnType<typeof actionCreators[keyof typeof actionCreators]>

export default (state = {}, action: Action) => {
    switch (action.type) {
        case a2:
            return {...state, b: action.b}
        default:
            return state
    }
}

Expected behavior:
Code with literal narrowing should compile, similar to the following code with an explicit const assertion:
const a1 = 'A1' as const
const a2 = 'A2' as const
...

Actual behavior:
Code does not compile.
Property 'b' does not exist on type '{ type: string; } | { type: string; b: number; }'.
  Property 'b' does not exist on type '{ type: string; }'.(2339)

Related Issues:
Might be related to #9998.
Output
const a1 = 'A1'; // as const
const a2 = 'A2'; // as const
const actionCreators = {
    a1: () => ({ type: a1 }),
    a2: (b) => ({ type: a2, b }),
};
export default (state = {}, action) => {
    switch (action.type) {
        case a2:
            return Object.assign(Object.assign({}, state), { b: action.b });
        default:
            return state;
    }
};

Compiler Options
{
  "compilerOptions": {
    "noImplicitAny": true,
    "strictNullChecks": true,
    "strictFunctionTypes": true,
    "strictPropertyInitialization": true,
    "strictBindCallApply": true,
    "noImplicitThis": true,
    "noImplicitReturns": true,
    "alwaysStrict": true,
    "esModuleInterop": true,
    "declaration": true,
    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,
    "moduleResolution": 2,
    "target": "ES2017",
    "jsx": "React",
    "module": "ESNext"
  }
}

Playground Link: Provided

This is working as intended. See #11126, and also #29510.

I see... Thanks for the explanation provided in the linked PRs - but my intuition was built from IDEs type hint tooltips, neither VS Code nor IntelliJ distinguish between the 2 different types (widening and non-widening literals both look the same):

I wanted to ask where it's documented, but now that I know what to look for, I found it in mentioned in cost assertions section:

no literal types in that expression should be widened (e.g. no going from "hello" to string)

The feature was very hard to discover for me (and surprising when I did), but I understand how it makes sense I guess, so I will either get used to it or wait for a new readonly-by-default mode proposed in #32758.

my intuition was built from IDEs type hint tooltips, neither VS Code nor IntelliJ distinguish between the 2 different types (widening and non-widening literals both look the same)

Yeah, we actually discussed displaying something to distinguish between the two, but we decided against the additional noise as in most cases it doesn't matter.


CommitTime exporter(s) ignore PROD_LABEL

Similar to #337

Actually, this shouldn't matter. Deployments to prod matter, but we don't care what namespace or environment the build came from, as long as the image winds up in prod.


panda-client 1.5.9 github action

automatic pull request

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


Postgres: is prepared statement feature currently used?

Hello, inspecting the implementation of postgres dialect and driver at https://github.com/koskimas/kysely/blob/fdc7b36c4823898ede69fe11959cb5bd695eeb94/src/dialect/postgres/postgres-driver.ts#L103 I do not see that the queryId is used (should be 2nd argument looking at the  usage references). Does this mean that the prepared statement feature is not used (https://node-postgres.com/features/queries#prepared-statements)?

NO.

You are right, but sometimes it can be an useful optimization especially when the pool is small and query is complex.
I guess current workaround is to .compile() and calling the pg.query directly.


Add jenkins tests for other steps.

Also, use the service account so it can actually deploy.

LGTM - Thanks.


Add Trello extension

Description
This pull request adds a draft of a trello extension, with a few questions, mostly as I am learning React…
Type of change

New extension

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing

And a general question… How can I create an action that sends something back to the Trello API?

@ChrisChinchilla Use a useEffect
function useMycustomPostRequest(data){
  const [response, setResponse] = useState(null);

  useEffect(() => {
    async function postData(postData){
      try {
        const response = await axios.post(url, {
          method: 'POST',
          headers: {
            'Content-Type': 'application/json',
            'Authorization': `Bearer ${preferences.token}`
          },
          data: postData
        });
        setResponse(response);
      } catch (error) {
        console.log(error);
      }
    }

    postData(data).then((r) => r);
  }, [data])

  return response;

}

export { useMycustomPostRequest };

Thanks so much @Ali-Parandeh will apply!

hey @ChrisChinchilla 👋 just wanted to ask what the status on this is - is this ready to be reviewed or do you still need some time to work on it?

Hey @sxn I am still having some issues with test failing (they pass locally), but I think they relate to an issue I am having with Typescript models as the Trello API returns different API responses with the two endpoints I am using, making it hard for me to make it er, "happy", although things all work… I posted a message in the Slack channel about it, but basically one endpoint returns a list of cards, the other returns the list inside a "cards" array, and I am not sure how to solve it…

@sxn I'm stumped on a better way to fix the issues with Typescript and the different API responses, but as part of recent changes, I fixed code that means if you changed a search term, things would crash and the only way I could find to resolve things, was return the object of the response and add an ignore to the typescript lint… It's not satisfactory, but everything works 🤷♂

@pernielsentikaer The read me I knew and kind of forgot about. I am a React and Typescript newbie, but I’ll attempt to understand what these things mean. A lot of the code came from other extensions, so they could also have the same issues…

@pernielsentikaer Regarding the empty state issue, as far as I can tell (and this code mostly came from the npm package search), I have implemented what you suggest?
From here down…
https://github.com/raycast/extensions/blob/c78aa513cd296900ca17967864eb3fce4e780633/extensions/trello/src/index.tsx#L36

Is it intended, that first view shows only my cases, and I can search in all cases in the boards I have? (as the screenshots earlier)

I don't fully understand the question. But If I understand correctly, then yes, at the moment the extension returns all cards assigned to you, and offers the ability to search the text of those. There will be further improvements soon though :)

@ChrisChinchilla I'm just a bit wondering why I only see one when loading the extension, and when I search there is alot more:

Maybe it's as expected, but as a user I would be pretty confused 😄

@pernielsentikaer erm, that’s not what I see at all, as you can see from my early screen grab, it works filtering across a wider list. It could maybe be because that’s the only card assigned to you, and search is searching all cards across all the boards you’re on. I am away from my laptop until Monday afternoon so can’t check until then.

It could be awesome if the initial load showed all instead it's confusing that I only see the one assigned directly to me, but I'm able to search for a lot more if I know what to search for
Could they be combined in one list with sections instead?

@pernielsentikaer OK, been looking into this and it seems unfortunately the way the Trello API is designed, others asking similar questions here - https://stackoverflow.com/questions/26552278/trello-api-getting-boards-lists-cards-information
In that there's an API for returning all cards assigned to you, and then an API for searching everything, but in the "search everything" API you can't filter by member… It's odd I agree, but that's the way it is 🤷
There might be a way to get it to work as Intend by using react state or something, but that would involve a significant rewrite and digging into topics I don't fully understand right now. So how about in the spirits of getting this merged, I cut the search feature, and stick with the simple "return all your cards" for now, and add that back when I've had time to investigate?

How about splitting the things apart?

Filter your cards, as today but but search, only filtering
Search for cards, starts with an empty array

That way it should be solved 😄

@pernielsentikaer Sorry it's been a very busy few weeks, not much time for open source work. I don't fully understand what you mean sorry…

Filter how? With the API? With Raycast/React? Filter on what?
If you mean just a "relevant Trello-wide" search, I still don't really feel like that actually makes much sense from the perspective of a personal tool like RayCast, but until i find a better way to get around that API limitation, I guess it will do…

OK, functionality split is done. I think i know what you mean regarding filtering, but I wonder what to filter on, there's potentially a lot of options…

Thanks @pernielsentikaer rerequested!

Awesome @ChrisChinchilla 🔥
Approved! 😄

🙀 🙀 🙀 🙀 🚀 🚀 🚀 🚀


ModuleNotFoundError: No module named 'scarches.dataset' while installing scArches in Google Colab

Hi,
I have been trying to install using scArches using pip in Colab. However, this gave me an error that has already been reported here. From that post, it seems that the way to go is to install scArches from the master. So I go like this:
!git clone https://github.com/theislab/scarches.git
%cd scarches
!python3 setup.py install
%cd ..

Then, if I try to load scArches it complains that there are some packages that are not there, so I have to install them:
!pip install rich
!pip install pyro
!pip install sinfo
!pip install umap-learn

So far, so good. Now I try to load scArches:
import os
import warnings
warnings.simplefilter(action = 'ignore', category = FutureWarning)
warnings.simplefilter(action = 'ignore', category = UserWarning)

import gdown
import torch
import numpy as np
import scanpy as sc
import scarches as sca
import matplotlib.pyplot as plt
from scarches.dataset.trvae.data_handling import remove_sparsity

but I get the following error:
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-14-8d888ccef61d> in <module>()
      3 import numpy as np
      4 import scanpy as sc
----> 5 import scarches as sca
      6 import matplotlib.pyplot as plt
      7 from scarches.dataset.trvae.data_handling import remove_sparsity

3 frames
/content/scarches/__init__.py in <module>()
----> 1 from . import scarches

/content/scarches/scarches/__init__.py in <module>()
----> 1 from . import dataset, metrics, trainers, models, zenodo, plotting
      2
      3 __author__ = ', '.join([
      4     'Marco Wagenstetter',
      5     'Mohsen Naghipourfar',

/content/scarches/scarches/metrics/__init__.py in <module>()
----> 1 from .metrics import entropy_batch_mixing, knn_purity, nmi, asw

/content/scarches/scarches/metrics/metrics.py in <module>()
      6 from sklearn.preprocessing import LabelEncoder
      7
----> 8 from scarches.dataset import remove_sparsity
      9 from .clustering import opt_louvain
     10

ModuleNotFoundError: No module named 'scarches.dataset'

---------------------------------------------------------------------------
NOTE: If your import is failing due to a missing package, you can
manually install dependencies using either !pip or !apt.

To view examples of installing some common dependencies, click the
"Open Examples" button below.
---------------------------------------------------------------------------

Is there any way I can fix this from the master repo? Or is there a package I am missing?

hi @cartal ,
I just released a new version, could you install that by running pip install scArches  and tell me if it works then?

It works now! Thanks a lot!

Hi, thanks for the previous fix.
Now it seems that I have another issue: After training, following the tutorial, I try to do this:
adata_latent = sc.AnnData(trvae.get_latent())
adata_latent.obs['cell_type'] = source_adata.obs[cell_type_key].tolist()
adata_latent.obs['batch'] = source_adata.obs[condition_key].tolist()

But it fails with the following error:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-18-aa20a9a6797b> in <module>()
----> 1 adata_latent = sc.AnnData(trvae.get_latent())
      2 adata_latent.obs['cell_type'] = source_adata.obs[cell_type_key].tolist()
      3 adata_latent.obs['batch'] = source_adata.obs[condition_key].tolist()

1 frames
/usr/local/lib/python3.7/dist-packages/scarches/models/trvae/trvae_model.py in get_latent(self, x, c, mean)
    176             c = torch.tensor(labels, device=device)
    177
--> 178         x = torch.tensor(x, device=device)
    179
    180         latents = []

/usr/local/lib/python3.7/dist-packages/scipy/sparse/base.py in __len__(self)
    289     # non-zeros is more important.  For now, raise an exception!
    290     def __len__(self):
--> 291         raise TypeError("sparse matrix length is ambiguous; use getnnz()"
    292                         " or shape[0]")
    293

TypeError: sparse matrix length is ambiguous; use getnnz() or shape[0]

I have tried in a local machine as well as colab and with two different datasets and I can confirm that I get the same error in both machines with both datasets.

Hi @cartal  you need to pass the adata to get latent function

Sorry, could you tell me how? because this was working before and now it isn't.

Sorry, could you tell me how? because this was working before and now it isn't.

sorry I thought it is another problem, seems to be scipy sparse problem @Cottoneyejoe95  do you have any idea?

@cartal Did you call the remove_sparsity() funtion before passing the adata to the model?

Ah, there I am at fault. After running this it worked! Thanks a lot for the help!

Hello, I am installing scArches using "pip install scArches" in Google Colab and encounter the same issue as Cartal. Could you please suggest me a solution? Thank you.
Below is my installation code:
!pip install scarches
!git clone 'https://github.com/theislab/scarches'
%cd scarches
!python3 setup.py install

@hanh-thepepsimist
!pip install -e git+https://github.com/theislab/scarches.git@master#egg=scarches
then click restart runtime
This should work for Google Colab.


docs: update SvelteKit integration

don't merge until we publish SvelteKit integration

@benmccann thx man

included in the new docs


Using orchestrator  with servers using replicate- variables

Hello!
I have MariaDB replication setups using additional variables to control which databases and tables are actually getting replicated (in multi-master scenario when one slave replicates two databases from different hosts).
e.g:
            (replicate db1)
master1 ----\
             >---------- slave
master2 ----/
             (replicate db2)

replication is controlled by per-connection variables like replicate-wild-do-table or replicate-wild-do-db
Is there a way to use orchestrator with such configuration?

regardless of replicate-* setup, orchestrator does not support multisource replication.

https://github.com/github/orchestrator/blob/master/docs/supported-topologies-and-versions.md
https://github.com/github/orchestrator/issues/433
https://github.com/github/orchestrator/issues/38

@shlomi-noach thank you. I missed it while browsing docs.
I am closing this issue because my question is now answered.


Template usage is undocumented

Hi, I had a hard time trying to figure out how to use wagtailnews in templates. I had to use template debugging tools to finally find out that these variables are mainly to be used:
newsitem_list
newsitem_page
newsitem_view

newsitem_list can be iterated through to receive a list of news pages / blog posts. That was the most important thing for me to find out by experimenting.
I am a wagtail beginner more or less, so please point me in the right direction if this would have been found somewhere in the docs or could be otherwise retrieved by reading code - In any case, mentioning it at least in you README.md is essential, at least to my understanding.
Thanks a lot for wagtailnews! :-)
and all the best,
Jojo

Fair call, some extra documentation would help peolpe find that info, at the moment you could only find that out by looking at the source.
I think the best place would be under 'Usage' in the docs with a bit of info about how to setup your templates for use with wagtailnews. Thanks for pointing this out :+1:

Fair call, some extra documentation would help peolpe find that info, at the moment you could only find that out by looking at the source.
I think the best place would be under 'Usage' in the docs with a bit of info about how to setup your templates for use with wagtailnews. Thanks for pointing this out :+1:


Replace calls to getProjectAndPackageIds() with collectProjectsAndPackages()

Please have a look at the individual commit messages for the details.

@fviernau are you fine with merging this, or do you use getProjectAndPackageIds() in rules?

@fviernau are you fine with merging this, or do you use getProjectAndPackageIds() in rules?

Can be merged from our internal setup perspective.


这个app的config不支持dns和rout ing

我直接复制了https://raw.githubusercontent.com/XTLS/Xray-examples/main/VLESS-TCP-XTLS-Vision/config_client.json这个链接，提示错误，如果把dns和routing都删掉就可以工作

Hello @Bearljj!
Thank you for posting this topic. That's true. We should fix this problem. We will investigate the issue and send you the feedback.

@Bearljj This problem occurs because there are no geoip.dat and geosite.dat files in the Invisible Man XRay root folder. We'll fix this problem in our new release. But if you want to use it in your current version, you should install these two files manually and replace them in the root folder.
Follow these steps:

Download geoip.dat from this link (https://github.com/v2fly/geoip/releases/latest/download/geoip.dat)
Download dlc.dat from this link (https://github.com/v2fly/domain-list-community/releases/latest/download/dlc.dat)
Rename dlc.dat to geosite.dat
Copy both dat files, go to the Invisible Man XRay root folder and paste them beside the Invisible Man Xray.exe file.
Run the app.

Don't close this topic because we should fix this problem and after that, we will close it.


Annotate lineages with described mutations

Following up on #61 we want here to include in the DB the mutations that are described to conform a given lineage. This is slightly different from the mutations that are observed in the samples from this lineage.
We could create a table LineageVariant with one FK pointing to the Lineage table and another FK pointing to the Variant table.

[x] Find an appropriate resource that describes the mutations in each lineage
[x] Define an appropriate data model (the same variant may occur in multiple lineages...)
[x] Parse the mutation representation
[x] Load the data in the DB at startup time

Potential issues:

Are indels represented in a normal form compatible with what we use in CoVigator?
MNVs may not be reported together, we may need to merge them
The Variant table contains the functional annotations coming from SnpEff. This information is obtained from the VCF files that come out of the pipeline. We won't be able to add this information if the database does not contain this variant beforehand.

https://covariants.org/variants
https://github.com/hodcroftlab/covariants/blob/master/web/data/clusters.json
The data provided by the Covariants team could be another source of lineage-defining mutations. They are divided into synonymous and non-synonymous mutations. The lineages are defined using the Nextstrain classification.

The covariants resource looks better structured. Feel free to use this if it helps you

Done. Closing the issue


Update Aria Team Project Page with our project information

We need to add all our info on the Aria Team Project Page for judging!

[ ] Add link to live app
[ ] Add link to code repo
[ ] Add link to presentation
[ ] Add team info (bios)
[ ] Finalize our content
[ ] Add project description and info

Final content added. Thanks everyone for your contributions!
https://2018.spaceappschallenge.org/challenges/help-others-discover-earth/artify-earth/teams/aria-earthmoog/project


Do a full reload if needed

Find out whether we need to do a full reload or not. If a full reload is needed, do it.

Would love some kind of choice in the API here ( sometimes we have work happening on init() that doesn't get changed when switching between stories of the same component) would love to be able to declare that this should get a fresh virtual DOM.

@orta you mean something like this? https://github.com/kadirahq/react-storybook/pull/285

I think so, I'm still quite fresh to all these pieces

We are doing this now with the latest version of Storybook. (2.14.0+)


Dashboard redesign waypoint.

This pull request:

deletes image assets no-longer needed in repo.
gathers new image assets for site redesign.
downloads and tests addons before moving forward.
upgrades to latest version of bootstrap.
gathers latest update of css for site redesign.
consolidates new css files into @imports in style(Dark|Light).scss

:+1:


Vscode Linux app context wrong?

See https://talonvoice.slack.com/archives/CCEE45RT7/p1613654398012400. Copying from there:

Looks like the knausj code for matching the VS code window title is looking for "Code - OSS" in the name. Try this change:

diff --git a/apps/vscode/vscode.py b/apps/vscode/vscode.py
index bb992ea..547e681 100644
--- a/apps/vscode/vscode.py
+++ b/apps/vscode/vscode.py
@@ -11,7 +11,7 @@ and app.bundle: com.microsoft.VSCode
 """
 mod.apps.vscode = """
 os: linux
-and app.name: Code - OSS
+and app.name: Code
 """
 mod.apps.vscode = """
 os: windows

Would be good to know why we were looking for the "- OSS" bit, maybe we should allow either.

the name is pretty fragile in my experience, developers like to randomly change it. I don't know if we need to add the executable for linux, similar to Windows?

I guess the reason it was looking for OSS is that certain distributions have multiple versions. the ones with OSS in the name are packages that are built directly from the git repository. the ones without are the official binaries provided by microsoft.
example from arch showing the title was Code - OSS:
aur/visual-studio-code-bin 1.53.2-1 (+995 19.08)
    Visual Studio Code (vscode): Editor for building and debugging modern web and cloud applications (official binary version)
community/code 1.53.2-1 (10.3 MiB 71.7 MiB) (Installed)
    The Open Source build of Visual Studio Code (vscode) editor

confirmed xprop gives the following for the OSS version.
WM_NAME(UTF8_STRING) = "Welcome - Code - OSS"
_NET_WM_NAME(UTF8_STRING) = "Welcome - Code - OSS"
_MOTIF_WM_HINTS(_MOTIF_WM_HINTS) = 0x2, 0x0, 0x1, 0x0, 0x0
XdndAware(ATOM) = BITMAP
_NET_WM_BYPASS_COMPOSITOR(CARDINAL) = 2
WM_WINDOW_ROLE(STRING) = "browser-window"
WM_CLASS(STRING) = "code-oss", "code-oss"

also just installed the official package:
WM_NAME(UTF8_STRING) = "Welcome - Visual Studio Code"
_NET_WM_NAME(UTF8_STRING) = "Welcome - Visual Studio Code"
_MOTIF_WM_HINTS(_MOTIF_WM_HINTS) = 0x2, 0x0, 0x1, 0x0, 0x0
XdndAware(ATOM) = BITMAP
_NET_WM_BYPASS_COMPOSITOR(CARDINAL) = 2
WM_WINDOW_ROLE(STRING) = "browser-window"
WM_CLASS(STRING) = "code", "Code"

so I think just matching code-oss and Code as the app names is correct.


Can't use @Convert on accessor method

Expected Behavior
I have a class hierarchy that uses generics for a field's type, T. I want to use the Convert annotation in a subclass where the field's type is defined so that I can customise the conversion between entity and database.
Current Behavior
The comment on neo4j-ogm/core/src/main/java/org/neo4j/ogm/annotation/typeconversion/Convert.java says: "Annotation to be applied to fields and accessor methods of entity properties [..]", but the annotation's definition allows it to be used only on fields. It isn't possible in Java to override the type of a field in a subclass.
Possible Solution
Allow the Convert annotation to be used on accessor methods, as suggested in the javadocs for the Convert annotation class.
Your Environment

OGM Version used: 3.0.0-RC1
Java Version used: 8
Neo4J Version used: 3.2.3

Hi,
thank you for reporting this.
Since 3.0.0 the fields are accessed directly, never through a getter. The documentation has not been updated to reflect this. We will fix this.
Regarding your use case - you are defining field T t; in a base class and you want to provide different converter for each subclass?

Hi, yes that is my use case.

Since it is reflected in the migration docs (http://neo4j.com/docs/ogm-manual/current/migration/#_field_access_only) it is still in the type conversion section (http://neo4j.com/docs/ogm-manual/current/reference/#reference:type-conversion:custom). Also the Javadoc on the class is still outdated.


1.22.4 -> 1.22.5 Regression for ServiceEntries with multiple VIPs

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
When upgrading from 1.22.4 to 1.22.5 we started to see intermittent outages in traffic relating to ServiceEntries with multiple addresses.
The clusters then split the ServiceEntries into 1 address per SE (this fixed the issue) and also rolled back to 1.22.4 (which also independently fixed the issue). Solutions were tested separately and then applied to all the affected clusters.
Version
$istioctl version
client version: 1.22.5
control plane version: 1.22.5
data plane version: 1.22.5 (19 proxies)
$kubectl version
Client Version: v1.31.1
Kustomize Version: v5.4.2
Server Version: v1.29.9+IKS

Additional Information
apiVersion: networking.istio.io/v1
kind: ServiceEntry
metadata:
  annotations:
    meta.helm.sh/release-name: istio
    meta.helm.sh/release-namespace: default
  creationTimestamp: "2022-01-22T00:49:39Z"
  generation: 1
  labels:
    app.kubernetes.io/managed-by: Helm
  name: ims-resolvers
  namespace: pdns
  resourceVersion: "286944889"
  uid: d773ab5c-19b9-4c61-95ba-031f8dd927df
spec:
  addresses:
  - IP address 1
  - IP address 2
  endpoints:
  - address: IP address 1
  - address: IP address 2
  hosts:
  - ims-resolvers.not.used
  location: MESH_EXTERNAL
  ports:
  - name: TCP
    number: 53
    protocol: TCP
  resolution: STATIC

istio-proxy error log when on 1.22.5. The error seen very intermittent i.e not every egress call to IP address 2 failed.
IP address 2 redacts actual IP address.
"- - -" 0 UH - - "-" 0 0 0 - "-" "-" "-" "-" "-" BlackHoleCluster - _IP address 2_:53 <IP_ADDRESS>:45382 - -

istio-proxy log did not report any BlackHoleCluster while it was on 1.22.4

See the release note here: https://istio.io/latest/news/releases/1.22.x/announcing-1.22.5/#changes.
There wa sa fix that caused many issues so it was reverted.
A simple config like
apiVersion: networking.istio.io/v1
kind: Sidecar
metadata:
  name: sidecar
spec:
  egress:
  - hosts:
    - '*/*'

in the same namespace likely would fix it. I didn't test with your specific config but think think it should apply. in 1.24 this workaround is not needed

The cluster upgraded from 1.22.3 (where traffic was fine) to 1.22.4 and traffic was fine. It stayed like that for 3 weeks with everything healthy. Then when it upgraded to 1.22.5 it ran into this issue.
It might be related to that SA change in 1.22.4 that was reverted in 1.22.5. However I don't understand the mechanism.

I see the same behavior on 1.22.3 and 1.22.5, and both work with the workaround (https://github.com/istio/istio/issues/53530#issuecomment-2412826059), and both work on master branch.

Prior to Oct 7 we were on 1.21.5. Between Oct 7 and Oct 8th we upgraded to Istio 1.22.4 in all regions. On Oct 9th Istio 1.22.5 was pushed and we started noticing the problem with ServiceEntry with multiple address logging BlackHoleCluster errors mostly on 2nd IP address in the ServiceEntry To mitigate the issue (While we were on 1.22.5) we updated our ServiceEntry into 2 individual ServiceEntry. We also rolled back to 1.22.4 while we had individual ServiceEntry.

Oh, I thought it had gone from 1.22.3 -> 1.22.4 -> 1.22.5. Thank you for correcting me about it being 1.21.5 -> 1.22.4 -> 1.22.5.


Update runtime-spec to aaa24815e0ab632c424e18d726fd47f403ce2981

Signed-off-by: Kenfe-Mickael Laventure<EMAIL_ADDRESS>--
Required for https://github.com/moby/moby/pull/33336

LGTM

@mlaventure if docker sets OomScoreAdj then we need to bump runc with a new version of the spec and update that as well.  The field was moved from the Linux Resource section to the Process


Update TOML parser to pick up a bugfix

Cargo has previously accepted invalid TOML as valid, but this bugfix should fix
the problem. In order to prevent breaking all crates immediately toml-rs has a
compatibility mode which emulates the bug that was fixed. Cargo will issue a
warning if this compatibility is required to parse a crate.

r? @brson

r? @wycats
(rust_highfive has picked a reviewer for you, use r? to override)

:umbrella: The latest upstream changes (presumably #2691) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably #2743) made this pull request unmergeable. Please resolve the merge conflicts.

@bors r+

:pushpin: Commit c17f9f0 has been approved by brson

:hourglass: Testing commit c17f9f0 with merge a00992f...

:broken_heart: Test failed - cargo-win-msvc-64

@bors: r=brson

:pushpin: Commit 0fa6f27 has been approved by brson

:hourglass: Testing commit 0fa6f27 with merge 0de788d...

:broken_heart: Test failed - cargo-mac-32

:pushpin: Commit 2a1e16c has been approved by brson

@bors: r=brson
On Sun, Jun 12, 2016 at 1:02 PM, bors<EMAIL_ADDRESS>wrote:

💔 Test failed - cargo-mac-32
http://buildbot.rust-lang.org/builders/cargo-mac-32/builds/1669
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang/cargo/pull/2680#issuecomment-225426545, or mute
the thread
https://github.com/notifications/unsubscribe/AAD95MktiRVfuFGduXHiuBzEU2-OwoBSks5qK-c-gaJpZM4IdXSK
.

:hourglass: Testing commit 2a1e16c with merge f8e75ec...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


wallet-ext: home UI update

adds support for different themes
uses new design for main view
moved network-switch under settings view
fixes  import order rule for scss aliases

https://user-images.githubusercontent.com/10210143/177606940-e952324b-6a20-4fa9-bb25-efd6f09a289f.mov
https://user-images.githubusercontent.com/10210143/177606986-f186e523-f67b-4c8b-9722-aebb4c97291b.mov
part of #2959

did we get the scss files from dreamten, otherwise can we do so?

No we didn't get them from dreamtem, I assume they will give us some style once they complete the design?

what's the plan to have the images like icons etc. in the new design?

Currently we use an icon pack from bootstrap. I think for the new design we will use something similar. For custom icons that can be converted to iconfonts will add some functionality for this in upcoming prs. For the rest I would prefer svg icons (to allow resizes) but since we only have one specific size to support (the popup) png images also should be ok

The icons used can be exported from Figma


Extract Curl header handling into separate class

Split off part of the work from https://github.com/newsboat/newsboat/pull/2340

coverage: 60.494% (+0.04%) from 60.45%
when pulling 68a6d2e288ef8be3ed683417da87d9a385e86147 on dennisschagt:curl-header-handling
into 323c4f6cb0ce012d93ea9e1ebe563b45047af092 on newsboat:master.


chore(ssa refactor): range constraints for integer params to main

Description

Problem*

Resolves
Summary*

Automatically add range constraints for integers found in the parameters to main
Example
fn main(x : u8) -> pub u8 {
    x
}

produces:
current witness index : 2
public parameters indices : []
return value indices : [1]
BLACKBOX::RANGE [(_1, num_bits: 8)] [ ]

Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Issue for adding test:
https://github.com/noir-lang/noir/issues/1452


Mmansurova/abc 262

When navigating to the global search, please update the title

When performing a search, update the url to have the search term. This will enable user to return to his previous search when navigating to the results page


Fix issues with applyMultiSignatures

Fixed signatures overwriting each other if applied one at a time
Fixed the order of signatures not being checked against redeemScript. signatures will now be sorted regardless of their initial order.

@cronicc do we have ci for this repo?

Yes, we do

for some reason the check is not reported then.

https://travis-ci.org/ZencashOfficial/zencashjs/builds/569927719?utm_source=github_status&utm_medium=notification
The repo still seems to be using the travis legacy open source integration. So it's running on travis.ci-org instead of travis-ci.com,

The pipeline was failing because I hadn't committed the output code as well. It's fixed now.
Also addressed your comments @nicolam1

Not a regression but an additional bug, zencashjs doesn't handle m-of-n multisig correctly if m=n.

What

Hi, I'm looking into multisig wallets for ZEN and this JS lib looks like a great programmatic way of using this feature. Are the issues reported above still relevant? Should use something else instead for ZEN MultiSig wallets?

Hi, I'm looking into multisig wallets for ZEN and this JS lib looks like a great programmatic way of using this feature. Are the issues reported above still relevant? Should use something else instead for ZEN MultiSig wallets?

This PR would extent the multisignature functionality of zencashjs, but even without this PR multisignature functionality works fine.
please take a look at the examples in https://github.com/HorizenOfficial/zencashjs/blob/master/README.md#example-usage-multi-sig


action output to github environments!

https://github.com/Azure/static-web-apps-deploy/blob/v1/action.yml#L43-L44

https://github.com/google/secrets-sync-action/


HTML & HTML Source message tabs are not available in Chrome

What is the reason that HTML & HTML Source tabs are not available in Chrome? Is it a bug, or not implemented yet feature?
Firefox:

Chrome:

It looks like is was an issue with Chrome Extensions.

Thanks @antonkomarev, could you please tell me what chrome extention was causing the issue?

I'm not sure. One of my team members faced the issue. She disabled all extensions and re-ensbled them. And now it's working.

@axllent I get the same problem. I can not see the tabs both in firefox and chrome.
Firefox:

Chrome:

I am using 1.5.1.


[Outlook] (online meeting) Update TOC title

Docs Build status updates of commit a4cfea9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.