Can't install Mopidy-Touchscreen

I'm trying to install Mopidy Touchscreen but everytime I try it gives me this error:
/usr/local/lib/python2.7/dist-packages/cryptography/hazmat/primitives/constant_time.py:26: CryptographyDeprecationWarning: Support for your Python version is deprecated. The next version of cryptography will remove support. Please upgrade to a 2.7.x release that supports hmac.compare_digest as soon as possible.
  utils.DeprecatedIn23,
/usr/local/lib/python2.7/dist-packages/cryptography/hazmat/bindings/openssl/binding.py:163: CryptographyDeprecationWarning: OpenSSL version 1.0.1 is no longer supported by the OpenSSL project, please upgrade. A future version of cryptography will drop support for it.
  utils.CryptographyDeprecationWarning
Collecting Mopidy-Touchscreen
  Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError("bad handshake: Error([('SSL routines', 'SSL3_GET_SERVER_CERTIFICATE', 'certificate verify failed')],)",),)': /packages/f4/ab/5311965d4b9609322e2baa42ad7d2fc63fa80e8c1199f41f953787920455/Mopidy-Touchscreen-1.0.0.tar.gz
  Retrying (Retry(total=3, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError("bad handshake: Error([('SSL routines', 'SSL3_GET_SERVER_CERTIFICATE', 'certificate verify failed')],)",),)': /packages/f4/ab/5311965d4b9609322e2baa42ad7d2fc63fa80e8c1199f41f953787920455/Mopidy-Touchscreen-1.0.0.tar.gz
  Retrying (Retry(total=2, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError("bad handshake: Error([('SSL routines', 'SSL3_GET_SERVER_CERTIFICATE', 'certificate verify failed')],)",),)': /packages/f4/ab/5311965d4b9609322e2baa42ad7d2fc63fa80e8c1199f41f953787920455/Mopidy-Touchscreen-1.0.0.tar.gz
  Retrying (Retry(total=1, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError("bad handshake: Error([('SSL routines', 'SSL3_GET_SERVER_CERTIFICATE', 'certificate verify failed')],)",),)': /packages/f4/ab/5311965d4b9609322e2baa42ad7d2fc63fa80e8c1199f41f953787920455/Mopidy-Touchscreen-1.0.0.tar.gz
  Retrying (Retry(total=0, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError("bad handshake: Error([('SSL routines', 'SSL3_GET_SERVER_CERTIFICATE', 'certificate verify failed')],)",),)': /packages/f4/ab/5311965d4b9609322e2baa42ad7d2fc63fa80e8c1199f41f953787920455/Mopidy-Touchscreen-1.0.0.tar.gz
Could not install packages due to an EnvironmentError: HTTPSConnectionPool(host='files.pythonhosted.org', port=443): Max retries exceeded with url: /packages/f4/ab/5311965d4b9609322e2baa42ad7d2fc63fa80e8c1199f41f953787920455/Mopidy-Touchscreen-1.0.0.tar.gz (Caused by SSLError(SSLError("bad handshake: Error([('SSL routines', 'SSL3_GET_SERVER_CERTIFICATE', 'certificate verify failed')],)",),))

You are using pip version 18.1, however version 20.0.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

When I try to upgrade pip it gives me the same error. I'm new to this kind of stuff so I hope anyone has an idea how to fix this.

Hi,
as the error sais "certificate verify failed", so you can try running pip  with "--trusted-host files.pythonhosted.org".
See https://pip.pypa.io/en/stable/reference/pip/#general-options:
--trusted-host <hostname>¶
Mark this host or host:port pair as trusted, even though it does not have valid or any HTTPS.

Hi,
as the error sais "certificate verify failed", so you can try running pip with "--trusted-host files.pythonhosted.org".
See https://pip.pypa.io/en/stable/reference/pip/#general-options:
--trusted-host <hostname>¶
Mark this host or host:port pair as trusted, even though it does not have valid or any HTTPS.

Thanks, that works but now I have another problem. After trying to install Mopidy Touchscreen it gives another error with the final sentence:
Unable to run "sdl-config". Please make sure a development version of SDL is installed.
I found https://pypi.org/project/PySDL2/ and I tried to download a dev version of SDL but I can't download it with git. I tried to install that as well but it just says that it is unable to locate package.


Fixing Travis with a new logo.svg

I tested different versions of gulp-imagemin, but there's just something wrong with our old logo.svg. I replaced it with a different file.
Test

Run Gulp Build
Check the logo on the homepage

Review

@jimmynotjim
@anselmbradford
@sebworks

https://media.giphy.com/media/hZj44bR9FVI3K/giphy.gif

:+1:


Text elements do not render correct background colors

Setting a background for text elements does not set the color in the actual character's cell to match the overall background color leading to a patchy appearance, not entirely sure how to go about looking into this @bryphe, as the meat of how revery works aka shaders etc. is very new to me but could have a look if you point in me the right direction

Nice catch @Akin909. Definitely a bug with how we render text.
The shader where we color / render the text is here:
https://github.com/bryphe/revery/blob/c20634b72ab39dea1372aaef7d0b519452773df2/src/UI/FontShader.re#L31
The 'texture' we use to read the font has only a single channel (t.a is a float value from 0.0 -> 1.0). In theory - this code should work OK to make the text transparent (since it's setting the alpha value to 0), but we don't set the appropriate blend modes (which tell the GPU how to combine pixels that are already drawn with pixels that you are sending).

@Akin909 - hope you don't mind - I took a go at this w/ #60.

This should be fixed now, @Akin909 ! Let me know if it's not for you 👍


Fixed colorbox in gallery view

Coverage decreased (-0.05%) to 83.704% when pulling 0b13fe3bb75b26eb1defcea1fd79fae5c9633af5 on gallery_view_colorbox_fix into 325c2e4a13a0b60a2fbd2648bdb6e3410ec949d8 on master.

:+1:


Link from "Start Here" page to whether, using, and maybe other docs?

It's a little sad that the home page's "Learn More" link leads to a "Start Here" page that currently links to those docs only from the sidebar. As more people come across our home page (as opposed to landing initially on our 1.0 blog post or our release notes), I think we'll want more prominent links to the discussions of whether and how to use our annotations. (Really, it might be nice for https://jspecify.dev/ itself to serve the "Start Here" page, since users currently have to follow a couple links just to get to, e.g., the Maven coordinates. I have no concept of how hard that part would be to pull off. Maybe we could at least get various direct links in the section that currently has items like "Standard Annotations?")

Please do edit the start-here page!


Provide mobile-friendly rendition of the "compile" pane for users with unusual network topology

 Issue by Katharine
Saturday Feb 15, 2014 at 17:14 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT
Originally opened as https://github.com/pebble/cloudpebble/issues/23

Some people are unable to connect their computer directly to their phone as a result of their network topology or other restrictions. We could just put the QR code back (it's actually still in there; it's just hidden…), but that still doesn't help iOS developers due to the restrictions on side-loading JavaScript.
Instead, we could provide the option to log in from a mobile device and get a mobile friendly layout that allows them to install/log/screenshot on 'localhost', as seen from the app. This does work now, but is difficult to actually use in practice.
Note that some users of potentially questionable sanity actually write code on their phones, so we have to handle that use case too.

 Comment by Katharine
Saturday Feb 15, 2014 at 17:30 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

Also note that this still won't work if the phone is on a mobile network instead of a wireless network. It would be nice if the Pebble app could still listen to localhost when there was no network connection, but doing that without causing (even more) user confusion would require some UI work.

 Comment by sarfata
Sunday Feb 16, 2014 at 00:46 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

That is a pretty awesome idea. The only sucky thing is that the developer connection will die after a few minutes if the iOS app is not in the foreground. We have not found a way around that yet.

 Comment by KeanWong
Sunday Feb 16, 2014 at 00:49 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

@Katharine - possible solution would be to send a push message to wake up the Pebble app on the device (and/or remind the user to foreground it) if the issue is that the iOS app is not currently foregrounded but needs to me

 Comment by Katharine
Sunday Feb 16, 2014 at 00:47 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

@sarfata Worst case, I can hack around it by asking you to switch to the Pebble app if it fails – when we're routing through local loopback the set of failure modes is fairly tiny.

 Comment by sarfata
Sunday Feb 16, 2014 at 01:14 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

Yeah. This reminded me that it would be great to have Pebble-SSO support: #26

 Comment by Katharine
Sunday Feb 16, 2014 at 01:00 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

@KeanWong If we can actually do that, it'd be great! Even if this doesn't actually bring the app back to life, it'd be a handy UX improvement.
I think actually getting a push from a CloudPebble project to the correct user's phone app would require some other integration work first, though.


Cleanup Batching Implementation

The batching implementation could be cleaned up in some areas.

[ ] Remove the tight while loop

[ ] Remove the Checkstyle suppression once it is done

[ ] Move where the response is set to retrieved
[x] Remove unnecessary portions in the pom.xml
[x] Add format templates for commonly used templates
[x] Fix ArrayIndexOutOfBoundsException when splitting strings

Examples

https://github.com/Azure/azure-sdk-for-java/pull/5734/files#r333257305
https://github.com/Azure/azure-sdk-for-java/pull/5734/files#r333260091

[x] Update comments that say URI
[x] Remove duplicate HttpPipeline.getPolicy() call
[x] Use default AtomicInteger constructor
[x] Use Mono.fromRunnable to make request construction asynchronous
[x] Cleanup links using en-us in the URL

Given the current implementation it isn't possible to remove the while loop.


mass rename of Intermediate challenges directory

Cleaned up the same as the Hard dir.

hm. I'm relatively new to forking and pulling, and might have messed up this pull request..
I meant to keep those other commits separate.


GitHub Actions Update

I changed cryptol-remote-api to run as a service in the CI so we don't have to use the huge dev image.
I left saw-remote-api as is for now since the saw-client python package in pip is old. Building the correct package and putting it in the right place doesn't seem like the best idea, but we could build a smaller container than the dev image if we wanted to have the saw jobs look similar to the cryptol ones.

Thanks for the polish here.


NETOBSERV-649 fix incorrect metrics on "common" filters

The issue is described here: https://issues.redhat.com/browse/NETOBSERV-649?focusedCommentId=21091465&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-21091465
The solution implemented is, for common filters (eg. source=X OR dest=X), to include a "not" match in the dest query: source=X OR (source!=X AND dest=X) to eliminate the overlap.
It was needed to add this NOT operator on backend side (so half the way done for NETOBSERV-387 which is about implementing NOT)
Also, the matrix-merger algorithm had to be updated so that values are added when the same metric is returned by several queries. We can do that precisely because we can now assume they don't overlap.

@memodi this might also affect your mocks, I'm not 100% sure. The way frontend calls backend for "common" filters is a little bit changed, as well as the backend query to loki

/hold
not done yet :/

@memodi this might also affect your mocks, I'm not 100% sure.

thanks for the heads up!

closing, as explained in https://issues.redhat.com//browse/NETOBSERV-649 comments: the issue needs this other PR to be reproduced / verified, so I'm moving this change into that PR


Fails to expand Windows paths with brackets

> const p2 = 'C:\\Users\\foo\\Workspace\\project\\routes\\[locale]'
> braces.expand(p2)
Error: no parsers registered for: "]"
    at parse (C:\Users\foo\Workspace\project\node_modules\snapdragon\lib\parser.js:475:15)

This is correct, you escaped the first brace, \\[ so the second one is a parsing error. Backticks are used for escaping in glob patterns. All path separators must be forward slashes. This isn't our rule, it's the rule for all globbing libraries.

Those aren't escaped brackets, they're escaped backslashes for a windows path.

This isn't our rule, it's the rule for all globbing libraries.

If that is true, perhaps this bug belongs to chokidar, we'll see what they say on their ticket.

To be clear, chokidar is broken on Windows due to this libraries inability to recognize this condition.

To be clear, \\[foo] is an invalid bracket expression.  This topic has been exhaustively discussed on many issues on several projects. If chokidar is creating the glob pattern, then that is a bug in chokidar, since glob patterns should only use backslashes for escaping a value. Given that \\ in a string is converted to \ in a regex, it is actually escaping [, not the backslash. I would be happy to discuss and help with creating a solution on chokidar. But it is not a bug in this library.

If that is true, perhaps this bug belongs to chokidar, we'll see what they say on their ticket.

I'd be happy to help there.

Yep, ok sorry, i had deleted that line in my comment, I'm not fully awake yet.

Yep, ok sorry, i had deleted that line in my comment, I'm not fully awake yet.

no that's ok, I didn't mean to sound harsh. it happens

Definitely looks like this is a chokidar bug: https://github.com/paulmillr/chokidar/issues/699#issuecomment-437293197


[FEATURE] - Windows Device Configuration Export for ADMX templates

Would love to see exports available for windows device configurations with ADMX templates.

I have a new version up on testpypi that includes backup of Group Policy Configurations. You can install it by running,
pip install -i https://test.pypi.org/simple/ IntuneCD-beta
You can then run a backup like this,
intunecdbeta-startbackup
Test this version and verify that you can now see the configurations.


Added default headers that other servers have.

Thanks for your help. I want to make sure that for the high performance frameworks we are compare  apple to apple. The these Java EE based platforms, these headers won't make any difference.


Dead link on svg4everybody polyfill in examples

Hi, thank you for great tool. I found a dead link on polyfill svg4everybody in templates for symbol and defs examples. This link is not working now. You can use this CDN or another.
Ps. Also, you forget call the script.
<script>svg4everybody();</script>

Thanks for pointing this out. Should be fixed with the next release of svg-sprite. :+1:


Add security hardening functionality

What version (commit) do you encounter this issue on?
master
What type of issue is this (bug, feature, documentation, regression)?
feature
What is the expected behavior?
Liberty now includes the ability to use openstack-ansible-security, this is a role to apply security best practices host configurations. RPCO needs updating to be able to selectively deploy this capability.
What is actually happening?
Currently the openstack-ansible submodule does not include to relevant commit and deploy.sh has no ability to control the use of this role.
How can we reproduce this issue?
How can we verify your issue is addressed?
Once updated the role will apply the security best practices it defines, this will be observed in the logs and config.
If this is a large issue, what can be done immediately to alleviate the issue while working on a longer-term solution?
What is your evaluation on the importance of this issue, and why?
High, this is a requirement for 12.2.0
What branches / milestones do you believe this issue should target?
The list below is what is currently supported for backports
Remove branches (lines) you do not wish to target.

[x] Master

merged


fix javaVersion extraction bug

fixes: #269
Signed-off-by: Yixin Qian<EMAIL_ADDRESS>

ok... the bug here is quite tricky... There are two kinds of java version message: one ends with JCL and one ends with Server VM. JCL comes first in the regex expression. The bug is that sometimes we are supposed to use Server VM extraction but we are actually using JCL extraction.
Example console output: https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_extended.system_aarch64_linux/230/consoleFull
10:31:26  openjdk version "1.8.0_262"
10:31:26  OpenJDK Runtime Environment (AdoptOpenJDK)(build 1.8.0_262-202006091656-b05)
10:31:26  OpenJDK 64-Bit Server VM (AdoptOpenJDK)(build 25.262-b05, mixed mode)
10:31:26  get testKitGen...
......
.......
10:32:01  (ant -f scripts/build_test.xml -DTEST_ROOT=/home/jenkins/workspace/Test_openjdk8_hs_extended.system_aarch64_linux/openjdk-tests/TKG/.. -DBUILD_ROOT=/home/jenkins/workspace/Test_openjdk8_hs_extended.system_aarch64_linux/openjdk-tests/TKG/../../jvmtest -DJDK_VERSION=8 -DJDK_IMPL=hotspot -DJCL_VERSION=latest -DBUILD_LIST=system -DRESOURCES_DIR=/home/jenkins/workspace/Test_openjdk8_hs_extended.system_aarch64_linux/openjdk-tests/TKG/../../jvmtest/TestConfig/resources -DSPEC=linux_aarch64_cmprssptrs -
here we just want to extract
10:31:26  openjdk version "1.8.0_262"
10:31:26  OpenJDK Runtime Environment (AdoptOpenJDK)(build 1.8.0_262-202006091656-b05)
10:31:26  OpenJDK 64-Bit Server VM (AdoptOpenJDK)(build 25.262-b05, mixed mode)

but unexpectedly end up extracting the whole bunch of stuff to include DJCL_VERSION
So I add a \s to indicate I only want this style:
10:51:33  openjdk version "1.8.0_262"
10:51:33  OpenJDK Runtime Environment (build 1.8.0_262-202006101710-b06)
10:51:33  Eclipse OpenJ9 VM (build master-2d4adaf1b, JRE 1.8.0 Windows Server 2012 R2 x86-32-Bit 20200610_587 (JIT enabled, AOT enabled)
10:51:33  OpenJ9   - 2d4adaf1b
10:51:33  OMR      - c690e9375
10:51:33  JCL      - def44e393b based on jdk8u262-b06)

to differ from DJCL_VERSION
java version is set correctly when reloading the database locally.
This might not be a decent solution, appreciate any suggestion

@smlambert @llxia

https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_extended.system_s390x_linux/214/consoleFull
I just found there exists a third JDK extraction type: Zero VM
do we have other JDK type? I will add them all to this PR. FYI @llxia

That is all of the variants we will see for now (Zero VM is from OpenJDK, for ease of platform ports, and out of interest sake, does not have a JIT atm, so expect most test execution time to be slower).

Talked with @OscarQQ , we will merge this PR so the users can use the create new issue feature. We will deliver an enhanced version in another PR - wrap java -version info with a specific message, so the regex can easily find java -version.

The server is updated. Please note, we only parse the output once, so the existing data in database will not be updated. The future build will have the correct java version info.


Merge development branch into the master branch

This PR contains project restructuring and installation changes from @lsetiawan and @uwcdc. The project now follows the Scientific Python standards laid out by Scientific Python. In addition, the following dependencies were updated:

neuraltoolkit
smart-open

Due to the fact that the dependence on neuraltoolkit has not been used in over a year, the pip install neuraltoolkit (v0.3.1) was substituted, and the project built without any error.
In addition, the dependence on smart-open diverged from the master repo, so the original fork has been maintained internally by the Braingeneers. Consequently, the fork was migrated to here.

@uwcdc There are a lot of files in this PR coming from docs/build this folder should be removed from git and ignored. Usually the built directory from sphinx doesn't go into the source code repository as it creates bloat. It should be generated on the fly by hosting services such as readthedocs.


Compile error with optimization level 'Fast [-O]'

I get a compile error for this line of code
    let tomorrow = today.add([.Day: 1])

where today is an NSDate object, but only when the optimization level is set to fast (which is the default for release builds).
SWIFT_OPTIMIZATION_LEVEL = -O

By the way, this is working:
    let tomorrow = today.add(days: 1)

Weird! Needs investigation. Thx for the issue.

It's okay with SwiftDate4.
The new function which accomplish your request is:
let tomorrow = try! Date().add(components: [.day : 1])


Return query if no match?

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[x] Mac OS X
[x] Windows
[x] Etc.

Shell

[x] bash
[x] zsh
[x] fish

Problem / Steps to reproduce
Is it possible to the query string if the search query wasn't found?
Something like --exit-0 but instead of exitting, return the search query...
If it's not an available flag option, can it be done with --bind and actions?
Thanks.

You can use --print-query option.

curious how to make this change in shell key bindings eg in
source "/opt/homebrew/opt/fzf/shell/key-bindings.zsh"

thanks

curious how to make this change in shell key bindings eg in

I assume you want to print the query when no matching history entry is found. Try the following[^1]:
# You can make CTRL-R paste the current query when there's no match
export FZF_CTRL_R_OPTS='--bind enter:accept-or-print-query'

[^1]: fzf/CHANGELOG.md 0.45.0


ORC-207: [C++] Enable users the ability to provide their own thirdparty libraries

A couple of things:

I'd prefer if the user had to explicitly set the environment to get a non-default version. That is particularly true for protobuf, which is well known to not be API compatible between versions.
I don't think it is necessary to have travis-ci do the build with the environment variables set.

I did not understand the first point. PROTOBUF_HOME must be explicitly set to use the user specified version. Otherwise, the library gets downloaded.
Will remove the travis-ci test.
Thanks!

It looked like you were searching for the protobuf headers and libraries. Is that not the case? Why do we need all of the FindX cmake files for a simple test of a variable?

In this PR, FindProtobuf.cmake checks if the specified path PROTOBUF_HOME contains the required headers and sets PROTOBUF_INCLUDE_DIRS, checks for the library and sets PROTOBUF_LIBRARIES, and checks for the executable and sets PROTOBUF_EXECUTABLE. Even if one of them is missing from the user specified location, the protobuf library gets downloaded. In a future patch, we can extend the FindX.cmake to check default system paths (except for protobuf). But by default, we should only download.

Ok

I find myself in need of this PR. Looks like this has a few merge conflicts, but other than that is this ready to merge?

@jcrist  Yes, this should be good to merge with rebasing. I will try to merge this in by today.

Thanks @majetideepak. If you also have a chance, it would be nice to get #185 reviewed and merged too. Waiting on both of these so I can use orc in a larger cmake project.

@jcrist  Will review that PR. Thanks!


Package Scope
@suspensive/react-query
Description
Add <HydrationBoundary/> in suspensive-react-query-4
Possible Solution
No response
etc.
No response

related: #1251

I will reopen the issue if the need.


Run in dev environment

Q
A

Bug?
yes

New Feature?
no

Sulu Version
latest

Browser Version
-

Actual Behavior
When i launch the server using :
php bin/console server:start

the default environment is prod, wich is not very convenient for development.
WebServerBundle is using app.php as entrypoint, because there is no app_dev.php.
Expected Behavior
Builtin server must use dev environment
Steps to Reproduce
composer create-project sulu/sulu-minimal my-project -n
bin/adminconsole sulu:build dev
php bin/console server:start

Possible Solutions
create a app_dev.php, like symfony distribution :
<?php

define('SYMFONY_ENV', 'dev');
require_once __DIR__ . '/app.php';

We've already been discussing this. We don't know exactly which solution we will take yet. We have not discussed yet if we only want that behavior for the php internal webserver, or if dev should be the general default.
In the mean time you can use SYMFONY_ENV=dev bin/console server:start to start in dev mode.

We've already been discussing this. We don't know exactly which solution we will take yet. We have not discussed yet if we only want that behavior for the php internal webserver, or if dev should be the general default.
In the mean time you can use SYMFONY_ENV=dev bin/console server:start to start in dev mode.
/cc @trickreich @chirimoya

We've already been discussing this

Sorry, i did'nt find it during my search
As a Symfony developer, it is to me the expected behavior for the php internal webserver, not for general default.

The issue just came up this week and we discussed it in person, not on github :-)


Provide integration with gulp

https://github.com/raphamorim/native-css/issues/1

u also think of gulp+webpack?
maybe add an loader for it? so you can combine it with stylus/sass loader to create your styleObject onimport? THAT would be awesome!
great work by the way... i also think css should stay in css - but now it can be used in js too!


Added ComplexDiagnostic

Hi everybody.
Currently there is no way to show complex diagnostics.
Rust have very useful and detailed messages.
For code:
fn main() {
    let mut x = &5;
    {
        let y = 5;
        x = &y;
    }
}

It gives very good explanation:
error: `y` does not live long enough
 --> main.rs:6:5
  |
5 |         x = &y;
  |              - borrow occurs here
6 |     }
  |     ^ `y` dropped here while still borrowed
7 | }
  | - borrowed value needs to live until here

I am using VSCode.
Rust Language Server (RLS) is currently being developed.
RLS can send all of aforementioned messages, but there will be problems when multiple messages have same range.
I'd like to have a way to show only primary message and if a user hover on the message to show all secondary messages.
Currently there is no way to show such kind of messages in VSCode.

Hi @KalitaAlexey, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@jonathandturner,
I have created an issue allowing RLS to show beautiful messages.
What do you think about it?

slaveDiagnostic? In 2017?..
Anyway, why the new type and not optional field in the original message?

@wiz,
What's wrong with slaveDiagnostic?
The new type because there is no such type.
Should I add ? to make a field optional?

@wiz,
Do you link slaveDiagnostic with slavery? Is secondary better?

The new type because there is no such type.

https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#diagnostic
There's message already, but now notion of how editor should behave with respect to multiline string.
If you've tried sending some \ns there and it doesn't work, a new details? field could be added.

@wiz,
Look at the provided information.
Diagnostics are on different lines.

Currently there is no way to show such kind of messages in VSCode.

What does it show instead?

Hi, I am closing and re-opening this PR to bump the CLA bot. Sorry for the inconvenience!

Hi @KalitaAlexey, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

@wiz

Currently there is no way to show such kind of messages in VSCode.

What does it show instead?

It show each message separately. I described when it is a problem:
...there will be problems when multiple messages have same range.
fn main() {
    let mut x1 = &5;
    let mut x2 = &5;
    {
        let y1 = 5;
        x1 = &y1;

        let y2 = 5;
        x2 = &y2;
    }
}

Output:
error: `y1` does not live long enough
  --> main.rs:10:5
   |
6  |         x1 = &y1;
   |               -- borrow occurs here
...
10 |     }
   |     ^ `y1` dropped here while still borrowed
11 | }
   | - borrowed value needs to live until here

error: `y2` does not live long enough
  --> main.rs:10:5
   |
9  |         x2 = &y2;
   |               -- borrow occurs here
10 |     }
   |     ^ `y2` dropped here while still borrowed
11 | }
   | - borrowed value needs to live until here

It show each message separately

You'd better start posting screenshots :smile: I understand RLS messages, but fail to imagine the result in the editor itself...

@wiz,
Okay. I will attach screenshots.

I have added a screenshot showing the problem.

@nrc,
Hi.
I am inviting you here to hear your opinion.

I have added a screenshot showing the problem.

Still can't see what message does the editor currently show on hover or anywhere else...

@wiz,
I'll add more screenshots.
I didn't think that it is needed.

I see the need for more complex diagnostics given the analysis modern languages perform. However using commands for this is IMO the wrong solution. VS Code for example has already support for showing inline error message (as other editors have).

In addition something like a complex diagnostic message must be handled by a problem view as well. This means that publishing diagnostic should support such complex diagnostics and the editor / client should handle them in an appropriate and consistent way in its user interface.

Rather then introducing a new type, couldn't a diagnostic have an optional reference to a master or causeother diagnostic?

@mickaelistria,
Why?

From our point of view (implementing the RLS) we would rather experiment by adding custom fields to the Diagnostic interface to begin with. I don't think we need to make official changes to the protocol to  make this work (aiui, VSCode and other editors should ignore unexpected fields, not error out). Once we've found what works, we'll report back. I think making any changes at this stage would be premature.
For reference, our initial plan is adding label: string and children: ChildMessage where
interface ChildMessage {
    range: Range,
    label: string,
}

I expect that the new label field is a bit Rust specific, so an alternative plan is to add:
interface LabelledRange {
    range: Range,
    label: string
}

Then change range: Range in Diagnostic to range: Range | LabelledRange and add childRanges: [Range| LabelledRange], or something like that. As I say, we're experimenting :-)

Just to follow-up - since we want to keep the structures compatible, we could do something like this:
interface LabelledRange  extends Range {
  label: string
}

This way, structurally we stay compatible with the shipping version of the error message object, but we can add a few additional fields that would be seamlessly ignored by LSP compatible plugins.
Actually, because we're still so early in the thinking here, it makes sense to close this issue until we've had time to create and test out a real proposal.

@jonathandturner,
Okay. I will close it for now.

I like the proposal very much to get something working and then to spec the LSP. If you do the prototyping in VS Code please consider the problems view and the F8 feature (Goto next Error or Warning) as well.

I am not going to open this PR since probably something will change.
Instead of it, I would propose to allow the client to notify about hovering:

text/willShowHover with the range
text/didHideHover with the range

This way our language server would be able to show code lenses on starting hovering and to remove code lenses on stopping hovering.


Night mode won't stick on this one VPS

SYSTEM INFORMATION

OS type and version
Ubuntu Linux 22.04.2

Webmin version
2.021

Authentic Theme version
20.21

Processor information
AMD EPYC 7282 16-Core Processor, 4 cores

I have this one VPS instance where night mode would not ‘survive’ once we navigate or refresh pages. Additionaly the graphics on System Information show blurred and without colors in dark mode.
Toggling night/day mode works inconsistently (don’t always toggles) both with shortcut or icons and navigating to any tab or page will turn night mode off but leave the blurred graphics behind. If we refresh the page the graphics would come back normal (in day mode only).
I have 3 other VPS instances working fine. I know it probably means nothing but the only difference is that the other VPSs are all Intel Xeon VMs.
This is how it looks when "normal"

Then if toggling to night mode...

Toggling back...

If I refresh the page anytime it goes day mode with the normal (sharp and colored) graphics.
Thenak you

Hello, Julio!
If you open browser's console and toggle day/night mode -- do you see any errors? Also, if you try another browser (or incognito tab), with all extensions disabled -- does it change anything? Or it just works fine in the same browser but for other servers?

It works just fine for the other sites on the same browser.
It behaves the same on edge or on Chrome with incognito window

Excellent! Could you now open browser console and see what errors do you get on switching dark/light modes?

panel.demimot.com-1686650757375.log
panel.demimot.com-1686650781474.log
panel.demimot.com-1686650792118.log
panel.demimot.com-1686650801679.log

Is this instance in Cloudflare or behind other proxy? If so, could you tell more about DNS and proxy configuration for the instance that's having an issue?

Please follow this guide to setup Apache gateway for Webmin:
https://webmin.com/faq/#can-i-run-webmin-or-usermin-behind-reverse-proxy

It gets me a "502 bad gateway" error.

This is not the theme issue!

I adjusted certificate paths, server name, etc. to suit my setup but all I get is a "502 bad gateway" error.

This error indicates that Apache proxy did not receive a valid response from the Webmin backend server.


Create an art for the play store

A simple design for the front page in the play store.

I'll do my suggestion for it!

Ok. You can edit the existing art too.


Vertical "spacers/joins" in Specializations, Force Powers and Signature Abilities are too small/thin

[v1.3] Request
Currently, the vertical spacers in Specializations, Force Powers and Signature Abilities are both hard to click/toggle (in edit mode) and hard to tell apart when the various layouts are fully filled in, to the point where a couple of players with sight difficulties literally cannot tell which talents/upgrades are connected and which aren't.
If those were stretched a bit more vertically, that would make the connections much more readily identified (and would make them much easier to click in the edit phase, too).
Thanks!

I've widened them a bit, but as you'll see there just isn't enough room on the specializations,  Users were adamant at the very beginning that specializations be display without scrolling.  I've also widened the clickable area on the connectors.

will be in alpha-8


Added Lemmy.Net DI Client

Added a reference to my Lemmy DI client for dotnet, it's a few weeks old and already got a few ratings on Nuget and some stars.

Thanks, looks good!


update the prow resource limits

update the prow resource limits
Signed-off-by: Harshad Reddy Nalla<EMAIL_ADDRESS>Related-to: https://github.com/thoth-station/thoth-application/issues/1332

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign pacospace after the PR has been reviewed.
You can assign the PR to them by writing /assign @pacospace 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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: tumido
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 [tumido]

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


Fix service latency test.

More accurate latency measurement.
More lenient/sensible pass requirements.

Fixes #10248.
My cluster is passing with 15s/16s median/99p. It seems like that's much slower than it used to be.

More review to come, but I got interrupted.

@lavalamp how big is your cluster?

@timothysc 4 nodes

/cc @rrati
Our latencies are much higher on 25 node HA cluster.

I added constants and a few more comments. PTAL

@k8s-bot test this please

@lavalamp Have you tried the test on a cluster larger than 4? We've gotten the test to pass on a cluster of 8 nodes, but it fails on 22 nodes.  It seems like either kube isn't scaling properly for this test, or the variances in this test don't scale to larger clusters.

@rrati do you have the output of those runs handy? Can you send it to me or post it here?
If this latency increases with cluster size, that's actually a bug that needs to be fixed.

@lavalamp Tim posted our results in #10348.  Is there other data you would want?
That data is 4 days old.  I can run the test again on our 22 node cluster if you want recent/more data.

@rrati Actually that would be helpful if you could run it with this PR patched in, as I'm not sure if the version of this test currently committed measures the time correctly. Also, how much CPU do you give to controller-manager?

INFO: 50 %ile: 1m28.003322392s
INFO: 90 %ile: 1m35.316141426s
INFO: 99 %ile: 1m44.917936666s
INFO: Total sample count: 1100
Median latency should be less than 30s
Tail (99 percentile) latency should be less than 1m0s
50, 90, 99 percentiles: 1m28.003322392s 1m35.316141426s 1m44.917936666s
This is on our 22 node cluster.  The controller-manager is running on a 40-core box.

Other than relatively minor nit, LGTM.  It sounds like this test is actually going to fail on non-trivial sized clusters, but that's good if it confirms a system problem and provides good data to build a fix.

@thockin for a second TL LGTM.  This is purely e2e test code, and very useful.  Worth merging for v1.0, IMO.

Awesome, thanks.
I think the problem is rate limiting in the e2e test's client. I will update this shortly with a fix.

@rrati I took out the parts that were scaling the workload by the size of the cluster. I think this should fix its behavior in your cluster.

@lavalamp Here's the results with the latest changes:
INFO: 50 %ile: 2.230045533s
INFO: 90 %ile: 2.630304389s
INFO: 99 %ile: 2.831806407s
INFO: Total sample count: 200
I ran this multiple times and it always passed with the "worst" results being:
INFO: 50 %ile: 2.260581261s
INFO: 90 %ile: 2.580007133s
INFO: 99 %ile: 3.028985929s
INFO: Total sample count: 200
Removing the scaling to the cluster scaling makes the test run much faster and seems to pass, but is it sacrificing the validity of the test?  I don't know what the requirement is that spawned this test but is there, or should there be, a cluster size component?

@rrati Thanks, your cluster is better than mine :)
We haven't defined our target here. We'll probably want to eventually define a "loaded" state and run this test when the cluster is in that state.

