How long to train the model

Nice work! How long and how much GPU memory is required to train this model on WebVid-10M?

Hi @ailearnerkyh,
With 8 A100 GPUs, you can have a good checkpoint at about 72k iterations in about two days.
Training longer may lead to better results.


Support for params for .bicepparams file only?

I am developing my first Bicep deployment template, and loving it so far. I have a question, though... There doesn't appear to be support (at least, not yet) in the .bicepparam file for a parameter that is only used to support another parameter in the .bicepparam file, and not in the main.bicep file.
For example:
param owner_name = 'Daffy Duck'
param environment_name = 'test'
param resourceTags = {
Environment: environment_name
'Application Owner': owner_name
'CREATED-BY': owner_name
}
The message I am getting is in the main.bicep file, "Parameter 'owner_name' is declared but never used.'
If I leave it out of the main.bicep file, then I get an error in the .bicepparam file.

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

ID: cdf73225-d2e6-b23e-b355-fbb9d3fcdca3
Version Independent ID: fa832878-9821-f222-5f15-ad8646737a04
Content: Create parameters files for Bicep deployment - Azure Resource Manager
Content Source: articles/azure-resource-manager/bicep/parameter-files.md
Service: azure-resource-manager
Sub-service: bicep
GitHub Login: @mumian
Microsoft Alias: jgao

@jewelerm Thanks for your comment. We will review the issue and get back to you shortly.

@Usman0111 - can you take a look at this issue?

The rule with .bicepparam file is that each file is attached to one .bicep file (so we can have multiple params files for one bicep file). This means that the .bicepparam file is well aware of what is going on in the main .bicep file. However, .bicep doesn't know anything about .bicepparam file so you get the warning about parameter not being used in main file. This would need more design discussion to decide if we can support it.
We do plan to add variable support in .bicepparam soon and since you mentioned removing the values from main file, I am thinking not having owner_name and environment_name  in main.bicep is not necessary for your use case so you might be able to convert those to variables as follows
var owner_name = 'Daffy Duck'
var environment_name = 'test'
param resourceTags = {
Environment: environment_name
'Application Owner': owner_name
'CREATED-BY': owner_name
}

Alternatively, you can look into utilizing environment variables for this as well.


Add information about VS Code as a git GUI to source-control section

We have a section on git tools that is missing one of the most popular, VS Code. Should mention how it's useful for staging changes and reviewing diffs. Add to ./source-control/git-guidance/readme.md#tools

@sapinderpalsingh expressed interest in this issue. He needs to join the Microsoft org on github then I can add him as a contributor, just FYI @rozele @TessFerrandez


[Program]: Write a Swift program to find the trace of a matrix

Description
Write a Swift program to find the trace of a matrix
The trace of a square matrix is the sum of the elements on the main diagonal (from the upper left to the lower right).
Input  :
1 2 3
4 5 6
7 8 9

Output :
15

How to contribute

Save the solution in program/program/find-the-trace-of-a-matrix/program/FindTheTraceOfAMatrix.swift file

!assign


Proofs for spectrum of pure states

Proved theorems spectrum_pure_eq_constant and pure_of_constant_spectrum.
The proofs are a mess, so any feedback is welcome!

Thanks Leo! 😄


Enhancement: jaeger UDP support

I have a spring sleuth app running locally that sends traces to jaeger using the zipkin http collector. However, we are running our apps in kubernetes with jaeger-agent sidecars. As far as I can tell, the jaeger agent does not support the http zipkin collector so there seems to be no way to use sleuth to send traces to jaeger agents.
We can use spring-jaeger-cloud-starter to achieve this but then we lose the trace ids in our logs.
I propose a new spring-cloud-sleuth-jaeger adapter that supports the UDP protocol. Unless there is another option I have overlooked.
Thanks.

things like this end up in different repos. Closest to what you are asking for is the X-Ray UDP reporter
https://github.com/openzipkin/zipkin-aws/tree/master/reporter-xray-udp
There are at least 6 other non-stock reporters in various repos, for different backends or transports.

PS if someone is feeling advanced, they could also implement this as a
Brave FinishedSpanHandler, too.
Just the zipkin.reporter option is a bit more understood, and can be used
in different libraries vs Brave.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Whack the mole skillmap has grey block in the hints

Get Animated tutorial->Step 6->hint

Fixed. Thanks!


Square effect when dragging in Chrome

Hello guys,
We used your example http://react-dnd.github.io/react-dnd/examples-sortable-simple.html to implement our solution for dnd.
We experienced an issue with card, which inside contains another DOM element with rounded angles.

Can you see this square around the circle, it appears when you dragging element in Chrome.
Can you navigate us, how we can solve this issue?
Thanks

Are you using a custom drag layer? https://react-dnd.github.io/react-dnd/docs-drag-layer.html

No, will it help?


add payload functionality for reactions

Description:
Lays groundwork for treating reactions as first class citizens in in the new payload architecture. Also updates interactive mode to properly handle them.
Checklist:

[x] Code Quality
[x] Pep-8
[x] Tests (if applicable)
[x] Success Criteria Met
[x] Inline Documentation
[ ] Wiki Documentation (if applicable)

ugh, bitten by the poetry2setup bug


First etl

Completed a basic version of the tap-wikipedia ETL.
Main features:

