Always give the owner of a pod control access to resources

See https://github.com/solid/node-solid-server/issues/1489
This requires:

[ ] Tracking the pod owner
[ ] Creating an AllowOwnerControlAuthorizer

handles all resources
allows Control when authenticated as the owner

[ ] Configuring the AllowOwnerControlAuthorizer

I see two possible strategies:

Either AllowOwnerControlAuthorizer is a wrapper that takes the existing authorizer, and adds Control to whatever permissions returned by that authorizer.
Either we build a CombinedAuthorizer that invokes AllowOwnerControlAuthorizer and AllowOwnerControlAuthorizer in parallel, and creates the union of the permissions.

In the first, we need to make the arbitrary decision which authorizers are wrappers and which are not (unless we all make them wrappers!). The second seems more flexible, but is an extra component and relies on the possibility of creating a union.

This is going to be easier when using a server with dynamic pod configuration (the one where there's a potentially different config per pod). We already store a link between owner and pod address there. The ConfigPodInitializer uses this information to instantiate all the stores and to add the corresponding routes to a KeyValueStorage. This storage is then used by the BaseUrlRouterRule to know where to send each request.
We could make a similar storage to link routes to owners (or put both those things in the same storage). The AllowAgentAuthorizer could then check if the incoming request overlaps with any of the stored pod URLs (just like the router rule) to find the owner.

The owner must have control access, so that they can get themselves out of any situation.  This does NOT mean that they should always have all access.   An owner can use their control access to deprive themselves of read and/or write and/or append access if they wantHeads up: .   Setting a file read-only should be quite common, or append-only for the proper operation of various kinds of workflow.  Changed title to what it should be.

Agreed with Control access; this means we will have to change the proposed steps.
Rather than a chain that gives owners all permissions, it should be a wrapper that adds Control permissions to the permissions granted by the underlying authorization system.

Since the dynamic pod part is not really used we can look at the IDP for a solution. We can store WebID <-> pod associations in a Storage object there and then re-use that in the authorizer. We could also store it in the accountStore together with the other IDP information (email/passwordhash/webid) but that would remove the option of having a pod without registering with the IDP (although we might want this anyway in the future to provide pod controls?).
Regarding the authorizer: Authorizers are AsyncHandlers and our current authorizer already is a waterfall handler: https://github.com/solid/community-server/blob/1173f98b5db64fcbb63436ed574697eaa6b1fecd/config/ldp/authorization/webacl.json#L9-L10
So this new authorizer can just be part of the waterfall and only handle requests where the only permission required is control, the credentials contain a WebID, and that WebID is associated with a pod that is a substring of the target identifier.

The owner must have control access, so that they can get themselves out of any situation. This does NOT mean that they should always have all access. An owner can use their control access to deprive themselves of read and/or write and/or append access if they wantHeads up: . Setting a file read-only should be quite common, or append-only for the proper operation of various kinds of workflow. Changed title to what it should be.

How does the proposed solution address this? Indeed, there are use-cases where it is not desirable that the owner of a pod has — or can give him/her-self — full access.

How does the proposed solution address this? Indeed, there are use-cases where it is not desirable that the owner of a pod has — or can give him/her-self — full access.

The solution covers the first part: they can deny themselves read/write access, but indeed not the second part of your statement since they will always have control access. But that is also not what is being requested here afaik. The request is that an owner should always have control access to the resources in the pod.

It may not be what is originally requested in this issue, but I'd like to put the attention on it, since the current solution neglects some very desirable use cases.

One option we could always add in the future is the option to revoke or change ownership of a pod, which would then disable the control access specified here.

Ok! This would, however only work pod-wide, which makes me wonder: is there a reason ownership is defined per pod instead of per container or per resource?
One often recurring use case I have in mind is the user delegating control over a container of his/her pod to an organisation or service which can write mission-critical data to it, only readable but not writable by the user.

Reason is simplicity 😅
This is pretty much the "simplest" solution to this problem, to make sure there is one at least in case someone gets locked out of their pod by accident.
At this point we're not making use yet of the new solid:owner predicate mentioned in the linked spec issue. (Which makes me realize we should probably add that triple to newly created pods after this issue is resolved).
That's not to say we shouldn't do this or that this will never be added. We could for example make a more advanced authorizer that goes looking for ownership triples somehow and also have the possibility to edit that metadata, but there are several questions that will need to be answered before doing that. This is also getting a bit off-topic for this issue since this was specifically about pod ownership so if this is needed I think a separate issue might be better suited.

Added in #938


Documentation is out-of-date

The documentation at http://clux.github.io/loggerv appears to be out-of-date/sync with v0.6. The old *_color methods are listed.
Is there a specific or automated procedure for generating the documentation with rustdoc?

Oh, whoops. Yeah, thought I pushed docs.
I usually follow this: https://github.com/clux/dotfiles/blob/ad3bf1f5419f5763f314e65b8e66f622a9e4e988/.functions#L410-L416 which uses the ghp-import pip module. I think you'll have access to basically follow those steps now that you are a collaborator. Feel free to try.

Okay, I followed the function you provided to update the docs. It did not quite work for me without some modification. I had to use the HTTP address for the repository and the ghp-import did not appear to work, so I had to manually push the docs to the gh-pages. I have not used the ghp-import module before, so I could be me. Regardless, I was able to update the documentation.

Ah, yeah. If you're cloning with http rather than ssh, that would cause that. Thanks for updating it :-)


allow DomainPool.generate_sequence() to take as input old sequence and mutate it

Currently, each new sequence returned from DomainPool.generate_sequence() is independent from the previous one used for that domain. But one may wish to choose the new sequence to be related to the old in some way, e.g., being some some edit distance from it. To support this, pass a (possibly None) parameter as input, which is the previous assigned DNA sequence.

See also comments on #35.


Dapr for .NET Developers - Chapter 8: Bindings

Chapter 8 of the Dapr for .NET devs ebook about bindings.

This is chapter 8 about the Bindings building block.
@nishanil / @IEvangelist please review. Thanks!

This is chapter 8 about the Bindings building block.
@nishanil / @IEvangelist please review. Thanks!

There are several compile errors with the C# snippets include, and a few other minor nits.

@IEvangelist I'll review your suggestions a.s.a.p.
Thanks!

@IEvangelist I've applied your suggestions. Please review.


I want to sent the form object back to the action but not sure how

First of all congrats for this awesome addon.
I have a bs-form with an action attribute pointing to my action in the route. One thing I haven't found how to do it is to send back (data up) the form object to the action. Using model=formObject doesn't work either. Do you know how I could achieve that? I think it's rather crucial since most actions in Ember 2.0 take place in the route and not in the controller. I can explicitly ask the controller from the route to get the formObject but I would prefer return it from the component's action directly.

The solution to this is

wrap the form in a component
pass the data down to the component from route as 1 object, like a formObject
set the action in the ember-bootstrap form and catch the action from the component
resend the action to the controller/router with the modified formObject

Hi @vasilakisfil , do you have the example code?


In multiprocess mode, ensure that metrics initialise to the correct file

Without this, when forking, children can initialise metrics to the
cached master pid file. This is racy, and can corrupt the master's
mmaped file, breaking all subsequent collections.

Thanks!


fat: attestation everywhere

Bring in the Bifold changes that allow attestation to work form anywhere in the application.

@jleach some sonarcloud issues


Allow for running tests on Windows and fix tests

This fixes running tests on Windows and the tests themselves.
This should also allow #7 to be merged.

@busches mind rebasing; looks like you have a timeout too; you might want to pull the 5s timeout out to 10s?

CLA is valid!

@bcoe I can rebase, I'm a bit confused on why this wasn't just merged, since it has the exact same fixes as #12 and was done a month earlier.

Rebase is done.

@busches don't suppose I could convince you to add an appveyor config 👍
thanks a bunch for the contribution.


Firestore memory footprint seems too high

[REQUIRED] Step 1: Describe your environment

Xcode version:  12.4
Firebase SDK version: 7.10.0
Installation method: Swift Package Manager
Firebase Component: Firestore

Steps to reproduce:
I created an iOS App extension, Network extension to be more precise, this extension is limit by iOS to run with a maximum usage of 15 MB of RAM.
I have noticie that just including FirebaseFirestore and FirebaseFirostoreSwift-Beta with SPM and running
FirebaseApp.configure()
Make the memory ram usage go up by around 6 MB without running any query or snapshot yet. Comparing this to adding FirebaseAuth, FirebaseCrashlytics or FirebaseeAnalytics, seems high, the other seems to be around 1 MB.
Do you guys think this is normal behavior? or could be a bug that make Firestore memory usage higher compare to the rest? if  it's the normal usage, is there anything possible to setup to reduce the memory usage and still use the SDK on the App Extension?
Thanks a lot for your answer and time

Hi @unxavi. Thank you for reporting this issue. I don't have any immediate response but will look into it and reply back. My first thought though is that I'm not terribly surprised that Firestore's memory usage is significantly larger than the other products. Its relative size is larger and is has quite a few dependencies.
If you have a chance, could you use some of the memory debugging tools from xcode to see if there are any obvious memory usage issues? (e.g. Gathering Information About Memory Use).

@unxavi Would you be able to create a project in GitHub that I could clone to reproduce? I'd like to make sure I'm using the same setup as your environment.

@dconeybe sure, I can do that... also I have opened the memory tools, but I'm not what I'm looking for.
I've implemented a gRPC Firestore client with swift-grpc, and I can see that it uses 2MiB, while Firestore SDK init takes 5MiB.
I attach a project here as a zip? it will be a network extension type, it can't be run on a simulator, has to be a real device.

Great! The best way to share your project would be to publish it to GitHub as a repository and post the link here. Would that be possible?

@dconeybe sure, honestly I don't why I didn't think of that 🤦🏽

@dconeybe I'm finishing the problem that I will share with you, but so far I'm not able to reproduce the issue as bad as in my project.
I'm running the memory tools, and I am able to see a region which Firestore is using of about 6MiB, right now when I enable Firestore is using in total 9MiB.

I'm not sure what's happening in this project. I'm not running any code on the extension, just Firestore


basic arithmetic, mem, branches

Please, replace for loop with while loop with !state.complete as condition in Hart::run() - it's the last change, just to avoid creating separate PR on this

done


Docker planetary-graphql

This PR adds a Dockerfile for building planetary-graphql as a docker image, for easier compatibility with ansible scripts.
NOTE: This generally works, but is an incredible memory hog.  I am uncertain whether the memory issues were due to the build or the app.  The dockerfile is meant to follow the deploy instructions in the README as close as possible, but there may be issues in the base image I used or some such.  I wanted to offer the PR as a stake in the ground, but would love to pair on looking at graphql's memory usage further.

 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.


Command Award

Allow users to specify a command to be executed on behalf of the server.
Considerations:

Templating engine, to allow different variables to be used in the command

Example:
{
    "type": "command",
    "command": "/somecommand ${playerName}"
}

This is absolutely crutial to my planed usage (advanced rankup system). Are you planning on making this possible soon?

@Mohron
It will be going out in the next release (probably today or tomorrow)

@Mohron v1.1.0 has support for this now.

A demo achievement should be added to showcase this new capability.

@Mohron I actually just created a demo command award and lumped it in with the "Green Thumb" achievement requiring you to plant 10 saplings. The command is time set day, which I thought was appropriate for the achievement.


Switched from isNan to typeof

I noticed an issue when using redux-form. I was creating a form with field names that were IDs that were long numbers. This stores fields as an array rather than an object. The trouble with this is that the IDs were large enough to overflow the max array size.Cause crashing within deleteIn because there is a loop dependent on the array lengths and the array lengths for the arrays created from these IDs were incredibly high. I am not sure of the motivation of using isNaN here but switching to typeof resolves this issue.

can you also add a test?

Sorry about the tests issue, I had trouble testing locally at first. My initialValues is an object and the problem persists. This is not as easy of a fix as I initially thought. I will close the PR but could you explain the motivation behind storing fields in an array in some cases? I want to try to make a more cohesive fix. At the very least large numbers should not be used for arrays like this.


Extract information from active kubeconfig context

Currently, kubectl-http will always try to extract the cluster and user information at index 0. This PR extracts information about the current context and passes them to follow-up jsonpath queries to make sure that calling this as a kubectl plugin has the effect of interacting with the same context as built-in commands would.

cc @sttts

I missed --minify and its meaning in the script so ... oops. This is not needed.


Discrepancy in reverse primer sequence in documentation

Hi,
I was running this pipeline on my samples. I noticed in the README documentation, the __adapter_P was default to R1492 (default: AAGTCGTAACAAGGTARCY), however in the actual run_dada_ccs.R script, the primerR was actually set to RGYTACCTTGTTACGACTT. I see the full-length primer sequence is the latter, so I want to confirm what was the true default of the pipeline?
Thanks!

Hi @marinachen , AAGTCGTAACAAGGTARCY is the reverse complement of RGYTACCTTGTTACGACTT, so they're the same thing.
Nonetheless, the pipeline uses cutadapt to trim adapter in the beginning so the primer trimming portion in the DADA2 script is always skipped and does not function, so you can ignore that.
Thanks.

Hi @proteinosome thank you for clarifying! That's good to know. I was trying to run separate DADA2 bc I want to take further look into some samples that had heavy chimera removal and unsure about the trimming step.

@marinachen the runtime parameter --front_p and --adapter_p are used by cutadapt in the very beginning for adapter trimming. Reads that have both primers are then fed into DADA2. I modified the DADA2 script so it will not do primers trimming as I found cutadapt to be much faster and more flexible.
DADA2 step does however come with a chimera detection and removal step. In the final HTML report you will see how much reads remain after chimera removal in a table.

Thank you for further explanation! The reason I wanted to further look into some samples was because there are a couple of ones that only had about less than 40% reads left after chimera removal, so I’m a bit concerned and want to compare pre vs post ASV table to see what are removed as chimeras. Please do let me know your insights! Thanks again!

From: Khi Pin, Chua @.>
Sent: Friday, April 28, 2023 4:20:02 AM
To: PacificBiosciences/pb-16S-nf @.>
Cc: Chen, Marina @.>; Mention @.>
Subject: Re: [PacificBiosciences/pb-16S-nf] Discrepancy in reverse primer sequence in documentation (Issue #32)
@marinachenhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_marinachen&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=3tSRy3a93tnSAPe3frFSyIeHYA3T5NVkcdqbYRSCExU&m=iOT-gG7UgUYQxy2Mq31eZnrv9iZS_RA1ZlPO8CLmpgAbzG2RVfledQy5vWBId9dT&s=c_xI5BdP3MqkMkTMmVIfRMrLEbPEtcCA9l62SPKq_Qw&e= the runtime parameter --front_p and --adapter_p are used by cutadapt in the very beginning for adapter trimming. Reads that have both primers are then fed into DADA2. I modified the DADA2 script so it will not do primers trimming as I found cutadapt to be much faster and more flexible.
DADA2 step does however come with a chimera detection and removal step. In the final HTML report you will see how much reads remain after chimera removal in a table.
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_PacificBiosciences_pb-2D16S-2Dnf_issues_32-23issuecomment-2D1527174726&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=3tSRy3a93tnSAPe3frFSyIeHYA3T5NVkcdqbYRSCExU&m=iOT-gG7UgUYQxy2Mq31eZnrv9iZS_RA1ZlPO8CLmpgAbzG2RVfledQy5vWBId9dT&s=9WYX-6LRvprdiOAIjV2JqfRJZZakp9H9ZyIXT0l5LX0&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ANC3264EJFKYPBW2C7226U3XDN4TFANCNFSM6AAAAAAXMZR42Q&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=3tSRy3a93tnSAPe3frFSyIeHYA3T5NVkcdqbYRSCExU&m=iOT-gG7UgUYQxy2Mq31eZnrv9iZS_RA1ZlPO8CLmpgAbzG2RVfledQy5vWBId9dT&s=W7B3YyAQN9eJBxUEdyuk-EUiPOYmROFmw3U5J6uieNM&e=.
You are receiving this because you were mentioned.Message ID: @.***>

@marinachen did you look at the read length distribution? Chimera usually shows up as molecule significantly longer than 1.5 kbp (full-length 16S is 1.5 kbp).

The distribution looks alright, plus the min and max length was as default, 1000 and 1600 respectively. The heaviest removed samples were about ~20-30% chimeric ASVs. I blasted a few and there seems to be hits. Would there be other possibilities causing that and any solutions? Thank you!

Can you show me the reads table from the HTML report from some of these samples? I.e. how many reads are lost in each step of DADA2.
Thanks.

Hi, please see attached screenshot of the top chimera-removed samples. For this result, I kept everything default in the pipeline. I'm a bit concerned of these samples losing half reads as chimeric. Thank you for any thoughts!

Sorry this slipped through my mind for a bit. The reads loss in the denoise step look reasonable to me. I generally think the percentage of input non-chimeric is ok as long as they're above 50%, but it does depend on the sample type. Environmental samples have a tendency of losing more reads. DADA2 has a pretty strict filtering process so it's not uncommon to see ~50% read loss. I do think there seems to be a higher than usual non-chimeric sequence (after "denoised" columns). Did these samples by any chance go through a higher number of PCR cycles?
BLASTing those are bound to hit 16S sequences, since these are chimeric reads between different 16S fragments, but are still 16S. Although I'd imagine usually chimeric reads would have a less tight read-length distribution. Overall thou you still have quite a lot of reads for your samples so I probably wouldn't worry too much about it.

Thank youf for the explanation! I don’t think they would have more PCR cycles than others in the same  batch. My concern rises because these chimeric reads are mostly above 1450bp and many can be blasted to complete genome with 100% coverage and identity.
I see DADA2 recommend raising minFoldAbundoverParent to at least above 3.5. I wonder what’s the default in pipeline and how can I change it when running workflow?

From: Khi Pin, Chua @.>
Sent: Tuesday, May 9, 2023 5:17:47 AM
To: PacificBiosciences/pb-16S-nf @.>
Cc: Chen, Marina @.>; Mention @.>
Subject: Re: [PacificBiosciences/pb-16S-nf] Discrepancy in reverse primer sequence in documentation (Issue #32)
Sorry this slipped through my mind for a bit. The reads loss in the denoise step look reasonable to me. I generally think the percentage of input non-chimeric is ok as long as they're above 50%, but it does depend on the sample type. Environmental samples have a tendency of losing more reads. DADA2 has a pretty strict filtering process so it's not uncommon to see ~50% read loss. I do think there seems to be a higher than usual non-chimeric sequence (after "denoised" columns). Did these samples by any chance go through a higher number of PCR cycles?
BLASTing those are bound to hit 16S sequences, since these are chimeric reads between different 16S fragments, but are still 16S. Although I'd imagine usually chimeric reads would have a less tight read-length distribution. Overall thou you still have quite a lot of reads for your samples so I probably wouldn't worry too much about it.
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_PacificBiosciences_pb-2D16S-2Dnf_issues_32-23issuecomment-2D1539488003&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=3tSRy3a93tnSAPe3frFSyIeHYA3T5NVkcdqbYRSCExU&m=WFlp3ZB9uQSBqXrqUiJspAgBDIaONDUrZNYA1hnlCyw2bwQ6ksFBISXPH2IjTmbL&s=AEZE094je4GdOvZtGyhQqx06M_6_v2Fun5s6NA7vjYY&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ANC3266UMJWCLJOOKLLL6ZLXFIDTXANCNFSM6AAAAAAXMZR42Q&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=3tSRy3a93tnSAPe3frFSyIeHYA3T5NVkcdqbYRSCExU&m=WFlp3ZB9uQSBqXrqUiJspAgBDIaONDUrZNYA1hnlCyw2bwQ6ksFBISXPH2IjTmbL&s=lMdWdPi8S4U2vrNarPPuAtY8cxa97d_WSdOMZjM8GCg&e=.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks again! I actually meant chimeric ASV sequence, not the actual reads, sorry for the confusion. And the way I extracted chimeric ASV sequence is by: anti_join(seqtab.df, seqtab.nochim.df, by="seq")

From: Khi Pin, Chua @.>
Sent: Tuesday, May 9, 2023 9:05 AM
To: PacificBiosciences/pb-16S-nf @.>
Cc: Chen, Marina @.>; Mention @.>
Subject: Re: [PacificBiosciences/pb-16S-nf] Discrepancy in reverse primer sequence in documentation (Issue #32)
@marinachenhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_marinachen&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=3tSRy3a93tnSAPe3frFSyIeHYA3T5NVkcdqbYRSCExU&m=Lv4CQESYAr2JjsxXqWWAFD1uLfFVgbDq0_dwBsHWm9CvUqjSLapwV_cwCh98loCl&s=nI0rADRPJYUWy0z-VTv8fKBARgsxkm3H4UK1CHGlDvw&e= The drop from "denoised" to "non-chimeric" is pretty significant across all samples except Soil8, which is why I wondered if there was any experimental step that could perhaps contribute to that. May I know how did you extract the chimeric reads?
For minFoldAbundoverParent, the default is 3.5 in the pipeline. I did not make it a runtime option but you can go into main.nf and under the dada2_denoise process, add --p-min-fold-parent-over-abundance X to the qiime dada2 denoise-ccs command.
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_PacificBiosciences_pb-2D16S-2Dnf_issues_32-23issuecomment-2D1540089587&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=3tSRy3a93tnSAPe3frFSyIeHYA3T5NVkcdqbYRSCExU&m=Lv4CQESYAr2JjsxXqWWAFD1uLfFVgbDq0_dwBsHWm9CvUqjSLapwV_cwCh98loCl&s=-OGfL6aYKTOvBySsOIIkTTdjrCtobDCGIz1X5-Y4sug&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ANC3265QG6XOTTI6QUEODX3XFI6KVANCNFSM6AAAAAAXMZR42Q&d=DwMCaQ&c=WO-RGvefibhHBZq3fL85hQ&r=3tSRy3a93tnSAPe3frFSyIeHYA3T5NVkcdqbYRSCExU&m=Lv4CQESYAr2JjsxXqWWAFD1uLfFVgbDq0_dwBsHWm9CvUqjSLapwV_cwCh98loCl&s=4doe85eGBMmqO_VBqWfNaJAupd567KamkIR6dcp8g7o&e=.
You are receiving this because you were mentioned.Message ID: @.***>

@marinachen Unfortunately I think that may be a question suited for DADA2 author, so I will close this for now. If I thought of any possibility I would let you know.


[Flash] Type coercion failure when loading maps from SharedObject

Describe the bug
Type coercion fails when attempting to use a map loaded from a SharedObject in the Flash target.
Steps to Reproduce

Save a Map in a SharedObject
Load SharedObject
Attempt to use saved Map

Minimal Example
import openfl.events.Event;
import openfl.events.MouseEvent;
import openfl.net.SharedObject;
import openfl.display.Sprite;

class Main extends Sprite {
    var myMap:Map<String, Int> = ["test" => 1];
    public function new () {
        super();
        this.addEventListener(Event.ADDED_TO_STAGE, setup);
    }

    public function setup(e:Event) {
        this.stage.addEventListener(MouseEvent.CLICK, save);
        this.stage.addEventListener(MouseEvent.RIGHT_CLICK, load);
    }

    function save(e:Event) {
        var save = SharedObject.getLocal("test", "/");
        save.data.myMap = myMap;
        save.flush();
        trace("Saved");
    }

    function load(e:Event) {
        var load = SharedObject.getLocal("test", "/");
        trace(Type.getClassName(Type.getClass(load.data.myMap)));
        myMap = load.data.myMap;
        trace("Loaded");
    }
}

Expected behavior
Map is loaded from the SharedObject
Screenshots
TypeError: Error #1034: Type Coercion failed: cannot convert Object@2800c5302e51 to haxe.IMap.
        at Main/load()[/Projects/Haxe/issues/SharedObjectMaps/Source/Main.hx:28]

OpenFL Targets
Flash
Additional context
Does not occur on other targets.

With SharedObject and ByteArray, only the core AS3 language types can be serialized automatically. As I understand it, custom types need to implement flash.utils.IExternalizable to be restored when read from a SharedObject or a ByteArray (and i think maybe flash.net.registerClassAlias needs to be involved too).
Since haxe.IMap does not implement the IExternalizable interface, and we can't force it to implement that interface because it's a Haxe core language type that isn't under OpenFL's control, there probably isn't much that can be done to improve this situation. You may want to consider using a different type that can be serialized. For instance, Flash's Dictionary type is relatively similar to Map (though it isn't strictly typed like Map, so it's not a perfect replacement).
It may also be possible to create a custom IMap implementation that also implements IExternalizable. It could use a regular Map internally to store the data, and act as kind of a wrapper. I've never tried this myself, but it might be something to look into, if you really want to use a map, and would prefer to avoid another type.


Can I encode/decode between public key and unlock hash?

Thanks.

To create an unlock hash you need a set of UnlockConditions.
For the vast majority of addresses, the unlock conditions are simple: they do not have a timelock and only require one signature. So if you have a pubkey, you can create the typical unlock conditions like so:
uc := types.UnlockConditions{
	PublicKeys:         []types.SiaPublicKey{pubkey},
	SignaturesRequired: 1,
}

The UnlockHash is then calculated via the appropriate method on uc:
addr := uc.UnlockHash()

The hash is the Merkle root of a tree that has the timelock, public keys, and number of signatures as leaves.


Publish to Maven Central

Publish mule-deploy-library artifacts to Maven Central.

[x] Update Github Actions Workflow
[x] Configure POM for Maven Central Publish

Implemented with #51


Fix issue #58

Include apple-system on font-family

Thank you very much! :)

Hi @DevEddy! so this pull request exists because of #58.
I don't understand why a pull request for fixing custom fonts breaks custom fonts...
@csteeg Can you help us understand what's happening? Use you using two different version of iOS maybe?

I would try to debug it but multitargeting doesn't work for me in the newest VS on Windows or Mac..

What's the error?

@DevEddy so you set the font family to a font that's embedded in included in your iOS project? This fix was indeed to let that work, if -apple-system wasn't included, on my end the font did not show correctly. What version of iOS did you test this on?

Before this update everything worked well for me.
I am using the newest iOS 12.3
@matteobortolazzo I don't remember, I will need to test it onre more time.
-- Eddy

@csteeg @DevEddy
since I am unable understand and test this issue I removed the build from NuGet for now.


Make it so that the message is edited when checking the Host node

🗿👌

it will be future update since im busy with other project like PteroControl


Make tld .dev configurable

There is a chance that .dev can be made globally available TLD, in which case using .dev for local development may not be viable, so I think we should aim to make TLD configurable so as developers can locally configure it.

This has become a priority now because .dev is a real TLD down and hence conflicts with other domains. We should get configurable TLD done.

@gnufied Something like?
tld = local

[redis]
command = redis-server /usr/local/etc/redis.conf

[memcached]
command = /usr/local/opt/memcached/bin/memcached

@nisanth074 I don't think that will work.
The TLD setup has to be part of invoker setup command and written once to invoker configuration file. the .ini file you shown is user written per project config file.
So something like invoker setup --tld local

Working on a pull request here https://github.com/code-mancers/invoker/pull/153

FTR, while accessing Invoker-provided .dev domains in Firefox works fine for me, Chromium 48.0 blocks it, stating:

This webpage is not available
ERR_ICANN_NAME_COLLISION
This site is using a new generic top-level domain (gTLD). […]


Create de.yml

Aditional German Language

Greetings and thank you for your PR.
Unfortunately, we as a group have decided to leave translations up to the discretion of language pack creators. In turn, this will remove any dependencies or blockers in the future if we choose to add, modify, or remove any existing English language translation strings within this extension.
We are sorry for any inconvenience this may have caused you.
Feel free to PR the translations to a German language pack.


Cannot run classify_webcam.py

I'm getting this error while running classify_webcam.py
Traceback (most recent call last):
File "classify_webcam.py", line 34, in
in tf.gfile.GFile("logs/output_labels.txt")]
File "classify_webcam.py", line 33, in
label_lines = [line.rstrip() for line
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 213, in next
return self.next()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 207, in next
retval = self.readline()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 176, in readline
self._preread_check()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 78, in _preread_check
compat.as_bytes(self.__name), 1024 * 512, status)
File "/usr/lib/python3.6/contextlib.py", line 88, in exit
next(self.gen)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/errors_impl.py", line 466, in raise_exception_on_not_ok_status
pywrap_tensorflow.TF_GetCode(status))
tensorflow.python.framework.errors_impl.NotFoundError: logs/output_labels.txt

In classify_webcam.py file, change output_labels.txt to trained_labels.txt and output_graph.pb to trained_graph.pb (as per the name that you have specified during retraining process)

I'm getting this error while running classify_webcam.py
Traceback (most recent call last): File "classify_webcam.py", line 34, in in tf.gfile.GFile("logs/output_labels.txt")] File "classify_webcam.py", line 33, in label_lines = [line.rstrip() for line File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 213, in next return self.next() File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 207, in next retval = self.readline() File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 176, in readline self._preread_check() File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/lib/io/file_io.py", line 78, in _preread_check compat.as_bytes(self.__name), 1024 * 512, status) File "/usr/lib/python3.6/contextlib.py", line 88, in exit next(self.gen) File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/errors_impl.py", line 466, in raise_exception_on_not_ok_status pywrap_tensorflow.TF_GetCode(status)) tensorflow.python.framework.errors_impl.NotFoundError: logs/output_labels.txt

