Error<EMAIL_ADDRESS>Invalid bottle tag symbol and thus no able to install

brew doctor output
brew doctor
Your system is ready to brew.

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.0.25-4-g59bc0e9
ORIGIN: https://github.com/Homebrew/brew
HEAD: 59bc0e96c661a9d98f58ca803f0a1d56a18468d7
Last commit: 7 hours ago
Core tap origin: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 83616b7938b965fd006a6ba14a6029a6fecb9a69
Core tap last commit: 4 months ago
Core tap branch: master
Core tap JSON: 27 Jun 02:51 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 10
Homebrew Ruby: 2.6.10 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: 10-core 64-bit arm_firestorm_icestorm
Clang: 14.0.0 build 1400
Git: 2.39.2 => /opt/homebrew/bin/git
Curl: 7.86.0 => /usr/bin/curl
macOS: 13.2.1-arm64
CLT: <IP_ADDRESS>.1.1668646533
Xcode: N/A
Rosetta 2: false

What were you trying to do (and why)?
I have already<EMAIL_ADDRESS>installed on my Mac using brew, I need to install 7.2 as well for another legacy project.
What happened (include all command output)?
When I ran the command brew install<EMAIL_ADDRESS>it gave me "Warning: No available formula with the name<EMAIL_ADDRESS>Did you mean<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>or php@5.6?"
Then I ran brew tap exolnet/homebrew-deprecated to make it work, and now when I run brew install<EMAIL_ADDRESS>, it gives me this error also:
Warning: No available formula with the name<EMAIL_ADDRESS>Did you mean<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>or<EMAIL_ADDRESS>==> Searching for similarly named formulae and casks...
Error<EMAIL_ADDRESS>Invalid bottle tag symbol
brew tap exolnet/homebrew-deprecated
==> Tapping exolnet/deprecated
Cloning into '/opt/homebrew/Library/Taps/exolnet/homebrew-deprecated'...
remote: Enumerating objects: 101, done.
remote: Counting objects: 100% (6/6), done.
remote: Compressing objects: 100% (4/4), done.
remote: Total 101 (delta 2), reused 5 (delta 2), pack-reused 95
Receiving objects: 100% (101/101), 23.49 KiB | 2.13 MiB/s, done.
Resolving deltas: 100% (48/48), done.
Tapped 3 formulae (20 files, 85.9KB).
What did you expect to happen?
Just wanted to install PHP 7.2 (Already have PHP 8.2 installed on my system and I need to have both 8.2 and 7.2 versions for different projects)
Step-by-step reproduction instructions (by running brew commands)
`brew install<EMAIL_ADDRESS>`brew tap exolnet/homebrew-deprecated`

Php 7.2 is no longer supported as November 2020. See https://endoflife.date/php. Barring a handful of exceptions, we only keep maintained software in Homebrew/core. See https://docs.brew.sh/Acceptable-Formulae.
Error<EMAIL_ADDRESS>Invalid bottle tag symbol is a problem in the exolnet/deprecated tap. That tap is archived, so I don't think that's going to get fixed.
Try using the shivammathur/php tap instead. Note: this is a third-party tap that is not affiliated with the Homebrew organisation, so we cannot vouch for anything you install from there.


Button onPress doesn't work using keyboard if onKeyUp/onKeyDown is passed

🐛 Bug Report

While using useButton, if you pass onKeyUp or onKeyDown as props, onPress callback doesn't fire on clicking Enter or Space keys
🤔 Expected Behavior

onPress should work independent of onKeyUp or onKeyDown
😯 Current Behavior

Accessibility breaks when keyboard event callbacks are passed. Here is a codesandbox. https://codesandbox.io/s/hopeful-gagarin-up2fc4?file=/src/App.js
🔦 Context

I am building a design system on storybook. Storybook automatically detects props that accept callbacks and passes one so that it can log all the callback responses. So this way the component broke as onPress never worked with keyboard. This is how I found out about this bug.
💻 Code Sample
https://codesandbox.io/s/hopeful-gagarin-up2fc4?file=/src/App.js
🌍 Your Environment

Software
Version(s)

react-aria
3.24.0

Browser
Chrome

Operating System
MacOS

Looks like onPress and onKeyDown are being called on Enter/Space in your attached code sandbox. What version are you running with locally?

@LFDanLu Sorry. I have updated the issue. This only happens when we use onKeyUp.

Also, if onKeyUp is passed, then is-active class stops working.


bug: Using .addPhrase with Angular script optimization causes error that prevents Angular from bootstrapping

Expected behavior
I expected .addPhrase to include my added obscene term and build properly.
Actual behavior
Using .addPhrase in an Angular component and optimization: scripts = true in angular.json's build config causes the following error:
Cannot read properties of undefined (reading 'Literal');
This error prevented Angular from bootstrapping
Minimal reproducible example
No response
Steps to reproduce
No response
Additional context
No response
Node.js version
v18.12.1
Obscenity version
v0.2.0
Priority

[ ] Low
[ ] Medium
[ ] High

Terms

[X] I agree to follow the project's Code of Conduct.
[X] I have searched existing issues for similar reports.

Please provide a MRE and steps to reproduce.


Transaction Identifier

Good morning,
Am I not understanding the code the right way, or is the transaction identifier in the frame the master sends, the function code, and not a unique number that identifies the transaction?
Sorry for the trouble, and thank you very much for your help

The transaction ID is used to identify the request when you use asynchronous request. In this case you can send multiple request but you may not get them back in order. You can use the ID to identify the request in the call back function.


SWARM-1427: Swarm hangs on shutdown with Consul Topology

Motivation
Topology consul unadvertise always sends a deregister to consul for every registered service, totally ignoring the provided name adress and port. This results in unwanted behaviour and leads to errors and deadlock on shutdown
Modifications
added a filter before the foreach to filter out only those registrations that match the provided data and only deregister those.
Result
only matching services are deregistered from consul

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you created a JIRA and used it in the commit message?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[ ] Have you built the project locally prior to submission with mvn clean install?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Okay to test

Strange failure.
Retest this please.


Add more test files

Added two json test files

Why did you close the other PR?
In the future, please submit individual PRs for different test definitions. If everything is bundled in a single PR, then any discussion on a single JSON file will prevent all the files that are fine from being merged.

Will do

On 18 Oct 2015, at 16:49, Katrina Owen<EMAIL_ADDRESS>wrote:
Why did you close the other PR?
In the future, please submit individual PRs for different test definitions. If everything is bundled in a single PR, then any discussion on a single JSON file will prevent all the files that are fine from being merged.
—
Reply to this email directly or view it on GitHub.

I believe all of the feedback in here is incorporated in the other individual PR's. I assume I should close this PR?

Good point. I'll close it now. Thanks!


ADPU Payload Error companion app sent wrong size

When I try to send HNT from my Ledger Nano X, I get the error message - ADPU Payload Error, companion app sent wrong size.
Can you please explain how to resolve this issue?
Thanks

I get the exact same notification on my ledger display. Am wondering if this is a fresh problem seeing as you and I are the only people posting about it. Regardless, as of now - my HNT is locked in my wallet and can't be sent. Very frustrating.

I would add for the sake of relevancy - am trying to send HNT to Binance. I notice Kucoin has nodes that are in perpetual maintenance.

Support to send Ledger payment on the Wallet (this repo is actually the hotspot app) app is not yet available. this is why you're seeing the problem. Please try again when it's announced.

Is there any update on this issue? I am receiving the same issue and am wondering if it is related to needing to update the device app version or the desktop version.

Apologies. I was able to resolve the issue by going and downloading the latest version of the helium wallet. Apparently my wallet at the time wasn’t the newest version.
Sent from Proton Mail for iOS
On Sat, Sep 24, 2022 at 4:09 PM, cameronnili5 @.***> wrote:

Is there any update on this issue? I am receiving the same issue and am wondering if it is related to needing to update the device app version or the desktop version.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

@Zerkaman , thanks for following up. I started using the Ledger HNT wallet prior to the new guidance around importing the ledger wallet via the mobile app (see here). That said, my mobile wallet HNT app is not synced with my ledger wallet. I created a wallet on the Helium blockchain totally just using my ledger device and have accessed it via the CLI. My suspicion is that there is a version incongruency with the device HNT app and the desktop app. No matter how many times I try to use the latest desktop app (v2.2.4), it keeps call the old version in the CLI (v2.1.3).
Any tips? I'd appreciate it! Worried my HNT is locked up :(

@Zerkaman this is the CLI. Notice the device app and desktop app difference.

I really don’t. I wish I did. If your HNT ends up locked perpetually, I’d say that eventually it’ll become ubiquitous and the problem will solve itself as more and more people use it and the problem becomes more common. I know that doesn’t help you in the short term, but I wouldn’t worry about never ever being able to get your hands on it properly. I think it will happen eventually just by default.
Sent from Proton Mail for iOS
On Sat, Sep 24, 2022 at 4:35 PM, cameronnili5 @.***> wrote:

@.***(https://github.com/Zerkaman) , thanks for following up. I started using the Ledger HNT wallet prior to the new guidance around importing the ledger wallet via the mobile app (see here). That said, my mobile wallet HNT app is not synced with my ledger wallet. I created a wallet on the Helium blockchain totally just using my ledger device and have accessed it via the CLI. My suspicion is that there is a version incongruency with the device HNT app and the desktop app. No matter how many times I try to use the latest desktop app (v2.2.4), it keeps call the old version in the CLI (v2.1.3).
Any tips? I'd appreciate it! Worried my HNT is locked up :(
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

I only had about a grand worth of money locked up in my HNT, so even if I didn’t manage to get unlocked (which I did), the hit wouldn’t have been the end of the world.
I was so frustrated that I thought about making a YouTube video posting and waiting for the comments to roll in. You’d have to assume that other people are experiencing the same problem and looking for answers there. Maybe that is a route you could go.
Sent from Proton Mail for iOS
On Sat, Sep 24, 2022 at 4:41 PM, cameronnili5 @.***> wrote:

@.***(https://github.com/Zerkaman) this is the CLI. Notice the device app and desktop app difference.
image
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks. I've submitted a ticket with Ledger. It feels as though they should resolve this as they were the ones who recommended using experimental mode in the first place. Will post the results here.

Just want to close this out... I resolved the issue. The problem was that when I setup my Ledger HNT wallet, there was no new Helium Wallet App (now there is) so I needed to use my CLI to access the wallet (super clunky UX!). Now, with your ledger, you can simply import your HNT wallet into the Helium Wallet App (an app that is separate from the hotspot app). Much better UI/UX!!

I am also having this same issue. Helium-ledger-cli not working, not able to send transactions from it

My solution ended up being to download version 2.2.3
I was trying to make the transfer on version 2.1.3 and that appears to have been the problem. Was unaware that a new version had been released at the time. Likewise, I also downloaded the Helium Wallet App and connected my ledger to it.
Beyond any of that, I don’t think I’d be any help. Good luck!
Sent from Proton Mail for iOS
On Sun, Nov 13, 2022 at 4:15 PM, fintechtrades @.***> wrote:

I am also having this same issue. Helium-ledger-cli not working, not able to send transactions from it
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

What version of ledger were you using?
The helium-ledger-cli version
Sent from Proton Mail mobile
-------- Original Message --------
On Nov 13, 2022, 4:20 PM, Zerkaman wrote:

My solution ended up being to download version 2.2.3
I was trying to make the transfer on version 2.1.3 and that appears to have been the problem. Was unaware that a new version had been released at the time. Likewise, I also downloaded the Helium Wallet App and connected my ledger to it.
Beyond any of that, I don’t think I’d be any help. Good luck!
Sent from Proton Mail for iOS
On Sun, Nov 13, 2022 at 4:15 PM, fintechtrades @.***> wrote:

I am also having this same issue. Helium-ledger-cli not working, not able to send transactions from it
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

I don’t remember what version the ledger firmware specifically was at that point, but the helium cli I was incorrectly trying to use was version 2.1.3
I was unaware that helium had released an update. I downloaded the latest version and boom - everything transferred out fine. I don’t know if they’ve released an even newer version since then, but you might wanna check.
Sent from Proton Mail for iOS
On Sun, Nov 13, 2022 at 4:27 PM, fintechtrades @.***> wrote:

What version of ledger were you using?
The helium-ledger-cli version
Sent from Proton Mail mobile
-------- Original Message --------
On Nov 13, 2022, 4:20 PM, Zerkaman wrote:

My solution ended up being to download version 2.2.3
I was trying to make the transfer on version 2.1.3 and that appears to have been the problem. Was unaware that a new version had been released at the time. Likewise, I also downloaded the Helium Wallet App and connected my ledger to it.
Beyond any of that, I don’t think I’d be any help. Good luck!
Sent from Proton Mail for iOS
On Sun, Nov 13, 2022 at 4:15 PM, fintechtrades @.***> wrote:

I am also having this same issue. Helium-ledger-cli not working, not able to send transactions from it
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

I recommend not using the CLI method as it seems like there is no active support for that anymore since the new Helium Wallet App is available.

Thx. Will do so if and when I sell remainder of my stash.
Sent from Proton Mail for iOS
On Tue, Nov 15, 2022 at 4:03 PM, cameronnili5 @.***> wrote:

I recommend not using the CLI method as it seems like there is no active support for that anymore since the new Helium Wallet App is available.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

You may experience this error if you are using a Macbook Pro M2, my solution was to switch to a Windows PC and try it, and it worked fine


Error : content-length with this repo zip archive.

What is happening here ?
>>> import unzip_http
>>> rzf = unzip_http.RemoteZipFile("https://github.com/saulpw/unzip-http/archive/refs/heads/master.zip")
>>> rzf.namelist()
github.com Accept-Ranges header ('') is not 'bytes'--trying anyway
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/askhento/.local/lib/python3.10/site-packages/unzip_http.py", line 150, in namelist
    return list(r.filename for r in self.infoiter())
  File "/home/askhento/.local/lib/python3.10/site-packages/unzip_http.py", line 150, in <genexpr>
    return list(r.filename for r in self.infoiter())
  File "/home/askhento/.local/lib/python3.10/site-packages/unzip_http.py", line 159, in infoiter
    self.zip_size = int(resp.headers['Content-Length'])
  File "/home/askhento/.local/lib/python3.10/site-packages/urllib3/_collections.py", line 260, in __getitem__
    val = self._container[key.lower()]
KeyError: 'content-length'

Using WSL2 on windows 10.

Hi @Askhento, github.com is not returning the Content-Length header, which is required.  According to https://stackoverflow.com/questions/76162297/github-api-not-always-returning-content-length-header, it might be possible to affect the returned headers by sending certain headers.   If you do a HEAD request manually with e.g. curl, you can experiment with sending different headers, and seeing which headers are returned.  If you find something that works reliably, we can try adding to the library.  Good luck.


Autoplay Video When Displaying Video Page

Looks like this already exists
https://github.com/lbryio/lbry-app/issues/584


Logout handling in backend

When receiving a logout event, we need to handle it by calling sessions.Logout()... but we need a handler first

frontend need to send request for logout.


jQuery 3 - window load inside ready state will not be triggered

In #14620 I proposed to put code from POS_LOAD section to jQuery ready state. But this not working in in jQuery 3. See https://github.com/jquery/jquery/issues/3194
Need to replace jQuery(function ($) { $(window).on('load', function () { /*code*/ }); }); with jQuery(window).on('load', function () { /*code*/ });
Sorry for this.
What steps will reproduce the problem?
Register some code in POS_LOAD section.
What is the expected result?
Code must be executed when page content fully loaded.
What do you get instead?
Code doesn't run at all.
Additional info

Q
A

Yii version
2.0.13

PHP version
*

Operating system
*

Would you please make a pull request?


Version 2.0.0 doesn't install /Styles/Colors.xaml correctly

Describe the bug
If you install version 2.0.0 via NuGet the files /Styles/Colors.xaml cannot be located.
To Reproduce
Steps to reproduce the behavior:

Install package version 2.0.0 via NuGet
Attempt to reference in App.xaml, e.g.:
Try to start application (or something VS will warn before)
Exception IOException: Cannot locate resource 'styles/colors.xaml' is thrown

Doing same steps with version 1.6.5 does not reproduce this issue.

@will-scc The styles, colors and brushes are now located in new resource dictionaries. Please read the new quick-start guide or here at the wiki. It should help you. Thx 🙏


markdown

I would like to help convert ebooks to markdown

Volunteer efforts to convert books to html include "Standard Ebooks" and "Distributed Proofreaders". If you are interested to help authors convert to Markdown, you might look at the forums for gitbook or leanpub. Free-programming-books does not convert books at all.


How to enable caching?

I am using this angularjs-pdf in IONIC framework ( which is entirely based on AngularJS). Everything works fine, except when I change the page, it flickers a lot and takes time to render, going from top to bottom. Even if the page is previously visited, it takes the same time to load it. Is there a way to simply cache at least the pages which are already rendered once? Please let me know if somehow I'm doing this wrong!

Hi @darshantejani007,
page is not cached, every time you change page, the page is reloaded and re-rendered.

@dennybiasiolli Thanks! That much I understood. Now is there a way to flip the switch of caching? I want to enable it!


update sha-256 value

Closing due to inactivity, @waletdevsec please reopen if you are able to address the requested changes.
Proposing also a solution to fix this package and avoid that it fails again in https://github.com/mandiant/VM-Packages/issues/828


Implement color hotkeys

Same key bindings as http://www.ircbeginner.com/ircinfo/colors.html
Even though the characters are invisible (in Chrome at least), it's still useful as an initial implementation.
I moved all hotkey bindings into their own scope (add ?w=1 to url for saner diff)

Nice, this works well 👍. After this gets merged perhaps making it easier to see that you've pressed this would be good -- right now it's just a little hard to tell that you've actually entered one of these chars until you hit enter.

Sadly this is almost impossible with the current textarea, but if we ever go for a rich-content equivalent, it would. Not on the agenda for now though.
I reckon @xPaw had a crazy idea with tailor-made font or something, but I can't find it in the issues/PRs.

@astorije it's a branch
On Tue, 20 Dec 2016, 03:28 Jérémie Astori<EMAIL_ADDRESS>wrote:

Sadly this is almost impossible with the current textarea, but if we ever
go for a rich-content equivalent, it would. Not on the agenda for now
though.
I reckon @xPaw https://github.com/xPaw had a crazy idea with
tailor-made font or something, but I can't find it in the issues/PRs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thelounge/lounge/pull/810#issuecomment-268127929, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAlb07RM58UPJe8b5bM6_8DHtJxQOdLvks5rJy8ygaJpZM4LP9-f
.

Ah ah, I see the result, pretty crazy indeed! I don't think the font-family falls back on the next font when a char is missing (i.e. all non-control-code chars in the case of a font with just B, I, U symbols), does it?

I've updated the PR to correctly insert formatting characters at the position where the cursor is. Also supports selections.

@xPaw, I added the new shortcuts in the help windows, let me know what you think. I'll review your recent changes as well.

I'm seeing issues like this on macOS;

@astorije I don't quite understand why we have separate sections for windows and macOS bindings. The only difference is the meta key (ctrl vs ⌘)

Because that might not be the case for long, due to the different nature of OSes. As a matter of fact, they used to be different before a recent PR.
Also, I think we should quickly hide this distinction. It must not be difficult to detect if we are on Windows/Linux or macOS on the front-end and display the according section only. I am less certain on how to be sure we have a keyboard on the client or not.


whenBlock - show nBlocks / second in most recent X blocks

The reason whenBlocks exsits is so people can see how close they are to being synced. It would be interesting to show the speed of the most recent X blocks synced per second. When the hack blocks are occuring, it's super slow. When the regular sync is happening, it's much faster.

Moved to https://github.com/Great-Hill-Corporation/quickBlocks/issues/22


SVG Scaling Issue in IE11 (and maybe others)

The SVG image on IE doesn't scale.
Good stuff @tylerlong.

What do you mean by "doesn't scale"?
I don't have IE to test. I tested in other browsers and I do notice some issue.  The labels and text will be misplaced when scaling up. Not sure it's the same issue you reported.

Sorry, should have been more clear and provided an example.
Attached what the default diagram looks like on IE11 vs. Chrome on the same monitor.
On IE11 the SVG is tiny and no text is even visible.

The IE issue is not a new issue. I guess we have to give up IE but only support Microsoft Edge.

ok thank you


How i can set Ui Like Ios in Android.

we have downloaded Q-municate application in Android and iOS, and tested by calling on each cross device, in iOS its shows perfect calling information like No Answer | missed Call | Audio / Video Call. But in Android it just shows Call Notification,
How i can make similar like iOS to android.

Hello
Replay Please?????


Ledger Generate Account Fails, missing dependency

Describe the bug
Im unable to generate a new account with ledger, appears error from missing dep.
To Reproduce
Steps to reproduce the behavior:
1. using NEAR Cli (v1.5.4)
2. Command: near generate-key --useLedgerKey name.near
3. Get error
Expected behavior
I should be able to generate a keypair and approve on a ledger
Additional context
 Error: Cannot find module '@ledgerhq/hw-transport-node-hid'
 Require stack:

*   /.nvm/versions/node/v15.3.0/lib/node_modules/near-cli/middleware/ledger.js
     ```

