download command also runs testdrive, apparently

After deletion of the data/ directory, when running
java -jar target/phenopacket2prompt.jar download
I get
SLF4J(W): No SLF4J providers were found. SLF4J(W): Defaulting to no-operation (NOP) logger implementation SLF4J(W): See https://www.slf4j.org/codes.html#noProviders for further details. SLF4J(W): Class path contains SLF4J bindings targeting slf4j-api versions 1.7.x or earlier. SLF4J(W): Ignoring binding found at [jar:file:/Users/leonardo/IdeaProjects/phenopacket2prompt/target/phenopacket2prompt.jar!/org/slf4j/impl/StaticLoggerBinder.class] SLF4J(W): See https://www.slf4j.org/codes.html#ignoredBindings for an explanation. [INFO] Added 9 simulated individuals. [INFO] Added 18 simulated individuals. [INFO] Added 27 simulated individuals. [INFO] Added 36 simulated individuals. [INFO] Added 45 simulated individuals. [INFO] Added 54 simulated individuals. [INFO] Added 63 simulated individuals. [==================================================]   100%      [==================================================]   100%
(NOTE: judging by the preview of this message, not all newlines of the terminal are correctly represented).
Is this as expected @pnrobinson ?

Similar behavior with the following, i.e. running the batch command runs also download and testdrive. Which could be a bug or a feature, not sure!
leonardo@MBP-GNFWN6C2R7 phenopacket2prompt % java -jar target/phenopacket2prompt.jar batch -d ~/data/phenopacket-store/ SLF4J(W): No SLF4J providers were found. SLF4J(W): Defaulting to no-operation (NOP) logger implementation SLF4J(W): See https://www.slf4j.org/codes.html#noProviders for further details. SLF4J(W): Class path contains SLF4J bindings targeting slf4j-api versions 1.7.x or earlier. SLF4J(W): Ignoring binding found at [jar:file:/Users/leonardo/IdeaProjects/phenopacket2prompt/target/phenopacket2prompt.jar!/org/slf4j/impl/StaticLoggerBinder.class] SLF4J(W): See https://www.slf4j.org/codes.html#ignoredBindings for an explanation. [INFO] Added 9 simulated individuals. [INFO] Added 18 simulated individuals. [INFO] Added 27 simulated individuals. [INFO] Added 36 simulated individuals. [INFO] Added 45 simulated individuals. [INFO] Added 54 simulated individuals. [INFO] Added 63 simulated individuals. Retrieved 4876 files.

@pnrobinson I understand now the issue. TestDriveCommand has a constructor, at https://github.com/monarch-initiative/phenopacket2prompt/blob/800438507aa702dd200d78201b4fef0d68ab7c67/src/main/java/org/monarchinitiative/phenopacket2prompt/cmd/TestDriveCommand.java#L220-L222 and apparently as a subcommand this is run even if the command is not invoked. This explains why running download or batch subcommands still outputs things like
[INFO] Added 27 simulated individuals.
since the print statement is within the constructor:
https://github.com/monarch-initiative/phenopacket2prompt/blob/800438507aa702dd200d78201b4fef0d68ab7c67/src/main/java/org/monarchinitiative/phenopacket2prompt/cmd/TestDriveCommand.java#L296
Maybe what is now a constructor should be some further class/method, called from within Integer call()..., namely here:
https://github.com/monarch-initiative/phenopacket2prompt/blob/800438507aa702dd200d78201b4fef0d68ab7c67/src/main/java/org/monarchinitiative/phenopacket2prompt/cmd/TestDriveCommand.java#L307
Does this make sense?

The problem was that I put some code into the Constructor of the TestDrive command which is a mistake with the picocli framework. I am fixing it.

Fixed by https://github.com/monarch-initiative/phenopacket2prompt/pull/53


local delivery issue

hi there, first of all thanks for this great software,
I've an issue with an uncommon setup, we run mox behind a NAT and we are having some issue only with local delivery.
Is there a way to force mox to deliver email to its own managed domains directly?
thanks for your support

hi @lesion, is it the spf check that is causing trouble?
the following has been known to happen: your outgoing connections to your own mx host (to its public ip) would be routed back to mox. instead of the connection to your public ip appearing to come from that public ip, the route may NAT it in a way that it appears to come from the router's internal/private ip, which of course isn't listed in your domain's spf record. an spf fail can be a reason for a message to be rejected. your messages should also have dkim signatures, which should still pass, but for first-time sending the spf fail can still make the delivery suspicious enough to be rejected. a solution could be to change the spf check to allow private IPs (as used in NATs) for incoming messages from domains that the mox instance is hosting.
i don't think bypassing the regular delivery mechanisms is a good idea for domains hosted on the mox instance. quite some functionality is happening in the smtp delivery path (e.g. rate limiting, junk filtering, alias processing). not delivering as usual would mean that functionality would have to be implemented again in that new special self-delivery path.
but it may be better to take a step back: what issue exactly are you having with local delivery? perhaps there are some error logs that show what's going on?

yep, the issue was spf check

a solution could be to change the spf check to allow private IPs (as used in NATs) for incoming messages from domains that the mox instance is hosting.

yep, this could be nice for IPs and NATIPs settings

i don't think bypassing the regular delivery mechanisms is a good idea for domains hosted on the mox instance. quite some functionality is happening in the smtp delivery path (e.g. rate limiting, junk filtering, alias processing). not delivering as usual would mean that functionality would have to be implemented again in that new special self-delivery path.

I see.
btw, I solved with a lower level solution, not so clean but it is working as expected:
LOCAL_MACHINE_IP=<IP_ADDRESS>
PUBLIC_IP=<IP_ADDRESS>

# route traffic directed to public ip directly to our own ip
iptables -t nat -A OUTPUT -d  $PUBLIC_IP -p tcp --dport 25 -j DNAT --to-destination $LOCAL_MACHINE_IP

# change source ip to our public so spf check is happy
iptables -t nat -D POSTROUTING -d $LOCAL_MACHINE_IP -p tcp --dport 25 -j SNAT --to-source $PUBLIC_IP

thanks, feel free to close this


allow users to specify float types from OneHotEncoder and ContinuousEncoder

I am using a LinearPipeline consisting of a OneHotEncoder in combination with a NeuralNetworkClassifier, which I think is a basic yet really neat use case for linear pipelines.
However, Flux gives this warning:
┌ Warning: Layer with Float32 parameters got Float64 input.
│   The input will be converted, but any earlier layers may be very slow.
│   layer = Dense(11 => 16, relu)  # 192 parameters
│   summary(x) = "11×20 Matrix{Float64}"
└ @ Flux ~/.julia/packages/Flux/CUn7U/src/layers/stateless.jl:60

I know MLJ tries to abstract away from data types to use scitypes instead, but in this case it would be great to be able to specify the data type a OneHotEncoder gives as output.
For OneHotEncoder this happens to be really easy to do (it calls float), for a ContinuousEncoder it would be a bit trickier (it calls coerce).
I also don't know if it conflicts too much with the scitype philosophy or if there's an easier way around this, so I thought I'd ask here before making a PR (which I would gladly do).

This sounds very reasonable. For the ContinuousEncoder, perhaps we can just call Float32.(...) after the coercion. Be great to have a PR, thanks!
Would it be enough to have a single option Float32 or Float64, or do we want more than that?
@EssamWisam Perhaps we can keep such an option in mind for the encoders you are working on. We should have a consistent name for the hyper-parameter. If it's a flag, how about use_float32?
@tiemvanderdeure What do you think?

I think it's common to train deep learning models at 16-bit precision and sometimes even lower. Thus, I think one way to approach this may be to add a transformer (e.g., CastingTransformer) that can cast solumn types (e.g., Float64 to Float32 or Int32 to Int16 and so on) according to user requirements.
Alternatively, I think it would be good for Flux models to do this conversion themselves (e.g., while batching). That would be easier because one can then try out their Flux model at different precisions for the parameters without also manuaully changing the precision of the data.

My initial idea was to be a bit more flexible by adding a datatype hyper-parameter that defaults to Float64.
Would a CastingTransformer just transform columns into the same scitype but a different DataType? I can see the use case, but it would make pipelines longer than if we can integrate this into existing transformers.
I think doing the conversion in MLJFlux is also reasonable - I can't see any reason why you wouldn't want the input data to be converted to whatever datatype the neural net has. I don't know if there are any other MLJ models that have specific requirements to input data? I can't think of any on the spot, but I only know a fraction of all the models.

When I proposed having a separate transformer, the reason was it would not make sense to have the hyperparameter take only specific values for the type (e.g., only Float64 or Float32) as the model may also be Float16 or it may use an embedding layer requiring the input to be ordinal encoded integers and because a separate transformer can also be used other applications (e.g., casting specific columns of a dataset to Float16 while casting others to Int16 to save memory before feeding the data into some model).
If I understand correctly now, you meant that datatype does not have to be restricted to specific types (e.g., user can supply the type). While I think this can work (while being tricky to implement), I still lean towards having Flux or MLJFlux be responsible for this as it can be argued that among all the models that one can insert in a pipeline, only Flux models need this; the rest only depend on the scientific type. Otherwise, developing something generic that could be used for Flux models or another purpose can also make sense; albeit, tricky as well.

Just made a quick PR to show what I had in mind - it might still be better to handle this in (MLJ)Flux instead, then we don't merge the PR.

After thinking about this more, my vote would be to add this to MLJFlux, as this is, for now, the only package fussy about float types. In the rare we want float32 in some other case, one can always insert and an ordinary function into the pipeline. For example, if you have a float64 table, you could insert float32ify, where
float32ify(X) = Float32.(Tables.matrix(X))

which returns a Float32 matrix, which MLJFlux models will accept. It seems overkill to have a new Static transformer to generalise this kind of simple conversion, no?
In MLJFlux, I wouldn't implement the conversion by adding a layer to the Flux model. Rather do it in the data pre-processing step, so that it only needs to happen once.

In MLJFlux, I wouldn't implement the conversion by adding a layer to the Flux model. Rather do it in the data pre-processing step, so that it only needs to happen once.

Admittedly, this is a bit of a pain because the reformat methods do not see the model, which would carry the float type hyperparameter. 🤷🏾

Admittedly, this is a bit of a pain because the reformat methods do not see the model, which would carry the float type hyperparameter. 🤷🏾

In the current state, Flux will behave as reported above (as far as I believe) if input is FloatX and model is FloatY where X≠Y. The most frequent case for this will have X=64 and Y=32. If we make a reformat method to handle just this case, then we don't perfectly solve this problem for all setups but at least do so for the vastly most frequent setup (i.e., still an improvement over the status quo).

It makes good sense to me to just add this to MLJFlux. I'm leaving on holiday today, but I can write a PR in august.

MLJFlux now automatically converts inputs to float32.


added_linked_list_in_ruby

Fixes #{506}
By submitting this pull request I confirm I've read and complied with the below declarations.

[ x ] I have read the Contribution guidelines and I am confident that my PR reflects them.
[ x ] I have followed the coding guidelines for this project.
[ x ] My code follows the skeleton code structure.
[ x ] This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
[ x ] This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.

Hi! Thanks for showing interest in contributing to this repository.
Please make sure that you have ticked the points mentioned in PR description correctly.
Thanks again!

@TUSHARZ you're not assigned for #506 . Please ask before sending a PR.


Error: Missing "./dist/runtime/composables" specifier in "@nuxtjs/i18n" package

Environment

Operating System: Linux
Node Version:     v16.19.1
Nuxt Version:     3.4.1
Nitro Version:    2.3.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      modules, i18n, pages, build, nitro, css, vite, runtimeConfig, typescript
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
Sadly, after a couple of hours of trying to reproduce the error, I could manage to do so. I hope, that you maybe can make sense out of this error and provide me some tips to reproduce the error
Describe the bug
Error occurs after running vitest
Additional context
nuxt dev functions perfectly, as does nuxt build and pretty much all the other commands.
Here is the list of packages used.
"dependencies": {
    "@pinia/nuxt": "^0.4.6",
    "axios": "^1.2.2",
    "leaflet": "^1.9.1",
    "leaflet-gesture-handling": "^1.2.2",
    "pinia": "^2.0.29",
    "vite-plugin-vuetify": "^1.0.1",
    "vuetify": "^3.1.11"
  },
  "devDependencies": {
    "@mdi/js": "^7.1.96",
    "@nuxt/test-utils": "^3.2.3",
    "@nuxtjs/eslint-config-typescript": "^12.0.0",
    "@nuxtjs/i18n": "8.0.0-beta.11",
    "@types/leaflet": "^1.9.3",
    "@types/lodash-es": "^4.17.6",
    "eslint": "^8.31.0",
    "eslint-config-prettier": "^8.6.0",
    "eslint-plugin-prettier": "^4.2.1",
    "nuxt": "3.4.1",
    "prettier": "^2.8.2",
    "sass": "^1.57.1",
    "sass-loader": "^10.4.1",
    "typescript": "^4.9.4",
    "vitest": "^0.30.1",
    "vue-tsc": "^1.2.0"
  },

Logs
FAIL  tests/pages.test.ts > Test Index pages
Error: Missing "./dist/runtime/composables" specifier in "@nuxtjs/i18n" package
 ❯ e node_modules/vite/dist/node/chunks/dep-d305c21f.js:16076:25
 ❯ n node_modules/vite/dist/node/chunks/dep-d305c21f.js:16076:627
 ❯ o node_modules/vite/dist/node/chunks/dep-d305c21f.js:16076:1297
 ❯ resolveExportsOrImports node_modules/vite/dist/node/chunks/dep-d305c21f.js:23295:20
 ❯ resolveDeepImport node_modules/vite/dist/node/chunks/dep-d305c21f.js:23314:31
 ❯ tryNodeResolve node_modules/vite/dist/node/chunks/dep-d305c21f.js:23000:20
 ❯ Object.resolveId node_modules/vite/dist/node/chunks/dep-d305c21f.js:22762:28
 ❯ node_modules/rollup/dist/es/shared/node-entry.js:24407:40
 ❯ PluginDriver.hookFirstAndGetPlugin node_modules/rollup/dist/es/shared/node-entry.js:24307:28

The issue is resolved. It was caused by malformed import of defineI18nConfig

defineI18nConfig

@mikhgaus Hey, could you tell what import did you use to fix it? Thanks!

I used "@nuxtjs/i18n": "^8.0.0-rc.5" version problem still exists, please how did you solve it in the end ?

defineI18nConfig

@mikhgaus Hey, could you tell what import did you use to fix it? Thanks!

I can confirm, I'm using 8.0.0-rc.5 and after updating to nuxt 3.8.0 i get the error message
Error: missing "./dist/runtime/composables" specifier in "@nuxtjs/i18n" package"
i have auto import set to true. Does anybody have an idea how to solve it

I was in the exact same situation and realized that it was the Nuxt update that broke some imports.
I was importing the LocaleObject type like this:
import { LocaleObject } from '@nuxtjs/i18n/dist/runtime/composables'
Which broke when updating Nuxt, needing me to change the import to this:
import type { LocaleObject } from '@nuxtjs/i18n/dist/runtime/composables'
Which makes sense. Hope this can help!


[WIP][test-maven][test-hadoop3.2] YarnClusterSuite

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Test build #122681 has started for PR 28546 at commit 6bcbcbc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27343/
Test PASSed.

Test build #122686 has started for PR 28546 at commit f55d551.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27348/
Test PASSed.

Test build #122681 has finished for PR 28546 at commit 6bcbcbc.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122681/
Test FAILed.

Test build #122686 has finished for PR 28546 at commit f55d551.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122686/
Test PASSed.


Automated Markdown Generation from ISSUES

Hi @vinitshahdeo I have created a script that will generated Markdown based on the ISSUES file. Whenever PR is raised on the Main branch.
Please do also add Hacktoberfest-accepted tag to it when merged
Script Standalone working :
https://user-images.githubusercontent.com/11576342/137005267-e8dcda35-6fb9-42bf-af99-c60ce882e006.mp4
Script trigger when PR is raised on main branch
https://user-images.githubusercontent.com/11576342/137007719-3f66b244-e07d-4269-99e2-798a8df061c7.mp4

This is awesome man. Can we have a live demo for the same?

Sure I will drop a screen recording of demo in PR

Hi @vinitshahdeo I found one bug please dont merge it now I will fix that and let you know meanwhile you can see the video of how it function

If this is done , pls close this issue and help us reduce mess in issue
Thanks 😃


Implement restricted access to GraphiQL/Playground/Voyager UIs.

Is your feature request related to a problem? Please describe.
Apply an Authorization role or policy to restrict access to GraphiQL, Playground, Voyager
Describe the solution you'd like
Probably you'd just add an option to specify an authorization policy name in GraphiQLOptions, PlaygroundOptions and VoyagerOptions.
Describe alternatives you've considered
Currently we just disable GraphiQL/Playground/Voyager for Release build.
Additional context
The more general problem is to apply an Authorization policy to ...

Loading of the Client UIs
Schema queries (v1 global)
Schema queries (v2 localized to specific models and perhaps even fields?)
I guess you'd need to research what direction the official GraphQL.org specs have taken on this.

@michaelstaib  we are going to production with hotchocolate in the upcoming weeks. thankssss!
We would love the Schema Authorization feature.
In our use case we want to be able to secure the introspection query all together.
i think its what @tohagan was refering as Schema Queries v1 global
we are already authorizing all queries and mutations with :hot_pepper: :chocolate_bar:

@cescoferraro we have that on our list for V11. So, it will come. We might even push this out with 10.3.0.

Actually the bigger issue here is restricting access (or disabling!) introspection queries.

https://www.abhaybhargav.com/from-the-trenches-diy-security-perspectives-of-graphql/

@tohagan that is possible, you can extend the introspection types and restrict them.

You just have to add our authorize directives.

So, just regarding security, you can already do a lot with hc to make your endpoint super secure. That is why I do not consider this pressing.
Securing Playground is just securing the route.
But you can already whitelists queries, use persisted queries, put authorization directives on any of your fields / including introspection. You can even put custom middleware on your introspection fields to secure them.
Moreover, you can control the execution depth and the allowed cost of a query. You can even intercept the input coercion if you so desire to handle the coercion of inputs yourself.
So, from a security point this is not really a big issue.

Also, you can add custom query validation rules which could further harden your endpoint.

Regarding the endpoint security: https://github.com/ChilliCream/hotchocolate/pull/1190

Thanks ... that all sounds great... so then the missing bit  is documenting how to do these things. I read some security articles on GraphQL and a common issue raised was securing introspection. Now that I know the issue I’ll act on it but the next person learning HC likely won’t know these security risks, nor what to do to fix them.

@tohagan do you want to do a pr for that? I can assist you with putting all the information together. You could add a section to the security section of the documentation, maybe even with a reference to the article that you have read.
We are using docusaurus and the docs are located here:
https://github.com/ChilliCream/hotchocolate-docs

I've not forgotten your PR request.  Just been buried other non GraphQL stuff + xmas holidays.
I think I'd need to refer to an example of how to code restricting access to introspection queries but not sure how to go about this despite your guidance above.

@tohagan that is possible, you can extend the introspection types and restrict them.

@michaelstaib, is it possible to somehow add authorize directive for the built-in introspection __schema query?

@vhatsura yes it is. you can add a authorize directive on the query type and associate it with a authorization policy.
In the policy you have access to the resolver context and you can check if the current field is a introspection field.
The auth policy context as a property Resource which is the IResolverContext
On the IResolverContext => context.Field.IsIntrospection...

you can add a authorize directive on the query type and associate it with a authorization policy.

@michaelstaib, sorry for that but I cannot familiar enough with HC, can you post a small example snippet for that? For our query type we have the following:
protected override void Configure(IObjectTypeDescriptor<Query> descriptor)
{
    //descriptor.Authorize(Scopes.Full); // authorize directive for all queries
    descriptor.Field(t => t.GetFetchRocket(default, default)).Authorize(Scopes.Full); // authorize directive for fetchRocket query
}

Is it possible to have similar for __schema introspection query without adding additional middleware?

From @tohagan:

I read some security articles on GraphQL and a common issue raised was securing introspection.

From @vhatsura:

sorry for that but I cannot familiar enough with HC, can you post a small example snippet for that?

Same issue, require some way to completely disable introspection on some deployments for security reasons but unsure how to do so.  @michaelstaib your sketch of solution sounds like would solve the issues if you can provide a bit more detail on where/how to hook up an IsIntrospection filter.

Playground has now been replaces with bcp and uses the new endpoint API, this allows for using authorization rules.

@michaelstaib echoing what was requested a couple of times above: Could you provide a simple example of how to disable introspection queries? Thanks! :)

I have figured something out for v11:
internal class NoIntrospectionDocumentValidatorVisitor : TypeDocumentValidatorVisitor
{
    protected override ISyntaxVisitorAction Enter(FieldNode node, IDocumentValidatorContext context)
    {
        if (node.Name.Value == IntrospectionFields.Schema || node.Name.Value == IntrospectionFields.Type)
        {
            IError error = ErrorBuilder.New().SetMessage("Introspection queries are disabled").Build();
            context.Errors.Add(error);
            return new BreakSyntaxVisitorAction();
        }

        return base.Enter(node, context);
    }
}

Register in Startup.cs as:
services
    .AddValidation()
    .TryAddValidationRule<NoIntrospectionDocumentValidatorVisitor>();

I have figured something out for v11:
...

Hi, your proposed solution does not work, it fails here

services
    .AddValidation()
    .TryAddValidationRule<NoIntrospectionDocumentValidatorVisitor>();

The error given is this one
The type 'NoIntrospectionDocumentValidatorVisitor' cannot be used as type parameter 'T' in the generic type or method 'HotChocolateValidationBuilderExtensions.TryAddValidationRule<T>(IValidationBuilder)'. There is no implicit reference conversion from 'NoIntrospectionDocumentValidatorVisitor' to 'HotChocolate.Validation.IDocumentValidatorRule'.

@huysentruitw  @gallargit
In the lastest preview you can just do
services
                 .AddGraphQLServer()
                 .AddIntrospectionAllowedRule() /// <-----
                 ...


KYLIN-2924 disable google error-prone for travis-ci

Travis-ci failed with message:
"
The job exceeded the maximum log length, and has been terminated."
I try to disable errorprone check to reduce the log output.

Just try if it can make travis-ci happy

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@09c5668). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #281   +/-   ##
=========================================
  Coverage          ?   21.29%
  Complexity        ?     4436
=========================================
  Files             ?     1087
  Lines             ?    69919
  Branches          ?    10101
=========================================
  Hits              ?    14889
  Misses            ?    53626
  Partials          ?     1404

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 09c5668...7ea312f. Read the comment docs.

Pull Request Test Coverage Report for Build 3742

0 of 0   changed or added relevant lines in 0 files are covered.
21 unchanged lines in 6 files lost coverage.
Overall coverage increased (+0.01%) to 23.306%

Files with Coverage Reduction
New Missed Lines
%

atopcalcite/src/main/java/org/apache/calcite/sql/type/SqlTypeUtil.java
1
0.0%

atopcalcite/src/main/java/org/apache/calcite/tools/Programs.java
1
0.0%

core-dictionary/src/main/java/org/apache/kylin/dict/lookup/cache/RocksDBLookupTable.java
1
81.08%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
4
78.12%

atopcalcite/src/main/java/org/apache/calcite/sql2rel/SqlToRelConverter.java
7
0.0%

atopcalcite/src/main/java/org/apache/calcite/prepare/CalcitePrepareImpl.java
7
0.0%

Totals

Change from base Build 3738:
0.01%

Covered Lines:
16295

Relevant Lines:
69919

💛  - Coveralls


Version <IP_ADDRESS>+ do not detect new versions/updates

PHP Version
8.1
Shopware Version
<IP_ADDRESS>
Expected behaviour
Shop should notify when an update is available. "in this case <IP_ADDRESS>"
As Popup or under configuration -> system -> Shopware-Updates
Actual behaviour
Nothing happens, no notification popup and Shopware-Updates tells me the Version is the latest version.
How to reproduce
Hard to they in this case.
In Version 6.4.x the notification worked without any problems.
Since <IP_ADDRESS> this does not work anymore.
The logs also do not provide any information on what is going on.
This Problem first appeared when we have installed a fresh version of <IP_ADDRESS> when it came out.
Then I had to do a upgrade by hand.
Downloading the installer from the change log page and do the upgrade manual.
Where the next problem "https://github.com/shopware/platform/issues/3190" appeared, which was cleared up by @shyim.
I hope in this case, it is a similar issue and can be fixed very easily.
Let me know if I can provide any information needed.

Looks like the issue "https://issues.shopware.com/issues/NEXT-27536" is not fixed.
The problem still exists in Version <IP_ADDRESS>.
At least in our case.


Merge Release <IP_ADDRESS>-rc bugfixes into master branch

This is a placeholder PR to merge all changes from Release Candidate branch <IP_ADDRESS>-rc into master

@maciejzaleski can you update the PR so that it only contains the drift adjustment?


Add Equatable functionality to JSONValue

Extend JSONValue to conform to Equatable and implement the == function so
JSONValue can be used in equality testing.

:+1:

:+1:


remove codecov.yml in the reedline repo

I find reviewing PRs with the code coverage turned on incredibly annoying and hard to understand
all of the other code that I need to understand what has changed...
Unless someone really feels strongly about having it in there I am going to remove it....
Thanks !

I'm wondering if we can disable it without deleting the file just in case we want to easily enable it again?

Here is a copy of the codecov.yml file
coverage:
  status:
    project:
      default:
        informational: true
    patch:
      default:
        informational: true

comment:
  layout: reach, diff, files
  behavior: default
  require_base: yes
  require_head: yes
  after_n_builds: 5 # Our test matrix currently has 5 configurations

Here is a copy of the associated ci.yml file
on:
  pull_request:
  push: # Run CI on the main branch after every merge. This is important to fill the GitHub Actions cache in a way that pull requests can see it
    branches:
      - main

name: continuous-integration

jobs:
  build-lint-test:
    strategy:
      fail-fast: false
      matrix:
        platform: [ubuntu-latest]
        rust:
          - stable
        # Define the feature sets that will be built here (for caching you define a separate name)
        style: [bashisms, default, sqlite, basqlite, external_printer]
        include:
          - style: bashisms
            flags: "--features bashisms"
          - style: external_printer
            flags: "--features external_printer"
          - style: default
            flags: ""
          - style: sqlite
            flags: "--features sqlite"
          - style: basqlite
            flags: "--features bashisms,sqlite"

    runs-on: ${{ matrix.platform }}

    steps:
      - uses: actions/checkout@v3

      - name: Setup Rust toolchain
        uses<EMAIL_ADDRESS>      - name: Setup nextest
        uses: taiki-e/install-action@nextest
      - name: Setup cargo-llvm-cov
        uses: taiki-e/install-action@cargo-llvm-cov

      - name: Rustfmt
        run: cargo fmt --all -- --check

      - name: Clippy
        run: cargo clippy ${{ matrix.flags }} --all-targets --all -- -D warnings

      - name: Tests
        run: cargo llvm-cov nextest --all ${{ matrix.flags }} --lcov --output-path lcov.info

      - name: Doctests
        run: cargo test --doc ${{ matrix.flags }}

      - name: Upload coverage
        uses: codecov/codecov-action@v3
        with:
          files: lcov.info

Note here is another copy of both files for archival reasons in case we want to change back the settings...
https://github.com/stormasm/nunotes/tree/main/archive

For starters I will just attempt to modify the ci.yml action without actually deleting the codecov.yml file to see if that works and turns off code coverage...
If this doesn't work then we will remove the codecov.yml file...
I wish there was a setting that individuals could set to toggle code coverage on and off when you are looking at PRs.
Then you can review the code with out seeing all of the extraneous code coverage messages which get in the way of trying to understand the code changes...


Update documentation on resource quota limits

Related to https://github.com/rancher/rancher/issues/35647

@jtravee There's some more information about this in https://github.com/rancher/dashboard/issues/5016 but it's complicated. To summarize the background context of these changes, I would say:

Currently in the Rancher UI, if you edit project quota limits or namespace default limits, it will update existing namespaces in the project. This is not desired behavior and the UI is going to change so that only new namespaces are affected by editing resource quotas at the project level, not existing namespaces.
Currently in the Rancher UI, if you make a project resource quota, then you have namespaces within the project that have quotas adding up to the project quota, there's no resource capacity remaining in the project, so if you try to create a new namespace in the project, you see resource limits set to zero or negative values in the namespace. Because to set the default resource limits for new namespaces in the project, the UI first adds up all the limits for namespaces within the project, then compares that with the project resource quota. If there's any resources left in the project, then the default limit is that remainder or the default namespace quota, whichever is smaller. There are several UI tickets open to try to improve the user experience around this in the UI. https://github.com/rancher/dashboard/issues/5016#issuecomment-1048383086
The user can create a namespace within a project by using kubectl, only if they add annotations about what project they want to put the namespace in. They can also use annotations to ask for resources to be allocated for the namespace. There is currently an open UI ticket where we want to stop using those annotations to calculate how many resources are left in a project, because those annotations only represent what was requested, not what was actually received. https://github.com/rancher/dashboard/issues/5188

I hope that is enough context. These docs changes are basically capturing how the UI is supposed to work when the bugs are fixed, in an effort to keep the UI and the docs in sync.

@btat I updated one little thing - i would say it's good to merge now.


add tags to experiments

🚀 Deployed on https://63349841da8431004683311d--cozy-fox-0defec.netlify.app


PropertyChanged delegate does not return which property is this BindablePropery raised for. Like we get in Framework 4.8 Dependency Property.

Type of issue
Other (describe below)
Description
In a custom control, if I create multiple BinableProperty and assign the same function, then the delegate does not have a parameter to understand for which property it is called. In that single function assigned to multiple properties, I should be able to know the property in the parameter.
Page URL
https://learn.microsoft.com/en-us/dotnet/api/microsoft.maui.controls.bindableproperty?view=net-maui-9.0
Content source URL
https://github.com/dotnet/dotnet-maui-api-docs/blob/live/docs/xml/Microsoft.Maui.Controls/BindableProperty.xml
Document Version Independent Id
028a9a57-7d07-d32f-5665-5d686b735a73
Article author
@dotnet-bot
Metadata

ID: 028a9a57-7d07-d32f-5665-5d686b735a73
Service: dotnet-mobile
Sub-service: dotnet-maui

Thanks for the report! I'm afraid I don't understand completely. Is this something that you wish has a different functionality or something that should be better reflected in the documentation?


Optimisation: Cache language files for Vorto properties

This was an issue raised by Simone (@simonech) at Code Garden before my back office optimisation talk. I don't know all the details but I'm hoping this description will be enough to understand the issue and do some more digging.
It seems that when loading a Vorto property it makes a call to the server for a language file (or list of languages of some sort). This is done per property so if you have several properties they all make this same call even though its doubtful they would change between properties.
Suggest this is called once and cached and then reused client-side. This does raise the issue of how do you invalidate the cache but that could either be done via a timestamp (micro caching for just 5 seconds while everything loads might do it) or something a bit more robust...I think micro cache should do, it can reload it each page just not multiple times on a page.
This "could" be fixable via the work in this issue (http://issues.umbraco.org/issue/U4-11386) with Service workers but I think better to fix it at source.

Good spot. Maybe just loading it once per "editor load" would be ok. So when you open an editor it just gets requested once and all vorto probs on the front end use the same result.
I'll have a play when I have a sec though.
Thanks for reporting.

Could also be done using the "sync" mechanism the same way you sync open tabs and current tab.
You just load the languages once and sync its content to all other properties in the page.
Anyway this seems like a more generic issue. There should be a way to adopt the "service worker" approach (or something similar) to all API calls done in the backend, and just load them once per editor (even if reused in different properties). For example the configuration of doctypegrid editor, where every "cell" loads the overall configuration. But this is beyond the scope of this issue :)

@simonech I logged that other issue with service workers we discussed here too btw: http://issues.umbraco.org/issue/U4-11388


Scatterplot: y axis range changes when smoothed mean shown

In toggling between raw data or BestFitLineWithRaw, the y-axis range changes little if it all. However, when selecting smoothedMean or smoothedMeanWithRaw, the y-axis range changes drastically.
Example:
Data: SCORE
Subset: none
Plot: scatter
x-axis: age at enrollment
y-axis: birth year
Here the y-axis changes from (1930, 2010) to (0, 2010) (approximately).

Hi @asizemore and @dmfalke
In summary, I tentatively conclude that it is a bug at plotly, thus I have implemented something to address this issue, which would work fine, I suppose.
For more details: I have tested this example and several other datasets (with different variables). Note that I have been relying on plotly's autorange feature for y-axis as I was reluctant to set the range manually: it was because several plots were involved in there (like raw data markers, smoothed mean line, confidence interval (bounds), and best fit line). Somehow, for the specific dataset that Ann noticed, the plotly's autorange did not work correctly. To overcome the issue, I have decided to manually set y-axis range. As a result, it shows reasonable plot now: screenshot is attached for the case of smoothedMeanWithRaw with the example above.
@asizemore Please note that it would take some time for this change to be reflected to the feature site as it needs review/merge and updating to the feature site. Please feel free to ask me if you find another issue concerning the scatter plot viz.

Thanks so much @moontrip!! The new plot looks great.
Thank you for the detail about plotly as well -- I understand the underlying problem much better. I agree it makes sense to use plotly's autorange in general, and I find the way plotly's autorange failed with the multiple plots very interesting.
Thanks again for looking into this!


Support for other sites

Is this planned? Kind of limited that it only supports YouTube.

