withHubProtocol(new signalR.protocols.msgpack.MessagePackHubProtocol()) Disconnect when connected

1.Use javascript:
connection = new signalR.HubConnectionBuilder().withUrl("/feedback")
.configureLogging(signalR.LogLevel.Information)
.withHubProtocol(new signalR.protocols.msgpack.MessagePackHubProtocol())
.build();
After connecting, the server pushes the information and immediately disconnects it.
2.Use javascript:
connection = new signalR.HubConnectionBuilder().withUrl("/feedback")
.configureLogging(signalR.LogLevel.Information)
.build();
Will not disconnect.
Why?

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

ID: 3cf283c9-5cd2-a9ff-b8c8-372d8a04356d
Version Independent ID: 729fddcf-24e4-b563-4952-a8045b231868
Content: ASP.NET Core SignalR JavaScript client
Content Source: aspnetcore/signalr/javascript-client.md
Product: aspnet-core
Technology: aspnetcore-signalr
GitHub Login: @bradygaster
Microsoft Alias: bradyg

Have you configured your server to support the MessagePackHub protocol?

Please provide a case, thank you.
I checked the official website and I didn't find it。

Configuration is added to the configureservices method in the statup.cs file
services.AddSignalR()                 .AddMessagePackProtocol(options=>                 {                     options.FormatterResolvers = new List<MessagePack.IFormatterResolver>()                     {                         MessagePack.Resolvers.StandardResolver.Instance                     };                 });

Looks like you found it. Is your issue resolved?

No solution，I hope you.

How can I help you?

If you're still having a problem, open an issue at https://github.com/dotnet/aspnetcore.


HTTP 408 upstream request timeout ERROR when making online endpoint request

Hi,
I've deployed my ML model and entry script into an online endpoint dyn-test-endpoint.
subscription: fcd57b7d-d387-4998-8349-63b97b5089c7
resource group: lsdresources1
workspace name: modelscoring-dev
When calling the endpoint, I always got 408 error. But from deployment log I can see the request is successfully handled without any problem. See below screenshot. From portal test place, the same error happens.

So please help me with this problem.
Thank you,
Harry

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

ID: a852f927-cbbd-634a-d81e-78359d539ee3
Version Independent ID: 0af3f631-2669-38a9-9b17-fe5e9893b711
Content: Deploy an ML model by using an online endpoint - Azure Machine Learning
Content Source: articles/machine-learning/how-to-deploy-managed-online-endpoints.md
Service: machine-learning
Sub-service: mlops
GitHub Login: @rsethur
Microsoft Alias: seramasu

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

Close this issue since I've figured out how to set up timeout limit in YAML config for deployment. BTW, default 5 seconds are too short.

@sunnyhay I am facing the exact same issue. Is it possible to tell me how to increase the timeout limit in the YAML file?
And is it the same YAML file used for deployment and specifications of dependencies?

@sunnyhay I am facing the exact same issue. Is it possible to tell me how to increase the timeout limit in the YAML file? And is it the same YAML file used for deployment and specifications of dependencies?

I just wanted to add that in my case, I can see the results in the deployment logs, but I am getting an upstream request timeout from postman

I had the same issue but I solved it in a different way: I added request settings in the ManagedOnlineDeployment class.

@AmmarMohanna saved my life dude !

@AmmarMohanna Thanks Bro.

Can somebody please provide more instructions on where exactly to do that? I am creating an prompt flow endpoint. After 90 seconds it stops with the 408 upstream request timeout, Can this be done on the endpoint itself as well?
Thank you

Is there a way to add the request settings when deploying using AKS?


Implement Additional Security Headers

