class: and class- shorthands are not stripping class prefix

Writting the following:
view! { cx,

}

results in

Same for the class- variant.

Thanks, this is a bug I introduced earlier this week and just caught today and it should be fixed on then,sin branch and in the latest release. Do you know what version you're using?

0.0.12

I hadn't checked there was a new release before creating the issue, will do next time! I keep forgetting the crate is moving fast!

Yeah no problem. I literally released the new version (okay, several new versions because I screwed a couple things up...) in the last thirty minutes so we just overlapped... This is why I'm staying on 0.0.x for now :-)
Thanks for catching this and letting me know. I'll close the issue, but feel free to reopen if the problem is fixed.


Plugin [id: 'com.moowork.node', version: '1.3.1'] was not found in any of the following sources:

Where:
Build file 'C:\projects\git\services\build.gradle' line: 16

What went wrong:
Plugin [id: 'com.moowork.node', version: '1.3.1'] was not found in any of the following sources:

Gradle Core Plugins (plugin is not in 'org.gradle' namespace)
Plugin Repositories (could not resolve plugin artifact 'com.moowork.node:com.moowork.node.gradle.plugin:1.3.1')
Searched in the following repositories:
Gradle Central Plugin Repository

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org.

Can you please help me to resolve this problem.

facing same problem

That is not helpful.

Had the same problem when I accidentally enabled Gradle "offline mode" in IntelliJ Idea

close it.

We are acutally trying to build while offline. Is there somewhere we can "install" the plugin or point to local repository so that build can work or will it always try to download from internet?

add mavenLocal() to your buildscript.repositories or pluginManagement.repositories (in settings.gradle) and build the plugin locally using publishToMavenLocal though you're going to need to switch to the fork to be able to get node, see: https://github.com/node-gradle/gradle-node-plugin/issues/66 for how to use distBaseUrl and how to add an ivyrepo
Are you sure you want to build while offline rather than serving all requests through your own nexus/artifactory/etc? Because one way or another you're going to need your dependencies, and installing everything is a bother in large environments


Informative implementation documentation

This issue is a temporary place to document existing, ongoing, or planned implementations of Solid Notifications Protocol Editor's Draft (see also proposed Version 0.2.0) and specific notification channel types.
Test suite and implementation reports will supersede this information when available, and will have a breakdown for applicable requirements.
The following table is informative only, in draft status, and intended to reflect intents. Information may be outdated, e.g., there is a planned shift to support notification channel types (as opposed to subscription types). Refer to project source as authoritative.

Product
ResourceServer
SubscriptionClient
SubscriptionServer
NotificationSender
NotificationReceiver

CSS
✔
-
WebHookSubscription2021
WebHookSubscription2021
?

dokieli
-
WebSocketSubscription2021 WebSocketChannel2023, LDNChannel2023
-
-
WebSocketSubscription2021 WebSocketChannel2023

solid-webhook-client
-
WebSocketSubscription2021, WebSocketChannel2023
-
-
WebHookSubscription2021, WebHookChannel2023

Regarding CSS: WebSocketSubscription2021 and WebHookSubscription2021 were implemented in the 6.0 branch so would be part of that upcoming release. These are based on the specs as they were in September/October 2021, so the 1.0 version (with some aspects of the editor's draft at that point). I haven't been following the Notification spec since then so I will need to have a look what changes might be required to support the 2.0 version.

I strongly recommend that *Subscription2021/2022 (expressing notification-subscription-type) is not released! They are considered obsolete at this point, and will be marked as such and refer to new versions which express "notification channel types".
Solid Notifications Protocol Version 0.2.0 is not compatible with 0.1.0.
There will be no tests for 0.1.0. I suggest implementations adapting 0.2.0/notification-notification-type (or later version if/when available).
(dokieli originally implemented 0.1.0/notification-subscription-type but recently rewrote to conform to 0.2.0/notification-channel-type. No point in keeping 0.1.0 around.

I've made a stand-alone implementation for subscribing to a WebSocketChannel2023 following v0.2.0 of the Solid Notification Protocol and tested it against the CSS add-notification branch:

Github repository
npm package

Off the top of my head, I think Solid-Nextcloud implements WebSocketChannel2023 and WebHookSubscription2021 and there are some basic jest-based tests for both in https://github.com/solid-contrib/solid-crud-tests/blob/main/test/surface/notifications.test.ts

CSS v6.0.0 has now been released which includes support for v0.2 of the spec. The built-in channels being WebSocketChannel2023 and WebHookChannel2023.

We had a call earlier this month with @woutslabbinck and @jaxoncreed and agreed to collaborate on TS common modular implementation in https://github.com/o-development/solid-notification-client
This will incorporate prior work done in:

https://github.com/o-development/solid-webhook-client
https://github.com/woutslabbinck/Solid-Notification

There is an open PR with DiscoveryClient and SubscriptionClient based on a branch that I'm already using in sai-js. Everyone interested in collaboration please feel invited to join the effort!


Mandelbrot script only works without overflow checks enabled

Due to an overflow check and recent optimisations the Mandelbrot script currently only runs when built in release mode.
The offending code is in bf_vm.rs:154
    AddCurrentUp(x) => {
           self.mem[self.data_ptr + x] += self.mem[self.data_ptr];
           self.mem[self.data_ptr] = 0;
    }

In a debug build with overflow checks enabled it will panic.
Changing this to a wrapping add completely breaks the mandelbrot script, investigation into why is needed.

Ok, wow, issue was I wasn't storing the result of the wrapping add back into the cell!
Fix:
AddCurrentUp(x) => {
        self.mem[self.data_ptr + x] = self.mem[self.data_ptr + x].wrapping_add(self.mem[self.data_ptr]);
        self.mem[self.data_ptr] = 0;
},

Still wouldn't reccomend running Mandelbrot script in debug mode, takes a while 😄


Deprecating password authentication

Hi all, do you have a fix for this issue:
https://developer.github.com/changes/2020-02-14-deprecating-password-auth/
I generated a personal token but I am not sure what is the best way to modify the setup.sh file.
Should I just replaced
curl -u '{{ cookiecutter.github_username }}' https://api.github.com/user/repos -d '{"name":"{{ cookiecutter.project_name.lower().replace(' ', '_') }}", "private":{{ cookiecutter.repo_private }}}'

by
curl -H 'Authorization: token '  {{ cookiecutter.github_accestoken}}  https://api.github.com/user/repos -d ...

After defining the cookiecutter.github_accestoken somewhere?

Hi @lhoupert,
thanks for raising this issue. I am currently sifting through my gh inbox after vacation and am somewhat overwhelmed. Ill try to look into this in the next weeks. Sorry for the delay.

Hey @jbusecke ! No problem, I thought I should flag it up before someone get stuck

Thanks a lot. I'll try to look into it soon!

@lhoupert do you think this problem is sufficiently addressed by #6? I would have to look into the token, but am wondering if setting up an SSH key for github solves this issue for you?

Curious. Glad it works for you now. Feel free to reopen anytime. And thanks for using the template!


Added ContentID to Attachments

Postmark offers the ContentID field for each element in the attachments array:

The picture above is taken from an actual Postmark JSON

This field is used primarily for embedded images in the HTML message body. To support their parsing in target applications, this PR adds the optional contentId field to attachments.
A Note on Field Presence
Theoretically, the ContentID field only makes sense for images or other embeddable attachments. According to our tests however, Gmail sets the ContentID on non-image attachments as well:

Outlook 365, on the other hand leaves it empty:

Therefore, it's not 100% sure that every attachment has a ContentID, so the code in this PR treats it as optional and not necessarily present. In such cases, the value of the field in the DTO will be null. Empty strings will be kept as-is

@alexbuis do you plan to accept this PR?
I'm perfectly fine with a "NO" answer. I just would like to know if it'll be merged anytime soon or if we should proceed with a fork in the project where we need it.
Btw, a big thank you for this library! :pray:

Thanks @fulopattila122 🙏


Improve russian translation

Editing translation for jump-straight-to-the-content-using-the-main-element.russian.md

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

This PR was closed and then re-opened in an attempt to clear a specific Travis build failure.


Improve error handling

Add check for at least one std stream required for /containers/{id}/logs
Add check for state in /containers/{id}/top
Fill in more fields for /info
Fixed error checking in service start code

Signed-off-by: Jhon Honce<EMAIL_ADDRESS>

@baude Still missing some data for /info.  I think it will be blank until we start pushing changes into main.


x/tools/gopls: go list error prevented initial workspace load

@katiehockman reported a separate issue, but in her logs I noticed the following, specifically these go list errors (which would come to gopls by way of go/packages):
internal error: go list gives conflicting information for package crypto/internal/subtle

internal error: go list gives conflicting information for package crypto/internal/randutil

@jayconrod or @bcmills: What does this go list error mean?
Log snippet:
2020/02/05 14:33:43 initial workspace load failed: internal error: go list gives conflicting information for package crypto/internal/subtle
[Info  - 2:33:43 PM] 2020/02/05 14:33:43 go/packages.Load
	snapshot = 0
	query = [./... builtin]
	packages = 0
[Error - 2:33:43 PM] 2020/02/05 14:33:43 initial workspace load failed: internal error: go list gives conflicting information for package crypto/internal/subtle
[Info  - 2:33:43 PM] 2020/02/05 14:33:43 go/packages.Load
	snapshot = 1
	query = [file=/<redacted>/go/src/crypto/ecdsa/ecdsa.go]
	packages = 0
2020/02/05 14:33:43 DocumentSymbols failed: getting file for DocumentSymbols: internal error: go list gives conflicting information for package crypto/internal/randutil
	URI = file:///<redacted>/go/src/crypto/ecdsa/ecdsa.go
[Info  - 2:33:43 PM] 2020/02/05 14:33:43 go/packages.Load
	snapshot = 1
	query = [file=/<redacted>/go/src/crypto/ecdsa/ecdsa.go]
	packages = 0

cc @matloob
This sounds like #36188, which was worked around in golang.org/x/tools/go/packages. What version of gopls was this, and what version of go/packages was it built against?

Yeah, it looks like #36188 didn't make it into gopls/v0.3.0, which was released around the time this issue was filed. Closing this as a result. Thanks, Jay!


ci: Handle twister.xml junit files being empty

junitparse doesn't deal well with being given an empty file.  So build
a filterted list of non-empty files to pass to junitparser.
Additionally handle the case that all junit xml files are empty, if
that is the case skip calling junitparser and junit2html.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Seeing issue with the junit merge here:
https://github.com/zephyrproject-rtos/zephyr/pull/34296
https://buildkite.com/zephyr/zephyr/builds/27457#522237cf-917d-460e-af96-2ddb0f2e8ef9


Reduce log level of components

Motivation
This will close https://github.com/ZcashFoundation/zebra/issues/2655 but it actually does a reduce of the log level of everything inside the components.
I see the majority of the messages of no value to a end user. With this change, after some initial messages the console start to look something as:
Jan 27 13:56:17.029  INFO sync:extend_tips:checkpoint: zebra_consensus::checkpoint: verified checkpoint range block_count=395 current_range=(Excluded(Height(279356)), Included(Height(279751)))
Jan 27 13:56:30.733  INFO sync:extend_tips:checkpoint: zebra_consensus::checkpoint: verified checkpoint range block_count=400 current_range=(Excluded(Height(279751)), Included(Height(280151)))
Jan 27 13:56:49.220  INFO sync:extend_tips:checkpoint: zebra_consensus::checkpoint: verified checkpoint range block_count=400 current_range=(Excluded(Height(280151)), Included(Height(280551)))
Jan 27 13:57:03.039  INFO sync:extend_tips:checkpoint: zebra_consensus::checkpoint: verified checkpoint range block_count=400 current_range=(Excluded(Height(280551)), Included(Height(280951)))
Jan 27 13:57:38.612  INFO sync:extend_tips:checkpoint: zebra_consensus::checkpoint: verified checkpoint range block_count=400 current_range=(Excluded(Height(280951)), Included(Height(281351)))
Jan 27 13:58:34.577  INFO sync:extend_tips:checkpoint: zebra_consensus::checkpoint: verified checkpoint range block_count=400 current_range=(Excluded(Height(281351)), Included(Height(281751)))
Jan 27 13:59:16.077  INFO sync:extend_tips:checkpoint: zebra_consensus::checkpoint: verified checkpoint range block_count=322 current_range=(Excluded(Height(281751)), Included(Height(282073)))

This just shows progress of the sync (and some other messages  from the sync occasionally) but i think progress is what most users will care about.
Solution
Reduce the log level of the components to debug! for a cleaner console output.
Please note a lot of the changes in this PR are just removing the tracing:: prefix where is not needed.
Review
Anyone can review.
Reviewer Checklist

[ ] Make sure info messages you want to see in the console by default were not removed by this PR.

Follow Up Work
I think we will do similar stuff for other zebra crates at some point.

Thank you @conradoplg i think we can merge this and then i will bring a new one with https://github.com/oxarbitrage/zebra/pull/169 into the foundation repo.


Error: invalid operands for multiplication

Got this error when compiling sass using latest gulp and sass

Error in plugin 'sass'
Message:
assets/scss/helpers/mixins/_rem.scss
Error: invalid operands for multiplication
on line 33 of assets/scss/helpers/mixins/_rem.scss
$val: parseInt($value); /* 6 */

when using
  @include rem('font-size', 1.6rem);

however, when trying
  @include rem('font-size', 1.6);

Does not error but outputs the following

#site-title #site-description {
/* 3 /
/ 3 /
/ 4 /
/ 6 /
/ 6 /
/ 8 /
/ 9 */... }

Sorry fixed it. forgot to add in the functions


Fix coordinates conversion with rotation.

That ended up quite big.
This first commit should fix:

coordinates conversion when there is rotation
image clipping with rotation
place holder with rotation but I did not test. What is the best way to test that?

Bugs to fix:

flickering at certain angles. It seems that it has been introduced by my previous PR. I suspect some float arithmetic issue.
tile edge smoothing does not work with rotation (the translation computations are off)

This is a big patch! Seems decent in theory...let me know when you've sorted those bugs out.
I think you should be able to test the placeholder by doing an ImageTileSource with a broken URL...that way the placeholder should stay up the whole time.

So I tested the place holder and it works fine.
The flickering is fixed.
The only remaining issue is the edge smoothing. I spent quite some time on it without coming up with a  fix. I will look again but ideas are welcome :)

