Zoom - Chat Support

Checklist

[X] I'm requesting a site-specific feature
[X] I've verified that I'm running yt-dlp version 2022.05.18 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Southern California, USA
Example URLs
https://zoom.us/rec/play/6JV_c-36-DM3T9GQsASDC_F7W46_J6msgSQZ-vcEn0q9VCVVNlKiNLcaYuNMjek9LoEdvtcD0YiquC00
Description
Please add support for zoom’s chat replays. They can be accessed via a button.
Verbose log
Not attempted

Not attempted

Can you please attempt it (using --list-subs)

--write-subs --sub-langs cc gives what you want.


TypeScript Client generator is generating PascalCase model prop references instead of camelCase - resulting in undefined values

Hi, I did come across an issue after migrating our app project to .NET 8.
The type script client generator is now incorrectly referencing the model properties as PascalCase rather than camelCase resulting in undefined objects.
Our web api is correctly passing the properties as camelCase, the typescript client generated the API model props in the same way (camelCase), but the code referencing the model props from the http response is using PascalCase, please see:

I do use MSBuild and the NSwagExe_Net80 command based on nswag.json file generated via NSwagStudio:
<Target Name="NSwag" AfterTargets="PostBuildEvent">
  <Message Importance="high" Text=" " />
  <Message Importance="high" Text="Running NSwag generate..." />
  <Exec WorkingDirectory="$(ProjectDir)" EnvironmentVariables="ASPNETCORE_ENVIRONMENT=NSwag" Command="$(NSwagExe_Net80) run nswag.json /variables:Configuration=$(Configuration)" />
</Target>

Used packages:

Minimal Api config:
services.AddOpenApiDocument(config =>
 {
      if (swaggerOptions != null)
      {
          config.DocumentName = "v1";
          config.Title = swaggerOptions.Title;
          config.Version = "v1";

          config.AddSecurity(swaggerOptions.Scheme, Enumerable.Empty<string>(), new OpenApiSecurityScheme
          {
              Type = OpenApiSecuritySchemeType.ApiKey,
              In = OpenApiSecurityApiKeyLocation.Header,
              Name = HeaderNames.Authorization,
              Scheme = JwtBearerDefaults.AuthenticationScheme,
              BearerFormat = "JWT",
              Description = "Copy 'Bearer ' + valid JWT token into field"
          });

          config.OperationProcessors.Add(new AspNetCoreOperationSecurityScopeProcessor(swaggerOptions.Scheme));
      }
  });

if (app.Environment.IsDevelopment())
{
    app.UseOpenApi();
    app.UseSwaggerUi();
}

Please help...what can I do about this, it worked perfectly before in .NET 7 before the migration?
Thank you.

Can you show openapi doc with schema part?
I'm asking because in the default example, everything is working fine:

(maybe you have different settings?)
I had similar issue, with camelCase but that's because of some settings;
you can always override by ex.
document.ApplySettings(CreateNewtonsoftSchemaWithCamelCaseStrategy(), null);

 private static NewtonsoftJsonSchemaGeneratorSettings CreateNewtonsoftSchemaWithCamelCaseStrategy()
     => new()
     {
         SchemaType = SchemaType.Swagger2,
         SerializerSettings = new JsonSerializerSettings
         {
             ContractResolver = new CamelCasePropertyNamesContractResolver
             {
                 NamingStrategy = new CamelCaseNamingStrategy()
             },
             ReferenceLoopHandling = ReferenceLoopHandling.Ignore,
         }
     };


No support for aliyun cs ListRepoSyncTask

Type of Demand:
Aliyun CLI does not support aliyun cs ListRepoSyncTask (see output below).

╰─ aliyun cr ListRepoSyncTask
ERROR: product 'cr' need restful call
Use `aliyun cr {GET|PUT|POST|DELETE} <path> ...`

This API is documented at https://www.alibabacloud.com/help/doc-detail/145283.htm
Also, there is no documented way to use the restful interface such as aliyun cr GET <path> to get call ListRepoSyncTask.

@JacksonTian it would be nice if you mention in which version of the cli this has been fixed.

The latest version(v3.0.137) has supported it.


Consider adding mechanism to register alternate Sinks with NewMetricSinkFromURL

In my case, I'd like to use the provided sub-package Dogstatsd sink but without loosing the abstraction this library provides.
While I could write my own config/setup method that parses URL and does something different if protocol is dogstatsd:// (i.e. instantiates dogstatsd sink directly instead of using NewMetricSinkFromURL) it seems undesirable to reinvent that wheel.
Proposal
Allow external packages to register syncs early in execution (i.e. in func init()) that can hook into the central mechanism.
That might look something like this:

// Existing registry
var sinkRegistry = map[string]sinkURLFactoryFunc{
	"statsd":   NewStatsdSinkFromURL,
	"statsite": NewStatsiteSinkFromURL,
	"inmem":    NewInmemSinkFromURL,
}

// Add new mutex - might be unnecessary if we document that it's only safe to register in
// `init` methods but I haven't found spec to say those are guaranteed not to be run in
// parallel yet either.
var registryMu sync.RWMutex

func RegisterSyncForScheme(scheme string, factory sinkURLFactoryFunc) {
    registryMu.Lock()
    defer registryMu.Unlock()

    sinkRegistry[scheme] = factory
}

// Existing func
func NewMetricSinkFromURL(urlStr string) (MetricSink, error) {
    // ...
    registryMu.RLock()
    sinkURLFactoryFunc := sinkRegistry[u.Scheme]
    registryMu.RUnlock()
    /// ...
}

Then the included additional sinks or custom sinks defined in outer packages can use the same method to be instantiated at runtime.
I made an issue rather than just a straight PR with that code in just to check that there is no good reason this wasn't done already. If you agree it's worthwhile @armon I can make a PR.

Golang spec says:

Package initialization—variable initialization and the invocation of init functions—happens in a single goroutine, sequentially, one package at a time.

https://golang.org/ref/spec#Package_initialization
So we could go without mutex and document that it's only valid to call Register... from func init.

@banks I like it! Seems like a good way to make it extensible


Get SRA experiment data

Added code to get more experiment data from SRA that can be associated with *.sra or *.fastq files that are downloaded.

Thanks! I modified it a bit to work with FASTQ files. I think it should still work with SRAs but I couldn't get any to download.


PrimitiveDateTime::assume_offset & OffsetDateTime::to_offset can result in an invalid underlying PrimitiveDateTime

An OffsetDateTime can have an underlying PrimitiveDateTime that is invalid on its own. This can be constructed by doing datetime!("9999-12-31 23:59:59").assume_offset(offset!("-12")), resulting in an underlying PrimitiveDateTime of
PrimitiveDateTime {
    date: Date {
        year: 10_000,
        ordinal: 1,
    },
    time: Time {
        hour: 11,
        minute: 59,
        second: 59,
        nanosecond: 0,
    },
}

PrimitiveDateTime cannot validly hold a year with more than four digits (or six, if the large-dates flag is enabled). As such, this is a bug. It is not limited to internal APIs, as this causes a Unix timestamp roundtrip to fail (the "from" conversion will return Err). This does not affect formatting, as the date and time displayed are the values in the attached offset.
As to the realistic effect of this: As long as a user is not up against the literal bounds of valid data, there is no issue whatsoever. But if a user were to do some perfectly valid code as exampled above, they might wind up in an unexpected situation.
Now, what should be done about this? There are a few options.

