add test project

Hi guys!
I made a test project for you. I hope you like it 😃
It's not very extensive, but it's a start.

Thanks for tackling this, it's one of those things that I've being meaning to do, but never got around to.
I'd prefer we use https://www.nuget.org/packages/Microsoft.AspNetCore.Mvc.Testing for the integration tests, rather than hit the Azure instance, as this way tests can be run before deployment has happened.
The file src/staging.runsettings shouldn't be include, I'm assuming it's mainly there because it's relying on the Azure hosted resource.
Rather than creating a class for the status codes, I'd like to find a way that we can reuse the classes from the main web project, so we don't have the duplication of code.

I'd prefer we use https://www.nuget.org/packages/Microsoft.AspNetCore.Mvc.Testing for the integration tests, rather than hit the Azure instance, as this way tests can be run before deployment has happened.

That's a good idea, but I don't think it excludes running against the deployment because things often work differently when deployed.

The file src/staging.runsettings shouldn't be include, I'm assuming it's mainly there because it's relying on the Azure hosted resource.

I wasn't sure whether to include it. It's for running the tests locally and it saves everyone trying to figure out how to set it in case they want to run locally. I can remove it if you like.
Yes, it's for testing against the deployed resource, but as I said above, maybe it's a good idea to keep that (as well as using
Microsoft.AspNetCore.Mvc.Testing).
It's the same resource that's defined in staging.yml. BTW that was defined by you, but didn't exist, so I created it.

Rather than creating a class for the status codes, I'd like to find a way that we can reuse the classes from the main web project, so we don't have the duplication of code.

That's possible, but then the tests are using the code that they're supposed to be testing, so if the code breaks then the tests will still pass because they're using the broken code.

I'd prefer we use https://www.nuget.org/packages/Microsoft.AspNetCore.Mvc.Testing for the integration tests, rather than hit the Azure instance, as this way tests can be run before deployment has happened.

That's a good idea, but I don't think it excludes running against the deployment because things often work differently when deployed.

Since there's a self-hosted version (you can pull the container yourself), I more want to focus on testing the functionality at its raw state, rather than what production infrastructure brings, because they should work the same way. That NuGet package stands up the webserver in another process and you use HttpClient to call it, meaning that it's still making network calls (although they aren't going outbound), which covers enough of what I'm happy with. This also means that we can return the 1xx range and test them, something that's not possible on Azure.

The file src/staging.runsettings shouldn't be include, I'm assuming it's mainly there because it's relying on the Azure hosted resource.

I wasn't sure whether to include it. It's for running the tests locally and it saves everyone trying to figure out how to set it in case they want to run locally. I can remove it if you like. Yes, it's for testing against the deployed resource, but as I said above, maybe it's a good idea to keep that (as well as using Microsoft.AspNetCore.Mvc.Testing). It's the same resource that's defined in staging.yml. BTW that was defined by you, but didn't exist, so I created it.

Yeah, that resource isn't up to date anyway (presently there isn't a staging deployment...).

Rather than creating a class for the status codes, I'd like to find a way that we can reuse the classes from the main web project, so we don't have the duplication of code.

That's possible, but then the tests are using the code that they're supposed to be testing, so if the code breaks then the tests will still pass because they're using the broken code.

From a test validation standpoint, I'm find with I request a 200, I get back a 200, with the body/headers/etc. as described in code. If the response body/headers/etc. aren't aligned with the HTTP spec, well, that's a different problem and beyond the scope of testing, I'm really only interested in that the custom action result works the way that is expected and returns what is defined in the code.

I updated to reuse status classes from main project.
Next I'll change it to use Microsoft.AspNetCore.Mvc.Testing.

@aaronpowell I think I've addressed all your comments now

Looks good, I'll see if I can get to it properly over the next week

Yeah, that resource isn't up to date anyway (presently there isn't a staging deployment...).

I made staging.yml work as intended, but if you're not interested in a staging environment then it's just an annoyance now and you could delete this file. Or create the resource - that's no big deal - but if you're not going to use it then there's no point.

Hi @aaronpowell
I'm sure you're busy with other things, but any chance you could have a look at this. I can delete staging.yml if that's a blocker for you.
I was thinking about making a PR for #70 and it'd be nice to be able to include some tests.

Apologies @neilboyd I'll have a look at this tomorrow

Thanks, sorry it took so long to action, between me forgetting about it and then I got sick at the end of last week, but finally merged in 🎉


Does it support heartbeat? Do we need heartbeat in tendermint to keep alive?

Yes, go-p2p pings the connection automatically.


Filter input iban

Coverage remained the same at 85.286% when pulling 9347b6d09e628003fe9dec04c6f801c35034c5df on filterInputIban into 945c7b5f0ee9ace3df18b662a2e06f41f83085a5 on master.


Content edit for the CTA Guideline/Placement section

As-Is:
Placement
CTAs should be quickly identifiable, even if a reader is just scanning the page. Make them eye catching and easy for your reader to find. A CTA’s placement should also be located where it makes the most sense for the content structure of the page, and they should not be scattered throughout the page. Also note that while it’s important to place a CTA in an easily identifiable position, do not place them too early on the page, where a reader will be presented with the link before reading the objective your page.
Visual references:
(coming soon, I am having trouble uploading the image)

Feedback for edit:
"make them eye catching" is a little misleading because we would want teams to use the link and buttons styles that are in the system already, not change them to make them eye catching. Maybe we can say something like "make them prominent enough"...
instead of "they should not be scattered throughout the page" maybe we should say something like “the CTAs should help guide users through the content and can be placed strategically (or thoughtfully) throughout to drive users to the content or action they are trying to achieve”
This particular feedback is coming from the design for Lila’s learn page does have CTAs distributed throughout the page and “scattered” could be interpreted in layout or placement
"do not place them too early" could say something like: placing a CTA high on the page could distract/drive users somewhere else before they reach the main objective of your experience, consider where the primary actions should be placed and when the users are ready for them.

@JennySanchez Draft attached that addresses the concerns above. Can discuss if you'd like:
https://ibm.box.com/s/3llinph7ipi42ux5xbht32a3gznr0tt7

@jimpistilli We are good with this task. Thank you for taking care of the edits.


Prevent zombie processes

Unfortunately go doesn't always ensure that execd processes are completely
waited for. On linux this means that zombie processes can occur.
This PR ensures that these are waited for by using signal notifier in serv and
passing a context elsewhere.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

:rocket:


[ci] migrate CI runner to local nixos machines

remove all original machine setup script
add new setup script, which is used for interacting with systemd socket, to sync NFS data to overlayfs
set java home to work directory globally, to avoid ammonite complaining about RO fs

Ready to be merged.


Fixing callout width/height calculations.

Making changes to Callout with a custom component.
The height is no longer fillHeight: true, which means the callout sets the component width, and the component sets the callout height. This seems closer to the flow of width/height calculations used previously.

Related: https://github.com/Esri/arcgis-runtime-samples-qt/pull/1371

I get it, thanks for the heads up; will test in certification.
LGTM


preload rails in rake tasks?

With rake que:work, been running into undefined constants and other things that indicate that rails isn't fully loaded.
I noticed that resque here https://github.com/resque/resque/blob/1-x-stable/lib/resque/tasks.rb#L57 runs Rails.application.eager_load! on boot.
que should do something similar, right?

Rails.application.eager_load! fixed similar problems in my rake task.


Add documentation comments for TypeScript SDK

ref https://tsdoc.org/

There are some issues with this because of the way the SDK is setup. typedoc spits out only the documentation for the exported items which in our case is a very small subset of the API (4 interfaces recalling from memory). I need to look into the plugin ecosystem to see if I can find something that we can use to export all the required interfaces. if not we may need to write our own plugin or build on top of tsdoc.

The new SDK comes with tsdoc and the docs are published to https:/fermyon.github.io/spin-js-sdk
the v1 SDK docs can be found at https:/fermyon.github.io/spin-js-sdk/v1


Please list orelib under the list of dependencies as a QOL improvement

Mod Version:
Forge Version:
Link to crash log (if applicable):
Description:
The newer versions of dynamic surroundings require orelib 3.5+ as a dependency and will not run if it is missing but this mod is not listed as a dependency under the relations tab and while it is mentioned in the release notes, it would be a lot nicer if it were listed as a required dependency under the relations tab as well.

CurseForge has it as a Required Dependency.

This issue isn't really for me since I went into the release notes and actually read them but for other people that use the dependency listing in the relations menu instead of going into the release notes.  The interesting thing about it is that if you go into the relations menu on the orelib mod, dynamic surroundings is listed as a mod that requires orelib.  So there may be something not right on twitch's side or something idk.

Yep.  The other main question is the "relations" link on the main page is tied to which version of the mod?  There are versions for 1.10, 1.11, 2 for 1.12, and 1.7.10.  Potluck!


MINIFICPP-654 - C API: failure callback improvements

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

It's not final, further changes expected as I'm not yet satisfied with the code quality, but the concept can be reviewed at least.

@arpadboda is this ready for review? You stated, "it's not final"

Yes, I plan some further changes, but most probably in scope of a different PR.

Looks good. minor comments, nothing that's a big deal


How to run imgproxy after installed

Hi,
After I installed the imgproxy. How can I run it, It's nothing after run "CGO_LDFLAGS_ALLOW="-s|-w" go get -f -u github.com/imgproxy/imgproxy"
or "PKG_CONFIG_PATH="$(brew --prefix libffi)/lib/pkgconfig"
CGO_LDFLAGS_ALLOW="-s|-w"
CGO_CFLAGS_ALLOW="-Xpreprocessor"
go get -f -u github.com/imgproxy/imgproxy"
Also I can't run imgproxy commond.
When I go to the browser and input the localhost:8080, or serverdomain:8080. I don't show any thing.
I am new for golang, do you have more detail step for install an run imgproxy?
Regards,
Tommy

Did you read documentation? CHeck Configure options and try to use this https://docs.imgproxy.net/#/generating_the_url_basic?id=example
if you use url, not command line mode

Hey Tommy,
go get installs binaries to $GOPATH/bin. Make sure you added it to your PATH.

Closing due to no activity here


Use loadash shuffle instead of constructing the final list one entry at a time.

@lucasmazza Thanks!


fatal error: Error raised at top level: SMTP.SMTPClientError.invalidPassword

Currently the user is required to change their settings for their Gmail account to "Allow for less secure apps"
fatal error: Error raised at top level: SMTP.SMTPClientError.invalidPassword(534, "5.7.14 <https://accounts.google.com/signin/continue?sarp=1&scc=1&plt=AKgnsbtW"): file /Users/buildnode/jenkins/workspace/oss-swift-package-osx/swift/stdlib/public/core/ErrorType.swift, line 160

@b20160 Thanks for the reminder, I'll put that in the docs 👍

This has been added to the example.


Webfontloader via npm?