By edge smoothing are you referring to #764? Perhaps @mindfreeze might have some ideas.
Meanwhile, I'm still thinking about function names. scalePointsFromPixels etc. seems pretty good, but it does ignore the fact that in addition to scaling it's also translating. I wonder if there is another option that addresses that. pointsFromPixelsNoRotate might be more accurate (if little less pretty). Anyway, @avandecreme, just mentioning it in case you've got any thoughts on the subject.

By edge smoothing are you referring to #764? Perhaps @mindfreeze might have some ideas.

Yes. There is a translation error when there is rotation and I am not sure of the best way to fix that.
To debug that, one can set debugMode: true and zoom in > 110%.
Note that this bug should be present on master as well (but might be easier to fix in this branch thanks to the new conversion methods).

Meanwhile, I'm still thinking about function names. scalePointsFromPixels etc. seems pretty good, but it does ignore the fact that in addition to scaling it's also translating.
I wonder if there is another option that addresses that. pointsFromPixelsNoRotate might be more accurate (if little less pretty). Anyway, @avandecreme, just mentioning it in case you've got any thoughts on the subject.

You are right, scale seems the best term for the delta functions but is not accurate for pointFromPixel.
pointsFromPixelsNoRotate is not pretty but I can't think of a better term either. I guess we should be consistent and name the delta versions with the same pattern.

Another option would be to go with a flag, so you'd have pointsFromPixels(pt) and pointsFromPixels(pt, { noRotate: true }). That second param is where the immediately param goes; we could make it so you can pass a boolean there (for immediately) or an object (for flags like noRotate)...in the latter case, you should be able to add immediately as a property of the object if you want. Just another thought.
Sounds like the edge smoothing translation error with rotation was already part of @mindfreeze's original patch? Or has this patch made it worse? Either way, hopefully he can weigh in here.

Another option would be to go with a flag, so you'd have pointsFromPixels(pt) and pointsFromPixels(pt, { noRotate: true }). That second param is where the immediately param goes; we could make it so you can pass a boolean there (for immediately) or an object (for flags like noRotate)...in the latter case, you should be able to add immediately as a property of the object if you want. Just another thought.

I started by adding a flag but I later felt that the code would be cleaner with separate methods.
An other way to look at the no rotation methods is to consider them like coordinates conversion from browser coordinates to "OSD image ratio" coordinates. With rotation you go from browser to viewport coordinates.

Sounds like the edge smoothing translation error with rotation was already part of @mindfreeze's original patch? Or has this patch made it worse? Either way, hopefully he can weigh in here.

I believe it was already present.

The problem with edge smoothing seems to be related to the scaling. I am looking into it. No luck yet.
But if the problem is there on master as well, you should merge this and create a separate issue for the smoothing thing.

I finally found the trick. See https://github.com/avandecreme/openseadragon/commit/eaed2d5d8a87c4e146ef24745a43750d661fbcc4

@iangilman Any thought on that?

I started by adding a flag but I later felt that the code would be cleaner with separate methods.
An other way to look at the no rotation methods is to consider them like coordinates conversion from browser coordinates to "OSD image ratio" coordinates. With rotation you go from browser to viewport coordinates.

I think the solution I prefer is the NoRotate suffix right now. If that ok with you, I will do the renaming. I think we should then merge this PR and work on the smoothing issue in a new PR.

Sure, I'm fine with the NoRotate suffix, and I'm down with merging this and doing smoothing in a new PR.

Done :)

Lovely! Does this close any tickets?

Lovely! Does this close any tickets?

Actually not. But we should add that to the changelog though. Let me know if you want me to do it.

I've added this PR to the changelog: https://github.com/openseadragon/openseadragon/commit/3c446663c3ce8f59d317ee898b210dda7d1f58a4

Excellent :+1:


Needs and update

Hey, looks like the workflow needs to be triggered.

The latest image built fine?

Not sure why you were encountering an issue. Either way, I've just pushed an unrelated commit, so the workflow will be triggered regardless.

Sorry about not responding. It hadn't updated on dockerhub for the latest update from yesterday for some reason. All is good now though.

No problem. I checked Docker Hub prior to responding here, and the updated timestamp corresponded with the latest commit and workflow/action. So I'm not sure why you were seeing an older image. Regardless, at least it's sorted now!


Getting Started reference to wasm-opt could be more clear. #125

link to old issue
Issue was opeed by @cldershem
In Getting Started the sentence mentioning wasm-opt is a little confusing. It states that in some circumstances one might want to install wasm-opt from the binaryen toolkit, but doesn't say how one does that. Even after clicking through to the toolkit's own README it isn't clear how to install the package.
I'm sure if I took sometime I could potentially figure this out by installing binaryen, but that looks to be a commitment since that involves compiling from source. Since this book is possibly the first substantial intro to Rust & Wasm someone might have, it would be good to keep the barrier of entry as low as possible. A little more of an explanation that this is advanced usage or leaving that as a footnote to move to a later section of the book might help make it less of an issue.

@cldershem the original link in your issue is not longer working, do you have an updated link of the Getting Started?

I already have proposed this, but it was in some PR that I'm not able to find anymore, so here it is again.
What about upgrading https://github.com/pepyakin/wabt-rs to include wasm-opt binary? This will allow installing wasm-opt via cargo.

It still requires a C++ compiler, and probably some other dependencies that I might forget.
It might introduce some confusion if wasm-opt is installed both from wabt-rs and wabt

This is a moved ticket from previous issue that has since been migrated here to the new repo for the book.


500 Internal Server Error

Hi,
I try to connect wordpress to my OpenLDAP but after "Enable Authentication via LDAP" I get "500 Internal Server Error" after login.
My logs:

[Thu Jul 06 13:00:02.817662 2017] [:error] [pid 13104] [client <IP_ADDRESS>:43416] [AuthLDAP] User 'heinz' logging in, referer: https://berlin.de/wp-login.php
[Thu Jul 06 13:00:02.817829 2017] [:error] [pid 13104] [client <IP_ADDRESS>:43416] [AuthLDAP] about to do LDAP authentication, referer: https://berlin.de/wp-login.php
[Thu Jul 06 13:00:02.817900 2017] [:error] [pid 13104] [client <IP_ADDRESS>:43416] [AuthLDAP] connect to LDAP server, referer: https://berlin.de/wp-login.php

WordPress version: 4.8

