Should the BearerStrategy callback set currentUser when adding a new token to authenticatedUserTokens?

In the BearerStrategy callback, the currentUser variable is only set when the token is already found in the authenticatedUserTokens array. If this is the first time a token is seen, then it's added to the authenticatedUserTokens array -- but currentUser is not set in that case.  That causes the second parameter to the "done" method to be null the first time into the callback -- this causes the example to return an error ("authentication failed due to: error: invalid_token"). However, the second attempt will succeed because now that token is then authenticatedUserTokens list.
Is that the desired behavior, that one has to try to call the endpoint twice before authentication succeeds?  Or should the BearerStrategy callback be setting the currentUser variable inside of the "if(!userToken)" block?
Thanks!

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

ID: 8553dca0-5b9a-480d-e6a0-5051eab6d04f
Version Independent ID: da05c9f1-38a3-59b7-8364-7a42451e73ff
Content: Secure a Web API with Azure AD
Content Source: articles/active-directory/develop/quickstart-v1-nodejs-webapi.md
Service: active-directory
GitHub Login: @CelesteDG
Microsoft Alias: celested

@AE-MS
Thanks for your feedback! We will investigate and update as appropriate.

#reassign:@TylerMSFT
Opened ADO work item 795560 to track this issue. Requires feature team input. Will loop back here once that work item has been resolved.
#please-close


Add GitHub Actions for Ruff and Black, and make them green

Addresses #233
Can squash as desired and/or open issues for the different pieces of work.

ruff can do what black does, and much faster.    do we need both?

ah, i see - not identical.  thanks.
two is definitely fine - until/if/when we have a stronger opinion to collapse to just ruff.

Noting for self: Waiting for https://github.com/irods/irods_testing_environment/pull/231 before merging this.


To Reproduce
Steps to reproduce the behavior:

Download https://asciinema.org/a/Muvp1oofnYotCnXHTpDezCYtA
Use agg to render into a Gif: agg 589983.cast 589983.gif
Watch the Gif: while the ascii recording on the web correctly shows that the command takes a while to get started, this isn't reflected in the Gif

Expected behavior
For the recording as shown on the web and the rendered Gif to be somewhat similar.
Versions:

OS: Arch Linux
agg: 1.4.1

Good catch. I think we're using first print for the initial frame,  while we need initial frame to be blank.

Just released 1.4.2 with a fix for this.

Excellent, thanks! It works splendidly.


Restructuring metrics meta-data

This PR restructures the way we store metrics in the meta-data:

Moves all model accuracy metrics in a metrics key on meta-data.
Renames several metrics to improve naming consistency.
Adds new tests to enforce schema fields and keep the number of fields low
Fixes the documentation

@NicolasHug Thanks for the review. You are right it looks ugly; let's follow up on the next PR. Any pointers on where this should be fixed?


Added longPress, Right Click on Desktop.

Added longPress (or Long Click on Desktop)
Added Right Click on Desktop

Hello @aroxu !
Thanks you for this contribution
LGMT 👍🏻


Add support for UffdPrivileged memory backend type

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Current dependencies on/for this PR:

main

PR #1

PR #2   👈

This comment was auto-generated by Graphite.


Add xkd_read() + clean-up issues w/ Imports + vignettes

I initially started this PR to add a simple xkd_read() function but I noticed some notes/warnings from check() that I thought it might be helpful to address. These included:

Removed unused files and relocated data files
Add resource_files metadata to vignettes to ensure file imports work correctly
Rename vignettes to strip invalid leading numbers
Add articles section to _pkgdown.yml to order vignettes correctly
Add svgchop and gridfonts to Remotes
Add missing Imports digest and R6 + new Imports rlang + tools
Move packages from Depends to Suggests + add missing packages to Suggests
Fill in missing parameter definitions + descriptions for R6 method
Update default source to URL (to match Excalidraw JSON schema spec)

To get the xkd_read() function to optionally return an ExcaliDoc object, I added a new replace method. I also added package documentation and improved handling of the updatedate attribute in the elements component of the diagram.

Thanks! I don't use this package much, but glad to see it revisited


Bring ABC size down to 16

I wanted to cement the excellent work done by @buehmann in #7450 by bringing the bar down to 16. That way we can't have new offenses creeping in without notice.

Before submitting the PR make sure the following are checked:

[X] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[X] Squashed related commits together.
[X] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[X] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[X] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

🙇

Reduce ABC size of AbcSizeCalculator (5eb411b)

so meta


devx: add more resource links

I added sentry and along the way, updated all slack links :)

Thanks for being so responsive to comments! 😁


iTerm issues with Powerline font rendering

Tried to file an issue with iTerm but their gitlab is broken ;)
So this happened out of nowhere, no changes on zsh, no changes on the theme, no changes on the selected font:

Top Terminal.app, bottom iTerm.app
Maybe someone here has a similar experience?
Thank you!

Not sure what you're talking about with GitLab being broken to be honest.
Works here.


Adjust MAX_FILE_SIZE to 10 MiB

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ] Internal code
[x] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
This PR adjusts MAX_FILE_SIZE to 10 MB as per the new limit for users and bots: https://github.com/discord/discord-api-docs/pull/7312

While this limit is already active for users and bot users...

Why now? It's the 16th December.


SpinBox update_on_text_changed = true regression: Can't input decimals.

Tested versions

Reproducible in: v4.3.stable, v4.4.dev6 [1f47e4c4e]
Not reproducible in: v4.2.stable

System information
Godot v4.3.stable.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1080 (NVIDIA; <IP_ADDRESS>03) - Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz (8 Threads)
Issue description
Setting Update on Text Changed to true prevents decimals from being input. It seems to be similar to what happened with #81989. You can copy and paste decimal numbers into the SpinBox just fine, but trying to type a decimal will do nothing. Changing step to a decimal value < 1 still doesn't allow a decimal to be typed. This is true for both GDscript and C# in 4.3 & 4.4-dev6, but not in 4.2 (as #81989 fixed it in that version)
Steps to reproduce

Create a new scene
Add a SpinBox node
Set Update on Text Changed to true
Run the current scene
Type decimal (.) into the SpinBox

Minimal reproduction project (MRP)
N/A

Submitted a PR (linked above), that should fix this.


Add an option to automatically launch the deploy script when the installer exits

Add an option to automatically launch the deploy script when the installer exits.  This would avoid the extra step of having to go through the Start menu to launch the deployment script.
This work item was migrated from CodePlex
CodePlex work item ID: '6085'
Assigned to: 'tfabraham'
Vote count: '1'

[UnknownUser@3/2/2010]
Resolved with changeset 37623.

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]


"Force" locale in joda time common test to work with any user setting

DayOfWeekFromSundayDateTimeFieldTest can fail for some user, depending the locale settings. Basically, the tests assume the Locale is set to US.

Better alternative is to use export LANG=en_US.UTF-8 (maybe worth to document it in README.md though).


chefdk/1.3.40

announced via https://discourse.chef.io/t/chefdk-1-3-40-released/10726
final Chef 12-based Chef Development Kit

$ chef -v
Chef Development Kit Version: 1.3.40
chef-client version: 12.19.36
delivery version: master (69bfa4a76959a8d093511a90fddd7a1f7e43e354)
berks version: 5.6.4
kitchen version: 1.16.0

[x] The commit message includes the cask’s name and version.

Thank you. Please fill in the pull request template next time.


Start adding files for favicon attempt 2 #14

As on issue #14, do icons again this time with same icon as on space.artifactory.org.au page (transparent).

Simple change to icons so going to merge it myself.


Can it support on-demand loading and virtual scrolling to solve the loading and memory consumption issues of large PDF files?

Can it support on-demand loading and virtual scrolling to solve the loading and memory consumption issues of large PDF files?
Here is the reference link： https://github.com/Lemonreds/react-pdf-p2

On-demand loading requires your API supports range-request and then you can set the pdf.js's behavior using any of DocumentInitParameter in usePDF.
Regarding scrolling, VuePDF component only renders one page so you can control how many pages to render on viewport by using Intersection Observer API


NIFTI library installation

There appears to be a library installation issue with the NIFIT update. This has been detected with the SimpleITK builds against ITK master:
$ cmake -G Ninja -DCMAKE_CXX_STANDARD:STRING=11 -DITK_GIT_TAG:STRING=master ~/src/SimpleITK/SuperBuild/
$ ninja
...
[10/23] Performing configure step for 'SimpleITK'
Not searching for unused variables given on the command line.
loading initial cache file /scratch/blowekamp/build/SimpleITK-ITK/SimpleITK-build/CMakeCacheInit.txt
-- Building SimpleITK version "1.2.0rc1.post2-g254b2"
CMake Error at CMakeLists.txt:62 (find_package):
  Found package configuration file:

    /scratch/blowekamp/build/SimpleITK-ITK/ITK-prefix/lib/cmake/ITK/ITKConfig.cmake

  but it set ITK_FOUND to FALSE so package "ITK" is considered to be NOT
  FOUND.  Reason given by package:

  The following imported targets are referenced, but are missing:
  ITKNIFTI::ITKznz ITKNIFTI::ITKniftiio

