Sorbet does not understand Ruby 2.6's endless range syntax

Input
→ View on sorbet.run
if (3..).include?(x)
  'yes'
else
  'no'
end

Observed output
editor.rb:1: Parse Error: unexpected token: syntax error https://srb.help/2001
     1 |if (3..).include?(x)

Expected behavior
The code should at least not throw a syntax error, since this is valid ruby 2.6

It looks like endless ranges (2..) don't work.  If ruby 2.6, where they were introduced, isn't officially supported, it might be good to document that somewhere.  Or, if that's already document somewhere and I missed it, ignore me!  :)

What's the version of sorbet's parser/lexer? Is it supposed to handle the latest version ruby? All of them? (the latter is impossible)
I see that lexer.rl has a bunch of ruby-version-specific checks, but there's only 1 grammar file.
I could help with that

More errors:
meth[] {} # valid starting from 2.5
class A; return; end # invalid starting from 2.5
m [] {} # invalid starting from 2.5

begin; 1; else; 2; end # invalid starting from 2.6
-> do rescue; end # valid starting from 2.6

however, it handles some new features of 2.5 (like meth do; foo; rescue; bar; end), so I guess current state of parser/lexer is somewhere between 2.5 and 2.6.

You're right. The state of the version of the parser/lexer used to be "Whatever Stripe uses" and now is "kind of a hybrid". If you want to tackle figuring out what the story is for multiple version support, please do, we haven't really put much thought into it other than kicking it down the road.
If you don't want to shave that yak, we will happily take patches for backwards compatible syntax changes.
Also for context, the original parser was a line-by-line port of the whitequark parser into C++. So keeping it close to that would be helpful so we can incorporate changes from them. (AKA don't go refactor the whole thing for fun please)

Fair enough!
I imagine that multiple-version support is going to be a thing we'll want to deal with sooner than later.  We can make backwards-compatible fixes to the parser to handle syntax, but we'll still get issues with the stdlib.  Eg Integer.sqrt(n) fails type checking because that method was introduced in 2.5.  Likewise, RubyVM::Env fails because that constant was removed in 2.4.  We could probably rely on syntax additions usually being additive, but that doesn't seem to be the case for stdlib changes.

+1 for implementing a superset of syntax all supported versions of Ruby, unless it makes it much harder.
Overall, we haven't found a problem of using unsupported syntax of Ruby to be a widespred issue so it doesn't seem that maintaining multiple parsers for different versions would be worth it.

@ptarjan @DarkDimius Is there any way to expose parser/lexer and reuse whitequark/parser's test suite? (similar to https://github.com/opal/c_lexer) I've started backporting missing updates and corresponding tests but it's very painful

@iliabylich How would you want it exposed? I pulled in the test suite maybe a year ago here: https://github.com/sorbet/sorbet/tree/master/test/testdata/parser/whitequark . You can use -p options to srb tc to print out various parts of the pipeline (and that is what is in those .exp files)

Yes, that's what I did locally as well (and it kinda works). It is possible to write tests this way, but it's very hard to debug it when things go wrong. whitequark/parser exposes more internal things after shifting each token (like lexer's state, values of cmdarg/cond/context stacks etc).
Also, every time when I did some changes to lexer.rl I had to recompile a lot of files

How would you want it exposed?

Is it possible to compile an isolated parser that takes ruby code via stdin and prints AST? We could try monkeypatching assert_parses to do something like Open3.capture3(parser_executable, 'ruby-code') under the hood (existing specs don't very complex things like source maps anyway, so we only need the structure of the AST). Maybe this executable could also dump it into something like json?

You're welcome to send us PRs for those output formats of capturing more
would be helpful. They are internally used so can be changed. So if you
want them to be real JSON you can do that too.
As for the executale
srb tc -p parse-tree-raw -e
Might be close to what you want?
On Mon, Jun 24, 2019 at 4:16 PM Ilya Bylich<EMAIL_ADDRESS>wrote:

Yes, that's what I did locally as well (and it kinda works). It is
possible to write tests this way, but it's very hard to debug it when
things go wrong. whitequark/parser exposes more internal things after
shifting each token (like lexer's state, values of cmdarg/cond/context
stacks etc).
Also, every time when I did some changes to lexer.rl I had to recompile a
lot of files
How would you want it exposed?
Is it possible to compile an isolated parser that takes ruby code via
stdin and prints AST? We could try monkeypatching assert_parses to do
something like Open3.capture3(parser_executable, 'ruby-code') under the
hood (existing specs don't very complex things like source maps anyway, so
we only need the structure of the AST). Maybe this executable could also
dump it into something like json?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sorbet/sorbet/issues/972?email_source=notifications&email_token=AAAJZT4O2UJR5DY4GFX5GX3P4FIVZA5CNFSM4H2RKR3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYOPZCQ#issuecomment-505216138,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAJZTYP5OBODHEZJ7XY5CDP4FIVZANCNFSM4H2RKR3A
.

It's parse-tree-json, thanks, I'll try it!

Quick question on endless ranges: .rbi files don't support it. Range<TBegin, TEnd> returns TBegin from #begin method and TEnd from #end method. Should we update them? Or should we introduce more types for literal-based ranges?

1..2 always has .begin and .end
a..b may return nil from both methods.
<whatever>.. always returns nil from .end

I'll send a PR with updates to AST, should be straightforward after merging https://github.com/sorbet/sorbet/pull/1670, but I don't know what do with types


Adapter Terminated by "Power Off"

If you turn off the TV, the adapter terminates with the following error message.
The adapter will then restart, so not a serious error, but you might want to check it out anyway.
Thank you.
adapter: 2.0.0
admin: 6.8.2
js-controller: 5.0.11
node: 18.17.0

2023-08-18 13:25:09.015  - info: host.Rpi4 instance system.adapter.netatmo-crawler.0 terminated with code 11 (ADAPTER_REQUESTED_TERMINATION)
2023-08-18 13:26:30.403  - error:  panasonic-viera.1 (22000) Unhandled promise rejection. This error  originated either by throwing inside of an async function without a  catch block, or by rejecting a promise which was not handled with  .catch().
2023-08-18 13:26:30.406  - error: panasonic-viera.1 (22000) unhandled promise rejection: connect ECONNREFUSED <IP_ADDRESS>:55000
2023-08-18 13:26:30.407  - error: panasonic-viera.1 (22000) Error: connect ECONNREFUSED <IP_ADDRESS>:55000
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1495:16)
2023-08-18 13:26:30.409  - error: panasonic-viera.1 (22000) Exception-Code: ECONNREFUSED: connect ECONNREFUSED <IP_ADDRESS>:55000
2023-08-18 13:26:30.967  - info: panasonic-viera.1 (22000) terminating
2023-08-18 13:26:30.968  - warn: panasonic-viera.1 (22000) Terminated (UNCAUGHT_EXCEPTION): Without reason
2023-08-18 13:26:31.620  - error: host.Rpi4 instance system.adapter.panasonic-viera.1 terminated with code 6 (UNCAUGHT_EXCEPTION)
2023-08-18 13:26:31.620  - info: host.Rpi4 Restart adapter system.adapter.panasonic-viera.1 because enabled
2023-08-18 13:27:01.726  - info: host.Rpi4 instance system.adapter.panasonic-viera.1 started with pid 22617
2023-08-18 13:27:07.227  - info:  panasonic-viera.1 (22617) starting. Version 2.0.0 in  /opt/iobroker/node_modules/iobroker.panasonic-viera, node: v18.17.0,  js-controller: 5.0.11

Sorry
I have added above


Identify when input is already a torrent

Don't create a torrent file from something that is already a torrent file.
This makes it so users of WebTorrent can create their own torrent files and seed them without WebTorrent creating a torrent of the torrent file passed in

It should be possible to create a torrent file that contains a torrent file. This isn't the right place to solve the problem you're experiencing.

Yeah, I closed the PR because of that and because WebTorrent.seed does more inside the create-torrent callback anyway

Cool, glad you got it figured out 👍


Support custom screen resolution via define settings when using SDL

define M5GFX_WINDOW_WIDTH and M5GFX_WINDOW_HEIGHT to set screen resolution.
M5GFX_WINDOW_WIDTH and M5GFX_WINDOW_HEIGHT need to be declared at the same time.

What made you need this feature?

I am creating an LVGL simulator based on M5GFX and adding a shell image. Since the sizes of different core device are different, I need to set different window sizes. It will be more convenient to set different simulator window sizes through build_flag.
Demo:

build flag:

I understood the purpose.
I'm guessing that your program is setting a special offset on the drawing area to draw the shell image.
I think when the user uses the drawing function, I need to behave in a way that doesn't cause them to draw outside the picture frame.
What you really need is the ability to set the frame, not the width, right?

Yes, the window size and the area rendered by lvgl are implemented through offsets.
The size of the window is set based on the size of the background image. Because I did not find a suitable API to change the default window size, so I added two macro definitions to change the default window size setting by use build_flag...
Core2 backgroud image:

CoreS3 backgroud image:

Rendering and drawing offset processing:

Is your wish as follows?

Add new flag to M5GFX build flags to display shell case frames.
If the display flag of the picture frame is disabled, it will be displayed as before.
If the display flag of the picture frame is enabled, a dedicated shell case PNG image for each model will be displayed.
Users cannot interfere with picture frames from M5GFX's drawing function.

Yes!
It would be best if the user's operations could be limited to the actual display area of the core device (such as the red box part), but the current implementation cannot achieve this.

It would be better if the area pointed by the arrow could be made transparent! However, an irregular shell like CoreS3 is a bit difficult to handle.

It is probably desirable to add new functionality to Panel_SDL. I'm sure SDL can be layered.
In other words, the shell case image is placed in a separate layer from the memory that the user manipulates with drawing functions, and overlapped during SDL rendering.
That way, it would be possible to reproduce a circular display like the M5Dial.

I will try making a prototype later, so please provide image data of the shell case.

Sure!
Core2

Size: 401 x 401 px
Display area: (41, 72) - (361, 312)
X offset: 41
Y offset: 72

CoreS3

Size: 401 x 501 px
Display area: (41, 123) - (361, 363)
X offset: 41
Y offset: 123

I checked the image, and it looks like the size within the frame is not exactly 320x240, but has an error of 1 pixel. It may be better to correct the error and adjust the size outside the frame to 400x400.

OK, I will ask our art designer check this.

Sure!
Core2

Size: 400 x 400 px
Display area: (40, 72) - (360, 312)
X offset: 40
Y offset: 72

CoreS3

Size: 400 x 500 px
Display area: (40, 122) - (360, 362)
X offset: 40
Y offset: 122

Updated, please download the image again.

I think...

This image was modified by me. I think the top and bottom margins of the actual machine are equal, so I think 80 is preferable instead of 72.

Yes, I asked the hardware engineer and they told me it was equal. I asked the art designer to make changes.
Here is the latest images:
Core
Fire
Core2
CoreS3

Thanks @lovyan03 support, this feature is supported now, so i close this.


Mention IOPS in deploy docs

@echohack

Merging per LGTM over Slack

Coming in here after the party a bit but this looks great! Thanks @msteffen !


Setup Persistent DB types

@wz1000 do you want to add the following to this issue, or treat the separately:

Enum support
JSONB support
Array support
Building out audit logs?

@saurabhnanda What do you mean by Enum support and Array support?
Also, why is JSONB support needed? Won't we be working with Haskell types and only convert to/from JSON while sending/recieving data?

What do you mean by Enum support and Array support?

Figuring out how Persistent can handle PG ENUMs and arrays. Check out the schema at https://github.com/vacationlabs/haskell-webapps/blob/master/ServantOpaleye/db/schema.sql to understand use-cases.

Also, why is JSONB support needed? Won't we be working with Haskell types and only convert to/from JSON while sending/recieving data?

JSONB is a datatype supported by PG. It is extremely useful in storing data that is structured, but the schema may not be well-known ahead of time. Possible use cases:

storing the change-set in audit_logs.changes column
storing user-defined product properties as general key-value pairs in products.properties column

Also, what about housekeeping columns, like created_at and updated_at, and how to deal with them in Persistent?

Figuring out how Persistent can handle PG ENUMs and arrays

AFAIK, Persistent automatically converts Haskell Enums to PG enums. However, it doesn't support PG arrays.

Also, what about housekeeping columns, like created_at and updated_at, and how to deal with them in Persistent?

They'll have to be dealt with manually, possibly with helper functions that take care of updating them.

However, it doesn't support PG arrays.

Do you want to spend time looking around some (unpublished) projects that solve this problem?

They'll have to be dealt with manually, possibly with helper functions that take care of updating them.

Here's a heads-up about this problem (because I've already spent some time with it):

Persistent doesn't have a good way of dealing with non-nullable columns that have a default value specified by the DB
If you make created_at and updated_at as NOT NULL then you have to always call getCurrentTime and specify those fields in the record before shipping them off to the respective Persistent function.
This probably needs some Template Haskell to make it more palatable to use.

Do you want to spend time looking around some (unpublished) projects that solve this problem?

I think I can just work around it for now(one possible solution is using a bitfield with some lenses to replicate a [Bool]). If needed we can solve this later.

If you make created_at and updated_at as NOT NULL then you have to always call getCurrentTime and specify those fields in the record before shipping them off to the respective Persistent function.

This can be solved by defining custom update/insert functions that automatically manage those fields.

@wz1000 has this been closed by #30


Flex Objects blueprint in theme's blueprints/flex-objects not showing in Admin sidebar

Hello,
as of #3255, blueprints appear in Flex Object Plugin and are confirmed to work at engine level, but are not displayed in the Admin sidebar and adding/editing objects is no more possible.
Thank you!

https://github.com/trilbymedia/grav-plugin-flex-objects/commit/785eb663b642261e305e73816ca0e400b9376f94


Bootstrap sliders not working - displaying as block instead of columns

Hello,
I am using Bootscore with Woocommerce theme on my website, I am trying to add a simple product slider based on bootstrap and ACF fields.
I can not make it work, even though it works on my other websites, using different themes. In fact, I have tried many different bootstrap sliders approaches and none of it works – everything gets displayed in one column, underneath each item, I have tried these two codes for example:
https://codepen.io/mephysto/pen/ZYVKRY
https://www.jqueryscript.net/slider/multi-item-carousel-bootstrap.html
and this is the desired code, I have been successfully using on other sites:

if( $featured_posts ): ?>

// Setup this post for WP functions (variable must be named $post).
setup_postdata($post); ?>

" .$tag->name. "";
}
}
?>