Hi. Thanks for rausing that issue. Sorry for not getting back to you earlier. I hope you managed to at least get your website back to responding.
To solve the issue I'd need some more information though. First of all: Did you install the plugin via the plugin-manager or did you download it from somewhere? Which version of the plugin is it? Are you sure that the LDAP-extension is available on your system? Was thst the complete output of the logfile you postet there?
Thanks for providing more information. Feel free to contact me via email at andreas AT heigl DOT org

When you used the plugin from the master-branch from GitHub there was an issue that thankfully has been resolved by @rossigee with #135.
So downloading and installing if from GitHub with a fresh download should fix the issue!

Sorry for my late answer, yes you are right I forgot to activate ldap for php5 in Debian:

Install php5-ldap:

sudo apt-get install php5-ldap

Reboot apache

/etc/init.d/apache2 restart

Modify php.ini, and uncomment the line:

;extension=php_ldap.so

I install master-branch and now every is fine.
Thank you for your help and your good plugin :).


Ubuntu 16.04 support

Fails to install with Ubuntu 16.04

The script does not support ubuntu 16. This is something that is being worked on over the next few days

Latest script supports Ubu16

Got sone problems with Webmin install using the script:
root@server:/etc/nginx/sites-available# sudo rtwebmin
Device "eth0" does not exist.
Installing Webmin
OK
grep: /etc/webmin/config: No such file or directory
/usr/local/bin/rtwebmin: line 31: /etc/webmin/config: No such file or directory
grep: /etc/webmin/config: No such file or directory
/usr/local/bin/rtwebmin: line 37: /etc/webmin/config: No such file or directory
grep: /etc/webmin/config: No such file or directory
/usr/local/bin/rtwebmin: line 43: /etc/webmin/config: No such file or directory
grep: /etc/webmin/config: No such file or directory
/usr/local/bin/rtwebmin: line 49: /etc/webmin/config: No such file or directory
grep: /etc/webmin/config: No such file or directory
/usr/local/bin/rtwebmin: line 55: /etc/webmin/config: No such file or directory
grep: /etc/webmin/miniserv.conf: No such file or directory
/usr/local/bin/rtwebmin: line 61: /etc/webmin/miniserv.conf: No such file or directory
Setting up Nginx Reverse Proxy
Completed
Webmin can be accessed at https:///webmin
login using root, or sudo user credentials

There seems to be an issue with Webmin on Ubu16. I will look into it.

suddenly the second user ain't working:
[01.05.2016 23:47:06] WebUI started.
[01.05.2016 23:47:06] Bad response from server: (500 [error,getplugins]) Internal Server Error
user 1 is fine...
tried to delete user 2 and make a new one but it's the same

