LG-3187: Upgrade USWDS to 2.9.0

Related (merges to): #158
Preview on Federalist: https://federalist-2f194a10-945e-4413-be01-46ca6dae5358.app.cloud.gov/preview/18f/identity-style-guide/aduth-uswds-upgrade/
This pull request upgrades USWDS dependency from ^2.0.3. to ^2.9.0 (latest).
Notes:

See additional documented findings in ticket LG-3187 description
There have already been a few conflicts in the use of colors. As noted in the research findings, USWDS v2.4.0 introduced additional support for non-token values. The USWDS internal implementation makes some assumptions about the readiness of tokens at initialization, which conflicted with previous usage of token assignment to theme variables. Furthermore, there's at least one upstream bug in USWDS bypassing color tokens which does not interoperate with custom theme colors (source, pending upstream bugfix).

Furthermore, there's at least one upstream bug in USWDS bypassing color tokens which does not interoperate with custom theme colors (source, pending upstream bugfix).

See https://github.com/uswds/uswds/pull/3762

Regression: Banner link reverts to blue, resulting in accessibility failures.
Screenshot:

Error:
    "http://<IP_ADDRESS>:35819/accessibility/policies/": [
      {
        "code": "WCAG2AA.Principle1.Guideline1_4.1_4_3.G18.Fail",
        "context": "Here’s how you know",
        "message": "This element has insufficient contrast at this conformance level. Expected a contrast ratio of at least 4.5:1, but text in this element has a contrast ratio of 2.66:1. Recommendation: change text colour to #fdfeff.",
        "type": "error",
        "typeCode": 1,
        "selector": "html > body > div:nth-child(2) > div > header > div > button > span"
      }
    ],

Fix: 23f63c9

Regression: USWDS upgraded the major version of Normalize.css, causing certain opinionated styling to be omitted. This causes pre tags to unexpectedly overflow.
Related: https://github.com/necolas/normalize.css/blob/master/CHANGELOG.md#400-march-19-2016
Screenshot (before):

Fix: b6dc182
Impact on other products: None, assuming upgrade, since changes are exclusive to stylesheet updates.
Screenshot (after):

Update from research findings:

Header and banner width now controlled in settings: Now there are explicit settings for controlling the max-width of the gov banner and the header.

Affected variables are renamed in 9fcc61c.
See: https://designsystem.digital.gov/about/releases/#version-220

Acknowledged and expected change: Footer text changes from Serif (Merriweather) to Sans-Serif (Source Sans Pro).
Screenshot:

Before
After

Related changelog: USWDS v2.3.0

