TomlArray serialization extra comma

Hi again!  Again, keep up the great work!
When serializing a TomlArray, an extra comma is appended to the end of the array in some cases.
Expected Result: array = [ 1 ]
Actual Result: array = [ 1, ]
[TestMethod]
public void TestArrayConstruct()
{
    TomlArray tomlArray = new TomlArray();

    tomlArray.Add(new TomlInteger()
    {
        Value = 1
    });

    string expectedResult = @"array = [ 1 ]";

    var table = new TomlTable
    {
        ["array"] = tomlArray
    };

    StringBuilder sb = new StringBuilder();
    using (var sw = new StringWriter(sb))
    {
        table.ToTomlString(sw);
    }

    string actualResult = sb.ToString();

    Assert.AreEqual(expectedResult, actualResult);
}

Hey!
So-called trailing comma is part of the TOML spec, so it's perfectly fine for the serializer to generate those.
I initially left in trailing comma for the sake of saving a few lines of code. Looking back at it, I might as well make the output a bit cleaner. I'll change the behaviour ASAP.


iOS: Swiping back from original story view breaks all navigation

Open any story and either double-tap or tap the original story button. If you swipe back (not just hit the Done button) from the SafariViewController, all navigation is broken. You can't do anything after that.

iPhone or iPad?  I can't reproduce this on either.  Can you provide any more details on how to reproduce and what happens?

iPhone. I was able to reproduce it regularly. I was in a dark theme. And bingo ... you have to be in a dark theme.

It does seem more reproducible in a dark theme, though I don't think that is the cause.  A symptom of the issue is that the -[StoryPageControl viewDidAppear:] method isn't getting called.  Experiments show that if you swipe slowly (or tap Done), it works properly, but if you swipe quickly, it fails.
This appears to be a known bug in iOS 9.2.
I've tried manually calling the -viewDidAppear: method, or manually dismissing the SFSafariViewController, but nothing helped.

 This is kind of a blocker. I wonder if this happens in the shipping app.

Dunno.  I've posted to that thread, and filed a Radar (#24560676).  If I think of anything else to try, I'll give it a go.

I can't seem to reproduce this bug - everything works fine after swiping back from in-app Safari on my iPhone 6s and iPad Air with 9.2.1 and iPhone 5s with 9.3b5. Maybe I'm incapable of swiping fast enough. There is a workaround to disable swiping in the thread you linked to now, though.

It is a bit sporadic, and seems to occur more on dark themes for some reason.
I offered to implement that workaround, but Sam said that since it's an iOS bug, it should get fixed in due course.  So I think we'll live with it, and hope not too many people encounter it.


go-ceph: require a newer version of testify

The older version has some bugs fixed in newer releases. These issues
were exposed on patches not yet merged. We can take this now and
rebase those more complex patch series afterwards.
Signed-off-by: John Mulligan<EMAIL_ADDRESS>Checklist

[ ] Added tests for features and functional changes
[ ] Public functions and types are documented
[ ] Standard formatting is applied to Go code
[ ] Is this a new API? Is this new API marked PREVIEW?

@mergifyio rebase


GC has rather big impact on snapshot size

Looking at Chapter 2.1 Collaborative Meeting Notes — Status document, here is some interesting stats:

Total number of summary blobs: 3.6Mb
567Kb out of that - catchUp blobs - something we should address (see https://github.com/microsoft/FluidFramework/issues/6364)
619Kb - gc blobs! That's 20% of total content if I exclude catchUp blobs

Granted this file has a lot of deleted content, and once we run sweep on it, sizes will reduce. But I expect percentage wise GC data to stay pretty high relative to useful content.
Is there anything we can do here to reduce burden on snapshot sizes?
Are there direct ways to reduce impact?
Can and should we write all this data as an attachment blob(s), such that it is excluded from snapshot and does not impact boot times? I believe it would work correctly in a sense of summarizing client always either reading these blobs or reusing in next summary, thus never running into situation where these blobs would be deleted by storage (today that can happen with other attachment blobs, and thus we ensure that all important blobs come in summary and cached by driver forever).

It would be great to have a discussion on what we can do about it.
And once / if we have a proposal, see who can help implement it.

I am not familiar with attachment blobs but we should definitely have a discussion.

Attachment blobs have the following properties that differ them from snapshot blobs:

Each individual blob is uploaded and downloaded separately via http request.

There is general agreement that we want to add batching capabilities here in the future.

Attachment blobs are not included into snapshot payload, and thus have overall positive impact on boot, assuming their content is not needed on boot.
Because they are requested via individual request, such requests are cacheable by browser

i.e. images are loaded only once - rebooting document will result in image being provided by browser cache.

All blobs can be garbage collected by storage.

in case of SPO, it reclaims snapshots quickly, and thus any blob that was references in snapshot that given client loaded from might be deleted at some point in time.
We solve this problem for summary blobs by ensuring than all summary blobs are included in snapshot and cached by driver
Images (that use attachment blobs) could end up being gone in case client is way behind on ops and attempts to resolve an image. Code (on FF & Bohemia side) deals with it and app recovers quickly when it catches up (there is no longer such image in this doc).

@marcmasmsft, @agarwal-navin, please take a look. This is not super urgent, but I'd love us to agree on high-level design and find a way to move forward here.
Here is a proposal making it possible to reduce impact of GC blobs to boot perf / bandwidth, solved via added generic building blocks that might be useful for other scenarios. It will allow only summarizer to download GC blobs, but do so in perfrormant way:

An API that allows client to download multiple blobs in fewer network calls (ideally - one).

From client perspective, no differences RE attachment or summary blobs
Server should be in position to return only subset of blobs requested. Client would follow up with more requests on as-needed bases
We should consider using new snapshot file format as a wire protocol for these responses. That said, it can be also used a staging for new format, i.e. a way to get new path going at smaller scale, as shipping multi-blob support would be easier than shipping snapshot format.
Request format should ideally be generic, supporting (in the future) ability to ask for combination of blobs, trees (with blobs) and/or full snapshot - all possibly added in the future on as needed bases, but format being flexible enough to allow such extension in the future. We might use it in the future for partial checkout (i.e. getting only sub-tree of the snapshot on boot).

An ability to specify prioritization of blobs in a summary:

I'd make it more generic than specifying attachment vs. summary blobs. For most part I want to keep that distinction abstracted away from client, and instead tell priority of blobs to be included in snapshot. For now, we could support only two priorities (guaranteed to be included into snapshot vs. possibly included or not - server could still include it into snapshot, but client can cope if it's not).
Over time, if / when we move to partial snapshots / skeleton, that mechanism can be used to querying partial data (though there is trade off between cached response vs. smaller response).

No new API will be added to runtime to request multiple blobs. Instead drivers supporting such API could defer fetch() to a continuation (end of JS turn) and as result batch all requests that come in given JS turn into single request

