Blur over different text when hovering over FlatButton in Flutter Web

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
Edit lib/main.dart and change the Scaffold's Center element to:

Center(
        child: Padding(
          padding: EdgeInsets.symmetric(
            horizontal: 50,
          ),
          child: ConstrainedBox(
            constraints: BoxConstraints(
              maxWidth: 500,
            ),
            child: ListView(
              children: [
                SizedBox(
                  height: 50,
                ),
                ConstrainedBox(
                  constraints: BoxConstraints(minWidth: 500),
                  child: Card(
                    child: Padding(
                      padding: EdgeInsets.all(50),
                      child: Column(
                        children: [
                          Text(
                            'Log out',
                            style: TextStyle(fontSize: 30),
                          ),
                          SizedBox(
                            height: 35,
                          ),
                          Column(
                            children: [
                              FlatButton(
                                child: Padding(
                                  padding: EdgeInsets.all(10),
                                  child: Text(
                                    'Leave',
                                    style: TextStyle(
                                      fontSize: 17,
                                    ),
                                  ),
                                ),
                                onPressed: () {},
                              ),
                            ],
                          ),
                        ],
                      ),
                    ),
                  ),
                ),
                SizedBox(
                  height: 50,
                ),
              ],
            ),
          ),
        ),
      ),

Expected results:
Just the FlatButton to change
Actual results:
Unrelated text, the "Log out," was blurred

Logs

