Order of prefixed properties have changed

This is an example, but when using flex-grow on items in 2.20.30, prefixed properties came before the spec properties, so the spec always won if supported:

However upgrading to 2.20.32 the order of these properties have changed, placing the prefixed properties last and therefore winning over the spec properties:

I also noticed that the number of prefixed properties have been reduced, and I assume that is from this commit (which is an awesome thing, btw), but I worry that the prefixed (legacy) property winning can cause weird regression issues.
I'm hoping the fix may be as simple as a basic sort (where anything with a leading - comes first) before being added to the style tag?

That's odd, I'll investigate.

I've traced it to the serialization that occurs in src/CSSPropertyOperations/createMarkupForStyles ... I'm gonna try to work a fix so that I can submit a PR instead of just complain haha

that should be fine though? I've made an issue in the prefixer package as well https://github.com/rofrischmann/inline-style-prefixer/issues/133

hmm, it's iterating the keys in a different order than expected

I have a fix locally, passes all the tests ... will be creating a PR in a few if its alright.

sure, go for it. you can share the snippet here too, I'm hacking on it myself rn

export function createMarkupForStyles(styles, component) {
  const serialized = Object.keys(styles)
    .sort((a, b) => {
      return (b.indexOf('--') === 0) ? b : processStyleName(b)[0] === '-' ? 1 : 0;
    })
    .reduce((styleString, styleName) => {
      const isCustomProp = (styleName.indexOf('--') === 0)
      const styleValue = styles[styleName]

      if (process.env.NODE_ENV !== 'production' && !isCustomProp) {
        warnValidStyle(styleName, styleValue, component)
      }

      if (styleValue == null) {
        return styleString;
      }

      return isCustomProp ? `${styleString}${styleName}:${styleValue};` :
        `${styleString}${processStyleName(styleName)}:${dangerousStyleValue(styleName, styleValue, component)};`
    }, '')

  return serialized || null
}

Basically converted it to be more function with use of Object.keys which allowed for sorting prior to serialization. I couldn't think of a way to order the keys without iterating over them prior to reduction, and this seemed the most efficient way to double-iterate.

not sure this is the best way to do it, might lead to weird behaviour when I'd need it in another order.
It looks like the object is created correctly, but it's pulling the keys in the wrong order. I'll investigate further for a better solution, and/or wait for i-s-p to come up with a solution.

weirdness

yup ... does not surface with display: flex;, only with flex-grow or shrink or whatever ... super weird

it's because isp uses the array/fallback method for display, but separate keys for the others, and if you look at the screenshot, the order is fine in the collapsed form, but reorders in the expanded form/ when keys are enumerated.

yupyup I got ya ... from what I can tell, it looks like alphabetic ordering is used, and capitalization matters in the expanded display, but not in the collapsed (and also not in iteration).

also just noticed that the prefixer mutates the input style, ack ><

How about something like this?
export function createMarkupForStyles(styles, component) {
  let serialized = '';

  for (let styleName in styles) {
    if (!styles.hasOwnProperty(styleName)) {
      continue;
    }

    const isCustomProp = (styleName.indexOf('--') === 0)
    const styleValue = styles[styleName]

    if (process.env.NODE_ENV !== 'production' && !isCustomProp) {
      warnValidStyle(styleName, styleValue, component)
    }

    if (styleValue == null) {
      continue;
    }

    if (isCustomProp) {
      serialized += `${styleName}:${styleValue};`
    } else {
      const processedName = processStyleName(styleName);

      serialized = processedName[0] === '-' ?
        `${processStyleName(styleName)}:${dangerousStyleValue(styleName, styleValue, component)};${serialized}` :
        `${serialized}${processStyleName(styleName)}:${dangerousStyleValue(styleName, styleValue, component)};`
    }
  }

  return serialized || null
}

No additional iterations, a better guarantee that the property is indeed prefixed before ordering the string, and you get to keep the original coding style of for-in (looks like Object.keys + reduce was less performant).

I'll consider it. I'm also looking at writing my own, non-mutating version of isp's createPrefixer.js (don't need it's nesting/array support anyway)

Fair enough, I'll just set up the PR with this and let it's fate rest in your hands. Anything I can do, lemme know.

do you have an example of a prefixed property having the different/wrong value than a non prefixed property? the author of the prefixer doesn't seem to think this is a problem per se.

Its not a problem with the values being incorrect, the values always line up ... the issue is that when you convert the objects into strings for insertion into the <style> tag it places the prefixed properties after the spec properties. This can cause unexpected behavior, and really is counter-intuitive (as with all fallback mechanisms, the spec always comes last because it should win out).
In the screenshots I provided that started this whole thing, flex-grow comes after -webkit-flex-grow in version 2.20.30 (which is correct behavior, based on the above logic), however -webkit-flex-grow comes after flex-grow in version 2.20.32, which means that the prefixed property "wins" over the spec property. This is why I provided the PR ... it simply places prefixed properties before non-prefixed properties when being serialized into a string, ensuring that correct order is maintained for CSS overrides.

I can confirm that the behavior of the prefixed properties is not the same as those for the spec properties, as I have witnessed visual bugs related to use of the prefixed properties. An example is creating an element that is both the child of a column flex container and is a column flex container itself where the height is not a specific pixel value. In all current browsers, the spec will let the element grow to fix its own container and then base its calculated height for its children on that growth. With the prefixed properties, the calculated height is determined to be 0, and simply unchecking the prefixed values in devtools to let the spec values take precedent fixes it.
It appears the assumption that the prefixed properties are no longer being maintained when the spec properties are available is indeed true. So just a heads up, labeling this as a "question" seems less appropriate than "bug".


Can't run tests in archive_test.py

Hello, I was trying to run the tests by executing python tests/archive_test.py but I get this error:
  File "/lge/tests/archive_test.py", line 19, in <module>
    from lge.modules.inverse_model import LinearInverseModel
ModuleNotFoundError: No module named 'lge.modules.inverse_model'

Are the tests supposed to be runnable?

Thank you for reporting this.
Most of the tests are remnants from the development phase and need to be updated. I am actively working on it. I am keeping the issue open until this is resolved.

Ok, thank you!


No stack trace printed nor does process exit.

I was having some trouble with my Heroku app exiting with status 139. I installed segfault-handler, but now this is all I get. No stack trace.
Apr 11 13:31:14 my-app app/worker.2:  PID 3 received SIGSEGV for address: 0x0
The process doesn't exit anymore.
Please advise.

Attempts to reproduce the segfault on my laptop (OS X) are proving unsuccessful.

@jspavlick Can you checkout the repo and run the tests?

Thanks for the quick response.
Running example.js on my laptop (OS X) provides a wonderfully nice stack trace.
Laptop:
~> node example.js
NodeSegfaultHandlerNative: about to dereference NULL (will cause a SIGSEGV)
PID 44292 received SIGSEGV for address: 0x1
0   segfault-handler.node               0x00000001008de47a _ZL16segfault_handleriP9__siginfoPv + 282
1   libsystem_platform.dylib            0x00007fff92ddbf1a _sigtramp + 26
2   ???                                 0x00007fff5fbfef10 0x0 +<PHONE_NUMBER>02128
3   segfault-handler.node               0x00000001008de246 _Z22segfault_stack_frame_2v + 22
4   segfault-handler.node               0x00000001008de276 _Z13CauseSegfaultRKN2v89ArgumentsE + 38
5   node                                0x0000000100145b9a _ZN2v88internalL21Builtin_HandleApiCallENS0_12_GLOBAL__N_116BuiltinArgumentsILNS0_21BuiltinExtraArgumentsE1EEEPNS0_7IsolateE + 459
6   ???                                 0x00002f747c00618e 0x0 +<PHONE_NUMBER>1566
7   ???                                 0x00002f747c063df3 0x0 +<PHONE_NUMBER>5667

Heroku:
~> heroku run crash --app my-app
Running `crash` attached to terminal... up, run.6659
Detected 512 MB available memory, 512 MB limit per process (WEB_MEMORY)
Recommending WEB_CONCURRENCY=1
NodeSegfaultHandlerNative: about to dereference NULL (will cause a SIGSEGV)
PID 3 received SIGSEGV for address: 0x1
/app/node_modules/segfault-handler/build/Release/segfault-handler.node(+0x10e5)[0x7f33b06960e5]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x10340)[0x7f33b0c6d340]
/app/node_modules/segfault-handler/build/Release/segfault-handler.node(_Z22segfault_stack_frame_1v+0x10)[0x7f33b06961d0]
/app/node_modules/segfault-handler/build/Release/segfault-handler.node(_Z13CauseSegfaultRKN2v89ArgumentsE+0x1c)[0x7f33b069620c]
node[0x70fdc7]
[0x25ebbdd06362]
stty: standard input: unable to perform all requested operations

However, I receive no such stack dump when not running the example code.
Thanks.

I was just able to get my laptop node process to segfault. Notice node-segfault-handler did not help me. I'm not sure what to do next.
node(43906,0x7fff7536a300) malloc: *** error for object 0x10298d008: incorrect checksum for freed object - object was probably modified after being freed.
*** set a breakpoint in malloc_error_break to debug
/Library/Ruby/Gems/2.0.0/gems/foreman-0.77.0/lib/foreman/cli.rb:103:in `exit': no implicit conversion from nil to integer (TypeError)
	from /Library/Ruby/Gems/2.0.0/gems/foreman-0.77.0/lib/foreman/cli.rb:103:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/foreman-0.77.0/bin/foreman:7:in `<top (required)>'
	from /usr/bin/foreman:23:in `load'
	from /usr/bin/foreman:23:in `<main>'

I haven't worked with heroku so you will have to bear with me here
but it looks like heroku are using foreman as the process manager?
So heroku run crash --app my-app
Is really a wrapper around forman
If that's the case can you run the node process directly on the server as you would on your local machine?
If you want a process manager then https://www.npmjs.com/package/forever might help.
Other than that you could ask the forman folks what might be preventing the stacktrace from surfacing up

Hey @jspavlick I am going to close this as it's been around for a while.
If you have more input or further questions please feel free to spin up another issue.

I'm having a similar issue, not with not exiting but I have no stack trace:
root@beaglebone:~/test/node_modules/segfault-handler# node example.js
NodeSegfaultHandlerNative: about to dereference NULL (will cause a SIGSEGV)
PID 1907 received SIGSEGV for address: 0x2
root@beaglebone:~/test/node_modules/segfault-handler# uname -a
Linux beaglebone 4.1.19-bone-rt-r20 #1 Sun Mar 13 04:30:34 UTC 2016 armv7l GNU/Linux
Any ideas?

