brew install fails if /tmp is open in Finder

brew doctor output
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed casks are deprecated or disabled.
You should find replacements for the following casks:
  intel-haxm
  wkhtmltopdf

Warning: Some installed formulae are deprecated or disabled.
You should find replacements for the following formulae:
<EMAIL_ADDRESS>  python-lxml
  python-packaging

Warning: You have unlinked kegs in your Cellar.
Leaving kegs unlinked can lead to build-trouble and cause formulae that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  fonttools

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 4.3.8
ORIGIN: https://github.com/Homebrew/brew
HEAD: e5f776b3e23cae8f1b6c3d1194b8329bf701965f
Last commit: 5 days ago
Core tap JSON: 06 Jul 10:12 UTC
Core cask tap JSON: 06 Jul 10:12 UTC
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 16
Homebrew Ruby: 3.3.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/bin/ruby
CPU: 16-core 64-bit kabylake
Clang: 15.0.0 build 1500
Git: 2.45.2 => /usr/local/bin/git
Curl: 8.7.1 => /usr/bin/curl
macOS: 14.6-x86_64
CLT: <IP_ADDRESS>.1.1708646388
Xcode: 15.4

What were you trying to do (and why)?
I'm trying to install a package (to use what it provides).
What happened (include all command output)?

Installation to finish successfully.
Step-by-step reproduction instructions (by running brew commands)

Open /tmp in Finder
Install a big package. I tried go<EMAIL_ADDRESS>and texlive: brew install go
...
No profit

I suspect Finder creates some weird files in directories that brew doesn't expect (maybe .DS_Store or something like that).
I tried some smaller packages and it doesn't happen with all of them. Finder probably needs some time to notice the directory and do its thing. For me installing go fails every time in this scenario.
Packages install with no issues when /tmp window is closed.
PS: I lied when I checked the brew doctor checkbox, obviously. I don't think it's relevant but I couldn't submit the issue without it.

@pointlessone Can you run brew update again and test if the bug is still occurring?
I just went through your test case and was unable to reproduce it using brew 4.3.9
HOMEBREW_VERSION: 4.3.9
ORIGIN: https://github.com/Homebrew/brew
HEAD: 1c98a33a376e5b35ebc801d0f7d6ba7b9eb118ac
Last commit: 8 days ago
Core tap JSON: 15 Jul 18:24 UTC
Core cask tap JSON: 15 Jul 18:24 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 3.3.3 => /opt/homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/bin/ruby
CPU: octa-core 64-bit arm_blizzard_avalanche
Clang: 15.0.0 build 1500
Git: 2.45.1 => /opt/homebrew/bin/git
Curl: 8.6.0 => /usr/bin/curl
macOS: 14.5-arm64
CLT: <IP_ADDRESS>.1.1708646388
Xcode: N/A
Rosetta 2: false

No, still can reproduce.
HOMEBREW_VERSION: 4.3.9
ORIGIN: https://github.com/Homebrew/brew
HEAD: 1c98a33a376e5b35ebc801d0f7d6ba7b9eb118ac
Last commit: 9 days ago
Core tap JSON: 17 Jul 07:26 UTC
Core cask tap JSON: 17 Jul 07:26 UTC
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 16
Homebrew Ruby: 3.3.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/bin/ruby
CPU: 16-core 64-bit kabylake
Clang: 15.0.0 build 1500
Git: 2.45.2 => /usr/local/bin/git
Curl: 8.7.1 => /usr/bin/curl
macOS: 15.0-x86_64
CLT: <IP_ADDRESS>.1.1720323432
Xcode: 15.4

brew install<EMAIL_ADDRESS>-d
/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading<EMAIL_ADDRESS>/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading<EMAIL_ADDRESS>/usr/local/Homebrew/Library/Homebrew/brew.rb (Cask::CaskLoader::NullLoader): loading<EMAIL_ADDRESS>Warning: You are using macOS 15.
We do not provide support for this pre-release version.
It is expected behaviour that some formulae will fail to build in this pre-release version.
It is expected behaviour that Homebrew will be buggy and slow.
Do not create any issues about this on Homebrew's GitHub repositories.
Do not create any issues even if you think this message is unrelated.
Any opened issues will be immediately closed without response.
Do not ask for help from Homebrew or its maintainers on social media.
You may ask for help in Homebrew's discussions but are unlikely to receive a response.
Try to figure out the problem yourself and submit a fix as a pull request.
We will review it but may or may not accept it.

==> Downloading https://ghcr.io/v2/homebrew/core/go/1.21/manifests/1.21.12
Already downloaded<EMAIL_ADDRESS>/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading go
/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading go
/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading go
/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading go
/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading go
==> Fetching<EMAIL_ADDRESS>==> Downloading https://ghcr.io/v2/homebrew/core/go/1.21/blobs/sha256:b4121b05ae63662ebd68a2807b3c7ea3f2363bf57
Already downloaded<EMAIL_ADDRESS>/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading go
/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading<EMAIL_ADDRESS>/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading go
/usr/local/Homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading go
==> Pouring<EMAIL_ADDRESS>/usr/bin/env tar --extract --no-same-owner --file<EMAIL_ADDRESS>--directory /private/tmp/homebrew-unpack20240717-72673-nss6ue
/usr/bin/env cp -pR /private/tmp/homebrew-unpack20240717-72673-nss6ue/.DS_Store /usr/local/Cellar/.DS_Store
/usr/bin/env cp -pR<EMAIL_ADDRESS><EMAIL_ADDRESS>Error: Directory not empty @ dir_s_rmdir -<EMAIL_ADDRESS>/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2324:in `rmdir'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2324:in `block in remove_dir1'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2335:in `platform_support'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2323:in `remove_dir1'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2316:in `remove'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:1452:in `block in remove_entry'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2377:in `block (2 levels) in postorder_traverse'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2377:in `block (2 levels) in postorder_traverse'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2377:in `block (2 levels) in postorder_traverse'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2381:in `postorder_traverse'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2376:in `block in postorder_traverse'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2375:in `each'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2375:in `postorder_traverse'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2376:in `block in postorder_traverse'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2375:in `each'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2375:in `postorder_traverse'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2376:in `block in postorder_traverse'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2375:in `each'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:2375:in `postorder_traverse'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/fileutils.rb:1450:in `remove_entry'
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.3/lib/ruby/3.3.0/tmpdir.rb:107:in `mktmpdir'
/usr/local/Homebrew/Library/Homebrew/unpack_strategy.rb:126:in `extract_nestedly'
/usr/local/Homebrew/Library/Homebrew/download_strategy.rb:103:in `stage'
/usr/local/Homebrew/Library/Homebrew/download_strategy.rb:36:in `stage'
/usr/local/Homebrew/Library/Homebrew/software_spec.rb:389:in `stage'
/usr/local/Homebrew/Library/Homebrew/formula_installer.rb:1291:in `block in pour'
/usr/local/Homebrew/Library/Homebrew/extend/pathname.rb:314:in `block in cd'
/usr/local/Homebrew/Library/Homebrew/extend/pathname.rb:314:in `chdir'
/usr/local/Homebrew/Library/Homebrew/extend/pathname.rb:314:in `cd'
/usr/local/Homebrew/Library/Homebrew/formula_installer.rb:1290:in `pour'
/usr/local/Homebrew/Library/Homebrew/formula_installer.rb:434:in `install'
/usr/local/Homebrew/Library/Homebrew/upgrade.rb:237:in `install_formula'
/usr/local/Homebrew/Library/Homebrew/install.rb:350:in `install_formula'
/usr/local/Homebrew/Library/Homebrew/install.rb:301:in `block in install_formulae'
/usr/local/Homebrew/Library/Homebrew/install.rb:300:in `each'
/usr/local/Homebrew/Library/Homebrew/install.rb:300:in `install_formulae'
/usr/local/Homebrew/Library/Homebrew/cmd/install.rb:297:in `run'
/usr/local/Homebrew/Library/Homebrew/brew.rb:92:in `<main>'

The underlaying issue that unknown files spook brew is still there. Maybe your Finder doesn't create those .DS_Store files? One thing to try is to enable "Calculate all sizes" for the folder (⌘-J or in main menu View → Show View Options). Or create a random file in the unpacked directory.
I narrowed it down to unexpected hidden files.
Here's a oneliner to try:
while true; do find /tmp/ -path '*/homebrew-unpack*' -type d -exec touch "{}/.hidden-file" \; ; done

This will create a .hidden-file in every homebrew-unpack directory. Running this while installing a package makes the installation fail.

Can't reproduce this either. I had /tmp open in Finder as I was installing<EMAIL_ADDRESS>and didn't have any problems.

I narrowed it down to unexpected hidden files.
Here's a oneliner to try:
while true; do find /tmp/ -path '*/homebrew-unpack*' -type d -exec touch "{}/.hidden-file" \; ; done

This will create a .hidden-file in every homebrew-unpack directory. Running this while installing a package makes the installation fail.

I can see why this would cause problems, but I don't see why this would happen if you had only /tmp open in Finder.

I don't see why this would happen if you had only /tmp open in Finder.

I don't have a definite answer but my first guess would be that Finder puts .DS_Store files all over the place when it calculates folder sizes.

I don't have a definite answer but my first guess would be that Finder puts .DS_Store files all over the place when it calculates folder sizes.

Sounds plausible, but I suspect you need to do more than just have /tmp open in Finder for this to happen.

I can make you a screen recording to demonstrate that I don't do anything else.
Just tried it again. Extra piece of information is that /tmp has to be in list view, so that folder size was actually displayed.
The issue is not present when the folder displayed as Icons.

Sure, understandable. I just wanted to document it in case it's a symptom of something bigger.

I can make you a screen recording to demonstrate that I don't do anything else.

That's not necessary; I believe that you don't think you're doing anything else. By "do more" I was also referring to configuration changes, and that won't be visible from a screen recording.

Just tried it again. Extra piece of information is that /tmp has to be in list view, so that folder size was actually displayed.
The issue is not present when the folder displayed as Icons.

That's good to know. Interestingly, folder sizes don't show up for me even in list view. I guess you need to enable "Calculate all sizes" in the view options in Finder in addition to using list view.
This does look like a bug (will need to try reproducing it later), but it seems difficult enough to encounter that it probably doesn't affect many users. We'll review PRs to fix this, but I think Homebrew maintainers are unlikely to pick this up due to the limited impact of the bug.