get_price_html()?>

Previous

Next

Can I get some help with this one, please?
Am I missing something?

Yes, that‘s because Bootstrap‘s carousel component is not able to show columns by default. Getting your Codepen examples working in a WP loop with Bootstrap carousel is quiet hard.
For this reason we‘ve created the bS Swiper plugin https://bootscore.me/documentation/bs-swiper/ which is able to show WooCommerce products or any other custom post types.
Use this plugin with a simple shortcode and show your customers products in a slideshow.
BTW I converted this issue into a discussion, because this isn‘t a theme bug.
Does this help?


Push To Master: Add data management scripts

PR adds 2 scripts

data download bash script
jupyter notebook of how the data was processed

Will be taking a look after lab meeting today. Just a heads up for future PR. Please convert jupyter notebooks to executable scripts and upload the converted script as well. It makes reviewing notebooks a bit easier. Conversion constructions here.

Please convert jupyter notebooks to executable scripts and upload the converted script as well. It makes reviewing notebooks a bit easier.

Agreed. Added in 0a33b69

Thanks @danich1 and @jaclyn-taroni - I believe I addressed your comments. Updates are ready for re-review


Add optional AST node type

The proposed OptionalAstNode<T> is a generic type that sets all properties, that could be potentially missing due to parser errors, to be optional. However, this excludes specificially:

Properties inherited from AstNode (mainly the $type property which is always set).
All boolean properties. Those will be set to false by the parser, even if the property wasn't parsed.
All array property. Those will be set to [] by the parser.

This change adjusts the arithmetics language validator to use that new type to ensure that all properties are properly checked for truthyness before they are accessed. Inspired by the discussion in https://github.com/eclipse-langium/langium/discussions/1758.

@sailingKieler @sorawee Making AST nodes deep optional, makes them very difficult to deal with. While technically correct on a type level, you cannot pass them into any method expecting a "normal" AstNode, due to type mismatch. This makes validations/scoping suddenly very difficult to write. I'll probably get back to this in the new year.


Update grpc-netty to 1.56.0

About this PR
📦 Updates io.grpc:grpc-netty from 1.55.1 to 1.56.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.grpc", artifactId = "grpc-netty" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.grpc", artifactId = "grpc-netty" }
}]

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

Codecov Report

Merging #2600 (e31cd9f) into main (18d6e2d) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2600      +/-   ##
==========================================
- Coverage   78.46%   78.43%   -0.03%
==========================================
  Files         445      445
  Lines       15639    15639
  Branches     2523     2523
==========================================
- Hits        12271    12267       -4
- Misses       3368     3372       +4

see 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove route from response errors

Motivation
The response_errors also included the route label, which can lead to a lots of data. This is pretty similar to #65
What
Removes the route from the response_errors. Also, I've updated the readme to align with the current responses.
Why
Avoid creating an excessive number of metrics.
Verification Steps
This part of the code is covered by test.
Checklist:

[x] Code has been tested locally by PR requester
[ ] Changes have been successfully verified by another team member

Progress

[x] Finished task
[ ] TODO

good catch, thanks @davidporos92
Will verify and merge asap


"madb shell" command

adb shell is such a popular command, and most users expect that there would be madb shell.

Done by 0fdb1db


Console log in nitro plugin breaks the server

Environment
Latest Nuxt 3.11.0
Reproduction

Describe the bug
Using console log of any type breaks server in server plugins
Additional context
No response
Logs
No response

Should be the same as mine https://github.com/nuxt/nuxt/issues/26329

Yes, this is already resolved and should be released in a patch later today.


SelectionArea causes inconsistent layout between web and other platforms

SelectionArea shrinks to the child size on the web while it expands to the parent size on other platforms. It lacks consistency. Tiny differences would be acceptable, but this bug causes a much bigger difference as shown below.
I see the issue regardless of the renderer type on the web.

Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const App());
}

class App extends StatelessWidget {
  const App();

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: Scaffold(
        body: SizedBox.square(
          dimension: 200,
          child: SelectionArea(
            child: Text('text'),
          ),
        ),
      ),
    );
  }
}

Windows
Web

Another example:

Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const App());
}

class App extends StatelessWidget {
  const App();

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: SizedBox.expand(
          child: SelectionArea(
            child: Scrollbar(
              thumbVisibility: true,
              trackVisibility: true,
              child: SingleChildScrollView(
                primary: true,
                padding: const EdgeInsets.all(16.0),
                child: Text('text\n' * 100),
              ),
            ),
          ),
        ),
      ),
    );
  }
}

Windows
Web

Steps to Reproduce
Just run each sample above on web and other platforms.

flutter doctor -v
[√] Flutter (Channel stable, 3.7.3, on Microsoft Windows [Version 10.0.22000.1574], locale ja-JP)
    • Flutter version 3.7.3 on channel stable at D:\xxxxx\flutter\sdk
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (10 days ago), 2023-02-08 15:46:04 -0800
    • Engine revision 248290d6d5
    • Dart version 2.19.2
    • DevTools version 2.20.1

[X] Windows Version (Unable to confirm if installed Windows version is 10 or greater)

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at D:\xxxxx\android\sdk
    • Platform android-33, build-tools 33.0.1
    • ANDROID_HOME = D:\xxxxx\android\sdk
    • Java binary at: C:\Program Files\JetBrains\IntelliJ IDEA 2021.3.3\jbr\bin\java
    • Java version OpenJDK Runtime Environment JBR-17.0.5+1-653.25-jcef (build 17.0.5+1-b653.25)
    • All Android licenses accepted.

[X] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.3.1)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.3.32811.315
    • Windows 10 SDK version 10.0.19041.0

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

[√] IntelliJ IDEA Ultimate Edition (version 2022.3)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2021.3.3
    • Flutter plugin version 72.1.4
    • Dart plugin version 223.8617.8

[√] VS Code (version 1.75.1)
    • VS Code at C:\Users\xxxxx\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.58.0

[√] Connected device (2 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22000.1574]
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 110.0.1587.49

[√] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 3 categories.

Triage report
I can reproduce this issue on Master (3.8.0-14.0.pre.2). The behaviour in web is consistent with both HTML and CanvasKit renderers

Code Sample (Same as OP)
import 'package:flutter/material.dart';

void main() {
  runApp(const App());
}

class App extends StatelessWidget {
  const App({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: Scaffold(
        body: SizedBox.square(
          dimension: 200,
          child: SelectionArea(
            child: Text('text'),
          ),
        ),
      ),
    );
  }
}

Screenshot