did classify.py run using docker if then please explain me how to give the path in command line for that

Hi Rohit,
Yes, you can run the classify_webcam.py file using docker command. From host machine you have to mount the location of source code to container and set that as working directory in the command.
The syntax is as follows,
docker run -it -v : --working_dir= python/python3

Hi Rohit,
Yes, you can run the classify_webcam.py file using docker command. From host machine you have to mount the location of source code to container and set that as working directory in the command.
The syntax is as follows,
docker run -it -v hostmachine_dir:container_dir --working_dir=path_on_container python/python3 program_name

thanks for the reply but i tried on my system it worked the only thing that concerned me to run classify.py which path/to/image.jpg should i mention

i m confuse to which path i should mention is it the dataset because it contains the image file but i tried and it didn't work saying dataset is a directory

Hi Rohit,
Yes, you can run the classify_webcam.py file using docker command. From host machine you have to mount the location of source code to container and set that as working directory in the command.
The syntax is as follows,
docker run -it -v hostmachine_dir:container_dir --working_dir=path_on_container python/python3 program_name

classify_webcam.py works fine for me only the issue is with classify.py sorry for the inconvenient.

i did the desired changes. but now it's not showing any output.
i am running classify_webcam.py but my computer is showing no response. what can be the reason? the webcam is not opening.