I don't think it is a good idea to support all video sharing services in one library, which would also be quite much work to keep them all updated. It would be better to create multiple libraries that work standalone and maybe then creating a library that parses the link and selects the right library.

^ would have to agree with that. It would be too much work. There are plans in the works though on adding more features to this library like the ability to bypass age restrictions, extract youtube comments, fetch related videos, etc... but the point is that all of that would deal with youtube.com exclusively.


Update Opera to version 33.0.1990.43

:point_up: Title says it all!

Thank you for the contribution. It was merged directly as 245d8ce42341db8ac31c386dc4ac2d78b25ebdbd to keep commit history cleaner. Your contribution is still credited to you.


[BUG] Client#search errors when no results exist

Describe the bug
When passing a query into Client#search that produces no results, the following error is thrown:
TypeError: Cannot read property 'continuationItemRenderer' of undefined
    at SearchResult.loadSearchResult (/Users/inquire_kinect/VSC/Repos/TommyBot/node_modules/.pnpm/youtubei@0.0.1-rc.10/node_modules/youtubei/dist/classes/SearchResult.js:107:19)
    at SearchResult.<anonymous> (/Users/inquire_kinect/VSC/Repos/TommyBot/node_modules/.pnpm/youtubei@0.0.1-rc.10/node_modules/youtubei/dist/classes/SearchResult.js:63:18)
    at Generator.next (<anonymous>)
    at fulfilled (/Users/inquire_kinect/VSC/Repos/TommyBot/node_modules/.pnpm/youtubei@0.0.1-rc.10/node_modules/youtubei/dist/classes/SearchResult.js:11:58)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

To Reproduce
const YTI = require("youtubei")
const yt = new YTI.Client()

yt.search("ewiqur6iuwraflojksadbvasadfgshp2sr")
  .then(console.log, console.error)

Expected behavior
What should happen instead is that either an empty array or null should be returned.

Thank you for reporting this!
It is indeed should return an empty array instead of throwing an error, will fix this soon.

Please upgrade rc.12 😁
Plus you can also now check the estimated result count
const result = await youtube.serach("...");

console.log(result.estimatedResults);


Enhancements

Addresses:

#295
#298
#240

The CI checks have passed, and I've seperately run this successfully with multiple test configurations from scratch on both windows and linux systems; I'll merge this one in.


Add an average feeding time graph

There could be some value in finding relationships between feeding time (and maybe frequency) and sleeping habits.

4272c9b075a2b3d92eacf684985af283b92477a1 adds a simple implementation that I will probably tweak a little further before closing this.

0646640 changes this to a cleaner scatter plot and includes number of feedings by date.


[feat] http client with cert

Support HTTP client to use specified certificates. It's necessary when two-way authentication is enabled (the server will verify the client's identity) on the HTTP server.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Allow arguments to container.stop()

Feature Request
It would be helpful to be able to pass arguments to to container.stop() which are propagated to the underlying dockerode container.stop().  This would allow specifying the timeout, t, that the container will wait before it is killed:
https://docs.docker.com/engine/api/v1.37/#operation/ContainerStop
When dealing with an ephemeral container for a test, you often don't care about it shutting down cleanly, since you'll be throwing it away.  In order to achieve this in my tests I have to currently do a hacky (and brittle) workaround of setting the underlying container's default options for stop:
(testContainer as any).container.container.defaultOptions.stop.t = 1;
await testContainer.stop();

It would be much nicer to instead just call the top-level stop with arguments:
await testContainer.stop({ t: 1 });

Available in 1.3.0.

Confirmed working in 1.3.1 with the command:
await testContainer.stop({
  timeout: new Duration(0, TemporalUnit.SECONDS)
});


bug: kubesphere/fluent-bit:v1.9.9 fluent-bit-watcher not executable

Describe the issue
Running kubesphere/fluent-bit:v1.9.9 gives output:
exec /fluent-bit/bin/fluent-bit-watcher

To Reproduce
docker run --rm -it kubesphere/fluent-bit:v1.9.9

of cause same when deployed via fluent-operator master in Kubernetes (kind)
Expected behavior
it runs, showing proper output similar to v1.9.8 :smile:
Your Environment
- Fluent Operator version: current master 91e1e98e0c08dd298e9f2dcf081ccbd20e6c27ea, but irrelevant here
- Container Runtime: Docker version 20.10.18, build b40c2f6
- Operating system: Ubuntu 20 LTS
- Kernel version: 5.15.0-48-generic

How did you install fluent operator?
No response
Additional context
No response

@wenchajun would you help to check if any of the changes introduced in v1.5.1 is related to this?
If the official fluentbit:v1.9.9 has the same problem or the fluentbit image we made has the problem?

I thought the /fluent-bit/bin/fluent-bit-watcher is the extra piece you are adding to the official fluentbit image, no?

I thought the /fluent-bit/bin/fluent-bit-watcher is the extra piece you are adding to the official fluentbit image, no?

Yes, our custom image is started by /fluent-bit/bin/fluent-bit-watcher

just wanted to report I also got this. I remember before I didnt get this, so does anyone know what version we might need to roll back to in order to get it working again?

kubectl get pods -n fluent
NAME                               READY   STATUS             RESTARTS      AGE
fluent-bit-bkzbj                   0/1     CrashLoopBackOff   4 (46s ago)   2m21s
fluent-bit-kf5q5                   0/1     CrashLoopBackOff   4 (53s ago)   2m21s
fluent-operator-749bfb89bd-kcc6m   1/1     Running            0             2m32s
jasonrperez@AFSAFS-9CV1JRHO:~$ kubectl logs fluent-bit-bkzbj -n fluent
exec /fluent-bit/bin/fluent-bit-watcher: no such file or directory

@

just wanted to report I also got this. I remember before I didnt get this, so does anyone know what version we might need to roll back to in order to get it working again?
edit: nvm you said "it runs, showing proper output similar to v1.9.8" so that answers that question, anyways for thoroughness heres my output:

kubectl get pods -n fluent
NAME                               READY   STATUS             RESTARTS      AGE
fluent-bit-bkzbj                   0/1     CrashLoopBackOff   4 (46s ago)   2m21s
fluent-bit-kf5q5                   0/1     CrashLoopBackOff   4 (53s ago)   2m21s
fluent-operator-749bfb89bd-kcc6m   1/1     Running            0             2m32s
username@hostname:~$ kubectl logs fluent-bit-bkzbj -n fluent
exec /fluent-bit/bin/fluent-bit-watcher: no such file or directory

@wenchajun Would you take a look if you can reproduce this

After my testing, it seems that the underlying image is the cause here, and the exact cause is still being troubleshot. You can switch the image to kubesphere/fluent-bit:v1.9.4 for now.

I've revert #416 by https://github.com/fluent/fluent-operator/pull/439 which might cause this problem and @wenchajun will release v1.6.1 @ITler @perezjasonr


space-age:  verify ordering of actual/expected

In the Jasmine testing framework, it is convention to:
expect(actual).fn(expected)

But some test suites have the following:
expect(expected).fn(actual)

This causes the error messages to be confusing.
Please verify the space-age exercise to make sure that the expectations are ordered correctly. If they are, then this can be closed.
This issue will help to close #111

Got it

This one is OK and can be closed

You rulez!


"zmap: could not detect default gateway address for 0." on VPS with strange interfaces

(git version of zmap, which BTW reports "zmap 1.0.0".)
Summary:
On a VPS, with a non-normal ifconfig, I get strange results from zmap which I cannot solve with -i nor -G
Long story:
On a VPS, I get:
root@superdoos:~# zmap -p 80 -N 10 -B 1M -o -
Aug 20 18:22:20.820 [INFO] zmap: started
Aug 20 18:22:20.899 [FATAL] zmap: could not detect default gateway address for 0. Try setting default gateway mac address (-G).
root@superdoos:~#
My ifconfig shows:
venet0    Link encap:UNSPEC  HWaddr 00-00-00-00-00-00-00-00-00-00-00-00-00-00-00-00
inet addr:<IP_ADDRESS>  P-t-P:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>
venet0:0  Link encap:UNSPEC  HWaddr 00-00-00-00-00-00-00-00-00-00-00-00-00-00-00-00
inet addr:<IP_ADDRESS>  P-t-P:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>
(public IPv4 address a bit altered)
Specifying "-i venet0:0" or "-i venet0:0" results in the same problem.
Additionally specifying -G results in another error message, a high ETA and no useful ports found:
zmap -p 80 -N 10 -B 1M -o - -i venet0 -G 00:00:00:00:00:00
Aug 20 18:27:12.031 [INFO] zmap: started
SIOCGIFADDR: Invalid argument
0:01 0%; send: 0 0 p/s (0 p/s avg); recv: 0 0 p/s (0 p/s avg); drops: 0 p/s (0 p/s avg); hits: -nan%
0:02 0%; send: 0 0 p/s (0 p/s avg); recv: 0 0 p/s (0 p/s avg); drops: 0 p/s (0 p/s avg); hits: -nan%
0:03 0%; send: 0 0 p/s (0 p/s avg); recv: 0 0 p/s (0 p/s avg); drops: 0 p/s (0 p/s avg); hits: -nan%
0:04 0%; send: 0 0 p/s (0 p/s avg); recv: 0 0 p/s (0 p/s avg); drops: 0 p/s (0 p/s avg); hits: -nan%
0:05 0% (68 years left); send: 0 0 p/s (0 p/s avg); recv: 0 0 p/s (0 p/s avg); drops: 0 p/s (0 p/s avg); hits: -nan%
0:06 0% (68 years left); send: 0 0 p/s (0 p/s avg); recv: 0 0 p/s (0 p/s avg); drops: 0 p/s (0 p/s avg); hits: -nan%
0:07 0% (68 years left); send: 0 0 p/s (0 p/s avg); recv: 0 0 p/s (0 p/s avg); drops: 0 p/s (0 p/s avg); hits: -nan%

Same issue here, masscan to. I juse evilscan now: https://github.com/eviltik/evilscan

So even though this issue as closed and I remember @zakird said in another post that this issue will not be fixed.
I found a way to baypass it and my solution is to install a docker on the VPS which is using a none kvm method.
It worked from within the docker (don;t use --vpn)


Text change to https://staging.creativecommons.org/share-your-work/licensing-types-examples/public-domain/

https://staging.creativecommons.org/share-your-work/licensing-types-examples/public-domain/
Under the CC0 description, change:
"Use this universal tool if you are a holder of copyright or database rights, and you wish to waive all of your interests in your work worldwide."
to
"...., and you wish to waive all of your interests that may exist in your work worldwide."

Hit publish too quickly. Additionally, on that same page after the edited sentence mentioned above, add this additional sentence as part of the same paragraph:
Because copyright laws differ around the world, you may use this tool even though you may not have copyright in your jurisdiction, but want to be sure to eliminate any copyrights you may have in other jurisdictions. Let me know if you have questions.

(and the Let me know if you have questions shouldn't be part of the added text, it's a message to jennie :-))

@peterspdx is this still an issue?

Yes, I have been out and haven't had time to make the change. Please feel free to go ahead @little-wow

done


Increment versions for reservations releases

Increment package versions for reservations releases

API change check
API changes are not detected in this pull request.


[Feature Request] Lazy Channels