Shippable is this flake, restarting:
F0629 22:50:56.456209 33315 integration.go:803] FAILED: unexpected endpoints: timed out waiting for the condition

LGTM

Since this PR is ok-to-merge and this test is constantly failing on jenkin I'm merging it now.

@wojtek-t: Thanks. I saw this in queue earlier, then totally forgot about when I pinged the other bug.


Accept multiple committees

As title.
e.g., "TC VCARD, TC CALENDAR" rather than currently "TC VCARD, CALENDAR" when I put "VCARD, CALENDAR" in the :technical-committee: document attribute.

Isn't this going to apply to all Metanorma documents? If so, it will mean a grammar change.

Yes please. We should support multiple authorships, whether individuals or committees or organizations.

But doesn’t already the Relaton model support this?

No, this is only in standards-specific extensions of Relaton, and it has only ever licensed one committee. I am allowing more than one for CSD; you need to tell me if more than one applies to other standards as well, including ISO and GB.

.... Don't use split on the technical-committee document attribute, because you have technical-committee-type as well. The solution should instead be the same as Asciidoctor does for multiple authors: technical-committee, technical-committee-type, technical-committee2, technical-committee-type2, technical-committee3, technical-committee-type3...

I think we should allow more than one committee (i.e. contributor = author) for the standards. The solution would be fine. This aligns us with AsciiRFC too where multiple authors are supported.