Validation is required to ensure that it would help in case of fetching all of GC blobs.
We can get batching (backup strategy via usage of rate limiter, though current limit is too high (24), i.e. client will make first 24 individual request before throttling any network requests and thus creating an opportunity for driver to batch them. This technique is even less efficient in drivers using http/2.0 where limits are much higher.

Alternative solution for this particular problem:

We can change GC processes to directly upload attachment blobs instead of using summary workflow.

this would add number of calls to SPO, and thus chances of being throttled.

We can do blob aggregation to reduce number of blobs, and thus reduce the need to have multi-blob read API.

This overall does not work well with current incremental summary upload where we do our best to not summarize paths that did not change. Moving to essentially single blob will substantially increase upload bandwidth requirements, unless we use some kind of differential compression that relies on content of prior blob.

We can also do nothing, and accept the cost of GC to bandwidth and boot perf, though at 20% impact to size that's rather big area to pass by.

Multi-blob read is tracked as individual item - see #4590 - Design ODSP API for requesting multiple blobs

The proposal sounds good to me. Since #4590 is a pre-requisite for this, we should discuss with ODSP and get work started on it. I remember we discussed with Marcelo and he was on board.

We will likely not have data before start of May, and I want to be data driven. Also, if data confirms GC blobs are large, we should look into that other item first about duplicate data.
So, moving it to June for now.

We have data for GC blob sizes. I'm not sure if this data has PR #10016 fix. If it does not, then numbers will continue to shrink over time.
union Office_Fluid_FluidRuntime_*
| where Data_eventName == "fluid:telemetry:Summarizer:Running:Summarize_generate"
| where isnotnull(Data_gcTotalBlobsSize)
| extend blobs = Data_totalBlobSize - Data_gcTotalBlobsSize
| summarize count(), percentile(Data_gcTotalBlobsSize, 99)
P99 is at 248K
Average - 21K
P50 - 10K
Which is much better that what we saw before!
I'd suggest revised plan:

We need a fix for https://github.com/microsoft/FluidFramework/pull/10334. This is super important to get process to be more pay-per-play.

Once it's in and we have data (including data with above mentioned PR #10016, we will re-evaluate if we need to make any improvements here, including having multi-blob breakdown of GC data. I'd think (based on data we have) that should not be needed.

Mark GC blob as not to be returned in snapshot and run experiment with it. Including starting with our stress tests, and eventually shipping to production.

Note that TreesLatest (excluding Whiteboard)

P99 is 2Mb
average is 221K
P50 is 81K

So not including GC blob in payload is still meaningful.
Though for each TreesLatest call that would add 0.2 calls to read GC blob (as that's how many summarizer sessions we have).

I'll keep it on my plate to figure out next steps / funding.

If I got it right, PR #10016 (see commit https://github.com/microsoft/FluidFramework/commit/f2dd3756892e4f3bfd8984e2e653472958835cae) went to client_v0.59.3000
And now we have more data that points out that gcTotalBlobsSize's P99 is above 1M:

Data_runtimeVersion
count_
percentile_blobs_99
percentile_Data_gcTotalBlobsSize_99

0.59.3000
172,169
176,232
1,125,882

0.59.3002
605,387
278,560
1,578,004

0.59.2000
28,227
169,074
33,3012

If I look at GC blob in range of 1M - 1.5M, that corresponds (on average) to 1600 data stores. I.e. 1K bytes per data store.
That seems about right - I get 13.3K for a file with 4x4 table in it.
There are 71 DDSs, and thus word "outboundRoutes" contributes ~1K to size. Every ID repeats at least twice.
It compresses 7x.

Key next steps:

Understand how often we write out GC blob after recent changes: https://dev.azure.com/fluidframework/internal/_workitems/edit/609

Possible next step - breaking GC blob into more blobs (using some kind of stable partitioning) and leveraging multi-blob read to keep load perf & storage scale reasonable for summarizer client

Exclude GC blobs from snapshots: https://dev.azure.com/fluidframework/internal/_workitems/edit/418

There are couple of ways we can compress the data written into GC blob. We should probably wait for the next steps outlined above before we go this route:

Create simple IDs (integers) for each node and add a simple ID -> real ID table in the GC data. However, there may be plans to switch data store / DDS to use compressed IDs similar to the ones SharedTree uses right? If so, this step is not needed.
Use small names for properties - outboundRoutes -> o. unreferencedTimestampMs -> u, etc. This will save us quite some space.

I thought about it. We have somewhat similar problems in other places. For example, with 80 lines of code I can compress 3x Sequence payloads for Scriptor.
I'm a bit worried to implement solutions like that. I'd rather add some very trivial (lz4) compression to all blobs, and make sure it's part of protocol (i.e. storage can decompress / recompress them as it sees fit - similar to usage of base64 - one can upload binary and get back base64 or vice versa). This allows us to have zero impact (if we want to) on long term storage, and much smaller payloads.
There are asks from 3rd parties to solve similar problems for other DDSs.

Yes I agree. I wouldn't want to do it unless necessary.


Failed to initialize Logshark request processor

Hi there
Even though I have created logshark account and granted relevant permission levels to postgres data base, I am getting the error "Failed to initialize Logshark request processor!
Please check to make sure that the results database was configured correctly" Excerpt from Logshark.log file is given below.

017-06-05 11:56:21,064 (null) [1] DEBUG Logshark.CLI.Program - Logshark execution arguments: D:\Tableau\ServerLogs\NonProdServer_5-06-17.zip
2017-06-05 11:56:21,110 (null) [1] INFO Logshark.Config.LogsharkConfigReader - Loading Logshark user configuration..
2017-06-05 11:56:21,224 (null) [1] DEBUG Logshark.Connections.PostgresConnectionInfo - Attempting to open connection to Postgres database 'localhost:5432\postgres' using user account 'logshark'..
2017-06-05 11:56:21,293 (null) [1] FATAL Logshark.CLI.LogsharkCLI - Failed to initialize Logshark request processor!
Please check to make sure that the results database was configured correctly. (Exception has been thrown by the target of an invocation.)
2017-06-05 11:56:21,293 (null) [1] DEBUG Logshark.CLI.LogsharkCLI - Logshark.Exceptions.DatabaseInitializationException: Failed to initialize Logshark request processor!
Please check to make sure that the results database was configured correctly. ---> System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.InvalidOperationException: This implementation is not part of the Windows Platform FIPS validated cryptographic algorithms.
at System.Security.Cryptography.MD5CryptoServiceProvider..ctor()
--- End of inner exception stack trace ---

Any help would be greatly appreciated.
Thanks
Samantha

Hi Samantha,
FIPS Mode is a setting in Windows that governs which encryption/hashing/digital signing can be used.
Logshark itself is FIPS-compliant, but Postgres uses MD5 hashing for it's username/password authentication, which causes problems on Windows systems where FIPS mode is enabled.
I believe disabling FIPS mode on the system hosting Postgres would likely resolve your problem, but of course, you should first make sure this is acceptable from a security standpoint.  FYI -- Microsoft actually no longer recommends enabling FIPS mode.
This setting can be disabled via a local security policy; here's an easy-to-follow set of instructions.
Hope that helps!
-Justin

Thanks heaps Justin :+1:
It sure did help - now the difficult part which is making IT security to agree with this approach


Spoof resource templates from server

spoof calls to sinopiaServer to retrieve a resource template with a data structure that gets appropriate JSON from files in our static directory.
Note that the approach taken here works for our tests, works for running the webpack server, and works when we run the express server as well.
Eventually, we will want to make an HTTP request out to the sinopia server, presumably using "fetch" or "axios" and we will have to figure out how to deal with the async nature of that call while rendering components.
Closes #125

Coverage increased (+0.5%) to 1.456% when pulling d643c22f932c07696179df77409120cbabb5585f on spoofResourceTemplatesFromServer into 39c884e18e464230bfc0804ba5c0f04eb0c8dbab on master.


Modal navigation sample does not look ok on iOS

Start the app -> Modal page -> Modal page example.
It looks some part of the app is under action bar:

Fixes with tns-core-modules@next


[debian plugin] Add an update alias using apt list -u

Add an update option which also lists the available updates before updating (apt list -u).
The new alias would be alias alu='sudo $apt_pref update && sudo $apt_pref list -u && sudo $apt_pref $apt_upgr'.
This is necessary to see the exact version change of a package

Closed, because it's part of #7899


[opencv3] no absolute paths

For https://github.com/microsoft/vcpkg/issues/20469

Thanks for your PR. Could you please also apply these changes for opencv2 and opencv4?

@LilyWangLL See https://github.com/microsoft/vcpkg/pull/28180 and I will create a PR for opencv4


ObjectInspector works, Inspector doesn't

I have this basic example:
import React from 'react';
import ReactDOM from 'react-dom';
import {ObjectInspector, TableInspector} from 'react-inspector';

// or use the shorthand
import {Inspector} from 'react-inspector';

let data = {
    name: 'Davide',
    age: 23
};

ReactDOM.render(
    <div>
    <Inspector data={data}/>
</div>, document.getElementById('app'));

But I get this error:
warning.js?8a56:44Warning: React.createElement: type should not be null, undefined, boolean, or number.

But if I use
    <ObjectInspector data={data} />
 <TableInspector data={data} />

Works just fine! Is there difference between ObjectInspector  and Inspector?
Is ObjectInspector  the old https://github.com/xyc/react-object-inspector?
I'm using
    "react": "^15.1.0",
    "react-dom": "^15.1.0",
    "react-inspector": "^1.0.0"

With this boilerplate: https://github.com/survivejs/react-boilerplate
If you need I can upload the repo so you can try!

Hi @ilbonte , please use
import Inspector from 'react-inspector';

Inspector is not named yet. Will add it in named export in the next version.
Difference between ObjectInspector and Inspector is that Inspector will check whether the object is a DOM Node and use DOM Inspector instead. <ObjectInspector> functions same as the old https://github.com/xyc/react-object-inspector, except that it supports surfacing non-enumerable properties now.

Oh ok thank you! Maybe you should update the readme 😉


Preload all js bundles required for a specific route

Which @angular/* package(s) are relevant/related to the feature request?
platform-server
Description
When loading a route, the browser downloads the HTML file, then the main.js file, and only after that it loads other files that are required by the main js file, for example, a component that is lazy loaded using loadComponent in the router.
Proposed solution
I think there is an opportunity to use the fact that SSR already had to render the page, so should know which bundles are required for this route. If the bundles can be identified, then they could be preloaded (https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/rel/preload) so that the browser can download all of the js files that are required to render the page at the same.
Alternatives considered
N/A

It would be great, adding link preload tags for such script will also allow CDNs such as Cloudflare to automatically generate the Early Hints, which can further improve the performance. Ref: https://developers.cloudflare.com/pages/platform/early-hints/#2-automatic-link-header-generation.
I would say that cli should generate preload tags for the css file and the main javacript bundles as well. So that CDNs can generate Early Hints for these resources as well.

@alan-agius4 what if in case of server side, it also create HTTP 103 Early Hints while generating modulepreload links? These could be added to [ResponseInit](https://developer.mozilla.org/en-US/docs/Web/API/Response/Response#parameters) as early hints headers.

Have you done any experiments with early hints? In our case, using our custom implementation we add early hints for:
stylesheet
eager loaded images
polyfills
main
modulepreload
And it in fact it improved the CWV. Whats your observation?


Thread-leak in CRT engine when internet connection is cut midway

Background
When using CRT HTTP client to get a 100MB file from a S3 bucket using getObject AND internet disconnects midway, it was observed that thread leak warning is thrown by the thread sanitizer. Note that if thread sanitizer is unchecked in Xcode, no indicator of the leak occurs.
Below is the log. Note that number of Body chunk received logs have been cut and replaced with <SKIPPED>  as they clog up the log needlessly.
CLITool(77001,0x1fb5ebac0) malloc: nano zone abandoned due to inability to reserve vm space.
2024-05-30T16:29:44-0700 info S3Client : [Logging] Request: GET https:
Path: /100MiB.txt
Headers: Host: share-bucket-sichanyoo.s3.us-west-2.amazonaws.com,
X-Amz-Date: 20240530T232944Z,
x-amz-content-sha256: e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855,
Authorization: AWS4-HMAC-SHA256 Credential=AKIA2NVENHBZH6NQSCUD/20240530/us-west-2/s3/aws4_request, SignedHeaders=host;x-amz-content-sha256;x-amz-date, Signature=89901dd8f63a2c487d61f923197469240b1eb9585a877d5e1f883dee567521b4,
User-Agent: aws-sdk-swift/1.0 ua/2.0 api/s3#1.0 os/macos#14.4.1 lang/swift#5.10 cfg/retry-mode#legacy
Query: x-id=GetObject
2024-05-30T16:29:44-0700 info SerialExecutor : [Logging] Creating connection pool for share-bucket-sichanyoo.s3.us-west-2.amazonaws.com with max connections: 50
2024-05-30T16:29:45-0700 info CRTClientEngine : [Logging] Connection was acquired to: Optional("https://share-bucket-sichanyoo.s3.us-west-2.amazonaws.com/100MiB.txt?x-id=GetObject")
2024-05-30T16:29:45-0700 info CRTClientEngine : [Logging] Using HTTP/1.1 connection
2024-05-30T16:29:45-0700 info CRTClientEngine : [Logging] Main headers received
2024-05-30T16:29:45-0700 info CRTClientEngine : [Logging] Body chunk received
2024-05-30T16:29:45-0700 info CRTClientEngine : [Logging] Body chunk received
2024-05-30T16:29:45-0700 info S3Client : [Logging] User requested checksum validation, but the response headers did not contain any valid checksums
2024-05-30T16:29:45-0700 info CRTClientEngine : [Logging] Body chunk received
2024-05-30T16:29:45-0700 info CRTClientEngine : [Logging] Body chunk received

<SKIPPED>

2024-05-30T16:29:47-0700 info CRTClientEngine : [Logging] Body chunk received
2024-05-30T16:29:47-0700 info CRTClientEngine : [Logging] Body chunk received
2024-05-30T16:30:47-0700 info CRTClientEngine : [Logging] Request/response completed
2024-05-30T16:30:47-0700 info CRTClientEngine : [Logging] Response encountered an error: crtError(AwsCommonRuntimeKit.CRTError(code: 2073, message: "Http connection channel shut down due to failure to meet throughput minimum", name: "AWS_ERROR_HTTP_CHANNEL_THROUGHPUT_FAILURE"))

WARNING: ThreadSanitizer: thread leak (pid=77001)
Thread T6 (tid=10110039, finished) created by thread T1 at:
#0 pthread_create <null> (libclang_rt.tsan_osx_dynamic.dylib:arm64e+0x3062c)
#1 aws_thread_launch <null> (CLITool:arm64+0x1002c5f84)
#2 create_and_init_host_entry <null> (CLITool:arm64+0x10031eb00)
#3 default_resolve_host <null> (CLITool:arm64+0x10031cc9c)
#4 aws_host_resolver_resolve_host <null> (CLITool:arm64+0x10031b800)
#5 aws_client_bootstrap_new_socket_channel <null> (CLITool:arm64+0x100300970)
#6 aws_http_client_connect_internal <null> (CLITool:arm64+0x10035392c)
#7 aws_http_client_connect <null> (CLITool:arm64+0x1003549a4)
#8 s_aws_http_connection_manager_new_connection <null> (CLITool:arm64+0x10035bf94)
#9 s_aws_http_connection_manager_execute_transaction <null> (CLITool:arm64+0x1003585f4)
#10 aws_http_connection_manager_acquire_connection <null> (CLITool:arm64+0x100358bac)
#11 static AwsCommonRuntimeKit.HTTPClientConnectionManagerCallbackCore.acquireConnection(continuation: Swift.CheckedContinuation<AwsCommonRuntimeKit.HTTPClientConnection, Swift.Error>, connectionManager: AwsCommonRuntimeKit.HTTPClientConnectionManager) → () <null> (CLITool:arm64+0x100232b3c)
#12 closure #1 (Swift.CheckedContinuation<AwsCommonRuntimeKit.HTTPClientConnection, Swift.Error>) → () in AwsCommonRuntimeKit.HTTPClientConnectionManager.acquireConnection() async throws → AwsCommonRuntimeKit.HTTPClientConnection <null> (CLITool:arm64+0x10023235c)
#13 partial apply forwarder for closure #1 (Swift.CheckedContinuation<AwsCommonRuntimeKit.HTTPClientConnection, Swift.Error>) → () in AwsCommonRuntimeKit.HTTPClientConnectionManager.acquireConnection() async throws → AwsCommonRuntimeKit.HTTPClientConnection <null> (CLITool:arm64+0x1002323b4)
#14 merged closure #1 (Swift.UnsafeContinuation<A, Swift.Never>) → () in Swift.withCheckedContinuation<A>(function: Swift.String, _: (Swift.CheckedContinuation<A, Swift.Never>) → ()) async → A <null> (libswift_Concurrency.dylib:arm64e+0x63c4)

SUMMARY: ThreadSanitizer: thread leak (/Users/chanyoo/Library/Developer/Xcode/DerivedData/SDKWorkbench-fbjdenesfcxsoyawnpgfulljkaxs/Build/Products/Debug/CLITool:arm64+0x1002c5f84) in aws_thread_launch+0x340

ThreadSanitizer report breakpoint hit. Use 'thread info -s' to get extended information about the report.

ThreadSanitizer: reported 1 warnings
Program ended with exit code: 66

Closing ticket as issue hasn't resurfaced.


Data in Grid Components not show correctly

times: [ {value: 'hello' },{value: 'how' },{value: 'are' }, ,{value: 'you' },]
I show my error in 3 images. can you guys solve this problem?
Why aren't my x-axis labels "hello - how -are -you" respectively?

Hi, can you bind the grid to time and the label in the grid to value?
Haiss2 @.***> schrieb am Fr., 30. Juli 2021, 07:22:

[image: 39592]
https://user-images.githubusercontent.com/34814121/127604290-06126615-3157-4bec-be42-5bad518421b8.jpg
[image: 39594]
https://user-images.githubusercontent.com/34814121/127604292-c5934199-5bb3-4b55-b43f-e73a31a538cc.jpg
[image: image]
https://user-images.githubusercontent.com/34814121/127604327-b6c309ba-b54f-4290-9e19-435adb61d1f1.png
times: [ {value: 'hello' },{value: 'how' },{value: 'are' }, ,{value: 'you'
},]
I show my error in 3 images. can you guys solve this problem?
Why aren't my x-axis labels "hello - how -are -you" respectively?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/KlausSchaefers/vue-low-code/issues/28, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWHOQC53ZVC4IV7CWVWFWLT2IZJPANCNFSM5BHZYSQQ
.

I tried, but the labels didn't show anything.


About Tokyo24/7 dataset

Excuse me, could you please tell me where to download the Tokyo 24/7 dataset?  The download link I find on the original paper is invalid. So where you download it? Thanks

Apologies for the late reply, I somehow missed this. Did you have any luck with the dataset project page? http://www.ok.ctrl.titech.ac.jp/~torii/project/247/
\


Can't krr use the nodeport port?

My kubernetes cluster is on another host, and it seems impossible to use nodeport to access the Prometheus server address. What should I do?

Hey, @llody55
Can you run share logs with -v flag added?

@LeaveMyYard

No, I mean the verbosity flag:
python3 krr.py simple -k /root/.kube/config -p http://<IP_ADDRESS>:32456 -v

@llody55 it looks like the issue is not in a prometheus but an issue that was found in https://github.com/robusta-dev/krr/issues/91 related to HPAv2 not available
I will open a PR soon on that


[Request] Need guide using menu with vue-router

Hello,
In Breadcrumb docs, It tells how to integrate with vue-router. But Menu will use more.
How do I use vue-router with menu component?

I haven't integrated vue-router into the menu component yet. I'll add the feature recently. Thanks for your feedback.😃

Thank you for reply :) currently I using on-select for router push with parameter(activeName). It works well but It can't change active class

@ChangJoo-Park  can you show your on-select code - I'm trying to use it and I cannot get it to work.
no example in the docs either....

Yes I will :)

@xtrimf here you are.
It is repo for explain.
https://github.com/ChangJoo-Park/at-ui-navigation-menu-example
Add @on-select event
https://github.com/ChangJoo-Park/at-ui-navigation-menu-example/blob/master/src/App.vue#L3
Handle Routes
https://github.com/ChangJoo-Park/at-ui-navigation-menu-example/blob/master/src/App.vue#L16

tnx @ChangJoo-Park  !
But what if I don't want to use the menu items as links? but rather as "buttons" - each one triggers a different script?
I saw in your example that the on-select is  per at-menu and I need per at-menu-item
any Idea if that's possible?

@ChangJoo-Park @xtrimf

@ChangJoo-Park @xtrimf
Sorry man, i'm obsessed with my job recently. No time to deal with this issues.
I had integrated vue-router into the menu component. It just hasn't been released yet.
I'll release it as soon as possible. Thanks :)

#47

Thank you @koppthe


Feature request - Support for video backgrounds in themes

Hi there,
Thanks for making this project, I'm rather new to Android emulation and find this to be the best implementation of a launcher. Snappy, powerful with lots of features yet straightforward to setup and use.
I have a feature request that would make it even better to me: supporting looping video backgrounds as platform themes, in order to get beautiful animations such as these: https://youtu.be/TASSFcTNslE (which I would definitely add to the theme repository as soon as it's supported).
Cheers !

Thanks that's actually a good idea. I will think about it.

The request is conflicts with the planned 1.5 The Plugin Update (start work on it Q3. See discord discussions.)


feat: rewrite trader in typescript

I want to rewrite the trader script in typescript :)
Just a small reminder for me^^
Also:

[ ] skip retry queue for {"code":-2010,"msg":"Account has insufficient balance for requested action."}
[ ] fall back to non-margin trading if margin trading fails
[x] add time to logs

I'm now pretty much finished rewriting the trader script in typescript. But I'll surely add extensive tests and paper trading before merging this.
ping fyi @jsappme @balance80 :)


Task execution

Are there any facilities for pausing or cancelling an in-progress task?  Or is this only at the level of the workflow?  If a workflow is cancelled, what happens to the tasks it was currently running?  Are they simply abandoned?  Does the orchestrator wait for them to complete prior to cancelling?  Is it possible that the cancelled workflow could later be restarted and result in re-running the task that was abandoned when the last workflow was cancelled?   We need to understand how much control is possible at the task level, and also whether there are certain sequences of operations at the level of the workflow that can result in tasks being run more than once.
Ultimately, we need a strong understanding of whether the task can rely on the orchestration layer to ensure exactly-once execution semantics, or it needs to be written in such a way that it can handle two versions of the task executing concurrently.

@clari-akhilesh Tasks are running in remote workers outside the control of Conductor server.  When a workflow is cancelled / terminated, the running tasks continue to execute (except as described in cases below).  Once the task completes and updates the status to Conductor, conductor treats that as NOP.
If your tasks are running in remote workers with a way to terminate them (e.g. containers) you can implement a WorkflowSystemTask and execute them on a remote/local server, in that case the cancel method implementation can be used to terminate the running instances of the tasks.
We use this mechanism to start and manage the remote tasks running in containers. start method spawns the container and starts the task execution, but if the workflow is cancelled, the cancel method talks to the container orchestrator to terminate the container and stop the task progress.
Hope this helps.

NOP is NoOp, I presume.
If it's not too much to ask, could you give an example of cancelling tasks using containers? Perhaps, add it in the documentation? Thanks!

@clari-akhilesh we work with container as a service infrastructure so scheduling is an HTTP call and cancelling is another HTTP call to cancel the running container.  the implementation will vary based on the orchestration service you use for containers.
In essence, it is an extension of HTTP task where the start method spins up a new container and cancel method implementation makes another call to the infra to terminate the container.


about config in per_sample_gradient_clip

Hi, my code reports a bug when I use optimizer.virtual_step() it says module 'config' has no attribute 'Z'. I dont know how to do, help me please!!

You need to import config library and set config.Z=something in your code.
E.g. https://github.com/woodyx218/opacus_global_clipping/blob/master/examples/mnist.py.
This is a step to set global environment.


Multiple collations in spellcheck

(moved from https://code.google.com/p/solrnet/issues/detail?id=211 )
SolrNet does not currently support multiple collations. SpellCheckResponseParser and SpellCheckResults should be modified to support this.
To avoid multiple breaking changes, things should be changed to support extended results as well ( http://wiki.apache.org/solr/SpellCheckComponent#spellcheck.collateExtendedResults ).
New objects encapsulating this data should be immutable.
The patch/pull request must include tests for the cases of single collation, multiple simple collation, and multiple extended collations.
There was a pull request for this some time ago, but it lacked tests and objects were not immutable: https://github.com/mausch/SolrNet/pull/24 , still it could be used as reference.

There's also another issue with a breaking change to do with collations in Solr 5.x.
See "6. The spellcheck response format has changed..." in "Upgrading from Solr 4.x" on http://lucene.apache.org/solr/5_0_0/changes/Changes.html.
I couldn't actually find anything about Solr 5 in SolrNet, so not sure if this should be raised as a separate issue or tackled as part of #110.

See https://github.com/mausch/SolrNet/pull/194

Released 0.5.1


Update ASUS component

There has been an alternative solution developed to the ASUS WRT component using the web interface of the router  https://community.home-assistant.io/t/alternate-device-tracker-component-for-asuswrt/979 . But due to changes in HASS it got broken. Maybe someone with deeper knowledge of python can take a look and integrate the code into HASS?

https://community.home-assistant.io/t/alternate-device-tracker-component-for-asuswrt/979 for anyone that wants the link without having to copy/paste it

Not integrated yet


Carousel scrolling interval on mobile

On mobile the carousel only seems to scroll half a slide instead of a full slide:

Only happens when the window gets resized, so should only happen in edge-cases on real devices. Would be nice to have this fixed, but it's really not important.


Removing guava dependency from the google_sign_in plugin

Removes the java guava dependency.
Reasons described here: https://github.com/flutter/flutter/issues/13991
Please only consider merging, when The issue above has been answered and the getToken method is required.

I'll try to fix these, in the next couple of days.

@andretietz I'm going to close this PR since you're not waiting for review right now, but please don't hesitate to reopen it, or submit a new one, once you are ready for further review. Thanks!
If you don't want to work further on this PR that's fine too, I've tagged it so that when we have some free time we can fix whatever remains to be fixed and land it for you.

@Hixie @mravn-google just adding handful of google libraries (play services, firebase, maps & google auth), I exceeded dex 64k limit. greatly appreciate your help on it. see also https://github.com/flutter/flutter/issues/41296 thanks


formula_installer: update syntax of maximum macOS check

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Fix a singular reference to how a maximum macOS requirement used to be specified. (xref: Homebrew/brew#662)

Review period ended.


Could withCount performance be improved?

Laravel Version: 5.6.2
PHP Version: 7.2
Database Driver & Version: MySQL 5.7.23-0ubuntu<IP_ADDRESS>

Description:
Currently withCount produces the following query:
select
	`artists`.*,
	(
		select count(*)
		from `songs`
		where `artists`.`id` = `songs`.`artist_id`
	) as `songs_count`
from `artists`

But this query seems pretty slow. For example, on our modest dataset of 332 artists and 1287 songs this query takes on average about 0.75 sceonds.
I tried different queries that produce the same result and these run in 0.05-0.1 seconds. Here are the queries:
select
	artists.*,
	count(*) as songs_count
from `songs`
	left join artists
		on artists.id = artist_id
group by artist_id

select
	`artists`.*,
    songs_count
from artists
left join
	(select count(*) as songs_count, artist_id
     from `songs`
     group by artist_id
    ) as `t`
    on t.artist_id = artists.id

Could any of these queries be implemented instead of the current or are there some catches as in #18109 ? It seems that soft deletion could still be honored in these queries.

The big advantage of the current subquery solution is the simple integration. Adding a JOIN clause is more complicated because you have to adjust the selected columns. By default, Laravel queries select all columns:
User::all(); // select * from "users"

Joining another table will override columns of the main table (https://github.com/laravel/ideas/issues/347).

Instead of looking at sql queries, have you tried looking at the query plan? Just do EXPLAIN FORMAT=JSON SELECT ... and you'll see details about the query, like which tables and indexes will be used, and how they will be used.
What you described matches a simple case of "the first queries loaded the data from slow disks into fast memory, the second queries loaded the data from faster memory".
Also keep in mind that you changed a simple query ( SELECT a.*, ... FROM a ) into something more complex by adding the GROUP BY clause. Your example assumes that you can use the functional dependence that mysql introduced in 5.7.5. So your example will not work on mysql <5.7.5. I havn't looked into how this works if you disable strict mode, but using mysql strict mode is basically telling it that you never care about your queries returning correct data anyway...

Hi there,
Welcome to Laravel and we are glad to have you as part of the community.
Unfortunately this GitHub area is not for ideas, suggestions etc. This is only for issues/bugs with the framework code itself.
I will be closing your ticket here. You are able to open a ticket at https://github.com/laravel/ideas
Alternatively you are able to open a PR using the Contributions guidelines: https://laravel.com/docs/5.7/contributions#which-branch
If you feel I've closed this issue in error, please provide more information about how this is a framework issue, and I'll reopen the ticket.
Thanks in advance.


v1.1

Some of features would be really awesome to have in the next release 🚀:

[ ] Parser: Handle single-quoted strings.
[ ] Parser: Other improvements to make sure parser's first step always return correct number of tokens.
[ ] Parser: Handle advanced syntax of options, (can use yargs or similar library to parse each option token).
[ ] Parser: Handle arguments with isOptional, isPersistent, isRepeatable, etc, differently.
[ ] Performance: Install Fig's completion spec locally and static render first result → avoid (many) network calls client-side.
[ ] SEO: Generate static sitemaps for all commands defined in spec → increase SEO, potentially make this the best place to search & learn shell command 🔥.
[ ] Socials: Generate OG image dynamically for command being queried.

Bumping the Parser: Handle advanced syntax !
Wanted to try calssic rm -rf / to look for easter eggs and i found out that -rf  option is unknown.
Morover, the error stops parsing becouse rm -rf -v  only shows the error insted of (at least) describing the -v option.

same issue with ls -la --sort=date

Bumping the Parser: Handle advanced syntax !
Wanted to try classic rm -rf / to look for easter eggs and i found out that -rf option is unknown.
Moreover, the error stops parsing because rm -rf -v  only shows the error instead of (at least) describing the -v option.

@Sepci0 grouped options worked now!

Can confirm ls -la works now, thanks.


Use StructArrays.jl to avoid duplicate codes

Codecov Report

Merging #14 (c9d97b7) into main (c2db98d) will increase coverage by 5.01%.
The diff coverage is 56.00%.

@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   30.06%   35.07%   +5.01%
==========================================
  Files           9       10       +1
  Lines         316      325       +9
==========================================
+ Hits           95      114      +19
+ Misses        221      211      -10

Impacted Files
Coverage Δ

src/common/CircularArraySARTTraces.jl
0.00% <0.00%> (ø)

src/common/CircularArraySLARTTraces.jl
0.00% <0.00%> (ø)

src/episodes.jl
0.00% <0.00%> (ø)

src/trajectory.jl
93.18% <ø> (+1.51%)
:arrow_up:

src/LastDimSlices.jl
8.33% <8.33%> (ø)

src/samplers.jl
62.50% <57.14%> (-37.50%)
:arrow_down:

src/traces.jl
73.97% <72.85%> (-11.75%)
:arrow_down:

src/common/sum_tree.jl
0.00% <0.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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

This might affect https://github.com/JuliaReinforcementLearning/Trajectories.jl/pull/12 .
The main change here is in the traces.jl file. I replaced the Trace with LastDimSlices. Generally speaking, each trace now can be any AbstractVector.

I think these changes are not too problematic for #12.

If I understand well the new Trace, the idea is that the container (parent) is, for example, a CircularArrayBuffer, but Trace make it look like a Vector and will return a view when fetched. I think an example in the docstring would help understand this better. I would also add unit tests that combine both because this will  probably be the main use case.

Sure, I'll add exhaustive tests to make sure the correctness for CircularArrayBuffers.

I'd do the same for Traces, MultiplexTraces and MergedTraces.

Will do.

I think MultiplexTraces is a really nice addition.

:smile:

I see the point of MergedTraces, it is like a Traces but that can accommodate different types of AbstractTraces. I think in practice, because of MultiplexTraces, it is always MergedTraces that will be used. Theferefore I'm wondering if Traces should not become what MergedTraces is at the moment.

This is worth rethinking. I'll give it a try.

There are plenty of changes in this PR, it's a bit hard to get a hang of the new design just by looking at the code. So I'll wait for this to be merged to check if #12 works well with it. But overall it seems nice. I may have new suggestions in the future after fiddling with this a bit longer.

🤗

I think a good addition right now would be unit tests this with non-scalar traces and with episodes. I think this may highlight potential caveats and also serve as a proto-documentation of examples.

OK


RSocket Client API

Changing the RSocketClient API to be more intuitive.
The only way to create clients is using RSocket class.  The clients can be created in two modes - regular-connected-client or cold-resumed-client.  The API forces the users to pass the appropriate parameters for both the constructors.  The only public method which RSocketClient exposes is resume().  client->resume() would be a warm-resumption attempt.  All the methods return futures.
RSocket::createConnectedClient(...)
      .then([&client](std::unique_ptr<RSocketClient> cl) mutable {
        client = std::move(cl);  // store a reference to the client
        client->getRequester()
            ->requestStream(Payload("BM_Stream"))
            ->subscribe(...);
      });

RSocket::createResumedClient() has a similar usage.
TODO:

Add more convenience constructors in RSocket to cover common cases.   All constructors would return Future<RSocketClient> and they would create a fresh-connected-client or a cold-resumed-client.
Resumption part in this PR is only about the API.  It will updated and tested once the server side changes are also made.
Error propagation from the ConnectionFactory to the user has to be implemented.  This would require probably a change in the ConnectionFactory::connect() method and RSocketClient::connect() method.

An alternative could be a static connect() that returns the client synchronously but also takes a callback for connection success/failure. That would work for us on mobile. However I would consider a builder API so don't have to repeat args, something like:
auto client = RSocket::clientBuilder(10 args).connect(callback);
// or
auto client = RSocket::clientBuilder(10 args).resume(callback);

This way I can write
auto builder =  RSocket::clientBuilder(10 args);
if (has_state) {
  client = builder.resume();
} else {
 client = builder.connect()
}


remove snippet tag

The snippet tags are being rendered on the docs and are not really needed
https://docs.microsoft.com/en-us/dotnet/visual-basic/programming-guide/language-features/procedures/ref-return-values
I'm wondering if the addition of the BOM marks in PR #1091 broke the logic to remove the snippet tags. Maybe if the snippet tag is in the first line, it doesn't behave as expected?

No clue. 😉


Tracker Transfer

We want to add support for visualising mapf plans. The mapf tracker (http://tracker.pathfinding.ai/) is a database with many best-known mapf plans, but the format in which the plans appear is different to the one accepted by PlanViz.
Proposal:
(1) extend PlanViz to accept mapf action model
(2) add a script/program to convert mapf tracker plans into PlanViz plans.

Implemented in
f7b99037636ab5ce709c4e0e366900873068667a

The documentation for PlanViz and Tracker Transfer tools is very long, and should be split up.
Proposal:
(1) readme.md should contain the description of the two main program in the repo and the examples of how to run them.
(2) details of instructions should be moved to seperate markdown files.
(3) the programs (plan_viz.py, tracker_transfer.py) live in the top folder of the repo, and other python files lives inside of source subfolder (src/)
(4) rename script folder to src

Documentation changes currently on branch  mapf-dev. @shchan13 you are the principal maintainer of the PlanViz repo. Any comments before we wrap this one up?

I have checked the code and the documents. I think it would be better if we merge this into the main branch after we finish the render function for generating agents without dots for the vanilla MAPF.

sounds good. it sounds like a minor change. could we get this done for
release early next week?
On Thu, 19 Oct 2023, 08:57 Shao-Hung Chan, @.***> wrote:

I have checked the code and the documents. I think it would be better if
we merge this into the main branch after we finish the render function for
generating agents without dots for the vanilla MAPF.
—
Reply to this email directly, view it on GitHub
https://github.com/MAPF-Competition/PlanViz/issues/11#issuecomment-1769428521,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWUWNFQGD27E6XQNUH554TYABGE5AVCNFSM6AAAAAA6E2DSWCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRZGQZDQNJSGE
.
You are receiving this because you commented.Message ID:
@.***>


OfficeProPlus.msi install does not complete, in some cases.

Steps taken:

Downloaded Office 365 ProPlus installation toolkit
Created MSI with embeded files (tried with 2 different MSIs) - copied to each workstation.
On some workstations, only the Microsoft Office 365 ProPlus Installer is ever installed
On other workstations the install completes successfully

Q: How can we determine what is causing this?  How can we remediate?
Thank you!

This was our testing methods causing the issue.
Closing


separate by group name for Xwing don't work anymore ?

tested 2 time this week the first round generated the guy's from the same group are paired .

created a pull request to fix this
https://github.com/Killerardvark/CryodexSource/pull/51

This should be correct in any 5.0.0+ build


Release/1.5.0

[1.5.0] - 2024-10-28
Changed

Update tutorial notebook to use PROD instead of UAT and improve readability (#241)(@danielfromearth)

Added

Expose sorting variable argument in the command line interface (#233)(@danielfromearth)

Removed

Remove the dask dependency (#235)(@danielfromearth)

Checklist:

[x] Tests all pass
[ ] Update CHANGELOG with release header

📚 Documentation preview 📚: https://stitchee--242.org.readthedocs.build/en/242/

Codecov Report
Attention: Patch coverage is 56.66667% with 13 lines in your changes missing coverage. Please review.

Project coverage is 83.64%. Comparing base (fb2ebdd) to head (70ebd6a).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

concatenator/dimension_cleanup.py
9.09%
10 Missing :warning:

concatenator/dataset_and_group_handling.py
50.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #242      +/-   ##
==========================================
- Coverage   86.27%   83.64%   -2.63%
==========================================
  Files          11       11
  Lines         641      648       +7
==========================================
- Hits          553      542      -11
- Misses         88      106      +18

Flag
Coverage Δ

integration
?

unittests
83.64% <56.66%> (-0.14%)
:arrow_down:

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

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


no cpu/usage_rate for nodes?

I see data in cpu/usage for nodes, but not for cpu/usage_rate

Can you please provide more information? What Heapster version, what deployment, where do you miss cpu/usage_rate, etc.

I'm running heapster v1.0.0 on k8s v1.1. The k8s is set up on ubuntu 14.04 machines on a cloud.
I can find result with this query:
select hostname,type,value from "cpu/usage"  where type='node' limit 40

but not this one:
select hostname,type,value from "cpu/usage_rate"  where type='node' limit 40

Ok, can you enable log sink (--sink=log)for a moment and send us a sample from the logs?

Some log is enclosed:
log.txt
It seems all metrics are present in the log

Thanks for the log. Looks like for unknown reason the rates are not generated for nodes and node system containers. One reason is that something is missing in the node metadata in cAdvisor.
What cloud are you running on?

I'm running k8s on a local IaaS provider, could the problem be caused by the version of k8s or the way I'm running the k8s on the infrastructure?

Please add -v=8 for more logging - if my suspicions are correct you should find: "Skipping rates for %s - different crate time new: xxxx  old: xxx"  there. Please paste these lines and/or all logs if possible.

Right, I did find something like:
I0323 17:54:05.088835       1 rate_calculator.go:51] Skipping rates for node:i-075twq8l - different crate time new:2016-03-22 20:06:16.842345 +0800 +0800  old:2016-03-22 20:06:16.842345 +0800 +0800

All logs are enclosed:
log1.txt

I spotted a bug in our code. We will push a new image tomorrow.

Thanks for reporting!

Sure. Thanks for fixing the problem so quick :)


chore(deps): add plugin metadata script/json...

What does this PR do?
chore(deps): add plugin metadata script/json (RHIDP-1502)
Signed-off-by: Nick Boldt<EMAIL_ADDRESS>Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
N/A (or see commit message above for issue number)
How to test this PR?
N/A
PR Checklist
As the author of this Pull Request I made sure that:

[x] The Eclipse Contributor Agreement is valid
[x] Code produced is complete
[x] Code builds without errors
[ ] Tests are covering the bugfix
[x] The repository devfile is up to date and works, if one exists
[x] Sections What issues does this PR fix or reference and How to test this PR completed
[x] Relevant user documentation updated
[x] Relevant contributing documentation updated
[x] CI/CD changes implemented, documented and communicated

Reviewers
Reviewers, please comment how you tested the PR when approving it.

Closing in favour of https://github.com/redhat-developer/rhdh-plugin-registry/pull/1/files#


Update BalenaOS to v2.48.0

Can one of the admins verify this patch?

@resin-jenkins add to whitelist


db tool missing the --datadir argument

Describe the bug
The current db tool missing the --datadir argument so we can't specify the data dir to operate on.
❯ ./op-reth db clear mdbx StoragesTrie --datadir ./optimism
error: unexpected argument '--datadir' found

  tip: to pass '--datadir' as a value, use '-- --datadir'

Usage: op-reth db clear mdbx

For more information, try '--help'.

The help info is:
❯ ./op-reth  db clear mdbx -h
Deletes all database table entries

Usage: op-reth db clear mdbx [OPTIONS]

Arguments:

Options:
      --instance <INSTANCE>  Add a new instance of a node [default: 1]
  -h, --help                 Print help (see more with '--help')

Logging:
      --log.stdout.format <FORMAT>    The format to use for logs written to stdout [default: terminal] [possible values: json, log-fmt, terminal]
      --log.stdout.filter <FILTER>    The filter to use for logs written to stdout [default: ]
      --log.file.format <FORMAT>      The format to use for logs written to the log file [default: terminal] [possible values: json, log-fmt, terminal]
      --log.file.filter <FILTER>      The filter to use for logs written to the log file [default: debug]
      --log.file.directory <PATH>     The path to put log files in [default: ]
      --log.file.max-size <SIZE>      The maximum size (in MB) of one log file [default: 200]
      --log.file.max-files <COUNT>    The maximum amount of log files that will be stored. If set to 0, background file logging is disabled [default: 5]
      --log.journald                  Write logs to journald
      --log.journald.filter <FILTER>  The filter to use for logs written to journald [default: error]
      --color <COLOR>                 Sets whether or not the formatter emits ANSI terminal escape codes for colors and other text formatting [default: always] [possible values: always, auto, never]

Display:
  -v, --verbosity...  Set the minimum log level.
  -q, --quiet         Silence all log output

I am not sure that it's by design or we missed the argument during the upgrade.
Steps to reproduce

build the op-reth with make install-op
run ./op-reth  db clear mdbx -h

Node logs
No response
Platform(s)
Mac (Apple Silicon)
What version/commit are you on?
❯ ./op-reth --version
reth Version: 1.0.0
Commit SHA: 31e247086
Build Timestamp: 2024-07-03T03:21:53.041444000Z
Build Features: jemalloc,optimism
Build Profile: release
What database version are you on?
❯ ./op-reth db version
Current database version: 2
Local database version: 2
Which chain / network are you on?
optimism
What type of node are you running?
Archive (default)
What prune config do you use, if any?
No response
If you've built Reth from source, provide the full command you used
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

the syntax for this command is not as convenient as it could be because the datadir args only match on the db command not on the subcommand. this could be improved.
reth db --datadir ../optimism clear mdbx StoragesTrie

closing in favor of #9256

StoragesTrie

thanks~


Confusing split error

Prerequisites

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

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

Description
In the following file:
import Std.Data.HashMap
open Std

/--
A circuit node declaration. These are not recursive but instead contain indices into an `Env`.
-/
inductive Decl where
  /--
  A node with a constant output value.
  -/
  | const (b : Bool)
  /--
  An input node to the circuit.
  -/
  | atom (idx : Nat)
  /--
  An AIG gate with configurable input nodes and polarity. `l` and `r` are the
  input node indices while `linv` and `rinv` say whether there is an inverter on
  the left or right input.
  -/
  | gate (l r : Nat) (linv rinv : Bool)
  deriving BEq, Hashable, DecidableEq

/--
A cache that is valid with respect to some `Array Decl`.
-/
def Cache (decls : Array Decl) := HashMap Decl Nat

/--
Lookup a `decl` in a `cache`.

If this returns `some i`, `Cache.find?_poperty` can be used to demonstrate: `decls[i] = decl`.
-/
@[irreducible]
def Cache.find? (cache : Cache decls) (decl : Decl) : Option Nat :=
  match cache.val.find? decl with
  | some hit =>
    if h1:hit < decls.size then
      if decls[hit]'h1 = decl then
        some hit
      else
        none
    else
      none
  | none => none

/--
This states that all indices, found in a `Cache` that is valid with respect to some `decls`,
are within bounds of `decls`.
-/
theorem Cache.find?_bounds {decls : Array Decl} {idx : Nat} (c : Cache decls) (decl : Decl)
    (h : c.find? decl = some idx) : idx < decls.size := by
  unfold find? at h
  split at h
  . split at h
    . split at h
      . injection h
        omega
      . contradiction
    . contradiction
  . contradiction

/--
This states that if `Cache.find? decl` returns `some i`, `decls[i] = decl`, holds.
-/
theorem Cache.find?_property {decls : Array Decl} {idx : Nat} (c : Cache decls) (decl : Decl)
    (h : c.find? decl = some idx) : decls[idx]'(Cache.find?_bounds c decl h) = decl := by
  unfold find? at h
  -- HERE HERE HERE HERE <<<<
  split at h
  . split at h
    . split at h
      . injection h
        omega
      . contradiction
    . contradiction
  . contradiction

split throws a confusing error in the last theorem:
tactic 'splitMatch' failed, nested error:
'applyMatchSplitter' failed, failed to generalize target

While it would be nice if this just worked, this is non trivial in the presence of dependent types like in this theorem. I thus propose that we at least improve the UX and present a better error than this.
Context
Verifying AIG implementations in LeanSAT
Steps to Reproduce

Copy above file into live.lean-lang.org or a lake project with std dependency
Observe error in split line

Expected behavior: Better error message, ideally the split works
Actual behavior: Confusing error message about split internals
Versions
"4.7.0-rc2"
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

Assuming it can't work, what about an error message like:
'split' could not generate a suitable proof because the types in the goal are too intricate.

In particular, 'split' failed because the type of $SUBTERM is:
  $TYPE


How to

Using leaflet I would do
L.geoJSON(myPoints, {
    style: myStyle,
   onEachFeature: onEachFeature
}).addTo(mymap);

To fill it pointers on a map from a GeoJson file. The myPoints was filled with the GeoJson variable.
How to reproduce this behaviour in Vue2Leaflet?
I've pulled the json data from here:
created () {
    axios.get("http://ponnuki.me/map/json/data.json").then(response => {
      this.geojson = response;
    });

But I would assume I need something different for the
data () {
...
geojson: null,
...

Thanks!

Hi @damarusama,
Did you have a look at the example folder?
Hope this helps.
Micka

Here is an example showing your data so Vue2Leaflet is working properly.
You probably have issue while loading data from your server (HTTP vs HTTPS or CORS problem)
These issue have nothing to do with Vue2Leaflet.
Hope this helps,
Micka


Add base test suite

Summary: Begin transition to a real application by defining some unit tests.
Included changes:

tests to validate processing of input yaml/json config file
tests to validate vault client initialization and retrieval of appsre TF credentials

Note: addressing testing of terraform utility will be separate effort due to complexity.

Shouldn't you update https://github.com/app-sre/terraform-repo-executor/blob/main/pr_check.sh to execute these newly defined tests in PR check?

@cubismod this change will include vet (and indirectly the test) when build is executed. Validated by proceeding output existing in latest build:
17:12:04 #11 12.48 === RUN   TestProcessConfig
17:12:04 #11 12.48 === RUN   TestProcessConfig/valid_yaml_returns_no_error_and_actual_equals_expected
17:12:04 #11 12.48 === RUN   TestProcessConfig/valid_json_returns_no_error_and_actual_matches_expected
17:12:04 #11 12.48 === RUN   TestProcessConfig/invalid_yaml_returns_error
17:12:04 #11 12.48 --- PASS: TestProcessConfig (0.00s)
17:12:04 #11 12.48     --- PASS: TestProcessConfig/valid_yaml_returns_no_error_and_actual_equals_expected (0.00s)
17:12:04 #11 12.48     --- PASS: TestProcessConfig/valid_json_returns_no_error_and_actual_matches_expected (0.00s)
17:12:04 #11 12.48     --- PASS: TestProcessConfig/invalid_yaml_returns_error (0.00s)
17:12:04 #11 12.48 === RUN   TestInitVaultClient
17:12:04 #11 12.48 --- PASS: TestInitVaultClient (0.00s)
17:12:04 #11 12.48 === RUN   TestGetVaultTfSecret
17:12:04 #11 12.48 --- PASS: TestGetVaultTfSecret (0.00s)
17:12:04 #11 12.48 PASS

@cubismod this change will include vet (and indirectly the test) when build is executed. Validated by proceeding output existing in latest build:
17:12:04 #11 12.48 === RUN   TestProcessConfig
17:12:04 #11 12.48 === RUN   TestProcessConfig/valid_yaml_returns_no_error_and_actual_equals_expected
17:12:04 #11 12.48 === RUN   TestProcessConfig/valid_json_returns_no_error_and_actual_matches_expected
17:12:04 #11 12.48 === RUN   TestProcessConfig/invalid_yaml_returns_error
17:12:04 #11 12.48 --- PASS: TestProcessConfig (0.00s)
17:12:04 #11 12.48     --- PASS: TestProcessConfig/valid_yaml_returns_no_error_and_actual_equals_expected (0.00s)
17:12:04 #11 12.48     --- PASS: TestProcessConfig/valid_json_returns_no_error_and_actual_matches_expected (0.00s)
17:12:04 #11 12.48     --- PASS: TestProcessConfig/invalid_yaml_returns_error (0.00s)
17:12:04 #11 12.48 === RUN   TestInitVaultClient
17:12:04 #11 12.48 --- PASS: TestInitVaultClient (0.00s)
17:12:04 #11 12.48 === RUN   TestGetVaultTfSecret
17:12:04 #11 12.48 --- PASS: TestGetVaultTfSecret (0.00s)
17:12:04 #11 12.48 PASS

Gotcha I see what you're saying. I took another look at the PR check output and see the test output before build in there. LGTM


Update README.md

added blazium link and removed the "GG".

Can you post your discord name here so we can give you a role in discord

me
thereaper5548


I got this image after training ,what does the last row mean? Thankyou!!!

Hello!   I got the following picture after training, i can understand from the second to the fourth row, they are reconstruction result, side view, 2D key point prediction, right? but I'm a little confused what does the last row represent please? Looking forward to your reply!!!

The last column visualizes the annotated keypoints. Here some keypoints have not been annotated, that's why they are missing.


[WIP] Add GitActions for integration tests

What does this PR do?
What issues does this PR fix or reference?
Is your PR tested? Consider putting some instruction how to test your changes

This PR is replaced by https://github.com/devfile/integration-tests/pull/43


Document upload of RPMs to repository server

I keep forgetting this step.
Thanks to @jouvin for reminding me.

Clearly I'm not awake as this is already in #200, will leave up to @jouvin which version of the text we merge!

I prefer that #200 is merged as it contains more than this issue. But I'm happy to rephrase  this particular part according to @jrha's text. @jrha if you agree, could you open a pull request on my #200 source branch so that your changes are incorporated in it.


fix: imageMaxWidth overrides maxWidth

fixes issue #151
note that this will only work with images from the same source as the configured wordpress host

Looks great :D

This is published in<EMAIL_ADDRESS>thanks @acao!

thanks for pairing with me!


Advancedtls

What problem are we solving?
close connections
https://github.com/chrislusf/seaweedfs/issues/2802
How are we solving the problem?
credential reloading
https://github.com/chrislusf/seaweedfs/issues/2802
Checks

[ok] I have added unit tests if possible.
[ok] I will add related wiki document changes and link to this PR after merging.

Verified via make dev_tls and and works on stage layer with duration 24h for cert Issuer
kubectl -n s3 describe certificates.cert-manager.io fast-cert
  Duration:  24h0m0s
  Issuer Ref:
    Kind:        ClusterIssuer
    Name:        vault-issuer
  Renew Before:  6h0m0s
  Secret Name:   fast-cert

After this PR, is the 10 hour max connection age still needed or not related?

After this PR, is the 10 hour max connection age still needed or not related?

Stell needed, for recconnet with new cert

I am not clear exactly why the re-establishing the connection is necessary.

I am not clear exactly why the re-establishing the connection is necessary.

Let's remove max connection age - it seems to be needed more for security reasons
https://github.com/chrislusf/seaweedfs/issues/2589#issuecomment-1166467411


updates max memory in lambda settings doc

The limit on AWS is now 3008 Mb - at least I can verify that I have uploaded a function with 2048 Mb using up.
Open an issue and discuss changes before spending time on them, unless the change is trivial or an issue already exists.
Use "VERB some thing here. Closes #n" to close the relevant issue, where VERB is one of:

add
remove
change
refactor

If the change is documentation related prefix with "docs: ", as these are filtered from the changelog.
docs: add ~/.aws/config
Run dep ensure if you introduce any new import's so they're included in the ./vendor dir.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

forgot about that, thanks!


Migration to swift 4

Any thoughts on that?

@AlexGee17 Migrated swift 4 and published a new version, 1.3.8. Could you please check.


floating-ui/vue does not export offset middleware options type

The @floating-ui/vue package does not export the types for the offset middleware. You'd have to manually make them or use TS magic to get the types, it's not very ideal.

I think this is part of a bigger problem of the types in the base libraries, where if core and dom have the same type, it's not being re-exported and is only available to import from @floating-ui/core.
Only SizeOptions appears to be as it's wrapped specifically.

I see TS just added support for export type * on the latest beta release. It may come in handy

We support TS 4.1 minimum and will have to for years, so new features are kinda useless to libraries :\ basically the same as browser support minimums.
In any case, there are conflict issues that have occurred due to naming collisions, so each type needs to be individually exported to prevent problems in certain cases it seems.


session close hint process

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: #86
What is the new behavior?

Other information

Codecov Report

Merging #103 (b2a6e1f) into master (a4f6a59) will increase coverage by 0.58%.
The diff coverage is 81.44%.

@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
+ Coverage   44.19%   44.77%   +0.58%
==========================================
  Files         102      103       +1
  Lines       12373    12609     +236
==========================================
+ Hits         5468     5646     +178
- Misses       6598     6649      +51
- Partials      307      314       +7

Flag
Coverage Δ

1.13.x
26.60% <57.97%> (+0.80%)
:arrow_up:

1.14.x
26.60% <57.97%> (+0.80%)
:arrow_up:

1.15.x
26.60% <57.97%> (+0.80%)
:arrow_up:

1.16.x
26.60% <57.97%> (+0.80%)
:arrow_up:

1.17.x
44.77% <81.44%> (+0.58%)
:arrow_up:

integration
43.46% <65.50%> (+0.38%)
:arrow_up:

macOS-latest
26.60% <57.97%> (+0.80%)
:arrow_up:

ubuntu-latest
54.10% <81.44%> (+0.49%)
:arrow_up:

unit
26.60% <57.97%> (+0.80%)
:arrow_up:

windows-latest
26.60% <57.97%> (+0.80%)
:arrow_up:

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

Impacted Files
Coverage Δ

internal/meta/meta.go
57.81% <ø> (ø)

internal/table/trailer.go
50.00% <50.00%> (ø)

internal/table/client.go
84.45% <70.37%> (+0.60%)
:arrow_up:

internal/table/session.go
76.91% <83.67%> (+1.41%)
:arrow_up:

internal/table/retry.go
77.45% <100.00%> (+0.67%)
:arrow_up:

table/options/models.go
40.00% <100.00%> (+0.47%)
:arrow_up:

internal/conn/conn.go
71.82% <0.00%> (-6.35%)
:arrow_down:

internal/conn/grpc_client_stream.go
65.78% <0.00%> (+5.26%)
: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 a4f6a59...b2a6e1f. Read the comment docs.


WIP: Streamline ImplSymbolVisitors

@seut @matriv you may want to take a look at this. It isn't ready yet (although I think tests pass)
But I'm not sure if this is even an improvement or just a different way of doing the same thing.
I started this for three reasons:

I didn't fully understand the difference between ImplementationSymbolVisitor and CollectInputSymbolVisitor
Complex class hierarchy with AbstractImplementationSymbolVisitor & BaseImplementationSymbolVisitor
Not really type safe - We use generics for the docExpressions, but the context itself is not used as a generic so we lose the type safety.

Here's why I'm not sure if these changes really make sense:

Could be solved easier by renaming the existing classes and adding docs.
I'm not sure if this became simpler
This would be fixed, although there are still a few places where I had to do it unsafe.

Current coverage is 81.14% (diff: 96.75%)

Merging #4569 into master will decrease coverage by 0.07%

@@             master      #4569   diff @@
==========================================
  Files          1294       1293     -1
  Lines         44096      44084    -12
  Methods           0          0
  Messages          0          0
  Branches       7013       7004     -9
==========================================
- Hits          35818      35773    -45
- Misses         5581       5619    +38
+ Partials       2697       2692     -5

Powered by Codecov. Last update f85a3a0...9c45478

In general I like it:

I think it's much better than just renaming and adding docs imho, but adding some docs specially to the ImplementationResolver would be great ;)
imho much cleaner, not sure if simpler...
great! not sure if the leftovers are really important as they afaik use Object as a concrete type anyway or?

Regarding 3)
The leftovers are mostly system tables where we have Iterable<?> and concrete Expression implementation which are not using Object but some specific type.
E.g. sys.jobs has Iterable<JobContext>, and all expressions expect a JobContext.
If we changed the data structure to have the referenceResolver and the iterable in the same place this could be solved. But doing so would require some more work.


Clicking outside an empty text element makes it unfocusable

This is better explained by following these steps:

put your cursor on the end of a paragraph
press enter to create an empty paragraph
click just a bit above or below the empty paragraph (outside to make the caret go away, but not on another element)
you can't click on the empty paragraph anymore to put the caret back inside.

May be related to #116

Empty paragraphs are removed once they lose focus, this is the intended behaviour.

Yup I know.  Perhaps that should be the bug :)

Ah I see, it's almost like it's removed from the virtual DOM but not the actual DOM. Will take a look.

Thanks!

OK it was a bit more complex than I at first thought because we've been relying on not blurring an element through native 'blur' events for a while now. Adding back native blur support broke a number of tests and it took a while to fix them.
Also the scope of this fix is a little more far reaching than I'd hoped so if you spot anything odd directly related to it please let me know.

Thanks for the fix. Found something unwanted. Now if you have a blank text element then click on the heading tool, the element now disappears. Prior to the fix the element won't disappear :)

Yeah I think we're going to have to back this out and think about it some more¸had a feeling it might have bigger knock on effects.

I agree, I think instead the fix should be to bring force the caret back on click if it's not there.

I think this issue got closed again by accident :)

Yeah the refert automatically closed it because of the message

I think this is okay :)

I take that back. Now because of the change, you can't add a blank text element, then add an image/video since the focus goes away.
I really think the right approach would be just to force back the caret if it's gone from an empty text element, since that's the problem. The fix has ramifications :( I'll check into how that should be done. I'm rolling back my copy.

Here's a solution. I took the approach that I mentioned. To avoid any unwanted effects, I only fixed the exact problem: if the text element is empty, it cannot be clicked on again to bring back the cursor.
Text.prototype._addDOMEventListeners = function() {
	Text.__super__._addDOMEventListeners.call(this);
	return this._domElement.addEventListener( 'mousedown', (function(_this) {
		return function(ev) {
			if ( ! _this.isMounted() ) {
				return;
			}
			if ( _this.content.isWhitespace() ) {
				var selection = new ContentSelect.Range(0, 0);
				return selection.select( _this.domElement() );
			}
		};
	})(this));
};

Just found out that the previous change also messed with the history :(

Thanks @bfintal - so does this mean that the empty element is retained after focus is lost and that it can then be clicked on again to give it back focus?

@anthonyjb Yup, exactly :)

        # HACK: If the content of the element is empty and it already has focus
        # then supress the event to stop odd behaviour in FireFox.
        #
        # Anthony Blackshaw<EMAIL_ADDRESS>2016-01-30
        if @content.length() == 0 and ContentEdit.Root.get().focused() is this
            ev.preventDefault()

OK looking a the source there's a reason that we didn't implement it in this way (e.g on click before) - not a super descriptive comment there though so I'm going to have to have a play.

OK so I can confirm this hack is the reason that this bug occurs.

Ah I remember that. From what I remember though, the issue why that was implemented is similar and may actually be also solved by the above fix.

To be honest I no longer see an issue in firefox if I remove the hack - I'm think about just removing it.

Reference: #116

Thank you :) That error still occurs - so I need to resolve the original issue without the hack, figure out why firefox behaves this way.

I think as per your code forcing the selection to be 0,0 is the most likely solution btw.

So this code seems to resolve the issue, and looks reasonable neat for a hack to me:
        # HACK: If the content of the element is empty and it already has focus
        # then supress the event to stop odd behaviour in FireFox. See issue:
        # https://github.com/GetmeUK/ContentTools/issues/118
        #
        # Anthony Blackshaw<EMAIL_ADDRESS>2016-01-30
        if @content.length() == 0 and ContentEdit.Root.get().focused() is this
            ev.preventDefault()
            new ContentSelect.Range(0, 0).select(this._domElement)

+1 I think this is functionally the same. Good that it can be placed in the same hack fix :)

Grrr this still has issue in FF, this is the final hack:
    _onMouseDown: (ev) ->
        # Give the element focus
        super(ev)

        # If the user holds the mouse down for an extended period then start
        # dragging the element.
        clearTimeout(@_dragTimeout)
        @_dragTimeout = setTimeout(
            () =>
                @drag(ev.pageX, ev.pageY)
            ContentEdit.DRAG_HOLD_DURATION
            )

        # HACK: If the content of the element is empty and it already has focus
        # then supress the event to stop odd behaviour in FireFox. See issue:
        # https://github.com/GetmeUK/ContentTools/issues/118
        #
        # Anthony Blackshaw<EMAIL_ADDRESS>2016-01-30
        if @content.length() == 0 and ContentEdit.Root.get().focused() is this
            ev.preventDefault()
            if document.activeElement != this._domElement
                this._domElement.focus()
            new ContentSelect.Range(0, 0).select(this._domElement)

Think that's it fixed - thanks for your help tracking that down, still not happy with the fix as I would like to know why FF displays this behaviour but for another day.


opdreport: Adding different ids based on dump type

Change:
-Adding different ids to dumps like sbe and hbdump, so that
user can identify the dump type by looking at ids.
Confusion was, once dumps get offloaded the name changes to
SYSDUMP_1_.gz so it was hard to distinguish the dump
type.
Tested:
-Post changes can see, sbe and hbdump are getting different dump
ids.
Previous dump ids were:
hbdump_1_1638868121.gz and sbedump_1_1638868193.gz
Examples below post change:
hbdump_20000001_1638868121.gz and sbedump_30000001_1638868193.gz
Signed-off-by: Chirag Sharma<EMAIL_ADDRESS>

https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/49413


RelationId breaks entity sorting

I have 2 entities: User and Follow, which is manual many-to-many relationship table. Both tables are identified by AdminBro and there are pages for them in navigation panel. But when I try to navigate to Follow page, it request fails with 500 error.
My debugging got me to find request failing in Core AdminBro package due to sortBy being undefined, because it could not find any property, by which is could sort.
Further digging got me to Property.isSortable() which apparently return false, even if my column is just a string or number. Which in turn, I think happens because typeorm marks my 2 primary columns as reference columns and admin-bro-typeorm` does not like that.
Is there any way of forcing adapter to sort by specified column or any other fix?
https://github.com/SoftwareBrothers/admin-bro-typeorm/blob/7ff78547db5eb0c8344692c92577a973593437d7/src/Property.ts#L28

Error: undefined column was not found in the Follow entity.
    at /home/m-nny/projects/junkyard/improved-spoon/src/find-options/FindOptionsUtils.ts:122:27
    at Array.forEach (<anonymous>)
    at Function.FindOptionsUtils.applyOptionsToQueryBuilder (/home/m-nny/projects/junkyard/improved-spoon/src/find-options/FindOptionsUtils.ts:118:40)
    at Function.FindOptionsUtils.applyFindManyOptionsOrConditionsToQueryBuilder (/home/m-nny/projects/junkyard/improved-spoon/src/find-options/FindOptionsUtils.ts:71:25)
    at EntityManager.<anonymous> (/home/m-nny/projects/junkyard/improved-spoon/src/entity-manager/EntityManager.ts:678:33)
    at step (/home/m-nny/projects/junkyard/improved-spoon/node_modules/tslib/tslib.js:143:27)
    at Object.next (/home/m-nny/projects/junkyard/improved-spoon/node_modules/tslib/tslib.js:124:57)
    at /home/m-nny/projects/junkyard/improved-spoon/node_modules/tslib/tslib.js:117:75
    at new Promise (<anonymous>)
    at Object.__awaiter (/home/m-nny/projects/junkyard/improved-spoon/node_modules/tslib/tslib.js:113:16)

Here are my entities. Also I created minimal repo (here)[https://github.com/m-nny/improved-spoon]
@Entity()
export class User extends BaseEntity {
  @PrimaryGeneratedColumn()
  id: number;

  @Column()
  firstName: string;

  @Column()
  lastName: string;

  @OneToMany(() => Follow, (follow) => follow.followingUser)
  isFollowing?: Follow[];

  @OneToMany(() => Follow, (follow) => follow.followedUser)
  followedBy?: Follow[];
}

@Entity()
export class Follow extends BaseEntity {
  @PrimaryColumn()
  @RelationId((follow: Follow) => follow.followedUser)
  followedUserId: number;

  @PrimaryColumn()
  @RelationId((follow: Follow) => follow.followingUser)
  followingUserId: number;

  @ManyToOne(() => User, (user) => user.isFollowing)
  followingUser?: User;

  @ManyToOne(() => User, (user) => user.followedBy)
  followedUser?: User;
}

Does the problem still persist in the latest version of the adapter?


⚠ SolarOscp has degraded performance

In 60d5671, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 117 ms

Resolved: SolarOscp performance has improved in a8a1749 after 11 minutes.


Val ROM

Change no-kats feature name to fast-rom
Added check to prohibit fast mode in production lifecycle

Replacing with https://github.com/chipsalliance/caliptra-sw/pull/620 so Vishal M can push additional changes


Add my school to JetBrains

@sendhello This domain is blacklisted because the college allows to register an email address there even if you are not a student of the college. If you are the student of this college, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).


Do you have an example of mixing with Rust?

I am curious on Pop_os interest and attempts to complement/replace linux gui's with Iced, found a tutorial on use.
https://brianch.github.io/iced-twentyone-tutorial/01intro.html
So I am wondering as a Haxe user how easy it might be to code Iced in Haxe with your c++ target ?
Through suggesting this I hope to clarify a real world use that Haxe community may have. Iced seems a use, which is not going to compromise any private use cases which you would not be able to share.  I think coding Iced in Haxe would be something that would really excite users and bring new users, the concept would make an excellent example use... if it is at all viable and show Haxe being used outside game industry.
Having only read a book on Rust my knowledge is very limited, so I have no idea how feasible, and not being a c/c++ coder I do not think I can assist currently, feel free to close but injecting the idea in hope. I realise this would be more than a Hello Rust use case, but one that would really test your implementation choices.

I can totally do a Rust integration example! I am a huge Rust fan, so would definitely like to try that once this project is a bit more stable. However, when it comes to working with Rust frameworks... that's a bit outside the scope of this project. This project can help run Haxe code in Rust, but running Rust code in Haxe is a different story.
While calling C++ from Rust isn't that difficult, arbitrarily calling Rust structs and functions from C++ is impossible without wrapping them in #[no_mangle] and extern "C" (afaik)? So unfortunately, being able to use Rust libraries in Haxe would require wrapping the Rust to work with C++.
Not to mention, when it comes to calling existing C++ code, the official Haxe/C++ target is already just as capable as this project is when it comes to simple function/struct calls. So... even if you go through all the effort of exposing Rust in C++, it's not like this project would provide any benefit.
Sorry if this was not the answer you were hoping for >.<

mainly just curious, I am not convinced that c/c++ is for me, and feel should be using something low level that blends well with haxe. I have no commercial need apart from being considered code employable!


Face Detector Batch Input

Apply score filtering and non max suppression to all input images in the post processing layer. Currently only the first image of the input batch is returned.

Hi, how close is this feature to being finished? It would be very helpful for a project I'm working on.


[Discussion] Gnupg to docker image to allow blocking unsigned commits on pre-receive-hook

Hello everyone,
Was about to add this pre-receive hook to my repository in gitea in order to block non-signed commits. However it failed and I found out the gnupg package wasnt included in the docker build.
Now this does not come without a cost of course to install the gnupg package costs us 54MB which is quite a lot considering the whole gitea image weighs in at just shy of ~75MB.
So I mainly started this issue to discuss if it would be:

Possible to have gnupg package added to Dockerfile?
If not are there other creative solutions/alternative libraries to circumvent this "limitation"?

.. Or should I simply just manually install it and call it a day?
Kindly,
sphrak

You can just create new dockerfile based in gitea and just add gnupg package

Closing this as hopefully we will have this feature baked into Gitea. See #4249 As a workaround @JonasFranzDEV option will likely work for you.


Documents of postgrest is not up-to-date

Describe the bug
From readme of postgrest-dart:
final response = await client.from('users').select();

From readme of supabase:
final data = await supabase
    .from('countries')
    .select()
    .order('name', ascending: true);

select() and order() don't return a Future, so I think await is wrong there.
Another sample code below also looks wrong or misleading regarding error handling.
try {
  final data = await client.from('users')
    .insert([
      {'username': 'supabot', 'status': 'ONLINE'}
    ]);
} on PostgrestException catch (error, stacktrace) {
  ...
}

According to the source code of the package, insert() returns PostgrestFilterBuilder, which inherits PostgrestTransformBuilder, which inherits PostgrestBuilder implementing Future. So the result of insert() is a kind of Future.
I'm guessing the inserting operation has not yet finished when insert() ends. If it is right, the try-catch above only catches the exception that occurs during the synchronous operation of the insert() method. An exception that occurs during the asynchronous operation after that has to be caught separately, but the above code does not show it. The code does not show how to wait for the insertion to complete either.
To Reproduce
N/A
Expected behavior
All documents have correct information.

Did you change your code? select and all the other methods do return a future. They all return descendants of PostgrestBuilder, which, as you said, implement Future.
I don't understand your exceptions catching problem. If you await the query, it will make the postgrest call. Why should the exception not being caught?

@Vinzent03
Ah, I got confused while I was looking into #252 using excessively simplified code without implementing Future. I'm sorry that I misunderstood select() and insert() didn't return a future.
So the document is correct. It is another issue that I get the "Missing variable type" error if I use await for those methods in a project with implicit dynamic disabled. I'll close this issue. Thank you.


Fix broken phrase

Seems like a phrase was cut in two by mistake.

/assign @kbhawkey

Page preview:
https://deploy-preview-21024--kubernetes-io-master-staging.netlify.app/docs/tasks/run-application/horizontal-pod-autoscale/#default-behavior


Expose the ContextMenu component

Is your feature request related to a problem? Please describe.
I have a context menu event doesn't use on the React's useContext and I umsure on how to use the mantine-contextmenu library with this constraint.
Describe the solution you'd like
I would like to render the ContextMenu component manually but it's not accessible on import.
Describe alternatives you've considered
Considering copy-pasting the components for the meantime.
Thank you very much for your work!

Thank you for your kind words!
The ContextMenu, ContextMenuItem and ContextMenuDivider components are exposed starting with v.1.3.0.


Fix: reorder task

This change is

Builds ready [e6686ff]builds: chrome, firefox, opera
bundle viz: background, ui, inpage, contentscript, ui-libs, bg-libs, phishing-detect
dep viz: background
all artifacts


use more Worldcat description when no Voyager record is available

A person be viewing a title in Worldcat, e.g., http://www.worldcat.org/title/brain/oclc/34473395 could select "Check Availability at GWU or request through ILL"

even though Worldcat doesn't list any WRLC library.  Our Worldcat API handles this scenario by displaying some information from the resultset returned from OCLC. We could show more of the resultset, and provide an explanation that the results are from OCLC.
For examples try /oclc/ and one of these numbers:
609102750 34473395 34474496
Sample result: https://github.com/gwu-libraries/launchpad/wiki/sample-results-from-oclc-api

After the update the

Looks great, I think this is really useful.
Testing this for the PR I noticed that the second author link in example /oclc/34473395 doesn't return any results (http://gwdev-zut.wrlc.org:8888/search?q=Author:"Thompson, Ian, 1964- illustrator.") and in example /oclc/34474496 the only author link also returns no results.

The author link a different problem unrelated to this work. The code for
building links includes a "role" from a subfield on the MARC tag we
shouldn't make part of the link (eg., illustrator).
If you remove the illustrator from the link, it works
http://gwdev-zut.wrlc.org:8888/search?q=Author:"Thompson, Ian, 1964"
Thanks,
*Mike *
On Tue, Mar 1, 2016 at 4:44 PM, Christian Aldridge<EMAIL_ADDRESS>
wrote:

Looks great, I think this is really useful.
Testing this for the PR I noticed that the second author link in example
/oclc/34473395 doesn't return any results (
http://gwdev-zut.wrlc.org:8888/search?q=Author:"Thompson, Ian, 1964- illustrator.")
and in example /oclc/34474496 the only author link also returns no results.
—
Reply to this email directly or view it on GitHub
https://github.com/gwu-libraries/launchpad/issues/1108#issuecomment-190919693
.


Doesn't disable spellchecking on some external links

Spell checking is still on for this link:  [GitHub - alangrainger/obsidian...](https://github.com/alangrainger/obsidian-lazy-plugins):

Here are my settings:

No this does work.  Nevermind.

For future reference - sometimes the document renderer needs a refresh after toggling the settings (or installing the plugin). Usually closing the note and opening it again does the trick.
A little annoying, but haven't looked into solving it.


lockFileMaintenance clarification

Still cant get it right.
let say I will use yarn, pinVersions:true for all kinds of deps and some days passed of autoupdating and automerging. I think after every new update it should regenerate the lockfile, so when u clone the repo and run install u will get correct deps. is that option for that?
I seen today some new opened issue at renovate repo for smth like that, but how it differs from the lockfilemaintenance option?

Every time a dependency is updated, Renovate will regenerate the lock file from scratch so you will have the very latest of both direct dependencies plus sub-dependencies. Hence, if you are updating dependencies frequently then your lock file will also be likely to be fairly up to date.
Note that Renovate won't keep updating lock files in a PR constantly - they will only be updated/regenerated if the package.json is updated by another dependency version.
If it has been a while since the last Renovate PR/lockfile then your dependencies may be "out of date" (if using semver ranges) plus also sub-dependencies can always get out of date quickly, if you care. That's where "lock file maintenance" can help, because it will also regenerate the lock file even if package.json hasn't been updated. By default lock file maintenance has a schedule of "before 5am on monday" so that it disturbs at most once per week. If you left it running continuously you'd probably end up with lock file updates every hour or two and not many people would really want that.

they will only be updated/regenerated if the package.json is updated by another dependency version.

same as in the docs. I cant understand exactly that. what mean only if is updated by another, when it is touched/changed in every pr for new dep..

If you left it running continuously you'd probably end up with lock file updates every hour or two and not many people would really want that.

why not? is the commit prefix chore?

same as in the docs. I cant understand exactly that. what mean only if is updated by another, when it is touched/changed in every pr for new dep..

lock files are regenerated only if package.json changes in a branch. e.g if PR is created to upgrade to prettier v1.7.1, and then soon after prettier v1.7.2 is released then the lock file will be updated too. But if v1.7.1 PR sits open for a week without you merging it then Renovate won't keep regenerating the lock file every hour "just in case" some sub-dependency has changed.

why not?

People don't want disruptions/notifications too often. And very wasteful for the server running Renovate too, i.e. you'd need a VCPU per project or every few projects if you insist on regenerating every lock file non stop.

im fan of the "rolling release" model, and so probably thats why I keep evrything uptodate at maximum.. :d
anyway, I think Im starting to get it. and would be good to run regenerate when new update/pr is sent (probably for that is the mentioned seen issue in the other repo).

oh, I think I got it. alright, i'll keep it "on mondays" only, i's okey.


Allow For The JSON View To Remember Last Viewed Position

Is your feature request related to a problem? Please describe:
When viewing the JSON and switching back to list view, users have to again scroll to the previous position when back in JSON view.
Describe the solution you'd like
All the JSON view to remember last used position.
Screen recording:
https://github.com/tokens-studio/figma-plugin/assets/93133012/87e5668d-27d0-4ffb-beeb-74e53bc30a49

This request has been moved to our new feedback tool. Go check out https://tokensstudio.featurebase.app/ to create a free account. You can upvote and comment on existing posts, make a new request, and subscribe to be notified when requests move into our roadmap.

Added to Featurebase Roadmap : 🧑💻 JSON view/edit enhancements


UpdateMode packet - invalid size. OpenRGB 0.9.1

My hardware isn't supported in stable OpenRGB build 0.9, therefore i installed 0.9.1 (Git Commit ID 7dcdef28c86122ab4fd446a1d1fa584c59df98ca / 2024-10-20).
Homebridge-openrgb successfully connects to SDK server and recognizes accessories, however I can't control any of them.
It works flawlessly with 0.9.
I know it looks like a problem with OpenRGB, not homebridge-openrgb, but honestly, I have no clue and I want to let you know that your plugin might not be compatible with future versions of OpenRGB.
I apologize for any inaccuracies. This is my first support request on GitHub, and I'm not familiar with it. Thanks in advance for your support and understanding.
homebridge-openrgb logs:
[10/23/2024, 7:12:10 AM] [OpenRGB] Restarting child bridge...
[10/23/2024, 7:12:10 AM] Got SIGTERM, shutting down child bridge process...
[10/23/2024, 7:12:15 AM] [OpenRGB] Child bridge process ended
[10/23/2024, 7:12:15 AM] [OpenRGB] Process Ended. Code: 143, Signal: null
[10/23/2024, 7:12:22 AM] [OpenRGB] Restarting Process...
[10/23/2024, 7:12:22 AM] [OpenRGB] Launched child bridge with PID 10982
[10/23/2024, 7:12:22 AM] Registering platform 'homebridge-openrgb.OpenRgbPlatform'
[10/23/2024, 7:12:22 AM] [OpenRGB] Loaded homebridge-openrgb v1.4.3 child bridge successfully
[10/23/2024, 7:12:22 AM] Loaded 3 cached accessories from cachedAccessories.0EB657827B6C.
[10/23/2024, 7:12:22 AM] [OpenRGB] Loading accessory from cache: ASRock B450M Pro4-F
[10/23/2024, 7:12:22 AM] [OpenRGB] Loading accessory from cache: ZOTAC GAMING GeForce RTX 4070 Ti Trinity OC
[10/23/2024, 7:12:22 AM] [OpenRGB] Loading accessory from cache: Keychron Gaming Keyboard 1
[10/23/2024, 7:12:22 AM] Homebridge v1.8.4 (HAP v0.12.2) (OpenRGB) is running on port 42633.
[10/23/2024, 7:12:22 AM]

OpenRGB logs:
[Info:] NetworkServer: Network server started
[Info:] NetworkServer: Network connection thread started on port 6742
[Info:] NetworkServer: Network server started
[ERROR:] NetworkServer: recv_select failed receiving magic, closing listener
[Info:] NetworkServer: Closing server connection: <IP_ADDRESS>
[Info:] NetworkServer: Network server started
[ERROR:] NetworkServer: UpdateMode packet has invalid size. Packet size: 64, Data size: 60
[Info:] NetworkServer: Closing server connection: <IP_ADDRESS>
[Info:] NetworkServer: Network server started
[ERROR:] NetworkServer: recv_select failed receiving magic, closing listener

Plugin Config:
{
    "name": "OpenRGB",
    "servers": [
        {
            "name": "homebridge",
            "host": "xxx.local",
            "port": 6742
        }
    ],
    "discoveryInterval": 60,
    "preserveDisconnected": false,
    "suppressConnectionErrors": false,
    "_bridge": {
        "username": "0E:A6:D3:59:DD:F9",
        "port": 41702
    },
    "platform": "OpenRgbPlatform"
}

Environment:

Plugin Version: 1.4.3
Homebridge Version: 1.8.4
Node.js Version: 20.18.0
NPM Version: 10.8.2
Operating System: Ubuntu Focal (20.04.5 LTS)

Update on this issue from OpenRGB community at Discord:

It's not an OpenRGB issue and needs to be fixed in the downstream project. Essentially it needs to update the packet it's sending as it's not including the size of the packet payload.