Now components reliably respect their font settings. Setting values like $theme-footer-font-family should set the font face for the entire component, but some CSS specificity quirks were overriding these values in some instances. Now, setting a component’s font face works as expected, with no secret overrides. (https://github.com/uswds/uswds/pull/3253)

Relevant variable assignment:
https://github.com/18F/identity-style-guide/blob/b196ccfcfc3da550275e43eb21e021f6581e2c83/src/scss/uswds-theme/_components.scss#L36

Acknowledged and expected change: Line-heights on most content has changed slightly, in direct correlation with a fix in USWDS 2.2.0 to improve rounding precision of the line-height functions. This is only explicitly mentioned to affect navigation, but the affected functions (px-to-rem, line-height, lh) are used extensively throughout USWDS. The effect is that pages may extend longer vertically than they had previously (verified on documentation site with visual regression test results).
Pull request: https://github.com/uswds/uswds/pull/3100

Fixes a spacing issue affecting the header nav indicator bars present since 2.0 that seems to be a result of rounding errors in line-height calculation.

Because this had a significant impact on visual regression testing, I was able to assess the impact of these and other changes by effectively reverting the fix: https://gist.github.com/aduth/77c4d3785666088282cee22f37acd981
The impact is noticeable only under scrutinous review of differences.
Example:

Before
After

⚠ Requires markup changes at time of upgrade in dependent projects ⚠
USWDS 2.5.0 includes updates to markup for the benefit of improved accessibility.

banner: is now a
 instead of a
 with the ARIA label Official government website
footer: nav includes the ARIA label of Footer navigation
graphic-list: uses h2 as a heading default instead of h3
hero: includes the ARIA label of Introduction
search: the search form is given the ARIA role of search
documentation template: includes only the main content in the  element. The nav is no longer treated as an

https://designsystem.digital.gov/about/releases/#version-250
Examples updated in: 64f32de
This will require corresponding markup changes in consuming projects at time of upgrade.

⚠ Requires markup changes at time of upgrade in dependent projects ⚠
USWDS 2.6.0 updates markup of search component.

Use more consistent search markup. We made the markup of usa-search more consistent and assured that the styling worked properly regardless of what version of the markup you’re using. This is not as breaking change, but we recommend updating your search markup to the most current version. (https://github.com/uswds/uswds/pull/3327)

https://designsystem.digital.gov/about/releases/#version-260
Examples updated in: a611187
This will require corresponding markup changes in consuming projects at time of upgrade.

⚠ Requires markup changes at time of upgrade in dependent projects ⚠
USWDS 2.7.0 updates markup guidance for numeric input fields:

Improved mobile experience for numeric fields. We updated our guidance and code for numeric fields to follow the lead of gov.uk’s recent guidance and research. This updates those fields to use text rather than number inputs with an inputmode of numeric. (https://github.com/uswds/uswds/pull/3392)

https://designsystem.digital.gov/about/releases/#version-270
Examples updated in: 231ebd3
This will require corresponding markup changes in consuming projects at time of upgrade.

Acknowledged and expected change: USWDS 2.7.0 fixes appearance of skip link to appear as link. This is desirable, no changes are necessary, and consuming projects will benefit immediately by upgrading.

Before
After

⚠ Requires markup changes at time of upgrade in dependent projects ⚠
USWDS 2.7.1 updates markup of SVG images to include role attribute.

Added role="img" to SVG images. (https://github.com/uswds/uswds/pull/3501)

https://designsystem.digital.gov/about/releases/#version-271
Examples updated in: a67fff0
This will require corresponding markup changes in consuming projects at time of upgrade. As a general accessibility best-practice, it could be implemented independently from the LGDS upgrade in consuming projects.

Marking this as ready for review, since all necessary changes from upgrades should be accounted for.
I'm planning to include a CHANGELOG.md file to help document some of the above-noted required changes for consuming projects.
Also noting: The build failure for ci/circleci: visual-regression is expected, as described in above "Acknowledged and expected change" notes and by visual regression tests documentation.

⚠ Requires markup changes at time of upgrade in dependent projects ⚠
USWDS 2.8.0 updates markup of the "official government website" banner.
https://designsystem.digital.gov/about/releases/#version-280
Examples updated in: 029b089
This will require corresponding markup changes in consuming projects at time of upgrade.

visual regression in the accordion but it looks like mostly padding and I'm OK with it if you are

Yeah, the prior comments worked to mitigate most of the major differences, but the remaining diff is attributed to a new line height calculation that's expected (https://github.com/18F/identity-style-guide/pull/159#issuecomment-709353682).

Couple more findings that will need reconciliation:

USWDS 2.8.0 includes a new Tooltip component, which conflicts with our custom component (live site, branch preview, USWDS component)

Suggestion: Remove our custom implementation, defer to USWDS, include note in CHANGELOG describing markup differences for upgrade.

USWDS 2.8.0 includes new Date Picker and Date Range Picker components. While these don't conflict with our date input component recommendation since the three-field input is still included in USWDS, there may need to be additional consideration if and how we propose to leverage these two new components.

Suggestion: Leave as-is for now.

USWDS 2.8.0 includes a new Tooltip component, which conflicts with our custom component (live site, branch preview, USWDS component)

Suggestion: Remove our custom implementation, defer to USWDS, include note in CHANGELOG describing markup differences for upgrade.

As far as impact is concerned, a naive search appears to reveal that we don't use it anywhere:
https://github.com/search?l=&q=usa-tooltip+user%3A18f&type=code
I can still document the change all the same.

Chatted with @nickttng on Slack regarding the notes in https://github.com/18F/identity-style-guide/pull/159#issuecomment-729209873:

For Date Picker and Date Range Picker components, we're in agreement to consider this if and when they become relevant for us to use and customize.
For Tooltip, given the fact that we don't currently use it anywhere, it seems reasonable to remove this for now to allow us to move forward with a release. If we discover a need for tooltips in the future, we can investigate how best to reconcile our guidance with USWDS'.

Another discovery I'd made is that USWDS introduces a Card component in Version 2.7.0. While there are some visual similarities to the LGDS Card component, the usage guidance is not quite the same, where the USWDS discourages standalone usage of a card.
While it's an unfortunate naming collision, the implementation of the LGDS card using custom lg--based classes means there should not be concern for conflicts in code. Thus, we can upgrade to the latest USWDS without concern for unexpected breakage. Technically, there's nothing to stop a developer from using both or either of lg-card and usa-card.
Recommendation: No action for now. In the future, consider how to reconcile where reconciliation could be one of (a) removing our card implementation in favor of USWDS, (b) renaming our card component if it should be distinct from USWDS card component, or (c) merge implementation and document guidance for different usage if desired to build upon USWDS card.
Note that future revisions or removal of this component should consider impact to dependent applications, where it is currently used in 18f/identity-dashboard and a test route in identity-idp (see search results).


Added Cascade XOR

Adds Cascade scheme to the XOR operation, as per #268.

Hi Matt, I wasn't planning to add a test for this PR, but can do so if you think that would be beneficial. I noticed that most of the bitwise operations don't have a test.
In any case, the CLA has been signed and submitted.

Ok, I'll look in to adding those tests shortly. Looks like I also committed something I shouldn't have a second ago so ignore that, reverting those changes as they're not ready yet. 🙃

Hi @PenguinGeorge, thanks very much for your work here, I'll look to get it merged soon.
The infoURL fields look great too, but they'd be better in a separate PR when you're ready. Cheers!

Created new PR (#342) for updated codebase.


Is it possible to render only the numbering of the titles without writing the numbers to the file?

Considering that introducing numbers that may change at any time into the title may cause some unexpected issues, is it possible to only render title numbers without writing them into the file?

I also believe this feature is necessary. At present, the plugin alters the file content directly. In other words, it modifies the actual content. A more suitable solution would be to utilize CSS techniques, such as the pseudo-element, to add numbering to the headings, thereby making the headings uneditable. Furthermore, the current speed of updating the numbering is not fast enough.

You can achieve this by adding the following to a custom CSS Snippet (see Settings -> Appearance -> CSS snippets, where you can open the appropriate folder, create a file, and add the content below):
.HyperMD-header-1 {
    counter-reset: h2counter;
}

.HyperMD-header-2:before {
    content: counter(h2counter) ".\0000a0\0000a0";
    counter-increment: h2counter;
}

.HyperMD-header-2 {
  counter-reset: h3counter;
}

.HyperMD-header-3:before {
    content: counter(h2counter) "." counter(h3counter) ".\0000a0\0000a0";
    counter-increment: h3counter;
}

.HyperMD-header-3 {
  counter-reset:h4counter;
}

.HyperMD-header-4:before {
  content: counter(h2counter) "." counter(h3counter) "." counter(h4counter) ".\0000a0\0000a0";
  counter-increment: h4counter;
}

This could probably pretty easily be incorporated into the plugin.

It was an intentional design choice of mine for this plugin to insert text and not use CSS. But maybe a good choice for a different plugin!


Start dockerized cluster with Consul service discovery

WIP

Current coverage is 66.40% (diff: 100%)

Merging #24 into master will not change coverage

@@             master        #24   diff @@
==========================================
  Files            32         32
  Lines           259        259
  Methods         254        254
  Messages          0          0
  Branches          5          5
==========================================
  Hits            172        172
  Misses           87         87
  Partials          0          0

Powered by Codecov. Last update 0ce24a8...a24f4cb


flutter-1.22-candidate.1 has dev framework build failures

framework hash: ce40de69b7b4f89c66d19c8dbd3bd86ae30f1bc6
seeing "Step('display builds.build(s) failed') (retcode: 1)" failures across a few builds @ https://ci.chromium.org/p/flutter/g/dev_framework/console
https://ci.chromium.org/p/flutter/builders/prod/Windows SDK Drone/1255? has CIPD('ensure_installed') failed with: failed to resolve flutter_internal/browsers/chrome/windows-amd64@latest (line 1): no such ref
cc @christopherfujino
chris, an aside, the dev engine builds always give infra failures because they can't upload the bits to cloud storage (as expected).  maybe we shouldn't build them at all?  it's a false failure and makes it hard to see if there are any real issues

Filed the "aside" as https://github.com/flutter/flutter/issues/64577


SeResNet50 as backbone

Hello, I have a question about using SeResNet50 as backbone, it seems like torch doesn't provide pre-trained SeResNet50 and I can't find one online.
from defaults.py
Options: 'imagenet' or 'self'
_C.MODEL.PRETRAIN_CHOICE = 'imagenet'
I can only choose 'imagenet' or 'self', but I don't have them either.
In this situation, how should I set  this _C.MODEL.PRETRAIN_PATH value?  What I need to do if I simply want to start training with random initialization?

I just find the model  from here
https://github.com/Cadene/pretrained-models.pytorch/blob/master/pretrainedmodels/models/senet.py
Another question is if I want to only save the backbone model after training the model, what should I do?, is there a interface for this?

I consider you can delete all 'torch.load' and 'load_param' in your setting.

Thank you very much


(feature request) More support for conditional installation

First of all, apologies if what I'm requesting is already possible! I did my best to search for anything relevant.
Context
We make use of Tensorflow in our codebase, which has the annoying quirk that there is a GPU-enabled wheel called "tensorflow-gpu" and a CPU-only wheel called "tensorflow". The GPU-enabled package will crash on a system without the CUDA libs installed. The wheels also share most of their package structure and seem to clobber each other if you accidentally install both. Some of that behaviour is probably a bit un-Pythonic, but that's what has been shipped.
Now, ideally Google would build a single wheel that gracefully falls back to CPU-only if CUDA insn't installed, but that probably isn't happening any time soon. Their official solution seems to be not to specify TF as a dependency for anything, and require the user to manually install the correct build [1]. This is less-than-ideal for configuration management.
Feature Request
I'd very much like it if I could set up one Pipfile that defaults to installing the CPU package unless I set some custom flag indicating GPU is available.
Discussion
The closest existing functionality I'm aware of is the support for environment markers - if there was an extension to allow user-defined markers then this could be a fairly clean solution. I'm unsure how pipenv handles the lockfiles when there are multiple possible configurations, but this would continue working the same way.
As a rough example of what I'm suggesting:
Pipfile:
[packages]

tensorflow-gpu = { version = "*", gpu_enabled = "=='1'" }
tensorflow = { version = "*", gpu_enabled = "==''"  }

Installing CPU version: pipenv install
Installing GPU version: pipenv --env_marker "gpu_enabled=1" install
[1] https://github.com/deepmind/sonnet "Sonnet will work with both the CPU and GPU version of tensorflow, but to allow for that it does not list Tensorflow as a requirement, so you need to install Tensorflow separately if you haven't already done so."
EDIT: Another downside to Google's approach that just occurred to me: manually installing tensorflow will really not work well with pipenv - every time the env gets purged, tensorflow will need to be manually installed again.

we currently support all the PEP 508 markers, so I'd start with getting gpu_enabled added to that list.

@kennethreitz do you support the "extra" marker?

@kennethreitz: Can you give some pointers on how to use PEP 508 markers for people without experience in this domain. I have exactly the same issue as the OP.

@Froskekongen What exactly conditions are you looking for? The linked PEP 508 itself is already very informational on available markers, and Pipfile’s readme already showcases how they can be used in Pipfile.

@uranusjr: It would be great, for instance, to showcase how one could solve this issue using "extra". Would something along these lines work, e.g.:
tensorflow_gpu = {*, markers="extra = 'gpu'"}
tensorflow = {*, markers="extra = 'cpu'"}

And how do I then set the extra marker?

I guess another way of doing this would be to extend the [dev-packages] support to add extra sets of packages that are optional.

@ed-alertedh: Did you contact python developers about adding gpu_enabled in PEP 508?

@Froskekongen I don't really want to go down that route. "gpu_enabled" seems like a fairly ill-defined ad-hoc flag to add to a multi-platform standard. "cuda_enabled" would be easier to define but is even more ad-hoc - I suspect the suggestion would be thrown out even if we did try to get it into the spec.
Also it turns out "extras" would not help here - that just allows a package to specify optional dependencies.
Basically, I can already tell pipenv to only install a package if it's running on Windows. The best example of this in our codebase is:
numpy = { version = "==1.13.3", os_name = "!='nt'", index="pypi" }
"c500f7b" = {path = "numpy-1.13.3+mkl-cp36-cp36m-win_amd64.whl", os_name = "=='nt'"}

My original request was simply a way to do the same thing, except instead of looking at os_name, it would look at a user-defined variable.

@ed-alertedh: Why do you say that "cuda_enabled" or "cuda_installed" is ad_hoc? I think that gpu-computing is fairly common, and dependence on cuda is very prevalent.
@kennethreitz: Can you please open this issue until there is a proper fix or workaround for this use case?
For me the use case is something like:
pipenv run python -m some_module.some_submodule --training_data_location=path --parameters=otherpath
Typically the many development platforms and jenkins will not have cuda installed on the systems, but the code still needs to work.

I would also like this feature to be extended. In my use case, i have some extra packages that I need only installed at some deployment environments.
It's a django app, and I import extra features depending on purpose of the server being installed. I would like to set an environment variable that flags the packages for installation. Maybe this is already available with PEP 508 support, but I cant figure out how can I do this. If It is doable I think the current documentation could be extended with this example.

you have to accept the constraints of the system.

I don't understand what is the constraint, is it the PEP 508 specification?

@kennethreitz as a matter of constructive feedback (which you can feel free to disregard), it may have been easiest to just (a) confirm that this use-case is not currently possible and then (b) state that it's not something you plan to implement. As a bonus (c) indicate whether contributions would be welcome or not.
Thanks for all your work on this tool, it's a huge improvement over wrestling with requirements.txt

I'm just saying that you need to read the documentation, here:
https://docs.pipenv.org/advanced/#specifying-basically-anything
And work within those constraints.

Sorry to comment in a closed issue, but it seems the most appropriate place for me.
In addition to @ed-alertedh
Is it possible to specify some extra flags based on some conditions? I would like to specify some pip flags (in this case --no-binary) based on some condition, e.g. current OS:
somoclu = {version = "*", sys_platform = "== 'win32'"}
somoclu = {version = "*", sys_platform = "== 'linux'",no-binary=":all:"}

Is there any possibility or workaround to this except specifying the wheel file directly?

Pip arguments get dropped in from the environment currently rather than being configured in the Pipfile. Just set PIP_NO_BINARY

Right the Pipfile doesn’t interpret no-binary and pipenv doesn’t either, I believe you can set the PIP_NO_BINARY variable and then set PIP_ONLY_BINARY to the package you want to get as a wheel

Would I would have to set PIP_NO_BINARY="somuclu" globally on every Linux machine I am running this on? This seems weird to me.
If yes, can I somehow incorporate PIP_NO_BINARY="somuclu" directly into the Pipfile?
(Thanks for the quick replies and sorry if I don't understand...)

I'm not sure of what you need, but I believe this may help:
https://docs.pipenv.org/advanced/#injecting-credentials-into-pipfiles-via-environment-variables
https://docs.pipenv.org/advanced/#automatic-loading-of-env
On Wed, 11 Apr 2018, 15:04 Timo Klerx<EMAIL_ADDRESS>wrote:

Would I would have to set PIP_NO_BINARY="somuclu" globally on every Linux
machine I am running this on? This seems weird to me.
If yes, can I somehow incorporate PIP_NO_BINARY="somuclu" directly into
the Pipfile?
(Thanks for the quick replies and sorry if I don't understand...)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pypa/pipenv/issues/1353#issuecomment-380463905, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADlMQ6Z6Dn3pbJzyo54lnFVyFhd0Pi9eks5tng1-gaJpZM4Rqqfb
.

Actually, somoclu fails to run smoothly on Linux using the wheel provided by pypi.org (some shared object is not installed; at least on my system), but compiling it myself (using pip install --no-binary :all:) works fine.
On Windows, compiling somoclu (and anything else) is always a mess. So I wanted to have a single Pipfile that installs everything smoothly on any (new) client (with any operating system) when calling pipenv install
Hence, I wanted to add OS-specific flags to the installation of somoclu.

No, we don't do os-specific pip-flag interpolation (I'm sure you understand this is a very specific edge case) -- and yeah I'd probably recommend handling this with an environment variable on your linux machine, since that's the most straightforward approach.  I doubt this is true on all linux machines, so I don't know that it's really fair to say you'd have to go do this globally on every linux machine you own, but on the ones that can't use the wheel, sure.

At the risk of throwing more "gold-plated" features into the mix... How difficult would it be to make pipenv think it is running on a different platform?
To keep my various lockfiles in sync I opted to set up a Docker container to lock the linux packages which I can run straight after I lock my windows packages. It would be really cool if I could do a "dry-run" and generate the lock file for another platform simply by changing the env markers. Although I guess there would be no check that the packages actually installed successfully by doing it that way.

Locking results should be the same regardless of your host platform. Pipenv currently can’t do this, but is constantly being improved toward the goal. Your proposal would be a step to the wrong direction, hiding the underlying problems instead of solving them.

@uranusjr Unfortunately due to the "extras" field in many packages, you will get a slightly different dependency graph on different platforms. E.g. I noticed jupyter pulls in pexpect on Linux but not on Windows. Trying to restore a lockfile with pexpect on Windows results in an error since the package is not available on that platform.

@ed-alertedh to be clear I really want this functionality and in many cases we actually do support this — it just depends on how the package maintainer has chosen to indicate their dependencies. If they do it dynamically via actual code then we are SOL.
With the latest release you should be able to keep all the markers and they will just drop through to the lockfile in theory

@techalchemy You're doing a great job with what you have to work with 😄

@techalchemy Yes, it is an edge case and I also agree: Great work!
Just wanted to know whether it is possible somehow.
Thanks alot!

I've read everything thrice, yet I fail to get anything constructive from this post.
I have exactly the same issue as OP, an application that runs with TF-CPU on dev machines and TF-GPU on production machines. I need to package it into a docker container (the ones from nvidia with TF already installed), and basically I need to tell pipenv to NOT install tensorflow since it is already provided inside the base container.
The environment markers just don't cut it, since docker containers have same markers than linux machines (sys.platform=linux,os.name=posix).
What is the recommended workaround ? How can I tell pipenv to not install a given package in Docker ?

@Overdrivr Whilst this is far from an ideal setup, this is how I'm currently handling it: https://github.com/ed-alertedh/multiple-pipenv-example

Hey that’s pretty clever, thanks for putting that example together! That really helps me understand also. One side comment which wasnt possible when this was originally discussed— if you give each Pipfile and Lockfile its own folder, you can just set PIPENV_PIPFILE to point at the right Pipfile and it should figure things out for you. I’m not totally sure about that, but I’m guessing.

Read.files

@techalchemy Oh, that's good to know, when I get a chance I might update the scripts to take advantage of that new feature! Really the biggest issue I have with this is that we can't take advantage of pipenv's CLI tools for editing the Pipfile (e.g. pipenv install), otherwise it is working reasonably well for us. We also find that updating the lockfiles on Windows is signficantly slower than on Linux, but I'm guessing a lot of that probably relates to pip's performance on Windows.

@kingmoduk please stop spamming the project.

@ed-alertedh Thanks for sharing.
I haven't had the time to try it yet, but I'm thinking about moving TensorFlow to dev-packages. That way I can install a deploy version of my package in Docker that does not have TF in it.

I realize this could be also useful between dev and test environment for modules.
In dev mode, the module must be installed in editable mode -e . to make development easy.
However in test environment (CI test runner like Jenkins or GitlabCI), it would be better to install to install the package directly from the wheel, to be closer to how the package will be used in reality.
@kennethreitz Any recommendations for this kind of scenario ?


[Feature Request] Default language for audio stream / subtitles

Some videos have multiple audio streams and subtitles for different languages.
It would be nice if we could specify a default language so that they're picked automatically when opening a video.
mpv-android has this for reference.

I've checked your player seems to enable "Forced" streams to be the Default.
If there's no "Forced" streams, no subtitles will be enabled.
How do we ALWAYS enable subtitles?

This was requested here https://github.com/moneytoo/Player/issues/200 (at least the audio part). I completely agree with this request, it would be nice not to have to choose the audio/subtitle track all the time.

Created an account just to +1 this, especially the subtitles part. I love this player, more than MX and Exo even, and my only gripe is having to manually choose subtitles every episode/movie.

Could you please add the option for “none” or at least a setting to default to off on the Android TV version? Subtitles are always on, and even when I turn them off they will turn back on after a pause goes into a screensaver.
This is on a Nvidia Shield.

Available now in the latest release.

Could you please add the option for “none” or at least a setting to default to off on the Android TV version? Subtitles are always on, and even when I turn them off they will turn back on after a pause goes into a screensaver.
This is on a Nvidia Shield.

Something like that shouldn't happen. Unless the app is instantly killed by the system, subtitle selection should be saved and restored when relaunching the app. Last few stable versions have issue with forced subtitle tracks. It's resolved now in latest betas as well as you should be able to use these new options. However let me know if you have something more I could go on (as so far it works fine for me).

The issue appears to be resolved with .132. Video starts with no subtitles enabled- paused into screensaver came back, still off.


webpack build gets stuck when using certain markdown syntax

[x] I confirm that this is an issue rather than a question.

Bug report
Version
v0.6.7
Steps to reproduce
Download this and follow the repro steps in README.md
It gets stuck when using $$\frac{A_1B_2}{C}$$ in markdown with @iktakahiro/markdown-it-katex

What is expected?
webpack build succeeds.
What is actually happening?
webpack build gets stuck.
Other relevant information

Result of running saber -v:
Browser version (optional):
Is Saber a global or local install?
Which package manager did you use for the install?
Does this issue occur when all plugins are disabled?

IssueHunt Summary

Sponsors (Total: $100.00)

 issuehunt ($100.00)

Become a sponsor now!
Or submit a pull request to get the deposits!
Tips

Checkout the Issuehunt explorer to discover more funded issues.
Need some help from other developers? Add your repositories on IssueHunt to raise funds.

info Using config file: saber-config.yml
success Compiled client successfully in 5m 9.3s!
info Available at http://localhost:3000
info 80.1 MB memory used

It's not that it get stuck it just take 5m 9.3s 🤣

I guess that there is a promise that resolve after a timeout

After tracking down this I ended up here
https://github.com/vuejs/component-compiler-utils/blob/442d7b221d2075b2b53d8f6d4e1a4b4e6ab911e3/lib/compileTemplate.ts#L167-L178
that's why it takes so much time in dev mode.
I don't know if this should be reported to prettier or not

The issue https://github.com/prettier/prettier/issues/4784

A work around is to set prettify: false in vue-loader options

This should be fixed in the next release, no ETA yet, probably in a couple of weeks, but I'll try to cut one before that

https://github.com/prettier/prettier/issues/4784#issuecomment-502602463


Fix secret name for openshift mode

Upgrading the gatekeeper causes cert-issue due to the fact that the same secret name has been created by the service-ca already
Ref: https://issues.redhat.com/browse/ACM-10859
Signed-off-by: yiraeChristineKim<EMAIL_ADDRESS>

@yiraeChristineKim this looks good to me after the log messages are fixed.

I think this PR should be in main and then cherry-picked to release-3.14? (You should be able to edit the target branch.)

Great!

Oops--it might need a rebase. Sorry about that--I was hoping it'd be more straightforward. 😞


tracing logging for better debug

What problem does this feature solve?
better dx
What does the proposed API of configuration look like?
reference:
https://rustc-dev-guide.rust-lang.org/tracing.html#using-tracing-to-debug-the-compiler

@bvanjoi Did this pull request fits your need? https://github.com/web-infra-dev/rspack/pull/3002

Closed due to stale and unclear feature request.


[Question] Disabling L1 cache for specific variable

Hi,
To test L2 cache, i would like to disable usage of L1 for specific variable in device functions. In CUDA, i can do it like this:
__device__ __inline__ double ld_gbl_cg(const double *addr) {
  double return_value;
  asm("ld.global.cg.f64 %0, [%1];" : "=d"(return_value) : "l"(addr));
  return return_value;
}

What and how should i inline to acquire same result with HIP? Checked ISA GFX9, GFX10 docs, found only simple load instructions and 0 data whether L1 / L2 is used.
--
Specs:
Rocm 4.5.0
GPU: RX 5500 XT
Arch: gfx1012
OS: Ubuntu 20.04

What exactly are you trying to do here? If you can give us some context we might be able to help you better with it.

Hi, @jatinx !
In general, i would like to measure L2 cache latency, and for that i would like to bypass L1 and load variable only to L2. More specifically, i would like to use an instruction like CUDA-specific ld.cg also on AMD GPUs.
What i have found out, is that for AMD GPU's load_dword with dlc = 1 is required, but i'm not sure, which load_dword instruction would suit best here.

We advice against writing asm in HIP kernels.
What you might be looking for in non temporal loads: https://clang.llvm.org/docs/LanguageExtensions.html#non-temporal-load-store-builtins

@HardMax71 Please advise if we can go ahead and close this ticket? Thanks!

@ppanchad-amd yes, you're free to close it. Thank you!


Remove external API calls in TestPassword

The test had a dependency on https://api.pwnedpasswords.com which caused many failures on CI recently:
--- FAIL: TestPassword (2.37s)
    pwn_test.go:41: Get "https://api.pwnedpasswords.com/range/e6b6a": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
FAIL
coverage: 82.9% of statements

This mocks the service using gock and tests one positive and one negative case, removing this dependency and removing the need to have a password generator in the test because we control the responses.
Regarding concerns about compatibility, i think it's totally fine to mock this because if that service introduces a breaking change in their API, they better make a /v2/ path or similar.

Rather than introduce a new dep we could just pull out the parse logic and make unit tests for this instead?

Rather than introduce a new dep we could just pull out the parse logic and make unit tests for that instead?

But is it a realistic test? I use such http mocks all the time, I think they are great.

If the mocking framework becomes a problem, it can be replaced with the more manual way of https://pkg.go.dev/net/http/httptest but I don't see that currently being necessary.


solution for investiny 403

please use which is working api
import requests
from requests.structures import CaseInsensitiveDict
while True:
i=['1138417','17955','101810','160',]
for k in i:
url = f"https://tvc6.investing.com/9368e857cc51ddcae69108bd8a3b6d49/1664515691/56/56/23/history?symbol={k}&resolution=D&from=1633411692&to=1664515752"
    headers = CaseInsensitiveDict()
    headers["authority"] = "tvc4.investing.com"
    headers["accept"] = "*/*"
    headers["accept-language"] = "en-US,en;q=0.9"
    headers["content-type"] = "text/plain"
    headers["origin"] = "https://tvc-invdn-com.investing.com"
    headers["referer"] = "https://tvc-invdn-com.investing.com/"
    headers["sec-fetch-dest"] = "empty"
    headers["sec-fetch-mode"] = "cors"
    headers["sec-fetch-site"] = "same-site"
    headers["user-agent"] = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/104.0.5112.102 Safari/537.36"

    resp = requests.get(url, headers=headers)

    print(resp.status_code)
    time.sleep(1)
    if resp.status_code==200:
        print(resp.json())

to make reliable solution we need to switch between multi api

Thanks, @ramakrishnamekala129 already been fixed, and released, it's available at https://github.com/alvarobartt/investiny/releases/tag/v0.7.2, should be installable in some minutes with pip install investiny==0.7.2 :hugs:

tvc6 is working fine they banned tvc4

I know that you already shared this URL with me in the past, but where did you get that URL from? Just by randomly testing different tvc combinations, or it's just the Investing.com page from another country?

FYI after some testing, it seems that tvc4 is working again! 🎉

Since investpy stopped working my main stopped working so I started searching backdoor in investing.com so I got tvc4 in Indian  site after so made request methods which worked fine for 3 to4 days after that I sent so that repair investpy along with that after searching countries investing.com then I got tvc6 Australia which had some sharing between American investing so concluded that they work differently.

Hey guys, I'm using the investpy 7.2.0 but I got the same preview error: 403.
Any clue to how deal with this problem?

Let me check this again later, as it shouldn't happen any more as far as I could test... Can you try with investiny 0.7.1? Thanks!

Ok already tested it @Emmanuel1524 aaaaand... bad news, all the Investing.com APIs seem to be Cloudflare protected now... let me check for alternative solutions as proposed at #56 😞


lodash huge files

is needed load complete this library, i dont understand what happend with this line
store.inject = _.partial(inject, store, _, persistConfig); this is neded ? no can use only the one function ?

i search and i find this solution,
import partial from 'lodash/partial';
const ph = partial.placeholder;
const inject = partial(fn, ph, fn2);

i dont sure is this working same to the old format, i dont ffamiliarize with lodash

store.inject = _.partial(inject, store, _, persistConfig);

has been replaced by something like that
store.inject = reducers => inject(store, reducers, persistConfig);


Light Detection Sensor is shown as motion sensor HomeKit

Xiaomi/LUMI Light Detection Sensor
homebridge-deconz v0.1.2
From Homebridge log
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: logLevel: set to 2
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Name: set to "LUX Køkken"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Serial Number: set to "04CF8CDF3C7D3019"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Manufacturer: set to "LUMI"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Model: set to "lumi.sen_ill.mgl01"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Firmware Revision: set to "0.0.0_0021"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: LUMI lumi.sen_ill.mgl01 v0.0.0_0021 (1 resources)
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Name: set to "LUX Køkken"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Configured Name: set to "LUX Køkken"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Enabled: set to true
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Status Active: set to true
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Status Fault: set to 0
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Current Ambient Light Level: set to 36 lux
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Dark: set to false
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken: Daylight: set to false
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken Battery: Name: set to "LUX Køkken Battery"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken Battery: Configured Name: set to "LUX Køkken Battery"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken Battery: Battery Level: set to 100%
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken Battery: Charging State: set to 2
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken Battery: Status Low Battery: set to 0
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken Battery: Low Battery Threshold: set to 20%

[03/02/2023, 09:53:43] [deCONZ] LUX Køkken Motion: Name: set to "LUX Køkken Motion"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken Motion: Configured Name: set to "LUX Køkken Motion"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken Motion: Motion Detected: set to 0

[03/02/2023, 09:53:43] [deCONZ] LUX Køkken History: Name: set to "LUX Køkken History"
[03/02/2023, 09:53:43] [deCONZ] LUX Køkken History: Configured Name: set to "LUX Køkken History"

From terminal
curl -s http://<IP_ADDRESS>:34699/gateways/00212EFFFF04DB5F/devices/04CF8CDF3C7D3019 | jq
{
  "id": "04CF8CDF3C7D3019",
  "manufacturer": "LUMI",
  "model": "lumi.sen_ill.mgl01",
  "name": "LUX Køkken",
  "resources": [
    "/sensors/18"
  ],
  "settings": {
    "expose": true
  },
  "type": "sensors",
  "zigbee": true
}

API info from Phoscon App
{
    "config": {
        "battery": 100,
        "on": true,
        "reachable": true,
        "tholddark": 12000,
        "tholdoffset": 7000
    },
    "ep": 1,
    "etag": "0a052af0cb5b483a3c2ffbde992ef591",
    "lastannounced": null,
    "lastseen": "2023-02-03T09:01Z",
    "manufacturername": "LUMI",
    "modelid": "lumi.sen_ill.mgl01",
    "name": "LUX Køkken",
    "state": {
        "dark": false,
        "daylight": false,
        "lastupdated": "2023-02-03T09:01:22.691",
        "lightlevel": 16813,
        "lux": 47
    },
    "swversion": "0.0.0_0021",
    "type": "ZHALightLevel",
    "uniqueid": "04:cf:8c:df:3c:7d:30:19-01-0400"
}

By design, Homebridge deCONZ exposes a hidden dummy motion sensor so Eve shows the history for Light Level.  I think Home ignores the hidden setting.  Eve honours it, but still shows the dummy motion sensor in the Types view.

That was not the case with homebridge-deconz v0.1.0 and I have never had a motion sensor in HomeKit

Homebridge deCONZ is still under development; new features are added with each version.

But why expose a motion sensor when it is not present. It's going to be a mess in HomeKit

See above: so Eve shows the history for Light Level.


USER - Allow users to upload profile pictures

Pausing work on this because we are blocked -- need access to Luisa's AWS account in order to use S3. Will leave a note in user-profile.html on current PR (branch called Feature/upload-profile-picture).


Find the Primorial

Before jumping into Pr first comment for assign.All Problem from Edabit. Link is mandatory to add
Problem statement: A Primorial is a product of the first n prime numbers (e.g. 2 x 3 x 5 = 30). 2, 3, 5, 7, 11, 13 are prime numbers. If n was 3, you'd multiply 2 x 3 x 5 = 30 or Primorial = 30.
Create a function that returns the Primorial of a number.
Problem Link: https://edabit.com/challenge/fRjfrCYXWJAaQqFXF

Before submitting a PR please Check some details.Check Edabit for problem task

[ ] fileName formet should be in camelcase
example- returnTheNextNumber.py
[ ]  Add label for Problem
[ ]  In file there should be problem statement and link to probelm
example -

'''
    Probem Task : This program will add two numbers
    Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
'''
*/
def main():
    return a + b

This repository is vaild for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck,

@aayushi-droid  Please Assign this issue to me... I will do my best .. :)
Add Hard label also.

I would love to work on this.

I would like to work on this.

hey! pls, assign to me. I would love to work on this.

I wanna work on this code generation. Please assign to me

I would love to work on this and I am good at problem-solving. I have a gold badge on hackerrank in problem-solving.You can check:https://www.hackerrank.com/Gvirmani1505.

Can I work on this problem?

Can I work on this problem?

I will solve this. Over and Out!

Please assign. I want to solve this :)

This is very easy question can i try ?Please assign me

Hi, can I be assigned? I want to contribute, it would be my first time contributing to a project.

Can I work on this problem? Thank you!

Assign me the task!
I have completed this challenge.
Waiting for assignment!

Thank you so much for the opportunity . I did it and now you can merge it ,if you want anything please inform me

Can you please guide me how to submit a pull request.
Like the steps to make it.
Should I clone your repo and merge it with my code attached?
On Fri, 2 Oct 2020, 01:33 Gautam-virmani<EMAIL_ADDRESS>wrote:

Thank you so much for the opportunity . I did it and now you can merge it
,if you want anything please inform me
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Py-Droid/Python-Thunder/issues/42#issuecomment-702367082,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN6MCVFJMJZPZ2WPCMMQ6Z3SITOAZANCNFSM4R74Y5FA
.


feat: API token auth - [merged]

Merges api-token-auth -> development

added 2m of time spent

mentioned in merge request nofusscomputing/infrastructure/configuration-management/project-production!48

added 1 commit
962ae2b8 - feat(api): initial token authentication implementation
Compare with previous version

mentioned in commit 9f0e03880bb3c956a809959f64659c5996d4e77d

mentioned in commit 962ae2b8dfaf7cccdfd449e2a7db087f9b3542c9

mentioned in commit nofusscomputing/infrastructure/configuration-management/playbooks-production@65de7e8cfdd5c85c46ffb5e5fce403460bc8061c

added 1h 11m of time spent


refactor: change package name

:tada: This PR is included in version 0.0.1 :tada:
The release is available on:

GitHub release
v0.0.1

Your semantic-release bot :package::rocket:


⚠ Secret Website 3 has degraded performance

In 035d05b, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 3160 ms

Resolved: Secret Website 3 performance has improved in 3c95eb7.


Feat: OAuth 2.0 SAML Bearer Assertion Flow

Describe the Feature
Our app is using this plugin to login to Azure AD. We would like to connect to an Enterprise system using SAML bearer assertion flow. Is there an ETA on support for this?
Platform(s) Support Requested

Web
IOS
Android

Hi,
this does not sound like a OAuth 2/Open ID standard flow.
https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-saml-bearer-assertion
This is a generic plugin for OAuth 2 so you have to read into the specifics of your providers flow and configure it. As soon as there is a working config I’m happy to add that to the docs for others to see.
So I guess you already tried it and therefore plz provide your configuration for it and tell us what part did not work.
Although I don’t not Microsoft’s  intentions for this flow but just a word to SAML (2.0): It is a completely different mechanism, which is simply out of scope of a OAuth 2/Open ID plugin.
BR Michael

I see, thank you for your prompt response :)


Icon request: Wealthfront

URL of the site or app (required): https://www.wealthfront.com/
Logo resources for the icon (optional, but recommended):

SVG: https://commons.wikimedia.org/wiki/File:Wealthfront_Logo.svg

Here you go!
https://github.com/aegis-icons/aegis-icons/blob/master/PNG/Wealthfront.png


ISSUE: Menu active checker doesn't work when url params applied

Example:
adminlte.php
url: admin/categories
active url: admin/categories?filter=active
doesnt trigger the menu.

https://github.com/jeroennoten/Laravel-AdminLTE/pull/226

closed with pr #226


strict boundary rules can return more matches than relaxed boundary rules

greedy / eager regex matching rules strike again! and result in some counter-intuitive behavior
with strict boundary rules, the constructed regex is different (more precise), and ends up matching the later "a" at the word boundary. with loose rules, the earlier "a" is matched but we discard the whole match because that "a" doesnt land on a boundary. when there are no boundary rules we generate the same regex as in loose, but don't discard:
needle: fi co ba de a
match with interLft: 0:
Complete Figure Drawing Course HD: 025 - Understanding Basic Design of Scapula and Formulas`
match with interLft: 1:
NOTHING (discarded the above match)
match with interLft: 2:
Complete Figure Drawing Course HD: 025 - Understanding Basic Design of Scapula and Formulas`
kinda of an edge case, but may be fixable.
unfortunately, JS regexp flavor doesn't support inline modifiers for selective case sensitivity, which would be the proper solution for loose bound rules: https://www.regular-expressions.info/modifiers.html
i think the solution will be pretty much similar to https://github.com/leeoniya/uFuzzy/issues/18. if we're about to discard a match due to a loose boundary rule, we can scan forward inter-junk using a /y sticky case-sensitive regex like /[\b_\d]app/gy (interLft: 1) and/or /app[\b_\dA-Z]/gy

linked commit improves the situation, so the discard doesn't happen.
however, the highlighting will still be wrong without additional range query refinement, which will require adding a peudo-lookahead into the junk before the exact term, to make sure that query picks up the correct term match. that requires deeper changes to the needle prepQuery and how it joins terms + inserts inter junk allowances.
https://github.com/leeoniya/uFuzzy/blob/2ab962bd36e135f5ab56e9a37d239654ba66c08b/src/uFuzzy.js#L480


Add shiawase VIP (Piano cover) by dion timmer

Song Name
shiawase VIP (Piano cover)
Artist Name
dion timmer
Source
Youtube
Youtube Link
qG-mxxkjBWo
Direct File Link
No response
Song ID
1266014
Start Offset [ms]
3000
Extra
Created from Auto Nong.
Created for level: 86407629
Downloaded: Yes

accept


Add model, factory, and migration

@drewroberts As with the notes package, should  the this package contain the same HasPackageTrait from the support package?
Also I'm assuming the Status model should also extend the BaseModel class?

@drewroberts As with the Notes package, should this package have the Status model also extend the BaseModel class?

Yes. Thank you


Integration Test - Release - 2019/05/15 13:21:36

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@bharney-msft : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


[no ticket] Upgrade to Ember* 3.4

Purpose
Upgrade to Ember* 3.4
devDependencies changes based on https://github.com/ember-cli/ember-new-output/blob/v3.4.3/package.json
Summary of Changes
Changed

Models:

draft-registration

define inverse for node relationship as draftRegistrations instead of null (required by ember-data 3.4)

preprint

define inverse for node relationship as preprints instead of null (required by ember-data 3.4)

MIsc:

upgrade to<EMAIL_ADDRESS>

Side Effects
Hopefully not. 🤞
Feature Flags
n/a
QA Notes
Full cross-browser regression test of all Embosf pages recommended.
Ticket
n/a
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Pull Request Test Coverage Report for Build 1603

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

Totals

Change from base Build 1594:
0.0%

Covered Lines:
3208

Relevant Lines:
4447

💛  - Coveralls


feat(get-started): Add change assets path page

Just saw it regenerated the radio snap. WIll try again


[GTPU] tunnel_gtpu controller

Hi,
I'm trying set up a controller for GTP-U traffic but when running ryu-manager tunnel_gtpu.py I'm facing with the following error:
TunnelGTPU: Exception occurred during handler processing. Backtrace from offending handler [switch_features_handler] servicing event [EventOFPSwitchFeatures] follows. Traceback (most recent call last):   File "/home/user/.local/lib/python2.7/site-packages/ryu/base/app_manager.py", line 290, in _event_loop     handler(ev)   File "/home/user/ryu-lagopus-ext/ryu/app/tunnel_gtpu.py", line 21, in switch_features_handler     type_eth  = (ofp.OFPHTN_ONF << 16) | ofp.OFPHTO_ETHERNET AttributeError: 'module' object has no attribute 'OFPHTN_ONF'
EDIT: OFPHTN_ONF is defined in ofproto_v1_5.py but this controller is using ofproto_v1_3.py and that is why the needed attribute is not found.
Any suggestion about how to handle this situation?
Regards,
Ginés.

Currently, Lagopus tunnel support is not used ONF standard match/action.
Try https://github.com/lagopus/ryu-lagopua-ext instead of original Ryu.
Thank you.

Hi ,
I try  https://github.com/lagopus/ryu-lagopus-ext instead of original Ryu.
but still Encounter the same problems
How to solve about this problem?

We defined OFPHTN_ONF in ryu/ofproto/ofproto_v1_3.py.
https://github.com/lagopus/ryu-lagopus-ext/blob/lagopus-general-tunnel-ext/ryu/ofproto/ofproto_v1_3.py#L809-L824
Your ryu application may be running with original ryu-manager.
Please try re-installing ryu of ryu-lagopus-ext branch.
Thank you.


chore/deprecation-of-ioutil

What kind of change does this PR introduce?
A small refactor that removes deprecated io/ioutil package and uses io package instead.
What is the current behavior?
Currently client.go and execute.go uses the deprecated package io/ioutil. Although io/ioutil will continue to work as expected, the go team encorages the usage of io and os packages as of go version 1.16. More info can be found here: https://golang.org/doc/go1.16#ioutil
What is the new behavior?
The behavior with io.ReadAll is the same as deprecated ioutil.ReadAll.
Additional context
No additional context 😅

Looks like golangci-lint is saying:

ReadAll not declared by package io

I checked the golangci.yml file and it appears the go version is set to 1.16. It also looks like we are usuing golangci version 1.38.0.
According to this, golangci supports go version 1.16. Unsure why the linter is not recognizing ReadAll as being a part of io package 😳

Looks like the reviewdog action is saying the following:
[golangci] reported by reviewdog 🐶 ReadAll not declared by package io (typecheck)
According to the reviewdog.yml file, we are not specifying a go version as specified here
Going to add a go version to the reviewdog.yml file and see if that helps 🤞🏾

LGTM. Thanks for contribution 💯


AWS SdkClientException on upload to S3

A user reports that while comparing two regional analysis results, an error window pops up with the following text:
Error on the server com.amazonaws.SdkClientException: Unable to execute HTTP request: Timeout waiting for connection from pool.

Taking a closer look at the stack trace, we found this was happening while attempting to lazy-initialize and upload a probability surface to S3. We are not currently using these probability surfaces, and I believe they are all zeros, so this step is not really necessary and could be removed. But this error points to something more general happening: the S3 client library's connection pool is being exhausted.
Looking at Javadoc, the default connection pool size is 50. If 50 connections are being held open long enough to get a timeout, something is wrong - presumably some other part of Analysis is grabbing connections and never releasing them. For example, some operation could be failing without proper cleanup in a catch clause.
It is notable that the error disappears after a server restart. This is consistent with the idea of connections being held and not returned to the pool, as some operation fails repeatedly over time.

This also uses a mechanism where a file is uploaded to S3 and then an S3 URL passed back to the UI. We should just remove this endpoint from the backend and remove the HTTP fetches from the front end, replacing with a stub function that always returns a probability of 1, and/or just eliminating all the probability filtering from the front end for now.


Update package_stm_index.json

board was download failed in Arduino 1.8.16,
the error is
com.fasterxml.jackson.core.JsonParseException: Unexpected character ('<' (code 60)): expected a valid value (number, String, array, object, 'true', 'false' or 'null') at [Source: (FileInputStream); line: 7, column: 2],
the email was empty.

sorry, forget it


Datebase system is shut down

I using different machine to import and serve.
I run the docker container in my k8s cluster when I completed the importing.
But I received the 500 error code and error message with Failed to establish database connection:SQLSTATE[08006] [7] could not connect to server: No such file or directory\n\tIs the server running locally and accepting\n\tconnections on Unix domain socket \"/var/run/postgresql/.s.PGSQL.5432\"?" when I request search.php.
The server log as below
+ trap stopServices TERM
+ /app/config.sh
+ id nominatim
+ useradd -m -p J2VdAjzbBkWKO nominatim
+ IMPORT_FINISHED=/var/lib/postgresql/12/main/import-finished
+ TOKENIZER_DIR=/nominatim/tokenizer
+ '[' '!' -f /var/lib/postgresql/12/main/import-finished ']'
+ chown -R nominatim:nominatim /nominatim
+ '[' '!' -d /nominatim/tokenizer ']'
+ echo 'No tokenizer configuration found. Copying from persistent volume into project directory.'
No tokenizer configuration found. Copying from persistent volume into project directory.
+ cp -r /var/lib/postgresql/12/main/tokenizer /nominatim/tokenizer
+ chown -R nominatim:nominatim /nominatim
+ service postgresql start
* Starting PostgreSQL 12 database server
...done.
+ cd /nominatim
+ sudo -E -u nominatim nominatim refresh --website --functions
2022-06-27 12:57:08: Using project directory: /nominatim
2022-06-27 12:57:08: Create functions
2022-06-27 12:57:09: Setting up website directory at /nominatim/website
+ service apache2 start
* Starting Apache httpd web server apache2
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
*
+ wait
+ tail -f /var/log/postgresql/postgresql-12-main.log
2022-06-27 12:57:05.023 UTC [39] LOG: starting PostgreSQL 12.9 (Ubuntu 12.9-0ubuntu<IP_ADDRESS>) on x86_64-pc-linux-gnu, compiled by gcc (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0, 64-bit
2022-06-27 12:57:05.023 UTC [39] LOG: listening on IPv4 address "<IP_ADDRESS>", port 5432
2022-06-27 12:57:05.023 UTC [39] LOG: listening on IPv6 address "::", port 5432
2022-06-27 12:57:05.030 UTC [39] LOG: listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
2022-06-27 12:57:05.505 UTC [40] LOG: database system was interrupted; last known up at 2022-06-27 12:57:03 UTC
2022-06-27 12:57:05.554 UTC [40] LOG: database system was not properly shut down; automatic recovery in progress
2022-06-27 12:57:05.584 UTC [40] LOG: invalid record length at 12E/D7344178: wanted 24, got 0
2022-06-27 12:57:05.584 UTC [40] LOG: redo is not required
2022-06-27 12:57:05.697 UTC [39] LOG: database system is ready to accept connections
2022-06-27 12:58:05.794 UTC [39] LOG: could not open file "postmaster.pid": No such file or directory
2022-06-27 12:58:05.794 UTC [39] LOG: performing immediate shutdown because data directory lock file is invalid

the data import process log has gone but I can ensure that the process has executed completed and outputed the last log database system is ready to accept connections
Could anyone suggest  a solution? I dont want to import the planet data again because it took two weeks.

Looks like one of three possibilities:

Database files are not in the correct location
Not all database files are available
Database is corrupt

In case of 3, you might be able to repair the files using a Postgresql 12 docker image..

I repair the database by run the pg_resetwal command, but I still don't know how the wal file was corrupt somehow.


Provide Gradle quick start instructions on README

As a Gradle user, I'd like quick start instructions for Trampoline. This is done for Maven only.

Would love to see Gradle there, too :)

Sure, completely agree. I will review it and solve it as soon as possible. Thanks for your feedback!

solved on v3.4.
Moreover, now you can start Trampoline using ./gradlew (or gradlew.bat) bootRun.
Cheers!

Thank you very much!


run train.py error

hi，sir，thank your great project！I am trying your  ’train.py‘， but there exist two issues：
a.   the 'dwpose_conditions' from dwpose is numpy-type, but model need is torch-type
b. image_prompts is not used in training interface
Is there any miss in your uploaded code?

Thank you for bringing up this issue. It appears there are some version incompatibilities within the released code. I will investigate this matter and release an updated version shortly.

thank you very much for your rely!!! I fixed these issues above and run this code. But unfortunatly，I got divergent result  in ubc fashion dataset. Is there other problem in training？

I update the train.py
This version should work normally.
The former version uses fine-grained image prompts during training, but it is unstable.
We use null text prompts in the updated version, like MagicAnimate.

very appreciated for your help！thank you！

@Wangxuefeng92 Hi, I'd like to know do you use batch_size=1 or >1, I noticed that there are some comments in python file as "FIXME: only support train_batch_size=1".


console.assert is not defined

Is it possible to rebuild & rerelease videojs-kit?? hls.js has an upstream fix for https://github.com/video-dev/hls.js/issues/5218 that I can't pull in because videojs-kit appears to bundle the library at build time
I see the following errors in prod builds:

Hi @awinograd  -- thanks for opening. Let me see if we can get someone to update Hls.js dependency and release a new version


Question: Override google assistant (nabu casa edition)

Home Assistant release with the issue: 94.3
Last working Home Assistant release (if known): none
Operating environment (Hass.io/Docker/Windows/etc.): HassOS
Component/platform:
Google Assistant (cloud / nabu edition)
Description of problem:
This is a question, is there a way to override the google assistant component , the cloud edition
seems when i load the google_assistant as a custom, it has only effect on the local edition, not the cloud, the cloud is still using the original files
thnx

No, that's not possible. As the integration is certified by Google, we don't want people to go around and change it.


[ADAM-1709] Add ability to left normalize reads containing INDELs.

Resolves #1709. Opens protection on main method in NormalizationUtils to be package-private to org.bdgenomics.adam. Adds code to AlignmentRecordRDD exposing INDEL normalization as a transformation.

Coverage decreased (-0.4%) to 83.05% when pulling 8fcb03f95984406147efe377e4dd57c4bd18eb01 on fnothaft:issues/1709-left-normalize-indels into 51efbafef44062dc0616735923ebd3fbf6051db2 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/2354/
Test PASSed.

Thank you, @fnothaft


resolve build errors to make project compile

Hello :crab: ,
the current master branch doesn't compile, so I made small fixes to make it compile again.
Thank you for reviewing this PR :crab:

Good catch, the updates look good to me, many thanks for the fix!


Installation fails with setuptools>=75.0.0

Hi! Even though this repo doesn't seem active any more, I'll put this out here in case anyone else has an issue installing this package.
Problem description
Cannot install pyinterval with setuptools>=75.0.0 due to major changes in setuptools.
Crlibm, which is a direct dependency of pyinterval fails to install due to the changes in setuptools (specific error below)
My speccs

Apple M1 Max
32GB memory

Tested with: Python 3.10, 3.11
The same error occurs in a Github runner on  Ubuntu 22.04.5 LTS with Python 3.10
Installation failure shell output
pip install --no-cache pyinterval                                                                                ml/aint/ci [6c4db28870]
Looking in indexes: https://pypi.org/simple
Collecting pyinterval
  Downloading pyinterval-1.2.0.tar.gz (25 kB)
  Preparing metadata (setup.py) ... done
Collecting crlibm==1.*,>=1.0.3 (from pyinterval)
  Downloading crlibm-1.0.3.tar.gz (1.6 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.6/1.6 MB 1.4 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [6 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/private/var/folders/km/d_mh08b54j52lf5b16g476k40000gn/T/pip-install-9drerz7x/crlibm_43dfd5bfd6bb44ed8de5cfd7748b06ef/setup.py", line 15, in <module>
          from distutils.command.upload import upload
      ModuleNotFoundError: No module named 'distutils.command.upload'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Looking at crlibm's setup.py, it shows the failing import.
I haven't looked into why setuptools removed this but it definitely makes this part fail. Their changelog states the removal in v75.0.0, which refers to this commit
Workaround
Use setuptools<=74.1.3

Thanks!

I'm using setuptools==74.1.3 with python 3.11.4 and still running into the same issue.
$ pip install --no-cache pyinterval
Collecting pyinterval
  Downloading pyinterval-1.2.0.tar.gz (25 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Collecting crlibm==1.*,>=1.0.3 (from pyinterval)
  Downloading crlibm-1.0.3.tar.gz (1.6 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.6/1.6 MB 45.9 MB/s eta 0:00:00
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [20 lines of output]
      Traceback (most recent call last):
File "/Users/myuser/.pyenv/versions/3.11.4/envs/venv3114/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/Users/myuser/.pyenv/versions/3.11.4/envs/venv3114/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/Users/myuser/.pyenv/versions/3.11.4/envs/venv3114/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^
        File "/private/var/folders/vt/2r91wcsj4s9dg7cd0y9tqrkr0000gn/T/pip-build-env-6x6k1js2/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 333, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=[])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/private/var/folders/vt/2r91wcsj4s9dg7cd0y9tqrkr0000gn/T/pip-build-env-6x6k1js2/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 303, in _get_build_requires
          self.run_setup()
        File "/private/var/folders/vt/2r91wcsj4s9dg7cd0y9tqrkr0000gn/T/pip-build-env-6x6k1js2/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 521, in run_setup
          super().run_setup(setup_script=setup_script)
        File "/private/var/folders/vt/2r91wcsj4s9dg7cd0y9tqrkr0000gn/T/pip-build-env-6x6k1js2/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 319, in run_setup
          exec(code, locals())
        File "<string>", line 15, in <module>
      ModuleNotFoundError: No module named 'distutils.command.upload'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.


Extension always calls SSM Parameter Store

Description
The extension always calls SSM Parameter Store, even if Secrets Manager is used for the license key. This results in permissions errors if the lambda doesn't have permission to SSM (which it doesn't actually need in this situation, and shouldn't have).
The errors create CloudTrail entries and may result in false alarms/unnecessary triage.
Steps to Reproduce
Install the extension layer, without giving the Lambda role permissions to access SSM.
Result: many errors in CloudTrail.
Expected Behavior
No calls to SSM if Secrets Manager has the key.
Relevant Logs / Console output
"errorCode":"AccessDenied","errorMessage":"User: arn:aws:sts::12345:assumed-role/example-region-lambdaRole/my-Lambda is not authorized to perform: ssm:GetParameter on resource: arn:aws:ssm:region:12345:parameter/NEW_RELIC_LICENSE_KEY because no identity-based policy allows the ssm:GetParameter action"

Your Environment
NR Layer: NewRelicNodeJS18X:55

working on a fix for this in an upcoming release. We will reduce the calls to the SSM store, and allow users to disable calling the SSM store entirely.

We have merged the fix, waiting for the new layers workflow such that it can be released to the layers.

@chaudharysaket Was this fix released?

@rittneje We will release all the changes as new layers for each runtime by the end of May. Thank you!

Hello,
I'm using the following layer arn:aws:lambda:eu-west-1:451483290750:layer:NewRelicPython312:8 and I'm also experiencing the same issue.
I'm not using the secrets manager but I'm using SSM parameter store (with NEW_RELIC_LICENSE_KEY_SSM_PARAMETER_NAME pointing to the correct place where the key resides and the lambda has access to it.
Even so, I'm flooded with logs in cloudtrail:
    "errorMessage": "User: arn:aws:sts::12345:assumed-role/bla/bla is not authorized to perform: secretsmanager:GetSecretValue on resource: NEW_RELIC_LICENSE_KEY because no identity-based policy allows the secretsmanager:GetSecretValue action",

@nunofernandes Here is the latest from New Relic support:

New target for release of this feature is 6/21 (end of sprint).
This release is also expected to resolve a similar issue involving Secrets Manager calls.

We have released extension v2.3.12 that solves the above issues with unwanted SSM & secrets manager calls. We will release layers [PR] after testing with each runtime - python, nodejs, golang, java & dotnet [6/25].

We have released layers with extension only, which are used by golang runtime lambda. We will release other runtime layers as mentioned above.

Hi @rittneje & @nunofernandes
We have released new layers with the updated extension. Please let us know if you still face issue with cloud trail logs due to unnecessary calls to either SSM or secrets manager.
Thank you!

@chaudharysaket From testing we have observed that the offending SSM calls no longer happen with the latest layer versions.

@chaudharysaket I did the change one hour ago and I can confirm that I can't see anymore those SSM calls being made.. :clap:  :fireworks:


docker: optimize layers

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Just noticed this one... perhaps https://github.com/datahub-project/datahub/pull/10878 can be made part of this?


Feature/moveToCircleRegion

Hey there is an implementation of MapRegion To a center point With a radius
With a little bit of refactoring because the Map wasn't moving like in iOS
Regards,

There is the PinInfoView.
With his events:
PinInfoViewClicked & PinInfoViewLongClicked

For the moveToCircleRegion feature
I changed the Move to region Button in the sample to the Circle Overlay Map Region.
So you can test it
Regards,


Telegram_bot not running after upgrade

After upgrading from v6.0.0 to 6.3.0 I noticed that I can't run the telegram bot anymore. I could solve the issue by checking out to the beta branch. Nevertheless I am reporting the issue which others may also encounter.
# python3.9 telegram_bot.py
2022-05-04 18:05:34,498 - telegram.helper - INFO - Telegram Bot is listening
Traceback (most recent call last):
  File "/home/h1/telegram_bot.py", line 922, in <module>
    main()
  File "/home/h1/telegram_bot.py", line 907, in main
    botconfig.setcommands(None, None)
  File "/home/h1/telegram_bot.py", line 260, in setcommands
    self.helper.send_telegram_message(
  File "/home/h1/models/telegram/helper.py", line 235, in send_telegram_message
    context.bot.send_message(
AttributeError: 'Updater' object has no attribute 'send_message'
^CException ignored in: <module 'threading' from '/usr/lib/python3.9/threading.py'>
Traceback (most recent call last):
  File "/usr/lib/python3.9/threading.py", line 1470, in _shutdown
    lock.acquire()

System info:
NAME="Ubuntu"
VERSION="20.04.4 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian

Fix is merged into main yesterday: [(https://github.com/whittlem/pycryptobot/pull/707)]

Fix is merged into main yesterday: https://github.com/whittlem/pycryptobot/pull/707

Thanks for confirming.


Add Media Driver PID to CnC file

Needs to be supported across all languages.

I can grab this. Will rev the CnC version. Anything else needed to be added?
Start timestamp?

Yes start timestamp would be useful.

All in for Java, C++, and C


Occasional sysreg_access test failure

Occasionally the first test inside test/regression/aarch64SystemRegisters.cc will fail on the out of order run.
Rather than General Purpose Register 2 containing the value 42, it will contain 0.

Issue has not been seen in recent releases. Closing as stale


Fixed package test ubuntu17.04 -> ubuntu17.10

PR Summary
This is related to #6028
Ubuntu 17.04 reached End of Life.
The supported upgrade path from Ubuntu 17.04 is via Ubuntu 17.10.
This change fixes test dockerfile by changing base docker image to ubuntu:artful
PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Change is not breaking
[N/A] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[X] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[X] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

We are still producing the package and calling it 17.04
https://github.com/PowerShell/PowerShell/blob/master/tools/packaging/packaging.psm1#L6

@TravisEz13 yes; that is not a problem; this package installs just fine on 17.10 image (ubuntu:artful).
The issue is that current test is Not working because wrong base image (ubuntu:zesty) is used (there are some issues during apt-get update), so this change has to go through to fix it.


Unable to update height info in Helium app

Describe the bug
I am unable to update my height info for my miner Rak HotSpotv2. It's 18m AGL but the Helium app won't let me update it.
To Reproduce
I click on the gear icon
I click on "Update Hotspot"
I enter my new height info and when I submit, the page goes blank and hangs there.
I have attached a screen recording for reference,
Expected behavior
I should be able to update my height info,
Smartphone (please complete the following information):

Device: iPhone 12 Pro Max
OS: iOS 15.1
App Version: Latest

Additional context
Please see attached screen recording. Thank you.
https://user-images.githubusercontent.com/46775664/141655455-e4bd3df8-cdd8-4cf4-ba88-47a2d65fc3a3.mov
.

Have you tried selecting your antenna?

I have the same exact issue.

@dknightcc11 can you record a video of updating height starting from pressing the settings icon?

I am on iOS 15.1 and I have an iPhone 11 Pro. I spent a week trying to update my antenna gain and height until I was able to figure out what was happening. The issue is that when I try to input the antenna gain and height the keyboard pops up (as expected, see second screen shot below). But crucially the keyboard is covering the Update Antenna button that you need to tap to confirm the update (you can see the button on the first screen shot). So when you tap on Done after you input the data the screen just goes all blur (see third screen shot) but it doesn't actually submit anything. And since the input screen it's gone you can't see the Update Antenna anymore. Also note that the keyboard that pops up doesn't have the Hide Keyboard key so I am unable to hide it to see the Update Antenna button. How did I get around this bug? I installed the app on my iPad and due to the different screen size I was able to finally update my antenna height and gain.

TURRIBEACH, well written!!!!, and I also concur this is ABSOLUTELY What I did to solve the issue!  I couldn't even assert a Brand new miner, with the same issue....so I set it up on my iPad, and Everything worked....good catch.... Guys , please fix this.  Thanks! and cheers, JB

this is really helpful. thanks for narrowing it down for us - makes it much easier to fix. we'll be looking at this next.

The latest app update 3.8.5 adds in better logic around closing the keyboard for users to submit the transaction.

Please comment/close if this is fixed for you! thanks for everyone's patience.

Hi, I updated my app to v3.8.5 and I see no improvement at all, it behaves in exactly the same way. Are you sure this is fixed?

Hi, I updated my app to v3.8.5 and I see no improvement at all, it behaves in exactly the same way. Are you sure this is fixed?

the new app version hides the keyboard when you press "Done". The Update Location card should still be visible. Are you saying that the card is now gone and the only thing you see is the blurred background? what iOS/phone are you using?

The new version does Not solve the problem.
iOS 15.1.1
iPhone 13 pro

Are you saying that the card is now gone and the only thing you see is the blurred background? what iOS/phone are you using? => Yes, same as before.
iOS 15.1
iPhone 11 Pro

Are you selecting a Custom Antenna in the Antenna options? I can reproduce it every time. It also happens after I set the antenna dBi and I tap in the Height field.

Are you selecting a Custom Antenna in the Antenna options? I can reproduce it every time. It also happens after I set the antenna dBi and I tap in the Height field.

can you take a video recording?

https://user-images.githubusercontent.com/34696469/145476574-8f84dc57-cda9-48b2-b93b-bea81e090845.mov

this is awesome. super helpful to understand visually what is happening. will help us narrow it down.

Why is this closed? It is still not fixed in v3.8.5.

Why is this closed? It is still not fixed in v3.8.5.

We have a fix that will hopefully work for you. It will be out in v3.8.6

I have the same problem since 2021 September :( I was never able to set up a correct Antenna value or height.

Why is this closed? It is still not fixed in v3.8.5.

We have a fix that will hopefully work for you. It will be out in v3.8.6

When is v3.8.6 supposed to be out? Shouldn't this issue be closed once v3.8.6 is out and confirmed as closed?

Why is this closed? It is still not fixed in v3.8.5.

We have a fix that will hopefully work for you. It will be out in v3.8.6

When is v3.8.6 supposed to be out? Shouldn't this issue be closed once v3.8.6 is out and confirmed as closed?

3.8.6 should be out today and we will re-open the issue if its not resolved. Github auto closes it when a PR linked to the issue is closed.

Just downloaded v3.8.6 and tested it and the issue is still there. Can you please reopen the issue?

This issue is not resolved v3.8.6, iOS 15.0 iPhone SE. There is no difference in behaviour from previous versions.
Having incorrect antenna information now negatively impacts the helium network, please remove all animations for these modals or look to correct this issue asap.
@tyler-whitman

This issue is not resolved v3.8.6, iOS 15.0 iPhone SE. There is no difference in behaviour from previous versions.
Having incorrect antenna information now negatively impacts the helium network, please remove all animations for these modals or look to correct this issue asap.
@tyler-whitman

@Mattish @turribeach would either of you be able to send me a friend request on discord (tyler.helium#0588) so I can reach out with a beta build of the app to test some fixes for this issue?

I am sending you a friend request now
I am RandomPlanet on Discord
Jason Buecker
770.428.4205
 MacintoshConsultants.Com
Apple Consultants Network
Apple Certified Technical Coordinator (ACTC)

On Jan 4, 2022, at 8:28 PM, Tyler Whitman @.***> wrote:
tyler.helium#0588

We have tracked it down to an iOS setting in Accessibility > Motion > Reduce Motion and enabling "Prefer Cross-Fade Transitions" is causing the bug. We have a fix coming in the next release which will be 3.9.0. If you want to resolve it as soon as possible, disabling "Prefer Cross-Fade Transitions" temporarily will work.

The fix for this is now available on 3.9.0. The root cause was an accessibility option to reduce motion.


[CARBONDATA-2491] Fix the error when reader read twice with SDK carbonReader

[CARBONDATA-2491] Fix the error when reader read twice with SDK carbonReader

1.Fix the error out of bound when reader read twice with SDK carbonReader
2.Fix the java.lang.NegativeArraySizeException
3.Add timestamp and  bad record test case
This PR dependency on https://github.com/apache/carbondata/pull/2246
How to fix the error:
Carbon throw the exception because the fileName is "" in org.apache.carbondata.core.util.path.CarbonTablePath.DataFileUtil#getTaskNo, I find the filePath is "" in prunedBlocklets of org.apache.carbondata.hadoop.api.CarbonInputFormat#getDataBlocksOfSegment. So it should the data map issue.
It'S UNSAFE issue
Carbon throw the second exception  because the length less than 0 in org.apache.carbondata.core.indexstore.row.UnsafeDataMapRow#convertToSafeRow
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[x] Any interfaces changed?
No
[x] Any backward compatibility impacted?
No
[x] Document update required?
No
[x] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.
No
[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
No

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4986/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4802/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5959/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5963/

retest this please

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5975/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4818/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5985/

retest this please

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5987/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4829/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6031/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6046/

retest this please

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4887/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6051/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4892/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5060/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6056/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4897/

@sounakr Done，and CI pass

@xubo245 Better to allow "*" as input in reader projection. This will help the user to specify all columns. Just like SQL select *.

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4902/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5066/

@sounakr Ok, done

LGTM

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6069/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4909/

@jackylk @ravipesala Hello, sounakr give LGTM and CI pass. Can you help to check and merge it if there are no problem, please.

retest this please

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5082/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4921/

retest this please

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4925/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4930/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6091/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6093/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4932/

@jackylk CI pass, please check.

LGTM


IBrowsingContext.OpenAsync doesn't pass CancellationToken on

This line in public static Task<IDocument> OpenAsync(this IBrowsingContext context, DocumentRequest request, CancellationToken cancel = default) throws the CancellationToken away
https://github.com/AngleSharp/AngleSharp/blob/3968eb050e142b1d94550fba407afe772232b126/src/AngleSharp/BrowsingContextExtensions.cs#L66

That's a great catch. Thanks for noticing. Want to make a PR?
Would be much appreciated :rocket:!

Landed in devel and the preview!
Thanks for your awesome contribution @singingknight


⚠ Bofrost.lu has degraded performance

In e01b24f, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1188 ms

Resolved: Bofrost.lu performance has improved in cf37909 after 7 minutes.


add javascript to track total changes in profits/losses

Code has been added to track total changes through out the whole period of profits/losses


Native threads calling back into Python

I have a C++ project wrapped in boost python and I'm using the resultant module created from this to build a Python Application. It works really well however I have noticed that threads that initiate from C++ are not tracked by the debugger and therefor python break points do not get hit on callbacks from C++-> Python. Essentially what is descrived here: https://github.com/Microsoft/PTVS/issues/3003
Can someone tell me what is the most non intrusive fix for this? Has the advice moved on a bit from what was described in the link above? It seems to recommend using ptvsd.debugger.new_external_thread() but I can't seem to import ptvsd as VS doesn't seem to know where the module exists.
Any help would be greatly appreciated. I am using VS2013 not exactly sure how to tell which version of PTVS I am using but it is fairly recent.

I don't think we're removing ptvsd from sys.modules in the most recent version, so it should be there once debugging begins; i.e. if you can't import it, that would imply that debugger is not attached yet when the thread is started - is that right?

No - The debugger is attached. The program fails to start because it can't import ptvsd. How do I find out where it is? Where should I find ptvsd module normally? Just so I can check my install is correct.

It lives in the PTVS extension folder under ...:\Program Files (x86)\Microsoft Visual Studio\...\Enterprise\Common7\IDE\Extensions (the folder names below that will differ, but you can just search for .py files).
A simpler way for you to be able to import it at any time is to pip install ptvsd it into your Python environment. But it's still strange that it won't let you just import it. Can you check what's in sys.modules?

It's here but I can't seem to import it.

Ah, your version of PTVS is much older than what that workaround was written for.
In your case, instead of an import, try this:
ptvsd = sys.modules['$visualstudio_py_debugger']

Ok, if I download 2.2.2 that should support it?

Ok, I've upgraded to 2.2.2 and created a class like this:
class NativeCallback():
# ptvsd
#import ptvsd.debugger
#ptvsd.enable_attach(secret = 'mysecret', address = ('<IP_ADDRESS>', 8080))
#ptvsd.wait_for_attach()
@staticmethod
def register():
    import ptvsd
    import ptvsd.debugger
    ptvsd.debugger.new_external_thread()

Used like this:
def onHeartbeatResponse(self, context, envelope):
    NativeCallback.register()

and the breakpoints are still not hit.

Its working now! Upgraded to VS2015 using 2.2.2. Thanks.

:+1:


Add syntax to paket.template to get version constraint from paket.dependencies

I think it would be quite useful to have a syntax similar to LOCKEDVERSION and CURRENTVERSION to tell paket pack to grab the version constraint for a given dependency from paket.dependencies. For example something like this:
dependencies
    Paket.Core DEPENDENCIESVERSION
    Fake DEPENDENCIESVERSION group: mygroup

which, given the following paket.dependencies:
source https://api.nuget.org/v3/index.json
nuget Paket.Core ~> 5.100.2
nuget Fake ~> 4.0
group mygroup
nuget Fake >= 5.0.0-alpha

would be equivalent to:
dependencies
    Paket.Core ~> 5.100.2
    Fake >= 5.0.0-alpha

This should already be the case. But I think you then need to omit the dependencies section from your paket.template.

@inosik I guess this is referring to paket.template with "type file". According to https://fsprojects.github.io/Paket/template-files.html#Dependencies dependencies for "type project" are added automatically:

In a project file, the following dependencies will be added:

Any Paket dependency with the range specified in the paket.dependencies
[...]

Missed this OP feature also today - had to copy constraints from paket.dependencies for "type file" to get it working...


Remove accessibility props that are no longer valid.

Please select one of the following

[x] I am removing an existing difference between facebook/react-native and microsoft/react-native :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native :thumbsup:
[ ] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Description of changes
The accessibilityTraits and accessibilityLabel props no longer exist in the official react-native code. I'm removing it to get rid of the errors during debugging of RNTester app.
Focus areas to test
(optional)
Summary
Remove obsolete accessibility props in RNTesterExampleList.js
Microsoft Reviewers: Open in CodeFlow

/azp run React Native Apple build from GitHub


stats verb tests occasionally failing

Describe the bug
From time to time the stats verb tests are failing with mismatched counts. I've included an example below but have seen multiple different tests fail with count mismatches.
To Reproduce

Do a few full test runs immediately after each other

Additional context
Here's some relevant output from a test run I just did in which the test stats verb for id 11 - for messageType text failed.
command sent: stats:11
stats verb response : data: [{"id":"11","name":"NotificationCount","value":"{\"total\":128,\"type\":{\"sent\":98,\"received\":30},\"status\":{\"delivered\":86,\"failed\":10,\"queued\":2},\"operations\":{\"update\":101,\"delete\":17},\"messageType\":{\"key\":113,\"text\":15},\"createdOn\":1644796786961}"}]
command sent: notify:messageType:text:ttr:-1:@cicd2:message@cicd1:Hi!!!
notify verb response data:7b3961bb-fdf8-4932-99d0-635c991b2e7a
command sent: stats:11
stats verb response : data: [{"id":"11","name":"NotificationCount","value":"{\"total\":73,\"type\":{\"sent\":53,\"received\":20},\"status\":{\"delivered\":52,\"failed\":0,\"queued\":1},\"operations\":{\"update\":45,\"delete\":17},\"messageType\":{\"key\":57,\"text\":16},\"createdOn\":1644796804975}"}]
package:test_api                 expect
test/stats_verb_test.dart 190:5  main.<fn>
queue cleared

Expected: <99>
  Actual: <53>

Fixed the failing stats test by verifying whether the notification has been delivered before retrieving the stats verb again.


Add TLS support for Redis

Ability to configure Redis with TLS endpoints and TLS replication for Redis HA.
Jira: OSPRH-3567

/retest


vrpn: 7.33.1-1 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository vrpn to 7.33.1-1:

upstream repository: https://github.com/vrpn/vrpn.git
release repository: https://github.com/ros-drivers-gbp/vrpn-release.git
distro file: lunar/distribution.yaml
bloom version: 0.6.1
previous version for package: 0.7.33-0

Thanks @paulbovbel for the quick release. Is the version scheme change intentional? (0.7.33-0 -> 7.33.1-1)

Yes. VRPN hasn't had a release in several years, so I had to bloom from a commit rather than a tag, and provide a 'patch' number for ROS.
I double checked and 7 is the major version in upstream packaging, not the minor (http://neuro.debian.net/pkgs/vrpn.html).

I double checked and 7 is the major version in upstream packaging, not the minor (http://neuro.debian.net/pkgs/vrpn.html).

Oh yeah, the leading 0 is confusing. Thanks for confirming. Merging then


Customization for WeChat Mini Program

Hi AppInsights Team,
This is Kris from Microsoft working on China online store  project. We are building a WeChat Mini Program (a.k.a. weapp) to sell some electronic products online and would like to consult whether it is suitable to customize AppInsights NodeJS SDK for weapp's telemetry.
What is WeChat Mini Program?
Here are some articles about WeChat Mini Program:
https://www.techinasia.com/wechat-mini-programs-launch
https://walkthechat.com/wechat-mini-programs-simple-introduction/
Official documentation: English | 中文
From technical perspective, a weapp is based on WeChat App on Android/iOS. There are 2 layers in a weapp: view layer and logical layer. View layer is like HTML and CSS. Logical layer is written in Javascript and runs in an environment that is similar as NodeJS. However, it lacks of many built-in NodeJS modules like http, https, Buffer, path, etc. Instead, WeChat Team provides a built-in library wx with which we can send http requests, login via WeChat, get system info, etc.
Problem
Because of weapp's nature that it lacks of many NodeJS modules, AppInsights NodeJS SDK is not suitable for it. Then I do some customization to make it work for weapp: remove a lot of functionalities that is not compatible, replace http and https modules with WeChat's wx.request() and replace built-in Buffer with a 3rd party one. However, it is worrisome because the SDK is for a web server but we use it in way that it doesn't intend to. A weapp runs on a phone where processor, memory, network and Javascript engine are quite different from those in a web server.
Could you give us some advice about this kind of usage? Do you have some system requirement or/and performance test (processor, memory, network, etc.) of AppInsights NodeJS SDK that can be shared with us?

A requirement for this Node.js SDK is certainly that it is currently running on top of Node.js (Note that it is not required to be a web server application [1], but at a minimum all APIs available in Node 0.10 are expected to be available for use. A version string is also expected to be available at process.versions.node that accurately represents which Node API level is available and what API behavior to expect).
You can maintain a fork like you currently do without this requirement, but be aware that it may be difficult to keep it up to date with changes if necessary, as we may add more dependencies on Node APIs that aren't available to you.
Performance impact of the base SDK itself is quite minimal. In a test app with 4 autocollected dependencies for every incoming request, sampling set to reduce telemetry rate to ~5 items / second, automatic correlation disabled, and a simulated load of 30 concurrent users, CPU overhead was measured at 3% and overhead to requests/second was 16%. (Requests/sec moved from 652 at baseline to 549, and CPU from 87% under baseline load to 90%). Assuming your secenario doesn't support autocollection either (so no monkeypatching), I'd expect even lower overhead.
These stats can certainly change, as you mention, with the JavaScript engine changing, but I still wouldn't expect significant impact. Especially if everything is manually collected in your modified SDK, as SDK code is only called when you call into it at that point.
[1] Request collection and autocorrelation only works for web servers but this doesn't prevent use of the TelemetryClient API to manually instrument your application.

Thank you @OsvaldoRosado for the detailed information!

I'd like to share some updates of my customization: On July 2018 we launched the Store WeChat Mini Program (Search "微软官方商城" in WeChat and you'll get it) with AppInsights telemetry via the customized SDK. It has been in production and working well since then.
@OsvaldoRosado