flutter doctor -v (Master)
[!] Flutter (Channel master, 3.8.0-14.0.pre.2, on Ubuntu 22.04.2 LTS 5.15.0-60-generic, locale en_US.UTF-8)
    • Flutter version 3.8.0-14.0.pre.2 on channel master at /home/exabyt3/fvm/versions/master
    ! Warning: `dart` on your path resolves to /usr/lib/dart/bin/dart, which is not inside your current Flutter SDK checkout at /home/exabyt3/fvm/versions/master. Consider adding /home/exabyt3/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision cc9b198213 (50 minutes ago), 2023-02-20 00:49:25 -0500
    • Engine revision 780279109e
    • Dart version 3.0.0 (build 3.0.0-257.0.dev)
    • DevTools version 2.21.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /home/exabyt3/Android/Sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/linux#android-setup for more details.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /usr/bin/brave-browser

[✓] Linux toolchain - develop for Linux desktop
    • Ubuntu clang version 14.0.0-1ubuntu1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2022.1)
    • Android Studio at /home/exabyt3/programs/android-studio
    • Flutter plugin version 72.1.2
    • Dart plugin version 221.6096
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.75.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.58.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.2 LTS 5.15.0-60-generic
    • Chrome (web)    • chrome • web-javascript • Brave Browser <IP_ADDRESS>

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

cc @chunhtai

Removing the web label for now just so this issue goes to the framework triage. There doesn't seem to be anything to be done in the web engine. There's a conditional import in the framework that does different things depending on the platform, which is what probably causes the discrepancy, but the fix is likely fully on the framework side.


Is support npm install?

how can i use npm to install this?

+1 but I think this project is died...


conflict warning where no conflict is

i have two packages from aur installed: ethereum-git and mist, they are not in conflict with each other. but somehow i get the following warning:
 ~ $ pacaur -Syua
:: Synchronizing package databases...
 core is up to date
 extra is up to date
 community is up to date
 multilib is up to date
:: Starting AUR upgrade...
:: haskell-deepseq-generics is not present in AUR -- skipping
:: mate-dialogs is not present in AUR -- skipping
:: mate-themes-extras is not present in AUR -- skipping
:: powerdown-git is not present in AUR -- skipping
:: slack-chat is not present in AUR -- skipping
:: uvcvideo-dkms is not present in AUR -- skipping
:: vala-012 is not present in AUR -- skipping
:: resolving dependencies...
:: looking for inter-conflicts...
:: ethereum-git and ethereum-git
mist are in conflict (ethereum). Remove ethereum-git
mist? [y/N] n
:: unresolvable package conflicts detected
:: failed to prepare transaction (conflicting dependencies)
:: ethereum-git and ethereum-git
mist are in conflict

note the linebreak before mist. something strange is going on. i wonder why this happens.
here is some more detailed output: https://gist.github.com/5chdn/6c457f2ef6bf18fe690d
please let me know if you need more details.

hitting Y allows me to upgrade with the following error tho:
:: resolving dependencies...
:: looking for inter-conflicts...
:: ethereum-git and ethereum-git
mist are in conflict (ethereum). Remove ethereum-git
mist? [y/N] y
grep<EMAIL_ADDRESS>No such file or directory

Indeed weird. Thanks for the report, I'll have a look as soon as possible.

Can you post your pacman -Qm output? There is something picky in that list:
 uvcvideo-dkms v8-3.15 vala-012

Not sure where that uvcvideo-dkms comes from, but it's version string isn't parsed correctly. Also, this is most likely unrelated, but the package vala-012 doesn't exist in AUR anymore, I think it has been superseded by vala0.12.

 ~ $ pacman -Qm
android-file-transfer 2.4-1
argtable 2.13-1
caja-dropbox 1.12.0-1
cloog 0.18.4-1
cower 14-2
dropbox 3.12.6-1
ethereum-git 1.0rc2.r106.gb315d82-2
google-chrome 47.0.2526.111-1
gtk-engine-unico 1.0.2-6
haskell-deepseq-generics <IP_ADDRESS>-2
hplip-plugin 3.15.11-1
isl 0.15-1
jdownloader2 latest-7
lib32-libpng12 1.2.56-1
libjson-rpc-cpp-git 20150829-1
libpng12 1.2.56-1
mate-dialogs 1.8.0-2
mate-themes-extras 3.16.3-2
meteor-js 1.2.1-3
mist 0.3.7-2
pacaur 4.4.4-1
pgmodeler 0.8.2_alpha1-1
powerdown-git 149.d834996-1
pybitmessage 0.4.4-2
pydio-sync 1.0.2-1
python2-potr 1.0.1-3
python2-socks 1.5.6-1
qgis 2.12.2-1
qwtpolar 1.1.r182-1
secp256k1-git 20151210-1
slack-chat 1.2.2-2
spatialindex 1.8.5-1
spideroak-one 6.1-1
sublime-text-dev 3.3083-3
teamviewer 11.0.52520-1.5
telegram-desktop-bin 0.9.18-1
tor-browser-en 5.0.7-1
transifex-client 0.10-2
uvcvideo-dkms r104-4
v8-3.15 3.15.11-1
vala-012 0.12.1-3
vidalia 0.2.21-4

I was wrong and assumed v8-3.15 was a version string, but it is really a package name. After installing ethereum-git and mist, I am able to reproduce the issue when simulating a ethereum-git update.

This issue is due to to the ethereum-git and mist packages both being part of the ethereum group, while the ethereum string is also a conflicting package.
As a result, the expac -Qs '%n' ^ethereum$ check not only returns packages and providers of "ethereum", but also all installed packages being part of the "ethereum" group.
Since the group field is rarely provided in PKGBUILD in the AUR, that issue has stayed undetected until now.
As of now, I am not sure how to fix that issue properly. Looking at the behavior of pacman, expac behavior seems to be correct. Also, all expac calls should be carefully checked to ensure this bug doesn't occur in other part of the code.

This particular issue has now been fixed in 7e799ad554a37048f65efe94dd7fce12550d6f6e (in pacaur-git).
The code still need to be adjusted to ensure the very same issue doesn't happen with version string and maybe when checking for VCS clients too, but for now this should be enough to get you out of that issue. Thanks again for reporting, that one was a surprisingly weird bug!

cheer :smile:

Fixing this for  -Qs '%v' isn't trivial, but on the 3 locations using it, one is entirely safe and the two other are non critical (use for display only). The location using -Qs '%n' to ensure the VCS program is installed is also safe, since this concern only 4 or 5 packages in the whole repositories. No action needed, closing.

Seems there might be a regression introduced by this change:
With pacaur on the system, installing pacaur-git doesn't trigger the conflict check anymore. Reopening.

Fixed in a009b75758fb721636476b36c9e122ba97a2b05f.


CI needs more instrumentation

Describe the bug
CI is getting flaky
+ ./hack/cluster-deploy.sh
waiting for cluster-clean to finish
+ source cluster-up/common.sh
++ set -e
++ '[' kubernetes = kind ']'
Deploying manifests...
+ CLUSTER_PROVIDER=kubernetes
+ MANIFESTS_OUT_DIR=_output/manifests/kubernetes/generated
+ main pipefail
+ '[' '!' -d _output/manifests/kubernetes/generated ']'
+ echo 'Deploying manifests...'
+ kubectl apply -f _output/manifests/kubernetes/generated
namespace/kepler created
clusterrole.rbac.authorization.k8s.io/kepler-clusterrole created
clusterrolebinding.rbac.authorization.k8s.io/kepler-clusterrole-binding created
serviceaccount/kepler-sa created
daemonset.apps/kepler-exporter created
service/kepler-exporter created
servicemonitor.monitoring.coreos.com/kepler-exporter created
+ kubectl rollout status daemonset kepler-exporter -n kepler --timeout 60s
Waiting for daemon set "kepler-exporter" rollout to finish: 0 of 1 updated pods are available...
error: timed out waiting for the condition
make: *** [Makefile:165: cluster-sync] Error 1
Error: Process completed with exit code 2.

To Reproduce
Recent CI results here
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

https://github.com/sustainable-computing-io/kepler/actions/runs/3073381944/jobs/4965474260#step:8:233
I see latest CI passed, @rootfs  is this issue able to reproduce easily?

let's keep an eye on it and re-open this issue if it happens again.

Maybe we need a larger timeout?
Let's see if it happen again

https://github.com/sustainable-computing-io/kepler/pull/231#issuecomment-1254422128

Thanks @SamYuan1990!
What we need is the artifacts https://docs.github.com/en/actions/using-workflows/storing-workflow-data-as-artifacts
So we will dump the logs and description of the pods to files in the artifact folder
Something like:
kubectl describe pod -n $1 $2 > $artifacts/describe/pod/$2
kubectl describe logs -n $1 $2 > $artifacts/logs/pod/$2

But, to save space we just dump the info when something fail

Thanks @SamYuan1990!
What we need is the artifacts https://docs.github.com/en/actions/using-workflows/storing-workflow-data-as-artifacts
So we will dump the logs and description of the pods to files in the artifact folder Something like:
kubectl describe pod -n $1 $2 > $artifacts/describe/pod/$2
kubectl logs -n $1 $2 > $artifacts/logs/pod/$2

But, to save space we just dump the info when something fail

any suggestion for fail handle?
I suppose our points is
when fail happnes.
kubectl describe pod -n $1 $2 > $artifacts/describe/pod/$2
kubectl logs -n $1 $2 > $artifacts/logs/pod/$2

@SamYuan1990 something like:
function logdump {
   kubectl get po -A -o custom-columns=namespace:.metadata.namespace,name:.metadata.name | tail +2 | while read -r line ; do
     line=($line)
     namespace=${line[0]}
     podname=${line[1]}

     mkdir -p $artifacts/describe/pod/$namespace
     mkdir -p$artifacts/logs/pod/$namespace

     kubectl describe pod -n $namespace $podname > $artifacts/describe/pod/$namespace/$podname
     kubectl logs -n $namespace $podname > $artifacts/logs/pod/$namespace/$podname
   done

}

trap 'logdump' ERR

@rootfs ,is this issue reproduced after we fixed panic issue?

@SamYuan1990 we are not seeing it recently.


Sometimes pugixml fails to find xpath values

I have the following code to extract data from an XML document using XPATH:
pugi::xml_document doc;

size_t length = strlen(message_text);
pugi::xml_parse_result result = doc.load_buffer_inplace(message_text, length);

