sbtn 1.8.1 regression on Debian 11 with glibc 2.31

steps
I just updated sbt on two Debian 11 ("bullseye" / "stable") x86_64 machines from 1.8.0 to 1.8.1 (a local machine and a vm used for GitHub Actions hosted-runner).
problem
On both I can no longer start sbtn and receive the following output instead.
This happens not only in a project folder which I compiled successfully just before the update, but also in a completely empty folder.
It happens with both sbt --client and manually downloaded sbtn.
ci@vm03:/tmp/foo$ sbt --client clean
/usr/bin/sbt: line 174: local: `=': not a valid identifier
/home/ci/.cache/sbt/boot/sbtn/1.8.1/sbtn: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by /home/ci/.cache/sbt/boot/sbtn/1.8.1/sbtn)
/home/ci/.cache/sbt/boot/sbtn/1.8.1/sbtn: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /home/ci/.cache/sbt/boot/sbtn/1.8.1/sbtn)

locke@latitude:~/Downloads/sbtn-x86_64-pc-linux$ file sbtn
sbtn: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, BuildID[sha1]=d14b2409d54f5d2b1a0cfbe9475a5b938a2b2479, for GNU/Linux 3.2.0, with debug_info, not stripped
locke@latitude:~/Downloads/sbtn-x86_64-pc-linux$ ./sbtn
./sbtn: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by ./sbtn)
./sbtn: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by ./sbtn)

expectation
sbt and sbtn should work on Debian 11 ("bullseye" / "stable") with glibc 2.31, as this is the current supported release.
Otherwise, it has to be clearly stated somewhere (e.g. https://www.scala-sbt.org/download.html), that systems with an older glibc like Debian 11 are no longer supported.
notes
full log on a vm
Linux vm03 5.10.0-20-amd64 #1 SMP Debian 5.10.158-2 (2022-12-13) x86_64

The programs included with the Debian GNU/Linux system are free software;
the exact distribution terms for each program are described in the
individual files in /usr/share/doc/*/copyright.

Debian GNU/Linux comes with ABSOLUTELY NO WARRANTY, to the extent
permitted by applicable law.
Last login: Sat Dec 31 12:31:23 2022 from 2a02:908:1867:f140:226:18ff:fefd:ef7
locke@vm03:~$ sudo aptitude upgrade
[sudo] password for locke:
The following packages will be upgraded:
  sbt
1 packages upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Need to get 0 B/20.0 kB of archives. After unpacking 1,024 B will be used.
Do you want to continue? [Y/n/?] v

Versions will be shown.

The following packages will be upgraded:
  sbt [1.8.0 -> 1.8.1]
1 packages upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Need to get 0 B/20.0 kB of archives. After unpacking 1,024 B will be used.
Do you want to continue? [Y/n/?]
Reading changelogs... Done
(Reading database ... 58821 files and directories currently installed.)
Preparing to unpack .../apt/archives/sbt_1.8.1_all.deb ...
Unpacking sbt (1.8.1) over (1.8.0) ...
Setting up sbt (1.8.1) ...
Processing triggers for man-db (2.9.4-2) ...

Current status: 0 (-1) upgradable.
locke@vm03:~$ cat /etc/debian_version
11.6
locke@vm03:~$ uname -a
Linux vm03 5.10.0-20-amd64 #1 SMP Debian 5.10.158-2 (2022-12-13) x86_64 GNU/Linux
locke@vm03:~$ apt-cache policy libc-bin
libc-bin:
  Installed: 2.31-13+deb11u5
  Candidate: 2.31-13+deb11u5
  Version table:
 *** 2.31-13+deb11u5 500
        500 http://ftp2.de.debian.org/debian bullseye/main amd64 Packages
        500 http://ftp2.de.debian.org/debian bullseye-updates/main amd64 Packages
        100 /var/lib/dpkg/status
locke@vm03:~$ cat /etc/apt/sources.list.d/sbt.list
# 2021-07-31
deb [signed-by=/usr/local/share/keyrings/scalasbt-release.gpg] https://repo.scala-sbt.org/scalasbt/debian all main
deb [signed-by=/usr/local/share/keyrings/scalasbt-release.gpg] https://repo.scala-sbt.org/scalasbt/debian /

locke@vm03:~$ sudo -u ci -s
ci@vm03:/home/locke$ cd /tmp/
ci@vm03:/tmp$ mkdir foo
ci@vm03:/tmp$ cd foo/
ci@vm03:/tmp/foo$ sbt --client clean
/usr/bin/sbt: line 174: local: `=': not a valid identifier
/home/ci/.cache/sbt/boot/sbtn/1.8.1/sbtn: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by /home/ci/.cache/sbt/boot/sbtn/1.8.1/sbtn)
/home/ci/.cache/sbt/boot/sbtn/1.8.1/sbtn: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /home/ci/.cache/sbt/boot/sbtn/1.8.1/sbtn)

after downgrading to 1.8.0, sbtn works
locke@vm03:~$ sudo aptitude install sbt=1.8.0
The following packages will be DOWNGRADED:
  sbt
0 packages upgraded, 0 newly installed, 1 downgraded, 0 to remove and 0 not upgraded.
Need to get 0 B/19.9 kB of archives. After unpacking 1,024 B will be freed.
dpkg: warning: downgrading sbt from 1.8.1 to 1.8.0
(Reading database ... 58821 files and directories currently installed.)
Preparing to unpack .../apt/archives/sbt_1.8.0_all.deb ...
Unpacking sbt (1.8.0) over (1.8.1) ...
Setting up sbt (1.8.0) ...
Processing triggers for man-db (2.9.4-2) ...

Current status: 1 (+1) upgradable.
locke@vm03:~$ sudo -u ci -s
ci@vm03:/home/locke$ cd /tmp/foo/
ci@vm03:/tmp/foo$ sbt --client clean
downloading sbtn 1.7.0
[info] entering *experimental* thin client - BEEP WHIRR
[info] server was not detected. starting an instance
[warn] No sbt.version set in project/build.properties, base directory: /tmp/foo
[info] welcome to sbt 1.8.0 (Debian Java 11.0.16)
[info] set current project to foo (in build file:/tmp/foo/)
[info] sbt server started at local:///home/ci/.sbt/1.0/server/4c0c08c28ca021b53943/sock
[info] terminate the server with `shutdown`
> clean
[success] Total time: 1 s, completed Jan 4, 2023, 6:43:14 PM
ci@vm03:/tmp/foo$ sbt --client shutdown
[info] entering *experimental* thin client - BEEP WHIRR
[info] terminate the server with `shutdown`
> shutdown
[info] disconnected

other systems
I manually downloaded sbt from the GitHub Release onto Debian 12 ("bookworm" / "testing", using glibc 2.36-7) on my RaspberryPI 4 (aarch64) and had no problem with the simple clean smoketest.