X-XSS-Protection: 1; mode=block
X-Frame-Options: DENY   (dont go into a frame!)
X-Content-Type-Options: NOSNIFF  (everything

Fixed by PR #192


Use old value

We should put the old value of skip_on_method_added back at the end, instead of blindly setting it to false. (I realized this when you were adding some skip_on_method_added to pay-server.)
Maybe also, instead of directly manipulating this flag, we should use something like
T::Configuration.without_on_method_added do
  # statements in here won't trigger our `_on_method_added` in T::Private::Methods
  # you may also nest these blocks arbitrarily and they won't clobber each other
end

If something like this ^ would be more preferred I'll repurpose this PR to do that.
Motivation
To make sure we don't clobber someone else's skip.
Test plan
No tests. Not sure if there could or should be tests for this.

I'd actually prefer the without helper, specifically because it should be implemented like this:
def self.without_on_method_added(&blk)
  begin
    old_value = skip_on_method_added
    yield
  ensure
    skip_on_method_added = old_value
  end
  nil
end

and put it on T::Private::DeclState, not T::Configuration.

ack, but i think there should be a skip_on_method_added = true before the yield


New interface for random.financial

Implements the interface discussed in #763, allowing for controlling the time increment #754.
Time increment is provided as a number of seconds, not a d3 time interval - it's not possible to create a d3 time interval of 5 mins, for example.
Replaces volume model with something simpler, and makes it configurable as a functor property.
Component function now does not mutate state - stateful usage is provided by the closure returned from generator.stream(), as proposed.

Todo:

[x] Convert usages
[x] Improve documentation

We have the use case of using this component to create an initial array, then add new data using the stream, and we want the open price of the first streamed point to be related to the last point of initial data.
What's preferred?
var generator = fc.data.random.financial()
  .start(new Date(2016, 0, 1));

// Currently
var data = generator(new Date(2016, 1, 1));
generator.startDate(new Date(2016, 1, 1));
generator.startPrice(/* something sensible */);
var stream = generator.stream();
data.push(stream());

// Or this?
var stream = generator.stream();
var data = stream(/* end date and/or number of points? */); // Returns array
data.push(stream()[0]);

// Or revert to having generator mutate date and price?
var data = generator(new Date(2016, 1, 1));
var stream = generator.stream();
data.push(stream());

Thanks @tlsim - this is going to be a big improvement. Here's some notes on our discussion:
The generator shouldn't have an API that distinguishes between generating an array of items versus streaming single datapoints. One good way to solve this is to make everything a streaming operation.
// a factory that creates streams with a given initial state.
var generator = fc.data.random.financial()
  .start(new Date(2016, 0, 1));

// construct a stream from the generator - you can of course do this multiple times.
var stream = generator();

// populate an array by asking for a number of points
var data = stream.next(20);

// stream new items into the array
data.push(stream.next());
data.push(stream.next());
data.push(stream.next());

// another way to populate the initial data
var stream2 = generator();
var data2 = stream2.until(function(datum) { datum.date > new Date(2015,12,12); });

One final (?!) tweak to the API -
var dataGenerator = fc.data.random.financial();

var stream = dataGenerator.stream();

var snapshot = stream.take(2); // starts from start (first call to stream) [{}, {}]

var datum = stream.next(); // continues from above {}

stream.until(function(datum) { datum.date > new Date(2015,12,12); }) // continues from above [{},...]

var data = dataGenerator(2); // always starts from start date [{}, {}]

The above code tries to ensure consistent return types and allow the following 3 use cases -

Generate n data sets of a fixed range
Subscribe to a stream of data points
Request a snapshot to back-fill a range, then stream in new data

Updated.
One more thing to check:
stream.until takes function(datum) but filter takes function(date). This might feel inconsistent. Is this worth changing?

One more thing to check: stream.until takes function(datum) but filter takes function(date). This might feel inconsistent. Is this worth changing?

:+1: for changing the filter function for consistency.

@ColinEberhardt @chrisprice thanks for the guidance! I think this is ready to review.

Updated to address comments. Matching RxJS takeUntil more closely has simplified things. Also now allows for setting the units of mu and sigma with an interval offset, so no more 'ms per year' constant.

Thanks for the update @chrisprice are you ok to review?

LGTM :+1:
Merging is going to break the site as per #873.


buildPythonPackage: always export LANG=UTF-8

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Note I will need to clean up the expressions a bit (used git grep and sed here).

As I said in https://github.com/NixOS/nixpkgs/pull/54485#issuecomment-461748252
we also need to support this for the macOS stdenv as well or
else we would break macOS support horrible.
It is probably just a simple patch. However I can not test it, since
I don't have access to the hardware/software. If somebody wants to help me out on that, let me know. Also I would move this to stdenv actually since also other application can profit from that.

I see, so we're blocked then.

For Darwin we can use the approach as in https://github.com/NixOS/nixpkgs/commit/8220b5bac8e934e8d6c6cc3510f3ed33b1bd2a50 for now.

Included https://github.com/NixOS/nixpkgs/commit/1fccd255953b6025a4d0342c2cf5641e5b6b6ced in staging-next. Cleaning up expressions is for another time.


Be more flexible about result code returned in test

Status code may vary from a bad 'ls'; which may be O.S. dependent. Travis returns a different value.
It's kind of odd, really; I merged a passing PR into master, and  it started failing here. Change in behavior on Travis?

Changes Unknown when pulling 4f1bc955bf490891a0390db3ab7aa5acfb948161 on status-value-in-test into ** on master**.


assignee on entry disappeared (only visually) when going backward or forward in firefox

Replaced by #52


feat(obstacle_avoidance_planner): parameterize non_fixed_trajectory_length

Signed-off-by: Mehmet Dogru<EMAIL_ADDRESS>Description
This PR parameterize non_fixed_trajectory_length to be able to tune it easier.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@takayuki5168 to please review.

Thanks for your contribution. I'll review this weekend.


Update wording of README.md > params > filter

I believe the documentation reads as if the files matching the predicate are going to be excluded when it is actually the opposite!

我也有相同问题，文档中filter的说明应该是指定哪些资源压缩


Support for typescript (angular 2 templates)

I would like to see this working in combination with angular2-inline extension. It wasn't clear to me when the css parser is triggerd. Is it only on html files?

@Aaltuj sorry for this late response. Yeah, currently it only works on HTML files.

Closing this issue as the latest version now has support for TypeScript.


Add helm chart and terraform module

I created a helm chart for the project I'm currently working on and have decided to port it to yace project. Included is the slightly modified helm chart and a terraform module to deploy yace via kind: ServiceAccount.
Hopefully, this helps!

Hey @withernet I currently have no intentions to maintain a helm repository for yace.
Feel free to create it in your own repository and pull request a link to your helm chart.
Greetings, Thomas


Update api level 5

Codecov Report

:exclamation: No coverage uploaded for pull request base (API_LEVEL_5@52f6b9d). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@              Coverage Diff               @@
##             API_LEVEL_5     #341   +/-   ##
==============================================
  Coverage               ?   85.86%
==============================================
  Files                  ?        8
  Lines                  ?      389
  Branches               ?        0
==============================================
  Hits                   ?      334
  Misses                 ?       55
  Partials               ?        0

Flag
Coverage Δ

unittests
85.86% <0.00%> (?)

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

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


Fix broken link in README.md

I was reading the README when I saw a link was broken, so here is the fixed version.

Thanks!


how can I to configure SSL flag when creating a connection?

Will be something like this?, somebody can help me please?
var connection = mysql.createConnection({
  host     : 'localhost',
  user     : 'me',
  password : 'secret',
  database : 'my_db',
  ssl: true
});

I'm having this error:
[Error: Server does not support secure connnection] stack: [Getter]

This error is because the MySQL server you are trying to connect to is not configured for SSL. You can read more about setting up your MySQL server for SSL connections in the MySQL manual here: https://dev.mysql.com/doc/refman/5.6/en/using-ssl-connections.html

@dougwilson  thanks for helping me. I've already configured MySQL to Use SSL Connections as https://dev.mysql.com/doc/refman/5.6/en/using-ssl-connections.html, but the error continues, maybe is there other step after configure? thanks in advance :smile:

It's no problem! In order to use SSL, you cannot simply just do ssl: true :( You have to provide all the options, including the certificate in an object (more explained here: https://github.com/felixge/node-mysql#ssl-options).
Also, are you still getting the error "Server does not support secure connnection" or a different error? The error "Server does not support secure connnection" specifically means that your MySQL server itself is not correctly configured, rather than anything you are doing with this client.

isn't there the possibility to only enable this flag? (ssl:true)  https://github.com/felixge/node-mysql#other-available-flags, without adding any certificate in an object?

It is not possible with how MySQL works. If you can determine a way, PRs are accepted!
Also, are you still getting the error "Server does not support secure connnection" or a different error? The error "Server does not support secure connnection" specifically means that your MySQL server itself is not correctly configured, rather than anything you are doing with this client.

yeah I think the main problem is I'm continue using this
ssl: true

jejeje I'll change that and test again :smile:
thanks you so much for help!

just a question more please, is not clear for me this flags https://github.com/felixge/node-mysql#other-available-flags, when and how can I use?. This way?
var connection = mysql.createConnection("mysql://localhost/test?flags=-FOUND_ROWS");

If I use SSL, what will the behavior?

If you need to use the flags, you can use them with that string format, but we recommend you use the object format so that all options are available to you:
var connection = mysql.createConnection({
  database: 'test',
  flags: '-FOUND_ROWS'
})

but what will do the flag SSL ?

Those flags are just listed because they are in the source code for the MySQL command line client as part of MySQL; they have no functionality in this module.

ohhh now I get it :smile:  thanks


Potential fixes for datadog exporter (very long initialization)

Hello.
This follows issue #591
After doing more investigation, it looks like the datadog agent is sometime (depending on where you are hosted) taking more than 1 minute to initialize. blocking the readiness/liveness probes until that
See: https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/16442
Right now there is no way to use the current chart in this case, there are potential fixes:

Allow to specify custom timeouts for liveness/readiness probes.

Allow to pass custom environment variables, this would allow to setup a workaround to force the hostname based on kubernetes attributes instead of the potentially long datadog hostname detection, to refer it in the configuration file

Example of custom environment variable that would be needed
        env:
        - name: NODE_NAME
          valueFrom:
            fieldRef:
              fieldPath: spec.nodeName

To be referred in the configuration file as
config:
  exporters:
    datadog:
      host_metadata:
        hostname: "${env:NODE_NAME}"

Ideally both options should be possible.
Thanks

@kedare does this work for you?
mode: deployment

config:
  exporters:
    datadog:
      host_metadata:
        hostname: "${NODE_NAME}"
  service:
    pipelines:
      traces:
        exporters:
          - datadog
          - logging

extraEnvs:
  - name: NODE_NAME
    valueFrom:
      fieldRef:
        fieldPath: spec.nodeName

Like you said, we can also update the chart to allow configuring the probes.

Thank you, this approach is working fine (also I made a mistake the hostname propery need to be just under the datadog property)


Fails to block video ads

Blockify works wonderfully so long as the ads are audio only. I can make Spotify play only audio ads by not having Spotify in focus. The thing is I like to have Spotify visibile quite often which renders blockify useless.

Thanks for the report, I wasn't aware. I'll definitely look into it but it might take me some time to get around to it.

I cannot seem to get video ads. They simply don't occur on my system.
To be able to block them, I'd need to collect some context information about video ads by which i can identify them.
It would be of great help if someone could provide the output of blockify -vvv, spotify and blockify-dbus get all while a video ad is playing.

I've got a premium trial at the moment, which I think it ends today or tomorrow. I'll have a go collecting some results, if no one provides anything in the interim.

Please try again, video ads should now be muted, too.

Unfortunately, although this fix does seem to work some of the time, it doesn't seem to work consistently.
Here's some output that might help in working out what's going on.
Example when blockify is working:
blockify-dbus get all:
artUrl		=
length		= 30000000
trackid	= spotify:ad:000000000000000000000015ff2513dc
album		=
albumArtist	= dbus.Array([dbus.String('')], signature=dbus.Signature('s'), variant_level=1)
artist		=
autoRating	= 0.0
discNumber	= 0
title		= House
trackNumber	= 0
url		= https://open.spotify.com/ad/000000000000000000000015ff2513dc

wmctrl -lx:
0x02c00001  1 spotify.Spotify       Hostname Spotify

where Hostname is the hostname.
Example when blockify isn't working:
blockify-dbus get all:
artUrl		= https://open.spotify.com/image/9d764c6feba0f5bc629f0ca76666efe1c2495563
length		= 166573000
trackid	= spotify:track:2i63MeQVdPuLvDH4EdaqXO
album		= Tourist History
albumArtist	= dbus.Array([dbus.String('Two Door Cinema Club')], signature=dbus.Signature('s'), variant_level=1)
artist		= Two Door Cinema Club
autoRating	= 0.6
discNumber	= 1
title		= Undercover Martyn
trackNumber	= 7
url		= https://open.spotify.com/track/2i63MeQVdPuLvDH4EdaqXO

These are the details of the previously played song.
wmctrl -lx:
0x04200001  1 spotify.Spotify       Hostname Spotify

It might be useful to compare the output of wmctrl -lx during an advert with the output of during a song.
e.g.
0x04200001  1 spotify.Spotify       Hydra Two Door Cinema Club - Undercover Martyn

It seems that blockify-dbus get all sometimes returns info from the last song but wmctrl -lx is always up to date.

Hi @Spagy,
what you're describing is expected behaviour. Spotify won't trigger DBus updates for some of their ads. That's why i'm cross checking via wmctrl. If Spotify is playing and wmctrl shows nothing besides "Spotify", the currently playing track is assumed to be an ad.
So, theoretically, from the information you have provided, the song should have been recognized as an ad in both cases because wmctrl did show only "Spotify".
Possible reasons why it didn't work:

The window title check is skipped because your mute method is not pulsesink_mute. If you can run pacmd list-sink-inputs, this is not your problem.
Spotify playback status is not correctly recognized as playing. This, again, can happen if your mute method is not pulsesink_mute.

Please check if you can run pacmd list-sink-inputs.

Or there's a third, slightly more embarrassing, option. I forgot to update one of my machines so I was old version of blockify. Thanks for trying to help


Use GraphQL port from env variable in server render

GraphQL port is currently hard coded. A quick try to use dotenv was not successful. GRAPHQL_PORT was still undefined. Maybe we need to set it in server webpack config

Closed via #46


tools: Add cmux command line tool

Summary
Usage: ./nuttx/tools/cmux -d  -n  -b
Please refer to the following link for more information:
https://docs.kernel.org/6.1/driver-api/tty/n_gsm.html
Impact
new host tool
Testing
ci

[Experimental Bot, please feedback here]
No, this PR does not fully meet the NuttX requirements. While it provides a summary and mentions testing via CI, several crucial pieces of information are missing.
Here's a breakdown of what's lacking:

Insufficient Summary Detail: The summary provides basic usage but doesn't explain why this new tool is necessary. What problem does it solve?  What existing functionality does it improve upon (or replace)?  How does it work internally?  Are there related NuttX issues?

Missing Impact Assessment:  Simply stating "new host tool" is insufficient.  Consider the impact points listed in the requirements:

Feature: Is it a new feature?  Or does it replace/modify an existing one?
User Impact: How will users interact with this tool?  Will existing workflows change?
Build Impact: Does adding this tool require changes to the NuttX build system?  Does it introduce new dependencies?
Hardware Impact: Does this tool interact with hardware in any specific way? Is it architecture-specific?
Documentation:  This new tool requires documentation explaining its purpose, usage, and options.
Security/Compatibility:  These points haven't been addressed. Even if the impact is "NO," it should be explicitly stated.

Inadequate Testing Information:  "ci" is not enough. While CI testing is important, the PR needs to show specific test results.

Build Host Details: Specify the OS, CPU architecture, and compiler version used for testing.
Target Details: Which architecture(s) and board configurations was this tested on?  Was it tested on real hardware or a simulator?
Logs:  The PR needs to include "before" and "after" logs demonstrating the tool's functionality and the problem it solves (or the improvement it delivers).  Simply saying "ci" provides no evidence that the tool actually works.

In short, the PR needs to be significantly more detailed to meet the NuttX requirements.  It needs to provide a thorough explanation of the changes, their impact, and clear evidence of testing.

@xiaoxiang781216 I think this application depends on CMUX support in the NuttX kernel, is there a CMUX driver coming to NuttX kernel?

@xiaoxiang781216 I think this application depends on CMUX support in the NuttX kernel, is there a CMUX driver coming to NuttX kernel?

Yes, it will come in the next patch.

@xiaoxiang781216 I was thinking this was CMUX support for NuttX, but seems like it has other purpose, probably to run on Linux side. Could you please explain the idea of this tool? Please create a usage() function explain the purpose of this tool.

already in the code:
static void usage(const char *progname)
{
  fprintf(stderr, "Usage: %s -d <device> -n <number> -b <baudrate>\n",
          progname);
  exit(EXIT_FAILURE);
}

Please could you add a brief explanation what is cmux and what it is for in the PR description and commit log? Thanks.

Update, you can learn the concept and spec from:
https://docs.kernel.org/6.1/driver-api/tty/n_gsm.html

Please could you add a brief explanation what is cmux and what it is for in the PR description and commit log? Thanks.

Update, you can learn the concept and spec from: https://docs.kernel.org/6.1/driver-api/tty/n_gsm.html

Thank you Xiang! What is the purpose of this external too to NuttX, since NuttX doesn't have CMUX support yet?

Please could you add a brief explanation what is cmux and what it is for in the PR description and commit log? Thanks.

Update, you can learn the concept and spec from: https://docs.kernel.org/6.1/driver-api/tty/n_gsm.html

Thank you Xiang! What is the purpose of this external too to NuttX, since NuttX doesn't have CMUX support yet?
I think this is the question I and everybody that find this program at nuttx/tools/ will ask. This application is for Linux, not for NuttX, correct?

Yes, that's why this tool is put into nuttx/tools, not apps/.

@xiaoxiang781216 I think /me and @hartmannathan are asking some question: why to include a program that is supported only for Linux into NuttX?

Because you need this tool to change host serial port into mux mode.

The documentation you provided is for Linux, please consider including a Documentation/ to explain the goal and purpose to have a Linux host CMUX GSM

@Gary-Hobson will provide a NuttX serial driver which implement CMUX protocol.

instead of adding CMUX support on NuttX and adding this cmux tool at nuttx-apps

This tool doesn't need for NuttX since NuttX CMUX driver auto switch to mux mode by config, not through ioctl dynamically.

@xiaoxiang781216 I think this application depends on CMUX support in the NuttX kernel, is there a CMUX driver coming to NuttX kernel?

@xiaoxiang781216 there is an issue already opened asking for CMUX support on NuttX: #7153

Yes, after @Gary-Hobson upstream MUX serial driver, #7153 should work.

Yes, it will come in the next patch.

But in this case the cmux application should be inside nuttx-apps, instead of nuttx/tools/, right?

No, NuttX mux driver doesn't need the setup like Linux.

Please could you add a brief explanation what is cmux and what it is for in the PR description and commit log? Thanks.

Update, you can learn the concept and spec from: https://docs.kernel.org/6.1/driver-api/tty/n_gsm.html

Thank you Xiang! What is the purpose of this external too to NuttX, since NuttX doesn't have CMUX support yet?
I think this is the question I and everybody that find this program at nuttx/tools/ will ask. This application is for Linux, not for NuttX, correct?

Yes, that's why this tool is put into nuttx/tools, not apps/.

@xiaoxiang781216 I think /me and @hartmannathan are asking some question: why to include a program that is supported only for Linux into NuttX?

Because you need this tool to change host serial port into mux mode.

Why? Are you using the modem with SIM? Normally the CMUX will work integrated with PPP and the user should issue a command (or an IOCTL) to setup the CMUX (multiplexed) ports.

The documentation you provided is for Linux, please consider including a Documentation/ to explain the goal and purpose to have a Linux host CMUX GSM

@Gary-Hobson will provide a NuttX serial driver which implement CMUX protocol.

Great! Kudos @Gary-Hobson !

instead of adding CMUX support on NuttX and adding this cmux tool at nuttx-apps

This tool doesn't need for NuttX since NuttX CMUX driver auto switch to mux mode by config, not through ioctl dynamically.

Hmm, I think having IOCTL support could be useful in case where we want to enter / leave CMUX during the PPP execution.

@xiaoxiang781216 I think this application depends on CMUX support in the NuttX kernel, is there a CMUX driver coming to NuttX kernel?

@xiaoxiang781216 there is an issue already opened asking for CMUX support on NuttX: #7153

Yes, after @Gary-Hobson upstream MUX serial driver, #7153 could be closed as complete.

Yes, it will come in the next patch.

But in this case the cmux application should be inside nuttx-apps, instead of nuttx/tools/, right?

No, NuttX mux driver could work automatically without the setup like Linux, that's why this tool put into nuttx/tools/ not apps/.

Hmm, by automatically you mean statically, right? It should be use some board config to enter in this CMUX mode, correct?

@xiaoxiang781216 maybe you can include a verbose explanation inside this C file (just before "Included Files") explaining the purpose of this file inside nuttx/tools. This way it could help the absence of proper Documentation/ to this script

Because you need this tool to change host serial port into mux mode.

Why? Are you using the modem with SIM?

We use cmux for debugging purpose to have the multiple virtual port for nsh/log/gdb at the same time on one physical uart.

Normally the CMUX will work integrated with PPP and the user should issue a command (or an IOCTL) to setup the CMUX (multiplexed) ports.

in this case, you can bypass cmux tool and do ioctl in your program.

This tool doesn't need for NuttX since NuttX CMUX driver auto switch to mux mode by config, not through ioctl dynamically.

Hmm, I think having IOCTL support could be useful in case where we want to enter / leave CMUX during the PPP execution.

it's hard to handle the enter/leave without the lost data, especially we want syslog can be worked as soon as possible in any environment.

No, NuttX mux driver could work automatically without the setup like Linux, that's why this tool put into nuttx/tools/ not apps/.

Hmm, by automatically you mean statically, right? It should be use some board config to enter in this CMUX mode, correct?

you can config through:

Which physical port you want to use CMUX
how many virtual serial ports you want

After initialization, you will see all virtual serial ports under /dev/ and use them like the normal uart. Of course, the other side(e.g. Linux) need config by their design as usually.


Attribute._value is not updated with the PV

Internal .get() on values just return the initial value
Steps To Reproduce
Steps to reproduce the behavior:
from fastcs.controller import Controller
from fastcs.attributes import AttrRW
from fastcs.datatypes import Int
from fastcs.launch import FastCS
from fastcs.transport.epics.options import EpicsIOCOptions, EpicsOptions

class TestValueConroller(Controller):
    value = AttrRW(Int())

if __name__ == "__main__":
    epics_options = EpicsOptions(ioc=EpicsIOCOptions(pv_prefix="DEVICE"))
    controller = TestValueConroller()
    fastcs = FastCS(controller, epics_options)
    fastcs.run()

The PV works as expected:
─>$ caput DEVICE:Value 10000
Old : DEVICE:Value                   0
New : DEVICE:Value                   10000
...
─>$ caget DEVICE:Value
DEVICE:Value                   10000

but in the IOC console:
>>> controller.value._value
0

regardless of what the PV is set to

I think this is because on_update (from ioc.py:259) only calls process_without_display_update() and not process(). I don't really know  enough about how this works, but can we make the default _process_callback be updating the Attribute value?


update README

for users, before next release, we should update README with instructions how to update to native-cuda version

will be handled by @McBs in #195


Mini player (with full cover) is updated only when main window is active

<IP_ADDRESS>-beta, 10.9.5

Is it possible to capture a video of this scenario? Or more explicit steps to reproduce? I'm unable to reproduce this right now. :confused:

Sorry, my bad.
When you open up mini player, the slider and seconds counter are frozen on a song current position. The artwork though is changed when the song is ended.
However, when you open up main window, then the slider begins to move forward synchronously.

I'm encountering the same behavior regarding the slider and progress bar being paused. With the main window open, it proceeds as expected.

Hey @nashbridges & @justonian  - I know this is a really old issue, but there's been a ton of fixes/improvements since it was logged.
If possible, could you check latest code and see if you're still seeing the issues you're reporting here? You can either build from master, or for convenience I've tagged a release on my fork of everything up to today: https://github.com/chrismou/radiant-player-mac/releases/tag/v1.5.0-20151213
Thanks!

@chrismou I've tested your fork and the reported bug is fixed there. However, album art is not loaded neither in mini player, nor in dock (which obviously is an unrelated issue, but I've stopped using radiant player long time ago, therefore didn't send a report)

Huh. That's interesting. I can't make my mini player look like that at all. I can't even get it to show the "playing" image in the status bar icon.
Thanks for getting back to me though. I'll look into the album art issue.

Hey @nashbridges - thanks for the heads up about the song art. Turns out Google made another change and it broke it. Now fixed - should be included in the new release
Cheers! :+1:


job cannot use the same name, even from different user

we need to modify launcher path on hdfs and zk from /Launcher/FrameworkName to /Launcher/UserName/FrameworkName @yqwang-ms

By design we ONLY use FrameworkName to identify Framework, i.e. FrameworkName is the key/id for the Framework. It helps to achieve all APIs are IDEMPOTENT and STATELESS, thus Work Preserving Client Restart.
If we add user into the key:

We need to change a lot codes.
You still cannot avoid the same name within the same user.

So, suggest RestServer(i.e. Launcher's clients) to:

set the FrameworkName to be UserName_JobName.
Or, if you can ensure you are submitting a new job instead of updating a job or retry the submission, you can generate a unique/un-conflict FrameworkName for user.

I like the first option: username_jobname

We also need to restore the concatenated string to show the original JobName on webportal correctly, right?

Yes

Duplicate of #562


Statevector.draw('qsphere') does not properly color phase

Environment

Qiskit Terra version: 0.19.1
Python version: 3.8.10
Operating system: Win11x64 (running on IBM's JypyterLab using FireFox)

What is happening?
At certain circumstances, Statevector.draw('qsphere') does not color the arrow-ball according to the resulting phase.
However, at other circumstances, it does.
How can we reproduce the issue?
qc = QuantumCircuit(1)
qc.x(0)
qc.z(0)
Statevector.from_label('0').evolve(qc).draw('qsphere')

This wrongly produces a visualization of |1> since the color matches the phase of 0.
see:

What should happen?
The color should match the phase at Pi to match the state -|1>
Any suggestions?
Note that at other times, it's ok. For example:
qc = QuantumCircuit(1)
qc.h(0)
qc.z(0)
Statevector.from_label('0').evolve(qc).draw('qsphere')

produces:

The top image isn't wrong, per se, it's just fixed the global phase to 0 - it's showing the same state.  I'm not 100% sure why, but it was definitely a design choice: https://github.com/Qiskit/qiskit-terra/blob/5ea6e9557655b144228c29d7099375f5d2c91120/qiskit/visualization/state_visualization.py#L824-L827

Thanks for your reply and code verification Jake!
I think it's a weird choice (losing information...), especially when composer doesn't
(I guess I will add it to here).
In any case, how about adding a remove/keep global phase parameter with a default?
No harm done and more power gained.

Jake, I went snooping into the code thinking I will just add a parameter. Seems the issue lies deeper...
If you remark those 3 lines of code the global phase will still be removed!
Apparently, the "state" which arrives to this function is already stripped of phase.
Going deeper into the call trace, I find that in line 766 the state is first converted into a density matrix:
eigvals, eigvecs = linalg.eigh(rho.data)

which intrinsically wipes the global phase. Then in line 822 the state is being reconstructed (without the phase) back from the density matrix:
state = eigvecs[:, idx]

So, I actually have 3 questions:

How do you suggest modifying the code to effectively control the global phase elimination with a parameter?
What is the meaning of those 3 lines of code (remove the glboal phase) if states already arrive there stripped of global phase?
Line 820 decides whether "there's something" based on a crude hardcoded value (0.001), how valid is that? and shouldn't we globalize such a setting to all drawing functions?

Thanks!

Sorry I didn't reply for a while.

Adding a normalize_global_phase=True keyword argument would be a fine solution to this, though I think it might be slightly fiddly to do while keeping all the current behaviour.
The updates in #4324 made the plotting functions work for anything that Statevector or DensityMatrix will accept as inputs, which is a couple of things.  "Not fixing the phase" only makes sense for statevector-like inputs, so any change would need to make sure it could detect what things should be converted to statevectors and what to density matrices.

There are two points at which the global phase was removed, likely because the update in #4324 was just simplifying existing code that handled density matrices and statevectors separately, and the removal of global phase wasn't originally in the statevector path.  For input density matrices, fixing the gauge of the eigenvectors is still a necessary step there to avoid being dependent on the orthogonalisation routine (i.e. if x is an eigenvector, so is -x, and you can't necessarily predict which you'll get)

Yeah, it probably should be standardised and controllable, but overall it's probably best not to confuse the interface - the plotting is quite a coarse tool anyway, and eigenvalue of 0.001 means that state participates with a probability of 0.1%, which seems pretty acceptable as a cut-off for "not really important", since it's barely measurable with current hardware.


Online simulator with drift and delays

Note: This MR is an adaptation of #176 based on the suggestions in https://github.com/st-tech/zr-obp/pull/176#issuecomment-1192528581
Why
As an applied researcher on applications of bandits, I want to be able to compare various online and off-policy bandit algorithms.
About the change
The current implementation of the bandit simulations is based on a replay method with a synthetic bandits feedback log under the assumption of some logging policy. While training an online policy (eg. epsilon greedy), the policy is only updated when the taken action of the online policy is identical to the action taken by the (synthetic) logging policy. This make sense when we want to optimise the bandit policy using logged data, however, using simulation we can have access to much more flexibility.
For this reason we've made the following changes to simulation aspect of the framework

The method run_bandit_simulation was renamed to run_bandit_replay and is moved to a separate replay.py file. The method itself is left unchanged. None of the synthetic data generators have changed.
We introduce a new BanditEnvironmentSimulator, loosely based on the synthetic datasets. This class can generate a single BanditRound or a batch BanditsRounds dataclass. These rounds contain a context, action_context, expected_rewards, and round delays (optional).
We also introduce a new BanditPolicySimulator class. This class can take one or multiple steps using the BanditRound dataclasses created by the BanditEnvironmentSimulator. This policy simulator keeps track of taken actions, rewards and handles delayed feedback when relevant.
In addition to the regular simulators, we also introduce an ExponentialDelaySampler and CoefficientDrifter that can be used in combination with the BanditEnvironmentSimulator and BanditPolicySimulator to simulate delayed rewards and concept drift.
The CoefficientDrift changes the coefficients used in the different reward functions. For this reason, I've added a coefficient function as an optional argument to the different reward functions.
We show how to bridge the gab between the simulations and off-policy methods in OBP in one of the tests (test_ipw_can_be_learned_from_logged_data_generated_by_simulation).

Next steps

We have some notebooks demonstrating the different types of simulations, such as more in-depth bridges between the simulations and off-policy methods and how drift affects the on-policy models. These will be added later.
Do another pass through the code to add docstrings.
The base_reward_function still applies a z-score to the expected rewards. However, this breaks the drifting functionality of the CoefficientDrifter as the stddev and mean are taken over the full data sample. I either need to take the z-score out all together or make it optional. WDYT?

@Braamling Thank you, and the high-level implementation looks great! (though I haven't yet gone through every detail)

Would you be able to provide the notebooks? Thot would be very helpful
In terms of base_reward_function, could you make the z-score procedure optional rather than taking it out altogether?

@usaito I got stuck in some other priorities, let me get back to you somewhere next week. I'll provide the notebooks and make the z-score optional. Hopefully that will give you all the information needed to start reviewing the MR :)

@usaito I've added three notebooks and made the z-score optional. I would be very interested to hear what you think!

@usaito did you have the chance to take another look at this MR? Would love to hear if you think this fits OBP :)

@Braamling Sorry for not being able to respond, because I'm getting into the new semester. I've started reviewing the new content, and the notebook examples look great. Let me see the detailed implementation over the weekend. Thank you!

@Braamling  Thank you so much again for the contributions! I made some minor comments above. I'd also suggest renaming the notebook examples as follows.

delay-simulation.ipynb -> online-bandit-with-delay-simulation.ipynb
drift-simulation.ipynb -> online-bandit-with-drift-simulation.ipynb
online-vs-ope-simulation.ipynb -> online-bandit-vs-opl-simulation.ipynb

@Braamling Thank you so much for the reminder, and I'm happy to merge, but it seems that some tests are failing, so it would be great if you could deal with these issues. Thank you!

@usaito, I should really turn on notifications for this thread 😆
The tests were broken due to some issues with seeds. I've made the tests more robust against these changes, and locally they have passed. I do think that you need to activate the tests on Github if I'm not mistaken.
Let's hope everything is ✅ now!

Forgot to run black, added another commit.

@Braamling Looks great! Thank you so much again for this and also your contribution to our workshop at RecSys!


Extend test coverage:

Test all validations of Campaign model
Test Campaign::to_csv
Further test confirming a match
Test that a user can update and delete their information
Test Vaccine model
Test robots.txt
Omit app/channels/ from coverage report

Thanks a lot for these tests @rthbound This is awesome 🙌


Some pointers and help here and there

Some clarifications to help users through.

@JJ : Thanks for your contribution to the Azure documentation! The author, @seanmck, has been notified to review your proposed change.

Thanks for the feedback. I'll do that in a moment.

@JJ Hi Juan, are you ready for a second review, or are you still working on edits? Thanks. -MM

Ready and able :-)

@JJ : Thanks for your contribution to the Azure documentation! The author, @seanmck, has been notified to review your proposed change.

Looks good. Thanks again, @JJ!
#sign-off
Cc: @seanmck

@mmacy: Only the author, @seanmck, can sign off on this pull request.

Thanks all!


Support Windows

This set of patches enables building and passing the test suite on Windows.  Primarily, treat Windows and Linux similarly.  The platform specific behaviour here is the construction of the input paths where we now use URL(fileURLWithPath:) and droppingLastPathComponent and addingPathComponent to compute the location of the test input.

🙄 of course the failure would be related to line endings!
Here's one of the failures, but the other is similar:
D:\a\SWXMLHash\SWXMLHash\Tests\SWXMLHashTests\WhiteSpaceParsingTests.swift:55: error: WhiteSpaceParsingTests.testShouldBeAbleToCorrectlyParseCDATASectionsWithWhitespace : XCTAssertEqual failed:
("Optional("\n        \r\n  this\r\n  has\r\n  white\r\n  space\r\n        \n    ")") is not equal to
("Optional("\n        \n  this\n  has\n  white\n  space\n        \n    ")")

That should be easy enough to add a platform check around them, though. I can add it to the PR later or you're welcome to if you get to it first.
Thanks for the contribution @compnerd! It looks great!

I think that the nicer way to handle this is via .gitattributes to ensure that the files are checked out in unix line endings rather than native.  That should give us the same behaviour on all the platforms.

Good call @compnerd and thanks again!

My pleasure!  One favour to ask - would it be possible to get a tagged release with that by any chance?

Absolutely, I just published 7.0.2 out.


Fork choice rule

Added the LMD GHOST fork choice rule.

"Set finalized_head to the finalized block with the highest slot number." => Can I confirm that finalized_head can be a placeholder block if the slot is empty?
"Set justified_head be the descendant of finalized_head that is justified" => What if no such descendant exists?
Are finalized_head and justified_head relative to both onchain and offchain attestations, or only onchain votes?
"store contains all attestations and blocks that the validator has received" => Can I confirm that store contains offchain attestestations?
"Let get_most_recent_attestation_target(store, v) be a function that returns the block that a validator attested to in the attestation in store that has the highest slot number" => Can I confirm that get_most_recent_attestation_target must first filter out blocks that are not connected (via a path of ancestors) to justified_head?
How is get_most_recent_attestation_target defined if a validator made more than one attestation at the same slot?

Just made an update to address these. For (1), the head should be an actual block (I don't think we use the "placeholder block" terminology in the spec....), and I think it's already the case that the block hash at slot N in a chain is the block hash of the most recent block in the chain with slot <= N.

Thanks :)

Can you check my edits make sense?
"it's already the case that the block hash at slot N in a chain is the block hash of the most recent block in the chain with slot <= N" => My intuition tells me it's not the "right" definition, i.e. that things progress (e.g. slot count increases, inactivity penalty is processed, ...) even if there are no "real" blocks.
Should we define the edge case where get_most_recent_attestation(store, validator) should return None?

"it's already the case that the block hash at slot N in a chain is the block hash of the most recent block in the chain with slot <= N" => My intuition tells me it's not the "right" definition, i.e. that things should progress (e.g. slot number increases, inactivity penalty is processed, ...) even if there are no "real" blocks, and block hashes should reflect that progress.

If we want to make things work this way, we can, though it would entail some deep changes to the state transition function. Essentially, the top-level block processing function would say "if block.slot - block.parent.slot == N, then run the underlying block processing function N-1 using a standardized empty block, and then once using block.

How does the fork choice rule work in the edge case where there are more than one justified or finalised block?

Higher slot wins. If more than one with the same slot number (ie. an attack happened), then I would say pick the one that the client learned about earlier.

"there are implementations that compute the head in logarithmic time" => logarithmic in what?

Logarithmic in slot count. Linear in the number of distinct objects that validators voted for in their most recent votes (ie. usually ~64).


App crashes when the react-native-vimeo-iframe leave the viewport

I have this strange problem on android, i-e if I have a scroller in place and when I try to scroll to the react-native-vimeo-iframe and go more below until react-native-vimeo-iframe is out of view port, the app crashes.
PS: It's working fine on iOS.

Hi @husnaintahir, are you wrapping the Vimeo component in a ScrollView?
If yes, could you try by putting overScrollMode='never' as explained here?

Still happens to me when adding overScrollMode, but adding another prop works for me. According to this source: https://github.com/meliorence/react-native-render-html/issues/605#issuecomment-1363638140
I have <Vimeo height={height} videoId={id} overScrollMode="never" androidLayerType="software" />
tried both software and hardward, both works and app no crash. not sure what the difference is, but if it works, then i'm good with either.


WIP DBCP-509: Consistency across methods in PerUserPooldataSource and its parent

For discussion for DBCP-509. A proposal to use primitives and use the default values instead of nulls, as in the other methods.

If this change is approved, then I'd update the test code from DBCP-504 later :+1:

-1 for 2.x as this patch breaks BC :-( See 'mvn clirr:check'.

@kinow May you please rebase on master? I'd like to see if the default goal in the POM properly causes the build to fail if JApiCmp fails.

@kinow
Closing since it breaks BC and there is no further activity.

No worries @garydgregory . Had completely forgotten about it, but no bandwidth to work on this issue now. We can revisit it later on :+1: Thanks!


Add shwap data types

Hey @fl0rek.
We had to make some changes to shwap, which landed in this commit. The TLDR is described in the commit-msg.


Replace deprecated color-adjust with print-color-adjust

https://developer.mozilla.org/en-US/docs/Web/CSS/print-color-adjust
This fixes issues with autoprefixer ^10.4.6
(https://github.com/postcss/autoprefixer/releases/tag/10.4.6)
Which throw warnings for use of color-adjust CSS
Addresses https://github.com/twbs/bootstrap/issues/36259
Replaced in a few files more than https://github.com/twbs/bootstrap/pull/36258
(Probably necessary to swap in the min files)

There's no need to update the dist files, that happens on release. Can you revert those changes?

@mdo done

By now this is a duplicate of #36258 so it can be closed IMO.

By now this is a duplicate of #36258 so it can be closed IMO.
(Not sure if a PR for 4.x is necessary or if a backport is possible though.)

They're separate, so we'll leave both PRs.

@mdo: I'm not sure we can apply this in v4 due to our supported browsers.

@XhmikosR
If this is the case and you want to leave it in, you should use an older version of autoprefixer, otherwise the warning will be treated as an error in production

It's not an error, it's a warning. And we can't control the tools the consumers of the package use.

@XhmikosR , I guess that its not specifically an error, but because its somewhat common to enable "treat warnings as errors" it does affect a lot of automatic deployments. It's true that you can't control what tools other users use, but you can control which tools that you use. So if you want to update the prefixer in v5, but not in v4, that's another way to get around these warnings.
https://caniuse.com/?search=print-color-adjust
Currently the only web browsers which really used only color-adjust appear to have been earlier builds of firefox for which 48-97, during which print-color-adjust enjoyed partial support. Other browsers do not support color-adjust or require the webkit prefix.
So maintaining this to support old versions of firefox eschews support of that functionality in Edge, Chrome, Safari etc

It's not an error, it's a warning. And we can't control the tools the consumers of the package use.

I am using bootstrap 4 via the ruby gem, which used to be a supported/recommended path to using bootstrap 4. The bootstrap 4 ruby gem has a dependency on autoprefixer-rails.  So autoprefixer is not a tool I chose to use, it's in fact one bootstrap did control.
While my current setup doesn't cause any builds to fail, I now get this autoprefixer warning in all my build output. Which is a bit annoying.
I don't think I have a way to remove autoprefixer from my app, as it's a bootstrap dependency; I guess I could lock to an older version that doesn't produce this warning.
If anyone has any other ideas for eliminating this warning from my use of bootstrap 4 via ruby gem, I'd be interested in hearing them.

I have the same issue as @jrochkind and would be glad for a way to suppress this warning

@XhmikosR thanks for merging this fix, any plans to release a patch version for Bootstrap 4?

Yeah, we will release a new version soon, maybe this week.
On Wed, Jun 8, 2022, 19:19 Rodrigo Arias @.***> wrote:

@XhmikosR https://github.com/XhmikosR thanks for merging this fix, any
plans to release a patch version for Bootstrap 4?
—
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/pull/36283#issuecomment-1150126891,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACVLNJFLYM7HOJGNSPZ5ZDVODBYRANCNFSM5VF7CGRA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I had the same problem. However, I tried to install bootstrap<EMAIL_ADDRESS>but then I had a problem with the localhost, which doesn't open the project even with not presenting errors in compiling. Now, I uninstalled bootstrap and tried to reinstall it, but still have the same autoprefixer error.
I tried also overriding autoprefixer, but still get the 6:29521 error. Does someone knows a way to get around it?

@XhmikosR any update on when this will be released? Thanks for your work on it.

This was also fixed in Bootstrap v4.6.2

@jrochkind IMO you can create an issue in https://github.com/twbs/bootstrap-rubygem


Errata in natural logarithm Taylor expansion

good catch!
please create a feature branch that only has this commit (and not the two merge commits) and do not merge the feature branch into your master before doing the PR
If you do not know how to do this, I can also do another PR and use git am on your commit which will preserve it, including your name.

I'm not sure how to do that. I'm using the github web interface. The fork button always takes me to my old fork where I've updated the changes from iohk. Seems like it doesn't have the possibility of making a separate branch. Would it work if I delete my fork and create a new one? Or must I use the git command line necessarily?
If it's easy for you to resolve the PR it's OK.

Would it work if I delete my fork and create a new one? Or must I use the git command line necessarily?

to be honest, I don't know. I haven't used the github interface directly myself. I created #418 which includes your commit verbatim, with you as the author. So we can just merge that one and close this here if you agree.

Would it work if I delete my fork and create a new one? Or must I use the git command line necessarily?

to be honest, I don't know. I haven't used the github interface directly myself. I created #418 which includes your commit verbatim, with you as the author. So we can just merge that one and close this here if you agree.

Perfect. Thank you!

@csoroz thanks for spotting and fixing this!


Raycast2D not casting to the right position when casting the ray in code

Godot version:

3.2.3
OS/device including version:

Windows 10
Issue description:

Casting a raycast2D to a position using code doesn't cast the ray to the right position.
Like this example where i cast the ray onto the mouse's global position but the ray doesn't cast to the correct position:

Steps to reproduce:
In code, just cast a ray into the position of any object and the ray won't cast to the exact location of the object.
Minimal reproduction project:

RaycastIssue.zip

This is due to mouse coordinates, not the raycast preview.
Duplicate of https://github.com/godotengine/godot/issues/34805 and possibly others.


Some queries are really just testing parquet

I was looking at some of the H2O queries and, the way that they're set up, they're like 90% just testing the performance of read_parquet.  It seems like we should either remove parquet from what is measured (by a persist and wait for example) or stop caring about these queries, and maybe just care about one of them.
Alternatively, was this known/intentional?

It's known.
As-is they show end-to-end performance from the POV of the user.  This surfaces things like the impact of  implementing column projection, string dtypes, and https://github.com/dask/dask/issues/9619.

In an ideal world, we could record and analyze both the end-to-end and io-free run times seperately. However, the persist and wait approach is not always possible for large-scale datasets.

It is possible here though right?  These datasets are small.  I'm totally fine if we want to do an end-to-end workflow, but currently it looks like we have several end-to-end workflows that are all testing 95% the same thing.  Maybe this is just an artifact of H2O and we're fine doing a weird job here because we're trying to match some existing standard.  I do want to make sure that we're on the same page though that lots of these queries are super-correlated.  If we're not aware of that then I recommend that we spend more time looking at profiles rather than just taking a birds-eye view.

Yes.  Given that all the queries now run reliably now on a 50GB dataset, I'm in favor of updating the test suite, so we could include this information as well  For the sake of completeness, the following table show the fraction of time spent in _read_table_from_path for each query on both a 5GB and 50GB dataset.

FS
Dataset Size (GiB)

1
2
3
4
5
6
7
8
9

0

s3fs
5

92
74
49
67
51
46
44
4
37

50

92
84
50
73
54
54
50
10
52

This table shows the total Wall Clock time (seconds) for each query:

FS
Dataset Size (GiB)

1
2
3
4
5
6
7
8
9

0

s3fs
5

4.7
3.4
26
11
16
16
23
155
23

50

36
41
622
199
357
245
589
1655
311


Placeholder overflows in IE and Firefox

When the placeholder is long it overflows in all browsers except chorme and look like the following image

This is caused because the corresponding style that calculates the max width has a vendor prefix. I guess is safe enough to just remove the webkit prefix.

Thanks for pointing this, will publish a release to npm with some other fixes soon.

fixed in 0.6.0

Thanks a lot!


Installation problem with the boost_python lib.

After I finished the Installation instruction on Ubuntu 16.04, I tried to test the tool working state:
./rs --help
the tool returned the Import Error:
Traceback (most recent call last):
File "/home/ql/anaconda3/envs/pytorch/lib/python3.6/runpy.py", line 193, in _run_module_as_main     "__main__", mod_spec)
File "/home/ql/anaconda3/envs/pytorch/lib/python3.6/runpy.py", line 85, in _run_code     exec(code, run_globals)
File "/home/ql/GitHubRepo/robosat/robosat/tools/__main__.py", line 5, in <module>     from robosat.tools import (
File "/home/ql/GitHubRepo/robosat/robosat/tools/extract.py", line 3, in <module>     from robosat.osm.parking import ParkingHandler
File "/home/ql/GitHubRepo/robosat/robosat/osm/parking.py", line 3, in <module>     import osmium
File "/home/ql/anaconda3/envs/pytorch/lib/python3.6/site-packages/osmium/__init__.py", line 1, in <module>     from osmium._osmium import *
ImportError: /usr/lib/x86_64-linux-gnu/libboost_python-py27.so.1.58.0: undefined symbol: PyClass_Type
I thought there might be something wrong with the Boost_Python lib, how could I figure it out?

You are running into https://github.com/osmcode/pyosmium/issues/52. The ticket has a workaround at the bottom. You have Python3.6 installed (probably through a ppa) but the Ubuntu 16.04 system comes with Boost.Python for Python3.5.

Thanks for your quick reply. I used conda to create a new Environment with python=3.5. After I replayed all the installation operations, Robosat works fine. Sorry for the noise. Feel free to close.

:+1: hopefully at some point we will just have self-contained Docker images.
Have fun with :robot: :framed_picture:!


Add support for neovim editor

dnote.io client seems to ignore neovim as an editor (EDITOR="nvim") and instead runs vi which is a default option. In my case this leads to errors as I have some settings in .vimrc file which are not recognized by vi.
I have created a minimal untested PR - I don't code in golang and after few failed attempts to run the code locally I gave up :-) but it seems like you don't to anything specific with editors so this should be enough.
The other, better solution would be probably using "vi" as default editor value only of EDITOR environment variable is unset and in all other unrecognized editors - and also all other editors which don't need any special flags - you would simply keep using the actual EDITOR value.
I am happy to change the PR to your liking.

in all other unrecognized editors - and also all other editors which don't need any special flags - you would simply keep using the actual EDITOR value.

I think, that way, we can definitely support many editors without having to make special cases for each of them. But a problem is that we cannot guarantee that Dnote will work for all of them as some editors require flags to make command wait until program is exited. Therefore I think whitelisting editors (as we are doing now) is a cleaner solution.


[rabbitmq] Update 3.9.11

https://github.com/rabbitmq/rabbitmq-server/releases/tag/v3.9.11

LGTM thx


Replace reduce in labels with flatMap and merged lines

Changes proposed
The labels unique section in the get-issues/+server.ts caught my attention and optimized it with flatMap function. Also merged the uniqueLabels and made inline with returnBody.
Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

This is already done in https://github.com/EddieHubCommunity/good-first-issue-finder/pull/264/


fix(core): allow wry to be an optional dep again (fix #1841)

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[x] No

The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
crate::manager::default_args! {
  /// A menu event that was triggered on a window.
  #[cfg(feature = "menu")]
  #[cfg_attr(doc_cfg, doc(cfg(feature = "menu")))]
  pub struct WindowMenuEvent<P: Params> {
    pub(crate) menu_item_id: P::MenuId,
    pub(crate) window: Window<P>,
  }
}

turns into
#[doc = " A menu event that was triggered on a window."]
#[cfg(feature = "menu")]
#[cfg_attr(doc_cfg, doc(cfg(feature = "menu")))]
#[cfg(feature = "wry")]
pub struct WindowMenuEvent<P: Params = crate::manager::DefaultArgs> {
    pub(crate) menu_item_id: P::MenuId,
    pub(crate) window: Window<P>,
}
#[doc = " A menu event that was triggered on a window."]
#[cfg(feature = "menu")]
#[cfg_attr(doc_cfg, doc(cfg(feature = "menu")))]
#[cfg(not(feature = "wry"))]
pub struct WindowMenuEvent<P: Params> {
    pub(crate) menu_item_id: P::MenuId,
    pub(crate) window: Window<P>,
}

thanks for the fix :heart:


Breaking change in registration of inherited providers

Laravel Version
11.9
PHP Version
8.2.13
Database Driver & Version
No response
Description
Before v11.9, the registration of services paid attention to instance of service (using instanceof) and avoided the registration if "similar" class was already registered. With v11.9 it matches exactly using  fully qualified name of the class.
php:instanceof - instanceof can also be used to determine whether a variable is an instantiated object of a class that inherits from a parent class.
Example:
let define CustomMigrationServiceProvider extends Illuminate\Database\MigrationServiceProvider.
and register the provider.
The order of registration during booting of an application is exactly the same for all 11 laravel versions:

...
CustomMigrationServiceProvider
...
Illuminate\Database\MigrationServiceProvider

but

before 11.9 the application had detected, that previously registered CustomMigrationServiceProvider is instance of Illuminate\Database\MigrationServiceProvider and the registration of Illuminate\Database\MigrationServiceProvider was skipped
** with 11.9+** the application detect only that the class (Illuminate\Database\MigrationServiceProvider) was still not registered and follows with the registration (it means, the migrator will be overwritten)

Reason for the breaking change:
https://github.com/laravel/framework/compare/ceb892..2fd3a25#diff-0a80606049a107748776bea5151a61fbec2afefe3361c340a5de9c8c1ccdf9c8

before v11.9
** \Illuminate\Foundation\Application::markAsRegistered  pushed the registered service onto the end of list
** \Illuminate\Foundation\Application::getProvider used getProviders (iterates over the list using instanceof)
with v11.9
** \Illuminate\Foundation\Application::markAsRegistered  set the registered service in the list using fully qualified name (key => value)
** \Illuminate\Foundation\Application::getProvider access the registered service directly using fully qualified name

Steps To Reproduce
<?php

declare(strict_types=1);

namespace Tests\Components;

use Illuminate\Contracts\Console\Kernel;
use Illuminate\Foundation\Application;
use Illuminate\Foundation\Testing\TestCase;
use Illuminate\Support\ServiceProvider;

interface ServiceInterface
{
    public function run(): string;
}

class ServiceA implements ServiceInterface
{
    public function run(): string
    {
        return "A";
    }
}

class ServiceB extends ServiceA
{
    public function run(): string
    {
        return "B";
    }
}

class AServiceProvider extends ServiceProvider
{
    public function register(): void
    {
        $this->app->singleton(ServiceInterface::class, ServiceA::class);
    }
}

class BServiceProvider extends AServiceProvider
{
    public function register(): void
    {
        $this->app->singleton(ServiceInterface::class, ServiceB::class);
    }
}

class ServiceProviderRegistrationTest extends TestCase
{
    /**
     * Creates the application.
     *
     * @return Application
     */
    public function createApplication(): Application
    {
        $app = require __DIR__ . '/../../bootstrap/app.php';

        $app->make(Kernel::class)->bootstrap();

        return $app;
    }

    public function testBreakingChange(): void
    {
        $this->app->register(BServiceProvider::class);
        $this->app->register(AServiceProvider::class);

        $service = $this->app->get(ServiceInterface::class);

        // laravel 11.8: the first service wins
        // laravel 11.9: the last service wins
        $this->assertEquals("B", $service->run()); // works with v11.8
    }
}

Hey there, thanks for reporting this issue.
We'll need more info and/or code to debug this further. Can you please create a repository with the command below, commit the code that reproduces the issue as one separate commit on the main/master branch and share the repository here?
Please make sure that you have the latest version of the Laravel installer in order to run this command. Please also make sure you have both Git & the GitHub CLI tool properly set up.
laravel new bug-report --github="--public"

Do not amend and create a separate commit with your custom changes. After you've posted the repository, we'll try to reproduce the issue.
Thanks!

I don't believe this is the correct way and is never meant to be supported, service providers should be registered via app.providers.

Hi Crynobone,
you are probably right, that it was never meant to be supported, but ... it has worked until 11.9 ;)
Affected application uses separate custom package and registered "CommonServiceProvider" (using app.providers).
The common-provider registers few other logically splitted ServiceProviders. Of course, it's possible to register each provider separately (is probably the best solution) - I reported the issue, because the public method Application::register no longer works exactly as before, and maybe it's relevant for other people/causes other bugs.
In case you want to check it, I've prepared the repository, that helps to reproduce it:
https://github.com/vnrmc/laravel-bug-report-51739
Thx!

The correct way to solve this is actually:

Remove App\Providers\CommonServiceProvider class and registration from bootstrap/providers.php:

<?php

return [
    App\Providers\AppServiceProvider::class,
-   App\Providers\CommonServiceProvider::class,
];

Update config/app.php and add the following configuration:

    'providers' => Illuminate\Support\ServiceProvider::defaultProviders()
        ->replace([
            'Illuminate\Database\MigrationServiceProvider' => 'App\Providers\CustomMigrationServiceProvider',
        ])
        ->toArray(),


Make Dask execution asynchronous

Currently, the Dask execution waits for all the endpoints to complete. I suspect when we build large dataset test cases (https://github.com/ethyca/fidesops/issues/39) we'll find that synchronous execution to be a significant limitation in query execution times. Fidesops should default to asynchronous task execution, and provide this as a configuration option.

is it Dask runner that's async or api?
api is ready to go, run dask distributed in background worker
instead of calling run directly, get dask to kick it off

Based on today's discussion, here's a few requirements:

fidesops should execute request tasks (database queries, uploads, etc.) on a dask worker, not the main webserver (in other words: use dask.distributed)
add some dask workers to the docker-compose
add the ability to start dask workers to the fidesops CLI (see https://github.com/ethyca/fidesops/blob/main/src/fidesops/cli.py where I'd imagine we just add fidesops start_worker as a command)
update the deployment guide accordingly


Describe the need
Workflow Cancel is not working from the API despite returning an ACCEPT
Apparently you can force cancel it but we appear to be missing the ability to cancel in the Oktokit library
API Version is: 2022-11-28
https://docs.github.com/rest/actions/workflow-runs#force-cancel-a-workflow-run
  -X POST \
  -H "Accept: application/vnd.github+json" \
  -H "Authorization: Bearer <YOUR-TOKEN>" \
  -H "X-GitHub-Api-Version: 2022-11-28" \
  https://api.github.com/repos/OWNER/REPO/actions/runs/RUN_ID/force-cancel

See:
https://github.com/actions/runner/issues/1846
https://stackoverflow.com/questions/77799456/workflow-stuck-and-not-cancellable
SDK Version
octokit/dotnet-sdk v13.0.1
API Version
2022-11-28
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Appears to be in this branch: https://github.com/octokit/dotnet-sdk/blob/668e0f0fca8a323a2ad95863c2cdd72dc118442d/src/GitHub/Repos/Item/Item/Actions/Runs/Item/ForceCancel/ForceCancelRequestBuilder.cs#L13

Hey @mikebollandajw Thanks for reaching out.
There might be a little confusion around which SDK you're using that I'd like to clear up.  This issue was created in the dotnet-sdk repository which is for our generated SDK - and the branch you reference has been shipped as part of the change sets.  The generated SDK dotnet-sdk, is currently in pre-release and is on v.0.0.23
The version number that you are referencing actually belongs to Octokit.net - that is our older, manually written SDK - v13.0.1 being the latest version there.
Please consider using the newer SDK; as you have discovered our generative SDK (dotnet-sdk - available on nuget) is updated with the latest from the GitHub REST API nightly.
Otherwise, we can transfer this issue to the classic Octokit.net repo and label it as "Up for Grabs" so that you or the community can pick it up and add the APIs to the older SDK.


SPFX: Configure web service urls per environment.

If a webpart is calling an external web service that has different urls per environment. Is there any standarard way to build the package for

Dev vs
QA

So when deployed to Dev the webpart will call dev endpoint and so on.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 819a6029-5ccb-0a7b-8c59-a1ebeffca4cb
Version Independent ID: 61b64d96-30a0-93d8-a0fd-6523e2dd5418
Content: SharePoint Framework toolchain
Content Source: docs/spfx/toolchain/sharepoint-framework-toolchain.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

No... there are lots of ways to deal with this. It depends on what you're comfortable with. Personally, I'd use a tenant scoped property.

Thank you @andrewconnell
I went ahead with creating .env files.
.env.production, .env.development ect.
And while building the package I used cross-env to pass env value
"bundle:production": "cross-env NODE_ENV=PRODUCTION gulp bundle --ship",
"bundle:development": "cross-env NODE_ENV=DEVELOPMENT gulp bundle --ship"
And finally in gulp file I did something like this
if (pluginDefine) {       pluginDefine.definitions = {         ...pluginDefine.definitions,         ...currentEnvValues,       };     } else {       cfg.plugins.push(new webpack.DefinePlugin(currentEnvValues));     }
Works really fine for me.

It would be really nice to have something like this built in though :blush:


Corrected Using A Library example HTML syntax.

Fixes #528
Changes:
Incorrect example HTML syntax was corrected.
Screenshots of the change:

thanks!


Does the actionButton have some playHideAnimationCompleationListner?

Thanks for grate lib! :)
Wanted to hide the actionButton using animation when user press backpress but the super.onBackPressed(); in parent Activity  don't wait for actionButton.playHideAnimation(); to finish.
I posted a Q at: to get a workaround
http://stackoverflow.com/questions/42878553/how-to-let-fragment-do-some-action-before-onbackpressed-kill-it

ok I solved thanks for your attention..


Custom status clears after a few days

I have written a bot that uses a custom status, and after a few days, the custom status that is set disappears. Is there a workaround for this?

You could just set the custom status again every 3 days or so. Just a workaround.


[perf] Migrate @azure-tests/perf-ai-metrics-advisor to ESM

Packages impacted by this PR

@azure-tests/perf-ai-metrics-advisor

Issues associated with this PR

https://github.com/Azure/azure-sdk-for-js/issues/31338

Describe the problem that is addressed by this PR
Migrates @azure-tests/perf-ai-metrics-advisor  to ESM via automation.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

/azp run js - pullrequest

/azp run js - pullrequest


Checklist for 5.0.0

Implementations

[x] Add Supermassive Special Power.
[ ] Add Excalibur Special Power.
[ ] Add Sigma Strike Special power that implements WeaponCriticalEdgeIncreaseStackable.
[x] Add Power leveling OP ability that increases exp gain using AddPlayerLeaveCombatTrigger.
Add Celestial Realm and Shadow Realm as OP abilities (require level 15).
[x] Update TabletopTweaks-Core requirement to 0.5.15.
[x] Add some wuxia/murim-like backgrounds (#114)

[x] Added Demonic Cultivator Isekai background.
[x] Added Enlightened Sage Isekai background.
[x] Added Musician Isekai background.
[x] Added Rationalist Isekai background.

[x] Class Reworks

#108
#123

[x] Rework Aura of Righteous Wrath.

[x] Add variants
[x] Remove effect on additional limbs.

[x] Change Killing Intent into a toggle ability (#101)
[x] Nerf Exceptional Weapon feats to no longer work with additional limbs (remember to update description)
[x] Nerf Beta Tester initiative bonus from 10 -> 4.
[x] Nerfed Magical Amplification spell dice from d12 -> d10.
[x] Buffed Gamer background from +4 to +8
[x] Fix Reflect to ignore attacks from owner (#125)
[x] Remove True Smite and True Mark from paladin legacy. (#107)
[x] Remove redundant Kinetic Power Special Power (#107)
[x] Remove redundant OverpoweredAbilitySelection2 and just use OverpoweredAbilitySelection instead.
[x] Remove alignment restrictions for Isekai Channel Positive Energy and Channel Negative Energy.
[x] Update icon for Exceptional Feats.
[x] add crossbreed human to human heritages
[x] #122
[x] #128
[x] Bugfix, selection array initilization logic seems to differ either due to other mods, game version, or just because; making the use of the Features reference proxy array questionable, so instead the underlying m_AllFeatures array needs to be used everywhere #129

Pre-Release Tasks

[ ] Update LocalizationPack.json
[ ] Update CHANGELOG.md
[ ] Update README.md
[x] Update Info.json
[x] Update Repository.json

Post-Release Tasks

Update mod description
Update mod version
New screenshots
Release description

Yeah I guess I could add an evil version. I was thinking about what you said about adding prerequisites and this one seemed to make sense.

Chaotic and Lawful would also make sense as options since those are the 4 choices smite usually goes by.
Or we are simply less strict and say Righteous wrath is centered at lawful good and will empower anyone within 2 steps (so neutral good, chaotic good, Lawful neutral, True Neutral, Lawful Evil) While the counterpart is centered on chaotic evil with a similiar restriction.
Oh and obviously having one would block you from picking the other to prevent any true neutral cheaters from gaming the system...

Splitting it along the smite options might be a really good idea because that way each aura could also have its own damagetype.
Good would be holy, evil negative, Lawful Fire, Chaotic Air, or something like that...

I was thinking about it, but somehow legend unlocks you to level 40, would it be possible to do the same with IP and make it a class with actually 40 levels of progression. (if not possible then maybe a prestige class that you can pick after reaching 20 lvl with protagonist?

It shouldn't be too hard to add a prestige class, but I wonder what we should call it?
Something like Isekai Protagonist+ would be so uncreative.
That makes me think about an off-topic question, how do most isekais end anyway? Like all the Isekais I've seen are still on-going.

That's the problem with Light Novel type of writing, they drag on for so long that usually authors stop before they reach final conclusion, or the story mostly ends with protagonist becoming existence equal to god.

As for Prestige class for IP, maybe Awakened Protagonist.

since Nerin mentioned the idea yesterday I had some thought on mechanics for the Prestige class..
Easy progression choice, continuing to grant an extra basic feat at every even level.
Grant an additional  overwhelming feat every 4-5 levels.
Grant an "Impossible Feature" at every odd level, this feature would allow the player to select or reselect a class feature of the IP and its archetypes. For example since you need to pick what to buff in beach episode you could make a second choice using this, or perhaps you want some of the additional attacks restricted to Edge Lord as a Hero, or want your God Emperor to use Charisma for everything the way the Hero does?
With every archetype getting a choice of legacy I was thinking of debuffing the Dual Option a bit by not just restricting it to be available only for level 5+ or perhaps even 10+ but also adjusting the dual progression that way as well.
So your second choice of legacy would only get 15 (or 10) levels amongst the original 20 and it would be only by going legend and picking this prestige class that you could get the final levels.
For non legends this would also make it a lot more important of a choice what you pick for first rather than second legacy.

See my statement above on keeping neutral characters from gaming the system.
Depending on what options we add we have different options on who can pick it and who it can affect while remaining fair to everyone.
If we go by the 4 smite options then centering them around the neutral option of their axis and allowing only 1 step away from it would be balanced.
With just two options I think we always leave a hole where a specific build feels that they are limited in some way, because the lawful evil character might consider lawful to be more important than evil, so why can he buff the chaotic neutral rogue but not the lawful good paladin despite them being closer?

Actually I think that means we need to keep the buff effect the same and delcare it non stackable, that way even if you are under the effect of two auras only the first counts...

It shouldn't be too hard to add a prestige class, but I wonder what we should call it? Something like Isekai Protagonist+ would be so uncreative.
That makes me think about an off-topic question, how do most isekais end anyway? Like all the Isekais I've seen are still on-going.

most end by defeating the villain and either going home or staying in this world to live happily ever after.

See my statement above on keeping neutral characters from gaming the system. Depending on what options we add we have different options on who can pick it and who it can affect while remaining fair to everyone. If we go by the 4 smite options then centering them around the neutral option of their axis and allowing only 1 step away from it would be balanced. With just two options I think we always leave a hole where a specific build feels that they are limited in some way, because the lawful evil character might consider lawful to be more important than evil, so why can he buff the chaotic neutral rogue but not the lawful good paladin despite them being closer?

yeah, something like Divine Anathema should be most balanced, but that would also be rather overpowered for neutral characters won't it?

Hello! Is it possible to remove alignment restriction on the Villain archetype or add a feat/backgroud/OP Ability that allows taking this class with good aligment (name like Heel–Face Turn)? I really want to play it, but standard ToyBox option doesn't work at all.

the current cillain is being split into two archetypes in this patch, overlord and mastermind with mastermind not being alignment restricted, so depending on what your reason for wanting to pick the current villain is that might already do it for you

the current cillain is being split into two archetypes in this patch, overlord and mastermind with mastermind not being alignment restricted, so depending on what your reason for wanting to pick the current villain is that might already do it for you

well, i really like double Overpowered Ability aka Villainous Overpowered Ability

If you're mostly picking the auto-metamagics the new Mastermind subclass coming in 5.0.0 should be suitable.
Otherwise you need to be evil and choose the new Overlord subclass (also coming in 5.0.0).
You can see the changes so far here. Not all changes are listed yet.

#123
#108

Still work in progress

If you know some C#, you can even build your own version.

If you're mostly picking the auto-metamagics the new Mastermind subclass coming in 5.0.0 should be suitable.
Actually, it's almost perfect. If there was a way to add "Mythic Spells", it would really greant. In fact, this is why I want double OA - collect all spells for ultimate spellbook :)

If you know some C#, you can even build your own version.
I know a little bit of C# and understand what needs to be edited specifically (I even found out what exactly needs to be changed), but I'm not sure how to correctly compile the code into a mod-zip file.

you don't, the zip file is a zipped version of the mod folder that is installed under D:\Steam\steamapps\common\Pathfinder Second Adventure\Mods or whatever your game Mod folder is.
If you compile the code locally it will be installed directly to that folder with no need to zip it first.

Well you can turn off the automatic installation because that is done by:
<PropertyGroup>
    <PostBuildEvent>(robocopy "$(TargetDir) " "%25WrathPath%25\Mods\IsekaiMod\ " /s /e )^& IF %25ERRORLEVEL%25 LEQ 1 exit 0</PostBuildEvent>
  </PropertyGroup>

as a compiler directive and then it will only be build to a build directory in the project from where you could copy it manually or zip it if you feel that you need to involve the modfinder tool by dropping it in

Also, as far as I can see there are 4 overpowered feat selections which would allow you to pick a mythic spellbook to be added, assuming that option hasn't also been added to the metamagic feats given its magical nature...

I believe someone still wanted the wisdom based version of kineticist for the god emperor, besides that feedback for names and descriptions for this version was fairly low anyway :(
I think about the only thing really left to do in this version is updating the availability chart for the new sub classes...

I believe someone still wanted the wisdom based version of kineticist for the god emperor

Guess that was me, was looking for a godling throwing lighting everywhere like a mini Zeus ^^
But if it cannot be done then...

@Mythalar
It can be done, it is actually a lot less work than most of the asked for customisation ideas so far because all it needs is the implementation of a single newly made burn feature that overrules the main stat to Wisdom and copies pretty much everything else from an existing burn feature.
The biggest problem with it is that all affected blasts have to be defined in the feature, which can lead to problems with expanded elements depending on when in the patchorder it is done.
The description is actually the bigger problem as far as I am concerned because the text of both the Custom Burn feature and the archetype should then make you sound appropriatly divine and any idea I come up with is zeus (or similiar) based and let us be honest he is a bad example for wisdom and makes a very good case for why charisma should have been the divine stat instead...

(Crossed posts with the legacies thread I guess)
@kjk001
Okay I thought you even could more or less copy/paste a class or subclass so you would not have to manually assign/modify too many things.
On the description side I partially disagree with you.
You are right that Zeus and quite a bit of similar gods are not paragon of wisdom (greeks and latin gods were made similar to mankind after all :p ) but the Psycho wisdom mechanic does not dwells into this.
The actual description and rules books emphasize this subclass as an indivudual that contains the power flow of the elements within his mind instead of within his physical body, hence the constant straining and he's then exposed to emotional outbursts.
So it's not sage wisdom, but more of concentration/discipline, whereas charisma is all innate.
In my mind (...) it does make sense as a (divine) power that is growing inside the character and that he must learn to wield alongside his others abilities, more so than some other kineticist subclasses.
Edit : From stat description "Wisdom describes a character’s willpower, common sense, awareness, and intuition."
I think willpower is precisely what define a god in those settings, they are just powerful wills without true actual bodies.
Don't know how do you and @JohN100x1 see this?

I am not disagreeing with the mechanical standpoint, I am saying that I am having problems coming up with a clever description for both to make them sound like features for a divinely touched person growing like that because all my ideas circle back to gods who are terrible examples for that...

Oh, will think about something revolving over power surge in the legacies topic for archetype and mind burn.

I won't be of much help with the descriptions. I've spent all my creative power trying to think of names for the reworked Aura of Righteous Wrath variants.

@JohN100x1 so apparently multiple classes in a ContextRankValueType.Classlevel do work?
I am fairly sure that was the first thing I had tried though and that the whole reason why I added a second version of the context was because my heal with the paladin legacy had not increased with that, when I just retried it though it did work now and now it does work even if I do not change its type to sum with multiple classes in the same array?
Either way I added some additional breakpoints for the method to realise that it retravels already patched features and abort and the change in handling ContextRankProgressions should now also cover domainpowers...

anyway with that I am done for today and going to bed good night

good night 😴


How to use my own rules when using the "Create eslintrc.json file" command?

Is there a template somewhere? I couldn't find it in the extension directory.
I just want to paste my own eslintrc file on top of it, as I have a standard one that is used by 99% of my projects.
Thanks.

This is currently not supported.

@mikevaleriano For your use case it might make more sense to publish your configuration and import it as a dependency to your project. That way, they also stay up-to-date.
Slightly related: https://github.com/Microsoft/vscode-eslint/pull/423

I agree with @Hirse
OK to close?

Yes, I've been doing something similar, but linking that way is faster and more maintainable. Thanks.


CloudFlare Workers Adapter can't serve prerendered pages

Describe the bug
We embed svelte kit blog inside existing site, using Cloudflare workers to intercept requests to /blog* route. Previously we had an issue that scenario https://github.com/sveltejs/kit/issues/10937 that was successfully fixed in recent upgrade. Now, pages are successfully served from subpath. But if we want to serve prerendered page content, attempt to open the page directly causes error. It means that if I go to /blog page, and then go over the link to /blog/prerender page - everything works as excepted. But if you try to open blog/prerender page directly - you will gate Cloudflare 1101 error.
As far I understand, prerendered page generated during build. And even more - it's contend stored inside corresponding KV. But when I try to open the page directly - there is some error in manifest, because of what matching KV entry isn't found.

I can see that in KV resulting key is not `prerender.html`, but `blog/prerender.cfcc6a4220.html`
I think that hash is added by vite, but actually I didn't figure out how key name related to manifest inside build. I mean, that file  `/blog/prerender.cfcc6a4220.html` doesn't served by direct link.
Reproduction
repository: https://github.com/nosovk/sveltekit-cloudflare-workers-repro
broken test link: https://sveltekit-cloudflare-workers-repro.nosovk.workers.dev/blog/prerender
it will fail if you will try to open it directly.
But you still can visit that page, by opening that link:
https://sveltekit-cloudflare-workers-repro.nosovk.workers.dev/blog/
and that pressing on a link to /prerender page
inside logs we can see error like that:

Logs
{
  "outcome": "exception",
  "scriptName": "sveltekit-cloudflare-workers-repro",
  "diagnosticsChannelEvents": [],
  "exceptions": [
    {
      "name": "Error",
      "message": "_KVError: could not find prerender.html in your content namespace",
      "timestamp":<PHONE_NUMBER>034
    }
  ],
  "logs": [],
....
}

System Info
System:
    OS: Windows 11 10.0.22621
    CPU: (20) x64 13th Gen Intel(R) Core(TM) i7-1370P
    Memory: 35.48 GB / 63.67 GB
  Binaries:
    Node: 20.5.1 - ~\AppData\Local\fnm_multishells\23348_1701858361774\node.EXE
    npm: 10.2.1 - ~\AppData\Local\fnm_multishells\23348_1701858361774\npm.CMD
    pnpm: 8.6.12 - ~\AppData\Local\fnm_multishells\23348_1701858361774\pnpm.CMD
  Browsers:
    Edge: Chromium (119.0.2151.97)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    @sveltejs/adapter-cloudflare-workers: ^1.2.2 => 1.2.2
    @sveltejs/enhanced-img: ^0.1.5 => 0.1.5
    @sveltejs/kit: ^1.27.7 => 1.27.7
    svelte: ^4.2.0 => 4.2.2
    vite: ^4.0.0 => 4.5.0

Severity
blocking all usage of SvelteKit
Additional Information
Unfortunately for a blog it's crucial to being indexed by Google, it means that without prerender it's not very usefull.

can you share the deployment logs?

wrangler deploy dev output:
log.txt

Thank you. The log was very helpful. I now understand it was my mistake for copying the cloudflare adapter prerendered routing logic because there is an important distinction between the two.
Cloudflare workers store prerendered pages as /about.html so we can't request for the pathname /about that doesn't have the .html suffix like we do with cloudflare pages.

https://sveltekit-cloudflare-workers-repro.nosovk.workers.dev/blog/prerender works now, thanks a lot for help!


[WIP]Vq on reworked model

Current code is modified from a draft what I was using, and will be tested if GPU is available.
Following experiments are performed with the draft code and clean-100h subset.
Results with commandgrep -A1 %WER modified_beam_search/errs-test-*-beam_size_4-epoch-19-avg-10-beam-4.txt  :
baseline no vq:

modified_beam_search/errs-test-clean-beam_size_4-epoch-19-avg-10-beam-4.txt:%WER = 7.08
modified_beam_search/errs-test-clean-beam_size_4-epoch-19-avg-10-beam-4.txt-Errors: 410 insertions, 347 deletions, 2963 substitutions, over 52576 reference words (49266 correct)
--
modified_beam_search/errs-test-other-beam_size_4-epoch-19-avg-10-beam-4.txt:%WER = 18.66
modified_beam_search/errs-test-other-beam_size_4-epoch-19-avg-10-beam-4.txt-Errors: 956 insertions, 1097 deletions, 7713 substitutions, over 52343 reference words (43533 correct)

With vq from HuBERT extra large fine-tuned model:

modified_beam_search/errs-test-clean-beam_size_4-epoch-19-avg-10-beam-4.txt:%WER = 5.68
modified_beam_search/errs-test-clean-beam_size_4-epoch-19-avg-10-beam-4.txt-Errors: 286 insertions, 258 deletions, 2444 substitutions, over 52576 reference words (49874 correct)
--
modified_beam_search/errs-test-other-beam_size_4-epoch-19-avg-10-beam-4.txt:%WER = 15.80
modified_beam_search/errs-test-other-beam_size_4-epoch-19-avg-10-beam-4.txt-Errors: 797 insertions, 875 deletions, 6597 substitutions, over 52343 reference words (44871 correct)

@pzelasko Here is that recipe.

Thanks!

Now results are quite reproducible after fixing some bugs:
Result of first run:
errs-test-clean-beam_size_4-epoch-19-avg-10-beam-4.txt:%WER = 5.61
errs-test-other-beam_size_4-epoch-19-avg-10-beam-4.txt:%WER = 15.95

second run:
errs-test-clean-beam_size_4-epoch-19-avg-10-beam-4.txt:%WER = 5.67
errs-test-other-beam_size_4-epoch-19-avg-10-beam-4.txt:%WER = 15.60

Now I am doing some experiment with function "compute_mask_indices"  to predict masked indices only.

Cool!
I'm hoping we can merge this fairly soon, which will make it easier to keep it up to date when other things change.

Sure. I am implementing your suggestions about predicting indices of masked region only. After that will finalize this pr and make it meargable.


Latest Facebook Updated SSL Bypassed Required

I would like to update update apk to latest version & also Create Separate Repository for Open source script how we can patch manually if any future updates occurs.

Canary Mail
You've received a secure email
@.*** has sent you a secure email via Canary Mail.
Read Secure Email (https://secure.canarymail.io/read?obj_id=36c7bfa2-f300-4c59-8515-74f69905ad04&obj_key=Y0JsMnFLUGloS3lSbFFzVHNMcTFiZz09&thr_id=36c7bfa2-f300-4c59-8515-74f69905ad04)
If you expect to correspond often with @.***, we recommend you download Canary Mail for free.
Download Canary (https://canarymail.io)
Privacy (https://canarymail.io/privacy.html) | Terms (https://canarymail.io/terms.html) | Docs (https://help.canarymail.io/) | Support (https://canarymail.zendesk.com/hc/en-us/requests/new)
Copyright © 2021 Canary Mail, All rights reserved.


Tracking audiences in a privacy-first manner

Is your feature request related to a problem? Please describe.
In segmentation (personalization) use cases, we need to report on how each defined audience performs relative to the others. Currently, we do not have any way to track this data. This used to be tracked under the audience checkpoint, but got removed for privacy concerns.
Describe the solution you'd like
We want to re-introduce the checkpoint, and we want to be able to track this in a privacy compliant way, as discussed over slack.
Differential audiences is what we settled upon where we will randomly select whether we report the "true" audience, or pick a random one from the list. And in the reporting, we'll compute the variance from a uniform distribution to estimate the audience distribution.

:tada: This issue has been resolved in version 2.27.0 :tada:
The release is available on:

v2.27.0
v2.27.0
GitHub release

Your semantic-release bot :package::rocket: