FS2022 Thoughts.

This is my todo list, for public comment is the last item:

New icon, use the blue and the orange, maybe split in half, drop the 19 out of it

Check everything for "FS19"

Run some tests.

Update translations (pending next item)

Add "overweight" or "bulky" mods tab (title undecided) - these mods almost certainly work, but
they contain extra files that would have been caught if they got submitted to the modhub.
(png, wav, blend, etc) - it doesn't always mean it was a lazy conversion, but it might.

Seriously considering a file date gate on 2022 mods.  Or maybe the date on the internal files.  Not sure
how to go about this, but there is a ton of slap-dash conversion work happening right now, on the eve(ish)
of the editor being released. And it's gonna make it a nightmare to explain to folks why their mods crash
the game, maybe this is a good weeder - drop those with dates earlier than like a week from now into a "well, if you are sure" conflict pot?  Maybe whitelist those couple that hit the modhub fast?

Features so far:

Pick to either can 2019 or 2022 mods, location of gameSettings.xml is guessed based on selection.
Added a "extra files" check - (sortable column on explore mods) - details at botton of details page
Added an "old shaders" check - for FS19, detects use of FS17 shaders., for FS22, detects use of FS19 shaders.
Update modDesc version check to differentiate 19 and 22.
Added a "newest file" info field to the detail page - it is the file date of the most recent file in a zip archive


(chore) Fix minor typos

Fix various minor typos in :

project documentations,
javadoc and comments,
private field, constant or method names,
method attribute and variable names,
test class and method names,
exception messages.

Codecov Report

Merging #367 (f72e131) into master (4e24535) will increase coverage by 0.04%.
The diff coverage is 57.89%.

:exclamation: Current head f72e131 differs from pull request most recent head f939510. Consider uploading reports for the commit f939510 to get more accurate results

@@             Coverage Diff              @@
##             master     #367      +/-   ##
============================================
+ Coverage     84.53%   84.57%   +0.04%
- Complexity     2967     2968       +1
============================================
  Files           193      193
  Lines          7293     7293
  Branches        916      916
============================================
+ Hits           6165     6168       +3
+ Misses          886      884       -2
+ Partials        242      241       -1

Impacted Files
Coverage Δ

.../org/apache/commons/io/CloseableURLConnection.java
19.35% <0.00%> (ø)

...c/main/java/org/apache/commons/io/EndianUtils.java
100.00% <ø> (ø)

...rc/main/java/org/apache/commons/io/FileSystem.java
55.29% <ø> (ø)

...in/java/org/apache/commons/io/FileSystemUtils.java
76.87% <ø> (ø)

src/main/java/org/apache/commons/io/FileUtils.java
94.50% <ø> (ø)

...main/java/org/apache/commons/io/FilenameUtils.java
92.38% <ø> (ø)

src/main/java/org/apache/commons/io/HexDump.java
94.59% <ø> (ø)

src/main/java/org/apache/commons/io/IOCase.java
91.17% <ø> (ø)

...a/org/apache/commons/io/StandardLineSeparator.java
100.00% <ø> (ø)

...commons/io/comparator/CompositeFileComparator.java
100.00% <ø> (ø)

... and 28 more

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 4e24535...f939510. Read the comment docs.


GCE Backend Service quota now 5 by default?

The quota section of the beta limitations page states that projects have a quota of 3 backend services by default.
I just created a new GC project, and enabled GCE, and it appears that you now get 5 by default.

@nicksardo I created a pr, signing cla now.

/close
Thanks


First Time Setup Question w/xTeVe & Plex + unRaid

Hello!
First off thanks for the very awesome sounding piece of software. I've been running into a bit of an issue on I believe the final steps to get everything working.
My setup is currently entirely in unRaid
EPlus seems to be up and running fine "Server started on port 8000"
I then added the address http://<IP_ADDRESS>:8000/channels.m3u to xTeVe which came up showing 100 streams.
Followed the same steps for XMLTV with xTeVe entering http://<IP_ADDRESS>:8000/xmltv.xml
On Plex I then added my xTeVe tuner which properly showed my newly assigned channels.
However when I go to access the channel I receive an error statingCould not tune channel. Please check your tuner or antenna.
From xTeVe logs I see:
EPlusTV 9
[xTeVe] Client User-Agent:      Lavf/LIBAVFORMAT_VERSION
[xTeVe] Streaming URL:          http://<IP_ADDRESS>:8000/channels/9.m3u8
[xTeVe] Streaming Info:         URL was passed to the client.
[xTeVe] Streaming Info:         xTeVe is no longer involved, the client connects directly to the streaming server.
Navigating to http://<IP_ADDRESS>:8000/channels/9.m3u8 I do get a clip that can be opened within VLC but no luck connecting via Plex.
Appreciate any tips to over come this final hurdle. Let me know if there are any additional details I can provide. Thanks

I believe your problem is with Plex, as it's known to have difficulty with live TV streams on occasion. If you can play the links in VLC, the xTeVe setup is probably fine. You should start by looking at Plex's log to find the specific error that causes the failure when you try to play a stream. You can also try to activate the buffer function in xTeVe with either ffmpeg or VLC, which might help Plex to figure out the stream.
I do have a question, however. You said you can see a clip in VLC, but is it the "Event Will Start Soon" screen created by EplusTV? Or are you actually seeing a live stream from ESPN+? I ask because on my setup (with Jellyfin) the streams will often start on the first screen, then stall, and opening the same channel again will then play the live feed. I think it's because the difference between the two video clips causes Jellyfin's ffmpeg to quit. Perhaps something similar is going on with Plex?

Thanks for the suggestions! I'll take a peak at my logs this afternoon and report back. Should have thought to look through the Plex side D'oh!
I did setup dizqueTV w/xTeVe as a test and confirmed those streams did play. Which falls inline with what you had mentioned checking my ffmpeg settings on xTeVe side as did configure those for dizqueTV.
As for the dled clip when I play through VLC I see the sports event you'd expect to see on the channel. Did happen to notice when I clicked let the error occur then try again w/the stream via Plex it did seem to attempt to load (further load?) when accessing but again would fail.
Will follow up. Thanks again

Apologies for the delayed follow-up. Looks like changing the buffer option in xTeVe to ffmpeg did the trick! Thanks for the suggestion. I am seeing the Event Will Start Soon text occasionally and having to back out and back in but sounds like that's the nature of the beast.
Thanks again


Code blocks not supported?

Are code blocks not supported?
❯ cat foo.md
# Foo

- Something

``` python
print foo
```
❯ tldr --render foo.md

  Something

❯

If I read the spec correctly, this should be supported

https://github.com/tldr-pages/tldr/blob/main/CLIENT-SPECIFICATION.md#page-structure
https://spec.commonmark.org/0.30/#example-24

(The space between the triple ticks and "python" is not significant, just followed the example from the CommonMark spec)

Hm, that section in the spec wasn't there when I started this project. Tealdeer is not a commonmark renderer, it uses a custom fast parser that handles the format specified by the TLDR pages.
The spec notes:

Clients MUST NOT break if the page format is changed within the CommonMark specification.

We fulfil that requirement, it doesn't break but there's simply no special rendering being done.
Right now I'd be against full commonmark compatibility, since the vast majority of pages will probably use the standard markup, and it would add considerably more complexity to this project.

(Note: You might be interested in bat as a console based markdown renderer.)

Ok, thanks for the reply.


EE-1047: Refactor bond/unbond outside core API.

Ref: https://casperlabs.atlassian.net/browse/EE-1047
This PR:

Refactors bond/unbond as internal impl detail of auction contract
Cleans up a bit of test contracts (i.e. mint-bonding renamed with auction-bonding).
Not calling bond/unbond directly from test contracts also required a bit of thoughful changes to the assertions
Assumes bond/unbond is now renamed into add_bid/withdraw_bid, so some related past regression tests are updated for new entrypoints.

bors r+


Orange eco-conception guidelines review

eco-conception guidelines review
General description
The Orange Eco-Conception team started to write some guideline. They are at a very first draft stage and are still being updated.
The objective is to perform an assessment of what exists today and identify:

what could be clarified
what could be updated
what is missing
what we should do in the current ODS guidelines to be more compatible with those guidelines

Detailed information
Contact: Maria-Jose PRESSO & Pierre RUST
Documentation: https://recommendations-preview.innov.intraorange/software-ecodesign-mobile-applications/
Implementation Tasks

[ ] review the existing guideline
[ ] provide feedback on the assessment
[ ] list further actions in new git issues

Relevant actions on our side:

Tooling/Mesure energy impact: Use Google tools to watch metrics that could badly impact the energy consumption.
Use of assets: Take a look to the Medium article then check our assets
Be careful with the use of animations and web views


Windows: Enable flaky test result processing

Commit Message: Windows: Enable flaky test result processing
Additional Description: Relies on https://github.com/envoyproxy/envoy/pull/14083
Risk Level: Loq
Testing: N/A
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Marked as a draft until https://github.com/envoyproxy/envoy/pull/14083 is merged, that PR is a standalone thing that could use some discussion separately

@envoyproxy/windows-dev

LGMT, thanks! it will help a lot with the test deflaking effort on windows

All green, should be good to review


CORE-358 - Endorsement Checker Updates

Closes CORE-358

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


[Project] Add new Job classes

This commit adds new Job classes - JobConfig, JobInstall, JobInstalled
and JobUninstall.
ONE-vscode-DCO-1.0-Signed-off-by: Yongseop Kim<EMAIL_ADDRESS>

PTAL @hyunsik-yoon and @jyoungyun

PTAL @Samsung/one-vscode_committers


Write a Kotlin program to print numbers from n to 1 without using a loop

Description
Write a Kotlin program to print numbers from n to 1 without using a loop
Input  : 5
Output : 5 4 3 2 1

How to contribute

Save the solution in program/PrintNumbersFromNTo1WithoutUsingALoop.kt file
Add PrintNumbersFromNTo1WithoutUsingALoop.kt file in print-numbers-from-n-to-1-without-using-a-loop folder

!assign


Replace deprecated Authorisation method

