Add flag to disable puppeteer sandbox

I understand that this may not be something you'd like to support, but it's useful to me and I figured I might as well submit a pull request.
And apologies for the large diff, prettier did its work it looks like.

Sorry for the delay and thanks for this! No worries about the prettier changes, this LGTM!


Drop Ubuntu 16.04 and 18.04 support

includes: #150

hmm, I am not really a fan of deleting the check. On the other hand: it only ever ran on ubuntu18.04 anyways. At the same time: I think the reload command for non systemd OSes is broken since forever (or relaod is somehow not a typo)


💀

Just update it every now and then 👍🏼

I just made a GitHub account and put the webosu files on here.
webosu.online is blocked at my school, so I just made this so I could play it.
Haven't updated it in a bit. But thanks.


chore: Removing partner program callout from invite modal

Description
Removing partner program callout from invite modal as we are rolling back the experiment to promote our partner program when user with a different domain was invited to an application.
PR fixes following issue(s)
Fixes #29813
Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing
How Has This Been Tested?

[x] Manual
[ ] JUnit
[ ] Jest
[x] Cypress

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Summary by CodeRabbit

Refactor

Simplified the Invite Users Form by removing unused references and streamlining state management.
Enhanced the Action Entity Context Menu by conditionally adding options based on plugin type.

Tests

Removed a specific regression test related to partner program callouts for users from different domains.

Chores

Cleaned up redundant functions and logic related to partner program callouts in workspace invitation processes.

/ok-to-test<EMAIL_ADDRESS>


Can not reptoduce the face and eye movement.

Hello.
I am trying to reproduce the character's face as well as eye movements in the demo video you provided. Since the female hair in that video is floating all the time, I assumed that it was generated from a single still image. So I generated a similar image and expanded it to 240 batches and generated it based on the image examples in the repository. However, the generated video only has the mouth movement, not the character's face and eyes following the speech as in the demo video.
How can I achieve the effect of the demo video?
Thank you for your help.

@OrvilleQ we are adding a new node that is based on SadTalker and you can generate video from an image, however, it only works with close-up faces, the node will be added shortly


With which manim version this addon works ?

I tried this addon on the last version of Manim following what it was wrote in the readme file and I got this error : manim.py: error: unrecognized arguments: --save_to_pptx.

This repo is an add-on, so it requires a fork of Manim that has the add-on system. At the moment, the only one that has it is my fork at https://github.com/yoshiask/manim, but Manim Community will have it too.

Thank you for your answer and thank you for this very useful add-on


V 2.0

I guess is time to write v 2.0 of the boilerplate. We should think about what to keep, refactor or add.
This are the things I would like in this version:

Setup script. To setup repo and install dependencies
Refactor grid, both fluid and fixed widths, in a way we all can agree to and also as a mixin. I advocate for the use of flex. I believe we can make a flex grid which work for fluid and fixed width.
Better package management, for plugins and dependencies
Better utility classes, as mixins, for margins, borders and stuff
Better site options page, with options to add more than one options page
ES6 support.
Test page, a plain HTML page that has examples of code that display grid, margins, utility classes, etc by examples. This will be useful to test changes and improvements to the boilerplate. It works as well as documentation.

I think we should also consider on looking at frameworks like Bootstrap, Foundation or Semantic to learn how they manage their code, media-queries, best practices, etc.
If you have more ideas comment them, I'll add them to my list.

I 100% agree with all of this. Also to be explicit we should move to composer for php package management. And possibly bower or similar for js dependencies

Yeah i think we should incorporate bower with gulp to generate our library.js file: https://github.com/ck86/main-bower-files

Also we should call the project and repo something better

@varffs Composer seems perfect for CMB2 intergation.
I don't understand what could we use bower for. What packages have we use/could use from bower in our projects?

js libs like Swiper, ImagesLoaded, Masonry etc

ohh true, we use those a lot. good point.

I'm wondering if we could have a cheat sheet(s) for common code that we often use in projects but not nec every project. For example when I use Moment PHP for localized date formatting setting the locale needs to be done and that code for this is always the same. Same for other qtranslate-x snippets. Etc. We could have a folder of these cheat sheets in the boiler (we could also do them in another repo but makes sense here to me)

make this into milestone. add issues for things outstanding and assign them


Deleting a product in Unified View selects the next one

https://stackable-workspace.slack.com/archives/C02R24UJBV3/p1647860808801859


Use SPDX identifier in license field of META.info

Use the standardized identifier for the license field.
For more details see https://design.perl6.org/S22.html#license

Awesome, thanks!


Why is wmain shown without a return type?

The declaration for wmain is shown as:
wmain( int argc, wchar_t *argv[ ], wchar_t *envp[ ] )

This relies on the obsolete "implicit int" feature, which has been invalid in C since the 1999 standard and in C++ for even longer.
The correct version:
int wmain( int argc, wchar_t *argv[ ], wchar_t *envp[ ] )

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

ID: 26b1131c-87d8-3199-9275-b3fe094aba40
Version Independent ID: 473540ca-baa7-d6bb-8f3c-327a29ef3804
Content: Using wmain
Content Source: docs/c-language/using-wmain.md
Product: visual-cpp
Technology: cpp-c-language
GitHub Login: @corob-msft
Microsoft Alias: corob

The same problem occurs in docs/c-language/main-function-and-program-execution.md.

@Keith-S-Thompson
Thanks for the report. These articles may be some of the oldest in our C-language content. They almost certainly predate the change from default int return types. A fix is ready to publish later today.


Upgrade to lit 2 0

@olange Please review and test this upgrade to lit 2.0.2. It builds, starts and runs  successfully; I did not try to deploy it.
I followed steps suggested in Lit Upgrade guide:

npm uninstall lit-element lit-html; npm install lit;  errors!
change code to import { LitElement, html, css } from 'lit'; still errors
replace @internalProperty by @state, replace import { internalProperty } by import { state }: it builds, starts and works

Here is  git diff main > diff-from-main.txt                                                                                [upgrade-to-lit-2-0|✔]
diff-from-main.txt


Add specs for optparse into argument

optparse
Add an into option. Feature #11191
Relates issue https://github.com/ruby/spec/issues/473

@eregon There are no specs for OptionParser

Right, the coverage of the stdlib is usually lower.
For pure-Ruby stdlib with their own tests and now their own gem/project in 2.4/2.5, I think it's less a priority to cover them than language and core.

Thank you for the PR!
I'll fix the small typos in the descriptions.


Provider Standards

I wanted to write a provider for my country (Nicaragua). After going over a few of the ones already defined; I noticed there are no standards. Many properties were not present in other providers. Is there an initiative in regards to this matter of inconsistency? if not I would love to hear your thoughts on this. :)

I think we should add interfaces in order to standardize providers

There is no initiative, and I don't think it's necessary. It's already hard enough to find contributors willing to add providers for a certain locale, so it'd be counterproductive to refuse PRs because they don't include all required providers.
Thanks for your suggestion anyway. And I'm looking forward to your Nicaraguayan provider!

@fzaninotto Thank you for your quick response. I really love this library its extremely useful but Its very sad to have such level of inconsistency. I strongly believe that adding interfaces would make it easier for future contributor. I'll love to give it try before just saying no to this idea :)


[FR] operator bool

wrapper around has_value
ps. don't know if you get notifications for new comments in closed issues: https://github.com/microsoft/proxy/issues/61

While std::any does not have operator bool, proxy is not sufficiently motivated to do so. Besides, since smart pointer usually have operator bool, if assign a default-constructed std::unique_ptr to proxy, proxy::operator bool will become ambiguous.


3.1 - ggplot

(I swapped Day with Day 4 to make our lives easier)
@monacoba - Ready for primetime: https://tidy-mn.github.io/R-camp-slides/day3/03-1-Plots-graphics-grammar.html#1
Some penguin observations had their island and species info lost to the elements. We know the observations were from an excursion to a single island, but which one? And what species are they? Plots to the rescue!

The gg  in ggplot stands for Grammar of graphics.
The first building block is the plot canvas, and  it is the foundation for all the other plot pieces.

Not sure if we need more for a live demo here. Up next the 3.2 slides introduce facets and labs() and legends.  Maybe we save the demo portion for after that?

Complete


Attempt to further reduce re-parsing for globally cached images (PR 11912, 16108 follow-up)

In PR #11912 we started caching images that occur on multiple pages globally, which improved performance a lot in many PDF documents.
However, one slightly annoying limitation of the implementation is the need to re-parse the image once the global-caching threshold has been reached. Previously this was difficult to avoid, since large image-resources will cause cleanup to run on the main-thread after rendering has finished. In PR #16108 we started delaying this cleanup a little bit, to improve performance if a user e.g. zooms and/or rotates the document immediately after rendering completes.
Taking those two PRs together, we now have a situation where it's much more likely that the main-thread has "globally used" images cached at the page-level. Hence we can instead attempt to copy a locally cached image into the global object-cache on the main-thread and thus reduce unnecessary re-parsing of large/complex global images, which significantly reduces the rendering time in many cases.
For the PDF document in issue #11878, the rendering time of the second page changes as follows (on my computer):

With the master-branch it takes >600 ms to render.
With this patch that goes down to ~50 ms, which is one order of magnitude faster.

(Note that all other pages are, as expected, completely unaffected by these changes.)
This new main-thread copying is limited to "large" global images, since:

Re-parsing of small images, on the worker-thread, is usually fast enough to not be an issue.
With the delayed cleanup after rendering, it's still not guaranteed that an image is available in a page-level cache on the main-thread.
This forces the worker-thread to wait for the main-thread, which is a pattern that you always want to avoid unless absolutely necessary.

/botio test

The pdf in #11878 is probably a bit unusual, I mean I'm not sure (it's just an opinion) that there are so many pdfs in the wild having a common big image on several pages.
Do you know if there are some improvements in other context (e.g. when rotating ? zooming ?...) ? or with more usual pdfs ?
Do you know if there are some possibles regressions (possibly due to the added sync point in the worker) ?

The pdf in #11878 is probably a bit unusual, I mean I'm not sure (it's just an opinion) that there are so many pdfs in the wild having a common big image on several pages.

While it's difficult for me to say how common this is, we've seen a number of cases (in both GitHub and Bugzilla) where the same large image is used as the background on all pages; note the issues/bugs listed at the end of https://github.com/mozilla/pdf.js/pull/11912#issue-620164894 and those are the ones I knew about three years ago.
Hence I'd say that a global image-cache definitely helps, to both reduce CPU and memory usage, and this patch simply improves things a little bit further.

Do you know if there are some improvements in other context (e.g. when rotating ? zooming ?...) ? or with more usual pdfs ?

This patch won't really affect those cases, as far as I can tell.

Do you know if there are some possibles regressions (possibly due to the added sync point in the worker) ?

There shouldn't be, since only images that we've previously seen at least once on another page will trigger this new code-path.