I tried reproducing again with "Calculate all sizes" checked and list new in the finder and couldn't recreate the issue. I am running an Apple chip Mac (ARM64) Mac and noticed the issue creator was running an intel chip Mac (x86_64). If someone can do a test with an Intel Mac that would be nice.

Multiple people can't reproduce so: closing, sorry.

I'm experiencing this right now trying to upgrade node from source (Monterey so no bottle). I also had /tmp open in Finder, every directory it complained about only had .DS_Store in it.
Perhaps the code involved in doing these installs could do the .DS_Store check like some parts of it already do, or better yet not consider that a problem in the first place unless a developer here can explain why that would be a problem.
Error: An exception occurred within a child process:
  Errno::ENOTEMPTY: Directory not empty @ dir_s_rmdir - /private/tmp/homebrew-unpack20240926-89310-crkwlz/node-v22.9.0/deps/v8

Just quoting what I said above, because it's still true:

This does look like a bug (will need to try reproducing it later), but it seems difficult enough to encounter that it probably doesn't affect many users. We'll review PRs to fix this, but I think Homebrew maintainers are unlikely to pick this up due to the limited impact of the bug.


feedback

@dlebauer  pinging you here instead of the usda repo
the readme has instructions https://github.com/sckott/usdaplantsapi#base-url
let me know what you think
I only have the original data you provided from this comment https://github.com/USDA/USDA-APIs/issues/7#issuecomment-187931791

@sckott this looks good, but I'd suggest using the file usda_plants_20160223.csv from a subsequent comment (https://github.com/USDA/USDA-APIs/issues/7#issuecomment-188002148). This is a copy of the database from Feb 23 2016; the other ones are five or more years old and the original content has not been preserved.

okay, will do

One other thought: the current documentation suggests that the search endpoint is currently used to define which fields are returned (as in select genus, species, accepted symbol from plants).
Another very useful feature would be to be able to subset analogous to a where clause like ... where genus = 'Acer'; etc.
In the development version of the BETYdb API this would be something like species?genus=Acer

Right, the last bullet point in the params header describes that https://github.com/sckott/usdaplantsapi#params - ability to search, that is, where statements

Got it. Thanks!
On Thu, Mar 17, 2016 at 12:52 PM Scott Chamberlain<EMAIL_ADDRESS>wrote:

Right, the last bullet point in the params header describes that
https://github.com/sckott/usdaplantsapi#params - ability to search, that
is, where statements
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/sckott/usdaplantsapi/issues/3#issuecomment-197999049

API now using the newer dataset


Change repoMasterBranch to repoDefaultBranch

Update Repo data type, based on https://developer.github.com/v3/repos/#get
There might be other fields that have changed, I haven't verified every field.

Merged in https://github.com/phadej/github/pull/222

Cool, thanks!


Updating old version of golang.org/x/sys to work with go 1.18

Using latest go (1.18) requires an update to golang.org/x/sys
https://github.com/golang/go/issues/49219
Changes:
PR updates this module

Closing as CLI uses go1.21 now and the package is updated


Follow code conventions and make code more easy to read.

Also see, https://github.com/SpigotMC/Spigot-API/blob/master/src/main/java/org/bukkit/plugin/java/JavaPlugin.java#L232

Thanks for the contribution, @hamzaxx!


Unsuccessful attempt at EventQueueItems

Just like we talked, I am pushing the EventQueueItem trial. I'm pretty sure that it is not needed, since like you mentioned, we can just push a message as a prop, but I didn't want to go ahead and put more time in before you reviewed it and gave me your opinion.
On a brighter note, the play/stop button is now a toggle, which can easily be implemented with backend, just by changing the function, crosses on queue items are dark red and we have a shuffle and repeat buttons, which are a WIP.
Give me a heads up when you go through the changes.

Frankly thinking that it may not even be necessary to have the events view visible at all time. Maybe use a tabbed view, or remove it entirely. It probably doesn't need to be in the first release


Repeatedly open new tabs

When I click on link from editor, the link opens in a new tab, this is great. But if I click on the link again, it opens again in a new tab. This problem does not happen in File explorer. The plugin "obsidian-no-dupe-leaves" has a bug, when I install this plugin, I can't open links like [[LinkDemo > Header1]], so I'm wondering if this plugin can solve the above problem at the same time.

The issue has been fixed in the new version.

The issue has been fixed in the new version.


Implementing darker as pre-commit

Implementing pre-commit rules to apply linting only in codes changed.
It was implemented a set of rules, using pre-commit, to run linting before commit is made.
For linting tool, kept using flake8, but added black.
On top of these two tools, Darker was used to apply the linting rules only to changed lines.

In flynt 0.68, some breaking changes were introduced as states in darker release notes 1.7.1

Hey @willianrocha!

There are some merge conflicts, so I guess this branch needs rebasing.

