Use GitStatusCache when it's installed.

Cache status information in external git-status-cache process and retrieve using git-status-cache-posh-client scripts. This pull request uses git-status-cache-posh-client's Get-GitStatusFromCache function when the module is installed. This function returns status information in a format that's easily consumable within posh-git.
The external cache is built on top of libgit2 and dramatically improves performance on large repositories. Cache hits do not become more expensive with larger repositories. Performance of both cache hits and cache misses with the cache is significantly faster than current status retrieval.
Inspired by:

https://github.com/dahlbyk/posh-git/issues/78
https://github.com/dahlbyk/posh-git/pull/52
https://github.com/dahlbyk/posh-git/pull/104
https://github.com/dahlbyk/posh-git/pull/145

My primary motivator was making posh-git usable on some very large repositories I interact with every day; this solution has worked great and has been baking for a while. Recognizing that there are previous attempts at this type of improvement I've attempted to incorporate feedback from the earlier conversations.
I'm happy to address any additional comments or feedback!

Performance
Cost for serving a cache hit in the cache process is generally between 0.1-0.3 ms, but this metric doesn't include the overhead involved in a full request.
The following measurements were taken on git repositories containing the specified file count. Each file was a text file containing a single sentence of text. Each case was run 5 times and the numbers reported below are averages. Each individual measurement was taken with a high resolution timer at 1 ms precision.
Request from git-status-cache-posh-client
The measurements below capture:

Serializing a request to JSON in PowerShell.
Sending the JSON request over the named pipe.
Compute the current git status (cache miss) or retrieving it from the cache (cache hit) int the cache process.
Sending the JSON response over the named pipe back to the PowerShell client.
Deserializing the JSON response into a PSCustomObject

No files modified

cache miss
cache hit

10 file repository
3.0 ms
1.0 ms

100 file respository
3.0 ms
1.0 ms

1,000 file respository
5.2 ms
1.0 ms

10,000 file respository
22.4 ms
1.0 ms

100,000 file repository
176.6 ms
1.0 ms

10 files modified

cache miss
cache hit

10 file repository
2.8 ms
1.0 ms

100 file respository
2.8 ms
1.0 ms

1,000 file respository
4.4 ms
1.0 ms

10,000 file respository
20.6 ms
1.0 ms

100,000 file repository
176.6 ms
1.0 ms

Using git-status-cache-posh-client to back posh-git
The measurements below capture all the steps from the git-status-cache-posh-client section as well as the cost for posh-git to render the prompt. This extra step has a fixed cost of around 7-10 ms.
Costs were gathered using timestamps from posh-git's debug output. Timings for posh-git without git-status-cache and git-status-cache-posh-client are included for in the "posh-git without cache" column for reference.
 Recognizing that there are previous attempts at this type of improvement I've attempted to incorporate feedback from the earlier conversations.

:100: This does seem to address all my concerns with the previous implementations.

[x] Cache is optional.
[x] No loss of information.
[x] Effectively zero changes to the mechanism for writing status.

To test this out:

Clone git-status-cache-posh-client somewhere.
Create a bin/ directory with GitStatusCache.exe from git-status-cache Latest
Add Import-Module <path-to>\git-status-cache-posh-client\GitStatusCachePoshClient to $PROFILE

So far the caching seems to be working exactly as I would expect!

I did just introduce a merge conflict on you... :crying_cat_face:

If you run install.ps1 in git-status-cache-posh-client, it should do the download on your behalf as part of setup as well as add the import to your profile. I like the idea of the warning or auto-download in the case where you enable it through another path and will follow-up on https://github.com/cmarcusreid/git-status-cache-posh-client/issues/4 (may be a few days before I get a chance).

If you run install.ps1 in git-status-cache-posh-client, it should do the download on your behalf as part of setup as well as add the import to your profile.

Ironic that I would overlook an install.ps1 modeled after the one we have here! You should mention that in the README. :grinning:

README clarified. Not sure how I forgot that. :)

README clarified. Not sure how I forgot that. :)

:+1:

I'm hoping to get a bit of visibility on this to kick the tires before merging, but I'm not too worried about it since the caching is optional.

I just tested this and it rocks. Waaay faster.

So much nicer. posh-git was starting to go incredibly slow.

Hi @cmarcusreid ,
this really works well!
Can you consider adding the StashStatus from #215 into your Pipelines?
I tried to file a PR at your repositories, but I'm no C++ guy and can't get the project building :-)

Fails on windows 10, had to add -UseBasicParsing to all wgets in GitStatusCachePoshClient.ps1 to make install.ps1 to run without errors.

Hey @MarkusAmshove!
Stash support was added to the cache for in https://github.com/cmarcusreid/git-status-cache/pull/12. Relevant integration for this PR is in 008836e. If you have both, posh-git should show it when the setting is enabled.
If you started using the cache before this change, you may need to update your cache bits (grab the latest git-status-cache-posh-client and call Update-GitStatusCache to download.)
You can check if the cache is returning stash information by call Get-GitStatusFromCache:
D:\posh-git [useGitStatusCache ≡ +1 ~0 -0 ~]> git stash save "My sample stash."
Saved working directory and index state On useGitStatusCache: My sample stash.
HEAD is now at 008836e Read stash count from cache.
D:\posh-git [useGitStatusCache ≡]> $global:GitPromptSettings.EnableStashStatus = $true
D:\posh-git [useGitStatusCache ≡ (1)]> Get-GitStatusFromCache

Version           : 1
Path              : D:\posh-git
RepoPath          : D:/posh-git/.git/
WorkingDir        : D:/posh-git/
State             :
Branch            : useGitStatusCache
Upstream          : origin/useGitStatusCache
AheadBy           : 0
BehindBy          : 0
IndexAdded        : {}
IndexModified     : {}
IndexDeleted      : {}
IndexTypeChange   : {}
IndexRenamed      : {}
WorkingAdded      : {}
WorkingModified   : {}
WorkingDeleted    : {}
WorkingTypeChange : {}
WorkingRenamed    : {}
WorkingUnreadable : {}
Ignored           : {}
Conflicted        : {}
Stashes           : {@{Name=stash@{0}; Sha1Id=771975d52c2507e9c3a0bb6bc151d66af03f2ea; Message=On useGitStatusCache: My sample stash.}}

@Kantis I don't repro on Windows 10, so I wonder what's different about our environments. Could you clarify the following to help me get this fixed?

What exactly was the failure?
Was this on a call to the install script, retrieving information from the cache, or both?

@cmarcusreid

Dont have it anymore, sadly. Sorry. I can't reproduce it any longer. Assuming it might be working without the -UseBasicParsing now after I installed the vcredist.
When callign the install script

@Kantis Thanks. Are you by chance on a machine without Internet Explorer? I found this:
-UseBasicParsing
Uses the response object for HTML content without Document Object Model (DOM) parsing.
This parameter is required when Internet Explorer is not installed on the computers, such as on a Server Core installation of a Windows Server operating system.

Either way, I've logged issue https://github.com/cmarcusreid/git-status-cache-posh-client/issues/9 on git-status-cache-posh-client and will get this fixed.

@cmarcusreid great stuff :+1: Just got it running and this is amazing. :star:

Works great here too! The speed improvement is incredible!

@dahlbyk: I just wanted to leave a note to check the current status on this PR and to make sure you're not waiting on anything from my end. Thanks!

Honestly I'm just swamped. The positive feedback bodes well, will try to get a final review for you soon. Sorry!

@dahlbyk any further movement on this? I've been testing it for a while and haven't run into any issues

How can I ensure the cache is being utilized? I have the client installed and the Get-GitStatusCacheStatistics works, but only increments when I directly call Get-GitStatusFromCache. If I do a ls within the directory it still takes ages and the stats don't change? I am using the forked posh-git.

@variab1e: If the call count isn't incrementing outside of your manual calls, then posh-git isn't calling it. Double check that you're on the "useGitStatusCache" branch and reload PowerShell. The PR is not included in the master branch.
Once you have the PR you should have be able to explicitly disable or enable via $global:GitPromptSettings.EnableFileStatusFromCache. On the first request posh-git will automatically set this to $true if it finds the cache. See 2d3cf044b43aec5c753a2f7ae87b25009aec5f0e for details of how this is set.
Please let me know if this doesn't resolve!

Any updates on this? :)

@MarkusAmshove Nothing from my end as I'm not aware of any outstanding feedback. I'm still using it daily. :)
Ultimately up to @dahlbyk on whether this makes sense to merge or whether it's more appropriate to continue to live in the fork.

Ultimately up to @dahlbyk on whether this makes sense to merge or whether it's more appropriate to continue to live in the fork.

I do plan to merge, I just haven't had a chance to properly review. :soon: I hope.

This works great! Thanks for putting it together. I was previously waiting ~3 seconds after every command. I would have had to disable posh git if not for this.

Hey @cmarcusreid, I'd like to include this in our next release. Unfortunately, #319 and #326 introduced merge conflicts, and the latter seems like it might require a new bit in the cache response.
If you're not able to resolve the conflicts, can you let us edit this PR?

If you're not able to resolve the conflicts, can you let us edit this PR?

I took a crack at resolving the conflicts (helps a lot to separately commit the indentation change): https://github.com/dahlbyk/posh-git/compare/git-status-cache.

Thanks for the ping. Taking a look at the conflicts now.

Since my last merge of master the Arrays changed to Lists which broke the cache response building code. I've merged in your conflict resolution and updated the cache response building code to add to the Lists rather than overwrite.

Amusing that we both pushed essentially equivalent commits a few minutes apart. In the spirit of #319, I wonder which version will perform better for very large changes? @lzybkr can you tell at a glance, or should we profile it?

Pipe to foreach is definitely slower than AddRange, but if possible, you should avoid the cast to [string[]] because that might copy the collection.
Instead, Get-GitStatusFromCache should ensure those properties implement IEnumerable<string> to make the cast unnecessary.

The response of Get-GitStatusFromCache is built by Convert-FromJson. Unfortunately Convert-FromJson converts arrays to Object[] even though they're strings in this case. I don't think we have a way to eliminate the cast (without doing equivalent work ourselves), but I'm open to suggestions if someone sees a better solution. I've merged 3e13ee8 based on the AddRange note.

if possible, you should avoid the cast to [string[]] because that might copy the collection.

We could use [Linq.Enumerable]::Cast or [Linq.Enumerable]::OfType, though calling static generic methods seems to require a workaround.

