Add docs about additional config settings for disagg arch

Enhancement
profiles.default.dt_read_thread_count_scale
2.0, "Number of read thread = number of logical cpu cores * dt_read_thread_count_scale.  Only has meaning at server startup."

profiles.default.io_thread_count_scale
5.0, "Number of thread of IOThreadPool = number of logical cpu cores * io_thread_count_scale.  Only has meaning at server startup."

profiles.default.init_thread_count_scale
100, "Number of thread = number of logical cpu cores * init_thread_count_scale. It just works for thread pool for initStores and loadMetadata. Only has meaning at server startup."

profiles.default.remote_gc_method
1, "The method of running GC task on the remote store. 1 - lifecycle, 2 - scan."

profiles.default.remote_gc_interval_seconds
3600, "The interval of running GC task on the remote store. Unit is second."

profiles.default.remote_gc_min_age_seconds
3600, "The file will NOT be compacted when the time difference between the last modification is less than this threshold"

profiles.default.remote_gc_ratio
0.5, "The files with valid rate less than this threshold will be compacted"

profiles.default.dt_filecache_max_downloading_count_scale
1.0, "Max downloading task count of FileCache = io thread count * dt_filecache_max_downloading_count_scale."

profiles.default.dt_filecache_min_age_seconds
1800, "Files of the same priority can only be evicted from files that were not accessed within `dt_filecache_min_age_seconds` seconds."

profiles.default.disagg_build_task_timeout
DEFAULT_DISAGG_TASK_BUILD_TIMEOUT_SEC, "disagg task establish timeout, unit is second."

profiles.default.disagg_task_snapshot_timeout
DEFAULT_DISAGG_TASK_TIMEOUT_SEC, "disagg task snapshot max endurable time, unit is second."

profiles.default.disagg_fetch_pages_timeout
DEFAULT_DISAGG_FETCH_PAGES_TIMEOUT_SEC, "fetch disagg pages timeout for one segment, unit is second."

Related work:
The doc team is working on reformatting the tiflash configures item
https://github.com/pingcap/docs-cn/pull/19168/files


Provision of attributes to optimizers

i had a quick question on how to provide arguments to optimzers.
Does code like per below work?
stytole_model = mx.model.FeedForward(ctx=devs,
        symbol             = network,
        num_epoch          = 40,
        optimizer          = mx.optimizer.Adam(),
        learning_rate      = 0.0001)

or will the learning rate argument be ignored, and one either has to enter like this:

stytole_model = mx.model.FeedForward(ctx=devs,
        symbol             = network,
        num_epoch          = 40,
        optimizer          = mx.optimizer.Adam(learning_rate      = 0.0001)

or like this?

stytole_model = mx.model.FeedForward(ctx=devs,
        symbol             = network,
        num_epoch          = 40,
        optimizer          =  "adam",
        learning_rate      = 0.0001)

Many thanks in advance,

W

2&3 will work. 1 doesn't


Fix dependency cycle injection

Relevant #134
Fixes #158
Hi @yoichitgy
Please take a look at a new approach to resolve dependency cycles without initiating objects twice.
The concept is only partially based on what I wrote in the bug thread. I could not think of a way to replace the Swift Optional with a reference type without breaking the API. So I took a different approach.
The PR is based on the premise that initComplete closures are not allowed to unwrap the Optionals resolved by the Resolver.  This should be in line with the documentation but it will break code that tries to unwrap them (I had to remove force unwrapping from some Tests for this).
This PR enables Swinject to do two things:

To investigate the dependencies needed by an initComplete closure using a DependencyAnalyzer as the Resolver. It works by returning nil and saving the requested types.
To call initComplete closures that only create new objects which are not in the process of being created.
Then after each Objects initialization, pending initCompletes are checked and executed once all dependencies are created (that is: not currently being created).

The code might be rough by I though to pitch this idea to you before investing more time in this concept.
Tests:

I added the case @petalvlad was describing.
A case testing if storyboards are injected only once now fails because of the additional analysis run – I could not yet look into that issue.
I had to remove force unwrapping from initComplete closures in tests.

Looking forward to your feedback!
Cheers,
Markus

@marcorei, thank you for your investigation and pull request to fix the issue👍👍👍 Adding the test case for #158 is also great!
Please give me time to understand the problem and resolution correctly. Your detailed comment is very helpful! I'll also try adding some test cases.
I'll try taking time by the weekend.

@marcorei I've read your code. It's a nice solution for the problem👍
One concern is side effect caused by the call of initComplete by DependencyAnalyzer. A unit test in SwinjectStoryboardSpec is failing because of the side effect. I think the side effect is related to the assumption that initComplete must not force-unwrap the resolved instance.
So, I think it's good to run the analyzer and pending completions code only if an option for it is specified. For example, .Lazy object scope or circularDependencyAnalized as below. (My preference is the latter one though I still didn't investigate much.) What do you think about the concern?
container.register(AType.self) { _ in AImpl() }
    .inObjectScope(.Lazy)
    .initCompleted { (r, a) in
        a.b = r.resolve(BType.self)
    }

or
container.register(AType.self) { _ in AImpl() }
    .initCompleted { (r, a) in
        a.b = r.resolve(BType.self)
    }.circularDependencyAnalized()

Here are the other tiny nitpicks:

With one of the options above, it's nice to put the analyzer and pending completion code to a function to simplify resolveEntry.
DependencyAnalyzer should be internal to hide our implementation.

@jakubvano, what do you think about the problem and resolution?
Though we have the concern, we are getting closer to a good resolution. Thank you @marcorei😃

Running the analyzer optionally is a great idea in my opinion. I much prefer circularDependencyAnalyzed() because it can be used together with .Container or .Hierarchy.

@yoichitgy My understanding of the problem is, that there are two premisies we might want to guarantee:

No throw-away objects will be created during graph resolution (i.e. temporary objects which are destroyed after resolution)

this was problem described in #134
this is relevant when there are side-effects of creating given object (i.e time consuming operation, or some interaction with passed dependencies)

Injected dependencies will be fully initialised, except ones which are part of the same dep. cycle as given object

AFAIK this is currently not possible to achieve when object is part of a dependency cycle
this covers #158

Initial solution (with immediate initComplete calls) fulfilled 2 but not 1, solution in #134 fulfilled 1 but not 2.
Proposed solution fulfills 2 as well, but with couple of new issues:

due to multiple initComplete invocations 1 does not hold for object scope .Nonedependencies. This is IMO not relevant, but should be mentioned in docs if we choose to go with this
due to multiple initComplete invocations, property dependencies will be assigned multiple times.
forced unrwapping is not allowed in initCompleted closures

As you mentioned, if we do not want to break current API we should make this an optional feature - I would suggest having #134 as default implementation and adding something like .attemptCompleteInitialisation() for opting to use DependencyAnalyser feature.
However, I do think that proposed solution should be a default behaviour, as both 1 and 2, seem like natural expectations for DI framework. Maybe add opt-in feature in 1.0 version and make it default in 2.0?

Thinking about this some more, maybe initial solution with immediate initCompleted calls was fine after all: it is much simpler, and in most cases (at least in my experience) "throw-away" instances do not cause problems in properly designed architecture.
And if they do, you can work around them by injecting both parts of dependency cycle in initCompleted closures, i.e:
container.register(A.self) { _ in A() }
    .initCompleted { r, a in a.b = r.resolve(B.self)! }

container.register(B.self) { _ in B() }
    .initCompleted { r, b in b.a = r.resolve(A.self)! }

instead of
container.register(A.self) { r in A(b: r.resolve(B.self)!) }

container.register(B.self) { _ in B() }
    .initCompleted { r, b in b.a = r.resolve(A.self)! }

Thanks @jakubvano for your deep investigation and understanding of the problem🙇 You structured my ambiguous concern. Invoking initCompleted is important for both of Swinject code and user's code. Swinject should be simple, stable, robust and consistent because DI framework is a fundamental framework for an app.
I would like to conclude we revert #134, and add documentation about the workaround @jakubvano  wrote. @marcorei, would you agree?
The idea using DependencyAnalyzer is technically very interesting, but a little bit experimental to add to Swinject Core. It may be good to add that kind of idea to a Swinject extension hosted in another repository. @marcorei, I really appreciate your pull request, idea and reference to Dagger😃 If you are interested, I appreciate another idea or pull request to a different problem in Swinject Core or a new extension!

This is off topic, but I want to simplify my code implementing the feature of parent container and hierarchy scope. It is the most complicated part in Swinject...

@yoichitgy I agree, the solution is rather complex for a small benefit. Working on it was fun nevertheless!

Thanks @marcorei! Ok, we will revert #134 and following some adjustment commits I did, and document the workaround.

#134 has been reverted by #164.


Exposing helpers

As discussed offline, a lot of potentially useful gltf-pipeline helpers aren't being exposed right now. We can keep a running list here. Some ideas for exposing them:

export a utils.js file to gltf-pipeline that exposes useful utils. this to avoid clutter in index.js
expose requested utilities as "_private"
just expose everything?

Wishlist:

[ ] readAccessor
[ ] getUniqueId
[ ] packArray

It is perfectly fine to treat the gltf-pipeline public API the same way we treat the Cesium API meaning that these functions could be exposed as individual functions, e.g., like Cesium.clone; there's no need to group them into one util class.

I will try to get to this today; it's been on my to-do list for a while.

Fixed in #157, closing this issue.


Notebook moves focus in and out of output

https://monacotools.visualstudio.com/DefaultCollection/Monaco/_build/results?buildId=99302&view=logs&j=3792f238-f35e-5f82-0dbc-272432d9a0fb&t=2912d057-d0f3-5270-1d09-777eeaba2eea&l=104
 1) VSCode Smoke Tests (Web)
       VSCode Smoke Tests (Web)
         Notebooks
           moves focus in and out of output:
     Error: Timeout: is active element 'webview, .webview' after 20 seconds.
      at /Users/runner/work/1/s/test/automation/out/code.js:181:23
      at Generator.next (<anonymous>)
      at fulfilled (/Users/runner/work/1/s/test/automation/out/code.js:9:58)

It seems it's still flaky https://dev.azure.com/monacotools/Monaco/_build/results?buildId=110155&view=logs&jobId=3792f238-f35e-5f82-0dbc-272432d9a0fb&j=3792f238-f35e-5f82-0dbc-272432d9a0fb&t=bd1c96ef-8f6c-518c-1bd9-e87b3a72835d

Consolidate notebook smoke-test failures? https://github.com/microsoft/vscode/issues/119164


chore(main): release uniform 0.14.1

:robot: I have created a release beep boop
0.14.1 (2024-12-09)
Bug Fixes

