Warn if with_them is used before where is defined

This PR emit a warning if with_them is used before where is defined.
See https://github.com/rspec-parameterized/rspec-parameterized-core/issues/8
Impact
On this repository we are now seeing 3 warnings:
/home/peter/devel/rspec-parameterized-core/spec/rspec/parameterized/core_spec.rb:265: `where` not defined.
/home/peter/devel/rspec-parameterized-core/spec/rspec/parameterized/core_spec.rb:271: `where` not defined.
/home/peter/devel/rspec-parameterized-core/spec/rspec/parameterized/core_spec.rb:286: `where` not defined.

They relate to https://github.com/rspec-parameterized/rspec-parameterized-core/blob/94ed77992543f76d849805fd22f3fd38798b8f9d/spec/rspec/parameterized/core_spec.rb#L264-L304 which makes sense
However, emitting a warning in those cases would also cover the following case:
describe 'empty' do
  where(:x) { [1] }

  describe 'another scope' do # !!! NOTE THIS EXTRA CONTEXT!!
    with_them do
      it 'works' do

      end
    end
  end
end

# 0 examples, 0 failures

Full output
$ bundle exec rspec -f p --dry-run
warning: parser/current is loading parser/ruby30, which recognizes 3.0.6-compliant syntax, but you are running 3.0.5.
Please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
/home/peter/devel/rspec-parameterized-core/spec/rspec/parameterized/core_spec.rb:265: `where` not defined.
/home/peter/devel/rspec-parameterized-core/spec/rspec/parameterized/core_spec.rb:271: `where` not defined.
/home/peter/devel/rspec-parameterized-core/spec/rspec/parameterized/core_spec.rb:286: `where` not defined.
Run options: include {:current=>true}

All examples were filtered out; ignoring {:current=>true}
......***..................................................................................

Pending: (Failures listed here are expected and do not affect your suite's status)

  1) RSpec::Parameterized where and with_them a: 1, b: 2, answer: 3 should do additions
     # PENDING
     # ./spec/rspec/parameterized/core_spec.rb:34

  2) RSpec::Parameterized where and with_them a: 5, b: 8, answer: 13 should do additions
     # PENDING
     # ./spec/rspec/parameterized/core_spec.rb:34

  3) RSpec::Parameterized where and with_them a: 0, b: 0, answer: 0 should do additions
     # PENDING
     # ./spec/rspec/parameterized/core_spec.rb:34

Deprecation Warnings:

RSpec::Core::Configuration#treat_symbols_as_metadata_keys_with_true_values= is deprecated, it is now set to true as default and setting it to false has no effect.

If you need more of the backtrace for any of these deprecations to
identify where to make the necessary changes, you can configure
`config.raise_errors_for_deprecations!`, and it will turn the
deprecation warnings into errors, giving you the full backtrace.

1 deprecation warning total

Finished in 0.00623 seconds (files took 0.15319 seconds to load)
91 examples, 0 failures, 3 pending

Marking as "Draft" for now as this idea and the implementation has to be tested at GitLab.

In https://gitlab.com/gitlab-org/gitlab/-/issues/430870#note_1637281194 I found some cases where where is defined in a parent example group and with_them is used in a nested context assuming that where is shared:
describe 'foo' do
  where(foo: [1, 2, 3])

  describe 'bar' do
    with_them do
      it('works') {}
    end
  end

  describe 'bar2' do
    with_them do
      it('works') {}
    end
  end
end

This :point_up: gives 0 examples, 0 failures.

:wave: @sue445 Please excuse the noise but I've made this PR ready for review again :sweat_smile:
Do you mind checking? :pray:

Therefore, I think we should only treat it as a warning when where is not defined in same scope as with_them.

I agree :+1:
I was wondering how we can achieve this :shrug: Currently, where and with_them are not lazy so I am not sure how to let with_them know if there's a where following it. For example:
describe 'foo' do
  with_them do # How do we know that `where` will follow in the same scope?
    ...
  end

  where(...)
end


simple test for plugin.xml

@patflynn @loosebazooka

LGTM. I guess we should include this in the login plugin as well.


Fix/input onchange

修复 onchange 在 input 上面无法触发的问题。

Codecov Report
Base: 82.45% // Head: 82.34% // Decreases project coverage by -0.11% :warning:

Coverage data is based on head (e5eb8bc) compared to base (57d4356).
Patch coverage: 47.82% of modified lines in pull request are covered.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-next     #507      +/-   ##
================================================
- Coverage         82.45%   82.34%   -0.12%
================================================
  Files                92       92
  Lines              7816     7851      +35
  Branches            978      984       +6
================================================
+ Hits               6445     6465      +20
- Misses             1364     1378      +14
- Partials              7        8       +1

Impacted Files
Coverage Δ

packages/rax-compat/src/create-element.ts
84.69% <47.82%> (-5.20%)
:arrow_down:

packages/ice/src/config.ts
75.00% <0.00%> (-0.59%)
:arrow_down:

packages/ice/src/service/config.ts
86.66% <0.00%> (ø)

packages/ice/src/tasks/web/index.ts
100.00% <0.00%> (ø)

packages/ice/src/createService.ts
95.12% <0.00%> (+0.27%)
:arrow_up:

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

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


dyld: Library not loaded: /usr/local/opt/z3/lib/libz3.4.12.dylib

Just followed the installation steps for macOS:
❯ cd /tmp

/private/tmp
❯ curl -OL https://github.com/roc-lang/roc/releases/download/nightly/roc_nightly-macos_x86_64-latest.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100 33.3M  100 33.3M    0     0  1269k      0  0:00:26  0:00:26 --:--:-- 1345k

/private/tmp took 26s
❯ tar xf roc_nightly-macos_x86_64-latest.tar.gz

/private/tmp
❯ cd roc_nightly-macos_x86_64-2023-11-21-2afd9ca/

/private/tmp/roc_nightly-macos_x86_64-2023-11-21-2afd9ca
❯ ./roc
dyld: Library not loaded: /usr/local/opt/z3/lib/libz3.4.12.dylib
  Referenced from: /private/tmp/roc_nightly-macos_x86_64-2023-11-21-2afd9ca/./roc
  Reason: image not found

❯ system_profiler SPHardwareDataType SPSoftwareDataType -detailLevel mini
Hardware:

    Hardware Overview:

      Model Name: MacBook Pro
      Model Identifier: MacBookPro9,2
      Processor Name: Dual-Core Intel Core i7
      Processor Speed: 2,9 GHz
      Number of Processors: 1
      Total Number of Cores: 2
      L2 Cache (per Core): 256 KB
      L3 Cache: 4 MB
      Hyper-Threading Technology: Enabled
      Memory: 16 GB
      Boot ROM Version: 4<IP_ADDRESS>.0
      SMC Version (system): 2.2f44
      Sudden Motion Sensor:
          State: Enabled

Software:

    System Software Overview:

      System Version: macOS 10.15.7 (19H2026)
      Kernel Version: Darwin 19.6.0
      Time since boot: 13 days 21:03

@Anton-4 I tried, but I am using MacPorts which installs into /opt/local/lib. Thus, I created a symlink from the requested location.
The z3 lib is found this way, but now another lib is requested – did I miss the prerequisites for roc?
❯ sudo port install z3
Password:
--->  Fetching archive for z3
--->  Attempting to fetch z3-4.12.2_1+lto+polly+polly_late+polly_vector+threads.darwin_19.x86_64.tbz2 from https://packages.macports.org/z3
--->  Attempting to fetch z3-4.12.2_1+lto+polly+polly_late+polly_vector+threads.darwin_19.x86_64.tbz2.rmd160 from https://packages.macports.org/z3
--->  Installing z3 @4.12.2_1+lto+polly+polly_late+polly_vector+threads
--->  Activating z3 @4.12.2_1+lto+polly+polly_late+polly_vector+threads
--->  Cleaning z3
--->  Scanning binaries for linking errors
--->  No broken files found.
--->  No broken ports found.

/private/tmp/roc_nightly-macos_x86_64-2023-11-21-2afd9ca ⚡
❯ mkdir /usr/local/opt/z3/lib

/private/tmp/roc_nightly-macos_x86_64-2023-11-21-2afd9ca ⚡
❯ ln -s /opt/local/lib/libz3.4.12.dylib /usr/local/opt/z3/lib/

/private/tmp/roc_nightly-macos_x86_64-2023-11-21-2afd9ca ⚡
❯ ./roc
dyld: Library not loaded: /usr/local/opt/zstd/lib/libzstd.1.dylib
  Referenced from: /private/tmp/roc_nightly-macos_x86_64-2023-11-21-2afd9ca/./roc
  Reason: image not found

Symlinking the zstd lib, too, finally lets me run roc:
❯ mkdir /usr/local/opt/zstd/lib

/private/tmp/roc_nightly-macos_x86_64-2023-11-21-2afd9ca
❯ ln -s /opt/local/lib/libzstd.1.dylib /usr/local/opt/zstd/lib/

/private/tmp/roc_nightly-macos_x86_64-2023-11-21-2afd9ca
❯ ./roc version
roc nightly pre-release, built from commit 2afd9ca on Tue Nov 21 09:09:05 UTC 2023

Would it be possible to make roc work with MacPorts out of the box, too?

The same happened to me as well, I don't use MacPorts, I solved it thanks to your help by doing:
brew install z3
brew install zstd

They end up installed in /usr/local/lib so Roc is able to find them once they are installed, but I think these should be listed as prerrequisites in the installation tutorial for macos_x86_64.

Thanks @marc-llop,
I've updated the install instructions.

For MacPorts support, it seems like we can modify the rpath when producing the nightly so it can use the MacPorts path as an additional place to search. I'll make a new issue for that and close this one.

I'll make a new issue for that and close this one.

https://github.com/roc-lang/roc/issues/6081


(PUP-3706) console format should pretty print nested structures

This makes the console format actually pretty print nested structures.
It already said it would, but didn't in fact.

Triage Notes

@dalen, do you plan to look at this soon?
If @MikaelSmith gets time, look at fixing and merging ourselves.

I've updated it now with some parts of PUP-3355, which I believe is necessary to use JSON instead of PSON.

Thanks for the update. There's a merge conflict requiring a rebase that's caused by the update to RSpec 3. This can be fixed by x.should == y to expect(x).to eq(y).
Also, it'd be nice to have a spec covering the default where --render-as json isn't specified, but it's not immediately clear to me how to do that. Why is the 2nd commit labelled with PUP-3355?

@MikaelSmith That is because that commit doesn't really have anything to do with this change. But people requested that I switch from PSON to JSON, and that commit is for enabling JSON support in Puppet overall, which would be a prerequisite for switching this change from PSON to JSON.
I'll update the specs to match rspec 3 syntax.