I would expect the LINQ version to have comparable cost to the earlier foreach as it also incurs an enumeration. Probably the next step if we want to  drill into this is to profile each option and see if there's an appreciable difference. Do you feel this is a worthwhile investigation?

I expect the LINQ version to always win. I improved the overhead of ForEach-Object some in PowerShell 5.1, but it's still much more expensive than a method call.
I suppose it's possible if you have a huge number of objects (approaching physical memory limits), then the pipeline might win, but for typical workloads sensitive to performance, you should prefer .Net methods over the pipeline where possible.

Got it. Right now we're using the string[] cast. Any intuition on that versus the LINQ cast?

They have differing semantics - LINQ cast will only do a C# cast whereas PowerShell [string[]] cast will convert each element of the array using PowerShell rules, e.g. calling ToString if the object is something other than a string.
That said, a LINQ cast avoids a copy of the array, so in theory it should be better, but it can't be called without reflection (PowerShell does not provide a way to specify a generic argument only used in the return type).
I measured the two options and LINQ is about 3x faster even with the reflection (tested with a 5000 strings in an object[]) using:
# Create 5000 strings
$objArray = @("a") * 5000
$collection = [System.Collections.Generic.List[string]]::new()
$m = [System.Linq.Enumerable].GetMethod('Cast').MakeGenericMethod([string])
$collection.AddRange($m.Invoke($null, (,$objArray)))

Excellent, thanks! I'll make this adjustment when I have a chance (probably sometime over the weekend).

Working on this now, will push to this branch.

Not thrilled with b07bce5, but it seems to work? PowerShell auto-enumerating stuff is tricky to work around. 😦

That is a reasonable concern, but the command invocation overhead is unfortunately not small, I would need to measure to know which is worse.

Per comments on https://github.com/dahlbyk/posh-git/commit/b07bce5ba557f7cdc6a14247171aff6f0837ed9e, I've inlined the reflected method call instead of wrapping it in a PowerShell function.

I know a few folks have mentioned they have been running with this locally for a while. Update to this branch's latest and confirm your projects still work as expected?

It looks like rename operations currently break the prompt. Other operations look fine on my machine.
Manual execution of the cast throws an InvalidOperationException. Unable to cast object of type System.Management.Automation.PSObject to type System.String.

@cmarcusreid Which rename operations?  Is this the rename of the prompt function that the Chocolatey install adds to your profile?  If so, that will be removed before 0.7.0 is released.  Importing the module now puts in place the default posh-git prompt.

Rename operations on files in the index or working directories. :)
I believe it's just the new cast bit in this PR; I'll take a look sometime this weekend if no one beats me to  it.

🎉
Appreciate everyone's patience with this getting this merged. Thanks for the contribution, @cmarcusreid!


Questions & Suggestions

Hi, I constantly monitor GH for new repos that have anything to do with package manager
Usually, it's just spam, but every now, and then I hit a jackpot and find a gem.
So I just want to say thank you for putting this awesome work on here.
I compiled the binary and wanted to test it, it seems like it's still in alpha/beta.
I read on the readme, that this is the successor to your YPM
I wanted to see how you managed repos/packages and came across this: https://github.com/avs-origami/ypm/tree/master/repo/core
I then tried to use arc to create a test package template:
$ arc new test-pkg
-> Created new package test-pkg

$ tree test-pkg/
test-pkg/
├── build
└── package.toml

$ cat test-pkg/build
#!/bin/sh -e

$ cat test-pkg/package.toml

[meta]
version = ""
maintainer = ""
sources = []
checksums = []

[deps]

[mkdeps]

I couldn't find any docs to follow, so I don't know how I should write it
There's docs here at https://avs-origami.github.io/projects/ypm/, but I think this maybe outdated.
Your project greatly interests me, and if you had like to discuss more or need some help adding packages, I can help.