Just a side note, wdyt about using Map objects for #objs and similar ?

Just a side note, wdyt about using Map objects for #objs and similar ?

I've also been thinking about that, but as far as I know Map (and Set) is still slower than an Object; see https://bugzilla.mozilla.org/show_bug.cgi?id=1799154

/botio test

/botio-windows test


Merge pull request #1464 from bgoonz/preview

Skip to content

Pull requestsIssues
Marketplace
Explore

bgoonz/BGOONZ_BLOG_2.0

Unpin

Unwatch

Fork

Starred 11

Code

Issues6

Pull requests1

Discussions

Actions

Projects1

Wiki64

Settings

Releases2

More
Edit Code
Gitpod
Merge pull request #1464 from bgoonz/preview #1465
Open
preview  master
Open
Merge pull request #1464 from bgoonz/preview#1465
bgoonz wants to merge 1 commit into preview from master
+0 -0
 Conversation 0 Commits 1 Checks 27 Files changed 0
Conversation

Owner
bgoonz now
|
No description provided.
|

[Merge pull request](https://github.com/bgoonz/BGOONZ_BLOG_2.0/pull/1465/commits/a653462fa8206b6b7692bc0b2676a90c252dcbe8 "Merge pull request #1464 from bgoonz/preview") [#1464](https://github.com/bgoonz/BGOONZ_BLOG_2.0/pull/1464) [from bgoonz/preview](https://github.com/bgoonz/BGOONZ_BLOG_2.0/pull/1465/commits/a653462fa8206b6b7692bc0b2676a90c252dcbe8 "Merge pull request #1464 from bgoonz/preview")
Verified
[a653462](https://github.com/bgoonz/BGOONZ_BLOG_2.0/pull/1465/commits/a653462fa8206b6b7692bc0b2676a90c252dcbe8)

vercel bot commented on [a653462](https://github.com/bgoonz/BGOONZ_BLOG_2.0/commit/a653462fa8206b6b7692bc0b2676a90c252dcbe8) 3 minutes ago
Successfully deployed to the following URLs:

bgoonz-blog-2-0-bgoonz.vercel.app
bgoonz-blog-2-0-git-master-bgoonz.vercel.app
bgoonz-blog-2-0.vercel.app

vercel bot commented now
|
This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.
🔍 Inspect: vercel.com/bgoonz/bgoonz-blog-2-0/A52JoZy6aYeNb5MRhYBWnkqQ83ZT
✅ Preview: bgoonz-blog-2-0-bgoonz.vercel.app
|

viezly bot commented now
|
Empty pull request. No need to analyze
|
Merge state
Add more commits by pushing to the [master](https://github.com/bgoonz/BGOONZ_BLOG_2.0) branch on bgoonz/BGOONZ_BLOG_2.0.
Show environments
This branch was successfully deployed
2 active deployments
github-pages--- a653462f Deployed 12 minutes ago by github-pages[bot] via deploy #164
View deployment
Production--- a653462f Deployed 11 minutes ago by vercel[bot]
View deployment
Hide all checks3 failing, 1 queued, 24 successful, 2 in progress, 1 pending, and 1 neutral checks

Tests: release process / release (push) Failing after 2m --- release
Details

Tests: release process / release (pull_request) Queued --- Waiting to run this check...
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (11) (push) Successful in 28s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (10) (push) Successful in 19s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9) (push) Successful in 27s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9.2) (push) Successful in 23s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9.1) (push) Successful in 16s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (9.0) (push) Successful in 16s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (8) (push) Successful in 18s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (7) (push) Successful in 21s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (6) (push) Successful in 21s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (6.1) (push) Successful in 26s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (5) (push) Successful in 24s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (5.9) (push) Successful in 36s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (4) (push) Successful in 21s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (4.6) (push) Successful in 27s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (4.5) (push) Successful in 23s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (4.4) (push) Successful in 23s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (3) (push) Successful in 27s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (2) (push) Successful in 20s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (1) (push) Successful in 21s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (0.12) (push) Successful in 26s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (0.10) (push) Successful in 24s
Details

Tests: nvm install-latest-npm / nvm install-latest-npm (push)
Details

Build<PHONE_NUMBER>98) --- Summary
Details

Deadpendency Check Failing after 7s --- Deadpendency Check Result
Details

Cloudflare Pages In progress --- Building
Details

LGTM analysis: JavaScript In progress --- Fetching git commits
Details

docs/readthedocs.org:bgoonzblog2 Pending --- Read the Docs build is in progress!
Details

Code scanning results / CodeQL --- No analyses found
Details

CommitCheck --- All commit messages are valid!
Details

Vercel --- Deployment has completed
Details
This branch has no conflicts with the base branch
Merge pull request
Convert to draft or view command line instructions.

Write Preview
Close pull request
Comment
Remember, contributions to this repository should follow its contributing guidelines and code of conduct.
ProTip! Add comments to specific lines under Files changed.
Reviewers -- review now
Still in progress? Convert to draft
Assignees -- assign yourself
Labels
Projects
Milestone
Linked issues
Notifications
Customize
Unsubscribe
You're receiving notifications because you're watching this repository.
1 participant

Lock conversation

© 2022 GitHub, Inc.

Terms

Privacy

Security

Status

Docs

Contact GitHub

Pricing

API

Training

Blog

About

viezly bot commented now Empty pull request. No need to analyze


one descriptorPool per DescriptorSet cause high cpu and gpu memory consumption in version 1.51

after i update to v1.51,  the memory consumption of cpu and gpu become incredible high, and vkCreateDescriptorPool returns VK_ERROR_OUT_OF_DEVICE_MEMORY
and i found the difference here:
https://github.com/ConfettiFX/The-Forge/blob/da36bb40f6ea11df76167f528032c52999912d82/Common_3/Renderer/Vulkan/Vulkan.cpp?_pjax=%23js-repo-pjax-container%2C div[itemtype%3D"http%3A%2F%2Fschema.org%2FSoftwareSourceCode"] main%2C [data-pjax-container]#L4892
am i missing somthing?

How many DescriptorSet structs do you have?

about 20k DescriptorSet  used for per draw word matrix and texture params, i use the-forge for a CAD program, 20k drawcalls is very common.
when use version 1.5 there seems no problem. but now the 8G gpu shared memory can be used up and vkCreateDescriptorPool return VK_ERROR_OUT_OF_DEVICE_MEMORY
i know i can use a storge buffer for my matrix and use dynamic descripor set, but what about the texture param if bindless is not support on target device
thank you(^_^)

If all the 20k descriptor sets are using the same root signature, you can allocate just one DescriptorSet struct with mMaxSets=20k. We will take a look on our side as well if this doesn't resolve your issue

@manas-kulkarni it works !
thank you very much! your idea makes gpu memory usage even lower than version 1.5, now it only takes  about 100M shared gpu memory. i think i shoud matain a descriptor pool list per root signature and create new pool on demand.
and here:
https://github.com/ConfettiFX/The-Forge/blob/master/Common_3/Renderer/Vulkan/Vulkan.cpp#L4877
i use tf_malloc instead alloca in case of stack overflow when mMaxSets is too big
thank you again! and merry christmas !

Glad that it worked. We will fix that alloca call in next release. Happy Holidays!


feat(iroh): use reflink to avoid copying files into/out of the flat store

Description
By using reflink, we can avoid having to copy files into/out of the flat store. For file systems that support COW, this should be basically instantaneous.
The reflink crate provides a function that automatically falls back to copying.
Notes & open questions
Does this cause trouble on any of the more obscure platforms, or is the fallback safe?
Change checklist

[x] Self-review.

Implements https://github.com/n0-computer/iroh/issues/1316

sigh... seems like none of the crates work well for some targets...

Asked the crate author of the forked crate in https://github.com/cargo-bins/reflink-copy/issues/16

@rklaehn can we close this for now?


Chumsky parser

Use the chumsky library to parse justfiles. Cf. https://github.com/casey/just/issues/1324

I took another look at this, and I think that although a structured approach has advantages, overall I think I like the current parser.


feat: add deletion feature in cfp

Related Issue

Information about the related issue

Closes: #558
Describe the changes you've made
Added deletion feature in cfp and add a confirmation box, so after confirming there one can delete a cfp
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update.
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented on my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)

Original
Updated

original screenshot
updated screenshot

Code of Conduct

[x] I agree to follow this project's Code of Conduct

@Deepu178 I have resolved the merge conflict


scipy error: undefined symbol: sgegv_

/usr/local/lib/python2.7/dist-packages/scipy/linalg/lapack.py in <module>()
    356 from .blas import find_best_blas_type as find_best_lapack_type
    357
--> 358 from scipy.linalg import _flapack
    359 try:
    360     from scipy.linalg import _clapack

ImportError: /usr/local/lib/python2.7/dist-packages/scipy/linalg/_flapack.so: undefined symbol: sgegv_

I have reinstalled scipy,numpy,thenao,lapack,atlas.. for many times...Anyone could help me?

I have the same issue...

me too.. anyone can help?

Ok, I solved it. I used apt-get install scipy and it work...

This is just a scipy install error. I suggest using Conda, which is also what we use for Travis CI in Keras. https://github.com/fchollet/keras/blob/master/.travis.yml
It will take care of setting up scipy for you.


[TechDocs HomePage] Create Docs Component Button

This is a sub-issue out of #2028.
On the TechDocs HomePage, a "Create Docs Component" Button should exist that takes a user to the default docs-template (/create/docs-template) or let a Backstage integrator decide where the button should take the user.

@emkidwell Could you assign me this issue, right now I have college exams but I will contribute to this in a few days after the end of my exams

@emkidwell Could you assign me this issue, right now I have college exams but I will contribute to this in a few days after the end of my exams

Sure, done! :D

Sure, done! :D

I have fixed the URL, but can you explain what that latter part of the description means so I could contribute to that too if required

Based on the discussions in the sample PR, I might suggest implementing this be broken up into two:

[ ] Add a create docs button to the docs homepage which just takes the user to create components (whether or not it shows only docs)
[ ] Design and figure out a way to identify "the docs template(s)" for a given instance and link directly to the setup page for that given template

One might be easier done than the other, and an incremental improvement towards @emmaindal 's outcome?

@OrkoHunter @emmaindal Will backstage be there in GSoC'21 ?
I'm trying for GSoC'21, could you please confirm ?

Good luck @Anshultilondiya with GSoC and thanks for showing interest! Backstage has not applied as a GSoC '21 organization.

Well I liked the project, I'm thinking to contribute as soon as I get better understanding, the demo video was really helpful in encouraging my interest in backstage 😅

Hi, reviving this a bit, I may be able to contribute here. What I have right now is simply duplicating the logic that the catalog page uses to produce this button, and applying a ?filters[type][0]=documentation to the end of the link. This is pretty brittle and not really extensible in the way one might like, given "Link to set of templates" seems like it would be commonly used.
I'd propose an additional property on the route refs to indicate query parameters. This would allow integrators full control over what the docs button links to while having the ability to provide a sane default. If this already exists in the routing system, let me know :)