For my current customer setup I need to read multiple parameters from an S7 PLC in high frequency (10ms)
The parameters change infrequently so i make use of a change filter ({"Type": "Always", "AtLeast": 60_000}).
One parameter is system time however this change rapidly by its nature, however I only need this parameter if any other parameter has changed.
Is the any way to achieve this behaviour?

Would it work for you to include timestamps at value level by setting  the "TimestampLevel" configuration to "Channel" or "Both"?

So basically you need an filter a value, in case the system date-time, if after applying the filters, one or more other values are included in the output set of values.

Yes, i need a piggyback filter. e.g.
{
    "Type": "Piggyback",
    "Channels": [ "OtherChannelA", "OtherChannelB" ],
}

if "Channels" is not set or empty, it should be send, if any other channel of the corresponding schedule is send.

Implemented in version 1.1.1  by adding Condition filtering.
https://github.com/aws-samples/shopfloor-connectivity/tree/mainline/docs#condition-filters
Add the new Condition filters section at top level of config
  "ConditionFilters":{
      "NotJustTheTimeStamp": {
             "Operator" : "only",
             "Value" : "false"
            }
      }
  }

Add
   "ConditionFilter" : "NotJustTheTimeStamp"

to the timestamp channel for your sources

Thanks a lot

What would be the easiest solution for ConditionFilters to send Channel A and B Values only if any other channel has a value?
Or would it work if I apply NotJustTheTimeStamp on channel A and B?

There is a set of new operators you can use to build condition filters
https://github.com/aws-samples/shopfloor-connectivity/tree/mainline/docs#condition-filters
The easiest one is to define a condition using the "present" operator which checks if that other channel has a value and apply that on channel A and B. So if you want to output channel A and channel B only if channel C exists then define this condition:
"ConditionFilters":{
    "OnlyIfCExists":
          {
           "Operator" : "present",
           "Value" : "C"
          }
    }

And then on channel A and B use the "OnlyIfCExists" as the value for "ConditionFilter".
"Channels" : {
  "A": {
          "NodeId": "ns=0;i=2256",
          "ConditionFilter" : "OnlyIfCExists"
      },
 "B": {
          "NodeId": "ns=0;i=2259",
          "ConditionFilter" : "OnlyIfCExists"
      }
}

There are also operator to check if a field is absent, if any, none or all fields in a list are there and you can combine multiple operators using the and and or operator. I've include some examples in the documentation.
p.s. instead of "NotJustTheTimeStamp" in my previous message or the name I've used in this one, you can give a condition filter any name which describes its purpose the best in your situation. The name just identifies a filter, which is one, or a combination of condition filter operators and is used to apply that filter on one or more channels by setting the value of "ConditionFilter" to that name.

The easiest way is to use the "present" operator in a condition filter. The condition filter named "OnlyIfCExists" (use any name you want) will check if channel C has a value in it's source.
  "CondtionFilters" : {
      "OnlyIfCExists" : {
          "Operator" : "present",
           "Value" : "C"
      }
  }

Then apply this filter to channel A and B. A and B are only in the output if C has a value.
 "Channels" : {
  "A": {
          "NodeId": "ns=0;i=2256",
          "ConditionFilter" : "OnlyIfCExists"
      },
 "B": {
          "NodeId": "ns=0;i=2259",
          "ConditionFilter" : "OnlyIfCExists"
      }
  "C": {
          "NodeId": "ns=0;i=2260"
      }
}

If you also have a ChangeFilter or a ValueFilter on a channels, these are applied first on all these channels before the condition filter is applied. So if C has a value, but is filtered out using a Value or a ChangeFilter, the because of the ConditonFilter A and B will filtered out as well.
There is a rich set op operators which let you test if channels are present or absent, or if any, none or all channels in a list of channels are present. It is even possible to combine multiple operators using the or and and operators. See complete list of operators and examples at https://github.com/aws-samples/shopfloor-connectivity/tree/mainline/docs#condition-filters


Create Objective-C version of MonkeyPaws?

It might be worth creating an Objective-C version of MonkeyPaws, so that it can be more easily included in Objective-C only projects.
The test side can still use Swift without forcing the rest of the app to use it, and doesn't really need an Objective-C version, but the embedded visualisation could benefit.
If this is something you would like, please comment.

I'm blocked by the same issue.
I'm importing SwiftMonkeyPaw into an Object-C project, but it throws a linker error:

Undefined symbols for architecture i386:
"OBJC_CLASS$__TtC15SwiftMonkeyPaws10MonkeyPaws", referenced from:
objc-class-ref in AppDelegate.o

I think this error is result from the mix compilation.

What happens exactly? Does it crash, or just not do anything?
If it does nothing, can you try putting a breakpoint in the append function to see if it ever gets called?

The paw's layer didn't display, however, I have found the problem.
I inited MonkeyPaws too earlier, as my Objective-C project modified the UIApplication in the method (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(nullable NSDictionary *)launchOptions
It worked after put the init statement in advance :)

Oh good, at least that is solved then!

Hi guys, help me please!  I did the following step:
//AppDelegate.h
@import SwiftMonkeyPaws;
@property (strong, nonatomic) MonkeyPaws *paws;
//AppDelegate.m
@import SwiftMonkeyPaws;
self.paws = [[MonkeyPaws alloc] initWithView:self.window tapUIApplication:true];
But in AppDelegate.h, failed in line  : @import SwiftMonkeyPaws;
error message: AppDelegate.h:15:1: Use of '@import' when C++ modules are disabled, consider using fmodules and fcxx-modules.
And in AppDelegate.m, no error for @import SwiftMonkeyPaws;
Could you guys help to take a look? How can I fix it? thanks  very much. :D

I might be a bit late, but I've been working on an Objective-C version of MonkeyPaws. Please have a look here: https://github.com/donaldhu/xc-auto-monkey

What if it is a mix and match project of Swift & Obj C. Will SwiftMonkey function the way it should?

Closing due to inactivity. Please reopen the issue if you think it should be addressed.


MegaPatching renamed imports results in an error

Currently, if you import:
from foo import Foo as Bar
Then you will get an error if you attempt to MegaPatch Bar, because it is attempting to find Bar in the foo module.

I think this is resolved if the error is improved since this usually indicates a user error, but its worth seeing if we can support this anyways, at least at first.


Use unique artifact identifier in upload-artifact step

Issue
Fixes https://github.com/gradle/gradle-enterprise-build-validation-scripts/issues/644
Since upload-artifact@v4, uploading again the same artifact name in a workflow fails the step.
Using a hardcoded experiment-X-receipt is not valid anymore
Fix
Use a unique name. This is not so straightforward as some collisions can occur in the case of matrix.
Using the job-index for matrix solves this.
Note that appending the job-index is happening only if job-total > 1 to not suffix artifacts from non matrix scenario
This could hopefully get simplified if/when GitHub starts to expose a unique job-id when running with matrix strategy.
Example
See a summary running with the PR fix here

This branch fixes the issue I was having in our codebase


ESP32S3 configuration USB-Serial output log will unexpectedly restarted (IDFGH-5921)

INSTRUCTIONS
Before submitting a new issue, please follow the checklist and try to find the answer.

[x] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[ ] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[ ] I have searched the issue tracker for a similar issue and not found a similar issue.

Environment

Development Kit:      [ESP32-S3-DevKitC-1]
Kit version (for WroverKit/PicoKit/DevKitC): [v1]
Module or chip used:  [ESP32-S3-WROOM-1]
IDF version :  v4.4-dev-3042-g220590d599
Build System:         [idf.py]
Compiler version : xtensa-esp32s3-elf-gcc.exe (crosstool-NG esp-2021r1) 8.4.0
Operating System:     [Windows]
(Windows only) environment type: [PowerShell].
Using an IDE?: [VSCode]
Power Supply:         [USB]

Problem Description
I have enabled the USB Serial/JTAG controller option in idf.py menuconfig, but after downloading to the chip, using the serial port to connect to the computer, the error of rst:0x10 (RTCWDT_RTC_RST), boot:0x8 (SPI_FAST_FLASH_BOOT) will occur, and it must be plugged in The program can run normally after usb. This will be very troublesome, because the device will not always be plugged in to run when it is running normally. How can I solve this problem?
Debug Logs
The following is the serial output
[2021/09/28 22:43:01.0518] → ESP-ROM:esp32s3-20210327
Build:Mar 27 2021
rst:0x1 (POWERON),boot:0x8 (SPI_FAST_FLASH_BOOT)
SPIWP:0xee
mode:DIO, clock div:1
load:0x3fcd0108,len:0x1414
load:0x403b6000,len:0x8ac
load:0x403ba000,len:0x2e60
entry 0x403b61b8
I (24) boot: ESP-IDF v4.4-dev-3042-g220590d599-dirty 2nd stage bootloader
I (25) boot: compile time 22:29:31
I (25) boot: chip revision: 0
I (26) boot.esp32s3: Boot SPI Speed : 80MHz
I (29) boot.esp32s3: SPI Mode       : DIO
I (33) boot.esp32s3: SPI Flash Size : 8MB
I (37) boot: Enabling RNG early entropy source...
W (41) bootloader_random: RNG for ESP32-S3 not currently supported
I (47) boot: Partition Table:
I (50) boot: ## Label            Usage          Type ST Offset   Length
I (56) boot:  0 nvs              WiFi data        01 02 00009000 00005000
I (63) boot:  1 otadata          OTA data         01 00 0000e000 00002000
I (69) boot:  2 app0             OTA app          00 10 00010000 00330000
I (76) boot:  3 app1             OTA app          00 11 00340000 00330000
I (82) boot:  4 spiffs           Unknown data     01 82 00670000 00190000
I (89) boot: End of partition table
I (92) esp_image: segment 0: paddr=00010020 vaddr=3c070020 size=122c0h ( 74432) map
I (113) esp_image: segment 1: paddr=000222e8 vaddr=3fc93f00 size=03f4ch ( 16204) load
I (117) esp_image: segment 2: paddr=0002623c vaddr=40374000 size=09ddch ( 40412) load
I (126) esp_image: segment 3: paddr=00030020 vaddr=42000020 size=65b00h (416512) map
I (201) esp_image: segment 4: paddr=00095b28 vaddr=4037dddc size=06118h ( 24856) load
I (207) esp_image: segment 5: paddr=0009bc48 vaddr=50000000 size=00010h (    16) load
I (213) boot: Loaded app from partition at offset 0x10000
I (213) boot: Disabling RNG early entropy source...
W (214) bootloader_random: RNG for ESP32-S3 not currently supported
I (221) opi psram: vendor id : 0x0d (AP)
I (224) opi psram: dev id    : 0x02 (generation 3)
I (229) opi psram: density   : 0x03 (64 Mbit)
I (233) opi psram: good-die  : 0x01 (Pass)
I (236) opi psram: Latency   : 0x01 (Fixed)
I (240) opi psram: VCC       : 0x01 (3V)
I (244) opi psram: SRF       : 0x01 (Fast Refresh)
I (249) opi psram: BurstType : 0x01 (Hybrid Wrap)
I (253) opi psram: BurstLen  : 0x01 (32 Byte)
I (257) opi psram: Readlatency  : 0x02 (10 cycles@Fixed)
I (262) opi psram: DriveStrength: 0x00 (1/1)
I (266) spiram: Found 64MBit SPI RAM device
I (270) spiram: SPI RAM mode: sram 40m
I (273) spiram: PSRAM initialized, cache is in normal (1-core) mode.
I (280) cpu_start: Pro cpu up.
I (282) cpu_start: Starting app cpu, entry point is 0x40375220