umm, right it should be PUP-3540 not PUP-3355, will update

@dalen looks like this just needs the merge conflict cleaned up and the commit message fixed and we can review this soon.

I'll try to look at that soon.  Very sorry to hear that. Best wishes for the recovery (and sympathies for one-handed typing - I've been there before).

PR Triage notes: Scott will take a look at this and if it's a simple merge conflict/spec test fix, finish the work.

I've rebased this and updated the spec tests in PR-3658. That PR now supercedes this one.


No valid renderer was found for the x file extension

Main astro file
---
import Chatbox from "../components/tsx/Chatbox";
---

    <Chatbox client:visible />

Chatbox.tsx
import React from "react";

class Chatbox extends React.Component {
  constructor(props: any) {
    super(props);
  }

  override render() {
    return

;
  }
}

export default Chatbox;

Error:
 error   Unable to render `Chatbox`.

  No valid renderer was found for the `.D:/Github/Repositories/BucksBoost/client/src/components/tsx/Chatbox` file extension.
  Hint:
    Did you mean to enable the `@astrojs/react`, `@astrojs/preact`, `@astrojs/solid`, `@astrojs/vue` or `@astrojs/svelte` integration?

    See https://docs.astro.build/en/core-concepts/framework-components/ for more information on how to install and configure integrations.
  File:
    D:\Github\Repositories\BucksBoost\client\src\pages\index.astro
  Stacktrace:
NoMatchingRenderer: Unable to render `Chatbox`.