@manoharmayurohitsharma
did it worked for you?
is that classify_webcam.py workin

I am having the same circumstances as @someshium
Did you find any alternatives ??

@eswarnitk the Classify_webcam.py is not working for mine. i have done the corrections but still when i run it, it shows nothing. remains stucks for infinity untill i exits myself.

@Aryamn Not yet. i have no hope left with this one. did you found any other good repo for reference?

@someshium , can you please elaborate the problem?. Show me your terminal error.
Please check your model and label names in the .py file.

I think there is a problem with cv2 library. I have not figured it out fully though

@eswarnitk  Thankyou for reply. sir this is what it's showing for me.

I am getting no error or nothing happens once i Run python3 classify_webcam.py file. stuck for infinity.
don't know what is the problem. the image classifier works fine. but this webcam one is not working.
It's a big problem. I am still figuring it out. if you have any other reference repo similar to this please suggest me.
thanks.


Refactor the grammer so that we can support highlighting of model keyword and model identifier

Right now grammar is laid out in a way that it is quite hard to highlight model <identifier> part.
Plugin needs some refactoring in order to highlight these fields properly. Kinda same goes for datamodel, enum
Example of the problem:

Notice model Block is not highlighted in the above image


Multi-class classification

I have two main questions.
My first question is which task number in this link https://instructions.apps.allenai.org/ is similar to a multi-class classification?
My second question is, if we want to use this model for multi-class classification for our own dataset where we have a definition for each class and a corresponding example for each class, what should be the format of the input prompt to the model?