@Locke Thanks for the detailed report!
I guess this could be caused by ipcsocket build (https://github.com/sbt/ipcsocket) or the GraalVM native-image build for sbtn. In general, any part of the sbt that requires native integration is somewhat finicky, and if the compatibility is not enforced by CI (often set to "ubuntu-latest") we won't notice when it slips.
@mkurz Do you know if this is mostly due to whatever the glibc lying around in the Github Actions image we use to build sbtn? If so using slightly older ubuntu would restore the Debian 11 compatibility?

https://repology.org/project/glibc/versions says Ubuntu 22.04 uses glibc 2.35, and Ubuntu 20.04 uses glibc 2.31. I'd be happy to try to downgrade the Ubuntu machine and see if this might get fixed.

If so using slightly older ubuntu would restore the Debian 11 compatibility?

I'd be happy to try to downgrade the Ubuntu machine and see if this might get fixed.

I am pretty sure that should fix the problem. You used ubuntu-latest to build sbtn 1.7.0 in July 2022, so back then effectively ubuntu-20.04 was used. Since beginning of November 2022 however ubuntu-latest points to ubuntu-22.04: https://github.blog/changelog/2022-11-09-github-actions-ubuntu-latest-workflows-will-use-ubuntu-22-04/

Thankfully this was fairly easy to reproduce on Github Actions as well using ubuntu-20.04 - https://github.com/sbt/sbt/pull/7119

:+1: IMHOT that bug basically was more or less unavoidable, and it was just waiting till a new ipcsocket release gets cut :wink: Guess we stay on ubuntu 20.04 for a bit longer.

sbt 1.8.2 is out - https://eed3si9n.com/sbt-1.8.2


Card data fields are not loaded

In OPC order mode, when customer logs in, the card data fields are not loaded. Only after refreshing page are these fields loaded.
Module version: 1.9.0

Fixed by 3dd033a8765fae7b72e30f551c944073e9572c4b.
New version of module released


docs(Select): disable single select options

Wanted to add this detail to the docs. I found it by chance passing the proper object into the options prop and thought it might be helpful to people searching the docs.

@benjamincanac it seems that the disabled option state is not shown in the preview page of the docs.
I copied the same code to my local environment and it's working there.
Do you know where the difference comes from?

Indeed, I merged it too fast. The disabled was missing in the actual component: https://github.com/nuxtlabs/ui/commit/0ea1f310a95361779ca449cd4f6fbbeb5528e91b


[TypeChecker] SE-0326: Allow ExprAvailabilityWalker to walk into mu…

…lti-statement closures
When experimental-multi-statement-closures flag is enabled,
multi-statement closures are type-checked together with enclosing
context, which means that availability checking is allowed to walk
into them.

@swift-ci please smoke test

@etcwilde Once check gets removed the whole block wouldn't be needed, we should be able to remove everything, which I'll just do.


Adding access to Michkov to see openshift-pipelines

Removing Guillaume
Adding myself (Michkov) to investigate why pruning is not working

/retest


[WIP] 13.12 Neuralstyle

Job d2l-zh/PR-875/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-875/

Job d2l-zh/PR-875/2 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-875/

Job d2l-zh/PR-875/3 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-875/

Job d2l-zh/PR-875/4 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-875/

Job d2l-zh/PR-875/5 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-875/

Job d2l-zh/PR-875/6 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-875/

Job d2l-zh/PR-875/7 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-875/

Job d2l-zh/PR-875/8 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-875/

Job d2l-zh/PR-875/9 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-875/

Render checked done:

sec_
'numref`
cite

Forum links worked


Autoformat TS to follow common practices

I started this project and coded TS like I did OCaml. This meant a syntax like let x :number = ... and other quirks. Introduce a formatter with rules that are the tolerable version of how people tend to write typescript.

Introduced prettier. Resolved.


Bug_NativeSelect> Inverse/Inverse Disabled/Has Error Inverse/Helper Message Inverse> 'isInverse' control prop> Text 'Select' appears faded if 'isInverse' control prop is 'False'.

Describe the bug
Text 'Select' appears faded if 'isInverse' control prop is 'False'.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://storybook-preview-290--upbeat-sinoussi-f675aa.netlify.app/?path=/story/nativeselect--inverse'
Click on 'Controls'
Select 'isInverse' control prop
Toggle 'isClearable' to 'true'
Verify the preview

Expected behavior
Dark background should disappear if 'isInverse' control prop is 'False'.
Screenshots
https://somup.com/c3nD69TeB2
Desktop:
OS: [Win 10]
Browser [chrome]
Version [Version 98.0.4758.82 (Official Build) (64-bit)]
Additional context
As per the documentation, If 'isInverse' is true, the component will have inverse styling to better appear on a dark background. However, for 'NativeSelect>Inverse/Inverse Disabled/Has Error Inverse/Helper Message Inverse', text gets faded instead of disappearing the dark background.

The NativeSelect component is working as expected and isInverse has the correct behavior. In Storybook, the team has chosen to add a background for Inverse to make it clearer when the isInverse toggled is switched, but we do not expect that background color to also be toggled as it is not part of the component.
Closing this bug as "wontfix".


Fallback to cpu when reading Delta log files for stats

fixes #1197
[BUG] java.lang.NullPointerException when exporting delta table #1197
It appears the delta stats reading is returning special internal Databricks types which we can't handle. To deal with this we try to recognize this is dealing with the Json delta log files and make the entire plan that reads from those fall back to use the CPU.
I added an internal config to allow disabling this behavior just incase something weird happens.
This only happens on databricks and is very hard to reproduce so I wasn't able to come up with test for it.

build

build

build

these failures are cast tests:
20:46:20  Caused by: java.lang.ArithmeticException: Casting 2.14748365E9 to int causes overflow
Nothing that changed here

build

build

build


boards: cy8ckit_062_wifi_bt_m{0,4}: Add OpenOCD support for flashing

The 0.9.5 SDK has support for the Cypress PSoC6 family.  Add ablilty to
flash with openocd.  I've only tested flashing hello_world on the m0.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

@OlegKapshii please take a look

Codecov Report

Merging #11452 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #11452   +/-   ##
=======================================
  Coverage   48.38%   48.38%
=======================================
  Files         265      265
  Lines       42193    42193
  Branches    10137    10137
=======================================
  Hits        20413    20413
  Misses      17703    17703
  Partials     4077     4077

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 e053a47...bf2e561. Read the comment docs.

@OlegKapshii : Cool, thanks for testing, can you approve this PR then?

@pfalcon Can I do this? I do not see any suitable control on the page.

@OlegKapshii :

@pfalcon Can I do this? I do not see any suitable control on the page.

Yep, it's a bit hidden for people who aren't in reviewers already. But anyone should be able to cast a vote by clicking on "Files changed" tab, then you'll see "Review changes" green button below the tab headers.

@MaureenHelm updated w/changes requested, can you re-review.

recheck


Webpack fails to build ES6 in external packages

I'm trying to use an ES6 npm package (in this case one called gamesystem) in a project and build with webpack (and babel) it fails to build any ES6 code inside the external dependency, why is that? If I have the same code as a dependency with a relative path it works fine.
App code:
'use strict';

import World from 'gamesystem'; // Breaks
//import World from '../node_modules/gamesystem'; // Also breaks
//import World from '../gamesystem'; // Works (copied the same directory gamesystem outside the node_modules directory)

let world = new World();

Error:
ERROR in ./~/gamesystem/World.js
Module parse failed: /home/user/project/node_modules/gamesystem/World.js Line 3: Unexpected token
You may need an appropriate loader to handle this file type.
| 'use strict';
|
| import System from './System';
|
| export default class World {
 @ ./src/app.js 3:18-39

Webpack config:
'use strict';

// Modules
let WebpackNotifierPlugin = require('webpack-notifier');
let HtmlWebpackPlugin = require('html-webpack-plugin');

let config = {
  entry: {
    app: __dirname + '/../src/app.js'
  },
  devtool: 'source-map',
  devServer: {
    stats: {
      modules: false,
      cached: false,
      colors: true,
      chunk: false
    },
    proxy: require('./devProxy')
  },
  module: {
    loaders: [{
      test: /\.js$/,
      loader: 'babel',
      exclude: /node_modules/,
      query: {
        presets: ['es2015', 'stage-0']
      }
    },
    {
      test: /\.css$/,
      loader: 'style!css'
    },
    {
      test: /\.html$/,
      loader: 'raw'
    }]
  },
  plugins: [
    new WebpackNotifierPlugin(),
    new HtmlWebpackPlugin({
      template: __dirname + '/../src/index.html',
      inject: 'body',
      minify: false
    })
  ]
};

module.exports = config;

I asked on stackoverflow as well: https://stackoverflow.com/questions/35273956/webpack-fails-to-build-es6-in-external-packages

My fault, I was excluding the node_modules directory in the webpack config, thanks for a great software, but lets hope that we soon can peal off more and more of the layers of tools needed to run ES2015+

Oh, actually the source of the error is right here...
exclude: /node_modules/

Exclude has preceedence over include as far as I know. So maybe you want to use an array of paths to include instead? It’s the preferred way.
include: [
    /node_modules\/(gameworkd|someOtherModule|...)/,
    path.join(__dirname, "./app") // assuming that your config and app are in the same folder.
]

Am 08.02.2016 um 19:16 schrieb Fredrik Söderström<EMAIL_ADDRESS>I'm trying to use an ES6 npm package (in this case one called gamesystem) in a project and build with webpack (and babel) it fails to build any ES6 code inside the external dependency, why is that? If I have the same code as a dependency with a relative path it works fine.
App code:
'use strict';
import World from 'gamesystem'; // Breaks
//import World from '../node_modules/gamesystem'; // Also breaks
//import World from '../gamesystem'; // Works (copied the same directory gamesystem outside the node_modules directory)
let world = new World();
Error:
ERROR in ./~/gamesystem/World.js
Module parse failed: /home/user/project/node_modules/gamesystem/World.js Line 3: Unexpected token
You may need an appropriate loader to handle this file type.
| 'use strict';
|
| import System from './System';
|
| export default class World {
@ ./src/app.js 3:18-39
Webpack config:
'use strict';
// Modules
let WebpackNotifierPlugin = require('webpack-notifier');
let HtmlWebpackPlugin = require('html-webpack-plugin');
let config = {
entry: {
app: __dirname + '/../src/app.js'
},
devtool: 'source-map',
devServer: {
stats: {
modules: false,
cached: false,
colors: true,
chunk: false
},
proxy: require('./devProxy')
},
module: {
loaders: [{
test: /.js$/,
loader: 'babel',
exclude: /node_modules/,
query: {
presets: ['es2015', 'stage-0']
}
},
{
test: /.css$/,
loader: 'style!css'
},
{
test: /.html$/,
loader: 'raw'
}]
},
plugins: [
new WebpackNotifierPlugin(),
new HtmlWebpackPlugin({
template: __dirname + '/../src/index.html',
inject: 'body',
minify: false
})
]
};
module.exports = config;
I asked on stackoverflow as well: https://stackoverflow.com/questions/35273956/webpack-fails-to-build-es6-in-external-packages https://stackoverflow.com/questions/35273956/webpack-fails-to-build-es6-in-external-packages
—
Reply to this email directly or view it on GitHub https://github.com/webpack/webpack/issues/2017.

Even by putting my module in the include I get the same error.
{
  test: /\.(js|jsx)$/,
  include: [
     path.resolve(__dirname, './app'),
     /node_modules\/(c360-structured-tree-component)/
  ],
  loaders: [
    require.resolve('react-hot-loader'),
    require.resolve('babel-loader')
 ]
},

Here is the error.
/Users/rparent/Documents/CTSProjects/arsenal/cms/node_modules/c360-structured-tree-component/src/component/TreeContainer.js:3
import { connectToStores } from 'fluxible-addons-react'
^^^^^^

SyntaxError: Unexpected reserved word
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:414:25)
    at Module._extensions..js (module.js:442:10)
    at Object.require.extensions.(anonymous function) [as .js] (/Users/rparent/Documents/CTSProjects/arsenal/cms/node_modules/babel-register/lib/node.js:134:7)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Module.require (module.js:366:17)
    at require (module.js:385:17)
    at Object.<anonymous> (AssetsPage.js:5:1)
    at Module._compile (module.js:435:26)

Any clue on this, other solutions or something I might have done wrong?

@raphaelparent try this:
      include: [
        path.resolve(__dirname, './app'),
        /c360-structured-tree-component/
      ],

@raphaelparent were you able to figure this one out?

@ThariqS Unfortunately I don't remember exactly what I did since this has been more than a year ago, but I think I just compiled my module into ES5 and used it that way. I hope that helps and that you find a solution to your problem!

Hi @raphaelparent, @ThariqS -
I'm having a similar problem.
I have two node_module packages that are written in ES6. One of them imports the other.
I'm trying to figure out how to have webpack transpile the files. I've added the directory for each to the include: [] array in the webpack config, but I still get SyntaxError: Unexpected token export:
export function loadScript(src, cb) {
^^^^^^
SyntaxError: Unexpected token export
    at Object.exports.runInThisContext (vm.js:76:16)
    at Module._compile (module.js:542:28)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)

any ideas?

I also have this issue
trace:
/home/sh_eldeeb_2010/@eng-dibo/dibo/node_modules/@babel/runtime/helpers/esm/asyncToGenerator.js:17
export default function _asyncToGenerator(fn) {
^^^^^^

SyntaxError: Unexpected token 'export'
    at Module._compile (internal/modules/cjs/loader.js:891:18)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:991:10)
    at Module.load (internal/modules/cjs/loader.js:811:32)
    at Function.Module._load (internal/modules/cjs/loader.js:723:14)
    at Module.require (internal/modules/cjs/loader.js:848:19)
    at require (internal/modules/cjs/helpers.js:74:18)
    at Object.4023 (/home/sh_eldeeb_2010/@eng-dibo/dibo/dist/ngx-cms/core/server/main.js:3374:18)
    at __webpack_require__ (/home/sh_eldeeb_2010/@eng-dibo/dibo/dist/ngx-cms/core/server/main.js:3758:41)
    at Object.3344 (/home/sh_eldeeb_2010/@eng-dibo/dibo/dist/ngx-cms/core/server/main.js:1988:148)
    at __webpack_require__ (/home/sh_eldeeb_2010/@eng-dibo/dibo/dist/ngx-cms/core/server/main.js:3758:41)
The terminal process "bash '-c', 'npm run serve'" terminated with exit code: 1.


AKTimelineTap

AKTimelineTap is an audio tap on the render thread, based on AKRenderTap.  AKSamplerMetronome is a class that utilizes it, It triggers audio with sample accuracy and can be synchronized perfectly with other audio sources like AKAudioPlayer.  There's a new example project, MetronomeSamplerSync, that shows how to use AKSamplerMetronome, but AKTimelineTap is the real advancement.
Also, there's a bug fix for AKButton in there, totally related.

Exciting!


Handle duplicate addresses [TE-490]

Now P2pPeers internally maintains the set of peer addresses and report the error when trying to add a peer with already existing address.

OK, not sending Nack yet.


Clean up some stuff

Under position check there was an extra separator, making it look weird, so I removed that
Made the Bruteforce Info window look a tiny bit cleaner

i did a bigbrain move, added to pr ^


Infinispan's default cache also creates it, not so when providing a name.

This is a somewhat obscure "feature". And got "broken" with the CLI #92 refactoring. Now, limitador always expects the cache to be there and never tries to create it.
To be verified: it looks like when the cache doesn't exist tho, limitador silently fails to update/create counters, and always falls back to brand new counters and lets all traffic thru.

See #108

That feature was 🔥


Improve code completion UI

Provide more completion options
Annotate options with icons as in Eclipse
Typing when the dialog is up needs to do the right thing
(Imported from Bitbucket #7 reported by maxg on 2010-03-13)

(Changed responsible from nobody to phy.)
(maxg on 2010-05-25)

Let's move icons to a separate issue and call this done!
(maxg on 2010-06-15)


InsufficientPermissions error when accessing Fit data?

I'm trying to access Fit data in my app, but whenever I try to create a datasource I get a Google::Apis::ClientError: insufficientPermissions: Insufficient Permission error.
Here's the code I've got
require 'google/apis/fitness_v1'
require 'googleauth'

scopes = ['https://www.googleapis.com/auth/fitness.activity.read']
authorization = Google::Auth.get_application_default(scopes)

some_headers = {}
authorization.apply(some_headers)

fitness = Google::Apis::FitnessV1
fit = fitness::FitnessService.new
fit.authorization = authorization

s = fit.create_user_data_source(google_id.uid)

google_id is accessed earlier in my code, so I know that's not what's raising the error, and when I check the link it runs the user's ID is included in it, so I know it's successfully retrieving the uid from my database, but I can't figure out why I'm getting this error. My application has the FitAPI enabled.

It's because you're not requesting the correct scopes. In your example, you're only asking for read access, but are attempting to write data (create a data source.)
See https://developers.google.com/fit/rest/v1/reference/users/dataSources/create


Issues with Subdirectory Setup on Apache2

Hey,
I got some issues setting up BookStack in a subdirectory. I have followed the instructions Subdirectory Setup and once added the configuration with custom folders in my vhost. Unfortunately, the site does not load at all.
BookStack should be available at https://domain.name/bookstack.
The location of BookStack is /var/www/bookstack/... .
I have also added the domain in the .env.
Here is my config.
<IfModule mod_ssl.c>

<VirtualHost domain.name:443>
        ServerName domain.name.tld
        ServerAdmin webmaster@localhost
        DocumentRoot /var/www/html

        ErrorLog ${APACHE_LOG_DIR}/error.log
        CustomLog ${APACHE_LOG_DIR}/access.log combined

       # BookStack Configuration
       Alias "/bookstack" "/var/www/bookstack/public"

       <Directory "/var/www/bookstack/public">
          Options FollowSymlinks
          AllowOverride None
          Require all granted

          RewriteEngine On
          # Redirect Trailing Slashes If Not A Folder...
          RewriteCond %{REQUEST_FILENAME} !-d
          RewriteRule ^(.*)/$ /$1 [L,R=301]

          # Handle Front Controller...
          RewriteCond %{REQUEST_FILENAME} !-d
          RewriteCond %{REQUEST_FILENAME} !-f
          RewriteRule ^ index.php [L]
       </Directory>

       <Directory "/var/www/bookstack">
          AllowOverride None
          Require all denied
       </Directory>
        # End BookStack Configuration

        SSLCertificateFile /path/to/certificate/file
        SSLCertificateKeyFile /path/to/certificate/key/file
        Include /etc/letsencrypt/options-ssl-apache.conf

        RewriteEngine on
        RewriteCond %{SERVER_NAME} =domain.name.tld
        RewriteRule ^ https://%{SERVER_NAME}%{REQUEST_URI} [END,NE,R=permanent]
</VirtualHost>

</IfModule>
# vim: syntax=apache ts=4 sw=4 sts=4 sr noet

Thanks for ur help.

I'm not sure if this might be the solution to your problem, but for mod_rewrite to work (which laravel uses for pretty URLs) you need at least
AllowOverride FileInfo

@JustNoopie Could you also confirm your version of BookStack (Look at the version file for this)?
Also, to help solve your CSS issues, Could you open on the login page (Where styles are not loading) then view the browser devtools (Ctrl+Shift+I) in chrome > View the console tab > Reload the page > Screenshot/copy any errors you have and paste here (May contain your domain name so blank out if you want to keep that hidden).
I suspect you're either using the development files which have no styles or the path to the styles, that BookStack generates, is incorrect.


Running the setup_cpu.sh script on Ubuntu 16.04 fails with errors in npm

Hi,
Just cloned the repo and ran the setup_cpu.sh script as instructed, gives me errors during the build with npm packages. Specifically the  ``sass-loaderrequiresnode-sass >=4. Please install a compatible version
Command Used:
source setup_cpu.sh
Screenshot:

Complete Log File:
2019-10-01T14_24_22_398Z-debug.log

Thanks. I will try to update the files such that the newest sass will be installed. For now, try running npm update sass and then npm run wp. That might work.

Updating sass module did not work, I went down this rabbit hole of fixing individual package dependencies with npm audit etc. and that did not work either.
Fix Used:
Finally, The trick was to installed the older version of nodejs ,
Commands:
conda uninstall nodejs
conda install nodejs==9.11.1 -c conda-forge
which node should give <your conda path>/bin/node
node --version #Just to verify
npm -V #Just to verify

Just a thought, pip freezeing all the python/conda packages for next version would be helpful in replicating.

Thanks @mohammedayub44
I have met the similar problem. And use the older version of nodejs solves this problem.


Return true on successful save! calls

For consistency with ActiveRecord, I've made it so ActiveNode and ActiveRel return true when save! does not error out.
TODO

[x] Not sure if it's worth fighting Rubocop for or instead of || -- is there a magic cookie?

This is great, thanks so much!
A bit more subtlety would be nice from rubocop regarding "or".  Rubocop goes mainly off of the ruby-style-guide, though which says "The and and or keywords are banned. It's just not worth it. Always use && and || instead."
Avdi Grimm has an excellent free RubyTapas on the topic, though:
http://devblog.avdi.org/2014/08/26/how-to-use-rubys-english-andor-operators-without-going-nuts/
I'm going to merge in your changes and test something in Rubocop...

Ok, it looks like Rubocop doesn't allow for any configuration of this cop.  You can disable the cop for specific files, but I definitely like making sure that we use the symbolic versions for boolean expressions.  So for now let's keep the line comments, but I've made a comment over here:
https://github.com/bbatsov/ruby-style-guide/issues/415
Thanks again!

Thanks, @cheerfulstoic! Agreed on the rubocop/style guide as well -- it'd be great if or and and were allowed for control flow. Maybe rubocop could look for lines containing or/and and if/unless, and flag those as no good, terrible Rubies. Otherwise, we would get a free pass for control flow.

One other issue is the lower precedence of = against the english operators.  There are cases where you might want it to go both ways, though (I think both are demoed in the screencast).
We'll see how it goes down with the style guide repo.  Though yeah, even if that gets changed it might take some clever hacking to find a way to have rules like you'd want.  I know Rubocop will actually tell you when your code is ambiguous, though, so maybe there's a shot...


create a automated changelog

prof of concept is working on https://github.com/Strangerware/sw-vast-errors/releases so closing this issue


A few Travis changes

Updates the STM32 build to the new Ubuntu to get a better compiler without needing the PPA. Also splits out the SDL install scripts as they're getting a bit big. (I may have something that requires adding more to them somewhere...)

Oh nice! All those wrinkles melt away if you just wait long enough 😆
Thanks for making a PR for this!


nixos/pantheon: make it possible to remove core packages

Description of changes
This is #151005 but for Pantheon.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Did some quick test by running github:bobby285271/test-pantheon with https://github.com/NixOS/nixpkgs/commit/f285f3afbdceca1fca3da411fd50edb7dbb3b3d5, the desktop seems to work™.
Screenshot

@ofborg test pantheon


Extend entity descriptor with human-readable fields

We should consider extending the entity descriptor with optional human-readable descriptor fields like name, website, security contact, etc. This would make it easier to compile a nice list of all entities participating in the system.

Yeah, it seems like this "public key to {metadata}" map can live outside the consensus layer (e.g., in a dashboard/explorer-like application). And entities can provide signed metadata to avoid spamming with random "claims".

One issue is that the human-label to public key map becomes security critical, since if it were subverted it would cause transaction targets to be mis-selected and such things to go awry.


license

I didn't find any information about the license in the readme.
What's the license of this repo?

Is it the same as the original implementation? MIT license?

Thank you.


feat: use adapter pattern for store finder

use adapter and connector for backend calls related to store finder
remove redundant types related to store-finder, (i.e. Location, LatitudeLongitude, OpeningHours) and replace with the standard ones (relevantly PointOfService, GeoPoint, WeekdayOpeningDay)
change type of Time.hour and Time.minute from Uint8Array to number
remove unnecessary mapping from property longitudeLatitude to geolocation in find-stores.effect

close GH-2434

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Crashes

There is an issue when I install the tweak it causes my setting app to completely crash then disabling some of my tweak forcing me to boot into safe mode and uninstalling the tweak.

I have the exact same problem 😩 I used to have cask 2 working perfectly then I rejailbroke my phone, downloaded it again and now it causes the same crash all the time

Version 1.2.0 should fix this issue. Can you please confirm @joshiepoo21 and @ddlawson?


Make console breadcrumbs safe to use

While the console breadcrumbs are great in concept, they've actually led to more crashes in our app than they've helped to diagnose. This PR resolves the underlying issue and adds safeguards to prevent other undiagnosed issues from crashing people's apps.
The underlying issue
If a console.log statement receives null or undefined as an argument, the code will throw a TypeError because you cannot call toString() on these values. In all cases, it's safer to use the String constructor than to call toString() on a value. (The former will ultimately delegate to toString() if defined, but will gracefully handle primitives like null and undefined.)
The safeguard
As new features are added to the tracker, it's important to wrap them in a top-most try/catch. We all make mistakes and cannot fully anticipate whether new code is going to result in problems. However, a top-level try/catch is an inexpensive way to mitigate the fallout from any unforeseen corner cases.
These precautions are especially important for a product like this.

👍  Can you publish a patch release, too?

Yes, it's on its way 🚢


⚠ Thumbnails API Endpoint has degraded performance

In b5342c8, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1071 ms

Resolved: Thumbnails API Endpoint performance has improved in 6198eea after 8 minutes.


e2e: out of gas in location

Describe the Bug
zetacore:
8:12PM ERR panic recovered in runTx err="out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 416639: out of gas" module=server
8:12PM ERR panic recovered in runTx err="out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 416639: out of gas" module=server
8:12PM ERR panic recovered in runTx err="out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 416639: out of gas" module=server
8:12PM ERR panic recovered in runTx err="out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 416639: out of gas" module=server
8:12PM ERR panic recovered in runTx err="out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 413601: out of gas" module=server
8:13PM ERR panic recovered in runTx err="out of gas in location: WritePerByte; gasWanted: 400000, gasUsed: 400445: out of gas" module=server
8:14PM ERR panic recovered in runTx err="out of gas in location: WritePerByte; gasWanted: 400000, gasUsed: 401848: out of gas" module=server
8:16PM ERR panic recovered in runTx err="out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 413525: out of gas" module=server

zetaclient:
2024-06-19T20:12:18Z INF MonitorInboundResult: successfully resent tx, txHash: 2D773AC5E60EC4583FB134640B6D99012FFF3050BC0E7E0154A3D0BD42CC7807, log out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 416639: out of gas module=ZetacoreClient
2024-06-19T20:12:22Z INF MonitorInboundResult: successfully resent tx, txHash: 748BF41DF4318C5A0CD4F660BF96A3FB5B1BB7E552DFD98E61F1D65296BB581E, log out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 416639: out of gas module=ZetacoreClient
2024-06-19T20:12:22Z INF MonitorInboundResult: successfully resent tx, txHash: E8D71EEF7C42BC039ECFC185EDCF85F5B5F670FC7303EB57C4E7A8E744BC2867, log out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 416639: out of gas module=ZetacoreClient
2024-06-19T20:12:31Z INF MonitorInboundResult: successfully resent tx, txHash: F8F264605C461CA10C9E2821F8A5B9FC4CFA125E17E02845661246427A763974, log out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 416639: out of gas module=ZetacoreClient
2024-06-19T20:12:31Z INF MonitorInboundResult: successfully resent tx, txHash: 3F8D1A8773FA1E3FB6783C6553556028DFAC56432168EBEDB4C16CB05E44AEF3, log out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 413601: out of gas module=ZetacoreClient
2024-06-19T20:13:18Z INF MonitorInboundResult: successfully resent tx, txHash: 4ED970946C707CE7D8E04B624E6C329E2E6E366F8C0AA5F3F5F1E3ED74864C40, log out of gas in location: WritePerByte; gasWanted: 400000, gasUsed: 400445: out of gas module=ZetacoreClient
2024-06-19T20:14:25Z INF MonitorInboundResult: successfully resent tx, txHash: E101E0C52BCEEA24236C7165C6800D738864463475DC2A5398AA74DD7A925099, log out of gas in location: WritePerByte; gasWanted: 400000, gasUsed: 401848: out of gas module=ZetacoreClient
2024-06-19T20:16:04Z INF MonitorInboundResult: successfully resent tx, txHash: C8DF51D86FE734A3410981CB06978A0760E101CB2BCC095DE476623B5E572F5D, log out of gas in location: ReadPerByte; gasWanted: 400000, gasUsed: 413525: out of gas module=ZetacoreClient

To Reproduce
make start-e2e-test
Expected Behavior
We should not see this during e2e this much right?

The max limits are set to
PostVoteInboundExecutionGasLimit = 4_000_000
PostVoteInboundMessagePassingExecutionGasLimit = 4_000_000
PostVoteOutboundRevertGasLimit = 1_500_000
I think it is fine to send with a lower limit of 40000 and retry from there as the finalizing vote would cost extra .
Curious, have these errors just started showing up ?


Fix: MudScrollToTop throws exception with a page change.

Description
fixes #5015
replaces this PR #5052
Made by analogue from mudScrollSpy.js
https://github.com/MudBlazor/MudBlazor/blob/66894b28cdee27b576ae5a82841c6bbbf519efc3/src/MudBlazor/TScripts/mudScrollSpy.js#L12
How Has This Been Tested?
with reproducing the steps from the #5015 and verifying that error doesn't appear.
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] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

Hi @ScarletKuro and thanks for your work
I'm sorry in that case that our previous discussion has disappointed you and reduced your ambition. You can, of course, propose changes to the JS interop service. I was only reluctant to the reactive part not your improvements to the service.  We can accept this PR as it is, but if you want to improve more, feel encouraged to do so.

@just-the-benno I'm not disappointed. It was only to to raise the attention that event doesn't really work well with Blazor because it's almost all the way async which forces you to use async void in some places which is not really good, and that you might end up with more ugly and boilerplate code, and here is well known reactive which can solve this problems, but it wasn't pretending to be the solution that should be accepted, so don't sweat about it.
The most important is to fix the bug, which this PR does.

@henon Ready for your review

Thanks!


Support dotnet watch as a launch target

Have tasks.json configured to use "dotnet watch" when a user initiates a "Start without debugging".

Hi @metoly, in VS Code, start without debugging still involves the debugger, and aren't setup to start the app a completely different way.

I should add - as of 1.18, the C# extension will automatically add a tasks.json task for starting "dotnet watch". So there is now an easy way to kick that off.


Provide a non-templated Optimizer type.

The structure tch::nn::Optimizer<T> has T as a template parameter in order to store the config used to create it in a private field. As far as I can tell, this config is not used by tch and cannot be used by users since it is private. As a result of this templating, one cannot write code that takes in an arbitrary optimizer (Adam/Sgd/etc) without pointlessly templating on T. (At least without writing a wrapper around Optimizer that forwards all of its methods).
It also means that you cannot write a function like fn make_optimizer(&str) -> Option<Optimizer>  to e.g. dynamically select the optimizer type from a command-line string.
I think that <T> and the associated config: T field should be dropped from tch::nn::Optimizer.
Alternatively, a backwards-compatible approach could be to implement From<Optimizer<T>> for Optimizer<()> that drops the config (maybe with an alias for Optimizer<()>).

Having COptimizer available is sufficient for my purposes in terms of a non-templated optimizer type. It might still be nice in general to have the methods of Optimizer be available without templating but feel free to close this issue if you prefer to leave Optimizer as-is.


Allow 10x cellranger sample indexes

This allows CellRanger style indexes to be specified, but does NOT check that validity of the indexes.
Closes #92

Thanks for the test!


After right-clicking, the playback section follows whenever the mouse pointer is moved.

Bug description
After right-clicking, the playback section follows whenever the mouse pointer is moved.
This is not the behavior I want. Even after right-clicking, I want to move to that section whenever I click as I want.
I will attach a reproduction video from the example code.
https://github.com/katspaugh/wavesurfer.js/assets/7166022/55a70155-ca6e-4424-be0e-07abb3e68f8f
Environment

Browser: Chrome

Minimal code snippet
Expected result
Even if you right-click the mouse I want to move to that section whenever I click as I want
Obtained result
After right-clicking, the playback section follows whenever the mouse pointer is moved
Screenshots

Oof, that's definitely not the behavior we want :)
Thanks for the report!

thanks @katspaugh !


Host documentation online

Hi there. Documenter.jl is the go-to standard in creating a documentation for Julia packages. Besides the plethora of features it involves, it also allows you to build your documentation using Julia code.
Documenter.jl also has clear instructions on how you can host your documentation on the very github repository that you host your code, see for example here: https://juliadocs.github.io/Documenter.jl/stable/man/hosting/
There seems to be a dedicated folder already containing some documentation docs/ but it is inaccessible.

The documentation is already at
https://fhell.github.io/NetworkDynamics.jl/
But there was no link. We need to update the readme to show this, and rework the Docs to start with.

I'd also recommend to replace all Julia code blocks in teh documentation with  @example code blocks, to ensure that the output of what you present is indeed what you would like it to be.

The documentation has been up for a while, so i think this issue can be closed. However one problem with the @example blocks remains. Apparently the ubuntu-worker that builds the docs is missing a library and prints the follwing error message into the documentation:
/home/runner/.julia/packages/GR/NSt7D/src/../deps/gr/bin/gksqt: error while loading shared libraries: libQt5Widgets.so.5: cannot open shared object file: No such file or directory
connect: Connection refused
GKS: can't connect to GKS socket application
Did you start 'gksqt'?

GKS: Open failed in routine OPEN_WS
GKS: GKS not in proper state. GKS must be either in the state WSOP or WSAC in routine ACTIVATE_WS

@Datseris, do you know how to fix this by any chance? It seems in the docs for DynamicalBillards.jl you don't have that problem.

I use PyPlot for DynamicalBilliards.jl . You should report this issue to GR's github page. However, I think that adding:
ENV["GKSwstype"] = "100"

as the first line of your make.jl tends to fix problems on Travis.

@lindnemi the docs have been up for a while, but at which link? This https://fhell.github.io/NetworkDynamics.jl/ doesn't work for me.

Thank, I'll try out your suggestions.
docs can be found here: https://fhell.github.io/NetworkDynamics.jl/dev/

I added a redirect from  https://fhell.github.io/NetworkDynamics.jl/  to the dev version of the docs. Eventually the redirect should be changed to the stable version, however, as of now that is terribly out of date. @FHell we should think of incrementing the version to 0.2.1 in order to include the improved docs in the package.


Parallelize Pipeline Generation

We are hitting timeouts. Filed a PR elsewhere to allow us to override that.
I'm not saying we should entirely parallelize it. But at least the the current serial for-each here into a Parallel.Invoke or the like.

Been a while since we saw timeouts on this.  Closing.  Sima did add caching logic which sped up a lot of the interactions with DevOps.  That may be why we're not seeing timeouts anymore.


Add new compiler defaults and mock-chain improvements

Compiler: Set ErgoTree output to v1 by default.
MockChain: Add MockChain.clearUTxOSet() method.
MockChain: Add MockChain.jumpTo() method.

Codecov Report

Merging #19 (6c73d4a) into master (ca38137) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #19   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           77        77
  Lines         5547      5582   +35
  Branches      1168      1170    +2
=========================================
+ Hits          5547      5582   +35

Impacted Files
Coverage Δ

packages/compiler/src/compiler.ts
100.00% <100.00%> (ø)

packages/mock-chain/src/mockChain.ts
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


rpm: Refine prefix matching for sub RPMs to reduce fragility

The current prefix matching for sub RPMs can potentially fail in two ways:

if we have two overlapping prefixes we may match the shorter prefix against the wrong RPM
the shorter prefix will match repeatedly even after finding its target

This change keeps track of which sub RPMs have already match so avoid
double matching a shorter prefix.  Additionally, we sort the sub RPM
list by name length in reverse order.  This ensures that we prefer to
match the longest prefixes first and avoid double matching with the
same prefix.

This fixes https://github.com/bazelbuild/rules_pkg/issues/875

Can we add a test that verifies the behavior being fixed is fixed?

Can we add a test that verifies the behavior being fixed is fixed?

Yes.  I'll try to get back to that this week.

Looks good, but it could use a test.

Done


no_std compatibility: handling the 4 incompatible crates

@romac In ibc-rs module and dependencies.tendermint module, there are 4 crates that's incompatible with no-std, as below:

[X] anomaly: to be removed? #988
[X] thiserror #988
[X] serde-repr: replace with alternative? no longer relevant
[ ] toml

If you have any progress of processing the crates above for no-std please update here, thank you.

anomaly
Yes, we need to remove it.
thiserror
Replace with displaydoc and a conditional std::error:Error instance.
See https://github.com/dtolnay/thiserror/pull/64#issuecomment-735805334
serde-repr
It looks like this crate is not used at all in tendermint, and we can thus just remove its from the dependencies.
toml
This crate is only used to parse a Tendermint configuration file. I would suggest only defining this function when no_std is not set, and thus only enable the toml dependency when the no_std feature is disabled.
@livelybug Does this make sense to you?

Note that we often use the .context method of anomaly to provide more context when throwing an error. Each use of .context should therefore be replace with a specific error variant:
Before:
use thiserror::Error;

type Error = anomaly::Error<Kind>;

#[derive(Clone, Debug, Error, PartialEq, Eq)]
enum Kind {
   #["foo error"]
    Foo,
}

fn some_func() -> Result<(), Error> {
  if true {
    Err(Kind::Foo.context(format!("bar: {}", n)).into())
  } else {
    Err(Kind::Foo.context("baz").into())
  }
}

After
use displaydoc::Display;

#[derive(Clone, Debug, Display, ...)]
enum Foo {
  // bar: {0}
  Baz(i32),
  // baz
  Baz
}

#[cfg(feature = "std")]
impl std::error::Error for Foo {}

#[derive(Clone, Debug, Display, ...)]
enum Error {
   // Foo error: {0}
    Foo(Foo),
}

#[cfg(feature = "std")]
impl std::error::Error for Error {}

fn some_func(n: i32) -> Result<(), Error> {
  if true {
    Err(Kind::Foo(Foo::Bar(n)))
  } else {
    Err(Kind::Foo(Foo::Baz))
  }
}

I'm not sure if the proposed composition of Foo and Bar errors handle the stack traces well? Actually, with this the backtrace is not captured at all. Or perhaps backtraces are not that useful in practice? (I don't really need it personally, just thought some developers may find the backtraces useful)

That's a good point, thanks! I will investigate to see if we can somehow retain backtraces when std is enabled. Backtraces are indeed useful imho but if we retain them at the relayer and relayer-cli it might be enough for our needs, cf. https://github.com/informalsystems/ibc-rs/pull/950#issuecomment-842557502.

anomaly
Yes, we need to remove it.
thiserror
Replace with displaydoc and a conditional std::error:Error instance.
See dtolnay/thiserror#64 (comment)
serde-repr
It looks like this crate is not used at all in tendermint, and we can thus just remove it from the dependencies.
toml
This crate is only used to parse a Tendermint configuration file. I would suggest only defining this function when no_std is not set, and thus only enable the toml dependency when the std feature is enabled.
@livelybug Does this make sense to you?

hi, For anomaly crate, have you some advise? I am try to use anyhow to re-code.

anomaly
Yes, we need to remove it.
thiserror
Replace with displaydoc and a conditional std::error:Error instance.
See dtolnay/thiserror#64 (comment)
serde-repr
It looks like this crate is not used at all in tendermint, and we can thus just remove it from the dependencies.
toml
This crate is only used to parse a Tendermint configuration file. I would suggest only defining this function when no_std is not set, and thus only enable the toml dependency when the std feature is enabled.
@livelybug Does this make sense to you?

I have re-code, and meet some  error, for anomaly. I see your use eyre move to anomaly, I have been do this. But, in no_std env, eyre and displaydoc (support no_std thiserror replace displaydoc). Have been some error. So I use anyhow and displaydoc to do this, But I have other error. https://github.com/informalsystems/tendermint-rs/issues/884

All the problematic dependencies have been removed. So closing this one.


[ISSUE #3367] selector match only once

// Describe your PR here; eg. Fixes #3367

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3368 (a87f188) into master (d47e617) will decrease coverage by 0.00%.
The diff coverage is 75.34%.

@@             Coverage Diff              @@
##             master    #3368      +/-   ##
============================================
- Coverage     65.25%   65.24%   -0.01%
- Complexity     5241     5251      +10
============================================
  Files           760      760
  Lines         21224    21282      +58
  Branches       1932     1934       +2
============================================
+ Hits          13849    13885      +36
- Misses         6089     6107      +18
- Partials       1286     1290       +4

Impacted Files
Coverage Δ

...apache/shenyu/plugin/base/cache/BaseDataCache.java
80.86% <66.66%> (-10.31%)
:arrow_down:

...pache/shenyu/plugin/base/AbstractShenyuPlugin.java
63.41% <90.00%> (-4.88%)
:arrow_down:

.../plugin/base/condition/strategy/MatchStrategy.java
91.66% <91.66%> (ø)

...ugin/base/condition/strategy/AndMatchStrategy.java
100.00% <100.00%> (ø)

.../base/condition/strategy/MatchStrategyFactory.java
100.00% <100.00%> (ø)

...lugin/base/condition/strategy/OrMatchStrategy.java
100.00% <100.00%> (ø)

...ng/rocketmq/rocketmq/RocketMQLogCollectClient.java
72.00% <0.00%> (-4.00%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
62.76% <0.00%> (-1.07%)
: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 d47e617...a87f188. Read the comment docs.

There is some logic wrong.

see https://github.com/apache/incubator-shenyu/pull/3390


Fix procedure definition blocks with trailing text

Resolves
Fixes #1658
Proposed Changes
Adds a bunch of special casing to the rendering code:

If a dummy input follows a statement input on a procedure definition block, let them stay in the same row
If a dummy input is in a row after a statement input, for reasons given above, don't add the input's field widths to the running tally.
When rendering a define block, check if there's a following dummy input and render its fields inline, then adjust size accordingly.

Reason for Changes
The old code broke the block into multiple lines.  This was surfaced in Magyar and Korean.
Test Coverage
New:

Old:

Making this code work:

Sweet! Glad to see my bug fixed!


Harry Potter and the Sorcerer's Stone errors when using Turnip mode

Hi. My device has Snapdragon 625 and as a PoC I tried to run this game. When using Software mode, the game runs at 8FPS. When using Turnip mode, the game gives error related to Unreal Engine (General Protection Fault
History: UGameWindow) or something like that.

Sorry, on Snapdragon 625 only VirGL or LLVMpipe is supported, and not all games can run


[Suggestion] Cache groups for components

Simple suggestion that I'd like to run past everyone.
Rather then an [SingleEntity] entity doing a look up this way:
public Entity audioSourceEntity { get { return GetGroup(Matcher.AudioSource).GetSingleEntity(); } }

Why not store a reference to the group so it's only looked up once?
public Entity audioSourceEntity { get { return groupRef.GetSingleEntity(); } }

I imagine dictionary lookup you are doing for the group is fairly fast, but I wonder for every frame execution systems if this would be a little bit faster. Enough to satisfy a little bit more complexity for the generated code.
Thoughts?

Groups are cached the first time you get one. So there's no lookup :)

There is a still a dictionary lookup:
public virtual Group GetGroup(IMatcher matcher) {
    Group group;
    if (!_groups.TryGetValue(matcher, out group)) {
        group = new Group(matcher);
        var entities = GetEntities();
        for (int i = 0, entitiesLength = entities.Length; i < entitiesLength; i++) {
            group.HandleEntitySilently(entities[i]);
        }
        _groups.Add(matcher, group);

        for (int i = 0, indicesLength = matcher.indices.Length; i < indicesLength; i++) {
            var index = matcher.indices[i];
            if (_groupsForIndex[index] == null) {
                _groupsForIndex[index] = new List<Group>();
            }
            _groupsForIndex[index].Add(group);
        }

        if (OnGroupCreated != null) {
            OnGroupCreated(this, group);
        }
    }

    return group;
}

_groups.TryGetValue(matcher, out group) is pretty fast, but for an extra boost it might be worth caching.

Thanks, I'm always happy if we can identify more places where we can save some cycles :)
Yes, this would save one dict lookup.
Thoughts:

We would need to store a group in the entity for each SingleEntity which increases the memory footprint of each entity.
When you store references to groups, you should also always subscribe to pool.OnGroupCleared and add logic to set the group field to null and retrieve it again from the pool if it's null.

I could add a test in the performance test suite and profile it, to see if there are any gains that would justify the increased complexity.

I guess the more obvious thing would be to cache the entity itself.
I find myself doing a enity lookup in a system and saving it to a local var just to avoid duplicate lookups. It would be nice to not have to do that but the performance difference is probably pretty slight.

I looked into this further in my system code where I am caching bool for component flags. I actually don't see much difference (doesn't show up in the visual debuger so it's less then 0.001ms) between caching in a boolean, using the current code or caching the group like the following:
Group _cachedGroup;

public Entity gameOverEntity {
    get {
        if (_cachedGroup == null)
        {
            _cachedGroup = GetGroup(Matcher.GameOver);
        }
        return _cachedGroup.GetSingleEntity();
    }
}

So this may be a case of premature optimization on my part. In this case it was just a system with a single lookup, it may make more a difference on systems with multiple lookups.


Typing issue with LayoutServerLoadEvent

Describe the bug
If I have a +layout.server.ts like the following, I get a TypeScript warning in VS Code "Parameter 'event' implicitly has an 'any' type"
import type { LayoutServerLoad } from './$types'

export function load(event): LayoutServerLoad {
  const locals = event.locals
  const { user }: { user: User } = locals
  return {
    user
  }
}

Reproduction
See above.
Logs
No response
System Info
System:
    OS: macOS 12.5.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 3.88 GB / 32.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.8.0 - /opt/homebrew/bin/node
    npm: 8.18.0 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 104.0.5112.101
    Firefox: 103.0.2
    Safari: 15.6.1
  npmPackages:
    @sveltejs/adapter-node: latest => 1.0.0-next.87
    @sveltejs/kit: latest => 1.0.0-next.445
    svelte: ^3.49.0 => 3.49.0
    vite: ^3.0.9 => 3.0.9

Severity
annoyance
Additional Information
No response

I believe you've made a mistake writing the type annotation.
Can you try replacing
export function load(event): LayoutServerLoad {

with
export const load: LayoutServerLoad = (event) => {

Yup, LayoutServerLoad is the type of the load function itself, not of the return value of the function.

You're right. Sorry about that. I've been switching back and forth between arrow and standard functions.


Update Versioning of Azure License Update Doc

Description Of Changes
Update versioning standard for chocolatey-license package as well as some minor linting.
Motivation and Context
Keeping with new versioning standard required by the intune integration.
Testing

[X] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

[ ] Minor documentation fix (typos etc.).
[X] Major documentation change (refactoring, reformatting or adding documentation to existing page).
[ ] New documentation page added.
[ ] The change I have made should have a video added, and I have raised an issue for this.

Issue #

Change Checklist

[ ] Requires a change to menu structure (top or left hand side)/
[ ] Menu structure has been updated

Related Issue
Fixes #628

Bumping for review. Someone, anyone ...


Min and Target SDK

Hey,
i saw that the App has Target SDK on 35 and Minimum SDK on 29 and I wanted to ask if you could make Minimum SDK to 31? I would appreciate that!

Can you explain why a higher min-sdk is better?

A higher Minimum SDK version implies that your app supports fewer Android versions, minimizing compatibility issues. Because higher SDK means also Versions which are supported better.

I think I can understand @Jeffrey2307's point of view: because Android 12 (SDK 31) is the last version that receives security patches. Further more, the less minSdk, the more internal compatibility layers, techniques etc. For example, lower SDK versions require v1/v2 signing, have somewhat different optimization levels, etc.

Ok, so it is not only about raising a number in a build file but also to drop compatibility code etc.

It is recommended. For example GrapheneOS uses for the App Store Minimum SDK 31. It makes the work easier.

@Jeffrey2307 it is very unlikely if @lberrymage resign from this. Look at the changelog, particularly v0.13.0. The support has been added for Android 10 and 11 (SDK 29 and 30, respectively).

@PatrykMis We never know. Let's see.

I don't plan on increasing the minimum SDK for now since it's not currently causing issues and it isn't much work to maintain support for Android 10 and 11. I agree that supported Android versions should be prioritized; however, existing users on Android 10 and 11 devices expect Accrescent to continue working, and it's not trivial for people to get a new device. I'll increase it once it becomes a burden to maintain. For what it's worth, I don't plan on adding support for Android versions lower than 10 (I forget exactly why, but it's significantly more work to support those).


Shopify theme : There is a layer of haze on my background pic

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

run command 'theme new --password=mypassword --store=store.myshopify.com --name=theme'
click the 'Customize' button and add a slideshow section ,then upload pic
there is a layer of haze on the slide background pic, u can see in the below pic

Expected behavior
This is a slideshow in shopify, I checked all the parent elements of the component, there is no corresponding css, it should be a css written separately by js, how can I remove the lager.
Environment (please complete the following information):

Additional context

We've flagged this with the internal Online Store Editor team here, but I'll close this issue as it's unrelated to Theme Kit.


Error Page

Route to an error page which will convey the site is not available at the moment. For any api related error we can route to this page.
Let me know do you want to include this feature .
Please assign me this issue under GSSOC'23

Nice idea @ahegde3 , make separate pages for 404 and api realted issue

hi @yashvardhan-verma  i am intrested to do it so please assign me

@PatilHarshh may be we can collaborate. You can make the UI and I can handle the routing and error boundary.

@ahegde3 That will great idea. Please tell @yashvardhan-verma  to assign me as well.

is it ok @ahegde3  and @yashvardhan-verma

Looks good to me, you and @ahegde3 can work collaboratively, When you both area done create a PR and mark that as done.


9/02 PM Publish

Docs Build status updates of commit 2d4b2b2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-6.3.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-6.3.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add in operator workplace showing clients who has been come by their appointment (new feature to release 2.0)

Need the feature which will show the list of waited clients in hall.

It's has implemented in release 1.0.2


Coordinate Transformation for pd_ee_delta_pose in PushCube-v1 Dataset

Hello,
We are using the dataset available on Hugging Face, specifically examining the first trajectory of PushCube-v1.  We have noticed that when using pd_ee_delta_pose, the action does not equate to delta_tcp_pose.
We found some explanations at this link, but we are still unclear about how to specifically transform the coordinates.
Could you please provide more detailed guidance on how to perform this transformation?

Could you share a script to reproduce this?
I will also note that we run by default 5 physics steps for every 1 control step with a sim freq of 100. It is possible that the robot has not reached the target (current_ee_pose + delta_pose) in the one control step. Although a difference of 0.2 in the X direction seems concerning. Difference in Z could be due to hitting the table and so you are prevented from going under it (hence larger z difference values).

Hi @zqy-an is this still an issue?

Thanks!
However, I am still very interested in how the action is converted into delta_pose. Could you please tell me where I can view the code responsible for this process?

See the replay trajectory tool under mani_skill/trajectory/replay_trajectory.py for how we convert action spaces.


[SwiftScan] Add APIs to load/check libSwiftScan directly

Add APIs for SwiftDriver library users to directly load/check the libSwiftScan directly before calling into its function.

Add APIs to query Driver for libSwiftScan location
Make verifyOrCreateScannerInstance to be public APIs to load or check the libSwiftScan.
Update verifyOrCreateScannerInstance to be more ergonomic to use. The function only returns true if the given path to libSwiftScan does not exist while most of the time caller actually errors out in those cases. Make throwing error the default behavior and caller can check for file existance before calling the function to perserve old behavior.

@swift-ci please test

ping

@swift-ci please test

@swift-ci please test windows platform

@artemcm @owenv Does those new APIs look ok to you?


Client can not get the header returned by server in the websocket handshaker

Describe the bug
I add a Set-Cookie header to the handshake response.  But the client can not receive it. And I found that client can only receive the handshake response returned by the gateway instead of the server. Maybe this is caused by the gateway has connected with the client before the server sends the handshaking response. By the code, the gateway really processes the WebSokcet request in this way.
Sample
This is the response headers without gateway: set-cookie: SOCKET_UUID=44b7d031-e0ca-414e-847e-3a5280db580a; path=/; expires=Tue, 20 Apr 2021 10:25:51 GMT; HTTPOnly, Access-Control-Allow-Credentials: true, Access-Control-Allow-Origin: xxx
With gateway:

Can you provide the things asked for when you created this issue, such as versions and steps to reproduce the problem?

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.

I just bumped into this and will furnish the details.
spring-boot-dependencies:2.5.2
spring-cloud-dependencies:2020.0.4 (spring-cloud-gateway-server:3.0.4)
We are using SCG to front a set of web applications implemented in Streamlit. Streamlit applications establish a websocket connection which is used to dynamically return data to the page. When the websocket connection is estabished, the streamlit server returns a Set-Cookie header that contains an XSRF token.  This token is then used for operations such as file upload.

After placing this app behind SCG, we noticed that we are no longer receiving the "additional" response headers. They are being lost as the response transits through the gateway.
I have put together a basic setup that replicates the issue using a simple Node.js websocket server, a client HTML page, and the SCG setup.
Steps to Reproduce

Run the node server.mjs file:

// install the one dependency:
// npm install ws

import { WebSocketServer } from 'ws';
import cookie from 'cookie';

const wss = new WebSocketServer({ port: 5678 });

wss.on('headers', (headers) => {
    headers.push('Set-Cookie: _xsrf=1234599');
    headers.push('X-Other-Header: hello');
});
wss.on('connection', function connection(ws) {
    console.log('Connection')
    ws.send('hi from the server');
});

Open the client.html in your browser. It should establish a connection to the websocket server and display hi from the server on the page.

<!DOCTYPE html>
<html lang="en">
<head>
    <title>WebSocket demo</title>
</head>
<body>
<script>
    port = "5678" // Go direct to server
    //port = "6600" // Go through gateway

    let ws = new WebSocket("ws://<IP_ADDRESS>:" + port + "/"), messages = document.createElement('ul');

    ws.onmessage = function (event) {
        let messages = document.getElementsByTagName('ul')[0],
            message = document.createElement('li'),
            content = document.createTextNode(event.data);
        message.appendChild(content);
        messages.appendChild(message);
    };
    document.body.appendChild(messages);
</script>
</body>
</html>

Open up the network tab, and you should also observe that the browser received two response headers (Set-Cookie and X-Other-Header).

Now configure and start up a gateway to front this server:

# Spring Cloud Gateway Config
spring:
  cloud:
    gateway:
      routes:
        - id: wss
          uri: http://localhost:5678
          predicates:
            - Host=**
server:
  port: 6600
logging:
  level:
    org:
      springframework:
        cloud:
          gateway: TRACE

The client.html page needs to be updated to go through the gateway. This is done by changing the port number from 5678 to 6600.

<script>
    //port = "5678" // Go direct to server
    port = "6600" // Go through gateway

Go back to the browser and refresh the client.html page. You should one again see hi from the server on the page. Now examine the network tab and notice our two added headers are absent from the server response.

I would appreciate any tips and/or hacks to get these headers through the gateway! Thanks.

@spencergibb, setting a breakpoint in WebsocketRoutingFilter.java, I can see the missing response headers in the proxySession.handshakeinfo.headers map.
But how to get them into the response back to the browser... 🤔.


gpodder: 3.11.1 -> 3.11.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gpodder is: A podcatcher written in python
meta.homepage for gpodder is: http://gpodder.org/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.11.2 with grep in /nix/store/8yvcb7ip2iva88gd9bw6ha5l25s69q8d-gpodder-3.11.2
found 3.11.2 in filename of file in /nix/store/8yvcb7ip2iva88gd9bw6ha5l25s69q8d-gpodder-3.11.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
gpodder

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8yvcb7ip2iva88gd9bw6ha5l25s69q8d-gpodder-3.11.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gpodder https://github.com/r-ryantm/nixpkgs/archive/8b0ab2c2db8bdc6d3fa2708c09b58cdf23b552da.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/8b0ab2c2db8bdc6d3fa2708c09b58cdf23b552da#gpodder

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8yvcb7ip2iva88gd9bw6ha5l25s69q8d-gpodder-3.11.2
ls -la /nix/store/8yvcb7ip2iva88gd9bw6ha5l25s69q8d-gpodder-3.11.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

gpodder

Maintainer pings
cc @svenkeidel @Mic92 for testing.

Builds and runs fine locally. Complains about missing Mutagen library if the tagging extension is enabled, but this is a longstanding issue not affected by this PR.

I've opened #257893 separately for the mutagen issue. It shouldn't interfere with this update.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gpodder is: A podcatcher written in python
meta.homepage for gpodder is: http://gpodder.org/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.11.3 with grep in /nix/store/6fzz17ahnm3iv00hh6bskbi90n437jb8-gpodder-3.11.3
found 3.11.3 in filename of file in /nix/store/6fzz17ahnm3iv00hh6bskbi90n437jb8-gpodder-3.11.3

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
gpodder

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6fzz17ahnm3iv00hh6bskbi90n437jb8-gpodder-3.11.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gpodder https://github.com/r-ryantm/nixpkgs/archive/76af224aa0b54a94b3e965ab0ebdb90e11f197a1.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/76af224aa0b54a94b3e965ab0ebdb90e11f197a1#gpodder

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6fzz17ahnm3iv00hh6bskbi90n437jb8-gpodder-3.11.3
ls -la /nix/store/6fzz17ahnm3iv00hh6bskbi90n437jb8-gpodder-3.11.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

gpodder

Maintainer pings
cc @svenkeidel @Mic92 for testing.


Add Initial Content

Videos
More Photos
Experience
Projects
Contact Info

Nice work! Can you combine some of these pages together to simplify your navigation menu?


ops/numpy.py: support key as list in GetItem

Please take a look, I had trouble loading a model that did things like this:
kernel_output = keras.layers.Concatenate()(
  [
    kernel(input[:, 2*other:2*other+2] - input[:, 2*ref:2*ref+2])
    for ref in range(n)
    for other in range(n)
    if other != ref
  ])

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 54.19%. Comparing base (c8700f4) to head (7e8532e).
Report is 102 commits behind head on master.

Files
Patch %
Lines

keras/ops/numpy.py
0.00%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #19310       +/-   ##
===========================================
- Coverage   80.14%   54.19%   -25.96%
===========================================
  Files         341      365       +24
  Lines       36163    39863     +3700
  Branches     7116     7748      +632
===========================================
- Hits        28982    21602     -7380
- Misses       5578    16683    +11105
+ Partials     1603     1578       -25

Flag
Coverage Δ

keras
54.19% <0.00%> (-25.80%)
:arrow_down:

keras-jax
?

keras-numpy
54.19% <0.00%> (-2.90%)
:arrow_down:

keras-tensorflow
?

keras-torch
?

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

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

Thanks for the PR! Please add a unit test.

Just did! I was not entirely sure which test to add to - I hope, I made a reasonable choice. Otherwise, please let me know!

You'll also need to run sh shell/format.sh to format the code.

You'll also need to run sh shell/format.sh to format the code.

I adjusted the patch with shell/format.sh.

@fchollet I updated the PR!


Moving Localization from JSON Files to Database

Currently using JSON files for localization in each service. Seeking advice on migrating to a database setup and managing it dynamically at runtime. Looking for tips and best practices.

The LanguageManagement Module uses dynamic localization. That is a commercial module.
You can also write your own similar module or make a request (or PR) on https://github.com/abpframework/abp/issues to add basic dynamic language management infrastructure.


Unintended limitations to skill scroll functionality

It seems the ASMC that adds a skill scroll to a unit's inventory can't do it without first removing the skill from their skills list, which makes no sense as it's not currently on their list?

I don't understand what you mean. Can you say it in more detail?

Sure, so in debug-event.c you have a function to add a skill scroll to Ephraim, but the item usability for that skill scroll is disabled unless you call Evt_RemoveSkill on the skill in that scroll before you add it into his inventory.
I'm confused why this is necessary, as he doesn't have the skill before he learns it via scroll?

because he has already learned that skill in debug mode

But he hasn't though. As you can see from my screenshot, he learns three skills in debug mode, MovBonus, Pass and Teleportation.
I went through the entire system, Fury isn't currently learned anywhere by anyone.

https://github.com/MokhaLeee/fe8u-cskillsys-kernel/blob/main/Wizardry/Core/SkillSys/kernel/LoadSkill.c#L150

Ah I think I understand now. It's available in his list of equippable skills that's why he can't learn it.


fix: Fixes BigNumber error due to incorrect number type

Fixes error Uncaught Error: lt() number type has more than 15 significant. It occurred when for example the users balance was 0.999368704999999998 ETH, the tx fee was 0.000063 (i.e. 3Gwei * 21000)

@ltfschoen I merged your PRs chronologically, but this one has some conflicts with 2 previous ones. Could you rebase this PR on top of latest master?
Feature-wise, we can re-test everything from this PR.

@amaurymartiny I've fixed the merge conflicts and used BigNumber in a couple of other places. The main change that resolves the error is .toString() though


Remove.h5ad suffix requirement for identifiers

See conversation here
The curator should not be required to, but may, include the .h5ad suffix when communicating either the curator tag OR the dataset id to the API. The backend must handle the determination accordingly for all endpoints. This includes the lambda logic that parses the s3 key.
Updated tag and id handling rules:

a tag may not match a uuid. (cc @Bento007 for approval)
A curator tag may not be an empty string.
Curators may not assign their own ids. They must assign a curator tag when uploading to s3, but are not required to use it thereafter; they may reference it by the dataset id which they can retrieve via GET /collections/{collection_id} and filtering for the new dataset, whether manually or by searching with the curator tag.

I would avoid special-casing .h5ad, because it increases the test matrix.

no h5ad at all please. We only need to add the h5ad suffix when they download the file, but even then they can modify the name themselves.

Ok.
I think we should make the change by allowing files whose url includes s3://cellxgene-dataset(env?-)submissions to skip this check

See a recent case where Jason tripped over the .h5ad requirement.

Per triage with @jahilton and @Bento007 - there was agreement that this refinement was required to simplify the API.

Special attn to one of the cases I hit, in case it isn't covered in the '.h5ad' work.
I believe it was rooted in uuid-formatted curator_tags, which should be blocked according to the notebook documentation. There was no error, file reports uploading but then nothing happens on the portal.

There was no error, file reports uploading but then nothing happens on the portal.

I captured this case above.

@Bento007 - you'll also update the notebooks?

Yes, I'll update the notebooks as well

Update the requirements of this ticket.

@Bento007 - Have the references to ".h5ad" been scrubbed from notebooks? For example - https://github.com/chanzuckerberg/single-cell-curation/blob/main/notebooks/curation_api/python/example_workflow.ipynb:
identifier = "topdir/subdir/file.h5ad"  # Or "<dataset_id>.h5ad"


[flutter_tools] read expression compilation results before starting next compilation

Currently it is possible to accidentally clobber expression compilation results, or read invalid/partial dill file.
While the expression compilation is structured to ensure only one compilation request at a time, this returns before the tool has read the file containing the dill - thus the next compile may be started or even finish before we begin reading. This can happen if multiple expression compilations are triggered near the same time.
To fix, add an additional hook for waiting before finishing each compilation request. Use this to read the expression compilation dill into memory before returning.

g3 targets are already failing.


APP-1357 Sync dev branch

Codecov Report

Merging #141 into dev will decrease coverage by 0.91%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##              dev     #141      +/-   ##
==========================================
- Coverage   90.93%   90.02%   -0.92%
==========================================
  Files         141      172      +31
  Lines        2768     3497     +729
  Branches      261      326      +65
==========================================
+ Hits         2517     3148     +631
- Misses        192      264      +72
- Partials       59       85      +26

Impacted Files
Coverage Δ

...heck/services/CompositeServiceHealthIndicator.java
0% <0%> (ø)
:arrow_up:

...check/application/ApplicationsHealthIndicator.java
69.23% <100%> (ø)
:arrow_up:

...ration/healthcheck/HealthCheckExecutorService.java
100% <100%> (ø)

...ion/healthcheck/AsyncCompositeHealthIndicator.java
78.04% <62.5%> (-13.62%)
:arrow_down:

...n/healthcheck/services/ServiceHealthIndicator.java
91.25% <81.25%> (-0.54%)
:arrow_down:

...ion/provisioning/properties/KeyPairProperties.java
0% <0%> (ø)

...ation/provisioning/service/ApplicationService.java
95.65% <0%> (ø)

...n/provisioning/IntegrationProvisioningService.java
79.81% <0%> (ø)

...ration/provisioning/service/AppKeyPairService.java
100% <0%> (ø)

...ning/service/ConfigurationProvisioningService.java
93.75% <0%> (ø)

... and 26 more

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 5456a6b...04b5014. Read the comment docs.

LGTM


If "FATAL:Cannot open database"

I am not able to start SyncTrayzor on PC although it was working there before. I tryied a complete new instalation, delete program folder, delete Roaming folder, reboot system - again and again the same.
Error message in log is:
FATAL: Error opening database: Syncthing v0.14.53 required
SyncTrayzor version 1.1.22,  OS Windows 10 Profesional
Can somebody help please?

I get this after following your steps:
2019-02-02 11:40:00.3769 #14152 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 131MiB. Private Memory Size: 84MiB. GC Total Memory: 9MiB
2019-02-02 11:40:00.8797 #14152 [Info] SyncTrayzor.Services.UpdateManagement.UpdateChecker: No updates available
2019-02-02 11:40:57.1069 #14152 [Info] SyncTrayzor.Bootstrapper: Shutting down: Shutdown
2019-02-02 11:40:57.1225 #14152 [Info] SyncTrayzor.Syncthing.SyncthingProcessRunner: Killing Syncthing process
2019-02-02 11:41:02.8221 #11160 [Info] SyncTrayzor.Bootstrapper: SyncTrazor version <IP_ADDRESS> (Amd64) started at C:\Program Files\SyncTrayzor\SyncTrayzor.exe (.NET version: Unknown (461308))
2019-02-02 11:41:02.8533 #11160 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: LogFilePath: C:\Users\Ram\AppData\Roaming\SyncTrayzor\logs
2019-02-02 11:41:02.8533 #11160 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: SyncthingBackupPath: C:\Program Files\SyncTrayzor\syncthing.exe
2019-02-02 11:41:02.8533 #11160 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: ConfigurationFilePath: C:\Users\Ram\AppData\Roaming\SyncTrayzor\config.xml
2019-02-02 11:41:02.8533 #11160 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: ConfigurationFileBackupPath: C:\Users\Ram\AppData\Roaming\SyncTrayzor\config-backups
2019-02-02 11:41:02.8533 #11160 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: CefCachePath: C:\Users\Ram\AppData\Local\SyncTrayzor\cef\cache
2019-02-02 11:41:02.8533 #11160 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: DefaultSyncthingPath: C:\Users\Ram\AppData\Roaming\SyncTrayzor\syncthing.exe
2019-02-02 11:41:02.8533 #11160 [Debug] SyncTrayzor.Services.Config.ApplicationPathsProvider: DefaultSyncthingHomePath:
2019-02-02 11:41:02.8689 #11160 [Debug] SyncTrayzor.Services.Ipc.IpcCommsClientFactory: Checking for other SyncTrayzor processes
2019-02-02 11:41:02.8689 #11160 [Debug] SyncTrayzor.Services.Ipc.IpcCommsClientFactory: Did not find any other processes, or they all responded with an error
2019-02-02 11:41:02.8846 #11160 [Info] SyncTrayzor.Services.Config.ConfigurationProvider: Loaded default configuration: <Configuration ShowTrayIconOnlyOnClose=False MinimizeToTray=False CloseToTray=True ShowDeviceConnectivityBalloons=True ShowDeviceOrFolderRejectedBalloons=True SyncthingAddress=localhost:8384 StartSyncthingAutomatically=True SyncthingCommandLineFlags=[] SyncthingEnvironmentalVariables=[] SyncthingDenyUpgrade=False SyncthingPriorityLevel=Normal Folders=[] NotifyOfNewVersions=True LatestNotifiedVersion= ObfuscateDeviceIDs=True UseComputerCulture=True SyncthingConsoleHeight=100 WindowPlacement= SyncthingWebBrowserZoomLevel=0 LastSeenInstallCount=0 SyncthingCustomPath= SyncthingCustomHomePath= ShowSynchronizedBalloonEvenIfNothingDownloaded=False DisableHardwareRendering=False EnableFailedTransferAlerts=True EnableConflictFileMonitoring=True ConflictResolverDeletesToRecycleBin=True PauseDevicesOnMeteredNetworks=True HaveDonated=False IconAnimationMode=DataTransferring OpenFolderCommand=explorer.exe "{0}" ShowFileInFolderCommand=explorer.exe /select, "{0}"LogLevel=Info>
2019-02-02 11:41:02.8846 #11160 [Info] SyncTrayzor.Services.Config.ConfigurationProvider: Configuration file C:\Users\Ram\AppData\Roaming\SyncTrayzor\config.xml doesn't exist, so creating
2019-02-02 11:41:02.9002 #11160 [Debug] SyncTrayzor.Services.Config.ConfigurationProvider: InstallCount changed from 0 to 1
2019-02-02 11:41:02.9002 #11160 [Warn] SyncTrayzor.Services.Config.ConfigurationProvider: Syncthing doesn't exist at C:\Users\Ram\AppData\Roaming\SyncTrayzor\syncthing.exe, so copying from C:\Program Files\SyncTrayzor\syncthing.exe
2019-02-02 11:41:02.9783 #11160 [Debug] SyncTrayzor.Services.AutostartProvider: Have read/write access to the registry
2019-02-02 11:41:02.9783 #11160 [Debug] SyncTrayzor.Services.AutostartProvider: Setting AutoStart to <AutostartConfiguration AutoStart=True StartMinimized=True>
2019-02-02 11:41:02.9783 #11160 [Debug] SyncTrayzor.Services.AutostartProvider: Autostart path: "C:\Program Files\SyncTrayzor\SyncTrayzor.exe" -minimized
2019-02-02 11:41:03.3533 #11160 [Debug] SyncTrayzor.Services.Conflicts.ConflictFileWatcher: Refreshing conflicted files. Found 0 from 0 options
2019-02-02 11:41:04.1557 #11160 [Info] SyncTrayzor.Syncthing.SyncthingProcessRunner: Starting syncthing: C:\Users\Ram\AppData\Roaming\SyncTrayzor\syncthing.exe
2019-02-02 11:41:04.3092 #11160 [Info] SyncTrayzor.Services.UpdateManagement.UpdateChecker: No updates available
2019-02-02 11:41:05.8430 #11160 [Info] SyncTrayzor.Syncthing.SyncthingProcessRunner: Syncthing exited. Restarting...
2019-02-02 11:41:05.8430 #11160 [Info] SyncTrayzor.Syncthing.SyncthingProcessRunner: Starting syncthing: C:\Users\Ram\AppData\Roaming\SyncTrayzor\syncthing.exe
2019-02-02 11:41:07.3540 #11160 [Info] SyncTrayzor.Syncthing.SyncthingProcessRunner: Syncthing exited. Restarting...
2019-02-02 11:41:07.3540 #11160 [Info] SyncTrayzor.Syncthing.SyncthingProcessRunner: Starting syncthing: C:\Users\Ram\AppData\Roaming\SyncTrayzor\syncthing.exe
2019-02-02 11:41:08.8571 #11160 [Info] SyncTrayzor.Syncthing.SyncthingProcessRunner: Syncthing exited. Restarting...
2019-02-02 11:41:08.8571 #11160 [Info] SyncTrayzor.Syncthing.SyncthingProcessRunner: Starting syncthing: C:\Users\Ram\AppData\Roaming\SyncTrayzor\syncthing.exe
2019-02-02 11:41:10.3659 #11160 [Warn] SyncTrayzor.Syncthing.SyncthingProcessRunner: 4 restarts in less than 00:01:00: not restarting again
2019-02-02 11:46:03.3863 #11160 [Info] SyncTrayzor.Services.MemoryUsageLogger: Working Set: 130MiB. Private Memory Size: 127MiB. GC Total Memory: 7MiB
2019-02-02 11:46:03.5895 #11160 [Info] SyncTrayzor.Services.UpdateManagement.UpdateChecker: No updates available

Please let me know how do I resolve this. I started getting this error as I was first using the portable version and then I closed the application and installed the .exe version. And when I ran synctrayzor, I started getting this error as shown in the screenshot below:

I was able to finally resolve the issue by deleting all folders related to SyncTrayzor and Syncthing in AppData. Can't recall the exact folder location.

@rampatra I'm afraid I need the entire log at the point that the problem started happening - I need to see why it started happening.

@jrongit Were you running Syncthing separately, before you started using SyncTrayzor?

This is happening to me and yes I did run syncthing outside synctrazor first. What happened is I'm just starting to use syncthing so I with the official Syncthing Getting Started Guide and set it up and tested it a bit. Then I looked into synctrayzor later when the manual mentioned it. I installed both syncthing and synctrazor using chocolatey.

@canton7 - Yes, I installed Syncthing separately first. I didn't realise that SyncTrayzor included Syncthing and thought I'd experiment with the core application first. From other replies it sounds like this might be the issue. Happy to help/reply where I can, just takes me a while to get back here to check the thread.

I think... Although syncthing.exe appears on program files, the exe that matters is in your hidden roaming appdata profile folder. Make sure it's current.
Get Outlook for Androidhttps://aka.ms/ghei36

From: jrongit<EMAIL_ADDRESS>Sent: Sunday, February 10, 2019 6:39:13 PM
To: canton7/SyncTrayzor
Cc: drink7up; Comment
Subject: Re: [canton7/SyncTrayzor] If "FATAL:Cannot open database" (#493)
@canton7https://github.com/canton7 - Yes, I installed Syncthing separately first. I didn't realise that SyncTrayzor included Syncthing and thought I'd experiment with the core application first. From other replies it sounds like this might be the issue. Happy to help/reply where I can, just takes me a while to get back here to check the thread.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/canton7/SyncTrayzor/issues/493#issuecomment-462110863, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AsoujP0nFfMTSE0x7xQlL1OPCCbOOs1Vks5vL8yhgaJpZM4Z39c-.

@jrongit Were you running Syncthing separately, before you started using SyncTrayzor?
I started the portable version first. Then I installed and got an error.
I was able to finally resolve the issue by deleting all folders related to SyncTrayzor and Syncthing in AppData. Can't recall the exact folder location.
AppData\Local\Syncthing
AppData\Local\SyncTrayzor
AppData\Roaming\SyncTrayzor\

The only explanation is that you have run Syncthing 0.14.53 or higher outside of SyncTrayzor (or another SyncTrayzor instance which contains a newer Syncthing version, but shares the same database location). Another possibility is that something deleted your Roaming profile folder, but not your Local profile folder.
The copy of Syncthing which SyncTrayzor uses is in C:\Users\<You>\AppData\Roaming\SyncTrayzor (see the README). If you were replacing the copy that's in C:\Program Files, that won't have made any difference.
The workaround is to download Syncthing 1.0.0 and put it in C:\Users\<You>\AppData\Roaming\SyncTrayzor (for the installed version) or data (for the portable version).
If one of you could send me your log files from the time that the problem started happening (File -> Settings -> Logging -> Show SyncTrayzor log files, then send me the file corresponding to the point in time where the problem first started happening), that might help track down what exactly is going on with Syncthing.

this helped and explains the problem perfectly. maybe the log should point out that under the roaming folder is the "real" version of Synthing which SyncTraytor uses.
It was exactly like this with my set up. I tried Trayor awhile ago, used the CLI version besides that and the roaming one was "too old".
thank you, maybe this should get fixed / logged that you know what to do.

Thanks a lot to @canton7.
Your fix worked perfectly for me (I had the same issue as described initially by @ppcina ) : just copied the latest syncthing.exe version available (1.1.0) into folder C:\Users<You>\AppData\Roaming\SyncTrayzor

Chiming in to say that the fix (replacing the Syncthing binary in %appdata%\SyncTrayzor\) worked wonderfully.
I'd be more than happy to send logs describing when this first started to happen, but this started happening months ago and %appdata%\SyncTrayzor\logs\logs archive only has three days worth of logs. (I papered over the problem by pointing SyncTrayzor to a Syncthing binary in %userprofile%\bin.)

The latest version will ship with Syncthing 1.1.1, which should solve the issue... until Syncthing next changes its database schema format. Ideally we want ot explicitly tell Syncthing to upgrade itself if we need to copy it to AppData (but only if updates to Syncthing are allowed)

syncthing won't work with the version built-in but I copy paste in %appdata%\SyncTray syncthing version 1.5.0 and now it works

I had the same error message (I think), although I have never tested a newer version. I read the console messages and set a flag:
-allow-newer-config
in in the advanced setting of the Syncthing tab in Preferences. Now it is working.

The copy of Syncthing which SyncTrayzor uses is in C:\Users<You>\AppData\Roaming\SyncTrayzor (see the README). If you were replacing the copy that's in C:\Program Files, that won't have made any difference.
The workaround is to download latest version of your OS build _Syncthing and put it in C:\Users<You>\AppData\Roaming\SyncTrayzor (for the installed version) , just overwrite the file it will work !

Solution from Reddit:
"I've finally got it to run. Go into synctrayzor, let it error out as usual. Then go to the settings button bottom right, syncthing tab, open advanced and put this code "-allow-newer-config" in the first line (without "")."

So, I downloaded, installed and ran synching windows version and quickly noticed that I would have to keep this black terminal window open for service to keep on running. Read a little bit more on download page saw that SyncTrayzor was the right way to go so downloaded, installed and ran getting this "FATAL:Cannot open database" error even after confirming that no other instance of Syncthing or SyncTrayzor was running.
Solution:  delete both "Syncthing" and "SyncTrayzor" folders from below two locations
C:\User<username>\AppData\Local
C:\User<username>\AppData\Roaming
Hope this helps. Regards!

thx to @jjarandab , u fixed my issue perfectly!!!


Add new Readme with installation instructions.

Wrong repo I guess

Oooops mb sorry. Yes, wrong repo.