Consider using cookstyle instead of rubocop in lint.rb

I could happily submit the PR to make this change, but wanted to bring it up for discussion.
The current chef generate in chefdk results in a cookbook that fails rubocop (and therefore these lint tests). Is there any reason to not switch to cookstyle for the lint tests?

I second the idea of switching to cookstyle. If cookstyle is the preferred method of linting cookbooks and is the defacto standard, why wouldn't it be the standard in delivery? (or anywhere else for that matter)

I'm working on a PR for this, and wonder about two possible implementations (I don't think we want to just immediately cut over, because it's possible that someone has older builders that don't have cookstyle installed)
These are the two options as I see them:

If cookstyle is installed, run cookstyle instead of rubocop. Period.
Set an option in config.json to over-ride and enable cookstyle instead of rubocop, which makes using cookstyle an opt-in.

I am more in favor of option 1, as it should be the default. Perhaps actually there is a Secret Option 3, which would be this:
If cookstyle is installed, run it. Unless enable_rubocop is set in the config.json. That way if someone really wants to use rubocop they can. This will also not break things for older builders.

It looks like cookstyle was added to ChefDK in v0.14 (2016-05-17).
Since that was quite some time ago, I think that it's fairly safe to go with option 1. I have a fork where if cookstyle is installed, it will run cookstyle tests in lint. If not, it will run rubocop.
I have not been able to yet test it on a builder with an older version of ChefDK that does not have cookstyle though.


Fixes create project error with chinese #664

Fixes https://github.com/azkaban/azkaban/issues/664
Add error output when create project failed.
Add create table with utf8

@EdwardsBean
What tests have you done?
What happens if the table was created using the original default character set ( Is it Latin1?) and they run this code to read from that table?
Do other tables suffer from the same issue?
How about project descriptions?
Could you include unit tests?

@HappyRay
Unit tests can reuse azkaban-common/../JdbcProjectLoaderTest
If the table was created using the original default character set (Latin1) and they can run this code to read from that table
When use input content contains none latin support character, mysql  will not allow to insert(SQLException).
Right now only project name and project descriptions suffer from the issue.

@EdwardsBean are you referring to
azkaban.project.JdbcProjectLoaderTest#testProjectRetrievalByFetchProjectByName ? It doesn't test non-ascII characters, does it?
The test is currently skipped for me because it requires a test DB to be setup. But the instruction is not clear how it should be set up. As the result, I don't think it runs on the CI system automatically.
===
According to
http://stackoverflow.com/questions/7048745/what-is-the-difference-between-utf-8-and-iso-8859-1
Only the ASCII characters are compatible between these two encodings.
If so, there can be problems reading tables created with latin1.
A DB update may be needed.

@HappyRay
You are right.
When project name or project description contains non ASCII characters(project table with latin1), in azkaban project dashboard show messy code of non ASCII characters.
Changing project table to UTF8 is really necessary for those languages none western world.

Hi ,
I want to use Chinese words in the job files , but I found that after inserting into the mysql db , the content of the job become messy, in table project_files and project_properties. How can I solve it ?
Thanks!

Sorry about the long delay.
@EdwardsBean  Are you still interested in working on this PR?
If you can merge in the latest code, I can consider merging it now that we have functional DB unit tests using in memory H2 DB.
@kunkun-tang @li-afaris  what do you think?

Following up from @HappyRay's comment- @EdwardsBean do you still want to merge this? Please rebase on latest code.


Use fast eval_at_point

This change is

#536  👈
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @spapinistarkware and the rest of your teammates on  Graphite


data augmentation and normalization

Thank you for sharing your method, it's great! I have some questions about data augmentation and normalization.

I notice that local voxel coordinates and reflectivities of points are usually smaller than 1 after building the point features of KITTI, while the raw x and y of the point cloud range from 0 to 80. Is it necessary to do some data normalization, like z-score normalization, before feeding the features to the network?
About data augmentation, except the rotation, scale, flip, is there any useful augmentation method that haven't been mentioned?

Since the network uses a stack of MLPs to process, the BN layer in it can already do the same thing. You can also perform the normalization on the input features to check its effect.
Instance augmentation is also a popular strategy which is proposed by Panoptic-PolarNet (CVPR 2021).

Thank you for your explanation!


Prepare for Gradle plugin release 3.1.1

#177  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rbro112 and the rest of your teammates on  Graphite


added a home section in all page

hey @Anushkabh i added a home section in all the page form which they can directly goes to home page . Please accept the Pr and level it

issue number?

#287

@Anushkabh  please merge it and label it

resolve the conflicts by pulling the latest code

hey @Anushkabh  please merge it i resolved

resolve the conflicts

Resolve the conflict and create a new PR


Fix for #114

The code looks good, we can merge once CI is happy

@nickjbrowning @Luthaf what's the status of this?

This was waiting on CI to be fixed, after what I can do a quick review and merge!

should be good for review @Luthaf


Slow initial load

Describe the bug

At the time of writing, latest version is v1.44.0
The initial load of a skeleton React app using the router is perceivably slow (~500ms from React profiler)
v1.28.1 is the latest good version (good as in not perceivably slow) which suggests that v1.28.2 introduced a regression

Your Example Website or App
https://stackblitz.com/edit/tanstack-router-vjwuwc?file=index.html,package.json,package-lock.json,src%2Fmain.tsx,vite.config.js
Steps to Reproduce the Bug or Issue

Bad: https://stackblitz.com/edit/tanstack-router-vjwuwc?file=index.html,package.json,package-lock.json,src%2Fmain.tsx,vite.config.js
Good: https://stackblitz.com/edit/tanstack-router-u6raco?file=index.html,package.json,package-lock.json,src%2Fmain.tsx,vite.config.js

The only difference between Good and Bad is the router version.

Refresh the page on the StackBlitz browser
Notice that This sentence is from React loads instantly in the Good version, almost at the same time as This sentence is from HTML.
Notice that This sentence is from React loads after a delay in the Bad version, ~500ms after This sentence is from HTML

Expected behavior
As a user, I expected no perceivable delays on initial app load
Screenshots or Videos
No response
Platform

macOS
Chrome Version 126.0.6478.127 (Official Build) (arm64)

Additional context
No response

Wonder if this is related to https://github.com/TanStack/router/issues/1899

Thanks @ericchernuka for the pointer! Sounds like a workaround would be to set defaultPendingMinMs to 0

#1646

this will be fixed by #1907
reproducer with pre-release package: https://stackblitz.com/edit/tanstack-router-mp74sr?file=index.html,package.json,package-lock.json,src%2Fmain.tsx,vite.config.js

@schiller-manuel I'm still seeing this 500ms delay on 1.45.11

@levrik  can you please open a new issue by and provide a minimal complete example by forking one of the existing router examples on stackblitz?
e.g. https://tanstack.com/router/v1/docs/framework/react/examples/quickstart-file-based


Turn off save to disk for GPU aquaplanet DYAMOND simulation

Purpose
Turn off save to disk for GPU aquaplanet DYAMOND simulation

[x] I have read and checked the items on the review checklist.

What's the purpose of this change?

What's the purpose of this change?

The GPU pipeline is primarily for looking at the performance benchmarks. We would want to turnoff file writes for measurements! We can test IO in the general CI pipeline and/or longrun cpu/gpu pipeline!

We can have a separate GPU pipeline focussed on IO, if we would like to tune IO performance!

Implemented in #2616


Update instrumentation.md

Try to trigger rebuild; might fix https://github.com/prometheus/docs/issues/1045

Changed the first thing I could find to nitpick.

Heh, DCO is an issue with in-web changes.

Erm, wrong PR. Disregard my confusion.

DCO is fine as per human inspection.


admin.password file gets created even if password for admin is specified in Hiera

When provisioning a server, the "nexus::config::admin::password" is set in Hiera and Nexus uses it, so everything looks good.
But even when the password has a value, the file /opt/sonatype/sonatype-work/nexus3/admin.password gets created and next time Puppet runs it throws an authentication error. As per the documentation, that random password should only be created if the admin password is not defined (to my understanding).
I tried deleting the file but it gets created every time Puppet runs, also, if I try to set it to absent I get a dependency cycle error.
Please your help.

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hi @yasser583,
/opt/sonatype/sonatype-work/nexus3/admin.password is created by Nexus during the installation. It’s not part of this Puppet module. If you have defined nexus::config::admin::password properly, simply delete the file mentioned above, and you should be good to go.
Ref : https://github.com/puppets-epic-show-theatre/puppet-nexus/blob/master/lib/puppet/transport/nexus_rest_api.rb#L19

Hello @PiR-B, I mean, I delete the file but every time nexus is started (for example with systemctl start nexus), a new admin.password file gets created. I've deleted the file a lot of times but it just comes back, can't get rid of it.
Thanks again.

Hi @yasser583,
Can you please provide your nexus class definition ?
As an example, I don't have any problem with this definition running on debian 12
mod 'puppet-archive', '7.1.0'
mod 'puppet-extlib', '7.0.0'
mod 'puppetlabs-device_manager', '4.1.0'
mod 'puppetlabs-java', '11.0.0'
mod 'puppetlabs-stdlib', '9.6.0'
mod 'pest-nexus', '4.1.0'

  java::adoptium { 'jdk17' :
    ensure        => 'present',
    version_major => '17',
    version_minor => '0',
    version_patch => '12',
    version_build => '7',
  }
  -> class { 'nexus':
    version                    => '3.71.0-06',
    host                       => '<IP_ADDRESS>',
    download_site              => 'https://download.sonatype.com/nexus/3',
    purge_default_repositories => true,
    manage_datastore           => true,
  }

nexus::config::anonymous::enabled: true
nexus::manage_user: false
nexus::config::admin::first_name: Firstname
nexus::config::admin::last_name: Lastname
nexus::config::admin::password: >
  ENC[PKCS7,whatever]

After the puppet run, did you manually finished the installation thanks to the GUI ?
Regards

Hello @PiR-B, this is what I have:
Puppetfile
mod 'puppetlabs-device_manager', '4.1.0'
mod 'puppet-extlib', '7.0.0'
mod 'pest-nexus', '4.1.0'

nexus.pp
class profile::nexus (
    ....
    $nexus_server_version  = lookup('nexus_server_version', undef, undef, undef),
) {

  package { 'java-17-amazon-corretto':
    ensure => 'present',
  }

  class { 'nexus':
    version                                    => $nexus_server_version,
    purge_default_repositories  => true,
    manage_datastore                => true,
    require                                    => [Package['java-17-amazon-corretto'], Class['nginx']],
  }
}