Retrieving Wikipedia dumps (https://dumps.wikimedia.org/enwiki/)
Retrieving a list of URLs of Featured Articles in Wikipedia  from https://en.wikipedia.org/wiki/Wikipedia:Featured_articles
Caching dumps downloads
Setting configuration values for tap

This is a good start.


Pattern

Added all pattern

Please write markdown files properly, The content is not visible properly.
https://github.com/aaryahjolia/dsa_competitive-coding/blob/191f8e5a9184c77119a72e76d94b5173971b4421/Pattern/Pattern1-5.md.
Also make a seperate .md file for each pattern.

Please write markdown files properly, The content is not visible properly.
-> Can you give example of how to write down markdown files, The content which is not visible is the shape of the pattern which we will get in output.

Also make a seperate .md file for each pattern.
-> I have 28 patterns, so I have to have 28 files ?

You can include shape inside 3 backtick (`) Like this : Hello inside backticks.
See other markdown files like that of sorting files for reference.
It's advisable to make seperate files as it will look good to practice one question at a time but you can do only 5 files also but with proper marking in .md files.

Changes done , you can check it and merge it

I have contributed under HCSOC22'

plz watch the changes
I have contributed under HSOC22

How to add GFG url ?

Go to gfg ide,Paste your code and click on Run+Generate URL,Paste that url back to your .md file

Changes done you can check and merge

LGTM!


Filter sensitive data on Sentry

Context
When the record is not unique Postgres raises the exception and it goes to Sentry like this:
  PG::UniqueViolation: ERROR:  duplicate key value violates unique constraint "index_provider_users_on_email_address"
DETAIL:  Key<EMAIL_ADDRESS>already exists.

We want to avoid having sensitive data, so this PR changes and filters the PG detail from the exception.
The result
As a test, you can see filtered here: https://dfe-teacher-services.sentry.io/issues/4046612784/?project=1765973&query=is%3Aunresolved&referrer=issue-stream&stream_index=0

This looks OK to me. I did wonder whether we might lose useful diagnostic info by doing this kind of thing. In the linked example I think there is still enough info to go on. There are other cases in which Postgres error messages echo the input data, e.g. missing not null column values. e.g.
insert into candidates(id, email_address) values<PHONE_NUMBER>, 'bob@example.com');
ERROR:  null value in column "created_at" violates not-null constraint
DETAIL:  Failing row contains<PHONE_NUMBER><EMAIL_ADDRESS>null, null, null, null, f, null, f, null, null, null, f, f, f).

Good catch. Yes, I think there is still enough info to go on. But I am doing the filtering only when RecordNotUnique is raised so it might be so few cases.


Your authorization do not allow you to access to this information

Hello,
When I make an API call (user creation, payin whatever) I sometimes get the following error:
{ Message: 'Your authorization do not allow you to access to this information',
Type: 'not_authorized',
Id: '3b54f507-6805-49c4-beef-5e92da838c66#1476384039',
Date:<PHONE_NUMBER>,
errors: null }

Your authorization do not allow you to access to this information { Message: 'Your authorization do not allow you to access to this information',
Type: 'not_authorized',
 Id: '3b54f507-6805-49c4-beef-5e92da838c66#1476384039',
 Date:<PHONE_NUMBER>,
errors: null }

On my local environment I had the error every once in a while but when the app is deployed it's way more frequent. Please note that I am using the sandbox for now.

Hey @yns01, are you using the latest version of the SDK? From v1.1.0, a fix is included for a known possible authorisation problem. To help with debugging, are you able to record when creating a token, when it expires, and then when you get this error, record what token you were using? It would be a huge help for us (since we don't log the authorization header due to its sensitive nature, so debugging this on our side is quite tricky).

Thanks @hobailey your for reply. I've updated my SDK yesterday, and for now it seems OK.
However, if it occurs again, how should I process to debug ?

OK, good news then. For the debug, could you not try what I previously suggested?

Also, just for the record, please provide the version of nodejs that you are currently running the SDK on for debugging purposes. Thanks!

Well tbh, I'm not sure what you mean by

are you able to record when creating a token, when it expires, and then when you get this error, record what token you were using?

I'll reformulate it, but let me know if it still isn't clear:

Generate a token as normal and log its value and when it will expire
In your app, add a catch for the "not_authorized" error and log what token you were using (and the datetime)

Since the problem is resolved, I'll close this issue for now, but don't hesitate to let us know if the problem comes back.


Types fixes

The major change here is that the types dir index file now imports all other types, and correspondingly exports them. Files that use an types/* import, have been changed to importing only the index file. Some types files were actually .d.ts files, but not named correctly. And safe-types file was renamed to safeTypes to be consistent (some other files follow hyphen case...). This also includes a few linting issues that made it passed preexisting commits. Specifically those using non primitive types (ie Symbol vs symbol, Boolean vs boolean). Some files I came across while fixing the import type/* contained some errors before/after.

Real quick question
in some component files you import a type via a relative path ../../types
in other d.ts you import a src/types
just wondering why the difference?

Some were from me specifically writing them, while others were auto generated (ctrl + space) intellisense. I'd prefer just using src/types. But ultimately they didn't make a difference. Once I'm done with all the interfaces (I think there's only two), I can go around and replace globally any ../types imports in favor of src/ . But I figure someone eventually will import a type as ../ so ¯_(ツ)_/¯


Feature request: allow label tags to be more than just a simple string

Describe the problem
I'm looking for help on implementing the user experience for editing edges. Our mockups suggest replacing the tag with a label that includes a context menu, allowing users to interact with the edge context menu to make edits.
Mockup screenshot:

/cc @marshmalien


client generation

Adds generation of client from IDL

[x] client generation from JS parser json output
[x] use rust parser now that it's out
[x] NativeSender
[ ] mockall (will need help with feature flags)
[ ] naming for all traits
[ ] tests

old todo:

[ ] port code macros from old PR to generate client straight from code skipping generator
[ ] ensure all three types clients work (now we have stable async traits now)

[ ] GClientSender
[ ] GTestSender

[ ] ensure all use cases are supported (e.g. rpc -> program, program -> program)
[ ] more tests
[ ] docs on generator binary?

still some todo's


Lutron Caseta cover SerenaHoneycombShades support

Description:
Adds support for the Lutron Caseta SerenaHoneycombShades in the cover component. This expands supported Lutron shades to Serena Honeycomb and Roller Shades.
Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io - **N/A

If the code communicates with devices, web services, or third-party tools:
Applicable

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Not Applicable

[ ] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[ ] New dependencies are only imported inside functions that use them ([example][ex-import]).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

This is my first pull request, so please let me know if I did anything incorrectly!

Accidentally used the master branch. Will resubmit.


Convert example docker command in README to use :latest rather than being pegged to :0.9

What does this PR do?
I had been encountering some issues with running a derivative of falcon over 4 GPU shards, and discovered that the example docker command I had been using from the README pegged the release version to 0.9.
Removing this limitation and switching to latest solved my issue (by the way, remarkable and great work making so many improvements on creating a general purpose environment for serving these models!)
Not sure what the rationale was behind pegging to 0.9 but if there's no remaining rationale I think this could certainly help people gain more value from this repo.
All the best!
bealbrown

Fixes # (issue)
Before submitting

[ x] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

:latest is bad practice:
The reason is that you never know what latest means after a while. And we've had already a few issues where users where using latest and didn't pull so where not actually on latest. Lots of confusing back&forth ensued.
docker will not check latest if you don't ask it too.
You're free to use it still but I would recommend avoiding it (basically everywhere, unless you're just trying out something.).

Ok! Will of course peg to a known working version for production usage, but for getting started demo seemed to me that using latest would be most appropriate.
In my case 0.9 was unable to run a model with some obscure errors, so I almost abandoned using the project, but discovered that the latest version was able to.
At any rate, thanks again for your great work on this project! All the best!

In my case 0.9 was unable to run a model with some obscure errors,

We indeed may want to make sure this is up-to-date.


how to setup isolated caches for multiple projects in the same repo

This is more of a question than a feature request :), asking for some advice.
In AndroidX, we build multiple projects in paralel. As such, their dependencies are different, hence we tried to separate them by giving each of them a different key (to avoid them overriding each-other's cache)
This created another downside where it simply does not upload the cache due to the hit. (see warm up gradle step here: https://github.com/androidx/androidx/runs/3550347353?check_suite_focus=true)
Is it possible to pass multiple list of keys where we can control the fallback mechanism when the first hit does not match.
e.g. we have N projects, for each of them, we want to specify two keys:
projectName-withGradleFilesHash
projectName

https://github.com/androidx/androidx/pull/242/files
We actually don't even want it to fallback to the default dependencies if these two caches fail, as it will result in building on top of another project's dependencies (hence use more space in the 5GB limit).
As far as I can read this code, it currently uses 2 keys:

a key with given parameters
fallback to dependencies
https://github.com/gradle/gradle-build-action/blob/b3afdc78a7849557ab26e243ccf07548086da025/src/cache-dependencies.ts#L39

It would be great if there was a way to pass the list of keys (and we can enable exact match for these keys to avoid the default dependency)
We are also open to suggestions if there is a better way to do this.
Thanks!

Thanks for the report @yigit, this is timely.
I'm currently working on a gradle-build-action@v2. In it's present state, the cache key generation/matching is similar to what you're asking for (although there is not yet any user-configurable options). You can see the logic here: https://github.com/gradle/gradle-build-action/blob/releases/v2/src/cache-utils.ts#L29-L51.
At first I was trying to hash the project-name and the arguments list, but this was problematic since the cache is restored/saved only at the start/end of the entire GitHub Workflow job, which can involve multiple Gradle invocations with multiple projects and different tasks requested. So instead, the cache key is made up of:

The job id/name
The hashed matrix for the job
The current git SHA for the entire repo

This has the benefit of tying the cache output with the actual job invocation, and provides the maximum caching benefit. There are downsides that I'm currently working to mitigate:

Tying the cache key to the repository commit ID means that even if the cached directories are unchanged after build invocation, a new cache entry will be saved with a new cache key.
Having different cache entries for different job invocations (eg JDK 8 vs JDK 11) can result in a lot of duplication of values within the cache.

My goal is to:

Provide the maximum benefit for small/medium projects with absolutely no additional configuration required.
Ensure that larger projects can tweak things to maximise their cache usage without suffering from endless cache eviction.

I'll add androidx to the set of projects that I've testing with, so I can monitor how I'm going on the second point.

PS I don't plan on addressing this in v1 of the build action, but would be open to a PR if you are particularly keen.

Oh interesting.
the multiple gradle invocations was the exact reason why we've added that warm up task. Each sub-project runs with the same arguments first and then runs with its own build actions (so the cache key for the actual build never worked as all parallel tasks used the key from the first invocation that was the same for all)
We are also not interested in things like os, jvm as the main goal here is to cache maven dependencies. (though could matter for other projects)
In terms of new cache entries, as long as there is a good fallback mechanism, i'm not sure how bad it is. Maybe the biggest downside is that every job run will upload the cache regardless, hence doing more upload work when not needed. (though it seems fast enough). I'm also hoping that Github implemented an LRU for that cache so it should act well in practice. But if someone invokes gradle multiple times, they probably want the gradle cache to be shared, otherwise, you'll just end up uploading the same content multiple times as it happens after job completes. Also, LRU may not perform best as people might have different post submit workflows which run way less, hence their cache may not stay recent in an LRU cache.
Btw, I'm not sure what matrix for the job contains. If it only contains the matrix parameters passed into that job (vs all matrix options), that might be the perfect solution for our case.
If that assumption is correct, this would even work out of the box for AndroidX(almost). Regardless though, i think for advanced use cases, letting caller take full control of caching keys seems like a feasible option? I'll see if we can prototype it over v1 and send a PR if it seems to make sense.
Thank you!

A job configured like this:
strategy:
  matrix:
    java: [ 11, 12 ]
    os: [ ubuntu-latest, macos-latest ]

will have a matrix context object like this for the first run:
{
  "java": 11,
  "os": "ubuntu-latest"
}
```.

So combined with the job name / id this provides a good way to uniquely identify a particular job execution over time.

Note that v2 of the action will do caching a bit differently:

The Gradle User Home (or portions thereof) will be cached with a key that is unique for the actual invocation. This key includes the git SHA so we make no assumptions about what particular files or build parameters will impact the Gradle User Home.
Within the Gradle User Home, many common zips and jars will be deduplicated and cached with a fixed id. This works by:

Before caching the Gradle User Home, each of these jars is replaced by a marker file and cached separately
When restoring the Gradle User Home, the jar/zip is restored for any marker file detected

In testing this has a huge impact on reducing the overall size of the cache entry that is specific to a job invocation. I will be interested to see how this works for the AndroidX project.

yea reducing cache size might actually help a lot for us.
I'm happy to collaborate here to make AndroidX a test bed. Our main development happens in AOSP so we have more flexibility in breaking Github (though not desired as we get a steady stream of PRs there).
Or we can even have a running PR that changes it to v2 and work on it (or on a branch).

so i made a quick and dirty prototype here on what i was thinking:
https://github.com/yigit/gradle-build-action/commit/eadf30afbc7bc3e33e53c1bfcf9f83f089d801bb
beware,I've not tried this nor I know TS so this might be stupid :) I basically copied how github's cache action reads restore-keys and copied.
Lmk what you think, if it sounds reasonable, i can test this and try to send a PR. Meanwhile, it seems like there is no v1 branch to merge back into? (I've created this from 1.5.1 tag)
Btw. I'm not sure why the action is failing, i've run npm run-script build and uploaded everything.
https://github.com/yigit/gradle-build-action/runs/3628537290

Closing this issue, as we've moved on a long way from the implementation in v1.


Add SOBR extent details

Description

Up to now there were no detailed information about extents of ScaleOut Backup Repositories (SOBR). Now they are part of "Backup Repository Configuration". There is also a new table-entry: "Extent of ScaleOut Backup Repository", when repository is an extent of a SOBR. Otherwise this line isn't visible.
Details:

shortened (if ($ScaleOuts))-section. So it was easily possible to keep extents-information in array.
disabled new creation of  $BackupRepos, because it already includes all SOBR-extents.
adding new line in repository table
if new line as no entry, remove it afterwards

Related Issue

Motivation and Context

SOBR extents are also normal repositories, so their settings are also interesting.
How Has This Been Tested?

Tested in my demo environment
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.

I have added this section back:
if ($ScaleOuts) {
    foreach ($ScaleOut in $ScaleOuts) {
        $Extents = Get-VBRRepositoryExtent -Repository $ScaleOut
        foreach ($Extent in $Extents) {
            $BackupRepos = $BackupRepos + $Extent.repository
        }
    }
}

Because there is the possibility to have multiple extent per SOBR.
Scale-Out Backup Repository
A scale-out backup repository is a repository system with horizontal scaling support for multi-tier storage of data. The scale-out backup repository consists of one or more backup repositories called performance tier, and can be expanded with object storage repositories for long-term and archive storage: capacity tier and archive tier. All the storage devices and systems inside the scale-out backup repository are joined into a system, with their capacities summarized.
PS C:\Users\jocolon> Get-VBRBackupRepository -ScaleOut | Select-Object Name,Extent,CapacityExtent

Name              Extent                                                         CapacityExtent
----              ------                                                         --------------
SOBR - Minio      {E - Backup Repository}                                        Object storage repository  - Minio
SOBR - AWS S3     {F - Backup Repository, E - Backup Repository - VEEAM-VBR-02V} AWS - S3 - Backup
SOBR - Azure Blob {F - Backup Repository - VEEAM-VBR-02V}                        OS - Azure Blob

PS C:\Users\jocolon>

Thank you for your ongoing collaboration!

Hi!
I know there can be more than one extent in a SOBR. I found an error in my code, but the short version works fine:
PS C:\Users\Administrator> Get-VBRBackupRepository | Where-Object {$_.Type -ne "SanSnapshotOnly"}| select name

Name
----
Default Backup Repository
Local_Repo
OLD
Proxy_Repo
xfs_5T

PS C:\Users\Administrator> Get-VBRBackupRepository -ScaleOut | select name, extent

Name      Extent
----      ------
SOBR_xfs  {ReFS_local, linux_1, xfs_1}
SOBR_blob {xfs_2}

PS C:\Users\Administrator> $BackupRepos = $null; $ScaleOuts = $null; $Extents = $null
[Array]$BackupRepos = Get-VBRBackupRepository | Where-Object {$_.Type -ne "SanSnapshotOnly"}
[Array]$ScaleOuts = Get-VBRBackupRepository -ScaleOut
if ($ScaleOuts) {
    foreach ($ScaleOut in $ScaleOuts) {
        $Extents = Get-VBRRepositoryExtent -Repository $ScaleOut
        foreach ($Extent in $Extents) {
            $BackupRepos = $BackupRepos + $Extent.repository
        }
    }
}
$BackupRepos | select name

Name
----
Default Backup Repository
Local_Repo
OLD
Proxy_Repo
xfs_5T
ReFS_local
linux_1
xfs_1
xfs_2

PS C:\Users\Administrator> $BackupRepos = $null; $ScaleOuts = $null; $Extents = $null
[Array]$BackupRepos = Get-VBRBackupRepository | Where-Object {$_.Type -ne "SanSnapshotOnly"}
[Array]$ScaleOuts = Get-VBRBackupRepository -ScaleOut
if ($ScaleOuts) {
    $Extents = Get-VBRRepositoryExtent -Repository $ScaleOuts
    $BackupRepos = $BackupRepos + $Extents.Repository
}
$BackupRepos | select name

Name
----
Default Backup Repository
Local_Repo
OLD
Proxy_Repo
xfs_5T
ReFS_local
linux_1
xfs_1
xfs_2

But I made a error with brackets. I wrote:
if ($ScaleOuts) {
    {$Extents = Get-VBRRepositoryExtent -Repository $ScaleOuts}
     $BackupRepos = $BackupRepos + $Extents.Repository
}

It should be:
if ($ScaleOuts) {
    $Extents = Get-VBRRepositoryExtent -Repository $ScaleOuts
     $BackupRepos = $BackupRepos + $Extents.Repository
}

Sorry for this mistake!
Would be fine, if the code could stay, because otherwise the display of the SOBR of the extent would not work correctly in the detail section. Should I update the code and pull it again?

Of course, you can create a new PR.
Additionally I have been documenting several missing features in the report in case anyone wants to collaborate more code!
https://github.com/AsBuiltReport/AsBuiltReport.Veeam.VBR/projects/1

There is also a slack channel where you can find Tim and many other ASbuiltReport contributors.
VMware {code} #asbuiltreport
https://vmwarecode.slack.com/archives/CCLMSHL30


Implement indentation handling

looks good, can you add a test?
then I will merge

A couple months later, added tests and made the output a little prettier


ManageIQ.toolbars is null error appears in Firebug

while navigating several pages (login, topology, etc.) , this error appears.

cc @martinpovolny
@miq-bot add_label ui

@skateman : can you deal with the remaining accurences of ManageIQ.toolbars, please?

@abonas can't reproduce on latest master, can you verify please?

@abonas can't reproduce on latest master, can you verify please?

rebased now on latest master - don't see it either.

@abonas please close the issue then

Thanks both!


register 1.3

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/92575
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.3.0 -m "<description of version>" b228adaea331f485948996ca03b5f1b9cc3b3146
git push origin v1.3.0


iOS - Dialog background becomes transparent Due to icon animation

When a KottieAnimation() is used inside a androidx.compose.ui.window.Dialog() in compose the background intead of becoming white it takes the root window background. and the icon background does not work.

@buypoint can you attach a screenshot/video please

my code
Dialog(
        onDismissRequest = {
        },
    ) {
        var loadingjsonString by remember { mutableStateOf("") }
        LaunchedEffect(Unit) {
            withContext(context = Dispatchers.Default) {
                loadingjsonString =
                    Res.readBytes("files/reactive_template_compose_loading1.json").decodeToString()
            }
        }
        if (loadingjsonString.isNotBlank()) {
            val composition = rememberKottieComposition(
                spec = KottieCompositionSpec.JsonString(loadingjsonString)
            )
            val animationState by animateKottieCompositionAsState(
                composition = composition,
                isPlaying = true,
                iterations = Int.MAX_VALUE,
            )
            KottieAnimation(
                modifier = Modifier
                    .size(60.dp)
                    .nothing(),
                composition = composition,
                progress = { animationState.progress },
            )
        }
    }

Same is happening to us

@bevice1 @xiaojinzi123 unfortunately, at the moment transparent background doesn't work for kottie in iOS, check out https://github.com/ismai117/kottie/issues/5


🐛 Make messages compatible with Conventional Commits

Discussed #176 and #87
Current messages is not compatible Conventional Commits. Look at Subject Case in Conventional Commits
I see 3 types of messages:

${ COMMIT_PREFIX } Synced file(s) with ${ GITHUB_REPOSITORY }
${ COMMIT_PREFIX } Synced local '${ file.dest }' with remote '${ file.source }
${ COMMIT_PREFIX } Created local '${ file.dest }' from remote '${ file.source }'

It can be difficult create 1 template for 3 type of messages...
So we can start with just lower case: Synced -> synced, Created -> created

I didn't test it yet because a new bug #289, I will test it later...

Thanks for the PR @phnx47!
If you fix the conflicts I'll be happy to merge it!

I rebased commit and regenerated dist, tested with with wagoid/commitlint-github-action@v5 https://github.com/phnx47/harvest-report/pull/79. All good now

:tada: This PR is included in version 1.20.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Cannot run gtestgengui.py on Python 3

Because of the changes between Python 2 and Python 3, gtestgengui.py cannot be executed on Python 3.x.

This happens because modules Tkinter and tkMessageBox were renamed to tkinter and tkinter.messagebox in Python 3. The solution to this problem is quite simple, just load the correct modules according to the version of the runtime.

Fixed on release 0.2.


PROD: custom eMails use BCC for mailing list

mailing list recipients should nto see each other
so DONT use :cc or :to, rather use :bcc

duplicate to #693


nonstationary for fedformer

Hello,
Thanks a lot for a very interesting work. Will this codebase continue to be updated with the code for FEDformer? Is adding non-starionary in fedformer more different than adding it in autoformer?

`


Removed white gap at the left in the menu when hovering over element

@WebFreak001 what browser are you using? I couldn't see neither this bug nor any change on my device (Chrome, FF)
Thanks for your help :)

Im using chrome. Before it looked like this for me: https://i.imgur.com/n72MADl.png but now it looks like this: https://i.imgur.com/2Jj3IDB.png

Oh I also saw that on your screen the app header wasn't correctly displayed - 743eab1cab38f64556d89962859a71fe56f1b4f2 fixes this :)
Thanks again!


V1.20.5 lightbits

create a makefile to generate new patched e2e.test binary and package it as docker image.
this image will be pushed to our registry so the test can pull it and use the patched binaries.
We should maintain such branch for each supported k8s version.
For more info about how to build and generate please look here
see:

https://github.com/LightBitsLabs/lb-csi/pull/124
https://github.com/LightBitsLabs/systests/pull/6902

@lev-lb @ronen-lb please notice i have another commit here cause we have a new leftover PV from the snapshottable test suite that had the same problem - sc deleted before PV deleted

@yogev-lb : same comments from me as for PR-1, except please change the commit subjects on the last 2 commits to explain the difference between them.


[WIP]Trigger Changes Part 1

NOTE: This PR is still WIP, and is not out for review. Currently we are waiting for this PR to get merged - https://github.com/openshift/pipelines-tutorial/pull/64
The following changes have been made as part of this PR:

Procedure for creating and adding triggers
Procedure for adding webhooks

@nkakkar81 What's the status of this MR?

This is a placeholder PR which was created since the folder structure was not in place openshift-docs repo ( at that time). This PR can be deleted ( without merging ) because:

The docs in this repo are only till 0.10 release which do not support Triggers.
This information is now already present in our main OCP docs - https://docs.openshift.com/container-platform/4.4/pipelines/creating-applications-with-cicd-pipelines.html#about-triggers_creating-applications-with-cicd-pipelines

@Preeticp - FYI, we had discussed this when I was documenting for Triggers ( for TP release)


Item order not respected when using SpanSize width = number of columns

I have the following code to collect my items to show in my RV:
    // Spanned
    public RecyclerView.LayoutManager getLayoutManager() {
        if (!mMainView || mDataHolder.getViewMode(mActivity).equals("grid")) { // We force the grid view in subset
            final Integer numColumns = UserPreferencesHandler.<Integer>readValue(mActivity, UserPreferencesHandler.PREF_KEY_NUM_COLUMNS);

            SpannedGridLayoutManager layoutManager = new SpannedGridLayoutManager(SpannedGridLayoutManager.Orientation.VERTICAL, numColumns);
            layoutManager.setSpanSizeLookup(new SpannedGridLayoutManager.SpanSizeLookup(position -> {
                try {
                    switch (RVAdapterTimeline.this.getItemViewType(position)) {
                        case RVAdapter.VIEW_TYPE_HEADER:
                            try {
                                return new SpanSize(numColumns, 1);
                            } catch (ClassCastException e) {
                                return new SpanSize(1, 1);
                            }
                        default:
                            LineItem lineitem = mItems.get(position);
                            if (lineitem != null && lineitem.mediaItem != null && lineitem.mediaItem.getRating() != null && lineitem.mediaItem.getRating() == 5) {
                                return new SpanSize(2, 2);
                            }

                            return new SpanSize(1, 1);
                    }
                } catch (IndexOutOfBoundsException e) {
                    MyApplication.toastSomething(mActivity, e); // Seen on firebase
                    return new SpanSize(1, 1);
                }
            }));

            return layoutManager;
        } else {
            return new LinearLayoutManager(mActivity, LinearLayoutManager.VERTICAL, false);
        }
    }

There are header items with a full column span:
return new SpanSize(numColumns, 1);
Other items are with span width 2 or 2. The number of columns is 5.
After collecting the items, they are used for the RV adapter. The problem is that the order in the rendered RV is not the same as the items. Result:
Pic
Could you please fix that? I can confirm 100% that the default GridLayoutManager is working with these items.

I'm afraid that's not possible at the moment. The LayoutManager will order items so they don't leave any gaps. That means that if an item with can fit in a previous position, it will be placed there.
That said, the LayoutManager could be modified to support sections and only do this kind of ordering inside those sections. However, tomorrow I'm leaving for a business trip and I won't be back for at least a week, so I don't think I'll have time to look at this until then.

Ok. Thanks for your message. Maybe you could check if the span is equal to the number of columns, in this case, you don't fill the gaps. This would be an easy workaround. I understand the problem.


[issue 3838] [schema] : Allow incompatible schemas to co-exist on a topic

Problem:
Fixes #3838 : The set-schema-autoupdate-strategy policy currently allows disabling schema update (--disabled) and one of --compatibility(FULL, BACKWARD, FORWARD). There is no way to currently allow multiple schemas for a topic.
Solution:
It would be useful to allow --compatibility so the schemas can be added without checking with the previous schemas. This allows for extra flexibility for instance having multiple objects on a topic, supports use cases like a simple event bus etc.
Modifications
This change added tests and can be verified as follows:

Added test cases under SchemaUpdateStrategyTest

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

The public API: (yes)
The schema: (yes)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (yes)
The admin cli options: (yes)
Anything that affects deployment: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (docs / JavaDocs added)

@sijie Hi, here is the patch as promised on slack. Please review this and let me know your comments.

@shiv4289 There's one header missing from the newly added file:
2019-03-16\T\19:37:30.279 [WARNING] Missing header in: /home/jenkins/jenkins-slave/workspace/pulsar_precommit_java8/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/schema/AlwaysSchemaValidator.java

You can use mvn license:format to automatically fix these.

You can use mvn license:format to automatically fix these.

Thats magic, thank you. Done!

run integration tests

run cpp tests

run cpp tests

Merged in 2.3.1 at
5dd2f47


Add Chris API

Exposes a few data points to be used with website building, tournament information or just nerd information. Spiritual successor to Morg HTTP Client with some added functionality.
Shows some information about your stats, quests, inventory, equipment and some miscellaneous events like player weight and combat level

Please contribute to the original plugin instead of submitting a new plugin. That way there is less pluginhub clutter and the current users of the plugin can benefit from potential improvements and bugfixes.
Additionally, a (large) part of your code seems to be copied from Morg Http Client, but I do not see any attribution to the author.
Furthermore, Morg Http Client has been unavailable for 1 day (see the issue you created), and someone submitted a PR to the repo that likely fixes the plugin?

I have completely rewritten the plugin now. Not a single line of code has been taken from any other plugin.
I've added four new endpoints being quests, combat, account info, bank.
I've added multithreading to the endpoints so if you call them fast enough it shouldn't crash the entire plugin.
I've extended the equipment and inventory endpoints with item names and slot name in the equipment handler.
I've extended the events endpoint to show last 5 chat messages and special attack energy and if you're idle or not.
I've extended the skills endpoint to show the amount of boost your skills have right now.
This is not even close to the same plugin as morg anymore and uses no code from that plugin either. If this can't be accepted because it shows some of the data that morg did, then it's weird that other plugins have been accepted which also show similar data

To reiterate what was said earlier,

We'd rather you try and contribute to an existing plugin.

The preference would be for you to contribute to the similar plugin that already exists. If it remains abandoned, transfer of ownership to a new maintainer who would be active would generally be the next step.

Alright since it's obvious that this won't ever be implemented, how long do I need to wait before either me or someone else is given ownership of morg?
Currently his plugin contains some datapoints that no other of the plugins do. All of my website endpoints are currently blank and my grafana dashboards are just showing errors because of morg being down.
I'll reiterate that morg is largely unfinished code with plenty of ideas that were never implemented. With no activity in the last 2 years and having a pull request open for over 8 months. How long will i need to wait before someone can take over the plugin? Or should i just move away from using the morg endpoints completely and hope i can find something else that does the same as morg does?

There is no strict amount of time (often 1 month is used though), but we'd like people to be added as contributor to a repo instead of transferring ownership, so you should:

Create a PR with this in their repo and ask to be added as contributor to the repo.
Contact the author via any way possible:

@ them in the RL discord if you know their discord name and ask to be added as contributor
Check their GH for contact info and use that to ask them if you can be added as contributor
Check their commits for their email so you can ask them via email if you can be added as contributor
Can create an issue as well asking to be added to the repo as contributor

If they haven't responded in 1 month, you can create a new PR here with pictures of them not responding. Also don't forget to @ them in the PR. In that case you'd transfer ownership of the plugin to your own repo, keeping the commit history intact without making any changes. After ownership has been transferred, you can update the plugin to whatever you want.
@MorgApps

Morg updated their app, is that satisfactory?

That's more than satisfactory. I will do pull requests to morg instead and improve that plugin.
Only reason i wrote chrisapi was because i didn't see any activity on morg but now that morg accepts pull requests my plugin is obsolete


Client tokengetter cannot differentiate multiple OAuthRemoteApp?

When the tokengetter is called in get_request_token should it not also pass the name of the OAuthRemoteApp to more easily differentiate between different clients? Another option is to use a specific tokengetter per client, but seems overly verbose.
Looking at the example client, passing the name as a parameter would allow multiple accesstokens to be stored in the session as in session.get('remote_oauth_%s' % name)

@lepture Hi, lepture. Do we have a better solution for this situation nowadays? If not, what's the best way to implement multiple OAuth providers in one application?

@greyli How about:
def create_oauth_getter(key):
    def getter():
        return session.get(key)
    return getter

remote1.tokengetter(create_oauth_getter('remote1')
remote2.tokengetter(create_oauth_getter('remote2')

BTW. I'm creating a new auth library in a sustainable way now. Please watch it here https://github.com/lepture/authlib

@lepture Thanks! I will keep an eye on it.


How to handle endpoints that don’t return a resource object(s) as its primary data

Hi everyone,
I am writing an api that exposes an Elastic Search api and was hoping to format the response using the JSON API Spec.
Currently the api looks like this:
Historical percentiles

Example:
/historical_percentiles?field=someField&start_time=1478810146&end_time=1478810158&bin_interval=second

Returns the percentiles for a field over a range of time.

Parameters:

start_time (required) Unix epoch time
end_time (required) Unix epoch time
field (required)
bin_interval (required) Elastic Search compatible interval string
filters (optional)

{
  "meta": {
    "time_to": "2016-11-05T01:34:31.597836+00:00",
    "time_from": "2016-10-08T01:37:51.597836+00:00",
    "bins": [
      {
        "5.0":<PHONE_NUMBER>497282,
        "25.0":<PHONE_NUMBER>881007,
        "1.0": 327.0,
        "95.0": 5520.63477722484,
        "75.0": 2844.4385814706948,
        "date": "2016-10-08T00:00:00.000Z",
        "99.0": 8380.986835876729,
        "50.0": 1617.47585721036
      },
      {
        "5.0":<PHONE_NUMBER>7286674,
        "25.0":<PHONE_NUMBER>130121,
        "1.0": 328.0,
        "95.0": 5440.772302540405,
        "75.0": 2795.7787375257903,
        "date": "2016-10-09T00:00:00.000Z",
        "99.0": 8345.435844243311,
        "50.0": 1819.0584232227932
      }
    ]
  }
}

Ideally, I would like to wrap the payload in the data property, but JSON API spec requires that data returns a resource or resources, which must have an id and type.
The only performant way (that I can think of) to have an id for this payload would be to base64 encode the endpoint with query params.
/historical_percentiles/base64encodedEndpointWithQueryParams
{
  "data": {
    "id": "base64encodedEndpointWithQueryParams",
    "type": "historical_percentiles",
    "time_to": "2016-11-05T01:34:31.597836+00:00",
    "time_from": "2016-10-08T01:37:51.597836+00:00",
    "bins": [
      {
        "5.0":<PHONE_NUMBER>497282,
        "25.0":<PHONE_NUMBER>881007,
        "1.0": 327.0,
        "95.0": 5520.63477722484,
        "75.0": 2844.4385814706948,
        "date": "2016-10-08T00:00:00.000Z",
        "99.0": 8380.986835876729,
        "50.0": 1617.47585721036
      },
      {
        "5.0":<PHONE_NUMBER>7286674,
        "25.0":<PHONE_NUMBER>130121,
        "1.0": 328.0,
        "95.0": 5440.772302540405,
        "75.0": 2795.7787375257903,
        "date": "2016-10-09T00:00:00.000Z",
        "99.0": 8345.435844243311,
        "50.0": 1819.0584232227932
      }
    ]
  }
}

I don't like this method for a number of reasons:

the id could take up more space than the actual data being returned eg. (user enters many filters and one bin is returned)
the documents are stored for a finite period of time; the user would not be able to retrieve the same resource again after some period of time by using the id and type.
added complexity and redunancy of having to handle query params and base64 encoded id
frequency of use, I doubt the consumers of this api would ever use the id to retrieve data

Because of these reasons, I've put it in meta for now.  My reasoning is the api is returning data about stored data, which technically is meta data of the resource.
Is there a better way to handle this that is JSON API compliant?
Should I handle this use case separately and add it to our version of the JSON API spec?
Thanks for reading,
Andrew Toy

A couple thoughts:
Do you really need to use a json-api endpoint here? As long as this exception is documented, I don't see any reason to try and force this to comply with the spec.
That said, I don't think there's really a problem with using a resource-based approach. For example:
POST /historical_percentiles

{
  "data": {
    "type": "historical percentile",
    "attributes": {
      "field": "someField",
      "time_to": "2016-11-05T01:34:31.597836+00:00",
      "time_from": "2016-10-08T01:37:51.597836+00:00"
    }
  }
}

Response:
HTTP/1.1 200 OK
Content-Type: application/vnd.api+json

{
  "data": {
    "type": "historical percentile",
    "attributes": {
      "field": "someField",
      "time_to": "2016-11-05T01:34:31.597836+00:00",
      "time_from": "2016-10-08T01:37:51.597836+00:00",
      "bins": [
        ...
      ]
    }
  }
}

As far as I can tell:

There doesn't seem to be anything that requires you to return an ID. The only language on resource creation and IDs seems to be about client-generated IDs.
There isn't anything in the spec that requires the object to be persisted.
It's perfectly acceptable to return other response codes. 200 might be appropriate here, since the request was successful, you're returning data, yet nothing is persisted.

@rpkilby The spec states

A resource object MUST contain at least the following top-level members:

id
type

(with the only exception being resource-creation payloads).

Whoops - was just scanning the resource creation section. Regardless, you could always just return a randomly generated uuid.

This endpoint does not need to be a jsonapi endpoint to be honest. You could go with a simple json endpoint.

Cool, thanks the feedback.  I really appreciate it.

I was really hoping that JSON API could serve as the spec for any and all json responses.  If anybody else has any thoughts it would be much appreciated.

Our team has been using the meta property to return aggregated data for a few months now.  We've decided to change direction and add an additional section to our fork of the JSON API Spec.
When we finalize on it, I'll put it here.
Here are some related discussions that also have no resolution:
http://discuss.jsonapi.org/t/extension-for-chart-graph-data/408
http://discuss.jsonapi.org/t/representing-non-resourceful-aggregated-data/431/7
I believe that this is an important and common use case to resolve.  Please let me know if you have any other ideas.

related https://github.com/json-api/json-api/issues/1131

I'm gonna close this; I think the discourse discussions and #1131 cover this well.


Remote end closed connection without response

I got this issue when try to animate own drawing.
Thank u for your helps.

Hi @justinduynguyen.
I'm unsure what's causing this issue, and whether it's something with the Docker container or TorchServe. If you ping the server again after you've received this message, does it still return 'healthy'?
If you ping the

Hi @justinduynguyen.
I'm unsure what's causing this issue, and whether it's something with the Docker container or TorchServe. If you ping the server again after you've received this message, does it still return 'healthy'?

It returns this message:
curl: (52) Empty reply from server

It looks like there's an issue with TorchServe then. Try increasing the RAM available to the docker container to 16 gb and try again.

I was increasing the RAM available to 16GB or 32GB even but still not working.
This log from docker

I'm not sure what's causing the issue. You could try rebooting the docker, rebuilding it, or finding and killing whatever is using port 7070.

I meet the same problem. It works after my increasing RAM available to the docker container to 32 GB (not worked for 16 GB).


feat(consumer): allow for consumers use bound exchanges

using the bound exchange parameter, consumers can specify an additional exchange to declare and bind to

Codecov Report

Merging #38 (b7574e0) into master (7252421) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##            master       #38    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files            5         6     +1
  Lines          407       520   +113
==========================================
+ Hits           407       520   +113

Flag
Coverage Δ

tests
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

pyrmq/consumer.py
100.00% <100.00%> (ø)

pyrmq/publisher.py
100.00% <100.00%> (ø)

pyrmq/tests/conftest.py
100.00% <100.00%> (ø)

pyrmq/tests/test_consumer.py
100.00% <100.00%> (ø)

pyrmq/tests/test_publisher.py
100.00% <100.00%> (ø)

pyrmq/__init__.py
100.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 651bb87...b7574e0. Read the comment docs.


fix: license in package.json to be spdx compatible

The license/licenses key in the package.json must be SPDX compatible.
The String "Apache 2.0" is not a valid SPDX identifier. The correct string is "Apache 2.0".
This can cause errors e.g. in projects that have license checks. For me this caused a nx-workspace update to fail.

Hey I have the same problem with a license-checker, the correct name is Apache-2.0.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1888 ms

Resolved: Bofrost.at performance has improved in 13d922e after 53 minutes.


Video Conference Mute: Hotkey is not captured but send to currently focused application

Microsoft PowerToys version
0.55.1
Running as admin

[ ] Yes

Area(s) with issue?
Video Conference Mute
Steps to reproduce

Define a hotkey for "Microphone mute" that contains the SHIFT key, i.e.: Win-Shift-Q
Open a text editor
Press Win-Shift-Q
=> "microphone mute" is switched -- OK
=> the letter "Q" is typed into the text editor -- not OK
This behaviour is very annoying, especially if you are taking notes during a video conference.

Whereas if the hotkey doesn't use she SHIFT key:

Define a hotkey for "Microphone mute" that doesn't contain the SHIFT key, i.e.: Win-Q
Open a text editor
Press Win-Q
=> "microphone mute" is switched -- OK
=> nothing is typed into the text editor -- OK

✔ Expected Behavior
The hotkeys defined for a video conference mute function are captured by video conference mute and don't bubble up to any other open application.
❌ Actual Behavior
If the hotkey uses the SHIFT key, the pressed letter is send to the currently open app.
Other Software
No response

Thanks for the feedback but VCM is moving to maintenance only in the 0.67 timeframe. Our team will only directly address critical bugs, security and accessibility issues.
We'll accept community PRs for enhancements.


Updates for moving OSS 1.8 content to Enterprise 1.9

[ ] Note to clarify OSS 1.8 is last 1.x OSS release, and reference 2.0
[ ] Move Downloads above Getting Started
[ ] Expand Getting Started content to include how to configure and manage clusters, monitor InfluxDB, log and trace, manage security, and backup/restore
[ ] Move concepts up a level out of reference
[ ] (lower priority) Enterprise features should be rephrased/reworked, for example "Managing users" and "Working with clusters"
[ ] Verify front matter menu names are correct (some weren't showing up in TOC)

Addressing these in https://github.com/influxdata/docs-v2/pull/2421.


Need comment about src

Hi. What does this block do? it finds "functions" and...
What numbers 500,1500 do?
  while True:
    next = view.find("\\bfunction\\b", cur)
    if next is None or next.b > pos or (next.a == -1 and next.b == -1): break
    line = view.line(next.a)
    if line.a < pos - 1500: line = sublime.Region(pos - 1500, line.b)
    indent = count_indentation(view.substr(line))
    if indent < min_indent:
      min_indent = indent
      start = line.a
    cur = line.b
  return sublime.Region(start, min(pos + 500, region.b))

It decides on which range around a changed piece of code to send to the server. It prefers starting at a function boundary, and does not scan back more than 1500 characters.


perkeep: 0.11 -> 0-unstable-2024-04-23

Update to current version. See https://github.com/perkeep/perkeep/compare/0.11..bb15e6eb48bc9d614673f3af9432c70a76707c22 for a detailed diff
@kalbasit as maintainer
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@gador what if  you add yourself as another maintainer? I haven't used perkeep for a long time.

@kalbasit sure!
Btw out of curiosity, have you found an alternative to perkeep?

@kalbasit sure! Btw out of curiosity, have you found an alternative to perkeep?

I use several different things; Mostly Obsidian for notes, links, etc... (I back it up to my NAS and an encrypted copy to GitHub). Pics are on my Synology using their software.


New package: AdversarialPrediction v0.2.0

Registering package: AdversarialPrediction
Repository: https://github.com/rizalzaf/AdversarialPrediction.jl
Created by: @rizalzaf
Version: v0.2.0
Commit: 900edf944d75c377e0d20053b794e52c2fa73b2d
Reviewed by: @rizalzaf
Reference: https://github.com/rizalzaf/AdversarialPrediction.jl/commit/900edf944d75c377e0d20053b794e52c2fa73b2d#commitcomment-38305594

This pull request is withdrawn. I modified the code and submitted another submission: https://github.com/JuliaRegistries/General/pull/12340


Method comments in generated C++ are misplaced

Rust struct method comments are not placed before the corresponding generated C++ class methods:
cxx = "1.0.43"
// lib.rs

#[cxx::bridge]
mod ffi {
    unsafe extern "C++" {
        include!("rust/cxx.h");
    }

    extern "Rust" {
        /// This struct comment is placed correctly.
        type SomeType;

        /// This method comment is misplaced.
        fn misplaced_comment(self: &SomeType);

        /// This free function comment is placed correctly.
        fn correctly_placed_comment();
    }
}

struct SomeType {}

impl SomeType {
    fn misplaced_comment(&self) {}
}

fn correctly_placed_comment() {}

this results in
// lib.rs.h

#pragma once
#include "rust/cxx.h"

struct SomeType;

#ifndef CXXBRIDGE1_STRUCT_SomeType
#define CXXBRIDGE1_STRUCT_SomeType
// This struct comment is placed correctly.
struct SomeType final : public ::rust::Opaque {
  void misplaced_comment() const noexcept;
  ~SomeType() = delete;

private:
  friend ::rust::layout;
  struct layout {
    static ::std::size_t size() noexcept;
    static ::std::size_t align() noexcept;
  };
};
#endif // CXXBRIDGE1_STRUCT_SomeType

// This method comment is misplaced.

// This free function comment is placed correctly.
void correctly_placed_comment() noexcept;

Good catch -- I've published a fix in 1.0.44.

Thanks for the quick fix!


feat(api): subdomain support

Checklist

[ ] I have read the coding guide
[ ] My change requires a documentation update and I have done it
[ ] I have added tests to cover my changes.

Description

Motivation and context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

This change is

pkg/api/router.go line 42 at r4 (raw file):
Previously, acud (acud) wrote…
i think that with @agazso we discussed having the locators in the form of *.bzz OR swarm.localhost/ i.e. to prefix everything with the subdomain of localhost, like ipfs are doing. not sure how to handle this here.

So it seems to be as simple as adding the string.


Does this support sqs broker?

I'm getting false purging messages while using it with sqs, does it not support it?

Hi :)
To quote the README:

This has only been tested with Redis so far.

Could you perhaps give some more details regarding the behavior you're experiencing?


Use reflog to calculate diff and allow 'git://' repo addresses

Some people prefer to use ssh addressing for their repo instead of https addressing. Changing the config restriction from url to str would allow them to do this.
For the reflog change to calculate changed files please refer to this answer from https://stackoverflow.com/questions/964876/head-and-orig-head-in-git

HEAD is (direct or indirect, i.e. symbolic) reference to the current commit. It is a commit that you have checked in the working directory (unless you made some changes, or equivalent), and it is a commit on top of which "git commit" would make a new one. Usually HEAD is symbolic reference to some other named branch; this branch is currently checked out branch, or current branch. HEAD can also point directly to a commit; this state is called "detached HEAD", and can be understood as being on unnamed, anonymous branch.
And @ alone is a shortcut for HEAD, since Git 1.8.5
ORIG_HEAD is previous state of HEAD, set by commands that have possibly dangerous behavior, to be easy to revert them. It is less useful now that Git has reflog: HEAD@{1} is roughly equivalent to ORIG_HEAD (HEAD@{1} is always last value of HEAD, ORIG_HEAD is last value of HEAD before dangerous operation).
For more information read git(1) manpage, Git User's Manual, the Git Community Book and Git Glossary

Hi @jangeador,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Nice work! Cant wait to see this merged. Currently it is impossible to clone a private repo, however there is a workaround
Workaround: setup the git repo in /config (using git init and git remote add origin <url>). Then, do cp /config/.git/HEAD /config/.git/ORIG_HEAD. After this, the plugin works, no matter what url you fill in in the config. This is because the git pull command uses the already configured origin remote.

Thanks for the fixes and for the merge.

"repository": "match((?:git|ssh|https?|git@[-\\w.]+):(\/\/)?(.*?)(\\.git)(\/?|\\#[-\\d\\w._]+?))",
I think we need to improve the regexp here. Because some of us aren't using github, for instance mine is gitlab@git.domain.com:group/repo.git
I'd suggest a regexp which can match any word before @.

you can make a PR, that will be update without create a new version

Awesome, thanks!


Add tests of equations from firms.py, government.py, and aggregates.py

@jdebacker
This is an update of PR #26 that should also fix issue #27. I opened a new PR from a new branch because thre would have been lots of unrelated commits. Sorry for not checking this before submitting the other PR.
The only tests I did not add to the test_aggregates.py, test_government.py, and test_firms.py scripts are the tests that require pulling data from the SAM and comparing it to output. I think those tests could be added to the execute.py script @rkasher wrote, which tests the output from the CGE model.
I kept placeholders for these tests to remind myself these functions need tests written, but I can take them out if you like.
Here is the output from my test sessions:
(cge_env) C:\Users\duncan.hobbs\Github\PSL\CGE\open_cge\tests> pytest test_aggre
gates.py
C:\Users\duncan.hobbs\AppData\Local\Continuum\anaconda3\envs\cge_env\lib\site-pa
ckages\pep8.py:110: FutureWarning: Possible nested set at position 1
  EXTRANEOUS_WHITESPACE_REGEX = re.compile(r'[[({] | []}),;:]')
============================= test session starts =============================
platform win32 -- Python 3.7.3, pytest-4.4.0, py-1.8.0, pluggy-0.9.0
rootdir: C:\Users\duncan.hobbs\Github\PSL\CGE
plugins: xdist-1.26.1, pep8-1.0.6, forked-1.0.2
collected 6 items

test_aggregates.py ......                                                [100%]

========================== 6 passed in 0.85 seconds ===========================

(cge_env) C:\Users\duncan.hobbs\Github\PSL\CGE\open_cge\tests> pytest test_firms
.py
C:\Users\duncan.hobbs\AppData\Local\Continuum\anaconda3\envs\cge_env\lib\site-pa
ckages\pep8.py:110: FutureWarning: Possible nested set at position 1
  EXTRANEOUS_WHITESPACE_REGEX = re.compile(r'[[({] | []}),;:]')
============================= test session starts =============================
platform win32 -- Python 3.7.3, pytest-4.4.0, py-1.8.0, pluggy-0.9.0
rootdir: C:\Users\duncan.hobbs\Github\PSL\CGE
plugins: xdist-1.26.1, pep8-1.0.6, forked-1.0.2
collected 15 items

test_firms.py ...............                                            [100%]

========================== 15 passed in 0.79 seconds ==========================

(cge_env) C:\Users\duncan.hobbs\Github\PSL\CGE\open_cge\tests> pytest test_gover
nment.py
C:\Users\duncan.hobbs\AppData\Local\Continuum\anaconda3\envs\cge_env\lib\site-pa
ckages\pep8.py:110: FutureWarning: Possible nested set at position 1
  EXTRANEOUS_WHITESPACE_REGEX = re.compile(r'[[({] | []}),;:]')
============================= test session starts =============================
platform win32 -- Python 3.7.3, pytest-4.4.0, py-1.8.0, pluggy-0.9.0
rootdir: C:\Users\duncan.hobbs\Github\PSL\CGE
plugins: xdist-1.26.1, pep8-1.0.6, forked-1.0.2
collected 5 items

test_government.py .....                                                 [100%]

========================== 5 passed in 0.78 seconds ===========================

(cge_env) C:\Users\duncan.hobbs\Github\PSL\CGE\open_cge> pytest execute.py
C:\Users\duncan.hobbs\AppData\Local\Continuum\anaconda3\envs\cge_env\lib\site-pa
ckages\pep8.py:110: FutureWarning: Possible nested set at position 1
  EXTRANEOUS_WHITESPACE_REGEX = re.compile(r'[[({] | []}),;:]')
============================= test session starts =============================
platform win32 -- Python 3.7.3, pytest-4.4.0, py-1.8.0, pluggy-0.9.0
rootdir: C:\Users\duncan.hobbs\Github\PSL\CGE
plugins: xdist-1.26.1, pep8-1.0.6, forked-1.0.2
collected 0 items

============================== warnings summary ===============================
C:\Users\duncan.hobbs\AppData\Local\Continuum\anaconda3\envs\cge_env\lib\site-pa
ckages\numpy\lib\type_check.py:546
C:\Users\duncan.hobbs\AppData\Local\Continuum\anaconda3\envs\cge_env\lib\site-pa
ckages\numpy\lib\type_check.py:546
C:\Users\duncan.hobbs\AppData\Local\Continuum\anaconda3\envs\cge_env\lib\site-pa
ckages\numpy\lib\type_check.py:546
  C:\Users\duncan.hobbs\AppData\Local\Continuum\anaconda3\envs\cge_env\lib\site-
packages\numpy\lib\type_check.py:546: DeprecationWarning: np.asscalar(a) is depr
ecated since NumPy v1.16, use a.item() instead
    'a.item() instead', DeprecationWarning, stacklevel=1)

-- Docs: https://docs.pytest.org/en/latest/warnings.html

Let me know if anything is unclear. I should have split this into smaller PRs, so I apologize in advance.

@duncanhobbs This looks good to me - I'll merge.
FYI - not entirely sure of the issue you had with merging changes from the master into the branch you had in the other PR, but if I can be helpful in showing you how to merge the latest changes in and resolve conflicts, please let me know.


Use ints to generete Rows range (to uniformly test against integers)

Codecov Report

Merging #5098 into batch-iterator will increase coverage by <.01%.
The diff coverage is n/a.

@@                 Coverage Diff                  @@
##             batch-iterator    #5098      +/-   ##
====================================================
+ Coverage             82.21%   82.22%   +<.01%
  Complexity            11091    11091
====================================================
  Files                  1343     1343
  Lines                 45504    45504
  Branches               5682     5682
====================================================
+ Hits                  37411    37415       +4
- Misses                 5419     5422       +3
+ Partials               2674     2667       -7

Impacted Files
Coverage Δ
Complexity Δ

...o/crate/analyze/SafeExpressionToStringVisitor.java
83.33% <0%> (-16.67%)
7% <0%> (-1%)

...transport/executionphases/ExecutionPhasesTask.java
82.4% <0%> (-9.26%)
20% <0%> (-3%)

.../java/io/crate/analyze/symbol/ParameterSymbol.java
87.5% <0%> (-6.25%)
6% <0%> (-1%)

...ava/io/crate/jobs/AbstractExecutionSubContext.java
84% <0%> (-4%)
17% <0%> (ø)

...in/java/io/crate/breaker/RamAccountingContext.java
80% <0%> (-3.34%)
19% <0%> (-2%)

...ration/collect/collectors/LuceneBatchIterator.java
86.81% <0%> (-3.3%)
35% <0%> (-1%)

...or/transport/distributed/DistributingConsumer.java
78.21% <0%> (-2.98%)
20% <0%> (ø)

.../src/main/java/io/crate/blob/RemoteDigestBlob.java
78.57% <0%> (-1.79%)
8% <0%> (ø)

...er/src/main/java/io/crate/sql/tree/AstVisitor.java
29.66% <0%> (-1.7%)
35% <0%> (-2%)

...main/java/io/crate/http/netty/HttpBlobHandler.java
70.28% <0%> (-1.61%)
50% <0%> (-1%)

... and 23 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 2966414...ad6faf4. Read the comment docs.


LineEdit, TextEdit nodes problem. Incorrect text editing on android

Godot version:
Godot 3, Godot 3.1.1, Godot 3.2 alpha2
OS/device including version:
Android 7.0
Issue description:
LineEdit, TextEdit nodes bug. Incorrect text editing on android.
If you move the caret to the middle of the text, when you type the keyboard copies a section of the text.
Steps to reproduce:
If you type
aaaaabbbbcccc|
move the caret inbetween the a and b
aaaaa|bbbbcccc
and type d, you get
aaaaabbbbccccd|bbbbcccc
instead of the expected
aaaaad|bbbbcccc
there is also a bug when deleting text.
If you type
aabb|
move the caret inbetween the a and b
aa|bb
and press backspace on virtual keybord, you get
aab|bb
instead of the expected
a|bb
The problem is noticed on an android with Google keyboard. No problem editing text on PC.
also there are no problems with editing numbers
Minimal reproduction project:
just a LineEdit ore TextEdit node compiled for android

Duplicate of #27065 ?

Duplicate of #27065 ?

with some additions

You can always include additional information to the opened issue.

Duplicate of #27065.


feat: create abi files per contract

What I did
Creating multiple abi files for compile

fixes: APE-479
How I did it
Changed ape.managers.project.types.create_manifest to have seperate files.
How to verify it
Just run compile command and /abis folder should have compiled contracts as seperate.
Checklist

[ ] All changes are completed
[ ] New test cases have been added
[ ] Documentation has been updated

As @fubuloubu mentioned, we have discussed doing this in a way such that we are not double-caching here. Instead, the local project manifest would get composed from the contract type artifacts during runtime as needed.
Can you describe your use-case more? Maybe there is a short-term solution we can offer. Else, we can try to work out the design now and help you implement the rest of it.
I still left other review comments anyway!

I think a lot of alternative tools to Ape (some in other languages) cache build files by contract type, instead of by project. So to integrate with them it would be easier if we did the same (which isn't really required for us until publishing time for the local package)

As @fubuloubu mentioned, we have discussed doing this in a way such that we are not double-caching here. Instead, the local project manifest would get composed from the contract type artifacts during runtime as needed.
Can you describe your use-case more? Maybe there is a short-term solution we can offer. Else, we can try to work out the design now and help you implement the rest of it.
I still left other review comments anyway!

So, we are using brownie and web3 heavily in our code. Normally, brownie is providing contract based compile files when you simply run brownie compile and it is in the form that you can easily crease web3.Contract instances etc so list of dicts directly. We would like to use this in ape also, those compiled files also used in many other areas like in frontend and other parts of the project.

So, we are using brownie and web3 heavily in our code.

What Brownie features is Ape missing that we can include in our roadmap for Brownie equivalency (besides coverage)

So, we are using brownie and web3 heavily in our code.

What Brownie features is Ape missing that we can include in our roadmap for Brownie equivalency (besides coverage)

In tests, we cant use easy usage of 'EventName' in tx.events kind of syntax with ape it is kind of
assert next(contract.Event.from_receipt(tx), None) is not None

Gas calculation problem also, all tests with eth.get_balance failing now.
Transactions has no timestamp, and i need to use below to create web3py Contract Instance with address and abi
[abi.dict() for abi in contract.contract_type.abi]

and for tx timestamp max i can use is
ape.chain.blocks[tx.block_number]

In tests, we cant use easy usage of 'EventName' in tx.events kind of syntax with ape it is kind of
assert next(contract.Event.from_receipt(tx), None) is not None

I am in favor of refactoring from_receipt to return a list here, since it should be a defined list (less than 100s of events) and it doesn't need additional data to fetch (the receipt contains all the raw event data)
Maybe can add this to 0.6.0 breaking changes?

Gas calculation problem also, all tests with eth.get_balance failing now.

Is there an issue open for this? We should solve any issues with this

Transactions has no timestamp, and i need to use below to create web3py Contract Instance with address and abi
[abi.dict() for abi in contract.contract_type.abi]

You want to be able to pull out a Web3py contract instance from an Ape contract?

and for tx timestamp max i can use is
ape.chain.blocks[tx.block_number]

Yeah, I've wanted to add ReceiptAPI.timestamp that's a computed property to fetch the timestamp from the corresponding block, I don't think we have tracking for this, but this is basically what you need?

In tests, we cant use easy usage of 'EventName' in tx.events kind of syntax with ape it is kind of
assert next(contract.Event.from_receipt(tx), None) is not None

I am in favor of refactoring from_receipt to return a list here, since it should be a defined list (less than 100s of events) and it doesn't need additional data to fetch (the receipt contains all the raw event data)
Maybe can add this to 0.6.0 breaking changes?

Yes, this is a grand idea. It should have been a list all along on the Receipt.
Everything else too, but right now focused on this first bit as we are moving towards 0.6.

In tests, we cant use easy usage of 'EventName' in tx.events kind of syntax with ape it is kind of
assert next(contract.Event.from_receipt(tx), None) is not None

I am in favor of refactoring from_receipt to return a list here, since it should be a defined list (less than 100s of events) and it doesn't need additional data to fetch (the receipt contains all the raw event data)
Maybe can add this to 0.6.0 breaking changes?

Gas calculation problem also, all tests with eth.get_balance failing now.

Is there an issue open for this? We should solve any issues with this

Transactions has no timestamp, and i need to use below to create web3py Contract Instance with address and abi
[abi.dict() for abi in contract.contract_type.abi]

You want to be able to pull out a Web3py contract instance from an Ape contract?

and for tx timestamp max i can use is
ape.chain.blocks[tx.block_number]

Yeah, I've wanted to add ReceiptAPI.timestamp that's a computed property to fetch the timestamp from the corresponding block, I don't think we have tracking for this, but this is basically what you need?

There is no issue regarding gas calculation or so only what i wrote to discord. And web3py contract part seems good to me.

In tests, we cant use easy usage of 'EventName' in tx.events kind of syntax with ape it is kind of
assert next(contract.Event.from_receipt(tx), None) is not None

I am in favor of refactoring from_receipt to return a list here, since it should be a defined list (less than 100s of events) and it doesn't need additional data to fetch (the receipt contains all the raw event data)
Maybe can add this to 0.6.0 breaking changes?

Yes, this is a grand idea. It should have been a list all along on the Receipt. Everything else too, but right now focused on this first bit as we are moving towards 0.6.

Yeah, a lot to do for 0.6.0 but this refactor to a list can be something we can do for 0.6.0 pretty easily, turns this:
assert next(contract.Event.from_receipt(tx), None) is not None

into:
assert len(contract.Event.from_receipt(tx)) == 0

which is definitely more intuitive, and can also maybe do stuff like this (eventually):
assert contract.Event.from_receipt(tx) == [contract.Event(1, 2, 3)]

https://github.com/ApeWorX/ape/pull/1244

@fubuloubu @unparalleled-js are we merging this? there is no comment left so this is ready imo.

@fubuloubu @unparalleled-js are we merging this? there is no comment left so this is ready imo.

I think this is something we should work on post-0.6.0 release

Yeah, I've wanted to add ReceiptAPI.timestamp that's a computed property to fetch the timestamp from the corresponding block, I don't think we have tracking for this, but this is basically what you need?

Yes, that would be a nice improvement in API ergonomics.

Tracking this here: https://github.com/ApeWorX/ape/issues/1257

@fubuloubu @unparalleled-js are we merging this? there is no comment left so this is ready imo.

Yes, some refactoring is needed! And I think there are some unknowns yet as well, regarding what exactly we want. We won't forget it, and I am thinking the design now.

@fubuloubu @unparalleled-js are we merging this? there is no comment left so this is ready imo.

Yes, some refactoring is needed! And I think there are some unknowns yet as well, regarding what exactly we want. We won't forget it, and I am thinking the design now.

Whenever it is ready i can also work on it as we need this in shorter time span.

Having discussed some with @fubuloubu , @bilbeyt this is the spec we would like to go with:

Create each ContractType in the project's .build/ folder like: .build/{ContractType.name}.json.
Don't include the contact types in the local .build/manifest.json - that will be just for the sources.
Step 2 will require altering how __getattr__ works in ProjectManager so that project.MyContract still works, which may be the challenging part here.

[ ] Approved but let me finish testing with ape-cairo to make sure it is not broken