[2021/09/28 22:43:12.6252] → ESP-ROM:esp32s3-20210327
Build:Mar 27 2021
rst:0x10 (RTCWDT_RTC_RST),boot:0x8 (SPI_FAST_FLASH_BOOT)
SPIWP:0xee
mode:DIO, clock div:1
load:0x3fcd0108,len:0x1414
load:0x403b6000,len:0x8ac
load:0x403ba000,len:0x2e60
entry 0x403b61b8
I (25) boot: ESP-IDF v4.4-dev-3042-g220590d599-dirty 2nd stage bootloader
I (25) boot: compile time 22:29:31
I (25) boot: chip revision: 0
I (26) boot.esp32s3: Boot SPI Speed : 80MHz
I (30) boot.esp32s3: SPI Mode       : DIO
I (34) boot.esp32s3: SPI Flash Size : 8MB
I (38) boot: Enabling RNG early entropy source...
W (42) bootloader_random: RNG for ESP32-S3 not currently supported
I (48) boot: Partition Table:
I (51) boot: ## Label            Usage          Type ST Offset   Length
I (57) boot:  0 nvs              WiFi data        01 02 00009000 00005000
I (64) boot:  1 otadata          OTA data         01 00 0000e000 00002000
I (70) boot:  2 app0             OTA app          00 10 00010000 00330000
I (77) boot:  3 app1             OTA app          00 11 00340000 00330000
I (83) boot:  4 spiffs           Unknown data     01 82 00670000 00190000
I (90) boot: End of partition table
I (93) esp_image: segment 0: paddr=00010020 vaddr=3c070020 size=122c0h ( 74432) map
I (114) esp_image: segment 1: paddr=000222e8 vaddr=3fc93f00 size=03f4ch ( 16204) load
I (117) esp_image: segment 2: paddr=0002623c vaddr=40374000 size=09ddch ( 40412) load
I (126) esp_image: segment 3: paddr=00030020 vaddr=42000020 size=65b00h (416512) map
I (202) esp_image: segment 4: paddr=00095b28 vaddr=4037dddc size=06118h ( 24856) load
I (207) esp_image: segment 5: paddr=0009bc48 vaddr=50000000 size=00010h (    16) load
I (214) boot: Loaded app from partition at offset 0x10000
I (214) boot: Disabling RNG early entropy source...
W (215) bootloader_random: RNG for ESP32-S3 not currently supported
I (221) opi psram: vendor id : 0x0d (AP)
I (225) opi psram: dev id    : 0x02 (generation 3)
I (229) opi psram: density   : 0x03 (64 Mbit)
I (233) opi psram: good-die  : 0x01 (Pass)
I (237) opi psram: Latency   : 0x01 (Fixed)
I (241) opi psram: VCC       : 0x01 (3V)
I (245) opi psram: SRF       : 0x01 (Fast Refresh)
I (249) opi psram: BurstType : 0x01 (Hybrid Wrap)
I (254) opi psram: BurstLen  : 0x01 (32 Byte)
I (258) opi psram: Readlatency  : 0x02 (10 cycles@Fixed)
I (263) opi psram: DriveStrength: 0x00 (1/1)
I (267) spiram: Found 64MBit SPI RAM device
I (271) spiram: SPI RAM mode: sram 40m
I (274) spiram: PSRAM initialized, cache is in normal (1-core) mode.
I (280) cpu_start: Pro cpu up.
I (283) cpu_start: Starting app cpu, entry point is 0x40375220

The following is the project file：
espnow.zip

Hi, Cheng.
Firstly, is it possible to try usb_serial port on linux? I'm not sure that usb_serial port on windows can work properly.
Secondly, are you pretty sure that the issue is caused by usb_serial. Two things can help you check.
-(1) using uart port to see whether there is still a restarting issue or not.
-(2) Run hello_world example with usb_serial port.

Thanks to answer:
(1) I use the usb_serial on windows
(2) I try to run the hello_world example with usb_serial port,but it still report error.
[2021/09/29 14:56:17.8737] → ESP-ROM:esp32s3-20210327
Build:Mar 27 2021
rst:0x1 (POWERON),boot:0x8 (SPI_FAST_FLASH_BOOT)
SPIWP:0xee
mode:DIO, clock div:1
load:0x3fcd0108,len:0x1718
load:0x403b6000,len:0x8ac
load:0x403ba000,len:0x2e60
entry 0x403b61b8
I (24) boot: ESP-IDF v4.4-dev-3042-g220590d599-dirty 2nd stage bootloader
I (25) boot: compile time 14:53:50
I (25) boot: chip revision: 0
I (28) boot.esp32s3: Boot SPI Speed : 80MHz
I (33) boot.esp32s3: SPI Mode       : DIO
I (38) boot.esp32s3: SPI Flash Size : 2MB
I (43) boot: Enabling RNG early entropy source...
W (48) bootloader_random: RNG for ESP32-S3 not currently supported
I (55) boot: Partition Table:
I (59) boot: ## Label            Usage          Type ST Offset   Length
I (66) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (73) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (81) boot:  2 factory          factory app      00 00 00010000 00100000
I (88) boot: End of partition table
I (92) esp_image: segment 0: paddr=00010020 vaddr=3c020020 size=06fc0h ( 28608) map
I (106) esp_image: segment 1: paddr=00016fe8 vaddr=3fc90640 size=025d4h (  9684) load
I (111) esp_image: segment 2: paddr=000195c4 vaddr=40374000 size=06a54h ( 27220) load
I (124) esp_image: segment 3: paddr=00020020 vaddr=42000020 size=14604h ( 83460) map
I (141) esp_image: segment 4: paddr=0003462c vaddr=4037aa54 size=05be8h ( 23528) load
I (147) esp_image: segment 5: paddr=0003a21c vaddr=50000000 size=00010h (    16) load
I (152) boot: Loaded app from partition at offset 0x10000
I (153) boot: Disabling RNG early entropy source...
W (159) bootloader_random: RNG for ESP32-S3 not currently supported
I (166) opi psram: vendor id : 0x0d (AP)
I (170) opi psram: dev id    : 0x02 (generation 3)
I (176) opi psram: density   : 0x03 (64 Mbit)
I (181) opi psram: good-die  : 0x01 (Pass)
I (185) opi psram: Latency   : 0x01 (Fixed)
I (190) opi psram: VCC       : 0x01 (3V)
I (195) opi psram: SRF       : 0x01 (Fast Refresh)
I (200) opi psram: BurstType : 0x01 (Hybrid Wrap)
I (206) opi psram: BurstLen  : 0x01 (32 Byte)
I (211) opi psram: Readlatency  : 0x02 (10 cycles@Fixed)
I (217) opi psram: DriveStrength: 0x00 (1/1)
I (222) spiram: Found 64MBit SPI RAM device
I (227) spiram: SPI RAM mode: sram 40m
I (231) spiram: PSRAM initialized, cache is in normal (1-core) mode.
I (238) cpu_start: Pro cpu up.
I (242) cpu_start: Starting app cpu, entry point is 0x40375124

[2021/09/29 14:56:29.2805] → ESP-ROM:esp32s3-20210327
Build:Mar 27 2021
rst:0x10 (RTCWDT_RTC_RST),boot:0x8 (SPI_FAST_FLASH_BOOT)
SPIWP:0xee
mode:DIO, clock div:1
load:0x3fcd0108,len:0x1718
load:0x403b6000,len:0x8ac
load:0x403ba000,len:0x2e60
entry 0x403b61b8
I (25) boot: ESP-IDF v4.4-dev-3042-g220590d599-dirty 2nd stage bootloader
I (25) boot: compile time 14:53:50
I (26) boot: chip revision: 0
I (29) boot.esp32s3: Boot SPI Speed : 80MHz
I (34) boot.esp32s3: SPI Mode       : DIO
I (39) boot.esp32s3: SPI Flash Size : 2MB
I (43) boot: Enabling RNG early entropy source...
W (49) bootloader_random: RNG for ESP32-S3 not currently supported
I (56) boot: Partition Table:
I (59) boot: ## Label            Usage          Type ST Offset   Length
I (67) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (74) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (81) boot:  2 factory          factory app      00 00 00010000 00100000
I (89) boot: End of partition table
I (93) esp_image: segment 0: paddr=00010020 vaddr=3c020020 size=06fc0h ( 28608) map
I (107) esp_image: segment 1: paddr=00016fe8 vaddr=3fc90640 size=025d4h (  9684) load
I (112) esp_image: segment 2: paddr=000195c4 vaddr=40374000 size=06a54h ( 27220) load
I (124) esp_image: segment 3: paddr=00020020 vaddr=42000020 size=14604h ( 83460) map
I (142) esp_image: segment 4: paddr=0003462c vaddr=4037aa54 size=05be8h ( 23528) load
I (147) esp_image: segment 5: paddr=0003a21c vaddr=50000000 size=00010h (    16) load
I (153) boot: Loaded app from partition at offset 0x10000
I (154) boot: Disabling RNG early entropy source...
W (159) bootloader_random: RNG for ESP32-S3 not currently supported
I (166) opi psram: vendor id : 0x0d (AP)
I (171) opi psram: dev id    : 0x02 (generation 3)
I (176) opi psram: density   : 0x03 (64 Mbit)
I (181) opi psram: good-die  : 0x01 (Pass)
I (186) opi psram: Latency   : 0x01 (Fixed)
I (191) opi psram: VCC       : 0x01 (3V)
I (196) opi psram: SRF       : 0x01 (Fast Refresh)
I (201) opi psram: BurstType : 0x01 (Hybrid Wrap)
I (206) opi psram: BurstLen  : 0x01 (32 Byte)
I (211) opi psram: Readlatency  : 0x02 (10 cycles@Fixed)
I (217) opi psram: DriveStrength: 0x00 (1/1)
I (222) spiram: Found 64MBit SPI RAM device
I (227) spiram: SPI RAM mode: sram 40m
I (232) spiram: PSRAM initialized, cache is in normal (1-core) mode.
I (239) cpu_start: Pro cpu up.
I (242) cpu_start: Starting app cpu, entry point is 0x40375124