Hieradata (don't mind the syntax since it gets converted to real hiera)
"profile::nexus::nexus_server_version": "3.71.0-06",
"nexus::config::admin::username": "admin",
"nexus::config::admin::first_name": "admin",
"nexus::config::admin::last_name": "admin",
"nexus::config::admin::email_address"<EMAIL_ADDRESS>"nexus::config::admin::password": "anypassword"

Regarding the GUI installation, I don't think Nexus asks for anything else after login.

Hi @yasser583,
It looks good. What is your OS & puppet-agent version ?
Can you please attach the output of the sudo puppet agent -t ?
Regards

OS: Amazon Linux 2
Puppet agent version: 7.21.0
puppet agent -t
Error: /Stage[main]/Nexus::Config::Admin/Nexus_user[admin]: Could not evaluate: authentication error
Notice: /Stage[main]/Nexus::Config::Anonymous/Nexus_setting[security/anonymous]: Dependency Nexus_user[admin] has failures: true
Warning: /Stage[main]/Nexus::Config::Anonymous/Nexus_setting[security/anonymous]: Skipping because of failed dependencies
Warning: /Stage[main]/Nexus::Config::Email/Nexus_setting[email]: Skipping because of failed dependencies
Warning: /Stage[main]/Nexus::Config::Default_repositories/Nexus_repository[maven-central]: Skipping because of failed dependencies
Warning: /Stage[main]/Nexus::Config::Default_repositories/Nexus_repository[maven-releases]: Skipping because of failed dependencies
Warning: /Stage[main]/Nexus::Config::Default_repositories/Nexus_repository[maven-public]: Skipping because of failed dependencies
Warning: /Stage[main]/Nexus::Config::Default_repositories/Nexus_repository[maven-snapshots]: Skipping because of failed dependencies
Warning: /Stage[main]/Nexus::Config::Default_repositories/Nexus_repository[nuget-group]: Skipping because of failed dependencies
Warning: /Stage[main]/Nexus::Config::Default_repositories/Nexus_repository[nuget-hosted]: Skipping because of failed dependencies
Warning: /Stage[main]/Nexus::Config::Default_repositories/Nexus_repository[nuget.org-proxy]: Skipping because of failed dependencies
Notice: Applied catalog in 15.73 seconds

Hi @yasser583,
This output is 100% normal if /opt/sonatype/sonatype-work/nexus3/admin.password file exist.
Otherwise, it's an issue. Can you please confirm that the file is present ?
If it's present, remove /opt/sonatype/sonatype-work/nexus3/admin.password file and post the output of sudo puppet agent -t ?
Regards

Hi @yasser583,
Ok I can reproduce your problem :)
pba@p1nexus1b:~$ ls -al /opt/sonatype/sonatype-work/nexus3/admin.password
ls: cannot access '/opt/sonatype/sonatype-work/nexus3/admin.password': No such file or directory
pba@p1nexus1b:~$ sudo service nexus restart
pba@p1nexus1b:~$ ls -al /opt/sonatype/sonatype-work/nexus3/admin.password
-rw-r--r-- 1 nexus nexus 36 Aug 28 20:57 /opt/sonatype/sonatype-work/nexus3/admin.password
pba@p1nexus1b:~$ sudo puppet agent -t
Info: Refreshing CA certificate
Info: CA certificate is unmodified, using existing CA certificate
Info: Refreshing CRL
Info: CRL is unmodified, using existing CRL
Info: Using environment 'production'
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Notice: Requesting catalog from puppet:8140 (<IP_ADDRESS>)
Notice: Catalog compiled by p1puppet1a.dc.local
Info: Caching catalog for p1nexus1b.dc.local
Info: Applying configuration version 'p1puppet1a-production-ab8e09f v1'
Error: /Stage[main]/Nexus::Config::Admin/Nexus_user[admin]: Could not evaluate: authentication error
[...]

@TuningYourCode, can you please change the behaviour of the build_options method ? I don't know Ruby, and I don't want to break anything
I believe something like this would be better :
      if @connection_info[:username].present? && @connection_info[:password].present?
        username = @connection_info[:username]
        password = @connection_info[:password].unwrap
      elseif File.exist?(@connection_info[:tmp_pw_file])
        username = 'admin'
        password = File.read(@connection_info[:tmp_pw_file])
      else
        # Throw an error
      end

If the username and the password are defined, in my opinion, the condition should be evaluated first.
As mentioned by @yasser583, if we restart the Nexus service, we notice that /opt/sonatype/sonatype-work/nexus3/admin.password is regenerated, and currently, this generated file has higher precedence, which is not intended.
@yasser583 please wait for @TuningYourCode response and don't restart the service or just manually delete the generated file for now (:
Regards

Looks like a regression, nexus shouldn't create that file if admin user was setup. Not much we can do here for now without breaking other versions.
Let's see how upstream behaves if they fix this issue soon or if this will be the new behaviour that we have to find some workaround.


Update required Python version in documentation

Updated the required Python version in the installation documentation to Python 3.10 or later as installation fails otherwise.

Can you just increment to python>=3.11?

Sure. Just confirming, we want to list 3.11? It works with 3.10, but should I just change that recommended installation piece to >=3.11 and leave the installation requirement as"3.10 or later"?

Well, the stated installation requirement should always match the actual installation requirement, whatever it is.
I have been meaning to drop support for 3.10, and now is as good of a time as any. So could you update the requirement as well as the statement?

Got it. Will do. Thanks for confirming

@nvanalfen could you also modify pyproject.toml to mandate python>=3.11, and also the CI workflows?


Batch sequence generation broken on H2 2.x

Due to https://github.com/h2database/h2database/issues/3483 batch sequence generation is broken on versions 2.x of H2.
There are two possible fixes:
We could special case the H2 dialect and use this query
SELECT NEXT VALUE FOR sequenceName
  FROM SYSTEM_RANGE(1, ?);

Or we could simplify / change the default query to
WITH RECURSIVE T(N) AS (
  SELECT 1
   UNION ALL
  SELECT N + 1
    FROM T
   WHERE N < ?
)
SELECT NEXT VALUE FOR sequenceName
  FROM T;

Thanks for spotting it. Let's change the default query. Would you like to send a Pull Request with the fix?

Yes, I can do a PR.

Looking forward to it

Thanks, I integrated it.


chore: add missing windows-sys features back