Declare this behavior unspecified, similar to how large dates are not specified in time 0.2. I do not like this option, as it could potentially lead to panics or the user falsely assuming something.
Allow the internally invalid values, but change the attached UtcOffset from UTC to something else (what?), such that conversion a Unix timestamp roundtrip always succeeds. Users may be expecting the offset returned from these methods to be UTC, however (including myself; see #305 for a request to formalize this).
Store the internal PrimitiveDateTime in terms of the attached UtcOffset, not UTC. While feasible for OffsetDateTime, this could prove disastrous down the road when it comes to an implementation using tzdb, as similar behavior would be expected. This would then mean that it is not possible to determine the actual moment in time that should be represented, as clocks change on occasion (notably daylight saving time), causing ambiguities in the local time.
Accept the status quo, formalizing that OffsetDateTime::from_unix_timestamp{_nanos} always returns an offset of UTC. The downside of this is that users who push the type boundaries may not be able to construct an OffsetDateTime from a Unix timestamp generated by valid code.

As I type this out, I realize that the addition of an OffsetDateTime::from_unix_timestamp{_nanos}in_offset method or similar could be feasible. This would alleviate the concerns, so long as the caller provides a UtcOffset of the same or larger magnitude as the original.

Make PrimitiveDateTime::assume_offset and OffsetDateTime::to_offset fallible. This would inconvenience the overwhelming majority of users, as any use not at the type boundaries would return a success value.

Upon initial thought, my intuition is pushing me towards (4), as it maintains the existing behavior, requires minimal changes, and would only inconvenience those who run up against the type boundaries.

Resolved the behavior as option (4).

Resolved the behavior as option (4).


Fix memory issues in ScrollView::MessageReceiver

This PR fixes memory issues that caused https://github.com/tesseract-ocr/tesseract/issues/3869. The first commit fixes the actual bug (unique_ptr::release() should be used instead of unique_ptr::get()). The second fixes the underlying problem of using plain pointers that caused the bug in the first place.
Fixes https://github.com/tesseract-ocr/tesseract/issues/3869.
Supersedes https://github.com/tesseract-ocr/tesseract/pull/3870
FYI @rmast @stweil.

@rmast Could you please check whether this PR fixes the issue you were seeing. I didn't try to reproduce the problem, only diagnosed the problem by looking into source code.

I still get an error when running with valgrind:
$ valgrind bin/debug/clang/tesseract -c invert_threshold=0.9 --dpi 300 -l Latin -c textord_debug_tabfind=1 -c textord_debug_bugs=1 -c tessedit_reject_block_percent=1 -c tessedit_reject_row_percent=1 -c debug_noise_removal=1 https://user-images.githubusercontent.com/3341558/179373903-ef6cc246-f4e5-4633-a762-ded4dd22708f.jpg output82
==1212620== Memcheck, a memory error detector
==1212620== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==1212620== Using Valgrind-3.16.1 and LibVEX; rerun with -h for copyright info
==1212620== Command: bin/debug/clang/tesseract -c invert_threshold=0.9 --dpi 300 -l tessdata_fast/script/Latin -c textord_debug_tabfind=1 -c textord_debug_bugs=1 -c tessedit_reject_block_percent=1 -c tessedit_reject_row_percent=1 -c debug_noise_removal=1 https://user-images.githubusercontent.com/3341558/179373903-ef6cc246-f4e5-4633-a762-ded4dd22708f.jpg output81
==1212620==
Vertical skew vector=(7,13173)
Starting sh -c "trap 'kill %1' 0 1 2 ; java -Xms1024m -Xmx2048m -jar ./ScrollView.jar & wait"
ScrollView: Waiting for server...
Socket started on port 8461
Client connected
Click at (855, 2749)
Click at (855, 2749)
==1212620== Thread 2:
==1212620== Invalid read of size 8
==1212620==    at 0x54AF90: tesseract::SVEvent::~SVEvent() (scrollview.h:70)
==1212620==    by 0x6013EA: std::default_delete<tesseract::SVEvent>::operator()(tesseract::SVEvent*) const (unique_ptr.h:85)
==1212620==    by 0x60050C: std::unique_ptr<tesseract::SVEvent, std::default_delete<tesseract::SVEvent> >::~unique_ptr() (unique_ptr.h:361)
==1212620==    by 0x5FD91F: tesseract::ScrollView::MessageReceiver() (scrollview.cpp:184)
==1212620==    by 0x604056: void std::__invoke_impl<void, void (*)()>(std::__invoke_other, void (*&&)()) (invoke.h:60)
==1212620==    by 0x603FEC: std::__invoke_result<void (*)()>::type std::__invoke<void (*)()>(void (*&&)()) (invoke.h:95)
==1212620==    by 0x603FC4: void std::thread::_Invoker<std::tuple<void (*)()> >::_M_invoke<0ul>(std::_Index_tuple<0ul>) (thread:264)
==1212620==    by 0x603F94: std::thread::_Invoker<std::tuple<void (*)()> >::operator()() (thread:271)
==1212620==    by 0x603EED: std::thread::_State_impl<std::thread::_Invoker<std::tuple<void (*)()> > >::_M_run() (thread:215)
==1212620==    by 0x50D3ECF: ??? (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.28)
==1212620==    by 0x4FEBEA6: start_thread (pthread_create.c:477)
==1212620==    by 0x52E9DEE: clone (clone.S:95)
==1212620==  Address 0xc15e4a0 is 16 bytes inside a block of size 48 free'd

With this additional change Tesseract no longer crashes:
diff --git a/src/viewer/scrollview.cpp b/src/viewer/scrollview.cpp
index 087dabe8..34d339f7 100644
--- a/src/viewer/scrollview.cpp
+++ b/src/viewer/scrollview.cpp
@@ -347,7 +347,7 @@ void ScrollView::StartEventHandler() {
       if (new_event->type == SVET_DESTROY) {
         // Signal the destructor that it is safe to terminate.
         event_handler_ended_ = true;
-        delete new_event; // Delete the pointer after it has been processed.
+        //delete new_event; // Delete the pointer after it has been processed.
         return;
       }
       delete new_event; // Delete the pointer after it has been processed.

I still don't know whether memory leaks remain.

I’m now running a build starting with the 2 commits from Povilas and adapted with two changes from Stefan. (One from this thread, one from a new commit on the main)

I still don't know whether memory leaks remain.

There remain two direct leaks (48 + 16 bytes) and an indirect leak (4 bytes). They are shown when I build tesseract with sanitizers (./configure --disable-shared --enable-debug CXX=clang++-11 'CXXFLAGS=-D_GLIBCXX_DEBUG -Wall -g -O0 -fsanitize=address,undefined -fstack-protector-strong').

I still get an error when running with valgrind

I couldn't reproduce the problem and looking into code this crash seems to be something different than what the initial two commits fix.
Anyway, I have converted event_table_ to std::unique_ptr too. This should fix any memory handling bugs in that area.

Thanks @rmast for testing.


Make credit card images customizable

That library has two constants - CARD_TYPES and images - both are static and hardcoded, in case it's needed to change a credit card image, a code change is required, no way to pass down a property to override the default image.
Said that, this PR makes possible to add custom images, also this PR makes possible to add any other credit card type.
For instance, by using that PR we can add a different card type if needed - the credit card type Discover: 3000-0000-0000-04.

@igorlima , this feature is great, thanks so much 👍

Published on 1.1.2


NBT contains bad sneak behavior value ""; using default value instead

Coming from MC Eternal v<IP_ADDRESS> modpack. I found this warning report in the server console.
MC Version: 1.12.2
Mod Version: 0.11.4
In the server console, I get this warning of "NBT contains bad sneak bnehavior value ""; using default value instead. This happens when the player joins the server. Screenshot posted below

Thanks for reporting this. I quietly fixed it in the 1.16.5 version a while back, but I haven't updated the 1.12.2 version since then. There's a bug in how player "sneak behavior" settings are being saved, so whenever players rejoin a server it'll always revert to "invert chopping". I'll make sure this is fixed in the next update!

This should be fixed as of 0.14.1. Sorry for the wait!


Ability to get current time (getCurrentTime) on the player

Is there currently the ability to get the current time of the player? (I.e. how much of the video has been played so far)

You can use this: https://pub.dev/documentation/youtube_player_iframe/latest/youtube_player_iframe/YoutubePlayerController/getCurrentPositionStream.html


[Request] Single key desktop switching

It would be nice to directly switch to a desktop with one key, I've used F1-F4 keys in the past, is this possible ?

Hello @ascii78,
what you are asking for is not possible at the moment (version beta 0.11.1) and I don't know if it would be feasible to implement it in any future version (it would probably require some heavy work to be done in the settings configuration, which would cause trouble when updating from an old version).
Maybe @sdias could give you some more details on this.

Insert this code anywhere in the script:
F1::
SwitchToDesktop(1)
Return


Replace right-click menu with cmp one

So my goal is replace mouse menu (right window) with cmp menu (left one) I could not find any info, is this even possible? thx

to spawn the completion menu you need to be in insert mode, so rn this is not possible


Error: exec proxy not found

I'm doing a project in Phonegap that requires an option to login with Facebook.
The installation of the plugin is correct but when i try it out on the browser i get the following message:
Error: exec proxy not found for :: CordovaFacebook :: login
Now, I understand that it only is supposed to work on Android or IOs so i also tried running the project in the phonegap app for Android that, as far as I understand it builds the project in a native Android enviroment. But even then it still doesn't work. I don't get any interaction to continue, like if my button wasn't working. I checked with some "alerts" with some messages and they appeared but the function dies once it gets to the CordovaFacebook.login()
Here's  the code i'm using if it's any help:
function getDataFB(e){
  //console.log('start FB');
  //alert('start FB');
  e.preventDefault();
  window.CordovaFacebook.login({
    permissions: ['email', 'user_likes'],
    onSuccess: function(result) {
      if(result.declined.length > 0) {
         alert("The User declined something!");
      }else{
         alert('Correct FB connection');
      }
    },
    onFailure: function(result) {
      if(result.cancelled) {
         alert("The user doesn't like my app");
      } else if(result.error) {
         alert("There was an error:" + result.errorLocalized);
      }
    }
  });
}

I'm having some issue. Can I help me, please?


Android sdk tools spec update

@chenlevy24 the PR is not passing the CI checks and has a conflict. Can you check?


chore: update git-age to version v0.1.10

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Automated with GoReleaser
Microsoft Reviewers: Open in CodeFlow

Manual Validation ended with:

Trojan:Win32/Wacatac.B!ml

(Automated response - build 812.)

😂 eh yeah sure...
I can assure you it's not any kind of malware. I'm already building it (for Windows) with an obfuscator to 'trick' the Windows built in Defender...
Is there anything I can do besides spending ~200€ for a code signing certificate?

@wingetbot run

@wingetbot waivers Add Validation-Executable-Error


jQuery error when selecting file from pop up in admin: Uncaught TypeError: $ is not a function

I'm getting a jquery error when trying to select a file from admin:
Uncaught TypeError: $ is not a function at window.dismissRelatedImageLookupPopup (popup_handling.js:18) at HTMLAnchorElement.onclick (?_pick=file&_popup=1:412)
Using 1.4.4 and Django 2.1.7

I also got this error, but only with a model that use FilerFileField, seems like Django jQuery is loaded after the dropzone and widget JS.

I'm getting this error with FilerImageField as well -- Django 2.1.7 and Filer 1.4.4.

seems like Django jQuery is loaded after the dropzone and widget JS.

I got the same problem. Seems to be solved by adding jquery and init script explicitly on top of the admin template, and add the jquery and init scripts as static files. However this feels very hacky, so a better fix would be appreciated!
templates/admin/base.html
{% extends 'admin/base.html' %}
  {% block extrahead %}
      <script type="text/javascript" src="/static/admin/js/vendor/jquery/jquery.js"></script>
      <script type="text/javascript" src="/static/admin/js/jquery.init.js"></script>
  {% endblock %}

Got same problem, ends up downgrading to Django 1.11.x

I did some work on media merging in Django because I also encountered problems where assets were ordered incorrectly. Django 2.2 fixes this for good but also makes it necessary to explicitly depend on django.jQuery. See
Release notes:
https://docs.djangoproject.com/en/2.2/releases/2.2/#merging-of-form-media-assets
Possible fix:
https://github.com/matthiask/django-cabinet/commit/f183dcb4db382ccb1ba9fe1e328695c1d7181709

See PR: #1116


{Graph} Refactor role module for Microsoft Graph migration

Description
Code refactoring for Microsoft Graph migration

Replace azure.graphrbac SDK interactions with model-less dict operations
Sort functions in src/azure-cli/azure/cli/command_modules/role/custom.py into logical groups for easier developer collaboration and source control
Extract src/azure-cli/azure/cli/command_modules/role/msgrpah (similar to https://github.com/microsoftgraph/msgraph-sdk-python-core) to officially introduce msgraph.GraphClient so that it can be re-used by other modules or extensions
Introduce msgraph.GraphError for model-less error handling
Rename msgraph.GraphClient methods: for non-create POST API like addPassword, use ...add_password, instead of changing it to ...password_add

Test refactoring

Migrate test replacers for Microsoft Graph: MSGraphClientPasswordReplacer, MSGraphUserReplacer
Manually test all az ad/role commands to be working
Re-organize test cases into reasonable logical groups like ApplicationScenarioTest
Rewrite all tests to adapt to Microsoft Graph, especially dropping AD Graph permission tests for az app permission
Re-run all tests for az ad/role so that the recording contains Microsoft Graph requests

This PR paves the way for complete Microsoft Graph migration (https://github.com/Azure/azure-cli/issues/22174).

Graph


Add vcpkg to macOS agents

Tool information

Tool name: vcpkg
Add or update? Add
Desired version: Same as on Windows and Ubuntu

Area for Triage:

Apple
C/C++

Question, Bug, or Feature?:
Feature
Virtual environments affected

[ x] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Can this tool be installed during the build?
Installing vcpkg on-the-fly would take about one to two minutes. But shipping it pre-installed would bring the macOS agents in line with Windows and Ubuntu where it is already provided. Also pre-installing it would allow more aggressive caching of packaging (see #11) and shave off some build time.
Are you willing to submit a PR?
Yes
Thanks,
Gregor

@gjasny thanks for your suggestion. We will consider adding vcpkg to macOS.

@gjasny vcpkg has been added and will be rolled out next week.

@gjasny the image has been rolled out. Could you check vcpkg is available, please?

@miketimofeev Working like a charm! Thank you!

@gjasny thanks for checking! Closing the issue.


Actor type don't failover when app crashes but sidecar remains healthy

In what area(s)?

/area runtime
/area operator
/area placement
/area docs
/area test-and-release

What version of Dapr?
0.4.x
Expected Behavior
Placement service should be aware of the health of the application for a given actor type, not only the sidecar instance. If the app crashes but another app is up and registered for the same actortype, then calls to actor instance (new or existing Id) fail. In other words, the sidecar should report the health of the app too, so placement service can trigger rebalancing.
 Open a new terminal (terminal #2), then:
dapr run --app-id demoactorclient -- java -jar examples/target/dapr-java-sdk-examples-exec.jar io.dapr.examples.actors.http.DemoActorClient

Open a third terminal, then:
pskill java.exe
dapr run --app-id demoactorservice2 --app-port 3001 -- java -jar examples/target/dapr-java-sdk-examples-exec.jar io.dapr.examples.actors.http.DemoActorService -p 3001

Go back to terminal #2 and rerun:
dapr run --app-id demoactorclient -- java -jar examples/target/dapr-java-sdk-examples-exec.jar io.dapr.examples.actors.http.DemoActorClient

Now, see errors:
== APP == io.dapr.exceptions.DaprException: ERR_ACTOR_INVOKE_METHOD: rpc error: code = Unknown desc = error activating actor type DemoActor with id 60dad782-a802-4b39-acff-0d5487471756: dial tcp4 <IP_ADDRESS>:3000: connectex: No connection could be made because the target machine actively refused it.

== APP ==       at io.dapr.client.DaprHttp.lambda$invokeApi$3(DaprHttp.java:201)

== APP ==       at reactor.core.publisher.MonoCallable.subscribe(MonoCallable.java:56)

== APP ==       at reactor.core.publisher.Mono.subscribe(Mono.java:4105)

== APP ==       at reactor.core.publisher.Mono.block(Mono.java:1662)

== APP ==       at io.dapr.examples.actors.http.DemoActorClient.callActorForever(DemoActorClient.java:61)

== APP ==       at io.dapr.examples.actors.http.DemoActorClient.lambda$main$0(DemoActorClient.java:42)

== APP ==       at java.base/java.lang.Thread.run(Thread.java:834)

== APP ==       Suppressed: java.lang.Exception: #block terminated with an error

== APP ==               at reactor.core.publisher.BlockingSingleSubscriber.blockingGet(BlockingSingleSubscriber.java:93)

== APP ==               at reactor.core.publisher.Mono.block(Mono.java:1663)

== APP ==               ... 3 more

This only happens locally, not in a clustered environment like k8s. This is because when running locally, the two processes register as the same node, and no rebalancing happens.
I guess we can close this issue.

@yaron2 Like we discussed (updated issue title and description to reflect this), the issue here is that the sidecar health should not be enough for the placement service to determine rebalancing. The sidecar should also send the health status of the app, so placement service can rebalance an actor type if app is unhealthy (continuing restarting in case of K8s).

@amanbha is this a dupe of something you opened in the past? My long-term memory is coming into play here..

@amanbha is this a dupe of something you opened in the past? My long-term memory is coming into play here..

@yaron2 It seems different, I think the issue which you are referring to is this one https://github.com/dapr/dapr/issues/1038

@amanbha is this a dupe of something you opened in the past? My long-term memory is coming into play here..

@yaron2 It seems different, I think the issue which you are referring to is this one #1038

Yeah that's the one. The common theme is we should get the runtime to detect app restarts. This is useful for metrics too.

About this issue: This is a broader question than placements, should Dapr ask user apps for health probes to handle errors for pub-sub/invocation etc.
I think, if the user service has crashed, then user has to bring it up based on monitoring, while the instance is down, the calls will fail.

@amanbha is this a dupe of something you opened in the past? My long-term memory is coming into play here..

@yaron2 It seems different, I think the issue which you are referring to is this one #1038

Yeah that's the one. The common theme is we should get the runtime to detect app restarts. This is useful for metrics too.

Yes, restarts should be handled.as per  #1038

@amanbha @yaron2 (IMO) In a production environment, if daprd is proxying calls into my app, it should detect unhealthy instances and failover to a healthy instance - similar to a load balancer. Users should be aware that some instances are failing but their service should not experience reduced availability if healthy redundancy is available.

@amanbha @yaron2 (IMO) In a production environment, if daprd is proxying calls into my app, it should detect unhealthy instances and failover to a healthy instance - similar to a load balancer. Users should be aware that some instances are failing but their service should not experience reduced availability if healthy redundancy is available.

We have a mechanism for retries today that will retry on errors coming from the destination Dapr sidecar.
Essentially in Kubernetes, production environments are recommended to set up health probe on the container level. In that case, Kubernetes itself will take the Pod out of the load balancing rules and requests will be routed to healthy pods.

@yaron2 Makes sense for K8s. What about for standalone mode?

Standalone mode uses the components to resolve remote addresses, so this really comes down as a concern of the hosting platform.
You will get the same type of behavior in DC/OS, Swarm and hopefully Service Fabric.
For a local node setup, this is probably less useful.

I was also thinking about making dapr cli exit when app exits. If we add the mode where dapr cli can run without app, we can add this feature to the cli as well.

@artursouza @yaron2  Based on the discussion I think this is not a issue any more(definatley not P0) and can be closed in favor of the cli issue which @artursouza  explained above.

@artursouza @yaron2 Based on the discussion I think this is not a issue any more(definatley not P0) and can be closed in favor of the cli issue which @artursouza explained above.

I agree this is not P0.
We should however talk about having actor sdks implement health check endpoints. If that turns into a separate issue I think we can close this one.

This issue is originally to report the issue, so I will revert it back to describe the problem and not the solution. Then we can keep it open unless we decide that the current behavior is acceptable.

I was also thinking about making dapr cli exit when app exits. If we add the mode where dapr cli can run without app, we can add this feature to the cli as well.

Can you clarify how having the cli exit when the app does solves the problem?

I was also thinking about making dapr cli exit when app exits. If we add the mode where dapr cli can run without app, we can add this feature to the cli as well.

Can you clarify how having the cli exit when the app does solves the problem?

It would only help in the context of cli - how this issue originated - but does not solve the root cause.

Design Proposal
Following is a design for implementing a health endpoint in the Actor runtime SDKs.
The Dapr runtime will continuously probe the health endpoint and upon 2 consecutive failures, 5 seconds apart, will remove itself from the placement ring.
Likewise when it receives an OK response after the same defined period, it will rejoin the ring.
The health endpoint url format:
<address:port>/healthz

A 200 OK response is a guarantee for a successful liveness probe from the Dapr runtime side.
Any other response will count as a failure.
The healthz endpoint is named so to be consistent with Dapr's own health endpoint.

/cc @amanbha

/cc @amanbha

lgtm

This means we should have the endpoint inside the Dotnet SDK and Java SDK for this release.
cc @artursouza

This means we should have the endpoint inside the Dotnet SDK and Java SDK for this release.
cc @artursouza

Yes, and any third party actor implementations(outside of our SDKs0 should also have it, so we will have to document it and explain this scenario as well.

If we have existing docs about the API endpoints used in an actor runtime, it should go in there.

Shouldn’t this be an API beyond just actors?

Shouldn’t this be an API beyond just actors?

Probably not, as a failure here means for Dapr to rebalance the actors. For non actors, I'm not sure what a failure in the app would cause.

Right now, it is only used for actors, but it can be used for other scenarios later. The API itself just says healthz, which is not specific to actors - which I agree with.

Right now, it is only used for actors, but it can be used for other scenarios later. The API itself just says healthz, which is not specific to actors - which I agree with.

Yeah. Let's focus on the immediate goal which is actors, then we can think next about other scenarios.

I agree. I am just asking to keep the API outside the actor namespace.

It mainly impacts actors, as no call would ever succeed for a particular actor id if user app is not responding. For non-actor calls the call is not tied to a specific instance.

I agree. My point is that if we want to use this feature outside actors later, it will be a breaking change.
If we want to make this specific to Actors, we don’t need a new API. We can use the mandatory /Dapr/config as health check.

I agree. My point is that if we want to use this feature outside actors later, it will be a breaking change.
If we want to make this specific to Actors, we don’t need a new API. We can use the mandatory /Dapr/config as health check.

Yes, we can use config endpoint, my concern is with overloading because user code might be doing lots of processing on that route assuming its called only once, now we have to educate them that cache the values for subsequent calls as its also used as a health probe.

Ok. So going back to the health API. I am proposing it to be outside the actor namespace. That is all.

@artursouza are you talking about the actor namespace in the SDK? in that case, isn't it dependent on where the HTTP server sits? as its just a 200 OK response.
If the HTTP server is outside of the actors namespace/package, then its outside..

Yes, but it depends on how we are extending the rest controller. So I can assume we agree that this is outside Actor namespace.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Yaron Schneider<EMAIL_ADDRESS>Sent: Tuesday, March 24, 2020 6:22:47 PM
To: dapr/dapr<EMAIL_ADDRESS>Cc: Artur Souza<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dapr/dapr] Actor type don't failover when app crashes but sidecar remains healthy (#1070)
@artursouzahttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fartursouza&data=02|01||64781a95c583418e163f08d7d05b0782|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637206961687751478&sdata=kJby146MHAU55ZeDXq%2BIj9XsCiMJ0GJS0IYiQN2aIx4%3D&reserved=0 are you talking about the actor namespace in the SDK? in that case, isn't it dependent on where the HTTP server sits? as its just a 200 OK response.
If the HTTP server is outside of the actors namespace/package, then its outside..
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdapr%2Fdapr%2Fissues%2F1070%23issuecomment-603588288&data=02|01||64781a95c583418e163f08d7d05b0782|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637206961687761486&sdata=Iqwx%2BmcLKIdjlAFFby2KuEbZb8k83M6EuUjFfC76lxc%3D&reserved=0, or unsubscribehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAA77CXMTFPRLETK55XJFO3RJFMGPANCNFSM4KYCJXVA&data=02|01||64781a95c583418e163f08d7d05b0782|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637206961687761486&sdata=mnv9dFsmI3LgmidGNGN34KYKKgt4Aw4YCzaEG0Ut59Y%3D&reserved=0.

I think you wanted to clarify if it will be a route like /actors/healthz or /healthz. It will be latter. For this change Dapr runtime will only use it for actors placement distribution.


Updated release diff targets and .npmignore

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

#3095  👈
#3082
main

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

Join @CatChen and the rest of your teammates on  Graphite


snd: 22.5 -> 23.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/snd/versions.
meta.description for snd is: "Sound editor"
meta.homepage for snd is: "https://ccrma.stanford.edu/software/snd/"
meta.changelog for snd is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 22.6 with grep in /nix/store/gnbh09yv9g7qg3s0qa5i6vrzffl99i3j-snd-22.6
found 22.6 in filename of file in /nix/store/gnbh09yv9g7qg3s0qa5i6vrzffl99i3j-snd-22.6

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
snd

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

snd

Maintainer pings

@ofborg eval

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/snd/versions.
meta.description for snd is: "Sound editor"
meta.homepage for snd is: "https://ccrma.stanford.edu/software/snd/"
meta.changelog for snd is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 22.7 with grep in /nix/store/86rs8q16xpl9n770kxwlf21my89al256-snd-22.7
found 22.7 in filename of file in /nix/store/86rs8q16xpl9n770kxwlf21my89al256-snd-22.7

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
snd

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

snd

Maintainer pings

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/snd/versions.
meta.description for snd is: Sound editor
meta.homepage for snd is: https://ccrma.stanford.edu/software/snd/
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 23.0 with grep in /nix/store/wch7ck0p5aia5w7n6i6i0lkd38kwsf87-snd-23.0
found 23.0 in filename of file in /nix/store/wch7ck0p5aia5w7n6i6i0lkd38kwsf87-snd-23.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
snd

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

snd

Maintainer pings


feat: blockscout support with sc verification

addresses #372

@barnabasbusa pls note:

blockscout execution block explorer not activated by default
up to date with latest changes in main
resources limit values are just a guess
not sure if its ok to return the 'blockscout verification url' from the main function so can be used in the diva-package that calls this eth-package

Could you add blockscout to a couple of the tests ?

.github/tests/mix-with-tools.yaml
.github/tests/mix-with-tools-mev.yaml

Do you see a value in enabling persistent storage for this tool (if users asks for persistence) ?

right, will do test and persistence!

I also returned the url for the verification sc service, pls check, I can rm if needed

This probably has more to do with the image than with kurtosis config, so lets merge this.


[FEATURE] enhanced shell completions

As of #213, we have support for static shell completions. There are a few enhancements to make:

more robust install support for linux and windows
automatic installation as part of our install script
dynamic completions, for example, completing component names on commands like exo start ...

This is all relatively low priority, but (3) in particular is probably a good "help wanted" feature for new commers!

Let's say that this is blocked by #301.

I had difficulty getting completions to generate on Fish. I have no clue why, as pkg-config --variable completionsdir fish seems to resolve the proper directory, and exo completion generate fish works fine to output a completions file (I was able to just pipe that where it belonged). I'm not sure if this should be a new issue or could be attached to this one, but I'll start here.
I wrote up my solution (to help others) here: https://stackoverflow.com/questions/70010126/generating-completions-file-for-exo-in-fish/70010127#70010127
I tried to tag it as exo but I don't have 1500 reputation so couldn't create the tag.

cc @BenElgar who can help with installer and shells stuff

I'm hoping #528 improves the fish use case.

Thanks @BenElgar!
@MaxmumP, we'll let you know when this lands in the next release & maybe you can give it a try and let us know how it works. Thanks for the report.

Yeah, that looks great. I had assumed you had already been using pkg-config to find the completions directory. Was impressed at how fast @BenElgar got that in! :-)

This was prematurely closed along with the Fish fix. Dynamic completions are not yet implemented.


MINOR: add github action trigger

Signed-off-by: Ruihang Xia<EMAIL_ADDRESS>Which issue does this PR close?

Ref #3321.
Rationale for this change

GitHub Action will only run on PR updating, we cannot trigger one on demand manually. This is inconvenient if we want to run one on master or without submitting a PR.
I've tried this patch in my fork repo, now I can see this button:

What changes are included in this PR?

Add an [action trigger](https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_dispatch] that allows us to trigger an action manually.
Are there any user-facing changes?

No

test failure caused by https://github.com/apache/arrow-datafusion/issues/3327

Thanks @waynexia  -- this looks great.

I took the liberty of merging this branch to master -- hope to merge it when green

Codecov Report

Merging #3323 (9140639) into master (70e702c) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3323      +/-   ##
==========================================
- Coverage   85.81%   85.81%   -0.01%
==========================================
  Files         294      294
  Lines       53764    53764
==========================================
- Hits        46136    46135       -1
- Misses       7628     7629       +1

Impacted Files
Coverage Δ

datafusion/expr/src/logical_plan/plan.rs
78.05% <0.00%> (-0.17%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks ❤


[App]: Account Creation Loop

Current Behavior
So I've recently just tried to create an account. After the first signup page, while being redirected, my browser tab crashed. Now upon returning to login, I am prompted to "Create your account" at the page https://wandb.ai/signup.
I already did that step, and selected a personal account. However whenever I go back, it prompts me to do so again and says my account already exists.
Error creating account: entity already exists for the user nichols
Expected Behavior
No response
Steps To Reproduce
No response
Screenshots
No response
Environment
OS: Tried logging in on both ubuntu, and ios
Browsers: Firefox and Safari
Version: 20.04
Additional Context
No response

Hi @nicholscrawford , thank you for writing in and our apologies for the inconvenience. I've asked the app team to investigate and we'll get you unblocked sometime today. I will provide an update by EOD

Hey @nicholscrawford , we appreciate you patience while we worked through this. We have identified the issue causing this loop behavior and have fixed it. You are now able to access your account without being prompted to input a new account name.


Add UpdateActions to the interop client helper

Add UpdateActions to the interop client helper, so that the helper can modify the list of supported test cases.

No significant performance differences

Known issues:
gRPC_interop_pull_requests: #10478


Listing Management - Recent Activity should show submitted listings

It seems to be working

Try this as a normal user and not as an Admin.

Issue moved to ozone-development/ozp-backend #167 via ZenHub.io

after making changes to the front end i am not getting expected results form the backend.
received
 instead of

@mannyrivera2010

Passed on latest.

Passed on staging


python310Packages.azure-mgmt-recoveryservicesbackup: 5.0.0 -> 5.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.azure-mgmt-recoveryservicesbackup is: This is the Microsoft Azure Recovery Services Backup Management Client Library
meta.homepage for python310Packages.azure-mgmt-recoveryservicesbackup is: https://github.com/Azure/azure-sdk-for-python
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.1.0 with grep in /nix/store/hhj69xwv61p25ypraa27hy4pdcpsag6i-python3.10-azure-mgmt-recoveryservicesbackup-5.1.0
found 5.1.0 in filename of file in /nix/store/hhj69xwv61p25ypraa27hy4pdcpsag6i-python3.10-azure-mgmt-recoveryservicesbackup-5.1.0

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
azure-cli
python310Packages.azure-mgmt-recoveryservicesbackup
python311Packages.azure-mgmt-recoveryservicesbackup

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hhj69xwv61p25ypraa27hy4pdcpsag6i-python3.10-azure-mgmt-recoveryservicesbackup-5.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hhj69xwv61p25ypraa27hy4pdcpsag6i-python3.10-azure-mgmt-recoveryservicesbackup-5.1.0
ls -la /nix/store/hhj69xwv61p25ypraa27hy4pdcpsag6i-python3.10-azure-mgmt-recoveryservicesbackup-5.1.0/bin

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

3 packages built:

azure-cli
python310Packages.azure-mgmt-recoveryservicesbackup
python311Packages.azure-mgmt-recoveryservicesbackup

Maintainer pings
cc @mwilsoncoding for testing.

@ofborg build azure-cli python310Packages.azure-mgmt-recoveryservicesbackup python311Packages.azure-mgmt-recoveryservicesbackup

Result of nixpkgs-review pr 213748 run on aarch64-darwin 1

3 packages built:

azure-cli
python310Packages.azure-mgmt-recoveryservicesbackup
python311Packages.azure-mgmt-recoveryservicesbackup


Issues with selecting supports

MAKE SURE YOU TESTED THE LATEST RELEASE.
Yes
Samsung user? See https://mathewsachin.github.io/Fate-Grand-Automata/SAMSUNG.html

Describe the bug
Bug where the app doesn't scroll through the support list but only taps on the left side of the screen. To add on, app only works properly when the top of my phone is facing the right. Not sure how to explain it but on a set orientation?
My settings are the prefered servants which are currently Raikou.png and Arjuna.png which were created using the Support Maker script.
Screenshots
If applicable, add screenshots to help explain your problem.
If an error occurred, you can find the message by expanding the app's notification.
It is recommended that you atleast add one screenshot of FGO running on your device.
Video
https://streamable.com/t795h8
Device Info (please fill atleast 'Device model'):

Device model: Asus ZenFone 5Z
Screen size: 1080 x 2246
Android Version: Android 10
RAM: 6GB

I forgot to say that my I set to preferred servants which are currently Raikou.png and Arjuna.png which were created using the Support Maker script.

The problem is FGO isn't taking up the whole screen. Pretty similar to what happens on Samsung phones.
Can you check if there is any option to make FGO run as a Full Screen app on your phone.
Those blue borders should fill the black area of the screen.

I've tried what you said and I've gotten it running in Full Screen. It managed to scroll up once and then it continued behaving like whats seen on the video again. I've tried it with and without notch option.

Upload the video in full screen please.

https://streamable.com/l3ir9w
Here you go. Thanks a lot by the way.

Seems like scrolling doesn't work for some reason. Also what is rotating the screen?

Something's really weird here, how does FGA look on your phone?
Is it not full screen? Can you post a screenshot of the app?
@reconman even the Debug mode rectangles aren't rendering completely.

Seems like scrolling doesn't work for some reason. Also what is rotating the screen?

Ahhh sorry, its just me trying it on the different orientation cause before this I had issues where only when it was counter clock-wise, the app would work properly.

Something's really weird here, how does FGA look on your phone?
Is it not full screen? Can you post a screenshot of the app?
@reconman even the Debug mode rectangles aren't rendering completely.
https://imgur.com/a/LL2sxUT

Here you go, its full screen though.

Oh, I meant an image of FGA (this app) not FGO

FGO is completely extending to the notch, right? So, the Ignore Notch option should be ON.

https://imgur.com/a/YsZwhl1
My bad, here you go.

I think the Back/Home buttons remaining always visible over FGA could be the problem 🤔
Can you see if there's any way to make FGA fullscreen too?

I've changed FGA to full screen and it started scrolling properly. I'll let you know if there any other issues after this. Thanks a lot for your help!


Use jumpscale with configmanager storing data in MongoDB

Issue migrated from https://api.github.com/repos/Jumpscale/core9/issues/244, opened by @FastGeert
For the Bancadati project its impossible to create git repo's for all the robots that will be running there. Therefore we need a way to configure jumpscale to store the data of the config manager into a MongoDB database.
In this modus, the data does not have to be encrypted.

in digital me the config data will all be in BCDB


TestEnvironment: Fix timeout arithmetic

Motivation:
The timeout calculation decrementing totalTimeoutRemainingNs is incorrect which results in premature timeouts and test failures.

@viktorklang - are you still merging commits? are there other contributors (do you need more)?

@rkuhn @akarnokd - wdyt?

Indeed the original code looks wrong but I don't remember tripping this failure case ever. Also isn't there a TCK test that verifies this timeout case?

The fact that this obvious bug only surfaces now means that this TCK facility is not used correctly anywhere in the code — long underflow seems unlikely to occur here. I can’t offer further comments as the TCK was mainly conceived and implemented by @ktoso (and I haven’t used JVM languages for the past few years).

If errors are delivered synchronously (like in the TCK tests) and the unit under test is behaving as expected, it is unlikely this code will be hit (e.g. env.asyncErrors.isEmpty() will be false). We occasionally hit the condition in ServiceTalk due to testing async signal delivery (different thread than test thread).

I added a unit test to the TCK to demonstrate the issue (if you remove the fix the test will likely fail).

@akarnokd - thanks for approval. What are the next steps to getting this merged and released?

I don't know. Most of the original crew have moved on afaik so no idea.

I’m afraid I lost my sonatype credentials, so I can’t publish anymore. If someone else wants to step up and needs me to confirm something, then I’ll happily do that!

@akarnokd @rkuhn - how does one become a contributor for this repo? I pinged victor offline and he is no longer involved. would love to get this merged and released.

No idea. Also no idea how the release infrastructure is working.
If I had this annoying issue, after so much time, I'd seriously consider cloning the TCK, applying the fixes and begin using that to verify my projects. Given the amount of development in RS, you probably won't have to worry about becoming incompatible.
(If I understood .NET better, I would have done it for the Reactive Streams .NET TCK repo, which practically unusable with newer test frameworks of .NET)

@ktoso - thx for review. comments addressed.

"worked ok enough" so that we didn't notice earlier
I'm assuming most tests were "synchronous" (publisher and verification on same thread) which would make it unlikely to come up.

I'd approve and merge but have long since lost merge rights here.
@viktorklang @rkuhn are you able to help out here?

I can merge, so I did that, but as I said I can’t publish anymore.

Thank you! It seems I can publish (but not merge) huh, so I’ll give that a try if we want a patch release with this :)

Sure, go ahead! I guess you’ll need another PR to bump the version, right?

version update: https://github.com/reactive-streams/reactive-streams-jvm/pull/547
anything else required for release?


[WFLY-9127] Add failover test for modcluster.

Worker failover test for modcluster. Creates basic scenario (one balancer and two workers) and sends a request when both workers are alive, then it undeploys worker which responded and sends the request again.
https://issues.jboss.org/browse/WFLY-9127

Can one of the admins verify this patch?

@rhusar @stuartwdouglas FYI;  THX @mmadzin


Styles are missing

There are no .sass file in the repository, but you are compiling them in gulp.
could it be that they are missing? at least I am missing a lot of styles in my app (see the picture, "fa fa-facebook" don't get any styles..).
There is nothing displayed, just when I hover over the invisible buttons their color change like it should, but I am missing for example the "\F09A" in the facebook sharing button and so on...
do you have any idea why? Did everything like the readme said.
I would be please if a fast solution is possible, because my project have to be finished in a few days... :(

Hi @Abszissex, I have noted that

The default icons requires fontawesome to be loaded into your project.

This means that you have to add fontawesome stylesheet in your project if you want the default icons, you can do this simply by adding the following line in your index.html head
<link href="https://maxcdn.bootstrapcdn.com/font-awesome/4.6.3/css/font-awesome.min.css" rel="stylesheet" integrity="sha384-T8Gy5hrqNKT+hzMclPo118YTQO6cYprQmhrYwIiQ/3axmI1hQomh7Ud2hPOy8SP1" crossorigin="anonymous">
There are many ways to load font-awesome stylesheet so I left it up to you


How to wait for commands?

Hi!
I'm write a small management interface script which allows me to handle some authentication stuff. I have to set management-client-auth on the openvpn server.
How to wait for an incoming request?

Currently this functionality is not yet supported by this library. There is a pull request (#14) open for the beginning of this feature which will enable you to do what you ask. Keep an eye on that PR and I'll try to remember to write back here after it's merged.

Thanks a lot!


Simple mint gasless toggle and small changes

ref

https://github.com/kodadot/nft-gallery/issues/7825

Abstract

added section in simple mint; when activated, it mints into our kodadot collective collection)
design updated in handoff file: https://www.figma.com/file/JXNtryaNYhY0XJNArzvPQx/Handoff?type=design&node-id=5000%3A1078&mode=design&t=bdDUIMGGuJOQsWYI-1
added also the banners etc.

blue one is for polkadot hub, grey for kusama hub, white for dev

Design changes

already mentioned added section
background UI changed to white (in dark mode, normal background color user everywhere)
reordered sections - Network put before collection selection; it did not make sense the other way around

Collection selection

The default state is simple
when you activate the kodadot collective, a glow is displayed behind the icon

Are there any additional transactions that the user has to sign? cc @vikiival
if you have any question feel free to ask

:wave:

Are there any additional transactions that the user has to sign?

Nope, everything is done by the API
Waiting for kodadot/private-workers#1

It's free for grabs if anyone take it
Some work was done here

https://github.com/kodadot/nft-gallery/pull/7962


c++utils: add double/float support in opt parsing

Add template specializations for options that are or type float and double
when parsing from command line arguments.

So this will throw an exception if it can't parse. We have three options:

do a try/catch and print out a meaningful error message, return 0.0
do a try/catch and print out a meaningful error message, rethrow
keep it as is and let the exception be thrown (which may not have a meaningful message)

What do you think we should do?

I like print and re-throw.


Feature/630 transform guards

closes #630
TODO: How to pass warnings and errors?

see branch https://github.com/FUSEEProjectTeam/Fusee/tree/feature/630-transform-guards


Revert "[Snyk] Security upgrade react-native from 0.73.4 to 0.73.5"

Reverts Expensify/App#42700

skipping checklists for a straight revert

not an emergency, skipped checklist for straight revert

Requested CP: https://expensify.slack.com/archives/C07J32337/p1717085948670429

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/roryabraham in version: 1.4.77-11 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/roryabraham in version: 1.4.77-11 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.4.78-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Add documentation on how to use prisma client extensions for "enum"-like validation

SQLite doesn't support Enums, but sometimes you want a string field to only allow certain specific values. This is especially useful for simplifying typings throughout the application.
Prisma client extensions should allow for this. I would welcome a PR to the docs demonstrating how to do this.

As stated here https://github.com/prisma/prisma/issues/2219#issuecomment-1506975020 this feature its not being developed in the future, maybe never, neither with check constraints or with any other method.
Here is another documentation explainig why they are opting-ing strictly for db-native features.
https://github.com/prisma/specs/issues/330#issue-525152162

As noted, I'm talking about using Prisma client extensions feature which does mean this wouldn't be a database-level enhancement, but just one for the client which is sufficient for us.

For people looking for it.
You can make client extensions like this:
import { z } from "zod";
import { Prisma } from "@prisma/client";

const schema = z.object({
  slug: z
    .string()
    .max(100)
    .regex(/^[a-z0-9]+(?:-[a-z0-9]+)*$/),
  name: z.string().max(100),
  description: z.string().max(1000),
  price: z
    .instanceof(Prisma.Decimal)
    .refine((price) => price.gte("0.01") && price.lt("1000000.00")),
}) satisfies z.Schema<Prisma.ProductUncheckedCreateInput>;

export const ProductValidation = Prisma.defineExtension({
  query: {
    product: {
      create({ args, query }) {
        args.data = schema.parse(args.data);
        return query(args);
      },
      update({ args, query }) {
        args.data = schema.partial().parse(args.data);
        return query(args);
      },
      updateMany({ args, query }) {
        args.data = schema.partial().parse(args.data);
        return query(args);
      },
      upsert({ args, query }) {
        args.create = schema.parse(args.create);
        args.update = schema.partial().parse(args.update);
        return query(args);
      },
    },
  },
});

And then use it like this:
import { Prisma, PrismaClient } from "@prisma/client";
import { ProductValidation } from "./models/product";
import { ReviewValidation } from "./models/review";

const prisma = new PrismaClient()
  .$extends(ProductValidation)
  .$extends(ReviewValidation);

See:
https://github.com/prisma/prisma-client-extensions/blob/main/input-validation/README.md

Thanks @L-Steinmacher!


[Bug] Unable to generate micronaut app without feature

Plugin Name
@nxrocks/nx-micronaut
Plugin Version
1.0.2
Nx Version
14.7.11
Expected Behaviour
User should be able to create a micronaut application without feature
Actual Behaviour
Generation crashes with a HTTP 400 error :
 >  NX   ❌ Error downloading Micronaut project zip from 'https://launch.micronaut.io'

   If the problem persists, please open an issue at https://github.com/tinesoft/nxrocks/issues, with the following information:
   ------------------------------------------------------
   Download URL: https://launch.micronaut.io/create/default/com.example.test?javaVersion=JDK_8&lang=JAVA&build=MAVEN&test=JUNIT&features=
   Response status: 400
   Response headers: {}
   ------------------------------------------------------
   Pass --verbose to see the stacktrace.

This is probably due to the empty feature parameter as :

https://launch.micronaut.io/create/default/com.example.test?javaVersion=JDK_8&lang=JAVA&build=MAVEN&test=JUNIT&features= return a 400
https://launch.micronaut.io/create/default/com.example.test?javaVersion=JDK_8&lang=JAVA&build=MAVEN&test=JUNIT  (please note the missing feature parameter) works fine

(Please also note Quarkus site seems to accept the empty feature parameter)
Steps to reproduce the behaviour

Create a new app in a workspace
When CLI ask for features, just press Enter without adding any feature.
Wait for CLI to download generated zip.

Closed by #136


[7.x] Allow postgresql to store blob (bytea) files when contains special characters

Hi, I need store bytea files into database. When file (binary format of file, not content) contains non printable characters (like pdf file), pdo failed. After this fix, everything working fine. When you are storing files like txt, which not contains non printable characters, it is storing bytea as PDO::PARAM_STR (because ctype_print return true), but in this situation there is no problem.
Edit: If you storing non printable chars into varchar field, there is no problem too (in this situation storing value into varchar with PDO::PARAM_LOB working without problem).

We don't include nested ternarys in Laravel.


On stickit:end Event Freeze the other sticky elements

Hi,
i would like to ask if it possible that one freeze all the sticky elements on the same position if one of them reach the bottom  so the other containers stopped and when one scroll up the sticky elements refresh.
For example:
I have three Container

Left sidebar,
Content Container in middle
Right Sidebar#

The Right and Left Sidebar have each different container heights and so the right Sidebar reach the bottom of the container before the left sidebar did it. I would like to know if it is possible that the left sidebar stops when the right sidebar reach the end and by scroll up they are again sticked
Thanks

Currently, it doesn't have this feature. I could add methods like 'pause' and 'resume' to control sticky elements by yourself.

Thanks for your reply.
This is would great if you add this methods, that i can controll them.

Hi, I found that you can just use extraHeight to do this. See this example
https://jsfiddle.net/jsmvyfwk/
Is this what you want?


Improve metatags on the homepage

This is how open collective shows up when shared on facebook :-(

This could be useful: https://metatags.io/


Node autoscaling enricher

#926
Part 1 - calculate percentage based on aggregated values.
Part 2 - next PR - enable enricher and set correct metric names for GCM autoscaling
cc: @fgrzadkowski

LGTM


Autocomplete Doesn't Automatically Select First Result In Some Cases

In some cases such as:
local ReplicatedStorage = game:GetService("ReplicatedStor

.. completions are generated, but none are selected by default.
This means that you must manually click the first result in the list or use the arrow keys, rather than just pressing tab to complete.
The first result is automatically selected in other cases (like "GetService" in the GIF below), so this seems like it could be a bug.
I've only noticed this issue occurring in GetService so far, but it could be present elsewhere that I haven't detected.

I can't reproduce this, the first item is always selected for me.
This seems more like a VSCode issue than the language server - do you have any VSCode settings modified?

Ah good find, thank you. That looks like the cause.
cc @Barocena I think you said you also suffer this issue?


Problem with decoding

OpenHTJ2K ver. https://github.com/osamu620/OpenHTJ2K/commit/7e59ff1535324484aaae10197168f99b6387d629
https://ibb.co/wpfj7Zg

I can see the attached images, however, the detailed information on; compilers, OS, encoding parameters, etc. should be provided from you.

GCC 11.3.1 20220429 no SIMD

How old is your computer? AVX2 is supported on Haswell or newer generations.

I don't know about SIMD compression in GCC.
I compiled for x86-64 processors in gcc. I donn't use the march=haswell option.
My computer.
I do not know about SIMD compression
[Microsoft Windows 10 Pro]
[10.0.19043 Kompilacja 19043]
[Procesor]
[Intel(R) Core(TM) i5-2500K CPU @ 3.30GHz, 3301 MHz, Rdzenie: 4, Procesory logiczne: 4]
[Wersja/data systemu BIOS]
[Intel Corp. BAP6710H.86A.0079.2012.0622.1722, 22.06.2012]

@Jamaika1
Thanks. The i5-2500K supports up to AVX, so AVX2 is not supported.
If possible, could you provide the codestream (.j2c) and commit-hash of the compiled .exe? I could not open .rar file.

It's better. The codec only supports standard photo sizes, e.g. 1280x720.
There is still a problem for 4000x3000.
I lack information in the SIMD codecs. Eg Error - Your CPU is not supported. I don't even know if the codec should show help for AVX2.
https://www.sendspace.com/filegroup/KO5NUl0y3HpE%2Bs54JIsldFcGQqP2RzzBNiqciGySVT0


Fixed 2 more tests that relied on non-deterministic APIs

The tests assume that HashMap is iterated in a certain order, which can cause the tests to fail under different java versions or platforms.
The fix simply changes HashMap to LinkedHashMap, as LinkedHashMap does guarantee an order of iteration.

[Current coverage][cc-pull] is 92.15% (diff: 100%)

Merging [#736][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #736   diff @@
==========================================
  Files           108        108
  Lines         16177      16177
  Methods           0          0
  Messages          0          0
  Branches       3766       3766
==========================================
  Hits          14908      14908
  Misses          594        594
  Partials        675        675

Powered by Codecov. Last update [80400e5...2d97a0e][cc-compare]
[cc-base-branch]: https://codecov.io/gh/alibaba/fastjson/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/alibaba/fastjson/compare/80400e57f5aad9375ac83c01306a958e4519cf6e...2d97a0ef2f907251313bdee1d49aba1e8b6308fc?src=pr
[cc-pull]: https://codecov.io/gh/alibaba/fastjson/pull/736?src=pr


Added logger mechanism for wdmp-c to support standalone logging & RDK…

… logger integration

Current coverage is 82.02% (diff: 82.64%)

Merging #16 into master will decrease coverage by 1.64%

@@             master        #16   diff @@
==========================================
  Files             2          3     +1
  Lines           533        562    +29
  Methods           0          0
  Messages          0          0
  Branches        116        124     +8
==========================================
+ Hits            446        461    +15
- Misses           34         43     +9
- Partials         53         58     +5

Powered by Codecov. Last update 9561a32...f6e5b51


Support non-standard julia registries

For my specific binder use case it would be great if one could specify a list of alternative julia registries, not just the general registry, from which packages that are listed in Project.toml should be loaded.
I assume that at some point julia will add an ability that one can specify alternative registries in the Project.toml itself (see https://github.com/JuliaLang/Pkg.jl/issues/635), but until that is the case, we could add an option for a .binderjuliaregistries file that is simply a text file with URLs for registries that should be used in addition to the default one.
Pinging @KristofferC just to get his thoughts whether that is a bad idea for some reason :)

I am not super familiar with Julia packaging but this sounds similar to personal package archives (PPA) in Debian or Ubuntu. We currently have no support for those when it comes to installing APT packages. This is on purpose as there is no community standard for specifying them in a config file. As a policy we don't want to invent new file formats and instead prefer to wait for the respective communities to create something that can be adopted by repo2docker.
I'd apply the same reasoning to custom registries for Julia. Especially if support via Project.toml is on the horizon.
The people who have asked for PPA support are vocal but there aren't many of them. To them the recommendation is to use a Dockerfile. I think for customer Julia registries one of the simpler escape hatches built in to repo2docker would work (postBuild).

Fair enough :) I think in terms of an escape hatch, we would need a way to run a script before the stuff that runs as part of the get_assemble_scripts to work around this.


add tagsToIgnore setting

FYI, this a bug posted in my branch: https://github.com/open-spaced-repetition/obsidian-spaced-repetition-recall/issues/72
And i think the plugin maybe need it.

Hi @Newdea.. Thanks but I think the better solution would be to let the user use glob patterns to ignore folders and/or files. I've opened a PR -> https://github.com/st3v3nmw/obsidian-spaced-repetition/pull/1086.

yes, using glob patterns is much better, i will close this pr.


Badlist Signatures updater issue

Hello,
Since my Asssembly Line setup is on, I wasnt able to pull signatures from badlist service.
I am using only default sources with following configuration :

Proxy (configured in the source config on the GUI), dont have ability to bypass it without tricky configuration.
Docker Compose (not K8s)
AL Version
Service Version : assemblyline-service-badlist:4.5.0.stable12

In the logs I have following error "MGet returned a document without any data"
Other services / sandboxes communicate well via proxy.
Accessing source url with a browser returned appropriate content.
Please note it is not easy to identify which source is failing that update.
Thank you for your helps on that issue.

Also it seems that a failing source is failing all sources from updating => resulting in no signatures at all.
Would like to have more logs/details concerning this issue. All I have for now. Dont hesitate to show me debug mode or things like that :)

You should be able to see which sources have failed in the UI under Update Sources

Thank you for your answer.
Yes, I see that. But the issue is that I cant tell why. Provided sources such as urlhaus or blackbook do not update and report no error in the GUI except "Last Successful Update: 54 years ago". I am looking in a way to have more verbose reports in case of a faillure. Other services/updaters seems to be easier to debug.

There should be logs from the updater container that should indicate why it failed per source.

Just forced an update to latest stable version of AL and fixed an obsolete proxy config in the compose file affecting updater container, now some sources are updating but dont know well to which root cause it is bound. I will check in next days if its definitely solved.
Thank for your time

I come back with some news, in my logs I found following entry "Looks like index BADLIST is not ready yet."
Is it an issue with index creation during init of the platform ?
How can I make sure to fix "maybe missing" indexes "safely" without having to rebuild the platform from scratch ?
If you have some way to handle that it will be a great news. :)

Maybe something went wrong during setup of the index in the elasticsearch. I built it without the elk monitoring stack so I might be a little limited in term of finding the root cause. Currently disabled Badlist service because that issue systematically leads to timeout during analysis.

Went a little deeper in elastic debug, seems my shards were corrupted but only for the badlist index. I am just sad it didn't throw an error on the GUI and entered in an silent error loop. I will improve my elastic skills before the next issue :). I mark as fixed. Thank you for your time.


Adds PrivateIPAddress in Digitalocean config.json

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "digitalocean-private-ip" git@github.com:jgrowl/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Hi @jgrowl I don't understand the goal of this PR. Can you explain? Thanks a lot

@dgageot Hi! Sorry, I'm not sure if anyone else would find this useful. I have been building tooling around docker-machine that sets up a consul server cluster. It is useful for me to have access to a machine's private ip address to be able to join servers to a bootstrapped server. I saw that the ec2 driver includes the PrivateIPAddress so that's why I added it in. Maybe it would be better to just rely on extracting it from a docker-machine ssh command?

Thanks for the feedback @jgrowl I think relying on docker-machine ssh is slower but reliable in time. As long as PrivateIPAddress is only exposed in config.json it's pretty sure it will disappear one day or another. @nathanleclaire wdyt?

I have always been wary about including IP address information in the serialized configuration directly.  IP addresses can change over time and putting them into the serialized file adds an additional "consistency" burden to the driver that the current Machine model does not support well.
I think it's probably not the end of the world to allow it, but I am also very hesitant to keep going down this slippery slope.  Most likely over time configs will need to be able to have a way to indicate a property is some kind of computed property (that inspect can properly query for), e.g. struct tags.

Thanks for the response.This PR can be closed then. I'll just use the ssh command to extract the private IP address for now.

OK thanks.  We'll keep private IP address querying in mind for a future feature though !


ostrstream deprecated

hi, i believe ostrstream is deprecated. well at least i keep getting annoying warnings about it being deprecated. here's a quick update:
a fix: cpplog.h
9: -> #include
180: -> std::ostringstream stream;
304: -> char lastChar = m_logData->buffer[m_logData->stream.str().size() - 1];
cheers!

oh whoops, that seemed to have broken it ... i will revert my changes and ignore the warnings for now.

Yeah, ostrstream is deprecated according to the standard - but as far as I can tell, there's no "official" replacement with the same features.  On my todo list is to write a replacement class that mimics the same functionality.

I don't know about this stream/buffer stuff ... but why can't u just use ostringstream?

In short, because of speed.  ostrstream allows you to specify the buffer to write to, whereas ostringstream manages its own buffer and requires additional copying - i.e the .str() method returns a copy, which is substantially less efficient.
Since a logging library might be called in a tight loop with speed requirements, I find that using ostringstream is too slow.

Ah right, no worries!


Ability to capture stderr?

Any recommendations on capturing stderr to the logger?

The only idea I have is to use standard Unix tricks. Stderr has fixed file handle. You can close it and open as a file which you would read, or a pipe to itself.


support for timestamps that use nanosecond resolution epoch time

Please add LogEntry.withTimestampNs(long)

Added in 3.0.4.


Change order of spawn points getter

Spawn points were used before being defined in the example code for setting up the simulation, this PR fixed that.

This change is

Updated to fix mistake in bounding boxes docs, analogously to this PR


Introduce response classifiers

By default, finagle treats all non-exceptional responses as a "success".  This
means that, for instance, 5XX http responses do not compute in success rates.
We introduce response classifiers to mark 5XX responses as failures.
Furthermore, we can use response classifiers to mark some failures as retryable
(i.e. depending on the request type). Unfortunately, Finagle doesn't yet honor
response classification when retrying requests.
Initially, response classification can be used simply to inform success rate
calculation. Later (#361), this classification may be used to automatically
retry some class of requests.
This change introduces a protocol-specific base response classifier and, for
http, marks all 5XX responses as failures. A response classifier plugin type is
introduced to allow this logic to be configurable, but this is of marginal
utility until requeues are supported.
Fixes #335

⭐ 🐑  Looks good to me.

:star: looks good, my comments are all minor suggestions

Can you add response classifiers to linkerd/CONFIG.md too?

I'd prefer to cover docs for this in the context of https://github.com/BuoyantIO/linkerd/pull/364.  Since we can't actually talk about retries yet, all of these classifiers are functionally equivalent (and so there's not really anything to configure...)

:star: 🐰


Foundation.toggler.js does not update aria-controls with ids of multiple toggled nodes

// Add ARIA attributes to triggers
    var id = this.$element[0].id;
    $(`[data-open="${id}"], [data-close="${id}"], [data-toggle="${id}"]`)
      .attr('aria-controls', id);

The current code only checks for a single id. I suggest updating to
// Add ARIA attributes to triggers
    var id = this.$element[0].id,
        $triggers = $(`[data-open~="${id}"], [data-close~="${id}"], [data-toggle~="${id}"]`);

    $triggers.each(function(index, element) {
       var $this = $(element);

       //make sure not to duplicate id in aria-controls
       $this.attr('aria-controls', `${$this.attr('aria-controls') ? $this.attr('aria-controls').replace(id, '') : ''} ${id}`.trim());
    });

How to reproduce this bug:
Compare https://foundation.zurb.com/sites/docs/toggler.html#toggle-a-css-class and https://foundation.zurb.com/sites/docs/toggler.html#multiple-targets
What should happen:
Toggler should get aria-controls
What happened instead:
Toggler only gets aria-controls when toggling a single node

https://www.w3.org/TR/wai-aria-1.1/#aria-controls
Should be a ID reference list

Hi @MoarCoding,
thanks for reporting this.
You are right, aria-controls should contain all of them.
Do you want to provide a PR?

https://github.com/zurb/foundation-sites/pull/11167


Reorganize the project content and add/updates features

UPDATES

reorganize the project structure
renforce uniqueness of roleAssignments name in bicep resources
[NOT COMPLETED] add the ability to disable in config.yaml unused llm tools

llm:
  excluded_llm_tools:
    - send_sms
    - search_document # not stable / still investigating
  slow: {}
  fast: {}

add the ability to disable Azure content safety during the bicep deployment

make deploy-bicep deploy-post name=my-rg-name enable_content_filter=false

[NOT FINALIZED] add the ability to specify in config.yaml the content vector field in the search index

ai_search:
  access_key: xxx
  endpoint: xxx
  index: trainings
  vector_content_field: "contentVector"

add the ability to disable language choice in config.yaml at the begining of the call

conversation:
  initiate:
    task: "xxx"
    agent_phone_number: "xxx"
    bot_company: xxx
    bot_name: xxx
    enable_language_choice: false

[NOT COMPLETED] add the ability to show debug info during the bicep deployment

make deploy-bicep deploy-post name=my-rg-name log_type=--debug

Observations (for developers)

Make sure to stop the function app (in Azure App Service) before function deployment
Please make sure you have Application Insights configured with your function app if you want to view function app logs: APPINSIGHTS_INSTRUMENTATIONKEY

@microsoft-github-policy-service agree

Hi @elvis-mdevo,
I recently managed to set up the original project after successfully overcoming various speed bumps on my way there (features not being available in my prefered regions, parameters no longer being available or being required like the content safety, openai quota exceeded, and so on).
Now that it's finally running I'm trying to improve/customize on similar things that you are mentioning in your pull request. Disabling IVR, disabling SMS feature, etc.
Love the improvements that you are mentioning/making and looking forward to them being included in the main branch!
Also just a quick question: Are you using synthetic-rag-index for supplying knowledge to AI Search? Is your vector size 0 bytes as well?

Hi @Bergdoktor,
Thanks you for your feedbacks. Hope that this version will be great for you.
For your question:

In my use case, I don't need to use synthetic-rag-index to build data injection pipeline. I just add the ability to configure ourself an index and set it to the configuration projet.

It there an incidence with this pull request ?

What already have been implemented:

Content Safety as a feature flag in the deployment
Reorganization with app and cicd folders

Would you be able to rebase your changes and send dedicated PR for each of your functional changes:

Folder for configuration named configs
Reliability for RBAC assignments (was not included in the commits if I'm not wrong)
Feature flag to disable the language selection in the initiate configuration

My 2 cents:

Have you been able to reproduce the async dns issue?
Is it really helpful for anyone to customize the vector field in AI Search? Data model customization is totally legit but in most cases many columns would requires to be added (ie security groups, business context, etc). In that case, just fork and edit the code.
What was the rational showing debug logs during bicep deployment?

Thank you by advance!


any_node: 0.5.0-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository any_node to 0.5.0-1:

upstream repository: https://github.com/ANYbotics/any_node.git
release repository: https://github.com/mikekaram/any_node-release.git
distro file: melodic/distribution.yaml
bloom version: 0.10.0
previous version for package: null

@clalancette thanks for the input. Although I'm not ANYbotics, I needed this package for releasing mine, so I took the liberty to do so. I will change the name as you suggested.
Thanks,
Mike


ARM and Intel builds create incompatible encrypted files

Hi,
I'm trying to encrypt a file using openssl 1.0.2l on a 64 bit x86 intel linux platform and decrypt it on a 32 bit arm cortex-A9 linux platform using openssl 1.0.2g. But this fails with a 'bad decrypt' message. Also the other way around fails. The x86 encrypted file can correctly be decrypted on windows with either 1.0.2g or 1.0.2o, so it seems it is not a version problem. Both systems can also decrypt their own generated files. I'm using AES256.
Should I assume the ARM build to be broken?
Maarten

I did not (consciously) build any version. The x86 linux version came with Ubuntu16, the Windows versions I downloaded from https://indy.fulgan.com/SSL/ and the ARM version came from Yocto Rocko which I think came precompiled as well.
From your comment I take it this not a known problem. I'll download the sources and try to cross-compile them myself. But I'm not sure yet how I should run 'make test' cross-platform.

I have pulled the 1.0.2p sources from git and cross-compiled them. I had to jump through some hoops to get the so's as well. The resulting openssl + libs create aes256 encrypted files on arm that the x86 can read and vice versa.
So I guess I'm barking up the wrong tree and should turn to Yocto instead.

On the openembedded mailing list I got an answer that tells me this problem was an issue (#4659) in openssl, caused by either openssl sources or by bin-utils (2.29) which should now be fixed.

Recording for posterity, cross-compiling and 'make test' do not currently play well together.

Btw. binutils have also reverted the behavior of ADR under normal conditions in 2.31.

Thanks for report!
As for 'make test' and cross-compiling not getting along. While this is not untrue in most general case, it's not that impossible on Linux for Linux targets, thanks to qemu-user. Quite contrary, very simple once you figure it out. One even has two options. One is env EXE_SHELL=qemu-<arch> make test, another is to setup binfmt to call qemu-, so that alien binaries can be executed from shell prompt as if they were native.


Wrong definition of COAP_MAX_TRANSMIT_WAIT

The definition of COAP_MAX_TRANSMIT_WAIT in er-coap.h is wrong.
The caret is a binary XOR and not an operator for power calculation. Probably a pow() function shall be used instead.

in the CoAP RFC:
ACK_TIMEOUT * ((2 ** (MAX_RETRANSMIT + 1)) - 1) * ACK_RANDOM_FACTOR
in the wakaama code:
#define COAP_MAX_TRANSMIT_WAIT               ((COAP_RESPONSE_TIMEOUT * ((2 ^ (COAP_MAX_RETRANSMIT + 1)) - 1) * COAP_ACK_RANDOM_FACTOR))
** was translated into ^
I suppose ** mean powerof

Indeed, it is 2 to the power of  (MAX_RETRANSMIT + 1).
It doesn't really have much of an impact on the correct behavior, since it is only used to guard against registration failure due to datagram loss.

Can you review the fix here: https://github.com/eclipse/wakaama/pull/222

Thats not quite correct. It has to be something like
#define COAP_MAX_TRANSMIT_WAIT      ((COAP_RESPONSE_TIMEOUT * ( pow(2, (COAP_MAX_RETRANSMIT + 1) ) - 1) * COAP_ACK_RANDOM_FACTOR))
And then of course the function pow needs the math.h header included.

I did it this way to avoid a dependency on math.h
using  (COAP_MAX_RETRANSMIT + 1) * (COAP_MAX_RETRANSMIT + 1) in place of pow(2,   (COAP_MAX_RETRANSMIT + 1))

That's what I thought too, but its not the same.
(COAP_MAX_RETRANSMIT + 1) * (COAP_MAX_RETRANSMIT + 1) is the same as (COAP_MAX_RETRANSMIT + 1) to the power of 2 and not 2 to the power of (COAP_MAX_RETRANSMIT + 1)
Alternatively a small pow function implementation might be added.

try #2 using bit shifting

That is definitely the more elegant solution 👍

Nice catch @vuzunov. How could I do that ?..
Thanks for the fix @jvermillard

Fixed by #222


Chore heroku deps and cluster

Tickets:

Fixes #23
Fixes #22

Notes:

Adds cluster via recluster
Moves dev deps back to devDependencies and uses heroku install script to conditionally install some of those...

Coverage remained the same at 88.41% when pulling 0c4ccd061d43d8ba2b527ab90e3893005c272a5b on chore-heroku-deps-and-cluster into e27fd150d0556b94bff0e35421bbc5307173e1dd on master.

Coverage remained the same at 88.41% when pulling 40799f5391acad9dbdbefd837a9d5a672c2c89d4 on chore-heroku-deps-and-cluster into e27fd150d0556b94bff0e35421bbc5307173e1dd on master.

Coverage remained the same at 88.41% when pulling 483165af054f8cfd1eae6e293712a51366569cbb on chore-heroku-deps-and-cluster into e27fd150d0556b94bff0e35421bbc5307173e1dd on master.

Coverage remained the same at 88.41% when pulling dd65a2234d6467754f65637fe6c8cac12b57e10e on chore-heroku-deps-and-cluster into e27fd150d0556b94bff0e35421bbc5307173e1dd on master.


Allow sort in ascending or descending order

I have defined sort by votes. When I search in AUR the last results I see in console are the least votes, with a reverse order the last results showed would be the most votes.

A --sort and --rsort options need to be added. At the moment, --rsort is passed to cower by default.
Also, the sort command line options (--asort, --psort, --vsort) also need to be reworked. Having two options only somewhat made sense, but since AUR 4 --psort has been added. This is hardly a clean implementation. They should be replaced by --[r]sort=(alpha|votes|popularity) with default set in the config file.

Done in 3a5b7a9340ae43e04c6cadeee0b21d314cde1530.


Development

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

binding/Binding/Definitions.cs
:white_check_mark:Succeeded

binding/Binding/GRBackendRenderTarget.cs
:white_check_mark:Succeeded

binding/Binding/GRBackendTexture.cs
:white_check_mark:Succeeded

binding/Binding/GRDefinitions.cs
:white_check_mark:Succeeded

binding/Binding/SKCodec.cs
:white_check_mark:Succeeded

binding/Binding/SKFontManager.cs
:white_check_mark:Succeeded

binding/Binding/SKFontStyleSet.cs
:white_check_mark:Succeeded

binding/Binding/SKImage.cs
:white_check_mark:Succeeded

binding/Binding/SKPath.cs
:white_check_mark:Succeeded

binding/Binding/SKPixelSerializer.cs
:white_check_mark:Succeeded

binding/Binding/SKTypeface.cs
:white_check_mark:Succeeded

binding/Binding/SkiaApi.cs
:white_check_mark:Succeeded

build.cake
:white_check_mark:Succeeded

cake/UpdateDocs.cake
:white_check_mark:Succeeded

cake/UtilsManaged.cake
:white_check_mark:Succeeded

changelogs/SkiaSharp.Views/1.55.0/SkiaSharp.Views.iOS.md
:white_check_mark:Succeeded

changelogs/SkiaSharp.Views/1.55.0/SkiaSharp.Views.tvOS.md
:white_check_mark:Succeeded

changelogs/SkiaSharp.Views/1.68.0/SkiaSharp.Views.UWP.md
:white_check_mark:Succeeded

changelogs/SkiaSharp/1.68.0/SkiaSharp.breaking.md
:white_check_mark:Succeeded

changelogs/SkiaSharp/1.68.0/SkiaSharp.md
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/harfbuzzsharp-<IP_ADDRESS>.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-harfbuzz-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-views-android-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-views-forms-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-views-ios-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-views-macos-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-views-net-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-views-tizen-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-views-tvos-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-views-uwp-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/FrameworksIndex/skiasharp-views-watchos-1.68.0.xml
:white_check_mark:Succeeded

docs/xml/HarfBuzzSharp/Blob.xml
:white_check_mark:Succeeded

docs/xml/HarfBuzzSharp/Buffer.xml
:white_check_mark:Succeeded

docs/xml/HarfBuzzSharp/Face.xml
:white_check_mark:Succeeded

docs/xml/HarfBuzzSharp/Font.xml
:white_check_mark:Succeeded

docs/xml/HarfBuzzSharp/NativeObject.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.HarfBuzz/BlobExtensions.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.HarfBuzz/CanvasExtensions.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.HarfBuzz/FontExtensions.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.HarfBuzz/SKShaper+Result.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.HarfBuzz/SKShaper.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.Views.Android/AndroidExtensions.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.Views.Android/Extensions.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.Views.Android/GLTextureView+IEGLConfigChooser.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.Views.Android/GLTextureView+IEGLContextFactory.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.Views.Android/GLTextureView+IEGLWindowSurfaceFactory.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.Views.Android/GLTextureView+IGLWrapper.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.Views.Android/GLTextureView+IRenderer.xml
:white_check_mark:Succeeded

docs/xml/SkiaSharp.Views.Android/GLTextureView.xml
:white_check_mark:Succeeded

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

build


Tests are broken on openbsd

Running tests on OpenBSD is currently non-functional. The issue is, that the setfattr tool cannot be found as openbsd does not support extended attributes. We need a different way to define attributes. OpenBSDs pkg currently ships an old version of squashfs-tools that doesn't support reading xattrs from pseudo files.

Fixed in 79bbc8c11d046f97fc4b5af8926c017038f4cf01


Allow PgSQL enums for discriminator columns

Feature Request

Q
A

New Feature
yes

RFC
no

BC Break
no

Summary

At this moment DiscriminatorColumn values are converted into SQL in SingleTablePersister::getSelectConditionDiscriminatorValueSQL like this:
return $tableAlias . '.' . $discColumn . ' IN (' . $values . ')';,
where $values is an comma-separated list of $this->conn->quoteed values of discriminatorMap.
Thus, it is impossible to turn it into a PgSQL-valid filter like t0.type IN ('type_a'::my_type_enum, 'type_b'::my_type_enum) without changing SingleTablePersister.
It would be great to add support for PgSQL enums for discriminator columns.

My bad. If you make the descriminator column values in the upper case, PgSQL will automatically convert strings to enums.

This seems to already be fixed in master: https://github.com/doctrine/doctrine2/blob/b40c2c6b7893c929f754150d480c20f9b22e0afe/lib/Doctrine/ORM/Persisters/Entity/SingleTablePersister.php#L167


Add gammatone filters to documentation

We forgot to do that.

https://pyfar.readthedocs.io/en/enhance-filter_type_documentation/index.html