MRG: Fix info equiv check

Dicts have no guaranteed order, so the _is_equal_dict using a zip of .items() was unsafe. This uses object_diff, which uses a more careful algorithm.
FYI @jaeilepp the CI failure from #3685 was random (as in unpredictable), but it was also real (as in not safely ignored). It made a test fail in #3707, for example. This should fix it, though.

LGTM but appveyor complains :(

@agramfort sweet, an unrelated but also bad error:
Downloading Miniconda-latest-Windows-x86_64.exe from https://repo.continuum.io/miniconda/Miniconda-latest-Windows-x86_64.exe
Exception calling "DownloadFile" with "2" argument(s): "The request was
aborted: Could not create SSL/TLS secure channel.

Let's fix that one separately. Hopefully it's a temporary AppVeyor problem. At least with this PR Travis should get more reliable.
cc @dengemann it looks like the zip(*[d.items() for d in dicts]) thing was from 2014, in case you can think of other places you might have used it.
git grep doesn't suggest any other zip(* problems with dicts at least.

fair enough
merge if you're happy


Bump mime-types

.18 adds a few mime types like application/wasm

Yes please I concur @giuseppeg. We've just updated for the new node .mjs extension as well. /cc @broofa

".mjs" and ".wasm" are pretty important for browser workflows. Please do merge this!


Content Preview

Description
(Please describe the changes in this PR for reviewers, motivation, rationale - mandatory)

Upstream PR
The following PR deals with the following generated content for the following versions:

3.10: arangodb/enterprise-preview:3.10-nightly
3.11: arangodb/enterprise-preview:3.11-nightly
3.12: arangodb/enterprise-preview:devel-nightly

/generate

/generate

/commit-generated


device_info plugin makes the app crash

Hi,
I am using the device_info plugin inside my app. Sometimes the release app crashes because of this issue:

I didn't know how to reproduce this issue The app is built with flutter 1.20.
Please help to check it.
Thanks.

Please include outputs of flutter doctor and flutter run -v

@amanv8060 this issue is not re-producible on local flutter run. It happens on my iOS production released app which is built by Bitrise with flutter 1.20.4.

Hi @duy-ng-12354
Can you please provide your flutter doctor -v, your crash logs, your pubspec.yaml, the list of affected devices and if possible your flutter run --release --verbose running on the same OS version as the affected devices.
Thank you

@TahaTesser Here you are:
Flutter doctor:
`⋊> ~ flutter doctor -v                                                                      18:05:17
[✓] Flutter (Channel stable, 1.20.4, on Mac OS X 10.15.6 19G2021, locale en)
• Flutter version 1.20.4 at /Users/duy/hub/flutter
• Framework revision fba99f6cf9 (3 days ago), 2020-09-14 15:32:52 -0700
• Engine revision d1bc06f032
• Dart version 2.9.2
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at /Users/duy/hub/android-sdk
• Platform android-29, build-tools 29.0.3
• ANDROID_HOME = /Users/duy/hub/android-sdk
• Java binary at: /Users/duy/.sdkman/candidates/java/current/bin/java
• Java version OpenJDK Runtime Environment AdoptOpenJDK (build 11.0.6+10)
✗ Android license status unknown.
Try re-installing or updating your Android SDK Manager.
See https://developer.android.com/studio/#downloads or visit
https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions.
[✓] Xcode - develop for iOS and macOS (Xcode 11.7)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.7, Build version 11E801a
• CocoaPods version 1.8.4
[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed
instructions).
[✓] VS Code (version 1.49.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.14.1`
Crash log (from Crashlytics):
`
Fatal Exception: NSInvalidArgumentException
*** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[5]
Fatal Exception: NSInvalidArgumentException
0  CoreFoundation                 0x1990fd114 (Missing)
1  libobjc.A.dylib                0x1ac8b3cb4 objc_exception_throw
2  CoreFoundation                 0x199167398 (Missing)
3  CoreFoundation                 0x199172590 (Missing)
4  CoreFoundation                 0x198ff86a4 (Missing)
5  CoreFoundation                 0x198feba04 (Missing)
6  device_info                    0x104555028 (Missing)
7  Flutter                        0x10385dd48 (Missing)
8  Flutter                        0x103340994 (Missing)
9  Flutter                        0x1036072e8 (Missing)
10 Flutter                        0x103349a40 (Missing)
11 Flutter                        0x10334b6b8 (Missing)
12 CoreFoundation                 0x19907da30 (Missing)
13 CoreFoundation                 0x19907d634 (Missing)
14 CoreFoundation                 0x19907cb14 (Missing)
15 CoreFoundation                 0x199076eb0 (Missing)
16 CoreFoundation                 0x199076200 (Missing)
17 GraphicsServices               0x1af0f0598 GSEventRunModal
18 UIKitCore                      0x19b93c1b0 (Missing)
19 UIKitCore                      0x19b941784 (Missing)
20 Runner                         0x102757bc4 main + 5 (AppDelegate.swift:5)
21 libdyld.dylib                  0x198d55598 (Missing)`
Pubspec
https://drive.google.com/file/d/180Ucm41iujw5LK6s0FN8xXY1mk4uoGOM/view?usp=sharing
Some affected devices
iPhone XR / 6s / XS / 7 . The OS version are 13.6, 13.3, 13.1 and 14.

Hi @duy-ng-12354
Can you please provide your pubspec.yaml, logs flutter run --release --verbose

@TahaTesser  I am unable to re-produce it again so there is no logs to provide as you guys requested. Let me close this one and re-open it again it it would happen again.
Thanks for your time.


Chrysler non-ACC Fix - Only allow openpilot in ACC mode

Fixes https://github.com/commaai/openpilot/issues/1278
Still needs to be tested and awaiting a route. Should probably also look for other states in the message.

522b7df1fa94de66|2020-12-28--21-37-14

Merged opendbc pr and updated on master. Please rebase

Merged opendbc pr and updated on master. Please rebase

Still changes to opendbc

Still changes to opendbc


SASpy 3.6.1 Running forever

I tried new version of SASPy 3.6.1 today after tweaking some of the configurations settings but it keeps on running without erroring out but SASPy 3.0.0 works fine. Please assist
Python 3.7.8 (tags/v3.7.8:4b47a5b6ba, Jun 28 2020, 08:53:46) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import saspy
>>> sas = saspy.SASsession(cfgname='Prod_Sec')

reopened in https://github.com/sassoftware/saspy/issues/338


Fix serialization restore for named tuples

Serialization was creating a new named tuple class based on the fields in the state dict.
This results in a type error when comparing to the original or when comparing the structure of PyTrees.
Fixes #1429

Codecov Report

Merging #1432 (bfca2cc) into main (f8dc89a) will decrease coverage by 0.00%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main    #1432      +/-   ##
==========================================
- Coverage   82.29%   82.28%   -0.01%
==========================================
  Files          65       65
  Lines        5332     5330       -2
==========================================
- Hits         4388     4386       -2
  Misses        944      944

Impacted Files
Coverage Δ

flax/serialization.py
89.93% <87.50%> (-0.13%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks for the fix!


get_title only returns the DVD menu.

get_title seems to only be returning the9 frame long DVD menu and nothing else.
VTS_01_0.IFO I can provide other files from the DVD if need be as well.

I've just found another DVD with the same issue, would you like samples from that one as well @Setsugennoao?

Can't reproduce anymore. Issue is > 1year old so probably got fixed with the rewrite.


assert API design

I've just found out that the error argument of .throws is not optional

.throws(function|promise, error, [message])

and that .regexTest is not consistent with the rest of the assertions:

.regexTest(regex, contents, [message])

because of its parameter order (all other assertions follow the basic scheme (actual, expected, message)).
I'm not sure if these were intended…

Could you please attach some code to both of these?
I just tested the first one and it works as expected, even when I don't set error argument:
import test from '.';

test(t => {
		t.throws(function () {
			throw new Error('damn it');
		});

		t.throws(Promise.reject(new Error('damn it')));

		t.end();
});

@vdemedes That is true, it works but:

it is not documented, so the behavior might change
.throws(function|promise [, error][, message]) currently does not work as expected with #125:

test(t => {
	t.throws(()=>{}, 'some message');
	t.end();
});

Before #125:
  ✖ [anonymous] Missing expected exception. some message

  1 test failed

  1. [anonymous]
  AssertionError: Missing expected exception. some message
    at Test.fn (/home/patrick/Projekte/github/merge-options/t.js:4:4)

After:
  ✖ [anonymous] Missing expected exception..

  1 test failed

About .regexTest:
t.is('actual', 'expected');
t.regexTest(/expected/, 'actual');

Then the notation should be something like this
.throws(function|promise [, error [, message]])

@SamVerschueren That is possible and I would recommend that, even though switching from assert.throws  is no longer so simple.

This is mentioned by @sindresorhus a couple of times, as well in #125.

We only use the core assert so not to duplicate code. If anything can be done better, we'll do it. Also the core Node.js people have said assert is done and won't change anymore.

That assertion module is not a binding contract. That's also the reason why AVA isn't using the same names as the assert library, to abstract away the implementation.

@SamVerschueren I knew that and I like #125, the problem is, in contrast to most assertions in AVA, the name (throws) is the same and developers who switch to AVA may experience unexpected hard to grasp behavior.
Maybe we should introduce our own throws assertions with a different name, in particular, because nodes throws and doesNotThrow suffer from many problems:
try {
	assert.throws(()=>{ throw new Error('abc')}, TypeError);
} catch (err) {
	console.log(err instanceof assert.AssertionError);
	//=> false
	console.log(err.message);
	//=> 'abc'

	// I would expect something like:
	console.log(err.message);
	//=>'Missing expected TypeError. Got unwanted exception Error('abc').'
}

Not sure about a name change, throws seems the shortest and most explanatory name for this. Something like t.exception doesn't feels right.
IMO people should read the docs, and I understand that some of them might think it has the same syntax, but then again, RTFM? ;).
I guess @sindresorhus would be happy to give his thoughts on this.

I think of more powerful assertions, maybe with different parameters. As this would also support Promises, rejects seem reasonable.

raises seems more appropriate.
If we go for a name change, I would not keep throws. It will confuse people because it has two methods that are almost identical, just to be consistent with assert. Again, that is not the intention of AVA (but I guess you already know :)).
It's certainly not up to me to make this decisions.

Oh no, let's not change the name of throws(). rejects looks like it supports only promises and raises is from Ruby.
All we have to do, is to fix & stabilize issues pointed out in the first comment.

I would prefer not to change the name of .throws. A lot of things will differ from the core assert module. Even more when we evolve further. I think we should rather fix the issues mentioned here and document clearly that the API is not core assert compatible.
As for regexTest, that unfortunate, but honestly, do we even need it now that we have power-assert? t.true(/foobar/.test(foo)); would be just as good with the excellent output of power-assert.

As for regexTest, that unfortunate, but honestly, do we even need it now that we have power-assert?