What does this PR try to resolve?
Fixes #12562
This is kinda a revert of https://github.com/rust-lang/cargo/pull/12176. I feel like it's better declaring all features direct in use in Cargo.toml regardless features are already unified from dependencies.
I've manually checked all windows-sys API usages in each workspace member and only cargo crate needs these being added.
How should we test and review this PR?
Build them on your windows machine (I don't have one and too lazy to create a VM at this moment 😜)
Additional information
Have some long-term plans in mind https://github.com/rust-lang/cargo/issues/12562#issuecomment-1694242076.
I believe backports is needed.

rust-1.73.0 branch (beta) for 0.74.0 crate release https://github.com/rust-lang/cargo/pull/12564
rust-1.72.0 branch (stable) for 0.73.1 crate release https://github.com/rust-lang/cargo/pull/12565

@bors r+

:pushpin: Commit 907b9c8ecf5164e57828862d73703220bab8f1e9 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 907b9c8ecf5164e57828862d73703220bab8f1e9 with merge b4a770bcb90814f1c4928d93ec835b598901878d...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing b4a770bcb90814f1c4928d93ec835b598901878d to master...

There can be ping for me, as author for that pr :-(
So, there no test that crates build independent.

Hey @klensy. Sorry I should have pinged you. No worries, that's a good sign for the entire ecosystem — Cargo now suffers from the same pain as other projects using workspaces have for a long while 😬. We'll start addressing that!


[tests] correct values expected by autoprefixer

update to correct issues detailed in https://github.com/callemall/material-ui/issues/7264

Deleting all cache, let's try again.

@djbuckley Thank you!

@oliviertassinari your welcome, didn't think there was any point in everyone pulling their hair out thinking they'd gone mad ;)


CPI dev.103 should be named 'photon' instead of 'cpi' to obtain meaningful info

Just a chore, but an annoyance:
curl -s -k https://<IP_ADDRESS>:25555/info | jq .
{
  "features": {
    "snapshots": {
      "status": false
    },
    "compiled_package_cache": {
      "extras": {
        "provider": null
      },
      "status": false
    },
    "dns": {
      "extras": {
        "domain_name": null
      },
      "status": false
    }
  },
  "user_authentication": {
    "options": {
      "urls": [
        "https://<IP_ADDRESS>:8443"
      ],
      "url": "https://<IP_ADDRESS>:8443"
    },
    "type": "uaa"
  },
  "cpi": "cpi",
  "user": null,
  "version": "1.3262.3.0 (00000000)",
  "uuid": "101da2c9-f611-4f1b-891e-9742cc25e7d9",
  "name": "photon-bosh-sandbox"
}

Bosh state file:
        {
            "id": "dfd32c14-860b-48d6-71cc-6f0d36166e77",
            "name": "bosh-photon-cpi",
            "version": "dev.103"
        }

Opened an item to attempt to address this: https://www.pivotaltracker.com/story/show/129865825


add serializers for TmShaHasher

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


do_moves

Hi!
This is an awesome project! Really cool.
I was reading the code, but how moves dons? I understand how normal moves done, but how capturing moves are done?

Yes, make sense. And what if there a few captures in one move?

That's the "ply" concept where a ply consists of one or more moves. A multi-move ply only occurs in presence of a capture while a capture doesn't necessarily lead to multi-move ply.
The respective logic is here, inside play_a_ply().

Got it. Thanks for explanation. Awesome project.

Thanks for your interest. Always happy to help if you have more questions.


Inconsistent metal disconnectors

@Aless-T and I observed the following.  There are two ways to import the MetalDisconnector. In version 2020.09 the behaviour of one of them, "MetalDisconnector2" below (from rdkit.Chem.MolStandardize.rdMolStandardize import MetalDisconnector ), changed and they are inconsistent.
Minimal example:
import rdkit
from rdkit import Chem
from rdkit.Chem.MolStandardize.metal import MetalDisconnector
from rdkit.Chem.MolStandardize.rdMolStandardize import MetalDisconnector as MetalDisconnector2
print(rdkit.__version__)
smi = "Cl[Pd+]"
old_mol = Chem.MolFromSmiles(smi)

md = MetalDisconnector()
new_mol = md.disconnect(old_mol)
new_smi = Chem.MolToSmiles(new_mol)
print(new_smi)
# v2020.09.1 prints: "[Cl-].[Pd+2]"
# v2020.03.3 prints:  "[Cl-].[Pd+2]"

md2 = MetalDisconnector2()
new_mol2 = md2.Disconnect(old_mol)
new_smi2 = Chem.MolToSmiles(new_mol2)
print(new_smi2)
# v2020.09.1 prints: "[Cl-].[Pd+]" <—— charge fixed at +1
# v2020.03.3 prints:  "[Cl-].[Pd+2]"

The change was probably introduced in this commit (https://github.com/rdkit/rdkit/commit/f1119f3980461f3c436b6de9ab10453730bc7621), @ptosco what is the rational behind?
It would be good if both MetalDisconnectors behaved the same.
Would it make sense to add a flag, so that the user chooses the behaviour?

@pschwllr
When you
from rdkit.Chem.MolStandardize.metal import MetalDisconnector

you import the original MolVS Python implementation by Matt Swain, while when you
from rdkit.Chem.MolStandardize.rdMolStandardize import MetalDisconnector as MetalDisconnector2

you import the C++ implementation of MolVS made by Susan Leung and Greg Landrum.
I have made a number of bug fixes and improvements to the C++ implementation, also involving the MetalDisconnector components, so the results are not going to be necessarily consistent with those yielded by the original MolVS implementation.
If you believe there are bugs in the C++ implementation, please do file them with reproducible test cases, but I don't think the inconsistency between the original MolVS Python implementation is a bug per se.
@greglandrum Any thoughts?

I agree that it is not a bug per se.

Would be good if users were made aware of the fact that the two MetalDisconnectors differ.
Looking at the example above "Cl[Pd+]" -> "[Cl-].[Pd+]", @ptosco why fix the charge of the metal in that case and hence, change the total charge?

The current documentation states:

• Disconnect N, O, F from any metal.
• Disconnect other non-metals from transition metals + Al (but not Hg, Ga, Ge, In, Sn, As, Tl, Pb, Bi, Po).
• For every bond broken, adjust the charges of the begin and end atoms accordingly.

How are the charges of the begin and end atoms adjusted in the new version? Because this behaviour clearly changed with the recent bug fixes and improvements.

@pschwllr You are absolutely right. I inadvertently introduced this regression in #3465. I have just submitted a PR that fixes it and added unit tests to avoid that this happens again.

@ptosco and @greglandrum thanks for the prompt response!


JTI Support

I'd like to see the JTI (JWT ID) claim implemented so that I could, say, keep track of 'active' tokens out in the wild. And like the spec says it could be used to help prevent any replay attacks.
Are there any plans for this?

You can add any claim you like in the payload. Is there something more you would want than that?

I meant native support within the library. I can do it myself (and likely will for this project), I was just wondering if there were plans to have it in the library itself for ID generation.

Would like to see this too. Would like to set this in options.
Otherwise I would set everything in payload. Makes the options thing obsolete.
http://self-issued.info/docs/draft-ietf-oauth-json-web-token.html#rfc.section.4.1.7

-1. the JWT spec intentionally only describes JTI by characteristics and not a specific identifier algorithm or format. I don't think a JWT library should make that decision for you. It might make more sense to expose it as a hook to provide an ID generator function, but in JavaScript there's not much benefit to adding api surface vs just specifying the claim via an object literal.

@jden Although I agree with what you're saying, I do think it would be nice to expose jti as a property in the options object. Currently, to add a JTI, one must assign it as a property on their payload object - and while that works, it doesn't really fit with the rest of the claims interface. I think it could be more streamlined.
I'd like to do the following:
var token = jwt.sign(user, process.env.JWT_PRIVATE, {
	algorithm: 'RS256',
	issuer: 'example.com',
	expiresInMinutes: 60,
	identifier: uuid.v4()
});

What does everyone think?

@jedd-ahyoung I agree with your suggestion and I would really like to see this implemented.

You can use payload.jti or options.jwtid now.


PyPi 2.3.1 and 2.4 packages missing

$ pip install -I Routes==2.3.1
Could not find a version that satisfies the requirement Routes==2.3.1 (from versions: 1.5, 1.5.1, 1.5.2, 1.6, 1.6.1, 1.6.2, 1.6.3, 1.7, 1.7.1, 1.7.2, 1.7.3, 1.8, 1.9, 1.9.1, 1.9.2, 1.10, 1.10.1, 1.10.2, 1.10.3, 1.11, 1.12, 1.12.1, 1.12.3, 1.13, 2.0, 2.1, 2.2, 2.3)
No matching distribution found for Routes==2.3.1

$ pip install -I Routes==2.4
Could not find a version that satisfies the requirement Routes==2.4 (from versions: 1.5, 1.5.1, 1.5.2, 1.6, 1.6.1, 1.6.2, 1.6.3, 1.7, 1.7.1, 1.7.2, 1.7.3, 1.8, 1.9, 1.9.1, 1.9.2, 1.10, 1.10.1, 1.10.2, 1.10.3, 1.11, 1.12, 1.12.1, 1.12.3, 1.13, 2.0, 2.1, 2.2, 2.3)
No matching distribution found for Routes==2.4

Yeah it's a result of this change:
https://github.com/bbangert/routes/pull/75
Sad this was released on 1/1 when most people are on vacation...

Yeah it's a result of this change:
https://github.com/bbangert/routes/pull/75
Sad this was released on 1/1 when most people are on vacation...

This is also breaking all of OpenStack CI:
http://logs.openstack.org/91/415991/2/check/gate-nova-pep8-ubuntu-xenial/42bbd87/console.html#_2017-01-01_16_24_19_796818

This is also breaking all of OpenStack CI:
http://logs.openstack.org/91/415991/2/check/gate-nova-pep8-ubuntu-xenial/42bbd87/console.html#_2017-01-01_16_24_19_796818

There is also no 2.4.0 package on pypi and the 2.3.1 package is gone:
https://pypi.python.org/pypi/Routes/2.3.1
And OpenStack can't use 2.3.0 because of a backward compat issue that forced us to move to using 2.3.1.

There is also no 2.4.0 package on pypi and the 2.3.1 package is gone:
https://pypi.python.org/pypi/Routes/2.3.1
And OpenStack can't use 2.3.0 because of a backward compat issue that forced us to move to using 2.3.1.

PyPi is not letting me upload the packages:
Upload failed (400): This filename has previously been used, you should use a different version.
error: Upload failed (400): This filename has previously been used, you should use a different version.

I have no idea why, there are no files there as can be clearly seen. I also have no idea how the 2.3.1 package disappeared. I'm looking into what happened.

PyPi is not letting me upload the packages:
Upload failed (400): This filename has previously been used, you should use a different version.
error: Upload failed (400): This filename has previously been used, you should use a different version.

I have no idea why, there are no files there as can be clearly seen. I also have no idea how the 2.3.1 package disappeared. I'm looking into what happened.

@bbangert @webknjaz ouch, guess that calls for a 2.4.1 then?

@bbangert @webknjaz ouch, guess that calls for a 2.4.1 then?

I was able to restore 2.3.1 version, but due to the existing bug with not being to upload a filename that has ever been used (even if its not currently present), I cannot restore the actual 2.3.1 files either.

I was able to restore 2.3.1 version, but due to the existing bug with not being to upload a filename that has ever been used (even if its not currently present), I cannot restore the actual 2.3.1 files either.

Ok, apparently this was a new change that I haven't run into before.
pypa/packaging-problems#74
I will create a 2.4.1, and I guess 2.3.1 is just hosed. Sigh.

Ok, apparently this was a new change that I haven't run into before.
pypa/packaging-problems#74
I will create a 2.4.1, and I guess 2.3.1 is just hosed. Sigh.

// off topic
@bbangert do you still have a twitter account? the one linked in your blog belongs to some other person now. I was going to ping you about this issue in twitter but couldn't find a handle..

// off topic
@bbangert do you still have a twitter account? the one linked in your blog belongs to some other person now. I was going to ping you about this issue in twitter but couldn't find a handle..

@bbangert I suggest you automating the deployments to PYPI, it's simple:
doc: https://docs.travis-ci.com/user/deployment/pypi/
example: https://github.com/cherrypy/cherrypy/blob/master/.travis.yml#L33-L42

@bbangert I suggest you automating the deployments to PYPI, it's simple:
doc: https://docs.travis-ci.com/user/deployment/pypi/
example: https://github.com/cherrypy/cherrypy/blob/master/.travis.yml#L33-L42

I was able to re-upload 2 2.3.1 wheels as I still had the exact same matching hash files around. Unfortunately I wasn't able to upload the source tgz as the hash changed.

I was able to re-upload 2 2.3.1 wheels as I still had the exact same matching hash files around. Unfortunately I wasn't able to upload the source tgz as the hash changed.

@webknjaz Thanks, I've added the appropriate config to the travis and will try it with 2.4.1 shortly.

@webknjaz Thanks, I've added the appropriate config to the travis and will try it with 2.4.1 shortly.

@bbangert I think you could add testing against 3.6 as well :)

@bbangert I think you could add testing against 3.6 as well :)

@bbangert
According to https://travis-ci.org/bbangert/routes/jobs/188075695:
HTTPError: 400 Client Error: A file named "Routes-2.4.1.dev0-py2.py3-none-any.whl" already exists for  Routes-2.4.1.dev0. To fix problems with that file you should create a new release. for url: https://pypi.python.org/pypi

@bbangert
According to https://travis-ci.org/bbangert/routes/jobs/188075695:
HTTPError: 400 Client Error: A file named "Routes-2.4.1.dev0-py2.py3-none-any.whl" already exists for  Routes-2.4.1.dev0. To fix problems with that file you should create a new release. for url: https://pypi.python.org/pypi

To avoid this you should restrict python version for the deployment provider:
  on:
    tags: true
    python: 3.5

To avoid this you should restrict python version for the deployment provider:
  on:
    tags: true
    python: 3.5

It also added the dev which it shouldn't have because its not running the command properly. Sigh.

It also added the dev which it shouldn't have because its not running the command properly. Sigh.

I think you should specify distributions: release like CherryPy does.

I think you should specify distributions: release like CherryPy does.

Guess not:
error: invalid command 'release'

Ok then...

Guess not:
error: invalid command 'release'

Ok then...

Yea.. I see. That's because it's an alias

Yea.. I see. That's because it's an alias

I can confirm that OpenStack CI is at least happy again with the 2.3.1 wheel being available on pypi. Thanks for addressing that part quickly.

I can confirm that OpenStack CI is at least happy again with the 2.3.1 wheel being available on pypi. Thanks for addressing that part quickly.

@bbangert see #77

@bbangert see #77

@webknjaz thanks, merged and pushed the tag, hopefully this does the trick.

@webknjaz thanks, merged and pushed the tag, hopefully this does the trick.

let's see

let's see

okay, it did the deployment https://travis-ci.org/bbangert/routes/jobs/188081884

okay, it did the deployment https://travis-ci.org/bbangert/routes/jobs/188081884


Point Nudge to specific update

We are trying to get nudge to start a specific update like you can with the python verison by using /Applications/.app and we are using this with the actionbuttonpath, but we can't seem to get it to work, either Nudge just starts and closes or it closes right after you press the update button.
I can't find anything in the wiki that describes how to point nudge to the specific OS version, is there anything we are missing or is the swift version not able to upgrade from a local downloaded macOS version