[ +118 ms] executing: [C:\flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +74 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[   +3 ms] 8b3760638a189741cd9ca881aa2dd237c1df1be5
[   +1 ms] executing: [C:\flutter/] git tag --points-at HEAD
[  +54 ms] Exit code 0 from: git tag --points-at HEAD
[        ] 1.22.0-12.1.pre
[   +9 ms] executing: [C:\flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +77 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[  +10 ms] origin/beta
[  +28 ms] executing: [C:\flutter/] git ls-remote --get-url origin
[  +55 ms] Exit code 0 from: git ls-remote --get-url origin
[  +14 ms] https://github.com/flutter/flutter.git
[ +304 ms] Unable to locate an Android SDK.
[   +4 ms] executing: [C:\flutter/] git rev-parse --abbrev-ref HEAD
[  +68 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] beta
[  +69 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[        ] 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.
[  +31 ms] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +3 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[  +36 ms] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[  +61 ms] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[  +40 ms] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[  +40 ms] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[  +44 ms] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[   +9 ms] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +63 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[  +29 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[  +41 ms] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[  +41 ms] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[   +6 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[  +41 ms] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[  +39 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.
[  +78 ms] Multiple devices found:
[  +65 ms] Web Server (web) • web-server • web-javascript • Flutter Tools
[  +15 ms] Chrome (web)     • chrome     • web-javascript • Google Chrome 85.0.4183.121
[  +35 ms] Edge (web)       • edge       • web-javascript • Microsoft Edge 85.0.564.51
[  +37 ms] [0]: Web Server (web-server)
[  +42 ms] [1]: Chrome (chrome)
[        ] [2]: Edge (edge)
[   +2 ms] Please choose one (To quit, press "q/Q")
[  +35 ms] :
[+3093 ms] 1
[ +153 ms] Generating C:\Users\Gabriel Pizarro\OneDrive\Code\Flutter bugs\FlatButton blur\bug\android\app\src\main\java\io\flutter\plugins\GeneratedPluginRegistrant.java
[  +75 ms] Launching lib\main.dart on Chrome in debug mode...
[  +66 ms] Updating assets
[  +82 ms] Syncing files to device Chrome...
[  +44 ms] Generating C:\Users\Gabriel Pizarro\OneDrive\Code\Flutter bugs\FlatButton blur\bug\android\app\src\main\java\io\flutter\plugins\GeneratedPluginRegistrant.java
[  +51 ms] <- reset
[   +7 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 -Ddart.developer.causal_async_stacks=true --output-dill C:\Users\GABRIE~1\AppData\Local\Temp\flutter_tools.c76e457b\flutter_tool.9ba4dfdb\app.dill --libraries-spec file:///C:/flutter/bin/cache/flutter_web_sdk/libraries.json
--packages .packages -Ddart.vm.profile=false -Ddart.vm.product=false --bytecode-options=source-positions,local-var-info,debugger-stops,instance-field-initializers,keep-unreachable-code,avoid-closure-call-instructions --enable-asserts --track-widget-creation
--filesystem-root C:\Users\GABRIE~1\AppData\Local\Temp\flutter_tools.c76e457b\flutter_tools.56c9184f --filesystem-scheme org-dartlang-app --initialize-from-dill build\cache.dill.track.dill --platform
file:///C:/flutter/bin/cache/flutter_web_sdk/kernel/flutter_ddc_sdk.dill
[  +31 ms] <- compile org-dartlang-app:/web_entrypoint.dart
[+14678 ms] Syncing files to device Chrome... (completed in 14,803ms, longer than expected)
[   +7 ms] Synced 26.9MB.
[  +59 ms] <- accept
[   +1 ms] Caching compiled dill
[ +416 ms] [CHROME]:
[   +1 ms] [CHROME]:DevTools listening on ws://<IP_ADDRESS>:50958/devtools/browser/f68d4d0d-aba3-4820-a67a-173e278654f1
[+5579 ms] Debug service listening on ws://<IP_ADDRESS>:50977/YuNjXBNhYEk=

[ +228 ms] Debug service listening on ws://<IP_ADDRESS>:50977/YuNjXBNhYEk=
[   +2 ms] Warning: Flutter's support for web development is not stable yet and hasn't
[  +27 ms] been thoroughly tested in production environments.
[        ] For more information see https://flutter.dev/web
[        ]   To hot restart changes while running, press "r" or "R".
[        ] For a more detailed help message, press "h". To quit, press "q".
[+9665 ms] Stopped debug service on ws://<IP_ADDRESS>:50977

[  +42 ms] Application finished.
[  +13 ms] "flutter run" took 35,199ms.
[ +125 ms] ensureAnalyticsSent: 90ms
[   +1 ms] Running shutdown hooks
[        ] Shutdown hook priority 4
[ +152 ms] Shutdown hooks complete
[   +1 ms] exiting with code 0

Analyzing bug...

   info - The value of the field '_counter' isn't used - lib\main.dart:53:7 - unused_field

1 issue found. (ran in 1.4s)

Downloading android-arm-profile/windows-x64 tools...                0.6s
Downloading android-arm-release/windows-x64 tools...                0.3s
Downloading android-arm64-profile/windows-x64 tools...              0.4s
Downloading android-arm64-release/windows-x64 tools...              0.3s
Downloading android-x64-profile/windows-x64 tools...                0.4s
Downloading android-x64-release/windows-x64 tools...                0.3s
[√] Flutter (Channel beta, 1.22.0-12.1.pre, on Microsoft Windows [Version 10.0.19042.508], locale en-US)
    • Flutter version 1.22.0-12.1.pre at C:\flutter
    • Framework revision 8b3760638a (9 days ago), 2020-09-15 17:47:13 -0700
    • Engine revision 4654fc6cf6
    • Dart version 2.10.0 (build 2.10.0-110.3.beta)

[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, set ANDROID_SDK_ROOT to that location.
      You may also want to add it to your PATH environment variable.

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

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[√] VS Code (version 1.49.1)
    • VS Code at C:\Users\Gabriel Pizarro\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.14.1

[√] Connected device (3 available)
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 85.0.4183.121
    • Edge (web)       • edge       • web-javascript • Microsoft Edge 85.0.564.51

! Doctor found issues in 2 categories.

Hi @TheNewJavaman,
I tried your code sample but didn't see the Log out text getting blur upon hovering over FlatButton.
66600.mov.zip

flutter doctor -v
    en-IN)
    • Flutter version 1.22.0-12.1.pre at /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 8b3760638a (9 days ago), 2020-09-15 17:47:13 -0700
    • Engine revision 4654fc6cf6
    • Dart version 2.10.0 (build 2.10.0-110.3.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.3

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

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (3 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    •
      Android 10 (API 29) (emulator)
    • Web Server (web)                   • web-server    • web-javascript •
      Flutter Tools
    • Chrome (web)                       • chrome        • web-javascript •
      Google Chrome 85.0.4183.121

• No issues found!

Can you share video of the issue you are facing.
Thanks.

Thank you for responding @darshankawar
I changed the "Log out" Text to:
Text(
  'Here is a bunch of small text so it\s obvious',
  style: TextStyle(fontSize: 14),
),

And recorded a video. Hopefully it looks clear.
Google Drive Link
One other thing to note is that Chrome has to be in windowed mode (not fullscreen or maximized) for this bug to appear.

Hi @TheNewJavaman
Tried again but didn't notice the blur on the text :-(
66000.mov.zip
I am on Mac and you seem to be on Windows, so not sure if that's causing the issue.
Also see a related but not similar issue, wherein I wasn't able to replicate the issue.
And yours could be similar to this.

I believe this issue is a duplicate of #58159, based on this comment. I will close this issue based on the information provided in the other thread.

I have this problem too.
I use Windows with scaling more than 100%
If I hover over a button, no matter what kind of, the button changes his color and the text get's unsharp!


Document support for IPv6

Adds a line in the CHANGELOG.md so that next release can list also this change


Enable parallel builds of OpenSSL on Unix

Share Cargo's jobserver with make so they can all build in parallel from the
same job pool.

@sfackler I think this was fixed in 1.1.0, right?

Yep, parallel builds work in 1.1.0.


To add working steps in Manhattan Distance Calculator

Is your feature request related to a problem? Please describe.
To add working steps in Manhattan Distance Calculator

Go ahead
Deadline 22 May,2021


✨(backend) bind credit card info to order admin serializer

Purpose
Now a credit card is linked to an order, we want to display that in our back
office application so we bind this data the AdminOrderSerializer.

Proposal

[x] Update admin api to bind credit card detail into order response
[x] Display credit card detail in order detail view
[x] Fix / Write tests

@kernicPanel and @jonathanreveille I added 2 fixup commits to this. I will handle your feedback if you have.


feat: WIP Input component

This is WIP to review today as we plan out the rest of the form components.

I think we're going to want to allow for an autocomplete prop so we can disable it for some of the sign up fields.
https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/autocomplete

 Input Component

@nathanyoung can i get access to the trello board? I can't view the Input Component ticket
have we considered styled-components for our css? that'll give us scoping for css classes so we dont have generic classes like .input

@JCarmine so hooks definitely need to called within the parent function component. It's more the subcomponents (renderLabel and renderInput) in this case that can go outside the main parent component. To me, 1 file represents 1 component, so its ok if a file has 1 main component and several sub components defined at the top level (as long as the main component is always at the top or bottom of the file) (i prefer top)

On the global/scoped CSS question. Global styles in the traditional sense don't make any sense since the components will be exported with only the styles contained within them.
I've used CSS Modules in the past and the solution for setting something like box-sizing globally would involve using sass, composes, a css var, or just remembering to set it manually in every component.
The solution could be to somehow build a template generator, that when invoked with a component name, generates a folder that contains boilerplate Component, storybook, sass files, etc. This way, we could ensure that the box-sizing rule is included in the boilerplate by default.


HACKING.md contains broken links to http://hastebin.com

The HACKING.md documentation contains two broken links to http://hastebin.com : http://hastebin.com/fajomitewu.yaml and http://hastebin.com/gihahuguvu.http.
Those links should be either fixed or removed.

Those links were added by https://github.com/endoflife-date/endoflife.date/commit/fa5e84cb43c713313791871536ca62c01cb8f6ae. @captn3m0, any idea what to do with those links ?

Will change to a gist.

@marcwrobel moving this entry to #1206


As a developer, I'd like to update all object comparisons to use object.sObjectType

Per #1323

Included in beta release version 3.35 (Beta 1) and higher

Included in production release version 3.35


empirically include the distro on rhe isotope vms names

no, instead work the hostname unless the name is defined

no


bug: Page content overflows on some viewports

ex: /reference/join-map-lanes.html the overflow is caused by large code blocks

Fixed by using max-w-prose on the content


Center property on Google map component is not changing when props was received

Center property on Google map component is not changing when props was received.
Please I need help its urgent

It will only change if you change it. Please show the code that you are using.

Try to use fitBounds or setCenter on componentDidUpdate to set new center.


[DO NOT MERGE] Rafter integration tests in Kyma

Description
Changes proposed in this pull request:

Rafter integration tests in Kyma, temporarily here to check them on pipelines

Related issue(s)

See also https://github.com/kyma-project/kyma/issues/6509

No longer relevant.


How about IDN domain implementation?

I want create IDN domain implementation for ru_RU locale. May be anybody already created that?

No, it doesn't exist yet (just look at the source, Luke). Feel free to create one. Don't forget unit tests.


HedgeArcPack - Broken Lost World PAC Generation

While HedgeArcPack in its current forms appears to work fine with most games, generating PAC files using the Lost World option causes the generation of broken PAC files. It can currently unpack files from the source games fine, but trying to use PAC files generated by the latest version of HedgeArcPack causes the game to refuse to load them. For instance, unpacking and repacking Sonic's archive causes his model to not load on the world map and crashing the game when loading into a zone. HedgeArcPack is fully able to extract these broken PACs too.
Attached are two zip files for reference. One contains the original Sonic PAC files from Lost World, the other contains PAC files with the same contents but repacked via the latest HedgeArcPack.
Sonic - Ripped from sonic2013_patch_0.cpk.zip
Sonic - Generated via Latest HedgeArcPack.zip

Thanks very much for reporting this, and especially for providing samples! I'm currently finishing up a rewrite of the packfile handling code in HedgeLib, which should also fix this issue. I'll look into this and try to discover its root cause to make sure it's fixed.


Fix incorrect comment wording

This better not end up being another s...

As an administrator, you may still merge this pull request.

muahaha


Response headers missing from logs when making use of default serializers

Description
When making use of default serializers as below, the response headers in log simply appears as "headers": "[Function]" rather than the expected list of headers.
Express example with logging middleware implementation:
app.use(function (req, res, next) {
  logger.info({ req, res });
  next();
})

It looks like the default seek-oss/logger serializer tries to extract res.headers which is a function in the res object.
Steps to reproduce (for bugs)
See description.
Possible Implementation
We could extract the headers with res.getHeaders(). See here: https://nodejs.org/api/http.html#http_response_getheaders
Context
Use case: we are trying to log all incoming HTTP requests.

Hey @arthurw1! A couple of generalities here:

We don't recommend logging every HTTP request in application code due to cost implications. If you're deploying to a SEEK-standard runtime environment then you likely have a sidecar that already handles generic request logging.

We don't recommend blasé logging as described in the example. Those objects are internal to Express and could conceivably contain sensitive (PII or security-related) information. These can be contained in request or response bodies, or propagated between middlewares via e.g. locals.

Happy to discuss this further internally if you'd prefer. You can reach out in our shared channels like #apna-jag.

Hey @arthurw1! A couple of generalities here:

We don't recommend logging every HTTP request in application code due to cost implications. If you're deploying to a SEEK-standard runtime environment then you likely have a sidecar that already handles generic request logging.

We don't recommend blasé logging as described in the example. Those objects are internal to Express and could conceivably contain sensitive (PII or security-related) information. These can be contained in request or response bodies, or propagated between middlewares via e.g. locals.

Happy to discuss this further internally if you'd prefer. You can reach out in our shared channels like #apna-jag.

@72636c Thanks for your input! I'll speak to my team about it.
But putting the use case aside, the code here doesn't seem to work as intended, i.e. the following are undefined (for a bare-bones nodejs http server):

res._headers
res.header
res.headers

However, res._header does provide you with header info as a string.

@72636c Thanks for your input! I'll speak to my team about it.
But putting the use case aside, the code here doesn't seem to work as intended, i.e. the following are undefined (for a bare-bones nodejs http server):

res._headers
res.header
res.headers

However, res._header does provide you with header info as a string.

👍 thanks for looking into this. Would you be open to submitting the fix?

👍 thanks for looking into this. Would you be open to submitting the fix?

Yup, I can do that!

Yup, I can do that!


How to get to the UI

I am playing with lagoon testing a few things... got the development Environment running and started looking around.. taking notes of my pain points on the way to later help with documentation...
I figured out how to get to the graphql api... and get auth...
from the code I see there should be the UI I saw in videos...
but going to.. http://localhost:3000/ gives me
{"errors":[{"message":"Unauthorized - Bearer Token Required"}]}
when I provide that with a browser extension I still only get a json ok
Questions:

Is there a login url
do I need to have the customer first project stuff done first
how do I get to see the UI

Thanks for short answers...

You can access the ui at http://localhost:8888/ which will redirect you to keycloak to log in, you can log in with lagoonadmin:lagoonadmin
You do need to either change the ui service in docker-compose.yaml from 8888:3003 to 8888:3000 or add an env var for setting the ui to dev mode and restart the ui service
    environment:
      - NODE_ENV=dev


openblas on windows 10 build fail

The following packages will be built and installed:

clapack[core]:x86-windows
dlib[core]:x86-windows
fftw3[core]:x86-windows
libjpeg-turbo[core]:x86-windows
openblas[core]:x86-windows
Additional packages (*) will be modified to complete this operation.
Starting package 1/5: openblas:x86-windows
Building package openblas[core]:x86-windows...
CMake Error at ports/openblas/portfile.cmake:16 (message):
openblas can only be built for x64 currently
Call Stack (most recent call first):
scripts/ports.cmake:71 (include)

Error: Building package openblas:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: openblas:x86-windows
Vcpkg version: 2018.11.23-nohash
Additionally, attach any relevant sections from the log files above.
I have the latest vcpkg portfiles and latest upgrades.

nevermind i didnt even read the error. can only be installed on x64.


Add draw samples example to gcm introduction notebook

Hey @bloebp, in terms of keeping the introduction streamlined, or focussed on the goal of answering causal queries, is this addition ideal? I think this actually helps for a deeper understanding of the library or our methods, so I'm wondering if there is a better place to show this. What do you think?


Hotfixes - url rewrites, UI fixes, increase code quality coverage

one more thing: please consider writing full PHPDoc blocks when your write new code

one more thing: please consider writing full PHPDoc blocks when you write new code: properties, methods, maybe classes too

definitely aiming to for new and go through and add to all. I have seen the light, thanks to you :)


Prevent running the crash_handler when a debugger is present on windows

Fixes #11347.
Also renamed the "debug/settings/backtrace/message" to "debug/settings/crash_handler/message" for consistency.

LGTM, assuming 'IsDebuggerPresent()' does what it says on the tin :D

I checked it with VSCode and it returned true so it seems to work as expected :).


so let me get this straight the only possible way to use eos is if you can download cleos .. and cml -> and the only way you can get cleos is if your not on a window...

what a joke how ever single thing i want to code around is perfectly designed to not work for me ....
how is windows not a supported system if you dont have any type of easy way to generate a wallet ... this currency will never thrive

Anchor wallet works fine on Windows.   If you need further assistance, contact EOS Support https://eossupport.io/


Fix SkyRD::Sky::free() freeing shared material

Fixes #95645
When a scene that has a WorldEnvironment that contains a Sky resource gets closed, SkyRD::Sky::free() gets called, which currently frees the material used by that Sky. This is a problem when multiple Skies use the same material.
This PR aims to fix that by removing the instruction which frees the material, since it already gets freed when Sky gets unreferenced.

Thanks! And congrats for your first merged Godot contribution :tada:

Cherry-picked for 4.3.1.


Improved thread management

Right now Cozy spawns one thread for every method on the data structure (that includes one thread for each new method it adds---which can be quite a few).
Because of this behavior, Cozy's ability to handle large specifications is more limited by the number of cores on your computer than by your willingness to wait. I would prefer a balance: Cozy uses all available cores, but not more, at the cost of a longer run on machines with fewer cores.
My proposal is:

[ ] change the semantics of the --timeout flag in the documentation to be "timeout per method" rather than global timeout
[ ] add a flag to control the number of threads (default is the number of cores on your machine)
[ ] in high_level_interface.py, do not start new synthesis jobs until an idle core is available
[ ] in high_level_interface.py, run each job until it has run for --timeout, rather than treating the timeout as a global timeout

just to comment, when I was trying out a few examples in repo, the utilization of cores is not very good (by good, I mean it can't fully utilize all cores I have, despite the workload. I expect this though).

Do you mean that Cozy does not spawn enough threads to use all available cores? This can happen if your specification does not contain very many methods.
Each thread synthesizes an implementation for a single query operation. Exploiting idle cores by parallelizing that task would be very difficult, and likely even worthy of a workshop paper for the synthesis community. If you have any good ideas, let us know!
In the meantime, I don't think finding more parallelism is a high priority. My experience has been that real-world specifications have quite a few methods.

Do you mean that Cozy does not spawn enough threads to use all available cores? This can happen if your specification does not contain very many methods.

Yes, but I do agree that parallelism is not a high-priority.


Defect/fix select app

Bugfix - we were storing the full app object in heroku.app.name, causing downstream errors interacting with the app.

@sfryffdc @mwoodffdc Ready for review


results are quite different from Cli98

Hi.
Now I want to get anchors of my own datasets.
Result from https://github.com/Cli98/anchor_computation_tool is:
anchors_scales = [8.3, 3.3, 18.0],
anchors_ratios = [(1, 1.02), (1, 0.92), (1, 0.97)]
Result from yours:
anchors_ratios =  [(0.7, 1.4), (1.0, 1.0), (1.3, 0.8)]
It seem quite different!
I'm really confused, and could you give an explain?
I also want to know how to get anchors_scales in your code?
Thanks a lot!

Hi @edwardning, in the tutorial of the repo that you linked as I commented here the bounding boxes aren't normalized before running K-Means and I think this is a very bad idea given how K-Means with IoU works.
From the issue linked above:

Immagine this situation:
10 bboxes 10x5
5 bboxes 2x1
5 bboxes 1x2
If K=2 the two clusters would be:
10 bboxes 10x5
and
5 bboxes 2x1
5 bboxes 1x2
and so the suggested ratios would be (1.4, 0.7) and (1., 1.) instead of (1.4, 0.7) and (0.7, 1.4) (or (1., 0.5), (1., 2.)).

As you can see from this example if you don't normalize, K-Means will be driven mainly by the bounding boxes sizes and your ratios will not make much sense.
Evan if you want to search for what the original TensorFlow repo calls the anchor_scale in the same K-Means run, you should consider that in EfficientDet (if you are using EfficientDet like I'm assuming) your sizes are determined by three factor: num_scales, anchor_scale and the layers.
The number of anchors generated is: num_ratios * num_scales * 5 (layers in the pyramid).
The repo that you linked doesn't consider this in the tutorial.
Also, you should use anchor ratios with a area (approximately) of 1, the normalization done in the tutorial of the repo that you linked is not correct considering how anchor ratios are used by the EfficientDet implementations.
Also, you should consider the input_size and adapt your bounding boxes to it before running K-Means to search for optimal scales.
See also this issue if you have the problem that in your dataset some bounding boxes are very small, if you haven't this problem probably the default sizes are already very good.
You can test how good one combination of sizes and ratios are using get_annotations_without_similar_anchors, see the tutorial of this repo.
Note that this list of sizes:
[32.0,
 35.91878554589994,
 40.31747359663594,
 64.0,
 71.83757109179987,
 80.63494719327188,
 128.0,
<PHONE_NUMBER>8359975,
<PHONE_NUMBER>8654377,
 256.0,
<PHONE_NUMBER>671995,
<PHONE_NUMBER>7308754,
 512.0,
<PHONE_NUMBER>34399,
<PHONE_NUMBER>461751]

Are what the official repo generated when you pass anchor_scale = 4 (the default value), you can easily play with sizes adapting this list based on your anchor_scale.


Feature/monitor changes

Additional support for #15 allowing users to monitor changes to both the push token and the activity status through a stream.

I tested and it works. Here's how I'm listening:
_liveActivitiesPlugin.activityUpdateStream.listen((activity) {
        print(activity);
      });

Here are some of the updates I got:

_ActiveActivityUpdate
_EndedActivityUpdate

unrelated to this PR but are you seeing this message in Flutter when sending push notifications?

Unable to decode content state: The data couldn’t be read because it is missing.

Published v1.6.0, thanks guys 👍
(@arnar-steinthors you just forget to update README.md, I updated for you, is it OK ?)

@istornz looks like you left some content regarding "Picture removed". I'd just say it returns status updates for new push tokens or when the activity ends.
I imagine once @arnar-steinthors or myself finish our projects we'll contribute examples to the docs of how to use push tokens and lifecycle events.

I tested and it works. Here's how I'm listening:
_liveActivitiesPlugin.activityUpdateStream.listen((activity) {
        print(activity);
      });

Here are some of the updates I got:

_ActiveActivityUpdate
_EndedActivityUpdate

Awesome, I will test this on production on monday/tuesday when the backend is ready to see how it goes when the app has been in the background for some time.

Published v1.6.0, thanks guys 👍 (@arnar-steinthors you just forget to update README.md, I updated for you, is it OK ?)

Sorry I forgot about that 😶🌫 thanks for taking care of it 👏
Only have the same comment that @jolamar left about the return value for the new docs
Stream<ActivityUpdate> Picture removed

unrelated to this PR but are you seeing this message in Flutter when sending push notifications?

Unable to decode content state: The data couldn’t be read because it is missing.

@jolamar  Yes I am receiving it, I think its because of the original ContentState thats inside the source code but everything is still working when I am testing locally, will have to see if it still works when I end to end test this on monday/tuesday

@arnar-steinthors @jolamar Oops my bad, fixed it. Thanks !

@jolamar I also have some thoughts on the .getPushToken() method you added.

If i am running my app on a simulator and call the method it will hang forever, probably because there are no updates to the token in the native code.
If I am running on a physical device the method will return a token but if I call the method consecutively it will hang on the second call, probably waiting for an update.

pseudocode example
final plugin = LiveActivities();
var token = await plugin.getPushToken(); // a token is returned
token = await plugin.getPushToken(); // the method will hang, never returning a value

What do you think about either removing this from the api since we now have the stream or changing the native implementation to use
let token = activity.pushToken;
result(token) // returns a nullable string

instead of what we currently have
for await data in activity.pushTokenUpdates {
            let pushToken = data.map {String(format: "%02x", $0)}.joined()
            result(pushToken) // returns a token when there is an update
          }

Then it would (hopefully) just return null on simulators and an identical token if you chain the calls

@arnar-steinthors that is also happening to me.. I assume that's just a weird side effect of using pushTokenUpdates maybe it gets it initially then immediately updates it? It doesn't really bug me that much :)
So I got everything working and I'm responding to dismisses and token updates. My implementation looks like this.. I'd like to hear your thoughts.
WidgetsBinding.instance.addPostFrameCallback((_) {
        _liveActivitiesPlugin.activityUpdateStream.listen((activity) {
          LiveActivityState state;
          String pushToken;
          activity.map(
            active: (newActivity) {
              state = LiveActivityState.active;
              pushToken = newActivity.activityToken;
            },
            ended: (_) => state = LiveActivityState.ended,
            unknown: (_) => state = LiveActivityState.unknown,
          );

          switch (state) {
            case LiveActivityState.active:
              context.read<MyStore>().setLiveActivityPushToken(pushToken, activity.activityId);
              break;
            case LiveActivityState.ended:
              context.read<MyStore>().revokeLiveActivityPushToken(activity.activityId);
              break;
            case LiveActivityState.unknown:
              break;
          }
        });
      });

I tried using _liveActivitiesPlugin.getActivityState(activityId) in the listener but I don't think the state had been set yet so I wasn't getting end state updates, so I used the map instead.

@arnar-steinthors I like your pushToken suggestion. I also get the hung behavior in simulator so this would be a great fix!

@arnar-steinthors I spent a lot of time yesterday trying to manage state for multiple live activities and hit a wall. Wondered if you have address this? Basically sharedDefaults isn't unique across live activities so all of them share the same data. I tried adding an ID to the sharedDefault key but that caused live activities to not appear at all. Not sure where to go from here.

@arnar-steinthors that is also happening to me.. I assume that's just a weird side effect of using pushTokenUpdates maybe it gets it initially then immediately updates it? It doesn't really bug me that much :)
So I got everything working and I'm responding to dismisses and token updates. My implementation looks like this.. I'd like to hear your thoughts.
WidgetsBinding.instance.addPostFrameCallback((_) {
        _liveActivitiesPlugin.activityUpdateStream.listen((activity) {
          LiveActivityState state;
          String pushToken;
          activity.map(
            active: (newActivity) {
              state = LiveActivityState.active;
              pushToken = newActivity.activityToken;
            },
            ended: (_) => state = LiveActivityState.ended,
            unknown: (_) => state = LiveActivityState.unknown,
          );

          switch (state) {
            case LiveActivityState.active:
              context.read<MyStore>().setLiveActivityPushToken(pushToken, activity.activityId);
              break;
            case LiveActivityState.ended:
              context.read<MyStore>().revokeLiveActivityPushToken(activity.activityId);
              break;
            case LiveActivityState.unknown:
              break;
          }
        });
      });

I tried using _liveActivitiesPlugin.getActivityState(activityId) in the listener but I don't think the state had been set yet so I wasn't getting end state updates, so I used the map instead.

Sorry for the late reply but yeah it looks good as long as this Widget that you are using this method in will live as long as the app does (i.e is never disposed unless the app is killed), otherwise you never stop listening to the stream and could end up with identical listeners.
You could also simplify this
WidgetsBinding.instance.addPostFrameCallback((_) {
        _liveActivitiesPlugin.activityUpdateStream.listen((activity) {
          activity.map(
            active: (newActivity) {
              context.read<MyStore>().setLiveActivityPushToken(newActivity.activityToken, activity.activityId);
            },
            ended: (_) =>  context.read<MyStore>().revokeLiveActivityPushToken(activity.activityId),
            unknown: (_) {},
          );
      });

@arnar-steinthors I spent a lot of time yesterday trying to manage state for multiple live activities and hit a wall. Wondered if you have address this? Basically sharedDefaults isn't unique across live activities so all of them share the same data. I tried adding an ID to the sharedDefault key but that caused live activities to not appear at all. Not sure where to go from here.
I tried adding a unique id to the key again today and it worked so I created a PR if you want to see #22

No I havent thought about this issue, my implementation prohibits using multiple activities at the same time since it doesnt fit my usecase.
I took a quick look at the PR and it looks a bit hacky but I guess we cant get around it since we are quite limited. I am wondering though if we need the extra attribute var sharedId: String, could we not use var id = UUID() for the same purpose?

@arnar-steinthors it's totally hacky. We need to pass in a unique ID at time of creation so that we can give the sharedDefault keys an ID. var id = UUID() doesn't exist until after the live activity is created. I bet others will want to supports multiple live activities, in my use case I want to be able to track multiple live sporting events.

@jolamar  I need help creating a live activity that does not appear, please


[WIP] spotify: <IP_ADDRESS>.gf6bc1b6b-12 -> <IP_ADDRESS>.gf614956d-16

Motivation for this change
Currently the updated spotify will segfault once you try to play anything. I'm opening this PR so that others know why spotify is outdated and do not try to update it themselves. Upstream links:
https://community.spotify.com/t5/Desktop-Linux/spotify-client-1-1-0-98-78-gb45d2a6b-10-from-Ubuntu-package/td-p/4675676
https://community.spotify.com/t5/Desktop-Linux/1-0-98-1-1-0-Crash-when-using-ALSA/td-p/4680398
Unfortunately spotify on linux is still not officially supported, so who knows when there will be a fix.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

1.1.5 is out (I feel like they use dice to determine their version numbers). Unfortunately it doesn't fix the issue. There is a workaround using pulseaudio emulation for alsa which we may want to adapt if this remains unfixed much longer.

I can't believe this is still broken :cry:

Maybe we could use the same workaround as Gentoo? https://gitweb.gentoo.org/repo/gentoo.git/commit/media-sound/spotify?id=d8c8d8abd012c709551cc96802d2615fa69b5212

Probably! I've seen that too, but since I don't particularly miss any new features in the spotify client (its pretty bad, but I doubt it'll get better in the ways I want it to get better) I wasn't motivated enough to try it.
If it works, feel free to do it :) Since you seem pretty interested in the spotify package, maybe even add yourself as a maintainer (no obligation to do so of course).

With apulse in LD_LIBRARY_PATH, Spotify does not crash anymore when playing music, but the song stays at 0:00 and these errors appears:
1564478349.003 [apulse 10368] Z pa_stream_set_underflow_callback
1564478349.003 [apulse 10368] Z pa_stream_set_started_callback
[apulse] [error] do_connect_pcm: can't get initial hw parameters for playback device "default". Error code 1 (Operation not permitted)
[apulse] [error] do_connect_pcm: failed to open ALSA device. Apulse does no resampling or format conversion, leaving that task to ALSA plugins. Ensure that selected device is capable of playing a particular sample format at a particular rate. They have to be supported by either hardware directly, or by "plug" and "dmix" ALSA plugins which will perform required conversions on CPU.

I don't know a thing about ALSA/PulseAudio or sound servers in general, so I admit I have no idea what I'm doing :slightly_smiling_face:

I got it working by exporting APULSE_PLAYBACK_DEVICE=plug:dmix before running spotify.

I created a new PR since the version is a newer version available.


Fix: issue #877 Typing fast in firefox skips the second character

Changing the focus on elements does not always go fast enough in firefox to reliably catch all keystrokes. This fix changes the order of events, and keeps catching keystrokes after the first until the focus has shifted.

Rebase this on top of current master.

I still experience this issue. Is is already fixed in the master code?

Pull request is rebased... ready for review...

 Also there shouldn't be any changes to the /dist files.

Seems I made a mess of things. Another attempt: #1785


Various additions and refactorizations to MSFS plugin

End user visible
Bugfixes

Fixed new value for some cases of wrapping around when adjusting with an encoder
Corrected border for buttons when disabled
Fixed background image of Flashlight in Lights folder
Fixed rounding error in display of fuel remaining
Fixed rounding error of N1 values in the RPM input
Fixed rounding error in the gallons per hour display of the Fuel display
Fixed these two problems in the airliner dynamic folder:
-- The last two buttons could not be shown because there was no navigation to second page.
-- The four reset commands were not implemented. They are now implemented as in the AP folder.

Added Functionality

Added a new encoder for showing and adjusting VOR OBS
Added a weather display input
Added tuning of ADF frequencies to the NAV dynamic folder
Barometer encoder shows pressure in both InHg and hPa
Added a cockpit view folder that also supports setting custom views
Added a lights display input
ConnectionSimConnect can now also disconnect
RPM input can now show N2 too
RPM input can now handle propeller planes with 4 engines
Fuel display now also shows pounds per hour
RPM input: for turbo props, Ng and RPM are shown

Other

Changed Barometer encoder push button to setting altimeter to current pressure rather than standard pressure
NAV frequency adjustment adjusts with step 0.05 rather than 0.01

Improvements to maintainability

Using environment variable to target the MSFS SDK rather than hardcoded path C:\MSFS_SDK
Made comments in the code for where to add code for new bindings
Added unit tests for ConvertTool
Added a class for debug tracing
Restricted access to properties of Binding class to be read only externally
Creation of Binding instances now mainly done in MsfsData.Register, which is now called by a BindingKeys value
Lowered instances of encoder value flipping back and forth when adjusting an encoder by multiple steps
Interface Notify renamed to INotify
Changed included SimConnect.dll to be that from SU14
Centralized definition of generally used colors
Added missing disposals of BitmapBuilder objects
Split SimConnectDAO.cs into smaller pieces
Simplified how:
-- Sim values get set to Binding objects
-- Data definitions are added
Removed unnecessary duplicate binding keys for lights and autopilot functionality
Started work to introduce names for all bindings such that changes and adjustments can refer to the relevant bindings by name rather than indexing in a list

Refactorizations

ImageTool: Avoid prefixing with class name inside the class
Removed superfluous this references
Change to coding preferences: prefer predefined types for .NET framework types
Introduced a new MSFSData Register method that avoids throwing bindings away
Introduced DecimalValueAdjuster to centralize logic for adjustment in connection with encoders
Removed leading underscore in various field names

Ok lot of changes ... I will cherrypick them if all if ok

Thank you. I have a couple of changes more that I would like to make a new PR for. The most important thing is that I have fixed (in the Binding class) more occurrences of encoder values flickering (e.g. heading and vor/obs) back and forth when the encoder is changed rapidly.


Protect config around SharedMutex

There were too many concerns about avoiding rw-lock around the config
var. The primary reason it was avoided was performance. We will roll
this back if this hits perf issues. folly/SharedMutex seems to be quite performant

Abandoning this to split into different PRs.


Build omits transitive dependencies

I'll just preface this issue by noting that I suspect this isn't a paket issue, but I am hopeful that someone from Paket might be able to recommend the correct place to post this, or provide an explanation or solution
Description
I have setup a trivial sample project using paket. Here we have a console application and a class library which the console depends on. The class library in turn depends on Microsoft.VisualStudio.Threading ~> 14.1.131. This has a transitive dependency on Microsoft.VisualStudio.Validation (>= 14.1.111). When paket resolves this, I end up with Theading at 14.1.131 and Valdiation at 15.0.82 - which all seems fine based on the constraints.
When I build the project, the bin directory for my class library contains both the threading and validation dlls as expected, however the console application only contains the class library and the threading dll. If we then run the console application we end up with an exception because Validation cannot be found, provided it isn't in the GAC. There are no warnings or errors emitted during the build.
I've run the build with the highest verbosity level and haven't been able to pinpoint where something might be going wrong. From what I can tell when building the console application it fails to recognize that Validation V15 is a dependency from the class library, instead its noting that Threading was built against V14 of validation. It however cannot find the V14 library locally and instead finds it in the GAC, which doesn't result in copying it locally. It should really however be trying to deal with V15 I'd have thought. If you then take the resulting build and run it on a machine without Threading in the GAC, it will crash.
If I manually add a binding redirect within the console application, it will work correctly. But it seems like there should be a warning or recommendation here to add that for me? I have AutoGenerateBindingRedirects set to true in the csproj's to try coax that behavior, to no avail
Here is an example build log
If anyone could provide some insight into this it'd be much appreciated, currently I am thinking that I need to switch from Max to Min resolver strategy within paket to avoid this happening in the future, but that doesn't really explain why this is happening without any warnings or errors.
Repro steps

Download the project here

Run the bootstrapper

Run paket install

Build the solution

Expected behavior
V15 of the Validation library to be present within the Console applications bin folder
Actual behavior
No version of Validation is present within the bin folder
Known workarounds
Changing the resolver strategy to min will prevent paket from fetching v15 of validation.
Manually adding a binding redirect for V15 will copy the library over.

Just tried. I don't actually get any errors with the app when I run it, although I did see the same behaviour that you observed i.e. no validation package is copied. As you observed, copying across a BR manually fixes the problem (which surprised me, I must admit).
Some other observations: -

The lock file in the sample project is out of date compared to the dependencies file. I don't know why that is though.
Paket has binding redirects support built in which currently isn't turned on for your project (add redirects: on to the .dependencies file). Unfortunately even turning that on does not fix it.
You can work around this by adding the a paket.references file to the console app so that it gets the project added explicitly (which then does get the binding redirect added by Paket).
As an aside, when I then removed the paket.references file, the BR was not removed.

BTW - you don't need to call the references .paket.references - just paket.references will do fine :-)

OK. I think I know what the issue is - you don't have a paket.references file in the console project. If you add into the same folder as the Console app - even if it's empty - then paket will realise that it needs to add BRs to the config file in there.

Interesting find. Unfortunately in my usecase I can't use paket within the Console project, we're using Nuget there. The reasons for this are complex unfortunately but I can explain if its really required. I've gone ahead and switched to using pessimistic resolution strategies in paket to avoid this situation for now.
It seems to me that this is likely an msbuild issue though, or is it genuinely pakets responsibility to be adding those BRs into the console project?

Hang on - so you're using paket for the class library and nuget for the console application? If that's the case, I have no clue why this is happening but yes, it's either a nuget or msbuild issue.
I'd suggest we close this.

I'll try dig into this a little more in the next couple days. I don't think the fact that our actual application is using Nuget is relevant, because there is no overlap between the dependencies were using. This is why the demo solution has the same issue.
I'll come back and update once I've got some more details. Will try experiment with Nuget to force it to use V15 of validation and V14 of threading, if at all possible. This would at-least rule paket out entirely.

Closing, lack of activity.


Publish new release with files defined in package.json

Came up from discussion on #259 - occured to me that npm only needs to publish the index.js file in releases, the rest are either published by default or can be ignored.
See https://docs.npmjs.com/files/package.json#files
Next release should be tested with the files attribute set first. If there's no problems, release it and we can action #259 using a GH pages directory with markdown tips for advanced use cases.

Have changed the package.json - just need to test it with the boilerplate before we publish.


Write an example/recipe for going from structure to chemiscope input

There should be an example (maybe in https://github.com/cosmo-epfl/kernel-tutorials/) on how to go from structure to chemiscope, using SOAP and PCA (maybe KPCovR).

We now have an example since #166 (https://github.com/lab-cosmo/chemiscope/blob/83f9929b2d226383fcf57bd1173546033e50fba5/python/examples/molecular_map.py), we only need to display it properly in the documentation

This has been done!


matching properties of serde

It seems that I am able to create tables with invalid properties. For e.g. "some_junk" is not valid property of csv serialization. The statement should fail.
CREATE EXTERNAL TABLE test2 (
f1 string,
s2 string)
ROW FORMAT SERDE 'org.apache.hadoop.hive.serde2.OpenCSVSerde'
WITH SERDEPROPERTIES ("some_junk" = ",", "escapeChar" = "\\")
LOCATION 's3://DOC-EXAMPLE-BUCKET/dataset/test1/'

I was expecting an error because of "some_junk". But the statement was executed successfully.

Raising an issue as suggested in the comments of this thread...
https://stackoverflow.com/questions/72908311/matching-properties-of-serde


Update README.md

I can't find any C# examples with the 'official' (still in pre-release) SDK. Am I missing anything? That is the reason I have been using this library,

The SDK was released yesterday. As a good MSFT citizen it's important for me to promote the official library from my company however I like this implementation as well since it comes with some DI goodness and allows for the community to contribute in a different way.
Since OpenAI supports a streaming response, I'd like to see if this library can support C# events to get a native callback experience when the platform streams data. Not sure if the official SDK will support this feature either.
Keep using it and contribute if you'd like. I'll maintain it for as long as people find it useful.

Yes, we are on the same page. You added the link to the "official" library so no one will get confused. However, a community library can move faster (ChatGPT could drop any day now and I would not expect an official SDK update for weeks). Also it can add helper methods (I'm thinking about uploading and displaying images with DALL-E when that becomes available).

Also see: https://github.com/lucabriguglia/OpenAI.NET


tarfile.ReadError

My module, based on deeppavlov, runs on my machine. However, it does not work on a remote server with Ubuntu 18.04 because of  tarfile.ReadError.

I use deeppavlov of 0.8.0 version with small modifications: github.com/alissiawells/deeppavlov (treshold instead of model’s verdict in the output)

Hi @alissiawells,
This is most likely related to #1109 (it will be fixed in the next release) as the console shows 2.76 KB downloaded while the file itself is about 5 MB.
There may be a typo in the download link or some troubles with network.
You can try and download and extract the file with other methods.


Unable to change the color of mxgraph.aws3d.dashedArrowlessEdge

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
The color of mxgraph.aws3d.dashedArrowlessEdge is aways blue no matter what the value of strokeColor is set to.
To Reproduce
Steps to reproduce the behavior:

Using the AWS 3D shapes, added a mxgraph.aws3d.dashedArrowlessEdge
Using the Format Panel change the Line color to black

Expected behavior
Color of Line changes to black
Screenshots
If applicable, add screenshots to help explain your problem.
draw.io version (In the Help->About menu of the draw.io editor):

diagrams.net 15.4.0

Desktop (please complete the following information):

OS: MacOS
Browser: App
Version

Additional context
Add any other context about the problem here.

The offending code appears to be
https://github.com/jgraph/drawio/blob/d76e39477bdf230ba6728dc488bd5896afb08425/src/main/webapp/shapes/mxAWS3D.js#L350

Done. Should be in the latest release.


Fix/#74 build on mac catalyst

Fixes the build on Catalyst + adds the needed @.available checks to the unit test target, so it can build on tvOS and watchOS

Fixes #74


Show Markup broken for some examples

Expected Behavior
Show Markup does what it promises to do, all of the time
Actual Behavior
Broken promises, broken hearts\
Steps to Reproduce

http://opensource.adobe.com/spectrum-css/2.8.0/docs/#toast
Click "Show Markup"
Cry

Spectrum-CSS version
2.8.0

Yeah, seen this issue quite a few times.

Same, like the Link examples. I need to open up devtools, like a sucker, to see the markup! 😉

In progress, here: https://github.com/naeohmi/spectrum-css/tree/issue/117


[SPARK-29398][core] Support dedicated thread pools for RPC endpoints.

The current RPC backend in Spark supports single- and multi-threaded
message delivery to endpoints, but they all share the same underlying
thread pool. So an RPC endpoint that blocks a dispatcher thread can
negatively affect other endpoints.
This can be more pronounced with configurations that limit the number
of RPC dispatch threads based on configuration and / or running
environment. And exposing the RPC layer to other code (for example
with something like SPARK-29396) could make it easy to affect normal
Spark operation with a badly written RPC handler.
This change adds a new RPC endpoint type that tells the RPC env to
create dedicated dispatch threads, so that those effects are minimised.
Other endpoints will still need CPU to process their messages, but
they won't be able to actively block the dispatch thread of these
isolated endpoints.
As part of the change, I've changed the most important Spark endpoints
(the driver, executor and block manager endpoints) to be isolated from
others. This means a couple of extra threads are created on the driver
and executor for these endpoints.
Tested with existing unit tests, which hammer the RPC system extensively,
and also by running applications on a cluster (with a prototype of
SPARK-29396).

Merged build finished. Test PASSed.

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

Test build #111919 has started for PR 26059 at commit 3316d5e.

Test build #111919 has finished for PR 26059 at commit 3316d5e.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 class DriverEndpoint extends IsolatedRpcEndpoint with Logging

Merged build finished. Test PASSed.

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

lgtm

Merged build finished. Test PASSed.

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

retest this please

Test build #112192 has started for PR 26059 at commit b674b4c.

Merged build finished. Test PASSed.

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

Test build #112192 has finished for PR 26059 at commit b674b4c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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


How can i locate elements in the application with browser stack?

Im not sure if anyone has encountered this already. But i want to do test automation on browser stack. The problem is i cannot locate the mobile elements of the application created in flutter.

It's not really clear what you're asking about here? Is this for flutter web?

Sorry i wasn't clear enough. Its flutter mobile app. In browser stack you can locate elements for mobile app

I don't know what you mean by browser stack or element.
The app switcher and views?


Document use of experimental syntax (e.g stage-0, stage-1)

An easy way to do this would be just remove stage-0 from .babelrc and chase down the errors.

PR on the v3.0.0 branch very welcome! Also added to #14

Came to this documentation section via #158.  fb React'ors are encouraging us to avoid the use of the this autobinding nature of the '=>' operator within React.Components`.
Rational:

The arrow function creates a new function (and therefore a new bind) on every render, so if you are using shouldComponentUpdate with the shallow-compare addon, it will always return true, and your sub-component will always re-render. This can have substantial impact on performance.

If you are still interested in pushing babel stages, I had been leveraging the @autobind decorator that did the same, yet didn't suffer from the aforementioned impact.  I ran into some Flow typing issues with @autobind and had to abandon it for the time being, but in lieu of that ...
... fb React'ors are saying that for now, it is best for React.Components (from a performance perspective) to bind in their constructors.
constructor(props) {
  super(props);
  this.tick = this.tick.bind(this);
}

... alas, 'function over form' still might matter. ;-)

Closing due to inactivity


[ENH] Storing basic provenance on JSON sidecars of derivatives

Summary
In the context of BEP 016 we found ourselves prescribing what metadata fields should/could be included within derivatives. We then realized that some of them would easily extend to other Imaging derivatives (see #310), and also tapped on the quite controversial SkullStripped metadata field.
Proposal
Since BIDS has always been reluctant to include much provenance information, this PR proposes to make all these metadata fields OPTIONAL, and provides some suggestions for naming.
Original Content of this PR
Mapped from @Lestropie's bids-standard/bids-bep016#5
Closes bids-standard/bids-bep016#2
I have removed some keys that I believe should remain in BEP16 for being
applicable to dwi only.
I have also changed FieldInhomogeneity* with
SusceptibilityDistortion* since I think the latter will be more
familiar to a wider audience.
EDIT: fields that remained in BEP16 are - EddyCurrentCorrection, GradientNonLinearityQSpaceCorrection, SliceDropoutDetection, SliceDropoutReplacement

cc/ @bids-standard/derivatives-mri-dwi

Since this is common derivatives, we should be pinging @bids-standard/derivatives.
These questions in particular are relevant to @bids-standard/derivatives-mri, but there has been discussion (apologies for not linking at the moment; I'll try to find it) with @bids-standard/derivatives-electrophys that SkullStripped is problematic, as it only applies to MRI derivatives. That seems to be true here, as well, although at least they are optional.
I guess I would say that either we should merge this quickly and address the MRI-centricity in #265, or address it there first, and hold off on this until that's resolved.

SkullStripped but also FieldInhomogeneity etc. would never apply to MEG, EEG, iEEG, but also not to BEH, which is also a full-featured (but probably often overlooked) BIDS data type.
I don't think that any datatype specific metadata fields should be required as per common-derivatives.

I drafted this PR to move out the discussion from BEP16, as we felt we were talking about metadata that could easily be present in other MRI derivatives.
That said, my opinion is that we should remove all these metadata bits for two reasons:

They do not apply to non-MRI data
They keep track of provenance, which is something beyond the scope of BIDS. Although it is difficult to keep provenance away in all the cases, I think in this particular time Skullstripped and the ones I added in this PR should be removed. IMHO it is a task for the workflow developers to allocate a dedicated provenance tracking (NIDM?) and/or insert the sufficient metadata information in more openly-scoped metadata fields/ READMEs / documentation / citation boilerplates.

I guess I would say that either we should merge this quickly and address the MRI-centricity in #265, or address it there first, and hold off on this until that's resolved.

I'm fine either way, but if Skullstripped is going away or a better solution is found in #265, then this PR does not make any sense. So I think I'm inclined to figure this out in #265 first and then downstream the decision here and into BEP16 (because we are defining there more of these metadata tracking keys - the non MRI side of it does not apply anymore, but the provenance argument is still valid).

@oesteban this PR is still marked for the common derivatives milestone.
How do you want to proceed with this PR? Can it be closed? Should it become an issue for further discussion? Some other idea?

@oesteban - i do think there is a need to curate a vocabulary around processes. perhaps we can work on that as a separate PR after the common derivatives is merged. as this vocabulary could be used in the provenance BEP #487 .

Sure, what's the course of action regarding BEP016? Should we just remove mentions to these provenance entries?


can't modify frozen String

The setup works great after update to jenkins (2.3.1) i get a error
RuntimeError
------------
can't modify frozen String

Cookbook Trace:
---------------
/var/lib/monsoon/repos/eventticketing/jenkins/chef/cookbooks/jenkins/libraries/_helper.rb:140:in `gsub!'
/var/lib/monsoon/repos/eventticketing/jenkins/chef/cookbooks/jenkins/libraries/_helper.rb:140:in `convert_to_groovy'
/var/lib/monsoon/repos/eventticketing/jenkins/chef/cookbooks/jenkins/libraries/credentials_private_key.rb:99:in `credentials_groovy'
/var/lib/monsoon/repos/eventticketing/jenkins/chef/cookbooks/jenkins/libraries/credentials.rb:117:in `block (2 levels) in <class:JenkinsCredentials>'
/var/lib/monsoon/repos/eventticketing/jenkins/chef/cookbooks/jenkins/libraries/credentials.rb:104:in `block in <class:JenkinsCredentials>'

my code
data_bag('deploy_keys').each do |project|
  jenkins_private_key_credentials project do
    description "#{project} pivate key for github access"
    private_key data_bag_item('deploy_keys', project)['private_key']
  end
end

i fix this changing the username from project to project.dup

Just wanted to comment, that I'm running into a similar issue with the jenkins_ssh_slave LWRP.  Have not investigated at all what may be causing this after coming across this issue and fixing with  .dup.
Code from wrapper cookbook:
if slaves.nil?
  Chef::Log.warn('Could not find any slaves, have they escaped?')
else
  # define the slaves launched via SSH
  slaves.each do |slave_name, slave|

    # There is an issue with this LWRP trying to alter slave_name
    # and running into issues with it being immutable/frozen.
    # At somepoint this may be fixed up stream and the .dup can
    # be removed.
    # Similar issue here: https://github.com/opscode-cookbooks/jenkins/issues/371
    jenkins_ssh_slave slave_name.dup do
      description 'Run builds as slaves'
      remote_fs   '/var/lib/jenkins'
      labels      ['executor']

      executors  slave['executors']

      # SSH specific attributes
      host        slave['fqdn']
      user        node['jenkins']['master']['user']
      credentials node['jenkins']['master']['user']
    end
  end
end

Great! Can we get a version bump on this?

We'll bump the version when we ship a new version of the cookbook. Still performing testing and merging  various fixes.


fix: incorrect protobuf body field used

Description:

This PR fixes _fromProtobuf inside of TokenUpdateNftsTransaction to use the correct tokenUpdateNfts field instead of tokenUpdate. Upon investigating https://github.com/hashgraph/hedera-wallet-connect/issues/265, we realized this property was not being referenced properly.
Related issue(s):
Fixes https://github.com/hashgraph/hedera-wallet-connect/issues/265
Notes for reviewer:
You can confirm the error happens outside of this branch by running the following code:
import Long from "long";
import { TokenUpdateNftsTransaction, Transaction } from "./exports.js";

const updateNftsTransaction = new TokenUpdateNftsTransaction()
    .setTokenId("0.0.123")
    .setSerialNumbers([Long.fromInt(1)])
    .setMetadata(new TextEncoder().encode("metadata"));

const parsedTx = Transaction.fromBytes(updateNftsTransaction.toBytes());

console.log("parsedTx: ", parsedTx._makeTransactionBody());

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Hi @kantorcodes , thank you for submitting this PR! It looks great.
Would you mind adding a unit test for this change in the TokenUpdateTransaction.js to satisfy the code coverage in the CI check?

Nice catch! This should be the correct behaviour.
Looks like we were missing some unit tests for this class as a whole and we also have a TokenNftsUpdateTransaction class which should be deleted.
@ivaylogarnev-limechain do you think we can do this in a follow up issue?

Nice catch! This should be the correct behaviour.
Looks like we were missing some unit tests for this class as a whole and we also have a TokenNftsUpdateTransaction class which should be deleted. @ivaylogarnev-limechain do you think we can do this in a follow up issue?

For sure, that sounds like a better option.

Hi @kantorcodes thanks for your effort. Your commit is not verified, it's signed, but it's not verified which may cause us problems in the future, can you try to sign it again may be using ammend.  I can also see that your commits are not verified in other repos.
Are you sure you've setup your GPG keys properly? You can look at this page for more info (Signing commits).

Hi @kantorcodes thanks for your effort. Your commit is not verified, it's signed, but it's not verified which may cause us problems in the future, can you try to sign it again may be using ammend. I can also see that your commits are not verified in other repos.
Are you sure you've setup your GPG keys properly? You can look at this page for more info (Signing commits).

Thanks! I think I've got these signed properly now :)

commits must be verified

Thanks for taking a look. I think commits should be signed properly. I realized afterwards (and likely why this went unnoticed initially) that fromProtobuf had no test coverage and this seems to have blocked the merge, so I added another commit to add a basic test in as well.


Selected row is not highlighted in value panel with dark theme

System information:

Windows 10 Pro 20H2 19042.928
Version <IP_ADDRESS>104181339

Connection specification:

Postgresql

Describe the problem you're observing:
With standard dark theme applied in the value panel the selected row is not highlighted.

Hello @Alterant-zz
Unfortunately, could not reproduce.

May be "Reset UI settings"/"Reset workspace state" can help

Or you can try enable this setting:

Made a reset of UI settings and workspace state.
Enabled option "Highlight rows with ..."
Nothing changed.
Switched Windows theme to Bright.
Also with no luck :(

thanks for the report

Fixed.

Verified

I can confirm that now everything works like a charm. Thanks a lot!


[1.20.1] Crash "Rendering entity in world"

Hi, I was doing some testing and when I teleported back to my main testing base, I got this crash: https://gist.github.com/katubug/4aa819f097bbecb025afb5018277aba9
I had had several Mooblooms (yellow and pink) around for a while, but it was only when I teleported away and came back that I got the crash. Unfortunately I don't have the matching latest.log at hand because this was a day or two ago, but if you need it I can try to hunt through them.
This is on B&B 1.9.0, Forge 47.2.19.

Will look into this later.

This was only fixed on 1.20.4, re-opening for that reason.


AttributeError: 'Operation' object has no attribute '_c_op'

tensorflow2.1
File "run.py", line 118, in
imgs_mask_test = model.predict(imagestest, verbose=1)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training.py", line 1013, in predict
use_multiprocessing=use_multiprocessing)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 498, in predict
workers=workers, use_multiprocessing=use_multiprocessing, **kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 475, in _model_iteration
total_epochs=1)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 128, in run_one_epoch
batch_outs = execution_function(iterator)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2_utils.py", line 98, in execution_function
distributed_function(input_fn))
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/eager/def_function.py", line 568, in call
result = self._call(*args, **kwds)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/eager/def_function.py", line 615, in _call
self._initialize(args, kwds, add_initializers_to=initializers)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/eager/def_function.py", line 497, in _initialize
*args, **kwds))
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/eager/function.py", line 2389, in _get_concrete_function_internal_garbage_collected
graph_function, _, _ = self._maybe_define_function(args, kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/eager/function.py", line 2703, in _maybe_define_function
graph_function = self._create_graph_function(args, kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/eager/function.py", line 2593, in _create_graph_function
capture_by_value=self._capture_by_value),
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/framework/func_graph.py", line 978, in func_graph_from_py_func
func_outputs = python_func(*func_args, **func_kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/eager/def_function.py", line 439, in wrapped_fn
return weak_wrapped_fn().wrapped(*args, **kwds)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2_utils.py", line 85, in distributed_function
per_replica_function, args=args)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/distribute/distribute_lib.py", line 763, in experimental_run_v2
return self._extended.call_for_each_replica(fn, args=args, kwargs=kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/distribute/distribute_lib.py", line 1819, in call_for_each_replica
return self._call_for_each_replica(fn, args, kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/distribute/distribute_lib.py", line 2164, in _call_for_each_replica
return fn(*args, **kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/autograph/impl/api.py", line 292, in wrapper
return func(*args, **kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2_utils.py", line 212, in _predict_on_batch
result = predict_on_batch(model, x)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training_v2_utils.py", line 556, in predict_on_batch
return predict_on_batch_fn(inputs)  # pylint: disable=not-callable
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/base_layer.py", line 778, in call
outputs = call_fn(cast_inputs, *args, **kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/network.py", line 717, in call
convert_kwargs_to_constants=base_layer_utils.call_context().saving)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/network.py", line 891, in _run_internal_graph
output_tensors = layer(computed_tensors, **kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/base_layer.py", line 778, in call
outputs = call_fn(cast_inputs, *args, **kwargs)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/keras/layers/convolutional.py", line 209, in call
outputs = self._convolution_op(inputs, self.kernel)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/ops/nn_ops.py", line 1135, in call
return self.conv_op(inp, filter)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/ops/nn_ops.py", line 640, in call
return self.call(inp, filter)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/ops/nn_ops.py", line 616, in _with_space_to_batch_call
block_shape=self.dilation_rate)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/ops/array_ops.py", line 3303, in required_space_to_batch_paddings
const_block_shape = tensor_util.constant_value(block_shape)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/framework/tensor_util.py", line 827, in constant_value
ret = _ConstantValue(tensor, partial)
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/framework/tensor_util.py", line 673, in _ConstantValue
if tensor.op.type == "Const":
File "/home/ma-user/anaconda3/envs/TensorFlow-2.1.0/lib/python3.7/site-packages/tensorflow_core/python/framework/ops.py", line 2220, in type
return c_api.TF_OperationOpType(self._c_op)
AttributeError: 'Operation' object has no attribute '_c_op'
I got this error, how to solve it?thank you.

@888yyh ,
In order to expedite the trouble-shooting process, could you please provide the complete code and dataset to reproduce the issue reported here.Also please try to execute the code using stable tf v2.5 and let us know if you are facing same issue.Thanks!

@tilakrayal @888yyh Has there been resolution to this issue?
I am getting this error with TF2.1.0 randomly. With 2.7.0 this never happens.
In order to be compatible with other things, we are not yet ready to do an upgrade to 2.7.0

@888yyh Hello，I encountered the same problem，Have you  adjust this error？thank you~


MD-2893: Modified queries involving limit & their baseline

Query drill3788.q - Table had multiple rows with same partkey, so also ordered by discount
Query 5level_sanity2.q - Table had 8 rows for each partkey, so reduced limit to 8
@Agirish please review

+1, pending review comments.


[Documentation suggestion] Provide executable examples and link to them

Readers of Catch' documentation may benefit from readily available, complete and executable examples that show a limited and coherent set of features and steps needed for compilation. The tutorial and reference documentation can then link to relevant examples that the reader can try, perhaps even online.
Compilation of these examples must be integrated with the continuous integration process. As such the examples become executable documentation geared towards the user.
For example:
// Assertions

// Compile:
// - g++ -I$(CATCH_SINGLE_INCLUDE) -o assertions assertions.cpp && assertions --success
// - cl -EHsc -I%CATCH_SINGLE_INCLUDE% assertions.cpp && assertions --success

// Let Catch provide a main():
#define CATCH_CONFIG_MAIN

#include "catch.hpp"

std::string one() {
    return "1";
}

TEST_CASE( "Assert that something is true (pass)", "[require]" ) {
    REQUIRE( one() == "1" );
}

TEST_CASE( "Assert that something is true (fail)", "[require]" ) {
    REQUIRE( one() == "x" );
}

TEST_CASE( "Assert that something is true (stop at first failure)", "[require]" ) {
    REQUIRE( one() == "x" );
    REQUIRE( one() == "1" );
}

TEST_CASE( "Assert that something is true (continue after failure)", "[check]" ) {
    CHECK(   one() == "x" );
    REQUIRE( one() == "1" );
}

Subjects that come to mind:

test case (test_case)
assertions (require, check)
exceptions (require_nothrow, require_throws, require_throws_as, require_throws_with, require_throws_matches)
matchers (require_that)
floating point (approx)
logging (capture, info, warn, fail, fail_check)
fixtures (test_case_method)
sections
bdd (scenario, given, when, then, and_when, and_then)
string conversions
reporters
event listeners
supplying your own main()
compile-time configuration

Basically, you want parts of SelfTest to become self-contained and hosted on Wandbox or similar, correct?

Indeed, with the difference that SelfTest may be geared (too much) towards implementers, complete and overwhelming where mere users may need another approach to support their understanding.
Key:

relatively simple
complete program
that compiles and executes and shows what's necessary for that
and is at the tip if reader's fingers from the documentation

See for example the examples linked from section Usage of lest.

Yeah, SelfTest's primary aim is to (mostly) guarantee that things didn't get broken by a change, secondary aim is a reference for someone who already knows Catch well and documentation for newbies is a very distant third.
Anyway, I can see some value in adding these, but honestly don't have much time (and what little I do, I'd like to use for different things), so this will likely either get put on a backburner, or someone else needs to put in a PR (hint, hint).

I can see some value in adding these

is what I liked to know ;)

Plan (in progress):
Overall:

[x] Create branch examples
[x] Create folder examples
[ ] Create several examples to get the gist, see below
[ ] Link to examples from Tutorial and from Reference
[ ] Publish branch for review feedback
[ ] ...

Examples:

[ ] 01-TestCase, single file
[ ] 02-TestCase, multiple files
[ ] 03-Assertions
[ ] 04-...

I installed the Catch2 framework using the CMake installation and when I try to compile and run any example I get this error: CATCH_SINGLE_INCLUDE: command not found
this is how I compile an example:
s$ g++ -std=c++11 -Wall -I$(CATCH_SINGLE_INCLUDE) -c 000-CatchMain.cpp
what is the purpose of the macro CATCH_SINGLE_INCLUDE ?

@ahadji05 It should contain path to the single-include version of catch.hpp.


Update BTC -Markets exchange

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Updating BTC-Markets exchange

What is the current behavior? (You can also link to an open issue here)
BTC-Markets code outdated. see (https://github.com/askmike/gekko/issues/1169)

What is the new behavior (if this is a feature change)?
Support for BTC-Markets trading

pull trading fee

fixed cancel order

Other information:

Update BTC-Markets package to v0.0.10
Update btc-markets.js
Its working fine so far but will testing a bit longer, I should be able to complete in the next few days. Any tweaks after testing I will commit to this branch
Keeping it in develop for now.

Incorrect


2.4.x update release test

update release test to match normal workflow

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.15 out of 16 committers have signed the CLA.:white_check_mark: FahadBSyed:white_check_mark: lbliii:white_check_mark: molinamelendezj:white_check_mark: chainlink:white_check_mark: seslattery:white_check_mark: brycemcanally:white_check_mark: robert-uhl:white_check_mark: jrockway:white_check_mark: tybritten:white_check_mark: albscui:white_check_mark: brendoncarroll:white_check_mark: acohen4:white_check_mark: smalyala:white_check_mark: armaanv:white_check_mark: djanicekpach:x: nikolay-pavlov-snkeosYou have signed the CLA already but the status is still pending? Let us recheck it.


Installation error when is no Husky installed

I have no Husky on my project so when I'm running
npm install serverless-plugin-static -D

I get the error
npm ERR! command failed
npm ERR! command sh -c husky install
npm ERR! sh: husky: command not found

since it's in the package.json file.

Thanks for reporting this issue! It's been fixed in 0.0.7.


bug nl.nn.adapterframework.batch.BatchFileTransformerPipe

Describe the bug
When Using the pipe the result doesn't include the whitespace before the values
Reporter
Tarik
Category

Frank Framework

To Reproduce
Steps to reproduce the behavior:

Use the nl.nn.adapterframework.batch.BatchFileTransformerPipe
config it so it uses a manager, flow, recordhandler

Expected behavior
When copying a value it should add whitespaces before and after the values
Screenshots

I would expect 2 whitespaces infront of this value
Environment:

ff.version=7.6-20200909.154756

Additional context

<recordHandler
name="NADTPHeadHandler"
className="nl.nn.adapterframework.batch.RecordXmlTransformer"
inputFields="5,7"
outputFields="tagName,VALUE"
rootTag="NADTP"
input: NADTP 390152
output:

A preserveWhitespace attribute could be a solution but is it really necessary?

I needed the whitespaces but now had to put them all back, this due to the backwards transformation on the childs level.
Which means in the body of the for eachchild the app needed both transformed data as the original form
Outlook voor Android downloadenhttps://aka.ms/ghei36

From: jacodg<EMAIL_ADDRESS>Sent: Monday, October 12, 2020 2:15:00 PM
To: ibissource/iaf<EMAIL_ADDRESS>Cc: LEEERRoooYYY JJJEENNKINNSSS<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [ibissource/iaf] bug nl.nn.adapterframework.batch.BatchFileTransformerPipe (#1084)
A preserveWhitespace attribute could be a solution but is it really necessary?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ibissource/iaf/issues/1084#issuecomment-707083491, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJYYHUAPBG6QLOQVV6DTHE3SKLXMJANCNFSM4RWWP7VA.

It appears that the whitespace is trimmed by the conversion to XML. We won't fix this, as we consider this to be too much of an edge case


Temporarily activate high-resolution tracking during trips

When the user starts a trip, turn up the accuracy (and any other necessary settings) while the trip is active. After the trip ends, restore the settings to the user-selected settings.

There is now a new settings screen that applies during trips, so you can choose your own high-resolution settings during a trip!


[SPARTA-508] add delete checkpoints when updating fragments

Given an existing policy which contains some fragments
When you modify any of these fragments
Then the checkpoint associated with this policy must be deleted

Added some fixes and line indentions (fixed my editor).
I disagree with reference to yield, or not, updateResponse. If updateResponse fails and returns a Failure, you need to include in pattern matching, with relatedPolicies.

+1

+1

:shipit:

Comment accepted. Will treat it ASAP

Squashing and checks in progress.


213 provide hd wallet implementation

Implementation of a first version HD wallet wrapper.

Hi @edridudi,
@Kammerlo  has implemented an initial version of the HD wallet module in this PR. We are currently reviewing it to finalize the api.  Feel free to review it as well and let us know if there are any specific requirements or enhancements you would like to see in this new API.
Thanks

Hi @edridudi,
@Kammerlo has implemented an initial version of the HD wallet module in this PR. We are currently reviewing it. Feel free to review it as well and let us know if there are any specific requirements or enhancements you would like to see in this new API.
Thanks

Hope you don't mind my adding a few comments / observations?! Thank you for doing this work!!

@gavinharris-dev Thanks for your comments / feedback. This PR is still in draft, and the current plan is to finalize it after the Conway HF, as it will impact some existing key APIs. So, please feel free to provide any feedback.

@Kammerlo
I have done some minor cleanup and changes to this PR. It seems the core functionalities are working fine. I think it's ready to be merged. Though there are a few enhancements required before the release, we can handle them in separate PRs.
Can you please have a quick look before we merge this?

I have added a separate issue to track future enhancements of the Wallet api
https://github.com/bloxbean/cardano-client-lib/issues/471


Version 0.1.1

@JuliaRegistrator register()
Release notes:
Major changes

loading a default CCconfig.toml file from the repository if an existing one is not found;

Minor changes

a built-in EC2 image is provided (see updates in README/documentation).
messages in cluster operations are improved.
warning if the configuration is empty for a supported provider.
README/documentation updates regarding the configuration file (CCconfig.toml).

Registration pull request created: JuliaRegistries/General/120445
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.1 -m "<description of version>" 161ac3351aec4af4b5fadc5814f7f5e26cdd8a9d
git push origin v0.1.1


[v1.4.x] Add pin_device_id option to Gluon DataLoader (#14136)

Cherry-pick the change that fixes out of memory error when using Gluon DataLoader with CPUPinned memory during distributed training on multiple GPUs with MXNet + Horovod to v1.4.x branch.

@junrushao1994 @szha Please help review and merge. Thanks.


Implemented ability to pass in multiple Locustfiles to select UserClasses via the WebUI

[Updated to reflect the Locustfile picker -> UserClass picker changes]
My team has a use case where we'd like to be able to pick a UserClass after spinning up the WebUI. The goal of the PR is to:

add this functionality without changing existing workflows
Run locust with multiple locustfiles. Primarily to be used with the User class Picker

The -f argument has been updated to allow for multiple locust files or a directory. All User classes in the specified locustfiles will be used by default.
The newly added -enable-userclass-picker enables the user to select the desired User classes & shape class in the Web Ui. If no selection is made for User classes, all User classes are used. The shape class defaults to the regular shape.
Other noteworthy changes:

Error raised if the specified directory doesn't exist, or doesn't have any valid Locustfiles
If the UserClass Picker is visible in the WebUI and there is no UserClass selection, /swarm will use all available UserClasses
load_locustfile, is_shape_class, is_user_class were moved from main.py to util/load_locustfile.py to avoid a circular import in web.py

This is no longer necessary for this PR, but I kept it in to cut down on the size of main.py

There is a semi-braking change with allowing multiple files

locust -f locustfiles QuickstartUser1 will throw an error as it expects QuickstartUser to be a Locustfile. The workaround is to use locust QuickstartUser1 -f locustfiles

Example:
Running locust -f locustfiles --enable-userclass-picker with the following file structure:
├── locustfiles/
│   ├── file1.py
│   ├── file2.py
│   └── shape_classes/
│       ├── shape1.py
│       ├── shape1.py
│   └── more_files/
│       ├── file_3.py
│       ├── locust.py
│       ├── _ignoreme.py

Nice! Couple comments, and I’ll need to do a full review when not on mobile :)

Hmm. When I think about it - could this be solved by allowing multiple parameters for -f / —locusfile instead? It seems a more obvious and user friendly approach (if you want a whole directory you would just do -f mydir/*

Hmm. When I think about it - could this be solved by allowing multiple parameters for -f / —locusfile instead? It seems a more obvious and user friendly approach (if you want a whole directory you would just do -f mydir/*)

Ah nice, I think this is a great idea!

Can you change it so that it works that way? I guess using parser.add_argument(…, nargs='+', …)
Also, maybe we could make it so that you select Users in the GUI instead of locustfiles (and default to run all users). That would make a lot of sense, I think.

@cyberw I just pushed up a few commits, but this isn't ready for re-review yet. I still have to update the docs and fix some failing tests (I'm on an M1 and it seems like some tests fail no matter what locally, but are fine in Github Actions. Some due to slowness)

I've nothing constructive to add other than to say thank you for working on this pull request. I've been thinking about this feature for about a week so decided to take a look today to see if it had already been discussed .. and found this :)

Merge with master for a fix for the build (flask made an api change in 2.2)

I've nothing constructive to add other than to say thank you for working on this pull request. I've been thinking about this feature for about a week so decided to take a look today to see if it had already been discussed .. and found this :)

Thanks! I'm glad others have use for this as well 🤘

@cyberw This is ready for re-review. I've updated with your requested changes, most notably that the UI allows you to pick from UserClasses instead of Locustfiles (which is a much better so thanks for the suggestion!). I also added in a ShapeClass picker which only accepts one value and defaults to the normal use-case and only appears when the UserClass picker is active.
Additionally, there is no need for any additional command-line arguments/flags. -f/--locustfile accepts:

A single Locustfile (no changes to this logic)

locust -f some_locustfile.py

Multiple Locustfiles

locust -f some_locustfile.py some_dir/another_locustfile.py

A directory containing locustfiles

locust -f locustfiles

This is starting to look really good. Anything else before we merge? Can you squash the commits that didnt really relate to the final product? Or I can squash everything into one commit

This is starting to look really good.
Thanks! Your input made it 10x better than what I initially had

Anything else before we merge? Can you squash the commits that didnt really relate to the final product? Or I can squash everything into one commit

Yeah, actually. Incase you didn't see my other comment:
There is a breaking change that was implemented with allowing multiple values to be passed into -f/--locustfile. I only just now caught it. Specifying User classes in the CLI cannot be done after the -f arguments, since it expects the values to be locustfiles.
Good:

locust -f locustfiles --csv test_csv QuickstartUser1
locust QuickstartUser1 -f locustfiles

Bad:

locust -f locustfiles QuickstartUser1

Error: Invalid file 'locustfiles'. File should have '.py' extension

I didn't update the docs on this yet as I wanted to see if I should keep it as is (maybe at a note in the error message), or perhaps add a flag to specify the User classes.
I'm thinking that the best move is to keep it as is and update the docs & error message, since that will affect the least amount of users.

Oh, right. Yea we probably dont want to do that. Maybe use a comma separated list? Argparse doesnt seem to support it out of the box so SO recommends something like
https://stackoverflow.com/questions/52132076/argparse-action-or-type-for-comma-separated-list

Oh, right. Yea we probably dont want to do that. Maybe use a comma separated list? Argparse doesnt seem to support it out of the box so SO recommends something like
https://stackoverflow.com/questions/52132076/argparse-action-or-type-for-comma-separated-list

Done. Excellent suggestion 🙏 . I also squashed all of the commits.

Minor details:

remove ”enable” from —enable-user-picker
make the default to actually select all users (then you can also remove the message about no selected users resulting in all users)

Btw, its kind of a ”user AND shape class” selector, isnt it? Maybe it should be named something different?

Minor details:

remove ”enable” from —enable-user-picker
That's how i originally had it, but test_specify_config_file was failing due to an argparse error: ambiguous option: --u=100 could match --users, --userclass-picker

make the default to actually select all users (then you can also remove the message about no selected users resulting in all users)
Update the image
You're referring to the UI select box, right?

Minor details:
remove ”enable” from —enable-user-picker

That's how I originally had it, but test_specify_config_file was failing due to an argparse error:
ambiguous option: --u=100 could match --users, --userclass-picker

make the default to actually select all users (then you can also remove the message about no selected users resulting in all users)
Update the image

You're referring to the UI select box, right?

Btw, its kind of a ”user AND shape class” selector, isn't it? Maybe it should be named something different?

Yup! I couldn't think of a better name 😐 . Open to suggestions

Not sure if it works, but maybe —class-selector?

@cyberw
I updated:

--enable-userclass-picker -> --class-picker
User class fields are pre-selected in the UI

Looks good now! Can you just update the PR description?

Looks good now! Can you just update the PR description?

Done 👍

Thx!


Search sessions or speakers by your voice in search screen

Kind (Required)

Improvement

Overview (Required)

Title says everything!

Links

Figma

https://www.figma.com/file/4r9becvhDy3GfXaXex8E8d/App?node-id=842%3A907

Image

Can I try it?

Of course, assigned 👍