Are you trying to add a key? It is rather confusing, but I believe you'll want to use add-key instead:
https://docs.near.org/docs/tools/near-cli#near-add-key

I've also tried installing near-cli today on a fresh new Mac M1. Node 16.6.0, getting the same error.

Solved this by using NodeJS 14.20.0


fix: topological sort for complicated intersection

Description
cherry-pick https://github.com/tier4/autoware.universe/pull/1482
Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@saka1-s Please confirm if this PR is intended


Use local main.tf by default

Proposed changes

Changes to resolve the following issues:

#52 - using a local backend now by default
#55 - using the terraform binary now and redirecting the output to a log file.
#41 - system deploy logs are now concise

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (changes to code, which do not change application behavior)

Checklist

[x] I have filled out this PR template
[x] I have read the CONTRIBUTING doc
[x] I have added automated tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (README.md, inline comments, etc.)
[ ] I have updated the CHANGELOG.md under a ## next release, with a short summary of my changes

Relevant Links

Further comments

@nathanagood -- do you think we could do an in person review of this PR, given its size?

@nathanagood -- do you think we could do an in person review of this PR, given its size?

@eschwartz - If by "in-person" you mean over vid, sure. I'd hate to wait until we are physically in-person.


coordinate on testing

I created a project:
https://c-testsuite.github.io/
https://github.com/c-testsuite/c-testsuite
and wondered if you would like to join in, things are sparse and need organizing, but perhaps you understand what I am aiming for.