Task 1541: agnews_classification is a good example of multi-class classification. The description states: "Given a short article, classify the article based on its category". This task involves classifying news articles into multiple predefined categories, which is a typical multi-class classification problem.
Task 1289: trec_classification
Description: "Given a question, detect which category better describes it."
This task involves classifying questions into multiple categories.
Task 1292: yelp_review_full_text_categorization
Description: "Given a review about a place, provide a rating for it."
While this involves ratings, it's essentially classifying reviews into multiple rating categories.
Task 1494: bengali_hate_speech_classification
Description: "Given a hateful post in Bengali, classify whether it is political, geopolitical, religious or personal in nature."
This is a multi-class classification task for hate speech.
Task 1531: daily_dialog_type_classification
Description: "Classify the input sentence into one of the 5 classes : unknown, commissive, question, information, directive"
This is a clear multi-class classification task for dialogue acts.
Task 1532: daily_dialog_emotion_classification
Description: "Classify the conversation sentences of input passage into the following emotions: anger, disgust, fear, happiness, sadness, No emotion"
Another clear multi-class classification task, this time for emotions.
Format of input prompt for multi-class classification with your own dataset:
Task: Classify the given text into one of the following categories:

[List your class names here]

Definitions:
[Class 1 name]: [Class 1 definition]
[Class 2 name]: [Class 2 definition]
...

Examples:
[Class 1 name]: [Example text for Class 1]
[Class 2 name]: [Example text for Class 2]
...

Text to classify: [Your input text here]

Classification:

Check promptify examples you'll get some ideas for the templates.


Added "redux implementation" link

Thanks, but I don't think we need to expand this section right now. If anything, we need to shrink it down.


Bugfix: Don't autoclose paths when using canvas (#4154)

When defining a path, any call to beginFill results in the path being closed,
but only for the Canvas renderer, not for the WebGL renderer.
This commit removes the check that sets the shape to be closed when a fill is
active. This unifies the rendering behavior between Canvas and WebGL.

Good catch! But I certainly know that it wont be accepted by other members of team if you dont make a fiddle for it :)

@ivanpopelyshev
I'll happily do that, but how do I get a build of pixi with my bugfix in a fiddle?
For the original misbehavior there was a fiddle in the referenced bug report.
This is the same fiddle, updated to use version 4.8.1 and show that the behavior is still the same:
https://jsfiddle.net/3snvu0dy/16/

Its possible to use pixi builds from PR's if they are made in this repo (if I make a branch based on your PR).
I believe that fiddle is enough. Lets wait when travis CI problem will be resolved.

Thanks @fmhdp!


[VS Code] Support for custom models in TS/JS

Target Platform or Language [e.g. Python]: JavaScript/TypeScript
Steps to Reproduce / Scenario:
Bring custom model training for TS/JS to the IntelliCode extension for VS Code.

Any update on this? Is there a certain amount of positive votes needed to get this implemented?

Normalizing the label to Product feedback

Are there any updates on this? It looked like an exciting idea, I'm curious if it will be implemented.

Hey folks, we've implemented a deep learning model for C# in VS and C# Dev Kit for VSCode, along with Python in VSCode. This should address the ask here!
https://devblogs.microsoft.com/visualstudio/upgrading-from-intellicode-custom-team-completions-models-to-deep-learning/


tsc check error TS1005: ',' expected

Describe the Bug
I added reactflow to my project, and everything works fine. It also builds fine. But when type-checking with tsc I get the following issues:
node_modules/@reactflow/controls/dist/esm/Controls.d.ts:1:15 - error TS1005: ',' expected.
1 import { type PropsWithChildren } from 'react';

node_modules/@reactflow/core/dist/esm/components/Handle/index.d.ts:2:15 - error TS1005: ',' expected.
2 import { type HandleProps } from '../../types';

node_modules/@reactflow/core/dist/esm/index.d.ts:25:47 - error TS1005: ',' expected.
25 export { default as useOnViewportChange, type UseOnViewportChangeOptions } from './hooks/useOnViewportChange';

node_modules/@reactflow/core/dist/esm/index.d.ts:26:48 - error TS1005: ',' expected.
26 export { default as useOnSelectionChange, type UseOnSelectionChangeOptions } from './hooks/useOnSelectionChange';

node_modules/@reactflow/core/dist/esm/index.d.ts:27:47 - error TS1005: ',' expected.
27 export { default as useNodesInitialized, type UseNodesInitializedOptions } from './hooks/useNodesInitialized';

They all seem to be coming from .d.ts files, and whenever doing import { type xxx }.
Adding skipLibCheck: true doesn't fix the issue.
Typescript: 4.4.4
React: 17.0.2
Reactflow: 11.7.2
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Create a project with the dependencies specified.
Create a random flow.
Run tsc
Expected behavior
tsc to compile without issues
Screenshots or Videos
No response
Platform

OS:  macOS
Issue is within terminal, both in local and in a linux jenkins environment

Additional context
No response

I have exact the same problem in my project.
Typescript: 4.4.4
React: 17.0.2
Reactflow: 11.7.2

@janarvaezkng I updated my typescript version to 5.1.3 and my project works

@janarvaezkng I updated my typescript version to 5.1.3 and my project works

Thanks @kike721 ! We ended up upgrading to 4.9.4 and it worked as well.

I face the same issue, but I do not want to upgrade typescript (which leads to more type error )

I face the same issue, but I do not want to upgrade typescript (which leads to more type error )

I have the same problem, how did you solve it in the end?

@guguji5 I have the same problem, how did you solve it in the end?


Switching AKS from using SPN to use Managed Identity. What happens when nodepool image is already the latest image?

The article mentions that, when migrating an existing AKS cluster from an SPN to MI, you need to run
az aks nodepool upgrade --node-image-only on your nodes to complete the update to managed identity
But what if the nodes are already on the latest image? How do we then force kubelet to use MI and let go of old SPN.

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

ID: 6a9c74a8-9f1a-3d3d-3aed-f8ec45f92e05
Version Independent ID: 01e98043-dc80-490e-a761-8d558ff4e6df
Content: Use managed identities in Azure Kubernetes Service - Azure Kubernetes Service
Content Source: articles/aks/use-managed-identity.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@haripraghash we'll review your feedback and get back to you shortly. Thanks.

@haripraghash
A nodepool upgrade operation will trigger an agentpool reconcile (which means that the cluster is going to try to get the nodepool to desired state: using MI) under the hood.
So it doesn't matter if the nodepool is on latest node image version.
Hope that helps.

Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


AnyFixture vs before_all

It seems both accomplish quite a similar goal - create shared setup only once instead of per example. However, I'm not sure which one is better to use. What are the considerations, that one has to take into account when choosing? Or are they complementary?

Btw, sorry if it's wrong to post questions here, but there's no Support section in the Readme and that's why I chose to write here.

before_all works only within an example group and should be considered as a replacement for before(:each) (when there is a lot of affected examples) and before(:all) ('cause it's not safe regarding to DB cleanliness).
AnyFixture, from the other hand, is kind of replacement for traditional, YAML-based, fixtures. The data generated within AnyFixture persists for the whole run, not only for the example group where it has been activated.
Take a look at the recent slides, where I give some explanations and tips when to use each tool (video is not there yet, unfortunately).
TLDR; Use AnyFixture when you have similar setup between files and use before_all/let_it_be to share setup within an example group.

Thank you very much for the clarification. I use AnyFixture heavily in my test suit. While it certainly keeps the speed of the suite high, it has several times bitten me in unexpected ways. When running my specs in random order I am never sure what is in the database. Also, sometimes tests may pass, when run as part of the whole suite, but might fail when running stand-alone. The reason is that, for example, they expect a record, which has been created in another file (full suite run), but is not created in the current file (individual suite run). So one has to be very careful. I guess one has to trade speed against readability and maintainability.

Well, yes, the same drawbacks as with usual fixtures. And that's the reason when other tools (such as before_all) exist–to avoid global state.
Btw, there is one useful thing missing in the guide–:with_clean_fixture shared context, which allows you to disable AnyFixture data for an example (i.e. allows you to work with clean state):
describe User do
  before(:all) { @account = TestProf::AnyFixture.register(:account) { create(:account) } }

  it "assigns new user to the first account if not specified" do
     expect(create(:user).account).to eq @account
  end

  it "raises an error if no accounts", :with_clean_fixture do
    expect { create(:user) }.to raise_error(ActiveRecord::RecordInvalid)
  end
end

Wow, this is indeed super useful!!! I'll definitely take it into account!


New variables are not used after refetch(newVars)

Hello! I don't know how to make it better. Please help.
I use the function:
    async update() {
        let variables = {
            company: this.props.match.params.companyId,
            filter: this.filter, order: this.order, limit: this.pagiation.limit, offset: this.pagiation.offset
        };
        await this.props.refetch(variables);
    }

But I see in Network two queries and nothing happens.
The first one is right, but the second is the default data (like variables: { company: props.match.params.companyId, filter: {}, order: [], limit: 10, offset: 0 }). Why?
@ApolloReact.graphql(ApolloReact.gql"
    query Departments($company: ID!, $filter: DepartmentFilter!, $order: [DepartmentOrder], $limit: Int, $offset: Int) {
        company {
            get(id: $company) {
                id,
                name,
                color,
                departments {
                    list(filter: $filter, order: $order, limit: $limit, offset: $offset) {
                        departments {
                            id,
                            color,
                            name,
                            users {
                                id, avatar, username
                            },
                            tags {
                                name
                            },
                            date
                        },
                        count
                    }
                }
            }
        }
    }", {
    props: ({ ownProps, data: { loading, company, refetch } }) => ({
        company: company && company.get,
        departments: company && company.get && company.get.departments && company.get.departments.list.departments,
        count: company && company.get && company.get.departments && company.get.departments.list.count,
        loading: loading,
        refetch: refetch
    }),
    options: (props) => ({
        fetchPolicy: 'cache-and-network',
        variables: { company: props.match.params.companyId, filter: {}, order: [], limit: 10, offset: 0 }
    }),
})