-- Configuring incomplete, errors occurred!
$ ls ITK-prefix/lib/*NIFTI*
ITK-prefix/lib/libITKIONIFTI-5.0.a
[blowekamp@lhcp-rh7  SimpleITK-ITK]$ ls ITK-prefix/lib/*nif*
ITK-prefix/lib/libITKniftiio-5.0.a

I think I figured out the solution. PR to upstream nifti_clib is occuring now:  https://github.com/NIFTI-Imaging/nifti_clib/pull/36

Well done! The SimpleITK builds against ITK master are clean now.

Thanks Brad.


Delay instant DelayedLoadWrapper intersection computation

The specific case this is fixing is DLWs within FillFlowContainers, where if timeBeforeLoad = 0, the intersection would be checked upon the first Update(), but the flow layout is only updated in a parent's UpdateAfterChildren().

(Fixes hundreds of requests happening on load of multiplayer lobby)


Add transforms between NAD datums

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.92%. Comparing base (f4f4922) to head (c4f4503).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
+ Coverage   96.77%   96.92%   +0.14%
==========================================
  Files           3        3
  Lines          62       65       +3
==========================================
+ Hits           60       63       +3
  Misses          2        2

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


Add isAfterWithGranularity function

Hello,
isAfter can take a second parameter to allow granularity. https://momentjs.com/docs/#/query/is-after/
Note : I have a big diff due to refmt, I don't know why.

Hey @DCKT,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 832ff080-0f78-11e9-94bf-8f8f7f7b1f8e

Hey @DCKT,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: dff396d0-2f46-11e9-9ea4-33062c9ac3a7

Hey @DCKT,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: af82e120-2f75-11e9-8adc-611d691b9f44

Hey @DCKT,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 349cd330-87a4-11e9-a2c3-13ee52ff072b

Hey @DCKT,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 7d8af760-89c6-11e9-b893-2559eff97ce8


Update site title tag from  to

Type of change

[x] Improvement (refactoring and improving code)

Description
I would like to eliminate the use of
for site title in the sidebar. Ref #1907
Additional context
I am having problems with search engines indexing blog posts. I would like to make the
 tag unique for post titles.

Hi @kungfux,
Thanks for fixing it! I think we could just keep the a element:
-

-      {{ site.title }}
-

+    {{ site.title }}

Proposed change will have visual difference as the following on hover rule will not work:
#sidebar .sidebar-bottom #mode-toggle:hover, #sidebar .sidebar-bottom a:hover, #sidebar .site-title a:hover {
    color: var(--sidebar-active-color);
}

Yes, I overlooked that. But it's easy to recover: just delete line 749 and 754 of _sass/addon/commons.scss.
https://github.com/cotes2020/jekyll-theme-chirpy/blob/73749067c5e94849bea9cd8146ebe4965f7ed894/_sass/addon/commons.scss#L749-L754

Yes, I overlooked that. But it's easy to recover: just delete line 749 and 754 of _sass/addon/commons.scss.
https://github.com/cotes2020/jekyll-theme-chirpy/blob/73749067c5e94849bea9cd8146ebe4965f7ed894/_sass/addon/commons.scss#L749-L754

These rules seem necessary. I made changes to preserve the current look and behavior on hover. Please have a look.

These rules seem necessary.

Of course it is. What I meant earlier was:
- a { // line 749
      @extend %clickable-transition;
      @extend %sidebar-link-hover;

      color: var(--site-title-color);
- } // line 754

These rules seem necessary.

Of course it is. What I meant earlier was:
- a { // line 749
      @extend %clickable-transition;
      @extend %sidebar-link-hover;

      color: var(--site-title-color);
- } // line 754

A coder sits, eyes heavy with sleep,
Writing code in the silence, deep.
Lines of logic begin to blur,
As hidden meanings start to stir.
Between the lines, the messages hide,
But through the yawns, they're hard to guide.
He dreams of rest, but code won't wait,
Another bug to investigate.
With sleepy sighs, mistakes appear,
The code's not quite as crystal clear.
He pushes it, then starts to sway,
And drifts to sleep before the day.


Fix join online video button after remove customer feature

This PR to rework join online video button feature in ticket shop.
As before, the token to join online video using customer's id for identifier in video, but since customer feature is removed, we will using User's email or Order's email as User identifier in video.
Summary by Sourcery
Rework the 'join online video' button feature by replacing the customer identifier with the user's email or order's email. Introduce a new API view to check customer orders for an event and refactor the video plugin check for better code clarity.
Bug Fixes:

Fix the 'join online video' button functionality by using the user's email or order's email as the identifier instead of the removed customer feature.

Enhancements:

Refactor the video plugin check to improve code readability and reliability.

Merging this for further testing.


Premultiplied alpha with 'GLFW_TRANSPARENT_FRAMEBUFFER'

Using the master of GLFW using the new transparency features that will be available in 3.4 yields some issues on X11 with compositors expecting premultiplied alpha. Currently, I can work around this in a GLSL shader using fragment.rgb *= fragment.a, but idealy this should be hidden behind GLFW's window abstractions.
Simply setting glfwWindowHint(GLFW_TRANSPARENT_FRAMEBUFFER, GLFW_TRUE) and playing around with glClearColor on X11 with a compositor requiring premultiplied alpha (ie. compton) will reveal this as an issue.
For example, glClearColor(1.0, 1.0, 1.0, 0.5) displays a solid white background, where glClearColor(0.5, 0.5, 0.5, 0.5) displays the expected results (translucent white background).
System information:
$ uname -r
4.14.12-1-ARCH
$ pacman -Qs glfw-x11-git
local/glfw-x11-git 3.2.1.r514.g58cc4b2c-1
    [... omitted for brevity]
$ pacman -Qs xorg-server
local/xorg-server 1.19.6-2 (xorg)
    [... omitted for brevity]
$ glxinfo | grep OpenGL
OpenGL vendor string: NVIDIA Corporation
OpenGL renderer string: GeForce GTX 1080 Ti/PCIe/SSE2
OpenGL core profile version string: 4.5.0 NVIDIA 387.34
OpenGL core profile shading language version string: 4.50 NVIDIA
OpenGL core profile context flags: (none)
OpenGL core profile profile mask: core profile
OpenGL core profile extensions:
OpenGL version string: 4.6.0 NVIDIA 387.34
OpenGL shading language version string: 4.60 NVIDIA
OpenGL context flags: (none)
OpenGL profile mask: (none)
OpenGL extensions:
OpenGL ES profile version string: OpenGL ES 3.2 NVIDIA 387.34
OpenGL ES profile shading language version string: OpenGL ES GLSL ES 3.20
OpenGL ES profile extensions:

This is a tricky issue, and one that has multiple potential resolutions depending on the answers to several questions.

How widespread is this behaviour (compositors expecting to see premultiplied alpha in window framebuffers)? All OSs? X11 only? Only some X11 compositors? If premultiplied alpha is expected nearly everywhere (this is my guess, but needs evidence), we should probably just document that users should output in that format.

Can the compositor be interrogated at runtime to see if it expects premultiplied or non-premultiplied data? Similarly, is there a mechanism by which we can tell the compositor which format we are outputting?

I don't honestly see glfw intercepting pixel data and converting its premultiplied-ness; that's really the job of the compositor.
Any other solutions?

As far as I know, this is X11-specific for Linux and BSD.

Can the compositor be interrogated at runtime to see if it expects premultiplied or non-premultiplied data?

There's no reliable way to query if the compositor in question actually wants premulitplied alpha (although I'm pretty sure most, if not all, do).

Similarly, is there a mechanism by which we can tell the compositor which format we are outputting?

Pretty sure the answer to this is no as well, unless compositors implement their own protocols for doing so.

I don't honestly see glfw intercepting pixel data and converting its premultiplied-ness

I agree, this would be a poor solution. If none is proposed, though, the behaviour should be clearly documented. A few other solutions include:

A glfwTranslateColor inline function or macro to perform whatever computations the underlying windowing system (and compositor) requires on color components.

A window attribute that could be used with glfwGetWindowAttrib, ie. GLFW_PREMULTIPLIED_ALPHA.

To determine whether the two proposed solutions need premulitplied alpha or not, GLFW could check during initalization with something like !getenv("WAYLAND_DISPLAY") && getenv("DISPLAY") on Linux/BSD, if all X11 compositors do indeed need premultiplied alpha.

!getenv("WAYLAND_DISPLAY") && getenv("DISPLAY")

Wayland compositors expect premultiplied alpha too.
e.g. https://github.com/swaywm/wlroots/issues/984#issuecomment-390221778 — wlroots expects pre-multiplied because Weston does that, and Weston is the reference compositor.
In Vulkan, there is a mechanism to query alpha formats, and Mesa's X11 and Wayland WSI code returns VK_COMPOSITE_ALPHA_OPAQUE_BIT_KHR | VK_COMPOSITE_ALPHA_PRE_MULTIPLIED_BIT_KHR.
So seems like it's a pretty safe assumption that anything on unix platforms expects premultiplied.


Feature request: add FreeBSD support via GNOME keyring

Considering that FreeBSD has go 1.17.5, libsecret, gnome-keyring, and dbus-daemon, please consider adding support for FreeBSD

@torculus can you test it?
Do you think we just need to create a build target?

Yeah, I'm willing to test it. I'm not much of a go programmer, but I think the Linux target should work well as a template

@torculus can you try to build https://github.com/zalando/go-keyring/pull/68 on your system?
I just checked https://github.com/vmactions/freebsd-vm and it's a bit weird to run a VM inside macos and then I have to figure what ports I need to install (maybe you want to create a PR for the GH actions?).

@torculus we merged the change and it should built also on *BSD.

So, I attempted to build on my machine and it's complaining about u_int and uintptr_t being invalid types
Long story short, upsteam godbus had an issue with compiling on FreeBSD that was fixed back in 2020 (godbus version 5.0.4). Unfortunately, go-keyring is pulling godbus version 5.0.3, which is unpatched
I don't know if the requirement could be updated, but that would fix this issue

@torculus Could you try with this PR? https://github.com/zalando/go-keyring/pull/70

Note that github.com/godbus/dbus currently does not compile on freebsd/386. See https://github.com/godbus/dbus/issues/315.

Ah, so the problem is in fact that godbus/gobus relies on cgo and cgo is not easily available when cross-compiling. This may be tricky.

@torculus We have released version v0.2.0 with the update you can try!

@twpayne From where do you cross-compile? Does it not work to compile directly on the target?
We were able to add a CI step running in freebsd VM which works: #71 (I don't know too many details about *BSD so maybe it's different from what you mean?)

It does work when you compile directly, but I'm cross-compiling as I'm using GoReleaser to build chezmoi's release binaries. The GoReleaser job runs on Linux (on GitHub Actions) and cross-compiles binaries for other operating systems and doesn't support cgo.
In practice I think zalando/go-keyring has done all that it's possible to reasonably do here. In chezmoi, I've used build tags to activate the GNOME Keyring functionality when it is available. https://github.com/twpayne/chezmoi/pull/1919 contains the gory details if you're interested.

Thanks for the information.
I will close this issue as it seems like we have done what we could from go-keyring side and support FreeBSD under the right conditions


getting segmentation error isoseq3 -cluster: similar to old unresolved issue

Hello,
I got a very similar error to this unresolved thread:
https://github.com/PacificBiosciences/pbbioconda/issues/310
The post is fairly old, is there any new advice or insight with regards to this issue?
I am also running isoseq3 cluster but on version 3.4.0.
I am trying to cluster multiple cells on a 1TB RAM Redhat 7 computing node and it appears to be using around 400GB of the RAM.
It crashed after running almost 2 days.
It got through the Read, Convert, and Sort steps then does a lot of Aligning Linear steps and ends with the following output:
Aligning Linear          : (1000) 323us 719ns  (805) 21ms 883us  (68743) 44ms 286us
Aligning Linear          : 4h 1m
Read to clusters
/var/spool/slurmd/job14326978/slurm_script: line 13: 82662 Segmentation fault
Thanks!
Anthony Abrantes

Hi! Could you please tell us how many SMRT cells you've used as input, including the number of CCS reads as input to Iso-Seq? Also please add all the calls of that workflow, including the ccs calls.
Thanks you

Hi, I'm experiencing the same issue. I don't believe this is due to memory. I'm receiving this error while running isoseq3 collapse and it immediately crashes.


Docs for prefer-compact missing an example using _.compact

Reading this:
https://github.com/wix/eslint-plugin-lodash/blob/master/docs/rules/prefer-compact.md
There's no example using _.compact.

Seconded. It's not apparent to me how _.compact could be useful (or used, at all) for me here:
_.filter(arr, item => item.property === 'foo')

Is this just a false positive?  Added an example.


Depth not working on Neumorphic Button Widget

Issue: Depth parameter is not working on Neumorphic Button Widget.
Dart SDK version: 2.12.0
Flutter SDK version: 2.0.0 stable channel
Library version: 3.1.0
The depth property is working completely fine for other widgets but not with Neumorphic Button. Here's the screenshot of the app:

Here's the code of my main.dart file:
import 'package:flutter_neumorphic/flutter_neumorphic.dart';

void main() {
  runApp(NeumorphicApplication());
}

class NeumorphicApplication extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return NeumorphicApp(
      debugShowCheckedModeBanner: false,
      title: 'Neumorphic Playground',
      themeMode: ThemeMode.system,
      theme: NeumorphicThemeData(
        baseColor: Color(0xFFF2F2F2),
        lightSource: LightSource.topLeft,
        depth: 5.0
      ),
      darkTheme: NeumorphicThemeData(
        baseColor: Color(0xFF2E3237),
        lightSource: LightSource.topLeft,
        depth: 5.0
      ),
      home: NeuHomePage(),
    );
  }
}

class NeuHomePage extends StatelessWidget {

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: NeumorphicAppBar(
        title: Text('Neumorhpic Playground')
      ),
      backgroundColor: NeumorphicTheme.baseColor(context),
      body: Center(
        child: Column(
          children: [
            Neumorphic(
              style: NeumorphicStyle(
                depth: 10,
                shape: NeumorphicShape.flat,
                lightSource: LightSource.topLeft,
                border: NeumorphicBorder(width: 0.5),
                boxShape: NeumorphicBoxShape.roundRect(BorderRadius.circular(25)),
              ),
              margin: EdgeInsets.all(25),
              child: Center(
                child: Container(
                  child: Text('This is something'),
                  margin: EdgeInsets.all(125),
                ),
              )
            ),
            NeumorphicButton(
              minDistance: -5.0,
              style: NeumorphicStyle(
                depth: 20,
                shape: NeumorphicShape.flat,
                lightSource: LightSource.topLeft,
                border: NeumorphicBorder(width: 0.5),
                boxShape: NeumorphicBoxShape.roundRect(BorderRadius.all(Radius.circular(25))),
              ),
              margin: EdgeInsets.all(10),
              padding: EdgeInsets.all(18.0),
              child: Container(
                child: Icon(Icons.play_arrow),
                margin: EdgeInsets.all(15),
              ),
            )
          ],
        ),
      ),
    );
  }
}

Here's my pubspec.yaml file as well:
name: neumorphic_playground
description: A Flutter app to play with some Neumorphic Libraries

# The following line prevents the package from being accidentally published to
# pub.dev using `pub publish`. This is preferred for private packages.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev

# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# In Android, build-name is used as versionName while build-number used as versionCode.
# Read more about Android versioning at https://developer.android.com/studio/publish/versioning
# In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
# Read more about iOS versioning at
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1

environment:
  sdk: ">=2.7.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter
  cupertino_icons: ^1.0.2
  flutter_neumorphic: ^3.1.0

dev_dependencies:
  flutter_test:
    sdk: flutter

# For information on the generic Dart part of this file, see the
# following page: https://dart.dev/tools/pub/pubspec

# The following section is specific to Flutter.
flutter:

  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

It's because you haven't passed anything to the onPressed parameter for the button.
Try this
NeumorphicButton(
             minDistance: -5.0,
             style: NeumorphicStyle(
               depth: 20,
               shape: NeumorphicShape.flat,
               lightSource: LightSource.topLeft,
               border: NeumorphicBorder(width: 0.5),
               boxShape: NeumorphicBoxShape.roundRect(
                   BorderRadius.all(Radius.circular(25))),
             ),
             margin: EdgeInsets.all(10),
             padding: EdgeInsets.all(18.0),
             child: Container(
               child: Icon(Icons.play_arrow),
               margin: EdgeInsets.all(15),
             ),
             onPressed: () {},
           )```

It's because you haven't passed anything to the onPressed parameter for the button.

Oh wow, didn't knew this parameter was required. I guess a blank lambda is required for button even if you don't know what action you'll perform on onPressed. Many thanks!


chore: remove readable-stream

We can use the native stream package of node. No need to use the readable-stream one.

@wolfy1339 ptal


TIMER_Int.h Comments

You  have to write a comments for APIs of TIMER driver to help the whole team member to work without forcing us to memorize  how it work .
comment template

List of APIs
API Comment
2.1 API function description
2.2 API function input (if it need any macros you should write it)
2.3 API function output

Attached template to make our work unified
MUSART_Interface.h

DONE !


[Bug]: TypeError: (0 , o.createContext) is not a function

Telegram Application
Other
Describe the Bug
I'm getting TypeError: (0 , o.createContext) is not a function when trying to use SDKProvider context.
To Reproduce
Steps to reproduce the behavior:

Create Next.js app using npx create-next-app@latest
Install tma.js:  npm i @tma.js/sdk @tma.js/sdk-react
Add SDKProvider to root layout.tsx:

  import {SDKProvider} from<EMAIL_ADDRESS>   ...
  <SDKProvider>
    <html lang="en">
      <body className={inter.className}>{children}</body>
    </html>
   </SDKProvider>

Run npm run build

Expected Behavior
Build success.

You should not use SDKProvider in the layout component. Please, refer to the Next.js template.


Add ability to get an SVG without encoding to base64 and decoding it again.

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
When saving an SVG as a text file, given the current signature_pad API it is mandatory to encode to base64 first.
**Which versions of SignaturePad?
4.0.10
What is the expected behavior?
I want to be a little more efficient and save the step of encoding to base64 since SVG is already a text format.
I would expose a toSVG() method that returns the SVG string and refactor a bit the private _toSVG to do the base64 operations.
With jpeg and png I can skip the base64 encoding just using toBlob on the canvas.
More info https://css-tricks.com/probably-dont-base64-svg/
What do you think?

I think that would be fine to expose a toSVG() function. @szimek any thoughts?

Thanks! I'll take a look at this this weekend.


Pop up Windows sometimes aren't shown correctly

Environment
Windows build number: 21298 rsmaster
Your Distribution version: Ubuntu 20.04
Your WSLG version: 0.2.11

Steps to reproduce

Install gitkraken (I used the download link and installed the .deb https://www.gitkraken.com/download )
Run gitkraken
Open a repository by clicking 'Browse' but do not select one
Navigate away from the Window and open other Windows infront of it
Click on gitkraken again
Only the gitkraken window will pop up, and it will be non-interactable. I'd expect the modal Window to pop up too.

Expected behavior
The pop up Window should pop up in place over the git kraken Window.
Actual behavior
The pop up Window remains hidden behind all other Windows, causing the user to go find it before they can go interact with the app again.

Really hope this problem can be fixed as soon as possible🥺.

Really hope this problem can be fixed as soon as possible🥺.

Really hope this problem can be fixed as soon as possible🥺.

@hideyukn88 I'm testing the fix and now some wayland apps like epiphany browser won't show the menu although you can see it in the taskbar thumbnail and I can open and see the About dialog. With WSLg 1.0.20 it shows the menu but it doesn't show the About dialog.
sudo apt install epiphany-browser
epiphany

$ cat /mnt/wslg/versions.txt
WSLg ( x86_64 ): <current>
Mariner: VERSION="1.0.20210224"
FreeRDP: 5f083fa0b97d433d6204985f6047886e29c1c61e
weston: 0af13ff2a2d8af2f9a0fe33f6b6a01f06c20a734
pulseaudio: 2f0f0b8c3872780f15e275fc12899f4564f01bd5
mesa: 2ad0684038f5732f7e4bd1a391ec9d833685fb48

@onomatopellan , thank you very much for report, I confirmed the issue, it looks wl_subsurface is not properly handled, I will look into more details, thanks!

@JYInMyHeart , thanks for your feedback. I just submit the fix for this issue. Once the release with fix is out, please try it, thanks!

I'm testing the issue with phpStorm popup windows reported in https://github.com/microsoft/wslg/issues/96
Just updated to 21382, but the problem still exists. Is this fix included in the 21382 build?
Running  wsl --update says
Checking for updates...
No updates are available.
Kernel version: 5.10.16
Do I miss something? How should I update WSLg to run on the latest version and test this fix?
Details:
Windows build number: 10.0.21382.0
Distribution version: Ubuntu-20.04
WSLG version: 1.0.19
cat /mnt/wslg/versions.txt
WSLg ( x86_64 ): 1.0.19+Branch.main.Sha.2a47bf20c239b523d7413f315cbc9ee60dfde831
Mariner: VERSION="1.0.20210224"
FreeRDP: 5f083fa0b97d433d6204985f6047886e29c1c61e
weston: 16de531f00aa3dfd17e0de74c8f49e9fd7cec617
pulseaudio: 2f0f0b8c3872780f15e275fc12899f4564f01bd5
mesa:

@Carobit There is no need to update the Windows build. Most important part from WSLg is the System distro package that comes via Windows update. You can wait for the updates or also just create a system.vhd with the latest fixes following the guide.
Thanks to latest @hideyukn88 fixes I compiled a new system.vhd and I have no more problems in wayland apps like Epiphany-browser.
The PhpStorm issue is not fixed because it's actually a bug that also happens in LInux. See https://youtrack.jetbrains.com/issue/IDEA-265390

@onomatopellan I had seen this jetbrains issue. But in x-server such as GWSL there is no problem to use find in files.

@JYInMyHeart Interesting. Then it could also be a XWayland bug.

Maybe. Now this issue and #188 still prevent me to use wslg.

@onomatopellan , thanks for verifying the my change, very appreciate. Yes, PnpStrom's "File In Files" issue is not Z order issue, since window is closed immediately.
@Carobit , currently we haven't released the update with this fix, you will need to build system-distro VHD by yourself to try out the fix at this point, but I'm hoping this will be released soon.

Thanks for the information. For me too, this issue and the #188 is a blocker for now.. Also sometimes the main menu disappears like the find in file popup, and I noticed some minor gui glitches. Looking forward to have these issues resolved, as wslg is a pretty promising feature.

WSLg 1.0.22 was just released externally. You can pick it up by doing the following from an elevated command prompt:
wsl --shutdown
wsl --update

This contains Hideyuki's fix for the z ordering issue and should address most issues with child window poping behind their parents. I'm going to close this issue. If you are still experiencing issue with some applications, please open new issue with details of the repro so we can track those down. For phpstorm, i've reactivated #96 to track that one.


Feature requests

Hi,
Because I'm not familiar with c++.
I have some feature request:

Allow resize find out windows.
Allow toggle topmost (on-off, all window; or fast hide show all window)
Allow select multible Instruction & Hits, and copy selected to clipboard.
Keep focus selected Instruction when click Follow in CPU (I can remember last selected).
Double click on instruction to Follow in CPU.
Allow set software breakpoint from selected Instruction by F2 or context menu.
Stop and resume (instead close)
Context menu (Find out...) in References windows? (without Follow dump).
Option use Exception breakpoint instead HWBP (this will make Find out more than 4 address).

Adding to the above:

find out what accesses on RET instructions (end of a function, where it exits -- who's the caller)


Typescript definition files ignored outside of node_modules/@types

VSCode Version: 1.17
OS Version: Windows 10

Steps to Reproduce:

Move a folder from node_modules/@types to anywhere else
Try editing tsconfig fields include, files, skipLibCheck, paths, and typeRoots to match new location
Only see intellisense when folder is in ./node_modeles/@types

Reproduces without extensions: Yes

Does reloading the window help? It sounds like the changes might not have been picked up.

Seems related to my issue
Before 1.17.0 I had my setup like this:
main folder
project1
project2
project3
and VSCode was able to find the @types folder inside each of the individual projects. once I upgraded it wasn't able to find @types/jasmine anymore. installing @types/jasmine inside of the main folder fixed it

@chrmarti I didn't reload the window because vscode automatically detected when I moved the dir back into node_modules/@types. Will give it a go.
Can someone explain the interaction between vscode and tsconfig as far as .d.ts files are concerned, and better yet, point us to a repo that's working?

@matt-erhart Are you still running into this problem?
d.ts files are part of the tsconfig.json project. VSCode should handle them the same way as tsc does, but it may have gotten out of sync in this instance.

@mjbvz what's a minimal tsconfig and folder structure I could use to test this out?

@matt-erhart You should just be able to put the .d.ts files next to your .ts files. Can you share your tsconfig.json and a basic description of your project structure


Issue with '=' in album name still persists...

I have attached screenshots of the discord application, the .dat file, the DRP window and the song details shown in groove for the relevant song.
By 'remove = from the keyset' I assume it means to just write in the album name as if the = was not there. Am I misinterpreting the instruction? The album artworks for all my other songs work perfectly fine.
What DRP shows (no purple confirmation text ): )

The image uploaded to Discord Application Assets:

The album in Groove:

What I have put in the .dat file:

I believe this was fixed in 1.3.1 so try that first. And yes, that is what i mean is to just remove the = from the album title. Is there a better way i could word it that might be less confusing?

Ok, I facepalmed when I realised I was in 1.3.0.
However, even after using 1.3.1, this happened:

All the information shown by the images from the previous post is still the same.
Also, when I link DRP with Groove, or try to use Launch DRP Hidden, this happens:

And just in case, but I don't think this matters, this dialog pops up when I try to run DRP Hidden from the shortcut:

I click the open prompt and it takes me to the previous image.
I have already attempted unlinking, locating and relinking.
About the instructions for dealing with albums with equal signs, it is perfectly clear. I just started questioning whether I was interpreting it wrong when it wasn't working.

The warning prompt is standard since I don't have the resources to get a verified windows signature.
I see you downloaded the release zip so good on you, but try cloning the repo instead because that will give more background stuff (that I may need to start including in future releases i suppose)
Im curious about the error since it hasn't been reported yet im wondering if there was a special set of circumstances to cause it
I'm baffled by the vbscript not working right. If it worked before and doesnt work now then that is kinda beyond me. if needed i can offer some debug tips but in the meantime i would use the bat instead because that is tried and true. if the bat doesnt work then theres something else going on

I downloaded my own release and it worked from there no problem. Apart from deleting and reinstalling the project I need a way to replicate the issue, I'm sorry. Thank you for the good documentation of the issue tho, its just hard to fix a problem I can't make on my own computer :/

See 1.3.2 and close when you confirm the issue has been resolved @LyingJay

The issue has been resolved. But maybe change the wording of the instructions a little to make it clear which '=' should become '==', as at first, I replaced the = in the album title, then realised that I was meant to replace the other = sign.

Done.


PulsarApi.proto

Getting error with the file PulsarApi.proto:
falanx --inputfile PulsarApi.proto --outputfile Pulsar.Proto.fs --defaultnamespace Pulsar.Proto --serializer binary
Generating code for: PulsarApi.proto
ERROR: proto file was not able to be parsed.

Error in Ln: 55 Col: 11
    optional int32 partition = 3;
          ^
Expecting: whitespace

Looks nothing suspect for me on this place:
message MessageIdData {
    required uint64 ledgerId = 1;
    required uint64 entryId  = 2;
    optional int32 partition = 3 [default = -1];
    optional int32 batch_index = 4 [default = -1];
}

Haven't worked this code in forever, but I think the root cause has to do with the core parsing for messages: something was consuming a token and leaving the parser in a bad state.
Can you try this patch in Parser.fs?
        // Message statement: field | enum | etc.
        and internal pMessageStatement =
            let pMessageMessage =
                pMessageDef |>> TMessageMessage

            choice [
                (isProto2 >>. (attempt <| pGroup))
                (isProto2 >>. (attempt <| pMessageExtend))
                (isProto2 >>. (attempt <| pExtensions))
                (attempt <| pMessageOption)
                (attempt <| pMap)
                (attempt <| pReserved)
                (attempt <| pOneOf)
                (attempt <| pMessageEnum)
                (attempt <| pMessageMessage)
                pField                 // must be parsed last for keywords to not be considered type names

                ]

@vilinski The commit above was included in release 0.7.2.  Can you verify the change worked, so that this issue can be closed?

sorry, waited for falanx updates it's deps.
Falanx 0.6.0 still doesn't work with actual PulsarApi.proto with the parser bug on the same line.
❯ falanx --inputfile PulsarApi.proto --outputfile Pulsar.Proto.fs --defaultnamespace Pulsar.Proto --serializer binary

Generating code for: PulsarApi.proto
ERROR: proto file was not able to be parsed.

Error in Ln: 60 Col: 12
     optional int32 partition = 3 [default = -1];
           ^
Expecting: whitespace

But type provider works without errors.
#r "nuget: Froto.TypeProvider"

open Froto.TypeProvider

let [<Literal>] PulsarApiProto = __SOURCE_DIRECTORY__ + "/PulsarApi.proto"
type PulsarApi = ProtocolBuffersTypeProvider<PulsarApiProto>

let msg = PulsarApi.Pulsar.Proto.MessageIdData()
msg.Partition <- Some 42

so not a Froto issue anymore.
Rather falanx still does not include the changes (09.04.2020)


Fix Lid Ridge

Fix the error I made in the last revision breaking the lid ridge.  Oops.

@rosenhauer,
BUG in Lid cutouts with Depth
cutouts in the Right plane:
cutoutsRight =
[
    [90,  5, 20, 15,  4, yappRoundedRect]
   ,[15, -5, 20, 15, 10, yappCircle, 15]      //--> [A(r)] OK
   ,[40, -5, 20, 15,  5, yappRoundedRect, 15] //--> [B(r)] OK
   ,[65, -5, 20, 15,  0, yappRectangle, 15]   //--> [C(r)] OK
   ,[65, 40, 20, 15,  0, yappRectangle, 15]   //--> [K(b)] OK
   ,[40, 40, 20, 15,  5, yappRoundedRect, 15] //--> [M(b)] OK
   ,[15, 40, 20, 15, 10, yappCircle, 15]      //--> [N(b)] OK
];

Given this code in cutoutsBase:
cutoutsBase =
[
 //--  0,  1,  2,  3,  4, n
   ,[115, 95, 20, 30, 10, yappCircle, 10]       //--> [D(b)] OK!
   ,[140, 90, 20, 30,  1, yappRectangle, 10]    //--> [E(b)] OK!
   ,[165, 90, 20, 30,  3, yappRoundedRect, 10]  //--> [F(b)] OK!
];

This is the result:

cutoutsLid  =
[
 //--  0,  1,  2,  3,  4, n
   ,[165, 90, 20, 30,  3, yappRoundedRect, 10] //--> [G(L)] BUG!
   ,[140, 90, 20, 30,  1, yappRectangle, 10]   //--> [H(L)] BUG!
   ,[115, 95, 20, 30, 10, yappCircle, 10]      //--> [J(L)] BUG!
];

This is the result:


Figure out why Zeppelin skin is being cached

The cache seems to take a really long time to clear, and I don't want users to think they did something wrong if the skin isn't changing immediately when they change the web files. If possible, I would like to get the cache to clear each time Zeppelin is restarted. If not, I will add a note to the README saying that adding the skin is only for aesthetic purposes so they shouldn't worry too much if they run into this issue.

@MBoustani @BrianWilson1

The solution is to press command + shift + r to reload the cache. I will update this on the README under possible pitfalls.

@chrismattmann @valeriearoth @BrianWilson1
I was going to add html header meta to no-cach on any browser, but seems like it is already there.
Here is part of  code from a random notebook on scispark zeppelin source code:
<!-- disable caches for all browser --> <meta http-equiv="cache-control" content="max-age=0"> <meta http-equiv="cache-control" content="no-cache"> <meta http-equiv="cache-control" content="no-store"> <meta http-equiv="expires" content="0"> <meta http-equiv="expires" content="Tue, 01 Jan 1980 1:00:00 GMT"> <meta http-equiv="pragma" content="no-cache">

@chrismattmann @MBoustani @BrianWilson1
Since we do want it to cache later on, what do you think about taking that part out and just making clearing the cache when you update the skin part of the requirements (instead of just a "possible pitfall")? It would be nice to have an explanation for this though.


email send button modified

it was just a normal looking button I have applied css to it

@harshita-2003 Can u please add a screenshot so that we can review it

oh sorry I did removed my avatar from readme , do look at the changes in policy.html
EARLIER:(not matching color theory):::::::::

now :

@harshita-2003 The existing branches has conflicts through I won't be able to merge it, please pull the repo propeerly and then make necessary changes.
And let the image near table of contents be as it is, please don't alter that


Add Asciidoc lexer

Here are references from other projects based on Scintilla:
http://hg.code.sf.net/p/scintilla/scite/file/tip/src/asciidoc.properties
https://github.com/notepad-plus-plus/userDefinedLanguages/blob/master/UDLs/AsciiDocWebPage_byEduardoSantana.xml

Thanks for the suggestion :)


ODK 2.0 sync can get confounded by filenames containing #'s

Migrated to opendatakit/aggregate#81 by spacetelescope/github-issues-import
The filename is used verbatim in the URL (with slashes recognized and preserved in the REST document structure).
Unfortunately, if you have emacs open on a file while you adb-push to the device, the edit buffer will be named something like #filename.ext# for the filename.ext file you are editing.
When you then reset app server and push everything up to the server, it gets stuck trying to process that name.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain beorse.net has exceeded the max emails per hour (284/250 (113%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o6.sgmail.github.com ([<IP_ADDRESS>]:50538)
by hp159.hostpapa.com with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.89)
(envelope-from<EMAIL_ADDRESS>id 1dgMCr-000efM-VP
for<EMAIL_ADDRESS>Fri, 11 Aug 2017 22:31:18 -0400
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=p6gE6x5ECUbZZ4Z69I3olhRta2I=; b=As/UNzA48ODTZgvW
qVT5q6MedyMI+/HUc5NsG90HynAkbZWu+pzbW5lnzi+8nbvUwml064wILkzYVmNS
S5S9QekP2wdC7nkNZZ6aXi+NjGUy5dYGwsbhojjS45510WfXM54uFJ456BuXCusr
Y77Asd82JYYZW1osI9GxWXPHT3c=
Received: by filter0945p1mdw1.sendgrid.net with SMTP id filter0945p1mdw1-10894-598E684D-27
2017-08-12 02:30:37.465579907 +0000 UTC
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2b-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0006p1iad1.sendgrid.net (SG) with ESMTP id xt_xTbpXQAeb_SW3qIt58g
for<EMAIL_ADDRESS>Sat, 12 Aug 2017 02:30:37.433 +0000 (UTC)
Date: Sat, 12 Aug 2017 02:30:37 +0000 (UTC)
From: Open Data Kit<EMAIL_ADDRESS>Reply-To: opendatakit/opendatakit<EMAIL_ADDRESS>To: opendatakit/opendatakit<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [opendatakit/opendatakit] ODK 2.0 sync can get confounded by
filenames containing #'s (#1256)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_598e684d525d2_34703ff6c7903c3061955";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: opendatakit-bot
X-GitHub-Recipient: jbeorse
X-GitHub-Reason: subscribed
List-ID: opendatakit/opendatakit <opendatakit.opendatakit.github.com>
List-Archive: https://github.com/opendatakit/opendatakit
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AAUZr-NIY52El7q0_0mFImJMrMB9IT3yks5sXQ5NgaJpZM4KqGpz
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: JVRD81wLmvjBgNlAmNtibjEbEEhcPGKEESixoxP2c46l6dFQebG2f7uwZQW6x10jYch0Dl2K+B7QF9
qgcVR3Huu7lhVSSqvgDKV1fbbkCuGXMo+/BVtN0XdFBcYfyQgc4auG7olwFDXG/L6z2bWTK2BIP5Ye
R4RuE/2tf+B1Q8SlbQ5f7TEq9rWbQ8kn5Y7nIOMOyyrTvJTpBp+HOc8nJ5wro4Nh/n37k4fgzKzo+6
s=
----==_mimepart_598e684d525d2_34703ff6c7903c3061955
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #1256.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/opendatakit/opendatakit/issues/1256#event-1203484352
----==_mimepart_598e684d525d2_34703ff6c7903c3061955
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #1256.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_598e684d525d2_34703ff6c7903c3061955--

This is a 2.0 sync protocol issue more than an aggregate issue, reopening


Feature/13 detecting dead players

Coverage increased (+0.08%) to 98.05% when pulling 94f570193d6f324556b7ed6458768e298d0a37db on feature/13-detecting-dead-players into 382ab602dc268efa3817a41c8b5e70068d511afc on master.


Pull all boathouse information shown on the homepage from the database

Pull all boathouse information shown on the homepage from the database.  Up until this issue all boathouse information appearing on the homepage is hardcoded in the Python code

Submitted code for this functionality ... see pull request 86


rsa-doc: fix typo

change "d_i in RFC8017" to "d_i" in RFC8017

Checklist

[ x] documentation is added or updated

Merged to the master, 3.2, 3.1, and 3.0 branches. Thank you.


Allow parallel tooltip on bar chart

I believe this PR should solve this issue : https://github.com/imaNNeoFighT/fl_chart/issues/260
Allows tooltip to be written parallel to bar like this:

I think we must implement a way to allow users to set an angle for showing the tooltip.
That solves all problems with angle.
BTW Your solution is not working correctly.
Check image below:

We are going to solve #260 problem in #679.


Memory leak after many requests have been made

Hi,
While I was debugging a hard-to-reproduce memory leak (which turned out to be a known-and-fixed MRI bug) I noted this:

This is the number of old_objects being plotted over time, when recording requests with VCR (using webmock as the backend).
I don't have the exact script at hand, at the moment, but it was really not so fancy: recording perhaps 10-20 different URLs with VCR, running a high number of requests would trigger this. It's likely that just making 1 000 000 requests with Webmock will show a similar behavior.
Do you consider this "by design" or could it be a bug in wemock or VCR?
Thanks for a great gem!

@perlun it is "by design"
Webmock keeps RequestRegistry and stores signatures of all executed requests,
therefore the registry grows over time.
The RequestRegistry is required for matching configured expectations of executed requests.
I guess it's a rare scenario to execute 1 000 000 requests via WebMock,
and that's why nobody has complained about that before.
You can always clear the RequestRegistry with WebMock::RequestRegistry.instance.reset!

Webmock keeps RequestRegistry and stores signatures of all executed requests,
therefore the registry grows over time.

OK, that's fine and expected. But is this the case even when making a request with the exact same URL over and over again also?
(It's not a major issue for me, I'm just curious now. The root cause of my original problem has been found and I am happy. 😄)

@perlun The same url should just increase the counter, therefore I would not expect the memory issues.
In that case it's worth investigating whether it's on VCR or Webmock side.

I will not be pursuing this further, so closing. Thanks for your help @bblimke!


authenticode: Fix typo

Codecov Report

Merging #26 (43d3789) into main (0e23755) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #26   +/-   ##
=======================================
  Coverage   87.29%   87.29%
=======================================
  Files           7        7
  Lines         480      480
=======================================
  Hits          419      419
  Misses         61       61

Impacted Files
Coverage Δ

authenticode/src/signature.rs
69.91% <ø> (ø)


Expecting syntax error for expression ();

Can be reproduced on current jshint.com.

This may come as a surprise, but some of our users are likely depending on this behavior. You can read more in the proposed fix over at gh-2865. In the mean time, I am going to label this as low-priority because it represents a small edge case.
Thanks for the report!


Make the Admin Profile Dynamic

[ ]  Currently, the admin's detail is static in profile section. Make this dynamic

[ ] Also make sure to change Welcome admin to Welcome {FirstName} (as you will get name in response, break it taking space as param and log the first part)

Depends on #476

@Kajol-Kumari  assign this isuue to me

@Kajol-Kumari  what is the end point for fetch the admin profile details

@Kajol-Kumari what is the end point for fetching the admin profile details

Currently we don't have any api to fetch the detail of a particular admin. @jackfrost13 please correct me here.
But in response of login, you get the required detaile, you can store it and use it to populate the profile

@Kajol-Kumari  we can do that too :)

but I think we have to use redux state

If you need an API, let me know I can provide one

It will be better because otherwise i have to save in redux state

Sure will do it today

Hi @Kajol-Kumari , if this issue is still open can I can work on this?

Hey @akshitakanojia we are waiting for it's api inorder to integrate it so it's not open as of now

@Kajol-Kumari If this is urgent I can take up the issue and close in within a few minutes

@Kajol-Kumari If this is urgent I can take up the issue and close in within a few minutes

Hey! Nothing urgent, just this issue is dependent on it so we are looking forward to that api in order to fit this isue.

@akshitakanojia I see you already have an in-progress issue, post completion of that ping us here if needed will close the dependent issue asap

@jackfrost13 I'm done with the in-progress issue.

Can this issue be assign to me if no other is working on it? Willing to work on this one!😊

I would like to work on this issue under GSSOC`22

@shobit1337 @Gurdeep475 can anyone of you please confirm if you would like to work on this issue?

@Kajol-Kumari yes I would like to work in this.

Hello @Kajol-Kumari, I would like to work on this issue under JWOC 2023.

can i work on this issue?

@Mandar007007  please go ahead with this issue.
Happy Learning :)

reducing the level of this issue as the actual implementation i was expecting was with the api integrate and not the local storage, it will be taken care in the another opened issue, so reducing the level of this one


Fake locale in doctor_test

Description
Fake out the doctor_test locale to account for locale variances between CI hosts.
12:32 +1590 ~5 -1: test/commands.shard/hermetic/doctor_test.dart: doctor with fake validators version checking does not work
12:32 +1590 ~5 -1: test/commands.shard/hermetic/doctor_test.dart: doctor with fake validators version checking does not work [E]
  Expected: 'Doctor summary (to see all details, run flutter doctor -v):\n'
              '[!] Flutter (Channel unknown, v0.0.0, on fake OS name and version, locale en-US)\n'
              '    ✗ version error\n'
              '\n'
              '! Doctor found issues in 1 category.\n'
              ''
    Actual: 'Doctor summary (to see all details, run flutter doctor -v):\n'
              '[!] Flutter (Channel unknown, v0.0.0, on fake OS name and version, locale en_US.UTF-8)\n'
              '    ✗ version error\n'
              '\n'
              '! Doctor found issues in 1 category.\n'
              ''
     Which: is different.
            Expected: ...  locale en-US)\n     ...
              Actual: ...  locale en_US.UTF-8) ...
                                    ^
             Differ at offset 137
  package:test_api                                                         expect
  org-dartlang-app:///test/commands.shard/hermetic/doctor_test.dart 529:7  main.<fn>.<fn>
  ===== asynchronous gap ===========================
  dart:async                                                               _asyncErrorWrapperHelper
  package:flutter_tools/src/base/context.dart                              AppContext.run
  org-dartlang-app:///test/src/context.dart 120:30                         testUsingContext.<fn>.<fn>.<fn>.<fn>
  dart:async                                                               runZoned
  org-dartlang-app:///test/src/context.dart 118:18                         testUsingContext.<fn>.<fn>.<fn>
  package:flutter_tools/src/base/context.dart 146:29                       AppContext.run.<fn>

Related Issues
Introduced with https://github.com/flutter/flutter/pull/44868.

Merging on red.


CRQ-6882: Add additional 'X-Odata-TE' header that is able to come in  http2 env.

…http2 env.

Looks OK


python310Packages.wandb: disable even more tests

Description of changes
The failure is observed twice on aarch64-darwin in #234022. Reported in https://github.com/wandb/wandb/issues/5423#issuecomment-1563458692.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

arrow-cpp times out on x86_64-darwin, and it could barely be cached since Hydra is busy on staging-next.


Change the default text for User-Created Policy Rooms

Details
Copy change for default user created policy rooms.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/176855
Tests

Click the '+' icon and select New room
Create a restricted room and verify that the welcome text reads: This is the beginning of #roomname, invite others by @mentioning them
Create a private room and verify that the welcome text reads: This is the beginning of the private #roomname room, invite others by @mentioning them
Create a chat with another user you haven't chatted with before and verify that the welcome text reads: This is the beginning of your chat history with <user name>.

QA Steps
Steps above.
Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

While testing, I noticed inconsistencies with the avatar icon not showing up. Since that's rendered by another component and not part of this PR, I'd create a separate issue for that.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @Gonals in version: 1.1.29-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by @roryabraham in version: 1.1.30-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[Event Request] Codeunit::370 "Bank Acc. Reconciliation Post" - Function TransferToBankStmt - OnBeforeTransferToBankStmt

Could You please, add the following event to the codeunit 370 "Bank Acc. Reconciliation Post" to manage the Transfer to Bank Statement?
Object: codeunit 370 "Bank Acc. Reconciliation Post"
Function TransferToBankStmt(BankAccRecon: Record "Bank Acc. Reconciliation")
Publisher Event: OnBeforeTransferToBankStmt
Reason: To implement custom business logic and manage the  Transefer
Where is the event placed (see image attached)?:

At the begining of the function TransferToBankStmt(BankAccRecon: Record "Bank Acc. Reconciliation")
Insert the following 4 lines:
IsHandled := false;
OnBeforeTransferToBankStmt(BankAccRecon,IsHandled);
if IsHandled then
Exit;
Event Code:
[IntegrationEvent(false, false)]
local procedure OnBeforeTransferToBankStmt(var BankAccRecon: Record "Bank Acc. Reconciliation"; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


roadmap

do we have a roadmap for mongoengine ?

Not really we have milestones - https://github.com/hmarr/mongoengine/issues/milestones and 0.6 should be out by the end of Feb.
Probably best to discuss on the google group user list.


Calculate correct lastRecords size

Motivation:
ResourceLeakDetector records at most MAX_RECORDS+1 records
Modifications:
Increase records size after add to lastRecords
Result:
ResourceLeakDetector will record at most MAX_RECORDS records

LGTM... @Scottmitch @trustin PTAL as well

@Scottmitch happy with the change ?

@louxiu - can you sign the ICLA?

@Scottmitch done

4.1 (96e06aa74) 4.0 (e8e317bdb)


portrait support ?

Does the following work, orientation is a Phaser thing so it should be as easy as making a couple Phaser calls, I haven't tried portrait yet so I'm not 100% ? src/states/boot.ts#L34-L36

I see , thanks.
But should the caculating code here on
https://github.com/rroylance/phaser-npm-webpack-typescript-starter-project/scripts/setupGameSize.js#L34-L36L#72-L76
be different ?

Oh ya... You're totally right. There's a few places that calculate and use width and height. I'll take a look into it and see what I can do. Thanks.

Any update on this? I've been trying to set up the game size with
npm run setupGameSize -- --width 1120 --height 1920 --aspect-ratio 1.71
This does not work. Shouldn't all calculations be based on the parameters provided, as opposed to having fixed values?

Not yet. Sorry. I've had no time lately to put towards this.

Also; just realized. 1120x1920 is an aspect ratio of 0.58, not 1.71. Maybe that's why you've been having some issues? @spacedragon


update data table icons

Updates all of our datatable open and close icons to match the overall styling of the page and be more explicit to their intention.
Example:

/cc @psalajova

/lgtm


Import Error

this is my torch and cudacanvas packages installed：
torch                     2.0.1+cu118
torchaudio                2.0.2+cu118
torchvision               0.15.2+cu118
cudacanvas                1.0.1.post212118

after pip install cudacanvas, i tried to import cudacanvas but got:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "D:\Software\programme\anaconda3\envs\pytorch_g_2.0.1\lib\site-packages\cudacanvas\__init__.py", line 13, in <module>
    from .cudaGLStream import CudaGLStreamer
ImportError: DLL load failed while importing cudaGLStream: The specified module could not be found.

Is this beacause of the 2.0.1 version of torch?

I just uploaded one for 2.0.1 try running the command below, should fix your issue
pip install cudacanvas==1.0.1.post201118 --force-reinstall

I just uploaded one for 2.0.1 try running the command below, should fix your issue
pip install cudacanvas==1.0.1.post201118 --force-reinstall

Thanks! It works now!


Fix fullmock integration with javamail 1.6.2

Closes #4

I know, this project is dead. I've just created the PR for people having the same problem as me ;-)


why did camera controls stop working properly when the script is connected to a broadcast

w h y
https://github.com/user-attachments/assets/dd8ad525-655b-4486-8a7d-5250ee8d93cb

you added no wait it all happens at the same time..

dude but it was working just fine befor and i didnt edit a single thing in my project i came back to it and the camera was acting super wack and the sprites were glitching and everything so i reloaded the page and then it wasnt as bad but then it just became this

oh the bug magically fixed itself  w h a t

wait a minute it doesnt happen on the beta servers but happens on the normal one
w     h     a     t

wait a minute it doesnt happen on the beta servers but happens on the normal one
w h a t

thats not a weird thing

unless thats jot the issue

w h y
2024-10-28.19-26-58.mp4

are all the sprites your trying to move bound to the same camera being changed

if sprites are trying to move while the camera is being moved?

oh yeah ca you test this again
it should be fixed now

ill checc tomorow

it still happen

the thing is u see how the arrows slightly zoom when i click the script

but then when i put it in a message and run it it zooms out so much for zoom reason

and its a whole new broadcast i made so no other sprites could be running something so idk what it is

i dont know

https://c.tenor.com/-CfhczC_cREAAAAd/tenor.gif


[S3] Parse AWS configuration with support for profile section

Hello :wave:!
I've faced a minor problem on my setup when I have a profile section on the configuration file [1] (valid with aws tools). In such case the netcdf library fails to load the AWS config, not critical but not obvious.
This is a quick fix I did for my myself, but looking at the parser and aws config specification I see parsing problems with:

Token delimiter characters in the values
Support for comments
Multiline service configuration

I'm not sure to which extend you want to support loading the AWS config when the building with the internal S3 implementation.
Thanks!
[1]
~/.aws/credentials
[default]
aws_access_key_id=123
aws_secret_access_key=456

[user1]
aws_access_key_id=123
aws_secret_access_key=789

~/.aws/config
[default]
region=us-west-2
output=json

[profile user1]
region=us-east-1
output=text

Let me make sure I understand.

When the profile name is specified as e.g "[user1]", the file (either credentials and config) parse correctly.
When the profile name is specified as e.g "[profile user1]", the file (either credentials and config) does not parse.

Have I stated the problem correctly?

Exactly!

Just checking in @DennisHeimbigner and @mannreis in regards to the latest changes requests.  It looks like @mannreis made some changes, how does this look now @DennisHeimbigner ?


[SPARK-32922][SHUFFLE][CORE] Adds support for executors to fetch local and remote merged shuffle data

What changes were proposed in this pull request?
This is the shuffle fetch side change where executors can fetch local/remote push-merged shuffle data from shuffle services. This is needed for push-based shuffle - SPIP SPARK-30602.
The change adds support to the ShuffleBlockFetchIterator to fetch push-merged block meta and shuffle chunks from local and remote ESS. If the fetch of any of these fails, then the iterator fallsback to fetch the original shuffle blocks that belonged to the push-merged block.
Why are the changes needed?
These changes are needed for push-based shuffle. Refer to the SPIP in SPARK-30602.
Does this PR introduce any user-facing change?
When push-based shuffle is turned on then that will fetch push-merged blocks from the remote shuffle service. The client logs will indicate this.
How was this patch tested?
Added unit tests.
The reference PR with the consolidated changes covering the complete implementation is also provided in SPARK-30602.
We have already verified the functionality and the improved performance as documented in the SPIP doc.
Lead-authored-by: Chandni Singh<EMAIL_ADDRESS>Co-authored-by: Min Shen<EMAIL_ADDRESS>Co-authored-by: Ye Zhou<EMAIL_ADDRESS>

Adding @Victsm @mridulm @zhouyejoe

Can one of the admins verify this patch?

Will resolve conflicts when this PR is merged as.

I have rebased the changes against the latest master. It only depends on https://github.com/apache/spark/pull/32140 but it interfaces at just 1 place with that PR so it can still be reviewed in parallel.
Given the size of this change and the feature freeze in early July, would really appreciate if folks can help with the review.
@mridulm @Victsm @Ngone51 @tgravescs @attilapiros

Gentle ping to help review this PR @tgravescs @attilapiros @Ngone51 @mridulm @Victsm @zhouyejoe

Took an initial pass, yet to look at ShuffleBlockFetcherIterator or test suites.
I am wondering, given the volume, whether we want to split between ESS side and client side. Thoughts ?

Thanks Mridul for reviewing!
My thoughts about splitting this change is that it completely encapsulates the fetch-side changes so it is easier to understand how the new messages introduced on the client side are being handled on the server-side. One of the feedbacks we got last year was that we broke things up in a way that made it difficult to understand.
That being said, I am still okay to break this change into client/sever PRs if that makes the review easier for the reviewers.
cc. @mridulm @Ngone51 @Victsm @tgravescs @attilapiros

@otterc Given the volume of the PR, does it cleanly separate out into ESS side and client side ?
If it does, we can merge the former first and then the latter.
If not, let us keep it as is.

@otterc Given the volume of the PR, does it cleanly separate out into ESS side and client side ?
If it does, we can merge the former first and then the latter.
If not, let us keep it as is.

Yes @mridulm. It will cleanly separate out into ESS side and client side. I will work on splitting the PR

Created SPARK-35671. The PR is https://github.com/apache/spark/pull/32811.
This PR will depend on the changes in SPARK-35671.
cc. @mridulm @Ngone51 @Victsm @tgravescs @attilapiros

Removed all the changes from here which are now part of SPARK-35671

This is still dependent on the changes in https://github.com/apache/spark/pull/32140 which has the protocol side of changes.

This is #32140 @otterc, you meant #32811 :-)
I did not check the PR id and followed the last comment twice :-)

This PR is not a WIP any more. @mridulm @Ngone51 I have addressed all the review comments so far. PTAL.

@otterc could you update the PR description? Looks like it's outdated.

Looks good to me, will wait for @Ngone51 to finish his review. Thanks for the changes and clarifications @otterc !

Jenkins, test this please

The github actions test failure looks unrelated, let me try jenkins anyway

+CC @Ngone51 Any further comments/suggestions ?
I was planning to merge this tomorrow, thanks !

@otterc Can you fix the conflict please?

@otterc Can you also please see if #33109 is relevant to push based shuffle ? Thx

@mridulm Resolved the conflict.

Sorry for the delay. I'll do a review today. BTW, are there any other necessary mgnet PRs that have to be merged for the 3.2 release?

Sorry for the delay. I'll do a review today. BTW, are there any other necessary magnet PRs that have to be merged for the 3.2 cut/release?

There are 2 pending tasks which are necessary for Magnet:

SPARK-35546: #33078. This one is ready for review
SPARK-32923: #33034. This is WIP

The test failures are unrelated.

I left some minor comments. I think we're ready to merge after addressing these comments.

Test build #140388 has started for PR 32140 at commit ad89a02.

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

Thanks for working on this @otterc !
Thanks for the detailed reviews @Ngone51 :-)

Merged to master

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

Thanks @mridulm and @Ngone51 for the thorough reviews!


Apply the ESlint rule to 'gulpfile.babel.js'

I fix various ESLint errors in the file.
(camelcase, semicolon, comma dangle, brace style, indentation)

Just curious: What tool did you use? Also, our lint test is not checking for gulpfile.babel.js?

Yes, 'gulpfile.babel.js' did not check for itself. because it checked only certain .js files in 'server' folder.
so i added './gulpfile.babel.js' on gulp.src (task of 'lint_server').
And I checked this file with ESlinter of the Atom at first.
Actually. I think the result is the same as checking with the gulp-eslint module in our gulpfile.babel.js.


Opacity for legacy subTool GUI

UI opacity slider does not work for legacySubTools,
eliminated possible memory leaks

it is still inconsistent imho. the speed limit signs in the UI should be same transparent as the main TMPE UI buttons.

additional the lane connectors are not getting transparent.

same for junction restrictions

And... sliders should be at 50% in the middle....

can you give an example of legacy subtool that is affected by this?

is this back-end only?

can you give an example of legacy subtool that is affected by this?

Basically any tool with OnGUI UI (does not have LegacySubTool base), e.g. traffic lights tool, vehicle restrictions window, etc.

I tested this and confirm that it works.

@chameleon-tbn I addressed all requested changes (skipped speed limits tool, because it will be replaced anyways).
If you are edit lane connections, transparency will be ignored for active node, same for junction restrictions otherwise it should take the setting into account 😉

Over all it looks OK. Overlay option activated all is faded. But for non-active junction if lane connector is selected, the transparency is not working

Faster than Flash our @krzychu124


Fully connected layer dimensions for GNN (p. 650)

Hi Sebastian,
There is a very useful picture on the page 650 with network layers description for GNN.
Probably that for fully connected layer the product W_fc (4x4) with h_fc (1x4) must be in swapped order (h_fc x W_fc) to get the same dimension as h_fc (1x4)?
Thank you.

Just updated this, thanks!


NPE in SwaggerConverter (Issue 732)

Hi team,
This PR is to solve the null pointer exception in swagger converter as mentioned in the issue #732
Please review the PR and merge the same.
Thanks,
Mohammed

Great work!

Thank you!


Meeting time

Implement meeting time
-Update MeetingDate to store meeting date as variable
-Add date and time to meeting
-Make meeting time optional
-Update meeting date constraints and documentation
-Update test util for meeting date
-Update UG example and add command example message
Separate Meeting class will be implemented when more requirements of meeting arise

There still seems to be some checkstyle errors in your pull request. Do fix it before merging!


Add CI setup, like other repositories

Done in https://github.com/open-contracting/pelican-frontend/commit/93bfdb41fc7ececb4481a54164be88f3adda03d4#diff-107e910e9f2ebfb9a741fa10b2aa7100cc1fc4f5f3aca2dfe78b905cbd73c0d2 and other commits.


Add missing dependency on joblib

Following your example from README.md I get:
>>> subject = genomicranges.read_ucsc(genome="hg38")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/foo/genomicranges/lib/python3.11/site-packages/genomicranges/io/ucsc.py", line 61, in read_ucsc
    data = parse_gtf(path, compressed=compressed)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/foo/genomicranges/lib/python3.11/site-packages/genomicranges/io/gtf.py", line 58, in parse_gtf
    from joblib import Parallel, delayed
ModuleNotFoundError: No module named 'joblib'
>>>

The package specified additional requirements that installs these optional packages.
pip install genomicranges[optional]

I'll add this to the documentation. Thank you for raising this issue.

The package specified additional requirements that installs these optional packages.
pip install genomicranges[optional]

Maybe I missed that, I just did pip install genomicranges per my shell history. I thought it will install all deps but this [optional] is fine with me.


Please support dynamic AuthConfig (needed to pull sandbox from Amazon ECR)

The static AuthConfig added in PR https://github.com/containerd/cri/pull/838 doesn't work well for Amazon ECR, which uses a short-lived token for registry authentication. We pull all containers from ECR to reduce the impact of bad network weather, so this missing feature is currently a blocker for migrating from dockershim to containerd-cri.
Docker's credential configuration supports running a separate helper binary, using the wrapper code in github.com/docker/docker-credential-helpers/client. This works pretty well, and the main cloud vendors have created drop-in credential helpers such as amazon-ecr-credential-helper.
Something like this would be perfect:
[plugins.cri.registry.auths]
  [plugins.cri.registry.auths."https://gcr.io"]
    dockerCredentialHelper = "/usr/local/bin/docker-credential-ecr-login"

Can you use this? https://kubernetes.io/docs/concepts/containers/images/#using-aws-ec2-container-registry

I think we prefer doing this at Kubernetes cluster level.

The Kubernetes level provides credentials for pulling Pod images, but these credentials are not used by containerd when it pulls the sandbox image.
We'd also like Kubelet to function in standalone mode, so we can use it to schedule the control plane.
Finally, the future roadmap for Kubernetes has the ECR credential fetch moving from the Kubelet (--cloud=provider=aws) into the control plane (--cloud-provider=external), so a standalone Kubelet would need to fully rely on containerd for ECR pull credentials.

@jmillikin-stripe Hm, I think this is a reasonable use case. We should think about how to support this use case.

@jmillikin-stripe Hey! Feel free to submit a change if you have bandwidth for this. I think this is a reasonable feature request. :)

We've worked around it using the ctr tool for now, so fixing this is more of a P3 feature request than a P1. If I can get the time later this year, I'll send in a PR.

This is useful, and not complex. Let's do it.

Hm, we recently hit a similar issue.
However, after some investigation, I found it is relatively simple to script out this:
$ echo "gcr.io" | /usr/bin/docker-credential-gcr get
{"ServerURL":"","Username":"aaa","Secret":"bbbbbb"}

$ crictl pull --creds aaa:bbb gcr.io/google-containers/busybox:latest

Given that this is not hard to make it work, I'm leaning toward not adding the docker specific logic into containerd.

In most cases, this should be handled by Kubernetes;
In some corner cases, such as bootstrapping or standalone kubelet, it is also easy to work around with the solution above.

@Random-Liu k3s is embedding containerd. While crictl is available and could be used as you described, it would be nice to be able to simply configure credential helpers that were used as necessary, rather than having to script image pulls ahead of time.

I think we want to be careful with this one and probably implement it not as CRI specific way or trying to re-use docker credential helpers. The credential helper model doesn't fit well here since it assumes a pre-fetch of the credentials and per-user calling from a client.  I've been thinking about this during the registry configuration updates I am making, but I can't see a good way to fetch credentials securely through a simple configuration file. Making a plugin interface may end up being best here.

I don't think it assumes prefetch and a per-user model. That may be the case for developer workflow but it's just as important on headless nodes, where it functions behind the scenes using cloud provider instance credentials to authenticate to that same providers secure registries. If you do make it a plug-in, there's going to be just one plug-in that interfaces to the Docker helper API since that's what everyone is already using.

We are facing same issue while using kubeadm upgrade, as containerd don't allow dynamic auth token - we are stuck while pulling images. Kubeadm upgrade is not entertaining images pulled from cri.

I think this is probably not going to be addressed in containerd, in favor of having the kubelet handle it: https://kubernetes.io/docs/tasks/kubelet-credential-provider/kubelet-credential-provider/


Click to Add Styling bug when hovering over Data Explorer/functions

Click to Add text is condensed and goes beyond the line height. Need clarity on whether this is the intended functionality or whether the helper text should be modified to fit the screen (i.e. be spliced rather than wrap, increase the line height whenever a text wraps, etc...)

@russorat what should be the best way to handle this moving forward?

Maybe it should just me a plus icon? or the word Add.

no longer an issue


resolve getInitialNotification for local notifications with notification schema

Summary:
Changelog: [Internal]
In the past, local notifications were stored in the launch options as UILocalNotification, whereas remote notifications were stored as NSDict. This means that we had different handling in PushNotificationIOS.js. This condition was not handling the differences in local and remote notifications that JS expected, which I'm fixing here.
Reviewed By: ingridwang
Differential Revision: D53734086

2740491e32f3c04cf270993d5e7dc6541f67e55d


Accessibility changes

This pull request contains accessibility checks for people with disabilities and people with reduced visual and auditory performances due to old age.
A couple of checks were conducted and errors were fixed according to the guidelines from the Easy Check - A First Review Of Web Accessibility article published by W3C.
Here 👇 is the report from the checks which were carried out;

Page Titles - No Issues Found

Image Text Alternative - Text alternatives for images at the header and social media sections were found to have accessibility errors and these errors were fixed in these lines of code.

This pull request contains accessibility checks for people with disabilities and people with reduced visual and auditory performances due to old age.
A couple of checks were conducted and errors were fixed according to the guidelines from the Easy Check - A First Review Of Web Accessibility article published by W3C.
Here 👇 is the report from the checks which were carried out;

Page Titles - No Issues Found
Image Text Alternative - Text alternatives for images at the header and social media sections were found to have accessibility errors and these errors were fixed in these lines of code.


Compilation failed after downloading the project

update library/build.gradle
classpath 'com.novoda:bintray-release:0.4.0' to classpath 'com.novoda:bintray-release:0.8.0'

facing this issue:
A problem occurred configuring project ':library'.
> Failed to notify project evaluation listener.
   > org/gradle/api/internal/component/Usage


Added few additional tests for prompting

Based on discussion in #268
Not able to run these tests on Ubuntu 18.04 with dotnet 3.1.201 in Rider 2020.1

@drewburlingame I don't see tests related to PrompterTests running at all. Do I miss something?

these scenarios are already covered in PromptForMissingArgumentTests so I'll close this PR


Bump supported Streamlit versions

This PR bumps the list of supported Streamlit versions to v1.10.0

Tweet: https://twitter.com/charlesbben/status/1534944089803829248?s=20&t=FNqW3ZMWUvyRRs9A1cSb9Q
Internal Slack thread: https://huggingface.slack.com/archives/C02136Y252P/p1654759216920019

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


[Feature Request] Optional firearms proficiencies

It would be nice to make firearms an available option for proficiencies. One of my players is an artificer, and they use firearms frequently. Here's a list of all the currently available types of firearms in D&D I've found so far.

From what I remember, the Exandria items aren't SRD, theyre from a Critical Role supplement to D&D Beyond

From what I remember, the Exandria items aren't SRD, theyre from a Critical Role supplement to D&D Beyond
Ah, right. In that instance, perhaps just the non-exandrian ones.

We should be able to add the DMG ones without trouble since they're just labels.

Sounds good to me!

Duplicate of #1327


WIP: release 2.4.0

How on earth did I overlook that in #98 ? 🤦♂

please do not merge yet, I want to double check the remaining build warnings


Allow forcing of multi line cases

for example
val decl = symbol.scope match {
case Local  => genLocalVariableDeclaration(symbol)
}
should become
val decl = symbol.scope match {
case Local  =>
genLocalVariableDeclaration(symbol)
}
This makes setting breakpoints in the eclipse debugger much simpler

Scalariform does not prevent writing the code as desired
val decl = symbol.scope match {
  case Local =>
    genLocalVariableDeclaration(symbol)
}


Follow up case Token list to buy is empty

Describe the task
Follow up case Token list to buy is empty
Actual: Token list to buy is empty => User do not select token -> button Buy is disable => User can not buy token

Same as https://github.com/Koniverse/SubWallet-Extension/issues/2322 => Close this issue


chore: reorganized message component guides and localization guide

This problem didn't spot my eye until after I merged #59. I should probably pay more attention next time.

I don’t think we need a separated category for app commands imo

I think it's reasonable. Plus, message components are in their own folder, so it wouldn't really make any sense to just leave Application Command guides in the main folder.

I think it's reasonable. Plus, message components are in their own folder, so it wouldn't really make any sense to just leave Application Command guides in the main folder.

But message components and app commands are nothing alike

I think it's reasonable. Plus, message components are in their own folder, so it wouldn't really make any sense to just leave Application Command guides in the main folder.

But message components and app commands are nothing alike

Ummmmm... what does that have to do with this discussion? I'm trying to say since Message Components have their own folder (and you aren't complaining), why not give Application Commands their own folder?


[positioner] Adjust Window Position Offsets

When using the Position::TrayCenter enum, I've noticed that the window opens extremely close to the system tray — there's practically no gap at all. For aesthetic and usability purposes, this is less than ideal.
I believe there isn't a way to adjust the position offsets, which would allow for a more customizable experience based on user or application needs. Thus, I would like to request the feature of adjustable window position offsets for the tauri-plugin-positioner. This could potentially be a configuration option that allows us to define additional pixel offsets (x and y) from the position provided by the Position:: enum.

@rp1231 Did you enable the system-tray feature in Cargo.toml?

I meant on the positioner plugin. If you also did that, please open a new issue, preferibly with a reproduction example but at least the output of the tauri info command, the dependency section from Cargo.toml, and some code of what you've tried.

@0Ky What operating system are you on?

Windows 11.

I'm not able to call any of the tray variants from rust on windows at all. Am able to call the non tray variants like Center just fine.

Are you following the correct usage guide?
Cargo.toml:
tauri = { version = "1.5.2", features = ["api-all", "system-tray"] }
tauri-plugin-positioner = { version = "1.0.4", features = ["system-tray"] }

main.rs:
Initialize the plugin:
use tauri_plugin_positioner::{WindowExt, Position};

fn main() {
    tauri::Builder::default()
        // include these
        .plugin(tauri_plugin_positioner::init())
        .on_system_tray_event(|app, event| {
           tauri_plugin_positioner::on_tray_event(app, &event);
        })
        .run(tauri::generate_context!())
        .expect("error while running tauri application");
}

Change position with adjustable offset values:
let window = app.get_window("main").unwrap();
let _ = window.move_window(Position::TrayCenter);
let current_position = window.outer_position().unwrap();
let offset_position = PhysicalPosition {
    x: current_position.x - 18,
    y: current_position.y - 12,
};
let _ = window.set_position(tauri::Position::Physical(offset_position));

Thanks for chiming in! Just wanted to add that the important thing here is the on_system_tray_event thing. There has to be one tray event before the first move_window call so that the plugin knows where the tray is. (In other words, if you call move_window before the first tray event you will get an error)

Thanks for chiming in! Just wanted to add that the important thing here is the on_system_tray_event thing. There has to be one tray event before the first move_window call so that the plugin knows where the tray is. (In other words, if you call move_window before the first tray event you will get an error)

Does not actually errors but panics. There should be a way to gracefully handle that and fallback to another position.

@truchi Feel free to open a new issue about this :)


Allow to assign a custom class to the monitor container

Hello! I'm using this useful monitor but I have a minor style problem. In the app I'm working on checkboxes have a custom style to be more nice to the user's eye, but that don't fit well with the monitor checkboxes.
I'm wondering if it would be a problem to assign a custom className to the monitor so I can reset checkbox's css in my stylesheet only for the monitor?
Thanks for your time!

Hi @ingro,
I'm glad you're finding the monitor useful. :)
I'm a bit reluctant to add much in the way of style overrides to the monitor since it's really only intended as a dev tool and not something end-users are going to be seeing.
That being said, I think you should be able to accomplish what you're looking for with the following:
Assuming you're doing something similar to the demo, then wrap your DevTools component when you put it into the DOM:
<div className={styles.DevToolsWrapper}>
  <DevTools/>
</div>

And then you can customize all checkbox inputs within the wrapped monitor like so:
.DevToolsWrapper input[type=checkbox] {
  /* Your custom styles here... */
}