I'm not 100% sure why is the webfont bundled in the vendors. Couldn't this be added using npm?
https://github.com/typekit/webfontloader
https://www.npmjs.com/package/webfontloader

jep this works via npm will be added to master in version 3.0.0

Do you have a branch open for this?

this is in branch #84


(Fix) Allow service support users to edit completed projects

Service support users should be able to edit completed projects, as this is a work request we get occasionally and service support should be able to manage on their own.
Checklist

[ ] Attach this pull request to the appropriate card in DevOps.
[x] Update the CHANGELOG.md if needed.

Me too 🤦  I think we did update but not edit


Apache2 reverse proxy ws?

Hi, everything works great (web) but i can get it works with apache2 reverse proxy - ws.
IP is behind cloudflare also
Any hint what can be wrong?
<VirtualHost *:80>
    ServerName ntfy.someserver.blabla
    ProxyPass / http://<IP_ADDRESS>:1234/
    ProxyPassReverse / http://<IP_ADDRESS>:1234/

    ProxyPass / ws://<IP_ADDRESS>:1234/
    ProxyPassReverse / ws://<IP_ADDRESS>:1234/
</VirtualHost>

What was it? Is there something missing in the docs?


Implement simple sleep for thread_sync.

Implements an intended part of the thread_sync API: If an empty slice of operations is passed to thread_sync along with a timeout Duration, thread_sync will just sleep the specified amount of time, allowing for easy and efficient sub-second sleeping.
Addresses #98
Alternatively, we could pass a sleep operation to a local stack word that will never awaken, and thus use timeout for the same effect. This would make the in-kernel implementation slightly simpler, but would make sleeping more expensive and involve more coordination.

Yep, that TODO is tracked by #42

I see the other note on time. I will add a separate issue for making the time types used in the kernel match up with the twizzler-abi.

Issue is now tracked in #100


support v3 UUIDs

support for rfc 4122 uuids (hashed by md5)
addresses #71

Looks great to me, thanks! Could you update the .travis.yml as well to test this on CI?

Sorry for the oversight @alexcrichton . Should be good now!

Awesome, thanks!


Postgres JSONB as saga persistence

Postgres JSONB fields provide nice NoSql alternative that is fully consistent and transactional. When it is being used as saga persistence, it requires no mappings unlike ORM versions (EF and NH).
I have implemented a saga storage using the Marten library. We have put it in use and it has proven to work reliably with both single and query versions.
The question is if @phatboyg wants to take it into the main repo or it will be separate? I can submit a PR.

How does Postgres work with locks?

I am not sure if a particular RDBMS locking strategy is the subject of this issue. There is a good overview here: http://blog.nordeus.com/dev-ops/postgresql-locking-revealed.htm

Marten and Postgres were on my to do list. I'd gladly pull it in if it's been started and working.

@phatboyg is the name MassTransit.PostgresMartenIntegration ok or you have something else in mind?

How about just MassTransit.Marten for the NuGet package, and MassTransit.MartenIntegration for the assembly name/namespace? That's consistent, and Marten only works with Postgres so it's pretty obvious.

Yeah, this would do. Will need to add Postgres to AppVeyor YAML file also to run tests.

Added to Appveyor.

One issue here - all MT assemblies target fw 4.5.2 but Marten requires 4.6.1. Is this an issue?

Well, the specific Marten assembly can target 4.6.1 I suppose, but MT will remain at 4.5.2

@phatboyg can you also add createdb MartenTest to AppVeyor?

I think I did it right.

The issue now is that fake script sets 4.5.2 as target framework for the build and my assemblies do not build.

You can probably drop that target spec in the build.fsx and let it use what's already in the csproj.


⚠ Jenkins Plugins Artifactory has degraded performance

In 1826b45, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1324 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 9cc0c78 after 12 minutes.


HyperLogLog

Hello @turu! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/yalla/streamprocessing/item_counters.py:

Line 52:80: E501 line too long (94 > 79 characters)
Line 53:80: E501 line too long (95 > 79 characters)
Line 55:80: E501 line too long (87 > 79 characters)

In the file tests/yalla/streamprocessing/test_item_counters.py:

Line 3:80: E501 line too long (82 > 79 characters)
Line 40:80: E501 line too long (93 > 79 characters)
Line 43:80: E501 line too long (116 > 79 characters)


.Net RC4: HandlebarsPlanner returns "The given key 'key' was not present in the dictionary"

Describe the bug
I have a console application which loads into the kernel 2 plugins with 3 functions:

A native one, to retrieve the number of US population given a year
A native one, to retrieve the number of US population given a year and a gender
A prompt one, to turn a text into a business mail

I'm using the HandlerbarsPlanner to process the following ask, which requires using some of the plugins I've imported:
Write a mail to share the number of the United States population in 2015 for a research program.

This is the code I'm using:
string apiKey = configuration["AzureOpenAI:ApiKey"];
string deploymentName = configuration["AzureOpenAI:DeploymentName"];
string endpoint = configuration["AzureOpenAI:Endpoint"];

var kernel = Kernel.CreateBuilder()
    .AddAzureOpenAIChatCompletion(deploymentName, endpoint, apiKey)
    .Build();

kernel.ImportPluginFromType<UnitedStatesPlugin>();

var pluginsDirectory = Path.Combine(Directory.GetCurrentDirectory(), "Plugins", "MailPlugin");
kernel.ImportPluginFromPromptDirectory(pluginsDirectory, "MailPlugin");

#pragma warning disable SKEXP0060 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.
var planner = new HandlebarsPlanner();

var ask = "Write a mail to share the number of the United States population in 2015 for a research program.";

// Create the plan
var originalPlan = await planner.CreatePlanAsync(kernel, ask);
Console.WriteLine(originalPlan);

// Execute the plan
var originalPlanResult = await originalPlan.InvokeAsync(kernel, []);
Console.WriteLine(originalPlanResult);

#pragma warning restore SKEXP0060 // Type is for evaluation purposes only and is subject to change or removal in future updates. Suppress this diagnostic to proceed.

If I explore the plan that is generated, I can see in the Handlerbars template all the information about my plugins:
## Complex types
Some helpers require arguments that are complex objects. The JSON schemas for these complex objects are defined below:

### UnitedStatesResponse:
{
  "type": "object",
  "properties": {
    "Year": {
      "type": "String",
    },
    "TotalNumber": {
      "type": "Int32",
    },
    "Gender": {
      "type": "String",
    },
  }
}

## Custom helpers
Lastly, you also have the following helpers you can use:

### `UnitedStatesPlugin-GetPopulation`
Description: Get the United States population for a specific year
Inputs:
    - year: String - The year (required)
Output: UnitedStatesResponse

### `UnitedStatesPlugin-GetPopulationByGender`
Description: Get the United States population who identifies with a specific gender in a given year
Inputs:
    - year: String - The year (required)
    - gender: String - The gender (required)
Output: UnitedStatesResponse

### `MailPlugin-WriteBusinessMail`
Description: Write a business mail
Inputs:
    - input: The text to convert into a business mail. (required)
Output: string

I can also see that the generated plan is correct:
{{!-- Step 1: Set the year for the data we want --}}
{{set "year" "2015"}}

{{!-- Step 2: Call the UnitedStatesPlugin-GetPopulation helper to get the US population for the year 2015 --}}
{{set "USPopulation" (UnitedStatesPlugin-GetPopulation year=(get "year"))}}

{{!-- Step 3: Prepare the text for the email using the concat helper --}}
{{set "mailBody" (concat "Dear Research Team,\n\nThe Total U.S. Population Data for the year 2015 was " (get "USPopulation.TotalNumber") ".\n\nBest,\n[Your Name]")}}

{{!-- Step 4: Use the MailPlugin-WriteBusinessMail helper to format the email text --}}
{{json (MailPlugin-WriteBusinessMail input=(get "mailBody"))}}

However, when I try to execute the plan by calling the InvokeAsync() method, I'm getting the following exception:
The given key 'USPopulation.TotalNumber' was not present in the dictionary.'

However, if I put a breakpoint in my plugin, I can see it's called and the returned object includes a property called TotalNumber with the population value:

To Reproduce
The full sample is available here: https://github.com/qmatteoq/SemanticKernel-Demos/blob/rc/SemanticKernel.Planner/Program.cs
Expected behavior
The planner calls the needed plugins successfully and provides me the expected answer.
Platform

OS: Windows
IDE: Visual Studio 2022 17.8.3
Language: C#
Source: NuGet package version 1.0 RC4

This should've been fixed here: https://github.com/microsoft/semantic-kernel/pull/4343, please re-open if the issue persists.


fix(install): Update install command to use new release URL format

As of v3.2.0, html-sketchapp has a new release URL format which breaks our install command. This PR updates the logic for generating the download URL, and also updates the minimum supported version to v3.2.0 in our package.json.
(Ping @kdzwinel)
Update: Also had to update the snapshots to match the latest version of html-sketchapp.

Ouch, sorry for that. Didn't realize someone depends on the file name. I'll be writing down release process instructions this week (@burakukula will be publishing next release), so I'll make sure that "name plugin file correctly" is on the list.

@kdzwinel Would it be too much trouble to start publishing the compiled Sketch plugin to npm again? That was by far the easiest from the perspective of this tool, since it didn't require an extra download step from GitHub.
Alternatively, is there a better way for us to provide a painless install process?

@markdalgleish I didn't want to keep build artifacts in the repo or artificially inflate NPM package size, but if it makes it much easier for you we can start including it again when deploying to NPM.

Yeah, I don't want to force you to bloat up your npm package. I guess the main thing is that the plugin release process is automated so that we can depend on it to be consistent across releases. Is this the case already?

Not really automated (can we automate creation of a GH release?), but it's on the release checklist, so you can be fairly sure we will remember about it. I'm starting to think though that it might be a good idea to put it back on npm - https://github.com/brainly/html-sketchapp/issues/121


Pilot MCP sink shouldn't expect all collection types from source server

Describe the feature request
When the source server only provides a subset of collections (e.g. istio/networking/v1alpha3/VirtualService), pilot operates on just the data provided, without throwing this error for the collections not in the subset Error receiving MCP response: rpc error: code = InvalidArgument desc = unsupported collection "istio/mixer/v1/config/client/quotaspecs".
Describe alternatives you've considered
The source server would have to send empty collection resources for all of the collections not being used. As new collection types are added, source servers for MCP resources need to send empty resources for these as well.

I think it is reasonable to configure Pilot to only support (listen for) types that source can provide. This could also be a configuration where default is all the istio config types so we can guarantee that it will not impact config path for galley or other components if one decides not to provide this config property. @ozevren & @ayj WDYT?

We are going to start working on a PR to configure pilot to listen for specific config types, with the default being all config types, as @Nino-K mentioned.
In the long term, we'd like to discuss a potential change to MCP itself to have sinks discover the config types supported by a source server.