This is possible, but there is a bug. This bug should be fixed with. https://github.com/macadmins/nudge/pull/284
In my testing the following configuration is sufficient to open /Applications/Install macOS Monterey.app if it exists. If you have the application in a different path you will have to set majorUpgradeAppPath as well.
{
  "osVersionRequirements": [
    {
      "aboutUpdateURL": "https://google.com",
      "requiredInstallationDate": "2021-11-28T00:00:00Z",
      "requiredMinimumOSVersion": "12.0.1",
      "targetedOSVersionsRule": "default"
    }
  ],
  "userExperience": {
    "randomDelay": false
  },
  "userInterface": {
    "iconDarkPath": "/Library/Application Support/Nudge/logoDark.png",
    "iconLightPath": "/Library/Application Support/Nudge/logoLight.png",
    "screenShotDarkPath": "/Library/Application Support/Nudge/screenShotDark.png",
    "screenShotLightPath": "/Library/Application Support/Nudge/screenShotLight.png"
  }
}


[v4] Bundled Coroutine

Hello,
I'm currently playing with coroutine with Lua and Sol and I'm actually a bit confused.
Indeed, I'm trying to reproduce something like that:
Lua Code:
co1 = coroutine.create(
	function()
	        print("AA : Step 1")
	        coroutine.yield()
	        print("AA : Step 2")
	end
)
coroutine.resume(co1)

co2 = coroutine.create(
	function()
	        print("BB : Step 1")
	        coroutine.yield()
	        print("BB : Step 2")
	end
)
coroutine.resume(co2)

coroutine.resume(co1)

Output:
$ lua test.lua
AA : Step 1
BB : Step 1
AA : Step 2

So I wanted to be able to do the same thing using sol::coroutine, but it seems that the wrong coroutine is executed:
#include <sol/sol.hpp>
#include <iostream>

int main()
{
    sol::state lua;
    lua.open_libraries(sol::lib::base, sol::lib::package, sol::lib::coroutine);

    lua["print"] = [](const sol::lua_value &v){
        std::cout << v.as<std::string>() << std::endl;
    };
    lua["cyield"] = sol::yielding([](){
        std::cout << "YIELDING" << std::endl;
    });

    sol::coroutine co1 = lua.load(R"(
        print("AA : Step 1")
        cyield()
        print("AA : Step 2")
    )");
    co1();

    sol::coroutine co2 = lua.load(R"(
        print("BB : Step 1")
        cyield()
        print("BB : Step 2")
    )");
    co2();

    co1();

    return 0;
}

Output :
AA : Step 1
YIELDING
AA : Step 2
AA : Step 1
YIELDING

I read some things about using sol::thread but I don't understand the way I should use it, for me I don't want to multithread my code, so I don't know if I'm on the good way.
Thanks for your help !

So, the reason you need sol::thread is because each of those coroutines needs its own execution stack.
In reality, it's closer to sol::execution_stack. It is a context in which the coroutine runs, and each coroutine needs its own sol::thread to have independent variables that don't clash with one another and its own return types. That's why the examples (https://sol2.readthedocs.io/en/latest/api/coroutine.html#yield-main-thread) go through some shenanigans, to give you the right code:
#include <sol/sol.hpp>
#include <iostream>

int main()
{
    sol::state lua;
    lua.open_libraries(sol::lib::base, sol::lib::package, sol::lib::coroutine);

    lua["print"] = [](sol::object v){
        std::cout << v.as<std::string>() << std::endl;
    };
    lua["cyield"] = sol::yielding([](){
        std::cout << "YIELDING" << std::endl;
    });

    // notice the new threads!
    sol::thread thread1 = sol::thread::create(lua);
    sol::thread thread2 = sol::thread::create(lua);

    // notice we load it FROM the new "execution stack"
    // we need it to have thread1's stack perspective
    sol::coroutine co1 = thread1.state().load(R"(
        print("AA : Step 1")
        cyield()
        print("AA : Step 2")
    )");
    // call first coroutine here
    co1();

    // notice we load it FROM the new "execution stack"
    // we need it to have thread2's stack and perspective
    sol::coroutine co2 = thread2.state().load(R"(
        print("BB : Step 1")
        cyield()
        print("BB : Step 2")
    )");

    // run the other coroutine
    co2();
    co1();
    // tada! they run on
    // independent stacks

    return 0;
}

Note to self
Given the number of times this has happened, I'm beginning to think I should write a sol::bundled_coroutine and completely change the docs to have people use that, where it automatically creates a new stack to be paired with every coroutine. The current interface is one-to-one with low-level concepts, and honestly that's causing people more heartache than benefit right now...

(To be clear, there is no sol::execution_stack. It doesn't exist; it's just a better name than sol::thread. Still, I need a good name for something like coroutine_but_it_has_the_thread_too...)

Thanks a lot for your reply, indeed this naming and the context itself is a bit confusing, maybe something named like coroutine_context could make it ?

Current name idea so far is sol::packed_coroutine.
This is also going to make me create a sol3 header. where I create aliases under a sol3:: namespace using good names for everything....

I am not sure if this is the right place to post but I am similarly confused by how coroutines are exposed and the docs are somewhat sparse.  I would like to be able to allow a function implemented in C++ to yield if the operation would block so that other Lua threads can run.  The sol::yielding wrapper doesn't appear to provide a mechanism for capturing the continuation that allows me to return a value and all of the documentation for the sol::thread and sol::coroutine machinery seems to be assuming that I am creating a thread in C++, rather than allowing Lua code to create coroutines and resuming them in response to external events.

@davidchisnall could you find a way to do that? I'm trying to implement the same funtionality you're talking about but every discussion or example I find here seems like a dead end.

I did, a year ago, but I'm not sure I can distill a useful example.  I'll try to get around to putting my code somewhere public soon.  Reading through my code now:
I wrote a simple cooperative scheduler that exposed some blocking calls, including the option to run functions in the background as coroutines.  The model for things involving the scheduler is that coroutines yield to the scheduler, which then yields to another thread.  The scheduler keeps a sol::thread (stack) and a sol::coroutine (callable function that returns when it yields, constructed from the sol::thread and a sol::function).  When Lua calls a function that you've exposed with sol::yielding, you need to capture the current coroutine (scheduler state) and later invoke it with whatever the result is.
I do this by packaging the current thread state (thread and coroutine) in an event object that has a callback that, when populated, returns the result (a sol::variadic_results).  When the event is ready, it's pushed into a list of ready events.  The scheduler iterates over this and then invokes the coroutine that the event captured with the result of its callback.
This is only a couple of hundred lines of code, including the kqueue integration, but reading the docs was frustrating because they assumed that I knew a lot more about the internals of both Sol and the Lua VM than I did.
The main problem is that the function invoked by sol::yielding does not take the current thread and coroutine as arguments.  I couldn't find a way of extracting them from Sol3, so I ended up having my scheduler track them.  This should be somewhat redundant (on invoking sol::yielding, Sol must at least know the current thread, and probably the current coroutine, so should be able to just pass them into the lambda).
I also needed to create the main thread like this:
        ThreadState(sol::state &state, sol::string_view code)
        {
            thread    = sol::thread::create(state);
            coroutine = thread.state().load(code);
        }

Without that, I could yield from the main thread but not get back to it.
If I remember correctly, calling a sol::yielding function causes the call into Lua to return back to C++.  You can then pick another coroutine to run.


Config in JSON

Giving an option to enter credentials into a JSON instead of environment variables?

Implemented by 13cfac1 . Environment variables removed completely.


Img2Img

Very good results, however is there a way to use it for Img2Img?

This needs to be implemented more natively to allow for the combination with other modules, like Latent and KSampler. That will be an even better work, and I'm looking forward to it as well.

@omarei @deng-wei Im currently trying to get the model and vae to work with ksamplers, loras etc. The architecture of the model is different to what those nodes are expecting so the challenge is converting it.

any news?

any news?

Been a bit busy at my day job, but I've been having issues getting it to properly denoise a supplied latent image, if its an empty latent it works fine. Once, I'll revisit it again today, and see if I can figure out why.


Add support for custom separator

What:
This adds support for custom "separators." By default, babel-plugin-react-intl-auto will join the key from a defineMessages and the path to that key by separating each part with a ".". For example:
// src/components/Button.js

import { defineMessags } from '...';

const messages = defineMessages({
  greeting: 'Hello',
});

import { defineMessags } from '...';

const messages = defineMessags({
  greeting: {
    id: 'src.components.Button.hello',
    defaultMessage: 'Hello',
  }
})

By defining a custom separator, you can replace the "." with a different value. For example, using separator: "_" would yield this:
import { defineMessags } from '...';

const messages = defineMessags({
  greeting: {
    id: 'src_components_Button_hello',
    defaultMessage: 'Hello',
  }
})

Why:
I am working on a project with Gatsby where I'd like to be able to:

Define my strings in my components
Extract those strings int JSON files for each supported locale
Be able to query the extracted JSON files via GraphQL

The first two parts of this are already supported by the library. However, when attempting to read the JSON files contents via GraphQL, I was unsuccessful. As best I can tell, GraphQL does not support . in the keys and was replacing them with _ by default. However, when the GraphQL query would query the value where the key was replaced (from '.' -> '_') the strings would not appear.
After some experimentation, I realized that if I manually replace the . with a different separator the GraphQL query would respond with the data as expected.

How:
Inside your babel config, you specify a "separator" value. If you do not specify one, internally we fall back the default "." value. This means this will not have any negative impact on existing users.
Checklist:

[x] Documentation
[x] Tests
[x] Ready to be merged
[x] Added myself to contributors table

I noticed that certain separator strings would throw an exception when running through a particular code path that involved a series of RegExp replacements. The best solution I could come to was ty wrap the RegExp in a try/catch and then if the expression failed, I would then escape the strings and try again. Originally I tried to always escape the strings but that caused existing tests to fail. That is why I went with the try/catch approach. I am open to feedback on a better way to implement this.

@ericmasiello Nice. Thank you 💯

Released v3.1.0 🚀


cannot use rdkit.Chem.MolStandardize.rdMolStandardize.Normalize

rdkit.Chem.MolStandardize.rdMolStandardize.Normalize needs RDKit/Data/MolStandardize/normalizations.txt, but it is not contained in wheel.
>>> from rdkit.Chem.MolStandardize.rdMolStandardize import Normalize
>>> from rdkit import Chem
>>> Normalize(Chem.MolFromSmiles('c1ccccc1'))
[01:23:44] Initializing Normalizer
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
OSError: File error: Bad input file /path/to/build/temp.macosx-10.9-x86_64-3.9/rdkit_install/share/RDKit/Data/MolStandardize/normalizations.txt

Thanks,