pugi::xpath_node_set node_set = doc.select_nodes(xpath);
pugi::xpath_node_set::const_iterator it;
for (it = node_set.begin(); it != node_set.end(); ++it)
{
    pugi::xpath_node node = *it;
    strncpy(value, node.node().text().get(), size);
    value[size] = '\0';

...
Sometimes this works fine and sometimes the contents of 'value' is an empty string even if
the document contains data in the element.  How can I troubleshoot this?  I have not used
pugixml before and I am on a very tight deadline.

You can locate the node without the text using node.node().offset_debug() - this is the byte offset from the beginning of the file where the node is

You can check node.node().child_value() (should be the same but who knows)

You're using load_buffer_inplace; make sure message_text is not being modified after load_buffer_inplace or reused - load_buffer_inplace is a destructive operation. Check if load_buffer works instead.

If you attach the document and/or XPath query you're using I may have more ideas. If you don't want to make them public feel free to send them to me by e-mail<EMAIL_ADDRESS>

Thanks for such a quick response.  Using 'load_buffer' instead seems to have fixed the problem.  However I am certain that the buffer is not modified until after all the pugixml methods have been called.

Can you provide a more complete version of the code (including details such as where the value is defined, how message_text is allocated & destroyed etc.)?

Both value & message_text are local variables of the caller to the parser function, both char[].  The prototype for the parser function is:
extern "C" {
int   iso20022_get_field(const char *xpath,
char *value, char *message_text,
int size);
}
Prior to me using the load_buffer_inplace, message_text was "const char*"

The only way I can imagine the code you posted malfunctioning is if [value, value + size] and [message_text, message_text + strlen(message_text)) overlap. If they do then strncpy will fill part of message_text with zeroes, which will essentially make some nodes in the document lose their names & contents.
If the above is true it's likely that the node you find will be very unusual - this:
 printf("%d <%s>\n", node.node().type(), node.node().name());

Would print:
2 <>

Also note that the code value[size] = 0 is suspicious - from the declaration of the function it seems that value is a buffer with size bytes, not size+1?

There is definitely no overlap.  And value is defined as [size+1]

I don't really know at this point how to make progress on this. I still suspect an application bug but I can't be sure.
Can you provide the source for iso20022_get_field function, the source for the function that calls it, an example XPath expression and an example XML where you could see the issue? (I understand that the issue is intermittent - this does not matter). Feel free to e-mail them to arseny.kapoulkine at gmail dot com if you don't want to attach them to the issue.

Wait, this is obvious. I'm not sure how I missed this.
I was assuming that the code checks the XML parse result but I don't see this in the code. If you call iso20022_get_field twice on the same buffer, the first call will destructively modify the buffer if you use load_buffer_inplace, and the second call will get a buffer that likely has the first tag abruptly ending, e.g. "<foo" - in which case the parsing will fail, and XPath query will not find anything.

That must be it.

Hello , Any API that i can use to find the node (anywhere in xml) without xpath?


Move InvalidPixelHandler from CameraCalibration to ImageProcessor?

Brought up in a discussion with @aleberti in this issue here: #2331
We added the InvalidPixelHandler as last step of CameraCalibrator when doing the dl0 → dl1 step.
That prevents applying it on input data that does not contain DL0 (e.g. DL1 images loaded with the HDF5EventSource or a custom source as is the case for the MAGIC plugin).
I think we could move the InvalidPixelHandler application from the CameraCalibrator to the ImageProcessor to enable applying it also on DL1 input.
@kosack what do you think?

Yes, it makes sense to have it in ImageProcessor, since that is generally where you want to avoid invalid pixels, so ok by me.

I tried to implement this today, but ran into some conceptual issues:

The broken pixels are per channel information
selected_gain_channel is only stored at dl0, not at dl1
The ImageProcessor currently works without needing dl0 information, it is purely dl1
Files containing only dl1 will not contain this information, do they currently contain the broken pixel monitoring data

These are the reasons, why the current InvalidPixelHandler is somewhat tied to the DL0 → DL1 step and is thus in CameraCalibrator.
Also: is_valid is currently a single bool, it should probably be a boolean mask for each pixel, same shape as image itself.


Shopify Connector Sales invoice not stamped on Shopify order

I created from a dev store a simple order with 1 item and it has shippng.
order comes into BC correctly.
this is for a location that is advanced warehouse.
ship the item via the warehouse shipment.
back on the sales order you have item line shipped but not the transport.
ship from SO the g/; transport line.
then set the qty to invoice on both lines and invoice them.
sales order invoices however the sales invoice number does not get stamped on the shopify order.
this also makes the order not fulfulled.

Thank you for reaching out.
If you are Microsoft partner, you can submit support request on behalf of your Business Central online customers. Start at the Business Central administration center where you can easily submit a support request in the Power Platform admin center.
To submit support requests on behalf of your customer, you must be a delegated admin on the customer tenant. Your company must also have the Advanced or Premier support plan.
The described cases though look like expected behavior:

The Sales Invoice No. field in the Shopify Sales Orders page is used when an imported Shopify Order is already fulfilled. For example, it contains only gift card, then in Business Central instead of Sales Order, system creates Sales Invoice. For more information, see https://docs.microsoft.com/en-us/dynamics365/business-central/shopify/synchronize-orders#to-create-sales-documents. It doesn't contain link to posted sales invoice.

To send information about fulfilment to Shopify you need to run the Sync Shipments to Shopify task. For more informations, see https://docs.microsoft.com/en-us/dynamics365/business-central/shopify/synchronize-orders#synchronize-shipments-to-shopify.

Kind regards,
Andrei Panko
PM, Microsoft

Thank you for reaching out.
If you are Microsoft partner, you can submit support request on behalf of your Business Central online customers. Start at the Business Central administration center where you can easily submit a support request in the Power Platform admin center.
To submit support requests on behalf of your customer, you must be a delegated admin on the customer tenant. Your company must also have the Advanced or Premier support plan.
The described cases though look like expected behavior:

The Sales Invoice No. field in the Shopify Sales Orders page is used when an imported Shopify Order is already fulfilled. For example, it contains only gift card, then in Business Central instead of Sales Order, system creates Sales Invoice. For more information, see https://docs.microsoft.com/en-us/dynamics365/business-central/shopify/synchronize-orders#to-create-sales-documents. It doesn't contain link to posted sales invoice.

To send information about fulfilment to Shopify you need to run the Sync Shipments to Shopify task. For more informations, see https://docs.microsoft.com/en-us/dynamics365/business-central/shopify/synchronize-orders#synchronize-shipments-to-shopify.

Kind regards,
Andrei Panko
PM, Microsoft

thank you for the reply.


Documentation using deprecated service rawgit

Seeing this with an old version of firefox, newer versions or Safari don't show the issue. The iframes show text like this:
https://rawgit.com/#19410754303789723334
"RawGit is now in a sunset phase and will soon shut down. It's been a fun five years, but all things must end."

Filed an issue: https://github.com/dabbott/react-native-web-player/issues/39


Basics  concept and Lasagna exercise

The Lasagna concept exercise was created by @jonmcalder : PR #202, dated January 2022. This remains unmerged as of April 2023 and is copied here for completeness.
This PR adds documentation to the exercise and adds the closely-related basics concept.
This double PR is a special case - I won't repeat this with others.

CI failures seem to be because armstrong-numbers is incompatible with 3.6.3 (use of pipes). No obvious issues with this PR, I think?
@jonmcalder previously wondered out loud about wheher it's time to drop 3.6.3 support. Not my decision, but it might make life easier for everyone.

Thanks @colinleach! Since part of this PR copies #202, would you be happy with that being merged first so that the diff for this PR (both what's added and what's changed if anything) is clearer? If so, please approve #202 for me and it can be merged as is.
The failing test on R 3.6.3 can be ignored for now - I agree that we can probably just drop the tests for R3.6.3 now as we'll likely want to start encouraging use of pipes going forward.

Of course, whatever you think is best.

Sorry for the hassle! And thanks again.

Waiting till morning seems to be not quite enough for a successful rebase: there are still too many changed files in this PR. I think it may be easier to close it and start again with a PR from a clean base.


Crash

It crashes after using for a while

It seems to only crash when you are playing games. It crashes fast when I try to play games that take up a lot of resources.

Seems like an issue with the games then


Changing column comment crashes with Postgresql

change_table :table1 do |t|
t.change_comment :column1, 'comment text'
end
crashes as table name is correcty fetched in
lib/migration_comments/active_record/connection_adapters/table.rb
def change_comment(column_name, comment_text)
@base.set_column_comment(@table_name, column_name, comment_text)
end
@table_name seems to be not initialized. What is, is @table.
I'm using PostgreSQL adapter, AR 4.2.4, 0.3.2 migration_comments

@urmorae this should be fixed in latest master, please confirm if that works

@pinnymz yes, master works perfectly


Searching for an API description yields no results.

I've installed the VSCode extension and am attempting to search for API descriptions, I have tried: Weather, GitHub and test. No results shown at all.
If you let me know what information you need to debug this.

Thanks for trying out the extension and for reaching out.
What OS are you running?
If you download the command line tool instead and run the kiota search command, do you get any result?
Are you getting any error message in the output from the kiota category or the extension host category?
Can you see the kiota version in the status bar? (Bottom right)

macOS
CLI tool returns this error when ran Failed to create CoreCLR, HRESULT: 0x80070008

Thanks. This is the exact same thing as #2559 that was reported internally.
I'm going to close this issue so we track everything at one place. You can subscribe to the other issue.
I'll provide a workaround there for people to use until we fix the mac releases.


Add a README

Something maybe following this template
https://gist.github.com/PurpleBooth/109311bb0361f32d87a2

link broken

You have to copy address because github put different address to actual link. Or just properly format link with markdown :D

believe it or now I actually used markdown the first time around


Discussion regarding performance / absurd

Since this implementation consistently beats IndexedDB implementation AND provides a consistent syntax for SQLite operation (e.g. you may use same-way defined operations like in an electron app) isn't that implementation is actually absurd?
Good use cases seem pretty clear to me
In case there some another, a well-maintained project for the same purpose, maybe it would be good to add a link to that repo in the description?

The code itself is not absurd, but the fact that it's needed at all and that we're stuck with something like IndexedDB is, and the fact that it beats it so handedly. Don't read too much in the name :)
There's only one other project that I know of that's attempted a similar thing, but it's not quite as fleshed out and proven yet, and doesn't have as much optimizations: https://github.com/rhashimoto/wa-sqlite

@jlongster I opened this issue just to have some fun but after reading README and running perf tests on my own I realized that there in no jokes happening in this repo, that's something that people could actually use in a real production environment to improve how their apps work. So please, just consider choosing another name for repo just to prevent misunderstanding of people about the goal of your project.
And thanks for all that stuff you did!


New version: Azul.Zulu.8.JRE version <IP_ADDRESS>

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.1 schema?

Microsoft Reviewers: Open in CodeFlow

Beep boop

/azp run


[Cypress] [WIP] Fix E2E and integration Tests for UI

Signed-off-by: Gaurav Chadha<EMAIL_ADDRESS>Description
This PR fixes #

[x] GraphQL Utils for handling Queries and Mutations.
[ ] E2E test for service_mesh_configuration_management_spec
[ ] Integration test Configuration_filters_spec
[x] E2E for userpreference_spec
[x] Integration for userperference_spec

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@Chadha93 if this first done, let's proceed with merge, then follow up on the other two checkmarks subsequently.

@Chadha93 if this first done, let's proceed with merge, then follow up on the other two checkmarks subsequently.

I just had a quick huddle with @Abhishek-kumar09 regarding this, since the new GraphQL endpoints are updated in catalog Server PR for Catalog and UI Pr is pending we need to wait to get that merge, and some more items need to be updated here after that we can test and change accordingly. Then we can proceed with it.
Found a login issue with filter_spec cypress file which was not allowing to visit /configuration/filters, making a fix for that separately.

@Chadha93, oh, very good.

@Chadha93, reminder to knock this one out:

Found a login issue with filter_spec cypress file which was not allowing to visit /configuration/filters, making a fix for that separately.

@Chadha93, reminder to knock this one out:

Found a login issue with filter_spec cypress file which was not allowing to visit /configuration/filters, making a fix for that separately.

Already done https://github.com/meshery/meshery/pull/6139

It depended on https://github.com/meshery/meshery/pull/6027 since the PR is merged, it needs a rebase to pull those changes, I am not updating the branch myself incase you have any uncomitted changes that may introduce you to some extra challenge time for pushing to the current git history.
The ask is if you have updated the branch, is the gql interceptors working as expected now?

It depended on #6027 since the PR is merged, it needs a rebase to pull those changes, I am not updating the branch myself incase you have any uncomitted changes that may introduce you to some extra challenge time for pushing to the current git history.
The ask is if you have updated the branch, is the gql interceptors working as expected now?

Updated it now.

beforeEach hook in integration/configuration_filters_spec.js is getting a 500 during request to set the provider @Chadha93:

@Chadha93 for e2e/service_mesh_configuration_management_spec.js the modified request interception alias never gets called by Meshery UI:
CypressError: Timed out retrying after 5000ms: `cy.wait()` timed out waiting `5000ms` for the 2nd request to the route: `getConfigItems`. No request ever occurred.

@leecalcote @Abhishek-kumar09 Cypress checks for Filters are disabled for now and other UI tests are passing.

Good. We’ll be down a test, but back to green in the meantime.

Merging this one to get green checks on tests. Opening a separate issue for filters test with gql request implementation.


Open Skype Bot with URL and pass in data

System Information

SDK Language: Node.js
SDK Version: 3.8.4
Development Environment: localhost or Auzure Bot Service

Issue Description
How can I open a Skype bot via a link while passing in some data? This is similar to how Facebook Messenger bots can do the same using a m.me/BotName?ref=1234. Is this possible in Skype?

@suvamsh This is not currently a feature of Skype.


Create all test results in GetTestRuns before disposing the TestManagementHttpClient

Create all test results in GetTestRuns before disposing the TestManagementHttpClient
Resolves #251

@christianbumann your changes have been merged, thanks for your contribution 👍


some questions about the paper

Thanks for sharing your great work!
I have read your paper, but I am having the following confusions w.r.t. to your methodology:

When constructing a counter-part, we need an ID and an OOD feature representation, how do you select ID? Is this coming from the energy function lower than a certain threshold?

Why we need to construct a one-one counter-part instead of one-multiple counter parts ?

When the detector network just get initialized, classification branch is still unreliable, is energy function a good representation of OOD?

Why high dissimilarity with an ID indicates OOD? How to certify they are not objects from some other classes? Objects from the rest classes might occupy larger feature space.

Hope to hear your answer!

Hi, @luoyuchenmlcv
Thanks for interest in our work!

The Id in the code is normally defined as the proposals after RPN with higher confidence to be foreground.

one-to-one mapping is because we aggregate all the potential unknown objects in a different frame to improve the diversity and also the smoothness of the object features. The latter is crucial for optimization.

Yeah, at the beginning of training, the energy function might not be optimal. but it does not do too much harm. The network will quickly learn discriminative object representations and thus produce meaningful logits and energy scores.

We actually introduce the energy filtering in the distillation part, The dissimilar objects with mild energy scores are used as the unknowns. So that hopefully "objects from some other classes" are filtered. You can visualize to check.

Let me know if you have more questions.


installing problem

i got this problem
http://localhost:8080/
MySQL: Invalid Query:
				SELECT ID, Name, Level, Secondary
				FROM permissions
				ORDER BY Level SQL error: 1146 (Table 'gazelle.permissions' doesn't exist)
Array ( [0] => Array ( [0] => SELECT ID, Name, Level, Secondary FROM permissions ORDER BY Level [1] => 0.080108642578125 [2] => ) )

installed vagrant with virtual machine on windows
also how i connecting to phpmyadmin?

Did you ever initialize the tables in the DB? Schema/script found here: https://github.com/ApolloRIP/Gazelle/blob/master/db/data/gazelle.sql
I don't know why you're linking to localhost, since we aren't in your network.

Closing as stale.


[Feature]docker deploy locally

Purpose of this pull request
add docker deploy locally

Brief change log

add docker deploy locally

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If necessary, please update the documentation to describe the new feature.

Codecov Report

Merging #82 (acb56dd) into dev (7e1e61d) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head acb56dd differs from pull request most recent head 840e229. Consider uploading reports for the commit 840e229 to get more accurate results

@@          Coverage Diff          @@
##             dev     #82   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files        142     142
  Lines       3782    3782
  Branches     230     230
=====================================
  Misses      3782    3782

Continue to review full report at Codecov.

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


En/Decode cookies with RFC6265

Hi,
Let's say the server returns a Set-Cookie header:
canalnet=Apv3FQl3fsWZskAZda1GN0rN9BJiTbujoi7Puq/Iq37rUQrJirrnAPOTHZo2aKlpyzWbdaH1MjKyV6s=;domain=bnpparibas.net; path=/
Then, having CookieDecoder.decode and CookieEncoder(false).encode produces:
canalnet="Apv3FQl3fsWZskAZda1GN0rN9BJiTbujoi7Puq/Iq37rUQrJirrnAPOTHZo2aKlpyzWbdaH1MjKyV6s="; $Path="/"; $Domain=bnpparibas.net
which is Version 1 minus the Version attribute.
According to Netscape draft, I would expect canalnet=Apv3FQl3fsWZskAZda1GN0rN9BJiTbujoi7Puq/Iq37rUQrJirrnAPOTHZo2aKlpyzWbdaH1MjKyV6s= (no double quotes, no path, no domain)
This causes non Version 1 compliant components, such as F5 BigIP, to fail parsing the Cookie header.
If you're OK with this, I can easily provide a PR.
Regards,
Stéphane

@trustin I guess this issue can now be closed as commit was cherry-picked to 4.1 and master.

Doing some final touches right now.

OK, thanks!

@slandelle I have a few questions/ideas:

It looks to me Rfc6265Client/ServerCookieEncoder can be a drop-in (or better) replacement of the old Client/ServerCookieEncoder. Correct? If so, I think we can simply remove the old encoders and remove the Rfc6265 prefix.
Similarly, we can remove the Rfc6265 prefix for the decoders because we don't have name clash here.
Would it be a good idea to port this pull request and the changes I proposed in this comment to 4.0 sans the rawValue part?

It looks to me Rfc6265Client/ServerCookieEncoder can be a drop-in (or better) replacement of the old Client/ServerCookieEncoder. Correct? If so, I think we can simply remove the old encoders and remove the Rfc6265 prefix.
Similarly, we can remove the Rfc6265 prefix for the decoders because we don't have name clash here.

Absolutely. I forked instead of replacing in order to not break compatibility and left the decision of such a move to you.

Would it be a good idea to port this pull request and the changes I proposed in this comment to 4.0 sans the rawValue part?

I'm not sure. People using the existing set would experience behavior changes in a fix version. For example, existing code relying on encoders returning null might break now that they return empty collections.

BTW, once you're done, I'd gladly fix #1466 :)

@slandelle One last question - it seems like the first local variable in Rfc6265ClientCookieDecoder.decode() is not really used at all.  When it is set to false, it always breaks out of the outer loop, and thus nobody will see the case where first is false.  Is it just a cruft I can clean up, or did you have something in your mind?

What have I done? It seems boolean first = true; should be defined outside the outer loop.
Let me check.

If you are going to push another commit, then you'd better start your branch from scratch because your work has been cherry-picked and the classes have been renamed as I proposed. Let me check back tomorrow morning.

OK

I checked, and yes, it's no longer used and can be dropped, sorry for this.

Done in #3340

Someone forgot to close this :)


Cannot Update 0.73.2 to any version - 403 Forbidden

I've been unable to update from 0.73.2.  I've tried to update with every release since and they all fail.  Trying to update to 0.79.3.  I am running hassio image on a rpi3.
➜  config git:(master) ✗ hassio homeassistant update
Error decoding json invalid character ':' after top-level value: 403: Forbidden#

Clicking update in the web UI does nothing.  I see no errors in the web browser console.  No log message appears in the home-assistant.log.
I should add the following as well:
➜  config git:(master) ✗ hassio su update
Error decoding json invalid character ':' after top-level value: 403: Forbidden#
➜  config git:(master) ✗ hassio host update
Error decoding json invalid character ':' after top-level value: 403: Forbidden#

So in short I cannot update anything.  This just started with 0.73.2.  Previously I've been able to update this hassio install without issue.

https://github.com/home-assistant/hassio/pull/744
This seems to be the fix...
How do I apply this patch to an already running hassio image?

The WebUI update button finally did something.   Error in log:
18-10-09 22:25:07 ERROR (SyncWorker_5) [hassio.docker.interface] Can't install homeassistant/raspberrypi3-64-homeassistant:0.79.3 -> 500 Server Error: Internal Server Error ("readlink /var/lib/docker/overlay2/l: invalid argument")

My image might be in trouble.

I have a similar error, after updating my hass.io instance from 0.79.3 to 0.80 on my RPI.
WebUI doesn't load and only the command hassio ha info works. Other CLI command return the same error message than you.

When trying to log the webUI using local IP I have a 404: Not Founderror.

After a reboot of the RPI by unplug/plug webUI is now OK but the CLI still have the initial error. Strange.

I have the same issue right now, running 0.79.1. Cannot upgrade.
@oncleben31  Did you find a solution?

The Raspberry reboot solve partially the problem. CLI is still KO.

@oncleben31 and @mmmente
I had this exact same issue. Now what was a bit different is that after I upgraded to the latest version, my CLI broke like yours. When looking at the hassio logs, I saw that the 403 error was caused by an "Invalid Token Key". I thought at first my CLI needed to be upgraded as I was running SSH Server 3.0 and an outdated CLI was the problem. However, I still had the same issue.
So what I done was uninstall SSH Server Completely from the webpage in hass.io. I then reinstalled SSH server. As soon as I done that, I was all good. All my CLI commands run now.

/move to home-assistant/hassio


exec $SHELL doesn't  has effect.

Hi, I use archlinux, the command exec $SHELL doesn't has effect, but the source ~/.bash_profile has effect.

It's a bash issue. Documentation enhancement is welcome.

Reinitializing the current shell session needs -l, i.e. exec -l $SHELL.


Image building fails with "Received bad response from Model Management Service"

Here is a description of an issue I encountered, which may be worth adding to this page:
If:

image building (e.g. az image create container) fails with unhelpful error messages such as:
"Received bad response from Model Management Service"
"Response Code: 500"
"An internal server error occurred. Please try again. If the problem persists, contact support"

and you are using an ARM template to provision the necessary Azure resources (Storage Account, Container Registry, Key Vault, App Insights, ML Service Workspace)

and you deployed the template multiple times (for example, as part of a CI/CD workflow),

the issue may be caused by the fact that the Key Vault ARM resource (Microsoft.KeyVault/vaults) is implemented in a way that assumes the accessPolicies property is always provided and exhaustive (contains all access policies that should be set on the vault) - when redeploying this resource all existing access policies not specified in the accessPolicies property in the ARM template are removed.
This breaks the functionality of the ML Workspace, because the Workspace adds its own access policy to the Key Vault and that policy gets wiped out when the Key Vault resource is redeployed (updated).
There are a couple of possible workarounds:

Do not deploy the ARM template more than once.
Add back the missing access policy using the Microsoft.KeyVault/vaults/accessPolicies resource.
Before starting the template deployment, examine the Key Vault if it exists and make a list of existing access policies. Pass the list to the template in order to fill the accessPolicies property of the Key Vault resource.
Before starting the template deployment, check if the Key Vault exists. If it does, pass a flag to the template in order to skip updating the Key Vault using a conditional expression in the template.

(I went with option 4.)
Here is a link to a feedback item for improving the behavior of the Key Vault resource:
https://feedback.azure.com/forums/906355-azure-key-vault/suggestions/34414747-arm-template-for-keyvault-to-have-accesspolicies-n
and another feedback item for teaching the Workspace resource to fix missing permissions:
https://feedback.azure.com/forums/257792-machine-learning/suggestions/36771241-workspace-arm-resource-should-repair-missing-permi

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

ID: 3a077225-4fb9-57c8-6bf7-f6e22eef0870
Version Independent ID: bf5ce9ce-51a8-43e2-7362-e3de0e2ac63e
Content: Deployment troubleshooting guide - Azure Machine Learning service
Content Source: articles/machine-learning/service/how-to-troubleshoot-deployment.md
Service: machine-learning
GitHub Login: @chris-lauren
Microsoft Alias: clauren

@jberezanski-mdg Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@hning86 Hi Hai, could you please check this product issue? I think the workaround is worth to add to the document. Thanks.

@Blackmist FYI

@YutongTie-MSFT - #please-assign to Blackmist

@jberezanski-mdg thanks for reporting this and apologies for the delay in getting back to you. Let me know if I have the order of the steps to reproduce this correct:

Use a template to create the resources for Azure Machine Learning, including Key Vault
Rerun the template to create another workspace, but using the same Key Vault as the previous one
This breaks the previous workspace.

Is that correct? Do you know if there's a different error from the SDK when trying to create the image, or the same one?
Thanks

It's even simpler, you can run the same template twice (with the same parameters). Most ARM resources are idempotent, so it should effectively be a no-op, but the Key Vault resource will clear the access policies, breaking the workspace.
I haven't used the SDK directly. However, I can see the error message ("Received bad response from Model Management Service") in the SDK code (azureml/core/image/image.py, class Image, method create), so it seems to be coming from there. I also seem to remember seeing the same message when trying to create an image from the Azure Portal.
I will try both ways (SDK and portal) once I set up a broken workspace again.

OK, here are the messages:
CLI:
ERROR: {'Azure-cli-ml Version': None, 'Error': WebserviceException('Received bad response from Model Management Service:\nResponse Code: 500\nHeaders: {\'Date\': \'Tue, 26 Feb 2019 17:34:05
GMT\', \'Content-Type\': \'application/json\', \'Transfer-Encoding\': \'chunked\', \'Connection\': \'keep-alive\', \'api-supported-versions\': \'2018-03-01-preview, 2018-11-19\', \'x-ms-client-request-id\':
\'bc89430916164412abe3d82acb1d1109\', \'x-ms-client-session-id\': \'\', \'Strict-Transport-Security\': \'max-age=15724800; includeSubDomains; preload\'}\nContent:
b\'{"code":"InternalServerError","statusCode":500,"message":"An internal server error occurred. Please try again. If the problem persists, contact support"}\'',)}

Portal:
Create image "myimage": An internal server error occurred. Please try again. If the problem persists, contact support. . Request ID: f000e6592ad74fa5a21f3a1c494e2cba. Session ID: cef9de06-34b0-2cdd-579c-ea743f5c1924

SDK:
>>> image = ContainerImage.create(name = "myimage", models = [model], image_config = image_config, workspace = ws)
Creating image
Traceback (most recent call last):
  File "C:\Python37\lib\site-packages\azureml\core\image\image.py", line 341, in create
    resp.raise_for_status()
  File "C:\Python37\lib\site-packages\requests\models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: https://eastus.modelmanagement.azureml.net/api/subscriptions/<redacted>/resourceGroups/<redacted>/providers/Microsoft.MachineLearningServices/workspaces/<redacted>/images?api-version=2018-11-19

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Python37\lib\site-packages\azureml\core\image\image.py", line 346, in create
    'Content: {}'.format(resp.status_code, resp.headers, resp.content))