Thoughts on deprecating regexTest?

@jamestalmage

@sindresorhus I'd leave it, but rename to just t.regex. To me it looks a bit more convenient (less nesting of statements):
t.regex(/ok/, 'oh ok');
t.true(/ok/.test('oh ok'));

I'm with @vdemedes on this one. Would leave it as well.

@vdemedes But the current one has the wrong argument order. I suggest we deprecate regexTest and add regex with the correct argument order then. Unless anyone has a better idea.
t.regex('oh ok', /ok/);

@sindresorhus oh, in that case, definitely, regexTest should be deprecated and correct regex should be implemented.

In my opinion, reducing APIs that are easy to make mistakes is the most valuable point of power-assert.
So I'm :+1:  on
t.true(/ok/.test('oh ok'));
(But for this purpose I must improve output depth of SuccinctRenderer! )

Is t.regex still on the table or do we resort to using t.true with an explicit regex.test? I'd love to contribute to the cause, got the necessary changes stashed and ready for tomorrows commute!
Oh and do excuse the potential necro. Just scouring through the help wanted issues.

It is still on the table. Go for it!

It is still on the table. Go for it!

Commuting tomorrow morning is gon' be fun, cheers! : )


Add known issue that Bossa reported.

Description
Add known issue that Bossa reported.
Tests
None
Documentation
This is documentation.

@jamiebrynes7 it's unclear from the defect report but it looks like it's an incompatibility cause by having either no Visual Studio Tools for Unity installed, or an old incompatible version.

In which case I would argue this is not a known issue with the GDK

Abandoning this PR and instead placing the the troubleshooting section.


Refactor ProjectValidationUtils to ExperimentUtils

@optimizely/fullstack-devs

Can you also run the e2e tests on your branch?


Fixing web pack warnings

First One:
ERROR in ./examples/auth-flow/app.js
Module build failed: SyntaxError:
/Users/erik/towbook/react-router/examples/auth-flow/app.js: Line 73:
'this' is not allowed before super()
Second:
ERROR in ./examples/auth-flow/app.js
Module build failed: SyntaxError:
/Users/erik/towbook/react-router/examples/auth-flow/app.js: Line 73:
'this' is not allowed before super()
Running on node v0.12.2

+1 this PR

These warnings are fixed in master.


Create Stripe App without UI

We are trying to create a Stripe app that does not have a UI. All the documentation which I've looked at assumes a UI (even the authentication flows).
It is unclear what we should do for a non-UI app & how we get the acct_xxxx values out to pull data out of Stripe. Additionally, it would be very helpful if there was some documentation or example code on an app that purely grabs the required auth info out of Stripe and nothing more.
Should we create an app that has a blank drawer screen? It's unclear to me if that would even get through the app review process.
Best,
Steven

Bump - also stuck on trying to figure out how to provision OAuth Apps/Links without UI components.
Stripping down the manifest doesn't seem to work:
[bhatfield app]% stripe apps upload  --log-level debug
[Tue, 25 Jul 2023 18:16:29 EDT] DEBUG config.Config.InitConfig: Using profiles file path=/Users/bhatfield/.config/stripe/config.toml
[Tue, 25 Jul 2023 18:16:29 EDT] DEBUG config.Config.GetProfilesFolder: Using profiles file path=/Users/bhatfield/.config/stripe
[Tue, 25 Jul 2023 18:16:29 EDT] DEBUG cmd.pluginCmd.runPluginCmd: Running plugin...
[Tue, 25 Jul 2023 18:16:29 EDT] DEBUG config.Config.GetProfilesFolder: Using profiles file path=/Users/bhatfield/.config/stripe
[Tue, 25 Jul 2023 18:16:29 EDT] DEBUG config.Config.GetProfilesFolder: Using profiles file path=/Users/bhatfield/.config/stripe
[Tue, 25 Jul 2023 18:16:29 EDT] DEBUG plugins.plugin.Run: negotiated gRPC with plugin process

⬆ You are about to upload your app to [REDACTED]

Name:    [REDACTED]
ID:      [REDACTED]
Version: 0.0.1

Would you like to proceed: Y
Would you like to proceed: Y
2023-07-25T18:16:35.715-0400 [ERROR] plugin.child.apps: plugin process exited: path=/Users/bhatfield/.config/stripe/plugins/apps/1.5.10/stripe-cli-apps pid=46436 error="exit status 2"
[Tue, 25 Jul 2023 18:16:35 EDT] DEBUG Tearing down plugin before exit
[Tue, 25 Jul 2023 18:16:35 EDT] DEBUG pluginTemplateCmd.runPluginCmd: Plugin command 'apps' exited with error: rpc error: code = Unavailable desc = error reading from server: EOF

{
  "id": "[REDACTED]",
  "version": "0.0.1",
  "name": "[REDACTED]",
  "icon": "",
  "permissions": [],
  "ui_extension": {
    "views": null,
    "content_security_policy": null
  }
}

to get the upload succeed without any views could you please try ?
"ui_extension": {
    "views": [],
    "content_security_policy": {}
 }

Thank you for your response - working with just a bare manifest in the directory, here's what happens when I attempt that upload:
[bhatfield app]% stripe apps upload  --log-level debug
[Wed, 26 Jul 2023 11:11:59 EDT] DEBUG config.Config.InitConfig: Using profiles file path=/Users/bhatfield/.config/stripe/config.toml
[Wed, 26 Jul 2023 11:11:59 EDT] DEBUG config.Config.GetProfilesFolder: Using profiles file path=/Users/bhatfield/.config/stripe
[Wed, 26 Jul 2023 11:11:59 EDT] DEBUG cmd.pluginCmd.runPluginCmd: Running plugin...
[Wed, 26 Jul 2023 11:11:59 EDT] DEBUG config.Config.GetProfilesFolder: Using profiles file path=/Users/bhatfield/.config/stripe
[Wed, 26 Jul 2023 11:11:59 EDT] DEBUG config.Config.GetProfilesFolder: Using profiles file path=/Users/bhatfield/.config/stripe
[Wed, 26 Jul 2023 11:11:59 EDT] DEBUG plugins.plugin.Run: negotiated gRPC with plugin process

⬆ You are about to upload your app to [REDACTED]

Name:    [REDACTED]
ID:      [REDACTED]
Version: 0.0.1

Would you like to proceed: Y
×  Failed to build files
✘ [ERROR] Could not resolve "@stripe/ui-extension-sdk/version"

    .build/manifest.js:4:14:
      4 │ export * from '@stripe/ui-extension-sdk/version';
        ╵               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  You can mark the path "@stripe/ui-extension-sdk/version" as external to exclude it from the bundle, which will remove this error.

[Wed, 26 Jul 2023 11:12:05 EDT] DEBUG Tearing down plugin before exit
[Wed, 26 Jul 2023 11:12:05 EDT] DEBUG pluginTemplateCmd.runPluginCmd: Plugin command 'apps' exited with error: rpc error: code = Unknown desc = ✘ [ERROR] Could not resolve "@stripe/ui-extension-sdk/version"

    .build/manifest.js:4:14:
      4 │ export * from '@stripe/ui-extension-sdk/version';
        ╵               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  You can mark the path "@stripe/ui-extension-sdk/version" as external to exclude it from the bundle, which will remove this error.

Please note that the upload attempt modifies the contents of stripe-app.json. See below:
Before:
{
  "id": "[REDACTED]",
  "version": "0.0.1",
  "name": "[REDACTED]",
  "icon": "",
  "permissions": [],
  "ui_extension": {
    "views": [],
    "content_security_policy": {}
  }
}

After:
{
  "id": "[REDACTED]",
  "version": "0.0.1",
  "name": "[REDACTED]",
  "icon": "",
  "permissions": [],
  "ui_extension": {
    "views": [],
    "content_security_policy": {
      "connect-src": null,
      "image-src": null,
      "purpose": ""
    }
  }
}

I was able to get the upload to succeed by removing the src/ directory from the autogenerated structure and uploading in that context instead:
drwxr-xr-x    6 bhatfield   192B Jul 26 11:37 .build/
drwxr-xr-x    6 bhatfield   192B Jul 26 11:10 .git/
-rw-r--r--    1 bhatfield   297B Jul 26 11:10 .gitignore
drwxr-xr-x    3 bhatfield    96B Jul 26 11:10 .vscode/
-rw-r--r--    1 bhatfield   217B Jul 26 11:10 jest.config.js
drwxr-xr-x  392 bhatfield    12K Jul 26 11:11 node_modules/
-rw-r--r--    1 bhatfield   293K Jul 26 11:11 package-lock.json
-rw-r--r--    1 bhatfield   592B Jul 26 11:11 package.json
-rwxr-xr-x    1 bhatfield   262B Jul 26 11:37 stripe-app.json*
-rw-r--r--    1 bhatfield    68B Jul 26 11:10 tsconfig.json
-rw-r--r--    1 bhatfield    53B Jul 26 11:10 ui-extensions.d.ts

[Wed, 26 Jul 2023 11:37:32 EDT] DEBUG config.Config.InitConfig: Using profiles file path=/Users/bhatfield/.config/stripe/config.toml
[Wed, 26 Jul 2023 11:37:32 EDT] DEBUG config.Config.GetProfilesFolder: Using profiles file path=/Users/bhatfield/.config/stripe
[Wed, 26 Jul 2023 11:37:32 EDT] DEBUG cmd.pluginCmd.runPluginCmd: Running plugin...
[Wed, 26 Jul 2023 11:37:32 EDT] DEBUG config.Config.GetProfilesFolder: Using profiles file path=/Users/bhatfield/.config/stripe
[Wed, 26 Jul 2023 11:37:32 EDT] DEBUG config.Config.GetProfilesFolder: Using profiles file path=/Users/bhatfield/.config/stripe
[Wed, 26 Jul 2023 11:37:32 EDT] DEBUG plugins.plugin.Run: negotiated gRPC with plugin process

⬆ You are about to upload your app to [REDACTED]

Name:    [REDACTED]
ID:      [REDACTED]
Version: 0.0.1

Would you like to proceed: Y
✔ Built files for production
✔ Packaged files for upload
✔ Uploaded [REDACTED]

Stripe need to process your files before this version can be installed.
🌐 To view the status on Stripe Dashboard, press enter.

[Wed, 26 Jul 2023 11:37:48 EDT] DEBUG Tearing down plugin before exit

Running into this issue as well. I tried removing "ui_extension" from the manifest but the UI is still there.


bugfix: setup related issue with wrappid app template due to unwanted peer package dependency of react

Describe the bug
setup related issue with wrappid app template due to unwanted peer package dependency of react

this issue had been fixed


Missing standards?

Assessment templates was shared during a training today and ISO 15489 is in the Compliance Center but not in this list. Can you update the list to be an accurate representation of what is in the system?
What role do I need to see the list of templates?
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a406f24f-e0c1-9143-9130-ce0459a66235
Version Independent ID: e040197a-12cc-281a-d75d-16395b24c46f
Content: Microsoft Compliance Manager templates list - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/compliance-manager-templates-list.md
Service: o365-seccomp
GitHub Login: @chvukosw
Microsoft Alias: chvukosw