I have pushed a new version to PyPi that includes the Data directory. Can you please check if this works for you on macOS?  @philopon?
https://pypi.org/project/rdkit-pypi/2<IP_ADDRESS>/
In [2]: from rdkit.Chem.MolStandardize.rdMolStandardize import Normalize
   ...: from rdkit import Chem
   ...: Normalize(Chem.MolFromSmiles('c1ccccc1'))
[09:47:32] Initializing Normalizer
[09:47:32] Running Normalizer
Out[2]: <rdkit.Chem.rdchem.Mol at 0x1480ff56fdc0>

It works. Thank you!


TotalBlocks metric is negative

Alluxio Version:
master branch
commit hash 6f8c5d2
Describe the bug
After less than an hour running, the totalBlocks metric displayed a negative value.
To Reproduce
Not sure it can repro exactly.
Expected behavior
A right count of total block
Urgency
Normal
Additional context
No

@yuzhu PTAL, thanks!

@yuzhu Please take a look thanks.


Don't display anything for TEST environment?

I know this comes from Facebook, but it's somewhat annoying to see bunch of warnings during tests which don't always make proper calls to test stuff. Would be possible to add condition for NODE_ENV === 'test'?

This is just a mirror of Facebook's warning module. As such, we won't be changing behavior here.
Happy Coding!

Ok I expected as much, I made my own fork for that. So if anyone is interested and want to avoid warnings during tests, just go with npm install github:BlackDice/warning.


Add vibration to morseCode()

This PR needs updating and rebasing.
Giving it a time limit to update otherwise it will be closed.


Create sitarameshcampus.edu.np

@roshanbudhathoki First stage of verification passed, official domain is sitarameshcampus.edu.np as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@roshanbudhathoki Thank you for your patience while waiting for a review and processing your request. Pull request merged.


feat(electron): pick random port

resolves #2992
Using get-port would have been nicer, but the library is built ESM only and can not be used from our CommonJS project. Hence I followed the title of the ticket and simply picked a random port between 10000 and 65535.

Going ahead and merge this so that we have it in the next release.
@holzeis : please address the comments in a follow-up PR. Thank you :)
bors r+

Tested locally and works great. The dis/advantage is that you don't know the port anymore so you can't open it in the browser as well :)

The default port electron will try is 7113. if that does not work - it will start to try random ports. So in the most cases you should know the port. But if it's really taken you can still check the logs ;)


pull request for e2e test at 11/08/2016 20:45:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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


Using rc2 nightly builds fails

Currently the nightly builds are beoken with a webapplication exception in startup.cs.  is there not a normal nuget package i can refernece to use this??

The nightly builds work fine, and no, there's no "normal" NuGet package, only RC2 nightly packages.

If i create a new web application and follow the instructions on the readme file it errors....

I'm not really good at psychic debugging. If you don't share - at least - the error message you're seeing, there's nothing we can do for you.

Sorry not at laptop atm. It is something about webapplication but i will get the error for u later

I was doing it with a brand new application thus why confused. Will post error as soon as i can

Pardon my jumping in here, but I have a related question...  If I want to incorporate OpenIddict into an existing "rc1-final" solution (consisting of several projects), is it a requirement that I convert the entire Solution to use rc2 nightly builds for all asp.net 5 dependencies?   Thanks.

Is it a requirement that I convert the entire Solution to use rc2 nightly builds for all asp.net 5 dependencies?

Yes, you can't mix RC1 and RC2 packages in the same application (at least, not without seeing tons of exceptions at runtime :smile:)

@Gillardo are you still seeing an error?

Yes i am, but i do not think it is your code, but maybe you can still help.  I have updated all my packages to rc2, but when i build, this line is invalid
// Entry point for the application.
public static void Main(string[] args) => new WebApplication.Run<Startup>(args);

If i add a using like so using Microsoft.AspNet.Hosting.Internal;
Then it appears that the Run method is no longer there??
What do i need to change this to?  Really want to try out your software, so if you can help, that would be great

The hosting API has been completely revamped: https://github.com/aspnet/Announcements/issues/131
Take a look at this commit to see how we fixed the OpenIddict samples to use the new API.

Thank you!!!!

Possible somebody knows if it's possible to use rc2 projects on Azure App Service? Does it support rc-2 version of dnx?


[GSSOC'23] Fixing the SignUp page

Feature idea.
The sign up page needs to be fixed a lot.
The input fields are not proportional and it makes the design look ugly. Also it is not responsive to screen sizes.

Describe the solution you'd like
All the inputs fields should be proportional to make the design look even.
Also effect can be added when an input is in focus.
The design breaks when viewed on smaller screens.
I would be willing to work on these issues and hence fixing the whole signup page and adding responsive design to it

hey, I wanted to contribute to this issue under GSsoc'23, can you please assign this issue to me.
I have made login and sign-up page in other issue. The webpages are responsive. I am attaching the screenshot of the page for examples.

Hey hey i think only single person needs to be assigned for an issue

@rishabhchaudhary0210 you can fix the responsiveness of signin sign up page!
@Siddhant422 i suggest you to open a new feature request for this!
Thankyou!

Since rishabh has not responded, @MonalikaPatnaik Can you assign this project to me.

Hey @MonalikaPatnaik I have started working on this and would issue the PR till tomorrow Thanks for assigning me!!!


Alps 1296 - Share current Camera & View parameters

Add short URL support for camera & view parameters into the generateHash method of History.
Short format sample : &-32.02y,6.03p,0.00r,90.00f,rectilinearv which is equivalent to &yaw=-32.02&pitch=6.03&roll=0.00&fov=90.00&view=rectilinear

 All committers have signed the CLA.


Grafana chart resource limits hardcoded

What did you do?
Specifiy resources for grafana chart
What did you expect to see?
Resources to be applied to grafana container
What did you see instead? Under which circumstances?
Hardcode resource limits in chart
https://github.com/coreos/prometheus-operator/blob/master/helm/grafana/templates/grafana-deployment.yaml#L61

Would you like to contribute to the project sending a PR?

Added a PR for this: https://github.com/coreos/prometheus-operator/pull/1388

@gianrubio I think we can close this ticket, since pr https://github.com/coreos/prometheus-operator/pull/1388 has been merged


App crashes while uploading

Summary:
Summarize your issue in one sentence (what goes wrong, what did you expect to happen)
App crashes when upload is somehow initiated from the background.
Steps to reproduce:
Yet to figure out, reported by email
System logs:
java.lang.RuntimeException: Unable to start activity ComponentInfo{fr.free.nrw.commons/fr.free.nrw.commons.contributions.MainActivity}: java.lang.IllegalStateException: Not allowed to start service Intent { act=fr.free.nrw.commons.upload.upload cmp=fr.free.nrw.commons/.upload.UploadService }: app is in background uid UidRecord{a10f242 u0a101 LAST bg:+10h50m21s78ms idle change:cached procs:1 seq(0,0,0)}
	at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2951)
	at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:3086)
	at android.app.ActivityThread.handleRelaunchActivityInner(ActivityThread.java:4823)
	at android.app.ActivityThread.handleRelaunchActivity(ActivityThread.java:4732)
	at android.app.servertransaction.ActivityRelaunchItem.execute(ActivityRelaunchItem.java:69)
	at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:108)
	at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:68)
	at android.app.ClientTransactionHandler.executeTransaction(ClientTransactionHandler.java:55)
	at android.app.ActivityThread.handleRelaunchActivityLocally(ActivityThread.java:4782)
	at android.app.ActivityThread.access$3200(ActivityThread.java:200)
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1826)
	at android.os.Handler.dispatchMessage(Handler.java:106)
	at android.os.Looper.loop(Looper.java:193)
	at android.app.ActivityThread.main(ActivityThread.java:6718)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:491)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)
Caused by: java.lang.IllegalStateException: Not allowed to start service Intent { act=fr.free.nrw.commons.upload.upload cmp=fr.free.nrw.commons/.upload.UploadService }: app is in background uid UidRecord{a10f242 u0a101 LAST bg:+10h50m21s78ms idle change:cached procs:1 seq(0,0,0)}
	at android.app.ContextImpl.startServiceCommon(ContextImpl.java:1577)
	at android.app.ContextImpl.startService(ContextImpl.java:1532)
	at android.content.ContextWrapper.startService(ContextWrapper.java:664)
	at fr.free.nrw.commons.contributions.MainActivity.initMain(MainActivity.java:118)
	at fr.free.nrw.commons.contributions.MainActivity.onCreate(MainActivity.java:90)
	at android.app.Activity.performCreate(Activity.java:7144)
	at android.app.Activity.performCreate(Activity.java:7135)
	at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1271)
	at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2931)
	... 16 more

Device and Android version:
ANDROID_VERSION=9
PHONE_MODEL=Redmi 3S
Commons app version:
APP_VERSION_CODE=709
APP_VERSION_NAME=2.13
Would you like to work on the issue?
Yes

The UploadService is either missing to call startForeground in some cases, we can probably think of including a combination of WorkManager and Foreground service to handle the same

We can add this to the stability tasks in the upcoming grant probably, as it only happens intermittently and the workload for the fix will be pretty significant.

From conversation with @ashishkumar468 : This will happen when the user presses the home button/app goes in background immediately after initiating upload before the app gets time to attach the service with the progress update notification. And only in Android O+.

I just got this by starting the upload process, and switching off the screen of my phone while on the depictions (or was it categories?) screen, then switching my phone's screen back on like 20 minutes later.

With the latest master, this happens anytime I try to upload an image (via either gallery or camera, from either home or nearby).
Happens even after uninstalling and reinstalling the app.

@nicolas-raoul  Can you send the logs for this, please?

@ashishkumar0207 30 minutes ago I wrongly posted a comment here, but then realized that it was a different error, so I filed a new issue: https://github.com/commons-app/apps-android-commons/issues/3997
Sorry for the noise!

No problem @nicolas-raoul


Docker Kubernetes

Hi,
I had trouble finding information if the helpy docker instance can be ran in kubernetes.
The usual problem is that docker images encode passwords, url in the image.
Has anyone tried making helpy work in kubernetes?
I'm interested in contributing.

Hey @fire would be great!  I have not done it yet myself, but there have been some changes over the last months to move passwords and api keys to env. variables where possible.  This makes it easier to separate the app server from the filestore and db.  This has worked great for running Helpy in a cluster with ephemeral app servers, even auto scaling.
The one issue with Helpy is that it uses a cached settings plugin that stores key-value settings both in the database and on disk.  Thus, in a cluster of app servers, it becomes impossible to invalidate the disk cached items immediately.  The solution is run a cron that deletes the cache every 1-n minutes.  Luckily these settings don't change very often except when you are first getting things configured.
As far as k8s goes, I know one team worked on this about a year ago, but I have no idea how well they got it working.  It should be easier now because you can use S3 for file storage by using env. vars.

List of tasks:

move passwords and api keys to env. variables where possible

run a cron that deletes the cache every 1-n minutes

use S3 for file storage by using env. vars.

