[BUG] Extensions on nested types do not work

Hi!
There seems to be a bug, as the following snippet doesn't compile:
@CowBox struct Person {    // compilation error(s): Circular reference; Circular reference resolving attached macro 'CowBox'.
    @CowBoxNonMutating let name: String
    @CowBoxNonMutating let pet: Pet

    @CowBox struct Pet {
        @CowBoxNonMutating let name: String
    }
}

extension Person.Pet {    // compilation error(s): 'Pet' is not a member type of struct 'CowBoxClient.Person'.
    func getName() -> String {
        name
    }
}

On the other hand this does:
@CowBox struct Person {
    @CowBoxNonMutating let name: String
    @CowBoxNonMutating let pet: Pet

    @CowBox struct Pet {
        @CowBoxNonMutating let name: String

        func getName() -> String {
            name
        }
    }
}

Looks like an error in swift - https://github.com/apple/swift/issues/66450

Ahh… yes! I encountered the 66450 bug before when using Observable macro… but it looks like Apple did ship a fix on the 6.0 toolchain. I tried building the first snippet (with an extension on the nested type) and confirmed the error on 5.10 and confirmed the fix on 6.0. Did switching to the 6.0 toolchain fix your builds?
For now… it looks like you identified a legit 5.10 workaround: remove the extension and define the necessary functions in the main declaration of the nested type.
We could leave this task open (for now) for others to reference. We can close the task once 6.0 release goes live to production and we confirm the fix. How does that sound?
Thanks for filing a task!

Did switching to the 6.0 toolchain fix your builds?

Yes, correct, it works with the following:
$ TOOLCHAINS=`plutil -extract CFBundleIdentifier raw /Library/Developer/Toolchains/swift-DEVELOPMENT-SNAPSHOT-2024-05-01-a.xctoolchain/Info.plist` swift -version
Apple Swift version 6.0-dev (LLVM b66077aefd3be08, Swift 84d36181a762913)
Target: arm64-apple-macosx14.0

But doesn't work with:
$ TOOLCHAINS=`plutil -extract CFBundleIdentifier raw /Library/Developer/Toolchains/swift-5.10-DEVELOPMENT-SNAPSHOT-2024-03-02-a.xctoolchain/Info.plist` swift -version
Apple Swift version 5.10-dev (LLVM 5dc9d563e5a6cd2, Swift 9bfe759d7048cb6)
Target: arm64-apple-macosx14.0

Does not repro from Xcode_16_Release_Candidate and Swift 6. I think we are safe to close this for now as a won't fix. Thanks for reporting!


Add BIP70 support

Summary
BIP 70 is a protocol for communication between a merchant and their customer, enabling both a better customer experience and better security against man-in-the-middle attacks on the payment process.
Add support for v2 to Badger.
https://github.com/bitcoin/bips/blob/master/bip-0070.mediawiki

implemented


Azubu.tv max lenght of playback fixed at 90min?

When playing back azubu.tv with the updated egg from the nightly build, the playback stops after about 90 minutes.
Is there a way, or option, to not have to reopen livestreamer every 90 minutes?
Thank you

Probably the same issue as described here: #1185


Kubescape findings by Severity

Description
I am trying to run a kubescape scan but trying to get only the critical and high findings. Is there any option to run all the frameworks and get only those
Environment
OS:   Rhel7 -->
Version:   V2.0.183  -->
Steps To Reproduce

Expected behavior

Actual Behavior

Additional context

@ssnmworld have you looked into the --severity-threshold flag?

Hi, when running the CLI, Kubescape will run all of the controls in the framework.
You can define a custom framework with only high and critical controls, however, this might be tiring.
On the other hand, if you scan and submit the data to the ARMO platform, you can easily perform such filtering:

If you do not have an account, you can run the following:
kubescape scan --create-account

@slashben I think we can close this because we are not planning on adding filters to the Kubescape CLI, what do you think?

Per @slashben , the --severity-threshold= will exit with an exit code <> 0 if there are any control severities at or above the threshold.   However, that still does not compensate for the very surprising lack of a severity in the output json of a scan.
In trying to understand the utter lack of obvious features like the severity, I can only conclude it was intentional.  Especially since the severity DOES appear in the textual output (but only in aggregated form).

Per @slashben , the --severity-threshold= will exit with an exit code <> 0 if there are any control severities at or above the threshold. However, that still does not compensate for the very surprising lack of a severity in the output json of a scan.

