Add ZkEVM

[Insert URL to the list here]
https://github.com/LuozhuZhang/awesome-zkevm
[Explain what this list is about and why it should be included here]
A curated list of awesome zkEVM resources, libraries, tools and more. zkEVM is a Fully EVM compatible and zk friendly virtual machine.
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
Requirements for your pull request

Don't waste my time. Do a good job, adhere to all the guidelines, and be responsive.
You have to review at least 2 other open pull requests.
Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration. Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions.
You have read and understood the instructions for creating a list.
This pull request has a title in the format Add Name of List.

✅ Add Swift
✅ Add Software Architecture
❌ Update readme.md
❌ Add Awesome Swift
❌ Add swift
❌ add Swift
❌ Adding Swift
❌ Added Swift

Your entry here should include a short description about the project/theme of the list. It should not describe the list itself. The first character should be uppercase and the description should end in a dot. It should be an objective description and not a tagline or marketing blurb.

✅ - [iOS](…) - Mobile operating system for Apple phones and tablets.
✅ - [Framer](…) - Prototyping interactive UI designs.
❌ - [iOS](…) - Resources and tools for iOS development.
❌ - [Framer](…)
❌ - [Framer](…) - prototyping interactive UI designs

Your entry should be added at the bottom of the appropriate category.
The title of your entry should be title-cased and the URL to your list should end in #readme.

