Decouple Micronaut Modules from Jackson

[ ] micronaut-gcp
[x] micronaut-problem-json
[ ] ❌ #94
[ ] micronaut-discovery-client
[x] micronaut-rabbitmq
[x] micronaut-security
[ ] micronaut-data
[x] micronaut-mqtt
[x] micronaut-nats
[x] micronaut-liquibase
[x] micronaut-pulsar
[ ] micronaut-aws
[x] micronaut-jms
[x] micronaut-sql
[x] micronaut-graphql
[x] micronaut-kafka

Micronaut GCP

https://github.com/micronaut-projects/micronaut-gcp/blob/ce266bc373eae9fd95d68fd7200696af2c57dbaa/gcp-function-http/src/main/java/io/micronaut/gcp/function/http/jackson/ObjectMapperCustomizer.java
https://github.com/micronaut-projects/micronaut-gcp/blob/ce266bc373eae9fd95d68fd7200696af2c57dbaa/gcp-pubsub/src/main/java/io/micronaut/gcp/pubsub/serdes/JsonPubSubMessageSerDes.java
https://github.com/micronaut-projects/micronaut-gcp/blob/c9f63645a2dde5efe80660760af12f23aff6ff24/gcp-logging/build.gradle#L9

Micronaut Problem JSON

https://github.com/micronaut-projects/micronaut-problem-json/blob/29f4645bf1244c9c3079053778be3e6ab1899fb1/problem-json/src/main/java/io/micronaut/problem/conf/MixInThrowableProblem.java

Micronaut Discovery Client

