[Flutter Web] Opens a blank Window

flutter run -d chrome or  flutter run -d edge either still opens a blank window and for it actual render someone always has to close the window and it automatically launches another window that renders my application
I had the same problem in beta .. I thought this would be fixed with flutter 2.0 😒

Logs

[ +131 ms] executing: [C:\flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +98 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[   +2 ms] c5a4b4029c0798f37c4a39b479d7cb75daa7b05c
[   +2 ms] executing: [C:\flutter/] git tag --points-at c5a4b4029c0798f37c4a39b479d7cb75daa7b05c
[  +71 ms] Exit code 0 from: git tag --points-at c5a4b4029c0798f37c4a39b479d7cb75daa7b05c
[        ] 2.0.1
[  +65 ms] executing: [C:\flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +46 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/stable
[        ] executing: [C:\flutter/] git ls-remote --get-url origin
[  +44 ms] Exit code 0 from: git ls-remote --get-url origin
[   +9 ms] https://github.com/flutter/flutter.git
[ +109 ms] executing: [C:\flutter/] git rev-parse --abbrev-ref HEAD
[  +49 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] stable
[ +125 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[  +22 ms] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +8 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[  +12 ms] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[  +19 ms] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[ +103 ms] executing: C:\Users\pato\AppData\Local\Android\sdk\platform-tools\adb.exe devices -l
[  +77 ms] List of devices attached
[  +10 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[   +4 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[  +14 ms] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[  +37 ms] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[ +135 ms] Skipping pub get: version match.
[  +48 ms] Found plugin desktop_window at C:\flutter\.pub-cache\hosted\pub.dartlang.org\desktop_window-0.3.0\
[ +599 ms] Found plugin desktop_window at C:\flutter\.pub-cache\hosted\pub.dartlang.org\desktop_window-0.3.0\
[  +60 ms] Generating C:\Users\pato\Documents\Flutter\hello\android\app\src\main\java\io\flutter\plugins\GeneratedPluginRegistrant.java
[ +262 ms] Launching lib\main.dart on Chrome in debug mode...
[ +188 ms] Updating assets
[ +141 ms] Manifest contained wildcard assets. Inserting missing file into build graph to force rerun. for more information see #56466.
[   +4 ms] Waiting for connection from debug service on Chrome...
[  +92 ms] Found plugin desktop_window at C:\flutter\.pub-cache\hosted\pub.dartlang.org\desktop_window-0.3.0\
[ +134 ms] Found plugin desktop_window at C:\flutter\.pub-cache\hosted\pub.dartlang.org\desktop_window-0.3.0\
[ +282 ms] <- reset
[  +25 ms] C:\flutter\bin\cache\dart-sdk\bin\dart.exe --disable-dart-dev C:\flutter\bin\cache\artifacts\engine\windows-x64\frontend_server.dart.snapshot --sdk-root C:\flutter\bin\cache\flutter_web_sdk/ --incremental
--target=dartdevc --debugger-module-names --experimental-emit-debug-metadata -DFLUTTER_WEB_AUTO_DETECT=true --output-dill C:\Users\pato\AppData\Local\Temp\flutter_tools.20a6698a\flutter_tool.62c2fcd9\app.dill
--libraries-spec file:///C:/flutter/bin/cache/flutter_web_sdk/libraries.json --packages C:\Users\pato\Documents\Flutter\hello\.dart_tool\package_config.json -Ddart.vm.profile=false -Ddart.vm.product=false
--enable-asserts --track-widget-creation --filesystem-root C:\Users\pato\AppData\Local\Temp\flutter_tools.20a6698a\flutter_tools.cadf743a --filesystem-scheme org-dartlang-app --initialize-from-dill
build\c7922d95bf4a2462b75c84a97c312edb.cache.dill.track.dill --platform file:///C:/flutter/bin/cache/flutter_web_sdk/kernel/flutter_ddc_sdk.dill --no-sound-null-safety
[  +22 ms] <- compile org-dartlang-app:/web_entrypoint.dart
[+33387 ms] Waiting for connection from debug service on Chrome... (completed in 33.9s)
[   +2 ms] Synced 35.4MB.
[   +1 ms] <- accept
[   +1 ms] Caching compiled dill
[+95097 ms] Using  <! ---- after closing the first windo --->
[ +806 ms] [CHROME]:
[ +427 ms] [CHROME]:DevTools listening on ws://<IP_ADDRESS>:61571/devtools/browser/5a63e291-3174-4f94-98c7-717e30fa38d3
[+2695 ms] DwdsInjector: Received request for entrypoint at http://localhost:61467/main_module.bootstrap.js
[  +10 ms] MetadataProvider: Loading debug metadata...
[  +47 ms] MetadataProvider: Loaded debug metadata
[  +25 ms] DwdsInjector: Injected debugging metadata for entrypoint at http://localhost:61467/main_module.bootstrap.js

[√] Flutter (Channel stable, 2.0.1, on Microsoft Windows [Version 10.0.19042.804], locale en-US)
    • Flutter version 2.0.1 at C:\flutter
    • Framework revision c5a4b4029c (14 hours ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Users\pato\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.8.2)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.8.30717.126
    • Windows 10 SDK version 10.0.18362.0

[√] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code (version 1.53.2)
    • VS Code at C:\Users\pato\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.19.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19042.804]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 88.0.4324.190
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 88.0.705.81

• No issues found!

Hi @Iampato
This used to happen a lot to me, I was gonna file an issue but had no time.  Now trying to reproduce it for the issue, surprisingly  I can't get the blank window when launching the flutter app on the stable channel.

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.1, on Microsoft Windows [Version 10.0.19042.804], locale en-US)
    • Flutter version 2.0.1 at C:\Users\Taha\Code\flutter_stable
    • Framework revision c5a4b4029c (18 hours ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\Taha\Code\SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = C:\Users\Taha\Code\SDK
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.
[✓] Chrome - develop for the web
    • Chrome at C:\Users\Taha\AppData\Local\Google\Chrome\Application\chrome.exe
[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.8.5)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.8.31005.135
    • Windows 10 SDK version 10.0.18362.0
[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Users\Taha\Code\android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
[✓] VS Code (version 1.54.1)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.20.0
[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19042.804]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 88.0.4324.190
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 88.0.705.81
• No issues found!

I'm using the same chrome as you, can you upgrade to 2.0.1 and try?
Thanks

Hi @TahaTesser  check my flutter doctor -v  logs I am on 2.0.1

Hi @Iampato
Thanks! This issue doesn't reproduce on the same version.  I had this issue in the past few weeks but not anymore. Tried in VS Code Terminal as well. Not sure what fixed it.

I tried it on master channel just now and it's not happening. When I close the window the tool exits with "Application finished" and doesn't restart the browser. I'll assume the issue is fixed. We can open a new open if this is happening in some environments.


[Fix] issue #11

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes

Fixed tickets
#11

License
MIT

This PR fix related to issue #11

We feel a certain joy of life in you :)
Thanks @orions


Fixed paths to images in docs

Description
Some links to the linked images in the documentation refer to the wrong folder.
Related Issues
[Reference any related issues or tasks that this pull request addresses.]
Proposed Changes
Fixed the paths
Checklist

[x] I have tested these changes locally.
[x] I have reviewed the code changes.
[ ] I have updated the documentation (if necessary).
[ ] I have added appropriate unit tests (if applicable).

Additional Notes
[Add any additional notes or context that may be helpful for the reviewer(s).]

@microsoft-github-policy-service agree

Thank you for your contribution! In this case, those paths are actually correct. It's a bit confusing, but due to the way the docsite is deployed, they actually are linked correctly even though it looks correct when viewing the markdown. E.g.: https://microsoft.github.io/graphrag/prompt_tuning/auto_prompt_tuning/

Thanks for your feedback. I do see that it works on your link, though the image links appear broken when vieed on Github only for these two. Any solution you can imagine?
https://github.com/microsoft/graphrag/blob/main/docs/query/drift_search.md


MatChip styles dependes on MatChipList

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Use MatChip without MatChipList.
What is the current behavior?
The styles applied to MatChip depends on MatChipList.
What are the steps to reproduce?
Demo: http://plnkr.co/edit/AFbxSJkkeCacnwKEPd2g?p=preview
What is the use-case or motivation for changing an existing behavior?
The ability to use MatChip inline with text since MatChipList adds a wrapper with display: flex.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
<EMAIL_ADDRESS>Is there anything else we should know?
There are a weird behavior, so the styles are applied despite of MatChip is not inside MatChipList. Seems that dependes but... not depends 🤔  (show demo).

This is because MatChip is a Directive and not associated with any css file while MatChipList is a Component and has chips.css in styleUrls.

This issue needs to be closed

Duplicate of https://github.com/angular/components/issues/9964.


ZEP1: Apply review feedback for terminology section

This PR applies some feedback from the ZEP1 review in https://github.com/zarr-developers/zarr-specs/pull/149. It adds illustrations to the terminology, and improves the store and storage transformer sections. In detail:

Added two illustrations, one for the general hierarchy and array terminology, and one for the codec/array/storage transformers/store workflow. Both were made with https://excalidraw.com. The png files include the source and can be loaded into excalidraw for further editing.
Added examples to store terminology section, as requested in
https://github.com/zarr-developers/zarr-specs/pull/149#discussion_r928313883
Improved the storage transformer terminology section, as requested in
https://github.com/zarr-developers/zarr-specs/pull/149#discussion_r928314024
Reordered the terminology definitions slightly, to fit better to the illustrations
(moved codec and compressor more to the end, after the second illustration).

Screenshots of the rendered specs with the illustrations
(The changes from the ZEP1 review were applied to the main branch for the screenshots)

This PR replaces the original PR https://github.com/alimanfoo/zarr-specs/pull/1, as we discussed to gather all PRs here onto the main branch (see https://github.com/zarr-developers/zarr-specs/pull/149#issuecomment-1302440391).

Thanks for sending this PR, @jstriebel.
The illustrations are excellent, and the text looks clear to me. I have one minor feedback:
If I try to view the illustrations here, I can't view them properly. This is because they don't have a background colour.
Should we add the illustrations with white background?
PS. I don't have a strong opinion on this, as I know these illustrations can be viewed on the web browser just fine. It's just the .md file.

If I try to view the illustrations here, I can't view them properly. This is because they don't have a background colour.
Should we add the illustrations with white background?

@MSanKeys963 Good point, next time I'll add the white background, but I'd leave it as-is for now for simplicity.


set comm_overlap as true by default for Zero3

Which can get 1%-2% percentage performance improvement for scale-out and this is the feature we are WIP, should set by default.

Performance improvement is expected from the separate streams of compute and communication when the flag is set. It will be explicit when allgather is fully implemented by copy engine.


Install GitOps operator

What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
Installing OpenShift GitOps operator through test script
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes NA
How to test changes / Special notes to the reviewer:

++ oc get csv -n openshift-operators
+ operator_status='openshift-gitops-operator.v1.0.0             Red Hat OpenShift GitOps               1.0.0                                                  Installing'
+ [[ openshift-gitops-operator.v1.0.0             Red Hat OpenShift GitOps               1.0.0                                                  Installing == *\S\u\c\c\e\e\d\e\d* ]]
++ expr 4 + 1
+ count=5
+ sleep 10
+ '[' 5 -lt 5 ']'
+ echo 'Completed OpenShift GitOps operator installation'

Increasing timeout

/lgtm

Sorry! Wrong PR
/lgtm cancel

++ oc get csv -n openshift-operators
++ grep openshift-gitops-operator
+ operator_status='openshift-gitops-operator.v1.0.0             Red Hat OpenShift GitOps               1.0.0                                                  Installing'
+ [[ openshift-gitops-operator.v1.0.0             Red Hat OpenShift GitOps               1.0.0                                                  Installing == *\S\u\c\c\e\e\d\e\d* ]]
++ expr 2 + 1
+ count=3
+ sleep 10
+ '[' 3 -lt 15 ']'
++ oc get csv -n openshift-operators
++ grep openshift-gitops-operator
+ operator_status='openshift-gitops-operator.v1.0.0             Red Hat OpenShift GitOps               1.0.0                                                  Succeeded'
+ [[ openshift-gitops-operator.v1.0.0             Red Hat OpenShift GitOps               1.0.0                                                  Succeeded == *\S\u\c\c\e\e\d\e\d* ]]
+ break
+ echo 'Completed OpenShift GitOps operator installation'

Gitops operator installation succeeded
Details log - https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/redhat-developer_kam/171/pull-ci-redhat-developer-kam-master-v4.6-integration-e2e/1361955981180801024/artifacts/integration-e2e/integration-e2e-steps/build-log.txt

/lgtm

/lgtm
/approve


pipenv

Checklist

[x] I've run make from the root directory to see all Dockerfiles are created successfully.

Motivation and Context
This PR relates to https://github.com/circleci/circleci-images/issues/168

LGTM !


DDFileLogger mixed log messages from framework and main project

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation or CocoaDocs
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
ios

Platform Version
12.0

CocoaLumberjack Version
3.5.1

Integration Method
cocoapods

Xcode Version
Xcode 10.1

Repro rate
all the time (100%)

Demo project link

Issue Description and Steps
Both main project and framework have a different DDFileLogger, and I want that framework fileLogger only record logs from framework. And The current result is both two loggers record all logs from framework and main project.

There are multiple ways to achieve this. For one, you could use your own DDLog instance in the main app (instead of the shared one). Or you could use a DDLogFormatter that filters logs from your main app (using e.g. the context parameter).

There are multiple ways to achieve this. For one, you could use your own DDLog instance in the main app (instead of the shared one). Or you could use a DDLogFormatter that filters logs from your main app (using e.g. the context parameter).

Thanks a lot. I'll try DDLogFormatter.


antd-mobile.less to be processed into PostCSS plugins, but not want to be processed into CSS Modules!

i want node_modules/antd-mobile/dist/antd-mobile.less to be proccessed into postcss-pxtorem plugin but not want to be proccessed into css-modules,  this is rep https://github.com/fangjj/test-css-module
if i config  passthroughPaths: [ "client/antd.less" ] then node_modules/antd-mobile/dist/antd-mobile.less will not be proccessed into postcss-pxtorem plugin.

any update?

This seems like a reasonable feature to include, but I'm not sure when I'll be able to do so. I'm open to PR's as well. In the meantime, the best solution I can think of is to add juliancwirko:postcss (see instructions here: https://github.com/juliancwirko/meteor-postcss/)  which will apply postcss to your files after nathantreid:css-modules has done it's part. I haven't tested it with my plugin, but I think they should be compatible.

thank you! i add juliancwirko:postcss to proccessed what CSS Modules passthroughPaths. like this.
 it works for me. but i have a question for this.

See here for more info:
https://github.com/nathantreid/meteor-css-modules/blob/master/README.md#postcss-plugins
On Tue, Jun 19, 2018, 8:08 PM redim<EMAIL_ADDRESS>wrote:

thank you! i add juliancwirko:postcss to proccessed what CSS Modules
passthroughPaths. like this.
[image: image]
https://user-images.githubusercontent.com/15318800/41631803-3a660fce-7469-11e8-8671-efcea42cf1ac.png
it works for me. but i have a question for this.
[image: image]
https://user-images.githubusercontent.com/15318800/41631845-7f27b5b8-7469-11e8-8012-e8bbcd3b24e3.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nathantreid/meteor-css-modules/issues/120#issuecomment-398592442,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLBNtwAFflXQb_hAkwtCmDzb8XZwtQmks5t-aCYgaJpZM4UlnJ4
.

yeah, got it. when  remove this required, but it still work.


Make it easy for users to get in touch

Not sure how to do this, but probably just including my email address on the main page, e.g. on the bottom, or in the docs.

I think this is fine. Docs have lots of details.


How to removeEventListener timeupdate?

How to remove the listener in Plyr player?
el.removeEventListener('timeupdate', event => { console.log("remove"); });

Use .on() .off() and .once():
https://github.com/sampotts/plyr#api

Use .on() .off() and .once():
https://github.com/sampotts/plyr#api

Thanks! But I got the error when I used .off()
[Vue warn]: Error in v-on handler: "TypeError: el.off is not a function"

What's el ? It needs to be an instance of Plyr.

What's el ? It needs to be an instance of Plyr.

Below is my code, but it doesn't word...
//todo removeListener let player = new Plyr(el); player.off('timeupdate', event => {     console.log("removeListener"); })

Hi @sampotts I don't know why but off isn't removing timeupdate event. Can you please help us with that  and take a look at this issue https://github.com/sampotts/plyr/issues/2249


Textbox Context Menu and Spell Check

I have come accross a problem with the context menu changes to TextBoxes. In the code (MaterialDesignTheme.TextBox.xaml) on line 27 is a comment "cludge the default context menu". It seems this "cludge" is preventing the normal spell check menu from displaying.
Sorry for not adding code examples; but I don't have access at the moment and I wanted to submit the bug before I forgot!

Pretty sure this behavior is expected. See https://msdn.microsoft.com/en-us/library/bb613557(v=vs.110).aspx on how to implement spell checking with a custom context menu.


Small optimisation in InspectAssembly's GetFiles

This replaces the single Directory.GetFiles call in InspectAssembly with two calls that only grab .exe and .dll files.  In my very limited testing when enumerating the whole of C:\Windows\System32, it reduces the file count by a few hundred and saves about half a second when iterating over them with GetAssemblyName.
Even though the final count of the result list was no different with these filters, I'm not sure if there are cases where you need to find files of different extensions?

LGTM!


suggested ux improvements

A few things i noticed playing with the current git version (4.0.1-16-ge0e59dd).

The File>Exit menu item should either be "E&xit" or "&Quit" (i.e. the keyboard shortcut should be x or q, as with any other app, but not "&Exit" as it currently is).
The "enter" key does nothing in the passphrase dialog when the checkbox has focus (i.e. enter passphrase, tab to checkbox, space to check it, pressing enter should click OK as it does when the text box has focus, but it does not)
When the search text box has focus, I'd expect the Escape key to clear the current search terms (muscle memory from using search in web browsers like Chrome or Firefox)

Another one, a bit more trivial:

I would suggest that the timer tick rate for updating the progress bar should probably be lower. You'd be surprised how much battery life is impacted in a laptop or tablet when something as simple as a blinking cursor is on screen (it prevents the Intel graphics chip from entering RC6p) -- or in this case, a very rapidly updating progress bar. You could probably update it every second or so and still show just as effectively how much time is remaining.

Thanks for the feedback, and I agree with all your first three points.
Regarding the timer, yes it does check every 100 milliseconds now, maybe it is  possible to check at a lower rate and still have a smooth progress bar (maybe some animations could help?).

@dagheyman I'm suggesting that even 100ms is too short a delay if you care about battery life. It keeps the GPU awake to redraw the progress bar. Also, I just noticed the timer doesn't stop ticking when minimized to the system tray, which means it's always consuming CPU even when backgrounded.

@dagheyman If you were to implement the change regarding the timer, would that affect desktops?  If so, maybe make it an option that can be enabled in Settings for laptops and such?

Closing since two of these are fixed and the third one (clear search box) has a separate issue.


CAMEL-17861, properly handle FileInputStreamCache

necessary fix for:
https://github.com/apache/camel/pull/7493
where determining the size of FileInputStreamCache was not possible

already fixed


After doing some operation, I am getting element not found error for previously accessed element.

Expected Behavior
It should found the element and continue the process.
Actual Behavior
its failing and showing error as "element not found"
Steps to Reproduce the Problem

Short Example of Code to Demonstrate the Problem
main_window=Application(backend="uia").connect(title_re="Editor*")
pane= main_window.window(class_name="WindowsForms10.MDICLIENT.app.0.2bf8098_r33_ad1",control_type="Pane",top_level_only=False)
Note :- getting errors as element not found for "WindowsForms10.MDICLIENT.app.0.2bf8098_r33_ad1" but the same element is displayed while running py_inspect.py
Specifications

Pywinauto version: 0.6.8
Python version and bitness: python 3.6.8 and 64 bit
Platform and OS: 64 bit and windows

please help me to find a solution.

This does not appear to be a bug or anything with pywinauto.
My bet is that your first statement is connecting to some other application that also matches Editor* regex.
I suggest you to improve your expression, since "Editor" is a very commom name (use path= instead).

No Actually, this is sample name, just to display the syntax I have given here. After connecting to application, I am able to do any operation. In my application, on each step same window will be updated with some new data. so when I am trying to use already accessed element properties , I am getting element not found error.

Without more info, it is hard to help.
We can only guess where is your problem, and I think this is not the correct place to do this (I am not a project maintainer, but this is not a community forum to ask for help). Try StackOverflow or the Mailing List.
Anyway, run these commands direct in a python terminal. Once you connect, try to run print_control_identifiers() to list the available controls, and check if your controls are available. I question myself if the control you're searching for is really there.
Another thing that you may consider is that, in .NET Applications, the class name is somewhat random.

WindowsForms10.MDICLIENT.app.0.2bf8098_r33_ad1 <-- these bold numbers may change

Double check if it is not the case.

Hi I checked that too, its okay thank you for your help, Actually the syntax and everything is correct. still getting a error. I don't know whether the problem is in pywinauto. I thought this might have any work around. thank you.

Maybe visible_only=False inside main_window.window(...) can help. Also app object returned by connect method is not a main window but a process that has a pid. That's why we name this object app in all examples. Then you can list top level windows in this process by app.windows(). For particular top level window there is method app.window(title_re="Editor.*").dump_tree().
Another way is using desktop = Desktop(backend="uia") to walk the whole windows hierarchy without binding to process. Hope it helps.


Fixes open pr check

Issue #, if available:
Description of changes:
I have no idea what i tested and how when i first made this change, but this tweaks the cli call just a bit to better utilize the newer version of the cli and should be correct now...
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Few comments

Hi, this idea seems lovely, but currently the implementation has some painful moments:

Encyrpting doubles file size, it will be probably super hard to avoid, if not impossible, but at least you should note that in README. I don't know if hacks like that: https://news.ycombinator.com/item?id=12262470 would make it possible to read the contents of the binary data, probably not, but that's something I remember reading, further research is needed

Encrypting freezes the browser. Opening a encrypted 9MB file (so 18MB HTML) freezes the browser. Chrome wants to kill the page when I try to focus input box - super bad UX here, not sure how could you embed this data within HTML without this. Encrypting could be moved to web workers, but with file:// protocol you have when you open local html file this is impossible. But you can at least have this for hosted HTML files I guess?

The encryption & decryption pages could use some nice UI design and UX. It's currently quite ugly and after using it once when decryptin all I care is the password box, yet most of the screen is covered with explainations, huge password hint, advertising product name and very cryptic (haha! a pun!) Details section

Proposed easy solution to UX:
Because there is no chance that all the people in the world would like my approach to decrypt page, which for me should look like that: https://dribbble.com/shots/5757939-Password-requirements (without the characters counter, and the text below input would be a password hint, OPTIONALLY I could display some text above the password field for first time users):

add a way to customize CSS
add a way to customize texts

Not sure how it'd be the best way to save my customizations so I don't have to copy/paste them each time.
Tell me what do you think about my thoughts

I think your thoughts are great! Agree 100%


What's new name of 'NzModalSubject' in 0.7?

What problem does this feature solve?
Chinese:
在0.6中的NzModalSubject现在叫什么名字?我找了好久也没找到,通过官方API,我只看到了父组件向子组件传递信息的代码
const instance = modal.getContentComponent();
但是没有找到子组件向父组件传递信息的代码,能给我个示例吗?
English:
NzModalSubject is version0.6's name,but I don't know it's new name.I spent much time to find it.From official API,I just find parent component transfer data to child component.
const instance = modal.getContentComponent();
But not find how child component transfer data to parent component,can u give me a demo?
What does the proposed API look like?
Chinese:
我不知道是这个功能目前没有还是我没找到关键代码,如果是没有的话,我认为有必要新增一个,子组件向父组件传递消息是一个很常见的应用场景.
English:
I don't know either Ng-Zorro doesn't make it or I didn't find core code.If doesn't make it,Personally,It's a very useful application scene.

查看 NzModalRef

@Eve-1995 Hi, can u share with me how to solve it please? I have  the same problem.thx!

@Flcwl Sorry,I can't help u.As matter of fact,I did't find the way.


Scrolling back to top

After I implemented it on body, the back to top link stopped working. How can I scroll back to top of the page with this plugin?

That worked perfectly! Thanks for the late reply.


cli: add peerDependencies to devDependencies in package.json.hbs templates

Refs: https://github.com/backstage/backstage/issues/24674#issuecomment-2098408642
Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hi! Sorry I've been slow getting to this. Happy to accept this but just wanted to check - is this something we should be doing in general everywhere? We don't do this in most of our regular packages; the idea probably has been that this is filled in by the test harness / dev app / app in all cases anyway. Should we revisit that?


WithFunction fallback gotcha for int64s

I'm having an issue where the comparison functions don't work properly if they are handed an int64, rather than an int. I discovered this when I tried to write a function that returned the result of time.Time.UnixNano().
Here's a minimal demonstration:
func demo() {
	doStuffInt64 := func(query string, value map[string]interface{}) interface{} {
		jqp, _ := gojq.Parse(query)
		code, _ := gojq.Compile(jqp,
			gojq.WithFunction("func", 0, 0, func(x interface{}, xs []interface{}) (out interface{}) {
				return int64(x.(int))
			}),
		)
		iter := code.Run(value)
		iv, _ := iter.Next()
		return iv
	}

	// Outputs 'true' as 3000 >= 2:
	fmt.Println(doStuffInt64(`(.a|func) >= (.b|func)`, map[string]interface{}{
		"a": 3000,
		"b": 2,
	}))

	// Outputs 'true' even though 3000 <= 2:
	fmt.Println(doStuffInt64(`(.a|func) >= (.b|func)`, map[string]interface{}{
		"a": 2,
		"b": 3000,
	}))
}

The type switch in binopTypeSwitch() doesn't account for int types that aren't int, so the int64 case hits fallback(), which compares only on type rather than value, and the result is that the code behaves incorrectly with no way of really knowing why without going for a spelunk through the source.
I believe the range of int types should be supported for comparisons, or an error should be raised when a disallowed type is encountered. I'm not very well steeped in the internals of the codebase though, maybe there's a better approach? Regardless, I think "silently behaving incorrectly" qualifies as a bug.
I am happy to have a crack at sending a PR, whatever your implementation preference, if it would be welcome.

The gojq library does not support int64 type so return int or *big.Int.

I understand that, I should've mentioned in my original post that I've "solved" my problem by converting to one of the supported types. The problem is that the library violates the "principle of least astonishment" by silently working incorrectly in the presence of an unsupported type, and the only way to discover that was to spend an hour reading the source.
Maybe this is simply a documentation deficiency, and the docs for WithFunction should make clearer that you should not return int64, but I think that's not quite enough. I think the API should either support int64 or protect against its use, not silently work incorrectly.

Sorry about that I will add notes on this, thanks.


Missing dll

エラー	1	'C:\Users\XXX\source\repos\OpenH264Lib.NET\Debug\OpenH264Lib.dll' メタデータが見つかりませんでした。
it seems like a circular dependency. I need OpenH264Lib.dll to make my app work, while the sample project complains about the same missing dll
What to do?

Hello. Thank you for your question.
After opening 'Open H264 Lib.sln' Visual Studio solution file,
build OpenH264Lib project at first by clicking Visual Studio "Build" -> "Build OpenH264Lib" menu. (Not "Build" -> "Build Solution")
Or from Solution Explorer, right click "OpenH264Lib" and build. (not OpenH264Sample!)
After build "OpenH264Lib", build "OpenH264Sample" project.

Thanks for your quick response. I have done as you mentioned. All worked.

I updated README.md. Thank you.


Add development requirements to package and instructions to README

This does not add PySide2 or PySide1, but it does add the dependencies for flake8 and coverage, as well as documentation for how to install the dependencies to the README.
PySide2 Snapshot wheels are available through the following:
pip install --index-url=http://download.qt.io/snapshots/ci/pyside/5.9/latest/ pyside2 --trusted-host download.qt.io
and are hosted at http://download.qt.io/snapshots/ci/pyside/5.9/latest/pyside2/
Since you may be about to merge in #160 I chose not to add PySide2 for 2 reasons.
1 ) To not get out of date immediately :)
2 ) The above are snapshots and therefore not versioned. I think until PySide2 enters into versioned tagging, it should be up to the person managing the distribution of OTIO in their facility to handle which PySide2 version they have available.
Something to ponder about once they do enter into a versioned tag schema is whether to actually add PySide2 as an installation dependency. I know that in my facility (Laika) we would have to fork and block this requirement in the setup.py as we build our own PySide/PyQt.

Of course, after I write that, I come across this blog post that states that they had a release earlier today:
https://blog.qt.io/blog/2018/06/13/qt-python-5-11-released/
http://download.qt.io/official_releases/QtForPython/pyside2/
I've checked and it's not in pypi yet, but additionally, the versioning scheme is still a little odd - I wouldn't expect to see PySide2-5.11 in a setup.py.

Yes, you are correct. We should be able to add a key called 'view' that will only install PySide and then users can optionally call it with pip install .[view] if they choose to install PySide2. This fulfills the condition of making the PySide install optional for studios as well.

@alexwidener did you want to add that [.view] target to this PR as well?  Or make that a separate PR?

We should add it as a separate PR once PySide2 is publicly available on pypi.org


perf: change target to ES2018

fix #26

It will remove _awaiter, __rest ts helper function from  lib/ and module/.

It is released in v2.1.0
https://github.com/azu/kvs/releases/tag/v2.1.0


Temporary tooltip upon load (not for mouse hover)

Hi,
Can I use your tooltips to show tooltip for a few seconds after page is loaded? The reason is to attract attention of  of users to some elements of the page. If a customer will visit the page the first time, it will help to understand navigation of the page.
Thanks in advance.
Best regards,
Galym

Hi,
sure, use timeouts and call the 'show' method. I thinkthere even is an example for this in the documentation. Thanks.

Hi,
Suppose I have element on the page with id="element". Can I show tooltip onload event like below shown ?
    $(document).ready(function () {

        $('element').tooltipster('show');
    });

$(window).load(function () {
    $('#element')..tooltipster().tooltipster('show');
});```
Please head over to stackoverflow for further implementation questions. Thank you

thanks!


Issue/16 create api routes

Unless the branches have really been messed up, this seems to be identical to the pull request just approved by @heart2read . Hopefully this is just a snafu with GitHub, but either way I'm closing this for now.


support re-initialization in PacketStream

Other side might re-initialize a packet-stream (e.g., send hello packet after client message). This case needs to be handled gracefully.

this will result in nonce fail. stream should close whenever something weird is detected. after timeout, other side will re-initialize


Edit install manual

Hey !
Thanks for the PR, it's true I forgot to update the documentation when I moved to gulp :)
Could you please just check the comment about the last step and I gladly merge your contribution :)

Thanks again for the PR.
However I decided to start using npm instead of gulp to build the extension so I add to do other modifications. I also updated the documentation in the gh-pages branch.
I took inspiration in what you did though !


Paniced at hot_targets.gete(t).unwrap()

In the tui-logger crate version 0.12.0, there is a panic occurring at src/lib.rs:833:59 with the following code:
let t = &self.targets[i + offset];
let hot_level_filter = hot_targets.get(t).unwrap(); // Paniced!
let level_filter = targets.get(t).unwrap();

https://github.com/gin66/tui-logger/blob/64129b13dc327a00342e110f81ce85c16da14656/src/lib.rs#L842-L846

tui-logger-0.12.0\src\lib.rs:833:59:
called Option::unwrap() on a None value

The panic happens occasionally under certain conditions, but the exact cause is unclear.

Thanks for the report. I have modified the code and hope, that it is now fixed.

Remaining issue is, that there is still an unexplained root cause. So the fix only masks an underlying bug….

My current usage involves first creating a TuiWidgetState and then setting the level for a specific target. However, if the target does not have any logs yet, this error occurs. Is this by design?
In my current setup, I set the default output level to info and also set default levels for certain targets.
let mut state = TuiWidgetState::new().set_default_display_level(LevelFilter::Info);
state = state.set_level_for_target("some::logger", LevelFilter::Off);

Additionally, another question is that most of logging systems are typically asynchronous(I have not fully read your code yet, sorry~). This means that even if I proactively log a message before call the render method, this error might still occur, right?.

Thanks for the explanation and based on your input I have been able to reproduce the problem. Based on my investigation, there has not been an underlying root cause. It is just an oversight, that every list can have an own mapping of target to log filter value, which is not mirrored to the logger. Consequently, the unwrap() is incorrect and unwrap_or() is a valid fix. I have added a comment to explain this in the code.
The new version is already published.

Thanks for you reply and quick fix😊


Fixes the issue of pytest failing with scheduled jobs

When you schedule a job it's in a background process with no terminal. When we invoked pytest it would die trying to report the status to the nonexistent terminal.  So, just unregister the TerminalReporter so it doesn't complain.
Also made it so if pytest dies the job reports as 'dead'.

Tests passed for this pull request.
Refer to this link for build results: http://jenkins.ceph.com/job/teuthology-pull-requests/842/

Tests passed for this pull request.
Refer to this link for build results: http://jenkins.ceph.com/job/teuthology-pull-requests/843/


feat: add rootless-containers/slirp4netns

rootless-containers/slirp4netns: User-mode networking for unprivileged network namespaces
$ aqua g -i rootless-containers/slirp4netns

Thanks!

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.75.0


RequestTimeout is not respected when API is called with context

We have set the RequestTimeout to 75ms in our DAX client.  However, when TCP connection is re-establshed, we still see some DAX request in the 6s range (which based on our discussion with AWS team are caused by connection being re-established).
Looking at the Go SDK, it appears that the issue is due to SDK not respecting the request timeout if it is called with context.
See https://github.com/aws/aws-dax-go/blob/264dfde337bc592b0ada5a8c587f14f610cd922a/dax/service.go#L163
Please note that we use QueryWithContext as per https://pkg.go.dev/github.com/zabawaba99/aws-dax-go/dax#Dax.QueryWithContext.

Are you setting timeouts in the context that's passed to DAX SDK? If you are explicitly passing in a context, it should look like below if you want to configure 50 ms timeout:
ctxTimeout, cancel := context.WithTimeout(context.Background(), time.Millisecond*50)

client.GetItemWithContext(ctxTimeout, in)

When context is passed to the SDK, we will honour the timeout that's set in the context and will not consider the RequestTimeout field value which was set during client initialisation.
That's why in the below code, the request timeout is overridden if context is provided -  https://github.com/aws/aws-dax-go/blob/264dfde337bc592b0ada5a8c587f14f610cd922a/dax/service.go#L163
I executed couple of tests to validate this behaviour and it works as expected:

I set the RequestTimeout configuration as 10 ms and while calling GetItemWithContext I provided context.Background() (infinite timeout)

There were no timeouts and we would see high latencies (note: simulated high server-side latencies by manually adding sleep on server-side)

I set the RequestTimeout configuration as 5 seconds and while calling GetItemWithContext, passed in context with 50 ms as timeout - context.WithTimeout(context.Background(), time.Millisecond*50)

I saw that client-side latencies were capped at 50 ms

Are you setting timeouts in the context that's passed to DAX SDK? If you are explicitly passing in a context, it should look like below if you want to configure 50 ms timeout:
ctxTimeout, cancel := context.WithTimeout(context.Background(), time.Millisecond*50)

client.GetItemWithContext(ctxTimeout, in)

When context is passed to the SDK, we will honour the timeout that's set in the context and will not consider the RequestTimeout field value which was set during client initialisation.
That's why in the below code, the request timeout is overridden if context is provided -
https://github.com/aws/aws-dax-go/blob/264dfde337bc592b0ada5a8c587f14f610cd922a/dax/service.go#L163
I executed couple of tests to validate this behaviour and it works as expected:

I set the RequestTimeout configuration as 10 ms and while calling GetItemWithContext I provided context.Background() (infinite timeout)

There were no timeouts and we would see high latencies (note: simulated high server-side latencies by manually adding sleep on server-side)

I set the RequestTimeout configuration as 5 seconds and while calling GetItemWithContext, passed in context with 50 ms as timeout - context.WithTimeout(context.Background(), time.Millisecond*50)

I saw that client-side latencies were capped at 50 ms

We did not set the timeout in the context that we passed into the QueryWithContext request.
When we call the API, we do something like
	cfg := dax.DefaultConfig()
        cfg.RequestTimeout = 75 * time.Millisecond
        client, err := dax.New(cfg)
        ...
        ctx := context.Background()
        // some code to add in our own context info
        output, err := client.QueryWithContext(ctx, input, opts)

There was nothing in the documentation to suggest that Config's RequestTimeout (i.e., https://github.com/aws/aws-dax-go/blob/264dfde337bc592b0ada5a8c587f14f610cd922a/dax/service.go#L49) will only be set if there is no context provided.

QQ - why have the RequestTimeout config if the context dominates, especially if context.Background() overrides the top-level config setting?
Perhaps I may suggest that RequestTimeout is just dropped altogether in favor of the more idiomatic Go style of providing solely the context with timeout.

Thank you for the feedback! We'll update the documentation in the next release to call out that the top-level configs are not honoured if the context is set.

Perhaps I may suggest that RequestTimeout is just dropped altogether in favor of the more idiomatic Go style of providing solely the context with timeout.

That's a good suggestion but unfortunately if we make that change now, the change will be backwards incompatible.

That's a good suggestion but unfortunately if we make that change now, the change will be backwards incompatible.

@anudeeb looking forward to github.com/aws/aws-dax-go/v2 😄 !


Implement toggle aware icons and labels in command actions

This PR fixes #92283
Add support for toggle aware icon and title in command action

@jrieken This PR is still in progress.
I am currently stuck at handling toggle state aware titles. I introduced toggle-aware icon (as discussed) and also needed to introduce the same for the title.
Menu items are sorted by title and in case of toggle-state aware items, title is computed based on toggle state.
https://github.com/microsoft/vscode/blob/37da787e91f01c6cfee3d4c50992782d9e072d92/src/vs/platform/actions/common/menuService.ts#L174-L176
Any suggestions on how to sort these? Shall I compute the title by evaluating the toggle state expr?
I am also not sure how to handle such commands while displaying in the keybindings editor. What title shall I have to show. This raises the point that whether this toggle command action is a single  or two command actions? IMO they are two different command actions as they perform two different actions. If so one has to register two command actions for each state. What do you think?

IMO they are two different command actions as they perform two different actions. If so one has to register two command actions for each state. What do you think?

Yeah - that was always my point of view and this is how it is currently achieved. Tho, I think it is a little cumbersome because most things are equals (e.g keybinding). On the other hand, having two distinct commands leaves no uncertainty in what direction to toggle e.g what your current state is. A related, alternative, design might be a richer toggled-property. Instead of a simple context key expression have an object with describes all toggle-properties (label, icons, tooltip (?)). Tho, you might endup repeating the whole command action data here... That would bring you to what's discussed here (which I am not a fan of): https://github.com/microsoft/vscode/commit/9f29782486dd19378ee1dceb0cb4064100096f73

@jrieken Looks like we cannot always split a toggle command into two, since there is a need to support a single command that can be toggle-able and users can attach keybindings to that.
Here is my suggestion, I would still want to go with current approach but instead   of having toggle-able titles, lets have toggle-able tooltips. Thinking about the toggle-able commands, I feel that the title shall always be fixed and saying that it is a toggle command. It's the tool tip that shall say the state of the command.
So I would introduce - toolTip property to ICommandAction. what do you think?

My preferred solution (in the name of simple consumer adoption) goes likes this
export interface ICommandAction {
	id: string;
	title: string | ILocalizedString;
	category?: string | ILocalizedString;
	icon?: { dark?: URI; light?: URI; } | ThemeIcon;
	precondition?: ContextKeyExpression;
	toggled?: ContextKeyExpression | { 	toggled: ContextKeyExpression, icon, title };
}

Overall, adding tooltip makes sense, both for all states. An alternative is to have ToggleTitle, ToggleIcon etc but would increase adoption cost and also somehow spread the toggle-ability everywhere

Discussed with @jrieken and agreed upon adding additional properties to toggled property and we will add icon and tooltip for now. title can be added after when needed.
export interface ICommandAction {
	id: string;
	title: string | ILocalizedString;
	category?: string | ILocalizedString;
    tooltip?: string | ILocalizedString;
	icon?: { dark?: URI; light?: URI; } | ThemeIcon;
	precondition?: ContextKeyExpression;
	toggled?: ContextKeyExpression | { toggled: ContextKeyExpression, icon?: { dark?: URI; light?: URI; } | ThemeIcon, tooltip?: string | ILocalizedString; };
}

To me this seems like unnecessary complexity, why build this on top of toggled (which currently is just for showing/hiding a checkmark -- right?). Menu placements are already conditional, so why not just allow changing the title/icon in the placement. IMO, that is more powerful, flexible, and easier to understand.
For example, if I have a command toggle.command, and depending on the context, I can have a menu/command palette placement for when it is on showing Toggle Off with a specific icon (if desired) and a placement for when it is off showing Toggle On with its specific icon.

Menu placements are already conditional, so why not just allow changing the title/icon in the placement. IMO, that is more powerful, flexible, and easier to understand.

The only reason they are conditional is that menus stay small and don't show a bunch of meaningless entries - this has nothing to do with the properties of an action (label, icon etc). I really don't understand, and you need to explain this, why for instance the save command should be called "Save" in the context menu and "Delete" in the command palette.

@jrieken Updated the PR aligning to our discussion. Please take a look.


Can I edit scss?

Am I able to safely edit the scss to my likings? As long as I stick with the concurrent divs and don't edit the file?  Thanks :smile:

Hi, I see Jekyll Now as simply a starting point from which you can customise the site however you like. For example, I used Jekyll Now as a starting point to create my blog. But then I changed the styling to use a different colour scheme, and even added new functionality.
So yes, you can edit the SCSS however you like. You can also edit the HTML or Jekyll code however you like. As for whether it is "safe" to do so… Naturally it is possible to make changes that will break your site or have some other detrimental effects. But if that happens, it's just a matter of fixing the problem. And if you use git and commit your changes regularly, you can simply revert any changes you've made, so it's not irreparable.
I hope this answers your question.


Support legacy storiesOf

Trying to convert from Webpack to Vite, I'm seeing this error:
Uncaught Error: Cannot add an argTypes enhancer to the store after a story has been added.
    at StoryStore2.addArgTypesEnhancer3 (story_store.js:536)
    at ClientApi2.addArgTypesEnhancer (client_api.js:132)
    at addArgTypesEnhancer2 (client_api.js:72)
    at vite-app.js:61

It's caused by having a storiesOf entry in a story file.
import React from 'react';
import { storiesOf } from '@storybook/react';

storiesOf('Dynamic', module).add('Story', () =>
Test
);

I'm guessing it's because module is undefined in the Vite context, but not sure what (if anything) to replace it with.
While it is a legacy API, it is also still the only way to add a dynamic list of stories (something that fits perfectly with import.meta.globEager).

I don't think it's possible. module doesn't exist in an ESM context. Maybe something like import.meta? @shilman should there be a way to support this?

There's a good chance we'll remove storiesOf support in 7.0 -- CSF will have been out for over two years -- and we're starting to release features that depend on CSF.
In short, I probably wouldn't prioritize this one.

@shilman there still isn't a way of dynamically creating stories using CSF? It is a nicer API, but we like to use it to dynamically include a glob of components/pages, as a way of ensuring we always include all components.
I can get around it, but I'll have to create a story file for each component manually (or with a generator).

For CSF we support globs for the name of the story files, but within the file the stories must able to be statically analyzed. I have a strawman for how to implement combinatorial tests in this way. But something simple like "create a story for every component" which would be pretty easy in storiesOf requires manual work in CSF (or implementing a webpack loader--does vite have a similar concept?).


Diagnose/investigate timeouts on TA live tests

Only happens on Live tests. I haven't been able to repro this locally.
Started happening after adding the custom features tests.

After multiple runs, it looks like custom features with multiple actions could be the cause. I will communicate with the service and meanwhile, we can disable those tests
PR: https://github.com/Azure/azure-sdk-for-net/pull/25042

Not currently seeing timeouts, outside of very occasional outliers.   Closing this out and will continue to monitor.

@jsquire  From Mariana's comment, it seems we just stopped seeing timeouts because the tests causing it have been disabled.

After multiple runs, it looks like custom features with multiple actions could be the cause. I will communicate with the service and meanwhile, we can disable those tests

From my understanding, this issue was still left open to track the work of checking with the service team and re-enabling those tests (we have three disabled tests that point to this issue). I'm opening it again so we don't forget about it.

@joseharriaga please check if this is still an issue.


potential bug for other size images (not 512 by 512)

Hello there,
Bug description: if the image size is not 512 by 512, the junctions and line_map we got from step 3 are based on the image which has been resized to 512 by 512. Now if we want to visualize the preds. We will go to train_preprocessing_exported (ex: in holicity_dataset), the image in data (one of the inputs) will be the original size of the image (ex: 2040 by 2040). The size is not equal to the size in the config file (512 by 512), so the code will resize the image, and the JUNCTIONS (the junction here is obtained based on 512 by 512 image).
In my opinion, you should not resize the junction here.
I don't know if there is any potential effect on line_map because I think the coordinate for line_map is based on the junctions.
Let me know if this is incorrect.
Thanks

Hey,
I think ur right. When I try to construct my datasets with non 512*512 image. The junction label is just unmatched,giving rise to a long while wrong training process.

when training with your own datasets, please make sure following codes commented:
junctions = (junctions * np.array(
self.config['preprocessing']['resize'], np.float)
/ np.array(size_old, np.float))

Hi, I think that you are indeed correct. This bug had never been caught because the Wireframe and Holicity images were already at 512x512, but it would not work for other image sizes.
I have fixed it now and pushed the new version. Thanks for reporting the bug!


Task :connectivity:compileReleaseJavaWithJavac FAILED

Hi there, I'm not able to build for release. It works on debug mode. Due to this error, I can not release my app.
See the complete report on Gradle scan.
https://scans.gradle.com/s/cdkyow3cp4jpo
Thank you., Prakash

Hello @prakash-indorkar ,
Could you please post the outputs of flutter run -v and flutter doctor -v and a complete minimal reproducible code sample?
Thank you!

@prakash-indorkar
What Version are you trying to build for release ? Please provide flutter build apk --verbose.

Hi, @sidrao2006 and @darshankawar
Surprisingly it built successfully today, after changing my JDK version from 14 to 1.8. I read your suggestion on one of the similar issues. I did install the jdk1.8 but forgot to change JAVA_HOME which was pointing on jdk14.
Thank you so much for your attention. I like being part of the flutter community.

Closing this as non-issue based on comment above.


Footer Visibility in mobile view

Describe the bug
Footer isn't completely visible in mobile.
To Reproduce
Steps to reproduce the behavior:

Go to Home page
Scroll down a little
See error

Expected behavior
The footer can be positioned as static, but when it is, in desktop view, the footer will be directly below the main tag, which can be adjusted by responsive design conditions.
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser: Brave
Version: v1.69.162

Smartphone (please complete the following information):

Device: Redmi Note 7 pro
OS: Android 10
Browser: Chrome
Version: 127.0.6533.103

for me its completely visible

can you help me reproduce it

here's the resolution i'm using

@maheshtamarakar
The footer is not completely visible add some padding and make it look good. Thank you!!


Fixes #766

Did the bond query override the single bond designation?  I.e. was this really a cosmetic issue?

It's kind of cosmetic. The query would always take precedence in substructure searching, but it made sense to me to fix this since there is an UNSPECIFIED bond type.


Script to download tinycore linux.

Work towards automated testing #24.
Download tinycorelinux from a mirror and patch.
Update hyperkitrun.sh to call script and also explicitly call make if
needed.
Tested commands by using hyperkitrun.sh
Signed-off-by: Pris Nasrat<EMAIL_ADDRESS>

LGTM. Thanks!

I had a couple of late-breaking very minor comments above, but nothing to prevent a merge, thanks!

Thanks! I'll do a follow up when I add a simple automated test which
addresses the comments.
On Wed, Jun 8, 2016 at 5:23 AM, Ian Campbell<EMAIL_ADDRESS>wrote:

I had a couple of late-breaking very minor comments above, but nothing to
prevent a merge, thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/docker/hyperkit/pull/34#issuecomment-224536006, or mute
the thread
https://github.com/notifications/unsubscribe/AAA15kLMpr_E3ZUsbisVToeZtDfJIEzlks5qJooogaJpZM4IvKUv
.


[Website] Increase maxHeight of codeblocks

Description

Make sure the codeblock maxHeight fits the content of the Usage source in Getting Started.
Motivation and context

Closes #477
Screenshots, videos, or demo, if appropriate

https://477-code-block-max-height--mineral-ui.netlify.com/
How to test

cd mineral-ui && npm start
Go to Getting Started
Make sure you don't have to scroll the code block
Go to a component demo with a long source (MenuItem's Custom Render is good)
Make sure you do have to scroll that code block

Types of changes

Other (provide details below)

Just a site thing.
Checklist

[x] Renders and functions properly in all supported browsers
[x] Automated tests written and passing [n/a]
[x] Accessibility and inclusivity considered
[x] Rendering performance (initial load time & 60fps) and perceived performance considered [n/a]
[x] Documentation created or updated [n/a]
[x] Tested in sandbox if new component or breaking change [n/a]

@kylegach this looks way better. thank you. is there any way to get the styles to be shared since they are for the same purpose?

@coldpour — We could put the max-height rule in the prismTheme css, but then we won't have access to theme variables. I kinda like that it'll "just work" if we ever adjust the font size or line height.

@kylegach yea definitely, I like that part too. But what about pulling this out into a function that takes the theme and spits out a maxHeight?

@coldpour — Check the first commit, please.
I also added a second commit (will be squashed post-review) that completely removes prismTheme.css in favor of placing all of the styles in the new getCodeBlockStyles util (plus a few on the inline code selector in Markdown).

@kylegach this is gorgeous! nice work sir


*: deprecate grpc.Code, grpc.ErrorDesc

grpc.Code, grpc.ErrorDesc, grpc.Errorf have been deprecated.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@bc50a45). Click here to learn what that means.
The diff coverage is 79.31%.

@@            Coverage Diff            @@
##             master    #8531   +/-   ##
=========================================
  Coverage          ?   65.93%
=========================================
  Files             ?      360
  Lines             ?    29539
  Branches          ?        0
=========================================
  Hits              ?    19477
  Misses            ?     8397
  Partials          ?     1665

Impacted Files
Coverage Δ

clientv3/leasing/kv.go
1% <0%> (ø)

proxy/grpcproxy/leader.go
59.18% <0%> (ø)

clientv3/client.go
68.55% <100%> (ø)

clientv3/retry.go
56.74% <100%> (ø)

proxy/grpcproxy/cluster.go
72.28% <100%> (ø)

etcdserver/api/v3rpc/rpctypes/error.go
80.95% <81.81%> (ø)

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 bc50a45...6f76d52. Read the comment docs.


Adding LinkedIn Link in the footer section

Adding LinkedIn Link in the footer under social connection section

/assign , assign me under GSSOC2023

Hey @codervivek5,I would like to add LinkedIn Link in the footer under social connection section.
I'm a contributor to GSSOC'23 please assign this issue to me as a part of GSSOC'23


remove wg text

What are your changes?
What is the urgency level of this change?

[ ] Blocker
[ ] High
[x] Medium

I have completed these items:

[x] I have built locally and tested for broken links and formatting.
[ ] If appropriate, I have added redirects.

If you included a comment with your commit, it appears here:

@mhausenblas can you please merge this fix?


Use Ruuvi icon in Ruuvi Station notifications

Description:
We want to change thermometer icon to Ruuvi icon in notification, because now collapsed notifications will not show app name and it's just clearer to use Ruuvi icon.
Current:

New, use same icon as on iOS:

Ruuvi eye files:
ruuvi-eye.zip

Let's use the white one whenever it is possible.

Background color when needed: #009088

Need to spec this, will not include in 2.10


Value of type 'PP' has no member 'setBadgeLabel' !

what happened to this function?
buttonItem?.pp.setBadgeLabel(attributes: { (badgeLabel) in
      badgeLabel.font = UIFont.systemFont(ofSize: 13)
      badgeLabel.textColor = UIColor.blue
})

in the last version and Swift 5
it gives me an error

Value of type 'PP' has no member 'setBadgeLabel'

From what I gathered, it has been removed in favor of BadgeControl!
See 1a7724d6a79a09c268985d788feef7477b4c0e7a for more information.

@X901 Any solution found?

Not a solution, but a workaround:

Set the property textLabel public instead of private in PPBadgeControl

Convert the code, from:

buttonItem?.pp.setBadgeLabel(attributes: { (badgeLabel) in
      badgeLabel.font = UIFont.systemFont(ofSize: 13)
      badgeLabel.textColor = UIColor.blue
})

to
buttonItem?.pp.badgeView.textLabel.font = UIFont.systemFont(ofSize: 13)
buttonItem?.pp.badgeView.textLabel.textColor = UIColor.blue

my 2c,

@jkpang could you help ?


Proposed repo restructuring

Following up on the discussion in our Wednesday meeting:
Proposed github repo structure:

 - bobanetwork/boba-v3
 	A top-level container, initialized as an empty repository.

 - bobanetwork/v3-optimism
 	Forked from ethereum-optimism/optimism

 - bobanetwork/v3-op-geth
 	Forked from ethereum-optimism/op-geth

 - bobanetwork/v3-erigon
 	Forked from ledgerwatch/erigon

 - bobanetwork/v3-erigon-lib
 	Forked from ledgerwatch/erigon-lib

 - bobanetwork/v3-erigon-interfaces
 	Forked from ledgerwatch/interfaces

 - bobanetwork/boba
 	Our current V2 repository

The boba-v3 repo would import each of the others as a submodule, with a
directory structure like:

boba-v3
  optimism [v3-optimism]
    op-geth [v3-op-geth]
  erigon [v3-erigon]
    erigon-lib [v3-erigon-lib]
      interfaces [v3-erigon-interfaces]
  legacy [boba]
  ops
    docker-compose.yml
  packages
    contracts
      (custom Boba contracts)
  boba_examples
    (...)
  boba_utilities
    (...)
  Makefile

The top-level Makefile (and/or the ops/docker-compose.yml file) would be able
to build the entire stack, recursively descending into the optimism and erigon
submodules as needed to build those packages and docker images. If there are
any aspects of the V3 system which are unchanged from the V2 version then they
can be built from the 'legacy' submodule.

I am proposing that we use branches named "boba-develop" and "boba-main" in
each of the v3-* repositories. These would be kept in sync with the "develop"
and "main" branches in the top-level boba-v3 repo. This way we will be able
to keep our versions of the repositories in sync with the upstream locations
without having to worry about conflicts on their "develop", "main", etc.
We will manually merge changes into our "boba-develop" branch at suitable
intervals, such as when they tag an upstream release candidate or fix an issue
which was blocking our progress. We would merge our "boba-develop" into our
"boba-main" when we were preparing to tag one of our own release candidates.

Note that the upstream repositories have different naming conventions, so we
might be tracking "devel" in one, "main", "alpha', etc. in others. This is
another reason to give our branches the "boba-" prefix.

For code that we want to contribute upstream, we can create a fork from their
development branch and cherry-pick the particular code changes that we wish
to propose.

Like it! Just a naming thing, shouldn't we stay consistent? boba-v3 --> v3-boba or switching all other repos to *-v3?

Love it! Created the structure above:
https://github.com/bobanetwork/v3-boba
All repos are synced up and clean and should work.
I'll add my custom logic in separate branches now.
Happy for feedback

The current repo may be archived for reference.


Add example number 3

Description:
$subject.

I think we can do something similar to this
https://github.com/gorilla/websocket/tree/master/examples/command

I think we can do something similar to this
https://github.com/gorilla/websocket/tree/master/examples/command

We will have to wait till OS command execution is implemented in Ballerina. It is planned to implement for the Swan Lake Update 2.
https://github.com/ballerina-platform/ballerina-standard-library/issues/2963


[WFLY-12087]:Upgrade CXF to 3.3.3

https://issues.jboss.org/browse/WFLY-12087
This upgrade will fix security vulnerability issue in org.apache.santuario:xmlsec dependency.

@jimma The JDK 11 failures are due to a
 [01:59:25][Step 2/3] [INFO] --- maven-enforcer-plugin:3.0.0-M2:enforce (ban-bad-dependencies) @ wildfly-galleon-pack ---
[01:59:54][Step 2/3] [WARNING]
[01:59:54][Step 2/3] Dependency convergence error for javax.xml.bind:jaxb-api:2.2.7 paths to dependency are:
[01:59:54][Step 2/3] +-org.wildfly:wildfly-galleon-pack:18.0.0.Beta1-SNAPSHOT
[01:59:54][Step 2/3]   +-org.apache.wss4j:wss4j-bindings:2.2.4
[01:59:54][Step 2/3]     +-javax.xml.ws:jaxws-api:2.2.9
[01:59:54][Step 2/3]       +-javax.xml.bind:jaxb-api:2.2.7
[01:59:54][Step 2/3] and
[01:59:54][Step 2/3] +-org.wildfly:wildfly-galleon-pack:18.0.0.Beta1-SNAPSHOT
[01:59:54][Step 2/3]   +-org.hibernate:hibernate-search-orm:5.10.5.Final
[01:59:54][Step 2/3]     +-javax.xml.bind:jaxb-api:2.3.0
[01:59:54][Step 2/3]
[01:59:54][Step 2/3] [WARNING] Rule 1: org.apache.maven.plugins.enforcer.DependencyConvergence failed with message:
[01:59:54][Step 2/3] Failed while enforcing releasability. See above detailed error message.
[01:59:54][Step 2/3] Failed to execute goal org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M2:enforce (ban-bad-dependencies) on project wildfly-galleon-pack: Some Enforcer rules have failed. Look above for specific messages explaining why the rule failed.
[01:59:54][Step 2/3] [INFO]

We don't use javax.xml.bind:jaxb-api, we use a JBoss forkorg.jboss.spec.javax.xml.bind:jboss-jaxb-api_2.3_spec so I think adding some exclusions to the root pom.xml will fix this.

Retest this please


Setting up the Kanban board

This is the link to the project https://github.com/Aquinasmoriasi/catalog-of-my-things
Our team has 2 members:

@TuyishimireEric - Team member 2
@Aquinasmoriasi - Team member 1

APPROVED 🟢
Hi @Aquinasmoriasi @TuyishimireEric,
Your setup is complete!  There is nothing else to say other than... it's time to start your group project :shipit:

Cheers and Happy coding!👏👏👏
Please, remember to tag me in your questions if you have any so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


binance: 1.25.0 -> 1.26.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/binance/desktop/releases.
meta.description for binance is: "Binance Cryptoexchange Official Desktop Client"
meta.homepage for binance is: "https://www.binance.com/en/desktop-download"
meta.changelog for binance is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/5klcb5d0x60pavpwb7nx2mqvisqwyj6a-binance-1.26.0/bin/binance had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.26.0 with grep in /nix/store/5klcb5d0x60pavpwb7nx2mqvisqwyj6a-binance-1.26.0
found 1.26.0 in filename of file in /nix/store/5klcb5d0x60pavpwb7nx2mqvisqwyj6a-binance-1.26.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
binance

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5klcb5d0x60pavpwb7nx2mqvisqwyj6a-binance-1.26.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A binance https://github.com/r-ryantm/nixpkgs/archive/55c44e3542ea87907b4b87acc272298c5ad47dcc.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5klcb5d0x60pavpwb7nx2mqvisqwyj6a-binance-1.26.0
ls -la /nix/store/5klcb5d0x60pavpwb7nx2mqvisqwyj6a-binance-1.26.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

binance

Maintainer pings
cc @wolfangaukang for testing.

Merging this request will affect #143070 (in the way of requiring a rebase)

@WolfangAukang do you still want me to merge this or do you want to incorporate it into your PR?

@WolfangAukang do you still want me to merge this or do you want to incorporate it into your PR?

You can merge this, no problem. I can work out with the other PR later.


Move to using gorm for richer query and tx support

WIP for Joins

@krishnamiriyala, what do you think about keeping AppUser, App, Group structs in datastore_test.go file, instead of moving them to a non-test file (data.go) and exposing them to the public? These structs are only used in datastore_test.go
Whatever would be left in data.go  could also be moved to datastore_test.go or protostore_test.go (a couple of constants would need to be duplicated in both of the test files, I believe)

@krishnamiriyala, what do you think about keeping AppUser, App, Group structs in datastore_test.go file, instead of moving them to a non-test file (data.go) and exposing them to the public? These structs are only used in datastore_test.go Whatever would be left in data.go could also be moved to datastore_test.go or protostore_test.go (a couple of constants would need to be duplicated in both of the test files, I believe)

*test is a common way to add test helper data and methods. let's leave it for now, it is easier than having m x n cross-referencing for test helper code.

@jeyhun Resolved all comments any other fixes?

@jeyhun Resolved all comments any other fixes?

Added couple of comments regarding .md file. All looks good


Calculator program in py

I want to add a calculator program in the Python section

could you please assign me for this, i wanna contribute

Sir,kindly assign me this issue
I am interested to contribute to this code.

could you please assign me for this, i wanna contribute !!

@hyprex-deva Could you please assign this task to me?
Issue no. #397

sir please assign this issue to me

@hyprex-deva I have added calculator program using GUI in python
Pull request no.#482


Observatory needs an icon

Observatory needs its own icon. It needs both a 48x48 pixel icon and a much smaller icon that can be used in Dart Editor for the console.
The attached image shows what we are using now inside of Dart Editor.
On the Tools index page, I am using the same icon that is used for Dart Editor, but it would be better to have an Observatory-specific icon.

Not planned.


Got some errors when setting the system-docker graph to RANCHER_OEM partition

Format the disk like this:
[root@rancher rancher]# blkid
/dev/sda1: LABEL="RANCHER_STATE" UUID="b5f98664-2a5c-4e84-9f2c-e5909684f64c" TYPE="ext4" PARTUUID="477d1443-01"
/dev/sda2: LABEL="RANCHER_OEM" UUID="c724524d-cf25-4374-a423-1d8e56c31c0b" TYPE="ext4" PARTUUID="477d1443-02"

Use ros install -t noformat -d /dev/sda ... to install ROS to the disk.
Change the system-docker graph:
ros config set rancher.system_docker.graph /usr/share/ros/oem

Check the system-docker after reboot:
[root@rancher rancher]# system-docker info
...
ID: 2Z4H:FTPT:QLT5:ICFC:NFSD:O2BM:LXRZ:22WU:UKP3:4ZN3:C4F2:6BFE
Docker Root Dir: /usr-c8ebc32/share/ros/oem
Debug Mode (client): false
...

[root@rancher rancher]# df -hT | grep oem
/dev/sda2            ext4            1.8G    191.3M      1.5G  11% /usr/share/ros/oem
/dev/sda2            ext4            1.8G    191.3M      1.5G  11% /usr/share/ros/oem/overlay2
overlay              overlay         1.8G    191.3M      1.5G  11% /usr/share/ros/oem/overlay2/b1e37639706516b0e10914af0f8856ddc99bc55628934be2c512402611837faa/merged

But the user-docker cannot work:
[root@rancher rancher]# docker info
Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?

[root@rancher rancher]# system-docker logs docker
time="2018-10-03T00:27:18Z" level=info msg="[PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin HOSTNAME=rancher HOME=/]"
time="2018-10-03T00:27:18Z" level=info msg="Running [system-docker-runc exec -- 1147e621e2866ff7d02e8e997bf640b907f753fb2b4fb5ba95ca65ca6c3d7805 env PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin HOSTNAME=rancher HOME=/ ros docker-init --group docker --log-opt max-file=2 --log-opt max-size=25m --host unix:///var/run/docker.sock]"
exec failed: container_linux.go:262: starting container process caused "process_linux.go:73: starting setns process caused \"fork/exec /proc/self/exe: no such file or directory\""

I should figure out this issue.
What to do if the user wants to change system-docker graph to a separate partition?

If anyone changes the data-root of system-docker to /usr/share/ros/oem (default mount point of RANCHER_OEM partition), and user-docker cannot work. Here is the workaround:

get the real root dir of system-docker

[root@rancher rancher]# system-docker info | grep "Docker Root Dir"
Docker Root Dir: /usr-9672fb7/share/ros/oem/system-docker

change the user-docker service in /usr/share/ros/os-config.yml

    docker:
      image: rancher/os-docker:18.03.1
      ....
      ....
      volumes_from:
      - all-volumes
      volumes:
      - /sys:/host/sys
      - /usr-9672fb7/share/ros/oem/system-docker:/usr-9672fb7/share/ros/oem/system-docker  #here

rebuild the user-docker service:

$ system-docker rm -f docker
$ ros s up docker

I don't recommend this method, it is too complicated to use.


De-Dupe developerID hashing logic.

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
Hashing logic to hash the developerID is duplicated across events.
TODO comment: DevHome.Common.RepoCloneEvent.
Expected result
No response
Actual result
No response
Included System Information
No response
Included Extensions Information
No response

Fixed in #1207


[video_player_web] Add a custom analysis_options file to video_player_web.

Description
flutter/plugins is currently red because inline analysis checks are no longer honored, as described here: https://github.com/flutter/flutter/issues/52899.
video_player_web used a web-only API that was being flagged (correctly) by the analyzer, but suppressed inline via // ignore comments.
Now that // ignore comments don't work, the tree has gone red.
The fix for video_player_web will take longer than acceptable to unblock the flutter/plugins tree, so we're putting this mitigation so work can continue normally.
This patch will be removed, once the affected APIs can be accessed without triggering analyzer errors.
Related Issues

https://github.com/flutter/flutter/issues/52899
https://github.com/flutter/flutter/issues/41563

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[x] No, this is not a breaking change.

This will have to be merged on red submit-queue. The point of this change is to fix the analyze check, which is the one currently broken in the pipeline.

I'll merge as soon as the CI finishes. analyze is passing!

Merging on red, after verifying analyze step (and everything else) now passes. This should unblock the tree.


Feature request: compatibility with Verbb Events plugin

I'm using https://github.com/verbb/events for events and sessions. Each event can have many sessions. I'd love to be able to set up two-way linking between them.

Hello! Is this something you need to do on your end, or does Josh need to do something as well? I created an issue on the Events repo too.

This would be my responsibility now! Looks like there's lots of other element types to add support for, so I'll review that.


Streamfield issues IE11

Issue Summary
While editing a page in IE11 I'm unable to add blocks to a StreamFields. Clicking the '+' button only triggers focus outline for the block above
Steps to Reproduce

Set up fresh bakerydemo instance
Create a new blogpost instance
Add a block through the + button

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: 3.6.0 / 3.6.9 .
Django version: 2.2.9 / 3.0.4.
Wagtail version: 2.7.1 / 2.8`.
Browser version: IE11 (version: 11.379.17763.0)

I can reproduce this issue.
Here are my findings:
Using the select element tool I cannot hover the add button as the containing element behind it receives focus. This probably explains why clicking the button triggers the focus outline for the block.

When I change the pointer-events property on the .c-sf-add-button--visible CSS class from unset to visible the button works again and I can add streamfields. It appears IE11 doesn't recognise unset as a valid property value.

I am short on time to investigate this issue in more depth but hopefully this is useful.

I can second this as an issue.

I stumbled upon this as well and was about to create a new issue before I found this one. I wouldn’t be surprised if this was broken since the new StreamField UI was introduced in Wagtail 2.7, https://docs.wagtail.io/en/stable/releases/2.7.html#improved-streamfield-design.
It does look like the issue is with the elements’ stacking order as mentioned above, so might need to be resolved in the stylesheet for https://github.com/wagtail/react-streamfield, which is where those new styles are defined.
Tested in Wagtail 2.10 (current master) for me.

As a workaround it seems that interacting with that button with the keyboard works as expected, although unfortunately the button’s focus indicator isn’t visible so that’s not really something to rely upon.

This may well have been fixed in the telepath StreamField rewrite for 2.13 - while the CSS is largely unchanged, the HTML structure has been altered so that the '+' buttons are at the same level as the blocks, rather than each one being part of the block container above it. Would be good to have some feedback about whether this issue is still present...

Unfortunately it appears that the StreamField rewrite in 2.13 does not fix this.

Due to #6170 likely happening in 2.15 I have removed the good first issue label

Closing as we will no longer be supporting IE11 as of 2.15.


Create an upgrade doc for contour

It would be really handy to have a doc on how to upgrade contour. I've run into a bug that requires an upgrade.

Hi @silvacraig, thanks for creating the issue. It seems this is something which should be added to https://github.com/kinvolk/lokomotive/blob/master/docs/development/updating-components.md document, as we currently hardcode the used version of contour in lokoctl binary.
For the actual updates, I think we should pull https://github.com/helm/charts/tree/master/stable/contour chart and then perhaps override the images and tags to use the latest version.
We also have an issue to update contour to the new version, here: https://github.com/kinvolk/lokomotive/issues/875.

Thanks @invidian - could it be a simple matter of using the stable/contour helm chart to deploy the newer version after removing the one deployed by lokoctls?

As far as I see, we ship some improvements over the stable/contour chart, metrics, alerts and Grafana dashboard for example, perhaps some more, so I don't think it will work the same.


Support tinyint

Support tinyint

Pull Request Test Coverage Report for Build 999558117

9 of 9 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 90.201%

Totals

Change from base Build 985213145:
0.01%

Covered Lines:
5477

Relevant Lines:
6072

💛  - Coveralls

Thanks @sundy-li

Pull Request Test Coverage Report for Build 999558117
Details

9 of 9 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 90.201%

Totals

Change from base Build 985213145:
0.02%

Covered Lines:
5477

Relevant Lines:
6072

💛  - Coveralls


Support for mysql2

I have built a plugin that adds support for the node-mysql2 module. Its based heavily on the existing mysql plugin in this repository and only required a few minor changes.
Is this something that GoogleCloudPlatform would like to consume and maintain, or should I publish it separately as a third party plugin? I will be happy to submit a PR if the former.

We would be happy to see a PR opened for this! Given that the changes are minor, it doesn't sound like it would require much additional maintenance.

@kjin Sorry for the late response. As the changes I made to the original mysql plugin are so minor, I am wondering what the best approach here would be.
It feels a little dirty to create a whole new plugin that duplicates 98% of the mysql plugins code, however as plugins modules are declared with module.exports = [ ... ] its difficult to import shared code from other plugins (not that you would really want to do this anyway).
Would it be acceptable to create a mysql-shared.js (or some such) that both the mysql and mysql2 plugins can require or would you rather keep plugins completely independent?

That's a valid concern, esp. given that mysql and mysql2 have very closely resembling public APIs -- and under different circumstances a shared JS module makes the most sense. However, I'd still prefer to keep the plugins independent, because it's nice to have each module's corresponding monkeypatches summed up into a single file. It helps reduce the barrier to entry for other users that also need custom tracing behavior for a module.
That being said, this is a really compelling point that we will consider going into the next semver-major version so I'm glad you brought that up. If you have any ideas on how to neatly handle duplicated code while still making the path for users to create custom plugins, please let us know (we are actively thinking about this). Our web framework plugins, to an extent, already see a certain amount of duplication, so this isn't the only instance where code is potentially being duplicated.
The mysql plugin is currently about ~60 lines of actual code, so it wouldn't be very difficult to recuperate differences if we decide to go down a shared code route in the near future.

As a note regarding a PR -- as you've probably noticed we switched to using TypeScript for our codebase. However, the plugins have yet to get typed, so they are really still just vanilla JS. So feel free to work based on plugin-mysql.ts as if it were a JS file, and upload plugin-mysql2.ts also as if it were pure JS. The only difference is the superfluous export default {}; at the end of the file; this just prevents the TS compiler from treating the plugin as a script rather than a file.
Sorry about the complications -- we'd really appreciate a PR to patch mysql2!

I'm being pretty terrible at replying here. Sorry again.
I also agree it makes more sense to keep the plugins separate from one another. Having plugins depending on plugins just creates 'plugin hell'.
In regard to sharing code, I am not really sure of what the best approach would be. I can think of two possible ways, but I don't really like either of them.
Helpers
Provide a collection of generic helper functions, perhaps under an import namespace like @google-cloud/trace-agent/plugin-helpers. This might reduce the code duplication slightly, but you wouldn't really be able to share very much.
Templates
Adding extendable templates that plugin authors could use as a base and overwrite portions of would perhaps allow for easier sharing of logic. I'm not really a fan of templates though, or classes for that matter, but this might be the easiest to adopt by new authors.
Other than that I don't have any solid ideas. If I think of anything else I will add it here.
I will submit a PR later today for the mysql2 plugin 👍


False positive on String.Empty

I'm moving old project from hand-written logger to Serilog. As a temporary solution, some of old usages were replaced with such code:

Although, formally, analyzer is right (String.Empty is not a constant) and this is actually a bad practise, it is clear that this field will never contain anything other than empty string. Shouldn't String.Empty case be exceptional? (and, may be, separate code smell rule to warn about empty string template?)

@ArXen42 I've pushed 0.14 to nuget and visualstudiogallery which treats String.Empty the same as ""

@Suchiman Thanks!


HDDS-11705. Snapshot operations on linked buckets should work on actual underlying bucket

What changes were proposed in this pull request?
Snapshot operations on linked buckets create a snapshot on a linked bucket. However, there are potential downsides since the key prefixes in the FileTable/KeyTable/DirTable would correspond to the prefix of the actual bucket. In such a case even though a snapshot is created on the linked bucket but an SSTFilteringService run on the snapshot, the fileTable, dirTable, and keyTable would be empty since there would be no keys with the linked volume and linked bucket prefix. The same problem with other operations on the snapshot as well CreateSnapshot, DeleteSnapshot, RenameSnapshot, GetSnapshot, get key from snapshot, listing snapshots, listing snapDiff jobs, and submitting a snapshot diff job.
The solution is to resolve the bucket link to the actual bucket and perform all the snapshot operations on the actual bucket instead of the linked bucket
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-11705
How was this patch tested?
Unit tests & integration tests to follow

Overall changes look good to me. Let run a full range of snapshot operations on a linked bucket to be sure that we didn't miss anything.

Overall changes look good to me. Let run a full range of snapshot operations on a linked bucket to be sure that we didn't miss anything.

That's the plan. I have made TestOmSnapshot & TestOmSnapshotFileSystemParameterized and testing a full test run on it

@sumitagrawl we did consider multiple links to the same destination bucket impact. It doesn't seem like it can create any problem because all the snapshot operations are still happening on the destination bucket. Let us know If you see any specific problem in allowing this.

@sumitagrawl we did consider multiple links to the same destination bucket impact. It doesn't seem like it can create any problem because all the snapshot operations are still happening on the destination bucket. Let us know If you see any specific problem in allowing this.

As such no problem in implementation, may be caution to be added to avoid creation of multiple snapshots unknowingly, for optimization in snapshot creation.

Thanks @swamirishi for the patch, @prashantpogde @sumitagrawl @sadanand48, and @adoroszlai for the review.


Better Unix pattern matching

While working on adding a new linting engine to ale I noticed things were not working properly for some reason even though the program was spitting out the errors in standard unix format. It turns out that the spec doesn't seem to say anything about whitespaces therefore not all programs implement this properly.
To put it simply, we're only covering the "green" use case right now:
- file:line:column: body
+ file:line:column:body

This PR makes the whitespace optional so it doesn't break error message parsing.

I have fixed this now, with some tests.

@w0rp Great work! Will try to test it as soon as possible.


IE issues with svg sprite technique

Hello!
I came across this IE issue/bug and I'm wondering a) if you have find a workaround for this b) if this should be documented in the README.
Thoughts?

@stream7 hi! Never heard about this before. Maybe replace whole <svg> instead of <use>?

@kisenka hey! How can you do that though? You don't have such control in React, after the VDOM reconciliation it will just update the xlinkHref attribute since that's the only thing that has changed.
The workaround I'm using now is to render both icons and use a css class to show only the "active" one.

@stream7 I am not good in React, but can not you force component rerender? If not, maybe use SVG as background image in this case?

but can not you force component rerender

I believe you can force a virtual DOM rerender but not a real one which won't help in this case.
Sure, you can use other techniques, I was just wondering if there is a workaround so we could stick to the sprite and the Icon component.
Anyways, thanks for the response. I'll let you know if I find a workaround.


Error while requiring module

Environment

Describe the bug
After install all things going is perfect, but when I'm enable the module in nuxt configuration and provide the URL to  GraphQL API, like .env file then run "npm run dev" I have this problem, what can i do .. PLEASE HELP
Expected behaviour
NO
Reproduction
No response
Additional context
No response
Logs
No response

I have the same issue


Dot plot example?

I've been struggling to figure out how to make a scanpy-style dot plot with Marsilea, though it seems like this should be possible.
Could an example of this kind of plot be added to the documentation?

Thank you for using the Marsilea. I've included an example below for you to refer to. Feel free to ask or open a new issue if anything is unclear.
The current documentation is a bit unclear. I will try to add more content and make it easier for new users in the next few months.
import numpy as np
import marsilea as ma
import marsilea.plotter as mp

# Synthetic data
color = np.random.randn(10, 10)
size = np.random.randn(10, 10)
genes = [f"gene_{i+1}" for i in range(10)]
cells = [f"cell_{i+1}" for i in range(10)]
chunks = ["T cell", "B cell", "myeloid"]
labels = np.random.choice(chunks, 10)

# Do the plotting
dot = ma.SizedHeatmap(size=size, color=color, cluster_data=size, cmap="Reds",
                      size_legend_kws=dict(title="Size Title"),
                      color_legend_kws=dict(title="Color Title")
                     )
# Add labels
dot.add_bottom(mp.Labels(genes), pad=.1)
dot.add_left(mp.Labels(cells), pad=.1)
# Add dendrogram
dot.add_dendrogram("right", method="ward")
# Add group labels
dot.vsplit(labels=labels, order=chunks)
dot.add_top(mp.Chunk(chunks, ["#90D26D", "#912BBC", "#496989"]))
# Add legends
dot.add_legends()
dot.render()


Make it easier to include as a submodule

Update lib to kicad7
Move footprints into a footprint library
(will duplicate and make variants)

Hi, did you ever finalize this? (It's still listed as a draft pull request.)

Oh yeah! Sorry about that! I tottally forgot to mark it as ready, we're using it in this project rn https://github.com/FRC-1721/RoomBouy
I don't know if i edited the readme or not, most of my PRs don't get merged! I'll check in a sec

Yep! Readme was updated, if you like the changes, this PR is good to go :+1:

Since you're using this, I'm going to accept the PR without checking. I haven't looked at this repo in years, so you probably understand it better than me.
Thanks for modernizing this!

Yeah ofc! Thanks for originally creating this :) I'll update the credits in my downstream projects :+1:


Map mouse to analog inputs (race wheel steering)

Hi, I'm trying to recreate this Daytona USA mouse steering setup in linux. http://forum.arcadecontrols.com/index.php/topic,115396.msg1224044.html#msg1224044
Does MoltenGamepad support mapping mouse movements to a virtual analog joystick? I'm not seeing much documentation on it, and unsure how linux handles a second mouse.
Thank you

There is not currently such support, which I guess would be a "rel2abs" event translator.
We'd need code to map relative mouse movements onto the absolute position reported by the virtual joystick.
(Pull requests welcome, can describe this more)