Would this be an acceptable solution for you?


Request for build against metrics-core 3.0.1

Looking for a build against metrics-core 3.0.1 (scala 2.11) - the most recent stable release. 3.1.0 has some issues with graphite:
https://github.com/dropwizard/metrics/issues/694

Correction, 3.1.0 is the current stable release, so perhaps a build with 3.0.2 re-release:
https://github.com/dropwizard/metrics/releases/tag/v3.0.2-dropwizard
Or include the patch with 3.1.0:
https://github.com/dropwizard/metrics/issues/694

Disregard this request, I just saw a more comprehensive version/build matrix:
https://github.com/erikvanoosten/metrics-scala/blob/master/docs/AvailableVersions.md

Glad it resolved itself :)


Add detailed direction mode

InfoHUD's coordinate overlay is very helpful while flying around with elytra, but I've still found myself pressing F3 to view my pitch and yaw when land isn't in sight. These changes split the direction display into its own mode and add a more detailed setting which shows the player's yaw and pitch in the same format as the debug screen. This also allows the direction display to be turned off entirely.
Here's a screenshot with the new mode enabled:

Hey!
I'm glad this plugin has been helpful to you and am grateful that you are showing interest in adding features to it :smiley:. However, I won't merge your PR for now. InfoHUD was created to be a plugin version of the CoordinatesHUD datapack without too many extra features added. I believe the feature may be a bit too advanced and may not have high interest. I for one had some trouble understanding what the numbers meant for a few moments.
I am aware that the same argument can be made about the CurrentTick and VillagerSchedule time modes and I do indeed regret going overboard in the past and implementing wild features that I must keep supporting into the future for those rare users that enabled them. Well, can't change the past I guess :disappointed:.
Your idea and implementation are good. If you wish, feel free to fork this project and release your own plugin (perhaps named some twist on InfoHUD) to gauge demand. If you wish to do so, here are some issues/tips I found:

If coordinates are disabled with everything else enabled, "N"/"NE"/"S"/"E"/"W"/... are still displayed. This might be intentional, however they are still left-padded, sometimes leaving an oddly large gap between the cardinal direction and the time. This is purely an aesthetic thing.
You may want to offset the yaw numbers. Most games I've seen that have a direction metre on top of the screen (like Apex or Fortnite) tend to set north as 0/360 degrees, then do a 360 degrees circle rather than [-180 to 180].
Perhaps invert the pitch numbers. Seems more intuitive to have looking up be positive and looking down negative.

Once again, thanks for your interest and hard work 🫡,
RoverIsADog

Understandable. I agree that this is a specialized feature and might be outside the scope of your plugin. No harm in maintaining a fork for my own use.
Regarding your feedback:

That is intentional. Frankly I'm not sure I understand the point of padding between the direction and time, but I chose to keep it to avoid upsetting current users. I'd prefer a header and single space separator between all fields myself.
I had the same thought, however I chose to keep the values identical to the debug screen as I didn't want to make the info provided "better" than vanilla, only cleaner.
Same as above.

Thanks for the quick review and response!


an error occurs when load the model "mobilenet-ssd"

I downloaded the model "mobilenet-ssd" with the downloader.py, then I got a file named "ssd_mobilenet_v2_coco.frozen.pb".
I try to load this model with command "net = cv2.dnn.readNetFromTensorflow("ssd_mobilenet_v2_coco.frozen.pb")", then I got following error: "cv2.error: OpenCV(3.4.4) C:\projects\opencv-python\opencv\modules\dnn\src\tensorflow\tf_importer.cpp:497: error: (-2:Unspecified error) Input layer not found: Postprocessor/BatchMultiClassNonMaxSuppression/map/while/NextIteration in function 'cv::dnn::experimental_dnn_34_v10::`anonymous-namespace'::TFImporter::connect'"
Anyone know what's going on here? And how to fix this?

@mazhongzhong, How this problem is connected with repository with models? Is ssd_mobilenet_v2_coco from Open Model Zoo repository?
Try to look at https://github.com/opencv/opencv/wiki/TensorFlow-Object-Detection-API

Let's assume that this is solved. Please, reopen if it's still relevant.