I met a mistake：Multipart body length limit 16384 exceeded

I use the middle to achieve large file upload, but encountered a mistake, I should be how to solve?

All of the settings below are invalid
public void ConfigureServices(IServiceCollection services)
    {
        services.Configure<FormOptions>(x =>
        {
            x.ValueLengthLimit = int.MaxValue;
            x.MultipartBodyLengthLimit = int.MaxValue;
            x.MultipartHeadersLengthLimit = int.MaxValue;
        });

        services.AddMvc();
    }

@Tratcher any idea on this limit?

Some multipart bodies include preamble data before the first mutipart section. MultipartReader automatically drains that portion but limits it to 16k. There's not currently a way to configure this behavior.
See https://github.com/aspnet/HttpAbstractions/blob/c0f937239a0a099b73c67c96ab9e1c875952f67f/src/Microsoft.AspNetCore.WebUtilities/MultipartReader.cs#L48-L50
Do you know why your upload includes a large preamble section? This is likely unintentional.

@Tratcher parking this on your plate until we can figure out what's wrong.

@Tratcher @Eilon
Thank you for your help.
I want to adjust the limit, but the setting is invalid, I want to achieve a separate file upload service through the middleware, I test only upload a 31.7KB file, it is temporarily unable to achieve this idea, I refer to this tutorial:
https://docs.microsoft.com/zh-cn/aspnet/core/mvc/models/file-uploads#uploading-large-files-with-streaming

What client are you using for the upload? An HTML form in a browser? Which browser?
Can you share a Fiddler trace of the upload? From your description it contains unexpected content.

@Tratcher
I use Postman to upload, and this is the fiddler content

This is the test code:
`
public class FileStreamUploadMiddleware
{
private readonly RequestDelegate _next;
    public FileStreamUploadMiddleware(RequestDelegate next)
    {
        _next = next;
    }

    public async Task Invoke(HttpContext context)
    {
        var boundary = HeaderUtilities.RemoveQuotes(context.Request.ContentType);
        var reader = new MultipartReader(boundary.Value, context.Request.Body);
        reader.HeadersLengthLimit = int.MaxValue;
        var section = await reader.ReadNextSectionAsync();
        while (section != null)
        {
            ContentDispositionHeaderValue contentDisposition;
            var hasContentDispositionHeader = ContentDispositionHeaderValue.TryParse(section.ContentDisposition, out contentDisposition);

            if (hasContentDispositionHeader)
            {
                if (HasFileContentDisposition(contentDisposition))
                {
                    //保存文件
                }
            }
            section = await reader.ReadNextSectionAsync();
        }
    }

    #region Helper
    public static bool IsMultipartContentType(string contentType)
    {
        return !string.IsNullOrEmpty(contentType)
               && contentType.IndexOf("multipart/", StringComparison.OrdinalIgnoreCase) >= 0;
    }
    public static bool HasFileContentDisposition(ContentDispositionHeaderValue contentDisposition)
    {
        // Content-Disposition: form-data; name="myfile1"; filename="Misc 002.jpg"
        return contentDisposition != null
               && contentDisposition.DispositionType.Equals("form-data")
               && (!string.IsNullOrEmpty(contentDisposition.FileName.Value)
                   || !string.IsNullOrEmpty(contentDisposition.FileNameStar.Value));
    }
    #endregion
}

`

Can you save and upload that Fiddler trace? I need to see the request body.

Is this one?
4_Full.txt

The request looks normal, but I see you're retrieving the boundary incorrectly.
var boundary = HeaderUtilities.RemoveQuotes(context.Request.ContentType);
The linked sample code does this instead:
    var boundary = MultipartRequestHelper.GetBoundary(
        MediaTypeHeaderValue.Parse(Request.ContentType),
        _defaultFormOptions.MultipartBoundaryLengthLimit);

I don't know how to express my feelings. I'm too tired. My code is the same as the core part of the official sample code. Can you look at the code with my heart?
https://github.com/aspnet/Docs/blob/master/aspnetcore/mvc/models/file-uploads/sample/FileUploadSample/MultipartRequestHelper.cs
` public static string GetBoundary(MediaTypeHeaderValue contentType, int lengthLimit)
{
var boundary = HeaderUtilities.RemoveQuotes(contentType.Boundary);
if (string.IsNullOrWhiteSpace(boundary))
{
throw new InvalidDataException("Missing content-type boundary.");
}
        if (boundary.Length > lengthLimit)
        {
            throw new InvalidDataException(
                $"Multipart boundary length limit {lengthLimit} exceeded.");
        }

        return boundary;
    }`

What @Tratcher is referring to:
You use:
var boundary = HeaderUtilities.RemoveQuotes(context.Request.ContentType);
context.Request.ContentType is in your request multipart/form-data; boundary=----WebKitFormBoundary8Su1nBMUX1qEXx1z, you need to first parse it, because you're only interested in the boundary parameter
In the reference code you posted, contentType is already a parsed MediaTypeHeaderValue, take a close look at your screenshot it says contentType.Boundary, and take a look at the method signature it says MediaTypeHeaderValue, so yes you need to do exactly what @Tratcher  said, parse the content-Type to MediaTypeHeaderValue then take the property Boundary and strip the (optional) quotes.
e.g.
var parsedContentType = MediaTypeHeaderValue.Parse(context.Request.ContentType);
var boundary = HeaderUtilities.RemoveQuotes(parsedContentType.Boundary);

I am sorry, I ignored this small detail, thank you very much for the patience solution. thank


Log out dropdown is hidden by alert

The distance between the toggle and log out dropdown is a bit too much
The dropdown is hidden when there are alert messages

Closing as we've addressed by moving the notifications to the bottom right


Incorrect JSON

[Enter feedback here]
Please remove the comma on the ExposedPorts JSON object

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

ID: 02676e4d-000b-ed97-7e9a-fadfb7019eb1
Version Independent ID: 3cd31ab9-9b25-8afc-da86-c9d9a53958ce
Content: Access built-in metrics - Azure IoT Edge
Content Source: articles/iot-edge/how-to-access-built-in-metrics.md
Service: iot-edge
GitHub Login: @v-tcassi
Microsoft Alias: v-tcassi

@lucarv Thanks for pointing this out. The document is updated and changes will go live soon.


Update dependencies

Pull requests:

https://github.com/senzing-garage/demo-entity-search/pull/79


Update static.py

Auto return "index.html" for dirs.

@huge-success/sanic-core-devs we probably need a policy about how and when to close PRs that die on the vine. I'm closing this one due to lack of response from the submitter.

We do: https://probot.github.io/apps/stale/


Incorrect outsideWidth, outsideHeight after Windows wake up?

Ebitengine Version
3eaa03e1
Operating System

[X] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] PlayStation 5
[ ] Xbox
[ ] Web Browsers

Go Version (go version)
1.22rc2
What steps will reproduce the problem?

Run an executable with an Ebiten app
Suspend (hibernate) Windows.
Wake up.
The working area of the window now has white padding at the bottom and on the right side:

What is the expected result?
No padding.
What happens instead?
The working area of the window now has white padding at the bottom and on the right side.
Anything else you feel useful to add?
If I resize the window (if the window is resizable) the padding disappears.

Does this happen with Ebitengine v2.6.6?

 It doesn't. Sorry, the ebiten-ui project uses v2.5.0 and I didn't pay enough attention to that.

Well, but didn't you confirm this happened with 3eaa03e...?

I made a mistake... I thought that my app which uses the latest Ebiten has the same problem but my padding was actually intentional. Then I didn't test thoroughly. I will be more careful next time. Thank you very much for this wonderful framework. I started to create a desktop learning application funcionally based on this one: https://play.octavian.app/piano And it looks very promising so far.

OK thank you for elaborating!


CI: broken merge_group tests