Thanks for your interest in this project!
This is very much still in development. While functions (like build, install, remove, etc) appear to be working, there are still several missing or incomplete features.
With regards to YPM, this project is more closely related to kiss (https://kisslinux.github.io/wiki/package-manager). It's the successor as in, "YPM is not in active development and is riddled with bugs, I'm starting over with something completely different."
If you're interested in testing it out, I do have some repos on github at https://github.com/tincan-linux/repo-core. I would say the easiest way to try it out (without wrecking your system) is to download a kiss chroot from https://codeberg.org/kiss-community/repo/releases, copy arc into the chroot's bin directory, then play around with it.
This project is really just something I'm doing on the side for fun, so expect development to be sporadic. I haven't gotten around to making docs yet, I am planning to do that once the major features are in place (see the readme for progress).
And if you find any bugs, or have any suggestions for improvements / features, feel free to open an issue or a PR!


PIKA-124: Standardize .gitignore

Context
Pikachu is standardizing the .gitignore files for all Pikachu repos.
https://zendesk.atlassian.net/browse/PIKA-124
Approach
Generate .gitignore from
gitignore.io,
and use it in all Pikachu repositories.
There's no standard way to keep the .gitignore files synced across repos right
now, so we might just potentially revisit this manually from time to time.
References
JIRA: PIKA-124
@zendesk/pikachu
Risks
None -- only affects .gitignore

Whoops, ignore this please. Runaway script.


WIP Pr/maacky/4856

testing if not copying over the index.html from public helps

See https://github.com/storybooks/storybook/pull/4856


Script returns error

When running the script, I consistently get this error:  Failure with StatusCode [Forbidden] and ReasonPhrase [Forbidden]
When running against a different tenant/connector setup, it works flawless. What could be causing this?

I get the exact same when testing this on a demo tenant, will try on a different tenant to validate there.

I get the exact same when testing this on a demo tenant, will try on a different tenant to validate there.

Just to update on this, testing on a different tenant worked ok. Seems limited to my demo tenant at the moment.

I'm having the same (reproduceable) problem...

--------------------------  PowerShell Script, output log ...
VERBOSE: Rows already ingested from File Count: 0
VERBOSE: Creating file C:\Users\<%username%>\AppData\Local\Tempctr###D\tmp0001.txt
VERBOSE: Created file with 2 records
VERBOSE: HTTP/1.1 POST with 268-byte payload
VERBOSE: received 1516-byte response of content type application/json
VERBOSE: Content encoding: utf-8
RetryCommand: C:\dev\HRDataConnector\Set-HRFile.v1a.ps1:333:17
Line |
 333 |                  RetryCommand -ScriptBlock {
     |                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Failure with StatusCode [BadRequest] and ReasonPhrase [Bad Request]
RetryCommand: C:\dev\HRDataConnector\Set-HRFile.v1a.ps1:333:17
Line |
 333 |                  RetryCommand -ScriptBlock {
     |                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Error body : Import status : Failed No Records were ingested We weren't able to process the file as it didn't contain the required column(s) :
     | "WORK_EMAIL","TERMINATION_ENTRY_DATE","ASSOCIATE_TERMINATION_DATE"
Write-Error: C:\dev\HRDataConnector\Set-HRFile.v1a.ps1:333:17
Line |
 333 |                  RetryCommand -ScriptBlock {
     |                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Failure with StatusCode [BadRequest] and ReasonPhrase [Bad Request]
VERBOSE: Will retry in [60] seconds

--------------------------  Azure HR Data Connector, Admin Log Entry ...
Import completion time: 11/3/2023 8:33:02 AM (UTC)
Import status : Failed
No Records were ingested
We weren't able to process the file as it didn't contain the required column(s) : "WORK_EMAIL","TERMINATION_ENTRY_DATE","ASSOCIATE_TERMINATION_DATE"

--------------------------  Troubleshooting Steps ...
I tried many things:

Changed Input Path to a path with no spaces
Both with and without "'s around the header column names
Both with and without "'s around the column values
I even copied the input data example from here:

https://learn.microsoft.com/en-us/purview/import-hr-data#step-1-prepare-a-csv-file-with-your-hr-data

--------------------------  Sample Data File, one of the many attempts ...
WORK_EMAIL,TERMINATION_ENTRY_DATE,ASSOCIATE_TERMINATION_DATE
sarad@contoso.com,2019-04-23T15:18:02.4675041+05:30,2019-04-29T15:18:02.4675041+05:30
pilarp@contoso.com,2019-04-24T09:15:49Z,2019-04-29T15:18:02.7117540


def of GO:0043625 "delta DNA polymerase complex"

Hi,
I'm working on transferring annotations from:
id: GO:0005659
name: delta DNA polymerase complex
to:
id: GO:0043625
name: delta DNA polymerase complex
and have some comments about the definition.
Firstly, it seems like it would be useful to transfer the relevant
definition dbxrefs from the original definition to the new one,
especially since the new term doesn't have a comment, or other info,
that allows people to trace the history that this def was largely derived
from the older, obsoleted term.  I didn't look at all three refs of the
original term, but this one, PMID: 12403614, should definitely be
associated with this def as it is the basis of the comment of the human
example.
Secondly, I was wondering if we could revise the definition so that the
human stuff is more of an example. Right now it seems a little like the
main focus.
This is the current def:
A multimeric enzyme complex, which in humans is a heterotetramer of
four subunits of approximately 125, 50, 68 and 12kDa. Functions in
DNA replication, mismatch repair and excision repair.
I'd like to suggest some additions and revisions:
A multimeric DNA polymerase enzyme complex which functions in
DNA replication, mismatch repair and excision repair. Common
characteristics of Pol delta include an active 3-prime to 5-prime
exonuclease activity and low processivity in the absence of PCNA.
There is heterogeneity between enzymes from different sources. In
humans, it is a heterotetramer of four subunits of approximately 125,
50, 68 and 12kDa, while in S. cerevisiae, there are three different
subunits and the enzyme is a hexamer that is a dimer of a
heterotrimer.
PMID: 12403614, PMID: 9745046
Lastly, I was also wondering if we should add two additional terms for"
delta DNA polymerase holoenzyme complex
epsilon DNA polymerase holoenzyme complex
based on this statement from the reveiw I'm reading (PMID: 9745046)
"The complex of Pol delta or Pol epsilon with PCNA and replication
factor C (RFC) is referred to as the holoenzyme form of the respective
enzyme."
thanks,
-Karen
Reported by: krchristie
Original Ticket: "geneontology/ontology-requests/3247":https://sourceforge.net/p/geneontology/ontology-requests/3247

thanks www.movied.org


Property schema using parent type instead of correct child type

Describe the issue
The part of swagger specifies that property osDiskImage is type GalleryOSDiskImage
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/compute/resource-manager/Microsoft.Compute/stable/2019-07-01/gallery.json#L1810-L1834
However after m4, it is now its parent type GalleryDiskImage.
                    - schema: &ref_355
                        type: object
                        apiVersions:
                          - version: '2019-07-01'
                        children:
                          all:
                            - &ref_357
                              type: object
                              apiVersions:
                                - version: '2019-07-01'
                              parents:
                                all:
                                  - *ref_355
                                immediate:
                                  - *ref_355
                              properties:
                                - schema: *ref_356
                                  required: true
                                  serializedName: lun
                                  language:
                                    default:
                                      name: lun
                                      description: >-
                                        This property specifies the logical unit number of the data disk. This value is used to identify data disks within the Virtual Machine and therefore must be unique for each data disk attached to the
                                        Virtual Machine.
                                  protocol: {}
                              serializationFormats:
                                - json
                              usage:
                                - input
                                - output
                              language:
                                default:
                                  name: GalleryDataDiskImage
                                  description: This is the data disk image.
                                  namespace: ''
                              protocol: {}
                            - &ref_358
                              type: object
                              apiVersions:
                                - version: '2019-07-01'
                              parents:
                                all:
                                  - *ref_355
                                immediate:
                                  - *ref_355
                              serializationFormats:
                                - json
                              usage:
                                - input
                                - output
                              language:
                                default:
                                  name: GalleryOSDiskImage
                                  description: This is the OS disk image.
                                  namespace: ''
                              protocol: {}
                          immediate:
                            - *ref_357
                            - *ref_358
                        properties:
                          - schema: *ref_359
                            readOnly: true
                            serializedName: sizeInGB
                            language:
                              default:
                                name: sizeInGB
                                description: This property indicates the size of the VHD to be created.
                            protocol: {}
                          - schema: *ref_360
                            serializedName: hostCaching
                            language:
                              default:
                                name: hostCaching
                                description: 'The host caching of the disk. Valid values are ''None'', ''ReadOnly'', and ''ReadWrite'''
                            protocol: {}
                          - schema: *ref_361
                            serializedName: source
                            language:
                              default:
                                name: source
                                description: The gallery artifact version source.
                            protocol: {}
                        serializationFormats:
                          - json
                        usage:
                          - input
                          - output
                        language:
                          default:
                            name: GalleryDiskImage
                            description: This is the disk image base class.
                            namespace: ''
                        protocol: {}
                      serializedName: osDiskImage
                      language:
                        default:
                          name: osDiskImage
                          description: This is the disk image base class.
                      protocol: {}

** REQUIRED **
ModelerFour version: 4.13.351
AutoRest-core version: 3.0.6282
CommandLine:
OpenAPI/Swagger file: https://github.com/Azure/azure-rest-api-specs/blob/master/specification/compute/resource-manager/Microsoft.Compute/stable/2019-07-01/gallery.json

Looking into this, I don't have an explanation quite yet.

@daviwil
Any update? Still reproduceable on latest core 3.0.6324 and m4 4.15.421
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/compute/resource-manager/Microsoft.Compute/stable/2019-12-01/gallery.json#L2261-L2263


Draft: Metal capture support for the presented MTLDrawable

Description

Generate the capture thumbnail image from the presented MTLTexture
Serialize the capture PresentedImage ID in the CaptureEnd chunk.

Added CA::MetalLayer::setFramebufferOnly to metal-cpp and deleted helper ObjC::CAMetalLayer_Set_framebufferOnly

https://github.com/baldurk/renderdoc/pull/2661


Handle Camunda Rest Failures

I came across an issue when trying to complete a Service Task. Ultimately, the process I was trying to complete errors out during a downstream Script Task. This causes Camunda to issue a 500 as the response to the request external-task/{taskId}/complete.
This gave me some confusion because the task seems to complete without issue. However, the process would still be waiting on the same Service Task. Tracking this down, it looks like ExternalTaskCamundaClient is not handling failed requests.
private async Task<HttpResponseMessage> SendRequest(string path, object body, CancellationToken cancellationToken)
{
    var basePath = _httpClient.BaseAddress.AbsolutePath.TrimEnd('/');
    var requestPath = $"{basePath}/{path}";
    var response = await _httpClient.PostJsonAsync(requestPath, body, cancellationToken);
    return response;
}

I know this is still version 0, but I just wanted to bring it to your attention in case you were not aware.
I love this library! Keep up the good work!

@sreagleeye Thank you for your issue. It will be fixed soon.

@sreagleeye Could you provide bpmn and handler to check that case?

CompletionFailure.bpmn.txt
[HandlerTopics("completesuccessfully", LockDuration = 10_000)]
public class CompleteSuccessfullyTaskHandler : IExternalTaskHandler
{
    public async Task<IExecutionResult> Process(ExternalTask externalTask)
    {
        return await Task.FromResult(new CompleteResult(new Dictionary<string, Variable>()));
    }
}

It seems to be fixed in 0.5.1. Can you check it?

This looks good! Once the ExternalTaskHandler completes, I am getting an incident on the service task. The stack trace for the incident is showing the error message from the failed request.


add team to checklist [fixes #129667517]

depends on https://github.com/cppforlife/checkman/pull/71

ok, will fix

thanks!


Reproduce should be the standard behavior.

In most cases --reproduce is used when checking out a new project and building it.
This should be replaced with something like --unpinned.

I think I agree with this. I think something like courier --update can handle the case where we want to pull the latest.

I'm not sure --update is correct because it just means pulling from deps.json rather than pins.json. deps.json may not contain floating references to repos which essentially means you aren't necessarily updating anything. However I think if you are looking at it from the point of view that you are "updating the pins" then it makes more sense.

+1 for adding courier --update and using pins.json by default when no flags are supplied.
We should also consider keeping the --reproduce flag for a few versions. The flag would have no effect (since using pins.json would be the default action), but would help with migration.


ListView onEndReached error

if contentComponent use ListView , and set ListView onEndReached .   will auto run onEndReached.

This is a react-native thing, more so than the component.  To test this, you make just a ListView with said callback, and it'll still trigger before anything is reached.  If you set onEndReachedThreshold in the ListView as well, you could have it not be called (I set it to 0, and it had to be scrolled to the bottom before it was called, for example).


Title: Improve About us Section

Is your feature request related to a problem? Please describe.
hii sir , i want to change about us section .change UI about section make blog more attractive and user friendly.
Describe the solution you'd like.
Change Images.
Improve About us content with proper heading along with SEO.
Describe alternatives you've considered.
Adding Images and Information
Add any other context or screenshots about the feature request here.
No response

Sure @singhvaishu

Thank you so much.
On Fri, 23 Jun, 2023, 12:27 am Agamjot Singh, @.***>
wrote:

Assigned #491 https://github.com/agamjotsingh18/codesetgo/issues/491 to
@singhvaishu https://github.com/singhvaishu.
—
Reply to this email directly, view it on GitHub
https://github.com/agamjotsingh18/codesetgo/issues/491#event-9610243600,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWQBEFGUIZMHH2443DMZDSLXMSIPZANCNFSM6AAAAAAZPVYGTA
.
You are receiving this because you were assigned.Message ID:
@.***>

@agamjotsingh18 can you assign this task to me under GSSoC'23

@AvinashChourasia  it's already assigned me . Currently  I'm working on it.


Invitation page closes #50

Coverage decreased (-0.1%) to 99.866% when pulling e1da39f23dd66cea81a773bb16e395d28ab8bcd4 on invitation-page-#50 into 4e70d7e581e338f83e88b480228a00efad3320e9 on develop.

Coverage remained the same at 100.0% when pulling c0776b8369b19bec7c8d5c33eb7404831d466e08 on invitation-page-#50 into 4e70d7e581e338f83e88b480228a00efad3320e9 on develop.


Bug fix: Fix bug in _map_template_vars method of BasePromptTemplate that was failing to properly assign the passed template_var_mappings

Description
For use cases like this:
template_var_1 = 'tv1'

new_prompt = PromptTemplate("This is template var 1: {template_var_1}",
                             template_var_mappings={'template_var_1': template_var_1})
new_prompt.format()

gives this error: KeyError: 'template_var_1'
The _map_template_vars method of BasePromptTemplate wasn't properly utilizing the template_var_mappings
After the change, now the above code works and outputs: 'This is template var 1: tv1'
Also after the change, If the template variable is passed as keyword argument, then the initial value will be overridden and passed argument will have precedence. Previously this was giving an error as well.
new_prompt.format(template_var_1='tv2')

# Outputs: 'This is template var 1: tv2'

Type of Change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

[x] I ran the code that caused the error initially after the change, and checked that it works now
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

Sorry, I had to close this, I looked at some of the failed tests and realized the _map_template_vars() method was used for a different use case than what I anticipated, hence closing this.


OSS standardization changes

resolves https://github.com/dbt-labs/dbt-core/issues/9221
Description
add CONTRIBUTING.md and dependabot.yml. Modify readme to bring this OSS repo up to the minimal standards for OSS at dbt Labs.
Also added templates for PRs.
This repo has more significant changes that need to be made per https://github.com/dbt-labs/dbt-core/issues/8409

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


Support overriding enum values for branch 1.5

I am using the 1.x.x version of the swagger-maven-plugin to generate OpenAPI v2 specification from my annotated model classes. Unfortunately, switching to OpenAPI v3 is not an option at the moment.
I need to generate enum values that have spaces or other special characters. This is currently not possible with the 1.x.x version. The generator always picks the name of the enum from the Java class, which has limits from the java language syntax. The latest release of the 2.x.x. swagger-api had an enhancement for that problem (issue #3552). This enhancement allows to annotate the enum values with @JsonProperty or a getter method with @JsonValue.
It would be nice, if this feature would also be available when generating OpenAPI v2 specs with the 1.x.x version.

Thanks for reporting and PR! implemented in #3617


Update partyfunctioncodes.csv

Adding the VGM responsible party, needed for submission of the VGM, and also a regular feature on shipping instructions from cargo owners, to forwarders and ocean carriers.
Code SPC is a UN standardised and EDI code see https://service.unece.org/trade/untdid/d20a/tred/tred3035.htm
Removed some redundant white space also

Kshitij has agreed to merge this :-)


Create Duplicate Frame Functionality

Added code to create a duplicate frame if an annotation is selected. Frame will have the same bounds as the last frame as the last frame with a time less than the current time. A duplicate frame will not be created if the current time minus the previous frame time is less than SAME_FRAME_THRESHOLD.

Has a bit of dependency on pull request #110 duplicate wont work after moving scrubber to new place because of the scrubber's focus trigger which is fixed in that pull request


[Bug]: Chrome browser requests leak memory when running with Puppeteer

Bug description
Steps:

Load a page in Chrome using puppeteer
Start executing requests from that page.
After about 30 seconds, you will see Chromium Helper (Renderer) memory starting to tick up by ~ 0.5 megabytes/second

Seems to happen on Mac and Windows.
Doesn't seem to happen in Firefox. Doesn't seem to happen if you disconnect puppeteer after setup.
Reproduction (run in node, requires Puppeteer):
require('http')
  .createServer(function (request, response) {
    console.log('received request');
    response.end('booyah');
  })
  .listen(20547);

const runPuppeteer = async () => {
  const browser = await require('puppeteer').launch();
  const page = await browser.newPage();

  const html = `<!DOCTYPE html>

`;
  await page.setContent(html);
};

runPuppeteer();

// Make run forever
setTimeout(() => {
  console.log('done');
}, 100000000);

Thanks!
Puppeteer version
19.2.2, think it is present in older versions too
Node.js version
18.11.0
npm version
8.19.2
What operating system are you seeing the problem on?
macOS
Configuration file
No response
Relevant log output
No response

Sounds a lot like https://github.com/puppeteer/puppeteer/issues/9186 Probably network manager is leaking some request content.

When I remove the .send('Network.enable') command from NetworkManager, I no longer have this issue. Sending 'Network.disable' afterwords also works, but I don't know of any way to get the CDPSession for the page without manually modifying puppeteer.
It seems like the memory is leaking within Chrome itself when the network is enabled. I don't see anything in the Chrome DevTools Protocol about disposing of requests. Does it just hold onto them indefinitely?

there is page._client() I think, although it's an internal method, it should not require modifying puppeteer. So you are saying that if you invoke Network.disable after requests have been made, the memory gets freed? Chrome definitely has a cache for all network requests because you can get request data Network.getResponseBody but I don't think holds onto them indefinitely. That should be also configurable via Network.enable params although Puppeteer uses the default configuration: https://chromedevtools.github.io/devtools-protocol/tot/Network/#method-enable

I'm using Network.disable proactively, calling it once right after I open the page. It looks like that clears out any existing request data and stops any more from being held onto.
I think chrome must otherwise hold onto requests indefinitely. I took a quick look at the chromium codebase, but I didn't see an obvious spot confirming that. I can see that being a good default for use with automated testing, but for long-running production instances it is problematic.

I believe this is the same issue as #9186. You should check the memory usage with inspector to find the leak location.

I don't think this is something you can debug with the memory usage inspector because the leak occurs in the chrome process itself, not javascript.

i am noticing this slow memory leak when deploying a small puppeteer server to digital ocean:

i am trying to work around this and looking for any solutions if anyone has.

@gregg-cbs memory usage might not be a problem on its own, the memory might get released later, e.g., if it becomes really low. Do you actually see out of memory crashes or swapping?

@gregg-cbs memory usage might not be a problem on its own, the memory might get released later, e.g., if it becomes really low. Do you actually see out of memory crashes or swapping?

No I have not seen that yet because i keep restarting the server to reduce ram consumption. The ram is on 73%, up 2% from yesterday.
My worry is that I dont have much ram left to operate with and this server is expected to do quite a lot of processing so if without any computations we are sitting at 70-80% ram usage then what is going to happen when traffic starts coming and the server has to work?
This server is going to fall over for sure because if I test this locally and process 100 pages to pdf the ram spikes up to 360mbs, this box only has 51mbs spare.

@gregg-cbs there is not reason for the server not to use available memory so pure memory usage is not a good stat (e.g., Linux caches files by default so generally the memory usage always grows but the memory can be freed up if needed), the OS and v8 would free up memory as soon as it is needed for something. In general, if you open a page/do things/close the page and do not have many concurrent pages open, it should be alright. In general, 512mb is not much for a browser dealing with many pages at once. So you want to rate limit. Also, you can stress test this locally and see if there are any out of memory errors.  Additional tip is to use browser.createBrowserContext to isolate the pages even better.

Each open tab requires at least 30-40 MB of memory, so you cannot open 30 pages at the same time on that server. You need to make sure that you only process 1 (or N) pages at a time and queue other HTTP requests.

Yesterday was emotional. I changed the puppeteer server to have a queue and to response via a webhook. So now i process 3 pdfs at a time ensuring i only use x amount of ram. That was a nice and necessary change so thank you for the suggestion.
The ram leak is still apparent and will catch me out at some point so I will have to investigate chrome versions and linux stuff to see how to alleviate it.

I played with the example a little more and although I see the memory increase in Chrome, I think it is expected since CDP retains request data to implement methods like https://chromedevtools.github.io/devtools-protocol/tot/Network/#method-getResponseBody that provide network data after responses completed.
I believe Chrome will clean it up if it runs low on memory and I do not think it will cause crashes.
Re-opening a page would also force release network data:
const puppeteer = require('puppeteer');

async function main() {
  require('http')
    .createServer(function (request, response) {
      console.log('received request');
      response.end('booyah');
    })
    .listen(20547);

  const browser = await require('puppeteer').launch();
  let page = await browser.newPage();

  const runPuppeteer = async () => {
    await page?.close();
   page = await browser.newPage();
    const html = `<!DOCTYPE html>

  `;
    await page.setContent(html);
  };

  runPuppeteer();

  setInterval(() => {
    runPuppeteer();
  }, 10000);

  // Make run forever
  setTimeout(() => {
    console.log('done');
  }, 100000000);
}

main()

@OrKoN - Thats what you/someone said "I believe Chrome will clean it up if it runs low on memory and I do not think it will cause crashes." - I can confirm memory and disk space is not released and I have experience both disk and memory full issues.
I cant remember what I did but i have solved the memory issue, i think i disabled some chrome functionality with args.
File space still becomes an issue because of dev profiles not being deleted and it seems the only way to clear dev profiles is to run a bash script every day that removes all profiles. On my server I ran through 24GBs of space because of this issue.

@gregg-cbs the clean up of the profile should be happening on the browser process exit https://github.com/puppeteer/puppeteer/blob/main/packages/puppeteer-core/src/node/BrowserLauncher.ts#L114 in the normal course of action, is it not invoked for you?  (disk issue is a different our though)

Looking at my code im not doing a browser exit after every page completion because the overhead of opening and closing the browser for every request is too much for my little server.
So what i do is shut down the browser every 2 hours. However this does not alleviate the file space issue and looking at the code suggested by this comment i dont think thats going to solve my problem either.
However this thread is for the memory leak so lets keep the discussion of disk space issue at the appropriate thread over here


Using Depreciated Module Distribution.PackageDescription.Parse

The code is using module ‘Distribution.PackageDescription.Parse’ which is apparently depreciated in Cabal 2.1.0. Hence it is returning this error
src/Hadrian/Haskell/Cabal/Parse.hs:18:1: error:
Failed to load interface for ‘Distribution.PackageDescription.Parse’
It is a member of the hidden package ‘Cabal-<IP_ADDRESS>’.
Perhaps you need to add ‘Cabal’ to the build-depends in your .cabal file.
It is a member of the hidden package ‘Cabal-<IP_ADDRESS>’.
Perhaps you need to add ‘Cabal’ to the build-depends in your .cabal file.
I believe this will be useful:
https://github.com/haskell/cabal/wiki/2.2-migration-guide
@snowleopard

@chitrak7 It looks like Hadrian submodule in GHC has fallen quite behind Hadrian HEAD, so I've just updated the submodule: https://github.com/ghc/ghc/commit/bf3f0a6808b89504a993dc9f2839daff9eaf6f2a.
This issue you refer to here has been fixed in #498.
Thanks for reporting! In future you can just do git pull https://github.com/snowleopard/hadrian.git in the Hadrian subdirectory of GHC to bring the submodule in sync with the HEAD.


Handle the case of have multi confirmation modal

Describe the bug
Version test: v1.1.0(220)
Device test: IP 11
To reproduce the bug
Steps:

Go to the Confirmation transaction screen when the transfer
Sign from Wallet connect
Sign the transaction in steps 1+2
Observe the displayed screen

Actual: Display the Amount screen
https://github.com/Koniverse/SubWallet-Mobile/assets/117999489/bf7d4a0d-4713-4864-a6ae-640cc4cf2b97
Expect: Display the Success screen

Version test: PR #1132 17:30 15/1/2023
Device test: IP 11
Validation passed.
https://github.com/Koniverse/SubWallet-Mobile/assets/117999489/a708c201-0ce6-4342-a730-5fc6f169159d


upgrade helix to 1.0.2

Description
In Apache Helix 0.9.8, a very expensive Jackson ObjectMapper is constructed for each and every serialized ZNRecord in ZNRecordSerializer:
  @Override
  public byte[] serialize(Object data) {
   ...
    // do serialization
    ObjectMapper mapper = new ObjectMapper();
    SerializationConfig serializationConfig = mapper.getSerializationConfig();
    serializationConfig.set(SerializationConfig.Feature.INDENT_OUTPUT, true);
    serializationConfig.set(SerializationConfig.Feature.AUTO_DETECT_FIELDS, true);
    serializationConfig.set(SerializationConfig.Feature.CAN_OVERRIDE_ACCESS_MODIFIERS, true);
    ByteArrayOutputStream baos = new ByteArrayOutputStream();
    byte[] serializedBytes;
    boolean isCompressed = false;

    try {
      mapper.writeValue(baos, data);
      serializedBytes = baos.toByteArray();
      ...

This was fixed last year: https://github.com/apache/helix/pull/1293/files#diff-3eaf04c3cf3dc03110404530dd4d18d7b9dbb8afeefebe4b4f57e73e043fe274R42
Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes

Documentation

Some packages were moved around in 1.0.0 and this ended up being quite a large change.

Codecov Report

Merging #7500 (c4ea3bc) into master (70374e2) will decrease coverage by 2.19%.
The diff coverage is n/a.

:exclamation: Current head c4ea3bc differs from pull request most recent head e05ae78. Consider uploading reports for the commit e05ae78 to get more accurate results

@@             Coverage Diff              @@
##             master    #7500      +/-   ##
============================================
- Coverage     72.00%   69.81%   -2.20%
+ Complexity     3404     3321      -83
============================================
  Files          1523     1129     -394
  Lines         75509    53402   -22107
  Branches      11004     8043    -2961
============================================
- Hits          54373    37283   -17090
+ Misses        17480    13472    -4008
+ Partials       3656     2647    -1009

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
69.81% <ø> (-0.02%)
:arrow_down:

unittests2
?

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

Impacted Files
Coverage Δ

...he/pinot/common/assignment/InstancePartitions.java
0.00% <ø> (-41.18%)
:arrow_down:

...not/common/assignment/InstancePartitionsUtils.java
0.00% <ø> (-67.50%)
:arrow_down:

...rg/apache/pinot/common/lineage/SegmentLineage.java
84.78% <ø> (-10.87%)
:arrow_down:

...not/common/lineage/SegmentLineageAccessHelper.java
0.00% <ø> (-88.89%)
:arrow_down:

...ot/common/messages/RoutingTableRebuildMessage.java
0.00% <ø> (-81.82%)
:arrow_down:

.../pinot/common/messages/RunPeriodicTaskMessage.java
0.00% <ø> (ø)

...e/pinot/common/messages/SegmentRefreshMessage.java
0.00% <ø> (-84.62%)
:arrow_down:

...he/pinot/common/messages/SegmentReloadMessage.java
0.00% <ø> (-100.00%)
:arrow_down:

...not/common/messages/TableConfigRefreshMessage.java
0.00% <ø> (-81.82%)
:arrow_down:

...ache/pinot/common/metadata/ZKMetadataProvider.java
20.30% <ø> (-62.41%)
:arrow_down:

... and 578 more

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 70374e2...e05ae78. Read the comment docs.

There seem to be nontrivial behaviour changes in Helix 1.0 which make this more challenging than expected.

@richardstartin we are planning to upgrade to helix 1.0. We are working closely with the the helix team in this regard. Helix 1.0 has been out for a while now, and is considered to be stable at this point. However, it is a source incompatible change. So, we need to move carefully.  We will raise an issue for it soon, and work on that.
We will probably do that in a branch, and merge to master when everything looks good in Linkedin production environment.
Meanwhile, can you please describe the behavior change you found?

Copy pasting message from @richardstartin  before it disappears from chat
hi Subbu, the upgrade wasn't simple and it's possible I made some mistakes doing it, but several controller tests time out on my branch. E.g. in ControllerInstanceToggleTest the cluster doesn't get set up properly.
09:26:27.461 [ZkClient-EventThread-33-localhost:2191] ERROR org.apache.helix.zookeeper.zkclient.callback.ZkAsyncCallbacks - Interrupted waiting for success
java.lang.InterruptedException: null
at java.lang.Object.wait(Native Method) ~[?:?]
at java.lang.Object.wait(Object.java:328) ~[?:?]
at org.apache.helix.zookeeper.zkclient.callback.ZkAsyncCallbacks$DefaultCallback.waitForSuccess(ZkAsyncCallbacks.java:249) [helix-core-1.0.2.jar:1.0.2]
at org.apache.helix.zookeeper.zkclient.ZkClient.issueSync(ZkClient.java:1306) [helix-core-1.0.2.jar:1.0.2]
at org.apache.helix.zookeeper.zkclient.ZkClient.access$300(ZkClient.java:85) [helix-core-1.0.2.jar:1.0.2]
at org.apache.helix.zookeeper.zkclient.ZkClient$4.run(ZkClient.java:1332) [helix-core-1.0.2.jar:1.0.2]
at org.apache.helix.zookeeper.zkclient.ZkEventThread.run(ZkEventThread.java:99) [helix-core-1.0.2.jar:1.0.2]
09:26:30.819 [HelixController-pipeline-default-ControllerTestUtils-(ed00bafa_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:30.828 [HelixController-pipeline-default-ControllerTestUtils-(6ca9c1f1_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:30.834 [HelixController-pipeline-default-ControllerTestUtils-(a5f87de1_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:30.841 [HelixController-pipeline-default-ControllerTestUtils-(3801afe7_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:30.852 [HelixController-pipeline-default-ControllerTestUtils-(3805570d_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:31.251 [HelixController-pipeline-default-ControllerTestUtils-(aaec8ce8_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:31.277 [HelixController-pipeline-default-ControllerTestUtils-(d8cc3e98_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:31.289 [HelixController-pipeline-default-ControllerTestUtils-(1b94a4b9_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:31.297 [HelixController-pipeline-default-ControllerTestUtils-(842b241b_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:31.307 [HelixController-pipeline-default-ControllerTestUtils-(0b4b5414_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:31.314 [HelixController-pipeline-default-ControllerTestUtils-(8bedd9f7_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
09:26:31.320 [HelixController-pipeline-default-ControllerTestUtils-(6477b0f8_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []
Oct 01, 2021 9:26:37 AM org.glassfish.grizzly.http.server.NetworkListener start
INFO: Started listener bound to [<IP_ADDRESS>:18998]
Oct 01, 2021 9:26:37 AM org.glassfish.grizzly.http.server.HttpServer start
INFO: [HttpServer] Started.
09:26:41.821 [ZkClient-EventThread-320-localhost:2191] ERROR org.apache.helix.messaging.handling.HelixTaskExecutor - Message 1375bc51-25a5-4e8e-a291-e407ecb959c2 cannot be processed: 1375bc51-25a5-4e8e-a291-e407ecb959c2, {CREATE_TIMESTAMP=1633076801818, ClusterEventName=InstanceConfigChange, FROM_STATE=ONLINE, MSG_ID=1375bc51-25a5-4e8e-a291-e407ecb959c2, MSG_STATE=new, MSG_TYPE=STATE_TRANSITION, PARTITION_NAME=toggleTable72090524845971, RESOURCE_NAME=toggleTable_OFFLINE, RESOURCE_TAG=toggleTable_OFFLINE, RETRY_COUNT=3, SRC_NAME=localhost_18998, SRC_SESSION_ID=100044bca960003, STATE_MODEL_DEF=SegmentOnlineOfflineStateModel, STATE_MODEL_FACTORY_NAME=DEFAULT, TGT_NAME=Server_localhost_3, TGT_SESSION_ID=100044bca960018, TO_STATE=OFFLINE}{}{}Partition toggleTable72090524845971 current state is same as toState (ONLINE->OFFLINE) from message.
09:26:41.822 [ZkClient-EventThread-320-localhost:2191] ERROR org.apache.helix.util.StatusUpdateUtil - StatusUpdate Error record: 100044bca960018__toggleTable_OFFLINE, {}{HELIX_ERROR     20211001-092641.000821 STATE_TRANSITION f04e6b17-edab-495a-bf33-ce056dec5c3a={AdditionalInfo=Partition toggleTable72090524845971 current state is same as toState (ONLINE->OFFLINE) from message., Class=class org.apache.helix.participant.HelixStateMachineEngine, MSG_ID=1375bc51-25a5-4e8e-a291-e407ecb959c2, Message state=NEW}}{}
My only motivation to upgrade was to avoid creating an ObjectMapper on each ZN serialization and it was taking more time than justified. So if you plan to do it at LinkedIn with the Helix team I think it's better that way.

Hey I am getting the same issue above. Any solution?

[HelixController-pipeline-default-ControllerTestUtils-(6477b0f8_DEFAULT)] ERROR org.apache.helix.controller.rebalancer.DelayedAutoRebalancer - No instances or active instances available for resource leadControllerResource, allInstances: [], liveInstances: [], activeInstances: []

@monicaluodialpad Which Helix version are you running?


RadioButtons selects with focus navigation when using gamepad

Describe the bug
When navigating through a RadioButtons list with gamepad, notice that the selection of the items follows focus.
Steps to reproduce the bug

Create an app and a RadioButtons list
Navigate through the list with a gamepad or Xbox controller
Observe that the selection follows focus, which is expected behavior with the keyboard, but not with gamepad

Expected behavior

Can navigate the list without selection following focus
Must explicitly press the A button to select an item in the RadioButtons list

Version Info
WinUI 2.1 Preview

Windows 10 version
Saw the problem?

Insider Build (18855)
Yes

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Anniversary Update (14393)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox
Potentially

Surface Hub

IoT

Additional context

This was fixed with the migration to repeater


node-gyp test with node.js v6.x doesn't work on travis

Summary
I added a test for native build modules to increase the coverage, but time-out happened only on node.js v6.
Steps to reproduce behavior
https://github.com/depjs/dep/pull/31
Expected behavior
all green.
Actual behavior
https://travis-ci.org/depjs/dep/builds/273831601

closing via https://github.com/depjs/dep/pull/37


Hard Delete Option

Does it make sense to expose a hard delete option for folks that don't want to soft-delete?

I think adding an options hash to delete(data, userId, _options_) makes sense, with ability to use options.__hardDelete__ = true means hard-delete ...

Closing, will revisit if this comes up again.


chore(shipyard-controller): cleanup code to do propper error formatting/handling

Action Item
The current code of the ShipyardController uses constructs like
if err == db.ErrNoEventFound {
	return ErrSequenceNotFound
}

to check for a given error type. However, comparing errors with == will fail on wrapped errors. (%w)
Clean up the code to use errors.Is/As as well as propper error wrapping to handle and format all kinds of errors.

We should split this ticket on a component based.


Docs(web-react, web-twig): Add ScrollView demo #DS-906

Description
ScrollView demos are now same in all packages
Additional info
In web demo there is also Custom Shadowsversion, but we don't have it integrated in react and twig yet.
Issue reference
Demo: ScrollView má ve všech technologiích stejná dema

coverage: 96.46%. remained the same when pulling cfa683d166e442ed5bde52f8059edfd848133852 on docs/DS-906-scrollview-demo into 4e1b5ebd2a3989261d4b1f04735a5025d6ee696d on main.

Although it is looking good, I am missing the purple color in the Custom shadows section.

In web demo there is also Custom Shadowsversion, but we don't have it integrated in react and twig yet.

Can you add it to the unified demo CSS, please?

In web demo there is also Custom Shadowsversion, but we don't have it integrated in react and twig yet.

Can you add it to the unified demo CSS, please?

we decided to add a bug issue instead


Race condition causing Scatter to never connect

When using the browser extension, if ScatterJS.scatter.connect is called after scatterLoaded would've been called then Scatter will never connect and timeout.
I think this code would replicate the issue (untested):
setTimeout(() => {
   ScatterJS.scatter.connect("testing").then(connected => console.log(connected));
}, 10000);

The fix would be for the connect function to resolve immediately if Scatter has already been loaded/connected.

Doesn't seem to be the case.
I just set a 10 second timeout and it still caught the extension.

Hm, any idea why I'm seeing this error then?

Can you help me debug this issue then? I'm not able to connect to Scatter with scatter-js but I can use the web extension fine.

I have Scatter Classic installed, and I've tried using scatter-js/dist/scatter.cjs and  scatter-js/dist/scatter.min.js, same issue with both.

Hit me up on the dev telegram tomorrow and I'll see what we can do

I 'm having the same issue as @liamcurry
Sometimes the connect promise resolves by timing out after 10 seconds.
Btw, I get the same request failure with status code 0, but this happens even when the promise resolves correctly. But from the code it looks like this is for Scatter Desktop only anyway and I'm testing with the Chrome extension only.
client.js?030d:82 [HMR] connected
polling-xhr.js?9d7b:263 GET http://<IP_ADDRESS>:50005/socket.io/?EIO=3&transport=polling&t=MKw0cAi 0 ()

Other than that nothing is happening.
I couldn't find a consistent reproduction right now, it fails randomly about 1/3 of the time. I'll see if I can find out something more.

Make sure you are doing import ScatterJS from.. in your base file ( main.js or equivalent, first thing that happens ). That way it sets up the addEventListener right when the page loads.
You don't have to call .connect right away, but it should set up the listener first.
This is something I'll have to fix with an interval function in the future to continuously check if scatter classic is loaded onto the DOM, but for now this should work.

For what it's worth this is also an issue with the web extension. If you bind to the scatterLoaded event after it's already been fired then it will seem like Scatter was never loaded. I'm getting around this by simply checking for window.scatter before setting up the event listener. Would that be an acceptable solution @nsjames ?

Looks like this could be implemented here: https://github.com/GetScatter/scatter-js/blob/master/src/scatter.js#L189
This section could be changed to the following (untested):
let holder = new Holder(new Scatter());
if(typeof window !== 'undefined') {
    // Check if extension has already been loaded
    if(typeof window.scatter !== 'undefined') {
        holder.scatter = window.scatter;
        holder.scatter.isExtension = true;
        holder.scatter.connect = () => new Promise(resolve => resolve(true));
    } else {
        window.scatter = holder.scatter;
    }
}

// Catching extension instead of Desktop
if(typeof document !== 'undefined'){
    document.addEventListener('scatterLoaded', scatterExtension => {
        holder.scatter = window.scatter;
        holder.scatter.isExtension = true;
        holder.scatter.connect = () => new Promise(resolve => resolve(true));
    });
}

I'm happy to create a PR for this if you approve of this approach.

@liamcurry I tested your code and it works fine. I would vote for this solution as it's more reliable than importing scatter-js early and hoping that the scatterLoaded event hasn't been fired yet.

FYI @MrToph @nsjames I created PR #5 to fix this issue. You can try it out with yarn add sagan-software/scatter-js.

There's a new version out which should handle this: https://github.com/GetScatter/scatter-js/commit/01d29857af1d26abb96a0e47673d7fbd6e38bcdc
Please pull 2.4.0 and see if it catches late

I can confirm that 2.4.0 fixes the issue. Thanks!


chore: correct comments per fields in BarmanCredentials

Correct comments in BarmanCredentials struct and update manifests accordingly.
Closes: #515
Signed-off-by: Hai He<EMAIL_ADDRESS>

/test

/ok-to-merge


Update readme.md

adding my lecture series that I think fits quite nicely :)

Awesome. Thank you!


Set Placement mode

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #516 (969ce9e) into master (d9d1754) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #516   +/-   ##
=======================================
  Coverage   19.60%   19.60%
=======================================
  Files          57       57
  Lines        2137     2137
=======================================
  Hits          419      419
  Misses       1672     1672
  Partials       46       46

Impacted Files
Coverage Δ

pkg/ddc/base/runtime.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Error making post

Hello,
Trying to use the post method for a text post.
t = Tumblpy(TUMBLR_CONSUMER_KEY, TUMBLR_CONSUMER_SECRET,
            TUMBLR_ACCESS_KEY, TUMBLR_ACCESS_SECRET)

        blog_url = t.post('user/info')
        blog_url = blog_url['user']['blogs'][0]['url']
        blog_url = blog_url.replace('http://', '')
        blog_url = blog_url.replace('/', '')
        print(blog_url)

        try:
            if image:
                #Create a photo post using a local filepath
                post = t.post('post', blog_url=blog_url, params={'type':'photo', 'caption': text, 'data': image})
            elif url:
                #Create a link post
                post = t.post('post', blog_url=blog_url, params={'type':'link', 'url': url, 'description': text})
            else:
                print('im definitely text')
                #Create a text post
                post = t.post('post', blog_url=blog_url, params={'type':'text', 'body': text})
        except Exception as e:
            import pdb; pdb.set_trace()
            print(e)

        return HttpResponse(status=200)

The text is just a normal string. I'm using Pythin 3.4.
Error is tumblpy.exceptions.TumblpyError: There was an error making your request. and error_code is 404
Any ideas why this isn't working? Thanks!

I had this too, for a temporary fix do: pip install python-tumblpy==1.1.0

Hi @amanthei, which version of Tumblpy are you using and what does blog_url print out as?

Hey @michaelhelmick I was using 1.1.3. Just tried @megagorilla's temporary fix of rolling back to 1.1.0 and that worked perfectly. blog_url (before replacing characters in the code above, which I tried in case it was supposed to be without the http://) correctly prints out as my blog url: http://thedirectory.tumblr.com/

Thanks! I don't use the Tumblr api too often anymore, but I'll try and debug this personally this week!

python-tumblpy==1.1.4 is out! Should fix this issue!

Brilliant, thanks!


set_custom_mouse_cursor(null) makes the cursor disappear.

Godot version
Godot Engine v4.0.stable.official.92bee43ad
System information
Windows 10
Issue description
The documentation states that passing null into the image parameter of set_custom_mouse_cursor will change it back to the system cursor, however I find that it removes the cursor's image entirely, rather than setting it to the system's cursor.
Steps to reproduce
var theimage = load("res://final.png")

func _on_my_button_pressed():
    if isimage == false:
        Input.set_custom_mouse_cursor(theimage, Input.CURSOR_ARROW, Vector2(16,16))
        isimage = true

    else:
        Input.set_custom_mouse_cursor(null)
        isimage = false```

When the button is pressed, the cursor is meant to change to the new cursor. When it has been pressed a second time, it changes back to the default cursor. However, upon pressing it a second time, it simply disappears. The cursor becomes visible again when the cursor exits and re-enters the window.

### Minimal reproduction project

[null mouse.zip](https://github.com/godotengine/godot/files/10901969/null.mouse.zip)

I am unable to replicate this on Linux X11 Xfce.

I believe I have the error will test soon

PR up #74511

Until this is fixed a workaround should be to call the function twice (haven't tested but it should fix it based on the code)

Calling the function twice works. I'll close the issue once the pull request has been approved, thank you.


Improve current viewport filtering

We relied on the bounding box provided by DOMSnapshot.captureSnapshot before. However, there are many problems with this method. Now we individually check each element by hooking it to its remote object through DOM.resolveNode and getclientboundingrect

why removing beartype?


add aws iam instance profile outputs

Tests failed due to the AWS Provider 3.x release. Fixes in https://github.com/hashicorp/terraform-aws-nomad/pull/73. We'll need to get that merged first.

https://github.com/hashicorp/terraform-aws-nomad/pull/73 is now merged. Could you rebase and I can kick off tests again?

@brikis98 rebased

Great, thanks. Kicking off tests now.

Tests passed. Merging now.


Add metadata for docs.rs

https://docs.rs/crate/pigg/latest/features
says
"There is very little structured metadata to build this page from currently. You should check the main library docs, readme, or Cargo.toml in case the author documented the features in them."
see if we can add meta-data about features to make this better, especially as we are about to add too more.

I see this in some other repos:
[package.metadata.embassy_docs]
src_base = "https://github.com/embassy-rs/embassy/blob/embassy-time-v$VERSION/embassy-time/src/"
src_base_git = "https://github.com/embassy-rs/embassy/blob/$COMMIT/embassy-time/src/"
features = ["defmt", "std"]
target = "x86_64-unknown-linux-gnu"

[package.metadata.docs.rs]
features = ["defmt", "std"]


Streaming issue

Whenever I turn ON or OFF the feature flag, the streaming status link is getting disconnected between GB service and GB proxy service if the SDK is chosen as JAVA

Both services are deployed in K8s env and min 2 pods are running for both.

Does streaming work as far as you can tell? I.e, is this a reporting/UI bug, or is there a true connectivity problem?

I have the same issue, but the streaming seems to be working fine.


Snapshot testing

I would like to talk about the benefits snapshot testing (with Jest) and when exactly to use it.

Useful presentation https://www.youtube.com/watch?v=HAuXJVI_bUs


[Log Analytics Solution] GA Log Analytics Solution

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[x] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

log analytics solution

Does your PR contains this issue https://github.com/Azure/azure-cli-extensions/issues/3316?

Does your PR contain this issue #3316?

Temporarily not. If there is any content that needs to be modified in CLI for this issue, I will add them into this PR.

Considering further - it is likely this RP will start deprecation process in next 6 months, the replacement functionality might GA in this time.  So it might be best to leave this as experimental.

OK, we'll keep it as experimental, so I close it first.
If we need GA this PR in the future, I will reopen it~


Updated USStreetNameExpander-PrefixDirectional.validator.mapcss

Updated USStreetNameExpander-PrefixDirectional.validator.mapcss to more properly handle addresses that have been already run through the ALLCAPS-to-TitleCase.validator.
Very likely an error or typo somewhere, so it should get a good look over.
Only my second PR ever, and I still don't really know how Github works so I may very well be doing all this wrong :-).

Love this. Getting the rules to work more cohesively is great. I left a comment about an alternative approach. I don't mind too much one way or the other. The changes look good overall.

I'm afraid I can't find your comment - where did you add it?

No worries! Line 91 on this view: https://github.com/watmildon/josm-validator-rules/pull/16/commits/2e4845fd7e8fd252ec7e02283e7dbe640fa274ad

I still don't see it, unfortunately. It says "0 comments on commit 2e4845f"...

Curious. Anyhow, here's my comment about adding to the existing regex instead of making new sets of checks: "A more compact way to do this is the change the regex above from "^NE " to "^N[eE] " and add the additional assertMatch there."

Gonna merge this so I don't forget. We can always keep fiddling.


Change type to Throwable

https://github.com/temporalio/java-sdk/issues/187

 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.


Fix confusing grammar in Assignment Overload spec

A previous commit (823d8a4) clarified that destructors are called upon assignment to an initialized object. In doing so it spliced the sentence in a very unnatural way. I've attempted to correct the sentence grammar while preserving the semantics.
It's not clear to me if calling the dtor on assignment to initialized objects is necessary, or if it's just an optional thing the default opAssign is meant to do. I'd appreciate clarification on this.

cc @RazvanN7

From my point of view, the sentence added by [1] should be deleted. If a struct defines a destructor it is used internally by the generated opAssign to destroy a temporary; for more information you can check [2] and the associated dmd code for buildOpAssign which is in src/dmd/clone.d in the dmd repo.
I don't think that we should mix compiler internals with public spec. [1] should not have been merged.
[1] https://github.com/dlang/dlang.org/commit/823d8a46d73b35f6f5836a636ca4f2169a0be99d
[2] https://dlang.org/phobos/dmd_clone.html#.buildOpAssign

BTW @RazvanN7, compared to C++, D's is behavior is a bit strange (even after reading  buildOpAssign's docs): https://gist.github.com/ZombineDev/5132213eb6950bd9853fc903394aa4c5

I've replied on the example in your repo.

Searching for trailing whitespace
./spec/struct.dd:1593:        assignment is defined as copying the contents of a source


chore: change import path to v4

This PR changes our import path to include v4 to correctly align ourselves with Go's naming convention.
It requires the dependency work done in #259, and should be merged after v4.0.0 is released!

@Mergifyio backport release/v4.1.x


Some colors are not rendered properly after upgrading to 3.0

Here's an example of my PHP code snippet using the Catppuccin v2.9

But after upgrading it to 3.0, some of the colors are missing

I'm wondering whether this is an issue or this is just how the color palette works, but I do prefer v2.

Thank you! It definitely looks better.

@nekowinston since golang theme also became a total shit after 3.0.0 — how can i contribute to theme definition? It looks like the syntax highlight broke after migration, or idk... Can i help somehow to update it for previous flavor?

@quenbyako have you tried with
{
  "gopls": {
    "ui.semanticTokens": true
  },
}

from the official Go extension?
I just gave go another look and pushed a few changes (https://github.com/catppuccin/vscode/commit/abf9b40b56e246cb1d8958fbd566d6ed5e839d7b).

@nekowinston yeaaaaaah, with this setting palette became less shitty, thanks 😆  Thank you, at least now i can code without blood in the eyes :)
What do you think about adding this info somewhere... somewhere? In readme, or in faq.md or sorta, cause i asked all my team members in my company and no one knew that you need to path settings a bit for gopls :(

Ah, sorry, cancel that, in go extension readme there is already a warning, that you need to do it manually, cancel my question


The first form lists the names of all xattrs on the given file(s).
The second form (-p) prints the value of the xattr attr_name.
The third form (-w) sets the value of the xattr attr_name to attr_value.
The fourth form (-d) deletes the xattr attr_name.
The fifth form (-c) deletes (clears) all xattrs.
options:
-h: print this help
-s: act on symbolic links themselves rather than their targets
-l: print long format (attr_name: attr_value)
-z: compress or decompress (if compressed) attribute value in zip format
Saving session...
...copying shared history...
...saving history...truncating history files...
...completed.
[Process completed]
`

I have GPT4ALL installed too. Could that be conflicting?

Also unable to install (M1, Ventura 13.6) - it seems like the xattr command doesn't have the -r or -d flags, here's the output from the help.
usage: xattr [-slz] file [file ...]
       xattr -p [-slz] attr_name file [file ...]
       xattr -w [-sz] attr_name attr_value file [file ...]
       xattr -d [-s] attr_name file [file ...]
       xattr -c [-s] file [file ...]

The first form lists the names of all xattrs on the given file(s).
The second form (-p) prints the value of the xattr attr_name.
The third form (-w) sets the value of the xattr attr_name to attr_value.
The fourth form (-d) deletes the xattr attr_name.
The fifth form (-c) deletes (clears) all xattrs.

options:
  -h: print this help
  -s: act on symbolic links themselves rather than their targets
  -l: print long format (attr_name: attr_value)
  -z: compress or decompress (if compressed) attribute value in zip format

I was able to run the script just fine, but get an infinite

whenever I try to install anything.

Just saw the post on discord will post there.

just run sudo xattr -d com.apple.quarantine /Applications/Pinokio.app in terminal instead of the patch command


Center Snap with support to snap to edge items

GravirySnapHelper is awesome comparing to default LinearSnapHelper. But It would be perfect if it is support also Gravity.CENTER and simultaneously snap to first and last item if scrolled to one of the edges.
Regretfully LinearSnapHelper (though snap to center) can't snap to side items as your GravirySnapHelper does. So in order your GravirySnapHelper become far more superior you should add Gravity.CENTER. Thanks. Also I found that if I set snapToLastItem(true) it does not snap, though it snap to edge items by default.

I don't understand why Gravity.CENTER support should be added. Care to explain an use case for this?

actually Gravity.CENTER with ability to snap for first and last item. LinearSnapHelper does not support snap for side items.

in LinearSnapHelper you dont have the snapListener, so +1 for Gravity.CENTER :)

@XtreeIvI, it doesn't make sense (in my opinion) to support CENTER and then snapping first and last items at the same time. That means it'll only be centered before the first scroll. After the first scroll, it'll either be snapped at START or END.
I'll consider adding CENTER support to replace the need for using 2 different SnapHelpers.

actually Gravity.CENTER with ability to snap for first and last item. LinearSnapHelper does not support snap for side items.

1
It is important when you have the first and last items with margins to screen edges

@XtreeIvI this will be added in 2.2.0

Released in 2.2.0. Just pass Gravity.CENTER and you get this behavior by default


Modularizing Counterblockd to use a plugin framework

I use Counterblockd as for a certain API methods, and because it's convenient to have only one daemon querying counterpartyd for new messsages every 3 seconds. On the other hand I don't use any of the database tables except for my own. I'm sure there are also cases where someone may want betting.py but not enhanced_asset info and trade updates or vice versa. If it's not overly ambitious I think it would be a great improvement.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/2823960-modularizing-counterblockd-to-use-a-plugin-framework?utm_campaign=plugin&utm_content=tracker%2F686853&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F686853&utm_medium=issues&utm_source=github).

This was done in counterblock 1.2.x


Appropriate package guidelines, for package requests

Context

I created a packaging request (#309024) which was closed.
The considerations for closing it seem fair, but I was not aware of those considerations before creating the issue.
I haven't been able to find guidelines, either linked from the README or CONTRIBUTING, for which types of projects are good fits to be packaged.
As a new NixOS user, I am relying on Nixpkgs while I learn how to make derivations.
I wasted maintainer time, as well as my own, in creating that issue.

Expectation

I would expect to find a list of rough criteria that package requesters can check, to ensure their package is right for Nixpkgs. e.g. that it's actively maintained and using modern libraries.

Reality

If that resource exists, it was not obvious on the README or CONTRIBUTING guides.

Relevance

I would not have needed to waste maintainer time closing my request.
Would allow new NixOS users like myself to feel more comfortable and welcomed: giving confidence about whether their suggested packages are good fits for the repository in advance.

Should be trivial to add a checklist to the Packaging Request template. Other templates like Missing Documentation already have one.

that it's actively maintained and using modern libraries.

I'll point out that this is not (AFAIK) a prerequisite for nixpkgs.
However, electron is a special case in that it basically ships a browser (chromium).
Old versions of electron use EOL versions of chromium that are known insecure.
We should not add known insecure packages into nixpkgs.
The only versions we have in the repo are electron 27 to 30, everything older has been dropped:
https://github.com/NixOS/nixpkgs/blob/c669412a552f31c45adad47894e7fd6a8698e53f/pkgs/top-level/aliases.nix#L308-L314


vbump data workspace

bumping the version after the August 2022 release

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
27926

Relevant Lines:
62430

💛  - Coveralls


AutoLogin Needs fixing

Hi,
Just wanted to share my expierence with AutoLogin.
So I was implementing the package as specified in the document where I encountered an issue with the autoLogin Param of this package.
What happens is that when we open our home page (in my case localhost:3000). Autologin which is set to TRUE redirects user to the required authentication point. But if the user after being redirecting, doesnot log in and tries to open home page again(localhost:3000), autoLogin fails to redirect him this time. The interesting thing is Now If the user refreshes his page (meaning that, he is loading the page for the third time), autoLogin will work and user will be redirected to the authentication point.
Summarizing, On Odd number of Page load count, autologin works. On even numbers, it doesnt.
Steps to replicate:
I replicated this using both of the examples given by you guyz on github but I changed the autoLogin to TRUE.

Load localhost:3000 -> You will be redirected to authenticator. Now dont log in there. Just go back to localhost: 3000 without logging in. This time you wont be ridrected.

Hi there,
This is working as intended.
If the login process in aborted in some way, this library will have a "bad state". This is returned in the error object from the AuthContext, and says just what you described; "Bad authorization state. Refreshing the page might solve the issue."
It's done in this way to avoid any infinite login loops. If that happens, the AuthProvider is likely to block either the user or the entire app (client_id) for a while. And also your app would be unusable.
If you have any suggestions on how to handle this in a more robust manner, I'd be happy for any ideas :slightly_smiling_face:

@ghauri899 Do you have any further suggestions, feel free to comment here.
For now, I'm closing the issue.


De-incubate getThreads for Pmd task and extension

De-incubates an input property for controlling the number of threads for the Pmd task and PmdExtension.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Replace static config variable injection with a deeper integration with Zookeeper

At the moment the master URL for mesos is injected as an environment variable at runtime to the node process. This could be problematic for example if the master changes during flight and we cannot retrieve the API data correctly.
We could integrate deeper with zookeeper perhaps using something like https://github.com/yfinkelstein/node-zookeeper and retrieve the leading master URL via that.

+1

+1. If you're running a HA Mesos cluster with multiple masters and an election happens then the Mesos-UI data gets stale and you need to restart the whole thing.

Closing via https://github.com/Capgemini/mesos-ui/commit/aecc56795b1bdbb13eadd14fe725b0bba6166758


Pin boost 1.64

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge/core could this be merged ?


Fix inaccurate fee estimations

Try out Leather build aab07ae — Extension build, Test report, Storybook, Chromatic

This PR fixes the issues relating to fee estimations describe in Notion.
The previously calculation lead to a situation whereby some estimations would be counting one fewer input than actually used, thus vastly decreasing the fee rate.
Open to feedback how we can extend the tests of the coin selection.
@fbwoolf can you please QA this to double check
https://github.com/leather-wallet/extension/assets/1618764/a604acca-8513-47a0-ab32-3662a5392f1d

cc @314159265359879 to help QA here?

@kyranjamie I believe we need to fix calc in in selectTaprootInscriptionTransferCoins as well?

checked btc send with 3ns inputs and 2ns outputs and it worked for me 👍

@kyranjamie I believe we need to fix calc in selectTaprootInscriptionTransferCoins as well?

Eurgh, this needs refactoring. Though I think it doesn't have same problem, as the other with the for loop.


Fix the last commit

Hi there,
Ahem! In the last pull request (Which was merged already) I was trying to enhance the error messages in case of failure due to (require or require_not) options. However, there is a little mistake in getting the failed string (actually it gets the next string in the array, or undefined).
It is fixed in this commit. I double tested it. And I'm sorry for the previous mistake.

I pushed a fix in 3699993f175170baa69c5c6c316c1f4979b8401d (0.5.6)


Fix #177 merge conflicts

I looked at #176 and tried adding NSOpenGLPFAOpenGLProfile, NSOpenGLProfileVersion3_2Core in the relevant spot in src/osx/mod.rs, and it worked - I got a 4.1 core context when I ran the examples. So, I fixed #177's merge conflicts, and made rustc stop complaining about a couple changes it had. (I also accidentally committed some files, but I got rid of them too. I totally know how to use git.)
I wanted to add a test for this but I wasn't exactly sure what it would look like.

:+1:


Allow dev to select which "organization-specific" package to include. Not just docassemble-massaccess

This could work by doing an API call to list the installed packages on the server, but need to really think through how we keep it simple for volunteers in MA and still work for people in other jurisdictions.

Note that I am not aware of a way for the API to see which YAML files are in an installed package. So this might be something we hardcode instead of scanning the server.

Note that I am not aware of a way for the API to see which YAML files are in an installed package. So this might be something we hardcode instead of scanning the server.

It actually is possible to list files inside an installed package. So that seems like a promising approach.
We also might have people who run the Weaver on a server that is different from the one that they have for their organization, but in that case it feels like the host of the Weaver is fairly likely to be willing to install the org-specific package.

#424 lays the groundwork here, but doesn't quite finish it. We still need some admin interview that's directly searching through other packages, so you don't need to manually enter the yml files per packages. Gonna keep this ticket open, and just bump it to next for now.

This is pretty well handled with our current approach of searching through the list of advertised packages on the server.


stop global pollution and clean up methods/exports

and for the love of all that is good in the world please merge #11

I'm not sure that enumerability matters in this case

Hey I think this PR is obsolete as per https://github.com/mikeal/r2/pull/38?


How to suppress entire folder or particular archive file

I am trying to use gradle dependency plugin. I would like to suppress the scanning of either an entire folder or a particular archive file (i.e. EAR or JAR) within a project. Please let me know how to suppress it?

I found it. I have used to --exclude option to ignore specific folder or specific files.


fix calendar showing previous day when using sliceMultiDayEvents

This bug is caused by #3543.
The calculation for midnight adds a day but for endDate we want the day to be subtracted again.

do you have the test case that shows the prior failure

ok our test system allows specific day, not today, so  set it to same day as testcase?

ok our test system allows specific day, not today, so set it to same day as testcase?

by today I meant 24th of September :)

i understand, i have to set the  testcase to always sept 24, 2024

For the "1 day repeat" event it shouldn't return it, but for the "2 days repeat" it should and the day in the past was shown too.

your fix is good.. can u update the changelog


[simple-romp] Webcam export not working

I wanted to test the live webcam export in Blender.
Exactly this way, how it's demonstrated in the README:

My first try was to create the required *.npz from webcam:
romp -t -sc=3 --onnx --show_largest --mode=webcam --calc_smpl --render_mesh --save_video -o "E:\Projekte\romp-test2\test_result.mp4"

After that i wanted to use it accourding to the export guidelines.
But: My first command did not create anything.
My pip freeze output:
black==22.3.0
certifi==2021.10.8
click==8.1.2
colorama==0.4.4
Cython==0.29.30
lap==0.4.0
mypy-extensions==0.4.3
numpy==1.22.3
opencv-python==<IP_ADDRESS>
pathspec==0.9.0
platformdirs==2.5.1
simple-romp==1.0.5
tomli==2.0.1
torch==1.11.0
typing_extensions==4.1.1
wincertstore==0.2

System:

Python: v3.9.12
OS: Windows 10 Enterprise (64-bit, Version 21H2, Build 19044.1706)

@Arthur151 The documentation seems outdated.
According to the export guidelines:
python tools/convert2fbx.py --input=/home/yusun/BEV_results/video_results.npz --output=/home/yusun/BEV_results/dance.fbx --gender=female

(Btw.: how exactly works --subject_id?)
https://github.com/Arthur151/ROMP/blob/1458ba0b05a63d64b7fff7898cd917d9fda378b5/simple_romp/tools/convert2fbx.py#L40
Where is tools/bpy-2.82.1 and post_install? Can't find it.
After looking around a bit, i found this one:
https://github.com/Arthur151/ROMP/blob/1458ba0b05a63d64b7fff7898cd917d9fda378b5/scripts/webcam_blender.sh#L1
Config file is available, but getting this:
(sromp) PS C:\Users\Vivien\Projekte\romp-test3> python -u -m romp.predict.webcam --configs_yml='E:\Projekte\romp-test3\webcam_blender.yml'
C:\tools\miniconda3\envs\sromp\python.exe: Error while finding module specification for 'romp.predict.webcam' (ModuleNotFoundError: No module named 'romp.predict')

I know, thats because romp.predict.webcam is not part of simple-romp.
Btw.: Currently i am, failing to install bpy on windows. Any ideas? 🙂

Hi, the way to install bpy on windows is at https://github.com/TylerGubala/blenderpy/releases
Windows

Download the *.whl file
Install your Windows library dependencies (see [the Blender wiki](https://wiki.blender.org/wiki/Building_Blender/Windows), install all dependencies that have lib in the name for your given platform)
pip install bpy-2.91a0-cp37-cp37m-win_amd64.whl && bpy_post_install

Please note that it only support python 3.7, so maybe conda env for python 3.7 would be great.
About live blender driving, please refer to this repo.
https://github.com/yanch2116/CharacterDriven-BlenderAddon
My colleague is responsible for maintaining this funciton now.
Best regard.

Hi, the way to install bpy on windows is at https://github.com/TylerGubala/blenderpy/releases
[..]
pip install bpy-2.91a0-cp37-cp37m-win_amd64.whl && bpy_post_install

Already tested, but not working because of && (works only on linux)
(base) PS C:\Users\Vivien\Projekte\romp-test3> pip install bpy-2.91a0-cp37-cp37m-win_amd64.whl && bpy_post_install
In Zeile:1 Zeichen:49
+ pip install bpy-2.91a0-cp37-cp37m-win_amd64.whl && bpy_post_install
+                                                 ~~
Das Token "&&" ist in dieser Version kein gültiges Anweisungstrennzeichen.
    + CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : InvalidEndOfLine

About live blender driving, please refer to this repo. https://github.com/yanch2116/CharacterDriven-BlenderAddon My colleague is responsible for maintaining this funciton now. Best regard.

Okay, then i will ask there.

ah, i see, sorry 🙂

Note:
I think the best way might be to use blender directly according to the blender documentation:

-P, --python
Run the given Python script file.

But tested it and failed:
(base) PS C:\Users\Vivien\Projekte\romp-test3> blender -b -P convert2fbx.py -- --input=E:/Projekte/romp-test3/video_results.npz --output=E:/Projekte/romp-test3/results.fbx --gender=female
Blender 2.93.5 (hash a791bdabd0b2 built 2021-10-06 06:44:49)
Read prefs: C:\Users\Vivien\AppData\Roaming\Blender Foundation\Blender\2.93\config\userpref.blend
usage: blender.exe [-h] [--input INPUT_PATH] [--output OUTPUT_PATH] [--fps_source FPS_SOURCE] [--fps_target FPS_TARGET] [--gender GENDER] [--subject_id SUBJECT_ID] [--rotate_y ROTATE_Y]
blender.exe: error: unrecognized arguments: -b -P convert2fbx.py -- --input=E:/Projekte/romp-test3/video_results.npz --output=E:/Projekte/romp-test3/results.fbx --gender=female


"Add Data Class" and "Add Rule" buttons shouldn't appear while logged out (cosmetic issue only- functionality is disabled/errors out)

Description
Two buttons shouldn't appear while logged out (cosmetic issue only- functionality is disabled/errors out)

"Add Data Class"
"Add Rule" x 2

Steps to reproduce
"Add Data Class"
0. Open $BROWSER in Incognito mode

Go to root of website
Click on a Data Asset
Click on "Schema" separator
See button "+ Add Data Class"

"Add Rule pt I"
0. Open $BROWSER in Incognito mode

Go to root of website
Click on a Data Asset
Click on "Rules" separator
See button "+ Add Rule"

"Add Rule pt II"
0. Open $BROWSER in Incognito mode

Go to root of website
Click on a Data Class
Click on "Rules" separator
See button "+ Add Rule"

Expected behavior
Buttons should not appear when logged out.
Screenshots

Environment
Desktop
Please complete the following information:

OS: Windows 10 22H2
Browser: Brave Browser: 1.61.116 Chromium: 120.0.6099.217 (Official Build) (64-bit)
Version: mdm-docker 7.2.0

Thanks!

Thanks @ItCatMod - bug reports are very much appreciated!  In some cases the back-end tells the UI what buttons can be shown to happen so it may be an error there - we'll take a look and sort it out!

Preliminary work on: https://github.com/ItCatMod/mdm-ui/tree/hide_buttons_when_logged_out . I'm pretty sure I forgot one of the buttons.
PS: based on 'main' branch.