I found a solution: adding -funwind-tables to CFLAGS in binding.gyp fixes this problem (see http://stackoverflow.com/questions/24700150/on-raspberry-pi-backtrace-returns-0-frames). Test after doing node-gyp rebuild:
root@beaglebone:~/test/node_modules/segfault-handler# node example.js
NodeSegfaultHandlerNative: about to dereference NULL (will cause a SIGSEGV)
PID 2268 received SIGSEGV for address: 0x2
/root/test/node_modules/segfault-handler/build/Release/segfault-handler.node(+0x12a0)[0xb68ba2a0]
/lib/arm-linux-gnueabihf/libc.so.6(+0x26b00)[0xb68f1b00]
/root/test/node_modules/segfault-handler/build/Release/segfault-handler.node(_Z22segfault_stack_frame_1v+0x1f)[0xb68ba5c4]
/root/test/node_modules/segfault-handler/build/Release/segfault-handler.node(+0x1434)[0xb68ba434]

I can create a pull request, but I haven't tested it on x86/x86_64, any idea if this will have any repercussions?


space POIs not registering as discovered

after updating to <IP_ADDRESS>, space POIs no longer register as discovered upon traveling to them.

Details
(e.g. derelict ships remain as "ship" with no discovery xp and no change to the filled in explored icon. with named radiant POI ships like "repair 'n go", the name shows up on the starmap, but the location similarly doesn't register as discovered. not sure exactly how deep the problem goes, but just found a bunch of reddit posts about it, some expressing difficulty even with starstations. one of those posts said the bug affected the deimos armored transport and another mentioned the clinic remaining "starstation", so i'm guessing locations like the colander and re-939 might also be impacted.
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
No
Reproduction Steps
as above, hop in your ship and travel to a space POI. my apologies for not testing this in a new game myself, but some of those reddit posts did (so far, i've found 7 threads about the issue). note: space locations i'd already discovered prior to updating were not affected.

from 1.12.30 patch notes: "Fixed an issue that caused visited space locations to appear undiscovered." tested in game and the issue is indeed resolved.


[gym] Add option to pass in specific toolchain

This adds features for https://github.com/fastlane/fastlane/issues/1828 and https://github.com/fastlane/fastlane/issues/1079

Add toolchain option to BuildCommandGenerator and PackageCommandGeneratorXcode7

You can now use toolchain option like this:
gym(
  workspace: WORKSPACE,
  scheme: SCHEME,
  toolchain: "com.apple.dt.toolchain.Swift_2_3",
  configuration: "AdHoc",
  use_legacy_build_api: false,
  export_method: "ad-hoc"
)

@nafu Looks like you also need to add it to package_command_generator_legacy.rb like this to work if use_legacy_build_api: true is set.

@bfcrampton
Thanks for your comment.
I think legacy build api doesn't accept toolchain option.
$ xcrun /tmp/PackageApplication4Gym --help

warning: PackageApplication is deprecated, use `xcodebuild -exportArchive` instead.

Usage:
    PackageApplication [-s signature] application [-o output_directory]
    [-verbose] [-plugin plugin] || -man || -help

    Options:

        -s <signature>  certificate name to resign application before packaging
        -o              specify output filename
        -plugin         specify an optional plugin
        -help           brief help message
        -man            full documentation
        -v[erbose]      provide details during operation

I'll add codes to print messages in print_legacy_information .

Hey @nafu, thanks for your pull request, this looks great. I added one comment before it's ready 👍

Hey @nafu, thanks for clarifying, this looks great to me. Could you only update the description of the new value, with an example value of what you can set?
Thanks 👍

@KrauseFx
I added example values.
https://github.com/fastlane/fastlane/pull/5864/commits/4c5da69e74e0a71b205516c3c2fd0702e4f213a0
Is this what you meant above? Let me know if I misunderstood anything.

Great, thanks for updating. I added one last question before we can merge 👍

Great work, thank you @nafu, this will be in the next release 🚀

@KrauseFx
Thank you, I appreciate it.

Hi @nafu @KrauseFx I'm currently struggling with this too. Do you know if this has been added to version 1.102.0?
Thanks!

@gbejarano01 There was no gym release yet https://rubygems.org/gems/gym

Thanks @KrauseFx, that's what I thought, do you know when this can be included in a new version?

@gbejarano01 we're working on it 🚀

Nice! Thanks @KrauseFx!


Error while using the playbook on a IPv6 network

Bug Report
Hi everyone,
We are trying to deploy a small cluster of espresso-bin card (ARMbian) for a university project.
We wanted our cluster to be in an IPv6 network, so we created 2 networks :

Public network : fd17:67e8:6b1f::/64
Cluster network :  fd17:67e8:6b1f:ffff::/64

We want to deploy 1 monitor and 4 osds.
We are having an issue playing the ceph-ansible playbook.
Thanks for you time.
What happened:
Error while playing the playbook to deploy Ceph on a IPv6 network using Ansible.
Task "ceph-mgr: fetch ceph mgr keyring" failed due to error parsing server name.
TASK [ceph-mgr : fetch ceph mgr keyring] *******************************************************************************
task path: /home/ansible/cluster-ceph-k3s/ceph-ansible/roles/ceph-mgr/tasks/common.yml:10
Friday 10 January 2020  14:09:10 +0000 (0:00:01.492)       0:12:09.814 ********
Using module file /home/ansible/cluster-ceph-k3s/ceph-ansible/library/ceph_key.py
Pipelining is enabled.
<localhost> ESTABLISH SSH CONNECTION FOR USER: None
<localhost> SSH: EXEC ssh -vvv -o ControlMaster=auto -o ControlPersist=600s -o StrictHostKeyChecking=no -o KbdInteractiveAuthentication=no -o PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey -o PasswordAuthentication=no -o ConnectTimeout=60 -o ControlPath=/home/ansible/.ansible/cp/%h-%r-%p localhost '/bin/sh -c '"'"'sudo -H -S -n  -u root /bin/sh -c '"'"'"'"'"'"'"'"'echo BECOME-SUCCESS-fgaofkzbloisvivnccqbdvhonzzyeoxv ; CEPH_CONTAINER_BINARY=docker CEPH_CONTAINER_IMAGE='"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"''"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"'"' /usr/bin/python'"'"'"'"'"'"'"'"' && sleep 0'"'"''
Escalation succeeded
<localhost> (1, '\n{"changed": true, "end": "2020-01-10 14:09:43.657866", "stdout": "server name not found: v2:[fd17:67e8:6b1f::1 (Name or service not known)", "cmd": ["ceph", "-n", "client.admin", "-k", "/etc/ceph/ceph.client.admin.keyring", "--cluster", "ceph", "auth", "import", "-i", "/var/lib/ceph/mgr/ceph-espressobin/keyring"], "rc": 1, "failed": true, "stderr": "unable to parse addrs in \'[v2:[fd17:67e8:6b1f::1]:3300,v1:[fd17:67e8:6b1f::1]:6789]\'\\n[errno 22] error connecting to the cluster", "delta": "0:00:31.776397", "invocation": {"module_args": {"directory_mode": null, "force": null, "remote_src": null, "mode": "0400", "cluster": "ceph", "owner": "ceph", "follow": false, "group": "ceph", "unsafe_writes": null, "secret": null, "content": null, "serole": null, "setype": null, "state": "present", "dest": "/var/lib/ceph/mgr/ceph-espressobin/keyring", "selevel": null, "caps": {"mds": "allow *", "mon": "allow profile mgr", "osd": "allow *"}, "regexp": null, "src": null, "name": "mgr.espressobin", "seuser": null, "delimiter": null, "import_key": true, "attributes": null, "backup": null}}, "start": "2020-01-10 14:09:11.881469", "msg": "non-zero return code"}\n', 'OpenSSH_7.6p1 Ubuntu-4ubuntu0.3, OpenSSL 1.0.2n  7 Dec 2017\r\ndebug1: Reading configuration data /etc/ssh/ssh_config\r\ndebug1: /etc/ssh/ssh_config line 19: Applying options for *\r\ndebug1: auto-mux: Trying existing master\r\ndebug2: fd 3 setting O_NONBLOCK\r\ndebug2: mux_client_hello_exchange: master version 4\r\ndebug3: mux_client_forwards: request forwardings: 0 local, 0 remote\r\ndebug3: mux_client_request_session: entering\r\ndebug3: mux_client_request_alive: entering\r\ndebug3: mux_client_request_alive: done pid = 3160\r\ndebug3: mux_client_request_session: session request sent\r\ndebug1: mux_client_request_session: master session id: 2\r\ndebug3: mux_client_read_packet: read header failed: Broken pipe\r\ndebug2: Received exit status from master 1\r\n')
<localhost> Failed to connect to the host via ssh: OpenSSH_7.6p1 Ubuntu-4ubuntu0.3, OpenSSL 1.0.2n  7 Dec 2017
debug1: Reading configuration data /etc/ssh/ssh_config
debug1: /etc/ssh/ssh_config line 19: Applying options for *
debug1: auto-mux: Trying existing master
debug2: fd 3 setting O_NONBLOCK
debug2: mux_client_hello_exchange: master version 4
debug3: mux_client_forwards: request forwardings: 0 local, 0 remote
debug3: mux_client_request_session: entering
debug3: mux_client_request_alive: entering
debug3: mux_client_request_alive: done pid = 3160
debug3: mux_client_request_session: session request sent
debug1: mux_client_request_session: master session id: 2
debug3: mux_client_read_packet: read header failed: Broken pipe
debug2: Received exit status from master 1
fatal: [localhost]: FAILED! => changed=true
  cmd:
  - ceph
  - -n
  - client.admin
  - -k
  - /etc/ceph/ceph.client.admin.keyring
  - --cluster
  - ceph
  - auth
  - import
  - -i
  - /var/lib/ceph/mgr/ceph-espressobin/keyring
  delta: '0:00:31.776397'
  end: '2020-01-10 14:09:43.657866'
  invocation:
    module_args:
      attributes: null
      backup: null
      caps:
        mds: allow *
        mon: allow profile mgr
        osd: allow *
      cluster: ceph
      content: null
      delimiter: null
      dest: /var/lib/ceph/mgr/ceph-espressobin/keyring
      directory_mode: null
      follow: false
      force: null
      group: ceph
      import_key: true
      mode: '0400'
      name: mgr.espressobin
      owner: ceph
      regexp: null
      remote_src: null
      secret: null
      selevel: null
      serole: null
      setype: null
      seuser: null
      src: null
      state: present
      unsafe_writes: null
  msg: non-zero return code
  rc: 1
  start: '2020-01-10 14:09:11.881469'
  stderr: |-
    unable to parse addrs in '[v2:[fd17:67e8:6b1f::1]:3300,v1:[fd17:67e8:6b1f::1]:6789]'
    [errno 22] error connecting to the cluster
  stderr_lines: <omitted>
  stdout: 'server name not found: v2:[fd17:67e8:6b1f::1 (Name or service not known)'
  stdout_lines: <omitted>

What you expected to happen:
We expected the playbook to fully configure the Ceph network.
How to reproduce it (minimal and precise):
Put IPv6 address in monitor_address.
Share your group_vars files, inventory
Find all the Ansible config at https://github.com/TGuimbert/cluster-ceph-k3s
More precisely at : https://github.com/TGuimbert/cluster-ceph-k3s/tree/master/ceph-ansible
Inventory file : inventory.ini (in the ceph-ansible folder)
Environment:

OS (e.g. from /etc/os-release): ARMbian/ based on Ubuntun 18.04
Kernel (e.g. uname -a): Linux espressobin 4.19.85-mvebu64
Docker version if applicable (e.g. docker version): None
Ansible version (e.g. ansible-playbook --version): 2.8.7
ceph-ansible version (e.g. git head or tag or stable branch):master
Ceph version (e.g. ceph -v): nautilus

ceph-ansible version (e.g. git head or tag or stable branch):master
 Ceph version (e.g. ceph -v): nautilus

Please use the stable-4.0 branch or v4.0.x tag for deploying nautilus
https://docs.ceph.com/ceph-ansible/master/#releases
Also I can find anything usefull at https://github.com/TGuimbert/cluster-ceph-k3s/tree/master/ceph-ansible.
Could you point me to your ansible inventory and group_vars variables ?

Hi,
inventory : https://github.com/TGuimbert/cluster-ceph-k3s/blob/master/ceph-ansible/inventory.ini
Oh sorry we forgot to modify gitignore so our config files are not in the git sent.
I will try to add it tomorrow (not in school currently)

@dsavineau
Just added upload the documents we modified in group_vars:
https://github.com/TGuimbert/cluster-ceph-k3s/tree/master/ceph-ansible/group_vars
Thanks in advance,


Added a new Medium impact entry for SPFx isolated solutions

@MikePlumleyMSFT
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


this.And throws an error

I have created a cucumber scenario that has the following lines:
Given I am on the Cart page
When I change the quantity of <Title> to <Quantity>
And I click Update cart button
Then I should see <Total> as Total of the item <Title>, <Overall Total> as Overall Total and <Grand Total> as the Grand Total

But when I execute the test, it throws an error that is shown on the screenshot:

Is this a legit issue or am I missing something? Thanks!

Hey @marktigno,
I'm guessing you're trying to define a cucumber step def along the lines of:
this.And('...', function() {})

Try doing this instead:
this.Given('...', function() {});

Did that help?

Hello @jan-molak,
Yeah I did that by replacing "And" with either "Given", "When" or "Then". But some of our scenarios are written on low-level format which includes "And" and "But".

I agree that the example scenarios you have provided are UI-focused and low-level.
Because of this, it might seem like using multiple Ands and Buts was needed, while in fact it highlights an issue with the abstraction level of the scenario.
Check out this excellent post.
Also, please note that the And and But keywords are just aliases for whatever step has occurred before them - see the source code.


chore(ci): invalidate CDN cache for latest & major versions (all env)

https://coveord.atlassian.net/browse/KIT-1171
Based myself of this CLI PR https://github.com/coveo/cli/pull/373 with the difference of implementing it for all environments (helps QA for example that staticdev is updated correctly)
Created a job here https://searchuibuilds.dev.cloud.coveo.com/job/ui-kit-cloudfront-invalidation/
I'm wondering if we have access to the same aws CloudfrontCacheInvalidation credentials (should be since we use the same jenkins)

I'm wondering if we have access to the same aws CloudfrontCacheInvalidation credentials (should be since we use the same jenkins)

Yes, that should not be a problem.


THRIFT-3595 Perl: Unify implementation of set as a hashref

https://issues.apache.org/jira/browse/THRIFT-3595

I have an open backlog item to use Set::Scalar for perl sets.  https://issues.apache.org/jira/browse/THRIFT-3190

Closing based on 2 years on inactivity; superceded by THRIFT-3190


fix: remove upper limit for required terraform version

/run pipeline

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


Only receive one notification per version.

Description of the new feature / enhancement
I would really appreciate having the option to only receive a single update notification when a new version is out.
If I ignore an update for some reason, I don't want to receive a notification every day until I either disable notifications out of rage or give in and update even if I don't want to.
Once is enough to know a new version is out for many people, they won't all forget about it every day.
Scenario when this would be used?
To reduce the amount of notifications if an update is ignored.
Supporting information
No response

/dup #25717

Doesn't sound like the same thing to me, none of the options they mention in that thread would notify me of a new version after I ignored the first notification of the next version available.
I, for example, would want to get a notification again when 0.71.1 releases after I ignore the update notification for 0.71.0 and not just a reminder that an update is pending.


edgeCollection().create() is creating a document collection instead of an edge collection

The problem seems to be that GraphEdgeCollection is inheriting from BaseCollection instead of EdgeCollection.

Thanks for catching that.


Relationships with expiry / TTL

Hey authzed team! 👋🏻  Opening up this issue to gauge interest on relationships with expiration
The ask
Write API should make it possible to write relationships with an expiration. Past the expiration, any operation that makes use of the relationship (Read, Check, Lookup, Expand... ) would effectively treat it as non-existent.

Native support is now being tested: https://github.com/authzed/spicedb/pull/2152


Approve with WalletConnect session?

I'm connecting to metamask app via wallet_connect_dart and want to swap DAI to ETH, so i want metamask to  approve the smart contract to have some DAI but i dont know how ?

Hi,
You need to create an ERC20 object with DAI smart contract as the address.
final dai = ERC20(
    address: EthereumAddress.fromHex(
        '0x6b175474e89094c44da98b954eedeac495271d0f'),
    client: client,
);

Then call dai.balanceOf.
Please look at the example usage in the README.md file.
Does this solve your problem?

The client is from infuria node right? but i'm connecting to Metamask mobile app with this package https://pub.dev/packages/walletconnect_dart. So how can i use your package with metamask connection? I'm trying to swap erc20 tokens with Uniswap smart contract

`
Future transferTokensToSmartContract(List args) async {
DeployedContract contract =
await getContract(erc20ContractAddress, erc20ContractAbi);
print(contract.address);
ContractFunction transferFunc = contract.function('transferFrom');
// ContractFunction symbol = contract.function('symbol');
return transferFunc.encodeCall(args);
}
var xx = await uniSc.transferTokensToSmartContract([
EthereumAddress.fromHex(session.accounts.first),
EthereumAddress.fromHex('0x7a250d5630B4cF539739dF2C5dAcb4c659F2488D'),
BigInt.from(0.00001 * pow(10, 18)),
]);
    var a = await EthereumWalletConnectProvider(connector).sendTransaction(
        from: session.accounts[0],
        to: '0x7a250d5630B4cF539739dF2C5dAcb4c659F2488D',
        data: xx,
        value: BigInt.from(0.00001 * pow(10, 18)),
        gas: 1000000,
        gasPrice: BigInt.from(900000));

`
This is what is do

If you want to check if the wallet has some DIAs, it doesn't matter where you are querying the account balance. Infura in the example can be replaced with anything.

i want to swap dai for eth through uniswap v2 smart contract. and i can't access the amount of dai in my metamask wallet


Misc config clean up

[ ] Verify whether we are using ring-defaults and/or transit-clj and remove them from deps.edn if not

[ ] Rename resources/public/downloads/DO_NOT_DELETE to .do_not_delete

[ ] Remove .meghanada from .gitignore and run sort-lines on entries by section

[ ] Add :check-deps, :check-reflection, and enhanced :rebel aliases to deps.edn and document them in README.org

[ ] Add a ":build-db only-changes" alias to deps.edn and document it in README.org

[ ] Document how to use announcement.txt in README.org

Gary added all these back as separate items


Unexpected ConditionalExpression for assignment

Summary
When assigning a displayName for a React component Stryker mutates this using a ConditionalExpression even though there is no condition here at all.
This is part of the following React component:
import React, { ReactElement, ReactChild } from "react";

type LabelProps = {
  caption: ReactChild;
  name?: string;
};

const Label = ({ caption, name }: LabelProps): ReactElement => {
  return <label htmlFor={name}>{caption}</label>;
};

Label.displayName = "Label";

export default Label;

Stryker config
// This config was generated using a preset.
// Please see the handbook for more information: https://github.com/stryker-mutator/stryker-handbook/blob/master/stryker/guides/react.md#react
module.exports = function(config) {
  config.set({
    mutate: [
      "src/**/*.ts?(x)",
      "!src/*.ts?(x)",
      "!src/**/*@(.test|.spec|Spec).ts?(x)"
    ],
    mutator: "typescript",
    testRunner: "jest",
    reporters: ["progress", "clear-text", "html"],
    coverageAnalysis: "off",
    tempDirName: "stryker-tmp",
    jest: {
      projectType: "react"
    }
  });
};

Test runner config
This is a non ejected Create-React-App application using Jest.
Stryker environment
+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>deduped
| +--<EMAIL_ADDRESS>deduped
+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>deduped
+--<EMAIL_ADDRESS>| +--<EMAIL_ADDRESS>deduped
| +--<EMAIL_ADDRESS>deduped```

    "react-scripts": "3.2.0",

Test runner environment
 "test": "react-scripts test",

Your Environment

software
version(s)

node
v10.15.3

npm
6.11.3

Operating System
Windows 10 Version 1809

Add stryker.log
stryker.log

Well well well... Turns out TypeScript classifies this as a BinaryExpression of all things..

In this case, the EqualsToken should be added to the invalid operators:
https://github.com/stryker-mutator/stryker/blob/f2ff418d73c94b79c5e823cb3f96b8caa52f70bc/packages/typescript/src/mutator/ConditionalExpressionMutator.ts#L25-L37


nixos/ircdHybrid fails to build with services.ircdHybrid.enable set to true

Describe the bug
On attempting to build the ircdHybrid NixOS module, I get the following error:
# nixos-rebuild switch
building Nix...
building the system configuration...
error: A definition for option `systemd.services.ircd-hybrid.after."[definition 1-entry 1]"' is not of type `string matching the pattern [a-zA-Z0-9@%:_.\-]+[.](service|socket|device|mount|automount|swap|target|path|timer|scope|slice)'. Definition values:
       - In `/nix/var/nix/profiles/per-user/root/channels/nixos/nixos/modules/services/networking/ircd-hybrid/default.nix': "started networking"
(use '--show-trace' to show detailed location information)

Steps To Reproduce
Steps to reproduce the behavior:

Add services.ircdHybrid.enable = true; to configuration.nix
nixos-rebuild switch

Expected behavior
I expected ircd-hybrid to be automatically configured using my specified settings.
Metadata
 - system: `"aarch64-linux"`
 - host os: `Linux 6.1.21, NixOS, 23.05 (Stoat), 23.05.2507.ad73028def6`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"nixos-23.05-small"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Looks like this was fixed in unstable, https://github.com/NixOS/nixpkgs/commit/c80398e5d2325c31a4a884b407607b34aa752abc
Workaround for 23.05:
systemd.services.ircd-hybrid = {
  wants = [ "network-online.target" ];
  after = lib.mkForce [ "network-online.target" ];
};

Unfortunately this still doesn't result in a working ircdHybrid...

Cool, thanks for letting me know :+1:


fix PR 'split wire with link nodes' #3416

Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes

fully qualify call to calculateGridSnapOffsets
use .x and .y properties of calculated offsets (not array indexes)

Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 67.323% when pulling c948573c2d7a887e103f93ead6e92821c506a0c4 on Steve-Mcl:fix-calculateGridSnapOffsets into 0e92f68b4ad3582fbb0b2f49f294db7951959ba1 on node-red:dev.


lost package-lock.json indentation

Line of code: here
You can see that 2-space-indentation is hardcoded for new package-lock.json file.
Is it possible to detect package-lock.json indentation? And if no is it possible to make option "tabWidth" for set this value?

It will be better to detect the origin indent from previous package-lock.json.
Can you send PR since i am locked on another open source project?

Yes I can. Thank you.

Fixed in 1.0.6.


Empty key names accepted

JSON:
"" : "value"
Result:
DotDict({'': 'value'})
Empty strings are accepted as key name. Here I have doubts.
Cross check:
"   " : "value"
Result:
Error: 'Invalid key name: "   ". Key names have to start with a letter, digit or underscore.'!
That is OK, and in my opinion also an empty string should be an invalid key name.

Yes, empty string as key must cause an error message.

The fix was pushed to stabi branch
Ready for ROBFW-AIO 0.13.2


tests/specs: elliminating #getKLabel

Fixes #725

LGTM

Please fix the lemma duplication, then we can merge.


interpolate_bads did not work

Description of the problem
Hello all! I have a problem interpolating bad channels of MEG data.
Here is my MEG data description:

I need to interpolate the MEG 079 channel based on other channels.
The original data looks like this:

After I called the meg_data.copy().interpolate_bads(reset_bads=False), it seemed that the MEG 079 channel was not changed.

I’m wondering what’s wrong during the process. I’d appreciate it if anyone could help! Thanks a lot!
Steps to reproduce
raw.info['bads']=['MEG 079']
raw.load_data()
meg_data = raw.copy().filter(l_freq=1, h_freq=40)
meg_data_interp = meg_data.copy().pick_types(meg=True, exclude=[])
meg_data_interp = meg_data_interp.interpolate_bads(reset_bads=False)
picks = mne.pick_channels(meg_data_interp.ch_names, include=['MEG 078','MEG 079','MEG 080'])
meg_data_interp.plot(duration=12, order=picks, n_channels=len(picks))

Link to data
No response
Expected results
interpolated data of MEG 079 channel
Actual results
no change to the data
Additional information
I'm using the data from https://osf.io/ag3kj/
I'm using sub-01_ses-1_task-1_meg.con and corresponding marker, elp, and hsp files
mne.sys_info():

I asked in the MNE Forum but the problem has not been solved. Here is the link:
https://mne.discourse.group/t/interpolate-bads-did-not-work/6531/9
Thanks a lot if anyone could help!

You are using an old MNE version, can you update to the latest version and see if the problem still persists?

No, the problem still exists.

I'm now working with the newest version.

And I found that it seems the interpolation was done, but made the initial values of channel 079 much smaller (the initial values of this channel was smaller than other channels but not zero).
Here is the initial values of channel 079:

And this is values after interpolation:

This is another channel (channel 078):

Do you know why the interpolation works like this? Thanks a lot!

Sadly, the interpolation seemed to fail again.

(Can it be because I did not regress the data based on the reference sensors at first? But I did not see this step in the tutorial. )

I may provide more information for this problem (I'm afraid it's because I'm doing something wrong before interpolation).
My data is from https://osf.io/ag3kj/. Currently the sample data for this problem is sub-01_ses-1_task-1. And following is all my codes:
import mne
import numpy as np
import pandas as pd
from mne.preprocessing import (ICA, corrmap, create_ecg_epochs, create_eog_epochs)

data_folder = '/bigvault/Projects/asymmetry_lang/MEG/sub-01/ses-1/meg'
data_raw_file = data_folder+'/sub-01_ses-1_task-1_meg.con'
marker_file_name = data_folder+'/sub-01_ses-1_task-1_markers.mrk'
marker_data = mne.io.kit.read_mrk(marker_file_name)
elp_file_name = data_folder+'/sub-01_ses-1_acq-ELP_headshape.pos'
hsp_file_name = data_folder+'/sub-01_ses-1_acq-HSP_headshape.pos'

def readPos(filename, start_line, point_num):
    df = pd.read_table(filename, header=None)
    line_num = len(df.values)-3
    save_data_row = []
    item = ''
    indicate_num = 0
    flag = False
    for line_index, m in enumerate(df.values[start_line:]):
        save_data_line = []
        for idx, n in enumerate(df.values[start_line+line_index][0].replace(' ','')):
            if flag:
                indicate_num += 1
            item += n
            if indicate_num == point_num:
                save_data_line.append(float(item))
                item = ''
                indicate_num = 0
                flag = False
            if n == '.':
                flag = True
        save_data_row.append(save_data_line)
    return np.array(save_data_row)

elp_data = readPos(elp_file_name, 3, 4)
hsp_data = readPos(hsp_file_name, 4, 2)

raw = mne.io.read_raw_kit(data_raw_file, mrk=marker_data/1000, elp=elp_data/1000, hsp=hsp_data/1000)
raw.info['bads']=['MEG 079']
raw.load_data()

meg_data = raw.copy().filter(l_freq=1, h_freq=40)
ref_channels = ['MEG 209','MEG 210','MEG 211','MEG 212','MEG 213','MEG 214','MEG 215','MEG 216',
               'MEG 217','MEG 218','MEG 219','MEG 220','MEG 221','MEG 222','MEG 223','MEG 224']
meg_data_interp = meg_data.copy()
meg_data_interp = meg_data_interp.interpolate_bads(reset_bads=False, exclude=ref_channels)
picks = mne.pick_channels(meg_data_interp.ch_names, include=['MEG 078','MEG 079','MEG 080'])
meg_data_interp.plot(duration=12, order=picks, n_channels=len(picks))

I think it's not a good idea to discuss this issue in both our Discourse forum and here. Would it be OK if I closed this issue and we continue on Discourse? If it turns out that this is indeed a bug, we can always re-open this issue.

I think it's not a good idea to discuss this issue in both our Discourse forum and here. Would it be OK if I closed this issue and we continue on Discourse? If it turns out that this is indeed a bug, we can always re-open this issue.

Sure. Thanks a lot for your time and help!

Great! Continuing the discussion here: https://mne.discourse.group/t/interpolate-bads-did-not-work/6531


IndexError because pytorch does not support int32 for indexing like onnx

Been trying a bunch of things to solve this now but the error message isn't very helpful. It just sounds like it's just the input dtype that's handled incorrectly.
  File "/home/richard/miniconda3/envs/3.9.13/lib/python3.9/site-packages/torch/fx/graph_module.py", line 630, in wrapped_call
    raise e.with_traceback(None)
IndexError: tensors used as indices must be long, byte or bool tensors

Here is the onnx file I'm trying to convert
https://drive.google.com/file/d/1FX_D6dcYEoVssr-y29F5RLbhmhf4RmyZ/view?usp=sharing
It is supposed to take a tensor of type long. Here's an example in json:
https://drive.google.com/file/d/1yTSxwOY10g0cULEVt3KQZWSzziDpD8bC/view?usp=sharing
This is how I try to run it:
    model = torch.load(torch_path).eval().requires_grad_(False)

    tokens = torch.tensor(
        json.loads(Path(f"tests/{model_name}/tokens.json").read_text())
    ).long()
    text_encodings = torch.from_numpy(np.array(model(tokens)))

The onnx file works with onnxruntime and gives the correct result.
    tokens = np.array(
        json.loads(Path(f"tests/{model_name}/tokens.json").read_text())
    ).astype(np.int64)

    providers = ["CPUExecutionProvider"]
    m = rt.InferenceSession(onnx_path, providers=providers)
    text_encoding = m.run(output_names, dict(inputs=tokens))

Do you have any tips for what could be wrong?

After some more debugging I think that that since the model is converted from JAX it's using int32 when slicing embeddings rather than long/int64. That works fine with onnx but isn't supported in pytorch
I tried converting explicitly to int64 inside the model and using jax.config.update("jax_enable_x64", True) when exporting but it didn't help

A similar issue also seems to happen when trying to use the model in onnxjs where it does not support int64 and they need to convert to int32. They insert cast operations into the graph to solve this type of issue here
https://github.com/microsoft/onnxruntime/blob/c6732c079b4fbf644bc939729c87d70195251d83/onnxruntime/python/tools/transformers/onnx_model_bert.py#L125-L202
Been trying to modify my onnx to replicate this for a while.

Hello, samedii.
We will look into the issue and try to come up with a solution next week.

@senysenyseny16 Thank you! I had to give up eventually on fixing the ONNX file manually. At the moment I'm running the text encoder via ONNX because it's not used as often as the image encoder (text-guided image generation) but it would be very nice if it wasn't necessary.
The package is open source if you want to know what it would be used for: https://github.com/samedii/pytorch-zero-lit

Ping

Hi @samedii
Sorry for the wait, we will get back to this issue as soon as possible.

Just showing there's still interest 🙂

Hi @samedii, fix in main branch.

Thank you! 🙏


TypeError: Ee.P.api is undefined

I am getting a TypeError: Ee.P.api is undefined when chosing a Mosaic visualization and then changing any of the options or entering some query for a postgreSQL data source.
I am using Grafana 10.4.0.
The rest of the error message is rather cryptic and contains seemingly random JS files with high line-numbers, but if you tell me how I can provide you with more information on how to fix it, I'd be happy to.

Hi @Lumiukko , can you provide a screenshot/paste of the full error message?  Also, by hanging any of the options are you referring to the options in the postgresql datasource, or in the Mosaic plot configuration?

I was referring to the Mosaic plot configuration on the right side. As soon as I change anything in that panel, the error occurs. It also happens when I enter some query in the bottom configuration, for example an SQL query.
The full error message reads like this (domain.com was chosen to censor the original URL):
ypeError: Ee.P.api is undefined

r@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:194:122062
div
div
x@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:214:30846
It@http://domain.com:8443/public/build/9569.a8d60b39654a160cb917.js:1:17495
Ce@http://domain.com:8443/public/build/9569.a8d60b39654a160cb917.js:1:26196
at@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:412:32691
div
div
div
b@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:454:100807
div
div
div
o@http://domain.com:8443/public/build/660.838743295c2108f3492d.js:1:2300
div
o@http://domain.com:8443/public/build/660.838743295c2108f3492d.js:1:4288
N@http://domain.com:8443/public/build/660.838743295c2108f3492d.js:1:9622
div
o@http://domain.com:8443/public/build/660.838743295c2108f3492d.js:1:2300
div
o@http://domain.com:8443/public/build/660.838743295c2108f3492d.js:1:4288
N@http://domain.com:8443/public/build/660.838743295c2108f3492d.js:1:9622
div
div
div
L@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:357:21098
Bn@http://domain.com:8443/public/build/DashboardPageProxy.9a6d60e5cac89c11c71c.js:130:526
at@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:412:32691
WithTheme(Connect(Bn))
yt@http://domain.com:8443/public/build/DashboardPageProxy.9a6d60e5cac89c11c71c.js:205:472
DashboardPage
at@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:412:32691
ms@http://domain.com:8443/public/build/DashboardPageProxy.9a6d60e5cac89c11c71c.js:205:6297
Suspense
r@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:194:122062
Ru@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:5705:9531
I@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:444:29242
di@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:435:4481
ni@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:435:12022
ht@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:444:17401
I@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:444:31410
div
div
main
div
Xf@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:5705:2385
di@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:435:4481
ni@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:435:12022
cr@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:435:11439
Hn@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:444:17692
I@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:444:25474
div
o@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:214:13882
h@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:136:9122
f@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:773:96558
d@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:499:21142
r@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:194:122062
g@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:194:122571
it@http://domain.com:8443/public/build/3644.93bade7d48e3d75173f7.js:416:521
mg@http://domain.com:8443/public/build/5017.830b14490d17e9cbdabb.js:5716:122

yeah... that error message doesn't really contain any useful information...
I'll try to setup a local environment with the postgreSQL data source and the same Grafana version and see if I can reproduce.

Yup, that's why I called it cryptic :D
I'm not sure if the data source is the issue here, but I am not that familiar with the Grafana ecosystem and was hoping someone already knew what may be the issue. I'd also be happy to help test possible solutions, re-installing.
My Grafana installation is running in a docker container (with the Grafana 10.4.0 image from docker hub), where I basically copied the folder into the respective plugin folder. Perhaps here something went wrong.

Can you test it with the TestData datasource?  That way we can tell if the issue is related to the postgreSQL datasource

It works with "Random Walk" from the TestData data source, but if I choose certain types, this error occurs again, one of them was "No Data" and another was "USA generated data".

but if I choose certain types, this error occurs again, one of them was "No Data" and another was "USA generated data".

Does that mean you see the TypeError: Ee.P.api is undefined error with the TestData datasource, or only the other errors (which are expected, those types would require certain transforms)

It is exactly the same error as above.
When the data does not match the expected format and has to be transformed first, I usually get the option to display the data as a table, but there is no "crash" with an error message.
I have attached a screenshot showing the error above (pixelized the domain, though).

It is exactly the same error as above.
When the data does not match the expected format and has to be transformed first, I usually get the option to display the data as a table, but there is no "crash" with an error message.
I have attached a screenshot showing the error above (pixelized the domain, though).

I'm able to reproduce the error now 👍 .  Will take a look into it.

Hi @Lumiukko , it looks like this error is due to panel monitoring feature from about 8 months ago.
I verified that if I disable that feature, I don't get the error anymore.
To disable it, just add panelMonitoring = false in your .ini configuration file


Trouble in "Train your own vGRF model for footskate cleanup"

hello，
when I run:
python -m utils.scripts.train_UnderPressure_model --dataset_name t2m

the problem is:
Traceback (most recent call last):
  File "stablemofusion/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "stablemofusion/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "StableMoFusion/utils/scripts/train_UnderPressure_model.py", line 93, in <module>
    skeleton = Skeletons.all()[0].to(device)
  File "StableMoFusion/UnderPressure/skeletons.py", line 17, in all
    files = [next(s.preprocessed_files()) for s in Subject.all()]
  File "StableMoFusion/UnderPressure/skeletons.py", line 17, in <listcomp>
    files = [next(s.preprocessed_files()) for s in Subject.all()]
TypeError: 'list' object is not an iterator

I change code in UnderPressure/skeletons.py:
class Skeletons:
	@classmethod
	def all(cls):
		files = [next(iter(s.preprocessed_files())) for s in Subject.all()]
		return torch.cat([torch.load(file)["skeleton"] for file in files])

	@classmethod
	def train(cls):
		files = [next(iter(s.preprocessed_files())) for s in Subject.train()]
		return torch.cat([torch.load(file)["skeleton"] for file in files])

	@classmethod
	def test(cls):
		files = [next(iter(s.preprocessed_files())) for s in Subject.test()]
		return torch.cat([torch.load(file)["skeleton"] for file in files])

the problem is:
File "stablemofusion/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "stablemofusion/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "StableMoFusion/utils/scripts/train_UnderPressure_model.py", line 93, in <module>
    skeleton = Skeletons.all()[0].to(device)
  File "StableMoFusion/UnderPressure/skeletons.py", line 33, in all
    files = [next(iter(s.preprocessed_files())) for s in Subject.all()]
  File "StableMoFusion/UnderPressure/skeletons.py", line 33, in <listcomp>
    files = [next(iter(s.preprocessed_files())) for s in Subject.all()]
StopIteration

it seems that "files" is empty, how to fix it?

hello， when I run:
python -m utils.scripts.train_UnderPressure_model --dataset_name t2m

the problem is:
Traceback (most recent call last):
  File "stablemofusion/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "stablemofusion/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "StableMoFusion/utils/scripts/train_UnderPressure_model.py", line 93, in <module>
    skeleton = Skeletons.all()[0].to(device)
  File "StableMoFusion/UnderPressure/skeletons.py", line 17, in all
    files = [next(s.preprocessed_files()) for s in Subject.all()]
  File "StableMoFusion/UnderPressure/skeletons.py", line 17, in <listcomp>
    files = [next(s.preprocessed_files()) for s in Subject.all()]
TypeError: 'list' object is not an iterator

I change code in UnderPressure/skeletons.py:
class Skeletons:
	@classmethod
	def all(cls):
		files = [next(iter(s.preprocessed_files())) for s in Subject.all()]
		return torch.cat([torch.load(file)["skeleton"] for file in files])

	@classmethod
	def train(cls):
		files = [next(iter(s.preprocessed_files())) for s in Subject.train()]
		return torch.cat([torch.load(file)["skeleton"] for file in files])

	@classmethod
	def test(cls):
		files = [next(iter(s.preprocessed_files())) for s in Subject.test()]
		return torch.cat([torch.load(file)["skeleton"] for file in files])

the problem is:
File "stablemofusion/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "stablemofusion/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "StableMoFusion/utils/scripts/train_UnderPressure_model.py", line 93, in <module>
    skeleton = Skeletons.all()[0].to(device)
  File "StableMoFusion/UnderPressure/skeletons.py", line 33, in all
    files = [next(iter(s.preprocessed_files())) for s in Subject.all()]
  File "StableMoFusion/UnderPressure/skeletons.py", line 33, in <listcomp>
    files = [next(iter(s.preprocessed_files())) for s in Subject.all()]
StopIteration

it seems that "files" is empty, how to fix it?

I am sorry for this issue. We've noticed that many people are experiencing this issue because the dataset in the UnderPressure are incomplete, which is needed in our original code to obtain their skeleton data. To make this issue easier, we've updated the code scripts/train_UnderPressure_model.py and utils/footskate_clean.py, and provided a processed UnderPressure skeleton data in ./UnderPressure/dataset/S1_HoppingLeftFootRightFoot.pth.  In this way, you can run our scripts without loading and process the dataset in UnderPressure.


MINOR: Avoid conflicting versions of org.apache.mina for MiniKdc deps

We have updated apacheda to a newer version that uses mina-core-2.0.18 while apacheds uses mina-core-2.0.16. Since these are used only for testing using MiniKdc, it would be better to use the versions from apacheds.
Committer Checklist (excluded from commit message)

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

@ijuma Thanks for the review. Test failure not related, merging to trunk.


fix Microsoft Azure Batch Readme issue

fix the issue #18805
@jongio  for notification

@jongio - Can you help ping the corresponding person to review this PR?

@joshfree - Who should review this?

@joshfree - Could you help us review this PR?

/check-enforcer evaluate


Added a replyLocale method to KlasaMessage

Description of the PR
I added a replyLocale method to KlasaMessage which will be useful instead of doing message.reply(message.language.get('KEY')) or message.sendLocale('KEY', { reply: msg.author }) you can now do message.replyLocale('KEY')
Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)

Semver Classification

[ ] This PR only includes documentation or non-code changes.
[ ] This PR fixes a bug and does not change the (intended) framework interface.
[x] This PR adds methods or properties to the framework interface.
[ ] This PR removes or renames methods or properties in the framework interface.

I don't think many people use Message#reply to begin with. Personally, I don't think this is something that belongs in core, but it would be accepted in Klasa Pieces.

Also, the linting failed for trailing whitespace.

Closing due to @tech6hutch. We'll accept your PR at klasa-pieces!


Extract supported technologies to separate page

Codecov Report

Merging #196 into master will increase coverage by 0.12%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master    #196      +/-   ##
===========================================
+ Coverage     72.57%   72.7%   +0.12%
  Complexity      991     991
===========================================
  Files           111     111
  Lines          3858    3858
  Branches        358     358
===========================================
+ Hits           2800    2805       +5
+ Misses          897     890       -7
- Partials        161     163       +2

Impacted Files
Coverage Δ
Complexity Δ

...in/java/co/elastic/apm/report/ReporterFactory.java
66.66% <0%> (+1.96%)
6% <0%> (ø)
:arrow_down:

.../co/elastic/apm/report/ApmServerHealthChecker.java
81.81% <0%> (+18.18%)
3% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bde5f2d...8e4a399. Read the comment docs.


Round rating to one decimal place in orders

Until now all orders that included a rating displayed a round number without decimals, but this is a result of the addition of the same number multiple times, and is not enforced by the format of the string. As you can see in the attached image, when the user receives mixed ratings, the numbers become strange fractions and reduce their readability.
I suggest to round the rating number to one decimal place.

Sounds good to me


Listening to window.toggle-full-screen using atom.commands.onWillDispatch does not retain originalEvent

Atom 1.3.2, Ubuntu 12.04
Run the following in the console:
atom.commands.onWillDispatch(function (e) {console.log(e.originalEvent)})

When using the default F11 to trigger window.toggle-full-screen the originalEvent property is not set like with other keybindings.
Output:
(left)
KeyboardEvent {}

(home)
KeyboardEvent {}

(F11)
undefined

@Tyriar Yeah, I believe this is true for any events which originate from the application menu, rather than from a regular keyboard event. What are you trying to use the originalEvent property for?

@maxbrunsfeld but I pressed F11 to trigger, why would that come from the application menu? I'm looking at building an extension that needs to know whether commands that were dispatched were dispatched via keyboard or not (command palette, menu).

Because that command is in the application menu, and has a keyboard shortcut defined in the application menu, the command is triggered by a different mechanism: it doesn't start from a KeyboardEvent.

@maxbrunsfeld But this doesn't happen for ctrl+alt+r to reload for example which is defined in the same menu section and the keymap is in the same scope?

Log after ctrl+alt+r:
KeyboardEvent {}
  altKey: true
  bubbles: true
  cancelBubble: false
  cancelable: true
  charCode: 0
  ctrlKey: true
  currentTarget: null
  defaultPrevented: true
  detail: 0
  eventPhase: 0
  keyCode: 82
  keyIdentifier: "U+0052"
  keyLocation: 0
  location: 0
  metaKey: false
  path: Array[16]
  repeat: false
  returnValue: false
  shiftKey: false
  srcElement: atom-text-editor.editor
  target: atom-text-editor.editor
  timeStamp:<PHONE_NUMBER>940
  type: "keydown"
  view: Window
  which: 82
  __proto__: KeyboardEvent

That's a good question. Do all of the F keys behave this way? If you're interested in figuring this out, you could put a breakpoint in CommandRegistry::dispatch.

F3 sets the originalEvent correctly.
When I stick a breakpoint in CommandRegistry::dispatch it is hit when I press F11, but not F3 or ctrl+alt+r.

In other words you seem to be right as F11 goes via dispatchApplicationMenuCommand, I'm still not understanding what exactly that distinction means though.

Found the problem, keybindings featuring uppercase F's in function keys simply don't work. It gets skipped over and ends up emitting 'did-fail-to-match-binding' in KeymapManager::handleKeyboardEvent. It must then fallback to check the application menu somewhere. The core issue here is captured in https://github.com/atom/atom/issues/9194 and https://github.com/atom/atom/issues/9917. In the long term KeymapManager should either be compatible with uppercase function keys or error out on them.


Compare partitioned files in order

This PR builds on https://github.com/NVIDIA/spark-rapids/pull/823 and adds support to CompareResults for comparing data sources with multiple partitions when ignoreOrdering=false by introspecting the RDD to get the underlying filenames and then reading each partition in order.
I have tested this on the YARN cluster against HDFS sources, and locally on my desktop running in local mode.
This closes https://github.com/NVIDIA/spark-rapids/issues/852

build


Added check for correct results after simulation

Uses OMJulia and is very very unstable and a complete overkill.
TODO: Replace diffSimulationResults() with more simple and pure Julia function.
Should solve issue #7

To much problems with docker and OpenMOdelica.
Also we don't want to have many dependecies to big tools.


Line with transparent fill

I have been trying to get a 'hollow' line to display in the sense that the stroke color only applies to the border and the fill is transparent. I have many of the component options but none seem to work. Is it possible what i am trying to do?
<Line
        x1="30"
        y1="15"
        x2={this.state.position}
        y2="15"
        stroke="blue"
         //fill="white"
         //fillRule="nonzero"
        fillOpacity={1}
        strokeWidth="15"
        strokeLinecap="round"

    />

Are you looking for fill="none" perhaps?


Fix broken link in mnist docs

Small fix, maybe too small.

Thanks for the fix! We'll add it as part of neon 1.6.


Replace crossbeam-channel by flume

Replace crossbeam-channel by  flume, which is coming with some advantages, see https://github.com/zesterer/flume#why-flume.

not sure if i'm right, but it seems flume::Sender/Reciever are not unwind-safe, because they are dependent on Spinlock, with no lock poisoning.
So it doesn't work well with slog::SendSyncUnwindSafeDrain trait...

Aren't spin locks kind of a bad idea in general?

Not sure, maybe they have good reasons to use spinlocks ...
I created this PR because we found some performance issues with slog::async + crossbeam-channel.
For our use-case, with multiple log producers, channel send long-tail performance can be quite bad, so we decided to downgrade slog-async to 0.2.3 for now, which uses std::mpsc.
According to our benchmarks, flume doesn't have this perf degradation.
But it's probably more a crossbeam issue, i'll open a ticket there.

Not sure, maybe they have good reasons to use spinlocks ...

They are generally quite fast because they are simple and they don't need OS support. But if the thread is ever preempted while the lock is held, other threads will burn CPU for the whole scheduler tick (and even yielding to the OS scheduler doesn't have to help, if enough of them are spinning, they will keep turns). So in a sense, they can be faster, but they also have much more severe „failure“ mode. That's why eg. parking_lot spins for few turns, but then suspends on a real OS-backed mutex.

Well... I'm not sure what to do about it. :D

Well... I'm not sure what to do about it. :D

OK, I've had a look at flume and it seems not to be using the spin lock as a mutex: https://github.com/zesterer/flume/issues/29#issuecomment-641530555. So my fear of spin-locks here is probably not warranted.

OK, I've had a look at flume and it seems not to be using the spin lock as a mutex: https://github.com/zesterer/flume/issues/29#issuecomment-641530555. So my fear of spin-locks here is probably not warranted.

So... all agree that flume is better? :D

So... all agree that flume is better? :D

Is this still a draft?

Just to add that @zazabe and I eventually settled on using std::sync::mpsc and not flume because the performance was better with mpsc for our use case.
See some benchmark data here: https://github.com/slog-rs/async/issues/21#issuecomment-762736542

I say we put both flume and crossbeam behind feature flags, that way people can pick whatever they want.....


cucumber-expressions: {word}-expression with unicode fix

Summary

fix {word}-expression using Unicode characters.
fix line separator in ExpressionExamplesTest.

Details
I think the summary is detailed enough :)
Motivation and Context
I have a problem when using {word}-expression with russian words.
How Has This Been Tested?
I've written unit-tests. An I'm testing it in my bdd-project.
Types of changes

[x ] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).

Checklist:

[ x] I've added tests for my code.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Hi @savkk,
Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾
In return for this generous offer we hope you will:

✅ Continue to use branches and pull requests. When someone on the core team approves a pull request (yours or someone else's), you're welcome to merge it yourself.
💚 Commit to setting a good example by following and upholding our code of conduct in your interactions with other collaborators and users.
💬 Join the community Slack channel to meet the rest of the team and make yourself at home.
ℹ Don't feel obliged to help, just do what you can if you have the time and the energy.
🙋 Ask if you need anything. We're looking for feedback about how to make the project more welcoming, so please tell us!

On behalf of the Cucumber core team,
Aslak Hellesøy
Creator of Cucumber

This broke cucumber on Android because of java.lang.IllegalArgumentException: Unsupported flags: 256. I''l check if can find workaround for this

Although this flag exists in Android and docs says that it has no effect on Android but can be passed it is omitted in source code and passing it throws exception.


Cannot import Google Sheet

It gives me this error after I click import button. I didn't set Google two-stage verification on.
GDataRequestException: Execution of authentication request returned unexpected result: 404
Google.GData.Client.Utilities.getAuthException (Google.GData.Client.TokenCollection tokens, System.Net.HttpWebResponse response)
Google.GData.Client.Utilities.QueryClientLoginToken (Google.GData.Client.GDataCredentials gc, System.String serviceName, System.String applicationName, Boolean fUseKeepAlive, IWebProxy proxyServer, System.Uri clientLoginHandler)
Google.GData.Client.GDataGAuthRequest.QueryAuthToken (Google.GData.Client.GDataCredentials gc)
Google.GData.Client.GDataGAuthRequest.EnsureCredentials ()
Google.GData.Client.GDataRequest.EnsureWebRequest ()
Google.GData.Client.GDataGAuthRequest.EnsureWebRequest ()
Google.GData.Client.GDataRequest.Execute ()
Google.GData.Client.GDataGAuthRequest.Execute (Int32 retryCounter)
UnityEditor.DockArea:OnGUI()

Yes, it's a known issue. Temporary the google side is broken at the moment due to the google's recent change of the authentication policy.
See that https://github.com/kimsama/Unity-QuickSheet/issues/8

So, when you can fix this issue? Thanks.

I've been so busy in these days. I'm not sure when I can fix that. Sorry for that. (but I guess it will not take a long)
Cheers,
-Kim

Completed to commit quick fix OAuth2 problem. See the googlesettings.asset file under Editor folder.

See the following link to to set up OAuth2 and download json file. (instead of generating p12, generate json)
http://stackoverflow.com/questions/30469058/google-gdata-client-gdatarequestexception-authentication-suddenly-fails-in-old


xtheadfmemidx: Fix instruction depenencies

The XTheadFMemIdx instruction that perform a zero-extension of the index register (those with a u in the name) are identical to the sign-extension counterparts on RV32. Therefore, it does not make sense to offer both for RV32.
Further, the table lists the 8-byte loads/stores as compatible with the F extensions. However, 8-byte loads won't fit into single-precision floating-point register. And 8-byte stores can't be performed from a single-precision floating-point register. Instead of defining truncation and extension for these cases, let's make the instructions only available for those cases where they make sense.
Fixes #49.

LGTM, Thanks!


Use bit-masking rather than modulo in levelized priority queue

The levelized priority queue currently allows for any number of buckets. Gerth Brodal pointed out, that if the number of tpie::merge_sorter in the _buckets_sorter field are a power-of-two, then one can do the modulo operations in push, next_bucket_label, setup_next_level, and setup_next_bucket much faster with mere bit-masking. Based on experiments by @jaschdoc it is at least 3x times faster to use bit-masking than to use the regular modulo operator.

The simple solution is to always have a power-of-two for the number of buckets. One can use static_assert to check the Buckets value at compile time. This way one can replace the % BUCKETS with & LOOK_AHEAD.

The fancy solution is to allow any choice of buckets. We can do so by creating an inlined modulo function that based on whether Buckets is some 2i - 1, i.e. 0, 1, 3, 7, ... , uses bit-masking instead. If done correctly, then the switch in logic is purely done at compile-time.

I'd favour the first of the two, since it is simple and does not add any unnecessary code complexity. The resulting performance increase is probably only visible for "small" instances, but given the number of calls it should be worth the effort.
Tasks

[x] Add a static_assert on the BUCKETS satisfying a constexpr is_pow2 predicate.
[x] Turn all % BUCKETS into & LOOK_AHEAD.

Our experiments in #278 show that this does not improve performance. Since the BUCKETS variable is known at compile time, then the compiler is already able to do this very optimisation.


Avidemux 2.7.8

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

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

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

Microsoft Reviewers: Open in CodeFlow

@ImJoakim Please review again.

@palenshus @jamespik will take a look, they will most likely add a waiver. Example: #24527. Waiver was added for this PR #24482.


Iterable becomes empty after checking

Be aware though, that the iterable will be emptied after being checked by your function, making it useless
console.log(isEmptyIterable(myIterable))  //return false if myIterable is not empty

console.log(isEmptyIterable(myIterable))  //return true always even if myIterable is not empty initially

Here is an explanation to the problem:
https://stackoverflow.com/questions/56811623/why-does-my-generator-become-empty-after-being-iterated-through


Refactor viz.scenario

Rename viz.scenario to viz.scenarios
Extract some parts of it to external functions
Adapt to blue style guide
Other minor refactorings

Part of #465

@ConnectedSystems Ready for another review.


Add tooltip for containers in Heron UI with cpu/ram/disk config

The related design doc can be found here: https://docs.google.com/document/d/1CaDVVifcWqViRO96tmePK5dwhOhYnhwjhMRuImg-aOs/edit#heading=h.rfm62inflf7w

PR: https://github.com/apache/incubator-heron/pull/3371


Improve link text in phase banner examples and restructure guidance

The link text in our examples isn’t very good. It’s just the word ‘feedback’ that’s linked, which is ambiguous and so it's unclear where the page takes you – it could be a way to view feedback that the service has received, for example.
Improve the link text by making more of the sentence part of the link, and also make it clear how the user will give feedback (by email, or in a form that opens in a new tab).
Restructure the sentence to put the link at the end of the sentence.
Also tweak the structure of the guidance to:

Move ‘Use an alpha banner when your service is in alpha, and a beta banner if your service is in private or public beta.’ under ‘How it works’ as it’s more about how to use the component than when to use the component.
Move the paragraph starting ‘There are 2 ways to use the phase banner component…’ further up ‘How it works’ as it feels like it’s the logical thing to say first.
Move the examples under ‘Add a feedback link’

Closes #3650

Thanks for the speedy review @claireashworth! 👍🏻


Update create-the-map.md

carter24343
Changes

Checks
By submitting your pull request for review, you agree to the following:

[ ] This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
[ ] I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
[ ] To the best of my knowledge, all proposed changes are accurate.

cvczgz

was called


BLE connection is not possible with Android SDK Demo App.

Hello.
I am trying to do MQTT communication with AWS with nRF52840 Device.
https://docs.aws.amazon.com/freertos/latest/userguide/getting_started_nordic.html
I am doing this while looking at the guide of this link.
However, it seems that the Android SDK Demo App and BLE are not connected.
Debug Log on nRF52840 is as follows. :
0 0 [iot_thread] [INFO ][DEMO][0] ---------STARTING DEMO---------

The RAM must start at 200044b0, it's size must be 3bb50
1 0 [iot_thread] [INFO ][INIT][0] SDK successfully initialized.
2 2 [iot_thread] [INFO ][DEMO][20] No networks connected for the demo. Waiting for a network connection.
3 370 [BLE] [ERROR][BLE_HAL][3700] sd_ble_gattc_exchange_mtu_request() on connection 0x0 returned busy, will retry.
4 370 [BLE] [INFO ][DEMO][3700] BLE Connected to remote device, connId = 0

5 370 [iot_thread] [INFO ][DEMO][3700] Successfully initialized the demo. Network type for the demo: 2
6 370 [iot_thread] [INFO ][MQTT][3700] MQTT library successfully initialized.
7 370 [iot_thread] [INFO ][DEMO][3700] MQTT demo client identifier is MyFMSBLE1 (length 9).
8 416 [InputTask] Numeric comparison:787566
9 416 [InputTask] Press 'y' to confirm
10 3370 [iot_thread] [ERROR][NET_BLE][33700] Failed to create BLE network connection after 30000 milliseconds.
11 3370 [iot_thread] [ERROR][MQTT][33700] Failed to establish new MQTT connection, error NETWORK ERROR.
12 3370 [iot_thread] [ERROR][DEMO][33700] MQTT CONNECT returned error NETWORK ERROR.
13 3370 [iot_thread] [INFO ][MQTT][33700] MQTT library cleanup done.
14 3370 [iot_thread] [INFO ][DEMO][33700] memory_metrics::freertos_heap::before::bytes::125128
15 3370 [iot_thread] [INFO ][DEMO][33700] memory_metrics::freertos_heap::after::bytes::96752
16 3370 [iot_thread] [INFO ][DEMO][33700] memory_metrics::demo_task_stack::before::bytes::19024
17 3370 [iot_thread] [INFO ][DEMO][33700] memory_metrics::demo_task_stack::after::bytes::18104
18 3370 [iot_thread] [ERROR][DEMO][33700] Error running demo.
19 3376 [BLE] [ERROR][BLE_HAL][33760] Failed to secure connection. Disconnecting.
20 3376 [BLE] [INFO ][DEMO][33760]  Disconnected from BLE device. Stopping the counter update

21 3376 [iot_thread] [INFO ][INIT][33760] SDK cleanup done.
22 3376 [iot_thread] [INFO ][DEMO][33760] -------DEMO FINISHED-------

I think BLE connect is disconnected due to Timeout because'y' msg does not come from Android App. Is that correct?
Can you solve this problem? Please.

From the logs its seems the device is not receiving the input for confirmation for numeric comparison from the terminal:
9 416 [InputTask] Press 'y' to confirm

Are you using Segger embedded studio terminal ? If so the terminal does not accept input characters as mentioned in the doc:
Note

The Segger embedded studio terminal emulator does not support an input capability. For this, use a terminal emulator like PuTTy, Tera Term, or GNU Screen. Configure the terminal to connect to your board by a serial connection as instructed in Installing a terminal emulator.

As mentioned you can use other terminals like TeraTerm or Putty and press y to confirm numeric comparison on the device.

Thanks for your kind answer.
What I want to know is that BLE connection is possible only by receiving'Y' MSG.
I don't understand. If this is not a problem, I would like to know why the BLE connection is not working.

@tobaro-4
By default device (peripheral) is configured to run with secure connection only mode, means encrypted connection with authentication using MITM (Man in the middle protection). This requires a numeric comparison code displayed at both device side and mobile phone and user need to confirm code on both sides.
You can choose to override this config parameters by setting Just Works Pairing mode (Unauthenticated paring with encryption) or no security as shown in this doc: https://docs.aws.amazon.com/freertos/latest/userguide/freertos-ble-library.html#freertos-ble-restrictions


awd-lstm training results on wikitext-2 do not match those in documentation

Listed on https://gluon-nlp.mxnet.io/scripts/index.html#language-model
[2] awd_lstm_lm_600_wikitext-2 (Val PPL 84.61 Test PPL 80.96)
python word_language_model.py --gpus 0 --emsize 200 --nhid 600 --epochs 750 --dropout 0.2 --dropout_h 0.1 --dropout_i 0.3 --dropout_e 0.05 --weight_drop 0.2 --tied --save awd_lstm_lm_600_wikitext-2
I'm getting near zero lr pretty early on and validation ppl is substantially higher (see below) than the published training log here: https://github.com/dmlc/web-data/blob/master/gluonnlp/logs/language_model/awd_lstm_lm_600_wikitext-2.log
My training log

[Epoch 169] time cost 306.39s, valid loss 6.06, valid ppl 429.32，lr 0.03
[Epoch 170 Batch 200/372] current loss 5.69, ppl 296.66, throughput 102.99 samples/s, lr 0.03
[Epoch 170] throughput 7052.65 samples/s
[Epoch 170] time cost 309.12s, valid loss 6.06, valid ppl 428.84，lr 0.03
Learning rate after interval update 0.003000
[Epoch 171 Batch 200/372] current loss 5.70, ppl 297.69, throughput 102.76 samples/s, lr 0.00
[Epoch 171] throughput 7029.03 samples/s
[Epoch 171] time cost 310.24s, valid loss 6.06, valid ppl 428.85，lr 0.00
[Epoch 172 Batch 200/372] current loss 5.69, ppl 295.60, throughput 106.35 samples/s, lr 0.00

Similar issue occurred for  awd_lstm_lm_1150_wikitext-2, i.e. learning rate substantially lower and ppl substantially higher compared to published log.
My training log

[Epoch 95] time cost 540.07s, valid loss 5.50, valid ppl 243.47，lr 3.00
[Epoch 96 Batch 200/372] current loss 5.33, ppl 207.33, throughput 61.25 samples/s, lr 3.17
[Epoch 96] throughput 4152.54 samples/s
[Epoch 96] time cost 536.21s, valid loss 5.48, valid ppl 240.60，lr 3.00
[Epoch 97 Batch 200/372] current loss 5.31, ppl 202.84, throughput 60.55 samples/s, lr 3.43
[Epoch 97] throughput 4136.88 samples/s
[Epoch 97] time cost 538.99s, valid loss 5.47, valid ppl 238.03，lr 3.00
[Epoch 98 Batch 200/372] current loss 5.30, ppl 199.51, throughput 60.91 samples/s, lr 1.54

BTW "--gpus" flag in the documentation page should be changed to "--gpu"
The training progress of standard-lstm models appear to match pretty well to the published training log.

Thanks for reporting, @dingran. @cgraywang, could you take a look?

@dingran by the way, at least with respect to the document page, we've updated the document on master branch: http://gluon-nlp.mxnet.io/master/model_zoo/language_model/index.html
The change will be included in 0.4.1

@dingran by the way, at least with respect to the document page, we've updated the document on master branch: http://gluon-nlp.mxnet.io/master/model_zoo/language_model/index.html
The change will be included in 0.4.1

Cool, overall, very happy with this toolkit, looking forward to future updates!

@dingran thanks for the kind words. Making the community productive is what we strive for. Keep letting us know how we can do better. If you are using the toolkit for something new and are willing to make it available to the community, let's work together and make it happen :)

Reran training with the following setting
python word_language_model.py --gpu 0 --tied --ntasgd --lr_update_interval 50 --lr_update_factor 0.5 --save awd_lstm_lm_1150_wikitext-2
See (unfinished) training log below, it does not seem to match the log here very well
I'm using the repo, at commit
% git rev-parse --short HEAD
bd68aaa

Use AWDRNN
/home/rding/.pyenv/versions/anaconda3-5.0.0/lib/python3.6/site-packages/mxnet/gluon/block.py:521: UserWarning: All children of this Sequential layer 'awdrnn1_sequential0_' are HybridBlocks.  We are now aware of that this might due to weight dropout with gradient clipping due to the gluonnlp's dependencies' version change. We are looking into it and will update once we got this right. It should not take long. At the meantime, please feel free to try the AWD pretrained model by using
        model, _ = nlp.model.get_model('awd_lstm_lm_1150', dataset_name='wikitext-2',                                        pretrained=True)

@dingran It looks like the script yet similar results as reported in the repo.
I used the same command line:
python word_language_model.py --gpu 0 --tied --ntasgd --lr_update_interval 50 --lr_update_factor 0.5 --save awd_lstm_lm_1150_wikitext-2
The mxnet version I am using:
Name: mxnet-cu90
Version: 1.5.0b20190210

And the master branch of gluonnlp is used.
It hasn't finished training yet, but I pasted the some intermediate results here:

[Epoch 0 Batch 200/372] current loss 7.83, ppl 2515.23, throughput 276.81 samples/s, lr 29.57
[Epoch 0] throughput 18380.53 samples/s
[Epoch 0] time cost 118.68s, valid loss 6.41, valid ppl 608.57，lr 30.00
[Epoch 0] test loss 6.34, test ppl 567.69
[Epoch 10 Batch 200/372] current loss 5.35, ppl 211.57, throughput 261.67 samples/s, lr 13.71
[Epoch 10] throughput 17765.78 samples/s
[Epoch 10] time cost 122.64s, valid loss 4.94, valid ppl 140.29，lr 30.00
[Epoch 10] test loss 4.87, test ppl 129.93
[Epoch 100 Batch 200/372] current loss 4.26, ppl 70.86, throughput 279.14 samples/s, lr 27.00
[Epoch 100] throughput 19054.15 samples/s
[Epoch 100] time cost 114.58s, valid loss 4.37, valid ppl 79.30，lr 30.00
[Epoch 100] test loss 4.32, test ppl 75.29
[Epoch 200 Batch 200/372] current loss 4.09, ppl 59.74, throughput 285.04 samples/s, lr 30.86
[Epoch 200] throughput 19220.91 samples/s
[Epoch 200] time cost 113.69s, valid loss 4.31, valid ppl 74.10，lr 30.00
[Epoch 200] test loss 4.26, test ppl 70.64
[Epoch 300 Batch 200/372] current loss 4.02, ppl 55.84, throughput 281.34 samples/s, lr 28.29
[Epoch 300] throughput 19174.19 samples/s
[Epoch 300] time cost 113.88s, valid loss 4.28, valid ppl 72.03，lr 30.00
[Epoch 300] test loss 4.23, test ppl 68.82
[Epoch 476 Batch 200/372] current loss 3.96, ppl 52.47, throughput 282.90 samples/s, lr 31.29
[Epoch 476] throughput 19116.56 samples/s
[Epoch 476] time cost 114.33s, valid loss 4.25, valid ppl 70.36，lr 30.00
[Epoch 476] test loss 4.21, test ppl 67.31

Could you please try to run again using the same setting and let us know if there is more issue?

Is this issue resolved?

@eric-haibin-lin  I just switched job and don't have access to gpu machines at the moment, let's consider it resolved given @cgraywang's experiment.

@dingran thanks for the response. Feel free to ping us or open another issue if this is still a problem.
Best of luck with the new adventure!


[Op] Add attention and bias_gelu ops

Description
This is a prerequisite PR for adding HF GPT-2 schedule.

Implement attention ops that use flash-attention and xformrs.
Implement bias_gelu ops that use torchscript or torch compiler.
Change the GPT-Neo schedule to use these ops, so that now GPT-Neo schedule doesn't depend on epoi anymore. Later we will update the schedules of other example models accordingly.
[Test] Add -rxXs to let pytest print reasons of skipped tests.
[Docker] Update flash-attention commit hash which improves the kernel performance by ~12%. CI image is not updated because this change doesn't impact the functionality.

Checklist

[x] PR's title starts with a category (e.g. [Bugfix], [Model], [Tutorial], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

@szhengac @chhzh123

Thanks @chhzh123


before and after scripts do not run under esm

Take the following configuration:
esm: true
before: scripts/before.js
after: scripts/after.js

I know you expressed an interest in disabling esm by default but I think the expectation is that these scripts should be executed with node -r esm if esm is enabled (especially if it's explicit as may be the case in tap 15).

The same applies to coverage-map, the script must be npm module.

As of tap 18, they will run no matter what kind of thing they are.  Check it out with npm i tap@pre. https://node-tap.org/changelog/#18.0


A mistake in 'Update Build Numbers Script' in Build Phases of DetoxInstruments and DTXProfiler targets.

Build ERROR:
$(CURRENT_PROJECT_VERSION) + 1: syntax error: operand expected (error token is "$(CURRENT_PROJECT_VERSION) + 1")
Because
buildNumber=$(($buildNumber) + 1)  is wrong.
You should change it to:
buildNumber=$(($($buildNumber) + 1))

What is the difference between $buildNumber and $($buildNumber))? For me the scripts work.

What is the difference between $buildNumber and $($buildNumber))? For me the scripts work.

sorry. It looks like it is a Xcode build bug. Just now, I use $buildNumber) to rerun, it was OK. But about 30min ago , it was wrong and show the tips 'syntax error' above.How odd!


After the vnode is created in the track UI, the normal child nodes will not be displayed and associated.

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[x] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happen?
If possible, provide a way for reproducing the error. e.g. demo application, component version.

After the vnode is created in the track UI, the normal child nodes will not be displayed and associated.

Requirement or improvement

Please describe about your requirements or improvement suggestions.

The vnode creation rule needs to be improved. @x22x22 Ping me if you have some uncertain things.

Please wait for me to fix this bug before considering release 6.4

When ref is missing, we need to create

Target spanId in the target segment id, with parentSegmentId = 0.
spanId = 0 in the target segment id, with parentSegmentId = -1.

For parent span, we should follow same rule, if span is not exist, create parent span and root span(spanid = 0)
All these vnode should only be added when it is not existing(or created before)

Have to repair, see https://github.com/apache/skywalking-rocketbot-ui/pull/154

Now  I have the same problem in 6.4.0.


String.toRequestBody() vs. String.asRequestBody()

We’ve implemented it as String.toRequestBody() but I think we might be mistaken here. to feels like a conversion; as feels like a cast. I’d like to confirm what the precedents are and follow them.

The distinction for string is mostly irrelevant because it's immutable. I think file is far more interesting and makes the argument for 'as'. The key difference being that changes to the file contents are reflected in the request body even after it's been created.

Lemme confirm I follow:
A.toB: returns a B getting the value from A. String.toInt(), Int.toString(), MutableList.toList().
A.asB: returns a B that is a view of A. Arrays.asList(), MutableList as List.

That is my understanding, yep!
On Sun, Jun 9, 2019 at 12:16 AM Jesse Wilson<EMAIL_ADDRESS>wrote:

Lemme confirm I follow:
A.toB: returns a B getting the value from A. String.toInt(),
Int.toString(), MutableList.toList().
A.asB: returns a B that is a view of A. Arrays.asList(), MutableList as
List.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/square/okhttp/issues/5177?email_source=notifications&email_token=AAAQIEPK2G7KJSSL3KYTNXDPZR73XA5CNFSM4HWIQS2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXIDFPY#issuecomment-500183743,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAQIEIJBVYS3UWSX6TVANLPZR73XANCNFSM4HWIQS2A
.

I wanna change File.toRequestBody() to File.asRequestBody() because we don’t snapshot the file at the time of the call. I should go through the other toXxx() methods to confirm they’re not views.


Update puppet env module usage

Depends on https://github.com/cargomedia/puppet-packages/pull/1468

@ppp0 please review

lgtm, thanks for noticing!


Allow custom properties to be visible inside the swagger.json file

For my needs, some extra 'x-AAAA' properties have to be available inside the final swagger.json files. As you didn't include this feature, and instead of adding every custom 'x-AAA' properties I need, I finally add only one custom property where you can put everything you need inside (see the optionsreference.md file).

Coverage remained the same at 100.0% when pulling 5681c92b01cb192df0be0871472183a6975e953e on srz09:master into f32d6e4a5f8e0c533fef825fc37e4c56378db2fc on glennjones:master.

@glennjones Do you think you'll gonna accept this PR one day ? Else I will need to publish an npm package for this PR to suit my needs...

It could be good to refer to my PR #396 as the other proposed way of including custom info properties.

Hi Thanks for taking the time to submit the PR. I have gone with @cvillerm PR #396. Can you check that this gives you what you need and gat back to me if it does not fulfil your use case. Thanks


PackageManager: Fix a broken assumption

PackageMangaer: Fix a broken assumption
The evaluator calls CompatibilityDependencyNavigator.create() in order
to construct a dependency navigator. That function creates an instance
of DependencyGraphNavigator if Project.scopeNames is not null [2].
The constructor of DependencyGraphNavigator in turn requires a
dependency graph to exist [3].
That requires check [3] fails if the project has an empty set as
scopeNames and at the same time there is no dependency graph available
for the type of the project. This happens e.g. if a source tree with
only one definition file is analyzed, whereas resolvedDependencies()
fails with an exception, see [4]. The failed requires check results in
a crash of the evaluator.
Guarantee that in case project.usesTree() == false a dependency graph
for the corresponding project type exists, to fix that problem.
[1] https://github.com/oss-review-toolkit/ort/blob/42845fa5eb5fbb1a9bc617aa778c2979ca0da26d/model/src/main/kotlin/CompatibilityDependencyNavigator.kt#L47-L60
[2] https://github.com/oss-review-toolkit/ort/blob/463a6ebb57cd3e9613b954a2094a509947c85dc3/model/src/main/kotlin/CompatibilityDependencyNavigator.kt#L96
[3] https://github.com/oss-review-toolkit/ort/blob/463a6ebb57cd3e9613b954a2094a509947c85dc3/model/src/main/kotlin/DependencyGraphNavigator.kt#L29-L31
[4] https://github.com/oss-review-toolkit/ort/blob/463a6ebb57cd3e9613b954a2094a509947c85dc3/analyzer/src/main/kotlin/PackageManager.kt#L255
Fixes #4441 .

@oheger-bosch as discussed, I'll work on a follow up PR to add a requires check to fail early in case a problematic OrtResult is constructed by the analyzer.


enable kata shim

Make sure kata-shim is installed at /usr/libexec/kata-containers/kata-shim and pass --agent kata to runv cli. Then runv will invoke kata-shim and use it to proxy for agent.
Like:
sudo runv --kernel /var/lib/hyper/kernel --initrd //golang/src/github.com/hyperhq/hyperstart/build/kata-initrd.img --agent kata --debug run test1

The last commit needs some extra attentions because it changes runv API such that vm.AddContainer actually creates a container in the guest, and therefore it must be called after all container resources are properly inserted.

It's working with kata agent/shim/initramfs image.
$sudo runv --kernel /var/lib/hyper/kernel --initrd //golang/src/github.com/hyperhq/hyperstart/build/kata-initrd.img --agent kata run test1
/ # ls
bin       dev       etc       lib       proc      root      sys       testdata  tmp       usr       var


Getting started with Gmail API in python

Topic Suggestion
Getting started with Gmail API in python
Proposed title of article
Getting started with Gmail API in python
Proposed article introduction
Gmail is one of the most dominant mail service, used globally, by most people and many organizations.
Over the past years, many Gmail features have been improved with the application of Artificial Intelligence, including suggestions while writing emails and security features.
Gmail API is found on RESTful APIs, that authorizes it's users to interact with our Gmail account, and it enables us to access it's elements using a python text.
Key takeaways

Installing the required libraries.
Enabling Gmail APIs
Sending Emails
Searching Emails
Reading Emails
Marking Emails as Read
Marking Emails as Unread
Deleting Emails

Article quality
In this article am going introduce users to Gmail APIs in python, I will take the user through installation of required libraries, and how to work with Gmail APIs in python, like sending, searching, reading, deleting, and marking as read or unread, using the Gmail API in python.

There are many existing articles on this topic, some of which detail all the topics you plan to cover. Can you tell me what unique content you plan to cover or unique perspective you will bring?

Hi @louisefindlay23 , am going to use a beginner friendly language and simplify the content. My article also introduces a new concept, installing required libraries, which has not been covered in official documents.

Hello @louisefindlay23 ,am seeing two labels, question and topic approved. Am not understanding please help.

Hello @louisefindlay23 ,am seeing two labels question and approved, am not understanding kindly help.


Dynamodb ingestion

DynamoDB ingestion is through

Load the attributes from aws dynamodb describe table
Load table recent streams to fetch item images, and derive the schema from the item images

We also support filter tables by AllowDenyPattern and report any table that didn't enable streaming.

Steps to run the ingestion job

aws login
populate dynamodb_to_datahub.yml recipe with configs (copy from ~/.aws/credentails)
2.1 aws_region
2.2 aws_access_key_id
2.3 aws_secret_key_id
2.4 aws_session_token
start k8s port forward for datahub gms pod
run command datahub ingest -c examples/recipes/dynamodb_to_datahub.yml

populate dynamodb_to_datahub.yml recipe with configs (copy from ~/.aws/credentails)

Pro-tip: you can avoid copying over creds to the datahub config by making a [default] section within ~/.aws/credentails that duplicates the [affirm-stage] section, or temporarily renaming that section to [default].

We also need the logic to fetch the current fields from DataHub and do a union with the attributes obtained from the streams.

rebased off 0.8.44.affirm


Change main branch name in GitHub actions

Default branch is now "main", so the GitHub Action snippet will not work out-of-box on newer repositories without swapping the default branch name.

The default branch of this repository is still called "master"...

I am not sure this changes anything, as the snippet should refer to the branch Github Actions is working on?

What I am saying is that our "main" branch is called master so if we implement the changes you are suggesting here, Actions will look for a branch called main and never find one and so never run.


Stop adding dummy edges to ensure one connected component for pymatching

pymatching no longer requires the input to have just one connected component

FYI in the same PyMatching v0.4.0 release the requirement for all qubit_id's up to the max index was also removed, so the lines
# Ensure invincible observables are seen by adding a boundary edge with all observables.
    g.add_node(num_detectors + 1)
    g.add_edge(num_detectors, num_detectors + 1, weight=1, qubit_id=list(range(num_observables)))

can also be removed

Eh, it's actually corner-casier to try to only use the max anyways since it fails in the degenerate case where there's no observables.

The number of observables (and hence shape of the array output by decode) is just the max fault id - 1. So if you want the matching.decode output to be consistent then yeah you'll want to make sure the max id is added somewhere (or reshape the decode output array if necessary). So maybe just easiest to leave it as is


Closing an "Untitled-1" editor causes the editor to re-launch

While in the the "F5" view...
If there is an editor that is not saved and you try to close it out... you can't as the editor immediately re-launches.
https://user-images.githubusercontent.com/5244553/110676896-80c4d380-81a2-11eb-8c0b-e92e94e3ae25.mp4

Hey Walter, this is interesting.  Can you reboot your computer to see if this continues?

I don't have to go that far, if I close VS code (and reopen), the editors return, but then I can close them from the standard files view.

I just re-produced the issue by doing File -> New from the F5 view.  The trick is that you have to have an editor opened (with a disk backed file) from the explorer view first.

ok.  thanks.  so, did you reboot your computer?

Yes, this morning.

It was discovered that a different plugin "codetogether" is causing this issue.  Thank-you for looking at this.