Significant uses of Jackson DataBind for Consul/Eureka (https://github.com/micronaut-projects/micronaut-discovery-client/blob/3d6c214f8652d9858580ac0e07e98d4563e9c545/discovery-client/src/main/java/io/micronaut/discovery/consul/client/v1/KeyValue.java) probably best to add a dependency on the data bind module

Micronaut RabbitMQ

https://github.com/micronaut-projects/micronaut-rabbitmq/blob/97d228b042c30c71e1665f51f9bfacbf346736ae/rabbitmq/src/main/java/io/micronaut/rabbitmq/serdes/JsonRabbitMessageSerDes.java

Micronaut Security

https://github.com/micronaut-projects/micronaut-security/blob/501868df501c1a52b6c90612fb041b023404961d/security/src/main/java/io/micronaut/security/authentication/jackson/SecurityJacksonModule.java#L36
https://github.com/micronaut-projects/micronaut-security/blob/98e2eb51015291541261ce7b1c2825a433b96ccc/security-jwt/src/main/java/io/micronaut/security/token/jwt/endpoints/KeysController.java
https://github.com/micronaut-projects/micronaut-security/blob/7fb43151fe663707fa769119a8e219169557cf27/security-oauth2/src/main/java/io/micronaut/security/oauth2/endpoint/token/response/OpenIdTokenResponse.java
https://github.com/micronaut-projects/micronaut-security/blob/aade741380a4a50dfc4ab120a6a15469c84b0c37/security-oauth2/src/main/java/io/micronaut/security/oauth2/grants/RefreshTokenGrant.java
https://github.com/micronaut-projects/micronaut-security/blob/aade741380a4a50dfc4ab120a6a15469c84b0c37/security-oauth2/src/main/java/io/micronaut/security/oauth2/grants/SamlBearerAssertionGrant.java
https://github.com/micronaut-projects/micronaut-security/blob/aade741380a4a50dfc4ab120a6a15469c84b0c37/security-oauth2/src/main/java/io/micronaut/security/oauth2/endpoint/token/response/TokenErrorResponse.java
https://github.com/micronaut-projects/micronaut-security/blob/aade741380a4a50dfc4ab120a6a15469c84b0c37/security/src/main/java/io/micronaut/security/endpoints/introspection/IntrospectionResponse.java
https://github.com/micronaut-projects/micronaut-security/blob/aade741380a4a50dfc4ab120a6a15469c84b0c37/security-oauth2/src/main/java/io/micronaut/security/oauth2/grants/ClientCredentialsGrant.java
https://github.com/micronaut-projects/micronaut-security/blob/98e2eb51015291541261ce7b1c2825a433b96ccc/security/src/main/java/io/micronaut/security/endpoints/introspection/IntrospectionController.java
https://github.com/micronaut-projects/micronaut-security/blob/501868df501c1a52b6c90612fb041b023404961d/security-oauth2/src/main/java/io/micronaut/security/oauth2/client/OpenIdClientFactory.java

Micronaut Data

https://github.com/micronaut-projects/micronaut-data/blob/797257aa8d1176df3bf01520f837c68b5a9da29f/data-model/src/main/java/io/micronaut/data/model/Pageable.java
https://github.com/micronaut-projects/micronaut-data/blob/fa62a8f3a4cb39e4d9b936c899816acd80e2016e/data-model/src/main/java/io/micronaut/data/model/Slice.java
https://github.com/micronaut-projects/micronaut-data/blob/91dbafaa59e2bf27e634c3cd536630889efe3e48/data-model/src/main/java/io/micronaut/data/model/DefaultPage.java

Micronaut MQTT

https://github.com/micronaut-projects/micronaut-mqtt/blob/cf4adeb37df9a0abcff6cdaff8aa9cb5c47740c3/mqtt-core/src/main/java/io/micronaut/mqtt/serdes/JsonMqttPayloadSerDes.java

Micronaut NATs

https://github.com/micronaut-projects/micronaut-nats/blob/09dcdf363de4729840087820ff4feb0b24f0e4ee/nats/src/main/java/io/micronaut/nats/serdes/JsonNatsMessageSerDes.java

Micronaut Liquibase

https://github.com/micronaut-projects/micronaut-liquibase/blob/2c2da5ffd41b5f1ea09fcfe9d9791c0134ad8bdd/liquibase/src/main/java/io/micronaut/liquibase/endpoint/RanChangeSetSerializer.java

Micronaut Pulsar

https://github.com/micronaut-projects/micronaut-pulsar/blob/281fba84ddda21b18444fa854b9de4ba44d40de4/pulsar/src/main/java/io/micronaut/pulsar/schemas/json/JacksonJsonWriter.java
https://github.com/micronaut-projects/micronaut-pulsar/blob/281fba84ddda21b18444fa854b9de4ba44d40de4/pulsar/src/main/java/io/micronaut/pulsar/schemas/JsonSchema.java

Micronaut AWS

https://github.com/micronaut-projects/micronaut-aws/blob/222f6025a66500d692ba1de570de8f186993df0d/function-aws-api-proxy/src/main/java/io/micronaut/function/aws/proxy/AbstractLambdaContainerHandler.java

Micronaut JMS

https://github.com/micronaut-projects/micronaut-jms/blob/246d25a9e2d2695a799d1d8a8b85a57b8bcb2f69/jms-core/build.gradle
https://github.com/micronaut-projects/micronaut-jms/blob/72b4a4366145cc1a2b20b0a9cc7cb8041c2772db/jms-core/src/main/java/io/micronaut/jms/serdes/DefaultSerializerDeserializer.java

Micronaut SQL

https://github.com/micronaut-projects/micronaut-sql/blob/5fda28e2392ae3bd6d30420bff76bb0046a2d2cf/jooq/src/main/java/io/micronaut/configuration/jooq/JacksonConverterProvider.java

Micronaut GraphQL

https://github.com/micronaut-projects/micronaut-graphql/blob/1baa6e48b1cf60037478c3e15e207583402e009b/graphql/src/main/java/io/micronaut/configuration/graphql/JacksonGraphQLJsonSerializer.java

Micronaut Kafka

https://github.com/micronaut-projects/micronaut-kafka/blob/66fd3b7248f76b3ee15e5a015e9289c38d3af390/kafka/src/main/java/io/micronaut/configuration/kafka/serde/JsonSerde.java

[ ] https://github.com/micronaut-projects/micronaut-problem-json/blob/29f4645bf1244c9c3079053778be3e6ab1899fb1/problem-json/src/main/java/io/micronaut/problem/conf/MixInThrowableProblem.java : This essentially "adds" an annotation to another class dynamically depending on a config value. We can't modify the target class (ThrowableProblem) because it's in a library. I see a few options:

support mixins in other json mappers

only allow this feature for jackson

create a custom serializer for this type if there's ever a cross-format approach to doing this

wrap ThrowableProblem in another class that hides these fields

The last option seems to be the least intrusive, so I'll take a look if this is feasible.

[ ] micronaut-rabbitmq: still on mn 3.0.3, can't use the json abstractions introduced in 3.1.

@yawkat have to upgrade the module at the same time

micronaut-rabbitmq: https://github.com/micronaut-projects/micronaut-rabbitmq/pull/293
micronaut-mqtt: https://github.com/micronaut-projects/micronaut-mqtt/pull/108

micronaut-nats: https://github.com/micronaut-projects/micronaut-nats/pull/192

micronaut-liquibase: https://github.com/micronaut-projects/micronaut-liquibase/pull/205

micronaut-pulsar: https://github.com/micronaut-projects/micronaut-pulsar/pull/141

micronaut-jms: https://github.com/micronaut-projects/micronaut-jms/pull/188

micronaut-sql: https://github.com/micronaut-projects/micronaut-sql/pull/516

micronaut-graphql: https://github.com/micronaut-projects/micronaut-graphql/pull/223

micronaut-kafka: https://github.com/micronaut-projects/micronaut-kafka/pull/444

micronaut-security: https://github.com/micronaut-projects/micronaut-security/pull/889

Problem has this bug https://github.com/micronaut-projects/micronaut-problem-json/issues/94

Doesn’t seem like a bug


⚠ Bofrost.at has degraded performance

In e071a14, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 903 ms

Resolved: Bofrost.at performance has improved in e360f13 after 6 minutes.


Bug: Trying to display file icon and path=1 or path=2 but lua throwing error.

Self Checks

[x] I'm using the latest lualine.
[x] I didn't find the issue in exsisting issues or prs.

How to reproduce the problem

I am using almost default config and not complicated configuration as below;
-- lualine config
require'lualine'.setup {
      options = {
        icons_enabled = true,
        theme = 'everforest',
        component_separators = { left = '', right = ''},
        section_separators = { left = '', right = ''},
        disabled_filetypes = {},
        always_divide_middle = true,
      },
      sections = {
        lualine_a = {'mode'},
        lualine_b = {'branch', 'diff',
                      {'diagnostics', sources={'nvim_lsp', 'coc'}}},
        lualine_c = {
                        'filename',
                        file_status = true,  -- displays file status (readonly status, modified status)
                        path = 1,            -- 0 = just filename, 1 = relative path, 2 = absolute path
                        shorting_target = 40 -- Shortens path to leave 40 space in the window
                           -- for other components. Terrible name any suggestions?
                      },
        lualine_x = {'filetype'},
        lualine_y = {},
        lualine_z = {'location'}
      },
      inactive_sections = {
        lualine_a = {},
        lualine_b = {},
        lualine_c = {'filename'},
        lualine_x = {'location'},
        lualine_y = {'progress'},
        lualine_z = {}
      },
      tabline = {},
      extensions = {}
    }

Expected behaviour
I expect a relative path and the icon next to this path
Actual behaviour
Getting;
E5108: Error executing lua ...toload/plugged/lualine.nvim/lua/lualine/utils/loader.lua:134: attempt to index local 'component' (a number value)
error.
Minimal config to reproduce the issue

Config
<!--
  =================
  ### IMPORTENT ###
  =================
<p>You need to provide a way to reproduce the issue without influence
of other plugins.
Issues without proper format and a proper way to
reproduce might be ignored.
This script can help you with that.
https://github.com/nvim-lualine/lualine.nvim/blob/master/scripts/nvim_isolated_conf.sh
Create an isolated config directory with:
`nvim_isolated_conf.sh -c DirectoryName`

Then modify the DirectoryName/.config/nvim/init.vim
so you can reproduce the issue .
You can load the config to see if the issue is occuring with
`nvim_isolated_conf.sh -l DirectoryName`

Paste the DirectoryName/.config/init.vim without comments below
-->

Aditional information
Thanks.

It's suppose to be
lualine_c = {
                        {'filename',
                        file_status = true,  -- displays file status (readonly status, modified status)
                        path = 1,            -- 0 = just filename, 1 = relative path, 2 = absolute path
                        shorting_target = 40 -- Shortens path to leave 40 space in the window
                           -- for other components. Terrible name any suggestions?
                      }},

notice the extra {} for options.

Thanks that did the trick.


Cannot publish to crates.io by git+rev dependency of PR #23

In PR #23, git+rev dependency was added to Cargo.toml but it is not acceptable for publishing to crates.io.
$ cargo publish --dry-run
    Updating crates.io index
error: all dependencies must have a version specified when publishing.
dependency `rpm-rs` does not specify a version
Note: The published dependency will use the version from crates.io,
the `git` specification will be removed from the dependency declaration.

It seems that there is no way except for creating and publishing a forked rpm-rs package.
https://doc.rust-lang.org/book/ch14-03-cargo-workspaces.html

If you publish the crates in the workspace to crates.io, each crate in the workspace will need to be published separately. The cargo publish command does not have an --all flag or a -p flag, so you must change to each crate’s directory and run cargo publish on each crate in the workspace to publish the crates.

Does the rpm-rs project not plan to do a release to crates.io themselves?

Have you contacted the owners of the rpm-rs project about this issue?
Could a horrible hack be to temporarily use cargo vendor to pull the unrelease rpm-rs sources (and everything else...) into your project and cargo publish that?


Dataverse support for datasource rules

"Dataverse" is not mentioned in the list of supported datasources (for the rules), so I am wondering what's the story there.
We had somewhat strange experience:

We used to be able to define rules for Dataverse datasource back in the beginning of September
This this feature broke and we had a support case which lasted for about a month
Where we were told Dataverse is not a supported datasource (for the pipeline rules)
However, on the same day, it started to work again (and it works at the moment)

So would it be supported or not?

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

ID: b9b120d7-3f8f-8bc7-2987-21dbf1090ff2
Version Independent ID: 6456d67d-cab6-01e1-3654-4848c5058253
Content: Get started using deployment pipelines, the Power BI Application lifecycle management (ALM) tool - Power BI
Content Source: powerbi-docs/create-reports/deployment-pipelines-get-started.md
Service: powerbi
Sub-service: pbi-deployment-pipeline
GitHub Login: @KesemSharabi
Microsoft Alias: kesharab

Hi @ashlega, thanks for your question! I am forwarding it on to the author of the article for clarification.

Hi @ashlega,
As per the email discussion you're having with the PM regarding this matter, I'm closing this GitHub issue.
Thanks,
Kesem


upgrade FakeItEasy and FakeItEasy.Analyzer to 2.2.0

Prerequisites

[x] I have written a descriptive pull-request title
[x] I have verified that there are no overlapping pull-requests open
[x] I have verified that I am following the Nancy code style guidelines
[x] I have provided test coverage for my change (where applicable)

Description
As it says on the tin.

@adamralph there is a pre-release version now, of FIE, that support core correct? Maybe we should look into using that and get our tests back up on core

@thecodejunkie not quite yet. You can grab it from TeamCity, but that feed isn't reliable. We can and should get a proper pre-release up fairly soon though.

@adamralph cheers for the update.. looking forward to the pre-release! :D

Any ETA?
On 13 September 2016 at 12:45, Andreas Håkansson<EMAIL_ADDRESS>wrote:

@adamralph https://github.com/adamralph cheers for the update.. looking
forward to the pre-release! :D
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NancyFx/Nancy/pull/2560#issuecomment-246655734, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGaptz5zbky0z62sRlaCJHbzqbqYw3tks5qpozXgaJpZM4J6Efe
.

Not yet.
Personally I'm fairly busy this week but I'll try and catch up with the rest of the FIE guys next week. We should be able to push out a beta to myget fairly soon.

@blairconrad I think with your porting work, this one can be closed right?

I didn't even know this existed. Yes. Superseded by my work.
On Mar 12, 2017 17:53, "Andreas Håkansson"<EMAIL_ADDRESS>wrote:

@blairconrad https://github.com/blairconrad I think with your porting
work, this one can be closed right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NancyFx/Nancy/pull/2560#issuecomment-285979855, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADH8FURgL-Qo4HwFEpFwbk4A9dCpJA2Qks5rlGlDgaJpZM4J6Efe
.

Thanks Adam!


There is no delete mechanism for attachments

Refering to:

Gitea version (or commit ref): v1.0.0-126-g302fa42
Feature Request.

There is no delete mechanism at all for attachments.

[ ] The user should be presented with the option to delete any / all attachments from an issue comment.
[x] Attachments should be deleted immediately when an issue / issue comment are deleted.
[x] Attachments should be deleted immediately when a release is deleted
[ ] "Orphaned"Attachments should be deleted periodically (eg: cron job).  Orphaned attachments have both attachment.CommentID and attachment.ReleaseId of 0.  They occur when a user starts creating an issue, uploads an attachment and then closes the browser without submitting.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Also when a release is removed? (since that is now a thing 😄 #673 )

Actually it looks like the code tries to delete an attachment on deleting a comment.
See Here
However it doesn't manage to do so.

Okay so this doesn't trigger the AfterDelete().  I guess that's an xorm bug.  I think for the time being I'm going to work around it and explicitly Delete the attachment in DeleteCommentById

change this to below code, then AfterDelete() will work.
if _, err = sess.Delete(&Comment{
ID: comment.ID,
}); err != nil {
		return err
	}

@lunny I will take it.

@lunny we need to merge this XORM PR first https://github.com/go-xorm/xorm/pull/525

#704 also fix item two.

Attachments should be deleted immediately when an issue / issue comment or release are deleted.

cc @couling

These aren't all fixed.

The user should be presented with the option to delete any / all previously added attachments while editing a release.

@gmastergreatee You are right! In addition to that, you cannot delete and recreate a release because it also deletes the git tag. One should split this.

As an additional issue, the attachments remain on disk even after deleting the release itself (see #2618).

closed by #14130


Fix ignored Realtime.Connection tests

just one of two iOS failed so it's qualified to merge because resto of them (tvOS and macOS tests) passed.


ZETA is not being sent from BSC testnet to any other connected chain

https://testnet.bscscan.com/tx/0x453e87f1ea31f1fd27364bd03a2e789226870fc3b98412e16b983a60f254d1e9
https://testnet.bscscan.com/tx/0xf587eda99e719e0a0f91766ba525eaba8793babe5c0b78609fec0bf0eee49b45
https://testnet.bscscan.com/tx/0x7b6471dea10f57ee3b21199b51cf1f646554d1c66e7640deef0e346a7827f728
These seem like valid calls to the connector, but these txs are not being registered by the protocol.

@andresaiello do you see anything wrong with these txs?

CCTXs were blocked 12 hours ago when the issue was created. It has since been resolved


range over zeros

i'm intending to use this as a free-map for a filesystem.
is there a reason there's no range function to find zeros?
am i holding it wrong?

@aep It seemed a bit strange to me at first, but if you understand the original use case, it kind makes sense.  Here's a function that will range over all the bits if you need it.
// RangeAll iterates over all the bits in the bitmap.
// For the rare instances where bitmap.Range is insufficient
func RangeAll(src bitmap.Bitmap, fn func(k uint32, v bool)) {
	l := uint32(len(src))
	for blkAt := uint32(0); blkAt < l; blkAt++ {
		blk := src[blkAt]
		bit := uint64(0x1)
		for bitAt := uint32(0); bitAt < 64; bitAt++ {
			k := blkAt*64 + bitAt
			v := (blk & bit) != 0
			fn(k, v)
			bit <<= 1
		}
	}
}

This and other functions are now available in https://github.com/chippyash/logicgate


Fullscreen toggle with dblclick does not work when video is showing subtitle

It's because .vjs-text-track-display has pointer-events: none

So it's not the problem to solve here. Should be closed then
https://openload.co/f/4y0Vo3OKduc - here the video with soft subs, fullscreen by dbclick works normal


how to change the model when I switch to other LLM service?

I have set up OPENAI_API_KEY and OPENAI_BASE_URL to other LLM but met error shown as below.

I've added two new environment variables to allow for arbitrary openai compatible resources as described here in the readme.
OPENAI_COMPATIBLE_URL=https://whatever.com/v1
OPENAI_COMPATIBLE_API_KEY=xxx


Not compiling so I am unable to use the template

Hi,
Thanks for sharing this useful CV which I find very compact and well-done. Unfortunately I cannot compile the file and continuously getting the following error:
eso-pic.sty
188
Undefined control sequence.
<argument> \nointerlineskip \AddToHook
                            {shipout/background}{\put (0,\ESO@yoffsetI ){\ESO@H
l.188 }

Please consider I am using TeXworks, just downloaded the latest version and tried both with LuaLaTex and XeLaTex both I get the same erro.
Can you help me?
Thanks

Hi. I just tried TeXworks on my machine and it works perfectly for all compilers. However, this is not surprising since TeXworks is only an editor and under the hood it utilizes the compilers you have installed separately on your system, e.g. via TeXlive or MikTeX (Windows).
Without further information it's hard to tell. My guess is that your LaTeX distribution needs to be updated.
You can use \listfiles in the preamble and then look at the log file.
For me it gives

eso-pic.sty    2020/10/14 v3.0a eso-pic (RN)

Thanks for your quick reply. Sorry if I missed some info. Yes, I am using Windows and I have downloaded the latest bundle of TeXworks (http://www.tug.org/texworks/).  And I am using MikTeX, I have just checked that. Do you mean I should open the Update Manager and update all the packages or reinstall MikTeX from scracth?
Thanks again

Updating should be sufficient if it is indeed a version problem.

I confirm I have reinstalled MikTeX (the latest version) and it works now.
Thank you very much for your help. Appreciated

Reinstalling MikTeX was fine and solved the issue.


Tests fail on Travis with error 'Network request failed'

Found tests fail with error 'Network request failed' on travis with sandbox environment.
Found sandbox return a odd 429 code. It's response header lack access control all origin and X-rate-limit.
Need to disable the test to allow build codes and push to latest branch.
I will still follow up this problem

I will be fixed after this PR merged.


Mod correlation when enabling or disabling

There exists a circumstance that one complete content needs more than one mod to implement. The most common example is campaign. Many campaigns are devided into parts because they are too big. Also some weapons have additional material mod or sound mod. You can enable users to create correlation among mods and sync their status. I just find the problem in the SHUFFLES, for I want to add a weapon to the shuffle, while it has additional sound mod. I can't make two of them binding together.

This has been on my mind for some time now, as i've stumbled into the same situation right as I added shuffles. A grouping feature would be great, but it would require me to rework how I reference mods, and such groupings, in one system, it would also require me to heavily adjust user configs. However, seeing that I'm not the only one wanting such a feature, I'll consider it going forward. Don't expect it super soon though.


Update the DEVELOPER_GUIDE.md to add Gradle Check Metrics Dashboard details

Description
Coming from https://github.com/opensearch-project/OpenSearch/issues/11217#issuecomment-2118322724 and https://github.com/opensearch-project/OpenSearch/issues/11217#issuecomment-2140815607 update the DEVELOPER_GUIDE.md to add Gradle Check Metrics Dashboard details.
Related Issues
Part of https://github.com/opensearch-project/OpenSearch/issues/11217
Check List

[ ] ~New functionality includes testing.~

[ ] ~All tests pass~

[ ] ~New functionality has been documented.~

[ ] ~New functionality has javadoc added~

[ ] ~API changes companion pull request created.~
[ ] ~Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)~
[x] Commits are signed per the DCO using --signoff
[ ]~Commit changes are listed out in CHANGELOG.md file (See: Changelog)~
[ ] ~Public documentation issue/PR created~

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@andrross @prudhvigodithi I think it belongs to CONTRIBUTING.md more, since the DEVELOPER_GUIDE.md is all about local development, and not shared CI infra

Hey @reta the DEVELOPER_GUIDE.md already has information about the flaky tests that has the details mentioning about the CI checks and gradle workflows. The idea of this PR is ensure developers are aware of this OpenSearch Gradle Check Metrics Dashboard to help investigate the flaky failures and resolve them, so more than CONTRIBUTING.md, the DEVELOPER_GUIDE.md seems to be like the right place for this information.

Yeah, I think this works well alongside the "Flaky Tests" subheading in the developer guide. I'm going to merge this as-is, but we can always iterate and improve on the documentation as needed. Thanks!


https://github.com/open-webui/open-webui/pull/5401

I took your work and built a new container. Now I see o1 in the list, but I'm still unable to send requests.

          https://github.com/open-webui/open-webui/pull/5401

Originally posted by @mjbommar in https://github.com/open-webui/open-webui/issues/5402#issuecomment-2349328590

see my comments in #5390.  if you don't have a proxy and use latest frontend, you need to fake the streaming.


Update linux-postinstall.md

In section "Manage Docker as a non-root user" added missing step to restart the docker service after making changes in user group membership.

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Is that in a situation where the docker group didn't exist yet? 🤔

@thaJeztah  sorry for delay in response
we are installing docker on rhel through  rhel-7-server-extras-rpms repo and it does not have docker group. It has dockerroot group available.
# yum info docker
.....
Installed Packages
Name        : docker
Arch        : x86_64
Epoch       : 2
Version     : 1.12.6
Release     : 48.git0fdc778.el7
Size        : 52 M
Repo        : installed
From repo   : rhel-7-server-extras-rpms

# cat /etc/redhat-release
Red Hat Enterprise Linux Server release 7.4 (Maipo)

I see. This means you're actually not installing using the instructions in Docker docs, so I don't think we should incorporate this change.
I think this is something that should be addressed in Red Hat's docs.
@thaJeztah what do you think?


Request: Rebranding

StaCoAn is a hard name to remember and pronounce. It could help to rebrand it to something else, for example 'Stacy'.

I will keep this issue open so others can vote for a namechange.

Hi,
Saw the question a few days ago but I didn't answer at the time.
StaCoAn can indeed prove hard to remember and/or share, however I feel like the name should still be meaningful, which Stacy is not as far as I can see.
SCA could be a straight-forward acronym but is already used for several projects, so it might not be the best idea.
StaCA could be a bit easier to use, as it doesn't require much work to pronounce, isn't widely spread and should be pronounceable in most language with little to no modifications (the 'St' might be a bit hard to say for some, but that's about it).
Or you could decide to have a bit of fun with something like SCeAl (Static CodE Analyser), pronounce 'skill', say 'ea' as in 'tea', write almost as 'seal'. Guaranteed confusion (or you could just say it's pronounced as 'seal' and has one 'c').
On a more serious note, you could very well use a name with no relation whatsoever with 'Static Code Analyser', but I think that you'd better avoid using something that could look like an acronym while it isn't.
I'd personally go for StaCA (short, meaningful, easy to say & to remember, only 6 repositories when searching that name in github and none with that exact name so no risk of confusion, ...).

Thanks everybody for your input. I have to agree that "StaCoAn" is indeed hard to remember and pronounce. But I think the tool came at a stage it is hard to do a namechange. Several blogs and articles have appeared and I think it's just to late to change the name.

Fyi, If you rename a repository on github (setting -> rename), the old URL (https://github.com/vincentcox/StaCoAn) will still exist and redirect you to the new page (e.g. https://github.com/vincentcox/Stacy) so I don't think that it's too late.
StaCoAn can still be the main title but you could add an a.k.a. 'new_name'.


Overhauled the secondary sparse extractor utility.

Codecov Report

Merging #51 (6024565) into master (c48f18e) will decrease coverage by 14.03%.
The diff coverage is 94.93%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##           master      #51       +/-   ##
===========================================
- Coverage   96.04%   82.01%   -14.03%
===========================================
  Files          76       31       -45
  Lines        6897     2825     -4072
  Branches       97       53       -44
===========================================
- Hits         6624     2317     -4307
- Misses        222      480      +258
+ Partials       51       28       -23

Impacted Files
Coverage Δ

...sparse/CompressedSparseSecondaryExtractorBasic.hpp
91.78% <88.88%> (+1.50%)
:arrow_up:

...lude/tatami/base/sparse/CompressedSparseMatrix.hpp
95.75% <100.00%> (+3.00%)
:arrow_up:

.../tatami/base/sparse/SemiCompressedSparseMatrix.hpp
96.12% <100.00%> (-0.10%)
:arrow_down:

...sparse/CompressedSparseSecondaryExtractorBasic.cpp
100.00% <100.00%> (ø)

... and 54 files with indirect coverage changes


Update cobol.json: Minimum mainframe program [uppercase] snippet.

When running "Minimum mainframe program [uppercase] snippet, got the following error:
E5108: Error executing lua ...ker/start/LuaSnip/lua/luasnip/util/parser/ast_parser.lua:339: cannot represent snippet: contains circular dependencies
stack traceback:
        [C]: in function 'assert'
        ...ker/start/LuaSnip/lua/luasnip/util/parser/ast_parser.lua:339: in function 'to_luasnip_nodes'
        ...ck/packer/start/LuaSnip/lua/luasnip/util/parser/init.lua:51: in function 'parse_fn'
        .../packer/start/LuaSnip/lua/luasnip/nodes/snippetProxy.lua:33: in function 'instantiate'
        .../packer/start/LuaSnip/lua/luasnip/nodes/snippetProxy.lua:86: in function '__index'
        .../packer/start/LuaSnip/lua/luasnip/nodes/snippetProxy.lua:99: in function 'copy'
        ...rvim/site/pack/packer/start/LuaSnip/lua/luasnip/init.lua:187: in function 'snip_expand'
        ...e/pack/packer/start/cmp_luasnip/lua/cmp_luasnip/init.lua:143: in function 'execute'
        ...arvim/site/pack/packer/start/nvim-cmp/lua/cmp/source.lua:378: in function 'execute'
        ...narvim/site/pack/packer/start/nvim-cmp/lua/cmp/entry.lua:472: in function 'execute'
        ...unarvim/site/pack/packer/start/nvim-cmp/lua/cmp/core.lua:467: in function <...unarvim/site/pack/packer/start/nvim-cmp/lua/cmp/core.lua:466>
        ...te/pack/packer/start/nvim-cmp/lua/cmp/utils/feedkeys.lua:47: in function <...te/pack/packer/start/nvim-cmp/lua/cmp/utils/feedkeys.lua:45>

Changed this to match the same string for PROGRAM-ID in other snippets.

Can you explain why you think it is a problem as it seems fine to me:

Something is off.
In the fork that I just did yesterday and the code that was installed by Packer the snippet looks like:
"Minimum mainframe program [uppercase]": {
		"prefix": "IDENTIFICATION",
		"body": [
			"       IDENTIFICATION DIVISION.",
			"       PROGRAM-ID. ${1:${1:${TM_FILENAME/(.*)\\..+$/$1/}}}.",
			"       AUTHOR. $2.",

But in the code that I see in your repo, it looks like:
"Minimum mainframe program [uppercase]": {
		"prefix": "IDENTIFICATION",
		"body": [
			"       IDENTIFICATION DIVISION.",
			"       PROGRAM-ID. ${1:${TM_FILENAME/(.*)\\..+$/$1/}}.",
			"       AUTHOR. $2.",

Note the extra {1:$
I'm not sure where that extra parameter is coming from.
But that extra parameter is causing the error that I posted above.
So the patch would make your code "look like your code"???
I guess hold off on the merge for now. I will delete my fork , fork the repo again and see what the code looks like.

Actually, I see you already changed it with b222f1a.
So, go ahead and delete this PR and I will do a Packer update and see if it pulls it down.


Sticky headers are no longer ios-only

Should have removed this in https://github.com/facebook/react-native/commit/77b8c097277b5cf248d08e772ea8bb8d8583e9a1
cc @janicduplessis

👍
@facebook-github-bot shipit


Je alfonso p patch 3 inline code

This PR appears to be a duplicate of #70, so I'm going to close it for now.


JOSS:  Update paper based on #25

Based on the comments of #25

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.353%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3389

Relevant Lines:
5909

💛  - Coveralls


Avoid deprecated methods in Page

While checking the code, I may have found a bug: see this line (which is now this line): shouldn't it be the opposite ('isOriginal' => $b->isAlias() ? 0 : 1)?

Should I resolve conflicts or close this pull request?

Lets resolve conflicts, but like with the other pull request, is there a way we can split this into several smaller pull requests?

I'll create smaller pull requests.


automountServiceAccountToken: false causes crashloop with Kuma DP sidecar

A recent issue #388 and related PR #389 addressed that a K8s Service Account shouldnt be auto mounted unless it was necessary for the pod to talk with the APIs i.e. when Kong Ingress Controller is deployed.
An unintended side effect appears to have been generated when deploying Kong Gateway on K8s (full rather than just ingress controller) hence ingressController.enabled=false AND you want to use Kuma/Kong Mesh sidecar which needs the K8s service account.
Without access this causes a crashloopbackoff in the Kuma DP sidecar
Error: could not read file /var/run/secrets/kubernetes.io/serviceaccount/token: stat /var/run/secrets/kubernetes.io/serviceaccount/token: no such file or directory
I think this will need to be driven off a seperate property rather than hooking with the ingress controller enablement; or a seperate property for identifying when the Kong Helm Chart is being deployed alongside Kuma/Kong Mesh
I have published further detail in Kong Discuss originally asking for some assistance getting to the root cause of the issue
https://discuss.konghq.com/t/kuma-sidecar-missing-serviceaccount-secret-mount-crashloopbackoff-occurs/9131
Any advice or steer on the best way to resolve this will be appreciated.  If any further detail is needed I'm happy to provide.
Rob

Issue #418 and PR #455 look to address this aspect looking through the Issue history the link with Kuma is known but not published.  Looks to be resolved if after this PR I set .Values.deployment.serviceAccount.name to be "default" to mount the default security account.
This is probably me being a helm/k8s newbie - but I'm not sure in the case of spinning up mesh with gateway this will be clear from what's been documented that this will need to be set in order not to fall into the sidecar trap?
Happy to be corrected!
Rob

Create the name of the service account to use */}} {{- define "kong.serviceAccountName" -}} {{- if .Values.ingressController.serviceAccount.create -}}     {{ default (include "kong.fullname" .) .Values.ingressController.serviceAccount.name }} {{- if .Values.deployment.serviceAccount.create -}}     {{ default (include "kong.fullname" .) .Values.deployment.serviceAccount.name }} {{- else -}}     {{ default "default" .Values.ingressController.serviceAccount.name }} {{- end -}}
Does it also need {{ default "default" .Values.deployment.serviceAccount.name }}  in _helpers.tpl so it's more out of the box?


Generic Error Screen & Error Widgets

For any unexpected issue, we can show the Generic Error Screen. We can also add some generic error widgets.

@all-contributors please add @dinurymomshad for architecture and code

@KalleHallden I don't know If this would be too much work, but maybe for issues like this were new UI items are being added it would be good if you provided a design? This way the app stays uniform design-wise and we don't get a bunch of mismatched widgets designed by different people.

@jorre127 yeah that is a good idea. Is there a convention for how to add design suggestions or do I just attach images in issue comments?

@KalleHallden I don't know what programme you use to design, but if you were using Figma you can just add links to the pages here, this way we can see the exact measurements, colours etc. I'm sure other design platforms have something similar.

Okay I'll take a look at that then, I used adobe xd to design it

@KalleHallden I'd suggest Figma because it lets you share links to specific screens. And it's a web app so easier to view.

Yeah I've done some googling and figma seems to be the best candidate


Add SDImageCoderWebImageContext coder option, which allow custom coder plugin, to receive the context option from top-level API

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necessary)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: ...
Pull Request Description
Reason
This is a solution for #2404 . SDWebImageContext contains all the extra information through view category, cache, manager API, but it's not accessable from the custom coder plugin.
It's not surprise because in my mind, a decoder, should only consider about the image & image data, or anything related to the decoding option (Like animation or not, scale factor).
But however, if some custom coder plugin user, who want do some business logic related process, during image decoding. They have no choice to know the context. So I guess, maybe we can open this possibility.
Related PR
Actually, this feature request, is from SDWebImageFLPlugin's PR, which will create a SDWebImageFLCoder, to directlly produce a FLAnimatedImage, to end up that previous hack code that do image decoding during View setImage process.
The image decoding process, should always happend in image coder, but not hack into the setImageBlock. And that PR provide a better solution. And since all the decoding method is called from global queue, this can keep better performance.

Codecov Report

Merging #2405 into 5.x will increase coverage by 0.1%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##              5.x    #2405     +/-   ##
=========================================
+ Coverage   68.72%   68.82%   +0.1%
=========================================
  Files          47       47
  Lines        6634     6649     +15
=========================================
+ Hits         4559     4576     +17
+ Misses       2075     2073      -2

Flag
Coverage Δ

#iOS
69.94% <66.66%> (+0.1%)
:arrow_up:

#macOS
68.67% <66.66%> (+0.06%)
:arrow_up:

Impacted Files
Coverage Δ

SDWebImage/SDImageCacheDefine.m
62.16% <50%> (-3.47%)
:arrow_down:

SDWebImage/SDImageLoader.m
86% <75%> (-1.78%)
:arrow_down:

SDWebImage/SDWebImageDownloader.m
80.82% <0%> (-0.35%)
:arrow_down:

SDWebImage/SDWebImageDownloaderOperation.m
87.62% <0%> (+3.09%)
: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 919751f...ef31239. Read the comment docs.


(scite4autohotkey) Not silent

This popsup:

I guess we should classify this as a bug then.
Maybe a need to use autohotkey script to automate it?

Still relevant I believe, but we need someone familiar with autohotkey to make this happen (IE, not me)

Looks like working now on second try. Probably some intermitient AHK issue as install script could be better.
I will close this as there is no much interest in improving this and it kinda works...
Anyway, it would be much better to switch to portable package as then no AHK installer is needed.


Simplify building when civetweb module is present

It was reported that Civetweb build is too complicated when we're building unrelated apps.
It is already fixed on civetweb master, this PR only bumps the module.
This PR is possibly related to #17996

This PR possibly fixes #17996

Right now, if we merge this PR, #17996 will close. If we are not sure, we should not have that text in the description

@ioannisg you're absolutely right - I updated the description.

While I am fine with this patch, we urgently need to document a set of rules for repos pointed to by the west.yml manifest. I think this should point to our own GitHub org, and not civetweb's. That said, this can wait since we don't have those yet.


hasNumberOfAtoms field showing wrong numbers

Hi,
I am trying to delete an atom in the lattice and writing it to a turtle file. The final hasNumberOfAtoms recorded in the turtle file is incorrect.
Steps to reproduce
from pyscal_rdf import KnowledgeGraph, System
kg = KnowledgeGraph()

# Create Al lattice
struct_Al = System.create.element.Al(graph=kg)

# Write initial data to ttl file
kg.write("initial.ttl", format="ttl")

# Delete one atom
struct_Al.delete(indices=[0])

# Write final data to file
kg.write("final.ttl", format="ttl")

Version
pyscal3: 3.1.7
pyscal-rdf: 0.2.3
Issue
The hasNumberOfAtoms field in initial.ttl is 4, which is correct. However, the hasNumberOfAtoms field in final.ttl is 2, which seems incorrect.
Can you kindly look into this?
Thanks

@NinadBhat Thanks for reporting, this was due to a self.natoms call after deleting the atoms. This will be fixed after merging in #73.

@NinadBhat This should be fixed in v 0.2.6, feel free to reopen if need. Thanks again!


Security Issue: Specify user/password via config file rather than command line

It is a rather large security problem to have user/password be on the command line, since any other user on the same machine can potentially see the login credentials simply with ps.
The user/password should be able to be specified via a config file of some kind.  This also makes using command line functionality easier since the user and password don't have to be specified every time.

I added this ability with the cache functionality in v0.13.1. You should be able to do abodepy --username [username] --password [password] --cache abode.pickle and that will generate a file named 'abode.pickle' which will contain your username, password, and the new UUID requirement. After this you can do abodepy --cache abode.pickle and it'll utilize the cached information.


powershell: fix darwin build

Motivation for this change
ZHF: #122042
cc @NixOS/nixos-release-managers
Hydra failure: https://hydra.nixos.org/build/142939687/nixlog/1
Things done

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

[ ] NixOS
[x] 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 nixpkgs-review --run "nixpkgs-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)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 123484 at af58cf29 run on x86_64-linux 1

2 packages built successfully:

powershell
tests.trivial

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/shells/powershell/default.nix:29:3:
   |
29 |   installPhase =
   |   ^


Heroes' Feast Spell

When you try to use the Heroes' Feast spell, the macro element kicks up an error(I had all the players targeted for the feast). Here is the console error in question. The rest of the DAE works fine.
https://gyazo.com/2e90ec1d93506d579cf850bb814a6098
Thanks.

I cant seem to replicate this on my end, just checking that you have Furnace with advanced macros enabled. If so, could you PM me a list of your modules on Discord?


Add collection id field

I added the collid field as collId. Ref:https://boardgamegeek.com/wiki/page/BGG_XML_API2

Codecov Report

Merging #6 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
+ Coverage   99.06%   99.07%   +<.01%
==========================================
  Files          12       12
  Lines        1072     1077       +5
==========================================
+ Hits         1062     1067       +5
  Misses         10       10

Impacted Files
Coverage Δ

...dGameGeek/Classes/Models/CollectionBoardGame.swift
100% <ø> (ø)
:white_check_mark:

...ses/Models/CollectionBoardGame+Deserializers.swift
100% <100%> (ø)
:white_check_mark:

Example/Tests/CollectionSpec.swift
100% <100%> (ø)
:white_check_mark:

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 8d680c4...7d47739. Read the comment docs.

Looks good! Thanks for submitting!


VersionUtil.cs housekeeping

Move ReleaseType enum from VersionInfo.cs to VersionUtil.cs
Update BRANCH from string to ReleaseType
VersionInfo.cs can now reference VersionUtil.BRANCH to get current build release type
Add ToUpper() extension for ReleaseType -> get uppercase string for in-game display
Add TmpeGuid getter to de-bloat two places its used
Deal with some (but not all) minor code analysis warnings
Update xmldoc for VersionUtil.cs

I also noticed that VersionUtil.CheckGameVersion() appears to be called twice in a normal session (TMPELifecycle.cs: Awake() and Load()) - is that intended?

Ditching this; it would break serialisation


Is it possible to use a sitemap provider that lies outside an MVC project?

Forgive me if the answer to this question already exists in the documentation, but I have not been able to locate it or recognize the answer if I saw it.
I use MvcSitemapProvider (version 4 for .NET MVC 5) to handle my site navigation from a single sitemap file. However, my MVC 5 project has grown to be very large. I would like to split the single application into different solutions because each of the areas really are (mostly) independent of each other.
In other words, I want to refactor my project from this:
Solution "One Mega MVC Project"

MVC 5 App
-- Area 1
-- Area 2
-- Area 3
-- ... more areas
One MvcSitemapProvider sitemap file for all links in this single mvc project

to this:

Solution "Website 1"
-- MVC 5 App #1
-- One MvcSitemapProvider sitemap file with links to this and links in the other projects

Solution "Website 2"
-- MVC 5 App #2
-- One MvcSitemapProvider sitemap file with links to this and links in the other projects

Solution "Website 3"
-- MVC 5 App #3
-- One MvcSitemapProvider sitemap file with links to this and links in the other projects

...More solutions

Having an identical sitemap file per project looks to me like breaking the "do not repeat yourself" rule. Is there a way for me to have only one sitemap file that all the projects read from, so I only need to maintain a single sitemap file?

Thinking about this some more. I could potentially follow this model.
For example, when browser is in a Sitemap View of Website 1, the user sees (in pseudo-code).

Sitemap of Website 1:
@Html.MvcSiteMap().SiteMap()

Other Project Sitemaps
Process the sitemaps from the other projects, pre-pending the correct server/folder names. @Html.MvcSiteMap().SiteMap()

It would be better if they could see the union of both link sets, so that they same order of links could be presented.

Closing this issue as the Nuget package is abandoned.


Add deprecation badge to Cargo.toml

it's kinda not? but just in case

DO NOT MERGE till all deprecation changes are ready


remove old script that is not used any more

I find it confusing that the packaging.sh script (from ACS) is still here, while one is supposed to use package_cosmic.sh. If I'm missing something, let me know.

It wasn't cleaned up, yet!


Can CDC server work with Oracle Streams instead of Golden Gate?

Gavin has done the investigation on the Oracle Streams and the result is negative. Details are documented in this link if you are interested.
https://www.networknt.com/faq/oracle-streams/

Oracle Streams can capture database change and creating a message that contains the information and enqueuing the message into a queue. So technically, it should be fine to work as CDC for the eventuate system. We will provide the detail implementation for it soon.

Given above comment, the issue is closed.


Swapped YouTube Video

Was the first lecture, but I propose the Ethereum Meetup talk.

No longer relevant


All of VS Code is very slow

Issue Type: Performance Issue
I've been using VS Code for awhile now and after updating and I think slightly before, vs code has been responding very slow to everything I try to do. I thought it was from the extensions I had but I uninstalled all of them and vs code isn't any faster than before. I don't know what it could be.
VS Code version: Code 1.67.0 (57fd6d0195bb9b9d1b49f6da5db789060795de47, 2022-05-04T12:06:02.889Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-1035G1 CPU @ 1.00GHz (8 x 1190)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.75GB (2.76GB free)

Process Argv
--crash-reporter-id 4ee18ccc-4377-4d91-ad8d-5011d69cbb31

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    1	   129	  5112	code main
    0	   110	  3640	   shared-process
    0	    78	  7724	     fileWatcher
    0	    74	  9596	     ptyHost
    0	    40	  6852	   utility-network-service
    0	    88	  7764	   issue-reporter
    0	    26	  9380	   crashpad-handler
    0	   109	 10000	   extensionHost
    0	   152	 10692	   window (Visual Studio Code)
    0	   144	 12448	   gpu-process
    0	    86	 15852	   utility

Workspace Info
;

Extensions: none
A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
cppdebug:30475216
vsclangdf:30486550

Unfortunately the vscode is like that because it is written in electron. A solution for weaker hardware may be e.g. https://gitpod.io

Never mind thanks for the help, for some reason vs code wouldn't update and now it allowed me to? Idk what happened but thanks!


Navigation approach

I'm working on making navigation and form post work in jsdom, so I can upstream things instead of adding them to zombie.js.  You can see what I've done so far in #1902.
The next step is to implement the hook for loading pages that get navigated to, and including them in the history.  What this means is that the JSDOM object may ultimately get new window objects, as the navigation occurs.  I think this is the simplest approach.  My suggestion is that there should also be a hook onNewWindow that is called when a link is opened in a new tab.  This hook will be given a reference to a separate JSDOM instance.
What this means is that JSDOM changes from representing a single loaded window, to being equivalent to a single tab within a modern, tabbed browser.  I don't think this notion of history can be simply made external, while keeping jsdom's implementation of the history and hash-change APIs.

Amazing. This is exactly what we've dreamed of. Having the JSDOM object change what it represents makes perfect sense to me.
The ideal solution would be to implement the WindowProxy so that the window is === itself before and after navigation. I think that can be layered on top though, later.
Can you say more about what the onNewWindow hook would be useful for?

If you imagine a implementing a full tabbed browser in JavaScript, by rendering a JSDOM dom to canvas or something: onNewWindow means that a new tab should be opened, displaying the contents of the new dom.  e.g. if you rendered the following page using jsdom:
Clicking me opens a new tab

I've started looking at this, but the spec is really hard to follow because there doesn't seem to be a class in jsdom called "Browsing Context".  I think currently all that functionality is tangled up with the window object.  Navigating across pages will mean multiple windows per browsing context, so I think it needs to be a separate object.
Is the right approach to make this a class and put it in lib/jsdom/browser/BrowsingContext?  I can try and create it according to https://html.spec.whatwg.org/#windows but I'm not 100% clear what each step of each algorithm involves.

That sounds right, yeah. The interesting part will be hooking it up to iframes. Also note that browsing contexts are 1:1 with WindowProxy objects.
And yeah, the spec is known to be pretty confusing. Anne and I are working to fix it but it's a long-term project. I'm at breakfast right now but will be in #jsdom IRC later today if you want more real-time discussion.

The biggest advice I think will be working on writing tests for observable differences, either from the outside or as web platform tests.

Also note that browsing contexts are 1:1 with WindowProxy objects.

That's useful to know.  I guess in that case it's about separating out a clear distinction between WindowProxy and Window object.  I'm not sure that's made very clear yet.

Right, jsdom does not have any WindowProxy implementation at all right now, just Window. WindowProxy isn't too hard to implement, I think, at least in the basics. I am a little worried about the performance consequences though, so I'd want to hold off on implementing it until we had some larger-scale benchmarks in place mimicking our biggest consumers (e.g. ReSpec and Ecmarkup).


Respect proxy CPU/memory annotations in proxy init

There is no reason to have the init container different than the
sidecar. I don't see any functional reason to make the requests bigger
than the sidecar, and there is no need to make it smaller as we will
always fit into quotas/limitrange adequately as long as we are equal to
sidecar.
For https://github.com/istio/istio/issues/26932
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
943f98a0bdf5c69b9740d9dd4dbfaa19e6fc3780
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Fix bbl executable name

was renamed, see assets of https://github.com/cloudfoundry/bosh-bootloader/releases/tag/v9.0.7

What is this change about?
Fix decota Docker image build.
Please provide contextual information.
Build is broken because bbl executable has been renamed: https://concourse.wg-ard.ci.cloudfoundry.org/teams/main/pipelines/cf-deployment-concourse-tasks/jobs/build-docker-image/builds/140
Please check all that apply for this PR:

[ ] introduces a new task
[ ] changes an existing task
[x] changes the Dockerfile
[ ] introduces a breaking change (other users will need to make manual changes when this is released)

Did you update the README as appropriate for this change?

[ ] YES
[x] N/A

How should this change be described in release notes?
Not relevant.
What is the level of urgency for publishing this change?

[ ] Urgent - unblocks current or future work
[x] Slightly Less than Urgent

Tag your pair, your PM, and/or team!
It's helpful to tag a few other folks on your team or your team alias in case we need to follow up later.

Already fixed with https://github.com/cloudfoundry/cf-deployment-concourse-tasks/pull/157


Create examples for making fine grained assertions

Problem
The conformance tests are not enough, albeit a great starting point.
Per cian on ibc-go team, they struggled with failing tests using the Interchain API. To be clear, the API worked well, but there's missed nuance in when to wait for blocks or other state.

I don't want to test all of these all the time, but the things I think would be useful are creating arbitrary wallets, being able to assert things are in escrow at certain times and being able to make sure that the correct wallet was refunded the correct amount etc.

I think that maybe some of these tests could belong in the conformance test suite, but I think there are quite a few fine grained assertions that we want to be able to make on the state of certain wallets/chains etc.

I'm thinking an examples directory as common with other projects. In this case, the examples will be tests that probably run on CI which ensures we keep the examples up to date and accurate.

My vote is we close this issue. The pattern is established!


render4k switch

This switch functionality was lost in previous revisions. need to add it back.

Fixed it for DX12 GDK version,


ListTile title overflow issue

I'm displaying a long text inside ListTile's title using the following code and I'm having UI issue with the Text overflow feature:
ListTile(
  contentPadding: const EdgeInsets.symmetric(horizontal: 32.0),
  leading: SizedBox(
    width: 32.0,
    child: Center(
      child: Container(
        width: 32.0,
        height: 32.0,
        decoration: BoxDecoration(
          shape: BoxShape.circle,
          color: HexColor.fromHex(_clientStatusList.first.color)
              .withOpacity(0.2),
          border: Border.all(
              color: HexColor.fromHex(_clientStatusList.first.color)),
        ),
        child: Center(
            child: Icon(
          CustomIcons.customer_important,
          size: 14.0,
          color: HexColor.fromHex(_clientStatusList.first.color),
        )),
      ),
    ),
  ),
  title: Text(
    _clientStatusList.map((e) => e.name).join(", "),
    maxLines: 1,
    overflow: TextOverflow.ellipsis,
  ),
  subtitle: Text("Status"),
),

Result from the above code:

Result without maxLines and overflow:

I guess that the expected result is:

Flutter doctor:
[✓] Flutter (Channel stable, v1.17.2, on Mac OS X 10.15.5 19F96, locale en-IL)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
[✓] Android Studio (version 4.0)
[✓] VS Code (version 1.45.1)
[✓] Connected device (1 available)

Hi @CripyIce
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Why it does that
Briefly state why the change was necessary.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes # <-- insert Issue number if one exists
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[ ] added relevant details to the changelog? (skip with label no-changelog)
[ ] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Codecov Report

Merging #220 (943c615) into main (59aefa1) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #220   +/-   ##
=========================================
  Coverage     62.63%   62.63%
  Complexity     2973     2973
=========================================
  Files           694      694
  Lines         15420    15420
  Branches       1043     1043
=========================================
+ Hits           9658     9659    +1
  Misses         5332     5332
+ Partials        430      429    -1

Impacted Files
Coverage Δ

.../dataspaceconnector/boot/util/TopologicalSort.java
95.23% <0.00%> (+2.38%)
: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 59aefa1...943c615. Read the comment docs.


[CPU] Enable dummy cache-level tiling for Mmt4dTilingExpert.

The cache-level tile sizes are as same as distribution. In this context, it is dummy because tiling does not happen at all. The revision moves the pipeline to use TilingConfig correctly.
It is a step towards to https://github.com/openxla/iree/issues/16025

I was too greedy because I wanted to move it to multi-level tiling and tune sizes afterward. This is just a noise to everyone, so I'm going to move it smoothly. The top goal to me is making mmt4d fusion work; I will rework on it.

Let me know if you are still working on this as this overlaps with what I'm doing as well :)

Let me know if you are still working on this as this overlaps with what I'm doing as well :)

no, I'm not working on this.


Documentation updates

The first thing I suggest is moving documentation from GitHub Wiki to the website:

It's properly versioned
It's easier to modify it for contributors
It can be edited locally, with a proper code editor
It would have more user-friendly interface
We would be able to add some interactive features, like embedding playground to docs (see https://github.com/microsoft/TypeScript-Website/issues/130 for more examples)

One thing that has to be decided is a static site generator we're going to use. I've checked out most of popular JS generator, and I think the best ones we can choose would be either Gatsby or Docusaurus. Docusaurus supports more things out of the box, but Gatsby has a rich plugin system, making it easier to customize.
Another thing I propose is moving website/documentation to this repo. That's a more controversial change (though that's a quite common practice), and it has it's tradeoffs, but there are a bunch of advantages as well:

It makes it even more easier to modify it, because adding a feature and documenting it can be done in one PR
It's much easier to do playground deployment previews for PRs, which is very useful for previewing changes without local checkout (we would be able to do it like Prettier does, otherwise we'll have to write a bot to do that)
It has full access to raw code, so it would be easier to generate API documentation from JSDoc

I think it makes sense to look at alternatives to the wiki. Having docs in this repo makes sense. We still want to deploy to typescript-to-lua.github.io though, which has to be done through the GitHub pages of the other repository. How are you planning on doing the deployment? I would VERY much prefer if we could restrict the docs in this repo to just markdown.


fix: vue parser

Closes #1281
The current logic:

usesString.indexOf to find the opening and closing template tag.
Is updating the tag attributes by locating indexes of the props from codestring

Problems.

Using String.indexOf means when there's nested templates, the first closing template tag is taken instead of the outermost one.
While we're updating the tag attributes, the code string gets mutated, this would mean we'll be searching for wrong locations in the string.

Solutions:

Instead of finding the template position with String methods or Regex, we can take advantage of the vue compiler which can directly provide us with the template content.
Instead of finding tag attributes/props with index locations, we can just find their exact string instead and replace it with the new format.

@anubra266 Can you include a test for the nested template that shows that this works?

Done @segunadebayo


Is Dockerfile.dockerignore supported?

I've searched the docs and glanced at the code and found nothing, so I guess the answer is no, however since this is a likely blocker for me I thought I'd ask explicitly.
Background: https://github.com/moby/buildkit/pull/901, https://github.com/GoogleContainerTools/kaniko/issues/776.

Hi! Werf supports .dockerignore.
https://github.com/werf/werf/blob/master/pkg/giterminism_manager/file_reader/dockerfile.go#L58

Thanks @distorhead. Too bad. Any plans of supporting Dockerfile.dockerignore too in the future?

@bard
The functionality will be available in the next release (https://github.com/werf/werf/pull/3516).

Available since https://github.com/werf/werf/releases/tag/v1.2.10%2Bfix25.


Upgrade Substrait and generate Substrait plan on driver

[x] Upgrade substrait to version e1b4c04a1b
[x] Generate substrait plan on driver
[x]  Add TPCH test data and queries

[ ] Update Substrait Java API


[Event Request] Table 36 Sales Header - OnHandleSupplementTypeOnRecreateSalesLines

Hi Team,
In Table 36 we need possibility to have our additional checks and/or skip standard code. Having event with handling in procedure RecreateSalesLines would be a perfect solution :

[IntegrationEvent(false, false)]
local procedure OnHandleSupplementTypeOnRecreateSalesLines(var TempSalesLine: Record "Sales Line" temporary; var IsHandled: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


chore: upgrade to gRPC-1.24.3

We need this version because openSUSE/Tumbleweed does not compile with
older versions of gRPC.

This change is

I tried to update all the builds to gRPC-1.24.3, but on some (generally older) distributions that would not compile, so I left those alone.


Server crash when VS install with AsmodeusCore

Tested with:
valkyrien-skies-1.1.3
vs-control-1.1.0
vs-world-1.0.0
forge 2847
AsmodeusCore 0.0.23 and 0.0.19
logs:
https://paste.ubuntu.com/p/YXxmdFzBqn/
crash report:
https://paste.ubuntu.com/p/fHjdmR3tF4/

oh sorry, the problem is not vs

oh sorry, the problem is not vs


Some references not displaying

Some disease gene annotation  have references in Artemis, but these are not displaying
Aco1 check ref disease associated, cerebellar atrophy; db_xref=PMID:25351951
Sec231  disease associated, Cowden syndrome; db_xref=PMID:26522472; date=20151103"
Trm82  disease associated, microcephalic primordial dwarfism; db_xref=PMID:26416026;
Trm82  disease associated, Galloway-Mowat syndrome; db_xref=PMID:30079490;

I can't see those references in the contig files.  Could you paste the lines into this issue?

AH it's OK. These were in my notes. I think I noted that I deleted them, but I would insert the reference into the  flatfile. TAking this one


Document how to use WaitCompletionPacket

The only efficient way to wait for a large number of waitable objects is to use the undocumented functions NtCreateWaitCompletionPacket, NtAssociateWaitCompletionPacket, and NtCancelWaitCompletionPacket.  These functions are used internally by the Windows thread pool implementation, but are not documented for use in user code.  The result is that event loops that are based on I/O Completion Ports do not support waitable objects.

Hi @DemiMarie, thank you for contacting us.  I just searched both the WDK and SDK headers (from the Win11 kits) and did not see the routines you list.  Our general rule is if it's not in a header that we ship, we don't document it.
You could try the forums for further help -- are you writing an app or a driver?

Hi @DemiMarie, thank you for contacting us. I just searched both the WDK and SDK headers (from the Win11 kits) and did not see the routines you list. Our general rule is if it's not in a header that we ship, we don't document it.
You could try the forums for further help -- are you writing an app or a driver?

I want to be able to wait for various waitable objects in a userspace application that uses I/O Completion Ports.  Using another thread is inefficient.

I'd suggest using the user-mode thread pool APIs. You can find more info at https://docs.microsoft.com/en-us/windows/win32/procthread/thread-pool-api.
Thanks for reaching out and best of luck!

Sadly the thread pool APIs are not practical to integrate into existing event loops.  WaitCompletionPacket, on the other hand, works fantastically in such event loops, and most portable applications do have their own event loops.


Plotting trees with TreeRecipe.jl

The following example shows how to manually plot the trees learned in DecisionTree.jl:
https://github.com/JuliaAI/TreeRecipe.jl/blob/master/examples/DecisionTree_iris.jl
Currently, the way to integrate a plot recipe in MLJ.jl is not documented, but is sketched in this comment.
So, can we somehow put this together to arrange that a workflow like this generates a plot of a decision tree?
edited again (x2):
using MLJBase
using Plots                 # <---- added in edit
import MLJDecisionTreeInterface
tree = MLJDecisionTreeInterface.DecisionTreeClassifier()
X, y = @load_iris
mach = machine(tree, X, y) |> fit!
plot(mach, 0.8, 0.7; size = (1400,600)))   # <---- added in edit

Note: It used to be that you made RecipesBase.jl your dependency, to avoid a full Plots.jl dependency. But now the recipes live in Plots.jl and you are expected to make Plots.jl a weak dependency. You can see an example of this here.

I've looked into this a bit further. I have an idea how to do it but it's a bit involved. The first step is to replace the current fitresult output of the fit methods for DecisionTreeClassifier and DecisionTreeRegressor models with wrapped versions. We need this because we are going to overload Plots.plot(fitresult, ...) for appropriate fitresult types.
So, we create a new struct
struct DecisionTreeClassifierFitResult{T,C,I}
    tree::T
    classes_seen::C
    integers_seen::I
    features::Vector{Symbol}
end

and instead of fit(::DecisionTreeClassifier, ...) returning fitresult = (tree, classes_seen, integers_seen, features) we will return
DecisionTreeClassifierFitResult(tree, classes_seen, integers_seen, features).
We will have to modify the definition of predict(::DecisionTreeClassifier, fitresult,...), fitted_parameters(::DecisionTreeClassifier, fitresult) and feature_importances(::TreeModel, ...) accordingly, so that they first unwrap the fitresult.
We do something similar for DecisionTreeRegressor, whose fitresult has a different form.
We should be careful that none of these changes breaks anything. Since fitresult is private (public access is through the fitted_params method we are fixing) this should not be a problem.
@adarshpalaskar1 You want to have a go at a PR to do this internal wrapping?

Sorry for the delayed response. I went through the implementation steps you provided, and I am eager to work on a PR.

I added the above mentioned changes for the DecisionTreeClassifier, but I am facing an issue while plotting:
In the above mentioned example, (https://github.com/JuliaAI/TreeRecipe.jl/blob/master/examples/DecisionTree_iris.jl), we have:
julia> typeof(dtree)
Node{Float64, String}

wrapped tree in example:
julia> typeof(wt)
InfoNode{Float64, String}

Where as in case of DecisionTreeInterface, we have:
julia> typeof(fitted_params(mach).raw_tree.node)
DecisionTree.Node{Float64, UInt32}

wrapped tree in fitted_params:
julia> typeof(fitted_params(mach).tree)
DecisionTree.InfoNode{Float64, UInt32}

Due to which we get:
julia> plot(fitted_params(mach).tree.node, 0.8, 0.7; size = (1400,600))
ERROR: Cannot convert DecisionTree.Node{Float64, UInt32} to series data for plotting

How can I handle this mismatch in the datatypes?


Special diagnostics & logging

Hi!
As part of DSC apply/verify monitoring process I would like to know EXACTLY, which DSC resource INSTANCE (given by a unique name) is currently processed. So far the ONLY way I found to achieve this, is to enable/read the Microsoft-Windows-Desired State Configuration/Analytic log with EventID equals to 4162 which fulfills the above requirement.
Example log, where we can see which DSC resource instance is beeing processed:

Via PowerShell it's easy to read:
Get-WinEvent -FilterHashtable @{ LogName = 'Microsoft-Windows-DSC/Analytic'; Id = 4162 } -Oldest
I also figured out, how to detect the current working DSC job GUID at runtime:
$JobGUID = [Microsoft.PowerShell.DesiredStateConfiguration.Internal.S_DscCoreR]::JobGuidString
But the problem is that I don't want to use event logs at all for diagnostics.
My idea is to do this kind of logging directly at DSC apply/verify runtime form inside the custom DSC resource.
Example (one instance of the DSC resource in a DSC configuration):
MyCustomResource "MyInstanceName" {
	Property1 = "value1";
	Property2 = 5;
	Property3 = @("value3","value4")
}

Then from the Test( ) method of MyCustomResource I would like to get somehow the instance name: “[MyCustomResource]MyInstanceName”
Something like:
[bool] Test() {
	$InstanceID = ... # the value of $InstanceID should be  "[MyCustomResource]MyInstanceName"
	return $result
}

Basically the question is, how to programatically get the instance name (in the above example the string
"[MyCustomResource]MyInstanceName") from the custom resource's Set( ) or Test( ) method?
What I tried so far:
Using Enable-DscDebug to debug my custom dsc resource and when stopped at whatever breakpoint, I tried to check all possible PowerShell (automatic) variables. Unfortunately none of the holds the value I am looking for.
BR,
Ladislav

Not sure that is possible since LCM has data and LCM does not pass that to the actual resources that are ran. You can possible get the resource name but not what the instance was named in the DSC configuration. You can try asking in the Slack/Discord #DSC channel https://dsccommunity.org/community/contact/

@johlju
Finally I found the solution to my problem. It is described in this article at the section Reading the active log file.


error

when i use the pre-train model,have an error:Exception encountered when calling layer "decoder__transformer" "                 f"(type Decoder_Transformer).

Because of some mistakes, pre_train model weights does not match the network. Please training your own model weights


API-1835: track specific failures from control loops

this allows for precise tracking of what failed in a control loop.

/cherry-pick release-4.18


Made system binaries (ls, readlink, etc.) discovery more generalized.

READLINK, LS, SYSTEM_PERL are now set via call to 'which' (which is set manually).

Thanks for your patch, good idea. This way, a different system Perl can be used.
By the way, SYSTEM_PERL gets assigned a wrong value if "which" is called before the path is cleared, but it doesn't have a negative effect as long as USER_PERL is set correctly. And it looks like this script needs some improvements anyway...

Glad to be helpful:) And your script saved me a good amount of time recently.

I didn't like how some variables were leaking and that the script sources itself, so I've reorganized it and made some ugly parts slightly less ugly.
As suggested, the script uses which to find the system Perl, but it does that only once now. So if your system Perl is in a different location, it should still work for you.
I've also added a note on "path" symlinks that can be used to prepend directories with Perl-related binaries to the $PATH.
I hope the updated script fits your needs.


Update Clarinet docs to include the new devnet package

Depends on https://github.com/hirosystems/clarinet/pull/1116.


HTTPErrors when accessing & downloading artifacts from wandb.Api()

wandb --version && python --version && uname

Weights and Biases version: 0.9.2
Python version: 3.6.10
Operating System: Windows 10

Description
I'm trying to download an artifact using Api().

I get an error "Bad Request for url: https://api.wandb.ai/graphql" upon getting artifact object when using API without parameters. This problem seems to be similar to https://github.com/wandb/client/issues/1111 (Note that I use wandb 0.9.2 where the https://github.com/wandb/client/issues/1111 issue is already fixed, as I understood)

api = wandb.Api()
artifact = api.artifact(name='shirts-artifacts-resume_2uhu1z1m_checkpoint:latest')
print('Downloading')
path = artifact.download('./wandb/test-dwld')
print(path)

Traceback (most recent call last):
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\apis\__init__.py", line 72, in wrapper
    return func(*args, **kwargs)
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\apis\public.py", line 454, in artifact
    return Artifact(self.client, entity, project, artifact_name)
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\apis\public.py", line 1959, in __init__
    self._load()
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\apis\public.py", line 2165, in _load
    'name': self.artifact_name,
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\retry.py", line 130, in wrapped_fn
    return retrier(*args, **kargs)
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\retry.py", line 95, in __call__
    result = self._call_fn(*args, **kwargs)
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\apis\public.py", line 151, in execute
    return self._client.execute(*args, **kwargs)
  File "C:\ProgramData\Anaconda3\lib\site-packages\gql\client.py", line 52, in execute
    result = self._get_result(document, *args, **kwargs)
  File "C:\ProgramData\Anaconda3\lib\site-packages\gql\client.py", line 60, in _get_result
    return self.transport.execute(document, *args, **kwargs)
  File "C:\ProgramData\Anaconda3\lib\site-packages\gql\transport\requests.py", line 39, in execute
    request.raise_for_status()
  File "C:\ProgramData\Anaconda3\lib\site-packages\requests\models.py", line 941, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://api.wandb.ai/graphql

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "d:\MyProject\tmp.py", line 7, in <module>
    artifact = api.artifact(name='shirts-artifacts-resume_2uhu1z1m_checkpoint:latest')
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\apis\__init__.py", line 74, in wrapper
    raise CommError(err.response, err)
wandb.apis.CommError: <Response [400]>

I tried to find a workaround to the above pproblem. The error goes away when I initialize Api() object with entity, project & run_id info, but then I get errors on downloading:

api = wb.Api({'entity': 'maria_korosteleva', 'project': 'Test-Garments-Reconstruction', 'run_id': '2uhu1z1m'})

artifact = api.artifact(name='shirts-artifacts-resume_2uhu1z1m_checkpoint:latest')
print('Downloading')
path = artifact.download('./wandb/test-dwld')
print(path)

Error:
Downloading
Traceback (most recent call last):
  File "d:\MyDocs\GigaKorea\Garment Pattern Estimation\tmp.py", line 12, in <module>
    path = artifact.download('./wandb/test-dwld')
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\apis\public.py", line 2085, in download
    pool.map(download_file, manifest.entries)
  File "C:\ProgramData\Anaconda3\lib\multiprocessing\pool.py", line 266, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "C:\ProgramData\Anaconda3\lib\multiprocessing\pool.py", line 644, in get
    raise self._value
  File "C:\ProgramData\Anaconda3\lib\multiprocessing\pool.py", line 119, in worker
    result = (True, func(*args, **kwds))
  File "C:\ProgramData\Anaconda3\lib\multiprocessing\pool.py", line 44, in mapstar
    return list(map(*args))
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\apis\public.py", line 2075, in download_file
    cache_path = self.get_path(name).download()
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\apis\public.py", line 2033, in download
    return storage_policy.load_file(self, name, manifest.entries[name])
  File "C:\ProgramData\Anaconda3\lib\site-packages\wandb\artifacts.py", line 442, in load_file
    response.raise_for_status()
  File "C:\ProgramData\Anaconda3\lib\site-packages\requests\models.py", line 941, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://api.wandb.ai/artifacts/None/ed5f7b1c16414b31e72706db1aef3cca

Note that if I change 'None' in the final URL to my wandb nickname, I'm able to download the file through the browser.
I place both problems into one issue because I have a feeling they are related. Let me know if I'm wrong and should separate them.

Thank you very much for reporting! We'll try to recreate.

We should have a fix in the next release.  The temporary workaround is to include your username in the artifact path. i.e.:
artifact = api.artifact(name='username/project/shirts-artifacts-resume_2uhu1z1m_checkpoint:latest')

Hi @vanpelt, thanks for the tip.
I've tried your suggestion and just wanted to let you know that I still get the 404 error when I do artifact.download('./wandb/test-dwld'): the URL still has None instead of the username. Just in case, I tried with both ways I initialize wandb.API() object  -- with or without parameters -- error is the same.

Hey @maria-korosteleva with the release of wandb 0.9.3 are you still seeing this issue?

@vanpelt the problems are gone in the new release. Thank you very much!


Bugs in multiple files

Hi,
this is not an issue. I'm trying to understand how the interprocedural part works... When infer analyzes a file, it tries to analyze all the methods, even the ones in other files that are called in the file under analyzation? So, if inside of a function a method from another file is called, the tool checks the function from the 2nd file? Now, imagine that the function from the 2nd file is a method that allocates memory with malloc and returns the created pointer... If this pointer is not freed on the first file, infer will detect a memory leak? I added an example...

file1.c

int method1() {
  int *ptr = method2();
  /* do stuff */
  /* If I don't free(ptr), infer gets a Memory Leak?*/
  return 1;
}

file2.c

int *method2() {
  int *ptr = (int*) malloc(100 * sizeof(int));
  return ptr;
}

I also would like to know if infer only checks memory leaks for malloc or if also does it to calloc. Thank you! :-)

Yes, Infer should report the memory leak in your example. Does it work if you try on a small example?
Infer will also treat calloc and malloc in the same way: https://github.com/facebook/infer/blob/master/infer/models/c/src/libc_basic.c#L812

Hi,
I haven't tried with a small example. But I'm running a few samples of memory and resource leaks collected from several github repositories. And one of them is this one,
         playlist = playlist_init(lpl_path, 99999);
         {
            for (j = 0; j < playlist->size; j++)
            {
               if (string_is_equal(playlist->entries[j].crc32, state->crc))
               {
                  printf("CRC Match %s\n", state->crc);
                  strlcpy(state->path, playlist->entries[j].path, sizeof(state->path));
                  state->found = true;
                  task_set_data(task, state);
                  task_set_progress(task, 100);
                  task_set_title(task, strdup("Compatible content found"));
                  task_set_finished(task, true);
                  string_list_free(state->lpl_list);
                  return;
               }
               task_set_progress(task, (int)(j/playlist->size*100.0));
            }
          }

where playlist_init is the initialization of the playlist with calloc in other file:
playlist_t *playlist_init(const char *path, size_t size)
{
   struct playlist_entry *entries = NULL;
   playlist_t           *playlist = (playlist_t*)calloc(1, sizeof(*playlist));
   if (!playlist)
      return NULL;

   entries = (struct playlist_entry*)calloc(size, sizeof(*entries));
   if (!entries)
   {
      free(playlist);
      return NULL;
   }

   playlist->entries   = entries;
   playlist->cap       = size;

   playlist_read_file(playlist, path);

   playlist->conf_path = strdup(path);
   return playlist;
}

There's a memory leak if you return in the first piece of code without freeing playlist, but infer doesn't detect it. It's a false negative right? I also think this is a messed up example and that infer could struggle in many ways... Thank you for helping! :)


File issues magic/v3

Ticket
If your pull request is related to a Suricata ticket, please provide
the full URL to the ticket here so this pull request can monitor
changes to the ticket status:
Redmine ticket:

replaced by #1277


More verrbose errors for JSON parsing

This commit adds information about the location of problematic JSON
files when reporting a JSON parsing error.
Before this commit:
[ERROR] Expecting property name: line 7 column 9 (char 188)

After this commit:
Error parsing 'core/mbed_lib.json':
[ERROR] Expecting property name: line 7 column 9 (char 188)

@0xc0170 @screamerbg

Forced-pushed the change suggested by @PrzemekWirkus

:+1:

+1


Backup showing error

Please dev resolve this error on 7plus jailbreak with checkra1n. Showing error something went wrong

This issue has already been posted (#3). Should be fixed as of v<IP_ADDRESS>.


develop ff-python based on ofn v1beta2

That's great, looking forward to this PR! @wanjunlei @wrongerror

I think this PR is ready for review, as it currently supports Dapr and HTTP triggers. (can be demonstrated in the sig meeting)
The observability feature and the hooks feature can be worked on later.
What are your comments?

@tpiperatgod That's great. @wanjunlei @wrongerror @webup @lizzzcai  would you help to review?

I'm okay with this PR, and it's great to have this significant improvement to the Python framework.
Just some minor issues remain.

any changes from the user side? maybe can update the docs if any.

any changes from the user side? maybe can update the docs if any.

I'll update the docs and examples later


run_simul_check_from_python.py

First implementation of ideas in https://github.com/exabl/snek5000/issues/101
Only works with OpenMPI.
Requires https://github.com/exabl/snek5000/pull/107
Of course, the detection of the instability saturation can be improved! It's just a proof of concept.
Associated todo list:

set environment variable in the launching script
more efficient sim.output.history_points.load

@akhoubani Can you please try this script and give some feedback?

@akhoubani Can you please try this script and give some feedback?

Yes, of course. I'll let you know.

It doesn't stop after saturation on my first try for aspect ratio 1.0.

I should check the algorithm.


PresetTimeCallback - add_tstop! method implementation

When I try to solve a SDDE with PresetTimeCallbacks the solver fails with this error message:
add_tstop!: method has not been implemented for the integrator
Julia Version v1.5.3
StochasticDelayDiffEq v0.2.8
DiffEqCallbacks v2.16.0
Example to reproduce the error:
using StochasticDelayDiffEq, DiffEqCallbacks

delay(u,h,p,t) = -0.1*u
stoc(u,h,p,t) = 1e-3

h(p,t) = 1.0

τ = 0.1
tspan = (0.0, 1.0)
lags = [τ];
p = []

data_stepper!(integrator) = 1.0
step_times = [0.1, 0.2]
cb = PresetTimeCallback(step_times, data_stepper!)

prob = SDDEProblem(
    delay,
    stoc,
    h,
    tspan,
    p;
    constant_lags=lags
);

sol = solve(prob, SROCK1(), dt=1e-3, callback=cb, adaptive=false)

If I'm not explicitly referencing DiffEqCallbacks the creation of the callback already fails with: UndefVarError: PresetTimeCallback not defined. Is this behavior intended?
If I'm using DifferentialEquations I get the same add_tstop!-error as above.
Thank you!

If I'm not explicitly referencing DiffEqCallbacks the creation of the callback already fails with: UndefVarError: PresetTimeCallback not defined. Is this behavior intended?

Yes, it needs DiffEqCallbacks.

When I try to solve a SDDE with PresetTimeCallbacks the solver fails with this error message

Looks like it's just a missing method.

Changing this function
https://github.com/SciML/StochasticDelayDiffEq.jl/blob/25524178668e5b40bb74417dfc0773ca03d564ab/src/integrators/interface.jl#L217
DiffEqBase.add_tstop!() seems to solve the problem.

That looks like it's the right solution. It just wasn't namespaced property to overload.

To merge https://github.com/SciML/StochasticDelayDiffEq.jl/pull/50


Refactor setup scripts and add helper functions

Attempting to trigger circle ci run from PR

Closing since CI did not trigger.

Me closing and re-opening this PR triggered the CI. @Arty-Maly can you refactor the setup-* scripts to use setup-helper-function? We can then see if that still triggers the CI. If this works, I can close and re-open the hdfs PR as well.

@majetideepak Can you review this pr, I think I fixed all the errors I was getting.
Thanks.

@majetideepak the linter is failing but I dont think I can see the FB Internal linter errors, is that required?


Admob integration

Hi Flet team,
This is an extraordinarily beautiful repo for python devs :)
I am just curious do you have any support now or future for Admob?

Sure, we could look into that when mobile version of Flet is ready!

admob and adsense

https://pub.dev/packages/google_mobile_ads

give me code for add ads in flet app

This feature is not yet available, but is on the roadmap.

+1

+1

+1

Check this https://pub.dev/packages/admanager_web

It's important to add functionality for monetization flet app and provide functionality for Mobile Ads and Web Ads Because Many Developers Create Products And Earn From Them Through Ads and Paid Subscription it's just my suggestions to make it possible as fast as because I also need this and waiting from 1 year I started working with flet from start of 2022 and waiting for these features @FeodorFitsner

any update?

@FeodorFitsner any update

@ndonkoHenri any update

Mobile Ads is like 90% done (PR).
Waiting on #3345 (WIP) to finalize as mentioned in https://github.com/flet-dev/flet/pull/3288#issuecomment-2257732918.
Please be patient.

Very Good! Iam very happy with that news.
Is this remaining 10% too critical?
in weeks? months? or more time?

Any update ?

 We needs this, precious!
Here's some good energy to speed up the process:  (∩ᄑ_ᄑ)⊃━☆ﾟ*･｡*･

when will this merged?

Install the latest pre and try mobile ads: pip install flet --pre -U
Example
For now, only two types are present: Banner and Interstitial. More types will be coming in the future.
import flet as ft
import flet.ads as ads

def main(page: ft.Page):
    page.horizontal_alignment = ft.CrossAxisAlignment.CENTER

    id_interstitial = (
        "ca-app-pub-3940256099942544/1033173712"
        if page.platform == ft.PagePlatform.ANDROID
        else "ca-app-pub-3940256099942544/4411468910"
    )

    id_banner = (
        "ca-app-pub-3940256099942544/6300978111"
        if page.platform == ft.PagePlatform.ANDROID
        else "ca-app-pub-3940256099942544/2934735716"
    )

    def handle_interstitial_close(e):
        nonlocal iad
        print("InterstitialAd closed")
        page.overlay.remove(e.control)
        page.overlay.append(iad := get_new_interstitial_ad())
        page.update()

    def get_new_interstitial_ad():
        return ads.InterstitialAd(
            unit_id=id_interstitial,
            on_load=lambda e: print("InterstitialAd loaded"),
            on_error=lambda e: print("InterstitialAd error", e.data),
            on_open=lambda e: print("InterstitialAd opened"),
            on_close=handle_interstitial_close,
            on_impression=lambda e: print("InterstitialAd impression"),
            on_click=lambda e: print("InterstitialAd clicked"),
        )

    def display_new_banner_ad():
        page.add(
            ft.Container(
                content=ads.BannerAd(
                    unit_id=id_banner,
                    on_click=lambda e: print("BannerAd clicked"),
                    on_load=lambda e: print("BannerAd loaded"),
                    on_error=lambda e: print("BannerAd error", e.data),
                    on_open=lambda e: print("BannerAd opened"),
                    on_close=lambda e: print("BannerAd closed"),
                    on_impression=lambda e: print("BannerAd impression"),
                    on_will_dismiss=lambda e: print("BannerAd will dismiss"),
                ),
                width=320,
                height=50,
                bgcolor=ft.colors.TRANSPARENT,
            )
        )

    page.overlay.append(iad := get_new_interstitial_ad())
    page.appbar = ft.AppBar(
        adaptive=True,
        title=ft.Text("Mobile Ads Playground"),
        bgcolor=ft.colors.LIGHT_BLUE_300,
    )
    page.add(
        ft.OutlinedButton("Show InterstitialAd", on_click=lambda e: iad.show()),
        ft.OutlinedButton("Show BannerAd", on_click=lambda e: display_new_banner_ad()),
    )

ft.app(main)

I am documenting this at the moment, but for those testing it right now, Interstitial ads can only be shown once. That's why in the example above we created new ones each time.
Feel free to ask any questions.

What about desktop and web?

Yay! Thank you guys and great job. Can't wait to try it! ٩(◕‿◕｡)۶

what about web

what about web

+1 @ndonkoHenri

what about web
Can we run Google AdSense ads via flet web view??? Is it possible @FeodorFitsner @ndonkoHenri

Web will investigated when possible.

Hey guys, so I'm a total newbie (only started learning about all this like 3-4 months ago). Is there a way to test this new feature with an emulator? I ask because I tried using Android Studio and the app either stays stuck on the splash screen or crashes. Without the Ads it works just fine.


KAFKA-15351: Update log-start-offset after leader election for topics enabled with remote storage

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@clolov @divijvaidya @showuon @abhijeetk88 @satishd
Call for review. Please take a look!

Unrelated test failures:
[Build / JDK 17 and Scala 2.13 / kafka.server.DescribeClusterRequestTest.testDescribeClusterRequestIncludingClusterAuthorizedOperations(String).quorum=kraft](https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-14340/5/testReport/junit/kafka.server/DescribeClusterRequestTest/Build___JDK_17_and_Scala_2_13___testDescribeClusterRequestIncludingClusterAuthorizedOperations_String__quorum_kraft/)
[Build / JDK 17 and Scala 2.13 / org.apache.kafka.controller.QuorumControllerTest.testBalancePartitionLeaders()](https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-14340/5/testReport/junit/org.apache.kafka.controller/QuorumControllerTest/Build___JDK_17_and_Scala_2_13___testBalancePartitionLeaders__/)
[Build / JDK 11 and Scala 2.13 / org.apache.kafka.connect.mirror.integration.IdentityReplicationIntegrationTest.testSyncTopicConfigs()](https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-14340/5/testReport/junit/org.apache.kafka.connect.mirror.integration/IdentityReplicationIntegrationTest/Build___JDK_11_and_Scala_2_13___testSyncTopicConfigs__/)

Merging in to trunk and 3.6


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test-source

Docs Build status updates of commit d7fe05f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-10-08-02-36-30 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-10-08-02-36-30

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


Extension

I was wondering if it was possible to get a day-extension on this project without penalty. I have a final on Wednesday which leaves only 1 more day to try to iron out any issues, along with that pdf documentation. Additionally, we need some more time trying to finish the react gui, which is new to all three of us.

@AznStevy To be fair to the class, we would have to extend the deadline for everyone... but I'm inclined to do that b/c the weather is hampering folks' efforts to get together to work on things.  How would Friday at midnight work?


Fix wrong redis replication connection config path

Key ".modes" was missing, always making it default

Codecov Report

Merging #613 (904a97c) into 2.x (b9ec590) will not change coverage.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##              2.x     #613   +/-   ##
=======================================
  Coverage   89.87%   89.87%
=======================================
  Files          60       60
  Lines        1560     1560
=======================================
  Hits         1402     1402
  Misses        158      158

Impacted Files
Coverage Δ

src/ChannelManagers/RedisChannelManager.php
96.05% <50.00%> (ø)

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 b9ec590...904a97c. Read the comment docs.