[2021/09/29 14:56:40.7425] → ESP-ROM:esp32s3-20210327
Build:Mar 27 2021
rst:0x10 (RTCWDT_RTC_RST),boot:0x8 (SPI_FAST_FLASH_BOOT)
SPIWP:0xee
mode:DIO, clock div:1
load:0x3fcd0108,len:0x1718
load:0x403b6000,len:0x8ac
load:0x403ba000,len:0x2e60
entry 0x403b61b8
I (25) boot: ESP-IDF v4.4-dev-3042-g220590d599-dirty 2nd stage bootloader
I (25) boot: compile time 14:53:50
I (26) boot: chip revision: 0
I (29) boot.esp32s3: Boot SPI Speed : 80MHz
I (34) boot.esp32s3: SPI Mode       : DIO
I (39) boot.esp32s3: SPI Flash Size : 2MB
I (43) boot: Enabling RNG early entropy source...
W (49) bootloader_random: RNG for ESP32-S3 not currently supported
I (56) boot: Partition Table:
I (59) boot: ## Label            Usage          Type ST Offset   Length
I (67) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (74) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (81) boot:  2 factory          factory app      00 00 00010000 00100000
I (89) boot: End of partition table
I (93) esp_image: segment 0: paddr=00010020 vaddr=3c020020 size=06fc0h ( 28608) map
I (107) esp_image: segment 1: paddr=00016fe8 vaddr=3fc90640 size=025d4h (  9684) load
I (112) esp_image: segment 2: paddr=000195c4 vaddr=40374000 size=06a54h ( 27220) load
I (124) esp_image: segment 3: paddr=00020020 vaddr=42000020 size=14604h ( 83460) map
I (142) esp_image: segment 4: paddr=0003462c vaddr=4037aa54 size=05be8h ( 23528) load
I (147) esp_image: segment 5: paddr=0003a21c vaddr=50000000 size=00010h (    16) load
I (153) boot: Loaded app from partition at offset 0x10000
I (154) boot: Disabling RNG early entropy source...
W (159) bootloader_random: RNG for ESP32-S3 not currently supported
I (166) opi psram: vendor id : 0x0d (AP)
I (171) opi psram: dev id    : 0x02 (generation 3)
I (176) opi psram: density   : 0x03 (64 Mbit)
I (181) opi psram: good-die  : 0x01 (Pass)
I (186) opi psram: Latency   : 0x01 (Fixed)
I (191) opi psram: VCC       : 0x01 (3V)
I (196) opi psram: SRF       : 0x01 (Fast Refresh)
I (201) opi psram: BurstType : 0x01 (Hybrid Wrap)
I (206) opi psram: BurstLen  : 0x01 (32 Byte)
I (211) opi psram: Readlatency  : 0x02 (10 cycles@Fixed)
I (217) opi psram: DriveStrength: 0x00 (1/1)
I (222) spiram: Found 64MBit SPI RAM device
I (227) spiram: SPI RAM mode: sram 40m
I (232) spiram: PSRAM initialized, cache is in normal (1-core) mode.
I (239) cpu_start: Pro cpu up.
I (242) cpu_start: Starting app cpu, entry point is 0x40375124

sdkconfig.zip
In the end,i did not think the problem was windows or linux problem ,because i did not plug in the usb and just used the serial port to view the log,thanks.

I get the same issue on linux. If I set the console to use USB/JTAG and don't use the USB port (plug it in to the UART port instead), I get constant bootloops. I also don't get any console output from the program once it's running. I'm running the ESP-Rainmaker led_light example on the ESP32-S3-DevkitC-1, 8MB, no PSRAM version

I get the same issue on linux. If I set the console to use USB/JTAG and don't use the USB port (plug it in to the UART port instead), I get constant bootloops. I also don't get any console output from the program once it's running. I'm running the ESP-Rainmaker led_light example on the ESP32-S3-DevkitC-1, 8MB, no PSRAM version

So I think the problem is on esp-idf, and it has nothing to do with the OS used, but the official has never responded,  I don’t know how to solve the problem.

I get the same issue on linux. If I set the console to use USB/JTAG and don't use the USB port (plug it in to the UART port instead), I get constant bootloops. I also don't get any console output from the program once it's running. I'm running the ESP-Rainmaker led_light example on the ESP32-S3-DevkitC-1, 8MB, no PSRAM version

Could you please show me the port on linux you are using. with ls -l /dev/tty*

This should be fixed by 4f5ab6be.


Allow to customize all settings on start page

First time to open the application, allow user to customize all setting when they choose

https://matrix.to/#/!oxOZZtibVUXxXtdPJS:decred.org/$TB_P_kASilD1-ZR6AArHUxVZMr-j8-p3U2VKMktc2wk?via=decred.org&via=matrix.org

oh wow, I think I chose Advanced then and just forgot it was there. It's great it's already there! But, I think the meaning of this screen (and effects that follow) could be improved with what I've described above, namely

(based on your screenshot from above) seems like the category of users who "want most user-friendly defaults, yet an ability to customize or check ALL the settings there are later on" is probably missing ?
so, I think it makes more sense to differentiate

not between those who like to tweak settings, and those who dont (which what it currently seems to suggest)
but rather between those who prefer best UX, and those who want max privacy


chore(master): release 1.1.1

:robot: I have created a release beep boop
1.1.1 (2023-11-17)
Bug Fixes