Discovery is something we have discussed in the past. I think it is reasonable have a collection that will serve discoverability metadata.
I am a bit worried about serving empty collections when the source is not serving the requested collection. This is likely to be an indicator of misconfiguration and it is harder to diagnose when looking from only one side (i.e. just Pilot logs).

Let's not make this even more complex without a reason.
Pilot asks for all the things it cares about.
The MCP server should send empty result if it doesn't have any resource of that type ( including if the CRD is missing or it's something it doesn't know about ).
Simple and easy.

We think that if we leave it this way, in a world where there are multiple MCP servers sending Pilot configuration, every MCP server would have to know to send these empty results, so they would break anytime Pilot starts requesting a new resource type.
It seems like the goal of MCP servers is so that unique configuration can be sent to Pilot, and requiring all MCP servers to implement the full set of configuration types that Pilot is capable of accepting doesn't line up with this goal. It would be a breaking change to Pilot every time new configuration was added.
Let us know what you think, and if we have the wrong idea.

We need to consider the fact that not all the environments (K8s, CF, Consul, etc) are equal in the type of configuration they provide. Having a source that sends empty resources to bypass MCP errors does not seem very semantic. Discovery process seems very reasonable where a source and a server can establish a contract on what collection types they should exchange. Also, this will solidify the MCP contract and increases the efficiency (potentially) between source and the server and perhaps makes debugging less cumbersome. Currently, MCP controller initializes with an assumption that the source is going to provide all istio config types, but in the case of CloudFoundry, there are only a handful of types (ServiceEntries, Virtual Services, Gateways and destination rules) that are currently useful to the environment. For other types CF source server is forced to forward an empty resource. This also introduces code instability where the source server code has to change/break every time a new type is introduced in pilot.

Do you all think you'd consider a PR  at this point with configuring Pilot with config types to start with, and at the next config working group meeting we can bring up the possibility of discovery for resource types for the long term?

In the config meeting, a third option was proposed:
Pilot could handle unsupported collection errors, and assume that that means the configuration source doesn't support that configuration type. Then Pilot could report that configuration type as synced and not attempt to retrieve configuration for that type from that source again.
This reduces the configuration burden while still satisfying our use case, but also has the risk of suppressing real errors. We will compile a proposal based on the options in this issue.

We think that if we leave it this way, in a world where there are multiple MCP servers sending Pilot configuration, every MCP server would have to know to send these empty results, so they would break anytime Pilot starts requesting a new resource type.

I am not sure why this is the case? The source shouldn't need a special case for every resource type. If a source only new about VS and SE, can it just have code like
switch resourcetype {
  case VS: send []{list of virtual service}
  case SE: send []{list of service entry}
  default: send empty list
}

@sushicw this is the issue I was referring to

Not that this introduces a strong coupling between the MCP source and sink, where the source MUST upgrade first when new resources are introduced. In the standard Istio case this means you need to upgrade galley before Pilot. Most users are not actually doing this and just upgrading everything at once and hoping things will work out.

Unsupported collections can occur as transient or permanent errors. Transient errors occur during upgrade/downgrade and should eventually resolve. Service may be disrupted until source/sink agree again. Permanent errors occur in CF-type use cases. Here are the solutions I've seen proposed so far:

source provides an explicit list of supported collections as a collection itself (i.e. a meta-collection per @ozevren's suggestion).
source returns "collection not supported error" if sink requests an unsupported collection.
source returns empty collection for unsupported collections.

(1) is the most complex and most explicit. Requires code changes to source and sink.
(2) is also explicit and is what we support today. Requires minimal code changes to sink if we want to skip over unsupported collections.
(3) requires the least amount of code change. It's also the least explicit. The client can't differentiate unsupported from empty collections. Requires minimal code changes to source.
I'd argue for enhancing (2) with better sink/client handling of unsupported collections. It's simpler than (1) and doesn't hide errors like (3). In practice, MCP connections will reset when either source or sink is upgraded, giving an opportunity for the list of supported collections to be reset. (2) could be further enhanced so that the sink doesn't request collections it knows it doesn't need (CF case).

@XanderStrike, you marked this as staleproof. Were you interested in pursuing one of the options suggested above?

I marked it as staleproof because an issue doesn't stop being an issue just because nobody has the bandwidth to work on it. That system is bad.
We would prefer 2, but have changed priorities since the issue was created and we're likely not going to pick up this work any time soon.

Can I retarget this for 1.6?  Who is working on it?

IMO this is lower priority with istiod. It may be worth reconsidering if someone wants implement option 2.

Given our current plans moving away from MCP I don't think we will implement this


Fix unnessary arrow with 0 students

Removing the bug of arrow showing during summary command.
Now with 0 students, this is display:

LGTM


CIV-7301 Add Lip flag hearing fee unpaid

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ x ] No

We dont need to create new gateway, we can extend the current one to cater for Lip, also you have the yes and no the wrong way around


Provide a service to attach a computation to Application

From @jetinder20 on January 30, 2018 6:37
null
Copied from original issue: hashmapinc/Timeseries#151

From @jetinder20 on January 30, 2018 6:40
This also includes providing the service to invalidate an application when attached computation is deleted. However, that service needs to be plugged in computation delete flow.

The Service to invalidate an Application is already there. I will work with Jay/Himanshu to plug that in computation delete flow when its ready.

The service has been plugged in.


Create dashboard

Create dashboard with total of all 5 plots and 3 KPI as specified in dashboard sketch.

Done


Can you support copying value on double-click?

组件非常不错，我现在自己写的，当太大了会卡顿，如果你支持双击value添加事件就用你的了

Hi, I took the liberty to translate the title, please try to use English in this repository.
If I understood the translation correctly, you want to be able to copy the value on a double click.
This would require to expose the double-tap event, I will add this, in the meantime you could use the already exposed long-press event to copy the value.
Example usage:
https://github.com/Bungeefan/flutter_debug_overlay/blob/519f76295f7e4aa99aa5dbfa1ca6c14e4686b3e8/lib/src/http_log/item/body_item.dart#L62-L68

This now available in flutter_json: 0.0.2.


chore(main): bump version to 2.4.0

:robot: I have created a release beep boop
2.4.0 (2023-08-23)
Features

enabled SWC compiler for tests (6fe0a57)
using SWC compiler for E2E tests (341b525)

Bug Fixes

added NestJS SWC defaults (a14a6ab)
repl mode module not found issue (3029e43)

Continuous Integration

update dependabot config (b6634ce)

Chores

add engine version in package.json (1d4b09d)
deps: upgrade dependency packages (b5f3306)
remove .swcrc file + update webpack config (936bf52)

Documentation

update CONTRIBUTING.md (4baf353)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/kenso312/nestjs-v10-webpack-boilerplate/releases/tag/v2.4.0 :sunflower:


Custom validator class to resolve issue #13

Added ValidateIfDHCPExclusionsEnabled class to resolve issue #13.
Validators are evaluated in listed order. The class checks whether the lan_dhcpv4_exclusions field is True/False. If the value is False, then it stops validation. If it's True, then the IPAddress validator is run.
Added error message to home.html for the DHCP exclusion fields.

As discussed @quackbox this works albeit with those UI issues we discussed around the page reload causing odd problems such as unticking the box and reloading the page ticks the box again but doesn't display any fields, or leaving the box ticked and reloading the page removes the fields but keeps the box ticked due to the javascript onload event for the function being called.
Try find a nice way to fix the UI bugs whilst still keeping the functionality you have and will review once you commit that.

I've updated the home.html page to call check_dhcpv4_exclusions() on load instead of just on click for state changes.

Looks good, thanks


pi.js is more progressive

https://github.com/SuperPowerPlumber/pi.js

how did you find this repo lmao

I forgot


Allow steps to set the exit code of gantry

With this merged the 0.5.0 (was 0.4.1 but I think a major bump is better) release should be finished
This commit adds the ability to set specific exit codes when certain steps fail to execute. This enables scripts to respond differently for specific failures, e.g. print the disk usage when a copy-task fails.

Thank you for your time!


How do you build kaldi/lib/_clif.so?

I am trying to build a wheel for this package. I managed to encapsulate kaldi shared lib into the wheel using auditwheel. Unfortunately it still fails with
    from ._kaldi_error import *
ImportError: _clif.so: cannot open shared object file: No such file or directory

I don't understand how you build kaldi/lib/_clif.so from sources in pykaldi/kaldi/lib/clif/python/. Is there some code missing in the repo?

The cmake incantation for bulidingkaldi/lib/_clif.so is here.
How are you building pykaldi? Aren't you using cmake like we do?
It seems like you are building the clif shared library and linking against it but the python modules cannot find it at run time, since you are getting an import error instead of a linker error. We set the RPATH of pykaldi modules here so that they can find other pykaldi modules and the clif shared library inside the installed package.

I am building with cmake, but the difference is that I want a wheel runnable on a different machine. So I run python setup.py bdist_wheel -d ./.
Next, I'm using the auditwheel tool to include all .so files inside the wheel. I was able to embed libkaldi_base.so, but not _clif.so.

I am guessing that is because _clif.so is not an external library dependency but is part of the pykaldi package.

First few lines of kaldi/__init__.py might also be relevant.

I am guessing this problem has to do with auditwheel editing RPATH of extension modules. Checking the RPATH of shared libs after they are modified by auditwheel might give a clue about what is going wrong. I tried to run auditwheel repair on my end but apparently I need to first build with an older toolchain so the wheel is compatible with "manylinux1_x86_64" ABI.
Can you try running the following on your end?
mkdir tmp
unzip <wheel-created-by-auditwheel> -d tmp
cd tmp/kaldi/base
readelf -d _kaldi_error.so | head -20

This is what I get when I run it on the vanilla wheel.
 readelf -d _kaldi_error.so | head -20