The severity can be found in the output json under summaryDetails.controls.[control-id].scoreFactor. I agree that this is not obvious since this field should have been called severity or severityFactor :(

@slashben : Thank you very much.  I have referenced the online severity of each "severityFactor" and arrived at the following, which satisfies the "Kubescape findings by Severity" goal of this ticket:

Low - 1,2,3
Medium - 4,5,6
High - 7,8
Critical - 9,10

Thanks again, @slashben

@HarrisKirk , you're right
See here the mapping:  https://github.com/kubescape/regolibrary/blob/22de0e1c70e131f1953f67fbfa970444a1e857d0/scripts/upload-readme.py#L193
If I recall we took this from CVSS scoring table


Fix a bit typos

- What I did
Fix a bit typos
- How I did it
- How to verify it
- Description for the changelog
Signed-off-by: yuexiao-wang<EMAIL_ADDRESS>

@AkihiroSuda
Thanks for review.
I agree that "eg" or "EG" is widely accepted,  However,  there are many "e.g." used in document and comment which is popular. So I suggest that use "e.g." instead of "eg"


SemanticDB find polymorphic candidates

object A {
  def foo(x: String): Int = 1     // M1
  def foo(y: Int): Int = 2          // M2
  val a = foo(1) // M2
//              ^
}

I would like to query the apply foo(1) and know the alternatives: M1, M2.
context: https://github.com/scalacenter/scalafix/pull/594#issuecomment-364429865

I'm currently investigating the scalac api to know if it's possible to do this.

This is one possible solution: https://github.com/scalacenter/scalafix/pull/594#issuecomment-364657589


[question] can this lib tell how many percentage is the GPU is running at?

I want to check if the GPU is currently occupied in certain percentage / by certain apps. Is this lib the right tool? Alternative if not? Thank you!

Hi @adamchenwei,
This is not the library for that unfortunately and I don't think this is possible on the web outside of a development environment.


Author - Prevent modification of public post (by others)

As an author, other authors cannot modify my public post

Closed with CMPUT404W23T05/Social-Distribution-Front-End#14


Revocation is key

As stated in the "Azure Active Directory": Refresh tokens [...] can be valid until revoked.
The very question (and requirement) is to be able to revoke the refresh token in such a way that it immediately blocks the access to all services.

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

ID: 1570de51-50b1-d4cc-c1c7-832b3242ccb3
Version Independent ID: f5250f91-0a48-20cf-8e2c-e2dc347edd88
Content: Session timeouts for Office 365
Content Source: Enterprise/session-timeouts.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

@JLBevierre - thanks for the feedback, I'll get this to the Enterprise content team for clarification.


fix: undo operation of eraser tool

The eraser tool does not work well in #20. This pr is to fix it.
As the comment says, if I follow the steps, there will be some weird chars on the canvas, since the version vector is not cleared. Then it is fixed with these changes: history.rs and eraser.rs.

Thank you so much for the fix! I just tested, LGTM!


[Documentation] Relative link to ../../examples doesn't work [404]

Documentation issues
In https://github.com/project-chip/connectedhomeip/pull/35846#discussion_r1781552590 I attempted to fix a broken link to the examples directory.
When you access that link from https://project-chip.github.io/connectedhomeip-doc/getting_started/first_example.html#an-sdk-example it results in a 404 because the master branch is not in the URL.
but when you access it from https://github.com/project-chip/connectedhomeip/blob/master/docs/getting_started/first_example.md it works just fine. So there's a discrepancy on how links are compiled on project-chip.github.io vs github.com/project-chip.
The most interesting thing is that the link to ../../examples/chip-tool works fine on both. But ../../examples/ breaks somehow.
I ended up just using the full hardcoded link to get around the issue but I'm afraid this problem may happen in other places and we might end up with other broken links throughout our documentation.

As far as I know, the documentation we upload gets built with sphyx and its config is at https://github.com/project-chip/connectedhomeip/blob/master/docs/conf.py#L76
We build it via make html so it uses https://github.com/project-chip/connectedhomeip/blob/master/docs/Makefile#L19
Workflow at https://github.com/project-chip/connectedhomeip/blob/master/.github/workflows/docbuild.yaml#L40
So we probably should reproduce this locally - can we see the wrong links being built after a make and once we reproduce, can we figure out what settings to fix up?


Fixed #28457 -- Updated the design of the 'Congrats' page for first time Django installs.

Developed by Timothy Allen and Chad Whitman of The Wharton School with shepherding from Aymeric Augustin and Collin Anderson.

Unfortunately, @keyframes and @-webkit-keyframes have to be kept separate for standard versus webkit browsers. In testing, when combined, the styles or animcations disappear for Chrome, Opera and Safari when separate (and don't get me started on MS Edge). I have, however, switched to using a single scale() parameter rather than scaleX() and scaleY() separately. Update coming in a few minutes.


Infra catalog migrations needed

Exactly once after upgrade to 1.2, we need some data migrations to support new catalogs:

Update catalog setting:

Change any entries with a value of https://github.com/rancher/rancher-catalog(.git)? to https://git.rancher.io/rancher-catalog
Change any entries with a value of https://github.com/rancher/community-catalog(.git)? to https://git.rancher.io/community-catalog

Change orchestration stack externalIds:

(system-)?catalog://library:(kubernetes|k8s):${num} to catalog://library:infra*k8s:${num}.

Additionally, if $num  == 7, change it to 8, because reasons.

(system-)?catalog://library:swarm:${num} to catalog://library:infra*swarm:${num}.
(system-)?catalog://library:mesos:${num} to catalog://library:infra*mesos:${num}.

Set Project orchestration fields

@deniseschannon These will not un-migrate on rollback, so the docs may need a warning or manual instructions to undo if the user wants to go back.

route53 also moved from templates to infra-templates, so its externalId needs to be migrated too.  And all the other external-dns templates should also probably be moved to infra for consistency (and then migrated).

@vincent99 Can you tell me exactly what needs to be migrated?  I can add it to the existing list.

templates := [
  "bind9",
  "cloudflare",
  "dnsimple",
  "dnsupdate-rfc2136",
  "pointhq",
  "powerdns-external-dns"
]

for k in range templates {
  replace(
    "(system-)?catalog://library:${k}:${num}",
    "catalog://library:infra*${k}:${num}"
  )
}

On an upgraded setup( from v1.1.4 -> v1.2.0-pre4-rc10) , existing K8s stack is showing as "upto date" and when hovered on shows "current:undefined" . Expect to be presented with option to upgrade to k8s 1.4.x.

On an upgraded setup( from v1.1.4 -> v1.2.0-pre4-rc10) , existing K8s stack is showing as "upto date" and when hovered on shows "current:undefined" . Expect to be presented with option to upgrade to k8s 1.4.x.

@sangeethah this is an issue in the catalog service.  The catalog entry was migrated correctly.

Created #6704 for the catalog service issue.

@sangeethah
Can you test the k8s upgrade migration as well as the DNS ones?
Route 53 and the other external DNS providers in community catalog have been migrated to infra-templates.

Tested with upgrade from 1.1.4 to 1.2.0-pre4-rc14. K8s catalog migration is fixed. "Upgrade available" option is seen to upgrade to K8s1.4.6.
I was able to see "Upgrade available" option for Route 53 DNS catalog item. The Route 53 DNS containers upgraded successfully

did you test any o the other dns providers @soumyalj? they come from a different catalog, so need to be tested separately.

Can you please test one of the external providers since it's coming form a different catalog starting point (community-catalog versus rancher-catalog)?

The specific test should be launch catalogs in 1.1.4, upgrae to 1.2.0/upgrade environment.
Check external ID of stacks are now coming from the new template source, which should be library:infra or community:infra

Using v1.2.0-pre4-rc14 - DNS/External DNS/k8s migrations are good.
Mesos/Swarm migration did not work.
Apparently in 1.1.4 mesos and swarm's externalId was system://mesos and system://swarm, so those don't get handled by migrateLibraryStacks() (Because they were never actually deployed by the catalog..)
What @vincent99 said above.

We won't be migrating the swarm service as the previous stack was using Docker's original swarm implementation and our new swarm service will be using the one packaged in Docker 1.12.

Version - 1.1.4 -> 1.2.0-pre4-rc15
Verified fixed


*: add etcd backup controller

only controller logic cc/ @hongchaodeng

checking crd logic added.

It's better to let @hasbro17 review this once he comes back on Monday.
It is better to make him catch up as well.

comments addressed. PTAL cc/ @hongchaodeng @hasbro17

all fixed. PTAL cc/ @hongchaodeng

lgtm
defer to @hasbro17 for final approval

LGTM

@fanminshi
I just realize that a lot of source code files don't have LICENSE headers.
Can you do a follow-up PR to fix them?

@hongchaodeng good catch.


Execute dbt SQL not working

Expected behavior
When I click on the execute dbt SQL button, the results are displayed
Actual behavior
When I click on the execute dbt SQL button I get the following error:

Database Error
Syntax error: Unexpected keyword LIMIT at [4:3]

Followed by:
{
  "code": -1,
  "message": "Database Error\n  Syntax error: Unexpected keyword LIMIT at [4:3]",
  "data": "\"Error: Database Error\\n  Syntax error: Unexpected keyword LIMIT at [4:3]\\n\\tat DBTCoreProjectIntegration_1.<anonymous> (/home/vscode/.vscode-server/extensions/innoverio.vscode-dbt-power-user-0.39.20/dist/extension.js:18410:127)\\n\\tat Generator.throw (<anonymous>)\\n\\tat rejected (/home/vscode/.vscode-server/extensions/innoverio.vscode-dbt-power-user-0.39.20/dist/extension.js:26056:28)\""
}

and I can see in my BigQuery query history that were 2 queries sent with just:
LIMIT 500

and nothing else in the query
Steps To Reproduce

Open a dbt model
Click on the execute dbt SQL button
See the error

Log output/Screenshots

Operating System
macos version 14.6.1
dbt version
1.8.6
dbt Adapter
dbt-bigquery
dbt Power User version
v0.45.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

@antoine-lefebvre, does this happen for all queries or specific ones?

Same here, it works well but when I add limit keyword it falls to fail. I am using dbt-bigquery as well!

Yeah, that seems a bug in dbt or the adapter
❯ dbt show -m customers
06:12:32  Running with dbt=1.8.0
06:12:33  Registered adapter: bigquery=1.8.0
06:12:33  Found 10 models, 3 seeds, 20 data tests, 2 sources, 478 macros
06:12:33
06:12:34  Concurrency: 4 threads (target='dev')
06:12:34
06:12:35  BigQuery adapter: https://console.cloud.google.com/bigquery?project=diesel-command-384802&j=bq:US:437bfd04-56d9-41dd-99e2-c60614642672&page=queryresults
06:12:35  Encountered an error:
Runtime Error
  Database Error in model customers (models/customers.sql)
    Syntax error: Expected end of input but got keyword LIMIT at [74:3]

@antoine-lefebvre, does this happen for all queries or specific ones?

This happens for all queries as far I as can tell

I also started with this issue out of the blue. No matter how i change the query template in the settings it allways previews SQL server using select * from ({query}) as query limit {limit} which for sql server will never run. in the past we used to change the template for {query} order by 1 OFFSET 0 ROWS FETCH FIRST {limit} ROWS ONLY to run sql server. But i believe this is not required for newer versions.
Also i am running inside a devcontainer.

Hello, I have a bug too. I don't store it on BigQuery; maybe it's an explanation?
limit 500  BigQuery adapter: https://console.cloud.google.com/bigquery?project=toto-extractors-sbx&j=bq:US:tata&page=queryresults BigQuery adapter: Retry attempt 1 of 1 after error: BadRequest('Syntax error: Unexpected keyword LIMIT at [55:3]; reason: invalidQuery, location: query, message: Syntax error: Unexpected keyword LIMIT at [55:3]') BigQuery adapter: https://console.cloud.google.com/bigquery?project=toto-extractors-sbx&j=bq:US:titi9&page=queryresults BigQuery adapter: https://console.cloud.google.com/bigquery?project=toto-extractors-sbx&j=bq:US:titi9&page=queryresults Database Error in test is_zero_open_or_closed_store (tests/is_zero_open_or_closed_store.sql) Syntax error: Unexpected keyword LIMIT at [55:3] Finished running node test.toto.is_zero_open_or_closed_store Marking all children of 'test.toto.is_zero_open_or_closed_store' to be skipped because of status 'error'. Reason: Database Error in test is_zero_open_or_closed_store (tests/is_zero_open_or_closed_store.sql) Syntax error: Unexpected keyword LIMIT at [55:3].


Add ability to pass tokens on connect for ACL control

Adds the ability to pass tokens on authenticate for ACL groups. This is useful for mumble servers that control group and channel permissions with ACL's.

:+1:

merge this pull request

Lies, GitHub.

GitHub such tease. :)


Consider removing the bindgen dependency

#22 and #24 show that clang is required as a build dependency because bindgen uses it to read in ggml.h and produce bindings. We should consider just writing the C bindings ourselves - it shouldn't be too difficult as long as we account for the idiosyncrasies (like #6)

Hmmm, good point. But one thing that concerns me a bit with doing this manually is that if we mistype something in the bindings and call a function, that's UB. Isn't it?

It can be - I hope that doesn't happen, but it's definitely a possibility. We don't depend on too much from ggml as it is now anyway, right?


Allow pyarrow>=5 in to_parquet and read_parquet

This PR changes operations like ddf.to_parquet(..., engine="pyarrow") and  dd.read_parquet(..., engine="pyarrow") to use ArrowDatasetEngine by default (for pyarrow>1).  This also add a ValueError when the user specifically request engine="pyarrow-legacy" for pyarrow>=5.

[x] Closes #7961
[x] Closes #7871

This also closes https://github.com/dask/dask/issues/7871

the last question I had was if datasets which were previously written using the pyarrow-legacy engine could be read in with the new default pyarrow-dataset engine

I suppose this is also tested with the cross-engine compatibility tests? (or otherwise we should add it to the tests)

For reference, the last question I had was if datasets which were previously written using the pyarrow-legacy engine could be read in with the new default pyarrow-dataset engine, and it appears the answer is "yes"

Ah  - this is actually a good point.  We certainly test cross-engine compatibility for pyarrow<4, but not really for pyarrow>=4.  I did add that test to check for a FutureWarning, but not that the result is still correct.

I suppose this is also tested with the cross-engine compatibility tests? (or otherwise we should add it to the tests)

This PR avoids using the "pyarrow-legacy" engine in most of the tests.  Perhaps we should continue testing "pyarrow-legacy" (with pytest.raises) until the ParquetDataset API is officially unavailable?

This PR avoids using the "pyarrow-legacy" engine in most of the tests. Perhaps we should continue testing "pyarrow-legacy" (with pytest.raises) until the ParquetDataset API is officially unavailable?

Yes, I think that would be best to ensure compatibility / consistent behaviour (and for those using the legacy engine, we can add a filterwarnings to ignore the specific warnings, so the tests don't fail?)

@rjzamora do you have the bandwidth to increase test coverage here?

@rjzamora do you have the bandwidth to increase test coverage here?

Hopefully #7988 will cover this?


Bump pnpm version to 9.10.0 because pnpm keeps complaining about it.

Bumps the pnpm version because pnpm complained about it when I installed all the packages lol. Because the project uses corepack, you'll either automatically have it upgraded or will be prompted to upgrade with a single command.

Thank you!


Failure to execute image

Please qavah.me I'm having problem on adding of image,on creating a campaign, it keeps telling me fail to execute  I should get image Data on canvas rendering context 2D the source width is 0 please I need assistance on please because I don't know how to go about it,

Could you please try again from another device?

Qavah.me
https://drive.google.com/file/d/1G0x9GXZeK46gcdTgS5Hfk0jcG9UqZ-hh/view?usp=drivesdk
This is my campaign image, please help execute it so that I can create my
on campaign on qavah
On Tue, Jul 26, 2022, 8:06 PM Théodore @.***> wrote:

Could you please try again from another device?
—
Reply to this email directly, view it on GitHub
https://github.com/Qavah/qavah/issues/1#issuecomment-1195873245, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXV67HA2GXWGFTJRHI3AJSLVWAZM5ANCNFSM54WY5TJA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm sorry, the website doesn't support PDF files yet.
Do you have it in another format, please?

No,I have tried all the formart I have on my Android phone but it keeps
telling me the same thing(fail to execute'get dataimage on canvas
rendering,context2D width will be 0) please help me on the format which
site can support,🙏🙏🙏
On Wed, Jul 27, 2022, 8:39 PM Théodore @.***> wrote:

I'm sorry, the website doesn't support PDF files yet.
Do you have it in another format, please?
—
Reply to this email directly, view it on GitHub
https://github.com/Qavah/qavah/issues/1#issuecomment-1197284938, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXV67HGOGWPYGJM6I6BCUHTVWGGA5ANCNFSM54WY5TJA
.
You are receiving this because you authored the thread.Message ID:
@.***>

What about this one? Also, does it work using your phone's camera directly?


minor fix neth not show on metamask mobile

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Type of change.

[x] FIX - a PR of this type patches a bug.
[ ] FEATURE - a PR of this type introduces a new feature.
[ ] BUILD - a PR of this type introduces build changes.
[ ] CI - a PR of this type introduces CI changes.
[ ] DOCS - a PR of this type introduces DOCS improvement.
[ ] STYLE - a PR of this type introduces style changes.
[ ] REFACTOR - a PR of this type introduces refactoring.
[ ] PERFORMANCE - a PR of this type introduces performance changes.
[ ] TEST - a PR of this type adds more tests.
[ ] CHORE - a PR introduces other changes than the specified above.

Hi @andreapn , thanks for your effort with this PR.
Unfortunately, Neth supports only desktop browser extension, so it's better do stay hidden from mobile UI.
You can access Neth from mobile on this link if you want to confirm this.


Default branch will be renamed main

On March 9th, 2021, we're going to rename the default branch in this repository to main. For more details, see dotnet's earlier announcement.

Closed in #5512


SerializationException when 'alert' state is set to 'null'

Apparently, it seems to be possible for the Alert state to be null (no idea how, might be a glitch). If so, the parsing fails with the following error:

Newtonsoft.Json.JsonSerializationException: 'Cannot convert null value to Q42.HueApi.Alert. Path 'state.alert', line 8, position 17.'

Debug data:
{   "manufacturername": "Philips",   "modelid": "LTW001",   "uniqueid": "00:00:00:00:00:00:3c:af-0b",   "type": "Color temperature light",   "state": {     "bri": 0,     "alert": null,     "reachable": true,     "on": false,     "xy": [       0.157973,       0.053367     ],     "colormode": "ct",     "ct": 344   },   "name": "Light name",   "swversion": "1.46.13_r26312",   "config": {     "direction": "omnidirectional",     "startup": {       "configured": false,       "mode": "safety"     },     "archetype": "classicbulb",     "function": "mixed"   } }

Alert is now nulllable.

Release 3.15.0 is now on NuGet


Create Open Collective Chapter organizer guides

With an uptick of communities across the globe using or evaluating Open Collective to support meetups and events in their region, it would be helpful for users and organizers to have readily accessible guidelines and policies available for them to use.
While regional chapters are currently working on a variation of docs for their own needs, it seems like a uniform set of guidelines and policies would reduce overhead for admins of the Open Collective nonprofits, the global chapter team, and regional organizers.
The following is an initial outline that can evolve should we move forward with this task.

[ ] Open Collective Chapter FAQ

[ ] What is an Open Collective Chapter?
[ ] What's the difference with an Open Collective Host?
[ ] Why join an Open Collective Chapter?
[ ] Why create an Open Collective Chapter?
[ ] What are the roles and responsibilities of Open Collective Chapter organizers?
[ ] What host options do the Chapters have?

[ ] Open Collective Chapter types

[ ] Regional Chapters
[ ] Vertical Chapters

[ ] Building And Growing A Meetup
[ ] Suggestions for Meetup Content
[ ] Group Management Topics
[ ] Event Formats

[ ] Turn Your City into an Open Collective
[ ] Charity Hackathon
[ ] Open Collective meetups
[ ] Vertical Chapter events
[ ] SustainOSS events

[ ] Managing Event Finances

[ ] Event Sponsorship
[ ] Using Open Collective
[ ] Set Event Budget
[ ] Submit Invoices and expenses

[ ] Getting Participants
[ ] Finding Sponsors
[ ] Promoting your event
[ ] Communication guidelines

As mentioned in https://docs.google.com/document/d/1nZR31Xfa9eWz1-2KQ9PGNfxjJdJLm45YoCSnOw13K8I/edit?disco=AAAAB0kt7Sc

Taking a fresh look at the copy on the chapters collective - I think the dual meaning of chapter being both a host and a regional group is confusing. Especially if a regional chapter does not necessarily have to be it's own entity (i.e. Open Collective could host).

Seems like it would less confusing if Open Collective was defining chapters like most other orgs or meetup groups - on a regional level - similar to https://opencollective.com/wwcode#members
or https://opencollective.com/operationcode-collective#members

While we could still make clear that an Open Collective chapter could also be an entity/host, I think more people would create an Open Collective chapter/meetup group in their region if they didn't have to create an entity until that was justified.

@xdamman @piamancini @hollomancer Does any of that make sense?

One of which is WordPress who have already created a good bit of policies and materials which we could essentially copy/fork and modify.

Since this comment, WordPress is now using Open Collective to manage the finances for charity hackathons and open-source workshops. They have also incorporated how they're using Open Collective into their guidelines and documentation (some of which is referenced in https://github.com/opencollective/documentation/issues/2#issuecomment-415519216)
This post provides more info regarding their process as well as who is contributing. If we're looking to create similar guides and tools for Open Collective and communities who use, it might not be a bad idea to reach out to whoever is administrating https://opencollective.com/wpf and/or followup in #wordpress regarding onboarding contributors and potential collaboration.

I created an initial outline in the issue summary https://github.com/opencollective/documentation/issues/2#issue-353069472 which incorporates elements of https://opencollective.com/chapters (which is outdated and contains broken links) and the WordPress organizer guide https://make.wordpress.org/community/handbook/meetup-organizer/welcome/
This would be a good project to fund with a bounty or similar incentive considering there's no obvious documentation for organizing Open Collective Chapters or events.


Fix deployment to GitHub Pages

Currently, when there's a new release, it is pushed to both dev and main, and both include the GitHub Pages workflow. If they are running in parallel, there can be conflicts, so this needs to be fixed.

Unfortunately, this currently needs a workaround, see also: https://github.community/t/how-to-retry-a-failed-step-in-github-actions-workflow/125880


update langchain to use 0.1.0

See: https://h2oai.slack.com/archives/C05MUTAHKU5/p1705074165449009?thread_ts=1705073212.277659&cid=C05MUTAHKU5, current version is good as is.


If +flyspell enabled, disable spell-fu keybindings and use flyspell ones

Fixing keybindings using recently introduced +flyspell flag ;)

Whoops. Forgot about those. Thanks!


Add a collision area scene for handling jumping down

Do not merge unless we're sure we're going with retiling instead of a collision area 2d workaround


CI/CD using GitHub Actions

This PR closes #358. (It also fixes a version metadata in dune-project)
Note: I guess the CI won't start as soon as I open this PR, because it comes from an external fork. See "checklist" below.
Anyway, the experiments I did in the pfitaxel fork (cf. "dry-run" part below) show this will fix #360 (Docker Hub deployment issue).
To sum up, it replaces Travis CI + Docker Hub automated build with GitHub Actions for both CI and CD, keeping the same semantics as much as possible.
Checklist
@yurug below are the steps I have applied to test this PR; you'll need to apply them on your side before merging the PR:

[x] 1. Browse https://hub.docker.com/repository/docker/ocamlsf/learn-ocaml/builds/edit and delete all build rules (I guess there are two), then click Save;
[x] 2. Browse https://hub.docker.com/settings/security and create a new personal access token dedicated to learn-ocaml CI/CD;
[x] 3. Browse https://github.com/ocaml-sf/learn-ocaml/settings/secrets and create two new secrets (repository secrets):

DOCKER_USERNAME := your Docker Hub login
DOCKER_PASSWORD := the token from item 2;

[x] 4. Merge this PR to trigger CI and CD from master.

Then I'll rebase #354 to restart CI on that other PR as I had suggested − https://github.com/ocaml-sf/learn-ocaml/pull/354#issuecomment-670603148
Dry-run
This GitHub Actions configuration gathers four YAML files (there is a little duplication between deploy-*.yml files but it appears GitHub Action doesn't provide yet ways to "include" a YAML in another one):

build-and-test.yml - example run
build-macos.yml - example run
deploy-master.yml - example run
deploy-tags.yml - example run

Comparison with Docker Hub automated build
According to the line cat /proc/cpuinfo /proc/meminfo I've added in (the first stage of) the Dockerfile, it is very possible issue #360 showed up because of lack of RAM:

Docker Hub: 1 processor; memory:
MemTotal:        3997656 kB
MemFree:         1341656 kB
MemAvailable:    3380948 kB

GitHub Actions: 2 processors; memory:
MemTotal:        7121280 kB
MemFree:         2074152 kB
MemAvailable:    6172100 kB

Thank you!


Fix exception type mismatch error in TestArrayElementwiseOp

See https://github.com/cupy/cupy/pull/2376
This tentative workaround should be resolved.

PR #2396 resolved this issue.


Let's add a hint to the plugin's name to explicitly explain its purpose

For a long time I was searching for ability to have a Ctrl+Tab List of the opened files (tabs) in MRU order similarly to the one in VS Code or Notepad++. And Sublime Text core developers for some reason do not want to add that to the editor.
So I was searching for existing plugins that could propose such an ability. I was searching for plugins containing anything like "File List", "Tab List", "Opened Files", "Opened Tabs", "MRU", "Ctrl Tab" in their name or description.
The closest one was Tab Filter, which is great except for its files (tabs) list is not in the MRU order.
Just recently, absolutely accidentally, I noticed that somebody mentioned Compass Navigator as a plugin that shows the files (tabs) list in the MRU order. So I tried Compass Navigator - and it is the closest implementation of the Ctrl+Tab List for Sublime Text I've ever seen.
So I'm wondering: could you add keywords similar to Opened tabs/files, MRU, Ctrl+Tab to the plugin description to allow to find it by these keywords among all the other plugins?
Here is a link to the corresponding topic in the Sublime Forum:
https://forum.sublimetext.com/t/shortcut-for-the-tabs-menu/63533

Sure. I updated the description. Afaik, PC filters packages using the description too. Thanks for bringing this up!


Roll Flutter Engine from 9d49175618f5 to eebe41f53610 (1 revision)

https://github.com/flutter/engine/compare/9d49175618f5...eebe41f53610
2023-10-19<EMAIL_ADDRESS>Add missing import (flutter/engine#47083)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Raise error when import is called on finalized container

Let me know whether the error should have another name, I thought the name ContainerAlreadyFinalizedError is adequate as it can be reused in the future for any operation on a finalized container which should raise.
I've also found the yard doc on Dry::System::Container.import is outdated, let me know if I should update that. Found the description in PR https://github.com/dry-rb/dry-system/pull/209 quite helpful and I could start to transfer some bits and pieces of it into the doc.
Closes #228

Thanks for taking care of this, @tak1n! I've resolved a merge conflict in #254 and brought this in :)


Set DEBUG env for github actions

Summary Change Description
Briefly describe the changes in this pull request (not intended as a full
recounting of the issue description).
Affected Issue Numbers

Resolves #<issue_number_1>
Resolves #<issue_number_2>

Code Review Notes
Describe any areas of concern that code reviewers should pay particular
attention to.  E.g. There are significant logic changes in function X.
Checklist

[ ] All issue requirements satisfied (or no linked issues)
[ ] Linting passes.
[ ] Migrations created & committed (or no model changes)
[ ] Tests implemented (or no code changes)
[ ] All tests pass

Separately implemented. This was helpful.

Awesome, it was simpler to show a working example than explain via comment/Slack. ;-)


Update to AGP3-alpha8.

Any idea what's missing here? I saw this failure in another project, so I think I'm missing something for Travis.

Build tools 26.0.0 were (accidentally?) deleted from maven. Need to update to 26.0.1.

Ah, thank you!

(kicking travis again.)


Using uzn file, output from command line and programmatically using .net dll are different.

I am using 3.04 tesseract ocr. Running command line like below gives me better result compared to programmatically calling the api.
"tesseract Page_00001_IP.tif Page_00001_IP -l eng -psm 4"
The api is called as follows.
var inputName = Path.GetFileName(imageFile).Replace(Path.GetExtension(imageFile), "");
using (var page = engine.Process(img, inputName, PageSegMode.SingleColumn))
{
text = page.GetText();
}
I have created the .uzn file same name as the image file. It contains regions specifying Left, Top, Width and Height. I appreciate any help to resolve this.
Thanks,
Ravi

We do not provide support for tesseract wrapers. Please use tesseract user forum.


[AUTOPATCHER-CORE] Upgrade python-sphinxcontrib-websupport to 1.2.7 3.0 - Upgrade

[AUTOPATCHER-CORE] Upgrade python-sphinxcontrib-websupport to 1.2.7 3.0 - Upgrade
Upgrade pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=509253&view=results
buddy build -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=509255&view=results

https://dev.azure.com/mariner-org/mariner/_build/results?buildId=512656&view=results


An error in the startup of the JPA entity manager was not showing in the logs

Stroom beta.13 started up and was missing the ACTIVITY table.  The error produced by the initialisation of the JPA entity manger was not shown in the logs.  The yml config had the root level as ERROR and stroom as INFO.
Need to change root level to WARN and to ensure any exception in the initialisation of the JPA entity manager is logged, possibly by catching and logging an explicit error ourselves.
When we change root level to INFO (to track down the problem) we saw:
INFO ..... java.sql.DatabaseMetaData .... Table not found: ACTIVITY
WARN .....org.springframework.context.annotation.AnnotationConfigApplicationContext - Exception encountered during context initialisation ....... BeanCreationException........HibernateException ........Missing table ACTIVITY

I have added a catch of Runtime errors in the app initialisation and logged them as errors before rethrowing


Update runlist.txt

This commit is also included in https://github.com/folio-org/folio-analytics/pull/588.  Closing.


wstunnel: exclude TLS arguments when enableHTTPS is false

Description of changes
When the module option enableHTTPS is false, omit tls-certificate and tls-private-key as attributes during command-line argument generation.
Prior to this change, the following error was seen:
       … while evaluating a path segment
         at /nix/store/5ls71zadnzh0api6ybg7p6y9hidhmnp9-source/nixos/modules/services/networking/wstunnel.nix:322:95:
          321|                   tls-certificate =
          322|                     if useACMEHost != null then "${certConfig.directory}/fullchain.pem" else "${tlsCertificate}";
             |                                                                                ^
          323|                   tls-private-key = if useACMEHost != null then "${certConfig.directory}/key.pem" else "${tlsKey}";

       error: cannot coerce null to a string: null

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

[x] Tested, as applicable:

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

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

@NixOS/nixpkgs-merge-bot merge


bug: download button on website is broken

Hi,
On DocKit website, the download button is not working.
I think the error comes from:
TypeError: navigator.userAgentData is undefined
    n https://dockit.geekfun.club/assets/chunks/theme.lK2rZp0k.js:1
    Ge https://dockit.geekfun.club/assets/chunks/framework.PNMSvVW_.js:1
    Se https://dockit.geekfun.club/assets/chunks/framework.PNMSvVW_.js:1
    n https://dockit.geekfun.club/assets/chunks/framework.PNMSvVW_.js:1

Hi @Blankll
Pretty sure it is being blocked by Firefox privacy settings.
I am using 124.0.1 (64-bit) on Ubuntu, ublock, tracking protection in strict mode.

Thanks, I'll have a look

the function to detect host arch and OS is supported only by limited browsers,  btw I have added auto-new version detect to directly download the latest version instead of visiting the browser again, this issue should be not necessary to fix then
see the support list: https://developer.mozilla.org/en-US/docs/Web/API/Navigator/userAgentData

Ok.
Suggestion: the Releases section of Github could be fallback if the download code failed to execute.


build: Move secrets to environment variables in run actions

Please describe the purpose of this pull request.
Secrets should be passed to run commands using environment variables so the secret value isn't visible in the host's process list. re: https://docs.github.com/en/actions/security-guides/using-secrets-in-github-actions#using-secrets-in-a-workflow
How to test
n/a
Have you tested this PR?
n/a
Related issues or PRs
n/a
Is your PR over 500 lines of code?
n/a
Additional context
n/a

Closed because #846 also contained this fix


On the mobile wallet Transaction page, currently don't have the options - Validate Proof File, Refresh, Export.

Description:
On the mobile wallet Transaction page, currently don't have the following options

Validate Proof File
Export.CSV
Refresh

Expecting options in mobile wallet same as QT wallet.

@sarawut11   @vinayaga07    Export to CSV is not needed for the first version of mobile.

Moved export feature into another issue - https://github.com/mwcproject/mwc-qt-wallet/issues/833

Verified, hence closing this ticket.


[ApiView] swagger api parser remove APIView dependencies

@ruowan not sure what the work here is/was. Checking if it is done or we should still do this

@ruowan I believe you are talking about dependency on APIView project from swagger parser. https://github.com/Azure/azure-sdk-tools/blob/main/tools/apiview/parsers/swagger-api-parser/SwaggerApiParser/SwaggerApiParser.csproj#L24
Can you please remind me why do we need this dependency? for CodeFile class?

Yes. For CodeFile class. It depends on APIView project.

Here is the PR to remove that dependency https://github.com/Azure/azure-sdk-tools/pull/5319


Insufficient installation process?

2022/02/22 20:53:25 open C:\Users*****.ssh\config: The system cannot find the specified path.
Appears after "successful" installation on Win 10 Pro 64 bit

You must have the ~/.ssh/config file in order for sshs to work. The table is generated using this file.

On Windows, the OpenSSH client creates the corresponding files only after first use. Can't SSHS create the .ssh directory and its contents itself? Just an idea.

Agreed. It would completely get rid of that error message.

Still does not work. Testet 2.1.1

Woops, path module doesn't work with Windows paths for reasons. a6d75776e9852fe2db1b5a79a539fdccd99f2fba fixes the issue, provided in release 2.1.2.

Still does not work. Tested 2.1.2

Yes works fine thank you!


Bug: (Linux) C4D render fails with false positive

Describe the bug
In my sample scene I can get C4D log message that says:
Description Error: ainode_image is already registered
When rendering with Arnold.
It doesn't seem to be a fatal error and the render still completes successfully.
However when I run it though the adaptor, the error regex catches that and fails the task
Expected Behaviour
I think we need to tighten up the error regex pattern.
There have been a few cases where the new regex has gotten us into trouble.
Looking back at D10, it looks like the error regexes were more specific (shown below)
".*Document not found.*"
".*Project not found.*"
".*Error rendering project.*"
".*Error loading project.*"
".*Error rendering document.*"
".*Error loading document.*"
".*Rendering failed.*"
".*Asset missing.*"
".*Asset Error.*"
".*Invalid License.*"
".*License Check error.*"
".*Files cannot be written.*"
".*Enter Registration Data.*"
".*Unable to write file.*"
".*RenderDocument failed with return code.*"
"Frame rendering aborted."
'Cannot find procedure "rsPreference"'

Current Behaviour
Job gets killed
Here is the log:
2024/12/20 06:48:03-08:00 ==============================================
2024/12/20 06:48:03-08:00 --------- Running Task
2024/12/20 06:48:03-08:00 ==============================================
2024/12/20 06:48:03-08:00 Parameter values:
2024/12/20 06:48:03-08:00 Frame(INT) = 0
2024/12/20 06:48:03-08:00 ----------------------------------------------
2024/12/20 06:48:03-08:00 Phase: Setup
2024/12/20 06:48:03-08:00 ----------------------------------------------
2024/12/20 06:48:03-08:00 Writing embedded files for Task to disk.
2024/12/20 06:48:03-08:00 Mapping: Task.File.runData -> /sessions/session-e18b137a0e72432994612c32a653b6eco063x6a3/embedded_files_4nnmsg0/run-data.yaml
2024/12/20 06:48:03-08:00 Wrote: runData -> /sessions/session-e18b137a0e72432994612c32a653b6eco063x6a3/embedded_files_4nnmsg0/run-data.yaml
2024/12/20 06:48:03-08:00 ----------------------------------------------
2024/12/20 06:48:03-08:00 Phase: Running action
2024/12/20 06:48:03-08:00 ----------------------------------------------
2024/12/20 06:48:03-08:00 Running command /sessions/session-e18b137a0e72432994612c32a653b6eco063x6a3/tmpb1aml8ja.sh
2024/12/20 06:48:03-08:00 Command started as pid: 3918
2024/12/20 06:48:03-08:00 Output:
2024/12/20 06:48:03-08:00 INFO: Applying user-level configuration: /home/deadline-worker-agent/.openjd/adaptors/runtime/configuration.json
2024/12/20 06:48:03-08:00 INFO: Applying user-level configuration: /home/deadline-worker-agent/.openjd/adaptors/Cinema4DAdaptor/Cinema4DAdaptor.json
2024/12/20 06:48:03-08:00 ADAPTOR_OUTPUT:
2024/12/20 06:48:03-08:00 ADAPTOR_OUTPUT: STDOUT: Performing action: {"name": "frame", "args": {"frame": 0}}
2024/12/20 06:48:03-08:00 ADAPTOR_OUTPUT: STDOUT: Performing action: {"name": "start_render", "args": {"frame": 0}}
2024/12/20 06:48:03-08:00 ADAPTOR_OUTPUT: STDOUT: Requesting Path Mapping for path '/mnt/MediaProduction/zENG/basic c4d/test'.
2024/12/20 06:48:03-08:00 ADAPTOR_OUTPUT: STDOUT: Mapped path '/mnt/MediaProduction/zENG/basic c4d/test' to '/mnt/MediaProduction/zENG/basic c4d/test'.
2024/12/20 06:48:04-08:00 ADAPTOR_OUTPUT: STDOUT: Requesting Path Mapping for path '/mnt/MediaProduction/zENG/basic c4d/tex/ame.PNG'.
2024/12/20 06:48:04-08:00 ADAPTOR_OUTPUT: STDOUT: Mapped path '/mnt/MediaProduction/zENG/basic c4d/tex/ame.PNG' to '/mnt/MediaProduction/zENG/basic c4d/tex/ame.PNG'.
2024/12/20 06:48:04-08:00 ADAPTOR_OUTPUT: STDOUT: Requesting Path Mapping for path '/mnt/MediaProduction/zENG/basic c4d/tex/ame_2.PNG'.
2024/12/20 06:48:04-08:00 ADAPTOR_OUTPUT: STDOUT: Mapped path '/mnt/MediaProduction/zENG/basic c4d/tex/ame_2.PNG' to '/mnt/MediaProduction/zENG/basic c4d/tex/ame_2.PNG'.
2024/12/20 06:48:04-08:00 ADAPTOR_OUTPUT: STDOUT: Description Error: ainode_image is already registered
2024/12/20 06:48:04-08:00 ADAPTOR_OUTPUT: STDERR: Rendering frame 0 at <Fri Dec 20 14:48:04 2024>
2024/12/20 06:48:04-08:00 openjd_fail: Error encountered while running adaptor: Cinema4D Encountered an Error: Description Error: ainode_image is already registered
2024/12/20 06:48:04-08:00 ADAPTOR_OUTPUT: STDOUT: Rendering frame 0 at <Fri Dec 20 14:48:04 2024>Progress: 5%
2024/12/20 06:48:04-08:00 ADAPTOR_OUTPUT: STDOUT: Progress: 11%
2024/12/20 06:48:04-08:00 ERROR: Entrypoint failed: STDOUT: Requesting Path Mapping for path '/mnt/MediaProduction/zENG/basic c4d/tex/ame.PNG'.
2024/12/20 06:48:04-08:00 STDOUT: Mapped path '/mnt/MediaProduction/zENG/basic c4d/tex/ame.PNG' to '/mnt/MediaProduction/zENG/basic c4d/tex/ame.PNG'.
2024/12/20 06:48:04-08:00 STDOUT: Requesting Path Mapping for path '/mnt/MediaProduction/zENG/basic c4d/tex/ame_2.PNG'.
2024/12/20 06:48:04-08:00 STDOUT: Mapped path '/mnt/MediaProduction/zENG/basic c4d/tex/ame_2.PNG' to '/mnt/MediaProduction/zENG/basic c4d/tex/ame_2.PNG'.
2024/12/20 06:48:04-08:00 STDOUT: Description Error: ainode_image is already registered
2024/12/20 06:48:04-08:00 STDERR: Rendering frame 0 at <Fri Dec 20 14:48:04 2024>
2024/12/20 06:48:04-08:00 openjd_fail: Error encountered while running adaptor: Cinema4D Encountered an Error: Description Error: ainode_image is already registered
2024/12/20 06:48:04-08:00 STDOUT: Rendering frame 0 at <Fri Dec 20 14:48:04 2024>Progress: 5%
2024/12/20 06:48:04-08:00 STDOUT: Progress: 11%
2024/12/20 06:48:04-08:00 Process pid 3918 exited with code: 1 (unsigned) / 0x1 (hex)

Reproduction Steps
C4D_ArnoldShaderTest.zip
Attached scene file.
You only see this message on a Linux worker.
Environment
At minimum:

Operating system: Rocky Linux worker
Version of Cinema 4D: 2024.4.1 / 2024_4_1_c6c50ba2c0c8_1301281096
3: Version of this package: 0.5.4

Please share other details about your environment that you think might be relevant to reproducing the bug.

Hi @Ahuge , thanks for raising this issue.
This is current expected behaviour. The adaptor is configured to detect errors in the print statements and stop the renders.
We use a similar approach in other DCC adaptors, like in Maya adaptor.
Tightening the regex could risk the Cinema 4D application running indefinitely. If we don't catch all errors, new fatal errors in future C4D versions might go undetected, potentially causing the application to run continuously. But I'll discuss with others in my team if we have any middle-ground.

Deadline 10's doesn't have issues with Cinema 4D processes running indefinitely so we should be fine to borrow from it's regex.
Also, I think it's more appropriate to learn from Deadline 10's C4D plugin than to learn from the Maya adaptor. Different DCCs behave differently and need different handling.

Switching this adaptor to use the exact same error regexes as Deadline 10 seems like a great idea to me. We don't need to re-learn all the individual lessons accumulated there over many years, we can just apply them directly.

Makes sense. We will work on making sure the adaptor uses the same exact error regexes as Deadline 10.


Avoid exception when unclosed directive container is found within another container and chunkDocument is empty (fixes #16, fixes micromark/micromark#173).

Initial checklist

[x] I read the support docs
[x] I read the contributing guide
[x] I agree to follow the code of conduct
[x] I searched issues and couldn’t find anything (or linked relevant results below)
[x] If applicable, I’ve added docs and tests

Description of changes
Avoid exception when unclosed directive container is found within another container and chunkDocument is empty (fixes #16, fixes micromark/micromark#173).
I tried for a while to parse the scenario successfully, but couldn't find a way to avoid creating an empty container that didn't break one of the existing "lazy" tests. As it is already an edge-case scenario, giving up like this seems okay to me - definitely better than crashing callers. :)

Thanks for working on this!
I appreciate that errors are not nice. Still, I believe that we must strive towards correctness.
You’re silencing an error, and introducing an incorrect result.
I spent some time with the code and introduced a proper fix.

Released in 3.0.2!

Love it, thank you!


Edit README

@homu r+

@homu r+

@WSlatton @Oksisane @homu is currently experiencing Internal Server Errors for this repository.
Until fixed, @homu will not be required as a status check to merge.

@homu r+

@homu r+

:pushpin: Commit b8af31f has been approved by smo-key

:hourglass: Testing commit b8af31f with merge 685ce32...

@homu fixed by removing the repo from homu.io, then removing both the webhook and the @homu user.

:sunny: Test successful - status


fix(redis-4): omit credentials from db.connection_string span attribute

Which problem is this PR solving?

Omit credentials from being exposed on redis-4 span attribute db.connection_string

Short description of the changes
Before attaching db.connection_string attribute to spans created by redis-4 we will remove credentials from the connection URL.
Specifically we remove Basic Auth and the query parameter user_pwd and reconstruct the URL to provide as much context as possible without leaking credentials.

@jarlefosen Hi, do you want to keep working on that? There are only a few small fixes left.

Hi @haddasbronfman, I sure do intend to update the PR according to comments. But at the moment I'm on vacation and will be updating as soon as I'm near my computer.
Terrible timing, sorry for delaying this.

@jarlefosen No rush. just wanted to know if you want to proceed with this. thanks!


Provide Solr sample requests and responses

@ohjuarez, could you please provide sample HTTP requests and responses for the following?

Creating a record in Solr (HTTP POST)
Reading a Solr record (HTTP GET)

curl examples would likely be enough.  Thanks!
CC: @cjav , @seandavi

You can query from the following URL:
http://localhost:8983/solr/#/sequence/query
You can browse the schema:
http://localhost:8983/solr/#/sequence/schema-browser?field=acxn
From the command line:
curl "http://<IP_ADDRESS>:8983/solr/sequence/select?q=%3A&wt=json&indent=true"
Query Syntax:
http://wiki.apache.org/solr/SolrQuerySyntax

Added Banana to Visualize solr repository
http://localhost:8983/solr/banana/src/index.html

http://localhost:8983/solr/sequence/select?q=*%3A*&wt=json&indent=true&facet=true&facet.pivot=gene_name_s,impact_s

Another example:
http://localhost:8983/solr/sequence/select?q=*%3A*&rows=0&wt=json&indent=true&facet=true&facet.pivot=gene_name_s,impact_s


[zh-cn]Remove duplicate parts

The description of the time zone section is duplicated

@ydFu PTAL

/lgtm
/approve
Thanks.


Code action to add a using declaration

Before:
void foo() {
  int x = bar::baz();
  //           ^^^
}

After:
using bar::baz;

void foo() {
  int x = baz();
}

Some questions about

what scope to add the using decl in (TU scope? Namespace scope?)
try to group with other using decls?
detect shadowing cases?

we've got that code action now. thanks to adamcz@


Any plans for typescript bindings?

Absolutely, and I welcome a PR for it.

Absolutely, and I welcome a PR for it.

Thanks for the quick response. If no one gets to it in a few weeks I can take a crack at it.

Thanks for the quick response. If no one gets to it in a few weeks I can take a crack at it.

I will be following this

I will be following this

I just opened a pull request for this. As best I can tell, the type defs were pretty straightforward. Of course, that makes me a little worried I missed something. But if nothing else, it's a starting point.

@chimericdream Thanks! Requested a few changes. I don't have much TS experience, so I'd love another set of eyes to review this.

Released in v0.12.1. @chimericdream, thanks for the contribution!


Properly Compare True and False Order in Sourcekit

Comparing "false" > "true" will return 0, which when returned by compare
indicates the two are equal which will result in a non deterministic
ordering of "false" and "true". Using .compare returns a negative number
as expected.

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 34f64084d1be799ec4fc12688270e1d9d0a57b59

Nice, maybe we can re-enable this now?
XPASS: Swift(linux-x86_64) :: SourceKit/CodeComplete/complete_sort_order.swift

Seems like it 👍
@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 34f64084d1be799ec4fc12688270e1d9d0a57b59

Build failed
Swift Test Linux Platform
Git Sha - 34f64084d1be799ec4fc12688270e1d9d0a57b59


[Feature][Connector-V2][Elasticsearch] Elasticsearch supports spaced write and is relatively real-time data source friendly #4347

…write and is relatively real-time data source friendly #4347

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

@Hisoka-X @TyrantLucifer @TaoZex Please  review thanks

-1. I don't think it is a good way to add additional threads to write data in the sink. This implementation method makes resources uncontrollable, and also loses the original intention of the committer interface design.

boss  Do you have any good ideas?

It's not a good change, for now we do not have the strong need to change it. So I will close it, if you have any questions, please send an e-mail to<EMAIL_ADDRESS>and discuss with community about your solution and your concern.


Which tokenizer does this model use?

As title.

Tokenisation happens here - it uses the NLTK tokenizer.


A syntax to improve in readme file

Hi,
For Typescript you propose to declare your module as:
import * as net 'net'
import { PromiseSocket } as 'promise-socket'

But you forgot to put from instruction. So it should be:
import * as net from "net";
import PromiseSocket from "promise-socket";

For what version of promise-socket? I checked at https://www.npmjs.com/package/promise-socket and there is
import PromiseSocket from 'promise-socket'

in README.
And it should be:
import net from 'net'

because this "star" notation is now obsoleted. See --esModuleInterop options for Typescript compiler.


Switch to 1ES servicing pools in release/3.1

Summary of the changes

We need to switch to the new 1ES servicing pools for all servicing branches. This does that for release/3.1.

Tracking issue: https://github.com/dotnet/core-eng/issues/14276
cc/ @ulisesh @lpatalas

@dougbu it seems like nobody on the Razor Tooling team has the permissions to merge this, would it be possible to give one of us permissions or have you press the button?

@ryanbrandenburg it's been a while 😃 I don't have push rights in this repo anymore and can't see who does. @jinujoseph❔

Please note however that we try pretty hard to put all changes for the 3.1.X release after updating the branding to 3.1.X i.e. this branch isn't open at the moment

@dougbu you say that this branch isn't open, but there exists no release/3.1.X branch (here or on dotnet/aspnetcore), so where are these changes (which are due this week) supposed to go?

@dougbu you say that this branch isn't open, but there exists no release/3.1.X branch (here or on dotnet/aspnetcore), so where are these changes (which are due this week) supposed to go?

My comment was about when, not where. release/3.1 is the right branch to use but we shouldn't check in before the branding update for 3.1.21.


Add descending by date comparators for run and job pojo types

Will facilitate dashboards to have sorted in descending order so can render as html without reverse.
2022-02-02 < 2011-01-01 < null

done


0.25 readme

0.25 readme update

I want to do https://github.com/microsoft/PowerToys/issues/7616 before we release and point loc issues to that so we can address them quickly


fix in resources button

Fixed the resource button when clicking the button it was giving the uneven box shadow

After fixing

@Aman1919 Thank you for contributing :)


alpine:latest: no matching manifest for linux/amd64 in the manifest list entries

$ docker pull alpine
Using default tag: latest
latest: Pulling from library/alpine
no matching manifest for linux/amd64 in the manifest list entries

yep, we've been hit by the same problem for ocaml/opam-repository's CI :/

This issue is probably duplicated with #418 though

+1, exact same output.

Same result with 3.8
docker pull alpine:3.8
3.8: Pulling from library/alpine
no matching manifest for linux/amd64 in the manifest list entries

+1, exact same output. Tested with Amazon Linux and Centos 7.5.1804.

Duplicate of https://github.com/gliderlabs/docker-alpine/issues/418


how to link duflor and duflor-gui?

duflor itself can be used gui-less, so we should have to link from duflor-gui -> duflor.
https://github.com/Claudius-Appel/duflor
Consider:
https://shiny.posit.co/r/getstarted/shiny-basics/lesson7/

this is resolved, as duflor.gui imports duflor as of 4c93170f89c0c9e6ed8b076718c398dfbbbbedaa


New vaccination parser

Fix #153

Just build it to host it myself and noticed that the attribute administeredVaccinations is missing in the response
response.txt

Just build it to host it myself and noticed that the attribute administeredVaccinations is missing in the response
response.txt
Edit: Maybe the issue is line 171 🤷♂

    "totvaccination",

You are right! will patch it.
Thanks a lot

administeredVaccinations is fixed

Looks good, thanks @Rubber1Duck! :)
Are you sure it's a good idea to change the type of the properties that were dropped by the RKI? I could imagine it wouldn't break as many apps if we just leave them as zero.

Looks good, thanks @Rubber1Duck! :)
Are you sure it's a good idea to change the type of the properties that were dropped by the RKI? I could imagine it wouldn't break as many apps if we just leave them as zero.