azureml.exceptions._azureml_exception.WebserviceException: Received bad response from Model Management Service:
Response Code: 500
Headers: {'Date': 'Tue, 26 Feb 2019 17:47:53 GMT', 'Content-Type': 'application/json', 'Transfer-Encoding': 'chunked', 'Connection': 'keep-alive', 'api-supported-versions': '2018-03-01-preview, 2018-11-19', 'x-ms-client-request-id': '3cdcf791f1214b9cbac93076ebfb5167', 'x-ms-client-session-id': '', 'Strict-Transport-Security': 'max-age=15724800; includeSubDomains; preload'}
Content: b'{"code":"InternalServerError","statusCode":500,"message":"An internal server error occurred. Please try again. If the problem persists, contact support"}'

Thanks for the detailed information @jberezanski-mdg , I'll start work on adding this to the document today.

Thanks again for the help and information on this @jberezanski-mdg ! The update to the document should be live later today.

#please-close

test

@jberezanski-mdg
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.

Thank you!

received same error when trying to create AML workspace with existing ACR. please find the logs below
=======================================================================
Error log*******************************
HTTPError                                 Traceback (most recent call last)
/databricks/python/lib/python3.7/site-packages/azureml/core/image/image.py in create(workspace, name, models, image_config)
359         try:
--> 360             resp.raise_for_status()
361         except requests.exceptions.HTTPError:
/databricks/python/lib/python3.7/site-packages/requests/models.py in raise_for_status(self)
939         if http_error_msg:
--> 940             raise HTTPError(http_error_msg, response=self)
941
HTTPError: 500 Server Error: Internal Server Error for url: https://eastus.modelmanagement.azureml.net/modelmanagement/v1.0/subscriptions/[REDACTED]/resourceGroups/[REDACTED]/providers/Microsoft.MachineLearningServices/workspaces/test/images
During handling of the above exception, another exception occurred:
WebserviceException                       Traceback (most recent call last)
 in