But if I call console.log(await this.props.refetch(variables)); this is what I need, but I want to see it in props.
    async update() {
        let variables = {
            company: this.props.match.params.companyId,
            filter: this.filter, order: this.order, limit: this.pagiation.limit, offset: this.pagiation.offset
        };
        console.log(await this.props.refetch(variables));
    }

➤ stale[bot] commented:
This issue has been automatically labled because it has not had recent activity. If you have not received a response from anyone, please mention the repository maintainer (most likely @jbaxleyiii). It will be closed if no further activity occurs. Thank you for your contributions to React Apollo!

➤ Makame commented:
It still doesn't work. I think you have too many stars.


Updated pull request template

Description
Updated pull request template
Fixes #291
Type of change

Documentation update

Testing steps
N/A
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report

Merging #292 (dab6fb6) into master (a60966c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #292   +/-   ##
=======================================
  Coverage   73.88%   73.88%
=======================================
  Files           9        9
  Lines         471      471
=======================================
  Hits          348      348
  Misses        105      105
  Partials       18       18

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 a60966c...dab6fb6. Read the comment docs.


[Zone Decommission] Decider to prevent shards moving into nodes of to-be-decommissioned zone

TBD

@imRishN could you share more information on the issue. Also are you actively working on it?

@imRishN Please make sure to add more details into the issue description before creating the issue


Experiment azure-search on API 29

The java azure-search is a reasonably complex module; while we prototype azure-core splitting, we will use azure-search as the Guinea pig to ensure the small modules continue to provide the same feature as azure-core as a whole.

Experimental azure-search (API level 29+) based on splitted azure-core is here: https://github.com/anuchandy/az-android-vnext/tree/master/sdk/search

Experimental azure-search (API level 29+) based on splitted azure-core is here: https://github.com/anuchandy/az-android-vnext/tree/master/sdk/search


False positive: branch protection rule violation on transferred repository

Hi there! Thanks for maintaining this tool.
We're currently using Allstar on sigstore/gh-action-sigstore-python, which was recently transferred from another org.
After we transferred the repository to the Sigstore org, we received an automated issue warning us of a stale review policy violation: https://github.com/sigstore/gh-action-sigstore-python/issues/32
Based on the current branch protection rules for main, this looks like a false positive. I've screencapped them below, for reference:

Please let me know if there's information I can provide!

Hi, thanks for reporting! I'll take a look at the logs and get back to you.

Thanks for the response @jeffmendoza! It turns out this was probably a misunderstanding on my part -- my colleague updated the setting after Allstar reported the violation, which I then saw and assumed wasn't detected correctly.
I'm closing this for now; will reopen if we see an actual false positive 🙂

Yes, Allstar is working a bit too slow now, We'd like to improve the responsiveness. Thanks for reaching out.


Follower Only Mode Command. Closes #2

Purpose
Sometimes things can get outta hand and we may need to moderate. As suggested in issue #2, we now have a follower only command that will allow only followers to change the theme.
Changed files
Authentication.ts

Added a twitch API call to check if the twitch user is following the streamer. The call to the API is required as the UserState object in tmi.js does not indicate if a user is following or not
Added a twitch API call to get all followers of the streamer

ChatClient.ts

Update disconnect to deactivate follower only mode before trying to reset the theme
Update onMessageHandler to pass through complete userState

Themer.ts

Added commands to activate and deactivate follower only mode
When follower only mode is activated, get new list of followers
Update commands, which can change the theme, to pass through the userState
Update changeTheme to handle allowing a theme to be updated if the user is following

Edited to explain more changes to files

Only check for follower status if follower mode is turned on
Save follower status on the _listRecipients object when checking for follower status
On !theme check if we have follower status for that user on _listRecipients.  If so use it, otherwise lookup it up yo.

Pushed up added functionality requested by MichaelJolley. Updated pull request changes.

:shipit:


Move the useLibrary declaration from a root extension to the embedded…

…Tools extension
By doing this, they are now type safe. This makes them easier to use from Java, and to extend on if necessary.
It also avoids poluting the root extension space. Its a slight breaking change, but in gradlerio we can actually just add duplicates on the wpi extension.
That would make in C++ the use library change to wpi.useLibrary(it, ...) which is much cleaner, and intellisense would work

That was my thought. I would rather shim that at the GR level for a year rather then keep it at the lower level like that.
I'll make the project upgrader automatically replace the old style with the new style, which will handle a lot of cases.


Add code signing

Not important at this stage and would require getting a certificate, which costs money.


Cluster - KubernetesProvider

Potential bug in KubernetesProvider.
We have observed pods not getting new topology from the KubernetesProvider.
This could potentially be any of the following issues:

Outdated Kubernetes client, the issue was seen in 0.17.0
Bug in KubernetesProvider, e.g. the watch for some reason hangs
Some form of deadlock in MemberList when processing topology changes
Other?

Actions.

More logging and added timeouts for KubernetesProvider watcher.
Make it possible to stream event stream data to the outside world for diagnostics.

Possibly related https://github.com/kubernetes-client/csharp/issues/533

Potentially an issue in AKS itself:
https://github.com/Azure/AKS/issues/1755
Seems to have been fixed in later releases

There could be multiple reasons for what we are seeing, keeping this open until we have verified of the issue goes away with the new patch

Multiple fixes went into 0.20.0. repoen if issues shows up again


Upgrade to OCaml 5.1

Upgrade to OCaml 5.1
Issue: https://github.com/NathanReb/ocaml-api-watch/issues/28

We just merged #30 so it seems you will have to rebase your work on top of main before we can merge!
Please let me know if you need help with this process!

We just merged #30 so it seems you will have to rebase your work on top of main before we can merge!
Please let me know if you need help with this process!

Even though i did Rebase from my side, can you conform it?

We just merged #30 so it seems you will have to rebase your work on top of main before we can merge!
Please let me know if you need help with this process!

Even though i did Rebase from my side, can you conform it?

It seems you didn't push your changes since github mentions there are still conflicts!

It seems you didn't push your changes since github mentions there are still conflicts!

can you tell me how to do that?

@nawarajshah you still haven't pushed your changes.
A rebase is roughly a rewriting of your local git history on your working branch. If you want this to impact this PR you will have to push this change to you remote branch on your fork of the project.
Since the history has been rewritten after your previously pushed, you'll have to run git push --force as the remote branch isn't an ancestor of your local branch anymore (i.e. you did not simply add new commits on top of what is on the remote branch).
What do you get when running git status in your local copy of the project?

@nawarajshah you still haven't pushed your changes.
A rebase is roughly a rewriting of your local git history on your working branch. If you want this to impact this PR you will have to push this change to you remote branch on your fork of the project.
Since the history has been rewritten after your previously pushed, you'll have to run git push --force as the remote branch isn't an ancestor of your local branch anymore (i.e. you did not simply add new commits on top of what is on the remote branch).
What do you get when running git status in your local copy of the project?

this Is what i get when i run git status

Can you show the output of these commands?

git remote -v
git log -1 upstream/main

Can you show the output of these commands?

git remote -v
git log -1 upstream/main

This is the output.

What rebase command did you run? If you rebased on top of your local main branch, it's possible that it was simply out of sync with the upstream main branch.
You should either update your local main branch and rebase again:
git checkout main
git pull
git checkout nawaraj_shah
git rebase main

or simply rebase on top of the upstream main branch:
git fetch --all
git rebase origin/main

git rebase origin/main

here it is

Can you show the output of these commands?

git remote -v
git log -1 upstream/main

after I get help from Bard, I can get here

That's progress! This shows what I suspected: that you did not have a remote configured to track the "upstream" repository, and as a result where not able to pull the latest updates. This documentation can help with understanding how to manage remotes and you may find it useful in the future :) https://git-scm.com/book/en/v2/Git-Basics-Working-with-Remotes
Now, to move forward, from the branch associated with this PR, you should be able to
git rebase upstream/main

And then finally
git push --force

Should update this pull request your changes.

Fixed in an alternate PR.


MO : Deduction of amount already paid

Questions
Answers

Branch?
develop

Description?
An order can have other payment on the same order process we can have a payment that is made before (eg payment by gift card or balance / have), so it is necessary to deduct the amount that has already been paid. This update have visual impact for the final customer. I regularly implement this update near my clients.Maybe one day we will be able to make multiple payments, this update is a first step :-).

Type?
Feature

Category?
MO

BC breaks?
no

Deprecations?
no

Fixed ticket?
no

How to test?
Paid with wire payment and check amount.

Has this been approved by the Product Team ?

Ping @PrestaShop/prestashop-product-team

Please note that we need an issue to be able to review it. For this PR i've just created it : https://github.com/PrestaShop/PrestaShop/issues/18413 but do not forget to create one next time ! :-)
Your improvement is a nice idea, let's test this PR !

Thanks @timactive


Proposed refactoring with new pattern for external APIs

This is a proposed refactoring of the OpenAI, Together, and HF APIs (and a model for what I think is a much cleaner pattern for pixeltable library functions in general). See in particular pixeltable/functions/openai.py, pixeltable/functions/together.py, and pixeltable/functions/huggingface.py.

This change is

pixeltable/env.py line 81 at r4 (raw file):

        # config
        self._config_file: Optional[Path] = None

Are there situations where there is no config? why is it optional.

pixeltable/env.py line 88 at r4 (raw file):

        self._stdout_handler.setFormatter(logging.Formatter(self._log_fmt_str))
        self._initialized = False

Why not just use .config. If it ever needs to change, @property can be added.

pixeltable/env.py line 408 at r4 (raw file):

    @property
    def openai_client(self) -> Optional['openai.OpenAI']:

if this was inited, it should be not None from the start (should be inited in constructor), otherwise, this is an error (like with engine()), no?

pixeltable/functions/openai.py line 111 at r5 (raw file):

@pxt.udf(batch_size=32, return_type=ts.ArrayType((None,), dtype=ts.FloatType()))

