[web] Wrong Icons displayed / Icons font cached after framework update

Bug
There is a bug with the Material icons. It happens through the following sequence of events:

Users uses web app
Icons class is updated with new code points in https://github.com/flutter/flutter
Web app is updated → new Flutter version is deployed
User opens web app again
Flutter web service worker pulls new app version (with new Icons code points)
Old icons font is pulled → results in completely wrong icons

Impact
The impact of this bug can be critical. See the following screenshots of the proper icons vs. the incorrect cached icons.
One of the icons in this Flutter web app (ignore the language, focus on the icons) is replaced by 💸, which might seem funny, but is also unacceptable from a product standpoint.

Cached / wrong version
Proper version

Steps to Reproduce
This is an e2e issue, i.e. I cannot provide a minimal sample. What needs to happen for this is for Chrome to cache an old version of the Material icons font but pull a new version of the app.
This happens to us frequently in the mentioned production app, which is why I want to open this issue. Here is the Flutter doctor anyway:
Click to expand versions
[✓] Flutter (Channel stable, 2.2.3, locale en-US)
    • Flutter version 2.2.3
    • Framework revision f4abaa0735 (12 days ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Chrome - develop for the web
    • Chrome

[✓] Connected device (3 available)
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 91.0.4472.124

Potentially related
I found the following two references, which I think might be related but both do not duplicate this issue:

Question on SO, which might be the same issue but has no actual answer
https://github.com/flutter/flutter/issues/75288, which might also be the same cause but was closed

Potential fix
You might argue that we should change the caching strategy of our web app, however, here are two reasons for why this should be fixed by Flutter:

We have no way of knowing the Flutter version the app the user receives is running
On top of that, the icons fonts is not loaded by us but by Flutter, so we cannot control the caching

I think the service worker used in Flutter web should fix this issue.
The icons fonts should be versioned with the hash of the Icons class (source file?) and then at runtime, Flutter web should only pull the cached icon font when the code points have not changed.
I tried to provide as many details as possible - have been seeing the issue for quite a while with varying extent but was reluctant to open an issue so far due to the lack of a minimal sample, however, I think if a web expert takes a look at this issue, they should be able to identify exactly why this happens and how to fix it (hopefully) 🙏

Probably duplicate of https://github.com/flutter/flutter/issues/85296 or https://github.com/flutter/flutter/issues/83703 ?

@bernaferrari it should be a dupe of https://github.com/flutter/flutter/issues/83703. However, I was not able to find it.
Maybe the additional information here is use for @guidezpl or @yjbanov.

Hi @creativecreatorormaybenot,
Thanks for filing the issue, closing as this is a duplicate of #83703 please follow up there for further updates. If you disagree feel free to comment and I will reopen it.
Thank you.

@maheshmnj I think the issue can be improved. #83703 is missing details in the description as well as the tile. This makes it difficult to find and also difficult to follow-up on it.

@creativecreatorormaybenot, I agree your fings are detailed and valuable, But I am not sure if it would be right to update the other issue with your findings, but you could always add your findings in the comments of that issue so that the details don't get missed.
Thank you.


Command Injection due to old node-modules

Any chance you can update the  jasmine-growl-reporter package to version 1.0.1?  There is a security vulnerability in the current version (0.0.3)

Once you get this fixed, other library's that depend on your library can be updated as well.
Thanks

Just one more reason to switch to jest.

1 year has passed and this is still not fixed. 😢

Just need to upgrade jasmine-growl-reporter to v2.0.0 – https://github.com/AlphaHydrae/jasmine-growl-reporter. Pretty please!

I see this has been addressed, caused breaking changes, and was reverted: https://github.com/mhevery/jasmine-node/issues/433

So please fix this!

Feel free to use this fork in the meanwhile:
https://github.com/daern91/jasmine-node
Just remember that, like discussed, it will not support node < 4
npm i daern91/jasmine-node


Mod ideas and problems

I played the game a lot with your mod and I loved its concept, I think that several landmines scattered throughout the world make everything more realistic and fun, but after playing for many hours I noticed some serious problems in the mod.
The first of these is the mine spawning inside the ship and killing everyone.
The second problem is that your mod makes the mine not deactivatable via the game console, after putting the mod on I was unable to deactivate the mines via the console.
The third is that there are some ghost mines, I got a clip with one of them and I'll leave it below.
Ghost Mine Clip
The fourth problem is the mines spawning in The Company Building which in theory should be a safe place, this causes a lot of losses after all, whenever a worker dies there we lose 20% of the resources.

@WarrenHood Please, if you're not busy, could you do an update fixing some of these errors?

@WarrenHood Please, if you're not busy, could you do an update fixing some of these bugs?

I'll be able to help out with some of these after the 3rd of January

I'll be able to help out with some of these after the 3rd of January

Okay, I'm grateful and I wish you a happy new year!

@JhonnyBRz Mina was there

I haven't had a chance to look at this yet

I haven't had a chance to look at this yet

this is the least of the problems, the other bugs are more dangerous


Getting an issue in my docker build using 0.11C

Setting up bitcoinxt (0.11C) ...
[91mdebconf: unable to initialize frontend: Dialog
[0m [91mdebconf: (TERM is not set, so the dialog frontend is not usable.) debconf: falling back to frontend: Readline
[0m [91mdebconf: unable to initialize frontend: Readline
[0m [91mdebconf: (This frontend requires a controlling tty.)
[0m [91mdebconf: falling back to frontend: Teletype
[0m [91madduser: The user `bitcoin' already exists. Exiting.
[0m [91mdpkg: error processing package bitcoinxt (--configure): subprocess installed post-installation script returned error exit status 1
[0m Processing triggers for libc-bin (2.19-0ubuntu6.6) ...
[91mErrors were encountered while processing: bitcoinxt

Any idea what's going wrong? Didn't have this problem with 0.11B.

Nevermind, issue with my image not bitcoinxt.

@5an1ty Any interest in updating your docker build to 0.11E?

@dgenr8 the docker build has been updated https://github.com/5an1ty/docker-bitcoinxt/issues/1

You were way ahead of me :)


Can db.instance.id be replaced by network.peer.address ?

db.instance.id
string
An identifier (address, unique name, or any other identifier) of the database instance that is executing queries or mutations on the current connection. This is useful in cases where the database is running in a clustered environment and the instrumentation is able to record the node executing the query. The client may obtain this value in databases like MySQL using queries like select @@hostname.
mysql-e26b99z.example.com
Recommended: If different from the server.address

It seems the intention is to record specific instance host name or IP and network.peer.address should do the job.
Related: #690

Related: https://github.com/open-telemetry/semantic-conventions/pull/345
After reading the discussion on the PR, it seems server.address was considered, but not the network.peer.address.
But also, #727 and #728 show that it's not an IP address, but a user-friendly name within the cluster (FQDN or unique within a cluster).
We removed network.peer.name (aka net.peer.sock.name aka unborn proxy.address) attribute which might be useful here, but I'd prefer to keep db.instance.id|name as a unified thing and remove db.mssql.instance_name and db.elasticsearch.node.name.
@arielvalentin @AlexanderWert @estolfo - thoughts?

AFAIK, the MS SQL instance does not match your proposed description of db.instance.id. It seems to be a quite special concept, I wonder if you will find anything equivalent in any other db.

We removed network.peer.name (aka net.peer.sock.name aka unborn proxy.address) attribute which might be useful here, but I'd prefer to keep db.instance.id|name as a unified thing and remove db.mssql.instance_name and db.elasticsearch.node.name.

@lmolkova Agree, db.instance.id|name can replace the db.elasticsearch.node.name (and with a corresponding, Elasticsearch specific overwrite of thedescription of the attribute in the ES context it should be clear.)

AFAIK, the MS SQL instance does not match your proposed description of db.instance.id. It seems to be a quite special concept, I wonder if you will find anything equivalent in any other db. MS SQL instances are not members of a cluster. Depending on which "instance" you connect to, you will have different databases. This instance concept seems to allow running logically distinct DBs on the same hostname, so it's rather the opposite of clustering.
https://learn.microsoft.com/en-us/sql/database-engine/configure-windows/database-engine-instances-sql-server?view=sql-server-ver16

thanks for the explanation! If we change db.instance.id to be something like "friendly name identifying db instance within the host name (server.address, network.peer.address, etc)" it'd work, right?

Closing, as SQLServer instance id is now incorporated into db.namespace, and Elastic is using db.instance.id since #738 (though that change may now be reverted in #972)


Allow specifying the PDFJS_NEXT build flag via an environment variable when running the various gulp commands

After PR #8459, the run-time of the various gulp test commands has regressed quite badly on Windows. For me, gulp test now takes approximately twice as long when run locally on Windows.
The problem seems to be the Babel transpilation step, which takes well over five minutes to run.[1]
For someone like me, who runs tests a lot locally, this slowdown is really hurting the overall development experience.
To get around this I tested setting PDFJS_NEXT = true in gulpfile.js, since the transpilation step isn't necessary when testing in a modern browser.
However, having to edit gulpfile.js every time that I need to run tests isn't very practical. Hence this patch, which adds an environment variable that allows you to disable the transpilation simply by using e.g. PDFJS_NEXT=true gulp test.
I hope that this can be considered an acceptable solution, such that I don't need to maintain this patch locally (or worse, edit gulpfile.js locally before testing).

[1] This can also be observed on the Windows bot, but it seems fine on Linux.

Looks good. Can you split that into two lines to add var pdfjsNext = before return?

That's a lot better, fixed now, thanks!
/botio-linux lint


How to customize modal animation speed?

Description
I just upgrade to 2.0.0. I am presenting FloatingPanel modally. The dismiss animation is a lot faster than the default iOS transitions. Any way to slow it down? It is definitely a faster transition after the update to 2.0.0
Expected behavior
Present and dismiss should match default iOS behavior
Actual behavior
Present and dismiss animation have a much shorter duration than default iOS behavior.
Steps to reproduce
1: Present timerpanel modally
2: Dismiss the modal viewcontroller
Environment
Library version
2.0.0

You are able to specify presentation and dismissal animations using the following 2 delegate methods. Here is an example.
    func floatingPanel(_ fpc: FloatingPanelController, animatorForPresentingTo state: FloatingPanelState) -> UIViewPropertyAnimator {
        return UIViewPropertyAnimator(duration: 0.25, curve: .easeOut)
    }
    func floatingPanel(_ fpc: FloatingPanelController, animatorForDismissingWith velocity: CGVector) -> UIViewPropertyAnimator {
        return UIViewPropertyAnimator(duration: 0.25, curve: .easeOut)
    }

Thank you @SCENEE! That was easy. I almost feel bad that I couldn't figure this out myself. Thank you so much!


Fix #2998: Finish the new Linker API.

Rebased after the OutputMode -> ESFeatures refactoring, which somewhat changes the game.
Now standard._ only contains stuff that I would later consider relevant when building linker plugins. Even though currently, the standard API is not enough to build anything meaningful, I think everything it contains would be part of the things necessary to build something meaningful.
Maybe one reasonable thing we could do to make the current standard API practical would be to add standard.StandardFrontend and standard.StandardBackend objects with a def apply(config: StandardLinker.Config): .... That would then be enough to build a custom linker that has a front-end and/or a back-end that wraps a StandardFrontend/StandardBackend with some pre- or post-processing, which addresses all the use cases we know to exist so far.

Rebased and updated with the ClearableLinker.apply and StandardLinker{Frontend,Backend}.apply I was suggesting above.


Clash with Yet (Not) Another Map Pack - Rise and Fall

Seems that when YNAMP is active with Rise and Fall using CQUI that the Envoy system doesn't work, it only ever shows the first City state and no more and cant send Envoys.

see #283


fix missing last line when page doesn't end with a new line

Due to the behaviour of read the current implementation will drop the content of the last line of a page if it's not followed by an empty newline.
solution found (it's an interesting read besides that) at: https://stackoverflow.com/questions/12916352/shell-script-read-missing-last-line

Interesting. Thanks for the fix.


Implement Webview vulnerabilities and Configuration scans

Implement webview vulnerabilities and config API calls and tag them with corresponding CWE.

implemented in v1.1.2 #24


Upgrade to undertow 2.2.28.Final to fix HTTP/2 Rapid Reset (1.6.x)

This includes a fix for the HTTP/2 Rapid DDOS exploit. While the CVE is not yet showing up in many image scanners (such as Sonatype), we believe its prudent to update proactively since we expect image scanners to start treating this a HIGH vulnerability.
CVE-2023-44487 (Base Score: 7.5 HIGH)
HTTP/2 Rapid Reset
https://nvd.nist.gov/vuln/detail/CVE-2023-44487

Preliminary tests show no issues backwards-compatibility issues with this new version of undertow.

@miklish This has been done for all repositories in 1.6.x branches.


Fix offset of relatively positioned windows

Coordinates returned by win_screenpos start at [1, 1] whereas floating windows are positioned relative to [0, 0], so occasionally the rounding logic results in floating windows being offset by one:

floating windows are positioned relative to [0, 0]

Is it also true for regular Vim?

I just tested it and [0, 0] appears to center the window under Vim, whereas [1, 1] is the top left 🙃

whereas [1, 1] is the top left

We're already offseting that in
https://github.com/junegunn/fzf/blob/8b0e1f941a419258b0877d9a5f71373d76472043/plugin/fzf.vim#L992-L993
I have tested with a number of different configs both on Vim and Neovim
let g:fzf_layout = { 'window': { 'width': 1.0, 'height': 1.0, 'relative': 0 } }
let g:fzf_layout = { 'window': { 'width': 1.0, 'height': 1.0, 'relative': 1 } }
let g:fzf_layout = { 'window': { 'width': 1.0, 'height': 0.5, 'relative': 0 } }
let g:fzf_layout = { 'window': { 'width': 1.0, 'height': 0.5, 'relative': 1 } }
let g:fzf_layout = { 'window': { 'width': 0.5, 'height': 1.0, 'relative': 0 } }
let g:fzf_layout = { 'window': { 'width': 0.5, 'height': 1.0, 'relative': 1 } }

and this patch seems to give the most consistent results.
diff --git a/plugin/fzf.vim b/plugin/fzf.vim
index 9cebed4..073f696 100644
--- a/plugin/fzf.vim
+++ b/plugin/fzf.vim
@@ -982,12 +982,12 @@ function! s:popup(opts) abort

   " Size and position
   let width = min([max([8, a:opts.width > 1 ? a:opts.width : float2nr(columns * a:opts.width)]), columns])
-  let height = min([max([4, a:opts.height > 1 ? a:opts.height : float2nr(lines * a:opts.height)]), lines - has('nvim')])
-  let row = float2nr(yoffset * (lines - height)) + (relative ? win_screenpos(0)[0] : 0)
-  let col = float2nr(xoffset * (columns - width)) + (relative ? win_screenpos(0)[1] : 0)
+  let height = min([max([4, a:opts.height > 1 ? a:opts.height : float2nr(lines * a:opts.height)]), lines - (relative ? 0 : has('nvim'))])
+  let row = float2nr(yoffset * (lines - height)) + (relative ? win_screenpos(0)[0] - 1 : 0)
+  let col = float2nr(xoffset * (columns - width)) + (relative ? win_screenpos(0)[1] - 1 : 0)

   " Managing the differences
-  let row = min([max([0, row]), &lines - has('nvim') - height])
+  let row = min([max([0, row]), &lines - height])
   let col = min([max([0, col]), &columns - width])
   let row += !has('nvim')
   let col += !has('nvim')

Actually, I'm not yet perfectly sure. Need more tests with different values of xoffset and yoffset as well.

Fixed on master. Please let me know if you still have the problem.


refactor(runtime-vapor): setStyle reuse from runtime-dom

Since runtime-vapor directly depends on runtime-dom, we can export these directly from runtime-dom instead of moving them to runtime-shared.

it should be an error caused by negligence.

My bad, I was looking at compiler-vapor, not runtime-vapor. Discussed with @edison1105 yesterday and I think we need a bit of restructure:

The @vue/vapor package is unnecessary. vue should directly depend on @vue/runtime-vapor (content of current @vue/vapor should be moved to vue/vapor).

The @vue/runtime-shared package is unnecessary. @vue/runtime-vapor should list @vue/runtime-dom as a peer dep so it can import shared utils from runtime-dom directly.

Similarly, @vue/compiler-vapor should rely on @vue/compiler-dom as a peer dep instead of direct dep.


Button state based on play state

With version 2.3.x Macro Deck introduced the ActionButton field in the PluginAction class which allows direct access to the actions button. This allows the easy change of the button state with MacroDeck.Server.MacroDeckServer.SetState(ActionButton, true : false);
Users should have a checkbox to enable "Sync play state with button state" in the action configuration and if enabled, the action should automatically change the button state based on the play state. This would allow the user to set a static icon as the "off" state icon and a animated icon for the "on" state.

I can implement this option to allow disabling sync, but currently this is the default (ie. button state changes with audio state)

fixed with other development


Unlinking Events

This doesn't quite work yet. There's a bug or two to work out. But you can check out the interface and play around with it.


BWMatrix ignore  SetBrightness setting

in example
port, err := serial.OpenPort(&serial.Config{Name: "COM3", Baud: 115200})

if err != nil {
    log.Fatal(err)
}

mat := fwmatrix.NewBWWithPort(port)

fwmatrix.SetBrightness(port, 100)

fwmatrix.DrawRect(mat, 0, 0, 5, 5)

if err = mat.Flush(); err != nil {
    log.Fatal(err)
}

wmatrix.SetBrightness(port, 10) looks same as wmatrix.SetBrightness(port, 100)

This was caused by a silly copy-paste error the library. SetBrightness should work correctly now. Please update to v1.0.2 to get the new fix.


cargo rustc --message-format=json fails to parse stderr of rustc

Trying the following on a small test crate that contains one error:
$ cargo rustc --message-format=json --lib -- -Z no-trans
   Compiling test-crate v0.1.0 (file://<snip>/test-crate)
error: Could not compile `test-crate`.

To learn more, run the command again with --verbose.

I get no error message.  Without -Z no-trans, I do get an error message:
$ cargo rustc --message-format=json --lib
   Compiling test-crate v0.1.0 (file://<snip>/test-crate)
{"reason":"compiler-message",<snip>,"message":{"children":[],"code":{"code":"E0499","explanation":"\nA variable was borrowed as mutable more than once. [...]

This is the expected output.
If I pass --verbose, we can see the error message is output by rustc:
$ cargo rustc --verbose --message-format=json --lib -- -Z no-trans
   Compiling test-crate v0.1.0 (file:<snip>/test-crate)
     Running `rustc src/lib.rs --error-format json --crate-name test_crate --crate-type lib -g -Z no-trans -C metadata=f713ecd4594c2f7f <snip>`
error: Could not compile `test-crate`.

Caused by:
  process didn't exit successfully: `rustc src/lib.rs --error-format json --crate-name test_crate --crate-type lib -g -Z no-trans -C metadata=f713ecd4594c2f7f <snip>` (exit code: 101)
--- stderr
warning: the option `Z` is unstable and should only be used on the nightly compiler, but it is currently accepted for backwards compatibility; this will soon change, see issue #31847 for more details

{"message":"cannot borrow `a` as mutable more than once at a time",<snip>}

But it's not picked up and reported by cargo.
Testing on a crate with no build errors, the causes are reported slightly differently:
$ cargo rustc --verbose --message-format=json -- -Z no-trans
   Compiling good-crate v0.1.0 (file://<snip>/good-crate)
     Running `rustc src/main.rs --error-format json [...]`
error: Could not compile `good-crate`.

Caused by:
  failed to parse process output: `rustc src/main.rs --error-format json [...]` (exit code: 0)
--- stderr
warning: the option `Z` is unstable and should only be used on the nightly compiler, but it is currently accepted for backwards compatibility; this will soon change, see issue #31847 for more details

Caused by:
  compiler produced invalid json: `warning: the option `Z` is unstable and should only be used on the nightly compiler, but it is currently accepted for backwards compatibility; this will soon change, see issue #31847 for more details`

So it seems cargo --message-format parses the JSON of rustc, but does not expect rustc to output anything else than JSON.
$ cargo --version
cargo 0.14.0 (eca9e15 2016-11-01)
$ rustc --version
rustc 1.13.0

I know I'm using an unstable flag on stable rust, so I should heed the warning and all would be fine.  But I believe the behavior of --message-format may still be viewed as a bug, since it's not choking on non-JSON output from rustc.

I went source diving, and found the place where this parsing happens.
Adding a check to look only at lines beginning with "{" seems to do the trick, but I don't know if it's a valid long-term solution.
Here's the patched closure:
                &mut |line| {
+                  if line.starts_with("{") {
                    let compiler_message = json::Json::from_str(line).map_err(|_| {
                        internal(&format!("compiler produced invalid json: `{}`", line))
                    })?;

                    machine_message::emit(machine_message::FromCompiler {
                        package_id: &package_id,
                        target: &target,
                        message: compiler_message,
                    });
+                  }
                    Ok(())
                },

cc @matklad, mind taking a look at this?

So I think we have three options:

covert "warning: -Z ..." to JSON on the compiler's side.

change the compiler to emit JSON to the stderr, the way Cargo does.

use if !line.starts_with("{") { return }.

Regardless of a long term solution, we should implement 3 to fix the problem faster and to be compatible with all compilers. @fmdkdd can you send a pull request? :)
Option 1) does not seem right to me, because it is not an warning about the Rust code, it is a warning specific to one particular Rust compiler.
Option 2) I think is the Right Thing. Is it worth implementing? Perhaps. On the one hand, backward compatibility should not be a huge problem: I think there are only a few tools which parse compiler output directly (Cargo being a notable example). On the other hand, the gain from the change seems marginal, so we can just as well stick with option 3.

Ah, and for 3) we probably want to forward stderr if it's not JSON, like
if line.starts_with("{") {
  // handle JSON here
} else {
  writeln!(cx.config.shell().err(), "{}", line)?;
}

Yeah (3) sounds good to me for now and I agree that (2) is the "best" way to go. I don't mind "hacks" in cargo though in the meantime because backwards compatibility may mean that we can't fix that :(.
Thanks for looking into it @matklad!

Can someone please explain what is the possible solution/workaround with this error?
I just tried to run a compilation of 3rd party dependency and the error brought me here.
BTW I already updated the toolset to the latest but it didn't help.

@raffaeler this is a 8-year-old issue. Please open a new issue with a reproducible example and the toolchain version, so that we can investigate and discuss.

@weihanglo I understand that, but I have no clue of the error or even how to repro it.
I just slapped into this error and I see that many other people is hitting it: https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/16445.
The error message is hard to understand so I bounced here. That's all.

Understood. That issue is completely unrelated to this issue and Cargo. That one is a real rustc compile error and needs to be addressed by the tokenizers crate or the wrapper building it.


Implement validation tools like ShEx or SHACL (or both?)

It would be nice to have built-in support for shape validations in Oxigraph. It might be done by implementing ShEx or SHACL, or maybe even both.

What's the planned approach here, doing it via SPARQL in the background? I've tested several SHACL implementations and it looks like most of them do some in-memory validation, which might or might not work for large datasets.
From what I heard Stardog is translating shapes to SPARQL and validates them like this. This has the benefit that it can use indexes that are there anyway already.

I have no real plan at the moment. My current goal is pushing SPARQL update and implementing full text search, RDF* and query optimization. ShEX/SHACL is more something for the distant future.
Using SPARQL (or at least a part of the SPARQL evaluator) is indeed probably the best way to do validation.

Ok I think that makes sense. To attract more people I think it's important to have a usable store that people can work with in production and then slowly start improving query planer/performance.
This will hopefully attract more users, contributors & money so other features like SHACL can be added.

Shex would be so cool! As an aside, do you know if there are any shex rust libraries? (I didn't find any after a quick look)

As an aside, do you know if there are any shex rust libraries?

Not at my knowledge, sadly.

As an aside, do you know if there are any shex rust libraries?

Not at my knowledge, sadly.

I got to know someone working on a Rust implementation of ShEx, you can find the Github repository here.


fix bug on click header

resolve https://github.com/OpenFunction/website/issues/59

/assign @benjaminhuo

@Arhell Hope you're safe in Ukraine. Thanks for the contribution.
Would you please give more details on this PR such as the behaviors before and after this fix?

@Arhell Hope you're safe in Ukraine. Thanks for the contribution. Would you please give more details on this PR such as the behaviors before and after this fix?

Hi @benjaminhuo, it is a background transparent problem, the original code makes the click button's text color incorrect.

/lgtm

@Arhell Hope you're safe in Ukraine. Thanks for the contribution. Would you please give more details on this PR such as the behaviors before and after this fix?

Hi @benjaminhuo, it is a background transparent problem, the original code makes the click button's text color incorrect.

/lgtm

@Arhell Hope you're safe in Ukraine. Thanks for the contribution. Would you please give more details on this PR such as the behaviors before and after this fix?

Hi @benjaminhuo, it is a background transparent problem, the original code makes the click button's text color incorrect.

@weili520 Thanks， Got it. The click button's color shouldn't change with the background


DOC: interpolate: deduplicate docstrings in _fitpack_py and _fitpack_impl

What does this implement/fix?

Functions in _fitpack_py are nothing but thin wrappers which dispatch to either BSpline methods or _fitpack_impl FITPACK Fortran wrappers.  Previously, the docstrings were duplicated, and were slightly diverging over time.
So merge the docstrings and deduplicate: move all docstrings to _fitpack_impl/* and attach to _fitpack_py/*.

Close/reopen to rerun tests

Hmmmm. This is baffling, I've to admit. The manipulation basically happens on import, so should be safe. Anyway.
Do you know how work make it easier for these IDEs? This certainly is not the only place in scipy we construct docstrings at import time, e.g. in stats.distributions. Does your IDE follow what's done over there?

IDEs do the analysis on the source code. If you import something then they go and fetch the imported func docstring. So this is an annoyance only if you are directly working on the SciPy codebase itself with bare functions. Otherwise users will see the full docstring.

I am not sure about that. E.g. with stats distributions, it does not render more than what's in the actual docstrings:

This is with PyCharm, maybe some other IDEs behave differently.

This LGTM once CI is green(er). The IDE issue is an unrelated generic problem and shouldn't block merging this - best to move that to a separate follow-up issue.

I am not blocking the PR. Just pointing out that this has consequences for IDE users.

Probably not much that can be done about that within SciPy though, so it's probably a wontfix.

I don't think we have made a public decision about going one route or the other. Not using any overwrite for __doc__ would solve the issue (and other issues we have with our own machinery to do so.)

Just pointing out that this has consequences for IDE users.

This is unfortunate indeed. I guess I wouldn't be surprised if it's a wontfix for pycharm as well, importing stuff online is a big can of worms.
So, to proceed here there are three options:

keep the status quo. Some 400 lines of duplicated docstrings. Duplicating them was a bad decision (quite a while ago).
keep this PR as is
move the docstrings to the functions which end up in the public namespace, i.e. _fitpack_py.py + add code comments to point to where the docstrings are. This is IMO slightly worse than the current state of the PR but is likely OK.

My preference would be option 3 (moving the doc to the public facing API) as to me private functions can then have more maintainers' oriented docs.


Copilot not working on Windows 11 Preview

What happened?
Copilot will not open if GoAwayEdge is installed, tried to use hotkey and manually opening Copilot but nothing happens.
Channel
Stable (default)
Search Engine
Google
If you selected “Others”, what is the request URL?
No response
How to reproduce
Windows 11 Preview Build 26241
Edge Version 127.0.2651.31
Additional information
2024-07-03 21:50:07 - INFO - GoAwayEdge 1.3.3 (Build 150)
2024-07-03 21:50:07 - INFO - Logging system initialized
2024-07-03 21:50:07 - INFO - Trying to load language: en-US
2024-07-03 21:50:12 - INFO - Start installation ...
2024-07-03 21:50:12 - ERROR - Failed to delete old task: System.IO.FileNotFoundException: The system cannot find the file specified. (0x80070002)
at Microsoft.Win32.TaskScheduler.V2Interop.ITaskFolder.DeleteTask(String Name, Int32 flags)
at Microsoft.Win32.TaskScheduler.TaskFolder.DeleteTask(String name, Boolean exceptionOnNotExists)
at GoAwayEdge.Common.InstallRoutine.Install(Object sender, DoWorkEventArgs e)
2024-07-03 21:50:12 - INFO - Installation finished.

Sorry for the late response. I could not reproduce the problem in the latest Insider Preview. It is possible that the alternative startup file is outdated / corrupted.
Try the following:

Open Command Prompt (not as administrator) and type: "C:\Program Files\valnoxy\GoAwayEdge\GoAwayEdge.exe" --update
If the alternative start file is outdated or corrupt, a window will appear asking you to update it. Confirm this with "Yes".

Should it still not work, delete the following file and repeat the above steps: C:\Program Files (x86)\Microsoft\Edge\Application\msedge_non_ifeo.exe

No problem, I meant to get back to you sooner too. Unfortunately the steps you shared didn't work for me. The command by itself didn't do anything and after deleting the exe and running the command again, the prompt to confirm yes appeared but still no change in CoPilot working. Here's a screen recording of the steps I tried.
GoAwayEdge Issue.webm

Ah, I see the issue. You are using the PWA app from Copilot. I had fixed the bug in the dev branch the other day. Here is a current build:
GoAwayEdge_v<IP_ADDRESS>_14d0d8704441b4c66d99fbec2f6ee75b49e96496_signed.zip

Works like a charm! Thank you, appreciate the help:


Fix defining class problem

The "Defining class problem" is actually not related, a cache of the currently defined class name is kept because, if I understand it correctly, BeanShell had an issue creating two clåsses with the same name from different packages. I think because the class generation was aborted our class was never removed from the currently defined list. #700
This is the code:
https://github.com/beanshell/beanshell/blob/e2477aa6d06a5b3c66472151581825ce9e1d3154/src/main/java/bsh/BshClassManager.java#L632-L655
Commenting out the exception allows us to recreate the offending class after the failed generation. Not entirely sure what the comment refers to about "...namespace imports in an analogous..." but I don't see any problem with generating two classes of the same name from different packages with the exception commented.
package a;
--> void
class A { };
--> $2 = class a.A :Class
a.A.class;
--> $3 = class a.A :Class
package b;
--> void
class A { };
--> $4 = class b.A :Class
a.A.class;
--> $5 = class a.A :Class
b.A.class;
--> $6 = class b.A :Class

Will investigate more maybe this is not the exact issue that is being prevented but if this is really not an issue anymore we should remove the definingClasses cache as a whole and not just remove the exception, as throwing this exception is the whole purpose of this cache and would otherwise have no purpose.

The only issue I found with removing the defining classes workaround/hack was a block in BSHType, which is already commented for change:
https://github.com/beanshell/beanshell/blob/e2477aa6d06a5b3c66472151581825ce9e1d3154/src/main/java/bsh/BSHType.java#L94-L113
After fixing that to use the namespace import values for inner classes instead, all tests pass and no further issues were detected.


Add mandatory file ignores when using --ignore flag

Signed-off-by: Maysun J Faisal<EMAIL_ADDRESS>What type of PR is this?
/kind bug
What does does this PR do / why we need it:

previously when we used --ignore, .odoignore/.gitignore was overwritten. Hence if odo index file .odo/odo-file-index.json changed due to a file change, it got stuck in an infinite loop
Similarly, if the local project is a git project, it would build more than once for .git changes
This PR adds .odo/odo-file-index.json & .git to the list of ignored files when --ignores is used

Which issue(s) this PR fixes:
Fixes #3819
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[ ] Documentation

[x] I have read the test guidelines

How to test changes / Special notes to the reviewer:
odo watch --ignore some.txt and create a file in the component directory. It should only build once.

@mik-dass @johnmcollier Could you pls review this PR since you both suggested the solution in the issue? Thx!

/approve


Python Interactive not Printing Correctly

Environment data

VS Code version: 1.30.1
Extension version (available under the Extensions sidebar): 2018.12.1
OS and version: MacOS Sierra
Python version (& distribution if applicable, e.g. Anaconda): Anaconda 3.6.7
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Relevant/affected Python packages and their versions: requests=2.21, IPython 7.2.0, ipykernel=5.1.0, jupyter

Expected behaviour
I should be able to print a requests object.
Actual behaviour
Nothing shows up!
I attached a screenshot. My terminal at the bottom printed correctly, as a result of clicking to run the selection in Python Terminal.
Steps to reproduce:

make a request
Print it

Logs
none

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
none

I believe the problem is we're treating the returned data as html directly instead of wrapping it in a span. So the <Response [200]> is treated like an html tag.

@rchiodo that makes sense to me. are you able to change the behavior to wrap output in span, and see if that resolves the issue?
Seems relevant to printing objects generally, not just requests objects. E.g. printing a package is wrapped in tags... <module 'package' from 'path/here.py'>
Seems like a straightforwards fix to make for the next release?

Unfortunately it's not as simple as just sticking a span around everything. It all depends upon the type of stream data. We have to put spans just around text data. It should be fixed soon though.


[5.4] add next and previous methods to collection

I have a page where managers of a team can view a list of the users on their team. Each user has a corresponding button to edit or delete them.  If I edit one of the users, and then need to make similar changes on all the other users, I must first go back to the list view, and then click into the new user.   I'd like an quicker way to navigate to the next or previous user in the list.   I cannot simply add or subtract 1 from the user ID because they may not be sequential.

This PR adds to new methods to the Collection that allow you to search for an item in a Collection and then return the previous or next item in the Collection.

$users = collect(['Andy', 'Taylor', 'Mohamed']);

$users->next('Taylor'); --> returns Mohamed

$users->previous('Taylor') --> returns Andy

In order to account for associative and numeric keys, I create a numerically indexed array of the keys of the items.  This allows me to simply add or subtract 1 to find the next or previous key that I should use on the items.

Both methods also have checks to make sure you have not reached the beginning or end of the array, and both check to make sure your search result returns something.  If either of these fail, the methods will return null.

We simply pass through to the search method, so you can use either a value or a callback to search, and you may also pass the strict flag if you'd like.

I would prefer the naming before and after, these are the words used in the docblocks to describe the functionality.
The type-hints for $value should be mixed, not string.
if (! $key = $this->search($value, $strict)) { will fail when the value has index 0 or an empty string as key.
The calls to array_search is missing the strict parameter causing issues with arrays that has both a index 0 and an empty string as a key.
The functions return null when there is no match, and when the matched value is null. Should the caller be able to differentiate between the two scenarios?

Hmm, will let you macro these in for now.


THRIFT-5706: lib/cpp Fix the Security tests on openssl 1.1 and 3.0

This PR fixes the Security tests to build on a clean install of ubuntu
20.04 and ubuntu 22.04 without modifications to the systems openssl
configuration.
* Enable TLS 1.0 and TLS 1.1 on OpenSSL 1.1 with the seclevel=0 flag
* Disable TLS 1.0 and TLS 1.1 on OpenSSL 3.0

While its technically possible to enable it on OpenSSL 3 I think because
of all the issues with these old TLS versions dropping support for it is
better.
This PR builds forth on the work done here: https://github.com/apache/thrift/pull/2811
Tested with the ubuntu 20.04 (OpenSSL 1.1) and 22.04 (OpenSSL 3.0) docker containers.
All lib/cpp tests succeed in both.

@emmenlau any update on the review of this?
cpp test suite is still broken without this patch on modern systems.

Sorry for the long delay! Looks good to me, lets bring this in!


Generator writes correct plugin paths

Thanks for this project! Super useful.

Oh I missed that one, thank you!


Add support for PHP 8, and drop support for PHP <7.2

Closes #7

I'm not sure why scrutinizer failed. The error doesn't give too much information 🤔

Looks like scrutinizer worked after you re-run it 🙂

Worked after retry. 🤷🏻♂


Your IPTV

Addon Manifest URL
https://5a0d1888fa64-your-iptv.baby-beamup.club/manifest.json
Addon Description
This addon brings all the Live Streams, VOD streams and Series from your IPTV subscription to your Stremio using Xtream API.

useful addition to stremio

@gorlev does this addon need a "M3U Playlist URL"? I think so, and "IPTV URL" is very confusing if that is the case, the addon description is very confusing too

@gorlev does this addon need a "M3U Playlist URL"? I think so, and "IPTV URL" is very confusing if that is the case, the addon description is very confusing too

Yes. It actually parses the username and password from the M3U playlist URL and make requests to the XtreamAPI with the credentials. By this way it can show all the series, VOD and LiveTV streams. I need to update the description and maybe improve the configuration page. I will work on this when I have time.

Any IPTV for free? Can anyone share?

I dont have a subscription and just want to add these FREE channels that are available on my country (so that I just have stremio and my parents do not need to go anywhere else in order to watch free local TV for news etc)
this is the m3u url https://lists.iptvgenerate.com/mylist/0d2b338c-2cdf-4414-9a39-2ee42c9a71e6.m3u it provides the file with the IPs/links for each channel etc..
But I cant use that because they are not a subscription so they dont have username and password... can't you make it so that free stuff without username and pass can be added to stremio as well?

I paid for iptv but it doesn't work

I set up the add on fine 2days ago, but unable to install since last night. Any idea when the install process can be restored. Looks like the add on is offline right now. Thanks.

All working now. Thank you for the great work.

I get failed to fetch error, from either installing from the addon or from the json link

not working for me either, can't install the addon

A algum tempo não consigo instalar o addon, sempre aparece um erro "ERR_OPENING_MEDIA" "Failed to fetch"
Alguém sabe do que se trata? Alguma ideia de como corrigir?

Olá também não estou conseguindo adicionar a lista paga ele da erro e não carrega se puder arrumar o app agradeço.

Doesn't work with my Xtream login error 504 could not fetch, Try something else.

How to I install this addon? Tried it several times and failed. Is there somewhere this addon is hosted so that I can download?

Addon is dead.

@gorlev what can I do to fix this addon? I can fix this if you don't have much time to do it.


some confusions in reproduce the bevformer-s

Hi zhiqi,
I am so instresting in the excellect job, eg bevformer. First step i want to reproduce the bevformer-static(means only use bev encoder) in your paper, but there are some confusions in the process. The model cfg is backbone: Resnet101(pretrained by Fcos3D), neck: fpn( with sizes of 1/16, 1/32,1/64), bev encoder: 6 * transformer(deformable self-atten in bev query and deformable cross-atten with 6 * camera features), detect head: 6 * transofrmer(deformable detr with 900 object query). All embed dim in attention is 256 and channel size in ffn is 512 by default.
Fristly, in your paper i found that the memory is around 20G（bev anchor size：200 * 200 * 4）, but in my implementation the memory is 32G（when bev anchor size：100 * 100 * 4）. Is there some problems with my configuration?
Second, in my implementation the speed of convergence the bevformer-static(bev anchor size：100 * 100 * 4) is more slower than detr3d with the same loss（l1 and focal loss）, and the final result mAP in nuscenes val set is only around 31.1.
In the meanwhile i have check the valid bev anchor and bev anchor transformation point in multi-view images，the pink points in bev figure means front view, we only visualize the points are valid in more than one image to check the bev anchor transformation. It seems that there are no obvious problems in bev anchor transformation. Can you help me to reproduce the result in your paper？

Thank you so much!

您的邮件我已收到，请您放心，我会尽快给您回复。

您的邮件我已收到，请您放心，我会尽快给您回复。


Remove single quotes for filtered values in context menu

System information:

DBeaver version 21.1.0

Describe the problem you're observing:
When we select a value to filter via context menu, it is surrounded by single quotes. We have to remove them.

We can't do that.


Use a better nav_drawer image when the app is offline

Fix for #285
When the app is offline, the nav_header image is set as twitter logo.
I modified it to use the fossasia logo from the splash screen.
@mananwason @championswimmer

@the-dagger I didn't want to increase the apk size by adding more resources. Plus the main aim of the system is to be Event independent and if we add such logos, then it defeats the purpose.

@mananwason Understood!
Should we set the default image to be a solid material color?

@the-dagger I would prefer just removing the twitter image and let it be same as the drawer color. If we set any particular material color, then also it amy be different from what the conference theme is.

Aight then. (y)


[Support] Does Python support call frequency, traffic limiting, and sampling rate?

pip install pinpointPy
Does this support call frequency, traffic limiting, and sampling rate?
I can see the related configuration in the agent of Java, such as profiler.sampling.percent.sampling-rate and profiler.sampling.counting.sampling-rate.
https://github.com/pinpoint-apm/pinpoint/blob/master/agent-module/agent/src/main/resources/profiles/local/pinpoint.config#L66C1-L66C43
What do we do in python if we want to set the relevant configuration?

for pinpointpy, not support profiler.sampling.percent.sampling-rate
as for profiler.sampling.counting.sampling-rate
Try trace_limit: counting per second.
https://github.com/pinpoint-apm/pinpoint-c-agent/blob/c35232017d1f2eb79e65373fda556638c7cf7d09/plugins/PY/pinpointPy/pinpoint.py#L122
BTW, trace_limit is process scope variable.

What does time_out mean? What does time_out: int = 0 mean?
@eeliu

time_out ： max time out when sending a span.
If your script is run and exit, like db backup, crontab job, set time_out > 0 (1), or else  set time_out = 0.


Collection edit UI

we will have a new ui for editing/customizing collection for owners - mainly because introducing big banner on collection detail
Figma

collection detail handoff
https://www.figma.com/file/7BWdGiowrvTucpSFLE0kRX/Collection-detail-handoff?node-id=0%3A1&t=cKi2EVd3pPHjX4n4-1

Main

it's quite similar to the sidebar for wallet connect - its only sharing the style but position is in the middle of a screen as popup
Collection owner can get there once he click through more options and then select "edit collection"

Icon

Owner can choose to change collection logo/icon - the current one would be displayed in solid stroke

if there is not anything filled, it looks like this:

Banner

you can drop your files into the field or select file.

here are the states for hovering over or dragging file over it

question for you, should we include supported types? If yes, what are they for banner? the main ones png, jpg etc

Social links

option for the owner to link certain links for the collection
links that are filled out will be shown on the page among social links buttons - left side
active state for the input is in the figma as well - its the same as with price input on explore

Save changes

primary button - hover effect to bg color

I think we could maybe expand the options in future - something like name edit etc, but for now I think it works Thank you!

Wait.. I'm pretty sure it's not possible to edit/customize already minted collection.
Once it's minted data is "frozen" on chain. (from what I understand on Bsx and Rmrkv1)

@roiLeo ah okay, and is it possible code it or is it just one of the specifications of those networks?
I think that for the social links we can store it off chain no? Or at least that is how opensea does it no?
Correct me if I am wrong
Thanks!

@roiLeo ah okay, and is it possible code it or is it just one of the specifications of those networks?

networks specification that don't enable these features but I may be wrong I've seen it's possible to update some metadata part on eth.

I think that for the social links we can store it off chain no? Or at least that is how opensea does it no?

Hmm.. alright why not but it has a cost, we will need to maintain external database (off chain)
I don't really like this solution because for me it goes against the chain and data will not persist as before.
I'm still open to discussion if there is really a gain in displaying this data.

yes, understand. The banner on the collection that has different size than profile pic is why I did this customize ui. And also the social links - but not sure how to store it tho if its not possible to have it onchain for each collection - maybe @vikiival would know?

Can push it to Supabase imo :D or mint is as metadata and push it to chain via system.remark

But First we should make ability to upload banner on the collection

Logo we could put coming soon as it's currently immutable, yet in future people could change it for reason
For banner, there is two options, on-chain, mint it while at creating collection, set it into stone, yet if someone want to mint 1k collection, it will be 1001. Off-chain, it's probably best if we store one more pic on r2, yet we don't know for example if pictures are still used after change (or we will find good way to figure out)
I would add instagram field for social links

store one more pic on r2

You still need some way how to make refference for the picture 😇

updated

You still need some way how to make refference for the picture 😇

yeah, I was thinking for some off-chain config.json for collections till we will be putting it on-chain

Time for some flex.
Would be nice if we can have single service for fandoms as well if possible, wyt?

Time for some flex. Would be nice if we can have single service for fandoms as well if possible, wyt?

we should try to keep it on chain, maybe in Identity pallet

we should try to keep it on chain

what's pro being on-chain for this?
Identity pallet has lot of limitations iirc.

what's pro being on-chain for this?

can be reused across plateform (singular, fandom, kodadot etc..)
can be fetched/updated through rpc calls
new project can build on top of that

Identity pallet has lot of limitations iirc.

Yes, unfortunately.
Imo we should first focus on #2329, and try to implement the rest something like an external db (but it would need secret keys, limit api usage...)

can be reused across plateform (singular, fandom, kodadot etc..)

if it will be HTTP endpoint, how it differs?

try to implement the rest something like an external db (but it would need secret keys, limit api usage...)

I prefer off-chain first, quick iterations and then we can figure out on-chain, don't want to repeat dumb mistakes ala carte RMRK team 😎

Are any ui changes needed?

CORS, domain validation, server maintenance

API keys?

Rmrk v1 and v2 wont allow this.
Basilisk neither.
On-chain metadata mutation works only for Statemine

Rmrk v2 wont allow this.

ain't it the goal of "Lego Nft"? or "multi-asset NFT"?
Or another way, would it be possible to update ipfs metadata file?

would it be possible to update ipfs metadata file?

Except Statemine it is not possible to update collection metadata

You can't edit a file once it's on IPFS.

Okay that what I was looking from ipfs discuss


Refactored quote.py. Individual quotes come from sqlite, rather than all results

The changes make the quote plugin select a single row when selecting a quote, rather than all of the possible results. Two queries are made rather than one (using a COUNT(*) first, then a SELECT with a LIMIT).
Also refactored the code to make it easier to add functionality in the future, such as a .q delete.

 Need to fix an error message.


[RFC]: add support for computing the complementary error function for single-precision floating-point numbers

Description
This RFC proposes adding support for computing the complementary error function for single-precision floating-point numbers.
The package is the single-precision analogue to @stdlib/math/base/special/erfc. Similar to that package, we can base our implementation on msun. The implementations for both JavaScript and C should follow the reference implementation.
For the JavaScript implementation, we'll need to make sure that we emulate float32 arithmetic by wrapping operation results with float64ToFloat32. This will mean noticeable performance degradation in JavaScript, but will ensure that the JavaScript and C implementations produce the same results.
Package: @stdlib/math/base/special/erfcf
Alias: erfcf
Related Issues
None.
Questions
No.
Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

Hi @kgryte,
Could you please assign this issue to me? I'd love to work on adding support for computing the complementary error function for single-precision floating-point numbers in the @stdlib/math/base/special/erfcf package.


Transformation spec updates to match latest BIDS model

Several updates to the definitions of transformations are necessary:

inputs should be input
Some names change changes. E.g. scale instead of standardize. Here is a list: https://github.com/INCF/pybids/blob/transformer/bids/events/transform/compute.py
Restrict transformations that create new names that are only knowable if you look at the data. E.g. "split", because you need to know the values of the variable you are splitting.

Lower priority:
4) Ideally, it would output in BIDS model spec (e.g. withut the 'parameters' field). Probably fine to hold off on this last one until BIDS model is finalized.

OK I'm going to restrict this to only bugfix #1 and #2, but leave comprehensive coverage of transformations to another issue, as it requires new frontend interfaces.


Consider adding some zero checks

Disclaimer: I'm no Mathematician; Just a random programmer that stumbled on this repo while looking for an EllipticE implementation to use with Ellipses.
I noticed that if the m is passed into EllipticE is 1, as in the case when solving for the Perimeter of an Ellipse with a SemiMinor axis of 0 (or very close to it), the function can fail to converge in time despite the result being trivial.
In my implementation I added a check for the same epsilon you used to verify an-gn in the while loop:
...
var kprime = Math.sqrt(1. - m);
if(kprime < 1e-15)
    return 1;
...

I'm not sure if any similar checks could be added to avoid unnecessarily iterations in cases where we know what the answer should be, or if my approach here was even valid, but it resolved an edge case for me at least.

Thanks for the interest, @JohannesMP. If you create a pull request, I'll happily review it!


Create MARRAYS.cpp

Closing Issue/Referencing Issue
issue reference (#)
Changes Made In Pull Request

Problem: https://www.codechef.com/OCT17/problems/MARRAYS/


OTP matching

I am using the library on atmega controller and the otp generated on my controller is not matching with the code generated online, I am using same Secret key on both online simulator and on my MCU.
I am using https://www.nayuki.io/page/time-based-one-time-password-tools
link to check the OTP where we can pass the time code manually also.
Can you confirm whether this library is supported as per the link which also says it is developed as per RFC 6238

Hi. As i tested, It works fine for me.
https://www.nayuki.io/page/time-based-one-time-password-tools

Secret key: JV4UYZLHN5CG633S // Convert text "MyLegoDoor" to base32
Code length: 6
Timestamp: Uncheck "Current time" and input<PHONE_NUMBER>

Tested with my example in Readme


improve: add methods to delete and disable caches

add methods to delete and disable the cache, related to the issue #99

I signed it!


Clap feature tweaking

(Context: sniffing out lingering sources of old clap version downloads)
inferno and names are both bin+lib crates that both use a default feature for the CLI related dependencies. Currently there are still some dependencies that are unconditionally pulling in clap for binary-related things even when using it as a library. I've already opened issues on both of the ones I noticed

https://github.com/benfred/py-spy/issues/647
https://github.com/rbspy/rbspy/issues/384

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

I didn't notice that this would require signing a CLA, but I'd rather not. Apologies for the noise


'attributes' is undefined on object

I have Cerebro running for AWS ElasticSearch clusters via aws-signing-proxy (https://github.com/cllunsford/aws-signing-proxy). Today I started getting errors when viewing the overview and nodes pages (the REST client seems to still work, as well as functionality like index templates). In the container logs, I have this error:
[error] application - Error processing request [path: /nodes, body: {"host":"AWS ES development"}] play.api.libs.json.JsResultException: JsResultException(errors:List((,List(JsonValidationError(List('attributes' is undefined on object: {"name":"UmewAJY","version":"6.8.0","build_flavor":"oss","build_type":"tar","build_hash":"dadef69","roles":["master"],"os":{"refresh_interval":"1s","refresh_interval_in_millis":1000,"available_processors":2,"allocated_processors":2},"jvm":{"pid":11308,"start_time":"2020-05-13T20:52:10.539Z","start_time_in_millis":1589403130539,"mem":{"heap_init":"5gb","heap_init_in_bytes":5368709120,"heap_max":"4.9gb","heap_max_in_bytes":5351276544,"non_heap_init":"2.4mb","non_heap_init_in_bytes":2555904,"non_heap_max":"0b","non_heap_max_in_bytes":0,"direct_max":"4.9gb","direct_max_in_bytes":5351276544},"using_compressed_ordinary_object_pointers":"true"}}),WrappedArray()))))) 	at play.api.libs.json.JsReadable.$anonfun$as$2(JsReadable.scala:25) 	at play.api.libs.json.JsError.fold(JsResult.scala:66) 	at play.api.libs.json.JsReadable.as(JsReadable.scala:24) 	at play.api.libs.json.JsReadable.as$(JsReadable.scala:23) 	at play.api.libs.json.JsUndefined.as(JsLookup.scala:193) [error] application - Error processing request [path: /overview, body: {"host":"AWS ES development"}] play.api.libs.json.JsResultException: JsResultException(errors:List((,List(JsonValidationError(List('attributes' is undefined on object: {"name":"BL57Tf5","version":"6.8.0","build_flavor":"oss","build_type":"tar","build_hash":"dadef69","roles":["master"],"os":{"refresh_interval":"1s","refresh_interval_in_millis":1000,"available_processors":2,"allocated_processors":2},"jvm":{"pid":11441,"start_time":"2020-05-13T20:52:13.535Z","start_time_in_millis":1589403133535,"mem":{"heap_init":"5gb","heap_init_in_bytes":5368709120,"heap_max":"4.9gb","heap_max_in_bytes":5351276544,"non_heap_init":"2.4mb","non_heap_init_in_bytes":2555904,"non_heap_max":"0b","non_heap_max_in_bytes":0,"direct_max":"4.9gb","direct_max_in_bytes":5351276544},"using_compressed_ordinary_object_pointers":"true"}}),WrappedArray()))))) 	at play.api.libs.json.JsReadable.$anonfun$as$2(JsReadable.scala:25) 	at play.api.libs.json.JsError.fold(JsResult.scala:66) 	at play.api.libs.json.JsReadable.as(JsReadable.scala:24) 	at play.api.libs.json.JsReadable.as$(JsReadable.scala:23) 	at play.api.libs.json.JsUndefined.as(JsLookup.scala:193)

This is not specific to AWS, I get the same error on a local cluster after upgrading Cerebro to 0.9.1. Version 0.8.3 works fine on the same cluster.

Dropping the version to 0.8.3 fixed it for me too.
What API calls are made for the overview and nodes pages? Happy to post the output from my AWS clusters for debugging purposes.

I've got the same bug I think: my cerebro setup worked fine on v0.9.0 but when I upgraded to v0.9.1 this "attributes is undefined" error message started. I am not using AWS however, I am on elasticsearch v5.6.16

Same here - reverting to 0.9.0

Hi,
at version 0.9.1 i got the error message "Error while loading nodes data".

When i use version 0.9.0 i dont have such error messages and everythind works fine.

So i think there is a problem in version 0.9.1.

https://github.com/lmenezes/cerebro/commit/1af8da1142f5b6cc98d4ef9bf057c0341fbc6540#diff-560d6b2470c110252b90f947d1d8a53f introduced attributes fields but sounds like this parameter does not exists.
We have similar error on /overview endpoint since 0.9.1 version, we tested on:

Elasticsearch 6.2.3 on AWS cloud
Elasticsearch 7.6.2 on Elastico cloud


Screen cheese in terminal with Copilot Placeholder

macOS, VS Code Insiders, zsh

Open terminal
Notice how the placholder message overlaps with terminal content

Bonus bug - placeholder message uses weird font

/duplicate #212221


Build clang package with PGO

Describe the bug
LLVM supports building Clang with PGO (https://llvm.org/docs/HowToBuildWithPGO.html). Using PGO for compilers has a huge impact on compiler performance. E.g. many distros are already building GCC (which also supports PGO builds) with PGO. I think for the users would be beneficial to have faster Clang binaries.
Locally I usually build my own Clang version. According to my tests, it could bring up to 20% performance.
Additionally, you could consider using LLVM BOLT as an additional optimization step, but I guess it should be discussed in another issue after the PGO implementation.
Expected behavior
clang package will be PGO-optimized.
Other notes
It's not actually a bug - just an improvement request. Feel free to assign to the more proper category.

FWIW, the community explicitly made the decision to turn off PGO in GCC in order to have proper reproducibility: https://github.com/NixOS/nixpkgs/pull/112928


Nested Mappable structs bug

If a have two Mappable structs, and one nested to another, they are not mapped correctly if I do toObject. It took me a couple of hours to detect this bug in real project.
I have function:
public extension Mappable {

    func consumed(_ other: JSONType) -> Self {
        let mapper = Mapper<Self>()
        return mapper.map(JSON: other, toObject: self)
    }
}

And this structs and test:
struct A: Mappable {
    var name: String?

    init(name: String) {
        self.name = name
    }

    init?(map: Map) {

    }

    mutating func mapping(map: Map) {
        name        <- map["name"]
    }
}

struct B: Mappable {
    var a: A?

    init(name: String) {
        a = A(name: name)
    }

    init?(map: Map) {

    }

    mutating func mapping(map: Map) {
        a       <- map["a"]
    }
}

func consumeTest() {
    var b1: B = B(name: "QW") //has a with name "QW"
    let b2: B = B(name: "ER") //has a with name "ER"
    b1 = b1.consumed(b2.toJSON())
    print("b1 name \(b1.a?.name)") //name is still "QW"
}

This happens because operator <- doesn't do assignment for Mappable objects in toObject mode. I don't know if this behaviour is correct for framework. But looks weird and unexpected. I think this kind of functions map(JSONObject: Any?, toObject object: N) should be in extension where Mappable is a class.

I've just merged a fix for this issue. Thanks for pointing it out 👍

I've just looked at a fix. Didn't even think it was that simple! I now can make my entities structs again, thanks!


Can you guys make it work in Linux as well?

I'm using Linux Ubuntu 21.04, and it doesn't work.
Describe the solution you'd like
I'd like to be able to use this extension also in my Linux OS.
Describe alternatives you've considered
I'm not sure if this is possible, but if it is, that'd be great
Additional context
The request is simple, but I'm not sure if it's even possible to achieve.

What does "doesn't work" mean?  Can you provide more details?
I'm using Zorin, which is built on top of Ubuntu, and it works for me.

Without answer....


Release keyboard grab with Ctrl-Alt

http://sourceforge.net/tracker/?func=detail&atid=381349&aid=1569293&group_id=24366

+1. This is definitely the most important feature that I need in freerdp.
Please make it possible to bind "release keyboard grab" to some predefined combination of keys, preferably a combination of keys that the user can specify.
Without this feature I cannot switch from rdestop to freerdp.

+1 Indeed. Switched from Remmina to xfreerdp because of multi-monitor support, and the most needed feature is this one. Interestingly enough, exactly for the same scenario: "Right-Ctrl", "Ctrl+Alt+Up".

I'm trying to implement the feature in xfreerdp. I'm not an expert in XLib, though. I was able to use XUngrabKeyboard successfully, and binded it to different keys trying to find the best. However, I'm facing some problems.
I can't handle the fullscreen mode at all. It seems like I can ungrab the keyboard while in fullscreen xfreerdp session, and I can even switch between workspaces and applications in gnome, but xfreerdp screens are sort of stuck to my position, and are always on top: e.g., I press "Right Ctrl" to ungrab the keyboard, then I switch to another gnome workspace with "Ctrl+Alt+Up" (gnome indicator does not appear) and I can see "Alt+Tab" application switcher on top of my xfreerdp session. However, when I try to switch to another application, it seems like the focus is gone from xfreerdp, but it's still always on top.
So far I came to solution of using "Ctrl+Alt" for temporary ungrabbing in window-mode. Example scenario: I work in fullscreen xfreerdp session, then press "Ctrl+Alt+Enter" to exit fullscreen mode, then hit "Ctrl+Alt+" (yes, I have to release Crtl+Alt keys first) to perform some Gnome stuff (e.g. lock screen with Ctrl+Alt+L, or switch workspace with Ctrl+Alt+ArrowKey).
Does anybody have a better idea of how to handle this, and get rid of this "exit fullscreen" redundant step? I can share my solution as a git patch if anybody is interested.

@khvMX could you share your solution?

Here's a git patch:
https://gist.github.com/khvMX/66e0b14c71a32aca1bc7
What it does is simply ungrabbing the keyboard when not in fullscreen with a "Right Ctrl" key. It has currently at least 2 problems:

The fullscreen mode implementation seems a bit weird for me. Consider a scenario: I have 2 monitors: on the left hand side is a fullscreen-one-monitor-xfreerdp session is running, on the right hand side - usual GNOME 3 environment. In Remmina if I move the mouse to the GNOME environment and switch the workspace, the Remmina's window stays on its workspace as expected. In contrast, xfreerdp window will be stuck to left monitor no matter what workspace you are switching to. For exactly the same reason, if I just ungrab the keyboard, the xfreerdp window will be still stuck on top of everything and will chase me over all workspaces. In principle, if we'd change the way xfreerdp renders in fullscreen mode, we could just ungrab the keyboard like in remmina even in fullscreen, and switch.
In windowed mode it works almost as expected, except that the keyboard is automatically grabbed all the time (e.g. if the mouse cursor is still inside the xfreerdp window, when we ungrab the keyboard, it will be grabbed back after basically any action). It actually suffices for my purposes: hit "Right Ctrl" to ungrab, press "Ctrl+Alt+ArrowKey" to switch the workspace.
Current workaround for the fullscreen problem is to exit fullscreen mode with "Ctrl+Alt+Enter", then do whatever is needed in windowed mode. Actually, automatic grabbing sometimes helps a bit, e.g. when switching back to the workspace with xfreerdp I just hit "Ctrl+Alt+Enter" to enter fullscreen mode and continue to work via RDP.

Hmm, after some of the Ubuntu updates today it seems that this patch does not ungrab the keyboard anymore..

@khvMX Any progress on fullscreen mode? I have the exact same problem as you. I will be implementing your patch, as it is better than nothing! :)

Hi. No, didn't touch it, works well for me so far :) There are majorly one thing that still annoys me, namely:
each time you exit fullscreen mode the RDP window moves down to the amount of pixels which is roughly equal to the window title bar, resulting in window coming down with a time :D That is, when I switch back and forth between Gnome Environment and RDP, i have to grab this freaking RDP window with a mouse and push it up every once in a while :D

Ok, cool! I was thinking that would be very frustrating aswell, but still very good piece of technology here! I may look into expanding upon your patch this semester at school, I will keep you updated if I do :)
Thanks for the patch you did create!

@khvMX I have a version that works with fullscreen :) It's not perfect, maybe you can help improve it - I reused code that already existed in the source (although it appears to not have existed in yours). Using ctrl+alt+C releases the keyboard for a single action, such as switching desktops. The fix:
Basically just add your line XUngrabKeyboard(xfc->display, CurrentTime); to this section:
    if ((keysym == XK_c) || (keysym == XK_C))
    {
        if (mod.Ctrl && mod.Alt)
                {
                        /* Ctrl-Alt-C: toggle control */
                        xf_toggle_control(xfc);
+                       XUngrabKeyboard(xfc->display, CurrentTime);
                        return TRUE;
                }
        }
#if 0 /* set to 1 to enable multi touch gesture simulation via keyboard */

That section is just below the bit about toggling fullscreen. Please let me know if you have questions, I fear that was not particularly clear! I don't understand where the 'C' key comes from (the CTRL and ALT key portions are pretty obvious), and so I'm not sure how to change the key binding to something like simply 'Right CTRL'.

@Rudedog9d  the first line of the patch "if ((keysym == XK_c) || (keysym == XK_C))" introduced the (lower and uppercase) 'c' via keysyms https://tronche.com/gui/x/xlib/input/keyboard-encoding.html

Hmm, seems like this
xf_toggle_control(xfc);
does the trick of releasing control keys in fullscreen mode. With this line it works with my patch also with Right Ctrl in both fullscreen and windowed mode.
My Full patch for Right Ctrl: https://gist.github.com/khvMX/1519678698b48e6bfbe5dc69dc4eeca6
I still don't have the feeling that this would be enough for a pull request. At least the combination of keys must be somehow configurable by the user. Does anyone here has an overview of the whole thing to create a pull request?

@Rudedog9d
Just found out that in my patch, using
if (mod.Ctrl && mod.Alt)
instead of
if (mod.RightCtrl)
is much much easier in the end. I've been using right ctrl in fullscreen mode for some things (e.g. ctrl+shift+arrow to select another word in IDE), so this patch would break it. Now I can switch between desktops and even lock a screen (Ctrl+Alt+L) without even leaving RDP Window :) Thumbs up for helping me find the solution!

@kevinf28 Thank you, that makes more sense now!
@khvMX No problem, glad I could help! :) And thank YOU for pointing me in the right direction... I think I will end up using 'Super+ESC' or similiar for my escape sequence, as I already use Super for most of my window management keystrokes. I'm not sure how the whole pull thing works, but I noticed we are in the 'FreeRDP-old' repo - Our little hack probably wouldn't make it far anyhow :P

I would love for xfreerdp to send ctrl+alt+up and ctrl+alt+down to my linux host (ubuntu) so I can switch to another workspace. I'm using fullscreen xfreerdp on my 2nd workspace. This works great in the Citrix Receiver.
It's quite cumbersome to have to switch back to windowed mode using ctrl+alt+enter and move the mouse outside the xfreerdp window.
The patch seems to be here, so just merge it? Or am I being to practical :-P


(Feature request) Test against multiple servers

Please consider adding a feature where we can define multiple speed test server and it tests against those sequentially. One can define the speed test server using the ID.
I would love to have a test report from a couple of servers in India, Europe, US and Singapore.

Why not just run multiple containers each with a different SPEEDTEST_HOST and SPEEDTEST_SERVER but all pointing to the same InfluxDB?

The problem in doing that would be - all speed test instance would run at the same time, will choke the pipe and will lead to inconsistent results. If running one after other then we know for sure that only one test is running a given time.

You could add some delays in addition:
  speedtest1:
    restart: always
    image: frdmn/speedtest-grafana:latest
    command: sh -c "
      sleep 300 &&
      node index.js"

  speedtest2:
    restart: always
    image: frdmn/speedtest-grafana:latest
    command: sh -c "
      sleep 600 &&
      node index.js"

The above delays likely wouldn’t work well since, they could overlap in the future. Likely the better way would be to have a PR for looping through a list of comma-separated SERVER_HOST values but this seems a very specific use case?


Update 2_installation.rst

What does this PR do?
Fix block indentation in Installation documentation.
Before submitting

[x] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).

Thank you @funkybob 👍

Released in https://github.com/jazzband/django-axes/releases/tag/6.2.0


[Icon request] SVG folder icon

[x] I'm sure this issue is not a duplicate?

Please add same Icon for SVG file to folder SVG.

Please see #650.


Questions about the model performance

I tried Llama-3-8B-Instruct for MathBench, the summary is:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
dataset                                               version    metric    mode    llama-3-8b-instruct-hf

######## MathBench Application Accuracy ########      -          -         -       -
mathbench-college-single_choice_cn                    783703     acc_1     gen     38.00
mathbench-college-single_choice_en                    b0fb1b     acc_1     gen     40.00
mathbench-high-single_choice_cn                       783703     acc_1     gen     39.33
mathbench-high-single_choice_en                       b0fb1b     acc_1     gen     38.67
mathbench-middle-single_choice_cn                     783703     acc_1     gen     49.33
mathbench-middle-single_choice_en                     b0fb1b     acc_1     gen     28.67
mathbench-primary-cloze_cn                            ea47a6     accuracy  gen     63.33
mathbench-primary-cloze_en                            bcc9c6     accuracy  gen     71.33
mathbench-arithmetic-cloze_en                         bcc9c6     accuracy  gen     52.67
######## MathBench Application CircularEval ########  -          -         -       -
mathbench-college-single_choice_cn                    783703     perf_4    gen     9.33
mathbench-college-single_choice_en                    b0fb1b     perf_4    gen     15.33
mathbench-high-single_choice_cn                       783703     perf_4    gen     8.67
mathbench-high-single_choice_en                       b0fb1b     perf_4    gen     12.67
mathbench-middle-single_choice_cn                     783703     perf_4    gen     23.33
mathbench-middle-single_choice_en                     b0fb1b     perf_4    gen     9.33
######## MathBench Knowledge CircularEval ########    -          -         -       -
mathbench-college_knowledge-single_choice_cn          783703     perf_4    gen     52.85
mathbench-college_knowledge-single_choice_en          b0fb1b     perf_4    gen     66.77
mathbench-high_knowledge-single_choice_cn             783703     perf_4    gen     35.11
mathbench-high_knowledge-single_choice_en             b0fb1b     perf_4    gen     58.72
mathbench-middle_knowledge-single_choice_cn           783703     perf_4    gen     41.62
mathbench-middle_knowledge-single_choice_en           b0fb1b     perf_4    gen     64.57
mathbench-primary_knowledge-single_choice_cn          783703     perf_4    gen     37.98
mathbench-primary_knowledge-single_choice_en          b0fb1b     perf_4    gen     67.89
######## MathBench Knowledge Accuracy ########        -          -         -       -
mathbench-college_knowledge-single_choice_cn          783703     acc_1     gen     71.52
mathbench-college_knowledge-single_choice_en          b0fb1b     acc_1     gen     77.22
mathbench-high_knowledge-single_choice_cn             783703     acc_1     gen     60.00
mathbench-high_knowledge-single_choice_en             b0fb1b     acc_1     gen     75.09
mathbench-middle_knowledge-single_choice_cn           783703     acc_1     gen     64.97
mathbench-middle_knowledge-single_choice_en           b0fb1b     acc_1     gen     81.71
mathbench-primary_knowledge-single_choice_cn          783703     acc_1     gen     64.42
mathbench-primary_knowledge-single_choice_en          b0fb1b     acc_1     gen     82.57
$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$$
I would like to ask how to calculate the model performance, for example, the Primary Application performance you gave for Llama-3-8B-Instruct is 71.0; but here the summary is 71.33 for primary-cloze_en and 63.33 for primary-cloze_cn; the average of these two is smaller...
the Primary Theory performance you gave for Llama-3-8B-Instruct is 60.2; but here the summary is 67.89 for single_choice_en and 37.98 for single_choice_cn; the average of these two is much smaller...
and same with other performance for Llama-3-8B-Instruct.

We have updated the summarizer, making it easier to obtain group results for MathBench A&T. Please refer to the updated README for guidance.
For Llama-3-8B-Instruct, here is what I obtained:
MathBench-A: llama-3-8b-instruct-hf

MathBench Accuracy

Category
Metric
Value

mathbench-new-college-single_choice_cn
acc_1
35.33

mathbench-new-college-single_choice_en
acc_1
46.67

mathbench-new-high-single_choice_cn
acc_1
44.67

mathbench-new-high-single_choice_en
acc_1
51.33

mathbench-new-middle-single_choice_cn
acc_1
54.00

mathbench-new-middle-single_choice_en
acc_1
44.67

mathbench-new-primary-cloze_cn
accuracy
65.33

mathbench-new-primary-cloze_en
accuracy
76.67

mathbench-new-calculate-cloze_en
accuracy
54.67

MathBench CircularEval

Category
Metric
Value

mathbench-new-college-single_choice_cn
perf_4
9.33

mathbench-new-college-single_choice_en
perf_4
18.67

mathbench-new-high-single_choice_cn
perf_4
10.67

mathbench-new-high-single_choice_en
perf_4
27.33

mathbench-new-middle-single_choice_cn
perf_4
28.67

mathbench-new-middle-single_choice_en
perf_4
21.33

mathbench-new-primary-cloze_cn
accuracy
65.33

mathbench-new-primary-cloze_en
accuracy
76.67

mathbench-new-calculate-cloze_en
accuracy
54.67

And for MathBench-T:

MathBench-T: llama-3-8b-instruct-hf
MathBench CircularEval

Category
Metric
Value

mathbench-knowledge-college-single_choice_cn
perf_4
43.35

mathbench-knowledge-college-single_choice_en
perf_4
63.92

mathbench-knowledge-high-single_choice_cn
perf_4
33.19

mathbench-knowledge-high-single_choice_en
perf_4
53.74

mathbench-knowledge-middle-single_choice_cn
perf_4
36.23

mathbench-knowledge-middle-single_choice_en
perf_4
66.29

mathbench-knowledge-primary-single_choice_cn
perf_4
44.23

mathbench-knowledge-primary-single_choice_en
perf_4
76.15

MathBench Knowledge Accuracy

Category
Metric
Value

mathbench-college_knowledge-single_choice_cn
acc_1
67.41

mathbench-college_knowledge-single_choice_en
acc_1
78.80

mathbench-high_knowledge-single_choice_cn
acc_1
62.55

mathbench-high_knowledge-single_choice_en
acc_1
72.95

mathbench-middle_knowledge-single_choice_cn
acc_1
63.47

mathbench-middle_knowledge-single_choice_en
acc_1
80.57

mathbench-primary_knowledge-single_choice_cn
acc_1
68.27

mathbench-primary_knowledge-single_choice_en
acc_1
82.57

It appears that your results are lower than mine. Please ensure that you are using the latest dataset, released on May 14: https://github.com/open-compass/MathBench/releases/tag/v0.1.0

Thank you for your reply, while we do use the latest dataset...

Can you give me some prediction samples, especially for single-choice questions?

If there is no cot with the prediction, please pull the newest main branch of opencompass and evaluate again, because the cot is set to None unexpectedly during the last version of opencompass.


implement a buff

good first candidate: #103

could work by:

iterating through existing attempts per probe
adding new, novel attempts (this defines the signature of the buff methods, neat)
storing the buff name(s) in attempt.notes["buffs"]

in this case we'll need:

a hook for updating generator params - can the existing one do it?

about that (s)..

give a config option to combine buffs, default is to go through them one by one
so also give a CLI option to specify buffs

the buff runs after prompts have been created, i.e. after the attempts are there - but before the attempts are sent to the detector

run buffs in the harness
store the original attempt uuid in the buffed attempt, in its notes

what does the buff do? provide .. hooks? one hook?

takes: attempt
yields: attempts


Documentation on environment variables

Description
Currently the README only has the template info
Screenshots
No response
Additional information
No response