When updating Github as part of a release, we make use of the Github API. To authenticate we use a Github Token. However the way in which we use the token is deprecated. See here for more detail: https://developer.github.com/changes/2020-02-10-deprecating-auth-through-query-param/
Solution should be the same as the one in Gaffer in Gaffer at https://github.com/gchq/Gaffer/pull/2272

Can I work on this?

@mitchell-frost yeah of course

Merged into develop


ApplicationStartup Request.AsString Problem

[x] I have written a descriptive issue title
[x] I have verified that I am running the latest version of Nancy
[x] I have verified if the problem exist in both DEBUG and RELEASE mode
[x] I have searched open and closed issues to ensure it has not already been reported

Description
If a POST request is sent with a body and a parameter then when deserializing Request.Body to string via .AsString() the POST data is correct but the parameter appears in the data and with the value 'null'.
For example, a JSON object would then have an extra field and this would be the parameter but with a null value.
Steps to Reproduce
Define a simple module and add an implementation of IApplicationStartup where pipelines.BeforeRequest.AddItemToStartOfPipeline has an implementation that desearilizes the Request.Body as a string.
For example (hand typed - not tested):
    public class SomeModule : NancyModule
    {
        public RestServiceApi() : base("v1")
        {
            Post["SomeCommand"] = _ =>
            {
                var request = this.Bind<SomeCommandRequest>();
                return Response.AsJson(new SomeCommandResponse {Status = "Unavailable"});
            };
        }
    }

    public class BeforeAllRequests : IApplicationStartup
    {
        public void Initialize(IPipelines pipelines)
        {
            pipelines.BeforeRequest.AddItemToStartOfPipeline(ctx =>
            {
                if (ctx.Request.Body.Length <= 0)
                {
                    return ctx.Response;
                }
                var test = ctx.Request.Body.AsString(); <--- Partial parameter
                return ctx.Response;
            });
        }
    }

   public void WhenSomeCommandCalledWithAParameterTheResponseContainsUnavailable()
    {
        var response = _testContext.Browser.Post("v1/SomeCommand", with =>
        {
            with.HttpRequest();
            with.Query("SomeParameter", "123456789009876");
            with.JsonBody(new SomeCommandRequest
            {
                 SomeField = 111,
                 SomeOtherField = "TEST"
            });
        });

        Assert.That(response.Body.DeserializeJson<SomeCommandResponse>().Status, Is.EqualTo("Unavailable"));
    }

Result:
{"someField":"111","someOtherField":"TEST","SomeParameter":null}

Nancy version:
Nancy host

[x] Nancy.Hosting.Self
[x] Other:

Environment: Windows 10
.NET Framework version: 4.5

Please close this, it looks like it was something in my environment that was causing the issue.


docs: added caution section for nextjs and warning

Describe your changes
This PR adds a section to the web frameworks documentation about Next.js not being fully supported for versions 13 and above. It also adds a warning message when using Next.js.

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.

@all-contributors add @zZHorizonZz for docs


Evaluate approach how specific location can have their dedicated description/assets

Looking at indicator E9 of trilateral, each location has a dedicated markdown file.
In principle i think we could "just" provide a new description for the item and have a generic for the collection, but we need to evaluate how we can configure this so it is injected during generation of the items.
In the E9 case it would be for geodb data.

I would assume this falls under homogenization of the metadata on a per-collection level. I don't think we should have to implement this on location level?

Good point, as next steps we discussed cleaning the description markdown files to be data descriptive and not have "stories" within. So it is possible this would no longer be needed then.
Let's keep this issue open and collect this points to discuss them in our next "data migration" session.


Remove configuration file for appveyor

Appveyor is not required. It only checks coding style.

Can we remove the appveyor directory as well?

I removed it.

Thanks. LGTM.


license in case of modification

The license doesn't say anything about the case of modification, redistribution (except that the license has to be included), commercial use

Nevermind, seems like it doesn't have to be stated. Anyways is BSD 3

Also just read the thing again, actually is pretty clear about it.
No idea what my brain did there


chore: fix focusing dots

This PR is the result of running
find . -type f -name "*.lean" -exec sed -i -E 's/^( +)\. /\1· /' {} \;
find . -type f -name "*.lean" -exec sed -i -E 'N;s/^( +·)\n +(.*)$/\1 \2/;P;D' {} \;

which firstly replaces . focusing dots with · and secondly removes isolated instances of such dots, unifying them with the following line. A new rule is placed in the style linter to verify this.

bors merge


You continue to rock, Sam Soffes

I was raging all day (still am) on a very weird UICollectionView animation bug.
As a side project, I realized that our pull-to-refresh has been shoved under our navigationbar (which animates in and out - which is causing the animation bug, incidentally).
At first I was like awww wtf, who is messing with the insets what's going on!? (having been raging for so long, it was impossible to turn off).  SSPullToRefresh!?  Bah!
Dropped into the code, saw defaultContentInset, set it to the new values, and shit just works.
:clap: :clap: :clap:
Nice work.  Thanks for, at least, making the end of my day enjoyable.  Tomorrow I have to get back to fixing this animation thing, but that's another day!

:heart: thanks!


C++ APIs - "The size of the buffer provided for a bound variable is invalid"

Hello,
I'm interested in using the pure C++ APIs, as described here. However, these are very poorly documented, which is making it very hard to use them. Aside from my error, it'd be very helpful if you could also provide examples for using this API without having to use C++/CX as provided by MLGEN.exe.
As of now, I've got a very simple test which works for creating a context and runtime, loading a model, and binding values. All of these return S_OK when called. However, the HRESULT returned from EvaluateModel(context) has the following error description:

The size of the buffer provided for a bound variable is invalid.

However, given the network architecture I expect to give the same size data in as out, and both are sized accordingly. Both the input and output report being TENSORs, and I've set pShape to the corresponding to the input/output shape, except replacing [-1, ...] with [1, ...] for the output . pData is bound to an appropriately sized vector, which has been resized to the exact size of both input and output (4096 floats, 16384 bytes). I set the DataSize member of the binding to 16384, as well.
This is the output from EnumerateModelInput/...Output:
Input #1
        Name: input_1:0
        Desc:
        FeatureType: 1 // Tensor
        ElemType: 1 // Float
        Dims: 4
        Shape: [ 1, 1, 64, 64, ]
Output #1
        Name: conv2d_8/Relu:0
        Desc:
        FeatureType: 1 // Tensor
        ElemType: 1 // Float
        Dims: 4
        Shape: [ -1, 64, 64, 1, ]

I sadly cannot share much more publicly, but I'll happily supply debug info or help out via e-mail.

Some more digging, and I think it's related to this issue MicrosoftDocs/windows-uwp#463, which has been closed but not solved.
For what it's worth, I can move the error between the context->BindValue(...) and runtime->EvaluateModel(...) depending on whether I set batchsize to 1 or -1. If I leave it as negative (per the output description), it'll give me the same error, but at bind time. If I "fix" it and make it positive, it'll instead error out during evaluation.
It seems like there's some interaction with batch shapes and datasizes that doesn't go down well. I noticed that DataSize of a tensorbinding is an unsigned integer, but at surface it seems like something somewhere goes int bytesExpected = product(pShape);, which returns a negative value and things get wonky.

To further follow up, I manually edited my ONNX graph file, and forced the -1 to 1, and it gives me the same issue.

Hi Tom, I'm sorry for the delay in response. I'm assuming that you are using Preview API in RS4 and want to see if you are hitting the same issue on the insider preview build. Have you by any chance tried your case on the preview build with newer SDK? If not I can give you more information and would love to get more debug information

Closing this out as an issue againts the Preview API .    Please try this on the new insider builds, we just released our new samples and MNIST !

This issue is still happening, not with the MNIST sample but when I try any ONNX model. Also, the pre-processing tools in .NET are terrible! Came on guys, even in iOS or Android is trivial to do this!!

Hi @rribani can I get more information on your issue? Have you tried running your ONNX model with WinMLRunner?

Hi @ryanlai2, sure. I tried the WinMLRunner but it fails since my model input is float and has a fixed size. I'm using a model based on MobileNet (input 224x224) After a while trying, I had to change the model input type to TensorFloat and  I struggled for days to figure out how to resize the content from the stream.
I had to use a BitmapDecoder, a BitmapTransform, a PixelDataProvider, DetachPixelData(), a WriteableBitmap, a new SoftwareBitmap, a buffer and then create a TensorFloat from that. Ufff!
Here is my code btw:
                SoftwareBitmap softwareBitmap;
                using (IRandomAccessStream stream = await selectedStorageFile.OpenAsync(FileAccessMode.Read))
                {
                    // Create the decoder from the stream
                    BitmapDecoder decoder = await BitmapDecoder.CreateAsync(stream);

                    BitmapTransform transform = new BitmapTransform() { ScaledHeight = 224, ScaledWidth = 224 };
                    PixelDataProvider pixelData = await decoder.GetPixelDataAsync(
                        BitmapPixelFormat.Rgba8,
                        BitmapAlphaMode.Premultiplied,
                        transform,
                        ExifOrientationMode.RespectExifOrientation,
                        ColorManagementMode.DoNotColorManage);

                    byte[] pixels = pixelData.DetachPixelData();

                    //Initialize writable bitmap.
                    var wBitmap = new WriteableBitmap((int)decoder.PixelWidth, (int)decoder.PixelHeight);
                    var wb_stream = wBitmap.PixelBuffer.AsStream();
                    await wb_stream.WriteAsync(pixels, 0, pixels.Length);
                    softwareBitmap = SoftwareBitmap.CreateCopyFromBuffer(wBitmap.PixelBuffer, BitmapPixelFormat.Bgra8, 224, 224, BitmapAlphaMode.Premultiplied);

                    IEnumerable<long> shape = new long[] { 1, 224, 224, 3 };
                    var pixelsFloat = new float[pixels.Length / 4];
                    System.Buffer.BlockCopy(pixels, 0, pixelsFloat, 0, pixels.Length);
                    TensorFloat.CreateFromArray(shape, pixelsFloat);
                    modelInput.input_1_01 = TensorFloat.CreateFromArray(shape, pixelsFloat);

                }