Trying to help a friend, and this happened.
root:~/working# clear
root:~/working# ls
rtinst.sh
root@ns3041775:~/working# bash rtinst.sh
rtinst.sh: line 139: [: : integer expression expected
rtinst.sh: line 139: [: : integer expression expected
rtinst.sh: line 139: [: : integer expression expected

Only Ubuntu release 12 and later, and Debian and Raspbian release 7 and later, are supported
Your system does not appear to be supported
root:~/working# uname -a ; echo ; lsb_release -a
Linux ns3041775.ip-5-135-186.eu 3.14.32-xxxx-grs-ipv6-64 #7 SMP Wed Jan 27 18:05:09 CET 2016 x86_64 x86_64 x86_64 GNU/Linux

No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04 LTS
Release:	16.04
Codename:	xenial
root:~/working#

So I'm honestly quite confused. As far as I know, it's a fresh install.

added an alternative way to check for OS details, using lsb_release when available.

webmin have resolved the issue with ubuntu 16, so this should work now, if you get a message saying that it isn't when you try to run it, just update the scripts, and run again.


Fix potential redis key leaking on channel unsubscription

close as potential risk condition to wrongly delete the channel key count.


Does superset support mobile device?

I am considering using superset as BI tool to our team and we have the mobile device requirement. I have not found the clear instruction that superset support mobile device.

Hi there, Superset doesn't work on mobile devices at the moment.

Mobile app: no mobile app for Superset
Mobile web support: some features may work on mobile but Superset isn't tested regularly for smaller viewports

Thanks Srini!
@xlhtc007 Are you looking for chart/dashboard view only on mobile? or more like interactive? Neither is in our near-term roadmap. It isn't hard to implement view-only mobile i believe, and some forks have done it in the past. It will be really great to have Superset work on mobile; please let us know if you would love to work on it :)


PLAT-90767: Scale the VirtualGridList while scrolling to the moveDistance distance

Checklist

[X] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[X] At least one test case is included for this feature or bug fix
[X] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
: Scale the VirtualGridList while scrolling to the moveDistance distance
Additional Considerations
When starting scrolling the VirtualList, the list moves upper. I expect that behavior would be implemented by a panel or the POC for controller because the VirtualList positions upper naturally if the panel body including the list enlarges and moves upper.  To verify this PR, I implemented that behavior app side. Please use the app described in the JIRA to verify this PR.
Links
PLAT-90767
Comments

Closing in favor of work in feature/PLAT-90799.


vcbuild.bat is terminated by "lf" end of line

Since "vcbuild.bat" is always used on Windows, this file should be end terminated by "crlf".
if vcbuild.bat is set with "lf" the script always end with :
the system cannot find the batch label specified - exit
so my script handle error but the build is ok

vcbuild is stored as a text file, make sure you have proper line ending option selected:
git config --global core.autocrlf true

See https://help.github.com/articles/dealing-with-line-endings/.


COPY button SLO content

Similar to issue #1794 it would be great to have the COPY button also for the content of SLO file on the evaluation done events.

Implementing this should be quite easy. Please look at PR #1977 to see how it's done for the JSON Payload.

I'd like to work on this issue.


True Colour in Python Curses

Support for true colour in python curses
Python curses fulfils almost all the needs for the program (keyboard inputs, timings and delays, seeking), except one. As of yet, python curses doesn't support true colours (it only supports 8 or 16 colors)
Therefore, any method to somehow use true colors in python curses is appreciated.
In case this is not possible, look at issue  #2

True colour support is not in curses. Solution is to maintain two releases for true colour and 8-colour/legacy terminals. Users then can choose per their preference


manifest.py logs to go into desktop folders too? :)

Thanks for requesting this! Fixed here: https://github.com/kieranjol/IFIscripts/commit/f70f7ae06efc340d517cada0e64beb289d7661d3 - update.py and lemme know if all works as it should :)


add license title

It's not strictly required, but it's useful metadata, and part of the recommended license template text (see http://choosealicense.com/licenses/isc/ and https://opensource.org/licenses/isc-license)

Coverage remained the same at 100.0% when pulling a6dd881ade239e35ccede7d6c963428422882564 on waldyrious:patch-1 into f0acd2a31150b0403c14eed53e6234b192704257 on floridoo:master.


remove unused package LegacyVersion and unused variables

Fresh install of deepdoctection results into missing legacyVersion error and unused variables.
AttributeError: module 'packaging.version' has no attribute 'legacyVersion'

NameError: name 'context' is not defined

Closes #178

Thanks for the PR.
I have a one question and one remark.
You can start a CI run in your branch by adding [force ci] in the git message, e.g.
git commit -m “add stuff [force ci]”

 This will run tests with Python 3.8.


Touch-based fling inertia does not work when using engine embedding

Summary
When using the Flutter engine embedder C API (flutter_embedder.h), when the FlutterPointerDeviceKind is kFlutterPointerDeviceKindTouch (i.e., using touch input), there is no scroll fling or inertia observed. When scrolling a ListView, when the "finger" leaves the "screen", there is no further inertia. The scroll stops immediately.
https://user-images.githubusercontent.com/5544935/216794808-9b91be1d-d8d7-4eda-b941-e9738c394051.mp4
I believe this is due to the C API lacking fields in the FlutterPointerEvent struct to describe touch delta values. There is only the absolute x and y fields.
When digging through the source code for how the Android platform dispatches touch events to the Flutter engine, I noticed that it does not actually use this API. Instead, it uses the PointerDataPacketConverter and a binary packet interface to embed mouse delta values.

The C API for Flutter embedding lacks the physical_delta_x and physical_delta_y fields, and the underlying engine implementation likely is unable to compute scroll inertia without these values.
Steps to Reproduce

Embed Flutter using the C API
Dispatch touch events to the engine using SendPointerEvent with the kMove phase and the kFlutterPointerDeviceKindTouch device type
Observe no scroll inertia or flinging.

Expected results:
Scroll flinging should work the same way it does on Android/iOS.
Actual results:
No scrolling inertia, likely due to the missing physical delta fields from the C API.

Logging of individual pointer events sent to Flutter over the C API during the above screen recording
Adding (0.615958, 0.260847)
Hover (0.609077, 0.262936)
Hover (0.599629, 0.262196)
Hover (0.588661, 0.263521)
Hover (0.578187, 0.267594)
Hover (0.56594, 0.273915)
Hover (0.551606, 0.282605)
Down (0.540295, 0.294847)Mov (0.540295, 0.294847)
Mov (0.52402, 0.314123)
Mov (0.506194, 0.336497)
Mov (0.49136, 0.36525)
Mov (0.478065, 0.40013)
Mov (0.471733, 0.436331)
Mov (0.469626, 0.483688)
Mov (0.466506, 0.535872)
Mov (0.465543, 0.589788)
Mov (0.46465, 0.644361)
Mov (0.458974, 0.693993)
Mov (0.456731, 0.734393)
Up (0.460067, 0.761879)
Hover (0.462284, 0.787889)
Hover (0.470545, 0.802322)
Hover (0.479675, 0.812034)
Hover (0.489785, 0.817329)
Hover (0.496692, 0.819357)
Removing (0.501063, 0.820074)
Adding (0.50233, 0.809722)
Hover (0.49676, 0.809108)
Hover (0.490991, 0.80841)
Hover (0.487995, 0.803566)
Hover (0.483943, 0.800553)
Hover (0.477866, 0.793941)
Hover (0.470102, 0.787162)
Hover (0.462693, 0.777975)
Hover (0.453636, 0.764289)
Down (0.444506, 0.7477)Mov (0.444506, 0.7477)
Mov (0.434431, 0.725796)
Mov (0.423657, 0.695749)
Mov (0.412802, 0.664183)
Mov (0.406776, 0.634159)
Mov (0.401406, 0.595647)
Mov (0.403411, 0.559436)
Mov (0.409139, 0.528334)
Mov (0.419717, 0.497461)
Mov (0.438535, 0.467352)
Mov (0.461323, 0.437837)
Mov (0.488728, 0.411929)
Mov (0.515137, 0.389213)
Mov (0.54017, 0.374823)
Up (0.567696, 0.361697)
Hover (0.590687, 0.356167)
Hover (0.607056, 0.350015)
Hover (0.618185, 0.345559)
Hover (0.625103, 0.34499)
Removing (0.630029, 0.343998)

Do you have an example I can run to see the problem, I would expect all events to still go through the converter in this case. Also, a printout of each ui.PointerData which arrives in dart would be helpful.

Hey @moffatman , thanks so much for the super rapid response. Honestly was not expecting such prompt response for such a weird niche use case of the embedder. I'll get the logging of the dart pointerdata asap, one moment!

Wow, OK, so you were right. Dart is in fact receiving physicalDeltaX/Y. Here's the entire dump of one "swipe" on the virtual touchscreen.

Expand meeee!

I already tried some ideas, including

Making a custom MaterialScrollBehavior that made sure dragDevices included .touch
Making the ListView have physics: const BouncingScrollPhysics()
Emulating other types of input devices
Fooling Flutter into thinking I was on TargetPlatform.android or TargetPlatform.fuschia or any other platform
Making sure I populate the device field with reasonable 0/1 values for left or right handed fingers
Setting FlutterPointerEvent.buttons to 0
Messing with the FlutterPointerEvent.timestamp, pushing it into the future or past, to see if it is a timing weirdness
Not removing the pointers at all (maybe issuing a kRemove was making the fling stop? Didn't seem to be the case...)
Not firing kHover
Lots of things I can't remember right now

I tried running it but just got a grey screen with the floating hand, no flutter window. The dart code was running, though, I added a printout and saw it in the console. Any library I need to install manually?
Two other things you can check for me though

The timestamps in dart - maybe if timestamps are off in units (ms vs us?), the calculated velocity-per-second could ~=0.
A print at the beginning of ScrollDragController::end to print out the details. Then we can see the actual final velocity.

Ah, what you're seeing is that the user's view in the 2D window is in front of that window. You can check out the controls for the simulator here: https://stereokit.net/Pages/Guides/Using-The-Simulator.html
To reorient the camera, you just hold Shift and use WASDQE + holding down right click to look around. You shouldn't need any other deps in your system, you just need to make the camera look in the right direction/position.
Oh, I think you might be onto something with the units of the timestamp. I am directly using the result of FlutterEngineGetCurrentTime(), which returns in nanoseconds. Looks like the pointer event requires microseconds! That might have fixed it, let me check.

https://user-images.githubusercontent.com/5544935/216851287-77577a7d-9964-40d8-aad8-799629e30aa7.mp4
That was it! @moffatman you rock :) It was the wrong timestamp units. The comments on engine_embedder.h definitely confused me because it said to use "the same clock as FlutterEngineGetCurrentTime".

But I conveniently didn't read the bit about the units :D
Thanks so much for the quick response and super helpful info. Cheers!!

Nice! Happy to help, and looks like a cool project 👍


Update sbt to 1.3.0

Updates org.scala-sbt:sbt from 1.2.8 to 1.3.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: semver-minor

Coverage remained the same at 95.205% when pulling 9cec1a29f9970224c02ec404e9e943e94fc432da on scala-steward:update/sbt-1.3.0 into 05ca406c633c8970133f1558bfccdb6cbcdc8397 on iheartradio:master.


Remove psutil usage in checker

Signed-off-by: Faseela K<EMAIL_ADDRESS>

oops spoke to soon - you need to do os.cpu_count() or 1 as i mentioned before


Implement backend for FiniteDifferences

This PR implements a backend for FiniteDifferences.jl. It basically combines the 3 declarations in the tests into 1, with generalization of pullback_function to multi-output functions.
Informal benchmarks showed that all three of these methods were necessary for speed. They also showed that with these defined, our default gradient and multi-arg derivative methods are faster than custom overloads using jvp and grad. A single-arg derivative method was faster than our default, yet for some reason it was less accurate.

Codecov Report

Merging #37 (4123a74) into master (979fa04) will increase coverage by 0.29%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #37      +/-   ##
==========================================
+ Coverage   80.05%   80.34%   +0.29%
==========================================
  Files           2        3       +1
  Lines         396      407      +11
==========================================
+ Hits          317      327      +10
- Misses         79       80       +1

Impacted Files
Coverage Δ

src/AbstractDifferentiation.jl
77.93% <100.00%> (-0.28%)
:arrow_down:

src/finitedifferences.jl
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

thanks Seth!


[Question] vshader issues when porting ctrcommon to buildtools

Since GBATemp is down at time of writing I figured I'd shoot this question as a github issue. I'm trying to port Nanquitas BootNTR Selector from using ctrcommon to buildtools and I got it nearly all correct I think but whenever I run make with buildtools I get the following error, which I do not get when using a clean download of the repo.
Building for 3DS...
build/3ds-arm/source/vshader.shbin.c
build/3ds-arm/source/vshader.shbin.o
build/3ds-arm/source/appInfo.o
build/3ds-arm/source/button.o
build/3ds-arm/source/common_functions.o
build/3ds-arm/source/config.o
build/3ds-arm/source/draw.o
source/draw.c: In function 'sceneInit':
source/draw.c:152:53: error: 'vshader_shbin_size' undeclared (first use in this function)
  vshader_dvlb = DVLB_ParseFile((u32*)vshader_shbin, vshader_shbin_size);
                                                     ^
source/draw.c:152:53: note: each undeclared identifier is reported only once for each function it appears in
make: *** [build/3ds-arm/source/draw.o] Error 1
rm build/3ds-arm/source/vshader.shbin.c

The make output from a clean download is: https://hastebin.com/jugebozuke.swift
The makefile I have for buildtools right now is: https://hastebin.com/mowawodili.makefile
As a side question, I can't find any references in buildtools how it would handle previously created bnr files, I just assumed defining it as BANNER would do but if that's wrong please also let me know.

Try chaning size to len.

Sorry, but could you clarify? I can't quite find what you mean with that.

"vshader_shbin_size". Try changing the "size" to "len".

Seems to be the thing that solves it indeed. I'm having some portlibs issues right now that I have to fix before the building will work (it is failing on libpng stuff). I'll close the issue with this comment.
(damn Cell9 it would've been great if you had mentioned dependancies)


Enhance support for (partial) table extraction

This PR adds support for access via the SOMAReader class to extract selected columns in a single pass. Query condition and range support will be added next.
The retrieved data structure is a from an Arrow-support package that works only on the lighterweight C interface to Arrow without linking.  Once data has been accessed, we can easily transfer to different Arrow data structures:
> library(tiledbsoma)
> uri <- "test/soco/pbmc3k_processed/obs"  # local data set
> columns <- c("n_counts", "n_genes", "louvain")
> z <- export_recordbatch(uri, columns)
[2022-10-07 18:00:12.785] [tiledbsoma] [Process: 66049] [Thread: 66049] [info] Reading from test/soco/pbmc3k_processed/obs
[2022-10-07 18:00:12.795] [tiledbsoma] [Process: 66049] [Thread: 66049] [info] Read complete with 2638 obs and 3 cols
[2022-10-07 18:00:12.795] [tiledbsoma] [Process: 66049] [Thread: 66049] [info] Accessing n_counts at 0
[2022-10-07 18:00:12.795] [tiledbsoma] [Process: 66049] [Thread: 66049] [info] Accessing n_genes at 1
[2022-10-07 18:00:12.795] [tiledbsoma] [Process: 66049] [Thread: 66049] [info] Accessing louvain at 2
>
> rb <- arch::from_arch_array(z, arrow::RecordBatch)   ## Arrow RecordBatch
> rb
RecordBatch
2638 rows x 3 columns
$ <float not null>
$ <int64 not null>
$ <large_binary not null>
>
> tb <- arrow::as_arrow_table(arch::from_arch_array(z, arrow::RecordBatch))  ## Arrow Table
> tb
Table
2638 rows x 3 columns
$ <float not null>
$ <int64 not null>
$ <large_binary not null>
>

(rebased and force-pushed)


Determine whether tabs should be scenes or frames

#84 and #658 are related.

Tabs will stay as frames as there is no need to make them scenes and doing so would just make things much more complicated for no reason.


Added InnerXml to all structs following https://github.com/GoComply/x…

…sd2go/pull/8
https://github.com/GoComply/xsd2go/pull/8

Thanks.
Would you mind to release v0.1.1?

Done.


S3 list operation requested when file not found

What happened:
When trying to open a file that does not exist in an AWS S3 bucket, a list operation is requested. In addition, a FileNotFoundError exception is raised.
What you expected to happen:
I would expect to see a FileNotFoundError exception being raised only. No list operations should be requested. It seems unnecessary and AWS S3 charges for it.
Minimal Complete Verifiable Example:
When running:
import s3fs

storage_options = {
    'key': 'my-key',
    'secret': 'my-secret',
    'use_ssl': False,
    'client_kwargs': {
        'endpoint_url': 'http://my-url:my-port'
    },
}
file_system = s3fs.S3FileSystem(**storage_options)
file_system.open('s3://my-bucket/my_file.txt', 'r')

The output is:
FileNotFoundError: my-bucket/my_file.txt

And my HTTP traffic monitor shows the GET request for the list operation (note the list-type=2):
GET     /my-bucket?list-type=2&prefix=my_file.txt%2F&delimiter=%2F&max-keys=1&encoding-type=url

Anything else we need to know?:
The list operation seems to be requested when this line is executed.
Environment:

Python version: 3.6.9
Operating System: Ubuntu 18.04.3 LTS
Install method (conda, pip, source): pip install s3fs
Output of pip list:

Package           Version
----------------- -------
aiobotocore       1.1.2
aiohttp           3.6.2
aioitertools      0.7.0
async-timeout     3.0.1
attrs             20.2.0
botocore          1.17.44
chardet           3.0.4
docutils          0.15.2
fsspec            0.8.3
idna              2.10
idna-ssl          1.1.0
jmespath          0.10.0
multidict         4.7.6
pip               20.2.3
pkg-resources     0.0.0
python-dateutil   2.8.1
s3fs              0.5.1
setuptools        50.3.0
six               1.15.0
typing-extensions <IP_ADDRESS>
urllib3           1.25.10
wheel             0.35.1
wrapt             1.12.1
yarl              1.6.0

(sorry fro the slow response)
Can you please turn on s3fs logging (S3FS_LOGGING_LEVEL=true or configure the "s3fs" logger). I am not seeing that there are TWO calls here.

Thanks for helping out. Here's the log output:
2020-10-19 19:10:59,056 - s3fs - DEBUG - Setting up s3fs instance
2020-10-19 19:10:59,157 - s3fs - DEBUG - CALL: head_object - ({},) - {'Bucket': 'my-bucket', 'Key': 'my_file.txt'}
2020-10-19 19:10:59,168 - s3fs - DEBUG - Nonretryable error: An error occurred (404) when calling the HeadObject operation: Not Found
2020-10-19 19:10:59,168 - s3fs - DEBUG - CALL: list_objects_v2 - ({},) - {'Bucket': 'my-bucket', 'Prefix': 'my_file.txt/', 'Delimiter': '/', 'MaxKeys': 1}

My HTTP traffic monitor reports:
HEAD    /my-bucket/my_file.txt
GET     /my-bucket?list-type=2&prefix=my_file.txt%2F&delimiter=%2F&max-keys=1&encoding-type=url

@martindurant In case this helps, I ran some additional tests and I see a single call (a HEAD request only) as long as I use an existing prefix.
For example:

Trying to open s3://my-bucket/existing-prefix/my_file.txt will generate a single call (a HEAD request).
Trying to open s3://my-bucket/non-existent-prefix/my_file.txt will generate a two calls (a HEAD request + a GET request for a list operation).

This seems related. When trying to delete an object that doesn't exist with recursive=True, like so:
import s3fs

storage_options = {
    'key': 'my-key',
    'secret': 'my-secret',
    'use_ssl': False,
    'client_kwargs': {
        'endpoint_url': 'http://my-url:my-port'
    },
}
file_system = s3fs.S3FileSystem(**storage_options)
file_system.rm('s3://my-bucket/my_file.txt', recursive=True)

the log output is (note the LIST operations):
2020-10-20 08:34:58,833 - s3fs - DEBUG - Setting up s3fs instance
2020-10-20 08:34:58,929 - s3fs - DEBUG - Get directory listing page for my-bucket/my_file.txt
2020-10-20 08:34:59,065 - s3fs - DEBUG - CALL: head_object - ({},) - {'Bucket': 'my-bucket', 'Key': 'my_file.txt'}
2020-10-20 08:34:59,149 - s3fs - DEBUG - Nonretryable error: An error occurred (404) when calling the HeadObject operation: Not Found
2020-10-20 08:34:59,149 - s3fs - DEBUG - CALL: list_objects_v2 - ({},) - {'Bucket': 'my-bucket', 'Prefix': 'my_file.txt/', 'Delimiter': '/', 'MaxKeys': 1}
2020-10-20 08:34:59,232 - s3fs - DEBUG - CALL: head_object - ({},) - {'Bucket': 'my-bucket', 'Key': 'my_file.txt'}
2020-10-20 08:34:59,274 - s3fs - DEBUG - Nonretryable error: An error occurred (404) when calling the HeadObject operation: Not Found
2020-10-20 08:34:59,274 - s3fs - DEBUG - CALL: list_objects_v2 - ({},) - {'Bucket': 'my-bucket', 'Prefix': 'my_file.txt/', 'Delimiter': '/', 'MaxKeys': 1}

OK, so this behaviour happens because info (and ls) doesn't know whether the path supplied is meant to be a file or a directory. info is needed to get the size of the file, to enable random access i.e., you won't see this if you get or cat. IF the parent directory has not yet been listed, then the expected fastest operation is head_object, which works if the target is a file, then list_objects (maxkeys=1) will tell you if in fact its a directory. This was done as a performance improvement over listing the parent, which could be expensive.
It would be reasonable for open to call a modified version of info, with some flag kwarg, which excludes a directory part, since you cannot open a directory.
I don't know offhand why rm might be listing twice, or calling head_object at all, would be worth looking into - but this is probably the rarer operation.

Thanks for the explanation! I'll look into it.

Would it be desirable to modify S3FileSystem._open() so that it does something like this before attempting to open a file?
...
bucket, key, _ = self.split_path(path)
try:
    self.s3.head_object(Bucket=bucket, Key=key))
except ClientError as e:
    raise translate_boto_error(e) from e
...

According to this, translate_boto_error(e) will return a FileNotFoundError when e.response['Error'].get('Code') is '404'.
In the meantime, I'm using file_system.s3.head_object() as a workaround.

The problem with this is, that you are ignoring the file listing cache, so that you do the lookup on every open, even if you have previously listed the prefix. You could have a middle-ground of checking self._ls_from_cache first, but you do end up repeating code.


Improvements to True Wind Acuracy

Made changes to TWD display.

Changes to improve display of True Wind data


[Admin] Publish

Docs Build status updates of commit 02a3433:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/overview/office-add-ins-community-call.md
:white_check_mark:Succeeded
View

README.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


@conda-forge-admin, please add user @wguanicedew

[x ] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:
Please help wguanicedew<EMAIL_ADDRESS>into this package maintainer list.

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @wguanicedew in conda-forge/idds-common-feedstock#16.


Added the ability to specify multiple init scripts for the CassandraContainer.

Currently, the CassandraContainer allows you to specify only one file containing the initialization script. I added the ability to specify multiple init scripts for the CassandraContainer.

How about a directory instead of a list? https://github.com/testcontainers/testcontainers-java/pull/3900

Thanks for the PR, @peneksglazami ! There is a liquibase-cassandra project which allows handling script migrations. Given that testcontainers allows testing against a genuine Cassandra service, the scripts applied to it can be performed by the project mentioned.


vimeo cannot get direct link to video

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.03.16. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.03.16

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v -g https://vimeo.com/208726519
C:\Users\Admin\Desktop>C:\Users\Admin\Desktop\youtube-dl.exe -v -g https://vimeo
.com/208726519
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-g', 'https://vimeo.com/208726519']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2017.03.16
[debug] Python version 3.4.4 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg 1.2
[debug] Proxy map: {}
WARNING: Unable to download JSON metadata: HTTP Error 403: Forbidden
https://skyfire.vimeocdn.com/1489930311-0x7c07b622031a676fcb6cb0c2574f29660745e2
bd/208726519/sep/video/713530452,713530460,713530459,713530453/../
https://skyfire.vimeocdn.com/1489930311-0x7c07b622031a676fcb6cb0c2574f29660745e2
bd/208726519/sep/video/713530452,713530460,713530459,713530453/../
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://vimeo.com/208726519

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Cannot get direct link to this video https://vimeo.com/208726519

I've get it as seen from the log.


fairseq-generate output reference sentence without space.

What is your question?
I am using mBart model to generate on a model for Hindi to English direction. The reference sentence which output from the fairseq-generate command with the T symbol has no spaces in between.
The translations are good, comparing it with the raw text. But I cannot get the reference directly from the output of the fairseq-generate.
Steps
Followed the following steps as mentioned in mBart example.

Run spm_encode from sentencepice model
fairseq-preprocess
fairseq-train
fairseq-genreate

Code
CUDA_VISIBLE_DEVICES=0 fairseq-generate $DATA_PATH \
    --path ${MODEL_PATH}/${MODEL_NAME} \
    --task translation_from_pretrained_bart \
    --gen-subset test \
    --source-lang ${SRC} --target-lang ${TGT} --bpe 'sentencepiece' \
    --sentencepiece-model $MODEL_BPE \
    --scoring sacrebleu \
    --remove-bpe 'sentencepiece' \
    --batch-size 32 --langs $langs \
    --results-path results/${ARCH}.${SRC}-${TGT}

A section of the output log
2020-12-23 15:50:21 | INFO | fairseq_cli.generate | Namespace(all_gather_list_size=16384, beam=5, bf16=False, bpe='sentencepiece', broadcast_buffers=False, bucket_cap_mb=25, checkpoint_suffix='', constraints=None, cpu=False, criterion='cross_entropy', data='/scratch/cse/btech/cs1170339/data/processed/IITB/fairseq_preprocess/normal_0.5', data_buffer_size=10, dataset_impl=None, ddp_backend='c10d', decoding_format=None, device_id=0, distributed_backend='nccl', distributed_init_method=None, distributed_no_spawn=False, distributed_num_procs=None, distributed_port=-1, distributed_rank=0, distributed_world_size=1, distributed_wrapper='DDP', diverse_beam_groups=-1, diverse_beam_strength=0.5, diversity_rate=-1.0, empty_cache_freq=0, eval_bleu=False, eval_bleu_args=None, eval_bleu_detok='space', eval_bleu_detok_args=None, eval_bleu_print_samples=False, eval_bleu_remove_bpe=None, eval_tokenized_bleu=False, fast_stat_sync=False, find_unused_parameters=False, fix_batches_to_gpus=False, force_anneal=None, fp16=False, fp16_init_scale=128, fp16_no_flatten_grads=False, fp16_scale_tolerance=0.0, fp16_scale_window=None, gen_subset='test', iter_decode_eos_penalty=0.0, iter_decode_force_max_iter=False, iter_decode_max_iter=10, iter_decode_with_beam=1, iter_decode_with_external_reranker=False, langs='ar_AR,cs_CZ,de_DE,en_XX,es_XX,et_EE,fi_FI,fr_XX,gu_IN,hi_IN,it_IT,ja_XX,kk_KZ,ko_KR,lt_LT,lv_LV,my_MM,ne_NP,nl_XX,ro_RO,ru_RU,si_LK,tr_TR,vi_VN,zh_CN', left_pad_source='True', left_pad_target='False', lenpen=1, load_alignments=False, localsgd_frequency=3, log_format=None, log_interval=100, lr_scheduler='fixed', lr_shrink=0.1, match_source_len=False, max_len_a=0, max_len_b=200, max_sentences=32, max_source_positions=1024, max_target_positions=1024, max_tokens=None, memory_efficient_bf16=False, memory_efficient_fp16=False, min_len=1, min_loss_scale=0.0001, model_overrides='{}', model_parallel_size=1, nbest=1, no_beamable_mm=False, no_early_stop=False, no_progress_bar=False, no_repeat_ngram_size=0, no_seed_provided=True, nprocs_per_node=1, num_batch_buckets=0, num_shards=1, num_workers=1, optimizer=None, path='/scratch/cse/btech/cs1170339/models/mbart_en-hi/checkpoints_hi_IN-en_XX_0.5/checkpoint_best.pt', pipeline_balance=None, pipeline_checkpoint='never', pipeline_chunks=None, pipeline_devices=None, pipeline_model_parallel=False, prefix_size=0, prepend_bos=False, print_alignment=False, print_step=False, profile=False, quantization_config_path=None, quiet=False, remove_bpe='sentencepiece', replace_unk=None, required_batch_size_multiple=8, results_path='results/transformer.hi_IN-en_XX', retain_dropout=False, retain_dropout_modules=None, retain_iter_history=False, sacrebleu=False, sampling=False, sampling_topk=-1, sampling_topp=-1.0, score_reference=False, scoring='sacrebleu', seed=1, sentencepiece_model='/scratch/cse/btech/cs1170339/models/mbart/mbart.cc25/sentence.bpe.model', shard_id=0, skip_invalid_size_inputs_valid_test=False, slowmo_algorithm='LocalSGD', slowmo_momentum=None, source_lang='hi_IN', target_lang='en_XX', task='translation_from_pretrained_bart', temperature=1.0, tensorboard_logdir='', threshold_loss_scale=None, tokenizer=None, tpu=False, truncate_source=False, unkpen=0, unnormalized=False, upsample_primary=1, user_dir=None, warmup_updates=0, zero_sharding='none')
2020-12-23 15:50:22 | INFO | fairseq.tasks.translation | [hi_IN] dictionary: 250001 types
2020-12-23 15:50:22 | INFO | fairseq.tasks.translation | [en_XX] dictionary: 250001 types
2020-12-23 15:50:22 | INFO | fairseq.data.data_utils | loaded 2507 examples from: /scratch/cse/btech/cs1170339/data/processed/IITB/fairseq_preprocess/normal_0.5/test.en_XX-hi_IN.hi_IN
2020-12-23 15:50:22 | INFO | fairseq.data.data_utils | loaded 2507 examples from: /scratch/cse/btech/cs1170339/data/processed/IITB/fairseq_preprocess/normal_0.5/test.en_XX-hi_IN.en_XX
2020-12-23 15:50:22 | INFO | fairseq.tasks.translation | /scratch/cse/btech/cs1170339/data/processed/IITB/fairseq_preprocess/normal_0.5 test hi_IN-en_XX 2507 examples
2020-12-23 15:50:22 | INFO | fairseq_cli.generate | loading model(s) from /scratch/cse/btech/cs1170339/models/mbart_en-hi/checkpoints_hi_IN-en_XX_0.5/checkpoint_best.pt
S-1553	अभीवहांकुल201कार्डिनल्सहैं।[hi_IN]
T-1553	Therearenow201cardinals.
H-1553	-0.8420987725257874	There are 201 cards right there.
D-1553	-0.8420987725257874	Thereare201cardsrightthere.
P-1553	-1.1698 -0.2925 -0.3764 -1.1276 -3.0362 -0.7208 -0.3474 -0.2848 -0.2235
S-2355	परिवारकेसदस्यखौफमेंहैं।[hi_IN]
T-2355	Thefamilymembersarefrightened.
H-2355	-0.6770707964897156	The family members are in fear.
D-2355	-0.6770707964897156	Thefamilymembersareinfear.
P-2355	-1.0040 -0.7288 -0.1219 -0.3241 -1.0873 -2.0004 -0.3123 -0.2953 -0.2197
S-1897	वेमीडियावालोंकेभीजवाबनहींदेते।[hi_IN]
T-1897	Theydonotanswertothemedia.
H-1897	-0.8603734970092773	They don 't even respond to the media.

Help
What am I doing wrong?
I expect the sentences marked as S (Source) and T (Translation/Reference) are expected to be with space.

Taking out --remove-bpe 'sentencepiece' should fix this problem. Same issue as https://github.com/pytorch/fairseq/issues/331.

@trina731 thanks :)
Yes, you are right.
Removing --remove-bpe 'sentencepeice' outputs the tokens in BPE directly. For example
T-2298	▁All ▁the ▁best ▁for ▁today ▁ .
H-2298	-0.8229939937591553	▁Good ▁wish es ▁for ▁today ▁ .
D-2298	-0.8229939937591553	▁Good ▁wish es ▁for ▁today ▁ .

But it is favorable to use --remove-bpe sentencepiece which is supposed to detokenize.
In an earlier version of fairseq it would remove the bpe and leave no space at all between words as mentioned in #331.
On updating the version to the latest v0.10.2 that too is fixed. For example now I get
T-961	This has gained wide acceptance .
H-961	-0.6537348628044128	It has received extensive approval .
D-961	-0.6537348628044128	It has received extensive approval .

Thus closing this issue.

I am not sure how this can happen. But on different tasks I get different behaviour

with task as translation_from_pretrained_bart

I get
S-255	Балаларсабағынабараалмайжатыр.[kk_KZ]
T-255	Childrencan'tgotoschool.
H-255	-1.3812958002090454	They are not going to go to school .
D-255	-1.3812958002090454	Theyarenotgoingtogotoschool.

Where there is no spacing between S,T and D whereas for H the spacing is correct.

with task as translation_multi_simple_epoch
I get

S-950	__hi_IN__ तमारी वात साची छे .
T-950	What you say is right .
H-950	-0.7007428407669067	You are right .
D-950	-0.7007428407669067	You are right .
P-950	-0.3226 -1.9887 -1.2709 -0.3252 -0.3545 -0.3353 -0.3080

I had the same problem and removing  --bpe 'sentencepiece'  --sentencepiece-model $MODEL_BPE  solved it for me.
Looking at the code in generate.py decode_fn it detokenized the sentences twice, for the bpe and for the tokenizer.

The sentence string is post-processed twice. For example, src_str is applied the spm based remove-bpe twice.
https://github.com/pytorch/fairseq/blob/a9f5741f58c05c581686b73465d7e3f9df5528f3/fairseq_cli/generate.py#L238
https://github.com/pytorch/fairseq/blob/a9f5741f58c05c581686b73465d7e3f9df5528f3/fairseq_cli/generate.py#L251
i.e.,
x.replace(" ", "").replace("\u2581", " ").strip()

For inference script, you can simply set --remove-bpe "none" and it will set the cfg.common_eval.post_process=None.

Do not use --remove-bpe and then manully post-process the text using scripts/spm_decode.py


Use lastest (master version) of node-plop with CLI

I'm trying to use some earliest features of node-plop (append action, manually passing data to actions, ...), but i can't figure out how to install plop globally with the latest version of node-plop from master branch.
When are you planning to upgrade version though ?
Those new features are really useful, and i don't want to lose time waiting for it, or do a work-around to always be up-to-date (isn't it the goal of this awesome tool - save time ?)
Thanks for your excellent tool.

I'll try to get this published today. Sorry, I don't normally let things wait for so long... things got really busy for me. I'll at least get node-plop out there today

v0.13.0 is now on NPM

@amwmedia Thanks, amazing !


Heating circuit operation mode (register 5006)

I get for register 5006 and 5106 the value -300.0.
Do you have here the mode number?
thx.

Sorry, can't help here. OP mode numbers go from 0 to 7, -300 must be an  error.
Cheers....Ralf

The question is if you have here a Number? If so lambda has to configure something that this is readable.

Yes, I have OP mode "3" (auto-heating):

Thx Ralf, so there will be a config action from lambda, to let me access the OP Mode register, additional I don’t have the „soll-temp“ (set point flow line/ 5005&5105).
What for a user level do you have?
thx

My heatpump is at v1.20 SP1, my HA installation at 2022.11.4, the "user level" at the heatpump when logged in is "1". Lambda does not give higher user levels to end users.
Cheers....Ralf

Thx Ralf, so there will be a config action from lambda, to let me access the OP Mode register, additional I don’t have the „soll-temp“ (set point flow line/ 5005&5105).
What for a user level do you have?
thx

Hello
does reading the registers work with the new yaml file ? I'm also interested, because I'm getting my Lambda mounted in January and would like to connect it to HomeAssistant as well.
Greetings Ralph

Hi Ralph,
yes, of course, I use the new yaml file in my own setup successfully. Compared to the original yaml file I made just 2 changes: removed a bug which was not seen on my installation and added new parameters as published by Lambda earlier this year. However, I could not yet test the new parameters because my Lambda is still on a 2021 SW-level which does not provide the new values which were introduced this year.
Cheers....Ralf

Hi Ralph, yes, of course, I use the new yaml file in my own setup successfully. Compared to the original yaml file I made just 2 changes: removed a bug which was not seen on my installation and added new parameters as published by Lambda earlier this year. However, I could not yet test the new parameters because my Lambda is still on a 2021 SW-level which does not provide the new values which were introduced this year.
Cheers....Ralf

Hi Ralf,
thanks for the info. As soon as my heat pump is installed, I will also test.
Greetings Ralph

Thx Ralf, so there will be a config action from lambda, to let me access the OP Mode register, additional I don’t have the „soll-temp“ (set point flow line/ 5005&5105).
What for a user level do you have?
thx

Hello does reading the registers work with the new yaml file ? I'm also interested, because I'm getting my Lambda mounted in January and would like to connect it to HomeAssistant as well. Greetings Ralph

I use my own yaml, but the only difference is the input_type: holding.
I had in the past a similar issue where lambda had to configure something, afterwards it worked.
my EU08L is on:

One more question, belong the readings from register 5006 in your case to the heating circuit mode in lambda gui?

I can’t set via write to register 5006 this to a different mode, in the drop down menu.
Only if I set it in the lambda gui to auto i‘am able to see that the icons switch to the modes corresponding to the values in the register, but the drop down don’t switch.

Hello,
maybe it has to be that Lambda has to change something on the register. In register 5006/5106 the unit is [0.1°C] and not register 5001 {No. as is comparable to the other registers. See Modbus documentation on the website.

Von: thecem @.>
Gesendet: Montag, 23. Oktober 2023 06:38
An: RalfWinter/lambda-heatpump-modbus-tcp-HA @.>
Cc: Subscribed @.***>
Betreff: [RalfWinter/lambda-heatpump-modbus-tcp-HA] Heating circuit operation mode (register 5006) (Issue #8)
I get for register 5006 and 5106 the value -300.0.
Do you have hier the mode number?
thx.
—
Reply to this email directly, view it on GitHubhttps://github.com/RalfWinter/lambda-heatpump-modbus-tcp-HA/issues/8, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7QCQTJENIHI2SLJUTNKOL3YAXYCZAVCNFSM6AAAAAA6LOOB2KVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2TMMZTG4YTCOI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

In between I get the information from lambda, that the actual SW is not supporting the values, like in the old controller software. There will be an update.


[Auth] Port IAuthenticator and adapt SDK to support MSI implementation

As per PR https://github.com/microsoft/botbuilder-dotnet/pull/3255
Azure App Service (and several other resource types) has a Managed Service Identity feature which simplifies management of application identity - see Microsoft.Azure.Services.AppAuthentication for more information.
By having AdalAuthenticator derive from an interface, I can then inject my own MsiAuthenticator that uses AzureServiceTokenProvider to get the token to call the channels.
I can inject my MsiAuthenticator via overriding BuildCredentialsAsync on BotFrameworkHttpAdapter and returning a MsiAppCredentials class that, in turn, returns the MsiAuthenticator

@rggammon are you interested in porting your IAuthenticator change to js? Would love your contribution on this one. @cleemullins and I were delighted with your initiative of proposing this change on C# 👍 If not interested, I'll be happy to port it, up to you.

@carlosscastro There currently isn't an AdalAuthenticator class. The idea here would be to create one, and have it implement IAuthenticator, correct?

Gabo and I had a conversation on this one. Since this IAuthenticator is in the deep auth layer and is not openly documented as an extension point, or particularly valuable as an extension point, we'll leave it as-is. Also this area might undergo refactors soon, in which the extension points will be reconsidered holistically.


bug: amortize jest mocking for AWS Sdk

Currently we to declare the jest callback at the top of every unit test that wants to hijack the aws-sdk calls. Would be much cleaner to amortize the callback to a single file:
Limitations
Jest Mocking happens at initialization. This means every file that we want to mock the architecture will have to have a copy of the below mock code.
NOTE: Callback in jest.mock() cannot be moved outside as the import happens after initialization.
Snippet

 Code Snippet

  const architecture = new MockArchitecture();
  jest.mock('aws-sdk', () => {
    return {
      config: { update: () => undefined },
      CloudFormation: jest.fn(() => {
        return {
          describeStackResources: jest.fn(({ StackName }) =>
            ({ promise: () => architecture.mockGetCfn(StackName) })
          ),
          describeStacks: jest.fn(({StackName}) =>
            ({ promise: () => ({ Stacks: [architecture.mockGetCfn(StackName)] }) })
          ),
          getTemplate: jest.fn(({StackName}) =>
            ({ promise: () => ({ TemplateBody: architecture.mockGetCfn(StackName) }) })
          ),
        };
      }),
      S3: jest.fn(() => {
        return {
          getObject: jest.fn(({Bucket, Key}) =>
            ({ promise: () => ({ Body: architecture.mockGetS3(Bucket, Key) }) })
          ),
        };
      }),
    };
  });

Closed by #23


NotImplementedError: Unsupported ONNX ops of type: Cast,Floor

When using coremltools to convert an ONNX model that had been exported from PyTorch 1.2.
Traceback (most recent call last):
  File "/Users/glennjocher/PycharmProjects/iD/yolov3-onnx_to_coreml.py", line 244, in <module>
    main()
  File "/Users/glennjocher/PycharmProjects/iD/yolov3-onnx_to_coreml.py", line 69, in main
    yolov3_model = convert(model_proto, image_input_names=['0'], preprocessing_args={'image_scale': 1. / 255})
  File "/Users/glennjocher/PycharmProjects/iD/onnx_coreml/converter.py", line 455, in convert
    _check_unsupported_ops(graph.nodes)
  File "/Users/glennjocher/PycharmProjects/iD/onnx_coreml/converter.py", line 143, in _check_unsupported_ops
    ','.join(unsupported_op_types)))
NotImplementedError: Unsupported ONNX ops of type: Cast,Floor

The offending lines seem to be complaining about the Upsample module:
  %335 : Float() = onnx::Cast[to=1](%334), scope: Darknet/Upsample # /Users/glennjocher/anaconda3/envs/yolov3/lib/python3.7/site-packages/torch/nn/functional.py:2466:0
  %336 : Float() = onnx::Floor(%335), scope: Darknet/Upsample # /Users/glennjocher/anaconda3/envs/yolov3/lib/python3.7/site-packages/torch/nn/functional.py:2466:0

but if I look at the upsample code there is nothing to correct:
        elif mdef['type'] == 'upsample':
            modules = nn.Upsample(scale_factor=int(mdef['stride']), mode='nearest')

@glenn-jocher looks like you are not disabling the rank5 mapping
Cast and Floor are implemented in CoreML 3.0 and hence, you need to pass the disable_coreml_rank5_mapping=True to convert method
check here: https://github.com/onnx/onnx-coreml/issues/415#issuecomment-518756496

@glenn-jocher closing as support for Cast and Floor is added
Please feel free to re-open if you need any assistance with the conversion.


Text is unreadable on white backgrounds

For my old locker script using GraphicsMagick I was sampling for the overall average brightness of the images on the screen (by converting an image to a 1x1 hsb colorspace) and taking anything from below 40% to show white text or above 40% to show black text. I even went further to change the lock image to a dark version on white backgrounds.
Currently with your implementation the text is always white.
I'd be interesting to see a implementation of something like this in the library you use (and how the performance is). In GM it takes a second or so.

I am aware of this issue and looking into it.


Feedback icon looks more like a chat box

Feedback icon looks more like a chat box
Change the icon to something more intuitive so that users can easily recognize the feedback icon

fbitem-publisher6fc06c4f-cbc6-408b-9d93-3061dfe015c9
Reported by: @siro1
System: Chrome 55.0.2883 / Windows 10 0.0.0
Referer: https://demo.gbif.org/publisher/6fc06c4f-cbc6-408b-9d93-3061dfe015c9
Window size: width 1536 - height 831
API log
Site log

Hi Siro
I've changed it from bug to an idea as I do not really consider it a bug. Good that you found it anyway.
If you have suggestions for what the icon should be, then please provide it here. I know Burke agree with you and have suggested a bullhorn as attached.

Personally I think it a fine icon and a fairly standard one (see the attached Google search) and do not think that a chatbox/message icon is misleading. On the contrary. But I guess it is a matter of taste and associations.

Mmm, may be try using one with the word "feedback" written in the icon?

@siro1  Perhaps. Remember that it should be possible to translate that. What is feedback called in Swahili?
In danish it would be something along the lines of "Tilbagemelding". In german i believe it is "Rückmeldung".
Point being that it could get very long. But that might also be ok. A bit odd on mobile perhaps. Do you have a suggestion for an icon?

Bullhorn will be suitable I believe. http://getbootstrap.com/components/

Given that we get as much feedback as on the previous site I guess people still know how to provide feedback. If someone have a specific suggestion and an svg and ideally an evidence based argument please reopen.


Detect valijson on the system and use it when requested

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area build

What this PR does / why we need it:
When not building valijson ourselves, it automatically detects where valijson is installed on the system.
Before that, it was necessary to specify the install prefix during configuration.
Which issue(s) this PR fixes:

Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

Welcome @ovalenti! It looks like this is your first PR to falcosecurity/libs 🎉

Hi @ovalenti. Thanks for your PR.
I'm waiting for a falcosecurity member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ovalenti
To complete the pull request process, please assign fededp after the PR has been reviewed.
You can assign the PR to them by writing /assign @fededp in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi! Thanks for this PR!
/ok-to-test

@ovalenti: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs-arm
2c2f326558f28031d9f3a2533521207cc14ecebe
link
true
/test build-libs-arm

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ovalenti: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs-arm
2c2f326558f28031d9f3a2533521207cc14ecebe
link
true
/test build-libs-arm

build-libs
2c2f326558f28031d9f3a2533521207cc14ecebe
link
true
/test build-libs

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Uh, you should update github actions to also install valijson package, given that multiple tests do not use BUNDLED_DEPS !

Uh sorry this is a draft :/ poiana requested my review. Please, prefix the PR title with a "wip: " while this is a draft :)
Thank you!