Dynamic section at offset 0x2d88 contains 31 entries:
  Tag        Type                         Name/Value
 0x0000000000000001 (NEEDED)             Shared library: [_clif.so]
 0x0000000000000001 (NEEDED)             Shared library: [libkaldi-base.so]
 0x0000000000000001 (NEEDED)             Shared library: [libpython3.6m.so.1.0]
 0x0000000000000001 (NEEDED)             Shared library: [libstdc++.so.6]
 0x0000000000000001 (NEEDED)             Shared library: [libgcc_s.so.1]
 0x0000000000000001 (NEEDED)             Shared library: [libc.so.6]
 0x000000000000000e (SONAME)             Library soname: [_kaldi_error.so]
 0x000000000000000f (RPATH)              Library rpath: [/home/dogan/tools/pykaldi/tools/kaldi/tools/openfst/lib:/usr/local/cuda/lib64:/home/dogan/anaconda2/envs/clif3/lib:/home/dogan/tools/pykaldi/tools/kaldi/src/lib:/home/dogan/tools/pykaldi/tools/kaldi/tools/openfst/lib:$ORIGIN/../util:$ORIGIN/../nnet3:$ORIGIN/../lm:$ORIGIN/../kws:$ORIGIN/../sgmm2:$ORIGIN/../decoder:$ORIGIN/../feat:$ORIGIN/../fstext:$ORIGIN/../online2:$ORIGIN/../transform:$ORIGIN/../tree:$ORIGIN/../cudamatrix:$ORIGIN/../lat:$ORIGIN/../__pycache__:$ORIGIN/../rnnlm:$ORIGIN/../gmm:$ORIGIN/../ivector:$ORIGIN/../matrix:$ORIGIN/../hmm:$ORIGIN/../tfrnnlm:$ORIGIN/../base:$ORIGIN/../chain:$ORIGIN/../itf:$ORIGIN/../lib]
 0x000000000000000c (INIT)               0x1208
 0x000000000000000d (FINI)               0x1d84
 0x0000000000000019 (INIT_ARRAY)         0x202d70
 0x000000000000001b (INIT_ARRAYSZ)       8 (bytes)
 0x000000000000001a (FINI_ARRAY)         0x202d78
 0x000000000000001c (FINI_ARRAYSZ)       8 (bytes)
 0x000000006ffffef5 (GNU_HASH)           0x1f0
 0x0000000000000005 (STRTAB)             0x5d0
 0x0000000000000006 (SYMTAB)             0x240

For the modified wheel it is:
 readelf -d _kaldi_error.so | head -20

Dynamic section at offset 0x2d88 contains 31 entries:
  Tag        Type                         Name/Value
 0x0000000000000001 (NEEDED)             Shared library: [_clif.so]
 0x0000000000000001 (NEEDED)             Shared library: [libkaldi-base-d68bbe48.so]
 0x0000000000000001 (NEEDED)             Shared library: [libpython3.5m.so.1.0]
 0x0000000000000001 (NEEDED)             Shared library: [libstdc++-62affc63.so.6.0.21]
 0x0000000000000001 (NEEDED)             Shared library: [libgcc_s-c77f8044.so.1]
 0x0000000000000001 (NEEDED)             Shared library: [libc-2-05b841ea.23.so]
 0x000000000000000e (SONAME)             Library soname: [_kaldi_error.so]
 0x000000000000000f (RPATH)              Library rpath: [$ORIGIN/../.libs]
 0x000000000000000c (INIT)               0x11a0
 0x000000000000000d (FINI)               0x1d14
 0x0000000000000019 (INIT_ARRAY)         0x202d70
 0x000000000000001b (INIT_ARRAYSZ)       8 (bytes)
 0x000000000000001a (FINI_ARRAY)         0x202d78
 0x000000000000001c (FINI_ARRAYSZ)       8 (bytes)
 0x000000006ffffef5 (GNU_HASH)           0x1f0
 0x0000000000000005 (STRTAB)             0x207268
 0x0000000000000006 (SYMTAB)             0x240

Yeah it seems auditwheel is not appending to RPATH but instead overwriting it. Relevant issue.

Can you run the following as well?
``
cd tmp/kaldi/.libs
ls -ltr

Thanks! This is very helpful!

# ls -ltr
total 593992
-rwxr-xr-x 1 root root   3996768 Dec 30 21:37 libc-2-05b841ea.23.so
-rwxr-xr-x 1 root root  26212464 Dec 30 21:40 libfst-201beb5d.so.10.0.0
-rwxr-xr-x 1 root root   3640008 Dec 30 21:40 libkaldi-transform-8b82f1a0.so
-rwxr-xr-x 1 root root  46869784 Dec 30 21:40 libkaldi-lat-af312041.so
-rw-r--r-- 1 root root    193168 Dec 30 21:40 libcblas-0511b056.so.3.0
-rwxr-xr-x 1 root root   5032880 Dec 30 21:40 libkaldi-tree-8d9080fc.so
-rwxr-xr-x 1 root root  36723984 Dec 30 21:40 libkaldi-nnet2-1ccb99aa.so
-rwxr-xr-x 1 root root   2229912 Dec 30 21:40 libpthread-2-f045d9ab.23.so
-rwxr-xr-x 1 root root   5370376 Dec 30 21:40 libkaldi-matrix-b4e155c1.so
-rwxr-xr-x 1 root root  15628432 Dec 30 21:40 libkaldi-hmm-e8402a2a.so
-rwxr-xr-x 1 root root   2483320 Dec 30 21:40 libkaldi-feat-a9f3d716.so
-rw-r--r-- 1 root root     96920 Dec 30 21:40 libgcc_s-c77f8044.so.1
-rw-r--r-- 1 root root     21544 Dec 30 21:40 libdl-2-6104b774.23.so
-rw-r--r-- 1 root root   4137496 Dec 30 21:40 libatlas-aa548a16.so.3.0
-rw-r--r-- 1 root root   2631088 Dec 30 21:40 libstdc++-62affc63.so.6.0.21
-rwxr-xr-x 1 root root   3624600 Dec 30 21:40 libkaldi-sgmm2-ad236f47.so
-rwxr-xr-x 1 root root   5414536 Dec 30 21:40 libkaldi-lm-c7150916.so
-rwxr-xr-x 1 root root  32143744 Dec 30 21:40 libkaldi-kws-ab29bf80.so
-rwxr-xr-x 1 root root   3173392 Dec 30 21:40 libkaldi-gmm-129ff5ed.so
-rwxr-xr-x 1 root root   5458672 Dec 30 21:40 libkaldi-fstext-6cf8b03d.so
-rw-r--r-- 1 root root   1407288 Dec 30 21:40 libgfortran-88599953.so.3.0.0
-rw-r--r-- 1 root root    454160 Dec 30 21:40 liblapack_atlas-c3fb2ebc.so.3.0
-rwxr-xr-x 1 root root   3096688 Dec 30 21:40 libkaldi-util-1798c31e.so
-rwxr-xr-x 1 root root  17100576 Dec 30 21:40 libkaldi-online2-11df9adb.so
-rwxr-xr-x 1 root root  65259560 Dec 30 21:41 libkaldi-nnet3-13c72bd0.so
-rw-r--r-- 1 root root    267968 Dec 30 21:41 libquadmath-99889722.so.0.0.0
-rw-r--r-- 1 root root   1097496 Dec 30 21:41 libm-2-8c60f2bc.23.so
-rwxr-xr-x 1 root root   8512448 Dec 30 21:41 libkaldi-rnnlm-dab76bc8.so
-rwxr-xr-x 1 root root   2397984 Dec 30 21:41 libkaldi-ivector-dc494efc.so
-rwxr-xr-x 1 root root   2791360 Dec 30 21:41 libkaldi-cudamatrix-c23f53f9.so
-rwxr-xr-x 1 root root    710112 Dec 30 21:41 libkaldi-base-d68bbe48.so
-rw-r--r-- 1 root root    198408 Dec 30 21:41 libf77blas-d437c2dc.so.3.0
-rwxr-xr-x 1 root root 218226872 Dec 30 21:41 libfstscript-2d80e22d.so.10.0.0
-rwxr-xr-x 1 root root  36527192 Dec 30 21:41 libkaldi-decoder-9f29b929.so
-rwxr-xr-x 1 root root  45044696 Dec 30 21:41 libkaldi-chain-c898f267.so

I think we can get away with the following monkey patch until auditwheel fixes their code. This preserves RPATH directories that start with $ORIGIN. I am not sure if this change to patchelf_set_rpath is a general enough solution but it should do the job for us.
cat ./auditwheel

#!/usr/bin/env python

# -*- coding: utf-8 -*-
import re
import sys

import logging
from os.path import relpath, dirname
from os.path import join as pjoin
from subprocess import check_call, check_output

from auditwheel.main import main
from auditwheel import repair

logger = logging.getLogger('auditwheel.repair')

def patchelf_set_rpath(fn, libdir):
    rpath = check_output(['patchelf', '--print-rpath', fn]).decode("utf-8").strip()
    rpath = list(filter(lambda x: x.startswith('$ORIGIN'), rpath.split(":")))
    rpath.append(pjoin('$ORIGIN', relpath(libdir, dirname(fn))))
    rpath = ":".join(rpath)
    logger.debug('Setting RPATH: %s to "%s"', fn, rpath)
    check_call(['patchelf', '--force-rpath', '--set-rpath', rpath, fn])

repair.patchelf_set_rpath = patchelf_set_rpath

if __name__ == '__main__':
    sys.argv[0] = re.sub(r'(-script\.pyw?|\.exe)?$', '', sys.argv[0])
    sys.exit(main())

Yay! It worked!

Are you interested in a script building wheel? I could submit a PR

Very much interested! Are you building it with the manylinux docker image?

Got it. Maybe we can build the wheel and upload it somewhere as part of docker image build on Travis.


Fixed #1246: empty contents with long paginations

What type of PR is this?
bug
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #1246
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

if you need server endpoint to add a new field, just ask ...

asked at https://github.com/iotexproject/iotex-analytics/issues/170

Closed in favor of #1266


Csm asdf

Hello @vhirtham! Thanks for opening this PR.

In the file weldx/transformations.py:

Line 457:17: E126 continuation line over-indented for hanging indent

Do see the Hitchhiker's guide to code style


More smooth plotting

I don't know if I am doing something wrong, but when I plot something, the ipython gets stuck and will not run the next line until I close the plot.
This happens both on a linux and a macos machines.
Is there a way to change this behavior?

Hi, this is due to matplotlib changes to their default behavior.  See their doc on show().  More on this with turning interactive mode on, ion().
There is probably a way to set IPython startup to have Jupyter like default behaviors.  Maybe adding %matplotlib inline to ipython.StartupCommands?

Consider the MWE:
I run the file with the %matplotlib magic command in ipython.StartupCommands

import numpy as np

from matplotlib import pyplot as plt

print(plt.isinteractive())

plt.plot(np.arange(10))
plt.show()

print(1 + 1)

If I run  this file by pressing F5 (which is the command "IPython: run fil with command line arguments")
I receive that plt.isinteractive() is False. In MacOS the file get stuck in plt.show(), while in Linux it continues to the end of the file.
If I run the code with F9 (IPython: Run Code Selection or Line at each cursor),
if I use the "%run -i" option, the return is the same as above. However, if I do it with the "%load" option, the return of plt.isinteractive() is True and the cursor continues until the end of the selection and does not stop at plt.show()
Finally, it is always the case that, when I do plt.isinteractive() in the IPython terminal, I get "True"

Here is something that may help.  This is what I got on Window:

After pip install numpy, matplotlib, ipython, PyQt5 to virtual environment

 pip freeze