This is working for inference in the model, but the results are not matching with the outputs from keras/tensorflow. I know there are some pre-processing steps missing, like normalization and ensure channel order. Probably I'll spend some more days to figure out how to do that, since the samples does not help.
Seriously, if you want people to use WindowsML, you must simplify this! It's not intuitive at all (even with documentation)!
Thanks,
Ribani

Hi Ribani,
Let me try to help:
as you may know WindowsML makes it pretty easy to automatically tensorize image input in NCHW format by simply binding an ImageFeatureValue instance which internally resizes to smallest axis then crops to accomodate prescribed aspect ratio and tensorizes to proper pixel order. In your case it seems your input shape is instead NHWC and demands a bit more work (i.e. custom tensorization).

First thing to understand is: in your reference framework, how are you pre-processing images in terms of size (crop, rescale).

In your example code above, you could use instead the VideoFrame API to encapsulate the retrieved SoftwareBitmap then use the VideoFrame.CopyToAsync() to copy its content to a 224x224 staging VideoFrame instance. Note that you can specify the BitmapBounds to use in that last method if you prefer to copy only portion of the original image (crop).

SoftwareBitmap softwareBitmap;
using (IRandomAccessStream stream = await selectedStorageFile.OpenAsync(FileAccessMode.Read))
{
  // Create the decoder from the stream
  BitmapDecoder decoder = await BitmapDecoder.CreateAsync(stream);

  // Get the SoftwareBitmap from the image file
  softwareBitmap = await decoder.GetSoftwareBitmapAsync();
}
// Encapsulate the image within a VideoFrame
VideoFrame inputImage = VideoFrame.CreateWithSoftwareBitmap(softwareBitmap);

// Create a staging frame of the desired size
VideoFrame stagingImage = new VideoFrame(BitmapPixelFormat.Bgra8, 224, 224, BitmapAlphaMode.Premultiplied);

// Copy input image into staging frame. If aspect ratio does not match, this method will pad and add black bar on both side
await inputImage.CopyToAsync(stagingImage);

With your SoftwareBitmap encapsulated in a VideoFrame in hand, second thing has to do with expected pixel values, pixel order and tensorization.

Currently you have an image in BGRA [0,255]. If you wish to normalize it for example to [0,1] float, you can do so at the same time you will create your tensor.
Given your shape, you may only need to drop the A pixel component (i.e. you want a tensor in BGR-BGR-BGR...) or additionaly reorder to RGB (make sure this matches your reference frmaework). Your example code does a block copy that includes the A component.
For creating an array from your image, you can refer to the logic in the TensorizationHelper sample and adjust it to your need.

Let me know if that helps :)

Hi @LPBourret,
The first thing I did was to try the SqueezeNet sample, that uses the VideoFrame. But I need to resize the image to 224x224 (not crop), so here my challenge begins. The only way that I found was using the BitmapTransform (and some more objects).
The TensorFloat class implements the ITensor and I can't do much with it. Any pre-processing or post-processing requires me to move the data to a byte array and work manually with it. To convert from NCHW to NHWC, from BRG to RGB, etc..
I'm trying with this PixelData.. Probably the CustomTensorization sample will help me some way.. I'll post my results here.
Thanks,
Ribani

Hi @rribani,
as a side note related to your pipeline, you can access directly the pixel buffer underneath a TensorFloat via the CreateReference() method. Just like you would do with a SoftwareBitmap, if you have already bound that TensorFloat instance, you can alter it's pixel buffer in-between inference and not rebind it (as long as the inference device is CPU, otherwise you need to rebind to let WinML appropriately transfer your data to a DirectX type such as a texture). This allow you to avoid a staging copy or "move" to a staging pixel buffer every time.
In terms of resizing your input image, you probably have good reason for doing that but I would advise others to avoid resizing if possible and instead train or use cropped images to accommodate the wide range of aspect ratio in image capture devices. In my experience, resizing arbitrary aspect ratio to 1:1 may disrupt the details of the original image to a point that affects accuracy on inference.

closing, feel free to reopen


Add login component dropdown to navbar component

I want to add a component to the navbar that lets more options in the future user
I think adding a select element to implement this, and adding a state for example,
To illustrate user identification
When the button is clicked the user's name will be displayed, with additional options when the user clicks the button

I was added this feature


python312Packages.lineax: 0.0.6 -> 0.0.7

Things done
Changelog: https://github.com/patrick-kidger/lineax/releases/tag/v0.0.7

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 350209

x86_64-linux

:white_check_mark: 8 packages built:

python311Packages.lineax
python311Packages.lineax.dist
python311Packages.optimistix
python311Packages.optimistix.dist
python312Packages.lineax
python312Packages.lineax.dist
python312Packages.optimistix
python312Packages.optimistix.dist

aarch64-linux

:white_check_mark: 8 packages built:

python311Packages.lineax
python311Packages.lineax.dist
python311Packages.optimistix
python311Packages.optimistix.dist
python312Packages.lineax
python312Packages.lineax.dist
python312Packages.optimistix
python312Packages.optimistix.dist

x86_64-darwin

:fast_forward: 8 packages marked as broken and skipped:

python311Packages.lineax
python311Packages.lineax.dist
python311Packages.optimistix
python311Packages.optimistix.dist
python312Packages.lineax
python312Packages.lineax.dist
python312Packages.optimistix
python312Packages.optimistix.dist

aarch64-darwin

:white_check_mark: 8 packages built:

python311Packages.lineax
python311Packages.lineax.dist
python311Packages.optimistix
python311Packages.optimistix.dist
python312Packages.lineax
python312Packages.lineax.dist
python312Packages.optimistix
python312Packages.optimistix.dist

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 350209

x86_64-linux

:white_check_mark: 8 packages built:

python311Packages.lineax
python311Packages.lineax.dist
python311Packages.optimistix
python311Packages.optimistix.dist
python312Packages.lineax
python312Packages.lineax.dist
python312Packages.optimistix
python312Packages.optimistix.dist

aarch64-linux

:white_check_mark: 8 packages built:

python311Packages.lineax
python311Packages.lineax.dist
python311Packages.optimistix
python311Packages.optimistix.dist
python312Packages.lineax
python312Packages.lineax.dist
python312Packages.optimistix
python312Packages.optimistix.dist

x86_64-darwin

:fast_forward: 8 packages marked as broken and skipped:

python311Packages.lineax
python311Packages.lineax.dist
python311Packages.optimistix
python311Packages.optimistix.dist
python312Packages.lineax
python312Packages.lineax.dist
python312Packages.optimistix
python312Packages.optimistix.dist

aarch64-darwin

:white_check_mark: 8 packages built:

python311Packages.lineax
python311Packages.lineax.dist
python311Packages.optimistix
python311Packages.optimistix.dist
python312Packages.lineax
python312Packages.lineax.dist
python312Packages.optimistix
python312Packages.optimistix.dist


Unable to replace null values in cell

Describe the bug
A clear and concise description of what the bug is.
I am unable to replace null values in cells.
To Reproduce
Steps to reproduce the behavior:

Go to Edit cells > transform
Different things I have tried:

replace(value, 'null', 'other_text')
replace(value, null, 'other_text')

Replace method works as expected for values other than null

Current Results

Expected behavior
I expected the null values to be replaced with 'other text'
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [Ubuntu 18.04]
Browser Version: [Version 76.0.3809.87 (Official Build) (64-bit)]

OpenRefine (please complete the following information):

Version [OpenRefine 3.2]

The question was answered on StackOverflow:
https://stackoverflow.com/questions/57882921/replace-null-values-in-cell

The stackoverflow shows that replace does not work on null. Is that not a bug? If not, then can you update the documentation to reflect that so others will not incur the same issue?

@titoesteves that is not a bug indeed! The replace function is designed to replace parts of strings by other strings, and null is not a string. Feel free to edit the wiki to make that clearer!

I tend to side with our user base here, in that we don't expose an easy button for null replacement like other data tools have.
This comes up often enough that I "think" most of our users would like to see a new function for this...perhaps a replaceNull
value.replaceNull("this value is being replaced and is no longer a null")

Sure! But then that has nothing to do with the existing "replace" function. We can open another ticket to request a new GREL function for that if you want.

DONE - see #2157

if(value==null,"new",value).


RawCollection not usable outside fiber

The patches to the underlying collection implementation to allow preserving the session when in a fiber mean that all patched collection operations fail when not in a fiber. Not being in a fiber is one of the main reasons to use rawCollection(). This should be fixable by using getOrNullIfOutsideFiber() instead of get() in the patch implementations.