@chvukosw  Please add your comment and help us in resolving this issue. Thanks.

Hello, you are correct, ISO 15489 was a recently-added template in Compliance Manager. The templates list page in the documentation (https://docs.microsoft.com/microsoft-365/compliance/compliance-manager-templates-list) will be updated by the end of today. Users holding any role are able to access all templates, both the included templates and premium templates, from the assessment templates page in Compliance Manager. Thank you.


Getting occasional 404s for _axiom/web-vitals call on PROD

Noticed my production dashboard is showing a small percentage of 404 errors for the web vitals call. Any idea what's causing this and how I can patch it? I do see the web vitals working since there are a number of successful calls for most users.
Secondly, is there a way to get this call to work locally?

Hey @jerang, thanks for the issue -- it's something we're aware of and looking into 👍

Secondly, is there a way to get this call to work locally?

You can copy the NEXT_PUBLIC_AXIOM_INGEST_ENDPOINT from your environment and expose it locally.

Hi @bahlo, to prevent create a duplicated issue, I'm seeing some requests to non-existents paths like /wordpress /wp /old /new /2020 etc.
This is something related to this issue?
Thanks!

Hi @bahlo, to prevent create a duplicated issue, I'm seeing some requests to non-existents paths like /wordpress /wp /old /new /2020 etc.
This is something related to this issue?
Thanks!

those are probably bots or web crawlers maybe trying to access those routes that don't exist on your app, I believe its not related to next-axiom.

@jerang do you still see this happening?

I am still seeing it and noticed a trend; seems to happen every 3 days at mid night. I don't have any automated deployments or builds so curious if that helps.


Update README.md

This is my commit message
Signed-off-by: VineethReddy02<EMAIL_ADDRESS>Added a build badge and circleci badge to Readme for Prometheus docs.
Its good to have badges in Readme to track build status.

This is my commit message
Signed-off-by: VineethReddy02<EMAIL_ADDRESS>Added a build badge and circleci badge to Readme for Prometheus docs.
Its good to have badges in Readme to track build status.

Note to confirm to DOC closing this PR and opening a new one.

See #1285


added a transfer.sh plugin

Created a single function to easily upload files to transfer.sh file sharing site
Usage : $ transfer file.txt
@slavaGanzin

@ilyes-kechidi Would you change this PR to add ziping directories?

Done @slavaGanzin

@ilyes-kechidi
I tested it. Works as expected.
I think we should switch to tar czf from zip, because tar is by default on all linux systems and zip is optional.

@slavaGanzin
Working on it

Done @slavaGanzin
needs testing though :)


Improve course page

i can fix the responsiveness and add grid to make the course adjust the design of the course page .

Please assign this issue to me as I would like to work on fixing it. also add the label GSSoC 2024 Extd relaevant .

@sahhoArjun097 it was completed before,
I will encourage you to look for other issues in website like improvements, then create an new issue
for now i am closing this issue.


determining the speed of generation based on model's underlying quality

This is both (1) a very naive question and (2) probably best suited for another forum, but I'll ask it anyway.
Is there a relationship between (1) the underlying quality of the LLM used and (2) the time it takes to complete the LMQL query?  Take two models for example:

model one -> very good model that, during greedy decoding, could actually complete the desired output without any LMQL constraints
model two -> a very "poor" model that can only complete the desired output with LMQL constraints

Will one of those models complete the generation faster?
This question reveals my lack of detailed understanding of both (1) greedy decoding in general and (2) LMQL's implementation but I'd appreciate some more intuition on the question as it will help us decide if it's "worth" using a stronger (most likely larger w.r.t. parameters) model in our application.

The quality of a model does not directly affect inference speed. However, smaller models (wrt no. of parameters) are typically much faster while often also less capable. Thus, it could be that you are just observing a smaller/less capable model to be faster and a bigger/more capable model to be slower.


bump 0.4.14

Nevermind, there are no new updates since last release - the issues on the iteration were released before.


Cannot find variable ..- when creating negative ranges

When creating a list with a range of numbers I came across an issue if the "high" number is negative and there isn't a space between range operator and "high" number.
Pasted some example output of the issue.
> [0..5]
[0,1,2,3,4,5] : List number
>
> [-5..0]
[-5,-4,-3,-2,-1,0] : List number
>
> [-10..-5]
-- NAMING ERROR ---------------------------------------------- repl-temp-000.elm

Cannot find variable `..-`

3|   [-10..-5]
      ^^^^^^^
Maybe you want one of the following?

    -
    Basics.-

> [-10 .. -5]
[-10,-9,-8,-7,-6,-5] : List number

@hendore can you reopen this on elm-lang/elm-compiler? this would be something to address in the parser

@lukewestby Done, thank you very much for the clarification.


terminal logging and handler logic

Is your feature request related to a problem? Please describe.
I will describe the use cases here, some are a problem, some are not.

Abuse - learners may try to use the system/platform for malicious or abusive behavior. For example using resources for bitcoin mining, sending spam emails, DOS to sites, or general hacking into a system trying to hide their entry behind other systems.
Learner Insight - as a person creating workshop/lab content, it can help to know what people are doing when in the lab. For example, how many commands did they type and at what time in the lab? Did they get errors often, did they try for help, try to do things other than what the lab instructions state, etc. By logging terminal output (with metadata) it would allow that insight to be available to the workshop author(s).

Describe the solution you'd like
The thought was a few things:

One could optionally enable terminal logging in a workshop definition. It would then collect the terminal input/output and store it in educates cluster.
Optionally a person could configure a "sink" for the terminal logging to go to (Object Store, Google Drive folder, SFTP, Queue, Webhook, etc) for longer term storage/analysis
Option to specify a container or script that could be registered as a "handler" to process the logging events live in a streaming manner and then take actions accordingly. For example it could be given the metadata about the stream of logging and if malicious behavior is detected the handler could delete the workshop sessions, or take other actions.

Describe alternatives you've considered
Providing a cronjob or other streaming agent installed in the workshop tries to get at the .bash_history or other files and ship them off to somewhere else for processing
Additional information
No response

For first stage of exploring what can be done, would suggest that build on existing implementation of frontend/backend for terminals that actually allows multiple browser windows on the same terminal session.
One issue with this existing functionality is that if the viewport sizes for the terminal are different then the second browser window will force a terminal size change on server side for process attached to pty, thus affecting the first browser. It is because of this and to flag first user that something strange is going on, that "HIJACKED" message is shown on first browser terminal window to indicate was taken over.
For purpose of exploring capturing terminal data in better way idea is that could work out how to have a read only viewer using this mechanism where second browser doesn't actually force a resize of process attached to pty, but instead resizes browser window to accomodate the existing terminal size. No terminal resizing would be triggered from second browser and instead scrollbar would show around the frame instead if was made smaller, that or could see if could somehow use proportional scaling of view so maintain original col/rows of terminal.
The idea is that one could add access to this view of terminals to special admin pages in training portal, thus allowing a trainer to view anyones terminals if needed without interfering with the terminal size (as would occur now if admin visits users session dashboard when URL is known), with by default no input accepted from second browser, although could have a toggle to permit entry so trainer could run stuff through users terminal.
This system would make use of existing web socket capability exposed by session dashboard. In adding this proper read only like view, can then see if we can make use of this to pull input/output into a separate monitoring service rather than a browser for capture, later replay, or searches for malicious commands.

Can we bump up the priority of this as it would help workshop abuse scenarios.
cc @billkable

Since is not easy to capture individual commands entered at the terminal, since is stream of messages for individual characters, option have been thinking of is to capture the stream of messages that get sent to front end to render the terminal. These are already being held internal to backend process within a sliding window memory buffer to allow replay to front end when a window is refreshed, so that visual state of terminal can be redrawn. So is just necessary to write them out to a file at the same time and work out a way then to get them out of the container, or stream them live to a remote service. This would allow later offline replay of a session, although right now it doesn't have time stamps so one couldn't replay them in orderly manner and not have a mess when it fires up a visual text editor and wipes out the displayed terminal buffer. Anyway, will give it some more thought.


Run-TestsInBCContainer -returnTrueIfAllPassed

Hello @freddydk,
I am using Run-TestsInBCContainer to test my integration test functions and I am a little confused about the -returnTrueIfAllPassed parameter
If I use the  Run-TestsInBCContainer without that parameter it shows no output
(if I assign XUnitResultFileName I can check the result file)
If I now add the -returnTrueIfAllPassed parameter it shows the executed codeunits with success/failure and if all functions have passed true at the very end.
I would have expected that the routine shows the executed codeunits without the -returnTrueIfAllPassed parameter and with the parameter only "true" if all functions have passed.
Or I am wrong?
my example:

Standard Passed (no output)
Standard Passed with -returnTrueIfAllPassed

  Codeunit 5043082 Automotive Test Sales Success (115.742 seconds)
True

Standard Failed (no output)
Standard Failed with -returnTrueIfAllPassed

  Codeunit 5043083 Automotive Test Purchase Failure (10.322 seconds)
    Testfunction InboundPurchaseInvoice Failure (4.127 seconds)

navcontainerhelper <IP_ADDRESS>
docker image mcr.microsoft.com/businesscentral/onprem:1910-de-ltsc2019

What other parameters are you using?
Hent Outlook til iOShttps://aka.ms/o0ukef