Implemented for CSD. Will roll out for other standards, other than ISO, GB.

Thanks @opoudjis !

Done in all gems, including ISO.


Broken since PSD2

After PSD2 has been enabled on September 12 the library no longer works although strong authentication has been completed once in the 90-day timeframe. Can you please adjust the code so it works again?
Errors look like this (netbank):
FinTS::DialogError:
       {"9050"=>"Die Nachricht enthält Fehler.", "9075"=>"Starke Kundenauthentifizierung notwendig.", "9800"=>"Dialog abgebrochen", "9340"=>"Ungültige Auftragsnachricht?: Ungültige Signatur."}

Renault Bank:
FinTS::DialogError:
       {"9050"=>"Die Nachricht enthält Fehler. (TRE)", "9075"=>"Starke Kundenauthentifizierung notwendig. (TRE)", "9800"=>"Dialog abgebrochen (TRE)", "9340"=>"Auftrag abgelehnt. (TRE)"}

Hi Alexander, we’re also seeing this issue but right now we don’t have any capacity to fix this as it seems to require bigger changes to the library.
Our de-facto upstream python experiences issues as well: https://github.com/raphaelm/python-fints/issues/72. Once a fix has been implemented there we might port it to this library.

@mkilling From the changes in the python-fints repo it seems that they basically catch the 9075 response (strong auth needed) and then start a TAN process. Is there any code to do any TAN processing in ruby_fints?