It seems to me that most of the changes were related to formatting (correct me if I'm wrong). Since the important changes don't include many code/configuration changes and not all commits have a working version of the feature, maybe you could reduce the number of commits to one or two(which would be working versions), and describe any important information(like the breaking changes from flynt) in the commits' description.

When I run git commit -m 'test', I receive the following error: FileNotFoundError: [Errno 2] No such file or directory: 'flake8'. Then I ran the same command with poetry: poetry run git commit -m 'test' and it worked. Is this the expected behavior? It does not seem so intuitive. But if it stays like this, I guess it should be documented in the CONTRIBUTING.rst file.

When I run git commit -m 'test', I receive the following error: FileNotFoundError: [Errno 2] No such file or directory: 'flake8'. Then I ran the same command with poetry: poetry run git commit -m 'test' and it worked. Is this the expected behavior? It does not seem so intuitive. But if it stays like this, I guess it should be documented in the CONTRIBUTING.rst file.

forgot to emphasize this one, git commit failing is not acceptable behaviour haha


Regarding new xsl files

Hi,
When I see your code you are referring to files in folder "MathServer/xslt" but, when i download your code  I don't see these files at  /srv/http/fastmath/MathServer/ folder. So, can you put the new modified files.
Thanks,

The location of the xsl files was hard-coded to an incorrect directory. I will remove the hard-coding. In the meantime, the xsl directory in your copy of the source can simply be copied to the appropriate location.

It looks like the entire MathServer directory was from an old version of the code. It has been completely removed. Its replacement (fm.mathengine) already properly located the xsl files.

But the existing xls files are not converting MathML to SymPy
understandably format right. They are converting to LaTex.
On Wed, Mar 25, 2015 at 8:42 PM, Stephen Just<EMAIL_ADDRESS>wrote:

Closed #1 https://github.com/stephenjust/fastmath/issues/1.
—
Reply to this email directly or view it on GitHub
https://github.com/stephenjust/fastmath/issues/1#event-264825292.

--
Vinay Mandava
91-9885127836

Ah, the XSLT files were originally for a MathML to LaTeX  conversion, but they are partially modified to handle the conversion to something understandable by SymPy.
The transform to SymPy only works for a small subset of MathML - only the parts needed to actually submit answers to the questions generated by the fastmath application.

Thanks alot for the information.
On Wed, Mar 25, 2015 at 8:57 PM, Stephen Just<EMAIL_ADDRESS>wrote:

Ah, the XSLT files were originally for a MathML to LaTeX conversion, but
they are partially modified to handle the conversion to something
understandable by SymPy.
The transform to SymPy only works for a small subset of MathML - only the
parts needed to actually submit answers to the questions generated by the
fastmath application.
—
Reply to this email directly or view it on GitHub
https://github.com/stephenjust/fastmath/issues/1#issuecomment-86081793.

--
Vinay Mandava
91-9885127836


lineage

Please come with a real account

Whats mean ? Real account i am not bot
On Thu, Jan 20, 2022, 11:24 PM Khalakuzzaman Apon @.***>
wrote:

Please come with a real account
—
Reply to this email directly, view it on GitHub
https://github.com/ROM-builders/temporary/pull/5391#issuecomment-1017767540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXMBNM3RM5NEC36XHGKPHADUXBD43ANCNFSM5MNNENYQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Please 🙏
Requesting its my real account i am still learning and new in development
so please give permission to use your server
Thank-you
Sir
On Thu, Jan 20, 2022, 11:27 PM [ Pitroxi ] @.***> wrote:

Whats mean ? Real account i am not bot
On Thu, Jan 20, 2022, 11:24 PM Khalakuzzaman Apon <
@.***> wrote:

Please come with a real account
—
Reply to this email directly, view it on GitHub
https://github.com/ROM-builders/temporary/pull/5391#issuecomment-1017767540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXMBNM3RM5NEC36XHGKPHADUXBD43ANCNFSM5MNNENYQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


This is dangerously incorrect

The interface is named subtle because it warns developers that many of the crypto algorithms have sophisticated usage requirements that must be strictly followed to get the expected algorithmic security guarantees. The subtle object is the main entry point for interacting with underlying crypto primitives.

You are not following these usage requirements, and the security guarantees are lost.
You are burying, apparently intentionally, all warnings to developers that their use of your library is deeply insecure.
Please reconsider this library.  Its usage structure cannot be repaired; you are passing too much of the buck to the downstream user, and hiding from them the information they need to do something correct.
The first law of the internet: "if you are not a cryptography expert, do not attempt to get involved."

This is particularly concerning coming from a cryptocurrency person.
Anyone with even a two week red team training program can walk right through this like it isn't even there.
This system is how bridges get owned, dude, is people like you saying "I'm going to make the hard parts easier" without recognizing the wild, wild damage they're doing

Note that this library is not intended for cryptographic purposes. If you need cryptographically secure hashing, there are extra steps you may need to take which this library will not be suited for.

This is the second sentence from my README. The first sentence links to the MDN documentation for Web Crypto, where you will find a similar warning to the one you quoted.
I very explicitly did not build this library as a security tool.  If you have suggestions for how I can more clearly warn developers about the limitations of this library, I am all ears and would be very likely to implement them.
I appreciate the feedback, and would love to continue the discussion if you are interested, but I reject the implication that there is no non-security use case for fast, easy, browser-side hashing. This library does not wrap Web Crypto's encryption or signing functionality. It is only for hashing and makes no security guarantees about those hashes.

In the interest of being as clear as possible about the use case for this library, I have reworded the warning at the beginning of the README to be more dire and formatted it as a callback.
https://github.com/delventhalz/browser-hash/commit/44c33a41d750c320c59583afa4d04c63fae215ab
If you have any further suggestions other than deleting the library entirely, I am very interested to hear them.

It's bad for even casual use.  This is literally the exact mistake that the
Chrome team made.

If you have any further suggestions other than deleting the library
entirely

Well, you could take the time to learn the difficult things that this name
was chosen to warn people about, and do them correctly, but between you and
me, I think that's even less likely than causing indirect harm because you
want to be a library author and don't grok the downside
There aren't even any t tests here, buddy
This is one of the six warning topics for a reason :(
This took more than two decades to get into the browser for a reason :(
But I guess I feel like we're already pretty clear into "I'm gonna do what
I want" territory.  You have fun
On Tue, Apr 5, 2022 at 11:59 AM Zac Delventhal @.***>
wrote:

In the interest of being as clear as possible about the use case for this
library, I have reworded the warning at the beginning of the README to be
more dire and formatted it as a callback.
44c33a4
https://github.com/delventhalz/browser-hash/commit/44c33a41d750c320c59583afa4d04c63fae215ab
If you have any further suggestions other than deleting the library
entirely, I am very interested to hear them.
—
Reply to this email directly, view it on GitHub
https://github.com/delventhalz/browser-hash/issues/1#issuecomment-1089194532,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAS5KQSODWFACXO33IP3LDVDSEPRANCNFSM5STFQC6A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks for your thoughts. If you have anything more specific to add later, I'll be here. Closing this issue for now.


Multiple conflicting errors in relation with using brew cask ...

Through normal operations with homebrew, I ended up learning that casks supposedly have moved (https://github.com/caskroom/homebrew-cask/issues/21913); fine. I tried to understand and follow all the differing pieces of information that people gave in that thread and also read reports in this project's issue threads. To no avail!
My goal is to get a back a working brew/cask environment on my Mac, so I can continue using it the way it was designed. Currently, I get the following errors:
$ brew cask list
Error: No available formula with the name "brew-cask"
Warning: The default Caskroom location has moved to /usr/local/Caskroom.

Please migrate your Casks to the new location and delete /opt/homebrew-cask/Caskroom,
or if you would like to keep your Caskroom at /opt/homebrew-cask/Caskroom, add the
following to your HOMEBREW_CASK_OPTS:

  --caskroom=/opt/homebrew-cask/Caskroom

For more details on each of those options, see https://github.com/caskroom/homebrew-cask/issues/21913.
android-file-transfer                              rowanj-gitx
javafx-scene-builder (!)                           xquartz

$ brew cask uninstall rowanj-gitx xquartz javafx-scene-builder android-file-transfer
Error: No available formula with the name "brew-cask"
Warning: The default Caskroom location has moved to /usr/local/Caskroom.

Please migrate your Casks to the new location and delete /opt/homebrew-cask/Caskroom,
or if you would like to keep your Caskroom at /opt/homebrew-cask/Caskroom, add the
following to your HOMEBREW_CASK_OPTS:

  --caskroom=/opt/homebrew-cask/Caskroom

For more details on each of those options, see https://github.com/caskroom/homebrew-cask/issues/21913.
Warning: Calling "cask :v1 => 'token'" is deprecated!
Use "cask 'token'" instead.
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:36:in `cask'

Warning: Calling MacOS.release is deprecated!
Use MacOS.version instead.
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:20:in `block in load'

Warning: Calling MacOS.release is deprecated!
Use MacOS.version instead.
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:23:in `block in load'

==> Removing Binary symlink: '/usr/local/bin/gitx'
rowanj-gitx latest is still installed.
Remove it with `brew cask uninstall --force rowanj-gitx`.
Error: xquartz is not installed

$ brew cask uninstall --force rowanj-gitx
Error: No available formula with the name "brew-cask"
Warning: The default Caskroom location has moved to /usr/local/Caskroom.

Please migrate your Casks to the new location and delete /opt/homebrew-cask/Caskroom,
or if you would like to keep your Caskroom at /opt/homebrew-cask/Caskroom, add the
following to your HOMEBREW_CASK_OPTS:

  --caskroom=/opt/homebrew-cask/Caskroom

For more details on each of those options, see https://github.com/caskroom/homebrew-cask/issues/21913.
Warning: Calling "cask :v1 => 'token'" is deprecated!
Use "cask 'token'" instead.
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:36:in `cask'

$ find /usr/local/Caskroom -name "*.app" -maxdepth 3
find: /usr/local/Caskroom: No such file or directory

$ brew cask install --force $(brew cask list)

Error: No available formula with the name "brew-cask"
Warning: The default Caskroom location has moved to /usr/local/Caskroom.

Please migrate your Casks to the new location and delete /opt/homebrew-cask/Caskroom,
or if you would like to keep your Caskroom at /opt/homebrew-cask/Caskroom, add the
following to your HOMEBREW_CASK_OPTS:

  --caskroom=/opt/homebrew-cask/Caskroom

For more details on each of those options, see https://github.com/caskroom/homebrew-cask/issues/21913.
Error: No available formula with the name "brew-cask"
Warning: The default Caskroom location has moved to /usr/local/Caskroom.

Please migrate your Casks to the new location and delete /opt/homebrew-cask/Caskroom,
or if you would like to keep your Caskroom at /opt/homebrew-cask/Caskroom, add the
following to your HOMEBREW_CASK_OPTS:

  --caskroom=/opt/homebrew-cask/Caskroom

For more details on each of those options, see https://github.com/caskroom/homebrew-cask/issues/21913.
==> Downloading https://dl.google.com/dl/androidjumper/mtp/current/androidfiletransfer.dmg
######################################################################## 100.0%
==> No checksum defined for Cask android-file-transfer, skipping verification
==> Moving App 'Android File Transfer.app' to '/Applications/Android File Transfer.app'
🍺  android-file-transfer was successfully installed!
Error: undefined method `latest?' for nil:NilClass
Follow the instructions here:
  https://github.com/caskroom/homebrew-cask/blob/master/doc/reporting_bugs/pre_bug_report.md

If this doesn’t fix the problem, please report this bug:
  https://github.com/caskroom/homebrew-cask#reporting-bugs

/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/download.rb:36:in `clear_cache'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/download.rb:14:in `perform'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:96:in `download'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:70:in `install'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:25:in `block in install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:10:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:110:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:149:in `process'
/usr/local/Homebrew/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Homebrew/Library/Homebrew/brew.rb:24:in `require?'
/usr/local/Homebrew/Library/Homebrew/brew.rb:100:in `<main>'

The only reason I even got into this mess was that roughly every month, I execute brew doctor to see if I have to clean up yet another mess due to someone deciding that breaking a perfectly working and stable API seems like a good idea (happened the last time when 'brew upgrade --all' was introduced for no useful reason to the end-user from a stability and workflow reproducibility point of view, only to be removed again a couple of months down the road, and many others in the years I have been using this wonderful system).
Basically, it all started with this harmless:
$ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: Your XQuartz (2.7.7) is outdated.
Please install XQuartz 2.7.9 (or delete the current version).
XQuartz can be updated using Homebrew-Cask by running
  brew cask reinstall xquartz

This time, it seems to have hit my system harder than other times ;). I am fine to spend some hours trying to figure this mess out myself, if someone would be so kind as to point me into the right direction.

Since my list of casks is very small, I have tried to manually remove them through the brew API. It sort of works:
$ brew cask uninstall xquartz   --caskroom=/opt/homebrew-cask/Caskroom
Error: No available formula with the name "brew-cask"
==> Running uninstall process for xquartz; your password may be necessary
==> Removing launchctl service org.macosforge.xquartz.startx
Password:
==> Quitting application ID org.macosforge.xquartz.X11
==> Removing files from pkgutil Bill-of-Materials
==> Removing files: ["/opt/X11/"]

Not sure what the weird error message (No available formula with the name "brew-cask") is supposed to tell me; however the directory xquartz is still there, as well as the android-file-transfer one, even after --force uninstall those packages for half a dozen times. Other packages are luckily gone:
$ find /opt/homebrew-cask/Caskroom -type d -maxdepth 2
/opt/homebrew-cask/Caskroom
/opt/homebrew-cask/Caskroom/android-file-transfer
/opt/homebrew-cask/Caskroom/android-file-transfer/.metadata
/opt/homebrew-cask/Caskroom/android-file-transfer/latest
/opt/homebrew-cask/Caskroom/xquartz
/opt/homebrew-cask/Caskroom/xquartz/2.7.7

My question: Can I just remove /opt/homebrew-cask/Caskroom and be done with it? I can then cleanly install those packages again afterwards in hope that brew finally decides to work with the new cask directory.

Can I just remove /opt/homebrew-cask/Caskroom and be done with it?

Yes.
Your installation of HBC is broken/outdated. Many clues to that:

Error: No available formula with the name "brew-cask"
Warning: Calling "cask :v1 => 'token'" is deprecated!
`Warning: Calling MacOS.release is deprecated!``

From the issue template that shows when opening a new issue:

Before opening your issue you must check our instructions for reporting bugs (…) [it] has several links to help diagnose and fix common issues.
(…)
if it is apparent you ignored the instructions for reporting issues, we may close the issue without review.

That seems very much to be the case. Not only will the problem probably be fixed by those instructions, you haven’t given the requested information. We worked hard  on those guides to keep things running smoothly, so I ask that you go back and follow them. It is OK to open a new issue for that.
I understand sometimes we’re tired and don’t want to sift through a guide to make what we feel is a simple bug report. But when each user ignores the guides to save a few minutes, maintainers lose hours and get fatigued. I ask you take that into consideration.
Closing for now, since you mentioned you might just nuke it and start over.

Thank you very much for your time. I have nuked that directory and proceeded to reinstall at least one cask (android-file-transfer), which worked correctly. For the remaining issues that still pertain now, I will read again the "reporting bugs" page and open a new issue if warranted.
Last note: The only thing I have done in the past months is "brew update; brew upgrade". How my system can get into this state of what you call broken/outdated is beyond me.

How my system can get into this state of what you call broken/outdated is beyond me.

May very well be our fault. Some breaking changes coupled with bad timing in updating on your side that screwed something up and you never got the fix.


update template project

❔ Question
https://github.com/DefinitelyTyped/DefinitelyTyped/pull/30364

Let me have a try.

腻害了~

fix by #66


Update to logos for Proton services

Pre-flight Checklist
Please ensure you've completed all of the following.

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.

Description of Change

Update to the outdated Proton Calendar logo. Protonmail and Proton Drive logos were mobile ones, so changed to website and desktop logos for better uniformity within Ferdium.

@all-contributors please add @Arthur-Huan for code


Add config option specify Gitlab groupname sanitizer separator

Add option to specify Gitlab groupname sanitizer separator.
The default option would remain the same as it currently is - to use a space.
In my use case I use "rough" groupnames as they come from my LDAP. E.g. I have example-group and that would remain to be example-group. Before this PR, it used to remove the dash.

This is not relevant anymore - https://github.com/Adambean/gitlab-ce-ldap-sync/tree/issue/43-dashes-removed-from-group-names - solves this problem for me.


About sha256WithRSAEncryption signature analysis using old version of bc

When using BC v1.71 to send a signed message using sha256WithRSAEncryption, if BC v1.37 is used for receiving, an error occurs in signature parsing.
Error Description
no such algorithm: SHA256with1.2.840.1135<IP_ADDRESS> for provider BC
The same problem does not occur with version 1.54 of the library used in sending signature messages.
It seems to occur with library version 1.55.
Please let me know which fix in the 1.55 release notes is causing this event.
I would also be glad to know the purpose of the modification.

It'll be because 1.37 has errors in it.


Fix behavior when userId is null or undefined

Hopefully the code speaks for itself.
This issue was discovered while debugging with IBM.

👍


Add scala unit tests

Add support for unit tests
Refactor to support better testing

Codecov Report

Merging #3 into master will decrease coverage by 22.97%.
The diff coverage is 85.00%.

@@             Coverage Diff              @@
##            master       #3       +/-   ##
============================================
- Coverage   100.00%   77.02%   -22.98%
============================================
  Files           13       29       +16
  Lines          129      235      +106
  Branches         9        9
============================================
+ Hits           129      181       +52
- Misses           0       54       +54

Flag
Coverage Δ

#react
100.00% <ø> (ø)

Impacted Files
Coverage Δ

...rc/main/scala/jscbe/modules/RepositoryModule.scala
100.00% <ø> (ø)

be/src/main/scala/jscbe/routes/SampleRoute.scala
3.33% <25.00%> (ø)

be/src/main/scala/jscbe/Starter.scala
100.00% <100.00%> (ø)

...main/scala/jscbe/configuration/Configuration.scala
100.00% <100.00%> (ø)

...in/scala/jscbe/configuration/DbConfiguration.scala
100.00% <100.00%> (ø)

.../scala/jscbe/configuration/HttpConfiguration.scala
100.00% <100.00%> (ø)

...main/scala/jscbe/modules/ConfigurationModule.scala
100.00% <100.00%> (ø)

.../src/main/scala/jscbe/modules/WebRouteModule.scala
100.00% <0.00%> (ø)

... and 13 more

Continue to review full report at Codecov.

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


During public env creation instruction shows two export cmds with override each other

Second export will override the first one

related to https://github.com/digma-ai/digma-ui/issues/625


Crash in C++ app when trying to instantiate type from C# library

Related to #693, #697, #698, #699

Created a C# NET 5 class library. Reference the WinUI 3 Preview 4 CI 1/26 nuget package.
Follow the authoring instructions to produce a nuget pkg for the class library
Created a cppwinrt WinUI 3 app from WinUI 3 Preview 3 VSIX
update cppwinrt app's WinUI reference to latest internal WinUI nuget preview 4 from 01/26
Referenced the NET 5 class library nupkg file from the cppwinrt app via a local nuget source
Build solution

After adding the winrt.host.runtimeconfig.json and the app1.exe.manifest to the WAP, the WAP project can start, however it crashes when instantiating a type from the library with "Error in DLL"

Potential dupe of #696

Potential dupe of #696


feat(module:icon): support add icon in feat modules & refactor

BREAKING CHANGE:

NZ_ICON_DEFAULT_TWOTONE_COLOR is removed. Use NzGlobalConfigService instead.

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information

@wendellhu95 测试挂了

GitHub 这是怎么了，我已经 rebase 到了 master


Clipped Camera not working in Godot physics

Godot version:
master: 94875f5
3.2.4.beta1
OS/device including version:
Ubuntu 20.10
Issue description:
ClippedCamera doesn't clip in Godot physics with v3.2 (crashes with master). Works in Bullet physics.

Master stack trace:
handle_crash: Program crashed with signal 11
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x41950) [0x7ff08dd3f950] (??:0)
[2] Vector3::is_equal_approx(Vector3 const&) const (/home/marcel/Git/godot/core/math/vector3.cpp:148)
[3] Plane::is_equal_approx(Plane const&) const (/home/marcel/Git/godot/core/math/plane.cpp:174)
[4] QuickHull::build(Vector<Vector3> const&, Geometry3D::MeshData&) (/home/marcel/Git/godot/core/math/quick_hull.cpp:384)
[5] ConvexPolygonShape3DSW::_setup(Vector<Vector3> const&) (/home/marcel/Git/godot/servers/physics_3d/shape_3d_sw.cpp:877)
[6] ConvexPolygonShape3DSW::set_data(Variant const&) (/home/marcel/Git/godot/servers/physics_3d/shape_3d_sw.cpp:896)
[7] PhysicsServer3DSW::shape_set_data(RID, Variant const&) (/home/marcel/Git/godot/servers/physics_3d/physics_server_3d_sw.cpp:92)
[8] ClippedCamera3D::_notification(int) (/home/marcel/Git/godot/scene/3d/camera_3d.cpp:749)
[9] ClippedCamera3D::_notificationv(int, bool) (/home/marcel/Git/godot/scene/3d/camera_3d.h:187)
[10] Object::notification(int, bool) (/home/marcel/Git/godot/core/object/object.cpp:796)
[11] SceneTree::_notify_group_pause(StringName const&, int) (/home/marcel/Git/godot/scene/main/scene_tree.cpp:818)
[12] SceneTree::iteration(float) (/home/marcel/Git/godot/scene/main/scene_tree.cpp:411)
[13] Main::iteration() (/home/marcel/Git/godot/main/main.cpp:2404)
[14] OS_LinuxBSD::run() (/home/marcel/Git/godot/platform/linuxbsd/os_linuxbsd.cpp:261)
[15] /home/marcel/Git/godot/bin/godot.linuxbsd.tools.64.llvm(main+0x1dc) [0x44b1e0c] (/home/marcel/Git/godot/platform/linuxbsd/godot_linuxbsd.cpp:58)
[16] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf2) [0x7ff08dd26cb2] (??:0)
[17] /home/marcel/Git/godot/bin/godot.linuxbsd.tools.64.llvm(_start+0x2e) [0x44b1b6e] (??:?)
-- END OF BACKTRACE --
Aborted (core dumped)

Steps to reproduce:
Create a ClippedCamera, and put a CollisionObject in the way. Notice that the camera is not clipped when using Godot physics (and crashes when using master).
Minimal reproduction project:
4.0: ClippedCameraFailure-4.0.zip
3.2: ClippedCameraFailure-3.2.zip

Removed crash label, because that was a separate issue fixed by #43775.


Links field missing from Resource.swift

In addition, a resource object MAY contain any of these top-level members:
links: a links object containing links related to the resource.

http://jsonapi.org/format/#document-resource-objects
This links object is currently not on Resource.swift

@aronbalog @vinceis1337 If you would like to check on this, I have made a PR to support links. Feel free to let me know if it needs changes.
https://github.com/aronbalog/Vox/pull/21


Suppress duplicate images for AIDS posters

All of the AIDS posters are duplicated in image search. This is because the same images were ingested into both Goobi and Miro. We can't just drop Miro images when there is a METS file, as they are not always in the same image. However, because of the number of AIDS posters and their popularity we need to come up with a way to fix this instance of the problem.
Fortunately, it looks like there is one. All AIDS posters that have been through the digitisation workflow have a digcode of digaids. Now that we have this as an identifier, we can use it to suppress the creation of images from Miro, where the Miro work is linked to a work with a wellcome-digcode of digaids.
Thinking forward, it is likely that I will suggest to the production team that any Miro images that are ingested through should be given a digcode of digmiro. If they agree to that, then we'll have a way to fix this across the board.

Dupe of #4875


Add graphical configuration

Currently, initial configuration in via command line; make it graphical too, using file chooser dialogue etc.

In fact, once we're at it, extend it so it can handle multiple configurations (i.e. email/filepath pairs) and manage those is a first screen, similar to how gitg does it.


EOF flag not set on exhausted input streams.

When reaching the end of a stream while parsing JSON, the eofbit is not set.
std::stringstream s;
try { json j ; j >> s; } catch (...) {}
s.eof(); // false

This seems like a reasonable expected behaviour, as our existing code relies of eofbit to determine when the end of a continuous stream of json commands is reached.

Addressed in https://github.com/nlohmann/json/pull/1343.


Design: Create design for Promotion components

Issue summary
There is currently no design for the promotion component and in order for the project to have a uniform styleguide, a design has to be created for each of the components.
Proposed fix
Create a design for the promotion component using the colors of the project which is already given in the README.
Tasks

[ ] Banner
[ ] Sale/Discount badges
[ ] Featured products

Before working on the issue, dm me or @AdityaSingh-02 and @purnima143 to be added to the Figma file and please, strictly follow the styleguide.
PR Rules

Create the PR against the dev branch not the master branch
Link the issue your PR is for by saying Fixes #PRNumber
Add a description of what your PR is about
Add screenshots or video of the working implementation of your PR

Can I work on this?


[Hotfix] when MLIR is not used in MIOpen

ref: https://github.com/ROCmSoftwarePlatform/MIOpen/pull/2353#discussion_r1405339967

See #2552


mux function could support a way to output its progress

Is your enhancement request related to a problem? Please describe.
When muxing something with mkvmerge it outputs the current progress in percent.
The current implementation of mux does not seem to support that information, which would be great to have, especially when running async or when muxing large files.
Describe the solution you'd like
Having an event function in the main object, which returns the progress, which is called when new progress information arrives.
Example:
mkv = MKVFile('/path/to/file.mkv')
mkv.remove_track(0)
mkv.mux('/path/to/output.mkv', gui.set_current_progress)

or
mkv = MKVFile('/path/to/file.mkv')
mkv.remove_track(0)
mkv.connect_event_new_progress(gui.set_current_progress)
mkv.mux('/path/to/output.mkv')

Implementation steps

[ ] Look at the output of mkvmerge and find a way to grab that information
[ ] Interpret that data
[ ] Add an event function which is called on new progress and returns that data
[ ] Add a variable to hold the connected function and a setter function to connect an external function to that event function

Additional context
I implemented something similar a few years ago, which still works, but the code is more a proof of concept, than a good implementation. It can be found here.
That function is used in combination with a gui progress bar.
Let me know, if you want help implementing this and the enhancement request from yesterday.

This is going to take more time. We need to figure out the best way to do it.

Hi, I'm back to working on my project and will try to implement this. If I'm successful, I'll try to open a pull request.
But I've now come across another serious problem and will open a new issue for it.


Fix bug in Webhooks metrics

Proposed Changes
Fix bug where the metrics name and description values were in the wrong slots, causing a type error where metrics conversion to text in the Prometheus and CLI output would crash.
Adds a call to the CLI metrics command to test that WebHooks metrics to text works.
Types of Changes
What types of changes does your code introduce to this project?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes issue #XXXX)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (correction or otherwise)
[ ] Cosmetics (whitespace, styles...)
[ ] DevOps (Build scripts, pipelines...)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating
the PR. If you're unsure about any of them, don't hesitate to ask on the
mailing list. We're here to help! This is simply a reminder of what we are
going to look for before merging your code.

[x] I have read the CODE_OF_CONDUCT.md document
[x] All tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if needed)
[x] Any dependent changes have been merged and published in related repositories
[x] I have updated changelog (At the bottom of the release version)
[x] I have squashed all my commits into one before merging

Further Comments
None

@varnerac thanks!


Merging multiple .nyc_output from different locations

Hi, just a small question.
I use this tool to do coverage on ejs templates. Due to specification of the project that i am working, I have multiple .nyc_output directories that shows coverage on the same ejs templates.
Is there any way to merge them and probably generate html report?
Thanks for this amazing tool.

@danziamo you should be able to move all the raw JSON files into a single folder, than run nyc report (rather than running the nyc foo.js which performs instrumentation).
This might be worth adding documentation for in the README, if you find this approach works for you perhaps you'd be willing to pitch in?

You can also run nyc merge source-dir dest-file.json on each directory.  We do this with the istanbuljs monorepo, see https://github.com/istanbuljs/istanbuljs/blob/master/monorepo-merge-reports.js.  In that monorepo each package writes coverage to packages/${packageName}/.nyc_output.  Then we run monorepo-merge-reports.js to merge each of those directories to .nyc_output/${packageName}.json.  From there we run nyc report which runs reports against the root .nyc_output (see the posttest script in https://github.com/istanbuljs/istanbuljs/blob/master/package.json).
An example command run by monorepo-merge-reports.js is nyc merge packages/test-exclude/.nyc_output .nyc_output/test-exclude.json.

Thanks, that worked for me.

@coreyfarrell I do as you said, but when I run nyc report finally, it reports nothing.

@yjjnls this issue is closed.  Please open a new issue and include all requested information.


Add Captcha service support

Court sites often use Captchas. We'll need to build in support for Captcha services that can be dropped in where needed.

Support for captchas introduced in OK dev. This version supports recaptcha V2 with a base library to support other types.

I have refactored the captcha solving classes to handle multiple types of captchas. This design required its own folder. The library now supports the two most common types of captchas we have encountered: Recaptcha Version 2 and its invisible counterpart.
Each require different strategies to inject the solution into the page and successfully bypass the captcha. The non-invisible kind has more variables in the way you move past the captcha and provides the opportunity for an xpath or javascript to be supplied by the developer or the option to inherit the class into a region or platform-specific class and overwrite the submit function entirely with a function that is a multi-line, tailored script.


Add warning when set "alias" in low level annontated field

Change Summary
Add warning message when user setup annotated field with alias kwarg at low level. #7828.
Related issue number

Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[x] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Fix #7828
Selected Reviewer: @adriangb

please review.

@nix010,
Great work here, thanks! Will merge once tests pass - I made a few minor modifications to function names + warning wording.

@sydney-runkle resolved the linting. Should be good now.


Unable to import TargetResponse from 'agenteval.targets'

A reproducible test case or series of steps
Failed when importing "from agenteval.targets import BaseTarget, TargetResponse"
The version of our code being used
agent-evaluation 0.1.0
Any modifications you've made relevant to the bug
NIL
Anything unusual about your environment or deployment
NIL, tested on AWS Sagemaker, Google Collab and local venv python. All python version 3.10.6 / 3.11.4

Whenever I import the line from agenteval.targets import BaseTarget, TargetResponse , i get the error:
ImportError: cannot import name 'TargetResponse' from 'agenteval.targets' (/opt/conda/lib/python3.10/site-packages/agenteval/targets/__init__.py)

Upon checking the source file, the TargetResponse does not exist on the folder level, importing this way works:
from agenteval.targets import BaseTarget
from agenteval.target_response import TargetResponse

If possible please update the user guide here

@tengfone Thanks for raising this! We will work on the fix.


Cloud Computing Resources Added

Hey, thanks for your contribution. Only one link per PR.
No new sections are required


numpy slice assignment failing on readinto

readinto failing to perform slice assignment:
~/active/cell/filesystem_spec/fsspec/spec.py in readinto(self, b)
   1249         # for i in range(len(data)):
   1250         #     b[i] = data[i]
-> 1251         b[: len(data)] = data
   1252         return len(data)
   1253

ValueError: invalid literal for int() with base 10: ''

fsspec version: 0.7.2
Context:
I am one of the developers for aicsimageio, relevant to this issue is the fact that we use delayed dask arrays to read microscopy images, but as a part of that we support very large files (usually this means >200GB). We natively support handing in str and Path to the AICSImage object as one would expect and I was in the process of extending support to reading from S3File or really, any fsspec file like object.
Bear with me...
Commands for reproduction setup:
git clone git@github.com:AllenCellModeling/aicsimageio.git
cd aicsimageio
git checkout feature/support-s3-file
pip install -e .
python

(The above was done in a conda environment on python 3.7)
Commands to reproduce:
from aicsimageio import AICSImage
from s3fs import S3File, S3FileSystem
fs = S3FileSystem(anon=True)
f = S3File(fs, "allencell/aics/pipeline_integrated_single_cell/cell_images_3d/00001bb5_16872_35317_reg.tiff")
img = AICSImage(f)
print(img.shape)

Traceback:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/maxfield/active/cell/aicsimageio/aicsimageio/aics_image.py", line 207, in shape
    return self.size()
  File "/home/maxfield/active/cell/aicsimageio/aicsimageio/aics_image.py", line 197, in size
    return tuple([self.dask_data.shape[self.dims.index(dim)] for dim in dims])
  File "/home/maxfield/active/cell/aicsimageio/aicsimageio/aics_image.py", line 197, in <listcomp>
    return tuple([self.dask_data.shape[self.dims.index(dim)] for dim in dims])
  File "/home/maxfield/active/cell/aicsimageio/aicsimageio/aics_image.py", line 152, in dask_data
    reader_data = self.reader.dask_data
  File "/home/maxfield/active/cell/aicsimageio/aicsimageio/readers/tiff_reader.py", line 176, in dask_data
    self._dask_data = self._construct_dask_array(self._file, tiff)
  File "/home/maxfield/active/cell/aicsimageio/aicsimageio/readers/tiff_reader.py", line 128, in _construct_dask_array
    sample = scenes[0].pages[0].asarray()
  File "/home/maxfield/miniconda3/envs/aicsimageio-test/lib/python3.7/site-packages/tifffile/tifffile.py", line 4587, in asarray
    out=out
  File "/home/maxfield/miniconda3/envs/aicsimageio-test/lib/python3.7/site-packages/tifffile/tifffile.py", line 6400, in read_array
    n = fh.readinto(result)
  File "/home/maxfield/miniconda3/envs/aicsimageio-test/lib/python3.7/site-packages/fsspec/spec.py", line 1241, in readinto
    b[: len(data)] = data
ValueError: invalid literal for int() with base 10: ''

Comments:
From some debugging, if we ignore all the aicsimageio reader code, the ultimate error that occurs occurs during the slice assignment in readinto. This same file, which can be downloaded here, (or with any other S3 downloader), can be read in to AICSImage just fine as a str or Path so it seems to be this slice assignment operation. I found what I would consider a really weird (and not optimal) but working solution of replacing the slice assignment with a simple loop and single item assignment: link
def readinto(self, b):
    """mirrors builtin file's readinto method

    https://docs.python.org/3/library/io.html#io.RawIOBase.readinto
    """
    data = self.read(len(b))
    for i in range(len(data)):
        b[i] = data[i]
    # b[: len(data)] = data
    return len(data)

Which I will be opening a PR with for feedback on how to solve the issue in a more optimized fashion.
Using the loop above, the above image reading code now works:
from aicsimageio import AICSImage
from s3fs import S3File, S3FileSystem
fs = S3FileSystem(anon=True)
f = S3File(fs, "allencell/aics/pipeline_integrated_single_cell/cell_images_3d/00001bb5_16872_35317_reg.tiff")
img = AICSImage(f)
print(img.shape)

(1, 1, 6, 64, 168, 104)

Is the length of the data zero here by any chance?
readinto is designed to work with bytearrays and buffers/memoryviews. I suppose in your case the assignee is a numpy array, so maybe all inputs should be wrapped in a memoryview?
def readinto(self, b):
    """mirrors builtin file's readinto method

    https://docs.python.org/3/library/io.html#io.RawIOBase.readinto
    """
    data = self.read(len(b))
    memoryview(b)[: len(data)] = data
    return len(data)

or memoryview(b).cast("B")[: len(data)] = data

TIL, memoryview
Both suggestions work!
To answer the question of what is in the data:
type: bytes
content: a whole bunch of \x00 (literally nothing else)
length: 17472

I recommend you use the latter suggestion in your PR. I suppose your numpy data type already uint8 (which is normal for images), but you need the cast for other cases where filling in the binary contents of a numpy array with any other type
I'd also appreciate a test showing that this works. The roundtrip would be something like
arr = numpy.arange(10, dtype=dt
arr.tofile(temporary_file)
arr2 = np.empty_like(arr)
 with fsspec.open('temp', 'rb') as f:
     f.readinto(arr2)
assert (arr == arr2).all()

which I would check for dt= 'uint8', 'int8', 'int32', 'float64'

a whole bunch of \x00

(might have been optimizable... :) )

Thanks! PR updated with tests. Sorry for not knowing too much about how to test this stuff. Appreciate the input :)


Fix performance problem for Tables.Columns(::KeyedArray)

There's a type stability issue which occurs in Tables.columns when the type of the keys are different on different axes.
I demonstrate that the problem exists, and that this PR fixes it. Not sure what kind of regression test would be optimal, but happy to add one / some if there's an obvious place to put them.
MWE
using AxisKeys, Tables

D1 = 2500;
D2 = 2500;
data = randn(D1, D2);
k1 = collect(1:D1);

# Choose between these two to swap between having the same and different key types.
k2 = Int32.(collect(1:D2));
# k2 = collect(1:D2);

X = KeyedArray(data; k1, k2);
@benchmark Tables.columns($X)

on master then output of benchmarking is
BenchmarkTools.Trial: 8 samples with 1 evaluation.
 Range (min … max):  654.951 ms … 766.341 ms  ┊ GC (min … max):  8.84% … 17.49%
 Time  (median):     696.806 ms               ┊ GC (median):    12.74%
 Time  (mean ± σ):   704.700 ms ±  42.018 ms  ┊ GC (mean ± σ):  13.40% ±  4.94%

  █             ▁      ▁  ▁                    ▁     ▁        ▁
  █▁▁▁▁▁▁▁▁▁▁▁▁▁█▁▁▁▁▁▁█▁▁█▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁█▁▁▁▁▁█▁▁▁▁▁▁▁▁█ ▁
  655 ms           Histogram: frequency by time          766 ms <

 Memory estimate: 604.74 MiB, allocs estimate: 22445012.

On this branch it's
BenchmarkTools.Trial: 252 samples with 1 evaluation.
 Range (min … max):  18.942 ms … 24.603 ms  ┊ GC (min … max): 0.00% … 21.99%
 Time  (median):     19.035 ms              ┊ GC (median):    0.00%
 Time  (mean ± σ):   19.889 ms ±  1.933 ms  ┊ GC (mean ± σ):  4.34% ±  7.99%

  █▇                                                        ▂
  ███▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▄▄▁▁▇█▄▇██ ▅
  18.9 ms      Histogram: log(frequency) by time      24.4 ms <

 Memory estimate: 71.53 MiB, allocs estimate: 10.

By changing the value of k2 to comprise Int64s, you can verify that this doesn't cause a performance regression for the situation in which the keys are the same type, in which case there isn't presently a type-instability.

This really kinda feels like a hack around a limitation with Julia? Maybe we should raise this as example against the core language?

Okay, I realize it's a bit more work, but would you mind extracting this logic out into a separate internal function to:

Add a type stability unit test
Use as a MWE
Make it easier to just delete / rewrite if the stability issue is fixed in the future

Very happy to do that. Could you suggest where I put the type-stability test? I'm not sure what the appropriate location is

Okay. I've factored out the code. The following should suffice as a unit test:
@inferred AxisKeys._get_keys_columns((randn(5), randn(Float32, 4)))

but I'm unsure what the correct location for it is.

I think at the end of the Tables.jl tests should be fine. Please and thank you!
https://github.com/mcabbott/AxisKeys.jl/blob/master/test/_packages.jl#L106

@rofinn tests added!

Awesome, thanks @willtebbutt!


[dv/kmac] cycle-accurate scb

Consider not support app interface for cycle accurate check.
I think it could be simpler as this:

SCB receive data_in,
SCB waits for design ack with a timeout check to ensure nothing stuck,
3). SCB check digest data when received done signal, and check intr_state.
In the meantime, check sha3_status Idle is set to 0.

Then we do not need this app_with_partial_data_vseq anymore.

Hi @cindychip, marking as M2, but please update if needed.

Done in RPs:
https://github.com/lowRISC/opentitan/pull/14519
https://github.com/lowRISC/opentitan/pull/14374
https://github.com/lowRISC/opentitan/pull/14295


Issue #270 radio driver

Issue #270 radio driver

Don’t think we need a separate PR for this since it’s already covered by the Checkbox PR https://github.com/sociomantic-tsunami/nessie/pull/273


Downloaded plots have cut off info

Hi,
first of all thank you for this amazing, intuitive tool. It's so great to be able to detect mutational signature and compare to existing ones using one button!
I would like to report an issue regarding downloaded plots. It seems that some information is cut off, e.g.
prof96_plot: n in Relative contribution, the name of the file if it's longer than ~10 characters
signatures_plot: the name of the file on the bottom, values on the scale are not displayed fully
comparison_with_cancers: names of signatures are pretty tight (that's lower priority) but the names of cancers like nasopharyngeal carcinoma or oral gingivo buccal squamous (cell carcinoma) are cut off, also the name of the file if it's as long
I am using only one sample.
Thanks so much!
Musica_infoCutOff.pdf

Thank you, that is a good workaround for now. Good luck with the application!

Closed by inactivity and focus on new projects.


Integrate Nano cryptocurrency in V4V spec, adding documentation

This adds a Value 4 Value specification and associated constants for the Nano cryptocurrency.
If there's another procedure for registering and declaring a cryptocurrency in the V4V spec, please let me know and I'll be sure to do it.

There is one constraint and perhaps we do need to lift it: the current Value Block spec says only one type of value block per feed. I always thought it should be more but I backed down on that.
I'm happy to re-add my Hive suggestion too but I'm not in a position to drive adoption of it.

There is one constraint and perhaps we do need to lift it: the current Value Block spec says only one type of value block per feed. I always thought it should be more but I backed down on that.

Related to that - https://github.com/Podcastindex-org/podcast-namespace/issues/300

@zhyatt Could you resolve the conflicts or do a new PR on this one?  I got behind in merging some other ones and it looks like it messed you up.

@daveajones I have resolved the conflicts.
The appendix section appears to be very lightning specific, and is a nested heading of the lightning section. In my commit I have kept the appendix together under the lightning heading, but this may be out of place?
I do think the document could perhaps do with some restructuring. For instance, the value override example is shown under the Supported currencies > Lightning heading, whereas it's more of a general example.
It may also be worth hiding the supported currencies section within <details> blocks (if they're still supported?), allowing the reader to expand only the relevant sections.

Thanks!!


vscode extension can not spawn electron process after package to vsix

VSCode Version: 1.25.1 (1.25.1)
OS Version:  mac os 10.13.6

Steps to Reproduce:

clone the project https://github.com/njleonzhang/vscode-extension-mardown-image-paste
install the dependency via npm install
open it in vscode.
run in test mode, press F5
run plugin command in the extension development vscode instance: Paste Image.
A electron app should be opened.
build the extension to vsix via npm run package
install the vsix to vscode.
run plugin command: Paste Image.

expected:
A electron app should be opened.
fact：
no electron app is opened.
About the extension
It spawn electron process for leverage the electron api clipboard.readImage for reading image from the clipboard. It now works as expected in debug mode, but fails in release mode because electron can not be started. No error is shown, I do not know how to debug further. Please help.

@isidorn it should be a bug, not a question. it is not work in release mode....

fyi @joaomoreno to decide

Yeah this is very likely an environment variable which messes Electron up.


MM-15457 E2E - Fix integrations Cypress spec

Summary

Update the integrations test to use better selectors.
Had to add ID's to a handful of elements for the test to use.
Instead of driving UI to allow these settings, now use an API call.

Ticket Link
JIRA

@reflog I want to abstract them out, but they should probably live somewhere else rather than off of cy. cy was getting a bit polluted with custom commands that were really only used for one test. Don't worry though, these are on my radar to fix :)

💪


Listeners??

Good re-work but You didnt forget about listeners for action like Open? Close ? for swiping items?

Yes, i removed the listeners for open or close action.
Are you want to use them?

Yes, I need listeners like in old version library. Have you working version with listeners?

I think listeners useless, so removed them.
I will add them in next version ：）

So how to check which item is opened?

SwipeMenuLayout.isMenuOpen()，but it is not case left or right menu.

public boolean isMenuOpen() {
	return (mLeftSwiper != null && mLeftSwiper.isMenuOpen(getScrollX()))
                || (mRightSwiper != null && mRightSwiper.isMenuOpen(getScrollX()));
}

For future would be nice if You will add listeners. But now I solved my problem so closing issues.

Yes, i will add that in upcoming major release :)


Typeahead "limit" if equal to results returned, shows "no-data".

Bloodhound returns correct number of results. But if the number of results is same as the limit option in the datasets, then this results in showing "no-results" instead of showing all of them.
I have created this JSFIDDLE that demonstrates this issue.
http://jsfiddle.net/rash805115/kosocz84/
Note: Searching for 1947 returns exactly 5 rows.
When limit is set to 4:
Searching for 1947 returns 4 results.
When limit is set to 5:
Searching for 1947 returns nothing.
When limit is set to 6:
Searching for 1947 returns one 1 result - the first result.
PS: My first time reporting a bug, so maybe I am not clear enough. Please teach me if I am wrong or something is missing.

I had the same bug today, here's what I found, hope it helps (it did for me)
http://stackoverflow.com/questions/31007825/bootstrap-typeahead-not-showing-hints-as-expected

Awesome. I will also update the issue I opened and mark it as duplicate of
yours. Can you tell me the issue number that you opened.
Regards.
On Tue, Jul 7, 2015 at 6:05 PM, Philippe Boyd<EMAIL_ADDRESS>wrote:

I had the same bug today, here's what I found, hope it helps (it did for
me)
http://stackoverflow.com/questions/31007825/bootstrap-typeahead-not-showing-hints-as-expected
—
Reply to this email directly or view it on GitHub
https://github.com/twitter/typeahead.js/issues/1312#issuecomment-119357711
.

--
Regards,
Rahul Chaudhary
Ph -<PHONE_NUMBER>

I'm not the one who asked the question on stackoverflow sorry...
but from the looks of it, i think it's this one : #1299

This appears to be the issue fixed by #1212.

@rsutphin thanks for the update...any ideas when the new build will be released. Closing this issue.

you can set limit Infinity,


[JN-864] Add international address validation

DESCRIPTION
Right now, there are only 3 supported countries - UK, Mexico, and Canada. I could add more, but I figured we wanted to do it slowly and these seem like the 'MVP' international countries we want to support - let me know your thoughts. We could add a separate ticket to go through and add more countries so that we can focus on that / spend more time ensuring each individual country is good.
To reduce input errors, I used a react select component of the supported countries, but you can add anything you want. It just won't allow you to validate in those situations.
Also refactors AddressValidationClient to be AddressValidationService to disambiguate from SmartyClient.
TO TEST:  (simple manual steps for confirming core behavior -- used for pre-release checks)

Obtain trial API key for international address validation from smarty.com (could be the same as old account key if you have one, just need to also enable international address trial)
Provide as SMARTY_AUTH_ID and SMARTY_AUTH_TOKEN in run config
Go back to any participant's profile
Try entering US address like before, ensure no regressions
Try entering random address in: Mexico, Great Britain, Canada
Be sure to validate multiple times to ensure no weird formatting changes (e.g. if you accept the changes, if you hit validate again it shouldn't suggest a different address again, it should just go green).

I just discovered https://stackoverflow.com/questions/29919596/how-do-i-get-a-list-of-countries-in-my-website so I will update this to display country names instead of hardcoding them. This will allow us to be sensitive to preferred language. I'll put it back to a draft until I get it working.

@devonbush @MatthewBemis After finding and fixing 15 edge cases and 5 different ways this could go wrong... This is now tentatively under review 🤕


The acorn's parse function fails on valid js code

The parser fails at the following line:
function a(){function b(){}/k/;}

This seems to be related to e6b433cd309846102c0cf97dc7ec8dcc4be87a89 and #396. Indeed, this bug is not reproducible on<EMAIL_ADDRESS>Interestingly,
function b() {} /k/;

works, but
function a() {
  function b() {} /k/;
}

doesn't.

There's some rather subtle state logic in the tokenizer to be able to distinguish regexps from division operators. This was caused by a bug in that (and only occurs when the function statement is preceded directly by an opening brace). Attached patch should help.


CupertinoNavigationBar animation flickering

Steps to Reproduce
After updating Flutter to v1.9.1+hotfix.4, I'm getting a weird flickering in CupertinoNavigationBar when changing pages. Here's a video that demonstrates the issue, please look at NavigationBar during changing screens:
https://youtu.be/MmUaFUVVtyo
Sometimes it's flickering, sometimes it's not, but most of the time it is flickering. If a button exists in the trailing property, this button sometimes is rendered below the navigation bar for a few frames, and it goes back into proper position. It looks very ugly.
I'm using very simple method for creation of the page:
import 'package:flutter/cupertino.dart';

class AddNewDuePage extends StatefulWidget {
  @override
  _AddNewDuePageState createState() => _AddNewDuePageState();
}

class _AddNewDuePageState extends State<AddNewDuePage> {
  Widget buildBody() {
    return Text("OK");
  }

  @override
  Widget build(BuildContext context) {
    var _navigationBar = CupertinoNavigationBar(
      middle: Text("Nowa składka"), // TODO: I18N
    );

    return CupertinoPageScaffold(
      navigationBar: _navigationBar,
      child: SafeArea(child: buildBody()),
    );
  }
}

Is this a problem with my Flutter installation, or is it a regression?
The problem exists in Debug compilations on Emulator & real hardware, as well as in Release compilations in real hardware (can't run release compilation in emulator).
Target Platform: Linux (emulator) + Android
Target OS version/browser: 9
Devices: Android Emulator, also Sony Xperia XZ Premium
Flutter doctor
$ flutter doctor -v
[✓] Flutter (Channel stable, v1.9.1+hotfix.4, on Linux, locale en_US.UTF-8)
    • Flutter version 1.9.1+hotfix.4 at /home/antek/dev/android/flutter
    • Framework revision cc949a8e8b (3 weeks ago), 2019-09-27 15:04:59 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /home/antek/Android/Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /home/antek/Android/Sdk
    • Java binary at: /home/antek/dev/android/android-studio/jre/bin/java
    • Java version openjdk version "1.8.0_152-release"
    ✗ Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup
      for detailed instructions.

[✓] Android Studio (version 3.4)
    • Android Studio at /home/antek/dev/android/android-studio
    • Flutter plugin version 35.3.1
    • Dart plugin version 183.6270
    • Java version openjdk version "1.8.0_152-release"

[!] IntelliJ IDEA Community Edition (version 2019.2-old)
    • IntelliJ at /home/antek/data/apps/idea/idea-IC-192.5728.98
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

! Doctor found issues in 2 categories.

Not sure why doctor didn't detect flutter/dart plugins for IDEA, because I'm using Idea with those plugins. Also everything is working with my Android toolchain, also not sure why doctor has issues with licenses.

CC @justinmc

I was able to reproduce this with a very simple app, so it seems to be a regression bug in Flutter!
import 'package:flutter/cupertino.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      title: 'Flutter Demo',
      home: MyHomePage(title: 'First Page'),
      routes: {
        '/second': (BuildContext context) => MyHomePage(title: 'Second Page'),
      },
    );
  }
}

class MyHomePage extends StatelessWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      navigationBar: CupertinoNavigationBar(
        middle: Text(title),
      ),
      child: Center(
        child: CupertinoButton(
          child: Text('Go'),
          onPressed: () {
            Navigator.pushNamed(context, '/second');
          },
        ),
      ),
    );
  }
}

It seems to have been caused by https://github.com/flutter/flutter/pull/31614 based on me trying old versions.

@gspencergoog Do you have any idea why that PR may have caused the text in CupertinoNavigationBar to flicker between routes?  Nothing jumps out as super obvious to me.

Some details:
I confirmed that https://github.com/flutter/flutter/pull/31614 is the first change that has the bug in it.  Checking out its merge commit, 7775c23784fdd7890a453031e0451fbed3731247, exhibits the bug, but checking out the commit directly before it, fdd554fb46e885292fb9c82e91969e21b603a1c3, does not.
That PR is a re-land of https://github.com/flutter/flutter/pull/30040.  If I check out the merge commit from that original PR, 4218c0bc3, then I do not see the bug.

There's a different issue reported here that looks related, video: http://treefortress.com/shared/2019-10-20_09-49-24.mp4.

I can manually size the container to get rid of it, but these containers are variable width, depending on the size of their text, so that's not ideal.

@GaryQian could there be a problem with text sizing? From the description it feels like we're not giving the Text widget the width it needs.

Here's a minimal code example that should reproduce the problem. Click the text to trigger the transition, you should see it flicker intermittently:
http://treefortress.com/shared/2019-10-21_00-19-14.mp4
It seems related to horizontal movement. If I change alignments so it animates vertically, (TopLeft > BottomLeft), it doesn't flicker at all.
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(home: Page1());
  }
}

class Page1 extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.black,
      body: GestureDetector(
        child: Container(
          alignment: Alignment.centerLeft,
          child: HeroContent(),
        ),
        onTap: () {
          Navigator.of(context).push(PageRouteBuilder(
            pageBuilder: (_, __, ___) => Page2(),
            transitionDuration: Duration(seconds: 2),
          ));
        },
      ),
    );
  }
}

class Page2 extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.black,
      body: Container(
        color: Colors.transparent,
        alignment: Alignment.centerRight,
        child: HeroContent(),
      ),
    );
  }
}

class HeroContent extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Hero(
      tag: "hero",
      child: Material(
        color: Colors.transparent,
        child: Container(
          child: Text(
            "Aries",
            style: TextStyle(fontSize: 42, color: Colors.white),
          ),
        ),
      ),
    );
  }
}

Small update, using Row() does not completely get rid of the issues. When using Row() + MaterialRectArcTween, I get overflow errors in extremely small values:

The following assertion was thrown during layout:
A RenderFlex overflowed by 0.000895 pixels on the right.

User-created ancestor of the error-causing widget was:
Column file:///D:/_Development/gskinner.com/flutter-vignettes/flutter-vignettes-git/constellations_list/lib/constellation_title_card.dart:31:18
The overflowing RenderFlex has an orientation of Axis.horizontal.
The edge of the RenderFlex that is overflowing has been marked in the rendering with a yellow and black striped pattern. This is usually caused by the contents being too big for the RenderFlex.

The most robust fix I've found so far is to just wrap the text in spaces " FooBar ".

Surrounding text with spaces indeed helps with flickering, but doesn't help when sometimes the "trailing" widget in CupertinoNavigationBar is displayed in the wrong place for few frames.
I'll try downgrading Flutter to previous version, it was much more stable than now.

Unfortunately this issue still exists in 1.12.13+hotfix.5

@antekone Can you try it on master?  I just tried my example app above on my iPhone 7 and I couldn't reproduce the problem.

You mean channel master?
When testing your example app on version 1.13.3-pre.60 / channel master on an Android phone (Xperia G8141), as well as on Android emulator, the problem still exists.
Flutter 1.13.3-pre.60 • channel master • https://github.com/flutter/flutter.git
Framework • revision ceef460a64 (6 hours ago) • 2019-12-18 00:23:02 -0800
Engine • revision 500f9cedf6
Tools • Dart 2.8.0 (build 2.8.0-dev.0.0 2d332ee1d3)

I don't own a Mac, so I can't check it on an iPhone.

Newest version (1.12.13+hotfix5) has broken lots of additional things (i.e. showCupertinoDialog is utterly broken now, where previously it worked well). It's now clear to me that nobody checks if Cupertino theme works on Android.
I've downgraded Flutter to 1.3.9:
Framework • revision f91df4abe1 (10 months ago) • 2019-03-09 21:19:28 -0500
Engine • revision 4e54bc93ca
Tools • Dart 2.2.1 (build 2.2.1-dev.1.0 2fb6cd9f5f)

Everything is smooth and nice on this version: there is no flickering that I'm describing in the original post, as well as showCupertinoDialog works without issues. I'm going to try to release newer version of my app on this older SDK. If it will go well, I'll port my app away from Flutter to some other framework.

Hi @antekone, sorry about the poor experience you've had with the latest stable. Could you elaborate a bit more on showCupertinoDialog being utterly broken (or maybe create a new issue for it)? The gallery app uses showCupertinoDialog and it seems to be working fine.
As for this issue, I believe we should keep the issue open since downgrading is not a real fix. I don't have a proper fix for this floating point arithmetic problem yet but I think multiple workarounds exist (e.g., the one @esDotDev pointed out, disabling softwrap on the text widget). In the meantime if you have a proper fix for the problem, pull requests are always welcome!

@LongCatIsLooong

I don't think this is merely floating point arithmetic problem. If the user has selected a bigger font in system settings, the fonts are jumping around the navigation bar even if I surround them with multiple spaces and disable softWrap property. Here's a link to the video that shows this problem (fast forward to 4:14 if you don't have time to watch through the full test):
https://www.youtube.com/watch?v=DbJePwR-lno

Sorry for using such vague descriptions like "utterly" when ranting about showCupertinoDialog. To show you what I mean, I've prepared two repositories:

one for old SDK that can be compiled with Flutter 1.3.9: https://github.com/antekone/show_cupertino_dialog_bug_old_sdk
another one for new SDK that can be compiled with Flutter 1.12.13+hotfix.5: https://github.com/antekone/show_cupertino_dialog_bug_new_sdk

Both repositories contain the same project with the same source files in the lib directory. Just the build scripts are different, because building 'old_sdk' version fails with new Flutter, and building 'new_sdk' fails when building with old Flutter.
This is the video what happens under different versions of Flutter. First I'm trying to run it on 1.3.9, then on 1.12.13+hotfix.5.
https://www.youtube.com/watch?v=Zh395g5wkd4

I think the showCupertinoDialog change is a feature: https://github.com/flutter/flutter/pull/42967/files. You can remove the SystemPadding widget now since the viewInsets is already taken care of by the dialog. To prevent future breakages one option is to provide your tests as suggested here: https://flutter.dev/docs/resources/compatibility.
Haven't had the chance to look into what might have caused 1 but I'll update when I find the issue.

Issue still exist
Code Sample
import 'package:flutter/cupertino.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      debugShowCheckedModeBanner: false,
      title: 'Cupertino App',
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      navigationBar: CupertinoNavigationBar(
        middle: Text('Cupertino App Bar'),
      ),
      child: Center(
          child: CupertinoButton(
              child: Text('Go Next'),
              onPressed: () {
                Navigator.push(context,
                    CupertinoPageRoute(builder: (_) => AddNewDuePage()));
              })),
    );
  }
}

class AddNewDuePage extends StatefulWidget {
  @override
  _AddNewDuePageState createState() => _AddNewDuePageState();
}

class _AddNewDuePageState extends State<AddNewDuePage> {
  Widget buildBody() {
    return Text("OK");
  }

  @override
  Widget build(BuildContext context) {
    var _navigationBar = CupertinoNavigationBar(
      middle: Text("Nowa składka"), // TODO: I18N
    );

    return CupertinoPageScaffold(
      navigationBar: _navigationBar,
      child: SafeArea(child: buildBody()),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel dev, v1.16.3, on Mac OS X 10.15.4 19E266, locale en-GB)
    • Flutter version 1.16.3 at /Users/taha/Code/flutter_dev
    • Framework revision be3a4b37b3 (4 days ago), 2020-03-26 18:08:05 -0700
    • Engine revision b2bdeb3f0f
    • Dart version 2.8.0 (build 2.8.0-dev.17.0 1402e8e1a4)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/taha/Code/sdk
    • Android NDK location not configured (optional; useful for native profiling
      support)
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/taha/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4, Build version 11E146
    • CocoaPods version 1.9.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 44.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build
      1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.43.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.8.1

[✓] Connected device (4 available)
    • iPhone 11  • 41EB2A85-7EB4-4FD2-8978-FBBE2A681D74 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-13-4 (simulator)
    • macOS      • macOS                                • darwin-x64     • Mac
      OS X 10.15.4 19E266
    • Chrome     • chrome                               • web-javascript •
      Google Chrome 80.0.3987.149
    • Web Server • web-server                           • web-javascript •
      Flutter Tools

• No issues found!

I experience the same issue, it really seems to be related to horizontal movement. Interesting though is that only the last part flickers, see reference here:
https://youtu.be/uO5BjCTElY8

Still there in this version:
Flutter 1.17.3 • channel stable • https://github.com/flutter/flutter.git
Framework • revision b041144f83 (3 days ago) • 2020-06-04 09:26:11 -0700
Engine • revision ee76268252
Tools • Dart 2.8.4

Any updates or workaround?

still there on 1.22.6

This is caused by RelativeRect in used in transitions may produce a small overflow from floating point subtraction. The fix is quite simple - just add precisionErrorTolerance to them. I'm wondering how many else places in the framework may contain something unobvious like this


Show original author name

First, awesome feature!!! Can't believe Discord doesn't have this :(
I tried it, the message was moved. But the new message doesn't show the original author name.
To be clear, I moved it , not the original author of the message.

If I understand you correctly, the message is moved to the new channel, but the author of the original message is not mentioned, is that correct? Could you provide a screenshot showing what you are describing?

hmm
the first time I got
<@null> | #bug-report
Mon Apr 19 2021 09:07:21 GMT+0000 (Coordinated Universal Time)

But now I tried again , and it worked
@fransyozef_rollerskate_app | #bug-report
Tue Apr 20 2021 18:46:07 GMT+0000 (Coordinated Universal Time)

Other users have reported this issue as well. I suspect this may have to do with the way the discord library handles caching of users. Was it by chance a new server or a user new to the server?
This is something I'm tentatively looking into, but am as of yet unsure if it is something I can resolve on my end or a problem with discord.


[test] Update APINotes, Serialization, and PrintAsObjC tests for Swift 3 removal

Mostly done with find/replace to compare Swift 4 and Swift 5 instead of Swift 3 and Swift 4. There are some interesting changes here because API notes for 4 also apply to 3 unless otherwise specified, and 4.2 is the next version after 4 rather than 5, but everything should still be correct and testing useful stuff.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 480e02b0cfd01c3c3cc53dc5bdb5aba808a910e6

More LLDB test flakery. I'll double-check that we have a bug tracking this.
@swift-ci Please test Linux

Filed SR-8245.

Same issue. One more try, then I'll give up and wait for the LLDB team.
@swift-ci Please test Linux


DAG workflow

Add DAG workflow view, like https://github.com/dagu-dev/dag

The repo you references seems to be private or does not exist. I get a 404.

I'm sorry, The correct link: https://github.com/dagu-dev/dagu

@dreamCodeMan neat UI. Not sure that it makes sense for Tork though as it's not a DAG-based workflow engine.


Is there a way to query historical weather averages for a location by API / JSON?

No, unfortunately not yet. The historical data is expensive and we have no access to historical data at the moment. This could be theoretically possible in future though.
We already have similar inquires, e.g. #551. I close this ticket in favor of that one..


Can't find jpg file

I have a problem that if I move outside the map I get errors that it no longer finds files for these coordinates because they do not exist. I wanted to add maxbound but it doesn't work do you have any good solution for this problem?

Leaflet has in the map object a config named:
max-bounds="[[-4000, -5500],[8000, 6000]]"
and
max-bounds-viscosity="1.2"
which limits the user from going out of bounds.
Leaflet will still try to load those non existing maps, but you can use
errorTileUrl to replace those tiles with a color matching one.


useSearch is triggered for all consumers when query parameters changed

Describe the bug
I'm currently working on an app where we use some query strings as part of our state and thus useSearch is used in a few key places, including some expensive components. Now the issue we have is that whenever we use navigate to update the search, it triggers all consumers of useSearch even if they read different keys. That is to be expected with the current API. So I'd expect to be able to do something like what you could do with most state managers that connect to react, having a selector API in between that only triggers a re-render when the selected slice changes. So something like:
const slice = useSearch(({ neededKey }) => neededKey);

Now, that being said, maybe there's already a way to achieve something similar and we've just been missing it or designing things poorly, in which case, what would the solution be like?
Your Example Website or App

Steps to Reproduce the Bug or Issue

Expected behavior

Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

The latest version out now has selector and track props that allow you return a slice to watch instead of the whole set. Let me know how it works out for you!


Add tags to the ServiceBinding secret

With this change an additional key tags will be added to the ServiceBinding secret, with the tags being fetched from the corresponding offering.
It's mostly a follow-up change to the one introduced as a fix to #61, in order to provide a better compatibility with the native CF services, which are being moved to K8s (tags is a mandatory key for the VCAP_SERVICES env variable).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: pbusko:x: kulhadiaYou have signed the CLA already but the status is still pending? Let us recheck it.

closed based on the comment


ga dataprotection

https://github.com/Azure/sdk-release-request/issues/3902

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-arm-dataprotection


how can I load remote data into a  hierarchy data's chidren with v5

for example :
let rootData = d3.hierarcy(data);
when I click dataNode, want to load more data on a node, but it's a d3.hierarcy data,
the remote data is a simple json, so i can't use the code in v3:
node.children = data;
bacause that data is not a h3.hierarcy.
so I use the coe like this:
const nodes = d3.hierarchy(res).children;
(nodes || []).forEach((n) => {
const nd = n;
nd.depth = c.depth + 1;
nd.parent = c;
});
c.children = nodes;
to add depth and parent to remote data's chidren, but I think it's not a right way.
so can you please help me ? thanks

See https://observablehq.com/@d3/d3-hierarchy for examples, in particular https://observablehq.com/@d3/tidy-tree?collection=@d3/d3-hierarchy uses a remote file.
To ask for help, please use the Stack Overflow tag d3.js instead. Thousands of D3-related questions have been asked there, and some answers may be relevant to you.


test failure in Scala 2.11 community build

recently seen at e.g. https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-community-build/345/consoleFull:
[scala-gopher] [error] Failed tests:
[scala-gopher] [error] 	gopher.channels.SchedulerStartupTest

maybe the test should have a longer timeout or something?

Let's check: I increased timeout to 3 sec instead one:
https://github.com/rssh/scala-gopher/commit/259e90ac16e0df7df75d4510478487d980daab0c
The test itself check the basic functionality of akka scheduler, so only two reasons of failure are possible: timeout and problem in akka scheduling initialization.

Looks like issue is fixed in 347: https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-community-build/347/console

thanks Ruslan!