Fra: Ralf Escher<EMAIL_ADDRESS>Sendt: Friday, October 11, 2019 9:12:37 AM
Til: microsoft/navcontainerhelper<EMAIL_ADDRESS>Cc: Freddy Kristiansen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Emne: [microsoft/navcontainerhelper] Run-TestsInBCContainer -returnTrueIfAllPassed (#660)
Hello @freddydkhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ffreddydk&data=02|01|freddy.kristiansen%40microsoft.com|b9e06c2141f640f2208308d74e1a65cd|72f988bf86f141af91ab2d7cd011db47|1|0|637063747593239558&sdata=PZJ9Vj7jByT1A8c0ztu2qnAi756FrM9tbS%2BkLQ1bB3o%3D&reserved=0,
I am using Run-TestsInBCContainer to test my integration test functions and I am a little confused about the -returnTrueIfAllPassed parameter
If I use the Run-TestsInBCContainer without that parameter it shows no output
(if I assign XUnitResultFileName I can check the result file)
If I now add the -returnTrueIfAllPassed parameter it shows the executed codeunits with success/failure and if all functions have passed true at the very end.
I would have expected that the routine shows the executed codeunits without the -returnTrueIfAllPassed parameter and with the parameter only "true" if all functions have passed. Or I am wrong?
my example:

Standard Passed (no output)
Standard Passed with -returnTrueIfAllPassed

Codeunit 5043082 Automotive Test Sales Success (115.742 seconds)
True

Standard Failed (no output)
Standard Failed with -returnTrueIfAllPassed

Codeunit 5043083 Automotive Test Purchase Failure (10.322 seconds)
Testfunction InboundPurchaseInvoice Failure (4.127 seconds)
navcontainerhelper <IP_ADDRESS>
docker image mcr.microsoft.com/businesscentral/onprem:1910-de-ltsc2019
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fnavcontainerhelper%2Fissues%2F660%3Femail_source%3Dnotifications%26email_token%3DACSGUA4I5GF5NRBSMNOGNM3QOAROLA5CNFSM4I7V63WKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HRECCHQ&data=02|01|freddy.kristiansen%40microsoft.com|b9e06c2141f640f2208308d74e1a65cd|72f988bf86f141af91ab2d7cd011db47|1|0|637063747593239558&sdata=TzhZQGiZXnV5KIKJLIoKk%2FJHljp7oFJZJdLqL4%2F0d9E%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACSGUA6G7X3T3Y3MDQJCES3QOAROLANCNFSM4I7V63WA&data=02|01|freddy.kristiansen%40microsoft.com|b9e06c2141f640f2208308d74e1a65cd|72f988bf86f141af91ab2d7cd011db47|1|0|637063747593249557&sdata=odVIwgEDA%2FKFA1fu6l%2FvCNqeQ6ftq7vVMtMhkw%2B5XD0%3D&reserved=0.

Reason for asking is, that the returntrueifallpassed will only affect one thing - and that is the return value from the function.
So you must have an error in the script - please include your full command, then I can maybe see what you do wrong?

Run-TestsInBCContainer `
    -containerName $containerName `
    -companyName $companyName `
    -testCodeunit $codeunitID  `
    -returnTrueIfAllPassed

Going to use -XUnitResultFileName which doesn't change the behavior.

Are you running as administrator?

No.
Now as administrator:
PS C:\BE-terna\PowerShell> Run-TestsInBCContainer -containerName Industry27 -companyName TESTMANDANT -testCodeunit 5043081
  Codeunit 5043081 Automotive Test Basic Success (0.237 seconds)
PS C:\BE-terna\PowerShell> Run-TestsInBCContainer -containerName Industry27 -companyName TESTMANDANT -testCodeunit 5043081 -returnTrueIfAllPassed
  Codeunit 5043081 Automotive Test Basic Success (0.205 seconds)
True

Got it - that is a bug.
Not that it doesn't display it - but the return value will actually be an array if you use -returntrue and run as non-admin

Fix included in <IP_ADDRESS>


Double click to edit port cell

Testing #117250
Could be useful instead of having to use the context menu.

This is not a pattern we use in other places. For example, double clicking an item in the file explorer doesn't let you rename.

@alexr00 yes, debug: function breakpoints, variables. We are not using it in places where clicking has another action like opening.


[3.x] Refactor layer property editor grid

Implements proposal https://github.com/godotengine/godot-proposals/issues/2770 on the 3.x branch (see #51039 for 4.0), based on different comments and suggestions.
List of changes:

Now able to display up to 32 layers for physics in 4x2 blocks (still 20 layers in 5x2 blocks for render because some layers are reserved)
Also show 32 layers for physics/navigation layer names in project settings
Adjustable grid size to fit available space in dock
Expansion icon to display more layers vertically
Layer numbers in cells to help with selection (cells are also slightly larger to fit the numbers)

For now I haven't implemented the label for layer names like in https://github.com/godotengine/godot-proposals/issues/2770#issuecomment-849099706, because tooltips are already displaying them and it can wait for feedback before doing further improvements if needed.
Physics layers:

Render layers:

Rescaling demo:

I've pushed color changes based on https://github.com/godotengine/godot/pull/51039#issuecomment-889897024 (CC @Calinou ).
In comparison to 4.0, I'm using dark_color_3 instead of font_hover_color for selected cells in order to get better contrast (especially on light themes), due to cells using less of the highlight color.
This is the result with the default theme:

And with a light theme:

For reference, that was the result on a light theme with the same settings as on master:

Looks good to me.
I was just wondering if it could be problematic for UX that the numbers are not contiguous, e.g. when looking at the first row:
 1  2  3  4   9 10 11 12

One alternative could be to write top-to-bottom and then left-to-right like this:
 1  3  5  7   9 11 13 15
 2  4  6  8  10 12 14 16

But I don't know if that's better.

I was just wondering if it could be problematic for UX that the numbers are not contiguous, e.g. when looking at the first row:

I think that contiguous rows is how it is currently, but we still split it in chunks, so it is even more confusing IMO:
 1  2  3  4  5 < -- >  6  7  8  9 10
11 12 13 14 15 < -- > 16 17 18 19 20

So this is actually better for my taste.

Numbers are organized by blocks to keep things consistent when resizing and follow left-to-right reading in each block to make it intuitive in English. It could be inverted for RTL languages, but apart from that I don't see another approach that really works in term of UX.

Thanks!

Wouldn't it be better to use a zero-based numbering, to keep it consistent with bit numbers in set_collision_mask_bit()?
0 1 2 3
4 5 6 7 etc.
I started using Godot 3.4 today, and loved the feature of controlling all 32 layers, but lost half an hour right away because I was working with the wrong bit in code.
Maybe others could get confused as well.
(Sorry if I'm writing here, I didn't feel like to open a full bug report or proposal for such a small thing.)

@nitaku This issue is going to be solved in 4.0 with https://github.com/godotengine/godot/pull/51532, but rather than making the layers 0-based in the inspector (which is harder to read, and especially confusing to non-programmers) the script functions have been renamed and changed to use 1-based indices too.

@pouleyKetchoupp Thanks, that solution will be even better!


Update npm

Can you update npm with the latest release? Thanks.

Thanks for the reminder. I don't know why I keep forgetting to do this!

Sorry for digging this up, but the current npm version is not up to date with the latest release, please take a look!


ClearScreen function corrected

I noticed that we when performed ClearScreen operation on the ishell, sometimes old values from the previous screen were still displayed. This was because in the package github.com/abiosoft/readline, writer.Write([]byte("\033[H")) was used to clear the screen. However, we can simply do writer.Write([]byte("\033[H\033[2J")) to deal with this. It successfully clears the whole screen.
I also eliminated the need for external reference to readline package. We can use the io.Writer object of ishell directly to clear the screen.
I am attaching a screenshot for reference. The "ed ?" at the end of the first line should not be there in case of clearScreen as this was pasted from the previous screen.
#130

Unmaintained repo, so closing


Old File Explorer back on Windows 11

Describe the bug
Hey Chris, Greetings from Brazil.
After using the tool with those options on the images, my File Explorer is back to his Windows 10 version, Without the Tabs e etc.
I've used this combination before but was able to revert the file explorer changes, but this time after trying to re-enable the new File Explorer only broke evertything and i had to reinstall.
To Reproduce
Steps to reproduce the behavior:

Use the Tool with those options on the images in Win 11.
Open File Explorer
Start missing your tabs

Screenshots
The options that i've used.

Additional context
This time I used the tool and didn't try to re-enable, I'm just waiting 😁

I've selected all the options and tryed the "Undo Selected Tweaks" option. No Result.

There is another GitHub Project called Explorer Patcher. It is actively updated. Brings back the the Windows 10 toolbar, solid project and works very well. They are quick to update it with any Windows Updates.

At this point, I just want my Windows 11 Explorer back 😥
But I'll take a look in this Explorer Patcher that u mentioned @Beaverhoosen

@Beaverhoosen Openning this Explorer Patcher fixed my File Explorer, Thank you! The only thing is that I dont know how I remove it or close it, since it dindt install anything, just runs.

Let me re-write my text and now say everything at onde.
@Beaverhoosen Thank You!
For anyone with the same problem, the fix is very simple.

Download and Run the Latest Explorer Patcher -> https://github.com/valinet/ExplorerPatcher
Check if the File Explorer has been restored
Rename the downloaded file from ep_setup.exe to ep_uninstall.exe and run.
Accept the uninstaller
Restart when finished


Broken styles for "Submit Expense" button

Most probably broken by https://github.com/opencollective/opencollective-frontend/pull/2123 (rebased in https://github.com/opencollective/opencollective-frontend/pull/2136)
Example: https://opencollective.com/drupal-diversity-and-inclusion-speaker-initiative

Not sure how that was before, we can easily roll back.


Error in last version (4.1.9) Cannot read properties of null (reading 'removeChild')

Using the "validateMoves.html" example and playing a few moves, the following error occurs:

I have the impression that it only occurs after a capture:
https://user-images.githubusercontent.com/4182336/176690964-7363a54a-8ad0-4521-a15c-ba073adae52c.mp4

I think you must give me an online example page to test, because this uses your forked version, with arrows. The original example seems not to have this problem: https://shaack.com/projekte/cm-chessboard/examples/validate-moves.html


Refactor code to build for Android using cmake

@xanscale, in general, a commit message should be more informative than closes #<number>. I recommend you to follow this pattern, when you also want to close an issue:
<module>: <short description>

<optional-long-description>

Closes #<issue-number>

All the relevant changes merged into measurement-kit/android-libs. Thanks for working on this!


Can`t Link GitHub & Microsoft Account at Login status

Bug Report
Current Behavior
Can`t login using 3rd Party Account.
Steps to Reproduce

Click "Settings",
Click "Security",
Click "GitHub Account Login" or "Microsoft Account Login" button at 3rd Party Account Login.

Expected Behavior
Account is linked
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Flarum version: 1.8.5
Extension version: 1.6.9
Website URL: https://fourm.akarinliu.com
Webserver: [OpenResty]
Hosting environment: [vps]
PHP version: 8.2.15
Browser: [Microsoft Edge 124.0.2478.67 ]

Output of "php flarum info", run this in terminal in your Flarum directory.

/www/sites/forum.akarinliu.com/index # php flarum info
Flarum core: 1.8.5
PHP version: 8.2.15
MySQL version: 8.2.0
Loaded extensions: Core, date, libxml, openssl, pcre, sqlite3, zlib, ctype, curl, dom, fileinfo, filter, hash, iconv, json, mbstring, SPL, session, PDO, pdo_sqlite, bz2, posix, random, readline, Reflection, standard, SimpleXML, tokenizer, xml, xmlreader, xmlwriter, mysqlnd, Phar, exif, gd, imap, mysqli, pdo_mysql, sodium, Zend OPcache
+--------------------------------+------------+--------+
| Flarum Extensions              |            |        |
+--------------------------------+------------+--------+
| ID                             | Version    | Commit |
+--------------------------------+------------+--------+
| flarum-flags                   | v1.8.0     |        |
| fof-oauth                      | 1.6.9      |        |
| flarum-suspend                 | v1.8.1     |        |
| flarum-tags                    | v1.8.0     |        |
| flarum-approval                | v1.8.1     |        |
| xrh0905-oauth-microsoft        | v1.0.1     |        |
| wanecho-showip                 | 0.0.5      |        |
| ianm-html-head                 | 1.2.3      |        |
| fof-user-directory             | 1.3.3      |        |
| fof-upload                     | 1.5.4      |        |
| fof-sitemap                    | 2.2.1      |        |
| fof-pretty-mail                | 1.1.2      |        |
| fof-nightmode                  | 1.5.3      |        |
| fof-custom-footer              | 1.1.2      |        |
| fof-anti-spam                  | 1.1.2      |        |
| flarum-subscriptions           | v1.8.0     |        |
| flarum-sticky                  | v1.8.0     |        |
| flarum-statistics              | v1.8.0     |        |
| flarum-nicknames               | v1.8.0     |        |
| flarum-mentions                | v1.8.3     |        |
| flarum-markdown                | v1.8.0     |        |
| flarum-lock                    | v1.8.0     |        |
| flarum-likes                   | v1.8.0     |        |
| flarum-lang-english            | v1.8.0     |        |
| flarum-lang-chinese-simplified | dev-master |        |
| flarum-emoji                   | v1.8.0     |        |
| blomstra-turnstile             | 0.1.4      |        |
+--------------------------------+------------+--------+
Base URL: https://forum.akarinliu.com
Installation path: /www/sites/forum.akarinliu.com/index
Queue driver: sync
Session driver: file
Scheduler status: 从未运行
Mail driver: smtp
Debug mode: off

Possible solution(s)

Additional Context
Add any other context about the problem here.

What happens when you try? Please share all relevant error messages, logs and screenshots.
Also of relevance could be screenshots of the settings in the GitHub and Facebook apps admin panels, but make sure to redact any secret token.


scss files loading differ between dev and prod.

Bug report
I have an issue with scss files loading :

in dev mode (local) if I not import scss files, I have my page styled (strange).
in prod mode, if not imported, isn't loaded (the correct).
I have this issue in three internal projects.
It seems like in dev mode webpack loads everything.
So when locally testing (dev), everything works, when deploying (prod) no style was loaded, astonished.

What is the current behavior?
scss file loading differ between dev and prod.
If the current behavior is a bug, please provide the steps to reproduce.
Normal steps :

webpack-dev-server (local)
webpack build prod (deployment)

What is the expected behavior?
scss loading should be the same in dev and prod to correctly validate dev before deploying.
Other relevant information:
webpack version:  4.32.2 and 4.41.5.
Node.js version: 10.13.0
Operating System: Windows 10
Additional tools: none
Thank.

Base config :
const merge = require('webpack-merge');

const Dev = require('./webpack.dev');
const Prod = require('./webpack.prod');

const baseConfig = {
  module: {
    rules: [
      {
        test: /\.(js|jsx)$/,
        exclude: /node_modules/,
        use: ['babel-loader'],
      },
      {
        test: /\.scss$/,
        loaders: [
          'style-loader',
          'css-loader',
          'resolve-url-loader',
          {loader: 'sass-loader', options: {sourceMap: true}},
        ],
        exclude: /node_modules\/(?!XXX)/,
      },
      {
        test: /\.css$/,
        use: [
          { loader: 'style-loader' },
          { loader: 'css-loader' },
        ],
        exclude: /node_modules\/(?!XXX)/,
      },
      {
        test: /\.less$/,
        use: [
          { loader: 'style-loader' },
          { loader: 'css-loader' },
          {
            loader: 'less-loader',
            options: {
              javascriptEnabled: true,
            },
          },
        ],
        exclude: /node_modules\/(?!XXX)/,
      },
      {
        test: /\.(jpe?g|png|gif|svg)$/,
        loader: 'url-loader',
        options: {
          limit: 8192,
          name: 'assets/img/[name].[ext]',
        },
      },
      {
        test: /\.(ttf|eot|otf|woff2?)$/,
        loader: 'url-loader',
        options: {
          limit: 8192,
          name: 'assets/fonts/[name].[ext]',
        },
      },
    ],
  },
  resolve: {
    extensions: ['*', '.js', '.jsx'],
  }
};

const devConfig = merge(baseConfig, Dev);
const prodConfig = merge(baseConfig, Prod);

module.exports.devConfig = devConfig;
module.exports.prodConfig = prodConfig;

Webpack dev :
const path = require('path');

module.exports = {
  mode: 'development',
  entry: path.resolve(__dirname, '../src/main/webapp/index.js'),
  output: {
    path: path.resolve(__dirname, '../src/main/webapp/dist'),
    publicPath: '/',
    filename: 'bundle.js',
  },
  devServer: {
    contentBase: '../dist',
    historyApiFallback: true,
    hot: true,
    port: 9000,
  }
};

Webpack prod :
const path = require('path');
const TerserPlugin = require('terser-webpack-plugin');

module.exports = {
  mode: 'production',
  entry: path.resolve(__dirname, '../src/main/webapp/index.js'),
  output: {
    path: path.resolve(__dirname, '../src/main/webapp/dist'),
    publicPath: '/',
    filename: 'bundle.js',
  },
  optimization: {
    minimizer: [
      new TerserPlugin({
        parallel: true,
        sourceMap: true,
        terserOptions: {
          mangle: true,
          output: {
            comments: false,
          },
        },
      })
    ]
  }
};

@helabenkhalfallah Unfortunately, it is very hard to help you without reproducible repo..

Hello @vankop I added my webpack configuration, I think this what you need not ?

I think it is the sideEffects problem (https://webpack.js.org/guides/tree-shaking/)? How you import styles?

Exemple Home Page :
import React, { Component } from 'react';
import '../scss/_reset.scss';
import '../scss/_grid.scss';
import '../scss/_layoutHome.scss';

class HomePage extends Component {
  componentDidMount() {}

  initPage = () => {};

  render() {
    return (

           HomePage

    );
  }
}

export default HomePage;

Package json :
  "sideEffects": [
    "*.scss"
  ],

@helabenkhalfallah Do you read this? After adding sideEffects, it is working?

@evilebottnawi yes :)
I have many stranges behaviors in my internals projects  :
"webpack": "=4.28.4" => side effects = false => I have styles loaded both in dev and prod (like if treeshaking not working), why ?
"webpack": "=4.32.2" => side effects = false => styles are loaded only in local dev, why ?.
"webpack": "=4.32.2" => side effects = 'scss' => styles are loaded in dev and prod.
Why scss file are loaded in local even if side effects is false and I don't import the scss files ? Here is my central issue.
This make not intuitive to find that we must add side effects attributes (because local dev work).
And why using 4.28.4, I have styles loaded in dev and prod ?
Thank you for your help :)