6                                                       image_name="winequality",
7                                                       description="Sklearn ElasticNet image for predicting wine quality",
----> 8                                                       synchronous=True)
9
10 #model_image.wait_for_creation(show_output=True)
/databricks/python/lib/python3.7/site-packages/mlflow/azureml/init.py in build_image(model_uri, workspace, image_name, model_name, mlflow_home, description, tags, synchronous)
226             name=image_name,
227             image_config=image_configuration,
--> 228             models=[registered_model],
229         )
230         _logger.info(
/databricks/python/lib/python3.7/site-packages/azureml/core/image/image.py in create(workspace, name, models, image_config)
364                                       'Headers: {}\n'
365                                       'Content: {}'.format(resp.status_code, resp.headers, resp.content),
--> 366                                       logger=module_logger)
367         if resp.status_code >= 400:
368             raise WebserviceException('Error occurred creating image:\n'
WebserviceException: WebserviceException:
Message: Received bad response from Model Management Service:
Response Code: 500
Headers: {'Date': 'Tue, 27 Oct 2020 06:42:17 GMT', 'Content-Type': 'application/json', 'Transfer-Encoding': 'chunked', 'Connection': 'keep-alive', 'x-ms-client-request-id': 'ef5cb682dc834ea3ab0c099fbf4098ec', 'x-ms-client-session-id': 'bc085591-809e-4df7-8389-aac74463564b', 'api-deprecated-versions': '1.0, 2018-03-01-preview, 2018-11-19', 'X-Content-Type-Options': 'nosniff', 'x-request-time': '0.901', 'Strict-Transport-Security': 'max-age=15724800; includeSubDomains; preload'}
Content: b'{"code":"InternalServerError","statusCode":500,"message":"An internal server error occurred. Please try again. If the problem persists, contact support.","correlation":{"RequestId":"ef5cb682dc834ea3ab0c099fbf4098ec"}}'
InnerException None
ErrorResponse
{
"error": {
"message": "Received bad response from Model Management Service:\nResponse Code: 500\nHeaders: {'Date': 'Tue, 27 Oct 2020 06:42:17 GMT', 'Content-Type': 'application/json', 'Transfer-Encoding': 'chunked', 'Connection': 'keep-alive', 'x-ms-client-request-id': 'ef5cb682dc834ea3ab0c099fbf4098ec', 'x-ms-client-session-id': 'bc085591-809e-4df7-8389-aac74463564b', 'api-deprecated-versions': '1.0, 2018-03-01-preview, 2018-11-19', 'X-Content-Type-Options': 'nosniff', 'x-request-time': '0.901', 'Strict-Transport-Security': 'max-age=15724800; includeSubDomains; preload'}\nContent: b'{"code":"InternalServerError","statusCode":500,"message":"An internal server error occurred. Please try again. If the problem persists, contact support.","correlation":{"RequestId":"ef5cb682dc834ea3ab0c099fbf4098ec"}}'"
}
}
==================Code we ran=============================================================
import os
from azureml.core import Workspace
from azureml.core.authentication import ServicePrincipalAuthentication
service_principal_password = os.environ.get("AZUREML_PASSWORD")
service_principal_auth = ServicePrincipalAuthentication(
tenant_id="",
username="",
password=service_principal_password)
ws = Workspace.create(name='myworkspace',
auth=service_principal_auth,
subscription_id='',
resource_group='myresourcegroup',
create_resource_group=False,
location='eastus2',
friendly_name='My workspace',
storage_account='subscriptions//resourcegroups/myresourcegroup/providers/microsoft.storage/storageaccounts/mystorageaccount',
key_vault='subscriptions//resourcegroups/myresourcegroup/providers/microsoft.keyvault/vaults/mykeyvault',
app_insights='subscriptions//resourcegroups/myresourcegroup/providers/microsoft.insights/components/myappinsights',
container_registry='subscriptions//resourcegroups/myresourcegroup/providers/microsoft.containerregistry/registries/mycontainerregistry',
exist_ok=False)

@yoganandmf Hi, thanks for reaching out to us. Could you please open a new ticket for this issue? The original thread is too old to make sense for reopening. Thanks a lot.

Stumbled upon this and related errors, apparently even deleting the serving endpoint left the state unclean and we were not able to redeploy with the same name. Killing the ACI container directly did not help either.


fix(build): add cross-env to support windows

You build scripts using ENV variables without cross-env, as a result, I didn't succeed to run a build on Windows.
This PR adding cross-env before MODULES= the same way as it's done in package.json scripts.
now:
await exec.promise(
    `MODULES=${format} npx babel src --out-dir ${outputDir}/${format} --ignore ${ignore.join(
      ',',
    )}`,
  );

to:
await exec.promise(
    `cross-env MODULES=${format} npx babel src --out-dir ${outputDir}/${format} --ignore ${ignore.join(
      ',',
    )}`,
  );

Error log:
$ npm run build:prod

><EMAIL_ADDRESS>build:prod D:\projects\swiper
> cross-env NODE_ENV=production node scripts/build

'MODULES' is not recognized as an internal or external command,
operable program or batch file.
'MODULES' is not recognized as an internal or external command,
operable program or batch file.
'MODULES' is not recognized as an internal or external command,
operable program or batch file.
'MODULES' is not recognized as an internal or external command,
operable program or batch file.
'MODULES' is not recognized as an internal or external command,
operable program or batch file.
'MODULES' is not recognized as an internal or external command,
operable program or batch file.
(node:16004) UnhandledPromiseRejectionWarning: Error: Shell command exit with non zero code: 1
    at ChildProcess.<anonymous> (D:\projects\swiper\node_modules\exec-sh\lib\exec-sh.js:73:15)
    at ChildProcess.emit (events.js:315:20)
    at maybeClose (internal/child_process.js:1048:16)

Closed #3820

Merged, thanks!


MB-16399 update SITDestinationFinalAddress with approved sit address update

Jira ticket
Summary
This ticket updates the SITDestinationFinalAddress on the service item when the prime creates an auto-approved sit address update
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.
Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Run DEVSEED_SUBSCENARIO=sit_address_updates make db_dev_e2e_populate
Find the DDDSIT service item ID for move SITUP1
make a POST request to https://primelocal:9443/prime/v1/sit-address-updates with this body

{
  "newAddress": {
    "streetAddress1": "7 Q St",
    "city": "San Antonio",
    "state": "TX",
    "postalCode": "78227",
  },
  "contractorRemarks": "test remarks",
  "mtoServiceItemID": {service_item_id}
}

Look at move SITUP1 again and verify that the SITDestinationFinalAddress matches newAddress

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Messages

:book:
:link: MB-16399

  Generated by :no_entry_sign: dangerJS against 2cbba58901260af22e94d6553264368bf952cb4f


paths.base not considered for Service worker

Describe the bug
When paths.base is set and service-worker.js is present in the root, the final build has register("/service-worker.js"), which throws 404
Logs
A bad HTTP response code (404) was received when fetching the script.
VM204 :1 Uncaught (in promise) TypeError: Failed to register a ServiceWorker for scope ('https://sidharthv96.github.io/') with script ('https://sidharthv96.github.io/service-worker.js'): A bad HTTP response code (404) was received when fetching the script.
To Reproduce
Open https://sidharthv96.github.io/mermaid-live-editor
To help us help you, if you've found a bug please consider the following:

If possible, we recommend creating a small repo that illustrates the problem.
Reproductions should be small, self-contained, correct examples – http://sscce.org.

Occasionally, this won't be possible, and that's fine – we still appreciate you raising the issue. But please understand that Svelte is run by unpaid volunteers in their free time, and issues that follow these instructions will get fixed faster.
Expected behavior
paths.base is prepended to service worker path.
Stacktraces
If you have a stack trace to include, we recommend putting inside a <details> block for the sake of the thread's readability:

Stack trace
Stack trace goes here...

Information about your SvelteKit Installation:

Diagnostics

The output of npx envinfo --system --npmPackages svelte,@sveltejs/kit,vite --binaries --browsers

Your browser

Your adapter (e.g. Node, static, Vercel, Begin, etc...)

Severity
How severe an issue is this bug to you? Is this annoying, blocking some users, blocking an upgrade or blocking your usage of SvelteKit entirely?
Medium-Low. As it blocks users who need to use Service workers.
Additional context
Add any other context about the problem here.

Not necessarily a duplicate, but could be probably be solved in #922 as well. Closing in favor of the mentioned issue.


Set up GraphQL queries

User Stories

As a user, I can retrieve data about users, their thoughts, reactions, and friends from the server

I added appropriate queries.


Update persistent-volumes.md

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Please delete this note before submitting the pull request.
For 1.12 Features: set Milestone to 1.12 and Base Branch to release-1.12
Help editing and submitting pull requests:  https://deploy-preview-9510--kubernetes-io-master-staging.netlify.com/docs/contribute/start/#submit-a-pull-request.
Help choosing which branch to use, see
https://kubernetes.io/docs/contribute/start#choose-which-git-branch-to-use.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

@karthikragu, thanks for the PR!  You will need to sign the CLA first before we can review it

Hey there! @karthikragu, looks like you haven't signed the CLA yet. Could I please have you do that? https://github.com/kubernetes/community/blob/master/CLA.md


run tests with coverage for correct coverage reporting

Pytest loads the plugin before the test is run, so we have to start coverage before pytest loads.
See https://github.com/iterative/pytest-servers/runs/7918315266?check_suite_focus=true#step:6:32.

Took the liberty of rebasing this on main. windows and macos testing failures were unrelated (and should now be fixed)

I have been trying to see if there's a simpler solution to this. I tried the suggestion from https://pytest-cov.readthedocs.io/en/latest/plugins.html, but it did not work. :(

I mean, the coverage report already works, so that solution also works.
The only issue here is that warning because the plugin gets loaded before the coverage.

Codecov Report

Merging #45 (a407634) into main (9be31d2) will increase coverage by 29.84%.
The diff coverage is 64.70%.

@@             Coverage Diff             @@
##             main      #45       +/-   ##
===========================================
+ Coverage   53.29%   83.14%   +29.84%
===========================================
  Files          11       11
  Lines         349      350        +1
  Branches       41       44        +3
===========================================
+ Hits          186      291      +105
+ Misses        148       42      -106
- Partials       15       17        +2

Impacted Files
Coverage Δ

src/pytest_servers/exceptions.py
50.00% <ø> (+50.00%)
:arrow_up:

src/pytest_servers/utils.py
78.04% <ø> (+34.86%)
:arrow_up:

tests/test_utils.py
100.00% <ø> (ø)

src/pytest_servers/factory.py
81.60% <64.70%> (+18.78%)
:arrow_up:

src/pytest_servers/s3.py
92.30% <0.00%> (+29.23%)
:arrow_up:

src/pytest_servers/gcs.py
92.30% <0.00%> (+34.61%)
:arrow_up:

src/pytest_servers/azure.py
92.85% <0.00%> (+46.42%)
:arrow_up:

... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Coverage   53.29%   83.71%   +30.41%
Looks like it isn't reporting coverage correctly?

https://github.com/iterative/pytest-servers/runs/7954317996?check_suite_focus=true#step:6:29
nox > coverage report --rcfile pyproject.toml --data-file .coverage.3.8
Name                                                                      Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------------------------------------------------------------------------
.nox/tests-3-8/lib/python3.8/site-packages/pytest_servers/__init__.py         0      0      0      0   100%
.nox/tests-3-8/lib/python3.8/site-packages/pytest_servers/azure.py           27      1      4      2    90%   40->37, 49
.nox/tests-3-8/lib/python3.8/site-packages/pytest_servers/exceptions.py       6      3      4      0    70%   5-7
.nox/tests-3-8/lib/python3.8/site-packages/pytest_servers/factory.py         87      8     30      8    86%   57-58, 65, 70, 78-79, 108->111, 119, 1[29](https://github.com/iterative/pytest-servers/runs/7954317996?check_suite_focus=true#step:6:30), 146->148, 148->151, 185->188
.nox/tests-3-8/lib/python3.8/site-packages/pytest_servers/fixtures.py        44     20     10      0    44%   18, [30](https://github.com/iterative/pytest-servers/runs/7954317996?check_suite_focus=true#step:6:31), 36-38, 44, 50, 56, 71-82
.nox/tests-3-8/lib/python3.8/site-packages/pytest_servers/gcs.py             25      1      4      2    90%   38->36, 43
.nox/tests-3-8/lib/python3.8/site-packages/pytest_servers/local.py            7      1      2      0    89%   11
.nox/tests-3-8/lib/python3.8/site-packages/pytest_servers/s3.py              65      2     14      3    91%   35-36, 53->58, 75->78, 90->exit
.nox/tests-3-8/lib/python3.8/site-packages/pytest_servers/utils.py           41      6      8      3    78%   18-19, 50->exit, 56-59
tests/__init__.py                                                             0      0      0      0   100%
tests/test_factory.py                                                        26      0      7      1    97%   34->exit
tests/test_memory.py                                                         11      0      0      0   100%
tests/test_utils.py                                                           9      0      0      0   100%
---------------------------------------------------------------------------------------------------------------------
TOTAL                                                                       [34](https://github.com/iterative/pytest-servers/runs/7954317996?check_suite_focus=true#step:6:35)8     [42](https://github.com/iterative/pytest-servers/runs/7954317996?check_suite_focus=true#step:6:43)     83     19    83%

Looks awesome, thanks!

Okay, let's merge this for now. Still not very happy about it. :(


evaluation error using Karpathy's test split

Hi Ruotian,,
I tried to run the following script:
python eval.py --dump_images 0 --num_images 500 --model log_fc/model-best.pth --infos_path log_fc/infos_fc-best.pkl --language_eval 1
But at the end I got the below error.
evaluating validation preformance... 499/500 (0.000000)
loading annotations into memory...
0:00:00.811912
creating index...
index created!
using 500/500 predictions
Loading and preparing results...
DONE (t=0.03s)
creating index...
index created!
tokenization...
Traceback (most recent call last):
File "eval.py", line 143, in
loss, split_predictions, lang_stats = eval_utils.eval_split(model, crit, loader, vars(opt))
File "/home/Durrani/icp/self-critical.pytorch/eval_utils.py", line 158, in eval_split
lang_stats = language_eval(dataset, predictions, eval_kwargs['id'], split)
File "/home/Durrani/icp/self-critical.pytorch/eval_utils.py", line 52, in language_eval
cocoEval.evaluate()
File "coco-caption/pycocoevalcap/eval.py", line 32, in evaluate
gts  = tokenizer.tokenize(gts)
File "coco-caption/pycocoevalcap/tokenizer/ptbtokenizer.py", line 52, in tokenize
stdout=subprocess.PIPE)
File "/home/Durrani/anaconda3/envs/py27/lib/python2.7/subprocess.py", line 394, in init
errread, errwrite)
File "/home/Durrani/anaconda3/envs/py27/lib/python2.7/subprocess.py", line 1047, in _execute_child
raise child_exception
OSError: [Errno 2] No such file or directory
Anybody who implement this code your reply will be appreciated.  thanks

@DurraniBit how did you solve it?

As for the coco-caption java is necessary when I Install java on the server then the error is solved.


icon button tests added

Coverage increased (+4.3%) to 100.0% when pulling 162e51e1338b0b9c50e4ff2f7d1147e36a45b2ed on icon-button into 1db587c6e05a34664e717978dae2d10f722e852c on master.


Update en-GB.json

The en-GB.json provided appears to be written in American English. The differences between American English (en-US) and British English (en-GB) often involve spelling, terminology, and occasionally punctuation, but not so much in syntax or grammar. In the case of the provided JSON file, most of the text seems to be written in a neutral way that could apply to both American and British English.
However, there are a few phrases that could be more culturally aligned to British English. This PR is the slightly adjusted version with a few cultural changes.
The changes are minimal and mainly involve using a more British English style of phrasing:
In the "description" field, "allows" has been changed to "lets", which might be slightly more common in British English.
In the "instruction" field, "primary" has been used instead of "main", which is a common alternative in British English. "Color" has been changed to "colour" in the "chargerImage" and "theme" fields, reflecting the British English spelling. In the "instruction" field, "attempt" has been used instead of "try", and "goes wrong" instead of "fails", to align more with British English phrasing. Remember, these changes are very subtle and may not even be necessary. The original text was already in a very clear and neutral form of English that would be understood by speakers of both American and British English.
The rest of the JSON file could remain the same, as it is not specific to any particular dialect of English. It is important to note that most of the text here is quite technical and specific to the application itself (like error messages, status updates, etc.), which would generally be the same across different English dialects.

Really sorry for the delay, but added in v0.2.7.


while ONPAD, broadcast only shows first GPS data point

This could be an issue with the GPS read() or the main loop. This can be narrowed down by adding logging in both. Notably, the test code seems to work fine and has a very similar architecture.

this is no longer unintended behavior, as the GPS read function takes too long in its current state to use during the ONPAD or ASCENDING phases. We will research methods to reimplement this feature at a later time #60


Ensure SDK sends device_id properly

Description
Device id was being returned as "" which resulted in it not being set properly.
Check List

[ ] Are you changing anything with the public API?
[ ] Are your changes backwards compatible with previous SDK Versions?
[ ] Have you tested this change on real device?
[ ] Have you added unit test coverage for your changes?
[ ] Have you verified that your changes are compatible with all Android versions the SDK currently supports?

Changelog / Code Overview

Test Plan
Tested locally with changes to ensure device gets perisisted.
Related Issues/Tickets

https://klaviyo.tpondemand.com/entity/199177


Research deployment on a Polkadot parachain

Contract: Investigate the feasibility and benefits of deploying Propcorn on a Polkadot parachain. We need to understand the technical requirements and if we are able to deploy EVM contracts easily.
UI: Investigate which wallets we need to support and what needs to be changed in the front-end code.

After talking with Sacha, the conclusion is that there are 2 options:

deploy on Plaza, a polkadot System chain that would allow us to deploy our current smart contracts on polkadot
build a wasm compatible chain with our own EVM and use our smart contracts

We are going to wait for the release of Plaza and take it from there (Q1/Q2 next year).


Build for iOS

Hi,
I finished my script to build the PDFium to iOS. All is here with manual patchs:
https://github.com/prsolucoes/mobile-pdfium
But i have some issues after compile. All issues is described here:
https://github.com/prsolucoes/mobile-pdfium/issues/1
Can you help me?

Sorry, I fail to see how this relates to this project qtpdfium.

Hi,
It is working. If anyone wanna, it is building on travis and all instructions is here:
https://github.com/prsolucoes/mobile-pdfium
Thanks.


Bug Report | course time, create date and other text inside app not showing but outline shows

Great extension, i wish they hired you to do the UX/UI for the Udemy site! I noticed the extension is not showing the text links inside the area where the course shows in the library. I tried clearing cookies and logging back in, but still saw the problem.
See screenshot: https://imagizer.imageshack.com/a/img924/8720/CcU0cD.jpg
browser: Microsoft Edge Version 114.0.1823.24 (Official build) beta (64-bit)

I'm surprised I'm the only one saying this is a problem. I'll try to test it in Chrome and also private mode.
I'm not sure what else could cause this.

Hey @lundeen-bryan, apologies for taking so long to respond!
You're absolutely right, the extension is broken due to a recent markup change introduced on Udemy's site.
Thank you for bringing it to my attention!
I will publish a fixed version as soon as possible. Thank you for your patience!

Reopening issue until fixed version v1.1.1 is live on Chrome Web Store and Firefox Addons.

Thank you SO much Tad! Fixed and working like a charm again! Tested in Microsoft Edge Version 114.0.1823.37 (Official build) beta (64-bit). Ext was installed via Chrome web store.

I still have same problem on Chrome and Edge. I was using Chrome Web Store version on both. Tried to use with userscript but nothing changed.
Google Chrome 114.0.5735.199 (Official build) (64-bit)
Microsoft Edge 115.0.1901.203 (Official build) (64-bit)


Fixed Trampoline could not be reused

Once the Trampoline operator worked once, it would not work a second time.

This is terrible, thank you!
I will release it immediately!


[Cache] Question

Hello.
The cache in Baremaps works with caffeine.
I was wondering what would be your approch to invalidate the cache when changes take place in the database.
The API states       --cache=CACHE           The caffeine cache directive.
But it is a bit obscure what is expected.
Thanks

We should probably include this in the documentation of the command.
Here is the documentation from caffeine:
https://www.javadoc.io/doc/com.github.ben-manes.caffeine/caffeine/2.2.2/com/github/benmanes/caffeine/cache/CaffeineSpec.html
And a command line example:
baremaps map serve \
  --cache 'maximumWeight=1073741824,expireAfterAccess=1h'  \
  --tileset tileset.js   --style style.js


use ziti mgmt service if available

Add config var for webhook like ${ZITI_MGMT_SERVICE:=ege-management} that, if matching the name of a Ziti service authorized for the admin identity, the webhook will wrap all mgmt API calls with a Ziti Edge SDK client dialing that service.
This allows the Ziti mgmt API to be protected by a Ziti service with a minimum of additional config while preserving the current behavior to call the mgmt API over underlay.

The mgmt API would be published as a Ziti service, and the agent would use the same Ziti identity to authenticate for the service via the client API and for admin ops in mgmt API.
I'm not working on it yet, but I wanted to get the idea out there for discussion, etc.


files had a problem

I tried to make a file research in a big folder (near 60/70GB) when, after some second of the searching line was sliding on the screen, the app has [crashed.]
Follow the debug log file...
debug_fulltrust.txt
Thank you for the beautiful app!
Desktop:

OS Version: [Windows 10 Pro 20H2 19042.804]
App version: [v<IP_ADDRESS>]

Thanks for this report. Could you post the other debug file as well? (debug.txt)

Merging with #3844


Turkish character issue on DBEaver 7.3.0

Hello, although I changed the text file encoding as explained in the screenshot, I see the result set with Turkish characters incorrectly. How can we fix this problem, does anyone know?

Cannot reproduce. Please tell DBeaver version and your OS

there is no update on issue for a month.
feel free to reopen it with an additional info

there is no update on issue for a month.
feel free to reopen it with an additional info

w11 dbeaver 23.0 has same problem


You have pushed your api keys

The API keys in this repo are all expired and for demo only. Thanks for your comment.


Improve accessibility of buttons and checkboxes used for pagination

Is your feature request related to a problem? Please describe.
Microsoft Edge has handy accessibility checks built-in and they detected these issues:

Describe the solution you'd like
Back/Forward buttons for switching pages needs to have e.g. aria-label or title attribute set: https://dequeuniversity.com/rules/axe/4.4/button-name?application=axeAPI
Checkboxes for records selection feature need to have a label, e.g. aria-label attribute since we don't want a visible label here: https://dequeuniversity.com/rules/axe/4.4/label?application=axeAPI
Describe alternatives you've considered
Ignore a11y, but it hurts the ecosystem

Thank you for raising this.
This should be fixed in a few minutes, as soon as v2.4.3 hits the npm registry.
There are a few properties you can use to override the defaults:

allRecordsSelectionCheckboxProps
getRecordSelectionCheckboxProps
getPaginationControlProps

I've also updated these examples in the docs website accordingly:

Pagination
Records selection

Cool, that was very quick, thank you! Great to see some overridable defaults set, good solution.


Add source entry for social_nav_ros

Please Add This Package to be indexed in the rosdistro.
social_nav_ros
The source is here:
https://github.com/MetroRobots/social_nav_ros
Checks

[x] All packages have a declared license in the package.xml
[x] This repository has a LICENSE file
[x] This package is expected to build on the submitted rosdistro

Hi @DLu can you please follow the release instructions here using Bloom https://docs.ros.org/en/rolling/How-To-Guides/Releasing/First-Time-Release.html

Step 1 is be part of release team - I did that
Step 2 says Create a new release repository, and the first sentence is

If your repository is new to the ROS community, you should first open a pull request on ros/rosdistro adding a source entry for your repository.

What am I missing?


Remove helper functions needed just for their types

This addresses the second code block in #13.

Yay, and I thought this is impossible.

Merging. Thank you!