cursor: Current character not correctly obtained (ed4e988)
highlight: Identify custom highlight group correctly (#1628) (da15b14)
viewport: Avoid awaiting the hack to refresh viewport (d5cb701)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/vscode-neovim/vscode-neovim/releases/tag/v1.1.1 :sunflower:


IndexMap: key not found when using entity resolver

Expected Behavior
When using federation V2 with the dynamic schema, I expect my entity resolver to return an array and for that array to be used as the response
Actual Behavior
I receive the following error:
thread 'tokio-runtime-worker' panicked at 'IndexMap: key not found', /home/adam/.cargo/registry/src/index.crates.io-6f17d22bba15001f/async-graphql-6.0.4/src/dynamic/resolve.rs:366:45

My code:
        FieldFuture::new(async move {
            println!("ENTITY RESOLVER");

            let datasource = ctx.data::<Datasource>()?;
            let user = ctx.data::<Option<User>>()?;
            let representations = ctx.args.try_get("representations").unwrap();
            let representations2 = representations.list().unwrap();
            let mut values = Vec::new();
            for item in representations2.iter() {

                let item = item.object()?;

                if val.is_some() {
                    println!("here");
                    values.push(FieldValue::owned_any(Account {
                        id: Some("cf896160-a53b-4955-b592-1028844eca21".to_string()),
                        username: Some("adminuser".to_string())
                    }))

                }
            }

            Ok(Some(FieldValue::list(values)))
        })

Specifications

Version: 6.0.4

It really doesn't work, I'm working on fixing it. 🙂

fixed in v6.0.5

Beautiful, I can confirm it works but there are two things I want to mention

You need to update the documentation so that you include the with_type to FieldValue. For example:

        FieldFuture::new(async move {
            println!("ENTITY RESOLVER");

            let datasource = ctx.data::<Datasource>()?;
            let user = ctx.data::<Option<User>>()?;
            let representations = ctx.args.try_get("representations").unwrap();
            let representations2 = representations.list().unwrap();
            let mut values = Vec::new();
            for item in representations2.iter() {

                let item = item.object()?;

                if val.is_some() {
                    println!("here");
                    values.push(FieldValue::owned_any(Account {
                        id: Some("cf896160-a53b-4955-b592-1028844eca21".to_string()),
                        username: Some("adminuser".to_string())
                    }).with_type("account"))

                }
            }

            Ok(Some(FieldValue::list(values)))
        })

It seems that you always need to return an array at the entity resolver, even if you're not expecting an array. For example.

{
  "data": {
    "address": [
      {
        "state": "Somewhere",
        "street1": "1234 Sesame Street",
        "city": null,
        "updated_at": "2023-08-25 20:58:56.693546",
        "created_at": "2023-08-25 20:58:56.693546",
        "updated_by": {
          "id": "cf896160-a53b-4955-b592-1028844eca21",
          "username": "admin",
          "created_at": "2023-08-25 20:52:28.680463",
          "updated_at": "2023-08-25 20:52:28.680463"
        }
      }
    ]
  }
}

updated_by is an entity resolver, even though it returns an non-array. At the entity resolver, I still need to return an array which doesn't make too much sense. If that's just the graphQL spec, I guess it's fine, I just found it odd.
return Ok(Some(FieldValue::list(vec![c.with_type("account")])))
-----
need to return a vec even though it's just expecting a single value

Actually after some thought. Leaving it an array makes sense as it keeps it consistent in all cases. Closing the issue.


Quality: quality improvements on Get-WindowsUpdateLog.md

PR Summary

resolves #3320
quality improvements from PSHSummit doc-a-thon

PR Checklist

[x] Descriptive Title: This PR's title is a synopsis of the changes it proposes.
[x] Summary: This PR's summary describes the scope and intent of the change.
[x] Contributor's Guide: I have read the [contributors guide][contrib].
[x] Style: This PR adheres to the [style guide][style].

@microsoft-github-policy-service agree


San Francisco on El Capitan

3ac69500b0627067db59ea60f02f456fee6791be
font-family: 'SF UI Text' is not working on El Capitan Beta 15A263e
font-family: '.SFNSText-Regular' does work.

I think SF UI Text works for the downloaded version of San Francisco. But good point about also adding .SFNSText-Regular that comes with El Capitan Beta.

From what I can tell this doesn't match the system's behaviour exactly. For smaller fonts, OS X uses the display variant of SF, not regular. Also fonts prefixed prefixed with a dot are considered private. The most stable approach would be to use -apple-system, but this doesn't seem to work in Chrome yet.

Apparently the Blink team considers adding a placeholder string BlinkMacSystemFont. However, it is currently reverted. See https://codereview.chromium.org/1266403002/.

The OS is not only switching between text and display. Turns out there is even more trickery involved! Compare https://jsbin.com/fumali/1/edit?css,output.

The most stable approach would be to use -apple-system, but this doesn't seem to work in Chrome yet.

That would be nice if something like that gets supported. There is another approach to get the system font with font: menu https://github.com/atom/atom/pull/8817. Not sure if that also would change between text and display.

@trkoch Here are the values for letter spacing adjustments

BlinkMacSystemFont has been reintroduced and now works in Chrome (confirmed with 46.0.2490.52). When using this symbolic font, Chrome automatically switches between text and display. However Chrome does not seem to apply the letter spacing adjustments mentioned by @williambout.

Awesome.. let's keep it in mind for when it lands in Electron/Atom.

SF UI tracking adjustment filed here — https://code.google.com/p/chromium/issues/detail?id=578799


python310Packages.aiomysensors: 0.3.1 -> 0.3.2

Description of changes
https://github.com/MartinHjelmare/aiomysensors/blob/main/CHANGELOG.md#v032-2022-10-16

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 199590 run on x86_64-linux 1

6 packages built:

python310Packages.aiomysensors
python310Packages.asyncio-mqtt
python310Packages.python-openzwave-mqtt
python39Packages.aiomysensors
python39Packages.asyncio-mqtt
python39Packages.python-openzwave-mqtt


Remove reference to wrapEmitter in README.md

This feature have been extracted into its own module: emitter-listener

+1

Rebased and landed as 253fb2279128eb5783c8c0363962762fd34434bc. Thanks, @watson!


Oplog Tailing

This enables Oplog Tailing by default for the high-performance example

May need to add instructions on how to setup mongo for oplog tailing.   By default,  I don't think it is enabled.

You mean creating a replica set? That's the only thing you have to do.


ponyc: 0.50.0 -> 0.54.0

ponyc: 0.50.0 -> 0.54.0

Release: https://github.com/ponylang/ponyc/releases/tag/0.54.0

pony-corral: 0.6.1 -> unstable-2023-02-11

upgrade to unstable-2023-02-11 is required because pony 0.54.0 breaks it.

CC: @redvers

Result of nixpkgs-review pr 220073 run on x86_64-linux 1

2 packages built:

pony-corral
ponyc

Result of nixpkgs-review pr 220073 run on x86_64-linux 1

2 packages built:

pony-corral
ponyc

@GabrielDougherty Feel free to review this.


Helm repo doesn't work: 404 error

Hi everyone,
In some case the helm repo https://summerwind.github.io/actions-runner-controller stopped working.
helm repo update

Hang tight while we grab the latest from your chart repositories...
...Unable to get an update from the "actions-runner-controller" chart repository (https://summerwind.github.io/actions-runner-controller):
	failed to fetch https://summerwind.github.io/actions-runner-controller/index.yaml : 404 Not Found
...

Expected:
helm repo update

Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from "actions-runner-controller" chart repository

Thanks!

Adding a +1 here, getting the same 404 error

same here

Ditto!

Oops, this is due to the repository transfer. Please try the following URL.
https://actions-runner-controller.github.io/actions-runner-controller/index.yaml

That works well for me, thanks for the quick response!

Oops, this is due to the repository transfer. Please try the following URL.
https://actions-runner-controller.github.io/actions-runner-controller/index.yaml

Thanks, works
P.s. Please don't forget change it in README.md


 For them even 1ms means more than 5% change (which is the
default). 1ms actually can result from rounding issue. Using absolute
values here is more suitable than the alarming large percentage swing.
This also adjusted some existing threshold accordingly.

This is for addressing the annoying flagging on https://perf.iree.dev/serie?IREE?MobileNetV3Small [fp32,imagenet] (TensorFlow) full-inference with IREE-Vulkan @ Pixel-4 (GPU-Adreno-640).


docs: update homebrew-pypi-poet instructions

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

Replaces up to https://github.com/Homebrew/brew/pull/9391
This PR modifies the instructions for python resource generation using homebrew-pypi-poet so installing virtualenvwrapper or virtualenv isn't necessary.
Instead, we can use the builtin venv module via python3 -m venv to more easily and reliably find resources.
CC: @claui, @dtrodrigues, and @MikeMcQuaid

Review period will end on 2021-01-08 at 18:28:33 UTC.

Review period will end on 2021-01-08 at 18:28:33 UTC.

Thanks @Rylan12, @claui!

Thanks @Rylan12, @claui!

Review period ended.

Review period ended.


Fixing OSGi resolution issues caused by jdk.internal.vm.annotation

Addressed https://github.com/openzipkin/brave/issues/1243, checked with 5.12.5-SNAPSHOT builds (local), the issue is gone. Thanks to  @dkulp and @adriancole .

@reta thanks! can you verify that this doesn't recurse? ex the jars that depend on brave don't end up with the same problem. you can search for bnd files here or in console https://github.com/openzipkin/brave/find/master
this was one reason I was wondering if this could be handled at the plugin level.. possibly we should raise a JIRA on it? wdyt

@adriancole yes, sure, so far brave seems to be the only module which needs this dependency. CXF uses
io.zipkin.zipkin2:zipkin
io.zipkin.reporter2:zipkin-reporter
io.zipkin.reporter2:zipkin-reporter-brave
io.zipkin.brave:brave
io.zipkin.brave:brave-instrumentation-http

I have checked also
io.zipkin.brave:brave-instrumentation-messaging
io.zipkin.brave:brave-instrumentation-rpc

Others require dedicated JARs but none of them required jdk.internal.vm.annotation

super weird.. I wonder then what is causing the dep on jdk.internal.vm.annotation...

@adriancole I found it, it is @Hidden annotation which comes from lambda- generated code for zipkin2.reporter.okhttp3.OkHttpSender class, it is in zipkin-sender-okhttp3 module

@reta cool. we can un-lambda that. great detective work!

So we only had that one lambda in all of the src/main of zipkin-reporter. I used intellij's "structural search" to verify the one you said was it (outside tests and benchmarks).

It is still strange, though... isn't it?.. There's no compile dep on okhttp from io.zipkin.brave:brave

@adriancole that's correct, so if you disassemble it using javap, you will see the jdk/internal/vm/annotation/Hidden in there (at the end of the snippet, this class seems to be generated by retrolambda), for example (this is taken from 2.15.0 version):
$ javap -v OkHttpSender\$\$Lambda\$1.class
Classfile /C:/Users/reta/Downloads/OkHttpSender$$Lambda$1.class
  Last modified May 17, 2020; size 709 bytes
  MD5 checksum 10eb8be2fbfee9f83f6a2b3b5b0193dd
  Compiled from "OkHttpSender.java"
final class zipkin2.reporter.okhttp3.OkHttpSender$$Lambda$1 implements java.util.concurrent.ThreadFactory
  minor version: 0
  major version: 51
  flags: (0x1030) ACC_FINAL, ACC_SUPER, ACC_SYNTHETIC
  this_class: #2                          // zipkin2/reporter/okhttp3/OkHttpSender$$Lambda$1
  super_class: #4                         // java/lang/Object
  interfaces: 1, fields: 1, methods: 4, attributes: 1
Constant pool:
   #1 = Utf8               zipkin2/reporter/okhttp3/OkHttpSender$$Lambda$1
   #2 = Class              #1             // zipkin2/reporter/okhttp3/OkHttpSender$$Lambda$1
   #3 = Utf8               java/lang/Object
   #4 = Class              #3             // java/lang/Object
   #5 = Utf8               java/util/concurrent/ThreadFactory
   #6 = Class              #5             // java/util/concurrent/ThreadFactory
   #7 = Utf8               OkHttpSender.java
   #8 = Utf8               instance
   #9 = Utf8               Lzipkin2/reporter/okhttp3/OkHttpSender$$Lambda$1;
  #10 = Utf8               <init>
  #11 = Utf8               ()V
  #12 = NameAndType        #10:#11        // "<init>":()V
  #13 = Methodref          #4.#12         // java/lang/Object."<init>":()V
  #14 = Utf8               newThread
  #15 = Utf8               (Ljava/lang/Runnable;)Ljava/lang/Thread;
  #16 = Utf8               Ljdk/internal/vm/annotation/Hidden;

What is mysterious though ... I was able to reproduce it only once locally by building the 2.15.1-SNAPSHOT ... I am still looking why it happened with 2.15.0 but not (anymore) with 2.15.1-SNAPSHOT. Building out of release tag (release-2.15.0) does not generate same bytecode either ....

It is still strange, though... isn't it?.. There's no compile dep on okhttp from io.zipkin.brave:brave

Not exactly but in test scope only:
$ mvn dependency:tree -pl :brave | grep okhttp3
[INFO] +- com.squareup.okhttp3:mockwebserver:jar:4.7.2:test
[INFO] |  \- com.squareup.okhttp3:okhttp:jar:4.7.2:test
[INFO] +- io.zipkin.reporter2:zipkin-sender-okhttp3:jar:2.15.0:test

opened https://github.com/luontola/retrolambda/issues/160 actually I found the culprit... we also had one here in BaggagePropagation

awesome, thanks a lot @adriancole , I think we can close this pull request

@reta I added a commit, that after you build https://github.com/openzipkin/zipkin-reporter-java/pull/179 can verify if all clear?

nah can't close this as there's one here :D

@adriancole the MANIFEST.MF (for brave and zipkin-sender-okhttp3) are good, CXF build passes against the snapshots with no issues, thanks!

thanks.. I'll go ahead and ship this as don't want people pinned.

Mind having a look? https://github.com/openzipkin/brave/pull/1252


Fix exit button link on introduction page

What is the context of this PR?
Fixes the missing exit button link on the introduction page.
How to review

Launch test_introduction.json
Ensure when you click Exit you are signed out and taken to the log-out URL.

Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

This is a slight pain, as usually I would expect this to be done in context of the handler, in this case Content, and then match the pattern we have used in multiple other places in templates (i.e {% set sign_out_url = content.sign_out_url %}) However I am thinking because this is used on at least error pages, that because they don't pass any content on rendering they would need updating as well?

I think we need to look to do this as a clean-up piece, we seem to only use content.sign_out_url in post submission pages. But we do have url_for littered though the templates. I prefer to pass to the templates the context it needs instead of url_for, but I think it needs a wider clean up to remove all the ones we can. We should be able to remove all, including error pages, as they still call render templates method, so we can pass it.
We also need to clean up the actual context passes, in question pages, we pass it as top level arguments, in post submission pages it is sent as content.* which is inconsistent. So if I were to pass this to template helpers now, it would be top level even though existing ones on post submission use content.sign_out_url.


Error when creating image for image-gallery from image in different resource group

This is autogenerated. Please review and update as needed.
Error when creating image for image-gallery from image
I try to create an image for an image-gallery from an image in a different resource group.
az sig image-version create
az sig image-version create \
   --resource-group rg-name \
   --gallery-name gallyer-name\
   --gallery-image-definition image-definition \
   --gallery-image-version 1.0.2 \
   --managed-image "/subscriptions/id/resourceGroups/rg-name/providers/Microsoft.Compute/images/image-name"

Errors:
CLIInternalError: The command failed with an unexpected error. Here is the traceback:
'NoneType' object is not subscriptable
Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.17.1/libexec/lib/python3.8/site-packages/knack/cli.py", line 215, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/local/Cellar/azure-cli/2.17.1/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 659, in execute
    raise ex
  File "/usr/local/Cellar/azure-cli/2.17.1/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 722, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/local/Cellar/azure-cli/2.17.1/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 715, in _run_job
    six.reraise(*sys.exc_info())
  File "/usr/local/Cellar/azure-cli/2.17.1/libexec/lib/python3.8/site-packages/six.py", line 703, in reraise
    raise value
  File "/usr/local/Cellar/azure-cli/2.17.1/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 693, in _run_job
    result = cmd_copy(params)
  File "/usr/local/Cellar/azure-cli/2.17.1/libexec/lib/python3.8/site-packages/azure/cli/core/commands/__init__.py", line 330, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.17.1/libexec/lib/python3.8/site-packages/azure/cli/core/__init__.py", line 784, in default_command_handler
    return op(**command_args)
  File "/usr/local/Cellar/azure-cli/2.17.1/libexec/lib/python3.8/site-packages/azure/cli/command_modules/vm/custom.py", line 3276, in create_image_version
    external_bearer_token = external_tokens[0][0] + ' ' + external_tokens[0][1]
TypeError: 'NoneType' object is not subscriptable

To Reproduce:
Create a virtual machine, deallocate it, generalize it, create an image from it.
Expected Behavior
It should create an image inside an image-gallery.
Environment Summary
macOS-11.1-x86_64-i386-64bit
Python 3.8.7
Installer: HOMEBREW

azure-cli 2.17.1

Extensions:
azure-devops 0.18.0

I think this can be closed as this error is pretty similar to the one here: #16466.
So I guess it will be fixed with the new version.
Can you guys confirm this?

I think this can be closed as this error is pretty similar to the one here: #16466.
So I guess it will be fixed with the new version.
Can you guys confirm this?

hi @qwordy could you pls help to confirm?

hi @qwordy could you pls help to confirm?

@TobiMichael96 Thank you for reporting this issue. It was fixed by https://github.com/Azure/azure-cli/commit/72c2203957760402e115e2056d018aa7a520d7bc.
Release date: 01/19/2021

@TobiMichael96 Thank you for reporting this issue. It was fixed by https://github.com/Azure/azure-cli/commit/72c2203957760402e115e2056d018aa7a520d7bc.
Release date: 01/19/2021

I think this can be closed as this error is pretty similar to the one here: #16466.
So I guess it will be fixed with the new version.
Can you guys confirm this?

Yes, you are right.

I think this can be closed as this error is pretty similar to the one here: #16466.
So I guess it will be fixed with the new version.
Can you guys confirm this?

Yes, you are right.


Updates to nr1-workshop setup guide

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.