I see this is carried over from the previous version, but I'm wondering where we're getting these batch sizes from.


Support for OpenPCDet v0.3.0

Hi,
I'm very interested in your code. Will you consider supporting OpenPCDet v0.3.0? Seems that my GPU (Nvidia RTX 3080) only supports CUDA 11 for now, and I've been kept having all kinds of issues with running CUDA 9.0 on my computer. It seems that I can only run OpenPCDet v0.3.0 with CUDA 11.
Thank you

This is one of the thing in my todo list. Actually, in my interval version, I have support v0.3.0. I will support it within a week.

Supported.


Thanks, and general issues

Hello Tsvi, first of all thank you for your great work. Your syntax really helps me in my work. I really wanted to use a modern text editor with specman and you made that possible.
I would like to point out a few issues that I have though:

I can't make a comment block, for some reason the ctrl / combination and going to edit and selecting the block comment won't work. So for now I have to manually comment each line.
2)The syntax sometimes is inconsistent for example:
ListOfUintToListOfBytes(input : list of uint(bits:64)) :list of byte is also{
var output : list of byte ;
messagef(MEDIUM, "Alex inside uint to bytes function got this as input %s  \n",input) ;
     for each uint(bits:64) (n) in input {
     	messagef(MEDIUM, "Alex inside uint to bytes function adding uint (not as a byte yet) %s to output \n",n) ;
         output.add(BusToListOfBytes(n,8));

     };
     return output;
 };

The second messagef function is highlighted (in lime) and and the first one isn't.
I have other syntax highlighting inconsistencies with comment blocks or function calls. (I'll point them out if you'd like). Thanks ,Alex.

Hi @tsalex1992 a lot of fixes were incorporated by @blutsvente. Could you check if these issues still exist?

Hi Tsvi,
I just started using sublime text 3 and it's great to have some syntax highlighting for e code, so thank you! I see the same problem that tsalex reported regarding the comment block 'ctrl +/', no comments are added or removed.
When I go to a commented line and type 'shift + ctrl + alt + p' I can see that comments are correctly detected.
Also enabling the console with > sublime.log_commands(True)
The console prints out the message: command: toggle_comment {"block": false}
when I type 'ctrl +/' so the key bindings seem ok.
I don't know the next step though who picks up this toggle_comment command
and performs an action...?
There's is no toggle_comment context in the SpecmanCode.sublime-syntax file,
but there's also none in a systemverilog syntax file and that one works ok...
Best regards
Damian

Hi @blutsvente
Thanks that works for me!
I see you've made other updates too, I appreciate it.
I've been trying to understand that sublime syntax file, it takes a bit of work...
Best regards
Damian

Closing this issue via PR #4
If issues are found, please open one for every bug. Thanks.


ponyc SIGSEV (Address boundary error) on OSX

On latest stable OSX el capitan, Xcode 7.3.1 with command line tools. I installed ponyc via homebrew --HEAD and llvm38 by editing the formula.
Compiling the hello world program from the tutorial produces this error (fish shell):
fish: 'ponyc' terminated by signal SIGSEGV (Address boundary error)

ponyc --version
b7e1747

Verbose or debug mode had the same result.

can you create a debug version of the compiler.
run it in lldb and report back what the backtrace is that you get?
make config=debug
lldb ponyc
> run
>  bt

I get this:
(lldb) run
Process 32596 launched: '/usr/local/bin/ponyc' (x86_64)
Building builtin -> /usr/local/Cellar/ponyc/HEAD/packages/builtin
Building . -> /Users/Shared/ponytest/server
Generating
Process 32596 stopped
* thread #1: tid = 0xba43, 0x0000000100066637 ponyc`lookup_base + 2157, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x60)
    frame #0: 0x0000000100066637 ponyc`lookup_base + 2157
