feat: support database snapshot

Adds support for the snapshot command and other commands that rely on database snapshots being supported. It does so by injecting a custom SnapshotGeneratorFactory using reflection, as there is no API support for doing this. The injected factory will only execute custom logic if the database is a Cloud Spanner database. For all other databases the original factory will be used.

I think the biggest concern is what it should look like long term - if we could get a feature request in Liquibase to disable unique constraints (or even just automatically disable them of the table doesn't exist in the information schema) then that would be good.
I'll look again Monday.

I think the biggest concern is what it should look like long term - if we could get a feature request in Liquibase to disable unique constraints (or even just automatically disable them of the table doesn't exist in the information schema) then that would be good.

If we could have some way of controlling / disabling it through some kind of supportsUniqueConstraint method in Liquibase, that would indeed be a lot better than this.


fixed clearfix problem on mobile, style of drodown menu

menu on mobile was broken because of floating, fixed
drowndown menu style was not working with wrong selectors, fixed
overrided max height of navbar on mobile (+20px) so no scroll on it

This thing throws warning on Firefox that 'empty string is passed to document.getElementById', I suspect that this is known problem of bootstrap 🤔


IE 11 Compatibility

Hello,
I found out that in IE 10/11, if the content height is greater than panel height, it will overlap the panel and scrollbar is missing too...Could you please take a look at that?
Many thanks,

This is no a bug of current control!
This would happen when you had a min/max height for the wizard, and you need to modify you css to make the scrollbar x and y compatible with all browsers.


Fix product parameter help message

Notes
Currently mypy_boto3-builder -h shows these choices for --product parameter:
{boto3,boto3_services,boto3_docs,aiobotocore,aiobotocore_services,aiobotocore_docs,aioboto3,aioboto3_docs} [{boto3,boto3_services,boto3_docs,aiobotocore,aiobotocore_services,aiobotocore_docs,aioboto3,aioboto3_docs} ...]

But it really expects input with - instead of _. This PR should fix it.
Type of change

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

Checklist
All of these are optional:

[x] I have performed a self-review of my own code
[x] I have run ./scripts/before_commit.sh to follow the style guidelines of this project
[x] I have tested my code changes

Thank you for the PR!


Simp fails to see proof irrelevance

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Check that your issue is not already filed.
Reduce the issue to a minimal, self-contained, reproducible test case. Avoid dependencies to mathlib4 or std4.

Description
A call to simp fails to close a goal, whereas the associated rw succeeds.
Context
@[simp] theorem imp_self {a : Prop} : (a → a) ↔ True := ⟨fun _ => trivial, fun _ => id⟩

example {P : Prop} : P → P := by simp only [imp_self]

example {α : Prop} {P : α → Prop} {f : ∀ {a}, P a → α} {a : α} : (h : P a) → P (f h) := by
  simp only [imp_self]

Zulip discussion: https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/Another.20kind.20of.20simp.20regression. The equivalent in Lean 3 succeeds. Thanks to @eric-wieser for minimising. This showed up in porting the exponential-ramsey project.
Steps to Reproduce
Code as above.
Expected behavior: The simp call should succeed and close the goal.
Actual behavior: The simp call fails, with error simp made no progress.
Versions
Lean (version 4.3.0-rc1, commit baa4b68a7192, Release)
MacOS
Additional Information
None
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

This is probably an issue in the discrimination tree module. It tries to index non dependent arrows, but then misses the possible rewrite on the dependent arrow in the goal.


overriding json encoder

Description
Overriding json_encoder is not propagated everywhere.
Expected behaviour
connexion is using same dump encoder
Actual behaviour
It uses flasks
Steps to reproduce
We have a custom encoder for json to encode enum.
So using
      gender:
        type: string
        enum:
          - m
          - f

I got error TypeError: <Gender.m: 'm'> is not JSON serializable
According to docs, I overriden default encoder:
application = connexion.App(__name__)
application.app.json_encoder = CustomJSONEncoder

but this introduces new problem:
On instance['passengers'][0]['gender']:\n    None
For some reason Flask json encoder dumps enum as null. This happens in /lib/python2.7/site-packages/connexion/apis/flask_api.py:_jsonify_data.
flask.json.dumps(data, indent=2) returns "gender": null.
flask.json.dumps(data, indent=2, default=custom_encoder) returns "gender": "f" as expected.
(custom_encoder is function used in CustomJSONEncoder as default method.)
Additional info:
Output of the commands:

python --version
Python 2.7.13
pip show connexion | grep "^Version\:"
Version: 1.1.14

@lucas03 sorry for the delay in responding. Could you provide a more complete example for reproduction, including the CustomJSONEncoder?

Thanks for replying, I can try to create a minimal example of connexion app with this issue, in few days I guess.
This is CustomJSONEncoder:
class CustomJSONEncoder(FlaskJSONEncoder):

    def default(self, o):  # pylint: disable=E0202
        json.default_encoder(o)

where default_encoder(o) would be something like this:
def default_encoder(obj):  # pylint: disable=method-hidden
    if type(obj) == Decimal:
        return str(float(obj))

    if hasattr(obj, 'isoformat'):
        return str(obj)

    if isinstance(obj, enum.Enum):
        return obj.name
    raise TypeError(repr(obj) + " is not JSON serializable")

@lucas03 thanks for the detailed response, I've verified that you're correct. Would you like to contribute a a fix by opening a PR? If you do not wish to do so, I can assign this to myself.

Yes, I can open a PR. But is overriding init best way how to solve this?

@lucas03 I had to recheck this as I felt that overriding the __init__ did not sound like the approach we should take. I think that you're missing a return in CustomJSONEncoder.default(), currently it calls default_encoder() and returns a None
class CustomJSONEncoder(FlaskJSONEncoder):
    def default(self, o):
        return default_encoder(o)

Oh my god, thanks. :(

@lucas03 no worries, it was a sneaky issue :)


Wepapi async

Implemented all Async methods
All query parameters re-written with the new SpotifyWebBuilder._builder methods

Not sure how to solve the marked Todo items.

Hey, big thanks! :+1:
I totally forgot to include await and async in the first method, thats why the TODO items didn't compile. Sorry!
It should be okay now, will have a another look at it tomorrow and release it afterwards.
Side-Note:
Also implemented some small helper Functions Next and Previous for Paging-Objects

Ofcoarse! lmao :+1:


Peer to peer code review

Hi @emmanuelsteven  you've done really amazing work but I think there's something you've to work on it while moving forward

I can see you have an image element above your body element and I think it's not a best practice also I can see you have wrap code in the div container with a class of full-wrap and I think its not a best practice

Your code is looking good Steven it was clean and you just need to add alt text for images.

thanks you guys for your kind review i would make these corrections


Add bounds check when trimming SIP number

@lukasIO Should your change (#646) go first, so that we don't have to sync the version number again?


Node package doesn't work with ESM modules

I just noticed this problem on a project I have where I have specificed "type": "module" in the package.json. With for example @parcel/css importing and running the native binary works fine, but for @minify-html/js I get the following error:
node:internal/errors:464
    ErrorCaptureStackTrace(err);
    ^

TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".node" for /home/sondre/Code/web/node_modules/.pnpm/@minify-html+js<EMAIL_ADDRESS>    at new NodeError (node:internal/errors:371:5)
    at Object.file: (node:internal/modules/esm/get_format:72:15)
    at defaultGetFormat (node:internal/modules/esm/get_format:85:38)
    at defaultLoad (node:internal/modules/esm/load:22:14)
    at ESMLoader.load (node:internal/modules/esm/loader:359:26)
    at ESMLoader.moduleProvider (node:internal/modules/esm/loader:280:58)
    at new ModuleJob (node:internal/modules/esm/module_job:66:26)
    at ESMLoader.#createModuleJob (node:internal/modules/esm/loader:297:17)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:261:34)
    at async ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:81:21) {
  code: 'ERR_UNKNOWN_FILE_EXTENSION'
}

Node.js v17.3.0

Instead of being able to do import * as minify from "@minify-html/js" I have to do
import * as minify from "@minify-html/js"
import { createRequire } from "module";
const require = createRequire(import.meta.url);
const mf = require("@minify-html/js") as typeof minify;

I have no idea really why this is a problem as both this and @parcel/css have .node-files in them, but this project gives me an error. The biggest difference that I can see is that this project builds the NodeJS package "manually" using node-gyp and such while @parcel/css uses napi to do the heavy lifting.

It would be nice, the whole node ecosystem is ever so slowly moving towards ESM so it'd be nice to have it working without fuzzing about with imports :+1:

The error produced by Vite is a little different. Including below for reference:
10:51:15 AM [vite] Error when evaluating SSR module //home/user/path/to/file:
Error: Failed to parse source for import analysis because the content contains invalid JS syntax. You may need to install appropriate plugins to handle the .node file format.
    at formatError (/home/user/path/to/node_modules/.pnpm/vite@2.7.13/node_modules/vite/dist/node/chunks/dep-f5552faa.js:36769:46)
    at TransformContext.error (/home/user/path/tonode_modules/.pnpm/vite@2.7.13/node_modules/vite/dist/node/chunks/dep-f5552faa.js:36765:19)
    at TransformContext.transform (/home/user/path/to/node_modules/.pnpm/vite@2.7.13/node_modules/vite/dist/node/chunks/dep-f5552faa.js:73659:22)
    at async Object.transform (/home/user/path/to/node_modules/.pnpm/vite@2.7.13/node_modules/vite/dist/node/chunks/dep-f5552faa.js:36985:30)
    at async doTransform (/home/user/path/to/node_modules/.pnpm/vite@2.7.13/node_modules/vite/dist/node/chunks/dep-f5552faa.js:52060:29)

Solution provided by OP works. Modified to add type after first import like:
import type * as minify from "@minify-html/js"

Then including the rest verbatim:
import { createRequire } from "module";
const require = createRequire(import.meta.url);
const mf = require("@minify-html/js") as typeof minify;

SvelteKit 1.0.0-next.247
GNU/Linux 5.15.11-3

Unfortunately the workaround seems to have its own warts. Here's what happens when running it inside a Node-alpine docker container:
Error: Error loading shared library ld-linux-x86-64.so.2: No such file or directory (needed by /app/node_modules/.pnpm/@minify-html+js<EMAIL_ADDRESS>    at Object.Module._extensions..node (node:internal/modules/cjs/loader:1183:18)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at /app/src/hooks.ts:11:19
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async instantiateModule (/app/node_modules/.pnpm/vite@2.7.13/node_modules/vite/dist/node/chunks/dep-f5552faa.js:60105:9)

Just going to leave this here for posterity.

When importing @minify-html/js from a child process (child_process.spawn()), and killing the main process, the child process will not terminate.

The latest version now has an index.js file that should allow for importing from ESM contexts. Let me know if it works on your end.
Note that the latest version is a minor version change, and there are changes to the Node.js library described in the changelog. Noticeably, esbuild has been replaced with minify-js, a JS minifier I wrote from scratch fully in Rust, so @endel there should no longer be any process/signals interference issues caused by the esbuild Go runtime similar to what you've described. Let me know if it works on your end.
Because of the aforementioned change, there are also now WASM versions available, if anyone is interested.

Seems wrong
import { minify } from '@minify-html/wasm';
         ^^^^^^
SyntaxError: Named export 'minify' not found. The requested module '@minify-html/wasm' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from '@minify-html/wasm';
const { minify } = pkg;

(node:3844) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.

E:\javascript\article-extractor\node_modules\@minify-html\wasm\index.js:1
import * as wasm from "./index_bg.wasm";
^^^^^^

SyntaxError: Cannot use import statement outside a module

And No default export found in imported module "@minify-html/wasm"


VSCode brought to halt after installing Forms

Current Behavior
Upon installing the new Forms component, my VSCode started crashing was unable to perform even simple tasks.
I believe this may have something to do with the svelte-language-server and the large type definitions generated by ZOD and Superforms (but this is just a guess).
Is anyone else experiencing similar issues?
Expected Behavior
The ability to use the Forms components without tears of sorrow.
Steps To Reproduce
Install forms component, import and use.
Link to Reproduction / Stackblitz
No response
More Information
No response

I haven't experienced this and (I hope) others haven't as well! Regardless, I'll look into this and see if I'm doing something weird with the types that could cause unexpected behavior.
Would you mind dropping the specs of your device so I can jump on a comparable one to test it out?

Going to transfer this to formsnap since that's where the issue would be if we can detect one!


API access based on public keys

Allowing users to be authenticated on the API using their public keys.
Scenario 1:  User has an Akord wallet + account

user logs in to cognito
downloads/decrypts wallet to access public key
using private key, generate JWT token to access API

Scenario 2: User has a public/private key generated outside of Akord

using private key, generate JWT token to access API

As a remark to this feature request: one thing to remember is not to throw private key over http while negotiating the token. To negotiate JWT using private key challenge-respons should be used.

included in airdrop release: https://github.com/Akord-com/akord-js/releases/tag/v4.2.0


Accuracy check in examples/full_rings/006_timing.py fails

Accuracy check in examples/full_rings/006_timing.py fails for some of the cases. Not clear from which version this has appeared

Resulting thresholds look reasonable, adjusted...


GDScript engine destruction causes deadlock when destroying lambdas

Godot version
v4.2.rc.custom_build [dfd61cdac]
System information
Debian, Vulkan (Forward+)
Issue description
Stack Trace
#0  futex_wait (private=0, expected=2, futex_word=0x55556474d280) at ../sysdeps/nptl/futex-internal.h:146
#1  __GI___lll_lock_wait (futex=futex@entry=0x55556474d280, private=0) at ./nptl/lowlevellock.c:49
#2  0x00007ffff7d61682 in lll_mutex_lock_optimized (mutex=0x55556474d280) at ./nptl/pthread_mutex_lock.c:48
#3  ___pthread_mutex_lock (mutex=0x55556474d280) at ./nptl/pthread_mutex_lock.c:93
#4  0x0000555557f673cd in __gthread_mutex_lock (__mutex=0x55556474d280) at /usr/include/x86_64-linux-gnu/c++/13/bits/gthr-default.h:749
#5  0x0000555557f6741d in __gthread_recursive_mutex_lock (__mutex=0x55556474d280) at /usr/include/x86_64-linux-gnu/c++/13/bits/gthr-default.h:811
#6  0x0000555557f67452 in std::recursive_mutex::lock (this=0x55556474d280) at /usr/include/c++/13/mutex:120
#7  0x0000555557f67d4f in std::unique_lock<std::recursive_mutex>::lock (this=0x7fffffffcc70) at /usr/include/c++/13/bits/unique_lock.h:141
#8  0x0000555557f67c7b in std::unique_lock<std::recursive_mutex>::unique_lock (this=0x7fffffffcc70, __m=...)
    at /usr/include/c++/13/bits/unique_lock.h:71
#9  0x00005555582a1d67 in MutexLock<MutexImpl<std::recursive_mutex> >::MutexLock (p_mutex=..., this=<optimized out>) at ./core/os/mutex.h:122
#10 GDScript::_remove_func_ptr_to_update (p_func_ptr_element=0x555564749d30) at modules/gdscript/gdscript.cpp:1424
#11 0x000055555848002c in GDScriptLambdaCallable::~GDScriptLambdaCallable (this=0x5555646bd690, __in_chrg=<optimized out>)
    at modules/gdscript/gdscript_lambda_callable.cpp:155
#12 0x000055555bf85196 in memdelete<CallableCustom> (p_class=0x5555646bd690) at ./core/os/memory.h:109
#13 0x000055555bf83e7a in Callable::~Callable (this=0x5555647355d8, __in_chrg=<optimized out>) at core/variant/callable.cpp:382
#14 0x000055555bf8b8a7 in Variant::_clear_internal (this=0x5555647355d0) at core/variant/variant.cpp:1373
#15 0x0000555557f58605 in Variant::clear (this=0x5555647355d0) at core/variant/variant.h:302
#16 0x0000555557f58672 in Variant::~Variant (this=0x5555647355d0, __in_chrg=<optimized out>) at core/variant/variant.h:788
#17 0x0000555557f9d0da in CowData<Variant>::_unref (this=0x555564749a18, p_data=0x5555647355a0) at ./core/templates/cowdata.h:216
#18 0x0000555557f985b0 in CowData<Variant>::~CowData (this=0x555564749a18, __in_chrg=<optimized out>) at ./core/templates/cowdata.h:415
#19 0x0000555557f92f28 in Vector<Variant>::~Vector (this=0x555564749a10, __in_chrg=<optimized out>) at ./core/templates/vector.h:290
#20 0x00005555582a6070 in GDScriptInstance::~GDScriptInstance (this=0x5555647499c0, __in_chrg=<optimized out>) at modules/gdscript/gdscript.cpp:2096
#21 0x000055555c2d7d5a in memdelete<ScriptInstance> (p_class=0x5555647499c0) at ./core/os/memory.h:109
#22 0x000055555c2d5493 in Object::~Object (this=0x555564749830, __in_chrg=<optimized out>) at core/object/object.cpp:1979
#23 0x000055555800d898 in RefCounted::~RefCounted (this=0x555564749830, __in_chrg=<optimized out>) at ./core/object/ref_counted.h:53
#24 0x000055555805c0a1 in memdelete<RefCounted> (p_class=0x555564749830) at ./core/os/memory.h:109
#25 0x000055555bf8b85a in Variant::_clear_internal (this=0x555564749530) at core/variant/variant.cpp:1360
#26 0x0000555557f58605 in Variant::clear (this=0x555564749530) at ./core/variant/variant.h:302
#27 0x0000555557f58672 in Variant::~Variant (this=0x555564749530, __in_chrg=<optimized out>) at ./core/variant/variant.h:788
#28 0x0000555557f9d0da in CowData<Variant>::_unref (this=0x55556474edb0, p_data=0x555564749530) at ./core/templates/cowdata.h:216
#29 0x0000555557f985b0 in CowData<Variant>::~CowData (this=0x55556474edb0, __in_chrg=<optimized out>) at ./core/templates/cowdata.h:415
#30 0x0000555557f92f28 in Vector<Variant>::~Vector (this=0x55556474eda8, __in_chrg=<optimized out>) at ./core/templates/vector.h:290
#31 0x000055555bf7f1c6 in ArrayPrivate::~ArrayPrivate (this=0x55556474eda0, __in_chrg=<optimized out>) at core/variant/array.cpp:44
#32 0x000055555bf7f1f5 in memdelete<ArrayPrivate> (p_class=0x55556474eda0) at ./core/os/memory.h:109
#33 0x000055555bf794cf in Array::_unref (this=0x5555645eca48) at core/variant/array.cpp:79
#34 0x000055555bf7d9d0 in Array::~Array (this=0x5555645eca48, __in_chrg=<optimized out>) at core/variant/array.cpp:823
#35 0x000055555bf8b8e6 in Variant::_clear_internal (this=0x5555645eca40) at core/variant/variant.cpp:1382
#36 0x0000555557f58605 in Variant::clear (this=0x5555645eca40) at ./core/variant/variant.h:302
#37 0x0000555557f58672 in Variant::~Variant (this=0x5555645eca40, __in_chrg=<optimized out>) at ./core/variant/variant.h:788
#38 0x0000555557f9d0da in CowData<Variant>::_unref (this=0x5555643a65d8, p_data=0x5555645ec8c0) at ./core/templates/cowdata.h:216
#39 0x0000555557f988a2 in CowData<Variant>::resize<false> (this=0x5555643a65d8, p_size=0) at ./core/templates/cowdata.h:275
#40 0x0000555557f931ea in Vector<Variant>::resize (this=0x5555643a65d0, p_size=0) at ./core/templates/vector.h:94
#41 0x00005555582b475d in Vector<Variant>::clear (this=0x5555643a65d0) at ./core/templates/vector.h:87
#42 0x00005555582a277a in GDScript::clear (this=0x5555643a6330, p_clear_data=0x0) at modules/gdscript/gdscript.cpp:1545
#43 0x00005555582a6c9d in GDScriptLanguage::finish (this=0x5555607fc2e0) at modules/gdscript/gdscript.cpp:2221
#44 0x000055555c2e7f82 in ScriptServer::finish_languages () at core/object/script_language.cpp:259
#45 0x0000555557fef64b in Main::cleanup (p_force=false) at main/main.cpp:3783
#46 0x0000555557f57961 in main (argc=1, argv=0x7fffffffdee8) at platform/linuxbsd/godot_linuxbsd.cpp:76

Not sure what causes this, but it seems having a vector/array of Callables/lambdas or of something that contains lambdas is causing a deadlock.
The deadlock only happens AFTER calling get_tree().quit() (or Alt+F4) and AFTER the tree has been removed. I am using static variables in classes to hold lists of callables, but even manually removing those references before quitting doesn't change the outcome.
It is impossible to pin it from the godot debugger as it blocks the keyboard interrupt signal, thus, a GDB backtrace. This causes the project to hang on the last frame and become unresponsive to anything save a kill.
I don't know how to provide more information on this, sorry.
Steps to reproduce
No clue!
Minimal reproduction project
N/A

I managed to figure out how to stop it from happening. It is definitely one of the many lists (vector/dictionary) of Callables or classes that hold a Callable. I explicitly clear()'d all the lists I had in static variables and now it closes properly. While this may be a circular reference problem, I would have assumed there's a way to run the logic in the right order to deallocate the static variables first? I don't know what that could break, tho.
I hope this helps, even if it's not enough for a MRP.

Judging by the stack trace, this may be fixed by https://github.com/godotengine/godot/pull/85170. Could you give it a test?

I tested with v4.2.rc.custom_build.1ed691914 from that PR and got the same deadlock and a similar stack trace. I think it could be related to lambdas capturing state in closures causing a circular reference, but I still can't get a MRP that does the same.
My project is public https://git.lubar.me/efi/cursed-farm-godot
To remove the workaround just comment out https://git.lubar.me/efi/cursed-farm-godot/src/commit/32441f0beac092bf94684df769f51f338e907978/Game.gd#L18 lines 18 and 19 and it will hang during quit.
Sorry that I can't reproduce it otherwise, it's beyond me how it happens.

I tested with v4.2.rc.custom_build.1ed691914 from that PR and got the same deadlock and a similar stack trace.

I can actually see a difference from that commit. I can reproduce an issue with a mutex lock before applying 1ed691914, and I get a distinct crash after applying it. Interesting enough, the lock issue doesn't seem to be consistent. But the new crash is consistent and happens on every exit. Which may indicate a new regression.
cc @RandomShaper

I can reproduce the deadlock on Linux with GCC compiled binary from 1faf2f5bfe97edcb606d82b81094cb35e2b8e18c, with the steps outlined in https://github.com/godotengine/godot/issues/85151#issuecomment-1821119466.
Almost the same stacktrace as in the OP.

I'm actively investigating this based on the Cursed Farm project. I already know what's going on, quite. I should be able to come up with a fix soon.


feat: add (optional) defaultValue parameters to firstValueFrom and lastValueFrom

Description:
This PR adds an optional defaultValue parameter to firstValueFrom and lastValueFrom. The parameters are added using an overload to avoid this issue (for some usage, at least).
The first and last operators both support defaults and I see no reason why firstValueFrom and lastValueFrom should not do the same. Sure, you could apply defaultIfEmpty to the source, but you say the same for first and last. And, IMO, including defaultValue in the signatures serves as a reminder that the functions expect a value and that not having an available value is an unexpected/error situation.
Also, supporting default values would make implementing a user-land equivalent that resolves with an Option straightforward. (IMO, introducing a Option isn't something we should do while there are bunch of APIs that throw for empty sources.)
Related issue (if exists): Nope

@cartant I think this should probably be in a config/options object. I say this because there's a strong possibility that this will support AbortSignal at some point:
const ac = new AbortController();

firstValueFrom(source$, { defaultValue: 2, signal: ac.signal });

@benlesh

I think this should probably be in a config/options object.

Nice catch. Done.


Enable crops

My use case needs to be able to crop images.
Uncommented the crops and removed what looked to be chaff, hopefully it wasn't supposed to be useful.

@Everlag The middle comment is an error that's being thrown with that function which is why it's commented out. If you can get it working with the test suite and your own images then I'm more than happy to merge!

My images are being processed with no issues.
The test isn't erroring for me, it looks like it produces nothing because the area of the logo being cropped is a solid color. I've modified the test suite to crop the letter in the center, would you prefer that was in another pull request?

Okay that's good stuff :)


Add detection for Server 2019

Now that Windows Server 2019 is released, the detection in https://github.com/puppetlabs/puppetlabs-facts/blob/master/tasks/powershell.ps1#L30-L40 should include this.

Is using something like (gwmi win32_operatingsystem).caption unreliable for generating these facts in the "real world". It seems to work fairly well for our vmpooler instances.
PS C:\cygwin64\home\Administrator> (gwmi win32_operatingsystem).caption
Microsoft Windows Server 2016 Standard

PS C:\cygwin64\home\Administrator> (gwmi win32_operatingsystem).caption
(gwmi win32_operatingsystem).caption
Microsoft Windows Server 2019 Standard

Resolved in 0.5.1 release. Thanks!


Support reading swarm version from plugins.yaml

Support reading swarm version from plugins.yaml
Version 0.1.0

@ablbot autosquash


vscode/vscodium crash on hyprland

Hyprland Version
Hyprland, built from branch  at commit f8008e4b3beb4aae2365973879a46fe0f15c79b4
Bug or Regression?
Bug
Description
vscode/vscodium frequently crash on mouse clicks (not always, but very frequently). The relevant part of the stack is:
                #0  0x000056347dbcd0e4 n/a (codium + 0x6e590e4)
                #1  0x000056347daa7a0e n/a (codium + 0x6d33a0e)
                #2  0x000056347dacdc13 n/a (codium + 0x6d59c13)
                #3  0x000056347dacd897 n/a (codium + 0x6d59897)
                #4  0x000056347dacd740 n/a (codium + 0x6d59740)
                #5  0x000056347dacd8ba n/a (codium + 0x6d598ba)
                #6  0x000056347dacd740 n/a (codium + 0x6d59740)
                #7  0x000056347dacd8ba n/a (codium + 0x6d598ba)
                #8  0x000056347dacd740 n/a (codium + 0x6d59740)
                #9  0x000056347dab2e1f n/a (codium + 0x6d3ee1f)
                #10 0x000056347d3564b6 n/a (codium + 0x65e24b6)
                #11 0x000056347dbbe61b n/a (codium + 0x6e4a61b)
                #12 0x00005634800e5436 n/a (codium + 0x9371436)
                #13 0x000056347d359cc9 n/a (codium + 0x65e5cc9)
                #14 0x0000563479b0f538 n/a (codium + 0x2d9b538)
                #15 0x0000563479b0f305 n/a (codium + 0x2d9b305)
                #16 0x000056347d3422eb n/a (codium + 0x65ce2eb)
                #17 0x0000563479aef4a4 n/a (codium + 0x2d7b4a4)
                #18 0x0000563479aef1d2 n/a (codium + 0x2d7b1d2)
                #19 0x0000563479b0126f n/a (codium + 0x2d8d26f)
                #20 0x0000563478fa8025 n/a (codium + 0x2234025)
                #21 0x0000563478fa7579 n/a (codium + 0x2233579)
                #22 0x000056347d3e4327 n/a (codium + 0x6670327)
                #23 0x000056347d3e281a n/a (codium + 0x666e81a)
                #24 0x000056347d3e228d wl_display_dispatch_queue_pending (codium + 0x666e28d)
                #25 0x0000563479b338e6 n/a (codium + 0x2dbf8e6)
                #26 0x0000563479b3420d n/a (codium + 0x2dc020d)
                #27 0x00007f364daf2ead g_main_context_dispatch (libglib-2.0.so.0 + 0x5dead)
                #28 0x00007f364daf3158 g_main_context_iterate.constprop.0 (libglib-2.0.so.0 + 0x5e158)
                #29 0x00007f364daf31ec g_main_context_iteration (libglib-2.0.so.0 + 0x5e1ec)
                #30 0x000056347c441eec n/a (codium + 0x56cdeec)
                #31 0x000056347c3e1b2c n/a (codium + 0x566db2c)
                #32 0x000056347c3a3b81 n/a (codium + 0x562fb81)
                #33 0x000056347b4f2c7f n/a (codium + 0x477ec7f)
                #34 0x000056347b4f4a62 n/a (codium + 0x4780a62)
                #35 0x000056347b4f02a9 n/a (codium + 0x477c2a9)
                #36 0x00005634792cb374 n/a (codium + 0x2557374)
                #37 0x00005634792ccbfe n/a (codium + 0x2558bfe)
                #38 0x00005634792cca18 n/a (codium + 0x2558a18)
                #39 0x00005634792ca355 n/a (codium + 0x2556355)
                #40 0x00005634792ca445 n/a (codium + 0x2556445)
                #41 0x0000563479007eed n/a (codium + 0x2293eed)
                #42 0x00007f364c59eace __libc_start_call_main (libc.so.6 + 0x23ace)
                #43 0x00007f364c59eb89 __libc_start_main@@GLIBC_2.34 (libc.so.6 + 0x23b89)
                #44 0x0000563478c6602a _start (codium + 0x1ef202a)

How to reproduce
Run vscode or vscodium, click into the window.

Besides, it's more of a codium issue to me.

FYI @pshirshov - this solution fixed it for me.

It doesn't actually fix things, it just makes them better. I have these settings and I'm able to open the first Code window in ~50% cases, but when I open a second one it would crash 100%.

It doesn't actually fix things, it just makes them better. I have these settings and I'm able to open the first Code window in ~50% cases, but when I open a second one it would crash 100%.

seems like setting "window.titleBarStyle": "custom" in the settings.json file to be fixing it for me

this also fixes it for me

It doesn't actually fix things, it just makes them better. I have these settings and I'm able to open the first Code window in ~50% cases, but when I open a second one it would crash 100%.

seems like setting "window.titleBarStyle": "custom" in the .config/VSCodium/User/settings.json file to be fixing it for me

Unfortunately that does not completely fix the problem for me. As I said before, frequently I'm able to open the first VSCode/VSCodium window but it always crashes on the second one. I'm on 6900XT if that matters.

@vaxerski not-hyprland label plz
https://github.com/microsoft/vscode/issues/184124

closing cuz not hl


Difference between documentation and guidance in compliance portal.

Hi
Documentation states : Priority: This value determines the order in which audit log retention policies in your organization are processed. A higher value indicates a higher priority. For example, a policy with a priority value of 5 would take priority over a policy with a priority value of 0. As previously explained, any custom audit log retention policy takes priority over the default policy for your organization.
But in the compliance center the guidance is the opposite :
"Valid priorities are a numerical value between 1-10000. 1 is considered the highest priority, and 10000 is the lowest priority"
Not sure if the doc is correct or if the statement in O365 compliance center is right.

Many thanks for clarifying.

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

ID: e52e1877-191e-5dbe-b805-e755c736b985
Version Independent ID: 69df5ed6-a86f-06d7-1156-e1de211f8e83
Content: Manage audit log retention policies - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/audit-log-retention-policies.md
Service: o365-seccomp
GitHub Login: @markjjo
Microsoft Alias: markjjo

@limp15000 Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/5143
Once the author approves, the document will be updated.
Thanks
Sri

@limp15000 Please contact me via internal MSFT email and I can provide status on this. I am also following up with engineering.

@limp15000 Confirmed with engineering that lower values take priority. Updated documentation will be published. We also updated the corresponding *UnifiedAuditLogRetentionPolicy cmdlet reference topics. Thanks again for heads up.


Revert "fix(deps): bump node-fetch from 2.6.1 to 2.6.2"

Reverts cujarrett/destiny-insights-frontend#17

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


Hover effects on Review Section

Your whole website have these hover effects on cards but this section lacks it.
I will add the best effects on this section
Kindly assign this issue to me under hactoberfest, gssoc-ext and relevant tags.

No needed @Bhum-ika


[DOC-3115] Apply updates to Harness Delegate FAQs topic

Apply updates to Harness Delegate FAQs topic
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Unit test for snack F0 function...that fails.

I've learned a bunch getting this far, especially I'm more comfortable with the
VoiceSauce source now.  I believe I have translated the way VoiceSauce uses
snack to compute the sF0 column in the output data correctly, but the data
values do no match up.  As you will see if you run the unit test, the values
are at least in the same ballpark (within 0.6, usually closer) at first, but
then things go off the rails.  I would expect the numbers to match with greater
than 0.6 precision, given we have the same input file and the same program
executing the calculation, but perhaps I am naive :).
Can you see anything I've done wrong, or incorrect assumptions I'm not aware
of?  Can you run the test and see what results you get?  The first few from my
run look like this:
(0, 247.31, (766.0, 247.262))
(1, 247.152, (767.0, 247.148))
(2, 246.636, (768.0, 246.649))
(3, 246.561, (769.0, 246.566))
(4, 246.277, (770.0, 246.299))
(5, 246.059, (771.0, 246.042))
(6, 246.422, (772.0, 246.432))
(7, 246.165, (773.0, 246.307))
(8, 245.713, (774.0, 245.741))
(9, 245.891, (775.0, 245.792))
(10, 246.717, (776.0, 246.595))
(11, 246.596, (777.0, 246.66))
(12, 246.321, (778.0, 246.312))
(13, 246.392, (779.0, 246.307))
(14, 246.57, (780.0, 246.387))
and it fails for me at sample 134:
AssertionError: 103.148 not less than 0.6 : row 134

OK, so I'll set this issue aside for the moment and move on.  I've marked the test as a failure and added a test of what we do get to make sure I don't break anything going forward.
I'll be building newer changes off this changeset...I'll merge this if there is no objection.

I tried to run the unit test on snack f0, but I don't think I'm doing it correctly. Based on the instructions in HACKING.md, I ran
python -m unittest test

but then I got the following error message (see end of message).  I'm probably missing something really obvious---can you let me know what might be going wrong? One line that looks suspicious to me in the error message is an error looking for a very strange file path:
'/var/folders/3h/qkl7zrt96bb6mznj2g0k8c2w0000gp/T/tmp86ZSKl/defaults/sounds/cant_c5_19a.f0'

can't find package snack
    while executing
"package require snack"
    (file "tclforsnackpitch.tcl" line 5)
Traceback (most recent call last):
  File "opensauce/process.py", line 59, in <module>
    process(indir, outdir, settings, params)
  File "opensauce/process.py", line 34, in process
    soundfile.measurements[param] = measurements[param](soundfile) # it is what it is...
  File "/Users/amoebe/Documents/research/voice-sauce/open-sauce/opensauce-python/opensauce/measure.py", line 16, in f0_snack
    f0 = snack_ks.get_snack_f0(soundfile)
  File "/Users/amoebe/Documents/research/voice-sauce/open-sauce/opensauce-python/opensauce/snack_ks.py", line 49, in get_snack_f0
    with open(f0file, "r") as f:
IOError: [Errno 2] No such file or directory: '/var/folders/3h/qkl7zrt96bb6mznj2g0k8c2w0000gp/T/tmp86ZSKl/defaults/sounds/cant_c5_19a.f0'
F.No handlers could be found for logger "opensauce.snack"
xE
======================================================================
ERROR: test_raw (test_snack.TestSnack)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/amoebe/Documents/research/voice-sauce/open-sauce/opensauce-python/test/test_snack.py", line 69, in test_raw
    F0, V = snack_pitch(fn, frame_length=f_len, window_length=w_len)
TypeError: 'NoneType' object is not iterable

======================================================================
FAIL: test_default_setup (test_cli.TestCLI)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/amoebe/Documents/research/voice-sauce/open-sauce/opensauce-python/test/test_cli.py", line 27, in test_default_setup
    self.assertEqual(rc, 0)
AssertionError: 1 != 0

----------------------------------------------------------------------
Ran 4 tests in 0.475s

FAILED (failures=1, errors=1, expected failures=1)


Added support of depthwise separable convolutions for Theano backend

Currently SeparableConv1D / SeparableConv2D layers are only supported by TensorFlow backend, this Pull Request adds support for Theano backend too. This may be useful for training models with SeparableConv2D layers and FP16 weights (TensorFlow can't train such kind of models).

Please add unit tests (by no longer the corresponding unit tests to TF).

SeparableConv2D layers and FP16 weights (TensorFlow can't train such kind of models)

You can submit a feature request for this to the TF repo if that's you're use case.

It seems that Keras doesn't have any unit tests for separable convolutions (only for regular convolutions). I'll try to add them in the nearest future.

Just, please add backend tests if there aren't any. There are existing tests for the corresponding layers though.

Ok, I'll add backend tests and refactor layers test (if it is necessary).

This PR is cool! As described in the official docs, the theano.tensor.nnet.separable_conv2d accepts depth_multiplier > 1. While TensorFlow assumes depthwise_kernel_shape = (row, col, input_dim, depth_multiplier), Theano does depthwise_kernel_shape = (input_dim * self.depth_multiplier, 1, row, col).

I hope the following codes may help you to check functionality when depth_multiplier > 1.
import numpy as np
from numpy.testing import assert_allclose
from theano.tensor import _shared
from theano.tensor.nnet import separable_conv2d

c1, c3 = 64, 64
b = 10
i1, i2 = 224, 224
k1, k2 = 7, 7
p1, p2 = 1, 1
d1, d2 = 1, 1
s1, s2 = 1, 1
inputs = _shared(np.random.random((b, c1, i1, i2)))
depthwise_val = np.random.random((c1, 1, k1, k2))

outs = []
for depth_multiplier in [1, 2, 3]:
    c2 = c1 * depth_multiplier
    depthwise_filters = _shared(depthwise_val.repeat(depth_multiplier, axis=0) / depth_multiplier)
    pointwise_filters = _shared(np.ones((c3, c2, 1, 1)))
    out = separable_conv2d(
        inputs, depthwise_filters, pointwise_filters, num_channels = c1,
        input_shape=(b, c1, i1, i2), depthwise_filter_shape=(c2, 1, k1, k2),
        pointwise_filter_shape=(c3, c2, 1, 1), border_mode=(p1, p2),
        subsample=(s1, s2), filter_dilation=(d1, d2))
    outs.append(out.eval())

assert_allclose(outs[0], outs[1], 1e-8)
assert_allclose(outs[1], outs[2], 1e-8)

I still believe this would be valuable to have. Are you still working on it?

Closing PR since @taehoonlee has added separable_conv2d for Theano. Thanks for the PR anyway!
We still need separable_conv1d though.


About 1.20.4

Er... is a 1.20.4 update still on the plans?

I forgor, but here ya go.
https://modrinth.com/mod/omnihopper/version/2.2.4+1.20.4


Please add getLatLng method like google_maps_flutter

Hello :)
First off thanks for using your time to build this package!! I really appreciate it.
Is your feature request related to a problem? Please describe.
I'm working on a project that has a requirement to allow users to draw an area on a map.
By following this example I was able to get it to work with the google maps package. What makes this possible is the getLatLng function of the controller. It takes screen coordinate and turns them into lat/long, but the apple_maps_flutter package does not.
Describe the solution you'd like
It'd be great if the apple_maps_flutter package had the same getLatLng function so I could allow users to draw on the map.

Agreed it will be nice if the package can provide access to getLatLng just like google maps package, I'm actually working on similar feature for ios device as explained by @sboyd  but i have no choice than to switch to google maps for now, hopefully this feature can be provided by apple_maps_flutter soon. Thanks

Hello :) First off thanks for using your time to build this package!! I really appreciate it.
Is your feature request related to a problem? Please describe. I'm working on a project that has a requirement to allow users to draw an area on a map. By following this example I was able to get it to work with the google maps package. What makes this possible is the getLatLng function of the controller. It takes screen coordinate and turns them into lat/long, but the apple_maps_flutter package does not.
Describe the solution you'd like It'd be great if the apple_maps_flutter package had the same getLatLng function so I could allow users to draw on the map.

Agreed it will be nice if the package can provide access to getLatLng just like google maps package, I'm actually working on similar feature for ios device as explained by @sboyd but i have no choice than to switch to google maps for now, hopefully this feature can be provided by apple_maps_flutter soon. Thanks


Running instructions

Updated the readme.md with running instructions. This will prevent downloading the dataset multiple times.

Good idea, that is much clearer.


pip install -e?

@yangkcatiu you mentioned in the user-lookup branch README.md ...

Local development
Clone this project, go to the source directory, type the following command to install the package locally:
pip install -e ./

Can you help me understand what this does exactly? The pip documentation says...

... which isn't very helpful.
From other Googling, it seems like maybe this allows me to import the version of the package that is present in the source folder on my local machine. Whereas, if I don't do this, running import osometweet would give me whatever the latest version of the package was when pushed to PyPi? I just tried this out and it seems like that is at least one difference, is there anything else that this does?

Oh, my bad, the / at the end is not supposed to be there.

Please close the issue if it's working now


Implement add_dur for Sequencer

When using the sequencer I find it more intuitive to use start time and duration instead of end time that requires an extra mental calculation. I had this implemented as an external trait which is easy enough so no worries if you prefer not to add this functionality.

I may change the name later but it's fine to add this for now, thanks.

Yes, I don't like the name either but didn't manage to think of something better 😄
Thanks 👍


Appendix A, Installing Ansible inside WSL

You recommend to install ansible using "pip3 install ansible" command.
This cause the package and its dependencies not yet installed, especially ansible-base package, to be installed under
/home//.local/lib/python3.8/site-packages/, and ansible executable modules, which are part of ansible-base package, to be installed under ./home//.local/bin/.
So if the above path not defined under user's PATH variable, the ansible command wouldn't be found.
On the other hand, if you install ansible using "sudo pip3 install ansible", the packages will be installed under
/usr/local/lib/python3.8/dist-packages and ansible executable files will be installed under /usr/local/bin/ which is always defined in the PATH variable.
So you can recommend either to add ./home//.local/bin/ to PATH or just to install ansible using sudo.
if you want to reproduce the case, please don't forget to remove ./home//.local/bin/ from PATH and uninstall ansible and ansible-base before reinstalling it.

Note: I solved the issue by running the command with sudo
sudo pip3 install ansible
based on #219  this is the wrong course of action
When following along in the book, I get the following:
Command 'ansible' not found, but can be installed with:
I'm on Win10, WSL2, Ubuntu 20.04
Below are the results when I re-ran the
python install followed by
the pip3 install followed by
checking the Ansible version
21:02:47 mba@Ubuntu:/mnt/c/Users/malco$ sudo apt-get install -y python3-pip python3-dev
Reading package lists... Done
Building dependency tree
Reading state information... Done
python3-dev is already the newest version (3.8.2-0ubuntu2).
python3-pip is already the newest version (20.0.2-5ubuntu1.6).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
21:03:46 mba@Ubuntu:/mnt/c/Users/malco$ pip3 install ansible
Requirement already satisfied: ansible in /home/mba/.local/lib/python3.8/site-packages (4.6.0)
Requirement already satisfied: ansible-core<2.12,>=2.11.5 in /home/mba/.local/lib/python3.8/site-packages (from ansible) (2.11.5)
Requirement already satisfied: PyYAML in /usr/lib/python3/dist-packages (from ansible-core<2.12,>=2.11.5->ansible) (5.3.1)
Requirement already satisfied: cryptography in /usr/lib/python3/dist-packages (from ansible-core<2.12,>=2.11.5->ansible) (2.8)
Requirement already satisfied: jinja2 in /usr/lib/python3/dist-packages (from ansible-core<2.12,>=2.11.5->ansible) (2.10.1)
Requirement already satisfied: resolvelib<0.6.0,>=0.5.3 in /home/mba/.local/lib/python3.8/site-packages (from ansible-core<2.12,>=2.11.5->ansible) (0.5.4)
Requirement already satisfied: packaging in /home/mba/.local/lib/python3.8/site-packages (from ansible-core<2.12,>=2.11.5->ansible) (21.0)
Requirement already satisfied: pyparsing>=2.0.2 in /home/mba/.local/lib/python3.8/site-packages (from packaging->ansible-core<2.12,>=2.11.5->ansible) (2.4.7)
21:03:55 mba@Ubuntu:/mnt/c/Users/malco$ ansible --version
Command 'ansible' not found, but can be installed with:
sudo apt install ansible


Leaderboards

In first place, when I scroll through the leaderboard, it feels weird, because it hides behind the upper wood instead of the first one.
FROM

TO

Also, in the selection screen, the "Time" word could be separated from the word "Today".

The first part is a bug; there should be less margin or padding on that child view, and it should be positioned lower, so that it would get cut off right about where the paper ends.
Styles for that are here and here.
Could you explain more about what you think would look better for the timespan / score category subtabs? I know the current design isn't very good, so some better design ideas would definitely help, but I'm not sure what you mean.

@nwinter simply "Today" instead of "Time" and "Today" in the same tab could look better like in the other sub tabs. Please let me know what do you think. I'll take up this bug.

Ah, but that wouldn't work well when there are two categories of scores, like when it can be "Time" and "Damage Taken".

I mean, the "Time" string could be separated from the "Today", but I'm am the not the best person to talk about design.

Hi. This is my first bug here. Can i work on this?

Ah, looks like @divyanshu013 beat you to it. There are plenty of older good-for-newbies bugs that need help, though!

Ok thanks anyways :)


Galaxy-stable chart doesn't start up with default settings.

How to recreate:
I checked out the develop branch and tested against minikube. Did not make any changes - simply ran a helm install .
The galaxy-stable chart installs without a hitch. However, when attempting to access the galaxy service via the NodePort, it simply returned a 404. The galaxy container's logs revealed the error:
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) unable to open database file
It appears that the GALAXY_CONFIG_DATABASE_CONNECTION is not being overridden as expected.
I then exec'd into the container and manually ran the following:
export GALAXY_CONFIG_DATABASE_CONNECTION=postgresql://galaxy:change-me@postgresql-for-galaxy/galaxydb?client_encoding=utf8
/usr/bin/startup

Still the same error.
Finally, I edited galaxy-central/config/galaxy.ini, and set the database connection manually there. It then proceeded to startup and connected to the postgres database, instead of sqlite. However, it ran all the migrations from scratch, indicating that it's not a database with migrations included.
Finally, it failed with the following:
Traceback (most recent call last):
  File "/export/galaxy-central/lib/galaxy/webapps/galaxy/buildapp.py", line 58, in paste_app_factory
    app = galaxy.app.UniverseApplication( global_conf=global_conf, **kwargs )
  File "/export/galaxy-central/lib/galaxy/app.py", line 185, in __init__
    self.job_manager = manager.JobManager( self )
  File "/export/galaxy-central/lib/galaxy/jobs/manager.py", line 23, in __init__
    self.job_handler = handler.JobHandler( app )
  File "/export/galaxy-central/lib/galaxy/jobs/handler.py", line 33, in __init__
    self.dispatcher = DefaultJobDispatcher( app )
  File "/export/galaxy-central/lib/galaxy/jobs/handler.py", line 764, in __init__
    self.job_runners = self.app.job_config.get_job_runner_plugins( self.app.config.server_name )
  File "/export/galaxy-central/lib/galaxy/jobs/__init__.py", line 632, in get_job_runner_plugins
    rval[id] = runner_class( self.app, runner[ 'workers' ], **runner.get( 'kwds', {} ) )
  File "/export/galaxy-central/lib/galaxy/jobs/runners/kubernetes.py", line 44, in __init__
    assert KubeConfig is not None, K8S_IMPORT_MESSAGE
AssertionError: The Python pykube package is required to use this feature, please install it or correct the following error:
ImportError No module named pykube.config
root@torpid-kudu-galaxy-stable-bbcf5b445-4

Is this a problem with an out-of-date container (I have not tested this chart on? @pcm32 Where can I view the DockerFile that was used to build pcm32/galaxy-stable-k8s? I checked here: https://hub.docker.com/r/pcm32/galaxy-stable-k8s/ but it does not display the Dockerfile.
The docker images used by minikube were listed as:
pcm32/galaxy-stable-k8s                                          latest              a4e5ec729667        6 months ago        898MB
pcm32/galaxy-stable-k8s-init                                     latest              01867b81731e        7 months ago        1.41GB

I forgot to add. The repo I used was:
helm repo add https://pcm32.github.io/galaxy-helm-charts

Thanks for pointing this out! Because of the minikube / actual cluster duality, I guess there is no clear default, but we could try to make it as smooth as possible. I don't think that in the documentation I mentioned any default case though, but using some example configs (which I will look into making more clear and standard). Currently blocked becuase of some issues with 18.01 images, but I will work on that this week.

This addition to docs should help as a default minikube, although the vanilla instance lifted is more or less useless as it doesn't include much tools and it won't submit jobs to k8s because stock images are not configured to do so.

Given #11 I will close this, but please reopen if needed.


Extending SMGL Serializer to support more banks

Bumping up version from 102 to 103
Toleration \r\n in received documents

Everything looks great! Thanks for contribution.
Please fix the test and I am not clear on \r\n thing. The tests may fail under windows but I am working on providing development in container feature.

Please fix the test and I am not clear on \r\n thing.

 The regex would not handle it. Perhaps I could amend the regular expression, but I did not want to make it unnecessary complex.

The tests may fail under windows but I am working on providing development in container feature.

I fixed the version and I assume it's now broken for the reason you mentioned.
Also I should mention why I made the new public method: the accounting software I use (proprietary) submits "102" for the version, and banks reject it. So I'm using your OFX library in my proxy that takes requests from the software towards my proxy, then replaces the Auth part of the payload with a working one, send it with the rest of the original payload to the bank and returns results back to the software. Works like a charm.

Sounds very good! I assume this will also allow users to send custom requests (like pay credit cards, query for due dates etc). OFX protocol has much more capabilities...
'\r\n' use case is well understood. Thanks again for contribution.

Just reviewed the pipeline - it says "The total line coverage is below the specified 80". I would appreciate if you add a test for PrepareOfxRequest


This doesn't support linux ?

Is your feature request related to a problem? Please describe.
I tested  this tool on windows 10, looks pretty, and I try to build on a  linux laptop, But it failed.
The tool doesn't support linux right ?

@iysheng 🤔 don't you watch the news buddy?

check this out https://abdenasser.github.io/neohtop/

@iysheng 🤔 don't you watch the news buddy?

aha, thks, I just read the readme before. and when build failed as

@iysheng no worries, let me know if it doesn't work properly for you.

aha, I couldn't use this tool, for the linux latop installed server edition without GUI support.
(NeoHtop:2688032): Gtk-WARNING **: 09:07:57.263: cannot open display:

what architecture ?

what architecture ?

X86-64, I connect the laptop use ssh

I have just downloaded only the .AppImage, I have given it permissions to run and it does not work, the neohtop window opens in white and in the console I get the message:
Could not create default EGL display: EGL_BAD_PARAMETER. Aborting...

** (NeoHtop:10945): WARNING **: 09:34:17.298: atk-bridge: get_device_events_reply: unknown signature

I'm on Fedora 41 (KDE plasma 6.2.3) am I missing something? any dependencies?
pd: i didn't build anything, i just downloaded the .AppImage

how about the .deb @rcv11x did you try that?

how about the .deb @rcv11x did you try that?

.deb? how do i install it in fedora if .deb is for debian/ubuntu?

how about the .deb @rcv11x did you try that?

.deb? how do i install it in fedora if .deb is for debian/ubuntu?

those are built on ubuntu, but you can try using a tool like Alien, which converts .deb files to .rpm format

how about the .deb @rcv11x did you try that?

.deb? how do i install it in fedora if .deb is for debian/ubuntu?

you can use dpkg command crack the deb file


[ci/cd] release multiple charts

We found a small “bug” into the chart release.
Since we release library and then charts if you merge a PR with both -> only libraries are released since the first step creates a tag.
Therefore now it is not possible to release at the same time library chart and "normal" ones

There is no way to fix the CI to fix it without making a contribution to the action itself.
The action can only get one folder: https://github.com/helm/chart-releaser-action/blob/a3454e4/cr.sh#L250
    local commit="$1"

    local changed_files
    changed_files=$(git diff --find-renames --name-only "$commit" -- "$charts_dir")

    local depth=$(( $(tr "/" "\n" <<< "$charts_dir" | sed '/^\(\.\)*$/d' | wc -l) + 1 ))
    local fields="1-${depth}"

    cut -d '/' -f "$fields" <<< "$changed_files" | uniq | filter_charts

As $charts_dir is quoted we cannot set more than one folder there. If we try to workaround it with something like putting there a . the cut command above will leave the charts out of the scope.
Looking up latest tag...
Discovering changed charts since 'nri-bundle-3.6.5'...
WARNING: .github/Chart.yaml is missing, assuming that '.github' is not a Helm chart. Skipping.
WARNING: charts/Chart.yaml is missing, assuming that 'charts' is not a Helm chart. Skipping.
WARNING: library/Chart.yaml is missing, assuming that 'library' is not a Helm chart. Skipping.
Nothing to do. No chart changes detected.

I have the time to improve the charts and the release process timeboxed so I have to low the priority of this issue.
I leave this comment to give context for the next who can take it.


Fix AnsiOutput write methods claiming zero bytes were written

Currently, the implementation of IStdout for AnsiOutputalways returns Ok(0) when a write completes successfully.
This behaviour is incorrect, as these methods are used elsewhere to implement std::io::Write, which expects  the writer to return the number of bytes written. This means that the implementation of Write for crossterm::Terminal doesn't play nice with the write! macro (and other consumers of the Write trait), causing uses of write! to panic with the message "failed to write whole buffer". (see mgattozzi/ysh#12).
This branch changes these methods to correctly return the number of bytes written, fixing the panics.

Thanks, my fault :( ! I'll merge it an do a release.
I have on question could the release wait for a couple of days? This because there maybe come another PR so I could have two patches in one release. If that doesn't work out let me know!
You could then use the GitHub repo with master branch in your cargo.toml

I published the new version with the changes.


Black doesn't split the line with string and format()

Operating system: Ubuntu 16.04
Python version: 3.6.6
Black version: 18.9b0
Does also happen on master: yes
Length limit: 80
Before Black:
logging.info("Evaluating the model with {} of the supplied data held out for tests ({}).".format(sup_opts["test_split"], len(test)))

After Black:
logging.info(
    "Evaluating the model with {} of the supplied data held out for tests ({}).".format(
        sup_opts["test_split"], len(test)
    )
)

The line with string should be splitted, because it exceeds the line length limit, but it's not. The same effect is for different settings, e.g. 60 chars, 50 chars.

This seems the same as #182.

this should be closed as a dup of #182  indeed


DOC: Description of kwargs in langchain_core.prompts.chat.HumanMessagePromptTemplate.from_template_file() misleading

URL
https://api.python.langchain.com/en/latest/prompts/langchain_core.prompts.chat.HumanMessagePromptTemplate.html
Checklist

[X] I added a very descriptive title to this issue.
[X] I included a link to the documentation page I am referring to (if applicable).

Issue with current documentation:
The documentation for the class method from_template_file() for the class langchain_core.prompts.chat.HumanMessagePromptTemplate is misleading, since it states that kwargs are "keyword arguments to pass to the constructor". However this is not true. kwargs will be passed to from_template(), which means that partial_variables will not be passed to the constructor, but will be handled by from_template.
Idea or request for content:
The description "keyword arguments to pass to the constructor." should probably be changed to "keyword arguments to pass to from_template.".
Note that there might be more alternative constructors that suffer from this problem.

I don't understand it. The arguments for from_template are as follows:
def from_template(
    cls: Type[_StringImageMessagePromptTemplateT],
    template: Union[str, List[Union[str, _TextTemplateParam, _ImageTemplateParam]]],
    template_format: str = "f-string",
    *,
    partial_variables: Optional[Dict[str, Any]] = None,
    **kwargs: Any,
) -> _StringImageMessagePromptTemplateT:

In this definition, kwargs are additional keyword arguments that are only passed to the constructor, as seen on line 506. Therefore, kwargs are never passed to from_template of PromptTemplate when instantiating HumanMessagePromptTemplate using from_template.

partial_variables is passed to PromptTemplate.from_template() in from_template(). But I'm not talking about from_template(), I'm talking about from_template_file() that looks like this:
    @classmethod
    def from_template_file(
        cls: Type[_StringImageMessagePromptTemplateT],
        template_file: Union[str, Path],
        input_variables: List[str],
        **kwargs: Any,
    ) -> _StringImageMessagePromptTemplateT:
        with open(str(template_file)) as f:
            template = f.read()
        return cls.from_template(template, input_variables=input_variables, **kwargs)

So when I pass a keyword parameter named partial_variables to from_template_file() this lands in kwargs under the key partial_variables. The line
        return cls.from_template(template, input_variables=input_variables, **kwargs)

passes those along to from_template. But since from_template does have an argument partial_variables the kwargs["partial_variables"] will be passed as that parameter and will not land as kwargs["partial_variables"] in from_template. To prove that run the following little test script:
class Test:
    @classmethod
    def from_template(
        cls,
        template,
        template_format,
        *,
        partial_variables,
        **kwargs,
    ):
        print("from_template - partial_variables", partial_variables)
        print("from_template - kwargs", kwargs)

    @classmethod
    def from_template_file(
        cls,
        template_file,
        input_variables,
        **kwargs,
    ):
        template = "file_content"
        print("from_template_file - kwargs", kwargs)
        return cls.from_template(template, input_variables=input_variables, **kwargs)

Test.from_template_file("template_file", "input_variables", template_format="template_format", partial_variables="foo")

This will output:
from_template_file - kwargs {'template_format': 'template_format', 'partial_variables': 'foo'}
from_template - partial_variables foo
from_template - kwargs {'input_variables': 'input_variables'}

which proves my point: Not all additional keyword arguments that get passed to from_template_file will be passed to the constructor: partial_variables will not. So the documentation is misleading.


Update Error Classification mechanism

Mercado Libre doesn't have a uniform error structure. For example the
invalid_token error has in the error key "not_found"
{ "message": "invalid_token", "error": "not_found", "status": 401, "cause":[] }

Or malformed access_token comes as a bad_request error
{ "message": "Malformed access_token: test","error": "bad_request", "status": 400, "cause":[] }

This updates the error classification mechanism to use the description
as the first alternative to classify the error.

@enortham friendly reminder

@enortham friendly reminder


Incorrect wall render order

Currently all walls render over all the entities.
Walking entities like the cook should render above walls to preserve the top-down perspective.

This was fixed in #15


Ability to choose a shell for scripts

What would you like to be added:
The ability to choose a shell for commands in TestAssert. For example:
---
apiVersion: kuttl.dev/v1beta1
kind: TestAssert
timeout: 600
commands:
  - shell: bash # runs bash -c instead of sh -c
    script: |
      set -euo pipefail
      ...
  - shell: bash -euo pipefail # runs bash -euo pipefail -c instead of sh -c
    script: |
      ...

Why is this needed:

Currently sh -c is hard coded for shell invocations, and without options like -e to ensure failures before the last command get picked up.
sh doesn't support setting -o pipefail to fail on errors in a pipeline.

What's worse, the behaviour of /bin/sh depends the environment. In some cases it is simply a symlink to bash, and then the bash features work. This reduces portability of tests. 😞
A couple of thoughts:

it might be nice to not have to specify shell: bash -whatever in every single place, but instead provide the default in the config and have it trickle down automatically to all commands 🤔
if we're going as far as allowing any shell, perhaps we should get rid of the assumption that -c is the way to pass the script to the shell, and instead rely on temporary files? 🤔

Temporary files would be fine too, as you can then write them like a script with the shebang.

The only potential issue would be that some folks might have /tmp mounted -o noexec 🤔
But I guess we could keep the current sh -c ... behaviour unless the text of script starts with #!?


Elasticsearch init scripts set max open files to 65535, but expects 65536

Same as: https://github.com/elastic/elasticsearch/issues/17430
Causes a "scary" false positive to be logged

This is fixed for 2.4.0 with https://github.com/elastic/elasticsearch/pull/18041


fix: show user logout reason message

I noticed that there's a message attached to the app URI when a session expires:
[main] Open Deep Link URL insomnia://app/auth/login?message=Your+session+has+expired.+Please+reauthenticate+to+continue+using+Insomnia+Sync.

I also included the ability to specify a custom timeout for fetch calls to the API; it's not in use right now but was helpful when debugging some connectivity issues

I see no reason not to add the timeout option but we should take caution before modifying it in production as it can contribute to outage conditions.


consider refining default blacklist?

in this case the redirect is not bypassed - i realize it's ultimately up to the user to configure the blacklist, but i'm wondering if the strings in the blacklist should be a bit more specific by default?
example URL: https://outgoing.prod.mozaws.net/v1/14b631a713d4684de03418f5a14d81de35dbe188d42c5085568a0f37ad0b9940/https%3A//github.com/mrdokenny/Cookie-AutoDelete
the redirection is allowed because of /cookie in the blacklist
i don't know how the strings in the blacklist typically appear in URLs, but perhaps this could all be a bit more specific to where, for example, /cookie becomes /cookie/ or /cookie[?&/] or something?

I am aware it's not perfect, it's more of a best guess from my side and what I saw in other extensions. My thinking is that a URL that is not skipped is better than a broken webpage. Also the only real way to fix this would be data analysis on URLs collected from the users. I think we can agree that this is not desirable at all.

I am not sure yet, but after "same domain detection" has been implemented, I have removed all preconfigured "parameters" in blacklist with exception to my custom rover.ebay.co.
Up to now I have 0 breakages and the upper sample does skip and a workable way.
I will wait for a week or two of browsing, then will report back the results with any possible exceptions that comes along.


setting default node version per folder

each time i go into a project folder and do $ nvm use ... it doesnt remember for the next time. is it possible have a default version using nvm per folder?

Yes - make a .nvmrc file in each directory.

And then add a snippet to your shell to switch automatically based on folder: https://github.com/nvm-sh/nvm#deeper-shell-integration
Could we make this simply happen by default @ljharb? It works like that for rbenv.

No, it is very intentionally not done by default, as that's exceedingly intrusive on a system.
Closing, since the question is answered.


NAT and Nested Virt in migration use-case

[Enter feedback here]
So Nested Virtualization only works with NAT networking. No possibility for Virtual Networking (https://docs.microsoft.com/en-ca/virtualization/hyper-v-on-windows/quick-start/connect-to-network)
If true ,this blocks migration scenarios where a on-prem cluster of Hyper-V servers need to be migrated to the cloud (for many reasons, like hosting old VMs with unsupported OSs). Those VMs are on-prem using "virtual networks" for east-west communication.
If we recommend any overlay workaround (such as LISP) it would be great to add a pointer to that documentation in the section where we simply say "enable NAT" on Azure.
This would also be useful to create greenfield Hyper-V clusters (with a virtual networking overlay) on Azure using nested virtualization.
Thanks
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7ffa9c12-4fdc-118a-4e84-624cbcc3f4c8
Version Independent ID: 6c5b1d1f-78a1-2c79-b37d-0367d76269da
Content: How to enable nested virtualization in Azure Virtual Machines
Content Source: articles/virtual-machines/windows/nested-virtualization.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the question. We are currently investigating and will update you shortly.

@marcosgm I am assigning this issue to the content author to investigate further and update the document as appropriate.

#please-close Doc now points to: https://docs.microsoft.com/en-us/virtualization/hyper-v-on-windows/user-guide/nested-virtualization-azure-virtual-network


Update links in YouTube video descriptions

How can community members best help update YouTube video descriptions for PyMC related videos on the Data Umbrella channel?
For example, how the link (https://github.com/pymc-devs/pymc-data-umbrella/blob/main/webinars/probabilistic_programming_with_pymc/notebook.ipynb) to the Jupyter Notebook for Austin’s Intro to Probabilistic Programming with PyMC
video (https://www.youtube.com/watch?v=Qu6-_AnRCs8&t=1044s) is inactive.
@reshamas, is there a structure for these types of pull requests, like the one you’ve outlined here (https://github.com/pymc-devs/video-timestamps/issues/11)?

Hi @zbraiterman, thanks for bringing this to our attention.
Link not working:
https://github.com/pymc-devs/pymc-data-umbrella/blob/main/webinars/probabilistic_programming_with_pymc/notebook.ipynb
Maybe it's this:
(a)
https://github.com/pymc-devs/pymc-data-umbrella/blob/main/about/probabilistic_programming_with_pymc/notebook.ipynb
Maybe it's this:
(b)
https://pymc-data-umbrella.xyz/en/latest/about/probabilistic_programming_with_pymc/notebook.html#probprog-pymc-nb
@OriolAbril  What's the better link to refer to?

@zbraiterman If you see more broken links like this, it's fine to open up an issue here, as you just did.
We restructured the whole sprint website, so that's why the links are broken, unfortunately.

@OriolAbril @symeneses
The "Run Code" button this page doesn't work.  I am not sure how to fix that?
https://pymc-data-umbrella.xyz/en/latest/about/probabilistic_programming_with_pymc/notebook.html#probprog-pymc-nb

Maybe it's this:
(b)
https://pymc-data-umbrella.xyz/en/latest/about/probabilistic_programming_with_pymc/notebook.html#probprog-pymc-nb

This one, always share links to the website.

We restructured the whole sprint website, so that's why the links are broken, unfortunately.

The links on the other webinars are not broken and never were. When restructuring I set up redirects so that the old links continue to work. However, as this link points to GitHub directly so setting up redirect was not possible.
I also remember the link being fixed already after flagging this on slack (link will only work if you have access to slack), could the change have been undone when adding the timestamps?

The "Run Code" button this page doesn't work. I am not sure how to fix that?

It is working for me, can you share a screenshot of what is happening in your case to try and see what might be going on?

from slack:

Link to webinar page in website:
https://pymc-data-umbrella.xyz/en/latest/about/probabilistic_programming_with_pymc/index.html
(this I can and will make sure it never breaks even if we move the notebook)

It looks like there were two separate links in the video description, one directly to the repo and one to the html file.
I do recall fixing one of them.
Confirming that I checked all the links in Austin's video description and they all work:

Key Links

Transcript: https://github.com/data-umbrella/event-transcripts/blob/main/2022/41-austin-pymc.md
Meetup Event: https://www.meetup.com/data-umbrella/events/282790294/
Video:  https://youtu.be/Qu6-_AnRCs8
GitHub repo: https://pymc-data-umbrella.xyz/en/latest/about/probabilistic_programming_with_pymc/notebook.html#probprog-pymc-nb

Resources

Jupyter Notebook: https://github.com/pymc-devs/pymc-data-umbrella/blob/main/about/probabilistic_programming_with_pymc/notebook.ipynb
Austin's website: https://austinrochford.com/talks.html
ArviZ: https://github.com/arviz-devs/arviz
PyMC Series of events: https://pymc-data-umbrella.xyz

@OriolAbril
Running notebook in the browser:  to discuss later, next time we meet.

Thank you, @reshamas and @OriolAbril, for following up on this.
Is the Slack open to / useful for folks like myself?

Also, is the Slack open to / useful for users like myself?

@zbraiterman Here is more information on the PyMC slack, including the "recurring contributor" role:
https://github.com/pymc-devs/pymc/blob/main/GOVERNANCE.md#recurring-contributors

Thank you, @reshamas, for sharing this information.

@zbraiterman Thanks for opening this issue. I am closing this.
If you see any more broken links, please open up a new issue here and I'll update the links.


Quickstart experience. User-friendly error handling

Here are some thoughts from newbie:

I expected that dpp init would guide me through pipeline creation
dpp --help didn't show how do I specify pipeline
Running dpp serve and going to http://localhost:5000/ resulted in builtins.KeyError KeyError: b'all-pipelines' which is not a user-friendly way to show that I don't have any pipelines.

Suggestions:

step-by-step pipeline creation (dpp init) wizard would be awesome
Elaborate help by showing some examples e.g. dpp run ./pipeline.yaml, dpp serve ./pipeline.yaml
dpp serve should validate whether there are any pipelines available and show nice user-friendly error with suggestions to fix

[x] python 3.5
[x] alpine 3.4

Well, dpp serve ./pipeline-spec.yaml didn't work either
Usage: dpp serve [OPTIONS]
Error: Got unexpected extra argument (./pipeline-spec.yaml)

I expected that dpp run would require yaml file, but it looks like yaml filename is hardcoded? Or maybe it just scans all the files? This is confusing.
If the default filename was dpp.yaml, it would make more sense. Additionally, I expected an option to override file name, e.g.:
dpp -f pipeline.yaml run albanian-treasury

I have an issue where errors in the YAML file cause dpp to silently fail (that is, show no avail. pipelines)

Could you please provide an example of such a flawed YAML file (so I can
add error messages in such cases)?
On Mon, 5 Jun 2017 at 12:24 Daniel Fowler<EMAIL_ADDRESS>wrote:

I have an issue where errors in the YAML file cause dpp to silently fail
(that is, show no avail. pipelines)
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/frictionlessdata/datapackage-pipelines/issues/17#issuecomment-306143441,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQMdfcShVs_pp1h3DefZ_QEFKUD48puks5sA8k0gaJpZM4LPGWf
.

This is an example of a broken yaml and the error it throws, it's not friendly at all in the sense that I cannot figure out what's wrong with it.
https://gist.github.com/martinszy/6f4a0b2021687fa729ccf6a2044a9fc1

I figured out that tabs don't work, and that spaces are very strinctly two


React Group Project

https://github.com/mahabubx7/space-cars/

[x] We are only two members @mahabubx7 @touredri

Hi @touredri @mahabubx7 ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
STATUS; APPROVED :tada:
Highlights

[ ] All tasks have been distributed equally :+1:
[ ] the tasks have been assigned correctly :+1:

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Performance: Dynamic import for draft-js

Feature Description
This is a follow-up issue for #8251 for draft-js specifically that requires a bit more work since it's used in multiple files in the code.

Alternatives Considered

Additional Context

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

cc @barklund For priority / points.

Perhaps this is relevant here:
I've experimented with moving draft-js related components to a dedicated @web-stories-wp/rich-text package in the past:


python3Packages.setuptools-rust.tests: fix eval

Description of changes
@trofi pointed out in this comment that #319173 failed to update the path.
This PR:

[x] Removes generic.nix, inlining its contents into default.nix
[x] Removes load-bearing defaults, requiring its callers to provide them.
[x] In python3Packages.setuptools-rust, use an override rather than assuming knowledge of where in the tree maturin is located.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Fixes eval for me. Thank you!


two blocks cannot be translated into Chinese

Describe the bug
These two blocks cannot be translated into Chinese.I can't find their information in GAME-STRINGS.JSON

They are in strings.json because they are used for all makecode editors. Or simply use in context translation.

I found it, thank you very much.

Great, so the issue can be closed now :-)


Internal Intrinsics

Issue description
When "intrinsics" (what Elide calls built-in VM symbols, often provided by the host VM) are provided to guest scripts, they are always included, even when exposed through internal facade modules.
This isn't super great for a few reasons: (1) they always exist on the global stack, presumably consuming memory etc; (2) user code can see these symbols. Even though they are ugly and unlikely to collide with user symbols, we shouldn't expose these to guest code at all.
Instead, we should devise a way to determine what is "runtime-internal" code that happens to execute as guest code, vs. actual guest code, and make such intrinsics available only where needed and safe.
Node has a similar feature to this in their codebase, referred to as primordials. This object is made available as a VM global only to internal sources. Injection of builtins is simplified by mounting each symbol on a single object.

As part of the experiments accompanying #954, we already have applicable results in this front. Bindings that are only required by guest initialization scripts (e.g. the console object in JavaScript) can be safely removed after the context is initialized. An API for this has already been designed.
However, the described behavior only fits initialization scripts, and is not applicable to builtin modules such as those providing the Node.js API. There are two difficult points here:

Determining whether a GraalVM Source is marked as internal is not as straightforward as it may seem, and in many cases I've been unable to do it entirely, even when explicitly creating a Source object for this intent.
The guest code that provides the builtin modules runs lazily, the first time it is imported by end-user code. This means that it will not have access to init-time bindings as described above. Additionally, we cannot control how the Source objects corresponding to those modules are created, since the GraalJS module loader API is completely internal (though they have considered opening it in the future). In practice, our own built-in modules are indistinguishable from end-user code when an intrinsic is accessed, which prevents us from implementing internal-only bindings.


JS Logic and PriceSearch JS files will need to be tweaked to match HTML/bootstrap layout

@19Lola89 @Lucasiti @W-JK  - JS files and ID's will need to be amended to match HTML file structure, ID attributes and targets.


Updates to fix grpc and docker CVEs

See commit messages for details.

There still an issue with<EMAIL_ADDRESS>that I'm not sure how to resolve.
[~/go/src/submariner-io/submariner-operator]$ go get -u go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp                                                                                [release-0.15]
go: downloading go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.45.0
go: downloading go.opentelemetry.io/contrib/instrumentation/net/http v0.11.0
go: downloading go.opentelemetry.io/contrib v1.20.0
go: downloading go.opentelemetry.io v0.1.0
go: downloading go.opentelemetry.io/otel v1.19.0
go: downloading go.opentelemetry.io/otel/metric v1.19.0
go: downloading go.opentelemetry.io/otel/trace v1.19.0
go: downloading github.com/go-logr/logr v1.3.0
go: upgraded github.com/go-logr/logr v1.2.3 => v1.3.0
go: upgraded github.com/stretchr/testify v1.8.1 => v1.8.4
go: upgraded go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.35.1 => v0.45.0
go: upgraded go.opentelemetry.io/otel v1.11.0 => v1.19.0
go: upgraded go.opentelemetry.io/otel/metric v0.31.0 => v1.19.0
go: upgraded go.opentelemetry.io/otel/trace v1.11.0 => v1.19.0
[~/go/src/submariner-io/submariner-operator]$ go mod tidy                                                                                *[release-0.15]
go: finding module for package go.opentelemetry.io/otel/exporters/otlp/internal/envconfig
go: finding module for package go.opentelemetry.io/otel/exporters/otlp/internal
go: downloading go.opentelemetry.io/otel/exporters/otlp v0.20.1
github.com/submariner-io/submariner-operator imports
	github.com/operator-framework/operator-sdk/cmd/operator-sdk imports
	github.com/operator-framework/operator-sdk/internal/cmd/operator-sdk/cli imports
	github.com/operator-framework/operator-sdk/internal/cmd/operator-sdk/bundle imports
	github.com/operator-framework/operator-sdk/internal/cmd/operator-sdk/bundle/validate imports
	github.com/operator-framework/api/pkg/validation imports
	github.com/operator-framework/api/pkg/validation/internal imports
	k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation imports
	k8s.io/apiserver/pkg/util/webhook imports
	k8s.io/component-base/tracing imports
	go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc imports
	go.opentelemetry.io/otel/exporters/otlp/internal: module go.opentelemetry.io/otel/exporters/otlp@latest found (v0.20.1), but does not contain package go.opentelemetry.io/otel/exporters/otlp/internal
github.com/submariner-io/submariner-operator imports
	github.com/operator-framework/operator-sdk/cmd/operator-sdk imports
	github.com/operator-framework/operator-sdk/internal/cmd/operator-sdk/cli imports
	github.com/operator-framework/operator-sdk/internal/cmd/operator-sdk/bundle imports
	github.com/operator-framework/operator-sdk/internal/cmd/operator-sdk/bundle/validate imports
	github.com/operator-framework/api/pkg/validation imports
	github.com/operator-framework/api/pkg/validation/internal imports
	k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation imports
	k8s.io/apiserver/pkg/util/webhook imports
	k8s.io/component-base/tracing imports
	go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc imports
	go.opentelemetry.io/otel/exporters/otlp/otlptrace/internal/otlpconfig imports
	go.opentelemetry.io/otel/exporters/otlp/internal/envconfig: module go.opentelemetry.io/otel/exporters/otlp@latest found (v0.20.1), but does not contain package go.opentelemetry.io/otel/exporters/otlp/internal/envconfig

Same thing if I do go get -u<EMAIL_ADDRESS>

The otel dependencies are a pain to deal with (not through otel’s fault, and there’s been recent work in k/k to improve things, but we’re still left to deal with the fallout — for CVEs which mostly don’t apply to us).

The way to fix this is to look for the first “integration point” in the dependency tree for a given family of dependencies. In
github.com/submariner-io/submariner-operator imports
	github.com/operator-framework/operator-sdk/cmd/operator-sdk imports
	github.com/operator-framework/operator-sdk/internal/cmd/operator-sdk/cli imports
	github.com/operator-framework/operator-sdk/internal/cmd/operator-sdk/bundle imports
	github.com/operator-framework/operator-sdk/internal/cmd/operator-sdk/bundle/validate imports
	github.com/operator-framework/api/pkg/validation imports
	github.com/operator-framework/api/pkg/validation/internal imports
	k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation imports
	k8s.io/apiserver/pkg/util/webhook imports
	k8s.io/component-base/tracing imports
	go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc imports
	go.opentelemetry.io/otel/exporters/otlp/internal: module go.opentelemetry.io/otel/exporters/otlp@latest found (v0.20.1), but does not contain package go.opentelemetry.io/otel/exporters/otlp/internal

that’s the transition from k8s.io dependencies to go.opentelemetry.io dependencies, which happens with go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc. That is also the integration point for the second error.
go get go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc fixes the dependency tree.

Oh I see you already added the final commit, thanks!