I'll see what I can do.
Btw, the page doesn't link to the tcc repo.
Also, there should be a way to separate C89 from C99 from C11. Perhaps, there should be 3 sets of tests (3 dirs, 3 prefixes, comments on 1st line, ifdefs, something).

Totally agree, that's one of the things I want feedback on, the biggest challenge is organization.


Basic HTML includes with simple javascript library for bootstrap 4

For static sites, Surely the best use case for html includes is static site generators like jekyll or hugo.
I am not denying that but many static websites ( specially bootstrap ones ) dont want to use these static site generators simply because its complicated at first and thus use this type of below crutch
document.write("<h1 class=\"task\">Task for the day<\/h1>");
This type of code is meant for testing only and isn't for production but still websites use this type of codes because its easy to get started with. Infact there are many html to js convertors for the same use case.
As per my best of knowledge and experience, this doc_write is mostly and widely used in navbars, sidebar, and footers of website which is surely not great!!
So i wanna purpose this simple below thing for **bootstrap.js**
http://www.w3schools.com/w3css/w3data_includes.asp
Demo => http://www.w3schools.com/w3css/tryit.asp?filename=tryw3data_html_include_2
Yup, maybe bootstrap is not meant for this type of stuff but what's better than saving the developer's and designer's day ??

+1 , Even i face this thing daily. Would be nice if this ads to bootstrap. Less Hacky then document write.

+1 , Even i face this thing daily. Would be nice if this ads to bootstrap. Less Hacky then document write.

I don't think we need this in Bootstrap. Besides HTML imports and web components seem like a much safer bet for this. If I even understand the problem correctly? Could you describe a use case?

I don't think we need this in Bootstrap. Besides HTML imports and web components seem like a much safer bet for this. If I even understand the problem correctly? Could you describe a use case?

Here is the use case,
My Company where i work has 1000's of pages in their website,
And we are using static html pages and dynamic java pages in between, its frankly imposible for us to use a static site generator
But our Navbar and footer remains the same throught the website ,
so we use JS File for both with doc_write statements in there. so that we dont have to make it repetitive
As we are using bootstrap, and if there is this feature in bootstrap, we dont need to use doc_write again, we can easily include the header.html and footer.html with a simple line of code.
Yes i admit this is not the best use case but not every site uses things like angular, react, jekyll etc and feel that this will be the best case for the old school static pages.
Rest on you guys to decide @hnrch02

Here is the use case,
My Company where i work has 1000's of pages in their website,
And we are using static html pages and dynamic java pages in between, its frankly imposible for us to use a static site generator
But our Navbar and footer remains the same throught the website ,
so we use JS File for both with doc_write statements in there. so that we dont have to make it repetitive
As we are using bootstrap, and if there is this feature in bootstrap, we dont need to use doc_write again, we can easily include the header.html and footer.html with a simple line of code.
Yes i admit this is not the best use case but not every site uses things like angular, react, jekyll etc and feel that this will be the best case for the old school static pages.
Rest on you guys to decide @hnrch02

I see. That seems to be a solution to a very specific problem though with not many other possible applications. Besides this should probably be handled server-side with your templating engine not on the client.
So I'm fairly confident in saying that we're not going to add this, considering A) the required amount of work we would have to put in and the arguably small benefit it would provide to the vast majority of Bootstrap users and B) the better first party solutions provided by browsers with HTML imports and web components.

I see. That seems to be a solution to a very specific problem though with not many other possible applications. Besides this should probably be handled server-side with your templating engine not on the client.
So I'm fairly confident in saying that we're not going to add this, considering A) the required amount of work we would have to put in and the arguably small benefit it would provide to the vast majority of Bootstrap users and B) the better first party solutions provided by browsers with HTML imports and web components.

no issue @hnrch02

no issue @hnrch02


Add open graph / twitter metadata

The current site doesn't show a preview / snippet when sharing it via Twitter. I assume the same thing happens on Facebook and other platforms.
It looks like the template for the new site already has the parts to make it work, not sure if the necessary data is actually provided somewhere:
https://github.com/OpenWebAdvocacy/website/blob/main/src/_includes/partials/meta-info.njk

Good catch.  Not sure where the current site is hosted from, but we'll probably want to sync the text images for both until a final cutover can be made to this repo.

https://github.com/open-web-advocacy/website << here's the current website if anyone's got time to do a push.

@mtom55 I shall take this on and file a PR.

This has been addressed, closing issue.


Provide reliable, locked-in-time method for starting Dockerized VinylDNS for client integration testing

Motivation
At present, go-vinyldns, terraform-provider-vinyldns, and vinyldns-cli run acceptance/integration tests in CI against a vinyldns/bin/docker-up-vinyldns.sh --api-only'd instance of VinylDNS running at localhost:9001. Per the directive and advice of the VinylDNS team, these tests assume various characteristics of the resulting localhost:9001 VinylDNS API, such as the presence of pre-loaded data.
However, it appears that various VinylDNS changes have been introduced that undermine these assumptions, in effect causing the above-mentioned acceptance tests -- and CI processes -- to break.
Here is an example failure, which illustrates the test assumptions about pre-loaded VinylDNS API data: https://travis-ci.org/vinyldns/vinyldns-cli/jobs/471905286
In my mind, it's necessary to agree on a testing strategy -- and a means of starting a localhost VinylDNS API -- that ensures against such drift in the future. It may be worth noting that this has happened a few times previously, as well.
Scope of change
The Dockerized instance of VinylDNS running at localhost:9001 via bin/docker-up-vinyldns.sh --api-only.

How to do proper compatibility testing at the api/version level across tooling has been problematic.  Recommend researching on a solution.


Tooltip doesn't show in fullscreen

Hi there,
This tool-tip has been very helpful for a project I'm working on; however, the tool-tip doesn't show up on hover (or at all) when in full screen. I haven't been able to get it to work and I noticed that in the example it doesn't show up unless the bar is clicked. http://bl.ocks.org/Caged/6476579
Any ideas on how to get it to function the same in full screen?
Thanks in advance!

Hi @jessmitch42,
Ohh, interesting.  I was able to reproduce this in Chrome.  It works fine in Safari. Thanks for the report.


ARROW-10879: [Packaging][deb] Restore Debian GNU/Linux Buster support

I removed Debian GNU/Linux Buster configuration wrongly in
20c5c6cb9f760bbb23e335c986aadcd9c9f09181 (ARROW-10814). I should have
removed Stretch configuration in it but removed Buster configuration.

@github-actions crossbow submit debian-* ubuntu-*

+1
The buster and bionic failures are related to #8472 and ARROW-10867.


Image Filenames are Transformed into all lower-case

Although I agree with the issue raised here: https://github.com/netlify/netlify-cms/pull/853 that the comparison of names should be case-insensitive, our team has a situation where we migrated a site over and have assets that have capitalized names (e.g. Organizational_Pledge.pdf); these files exist and function fine in our site. However, when we try to update them (replacing in place) using the NetlifyCMS Media interface, it does so, but it also transforms the filenames into all lower case (e.g. organizational_pledge.pdf).  This breaks a lot of links coming from outside of the website (ads, external sites, etc.), so we have to manually update it in Git, which obviously is not ideal.

Closing as stale and/or outdated.


Update MOZ 3W data

MOZ is now missing sector data and org name so will not make it into the database.
cc @turnerm

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1352

Relevant Lines:
1455

💛  - Coveralls


openmw-tes3mp: 0.7.1 -> 0.7.0-alpha

Describe the bug
the latest version of openmw-tes3mp in nixpkgs does not match the release version people use to host TES3MP servers ( 0.7.0-alpha). as a result, when someone tries to connect, they are kicked because their client's version doesn't match the server version.
Steps To Reproduce
steps to reproduce the behavior:

install openmw-tes3mp (this will require data files from the game The Elder Scrolls III: Morrowind)
connect to a server ( I have a server up that is hosted at https://bottom.rs )
watch yourself get kicked

Expected behavior
the packaged version of TES3MP should connect to a server without an issue
Additional Content
the 0.7.0-alpha builds are advertised by the TES3MP community and it's developers as stable builds, listed here: https://github.com/TES3MP/openmw-tes3mp/releases/tag/0.7.0-alpha and that 0.7.1 is a dev build that is not meant for general use, usually with breaking features and additions that are still being tested out.
this is my first time filing a bug in this repo and i am not sure if this is the right place

it seems this pr is what breaks this https://github.com/NixOS/nixpkgs/pull/123607
this pr rectifies ffmpeg3 vulns, but also changes the branch used from a stable build to an indev build, breaking compatibility with currently hosted public servers as a unintended side effect.
cc @peterhoeg

Yeah, we should probably switch back to the officially sanctioned branch. Feel like taking a stab at it? Ping me if you need help.

admittedly i dont know much about nix or how to package stuff for nixos, but (in theory) it may be as simple as swapping out a few lines
i guess this is a good place to start learning though, so ill give it a stab and see what comes of it

Any luck @nixdaemon  ?

@nixdaemon , there's a new 0.8.0 out just FYI.


start a node: add clarification on specifying flags every time

Fixes #1855, fixes #1813.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/2d9f29f323bccbb60116a33e9b545e29db6c9c22/

Done.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3d4e72581725e8333830a06817e64b661ba325be/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3d4e72581725e8333830a06817e64b661ba325be/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b20d90394e20c38f73d71bff8dbf138c149c9952/

@arjunravinarayan, you need to update your copy of master, rebase this branch against it, and push again to get past the TC failure. Then we can merge!

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4ad29ea8d7c77caf5eeef772c723f12bac3ce370/


[FEATURE] Visibility of current tenant

What is the bug?
Tenancy if enabled is a central concept in OpenSearch Dashboards however visibility into the tenant a user is currently in is hidden within menus or a few clicks away. This experience is frustrating for new and frequent users and can lead to the creation of Dashboard objects in the wrong tenant or confusion when they are loaded into a default tenant.
Currently if I want to see which tenant I am in then switch a tenant I have too:

Click the user icon in the top right
Notice the small text under the user role
Click the "switch tenant" CTA
Interact with the modal
Confirm

What is the expected behavior?
Due to the importance of tenancy as a concept in Dashboards this information should be more visible in the interface. For example, a part of the bread-crumbing or in the navigation header so users are always aware of the tenant they are in and can easily switch to another if they would like too. They should not have to look through menus or left to guess what tenant they are in.
Idea: Tenant as a fixed breadcrumb item - clicking it opens the tenant selection modal

What is your host/environment?

OpenSearch Dashboards Version: 2.3.0
Browser: Google Chrome

Work to address this (and other) issues is being tracked here https://github.com/opensearch-project/OpenSearch-Dashboards/issues/4298


Liquidity Withdrawal

I encountered the error when trying to remove liquidity from the pool as market creator/owner.
When trying to remove anything less than 100%, there is an error popping up after signing tx.
"A transferal of funds out of a swaps pool was below a threshold specified by the receiver."

When You try to remove 100% of liquidity from the pool, it gives the below error:
"A pool balance dropped below the allowed minimum."

The operations were performed on the LP:
https://app.zeitgeist.pm/liquidity/22
The market is:
https://app.zeitgeist.pm/markets/31

Some details on this... It seems like the min_asset_amount is specified incorrectly by the UI. For example, the pool contains 1827 ZTG, but when withdrawing about 25% of the liquidity, the minimum amount of ZTG specified is 511.7 (OP is the only pool owner). It should be less than a quarter of the amount of ZTG held in the pool.


Update user_tables.cql

Add index to user roles.

Can one of the admins verify this patch?

Okay to test

Heheh very good, thanks!

Build here: http://jenkins.sirwellington.tech/job/Banana Data Operations/166/


removing CircleCI workflow(not in use)

CircleCI no longer in use

Not sure I understand -- why can't we add it (actually not even add -- replace) in this PR?


Fix for .data() method to read scientific notation as string as is.

Problem

// This is true
jQuery('.data-scientific').data('unsigned') === '1234e5';

// This is false, because cash will return 123400000 due to JSON.parse('1234e5') parses it to a number.
cash('.data-scientific').data('unsigned') === '1234e5';

Solution

test for scientific notation using regex, if true then return value and do not JSON.parse'd it

Can you delete all the dist and unnecessary changes from this PR?
Looking at the code shouldn't this be working already? Like JSON.parse ( '1e99' ) is parsed correctly for me, is the string not reaching JSON.parse for some reason?

Hi Fabio, JSON.parse will convert the '1e99' into a number. What we want for .data() is to return '1e99' as '1e99' and not as<PHONE_NUMBER>. So before we pass the string to JSON.parse, we need to check if it has a scientific notation format. This is consistent with jQuery, Zepto, and native Element.dataset.get() routine.
I will make another pull request with only the changes meant for the fix for .data() method. We can disregard this request if you want.
Thanks!

Thank you but our data function is explicitly not fully aligned with jQuery, as jQuery's implementation doesn't work with stuff like React, which some people, me included, use alongside more direct DOM manipulation libraries.
Moreover if there's a data-foo="123"attribute somewhere jQuery's data function will read that as a 123 number, so not reading things like 1e10 as a number too seems inconsistent to me, this should probably be something jQuery should change, not the other way around.

I respect your decision on this.
Regarding jQuery and Element.dataset, they probably based it on HTML Spec from WHATWG, where values of dataset properties are returned as DOMString.
https://html.spec.whatwg.org/multipage/dom.html#domstringmap
https://webidl.spec.whatwg.org/#idl-DOMString
Still, cash-dom's .data() method might be a more practical implementation in real world case than it is to follow the specs strictly.
Thank you for your review and have a nice day!

You might consider using the attr method if what you want is always a string. Potentially you could also override Cash's data implementation. Unfortunately the best way to write such function isn't obvious, there are pros and cons of both approaches.
Sorry for the inconvenience.


Switching work week to day view Not getting current day (CRITICAL BUG)

@howljs
Same as week view to day view also not working.very critical issue
version : 1.13.2

@howljs
I think it works based on first day of week.
For current week it should be show current day.other weeks first day of week day

same here

Fixed in v2.0.0


Excel integration

@dcharbon Could you assign myself and @zyzhu as owners of the Deedle.Excel package please? https://www.nuget.org/packages/Deedle.Excel/1.0.4-rc1

@zyzhu what is your nuget username? Is it the same as your github username?

@dcharbon I just registered. It's the same as GitHub. Thanks!


context.entityId not correct in Android web view of group tab

It looks like issue #1335, which was reported about a year ago, still isn't fixed. As it was closed by a bot and also locked down I can't add my report there or ask for the status.
The problem is that context.entityId returns a very different value in the Android web view version of my group tab. When running in the browser, the Desktop app or the iOS app, the entityId I receive is the one I configured when the tab was created. In the Android web view I get a GUID f9d43794-c483-4952-b728-575678c3013b which I never set myself but which seems to come from the Teams core. The result is that the tab obviously doesn't work in the Android app, as I can't load the data for my tab.
If there won't be a fix for this, is there a workaround I can use? I'd be willing to save the GUID the Android app gets in my data model and have an alternative way of loading the data. But where does this GUID come from?

Hi @sr258,
I have gone through the issue, we are investigating on it.
Sorry for the delay, It's been pending form long period. Will get back to you as soon as possible.

@sr258 , We have a Bug for this in our backlog, we will inform you once this sorted out

Hi @sr258, We have got confirmation from Engineering Team that the fix is not yet verified and the feature is not yet enabled for android. Hence, we are closing this question for now.

Bug is Resolved now, we have tested this at our end, could you please also check and confirm?


Fix for error returns in wait for any expression and add a method in runtime for timeout

$subject

Closing this PR due to changes requested


Frame synchronization for GoPro, Pupil Core, and Surveillance camera

Hello! Thank you so much for your outstanding work.
As Pupil core records a video at varying frame rates, how did you synchronize the gaze and head directions with the chest direction computed from different cameras in each frame?
To the best of my knowledge, the average fps of videos recorded by Pupil Core can be calculated, but the starting frame and ending frames are still different from the frames extracted from the videos recorded by GoPro cameras or surveillance cameras, which create videos at almost constant frame rates.
I would really appreciate your guidance.
Thank you very much

Thank you for your interest!
We used SyncBac PRO with :pulse by Timecode Systems. These devices allow timestamps to be stored as metadata in the video recorded by each GoPro camera, so we it this for synchronizing all GoPro cameras.
To synchronize Pupil Core and GoPro, we recorded display screen of :pulse, which shows the timestamp, with the outside camera of Pupil Core. We then manually align starting frame of Pupil Core with the timestamp.
Because Pupil Core also stores timestamps separately by itself, we convert the format of Pupil Core's timestamps to GoPro's timestamps. We then interpolate head and gaze directions and picked up interpolated values at each GoPro's frame (timestamp) to deal with the varying frame rates of Pupil Core.

Thank you very much.
Now, it is clear to me how Gopro cameras and Pupil Core are synchronized.
Did you also record the display screen of pulse with the surveillance camera for synchronization?

No, we didn't. We used GoPro for surveillance cameras, too.

Thank you very much!


Cannot install Percollate dependencies

I am attempting to install Percollate on a machine running Linux Mint Debian Edition 32bit. I have used NVM to install NPM 10.22.0, which has worked on other distros before. I believe that the issue may be due to this machine being 32bit? My virtual machines are 64bit.
I want to give this laptop to a friend who would find percollate useful.
Here is some of my terminal logging:
(name)@(name)-mint:~$ npm i -g percollate
/home/(name)/.nvm/versions/node/v10.22.0/bin/percollate -> /home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/percollate/cli.js
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

<EMAIL_ADDRESS>updated 1 package in 11.858s
(name)@(name)-mint:~$ npm i -g canvas

<EMAIL_ADDRESS>install /home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas
node-pre-gyp install --fallback-to-build

node-pre-gyp ERR! install response status 404 Not Found on https://github.com/Automattic/node-canvas/releases/download/v2.8.0/canvas-v2.8.0-node-v64-linux-glibc-ia32.tar.gz
node-pre-gyp WARN Pre-built binaries not installable for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v64 ABI, glibc) (falling back to source compile with node-gyp)
node-pre-gyp WARN Hit error response status 404 Not Found on https://github.com/Automattic/node-canvas/releases/download/v2.8.0/canvas-v2.8.0-node-v64-linux-glibc-ia32.tar.gz
Package pixman-1 was not found in the pkg-config search path.
Perhaps you should add the directory containing pixman-1.pc' to the PKG_CONFIG_PATH environment variable No package 'pixman-1' found gyp: Call to 'pkg-config pixman-1 --libs' returned exit status 1 while in binding.gyp. while trying to load binding.gyp gyp ERR! configure error  gyp ERR! stack Error: gypfailed with exit code: 1 gyp ERR! stack     at ChildProcess.onCpExit (/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:351:16) gyp ERR! stack     at ChildProcess.emit (events.js:198:13) gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:248:12) gyp ERR! System Linux 4.19.0-17-686 gyp ERR! command "/home/(name)/.nvm/versions/node/v10.22.0/bin/node" "/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "configure" "--fallback-to-build" "--module=/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas/build/Release/canvas.node" "--module_name=canvas" "--module_path=/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas/build/Release" "--napi_version=6" "--node_abi_napi=napi" "--napi_build_version=0" "--node_napi_label=node-v64" gyp ERR! cwd /home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas gyp ERR! node -v v10.22.0 gyp ERR! node-gyp -v v5.1.0 gyp ERR! not ok  node-pre-gyp ERR! build error  node-pre-gyp ERR! stack Error: Failed to execute '/home/(name)/.nvm/versions/node/v10.22.0/bin/node /home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js configure --fallback-to-build --module=/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas/build/Release/canvas.node --module_name=canvas --module_path=/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas/build/Release --napi_version=6 --node_abi_napi=napi --napi_build_version=0 --node_napi_label=node-v64' (1) node-pre-gyp ERR! stack     at ChildProcess.cmd.on (/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas/node_modules/@mapbox/node-pre-gyp/lib/util/compile.js:89:23) node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:198:13) node-pre-gyp ERR! stack     at maybeClose (internal/child_process.js:982:16) node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5) node-pre-gyp ERR! System Linux 4.19.0-17-686 node-pre-gyp ERR! command "/home/(name)/.nvm/versions/node/v10.22.0/bin/node" "/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build" node-pre-gyp ERR! cwd /home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas node-pre-gyp ERR! node -v v10.22.0 node-pre-gyp ERR! node-pre-gyp -v v1.0.5 node-pre-gyp ERR! not ok  Failed to execute '/home/(name)/.nvm/versions/node/v10.22.0/bin/node /home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js configure --fallback-to-build --module=/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas/build/Release/canvas.node --module_name=canvas --module_path=/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/canvas/build/Release --napi_version=6 --node_abi_napi=napi --napi_build_version=0 --node_napi_label=node-v64' (1) npm ERR! code ELIFECYCLE npm ERR! errno 1 npm ERR<EMAIL_ADDRESS>install:node-pre-gyp install --fallback-to-build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/(name)/.npm/_logs/2021-07-07T13_47_19_661Z-debug.log
(name)@(name)-mint:~$ node -v
v10.22.0
Attempting to create a PDF results in the following:
(name)@(name)-mint:~$ percollate pdf https://www.dailymaverick.co.za/article/2021-07-06-new-evidence-confirms-allegations-of-murder-and-torture-by-eswatini-military-and-police/
Fetching: https://www.dailymaverick.co.za/article/2021-07-06-new-evidence-confirms-allegations-of-murder-and-torture-by-eswatini-military-and-police/ ✓
Enhancing web page: https://www.dailymaverick.co.za/article/2021-07-06-new-evidence-confirms-allegations-of-murder-and-torture-by-eswatini-military-and-police/ ✓
(node:9062) UnhandledPromiseRejectionWarning: Error: Failed to launch the browser process!
/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/percollate/node_modules/puppeteer/.local-chromium/linux-818858/chrome-linux/chrome: 1: /home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/percollate/node_modules/puppeteer/.local-chromium/linux-818858/chrome-linux/chrome: Syntax error: "(" unexpected
TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md
at onClose (/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/percollate/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserRunner.js:193:20)
at Interface.helper_js_1.helper.addEventListener (/home/(name)/.nvm/versions/node/v10.22.0/lib/node_modules/percollate/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserRunner.js:183:68)
at Interface.emit (events.js:203:15)
at Interface.close (readline.js:397:8)
at Socket.onend (readline.js:173:10)
at Socket.emit (events.js:203:15)
at endReadableNT (_stream_readable.js:1145:12)
at process._tickCallback (internal/process/next_tick.js:63:19)