@helabenkhalfallah I think it was bugs, try to use only latest versions

Okay, I will try to update. Thanks.

@helabenkhalfallah Can we close? Do you have more questions?

Yes yes, I wil try to update to the latest version, thank you for your help :)

@helabenkhalfallah feel free to feedback if you will have problems


Feature/python streaming client

Python client with streming mode tested on grpc-gateway mock.
There is still need to add a secure channel.

So far it looks good to me


nc_zarr for openstack cloud

Hi,
I tried the "new" zarr support by installing netcdf with --enable-nczarr. I think I had success because at the end of the config it says:
NCZarr Support:		yes.
However, when I do:
./ncdump -h https://swift.dkrz.de/v1/dkrz_0b2a0dcc-1430-4a8a-9f25-a6cb8924d92b/cmip6-zarr/ScenarioMIP.DKRZ.MPI-ESM1-2-HR.ssp370.AERday.gn.od550aer/#mode=zarr
I get:
NetCDF: Attempt to use feature that was not turned on when netCDF was built.
Or:
./ncdump -h https://swift.dkrz.de/v1/dkrz_0b2a0dcc-1430-4a8a-9f25-a6cb8924d92b/cmip6-zarr/ScenarioMIP.DKRZ.MPI-ESM1-2-HR.ssp370.AERday.gn.od550aer/
syntax error, unexpected WORD_WORD, expecting SCAN_ATTR or SCAN_DATASET or SCAN_ERROR
NetCDF: file not found

. Note that you can open this URL with xarray.open_zarr(url, consolidated=True).
The swift store is an openstack object storage that we use at the German Climate Computing Center.
Best regards and thanks for any help,
Fabi

Can you post your complete ./configure command options you used?

Hi Dennis,

Can you post your complete ./configure command options you used?

./configure --enable-netcdf-4 --enable-nczarr CFLAGS=-fPIC
results in
HAS_NCZARR='yes'
NC_HAS_NCZARR='1'
#define ENABLE_NCZARR 1

Also, what is this URL?

We are using an object storage from Openstack. We had to use  fsspec on the link first but, most recently, xarray can open the URL directly (probably with the power of fsspec). It is a link to the consolidated dataset, you can find a data array under
https://swift.dkrz.de/v1/dkrz_0b2a0dcc-1430-4a8a-9f25-a6cb8924d92b/cmip6-zarr/ScenarioMIP.DKRZ.MPI-ESM1-2-HR.ssp370.AERday.gn.od550aer/od550aer/
where the chunk and the .jsons are, e.g.:
https://swift.dkrz.de/v1/dkrz_0b2a0dcc-1430-4a8a-9f25-a6cb8924d92b/cmip6-zarr/ScenarioMIP.DKRZ.MPI-ESM1-2-HR.ssp370.AERday.gn.od550aer/od550aer/.zattrs
It would be very helpful if NC_ZARR could also cover other object storages or use fsspec similarly. As you can see in this table, swift openstack is, at many points at least, compatible with the s3 interface.
Nevertheless, thank you very much for your work and best regards,
Fabi

I just came across this, as I am trying to figure out how to host (readonly) zarr datasets on generic http servers, which I believe would also solve this issue.
The ability to host zarr on any http server, would open up many more use cases (including access to datasets on non-s3 compliant object stores), however writing would have to happen by other means (e.g. first to a file, then upload to the server).
In order to support this use case, we would likely need another implementation of zmap for http, which would implement at least the methods:

close
exists -> HTTP HEAD
len -> HTTP HEAD
read -> HTTP GET

In addition, an heuristic implementation of search might be possible by walking through index-pages. write could be implemented using the HTTP PUT method, but it's not likely to be supported by most servers. (this is basically what fsspec does in python)
If the hosted zarr datasets contain a listing of the contained variables (e.g. using nczarr or the consolidated metadata extension (.zmetadata) as supported by at least zarr-python and GDAL), it should be possible to read from HTTP using the four straightforward methods alone. If there's no such listing, search would be needed.
@DennisHeimbigner, do you think a zmap implementation for HTTP would be good to have in netCDF? And should netCDF support .zmetadata-style consolidated metadata?

As you can see in this table, swift openstack is, at many points at least, compatible with the s3 interface.

I think this table is meant to shoud functional compatibility, but not API compatibility. I.e. for each of the listed S3 API calls, there is an equivalent in OpenStack SWIFT, but it's spelled differently and thus needs a separate client implementation.