@FedeDP, do you think that it would be acceptable to install valijson from source during tests using system deps ?
Indeed, valijson doesn't seem to be available as a package in debian:buster.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

266e398 Detect a valijson system install using headers

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I am not sure why CI is not triggering here :/
Trying to close and reopen!
/close

@FedeDP: Closed this PR.

In response to this:

I am not sure why CI is not triggering here :/
Trying to close and reopen!
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/reopen

@FedeDP: Reopened this PR.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

There was a conflict with master, so I rebased :)

It seems that I am hitting this issue: https://github.com/uraimo/run-on-arch-action/issues/54

:/ if we only could get rid of that action and run on bare metal it would be much faster and simpler...

The ci for build-modern-bpf-arm64 is failing with:

/usr/sbin/wget: error while loading shared libraries: libffi.so.7: cannot open shared object file: No such file or directory

The ci for build-modern-bpf-arm64 is failing with:

/usr/sbin/wget: error while loading shared libraries: libffi.so.7: cannot open shared object file: No such file or directory

This exact same workflow on master is able to download the sources through cmake (ExternalProject_Add). It might be using something else than wget...
I could either try to use an other download tool, or fix the archarm_latest image, so that wget works...

You can try installing libffi!

pacman does install libffi as an indirect dependency of wget. However:
[root@25f67fd6f48d /]# wget
/usr/sbin/wget: error while loading shared libraries: libffi.so.7: cannot open shared object file: No such file or directory
[root@25f67fd6f48d /]# pacman -Q -l libffi
...
libffi /usr/lib/libffi.so
libffi /usr/lib/libffi.so.8
libffi /usr/lib/libffi.so.8.1.0
...