(node:9062) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:9062) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
I would appreciate your suggestions in regards to correctly installing the required dependencies?
Also, will the Snap be ready soon? Or perhaps an apt package? This is hell.

Hi @captainmalarky, from the log I see a couple of things:

npm install -g percollate produces some (false) requirements — jsdom should work well without canvas, so there's no need to npm install -g canvas (which seems to fail).
the actual problem seems to be that Puppeteeer can't launch the Chromium browser. Have you had a chance to take a look at this section from the Puppeteer troubleshooting page? It lists dependencies on various distributions.

I'm not sure of the status of @mcphail's percollate Snap (#103), I don't have the knowledge necessary to contribute there...

The snap package is usable, as it stands, but would need some upstream changes in snapd to be truly useful. I've watched some important members of the snapd team move to new jobs and I don't think it is likely these changes will come soon, if ever. For the time being I've had to park development of the snap, I'm afraid.

That's unfortunate but understandable.
On Wed, 07 Jul 2021, 19:53 Neil McPhail, @.***> wrote:

The snap package is usable, as it stands, but would need some upstream
changes in snapd to be truly useful. I've watched some important members of
the snapd team move to new jobs and I don't think it is likely these
changes will come soon, if ever. For the time being I've had to park
development of the snap, I'm afraid.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/danburzo/percollate/issues/128#issuecomment-875807393,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQWZPCEB6NAW5W2O6BGHRLDTWSIAFANCNFSM476XPX5Q
.

I'm going to close this issue, please let me know if this continues to be a problem.


0.3.2 breaks kv1 export (Darwin release)

Not sure if this happens on other architectures, but something about 0.3.2 broke export with --engine-type="kv1".
Version 0.3.1 works for me though.  Not sure if import is also affected as I'm only using medusa for export at the moment.

Hi @minorgod
Can you post the error message you are getting? I don't have a Mac, so it's hard for me to debug this issue.

Hi @minorgod Can you post the error message you are getting? I don't have a Mac, so it's hard for me to debug this issue.

Thanks for the speedy reply!
There's no "error" message per se...it just can't find any keys to read...
./medusa export mountname --format="json" --engine-type="kv1"
no keys found using path [/path] on Vault instance [http://vault.mydomain.test:8200]
Error: no keys found using path [/path] on Vault instance [http://vault.mydomain.test:8200]

Fairly sure it's not an issue with needing any other flags or env vars since the previous version of medusa works fine. Seems like something in that latest commit  has broken the path resoution. Maybe something adding or stripping off a slash somewhere it shouldn't? I'm not a golang programmer so it's hard for me to tell just by looking at the code.

How does your mountname look? Does it contain special characters like / or something like that?

I can confirm having the same issue with 0.3.2 and 0.3.3 on WLS running Ubuntu 20.04 LTS - we've reverted to 0.3.1.  The error message with %s = /path is the same regardless of whether medusa export secrets or medusa export secrets/ or medusa export my/secrets - looks like finalPath variable is not being built correctly.

I have made a new release v0.3.5. Thank you @landrew57 for solving this issue.


Handle indentation for multi-line blocks

Another attempt at multiline indentation, this one better.
The only case that's a little bit weird still is if I insert at the end of a line and had removed the indentation from the following line (see the last thing I did in the attached screencap)

LGTM


Added CountRotation.py

It will count the no. of rotations in rotated sorted array.

SOME OF THE CODES ARE ALREADY AVAILABLE TRY TO MAKE PR OF CODES WHICH ARE NOT AVAILABLE


Ft/zenko 160 add lvm storage

This PR brings the setup of an local LVM based storage for K8S PVs.
You need to setup either group_vars file or host_vars files containing the attributes found into roles/setup_lvm/defaults/main.yml and roles/kube_lvm_storage/defaults/main.yml
# ################ #
# LVM confguration #
# ################ #

# Specify which VG and which drive to use in host_vars for each node
metal_k8s_lvm:
  vgs:
    kubevg:
      drives: []

# Set the storage class setup on kubernetes node
metal_k8s_storage_class:
  storage_classes:
    local-lvm:
      is_default: true
  lvm_conf:
    default_fstype: 'ext4'
    default_fs_force: False
    default_fs_opts: '-m 0'
    default_mount_opts: 'defaults'
    vgs:
      kubevg:
        host_path: '/mnt/kubevg'
        storage_class: 'local-lvm'
        volumes:
          - name: lv01
            size: 10G
          - name: lv02
            size: 10G
          - name: lv03
            size: 10G
          - name: lv04
            size: 10G
          - name: lv05
            size: 10G

#
# You can also customize a specific volume like this
#
#    vgs:
#      kubevg:
#        volumes:
#          - name: lv01
#            size: 10G
#            fstype: xfs
#            fs_force: True
#            fs_opts: '-m 0 -cc'
#            mount_opts: 'defaults,noatime'

metal_k8s_storage_class:
  storage_classes:
    local-lvm:
      is_default: true
  lvm_conf:
    default_fstype: 'ext4'
    default_fs_force: False
    default_fs_opts: '-m 0'
    default_mount_opts: 'defaults'
    vgs:
      kubevg:
        host_path: '/mnt/kubevg'
        storage_class: 'local-lvm'
        volumes:
          - name: lv01
            size: 10G
          - name: lv02
            size: 10G
          - name: lv03
            size: 10G
          - name: lv04
            size: 10G
          - name: lv05
            size: 10G

I don't think this is the right approach.
In general, one will want to create one LVM VG per 'storage type', e.g. over a bunch of SSD-backed LVM PVs ('vg-ssd'), or some spindles ('vg-hdd')
Now, this means we should have 2 Kubernetes StorageClasses, and LVM LVs with Kubernetes PVs for said StorageClasses.
E.g.
metal_k8s_lvm:
  vgs:
    vg-ssd:
      drives: ['/dev/sdb', '/dev/sdc']
    vg-hdd:
      drives: ['/dev/sdd', '/dev/sde']

metal_k8s_storageClasses:
    - name: local-fast
      default: True
      lvmVG: vg-ssd
      hostPath: /mnt/local-fast
      fs:
        type: ext4
        options: '-m 0 --optimize-for-ssd' #Haha
        mountOptions: 'defaults'

    - name: local-slow
      lvmVG: vg-hdd

metal_k8s_volumes:
    - name: lv01
      storageClassName: local-fast
      size: 10Gi
    - name: lv02
      storageClassName: local-slow
      size: 128Ti

Exact naming of fields etc. bikeshed'able.

I agree with @NicolasT, this approach makes more sense in a TS perspective.
Moreover I would like to know what is the current state of art in Eng for specifying volume space. Is it:

GB
G
Gi
GiB
All of them ?

Okay.
@xaf-scalitywhatever is supported by the "lvol" module : http://docs.ansible.com/ansible/latest/lvol_module.html
So in this case "G|g" is the only valid choice
@NicolasT Okay, although this model allow precisely what you describe (this was the intention)
Now I Agree I could have some defaults for the filesystem and mount options and a way to override that per VG and by LV compared to only LV right now, so they would make sense for a type of storage.
However I disagree on the data modeling of the storage classes. I found this a recurrent "issue" in the ansible world and discussed it with @Zempashi , if you use a list, you have to manually check for the name collision as opposite of a dict, where the key is by design unique.
So I would rather do something like this:
metal_k8s_lvm:
  vgs:
    vg-ssd:
      drives: ['/dev/sdb', '/dev/sdc']
    vg-hdd:
      drives: ['/dev/sdd', '/dev/sde']

metal_k8s_storageClasses:
  local-fast:
    default: True
    lvmVG: vg-ssd
    hostPath: /mnt/local-fast
     fs:
       type: ext4
       options: '-m 0 --optimize-for-ssd' #Haha (<- I see what you did there !)
       mountOptions: 'defaults,nattime'
  local-slow
    lvmVG: vg-hdd

metal_k8s_volumes:
  default_fs:
    type: ext4
    options: '-m 0 --barrier=0' #IWantToLoseMyData
    mountOptions: 'defaults'
  volumes:
    lv01:
      storageClassName: local-fast
      size: 10G
    lv02:
      storageClassName: local-slow
      size: 128T
      fs:
        type: xfs
        options: '-m 0'

What do you think ?

It's ok for me, what I will challenge is the #Haha and the #IWantToLoseMyData stuff ;-)

Tested, works for me and Zenko.
Note: according to https://github.com/kubernetes-incubator/external-storage/tree/master/local-volume

Note that with manual PV creation, the volume has to be manually reclaimed when deleted
Confirm, I experienced this as well.


Retrieve bounding box of text on a page

I would like to determine the margins of the text in a PDF document. One possibility would be to render the PDF and look at the text layer of each page, specifically the positionins of their div children (which represent rows of text).
That strikes me as a little too cumbersome, though. Is there a way to retrieve the bounding box of all text on a page from the PDFJS object?

You would want to use getTextContent() https://github.com/mozilla/pdf.js/blob/master/src/display/api.js#L812 instead -- it is used by TextLayerBuilder. TextItem has transform and width/height https://github.com/mozilla/pdf.js/blob/master/src/display/api.js#L567. Does this help?

That strikes me as a little too cumbersome, though. Is there a way to retrieve the bounding box of all text on a page from the PDFJS object?

That's not a use case that needed for PDF Viewer (yet). You have to implemented it on your side.

@yurydelendik Thanks, that'll do the trick.
Is there documentation on transform? I suppose those are entries in a transformation matrix, but I'm not sure which.

I just found CSS3's transform and assume that the same logic is applied.
http://www.w3schools.com/cssref/css3_pr_transform.asp
https://dev.opera.com/articles/understanding-the-css-transforms-matrix/

Is the unit of the translations is px or some PDF-related unit (e.g., in)?

PDF user unit, you have to use PageViewport to map it to the screen presentation, see e.g. https://github.com/mozilla/pdf.js/blob/master/web/text_layer_builder.js#L159

Hm, I can't seem to get it right. The transform property on the individual element reads
element transform: [9.9626, 0, 0, 9.9626, 74.558, 673.034]

where I don't know how to to interpret it. I'm certainly not supposed to stretch the element by a factor of almost 10, right? Curiously, the 9.9626 always coincide with the height property. What to make of this?

Sorry we did not extend the documentation of those getTextContent that far (hoping we can improve the API). Currently we operate under assumption that user of the advanced features will have some knowledge in computer graphics. Let me help, based on https://github.com/mozilla/pdf.js/blob/master/web/text_layer_builder.js#L159 code, and let's start from simple example:
var text; // pdfPage.getTextContent(function (t) { text = t; });
var viewport = pdfPage.getViewport(1.0 /* scale */);
// find all text start points
var xs = [], ys = [];
text.items.forEach(function (item) {
  var tx = PDFJS.Util.transform(viewport.transform, item.transform);
  // avoiding tx * [0,0,1] taking x, y dirrectly from the transform
  xs.push(tx[4]); ys.push(tx[5]);
});
var boundsOfStartPoints = [Math.min.apply(null, xs), Math.min.apply(null, ys), Math.max.apply(null, xs), Math.max.apply(null, ys)];

But your task you need to take in account all four points of the rectangle, and tx calculation must be performed a little bit different. It's unfortunate width and height are scaled by fontHeight, but you can easily revert that and calculate position of the [0,0], [width/fontHeight,0], [0,height/fontHeight], [width/fontHeight, height/fontHeight] for bounds calculation.

It's unfortunate width and height are scaled by fontHeight,

Aha, I didn't know that. How do I retrieve the font height of an object? Does that coincide with content.items[i].height?

From https://github.com/mozilla/pdf.js/blob/master/web/text_layer_builder.js#L164: var fontHeight = Math.sqrt((tx[2] * tx[2]) + (tx[3] * tx[3]));

When inspecting tx[4] of an element on on the PDF, I notice that the actual x-y-pixel-coordinates of the top-left point do not coincide with tx[4] as computed from
tx = PDFJS.Util.transform(
  viewport.transform,
  content.items[i].transform
);

See:

tx[4] = 76.6...

while the actual translation in x-direction exceeds 100px.
Any idea what might be causing this?

I created a little plunk at http://plnkr.co/edit/m9oPJg80XHeeQ1nquxkz which confirms exactly what you're saying, @yurydelendik. However, in my full application, I still get the wrong offsets. I'm investigating why, and it might be due to the pdf_viewer I'm using. I'd like to create a plunk with it, too, but I can't find pdf_viewer.js served from github.io like http://mozilla.github.io/pdf.js/build/pdf.js or http://mozilla.github.io/pdf.js/build/pdf.worker.js. Any pointer?

Developers and leaders of Mozilla, which are shemales ass!

They should then put it long as needed by the throat until he swallows themselves in suicide!
You Mozilla is...

 And to do a favour... I'm going to do that!
I kicked her so hard failure golden eggs until they are ringing and sing you all the melodies of forward and back pain in an instant!

Was it expected?
Oh, Yes, that was!
A movement such as the Mozilla!
So happy that I never upgraded to Firefox 29 and beyond!
Let me tell you something Mozilla developers! If I would be still using your latest crap-ware that really my mission would at this time to find out where you live and then drill him often in Golden's failure until you analyzes and understands how to threaten power users in the correct way!
Without a doubt, has been said before - but may not be repeated enough... You are...
 How far did you come?

Here is how I approached this.  First, I ignore the scale factor in the textContent transform.  (The transformation matrix provided in the textContent.item[x].transform doesn't make any sense to me because it sets a scale in both the x and y directions equal to font height.  I don't know why you'd ever want to do canvas operations in those units.  But that's beside the point.)
The numbers that matter are:
const item = textContent.items[0];
const transform = item.transform;
const x = transform[4];
const y = transform[5];
const width = item.width;
const height = item.height;

In order to do any operations on the canvas using these values, you have to (1) fix the y coordinate from the PDF origin to the canvas origin and (2) scale the whole thing by whatever you've scaled the viewport (i.e. by whatever you passed getViewport).  So I do this:
convertToCanvasCoords([x, y, width, height]) {
  const { scale } = this;
  return [x * scale, this.canvas.height - ((y + height) * scale), width * scale, height * scale];
}

Where this.scale is the same number I passed to getViewport.  Then the following draws an accurate box around the text:
ctx.strokeRect(...this.convertToCanvasCoords([x, y, width, height]));

Note, I had to adjust y again by the height of the box because strokeRect wants the top left corner and even after adjusting for the PDF origin issue, what you end up with is the bottom left corner of the box.  So you add the height to get the top, then scale, then fix for origin.  There's probably a cleaner way of doing this, but this works, and it has the advantage that I kind of understand what's going on.  :)  Hope that helps.

For others digging around for what pdf.js is actually doing with transformation vectors, the PDF Reference includes a definition of how transformation vectors are laid out and how they relate to mapping into a two dimensional coordinate space.
Specifically, the components of a transformation matrix are described on page 142:

Translations are specified as [ 1 0 0 1 tx ty ], where tx and ty are the distances to translate the origin of the coordinate system in the horizontal and vertical dimensions, respectively.
Scaling is obtained by [sx 0 0 sy 0 0]. This scales the coordinates so that 1 unit in the horizontal and vertical dimensions of the new coordinate system is the same size as sx and sy units, respectively, in the previous coordinate system.
Rotations are produced by [cos θ sin θ −sin θ cos θ 0 0], which has the effect of rotating the coordinate system axes by an angle θ counterclockwise.
Skew is specified by [1 tan α tan β 1 0 0], which skews the x axis by an angle α and the y axis by an angle β.

(there's an accompanying chart in the reference as well)
And the vector itself is defined thus:

PDF represents coordinates in a two-dimensional space. The point (x, y) in such a space can be expressed in vector form as [x y 1]. The constant third element of this vector (1) is needed so that the vector can be used with 3-by-3 matrices in the calculations described below.
The transformation between two coordinate systems is represented by a 3-by-3 transformation matrix written as
[ed: pretend this is a matrix]

a
b
0

c
d
0

e
f
1

Because a transformation matrix has only six elements that can be changed, it is usually specified in PDF as the six-element array [a b c d e f].

Closing since this is answered now.

I have a use case where I would like to get the bounding box of each word within an item. For example, the str of an item can be "hello world!", but the transformation only gives the coordinates of the entire string. In my use case, I would like to get the coordinates of each of the "hello", "world", and "!". These words are not selected or highlighted.

@haijun-ucsd have you found a way to achieve this?

@haijun-ucsd have you found a way to achieve this?
@BernhardBehrendt You could do something like this:

Get the width of the item, for example x1-x0
Divide the width by the total character in that item including space and symbol (you get average width for each character)
Multiple the average width by the index in the item and you will get the x


Multiple Levels: No need for the tip for step 14

Describe the bug
Multiple Levels: No need for the tip for step 14
Steps to reproduce the behavior
1.Navigate to https://arcade.makecode.com/beta--skillmap
2.Multiple Levels------->START
Actual behavior
No need for the tip for step 14. Because it already adds an array of sprites of kind [Player] in the toolbox.

Additional context
1.OS: Windows(rs6)
2.arcade version: 1.4.19
3.Microsoft MakeCode version: 6.11.2

Really good find.  I feel like this might exist somewhere else, as well...but now that we have the block in Sprites, the tip is outdated.  This instance is fixed.
https://github.com/microsoft/pxt-skillmap-sample/pull/50


Add production flag for Survicate

Use isProduction from src/constants to only load the survey in production

Coverage increased (+0.3%) to 25.964% when pulling 35ec1e40d4b4ac28989cf2eeaf664cf834c99ff3 on Jskobos:survicate-flag into c439ec1d51a4dfda194a9b0fa1a9f0dca32ee1b5 on linode:develop.


Possible improvements for some non-standard address formats

Hello,
I'm processing an email dump of many thousand emails, many of which seem to contain addresses that appear to be in a non-standard format. I'm curious if you have seen such formats before, and if you have any suggestions for handling them. If you think addressparser should support such formats, I could try to work on a PR.
Here is an example of the type of To header I'm talking about:
To: 'John Doe'<EMAIL_ADDRESS>Bob Jones (bjones@domain.com); Shmoe, Joe; Kelly Johnson

The current output from this module is:
[
  {
    "address": "",
    "name": "'John Doe'"
  },
  {
    "address"<EMAIL_ADDRESS>    "name": ""
  },
  {
    "address": "",
    "name": "Bob Jones"
  },
  {
    "address": "",
    "name": "Shmoe"
  },
  {
    "address": "",
    "name": "Joe"
  },
  {
    "address": "",
    "name": "Kelly Johnson"
  }
]

Ideally, the output would be as follows:
[
  {
    "address": "",
    "name": "John Doe"
  },
  {
    "address"<EMAIL_ADDRESS>    "name": ""
  },
  {
    "address"<EMAIL_ADDRESS>    "name": "Bob Jones"
  },
  {
    "address": "",
    "name": "Shmoe, Joe"
  },
  {
    "address": "",
    "name": "Kelly Johnson"
  }
]

I'm not sure how common such address headers are out in the wild... I believe these emails were exported using software called OUTLOOK2MAC, but I'm not 100% sure. Apart from the address headers, the rest of the email format seems to be fine.

I understand the problem but this input is way too vague, this module is too simplistic to do some kind of heuristics and stuff to figure out the meaning of values.
This isn't supported by "normal" e-mail clients as well. For example this is what the OSX Mail app thinks of such To: value:

\
Name
Address

1
<EMAIL_ADDRESS>John Doe'<EMAIL_ADDRESS>Bob Jones ; Shmoe

2
Joe; Kelly Johnson
Joe; Kelly Johnson


Implement bandwidth throttling

If input (e.g. log file) volume goes crazy (e.g. due to a bug) Logagent will of course try to read the input and ship the data as fast as it can.  That could have a negative impact on network, CPU, etc.
Maybe now, when working on #28 it makes sense to think about throttling because it probably makes the most sense for throttling to happens as close to input as possible (e.g., don't even read data from input is specified max input bytes per second limit has been reached).

To limit the rate from files via tail-forever we would need to get this issue solved: https://github.com/mingqi/tailF/issues/8
--maxInputRate 10mb can be used by any plugin, currently only implemented for input from stdin.

This module seems to be interesting, as it exposes streams for tailed files: https://www.npmjs.com/package/safe-log-reader - to be tested ...

https://github.com/Juul/tail-stream


MTL-2112 csm repo is not available at this point

Description

The CSM repos are not added to the NCN at this point in time, short of adding the repos the user can just follow the check-for-latest documentation guide again.
root@ncn-m002 2023-04-01 01:53:37 ~ # zypper --plus-repo="/etc/cray/upgrade/csm/csm-${CSM_RELEASE}/tarball/${CSM_RELEASE}/rpm/cray/csm/sle-$(awk -F= '/VERSION=/{gsub(/["-]/, "") ; print tolower($NF)}' /etc/os-release)" --no-gpg-checks install -y cray-site-init
Specified local path does not exist or is not accessible.
root@ncn-m002 2023-04-01 01:53:47 ~ # scp ncn-m001:/root/*.noarch.rpm /root/
docs-csm-latest.noarch.rpm                                                                                100%   30MB 282.8MB/s   00:00
libcsm-latest.noarch.rpm                                                                                100%   33KB  62.7MB/s   00:00
root@ncn-m002 2023-04-01 01:53:48 ~ # rpm -Uvh --force /root/docs-csm-latest.noarch.rpm
Preparing...                          ################################# [100%]
Updating / installing...
   1:docs-csm-1.4.93-1                ################################# [100%]
root@ncn-m002 2023-04-01 01:53:54 ~ # zypper -n in -y --repo csm-sle-$(awk -F= '/VERSION=/{gsub(/["-]/, "") ; print tolower($NF)}' /etc/os-release) libcsm
Repository 'csm-sle-15sp4' not found by its alias, number, or URI.
Use 'zypper repos' to get the list of defined repositories.
root@ncn-m002 2023-04-01 01:53:57 ~ # zypper repos
Warning: No repositories defined.

Checklist

[ ] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[ ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[ ] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport release/1.4

/backport release/1.3


Missing functionalty - upload/download file from/to buffer

Hi,
I see that upload/download is working with the file system - is there a reason for not having methods to upload/download from/buffer ?
Assuming I have small files, then it's better performance.
10x,
Shay

It may be simply that nobody has submitted a PR adding this functionality yet :dancer:


Could not resolve Kotlin Compiler Embeddable after upgrade to 1.21.0

Expected Behavior

After upgrade Detekt should work
Observed Behavior

Error is thrown:
> Could not resolve all files for configuration ':detekt'.
   > Could not resolve org.jetbrains.kotlin:kotlin-compiler-embeddable:1.6.21.
     Required by:
         project : > org.springframework.boot:spring-boot-dependencies:2.7.1
         project : > io.gitlab.arturbosch.detekt:detekt-cli:1.21.0 > io.gitlab.arturbosch.detekt:detekt-parser:1.21.0
         project : > io.gitlab.arturbosch.detekt:detekt-cli:1.21.0 > io.gitlab.arturbosch.detekt:detekt-tooling:1.21.0 > io.gitlab.arturbosch.detekt:detekt-api:1.21.0
         project : > io.gitlab.arturbosch.detekt:detekt-cli:1.21.0 > io.gitlab.arturbosch.detekt:detekt-parser:1.21.0 > io.gitlab.arturbosch.detekt:detekt-psi-utils:1.21.0
      > Cannot find a version of 'org.jetbrains.kotlin:kotlin-compiler-embeddable' that satisfies the version constraints:
           Dependency path ':myproject:unspecified' --> 'io.gitlab.arturbosch.detekt:detekt-cli:1.21.0' (runtimeElements) --> 'org.jetbrains.kotlin:kotlin-compiler-embeddable:{strictly 1.6.21}'
           Constraint path ':myproject:unspecified' --> 'org.jetbrains.kotlin:kotlin-bom:1.7.10' (enforced-platform-runtime) --> 'org.jetbrains.kotlin:kotlin-compiler-embeddable:1.7.10'
           Constraint path ':myproject:unspecified' --> 'org.springframework.boot:spring-boot-dependencies:2.7.1' (platform-runtime) --> 'org.jetbrains.kotlin:kotlin-compiler-embeddable:1.6.21'
           Dependency path ':myproject:unspecified' --> 'io.gitlab.arturbosch.detekt:detekt-cli:1.21.0' (runtimeElements) --> 'io.gitlab.arturbosch.detekt:detekt-parser:1.21.0' (runtimeElements) --> 'org.jetbrains.kotlin:kotlin-compiler-embeddable:1.6.21'
           Dependency path ':myproject:unspecified' --> 'io.gitlab.arturbosch.detekt:detekt-cli:1.21.0' (runtimeElements) --> 'io.gitlab.arturbosch.detekt:detekt-tooling:1.21.0' (runtimeElements) --> 'io.gitlab.arturbosch.detekt:detekt-api:1.21.0' (runtimeElements) --> 'org.jetbrains.kotlin:kotlin-compiler-embeddable:1.6.21'
           Dependency path ':myproject:unspecified' --> 'io.gitlab.arturbosch.detekt:detekt-cli:1.21.0' (runtimeElements) --> 'io.gitlab.arturbosch.detekt:detekt-parser:1.21.0' (runtimeElements) --> 'io.gitlab.arturbosch.detekt:detekt-psi-utils:1.21.0' (runtimeElements) --> 'org.jetbrains.kotlin:kotlin-compiler-embeddable:1.6.21'

Steps to Reproduce

Upgrade detekt from 1.20.0 to 1.21.0
Run detekt

Context

Dependabot opened a PR in a project we have to upgrade detekt version from 1.20.0 to 1.21.0, we are using kotlin 1.7.10
Your Environment

Version of detekt used: 1.21.0
Version of Gradle used (if applicable): 7.5
Gradle scan link (add --scan option when running the gradle task): can't provide one cause is a private project
Operating System and version: Mac OS X 12.4
Link to your project (if it's a public repository): can't provide one cause is a private project

I can't confirm without your build files but this is almost certainly happening because kotlin-bom enforced platform is being applied to all configurations in the build, including the "detekt" configuration.
detekt relies on a specific version of kotlin-compiler-embeddable, and that specific  dependency version is now strictly enforced in the "detekt" Gradle configuration.
Gradle is reporting the conflict between the version detekt needs and the enforced platform override applied in your project.
You'll need to stop the kotlin-bom dependency from being applied to the "detekt" configuration.

yep, I could run the detect task removing the constraint that enforced kotlin version. Thanks for pointing it out.


Modify ModelCheckpoint class to support additional options for cloud storages

What does this PR do?

Fixes #5836
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

@ilia-zaitcev-by Hello, we are also facing the same issue of storage options not being passed for remote storages like azure. The solution proposed in this PR seems suitable. Can we look at revisiting this PR ?

@ananthsub We have a similar case to support additional options for remote stores specifically for Azure Blob File System(abfs).
One way is to add storage_options to every path needed for Trainer class. For abfs, these options cannot be added as part of the url. Fsspec does not support that currently. Having a separate storage_options dict works.
Since Path is used for various file system paths in the code, was thinking if it is possible to add a storage_options attribute to Path class or a subclass.
@ananthsub @carmocca Let me know your thoughts on this

@apoorvaagg @carmocca Sorry for the late reply! Yeah, having this kind of functionality would be great. We did some custom callback for that, which is rather simple, as it doesn't cover all possible functionality available in PL, like distributed training. It works for us, but not ready for upstreaming, I believe. Feel free to open a new PR to do this properly.


Bug: Perp Position Value is not correct.

Dear Team,
I have updated the latest mango-explorer version, however, my perp positions are not correct.eg: my BTC-PERP is 0.087,but the value of mango is 87,800. SOL-PERP has the same issue.
pls help check it.
The log is below after running "./bin/show-accounts"
« AccountSlot [1] BTC
            Net Value:     « InstrumentValue:         0.00000000 BTC »
                Deposited: « InstrumentValue:         0.00000000 BTC » (raw value: 0E-20)
                Borrowed:  « InstrumentValue:         0.00000000 BTC » (raw value 0E-20)
            Spot OpenOrders: None
            Perp Account:
                « PerpAccount
                    Base Position: « InstrumentValue:    87,800.00000000 BTC »
                    Quote Position: -3783574498.53515607565438472193
                    Long Settled Funding:<PHONE_NUMBER>6902058174692343
                    Short Settled Funding:<PHONE_NUMBER>4996363400827590
                    Bids Quantity: 0
                    Asks Quantity: 0
                    Taker Base: 0
                    Taker Quote: 0
                    MNGO Accrued: « InstrumentValue:         0.45457900 MNGO »
                    OpenOrders:
                        « PerpOpenOrders
                            Orders:
                                None
                        »
                »

The value stored (and shown) in the PerpAccount is - confusingly - in 'lot sizes' rather than in the quote currency.
The value shown in the PerpPositionSize column in the DataFrame should be correctly adjusted to the quote currency - check out the example of loading the Account data into a DataFrame.

Thanks.I have used the account frame.It's great!


Improve isRoute logic

This PR improves the logic for distinguishing a route from a component. Queries can now be inside of _NonRouteComponent.svelte and it will be treated as a component query instead of a route.
fixes #281

Hmm, maybe I have a (or more) misconception(s) about the static config value.
In my new project I need to use the static adapter (it is the frontend of a tauri app) and naturally I set static: true for SPA mode. This of course means no load is generated and the query is executed after mounting. Now, in my route routes/contracts/[id=integer].svelte I need to access the id param and I tried it like the "normal" way inside the ...Variables function, which, according to the docs, gets the same arguments as without static: true.
But I get this (outputted to the browser console):

Using the page store also does not work (Cannot reference store value inside


Limit cache sizes based on memory usage

Added option for limiting cache sizes based on total memory used by the objects in each cache.
Prune function is modified to accommodate for this new option.

@tst-jkent some thoughts about this - it would be great to have the option to limit the cache by number of objects OR object size. Also, khCache is used in multiple places - I would think we'd only want to use the new code for the Asset and AssetVersion classes and leave the other khCache uses the same as they are now. I'm not sure how important that is.

small changes, mostly cosmetic to use more c++11 constructs, won't effect logic

Our conversation reminded me - we need to update the docs and the release notes.

The only big thing I can think of that might be missing here is code to calculate the sizes of classes that are derived from AssetImpl or AssetVersionImpl. For example, one inheritance chain goes AssetVersionStorage -> AssetVersionImpl -> AssetVersionImplD -> CompositeAssetVersionImplD -> RasterProjectAssetVersionImplD. We're only calculating size at the AssetVersionImpl level and below, but the other classes can add fields and we aren't accounting for those right now.

minor comments, mostly performance related


Move links

Remove the lists of links at the end of testable-architecture and bdd-tutorial.
Add relevant blogs to blog-posts
Link from "Who does what" to internal "Example mapping" page, instead of directly to the blog. (Link to blog is also provided at the end of that page)

Thanks for the quick review @olleolleolle !


Fix EventsRegistry.registerNotificationOpened typing

EventsRegistry.registerNotificationOpened registers a callback that expects a response: NotificationResponse object as the first parameter. In the registered callback, we can expect response.notificaiton to be the notification opened.
However, NativeEventsReceiver.registerNotificationOpened wants a callback that expects a notification: Notification object instead. So we are passing back notification as the response which will cause the callback code to get undefined as the notification.
This fixes the mismatch to supply both the notification and the action to the registered callback.

Resolves #469.

Any movement on this one to get it merged since it's already been approved?

The issue is pretty much already addressed in https://github.com/wix/react-native-notifications/pull/566. I just rebased my change to remove that unused completion param.

@yogevbd I cleaned it up and all checks have passed, can this be merged now?


OHDSI Demonstrator

As product owner I would like a demo of OHDSI integrated putting into dev/prod environments

@vvcb per my email I’ve release OHDSI stack without AuthN/Z to prod at the following addresses
ATLAS:
https://ohdsi.xlscsde.nhs.uk/atlas/#/home
HADES:
https://ohdsi.xlscsde.nhs.uk/
Username for hades is ohdsi and I’ll send the password on a separate email shortly


fix(subscribe): don't swallow internal errors

Description:
This PR reports errors that would otherwise be swallowed - because of a closed subscriber - to the console. (This behaviour is something that bit me again, today, when I was investigating a snippet that effected a stack overflow error in v5, but didn't effect the error in v6. The error was still occurring, of course, it just wasn't being thrown.)
As discussed in the linked issue, calling hostReportError would be preferred - rather than writing to the console - but doing so would be a breaking change.
Related issue (if exists): #3803

kaboom - 👍 LGTM

@benlesh I've made a bunch of changes inline with your requests:

Removed reportError and used canReportError for readability.
Renamed to consoleWarn.
The canReportError error tests now just test the logic - not the reporting.
I've fixed a logic error in the canReportError implementation.
canReportError is no longer recursive.
canReportError error is also used in bindCallback and bindNodeCallback - see #3487. (Your mentioning that it's only used in one place reminded me of errors that were swallowed elsewhere.)
There are integration tests for Observable.subscribe, bindCallback and bindNodeCallback that stub console.warn.

Looking good, just a couple of nits.

@benlesh I've made changes inline with your requests:

isTrustedSubscriber has been used.
ErrorObserver<any> has been changed to Subscriber<any> | Subject<any>.

However, I've not changed consoleWarn to test console.warn only once for these reasons:

Doing so means that stubbing console.warn won't stub consoleWarn.
There are issues stubbing exported functions.
The actual logging to the console will be so much slower that it will make next to no difference, IMO. And it's only going to be in an error condition.

console.warn is now called directly. It's called directly elsewhere in the codebase, anyway.

which version of RxJs will this be included?


Update copyright and license for open source cFS framework (dev branch)

Update/add copyright and license to Apache 2.0

Imported from trac issue 288.  Created by jhageman on 2019-05-03T14:23:42, last modified: 2019-08-29T13:02:29

Trac comment by jhageman on 2019-05-03 14:25:14:
Likely start from #316, and do an additional scan for any other required changes.

Trac comment by jhageman on 2019-07-03 13:16:47:
Moved all open tickets targeting 6.6.1 to 6.7.0 end of summer release

Trac comment by jhageman on 2019-08-21 14:06:48:
CCB 8/21/2019 - Changeset approved and ready for test

Trac comment by jhageman on 2019-07-23 16:32:59:
See [changeset:bd1ede0] in trac-288-copyright-license

Trac comment by jphickey on 2019-08-22 16:06:59:
Merged into branch ic-ccb-20190821

Trac comment by jphickey on 2019-08-29 10:53:33:
Merged to development branch

Trac comment by jphickey on 2019-08-29 10:48:13:
Confirmed that ic-ccb-20190821 merge is acceptable for development merge

Trac comment by jhageman on 2019-08-29 13:02:29:
CCB 8/28/2910 - Reviewed and approved for merge to developement


Move entity between views

The library put windows and blinds in the same view or domain (named cover). I need to separate them, blinds is one "group" and windows is another "group"; so the views need to be separated, but also the section inside areas. I fount the way to create a new custom view (that I named blinds), but I can't remove the blind entities from cover and put into blinds, or move the entities between domains.
Is there a way to do this?

The best way to achieve what you want is to hide the cover view and create a custom view for blinds, windows and covers. It is a bit of work but can be done if you want. I may consider adding separate views for them but currently, it's not on my radar.


junit5 support?

Feature-Request
Hello, is there a way to reallize parameterized tests with JUnitParams at junit 5?

Hello,
An issue already exists for parameterized tests support in JUnit5 project
https://github.com/junit-team/junit5/issues/14
It is apparently waiting for @TestTemplate to be merged : https://github.com/junit-team/junit5/tree/issues/14-test-template

After last milestone JUnit5 supports parametrized tests :
http://junit.org/junit5/docs/current/user-guide/#writing-tests-parameterized-tests

JUnit 5 parametrized tests are terrible, you cannot have parameter pairs (e. g. to specify both input and asserted value) without boilerplate code in the form of provider method.
It's not a good replacement for JUnitParams, so we still need a JUnit 5-compatible version of them.

Maybe Dynamic Tests are a good fit for the use case you describe.
This syntax allows you to write straighforward test cases :
@TestFactory
Collection<DynamicTest> myTestMethodFactory() {
    return Arrays.asList(
        dynamicTest("1st dynamic test", () -> myTestMethod("param1", "expected1")),
        dynamicTest("2nd dynamic test", () -> myTestMethod("param2", "expected2"))
    );
}

private void myTestMethod(String param, String expected) {
	...
}

A more JUnitParams-like syntax can also be used, but it feels weird:
@TestFactory
Stream<DynamicTest> myTestMethodFactory() {
	return Arrays.stream(new Object[][] {
			new Object[] {"title1", "param1", "expected1"},
			new Object[] {"title2", "param2", "expected2"}
	}).map(testCase ->
		dynamicTest(
				testCase[0].toString(),
				() -> myTestMethod(String.class.cast(testCase[1]), String.class.cast(testCase[2]))
		)
	);
}

Ultimately a JUnit5 version of JUnitParams will solve the awkward boiler needed today.
I wrote a POC, but JUnit5 API is still moving.


Should psgeostat be removed?

Not currently used, but...

but some statistics are already present as nCaves, nCities, ...
Could these, perhaps, find a new home?

Yes, possibly :-)

Let keep this. Closing issue as Wontfix.