uniform: minor issues with FieldArray and Input (e30746c)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

uniform-v0.14.1
:sunflower:


Move django-libsass dependency from the wagtail package to the project template

I think this should be moved into the templates requirements.txt. AFAIK, it isn't called directly within Wagtail (it is pulled in through the COMPRESS_PRECOMPILERS setting in the project itself).
Removing it will also greatly improve speed of installing wagtail to the host machine for running the wagtail start command.

I don't undertstando you very good, but, i have a big problem because i couldn't install libsass in my web server, i have ask help to web servers supporters, and then of a large and hardly process, i could install libsass, then i could install django-libsass and finally, i can see all styles of wagtail admin.

@gasman and @kaedroho can we discuss? @kaedroho, did you mean "it is not called directly within Wagtail"?

I already have a fork for which I removed the libsass dependency. https://github.com/dbsima/wagtail/tree/stable/0.8.x

@dbsima Just to clarify, we're not considering dropping libsass completely, as developing wagtailadmin's styles in SCSS is still useful to us. This is just about moving the dependency from the wagtail package to the project template, so that people using virtual machines (e.g. Vagrant) don't have to install it on the host machine.

Done in #985/#1019


Ordered search mode with List[Union[]]

I have the model, which has  field with type:
children: list[
    Union[
        FieldBool,
        FieldInt8,
        FieldInt16,
        FieldInt32,
        FieldInt64,
        FieldUint8,
        FieldUint16,
        FieldUint32,
        FieldUint64,
        FieldEnum,
        FieldBitfield,
        FieldCompound,
        FieldStaticArray,
        FieldVector,
        FieldDeque,
        FieldSet,
        FieldBitVector,
        FieldPointer,
        FieldDfFlagArray,
        FieldDfStaticFlagArray,
        FieldDfArray,
        FieldDfLinkedList,
        FieldDfLinkedListType,
        FieldDfOtherVectorsType,
        FieldString,
        FieldPtrString,
        FieldStaticString,
    ]
] = element(default=None)

I need to parse tag and save the order of children in list. unordered search mode found 100 items, but ordered found only 1.
I mean, i don't know exact order of children, but i want to keep it. How can i do it?

@shevernitskiy hi,
Could you provide the document you are trying to parse?

It is just a chunk of big document.
Erase .txt.
doc.xml.txt

The problem is that in ordered search mode Union typed field is bound to the first matched element skipping all the elements before.
In your example FieldBool is not found, then FieldInt8 is bound to the <int8_t name='cached_glowtile_type'/> skipping all the elements before and search continues after it. Therefore only 1 element is found in ordered search mode.
In unordered search mode the elements order is not kept and right now there is not way to keep it.

The problem is that in ordered search mode Union typed field is bound to the first matched element skipping all the elements before.
In your example FieldBool is not found, then FieldInt8 is bound to the <int8_t name='cached_glowtile_type'/> skipping all the elements before and the search continues after it. Therefore only 1 element is found in ordered search mode. In unordered search mode the elements order is not kept and right now there is not way to keep it.

I see. That sad. As you can see there is cpp struct in xml, so the order matters.
Could the unordered_sequenced mode be the possible future improvement?

Yes, I will think about it. It is not easy to implement the search keeping elements order due to internal implementation details, but I will try to find a workaround.

Thanks!


DOC-3776 Patch release: log4j v2.17.1 docs update Part I

DOC-3791 Added legal info for CE 4.4.11

DOC-3788 Added legal info for CE 4.5.9

DOC-3785 Added legal info for CE 4.6.8

DOC-3782 Added legal info for CE 4.7.6

Sachin reviewed and approved preview site.


feature/99 - update to checkPopulate util

Fixes #99
Discussion

as discussed on Slack, we want to weaken this util to not check anything if the node has no includes, as that's including everything and that's the most one can include

Fails

:no_entry_sign:
The version was not updated. Please update the version.

:no_entry_sign:
The changelog has not been updated. Please update the changelog.

Warnings

:warning:
The README has not been updated. Please update the README.

Messages

:book:
We were able to automatically fix some formatting issues in this PR for you!

:book:
Could not find any browser results.

:book:
You reduced the total lines of code! Awesome! :+1:

Some things that were possibly fixed:

Code that could be fixed via the --fix flag
Formatting that could be fixed by prettier

Take a look at this commit to see what happened in detail: 5e35eaed9d1389c1888df32963893cb7b5765727
And look at this wiki page to see the reasoning behind the ESLint rules: https://github.com/smartprocure/eslint-config-smartprocure/wiki/Rules-and-Why-We-Chose-Them

  Generated by :no_entry_sign: dangerJS against 03faa3f80dc9e31e067ef386fbd6463ae7fe683c

Fails

:no_entry_sign:
The version was not updated. Please update the version.

:no_entry_sign:
The changelog has not been updated. Please update the changelog.

Warnings

:warning:
The README has not been updated. Please update the README.

Messages

:book:
We were able to automatically fix some formatting issues in this PR for you!

:book:
Could not find any browser results.

:book:
You reduced the total lines of code! Awesome! :+1:

Some things that were possibly fixed:

Code that could be fixed via the --fix flag
Formatting that could be fixed by prettier

Take a look at this commit to see what happened in detail: 5e35eaed9d1389c1888df32963893cb7b5765727
And look at this wiki page to see the reasoning behind the ESLint rules: https://github.com/smartprocure/eslint-config-smartprocure/wiki/Rules-and-Why-We-Chose-Them

  Generated by :no_entry_sign: dangerJS against 03faa3f80dc9e31e067ef386fbd6463ae7fe683c

updated, ready for review

updated, ready for review


Add datapackage.json support to Table View with a dropdown of listed data sources to load and view

This is the last remaining feature we planned for adopting frictionless data standards in #15
See datapackage.json specs: https://specs.frictionlessdata.io/data-package/
And https://specs.frictionlessdata.io/data-resource/#introduction

Cool! Being able to explore a dataset from a datapackage.json file (only) in Visual Studio Code would be a feature I'd likely use. If you need examples of large data packages, let me know.

Great! I think your camtrap example dataset could be a good test bed for it: https://github.com/tdwg/camtrap-dp/tree/main/example
Remoted data loading will be added in #53 & I jotted some notes from our table schema naming discussion in #84.

Created simple Observable JS notebook to explore some of the public datapackage.json sources published on DataHub:
https://observablehq.com/@randomfractals/data-package

Closing this since new public Data Packages picker added in #102 provides over 750 data packages to explore from github, and there is also new List Data Resources context menu in editor titlebar to view tabular data sources and table views from open datapackage.json files (see #104).


Get rid of global npm/yarn dependencies

Global npm or yarn dependencies can be problematic because their versions aren't tied to the project, you use a single version for your entire system.
The reason we added them, is to be able to use certain tools from the command line. (e.g. webpack, rimraf, karma, ...)
But when a script defined in package.json uses these tools, it's not necessary to globally install them, because yarn and npm will automatically add the folder ./node_modules/bin to the PATH when running scripts.
Since all our build scripts are in package.json, we no longer have any need  of global dependencies. We should therefore ensure that all global dependencies are also devDependencies, and remove the global script from package.json, and the docs

This ticket is still applicable even though it's quite old. These global dependencies still exist in our package.json here: https://github.com/DSpace/dspace-angular/blob/master/package.json#L18
One note here is that included in those global dependencies is pm2, which is NOT being used for development/builds, but rather for production deployment (specifically, it's used by the deploy and undeploy commands in our package.json).  We may want to consider whether pm2 should instead be a documented prerequisite (alongside node & yarn, but only required for production setups), instead of treating it like a dev/build dependency. More info about PM2 is at https://pm2.keymetrics.io/

Regarding PM2, I will just mention that I had been planning to use something else, probably Passenger.  More generally, it seems to me that this is something that should be left up to the site, with a well-documented suggestion for sites that are not so opinionated. :-)

This issue seems to have been partially addressed, but there is no linked PR.  The global script was removed in commit 8bf241c182479f9fb35ca3028397b7c01e980316 at 2020-03-18 11:03:40.  However, README.md still instructs one to yarn run global.

@mwoodiupui : Good point. Yes, it seems like we fixed this as a "side effect" of the move to Angular CLI in #625
So, I think this ticket can be closed as soon as we remove any remaining references to global...likely just that one in the README.

There are no longer any references to installing global packages in package.json or README.md so I think this is resolved.
The DSpace 7/8 Install docs mention installing yarn and pm2, but I think that's the right place for detailing global dependencies (not in our project's package.json).


windowrulev2 = immediate,fullscreen:1 , kitty freeze

Regression?
No
System Info and Version
 System/Version info
Hyprland, built from branch  at commit 9a09eac79b85c846e3a865a9078a3f8ff65a9259  (props: bump version to 0.42.0).
Date: Wed Aug 7 19:17:10 2024
Tag: v0.42.0, commits: 5069

Description
kitty freeze in full screen when i enable tearing
How to reproduce
enable tearing and fullscreen kitty
Crash reports, logs, images, videos
No response

Only kitty or any program ?

Why would you want to enable tearing for Kitty?

Why would you want to enable tearing for Kitty?

Probably to reduce input latency

Actually I can reproduce this. I cannot use immediate mode roughly since version 0.43 on any app as it will freeze as soon as the window gains solitarity. Doesn't matter if it's a wayland native or xwayland window.

Only kitty or any program ?

any program

Why would you want to enable tearing for Kitty?

i want to set tearing for any fullscreen application so my games have it , and i dont wan't other applications to be effcted or freezed , i dont need tearing in my applications but i just want everything to work and not adding endless window roles for every game i run

You could do something like windowrulev2 = immediate, class:^(steam_app_.*)$.
This would enable tearing for all games running with proton.

@firashacker Which GPU do you run? Any chance that it is an NVIDIA card? Me and a frined cannot enable immediate mode anymore without getting freezes and we both run NVIDIA, but it works on an AMD system.

@firashacker Which GPU do you run? Any chance that it is an NVIDIA card? Me and a frined cannot enable immediate mode anymore without getting freezes and we both run NVIDIA, but it works on an AMD system.

i have radeon rx6600

You could do something like windowrulev2 = immediate, class:^(steam_app_.*)$.
This would enable tearing for all games running with proton.

this might be helpful but it doesn't apply for all the titles i play


Overhaul @astrojs/sitemap docs

This PR updates the docs for the @astrojs/sitemap integration.
My main aim was to be able to link directly to the information about linking to the sitemap in robots.txt for https://github.com/withastro/starlight/pull/2083 (to avoid duplicating these docs in the Starlight docs).
But while there I realised this page is fairly dusty and could use a small refresh.
Reorganised, reworded, and re-prioritised things.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/integrations-guide/sitemap.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