I am unsure if i am sure :-) I had the same thoughts. And maybe it is the better way to set this to "Null" (empty).
Okay, i think thats the better way.... will patch this.

Do you mean null as in null or as in 0? :D

I hate to be that guy but I think the number zero would be safer for the users of this API. However, if I'm the only one that thinks that way, I'm fine with it :)
I think i should be clear for all that this information is not available, so null should by the better part.
do you remember the case that some fields at "Bund" are filled with "-", there also this fields are changed to null, and that i think was okay!
Lets see what @marlon360 opinion is. to sitch this to 0 is not a big deal :-) But unfortnatly he is realy very busy at the moment


[Vue warn]: Invalid vnode type when creating vnode: undefined.

helloworld.vue内容

<template>

hello world

</template>

react中引用:
import { applyVueInReact, applyPureVueInReact } from 'veaury';
import helloworld from './helloworld.vue';

const HelloWorld = applyVueInReact(helloworld);

// <HelloWorld></HelloWorld>

此时控制台提示

界面的元素:

@zhaozhongyu Were you able to solve this issue?

I have the same issue

@glepka , nope, I couldn't figure out. I ended up not using veaury. Instead, I'm developing the vue app screen by screen and using routing to run it by the side of our react app.

@Parag2385 @glepka Can anyone provide an example project?