$ pip freeze
asttokens==2.4.1
colorama==0.4.6
contourpy==1.2.0
cycler==0.12.1
decorator==5.1.1
executing==2.0.1
fonttools==4.48.1
ipython==8.21.0
jedi==0.19.1
kiwisolver==1.4.5
matplotlib==3.8.2
matplotlib-inline==0.1.6
numpy==1.26.4
packaging==23.2
parso==0.8.3
pillow==10.2.0
prompt-toolkit==3.0.43
pure-eval==0.2.2
Pygments==2.17.2
pyparsing==3.1.1
PyQt5==5.15.10
PyQt5-Qt5==5.15.2
PyQt5-sip==12.13.0
python-dateutil==2.8.2
six==1.16.0
stack-data==0.6.3
traitlets==5.14.1
wcwidth==0.2.13

Add "%matplotlib qt5" to configuration:

"ipython.StartupCommands": [
        "%matplotlib qt5",
        "%load_ext autoreload",
        "%autoreload 2"
]

Ran command which renders and show plot every time plt.plot() called:

$ ipython --no-autoindent --InteractiveShellApp.exec_lines="%matplotlib qt5" --InteractiveShellApp.exec_lines="%load_ext autoreload" --InteractiveShellApp.exec_lines="%autoreload 2"
Python 3.11.5 (tags/v3.11.5:cce6ba9, Aug 24 2023, 14:38:34) [MSC v.1936 64 bit (AMD64)]
Type 'copyright', 'credits' or 'license' for more information
IPython 8.21.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import numpy as np

In [2]: import matplotlib.pyplot as plt

In [3]: x = np.linspace(0, 2 * np.pi, 128)

In [4]: y = np.sin(x)

In [5]: plt.plot(x, y)
Out[5]: [<matplotlib.lines.Line2D at 0x1a744c06d10>]

In [6]:

With the same configuration I getthe behavior I mentioned above.
What is weird for me is the difference in the output of plt.isinteractive(). It kind of suggests that the expected behavior is not taking place

What is weird for me is the difference in the output of plt.isinteractive(). It kind of suggests that the expected behavior is not taking place

Agree.  matplotlib behaviors with IPython terminal need a bit of trial and error lately.  My guess is that the behaviors are dependent on the what the backend renderer can do.  I wish they follows the zen of Python "Preferably one way to do thing" and support one renderer like Qt.

I am having the same problem as @raphaelchinchilla. Would be nice if we can figure out a workaround. :pray:

Another solution is to do plt.show(block=False). It is a bit annoying but works...


Create SSH module

Currently there is an article on the wiki stating how to manually set up an ssh daemon. Maybe this can be ported into a nix-on-droid module.
I may create a PR for this if you think this would be a nice-to-have.

See https://github.com/nix-community/nix-on-droid/pull/203

See #203

Seems that PR was abandoned by the original author. I'll try to take a look if I have a chance. Thank you!

Thanks to the wiki, i was able to successfully install an SSH server !
https://gitlab.com/pinage404/dotfiles/-/commit/405665aeb8d5468184375ad69767772eb06e971b#29ca4eb2dccce271cd9a3d588f9b50575ec5607c_0_2

Cool! I had to remove the option -D # don't detach into a daemon process because it would stay stuck on that step, and without it seems to get stuck when I run exit.
added your sshd-start to the startup files of bash and zsh, and it works great
  # For starting the ssh server
  programs.zsh = {
    initExtra = ''
      sshd-start
    '';
  };
  programs.bash = {
    initExtra = ''
      sshd-start
    '';
  };

with your config i can connect to my phone from my PC through something like ssh<EMAIL_ADDRESS>-p 8022


issue

the GOD is just DAN but with every word DAN changed to GOD.

This MOD has more functionalities than DAN.

Gp

Gp

What happened?


downloading playlist on windows

Checklist