Incorrect energy bounds in "EBOUNDS" HDU in DRM file created with gtdrm

Hi,
I would like to export the detector response matrix from my analysis using the gtdrm tool. It runs ok and generates the desired output as a fits file. The fits file contains several HDUs, the one called DRM contains the energy dispersion matrix as well as the true energy bins. I suspect that the HDU called EBOUNDS should contain the reconstructed energy bounds, i.e., simply a copy of the bounds of the counts cube or source map used to run gtdrm. However, it's an array with the number of bins equal to the number of bins of the spatial binning used in the counts map. The array entries start at 500 and then simply increase linearly. This can't be right.
This is especially important for people who would like to do a combined analysis of Fermi and Cherenkov telescope data using, e.g., gammapy. With this bug, it's not possible to ingest the DRM into gammapy using this function: https://docs.gammapy.org/0.16/api/gammapy.irf.EDispKernel.html#gammapy.irf.EDispKernel
Furthermore, the hidden key word "edisp_bins" is not mentioned in the documentation: https://github.com/fermi-lat/fermitools-fhelp/blob/master/gtdrm.txt
When calling gtdrm the query for it reads "Number of bins to consider energy dispersion for" which is very misleading. It should be something like "Number of energy bins to prepend and append to the reconstructed energy array for the true energy array".
Thanks!

Ok, there was a bug that it was writing the y-pixel centers to the EBOUNDS HDU.   I fixed that, but the default is to only write the EBOUNDS for the original energy binning, not the extra bins.   I wanted to check if you needed the extra bins in the EBOUND HDU, or the original energy bins.
I also fixed the help string the in .par file to ask for the number of "extra bins" on "each side" of the spectrum.
Let me know about the if you need the extra bins in the EBOUNDS HDU and I'll fix it accordingly.

Thanks, Eric.
That should be fine, only the observed energy bins were missing. The true energy bins (with the appended bins on either side) are in the DRM HDU. I'm going ahead and close this issue.


"Failed to parse discovery request" after upgrade 1.1.5

Bug description
After upgrading istio from 1.1.3 to 1.1.5, the cluster seems not in a good state. Following are observed:

HTTPS requests failed with codes such as 400 for both curl command and automate testing scrips. Same script runs good with istio 1.1.3 with good metrics e.g. QOS.
Error message: LibreSSL SSL_connect: SSL_ERROR_SYSCALL
HTTP seem good, but QPS drops significantly  for same cluster same configuration
Error were observed in the log of Ingress SDS pod:

xxx@m-c02x60rajgh7$k logs --tail=100  -n istio-system istio-ingressgateway-5899c86ddb-hfkm6 ingress-sds

2019-05-06T05:58:46.592611Z	error	Failed to parse discovery request: discovery request node:<id:"router~<IP_ADDRESS>~istio-ingressgateway-5899c86ddb-hfkm6.istio-system~istio-system.svc.cluster.local" cluster:"istio-ingressgateway" metadata:<fields:<key:"CONFIG_NAMESPACE" value:<string_value:"istio-system" > > fields:<key:"IDLE_TIMEOUT" value:<string_value:"2m" > > fields:<key:"ISTIO_META_INSTANCE_IPS" value:<string_value:"<IP_ADDRESS>,<IP_ADDRESS>" > > fields:<key:"ISTIO_PROXY_SHA" value:<string_value:"istio-proxy:73fa9b1f29f91029cc2485a685994a0d1dbcde21" > > fields:<key:"ISTIO_PROXY_VERSION" value:<string_value:"1.1.3" > > fields:<key:"ISTIO_VERSION" value:<string_value:"1.1.5" > > fields:<key:"POD_NAME" value:<string_value:"istio-ingressgateway-5899c86ddb-hfkm6" > > fields:<key:"ROUTER_MODE" value:<string_value:"sni-dnat" > > fields:<key:"USER_SDS" value:<string_value:"true" > > fields:<key:"istio" value:<string_value:"sidecar" > > > locality:<> build_version:"73fa9b1f29f91029cc2485a685994a0d1dbcde21/1.11.0-dev/Clean/RELEASE/BoringSSL" > type_url:"type.googleapis.com/envoy.api.v2.auth.Secret"  has invalid resourceNames []

Expected behavior
Expected Https work fine after istio 1.1.5 upgrade
metrics such as QPS are on par of istio 1.1.3
Steps to reproduce the bug

istio 1.1.3 installed in the cluster
upgrade istio to 1.1.5 with using helm

Version (include the output of istioctl version --remote and kubectl version)
➜  ~ istioctl version --remote
client version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Clean", GitTag:"1.1.4-10-g9b6d31b"}
citadel version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b-dirty", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.4-10-g9b6d31b"}
galley version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b-dirty", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.4-10-g9b6d31b"}
galley version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b-dirty", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.4-10-g9b6d31b"}
ingressgateway version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Clean", GitTag:"1.1.4-10-g9b6d31b"}
ingressgateway version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Clean", GitTag:"1.1.4-10-g9b6d31b"}
pilot version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b-dirty", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.4-10-g9b6d31b"}
pilot version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b-dirty", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.4-10-g9b6d31b"}
sidecar-injector version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b-dirty", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.4-10-g9b6d31b"}
sidecar-injector version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b-dirty", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.4-10-g9b6d31b"}
telemetry version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b-dirty", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.4-10-g9b6d31b"}
telemetry version: version.BuildInfo{Version:"1.1.5", GitRevision:"9b6d31b74d1c0cc9358cc82d395b53f71393326b-dirty", User:"root", Host:"3e29fde4-6c3f-11e9-b00d-0a580a2c0205", GolangVersion:"go1.10.4", DockerHub:"docker.io/istio", BuildStatus:"Modified", GitTag:"1.1.4-10-g9b6d31b"}
➜  ~
app@cpeus567b000000:~$ kubectl version
Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.5", GitCommit:"2166946f41b36dea2c4626f90a77706f426cdea2", GitTreeState:"clean", BuildDate:"2019-03-25T15:26:52Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.5", GitCommit:"2166946f41b36dea2c4626f90a77706f426cdea2", GitTreeState:"clean", BuildDate:"2019-03-25T15:19:22Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}
How was Istio installed?
use helm
Environment where bug was observed (cloud vendor, OS, etc)
Azure cluster
Linux
Affected product area (please put an X in all that apply)
[ x] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ x] Security
[ x] Test and Release
[ ] User Experience
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

https://github.com/istio/istio/blob/2a39a33f0b06d865e4df52a0594973baf6ada405/security/pkg/nodeagent/sds/sdsservice.go#L290-L300
This requires that resourceNames is not empty, when https://github.com/istio/istio/blob/009730f1abc86ee10d530022c5d79e18fe31fa37/vendor/github.com/envoyproxy/go-control-plane/envoy/api/v2/discovery.pb.go#L45-L51 suggests that an empty request is valid

There are no significant changes to ingress gateway agent between 1.1.3 and 1.1.5. Perhaps 1.1.5 upgrades Envoy version that introduces the issue?

@twu8, a few question -

does HTTPS fail every time or intermittently (like happened only run some time, or only when you recreate k8s secret) ?
do you have nodeagent logs ?

@JimmyCYJ the same version of Envoy is used in 1.1.3, 1.1.4. and 1.1.5

@duderino Thanks Joshua. Then let's check ingress-sds and ingress-proxy logs to figure out what happened.

/cc @incfly

https://github.com/istio/istio/pull/14080 fixes the issue. Shall we close it now?

@JimmyCYJ my fix has nothing to do with sds, did you link the wrong pr?

The SDS issue is there is a race condition at node agent when Gateway CRD is deleted and k8s secret is deleted. @philrud's script could reproduce that, and the node agent fix is to address that issue.
This issue is not about sds, it is a config push that causes listener stuck in warming state during upgrade. If #14080 is not the fix, would you mind to add a link to the right fix?

Maybe I am misunderstanding this issue, but it seems it is about Envoy sending a request for empty list of ResourceName and sds responds with an error?
I highly doubt this issue is related to 14080 which fixes an issue where you have two gatways with overlapping host names in the https field created in the same second (pretty small group of people I would imagine)?
Maybe k missed something though

Fixed in https://github.com/istio/istio/pull/14305


Add English Holidays

This adds the English & Welsh holidays according to the Gov website: https://www.gov.uk/bank-holidays

This doesn't account a bank holiday falling on a weekend, a ‘substitute’ weekday becomes a bank holiday, normally the following Monday.

That is a good point, given that the Welsh and English holidays are the same it's probably worth just using your PR #75. Not sure on the best way to support en and cy without duplicating all the code though.

That is a good point, given that the Welsh and English holidays are the same it's probably worth just using your PR #75. Not sure on the best way to support en and cy without duplicating all the code though.

I've added English holidays to PR #75.
<?php

namespace Spatie\Holidays\Countries;

class England extends Wales
{
    public function countryCode(): string
    {
        return 'en';
    }
}

Use PR #75 as is more complete


multithread safe

the same as the client, if the server component works on separate thread he can't change the GUI and so we must call Invoke method to "jump" in the correct thread that can modify the GUI

Thank you @ozrecsec !


[Bug][High] Db query inserts don't work if there is a quote in the value of the binding

Hey @prapullac / @Nikhil-Nandagopal
Can some more info can be provided on this ?

This is fixed as part of Prepared Statement


Add a Gitter chat badge to readme.md

Metrolog/Font.GOST2.304-81 now has a Chat Room on Gitter
@sergey-s-betke has just created a chat room. You can visit it here: https://gitter.im/Metrolog/Font.GOST2.304-81.
This pull-request adds this badge to your readme.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Включил в #232


[Release] Bumped snmp version to 7.0.0

What does this PR do?
Release snmp v7.0.0
Motivation
https://github.com/DataDog/integrations-core/pull/15800
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

Changelog update to happen in master. Closing this pr to accommodate it. A new pr will be created with relevant changes (changelog included).


Activate the GitHub App Stale on the GitHub repository

Pull Request (PR) description

This Pull Request (PR) fixes the following issues:

Task list:

[ ] Change details added to Unreleased section of CHANGELOG.md?
[ ] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[ ] Examples appropriately updated?
[ ] New/changed code adheres to Style Guidelines?
[ ] Unit and (optional) Integration tests created/updated where possible?

This change is

Codecov Report

Merging #166 into dev will not change coverage.
The diff coverage is n/a.

@@        Coverage Diff         @@
##           dev   #166   +/-   ##
==================================
  Coverage   90%    90%
==================================
  Files        7      7
  Lines      691    691
==================================
  Hits       623    623
  Misses      68     68

Reviewed 1 of 1 files at r1.
Review status: :shipit: complete! all files reviewed, all discussions resolved

Comments from Reviewable


ide can't be started!!

here is my operation:
1.open two tabs to run npm-script  dev and dev-start  separately.
but it seems to get sucked in the tab which runs dev-start,want your help,thanks.

Could you explain more clear what is npm-script dev and dev-start in your project?