Example: - [Software Architecture](https://github.com/simskij/awesome-software-architecture#readme) - The discipline of designing and building software.

The suggested Awesome list complies with the below requirements.

Requirements for your Awesome list

Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
Don't open a Draft / WIP pull request while you work on the guidelines. A pull request should be 100% ready and should adhere to all the guidelines when you open it. Instead use #2242 for incubation visibility.
Run awesome-lint on your list and fix the reported issues. If there are false-positives or things that cannot/shouldn't be fixed, please report it.
The default branch should be named main, not master.
Includes a succinct description of the project/theme at the top of the readme. (Example)

✅ Mobile operating system for Apple phones and tablets.
✅ Prototyping interactive UI designs.
❌ Resources and tools for iOS development.
❌ Awesome Framer packages and tools.

It's the result of hard work and the best I could possibly produce.
If you have not put in considerable effort into your list, your pull request will be immediately closed.
The repo name of your list should be in lowercase slug format: awesome-name-of-list.

✅ awesome-swift
✅ awesome-web-typography
❌ awesome-Swift
❌ AwesomeWebTypography

The heading title of your list should be in title case format: # Awesome Name of List.

✅ # Awesome Swift
✅ # Awesome Web Typography
❌ # awesome-swift
❌ # AwesomeSwift

Non-generated Markdown file in a GitHub repo.
The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
Not a duplicate. Please search for existing submissions.
Only has awesome items. Awesome lists are curations of the best, not everything.
Does not contain items that are unmaintained, has archived repo, deprecated, or missing docs. If you really need to include such items, they should be in a separate Markdown file.
Includes a project logo/illustration whenever possible.

Either centered, fullwidth, or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to maximum half the width of the original image.

Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
Includes the Awesome badge.

Should be placed on the right side of the readme heading.

Can be placed centered if the list has a centered graphics header.

Should link back to this list.

Has a Table of Contents section.

Should be named Contents, not Table of Contents.
Should be the first section in the list.
Should only have one level of nested lists, preferably none.
Must not feature Contributing or Footnotes sections.

Has an appropriate license.

We strongly recommend the CC0 license, but any Creative Commons license will work.

Tip: You can quickly add it to your repo by going to this URL: https://github.com/<user>/<repo>/community/license/new?branch=main&template=cc0-1.0 (replace <user> and <repo> accordingly).

A code license like MIT, BSD, Apache, GPL, etc, is not acceptable. Neither are WTFPL and Unlicense.
Place a file named license or LICENSE in the repo root with the license text.
Do not add the license name, text, or a Licence section to the readme. GitHub already shows the license name and link to the full text at the top of the repo.
To verify that you've read all the guidelines, please comment on your pull request with just the word unicorn.

Has contribution guidelines.

The file should be named contributing.md. Casing is up to you.
It can optionally be linked from the readme in a dedicated section titled Contributing, positioned at the top or bottom of the main content.
The section should not appear in the Table of Contents.

All non-important but necessary content (like extra copyright notices, hyperlinks to sources, pointers to expansive content, etc) should be grouped in a Footnotes section at the bottom of the readme. The section should not be present in the Table of Contents.
Has consistent formatting and proper spelling/grammar.

The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
The description starts with an uppercase character and ends with a period.
Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

Doesn't use hard-wrapping.
Doesn't include a Travis badge.You can still use Travis for list linting, but the badge has no value in the readme.
Doesn't include an Inspired by awesome-foo or Inspired by the Awesome project kinda link at the top of the readme. The Awesome badge is enough.

Go to the top and read it again.

hi!
On Tue, 07 Jun 2022 02:24:02 -0700
Luozhu @.***> wrote:

<!-- Congrats on creating an Awesome list! 🎉 -->
<!-- Please fill in the below placeholders -->
[Insert URL to the list here]
https://github.com/LuozhuZhang/awesome-zkevm
[Explain what this list is about and why it should be included here]
A curated list of awesome zkEVM resources, libraries, tools and more. zkEVM
is a Fully EVM compatible and zk friendly virtual machine.

I do not know what "EVM" and "zk" are.
--
Shlomi Fish       https://www.shlomifish.org/
https://www.shlomifish.org/humour/bits/facts/Taylor-Swift/
If God exists and is the ego‐maniacal, sadistic and helpless creature that is
described in the Old Testament, then we’re in deep trouble.
— https://www.shlomifish.org/humour.html
Please reply to list if it's a mailing list post - https://shlom.in/reply .

hi!
On Tue, 07 Jun 2022 02:24:02 -0700 Luozhu @.***> wrote: <!-- Congrats on creating an Awesome list! 🎉 --> <!-- Please fill in the below placeholders --> [Insert URL to the list here] https://github.com/LuozhuZhang/awesome-zkevm [Explain what this list is about and why it should be included here] A curated list of awesome zkEVM resources, libraries, tools and more. zkEVM is a Fully EVM compatible and zk friendly virtual machine.
I do not know what "EVM" and "zk" are.
…
-- Shlomi Fish https://www.shlomifish.org/ https://www.shlomifish.org/humour/bits/facts/Taylor-Swift/ If God exists and is the ego‐maniacal, sadistic and helpless creature that is described in the Old Testament, then we’re in deep trouble. — https://www.shlomifish.org/humour.html Please reply to list if it's a mailing list post - https://shlom.in/reply .

hi! Thanks for your attention
1）You should know Ethereum! A blockchain that can run all smart contracts (applications), everyone compares it to "a global computer", the OS of this computing is EVM (the full name is ethereum virtual machine)
This article is good for u to understand EVM!
2）The full name of zk is zero-knowledge-proof, which is a cryptographic concept, we don't need to understand that deep :)
We only need to know that EVM is undergoing an upgrade. In order to carry more users, zkp technology is used. This awesome-zkEVM repository is a summary of these technologies and resources

Will fix it

Bump


Unable to locate Guild Wars 2 /bin

Hi,
Running v1.4 Alpha. I don't have GW2 installed in the default folder and the program doesn't detect the installation location. This doesn't happen with v1.2.2

I had the same issue. Workaround is to set the custom path ("Select Directory" button) and then restart the addon manager.

Your game path appears wrong ("GamesD"?), so that's probably the reason. Please report back if the issue persists.

Issue is that _configurationManager.UserConfig.GamePath gets updated when changing the Game path in the OpeningViewModel, but the _configurationManager.UserConfig.BinFolder doesn't. So setting the path, closing the manager and re-opening it fixes it since that updates the _configurationManager.UserConfig.BinFolder variable when it gets instantiated.
configuration.DetermineSystemType() should probably be called in the update_button_clicked before the check, rather than in OpeningView()

Your game path appears wrong ("GamesD"?), so that's probably the reason. Please report back if the issue persists.

Not wrong, it's the name of the folder

I had the same issue. Workaround is to set the custom path ("Select Directory" button) and then restart the addon manager.

Issue is that _configurationManager.UserConfig.GamePath gets updated when changing the Game path in the OpeningViewModel, but the _configurationManager.UserConfig.BinFolder doesn't. So setting the path, closing the manager and re-opening it fixes it since that updates the _configurationManager.UserConfig.BinFolder variable when it gets instantiated.
configuration.DetermineSystemType() should probably be called in the update_button_clicked before the check, rather than in OpeningView()

It worked, thanks! I still think this should be considered a bug because it shouldn't be necessary to restart the addon manager, right?

I had the same issue. Workaround is to set the custom path ("Select Directory" button) and then restart the addon manager.

Issue is that _configurationManager.UserConfig.GamePath gets updated when changing the Game path in the OpeningViewModel, but the _configurationManager.UserConfig.BinFolder doesn't. So setting the path, closing the manager and re-opening it fixes it since that updates the _configurationManager.UserConfig.BinFolder variable when it gets instantiated.
configuration.DetermineSystemType() should probably be called in the update_button_clicked before the check, rather than in OpeningView()

It worked, thanks! I still think this should be considered a bug because it shouldn't be necessary to restart the addon manager, right?

Yup, I've created a pull requested with a suggested fix for it for Friendly0Fire to review.

The Redesign definitely won't have that issue, I can assure you that much. Stay tuned!


refactor(transformer): arrow function transform: remove unnecessary assertion

This assertion was required previously, but now SparseStack::pop makes it impossible for self.this_var_stack.len() to be 0. Being exactly 1 is not a safety invariant.

#6002  👈
main

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

Join @overlookmotel and the rest of your teammates on  Graphite

My mistake retaining this assertion, so merging without review.


Discord update broke urgent color

The latest Discord update seems to have broken the $colorUrgent. My urgent color is #81abd9 and appears in the css generated yet does not appear in the client. Other colors do not seem to work either. Neither BetterDiscord or BeautifulDiscord work.

I'm experiencing this problem as well

Thanks for the report. I'm a little busy right now so I can't fix it right away. If anyone finds a way to solve it, PRs are always welcome.

I can fix it but I don't know how it looked before so I need a screenshot or a better description. Either that or you can go to any server in your browser on https://discordapp.com, right click whatever is supposed to be painted and select "Inspect element", then go up the DOM tree until you find the correct class witch CSS colour on it and paste it here.

been trying to work on this but the inspection tool turns into a blueish rectangle as soon as it turns up, does anyone know how to solve it or have the same issue?

Well, nope. Maybe when you turn on the developer mode in options? I don't know much about electron. I checked class names on discordapp.com last time I sent a PR.

Nope, I do have developer mode turned on. Can't seem to find a way to fix it, I guess I'll wait for a discord update or something.


Exchange / OneDrive/SharePoint Links not working

Hi,
I've not undertaken a lot of troubleshooting yet, but I thought I'd quickly ask before I start.
I've successfully installed this on a Linux (CentOS 7) box, and I'm able to connect successfully to my VBO365 server etc.
All of the items on the menu to the left are working (Jobs/Org/Repo/proxy etc), and I can see jobs, run them etc.
When I click on any of the menu items at the top (Exchange/OneDrive/SharePoint) I get a 404/file not found. The link appears to be /exchange, rather than the exchange.php file I can see.
As a quick test, I try manually going to exchange.php, and it loads. I then see the tenant link on the left. If I click on it I get another 404/file not found. This time the URL is /exchange/XXXXXXX-XXXXX-XXXXXX where the XXXX are the org ID.
Have I missed a line of config somewhere, or is this a bug?

Hi,
This is related due to htaccess controlling how the pages are rewritten. By default this is not enabled on Apache/httpd on Centos and needs to be modified. Please follow the guide on https://devops.profitbricks.com/tutorials/install-and-configure-mod_rewrite-for-apache-on-centos-7/ and let me know if this solves it.
I will make a note on this and improve the readme file. Thanks!

Perfect - thanks for the quick response, and that's fixed it. I'd missed one step in that process.

Hello Niels
We have got the portal working but when we try to assess a users mailbox to restore we get an error:
The requested URL /exchange/xxx is not found on this server.
I believe this is related to certificates or .htaccess but cannot find enough info to work out which one.
In REST log file
29/03/2019 12:29:04    8 (8668) Certificate claims:
29/03/2019 12:29:04    8 (8668)   Veeam Software
29/03/2019 12:29:04    8 (8668) Validation failed. Certificate is not valid for host localhost
I believe this relates to the Veeam O365 Server rather than the portal server.
I have regenerated new certificates under General Options and used it for both REST API and Authentication Tabs in General Options – Have restarted services but the problem persists.
Any ideas re the certificates or is there an entry I need to make in .htaccess?
Many Thanks
Clive

Clive, the htaccess which you should leverage for the rewrite rules is available in the base repository: https://github.com/nielsengelen/vbo365-rest/blob/master/.htaccess
This file rewrites everything as it should be (Exchange, OneDrive & SharePoint).
The REST log is normally just a warning due to using a self-signed certificate.

Niels
Many thanks
I have used that and success at last.
Best regards
Clive
Clive Harris
Managed Services Consultant
Covenco Recovery Services
Mobile: +44
Direct Dial: +44 (0)01295 274937
Fax: +44 (0)1753 732500
Email: clive.harris<EMAIL_ADDRESS>For more information about Covenco Recovery Services please visit our website at http://www.covenco-dr.co.ukhttp://www.covenco-dr.co.uk/
Follow us on: LinkedInmhtml:file://C:/Users/User/AppData/Roaming/Microsoft/Signatures/Clive Harris.mht!https://www.linkedin.com/company/1228295?trk=tyah&trkInfo=clickedVertical%3Acompany%2CentityType%3AentityHistoryName%2CclickedEntityId%3Acompany_1228295%2Cidx%3A0 | Twittermhtml:file://C:/Users/User/AppData/Roaming/Microsoft/Signatures/Clive Harris.mht!https://twitter.com/CovencoDR
[819EC020]
[85F1BB2E]   [60027F6C]
Company Registration Number: 3059168
Head Office Address:  Unit 4, MXL Centre, Lombard Way, Banbury, OX16 4TJ
This e-mail communication does not create or vary any contractual relationship between Covenco Recovery Services Limited (CRS) and you. Internet communications are not secure and accordingly CRS does not accept any legal liability for the contents of this message. The contents of this e-mail are confidential to the intended recipient at the e-mail address to which it has been addressed. It may not be disclosed to or used by anyone other than this addressee, nor may it be copied in any way. If received in error, please contact CRS on the above office number quoting the name of the sender and the addressee and then delete it from your system. Please note that neither CRS nor the sender accepts any responsibility for viruses and it is your responsibility to scan the e-mail and attachments (if any).
Registered Address: 5 South Parade, Summertown, Oxford, OX2 7JL, United Kingdom
From: nielsengelen<EMAIL_ADDRESS>Sent: 29 March 2019 15:51
To: nielsengelen/vbo365-rest<EMAIL_ADDRESS>Cc: Clive Harris<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [nielsengelen/vbo365-rest] Exchange / OneDrive/SharePoint Links not working (#8)
Clive, the htaccess which you should leverage for the rewrite rules is available in the base repository: https://github.com/nielsengelen/vbo365-rest/blob/master/.htaccess
This file rewrites everything as it should be (Exchange, OneDrive & SharePoint).
The REST log is normally just a warning due to using a self-signed certificate.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/nielsengelen/vbo365-rest/issues/8#issuecomment-478049258, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AusL2Z9ydaE5ljU5mJ9KukemjXF9iE4Lks5vbjZxgaJpZM4Wxy6h.


Add syntax specification to all .proto files

Newest protoc complains if the .proto file does not have a syntax specification:
[libprotobuf WARNING google/protobuf/compiler/parser.cc:492] No syntax specified for the proto file. Please use 'syntax = "proto2";' or 'syntax = "proto3";' to specify a syntax version. (Defaulted to proto2 syntax.)
Should add this line to all files in nanopb.

Fix released in nanopb-0.3.4.

At bc2c47e I get:
`         ********************************************************************
*** Failed to import the protocol definitions for generator.     ***
*** You have to run 'make' in the nanopb/generator/proto folder. ***
********************************************************************
Traceback (most recent call last):
File "/Users/marco/dev/exqbe/Bridge/nanopb/../../Vendor/nanopb/generator/nanopb_generator.py", line 33, in
import proto.nanopb_pb2 as nanopb_pb2
File "/Users/marco/dev/exqbe/Vendor/nanopb/generator/proto/nanopb_pb2.py", line 26, in
dependencies=[google_dot_protobuf_dot_descriptor__pb2.DESCRIPTOR,])
TypeError: init() got an unexpected keyword argument 'syntax'`
This is with protobuf 2.6.1 and 3.1.0. I have tried to clean and re-done make in 'generator' after switching versions.

@denravonska Sounds like your protoc version and python protobuf version are mismatched.
You can try to check that it is imported from the place where you expect:
python -c 'import google.protobuf; print(google.protobuf.file)'
If you have both protobuf 2.6.1 and 3.1.0 installed on your system, you have to be especially careful to keep the python path matched to the protoc version you use.

Aaaah, I had 2.6 installed via pip as well. It took precedence over the one from Homebrew.


Project and Visio Pro/Sts 2019 via SCCM

hi,
i was wondering if you can create separate packages for Project and Visio 2019 Std and Pro.
As now if we want to deploy those apps via SCCM we have for each application to include all Office source files around 2.1GB - 3 GB  .
Or you can suggest a best practice solution for  those scenarios
thanks in advance
Nick

Hi Nick,
The source files for Visio and Project are bundled together with the rest of Office.  I recommend managing a single location that contains all the Office source files for the architecture and channel you are deploying as well as all the languages you plan to support.  Then, within this location, create separate configuration XML files for each of the different configurations, i.e. Office only, Visio only, Office w/ Visio.
Cheers,
Chris Hopkins
Microsoft - Office Deployment

hi Chris and thanks for the quick reply.
What you are suggesting is ok but the problem is that we have  a lot of remote offices.So one single location as a source it is not a very good idea for us. What we need is to use Software Center and distribute the files to the local DPs .The problem is that we cannot create separate "Applications"  in CM with separate configuration XML files and in the same time use a common source file .So for each one of them  i.e. Office,Visio Std and Pro, Project Std and Pro we need to create a separate application using the same source files multiple times ,occupying more space on the servers and creating unessacary traffic during distribution and installation,
I hope you understand my concerns.
thanks
Nick
Nick

Hi Nick,
If you don't mind maybe we can connect on a call to discuss.  My email is<EMAIL_ADDRESS>

Closing the issue since Chris & Nick will be taking this discussion offline.


Search box incorrectly encodes special characters

When I use the search box in the upper right corner to search for a phrase that contains special characters (including spaces), those special characters are showing encoded in the same search box on the search results page.
For example, when I search for "overloading equals()", I'm correctly sent to https://docs.microsoft.com/en-us/search/index?search=overloading+equals()&scope=.NET, but the search box on that page now says "overloading+equals%28%29". This means I can't easily refine my search and try again if the results are not what I expected.

Thank you @svick. I've open an internal issue for us to track this 1016988

@svick I tried this today, and it appears to be fixed.  Can you verify and close if so.

Yeah, this seems to be fixed, closing.


Ability to inject existing target ref

Thank you for all of your hard work on a wonderful library!
The old Popper interface allowed the consumer to inject their own target and floating element refs.
We have a FloatingBox component that wraps the hook and want to inject an external pre-existing ref for a target element (which comes from a prop).
Is there a way to invert control of the ref to the consumer? I notice the arrow middleware does this.
https://github.com/floating-ui/floating-ui/blob/5c07610beac68b380850d421bdc1a96681b49bb9/packages/react-dom/src/index.ts#L95

No problem - i'll create a quick example later.

See https://github.com/floating-ui/floating-ui/issues/1503#issuecomment-1011975726. Calling reference achieves this and doesn't have the update() drawback.
Added to the docs: https://floating-ui.com/docs/react-dom#external-reference


Frontend : HTML Editors

Description 📜
I want to add Documentation of HTML Editors.
Assign this issue to me.

Domain of Contribution 📊

[X]  Frontend Dev HTML/CSS/JS

/assign

/assign

/assign

I  also want to add documentation of HTML Editors. Kindly Assign this issue to me.

I created the issue and i self assigned it to myself

/assign


Guidance needed on required inputs and on interpretation of outputs

--- It is very nice to see all these tools integrated in one environment. Hopefully all tools referred by Digitaf (youtube channell) would find a place here!
--- comment for all tools: scale and units unclear on the graphic that results after clicking "Compare trees !"; it would benefit all tools if each would have a link to a short manual to help understanding required input and to help interpretation of results.

Thank you Jo for your feedback!
--- Regarding your first remark: the objective of AgroforesTreeAdvice is to integrated the tools for tree (and crop) species selection, not all tools in the DigitAF catalog. But for some of the other tools, we are setting up other working groups to do the same kind of integration: we have started working on Financial tools, and we might look at biodiversity tools afterwards. You might also be interested in the carbon tool (https://github.com/euraf/OpenFarmCarbonTracker)
--- Regarding your second remark: thank you for this remark: yes, it was unclear (actually the score is dimensionless because it is a sum of different number of criteria in the different tools => following your remark we simply deleted the axis (will be implemented in the next version). (https://github.com/euraf/agroforestreeadvice/commit/2ea47b4eaf4c428f4889a6bb5f6116fd65a01aff)

Regarding the need for a manual, we won't write a manual in this repository, which is supposed to be just the crossroads to all the individual tools (which will be referenced in the "databases" page (not yet finished). Hopefully each individual tool will have its own documentation (having a user manual is one of the criteria to obtain a high FAIRness score, which is the overall aim of all DigitAF tools). To read more about the FAIRness score, see this: ... well actually the doc is not yet published online, I asked the developpers to publish it.


feat(opendataset): add dataloader for SegTrack dataset

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6065

Relevant Lines:
7027

💛  - Coveralls


Fixed #28687 -- Added a 'Not Empty' option to admin's related filter.

Hi @carltongibson,
Thank you so much for the comments.

I moved it next to Empty option.
I added relase note for it.
I added test for it.

In my point of view there is no need to an equivalent of self.empty_value_display, but please make final decision about it.

@carltongibson thanks again.
I wrote a better comment for the test. and also change the selected condition. so the test selected assert changed too.

@carltongibson , thanks.
I added the option for RelatedFieldListFilter.

@carltongibson, yes I will add this as soon as possible.

@carltongibson , I added get_not_empty_value_display().

@carltongibson is there anything left?

Hi @hramezani — Only that I need to have another look. (I will do that this week.) Thanks.

Hi @carltongibson,
Thank you so much for the comments.
I added the tests and also changed release note.

Hi again @carltongibson,

Whilst I can see the benefit of the feature, I have a slight reservation about adding yet another admin option for this. I wonder if it wouldn't be better fully encapsulated in a filter subclass?

can you help me about this paragraph?

Hi @hramezani.

can you help me about this paragraph?

Sure.
What I mean is that list_filter [accepts FieldListFilter subclasses directly]. There are already many admin options. A part of me thinks adding an extra one here is just more noise in the API, for (maybe) not so much benefit. That part of me would be inclined to say that users should create their own FieldListFilter subclass if they want this extra behaviour and specify it directly. That way we don't add extra clutter for everyone.
However, the PR itself itself looks OK. I will add my reservation to the ticket and mark it RFC to invite another review. Thanks for your effort.

@timgraham I added missed versionadded.

Did you see my comment on the ticket? I also agree with Carlton's assessment that it's not clear this is a feature that everybody wants. It would be interesting to see what's required to implement this in a subclass.

@timgraham, could you help me, please? I can't understand what is your mean. how can I implement it in a subclass? thank you

The idea is that maybe it would be enough for anyone who wants this feature to subclass RelatedFieldListFilter and override the choices() method to add the extra choice. That way we don't have to make any changes in Django and people who don't want this option won't be bothered by it.

ok, I got it. it seems you can close this pull request

Okay, I closed the ticket as wontfix.


Fix static test bash scripts

The run_tests.sh script runs in the wrong directory and produces errors. However, the bash step doesn't fail on error, so the error doesn't show up.

Fails correctly - good.

@steinimo You were interested in tests, right? Would you like to review my changes to the static build output tests?


s/insert_annotaions_from_mask/insert_annotations_from_mask

Fix typo in spelling of annotations

Is the new notebook meant to be included in this PR?

Yep, @vaughnproscia it calls that method so it's been updated too.

Is the new notebook meant to be included in this PR?

Yep, @vaughnproscia it calls that method so it's been updated too.

Got it 👍 I was expecting like a one line change but git measures the entire file to be changed ha so I wasn't sure. One sec and I'll finish reviewing.


Getting error while installing coolify

I am getting an error after adding the domain name and email address. please check the screenshot below

Can you please re-run the script with the debug option (2)?

Getting the same error with option 2

It's a fresh Digitalocean droplet with Ubuntu 20.04

Thanks. I know what the problem is. I've updated the installer so that it will output the cause - hopefully.
It looks like your domain name cannot be checked against the DNS server.
If you run the following command: dig @<IP_ADDRESS> A <domain> +short but replace  with your actual domain, it should not get back anything (no output).

umm okay, is it possible to use coolify without adding the domain? because my droplet IP is static and it won't change ever...

Atm, it only works with a domain. I’ll check how it could work without it.

You can use nip.io to make domain for IP addresses.

Sorry, but there are too many things that need to be reviewed to use without HTTPS only through an IP address.
Closing now, as the initial issue is solved.


GEODE-3638: cleanup minor issues found while prepping dunit talk

fix some dunit rules
add some dunit rule tests

@kirklund you asked me to hold #778 for this. Are you planning on merging this PR?


Add an option to create glosssary divisions automatically

Given a sorted list of glossentry elements, it should be possible to add divisions. Ideally, ones that can be customized.

With functional programming we would pass a user defined function with signature
function (element(glossentry)*) as element()*

as an argument for a function or template which constructs the glossary content. The result type element()* would allow to leave the sequence of glossentry untouched, or to construct a sequence of glossdiv elements.
Since this is not possible with SAXON HE, we need a substitute. A template in the http://docbook.org/ns/docbook/templates namespace with the simple identity function as default implementation, so that users can customize it?
Greetings, Frank

I think this can be implemented in the code that processes the actual glossary, so it can just be a named template or option that uses for-each-group. It's a little bit like the way glossary sorting is handled. At least I think it is. I could be wrong :-)

Fixed by #462.


Reflector - separate into read and write?

Hi there!
I've been using reflect-cpp for a few weeks, and it's really helping me avoid boilerplate code.  I have an application model, which I'm using reflection to implement a simple CLI where the model can be viewed container hierarchy (with names and types) and values can be retrieved or set. (The CLI works on the desktop and also targeted to embedded devices.)
The recent addition of Reflector support makes customizing serialization/deserialization even easier. However, I have a case where not all types are writable, and it makes no sense to write to them. For example, I have a queue of events, which can be inspected, but not set. (And the queue is not const because parts of the app need to write to the queue.)
One possible way of implementing this is to separate the Reflector into two distinct capabilities - one for reading one for writing. At present, the reflector concept covers both inseparably.
I'd be happy to help submit a PR if these ideas seem like a good way to go.

Hi @m-mcgowan ,
yeah, I agree, this makes a lot of sense and it is also very easy to do.
All you have to do is to split up the has_reflector concept in here into two separate ones (if you could also put them in a separate file internal/has_reflector.hpp, that would be great):
https://github.com/getml/reflect-cpp/blob/main/include/rfl/internal/has_reflection_method_v.hpp
Then you have to insert them in the appropriate places in here.
https://github.com/getml/reflect-cpp/blob/main/include/rfl/parsing/Parser_default.hpp
That's it. A PR is welcome, otherwise I can just do it myself.

And when you do this, if you could also adapt the documentation that would be great:
https://github.com/getml/reflect-cpp/blob/main/docs/custom_parser.md

Thanks for the pointers, I'll give it a shot.

Resolved.


docs: Typo in list doc

Type: docs
Platform: all
At CSS List, http://ionicframework.com/docs/components/#list, second paragraph, there is a typo:
The list view is a very versitile and powerful component.
should be
The list view is a very versatile and powerful component.
Thanks,
Pierre

Thanks!


Convert JSON to Parquet directly

In order to convert JSON payload to Parquet, the "established" way seems to be to go first through a JSON to Avro conversion, provided the JSON payload is covered by a JSON-Schema or part of a OpenAPI definition. The rest of the process relies on the parquet-avro implementation.
Since JSON payload can be loaded to a generic JsonNode (jackson library) and since a JSON payload can be described and checked against a JSON-Schema or an OpenAPI definition, wouldn't it be theoretically possible to convert a JSON payload to a Parquet "record" directly (somehow implementing a similar approach than parquet-avro but this a JsonNode as a data holder)?
Reporter: Emmanuel Brard / @ebrard
Note: This issue was originally created as PARQUET-1880. Please see the migration documentation for further details.

Emmanuel Brard / @ebrard:
FYI, I am working on an implementation with Jackson JsonNode as the data support and an OpenAPI schema definition to carry the schema information.


Client sessions change dimensions of host terminal.

Hi, thank you for the great project.
I noticed that client sessions, including read-only sessions, affect dimensions of host terminal. (https://github.com/owenthereal/upterm/blob/master/host/internal/server.go#L253)
In my case it shouldn't be like that but I wonder if it's something that's expected by other use cases. (e.g. tmux based sessions).

👋 You are correct. The dimensions of the host pty change as a new client joins the session.  The current algorithm is that the host pty will be set to the smallest width & smallest height of all clients + the host. This might be problematic to the host if a client with a very small screen size joins.
What would you expect the size of the host would be in your use cases?


Why does it not reload the site when changes are made in dev mode?

if i run the repo after doing pnpm install and pnpm ad sharp
when i run pnpm run dev
it should hot reload like any other astro template so i can actually see the changes live
how can i make it hot reload as in reload when i make changes to the code?

The astro dev command run by pnpm run dev supports hot reload and should be enabled by default.  Or is it something like --watch?

@L4Ph
It still doesnt hot reload for me even if I use --watch
can you confirm it works for you as in it hot reloads when you make changes with --watch?

@golam71
I have confirmed that HMR do not working.
Currently, I'm in the middle of debugging, so I can't say for sure, but it appears that --watch is compiling a large amount of CSS from <style></style> tags written within .astro files.
I suspect this is the cause, but it may take some time to investigate.

I succeeded in activating it.

@golam71
It seems just as I imagined.  I believe Tailwind CSS is fundamentally supposed to be written through base.css, which is then compiled.  Writing it within <style></style> caused some unusual compilation.
Please try this branch when you have time.  I will issue a PR then.

pls me assign @saicaca

Incredible guys, i was struggling to find the cause and wow you already found the cause

It has been incorporated into main.
Is it okay to close this issue?
@saicaca @golam71

It works
thank you very much :)


All loot tables breaking when CraftTweaker is installed

Minecraft version
1.20.1
Farmer's Delight version
2.0.7
Forge version
Fabric 0.15.7 (this needs to be changed on the issue template lol)
Description
All loot tables return empty when FDR is installed alongside CraftTweaker.
Steps to reproduce
Install CraftTweaker
use /loot give @p loot minecraft:blocks/cobblestone
Observe no cobblestone is dropped
See also https://github.com/Fabricators-of-Create/Porting-Lib/issues/74
Mod list
Amendments
CraftTweaker
FAPI
FDR
Moonlight Lib
Supplementaries
Supplementaries Squared
Logs
No response
Minimal instance

[X] I have tested this on a minimal instance

Performance and shader mods

[ ] I am using performance or shader mods

I'll see if Porting Lib has an update.

Yup. We were out of date, I'll update the bundled porting lib asap.


Documentation issues when upgrading to Doxygen 1.9.8

The documentation for this SDK builds fine with Doxygen 1.8.17 (as indicated by the comment in the DoxygenLayout.xml) but has at least two issues when upgrading to the latest version of Doxygen: 1.9.8, released on 25 Aug 2023.
The "API Documentation" tab is missing from the main menu and no modules.html is output

Doxygen 1.8.17
Doxygen 1.9.8

Using the latest default DoxygenLayout.xml produced by Doxygen 1.9.8's doxygen -l restores the contents of the old modules.html but now refers to it as "Topics" instead. Updating our DoxygenLayout.xml restores the previous page (albeit under a new URL):
-    <tab type="modules" visible="yes" title="API Documentation" intro="These are the libraries supplied in the Raspberry Pi Pico SDK"/>
+    <tab type="topics" visible="yes" title="API Documentation" intro="These are the libraries supplied in the Raspberry Pi Pico SDK"/>
+    <tab type="modules" visible="no" title="" intro=""/>

Brief descriptions are no longer extracted for most modules by default

Doxygen 1.8.17
Doxygen 1.9.8

The latter can be fixed by adding JAVADOC_AUTOBRIEF = YES to the Doxyfile to opt into the following functionality:

If the JAVADOC_AUTOBRIEF tag is set to YES then doxygen will interpret the first line (until the first dot) of a Javadoc-style comment as the brief description. If set to NO, the Javadoc-style will behave just like regular Qt-style comments (thus requiring an explicit @brief command for a brief description.)

Alternatively, we may need to switch to using explicit @brief commands to separate brief descriptions from detailed ones.

As commented in https://github.com/raspberrypi/pico-sdk/pull/1564#issuecomment-1932279064, I'm seeing incorrectly extracted brief descriptions in newer versions of Doxygen (e.g. 1.10.0) even when we're using the explicit \brief command if it isn't separated from the detailed description by a blank line, see https://github.com/raspberrypi/pico-sdk/blob/6a7db34ff63345a7badec79ebea3aaef1712f374/src/rp2_common/hardware_spi/include/hardware/spi.h#L100-L103

In the short term, maybe it's worth defining what version of Doxygen is supported and we advise using that. I believe the latest version that behaves the way we expect is 1.8.17 (before the changes to brief description extraction)?

See #1660.

Hi,
Is lack of backward compatibility in doxygenlayout .xml file expected?
I have updated doxygen locally but I cannot update doxygen in our CI currently and ended up with errors from older doxygen version in CI.
Also if there are breaking changes introduced IMO it requires changing format version and providing meaningful error if version is not supported/compatible: both versions are using layout 1.0

It's not so much a lack of backwards compatibility in the XML layout, as in the parser. Layout that parsed just fine with older versions of the library don't parse correctly with newer versions which seem to be a lot stricter in what it regards as valid markup.

this is likely fixed with the latest sdk release


Adding new MDO related queries from recent Microsoft Threat Intellige…

Adding new MDO related hunting queries from the recent Microsoft Threat Intelligence blog- File hosting services misused for identity phishing.
https://www.microsoft.com/en-us/security/blog/2024/10/08/file-hosting-services-misused-for-identity-phishing/
Required items, please complete
Change(s):

Add 4 new hunting queries relevant to MDO to identify phishing attempts

Reason for Change(s):

New hunting content from Microsoft Threat Intelligence

Testing Completed:

Tested the queries in Defender XDR Advance hunting

Checked that the validations are passing and have addressed any issues that are present:

Yes

Hi @damozes1,
Please add the new hunting queries to the data file and repackage this solution using the V3 tool. You can refer to this https://github.com/Azure/Azure-Sentinel/blob/master/Tools/Create-Azure-Sentinel-Solution/V3/README.md. Thanks...!!


My code gives the error "Please check if you have Photoshop installed correctly."

Hello,
I'm trying to collect information from a user using telegram chat (pyTelegramBotAPI) so I can transfer this to photoshop inside the function in the future, but when I use the bot, the code gives the error "Please check if you have Photoshop installed correctly." Although if you run the same, but not through the bot everything will run and work fine.
I tried - get a newer version of photoshop (CC2018 -> 2020), and also specify the path in the Windows environment variables. If you open photoshop using pyWin32 it works fine, but it's very unprofitable for me, I was going to use only pythonPhotoshopAPI.
Here's the function
def test_edit_text(info_from):
try:
    psApp = ps.Application()
    psApp.Open(r"mypath\first.psd")
    doc = psApp.Application.ActiveDocument
    print(info_from['test'])

    text_from_user = info_from['test']
    layer1init = doc.ArtLayers["layer1"]
    text_new_layer1 = layer1init.TextItem
    text_new_layer1 .contents = f"{text_from_user .upper()}"

    options = ps.JPEGSaveOptions(quality=5)
    jpg = r'mypath\photo.jpg'
    doc.saveAs(jpg, options, True)
except Exception as e:
    print(e)

If you run the function like this, nothing works
@bot.message_handler(content_types=['text'])
def func(message):
    if(message.text == '/ph'):    #Here we send a request to the function via telegram chat
        user_info = {'test' : 'asdsaasd'}
        test_edit_text(user_info)

But if you run it like this, it works fine
if __name__ == '__main__':
     user_info = {'test' : 'asdsaasd'}
     test_edit_text(user_info) `

Why does the code suddenly stop seeing Photoshop? Is it possible that the two libraries are incompatible?

Hi @emogothyy
We only have two places that have Please check if you have Photoshop installed correctly.
https://github.com/loonghao/photoshop-python-api/blob/main/photoshop/api/_core.py#L43
https://github.com/loonghao/photoshop-python-api/blob/main/photoshop/api/_core.py#L97
This API then relies on having Photoshop installed locally,
You can try to open Photoshop manually first, and then start your bot server to test.
If possible, please take a screenshot of your running process so that we can take a look

Why does the code suddenly stop seeing Photoshop? Is it possible that the two libraries are incompatible?

I checked that the dependencies between the two sides are not in conflict.

@loonghao Thanks a lot! It really helped me to open Photoshop first with import os and then everything started working properly.
I'll attach screenshots now. If necessary, I can send all the files.

Thanks again

@emogothyy
By default, If your Photoshop is not opened, the PYPS API will automatically open Photoshop when it is executed.
You can try tweaking your code a bit, similar to the following.
# main.py

from photoshop import Session

@bot.message_handler(content_types=['text'])
def func_a(message):
    if(message.text == '/ph'):
        with Session() as ps:
            user_info = {'test' : 'func_a'}
            test_edit_text(user_info, ps_api=ps)

@bot.message_handler(content_types=['text'])
def func_b(message):
    if(message.text == '/ph'):
        with Session() as ps:
            user_info = {'test' : 'func_b'}
            test_edit_text(user_info, ps_api=ps)

# photoshop_function.py
def test_edit_text(info_from, ps_api):
    try:
        psApp = ps_api.Application()
        psApp.Open(r"mypath\first.psd")
        doc = psApp.Application.ActiveDocument
        print(info_from['test'])
        text_from_user = info_from['test']
        layer1init = doc.ArtLayers["layer1"]
        text_new_layer1 = layer1init.TextItem
        text_new_layer1 .contents = f"{text_from_user .upper()}"

        options = ps.JPEGSaveOptions(quality=5)
        jpg = r'mypath\photo.jpg'
        doc.saveAs(jpg, options, True)
    except Exception as e:
        print(e)

I tried this method, but these places still have the same error, maybe the problem is in my photoshop, even after reinstalling, but if as before open with os or just open, everything works fine. Thanks so much for helping me with this error. Can I close the issue as solved?


About router training

Hi! Thank you for the great work!
Are you planning to release the code for router training?

Hi! Sorry for the delay. I’ve updated the Router training process, and you can find the updated version in the following notebook: Router_Train.ipynb. Feel free to check it out!

Thanks!


[Test] Disable some failing tests on ARM64e.

Disabling these tests:
IRGen/ptrauth-foreign.sil
IRGen/ptrauth_field_fptr_import.swift
Interop/SwiftToCxx/class/swift-class-virtual-method-dispatch-arm64e.swift
Interop/SwiftToCxx/class/swift-class-virtual-method-dispatch.swift
Interop/SwiftToCxx/class/swift-subclass-of-resilient-class-virtual-method-dispatch.swift

@swift-ci please test and merge

Thanks, will fix them in a follow-up.


Quick TODO Fixes

There are 10 instances of TODO in the current docs:

Seems like a quick sprint could help resolve these issues and remove the TODOs

Porting to https://github.com/substrate-developer-hub/substrate-docs/issues/158


how to connect a vdi to a vm?

I am new to terraform and xen server so forgive me if this is a noob question.  I would like to use terraform and xen server to create a new VM with two separate disks - one for the OS/Application and one for the application data.  My VM template already has the OS disk connected to it so I think I have that covered.  I am also able to create a VDI using xenserver_vdi, but I don't know how to attach that VDI to the VM and mount it say /data.  Any ideas how to do this?

To add the VDI to a VM you would do something like:
  hard_drive {
    vdi_uuid = "${xenserver_vdi.data.id}"
    mode = "RW"
  }

In the VM resource block where data is the identifier given to the VDI you're creating.
To mount it within the VM you would have to do some provisioning after Terraform, for example with Ansible.

I added this, but the VDI is not attached to the VM, here is what my tf file looks like (note that the template has the OS hard drive attached to it):
resource "xenserver_vdi" "xxx-data-vdi" {
sr_uuid = "4758064c-924a-c671-9c26-8321b6c207c8"
name_label = "xxx-data-vdi"
size =<PHONE_NUMBER>4 # 20GB
}
resource "xenserver_vm" "xxx-vm" {
base_template_name = "Ubuntu 16.04 Template"
name_label = "xxx-ubuntu-vdi"
static_mem_min =<PHONE_NUMBER>
static_mem_max =<PHONE_NUMBER>
dynamic_mem_min =<PHONE_NUMBER>
dynamic_mem_max =<PHONE_NUMBER>
vcpus = 1
boot_order = "c"
hard_drive {
is_from_template = true
user_device = "0"
} # Template VM HDD
hard_drive {
is_from_template = false
user_device = "4"
vdi_uuid = "${xenserver_vdi.xxx-data-vdi.id}"
mode = "RW"
}
cdrom {
is_from_template = true
user_device = "3"
}
}

Further investigation is indicating that no VBD is being created, hence the VDI is not attached to the VM.  Any help would be appreciated

Bug seems to be here:
https://github.com/ringods/terraform-provider-xenserver/blob/master/xenserver/resource_vbd.go#L371
This condition is always passing for my VDI.  I was able to workaround this by changing it to:
                if ud, ok := data[vbdSchemaUserDevice]; ok && ud != "" {
                        log.Println("Skipping VBDm user_device=%s", ud)
                        continue
                }

And not setting user_device.  No idea if this is the right fix, but thought I would post it here

Removing user_device and changing that check is exactly what I did.
I actually already opened a pull request to fix it: #33. Forgot about it when replying previously though.

Nice - hopefully we can that merged soon.  Do you have any documentation about the purpose of user_device?  I am new to xen/terraform so not sure what this is suppose to do.  Thanks

My understanding is that it's just a number (0 to 7) identifying the virtual device on a VM. In this Terraform provider it's only used to specify existing devices from a template. So when you have multiple devices in a template you can select them by the userdevice number.
See also http://xapi-project.github.io/xen-api/classes/vbd.html.

Interesting - how can I tell which user_device is which?  (e.g. how do I know user_device #0 is the xvda1 vs the cdrom?)


NodeId.Key should be of type PublicKey

NodeId.Key holds PublicKey.String()
PublicKey.String() returns BytesToHex - which means it is not the real public key
That could cause bugs if validation will be attempted with this "public key"
Moreover, NodeId.Key should be of type PublicKey and not just a string. That will help us avoid such issues.

closing in favor of #2269


How to add floating button?

Hey thanks for the library.
But i want to use a floating button with this library but unable to do so. Can you guide me?

Hi, thank you for using it and contributing to it!
Depends a lot on what you are trying to implement exactly. As you suggested, I implemented a transition builder example that makes use of the transition to present an icon button with an animation that matches the soft keyboard animation. You can find the example in the project.
Take a look at it and see if it helps (if you haven't already). If the example has nothing to do with what you want to implement, please provide more details about what you are trying to implement exactly, and I will try to reproduce it using the library.

It sort of same but not much... But i had done the required thing using stack and positioned.fill. Thanks for the help though :)

No problem! Happy to read that you managed to implement it!


Make GitHub Actions run tests on Linux and Windows, adjust tests for Linux

Why
@misterspeedy Thank you for creating this amazing library! FsExcel allowed me to quickly create readable code generating Excel sheets and avoid unwanted exposure to low-level libs.
The only problem was that my code base was on netstandard20. I locally resolved it by copying the code and compiling my netstandard20 version ( no code changes needed) but I felt that it would be nice to share this possibility with everyone.
I quickly found out that the current GitHub Actions build does not execute tests and that current tests are not Linux-ready. As I don't feel comfortable with proposing any changes (like the netstandard21 -> netstandard20 switch) without test results here I am with this PR that resolves issues blocking GitHub Action testing on Windows and Linux.
If I get this merged I will come back with the netstandard21 -> netstandard20 change trying to convince you that it is safe based on the tests that didn't get red from the change :)
How

Changed the GitHub Actions build to work on the sln file rather than fsproj of the lib project. Additionally, added Windows build and the steps executed in vs code tasks to ensure that generated files are fresh and up to date.
Added a workaround for the missing Calibri font preventing the opening of Excel files on Linux (https://docs.closedxml.io/en/latest/tips/missing-font.html)
Resolved "/temp" path issues
Ignored enumerate fonts test

(Build results on my fork can be found here https://github.com/gdziadkiewicz/FsExcel/pull/1 )

@gdziadkiewicz I also want the library to target netstandard2.0 (to be useful on .NET Framework), and I faced similar issue for running the tests on macos.
I've made a PR which does only the test adjustments (#53) but without the need to remove the test with fonts, only skipping it if it is not running on windows for now.
I think rebasing this PR on top of #53 to only focus on CI and target would make sense (assuming the tests come green in your environment, without changing any .fs, .fsx, .xlsx files in this branch).

WOW! Thank you so much @gdziadkiewicz  and @smoothdeveloper !  Sorry for the delay but I have now merged the PRs.  I didn't read your comment about rebasing until I had merged the main PR, but hopefully I have reinstated everything to what you both intended.  The test pass on Windows for me.
Thank you so much for these improvements.

@misterspeedy I'm confirming that the changes in my PR are indeed effective, able to run the tests and get those green.
If you could check #54 and publish the library again, I think it should enable the .net framework users to leverage the library.
Thanks for the stylish library!


tutorial-adding-command-fix1

Resolved issue of checkStyle Error
Resolved issue of parsing error for typicalPersonAddressBook.json

Fix: changed "tags" to "tagged" in typicalPersonAddressBook.json. Code is somehow fixed

Cant pass CI :(


Failed to extract bootstrap unc0ver b29

Failed to extract bootstrap 11-12.1.2 unc0ver b29

I'm also having this issue. Gets to step 27 then gives me that notification.

Same here. Any solution for this?

I still haven’t figured it out but sometimes my phone restarts and doesn’t install Cydia, sometimes it just gets stuck on step 27. I’m on an iPhone 6s, iOS 12.1.1 beta 3

Does this still happen on the latest beta?

i have this problem with b40 on iphone 6


enable cruise preview

submits cruise to QC editor with special flag and soon after removes the cruise
from the QC editor

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 20 Apr 2014 at 6:06

Change of plans - preview now only shows plots of the one dataset in the dashboard and does not use the QC editor (LAS) at all.


Create path from uncommitted changes

Use git diff for unstaged changes, otherwise git diff --cached
Command like: git diff --cached >> changes.patch

This feature is not relevant now.


fix: datatype of targetAverageUtilization for Horizontal Pod Autoscaling

For Horizontal Pod Autoscaling,
While using autoscaling based on targetAverageUtilization, in was facing this issue in DevSpace version : 6.0.1
Input:
  replicas: 1
  autoScaling:
    horizontal:
      maxReplicas: 3
      averageRelativeMemory: 40

Error:
ValidationError(HorizontalPodAutoscaler.spec.metrics[0].resource.targetAverageUtilization): invalid type for io.k8s.api.autoscaling.v2beta1.ResourceMetricSource.targetAverageUtilization: got "string", expected "integer"

References:
https://kubernetes.io/docs/tasks/run-application/horizontal-pod-autoscale
targetAverageUtilization requires integer, but string was been given, so have prvided fixes for that.

Fixes: for issue #81


Decide how to handle datetimes in jxml.

Right now it assumes everything is already UTC. This is OK because it's filled only from the database where everything actually is already UTC.
There should be a nicer way to do it. Hooking into variable setting is made more difficult with the weird subclassing I'm doing. There's more to learn there though.

Hey wait did I break episode uploading by removing the type conversion?


feat: refactor kakarot client

Time spent on this PR:
Resolves: #642
Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Testing

What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[x] No

Closing, will rework


Rename tail transaction padding

Will we always have only transactions in the reserved namespaces? For example, what happens when we add ISRs? should we change this name then?
Originally posted by @evan-forbes in https://github.com/celestiaorg/celestia-app/pull/1207#discussion_r1068731238

Proposed alternative names:

TailReservedNamespacePadding
ReservedNamespacePadding

This issue involves a rename to the implementation (i.e. appconsts.go) and the specs (the specs-staging branch)

Closed by https://github.com/celestiaorg/celestia-app/pull/1228


Error during provisioning (docker-machine 0.7.0/a650a40 on Win10 to VCenter/ESXi 5.5)

Attempting to provision a new VM via docker-machine results in the following error:
About to run SSH command:
if [ ! -z "$(ip link show docker0)" ]; then sudo ip link delete docker0; fi
SSH cmd err, output: exit status 255:
Error creating machine: Error running provisioning: Something went wrong running an SSH command!
command : if [ ! -z "$(ip link show docker0)" ]; then sudo ip link delete docker0; fi
err     : exit status 255
output  :

Docker-machine command used is simple/basic :
docker-machine --debug create dockervm01 \
--driver vmwarevsphere \
--vmwarevsphere-username '**REDACTED**' \
--vmwarevsphere-password '**REDACTED**' \
--vmwarevsphere-vcenter **REDACTED** \
--vmwarevsphere-datacenter Concord \
--vmwarevsphere-network 'VM Network' \
--vmwarevsphere-datastore 'p2k/p1_d1_l04' \
--vmwarevsphere-hostsystem '**REDACTED**'

This command is being run from WIndows10 in the docker quickstart console (bash) from docker toolbox, deploying to a 3-node vSphere cluster w/ DRS & SDRS, though I can't imagine any of this matters when it comes to an error in a shell script/command running on the boot2docker VM.

Updated to 0.8.0 and all is well.  Didn't know it was available when opening the ticket.


Some issues with upload

There are two minor issues with the uploader. I observed this when uploading https://zenodo.org/record/5764893#.Ya-hqbso_JU.
Maintainers
The maintainers field is not auto-populated from the rdf.yaml and the name is marked as mandatory but should be optional (we had a very long discussion about this and should stick with the conclusion of it!), see

Tags
When importing the tags from the rdf.yaml they are marked as The tags field format is invalid:

After some removing and re-adding of the tags it works.

Thanks for reporting, it was a fast implementation and the input components are shared with the authors field. I will improve it when I get some bandwidth on it.

@oeway both of these issues still exist and especially the one with the tags is annoying, because it appears although the tags are valid.

@oeway maybe it only happens when importing from zip?! I can easily reproduce it for new models, e.g.

when trying to import this model: https://oc.embl.de/index.php/s/KtXrlvh1IjqzX9D

I am using Mozilla Firefox 96.0 on Ubuntu 20.04

@oeway Ok, I realized now that the tag issue is because some of the tags I used were upper case.
This behaviour is however very very confusing, because the tags listed in https://github.com/bioimage-io/bioimage.io/blob/main/site.config.json#L115-L158 use upper case letters (this is what I used as reference for selecting tags).
I suggest we make this consistent and either make all tags lower case in the site.config.json or allow upper case.

@constantinpape Right, the preconfigured tags are not valid, I have fixed them and also added an auto conversion step to make sure the newly loaded tags are in lower case. The loading of maintainers should also work now, I will set the name the same as github_user for now to avoid more work on the author input UI. Will close this for now and feel free to reopen if there is still more issue.

Thanks! This part of the uploader is working now.


Add trigger when wrapping word in template string

First off, thanks for this extension - started using it today and it seems really useful... but one thing I was surprised didn't work was doing this:
https://user-images.githubusercontent.com/1249597/145416157-927e6941-5598-4f12-beb6-ff9083e4c536.mp4
It works fine if I do it like this:
https://user-images.githubusercontent.com/1249597/145416239-9093e9a1-93db-4c14-b569-fa929640bd69.mp4
But obviously that's a bit more fiddly in terms of the keystrokes/actions required.

Also,

When $ or { is deleted, replace backticks with quotes
Adds brackets to the template string for JSX properties

@truepatch both of those can already happen with settings

@truepatch both of those can already happen with settings

That's exactly how it doesn't work.

Oh you mean when wrapped?


Enhancement: Randomize execution order of tests

This PR

[x] randomizes the execution order of tests

💁♂ For reference, see https://phpunit.readthedocs.io/en/8.4/configuration.html#the-executionorder-attribute.

Thank you, @svpernova09!


Group renaming in the UI is now linked to the ordering

Group renaming in the UI is now linked to the ordering.
Fixes #230

There was an issue created for this, right?

Group descriptions are now also linked

Confirmed working


why the callback "onTouchEnd" is call for drag every time ?

"onTouchEnd" is last page,if you continue drag, it will be call。but now,  drag every time ,it has be call. why?

https://developer.mozilla.org/en-US/docs/Web/Events/touchend

The touchend event is fired when a touch point is removed from the touch surface.


[Feature] Search results page layout

Layout the search results page

Closed as obsolete, this feature was not implemented.


This project has not been completed yet, right? Please go ahead and implement the algorithm.

This project has not been completed yet, right? Please go ahead and implement the algorithm.
Come on! I support you!

only found two


meet trouble when using sourcemaps with gulp-babel and gulp-uglify

Here is my demo code.
https://github.com/fenglinzeng/gulp-sourcemap-babel-error-code
When I use babel and uglify, looks like sourcemap can not locate the correct console code line.
Is there any error of my usage?
thx.

Support questions should be directed to StackOverflow.


Darkmode flags don't work anymore

OS/Platform
Arch Linux
Installed
OS/Platform's package manager
Version
101.0.4951.54
Tested upstream?

[ ] I have tried reproducing this issue in Chrome and it could not be reproduced there
[ ] I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there

Description
The flags --force-dark-mode --enable-features=WebUIDarkMode don't work anymore
How to Reproduce?

Open terminal
Type:  chromium --force-dark-mode --enable-features=WebUIDarkMode
No dark mode :(

Actual behaviour
The browsers starts in lightmode
Expected behaviour
The browser should open in dark mode like it used to do.
Relevant log output
No response
Additional context
No response

Work's on my machine™, are you perphaps using the gtk+ theme on chromium?

I am using Classic. It worked before but not since I updated.

No problems on GTK:

Editing the flags in the browser worked for me. Thanks :+1:

the flags in the browser page (chrome://flags#enable-force-dark) inverts on all pages whereas the --force-dark-mode only inverted chromium's pages and made websites prefer dark mode
This makes it not suitable for using along with dark reader

Does it work in vanilla Chromium?


not working channels are ready in laravel echo

I have a problem i could not resolve.
when I use the program in localhost work perfect but when I move it to the server production it does not work.
I saw when i run laravel-echo-server start in localhost it shows:
✔  Running at localhost on port 6001
✔  Channels are ready.
✔  Listening for http events...
✔  Listening for redis events...
but in production it shows:
✔  Running at localhost on port 6001
✔  Listening for http events...
✔  Listening for redis events...
What could be happening??? why the option Channels are ready. is not working??? thank you

it show just in dev mode, sir.. you can see at line: https://github.com/tlaverdure/laravel-echo-server/blob/96e8a93757109ac0b7cc66cbb789edf03428d9fe/src/channels/channel.ts#L42

Hey there, this issue was opened a while ago. I'm going to close this issue, if this issue still exists, please open a new issue or open a Pull Request.


All Functionality Work but after giving space between two letter but require withought space

Eg.
HiDilip---->i type Hi and select Bold and type Dilip i want output actual HiDilip

I am sorry, but I do not understand your issue.

Hey My Issue is that Suppose is type word MarkMurphy afterfinish Mark i select Bold attribute and type Murphy withought adding space as it is counitinue it not Bold Murphy

You may wish to look into using other libraries, such as Knife or PoorEdit.

eg I type Mark select Bold type Murphy then whole word is MarkMurphy but it disply MarkMurphy

Thanks Mark Murphy.....


Re-add support for Laravel 5.1 LTS

This re-adds support for Laravel 5.1 LTS initially added in #104 and then removed in #122 by replacing the one call to the mapWithKeys collection helper method with simply iterating over the initial array instead of performing a bunch of conversions to collections and back again.

Thank you for this. I've decided to merge this in. But going to repeat the comment I give at #104

Be warned however that I'm likely to drop 5.1 support again if it makes introducing a new feature difficult.

Thanks, sounds good. Just so long as the semver stays up to date that's fine with me :) Personally I only need 5.1 until 5.5 comes out anyways.


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
6980d55 -> f2e1cf2

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
minor
v1.26.3 -> v1.27.2

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.27.2
Compare Source
Bug Fixes

deps: update module github.com/ibm/platform-services-go-sdk to v0.56.3 (#749) (2f431a0)

v1.27.1
Compare Source
Bug Fixes

deps: update module github.com/ibm/platform-services-go-sdk to v0.56.2 (#747) (d6dbbcb)

v1.27.0
Compare Source
Features

delete iam policy (#745) (82603f4)

v1.26.5
Compare Source
Bug Fixes

deps: update gomod (#743) (1ab2e92)

v1.26.4
Compare Source
Bug Fixes

deps: update gomod (#738) (a9275e6)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

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


settings card: Add header - Label and separator.

The settings card by itself looks a little empty. @timabbot
suggested that this was due to a label being absent from the
settings screen.
https://chat.zulip.org/#narrow/stream/243-mobile-team/topic/mascot/near/756544
Also added a separator to make things cleaner. Used a Label
component for including translations.

Thanks!
Let's make this an app bar with the same design as the app bar elsewhere -- see in particular the other settings screens (e.g. "Notifications") for examples.

Fair warning: The second commit (Make titles bold) isn't quite ready for a merge. This is because on certain devices from brands like Oneplus/Oppo which use custom fonts cut off the text at certain places. I'm guessing #2447 is a blocker for this for the moment -- in the case that we do want the titles to be bold.
The first and the third commit is ready though, in case we need to fix this ASAP.

Updated: There were some issues happening where on iOS an 88px gap would appear on top of the settings card. This was because instead of applying the safe area insets as 44px (normal), because in the previous version I simply used the Screen component to SettingsCard which nested two Screen components -- hence adding two ZulipStatusBars resulting in two safe area insets padding.

To resolve this I removed the use of Screen and directly added the ModalNavBar which is the component used in Screen to display the nav bar.

This was because instead of applying the safe area insets as 44px (normal), because in the previous version I simply used the Screen component to SettingsCard which nested two Screen components -- hence adding two ZulipStatusBars resulting in two safe area insets padding.

Hmm interesting. Can we change the existing invocation of Screen on that screen to take care of this, then? Currently we only invoke ModalNavBar from just one place, namely Screen, so it's probably cleanest to maintain that.
(I haven't gone and looked closely at the code in this PR yet.)

Also, one potentially useful bit of context: we used to always keep files/modules split up real small so there was just one React component defined per file.
Pretty recently we reversed that, and have started putting several closely related components in the same file where that makes the code easier to read and understand. But most code you see in the codebase now still reflects the old way.
I think SettingsCard + SettingsScreen are likely an excellent example of two components that would be easier to read and understand if combined in the same file. :slightly_smiling_face: Feel free to go ahead and combine them, as a commit either early or late in the branch, if that seems to help clarify things.

Fair warning: The second commit (Make titles bold) isn't quite ready for a merge. This is because on certain devices from brands like Oneplus/Oppo which use custom fonts cut off the text at certain places. I'm guessing #2447 is a blocker for this for the moment -- in the case that we do want the titles to be bold.

(BTW see the comment I just added on #2447 -- we hadn't known this when we first hit that bug, but it looks like it is in fact specifically related to making things bold. So, yeah.)

Hmm interesting. Can we change the existing invocation of Screen on that screen to take care of this, then? Currently we only invoke ModalNavBar from just one place, namely Screen, so it's probably cleanest to maintain that.

The Screen widget would require another parameter if we were to implement it that way (perhaps isCard?), which is why I was reluctant to change anything. I think in hindsight I agree though, adding that parameter would be better than changing a whole bunch of structure and the cleanness achieved by using ModalNavBar at one place.

I think SettingsCard + SettingsScreen are likely an excellent example of two components that would be easier to read and understand if combined in the same file. 🙂 Feel free to go ahead and combine them, as a commit either early or late in the branch, if that seems to help clarify things.

Actually, I don't think SettingsScreen is actually used anywhere!
zulip-mobile isham$ rg -C4 SettingsScreen
src/nav/AppNavigator.js
12-import UsersScreen from '../users/UsersScreen';
13-import SubscriptionsScreen from '../subscriptions/SubscriptionsScreen';
14-import ChatScreen from '../chat/ChatScreen';
15-import LoadingScreen from '../start/LoadingScreen';
16:import SettingsScreen from '../settings/SettingsScreen';
17-import LanguageScreen from '../settings/LanguageScreen';
18-import PasswordAuthScreen from '../start/PasswordAuthScreen';
19-import DebugScreen from '../settings/DebugScreen';
20-import DiagnosticsScreen from '../diagnostics/DiagnosticsScreen';
--
51-    realm: { screen: RealmScreen },
52-    search: { screen: SearchMessagesScreen },
53-    subscriptions: { screen: SubscriptionsScreen },
54-    users: { screen: UsersScreen },
55:    settings: { screen: SettingsScreen },
56-    language: { screen: LanguageScreen },
57-    lightbox: { screen: LightboxScreen },
58-    group: { screen: CreateGroupScreen },
59-    'invite-users': { screen: InviteUsersScreen },
zulip-mobile isham$

Confirmed this by deleting that file and checking behaviour. Doesn't seem to change!

Actually, I don't think SettingsScreen is actually used anywhere!

Well, your search results there show this line in AppNavigator.js:
55:    settings: { screen: SettingsScreen },

So that's some kind of reference to it! It's not totally obvious what it means until you dig more... but that just means that, until you work out what this is doing, it could be doing anything at all.
In context, this turns out to mean that navigating to the settings route opens up SettingsScreen. Which, it turns out, is what we do on touching the "settings" icon in the bottom nav bar in the main nav of the app. In other words: that's the component that defines our settings screen. :wink:

One tool that can be quite handy for seeing the tree of React components we're using: the React Developer Tools.
If you open that up when on the settings screen, I expect it will show you that there's a SettingsScreen, and a SettingsCard inside it.
I think it'll also show you that with the current version of this PR, there are now two Screen components, one nested inside another. That will be a clue as to why we were double-counting the safe area. :wink: The one in SettingsScreen is the one I had in mind above as the existing invocation of Screen.

In context, this turns out to mean that navigating to the settings route opens up SettingsScreen. Which, it turns out, is what we do on touching the "settings" icon in the bottom nav bar in the main nav of the app. In other words: that's the component that defines our settings screen.

Oh huh! I seem to be mistaken -- and it looks like this is still more confusing than that.
The settings route would be reached by the navigateToSettings action... but we never actually invoke that action:
$ rg -A1 navigateToSettings
src/nav/navActions.js
32:export const navigateToSettings = (): NavigateAction =>
33-  NavigationActions.navigate({ routeName: 'settings' });

And unless this makes a super confusing exception to our normal convention, we never navigate to a route except by invoking one of those action creators in navActions.js.
So maybe this really isn't used after all? If that's true, we should cut it out.

Anyway, backing up from the SettingsScreen issue:

I'd like to reserve the use of Screen for just the one outermost component that draws stuff visibly on the screen -- let's not introduce it inside a more-nested component like SettingsCard.

Especially let's not have one Screen nested inside another. If you look with the React Developer Tools, I think you'll find there's a Screen somewhere above SettingsCard in the tree.
The isCard flag feels like it makes the meaning of Screen pretty complicated; and I think trying to stick it in a more-nested place is the root cause of that complication.

93d2cfe15 screen: Change canGoBack to isRootNav.

Hmm interesting. Can you say more about why this feels like a better name for this value?
I don't totally love the existing name, but it's not clear to me this new one is less confusing.

In AccountPickScreen it looks like it didn't get flipped.

Also in the return value of handleBackButtonPress on BackNavigationHandler.

e42cd9224 navStyles title: Make titles bold.

This is the one that triggers the Oneplus/Oppo bug, right?

6fc21d0a1 nav bar: Increase left margin for text if isRootNav={true}.

Hmm. Honestly this line is already pretty puzzling:

      isRootNav ? { marginLeft: 16 } : { marginRight: NAVBAR_SIZE },

Can you work out why we apparently want a marginRight in one case, but not the other? The difference between the two cases seems like it should only be about the left side, where there might or might not be a back button.
Perhaps we should add a right margin in both cases? Also is NAVBAR_SIZE an appropriate width for it? That's pretty wide.
I guess this change is independent of that, though.

The desired behavior (which this commit accomplishes) is having the label stick further to the right, starting from the vertical line drawn from the middle of the icons under it.

I went and briefly played with the system settings app in Android 9, as a model for this kind of UI.
The titles there seem to be left-aligned with the labels on the settings, so to the right of the icons. That looks pretty good. I think a line through the middle of the icons is less helpful for alignment.

Thanks for the review @gnprice !

If you open that up when on the settings screen, I expect it will show you that there's a SettingsScreen, and a SettingsCard inside it.

Hmm, in MainTabs.js, the component we use to show the main UI, has some code like this.
export default TabNavigator(
  {
    home: {
      screen: HomeTab,
      navigationOptions: {
        tabBarLabel: 'Home',
        tabBarIcon: (props: TabNavigationOptionsPropsType) => (
          <IconInbox size={24} color={props.tintColor} />
        ),
      },
    },
	[...],
    settings: {
      screen: SettingsCard,
      navigationOptions: {
        tabBarLabel: 'Settings',
        tabBarIcon: (props: TabNavigationOptionsPropsType) => (
          <IconSettings size={24} color={props.tintColor} />
        ),
      },
    },
	[...]
  },
  {
    backBehavior: 'none',
    tabBarComponent: TabBarBottom,
    tabBarPosition: 'bottom',
    ...tabsOptions({
      showLabel: !!Platform.isPad,
      showIcon: true,
    }),
  },
);

So, I think in this case we call SettingsCard directly. The two Screens come from one in MainScreenWithTabs.js which renders like this.
render() {
    const { styles: contextStyles } = this.context;

    return (
      <View style={[styles.flexed, contextStyles.backgroundColor]}>
        <ZulipStatusBar /> <--- This has a screen under it
        <OfflineNotice />
        <MainTabs />
      </View>
    );
  }

This is why I introduced the isCard property to Screen.

The isCard flag feels like it makes the meaning of Screen pretty complicated; and I think trying to stick it in a more-nested place is the root cause of that complication.

Do you think we should instead introduce a ModalNavBar directly? Although as you said before (and I agree),

Currently we only invoke ModalNavBar from just one place, namely Screen, so it's probably cleanest to maintain that.

So in this case I don't know what the best thing to do might be.

but it's not clear to me this new one is less confusing

Well, I didn't make it so that this was less confusing per se, just to make it more applicable. For example, all the usages of canGoBack (adding of the back nav button) are actually pointing towards the fact that it is a root navigation. It isn't something related to this PR, and in retrospect I don't even think it's a good name change, since it's current reflection does only reflect only one aspect of Screen. I think I'll drop this commit.

This is the one that triggers the Oneplus/Oppo bug, right?

Yep, this is the one. I haven't found a solution yet (except including a font bundled in the app itself #2447, which I think is a very good thing to do because of other reasons as well -- primarily because every messaging app now-a-days has a unique font, which adds a certain unique flavor to it, although what font to choose might be a huge debate in itself).

Can you work out why we apparently want a marginRight in one case, but not the other?

I'm not exactly sure, but I believe it is because of the hamburger menu on the right for the list of topics in a stream (ExtraButton in ChatNavBar.js)

This becomes apparent when you look at a stream with a really long stream name.

The titles there seem to be left-aligned with the labels on the settings, so to the right of the icons. That looks pretty good. I think a line through the middle of the icons is less helpful for alignment.

True, what I picked was a little out of the random. I'll fix it, thanks for looking into it! :)

Hmm. So if I align the title to the left, the screen for picking accounts and the welcome screen get disturbed.

Made use of the isCard here as well to fix it (the previous usage was also due to this, since if we set the things to dynamically change based on canGoBack, it affects both this one and the welcome screen, because both of them don't need to have a goBack button and yet are different in the way they handle the status bars (because one is a card and another is a screen)).
I don't think we can do without isCard because it is an important distinction between aforementioned screens and this Card usage.

Added more details in the separated commit 🙂 .

A few weeks after the last comments above, I was studying our navigation and I confirmed that SettingsScreen isn't used anywhere, and deleted it: 2f0ac45d4 .
So that removes one source of confusion here.

The two Screens come from one in MainScreenWithTabs.js which renders like this.
render() {
    const { styles: contextStyles } = this.context;

    return (
      <View style={[styles.flexed, contextStyles.backgroundColor]}>
        <ZulipStatusBar /> <--- This has a screen under it

Hmm, does it, though? I don't see one:
        <View style={style}>
          <StatusBar
            animated
            showHideTransition="slide"
            hidden={hidden && Platform.OS !== 'android'}
            backgroundColor={Color(statusBarColor).darken(0.1)}
            barStyle={getStatusBarStyle(statusBarColor)}
          />
        </View>

Here StatusBar is an RN upstream component.
It does try to muck with a safe-area inset, though, so it may explain double-counting there:
    const style = { height: hidden ? 0 : safeAreaInsets.top, backgroundColor };

But it looks like there may indeed be no existing Screen involved on the settings screen! Hmm.

68e42174f screen: Add isCard property.

One essential thing that this commit should do is to actually implement the new prop! I.e. this bit that's currently in the next commit:

-        <ZulipStatusBar />
+        {!isCard && <ZulipStatusBar />}

Without that, this commit just adds some false advertising :slightly_smiling_face: . It also makes it impossible to understand on its own, because as is it doesn't actually work -- the prop doesn't do what it says it does.

1e81a5746 settings card: Add header - Label and separator.
OK, after looking closer at Screen vs. MainScreenWithTabs (as described above), I'm convinced that using Screen is definitely the wrong approach here. Instead we should invoke ModalNavBar directly.
Consider all the stuff that Screen does:
      <View style={[contextStyles.screen, { paddingBottom: safeAreaInsets.bottom }]}>
        {!isCard && <ZulipStatusBar />}
        {search ? (
          <ModalSearchNavBar autoFocus={autoFocus} searchBarOnChange={searchBarOnChange} />
        ) : (
          <ModalNavBar canGoBack={canGoBack} isCard={isCard} title={title} />
        )}
        <OfflineNotice />
        <KeyboardAvoider
          behavior="padding"
          style={[componentStyles.wrapper, padding && styles.padding]}
          contentContainerStyle={[padding && styles.padding]}
        >
          <ScrollView
            contentContainerStyle={[styles.flexed, centerContent && componentStyles.content, style]}
            style={componentStyles.childrenWrapper}
            keyboardShouldPersistTaps={keyboardShouldPersistTaps}
            scrollEnabled={scrollEnabled}
          >
            {children}
          </ScrollView>
        </KeyboardAvoider>
      </View>

(as seen at the tip of your branch; very similar in master.)
That's a lot of things! Do we want all of those things? You've added a workaround (the isCard) to eliminate one of them, but there are a couple more I'm pretty sure we don't want:

the paddingBottom
the OfflineNotice

and then there's all the KeyboardAvoider and ScrollView stuff that we don't know any reason to think we want here.
The original motivation was to add an app bar, which we call ModalNavBar; so let's just add ModalNavBar.
I think that simplifies these changes a fair bit, too.

One essential thing that this commit should do is to actually implement the new prop!

Ah, thanks for noticing that! Fixed.

I think that simplifies these changes a fair bit, too.

I agree. The offline notice bit is why I kept using the Screen, but since you specified we don't actually need it, removing Screen.

I have also dropped the left-margin on the ModelNavBar because I feel like it doesn't suit there + there's no unnecessary flag added just for that.

The titles there seem to be left-aligned with the labels on the settings, so to the right of the icons.

Maybe it is because of Android 10, or because of the OEM of my device, but if I check now the settings label on the Settings app on my device appear to not be aligned (left padding 16px) if they don't have any Back Arrow icon for navigating to the previous screen. That looks good too, IMO.

Thanks @ishammahajan !  I like how very much simpler this gets, too :smile:  Looking now.

Maybe it is because of Android 10, or because of the OEM of my device, but if I check now the settings label on the Settings app on my device appear to not be aligned (left padding 16px) if they don't have any Back Arrow icon for navigating to the previous screen.

Here's the system settings app on stock Android 10, in an emulator:

That's a screen with a Back arrow. But I can't actually find a screen there that has an app bar with a title and doesn't have a Back arrow. The main screen of the settings app looks like this:

And merged -- thanks again @ishammahajan !
Glad this turned out so very simple in the end. :slightly_smiling_face:
One bit of commit-message style (or two, I guess):

Tim's username is @timabbott, with a double "t" at the end

But actually on GitHub @-mentioning someone in a commit message is kind of a misfeature, unfortunately. GitHub turns that into a notification for them every time a version of the commit is rebased and pushed somewhere, which is usually totally irrelevant for them -- e.g. someone took a long stretch of history and rebased it and pushed that to their own clone. That person is doing something perfectly reasonable because it's off in their own sandbox, and the author (you) had a perfectly reasonable intention to mention the person when making a change in the project, but GitHub turns the combination of those into spam.
For that reason I usually avoid GitHub @-mentions in commit messages. Good alternatives include:

just their name -- can be unambiguous with further context (like the chat link in this case), or referring to a maintainer
for complete unambiguity, the Linux-kernel / Git-upstream convention -- see the Reported-By: in this random recent kernel commit or similar in lots of others, or see 338036e0c where I recently wrote Suggested-by: Anders Kaseorg<EMAIL_ADDRESS>to actually send a notification, @-mention the person in the PR thread. This doesn't get duplicated over and over like commits can, and so this problem doesn't arise.

The offline notice bit is why I kept using the Screen, but since you specified we don't actually need it, removing Screen.

Just to clarify this bit: the reason we don't want it from Screen here is that we already have one!
Try it out -- go to the settings screen and enter airplane mode, and you should get the red "No Internet connection" banner at the top.
That's supplied by MainScreenWithTabs, as quoted in your helpful analysis up here.
So I believe invoking Screen in SettingsCard would produce a second one.

But actually on GitHub @-mentioning someone in a commit message is kind of a misfeature, unfortunately.

(I just went and turned this into a bit of docs: https://github.com/zulip/zulip-mobile/blob/master/docs/style.md#mentioning-people as 9164c2b60 .)


Wrong selected style of active nz-tab when embedded in expandable nz-table

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-start-3inmuq
Steps to reproduce

Create <nz-table> with expand-feature using nzExpand
Place a <nz-tabset> with tabs within this expandable area

What is expected?
The active tab of the tabset within the expandable area should have correct style with blue title and blue line under the title.
What is actually happening?
The tab title is just colored blue but has no blue line under the title. Instead the style is applied after active tab change when clicking one of the tabs.

Environment
Info

ng-zorro-antd
8.5.1

Browser
All Browsers

Same if nested tabset, see https://stackblitz.com/edit/angular-426xym-9x8waa - select tab 2. Would it be commonly fixed or another issue needed?

@worki2k1 Programmatically dispatched window resize works as workaround (at least for nested tabs).

@marlag

See

@marlag Seems to work as workaround.


Add action to upload to PyPI on new release

This PR adds a GitHub action to publish new releases to PyPI. This action will be triggered when a new release is made in the careless repo.
For this to work, the version number for careless should also be incremented in setup.py prior to making the release.

Codecov Report

Merging #24 (ddc146d) into main (3989390) will decrease coverage by 0.42%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
- Coverage   69.04%   68.61%   -0.43%
==========================================
  Files          27       27
  Lines        1095     1080      -15
==========================================
- Hits          756      741      -15
  Misses        339      339

Flag
Coverage Δ

unittests
68.61% <ø> (-0.43%)
:arrow_down:

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

Impacted Files
Coverage Δ

careless/utils/distributions.py
51.16% <0.00%> (-2.69%)
:arrow_down:

careless/models/scaling/image.py
52.77% <0.00%> (-2.49%)
:arrow_down:

careless/models/scaling/nn.py
80.00% <0.00%> (-0.96%)
:arrow_down:

careless/models/merging/variational.py
81.25% <0.00%> (-0.17%)
:arrow_down:

Continue to review full report at Codecov.

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


Reduce Padding on Search Results Page to Reflect Design

Fixes #1090
Changed padding from 1.5 rem to .875 rem to make it 14px for the padding.

The figma task requires us to reduce only the padding above the label. In this PR, I think you changed the padding for all four sides, which is not what we want.

@KeithBonesJr I agree with Ashwin. I believe it's "paddingTop: "

Changed the Padding to Padding Top as mentioned in the comments:


Bug fix for hco_config_mod.F90

GCC 9.3.0 found a bug in HEMCO. Namely:
https://github.com/GEOS-ESM/GEOSchem_GridComp/blob/99737f3210e62c8e6a37295e7646283b768efe24/Shared/HEMCO/Core/hco_config_mod.F90#L994-L1005
this code tries to call HCO_CharSplit  outputting N (it's intent(out)). The problem is that N is the loop variable in which this code is executing. gfortran 9.3.0 really doesn't like you over-writing a loop variable!
Unfortunately, I don't know if this is non-zero-diff or not. I'm making a PR, but @christophkeller might have to test it to see what happens.

I double-checked,  verified zero-diff for case of both GOCART and GMI running HEMCO.


Deploying to rpi arch erroring

When pushing to an rpi architecture fleet with the Balena CLI, I'm seeing:
...
[bridge]        Successfully built c62c120b6597
[Info]          Uploading images
[Success]       Successfully uploaded images
[Error]         Some services failed to build:
[Error]           Service: wifi-connect
[Error]             Error: The command '/bin/sh -c export BASE_URL="https://github.com/balena-os/wifi-connect/releases" &&    case $BALENA_ARCH in         "aarch64")             BINARY_ARCH_NAME="aarch64-unknown-linux-gnu" ;;         "amd64")             BINARY_ARCH_NAME="x86_64-unknown-linux-gnu" ;;        "armv7hf")             BINARY_ARCH_NAME="armv7-unknown-linux-gnueabihf" ;;        *)             echo >&2 "error: unsupported architecture ($BALENA_ARCH)"; exit 1 ;;     esac;    if [ ${VERSION} = "latest" ]; then         export URL_PARTIAL="latest/download" ;     else         export URL_PARTIAL="download/${VERSION}" ;     fi;     curl -Ls "$BASE_URL/$URL_PARTIAL/wifi-connect-$BINARY_ARCH_NAME.tar.gz"     | tar -xvz -C  /usr/src/app/' returned a non-zero code: 1
[Info]          Built on c617b98
[Error]         Not deploying release.

It looks like a binary for rpi isn't being built. The documentation says that wifi-connects works with all boards support by Balena, which the RPi W and 3 are support by Balena.

Duplicate of https://github.com/balena-os/wifi-connect/issues/560


Add info message on package index auth error

When an authentication error occurs fetching the package index, it's difficult to tell the cause of the error.  This PR adds a short error message that indicates an authentication error occured.
Before:
Searching for flask
Reading http://localhost:8081/repository/pypi-proxy/simple/flask/
Couldn't find index page for 'flask' (maybe misspelled?)
Scanning index of all packages (this may take a while)
Reading http://localhost:8081/repository/pypi-proxy/simple/
No local packages or working download links found for flask
error: Could not find suitable distribution for Requirement.parse('flask')

After:
Searching for flask
Reading http://localhost:8081/repository/pypi-proxy/simple/flask/
Authentication error: Unauthorized
Couldn't find index page for 'flask' (maybe misspelled?)
Scanning index of all packages (this may take a while)
Reading http://localhost:8081/repository/pypi-proxy/simple/
No local packages or working download links found for flask
error: Could not find suitable distribution for Requirement.parse('flask')

Happy to accept this. Just be aware that this module is deprecated. You should really use pip instead.


UnboundLocalError when setting data to a Panel: "local variable 'mat' referenced before assignment"

Hi,
the following code produces a UnboundLocalError when executed with pandas 0.6.0 / Python 2.7:
import datetime
import pandas
timeidx = pandas.DateRange(start=datetime.datetime(2009,1,1),
                       end=datetime.datetime(2009,12,31),
                       offset=pandas.datetools.MonthEnd())
lons_coarse = np.linspace(-177.5, 177.5, 72)
lats_coarse = np.linspace(-87.5, 87.5, 36)
P = pandas.Panel(items=timeidx, major_axis=lons_coarse, minor_axis=lats_coarse)
P[datetime.datetime(2009,2,28)] = np.random.randn(72*36).reshape((72,36))

Maybe this is to be expected, but it's definitely not intuitive. Since both P[datetime.datetime(2009,2,28)] and np.random.randn(72*36).reshape((72,36)) have shape (72,36), I would expect this to work.
Until it does, what would be the way to do this?

fixed in the above commit


TypeScript support?

I'm working on an overdue migration from TSLint to ESLint due to the former's deprecation in favor of the latter. I had been using a plugin called tslint-plugin-relay, which ported rules from this project over to TypeScript. Is there any interest in supporting TypeScript rules to parallel the Flow rules? If so, I'd be happy to work on upstreaming work from tslint-plugin-relay. If not, I can work on separate plugin instead.

The project at large works with TypeScript, but plugin:relay/recommended enables relay/generated-flow-types, which doesn't work in any non-Flow project. That needs to be explicitly disabled in the ESLint rules configuration. I was inquiring about a parallel rule set to relay/generated-flow-types that would provide a similar type of linting with TypeScript in mind.

I do have an issue where relay/unused-fields does not output information about the line number, making the warning easy to miss when only looking at the IDE

I'll try with PR #115, no promises

I managed to get it working in #128, thanks to @llldar work in #115 :D

What dose generated-flow-types do?

#128 has been merged


Mag boots let you walk in space now.

Description
When you are walking around in space magboots magically walk around like you were on a grid.
Reproduction

mag boots on
walk around in space away from station.

Screenshots
https://github.com/space-wizards/space-station-14/assets/47093363/caff721e-d88d-4590-bccd-44dcb8cfd8fa
Additional context
NA

Probably related to this new PR?
#29009

ABSOLUTELY NO WAY

t3 gravboots that keep this behaviour :trollface:

honestly giving every discipline a type of timbs would be great


[core-client-rest] Add support for allowReserved and explode

Packages impacted by this PR

@azure-rest/core-client

Issues associated with this PR

Fixes #30943
Fixes #30944

Describe the problem that is addressed by this PR
Adds support for passing in an object as a query or path parameter. At present, this object allows for the user to specify a couple of encoding-related options in addition to the value of the parameter:

For path parameters: allowReserved can be set to true to indicate that special characters appearing in the parameter should not be URL-encoded. This allows for path parameters to contain characters like /.
For query parameters: explode can be set to true to change how arrays are encoded. By default, array values are encoded as a single query parameter, with each value separated by a comma (e.g. foo=a,b,c. With explode set to true, each array value will be converted into a single query parameter (e.g. foo=a&foo=b&foo=c).

What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
See design document Loop.
Are there test cases added in this PR? (If not, why?)
Yes

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure-rest/core-client