[release/5.0-preview2] Update hosted and BYOC agents used in our builds

dotnet/aspnetcore-internal#3540

NO-MERGE because I'm waiting for @Pilchie regardless of other reviewers. But @dotnet/aspnet-build please do review!


Confirm pages timeliness

Short description

Users can now confirm pages timeliness and refresh the date/timestamp of the latest version of all translations of a page.
Proposed changes

Enhances page form actions with a 'refresh date' button
~Adds a 'refresh dates' bulk action to the page tree view~

Resolved issues

Fixes: #1258

@membralala Ok, so the conclusion was to remove the bulk action and only have the button in the form to create the incentive to look at the page before confirming the actuality.

@membralala As the result of the discussion it seems like we don't really need any functionality in the tree view. Sou you can remove it from there and only keep the function on the page form.

@ulliholtgrave , @timoludwig  Thanks for your detailed feedback, I removed the bulk action and enhanced my first solution.


WIP: derive schema for oranda.json

Had to replace LinkedHashMap with IndexMap to get derive support (fine)
Derived schemars::JsonSchema for everything
TEMP HACK TO DEMO: emit the schema at the end of oranda build

main outstanding issue is when we produce the schema and how we surface/archive/publish it


SPH package type is conflicted

From the python code:
    packet['SPH_type'] = int(A_bin[0][3],2)
    if packet['SPH_type'] == 0:
        packet['SPH_type_mnemonic'] = 'TM'
    elif packet['SPH_type'] == 1:
        packet['SPH_type_mnemonic'] = 'TC'

Our rac files seem to indicate wrong package type sometimes.
Ask @OleMartinChristensen  to produce proper files or for the spec to be updated.

Do you get another number that 0 or 1? Or is it that you sometimes get TC packets?

In the rac files we got, I think they only have TC packages, or so it would seem, if we disregard the type and parse it as TM packages we can read the content fine, at least that was as far as I understood @joakimmoller

ok, lets to a telecon on it when you guys want, show me some examples.

in "source package header" the "Type" is always 0 which means it should be TC-packages in the payload. But there is mainly TM-packages in the payload (type should be 1), but some TC-packages also exists but it is hard to distinguish between them (without workarounds).

I'll prepare some stats from the files we have - then we talk on Monday.

After receiving the new update of aez icd (issue h) this is not an issue. One of the differences from previous versions was the removal of some element in the HTR structure. Now we can process the housekeeping data from the instrument.


Missing tag for Kedro 0.18.2

Missing tag for Kedro 0.18.2

You found the release quick! We haven't officially announced the release yet, this would be done on Monday probably, stay tuned.
For the time being adding  - - checkout 0.18.1 should work

The release is out and you should be able to remove any ad-hoc fix now. :)
https://github.com/kedro-org/kedro/tree/0.18.2


non-falsy-string breaks Attribute testing

Larastan Version: 2.2.0 (on phpstan v1.8.4)
--level used: 5

Description
Since phpstan v1.8.3, the following error occurs:
Parameter $get of static method Illuminate\Database\Eloquent\Casts\Attribute<mixed,mixed>::make() expects
  (callable(mixed, mixed=): string)|null,
  Closure(mixed, mixed): non-falsy-string given.

The non-falsy-string is introduced in phpstan v1.8.3. I think this test should pass, but it doesn't.
Laravel code where the issue was found

    /**
     * Returns the full name of a person.
     *
     * @return \Illuminate\Database\Eloquent\Casts\Attribute<string, never>
     */
    protected function fullname(): Attribute
    {
        return Attribute::make(
            get: function ($value, $attributes): string {
                if (! is_null($attributes['prefix'])) {
                    return $attributes['firstname'].' '.$attributes['prefix'].' '.$attributes['lastname'];
                }

                return $attributes['firstname'].' '.$attributes['lastname'];
            },
        );
    }

Hello Joost! 👋🏻
Thank you for your report.

@szepeviktor Thanks for you blazing fast reply! I reported over there. Would you prefer to close this one for now or just leave it open untill they've responded upstream?

The problem isn't the non-falsy-string part, the problem is that the code expects callable(mixed, mixed=) (one required parameter, one optional parameter), but your Closure requires two parameters.
This is a problem because the code that accepts the callable might call it only with one argument but when you give it a callable with two required parameters, PHP will crash.

So this is a phpstan problem? I am experiencing exactly this problem as well.
protected function fullImageUrl(): Attribute
{
    return Attribute::make(
        get: fn ($value, $attributes) => global_asset($attributes['image_url']),
    );
}

AFAIK the given callback may not get a second argument $attributes.
https://github.com/nunomaduro/larastan/blob/8fbe7423782dac778f2383f8cc895b0da0faf733/stubs/Attribute.stub#L30

Theory says it should have a default value fn ($value, $attributes = [])
But do not rely on me! I'm just a janitor here.

Theory says it should have a default value fn ($value, $attributes = []) But do not rely on me! I'm just a janitor here.

According to the Laravel documentation the second parameter is the attributes.

How was this ever solved? I'm struggling with this too and I can't for the life of me figure out what the correct syntax is to specify here:
    /**
     * @return \Illuminate\Database\Eloquent\Casts\Attribute<string, string>
     */
    protected function number(): Attribute
    {
        return Attribute::make(
            get: fn (string $value): string => "{$this->team->handle}-$value",
        )->shouldCache();
    }

The message I get is: Parameter $get of static method Illuminate\Database\Eloquent\Casts\Attribute<mixed,mixed>::make() expects (callable(mixed, mixed): string)|null, Closure(string): non-falsy-string given.

Your generic should be Attribute<string, never> since you don't have a set method

@calebdw Thanks for the pointer, however, I changed that but am still getting the same error: Parameter $get of static method Illuminate\Database\Eloquent\Casts\Attribute<mixed,mixed>::make() expects (callable(mixed, mixed): string)|null, Closure(string): non-falsy-string given. Feels like I've tried every single combination of typing to get this one to pass. Any other ideas?

@nsrosenqvist, please open a new issue to continue the discussion in (can link to this issue).

Hmm, today when I test this, the error is gone 🤔 I'll hold off creating a ticket until it reappears again.


Update Lachesis Hardware pages

Issues page
Bootloader page (still needs work but I will add more later)
Updated main page for new board, new findings

discovered katex is misconfigured, renders the text twice with different styling. Tried to fix. couldnt. didn't commit any config changes

Dont understand why Yarn is updating but it keeps causing conflicts that I dont know how to fix and dont have time to learn at the moment

other people been uploading with no issues regarding to yarn.lock please fix this !


Double reference to an author in a cheep in DataModel.cs