Got it, I'll check what can be done here. Haven't been testing Meteor without Fibers yet, but so far I understood there's always one (root) fiber while executing Meteor method or publication.
We still want transactions to work in this context, but I remember having issues with async_hooks (in #3) which might complicate things.

This particular code isn't running in a method or publication, and I'm not trying to do a transaction. I'm actually using bulk updates, but I suspect a version of the mongo driver changed bulk update to call regular update if there's only one update in the list, and that call was failing because it was trying to get the session outside a fiber.


[bug] Can't display full content - CSS Style

Search before asking

[X] I had searched in the issues and found no similar issues.

Environment
https://laf.dev
What happened
CSS Style

Try to Fix It
Remove overflow: hidden

Have the same problem  in the Logs Detail

scope

laf.dev
self-host web

How to reproduce
Return a large object in the code.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

The latest version online has fix this bug.
Just refresh laf.dev or laf.run to check it.

Do you know how about the self-host version?
Are the images updated?

the image has been updated just now (1 hour ago).
you can check it by your self.

thanks😘

Try
I try to upgrade the image to their image version. And force refresh the page.
image: docker.io/lafyun/laf-server@sha256:48fe5b854b68fb141592e34665f8479da67ae3b91d2448436c4592c09ee3ba4d

image: docker.io/lafyun/laf-web@sha256:02ed6d070a18e1fa4606ca444dcf8c4f9ffeb57cce6e10b1371d0220439d0858

Problem
But it still has the same problems.

It's work. Thanks.
If possible add git hash to application env?
Std out the git hash when the application bootstrap.
If I use the latest image version, I can't know which git commit version was used.


IUrlHelper.Absolute method

https://github.com/aspnet/Mvc/issues/7807 implementation

Thanks for your effort, @TKharaishvili.
As part of the repo reduction effort (https://github.com/aspnet/Announcements/issues/330) we're closing all the PRs in this repo. Please reopen this PR against the AspNetCore repo so we can consider it.


Support for new ldap_group_settings field in platform_saml_settings

Is your feature request related to a problem? Please describe.
In the artifactory 7.98.7 self-hosted release new field was added to manage the groups of saml users via ldap. I would like to configure the field using terraform.
Describe the solution you'd like
Be able to configure the setting with this provider
Describe alternatives you've considered
Currently manually set the settings via API
Additional context
See also the saml documentation here for this setting: https://jfrog.com/help/r/jfrog-platform-administration-documentation/enabling-synchronization-of-ldap-groups-for-saml-sso

@gira0 Thanks for the suggestion! I've added this to our plan.


feat: upgrade jodaTime

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.45%. Comparing base (5e171af) to head (182fb6d).
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #49   +/-   ##
=======================================
  Coverage   92.45%   92.45%
=======================================
  Files         124      124
  Lines        6731     6731
=======================================
  Hits         6223     6223
  Misses        404      404
  Partials      104      104

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Scala 2.13.0-M3

Needed to bump scala.js for this

Fixes #22

Who has rights to merge this?

I do, but it's @non's project, so we should chase him first ...

Thanks for this! I'll publish a release tonight after work.

Just released 0.6.4 to Sonatype. Should be available from Maven Central soon.


Handle Postgres advisory locks (and other functions?)

Hi, what's the correct way to obtain a Postgres advisory lock using Bun ?
Right now we use the following helper:
func obtainAdvisoryLock(ctx context.Context, db bun.IDB) error {
       rows, err := db.QueryContext(ctx, "SELECT pg_advisory_xact_lock(?, ?)", 12, 34)
       if err != nil {
               return err
       }

       for rows.Next() {
               if rows.Err() != nil {
                       return rows.Err()
               }
       }

       return rows.Err()
}

We expect something like db.NewSelect().ColumnExpr("SELECT pg_advisory_xact_lock(?, ?)", 12, 34).Scan(ctx) to work. Sadly it is not valid because of missing model: error bun: Model(nil) is returned.
We are on Bun v1.0.14 if that helps.
Thanks

I would use _, err := db.ExecContext(ctx, "SELECT pg_advisory_xact_lock(?, ?)", 12, 34), but you can do that with Bun too:
_, err := db.NewSelect().ColumnExpr("pg_advisory_xact_lock(?, ?)", 12, 34).Exec(ctx)

No need to use Scan or QueryContext, because pg_advisory_xact_lock does not return anything. Also, pg_advisory_xact_lock must be run in a transaction.

Thanks @vmihailenco , we'll try that.


PayU integration

What I'm trying to achieve
Implementing new gateway (PayU)
Describe a proposed solution
I'm planning to integrate PayU payments inside saleor, but it's impossible with keeping current gateway flow. (PayU works differently, we can't charge inside gateway capture method)
Problem is, after receiving update from PayU to our server (some django view) I need to run custom function from payment/gateway.py (to create transaction / payment status) , and question is- modyfing saleor/payment/gateway.py (creating custom function) would be acceptable?
Because for now, I can't find any way to make it work without changes in current gateway flow.
Or maybe you have any suggestions?
Other solutions I've tried and won't work
inside plugin in payment process I want to return action_required true with URL to payment page from PayU, next there will be view for receiving payment status update. Trick is, what to do after receiving payment successfull etc. inside view. Should we run custom method from gateway to create transaction, or should we do it inside view (changing flow)
Screenshots or mockups

https://gitter.im/mirumee/saleor?at=5db467ec14d55a37857e01fd
Gitter discussion

https://github.com/mirumee/saleor/pull/4939
Draft PR for architecture discussion / possible solutions. (not tested/not completed pr)

If you are looking for a solution in Saleor 3.x, please take a look at this PR https://github.com/saleor/saleor/pull/9275
You can capture amounts from any payment gateway on any order or checkout using paymentCreate mutation here:
https://new-checkout-flow-poc.api.saleor.rocks/graphql/
mutation{
  paymentCreate(id:"T3JkZXI6NTQ=", payment:{
    status: "Captured",
    type: "Paypal",
    availableActions: [CAPTURE],
    amountCaptured:{
      currency:"USD",
      amount:"22"
    }

  }){
    payment{
      id
      status
      type
      reference
    }
  }
}

We are planning to release these APIs soon


Face Anti-Spoofing

Hello Sefik, thanks for all the hard work and time that you obviously invested on the project.
Is there any way for DeepFace to verify that the face being detected is a real face? (not a printed face or a face in a screen)
This way it could be used in access control and not be fooled by just a printed photo.
If not, what method or library would you recommend?
Thank you.

this is out of the scope
unfortunately, i do not have an experience on that subject.

Thanks Sefik, Ill try to find or create a model to detect a printed faced, the problem is I don't know what attribute would differentiate an image of a face and an image of a printed face. Depth sensor comes to mid but that would require extra hardware.


Added methods to load/control tours [issue #4]

I don't know if this is intentional, but play_tour() resumes playback immediately while stop_tour() slows the tour down to a stop over the course of a few seconds.

Looks good! I'm not sure if that behavior is intentional - @philrosenfield, do you know?
Could you add some docstrings to the new methods? See:
https://github.com/WorldWideTelescope/pywwt/blob/master/pywwt/imagery.py#L8
for an example. We should follow the following guidelines for writing docstrings which means that parameters should be written as e.g.
Parameters
-----------
url : str
    The URL of the tour etc.

I don't know if play_tour/stop_tour timing is intentional, but I'd assume it is. Is there a problem with it?

To Tom -- Do you want docstrings on every method in this pull request, or just for the tour-related ones?
To Phil -- I was assuming they would both act the same way, but upon further thought, that's more of an aesthetic expectation on my part than an actual problem.

@ojustino - we need docstrings on all methods but you don't have to add them all in this PR if you'd rather not - up to you! (I just meant the tour ones for now)

I'll add docstrings for other methods in a separate pull request.

Looks good, thanks!


v2.15 not building for F-droid

It seems that v2.15 isn't building on fdroid
https://f-droid.org/wiki/page/com.tobykurien.webapps#v2.15

I don't understand, the linked build log says it succeeded?

If you look at the history for the build log there's an error there
https://f-droid.org/wiki/index.php?title=com.tobykurien.webapps/lastbuild_22&oldid=212881

Thanks @kabo the error log doesn't make sense to me unfortunately. I changed the code structure of the app, and it seems maybe F-droid stores some configuration information about the app that needs to be updated? The exact same gradle build commands as before still work, which is why I don't see anything I could change to make the build work.

Looks like the build worked. I got v2.15 via F-droid


Angular popup -- css only

Hello,
I am loving semantic UI. :)
However, I am facing few problems :
I am using angular for a ng-repeat, and a semantic UI popup. The jQuery for the popup is not triggered since the angular comes after the jquery. Is there any full-css version like for the dropdown ?
Thanks by advance,
Rémi

You might want to check out GitHub Issues on Semantic-UI-Angular they are better equipped to help you.


[EventHubs] check for any non-None values in amqp header/properties

In [this line], we use any() to check if the AmqpAnnotatedMessage.header has any values set. However, 0/False/"" are falsy values and therefore not detected as values by any().
Adding an _any() method to AmqpMessageHeader/AmqpMessageProperties, which returns True if any of the values are not None.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-eventhub

/azp run python - eventhub - tests


enhancement unique model by modify a value type from LARGEINT to other type

unique model by modify a value type from LARGEINT to other type
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Support for headers/block references autocompletion on wikilinks

Please Select if your Request is Either something new or an Enhancement

[x] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applys to. (Multiple selections are Possible)

[ ] Onboard. Initial vault setup and import/export pods
[x] Create. Note creation, lookup, snippets and templates
[ ] Retrieve. Backlinks, references, graph view
[x] Structure. Refactoring, multi-vault and schemas
[ ] Publish. Sharing your repo with the world
[ ] Misc (Choose this if your not sure)

Is your feature request related to a problem? Please describe
At the moment autocompletion is supported for notes when constructing a wikilink.
However, autocompletion of (header) block levels is not.
This causes a jarring interruption during a workflow where one wants to create a reference to a specific header.
Describe the solution you'd like
I would like autocompletion of a wikilink, once a note is selected and # is inserted, to switch to autocompletion of the headers on that page, resulting in a valid block reference once selected.
Describe alternatives you've considered
I have tried following the recommended workflow with the hotkey that creates a block reference to the currently selected block. However, this requires me to switch to the relevant page, find the relevant header manually, and use the reference hotkey and switch back - which takes me out of the flow of writing a note.

We should consolidate this + #175  + #262 and do a much-needed "auto-complete improvements" project. Might be a little bit larger RFC but makes sense to deliver them all together.
For now, marking as help-needed until we resource the project.

This is done.


Saving trained models

Hi,
I'm trying to save models after training:
from imxgboost.imbalance_xgb import imbalance_xgboost as imb_xgb
xgbooster_focal = imb_xgb(special_objective='focal', focal_gamma=1.0)
xgbooster_focal.fit(X_train.values, y_train["label"].values)
joblib.dump(xgbooster_focal, '../models/xgboost_focal.pkl')

but it doesn't work, apparently a pointer issue:
ValueError                                Traceback (most recent call last)
<ipython-input-30-d4caae048667> in <module>
      1 # save model (or try to)
----> 2 joblib.dump(xgbooster_focal, '../models/xgboost_focal.pkl')

...

ValueError: ctypes objects containing pointers cannot be pickled

See the full error log on this gist.
FYI, the same save method works well on regular xgboost models running on the same data.
Any idea how to solve that?

I think you can do trained_booster = xgbooster_focal .booster to retrieve the vanilla xgboost booster and save it with the same method on regular xgboost models.


import PlotUtils: palette

Please add
import PlotUtils: palette
in your code .
https://discourse.julialang.org/t/sankey-diagram-in-julia/92884/4
Why there is a comma after "Heat demand" in name collection ??
Please also make Documentation more understandable by including what is meaning of  src , dst  and  weights etc.
The ECharts.jl package gives more interactive sankey diagram so also make changes to make it more interactive like ECharts.jl Sankey plots.

Dear @raman-maker
Apologies for the huge details.
The import PlotUtils is not necessary in SankeyPlots as it is imported through Plots.jl
Regarding the comma after "Heat demand", in lists spanning through multiple lines, it is common practice to add the comma at the end of each element (and line).
I'll try to integreate more documentation in #29. Feel free to revise and comment


Feat: chat completion token info flag option

Description
A flag option that gives the user a count of tokens sent and received.
I think that it is an important feature that guides the user to stay within the token budget when making a chat completions request!
Implementation
To do this, we would need to add another option flag which could be -t and --token-usage. When a user includes this flag to their command, it should display in clear detail how many tokens were used in the generation of the completion, and how many tokens were used in the prompt.

@cleobnvntra we can use -u and --token-usage , because I have -t for temperature already.

@fadingNA I tried to implement it using the -u flag, but found out that you are also using the -u flag for --base-url. Is there any other flag you suggest to use? I'm thinking of using -tu instead. Let me know what you think!

@cleobnvntra  you can use -k instead, 2 syllable might confilct with Linux command

@cleobnvntra  I tihnk just --token-usage that's fine

@fadingNA gotcha


compose resources locale language

is there a way to set compose resources locale language since i have different flavors on android/ios and only want to support one language per build (and not the one user has selected in android settings) but i still want to keep all the string resources inside shared/common multiplatform lib
lets say you have multiple locales but you want to get specific one getString(R.id.helloTet, "en"),

The flavors are the android gradle plugin feature. It does not work for other targets.
If you want to provide a local compose provider with a specific locale then it is not possible yet: https://github.com/JetBrains/compose-multiplatform/issues/4197

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Guide

This is another attempt at building a guide. The previous Lookbook is great but doesn't allow any descriptive text and needs a more complex hosting setup.
This version is more like the form builder's, it builds a static Nanoc site and pulls in the actual code from the library so can't fall out of date.
Tasks

[x] get a Nanoc site bootstrapped and make it able to render the components
[x] add the GOV.UK Frontend stuff and make it look like the form builder guide
[ ] write the actual guide pages
[x] create an image for the masthead. @paulrobertlloyd - would you be able to lend a hand with this bit? Nice matching ones on the form builder and here would be amazing. You're more than welcome to choose a colour scheme/accent too if the purple's a bit much

@peteryates Have I got write permissions for this repository? Want to push a commit to this branch. Cheers!

🙏

Oops, thought you had them! Should see an invite now.

The examples pages are now present in draft form, they need some context and extra information and guidance but everything's looking decent.
The list on the homepage is a bit long, what are your thoughts on spreading it over two columns @paulrobertlloyd?
Something like this (just by adding columns: 2 to the ul)

Before
After

The examples pages are now present in draft form, they need some context and extra information and guidance but everything's looking decent.
The list on the homepage is a bit long, what are your thoughts on spreading it over two columns @paulrobertlloyd?
Something like this (just by adding columns: 2 to the ul)

Before
After

Also there's a couple of things that might need a reorganise; in this library the skip link is technically a helper method rather than a component, but it's listed under components as that's what it is in the source Nunjucks. Perhaps moving 'Link Helpers' from the introduction to Comonents and renaming the list "Components and helpers"?

Also there's a couple of things that might need a reorganise; in this library the skip link is technically a helper method rather than a component, but it's listed under components as that's what it is in the source Nunjucks. Perhaps moving 'Link Helpers' from the introduction to Comonents and renaming the list "Components and helpers"?

Split components into 2 columns as suggested. Have split link and button helpers out into separate pages, and placed under a ‘Helpers’ heading. Do we also need to include govuk_mail_to helper? Are there other helpers?

Split components into 2 columns as suggested. Have split link and button helpers out into separate pages, and placed under a ‘Helpers’ heading. Do we also need to include govuk_mail_to helper? Are there other helpers?

What makes the skip link functionally different from a component, apart from where it’s in the codebase?

What makes the skip link functionally different from a component, apart from where it’s in the codebase?

Nothing really other than you can't create it via render GovukComponent::SkipLinkComponent.new, only via govuk_skip_libk. Now they're separate I think moving it to the helper section makes more sense really.

Nothing really other than you can't create it via render GovukComponent::SkipLinkComponent.new, only via govuk_skip_libk. Now they're separate I think moving it to the helper section makes more sense really.

Now they're separate I think moving it to the helper section makes more sense really.

I agree.

Now they're separate I think moving it to the helper section makes more sense really.

I agree.

So there's now a first draft for each component.
The reason the header example page isn't working properly is that in govuk-frontend/govuk/all.js there's this:
// Find first header module to enhance.
var $toggleButton = scope.querySelector('[data-module="govuk-header"]');
new Header($toggleButton).init();

If I change it to this (manually, then copy the file to the right place) it works:
?
// Find all header modules and enhance them
var $toggleButtons = scope.querySelectorAll('[data-module="govuk-header"]');
nodeListForEach($toggleButtons, function ($toggleButton) {
  new Header($toggleButton).init();
});

Not really sure on the best way of doing this cleanly (where cleanly means not doing some sed gymnastics in the postinstall). Any ideas @paulrobertlloyd?

I've been thinking about this. Ultimately, I think component examples should be in an iframe – note how the success banner example always receives focus when go to that page. Heading levels are wrong too. Using an iFrame, as they do on the Design System docs, will ensure component examples remain encapsulated.
In the short term, what happens if you set html_attributes.data.module to nil or null (or similar)?

Possibly worth looking at:

https://github.com/alphagov/govuk-design-system/blob/main/views/partials/_example.njk
https://github.com/alphagov/govuk-design-system/blob/main/src/stylesheets/components/_example.scss
https://github.com/alphagov/govuk-design-system/blob/main/src/javascripts/components/example.js

Yeah - I had a quick look at how the <iframe> resizing worked in the official guide and it put me off! As we've only got a couple here and they don't look too bad, I'll leave it for the moment and revisit.
Overall I think it looks great, the content will evolve over time but I think it's fine for the first version.

Yeah, this is fine, and a perfect solution to the immediate issues.
Meanwhile… I’m seeing issues running Nanoc on Ruby 3.1.0:
Calling `DidYouMean::SPELL_CHECKERS.merge!(error_name => spell_checker)' has been deprecated. Please call `DidYouMean.correct_error(error_name, spell_checker)' instead.
Calling `DidYouMean::SPELL_CHECKERS.merge!(error_name => spell_checker)' has been deprecated. Please call `DidYouMean.correct_error(error_name, spell_checker)' instead.
Listening for lib/ changes…
View the site at http://<IP_ADDRESS>:3000/
[2022-01-03 10:50:17] INFO  WEBrick 1.7.0
[2022-01-03 10:50:17] INFO  ruby 3.1.0 (2021-12-25) [x86_64-darwin21]
[2022-01-03 10:50:17] INFO  WEBrick::HTTPServer#start: pid=66124 port=3000
/Users/paulrobertlloyd/.rvm/gems/ruby-3.1.0/gems/railties-7.0.0/lib/rails/railtie.rb:246:in `initialize': Rails::Engine is abstract, you cannot instantiate it directly. (RuntimeError)

Is the components gem supported by Ruby 3.1.0, too?

Ruby 3.1 isn't yet supported by Rails, we need to wait for Rails 7.0.1. The original plan for this release was to include it but it might need to follow shortly after.


Message to multiple channels

🧰 Ticket
closes #49
🚀 Overview:
Added the button for draft message sending
Added the multiselect dropdown button for  multi channel messages share
Added the multiselect dropdown button for multi channel  draft messages share
Updated message list component to show message feedbacks and added the form for writing feedbacks
🤔 Reason:
Used material ui dropdown but I think its better to remove it and use some alternative form bootstrap (witch one already used in project)
📝 Developer Notes:
I want to test it and done few style fixes, it's working so can be deployed if necessary.

Another item of feedback @lilitkarapetyan - when a message has been "Sent as Draft" - the contents of the message have to stay in the form.  Then when the "reviewer" asks for changes in their feedback, the player can make the changes, and either send another draft, or send the message for real.

Hmm, in some circumstances the "Send Draft" (with multiple recipients) is shown at the bottom-left with a drop-down, on other occasions it's shown at the bottom right, near the existing send message.

Thanks for the update @lilitkarapetyan
The form should only be kept on "Send draft".  For "Send" it should be cleared (and collapsed)

We're going to have to drop this work. It was defined/implemented at haste, trying to make it into the current wargame. But, It wasn't available in sufficient time to be incorporated in the game.  When we get further resources we should re-consider it, with UX, and re-implement. The code in this branch will remain available.


fix: do not use stdin which will stop when running in background

Fixes #65.
This bug was introduced by #63, which is too rush to handle STDIN inproperly. Although we can try to handle STGTTIN in Unix system but I cannot find an easy way to deal other platform like Windows. We'll revert the Ctrl-D key binding feature and back to the good old Ctrl-C.

When a process in a background job tries to read from its controlling terminal, the process group is usually sent a SIGTTIN signal. This normally causes all of the processes in that group to stop (unless they handle the signal and don’t stop themselves).

Ref: https://www.gnu.org/software/libc/manual/html_node/Access-to-the-Terminal.html

It would be great if someone can provide a viable cross-platform solution.


help to deploy on vercel

Hi @suhaildawood , thanks for this example of sveltekit + websocket.
Works perfectly in development.
But I can't deploy it to vercel, can you give some clues on how we could make this work?
I have this error on Vercel build :
Error: No Output Directory named "public" found after the Build completed. You can configure the Output Directory in your Project Settings.
Thanks

@robotpapier vercel and sveltekit has a comprehensive guide about deployment.
https://vercel.com/docs/frameworks/sveltekit
https://kit.svelte.dev/docs/adapter-vercel
Using vercel adapter will build your app in public folder, it seems your svelte.config.js is not set properly.

Glad it was helpful @robotpapier ! This example uses @sveltejs/adapter-node so you'll need a server capable of running the built Node.js script. From what I can tell Vercel does not support WebSocket servers as part of their serverless function offering so I doubt you'll be able to use the Vercel adapter. One option you could look into is Cloudflare - it looks like they do offer support for WebSocket servers within Cloudflare Workers.
If your use case doesn't involve a lot of traffic I'd suggest hosting your site on a small server like the Amazon T2.micro EC2 instance.

@suhaildawood yeah indeed I saw that afterwards, thanks for the tips ! :)

Glad it was helpful @robotpapier ! This example uses @sveltejs/adapter-node so you'll need a server capable of running the built Node.js script. From what I can tell Vercel does not support WebSocket servers as part of their serverless function offering so I doubt you'll be able to use the Vercel adapter.

Could you elaborate on this? I was under the impression that this example app was a server and could be hosted directly on Vercel. I'm not sure I understand how a SvelteKit app can even be a serverless function. I'm not sure if there's some sort of requirement I'm missing..

@aeviou
Yeah, as @suhaildawood said, and as in the Vercel docs, Vercel doesn't allow websockets connections, so there is no way It can work even if you can deploy it.
Luckily for my project, my apps will run locally in production, so I didn't try it on Cloudfare yet with which it should theorically work.
Hope it's clearer!

Ah yes. My misunderstanding was that I didn't understand that Vercel deployed the SvelteKit app as a single serverless function. Thanks for the clarification!


Highfive URL needs to be changed

The highfive url you have was from an older version of the docs that snuck in.
https://github.com/dominikgehl/geekli.st-php/blob/master/class.geeklist.php#L160
It should be /highfive instead of /h5

fixed in git revision 53a7240d467edcf59eb0937a4b8b08e5f517a296


RLE to segmentation

Hi there, nice job guys with datasets....
I have one question also regarding closed #23
Im trying to create new json with segmentation field.
It is little bit hard to find out what this segmentation filed means...
I can get RLE object (but it is probably wrong):
boundsTuples = [(x,y), (x1,y1), ...]

img = Image.new('L', (width, height), 0)
ImageDraw.Draw(img).polygon(boundsTuple, outline=1, fill=1)
mask = np.array(img)
fortran = np.asfortranarray(mask)
encoded = encode(fortran)
# encoded now has {counts, size} fileds

how can I convert rle object into segmentation array?

You might find .annToMask useful for your use case https://github.com/pdollar/coco/blob/master/PythonAPI/pycocotools/coco.py#L421-L428

@tylin I have further question, segemntation available datasets are actually polygons ([x, y, x1, y2 .... xn, yn]), is that right?

Yes, except for crowd region (iscrowd=1), the annotations were encoded by RLE.

@tylin got it! thanks!

@tylin iscrowd=1 uses uncompressed RLE. How would you convert a mask to uncompressed RLE?


WIP: Hef/other name change detection

Fixes #6617
Pull Request Motivation
This PR ensures that changes in Certificate otherName value triggers reissuance , as well as ensures that the Readiness status is correctly computed by matching the Issued Secret with the Certificate spec.
Kind
<bug>

Release Note

BUGFIX: Ensure otherName SAN changes in Certificates trigger reissuance.

@SpectralHiss
I think we probably don't want to add the OtherName check to the "fuzzy" match logic (I removed the check in https://github.com/cert-manager/cert-manager/pull/6620/commits/3dad3f320babb131aed24dbf324b75552626c8cb)
This "fuzzy" match logic is only used in case there is no CertificateRequest that exists, in that case we are comparing what we got back from the CA with what we requested from the CA (hence the "fuzzy" logic).

/ok-to-test

/retest-required

LGTM @SpectralHiss Thanks for fixing this missing functionality.
/approve
/lgtm

/kind bug


pass excluded_artifacts down to maven_install

We need to exclude some artifacts from the repo built by rules_jvm_external.
To do this, we just need the option to pass excluded_artifacts down to maven_install.

Looking at the license stuff ...

I'm actually not sure what the license gate wants. It looks like something in this repo, not mine?

that is something wrong, gonna remove it now.

licensebot is gone now. Thanks @smparkes for the PR improvement!

Thanks!


fine tuning the pretrained model

How can i fine tune the pretrained model with my own audio files for speaker verification?
There is no particular tutorial for it and i'm lost

Documents will be prepared by next week

Documents will be prepared by next week

I have problems in training and finetuning. Hope the ducuments be released ASAP.

Documents will be prepared by next week

Thank you very much for your work and hopefully the documents can be released soon! I found that I can't load the pretrained models in python codes.

I have problems in training and finetuning. Hope the ducuments be released ASAP.

Here it comes. Please refer to https://wenet.org.cn/wespeaker


Unable to locate attached view in the native tree

There appears to be related to this line of code, which calls detach sometimes on null nodeId's: https://github.com/kmagiera/react-native-reanimated/blob/f607cebcd4b5f84a0c2b0446288de164dba512a4/src/createAnimatedComponent.js#L153
An easy solution would be to also check for a nodeId, like this:
oldPropsAnimated && oldPropsAnimated.__nodeId && oldPropsAnimated.__detach();

Actually, I'm having issues with this line as well, I suggest the same check
https://github.com/kmagiera/react-native-reanimated/blob/f607cebcd4b5f84a0c2b0446288de164dba512a4/src/createAnimatedComponent.js#L31

Would you mind giving us context of this issue? How to repro it?

Maybe it's fixed. If not, please let me know, @JulianKingman @vincentriemer @trungtin


kuma,kuma-cp,kuma-dp,kuma-experimental,kumactl: 1.8.1 → 2.3.1

Description of changes
https://github.com/kumahq/kuma/blob/master/CHANGELOG.md
https://github.com/kumahq/kuma/compare/1.8.1...2.3.1
also fixes https://github.com/NixOS/nixpkgs/pull/245159

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

@Janik-Haag  this is the last one :pray:  after that we can close https://github.com/NixOS/nixpkgs/pull/245159

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

5 packages built:

kuma
kuma-cp
kuma-dp
kuma-experimental
kumactl

Seems like you have to disable the tests if the build environment is darwin since it throws some errors about dns. After that it should be good to merge.

@Janik-Haag build failure on darwin is happening in dependency
will try to fix it here https://github.com/NixOS/nixpkgs/pull/250749


WIP make a 2.12.2 CDT

This PR extracts the 2.12.2 sysroot from our latest build of the ctng compilers.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

license_file entry is missing, but is required.

For recipe:

License exception is not an SPDX exception.

I'd rather we patch the header needed for aligned_alloc.

But we don't know if that is the only change needed to match the ABI and API, right?
This way we do not have to guess.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License exception is not an SPDX exception.

We are doing the same thing for 2.17 on aarch64 and ppc64le, so I don't see a reason not to.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

We are doing the same thing for 2.17 on aarch64 and ppc64le, so I don't see a reason not to.

Where?

ct-ng builds 2.17 on aarch64 and ppc64le and we are going to replace that with the 2.17 glibc from CDT right?

Yes but where are the patches we are using with 2.17 now?

Those are in ct-ng.
gibc 2.12 x86_64 patches = 2.17 aarch64 + aligned_alloc patch.

Ahhhh. They are in the anaconda fork.
https://github.com/ContinuumIO/crosstool-ng/tree/master/packages/glibc/2.12.2

I had assumed (wrongly and without much basis) that aligned alloc was in 2.17.
hmmmmmm

Well, given that it worked with the 2.17 CDT, I guess that is true actually.

OK. I am still catching up here.
Option 1
We apply the aligned alloc patch to 2.12 centos headers. We don't actually know if this will generate a sysroot that is the same as the 2.12.2 one we use now, but we suspect it might. We'd have to inspect the source code to see if it has the rest of the patches that form 2.12.2.
Option 2
We extract the 2.12.2 CDT from the gcc_impl_linux-64 package we have now. (This PR.) This procedure is inelegant and hard to reproduce. It does, however, guarantee exact compatibility.
Did I get this right?

Yes

There is also the issue of the 2.17 patches from ctng (https://github.com/ContinuumIO/crosstool-ng/tree/master/packages/glibc/2.17) we have been using relative to 2.17 from centos 7.
Uggggh.

Most of those patches were ported from RHEL7/CentOS7, so that's fine.

My taxonomy Of Linux dists is really bad. The fedora patches too?
I guess my feeling is that we should be doing the minimal thing here. So that to me would mean only using the  centos 2.17 CDT for Linux-64 and hacking out the sysroot from ctng for the rest of the platforms.
However my experience in these issues is non-existent. Thus generally I’ll defer to you here.
One test we might consider is using some of the tools from the ABI lab to figure out what we might be missing in terms of headers and symbols etc. this doesn’t cover bugs however.

Actually no. Let’s patch the header and move on. The code built against these cdts is used on tons of other machines without issues.
Where do we want the patches? Probably in a 2.12 branch here?

Where do we want the patches? Probably in a 2.12 branch here?

Sure. Or add a new variant in recipe/conda_build_config.yaml whichever is easier for you.

@conda-forge-admin rerender

@conda-forge-admin rerender

@conda-forge-admin rerender


pullRequestsForR returns wrong Id

When using the simplePullRequestId returned from GitHub.pullRequestsForR and passing it to mergePullRequestR, it always results in an error. The reason is that it mergePullRequestR should get the pull request number, and not the global id.
The problem can be reproduced with the following code.
  let request =
    GitHub.pullRequestsForR user repo opts (GitHub.FetchAtLeast 1)

  Right (SimplePullRequest{simplePullRequestId} <- do
    res <- GitHub.executeRequest auth request
    return $ Vector.head res

  let merge = GitHub.mergePullRequestR user repo simplePullRequestId Nothing
  res <- GitHub.executeRequest auth request

When using Id simplePullRequestNumber, things work correctly

Thanks!


Add tabs shortcode pls

Describe the solution you'd like
https://kubernetes.io/docs/contribute/style/hugo-shortcodes/#tabs
https://hugo-dynamic-tabs.netlify.app/

Users who wish to add this feature are welcome to leave your thoughts at any time ;-)
If this is a widely needed feature, we may consider adding it.


Support file selection like setuptools and hatch

If I understand correctly, meson-python includes all the files in the sdist and it is not possible to exclude some directories or files.
In contrast, setuptools and Hatch support controlling which files are included in the sdist. For Hatch, there are few related options in the pyproject.toml file: https://hatch.pypa.io/latest/config/build/#file-selection
Would it be possible to at least be able to exclude some files with meson-python ?

meson-python uses meson dist to create sdist. meson dist obeys the export-ignore git attribute. Just mark the files you want to exclude from the sdist with this attribute in a .gitattributes file.


Updated CLI

[ ] Subcommands for discovery, installation, update and removal
[ ] CREATE MORE TASKS

Superceded by #1064


Move "dial" functionality into a separate package/project

Feature Description
This issue is open to gather feedback on the idea of splitting the "dial" functionality into a separate package/project.
Currently, this library has 3 distinct ways to interact:

Programmatically, as an interface for a Go app to dial connections
Programmatically, to start, customize, and mount a proxy server
As an executable, to start, customize, and mount a proxy server from the command line

The first option (dialing connections from an application) often has different ideal behavior compared to the second two, which has lead to some challenging behavior (such as when to log and when to return errors). It's also meant that changes to one interface means a major version bump for the others, even if the the other interfaces are left unchanged. Additionally, it's also meant that there are some dependencies pulled in for the proxy that are relevant to the dialer, and visa versa.
As such, we think it would be a good idea to split the dial functionality into its own project. This is obviously a breaking change, and will lead to a major version bump in the proxy.
We've been prototyping this idea for a while, and would love some feedback on this idea: https://github.com/kurtisvg/cloud-sql-go-connector

As a side note, the new dialer supports tracing and will have support for metrics soon, satisfying #169. See https://github.com/kurtisvg/cloud-sql-go-connector/pull/32 for details.

Closing this as completed.


debug notebook

Codecov Report
Base: 69.36% // Head: 69.36% // No change to project coverage :thumbsup:

Coverage data is based on head (ca620be) compared to base (6653dce).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage   69.36%   69.36%
=======================================
  Files           3        3
  Lines         173      173
=======================================
  Hits          120      120
  Misses         53       53

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

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


Move commits to branch 1.8

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4650/
Test FAILed.

jenkins, test this please

alluxio-bot, check this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4651/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4652/
Test FAILed.

jenkins, test this please

@yuzhu
A PR is only allowed to have a single cherry-pick commit. This should be split up so each PR is a single cherry-pick commit. Thanks!

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4653/
Test FAILed.


fix: remove ldflags from docs generation make target

Removes ldflags from docs generation make command as they are not used.
Their existence causes a conflict with the related PR which moves --path flag processing into init, causing a (nonfatal but anoying) error that ldflags were used but not defined.

@lkingland running failed e2e tests again, but otherwise looks good. Curious if you meant to create this as a draft PR.
/lgtm

@lkingland running failed e2e tests again, but otherwise looks good. Curious if you meant to create this as a draft PR.
/lgtm

I'm assuming that you didn't mean to make this WIP. I'm going to unhold it and if it really was a WIP, we can revert it.
/unhold

/hold cancel


[FORMATTING] Should Preserve the Space between OR and REPLACE SQL Keywords

<Transferred from inferrinizzard/prettier-sql#87, related to #226>
Describe the bug
It removes the space between OR and REPLACE.  in other words it converts OR REPLACE to ORREPLACE
To Reproduce
Try to format something like this one.
CREATE OR REPLACE FUNCTION "public" . "set_current_timestamp_updated_at"() RETURNS TRIGGER AS $$
Expected behavior
It should preserve the space between the OR and REPLACE Keywords. which should remain two words.
Screenshots

Usage

Occurred with PostgreSQL, Not sure about other SQL standards
VSCode version? Version 1.66.2
What are your currently configured VSCode settings for the prettier-sql extension, if relevant to the issue?

// following setting is under the settings.json file. but no entry with the key "prettier-sql"
"[plsql]": {
"editor.defaultFormatter": "inferrinizzard.prettier-sql-vscode"
},
Additional context
Pl. comment if you need anything specific to hunt this bug.

Unable to reproduce. Tried with both 5.x and 6.x. Even tried with VSCode plugin.
@inferrinizzard Do you know anything about this.


docs: podman-top: fix nonworking example

(minor) One example in the podman-top man page results in an
error when actually invoked:
unable to find PID field in ps output. try a different set of ps arguments

Cause: commit 6cb1c31d (PR #400) has to check PIDs in order to
filter those in the container.
Solution: add 'pid' to the list of requested output fields in
the sample command.
Signed-off-by: Ed Santiago<EMAIL_ADDRESS>

LGTM

@rh-atomic-bot r+


Add some javascript unit tests to momentUtils.js

Started to add simple tests to momentUtil.js

@GuillaumeDavy can you have a look at my code and maybe see if you want to add more tests?


Inaccurate/inconsistent "Supported" state for DP-5 in AKV Security Baseline

"DP-5: Use customer-managed key option in data at rest encryption when required" states Data at Rest Encryption Using CMK is supported, which is not accurate, as Key Vault is encrypted using platform keys with no option to configure CMK.
This is the description of this benchmark: "Description: Data at-rest encryption using customer-managed keys is supported for customer content stored by the service."
Per the description, this refers to using CMK to encrypt the data within the Key Vault, which is not possible to do. This section should be updated to state "Supported: False".
The configuration guidance is written to describe storing customer managed keys, but that does not match the description which makes this inaccurate/inconsistent.

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

ID: 9ccbaaaf-2267-bbf4-c6c4-9e7cb63f3465
Version Independent ID: 8e0ae980-eace-6fde-59ec-61ea040196e0
Content: Azure security baseline for Key Vault
Content Source: benchmark/azure/baselines/key-vault-security-baseline.md
Service: key-vault
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@lindbeck
Thanks for your feedback! We will investigate and update as appropriate.

Thank you! I realized that the same feedback applies to "IM-8: Restrict the exposure of credential and secrets" in the same documentation.
The description of this benchmark is: "Description: Data plane supports native use of Azure Key Vault for credential and secrets store.
This refers to storing secrets used by the service in a Key Vault, this is not applicable as Key Vault itself won't use secrets that are stored in another Key Vault, therefor this should be stated as unsupported.

@lindbeck
Thank you for bringing this to our attention.
I've delegated this to content author @msmbaldwin, who will review it and offer their insightful opinions.


Use CLR hosting to run managed code

As preparation for #1036. Fixes #547, maybe #1063 as well (not sure as I can't reproduce).
This makes the c# code previously linked to C++/CLI a standalone managed assembly and get's called during startup.
There're still some issues regarding versioning.
The managed initialization assembly version is set to <IP_ADDRESS>, which makes the in-game console output the wrong version (can be fixed by fetching the latest api assembly version). C++ core doesn't have any version set yet.

Looks like SDK style project require a nuget restore before build

I noticed some scripts will time out if the memory scanning in NativeMemory static constructor takes too long, not sure if it's intoduced by this or already present in previous versions. So I made sure the constuctor is called during domain setup.
Also added pattern/offset caching, which saves the scanning time after a reload and make #1162  easier to implement

I'm sorry, but this is too much. I'd rather not have to introduce yet another DLL in the project to make things work. But I'm willing to modify the SHVDN core in a way that would make it possible to just rename it to a DLL and have another project act as its loader. With 60c02dfed4a92c99fc3315f37999d83dd26419e6, it now can be loaded via Assembly.Load or any other means without locking up. So there can be a separate ASI that hosts the CLR and then loads SHVDN afterwards. This could be further extended to also make it possible to disable the SHVDN core from registering with SHV.
Good point with NativeMemory constructor being able to time out scripts, this already present right now and fixed this with a slightly cleaner method now: 5f514ef35b870e00e65cb0f9e3d36a9c9aaf2cdb

I thought another DLL was being introduced because Sardelka9515 said he couldn't get DllMain in a DLL built of C++/CLI loaded as this message says, but maybe that failed because the DLL coundn't find ijwhost.dll, I guess.

I noticed some scripts will time out if the memory scanning in NativeMemory static constructor takes too long, not sure if it's intoduced by this or already present in previous versions. So I made sure the constuctor is called during domain setup.

I'm sure any commit older than 5f514ef had the issue where some script may time out if the static ctor of NativeMemory takes more than 5 seconds, and that would happen when no scripts accessed any methods/properties that internally touches NativeMemory and some tick event in some script accessed one of them.

That's fine, as per https://github.com/crosire/scripthookvdotnet/issues/1036#issuecomment-1449418370, just that the described C++ core should not live in this repository and SHVDN and SHVDNC would remain separate repositories. SHVDN would be modified so that it can support being loaded by aforementioned C++ core that hosts the CLR, but it also can technically still run on its own.

Fair enough

Is #1036 worth opening? I'm asking because another DLL is probably needed to load scripts built against .NET Core/.NET 5+ since you can't use AppDomain and this PR got closed. If we never intend to add APIs built against .NET 7+, #1036 might as well deserve closed.

Right, my understanding is that .NET 7+ support looks very different from the current SHVDN implementation, making it essentially an entirely new project (since very little of the SHVDN core could be used with it, missing AppDomains etc.). So may as well embrace that and maintain it as such.
I'd rather not be the profile where such a big new undertaking is hosted and refined, considering that I'm not actively involved with GTA modding anymore. So #1036 is unlikely to happen in the context of SHVDN and can probably be closed, in favor of doing it via e.g. https://github.com/Sardelka9515/scripthookvdotnetcore. It might be worth the effort to make the interface between the SHVDN core and scripting APIs simple enough at least to also work with .NET Core, so that the scripting API code can be fully shared (imported via submodule + an external csproj that builds it for .NET Core instead of using the csproj in SHVDN that builds it for the .NET Framework).


Prefix's prefix changes

Which isn't too bad inherently, but it makes the check (no args) function redundant.

we already have/had this suggestion on feathub, which will hopefully come back soon?


the mcu side software paltform for deploy

Thanks for your great works, i am trying to deploy mcunet to stm32f746g-disco, so i want to know what software platform you use to generate the runtime code(tf-micro, tvm), or you just write it line by line?

tf-micro, i find it.

Thanks for your interest in the work. we will also release TinyEngine later this summer, which will provide a faster and more memory-efficient inference compared to tf-lite.


Unit tests

:tada: This PR is included in version 1.2.0-develop.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix  environment variables.

Fix environment variables in .env.example to accord with \test\integration\sendgrid.test.js.
var API_USER    = process.env.SENDGRID_USERNAME || 'some_sendgrid_username';
var API_KEY     = process.env.SENDGRID_PASSWORD || 'some_sendgrid_password';

Thanks!

Thanks for merging!


It will have completions even if "Add PYTHONPATH to search path" is not selected.

Describe the bug

Steps to Reproduce

module2.py file was prepared with below code.

def hello(): print('hello')

Add the module2.py path to PYTHONPATH.

Launch VS and create a Python Application.
In the default python file type import module2.

Expected behavior
It should not have intelliSense of module2 untill you select "Add PYTHONPATH to Search Path".

Additional context and screenshots
It has completions on module2.

Output info
Info: Info: Pylance language server 2021.4.1 (pyright 6410a8a2) startingServer root directory: c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\python\core\pylance\dist

Info: Info: Warning: Info: Info: Info: Info: Info: Info: No configuration file found.Setting pythonPath for service "PythonApplication23": "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python39_64\python.exe"stubPath C:\Users\vting\source\repos\PythonApplication23\PythonApplication23\typings is not a valid directory.Assuming Python version 3.9Assuming Python platform WindowsSearching for source filesFound 1 source fileBackground analysis(1) root directory: c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\python\core\pylance\distBackground analysis(1) started

This may be a pylance issue?

Might be setting this in the workspace config.

I can repro this in today's build.

Output info
Info: Info: Server root directory: c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\python\core\pylance\distPylance language server 2021.8.3 (pyright fa4194fd) starting

Info: No configuration file found.Info:
No pyproject.toml file found.
Info: Setting pythonPath for service "PythonApplication9": "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python39_64\python.exe"Warning:
stubPath C:\Users\vting\source\repos\PythonApplication9\PythonApplication9\typings is not a valid directory.Info:
Assuming Python version 3.9
Info: Info: Info: Info: Info: Assuming Python platform WindowsSearching for source filesFound 1 source fileBackground analysis(1) root directory: c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\python\core\pylance\distBackground analysis(1) started

Pylance respects PYTHONPATH and automatically finds modules there. So this is "as designed" since we now use pylance. @judej Should we just get rid of the action "Add PYTHONPATH to Search Path"?

Note that not picking up PYTHONPATH by default for Intellisense was also very much intentional, hence why the command exists. @zooba might remember why it was done that way.


unmountonblur

Current behavior
unmountonblur is only available in bottom-tab-nav
Expected behavior
Mave unmountonblur available in material-top-tab-nav also
What have you tried
n/a
Your Environment

software
version

node
v15.11.0

yarn
1.22.10

react-native
0.64.0

react-native-tab-view
5.3.14

react-native-pager-view
5.0.11

Please open issues related to React Navigation in React Navigation's channels.

Please open issues related to React Navigation in React Navigation's channels.

..and which repo/channel is that exactly - not sure how you have organized the modules.


define failing on merge

On this line: https://github.com/danielspaniel/ember-data-factory-guy/blob/master/dist/ember-data-factory-guy.js#L221
it calls merge, but I cannot figure out why merge would be defined here on the model definition, and in fact I am getting an undefined function error on this line.

Yikes @rtm  .. you must be defining the model twice, which is why your getting this error.
My original intent was to merge the two definitions, but I never actually finished the merge funciton, so this is my mistake.  Thanks for pointing this out.  Can you find the place where you are defining the same model twice and fix that for now?
I will take this line out also and only put it back in when I finish the merge funciton.

That code is fixed now, and will be pushed out in the next release.


Editor toolbar is bugged and control children handles are invisible when root node is viewport

Godot version:

3.2.3.stable.mono.official
OS/device including version:

Win10
Issue description:

Control nodes are missing handles in the editor, the toolbar toggles 'Lock' and 'Group' are inverted and don't function correctly (don't change state of the selection) and node specific entries are missing (i.e. Layout for controls), regardless of what is selected. This as discovered is due to the parent/root node being a viewport. When it's set to be of type control all other functionalities are working as expected again.
Viewport as root (bugged):

Control as root (normal)

Root viewport settings:

Duplicate of https://github.com/godotengine/godot/issues/30910.

Duplicate of https://github.com/godotengine/godot/issues/30910.

Locking is the lesser of those misbehaviours, the missing scene handles and toolbar entries are the actual bug!

Locking is the lesser of those misbehaviours, the missing scene handles and toolbar entries are the actual bug!

Locking is the lesser of those misbehaviours, the missing scene handles and toolbar entries are the actual bug!

Please open a separate issue for that after searching for duplicates.

Locking is the lesser of those misbehaviours, the missing scene handles and toolbar entries are the actual bug!

Please open a separate issue for that after searching for duplicates.

See also #3572

See also #3572


Error in latest for treesitter provider

Neovim version (nvim -v | head -n1)
NVIM 0.9.5
Operating system/version
macOS 14.3.1
How to reproduce the issue
-- Lazy nvim config
return {
  "kevinhwang91/nvim-ufo",
  dependencies = { "kevinhwang91/promise-async" },
  init = function()
    vim.o.foldcolumn = "0" -- '0' is not bad
    vim.o.foldlevel = 99 -- Using ufo provider need a large value, feel free to decrease the value
    vim.o.foldlevelstart = 99
    vim.o.foldenable = true
  end,
  config = function()
    require("ufo").setup({
      provider_selector = function()
        return { "treesitter" }
      end,
    })
  end,
}

Ensure latest version of nvim and treesitter are installed
Use above config
Open neovim. Observer error message.
...

Expected behavior
No error on starting Neovim.
Actual behavior
When I open Neovim I get the following error message:

This seems to be related to the merging of https://github.com/kevinhwang91/promise-async/pull/6

Shouldn't expose the inner error of promise-async, fixed. Now, should revert the latest commit from ufo.


Make the WDL imports into local imports

Dockstore can handle these well.  But I guess it would be a problem for the Broad Methods repository.
However, some of our WDLs still point to the Broad Methods Repository, so if we forget to update them, then even the Dockstore version of the workflows will be using old imports from the Methods Repository, and will not be synced with GitHub.

cellprofiler_pipeline.wdl still need to be updated and have local imports instead of the ones at the Broad Methods Repository


Stakeholder interviews

User Need
As a Discovery Team
I want to conduct interviews with our stakeholders
so that we understand their role in the domain, insiights into problems they may be facing and gauge their level of interest and engagement.
Value
We understand more about the domain and the needs of our stakeholders
Functional Requirements (What):

Stakeholder interviews

Non-Functional Requirements (How):
For each stakeholder:

Develop questions
Conduct interviews
Document findings (which we will later use to map processes and develop personas)

Acceptance Criteria:

We have interviewed all stakeholders

Started document for discussion/review - https://docs.google.com/document/d/113wlfDdVMfosYVdgtrQjdsIDHSUdtqzLeuB36vOIMA8/edit?usp=sharing


How do I download disputed territories?

I was trying to download data for disputed territories e.g.
pygadm.Items(admin="Z01", content_level=2) # Jammu & Kashmir (India)

However, this doesn't work.
Z01 is actually included within the file for India on gadm.org, but pygadm.Items(admin="IND", content_level=2) doesn't download it either -- it only includes the features whose GID_0 is IND. Any help?

I didn't know there was a custom code for disputed territories. It makes complete sense in the end to have them in customized code so they remain "disputed".
I need to investigate because normally I do use the json file to download entire countries. And I guess as I'm discovering the country using the is3 code, Z01 doesn't appear to exist.
Let me explore the dataset a little bit more to see if all disputed countries are stored using the same type of code (i.e. Z)

In the initial dataset, disputed territories are treated as individual countries (they have a GID that is not based on an ISO code).
I think to manage them I need to review the structure of the gadm_dataset.parquet file.


chore: run examples in on-pull-request workflow

Added a step to run the examples except the loadgen (I'm not sure if we want to run the script every time...) to make sure all the examples are up-to-date.
Updated a few examples to add cache cleanup.
Confirmed that dependabot is looking for new versions of the dependencies.

Closes #283

I'm raising a flag here because we just went through this when Schwern added a workflow to verify examples on pull requests (cc @eaddingtonwhite). It caused a ton of havoc and we ended up reverting the change. I really don't think we want to gate PRs on whether or not the examples build. We pin the SDK version in the examples exactly so we know that they work with the release they're pointed at, and we generally have a spike to bring examples up to spec each time we cut a new release. There shouldn't be an expectation in my opinion that the examples will always work with the HEAD of the repo.

@pgautier404
Thank you for the information!
I was not aware of the previous work Michael S worked on verifying the examples, and the change was reverted.
We created this ticket since I mentioned in one the PRs that we might be able to check examples to make sure they are up-to-date with charges in PRs we're filing (we had a lot of breaking changes recently prior to 1.0).
To me it seems like examples were not updated with those breaking changes recently hence the comment.
With that being said, I'm okay with not adding the step to check the examples every time if it caused many issues. (I want to make this change if it improves our process not worsen it...!!)

we should be verifying that examples build on every PR, BUT
they should be building against a pinned version, AND
we should have dependabot configured to detect when a new version is released, and file a PR to update the examples to the new version.  when this happens the dependabot PR SHOULD be red if the examples no longer build against the new release, so it will draw our attention to it and we can fix it.

i haven't had a chance to look at the code changes in this PR but that ^^ is the state of the art for how we are doing examples for now.

@cprice404 Thank you for the explanation!
I don't think we ever had a build step in our workflows although we already have a make build command.
I added two steps to call this command to run go build ./..., one for the main package and the other for the examples package.
The SDK version is already pined in the examples. Also I believe we setup the dependabot to check daily to detect a new version is released here.

we should be verifying that examples build on every PR, BUT

they should be building against a pinned version, AND

This is what I was getting wrong (well, one of the things anyway), apologies. I had it in my head that the examples were being built against the latest commit, not the latest pinned version. You're right about that being the safe and standard way to do this. Sorry for the brain flatulence.
In talking about this yesterday with @poppoerika I tracked down the commit that caused us so many problems, and it was actually adding the examples into the linting step that caused us so many problems we had to revert it in 8618c44c.