Create kubernetes docker image

Create statefulset

Pick a postgresql operator

Create a service

Create an ingress

See https://github.com/helpyio/helpy/blob/master/docker/.env.sample for the environmental variables.

postgres
rails secret
remote storage local
remote storage s3
transfer remote local to s3 to local?
Always prepare / run migrations

@scott is unicorn required, or can I use a loadbalance that roundrobins with affinity?

I think that the current docker container uses Unicorn, but you can use any Rails app server.  I mostly use Puma or Passenger with nginx on the front end.

Hey @fire were you able to get Helpy to work in Kubernetes?

Hey @dwene I am not sure about @fire, but I have been able to run it in Kubernetes and am almost finished with the PR and documentation

That's great to hear @scott ! Let me know if you need a guinea pig to try anything out, or if you need help getting it across the line. I'm definitely interested in trying it.

Ah, I was not able to get Kubernetes running. Other priorities got me.

Hi @dwene Here is what you will want to start with https://github.com/helpyio/helpy/tree/kubernetes/deploy

I should add I left the postgres deployment in there but have not tried to get it working.  I am only interested in using DBAAS for kubernetes myself.  If you can get it to work that would surely be helpful to others.

I'd do Helm chart if you guys create the deployment yaml files.


Feature/multi call roles

Now with multicall to grant and revoke roles.

Love the suggestion here! Thank you!
But could we fix the PR so it's easier to review? Most of the changes look like they could be in a .gitignore

OK I have fixed the formatting -now with 4 spaces tab width. (I was using 2 spaces tab width).
Sorry I am trying to understand your comment:
Most of the changes look like they could be in a .gitignore
Anyway I added artifacts and [._]*.s[a-v][a-z] in the file because I am using vim as the editor.
Let me know.

Right. Sorry it's been hard to see what your changes are doing.
I think this one too is just looking to implement this with hardhat? Could you check this repo and see if it has everything you need?
https://github.com/PatrickAlphaC/dao-template

Thank you for replying my request. I will check it out and let you know.
On Mon, 21 Mar 2022 at 7:52 PM, Patrick Collins @.***>
wrote:

Right. Sorry it's been hard to see what your changes are doing.
I think this one too is just looking to implement this with hardhat? Could
you check this repo and see if it has everything you need?
https://github.com/PatrickAlphaC/dao-template
—
Reply to this email directly, view it on GitHub
https://github.com/brownie-mix/dao-mix/pull/4#issuecomment-1073802339,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSXJ6EKEGAMDWAREUV4QLVBBPIBANCNFSM5RBR2YWQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Below are your comments in the script file deploy_and_run.py
    # Now, we set the roles...
    # Multicall would be great here ;)

So, I was trying to do multicall to set the roles and at the same time do it hardhat task.
What I should have done is implement it in brownie/python script - this would be a clearer "pull request".
I learnt my mistake and sorry I make you confuse.

Ahhhhh, yes exactly!


Support aggregation with zip files

Codecov Report
Attention: Patch coverage is 42.97521% with 69 lines in your changes missing coverage. Please review.

Project coverage is 73.44%. Comparing base (44a11bc) to head (ad0d776).

Files with missing lines
Patch %
Lines

...nt/torch_bench/_bash_bench_benchmark_runner_agg.py
57.14%
39 Missing :warning:

...erimental_experiment/torch_bench/bash_bench_agg.py
0.00%
30 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #253      +/-   ##
==========================================
- Coverage   73.60%   73.44%   -0.17%
==========================================
  Files         160      160
  Lines       23054    23123      +69
==========================================
+ Hits        16970    16983      +13
- Misses       6084     6140      +56

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


ImportError: No module named pyocd.tools.flash_tool

$ pip show pyocd
Name: pyocd
Version: 0.13.0
....
$ pyocd-flashtool
Traceback (most recent call last):
  File "/usr/local/bin/pyocd-flashtool", line 7, in <module>
    from pyocd.tools.flash_tool import main
ImportError: No module named pyocd.tools.flash_tool

Thanks for the bug report!
Did you have pyOCD already installed? What OS?
I'm guessing this is a very unfortunate side effect of the (necessary) package name change to all lowercase pyocd. On case-insensitive filesystems, pip may not be renaming the previous pyOCD to pyocd when it updates to version 0.13.0. But Python does perform case-sensitive package imports.
Can you try uninstalling and reinstalling pyOCD to see if that recreates the package directory with the correct name?
If that solves the problem, I'll add some info about this to the top of the README.

I had a very similar problem, but on windows. I was getting following exception:
E:\mbed_projects\Nucleo_STF401RE\encoder-test\BUILD\NUCLEO_F401RE\GCC_ARM-DEBUG>pyocd-gdbserver -t NUCLEO_F401RE
Traceback (most recent call last):
  File "C:\Python27\Scripts\pyocd-gdbserver-script.py", line 11, in <module>
    load_entry_point('pyocd==0.13.0', 'console_scripts', 'pyocd-gdbserver')()
  File "c:\python27\lib\site-packages\pkg_resources\__init__.py", line 479, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "c:\python27\lib\site-packages\pkg_resources\__init__.py", line 2703, in load_entry_point
    return ep.load()
  File "c:\python27\lib\site-packages\pkg_resources\__init__.py", line 2321, in load
    return self.resolve()
  File "c:\python27\lib\site-packages\pkg_resources\__init__.py", line 2327, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
ImportError: No module named pyocd.tools.gdb_server

I tired to reinstall pyocd with pip, but it didn't help.
However, changing folder name in C:\Python27\Lib\site-packages\ from pyOCD to pyocd solved the problem ;-) Now everything is working fine. Maybe it will help somebody.

I'm able to reproduce the problem with upgrading on Windows.
Although, strangely, I get a slightly different exception:
Traceback (most recent call last):
  File "C:\Program Files\Python37\Lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Program Files\Python37\Lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "Z:\creed\foo\winenv3\Scripts\pyocd-gdbserver.exe\__main__.py", line 5, in <module>
ModuleNotFoundError: No module named 'pyocd'```

Either way, this seems like a setuptools bug to me. I'll double check, create a setuptools issue and maybe patch, and update the pyOCD README.md with help for doing the upgrade.

I think I found the problem: I use a separate, clean working copy of the pyOCD repo to perform releases to PyPI. Because I use macOS with a case-insensitive APFS filesystem, when I pulled the 0.13.0 changes into this working copy, git did not rename the pyOCD directory to pyocd. Nor did setuptools have any issues building the sdist. So the tarball released to PyPI has the incorrect mixed case name for the pyOCD package. This causes different failures depending on whether the end user's filesystem is case-sensitive. Will fix and release 0.13.1 very shortly.

Just released v0.13.1 that fixes the issues on macOS, Linux, and Windows.
Closing this issue. If you continue to have problems please reopen.


The lesson accepted a wrong answer

On the lesson "Waypoint: Change the font size of an element" I noticed that i put in a code that looked wrong but I wasn't sure if it was correct or not.  The code I put in is located below:
This is what I put but the lesson accepted it and moved on to the next one:

This is the correct code which is shown in the next lesson:

Your code works because both P elements have the .red-text class. The first solutions sets all elements with the class .red-text to 16px , including the h2 element. If you remove the red-text class from your p elements, your solution will not be accepted.

Awesome thanks!


Neo4j APT Source Key Signature Expired

🐛 Summary
The current signature for the Neo4j apt source is expired
To reproduce
Steps to reproduce the behavior:

Run sudo apt update

Expected behavior
I expected no errors running apt update
Any helpful log output or screenshots
Paste the results here:
sudo apt update
Get:1 https://download.docker.com/linux/debian bookworm InRelease [43.3 kB]
Get:2 https://debian.neo4j.com stable InRelease [44.3 kB]
Get:3 http://kali.download/kali kali-rolling InRelease [41.5 kB]
Get:4 http://downloads.metasploit.com/data/releases/metasploit-framework/apt lucid InRelease [3956 B]
Get:5 https://packages.microsoft.com/repos/code stable InRelease [3590 B]
Err:2 https://debian.neo4j.com stable InRelease
  The following signatures were invalid: EXPKEYSIG 59D700E4D37F5F19 Neo4j Admins<EMAIL_ADDRESS>Get:6 http://kali.download/kali kali-rolling/main amd64 Packages [20.2 MB]
Get:7 http://downloads.metasploit.com/data/releases/metasploit-framework/apt lucid/main amd64 Packages [418 B]
Get:8 https://packages.microsoft.com/repos/code stable/main amd64 Packages [18.5 kB]
Get:9 http://kali.download/kali kali-rolling/main amd64 Contents (deb) [48.2 MB]
Get:10 http://kali.download/kali kali-rolling/contrib amd64 Packages [112 kB]
Get:11 http://kali.download/kali kali-rolling/contrib amd64 Contents (deb) [273 kB]
Fetched 68.9 MB in 9s (7812 kB/s)
1246 packages can be upgraded. Run 'apt list --upgradable' to see them.
Warning: https://download.docker.com/linux/debian/dists/bookworm/InRelease: Key is stored in legacy trusted.gpg keyring (/etc/apt/trusted.gpg), see the DEPRECATION section in apt-key(8) for details.
Warning: https://debian.neo4j.com/dists/stable/InRelease: Key is stored in legacy trusted.gpg keyring (/etc/apt/trusted.gpg), see the DEPRECATION section in apt-key(8) for details.
Warning: An error occurred during the signature verification. The repository is not updated and the previous index files will be used. GPG error: https://debian.neo4j.com stable InRelease: The following signatures were invalid: EXPKEYSIG 59D700E4D37F5F19 Neo4j Admins<EMAIL_ADDRESS>Warning: Failed to fetch https://debian.neo4j.com/dists/stable/InRelease  The following signatures were invalid: EXPKEYSIG 59D700E4D37F5F19 Neo4j Admins<EMAIL_ADDRESS>Warning: Some index files failed to download. They have been ignored, or old ones used instead.

I can't reproduce this with the latest Kali AMI, which appears to be using the same key.  Can you tell me which environment (production or staging) has this issue so I can look on that instance?

@jsf9k I've noticed it in env42, env27, and env39 so far. I can't reproduce in staging7 though.

Ah, it's because they extended the life of the key that originally expired on October 25.  You just need to run this command to update the expiration date on the key:
$ wget -O - https://debian.neo4j.com/neotechnology.gpg.key | sudo apt-key add -

(I already ran this command on kali1 in env42-production.)
New instances (like the ones in env7-staging) already have the updated expiration date and hence do not require this.

Thank you sir, great to hear!


Type hint check and corrections

Add type hint checks with mypy and typeguard following best practices of hypermodern_python (see #18).
Code update was needed to correct all errors detected by mypy.

Don't merge yet, I need to correct the GitHub workflow about code coverage.

Code coverage check in GA is OK now. We can merge as soon as the code review is done.