Hi @coezbek not right now, the ruby_fints codebase is basically a translation of python-fints from July 2017. It should be pretty straightforward to port the python-fints changes for PSD2, but I unfortunately don’t have time to work on that right now. It would be amazing if you or anyone could provide a pull request - I’ll be happy to review + merge it.


feat: MSSQL - Expose basic properties

Checklist:

[x] Have you added Release Notes in the docs repositories?
[x] Have you followed the Conventional Commits specification?

Thanks @zucchinidev for catching several issues and inconsistencies. :)


Support for axios 0.27.x

I'm working on a codebase that is pinned on axios 0.27.2.
The current peer dependency specified is ^0.25.0 -- this only covers 0.25.0 (check here) and not 0.26.x or 0.27.x because only patch updates are allowed for ^0.x caret ranges.
Can we extend support to 0.27.x? Thinking a range like >=0.25 <=0.27 should work.
Thanks!

Hi @dvrylc! Thanks for the detailed bug report.
Support for<EMAIL_ADDRESS>was added in<EMAIL_ADDRESS>https://github.com/anttiviljami/openapi-client-axios/commit/549bf9b8a9d5dbb692db45f6e521ea2912843e8f#diff-02b96681c77e402830334332f95b59e318db55981ef8cbf00bea768fac0a0bc2R42


not installing without --force flag with react ^17.0.0

if there is no reason to not support react 17+ i suggest adding react ^17.0.0 to peer dependencies and there is ^17.0.1
I found many other libs to have ^17.0.0 and this is the only which has ^17.0.1
made PR for this, if agreed

We really need this update. There are warnings about incorrect peer deps.

I am also having this same issue, I'm using react-virtualized for electron and electron-builder npm installs all the packages and keeps failing. Not to mention that I can't add any arguments (like --force or --legacy-peer-deps). The temporary fix I have done for now is to manually change the peer deps on my own

ping @bvaughn 😃

npm install --legacy-peer-deps works as an alternative to --force

Good lordy I see that this was added to the package.json on the github repo:
"peerDependencies": {
    "react": "^15.3.0 || ^16.0.0-alpha || ^17.0.1",
    "react-dom": "^15.3.0 || ^16.0.0-alpha || ^17.0.1"
  },

However this is not yet published to npm.  It is causing a disastrous monstrosity to occur when trying to install. Please publish this up to npm so I don't have to hack my install with --legacy-peer-deps!

@bvaughn please publish!  We're being blocked by this on Firebase cloud functions.

I haven't used this package in years. I don't have the mental bandwidth the maintain it. If someone else (preferably someone who actively uses it) wants to step up and publish this release (and deal with any potential breaks it causes, since there are several changes committed since the last release) then please email me.

Version 9.22.4 has been published with peer dependencies updated to include React 17 and 18. (No other changes from 9.22.3)
https://github.com/bvaughn/react-virtualized/releases/tag/v9.22.4


[GUI-164] Change setup progress appearance.

Description
Now: progress bar is divided by 3 section. Each section represents a set up step.
Then: unify it and make a single line, not tabs.
Related JIRA's task: https://mongoose-issues.atlassian.net/projects/GUI/issues/GUI-164

This one will be reimplemented since a lot of the functionality has changed. Closing the PR for now, leave the JIRA task open.


Allow null in optional payload_data fields

An event now fails validation when it's sent with uid: null. We have seen this happen with events from the Android tracker, perhaps other trackers allow setUserId(null) too.
Related to:

https://github.com/snowplow/iglu/issues/231
https://github.com/snowplow/snowplow-android-tracker/issues/250