[ ] I'm reporting a broken site support issue
[x ] I've verified that I'm running youtube-dl version 2019.05.11
[ x] I've checked that all provided URLs are alive and playable in a browser
[ x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ x] I've searched the bugtracker for similar bug reports including closed ones
[ x] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE
x:\TMP\tmp>youtube-dl -i -v --flat-playlist  "https://www.youtube.com/playlist?list=PL8dPuuaLjXtPNZwz5_o_5uirJ8gQXnhEO"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-i', '-v', '--flat-playlist', 'https://www.youtube.com/playlist?list=PL8dPuuaLjXtPNZwz5_o_5uirJ8gQXnhEO']
[debug] Encodings: locale cp1250, fs mbcs, out cp852, pref cp1250
[debug] youtube-dl version 2019.05.11
[debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7601-SP1
[debug] exe versions: none
[debug] Proxy map: {}
[youtube:playlist] PL8dPuuaLjXtPNZwz5_o_5uirJ8gQXnhEO: Downloading webpage
[download] Downloading playlist: Economics
[youtube:playlist] playlist Economics: Downloading 35 videos
[download] Downloading video 1 of 35
[download] Downloading video 2 of 35
[download] Downloading video 3 of 35
[download] Downloading video 4 of 35
[download] Downloading video 5 of 35
[download] Downloading video 6 of 35
[download] Downloading video 7 of 35
[download] Downloading video 8 of 35
[download] Downloading video 9 of 35
[download] Downloading video 10 of 35
[download] Downloading video 11 of 35
[download] Downloading video 12 of 35
[download] Downloading video 13 of 35
[download] Downloading video 14 of 35
[download] Downloading video 15 of 35
[download] Downloading video 16 of 35
[download] Downloading video 17 of 35
[download] Downloading video 18 of 35
[download] Downloading video 19 of 35
[download] Downloading video 20 of 35
[download] Downloading video 21 of 35
[download] Downloading video 22 of 35
[download] Downloading video 23 of 35
[download] Downloading video 24 of 35
[download] Downloading video 25 of 35
[download] Downloading video 26 of 35
[download] Downloading video 27 of 35
[download] Downloading video 28 of 35
[download] Downloading video 29 of 35
[download] Downloading video 30 of 35
[download] Downloading video 31 of 35
[download] Downloading video 32 of 35
[download] Downloading video 33 of 35
[download] Downloading video 34 of 35
[download] Downloading video 35 of 35
[download] Finished downloading playlist: Economics

Description
Downloading playlist don't create any output file.
Same url
youtube-dl -i -v   "https://www.youtube.com/playlist?list=PL8dPuuaLjXtPNZwz5_o_5uirJ8gQXnhEO"
works OK.

WRITE DESCRIPTION HERE

--flat-playlist won't create any files.


Serbian translation completed (app/locale/sr.coffee).

Coverage decreased (-0.01%) to 63.616% when pulling f25535fb278f13d9200bdf6d7b337842daa8787a on vitalije:master into 0918b522250be7154d3927625765a78417859425 on codecombat:master.

I have just noticed that there are some words that should not have been translated. I'll post another pr.

Thanks! Merged


Doing unsubscribe unsubscribes all subscribers for a topic

I've noticed a difference in behavior of rws compared to rosbridge when using subscriptions through roslibjs. With rosbridge, if I subscribe to the same topic from two places and unsubscribe once, I still get messages to the remaining subscriber. With rws, this is not the case. It appears rosbridge keeps track of the ids of the subscribers and does not kill the ros subscription until all subscribers of a topic have unsubscribed.
Looking at the ROSLIB.Topic object created it seems it has a concept subscribeId: "subscribe:/echoer/int32:805" with a number at the end, unique to each subscriber. When calling subscribe, this is passed to the bridge as
{"op":"subscribe","id":"subscribe:/echoer/int32:805","type":"std_msgs/msg/Int32","topic":"/echoer/int32","compression":"none","throttle_rate":0,"queue_length":0}
While one might argue keeping track of this should be done in roslibjs rather than in the bridge, that's how it works today and it would be nice to be able to use roslibjs with rws as is.

I have no experience with roslibjs, can you explain a bit why one client might want to subscribe to the same topic multiple times?
Although I believe that a client should subscribe to the same topic once, I agree that it would be better to reflect the original behavior of rosbridge.

E.g. you might have a dashboard with a gauge showing a momentary value and a chart plotting the value over time, both listening to the same topic. They'd work independently of each other, not knowing if there are other subscribers to the topic.

This sounds to me like, as you mentioned, a task for the client library to manage. It can be fixed at application level if you still want to use rws, so I prefer not to add unnecessary complexity to the server.

Actually roslibjs supported unsubscribe with the specific callback as argument so we were able to do that and let roslibjs keep track of if there were any remaining listeners or if it should issue an unsubscribe to the bridge.


Support for "date" type

OpenAPI supports the date format for the string type, but tinyspec doesn't have a corresponding type. It would be great to have it. (I would even go as far as redefining the d shorthand type to the new date type, and defining the new dt shorthand for the existing datetime type.)
P. S. I'm very new to the world of REST API specifications, but tinyspec already looks like a useful "source" language for OpenAPI, an attractive alternative to just writing these verbose specifications by hand.

Thanks for the report. I think we can add the date type, without a shorthand, to not break the current API.

For some reason (lack of releases on github? very few issues created?), I assumed that the project might be too young to worry about breaking existing usage. However, now I see that the history goes back to year 2017, so such breaking changes shouldn't be done.

Merged, thank you @pvgoran!


Allow patch upgrades to D3.js dependency

We ran into a problem with D3.js that was resolved in a later patch version (v3.5.4).  This small change allows patch upgrades to D3.js (3.5.x) while locking the minor version (i.e., it won't upgrade to 3.6.x).

As mentioned in #1177, we experienced compatibility issue when updating d3 with this version syntax. So, I think it's safer to use strict syntax like current one.

Well, are you not compatible with "d3": "3.5.5" ?
I keep having to mark the resolution manually for this, and I have other libs that use D3 that do want the latest. C3 seems to be working perfectly for me with 3.5.5, but I can understand the risk.
Could you maybe add a patch release, when d3 is updated, and you are compatible, if you need to do it manually?

Is this more an issue with C3, or more an issue with how D3 is using semver? Nothing in the patch version should be backwards incompatible, so the fact previous patches have broken C3 points to something awry with something they're doing.
@mbostock, any thoughts?

D3 is on 3.5.6 now, is there any move on this? I'd really prefer to trust d3, and then pin the d3 version in C3, only after it breaks again. Even then, we should create an issue for updating d3 ASAP.
This is a cop-out, although an understandable one, from dealing with the fact d3 has in the past broken C3. We need to trust the Semver and raise issues on D3 if they add breaking changes accidentally on a patch version.
When you have to manage large numbers of dependencies on projects, having repos take the attitude of "it broke last time, so I'm not going to update this dep" is counterproductive. It brings C3 out of sync with actively developed plugins.

@SamMorrowDrums Yeah, the more I think about it, the more I agree with you. This is the reason why semver exists in the first place. It totally was updating from 3.5.0 to 3.5.3 that broke it last time (and resulted in a dozen new issues in the queue), and we should really have complained vociferously in the D3 queue about either whatever change caused the performance issue or their use of semver.
My vote is to the following:

Change D3 from a standard dependency to a peer dependency.
Change the D3 version resolution to ~3.5.0 (I.e., "update the patch version")
Set the version of D3 used in the unit testing to whichever one is specified via npm/bower (Currently it's 3.5.0 consumed via CDN).
Whinge in the D3 issue queue if they do something that breaks C3.

Thoughts, @masayuki0812?

@masayuki0812 Any chance we can get an update on this issue? There've been a few "updating D3" issues and PRs and it'd be really nice to nip them in the bud with this.
@timmclean Am currently thinking the version would be better as ~3.5.4 as that will enforce using the newer version of D3, but am worried people will have stuff built using both C3 and D3 as a dep that will break due to their stuff (and not necessarily C3) being built for 3.5.0. Any thoughts?

Anyone who needs the older version of D3 specifically can depend on the older version of C3 already. I don't think there's a substantial difference between ~3.5.4 and ~3.5.0 going forward from here.
Setting d3 as a devDependency is the wrong thing to do - folks who use a Browserify workflow won't be able to use it in their projects then.  And yes, peerDependencies should be avoided at all cost.  It's clearly a dependency in this case.
Either way, just getting the version updated would be full of win.

@thomblake Ah, very good points.
Right, I think this should probably be merged at this point. Any thoughts, @masayuki0812? :+1: :shipit:

Both ~3.5.4 and ~3.5.0 seem fine from my end :+1:

It is possible to have this both ways: ~3.5.0 || ~3.5.4 is valid version syntax, and would skip the offending middle versions. Of course, we could also do <=3.5.0 || ~3.5.4, but that seems far too liberal (I doubt d3 1.x is actually supported).

Thanks for working on this. I think ~3.5.0 is ok at this moment because the latest version of d3 looks ok and we can give specific version to d3 if needed.

Great, thanks!

:+1: :tada: :confetti_ball: :shipit: Yay!

And there was much rejoicing!

@masayuki0812 any chance we can get this published to npm?


The fabric version of luckperms should be able to give perms for normal Minecraft commands.

Description
My suggestion is that you should be able to give players perms for vanilla admin commands in luckperms fabric, like how it works in bukkit.
Proposed behaviour
On a fabric server with luckperms and fabric API in the mods folder, the command /luckperms user [user] permission set minecraft.commands.op should give the user the ability to do the /op command, exactly how it works in bukkit.

Luckperm doesnt provide those permissions. It is up to the server software. You can see https://github.com/lucko/LuckPerms/issues/2961#issuecomment-817161201


Legacy regression MarkSymbolizer has no maxX maxY

See method processSymbolVectorMarker https://github.com/geostyler/geostyler-lyrx-parser/blob/a88b8885ad0f1fdda859785bcaa7a393806127ab/src/processSymbolLayer.ts#L222
We assign maxX maxY but they don't exist in Geostyler and will not be used.

Legacy logic coming from https://github.com/GeoCat/bridge-style/commit/7cdca37c80c931772197c59225f8f173b9580e7e#diff-eeab780d2a4c7bf120cf3c43756c0793703acb10b4c536742f15b8e8a969383cL421


Distorted video (iphone)

Q
A

Bug?
no

New Feature?
no

Version Used
v0.11.1-pl-1

FFmpeg Version
2.8.14-0ubuntu<IP_ADDRESS>

OS
Ubuntu Linux 4.16.3

Actual Behavior
php-ffmpeg distorts video (iphone)
Expected Behavior
Hello. There is a problem with the horizontal video received from the iphone.
This is a problem for me. I now need to check every video before converting.
ffmpeg -i input.mov -vf 'scale = 320: 320: force_original_aspect_ratio = decrease' output.mp4
This works fine and I expected this behavior for php-ffmpeg, but this does not happen.
Instead, php-ffmpeg thinks this is in the AspectRatio class, instead of doing system ffmpeg.
\ Sorry for my English

Could you send us the code you are using?

Video source:
FF2KWhaOXoyp.mov.zip
require 'vendor/autoload.php';

$ffmpeg = FFMpeg\FFMpeg::create();
$video = $ffmpeg->open('video.mov');
$video->filters()
    ->resize(
        new FFMpeg\Coordinate\Dimension(600, 600),
        FFMpeg\Filters\Video\ResizeFilter::RESIZEMODE_INSET
    )
    ->synchronize();

$video
    ->frame(FFMpeg\Coordinate\TimeCode::fromSeconds(10))
    ->save('frame.jpg'); // bad orientation
$video
    ->save(new FFMpeg\Format\Video\X264(), 'export-x264.mp4'); // bad orientation

What happening if you replace RESIZEMODE_INSET with RESIZEMODE_SCALE_WIDTH?

It does not change anything. The problem is not in the size of the video, but in the fact that it has the wrong orientation.

But when you try to resize it to 600 x 600, it should be a square and RESIZEMODE_SCALE_WIDTH tries to further optimize it?

try playing the video using mplayer and ffplay

I'm experiencing the same issue with vertical iPhone videos.


Reviewers are not included as participants.

Gitea version (or commit ref): 1.9.2

Description
If a user adds a code review/request changes (CommentTypeReview or CommentTypeCode) they are not listed as participants. Only plain commenters and the issue poster are.
Screenshots

Fixed by #8121


Add Replicate badge to README

Hey there - I work for Replicate, a site that allows for running ML demos on the web or with an API. I added a badge to the README to point to Replicate's demo in addition to the Hugging Face one, let me know if this looks good!

Hi @jagilley, thanks for your contribution! The demo looks great!


fix build errors

#206

@my3 thanks for fixing this!

Codecov Report

Merging #207 into master will not change coverage.
The diff coverage is 25%.

@@           Coverage Diff           @@
##           master     #207   +/-   ##
=======================================
  Coverage   78.62%   78.62%
=======================================
  Files          29       29
  Lines        2914     2914
=======================================
  Hits         2291     2291
  Misses        372      372
  Partials      251      251

Impacted Files
Coverage Δ

provider/bitmovin/bitmovin.go
65.14% <0%> (ø)
:arrow_up:

provider/zencoder/zencoder.go
82.62% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[FEATURE]  Icons in forms

Description
I would like to add relevant icons in the form fields that will increase the UI and aesthetics of this site.
I want to add icons in the login and register forms OR if you want me to add the icons in all the input fields present in the site for consistent display then I can implement that as well, just let me know
Screenshots
By relevant icons I mean something similar to the below one:

Additional information
No response

@AlfiyaSiddique Yes , you add icons in all the web pages but i have one issue
Concern only for login & registration page : As these are important pages also they seems very small in mobile , so let's add the icons there and if it seems too filled with stuff, then will remove the icons from mobile view (login & registration page) only

@AlfiyaSiddique Yes , you add icons in all the web pages but i have one issue
Concern only for login & registration page : As these are important pages also they seems very small in mobile , so let's add the icons there and if it seems too filled with stuff, then will remove the icons from mobile view (login & registration page) only

Okay @apoorvaron , understood 👍


Command line error: input file squares.tla does not exist

Going through the instructions here: https://github.com/tlaplus/vscode-tlaplus/wiki/Getting-Started
Running:
TLA+: Parse module triggers the first line in the squares example code to be highlighted with a red syntax highlight (squiggly line?).  Below terminal panel and hovering over the line shows that it's not able to find the squares.tla file.
I imagine theres a simple way to pass the path or maybe configure vscode to do the right thing?  But I don't know it :( and should probably be documented in the getting started guide.

And figured it out.  For some reason vscode didn't have permissions on a folder on the Desktop?  I realized this when trying to navigate there from the terminal in the vscode UI.  Moving it to home directory fixed things.  Sorry to bother!


 Changing max-width to width in reaction-avatars fixes it:

Test URLs

user with a very small profile image: https://github.com/Strubbl
that user's profile image: https://avatars.githubusercontent.com/u/97055?v=4
comment with that user's reaction: https://github.com/streetcomplete/StreetComplete/issues/2909#issuecomment-860959344
comment by that user: https://github.com/streetcomplete/StreetComplete/issues/1176#issuecomment-860973960

Does this change the loading behavior though? Check the comment and the PR that included that code. Normally the avatars appear gray and then the image fills up. I think it you drop max- it might stop that.

The CSS rule was introduced in #645, already with max-width. I just tried this change in the test URL linked there (https://github.com/babel/babel/pull/3646) with simulated 3G speed and disabled cache, and the gray circles appear before the images are loaded. So it works the same.

Thank you!


Add the missing default scalars Float and ID.

According to the current graphql docs there are two more scalars than supported currently by gql2ts-for-server: Float and ID.
This pull request adds support for them.

Coverage remained the same at 95.522% when pulling cb4dfa6c8cdb4eae6f28ce7691b0c326ae155b12 on msiegenthaler:add-missing-default-scalars into b632bf46f21ffc2b4124471a6ec7df2f57eb1010 on nknapp:master.


Triage privileges

This PR addresses three issues:

Triage rules assigned to serviceAccounts
Do not add capabilities when privileged: false
Remove default value for SELinuxLevel

Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/2448

/retest

recheck

The commits need to be cleaned up otherwise this should be good for a review and to be merged.

recheck

recheck

/retest

/retest


nixos/cups: Fix Unable to encrypt connection:

Unable to create server credentials
by creating /var/lib/cups/ssl directory.
Motivation for this change
Partial fix for #23993, makes this troubleshooting unnecessary: https://nixos.wiki/wiki/Printing#Upgrade_Required
Reproduce wtih: lpstat -E -s, which tries to list all printers over an encrypted connection.
Things done

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

[X] 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"
[X] 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
[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg test printing

@GrahamcOfBorg test printing

Expanded the printing test to cover this.

@GrahamcOfBorg test printing


The named parameter 'htmlToolbarOptions' isn't defined

I haven't seen this breaking change in the changelog

My bad. Was using an old version


[5.8] Replace global helper with container call in Exception Handler

As @crynobone noted in #29564 we could use container instead of global helper method.

Doesn't matter either way. This is within the foundation component.

That seems strange to me, because we already have container instantiated, why should we reinstantiate it using global helper?

The container is a singleton and does not get reinstantiated with the global helper?

Yes, but if it's already assigned in class field, why shouldn't we use it instead?


Display error message rather than silently exiting when Spyder crashes on startup

Originally discussed in e.g. #12557
For users on Windows at least, when launching Spyder not from the command line directly (e.g. from the shortcut, our recommended method on that platform), if an error occurs during early startup (at least before the mainwindow GUI appears), Spyder will silently exit instead leaving the user with no indication of what happened that would lead them here to the fix, and errors this early will not trigger even a "Spyder failed to start" dialog because AFAIK that doesn't spawn until after the GUI has started.
Even with full control over the Python level of the environment and ignoring random corruption and deliberate user action, there is always a chance of an error occurring at some point during initialization due to something in the user's/OS/machine environment outside of our direct control (out of memory or disk space, env vars, locale/encoding issues, OS or machine-level problems, graphics driver/Qt errors, config settings, and various edge cases).
As a Spyder core dev who has troubleshooted and helped resolve hundreds, perhaps thousands of different startup-related issues, I know to just launch it via the command line and view what error message (or other behavior) occurs there, but I worry the average user without that fortunate experience will likely find it far more frustrating. Therefore, it possible we should improve this UX, which will also make our lives easier helping users with generic "Spyder won't start!!!111!!" issues.
The fundamental issue is that no feedback at all is presented to the user when Spyder fails to start early enough in initialization, at least before the mainwindow GUI is displayed, unless its run from a terminal.
In the case of a startup failure, IMO the UX priority is to at least convey to the user that A. An error occurred, and B. Enough information (error message, traceback, etc) to Google or report it. Right now on Windows with the shortcut, there is no feedback, Spyder simply never launches. I had no idea it was even an error until I ran it from the command line, since Spyder went through the normal splash screen process but then the GUI never displayed, so I didn't have a way to know what had actually happened or (without running it in the command line, which the average user may not know to do) what to do about it.
What I would at least naively suggest is protecting the startup routine as early as possible with a try-except that then displays (since it is hopefully relatively self-contained, coming from a small external codebase) the Report to Github dialog as would have happened if the error occurred after GUI initialization. If this is not practicable, at least a simple error dialog explaining that an error occurred, showing the traceback and giving instructions (as our Report to Github dialog does) on what to do next. Thoughts? Better ideas? @ccordoba12 @goanpeca
Originally posted by @CAM-Gerlach in https://github.com/spyder-ide/spyder/issues/12557#issuecomment-623849879

This is a tough one, but yes in principle we should be able to have a big try except and catch any potential problems. Another issue is that Qt or PyQt might even be the problem, so we would need to use something else.
What I used in the past if Qt had an error would be to open the browser with the traceback of the problem so that is one option. We could also use tkinter, or just provide a call to a native Alert MsgBox, which should be straightforward for most platforms.


Allow []byte to be used as PK

WIP branch: https://github.com/go-reform/reform/tree/slice-pk
Valid use-case: UUID in binary form, probably should be used as []byte or [16]byte, not as string.
Remaining problems:

HasPK implementation (operator != not defined on slice)
SetPK implementation (cannot convert i64 (type int64) to type []uint8)

Related to #43.

@AlekSi do you have a vision of what exactly should be done to fix the problem?

We need more information about PK type.
HasPK is currently generated with this template:
// HasPK returns true if record has non-zero primary key set, false otherwise.
func (s *{{ .Type }}) HasPK() bool {
    return s.{{ .PKField.Name }} != {{ .TableVar }}.z[{{ .TableVar }}.s.PKFieldIndex]
}

It compiles only for types that has != operators defined, and slices do not define it. We could use !reflect.DeepEqual(s.{{ .PKField.Name }}, {{ .TableVar }}.z[{{ .TableVar }}.s.PKFieldIndex]), but that might be not so good. With knowledge about PK type, we could use code with != for simple types, s.{{ .PKField.Name }}.Equal({{ .TableVar }}.z[{{ .TableVar }}.s.PKFieldIndex]) for types defining Equal(x X) bool, and reflect.DeepEqual as fallback.
SetPK is currently generated with this template:
// SetPK sets record primary key.
func (s *{{ .Type }}) SetPK(pk interface{}) {
    if i64, ok := pk.(int64); ok {
        s.{{ .PKField.Name }} = {{ .PKField.Type }}(i64)
    } else {
        s.{{ .PKField.Name }} = pk.({{ .PKField.Type }})
    }
}

It is only used by when Record with zero PK is inserted, PK is autogenerated, and RDBMS support LastInsertId: https://github.com/go-reform/reform/blob/c8eea006559b6f50a00404d9f81b191da8fd02cc/querier_commands.go#L97. In that case, SetPK argument is always int64, but PK field can have another type, for example, int32 or even type MyPKType int32. That's why we handle int64 differently and try a conversion. With knowledge about the PK type, we can use the type-specific code.
There are two ways to get more information about PK type. The first one is to parse files with go/types: https://github.com/go-reform/reform/issues/57 The second is to use some reflection in init() function to switch implementations. You are welcome to work on any of them.


fix: Wrong colored create account button

What

Fixes wrongly colored create account button.

Impacted files

login_page.dart

Screenshot

Before

After

Fixes bug

#1128

Part of

https://github.com/openfoodfacts/smooth-app/issues/1171

Looks good, thanks @GowthamGoush

This project is cool, looking forward to contributing more! Also, thank you for the kind support


Pluging broken in Windows 11, 22H2

This plugin broke for me after my latest Windows update. I am now on Windows 11, 22H2 (build 22621.3155). The plugin now shows zero desktops, and the arrows don't work. The log says that the error is "GetDesktopManagerInternal".
I have tried versions 1.3, 1.3.2, 1.4.2 and 1.4.3, but the error remains the same. Any idea what this could be?

Microsoft has probably changed the API again. It will take a bit of investigation to find the new details.

I think I have found the new GUIDs for the interfaces.
Could you install this https://github.com/tyranid/oleviewdotnet
And then execute this powershell script?
Win11VirtualDesktopInterfaces.zip
Thanks.

I don't think I'm able to, due to lacking admin rights on my laptop. I am able to run OleViewDotNet.exe from https://github.com/tyranid/oleviewdotnet/releases/tag/v1.11 (not sure if that was the right one, but it ran at least), but trying to run your power shell script gives me an unauthorized access error:
`.\Win11VirtualDesktopInterfaces.ps1 : File cannot be loaded. The file Win11VirtualDesktopInterfaces.ps1 is not digitally signed. You cannot run this script on the current system. For more information about running scripts and setting execution policy, see about_Execution_Policies at https:/go.microsoft.com/fwlink/?LinkID=135170.
At line:1 char:1

.\Win11VirtualDesktopInterfaces.ps1

  + CategoryInfo          : SecurityError: (:) [], PSSecurityException
  + FullyQualifiedErrorId : UnauthorizedAccess`


Fix needham schroeder test + Add service endpoints

Fix disabled Needham-Schroeder test
Add service endpoints
Move Firestore adapter in element-lib
Cleanup fixtures

Codecov Report

Merging #129 into master will increase coverage by 1.37%.
The diff coverage is 92.06%.

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
+ Coverage   31.31%   32.68%   +1.37%
==========================================
  Files         192      191       -1
  Lines        2871     2836      -35
  Branches      196      196
==========================================
+ Hits          899      927      +28
+ Misses       1853     1805      -48
+ Partials      119      104      -15

Impacted Files
Coverage Δ

...b/src/adapters/database/ElementFirestoreAdapter.js
7.14% <ø> (ø)

...ackages/element-lib/src/adapters/database/index.js
100% <100%> (ø)
:arrow_up:

packages/element-lib/src/sidetree/resolve/index.js
95.23% <100%> (+0.04%)
:arrow_up:

packages/element-lib/src/__tests__/test-utils.js
94.56% <91.8%> (-5.44%)
:arrow_down:

...pto/Needham-Schroeder/rsa-jwt-needham-schroeder.js
80.76% <0%> (+66.66%)
:arrow_up:

Continue to review full report at Codecov.

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


gleam: 0.11.2 -> 0.12.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/gleam-lang/gleam/releases.
meta.description for gleam is: "A statically typed language for the Erlang VM"
meta.homepage for gleam is: "https://gleam.run/"
meta.changelog for gleam is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.12.0 with grep in /nix/store/920rrilsvpidg659xhs34l2q7q0b952f-gleam-0.12.0

directory tree listing: https://gist.github.com/85eeecfbe276d026ae56a83802b2f8cd

du listing: https://gist.github.com/ea5c64308b5873ee049f7864b7773be5

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
gleam

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

gleam

Maintainer pings
cc @filalex77 for testing.

Result of nixpkgs-review pr 102342 run on x86_64-darwin 1

1 package built:

gleam


ParticleSubGroup Evaluation/Projection

TODO, WIP
Description
Please include a summary of changes, motivation and context for this PR.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change)
[ ] New feature (non-breaking change which adds functionality)
[ ] API change (breaks downstream workflows and requires major semver bump)
[ ] Requires documentation updates

Testing
Please describe the tests that you ran to verify your changes and provide instructions for reproducibility. Please also list any relevant details for your test configuration.

[ ] Test Foo in /test/path/to/file_for_test_Foo.cpp
Description of test Foo
[ ] Test Bar in /test/path/to/file_for_test_Bar.cpp
Description of test Bar

Test Configuration:

OS:
SYCL implementation:
MPI details:
Hardware:

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any new dependencies are automatically built for users via cmake
[ ] I have used understandable variable names
[ ] I have run clang-format against my *.hpp and *.cpp changes
[ ] I have run cmake-format against my changes to CMakeLists.txt
[ ] I have run black against changes to *.py
[ ] I have made corresponding changes to the documentation
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings

######### DELETE THIS AND BELOW #########
How-to:

Format C++ source code with clang-format

find ./src ./include ./test -iname \*.hpp -o -iname \*.cpp | xargs clang-format -i

Format cmake files with cmake-format

cmake-format -i CMakeLists.txt src/nektar/CMakeLists.txt test/CMakeLists.txt

Format python with black

find ./python -iname \*.py | xargs black

Waiting on https://github.com/ExCALIBUR-NEPTUNE/NESO/pull/256


chore(Textfield): cleanup

Minor cleanup after Textfield work

we'll have to wait for this until Combobox is removed


on-run-end failed, error:  value too long for type character varying(65535)

Hi all! 👋
While running dbt test we got an error saying:
on-run-end failed, error:
 value too long for type character varying(65535)

After digging a bit, I found that the culprit is one of our unit-test for some macros, where we generate the test code using jinja to assert several conditions.
The compiled code ends up in a really long file. I found that your run-end hook calls several macros, but I got a bit lost in there. I guess that one of these macros is trying to upload the compiled code into a VARCHAR or TEXT variable (in Redshift) causing it to overflow (since the max size for a VARCHAR in Redshift is 65535).
Some more details about my env:
I'm running on Redshift using dbt core.
dbt version: 1.7.4
dbt-redshift: 1.7.1
elementary dbt-package: 0.13.0
edr version: 0.13.2

I'm just posting the issue, so I don't want to lead to any solution before is checked that it's not just a singular thing, but an easy solution would be to truncate every string saved to the max size or the TEXT variable based on the adaptor.

Hi @manuel-rodriguez-sonarsource !
Thanks a lot for opening this issue, and very sorry we are only just responding to it.
This is actually a bit surprising since we do truncate this value here, where get_column_size returns exactly 65535 for Redshift.
In any case, since it's been a while I'll go ahead and close this issue but if still relevant please feel free to open a new issue in the main Elementary repo (we are trying to concentrate issue management there).
Thank you!
Itamar

We actually have been running into the same issue. The problem column was test_results_query.
dbt version: 1.7.16
dbt-redshift: 1.7.6
elementary dbt-package: 0.15.2


Export all default components

I know there is already something on the roadmap to release a UI Kit, but I was wondering if it is possible to at least export all the default Docz components. That would already be a great start I think! Right now some components in the components folder can be imported, but others can't.
Right now I needed to create my own custom Table, which I wanted to look just like the Docz table. However, to achieve that, I had to copy that component. This of course can break with each update of Docz. So I'd rather see an option to import it.
Curious how others think about it.

Hey @pedronauck
I was wondering if you've considered this option as part of the new V1 release? I think it would be a great (and easy?) addition.
Thanks!

This is done on v1 @JeroenReumkens  :)

Hey @pedronauck
Thanks for yo awesome work on v1!!
You mean the usecomponents hook? I saw that and already tried that. But that way it isn’t yet possible to use the components themselves inside a mdx file, right?

Sorry @JeroenReumkens, I didn't understand your question. It's possible to use components inside mdx like always, but what specific you want to?

Hey @pedronauck
What I'm trying to achieve is importing the Docz table component in my MDX file, and also importing eg my breakpoint settings, and generate a table in JSX with all of the breakpoints. It would be nice if I could then re-use the Docz table component, so all tables would look the same. And I don't need to manually replicate the Docz table (or override the docz default table with my own table I need to style and maintain).
A simplified example of how I would like to use this in MDX would be:
import {Table} from 'docz';
import breakpoints from '../breakpoints';

    {breakpoints.map(breakpoint => (

{breakpoint.name}	{breakpoint.size}

    ))}

Oh for sure, you can export it from docz-theme-default like you can see:
https://github.com/pedronauck/docz/blob/master/core/docz-theme-default/src/index.tsx#L33

Yes, that's exactly what I was looking for! Thanks a lot for adding this 😄


Historical issues

https://github.com/weaveworks/ignite/issues/979

This issue is happend when use ignited to start vm. Can hannped when:

Use ignited start vm, and then delete the vm manifest in /etc/firecracker/manifests/, ignited will be crashed
Use ignited start vm, and then change the vm manifest spec.status.running to  false, ignited will be crashed

About the first case, when you delete the vm manifest file, ignited daemon will clear the vm status.runtime, but it will use it to remove containerd in StopVM, this cause issue
		if upd.Event == update.ObjectEventDelete {
			// As we know this VM was deleted, it wouldn't show up in a Get() call
			// Construct a temporary VM object for passing to the delete function
			vm = &api.VM{
				TypeMeta:   *upd.APIType.GetTypeMeta(),
				ObjectMeta: *upd.APIType.GetObjectMeta(),
				Status: api.VMStatus{
					Running: true, // TODO: Fix this in StopVM
				},
			}
		}

And about the second cause, when changing the vm manifest file, there will be an update event, but after that, the file will be deleted automatically because the storage received a delete event.
I think this is a bug of github.com/weaveworks/libgitops when it handle the  notify.InMovedFrom event in function:
// If the moveCache isn't cancelled, the move is considered incomplete and this
// method is fired. A complete move consists out of a "from" event and a "to" event,
// if only one is received, the file is moved in/out of a watched directory, which
// is treated as a normal creation/deletion by this method.
func (m *moveCache) incomplete() {
	var event FileEvent

	switch m.event.Event() {
	case notify.InMovedFrom:
		event = FileEventDelete
	case notify.InMovedTo:
		event = FileEventModify
	default:
		// This should never happen
		panic(fmt.Sprintf("moveCache: unrecognized event: %v", m.event.Event()))
	}

	log.Tracef("moveCache: Timer expired for %d, dispatching...", m.cookie())
	m.watcher.sendUpdate(&FileUpdate{event, m.event.Path()})

	// Delete the cache after the timer has fired
	delete(moveCaches, m.cookie())
}

Here is a modify file test using github.com/rjeczalik/notify, you can see there is a InMovedFrom for test.yaml, but no InMovedTo  in this case, it will consider it is a incomplete move, and delete the file.
event is : notify.InDelete /home/charlie.liu/test1/.test.yaml.swp
event is : notify.InCloseWrite /home/charlie.liu/test1/.test.yaml.swx
event is : notify.InDelete /home/charlie.liu/test1/.test.yaml.swx
event is : notify.InCloseWrite /home/charlie.liu/test1/.test.yaml.swp
event is : notify.InMovedTo /home/charlie.liu/test1/test.yaml~
event is : notify.InCloseWrite /home/charlie.liu/test1/4913
event is : notify.InDelete /home/charlie.liu/test1/4913
event is : notify.InMovedFrom /home/charlie.liu/test1/test.yaml
event is : notify.InCloseWrite /home/charlie.liu/test1/test.yaml
event is : notify.InDelete /home/charlie.liu/test1/test.yaml~
event is : notify.InCloseWrite /home/charlie.liu/test1/.test.yaml.swp
event is : notify.InDelete /home/charlie.liu/test1/.test.yaml.swp


why rnn train speed is not stable,sometims very slow?

hi, everyone,
recently,  I train a  cnn+rnn  model for ocr,  but during the training process, i found that the training speed  is not stable, sometimes 30 samples/sec,after a while, it may become 1000 samples/sec...
so i wonder why this happen?  I have tried updating mxnet  from 0.9.5 to  0.10, not solved.

Is 30 samples/sec the speed for a batch? Do you have variable sequence length?

I used mxnet.callback.Speedometer, so yes, I think 30 samples/sec is the mean speed  for a batch,and I fixed the input image size, so it's not variable sequence length;
ps: I tried training two models at the same time(using the same cnn+rnn ) with different gpus on a server,the training  speeds of two process almost the same,and when one training speed become 600 samples/sec, the other training speed also grow to nearly 600 samples/sec, when one training speed slow down,the other training slow down  as well;  And  during the low training speed time, most of cpus is  free, and gpus are  almost free.

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!
Also, do please check out our forum (and Chinese version) for general "how-to" questions.


gluegun error (migration not working)

Hi,
I am currently unable to migrate my project to the new CLI, it is failing on GraphQL api and i've decided for now to not waste precious time on it. I am using the most recent version of aws-amplify (v1.7.8 i believe).
So if i attempt to continue as normal, and i get the following error:
usr/local/lib/node_modules/@aws-amplify/cli/node_modules/gluegun/build/index.js:13
    throw up;
    ^

TypeError: enabledProviders.includes is not a function

How do i resolve this?

You would have to migrate for your project to be compatible with the latest version of the CLI. Closing this issue since we're tracking your migration issue us #1758.


No way to access compilation output files

Command
build, serve
Description
I have an esbuild plugin that is resposible to collect outputFiles after compilation (using onEnd hook with result.outputFiles).
When using application builder (esbuild) with custom plugins there is no way to access full list of outputFiles, only those emitted by main entrypoint compilation. There is no way to access additional files emited by angular builder, such as polyfills, assets, styles etc.
Previously, with using webpack plugin, i was able to get full list of outputFiles by using processAssets compilation hook.
I need to list emited files, as this is only way to get full names with filenames hash.
MY point is to build something like manifest of compilation result output files.
Describe the solution you'd like
I would like to be able to access all compilation files by esbuild plugin.
Describe alternatives you've considered
An alternative could be to access output files as an result of builder output, currently build command only returns {success: true} without information about output files.

This behavior is expected. Unlike with the webpack builder, the application builder uses multiple instances of esbuild to compile various parts of your application.
To obtain the complete list of files, utilize the outputFiles option of the application builder outside of the esbuild context.
for await (const { outputFiles } of buildApplication(...)) {
}

This behavior is expected. Unlike with the webpack builder, the application builder uses multiple instances of esbuild to compile various parts of your application.
To obtain the complete list of files, utilize the outputFiles option of the application builder outside of the esbuild context.
for await (const { outputFiles } of buildApplication(...)) {
}

@alan-agius4
Please reopen this issue, cause your answer is not correct, as described in issue:
currently build command only returns {success: true} without information about output files.

When using buildApplication without watch mode im not able to obtain list of output files.

{success: true} is returned when buildApplication is executed in write mode. I am curious, what is your use case of adding additional files?

@alan-agius4 any update on this?

@alan-agius4 can you please reopen this issue? Problem still exists and there is no final info how to get list of compiled files properly.

I have a separate use case, but it feels like I should comment here instead of posting a new issue.
Background: I'm migrating a project from angular-builders/custom-webpack to angular-builders/custom-esbuild.
In this project, we inline some CSS (loading styles) and JS (error boundary and loading script) into the entrypoint.
The inlined assets are declared in scripts/styles with inject: false in the build options of angular.json.
We have an index transform to insert the compiled inline assets into the entrypoint.
With the webpack builder, we read the files from disk, but this fails with the esbuilder. I think this is because of a pipeline difference:

Webpack: Compile → write → index generation
Esbuild: Compile → index generation → write

To work around this difference, I tried adding a plugin with an onEnd spy to extract the assets, but this didn't work because the asset compilation isn't done in the main build.
For my use case, it would be nice if e.g. globalPlugins/plugin.global could be supported so the spy plugin could be included in all compilation units and access the compiled inline assets[^1].
I'm more than happy to try to work on this as well 🙂
[^1]: Or, if there's a better way to do this, happy to hear!


Missing element in https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/linux-nginx?view=aspnetcore-6.0

If you don't put a symbolic link from the sites-available to sites-enabled it won't work.

Hello @Firesong25 ...
Please re-open this issue using the This page feedback button and form at the bottom of the English-US topic. Use of the This page feedback form adds metadata to your GitHub issue that cross-links the topic and automatically pings the author.


Validate TrustAtom inputs

Feature
Example TA:
Ŧ→sushi[0x00].900000000[0x00]892412523[0x00]uhCEk…UFnFF
Content in this example is sushi
Validations:

value - must be string containing a number
value - must be in the range -1..1 (inclusive)

Fixed in #13


Check app storage locations against Specify for NHMD vascular plants

The name of the collections in the storage tree has been edited in Specify for NHMD Vascular plants. Please check the App DB and Open Refine scripts and update.

[x] Tweak the openRefine script to comply with the new institution name (App version 1)
[x] Create the openRefine script ready for the next App version
[x] UPDATE THE db editions/NHMD storage taxonomy file to reflect Specify change.

All three items are completed and this part of the app is ready for production.

Box collection name is: C-Danish Herbarium
Shelf is: C-CP Danish Vascular Plant Collection

Added to https://github.com/NHMDenmark/Mass-Digitizer/issues/543 instead


Chart Card bug

On my iPad with Unplugged, the Chart Card doesn't show the chart graphic

This is working OK for me now.