Issue
Merge group GPU benchmark tests are broken:
https://github.com/lurk-lab/lurk-rs/actions/runs/6777848351/job/18422227933
Run cat fc495987b972a3736da3d9[1](https://github.com/lurk-lab/lurk-rs/actions/runs/6777848351/job/18422227933#step:17:1)46e561559[2](https://github.com/lurk-lab/lurk-rs/actions/runs/6777848351/job/18422227933#step:17:2)0c06036.json | criterion-table > BENCHMARKS.md
  cat fc495987b972a3736da3d9146e56155920c06036.json | criterion-table > BENCHMARKS.md
  shell: /usr/bin/bash -e {0}
  env:
    CACHE_ON_FAILURE: false
    CARGO_INCREMENTAL: 0
    BASE_REF: 25c170bf85cd0676fd012[3](https://github.com/lurk-lab/lurk-rs/actions/runs/6777848351/job/18422227933#step:17:3)7acdf[4](https://github.com/lurk-lab/lurk-rs/actions/runs/6777848351/job/18422227933#step:17:4)0e1632d39[5](https://github.com/lurk-lab/lurk-rs/actions/runs/6777848351/job/18422227933#step:17:5)9[8](https://github.com/lurk-lab/lurk-rs/actions/runs/6777848351/job/18422227933#step:17:8)
    LURK_BENCH_OUTPUT: commit-comment
An error occurred processing Criterion data: Duplicate column: fib-branch=gh-readonly-queue

Which is logically followed by Error: Missing comment 'body' or 'body-path'. on the next step (commit-comment)
Origin
Introduced in #852 and made visible by #857.

I've added a gist of :
LURK_RC="100,600" LURK_BENCH_OUTPUT=commit-comment cargo criterion --bench fibonacci_lem --message-format=json

At:
https://gist.github.com/huitseeker/2b2ffc93d2963dda8e1d920726b375f8
Passing it to criterion-table does produce
An error occurred processing Criterion data: Duplicate column: fib-branch=master

Seemingly, using the bench group as a separator with the function name duplicated is not helpful when using a single json, contrarily to two (differential bench), which is what I used to demo the output in #852.


Incompatible with release-it v15

ERROR Package subpath './lib/util.js' is not defined by "exports" in /Users/jhawker/Projects/GitHub/ember-resize-modifier/node_modules/release-it/package.json imported from /Users/jhawker/Projects/GitHub/ember-resize-modifier/node_modules/release-it-lerna-changelog/index.js

This error is reproducible in #201:
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './test/util/index.js' is not defined by "exports" in /home/runner/work/release-it-lerna-changelog/release-it-lerna-changelog/node_modules/release-it/package.json imported from /home/runner/work/release-it-lerna-changelog/release-it-lerna-changelog/test.js

Just hit this as well myself.

https://github.com/rwjblue/release-it-lerna-changelog/pull/208 should hopefully fix this.


task: delete .github/workflows/report.yaml

Description

This is not needed anymore as the reports are not done by a cronjob using knowledge-middleware-report package made using the Dockerfile.report.

This has changed, so I'm not sure if we should remove this workflow now @fivegrant ?

@YohannParis yes we want to keep this


Add curl to container

Adds curl package to the container for the purpose of creating a custom healthcheck in user-side docker compose files.
This is a compromise to the denied PR in: https://github.com/ollama/ollama/pull/1909 - instead of adding an intergrated health check, just add the tool so users can create one themselves in their local docker compose files.
Solves feature request: https://github.com/ollama/ollama/issues/5389

As a workaround until this PR is merged, you can modify your compose file with the following lines to add curl on startup:
    entrypoint: ["/bin/bash","-c"]
    command: ["apt-get update && apt-get install -y curl && /bin/ollama serve"]

Thank you so much for contributing this. The container here is meant to be minimal to help users get started. I'm less opinionated about this so happy to change my mind as well.

I use curl on container startup to access the ollama api to download and preload certain models. So it would be nice to have a „native“ way to access the API without downloading curl every time the container is recreated.


Application can crash if linked against both libppd and libcups

CUPS, on its sources, implements (duplicates implementation) of some, but not all, libppd functions, and exports them from the libcups.so library.
For example, the ppdOpenFd and ppdClose functions are exported from the libcups.so, while ppdLoadAttributes is not.
If the application is linked against both libcups.so and libppd.so libraries, the following scenario is possible:

application calls ppdOpenFd from the libcups.so
application calls ppdLoadAttributes, and this call is now goes into the libppd.so library. This function modifies the ppd_file_t structure, but the definition of this type is not exactly the same between these two libraries (the actual difference seems to be in the ppd_cache_t in the libppd vs _ppd_cache_s in the libcups.so)
application calls ppdClose and this call goes to the libcups.so and crashes there in attempt to destroy the _ppd_cache_s structure, which was actually created in the libppd.so

The exact behaviour depends on a link order; if libppd is linked first, everything is OK, while if libcups is linked first, application crashed.
I've caught this issue at Fedora 40 with cups-2.4.11-1.fc40.x86_64 and libppd-2.1~b1-2.fc40.x86_64 packages installed, but most likely it may affect other distros and other version.
It is hard to say what could be a perfect solution for this problem, because libcups header files announces exporting many ppdXXX functions, so this is a part of the libcups public API. And without linking of the libcups.so the libppd.so functions that use ipp_t and ipp_attribute_t become unusable, because these types are opaque and accessible only via libcups.so-exported API...

What if we mark all duplicated symbols as weak at the libcups.so and somewhere document this behaviour change?
If it breaks something, it will (at the first glance) break only apps that really and intentionally use mix of the overlapping APIs from the both libraries, which is incorrect and extremely dangerous by itself.
It is not a perfect solution, but it least it will make things deterministic and result will not depend on a link order.

Again, libcups.so.2 (CUPS 2.x libcups) always has the PPD functions in it. Any notion of doing weak linkage (not supported on all CUPS platforms) is incorrect since libcups is the original library.
Either libppd should rename its functions or it should not include them when building on a system with CUPS 2.x since libcups will provide them.

I don't see any equivalent of the  ppdLoadAttributes function in the libcups.so...

@alexpevzner It is called _ppdCacheCreateWithFile in libcups.so.2...

@alexpevzner Note that libppd is putting together the PPD-related API functions of libcups, the functions needed for the PPD compiler (*.drv -> *.ppd) utilities, functions of cups-driverd (for PPD collection handling), and making some internal libcups functions public API. There are also added several new functions. This is all clearly marked in ppd/ppd.h, in the part of the function prototypes at the end. There I have also told for what the new functions are good for and from where in CUPS 2.x they come from.

@tillkamppeter One thing you could do here is change the main header to have a block of defines that rename the functions to names that don't conflict with libcups, e.g.:
#define ppdClose libppdClose
#define ppdCollect libppdCollect
#define ppdCollect2 libppdCollect2
#define ppdConflicts libppdConflicts
#define ppdEmit libppdEmit
#define ppdEmitAfterOrder libppdEmitAfterOrder
#define ppdEmitFd libppdEmitFd
#define ppdEmitJCL libppdEmitJCL
#define ppdEmitJCLEnd libppdEmitJCLEnd
#define ppdEmitString libppdEmitString
#define ppdErrorString libppdErrorString
#define ppdFindAttr libppdFindAttr
#define ppdFindChoice libppdFindChoice
#define ppdFindCustomOption libppdFindCustomOption
#define ppdFindCustomParam libppdFindCustomParam
#define ppdFindMarkedChoice libppdFindMarkedChoice
#define ppdFindNextAttr libppdFindNextAttr
#define ppdFindOption libppdFindOption
#define ppdFirstCustomParam libppdFirstCustomParam
#define ppdFirstOption libppdFirstOption
#define ppdInstallableConflict libppdInstallableConflict
#define ppdIsMarked libppdIsMarked
#define ppdLastError libppdLastError
#define ppdLocalize libppdLocalize
#define ppdLocalizeAttr libppdLocalizeAttr
#define ppdLocalizeIPPReason libppdLocalizeIPPReason
#define ppdLocalizeMarkerName libppdLocalizeMarkerName
#define ppdMarkDefaults libppdMarkDefaults
#define ppdMarkOption libppdMarkOption
#define ppdNextCustomParam libppdNextCustomParam
#define ppdNextOption libppdNextOption
#define ppdOpen libppdOpen
#define ppdOpen2 libppdOpen2
#define ppdOpenFd libppdOpenFd
#define ppdOpenFile libppdOpenFile
#define ppdPageLength libppdPageLength
#define ppdPageSize libppdPageSize
#define ppdPageSizeLimits libppdPageSizeLimits
#define ppdPageWidth libppdPageWidth
#define ppdSetConformance libppdSetConformance


Handling null resources and jtwig-core version up2date

Coverage remained the same at 97.927% when pulling 4075c6f6a056be439f51c61c83c3248f6568a33b on lyncodev:master into fc530ac672ca5e669f4a96024eaec7677586a3e4 on jtwig:master.

Coverage remained the same at 97.927% when pulling d0e04490440ce0f729a7ee9248d2051a887199a0 on lyncodev:master into fc530ac672ca5e669f4a96024eaec7677586a3e4 on jtwig:master.


sdk.ide.Diff() returns nothing even when git diff --cached does

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS:macOS 14.5
- Continue: v0.8.27
- IDE: VSCode 1.89.1

Description
tried implementing the example from https://docs.continue.dev/customization/slash-commands#custom-slash-commands-1 and it keeps returning a null diff, even when I have staged changes.  I put together https://gist.github.com/cristoslc/0d9a555f3aab9d27ca6f2e3e60294d64 which tries git diff --cached, and that's returning what I expect when I have staged files.
To reproduce

Implement https://docs.continue.dev/customization/slash-commands#custom-slash-commands-1  in your config.ts
run custom slash command
EXPECTED: Commit message summarizing diff
ACTUAL: Response indicating LLM did not receive any diff

WORKAROUND: https://gist.github.com/cristoslc/0d9a555f3aab9d27ca6f2e3e60294d64
Workaround uses git diff --cached in the project folder and works as expected.
Log output
[Extension Host]% {text: 'const { exec } = require("child_process");\nconst {…rn config;\n}\n\nmodule.exports = { modifyConfig };\n'}
console.ts:137 [Extension Host]% {text: 'export function modifyConfig(config: Config): Conf…ssage;\n      }\n    },\n  });\n  return config;\n}\n\n\n'}
console.ts:137

Discord ref: https://discord.com/channels/1108621136150929458/1108621136830398496/1240360160766066779

looks like it's only showing unstaged changes. looking into this now

Relevant line of code is here:
https://github.com/continuedev/continue/blob/preview/extensions/vscode/src/util/ideUtils.ts#L593-L594
It looks like the one arg to this function is whether or not to use caching

So it seems like this cache parameter actually is a "staged or not staged" parameter after having tested it a bunch. I'm weary of this, probably there's some reason something staged would be cached, but haven't found the behavior to be otherwise yet. So this commit is my solution for now, but I'm going to keep my eye on it. The thing to look for at this point would be duplication
https://github.com/continuedev/continue/commit/b5052a3176af984dc82aa6a8994aefce6b3de0fc

I noticed this problem myself, just trying to use @Git Diff summarize this commit for me in the chat panel.  I assume it's the same problem, because all of the files in the commit were new, and staged to be committed.  Because they're all new (untracked) files, even running git diff from the terminal shows no diff.  If I stage them (adding them to the index), then git diff also shows no changes, because they're already staged.  I think what would be nice is for @Git Diff to use git diff --cached to just look at the staged changes (that way I can control exactly what I want it to summarize for me).
If I should open a separate ticket for this, let me know.  It looks like your proposed solution is to include staged and unstaged changes in the context, which would also be fine I suppose.  It still doesn't include untracked files, and doesn't allow as much control (in terms of what changes are being included), but it would work well for 90% of cases.


Support Connection get/setNetworkTimeout().

This is essential to minimize the effects of network partition events.
Related to #72 and #85.

Codecov Report

Merging #253 into dev will increase coverage by <.01%.
The diff coverage is 55.55%.

@@             Coverage Diff              @@
##                dev     #253      +/-   ##
============================================
+ Coverage     33.46%   33.47%   +<.01%
+ Complexity     1491     1486       -5
============================================
  Files            97       97
  Lines         23390    23415      +25
  Branches       3840     3841       +1
============================================
+ Hits           7828     7838      +10
- Misses        14001    14010       +9
- Partials       1561     1567       +6

Flag
Coverage Δ
Complexity Δ

#JDBC41
33.35% <55.55%> (-0.02%)
1478 <2> (-5)

#JDBC42
33.39% <55.55%> (+0.04%)
1481 <2> (ø)
:arrow_down:

Impacted Files
Coverage Δ
Complexity Δ

...t/sqlserver/jdbc/SQLServerConnectionPoolProxy.java
13.61% <0%> (-0.22%)
12 <0> (ø)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
37.01% <100%> (+0.16%)
0 <0> (ø)
:arrow_down:

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
41.27% <60%> (+0.09%)
228 <2> (-1)
:arrow_down:

src/main/java/microsoft/sql/DateTimeOffset.java
37.14% <0%> (-2.86%)
8% <0%> (-2%)

...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java
24.55% <0%> (-1.12%)
39% <0%> (-3%)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
45.05% <0%> (ø)
183% <0%> (ø)
:arrow_down:

...om/microsoft/sqlserver/jdbc/ReaderInputStream.java
42.69% <0%> (+1.12%)
15% <0%> (+1%)
:arrow_up:

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 598a3b4...3abcebc. Read the comment docs.

@v-xiangs Review recommendations integrated.

@brettwooldridge Thank you for your contribution.


Can't bind to 'options' since it isn't a known property of 'ss-multiselect-dropdown'.

I wanted to use this control in my project. I've added the basic functionality and amongs other things I'm getting this error:
Can't bind to 'options' since it isn't a known property of 'ss-multiselect-dropdown'.

 If 'ss-multiselect-dropdown' is an Angular component and it has 'options' input, then verify that it is part of this module.
 If 'ss-multiselect-dropdown' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message.
 (" </button -->
 <ss-multiselect-dropdown [ERROR ->][options]="myOptions" [(ngModel)]="selectedOptions" (ngModelChange)="onChange($event)"></ss-multisele"): e@40:59
 'ss-multiselect-dropdown' is not a known element:
If 'ss-multiselect-dropdown' is an Angular component, then verify that it is part of this module.
If 'ss-multiselect-dropdown' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message.

This was mentioned here https://github.com/softsimon/angular-2-dropdown-multiselect/issues/14 but I see that this issue was closed.
Maybe it's worth to mention that, I'm installing this from npm install and the version that installs is 1.5.4. I've seen that in releases is version 1.5.0. Just for tests I've done this with bower and with bower the 1.5.0 is downloaded. Npm is downloading wrong version or maybe there is a bug in version 1.5.4?

I've managecd to solve the issue, I've peasted it to the wrong module.....


no offline mode

https://github.com/SpockBotMC/SpockBot/commit/47919485da01f55565932b6e985f380a3a2d4a88
I think this commit breaks offline demo

{'auth': {'authenticated': False}}

I did that, and is not working
I have
start_settings = {
'username': 'player_name',
'authenticated': False,
}
the only way I got to run the demo was changing
default_settings = {
'authenticated': True,  <-- This True to False
'sess_quit': True,
}
in file spock/plugins/core/auth.py

Settings parser doesn't work like that, I haven't documented it (yet?)
By passing a dict to the "start" parameter, you're telling ONLY the "stayrt" plugin that those are the settings you want it to use. Start plugin doesn't care about authentication, you can either pass {'authenticated': False} as an "auth" parameter or you can build a more general settings dict and pass that to the "settings" parameter.
Anyway, fixed in https://github.com/SpockBotMC/SpockBot-Contrib/commit/8f39df86795b19bc2f36ce79c9e62e321c8ea4cf

humm ok.
Now, I understand, thanks


Cannot run / build for web

[🔴] × flutter build web

💪 Building with sound null safety 💪

Target dart2js failed: Exception: /home/praharshb/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_for_web-2.1.4/lib/image_picker_for_web.dart:246:23:
Error: 'is' can't be used as an identifier because it's a keyword.
      if (!_completer.is Completed && files != null) {
                      ^^
/home/praharshb/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_for_web-2.1.4/lib/image_picker_for_web.dart:246:26:
Error: Expected ')' before this.
      if (!_completer.is Completed && files != null) {
                         ^^^^^^^^^
/home/praharshb/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_for_web-2.1.4/lib/image_picker_for_web.dart:246:23:
Error: The getter 'is' isn't defined for the class 'Completer<PickedFile>'.
 - 'Completer' is from 'dart:async'.
 - 'PickedFile' is from 'package:image_picker_platform_interface/src/types/picked_file/html.dart'
 ('/home/praharshb/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_platform_interface-2.4.1/lib/src/types/picked_file/html.dart').
      if (!_completer.is Completed && files != null) {
                      ^^
/home/praharshb/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_for_web-2.1.4/lib/image_picker_for_web.dart:248:42:
Error: Property 'first' cannot be accessed on 'List<File>?' because it is potentially null.
 - 'List' is from 'dart:core'.
 - 'File' is from 'dart:html'.
          html.Url.createObjectUrl(files.first),
                                         ^^^^^
Error: Compilation failed.

Compiling lib/main.dart for the Web...                             10.6s
Exception: Failed to compile application for the Web.

Please add more details to your issue such as the package version you are using and your Flutter version. A minimal reproductible code sample would be great too.

I retried after changing systems and compiled it perfectly. I think it's an issue in Flutter installation? In any case, should not be relevant here.
Thanks, closing for now.

I retried after changing systems and compiled it perfectly. I think it's an issue in Flutter installation? In any case, should not be relevant here. Thanks, closing for now.

Hi, I don't know what do you mean by "changing systems". but I have the same problem and I can't solve it by now.
so I reopen the issue.

my flutter & dart versions :
Flutter 3.0.1 • channel stable • https://github.com/flutter/flutter.git
Framework • revision fb57da5f94 (6 months ago) • 2022-05-19 15:50:29 -0700
Engine • revision caaafc5604
Tools • Dart 2.17.1 • DevTools 2.12.2
project:
environment:
sdk: ">=2.17.1 <3.0.0"
error complete info:
/F:/Softwares/fluttersdk/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-3.0.0+1/lib/image_picker_web.dart:144:17: Error: Type 'Uint8List' not found.
static Future<Uint8List?> getImageAsBytes() async {
^^^^^^^^^
/F:/Softwares/fluttersdk/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-3.0.0+1/lib/image_picker_web.dart:176:22: Error: Type 'Uint8List' not found.
static Future<List?> getMultiImagesAsBytes() async {
^^^^^^^^^
/F:/Softwares/fluttersdk/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-3.0.0+1/lib/image_picker_web.dart:207:17: Error: Type 'Uint8List' not found.
static Future<Uint8List?> getVideoAsBytes() async {
^^^^^^^^^
/F:/Softwares/fluttersdk/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-3.0.0+1/lib/image_picker_web.dart:233:22: Error: Type 'Uint8List' not found.
static Future<List?> getMultiVideosAsBytes() async {
^^^^^^^^^
/F:/Softwares/fluttersdk/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-3.0.0+1/lib/image_picker_web.dart:179:18: Error: 'Uint8List' isn't a type.
var files = [];
^^^^^^^^^
/F:/Softwares/fluttersdk/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-3.0.0+1/lib/image_picker_web.dart:191:18: Error: 'Uint8List' isn't a type.
var files = [];
^^^^^^^^^
/F:/Softwares/fluttersdk/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-3.0.0+1/lib/image_picker_web.dart:196:49: Error: The argument type 'Object?' can't be assigned to the parameter type 'Uint8List'.

'Object' is from 'dart:core'.
'Uint8List' is from 'dart:typed_data'.
return files.map((e) => Image.memory(e)).toList();
^
/F:/Softwares/fluttersdk/flutter/.pub-cache/hosted/pub.dartlang.org/image_picker_web-3.0.0+1/lib/image_picker_web.dart:236:18: Error: 'Uint8List' isn't a type.
var files = [];
^^^^^^^^^
Failed to compile application.


The public connected ADLS Gen2 blob container displays as a regular blob container

Storage Explorer Version: 1.17.0
Build Number: 20210129.3
Branch: main
Platform/OS: Windows 10/ Linux Ubuntu 18.04/ MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to Reproduce

Expand one ADLS Gen2 storage account -> Blob Containers.
Create a blob container -> Right click the blob container.
Select 'Set Public Access Level...' -> Set the container public access level to 'container'.
Connect to the public blob container.
Observe the public connected blob container -> Check whether it displays as an ADLS Gen2 blob container.

Expected Experience
The public connected blob container displays as an ADLS Gen2 blob container.
Actual Experience
The public connected ADLS Gen2 blob container displays as a regular blob container.

By design. When connected publicly, we cannot detect HNS being enabled. Additionally, when connecting publicly, you cannot use any of the Gen2 specific features anyways.


Rad patch's should propagate

A patch fails to propagate from a contributor to the project maintainer.
The error is related to the maintainer attempting to fetch a non-existing git commit object from the contributor:
2023-03-04T11:41:42.219-05:00 DEBUG service  Draining outbox for session z6MknegFQPn3LFCkpM1f1d9a3AerPcqiN8DcmCGetZxkibvh (0 message(s))
2023-03-04T11:41:42.255-05:00 DEBUG worker   Git: remote: error: Object 2eaf5835751798dce20d9805de33da09ad3b41f5 not a commit
2023-03-04T11:41:42.255-05:00 DEBUG worker   Git: remote: fatal: revision walk setup failed

To recreate the error, run this script from the root of this project.  Beware it will delete any existing 'tmp' directory.
https://github.com/slack-coder/heartwood/blob/slack-coder/test-patch-propagation/script/propagate-patch-test.sh

Is this related to the "corruption" issue? Otherwise it's just a matter of calling announce-refs after creating the patch.

I wonder if it's because they're not in the signed refs @cloudhead

Yeah, this should be fixed by one of my PRs :)

Closed by #375


add JsonRpcException for error server responses without wrapping it into ErrorData

use case: controller returns some object or throws specific error. user wants to control error response (code/msg/data) without changing method signature to IActionResult.

fixed with #46


Update parameter to be savable

An attempt to make the custom tensor Parameter savable (loadable) with tf.saved_model.

Hey @awav, how would you like to proceed with this?

these types can be removed if they're not in use any more https://github.com/GPflow/GPflow/pull/1518/files#diff-e2281e00a6a807545fc8c393630c5c36L17-L19

@vdutor, @sam-willis I added a test.


bug: ui sends patch each time you load /configure endpoint

not that it does harm it's just useless

fixed elsewhere


Roslyn target stops server if no option is selected

If you use the Roslyn target to select the solution and you don't select any, because for example you changed your mind, i think the behaviour should be that nothing happens. Instead it stops the server. If you use something like telescope ui select this will be a common occurrence.

Absolutely. I never use the target command myself so thought it worked as expected. Will fix

Can you test this: https://github.com/seblj/roslyn.nvim/pull/114

I don't have the same problem btw using my own vim.ui.select and the built in one

Can you test this: #114

Can confirm no longer happens, using telescope ui select aswell


Make daemon newable

Do have manual tests done?

yes

/cc @antho1404 I remember that you wanted to do your own manual tests as well, in Discord.


Drop support for Ruby 2.3 and 2.4

Starting with the 2.0.0 release, drop support for Ruby 2.3 and 2.4, which are both EOL. This won't present a problem for Debian and Ubuntu since the oldest distributions for which asciidoctor-pdf packaged (buster and eoan, respectively) provide Ruby 2.5.
We'll continue to leave Ruby 2.3 enabled in the 1.5.x branch since it is not introducing breaking changes (and will go into maintenance mode once 2.0.0 is released).

resolved by 95785d4


update blame command to open on github with -g|--github option

https://git-scm.com/docs/git-blame

Hey @iki that's not a bad idea, but we'll take a break from extending core git commands right now and instead focus on improving hub custom commands like hub browse. You should be able to more easily navigate to online blame view with a better browse command. But we won't add any new flags to git blame right now.

@mislav imo extending git blame cmd has much better ux: the user already knows the syntax and can easily switch between console/github display command

I understand, it's just that I don't believe right now online blame would be any better than blame in the console. If you already have the syntax for blame on the command-line, I would say just go ahead and do it in the console since it would be much faster and you have more options to drill down into history.

That's matter of preference, github blame is faster to overview with user icons and clickable commits. The main reason is the blame URL though, that can be passed to discussion on issues/PRs/slack/...


Thick icons in electron/angular

Select one ...  (check one with "x")
[x] bug
[ ] feature request
[ ] enhancement

Expected behavior
I am using angular in an electron application with Clarity as front-end framework (here is the seed I use : https://github.com/maximegris/angular-electron). I expected icons to look like same as in web mode (as electron is chromium based).
Actual behavior
When I use icons (in header brand or nav, or anywhere else in the app), they are too much thick in dev mode. This is note the case when the angular part is run in web-mode or when the final electron app is built.
I doesn't see differences in DOM or style properties between electron dev-mode/built-mode/angular web-mode.
Reproduction of behavior
I use angular-electron version 1.4.2 with electron version 1.6.10.
I simply clone this project, add Clarity following instructions (https://vmware.github.io/clarity/get-started#seedProjectAngular starting at step 2, skipping MutationObserver polyfill as I will never be on IE10) and simply use some sample from Clarity documentation.
Environment details

Angular version: 4.1.3

Clarity version: 0.9.7

OS and version: MacOS Sierra

Browser: Electron app (Chromium based)

@Sparkoff: Wanted to check if the issue you raised is the same as this one: https://github.com/vmware/clarity/issues/969

It looks like to be same (I didn't saw this issue).
But I am on angular 4.1.3 and electron 1.6.10, which solved the issue and icons are still odds. Removing @webcomponents/custom-elements doesn't make any differences.

Hi @Sparkoff I don't believe the problem is occuring because of Clarity Icons. I have successfully installed clarity-icons in Electron in the past (multiple times) as I mentioned here in this issue #969.  It might be due to other packages that's causing SVG graphic elements to render improperly. I would also make sure to clear all the caches involved in SVG rendering.

@Sparkoff
We haven't heard back on this issue in a while. Were you able to resolve it? I'm closing for now due to inactivity.

I'm having the same issue -again-.
Despite it was apparently solved, I'm experiencing the thick icons issue again with clarity and electron.
I'm not sure where the problem resides, I'm trying to remove stuff from my package.json and trying to run the application again, but nothing seems to be solving the problem at all.
 There may be some styling introduced by one of these packages that is affecting the icons.
Barring that, it may be worth upgrading electron to 1.7.2+ because there appears to be a chromium+electron issue that was resolved there...

@mathisscott Sure, I can inspect anything needed, I will upload everything today's afternoon though, since I don't have my mac with me right now.
I will also try to update to Electron 1.7.2+ at this point, maybe it's just enough to solve the issue :).


Dead links on Zenodo

Hi all,
I just noticed that links to RDF packages pointing from zenodo don't work. Example:
https://zenodo.org/records/6817639
If you click on the RDF link on that page you get an error message from bioimage.io:

Also, beginner might not know what RDF packages are. People stumbling over the models on Zenodo might not have any idea what to do with them. If the zenodo description is auto-generated, consider adding an explanatory sentence there.
CC @constantinpape

Hi @haesleinhuepf
Thanks for highlighting this one! We are moving away from Zenodo and close to releasing a new model contribution system. Hopefully with that, this issue will be fixed!

Hi @esgomezm ,
thanks for the feedback! Just oit of curiosity: Will the links on zenodo stay dead links then? May there be any chance to make them point to a place where users see how to download a goven model with the new infrastructure?
Thanks!
Best,
Robert

Hi @haesleinhuepf
The links are not dead, as the files remain in zenodo repositories. Yet another thing is whether a zenodo repository was at any time, a valid model in the zoo and therefore, a model from the BioImage Model Zoo. This content should always be searched and accessed from the BioImage Model Zoo, to avoid any confusion. We encourage anyone to use our site to search for bioimage model zoo models rather than Zenodo or any other storage platform we cannot always maintain or control.

The links are not dead, as the files remain in zenodo repositories.

The zenodo repositories come with a link in the description (see screenshot above). This link leads to an error message. There is nothing else in the description. Hence, if people search on zenodo for models and find your model, they have no chance to know what to do with it. Consider updating the error message to be more helpful 😉

Yes, accepted models have all the links correctly. What I'm trying to explain is that this is not a model from the BioImage Model Zoo, and in reality, it is not even a model. It is not validated or correct, and people should not search for BioImage Model Zoo models in Zenodo but in the BioImage Model Zoo. The main issue here is that as far as we know, Zenodo repositories cannot be removed so this record (or any incorrect one) will remain there. Do you know how to do so by any chance? Thanks

@haesleinhuepf can you maybe explain how you happened to find this model? Because this looks like a failed upload, which should not be listed in the modelzoo at all. (Neither in the old system where models where stored directly on zenodo, nor in the new system where models are hosted on EBI S3 instead.)

I went through the Bioimage Zoo community resources:
https://zenodo.org/communities/bioimage-io/records?q=&l=list&p=1&s=10&sort=newest

There are plenty of such records with broken links

Yes, these are models where the upload process went wrong. As @esgomezm mentioned these are not models that are not actually available on the zoo. I don't think we can / should do much about this.
We should probably mark the bioimage-io zenodo community as deprecated in our docs.


interp: add a function to compile AST (instead of source)

Proposal
func (interp *Interpreter) CompileAST(ast.Node) (*Program, error)

This would accept Go AST provided by the user. Currently, the two supported pathways, compile a file and compile a function body, are equivalent to passing in an *ast.File or an *ast.BlockStmt, respectively. The new function could explicitly support only a File or BlockStmt input, though the existing (*Interpreter).ast may handle arbitrary (valid) ASTs.
I wrote a relatively small patch for this. Would you accept this as a PR?
Background
I'm playing around with writing a Go-/Yaegi-based shell. I want to support string interpolation and other features foreign to Go. This would require manipulation of the AST. While I could generate source, I would much prefer to directly pass the modified AST to Yaegi.
Workarounds
Parsing the source, modifying the AST, writing it back out to a buffer, and handing that buffer to Yaegi, which would parse the source again.

Thanks for this proposal. It is ok to me, as the standard lib AST format is stable and already in use by many tools / libraries in the Go ecosystem. The patch looks good too.


Make initial buffer size configurable like stream buffer size

I need to be able to configure the player to use an initial buffer size that overrides the buffer size set by the manifest. The initial buffer size value can be set independently of the buffer size.
I propose to add the following apis on player.js:
getInitialStreamBufferSize()
setInitialStreamBufferSize(bufferSizeInSeconds)
The initial buffer size will be used for the amount of content to buffer before playback starts. Once playback has started, the buffering goal revert to use the defined buffer size.

Interesting... Can you help me understand your use case?
My understanding is that the bandwidth attribute on a Representation is the minimum bandwidth required to assure uninterrupted playback, assuming that minBufferTime has been buffered before playback begins.
So to me, it seems like overriding minBufferTime with something lower means that you might cause a stall later, while overriding with something higher will increase startup latency.  Is the goal to buffer more at startup to try to avoid rebuffers later?  Or to start faster because the content's minBufferTime is too conservative?  Or perhaps to startup faster for clients which are estimated to have significantly more bandwidth than required, such that minBufferTime can effectively be ignored?

Our use cases are within the VOD context and our goal is to reduce re-buffering. Typically we use a large buffer size of 120 seconds.
Below are the situations why being able to set the initial buffer size would be useful:

Through experimentation in live, we have found that tweaking this value has helped to reduce re-buffering during start up for VOD.

Having some way to customise this value will allow us to easily A/B test values.

We cannot edit the manifest values as they are shared with players across multiple platforms.

In the future we would like to set this buffer size based upon estimated line speed or previous playback history to reduce start-up latency without causing re-buffering events.

I've already put together a potential pull request https://goo.gl/nYhbm0 with the proposed changes.

Ah, I see.  This sounds pretty reasonable.  I would suggest a couple of changes to the commit you referenced.
shaka.media.Stream.initialBufferSizeSeconds = Math.max(initialBufferSize, shaka.media.Stream.initialBufferSizeSeconds);
If the static value is low (as in the default value), this line would cause the static value to accumulate across all playbacks in this browser session.  For example, if I were to play content A with a buffer size of 6, then content B with a buffer size of 2, I would want B to start up as fast as specified in the manifest.  With your change, though, B would also buffer 6 seconds worth of content.
I believe it would be preferable to do something like this:
this.initialBufferSize_ = Math.max(initialBufferSize, shaka.media.Stream.initialBufferSizeSeconds);
and then later:
var segmentRange = streamInfo.segmentIndex.getRangeForInterval(this.video_.currentTime, this.initialBufferSize_);
I also think getBufferingGoal_ should revert to its original state.  If the initial buffering goal is larger than 15s, we should attempt to maintain that goal throughout playback.
I would prefer it if the test 'does not hang when seeking to evicted data' did not rely on setStreamBufferSize, since that is not what is under test.  I think the original way of shimming the minBufferTime should be restored.

Sorry, this is bad timing.  The next Stream refactoring change in progress on our side is actually going to move all of the minBufferTime logic to StreamVideoSource.  I apologize for the inconvenience.

Not to worry, we're going to wait until this lands in 1.4.0 and review what changes, if any, we need to make.
Thanks.


chore(deps): update common-dev-assets digest to c505994

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
3c001d4 -> c505994

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline


Support two more arguments

Adding those arguments:
--username
--password
getpass has some problems when reading stdin.

Or maybe use input instead of getpass?

Or can you simply replace getpass with input?

The problem with getpass is that it reads input from TTY.

I think since you need to input the captcha code each time, you show run the tool manually instead of using redirection.

I think since you need to input the captcha code each time, you show run the tool manually instead of using redirection.

I use child_process to interact with the python script.
You can see my VSCode JOJ Tools, https://github.com/linsyking/vscode-joj-tools, I integrate JOJ tools.

It seems that you use a self-made joj-auth for getting JOJ_SID? Anyway, could you please create a PR for that?

It seems that you use a self-made joj-auth for getting JOJ_SID? Anyway, could you please create a PR for that?

right away

btw, i wonder if there is a jaccount_cli for nodejs... :sob:

You can just translate it to nodejs https://github.com/tc-imba/python-jaccount-cli/blob/master/jaccount_cli/asyncio.py


[Question] Why should we call stream_bucket_make_writeable inside a loop?

Every single implementation of php_user_filter I see calls the stream_bucket_make_writeable function inside a loop but in every single test I made it doesn't make any difference. The second call to stream_bucket_make_writeable always returns null in my tests.
In which case is this loop really needed?

Hi @CViniciusSDias, that's a good question!
For reference, I assume that what you're asking for deals with an implementation and nothing that is exposed to consumers of this library:
https://github.com/clue/php-stream-filter/blob/5a58cc30a8bd6a4eb8f856adf61dd3e013f53f71/src/CallbackFilter.php#L60-L65
PHP's documentation suggests the stream_bucket_make_writable() should be called in a loop to get all "buckets" from a "bucket brigade". This suggests it may return multiple "chunks" for a single write. As such, I would rephrase the question: Why should we not call it in a loop?
I agree that the underlying stream filter API is somewhat weird and that's exactly the motivation for creating this project (https://github.com/clue/php-stream-filter#why). By relying on this project's API, you shouldn't have to worry about these details and can simply rely on a single function signature as documented.
I believe this has been answered, so I'm closing this for now. Please come back with more details if you feel this needs clarification and we can reopen this :+1:

Hey, @clue ! Thanks for answering.
I won't reopen it but my question was more like: Have you ever encoutered a case where using it without the loop led to different results? Cause in every sing test I made it does't make any difference.
About using you project, I already do! I was studying further the filters API so I stumbled into this "problem". I actually recommended this package in an online course I recorded for Alura (https://alura.com.br). It's still not available but I just wanted to let you know! :-)

@CViniciusSDias Thanks for getting back at this :-)
Tbqh I don't recall whether I've been able to reproduce this, but I'd figure it doesn't make much of a difference in terms of the code base anyway, so I'd rather be safe than sorry. Also note that this project still supports legacy PHP 5.3 (and legacy HHVM to some degree), so I think sticking to what PHP's documentation suggests makes even more sense.

According to my tests, while ($bucket = stream_bucket_make_writeable($in)) {  reproduce the same as:
https://github.com/KennedyTedesco/simple-stream-filter/blob/0.1/src/CallbackFilter.php#L11-L13
The filter() function is hit more than once and the stream_bucket_make_writeable() doesn't return more than one bucket per time.
And, yeah, the documentation is a bit poor.

@KennedyTedesco What you're saying may have applied to your tests, but it's in conflict what PHP's documentation says (multiple times):

https://www.php.net/manual/en/function.stream-filter-register.php#refsect1-function.stream-filter-register-examples
https://www.php.net/manual/en/php-user-filter.filter.php#refsect1-php-user-filter.filter-parameters

in is a resource pointing to a bucket brigade which contains one or more bucket objects containing data to be filtered. [emphasis mine]

Tbqh, I'm not sure I follow what you're trying to achieve in this ticket, so let's approach this from another direction: What would be gained by removing the 3-line while loop and replacing it with a 3-line if statement?

@clue You are right. No only documentation says, but also any single test about this in the PHP repository.
I just came out with this question (same as @CViniciusSDias ) when I was studying all the possibilities. Just trying to keep an open mind here, because every single test and use case I ever had, the while() wasn't touch more than once.
But you made the right choice following what the documentation says. Other libraries use the same approach.
I'm not saying I'll cut off the while (probably not), as I said before, I'm just exploring things.
@clue Thank you so much for your work and patience to answer the issues, it's inspiring!


WiFi not turning on after flashing the custom I225-V firmware

Hello @5T33Z0,
I followed your great tutorial and flashed my I225-V with the custom firmware (I have the same motherboard, Vision G). Ethernet now works and it's great.
However my PCI-e Broadcom BCM94360CS2 card (Fenvi HB1200) no longer works properly. Turning on WiFi is not doing anything. BT still works.
I can confirm that:

I read both the tutorial and the stock vs custom firmware guides (including the ones on the german forum and insanelymac)
VT-d is enabled in BIOS
I have DisableIOMapper disabled (in Kernel - Quirks)
I do not have any DMAR or .aml files in either Add or Delete section of ACPI
I do not have the e1000=0 or dk.e1000=0 boot-args
I already reseted the NVRAM

Here is my EFI config.plist
Thanks for your guides and your help

With DisableIOMapper enabled macOS crashes after boot and using the DMAR files will make no difference to WiFi availability.
What else can I try?

Have you read this: https://github.com/5T33Z0/Gigabyte-Z490-Vision-G-Hackintosh-OpenCore/blob/main/I225_stock_vs_cstmfw.md
Which OS are ypu referring to?

I did read it, Monteray 12.3.1 (will update to 12.4) soon

Since I don't use wifi I can't help you for now. I suggest either reverting to stock firmware or asking someone in the thread at insanelymac or on reddit.

I can't check your config right now. I am on the road for a couple of days. The info in the table is based on reports made by other others of this board on the referenced insanelymac thread.

Add SBUS-MCHC.aml and try again or use my EFI folder.

I added SBUS-MCHC.aml from your latest OC release, flashed the custom fw again but the WiFi still is not turning on

I can't help you with Wifi issues.

maybe leave the issue open so others can contribute or know there are drawbacks to flashing the custom fw?

From what I understand, you are not using my EFI and Config. So how's that supposed to be an issue of my EFI?  Might as well be an issue with your config and Wifi Card

since I'm not so good at this I just copied @rursache EFI folder and I'm having the same issue, the Fenvi WiFi & Bluetooth works but on the other side ethernet won't. I don't want to take the risk of flashing, but just wanted to point that I have the same problem 😞 on the other side everything it's working as expected, so I decided to buy a PCI-Express Ethernet so I don't have to flash my board.
note: I have z490 vision G w/  I7 10700k

@Stukz this issue is about flashed boards. ethernet won't ever work without flashing the custom fw

@rursache I don't feel confident on how to do it and also how to rollback in case I want the WiFi back 😔

@Stukz

Test Wifi using my EFI folder
I don't use Wifi on this board and I won't - ever - so I can't test it.
There have been reports that Wifi works for them with the combinations of settings I explained here: https://github.com/5T33Z0/Gigabyte-Z490-Vision-G-Hackintosh-OpenCore/blob/main/I225_stock_vs_cstmfw.md
You can rollback the custom fw any time as explained in the flash guide: https://github.com/5T33Z0/Gigabyte-Z490-Vision-G-Hackintosh-OpenCore/blob/main/I225-V_FIX.md

It is not related to custom fw.
I flashed custom fw and use another efi and it worked both Ethernet (have to add e1000=0) and Wifi.
Maybe it related to aml files

@hmthang96 its weird tho, i'm using the aml files from @5T33Z0's EFI


Is there a way to map a TensorDict?

I am currently using HuggingFace Datasets to load, process, and save some data. However, HF Datasets saves the data in the Arrow format and wastes a lot of time converting between Arrow and PyTorch tensors.
I am wondering if I can use memory-mapped TensorDicts for this purpose?
How can I do a map on batches of TensorDict?
Looking through the tutorials, the nearest example I found was using a Dataloader with collate_fn as the map function:
DataLoader(
    tensor_dict, batch_size=batch_size, collate_fn=map_fn,
)

But this does not allow me to form a pipeline of map functions. I also don't know how to save and load the resulting dataset.

This is on our radar!
I like the idea of having a (possibly multiprocessed) map to execute some transform over all the elements of a tensordict.
Stay tuned, I'll ping you once we have PR with this


Memory information backwards in Linux Mint

Description
I just installed System-Bridge on a computer running Linux Mint.
My approximate memory usage is 1.7GB of 12G. However, System-Bridge has the logic backwards and thinks I'm using +98% of my memory.
What Platform / OS are you running?
Linux
What version are you running?
2.5
Anything in the logs or a references that might be useful?
No response
Additional information
No response

You'll want to report this to https://github.com/sebhildebrandt/systeminformation as this is where the information comes from

@julianrinaldi ... investigating this on our side (systeminformation) ...

You'll want to report this to https://github.com/sebhildebrandt/systeminformation as this is where the information comes from

Is it possible you're using the MemFree number instead of MemAvailable?
In this screenshot the MemAvailable is the number that's reporting the correct memory usage, which is what the System Monitor is reporting.

@timmo001 so after investigation, we see that there is no problem in the systeminformation package. The issue (that was open there) is already closed.


remove unused frontend source code files

Related to: https://github.com/opendatahub-io/odh-dashboard/issues/1620
Description
Ran knip on the following branches:

main
incubation
accelerator-support
f/byon
f/mserving-metrics
f/pipeline-enhancements
f/pipeline-experiments

The following files were not used by any branch:

src/components/FormGroupSettings.tsx

contributed by https://github.com/opendatahub-io/odh-dashboard/pull/476

src/concepts/pipelines/content/configurePipelinesServer/DatabaseConnectionInputField.tsx

contributed by https://github.com/opendatahub-io/odh-dashboard/pull/689

src/concepts/secrets/apiHooks/useSecret.ts

contributed by https://github.com/opendatahub-io/odh-dashboard/pull/1176

src/pages/projects/screens/spawner/storage/useNotebookRootStorage.ts

contributed by https://github.com/opendatahub-io/odh-dashboard/pull/777

src/pages/projects/typeHelpers.ts

empty file

src/utilities/useDebounce.ts

contributed by https://github.com/opendatahub-io/odh-dashboard/pull/1199

@andrewballantyne @lucferbux @DaoDaoNoCode let me know if any of these files are worth keeping

The following files were not used by any branch:

src/components/FormGroupSettings.tsx

contributed by Refactor user group settings #476

Hah -- seems to have been a refactor without properly cleaning up the old file. The new one seems to be frontend/src/pages/groupSettings/GroupSettings.tsx
✅ This can be deleted

src/concepts/pipelines/content/configurePipelinesServer/DatabaseConnectionInputField.tsx

contributed by [Feature] Data Science Grouping #689

Looks like this is related to useAWSSecret -- likely we had limited logic at the start and needed to improve this. This is not needed today. DS Projects (DS Grouping) was under a lot of time pressure and we did a lot of corner cutting.
✅ This can be deleted

src/concepts/secrets/apiHooks/useSecret.ts

contributed by Data Science Pipelines (DSP) #1176

Seems like this may have been a merge conflict in the DS Pipelines branch. Deleted in main, readded back by a merge conflict resolution 🤔
✅ This can be deleted

src/pages/projects/screens/spawner/storage/useNotebookRootStorage.ts

contributed by Add editing environment variables and storage features for notebook #777

Most interesting... this was likely created in a PR for managing SpawnerPage state -- but was tossed when I probably made a comment. The effort behind this is covered via other means, no need to keep it around.
✅ This can be deleted

src/pages/projects/typeHelpers.ts

empty file

Hah, impressive -- this was the original location for helpers -- which was crafted during the DS Projects work ... but it was moved to the root as it was nothing to do with Projects directly.
✅ This can be deleted

src/utilities/useDebounce.ts

contributed by Custom Model Serving Runtimes #1199

I recall this being part of the work for Summit -- I recall it being created for some reason... seems it was eventually no longer needed. @DaoDaoNoCode & @lucferbux worked on that feature and I didn't see much of it until the end.
✅ This can be deleted


Sum Fields.

Hi i am trying to sum fields but i cant do it. I downloaded your files and when i am editing fields value i define:
A1 => 1
A2 => 1
Then i try at A3 => =SUM(A1:A2) or A1+A2 or SUM(A1:A2) but nothing happens.
I need to sum every column, how can i do it ?
Would be good to have formula examples at the info section.
Thanks alot !

Hello
for me demo does not work, even not JSFiddle. Any other info of how to use it?
Thank you.

Love the app... When I do an average calculation =b2/c2 My answer is an integer.
Is there a way to get the answer to the first or second decimal place? thanks in advance..

7 years later  :-)
Sorry, I discovered today this excellent control
Change the line 1236 in jquery.handsontable-excel.js
Handsontable.NumericCell.renderer.apply(this, [ instance, td, row, col, prop, newValue, cellProperties ]);
by
Handsontable.NumericCell.renderer.apply(this, [ instance, td, row, col, prop, newValue.toFixed(2), cellProperties ]);


Upgrade minimum supported Maven version to 3.0

It will be required for running Play! in development mode with hot reloading (issue #6).

New 1.0.0-beta6-SNAPSHOT snapshot deployed.

New 1.0.0-beta6-SNAPSHOT snapshot deployed, documentation updated.


Is it possible to make resizable windows ?

I have discovered this game engine recently and it seems quite interesenting, it is the engine that I was looking for! There is no much  documentation, but the examples are very useful. Now I wonder if there is a way to make resizable windows that scale and keep content ratio since I couldn't realize how to do this. Thank you so much for your engine, btw. Keep it up!

Hi! Sorry for the very late answer.
At the moment


Update OS collection description w/ link to Garage

Scratch that, OS already links to tableland.xyz/rigs which is better landing page. It's main call to action is a link to garage... I think that works well


Can I use this with Play JsObject

I would like to use this with play default JsObject but not sure if this is possible. This is what i have been trying but getting the message:
could not find implicit value for parameter um.
val json = Unmarshal(response).to[JsObject]
Are there any default unmarshallers for JsObject and if not how could I make this my selfe?

This doesn't seem to be an issue. Please ask questions on the Gitter channel. Also notice that you have to provide the full code in question in order for others to understand possible issues.


AssertionError: DNA sequence must be a nucleotide sequence of ATGCRYSWKMBDHVN

Hello,
Thanks for the wonderful tool!
It seems to run fine on many datasets, but for me it fails for one specific dataset, with the following error report:
Activating conda environment: .snakemake/conda/1c99e828952b31ca3eef31b2ad43ef5c_
WARNING:root:Output directory results/F6D39/NanoMotif/ already exists
Traceback (most recent call last):
  File "/scratch/brbloemen/FLUPOUL/FLUPOULmeta/workflow/.snakemake/conda/1c99e828952b31ca3eef31b2ad43ef5c_/bin/nanomotif", line 10, in <module>
    sys.exit(main())
  File "/scratch/brbloemen/FLUPOUL/FLUPOULmeta/workflow/.snakemake/conda/1c99e828952b31ca3eef31b2ad43ef5c_/lib/python3.9/site-packages/nanomotif/main.py", line 514, in main
    motif_discovery(args)
  File "/scratch/brbloemen/FLUPOUL/FLUPOULmeta/workflow/.snakemake/conda/1c99e828952b31ca3eef31b2ad43ef5c_/lib/python3.9/site-packages/nanomotif/main.py", line 299, in motif_discovery
    assembly = nm.load_assembly(args.assembly)
  File "/scratch/brbloemen/FLUPOUL/FLUPOULmeta/workflow/.snakemake/conda/1c99e828952b31ca3eef31b2ad43ef5c_/lib/python3.9/site-packages/nanomotif/dataload.py", line 46, in load_assembly
    return Assembly(load_fasta(path))
  File "/scratch/brbloemen/FLUPOUL/FLUPOULmeta/workflow/.snakemake/conda/1c99e828952b31ca3eef31b2ad43ef5c_/lib/python3.9/site-packages/nanomotif/seq.py", line 13, in __init__
    self.assembly = {name:DNAsequence(seq) for name, seq in assembly.items()}
  File "/scratch/brbloemen/FLUPOUL/FLUPOULmeta/workflow/.snakemake/conda/1c99e828952b31ca3eef31b2ad43ef5c_/lib/python3.9/site-packages/nanomotif/seq.py", line 13, in <dictcomp>
    self.assembly = {name:DNAsequence(seq) for name, seq in assembly.items()}
  File "/scratch/brbloemen/FLUPOUL/FLUPOULmeta/workflow/.snakemake/conda/1c99e828952b31ca3eef31b2ad43ef5c_/lib/python3.9/site-packages/nanomotif/seq.py", line 54, in __init__
    self._check_sequence(sequence)
  File "/scratch/brbloemen/FLUPOUL/FLUPOULmeta/workflow/.snakemake/conda/1c99e828952b31ca3eef31b2ad43ef5c_/lib/python3.9/site-packages/nanomotif/seq.py", line 71, in _check_sequence
    assert all(letter in self.iupac for letter in list(set(sequence))), f"DNA sequence must be a nucleotide sequence of {''.join(self.iupac)}"
AssertionError: DNA sequence must be a nucleotide sequence of ATGCRYSWKMBDHVN

The fastq files with methylation data were created as follows with dorado v0.7.0:
<EMAIL_ADDRESS>dorado basecaller $model "$INPUT_POD5_PATH" --device cuda:all -b 320 --modified-bases 4mC_5mC 6mA > "basecalling_dor7mod5_modified/${filename}_mods.bam"

samtools fastq "basecalling_dor7mod5_modified/${filename}_mods.bam" -T '*' | pigz -p9 > "basecalling_dor7mod5_modified/${filename}_mods.fastq.gz"

For the assembly I used Flye v2.9.3, using the --meta and --nano-hq parameters.
I aligned the fastq to the assembly fasta with minimap 2.26:
 minimap2 -ax map-ont -y -t {threads} {input.assembly_fasta} {input.fastq} > {output.sam}
and filtered, sorted and indexed with samtools:
samtools view -hb -F0x904 -@{threads} {input.sam} | samtools sort -@{threads} > {output.bam}
samtools index -@{threads} {output.bam} -o {output.bai}
samtools faidx {input.assembly} -o {output.fai}

The problem is loading the contigs in the assembly file. It seems there are characters that are not on the IUPAC form in a contig sequence. If you could check if there are characters that are not one of ATGCRYSWKMBDHVN

It looks like the issue was indeed caused by one of the input files.


File MoUSE/src/mouse/denoising/denoising.py needs to be checked

Needs to be checked the performance of the gaussian_kernel1d in the file MoUSE/src/mouse/denoising/denoising.py

we do not use now the import shown below:
"from scipy.ndimage.filters import _gaussian_kernel1d"
but it is necessary, to check if it works correctly now


handle deletes of gateway, httproute if gatewayclass is deleted

Issue #, if available: #115
Tested and this resolves the reproduce steps in the issue above.
Description of changes:
This is much simpler change than it appears, due to removing a nested if/else
This changes the logic of 2 controllers: Gateway and HTTPRoute very simply.

During a reconcile, the resource is considered in scope if it has the VPC Lattice finalizer on it

Before, the reconcile logic would skip any Gateway or HTTPRoute if there wasn't a matching GatewayClass. While this makes sense for all creates, it doesn't make sense for a delete when we know it's relevant because the controller put a finalizer on it there is a VPC Lattice resources created.
The change in controllers/httproute_controller.go is very straightforward.
The logic change is the same for controllers/gateway_controller.go but there was nested if/else clauses
The reconcile function had a nested if/else:
if gwClass.Spec.ControllerName == config.LatticeGatewayControllerName {

	if !gw.DeletionTimestamp.IsZero() {

The gwClass.Spec.ControllerName == config.LatticeGatewayControllerName logic was moved to a new function isGatewayRelevant that makes it clear the conditions that decide whether to reconcile or ignore a Gateway.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I am closing this for now.  We can re-open for discussion in future if needed.


⚠ Cioos National CKAN has degraded performance

In 1f3049c, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 14033 ms

Resolved: Cioos National CKAN performance has improved in 0665dce after 23 minutes.


Branches after SNOMED DELTA update

Hi,
I currently have a branching model of:

MAIN is SNOMED INT (January 2019)
MAIN/SNOMEDCT-UK is the UK edition extension
MAIN/SNOMEDCT-UK/BABYLON-UK is the Babylon extension to the UK edition

After updating SNOMED INT using the latest July 2019 delta using the /snomed-ct/v3/imports endpoint MAIN has the latest content in it.
To make the latest content available to the other branches presumably the best way would be to rebase them using the /snomed-ct/v3/merges. If I rebase MAIN/SNOMEDCT-UK will MAIN/SNOMEDCT-UK/BABYLON-UK also be rebased? What's the best way of doing it?

Hi @joshlk,
Yes, that branching model looks good. You can even depend on specific versions of SNOMED CT if you want and if you have the full history. A version (or tag) is represented as an empty branch basically.
If you rebase a branch, only that branch will be rebased to the new base. Child branches remain there until you rebase them on their parent.
So to get the new July release content to those two child branches of MAIN, you need to execute two rebase operations:

POST /snomed-ct/v3/merges source: MAIN target: MAIN/SNOMEDCT-UK
POST /snomed-ct/v3/merges source: MAIN/SNOMEDCT-UK target: MAIN/BABYLON-UK

I hope this helps.
Cheers,
Mark


Feature/ted 635

Codecov Report

Merging #219 (e1ff21a) into main (51b9cf8) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #219      +/-   ##
==========================================
- Coverage   98.70%   98.68%   -0.03%
==========================================
  Files         115      111       -4
  Lines        5030     5084      +54
==========================================
+ Hits         4965     5017      +52
- Misses         65       67       +2

Impacted Files
Coverage Δ

...ices/conceptual_mapping_generate_sparql_queries.py
100.00% <ø> (ø)

.../notice_transformer/services/notice_transformer.py
100.00% <ø> (ø)

ted_sws/__init__.py
92.91% <100.00%> (+0.23%)
:arrow_up:

ted_sws/core/adapters/cmd_runner.py
100.00% <100.00%> (ø)

ted_sws/core/model/manifestation.py
100.00% <100.00%> (ø)

ted_sws/core/model/supra_notice.py
100.00% <100.00%> (ø)

ted_sws/core/model/transform.py
100.00% <100.00%> (ø)

.../data_manager/adapters/mapping_suite_repository.py
100.00% <100.00%> (ø)

ted_sws/data_manager/adapters/notice_repository.py
96.96% <100.00%> (+0.01%)
:arrow_up:

ted_sws/data_manager/adapters/sparql_endpoint.py
96.55% <100.00%> (-2.26%)
:arrow_down:

... and 41 more

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


Honor sort_as with multi_select_custom_func

This could be undesirable to anyone depending on it working as it did, but it doesn't seem like it's documented to ignore sorting when using multi_select_custom_func.
There could be other places that need this, but this change seemed to work for my usecase at least.

Thanks

Thanks to you too for making yadcf :)


Add a bsp_peripherals! macro

Summary
Taking inspiration from bsp_pins!, this macro abstracts over the names
of peripherals within the Peripherals struct. BSP authors can use the
bsp_peripherals! macro to define the aliases, while users can use the
periph_alias! macro to refer to the aliased peripherals.
Update the feather_m0 to use this new approach.
Also fix a bug in bsp_pins! discovered while developing the new macro.
Checklist

[x] CHANGELOG.md for the BSP or HAL updated
[x] All new or modified code is well documented, especially public items
[x] No new warnings or clippy suggestions have been introduced (see CI or check locally)

I think abstracting over the clocks is the last remaining barrier to making fully generic examples. That will be made easier once clock::v2 lands.

Pinging anyone following this PR. It should be good to go now. Check out the uses in feather_m0.

Whoops, forgot to update the HAL changelog as well

Fixed

Love it


Error pip install pyopenjtalk

oi estou tentando instalar o pyopenjtalk para poder excutar o codigo, mas está aparecedo esse erro
`(venv) PS D:\python\tts-teste\tacotron2-kaggle> pip install pyopenjtalk
Collecting pyopenjtalk
Using cached pyopenjtalk-0.3.0.tar.gz (1.5 MB)
Installing build dependencies ... done
Getting requirements to build wheel ... error
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> [25 lines of output]
setup.py:26: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
_CYTHON_INSTALLED = ver >= LooseVersion(min_cython_ver)
Traceback (most recent call last):
File "D:\python\tts-teste\tacotron2-kaggle\venv\lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 353, in
main()
File "D:\python\tts-teste\tacotron2-kaggle\venv\lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 335, in main
json_out['return_val'] = hook(**hook_input['kwargs'])
File "D:\python\tts-teste\tacotron2-kaggle\venv\lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 118, in get_requires_for_build_wheel
return hook(config_settings)
File "C:\Users\felip\AppData\Local\Temp\pip-build-env-q90m0xd4\overlay\Lib\site-packages\setuptools\build_meta.py", line 162, in get_requires_for_build_wheel
return self._get_build_requires(
File "C:\Users\felip\AppData\Local\Temp\pip-build-env-q90m0xd4\overlay\Lib\site-packages\setuptools\build_meta.py", line 143, in _get_build_requires
self.run_setup()
File "C:\Users\felip\AppData\Local\Temp\pip-build-env-q90m0xd4\overlay\Lib\site-packages\setuptools\build_meta.py", line 267, in run_setup
super(_BuildMetaLegacyBackend,
File "C:\Users\felip\AppData\Local\Temp\pip-build-env-q90m0xd4\overlay\Lib\site-packages\setuptools\build_meta.py", line 158, in run_setup
exec(compile(code, file, 'exec'), locals())
File "setup.py", line 153, in
File "C:\Users\felip\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 501, in run
with Popen(*popenargs, **kwargs) as process:
File "C:\Users\felip\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 969, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "C:\Users\felip\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 1438, in _execute_child
hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] O sistema não pode encontrar o arquivo especificado
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
`

That's an error coming from pyopenjtalk's setup.py file; pyproject_hooks is just part of the machinery that runs that.
It looks like it's trying to run some non-Python command which isn't found on your system.
See e.g. https://github.com/r9y9/pyopenjtalk/issues/32


google play warning about vulnerable openssl version

I got an email from google play
"The vulnerabilities were addressed in OpenSSL versions beginning with 1.0.1h, 1.0.0m, and 0.9.8za. To confirm your OpenSSL version, you can do a grep via ("$ unzip -p YourApp.apk | strings | grep "OpenSSL""). For more information about the vulnerability, please consult http://www.openssl.org/news/secadv_20140605.txt."
I found a bash script from stackoverflow to help narrow down where the problem is.  Running it on my project, I get the following output.
Found OpenSSL versions
ssl_work_dir/lib/armeabi/libopentok.so:OpenSSL 1.0.1e 11 Feb 2013
ssl_work_dir/lib/x86/libopentok.so:OpenSSL 1.0.1e 11 Feb 2013
I installed the most recent version via git (com.tokbox.cordova.opentok 1.0.2)
Is there anyway to upgrade openssl that appears to be bundled into opentok?  They didn't give me a timeline, so I don't believe this is urgent, but also not certain.

Should be fixed from version 2.4.1


chore(deps): update sidecar versions

chore(deps): update sidecar versions
chore(deps): update minimum K8s version to 1.17

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#326  👈
#323
#322
#320
#319
#318
#317
#316
#314
#313
#312
#311
#315
#325
#299
#310
#324
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sergeyberezansky and the rest of your teammates on  Graphite


Allow to use slug instead of id in game directory name

As title says using just id for game directory stinks and it's very hard to find game you are looking for.
Please allow to use just slug (or if you are worried about slug not being uniqe slug-id combination).

Thanks for the feedback. I was investigating this when implementing the grpc backend.
The overhead is greater than it seems because of the repair command (which basically reconstructs an internal representation of the manifest from the directory tree and you need the game ids for that), but I haven't given up on it.
It will just require that the game id gets stored in each game directory for ease of reconstruction (but currently the tool doesn't do that so it will necessitate an adaptation).

I understand why it not here from the beginning.
From my point of view using $slug-$id would be good compromise (I have slug in directory name, app have id to work with and ignores slug if directory with this id exists).
Also this maybe and option to turn on for those who want it.
Another approach is to create header file inside game directory witch contains all information for app to work (.id file or something like that)?
For now I extracted id-slug pairs from json and renaming directories semi-automated way.
Thank You for looking into this!

I think optionally including the slug in the folder path in addition to a mandatory would be a good starting compromise that is faster to implement.
An extra complication with allowing just the slug is the following: Sometimes, they'll release a remake of a given game and give the remake the exact same title as the original game (in which case they'll rename the original by including something like "classic" in the title).
A good example is Master of Orion where the original has the slug "master_of_orion_classic" (was probably "master_of_orion" before the remake) and the newer remake has the slug "master_of_orion" (essentially taking over the slug of the original game).
When that happens, there is a slug name clash that occurs and the ordering of operations is very sensitive (the directory of the original game should be renamed before the remake is added). Admittedly, it is an edge case, but it might happen to someone and I don't like bugs.
Obviously, the solution to avoid the edge case would be to add a new category of actions just to rename directories and always execute those actions first so it is doable, it is just more overhead than it initially seems.

A good example is Master of Orion where the original has the slug "master_of_orion_classic" (was probably "master_of_orion" before the remake) and the newer remake has the slug "master_of_orion" (essentially taking over the slug of the original game).
Maybe for this situation add confirmation dialog? In case someone wants to use tool in automated way you can add switch behavior (like force download changed names / ignore and download anyway / etc).

For me it's not an issue to download whole library again (~500GB) and probably will do it some day again since i have few new games.

A good example is Master of Orion where the original has the slug "master_of_orion_classic" (was probably "master_of_orion" before the remake) and the newer remake has the slug "master_of_orion" (essentially taking over the slug of the original game).

Maybe for this situation add confirmation dialog? In case someone wants to use tool in automated way you can add switch behavior (like force download changed names / ignore and download anyway / etc).
For me it's not an issue to download whole library again (~500GB) and probably will do it some day again since i have few new games.

In terms of implementation, while not as use friendly, I very much like the idea of fixating the directory name and never changing it (I don't think slugs change that often), appending a number when there is a conflict. The complexity to manage that would be well bounded.
Honestly, I'm unsure at this point if I'll implement the above feature as a direct part of the tool or whether I'll implement it as standalone grpc storage plugin (once I've finalized the grpc storage). I'll see once I'm there what makes the most sense.
In terms of timeline, realistically there are some features I'll implement first because these are things I want for myself in the tool:

Downloading all the metadata (description, screenshots, etc) and have a nice dashboard to visualize and download your games
Support for error-resilient storages and minimization of downloads (for file integrity checks) on those storages when uploading file (mostly to minimise egress network costs on cloud solutions)
Depending on the work required, I might also look at leveraging the Galaxy api, because I find the api the browser uses increasingly flaky

After the above, in terms of priority, it will be a toss between this and having a cross-platform gui complementary tool to run the commands seamlessly from a user interface (not things I desperately need for myself, but interesting technical challenges and a good final layer of polish for the tool).

I've just been looking through the source for the argument that would enable saving by slug because I was surprised it wasn't in the usage examples and would not have believed for a second that only the ID was available. Please do add this. The inability to save by slug (or $slug-$id or basically any combination other than $id alone) is actually a blocker and I'll need to look for a different tool for now.

While waiting for new version of gogcli I've created this tool:
https://github.com/jerry1333/gogcli-renamer
This is not a solution to this issue/enchantment as renaming library will make it unusable with gogcli in future but it's something :)

There are a couple of things I really want to add to the tool (and limited time to do so, so it won't be immediate):

A built-in dashboard (similar experience to the GOG library) to view and access your games
A plugable grpc storage engine
A user-interface for a more high-level experience managing backups

I would probably implement this as a storage engine in 2. However, my immediate priority is 1 and I feel it might solve the usability use-case of trying to access game directories by id for a lot of people.

There's no rush, I found a workaround so you can proceed as planned.
Thanks for a brilliant tool!


[Enhancement] assign tablets to driver sequence only if num_tablets>dop/2

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Problem Summary(Required) ：

The strategy of assigning scanRanges to each driver sequence is to avoid local shuffle.
However, if the number of buckets assigned to a fragment instance is less than pipelineDop, pipelineDop will be set to num_buckets, which will reduce the degree of operator parallelism.
Therefore, when there are few buckets, insert local shuffle instead of using this strategy to improve the degree of parallelism.
To balance the cost of local shuffle and lower DOP, we insert local shuffle when num_tablets <=pipeline_dop/2.

If num_tablets is not very small, inserting local shuffle cannot significantly improve DOP but the cost of local shuffle may be expensive.
If num_tablets is very small,  inserting local shuffle can significantly improve DOP.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 5 / 6 (83.33%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/Coordinator.java
5
6
83.33%
[278]

@Mergifyio backport branch-2.4

run starrocks_admit_test

run starrocks_admit_test


angular_router 2: JsObject has no instance getter 'queryParameters' under Dartium

I'm upgrading toh-5, the Angular tutorial example that introduced the router, to use the angular_router 2.0.
The upgraded app runs fine under Chrome, but under Dartium the following exception is thrown:
VM44:1 EXCEPTION: Unhandled exception:
NoSuchMethodError: Class 'JsObject' has no instance getter 'queryParameters'.
Receiver: Instance of 'JsObject'
Tried calling: queryParameters
#0      Object._noSuchMethod (dart:core-patch/object_patch.dart:43)
#1      Object.noSuchMethod (dart:core-patch/object_patch.dart:47)
#2      new _RouterJs.<anonymous closure> (package:angular_router/src/router/router_coordinator.dart:86:57)
#3      JsObject._callMethodLegacy (dart:js:1282)
#4      JsObject.callMethod (dart:js:1271)
#5      _RouterJs.navigate (package:angular_router/src/router/router_coordinator.dart:102:15)
#6      RouterCoordinator.navigate (package:angular_router/src/router/router_coordinator.dart:50:14)
<asynchronous suspension>
#7      RouterImpl.navigate (package:angular_router/src/router/router_impl.dart:102:39)
#8      RouterLink.onClick (package:angular_router/src/directives/router_link_directive.dart:56:15)
#9      AppView.eventHandler1.<anonymous closure>.<anonymous closure> (package:angular/src/core/linker/app_view.dart:594:38)
#10     _rootRun (dart:async/zone.dart:1120)
#11     _ZoneDelegate.run (dart:async/zone.dart:675)
#12     NgZone._run (package:angular/src/core/zone/ng_zone.dart:175:21)
#13     _CustomZone.run (dart:async/zone.dart:1001)
#14     _CustomZone.runGuarded (dart:async/zone.dart:901)
#15     NgZone.runGuarded (package:angular/src/core/zone/ng_zone.dart:310:23)
#16     AppView.eventHandler1.<anonymous closure> (package:angular/src/core/linker/app_view.dart:594:14)
#17     DebugAppView.eventHandler1.<anonymous closure> (package:angular/src/debug/debug_app_view.dart:164:44)

I don't have the updated code out in a PR yet, but I will shortly (as soon as I'm done upgrading the component tests).
cc @kwalrath @matanlurey

Closed by https://github.com/dart-lang/angular/commit/b124d8b4810e3cb09506a1710ffb5dbb09880750


LAA File Transfer Architecture

Diagram of the overall LAA File Transfer interactions
What does this pull request do?
Provides information of end to end file transfers affecting LAA  systems interactions with 3rd parties
Why make these changes?
Capturing architecture landscape
Show me the result

Hm, I wonder if this kind of diagram would be better in https://github.com/ministryofjustice/laa-hosting-diagrams?


Talbpaul/dummy output

Pull Request Description
Fixes methods evaluateSample and collectOutput on both Dummy and ExternalModel so that they work with the new data object API.  evaluateSample returns the expected realization, and collectOutput is functionally a pass-through to the data object output collection.
Note this is being merged to a rework branch, not devel.

Good WP


Additional German translations

I added most of the missing translations and fixed some existing ones.

Thanks a lot Johannes!  Much appreciated.


[📚] Certificate Update

We've got email from Apple stating
As we announced in October, the Certification Authority (CA) for Apple Push Notification service (APNs) is changing. APNs will update the server certificates in sandbox on January 20, 2025, and in production on February 24, 2025. To continue using APNs without interruption, you’ll need to update your application’s Trust Store to include the new server certificate: SHA-2 Root : USERTrust RSA Certification Authority certificate....
To ensure a smooth transition and avoid push notification delivery failures, please make sure that both old and new server certificates are included in the Trust Store before the cut-off date for each of your application servers that connect to sandbox and production. At this time, you don’t need to update the APNs SSL provider certificates issued to you by Apple.
Do we need to do anything on our site if we are using Firebase Messaging?

Hi @petrnymsa - as far as I'm aware and according to Firebase team member, you don't have to do anything: https://groups.google.com/g/firebase-talk/c/KM9qydwykyg/m/V130qhduAQAJ?pli=1
I think this update only applies if you handle push notifications yourself. In the case, Firebase handles push notifications so it is up to them to handle certificate updates.


changes to makeEvent.tsx and added Evenet.tsx to display event details

functionality that can be attached to the calendar buttons

@KennethLeeJE8 Go ahead and merge whenever you feel like. I'm not looking to do code reviews for this hackathon. I'm just using PRs to prevent us accidentally pushing changes to main, and to get better alerts about merge conflicts.


REST autosave formData configuration support

Fixed with https://github.com/RxNT/react-jsonschema-form-manager/commit/d0f7cb4868acf008ec59b846dcadb4723954578d


Model Otter has a pitch angle that is not 0.

Hi there , professor Fossen, i have a problem that the model Otter has a itch angle that is not converged to 0，even its thrust power is se t to zero, is something went wrong or is just normal?

This is due to the payload. You can change the mass distribution to obtain zero pitch, but the vehicle usually operates with a non-zero pitch angle caused by the payload.

This is due to the payload. You can change the mass distribution to obtain zero pitch, but the vehicle usually operates with a non-zero pitch angle caused by the payload.

Thanks for replying me ,professor Fossen. But i still have another problem in this simulation: the heave angle is not converged to zero but a small constant negative value (around -0.3) ,which may mean that the vessel is going up or down, obviously it is strange, so what is the reason behind that ?

I can see that this is confusing. The idea is that changing the payload mass and location will change the steady-state heave position, roll angle, and pitch angle. I have updated the SIMotter.m and otter.m files on GitHub to make these more intuitive. In the SIMotter.m file, you can specify a non-zero payload (load condition) by:
% Load condition
mp = 25;                       % payload mass (kg), max value 45 kg
rp = [0.05 0 -0.35]';     % location of payload (m)
This will result in a non-zero steady-state heave position and pitch angle. If you remove the payload mass
mp = 0;
all the states converge to zero except the roll angle (the propeller generates a non-zero roll moment).

Thanks a lot, i will examine the new version and see the results later. Thanks again for your on-time reply.


doc(gi): improved module and package descriptions

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (4b7ab75) to head (8f80700).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #46   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           73        73
  Lines         1149      1149
=========================================
  Hits          1149      1149

Flag
Coverage Δ

gi
100.00% <ø> (ø)

optional
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

coverage: 100.0%. remained the same
when pulling 8f80700ba1d4687a8fbc3887d34773a685888bc7 on feature/gi
into 4b7ab75a077c25ea3d2c9dafef25a342baa39dd0 on main.


Symfony YAML compat

Not quoting the scalar "%locale%" starting with the "%" indicator character is deprecated since Symfony 3.1 and will throw a ParseException in 4.0.

thanks!


Use gcloud's cbt + emulator in Kokoro builds.

In the Travis (or Kokoro+Docker) builds we use the version of cbt and the bigtable emulator distributed with google-cloud-sdk.  In the Kokoro+Ubuntu build we get cbt using go get because it was "easier".
Getting the latest and greatest cbt sometimes breaks, e.g.:
    Getting cbt tool
# cd .; git clone https://go.googlesource.com/sys /tmpfs/golang/src/golang.org/x/sys
Cloning into '/tmpfs/golang/src/golang.org/x/sys'...
fatal: unable to access 'https://go.googlesource.com/sys/': The requested URL returned error: 502
package golang.org/x/sys/unix: exit status 128

But also sometimes it fails because the code is broken, or does not support an old version of the golang compiler.  I think it would be better to stick to the version of cbt included with gcloud in all our builds.

Another failure:
https://source.cloud.google.com/results/invocations/932a072f-4493-4e7c-b144-1347c1023327
    Getting cbt tool
# cd .; git clone https://go.googlesource.com/text /tmpfs/golang/src/golang.org/x/text
Cloning into '/tmpfs/golang/src/golang.org/x/text'...
fatal: unable to access 'https://go.googlesource.com/text/': The requested URL returned error: 502
package golang.org/x/text/secure/bidirule: exit status 128
package golang.org/x/text/unicode/bidi: cannot find package "golang.org/x/text/unicode/bidi" in any of:
	/usr/local/go/src/golang.org/x/text/unicode/bidi (from $GOROOT)
	/tmpfs/golang/src/golang.org/x/text/unicode/bidi (from $GOPATH)
package golang.org/x/text/unicode/norm: cannot find package "golang.org/x/text/unicode/norm" in any of:
	/usr/local/go/src/golang.org/x/text/unicode/norm (from $GOROOT)
	/tmpfs/golang/src/golang.org/x/text/unicode/norm (from $GOPATH)

[ID: 9873226] Build finished after 1019 secs, exit value: 1

And yet another failure:
https://source.cloud.google.com/results/invocations/50551f72-38c8-45ac-a124-9222246c0970
================================================================
Download dependencies for integration tests Mon Mar  4 11:14:43 PST 2019.
================================================================
    Getting roots.pem for gRPC.
    Getting cbt tool
# cd .; git clone https://go.googlesource.com/net /tmpfs/golang/src/golang.org/x/net
Cloning into '/tmpfs/golang/src/golang.org/x/net'...
fatal: git fetch-pack: expected ACK/NAK, got '?error: Access denied to IP <IP_ADDRESS>'
fatal: the remote end hung up unexpectedly
package golang.org/x/net/trace: exit status 128
package golang.org/x/net/http2: cannot find package "golang.org/x/net/http2" in any of:
	/usr/local/go/src/golang.org/x/net/http2 (from $GOROOT)
	/tmpfs/golang/src/golang.org/x/net/http2 (from $GOPATH)
package golang.org/x/net/http2/hpack: cannot find package "golang.org/x/net/http2/hpack" in any of:
	/usr/local/go/src/golang.org/x/net/http2/hpack (from $GOROOT)
	/tmpfs/golang/src/golang.org/x/net/http2/hpack (from $GOPATH)
package golang.org/x/net/context/ctxhttp: cannot find package "golang.org/x/net/context/ctxhttp" in any of:
	/usr/local/go/src/golang.org/x/net/context/ctxhttp (from $GOROOT)
	/tmpfs/golang/src/golang.org/x/net/context/ctxhttp (from $GOPATH)
package golang.org/x/net/context: cannot find package "golang.org/x/net/context" in any of:
	/usr/local/go/src/golang.org/x/net/context (from $GOROOT)
	/tmpfs/golang/src/golang.org/x/net/context (from $GOPATH)

[ID: 7012965] Build finished after 1136 secs, exit value: 1


BUildFrom Source resource build was incorrect

It didn't embed generate or embed fssrgen generated resx files.
Now it does.

Great :)


Remove ministers index page from links in roles

At the moment, we are including a link to the ministers index page in the links for every ministerial role (including past roles).
This results in a reverse link in the ministers index page content item that has become too large to work with.  The content item is not currently being used by any frontend applications.
In later work, this will be replaced with a link to the current and relevant ministers only in the index page's content item.
We are having to make this change in two steps (step one: empty the array, step two: remove the ministerial link completely), as patching the links won't remove the link if the key is not included. A later PR will remove the key completely from the presenter.
After deployment, all roles will need to be republished using the following rake task:
publishing_api:bulk_republish:document_type[Role]

Trello card
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

Everything republished now on integration and production


Rationalize the use of --wait for link/unlink

We now:

Re-purpose wait to enforce waiting for the pod being updated to be
in running state (meaning that is can be immediately used by
other tools)
Introduce wait-for-target that would do what wait does currently

Was the change discussed in an issue?
Fixes #1337
How to test changes?
When using link/unlink with the -w flag should no wait until the component has been redeployed.
If the flag is not specified, the command returns immediately.
The old -w flag is now --wait-for-target. Use it when linking to a service to have odo wait until the service has been fully provisioned.

:white_check_mark: odo build 1626 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/aaa1d37e89 by @geoand)

@geoand do we need more documentation on this, change some examples maybe?

@geoand do we need more documentation on this, change some examples maybe?

I don't really think so. The help info on the command should be good enough - plus now the wait flag is consistent with the rest of odo

@geoand do we need more documentation on this, change some examples maybe?

I don't really think so. The help info on the command should be good enough - plus now the wait flag is consistent with the rest of odo

👍 I think that help message for the flag is clear and it should be enought

:white_check_mark: odo build 1704 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/660a29e844 by @geoand)