No, this isn't how payload_data is conceived - payload_data is based on the concept of a GET querystring of name-value pairs - where a value is either there as a string (even an empty string), or the whole name-value is not set. There is no concept of a null value in a GET querystring, so there is no concept of null in payload_data.
This is a bug in the Android Tracker, closing.


Error in running in profile mode iOS

Steps to Reproduce
Ever since upgrading Flutter, when I try to run the app in profile mode, the app quit after Xcode Build Done. I have included the last few lines of the logs when running in verbose mode

Logs

[        ] Application launched on the device. Waiting for observatory port.
[+30017 ms] Installing and launching... (completed in 53.0s)
[   +1 ms] Error running application on TapPhone.
[   +4 ms] "flutter run" took 622,320ms.
[   +8 ms]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
           #1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:617:7)
           <asynchronous suspension>
           #2      FlutterCommand.verifyThenRunCommand
           (package:flutter_tools/src/runner/flutter_command.dart:1080:12)
           <asynchronous suspension>
           #3      FlutterCommand.run.<anonymous closure>
           (package:flutter_tools/src/runner/flutter_command.dart:931:27)
           <asynchronous suspension>
           #4      AppContext.run.<anonymous closure>
(package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #5      AppContext.run (package:flutter_tools/src/base/context.dart:149:12)
           <asynchronous suspension>
           #6      CommandRunner.runCommand (package:args/command_runner.dart:197:13)
           <asynchronous suspension>
           #7      FlutterCommandRunner.runCommand.<anonymous closure>
           (package:flutter_tools/src/runner/flutter_command_runner.dart:284:9)
           <asynchronous suspension>
           #8      AppContext.run.<anonymous closure>
(package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #9      AppContext.run (package:flutter_tools/src/base/context.dart:149:12)
           <asynchronous suspension>
           #10     FlutterCommandRunner.runCommand
           (package:flutter_tools/src/runner/flutter_command_runner.dart:230:5)
           <asynchronous suspension>
           #11     run.<anonymous closure>.<anonymous closure>
(package:flutter_tools/runner.dart:63:9)
           <asynchronous suspension>
           #12     run.<anonymous closure> (package:flutter_tools/runner.dart:61:12)
           <asynchronous suspension>
           #13     AppContext.run.<anonymous closure>
(package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           #14     AppContext.run (package:flutter_tools/src/base/context.dart:149:12)
           <asynchronous suspension>
           #15     runInContext (package:flutter_tools/src/context_runner.dart:70:10)
           <asynchronous suspension>
           #16     main (package:flutter_tools/executable.dart:90:3)
           <asynchronous suspension>

[  +10 ms] (lldb) process detach
[  +97 ms] ensureAnalyticsSent: 100ms
[   +1 ms] Running shutdown hooks
[        ] Shutdown hook priority 4
[   +1 ms] Shutdown hooks complete
[        ] exiting with code 2

Running "flutter pub get" in icay...                             1,311ms
Analyzing icay...
No issues found! (ran in 8.3s)

[✓] Flutter (Channel beta, 1.24.0-10.2.pre, on macOS 11.0.1 20B50 darwin-x64, locale en-US)
    • Flutter version 1.24.0-10.2.pre at /Users/josetapizquent/Projects/Tools/flutter
    • Framework revision 022b333a08 (4 weeks ago), 2020-11-18 11:35:09 -0800
    • Engine revision 07c1eed46b
    • Dart version 2.12.0 (build 2.12.0-29.10.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/josetapizquent/Library/Android/sdk
    • Platform android-30, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

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

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

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

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

[✓] Connected device (3 available)
    • TapPhone (mobile) • 00008020-000E58220E99002E • ios            • iOS 14.2
    • Web Server (web)  • web-server                • web-javascript • Flutter Tools
    • Chrome (web)      • chrome                    • web-javascript • Google Chrome
      87.0.4280.88
    ! Error: TapPhone is busy: Copying cache files from device. Xcode will continue when
      TapPhone is finished. (code -10)

• No issues found!

Hi @tapizquent
Tried to reproduce using it on a physical device in profile mode, no issues on beta or stable.
Can you please provide your complete flutter run  --profile --verbose and a complete reproducible minimal code sample
Thank you

Hi @TahaTesser
The full terminal output is here:
https://gist.github.com/tapizquent/c021355916b4c494851381570be927cb
I do not have a reproducible project, as I have no way of identifying why this is not working. It only happens when I run it as Profile mode. It runs correctly when running it from Xcode.
Note that everything runs fine until it tries to open the observatory port. It even launches the app in the phone but then quits.

Closing as this seems to be a duplicate of #72161.
I will add that this happens only on v1.24, I went back to the stable channel, which is currently v1.22 and the launch error is no longer present.


Sometimes Geofence radius is 0.0 when trying to add to LocationService

See
https://fabric.io/locative/android/apps/io.locative.app/issues/57989893ffcdc042508b6137
and
https://fabric.io/locative/android/apps/io.locative.app/issues/578195cbffcdc04250759c47
Immediate fix has been deployed in v1.1.1

Never seen this again, won't fix.


Deletion of abandoned target outputs

If a target is removed from the build file but still exists in the build log,
it will be removed when building.
This is similar to what tup does as described in http://gittup.org/tup/build_system_rules_and_algorithms.pdf.
It's mentioned around page 17 why this is a failing of "alpha build systems".
Since the build log provides a form of persistence, it's possible to find the stray targets based on which entries exist in the build log, but haven't been added to the state.

I'm reluctant to delete files automatically.  I understand tup's reasoning for this, but to me it falls more into the "policy" area of a build system -- Ninja (tries to) just build files, not provide build system features.

Unfortunately, this isn't a feature that can be added outside of the build system. Debug and release modes are things that can be generated outside of the build system and would qualify in "policy". Without this type of feature, the build is "incorrect".
I think this type of feature works towards ninja's goals. One thing that makes builds faster is not requiring a clean build or to do "rm -rf build" error.
What do you define as "policy"? I've never heard the term used that way before. I'm guessing things like debug/release mode would count.
The only concern that I would have is having to mark every output as "seen" and how that effects performance. I haven't tried using this feature on a huge project at the moment (it was tested on the ninja build). Maybe try adding a dummy rule to the chrome build and then remove it, and see if it takes longer with this feature added?
This kind of thing is one of the weaknesses of make, and I've liked ninja so far in that it addresses a lot of the things in make that are just frankly... awful would be the word?

This policy change on ninja would be very unwelcome in our project, thus we'd rather this request did not go through, or alternately, was made an optional feature in either the command line arguments to ninja or high level .ninja file configuration.
The difference between ninja and tup here is that .ninja files are meant to be an intermediate stage generated from a high-level description.
Our project uses the user's high-level build command (which subset of projects to build, which debug or release configuration, and so forth) to then write out the .ninja files. New .ninja files are written with potentially every user build command.
If this pull request goes through, then every time the user switches from debug to release, all of the other configuration's files will be deleted.
This would be unwelcome behaviour.


AccessMiddleware and OperaLogMiddleware cannot exist at the same time.

If you turn on AccessMiddleware settings, an exception "'async for' requires an object with aiter method, got method" will occur. Besides that, should request.state.ip be set in OperaLogMiddleware? Since OperaLogMiddleware is an optional plugin, if the user disables this plugin, the auth module will throw an exception for missing state.ip because it is used by captcha.

In fact, OperaLogMiddleware is not optional!

if OperaLogMiddleware is not optional, the PR does not need to handle. but can AccessMiddleware and OperaLogMiddleware exist at the same time?

I would check out AccessMiddleware, because I hardly ever use it.

OperaLogMiddleware in order to achieve the function to do some special operations, it seems that with AccessMiddleware (BaseHTTPMiddleware) there are some conflicts, according to the fastapi middleware from top to bottom order of registration, modify the OperaLogMiddleware registration location, make it accessible to more raw data

OperaLogMiddleware in order to achieve the function to do some special operations, it seems that with AccessMiddleware (BaseHTTPMiddleware) there are some conflicts, according to the fastapi middleware from bottom to top order of registration, modify the OperaLogMiddleware registration location, make it as non-destructive as possible


Why there is no property "panzoom" in my cy? version 3.21.1

What was the solution to this? We are seeing the same issue. Why was this resolved @maxkfranz @DiligentTuolaji

Here are my cytoscape package versions in my package.json:
    "cytoscape": "^3.29.2",
    "cytoscape-expand-collapse": "^4.1.0",
    "cytoscape-panzoom": "^2.5.3",
    "cytoscape-undo-redo": "^1.3.3",
    "react-cytoscapejs": "^1.2.1"

and here is how the panzoom library is initialized and then called:
import Cytoscape from 'cytoscape';
import panzoom from 'cytoscape-panzoom';
import 'cytoscape-panzoom/cytoscape.js-panzoom.css';
import merge from 'lodash.merge';
import CytoscapeComponent from 'react-cytoscapejs';

panzoom(Cytoscape);

...

return

(<CytoscapeComponent
        elements={ CytoscapeComponent.normalizeElements(graphData) }
        style={{ height: graphHeight }}
        zoomingEnabled={ true }
        maxZoom={ 3 }
        minZoom={ 0.001 }
        autounselectify={ false }
        boxSelectionEnabled={ true }
        layout={ merge([], defaultLayout, graphLayout) }
        stylesheet={ merge([], defaultStyles, graphStyles) }
        cy={ (cy) => {
          cy.panzoom(defaultPanZoom);
        } }
      />
    </div>
  );

This used to work but recently I've been getting the error e.panzoom is not a function. I suspect it has to do with the way the dependencies are initialized but I'm not sure


Feature/23and me split

Handle 23andMe files with split genotype alleles

This has pulled in several upstream changes too, don't think its a huge problem though it's not as clean as I would like.

Since all the tests pass, and I'd like to get this in ASAP while the ZFactor recruitment is ongoing, I'm going to force-merge this.


Update app-insights-tutorial-performance.md

@tysonn : Thanks for your contribution to the Azure documentation! The author, @mrbullwinkle, has been notified to review your proposed change.

#sign-off


images: update cilium-{runtime,builder}

Fix master after the merge of https://github.com/cilium/cilium/pull/22565.
Fixes: https://github.com/cilium/cilium/pull/22565

Awaiting our :robot: overlords to complete the PR: https://github.com/cilium/cilium/actions/runs/3944818161/jobs/6751121941

Our overlord's overlords got impatient and cancelled the automated action: https://github.com/cilium/cilium/actions/runs/3944818161

https://github.com/cilium/cilium/actions/runs/3944818161

Given that the runtime and builder images were pushed, I've just manually updated the commit.

Already got Andre's review, and this fixes master. Merging.


Support LICENSE.md files

Not all repo's have the specific LICENSE or LICENSE.txt file that AckGen is searching for. Sometimes, license files are named LICENSE.md
An example of a license with the md extension is https://github.com/mapbox/turf-swift

@devmat23
Thank you! That seems reasonable. 👍🏼


Add error handling on API fetches

Currently some of the code was removed because of the switch from RPC to REST query.
We want to have this functionality back and show to user adequate unified errors on UI.

We thought about adding toaster for this


Add "bright" mode option

I noticed in the 2.0.0 release this was removed, so I'm not sure if this is actually a desired feature or not. Pre v2, I liked having high contrast and bright both set, causing a very high contrast setup. I have re-added bright using the saturation and luminance values from v1, but if this is a desired feature and these should be different from a design/visual perspective, let me know and I'll change it. Below is what this setup looks like with the following options:
{
  theme = {
    style = 'dark',
    contrast = 'high',
  },
  colors = {
    mode = 'bright',
  },
  ui = {
    borders = 'fluo',
  },
}

Hey @ctffarley, happy to see you've already tested out the new version! The change was intentional tbh. What I wanted to do for the v2 is to create a palette that is working good with both dark and light theme. The issue with bright mode is that it I didn't find a way to make it usable with the light scheme:

As you can see it is almost unreadable. What I was considering to add is something like this:
  on_colors = function(colors)
    colors.hint = colors.orange
    colors.error = "#ff0000"
  end

But for the values of Saturation and Light of the palette. Do you have any alternative idea?

Ah yeah I see what you mean. If that's the goal then bright definitely won't work well with a light mode. I think your on_colors suggestion is a good one since it would allow me (and anyone else that wants it) to be able to recreate something more like their v1.0 setup if they used bright or desaturated.

Hey @ctffarley, I just created a PR #21  with the discussed feature. Do you mind checking if it allows to customize the colorscheme to provide the colors you were using?

Close in favor of #21

@0xstepit yup, just tested it out and it has the same effect. The only small addition I would add is that the config example in the readme pins the tag as 1.0.0 when it should be 2.0.0 (or ^2.0.0 or similar).


Project Requirements

Hello Harish Chandra Thuwal.
I'm from Turkey.I am studying at a university and working on a project like your project.
But I'm new to this area and I do not know how to access the model to run the project. Can you help me with what I should do?
My e-mail address is<EMAIL_ADDRESS>so please be very happy if you return.

Hi,

You can go through the Final Report.pdf to get an idea about the methodology that we used.
Once you go through the files and the pdf, you'll get an idea about how to do it.
I am planning to add a readme with steps in the future but it will take time.

You can learn basic steps about training inception(tensorflow) from here

Hi Harish, thank you for your kind answer, i try to open and run the project, but i can't find data sets folder.Could you share data sets folders,please?

Reference to the dataset used is present in the Final Report.pdf. You can check that out.

Thank you very much


Add "relation" model type

Per 2024/12/16 meeting:

defined a new "relation" type in the model - similar to a URL
when used, a new target attribute is available that points to the type of registry entity being pointed to
e.g. `"target": "schemagroups/schema"
MUST be one of:

/ (the registry itself)
/GROUPS
/GROUPS/RESOURCES
/GROUPS/RESOURCES/version
/GROUPS/RESOURCES/version? - runtime value can be a resource or a version

runtime value of this attribute MUST be the xid of the target

Do we want to support a relation that points to any entity in the Registry? E.g. "target": "any"

Should it be singular or plural names?  e.g. /endpoints/messages vs /endpoint/message
Normally we use plural for everything, but since the entity attribute itself will point to a single instance of a message, it might read better if we used "singular" for the target value.
Is it ok that I added / to point to the registry itself? I added this just for completeness.

Since an attribute of type relation isn't a full URL (it's just an xid), do we expect people to construct a real URL themselves? It would be kind of annoying to have to add a 2nd attribute just to give people the full URL.
Some options:
1 - leave it as is, yes they need to construct the URL themselves
2 - automatically create a 2nd (read-only) attribute (e.g. "foo-resolved": "URL" for each foo relation attribute)
3 - convert the xid into a full URL automatically during a GET. Would make the PUT and GET have different values for that attribute. Can't reuse GET response as PUT input. Yuck.


feat(post): add the consul DO + cloudflare post

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


initial api

@geekbeast

LGTM, would consider renaming service to DockerInfoService or something of the sort unless you have bigger things in mind for the class.

Weird build issues...

DockerInfoService sounds good, I was a bit tired at the time :p


python311Packages.aioesphomeapi: 18.2.0 -> 18.2.1

Diff: https://github.com/esphome/aioesphomeapi/compare/refs/tags/v18.2.0...v18.2.1
Changelog: https://github.com/esphome/aioesphomeapi/releases/tag/v18.2.1
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 264743 run on x86_64-linux 1

41 packages built:

esphome
esphome.dist
home-assistant-component-tests.airthings_ble
home-assistant-component-tests.aranet
home-assistant-component-tests.bluemaestro
home-assistant-component-tests.bluetooth_adapters
home-assistant-component-tests.bluetooth_le_tracker
home-assistant-component-tests.bthome
home-assistant-component-tests.dormakaba_dkey
home-assistant-component-tests.esphome
home-assistant-component-tests.eufylife_ble
home-assistant-component-tests.fjaraskupan
home-assistant-component-tests.govee_ble
home-assistant-component-tests.homekit_controller
home-assistant-component-tests.ibeacon
home-assistant-component-tests.inkbird
home-assistant-component-tests.kegtron
home-assistant-component-tests.keymitt_ble
home-assistant-component-tests.ld2410_ble
home-assistant-component-tests.led_ble
home-assistant-component-tests.moat
home-assistant-component-tests.mopeka
home-assistant-component-tests.oralb
home-assistant-component-tests.private_ble_device
home-assistant-component-tests.qingping
home-assistant-component-tests.rapt_ble
home-assistant-component-tests.ruuvitag_ble
home-assistant-component-tests.sensirion_ble
home-assistant-component-tests.sensorpro
home-assistant-component-tests.sensorpush
home-assistant-component-tests.snooz
home-assistant-component-tests.switchbot
home-assistant-component-tests.thermobeacon
home-assistant-component-tests.thermopro
home-assistant-component-tests.tilt_ble
home-assistant-component-tests.xiaomi_ble
home-assistant-component-tests.yalexs_ble
python310Packages.aioesphomeapi
python310Packages.aioesphomeapi.dist
python311Packages.aioesphomeapi
python311Packages.aioesphomeapi.dist


Added feature of social media sharing buttons

Description

Added social media sharing buttons with 3D effects.
Made the buttons responsive across devices.

Preview

Related Issues

Addresses issue #3

Thank you @Joshua-Vankayalapati for adding this feature.

It's my pleasure!!


commuity.r-multiverse.org failing when added to CI

The problem manifests itself for example here:
https://github.com/shikokuchuo/mirai/actions/runs/9641056206/job/26586246583#step:6:143
pak::pak() claims:

source packages are missing from community.r-multiverse.org

This doesn't reproduce if I run pak::pak() locally.
Also it seems fine to grab the binaries for Windows and Mac so those workflows succeed, e.g. at:
https://github.com/shikokuchuo/mirai/actions/runs/9641056206/job/26586247306#step:6:1188
Adding the repo is via amending the GitHub workflow yaml as per the below:
      - name: Add r-universe to repos
        run: |
          cat("\noptions(repos=c(RMV='https://community.r-multiverse.org',CRAN ='https://cloud.r-project.org'))\n", file = "~/.Rprofile", append = TRUE)
        shell: Rscript {0}

This is what I use all the time, and works fine if I swap in my personal R-universe instead.
@jeroen it would be great if you have an idea what this is - whether it is something to do with the re-direct, or a bug that can be fixed in pak itself? Thanks.
cc. @wlandau

Hmm weird, could you try the dev version of pak? https://github.com/r-lib/pak/issues/266#issuecomment-2156115228

@gaborcsardi what does the error source packages are missing from community.r-multiverse.org mean?

Fails to get the metadata from that site. Either a HTTP error or the file does not parse as a PACKAGES file.

This is very strange. If I change the domain to https://r-multiverse.r-universe.dev then the problem disappears, but they point to the same backend. Perhaps something in your r-multiverse.org cloudflare settings that triggers a bug?
I'm trying to find a way to figure out what pak is actually downloading and parsing.

Strange indeed. I don't have anything additional set up for Cloudflare. I've invited you to the Cloudflare account.
I'm happy for you to transfer the settings to the account you use for R-universe, and then I can disable this one.

Thanks. I can see your clouldflare settings now but somehow I can't edit them yet (I think it takes a while). Could you try the following for me: under speed > optimization you can disable HTTP/3.

I've changed those settings.

Very strange I'm seeing identical payloads for https://community.r-multiverse.org/src/contrib/PACKAGES and https://r-multiverse..r-universe.dev/src/contrib/PACKAGES. Testing using nanonext::ncurl() locally and using base url() and readLines() on the demo webr platform.

OK I was able to reproduce the problem. Somehow cloudflare identifies R as a potential spammer, and shows an interactive captcha page with a 403.
@shikokuchuo in your cloudflare settings, under "security" > "settings" can you try to set Security Level to the lowest possible value?

Fantastic! I can confirm that this is the issue, and it now works e.g. https://github.com/shikokuchuo/mirai/actions/runs/9646460548/job/26606426809
If you see security > events, you can see "bot fight mode" triggering. I think it's because it detects the request coming from GHA (a cloud platform) and sees it likely as a bot attempt. I've disabled this without changing the security setting. I turned on the AI scrape shield at the same time!
Thanks for investigating this.


Template typo

Last issue - just spotted a small typo at https://github.com/TwilioDevEd/eta-notifications-flask/blob/master/eta_notifications_flask/templates/show.html#L30
The button should read "Deliver the package" instead of "Deliver up the package."
Overall this app looks really good! And I especially appreciate the high test coverage.

Done


feat: upgrade native sdk <IP_ADDRESS>

Update native sdk <IP_ADDRESS> dependencies

This pull request is trigger by bot, you can checkout this branch and update it.

Examples built:

https://nightly.link/littleGnAl/hoe/actions/runs/7346206975/agora_rtc_engine_android_6.2.6-sp.4263_20231228092324.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7346206975/agora_rtc_engine_ios_6.2.6-sp.4263_20231228092505.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7346206975/agora_rtc_engine_macos_6.2.6-sp.4263_20231228092143.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7346206975/agora_rtc_engine_windows_6.2.6-sp.4263_20231228092043.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7346206975/agora_rtc_engine_web_6.2.6-sp.4263_20231228091749.zip

This comment is commented by bot, do not edit it directly

Examples built:

https://nightly.link/littleGnAl/hoe/actions/runs/7353399180/agora_rtc_engine_android_6.2.6-sp.4263_20231229032753.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7353399180/agora_rtc_engine_ios_6.2.6-sp.4263_20231229032728.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7353399180/agora_rtc_engine_macos_6.2.6-sp.4263_20231229032223.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7353399180/agora_rtc_engine_windows_6.2.6-sp.4263_20231229032352.zip
https://nightly.link/littleGnAl/hoe/actions/runs/7353399180/agora_rtc_engine_web_6.2.6-sp.4263_20231229032011.zip

This comment is commented by bot, do not edit it directly


Azure Government Availability Tests IP addresses used in production no longer match documentation

In the addresses-grouped-by-location section of the document, at the bottom is the Azure Government IP addresses. We recently added firewall rules to one of our applications which had App Insights availability tests. We added the 5 address groups indicated and everything was working fine for about 2 weeks until 2 days ago when some new IP address not in these ranges started being blocked by the firewall rules. The IP address ranges listed for Azure Governement are:
<IP_ADDRESS>/27
<IP_ADDRESS>/27
<IP_ADDRESS>/27
<IP_ADDRESS>/27
<IP_ADDRESS>/27
However, we are seeing Availability Tests originating from the following IP addresses:
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
Can the documentation be updated to correctly include all of the possible IP addresses from which Availability Tests can originate?

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

ID: 3942b937-b212-3ceb-d356-450b7f598f68
Version Independent ID: 8b015b42-cf02-3942-3cc3-6e0f977d191a
Content: IP addresses used by Application Insights and Log Analytics - Azure Monitor
Content Source: articles/azure-monitor/app/ip-addresses.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @lgayhardt
Microsoft Alias: lagayhar

@Quintinon Thanks for the comment!
Assigning @lgayhardt for visibility, review and document enhancement consideration if appropriate.

Any updates to this?  it's been 13 days since any of our Azure Government Availability tests have worked because the IP's are not coming from the ones in this list.

Hi @Quintinon
The Azure Gov IP addresses have been updated.
https://docs.microsoft.com/en-us/azure/azure-monitor/app/ip-addresses#azure-government
Thank you!

#please-close


Add TOML config file loader

I tried to make the loader of TOML files in issue #235
It uses the https://github.com/BurntSushi/toml to implement
The same test as the YAML loader has path

LGTM :+1: and I'd like someone more familiar with the guts of ./altsrc to review, too :smile_cat:

@roboll @ChrisPRobinson :wave: 😺

I would update Consider updating front page markup to indicate that now it supports TOML too

👍 Looks good.

Added notes to README and CHANGELOG in https://github.com/urfave/cli/commit/b616f6088660d2eaa33739718f0583f8d467a178
Thanks @ykanda and @ChrisPRobinson !


Show log natively

Add functionality to read the log file from within nyaadle.

Current implementation is fine for normal use. Therefore this is considered not necessary. Closing this issue.


Integrate Cosmic Text

Specific Demand
Cosmic text implements some more advanced text layout and editing. It supports rendering bold, italicized, and multiline text. It also implements text editing on Unicode characters.
Implement Suggestion
Replace the current text layout engine with Cosmic text and use Cosmic text to implement editing when input widgets are implemented.
Related: https://github.com/linebender/vello/issues/306

Are there issues with cosmic-text blocking this?

Thanks for the feedback. I'll be looking into layout with custom rectangles soon.


Implemented sort using the merge sort algorithm

Proposed changes
Implemented sort method using the merge sort algorithm. We can now perform sort operations on our container in O(n log n) time, without copying, destroying, or manipulating the data members of each node. Instead we manipulate the links each node is pointing to. The drawback of this implementation is that the algorithm is recursive, therefore there is a limit to the size of the container that can be sorted.
We will need to ascertain what this limit is, and update the sort algorithm in the future to either be iterative or incorporate a dynamic bottom-up approach. However, for version 1.0 of our linked list release, I believe this algorithm to be adequate.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

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

[x] I have read the CONTRIBUTING doc
[x] Unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Codecov Report

Merging #50 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #50   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines         532    603   +71
=====================================
+ Hits          532    603   +71

Impacted Files
Coverage Δ

tests/linkedListTest.cpp
100% <ø> (ø)
:arrow_up:

src/linkedList.cpp
100% <100%> (ø)
: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 3810eea...2ba768d. Read the comment docs.


Logging and verbose mode

I've just added the verbose mode and swapped fmt by log to make everything more uniform.

LGTM


Allow Escalating Location Permissions

If a user upon first launch authorizes .locationWhenInUse, and then later, when taking a certain action, needs to authorize .locationAlways, it shows as denied and that they need to change the settings from iPhone App settings. In fact, it is possible to upgrade from here, in-app, without going to the iPhone App Settings. (:

@mgt-la It's not posssible. If later you need use . locationAlways, you should request it. Apple allows to request location permissions only once

It works on PermissionScope which is another Pod like yours and apple has approved it. I just preferred this UX out of the box 🙂

Just tested, this works, if you press btn1, accept, then press btn2 and accept.
import UIKit
import Sparrow
import CoreLocation
class ViewController: UIViewController, CLLocationManagerDelegate {

  let manager = CLLocationManager()

  @IBAction func btn1(_ sender: Any) {
      //SPRequestPermission.dialog.interactive.present(on: self, with: [.locationWhenInUse])
      manager.requestWhenInUseAuthorization()
  }

  @IBAction func btn2(_ sender: Any) {

      //SPRequestPermission.dialog.interactive.present(on: self, with: [.notification,.locationAlways])
      manager.requestAlwaysAuthorization()
  }
}

@mgt-la I find how do it, in one hour I will update pod

🙏🤞😬 😃🏆

@mgt-la please, say me, need change permission from .always to .whenInUse?

@mgt-la I am updated pod, you can use it now. Thanks for your help!

@IvanVorobei sorry for the late reply- no, will never need to de-escalate back down to .whenInUse once .always is allowed.
Thank you!! You're awesome! This is a great pod and the other parts of Sparrow are a nice addition!


Commands in run_after_clone.sh script in panorama_sdk are incorrect

In the README.md file, there are instructions to run sh-4.2$ sudo sh aws-panorama-samples/panorama_sdk/run_after_clone.sh but this produces the following:
chown: cannot access ‘AWSOmniCVExamples/’: No such file or directory
chgrp: cannot access ‘AWSOmniCVExamples/’: No such file or directory

It appears that the folder structure has changed. Recommend either removing this step or telling users to manually run the appropriate chown and chgrp commands.

Thank you for flagging this.
The link https://panorama-starter-kit.s3.amazonaws.com/public/v1/Models/panorama_sdk.zip has been updated with the right run_after_clone.sh file. Please re-pull the panorama_sdk and it should work.

Appears to have been resolved. Thanks!


(Feature Request) Music keyword search brings up multiple choices

This would be a way to have the bot show a list of selections for the search query instead of auto-selecting the first result, which is often not what the user wanted played in the first place.
I've seen this on another bot before and would be a good addition to this one.
An example would be: "!!play nujabes"

Yeah it's possible and not terribly hard to do, but with youtube API it's just a bit annoying and inherits some issues from that. Plus I'm not sure how much faster that is over searching again.
Anyways, I know that feature and I think that it would come after a bigger re haul of the music module first

Yea this is a nice suggestions

Not a priority. After 1.3

I won't do this for 1.42, I'll go through the whole music module soon:tm: to stabilize it and add this too.

Scrap that, seems fun, I'll do it tomorrow as the last thing for 1.42

Was fun to make


azkaban  scheduler.xml

can you  azkaban  conf  at azkaban server  executor.metric.reports=true" to azkaban.properties but dr-elephant  at azkaban SchedulerConf.xml
Azkaban is turned on by default but why Azkaban works and flow

@ljank

@fusonghe, can you first verify if Azkaban is injecting the properties into the job config. Search for properties like azkaban.link.workflow.url in the job configuration.

@fusonghe
I also encountered this problem, do you solve it?


Items: Use Item base struct

Fixes #28

@tiltingpenguin good catch! Thanks for the review. Should be fixed now. :)


Include all file cache access in privileged section

This expands on #6914 a bit to include the full cache access. Adding an entry to the cache can result in an eviction, and the subsequent file deletion needs the elevated permissions as well.
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #6920 (4a20bfd) into main (805e6d9) will decrease coverage by 0.02%.
The diff coverage is 46.15%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #6920      +/-   ##
============================================
- Coverage     70.64%   70.62%   -0.02%
- Complexity    59160    59224      +64
============================================
  Files          4812     4812
  Lines        283750   283751       +1
  Branches      40915    40915
============================================
- Hits         200452   200400      -52
- Misses        66817    66862      +45
- Partials      16481    16489       +8

Impacted Files
Coverage Δ

...arch/index/store/remote/utils/TransferManager.java
78.78% <46.15%> (+0.66%)
:arrow_up:

... and 506 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Fix consecutive header bug

Fix #2862

Proven header store looks at height not prev hash, so there can be a race condition where consensus is reorging and signaling new blocks, those will get in to the PH store but while a reorg is taking place this situation can still happen (if a getprovenheader is send in the middle of a reorg PH store will have none consecutive headers).
There is nothing to fix really this is the deign of PH store.

Proven header store looks at height not prev hash, so there can be a race condition where consensus is reorging and signaling new blocks, those will get in to the PH store but while a reorg is taking place this situation can still happen (if a getprovenheader is send in the middle of a reorg PH store will have none consecutive headers).

If this is the case then the fix is correct.
So we are looking at the scenario when chain reorged already and PH store not yet.
Is it possible to go into that if on ln 129?
if (!(header.Header is ProvenBlockHeader provenBlockHeader))

All headers we received for the chain we've reorged to are PH.
Unless I'm missing smth it shouldn't be possible that we don't have PH in memory for chain we reorged to.

Actually, it looks like we can't go into that if but we still can get not consecutive header.
ConcurrentChain's EnumerateToTip is just getting block's by height from fork's height to tip's height.
So if we've reorged and set chain's tip to another header from a different chain while we are inside of foreach (ChainedHeader header in this.chain.EnumerateToTip(fork).Skip(1)) we will get a bad header.
In that case this check makes sense, but only if it's after the if, not inside of it.

In that case this check makes sense, but only if it's after the if, not inside of it.

@noescape00 to what if are you referring to exactly?

I actually think the problem is indeed also in ConcurrentChain and we should avoid using it (yeah it will get deleted but we can remove it from here already now)
But also PH store can still be in an inconsistent state.

Looks good, reminder to make the same change in ConesnsusManagerBehaviour

Looks good, reminder to make the same change in ConesnsusManagerBehaviour

I have also made the changes in CMB 👍


Add Vietnamese version: vie-RVV11

Add Vietnamese translation: vie-RVV11 from bibles.org. We had a user request.
http://m.bibles.org/versions/vie-RVV11

While this translation is available there, I'm having trouble loading it through the API.


Contact Us Page

A page where we can leave some info for curious participant to contact us

A form that will submit an email to the drexel weekend warrior gmail

Create a backend endpoint that will allow Strapi to send the email

Set up the email service on Strapi (back-end) side

Create a replacement div to appear once email is sent to confirm it has been sent

Optional: Allow them to send to individual trip leaders (Might be problematic since people might not want their emails assessed)

Finished


depreciate default alchemy key (#955)

Note we shouldn't squash and merge this, instead of a merge commit.
Also planning to release new version after we merge this PR since current latest version is broken :/

Thanks @rin-st! Lol sooo fast!!


Bugfix - Android no return if picker is dismissed

First thank you very much for this plugin. It works very well.
I found one small problem: If you cancel the process of saving a file on android with FileSaver.instance.saveAs and you dismiss the folder selection, the method never returns.

Why did you close it? I am facing the same issue.

Hey, @Maz47 is it working now?


Traffic Ops not listening on IPv6

The current Mojo listen line is "https://*:443", I changed it to "https://[::]:443" to make it work with IPv6.
Not having this causes failures to ORT reaching out to Traffic Ops (when the AAAA DNS records are present)
WARN Error receiving https://cdn1cdcms01.coxlab.net/update/cdn1cdedge01 from Traffic Ops: curl: (6) name lookup timed out 000

Looks like the issue wasn't with reaching "port refused" but caused by a secondary DNS not responding in my /etc/resolv.conf file. I'll file a separate issue about that new condition. But the issue here is still relevant that by default IPv6 listening isn't enabled.


ViewGroupDescriptor.registerDecorView() should move to AndroidDescriptorHost

If you look at where this method is called, in WindowDescriptor, we have some uncomfortable coupling between WindowDescriptor and ViewGroupDescriptor:
  private void registerDecorView(View decorView) {
    if (decorView instanceof ViewGroup) {
      Descriptor descriptor = getHost().getDescriptor(decorView);
      if (descriptor instanceof ViewGroupDescriptor) {
        ((ViewGroupDescriptor) descriptor).registerDecorView((ViewGroup) decorView);
      }
    }
  }

This code is also coupled to the fact that decorView doesn't have a more specific Descriptor registered for it.
I believe a better way to do this would be something like:
  private void registerDecorView(View decorView) {
    if (getHost() is AndroidDescriptorHost) {
      ((AndroidDescriptorHost) getHost()).registerDecorView();
    }
  }

Hmm, actually this might not even be necessary to have at all. We're not doing much with it: we only use it to control whether a ViewGroup will filter out DOMHiddenViews. And we only cared about that for performance with the getChildCount()/getChildAt() protocol because Decor Views need to count DOMHiddenViews and subtract them from the getChildCount(), etc. With the new protocol, i.e. getChildren(Accumulator), we don't have that problem. So we can just remove this whole decor view marking fiasco from ViewGroupDescriptor entirely.

Reopening because I tagged the wrong issue in my initial commit for #206


Mac M1 load error: dlsym(0x309d14370, sqlite3_vss_init): symbol not found

M1 builds successfully but unable to load extension into SQLite version 3.36.0
Mac 13.4.1
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX13.3.sdk
Source package sqlite-vss-0.1.1-alpha.20
sqlite> .load /Users/user/Desktop/sqlite-vss/sqlite-vss/build_release/vss0.dylib
Error: dlsym(0x309d14370, sqlite3_vss_init): symbol not found

Do you get the same error if you're using the pre-built libraries?

Yes. Thanks for quick response.
(including .dylib it seems to duplicate)
sqlite> .load /Users/user/Downloads/sqlite-vss-v0.1.1-alpha.20-loadable-macos-aarch64/vector0.dylib
Error: dlopen(/Users/user/Downloads/sqlite-vss-v0.1.1-alpha.20-loadable-macos-aarch64/vector0.dylib.dylib, 0x000A): tried: '/Users/user/Downloads/sqlite-vss-v0.1.1-alpha.20-loadable-macos-aarch64/vector0.dylib.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Users/user/Downloads/sqlite-vss-v0.1.1-alpha.20-loadable-macos-aarch64/vector0.dylib.dylib' (no such file), '/U
(removing it seems to not like architecture)
sqlite> .load /Users/user/Downloads/sqlite-vss-v0.1.1-alpha.20-loadable-macos-aarch64/vector0
Error: dlopen(/Users/user/Downloads/sqlite-vss-v0.1.1-alpha.20-loadable-macos-aarch64/vector0.dylib, 0x000A): tried: '/Users/user/Downloads/sqlite-vss-v0.1.1-alpha.20-loadable-macos-aarch64/vector0.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/System/Volumes/Preboot/Cryptexes/OS/Users/user/Downloads/sqlite-vss-v0.1.1-alpha.20-loa

Can you run uname -s -m on your machine and share what it prints out?
The mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64') message makes you think you might be on a x86_64 machine.
But regardless of that, if you build it yourself you shouldnt get the 'sqlite3_vss_init` symbol not found error. Can you also run:
nm -g /Users/user/Desktop/sqlite-vss/sqlite-vss/build_release/vss0.dylib | grep sqlite3_

hm alright that's really weird, a few questions:

How did you build build_release/vss0.dylib? Did you git clone and run make loadable-release?
Can you try the binaries at https://github.com/asg017/sqlite-vss/releases/download/v0.1.1-alpha.20/sqlite-vss-v0.1.1-alpha.20-loadable-macos-aarch64.tar.gz ? And also run the nm -g command on those .dylib files?
Can you run sw_vers?

 Be careful with the space (" ") character in that file path when you try, you can cd into that directory first and run sqlite3 with  .load ./vector0 .
I suspect that .load ./vss0 in that directory should also work. If it doesn't, could you run nm -g vss0.dylib | grep sqlite3_ on that as well?
Re building yourself: if you did follow those steps then I have no idea why you get no symbols on there. You can also look in the sqlite-vss/dist/release directory, which should also have a vector0.dylib and vss0.dylib files. They're normally the same as the files in build_release, but something might've gotten corrupted

Ugh same incompatible message. Really weird.
@McLap > ~/Downloads/sqlite-vss-v0.1.1-alpha.20-loadable-macos-aarch64  /usr/local/opt/sqlite/bin/sqlite3
SQLite version 3.36.0 2021-06-18 18:36:39
Enter ".help" for usage hints.
Connected to a transient in-memory database.
Use ".open FILENAME" to reopen on a persistent database.
sqlite> .load vector0
Error: dlopen(vector0.dylib, 0x000A): tried: 'vector0.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')), '/System/Volumes/Preboot/Cryptexes/OSvector0.dylib' (no such file), '/usr/lib/vector0.dylib' (no such file, not in dyld cache), 'vector0.dylib' (mach-o file, but is an i
sqlite> .load vector0.dylib
Error: dlopen(vector0.dylib.dylib, 0x000A): tried: 'vector0.dylib.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OSvector0.dylib.dylib' (no such file), '/usr/lib/vector0.dylib.dylib' (no such file, not in dyld cache), 'vector0.dylib.dylib' (no such file), '/usr/local/lib/vector0.dylib.dylib' (no such file)
sqlite> .load vector0.dylib
@McLap > nm -g /Users/cmcilvoy/Desktop/sqlite-vss/sqlite-vss/dist/release/vector0.dylib
/Users/cmcilvoy/Desktop/sqlite-vss/sqlite-vss/dist/release/vector0.dylib: no symbols
@McLap > nm -g /Users/cmcilvoy/Desktop/sqlite-vss/sqlite-vss/dist/release/vss0.dylib
/Users/cmcilvoy/Desktop/sqlite-vss/sqlite-vss/dist/release/vss0.dylib: no symbols

Can you run file /usr/local/opt/sqlite/bin/sqlite3 ?
On my M1 mac, I installed sqlite3 with homebrew so it returns:
file /opt/homebrew/opt/sqlite/bin/sqlite3
/opt/homebrew/opt/sqlite/bin/sqlite3: Mach-O 64-bit executable arm64

I suspect that the 'vector0.dylib' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')) error may be coming from some MacOS Rosetta stuff that's trying to run x86_64 programs on your M1. So I'm thinking  /usr/local/opt/sqlite/bin/sqlite3 may be a x86_64 build of sqlite3 and is causing this issue

That was it. I did a brew reinstall and its working properly. thanks!

Glad to hear! Will add this problem to a "troubleshooting" doc in the near future