@Parag2385 @glepka Can anyone provide an example project?

what exactly needs to be sent?

Provide the github URL of a test project. @glepka

I created a private repository and added you as a collaborator
https://github.com/glepka/vueinreact
There is a simple structure
The Vue component should appear on the "/test" route, but this does not happen
The vue component itself is located in the dist folder

@Parag2385 @glepka Can anyone provide an example project?

what exactly needs to be sent?

Provide the github URL of a test project. @glepka

I created a private repository and added you as a collaborator https://github.com/glepka/vueinreact
There is a simple structure The Vue component should appear on the "/test" route, but this does not happen
The vue component itself is located in the dist folder

@glepka
So is this project built using create reat app? It looks like you don't have webpack.config.js configured, I think you may need to run npm run eject first.
For details on how to configure webpack, please refer to https://github.com/gloriasoft/veaury/blob/master/dev-project-react/README.md

@Parag2385 @glepka Can anyone provide an example project?

what exactly needs to be sent?

Provide the github URL of a test project. @glepka

I created a private repository and added you as a collaborator https://github.com/glepka/vueinreact
There is a simple structure The Vue component should appear on the "/test" route, but this does not happen
The vue component itself is located in the dist folder

@glepka
So is this project built using create reat app? It looks like you don't have webpack.config.js configured, I think you may need to run npm run eject first.
For details on how to configure webpack, please refer to https://github.com/gloriasoft/veaury/tree/master/dev-project-react/README.md