ln -s /usr/lib/libffi.so /usr/lib/libffi.so.7 :laughing:
It seems a bug in the image :/ i will retry the build, perhaps they already fixed it!

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ovalenti
Once this PR has been reviewed and has the lgtm label, please assign fededp for approval by writing /assign @fededp in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/test all

@ovalenti: No presubmit jobs available for falcosecurity/libs@master

In response to this:

/test all

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@FedeDP Woohoo ! \o/

LGTM label has been added.  Git tree hash: faaacabb2c32821f348f4392177ffd35249ca445

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, ovalenti
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, Molter73, ovalenti
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,Molter73]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Start markdown

new
pull request
first pull request
I think its working

[x]  added an image
[x] added headers
[x]  added a code snipped
[ ] added a waifu


Unable to use the "Browse" function of MA

What version of Music Assistant has the issue?
2.0.0b1
The problem
Even if i see all my music in Artists, Albums and Tracks, if i go to "Browse" i just see a "Local FileSystem" folder and anything else.
If i click on "Local FileSystem" nothing append and all my music is already sync.
Strange behavior is i allready rename Local FileSystem with Media but it still "Local FileSystem" for the namefolder under "Browse".
How to reproduce
dont know sorry.
I cant delete the HACS version first because it ask to delete the integration first and i cant delete it even with a restart of HA.
So first i stop the AddOn, then i delete all the entities in the integration, then go to HACS and ignore the warning and delete the HACS part. Then restart HA, and delete the integration, now it works. Then restart and start the AddOn MA.
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2023.3.3
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Known issue https://github.com/orgs/music-assistant/projects/2/views/1