No valid renderer was found for the `.D:/Github/Repositories/BucksBoost/client/src/components/tsx/Chatbox` file extension.
    at renderFrameworkComponent (file:///D:/Github/Repositories/BucksBoost/client/node_modules/astro/dist/runtime/server/render/component.js:131:15)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    ```

Hi there! This is a place to file issues with documentation itself (problems with our website, corrections to documentation) and not a place where we can give support.
I can tell you that the error message is asking you whether you enabled the React integration, so that is a place to start looking. Astro does not include renderers for React, Vue etc. by default, but you can add integrations to be able to use your React components.
So I'm closing this issue, and in future if you need help, please use our Discord at https://astro.build/chat and post in our support forums! We have lots of people there who are happy to help and ready to answer your questions!

I fixed it by installing the Astro-react module and configuring my config file.
npm install --save-dev @astrojs/react react react-dom

import { defineConfig } from 'astro/config';
import react from '@astrojs/react';

export default defineConfig({
  integrations: [react()],
});

Excellent! Glad you're good to go again!  :rocket:


Create a Single initialization/bootstrap PHP file and Configuration Container

Currently, there's lots of configuration info and setup code scattered through different files. There should really only be a single place where configuration and application setup takes place, and it would be a good idea to move your config and setup there. Ideally, your bootstrap file should:

Setup autoloading (currently in includes/functions.php)
autoload any global functions
load all configuration from a separate file
Instantiate any global objects necessary (an application kernel or configuration container, for example)

Then all of your public PHP files only need to include the boostrap file. Instead of including various config files in your class constructors, you can use the configuration container to pull out the specific configuration they need. For example, your DbConn class might look like this:
$database = DbConn::createFromConfig($configContainer);
This also forces you to think more about application architecture because you can't just access everything from everywhere.

if you feel thats a way to go then fork and do a pull request with the amended files, i for one would be keen to see it


Status: {{connectionState}}

Hi
thx for your moped-website.
I have entered the setting-site to see, how i can configure the mopidy from there. There was a button to check ...something and it appear another setting-screen with "Status: {{connectionState}}"  on the upper right corner. And now my mopidy does not react on any click (back, or home).
Has someone an idea what happens here and how i can fixe this
thx
Ole

Check the javascript console for errors (F12 or CMD-ALT-I in Chrome). When you see {{}} texts, it's usually a sign that some javascript had an error during execution.

I got these issues related to this status: Mixed Content: The page at 'https://music.gitnote.eu/moped/' was loaded over HTTPS, but requested an insecure stylesheet 'http://fonts.googleapis.com/css?family=Open+Sans'. This request has been blocked; the content must be served over HTTPS.
moped-0.4.2.js:9 Mixed Content: The page at 'https://music.gitnote.eu/moped/' was loaded over HTTPS, but attempted to connect to the insecure WebSocket endpoint 'ws://music.gitnote.eu/mopidy/ws'. This request has been blocked; this endpoint must be available over WSS.c.connect @ moped-0.4.2.js:9
moped-0.4.2.js:14 Error: Failed to construct 'WebSocket': An insecure WebSocket connection may not be initiated from a page loaded over HTTPS.
at Error (native)
at c.connect (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:9:20759)
at new c (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:9:18901)
at Object.start (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:18:2379)
at new  (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:17:19794)
at d (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:4137)
at Object.e [as instantiate] (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:4250)
at https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:21322
at https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:13852
at f (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:12:20548)(anonymous function) @ moped-0.4.2.js:14

I got these issues related to this status:
Mixed Content: The page at 'https://music.gitnote.eu/moped/' was loaded over HTTPS, but requested an insecure stylesheet 'http://fonts.googleapis.com/css?family=Open+Sans'. This request has been blocked; the content must be served over HTTPS.
moped-0.4.2.js:9 Mixed Content: The page at 'https://music.gitnote.eu/moped/' was loaded over HTTPS, but attempted to connect to the insecure WebSocket endpoint 'ws://music.gitnote.eu/mopidy/ws'. This request has been blocked; this endpoint must be available over WSS.c.connect @ moped-0.4.2.js:9
moped-0.4.2.js:14 Error: Failed to construct 'WebSocket': An insecure WebSocket connection may not be initiated from a page loaded over HTTPS.
    at Error (native)
    at c.connect (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:9:20759)
    at new c (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:9:18901)
    at Object.start (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:18:2379)
    at new <anonymous> (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:17:19794)
    at d (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:4137)
    at Object.e [as instantiate] (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:4250)
    at https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:21322
    at https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:13852
    at f (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:12:20548)(anonymous function) @ moped-0.4.2.js:14

It seems related to this PR: https://github.com/andrenarchy/mopidy.js/commit/1e5ebe6059c44d02dae2fa88e7a6b577d5c2b77e

Thanks for the heads up.

@martijnboland No I believe that @olemde and my issue are the same.
Mixed Content: The page at 'https://music.gitnote.eu/moped/' was loaded over HTTPS, but requested an insecure stylesheet 'http://fonts.googleapis.com/css?family=Open+Sans'. This request has been blocked; the content must be served over HTTPS.
moped-0.4.2.js:9 Mixed Content: The page at 'https://music.gitnote.eu/moped/' was loaded over HTTPS, but attempted to connect to the insecure WebSocket endpoint 'ws://music.gitnote.eu/mopidy/ws'. This request has been blocked; this endpoint must be available over WSS.21.c.connect @ moped-0.4.2.js:9
moped-0.4.2.js:14 Error: Failed to construct 'WebSocket': An insecure WebSocket connection may not be initiated from a page loaded over HTTPS.
    at Error (native)
    at c.21.c.connect (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:9:20759)
    at new c (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:9:18901)
    at Object.start (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:18:2379)
    at new <anonymous> (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:17:19794)
    at d (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:4137)
    at Object.e [as instantiate] (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:4250)
    at $get (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:21322)
    at https://music.gitnote.eu/moped/assets/moped-0.4.2.js:13:13852
    at f (https://music.gitnote.eu/moped/assets/moped-0.4.2.js:12:20548)
'''

mopidy-moped does not seem to be ready for HTTPS & WebSocket Secure: a PR should be done for moped-0.4.2.js

So, is this fixed with the latest version?

[ solved ] clear everything on your web browser , chrome , firefox , iceweasel..
close web browser then start up your mopidy on moped web client.


💡 Support https://webpack.js.org/plugins/provide-plugin/

Suggest an idea for this project
I wonder if we could support ProvidePlugin somehow. I see https://github.com/webpro-nl/knip/blob/1cf68c8987bca64de78d89fac5e7bd6f088ea698/packages/knip/fixtures/plugins/webpack/webpack.common.js#L89 references it but it's not listed as an issue. I found that in our project when buffer is in package.json and
    new ProvidePlugin({
      process: 'process/browser',
      Buffer: ['buffer', 'Buffer'],
    }),

is defined in a webpack config, buffer gets listed as a unused dependency

In the webpack plugin, is there any way to find this in localConfig or resolvedConfig? That's the resolved exported value of the webpack config file (e.g. your webpack.config.js).

Right, good call. No, I don't think there is a way through the config value.
When I log resolvedConfig.plugins out in src/plugins/webpack/index.ts and run the webpack test I get
[ MiniCssExtractPlugin {}, ProvidePlugin {}, CopyWebpackPlugin {} ]

not sure what I'm looking at. At first I thought it was instances of these plugins, so basically an instance of https://github.com/webpack/webpack/blob/96c543c0a976e7d0a2dadd8ff0d6525f99bc3417/lib/ProvidePlugin.js#L25
but it doesn't seem to be since I can't access this.definitions and I thought to myself: "that makes sense, Knip is not actually executing the webpack config, that might have all kinds of side-effects".
So I guess the plugins are just opaque named empty objects or something?
Hoping my hunch is right.
This seems hard to solve with the current plugin API interface where you get a resolved config (which is a really nice api). The plugin would need access to the AST or raw string contents of the file to parse out this ProvidePlugin dependency

Knip is not actually executing the webpack config

So I guess the plugins are just opaque named empty objects/classes or something?

Knip loads and executes files like webpack.config.ts, but nothing else (i.e. it does not call functions or instantiate classes). Whatever that config file exports is handed over as the first argument in resolveConfig(localConfig).

The plugin would need access to the AST

Exactly, this has been on my mind for a while (I should add this to the list: https://knip.dev/reference/faq#why-doesnt-knip-have). But so far, the need has surfaced only rarely and it would mean opening up a new API surface area I'm currently not too fond of.

Yeah I completely agree. That type of API is bloated. Maybe there could be some sane way to solve this (or maybe not), but rushing an API for it is not a good idea.


Improve Handling Windows Decode Error

Changes

Replace backslashes if we hit an error decoding the temporary logfile

I still need to see if I can write a Windows only test for this.

The test probably doesn't need to be Windows-specific. If the tempfile is corrupt/non-utf-8 for any reason, on any platform, the same error will be triggered.
I'm quite confused about how the tempfile ends up non-utf-8 in the first place.
get_tempfile_logger seems fine....

David Glick prefers that we get to the root cause: https://github.com/SFDO-Tooling/CumulusCI/issues/2250#issuecomment-744612290

I'd prefer to figure out why the tempfile isn't getting written as utf-8, but it's not obvious to me either. I'd suggest a time-boxed debugging session in Windows, and then go with this workaround if it's still unclear.

@Br4nd0R If you can reproduce the problem at all, one way of figuring out where exactly the bad data is inserted might be to insert some kind of mock file object that inspects every write. But that's assuming that you have a test case.

I wonder if this is somehow related to the capturing of output from sfdx, and its output being cp1252 (the Windows default) rather than utf-8. But if that's the case I would expect it to blow up as soon as the sfdx output is processed, not at the end of the command while processing the tempfile log.


Files created from launcher are not mapped to the remote filesystem

If I try to create a new notebook by clicking the launcher button, the new notebook does not appear in the remote filesystem directory and is not listed in the remote filesystem listing in the file browser.
If I right-click in the file browser and create a new file there, it does appear in the file listing in the sidebar and is saved to the remote filesystem.
I also have the jupyterlab-git and jupyterlab-unfold extensions installed which interact with the filesytem and file listing display.
The ability for the UI to display notebooks that are saved to different file systems makes me wonder if it might be useful to identify the file system for example by colouring the background of the filename tab, or changing it's favicon?

This should be fixed by https://github.com/jupyterlab/jupyterlab/issues/15290


Flag Commands Cause Panic When Given $nil Arguments.

What happened, and what did you expect to happen?
The various functions in the flag module cause a Go panic via nil dereference if passed a $nil value as any of their arguments. I would expect these to throw an exception, rather than panicking.
Examples:
flag:call $nil $nil
flag:parse $nil []
flag:parse-getopt $args $nil

Example Stacktrace:
log.txt
Output of "elvish -version"
0.22.0-dev.0.20241021163419-8527641024a2
Code of Conduct

[x] I agree to follow Elvish's Code of Conduct.

This issue was reported again in #1869 and fixed.

Thanks for reporting! Left too many issues untriaged and still slowly going thru things...


Hello, I can't find your paper online. Can you send it to me?

Hello, I can't find your paper online. Can you send it to me?

Compact Global Descriptor for Neural Networks

FYI.
https://arxiv.org/abs/1907.09665v3

供参考。 https://arxiv.org/abs/1907.09665v3

Thank you for your help


Adding secrets as a volume.

A PR to use the auth files will show up soon.
Also removed kube auth version flag.

LGTM


add interaction success/failure events

About the PR
raises an event when you succeed or fail to pet something
Why / Balance
needed downstream

raises an event when you succeed or fail to pet something

i'm gonna take a wild guess and say this pr is rmc14 related :trollface:

you forgot to import the namespace

i'm gonna take a wild guess and say this pr is rmc14 related :trollface:

for felenid hunters actually

Is this still needed downstream for another fork or useful in general?
It seems like the linked PR was declined.

Is this still needed downstream for another fork or useful in general? It seems like the linked PR was declined.

Yes

Well then :)


Reimplement immutable vector fold without builder

Builder provides more features than we need for which we have to pay with performance. In this PR I'm replacing it altogether with a simple and efficient strategy.
The strategy is to build it in two passes:

Accumulate a reverse list and its length.
Allocate a vector of that length and traverse it in reverse order populating it with elements of the list.

Performance of this function should now be comparable to the list fold.

@nikita-volkov: Thank you! 🙂


Code snippets must be clearer on the version of the SDK

Microsoft Graph API reference docs and Graph Explorer provide code snippets given an HTTP call/scenario.
However, just by looking at the code snippet, it's impossible to know which SDK version it is compatible with.
We need to be clear which SDK version is supported in both platforms, Graph API reference docs and Graph Explorer.
Any suggestions on what it should look like? @RabebOthmani @lramosvea

@maisarissi, @jasonjoh and I had a discussion prior to our offsite about this. One of the things we discussed was having a policy we can publish about versioning, which I think currently does not exist. We need to discuss this also with Seb and Vivian.

@maisarissi @lramosvea do we still need to close on anything here?

We should have a new sentence highlighting the version used to generate the snippet:

@isvargasmsft @sebastienlevert @CarolKigoonya can you take a look at the suggestion and add any other thoughts?

I agree with adding a small flag with the version just for clarification. That could also promote upgrading to newer versions of the SDK.

We do need to add the SDK version used to generate the snippets.

This should be part of the snippet so it would also work with Graph Explorer. I agree this would deliver clear expectations for developers.

For the docs, we could probably do something like this. If we place a note inside the snippet block we run into the issue where using the copy functionality results in the user copying invalid code that may not compile.
Update user - Microsoft Graph beta | Microsoft Learn

The alternative is to use the option below with the text appearing as a code comment. This will also easily work with the Graph Explorer.
Update user - Microsoft Graph beta | Microsoft Learn

I like this option as it easily spans across all properties where we show snippets!

If we choose to go with the first option, which looks way better in our docs than just a comment line, would Graph Explorer be able to follow a similar pattern?

Also, the fact that the version is copy / pasted is fairly interesting as it bring this context over to the developers IDE.

I agree with those points, however I feel that the code comment is "less visible", in a sense that, it's easier to ignore the information as opposed to the Note option that creates a block to draw the readers' attention to it.
What if we add the comment in the code snippet but in the docs we keep the Note as well?

Ok! So I'll summarize our agreement:

Have a include file with "Code snippets are only available for the latest version. Current version is 5.x"
Use the include file to generate the Note section in the docs
Also have the comment in the code showing the version so we can have the same information in Graph Explorer (this will duplicate the information for people in docs, but I don't think this is a problem)

The final version would be:

@sebastienlevert @isvargasmsft @andrueastman

I like having the note although not sure if this will look good on docs where we have many notes already and this might be a bit overwhelming for users. Let me share this with Steve.

Hey @lramosvea have you checked with Steve? We believe this will bring clarity to users.

Can't tag Steve here but did share this with him. Let me follow up and I'll get back to you.

@lramosvea awaiting word of decision.

@ddyett @maisarissi after discussing this with the rest of the docs team, our recommendation is to just keep the comment in the code. We don't think the note needs to be added,

@ddyett to move to global backlog. .NET complete but needs tracking for other languages.

Closing this for now given the available spec. This has been broken down into individual language issues below
CLI - https://github.com/microsoftgraph/microsoft-graph-devx-api/issues/1935
Go - https://github.com/microsoftgraph/microsoft-graph-devx-api/issues/1936
Python - https://github.com/microsoftgraph/microsoft-graph-devx-api/issues/1937
PHP - https://github.com/microsoftgraph/microsoft-graph-devx-api/issues/1938
Typescript - https://github.com/microsoftgraph/microsoft-graph-devx-api/issues/1939
Powershell - https://github.com/microsoftgraph/microsoft-graph-devx-api/issues/1940


replaced old url with new one on swiftbrowser …

…that includes tco1279 and tco319 files

While running the tests I noticed that it takes quite some time now to download the data. However, since you only have to download the data once, I think it should be fine.

closes #94


The plugin breaks window-snapping for Hyper on Windows 10

Plugin version tested: 1.7.6
Hyper version tested: 2.0.0
Windows version: Windows 10 Pro 1709 (Fall Creators Update)
Repro steps:
1- Fresh Hyper install, with no plugin
2- Open Hyper and try to snap the window to screen borders, either by dragging the window with the mouse, or by using WIN+right or WIN+left --> It works
3- install hyper-statusline via hyper i hyper-statusline
2- restart Hyper
3- try to snap the window to screen borders again --> It fails

Just realized I accused the wrong plugin. Sorry for the noise...


Splunk Forwarder installation fails

Operating System Version: OS Windows 10 x64 1809
Provider (VirtualBox/VMWare): VirtualBox
Vagrant Version: 2.2.5
Packer Version: -
Are you using stock boxes (downloaded) or were they built from scratch using Packer? Donwloaded.
Is the issue reproducible or intermittent? Yes

Description of the issue:
install-splunkuf.ps1 fails on wef host.
Errors starting like that in vagrant logs for wef:
==> wef: Running provisioner: shell...
    wef: Running: scripts/install-splunkuf.ps1 as c:\tmp\vagrant-shell.ps1
    wef: Downloading Splunk Universal Forwarder
    wef: [11:46] Installing & Starting Splunk
    wef: powershell.exe : Get-Service : Cannot find any service with service name 'splunkforwarder'.
    wef:     + CategoryInfo          : NotSpecified: (Get-Service : C...lunkforwarder'.:String) [], RemoteException

This is due to .msi file link doesn't provides a file anymore:
root@me:curl -s "https://www.splunk.com/bin/splunk/DownloadActivityServlet?architecture=x86_64&platform=windows&version=7.1.0&product=universalforwarder&filename=splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi&wget=true" | html2text | grep Page
**** 404 Error: Page not found. ****

It may be worth to attach necessary file to repository is someone still holds it or to allow it to download not from splunk.com domain (e.g. along with Box images).
For me I found only 6.5.1 version here:
https://download.splunk.com/products/splunk/releases/6.5.1/universalforwarder/windows/splunkforwarder-6.5.1-f74036626f0c-x64-release.msi
but this version's installation doesn't accept msi options given in script after file name...

Thanks for the report! I'll look into this this weekend and get a fix out ASAP if I can reproduce

Hi @dtrizna, I'm unable to reproduce this problem. Maybe it was a temporary issue with the Splunk website?
clong:~$ wget "https://www.splunk.com/bin/splunk/DownloadActivityServlet?architecture=x86_64&platform=windows&version=7.1.0&product=universalforwarder&filename=splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi&wget=true" -O splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi
--2019-07-26 23:27:44--  https://www.splunk.com/bin/splunk/DownloadActivityServlet?architecture=x86_64&platform=windows&version=7.1.0&product=universalforwarder&filename=splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi&wget=true
Resolving www.splunk.com (www.splunk.com)... <IP_ADDRESS>, <IP_ADDRESS>
Connecting to www.splunk.com (www.splunk.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Moved Temporarily
Location: https://download.splunk.com/products/universalforwarder/releases/7.1.0/windows/splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi [following]
--2019-07-26 23:27:46--  https://download.splunk.com/products/universalforwarder/releases/7.1.0/windows/splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi
Resolving download.splunk.com (download.splunk.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to download.splunk.com (download.splunk.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 58298368 (56M) [application/x-msi]
Saving to: ‘splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi’

splunkforwarder-7. 100%[================>]  55.60M  14.9MB/s    in 3.9s

2019-07-26 23:27:50 (14.2 MB/s) - ‘splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi’ saved [58298368/58298368]

clong:~$ file splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi
splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi: Composite Document File V2 Document, Little Endian, Os: Windows, Version 6.2, MSI Installer, Code page: 1252, Title: Installation Database, Subject: Splunk UniversalForwarder Wix 1.0 Installer, Author: Splunk, Inc., Keywords: Installer, Comments: UniversalForwarder is a registered trademark of Splunk Inc., Template: x64;1033, Revision Number: {6ED8FACF-CB03-4D12-B141-C6A35E365553}, Create Time/Date: Tue Apr 24 01:36:00 2018, Last Saved Time/Date: Tue Apr 24 01:36:00 2018, Number of Pages: 200, Number of Words: 2, Name of Creating Application: Windows Installer XML Toolset (<IP_ADDRESS>3), Security: 2

Am acknowledging that now download-link works..
root@me:~# curl -I "https://www.splunk.com/bin/splunk/DownloadActivityServlet?architecture=x86_64&platform=windows&version=7.1.0&product=universalforwarder&file
name=splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi&wget=true"
HTTP/2 302
server: Apache
x-webkit-csp: frame-ancestors *.splunk.com *.touchcast.com, frame-ancestors *.splunk.com *.touchcast.com
content-security-policy: frame-ancestors *.splunk.com *.touchcast.com, frame-ancestors *.splunk.com *.touchcast.com
x-content-security-policy: frame-ancestors *.splunk.com *.touchcast.com, frame-ancestors *.splunk.com *.touchcast.com
x-content-type-options: nosniff
x-frame-options: SAMEORIGIN
location: https://download.splunk.com/products/universalforwarder/releases/7.1.0/windows/splunkforwarder-7.1.0-2e75b3406c5b-x64-release.msi
serverid: prod-web02
expires: Mon, 29 Jul 2019 07:34:16 GMT
cache-control: max-age=0, no-cache, no-store
pragma: no-cache
date: Mon, 29 Jul 2019 07:34:16 GMT

Can close this issue.

Operating System Version: OS Windows 10 x64 (version 2004, build 19041.630)
Provider (VirtualBox/VMWare): VirtualBox
Vagrant Version: 2.2.5
Packer Version: -
Are you using stock boxes (downloaded) or were they built from scratch using Packer? Downloaded.
Is the issue reproducible or intermittent? I hope so
Hi, I have the same problem and it keeps occurring for me. I rebuilt and reloaded it multiple time but it still keeps giving me this error.

wef: [14:49] Downloading Splunk Universal Forwarder...
wef: [14:49] Installing & Starting Splunk
wef: powershell.exe : Get-Service : Cannot find any service with service name 'splunkforwarder'.
wef:     + CategoryInfo          : NotSpecified: (Get-Service : C...lunkforwarder'.:String) [], RemoteException
wef:     + FullyQualifiedErrorId : NativeCommandError
wef: At C:\tmp\vagrant-shell.ps1:14 char:6
wef: + If ((Get-Service -name splunkforwarder).Status -ne "Running")
wef: +      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
wef:     + CategoryInfo          : ObjectNotFound: (splunkforwarder:String) [Get-Service], ServiceComm
wef:    andException
wef:     + FullyQualifiedErrorId : NoServiceFoundForGivenName,Microsoft.PowerShell.Commands.GetService
wef:    Command
wef:
wef: Splunk forwarder service not running
wef: At C:\tmp\vagrant-shell.ps1:16 char:3
wef: +   throw "Splunk forwarder service not running"
wef: +   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
wef:     + CategoryInfo          : OperationStopped: (Splunk forwarder service not running:String) [],
wef:     RuntimeException
wef:     + FullyQualifiedErrorId : Splunk forwarder service not running
wef:


Add to beta please

Hi can you add me to beta please<EMAIL_ADDRESS>

https://github.com/kodi-connect/kodi-connect/blob/master/tutorial-alexa/readme.md


Add Tag property for all plotable objects.

Suggestion: (Add Tag property for all plotable objects.)
Add Tag property can reduce coding for a lot of condition.
ScottPlot.Plot myPlot = new();

var span = myPlot.Plot.Add.HorizontalSpan(5, 10);
span.Tag = "range1";

var span2 = myPlot.Plot.Add.HorizontalSpan(15, 20);
span2.Tag = "range2";

// MouseMove event can judge from tag
if (curSpan?.Tag.Equals("rang1")) {
   // do something......
}

@sdhongjun there is no good reason to add a Tag property for each plotable object. This will only consume more memory is 99% use cases for nothing.
You could easily replace those tags with a simple Map:
ScottPlot.Plot myPlot = new();
var map = new Dictionary<IPlotable, string>();

var span = myPlot.Plot.Add.HorizontalSpan(5, 10);
map.add(span, "range1");

var span2 = myPlot.Plot.Add.HorizontalSpan(15, 20);
map.add(span2,"range2");

// MouseMove event can judge from tag
if ( map.GetValueOrDefault(curSpan).Equals("rang1")) {
   // do something......
}

Almost all wpf control has Tag property, is the easiest way to attatch something to the control, and user can access the attached object directly from Tag propery no need more other coding.

Maybe, but that's not the point.
Again, there is no reason to add a Tag property which would be use is some minor cases and only consume memory most of the time.
There are many other ways to build a kind of lookup table to fit your needs without having to add a property.

Hi @sdhongjun, thanks for this question! It comes up frequently. See https://scottplot.net/faq/tag/ for a code example that uses a Dictionary to pair custom objects with plottables as a safer and more universal alternative to adding an object? tag.


Feature request: each torrent may have different store location

All torrents in a session share the same data storage configuration, it's more flexible if we can choose store directory for each torrent task.

No plans to implement this. Sorry.


remove Decode to read images in correct orientation

Thank you for your contribution to the MindOCR repo.
Before submitting this PR, please make sure:

[x] You have read the Contributing Guidelines on pull requests
[x] Your code builds clean without any errors or warnings
[x] You are using approved terminology
[ ] You have added unit tests

Motivation
MindSpore's Decode can't automatically rotate images based on the EXIF orientation information. Therefore, this PR replaces Decode with OpenCV for image reading in the detection task.
On the left is our original data pipeline output (uses OpenCV) and on the right is the new data pipeline (uses Decode):

I think we can create a issue in mindspore gitee repo, allow decode to be funtional well for EXIF format

Yes, this is a very necessary functionality. Will open an issue there.

I think we can create a issue in mindspore gitee repo, allow decode to be funtional well for EXIF format

Yes, this is a very necessary functionality. Will open an issue there.

opened， https://gitee.com/mindspore/mindspore/issues/I7SF23


Worker killed error

I got an error message that comes from this line of code:
https://github.com/vatlab/sos/blob/cf3717e280d01a3ce5f6650f275aa1f91832d97a/src/sos/workers.py#L689
I wonder if it is a critical or benign error -- I am thinking a warning might do because if the worker is killed and the task did not have an output then some other errors will be triggered anyways, for, say missing output files?
INFO: Waiting for the completion of 50 tasks before submitting 554 pending ones.
WARNING: Task M200_d439b16a19c9fd37 considered as aborted due to missing pulse file.
INFO: M200_14c70cf6212440ef submitted to midway2 with job id 63661901
INFO: M200_b4cc366abb70d754 submitted to midway2 with job id 63661904
INFO: M200_ed7d96c64609238e submitted to midway2 with job id 63661905
INFO: M200_46faec8ac706332e submitted to midway2 with job id 63661906
ERROR: One of the local workers has been killed.

This means one of the workers has been killed by external force, and currently SoS exits without trying to recover from the error. Do you have any idea why it happened? (e.g. process killed due to out of memory).

Yes i figured it is the manager thread that failed. It is a run from the headnode. I've no idea why it fails though because I've been running similar pipelines on headnode it never have complained before. Do you think it is still good idea to raise an error here, or are there any attempts we can possibly make to recover?

Recovering from a failed worker is very difficult because the master has to know who is doing what, use a ping-pong protocol to check if the status of workers, then try to restart a worker and re-send the job if one died. A robust workflow system should be able to do this but it is too difficult (time-consuming) for me to do at this point.


Padding on fluid input makes element too wide

Describe the bug
I created a small page using a form with a fluid input. An example can be seen here:
https://codepen.io/anon/pen/LBOBGP
I expected the fluid input to be as wide as the outer container. But it is wider.
When I remove the 10px padding on each side of the input it fits.
Am I doing something wrong? Or is this a bug?
To Reproduce
See codepen: https://codepen.io/anon/pen/LBOBGP
Expected behavior
Input as wide as outer container.
Screenshots or Project link
https://codepen.io/anon/pen/LBOBGP
Desktop (please complete the following information):

iOs, Chrome and Firefox
<EMAIL_ADDRESS>

Hello @Guysbert . Thanks for your bug report. I have fixed it on input element. Also, I have fixed the same issue on other elements that have the fluid modifier (select and textarea).

Great! When will you publish the next release? :D

I hope that tomorrow will be ready to be published :smile:

Please @Guysbert , could you check if the input in the search box of the documentation is displayed well on your device? https://docs.siimple.xyz/getting-started/installation.html
And the fluid input of the documentation? file:///Users/jmjuanes/workspace/siimple/siimple/docs/_site/form/input.html

Oops, sorry. The correct link is: https://docs.siimple.xyz/form/input.html
Ok, so I don't know why in your codepen it looks wider and in the documentation it fits to it's parent size...

On your documentation page, the browser adds the user agent stylesheet setting box-sizing: border-box;

On the codepen page it does not do that:

After googling the issue it seems that this behaviour is caused, when <!doctype html> is added to the head of your page which is not the case on your documentation page. I guess if you'll add it, the input field will also be too wide in the documentation.
See also:
https://stackoverflow.com/questions/7534222/what-causes-the-user-agent-stylesheet-to-use-border-box-instead-of-content

I suggest to add the <!doctype html> tag to your webpage (because it's good style) and then add a box-sizing: border-box to the siimple-input--fluid class.

Confirmed. This is how your page looks when you add the doctype tag:

Closing issue. New version v3.1.1 will be published in few minutes with the fix of this bug. Thank you :smile:


Q: Is there a way to suppress asking for the user's location when loading an ad?

I'm loading an interstitial on the UI thread, but despite me not adding
ACCESS_FINE_LOCATION on the manifest, MobFox still asks for it, is there a way to skip this, or is this mandatory?
We have several reasons not to ask our users for fine location:

Privacy
Our app has nothing to do with the user's location
We want to be energy efficient and not have to use the GPS, everytime our app starts to drain more battery we see our installation numbers drop.

hmm, perhaps with reflection I can remove the BannerInitTasks.Tasks.GET_LOCATION?
wouldn't want to hack like that.

I was trying this, but no go:
interstitialAd = new InterstitialAd(activity);
final BannerInitTasks initTasks = interstitialAd.getBanner().getInitTasks();
        try {
            final Field tasksField = initTasks.getClass().getDeclaredField("tasks");
            tasksField.setAccessible(true);
            Map<BannerInitTasks.Tasks, Boolean> tasks = (Map<BannerInitTasks.Tasks, Boolean>) tasksField.get(initTasks);
            if (tasks != null) {
                tasks.remove(BannerInitTasks.Tasks.GET_LOCATION);
                LOG.info("Removed GET_LOCATION task");
            }
        } catch (Throwable t) {
            t.printStackTrace();
        }

Also, if the user denies permission it asks over and over, and then I got another permission screen to draw on screen.
Are there any settings available to be less aggressive?

yes, there is (static method):
Banner.setGetLocation(false)

We didn't document it because we assumed that in order to get better CPMs most publishers will want the location. With Android 6 permission prompts this is changing. Will have to rethink.

Thank you so much!

@nabriski has Banner.setGetLocation(false) been tested in the latest Android .jar ?
I'm getting errors now that I've updated:
error: cannot find symbol
    Banner.setGetLocation(false);
    ^
symbol:   method setGetLocation(boolean)

Hi,
Yes, we gave it a slight overhaul. You can remove this line, this method was removed and now it's false by default.
See here if you want to enable it:
https://github.com/mobfox/MobFox-Android-SDK#location

Ah, thanks even better. I realized I was actually compiling the wrong
version number anyways is what my issue was.
On Tue, Nov 29, 2016 at 8:53 PM Itamar Nabriski<EMAIL_ADDRESS>wrote:

Hi,
Yes, we gave it a slight overhaul. You can remove this line, this method
was removed and now it's false by default.
See here if you want to enable it:
https://github.com/mobfox/MobFox-Android-SDK#location
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mobfox/MobFox-Android-SDK/issues/35#issuecomment-263695314,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABMwwHMoBUjvd70YOyMFRSvjD51HJa2_ks5rDJC4gaJpZM4JgiJz
.


Error: Could not find Google Chrome executable for channel 'stable' at '/opt/google/chrome/chrome'.

Since v0.3.1 this error shows up on start, with v0.3.0 everything works fine

Hi @Beluk , what browser are you using Chromium ? Seem related to this puppeteer PR. Unforunatly, I have to keep this version because it resolves a critical CVE on puppeteer side. I will try to find an other way to do it

Hi @jean-humann not sure about your question. In Windows i use Chromium but i get the same error. The build machine which running on debian there is no chromium installed. With v0.3.0 it was not necessary. On both machine the same error occurs. So with v0.3.1 i'm no able to use it anymore. Do you know if i must install or setup anything different?

Thanks for those details, I clearly see the issue now. I remove the dependency to puppeteer to only use puppeteer-core. The downsize of that, is that puppeteer-core doesn't install a builtin chromium. I will add this feature this week.

@Beluk sorry for the delay, could you try again with release 0.5.1 please ?

@Beluk sorry for the delay, could you try again with release 0.5.1 please ?

I will do next week, when back from vacation.

Still gettting the same error on linux and on windows machine. :-(

@Beluk the fix #210 is available in V0.6.1.
Could you try and validate if it works for you, please ?
Thanks !

LGTM! Thx for the fix!


v2 - Backup Mode & Edit Start Time

[ ] Closes #30
[ ] Closes #31
[ ] Closes #35
[ ] Closes #39
[ ] Closes #44
[ ] Closes #46
[ ] Closes #48
[ ] Closes #50
[ ] Closes #51
[ ] Closes #52
[ ] Closes #56

Hey @nathanael-rsu, these changes look good. Since I am going to be doing a lot of UI testing right now, I was wondering if you could do me a favor and whenever you push a commit just leave a short comment about what was going on/what you were seeing in the app. That'll definitely help me in my testing. :)

Okay, will do. Thanks.


Controller model: Gateway API Policies controllers

Proposed model

After thinking about this model and trying to apply to existing uses cases, I believe that there is no single pattern or controller model for all controllers dealing with Gateway API policies.
On simple example to "break" the model: when there are broken paths in the Gateway API hierarchy, the mappers cannot scale up to the top type being reconciled. So, for instance, if you create a policy targeting a missing resource, the mapper cannot reach the gateway or kuadrant CR, so no reconciliation is created for the gateway or kuadrant CR, hence the policy status is never updated.

cc @Boomatang

In general I do agree. The case of missing targets is a problem, which will happen. I can see the use of gitops where you can not guarantee the order of creation. The idea there is no single pattern to solve this could be very true. The FOR type on the RateLimitingPolicy is much simpler than the AuthPolicy as the Limitador CR is a one to many but the Authconfig CR is a one to one unless the defaults and overrides are used.
I do not think we will be able to create a generalized solution to solve all the cases, without first solving the problem for the specific use case.
There is a gut feel saying that using mappers on the FOR type might not be the best approach. For instance if Gateway is update or created I do not think that should map back to reconcile functions for any of the Policies that could be attached, but instead we reconcile the gateway. I know this would mean more controllers and possibly more complexity  with a controller, as the controller would have to handle all possible policies that  are attached to it. However that complexity is also triggered today with every policy controller triggering on gateway events.

There is a gut feel saying that using mappers on the FOR type might not be the best approach. For instance if Gateway is update or created I do not think that should map back to reconcile functions for any of the Policies that could be attached, but instead we reconcile the gateway. I know this would mean more controllers and possibly more complexity with a controller, as the controller would have to handle all possible policies that are attached to it. However that complexity is also triggered today with every policy controller triggering on gateway events.

That is the idea behind the concept: "target the top resource type out of the set of resources that impact on the thing you are reconciling. "


Be able to set a custom logo

This will be shown in the nav bar.

@dantownsend, thank you for this great theme. Actually, adding an option for the custom logo would be really really helpful.
Do you have any estimation on when this option will be available?
And do you have any ideas for a temporary workaround for it?
Thank you so much,
Arseniy

@dantownsend, will it be okay if I suggest an implementation for this feature and create a PR?

@are-scenic Yes, feel free to make a PR!
I think we should be able to implement this feature without too much trouble, and get it merged in soon.

Thank you.

@dantownsend, finally I've prepared a PR#37 for this feature. Please review it.
Thank you.


OS names appear with underscores in various Plaster created files

In several spots there are underscores where spaces should be. These will need fixed similar to the account rename and legal properties with dots and without as shown below.
https://github.com/techservicesillinois/SecOps-Powershell-CISDSC/blob/ddf09ac74cb56d4133d5abfaa27988c06ce0f10e/src/CISDSCResourceGeneration/functions/public/ConvertTo-DSC.ps1#L121-L141

resolved in mentioned PRs


"Error: feed contains no items!" displayed while valid feed contains items

I have 2 Atom feeds which items are not displayed within newsboat:

https://sanctum.geek.nz/arabesque/feed/atom
https://junegunn.kr/atom.xml

"Error: feed contains no items!" is reported in the status bar
I've checked their validity against https://validator.w3.org/feed/ and they are both reported as valid there
Newsboat version: 2.10.2

This sounds like you didn't load the feeds. By default the key binding to manually fetch a feed is r or R to reload all feeds. You probably want to put (refresh-on-startup yes)[https://newsboat.org/releases/2.10.2/docs/newsboat.html#refresh-on-startup] in your ~/.newsboat/config to fetch the feeds automatically when newsboat starts. See also the (auto-reload)[https://newsboat.org/releases/2.10.2/docs/newsboat.html#auto-reload] and reload-time settings.

No I have many other feeds working in my config.
Only those 2 seem not to be parsed properly.
Whether on the startup refresh or via the fetch binding, the result is the same
Could you please add those in your config and test to check if you see their items?

In case it matters, I'm using Arch now
I used to have Ubuntu and a previous version of then newbeuter, those feeds didn't work as well there.
One of those feed used to report a SSL issue, and the other nothing AFAIR.
I waited to report the issue because I hoped it would be different on Arch and a newer newsboat version

Both feeds worked for me. Can you show your config? Can you also try with empty config and just these two feeds in your urls file (don't forget to backup your cache.db beforehands!)

When you run Newsboat with just two feeds, start it like this:
$ newsboat --log-level=6 --log-file=newsboat.log

reload all feeds and immediately close Newsboat, then post the resulting newsboat.log. It will be a big file, but that's okay.

Argh... nevermind I'm stupid...
I tested a bit more and I have a setting keep-articles-days 30 indeed
Since those feeds only have old entries, it makes sense none were shown...
Sorry for the noise :/
I was so convinced something was wrong with them given that SSL error I used to have on my old system
However a small thing: if I remove that setting, the feed items still don't show up unless I remove ~/.local/share/newsboat too.
Not sure if that's normal

No problem, glad you resolved it so fast @tardypad!

However a small thing: if I remove that setting, the feed items still don't show up unless I remove ~/.local/share/newsboat too.
Not sure if that's normal

That's expected. Articles older than keep-articles-days got deleted, and they will stay deleted no matter how you change the settings. I've updated the docs to reflect that; hope it helps!

Nice, at least something got a bit improved from my noise :stuck_out_tongue:
Thanks for the help and the quick replies


Update to 0.6.0

This PR updates to the latest KrakenD version and introduces a simple extension of the auth0 sample SPA for a better demonstration of the supported features

LGTM


Slack mention etiquette

This pull request doesn't fix an issue explicitly, but addresses a conversation had in Slack over Slack etiquette from here https://hospitalrun.slack.com/archives/C02BT7ZBK/p1505380410000243
This updates the advice about what commands can and should be used, with the usage of @everyone and @channel being discouraged, and @here being deemed acceptable as it only notifies people currently active on Slack, as per https://get.slack.help/hc/en-us/articles/202009646-Make-an-announcement
cc @HospitalRun/core-maintainers

Thank again, @WeeSkilz!!

Always a pleasure!


improve experience when attempting install on python2.6

follow up to https://github.com/PyCQA/pydocstyle/issues/206

[ ] add python_requires='>=2.7 to setup.py https://stackoverflow.com/a/42792413/1585509
[ ] try to fail early with nice message if installed with older pip/setuptools ?

The second option requires we do some hacky detection in setup.py which really won't go very far, honestly, especially if pip is new enough to prefer wheels.

This sounds okay, PRs welcome.

The final Python 2.6 release was way back in 2013 (https://peps.python.org/pep-0361/#release-schedule), and python_requires has been properly set since eebb17b37401235a79836ca83b6135b76d457c77, so the changes requested here are no longer needed.


Fixed static file extension

This suggestion works: https://github.com/ebekker/ACMESharp/issues/55

Not needed after pull #33.
Included web.config has been tested with stock IIS on Server 2012, Server 2012 R2, Server 2016, Win 7, and Win 8.1. If anyone needs to make changes to the web.config file, you can update the web_config.xml file in the root of the app.That is copied instead of using text hard coded in the app. So no more need to recompile to change the web.config.


Fixed TestBucketStore_ManyParts_e2e

https://app.circleci.com/pipelines/github/thanos-io/thanos/5120/workflows/efd2a21d-13b7-4035-99e3-cb1af8023694/jobs/13809
This fail is only visible for anyone from Thanos Team proposing PR, due
to extra tests against bucket providers.
Bug: https://github.com/thanos-io/thanos/pull/3823/files#r583476067
Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

OSS is bugged too 🤦


fix: undefined txid length

:tada: This PR is included in version 2.3.3 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.3.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


libsystem_kernel.dylib`__pthread_kill:

Steps to reproduce

Installed react native 0.40
npm i react-native-camera@next
-followed steps in manual setup for xcode
followed code in readme

Expected behaviour
content renders on page and allows you to click on [CAPTURE]
Actual behaviour
Builds successful but after simulator opens xcode goes to a new screen with:
libsystem_kernel.dylib`__pthread_kill:
0x1077d8efc <+0>:  movl   $0x2000148, %eax          ; imm = 0x2000148
0x1077d8f01 <+5>:  movq   %rcx, %r10
0x1077d8f04 <+8>:  syscall
->  0x1077d8f06 <+10>: jae    0x1077d8f10               ; <+20>
0x1077d8f08 <+12>: movq   %rax, %rdi
0x1077d8f0b <+15>: jmp    0x1077d37cd               ; cerror_nocancel
0x1077d8f10 <+20>: retq
0x1077d8f11 <+21>: nop
0x1077d8f12 <+22>: nop
0x1077d8f13 <+23>: nop
Environment

Node.js version:
React Native version:
React Native platform + platform version: iOS 9.0, Android 5.0, etc

react-native-camera
"^1.0.0-alpha1"
I cant seem to get this working right out of the box following the README. What am I forgetting or whats happening with this issue?

Sorry. Ignore. I missed the step of code I had to add if on ios 10+
Please Close

Sorry @chipe13, where is this step?

I've already found it. Thanks @chipe13

I have the same problem.  @chipe13, where is this step?


Handling Interrupts During Recording

Version of react-native-audio-recorder-player
2.4.4-rc.1
Platforms you faced the error (IOS or Android or both?)
iOS
Expected behavior
When recording an audio file, an interrupt event occurs. For example, a countdown timer expires in the iOS Timer app, generating an alarm. Or, a phone call comes in and is left unanswered. https://developer.apple.com/library/archive/documentation/Audio/Conceptual/AudioSessionProgrammingGuide/HandlingAudioInterruptions/HandlingAudioInterruptions.html
Is it possible to handle this type of interrupt? Ideally, triggering a Pause recording action when the interrupt occurs, then a Resume recording once the interrupt is completed.
Actual behavior
The interrupt occurs, and recording stops and does not resume.
Steps to reprodue the behabior
Start a recording, trigger an Alarm by setting a short countdown timer in the iOS Timer app.

Looks like this is another feature request. Currently, we don't take care of that situation and you should have to re-record them.

Related # 15

does anyone know if this ever got fixed in the subsequent 3 years? I recall this working for me properly a few months ago - a phone call or alarm would pause the recording - but now it seems to just scrap it.

Look at this

This is fantastic - thank you @pipi32167 @hyochan Works great!


add support for mysql and postgresql 9 - 10

add support for mysql and postgresql 9 - 10

Any version of PostgreSQL older than 12 is already "obsolete", you can see them at this link: https://www.postgresql.org/support/versioning/
Even version 12 has its EOL on November 14, 2024, and that is the main reason why it is not supported.
You should really consider upgrading your Postgres version, both for security and compatibility.

Where I work we have many legacy systems that we cannot update, the people who developed them are no longer with us and we have few resources and people, so it is not possible to simply update these applications, we have PostgreSQL 9.3, 10,11,12 and 16 databases and MySQL

@insinfo But don't expect maintainers of other software to keep those old versions. It may be very hard to have old library versions mixed with new ones in one software.
And this is also responsibility of management to keep enough people to maintain all the legacy apps that are in the company and also keep them up to date with current library versions and security vulnerabilities patched.

@camaeel you are completly right. For the moment PG Back Web will only include the postgresql versions that are officially supported by them and in the future, older versions will be removed from the latest releases
https://www.postgresql.org/support/versioning/

@insinfo In the company you work, they should update the components, or at least sponsor or help the project so we can add that kind of support

I work in a small city hall in a small town in the interior of Rio de Janeiro / Brazil, we simply do not have the financial resources and we also cannot find qualified workers with the salary we can offer. I believe that there are many who find themselves in this situation.
I understand that it is difficult to maintain support for older versions, but the PostgreSQL protocol has not changed much since version 9, and for simple databases that do not use specialized features of newer PostgreSQL versions there is no difference, and PostgreSQL 16 tools such as pg_dump and psql work with older databases without problems, and almost all PostgreSQL drivers work with older versions.

And this support for version 9-10 does not need to be forever, it could be limited to the current version and then deprecate them.

We can invariably update legacy applications but this will be slow and time consuming, we currently only have 2 developers to support multiple applications.


DAGCircuitError: 'bit mapping invalid

Informations

Qiskit: 0.39.2:
Python: 3.10.9:
Mac:

What is the current behavior?
I'm implementing quantum half adder on Jupyter Notebook.
When I try running my circuit on the simulator "qasm_simulator", Jupyter said
DAGCircuitError: 'bit mapping invalid: expected 4, got 8'
here is the code I've written. The error occurs on the last line of the third code.
from qiskit import QuantumCircuit, QuantumRegister, ClassicalRegister, execute, Aer
#SUM
X = QuantumRegister(1, "in |X⟩")
Y = QuantumRegister(1, "in |Y⟩")
sum_out = QuantumRegister(1, "out SUM |0⟩")
SUM = QuantumCircuit(X, Y, sum_out, name='SUM')
SUM.cx(1, 2)
SUM.cx(0, 2)

fig = SUM.draw('mpl', True)
SUM = SUM.to_instruction()
fig

#half_adder
cout = QuantumRegister(1, 'out Carry |0⟩')
c = ClassicalRegister(4)
hadder = QuantumCircuit(X,Y,sum_out,cout,c)
hadder.ccx(X,Y,cout)
hadder.append(SUM,[0,1,2])

show = hadder.draw("mpl",True)
hadder = hadder.to_instruction()
show

#testing half_adder
qu = QuantumRegister(4)
cl = ClassicalRegister(4)
circ = QuantumCircuit(qu,cl)
circ.x(qu[0])
circ.x(qu[1])
circ.append(hadder,[0,1,2,3])

for i in range(0,4):
    circ.measure(qu[i],cl[i])
circ.draw("mpl",True)
print(execute(circ,Aer.get_backend('qasm_simulator'), shots = 1).result().get_counts())

What is the expected behavior?
I don't totally understand the error. I hope to troubleshoot to see the result.
Suggested solutions

Your immediate problem is that the line
circ.append(hadder, [0, 1, 2, 3])

doesn't include any classical arguments to apply hadder to, but it expects 4 (though they're not used).  Perhaps you either meant not to have the ClassicalRegister c in hadder, or you meant to write the above line as
circ.append(hadder, [0, 1, 2, 3], [0, 1, 2, 3])

On our side, the append call I pulled out should have raised an error.  I'm not certain why it didn't, but it definitely looks like a bug that it didn't.


Authentication Error

I am trying to install from command prompt. Below is the command.
go get github.com/aws/aws-lambda-go/cmd/build-lambda-zip
Receiving the below error.
"package gopkg.in/urfave/cli.v1: unrecognized import path "gopkg.in/urfave/cli.v1" (https fetch: Get https://gopkg.in/urfave/cli.v1?go-get=1: authenticationrequired)"
Tried installing "go get gopkg.in/urfave/cli.v1"
Received below error "package gopkg.in/urfave/cli.v1: unrecognized import path "gopkg.in/urfave/cli.v1" (https fetch: Get https://gopkg.in/urfave/cli.v1?go-get=1: authenticationrequired)"

Need help in resolving this issue

I believe this was resolved by #223.


Define a setting

Where and when in time the game takes place is pretty important for our assets and sound effects.

For time I'm thinking more in the future, which can explain the device. Maybe the player is a researcher in a giant lab and an experiment got out of control, causing dark matter to spread everywhere. Or maybe the player is an explorer and finds a cave with evil magic that they need to escape from. I think the cave could make more sense for level design, since stuff can look more randomly made when it's natural.

I think we're set on this one: the player is a slime who was doing its thing in a cave until it stumbled upon a magical object, like Pandora's box. When it touched it, evil was released and dark matter started spreading everywhere. The slime must escape the rapidly decaying cave to survive and live it's slimy life.

agree with cave setting (already working on tiles with that theme)


refactor Form render props

Ability to pass form element attributes directly in <Form>.
Remove default form-horizontal class on form element

Fixed it in another commit.


"Unexpected keyword argument" error on JSON serialization

After simplejson usage was removed in #780, the serialization error handling no longer works as expected, because the standard lib json.JSONEncoder.__init__() does not accept the encoding keyword argument, producing unexpected keyword argument error.
In addition to that, simplejson.dumps can handle bytestring keys in dicts, while json.dumps can't: https://bugs.python.org/issue18820.
E. g. (with Python 3.4):
>>> simplejson.dumps({b't': 1})
'{"t": 1}'
>>> json.dumps({b't': 1})
...
TypeError: keys must be a string

I think we either need to revert and require simplejson, or we need to try and work around json's limits. It seems like requiring simplejson is the better solution here, as I'm concerned about removing any of the other functionality.

more thought on this: im ok reverting on any behavior that is not bound to tests. I think the Decimal behavior should be, but any use of invalid params should not exist here, and any edge case handling without tests is deemed inappropriate.


Manifest changes to work for IBM Cloud Tools for Swift

Verified to work with standard Bluemix deployment, as well as with ICT.

Thanks @egross91 !


feat (RingTheory/Valuation/Integers) : add lemma one_of_isUnit'

Add lemma one_of_isUnit' , that is a generalization of one_of_isUnit with a slightly weaker assumption.
Co-authored-by: María Inés de Frutos Fernández @mariainesdff

@erdOne  Did you find the time to have another look at my modifications following your suggestions?

Oops didn't notice that this is ready for review again. Thanks for the ping.
maintainer merge

bors merge

Oops didn't notice that this is ready for review again. Thanks for the ping. maintainer merge

Thanks ;-)


fix(lifecycle): scope might changed when call hook

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the main branch for v2.x (or to a previous version branch)
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
This pull request's issue originates from a specific problem mentioned in https://github.com/vuejs/pinia/issues/1800 . Due to the potential for callHook to modify the upper-level scope's behavior of collecting effects, it subsequently leads to incorrect effects being torn down when stopping in the wrong scope.

This'd be great to have reviewed and merged. We're in the middle of moving a fairly large Vue 2 + Vuex application into Pinia, and we'd like to avoid the boilerplate in working around this issue.


Mnemonic displayed as invalid

I have breadwallet on one iphone and try to restore the same wallet on another iphone (active/active availability ;-). When entering the 12 word mnemonic on the second (identical) breadwallet app, it shows that it is invalid. However it is valid and if I import it in electrum using BIP39, it works out fine.
Any idea what could be wrong here?

Hi thank you for reporting. This is very unfortunate, but most likely a simple error. Please contact<EMAIL_ADDRESS>for further assistance.


Extend WebAssembly with Host Functions

Is your feature request related to a problem? Please describe.
Right now what is possible: Wry loads HTML/JavaScript, which can load a WASM module, which can call code that was written in Rust that has been compiled to WebAssembly. What if the Rust you've written can't compile to WebAssembly due to thinks like dlopen / dlsym (FFI)? If you could provide host functions (in this case dlopen / dlsym) into the Wry WebView WASM runtime, it opens a new world of opportunity.
Describe the solution you'd like
Wry to somehow mesh the WebView WASM logic/runtime with something more powerful/robust (like Wasmer, etc.)
Describe alternatives you've considered
WebView IPC bridge through JSON messaging.
Would you want to assign yourself to implement this feature?

[ ] Yes
[X] No

Additional context
https://www.secondstate.io/articles/extend-webassembly/#:~:text=For Wasm%2C the Host Function,functionality to a large extent.
https://wasmedge.org/book/en/sdk/c/hostfunction.html
https://docs.wasmer.io/integrations/examples/host-functions

unfrotunately this is not possible since we don't have much control over the js/wasm runtimes and I don't think we would work on this any time soon plus I think your use-case could be off-loaded to the rust process that runs Wry instead of through wasm.

@amrbashir would it have been a better question if I asked "could the existing with_ipc_handler be extended to pass Uint8Array buffers instead of just String to enable more efficient passing of binary encoded payloads"?

While it is possible right now with the current method, it requires serialization on both ends and results in slow performance.
We are actively investigating improving the IapC performance, see #454 and https://github.com/tauri-apps/tauri/discussions/5690

While it is possible right now with the current method

Passing something other than a String does not seem to be supported on Mac OS X from what I can tell.
window.ipc.postMessage(new TextEncoder().encode('Hello, world!'))

2023-04-24 16:51:40.059 foo[60701:659390] -[__NSFrozenDictionaryM UTF8String]: unrecognized selector sent to instance 0x6000015b1000
thread 'main' panicked at 'Uncaught exception <NSException: 0x600001b915f0>', /Users/brandonros/.cargo/registry/src/github.com-1ecc6299db9ec823/wry-0.28.0/src/webview/wkwebview/mod.rs:103:37
stack backtrace:
   0:        0x100b18c64 - std::backtrace_rs::backtrace::libunwind::trace::h2000fb4d08dbcc59
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/../../backtrace/src/backtrace/libunwind.rs:93:5
   1:        0x100b18c64 - std::backtrace_rs::backtrace::trace_unsynchronized::h2b5e61495350674d
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
   2:        0x100b18c64 - std::sys_common::backtrace::_print_fmt::h05f5bfbdb3415936
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/sys_common/backtrace.rs:65:5
   3:        0x100b18c64 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h105074e3d85f800b
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/sys_common/backtrace.rs:44:22
   4:        0x100b301a8 - core::fmt::write::h34766cf8fff7af1e
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/core/src/fmt/mod.rs:1232:17
   5:        0x100b16890 - std::io::Write::write_fmt::hd64c4cf6e7adea59
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/io/mod.rs:1684:15
   6:        0x100b18a78 - std::sys_common::backtrace::_print::hd92783a665d3ebfb
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/sys_common/backtrace.rs:47:5
   7:        0x100b18a78 - std::sys_common::backtrace::print::h2a6828a537036cf9
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/sys_common/backtrace.rs:34:9
   8:        0x100b1a154 - std::panicking::default_hook::{{closure}}::h4e82ce6ccef941b2
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/panicking.rs:271:22
   9:        0x100b19eac - std::panicking::default_hook::h29f62f8795c5cb00
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/panicking.rs:290:9
  10:        0x100b1a664 - std::panicking::rust_panic_with_hook::h19862cbd0fbda7ba
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/panicking.rs:692:13
  11:        0x100b1a598 - std::panicking::begin_panic_handler::{{closure}}::h3f3626935e1669fe
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/panicking.rs:583:13
  12:        0x100b19084 - std::sys_common::backtrace::__rust_end_short_backtrace::h5054ef52bd507d0a
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/sys_common/backtrace.rs:150:18
  13:        0x100b1a2f4 - rust_begin_unwind
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/std/src/panicking.rs:579:5
  14:        0x100b3efb0 - core::panicking::panic_fmt::h7e47e10600a90221
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/core/src/panicking.rs:64:14
  15:        0x100a89474 - core::panicking::panic_display::h509e9028a372fa53
                               at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/core/src/panicking.rs:147:5
  16:        0x100991b2c - wry::webview::wkwebview::InnerWebView::new::did_receive::h3fa5c70ecfe6125d
                               at /Users/brandonros/.cargo/registry/src/github.com-1ecc6299db9ec823/wry-0.28.0/src/webview/wkwebview/mod.rs:103:37
  17:        0x1beee272c - <unknown>
fatal runtime error: failed to initiate panic, error 5
Abort trap: 6

That's what I meant by serialization on both sides, you need to use JSON.stringify on JS side before sending to Rust, then deserialize on it on rust

I've explored quite a bit on better IPC in #454.
Theoretically, all platforms can do so. But it's rough to come up with unified interface.
I suggest using custom protocol which can be more flexible and performant.

You can use https but will need to handle CORS.
Tauri itself hosts a dev server when developing but uses custom protocol directly in release build.

access-control-allow-origin: *
content-security-policy: default-src 'self' 'unsafe-inline' ipc:;

if those 2 response headers aren't enough, what else is needed to circumvent:
[Warning] [blocked] The page at https://redacted.app/ was not allowed to display insecure content from ipc://test. (redacted.app, line 8)

[Error] Not allowed to request resource

  (anonymous function) (redacted.app:8)
  run (redacted.app:7)
  Module Code (redacted.app:19)
[Error] Fetch API cannot load ipc://test due to access control checks.

  (anonymous function) (redacted.app:8)
  run (redacted.app:7)
  Module Code (redacted.app:19)
[Error] Unhandled Promise Rejection: TypeError: Load failed

  (anonymous function) (redacted.app:8)

According to doc, you should set ACAO to ipc://test if you are on macOS.

hmmm could you file another issue with steps to reproduce.
Because from tauri side, it can work with dev server with custom protocol enabled.
I'm curious what doesn't work in this case.


Does onnxruntime EP tensorrt quantized model can directly inference via tensorrt?

EP tensorrt quantized int8 model, I want direcly inference via tensorrt, doesn't through onnxruntime, is that possible?

The code is opensourced, it is not hard to find the answer from it. So based on the following implementation detail
https://github.com/microsoft/onnxruntime/blob/60db14bac39d5b0e7809d604d38e5b9aadf1df6e/onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L1558-L1568
if the engine encryption it not enabled, then the answer is simply yes.

@cloudhan thanks. May I ask further, what's the pros and cons in onnxruntime quantization with tensorrt EP compare with original tensorrt built-in int8 quantization?


Seem to be not supported web

Hello frideos team,
I try to build web application, but it stuck with error:
══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
The following UnsupportedError was thrown building _InheritedState:
Unsupported operation: Platform._operatingSystem

The relevant error-causing widget was:
  _InheritedState
  file:///Users/macbook/flutter/.pub-cache/hosted/pub.dartlang.org/frideos-1.0.0+1/lib/src/app_state_provider.dart:51:7

Please help me check it again, thank you!

frideos 1.0.1 seems to work on web. But it doesn't support null safe.
Warnings pop up:
The library 'package:frideos/frideos.dart' is legacy, and should not be imported into a null safe library.


Options for setOptions?

Hello, i got a question.
Which commands has the setOption function? I want to pass it a html file generated by a function on my zf2 module. I'm very noob in zf2, sorry for that.
Thanks a lot!! And thanks for the module!

Hi @tupicho,
As you've noticed DOMPDFModule hasn't received the time and attention it deserves. I understand how frustrating this is and I'm sorry you've had to deal with this frustration 😢
I'm currently taking steps to stabilize DOMPDFModule which includes code updates as well as creating a backlog that will ensure its continued success.
You can follow project Phoneix  to see what immediate work is being prioritized.
Thank you for your understanding and patience.


[DO-NOT-MERGE] Remove temporary fallbacks in HiveExternalCatalogVersionsSuite

What changes were proposed in this pull request?
The fallback was added because Jenkins builders were unstable. This PR removes it.
Why are the changes needed?
TBD
Does this PR introduce any user-facing change?
TBD
How was this patch tested?
TBD

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33076/
Test PASSed.

retest this please

Test build #128456 has started for PR 29694 at commit 355be55.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33080/
Test PASSed.

Test build #128452 has finished for PR 29694 at commit 355be55.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128452/
Test FAILed.

Test build #128456 has finished for PR 29694 at commit 355be55.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128456/
Test FAILed.


[ADDED] More badges fun :-)

Updated readme with more badges! yay!

Current coverage is 0.00% (diff: 100%)

Merging #64 into develop will not change coverage

@@           develop   #64   diff @@
====================================
  Files            1     1
  Lines           25    25
  Methods          0     0
  Messages         0     0
  Branches         0     0
====================================
  Hits             0     0
  Misses          25    25
  Partials         0     0

Powered by Codecov. Last update 6895188...256f17e


Updating to QDM 4.3

Adding 'Assessment, Performed' and 'Assessment, Recommended' data criteria.
Other relevant pull requests are in:

Patient API
health-data-standards

File to test the QDM 4.3 updates with:
AsthmaQDM4_3_v4_6_Artifacts.zip


only parse task in updateTask

only parse task in updateTask.
fix bucket names in remove-task-workflow.
delete extraneous zip file.

OK


Libraries::ResourcePostgresqlCloudBackup: default argument value should ...

...be nil because of set_or_return behaviour:
  if arg == nil && self.instance_variable_defined?(iv_symbol) == true

In this way, assigned value will be returned only if default value will be nil

Right, but in pr you set required: true for command_prefix

Sorry for this.
As far as I can understand fresh chef sources (https://github.com/chef/chef/blob/2603e2153d6ab50179d2278025a51579edb9033f/lib/chef/mixin/params_validate.rb#L100-L107) this is not necessary anymore.
Anyway, fixed in fdfa1c0cc7f10635b4abd8aec91af492e8762ca2


feat: Mandatory depends on logic added

port from https://github.com/frappe/frappe/pull/8820
ASNA Task: https://app.asana.com/0/1200414497004131/1200413538586244

@vishdha fix Sider issues

@vishdha fix Sider issues

Resolved


Update controller generator templates for new ParameterSanitizer syntax

Hi, just playing around on a new project, and having a poke around HEAD on devise (mainly because I had read the README of HEAD and not 3.5.3, so stuck with what I'd learned about).
Not really too familiar with devise, so not sure if I'm missing something here, but think I may have noticed something you've missed, so here's a PR.
Devise::ParameterSanitizer has a new syntax for permitting additional attributes to a model. This commit updates the generated controllers to reflect that.

@james yeah, we missed those comments. Thanks a lot :heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:.


Only able to make it work on initial view controller

Hi
Firstly, thank you for this alert view.  I really like it!
I am having an issue however with it that.

When the alert view is presented in the initial view controller - ALERTS WORK!
When the user goes through the view controllers - ALERTS NO LONGER WORK :(

Could you advise a novice developer as to why and what I should be looking out for?

having the same issue. Temporary solution was importing SCLAlertView-ObjectiveC

I have the same issue.
Please, how do I import and use  SCLAlertView-ObjectiveC in my swift project?
Thank you!

May want to look into this https://github.com/vikmeup/SCLAlertView-Swift/issues/38


[ENH] Moredocs and experiment with api implementation

PR Description
Addresses #5 and #19
Merge checklist
Maintainer, please confirm the following before merging:

[ ] All comments resolved
[ ] This is not your own PR
[ ] All CIs are happy
[ ] PR title starts with [MRG]
[ ] whats_new.rst is updated
[ ] PR description includes phrase "closes <#issue-number>"

One think that could be useful is to keep the np.ndarray (n_components, n_classes) format which has merit in itself for from mne_icalabel.iclabel import label_components while providing a 'simpler' user-friendly output for from mne_icalabel import label_components.

Nice, I also added rendering docs directly with a circleCI redirect:
https://output.circle-artifacts.com/output/job/af197406-a7a9-45c9-8093-6c3c0d5c2043/artifacts/0/dev/auto_examples/iclabel_automatic_artifact_correction_ica.html#sphx-glr-auto-examples-iclabel-automatic-artifact-correction-ica-py

I will merge this once the CI complete.


Relax rule: import/no-absolute-path

I would like to relax the import/no-absolute-path rule which "Forbids import of modules using absolute paths". It's not that I think absolute imports are a good idea, it's just that this rule is incredibly slow!
I opened an issue about it on the eslint-plugin-import repo: https://github.com/benmosher/eslint-plugin-import/issues/803
Almost 25% of linting time is spent in that one rule. Since this isn't actually a very common error (0 repos in our test suite actually failed when this was added, it's no big deal to just remove it, IMO)
Here's the raw output:
TIMING=1 standard                                                                                master
Rule                         | Time (ms) | Relative
:----------------------------|----------:|--------:
import/no-absolute-path      |    27.906 |    24.3%
indent                       |     7.070 |     6.2%
no-extra-parens              |     4.939 |     4.3%
comma-dangle                 |     4.313 |     3.8%
no-unused-vars               |     3.695 |     3.2%
no-regex-spaces              |     2.834 |     2.5%
space-in-parens              |     2.611 |     2.3%
no-useless-return            |     2.361 |     2.1%
no-unmodified-loop-condition |     2.310 |     2.0%
block-spacing                |     2.126 |     1.9%

Confirmed a noticable (150-200ms) improvement on a real-world repo (webtorrent):
Before:
$ rm -rf ~/.standard-cache && time ~/code/standard/bin/cmd.js
2.74s user 0.16s system 123% cpu 2.353 total

After:
$ rm -rf ~/.standard-cache && time ~/code/standard/bin/cmd.js
2.55s user 0.15s system 124% cpu 2.178 total

Released as 10.0.2

This rule will be re-enabled in standard v14 since the performance issues with it have been fixed now See: https://github.com/standard/standard/issues/1343
✨


[BEAM-13232] Close clients properly in KinesisSource.

Close AWS clients properly in KinesisSource.
Additionally using lazy init for AWS clients to avoid unnecessary allocations for some use cases, e.g. for splitting (doesn't require cloudwatch).

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @aromanenko-dev


GH-15045: [CI][conda] don't build pyarrow in r-jobs

Follow-up to #15014
Fixes #15045

Closes: #15045

@github-actions crossbow submit -g conda

@github-actions crossbow submit -g conda

@github-actions crossbow submit -g conda

@github-actions crossbow submit -g conda


TEXT CHANGE

Please don't make such useless changes.
For contributing, strictly follow issue tracker and
maintain community guidelines.


Error in stage 7?

Hi there, I think there is an error in your test for stage 7 of the DNS Server challenge. The pointer is 0xc010 when I think the correct value should be 0xc004. Could you please verify this?

@RamitGoolry feel free to add any extra details here - if you could print out the full bytes you're receiving in hex format, vs. what you think they should be, that'd be ideal

@rohitpaulk, thanks for reaching out! I actually realized as I was explaining the error in this comment that I was mistaken. I was mistakenly reading the 16th byte from the start of the question section not the start of the whole DNS message.
Since the header section is 12 bytes, that explains the difference of 12 bytes from 0x10 to 0x04. We can close this issue. Thanks again.

Gotcha, no problem. Thank you for sharing the context!


BartyCrouch runs much slower with Xcode 10.2

Since the update to Xcode 10.2, it feels like BartyCrouch runs much slower than previously.
Times like this happen on every build of a project where BartyCrouch once didn't noticeably delay builds with Xcode 10.1:

Task 'Update Interfaces' took 3.45 seconds.
Task 'Code Transform' took 11.587 seconds.
Task 'Normalize' took 22.055 seconds.
Task 'Lint' took 22.768 seconds.

Make BartyCrouch fast again! 🥇

Thanks @fredpi for reporting this. I will investigate shortly, I didn't notice any slowdowns in BartyCrouch and the reported times are definitely a no-go. Could you please tell me how exactly you installed BartyCrouch? Did you use brew install/upgrade bartycrouch? Swift code optimizations may play an important role regarding performance ...

@Dschee Thanks for looking into this.
I installed BartyCrouch via brew install and recently upgraded to version 4.0.1 via brew upgrade.

In trying to upgrade to BartyCrouch 4.0, we also saw some slowdown in Xcode 10.2
kevin-3:ios [kd/string_test]$bartycrouch update -v
Starting Task 'Update Code' ...
2019-04-11 12:16:50.048: ℹ  Incrementally updated keys of file '/Users/kevin/git/ios/Foo/de.lproj/Localizable.strings'.
2019-04-11 12:17:00.398: ℹ  Incrementally updated keys of file '/Users/kevin/git/ios/Foo/ja.lproj/Localizable.strings'.
2019-04-11 12:17:09.753: ℹ  Incrementally updated keys of file '/Users/kevin/git/ios/Foo/en.lproj/Localizable.strings'.
2019-04-11 12:17:19.280: ℹ  Incrementally updated keys of file '/Users/kevin/git/ios/Foo/es.lproj/Localizable.strings'.
2019-04-11 12:17:28.730: ℹ  Incrementally updated keys of file '/Users/kevin/git/ios/Foo/fr.lproj/Localizable.strings'.
2019-04-11 12:17:38.425: ✅  ./Foo:  Successfully updated strings file(s) of Code files.
Task 'Update Code' took 66.402 seconds.

I believe the previous version we were on, 3.something (kinda old) was in the range of 5 seconds. We have a pretty big project, something in the order of 1500 swift files and our localizable strings files are ~3000 lines.
Looks like most of the time is spend writing the files?
Installed via brew, 4.0.1

Today, I investigated a bit about this issue and found out that while the following line performs quickly, times add up so that it accounts for an estimated 80 - 90 % of the run times.
https://github.com/Flinesoft/BartyCrouch/blob/b16ee0255908b9d0bf27b76126d42112b472a717/Sources/BartyCrouchKit/FileHandling/StringsFileUpdater.swift#L346
The .count at the end doesn't do much harm; .prefix is about 2,5x fast than .components. I've tried multiple approaches to make things faster but didn't come up with something relevant yet. As for the components().count: We could use .filter { $0 == "\n" }, but that still leaves the .prefix...

Here are two approaches, that don't make things faster:
var idx = string.startIndex
var line: Int = 0
while idx < string.index(string.startIndex, offsetBy: valueRange.location) {
    if string[idx] == "\n" { line += 1 }
    string.formIndex(after: &idx)
}

var line: Int = 1
for index in 0 ..< string.count {
    if index >= valueRange.location { break }
    if string[string.index(string.startIndex, offsetBy: index)] == "\n" {
        line += 1
     }
}

I just did another measurement by removing the heavy line calculation and found out that 'Normalize' only took 0.373 sec (compared to ~22 sec previously). If we can't find a solution to make the calculation much faster, maybe there should be an option not to calculate the line...

This fix doesn't work for me. I still get Task 'Code Transform' took 21.23 seconds with BartyCrouch 4.0.2, same as @fredpi.

@Kondamon Is the normalize task slow for you, too?

@Kondamon Is the normalize task slow for you, too?
No, the normalize task finishes extremly fast without any delay (<0.1 sec). I don't even receive a message for how long it took in the console.

Okay, in that case, could you please open a new issue specifically stating that the code transform task is slow for you? Also could you provide your configuration file contents and Xcode, macOS and BartyCrouch versions there? Additionally, a guess about how many files or code lines your project contains would be helpful, too.