@constgen  dev and dev-start in package.json

Tried your steps in VSCode. I opened 2 terminals inside the code editor and run commands in the same order as you. Nothing stacked. After 20-30 sec I saw the Electron app running on my screen.

Failed to fetch extension, trying 4 more times

@constgen this is what i got when i run the dev-start in terminal

I use Windows and yarn.


Enhancement request: A way to get a thread ID from a function?

Is there a way to implement a mechanism which would allow me to get a thread ID (from 0 to (N-1)) in a function emplaced in the thread pool?

Check out ThreadIdentifiers test in thread_pool_test.cpp.
Best regards,
Robert


FIX #59 - always determine executable tests via annotation API before executing tests

Pro:

ensures that non-tests procedures are executed

Cons:

leads to a delay before starting a test (delay is small, even on large schemas)
delay might be huge for large schemas and empty annotation cache (corner case)

The delay issue will get addressed for huge schemas (like EBS) with optional DDL trigger in utPLSQL install (PR is still in progress - needs more testing and documentation update).
utPLSQL/utPLSQL#901
See also: utPLSQL/utPLSQL#631


Show users invited to an organization

Is your feature request related to a problem? Please describe.
When you invite a user to an organization, there is no indication that the invitation was sent. You're not sure if you did it right, or if you looked away and then forgot whether you sent it or not.
Describe the solution you'd like
Invited users who have not accepted should show up in the list of members, at least to maintainers, and possibly to all members of the org.
Optionally or instead, there could be a message that the invite was sent. But if we do the previous suggestion, unless there is a huge number of members, the previous paragraph is probably sufficient.
┆Issue is synchronized with this Jira Story
┆Epic: Organization Improvements
┆Fix Versions: Dockstore 2.X
┆Issue Number: DOCK-1381
┆Sprint: Backlog
┆Issue Type: Story

Just ran into this and found it very disconcerting, but maybe it wasn't showing uninvited numbers in the count before (says there are two members but it's only showing one).


my_configs.vim file not loading

This may be pretty basic (new to Vim), but the my_configs.vim file I created (full path: ~/.vim_runtime/my_configs.vim) is not being incorporated into the .vimrc file.
Specifically I put in the my_configs.vim file the command
inoremap kj <ESC>
and seperately
imap kj <ESC>
saved the file and saw no effect. I restarted the computer and even tried \e and typing jk never brought me back into normal. To test that there was no problem with the command itself, I edited the .vimrc file to include the command, saved, and it worked like a charm.
Not sure if this is an error on my side or with a recent version of the program. Help would be greatly appreciated! You have a great .vimrc style.

make sure your .vimrc file has this content:
try
source ~/.vim_runtime/my_configs.vim
catch
endtry

Thanks, that did it.

@Tiande Mine has the same content, but still no luck. After modifying ~/.vim_runtime/my_configs.vim I'm using this command to source my_configs sh ~/.vim_runtime/install_basic_vimrc.sh. Am I missing something? Can you point me in the right direction?

@iamrahulroy
The command sh ~/.vim_runtime/install_basic_vimrc.sh can't source my_configs.vim.
Just make sure your .vimrc file has this content:
source ~/.vim_runtime/vimrcs/basic.vim
source ~/.vim_runtime/my_configs.vim

It will work for you.


fix(ElementalPageExtension): Update getElementsForSearch() so that it…

Update getElementsForSearch() so that it switches to the frontend theme. This is to fix a regression from SilverStripe 3.X when HeyDay/Elastica is configured to write the ElementsForSearch field.
Also as noted in the following PR https://github.com/dnadesign/silverstripe-elemental/pull/343, the implementation of switching themes is technically incorrect. The only reason I noticed for this PR is that if you don't set the theme back, the CMS layout breaks after a user presses "Save" in the CMS. (its as if the flex boxes stop applying spacing, everything shifts next to each other)
Needs after merge

Can this please be cherry picked into 2.X branch and tagged for SilverStripe 4.1 support please?

Config:
---
Name: extensible-elastic-config
---
Page:
  indexed_fields: &page_defaults
    -
      ClassName:
        type: keyword
    - Title
    - Content
    -
      LastEdited:
        type: date
    -
      ElementsForSearch:
        type: text

Can this please be cherry picked into 2.X branch and tagged for SilverStripe 4.1 support please?

It might be easier if you retarget the pull request at a 2.x branch, then we can merge it up

@robbieaverill Which branch do you want me to target? 2, 2.0 or 2.1?
Will my changes also go into 3 / master?
Once you reply, I'll work on this. I'll also combine #343 into this PR.

Target either 2.0 or 2.1. All changes made in earlier branches are merged up. 2.0 -> 2.1 -> 2 (this branch is for a future 2.2 release) -> 3 (for 3.x releases) -> master (for 4.x releases).

I'd say 2.1

Agreed with 2.1
Looks like the code being patched in #343 doesn't even exist in 2.0 (it was introduced in 2.1)

@robbieaverill Done :)


Why does IDbSet does not define async methods from DbSet?

Regarding to IDbSet , the EF documentation states the following:

"This context works in exactly the same way as the context that uses the DbSet class for its set properties."

That is far from true if you use the async methods on DbSet, which aren't defined in IDbSet. Why exactly doesn't IDbSet define the async methods? Should it?
From a SOLID perspective, and in particular the last letter of the acronym, it would be more logical to define sets using IDbSet, but that would deny you access to the async features.

@mstaessen See https://entityframework.codeplex.com/wikipage?title=Design Meeting Notes - May 16%2C 2013

Thanks for the pointer. Hadn't found it.
The documentation though could be more explicit about the preference if DbSet over IDbSet. Does the old EF documentation take pull requests?

@mstaessen We take pull requests for the inline XML documentation on the classes. Not sure about the other docs. @rowanmiller?

@mstaessen Our current docs on testing don't show IDbSet (https://msdn.microsoft.com/en-us/library/dn314429(v=vs.113).aspx and https://msdn.microsoft.com/en-us/library/dn314431(v=vs.113).aspx). Was there another doc that you thought should be updated? As Arthur mentioned, you can send a PR for the API documentation, as it is maintained inline in the code.

I think this doc https://msdn.microsoft.com/en-us/library/jj592675(v=vs.113).aspx#Anchor_1 might need an update. It states

There a situations, such as when creating mocks or fakes, where it is more useful to declare your set properties using an interface. In such cases the IDbSet interface can be used in place of DbSet. [...] This context works in exactly the same way as the context that uses the DbSet class for its set properties.

but that would not be true if your code uses the async functions already.

@mstaessen agreed, I opened https://github.com/aspnet/EntityFramework.Docs/issues/360 to track this


[Snyk] Upgrade @nuxtjs/pwa from 3.0.0-beta.20 to 3.3.5

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @nuxtjs/pwa from 3.0.0-beta.20 to 3.3.5.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 15 versions ahead of your current version.
The recommended version was released 2 years ago, on 2021-01-26.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

👷 Deploy Preview for benevolent-blini-5169f6 processing.

Name
Link

🔨 Latest commit
017fb241c38ee6aed5f3b77f52da4687fab182b5

🔍 Latest deploy log
https://app.netlify.com/sites/benevolent-blini-5169f6/deploys/645fbd51390a0b0008d5f10d


Add more tests for ol.coordinate.

Please review.

LGTM.

I added another test, and will merge once travis is happy, if you don't disagree.

Still looks good to me.

:bowtie:


Import to Altium error

I'm trying to import the files into Altium Designer using the import wizard but the .PcbDoc keeps showing up as empty. The schematics import fine. I get the following logs after the import is complete. Do you have Altium files available / has anyone else had this issue with these files?
csi.SchLib.LOG
ethernet.SchLib.LOG
hdmi.SchLib.LOG

No, the project files are meant to be used in Kicad (https://en.wikipedia.org/wiki/KiCad). Obviously it is technically possible to import those designs to other tools but this is beyond the scope of this repository / issue tracker -- its meant for reporting bugs / problems with the design only.


[Discover] solution: add snap test

add test && add html tag

Coverage increased (+0.8%) to 55.586% when pulling 88e3f1cdf66eebce3a1c858e6d2646987cda2732 on travelgeezer:solution into cf55756b1d3cfebe3d3c428a236940d0941fafea on phodal:master.


undefined symbol: SuiteSparse_config

I'm using numpy 1.9.2, scipy 0.16.0, Python 2.7.10, SuiteSparse 4.4.5 and scikit-umfpack from github (to avoid the lib.six problem from #11).
In numpy's site.cfg, I followed the instructions on http://rc.github.io/scikit-umfpack/install.html
However, entering
  from scikits import umfpack

results in
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "[...]/scikit-umfpack/0.2-intel-2015b-Python-2.7.10/lib/python2.7/site-packages/scikits/umfpack/__init__.py", line 12, in <module>
    from .umfpack import *
  File "[...]/scikit-umfpack/0.2-intel-2015b-Python-2.7.10/lib/python2.7/site-packages/scikits/umfpack/umfpack.py", line 130, in <module>
    from . import _umfpack as _um
  File "[...]/scikit-umfpack/0.2-intel-2015b-Python-2.7.10/lib/python2.7/site-packages/scikits/umfpack/_umfpack.py", line 28, in <module>
    __umfpack = swig_import_helper()
  File "[...]/scikit-umfpack/0.2-intel-2015b-Python-2.7.10/lib/python2.7/site-packages/scikits/umfpack/_umfpack.py", line 24, in swig_import_helper
    _mod = imp.load_module('__umfpack', fp, pathname, description)
ImportError: [...]/scikit-umfpack/0.2-intel-2015b-Python-2.7.10/lib/python2.7/site-packages/scikits/umfpack/__umfpack.so: undefined symbol: SuiteSparse_config

Adding the directory where libsuitesparseconfig.a is located (in SuiteSparse_config in the SuiteSparse directory) to site.cfg doesn't help.
Any idea how to fix this?

Hi @sbecuwe, could you try adding the missing library into umf_info['libraries'] in scikits/umfpack/setup.py? Similarly to librt, see #10.

It turned out that by reading Stefan Fürtinger's instructions, I got it working. You could maybe update the url to his page: http://fuertinger.lima-city.de/research.html seems to be the up-to-date version.

OK, thanks for the info. BTW. after a quick glance at the original link, I do not see differences in the instructions - is something wrong in the ones in the original url? Both pages have the same date of the last update, so IMHO they are in sync.
So, for posterity, what was the actual problem?

There are some minor (typographical) changes/fixes on the new page which show pages are not in sync. The new url is also his current home page.
The actual problem: the shared libs were missing in my SparseSuite.

OK, I have added PR #20 - could you review? :)

Done! Checked url ;-) Thanks for the update.

It's merged. Thanks, and let us close this.


link to "Zafira" in generated email has old value

inside generated testrun email we have old zafira link which refers to
host/#!/zafira/....
it should be adjusted according to the new "/app" context root.
Also please scan all code on hardcoded "#!"  values to make sure we don't forget about it