@glepka Is the problem solved?


Adding university

Universidad de Palermo is added

@zittermann I am sorry, the domain palermo.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


select.js line41 should be value = {option.value}; not value = {i}

value is an index into the resolvedoptions array, not an actual value. If you look at onChange you'll see that we setValue resolvedOptions[thatIndex] to set the actual form state.
Closing for now, please re-open if I misunderstood you.

Thx


??? triggers dead-code warning, with 'sbt test'

Hadn't noticed this in #87 - sorry.
With the code out of the comments, the use of -Ywarn-dead-code kicks in with sbt test (kind of surprising, I would suppose ??? not to be treated as dead code). Anyways, it may be best to do other stuff and keep the graph in comments.
Or remove the whole file, as unneeded?

Do we still need this example after moving DummyProducer and DummyConsumer into examples?

Nope. Removing the file in #86 so closing this PR.


Current Guidance Around Limits?

Hi, this is a great library, thank you for maintaining it.
I went to open this in Discussions (as was suggested here https://github.com/nidhaloff/deep-translator/issues/94#issuecomment-904040700) but that seems quite inactive so I wanted to ask here since I do believe it constitutes an issue, technically.
I've seen a few mentions of limit numbers:
https://github.com/nidhaloff/deep-translator/issues/154 suggests 5 requests per second and up to 200k requests per day for Google
But https://github.com/nidhaloff/deep-translator/discussions/132 suggests being rate limited after fewer than 100 calls
https://github.com/nidhaloff/deep-translator/issues/161 suggests only a few words for lingueeTranslator for example
I also read through some other issues like https://github.com/nidhaloff/deep-translator/issues/149 but those weren't very useful.
So the question is: Are there any general benchmarks on how many requests this package can handle? I'm looking to translate in batches of ~50 characters but it could be bursts of many requests at once. Let's say 100 requests/second. I can pay for API access as a fallback, but building out my project, this library seemed to have incredible potential but I'm concerned about being rate limited basically right away and I can't tell if that's a consistent issue or not. Given this has 1K+ stars I'm assuming people are using it fairly heavily?
I'm basically wondering if I should spend time building my project around this library or if I should just bypass it and eat the cost of Google's API directly if I need to scale at all.
Finally, are there any more examples or explanations on how the proxies work?
Basically any guidance around limits, benchmarks, etc. would be greatly appreciated!

I intend to use GoogleTranslator in my api, so if the user inputs search query in other languages, I can translate them to my desired language. but, there may be a lot of users trying my api, many of them simultaniously. So, I won't be able to use this package for more than 5 requests per second?

This package is just a collection of API clients; the rate limits are not different than if you were calling the Google API by yourself.


Chore (add extra class for overlay trigger element) - Minor column width patch

Sorry again @AllenFang. Last time i do this. Promise. haha

@khinlatt, That's fine :)


no-unknown-property & styleName attribute (v7.31.6)

React CSS Modules uses the styleName attribute (https://github.com/gajus/babel-plugin-react-css-modules). Not sure if this is standard enough to be included...
<div styleName="styles.container" />
Thanks,
Mark.

I think this is in the same category as emotion's css attribute - both are utterly nonstandard, and neither should be used without a data- prefix, so the best approach is for you to add it to ignore in your rule config.


getnameclaims returns error when passing claim_id

Sometimes it works and sometimes it doesn't. (For working example see #161)
This
lbryum getnameclaims --claim_id=2e821d91ffeb0cd100a3570753392d4b2116862d
Returns
Traceback (most recent call last):
  File "/root/git-repos/lbry/lbry-venv/bin/lbryum", line 11, in <module>
    load_entry_point('lbryum', 'console_scripts', 'lbryum')()
  File "/root/git-repos/lbry-repos/lbryum/lbryum/main.py", line 273, in main
    result = server.run_cmdline(config_options)
  File "/root/git-repos/lbry/lbry-venv/local/lib/python2.7/site-packages/jsonrpclib/jsonrpc.py", line 288, in __call__
    return self.__send(self.__name, args)
  File "/root/git-repos/lbry/lbry-venv/local/lib/python2.7/site-packages/jsonrpclib/jsonrpc.py", line 238, in _request
    check_for_errors(response)
  File "/root/git-repos/lbry/lbry-venv/local/lib/python2.7/site-packages/jsonrpclib/jsonrpc.py", line 567, in check_for_errors
    raise ProtocolError((code, message))
jsonrpclib.jsonrpc.ProtocolError: (-32603, u"<type 'exceptions.AttributeError'>:'Decimal' object has no attribute '__dict__'")

this has been fixed


Fix Bonsly's Evolution level

Bonsly is required to level up while knowing Mimic

Yikes i messed up

@Marty-D? How do we usually represent this?

It's done by setting evoMove. See Ambipom, Lickilicky, Mamoswine, Tangrowth or Yanmega.
Mr. Mime has its evoMove set but its evoLevel is 1 which is presumably incorrect.
By the way, Veekun seems to think that Tsareena evolves from Steene if it knows Stomp.

Veekun is incorrect in that regard, ill look more closely into the other move-based evolutions

i just cant seem to do anything correctly, ive done this twice in a row

No, Veekun is right. I actually completely forgot evoMove even existed when I added the Gen 7 Pokemon, since it's used literally nowhere else; not even the PSdex uses it. And it's been around just as long as evoLevel, apparently.
I put 29 as the evoLevel because Steenee learns Stomp then. And of course, before the games came out no one knew they would let you relearn moves even above your current level. So evoMove should be "Stomp" and evoLevel should be 19.

I PR'ed Tsareena fix @Marty-D

I mean I could’ve just added the Tsareena fix into here but alright

Want me to PR it into your fork of this and just send it into this PR @KrisOnline

No I'm sure it's fine

Oh, now I remember why Mr. Mime and Sudowoodo have an evoLevel of 1 here; you can just breed them without an Incense to get them at level 1. So I'm just gonna merge the other PR then.


pynodegl-utils: add log messages for test passed / failed

Currently, we only have an exit code to indicate if the test passed or failed.
This makes it more clear to the user if the test passed or failed, especially if running the test manually (not via meson)

Should be fine but please use utils/tests: as prefix

updated


If Possible, Cleanups Should Be Handled By A Single Subscription

The work done by JobCreator often scales with the number of subscriptions active, since it has to run the split_algo for each subscription.  If we can reduce the number of cleanup subscriptions per workflow by putting all the cleanup in one subscription that might reduce the load.
Don't know how plausible this is.  It might help with our massive slowdown issues.

I believe it would be quite hard to come to a good approach. Which may not be worth in the end.
@ericvaandering @ticoann should we close it?


setup with pixi

set everything up with pixi.
# installs emsdk, builds compiler pkg localy
pixi run setup

# build a pkg for emscripten-wasm32
pixi run build-emscripten-wasm32-pkg recipes/recipes_emscripten/regex

Looking good!

Should the README be updated to recommend using Pixi for the local setup?

@jtpio jeah good suggestion!


Update CI & goreleaser to support releasing off of master

Our CI config on master currently has logic for running nightly releases, but no logic for pushing "proper" releases when a tag is pushed (that logic lives on 2.0). We plan to move towards releasing directly off of master starting with v2.1.0, so this needs updating.
Port the relevant CI and goreleaser configs from the 2.0 branch to master. We (hopefully) shouldn't need to maintain 2 separate goreleaser configs; see the docs for info about overriding the derived version from the outside (i.e. to specify a nightly version), and the influx-cli Makefile for examples of how that can be used.

#22877 laid most of the groundwork for this. After it merges, the remaining work will include:

Add a top-level workflow that triggers on tag pushes
Make sure we pass build-type: release to the build & package steps in the workflow
Add a .goreleaser/release.yml and any associated helper files (at least .goreleaser/_uploads-release.yml)

Getting this to work might require editing the .goreleaser/_common.yml and .goreleaser/_uploads-nightly.yml to support using the dynamic release version in filenames instead of a hard-coded constant like "nightly" or "snapshot"

Some considerations related to the UI:

We currently build master off of a tagged UI release approximating the most recent UI available. For OSS releases, we frequently do not want to use the very latest UI though - for example, the recent clockface upgrade.
For release branches, we link the release build to a tagged release in the UI repo. This isn't necessarily on a branch in the UI repo but often ends up being on one.
We always run e2e tests on the master branches of influxdb and ui with the latest versions of both. This is done using docker images that are pushed on merges to master of the respective repos. These tests are very valuable to prevent incompatibilities from developing between the two.
On release branches of influxdb, we run a different kind of e2e test that uses the UI embedded in the built binary and the e2e tests specific to the tagged release. These tests ensure that the actual UI embedded in the binary works with the version of the e2e tests applicable to it.

All that being said, for releasing off of master to work with the UI we would need to:

Build off of master from a tagged UI release, rather than the floating "OSS-Master" tag.
Continue running both the docker-based e2e tests of the latest build from each repo, in addition to running the e2e tests for the embedded UI that is currently linked for the release on master.
It would be nice, but not entirely necessary, to still be able to do a local build of OSS with the latest UI assets from the UI repo. I think this is something that could be done in the Makefile, and could possibly involve pulling the latest UI docker image, copying the built assets out of it, and using those to embed into the OSS binary.


Improve container support

This PR overhauls our container images to more idiomatically accommodate two use cases:

ldbc/datagen-standalone: Running a standalone version of Datagen - batteries included. This image is for users who want to try out Datagen or run on small datasets without having to build the JAR themselves, install Spark, etc. This image exposes the familiar run.py entry script used for local runs so users don't have to learn to use a separate script.
ldbc/datagen-jar: Abusing the Docker registry to store the assembly JAR in lack of a better alternative. An image composed of a single layer adding the jar at /jar. Users can use this image to add Datagen to their custom containers, without the need to build it themselves.

Additionally I removed the entry point that changes the user. I think it's more idiomatic if we let the users map the system user to the container root user with userns-remap.


Test
Tested bats 1.8.2 locally against the verify-image-signatures e2e tests.
Additional Information
Needs to be tagged as v3.1.4.
Tradeoff

Potential improvement

Tagged as v3.1.4.


[API change] Making the seeding more intuitive.

After a lot of thinking, here is my proposal for the new seeding setup.
Fixes #432
Fixes #586
Fixes part of #610 #193
My take here is:
We add too little value compared to a manual seeding done by the user and passed through the config. In the end, it requires a lot of documentation and engineering for a very simple result: Seeding when the experiment starts and get some seeds here and there. This can be done in a couple of lines by the user, and will be much more explicit and clear. We can encourage users to put their seed in the config so that it's saved and can be changed from the command line.
This is only a couples of LOC for the user, but a lot less LOC for sacred. This should prepare the ground to make something simpler for the config as mentioned in #610.
Of course this is a big breaking change. Because of this, we'll use the strategy of the global flag to avoid users shooting themselves in the foot.
I'm not sure the PR actually works if the flag is set to False. The doc isn't updated. This PR serves as a base for discussion about the behavior that we want.
We provide two things:

Helper functions to set the seeds of common libraries, those are often one liners, but we offer a common api and avoid the cognitive workload of looking into those libraries. Those could be removed since they're really simple, it can be discussed, we add very little value:

import sacred.random
sacred.random.set_python_random_seed(44)
sacred.random.set_numpy_seed(44)
sacred.random.set_tensorflow_seed(44)
sacred.random.set_torch_seed(44)

We offer a context manager/function decorator, which can isolate the piece of code/function from the rest of the code by saving and reloading random states. This enables local seeding. Example:

import sacred.random

@sacred.random.save_numpy_random_state
def my_function()
      sacred.random.set_numpy_seed(84) # this has no effect on the code outside the function

with sacred.random.save_torch_random_state():
      # sample distributions with torch here without affecting the random state of the program.
      # you can even use a local seed with sacred.randomness.set_torch_seed.

I realised that my understanding of libraries producing random numbers is really bad haha, let's close that for now and I'll come up with a better proposal later on.

Now I am curious, what it is that you discovered, that made you write this.
I don't necessarily agree with this proposal, but it does seem reasonable to me. So if you found a problem, I'd be interested to learn about it.

Basically, I didn't know that there was a way to generate random numbers without touching the global seed. RandomStates should be encouraged because touching the global random seed isn't thread safe. Furthermore, you can't know if some other libraries will mess up the global random state without your knowledge. See #573. In the end, to allow isolation, we should use random numbers generators, like it is currently done in the sacred API. One thing I'd like to change though, is the automatic seeding. Touching a global variable without explicit user approval can lead to some unexpected results if the user skip this part of the docs.


wand crashing when window resized

WARNING:wand.tools:Unable to find server configuration file, restoring from backup
Traceback (most recent call last):
  File "C:\Users\ThomasHarty\miniconda3\envs\wand\lib\site-packages\pyqtgraph\widgets\VerticalLabel.py", line 52, in paintEvent
    self.hint = p.drawText(rgn, align, self.text())
TypeError: arguments did not match any overloaded call:
  drawText(self, Union[QPointF, QPoint], str): argument 1 has unexpected type 'QRect'
  drawText(self, QRectF, int, str): argument 1 has unexpected type 'QRect'
  drawText(self, QRect, int, str): argument 2 has unexpected type 'Alignment'
  drawText(self, QRectF, str, option: QTextOption = QTextOption()): argument 1 has unexpected type 'QRect'
  drawText(self, QPoint, str): argument 1 has unexpected type 'QRect'
  drawText(self, int, int, int, int, int, str): argument 1 has unexpected type 'QRect'
  drawText(self, int, int, str): argument 1 has unexpected type 'QRect'
Traceback (most recent call last):
  File "C:\Users\ThomasHarty\miniconda3\envs\wand\lib\site-packages\pyqtgraph\widgets\VerticalLabel.py", line 52, in paintEvent
    self.hint = p.drawText(rgn, align, self.text())
TypeError: arguments did not match any overloaded call:
  drawText(self, Union[QPointF, QPoint], str): argument 1 has unexpected type 'QRect'
  drawText(self, QRectF, int, str): argument 1 has unexpected type 'QRect'
  drawText(self, QRect, int, str): argument 2 has unexpected type 'Alignment'
  drawText(self, QRectF, str, option: QTextOption = QTextOption()): argument 1 has unexpected type 'QRect'
  drawText(self, QPoint, str): argument 1 has unexpected type 'QRect'
  drawText(self, int, int, int, int, int, str): argument 1 has unexpected type 'QRect'
  drawText(self, int, int, str): argument 1 has unexpected type 'QRect'
Traceback (most recent call last):
  File "C:\Users\ThomasHarty\miniconda3\envs\wand\lib\site-packages\pyqtgraph\widgets\VerticalLabel.py", line 52, in paintEvent
    self.hint = p.drawText(rgn, align, self.text())
TypeError: arguments did not match any overloaded call:
  drawText(self, Union[QPointF, QPoint], str): argument 1 has unexpected type 'QRect'
  drawText(self, QRectF, int, str): argument 1 has unexpected type 'QRect'
  drawText(self, QRect, int, str): argument 2 has unexpected type 'Alignment'
  drawText(self, QRectF, str, option: QTextOption = QTextOption()): argument 1 has unexpected type 'QRect'
  drawText(self, QPoint, str): argument 1 has unexpected type 'QRect'
  drawText(self, int, int, int, int, int, str): argument 1 has unexpected type 'QRect'
  drawText(self, int, int, str): argument 1 has unexpected type 'QRect'
Traceback (most recent call last):
  File "C:\Users\ThomasHarty\miniconda3\envs\wand\lib\site-packages\pyqtgraph\widgets\VerticalLabel.py", line 52, in paintEvent
    self.hint = p.drawText(rgn, align, self.text())
TypeError: arguments did not match any overloaded call:
  drawText(self, Union[QPointF, QPoint], str): argument 1 has unexpected type 'QRect'
  drawText(self, QRectF, int, str): argument 1 has unexpected type 'QRect'
  drawText(self, QRect, int, str): argument 2 has unexpected type 'Alignment'
  drawText(self, QRectF, str, option: QTextOption = QTextOption()): argument 1 has unexpected type 'QRect'
  drawText(self, QPoint, str): argument 1 has unexpected type 'QRect'
  drawText(self, int, int, int, int, int, str): argument 1 has unexpected type 'QRect'
  drawText(self, int, int, str): argument 1 has unexpected type 'QRect'
Traceback (most recent call last):
  File "C:\Users\ThomasHarty\miniconda3\envs\wand\lib\site-packages\pyqtgraph\widgets\VerticalLabel.py", line 52, in paintEvent
    self.hint = p.drawText(rgn, align, self.text())
TypeError: arguments did not match any overloaded call:
  drawText(self, Union[QPointF, QPoint], str): argument 1 has unexpected type 'QRect'
  drawText(self, QRectF, int, str): argument 1 has unexpected type 'QRect'
  drawText(self, QRect, int, str): argument 2 has unexpected type 'Alignment'
  drawText(self, QRectF, str, option: QTextOption = QTextOption()): argument 1 has unexpected type 'QRect'
  drawText(self, QPoint, str): argument 1 has unexpected type 'QRect'
  drawText(self, int, int, int, int, int, str): argument 1 has unexpected type 'QRect'
  drawText(self, int, int, str): argument 1 has unexpected type 'QRect'
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
QBackingStore::endPaint() called with active painter; did you forget to destroy it or call QPainter::end() on it?
 See usage in https://doc.qt.io/qt-5/qpainter.html#drawText
Typing error? Something to do with a recent change to pyqtgraph?

here is a conda environment which doesn't have this problem
#
# Name                    Version                   Build  Channel
blas                      1.0                         mkl
ca-certificates           2020.12.8            haa95532_0
certifi                   2020.12.5        py38haa95532_0
icc_rt                    2019.0.0             h0cc432a_1
icu                       58.2                 ha925a31_3
intel-openmp              2020.2                      254
jpeg                      9b                   hb83a4c4_2
libpng                    1.6.37               h2a8f88b_0
mkl                       2020.2                      256
mkl-service               2.3.0            py38h196d8e1_0
mkl_fft                   1.2.0            py38h45dec08_0
mkl_random                1.1.1            py38h47e9c7a_0
numpy                     1.19.2           py38hadc3359_0
numpy-base                1.19.2           py38ha3acd2a_0
openssl                   1.1.1i               h2bbff1b_0
pip                       20.3.3           py38haa95532_0
pyqt                      5.9.2            py38ha925a31_4
pyqtgraph                 0.11.0                     py_0
python                    3.8.5                h5fd99cc_1
qasync                    None            py38_0+gitce95ce9    https://condafeedwin.azurewebsites.net/xVVr5vkQoSF0U4GJSK5e
qt                        5.9.7            vc14h73c81de_0
scipy                     1.5.2            py38h14eb087_0
setuptools                51.1.2           py38haa95532_4
sip                       4.19.13          py38ha925a31_0
sipyco                    1.1             py38_21+gitbcc11ad    https://condafeedwin.azurewebsites.net/xVVr5vkQoSF0U4GJSK5e
six                       1.15.0           py38haa95532_0
sqlite                    3.33.0               h2a8f88b_0
vc                        14.2                 h21ff451_1
vs2015_runtime            14.27.29016          h5e58377_2
wand-client               v1.0            py38_130+gitb8aeb02    https://condafeedwin.azurewebsites.net/xVVr5vkQoSF0U4GJSK5e
wheel                     0.36.2             pyhd3eb1b0_0
wincertstore              0.2                      py38_0
zlib                      1.2.11               h62dcd97_4

seems like putting python to 3.85 fixes it.

See https://github.com/pyqtgraph/pyqtgraph#qt-bindings-test-matrix

Not reproduced in recent memory, and potentially seems related to pyqtgraph version issues. Closing purely for triage reasons.


PowerToys Run shows not all files are indexed despite windows shows all files are indexed

Microsoft PowerToys version
0.63.0
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce

✔ Expected Behavior
Search should be fully indexed
❌ Actual Behavior
Still got the error message
Other Software
No response

I have the same issue. But I think the real issue here is on Windows. Even though I have selected "Enhanced" in Settings, windows still doesn't index everything. In settings app I only see 91k files being indexed; I have 3 drives, besides my boot drive so 91k files is abysmal. One of them drives contain 161k files

If there's additional troubleshooting or resources about this issue I'd be more than thankful on how to solve my issue.

/dup https://github.com/microsoft/PowerToys/issues/13365


Still support extensions with vendor dir

How to categorize this PR?

/kind bug
What this PR does / why we need it:
While vendoring<EMAIL_ADDRESS>in the registry-cache extension I hit the following issues related to https://github.com/gardener/gardener/issues/8775:

hack/check-generate.sh script requires the target make tidy instead of make revendor. This PR adapts the logic to check for the vendor dir - if it exists - use make revendor, otherwise - make tidy
hack/check-skaffold-deps-for-binary.sh script now longer adds the vendor dir and this breaks extensions using this script. See https://github.com/gardener/gardener-extension-shoot-rsyslog-relp/pull/37#discussion_r1401030446. This PR adapts the logic to check for the vendor dir - if it exists add also vendor to the to the list of actual dependencies.

Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
N/A
Release note:

The `hack/check-skaffold-deps-for-binary.sh` and `hack/check-generate.sh` scripts are adapted to support also extensions that have a vendor dir.

/lgtm

/approve

/cherry-pick release-v1.84

/test pull-gardener-e2e-kind


[SMALLFIX] Fix mount table resolution when the root path is omitted

When the path is omitted such as in hdfs://localhost, the getPath method returns emtpy string, which in turn would fail the AlluxioURI construction with Path  is invalid exception.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17590/Build result: FAILURE[...truncated 2172 lines...][JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/keyvalue/common/target/alluxio-keyvalue-common-1.7.0-SNAPSHOT.jar to org.alluxio/alluxio-keyvalue-common/1.7.0-SNAPSHOT/alluxio-keyvalue-common-1.7.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/keyvalue/common/target/alluxio-keyvalue-common-1.7.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-keyvalue-common/1.7.0-SNAPSHOT/alluxio-keyvalue-common-1.7.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/fs/pom.xml to org.alluxio/alluxio-core-client-fs/1.7.0-SNAPSHOT/alluxio-core-client-fs-1.7.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/fs/target/alluxio-core-client-fs-1.7.0-SNAPSHOT.jar to org.alluxio/alluxio-core-client-fs/1.7.0-SNAPSHOT/alluxio-core-client-fs-1.7.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/fs/target/alluxio-core-client-fs-1.7.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-client-fs/1.7.0-SNAPSHOT/alluxio-core-client-fs-1.7.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/fs/target/alluxio-core-client-fs-1.7.0-SNAPSHOT-tests.jar to org.alluxio/alluxio-core-client-fs/1.7.0-SNAPSHOT/alluxio-core-client-fs-1.7.0-SNAPSHOT-tests.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/pom.xml to org.alluxio/alluxio-core-common/1.7.0-SNAPSHOT/alluxio-core-common-1.7.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.7.0-SNAPSHOT.jar to org.alluxio/alluxio-core-common/1.7.0-SNAPSHOT/alluxio-core-common-1.7.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.7.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-common/1.7.0-SNAPSHOT/alluxio-core-common-1.7.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.7.0-SNAPSHOT-tests.jar to org.alluxio/alluxio-core-common/1.7.0-SNAPSHOT/alluxio-core-common-1.7.0-SNAPSHOT-tests.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/worker/pom.xml to org.alluxio/alluxio-core-server-worker/1.7.0-SNAPSHOT/alluxio-core-server-worker-1.7.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/logserver/pom.xml to org.alluxio/alluxio-logserver/1.7.0-SNAPSHOT/alluxio-logserver-1.7.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/client/pom.xml to org.alluxio/alluxio-core-client/1.7.0-SNAPSHOT/alluxio-core-client-1.7.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/pom.xml to org.alluxio/alluxio-examples/1.7.0-SNAPSHOT/alluxio-examples-1.7.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.7.0-SNAPSHOT.jar to org.alluxio/alluxio-examples/1.7.0-SNAPSHOT/alluxio-examples-1.7.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/target/alluxio-examples-1.7.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-examples/1.7.0-SNAPSHOT/alluxio-examples-1.7.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/pom.xml to org.alluxio/alluxio-core/1.7.0-SNAPSHOT/alluxio-core-1.7.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/pom.xml to org.alluxio/alluxio-core-server-proxy/1.7.0-SNAPSHOT/alluxio-core-server-proxy-1.7.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/pom.xml to org.alluxio/alluxio-shell/1.7.0-SNAPSHOT/alluxio-shell-1.7.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.7.0-SNAPSHOT.jar to org.alluxio/alluxio-shell/1.7.0-SNAPSHOT/alluxio-shell-1.7.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/target/alluxio-shell-1.7.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-shell/1.7.0-SNAPSHOT/alluxio-shell-1.7.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/tests/pom.xml to org.alluxio/alluxio-tests/1.7.0-SNAPSHOT/alluxio-tests-1.7.0-SNAPSHOT.pomchannel stoppedArchiving artifacts
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17591/
Test PASSed.

Merged build finished. Test PASSed.

@gpang I didn't make it clear. "" is a valid AlluxioURI, however, the failure is on PathUtils.hasPrefix  which does not take empty string as input.

@yupeng9 , I think @gpang means the same suggestion as mine: making the change in AlluxioUri rather than the mount table.

@gpang what do you think?

@gpang any idea?