Data type long values can be rounded

I am running the .Net Core version, 1.3.2.  I'm running VS 2017 on Windows 10 (64 bit).
I am using some entities that have values with very large long values, and it seems that these are being rounded in an odd way and not stored in CosmosDB correctly.
I created a scaled down entity for testing that looks like this:
public class TestAccount
{
    [JsonProperty(PropertyName = "id")]
    public string Id { get; set; }

    [JsonProperty(PropertyName = "idLong")]
    public long IdLong { get; set; }
}

I instantiated it and saved it like so:
TestAccount testAccount = new TestAccount();
testAccount.Id = "769308320146468865";
testAccount.IdLong =<PHONE_NUMBER>46468865;
Assert.IsTrue(testAccount.IdLong < long.MaxValue, "testAccount.Id < long.MaxValue");

await cosmosRepository.CreateDocumentAsync(testAccount);

And the persisted entity looks like this.  Note the difference in the values.
{
  "id": "769308320146468865",
  "idLong":<PHONE_NUMBER>46468900
}

I was able to duplicate this in three scenarios:
Running Release code in Azure App Service (web site set to 64 bit)
Running under 64 bit IIS express
Running under Resharper unit test runner
My active platform is set to x64 (not AnyCPU).  Can you replicate this issue, or do you have any suggestions?

It could be serious issue for us too. I guess it uses double internally because of JavaScript etc. We'll try to reproduce later.

We tried to reproduce it and found, that data is serialized and stored correctly.
The issue is only at portal, Document/Query Explorer view "rounds" them.

You're correct.  It's just a UI issue in the portal.  Do you think that this may be fixed in the future?

OK, I have something that's also interesting.
Assume we've created the same document as previously.
TestAccount testAccount = new TestAccount();
testAccount.Id = "769308320146468865";
testAccount.IdLong =<PHONE_NUMBER>46468865;

await cosmosRepository.CreateDocumentAsync(testAccount);

which will create this document:
{
  "id": "769308320146468865",
  "idLong":<PHONE_NUMBER>46468865
}

If we use a query to pull the document back by it's ID, yes we can verify that all is well and the idLong property has the expected value:
var document = await cosmosRepository.GetDocumentByIdAsync("769308320146468865");
long prop = document.GetPropertyValue<long>("idLong");
Assert.AreEqual(769308320146468865, prop);

This is probably what you did to check the persistence of the document.
Now, let's use a query to find it.  We build a simple query using either a SqlQuerySpec or using a string to pull a document collection.
SqlQuerySpec query = new SqlQuerySpec("Select * from c where c.id = @docId",
                new SqlParameterCollection{new SqlParameter("@docId", "769308320146468865") });

var documents = cosmosRepository.GetDocuments<TestAccount>(query);
Assert.AreEqual(769308320146468865, documents.First().IdLong, "SqlQuerySpec: Document Ids are not equal");

const string querytest = "select * from c where c.id = \"769308320146468865\"";
documents = cosmosRepository.GetDocuments<TestAccount>(querytest);
Assert.AreEqual(769308320146468865, documents.First().IdLong, "StringQuery: Document Ids are not equal");

In both queries above, the IdLong value of the document returned is<PHONE_NUMBER>46468864, which causes our test to fail (expected value<PHONE_NUMBER>46468865).
Just for completeness, here's the methods in the repository used above.  They are trivial.
        public List<T> GetDocuments<T>(string query)
        {
            return documentClient.CreateDocumentQuery<T>(UriFactory.CreateDocumentCollectionUri(database, collection), query)
                .AsEnumerable().ToList();
        }

        public List<T> GetDocuments<T>(SqlQuerySpec query)
        {
            return documentClient.CreateDocumentQuery<T>(UriFactory.CreateDocumentCollectionUri(database, collection), query)
                .AsEnumerable().ToList();
        }

Can you see similar results?  These are done in the Resharper unit test runner, and I haven't tested the exact test in other ways (IISExpress), but it matches what I've been seeing.

This appears to be the same issue as #293.


Implement a "deferred" renderer

Prereqs:

[x] https://github.com/ppy/osu-framework/pull/6189
[x] https://github.com/ppy/osu-framework/pull/6187

This is something I've wanted to do for a while now, so I'm glad to finally get this out of the way and hopefully in a release in a somewhat experimental state for the time being. It's 100% isolated, so it should be an easy revert/delete if it doesn't go as planned.
Introduction
At its core, the idea is to record everything we need to do within a frame - all state changes, all vertices, all UBO data - and run through all of those "events" in a single pass at the end of the frame.
By doing so we can eliminate some hackery that's employed by VeldridRenderer such as different implementations of staging buffers for different scenarios (sometimes causing breakage), and haphazardly using multiple command buffers to split out buffer/texture updates from the main command buffer.
Furthermore, there's a hope that this can lead to further optimisations down the track, but there's significant work to be done before that's possible.
Expectations
Before going further, I want to set some expectations:

You should treat this as doing things correctly first, and improving performance second.
Generally slightly lower performance across the board, though may be better on D3D11.
Vulkan is stable on platforms that support it.
OpenGL doesn't allocate nearly as much as Veldrid-OpenGL.

OpenGL (Legacy) will continue to be preferred.

Deep dive
This effectively defines two passes:

The "draw" pass starts at Renderer.BeginFrame() and ends just before Renderer.FinishFrame(). It covers the DrawNode.Draw() recursion.
The "paint" pass starts and ends inside Renderer.FinishFrame().

The draw and paint passes are functionally air-gapped via subtypes of IRenderEvent.
ResourceAllocator is a sort of in-process heap that facilitates serialisation between the two passes. You give it data, it gives you references that can be used later. This is because we can't pass managed objects between the two passes.
EventList holds all the draw events emitted by the deferred classes (DeferredRenderer, DeferredShader, et. al.)
EventProcessor is the "paint" pass - it runs through all events in the EventList and does the necessary graphics operations.
UniformBufferManager and VertexManager hold all UBO and vertex data as a contiguous array. There are some specifics as to how each of these work:

Alignment is of utmost importance for UBOs.
UBOs have strict size requirements, so they're usually exposed as 65KiB "chunks". If binding incorrectly, D3D11 will silently ignore the bind request (not error or anything - ignore).
Vertices are a little bit more lenient, but we're using the new vertexIndexOffset parameter of GraphicsPipeline.DrawVertices() which requires them to at least be aligned to their own struct size.

