[Bug]:   Error: Cask 'stretchly' is unavailable

Version

[ ] I'm using version 1.12.0

Known issues

[X] I've checked Known issues

Existing issues

[X] I've checked Existing issues

What operating system are you using?
macOS
Operating System Version
macOS Monterey 12.6
Reproduction steps

brew update && brew install --cask stretchly

...
Reproduces how often: [What percentage of the time does it reproduce?]
always
Expected Behavior
install stretchly
Actual Behavior
Error: Cask 'stretchly' is unavailable: No Cask with this name exists.
Relevant log output
No response
Preferences
No response
Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

According to web, it should be there: https://formulae.brew.sh/cask/stretchly#default . I can also see the code: https://github.com/Homebrew/homebrew-cask/blob/master/Casks/stretchly.rb
If your issue persists, you should talk to folks here: https://github.com/Homebrew/homebrew-cask

My homebrew-cask installation was corrupt. After reinstalling homebrew-cask i was able to install it.
Thanks for pointing me in the right direction.

$ brew doctor > Warning: Homebrew/homebrew-cask was not tapped properly! Run:
rm -rf "/usr/local/Homebrew/Library/Taps/homebrew/homebrew-cask"
brew tap homebrew/cask
$ rm -rf "/usr/local/Homebrew/Library/Taps/homebrew/homebrew-cask"
$ brew tap homebrew/cask
$ brew search stretchly
$ brew install stretchly

Glad to help :)


Bump version

Bump Scala to 2.12.6
Bump sbt to 1.2.0
Bump akka to 2.5.14
Bump ScalaTest to 3.0.5
Bump Kafka to 1.1.1
Fix hyperlink directed to old API

Codecov Report

Merging #144 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #144   +/-   ##
=======================================
  Coverage   65.04%   65.04%
=======================================
  Files          16       16
  Lines         535      535
  Branches       34       31    -3
=======================================
  Hits          348      348
  Misses        187      187

Continue to review full report at Codecov.

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

👍 Ill push out a new release soon, cheers


HDInsight/cluster incorrectly marked secret properties

We(Azure Service Operator) have an ask to support HDInsight/Cluster as an ASO resource. However, I noticed a few below properties that are marked as x-ms-secret in the HDInsight/clusters spec which maybe are incorrectly marked as secrets.
Configurations - An empty object
ScriptActions - An array of object
MinInstanceCount - An int
SshProfile - An object

Hey, just checking if there's any updates on above? @aim-for-better, @idear1203, @deshriva


[WIP] NetworkResolver #119

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Tests pass?
✔

Fixed issues
#119

MiddlewareDispatcher: this one will adjust Request to contain trusted info / additional info

Could MiddlewareDispatcher pass the network resolver to middleware objects? (IpFilter::networkResolver) Either need it to use it at all or will ServerRequest already contain the modified information you want in Middleware::process()?

Could MiddlewareDispatcher pass the network resolver to middleware objects?

Yes.

(IpFilter::networkResolver) Either need it to use it at all or will ServerRequest already contain the modified information you want in Middleware::process()?

I think it should contain already modified info that could be trusted.

Seems to fail tests.

Seems to fail tests.

Yes. There is a special case that is not supported by yii2: the user is connected via HTTP, but the last proxy is already using HTTPS (eg more efficient HTTP2/QUIC)
client <-----http----> CDN <-----http2/quic----> server

A test has been added, this is what is causing the error. I want to read if this implementation makes sense.

I think such configuration is possible.

👍 What's left?

What's left?

TrustedHostNetworkResolver, test separation, more tests. and minimal phpdoc for complex method's parameters.

@samdark composer.lock is ignored in git, why? The TrustedHostsNetworkResolver will require the Ip validator from the yiisoft/validator package.
https://github.com/yiisoft/yii-web/blob/3398f9b5ccdeea2365d51f62498f4e1ce03910a8/.gitignore#L19-L21

I have modified the interface because this must be able to modify the many of parameters in the request object eg in TrustedHostsNetworkResolver. Not least, it makes the interface simpler and more flexible.

In order not to stuck with outdated dependencies. composer.json is read/evaluated every time.

@samdark The Ip rule can't find classes. Need more entries in composer.json?
1) Yiisoft\Yii\Web\Tests\NetworkResolver\TrustedHostsNetworkResolverTest::testIps with data set #0 ('<IP_ADDRESS>', array(array('<IP_ADDRESS>, <IP_ADDRESS>')), array('<IP_ADDRESS>'), null, '<IP_ADDRESS>')
Error: Class 'yii\helpers\IpHelper' not found

/home/smarton/work/source/web/yii2/kamarton-yii3-web/vendor/yiisoft/validator/src/Rule/Ip.php:464
/home/smarton/work/source/web/yii2/kamarton-yii3-web/vendor/yiisoft/validator/src/Rule/Ip.php:304
/home/smarton/work/source/web/yii2/kamarton-yii3-web/vendor/yiisoft/validator/src/Rule/Ip.php:260
/home/smarton/work/source/web/yii2/kamarton-yii3-web/src/NetworkResolver/TrustedHostsNetworkResolver.php:228
/home/smarton/work/source/web/yii2/kamarton-yii3-web/src/NetworkResolver/TrustedHostsNetworkResolver.php:167
/home/smarton/work/source/web/yii2/kamarton-yii3-web/src/NetworkResolver/TrustedHostsNetworkResolver.php:125
/home/smarton/work/source/web/yii2/kamarton-yii3-web/tests/NetworkResolver/TrustedHostsNetworkResolverTest.php:36

IpRule isn't ready. We can extract something like yiisoft/network package containing tools for various network calculations such as IP matching, masks, ranges etc.

@samdark Should I put TrustedHostsNetworkResolver in a separate branch in the absence of an IP validator and delete it from here?

Should I put TrustedHostsNetworkResolver in a separate branch in the absence of an IP validator and delete it from here?

Yes, let's do that after this one is merged.

Yes, let's do that after this one is merged.

removed

@samdark the code is reviewable.

Merged. Thank you!
Do you need a repository for IP / network helper?

Do you need a repository for IP / network helper?

I think for the time being I can work it out to make the IP validator workable in local, but it will definitely be necessary for a PR.

https://github.com/yiisoft/netutils. I'll add files there soon.

https://github.com/yiisoft/netutils. I'll add files there soon.

The yiisoft/network-utility would not be nicer?

https://github.com/yiisoft/network-utilities

Package is ready.


1.16.1 - Cannot Place Water when Paired with Some Mods

For some reason, when I have this mod installed with some other mods, I cannot place water in the world using a bucket of water.
If it is just this mod, it works fine. But if any of the following are included, this issue happens:

campanion-1.1.10+build.24
bunch-o-trinkets-1.0.1

Would love to have this fixed so I can slap this bad boy into All of Fabric 3.


Saving each frame to a separate file should not require a file name

When saving each frame in a wxg file to a separate file, the user should not be required to specify a file name in the dialog that the "..." button displays because it would not work. If the user does not delete this file from the Output Path field wxglade will fail when the Generate button is pushed.
Steps to reproduce:
Create a new wxg file (or see attachment).
Add two frames to it.
Add a menubar with menus to the first frame.
Click on "Application" at the top of the tree.
Click on the "Separate file for each class" radio button.
Click on the "Python" radio button under the Language area.
Click the "..." button next to "Output Path" field.
What happens:
A file save dialog will appear asking you to enter a name for a new file.
What should happen:
A dialog should appear that has you select the destination for all the class files without having to name a new file.
Pushing the "Generate Source" button in the Properties area causes an error dialog to appear saying "Output directory does not exist".
My info:
wxPython 4.0.6
Python 2.7.14
wxGlade 1.0.1
Mac OS 11.1

menu_test.zip

Well, I did not expect anyone to use the file selector - just enter the directory name. It's much easier.
Now a DirSelector is used if multiple_files is active.


refactor(config): Export loadEnv() function

For some projects, such as electron-vue-next, there is a need to load environment variables from non-standard locations and to transfer these environments to other builders (such as rollup, etc.).
This PR will allow simply import loadEnv function from vite and use its algorithm instead of duplicating this algorithm in your own code.
Use cases
Configuring the building of two bundles using a single mechanism for loading environment variables.
import { loadEnv } from 'vite';

// The location of env files can be arbitrary
const dotenvLocation = path.resolve(__dirname, '../../')

// Load all environment variables with the prefix VITE_ to build a web interface
const envForRenderer = loadEnv('production', dotenvLocation)
buildRendererProcessWithEnv(envForRenderer)

// Load all environment variables with the prefix ELECTRON_ to build a backend
const envForMain = loadEnv('production', dotenvLocation, 'ELECTRON_')
buildMainProcessWithEnv(envForMain)

Another use case:
The development environment can be configured to automatically generate *.d.ts files based on .env* and describing environment variables and their types for each mode.

I split loadEnv function into two:

loadEnv - reads files and returns an object with environment variables. Without side effects.
expandEnv - mutate process.env. For backward compatibility, I call the expandEnv wherever  loadEnv  is called. But, I assume that expandEnv is not needed everywhere, and its calls can be reduced or completely removed.

I don't have publish rights, so you'll have to wait for @yyx990803. Hopefully we can get auto-release configured once v1.0.0 is released.


Migrate to crust-client-api

Remove custom API client implementation and migrate to generated code in crust-client-api.
See examples in CRM and adapt:

[x] src/api/* configuring the  client
[x] src/store for injecting (configured) client into store
[ ] apply same design principles for vuex we have for crm/module, crm/page... for messaging/channels and users and fix/update all usages.

Obsolete.


Update fs2-core, fs2-io to 3.2.5

Updates

co.fs2:fs2-core
co.fs2:fs2-io

from 3.2.3 to 3.2.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #214.


[feature request] Don't host downloads at GitHub please

Is your feature request related to a problem? Please describe.
It has been throttling download speed since MS takeover or even longer:

(base connection speed is 10-20 Mbps)
When I try to download in the browser it 100% fails within a few minutes, with no ability to resume the download. Do you expect every user to be familiar with wget?
APK build gets auto mirrored to some sites, but I've only found 2.0.10 among crypto enabled builds, no 2.0.15 around. As for desktop version I haven't found any mirrors at all. Are there any?
Describe the solution you'd like
https://www.fosshub.com/ is a good place to put binaries, never had problems with it. Or since you're building for Web3, use Sia or smth.

I think this has to do with their lazy reliance on electron-updater. Also might be related to #419.


handle american ordinals

This should resolve #10

@akshaynagpal Is there anything else this needs to be merged?

Have been busy lately. Will look at it and merge if all requirements of merging are fulfilled.

@akshaynagpal Thanks!
If you can let me know what tests you think are missing, I can write them. I thought I had covered all the equivalence classes, but I could have missed something.
I also see there are some complexity issues that codebeat is complaining about. I can do some refactoring to make it a little cleaner.


Explore adding react-docgen

I know there's some work going on in storybook core (https://github.com/eirslett/storybook-builder-vite/issues/2#issuecomment-823064989) as @shilman mentioned, but I've managed to get react-docgen working (I haven't really tested it's performance) but it might be worth exploring adding this temporarily so the feature works?

There are now two solutions, one which is relying on the babel plugin (https://github.com/eirslett/storybook-builder-vite/commit/6cbc5440595fd975b6fd557788134ef53a48bb3b) and one which emulates it (https://github.com/eirslett/storybook-builder-vite/commit/480899fdb71b9e4f85225fcb03a10de94d136bdd).
Relying on the babel plugin is definitely cleaner but we can't make it easy to use<EMAIL_ADDRESS>maybe an alpha of babel-plugin-react-docgen could use it?
Currently I've added the babel plugin as a dependency but it's always included in @storybook/react so maybe it's not needed?

@joshwooding what's your sense for how important it is to use<EMAIL_ADDRESS>
Allow resolving of imported types/proptypes

Seems kind of important, right?  I think it could be okay to use your re-implementation for the time being, perhaps with a note to swap it out once (if) there's ever a stable release of react-docgen@6 (tracking issue: https://github.com/reactjs/react-docgen/issues/502).
Alternatively, like you said we could make an alpha version of babel-plugin-react-docgen, it looks like a similar approach has been taken in the past: https://github.com/storybookjs/babel-plugin-react-docgen/issues/72

I talked to @shilman and others about this, and they'd be willing to consider updating babel-plugin-react-docgen to use the alpha version of react-docgen if it solves the type import issue.  There's even a good chance that it will end up being faster than the typescript version that's in use right now.  We should explore it and see if it does indeed work for typescript projects.

@IanVS Cool, I was looking at this today. I think releasing an alpha of babel-plugin-react-docgen is the best option.

I published<EMAIL_ADDRESS>that we can experiment with until/unless storybook merges and releases https://github.com/storybookjs/babel-plugin-react-docgen/pull/104.


Is this package compatible with newest Meteor?

Hey guys! first of all - very well done with your package. Scondly I was wondering if it is still ok to use it with Meteor 1.6?

It seems this other repo was updated to the latest Meteor version:
https://github.com/tdukart/react-redux-meteor
Hope it helps!
Disclaimer: I haven't test it yet.


[BUG] CLI incorrectly reports missing arguments in the map step

Running sf map command when I miss the provider argument prints me incorrect and confusing message
Expected Behavior
The CLI reports that provider parameter is missing.
Current Behavior
CLI reports that profile id is missing, suggests to submit it as a first parameter:
sf map organizations-management/get-organizations
⠙ Loading profile
 ›   Error: ❌ Missing profile id. Please provide it as first argument.

Steps to Reproduce
Run a map command, omit the provider id
Your Environment

macOS 13.4.1
@superfaceai/cli/4.0.0 darwin-x64 node-v18.13.0

This is due to use of positional arguments - values are not tied to ids (provider name or profile id). In this case it should be addressed by error with message similar to:
Error: ❌ Missing provider name or profile id. Please provide provider name as first argument and profile id as second argument.


[TS-664] Analysis upload fails with "rejecting SARIF, as there are more runs than allowed"

This only fails for codacy, so I am wondering if the error is referring to the sarif file itself having too many "runs"? I tried running it in isolation w/o any other jobs and it failed, so I think it has to be something to do with the report contents.
Run github/codeql-action/upload-sarif@cdcdbb57970684[1](https://github.com/ben-manes/caffeine/actions/runs/6915822206/job/18815190960#step:7:1)c47f7063dda365e292e5cad7a
Uploading results
Waiting for processing to finish
Error: Code Scanning could not process the submitted SARIF file:
rejecting SARIF, as there are more runs than allowed (21 > 20)
Error: Code Scanning could not process the submitted SARIF file:
rejecting SARIF, as there are more runs than allowed (21 > 20)
    at Object.waitForProcessing (/home/runner/work/_actions/github/codeql-action/cdcdbb579706841c47f7063dda365e292e5cad7a/lib/upload-lib.js:337:23)
    at async run (/home/runner/work/_actions/github/codeql-action/cdcdbb579706841c47f7063dda365e292e5cad7a/lib/upload-sarif-action.js:55:13)
    at async runWrapper (/home/runner/work/_actions/github/codeql-action/cdcdbb579706841c47f7063dda365e292e5cad7a/lib/upload-sarif-action.js:70:9)

See the github-actions log.
https://github.com/github/codeql-action/issues/2000

From the GitHub docs, this is the analyzer exceeding the limits.

I went ahead and fixed this by removing the duplicate runs to fix the integration.
jq '.runs |= unique_by({tool, invocations})' results.sarif > codacy.sarif

original sarif
$ gron results.sarif.json | rg "tool.driver.name"
json.runs[0].tool.driver.name = "Checkov (reported by Codacy)";
json.runs[1].tool.driver.name = "Semgrep (reported by Codacy)";
json.runs[2].tool.driver.name = "Pmd (reported by Codacy)";
json.runs[3].tool.driver.name = "Semgrep (reported by Codacy)";
json.runs[4].tool.driver.name = "Cppcheck (reported by Codacy)";
json.runs[5].tool.driver.name = "Pmd (reported by Codacy)";
json.runs[6].tool.driver.name = "Markdownlint (reported by Codacy)";
json.runs[7].tool.driver.name = "Trivy (reported by Codacy)";
json.runs[8].tool.driver.name = "Semgrep (reported by Codacy)";
json.runs[9].tool.driver.name = "Shellcheck (reported by Codacy)";
json.runs[10].tool.driver.name = "Semgrep (reported by Codacy)";
json.runs[11].tool.driver.name = "Trivy (reported by Codacy)";
json.runs[12].tool.driver.name = "Jacksonlinter (reported by Codacy)";
json.runs[13].tool.driver.name = "Detekt (reported by Codacy)";
json.runs[14].tool.driver.name = "Flawfinder (reported by Codacy)";
json.runs[15].tool.driver.name = "Remark-lint (reported by Codacy)";
json.runs[16].tool.driver.name = "Spectral (reported by Codacy)";
json.runs[17].tool.driver.name = "Trivy (reported by Codacy)";
json.runs[18].tool.driver.name = "Trivy (reported by Codacy)";
json.runs[19].tool.driver.name = "Spectral (reported by Codacy)";
json.runs[20].tool.driver.name = "Checkov (reported by Codacy)";

fixed sarif
$ gron codacy.sarif | rg "tool.driver.name"
json.runs[0].tool.driver.name = "Checkov (reported by Codacy)";
json.runs[1].tool.driver.name = "Cppcheck (reported by Codacy)";
json.runs[2].tool.driver.name = "Detekt (reported by Codacy)";
json.runs[3].tool.driver.name = "Flawfinder (reported by Codacy)";
json.runs[4].tool.driver.name = "Jacksonlinter (reported by Codacy)";
json.runs[5].tool.driver.name = "Markdownlint (reported by Codacy)";
json.runs[6].tool.driver.name = "Pmd (reported by Codacy)";
json.runs[7].tool.driver.name = "Remark-lint (reported by Codacy)";
json.runs[8].tool.driver.name = "Semgrep (reported by Codacy)";
json.runs[9].tool.driver.name = "Shellcheck (reported by Codacy)";
json.runs[10].tool.driver.name = "Spectral (reported by Codacy)";
json.runs[11].tool.driver.name = "Trivy (reported by Codacy)";


CI: remove EOL Fedora-35

I am just curious why https://github.com/aufover/aufover-benchmark/pull/39/files#diff-9d14d1268d1222ca14842f37bd15484866bea6bad520ad45c09da8ccece668afL38
is still if [[ "${{matrix.release}}" =~ 3[56] ]]; then
I would understand that it matches f35 and f36, and f35 should have been removed in this PR.

@psimovec Yeah, it's redundant. You can remove it in an additional commit.


Find control's properties

Hi,
I'm using appium in visual studio for testing a simple WPF application,
I'm using WindowsDriver
WindowsDriver<WindowsElement> driver = new WindowsDriver<WindowsElement>(new Uri("http://<IP_ADDRESS>:4723"), desiredCapabilities);
I know how to enter text and how to get text from TextBox:
driver.FindElementByAccessibilityId("MyTextBox").SendKeys("123");
I wanted to know if there any way to get and set control's properties, For example: get IsChecked from CheckBox, get Background from TextBlock etc.
Thanks.

You can try exporting the PageSource to an XML file and then
driver.FindElementByAccessibilityId("MyTextBox").GetAttribute("innerHTML"));
You will get clue for your Intended property or attribute values to pass as an argument in GetAttribute function from this exported XML file.
I'm not sure if we can set value of the control by setting the value of attribute, even if we can it would not be a good idea because the background events are not triggered for some controls.
You will have to utilize the exposed methods like Click(), SendKeys() etc. to set the values.
Regards
PD

@dhapolapankaj
Thanks for your answer,
I guess that your solution is for a web app,
but like I said i'm testing a desktop app and the ui of this app
is written in XAML (using WPF) , is there a way to get properties from controls in my desktop app?
Thanks.

:) I would still insist you to export the PageSource
try below code
System.Xml.XmlDocument doc = new System.Xml.XmlDocument();
doc.LoadXml(driver.PageSource.ToString());
System.Xml.XmlTextWriter writer = new System.Xml.XmlTextWriter(@"c:\App_To_XML.xml", new System.Text.UnicodeEncoding());
writer.Formatting = System.Xml.Formatting.Indented;
doc.WriteContentTo(writer);
See if you have got the "App_To_XML.xml" file generated

Hi @CheNMorD,
Typically, I would use Selected property of the element to see if a Checkbox is checked.  Below is a snippet of interaction with a XAML CheckBox control in a UWP app taken from https://github.com/Microsoft/WinAppDriver/blob/v1.0-RC/Tests/UWPControls/CheckBox.cs#L97
var originalState = checkBoxElement1.Selected;
checkBoxElement1.Click();
Assert.AreNotEqual(originalState, checkBoxElement1.Selected);
checkBoxElement1.Click();
Assert.AreEqual(originalState, checkBoxElement1.Selected);

For any other element properties shown on inspect.exe, you can use the GetAttribute API. You can find an example of getting various attributes/properties on the following sample test:
https://github.com/Microsoft/WinAppDriver/blob/v1.0-RC/Tests/WebDriverAPI/ElementAttribute.cs#L38
Note that element properties that are not exposed to the accessibility layer such as background color and font, cannot be retrieved by Windows Application Driver nor inspect.exe.

As described by @bharathp666 in https://github.com/Microsoft/WinAppDriver/issues/590 we can check the whole xml document through driver.PageSource.


To rocm6.0 onnxrt whl

Update the Jenkinsfile to use a prebuild wheel file instead of always performing a full onnxruntime during our CI.
Should speed up CI by  ~30 minutes per run as we're not building Onnxruntime anymore.
Also moved CI dockerfile to 6.0 here to match with the prebuild onnxruntime wheel

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (7091dec) 91.38% compared to head (d166e02) 91.38%.
 Consider uploading reports for the commit 2e16447 to get more accurate results

Files
Patch %
Lines

src/include/migraphx/op/nonmaxsuppression.hpp
82.14%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #2579   +/-   ##
========================================
  Coverage    91.38%   91.38%
========================================
  Files          454      456    +2
  Lines        17189    17241   +52
========================================
+ Hits         15708    15756   +48
- Misses        1481     1485    +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

this still needs a way to add the build folder so we can pull that blob in to run our CI.

this still needs a way to add the build folder so we can pull that blob in to run our CI.

I saw that.  I approved because I was thinking the Jenkins file is part of AMDs CI infrastructure.  Ideally the test binaries would be published in a location that is publicly available

@TedThemistokleous  should this be updated to support the current 6.1.3 CI environment? It has published whl and zip now.  The move to rocm 6.2 wouldn't likely think of this if it is not there for 6.1 already

@Causten looks like we still don't have the .zip files on repo.radeon even for rocm 6.2. Ill need to figure out why. whls are up though


Selfie Segmentation drops fps when i switch to another browser

System information

OS Platform and Distribution (Win10)
Programming Language : JavaScript

Describe the problem:
When I switch to another browser tab or open another app window in full screen, the fps will drop
I use console.log in the onFrame function of the Camera , but it won't log when I switch to another browser tab
Code: https://google.github.io/mediapipe/solutions/selfie_segmentation.html#javascript-solution-api

Hi @dattt117, Can you provide any references with the above behaviour to investigate further. Thanks!

I use log in the onFrame function of the Camera.
Code:

Result:

There are about 7 log/sec
But when I switch to another browser tab or open another app window in full screen, no log shows up

This sounds like standard browser behavior.  Our simple Camera helper uses requestAnimationFrame in order to register for new video frames, and also checks to make sure it's receiving a new video frame.  However, if the user backgrounds the tab, then to save power/resources, the browser will automatically stop updating the video frames as frequently (or will stop updating them at all).  From the browser's point of view, it usually doesn't want to spend lots of power and CPU/GPU cycles if the animation isn't even visible.  So if you need to keep this alive in the background for some reason, you'd have to write something custom and more complex than our simple for-demo Camera helper utility (and you'd need to use something else besides requestAnimationFrame).


#50 - Update the view when <video> elements are added or removed from the DOM (or when it's style or any parent style changes)

Solves #50

@msach22 Can we get this one out? Maybe in a 3.4.0?

@wolfenrain @mark-veenstra If possible, could you please update the PR so we can get this merged in!
This would be great for a 3.4.0 release

@wolfenrain @mark-veenstra It looks like this PR would break any Android device using API Level 16 or many other newer versions for that matter. It looks like MutationObserver has just been commoditized in the recent versions and will break applications that are using and older version for Android. Based on this, I'm very hesitant to merge this PR.
https://developer.mozilla.org/en-US/docs/Web/API/MutationObserver#Browser_compatibility

@msach22 Regarding the MutationObserver we are using this solution already in production since we migrated to TokBox. And it is working great, as is, from Android 5 and up as we know since we tested this. So I am a bit surprised about the compatibility they claim here (even if you look into the old compatibility tables on that website).
But on the compatibility table they also state that when you want support from Android API level 18 to 26 you can use the vendor prefix WebKit. So if we want to be absolutely sure we could say: var MutationObserver = MutationObserver || WebKitMutationObserver. Then it should be no problem at all, but as I said above on this moment it is not necessary to get it working on these API levels. @wolfenrain can you add this extra check?
Also note that Cordova is only supporting API level 19 ( Android 4.4) - 27 (Android 8) from the most recent version. See this link: https://cordova.apache.org/docs/en/latest/guide/platforms/android/#requirements-and-support. They also will dropped support when Android version dip below 5% on Google's distribution dashboard.
So Android level 16 is even not supported by the most recent Cordova version. Also the market share of API level 16 and 17 is too small to keep it supported, that is why Cordova dropped it also.
This PR helps you to update the views automatically. For everyone that has an unsupported WebView or browser they still can use the old API by calling updateViews() themselves.
So why shouldn't we merge it in, with the change mentioned before?

@wolfenrain @mark-veenstra Thanks for your comments. The OpenTok Android SDK supports Android API level 16 and above and this plugin currently supports Cordova 3.5.0 and above. This means that we would have to do a breaking change to add this functionality.
I would really like to see this feature get merged in because I think it would make using the plugin easier, but we cannot drop support for Android 16.

@msach22 If you still want to support API level 16 (Android 4.1.x) and 17 (Android 4.2.x), then this PR is blocking it. I understand you will not merge this in if this is the business decision.
But we can keep it open for users who want a seperate solution and merge it in themselves. Or when you drop support for those old versions, then it can be merged as well.

@wolfenrain @mark-veenstra Yes, unfortunately I cannot merge this in, but I will leave this open in case developers want to add it to their forks. Thanks again for the great PR!

@mark-veenstra @wolfenrain Do you think we can add conditional observers where if the platform is the supported version we can use this observer, if not they will not have this functionality?

It should be technically possible. It just dawned to me that a polyfill might be possible aswell? Could that be a more all round solution?
On Dec 17, 2018, 7:58 PM, at 7:58 PM, Manik Sachdeva<EMAIL_ADDRESS>wrote:

@mark-veenstra @wolfenrain Do you think we can add conditional
observers where if the platform is the supported version we can use
this observer, if not they will not have this functionality?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/opentok/cordova-plugin-opentok/pull/59#issuecomment-447958511

@wolfenrain Yea having a polyfill might be the best approach. Let me know what you think


Sometimes results' filename is distorted

Sometimes when i use rg and fzf the mappings below, I get incomplete filenames on my results. For example, plugins.lua becomes plugins.lu and etc. This occurs only on some files and they're inconsistent.:
snap.register.map(   {"n"},   {"<Leader>ff"},   function()     snap.run {       producer = snap.get "consumer.fzf"(snap.get "producer.ripgrep.file"),       select = snap.get "select.file".select,       views = {snap.get "preview.file"}     }   end )
Image for assistance.

Nevermind. My bad, a related issue has already been opened #49. Closing.


Associate parameter group with rds cluster

It looks like the rds parameter group isn't attached to the rds cluster at all we should fix that
I believe that this bug causes data definition statements and long queries not to be logged. After we fix this we should verify that we can see those sorts of logs.

There is not a blocked status option. This is very blocked.

@charles-nava thanks. moved to blocked

I see the logs just fine when I attach the parameter group to the cluster. Going to move this back to in progress. I'll pick it up.


thrift_proxy: add upstream cluster metrics for MessageType

Signed-off-by: William Fu<EMAIL_ADDRESS>We have messagetype thrift metrics on the listener side, aka thrift_proxy's connection_manager implementation, but it is also useful to distinguish these counters across each upstream when we have multiple thrift upstreams handled by the router.
Commit Message: thrift_proxy: add upstream cluster metrics for MessageType
Additional Description:
Risk Level: medium (affects thrift customers)
Testing: new unit tests
Docs Changes:
Release Notes:
Platform Specific Features:

cc @rgs1 @fishcakez

Abandoning this PR so recreating a new one so we can emit upstream metrics from the conn_manager, instead of the thrift router (which has no response decoding direction)

@rgs1 can you have another look since a couple things changed?


[BUG] Add test coverage for #1518

Add additional tests to cover automatic session id refreshing to avoid regression of #1518
[BUG] v2.6.0 is not re-hydrating the automatic session id correctly for each request

v2.6.2 is now fully deployed


Add "android" browser to Intl locales

This should hopefully fix the issue reported in https://github.com/Financial-Times/polyfill-service/pull/613#issuecomment-231381636
It seems that the same thing was done in https://github.com/Financial-Times/polyfill-service/pull/698 but that PR also contains other changes which seem to take longer to land.

Thanks!

@triblondon any idea when this change will go live?

It'll be a while. We're about to release 3.8 to live.  Expect 3.9 in a few weeks.

hmm, no chance for a quick bugfix release? I guess the only way around it for now is using flags=always


Jdoze/npmpackageconflict

Remove react-dom/client from index.js.

Reinstall node_modules.

Update package/package-lock.json

Signed-off-by: Jason Doze<EMAIL_ADDRESS>

Let's consolidate all the vm_deploy into one script instead of having darwin/linux_deploy and vm_deploy.


Mutation Refactoring

Just an idea I had, writing it down for the record:
To support mutation change the underlying data storage from &'a [u8] to *mut [u8] (and associated phantom data) then any copy of the Pe<'a> doesn't lock the entire image. Then provide accessor methods which return mut pointers (instead of shared borrows).
There will still be potential for UB, but it gets pushed to the user code and the library remains safe.

wip


Peer review code 07-02

https://github.com/damian1997/web-app-from-scratch-1920/blob/c25b9438fde7a276b2337614e72f51b1accfc15a/docs/js/app.js#L6

Classmates is a global variable, so there is no need to use it as a param

https://github.com/damian1997/web-app-from-scratch-1920/blob/c25b9438fde7a276b2337614e72f51b1accfc15a/docs/js/app.js#L17

Well done on creating an async loop, you might want to check .map instead of a classic loop.

Fixed after refactoring


Undefined error while using with Eleventy v1.0.0-canary

When using this plugin with Eleventy v1.0.0-canary.44, I'm getting TypeError: Cannot read property 'dir' of undefined
Likely this._config in index.js#L9 is undefined.

Thanks for raising this issue @saneef, I thought this might happen at some point! I'll take a look at that version of Eleventy and see if it exposes the config in a different way.


dns lookup's params is a stable ip when i post domin to request under the Charles circumstance

class DMDns :Dns{     override fun lookup(hostname: String): MutableList<InetAddress>      } }
I use add this dns to okhtttp request and under the Charles circumstance I find i get the hostname always is a stable ip, but if my phone is not connect the proxy request directly the hostname is the domin.how to solve that problem?

Thanks for your question. This issue tracker is only for bug reports with test cases and feature requests. Please ask usage questions on Stack Overflow.
https://stackoverflow.com/questions/tagged/okhttp


Is it supposed to be useful also in a scripts?

I plan to make small scale crawling script (between 5 000 and 25 000 pages), is it an usable tool for that or would you recommend an alternative?

It is being used in scripts (ex: here and there, but I guess it ultimately depends on what you want to do in your script: feel welcome to describe more in details how you're thinking to use it, or link to a first implementation of your script, maybe I can make recommendations from there

I planned to write script that will primarily read data, but from what I see in all cases reading data is not done at all or done using SPARQL.
https://github.com/maxlath/wikidata-scripting/tree/master/youtube_links - reads no data - probably will either fail if claim (possibly conflicting one) is existing or it will be overriden.
https://github.com/maxlath/wikidata-scripting/tree/master/import_arbitrary_data_to_wikibase  - seems that deduplication is supposed to happen before this script is run
https://github.com/maxlath/wikidata-scripting/tree/master/import_writers_pseudonymes_from_dbpedia - reading happens via SPQRQL

yes, we currently lack some "smart patching" features, all the data control is indeed expected to be done ahead

This is not a problem for me - rather about is there a good way to get value of a specific claim and so on.

a possibility to get data in a script could be to use wb data, and eventually manipulate the result with something like jq:
# get all Q1 data
wb data Q1

# get all Q1 data in a simplified format
wb data Q1 --simplify

# get all Q1 P1424 claims data
wb data Q1#P1424

# get all Q1 P1424 claims data in a simplified format
wb data Q1#P1424 --simplify

# get the data for the claim identified by Q1-9741c622-fc42-4646-96ec-c594933d74c0
wb data Q1-9741c622-fc42-4646-96ec-c594933d74c0

# get the data for the claim identified by Q1-9741c622-fc42-4646-96ec-c594933d74c0 in a customized simplified format
wb data Q1-9741c622-fc42-4646-96ec-c594933d74c0 --simplify --keep ids,references,qualifiers,hashes,nontruthy

And capture stdin for processing? I guess that main worry is that I still need to manually parse text to hadle errors.

you could either check stdin
Q1_P31_claims_data=$(wd data Q1#P31)
if [[ "$Q1_P31_claims_data" != "" ]] ; do
  echo "Q1 has a P31 claim"
else
  echo "Q1 doesn't have a P31 claim"
fi

or the exit code, which will be 1 if no value can be found
wd data Q1#P31 > /dev/null && {
  echo "Q1 has a P31 claim"
} || {
  echo "Q1 does't have a P31 claim"
}

OK, thanks for answering! I think that I will close it.
Thanks again!


Update service broker and add bind/unbind operation

What this PR does / why we need it:
In this patch, I updated the service broker module and add the bind/unbind operation of storage service, for more information plz refer to the code.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Fix some missing parts.
Special notes for your reviewer:
During the integration test, I found that service-broker couldn't run after I built it, and the error is below:
./service-broker flag redefined: log_dir
panic: ./service-broker flag redefined: log_dir

goroutine 1 [running]:
flag.(*FlagSet).Var(0xc420078060, 0x8909c0, 0xc420010b80, 0x7231ab, 0x7, 0x72f98f, 0x2f)
	/usr/local/go/src/flag/flag.go:793 +0x5e1
flag.(*FlagSet).StringVar(0xc420078060, 0xc420010b80, 0x7231ab, 0x7, 0x0, 0x0, 0x72f98f, 0x2f)
	/usr/local/go/src/flag/flag.go:696 +0x8b
flag.(*FlagSet).String(0xc420078060, 0x7231ab, 0x7, 0x0, 0x0, 0x72f98f, 0x2f, 0x45f75d)
	/usr/local/go/src/flag/flag.go:709 +0x8b
flag.String(0x7231ab, 0x7, 0x0, 0x0, 0x72f98f, 0x2f, 0xc420010b50)
	/usr/local/go/src/flag/flag.go:716 +0x69
github.com/opensds/nbp/vendor/github.com/opensds/opensds/vendor/github.com/golang/glog.init()
	/home/krej/gopath/src/github.com/opensds/nbp/vendor/github.com/opensds/opensds/vendor/github.com/golang/glog/glog_file.go:41 +0x14a
github.com/opensds/nbp/vendor/github.com/opensds/opensds/pkg/utils.init()
	<autogenerated>:1 +0x5a
github.com/opensds/nbp/vendor/github.com/opensds/opensds/client.init()
	<autogenerated>:1 +0x6c
github.com/opensds/nbp/client/opensds.init()
	<autogenerated>:1 +0x4e
github.com/opensds/nbp/service-broker/controller.init()
	<autogenerated>:1 +0x53

I'm wondering that maybe we missed to import some k8s parts, plz take a look at it, thx!
Release note:

@leonwanghui Hi leon, I guess this problem is connected with 'glog' referred by OpenSDS.
I am not sure whether flag.xxx in https://github.com/opensds/nbp/blob/master/service-broker/main.go is conflicted with glog flag.

Got it, thanks!

LGTM


Create CODE_OF_CONDUCT.md

Change Summary

Merge Checklist

[ ] PR has a Changeset
[ ] PR includes documentation if necessary
[ ] PR updates the boilerplates if necessary

I don't think this is needed atm - not many contributors & they're unlikely to read this


Trying out a new thing for pr-4

Testing testing

Before
After

Holy
Moly

#merge-it

#merge-it

#merge

#merge

#merge

#merge

#merge

#merge

#merge

#merge

#merge

#merge

#merge


Clear method does not work in real device when a pin was dragged

This issue happens just in a real device. In my case I am using iphone 5 with IOS 10.3.
This is my scenario:
In my app the user can only have one pin at a time.
So when he tries to add a second pin, instead of displaying Add Pin, it says Move Pin.
When he taps in Move Pin I do the following:
this._pins.Clear();

var pin = new TKCustomMapPin
{
    Position = position,
    Title = string.Format("Pin {0}, {1}", position.Latitude,
    position.Longitude),
    ShowCallout = true,
    IsDraggable = true
};
this._pins.Add(pin);

This works fine in the emulators. But in the real device, only when the user had previously dragged the pin, it doesn't get cleared. So when he Moves the pin, he can still see the previous one, he can see two pins! After that the app crashes trying to do anything.
It seems like a bug in the Clear method that doesn't work when the pin got dragged before.
Please let me know if there is a work around for this issue, ok?
Besides that the API is working great and the business users are loving it.
Thanks

this is an old bug and should be fixed some tme ago, really wondering here which version you are using

I am using 1.5.1

I think it was fixed in 1.5.2-pre1 and is the same as #177 and #185.

Great! I upgraded to 1.6.1 and the bug is gone.
Thanks for your brilliant work!


keras.fit not working with conv2D layer

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
I am using the fashion_mnist dataset to train a convolutional network,
`fashion_mnist = keras.datasets.fashion_mnist
(train_images, train_labels),(test_images, test_labels) = fashion_mnist.load_data()when I use a model with conv2D as first layer, more specifically:model = keras.models.Sequential([
keras.layers.Conv2D(64, (3,3), activation=tf.nn.relu, input_shape=(28,28,1)),
...`
then use the fit function:
`model.fit(train_images, train_labels, batch_size=1, epochs=5)`
an error pops up:
`
ValueError: Error when checking input: expected conv2d_2_input to have 4 dimensions, but got array with shape (60000, 28, 28)`
so I changed the input shape to (28,28,1,0), but another error pops up:
ValueError: Input 0 of layer conv2d_6 is incompatible with the layer: expected ndim=4, found ndim=5. Full shape received: [None, 28, 28, 1, 0]
Describe the expected behavior
The conv nn should have worked just fine
Code to reproduce the issue
`
import tensorflow as tf
from tensorflow import keras
fashion_mnist = keras.datasets.fashion_mnist
(train_images, train_labels),(test_images, test_labels) = fashion_mnist.load_data()
model = keras.models.Sequential([
keras.layers.Conv2D(64, (3,3), activation=tf.nn.relu, input_shape=(28,28,1)),
keras.layers.MaxPooling2D(2,2),
keras.layers.Flatten(),
keras.layers.Dense(128, activation=tf.nn.relu),
keras.layers.Dense(10, activation=tf.nn.softmax)
])
model.compile(optimizer=tf.train.AdamOptimizer(),
loss='sparse_categorical_crossentropy')
model.fit(train_images, train_labels, epochs=5)
'
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
`
ValueError                                Traceback (most recent call last)
 in ()
10     keras.layers.Flatten(),
11     keras.layers.Dense(128, activation=tf.nn.relu),
---> 12     keras.layers.Dense(10, activation=tf.nn.softmax)
13 ])
14
/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/tracking/base.py in _method_wrapper(self, *args, **kwargs)
455     self._self_setattr_tracking = False  # pylint: disable=protected-access
456     try:
--> 457       result = method(self, *args, **kwargs)
458     finally:
459       self._self_setattr_tracking = previous_value  # pylint: disable=protected-access
/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/sequential.py in init(self, layers, name)
108       tf_utils.assert_no_legacy_layers(layers)
109       for layer in layers:
--> 110         self.add(layer)
111
112   @property
/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/tracking/base.py in _method_wrapper(self, *args, **kwargs)
455     self._self_setattr_tracking = False  # pylint: disable=protected-access
456     try:
--> 457       result = method(self, *args, **kwargs)
458     finally:
459       self._self_setattr_tracking = previous_value  # pylint: disable=protected-access
/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/sequential.py in add(self, layer)
172           # and create the node connecting the current layer
173           # to the input layer we just created.
--> 174           layer(x)
175           set_inputs = True
176
/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/base_layer.py in call(self, inputs, *args, **kwargs)
584         # the corresponding TF subgraph inside backend.get_graph()
585         input_spec.assert_input_compatibility(self.input_spec, inputs,
--> 586                                               self.name)
587         graph = backend.get_graph()
588         with graph.as_default(), backend.name_scope(self._name_scope()):
/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/input_spec.py in assert_input_compatibility(input_spec, inputs, layer_name)
121                          'expected ndim=' + str(spec.ndim) + ', found ndim=' +
122                          str(ndim) + '. Full shape received: ' +
--> 123                          str(x.shape.as_list()))
124     if spec.max_ndim is not None:
125       ndim = x.shape.ndims
ValueError: Input 0 of layer conv2d_6 is incompatible with the layer: expected ndim=4, found ndim=5. Full shape received: [None, 28, 28, 1, 0]
`

apparently it was an issue with the input, so I did some reshaping and now it works just fine.


feat: add functions CharLength & CharacterLength

What problem does this PR solve?
Add functions CharacterLength & Charlength
What is changed and how it works?
Code changes

[x] Has Rust code change
[ ] Has CI related scripts change

Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Note for reviewer

I think the functions of Char_Length and Character_Length are same. CharacterLength should be deleted, but when registering, you can create a FunctionSummary with a function name of CharacterLength but other functions consistent with Char_Length.


Registration error

Hi there,
I run the up with docker compose and when I submitted my registration I got the following error:
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) no such table: user [SQL: 'SELECT user.username AS user_username, user.email AS user_email, user.id AS user_id, user.password_hash AS user_password_hash, user.member_since AS user_member_since, user.last_seen AS user_last_seen, user.is_admin AS user_is_admin \nFROM user \nWHERE user.email = ?\n LIMIT ? OFFSET ?'] [parameters<EMAIL_ADDRESS>1, 0)] (Background on this error at: http://sqlalche.me/e/e3q8)

It seems that the table does not exist. Aren't the tables created automatically at runtime ?

Yes, it should be created by running flask db upgrade which is in the docker-compose.yml.
But I could reproduce the problem... :confused:
To workaround this you can log into the container while it's running: docker exec -it todolist sh and then do flask db upgrade. Now the tables are created.
Not sure when I'll find time to understand/fix this. I hope this is helpful for now.

Yeah, that fixed the issue. Thanks for the quick reply.
I am not very familiar yet with how docker-compose actually works. I work only with docker for now.
I will take a loot at it and try to figure out why the command was not running

Cool !!!


MIT OL Devops should get paged when production Concourse fails

Concourse was returning 404s for a time today due to LetsEncrypt rate limiting us. Given that there is an external (OCW) dependency on this service we need monitoring so we can take immediate action upon failure.
Expected Behavior

The devops oncall should be paged when the Concourse production web UI fails.
Current Behavior

No page if Concourse fails.
Steps to Reproduce
N/A - issue occurs whenever Concourse goes down.
Possible Solution

A simple check for an HTTPS 200 would be a good start. Some combination of Pingdom/OpsGenie should do nicely.

This is all set.
Pingdom uptime alert created: https://my.pingdom.com/app/reports/uptime#check=12481603


Density scaling and bandwidth in type_ridge()

In get_density() within type_ridge() the density estimates are scaled to the same maximum which I believe is incorrect. Also the bandwidths are by default computed separately whereas ggridges employs a joint bandwidth estimate which typically seems to be better. But maybe I'm overlooking something here, Vincent @vincentarelbundock ?
For illustration let's consider a sample with two groups with very different variances:
set.seed(0)
d <- data.frame(
  y = rep(1, 100),
  x = c(rnorm(50), rnorm(50, mean = 3, sd = 0.2)),
  z = factor(rep(1:2, each = 50))
)

Using ggridges a joint bandwith of 0.218 is selected and the resulting densities have very different maxima - as you would expect if both should have the area 1.
ggplot(d, aes(x = x, y = z)) + geom_density_ridges() + theme_minimal()
ggplot(d, aes(x = x, y = y, group = z)) + geom_density_ridges() + theme_minimal()

In contrast, because get_density() scales the density to have the same maximum, the second group has a much smaller area than the first in tinyplot(..., type = "ridge").
tinyplot(z ~ x, data = d, type = "ridge", grid = TRUE)
tinyplot(y ~ x | z, data = d, type = "ridge", grid = TRUE)

However, if I drop the scaling in line https://github.com/grantmcdermott/tinyplot/blob/main/R/type_ridge.R#L99 and select the same bandwidth of 0.218, then I get virtually the same result as in ggridges.
tinyplot(z ~ x, data = d, type = type_ridge(bw = 0.218), grid = TRUE)
tinyplot(y ~ x | z, data = d, type = type_ridge(bw = 0.218), grid = TRUE)

Note that type = "density" agrees here and chooses the same joint bandwidth without applying any scaling of the maximum:
tinyplot(~ x | z, data = d, type = "density", grid = TRUE)

Oh that's interesting. I chose this purely for visual reason, and had no principled reason. Sorry!
I'll defer to you on best defaults.

OK, good, thanks for the quick feedback. Removing the scaling to the same maximum is straightforward.
Regarding the bandwidth: Both ggridges and type_density() seem to use the average of the individual bandwidths per group, see: https://github.com/wilkelab/ggridges/blob/master/R/stats.R#L109-L112 and https://github.com/grantmcdermott/tinyplot/blob/main/R/type_density.R#L83-L86
As the code is so similar: Is the type_density code inspired by ggridges or are they both inspired by something else? Why is this simply using the mean rather than the weighted mean (so that larger groups would receive more weight)?
Should we then always enforce that the same bandwith is used throughout? Or should we allow different bandwidth per ridgeline? If the latter, how should we specify this, with an additional argument or does anyone have a better idea?

OK, thanks for the explanation. In any case, I think that type_density and type_ridge should use the same default bandwidth. And I lean towards weighted.mean but I haven't explored this more systematically to check whether it really works better in unbalanced data.


"Same Tree"

Closing Issue/Referencing Issue
issue reference (#1073 )
Changes Made In Pull Request

Created Leetcode folder and added a Leetcode problem solution - https://leetcode.com/problems/same-tree/
Readme file updated

I am genuinely thrilled!!


If images aren't done uploading it crashes the app

When in the admin page if you click upload image for a product and select your image, there is no prompt to let you know when the file is done uploading. If you click upload before the file is done uploading it will crash the app

Cannot reproduce. Needs more info.


Decide what we are doing with normalized cpu

In docs/resources.md we state that cpu is actually in units of KCU, which is normalized CPU.
But, on GCE/GKE we don't do that.  Nor on Vagrant, or anything else.
As @justinsb points out, the AWS cloud provider is reporting cores in ECUs (amazon's normalized core).
GCE is reporting actual cores.
GCE used to have GCEUs but it seems the documentation for that is gone.
So, we need to do something.  Some options:

kubernetes.io/cpu resource is implementation defined.

Can continue to use ECU on amazon
Can continue to use absolute on GCE
Remove mention of KCU from docs.
Config is portable, but performance varies.

kubernetes.io/cpu resource is cores on all cloud-providers.

Change Amazon
Fixup docs
Config is portable, but performance varies

kubernetes.io/cpu resource is normalized for all platforms, and ostensibly comparable across cloud-providers

put actual core count as a new field on nodes, in case anyone cares
define values for GCE, other platforms
Let the battle begin over who's cores are better
Constantly update a table of (processor type, core) to KCU mappings, as there are vastly more types of cpus in the wide world than there are just on amazon and GCE.
Config is portable, and performance varies less

Each platform can define its own normalized unit, with cores as a fallback

kubernetes.io/ecu means AWS ECUs
cloud.google.com/kcu means some normalized unit for GCE processors
Config is not portable, and you are forced to think about performance when porting.

@justinsb pointed out this issue on #10399

subscribe to this issue.

I assumed it would be something like each cluster can optionally define a
mapping of platform-CPU to KCUs and that would be used at scheduling/run
time.  I can think of no better person than you to consider the merits of
these options, though :)
On Fri, Jun 26, 2015 at 4:49 PM, Dawn Chen<EMAIL_ADDRESS>wrote:

subscribe to this issue.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/10438#issuecomment-115920456
.


Support for adding custom extensions for Asciidoctor

I would like to register custom extensions for the Asciidoctor.
Currently Asciidoctor is created inside AsciidocConfluencePage and I do not like an idea to pass extensions to newAsciidocConfluencePage method. Instead I would rather make AsciidocConfluencePage a simple pojo and extract all conversion to the separate class AsciidocConfluencePageBuilder. Then providing extensions to the constructor of AsciidocConfluencePageBuilder will be quite organic:
public class AsciidocConfluencePageBuilder {

    private static final Pattern CDATA_PATTERN = compile("<!\\[CDATA\\[.*?\\]\\]>", DOTALL);
    private static final Pattern ATTACHMENT_PATH_PATTERN = compile("<ri:attachment ri:filename=\"(.*?)\"");
    private static final Pattern PAGE_TITLE_PATTERN = compile("<ri:page ri:content-title=\"(.*?)\"");

    private static final Asciidoctor ASCIIDOCTOR = create();

    static {
        ASCIIDOCTOR.requireLibrary("asciidoctor-diagram");
    }

    public AsciidocConfluencePageBuilder(List<BlockMacroProcessor> blockMacroProcessors) {
        blockMacroProcessors.stream().forEach(processor -> ASCIIDOCTOR.javaExtensionRegistry().blockMacro(processor));
    }

    public AsciidocConfluencePage newAsciidocConfluencePage(AsciidocPage asciidocPage, Charset sourceEncoding, Path templatesDir, Path pageAssetsFolder) {
        return newAsciidocConfluencePage(asciidocPage, sourceEncoding, templatesDir, pageAssetsFolder, new NoOpPageTitlePostProcessor(), emptyMap());
    }
// ... other code omitted
}

public class AsciidocConfluencePage {

    private final String pageTitle;
    private final String htmlContent;
    private final Map<String, String> attachments;
    private final List<String> keywords;

    public AsciidocConfluencePage(String pageTitle, String htmlContent, Map<String, String> attachments, List<String> keywords) {
        this.pageTitle = pageTitle;
        this.htmlContent = htmlContent;
        this.attachments = attachments;
        this.keywords = keywords;
    }

    public String content() {
        return this.htmlContent;
    }

    public String pageTitle() {
        return this.pageTitle;
    }

    public Map<String, String> attachments() {
        return unmodifiableMap(this.attachments);
    }

    public List<String> keywords() {
        return unmodifiableList(this.keywords);
    }
}

@cstettler what do you think regarding this approach?

There is no need to change anything inside the confluence-publisher project for this feature.
Here is a short example how to add custom extensions to the Maven plugin: https://github.com/confluence-publisher/confluence-publisher/issues/95#issuecomment-961808303
I think this issue should be instead resulted in the documentation page on how to do this.


Enhance manager setup based on available CRDs

/kind feature
Describe the solution you'd like
This issue specifically refers to the code of this func: https://github.com/kserve/kserve/blob/99ac7b2050fafb14b7114b94ad6e3fd7ecfe3d15/pkg/controller/v1beta1/inferenceservice/controller.go#L298
Despite the KServe website has instructions to setup KServe under a Serverless environment and also under Raw, the manager setup uses configs to decide what set of CRDs are going to be watched.
Specifically, the decisions around the default deployment mode are not optimal. Users may still prefer using RawDeployment by default while using Serverless mode for the workloads where KNative-related features are needed. However, configuring RawDeployment as the default mode will lead to not watching KNative and Istio related resources, which is probably not OK because users may modify them and break the setup and the controller won't fix the broken resources.
Therefore, rather than using the default deployment mode as a helper to set what resources to watch, it may be better to resolve if the CRDs are available to watch them. In case the user configures Serverless as default deployment mode, the controller would still run fine (right now it continuously restarts because of the missing CRDs) if the user has the policy to annotate all the ISVCs with RawDeployment mode and, if the annotation is forgotten, the manager would start logging errors and that would be fine.

/assign


Host should load the libraries file once at boot

Currently we load the library on every request, but we don't allow the library file to vary.
We should fetch the library once when the python file is executed and then use it for all requests. This will be a bit more efficient in the cases where multiple queries are issued to the same endpoint within a minute or two.

yes please!


"Duplicate this page" not working without GlobalPagePermission of Add page.

Summary

Use "Page -> Create Page -> Duplicate this Page" in toolbar with not superuser, 403 Forbidden error occured.
When login as superuser or user has permission of "Add page (global)" is ok, but created at top of the nodes. Not created in siblings.
No problem with New Page or New Sub Page, because attach GET param named parent_node by PageToolbar.
Expected behaviour
Not need GlobalPagePermission, and copy to siblings.
Actual behaviour
403 error occured.
Environment

Python version: 3.6.5
Django version: 1.11.17
django CMS version: 3.5.3

Hi @victor-yunenko
I think this is not related to #6801.
because this is the issue of CMS_PERMISSION (not django permission) and you wrote "with no code modifications" in that.
I think the setting of CMS_PERMISSION is False by default.

If CMS_PERMISSION is True, the staff user needs both django's and cms's permission to add page, change, etc.
# cms/utils/page_permissions.py

# auth_permission_required checks django's permission.
# has_generic_permission checks cms's permission.

@cached_func
@auth_permission_required('add_page')
def user_can_add_subpage(user, target, site=None):
    """
    Return true if the current user has permission to add a new page
    under target.
    :param user:
    :param target: a Page object
    :param site: optional Site object (not just PK)
    :return: Boolean
    """
    has_perm = has_generic_permission(
        page=target,
        user=user,
        action='add_page',
        site=site,
    )
    return has_perm


Auto scroll chat to bottom when vs code start

Validations

[X] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that requests the same enhancement

Problem
When start vscode and go to continue chat, user need to scroll down all the way to bottom to see the last conversation with llm.
Solution
Make auto scroll to bottom when vs code start and user load extension.

Yeah note that there's a commented-out useEffect in the chat page that used to do this
from here https://github.com/continuedev/continue/pull/2848
But the wait a bit and then scroll to bottom approach is pretty hacky and fragile
flex-col-reverse on session.history is the perfect solution but would be a big change and make backward compatibility with older sessions a real issue (I suppose we could check sessions on load and invert them if e.g. first message is user) but there's a lot of index -1 type of logic that would be sensitive
the challenge is detecting when the chat content has loaded so that the scroll height is valid
A solid but imperfect solution might be to

store a map of index -> loaded on redux
detect when styled markdown previews have loaded and have them dispatch updateIndexLoaded true
when all have loaded (num loaded = history length), scroll to bottom, only once per session change


make sure was can handle mixed label and test that functionality

Some stm files have a mixed label we should allow for. Adds test that exercizes possible labels and I confirmed locally that using an invalid lable causes tests to fail.

I know I've seen this in various corpora, yet this convention challenges my assumption that a line of an STM file is a single speaker. Do we know that downstream digestion of this field in this data structure is okay with the other tools we use?

Yeah, I don't like stm lines not being a single speaker, but if we want this repo to handle most ASR data then it feels like we should allow this.
I don't think this will break any of our downstream digestion. AFAIK, most of our tools ignore this field.


Q: How stable is the current version?

I noticed that there are not user filed issues here.
Is urllib3.future stable?
Any known users/consumers yet?

Indeed, for now. The vast majority of users are silent.
I get reports higher in the stack, but usage constantly increase based on public datasets.
It is stable.

You need to stop closing issues that fast.


Updating Live with the latest changes in Main

Docs Build status updates of commit b0de9d6:
:warning: Validation status: warnings

File
Status
Preview URL
Details

azureadps-2.0/AzureAD/New-AzureADServiceAppRoleAssignment.md
:warning:Warning
View (azureadps-2.0)
Details

azureadps-2.0/AzureAD/New-AzureADMSInvitation.md
:white_check_mark:Succeeded
View (azureadps-2.0)

azureadps-2.0/AzureAD/New-AzureADServiceAppRoleAssignment.yml
:white_check_mark:Succeeded

azureadps-2.0/AzureAD/New-AzureADServiceAppRoleAssignment.md

Line 0, Column 0: [Warning: bookmark-not-found] Cannot find bookmark '#example-4--when-connected-using-a-customer-owned-app-or-service-identity' in 'azureadps-2.0/AzureAD/New-AzureADServiceAppRoleAssignment.yml'.
Line 0, Column 0: [Warning: bookmark-not-found] Cannot find bookmark '#example-4--when-connected-using-a-customer-owned-app-or-service-identity' in 'azureadps-2.0/AzureAD/New-AzureADServiceAppRoleAssignment.yml'.
Line 0, Column 0: [Warning: bookmark-not-found] Cannot find bookmark '#example-4--when-connected-using-a-customer-owned-app-or-service-identity' in 'azureadps-2.0/AzureAD/New-AzureADServiceAppRoleAssignment.yml'.
Line 0, Column 0: [Warning: bookmark-not-found] Cannot find bookmark '#example-4--when-connected-using-a-customer-owned-app-or-service-identity' in 'azureadps-2.0/AzureAD/New-AzureADServiceAppRoleAssignment.yml'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Expand content like theme demo

How to make the content expand like in this theme demo:
https://themes.materializecss.com/pages/demo
Also this seem a Shopify theme, but you said html template.

This is a specialized effect made for the theme. That is also the html theme. There is a separate shopify theme demo. But more importantly, we want to keep this repo focused on just Materialize.
If you have further questions about themes you can email us at<EMAIL_ADDRESS>


Make a 'just changed' cell.

It would be nice if we could make a cell that temporarily highlights (and then fades away) whenever anything inside of it gets updated.  For instance,
cells.ChangeHighlight(cells.Subscribed(lambda: someSlot.get()))

would show the contents of the slot, and any time it changes, flash an outline around the cell for 500 ms, say.

Yes, exactly. For things like marketdata screens that can be super helpful.
On Thu, Jul 11, 2019 at 10:44 AM Eric Gade<EMAIL_ADDRESS>wrote:

Are you imagining something like a border transition/animation, where the
border temporarily "highlights" and then fades away?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/APrioriInvestments/nativepython/issues/172?email_source=notifications&email_token=AB6OHBBJTXX4RB74R4G3A53P65BNBA5CNFSM4IASW4DKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZW5XXI#issuecomment-510516189,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB6OHBESO5EEX3SLJAZWTTDP65BNBANCNFSM4IASW4DA
.

After starting to work on this problem, I've come up with a couple of questions. What do we mean by "update" here?
If we mean that the value of a slot's data has changed, then we are going to need some methods for reflection on Cells that allow us to see which Slots they have at any moment during their lifetimes. Then we can determine which props we pass (ie exportData items) require notification of updates.
If we simply mean that whenever the visual contents of some Cell change, then we have a different problem which is DOM related. In this case "change" can be a child of a child of a child whose contents have changed (and which manifests itself visually), and which, at the moment, triggers all kinds of re-renderings and DOM updates.
One final option -- and right now I think it's the best one -- is to simply kick the can on this Cell until we have removed the "replacements method" of updating the UI. In this near-future system, updates are sent to individual Cells specifically (when they have been recalculated), so we will know which ones have been updated or not. We can then build the ability to "flash" whenever the Component receives an update message into all components, but only "activate" this capability when some parameters have changed -- parameters that will be adjusted by the wrapping function cells.ChangeHighlight(). In other words, ChangeHighlight will tell the enclosed Cell that it's UI component should activate flashing whenever it updates, and "update" here is anything that triggered a re-render.

yeah we don't have to do this now, but i wanted to keep track of it.
the simplest way in my mind would be to have all cells know their parent and walk up the tree any time a cell changes and trigger an action on any parent that wants to know when it changes. A simpler alternative would be to have this be an option we place on 'Subscribed' cels and only trigger it when the "Subscribed" rebuilds itself. not as flexible, but would cover most cases.


feat: updated k8s versions in chart-powerlint.sh to 1.25, 1.26, 1.27

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


Add shorthand prop feature.

Thanks @samuelvanderwaal! Could you add some tests for this, please?

Sure!

Hi @samuelvanderwaal ! May I ask what this PR is about? I'm not sure by reading the code alone :)
I looks like it allows the following syntax, but I'm not sure:
html! {
    <input type="checkbox" checked>
    //                            ^- no equal sign nor value here
}

Thank you!

@totorigolo This PR is for issue #769 but obviously isn't finished and I got stuck with it failing tests. I need to pick it up again and figure out how to complete it. Thanks for the bump!

@samuelvanderwaal It is looks like test failing due to formatting.  Should I be rebasing it on master since there's been a minor version release?

@jstarry Yes, I will pick this back up. Should I be rebasing it on master since there's been a minor version release?

Yes, please!

If it's crucial that this get done soon I can hand it over to someone else if anyone wants to pick it up. I thought I was going to have more time to work on it, but unfortunately with the COVID-19 pandemic I have my special-needs child at home all day and we have a newborn as well so my productive project time has dropped close to zero.
I have rebased my commits onto the lastest commits on master and wrote a basic test for it and can push that. My code no longer passes the test I wrote after being rebased and also appears to have broken the html-component-fail.rs test. The last thing I was working on was tracking down the mismatch in expected error messages for that.

@jstarry I'm not sure how many tests are appropriate for a change like this so let me know if I should add more.

Your tests look good, I'll take it from here. Thanks for finishing this up!

Implementation and tests are solid!
But the lack of contrast between the pass/fail tests really makes me hesitate. It's especially odd-looking in your test that mixes the new shorthand with the standard notation:
html! { <input style="color:red" size= />};

Can you elaborate on what you mean by the lack of contrast?

Can you elaborate on what you mean by the lack of contrast?
Sorry, I don't think "lack of contrast" was a legitimate reason (and I'm not too sure what I meant either)

What I should have said was that the difference between

not adding the value to a non-shorthand property
intending to use a shorthand property but not having the value in scope

is not clear from the syntax. The same syntax for two very different problems basically. I think this could be solved by opting into this shorthand prop syntax with a feature flag so that devs don't stumble upon confusing error messages.


Port option may not be working

I'm running my node server in a Docker container. I was already using --inspect flag so I could bind VSCode debugging system to my server inside the container so I tried using the port option to match the custom port number I was using. However I'm getting this error in the console:
Debugger is not listening on port 9229, 'node-oom-heapdump' cannot function correctly. Is the Node.js process started with the --inspect=9229 flag? { Error: connect ECONNREFUSED <IP_ADDRESS>:9229
Here is the command I run to execute my server:
npx nodemon -e html,js,ts --watch src --inspect=<IP_ADDRESS>:5858 --nolazy dist/index.js
Hence in the main file of my server, I did:
import path from 'path';
import oom from 'node-oom-heapdump';

oom({ path: path.resolve(__dirname, 'my_heapdump'), port: 5858 });

If I change my command to:
npx nodemon -e html,js,ts --watch src --inspect=<IP_ADDRESS>:9229 --nolazy dist/index.js
it now works.

I will look into it and get back to you. Thanks for the report.

I tried to reproduce the issue myself, but wasn't able to do so.
I guess we're using different Node.js versions. This module is tested up until version 10.x.
The following snippet works for me (notice i haven't used ESM, since this is not enabled out of the box in Node.js 10):
`
// started node with: node --inspect=<IP_ADDRESS>:5858 .\index.js
let path = require('path');
let oom = require('node-oom-heapdump');
oom({ path: path.resolve(__dirname, 'my_heapdump'), port: 5858 });
// output:
// Debugger listening on ws://<IP_ADDRESS>:5858/eef6d95f-e3c0-4a06-aed1-a79ef0334208
// For help, see: https://nodejs.org/en/docs/inspector
// Debugger is listening on port 5858, 'node-oom-heapdump' can function correctly.
`
With ESM enabled (experimental in Node.js 10):
`
// started node with: node  --experimental-modules --inspect=<IP_ADDRESS>:5858 .\index2.mjs
import path from 'path';
import oom from 'node-oom-heapdump';
oom({ path: path.resolve('my_heapdump'), port: 5858 });
// output:
// Debugger listening on ws://<IP_ADDRESS>:5858/13d850e3-639b-492b-b7b4-83d238fc5acc
// For help, see: https://nodejs.org/en/docs/inspector
// (node:27252) ExperimentalWarning: The ESM module loader is experimental.
// Debugger is listening on port 5858, 'node-oom-heapdump' can function correctly.
`
So i think it comes down to you using a newer Node.js version.
Can you confirm?

@Nargonath any update on this? Thanks.

@paulrutter Sorry I was in holidays. Concerning the node version we are on the version 10. Here is our docker FROM statement taken from our dockerfile FROM node:10.11-alpine.
I'll try without nodemon when I'll get the time. I'll let you know. Thanks for helping on the matter.

Closing due to inactivity. Reopen if needed.


Garbage key events on startup?

Description

I'm occasionally seeing random garbage key events immediately on startup:
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '?',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '6',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '5',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            ';',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '1',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            ';',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '9',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            'c',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)

Here's another bit of garbage:
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '[',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '?',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '6',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '5',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            ';',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '1',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            ';',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            '9',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:383:5] &event = Key(
    KeyEvent {
        code: Char(
            'c',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)

To Reproduce

Can't figure out anything consistent.
Expected behavior

Environment

OS: Linux
Terminal Emulator: has occurred in Gnome terminal and COSMIC terminal
Font:
Crate version: 0.27
Backend: crossterm

Additional context

My first guess is those might be ANSI escape codes that are feeded into the TUI for some reason.

[?65;1;9;c
This specific code appears to be part of a Device Status Report (DSR) command sequence.

But there is no way to be sure without more info/samples.
I'm curious how you log these events.

I put a dbg!(&event); right here: https://github.com/SUPERCILEX/clipboard-history/blob/bdf4aead1f0bcf81f71c476e539f3ad997aa1212/tui/src/main.rs#L382 and events from here: https://github.com/SUPERCILEX/clipboard-history/blob/bdf4aead1f0bcf81f71c476e539f3ad997aa1212/tui/src/main.rs#L201
I'll see if I can get more samples.

[tui/src/main.rs:382:5] &event = Key(
    KeyEvent {
        code: Char(
            'c',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)

[tui/src/main.rs:382:5] &event = Key(
    KeyEvent {
        code: Char(
            '6',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:382:5] &event = Key(
    KeyEvent {
        code: Char(
            'c',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)

[tui/src/main.rs:382:5] &event = Key(
    KeyEvent {
        code: Char(
            '?',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:382:5] &event = Key(
    KeyEvent {
        code: Char(
            '6',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:382:5] &event = Key(
    KeyEvent {
        code: Char(
            'c',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)

[tui/src/main.rs:382:5] &event = Key(
    KeyEvent {
        code: Char(
            ';',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:382:5] &event = Key(
    KeyEvent {
        code: Char(
            '9',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)
[tui/src/main.rs:382:5] &event = Key(
    KeyEvent {
        code: Char(
            'c',
        ),
        modifiers: KeyModifiers(
            0x0,
        ),
        kind: Press,
        state: KeyEventState(
            0x0,
        ),
    },
)

When it occurs is super inconsistent, but it does looks like the set of characters it's trying to spit out is consistent.

This is more of a crossterm problem than a Ratatui one, so it might be worth asking at https://github.com/crossterm-rs/crossterm for more info on this. The only things that might be Ratatui specific are any calls to crossterm terminal functions that report back strings on the stdin / terminal tty. As far as I know we don't have anything that does this, but there could be something somewhere in the stack that I'm unaware of.
I'd be looking at whether you're fully restoring the termios after running the app. I.e. perhaps you're enabling the mouse capture but not disabling it when exiting, or something similar?

wezterm seem to have explicit support for Device Attributes, so you can read more about this in the source code:
https://github.com/wez/wezterm/blob/56a27e93a9ee50aab50ff4d78308f9b3154b5122/termwiz/src/escape/csi.rs#L3115-L3129
https://github.com/wez/wezterm/blob/56a27e93a9ee50aab50ff4d78308f9b3154b5122/term/src/terminalstate/mod.rs#L1283-L1293
My guess is somewhere in your application there's a request for device attributes and the response is being read by crossterm instead. You can either 1) remove dependencies one by one to filter down to when the issue doesn't happen, and then once you identify the dependency filter down to the lines of code there that is being called, and wrap that with a disable_raw_mode and enable_raw_mode, 2) you can handle the response yourself by reading the keycodes you get from crossterm (if you do handle it yourself, the dependency that made the request will no longer get the response, and may not function correctly).
You can also try to move your terminal init to as "late" as possible and see if that helps.

https://github.com/SUPERCILEX/clipboard-history/blob/bdf4aead1f0bcf81f71c476e539f3ad997aa1212/tui/src/main.rs#L225-L226
    let mut picker = Picker::from_termios().unwrap_or_else(|_| Picker::new((2, 4)));
    picker.guess_protocol();

https://docs.rs/ratatui-image/latest/ratatui_image/picker/struct.Picker.html#method.from_termios

Query terminal for font-size with some escape sequence.
This writes and reads from stdin momentarily.

https://docs.rs/ratatui-image/latest/ratatui_image/picker/struct.Picker.html#method.guess_protocol (also does stdout/in write/reads. See https://docs.rs/ratatui-image/latest/src/ratatui_image/picker.rs.html#103-106 and  https://docs.rs/ratatui-image/latest/src/ratatui_image/picker.rs.html#185-187
// Guess what protocol should be used, first from some program-specific magical env vars, then with
// the typical $TERM* env vars, and then with termios stdin/out queries.

These probably need to happen before you start reading events. You're starting the event reading thread before this code is run.

 Thank you so much for finding that! I created https://github.com/benjajaja/ratatui-image/pull/33 to hopefully help people avoid that mistake in the future.

Thank you to everybody!


OpenAL efx fixes for macOS/Linux

Fixes for using the new efx code with macOS and Linux. This PR also changes linux to use the T3D supplied openal-soft instead of the system supplied one, also changes macOS to use the T3D openal-soft instead of the now deprecated apple opeanl version (that didn't support efx extensions anyway). All 3 platforms now use the same openal-soft build.

Oh yeah should give credit to @practicing01 , he created the original linux efx fix see #2353


[wsi] Implement new WSI abstraction

Implements a WSI abstraction and uses it for DXGI. This code has been used in dxvk native for a while, cleaned it up a little bit.
Primarily looking for feedback that this is going in the right direction.
Didn't bother bringing over D3D9 yet, need to rebase and rework some things there.

All the places that actually use WSI functions should really just include a common wsi.h or wsi_include.h or whatever that just includes wsi/win32/wsi_win32_.h, or wsi/sd2/wsi_sdl2_.h, depending on configuration.

This is handled by the fake windows.h for native already fwiw. The native headers are not included in this PR, just migrating Win32 over to it for now.

I'll still add one for DxvkWindowState though.

Implemented it for D3D9, and addressed the feedback. Opening up for proper review now.

Addressed feedback


Cookie attributes are sent in request

When trying to send a cookie with attributes, REST-assured includes its attributed in HTTP request:
List<Cookie> cookies = new ArrayList<>();
cookies.add(new Cookie.Builder("token", "tokenValue").setDomain("testing.com").setPath("/access").build());

given()
    .cookies(new Cookies(cookies))
    .get("/access")

will result in HTTP GET Request with
Cookie: token=tokenValue;Path=/access;Domain=testing.com

However, RFC6265 is explicitly saying to include cookie name and value in Cookie header. It does not mention attributes to be included. This makes sense, because Cookies attributes are there only for the client to decide whether a cookie should be included in the request or not.
RequestSpecificationImpl.groovy is responsible for that:
if (cookies.exist()) {
  http.getHeaders() << [Cookie: cookies.collect { it.toString() }.join("; ")]
}

where Cookie#toString is serializing not only cookie's name and value but also its attributed.
I will provide a pull request.

@johanhaleby I can fix it but do you have some tests already that verify integration between REST-assured RequestSpecificationImpl and Apache HTTP Client? Basically, the tests that verify that REST-Assured requests are properly translated into HTTP Requests

I guess we can close it now as you merged to master.


replace OpaqueToken by InjectionToken

In ngTS, OpaqueToken has been replaced by InjectionToken, see https://github.com/angular/angular/commit/d169c2434e3b5cd5991e38ffd8904e0919f11788.
In a recent meeting it was felt that this could be a good thing to do in ngDart as well.
cc @kwalrath

I like this change, and we have a volunteer to make it, but it will be post-V3 (new compiler).

Just a clarification: as is explained in #35, const instances are canonicalized, and so only certain values should be used as arguments to an InjectionToken constructor if we want to guarantee uniqueness of InjectionToken.
The most useful values are Types. So I'd suggest not having a generic type argument, and constraining the constructor argument type as follows:
class InjectionToken {
  final Type type;
  const InjectionToken(this.type);
}

(Cf. https://github.com/dart-lang/site-webdev/issues/256)
Relates to #364.

Blocked by https://github.com/dart-lang/angular2/issues/404.

@matanlurey Is this in progress now that #404 is resolved? Do we still have a volunteer for this?

Pushed to 5x.

Straw-man design: Use Inject for both an annotation and a token. For example:
class Inject<T> {
  final Symbol _name;
  const Inject([this._name]);
}

Use in a Provider:
const cachedRpcService = const Inject<RpcService>(#cachedRpcService);

const [
  const Provider(cachedRpcService, useClass: CachedRpcServiceImpl);
  // Can type-check at compile-time this is-a `RpcService`
];

Use in a Component:
class Comp {
  Comp(@cachedRpcService RpcService rpcService) { ... }
  // Can type-check at compile-time this is-a `RpcService`
}

One more potential requirement is that internal users are sometimes scared of using a single application global (i.e. #...) as the identifying token. We should explore allowing users to sub-class Inject:
class XsrfToken extends Inject<String> {
  const XsrfToken();
}


.diff(x, 'days') doesn't seem to use 24 hour days

Description of the Issue and Steps to Reproduce:
There seems to be handling for DST that causes .diff to fail when used with days as units
Please include the values of all variables used.
> a = moment("2017-10-30T00:00:00Z")
> b = moment("2017-10-29T00:00:00Z")
> a.diff(b, 'hours')
24
> a.diff(b, 'days')
0
>

Environment:
OS X timezone CET
node v6.11.3
> console.log( (new Date()).toString())
Mon Oct 30 2017 17:59:49 GMT+0100 (CET)
> console.log((new Date()).toLocaleString())
10/30/2017, 5:59:49 PM
> console.log( (new Date()).getTimezoneOffset())
-60
> console.log(moment.version)
2.19.1

I don't see this mentioned in the documentation, but it looks like the diff for days is operating against local time instead of absolute time.  The local time for your first moment is something like 29th Oct at 2AM, and the second moment is 30th Oct at 1AM.  These look like they're less than 1 day apart in local time.
If you want the result of a diff to use 24 hour units instead of variable-length days (23..25 hours) I think it's more reliable to use foo.diff(bar, 'hours') / 24.
I think the documentation for diff would benefit from explicitly discussing how days are treated (as it already calls out the special behaviour for month and year diffs.)

The relevant documentation isn't linked to in all the right places, but it's here.


Make it clear that properties resourceLinkedService and `folderPath…

…` should either be specified or omitted together.

@StanislawSwierc : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

FYI waiting for confirmation from Data Factory team.

Thank you @douglaslMS!

FYI sent my question again to the Data Factory team. Waiting for confirmation.

@StanislawSwierc Thank you for your patience while I waited to confirm this information with the Data Factory team.

#sign-off

Thank you @douglaslMS. I'm sure other Data Factory users will appreciate this effort and successful resolution.


Fix merging of stream datasets

Summary
When importing a stream dataset with multiple sources in eager mode by specifying error_policy or progress_reporting, an error occurs:
'_MergedStreamDataset' object has no attribute '_data'

How to test

Checklist

[x] I have added unit tests to cover my changes.
[x] I have added integration tests to cover my changes.
[x] I have added the description of my changes into CHANGELOG.
[x] I have updated the documentation accordingly

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below).

# Copyright (C) 2024 Intel Corporation
#
# SPDX-License-Identifier: MIT

@wonjuleee Retargeted this PR to 1.9 release branch. All Geti tests are green. Ready to merge!


Tulip Modifications

Remove precedence
Always prepend instead of append (to have a default)

What's up with this PR? It is referenced from the corresponding helm chart. Was it meant to be merged?

It is used to built the image used for the chart (but is not strictly necessary to merge)


IntelliJ 13.0.2 Community Edition not showing scenario fails when step fails

I have a bunch of stories that I am trying to run using:
@RunWith(JUnitReportingRunner.class)
public class AllStories extends JUnitStories
{
...
}
In my pom.xml I have:
    <dependency>
        <groupId>de.codecentric</groupId>
        <artifactId>jbehave-junit-runner</artifactId>
        <version>1.1.0</version>
    </dependency>

The screenshot shows a step that has failed yet the scenario shows as passed. What could be causing this?

If you don't mind, I'd like to remove the fix since:

recent intelliJ versions don't have the problem any longer and
it introduces two additional failed tests (story failed, scenario failed)


WebView and RTCView playing audio at the same time?

Expected behavior
I want to have the audio from the WebView and RTCView play at the same time.
Observerd behavior
I have a WebView and a RTCView that both play audio. When I render the WebView the audio from the RTCView stops
Steps to reproduce the problem
Platform information

React Native version:
0.43.0
Plugin version:
"^1.58.1
OS:
ios

You need to configure the AVAudioSession with the "mixing with others" option.


METAMODEL-6: Execute update summary

Hereby a new PR for METAMODEL-6. The previous one (#77) would target the master branch. This one targets the new 5.x branch and requires the Java 8 PR to be merged first (see #80).

Looks GREAT to me! Good job Kasper, I'm really happy to see the first Java 8 support PR!! ;)

Thanks, it's merged into "5.x" branch now then :-)


The application crashes when going to the plugins feature on Windows 10

Lapce Version
0.4.0
System information
Windows 10
Describe the bug
The application crashes when going to the plugins feature on Windows 10.
Additional information

Duplicate of https://github.com/lapce/lapce/issues/2781


fix(Makefile): add retrying to workflow CLI download

Also run chmod as a separate command as conditionals negate exit on error that -e is trying to provide
This should prevent the CI system form continuing when there is a problem downloading the CLI and also should let GCS settle down a bit after the binary is uploaded by providing the retry and some preprive (10 seconds) between each retry

Added response header dumping so that we can compare our custom headers (manually) to what the binary thinks it is. Will help with debuging

Things are not green since an older binary is still being downloaded but we have enough data going on now to pin point things better so I am going to merge this