Also see here https://github.com/music-assistant/hass-music-assistant/issues/1106

Thanks @OzGav
but for the "Browse" problem?

i delete addOn, restart HA, install again, same problem, no "Browse" function working
any idea please?

https://github.com/orgs/music-assistant/projects/2/views/1?pane=issue&itemId=22570112

thanks!


Will this project be open-source?

Will this project be open-source or is it a closed source company thing?
I'd love to contribute, having EQ working on my Mac is a top priority right now ^^

Currently the plan is to keep the app closed source as I have invested a lot of time into R&D and would like to keep some intellectual property to myself. Maybe if the project is going to be either super unpopular or super popular I will consider open sourcing it.

Hey everyone 👋
I will be open sourcing the new version of eqMac, therefore, I am closing this issue and repository as I will be transfering nodeful/eqMac2 repo to bitgapp/eqMac
Please bear with me for the next couple of days as I setup that new repo. The eqMac2 code will be living in the old-eqMac2 branch if anyone is interested in referencing it for whatever reason.
Please try out the new version and report bugs to the new repo.
Thanks for using eqMac.


Start README as jupyter notebook.

We're currently processing your upload.  This comment will be updated when the results are available.


Add fullscreen toggle button

Add fullscreen toggle button, ideally for all platforms

D12.2 has bug and can't exit fullscreen, implemented via workaround