ponyc`lookup_base:
->  0x100066637 <+2157>: movq   0x60(%r15), %rdi
    0x10006663b <+2161>: leaq   0xaeb38b(%rip), %rdx      ; "couldn't find '%s' in '%s'"
    0x100066642 <+2168>: xorl   %r12d, %r12d
    0x100066645 <+2171>: xorl   %eax, %eax

Are you running a debug version of the compiler? That looks like a release version.

In my brew file it says config=debug, is there a way i can confirm this?

Ok I recompiled without brew and now got a better stack trace:
(lldb) run /Users/Shared/ponytest/server/
Process 36091 launched: './build/debug/ponyc' (x86_64)
Building builtin -> /Users/myuser/ponycgit/packages/builtin
Building /Users/Shared/ponytest/server/ -> /Users/Shared/ponytest/server
Generating
Process 36091 stopped
* thread #1: tid = 0x1121d, 0x00000001000b4a52 ponyc`lookup_nominal(opt=0x0000000000000000, from=0x0000000107025e40, orig=0x0000000107025e40, type=0x0000000107025e40, name="create", errors=true) + 1042 at lookup.c:95, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x60)
    frame #0: 0x00000001000b4a52 ponyc`lookup_nominal(opt=0x0000000000000000, from=0x0000000107025e40, orig=0x0000000107025e40, type=0x0000000107025e40, name="create", errors=true) + 1042 at lookup.c:95
   92  	  if(find == NULL)
   93  	  {
   94  	    if(errors)
-> 95  	      ast_error(opt->check.errors, from, "couldn't find '%s' in '%s'", name,
   96  	        type_name);
   97
   98  	    return NULL;
(lldb) bt
* thread #1: tid = 0x1121d, 0x00000001000b4a52 ponyc`lookup_nominal(opt=0x0000000000000000, from=0x0000000107025e40, orig=0x0000000107025e40, type=0x0000000107025e40, name="create", errors=true) + 1042 at lookup.c:95, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x60)
  * frame #0: 0x00000001000b4a52 ponyc`lookup_nominal(opt=0x0000000000000000, from=0x0000000107025e40, orig=0x0000000107025e40, type=0x0000000107025e40, name="create", errors=true) + 1042 at lookup.c:95
    frame #1: 0x00000001000b4091 ponyc`lookup_base(opt=0x0000000000000000, from=0x0000000107025e40, orig=0x0000000107025e40, type=0x0000000107025e40, name="create", errors=true) + 385 at lookup.c:357
    frame #2: 0x00000001000b3f07 ponyc`lookup(opt=0x0000000000000000, from=0x0000000107025e40, type=0x0000000107025e40, name="create") + 55 at lookup.c:386
    frame #3: 0x000000010003eb90 ponyc`genexe(c=0x00007fff5fbff6e8, program=0x000000010a1efd00) + 224 at genexe.c:375
    frame #4: 0x0000000100032799 ponyc`codegen(program=0x000000010a1efd00, opt=0x00007fff5fbffad8) + 217 at codegen.c:573
    frame #5: 0x0000000100086df3 ponyc`generate_passes(program=0x000000010a1efd00, options=0x00007fff5fbffad8) + 51 at pass.c:284
    frame #6: 0x0000000100001ab1 ponyc`compile_package(path="/Users/Shared/ponytest/server/", opt=0x00007fff5fbffad8, print_program_ast=false, print_package_ast=false) + 161 at main.c:224
    frame #7: 0x00000001000018f8 ponyc`main(argc=2, argv=0x00007fff5fbffb78) + 1112 at main.c:345
    frame #8: 0x00007fff823d35ad libdyld.dylib`start + 1
    frame #9: 0x00007fff823d35ad libdyld.dylib`start + 1
(lldb) p opt
(pass_opt_t *) $1 = 0x0000000000000000
(lldb) p name
(const char *) $4 = 0x000000010a1e47a0 "create"
(lldb) p type_name
(const char *) $5 = 0x000000010a1ef7c0 "Main"

optseems to be null at this point.

Ok I had a typo in my hello world: If create is not  called create this crash occurs.


GraphDBSUT [00001] - Add JUnit 5 support and test coverage

Add JUnit 5 support and Surefire plugin in POM.
Use cases should include:
Create new repository after optionally deleting an existing repository

Fixed in commit 879e565 branch test_graphdbsut_00001.


Update tests.rst

Added one line on how to test. This is mentioned in the Iris Check List in pulls.rst list, but would be good to have it here as well (useful for new contributors like me).

Hi @pelson, thanks for your message! I submitted a CLA to the mentioned e-mail address a few days ago, I'm awaiting a reply.

@pelson : I still didn't here back about  my CLA. I submitted my request both to the addresses<EMAIL_ADDRESS>and<EMAIL_ADDRESS>. I will resubmit today. Could you maybe check what's going wrong here? Or maybe @bjlittle can help?

Hi @bascrezee – very sorry about the delay processing your CLA; this was entirely our fault. Still, we've got it all done now and that means that the CLA checker bot is now happy 🙂
It looks like you've picked up some test failures that don't seem to be related to the change you've proposed here, so I'm going to restart the tests and see if that fixes it.

you've picked up some test failures that don't seem to be related to the change you've proposed here

That was a problem with #3177, which was reverted in #3178.
Not a problem for this, I'll just merge.
Welcome + thanks @bascrezee !


Fixes build issues of Android example project

Migrates Android's Gradle plugins appliance from imperative to declarative way

Fixing build issues of Android example project

fix #86

@eyoeldefare


Add package header

Without this header the package doesn't install (using Spacemacs and quelpa). This is just a simple header but you can see more possibilities here.

@seubert thank you! So I think I saw your fork and realized I needed the package header, but totally missed this PR somehow, so I added it manually. I merged the PR anyways to give you credit.
p.s. I'd like to more fully flesh out this mode, but would need some help. If you're familiar with writing language modes, I'd love to collaborate!


Fix testKitGen output option

Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

reviewer: @sophia-guo

make sanity

add to whitelist

ok to test

PR triggered and passed. For unknown reason, no write permission to this  pull request. Will create an issue.
https://ci.adoptopenjdk.net/view/OpenJDK tests/job/PR_sanity_openjdk_test_x86-64_linux/41/
Approve this PR.


Make AnnotatedString.Companion.fromHtml available to multiplatform

Currently, this function is not implemented. Maybe make it available in the future. This library “com.mohamedrejeb.richeditor” did it, but we want official support.
actual fun AnnotatedString.Companion.fromHtml(
    htmlString: String,
    linkStyles: TextLinkStyles?,
    linkInteractionListener: LinkInteractionListener?
): AnnotatedString {
    throw UnsupportedOperationException("Compose Multiplatform doesn't support fromHtml")
}

We're not planning to support this helper in the next version. It was removed from common and now it's android-only, see https://android-review.googlesource.com/c/platform/frameworks/support/+/3150316
Also, it's questionable to add this in the core ui module because it requires extra dependencies for parsing that will increase binary size for all Compose applications.
The current recommendation here is to use 3rd party solutions

@MatkovIvan Fine, for anyone who have same request by do not wanna third party lib, I implemented myself by recursing the source string to find html tags, then adding spanstyle to each tag.
Let's close this issue for now, but maybe reopen in the future.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Count or set additional field for sub-sub queries.

Hi there. Have a question about sub-sub queries.
For example I have User which has Posts ( HasMany) and Posts has Likes which belongs to User and Post. So, I tying to get total count of likes when try to get all users with post related to users.
const users = User.query().withGraphFetched('[posts.[likes, files]]')
      .select(['user_table.*', User.relatedQuery('posts').count().as('totalPosts')])

in above example I got totalPosts. It's ok, but what if I want to get in each post total Likes?
Eventually expecting something like this:
const users = User.query().withGraphFetched('[posts.[likes, files]]')
      .select(['user_table.*', User.relatedQuery('posts').innerRelated('likes').count().as('totalLikes')])

And expect output array with users like below:
[{
   id: 9323,
   firstName: 'John',
   lastName: 'Travolta',
   posts: [{
      id: 2,
      totalLikes: 99,
      files: [File],
   }],
   bio: 'Actor'
}]

This should work:
const users = User
  .query()
  .withGraphFetched('posts.[likes, files]')
  .modifyGraph('posts', postsQuery => {
    postsQuery.select('posts.*', Post.relatedQuery('likes').count().as('totalLikes')
  })

Let me know if it doesn't.


Slashing DB import/export

Need to be able to import/export the dataset from one DB to another. This needs to be based on the standard dataset agreed by eth2 clients.

https://hackmd.io/@sproul/Bk0Y0qdGD


TNT Village closed but replaced by corsaro.red

Hi.
"TNT Village" has been closed but it was replaced by https://corsaro.red/
Please remove "TNT Village" and add "Corsaro.red" https://corsaro.red/
Thank you!!!

WIll be added in the next version.

Great!!

Please, try v0.0.42.


resolve #15 - Check how using pull request to push in branch with verification - Update README.md

The change after creating ruleset and enabling Require a pull request before merging! (SUCCEED!)

resolved #15


Python 2.4 compatibility problem with urlparse

I have setup a simple Client with v1.1.3 as in your Twitter example but I get the following error
File "/var/lib/zope3/instance/satisfly/lib/python/satisfly/browser/connections.py", line 272, in call
print client.request(request_token_url, "GET")
File "build/bdist.linux-x86_64/egg/oauth2/init.py", line 622, in request
File "build/bdist.linux-x86_64/egg/oauth2/init.py", line 317, in to_url
I investigated a bit and I noticed that you are expecting urlparse.urlparse returning an object with a query property:
def to_url(self):
    """Serialize as a URL for a GET request."""
    base_url = urlparse.urlparse(self.url)
    query = parse_qs(base_url.query)
    [...]

But my version returns a 6-tuple
Tnx for your help
Regards

Refer to #54 question about py24 support.

Sorry @eserra, Python 2.4 is no longer supported. See #54.


Update submodules using https instead of ssh

No longer planned to be actioned as the recommended route is to have ssh setup.


Not Tracking Newly Created Windows

I have been using chunkwm on my desktop for a long time. That works great. First I want to thanks @koekeishiya and many many contributors.
Now I want to install it on a new macbook pro. I install it from the brew. After install, I restart the computer and open some windows, it has no effects at all. Not resized and moved. After I restart using brew services restart chunkwm, it tracks all the window I have opened, starting resize them and move them. However if I open a new window, that window will not get tracked again.
Chunkwm has become one of the essential tools I use. Any help appreciated.

Here is my .chunkwmrc file Click Here

Here is my .khdrc file Click Here

UPDATE: Tracked the first open window of the app.
One thing I noticed is if I open an app that is previously closed, the window of that app is always tracked by chunkwm. However if it is an app that is already opened and I create a new window using cmd-n or file->new window, the window is not tracked

Problem Solved.
Recompile the plugin and problem solved.


I'm researching if I can move to expr from the govaluate dependency since it's not maintaining anymore. And I'm wondering about how can I achieve the following using expr:
expression, err := govaluate.NewEvaluableExpression("foo > 0");

parameters := make(map[string]interface{}, 8)
parameters["foo"] = -1;

result, err := expression.Evaluate(parameters);

It would be nice to have a migration guide for govaluate -> expr. Thanks!

Exactly the same. Please read the manual.


JSON/XML Schema Support

Add JSON Schema/XSD support (import and export).
Validation by schema, remote loading (ideally), & index identification.

Kamino closed and cloned this issue to ngnjs/NGN


Empty String Converted to true, Causing React Warning

Test Case

Test

Error
react-dom.development.js?f8c1:67 Warning: Received `true` for a non-boolean attribute `className`.

If you want to write it to the DOM, pass a string instead: className="true" or className={value.toString()}.

Expected Outcome
Just pass the empty string through (or omit className entirely).

Thanks for the detailed issue
Based on HTML spec https://html.spec.whatwg.org/multipage/common-microsyntaxes.html#boolean-attributes, empty string is treated as a boolean attribute. htmr will treat the attribute as such, and forward the boolean value to react instead. See #103
If you want to opt-out of this behavior, you can use custom transform, for example:
htmr(html, {
  transform: {
    p: props => {
      if (props.className === '') delete props.className
      return

    }
  }
})


fix fix core at memtable writer mem_consumption

Proposed changes
Fix core when calculating mem consumption of MemTableWriter in dev branch:
*** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1691737795" if you are using GNU date ***
*** Current BE git commitID: ab2a5f9c4a ***
*** SIGSEGV unknown detail explain (@0x0) received by PID 88836 (TID 89127 OR 0x7fd30500a700) from PID 0; stack trace: ***
 0# doris::signal::(anonymous namespace)::FailureSignalHandler(int, siginfo_t*, void*) at /mnt/disk1/chenkaijie/palo/be/src/common/signal_handler.h:413
 1# os::Linux::chained_handler(int, siginfo*, void*) in /root/doris/java8/jre/lib/amd64/server/libjvm.so
 2# JVM_handle_linux_signal in /root/doris/java8/jre/lib/amd64/server/libjvm.so
 3# signalHandler(int, siginfo*, void*) in /root/doris/java8/jre/lib/amd64/server/libjvm.so
 4# 0x00007FD3548D3770 in /lib64/libc.so.6
 5# doris::MemTableWriter::mem_consumption(doris::MemType) at /mnt/disk1/chenkaijie/palo/be/src/olap/memtable_writer.cpp:347
 6# doris::MemTableMemoryLimiter::_refresh_mem_tracker_without_lock() at /mnt/disk1/chenkaijie/palo/be/src/olap/memtable_memory_limiter.cpp:217
 7# doris::Daemon::memtable_memory_limiter_tracker_refresh_thread() at /mnt/disk1/chenkaijie/palo/be/src/common/daemon.cpp:286
 8# doris::Thread::supervise_thread(void*) at /mnt/disk1/chenkaijie/palo/be/src/util/thread.cpp:466
 9# 0x00007FD3546F9FED in /lib64/libpthread.so.0
10# clone in /lib64/libc.so.6

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.82 seconds
stream load tsv:          510 seconds loaded<PHONE_NUMBER>9 Bytes, about 139 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.2 seconds inserted 10000000 Rows, about 342K ops/s
storage size:<PHONE_NUMBER>0 Bytes


Add ability to change Service type and exclude some k8s resources using vars  (Helm Chart)

not create lighthouse resources if "lighthouse.enabled: true"
ability not to use CA for postgresql
ability to change Service type

:heavy_check_mark: Checklist

[x] Added or updated documentation
[x] All your commits have a Signed-off-by line in the message. (more info)

Thanks!
@martina-if, wanna have a look at this one? :grin: 🙏


add option to change data store location

Give me the option to specify where I want to have my data stored.

Attempted to add this option but was not able to at the moment. The data-store package I'm using creates the data location at point of requirement. This makes it difficult to require the package at either a different point or change the directory and/or file name. Will close for now as it's not pertinent for this package's functionality or has it been requested by anyone. C'est la vie!


Add project urls to PyPI

Merged. Note that the build is currently broken and effectively blocks release, so I'm not sure when this will be reflected in PyPI.

thanks, I see actionlint released v2: https://github.com/raven-actions/actionlint/releases/tag/v2.0.0


[CLI]: wand.log doesn't seem to work when including step=

Describe the bug

if i wandb.log it does not seem to upload when i use step argument as well.  i kind of believe it is something i am doing wrong because it seems like something others would have mentioned and i dont see any other issues or info in release note that is similar from a quick search.
an example below, if there is a better way to do this (such that step on the graphs doesnt have large "jumps" from point to point) i would love to know but didnt seem to find anything in docs that didnt require multiple wandb.runs at once?
example below but if i use the line wandb.log({"eval/loss": running_loss}, step=i, commit=True, sync=True) (and have tried without sync=True or commit=True etc but including here to be more explicit) it does not seem to upload this metric/data as i cannot see it on browser or through api.

import wandb

wandb.init(project="wanb-test-log-wahoo", config={"a": 1, "b": 2})

num_epochs = 100
num_batches = 1000
eval_every = 1
log_every = 10

running_loss = 1000
eval_metric = 1.0

for i in range(num_epochs):
    for ii in range(num_batches):
        running_loss *= 0.99
        if ii % log_every == 0:
            wandb.log({"train/loss_batch": running_loss})

    if i % eval_every == 0:
        eval_metric += 1 / num_epochs
        wandb.log({"eval/loss": running_loss}, step=i, commit=True, sync=True)

    wandb.log({"train/epoch": i, "train/loss": i / running_loss})

Additional Files
No response
Environment
WandB version: 0.14.0
Python version: 3.10
Additional Context
No response

@grahamannett Thanks for writing in. Few points:

sync is deprecated and it doesn't change the behavior of log.
Now, if you look in the logs of the run (you can find it under the folder you ran your the script from, the path would be something like this: /wandb/run-<date>_<timestemp>-<run_id>/logs/debug-internal.log) you will find warning that says that we drop all the "eval/loss" values, search for  Dropping entry: {'eval/loss'....

Why?

We currently only support logging monotonically increasing steps. Every time you do  wandb.log({"train/loss_batch": running_loss}) without providing step we increment the internal step by 1 so when we get to the wandb.log({"eval/loss": running_loss}, step=i, commit=True, sync=True) the step is way ahead of i and we drop this entry.

How to fix?

If you want to log against a custom value for your step you could try using define_metric. See docs for more details on usage.
If you want to use the same global step you need to make sure the values for the step are monotonically increasing.

Hope this helps, please let us know if you have additional questions.

I have some problems with logging on "same step". I run both validation and test dataset for evaluation in the same training step, and I log them individually. If I input same "step" number to both test and validation score (i.e. accuracy), wandb only logs either one of them. Now I avoid this issue by incrementing "step" by 1 when passing the value to log test scores. Do you have any idea why this happens?


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 9cc6b77:
:clock10: Pending: waiting for processors (22 builds ahead of you)
:warning: Docs Build is busy, currently there are 22 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 9cc6b77:
:clock10: Full build: the average full build time is 7 min(s) 15 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 9cc6b77:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

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


updating of experimental options

With the v4 release, we have an opportunity to simplify our current list of 'experimental' features:

[ ] remove treeshakeClientOnly experimental option (enabled by default since v3.0.0)
[ ] remove configSchema option (enabled by default since v3.3.0)
[ ] remove polyfillVueUseHead (disabled since v3.4.0) - implementable in user-land with plugin
[ ] remove respectNoSSRHeader (disabled since v3.4.0) - implementable in user-land with server middleware

I would also like to remove renderJsonPayloads option (enabled by default since v3.5.0) but there seem to be a number of repositories using it and I think we need to investigate why and help them migrate to JSON payloads first.
In addition, I think we can consider enabling the following options by default sooner than 4.x:

[ ] headNext (cc: @harlan-zw - what do you think?)
[ ] scanPageMeta
[ ] sharedPrerenderData

Thoughts welcome 🙏

polyfillVueUseHead can definitely be removed, @vueuse/head has been sunset for almost a year
headNext may also be worth removing if we're considering moving things under the module key meta, the Capo implementation can be enabled by default in the next Unhead release

Happy with all the considerations 👍🏻
For each removed feature/flag which could be a user-land implementation it'd be great to give an example in the release notes / migration guide / blog post, so people can easily "migrate" if they need said behavior.

Thanks, looking forward to


lizzy-client installation makes senza unusable

If someone installs lizzy-client on an environment where senza is already installed then lizzy-client would somehow overwrite senza and makes it unusable.

Why would you install Lizzy-client and Senza locally?

It happened in our CD toolchains where lizzy client AND senza was installed, and besides that, it doesn't matter if i would install lizzy-client locally or not. The fact is that lizzy-client corrupts a fully functional senza installation.

Ok. Thanks for being so nice.
On Aug 23, 2016 09:51, "Christian Lohmann"<EMAIL_ADDRESS>wrote:

It happened in our CD toolchains where lizzy client AND senza was
installed, and besides that, it doesn't matter if i would install
lizzy-client locally or not. The fact is that lizzy-client corrupts a fully
functional senza installation.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zalando/lizzy-client/issues/77#issuecomment-241654532,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGkXeqWfK7FaG1Tu5ESXkaZJdumnD1Cks5qiqaLgaJpZM4Jpmgm
.

 On the long term the plan is to merge senza and lizzy-client but senza still needs to be refactored before that.
Is there any reason why you need senza installed on your toolchain? We can probably implement it in lizzy and remove your need to have senza installed if you create an issue for it.

so lizzy-client needs to overwrite the senza command

Hi, I'm not so proficient in Python. Could you elaborate on what the alternatives were and why they didn't fit?
Thanks 🙌

@prayerslayer the only way to allow someone to copy "senza create ..." from their local machine to their CI environment and having lizzy handle it seamlessly is for lizzy-client to pretend it's senza.

And what about offering the same API as senza commands, but using lizzy and not touching senza (but maybe calling it under the hodd) so people can easily search & replace on their scripts?

Hmm, why couldn't you just add an alias in the CI environment to emulate senza? I still want to be able to install lizzy-client locally (even if it's only for debugging, playing around) without corrupting Senza...


livereload.js not found on GAE

BUG REPORT or FEATURE REQUEST?:
bug
What happened:
The website perfectly works locally, but once pushed on Google AppEngine, I have a 404 error on livereload.js. and my SVG images are not loaded.
What you expected to happen:
All JS files should be located on my "public/script" folder but I cannot found the livereload.js file.
How to reproduce it (as minimally and precisely as possible):
Here is my app.yaml file:
`runtime: python37
instance_class: F1
handlers:

url: /
static_files: www/public/index.html
upload: www/public/index.html
secure: always
redirect_http_response_code: 301

url: /membres
static_files: www/public/membres/index.html
upload: www/public/membres/index.html
secure: always
redirect_http_response_code: 301

url: /tarifs
static_files: www/public/tarifs/index.html
upload: www/public/tarifs/index.html
secure: always
redirect_http_response_code: 301

url: /cnntact
static_files: www/public/contact/index.html
upload: www/public/contact/index.html
secure: always
redirect_http_response_code: 301

url: /about
static_files: www/public/about/index.html
upload: www/public/about/index.html
secure: always
redirect_http_response_code: 301

url: /*.css
static_dir: www/public/
mime_type: text/css
secure: always
redirect_http_response_code: 301

url: /images/.svg
static_dir: www/public/images/
mime_type: image/svg+xml
secure: always
redirect_http_response_code: 301

url: /scripts/.js
static_dir: www/public/scripts/
mime_type: application/javascript
secure: always
redirect_http_response_code: 301

url: /(.)
static_files: www/public/\1
upload: www/public/(.)
secure: always
redirect_http_response_code: 301`

Anything else we need to know?:
How the livereload.js is generated ?

I found my answer, livereload is used to reload the website content each time we apply a modification.
It is not related to my SVG issue.

Yeah livereload is a Hugo thing helping during development.
It's only injected with the development server and not with the final static files.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 17:42:29 on 2023.07.14
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 0, errors out: 0
Original problem ID: 32658
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=32658

Problem has been resolved in 3m 0s at 17:45:29 on 2023.07.14
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 32658


ComfyUI freezes with save_db Request Entity Too Large error

Hi,
Around yesterday, after updating everything, ComfyUI interface completely freezes every few seconds with the error:
POST http://<IP_ADDRESS>:8188/workspace/save_db 413 (Request Entity Too Large)
I set auto-save of workspace manager to off, but it does not help right away. I rebooted comfy and now the error and freeze have stopped.

hi can you help check network tab and screenshot the network payload for me? thank you!

make sure you screenshot the payload size too

in your settings, can you change this settings to a lower number like 10 or 20, to see if it helps with the issue?

Got screenshot of the payload, it is 10MB

I'll try changing the setting.
My folder of workflows has 96 json files with a total size of 3.3MB

I'll try changing the setting.
My folder of workflows has 96 json files with a total size of 3.3MB

thanks for digging in. I just published a change that may help with this. We are planning to migrate our backend db to sqlite to properly mitigate this issue too in long term.

Thanks! I'll try to update.
I've just lowered the version history number, and also cleared all files in my workflow folder. but seems the payload is till 10.3MB.
I checked the db folder and saw that the changelogs file is very big.

Right the main problem is the changelogs table. I’ve changed the code to not store it to disk anymore. They will now only be stored in browser db (they are not that important so not worth it to backup into disk i think, especially when they can become very big..)

From: pto2k @.>
Sent: Friday, March 29, 2024 12:12:55 AM
To: 11cafe/comfyui-workspace-manager @.>
Cc: Weixuan Fu @.>; Comment @.>
Subject: Re: [11cafe/comfyui-workspace-manager] ComfyUI freezes with save_db Request Entity Too Large error (Issue #325)
Thanks! I'll try to update.
I've just lowered the version history number, and also cleared all files in my workflow folder. but seems the payload is till 10.3MB.
I checked the db folder and saw that the changelogs file is very big.
Snipaste_2024-03-29_00-11-50.png (view on web)https://github.com/11cafe/comfyui-workspace-manager/assets/4364117/f2a4db50-cea0-4901-b28f-b9dd4296d9c9
—
Reply to this email directly, view it on GitHubhttps://github.com/11cafe/comfyui-workspace-manager/issues/325#issuecomment-2025594049, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEMEEONSEFVFGCND4RQOX23Y2Q6QPAVCNFSM6AAAAABFM76QY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRVGU4TIMBUHE.
You are receiving this because you commented.Message ID: @.***>

The workflows payload you're seeing (10.3MB) cannot be reduced now cuz we still need to backup them into disk. But in future we plan to migrate server db to reduce payload size fundamentally.

From: Weixuan F @.>
Sent: Friday, March 29, 2024 12:21:16 AM
To: 11cafe/comfyui-workspace-manager @.>; 11cafe/comfyui-workspace-manager @.>
Cc: Comment @.>
Subject: Re: [11cafe/comfyui-workspace-manager] ComfyUI freezes with save_db Request Entity Too Large error (Issue #325)
Right the main problem is the changelogs table. I’ve changed the code to not store it to disk anymore. They will now only be stored in browser db (they are not that important so not worth it to backup into disk i think, especially when they can become very big..)

From: pto2k @.>
Sent: Friday, March 29, 2024 12:12:55 AM
To: 11cafe/comfyui-workspace-manager @.>
Cc: Weixuan Fu @.>; Comment @.>
Subject: Re: [11cafe/comfyui-workspace-manager] ComfyUI freezes with save_db Request Entity Too Large error (Issue #325)
Thanks! I'll try to update.
I've just lowered the version history number, and also cleared all files in my workflow folder. but seems the payload is till 10.3MB.
I checked the db folder and saw that the changelogs file is very big.
Snipaste_2024-03-29_00-11-50.png (view on web)https://github.com/11cafe/comfyui-workspace-manager/assets/4364117/f2a4db50-cea0-4901-b28f-b9dd4296d9c9
—
Reply to this email directly, view it on GitHubhttps://github.com/11cafe/comfyui-workspace-manager/issues/325#issuecomment-2025594049, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEMEEONSEFVFGCND4RQOX23Y2Q6QPAVCNFSM6AAAAABFM76QY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRVGU4TIMBUHE.
You are receiving this because you commented.Message ID: @.***>

Thanks for the explanation.  And the fix seems to be working =) no more errors
Another question, for the auto-save, does it save on every change in the graph?
If so, I wonder if you can add an option to exclude changes that do not affect the outcome of the graph, for example, node positions, color, etc.. and so they do not save into versions.

Thank you very much for the quick fix!