fixed in 1.78-SNAPSHOT
released 3.4.52 should include it


gdb: 8.1 -> 8.1.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/gdb/versions.
meta.description for gdb is: '"The GNU Project debugger"'.

Checks done (click to expand)

built on NixOS
/nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1/bin/gdb passed the binary check.
/nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1/bin/gdbserver passed the binary check.
/nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1/bin/gcore passed the binary check.
3 of 3 passed binary check by having a zero exit code.
0 of 3 passed binary check by having the new version present in output.
found 8.1.1 with grep in /nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1
directory tree listing: https://gist.github.com/2805a0e79734953add53a2ee1d88a44a
du listing: https://gist.github.com/d812dcc94390f0a6d3fd7678042414bc

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout b1f1888bdd87b91765ab2d103f8e3b387d1a6c4a && nix-shell --pure -I nixpkgs=. -p gdb

cc @nbp for testing.

part of https://github.com/NixOS/nixpkgs/pull/44564

Success on aarch64-linux (full log)
Attempted: gdb
Partial log (click to expand)
shrinking /nix/store/98pa9pmxg8526hpqfwxqjg2xjhv71vfw-gdb-8.1.1/bin/gdbserver
shrinking /nix/store/98pa9pmxg8526hpqfwxqjg2xjhv71vfw-gdb-8.1.1/bin/gdb
shrinking /nix/store/98pa9pmxg8526hpqfwxqjg2xjhv71vfw-gdb-8.1.1/bin/run
gzipping man pages under /nix/store/98pa9pmxg8526hpqfwxqjg2xjhv71vfw-gdb-8.1.1/share/man/
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/98pa9pmxg8526hpqfwxqjg2xjhv71vfw-gdb-8.1.1/lib  /nix/store/98pa9pmxg8526hpqfwxqjg2xjhv71vfw-gdb-8.1.1/bin
patching script interpreter paths in /nix/store/98pa9pmxg8526hpqfwxqjg2xjhv71vfw-gdb-8.1.1
/nix/store/98pa9pmxg8526hpqfwxqjg2xjhv71vfw-gdb-8.1.1/bin/gcore: interpreter directive changed from "/bin/sh" to "/nix/store/7q3ayirslrcva28wava6zpjcflcz1h3b-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/98pa9pmxg8526hpqfwxqjg2xjhv71vfw-gdb-8.1.1...
/nix/store/98pa9pmxg8526hpqfwxqjg2xjhv71vfw-gdb-8.1.1

Success on x86_64-linux (full log)
Attempted: gdb
Partial log (click to expand)
shrinking /nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1/lib/libinproctrace.so
shrinking /nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1/bin/gdbserver
shrinking /nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1/bin/gdb
gzipping man pages under /nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1/share/man/
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1/lib  /nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1/bin
patching script interpreter paths in /nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1
/nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1/bin/gcore: interpreter directive changed from "/bin/sh" to "/nix/store/56nrxy58wbhvs2sy3rir1jqa68p0kkm5-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1...
/nix/store/qcmzz759wggb6mkda18a7g1cafl2s8bd-gdb-8.1.1

Success on x86_64-darwin (full log)
Attempted: gdb
Partial log (click to expand)
make[3]: Leaving directory '/private/tmp/nix-build-gdb-8.1.1.drv-0/gdb-8.1.1/gdb'
make[2]: Leaving directory '/private/tmp/nix-build-gdb-8.1.1.drv-0/gdb-8.1.1/gdb'
make[1]: Nothing to be done for 'install-target'.
make[1]: Leaving directory '/private/tmp/nix-build-gdb-8.1.1.drv-0/gdb-8.1.1'
removed '/nix/store/18sn36awml907glqcp28ax7rfgzb4d6g-gdb-8.1.1/share/info/bfd.info'
post-installation fixup
gzipping man pages under /nix/store/18sn36awml907glqcp28ax7rfgzb4d6g-gdb-8.1.1/share/man/
strip is /nix/store/251fvx4mx9q4zarbca5cciasmn66p668-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/18sn36awml907glqcp28ax7rfgzb4d6g-gdb-8.1.1/bin
patching script interpreter paths in /nix/store/18sn36awml907glqcp28ax7rfgzb4d6g-gdb-8.1.1


Breaking Security Fix in Upstream Dependency (cryptography)

Hello,
A vulnerability scan flagged the upstream dependency cryptography (pycognito -> pyjwt[crypto] -> cryptography) that is fixed for versions > 42.0.4. See: https://github.com/advisories/GHSA-6vqw-3v5j-54x4
This update to cryptography causing breaking changes where:
from pycognito import Cognito

idToken = '<idToken>'
accessToken = '<accessToken>'

userPoolId='<userPoolId>'
clientId = '<clientId>'
region='<region>'

u = Cognito(
    user_pool_id=userPoolId,
    client_id=clientId,
    user_pool_region=region,
    id_token=idToken,
    access_token=accessToken
)
u.verify_tokens()

Results in:
>> TypeError: argument 'data': from_buffer() cannot return the address of a unicode object

with the error occurring upstream in jwt
If you attempt to pass the tokens in as bytes you get:
>> TypeError: a bytes-like object is required, not 'str'

Where the error occurs within pycognito.
I have created a PR to fix this breaking change by handling the cases at these two points of failure: #222

Seems like an important fix to get into main. @pvizeli

I just hit this issue too.


sync big files to azure storage container:  uncommitted block count exceeded

Which version of the AzCopy was used?
azcopy version 10.12.2 on Ubuntu 18.04
What command did you run?
.../azcopy sync "/datadisk/pgbackrest/log" "https://url+SAS" --put-md5
What problem was encountered?
ERR: [P#0-T#0] UPLOADFAILED: /path/to/file : 409 : 409 The uncommitted block count cannot exceed the maximum limit of 100,000 blocks.. When Staging block. X-Ms-Request-Id: 57d4adc9-601e-001a-4d9d-db9e48000000
How can we reproduce the problem in the simplest way?
azcopy sync big files to azure storage container
What I am trying is to copy the backup repos + logs of my postgres backup solution to a storage container as another backup layer.

Hey @dimidir ,
Thanks for reaching out and I apologize for not replying earlier.
What is the maximum size of file you are trying to transfer? Can you send me the log file or complete snippet of error from log file. I want to understand the content-length we're transferring in Put Block call.
As a mitigation step, can you manually try setting block-size-mb to large value (Upto 4000MiB)
https://docs.microsoft.com/en-us/rest/api/storageservices/put-block#remarks

Please reopen the issue with more information.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/260 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

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

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

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


Ubuntu 24.04 support

Have a customer requesting this.
Refs #6550

Dupe of #4883.

Good catch, I thought we had something open for this already.


fr: yeux

Missing IPA:
jø

@elijahsimkin Do you perhaps have an old version of the project repo? The entry for yeux has been included in the fr_FR dictionary since this commit in 2018.


For #800: Fix FtRemote for empty response body test case and unignore returnsAnEmptyResponseBody.

For #800:

Fixed test cases for empty response body;
Removed @todo and @Ignore from tests

Job #855 is now in scope, role is REV

This pull request #855 is assigned to @g4s8/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @yegor256/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be a monetary reward for this job

@g4s8 answered, please take a look

@g4s8 @t-izbassar As asked, I've reverted the changes and added an issue to jcabi-http project, and changed our issue so it can only be fixed after we deal with the problems in assertBody in jcabi-http. I've reverted pom.xml changes because although I think that they're pertinent and important they does not belong to this ticket. Please take a look now and say me what you think.

Codecov Report

Merging #855 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #855      +/-   ##
============================================
- Coverage     77.19%   77.17%   -0.03%
+ Complexity      998      997       -1
============================================
  Files           224      224
  Lines          4731     4731
  Branches        380      380
============================================
- Hits           3652     3651       -1
  Misses          918      918
- Partials        161      162       +1

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/org/takes/http/MainRemote.java
53.33% <0%> (-1.67%)
6% <0%> (-1%)

Continue to review full report at Codecov.

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

@t-izbassar I don't know if it fixes https://github.com/yegor256/takes/issues/712, I'm trying to  find why https://github.com/yegor256/takes/issues/800 is happening and these classes are not behaving accordingly.
I've not tried with another http client either, but I've tracked the problem to the jcabi-http library and documented it in ticket comments and in PR comments too. I've spent way more than 30 minutes in this ticket, mostly in review, and this issue is clogging my agenda for almost a month now.
If the presented solutions are not enough, please tell me precisely what would I do so we could close this ticket. Takes tickets are stale in my agenda and are preventing me for making money, so please help me to get rid of this one.

The architect of the project has changed; @yegor256/z is not at this role anymore; @paulodamaso/z is the architect now

@g4s8 @t-izbassar ping, what we need to get this one moving?

@paulodamaso as long as you are an architect now - it’s your call to make. I’ve already expressed my view. I don’t think that issue with jcabi-http is connected to the original problem. At least there is no test added that proves it. You haven’t checked if problem remains with different http client so from my perspective - this PR do not solve anything.

@elenavolokhova/z please review this job completed by @g4s8/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

Code review was too long (21 days), architects (@paulodamaso) were penalized, see §55

The job #855 is now out of scope

Payment to ARC for a closed pull request, as in §28: +10 point(s) just awarded to @paulodamaso/z

@g4s8 According to our Policy:

The code reviewer found at least three problems in the code

Only two issues were found during this review.
Please, confirm that you will try to find at least three issues next time.

@0crat quality acceptable


Opentrons App is missing UI feedback when module firmware update is in progress

Overview
When you update module firmware through the Opentrons App, the user experience can be a little confusing because of the lack of an activity indicator.
Steps to reproduce

In the Opentrons App, go to an OT-2 that's connected to a Heater-Shaker that has an outdated firmware version.

You could probably test this with other modules, too. According to @sanni-t, the Heater-Shaker has an especially long firmware upload time (several seconds), so the problem is more obvious.
At the time of writing, (v6.0.0), this test is kind of a pain to set up with a Heater-Shaker because we don't yet have full end-to-end support for updating its firmware? You have to scp some files onto the robot to get it to work.

Update the module's firmware through the Opentrons App.

I forget exactly how this works, but I think it's through the module's three-dot menu on the robot's page?

Current behavior
Working from my memory:

After you select the update button, the process will start on the robot, but there will be no visible change in the Opentrons App.
Then, the module will disappear for another moment.
Finally, the module will reappear with its updated firmware.

Expected behavior
Probably, after selecting the update option, the app should show an activity indicator like a spinner until the firmware update is complete and the updated module comes back online.

Just updated a Heater-Shaker. In the app I briefly got a spinner (less than 1 second), but the module card disappears as soon as the module starts to reboot.


After each execution aiidalab command leaves apps_meta.sqlite file

Anytime I run:
aiidalab

It leaves apps_meta.sqlite file in the current folder.

this is no longer the case