Both ´AuthorId´ and ´Author´ tracks the author of a given cheep, which causes the following error:
"The instance of entity type 'Author' cannot be tracked because another instance with the same key value for {'AuthorId'} is already being tracked. When attaching existing entities, ensure that only one entity instance with a given key value is attached."
Temporary workaround: Only instantiate a cheep with either AuthorId or Author object
Permenant fix: Remove either AuthorId or Author object from the Cheep object in the datamodel
(I am rooting for removing 'Author'

Just a headsup, the DBInitializer.cs file, which seeds the database has both Author and AuthorId, removing 1 would mean manually adjusting 600 lines.

This turned out to not be a real issue as it's not stored double in the actual database.


[VQC-CHALLENGE] [RaFlaNeQ] [27.10.2022]

Team members
Rahul Bhowmick
Neelkanth Rawat
Rajarsi Pal
Submission summary
We added a feature map and discriminator circuit of depth=2. We fixed the iterations at 50 at a stepsize=0.2 and batchsize =40. The accuracy on amazon SV1 simulator is 100%.
Additional comments
None

Congrats! Your model achieved 100% accuracy on the judging data set and is currently tied for 1st on the leaderboard.
We will soon release an optional second leg of the challenge, which will involve solving the same classification task, but this time using a kernel estimator instead of a VQC. This will act as the tie-breaker between teams whose VQC models achieve 100% accuracy.


add US ANSI layout variant

This is not professional quality work; it's my first time working with vector graphics. The left/right position of symbols on the keys might need some fine adjustment.

NOTE: I added the custom SUPER key text I requested to this PR; I'll make a second branch and add a PR with the usual TUX logo for anyone else who wants to make a custom layout.

Here is the updated pull request as this one has a couple of mistakes https://github.com/tuxedocomputers/keyboard-layouts/pull/61


generated .img is not recognized as a bootable disk by qemu

I get the following error:

I used this command to run it: qemu-system-i386 -m 1024 -hda toaruos-disk.img -vga std -sdl (after make toolchain and make)

The image built by the standard toolchain explicitly isn't a bootable disk. Look at the various make targets for running QEMU, such as make run/make kvm.


[BUG] Short bug description

How do i fix this?
node_modules\discord-music-player\dist\managers\Queue.js:117
throw new __1.DMPError(__1.DMPErrors.VOICE_CONNECTION_ERROR);
^
DMPError [VoiceConnectionError]: There was an Error while starting the Voice Stream

nvm i fix it :" sorry

How did you fix this got the exact error right now,,,

Add Voice intents to your client.

forgot to mention voice intent is already enabled as well as ffmpeg-static being installed.

Add Voice intents to your client.

never mind. was due to the bot lacking permission to join the channel. This should be stated more clearly...

I will state it more clearly next update, thanks for letting me know.

I will state it more clearly next update, thanks for letting me know.

no stress cheers for being awesome


Fixed: Bug | LoggedIn User does not see correct Follower count

fix issue: #168

_ No description provided. _

Attach the Issue link to the PR description.
You can do this by adding # followed by the issue number.

updated the code along with the description

Merging this PR. Next time ensure that you squash your commits into 1 for bugfixes.
You can know more about git squash here


chore: add User-Agent prefix to test helper client

Adding a different User-Agent for the test helper client will help us correctly identify traffic that was generated from running CLI commands as opposed to traffic that was generated by using the SDK directly.

This was broken out of #423 and should be merged before that PR.


Add set_with_type to writer

Related to #18

sorry for delay. Will have a look soon. But quite busy these days :(

will have a look this weekend, i promise ...

No rush

OK, so I'm merging it, but I will integrate set_with_type into set. There seems to be no benefit from a separate method

👍  great. thanks.


Monobuild request: Import bottom up so I can refactor across multiple projects (composite build)

I am setting up a monorepo outside twitter and there are many features that twitter's build system had(pants) that I would LOVE to see in gradle.
This relates to the previous request I opened as well.  In Intellij(and eclipse), I can open a project and all the projects it depends on BUT I want to do the reverse.  I want to open a library and the THREE things that depend on that library so that I can refactor library and Intellij will change all 4 projects.  Currently refactoring is a bit of a pain unlike at twitter's monorepo where I can bring in all 4 projects into intellij just with the click of a button.

The feature request as it is will not be done in Gradle. However Gradle is working on different things that might end up helping because they would allow either to not pay configuration time at each invocation and/or to select which projects to import in an IDE as opposed to all of them.


mWeb - Settings – Banner in About, Preference and Security is partly visible when open it Offline

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.39-0
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to https://staging.new.expensify.com/
Login and go Offline
Navigate to Settings
Click on About
Scroll to view the banner
Click on Preferences
Scroll to view the banner
Click on Security
Scroll to view the banner

Expected Result:
Banner in About, Preference and Security is fully visible when open it Offline
Actual Result:
Banner in About, Preference and Security is partly visible when open it Offline
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/d01e76de-ec2d-4ca5-9cc5-82c3660be648
View all open jobs on GitHub

I don't think this has anything to do with offline. It just seems like the new design we implemented here https://github.com/Expensify/App/pull/35662
can someone confirm @mountiny @shawnborton ?

Oh I think "partly visible" means we are missing the animation section. Got it now. Yes, reproduced on Android web.
Given that this only affects offline and very specific to this platform I'm not sure we need to make this a deploy blocker, but seems possibly related to that PR linked above.

Interesting, thanks, I think this can be handled by SWM as a follow up @kosmydel @filip-solecki

Sure, we will work on this!

All of the images should be in the card now

so I feel like this hsould be not repro anymore

On main

Shouldn't it be closed as we moved illustrations on those pages to Card component? (as @mountiny mentioned above)

Ah, good call. Closing.


⚠ ESPN has degraded performance

In e81ac2b, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 607 ms

Resolved: ESPN performance has improved in b8e36fb.


SAK-50438 Rubrics trigger event so that criteria get updated on a reorder

Add unit test

conflicts

I see that you merged my other rubric PR, so I'll get this one updated to fix the conflicts, hopefully tonight.


Fix empty menu being displayed like an error

fixes #59
This should work for now :)

So prettyyyyy! ✨ 😍


Joystick question

Cool project,
How do you have joysticks implemented into ue4? I’ve been trying to get my FrSky taranis (running EdgeTX firmware) through the raw input plugin.
I’ve been able to get it running through python using the winmm.dll using some code I found here: https://stackoverflow.com/questions/60309652/how-to-get-usb-controller-gamepad-to-work-with-python
I see that my product and vendor ids differ to the ones you are using in your source files for this project.

Thanks, Bryan!  I too struggled with various ways of getting joysticks / game controller / RC transmitters input to UE4/5, and settled on writing my own standalone C++ class to do it.  Most of the relevant code (product ID detection, axis polling) is in the header file, with the .cpp file containing only just the Windows code for reading the axes and buttons.   When I get a new game controller, I do the following:
(1) Get the device's product ID (which you appear to have done, getting 0x4f54)
(2) Use the Game Controller setup dialog to map the device's axes.
(3) Add a symbolic constant to the product IDs in the header file.
(4) Add a case statement for the product IDto the switch in the poll() method.
I've attached a couple of screenshots illustrating this process.  Let me know if you run into any trouble.

Cool, Yeah I’ll do it through the win32 api and forget about trying to get unreal to read it correctly. I had the same issues when trying to get a PS4 controller to work in unreal for a hexapod simulator and had to rely on third party intermediary apps to turn it into a different generic joystick so unreal would see it on the raw input plugin.
From the looks of it (have not tried) but the ability to remap input axis in ue4/5 look to be easy and even blueprintable. Probably something to leave till later and build a whole UMG interface to make the axis mapping easier.

FYI I’m running UE5.0.3 and VS2022 and didn’t have to do anything special to get them working together. Even your MultiSim project compiled and ran correct. Python server runs as well and talks to ue5 for me during play. It doesn’t pick up my FrSky taranis though from what I can tell. I tried changing the PID to the id that mine shows up in windows (0x4f54). The difference in PID is likely attributed to the fact that I’m running edgetx.org firmware instead of OpenTX on the taranis and they wanted to differentiate the firmwares.
I’m not sure how the devs that built Liftoff did it because it shows up correctly. Possibly they query all available joysticks and save the final joystick config mapping in an ini file somewhere.

Cool, maybe I should give UE5+VS2022 another try!


Support for network labels

docker-client lacks support for network labels and searching networks based on network labels.
I'd like to contribute this. Would this be appreciated?

Absolutely!

OK, then I'll go for it.
This might take a month or so because I can't work on it with high priority.

[X] query network labels (#705)
[X] create networks with labels (#705)
[ ] add filters to listing networks


[HOLD for payment 2023-10-27] Rename CONST.IOU. MONEY_REQUEST_TYPE to CONST.IOU.TYPE

Problem
This constant name isn't really correct. It does not hold the type of "money request" (which is manual/scan/distance), but instead it holds the type of IOU (which is split/request/send).
Why is this important?
It creates confusion in the code
Solution
Rename the const to CONST.IOU.TYPE

Oops, I did that wrong. Ignore me please, @kadiealexander :)
This was an issue handled internally so it is fine to close and no one requires payment.

Oops, sorry about that! I swear I looked for a review... @kadiealexander can you get that paid out please?

Payouts due:

[ ] Contributor+: $500 @thesahindia (via Manual Requests)

Eligible for 50% #urgency bonus? No

Shifting to weekly while the manual request is processed :)

$500 payment approved for @thesahindia based on summary above.

What's the next step with this one? Can it be closed out?

Ah, yup! Sorry I missed Jason's ping.


[] for array bindings instead of ()?

I'm just curious why () was chosen instead of [] for array binding patterns:
Foo[y] = x;

vs.
Foo(y) = x;

I'm just curious why () was chosen instead of [] for array binding patterns:
Foo[y] = x;

vs.
Foo(y) = x;

The syntax Foo[y] = x is already valid and meaningful JavaScript and thus can't be used for this. () was chosen because a call expression isn't a valid assignment target and will always throw when assigned to today.


How about adding properties that provide optical zoom min value?

i'm using this open source very well! Thank you to all the contenters.
However, there is a function that I would like to add.
Why not add properties that provide the minimum magnification (1.0, 0.5, 3.0, 5.0 and so on) of the camera's physical lenses (ex. builtInWideAngleCamera, builtInTelephotoOCamera, and builtInultraWideCamera) that you have for each type of device? That property will likely be used a lot for customizing the camera.

Would that be a property on the camera or does it vary per device (do some devices have built-in wide angle that go to something other than 0.5)?  If you have a mapping, I can add it to http://github.com/kudit/Device


Update merge_all to run with python 3.

parens for print() and remove .decode('utf-8')
I've also formatted it slightly.
Fixes #72

Thank you for the work.


How can I close the advertisement on Youtube Player?

Hi, I'm developing an app playing youtube video clip.
I want to close the advertisement but I can't because the playback button shows up. Is there anyway that I can do?
Thank you in advance!

You should be using youtube_player_iframe on that case.

You should be using youtube_player_iframe on that case.

Thank you for the reply!


Dev tunnels extremely slow all of the sudden

Running
devtunnel delete-all
devtunnel create servefast
devtunnel access create servefast.euw --anonymous --expiration 30d
devtunnel port create servefast.euw --port-number=5000
devtunnel port create servefast.euw --port-number=5001
devtunnel port create servefast.euw --port-number=8080
devtunnel port create servefast.euw --port-number=9099
devtunnel port create servefast.euw --port-number=9199
devtunnel port create servefast.euw --port-number=9000
devtunnel port create servefast.euw --port-number=9998

devtunnel host servefast.euw

devtunnel access list
devtunnel access delete servefast.euw --index 0

I have been using this for 2 months now and had no problem until today now all of the sudden my angular app takes 40s to load. When i access it on localhost it is instant and my internet connection is 200mbit/30mbit so it should be fine.
Loading times:

Please run devtunnel clusters --ping from your device to check the latency to the dev tunnel regions.
Also, you can run `devtunnel limits to ensure you haven't hit any quota limits.
Further limits are documented at https://aka.ms/devtunnels/limits

@servefast-cto Can you point to a different region and let us know if that helps?
devtunnel create --service-uri https://eun1.rel.tunnels.api.visualstudio.com
devtunnel post create TUNNELID --port-number PORT
devtunnel host TUNNEL ID

@derekbekoe thank you but tunnels started working today. I spend 100$ for ngrok in the mean time :)

Okay thanks. Closing this issue.

@derekbekoe happening again tunnels just stop working after some time


Properly detect SCSS violation line

After the upgrade of the scss-lint gem the format for the violations
changed. It caused us to pick up the column number of the violation as
the line number. This is a fix for it, with slightly more complex SCSS
code in the test.

:+1: :house:


File browser prototype

This adds a notebook with a load widget containing a file browser. This currently works but I am not that happy with the usability and would be interested in what others think.
See file_browser_examples.ipynb for a fuller explanation.
There is a question about security and access permissions on whatever the final server solution is here as well. This is best handled by setting up the user permissions on the server correctly however. Here I just attempt to implement a way easily browse to whatever directories or files you do have access to.

I think you are right when you say the look and feel is a bit "unusual". Here is a screenshot for the benefit of those who don't want to install:

That beging said, unless you have any better idea, maybe we could just use this for now and see what (if any) complaints we get in practice?

I think you are right when you say the look and feel is a bit "unusual". Here is a screenshot for the benefit of those who don't want to install:

That beging said, unless you have any better idea, maybe we could just use this for now and see what (if any) complaints we get in practice?

I think what users will most likely miss is options for filtering (by file type) and sorting (e.g., by creation date instead of alphabetically).

I think what users will most likely miss is options for filtering (by file type) and sorting (e.g., by creation date instead of alphabetically).


Support skipping polling.

fixes: #57

Hey, its now relased under v1 version for actions? I tried it but it doesnt look like it uses new code.

Hey, its now relased under v1 version for actions? I tried it but it doesnt look like it uses new code.

Not yet. Tag should be moved in a few days. Internal repos run off master for a few days to ensure we aren't regression anything before moving v1 tag.

v1 tag moved to v1.3.0

@iBotPeaches Thanks a lot! Tested it works very well 👍
But i think there is issue with colors definition as you can see in screenshots.

But i think there is issue with colors definition as you can see in screenshots.

thanks good find - https://github.com/sourcetoad/aws-codedeploy-action/pull/59


Integration with Azure Cognitive Search

This would be great - I'm building an enterprise chat but want to extrapolate on this eventually to add agents, etc. using langchain.

Hey people, I'm working on this - see: https://github.com/hwchase17/langchain/compare/master...UmerHA:langchain:master
Will open a PR when I've added integration tests.
@vowelparrot iiuc, you're the maintainer for tools/toolkits and therefore receivers. Could you assign this issue to me?
Thanks!

This can be closed since #4406 was merged

fyi @leo-gan This issue can be closed as it's completed


llvm 3.9: Download and build Polly

Motivation for this change
#20890
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Well, this is going to take a while to check and build...

Well, it looks like the travis instances don't have enough RAM to build it.

So, I get this error:
bashCMakeFiles/clang.dir/cc1_main.cpp.o: In function `cc1_main(llvm::ArrayRef<char const*>, char const*, void*)': cc1_main.cpp:(.text._Z8cc1_mainN4llvm8ArrayRefIPKcEES2_Pv+0x35b): undefined reference to `polly::initializePollyPasses(llvm::PassRegistry&)'

If I can't figure out how to appropriately patch the CMakeLists.txt, I don't think I can get around building polly twice: once in llvm.nix, and again in clang.nix.
I think this might be an example of the perfect use-case of multiple build outputs. Polly has to be in the llvm/tools/polly dir upon building clang in order to statically link it, unless the cmake file is patched.

@shlevy has also worked a fair amount on LLVM, if I remember correctly

So, I currently get the following error when using clang: : CommandLine Error: Option 'aarch64-ccmp' registered more than once! LLVM ERROR: inconsistency in registered CommandLine options
Possibly related: https://llvm.org/bugs/show_bug.cgi?id=24953
It might be because it's linking against a .so instead of a .a?

It looks like LLVM_BUILD_LLVM_DYLIB is broken, actually.

There's this message, too:
WARNING: substitution variables should be valid bash names, "ccLDFlags+" isn't and therefore was skipped; it might be caused by multi-line phases in variables - see #14907 for details.

See here.
I think there might be a problem with the spacing. I'll try changing that.

I think it's a problem in this bit of code.
It thinks that ccCFlags+= is a variable being assigned called ccCFlags+, as opposed to the variableccCFlags being updated.

Damn, #ddb1321 didn't fix it. Anyone have any ideas?

By the way, did the problems motivating https://github.com/NixOS/nixpkgs/pull/20941/commits/31bdc2f770b28e7bfc46d24f665dad3c7c888810 ever get reported/fixed in Nixpkgs master?
Also--what's the status of this? Are there outstanding problems with this
(other than probably needing a squash/rebase to cleanup the history)?

I don't think it's been fixed, and I've never seen that problem causing anything more than warnings.

This needs to be rewritten for newer llvm expressions


Multiple paths

Hi Tuurlijk,
Both things (Gitlab & multiple paths) are still a thing. I solved them in my fork for myself but I don't think that the solution would be suiteable for everyone which is why I closed the PR.


Create Optional examples

Add some examples : it already exist for Prism and Iso.

@mdulac these examples are mainly there because of a https://issues.scala-lang.org/browse/SI-7139 which is going to be fix in scala 2.12.1. Once it is done, I would like to move almost all examples to the website.
Sorry for the lack of documentation :(


Call For Maintainers

When I start longclaw, I was running an e-commerce website and the marriage of wagtail with some basic e-commerce functionality perfectly met my needs for a lower cost than using a hosted solution.
Now, I work on totally different areas of the web and have no e-commerce interests and no time to keep this project going.
Which is a shame, since I think there is a need for an e-commerce solution tightly integrated within Wagtail.
If anyone is interested in taking this project forward, please get in touch below. I am happy to add interested & active developers as full maintainers and/or move longclaw & related projects to their own Github org.

Hi James,
I'm using it in a project and would like to get it to support Django 3. I'd be happy to help maintain it along with anyone else who might like to help. I wont be able to give it a lot of time but hopefully it wont need a lot once its on 3.2

@sam-mi I've invited you as a collaborator

Hi @sam-mi,
I am also using this project on several projects (with a friends). i would like to work on it also.

@sam-mi
@simo97
You have been added as collaborators to the repository - you'll need to accept the invite

I am still interested in this project as well

Salut James,
Apologies. I was away and the invitation has lapsed, would you mind re-inviting me when you have a moment?

Hi James, I also could potentially help out, I'm very busy, but I could do what I can.

Hi James im also interested in being part of this project

Hi
@DarkArtek , @chrisbarry @tekanokhambane
I have sent you invites to the repository

Hi James,
I wasn't able to get to the invitation in time, would you please resend it?
Thanks,
Sam
On Tue, May 18, 2021 at 4:26 PM James Ramm @.***> wrote:

@sam-mi https://github.com/sam-mi
@simo97 https://github.com/simo97
You have been added as collaborators to the repository - you'll need to
accept the invite
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JamesRamm/longclaw/issues/375#issuecomment-842883987,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBH3MJDNNN4JUXFJJINRPDTOICATANCNFSM42QS5X5Q
.

@sam-mi
resent

HI James, i'd like to help you mantain the project. Is there a TODO list?

@JamesRamm Are you still looking for maintainers? We would like to use longclaw in our projects, but it seems it needs some work done. I'd be happy to hop in on this and maybe some other team members in the future as well.

@Xergon-sci
Yes absolutely. I will send you an invite

Watching this issue, as my team is also interested in seeing this package resurrected. Have the volunteers in this thread managed to meet up and are there next steps / a roadmap of any kind yet?

Well i am about to start a few ecommerce and multi-vendor projects so i did love to join so that the features I need are created and added to this repo. So @JamesRamm you can add me to the list. Thanks.

Hey all. I spoke to James on email and gained member status on the org. I can get PRs merged where they're needed but am not actively using this project. I just want to ensure that there's some movement on this important repo!
It would be good to get everyone together to figure out how we can collaborate. I don't have lots of experience with open source governance so I'm just spitballing here, but some next steps to me looks like:

[ ] Clean up issues
[ ] Backlog of priority issues that means this repo can be spun up

[ ] Propose other issues to go in https://github.com/longclawshop/longclaw/milestone/10
[ ] E.g. propose issues that will get PRs working again (e.g. Travis CI — is this still usable, and what needs to be changed to make that work?)

[ ] Review of forks of this repo that could be merged back in

Reach out to these maintainers to discuss this.

[ ] Set up some community governance. See https://github.com/longclawshop/longclaw/community

[ ] Create a code of conduct
[ ] Create issue template(s)
[ ] Create a pull request template(s)
[ ] Set up discussions section — @JamesRamm
[ ] Propose that more contributors get repo privileges

[ ] Identify a direction of travel past 'OK, it works'

[ ] Propose "what this is" / "what this is not"
[ ] Review the milestones and propose changes

CC @sam-mi @simo97 @DarkArtek @chrisbarry @tekanokhambane @oneyedoll @xergon-sci @whizfactor: could any of you pick up any of these tasks, start making some issues / docs / proposals / PRs towards them? Or propose alternative ways forwards.

Hey,
Thank you for doing this! I'm not sure I'm going to be help that much, but I have recently built a fairly large (600+ product variant site) and I would have some architectural thoughts that I learnt from doing this.
Essentually, I think I'd like to see a version of longclaw that is stripped back a bit, almost to a core first, which would include the most basic of things first (Basically the Product, ProductVariant & Cart functionality) and essentially just start with that, and then have other stuff / things added via extra dependencies or apps. Perhaps also having a route to starting without using the longclaw command (this probably already exists, but it can be a bit overbearing to see things having a templater, unless they are wagtail of course ;)
The reason for this is that in this project, I quite often found that I just wanted to implement some things myself or had to make the api conform to a pre-built frontend, so it was / is going to be easier just to do those things myself, so it would be nice to feel that long claw could get out of your way if you wanted it to..
TBH I didn't have any problem with anything in long claw (I'm not sure if carts should have their own model object, or just linking the items by session key as you've done is better, again, wasn't really a problem, I just might have had a cart object, but maybe that makes more of a mess!)
Perhaps also we should consider who is our key user? Is it someone who is experienced with wagtail, or might this be the first wagtail site they work with. Just a thing to consider about who the solution is aimed at.

Hello
i m not a programmer but i can do some test to help

@JamesRamm Many thanks I have accepted the invitation 😄

Hi, James currently looking at integrating this with a project and wouldn't mind tagging along for the ride, especially being my first attempt on an open source colllaboration, thanks. ALso thanks for this great Repo

Hi all. My team's workload shifted and this no longer is a priority area for us, so I can be removed from the maintainer's list.


Scaling not working, external scaler throws exception

I want to use KEDA for durable functions. I have been following Getting Started doc of this repo to test scaling of durable functions, but can't get to make it work. I have a simple durable function orchestration like this one. Also tried with QueueTriggered orchestration (actually this is what I to implement in my project), but doesn't work for that too.
Steps which I followed:

Deployed KEDA to keda namespace.
Deployed secret (create_secrets.sh) and external scaler to keda namespace.
Deployed the function app to default namespace along with secrets converted to base64.
Pods are running fine. I can start the orchestration by hitting the URL (http://localhost/api/Function1_HttpStart).
Started load testing. Number of pods didn't scale up as expected.

Logs of events and external scaler pod:
kubectl get events
FailedGetExternalMetric        horizontalpodautoscaler/keda-hpa-durable-keda   unable to get external metric default/ScaleRecommendation/&LabelSelector{MatchLabels:map[string]string{deploymentName: durable-keda,},MatchExpressions:[],}: unable to fetch metrics from external metrics API: No matching metrics found for scalerecommendation FailedComputeMetricsReplicas   horizontalpodautoscaler/keda-hpa-durable-keda   invalid metrics (1 invalid out of 1), first error is: failed to get ScaleRecommendation external metric: unable to get external metric default/ScaleRecommendation/&LabelSelector{MatchLabels:map[string]string{deploymentName: durable-keda,},MatchExpressions:[]LabelSelectorRequirement{},}: unable to fetch metrics from external metrics API: No matching metrics found for scalerecommendation
kubectl logs keda-durable-external-scaler-86c6f85bcc-s77qz -n keda -f
Request starting HTTP/2 POST http://durable-external-scaler-service.keda.svc.cluster.local:5000/externalscaler.ExternalScaler/GetMetrics application/grpc info: Microsoft.AspNetCore.Routing.EndpointMiddleware[0]       Executing endpoint 'gRPC - /externalscaler.ExternalScaler/GetMetrics' dbug: Grpc.AspNetCore.Server.ServerCallHandler[1]       Reading message. info: Keda.Durable.Scaler.Server.Services.ExternalScalerService[0]       Namespace: default DeploymentName: durable-keda GetMetrics() called. fail: Grpc.AspNetCore.Server.ServerCallHandler[3]       Error when executing service method 'GetMetrics'. System.NullReferenceException: Object reference not set to an instance of an object.    at Keda.Durable.Scaler.Server.Services.ExternalScalerService.GetMetrics(GetMetricsRequest request, ServerCallContext context) in /src/src/Keda.Durable.Scaler.Server/Services/ExternalScalerService.cs:line 112    at Grpc.AspNetCore.Server.Internal.CallHandlers.UnaryServerCallHandler3.HandleCallAsyncCore(HttpContext httpContext, HttpContextServerCallContext serverCallContext)    at Grpc.AspNetCore.Server.Internal.CallHandlers.UnaryServerCallHandler.HandleCallAsyncCore(HttpContext httpContext, HttpContextServerCallContext serverCallContext)    at Grpc.AspNetCore.Server.Internal.CallHandlers.ServerCallHandlerBase3.<HandleCallAsync>g__AwaitHandleCall|17_0(HttpContextServerCallContext serverCallContext, Method2 method, Task handleCall)
Specifications: Docker Desktop for Windows (<IP_ADDRESS>), Kubernetes (1.18.8)
Please let me know if more information is needed, I'd be happy to dig out as much as I can. I'm still new to Kubernetes so not sure whether this is a bug or I'm just missing something. Thanks!

Which version of KEDA are you running?

I'm using 2.0 stable version of KEDA, which I installed using helm: helm install kedacore/keda --namespace keda --name keda.
I also tried using the release candidate version helm install keda kedacore/keda --version 2.0.0-rc --namespace keda, but it shows this error while deploying the function app:
error: unable to recognize "deployone.yml": no matches for kind "ScaledObject" in version "keda.k8s.io/v1alpha1"
Deployment and service are created, but fails for ScaledObject.

We have changed the name of the scaled object type, you can find it in our migration docs.
However, I think this scaler has not ben migrated to 2.0 as well, right @TsuyoshiUshio ?
Function tooling is being updated to support 2.0 and should land soon.

Thank you for your guidance @tomkerkhove, I changed values of scaled object after referring to migration docs, and I'm able to run 2.0.0-rc version. However, I'm getting the same error.
Will wait for @TsuyoshiUshio 's inputs regarding migrating the scaler to 2.0. Please let me know if I could be of any help. Thanks!

@rohannevrikar
Sorry for being late. Yes. We need to migrate to 2.0.  I need to work on it.

Hi @samfisher07
Sorry for the inconvenience. I'd like to introduce, one workaround.
You can consider SQL backend for the Durable.  @cgillum develops SQL backend and keda support it.  SQL Provider is developed for k8s usage since on-prem scenario sometimes, they can't access to the Storage Account. If you are interested, please have a look!
https://techcommunity.microsoft.com/t5/apps-on-azure/new-storage-providers-for-azure-durable-functions/ba-p/2382044
MSSQL scaler
https://keda.sh/docs/2.2/scalers/mssql/
Durable Task SQL Provider
https://microsoft.github.io/durabletask-mssql/#/taskhubs

Hi @TsuyoshiUshio,
Thanks for the workaround, it works very well.
Not sure if this belongs here, but I had to modify the "GetScaleRecommendation" query to return a minimum value of 1 for the mssql scaler to work correctly with durable functions.
Cheers,
Sam

@samfisher07  I’m glad to hear the SQL provider worked well for you!
Would you be willing to share more details of your feedback on dt.GetScaleRecommendation here? https://github.com/microsoft/durabletask-mssql/issues?


Edit

👷 Deploy Preview for adoring-leavitt-e31a11 processing.

Name
Link

🔨 Latest commit
c3243f6777efa2c715125c20e06f08829eb3bf00

🔍 Latest deploy log
https://app.netlify.com/sites/adoring-leavitt-e31a11/deploys/62ab2195d1a6cc00081465b5


Various fixes for 0.6.x

Fixes for #1236 to continue support on Jubilinux 0.2.0 (Debian Jessie):
-Modifies /etc/sources.list to use archive.debian.org
-Disables "Valid Until" check (certificates are expired on archive.debian.org)
Install node/npm v8 (using Nodesource) on the Edison
Install v1.7.0 of MRAA
Only install bluez from source if on Jubilinux 0.2.0 (Debian Jessie), Jubilinux 0.3.0 uses a newer version (5.43).

It's a bit of a pain to test it on the edison:

Do a fresh flash of jubilinux 0.2.0
Change the last few lines of the edison bootstrap to:

[bunch of wifi setup stuff]
sleep 10
echo -ne "\nWifi SSID: "; iwgetid -r
sleep 5
curl https://raw.githubusercontent.com/cluckj/oref0/master/bin/openaps-install.sh > /tmp/openaps-install.sh
)

Once that script completes, edit  /tmp/openaps-install.sh so that the lines towards the end are pointing to cluckj/oref0 instead of openaps/oref0, like so:

curl -s https://raw.githubusercontent.com/cluckj/oref0/master/bin/openaps-packages.sh | bash -
mkdir -p ~/src; cd ~/src && git clone git://github.com/cluckj/oref0.git || (cd oref0 && git checkout master && git pull)

Then run cd && bash /tmp/openaps-install.sh, as usual...

For anyone testing this before we get it released, here's a copy and paste version of the instructions for testing: https://gist.github.com/danamlewis/6f2cf0d997e834269063c48dd9d7dc2b

Tested with a freshly flashed jubilinux 0.3.0 Edison using the instructions at https://gist.github.com/scottleibrand/9372092ac914c7edbe23fa4eb94b83d8 and confirmed that it loops successfully on git://github.com/cluckj/oref0.git master branch without any SPI errors.


[Objective-C] [false-positive]?? buffer malloc-ed return null or malloc(0)

@autoreleasepool {
// warning here: memory dynamically allocated by call to malloc() at pos A is not reachable after ***
    ...
    Byte* buffer = (Byte*)malloc(length);    // pos A
    if (buffer == NULL || length == 0)
        break;
   ...
}

when malloc-call returns null or malloc(0), should I invoke the free() call? My coder do not think so.. so false-positive produced here.~

When malloc returns null you don't need to invoke free(). However, I cannot reproduce what is the problem just based on your snippet of code. Could you paste a bit bigger snippet of code, so that I can see why Infer is reporting a leak there? Also, could you mention in which line is Infer reporting the leak?

a-ha... I think the question is here:

when malloc-ed length is zero, we need manually free the "malloc(0)" memory? IF malloc(0), system wont allocate the memory, free is non-necessary.. so maybe we think Infer gives the false-positive... ?
But in reality, finally, I think my coder's problem is here: bcoz now Infer do not have a very deep defect guideline to guide the programmer why defect here..  Below, skip the second branch, MEMORY_LEAK will happened?

do
                {
                    //原图
                    NSData *imageData = nil;
                    QINT64 length = [presentation size];
                    Byte*  buffer = (Byte*)malloc(length);
                    if (buffer == NULL || length == 0)
                    {
                        break;
                    }

                    NSError* error = nil;
                    QINT64 readBytes = [presentation getBytes:buffer fromOffset:0 length:length error:&error];
                    if (error != nil || readBytes != length)
                    {
                        free(buffer);
                        break;
                    }

                    imageData = [NSData dataWithBytesNoCopy:buffer length:length];
                    if ([imageData writeToFile:filePath atomically:YES] == NO)
                    {
                        break;
                    }

                    //缩略图
                    NSString * thumbPath = [[self getThumbFolder] stringByAppendingString:[filePath GetFileShortName]];
                    if (![fileManager fileExistsAtPath:thumbPath])
                    {
                        fileMd5 = [SmartDeviceMsgCenter getFileMd5: filePath];
                        [self createThumbImageActual: filePath md5: fileMd5];
                    }
                }while (0);

In the man of malloc says: "If size is 0, then malloc() returns either NULL, or a unique pointer value that can later be successfully passed to free()."
It looks like what you get from malloc(0) is implementation dependent, and it is always safe to pass that result to free. See http://stackoverflow.com/questions/8808303/c-freeing-malloc0.
Infer is complaining on that case, when length is 0. So yes, in that case you should call free to be on the safe side.

but not considering this point, in the following code, if not entering the second branch[ if (error**)], the buffer still not be freed~? From this point, Infer should give the warning, but as u know, the tool do do not give a whole issue roadmap(not like clang), I think the current error message may make the programmer being misunderstooooood?  Like here:
@autoreleasepool { // memory .....by call to malloc at pos A is not reachable after current line..
do
{
    Byte*  buffer = (Byte*)malloc(length);  // pos A
    if (buffer == NULL || length == 0)
    {
        break;
    }
    ...
    QINT64 readBytes = [presentation getBytes:buffer fromOffset:0 length:length error:&error];
    if (error != nil || readBytes != length)
    {
        free(buffer);
        break;
    }
    imageData = [NSData dataWithBytesNoCopy:buffer length:length];
    if ([imageData writeToFile:filePath atomically:YES] == NO)
    {
        break;
    }
    ...
}while (0);
}

The message of Infer is only part of the information it provides. There is also an error trace where it shows you the path that lead to the error. You could try using the command inferTraceBugs to see the trace.

I indeed manually parsed the report.json (equals to interTraceBugs -o report_json_dir), but I do not think the information is enough for me ~ like what I have said in the above post~

OK, thanks for the feedback. We'll take it into account.


fix: button disappears when orientation changes

Fixes #2059  previous button disappear when orientation changes while creating event
Checklist:

[x] I have checked for PMD and check-style issues
[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream development branch.

Changes: override the onPageSelected method in CreateEventActivity.java
Screenshots for the change:

@liveHarshit  @iamareebjamal  please check this.

@nulllpointerexception Do not raise multiple PRs for the same issue, I'll not consider it again.

@nulllpointerexception There is no previous button in the attached GIF

@liveHarshit really sorry. this is my first time. updated the gif


Test_tools exploitation notes

I need to put more user and chat args into BotClient object. Now I use that workaround:

client = BotClient(dp)
client.user = User(...)
client.chat = Chat(...)

for bot command testcases it will be usefull to put BotCommand objects into BotClient.send()

FOO_COMMAND = BotCommand("foo", "my awesome command for usefull actions")
# Now
client.send("/" + FOO_COMMAND.command)
# Suggestion
client.send(FOO_COMMAND)

For history. Discussion about note 2 was here:
https://t.me/aiogram_dialog/30363

context manager instead (in addition to) .reset_history
start dialog and get result


Double-root when path() its called and sep == root

Code to reproduce the bug :
var Resolver = require('dotresolver');
var r = new Resolver();
r.setRoot('/');
r.sep = '/';
console.log(r.path('test'));

Ouput : "//test"
Expected : "/test"

Add condition in Resolver.prototype.path() to fix this issue.


Convert Names to CMS

This converts all the names I could find which referenced "Computational
Chemistry Cookiecutter" to "Computational Molecular Science Cookiecutter,"
including all the shorthand and substitution.
Re-licenses under MolSSI as owner instead of Chodera Lab
Fixed badges to point to new builds under MolSSI/cookiecutter-cms
Fixes #45

"Computational Molecular Sciences"


[SPARK-46470][PS][TESTS] Move test_series_datetime to pyspark.pandas.tests.series.*

What changes were proposed in this pull request?
Move test_series_datetime to pyspark.pandas.tests.series.*
Why are the changes needed?
move the test to the right place
Does this PR introduce any user-facing change?
no, test only
How was this patch tested?
ci
Was this patch authored or co-authored using generative AI tooling?
no

merged to master


how to use rdpwrap for one user as multi-session and another user as single session?

Please, read README and ask this in official Telegram chat (if don't understand).


[machinst x64]: enable packed integer comparisons and lane operations

This change was originally intended to implement the remaining lane operations (any_true, all_true, and splat) in order to enable the simd_lane.wast spec test; however, I had forgotten that comparisons were necessary for some of those sequences so this change adds those as well.

Not a huge fan of this; instead could we just xorps the destination register, in the place where it's used? (There's already special handling of xorps X X in regalloc.) The slow down caused by the additional instruction in the pipeline may be recouped by the speedup caused by the lane dependency clearing.

@bnjbvr, me neither really. I didn't like the idea of adding extra instructions either without data on the performance impact so I left it in and moved it to the bottom of the Inst enum. Maybe there is something that can be done to the context that will provide an escape hatch? E.g. some way to "unsafely" mess with the use/defs?

without data on the performance impact

Could we make a tiny benchmark out of this to compare the two variants?

@bnjbvr, created https://github.com/bytecodealliance/wasmtime/issues/2256 to try to get rid of XmmUninitializedValue.


add 8080 port in the supported ports for spaces

Add to documentation that port 8080 is now opened by default for spaces

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Examples are not working due to missing throttling configurations

I'm testing this with terraform 0.14.3, aws provider 3.19.0, and the module version 0.5.0.
After applying the examples, I have received the response:
HTTP/1.1 429 Too Many Requests
{
    "message": "Too Many Requests"
}

Then I changed the throttling configuration manually through the WEB console, to see if it was going to work. The requests then returned 200 successfully. To see what was changed, I ran terraform plan, and here is the difference:
  ~ resource "aws_apigatewayv2_stage" "default" {
        id              = "$default"
        name            = "$default"
        # (7 unchanged attributes hidden)

      ~ default_route_settings {
          - throttling_burst_limit   = 100 -> null
          - throttling_rate_limit    = 100 -> null
            # (2 unchanged attributes hidden)
        }
        # (1 unchanged block hidden)
    }

I saw that the throttling configurations are commented out. Is this something expected? Am I missing a configuration here? Here is my configuration:
module "api_gateway" {
  source = "terraform-aws-modules/apigateway-v2/aws"

  name          = "http-lambda-example"
  description   = "HTTP API Gateway Template"
  protocol_type = "HTTP"

  cors_configuration = {
    allow_headers = ["content-type", "x-amz-date", "authorization", "x-api-key", "x-amz-security-token", "x-amz-user-agent"]
    allow_methods = ["*"]
    allow_origins = ["*"]
  }

  domain_name                 = "${local.subdomain}.${var.hosted_zone}"
  domain_name_certificate_arn = data.aws_acm_certificate.hosted_zone_certificate.arn

  default_stage_access_log_destination_arn = aws_cloudwatch_log_group.logs.arn
  default_stage_access_log_format          = "$context.identity.sourceIp - - [$context.requestTime] \"$context.httpMethod $context.routeKey $context.protocol\" $context.status $context.responseLength $context.requestId $context.integrationErrorMessage"

  integrations = {
    "ANY /" = {
      lambda_arn             = module.lambda_function.this_lambda_function_arn
      payload_format_version = "2.0"
      timeout_milliseconds   = 12000
    }

    "$default" = {
      lambda_arn = module.lambda_function.this_lambda_function_arn
    }
  }

}

I see error:
The latest automatic deployment attempt failed.
Deployment attempt failed: Unable to deploy API because no routes exist in this API
Your stage has no deployment attached. Update your API to trigger a new deployment.

after such deploy.
I believe the problem is not because throttling_burst_limit but because default_route_settings and route_settings are commented out.
Did someone has the same issue?
Or maybe someone knows workaround?

Fixed in #30

@antonbabenko I think this can be closed with the latest fixes now added.


Crash on controller detect

I'm using anitech J235 game controller.
XOutput is first seen as broken and frozen, then, a little lag, and finally, terminates itself. But, after I tried unplugging my controller and open it, it works fine!
However, after plugging my controller back in, it exhibits the same issues.
I have my VIsual Studio 2019 running a debug from Task Manager to see if I can have more information. Here's what pops up when it finally closed itself:
Exception thrown at 0x0000000010003654 (EZFRD64.dll) in XOutput.exe: 0xC0000005: Access violation reading location 0x0000000040550090.

It seems to be a library for USB Vibration.

It appears that the official driver causes the issue by forcing the controller to use it's GameController Network instead of USB Joystick. So just uninstall it. And, thanks to this TruckerSMP forum topic to point me out that it might not work with the driver. And, since I solved it, this issue is closed here.


NeoVim Config: mini.pairs with buffer type

Follow this reddit post

Required result achieved with autopairs in 3971032


Image in readme has old variable syntax

Describe the bug
The image that is in the readme (and therefore on pub.dev) contains the old fast_i18n variable syntax, with {variable} instead of $variable.
Expected behavior
Correct syntax in readme (and docs).

Hi, slang supports multiple syntaxes. $variable, {variable} and {{variable}} are valid, you can configure it in string_interpolation

Interesting, I have probably done something wrong then.
I am migrating from fast_i18n to slang and I now realized that I have:
targets:
  $default:
    builders:
      slang:
        options:
          ...
          string_interpolation: braces

in my build.yaml file, but it only works if I use $, which seems to be the default?

You need to import slang_build_runner and use it in the yaml:
targets:
  $default:
    builders:
      slang_build_runner: // <--
        options:
          ...
          string_interpolation: braces

Here is the migration guide: https://github.com/Tienisto/slang/blob/master/slang/MIGRATION.md

Thanks a lot for the quick response! 💙


Automated cherry pick of #6849: fix: disable policy rule compaction

Cherry pick of #6849 on release/3.0.
#6849: fix: disable policy rule compaction

Codecov Report

Merging #6852 into release/3.0 will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@              Coverage Diff               @@
##           release/3.0   #6852      +/-   ##
==============================================
- Coverage         6.55%   6.54%   -0.01%
==============================================
  Files              556     556
  Lines            98578   98575       -3
==============================================
- Hits              6457    6448       -9
- Misses           91598   91603       +5
- Partials           523     524       +1

Flag
Coverage Δ

#aFlag
6.54% <0.00%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/cloudcommon/policy/policy.go
1.57% <0.00%> (-0.02%)
:arrow_down:

pkg/util/rbacutils/rbac.go
55.34% <ø> (+0.59%)
:arrow_up:

pkg/util/rbacutils/reduce.go
72.22% <ø> (-3.78%)
: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 78b70d9...3b9cd6d. Read the comment docs.


Change redacted behaviour on the line information

The line information sometimes contains useful context information so
instead of redacting the whole line, this change redacts only the
secret

appreciate these PRs! will be putting together a new release soon


Meson formatters do not recurse into child directories

Describe the bug

To Reproduce

Create a top-level meson.build file
Create a meson.build file with any child directory i.e. mkdir foo && touch foo/meson.build
Run formatter

For example mine:

It does format the top-level one though.
Expected behavior
Formats all files. I tried fixing this by using the rules here:
https://github.com/numtide/treefmt-nix/compare/main...RossSmyth:treefmt-nix:main
As that globs the files in my shell. But it still outputs that the files are not formatted.
System information
35dfece10c642eb52928a48bee7ac06a59f93e9a
https://github.com/NixOS/nixpkgs/commit/a1d92660c6b3b7c26fb883500a80ea9d33321be2
Additional context
I'll fix it if desired, I just need to know the globbing rules

Alright I have found rules that glob it


Extension causes high cpu load

Issue Type: Performance
Extension Name: LiveServer
Extension Version: 5.5.1
OS Version: Darwin x64 18.2.0
VSCode version: 1.31.1

:warning: Make sure to attach this file from your home-directory:  :warning:

Hello. few questions!!,
When you're getting this issue? after load of vscode or after clicking to Go Live ?
Is this happen every-time ? or random ?
Please reply here https://github.com/ritwickdey/vscode-live-server/issues/278
Duplicate of https://github.com/ritwickdey/vscode-live-server/issues/278

[Note: This issue is closed by a javascript code (written by me).
https://gist.github.com/ritwickdey/36682dabe4a992c57e4562c935bfbbdd ]

Thank You.


wrong regular expression for exact match scoring

Hi, I would like to report a problem about Natural Question evaluation, especially about regexes_to_ignore.
TL;DR
I believe we should change regexes_to_ignore in nq_open.yaml
from
- "\ban|a|the\b"

to
- "\\b(?:The |the |An |A |The |a |an )"

And I think this modification should also be considered for other tasks using exact_match such as math reasoning.
Why
Firstly I git clone the latest version(ada4a31d69fd58ce794b60944900bd1471fbad3e) and ran the 0-shot meta-llama/Llama-2-7b-hf evaluation on nq_open dataset with this command:
lm-eval --model hf --batch_size auto --task nq_open --model_args pretrained=meta-llama/Llama-2-7b-hf \
--output output/llama2-7b-0-shot-no-escape --log_samples

When manually checking the output, I found there are some weird exact_match score calculation, for example this one:
  {
    "doc_id": 160,
    "doc": {
      "question": "who is the biggest selling female group of all time",
      "answer": [
        "Spice Girls"
      ]
    },
    "target": [
      "Spice Girls"
    ],
    "arguments": [
      [
        "Answer these questions:\n\nQ: who is the biggest selling female group of all time?\nA:",
        {
          "until": [
            "\n",
            ".",
            ","
          ],
          "do_sample": false,
          "temperature": 0.0
        }
      ]
    ],
    "resps": [
      [
        "The Spice Girls"
      ]
    ],
    "filtered_resps": [
      "The Spice Girls"
    ],
    "exact_match": 0.0
  },

Adding An|A|The
Although we have set ignore_case=True and regular expression for filter out the as in the config file, it still have "exact_match": 0.0.
After digging into the code, I found the problem lies in the evaluate package. As stated here, it would do regex filter before lowercasing. So I believe we should also add An|A|The into the regexes_to_ignore term.

Adding whitespace
And since there is also no white_space_fixing operation as in the KILT, it simply uses re.sub, as shown here, it would cause problems such as "the Animal" to " Animal", which is incompatible with "Animal".
Adding extra backslash for \b
The last problem is that lm-eval would read the config using yaml.load method, and directly sending this regex string to third party evaluation toolkit (huggingface/evaluate). However, the evaluate function needs a raw string. So we should add other \ to \b.
BTW, I think this would also affect other tasks requiring exact_match score.
After doing all the modification above, I got the right outcome:
 {
    "doc_id": 160,
    "doc": {
      "question": "who is the biggest selling female group of all time",
      "answer": [
        "Spice Girls"
      ]
    },
    "target": [
      "Spice Girls"
    ],
    "arguments": [
      [
        "Answer these questions:\n\nQ: who is the biggest selling female group of all time?\nA:",
        {
          "until": [
            "\n",
            ".",
            ","
          ],
          "do_sample": false,
          "temperature": 0.0
        }
      ]
    ],
    "resps": [
      [
        "The Spice Girls"
      ]
    ],
    "filtered_resps": [
      "The Spice Girls"
    ],
    "exact_match": 1.0
  },

Thanks for investigating this and for the thorough reporting!
Would you be willing to PR this fix? I can proceed to look at whether it affects other tasks.


Fixing re flags deprecation warning in python 3.6

In Python 3.6 and beyond they have deprecated flags in re.compile that do not come at the beginning. I moved the flag to the beginning of the string instead.

Thanks!


Expand instruction_supported() to optionally check parameters

Summary
This commit expands the recently added instruction_supported() method of
the Target class to also enable optionally checking if a parameter on an
instruction is supported. This is anticipating future changes around how
instruction parameters are represented in terra more generally as part
of #7624 and specifically in the Target for #7797. Before we do any
refactoring around this having a standard API for checking if a
parameter is supported is useful. This commit expands the interface for
the insturction_supported() method to check any combination of
operation name, qargs, operation class, or parameters (either operation
name or class is required) and the method will return True if that
instruction can be run on the Target device.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

29 of 34 (85.29%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 83.524%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/transpiler/target.py
29
34
85.29%

Totals

Change from base Build<PHONE_NUMBER>:
0.005%

Covered Lines:
52569

Relevant Lines:
62939

💛  - Coveralls

I need to add a release note now since instruction_supported() was initially added in 0.20.0 this PR now is adding new functionality on top of that API now. I'll do that in a little while.

Oops my bad I left the comment to remind myself and forgot to circle back and add the release note. I just pushed it up in https://github.com/Qiskit/qiskit-terra/pull/7807/commits/2777b1fc0f38468fb8cde26e974448133edbc218


Auth: Password Reset Functionality

Create an API for password reset using email, allowing users to reset forgotten passwords.

assign it to me.


Play generated gif in reverse

How can I play the gif in reverse?

One way you can achieve this is by looping through the frames from the end of the video and adding them to the context.
This is beyond the scope of the library, but I would love to hear if that worked for you.


Use user initials if avatar image is broken

Use user initials if avatar image is broken

you might want to run npm install to so that the version in the package lock file gets updated too


migrate to null safety

I have updated all project files to support null safety.
This merge still needs to be checked and tested for errors.
But the project is already started and running.

Hey @viplifes, thanks a lot 👍 .  can you resolve the conflict? It seems to be caused by a different fork

Conflicts have been resolved.

@viplifes bug on hangup

Thanks, fixed. You can upgrade and try again.

Solved, thank you!

@viplifes on registering with the demo app i get the error:
I/flutter (15018): [2022-02-02 15:18:30.675] Level.error event_manager.dart:110 ::: type 'Null' is not a subtype of type 'bool'
I/flutter (15018): #0      new Challenge.fromMap (package:sip_ua/src/digest_authentication.dart:10:27)
I/flutter (15018): #1      RequestSender._receiveResponse (package:sip_ua/src/request_sender.dart:122:23)
I/flutter (15018): #2      RequestSender.send. (package:sip_ua/src/request_sender.dart:57:7)
I/flutter (15018): #3      EventManager.emit. (package:sip_ua/src/event_manager/event_manager.dart:108:17)
I/flutter (15018): #4      List.forEach (dart:core-patch/growable_array.dart:433:8)
I/flutter (15018): #5      EventManager.emit (package:sip_ua/src/event_manager/event_manager.dart:105:12)
I/flutter (15018): #6      NonInviteClientTransaction.receiveResponse (package:sip_ua/src/transactions/non_invite_client.dart:96:28)
I/flutter (15018): #7      UA.onTransportData (package:sip_ua/src/ua.dart:882:25)
I/flutter (15018): #8      Transport._onData (package:sip_ua/src/transport.dart:316:11)
I/flutter (15018): #9      WebSocketInterface._onMessage (package:sip_ua/src/transports/websocket_interface.dart:172:16)
I/flutter (15018): #10     WebSocketInterface.connect. (package:sip_ua/src/transports/websocket_interface.dart:92:9)
I/flutter (15018): #11     WebSocketImpl.co
This stops the whole registering proces. With the non null-safety version i get:
I/flutter ( 8017): [2022-02-01 11:54:30.796] Level.debug digest_authentication.dart:178 ::: authenticate() | using qop=auth [a2:REGISTER:sip:weburl.nl]
I/flutter ( 8017): [2022-02-01 11:54:30.801] Level.debug digest_authentication.dart:205 ::: authenticate() | response generated
I/flutter ( 8017): [2022-02-01 11:54:30.808] Level.debug transport.dart:168 ::: send()
I/flutter ( 8017): [2022-02-01 11:54:30.812] Level.debug websocket_interface.dart:131 ::: send()
I/flutter ( 8017): [2022-02-01 11:54:30.815] Level.debug websocket_dart_impl.dart:49 ::: send:
and after some seconds a registration succes:
I/flutter ( 8017): [2022-02-01 11:54:31.291] Level.debug sip_ua_helper.dart:144 ::: registered => Code: [200], Cause: registered, Reason: OK
I/flutter ( 8017): [2022-02-01 11:54:34.84] Level.debug websocket_interface.dart:169 ::: Received WebSocket message
I/flutter ( 8017): [2022-02-01 11:54:34.87] Level.debug websocket_interface.dart:174 ::: Received and ignored empty packet
How can we resolve this failure?

@jongerenchaos I fixed this and you can test it out, so we can move this PR forward.

Tnx! It works without any problem!


Exposure of the csrf_token field value

Previously Flask-WTF stripped away the csrf_token field value when accessing the form data. But in 42befd0420d1f4896a70339de3d474044461a1c9 this was removed.
Is this intentional? Now a form will expose the token as part of the data, even though it's an implicit value not generally useful outside the form.
I realize it is WTForms that implements the general logic for supporting CSRF validation, so maybe this is viewed as the responsibility of WTForms in the same way that form.populate_obj explicitly avoids populating the CSRF field value on the object. Sadly WTForms has no such filtering when accessing form.data, and the filtering in Flask-WTF was useful as it was.
I'll raise an issue with WTForms if that's where you think this should be fixed.

Is there any update on this issue? We are currently forced to pinning Flask-WTF to 0.13.1, while considering simliar issues/solutions described by @fdanielsen.

@slint We're still "manually" filtering out the CSRF token as needed. We would be happy to try and provide a pull request once time allows, but we've been waiting on feedback on whether or not this should be fixed in Flask-WTForms or WTForms.

@fdanielsen You can leave a helper function here right now. In case anyone seeks the same problem.

I've run into a tangentially related issue recently while updating old dependencies (including an old version of flask-wtf). For us, it broke forms dynamically generated and used by Flask-SuperAdmin.
Here's what fixed it for us:
from flask_superadmin import Admin, AdminIndexView, expose, model
from flask_superadmin.model.backends.mongoengine.orm import data_to_document

class OurAdmin(model.ModelAdmin):
    @classmethod
    def _patch_form_populate_obj(cls, form):
        # wtforms started returning csrf token as a part of the data as well,
        # which we don't want to use when populating models
        def new_populate_obj(form, obj):
            data = {k: v for k, v in form.data.items() if k != 'csrf_token'}
            data_to_document(obj, data)
            return obj

        form.populate_obj = new_populate_obj
        return form

    def get_form(self):
        return self._patch_form_populate_obj(super().get_form())

Sorry, I know this isn't terribly relevant to flask-wtf, but hopefully it'll help someone else who has the same problem and finds this github issue.

@lepture Sorry for not getting back with an example fix for others. Reminded by @AlecRosenbaum's recent post, I'll share our FlaskForm specific solution which is just a simple wrapper around the data property in a subclass:
class BaseFlaskForm(FlaskForm):
    @property
    def data(self):
        return dict(
            (name, f.data) for name, f in self._fields.items() if name != "csrf_token"
        )


kustomize: 5.4.1 -> 5.4.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/kustomize/versions.
meta.description for kustomize is: Customization of kubernetes YAML configurations
meta.homepage for kustomize is: https://github.com/kubernetes-sigs/kustomize
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.4.2 with grep in /nix/store/dr124zy69xxahw9iycvbbf1ss8jxh7m7-kustomize-5.4.2
found 5.4.2 in filename of file in /nix/store/dr124zy69xxahw9iycvbbf1ss8jxh7m7-kustomize-5.4.2

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

kustomize

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/ca06a4ab76ebbcc476b79a034a9d60e9f1dd5a3f#kustomize

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

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

1 package built:

kustomize

Maintainer pings
cc @carlosdagos @vdemeester @periklis @zaninime @Chili-Man @saschagrunert for testing.

Result of nixpkgs-review pr 317139 run on x86_64-linux 1

1 package built:

kustomize


Current LeaguePositions issue

 kayn.Summoner.by.name('hide on bush')
        .region(REGIONS.KOREA)
        .then(summoner => {
            const id = summoner.id;
            kayn.LeaguePositions.by.summonerID(id).then(rank => {
                //. . .
            })
        })

When I try to get all the leagues of 'hide on bush' I get the following error, Is this an issue or am I doing it wrong?
(node:9644) DeprecationWarning: Collection#filterArray: use Collection#filter instead
TypeError: Cannot read property 'by' of undefined
    at kayn.Summoner.by.name.region.then.summoner (C:\Users\VICTOR\Desktop\es-shiba-discord\index.js:61:38)

Hi, sorry for the late response. This has been deprecated in June 17th and in v0.9.10