bookmark fragment fixed

Description (required)
Fixes #4178
What changes did you make and why?
Apparently, the removed bookmarks weren't tracked so I added an ArrayList<Integer> removedItemsin the view pager to track the removed items from the bookmarks. It is updated as the user clicks on the bookmark option on any of the pages.
As soon as the user exits the details view Pager, The root fragments backPressed function is called and therein I've accessed the removed items and removed them one by one, and clears the list finally.
For access to the removed ArrayList, I've created a getter and also a clear function that clears the list.
Tests performed (required)
Tested betaDebug on Nokia 6.1 with API level 29.

@madhurgupta10 @misaochan @neslihanturan
Up for review and suggestions.
Thanks.

@madhurgupta10 @misaochan @neslihanturan
Up for review and suggestions.
Thanks.

Codecov Report

Merging #4185 (c681e88) into master (ca82add) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #4185      +/-   ##
============================================
- Coverage     10.56%   10.55%   -0.02%
  Complexity      476      476
============================================
  Files           343      343
  Lines         12927    12944      +17
  Branches       1015     1020       +5
============================================
  Hits           1366     1366
- Misses        11493    11510      +17
  Partials         68       68

Impacted Files
Coverage Δ
Complexity Δ

...rw/commons/bookmarks/BookmarkListRootFragment.java
9.00% <0.00%> (-0.79%)
1.00 <0.00> (ø)

...ee/nrw/commons/media/MediaDetailPagerFragment.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #4185 (c681e88) into master (ca82add) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #4185      +/-   ##
============================================
- Coverage     10.56%   10.55%   -0.02%
  Complexity      476      476
============================================
  Files           343      343
  Lines         12927    12944      +17
  Branches       1015     1020       +5
============================================
  Hits           1366     1366
- Misses        11493    11510      +17
  Partials         68       68

Impacted Files
Coverage Δ
Complexity Δ

...rw/commons/bookmarks/BookmarkListRootFragment.java
9.00% <0.00%> (-0.79%)
1.00 <0.00> (ø)

...ee/nrw/commons/media/MediaDetailPagerFragment.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

Continue to review full report at Codecov.

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


Cursor not hollow when window starts unfocused

Platform: Linux X11 (i3 specifically)
When launching a new instance of alacritty, the cursor is a solid block despite the window being out of focus (I have no_focus [tiling] set in my i3 config). Focusing the window, then un-focusing it behaves correctly. For reference, the related settings are the defaults of cursor_style: Block and unfocused_hollow_cursor: true.

Thanks for the report! Sounds like a bug.

https://github.com/jwilm/alacritty/pull/1569 resolves this issue on i3. I'd love some testing on macOS/Wayland if possible, but it should be good to go the way it is.


Emulate A-Frame cursor events in Altspace

listen for Altspace cursor events, and fire new events with names used by A-Frame: mouseup, mousedown, mouseenter, mouseleave, and click
added example: test-cursor-events

:+1: Please also update the aframe examples in the gh-pages branch: https://github.com/AltspaceVR/aframe/tree/gh-pages


scrollLeft setter is not working

Hi,
I have this code
  var divWrapper = input.parent('.tags')[0];
  divWrapper.scrollLeft = divWrapper.scrollWidth;
  console.log(divWrapper.scrollWidth);
  console.log(divWrapper.scrollLeft);

but scrollLeft is never updated it keeps old value.

@carlitux it doesn't look like AngularJS is used anywhere in your code. On top of this this is sounds like a general support question that should be asked using a dedicated support channel: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#question

@pkozlowski-opensource if you need more code
/**
 * The search input.
 */
var input = element.find('input').eq(0);

/**
 * Focus the input element.
 */
scope.focus = function() {
  if (!input.length) return;
  input[0].focus();
  input[0].select();
  var divWrapper = input.parent('.tags')[0];
  divWrapper.scrollLeft = divWrapper.scrollWidth;
  console.log(divWrapper.scrollWidth);
  console.log(divWrapper.scrollLeft);
};

This is inside a directive fnLink and focus is called when ui is changed this because use overflow: hidden and input disapear when many items on left... so I need to scroll to far right and keep input visible.
The
divWrapper.scrollLeft = divWrapper.scrollWidth;
works fine on chrome dev tools but when focus is called don't update the scrollLeft attribute.

@carlitux this really sounds like a question for a support channel, please let's move the discussion there.

@pkozlowski-opensource ok, thanks


flake8-logging-format catches more things than ruff

Tested with ruff 0.0.236 and the latest flake8-logging-format. This is an error (G004) with flake8 but passes with ruff:
import logging
log = logging.getLogger(__name__)
msg = "message"
log.warning(f"A {msg}")

When using directly logging.warning, both fail.
Using logging.getLogger is a common idiom when logging messages on large projects, so it would be super useful to catch it as well.

flake8-logging-format only excludes the warnings. and parser. call paths, so log.warning is identified as a logger call. However, ruff only includes logging and *logger call paths, so log.warning is not identified as a logger.
This is probably easy to fix by changing is_logger_candidate: https://github.com/charliermarsh/ruff/blob/3ec46f093604c4fe79f724a5f927b241387a239c/src/ast/helpers.rs#L1009-L1019

Yeah we could extend to anything that includes log, I think.

(It’s also possible that I was wrong to restrict the checks in this way.)

The issue of correctly identifying how loggers are named arises for multiple rules.
Maybe we could define in one place the accepted names for some object to be considered a logger ? (I'm thinking about TRY400 which is only triggered by the logger being called "logging" or "logger")

We do have that centralized helper (is_logger_candidate), which TRY400 uses too. So if we just tweak that, it'll fix this for both rules.

Thanks a lot for the quick fix!


fix: data race in Broker.AsyncProduce

Closes #2673

This fix would help stabilize our production producers. Thanks!

remove whitespace

@prestona sorry for disturbing, can we merge it now?

@lzakharov - np. Merged.


docs: adjust readme file

:tada: This issue has been resolved in version 1.48.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error README example exception £Configuration

Error in the exception example from : https://github.com/lucagrulla/node-tail/blob/master/README.md#configuration
Forget '(' for condition
Expected =>
try {
  new Tail('missingFile.txt')
} catch (ex) {
  console.log(ex)
}

Fixed.


HW #3 (Tests)

This pull request contains all test implementations

I might be wrong, so, I would like to clarify: there is one file with integrational tests, right? What integration is being tested?)
Finally, I wonder how developers come up with the test cases they believe should be covered) I would appreciate it if you shared some of your ideas)


After Build Prozess doesn't end...

Version
3.0.0-rc.3
Reproduction link
https://bitbucket.org/mathe42/vuecli3-issue/
Steps to reproduce
vue create buildfails
select no Items
cd buildfails
vue add apollo
yarn build
Or in Repro:
yarn
yarn build
What is expected?
Build does fails or is successfull but ENDs after that
What is actually happening?
Build doesn't end

Maybe you have build problem because of version mismatch (vue apollo beta-10 and vue cli rc-3) in your package.jsom

I fixt it with install apollo-* directly and remove vue-cli-plugin-apollo from dependencies


[INLONG-1464] Add code CheckStyle rules

Fixes #1464

Codecov Report

Merging #1465 (ccee82f) into master (f2b1660) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1465   +/-   ##
=========================================
  Coverage     12.21%   12.22%
- Complexity     1046     1050    +4
=========================================
  Files           392      392
  Lines         32691    32691
  Branches       5145     5145
=========================================
+ Hits           3994     3995    +1
+ Misses        27933    27931    -2
- Partials        764      765    +1

Impacted Files
Coverage Δ

.../java/org/apache/flume/sink/tubemq/TubemqSink.java
51.42% <0.00%> (-4.00%)
:arrow_down:

.../tubemq/corebase/policies/FlowCtrlRuleHandler.java
34.51% <0.00%> (+0.44%)
:arrow_up:

.../inlong/tubemq/corebase/policies/FlowCtrlItem.java
40.00% <0.00%> (+1.11%)
:arrow_up:

.../producer/qltystats/DefaultBrokerRcvQltyStats.java
45.31% <0.00%> (+1.17%)
:arrow_up:

.../client/producer/qltystats/BrokerStatsItemSet.java
72.09% <0.00%> (+4.65%)
:arrow_up:

Continue to review full report at Codecov.

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