Testing
Can be tested in-game by changing the renderer.
Can be tested via OSU_GRAPHICS_RENDERER=deferred;OSU_GRAPHICS_SURFACE=x (you must provide both, otherwise it'll fallback to deferred+opengl).

GT 710: D3D11 corruption fixed. Vulkan still dies in gameplay (driver issue).
GTX 1650: Works.
ARC A380: System dies on more than just lazer (driver issue).
9800 GT: Need to fiddle around with drivers - uses software rendering atm.
Apple M1: Works.
RX 5700XT: Works.
RTX 3070Ti: Works.
iOS: Works.

Future work

Improve alignment.

The performance of this all is dependent upon the serialisation & structures therein. I've implemented everything (e.g. EventList) without considering alignment (hence using Unsafe.CopyUnaligned()).

Reimplement the masking SSBO. This is one thing that couldn't be done efficiently before due to in-line buffer updates.
Avoid processing of DrawNodes that haven't changed during the "draw" pass.

Hmm this failed to compile... @smoogipoo probably needs "silent" conflict resolution. git let me merge master just fine.

One more thing - if there's any suggestions for naming then I'm all ears. "QueuedRenderer", "BufferedRenderer", dunno.
"Deferred" rendering is a very specific rendering technique, so I can understand the naming being confusing.

Something broke with this branch causing texture corruption around buffered containers. Looking into it.

Texture corruption will be fixed via https://github.com/ppy/osu-framework/pull/6206


Remove "attempt" from ssh parameters when checking OSP host auth

Fixes a situation where "attempt" would accidentally get passed as an SSH parameter to Openstack hosts during scheduled authentication checks, resulting in a cryptic authentication status in the UI.
INFO -- : MIQ(ManageIQ::Providers::Openstack::InfraManager::Host#connect_ssh) SSH connection established to [<IP_ADDRESS>]
ERROR -- : MIQ(ManageIQ::Providers::Openstack::InfraManager::Host#connect_ssh) SSH connection failed for [<IP_ADDRESS>] with [ArgumentError: invalid option(s): attempt]

https://bugzilla.redhat.com/show_bug.cgi?id=1392460

@miq-bot add_label euwe/yes
@miq-bot add_label blocker

@mansam it looks like attempt is something ManageIQ specific, and not something that other provider hosts need either - but on the off-chance that's not the case, would it make sense to update the verify_credentials method (https://github.com/ManageIQ/manageiq/blob/master/app/models/manageiq/providers/openstack/infra_manager/host.rb#L89) to pull out the options it's planning on using?
@blomquisg you probably have the best perspective on how this change might affect other providers - any opinion?

@mansam this was supposed to be fixed by ab40ebedc07c7ebd06ac4d9f759cafada5bbfdfb.  Can you verify this?

This change no longer seems to be necessary as far as I can tell. Closing.


Change FLAGS_trusted_root_certs to use SSL_CTX_set_default_verify_paths unless specifically overridden

And change the default to "default" to detect.
Result: I can now start a server on Mac OS X.  Have not tested on Linux.
Unable to get tests to work on Mac before or after fix.

Made change as suggested.

I'm not sure why Travis is moaning, I think the build was just flaky.  How can I tell it to try again?

Rebased to try to kick travis.

There is one test that doesn't work on OSX as it exercises undefined C++
behaviour, the others should pass. If not I might have another look at them.
I'm not sure removing the default Linux cert location from this flag is a
good idea, as this default is used in several other places.
Can you explain the problem this is addressing?
Martin
On 28 September 2015 at 04:15, Adam Eijdenberg<EMAIL_ADDRESS>wrote:

Rebased to try to kick travis.
—
Reply to this email directly or view it on GitHub
https://github.com/google/certificate-transparency/pull/947#issuecomment-143628757
.

The problem isn't a test (that's a meta-problem!), but rather starting a log server would fail for me on Mac OS X because the file "/etc/ssl/certs/ca-certificates.crt" did not exist (not at Mac right now so I don't have the exact incantation that I used).
openssl already has a notion of a default certificate location in a platform portable manner - it seems strange (wrong) to reinvent this.
On my Ubuntu-like workstation, openssl version -d shows:
OPENSSLDIR: "/usr/lib/ssl"
$ ls -l /usr/lib/ssl/certs
... /usr/lib/ssl/certs -> /etc/ssl/certs
Using SSL_CTX_set_default_verify_paths will actually use all of the certificates in /etc/ssl/certs and per http://manpages.ubuntu.com/manpages/karmic/man8/update-ca-certificates.8.html, ca-certificates.crt should be a duplicate of that data, and as such this should give identical behavior.
One could argue that we drop this flag completely, as if we always use SSL_CTX_set_default_verify_paths then the cert file used can be overridden by setting common openssl environment variables that off the top of my head I cannot recall.

OK, I now realize this is for outgoing connections. It seems reasonable to
me that this should use the platform defaults.
From reading the docs it appears that your suggestion of removing the flag
and setting default verify paths is a good one.
Martin
On 28 September 2015 at 16:01, Adam Eijdenberg<EMAIL_ADDRESS>wrote:

The problem isn't a test (that's a meta-problem!), but rather starting a
log server would fail for me on Mac OS X because the file
"/etc/ssl/certs/ca-certificates.crt" did not exist (not at Mac right now so
I don't have the exact incantation that I used).
openssl already has a notion of a default certificate location in a
platform portable manner - it seems strange (wrong) to reinvent this.
On my Ubuntu-like workstation, openssl version -d shows:
OPENSSLDIR: "/usr/lib/ssl"
$ ls -l /usr/lib/ssl/certs
... /usr/lib/ssl/certs -> /etc/ssl/certs
Using SSL_CTX_set_default_verify_paths will actually use all of the
certificates in /etc/ssl/certs and per
http://manpages.ubuntu.com/manpages/karmic/man8/update-ca-certificates.8.html,
ca-certificates.crt should be a duplicate of that data, and as such this
should give identical behavior.
One could argue that we drop this flag completely, as if we always use
SSL_CTX_set_default_verify_paths then the cert file used can be overridden
by setting common openssl environment variables that off the top of my head
I cannot recall.
—
Reply to this email directly or view it on GitHub
https://github.com/google/certificate-transparency/pull/947#issuecomment-143768154
.

@AlCutter - Al, this code is your fault.  What do you think we should do?
(a) Keep the flag, but change the defaults as per this commit (shouldn't change behavior)
(b) Drop the flag entirely and always use SSL_CTX_set_default_verify_paths since I think users can get the same effect by setting SSL_CERT_FILE.

@aeijdenberg, everything is always my fault, and I'm cool with that :)
In general I'm less of a fan of environment variables than explicit flags as it's a lot easier to spot the latter when someone comes to you with a problem - but, that said, having a flag whose default value implies using the system roots but then which may or may not use them depending on an environment variable seems like the worst of both worlds, so it seems like the best thing to do would be your option (b): if there's an "officially supported" OpenSSL way of setting this, and it sounds like there is, then we should probably do it that way.

On Wed, 30 Sep 2015 at 10:36 Al Cutter<EMAIL_ADDRESS>wrote:

@aeijdenberg https://github.com/aeijdenberg, everything is always
my fault, and I'm cool with that :)
In general I'm less of a fan of environment variables than explicit flags
as it's a lot easier to spot the latter when someone comes to you with a
problem - but, that said, having a flag whose default value implies using
the system roots but then which may or may not use them depending on an
environment variable seems like the worst of both worlds, so it seems like
the best thing to do would be your option (b): if there's an "officially
supported" OpenSSL way of setting this, and it sounds like there is, then
we should probably do it that way.

Not sure about that. Setting env variables for things buried deep in other
things can be quite exasperating.
Also, I think that if you use the env variable, you are expecting it to
override the default roots - that is its purpose.
So, disagree. Think we should have a flag whose default means "use
defaults/env".

—
Reply to this email directly or view it on GitHub
https://github.com/google/certificate-transparency/pull/947#issuecomment-144339035
.

Ben, I think that's what the current proposed patch does.  By default we use the defaults (which includes env, because that's what openssl default does) or if you specify it, then we use that instead.
So if that's the case, I'll push the big green button as soon as anyone LGTMs it.


Language-specific guides request

Problem description

Problem location

I saw a problem on the following URL:

I couldn't find the information I wanted. I expected to find it near the following URL:

Other:

Project version(s) affected

Suggestions for a fix

Message-ID

<EMAIL_ADDRESS>

12899

would you mind providing any details?

let me close this one as we need more information


feat: add new cookie consent library

Have submitted a PR here: https://github.com/whitecube/laravel-cookie-consent/pull/3 to address the test failures.


The return values of the framework and the results of MySQL's internal command-line execution are inconsistent.

Challenges encountered when using Sharding-Proxy for database sharding and table partitioning
screenshot：

result:
gorm return: 0
mysql return: 1

@shig-cn please show us a demo, or paste your full codes here, or no one else know what exactly going on with your scenario.


flutter foreground task

@govindmaheshwari2 Let's a add video showing working of Android Foreground Service in this Doc as well


Plans to support format strings?

Neat little lib, i was wondering if you had plans to support formatted strings.

No current plans. Do you have a use case in mind that the example below wouldn't be appropriate for? If so, I'd be curious to know.
var firstName = "Nick";
var lastName = "Spreitzer";

// C# >= 6
$"My first name is '{firstName}' and my last name is '{lastName}'.".WriteLine();

// C# < 6
string.Format("My first name is '{0}' and my last name is '{1}'.", firstName, lastName)
    .WriteLineWait();

No specific use case, I just think it would be a tiny bit easier with a couple overloads for C# 5 and below. I could write then in a PR if you think it's a good idea.

Can you show me an example of the potential usage? Are thinking something like this?
"My first name is '{0}' and my last name is '{1}'.".WriteLine("Nick", "Spreitzer");

I thought about this again this morning and i think that the C# 6 syntax does everything already including numerical formatting ( e.g. {myNum:N0} ) which i didn't realize. I think now this change is unnecessary.

I actually think it was a good idea, for those who aren't on C# 6 yet. I may still implement the example I wrote above.


Cache outline headers & ref count outline provider

For https://github.com/microsoft/vscode/issues/209787

VS Code Stable (typing around 20 chars in quick succession)

VS Code OSS (typing around 40 chars in quick succession)
The changes are not yet in VS Code insiders.


BeamGuard wrongly blocks beamTo

Describe the bug
I have a bug where I use a BeamGuard to redirect from one BeamLocation to another. The issue is that is never updates the pages and never builds the new BeamLocation.
After digging around, I found the responsible code lines:
package/lib/src/beam_guard.dart#L132-L136
The issue is that my redirected BeamLocations does not change the url location, but instead displays a different page for the same url.
I see why the specified lines are there but for me this does not work. Maybe the if block could also check whether the runtimeType of both BeamLocations are different.
If there is some other way that allows a BeamGuard to redirect to a BeamLocation without changing the url let me know.
Beamer version: 1.4.1

Hey old friend @schultek :slightly_smiling_face:
I see what you mean... This is an interesting problem that should definitely allow more customization, e.g. that you can optionally customize what should BeamGuard treat as equal.
I'm planning to rework the guards for v2 so will assign this there. I plan to start working heavily on v2 next week.


[BUG] Settings link in networks doesn't work

Describe the bug

If you don't have a VLAN added to the harvester cluste you get a link when going to the networks page. The link is broken
To Reproduce
Steps to reproduce the behavior:

Navigate to networks in settings
Click the settings link
Expected behavior

It should take you to the advanced settings vlan section of settings
Support bundle

Environment:

Harvester ISO version:
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):

Additional context
Add any other context about the problem here.
It looks like the relative URL isn't correct.

caused by https://github.com/harvester/harvester/issues/1096

should be addressed via https://github.com/harvester/harvester/pull/1098, and the network CRD is updated to the non-namespaced scope via https://github.com/harvester/network-controller-harvester/commit/c68dcf51f5fd09625640a4c219aff6a1869941d5

Tested with latest master-head build: the link (https://:30443/dashboard/c/local/virtual/network.harvesterhci.io.clusternetwork/vlan?mode=edit) is working again.
This issue can be closed.


Update Get-RMSTrustedPublishingDomain.md

Added cmdlet deprecated note per PM request

Docs Build status updates of commit 1bcbfc7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-RMSTrustedPublishingDomain.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


PSD-735 set perfana-gitops config for test env

https://perfana.atlassian.net/browse/PSD-735

I think we do not (yet) need a separate develop branch and a make-a-check-test.yml flow: the same newly build afterburner can be deployed to all envs.


Update Keras Mixin

[x] Use SavedModel format instead of h5
[x] Add functional API

Both mentioned in #230

Note -  I had to switch to using snapshot_download because:

The SavedModel format doesn't play nice when filenames are hashes instead of folder containing keras_model.pb metadata.pb, variables/, etc.
You have to enforce the structure of the directory tree.

Using forced_filename also could have worked, but you'd have to gather up all the files within variables/assets/etc
I've seen some folks just zip up the specific file tree instead due to similar issues.

It would be nice to also add a Github Actions job with Keras to test this automatically on CI!

I came late to the last round of review, but great work! 🔥


Error in latest.log if mod is active

Hi,
i don't know if this error is crashing the game sometimes, but i think any error which isn't there is better :D
[12:56:06] [main/INFO]: (Dark Matter / InstrumentationAccess#bootstrap) Attaching instrumentation agent to VM.
[12:56:06] [main/ERROR]: (Dark Matter / InstrumentationAccess#bootstrap) An error occurred during an attempt to attach an instrumentation agent, which might be due to spaces in the path of the game's installation.
java.lang.NullPointerException: Couldn't find included "jar/dark_matter_instrumentation_agent.jar"!
at me.melontini.dark_matter.danger.instrumentation.InstrumentationAccess.bootstrap(InstrumentationAccess.java:184) ~[dark-matter-danger-0.6.0-1.18.2-144ffa74e35952d.jar:?]
at me.melontini.dark_matter.danger.instrumentation.InstrumentationAccess.(InstrumentationAccess.java:154) ~[dark-matter-danger-0.6.0-1.18.2-144ffa74e35952d.jar:?]
at me.melontini.blamelog.BlamePlugin.(BlamePlugin.java:40) ~[blame-log-0.3.0-1.18.jar:?]
at java.lang.Class.forName0(Native Method) ~[?:?]
at java.lang.Class.forName(Class.java:467) ~[?:?]
at net.fabricmc.loader.impl.launch.knot.MixinServiceKnot.findClass(MixinServiceKnot.java:96) ~[fabric-loader-0.14.21.jar:?]
at org.spongepowered.asm.mixin.transformer.PluginHandle.(PluginHandle.java:96) ~[sponge-mixin-0.12.5+mixin.0.8.5.jar:0.12.5+mixin.0.8.5]
at org.spongepowered.asm.mixin.transformer.MixinConfig.onSelect(MixinConfig.java:708) ~[sponge-mixin-0.12.5+mixin.0.8.5.jar:0.12.5+mixin.0.8.5]
at org.spongepowered.asm.mixin.transformer.MixinProcessor.selectConfigs(MixinProcessor.java:498) ~[sponge-mixin-0.12.5+mixin.0.8.5.jar:0.12.5+mixin.0.8.5]
at org.spongepowered.asm.mixin.transformer.MixinProcessor.select(MixinProcessor.java:460) ~[sponge-mixin-0.12.5+mixin.0.8.5.jar:0.12.5+mixin.0.8.5]
at org.spongepowered.asm.mixin.transformer.MixinProcessor.checkSelect(MixinProcessor.java:438) ~[sponge-mixin-0.12.5+mixin.0.8.5.jar:0.12.5+mixin.0.8.5]
at org.spongepowered.asm.mixin.transformer.MixinProcessor.applyMixins(MixinProcessor.java:290) ~[sponge-mixin-0.12.5+mixin.0.8.5.jar:0.12.5+mixin.0.8.5]
at org.spongepowered.asm.mixin.transformer.MixinTransformer.transformClass(MixinTransformer.java:234) ~[sponge-mixin-0.12.5+mixin.0.8.5.jar:0.12.5+mixin.0.8.5]
at org.spongepowered.asm.mixin.transformer.MixinTransformer.transformClassBytes(MixinTransformer.java:202) ~[sponge-mixin-0.12.5+mixin.0.8.5.jar:0.12.5+mixin.0.8.5]
at net.fabricmc.loader.impl.launch.knot.KnotClassDelegate.getPostMixinClassByteArray(KnotClassDelegate.java:422) ~[fabric-loader-0.14.21.jar:?]
at net.fabricmc.loader.impl.launch.knot.KnotClassDelegate.tryLoadClass(KnotClassDelegate.java:323) ~[fabric-loader-0.14.21.jar:?]
at net.fabricmc.loader.impl.launch.knot.KnotClassDelegate.loadClass(KnotClassDelegate.java:218) ~[fabric-loader-0.14.21.jar:?]
at net.fabricmc.loader.impl.launch.knot.KnotClassLoader.loadClass(KnotClassLoader.java:112) ~[fabric-loader-0.14.21.jar:?]
at java.lang.ClassLoader.loadClass(ClassLoader.java:520) ~[?:?]
at java.lang.Class.forName0(Native Method) ~[?:?]
at java.lang.Class.forName(Class.java:467) ~[?:?]
at net.fabricmc.loader.impl.util.DefaultLanguageAdapter.create(DefaultLanguageAdapter.java:50) ~[fabric-loader-0.14.21.jar:?]
at net.fabricmc.loader.impl.entrypoint.EntrypointStorage$NewEntry.getOrCreate(EntrypointStorage.java:117) ~[fabric-loader-0.14.21.jar:?]
at net.fabricmc.loader.impl.entrypoint.EntrypointContainerImpl.getEntrypoint(EntrypointContainerImpl.java:53) ~[fabric-loader-0.14.21.jar:?]
at net.fabricmc.loader.impl.entrypoint.EntrypointUtils.invoke0(EntrypointUtils.java:47) ~[fabric-loader-0.14.21.jar:?]
at net.fabricmc.loader.impl.entrypoint.EntrypointUtils.invoke(EntrypointUtils.java:35) ~[fabric-loader-0.14.21.jar:?]
at net.fabricmc.loader.impl.launch.knot.Knot.init(Knot.java:162) ~[fabric-loader-0.14.21.jar:?]
at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:68) ~[fabric-loader-0.14.21.jar:?]
at net.fabricmc.loader.impl.launch.knot.KnotClient.main(KnotClient.java:23) ~[fabric-loader-0.14.21.jar:?]
[12:56:06] [main/ERROR]: [BlameLog] Instrumentation went to get some milk, but never came back...
latest.log
It's funny the error says something about "Dark Matter", but this mod wasn't active at this time and when it's active, the error is the same.

Hmm, this was supposed to be fixed 😶🌫
You're also saying that it only happens sometimes?

The error is shown always in latest.log when the mod BlameLog is active. I was just confused 'coz the error said something with "Dark Matter" and i thought it was the fault of this mod and disabled it to try it out, but the error stays. Then i found out, only if the mod BlameLog is active, this error is shown up in latest.log.

Should be fixed in 0.3.1


Add ESP32 GPIO control and temperature example

Couple of nitpicks are added to other pages as well.

 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.


[CPU] Clean-up tile size computation for no-pad matmul strategy

This is mostly a test to evaluate the complexity of simplifying the current tile size computation logic for vectorization/unrolling. I commented out the logic around getMaxTileSizes in the no-pad matmul strategy hoping to see some test failures but everything seems to pass. However, I do see some performance regressions. It's difficult to infer which cases are covered by the commented logic without tests to look at. I would need some help to identify those cases.

I'm cleaning the PRs that need my review. Do we still need the PR or we can close it?


Add rule for unused Conditions

Issue #39:
Added a check to see if Conditionals specified are acutally used, made it a warning (W8001)
Added test and updated the existing test files to match. Found a gltch in the  _search_deep_keys  method that stopped the search if a key if found. But conditions can be used in conditions again (See: https://github.com/awslabs/cfn-python-lint/blob/master/test/templates/good/conditions.yaml#L29 )
Found more errors in the quickstart templates when applying this.
In regard to the quickstart templates, added a quick first version of a Readme stating that these templates are "as is" and should not be modified.

Updated the Test to leave alone the "base" good templates and use it's own list

Yes, it's in there... Sorry, had to stop working due to an appointment. Fixed and improved the error message for clarifivcation


Show all buildings at z16

Connects to #557.
@zerebubuth, @nvkelso could you review please?

There's something else going on here. I've got some tests which I don't understand why they're failing... I've pushed what I've got at https://github.com/mapzen/vector-datasource/tree/557-missing-buildings. Not sure why the test for this small building is failing... It seems to pass all the filters and get discarded by the formatting step?!

:+1: looks good to me!
Do we need to add something in the docs, or is min_zoom already documented?

Cool.


Add multitenancy tests

This PR adds multitenancy tests which execute various operations under different users and check that one user can not use the keys of another one.
Fix #245

I keep facing issues because of switching user to compile and execute the tests. I think I would have to clone the parsec repo under each user's home directory and run the cargo test command there.
Because the tests are not doing anything fancy, it might just be easier to write them in bash and use parsec-tool. That way they will be easier to work, understand and maintain.

To use parsec-tool for testing, there will need to be an option there to modify the socket path, to use /tmp/parsec.sock

I keep facing issues because of switching user to compile and execute the tests. I think I would have to clone the parsec repo under each user's home directory and run the cargo test command there.
Because the tests are not doing anything fancy, it might just be easier to write them in bash and use parsec-tool. That way they will be easier to work, understand and maintain.

Is it not possible to create symlinks for every user to the "root" Parsec folder?


Refresh Catalog Cache in Quota App

In the current code, if the given planId is not found in the catalog of quota_app we return 500- Internal server error which is a generic error. This PR modifies the code to instead return 404 NotFound in such case.
Resolves HCPCFS-3085.

Pull Request Test Coverage Report for Build 5507

11 of 19 (57.89%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 88.714%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

broker/data-access-layer/quota/src/BaseQuotaManager.js
5
7
71.43%

broker/applications/quota-app/src/QuotaApiController.js
6
12
50.0%

Totals

Change from base Build 5494:
-0.07%

Covered Lines:
9716

Relevant Lines:
10572

💛  - Coveralls


TypeScript - Find all references not working for default exports

"Find all references" feature doesn't report any usages for a class that is exported as default. See example below:
// B.ts
export default class B { // <-- "find all references" on B returns no usages
    test() {
    }
}
// A.ts
import B from "./B";

let b = new B();
b.test();

VSCode version:
Version 0.10.6
Commit a80232bbcfe8a5cdad1ebc98638673d9dcb02458
Date 2015-12-19T11:26:42.359Z
Shell 0.34.5
Renderer 45.0.2454.85
Node 4.1.1

Please give Salsa our new JS infrastructure a try. Here is how:
get the insider channel: https://blogs.msdn.microsoft.com/vscode/2016/02/01/introducing-insiders-builds/
enable salsa: https://github.com/Microsoft/vscode-docs/blob/master/release-notes/latest.md#javascript---salsa-preview

Great stuff. I got it set up and I'm pretty sure it's using the new language service since I can use the 'readonly' keyword. However, I'm not seeing the "Salsa" indicator in the status bar, as indicated in the documentation.
Unfortunately, the issue described can still be reproduced.

No Indicator means you are not running on Salsa :disappointed:
Are you setting the environment variable as described in the release notes:

Set the environment variable VSCODE_TSJS. On OS X, it is recommended that you change this in your .bash_profile using export VSCODE_TSJS=1. That way the environment variable is persisted.

I am using the windows version now. Setting the environment variable had no effect. Still not seeing any indicator about salsa, be it active or grayed out.
tsc is installed globally (Version 1.9.0-dev.20160205) and "typescript.tsdk" points to the correct path.

I'll try it on Monday also at work on my Linux machine, maybe the results will be different

Just tried it and it works for me. The indicator only shows up when a JS file is open and in focus. Also do not forget to restart Code.

Yes, you are right. It does show up for JavaScript files now. However, my issue was with a TypeScript-only project. I wouldn't expect "Find references" to work with plain JavaScript anyway.

I'm really really sorry, I didn't read your sample careful enough... My head was on Salsa and I was in the Salsa-solves-this-all mood. I'll later delete some of the comments to keep the issue clear.

This issue was moved to Microsoft/TypeScript#7024

Still doesn't work for decorated exports of React functional components:
const B = (props) => { return (<div/>); };
export default (() => B)();


Dev search v201

Merge contains mainly code refactoring commits.
New feature: filter with videos.Live check here:
http://gofullstack.me:3001/search?q=%23berlin%2B%2Fvideo&timezoneOffset=-180
It should be able to render also native video from twitter.

Video embeds in page work perfectly.
Search is still working

Those tabs for source type selection and the display of video looks good!


Wrong mining productivity

For example, iron/copper/coal ores productivity mined by electric drill is 0.525/s, but planner uses value 0.5/s. For stone - 0.65/s but planner uses still 0.5/s. There is small workaround - using 5% productivity bonus makes ore mining speed equal to 0.525/s, but for stone...
Formula uses (Mining_power - Mining_hardness) factor, that depends on ore-drill pair, so it's not so simple to make this change.

Thanks for the notice and for the patch! I like how you implemented it :)


Unlock all Option

Describe the feature you'd like to see added
pretty self explanitory on what this would do
pressing this will just Unlock everything in the game (mercs, weapons, difficultys)

I already talked to Nipkow about including all useful options for the Raz0r Trainer within Tweaks, so this is included, but it's interesting to remember, this can be very useful for people who are coming to play the game with the intention of just playing directly the mods as they've played the game many times in the past and all mods are usually made based on Professional difficulty, so including this option might be helpful, although you can teleport to the saddler, kill him with the "" Weak Enemies" and finish the game to unlock some stuff. But unlock everything at once is still useful but not primordial and i know you are not saying that, you are just recommending feature.
And just a commentary, put everything useful of the Trainer on the Tweaks can be very burningout, i believe they will not do it, and i understand compleatly. Mariokart have done a break on this for maybe a moment, maybe he back, maybe not, i'm very calm with the game, i feel everyone here is doing too much things very fast and i recommend relax, i hope they are not feeling overtasked. I'm glad to see all the new stuffs working so good and i can only thanks.
Ah and taking advantage about a option idea too what is good in my opnion, is the "True" Invencibility for Leon like Raz0r Trainer, so Leon will not be grabed by ganados while invencible, will not move at all for all game interferences. And please guys relax a bit for not get overhelmed, do only the necessary, Qingsheng is tired too and is off for a month, almost 2 already, do things only if you guys want, be at peace, and i recommend you guys if want open a method to donate for people be able to help in the project, love you guys including my friend scoovy.

Seems unlocks are controlled by the EXTRA flags, you can change those in the flag editor, but main menu needs to be reloaded for the unlocks to show, and reloading main-menu then reloads your save and overwrites the flags you set, catch-22 :/
Maybe will need to look into a way of saving the EXTRA flags when user modifies them, and then hook the save-loading code to restore the modified ones, then could just add a "Unlock All" button/setting that overwrites those flags for us.

Does the EXTRA flag also enable being able to select a difficulty when wanting to start a new game? I always found it weird how RE4 just immediately starts upon pressing New Game if no save file is detected. I think that could be a good accessibility change.


Publish to Chocolatey

I've noticed that there isn't any installer available which makes it a bit of a hassle to upgrade RoslynPad.
The software package manager Chocolatey would be a fairly convenient solutions towards this, I think.
EDIT: I've also read in a separate issue that Squirrel is being considered. I'd like to note that this is also .NET Framework friendly as well.

Good idea.
You can also install RoslynPad from the Microsoft Store, where you get automatic updates.

I've seen that as well, and that's certainly a viable solution.
I think the conflict primarily lies with those who wish to avoid the Microsoft Store for privacy concerns.
To my knowledge, the Microsoft Store solution includes compulsory Microsoft accounts in addition to agreeing to the Terms of Service, of which allows for the active collection of telemetry, and other such potentially personally-identifiable information.

And some companies to not allow the usage of the Microsoft store. Plus for automatic installation chocolatey would be great. (currently using LinqPad for that reason)

The next version will be published to winget


PowerRename gives no warning or error when failing to rename a file with a path longer than 259 characters

Microsoft PowerToys version
0.59.0
Running as admin

[X] Yes

Area(s) with issue?
PowerRename
Steps to reproduce

Have a file with a full file path longer than 259 characters (mine was W:\URUHA RUSHIA CH - ARCHIVE\Rushia Ch. 潤羽るしあ\20220202 - 【Getting Over It with】ごめんあさい！壺おじ！！クリアしたいの！！応援してくだちい；；（未クリア）【潤羽るしあ_ホロライブ】\Rushia Ch. 潤羽るしあ - 20220202 - 【Getting Over It with】ごめんあさい！壺おじ！！クリアしたいの！！応援してくだちい；；（未クリア）【潤羽るしあ_ホロライブ】 [beHrlSK5kr0] f251.info.json (was multiple files, extensions were description,info.json)) on an external NTFS drive

Use PowerRename to change the filename while keeping it above 259 characters (mine was
 )

No error message or warning is thrown, PowerRename just silently fails to rename the file

✔ Expected Behavior
PowerRename to either rename the file (Registry key HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\FileSystem\LongPathsEnabled is 1) or to fail to rename it, throwing me a warning.
❌ Actual Behavior
PowerRename silently fails to rename the file
(on a side note, the windows Explorer also doesn't let me add any characters after the path exceeds 259 characters, only letting me remove them until it goes back down)
Other Software
No response

Addition: Choosing a list of files where even one has a path longer than 259 results in none of the files showing up in PowerRename
One file, path < 260:

One file, path > 260:

Multiple files, one with a path > 260:

This can be worked around by opening PowerRename on the parent folder, like I did in the original post/

Good point. Similar to #8748.
Also (loosely) related #1853 and #17766


Boost reset when staking again [NAY-1] [CU-8694aecyq]

Boost is not properly reset after unstaking, causing more boost to be assigned to the user than it should.

In _unstake() , we only set s.stakeBoost[vTokenId][_stakerId] = 0; back to zero, but we do not set the stake boost for the next vTokenId back to zero. When a user stake in the current interval, part of the stake boost will be allocated to the next vTokenId proportional to the time it was staked between the current interval and the next interval

It would add up the the boost from previous staking, since it was not reset back to zero.

Task linked: CU-8694aecyq User can artificially inflate stake boosts for the next interval by repeating stake and unstake actions (NAY-1/HIGH)


meta-issue: stabilize OpenTracing 1.0RC semantics

I mentioned this in my OpenTracing status report, but to reiterate here...
This is the rough sequence of steps before I would feel ok declaring an OpenTracing 1.0 semantic spec:

Agree that there’s a problem to solve
Agree on the basic approach / vision
Propose initial semantic spec and specific platform sketches, with examples
Converge on a “1.0RC” semantic specification (with rough implementations on multiple platforms)
Allow early-adopters to bind to the “1.0RC” spec (app programmers, IPC frameworks, control flow systems, and of course tracing implementations)
Converge on a “1.0” semantic spec and announce more broadly, "co-branded" with various OpenTracing users and implementations

Right now we are somewhere between (3) and (4) above, and this particular Issue is about getting to (4).
Here are the more controversial issues I'm remembering, some of which I think are mostly resolved:

[x] Should DCP be modeled separately from Spans/etc? (Current answer: “no”)
[x] Should we support a robust message logging API? (Current answer: “maybe, but not yet”)
[x] Should we support (auto-propagated) “Trace Attributes”? (Current answer: “yes, with copious warnings in comments”)
[x] How should the Propagator interface interoperate with IPC frameworks? (Undecided)
[x] The best API for child-Span creation (Undecided)

There are also some lingering naming concerns, but I'm confident we can get through those if we deal with the semantics.
I would further suggest that we try to hash out the remaining sticking points via a google hangout rather than on Github. On that note, please fill out:
http://doodle.com/poll/q82qiz932q9gm3dr
If we need to use Github due to timezone sadness, I would ask that we restrict ourselves to issues in opentracing.github.io (creating new ones as needed) rather than fragmenting the discussions across per-language repos. And concerns about the current semantic spec should be accompanied by specific suggestions to the relevant use cases.
In alphabetical order: I suspect that some of @adriancole @bcronin @dkuebric @michaelsembwever @slimsag @yurishkuro might care about this... of course all are welcome to chime in!
(^^^ alphabetical order on the github usernames, btw :)

I was joined by @michaelsembwever @dkuebric @yurishkuro @bg451 for a hangout today, and we actually resolved a bunch of stuff. Thank you again to all of the above, much obliged!
Should we consolidate the span-creation methods?: basically, "Yes." There was agreement that we should do our best (within the confines of the platform) to consolidate around a Tracer method that creates a Span, regardless of whether that span will ultimately be a root or whether its parent is local or remote, etc. I don't think we came to a consensus about whether there should also be a way to create a child Span given a parent Span instance alone (i.e., without a Tracer)... But that decision can come later and would be built upon the singular mechanism mentioned above.
Should we support Trace Attributes? Yes, but with caveats in the comments, both about the possible dangers of the feature and also about the fact that the API may go away or change in incompatible ways if its "considered harmful" in the future.
Should we go back to separated Span and TraceContext concepts/types? No... it's conceptually tempting for sure, but it "makes the simple things complex" and we don't have enough adoption to get the "Span-free" version of DCP correct anyway.
Can we do something like https://github.com/opentracing/opentracing.github.io/pull/45? Yes. Though the details will be platform-dependent, there was support for the general idea of adding one more layer of indirection to get to a particular Injector/Extractor (or whatever we end up calling them).
Next steps:

[ ] get Python to parity with Go
[ ] experiment with injector/extractor registration patterns

(all should feel more than welcome to add to or correct anything about the above)

Should we consolidate the span-creation methods?

Not sure of practical implications of this one. I think we agreed to keep span.start_child, as well as move injector to the span level, so that the only place in the application that needs a Tracer reference is the server endpoint dealing with inbound requests. In addition, we still need two other span creation methods

tracer.getExtractor(typeOf[x]).extract(x) -> Span
tracer.start_trace() for standalone apps/clients

@yurishkuro tracer.start_trace() could be a special case of tracer.create_span() where there's no parentage info (at least conceptually; in Go there might be separate method variants).

@bensigelman but when would we use tracer.create_span() except to start a new trace? The "parentage info" can only be another span, in which case we can do span.start_chid().

I would probably break it down into three cases:

you have the propagated form of a Span (presumably from a different process) and want to resume tracing
you have a Span instance and you want to create a span to model some activity that's about to happen
you don't have a Span instance and you want to create a span to model some activity that's about to happen

Case (1) seems like a good reason to have tracer.create_span() independent of anything else. Case (2) is of course why I've been arguing for span.start_child() as a convenience method/synonym.
Case (3) is interesting and tricky... certainly it would be nice for the user if, once the in-process propagation story is clearer, cases (2) and (3) were covered by the same function call that did the equivalent of get_current_span() (from the use-cases doc) internally. In the meantime I do see an argument for separating the methods (esp in Go where it's almost required since the param counts are different), but the spirit of both is the same in my eyes: "create a span to model some activity that's about to happen".
Maybe the above is a long way of saying that I'm leaning towards a common method prefix of Tracer.CreateSpan* in Go (for both 2 and 3 above) and overloading Tracer.create_span(op_name, parent_info=None) in Python/etc.

Isn't case (1) solved by the Extractor pattern?  I guess I am not seeing where Tracer.create_span(op_name, parent_info=None) would be used with parent_info not being None. Since we don't have a class to model "trace context" anymore, parent_info can only be another Span, thus this child can be started from there.
Case (3) is indeed interesting because it can occur not only at the edge, but sometimes deep inside the app, as here: https://github.com/uber-common/opentracing-python-instrumentation/blob/master/opentracing_instrumentation/local_span.py#L125.

I skipped a step in my reasoning, apologies... I am arguing that methods which return a newly-started Span should be in the same naming constellation, whether that implies overloading or a common prefix for a particular platform. This means that the Extractor interface would feature a method matching CreateSpan*, too.

re getting python to parity with Go: https://github.com/opentracing/opentracing-python/pull/14

We are in a good place after the recent merges in opentracing-go... I will make parallel changes to the spec.md and python, at which point we can close this issue.

[ ] update docs site per opentracing/opentracing-go#48 and opentracing/opentracing-go#50
[ ] update the python API to follow suit with the above

(yay)


Add site mapledesign.co.uk

Thanks for your work on 11ty!

got a trailing comma in there!

Sorted!

Thank you!


Need a cancel option

I'm not 100% sure how I got here.

But you should have a Cancel or Exit option to quit without doing anything.

That is a very good point.
I'll add that in, thanks for the feedback.

Added a quit option.
Thanks for the feedback.


oData example, slight problem, "Not found"

Hi
I am using the standard oData example. However, I keep getting a "Not found" message in the grid in the browser instead of the data.
Here's the code:
$(function() {

    $("##jsGrid").jsGrid({
        height: "auto",
        width: "100%",

        sorting: true,
        paging: false,
        autoload: true,

        controller: {
            loadData: function() {
                var d = $.Deferred();

                $.ajax({
                    url: "#event.buildLink("Users.rolesAsJson")#",
                    dataType: "json"
                }).done(function(response) {
                    d.resolve(response.value);
                    console.log(response);
                    console.log(response.value);
                    console.log(d.resolve(response.value));
                });

                return d.promise();
            }
        },

        fields: [
            { name: "role_ky", type: "number" },
            { name: "role_name", type: "text"},
            { name: "ROLE_TEXT", type: "text"},
            { name: "ROLE_RANKING", type: "number"},
            { name: "ROLE_STATUS", type: "number"},
            { name: "ROLE_DATE_CREATED", type: "text"},
            { name: "ROLE_DATE_AMENDED", type: "text"}
         ]
    });

});

If you look at the three console.log()s they give me this, in order:
An object containing the COLUMNS and DATA (the json result of my query)
undefined
An empty object
So I guess my question is, why is response.value undefined? And what do I do about it?

This has nothing to do with js-grid itself.
Most probably there is an issue with your OData service.
Check your backend configuration so that you can get data w/o js-grid (open the url in the browser or in Postman).
Ensure there are data in DB (probably there are no data).
Check this post about OData service creation: https://www.codeproject.com/Articles/1033244/OData-Service-using-WCF-Service.
Or at this one OData service + Backbone: https://dzone.com/articles/backbonejs-app-wcf-odata


fix: typo in MouseCursor.js

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fixed typo.

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
no

Other information:
seperated -> separated

Thanks!

:tada: This PR is included in version 5.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Batch CAS writes to reduce round trips between bst and casd

See original issue on GitLab
In GitLab by [Gitlab user @juergbi] on Sep 11, 2019, 06:22
Adding large directories to CAS (sources and artifacts) is fairly slow right now as files are added one at a time with gRPC calls in add_object(). We should batch CAS writes either by using a single CaptureFiles request for many files or using CaptureTree to add a whole directory tree in a single request.

In GitLab by [Gitlab user @juergbi] on Sep 24, 2019, 10:27
assigned to [Gitlab user @juergbi]

In GitLab by [Gitlab user @juergbi] on Sep 24, 2019, 10:27
assigned to [Gitlab user @juergbi]

In GitLab by [Gitlab user @juergbi] on Sep 25, 2019, 15:25
mentioned in merge request !1611

In GitLab by [Gitlab user @juergbi] on Sep 25, 2019, 15:25
mentioned in merge request !1611

In GitLab by [Gitlab user @marge-bot123] on Oct 2, 2019, 11:18
closed via merge request !1611

In GitLab by [Gitlab user @marge-bot123] on Oct 2, 2019, 11:18
closed via merge request !1611

In GitLab by [Gitlab user @marge-bot123] on Oct 2, 2019, 11:18
mentioned in commit 7d050faca8a9c4bd70afd26121d1800de74f119b

In GitLab by [Gitlab user @marge-bot123] on Oct 2, 2019, 11:18
mentioned in commit 7d050faca8a9c4bd70afd26121d1800de74f119b


Maximum Valid Lineups

I have multiple lineup constraints. When I try to create, say 150 lineups, pydfs-lineup-optimizer will create well over a hundred then return an error code (see screen grabs below). I can see the lineups with the print function up to the error, but cannot export any of the good lineups prior to failure.
My question is this: Is there a way to stipulate the maximum number of lineups until failure and then allow for the export of those good lineups? Or, conversely, a way to simply print all lineups that meet criteria?
In advance, thanks for helping out a noob.

What constraints are you running cause I can still get 150, also can run a try pass and still do export

In the lineup_export.py I added the following except/pass to deal with when I ended up in the situation where my constraints didn't allow as many lineups as I was expecting
def export(self, filename, render_func=None):
    with open(filename, 'a', newline='') as csvfile:
        lineup_writer = csv.writer(csvfile, delimiter=',')
        try: #added sep1 2022
            for index, lineup in enumerate(self.lineups):
                if index == 0:
                    header = [self.COLUMNS_MAPPING.get(player.lineup_position, player.lineup_position)
                              for player in lineup.lineup]
                    header.extend(self.EXTRA_COLUMNS)
                    lineup_writer.writerow(header)
                row = [(render_func or self.render_player)(player) for player in lineup.lineup]
                row.extend(self._get_extra_columns(lineup))
                lineup_writer.writerow(row)
        **except: #added
            pass #added**

Same thing  I did, just in my normal code config

Thank you! I'll give it a try.


imgage resize

Hello,
Probably not the right place but, I can't resize my image during export. I use this in mk  :
""
I see the resizing image on typora but the final pdf as only full size image. Can someone have any idea, why ?
Julien

you could give something like ![](tab1.png){width="80%"} a try (see https://pandoc.org/MANUAL.html#images).

ok with --listings options it's working. thanks a lot!

ok with --listings options it's working. thanks a lot!

strange. this should work no matter what syntax highlighting is used for code blocks?


Convert to pure DotNetCore project

Do not merge yet. I'll modify the readme if ACK.

Have you tried that:

Build correctly in VS
Tests show correctly in VS
Build and can Test on linux


Return custom objects instead of raw web3 transactions from API

EDIT: After discussion, we changed desired approach and title was updated. Old title was "Move waitTransactionFinished to origin object"
It's a common pattern to create and then wait for transaction. For example, this example from our docs:
const transaction = await origin.listings.close(
      listing.address
    )
await origin.contractService.waitTransactionFinished(transaction.tx)

However, we should strive to hide contractService from developers, so I'd suggest moving waitTransactionFinished() directly to the origin object, or making a method on origin that relays the call to contractService.
EDIT: We should not return a raw web3 transaction, but should return an abstracted transaction object with a convenience function isFinished() for waiting for it to finish.
The above line would then read
const transaction = await origin.listings.close(
      listing.address
    )
await transaction.isFinished()

@DanielVF what do you think?

@wanderingstan Couple thoughts.

If we're wanting to hide contractService from developers, then we shouldn't export it, right?

I like this approach; however, I think if we're dividing the origin package into smaller modules, then maybe we want to place all of our methods into modules? So instead of putting this method directly on the origin object, maybe we could put it in a separate package. (Something like util maybe?) Not entirely sure about this - need to look at the code a bit more first. But just trying to think about keeping everything clean and organized.

I may be opening a bigger can of worms than intended here. :)
In general, our promise is "origin.js users don't need to know anything about blockchain or IPFS, they just use our library". So I'd consider exposing contractService for such a common pattern to be a slight breaking of that promise.
On second glance, perhaps its already a misstep that our methods return a web3 transaction. Can we instead return an abstracted "origin transaction", with a convenience method like isFinished()? Then the above code might look like this:
const transaction = await origin.listings.close(
      listing.address
    )
await transaction.isFinished()

@wanderingstan I agree with you!
Returning the web3 transaction should be a temporary thing -  we should definitely return our own instead.

@DanielVF cool. I'll rename this issue to reflect this design choice, and we'll put it in the backlog for a future sprint.

I'm coding this as we speak. 🛩

NOTE: There was discussion on discord around possibly naming the function whenMined
I don't like whenMined, as it requires our developers know something about blockchain, again breaking our promise that origin.js users don't need to know blockchain, only markerts.
All they care about is whether the thing is finished / done / complete. "Mined" doesn't mean much outside of the crypto world.

While I'm not super opinionated on this (hence my flip-flopping), I do tend to agree with @wanderingstan here. I think he makes a good case for keeping the name more generic. In terms of interfaces, I think a good principle is to hide as many details as possible. And origin.js of course is an interface. Do users of the exported class actually need to know that a block has been mined in order to use it? No. They just need to know that the thing has happened. Of course some people might be curious what's actually going on under the hood - and I believe proper documentation would be the place to explain that.

I believe we're done here...right @DanielVF ? Can this be closed?

No, not ready to be closed.
We're still returning what comes back from web3, plus some one of our own annotations. We've not decided yet exactly what we'll be returning.

Started in https://github.com/OriginProtocol/origin-dapp/pull/319 ☑


Missing omitempty tag

UserInfo field in ServiceInstanceSpec should have omitempty JSON tag. Because it is missing  ServiceInstance is not round tripped properly from and to json (and unstructured map[string]interface{} representation too) because userInfo field with null is added to the encoding result.
See https://github.com/kubernetes/community/blob/master/contributors/devel/api-conventions.md#optional-vs-required
I've just tried to update to new Service Catalog types and some of my tests blew up.

@ash2k - do you think there's a problem that our roundtrip tests should have caught but didn't? (In addition to the missing omitempty tag, I mean)

I'm not sure I understand the question, sorry.
I think a test like this one would be useful - where each type is converted to and from the "unstructured" representation (map[string]interface{}).

Closed by #1263


Not supported Node.isConnected nor Node.getRootNode()

Attending to the Node API are missing isConnected readonly property and getRootNode method

Feel free to put up a PR

Sorry I've taken a while to merge this in-- there's been a whole bunch of updates to the libdefs recently which cause conflicts. I will make sure this is in the 0.89 release.


can't set currency

I did the module update a few days ago. Now I do not receive any orders or payments via Stripe.
I have already uninstalled and reinstalled the module. Unfortunately, it still did not help.
Now I noticed in the settings of the payment methods that I no longer have a radio button or check box in the Currencies section. For the sections Groups, Countries and Delivery Services I can see and select the checkboxes.
you can see full message here
https://forum.thirtybees.com/topic/1619/stripe-bug-can-t-set-currency

Looks more like a core bug since modules can't influence the way this page is displayed.


samples: bluetooth: BTHome sensor template fix whitespace

Whitespace fix.

Please rebase to fix the manifest and assigner workflow failures.

@osaether I think you need to tab until you can't, and then spaces to align the remaining.

@osaether I pushed what I wanted if you don't mind. You just have to configure your editor to use the zephyr-provided .editorconfig, and it will do the right thing (start indenting w/ tabs instead of spaces).

Zephyr does not accept spaces at the start of a line so I will leave the (already approved) indent as it was. So now this PR is back to fix the single line.

While I don't think it's enforced anywhere, 0-N tabs in the beginning of a line is the only place where tabs should ever be found. Using spaces everywhere else tends to ensure that the alignment looks right regardless of if you're viewing the code in a web browser (on GitHub) or in your editor. Right now the PR adds tabs before \ on a couple of lines, which violates this rule.

@jhedberg where's that rule?

@jhedberg where's that rule?

Maybe I should have been clearer: it's neither enforced nor documented as a rule, as far as I know. However I'm recommending to follow it, since the practical consequences are as I outlined (we get the least messy result from having both tabs and spaces in this project).

edit: have to add that right now the code looks "right" on github & in both emacs (that loads editorconfig) & vim (that doesn't).

It really depends on luck, i.e. at what column of the line do you start adding tabs again (after having some non-tab characters).

I'm recommending to follow it

Not to be harsh, but that's just your personal preference at this point. If it's not enforceable/ed, we should at the very least document it somewhere as a project preference.
However I do agree with the rule you're proposing, I pushed an update removing those tabs.

I'm recommending to follow it

Not to be harsh, but that's just your personal preference at this point. If it's not enforceable/ed, we should at the very least document it somewhere as a project preference. However I do agree with the rule you're proposing, I pushed an update removing those tabs.

It's a bit more than a personal preference though: It is a rule that we've tacitly agreed to in the Bluetooth subsystem for years now, with most (if not all) contributors adhering to it.
Sometimes unwritten rules can be rules as well, and I do think this is one of these cases.


Implement Snapshot Testing (UI) with JEST

Don't need this because we are implementing Cypress E2E testing instead


How should we create multi manager in swarmkit?

As we know, we could create a manager using "swarmd -d /tmp/node-1 --listen-control-api /tmp/manager1/swarm.sock --hostname node-1",
But how to create another manager? and automatically failover when the first one crash?

@jmzwcn
In another node, join cluster
swarmd -d /tmp/node-2 --hostname node-2  --join-addr <first_node_ip>:4242

Then, promote node2 to a manager:
swarmctl -s /tmp/manager1/swarm.sock node promote node-2

Finally, you can use swarmctl node ls  to check.

When I run node1 and node2 in the same host, after promote node-2, it alert port conflict and node-2  will be down.
When I run two nodes in the different hosts,  node-2 could be promoted to a manager, but "-s /tmp/manager1/swarm.sock" will not work, node-2 will listen to "./swarmkitstate/swarmd.sock".
@jimmyxian, any problem for my step please?

I'm going to try to repro this right now. I changed the default directory in a PR a few days ago and maybe there is a mistake.
As a side note, the best way to test swarmkit is to use Docker Swarm Mode, which is embedded in Docker 1.12. The tools in the swarmkit repo are mostly there for hacking in development.

Ah, ok, your problem is this: the option to set the command API socket is --listen-comand-api. The -s option sets the secret.

[daniel@localhost ~]$ swarmctl --help
Control a swarm cluster
Usage:
swarmctl [command]
Available Commands:
node        Node management
service     Service management
task        Task management
version     Print version number of swarm.
network     Network management
cluster     Cluster management
Flags:
-n, --no-resolve      Do not try to map IDs to Names when displaying them
-s, --socket string   Socket to connect to the Swarm manager (default "./swarmkitstate/swarmd.sock")

@jimmyxian
About "swarmctl -s /tmp/manager1/swarm.sock node promote node-2", if node-1 and node2 don't  reside the same host, "/tmp/manager1/swarm.sock" will be different? How to ensure node-2 take over the node-1?

@jmzwcn
The default sock path in node-2 is ./swarmkitstate/swarmd.sock
You can use this sock file to talk with swarmkit in node-2.

Seems we couldn't run two managers in the same one host?

If you want to run two or more swarmkit manager on the same host. You should use different options when run swarmd.
Such as:
node 1:
swarmd
-d /tmp/node-1
--listen-remote-api <IP_ADDRESS>:4242
--listen-control-api /tmp/manager1/swarm.sock
--hostname node-1

node2:
swarmd
-d /tmp/node-2
--listen-remote-api <IP_ADDRESS>:4243
--listen-control-api /tmp/manager2/swarm.sock
--hostname node-2
--join-addr <IP_ADDRESS>:4242


Truncate bottom overflows for fixed height cards

Updates the TypeScript renderer to enable automatically truncating content that overflows the bottom of a fixed height card. The truncation logic is triggered in the updateLayout() method of the AdaptiveCard class, though it could be easily moved to the base container class if we ever want to be able to handle content overflows on any container, not just the bottom of the card. (This isn't really necessary at the moment since there's no notion of fixed height containers yet).
The default behavior for when an element overflows the bottom of the card is to hide the entire element. Two new methods are added to the CardElement class (truncateOverflows() and undoOverflowTruncation()) for elements that want to override this behavior (e.g. for TextBlocks, these methods are overridden so that text content can be partially clipped).
Example:

Everything is hidden behind the AdaptiveCard.useAdvancedCardBottomTruncation flag (defaults to false, since this should only be enabled for fixed height cards).
This addresses #517.

Nice!

Approved - let me know if it's ok to merge at this time or you plan other minor changes.

Feel free to merge 👍


[helm/alertmanager] Choose custom prometheus label value

The value of the prometheus label is fixed to Release.Name, this means that alertmanager and prometheus should be deployed on the same release. In cases where this is not desirable or possible is useful to be able to specify a custom value for that label.
This can be more flexible allowing any generic set of labels but I didn't want to complicate the PR for now and preferred to maintain compatibility.

@gianrubio I saw #1218, if that's going to be merged then maybe this can be included there, as it doesn't address this use case specifically. If not this will not cause conflicts anyway.

CI seems to be stuck

Rebased

thanks @richerve


Feature: Add Support for F# and VB Project Types

PR Details
This PR adds support for additional project types: .fsproj and .vbproj, enabling F# and Visual Basic coding in Stride's code-only approach. Further work may be required to integrate F# and Visual Basic into the Stride Game Studio, which I will leave to more experienced contributors.
Description
This PR serves as a follow-up to the incomplete https://github.com/stride3d/stride/pull/1078. I've implemented the changes necessary for a "code-only" approach, as I'm currently not familiar with Stride Game Studio.
To accommodate these changes, I've introduced a SupportedProgrammingLanguage record and a SupportedProgrammingLanguages class. I opted for these names as SupportedLanguage is already used within the codebase for a different purpose.
I'm open to feedback on the naming conventions and the overall implementation. The core change involves extending the existing logic to account for .fsproj and .vbproj in addition to .csproj.
Related Issue
https://github.com/stride3d/stride/pull/1078
Typical error when using other project types
Severity	Code	Description	Project	File	Line	Suppression State
Error	0.157s	[AssetCompiler] Unsupported file extension (only .sln, .csproj and .sdpkg are supported)	Example06_VBasic_Basic3DScene	D:\Projects\GitHub\stride-community-toolkit\examples\code-only\Example06_VBasic_Basic3DScene\EXEC	1

Motivation and Context
Allowing to use other .NET languages as they should be working.
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My change requires a change to the documentation - announcing this feature
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Looks good, thanks @VaclavElias !
Maybe IsSupportedByExtension could be changed to something a bit less vague, like IsProjectExtensionSupported ?

Of course, that is much, better, let me fix that :)

Changed.

Thanks !

My pleasure. Once we release the new NuGet packages, I can create more examples, and we can begin promoting this exciting option in a sensible manner for other developers to explore and experiment with.


pyenv plugin refactor (8x faster)

The current version of the pyenv plugin searches for pyenv using a predefined list of paths. If found, the plugin adds the pyenv bin directory to the user's PATH variable, and finally runs pyenv to load it into the environment. This is backwards. No matter which pyenv installation method you use, the pyenv utility will already be in your path.
I've refactored the plugin to simply load pyenv directly. It will also load pyenv-virtualenv if it's available, as in the original version. If pyenv is not in the user's path, it gracefully falls back to displaying the system-installed python version in the prompt.
Speed Increase
After refactoring, I noticed this made the plugin much faster. Let's profile it (with python of course):
profiler command: $ python -m timeit "__import__('subprocess').call(['zsh', '/Users/cyphus/.oh-my-zsh/plugins/pyenv/pyenv.plugin.zsh'])"

Original:
10 loops, best of 3: 822 msec per loop
Refactored:
10 loops, best of 3: 103 msec per loop.

This PR makes the pyenv plugin load 8x faster, while fixing underlying issues. Winner winner 🍗 dinner.
Issues closed by this PR

#6142
#5884
#6139
#6017
#4674 (Note: Possibly. This looks to be an issue of stemming from initializing two separate pyenv installations into the same shell. The first one is initialized based on the user's PATH, and the second is initialized based on the plugin's search order. This PR would change the pyenv plugin to also initialize based on the user's PATH, it would initialize the same pyenv. I can't say for sure without recreating the user's issue.)

Here to add my $0.02: The default-provided pyenv plugin (for some reason or another) only insists on using the /opt/... path for looking for pyenv version installations. The PR here as-is works perfectly and I'll be using this one in place of the default one. Thanks @cyphus!

Any progress on merging this? Anything that would make pyenv faster (while keeping functionality intact) is a big gain!

Thanks for the support on this.
I read about oh-my-zsh's plugin maintainers in the contributing docs. I'd like to offer to be the official maintainer for the pyenv plugin. As far as my qualifications, well, I read a ton of pyenv documentation and source code when developing these changes, and I may as well continue to put that knowledge to good use. Other than that, hopefully the code speaks for itself.

I really think this should be merged as soon as possible. I believe that its the only pyenv-related PR that really addresses the root problem with the default pyenv plugin. Not only does the current plugin use the Homebrew installation directory to install versions into (by setting PYENV_ROOT to this directory, which of course gets overwritten on an upgrade, destroying everything that you've done with pyenv), but there is, of course, the egregious performance issues that you outlined. The choice that it makes of setting PYENV_ROOT to be the Homebrew installation directory is not only a terrible idea (I'm so devistated that I lost all my packages (custom and otherwise) when I upgraded pyenv), but it goes against the settings recommended by the pyenv package itself!
Honestly, it sort of beyond me that this has been overlooked for this long. I now, of course, am using this pull request's version of the plugin, but this is something that I think should really be changed as soon as possible to avoid people losing their Python packages.
If this were to be merged, I think that it would be a great idea to include a note in the upgrade process to move the versions from the Homebrew install directory typically (/usr/local/opt/pyenv/versions/) to the pyenv user directory (typically ~/.pyenv/versions/).

Thanks @cyphus!

Thanks for merging this!
Out of curiosity, would this also work for the rbenv plugin? https://github.com/robbyrussell/oh-my-zsh/blob/master/plugins/rbenv/rbenv.plugin.zsh

which advantage have now the plugin if i need to define my $PATH BEFORE the plugin install ? the normal way is redefine your $PATH in the # User configuration AFTER the plugin activation in your ~/.zshrc file


DCHECK fixes

Description
This is a series of commits to fix issues that cause V8 debug check (DCHECK) failures when running the test suite using debug-enabled V8 libraries. Each commit message contains a more specific description, so I recommend reviewing it commit-by-commit.
(The last commit is not related to DCHECK fixes — it pulls in an update to common-runtime-tests-app which will be needed to upgrade V8 to 11.x.)
Related Pull Requests

https://github.com/NativeScript/common-runtime-tests-app/pull/23 (the common-runtime-tests-app update brought in by the last commit)

Does your pull request have unit tests?
Most of these changes are either covered by existing tests, or are validated by the DCHECK failures no longer occurring. The commit "Avoid setting the same property on an ObjectTemplate twice" has tests specifically for an API that I found had both an Android definition and a Kotlin extension, just to confirm that the new code handles it.

LGTM! Actually quite surprised by the dated GetImplementedInterfaces implementation. Nice catch!
Regarding https://github.com/NativeScript/android/pull/1764/commits/7a479c81c0a19af27df64337b3013e9b26149274, do you know if the properties were overwritten even if the DCHECK failed? Asking mostly because if the DCHECK failed but it did overwrite anyway, it'd change the behavior (previously it would set to the last property written, now to the first).

Good question, I assumed it was the case but it bears following up on.
The DCHECK fails because the ToLocalChecked() in ctorFuncTemplate->GetFunction(context).ToLocalChecked(); is called on an empty handle. So the failure happens quite a lot later than when the second property is added to the template. What I believe happens is that the template does "contain" both properties (you can see them in Object.keys of the prototype object) but only one can be accessed.
The test I added in https://github.com/NativeScript/android/pull/1764/files#diff-4187621cb8f9cc3730f47a10d0ba131f90f83cf0321251b0ea66ebdf9493b265R119 passes both with and without the patch, so that suggests that the behaviour of which function is called is the same.
For android.os.Handler.postAtTime there are three possible functions:

Java, 2-argument overload: https://developer.android.com/reference/android/os/Handler#postAtTime(java.lang.Runnable, long)
Java, 3-argument overload: https://developer.android.com/reference/android/os/Handler#postAtTime(java.lang.Runnable, java.lang.Object, long)
Kotlin extension: https://developer.android.com/reference/kotlin/androidx/core/os/package-summary#(android.os.Handler).postAtTime(kotlin.Long, kotlin.Any, kotlin.Function0)

What I've observed is that the 2-argument Java version is called if you pass 2 arguments, and the Kotlin extension overshadows the 3-argument Java version, so the Kotlin one is called if you pass 3 arguments. The patch doesn't seem to change this.
I'm not 100% sure of this, but I think from my understanding of the code, it doesn't matter whether the first or last property is kept - they both have the same MethodCallbackData. The distinction is only made at runtime in MetadataNode::MethodCallback() where the overload is selected based on the number of arguments passed to the JS function.
Here's the only difference in behaviour that I see with and without the patch.
console.log(Object.keys(android.os.Handler.prototype).filter(name => name === 'postAtTime'));
// output before: [postAtTime, postAtTime]
// output after: [postAtTime]

I believe that additionally, without the patch, the instantiation of the constructor function template is never added to the cache in MetadataNode::GetConstructorFunctionTemplate() either. I'm not certain what that affects, though, because you can still use the constructor function like new android.os.Handler().


Broken installation with installer

Description of the problem
After installing latest version of Coq I get the following message:

(coqide.exe:12032): GdkPixbuf-WARNING **: Cannot open pixbuf loader module file 'C:\Coq-Platform~8.15~2022.04\lib\gdk-pixbuf-2.0\2.10.0\loaders.cache': No such file or directory
This likely means that your installation is broken.
Try running the command
gdk-pixbuf-query-loaders > C:\Coq-Platform~8.15~2022.04\lib\gdk-pixbuf-2.0\2.10.0\loaders.cache
to make things work again for the time being.

the same with lower version:

(coqide.exe:4012): GdkPixbuf-WARNING **: Cannot open pixbuf loader module file 'C:\Coq-Platform~8.14~2022.04\lib\gdk-pixbuf-2.0\2.10.0\loaders.cache': No such file or directory
This likely means that your installation is broken.
Try running the command
gdk-pixbuf-query-loaders > C:\Coq-Platform~8.14~2022.04\lib\gdk-pixbuf-2.0\2.10.0\loaders.cache
to make things work again for the time being.

running presented commands does not help
there is no directory "gdk-pixbuf-2.0" in "C:\Coq-Platform~8.15~2022.04\lib"

Coq Version
8.15.2 and 8.14.2
(installed with binary installer "Coq-Platform-release-2022.04.1-version_8.15_2022.04-arch-x86_64_signed.exe" on Windows 10 Home)

This is a duplicate of (https://github.com/coq/platform/issues/212).
As mentioned there, CoqIDE doesn't use any dynamic pixbuf loaders (it doesn't load JPEG images and the like), so this does not hurt. Of course one could add them to silence the warning.
If you have any other issue with CoqIDE (say that it actually does not work), please create an issue describing the symptoms.


Update notifications.js

inject into service also

fixes #55

Merged and and released to npm :+1:


Implemented authentication with simple database

Make sure you have 2 new Python packages installed: flask-login and SQLAlchemy

Hi, I'm wondering, if a front end user is signed in via the implemented authentication process, is their data on the server side stored as an encrypted file (so mainly talking about the state files here)? And how will this work with multi user labeling tasks?


[Bug] Integration not loading after update to 0.19.0

The problem
After updating to 0.19.0, the integration is not loading, with this in the log (added in the details box)
My enviroment is an aimesh setup, with a TUF-AX5400 as main, another one as node, and a RT-AC57U V3 as another node.
Working without any problem until the update :/
Firmware <IP_ADDRESS>.386_50224-gc45ea49 (downgraded from 388 which is a complete MESS with aiMesh...)
Your device model
TUF-AX5400
Firmware type
Stock
Firmware version
<IP_ADDRESS>.386_50224-gc45ea49
Integration version with the issue
0.19.0
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
2023.2.2
Is there anything useful in the logs?
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/asusrouter/asusrouter.py", line 311, in async_identify
    data = item.method(raw[item.value]) if item.method else raw[item.value]
KeyError: 'serial_no'

Diagnostics information
No response
Additional information
No response

Hey,
Sorry to hear you are experiencing problems with AsusRouter
The error message says that the serial number of the device was not properly fetched from the device NVRAM. It is required in order to create a unique_id for the device in HA. When downgrading from 388 to 386 build, did you reset your device or just installed 386 on top of 388?
Could you please also try restarting the device with the error and trying to reload the integration (or restart HA) afterwards? I know, that might be obvious, but actually, there are quite several bugs with the Asus API, that are resolved with a simple restart (or reboot of the whole AiMesh if in mesh mode)

Thanks for the support. The downgrade was made weeks ago (in fact I'm downgraded now the component to 0.18.4 and it's working again), and it was with a full reset.
Is there any way of knowing with of the three routers is causing the problem? Restarting HA didn't work (but has worked after downgrading the integration...).
Thanks again

If all 3 devices are set up in the HA, the one that gets into the error with KeyError: 'serial_no' in the log should probably become unavailable (all of its sensors and other entities will be greyed out and have unavailable state)

Ok, after downgrading and upgrading again the integration, and rebooting home-assistant a couple of times, now magically works. Unexplainable but, as you were saying, it seems to be a not-so-reliable API.
Thank you for the support and the amazing work.

👍🏻 Great that it works now. In case you will be experiencing any other problems, please open a new issue


Add better support for development in gulp watch

Added dev server as a gulp task, can be used by running yarn dev


useNavigation

On importing useNavigation() an error appear.
The screenshot is provided below.

I think you're using useNavigate hook outside of the router.

Yep, you need to use it inside the Router component


Can you please add some example code of receving snmp V3 Traps and Notifications

Can you please add some example code of receving snmp V3 Traps and Notifications

How about these examples?

Thanks lot. It's useful.
And how can the manager deal with the received OIDs usually ? (how to interpret the OIDs)
And how can I store the received data into ElasticSearch or forward to Kafka ?

In pysnmp, OID acts like a tuple of integer sub-OIDs. Or as a human-friendly string of dot-separated integers. So manager can use either of these two representations.
Take a look at how people deal with pysnmp and MongoDB for inspiration. ;)

@etingof it seems that snmplabs.com is down


PowerShell Compatibility Issue

Params not parsing when server is running under PowerShell 5.1
The following server code running under 5.1.17763. 503
server {
    listen localhost:8080 http

    route post '/users' {
        param($s)

        # get the user
        $user = $s.Data['userId']
        $age = "21"

        # return the user
        json @{
            'Username' = $user;
            'Age' = $age;
        }
    }
}

Client code
[Net.ServicePointManager]::ServerCertificateValidationCallback = {$true}
$params = @{"userId"="someguy5";
 }
 $Json = $params|ConvertTo-Json
$response = Invoke-WebRequest -Uri 'http://localhost:8080/users' -Method Post -Body $Json -ContentType 'application/json'
Write-Host "Response - $response"

Produces this output:
Response {"Username":{},"Age":"21"}
The same server code and client code  running under PowerShell 6.1.2 parses the params correctly and produces this
Response {"Username":"someguy5","Age":"21"}
Documents say
Cross-platform using PowerShell Core (with support for PS4.0+)

Try:
server {
    listen localhost:8080 http

    route post '/users' {
        param($s)

        # get the user
        $user = $s.Data.userId
        $age = "21"

        # return the user
        json @{
            'Username' = $user;
            'Age' = $age;
        }
    }
}

I was just about to recommend this!
ConvertFrom-Json in PS4/5 uses a psobject, so $data['userId'] doesn't work unfortunately.
A gab in documentation, so if $s.Data.userId works, I'll add a note in for PS4/5.

I tested $s.Data.userId and it does work. Thank you

Excellent! I'll look at updating the docs.


Update testing docs

Changes references to run_tests to the more up-to-date azdev test.  Updates the syntax for the test script based on recent changes.

View a preview at https://prompt.ws/r/Azure/azure-cli/5508
This is an experimental preview for @microsoft users.


Add billboard option

Background

v7.0 API audit
For #2834

Change List

Add billboard option in both TextLayer and IconLayer, default to true - facing camera as the old code, so this should not impact the users.
Add render test

Coverage decreased (-0.001%) to 60.86% when pulling ae6fa0bf1a2b29e527ae4c4af8d3ff67d0cfba14 on xx/billboard into 0ed3cbba635469aa178afb0ca8dd218d356bc1ce on master.


refactor: network

I'm refactoring ES6 classes to functions. Functions are more friendly for tree-shaking. /cc @Songkeys


♻ Simplify PRNG import by defining struct externally

Description
I moved the PRNG and LazyShuffler structs outside of library LibPRNG so that they could be individually imported and you didn't always have to type LibPRNG.PRNG.
Note: This is a breaking change.
Checklist
Ensure you completed all of the steps below before submitting your pull request:

[x] Ran forge fmt?
[x] Ran forge snapshot?
[x] Ran forge test?

😱 I’m allergic to the inconsistency. 😱

😱 I’m allergic to the inconsistency. 😱

What inconsistency? Is this not done for the other libs, I can open larger PR to move it all. Although CI is failing me because it's not compiling for older versions, I thought top-level using statements were introduced in 0.8.4

I prefer the types to be in the library itself.
The verbose ritual of typing LibPRNG.PRNG reminds me that this struct belongs to the LibPRNG namespace.
It gives me dopamine.

Also, this has been on the wish list on sometime.
Not sure the team deprioritised this small syntax sugar QoL thingy over generics.
https://github.com/ethereum/solidity/issues/1100


ComponentVersionTelemetryInitializer is not having implementationType in services

As a result, customers can never get access to this service, when they check the initializers through ImplementationType property of service.

Fixed it as a part of : https://github.com/Microsoft/ApplicationInsights-aspnetcore/pull/163


Accuracy System

What feature do you want to get added on the base engine?
Seriously needs to be reworked.
I'd get 53 sicks and 2 goods would bring down all the way to 99.09.
Also color the ratings I can't see what I got because its all white.
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

Also color the ratings I can't see what I got because its all white.

true

Also color the ratings I can't see what I got because its all white.

true

but you can still edit the images to be colored.

that's honestly the main reason I despise pysch engine

Also color the ratings I can't see what I got because its all white.

true

but you can still edit the images to be colored.

yeah but that would mean I would have to do that with EVERY SINGLE MOD

Also color the ratings I can't see what I got because its all white.

true

but you can still edit the images to be colored.

yeah but that would mean I would have to do that with EVERY SINGLE MOD

I’ll just make a pr.

Also color the ratings I can't see what I got because its all white.

true

but you can still edit the images to be colored.

yeah but that would mean I would have to do that with EVERY SINGLE MOD

I’ll just make a pr.

?

Make a pull request?

Nah

Nah, not every FNF engine has to work in the same way regarding judgements/ratings etc

true very true but when your accuracy is this inaccurate then we have a problem
the ONLY song I feel works for this is Ballistic and Ballistic Remix

My brother in Christ there are gameplay modifiers you can use
Playing a rhythm game that's actually meant to be hard is also an option

My brother in Christ there are gameplay modifiers you can use Playing a rhythm game that's actually meant to be hard is also an option

its not hard coded is the problem

My brother in Christ there are gameplay modifiers you can use Playing a rhythm game that's actually meant to be hard is also an option

its not hard coded is the problem

wdym?

My brother in Christ there are gameplay modifiers you can use Playing a rhythm game that's actually meant to be hard is also an option

its not hard coded is the problem

wdym?

if the health is hardcoded to be hard then that actually feels like its hard not me manually having to do it
and it really doesn't matter if my eyes are glued to the accuracy

My brother in Christ there are gameplay modifiers you can use Playing a rhythm game that's actually meant to be hard is also an option

its not hard coded is the problem

wdym?

if the health is hardcoded to be hard then that actually feels like its hard not me manually having to do it and it really doesn't matter if my eyes are glued to the accuracy

oh


[Network] application gateway 100% support

Description

Testing Guide

History Notes

[Network] az network application-gateway show-backend-health: support probe operation arguments.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

network


udocker seems to not work

Issue description
I just discovered udocker and i can't use it on NixOS
Steps to reproduce
Pulling image seems good
udocker pull alpine
Warning: no engine available and autoinstall disabled
Downloading layer: sha256:e7c96db7181be991f19a9fb6975cdbbd73c65f4a2681348e63a141a2192a5f10
Downloading layer: sha256:a3ed95caeb02ffe68cdd9fd84406680ae93d633cb16422d00e8a7c22955b46d4

Trying to run image fail because it try to mount some specials folders
udocker run alpine
Warning: no engine available and autoinstall disabled
Error: invalid host volume path: /lib/modules

The documentation says:

--nosysdirs prevent udocker from mapping /proc /sys /run and /dev inside the container

Trying without mounting folder fail because it can't find proot
udocker run --nosysdirs alpine
Warning: no engine available and autoinstall disabled
Error: proot executable not found

Start a shell with proot still fail because it can't find proot
nix-shell --command fish --packages proot
udocker run --nosysdirs alpine
Warning: no engine available and autoinstall disabled
Error: proot executable not found

Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.

system: "x86_64-linux"
host os: Linux 4.19.48, NixOS, 19.03.172844.caacbe98ce1 (Koi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.2.2
channels(pinage404): "home-manager-19.03, nixos-19.03.172852.5121b483edc, nixos-unstable-19.09pre182256.168d1031af4"
channels(root): "nixos-19.03.172852.5121b483edc"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

It could still be important.


Address wrong for runnable scripts

When it is said You can find a runnable script for each tutorial here the address is wrong (error 404).

@sbacelar thanks could you give a full link to a page where you found this wrong link?
the links at the top of the pages like here: https://juliaai.github.io/DataScienceTutorials.jl/data/loading/ should all work fine but I'm not sure whether you're referring to this or something else

ah on the README ok thanks


[lcars/notanalog/limelight/daisy/thering/widviz/90sclk/widviztime/neonx] Add widget utils

I've converted the remaining clocks and widgets in #2196 to use widget utils.
I have not tested any of them and don't really intend to.
Especially widviz and widviztime I don't know if I've messed up or if they are ok. I can drop them from the PR if that's better.
For the others I feel it could be ok to merge them and if bugs come up we can fix them then?

Updated per suggestions.

Thanks, but I don't think you quite updated the widgets properly... See the comments - GitHub's message thing is annoying as I left the comment for saved=true here instead next to  saved = []; - but it looks from the section posted above like I meant it for var saved = null;

Yeah I see, if I were a little more observant I could have worked that out, sorry :P

Great - thanks!


Not compatible with php7.2+

This module does not seem to be compatible with php7.2, or at very least it's composer.json states it is only compatible with 7/7.1

Also is locked to specific magento/framework, magento/module-backend, magento/module-config, magento/module-cms versions

Closing as this repo is no longer maintained and will be marked as read-only.


Add string for NoteAir2

Used the GetBooxUpxKeys App to fetch strings.
Other helpful resources for posterity:
https://github.com/bkerler/edl/issues/200
https://www.mobileread.com/forums/showpost.php?p=4187479&postcount=52

Could you please provide the fingerprint of your firmware (the line starting with # fingerprint:)?
It can be used for getting a update.upx from onyx web api.

Sure thing! Below is the stock fingerprint before root/updating

#fingerprint: Onyx/NoteAir2/NoteAir2:11/2021-10-27-11-45-3.2_69979ac6ac/31052:user/dev-keys

Sure thing! Below is the stock fingerprint before root/updating

#fingerprint: Onyx/NoteAir2/NoteAir2:11/2021-10-27-11-45-3.2_69979ac6ac/31052:user/dev-keys

Thanks to your fingerprint, an update.upx was found in http://data.onyx-international.cn/api/firmware/update?where={"buildNumber":0,"buildType":"user","deviceMAC":"","lang":"en_US","model":"NoteAir2","submodel":"","fingerprint":"Onyx/NoteAir2/NoteAir2:11/2021-10-27_11-45_3.2_69979ac6ac/31052:user/dev-keys"} and the strings are verified now.


[Feature request] Support generate test report

HI team,
We love your work ! keep it up, please help support the feature of support generate test report and/or related API for the feature.
Tons of thanks

This is tracked in #171.
Thanks


MySQL and transactions deadlock check

The newer versions of MySQL have some logic about the transaction deadlocks:
if there are more than 200 active writing transaction you will get a transaction deadlock. The same is true if you acquire more than 1 million row locks.
There is a setting to turn off this behaviour and only use a transaction timeout in seconds. If the transaction doesn't finish till the deadline than it will be roll-backed.
Also this behaviour means that you should not use connection pools with more than 200 connections to the database?! We could loose configuration parity with PostgreSQL database. I.e. one variant of the framework is using 256 connections to the PostgreSQL, another 256 connections to the MySQL. The second one will be problematic for the update test-case.

A pull request is submitted - connections are decreased to 192.

Accepted.


0 Inputs working at all in World

On my controller and keyboard, nothing is working. I dragged the Scripts folder and replaced DInput8.dll with the one in the zip. Why?

Specify the game and its executable.

World is at alpha stage, this is known. It'll work randomly. I yet have to find out why this is the case.


Idea: set up an Azure function (PnP Powershell ) that monitors the update rate of the SharePoint search crawler

Not sure if the SharePoint Search Crawler picks up your changes in a timely fashion?
This azure function will update a list item and check if the update is picked up by Search within x minutes.

This idea has grown into a minor project and not fit as a sample


kaizen: vanity import path

Worth mentioning that the path is: quamina.net/go/quamina
Signed-off-by: Tim Bray<EMAIL_ADDRESS>

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

Coverage data is based on head (9cf7562) compared to base (c402b54).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #182   +/-   ##
=======================================
  Coverage   95.93%   95.93%
=======================================
  Files          17       17
  Lines        2067     2067
=======================================
  Hits         1983     1983
  Misses         60       60
  Partials       24       24

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

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

@timbray to fix the "out-of-date" warning please use this (non-default) option

Quiet nice and easy instead of doing this locally.

Question: Your current redirect returns:
<meta name="go-import" content="quamina.net/go/quamina git https://github.com/timbray/quamina">

IMHO we should follow this guidance:
When using modules, an additional variant of the go-import meta tag is recognized and is preferred over those listing version control systems. That variant uses "mod" as the vcs in the content value, as in:
https://pkg.go.dev/cmd/go#hdr-Remote_import_paths

IMHO we should follow this guidance:
When using modules, an additional variant of the go-import meta tag is recognized and is preferred over those listing version control systems.
That variant uses "mod" as the vcs in the content value, as in:

<meta name="go-import" content="example.org mod https://code.org/moduleproxy">
This tag means to fetch modules with paths beginning with example.org from the module proxy available at the URL https://code.org/moduleproxy. See https://golang.org/ref/mod#goproxy-protocol for details about the proxy protocol.

But that would imply that quamina.net should implement the GOPROXY protocol, and have to respond to five different endpoints. So I would actually have to build a real website there at quamina.net so that it would track versions and provide lists and so on.  If the current simplified form actually works it's
Or am I missing something?

Source: https://pkg.go.dev/cmd/go#hdr-Remote_import_paths