How to set an area size on onDividerDoubleTap?

How to set an area size on onDividerDoubleTap?
class MyHomePage extends StatelessWidget {
  final _multiSplitViewController = MultiSplitViewController(
    areas: [
      Area(
        flex: 0.5,
        builder: (context, area) {
          return const MyNavigationRail();
        },
      ),
      Area(
        builder: (context, area) {
          return const MainUI();
        },
      )
    ],
  );

  MyHomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: MultiSplitViewTheme(
        data: MultiSplitViewThemeData(
            dividerThickness: 8,
            dividerPainter: DividerPainters.grooved1(
                backgroundColor: const Color.fromRGBO(73, 73, 73, 1),
                color: const Color.fromARGB(255, 42, 42, 42),
                highlightedColor: const Color.fromRGBO(33, 33, 33, 1))),
        child: MultiSplitView(
          controller: _multiSplitViewController,
          onDividerDoubleTap: (dividerIndex) {
            _multiSplitViewController.getArea(0).size = 0; // no setter
            _multiSplitViewController.getArea(0).flex = 0; // no setter
          },
        ),
      ),
    );
  }
}

I want to control the divider position on onDividerDoubleTap

here is my current solution:
class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  var _multiSplitViewController = MultiSplitViewController(
    areas: [
      Area(
        flex: 0.5,
        builder: (context, area) {
          return const MyNavigationRail();
        },
      ),
      Area(
        builder: (context, area) {
          return const MainUI();
        },
      )
    ],
  );

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: MultiSplitViewTheme(
        data: MultiSplitViewThemeData(
            dividerThickness: 8,
            dividerPainter: DividerPainters.grooved1(
                backgroundColor: const Color.fromRGBO(73, 73, 73, 1),
                color: const Color.fromARGB(255, 42, 42, 42),
                highlightedColor: const Color.fromRGBO(33, 33, 33, 1))),
        child: MultiSplitView(
          controller: _multiSplitViewController,
          onDividerDoubleTap: (dividerIndex) {
            setState(() {
              _multiSplitViewController = MultiSplitViewController(
                areas: [
                  Area(
                    flex: 0,
                    builder: (context, area) {
                      return const MyNavigationRail();
                    },
                  ),
                  Area(
                    builder: (context, area) {
                      return const MainUI();
                    },
                  )
                ],
              );
            });
          },
        ),
      ),
    );
  }
}

it's ugly but it works for now
I hope we can improve the situation

Hi @stephane-archer!
Currently, to change the value of an area it is necessary to recreate the entire layout like any other layout. It would also not be possible to change the flex of just one area since this influences the neighboring area (when one stretches, another needs to shrink). I believe that setting the flex value directly in a single area would not be sufficient for the API. Perhaps what could exist is a method to only set the values of all areas at the same time without changing their Widgets.

@caduandrade, Can you clarify what is the issue with resizing an existing area (in that case this would be resizing 2 areas, one bigger and one smaller)? can you just change the constraints and then call the build function of the child widget?

@caduandrade these a great ways to simplify my code, thank you.
I will update my code now!
Do you think we can improve MultiSplitView API to avoid recreating areas?
onDividerDoubleTap gives us a dividerIndex Would you be able to add something to get from a dividerIndex a position and a possibility to change this position?

I think it's not impossible. But needs to be as intuitive as possible as you will also need to change the size, max and min values. It will never be perfect :smile: .

@caduandrade What I don't understand is that with my mouse I'm able to resize each panel without any complexity, what makes it impossible with an API?
When I double tap on a divider, I want to click on this divider and push it as much as possible to the left.
If I do it myself with my mouse it would be easy.
What I'm missing?

Hi @stephane-archer!
Short version: Yes, it would be possible, but not with the current API. Needs a new method. The current one is made for the drag event with/without pushing option and works only with pixels.
Large version: :open_mouth:
Drag uses DividerUtil's protected _resizeAreas and move methods. The move is just an entry point for _resizeAreas.
static double move(
      {required MultiSplitViewController controller,
      required LayoutConstraints layoutConstraints,
      required int dividerIndex,
      required double pixels,
      required bool pushDividers}) {

static double _resizeAreas(
      {required double pixelsToMove,
      required int direction,
      required MultiSplitViewController controller,
      required LayoutConstraints layoutConstraints,
      required int shrinkAreaIndex,
      required int growAreaIndex,
      required bool pushDividers}) {

It only works with pixels and internally knows how to convert to flex when necessary. This is the first problem in your case, there is nothing to directly change the flex. In the current implementation, when you completely shrink a flex area 1, its value is transferred to the neighboring flex area. So [1][1] turns into [0][2]. This is the second point, to adjust an area, it is currently necessary to adjust the neighboring area at the same time. In your most basic case with 2 areas, it would make no difference [1][1] to become [0][1] (changing only one area directly) or to [0][2] (via the drag event). The final effect on the screen would be [0%][100%]. But this is not true when there are other flex areas. For example, [1][1][2] represents [25%][25%][50%]. Shrinking the first via drag [0][2][2], the user would see [0%][50%][50%]. But if you only reset the size of the first [0][1][2], the last one will deform and the screen will have the proportion [0%][33.33%][66.66%]. This transfer of value to the neighbor also occurs with pixels. It would not be possible to reset pixels in one area and not change the neighboring area as this would result in inconsistency resolution when the total number of pixels is smaller than the size of the component.
I don't see a problem with there being a method to make it easier to change just one area and generate a different visual effect than the current drag. It is acceptable and expected. But what I wanted to say is that there is nothing ready and easy to expose this feature.
Conclusion, finally, :laughing: : It is in my plans to review this along with the possibility of having pixel + min/max in flex or flex + min/max in pixels. Different uses generate more complexity.

I created the issues:

#80
#81

@caduandrade I would like to thank you for all your help and explanation, and I would like to thank you for considering my needs and the willingness to improve the API/Library


Adds request modal

Adds a request modal. Doesn't actually send data anywhere yet; that's for #15
fixes #14

Maybe -  but let's leave futzing with that for later...it's also something we could user test.


adds encryption.Provider

that abstracts external dependencies and preconditions that need to be dynamic during a downgrade/upgrade

/assign @sttts

/lgtm
/approve


remove unsafe presents from present tree.

About the PR

Removed unsafe presents from the present tree.
Added the random coal present to the tree.
Changed the unsafe present prefix to be more clear.
Why / Balance

Insane presents can be any entity in the game

This is what results in signs, walls, singularities, etc, appearing in presents

unsafe presents can be any entity in the game with item component

This is what results in debug items and round ending items from being given to random players

(Possibly bad actor) Players take this as a opportunity to self antag with round ending items. (It's just a winter event who cares! Once a year thing! etc etc)

when not used for this, are used to powergame against antagonists

Admins endlessly on the conflict of having to ban people for using debug items then recieving numerous complaints about being banned for using debug weapons
Admins endlessly on the conflict of being told they shouldn't be banning people for using debug weapons and reciving complaints from players about the round being ended by said items

Going forward, presents should be further filtered (we have component filtering technology! See the randompresent system.) to exclude problem items like weapons and explosives.
If that is not reasonable, items will otherwise need to be handpicked and added to the whitelist, like the safe present.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Did we not already discuss this like a few weeks ago and decide not to do this?

@EmoGarbage404 It's still a reoccurring issue and I'm sick of seeing complaints from all sides

i feel like we could probably just make presents only spawn on christmas day, rather than all month long. Yeah, you could still have issues, but it'd be a lot more limited than dealing with it for literally all month.

i feel like we could probably just make presents only spawn on christmas day, rather than all month long. Yeah, you could still have issues, but it'd be a lot more limited than dealing with it for literally all month.

Ultimately it's a headmin decision on what to do.

i mean they are basically just meant to be a little bit of goofy fun for christmas. Tbh, I don't even know why they're available all month long. I feel like the main issue is just overexposure and being present in the game for so long that relatively rare things (singulo, debug items, etc) appear frequently.

i mean they are basically just meant to be a little bit of goofy fun for christmas. Tbh, I don't even know why they're available all month long. I feel like the main issue is just overexposure and being present in the game for so long that relatively rare things (singulo, debug items, etc) appear frequently.

That's probably the case when you have 80 players and at minimum 20 rounds per server per day

There should be an admin only christmas tree with only Insane presents and infinite uses per person for goofy admeme :godo:

i already removed all the trees


Separate out core SDK dependencies

A lot of the robotics stuff requires large dependencies that are not critical for working with the core components (e.g. miniscreen, battery, display).
This means that, in a Python context (particularly a virtualenv), many dependencies are required to be installed that are not explicitly required.
Separating out the dependencies like this will help to ensure that only dependencies that are required are included for a given application.
This is also a good time to consider the roll of the common library - should this simply become part of the "core" SDK?

One other thing to consider here is whether or not the common lib should just become part of the "core" SDK.

Proposal:
from pitop_sdk.pitop import ...
from pitop_sdk.robotics import ...


adjust snapPoints on window resize

changes based on https://github.com/emilkowalski/vaul/pull/399

Thanks for the PR @anbraten ! I believe this would resolve this issue https://github.com/radix-vue/vaul-vue/issues/63 ? 😁

@zernonia This fixes another bug when resizing the window, but unfortunately doesn't solve #63 yet. I am trying to finally fix it in #65.

Cool! Then I'll merge this in first 😁


Fix #138 - Use x to extract with 7-zip, not e. Use e only for dump, which strips directories.

Description
Changes from using the 'e' command to 'x' for 7-zip on all actions other than dump. 'e' extracts and strips directories. 'x' extracts with full paths. See https://sevenzip.osdn.jp/chm/cmdline/commands/index.htm
⚠ I'm not sure if we actually need to use 'e' for the special case of tar files. That particular use case involves using two piped 7zip commands, and I don't know if one of those two needs to use 'e' for some reason.
Issues Resolved

#138

Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[X] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Signed-off-by: Christopher Williams<EMAIL_ADDRESS>

@sgtcoolguy Travis failure here too


Limits are constrained by quotas on subscription level

It seems this document needs an update. For example the Storage Account resource type gets 250 instances by default and can be extended by raising a quota increase request. I would suggest you to include some guidance towards the subscription's usage and quotas menu in Azure Portal.

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

ID: 0bd67cd6-1517-b685-e2bf-a2bd0a063fea
Version Independent ID: 3745d93e-541c-f697-34a4-50b73b831161
Content: Resources without 800 count limit - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/resources-without-resource-group-limit.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@cveld - thanks for this feedback. The 250 limit is instances per region whereas this article covers instance per resource group. However, I will add a link for checking the region limit to this article.

The update has been published.
#please-close


chore(ci): check code format

Using Google's Java format checker[1].
[1] https://github.com/marketplace/actions/google-java-format#examples

My IntelliJ is refusing to the the Google formater. Will need help to do this if anyone has some time.


Pass the event as well as the encoded data to the on_event callback

Pass the event as well as the encoded data in the on_event callback
Fixes issue https://github.com/logstash-plugins/logstash-codec-compress_spooler/issues/13
Error encoding event {:exception=> # ArgumentError: wrong number of arguments (1 for 2)

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.

Finally got the CLA signed

What else needs to be done, to get this pull request moving.


clean up mcmc module

Codecov Report
Attention: Patch coverage is 10.00000% with 9 lines in your changes missing coverage. Please review.

Project coverage is 69.10%. Comparing base (fb42746) to head (6a54de6).
Report is 7 commits behind head on main.

Files with missing lines
Patch %
Lines

src/scope/emcee_fit_hires.py
10.00%
9 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #88      +/-   ##
==========================================
- Coverage   69.62%   69.10%   -0.52%
==========================================
  Files          12       12
  Lines         859      864       +5
==========================================
- Hits          598      597       -1
- Misses        261      267       +6

Flag
Coverage Δ

unittests
69.10% <10.00%> (-0.52%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Enhanced ColorPicker Component

Added new BIAL Method 'getColorAsHex()' so that you can retrieve hexadecimal color instead of the rgb(255,200,100) notation.  Useful for components that receive color as hex.

Cleaning up old issues.


Support conversion of MIDI's with multiple channels in a track

For example:

Pokemon Center
Pallet Town

My sheer surprise at someone else working with that hack of a script 😁
Nice one! Thank you


posts URL

How can I omit the date in the posts URL? E.g:
https://me.github.io/blog/2018/03/05/fritz
should be
https://me.github.io/blog/fritz

You can control this via permalink in your _config.yml
Closing as this is a Jekyll thing you can inquire with then if you need more assistance.
Ref: https://jekyllrb.com/docs/permalinks/

Perfect! Thanks for the immediate response!


Adopt updated builds of CI runners

Description
per request from ops
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.

/merge


No Browse tab in C++ project in vs2019

No Browse tab in C++ project in vs2019

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

ID: 55284369-f16f-2748-2a55-8cd24a06b940
Version Independent ID: f7b0c577-b584-98dc-90a2-4efcf095b35d
Content: Add references in the Reference Manager - Visual Studio
Content Source: docs/ide/how-to-add-or-remove-references-by-using-the-reference-manager.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

Related to https://github.com/MicrosoftDocs/visualstudio-docs/issues/3024

@ortymd Thanks for bringing this issue to our attention. We've added a Note our documentation to let folks know about the issue. Meanwhile, to make sure that this product issue is logged in the right place, please use the Report a Problem tool to report it. When you do so, your report becomes part of the work items that are tracked on Visual Studio Developer Community, where you can engage directly with the engineering team while they address and fix the problem.

Hello there! I'm having this same issue with VS2019
Screenshot

Project settings:

I'm having the same issue in VS 2022, missing the 'Browse' button in Project -> add reference

It looks like the Developer Community work item was closed as low priority: https://developercommunity.visualstudio.com/t/Add-Reference-is-missing-browse-tab/181732?ftype=problem&q=browse+button+add+reference+c%2B%2B


Not up-to-date on Cloudflare

Not sure how things end up on Cloudflare, but Picturefill's still on the 2.2.0 beta on Cloudflare.

@wilto or @scottjehl do either of you manage picturefill on cloudflare?

Not us. Looks like 2.2.0 stable is up there now, though? I think it just pulls from the repo tags; there must be a delay.


Add the customer id to the cart

Hi guys,
thank you for this easy to use hooks.
When the addItem hook create the cart, it would be nice to add the customer id if the user is connected.
I guess we could just get the customer_id in the useAddItem hook then use it as a param to the api call.
Let me know if this is something you may want to change.

Good idea. Necessary steps:

Get (if exists) the the customerToken

// src/api/cart/index.ts
const customerToken = req.cookies[config.customerCookie]

Add the customerToken to the body sent to handlers

// src/api/cart/index.ts
const body = { cartId, include, customerToken }

Get the customerId from the customerToken

// src/api/cart/handlers/add-item.ts
const customerId = customerToken && (await getCustomerId({ customerToken, config }))

Add (if exists) the customerId to the request body

It will be released in 1.6.0


Options.Namespace takes priority over ChartSpec.Namespace

My use case:
I have initialized a helm client, and I'm iterating through multiple charts and installing them one by one. Different charts go to different namespaces.
I've initialized the helm client like this:
        opt := &helmclient.KubeConfClientOptions{
            Options: &helmclient.Options{
                Namespace:        "default",
                RepositoryCache:  "/tmp/.helmcache",
                RepositoryConfig: "/tmp/.helmrepo",
                Debug:            true,
                Linting:          true,
                DebugLog: func(format string, v ...interface{}) {
                    fmt.Printf(format, v...)
                    fmt.Printf("\n")
                },
            },
            KubeContext: kubecontext,
            KubeConfig: kubeconfig,
        }

        // initialize helm client with the correct kube context
        helmClient, err := helmclient.NewClientFromKubeConf(opt, helmclient.Burst(100), helmclient.Timeout(time.Duration(time.Minute*15)))
        if err != nil {
            return err
        }

and my ChartSpec looks like:
    chartSpec := helmclient.ChartSpec{
        ReleaseName: m.Name,
        ChartName: m.HelmPackage,
        Version: "0.1.0",
        ValuesYaml: string(buf),
        DryRun: m.dryRun,
        Wait: false,
        Timeout: time.Duration(time.Minute*15),
        Namespace: m.Namespace,
        CreateNamespace: true
    }

I've set the ChartSpec.Namespace = "nopeus" and ChartSpec.CreateNamespace = true accordingly and ran InstallOrUpgradeChart with the chart.
Yet when I run this, the charts are deployed to the default namespace instead of the intended nopeus namespace. Which isn't a behavior I would expect.
I would love your help on this. Am I missing something? Is it a bug? Should I just create a new client every time (sounds weird)?

Also, it doesn't look like CreateNamespace actually works

I too would like to know how best to handle this.
Should you create a new client each time you change namespace?

👍 for this issue, having the same problem

same issue here...

I really hope that the problem reflected in this issue can be fixed as soon as possible

just rolled into the same issue


MPA Planning Domains shapefile not available

I need to access the MPA Planning Domains shapefile described here: https://www.ccamlr.org/en/science/mpa-planning-domains. I did go to the CCAMLR GIS portal, where I can see this layer, but when I clicked the “download” button, it directs me to this repository. The problem is that this layer is not available here. How can I access this MPA planning domain layer?
I initially thought it was similar to the statistical areas, and I could merge some of the subareas, but it is not the same. Some boundaries are different, so it would be great to access the correct layer, instead of attempting to recreate it from what I can see in your website.

The MPA Planning Domains shapefiles have been added to the repository and is included in release v0.5.0


fix(types) - add children for Marker, GoogleMaps, InfoWindow

PLEASE CHECK "Allow edits from maintainers"
help
FOLLOW "Conventional Commits Specification" FOR ALL COMMITS
https://conventionalcommits.org/

Background
I am using Typescript 4.8.4 and getting a type error when trying to use Marker and InfoWindow as children of the GoogleMap component.
Fix
Add children as an optional prop of type ReactNode to the interfaces for these components.
Additional
Please feel free to make a suggestion for how to better fix or how to change my code as to not need to apply this fix.


reloadIgnoringCache is deprecated

reloadIgnoringCache is deprecated in electron v.035

Duplicate of https://github.com/sindresorhus/electron-debug/pull/23


E2E testing framework

Description
This PR proposes an e2e testing mechanism using the bats framework. (You can install it with dnf install -y bats)
Some of its benefits:

Lazy people friendly. Only bash knowledge is required.
Code re-usability. Capability of reusing setup() and teardown()  functions across all tests besides of other common functions like checking ES indexing and artifact generation, etc.
Can run in parallel mode, across test files or within files -> Resulting in a high speed up of the e2e testing of this repo
Unit-test like implementation.

Working example:
$ bats --no-parallelize-across-files -j 8 fio.bats
 ✓ fio-standard
 ✓ fio-bsrange
 ✓ fio-hostpath
 ✓ fio-ocs-cachedrop
4 tests, 0 failures

Failing example (after adding an errata to one of the Vegeta  CRs) :
rsevilla@wonderland ~/ripsaw/e2e (e2e-testing) $ bats --no-parallelize-across-files -j 8 vegeta.bats
 ✓ vegeta-standard
 ✗ vegeta-hostpath
   (from function `die' in file helpers.bash, line 90,
    in test file vegeta.bats, line 25)
     `check_benchmark 300 || die "Timeout waiting for benchmark/${CR_NAME} to complete"' failed
   benchmark.ripsaw.cloudbulldozer.io/vegeta-benchmark-hostnetwork created
   Waiting for UUID from vegeta-benchmark-hostnetwork
   Timeout waiting for benchmark/vegeta-benchmark-hostnetwork to complete
   Dumping logs at artifacts/vegeta-benchmark-hostnetwork
   Saving log from pod vegeta-0-be7508d6-prrbw in artifacts/vegeta-benchmark-hostnetwork/vegeta-0-be7508d6-prrbw.log
   Saving log from pod vegeta-1-be7508d6-l67lt in artifacts/vegeta-benchmark-hostnetwork/vegeta-1-be7508d6-l67lt.log
   benchmark.ripsaw.cloudbulldozer.io "vegeta-benchmark-hostnetwork" deleted

2 tests, 1 failure
$ ls -lh artifacts/vegeta-benchmark-hostnetwork/
total 1.3M
-rw-rw-r--. 1 rsevilla rsevilla 1.2M Aug  3 14:22 benchmark-controller-manager.log
-rw-rw-r--. 1 rsevilla rsevilla 2.3K Aug  3 14:22 vegeta-0-be7508d6-prrbw.log
-rw-rw-r--. 1 rsevilla rsevilla 2.3K Aug  3 14:22 vegeta-1-be7508d6-l67lt.log
-rw-rw-r--. 1 rsevilla rsevilla 5.9K Aug  3 14:22 vegeta-benchmark-hostnetwork.yaml

When manually triggering the github actions workflow we can set a custom image location and select what tests we want to run

Next steps:

[x] Add remaining test cases.
[x] Integration with github actions.
[x] Safe pull request container build mechanism.
[x] Mechanism to upload artifacts generated (pod logs and benchmark YAML), github workflow run.
[x] Documentation

The workflow is executed when in the following conditions:

Pull request with label "ok to test"
Commit merged in master branch
Manually triggered from the "actions" tab of the repository. This option is better described in the documentation

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e991ac4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #625   +/-   ##
=========================================
  Coverage          ?   53.48%
=========================================
  Files             ?        8
  Lines             ?      316
  Branches          ?        0
=========================================
  Hits              ?      169
  Misses            ?      147
  Partials          ?        0

Flag
Coverage Δ

gha
53.48% <0.00%> (?)

python-3.9
53.48% <0.00%> (?)

unit
53.48% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 e991ac4...4d9b180. Read the comment docs.

!!!     Couldn't read commit file !!!


Cannot run Cypress 4.0.0 and CVU together with the same bundle.

Hello,
I tried using the Chef option to install Cypress and CVU side by side since its pretty much hassle-free after many trials and errors trying to manually set up everything. This looks like it worked and I could successfully import the bundle (bundle-2018.zip) through the Cypress GUI but this does not work in CVU. Ngninx/CVU gives a familiar error (502) that I've only seen happen when I have a CVU instance installed without a bundle.
I've seen two conflicting views on this page:

comment by okeefm stating that CVU and Cypress can use the same bundle.

Chef installation seems to imply that you need to import the bundle for both or doesn't really clarify the case of installing both together.

Is there anything I'm missing? I have a clean Ubuntu machine, with a chef install of the Cypress+CVU option. How do I get the CVU working?
-----------------------------EDIT------------------------------
Clicking on the "Validation Utility" tab in the Cypress 4 GUI at localhost redirects me to where the CVU should be running localhost:8080
This is the default config file in sites-enabled folder in NGINX left untouched:
upstream primary {
  server <IP_ADDRESS>:8000;
}

upstream secondary {
  server <IP_ADDRESS>:8001;
}

server {
  listen 80;
  server_name _;

  root /opt/cypress/public;

  try_files $uri/index.html $uri @app;

  location @app {
    proxy_pass http://primary;
    proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
    proxy_set_header Host $http_host;
    proxy_redirect off;
  }

  location /cvu {
    return 301 http://$host:8080;
  }

  error_page 500 502 503 504 /500.html;

server {
  listen 8080;
  server_name _;

  root /opt/cypress-validation-utility/public;

  try_files $uri/index.html $uri @app;

  location @app {
    proxy_pass http://secondary;
    proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
    proxy_set_header Host $http_host;
    proxy_redirect off;
  }

  error_page 500 502 503 504 /500.html;
  client_max_body_size 4G;
  keepalive_timeout 10;
}

Do I run this command on a clean Ubuntu install or on your Cypress 4.0.0 VM?

@geddawi I see there are 2 additional comments that you made which seem to be missing from GitHub (not sure if this is due to the downtime yesterday or not). Are those comments still relevant?

@rbclark No, I had an internal problem which I thought was a problem with your Chef install. So I deleted them so I don't cause you guys any hassle. Thanks for your support, you can close the issue now if you wish! I had a few questions about the actual platform though if you have any spare time, I can send you an email!


Shape disappearing from SVG file after being optimized

The following SVG file:
<svg
    version="1.1"
    id="Layer_1"
    xmlns="http://www.w3.org/2000/svg"
    xmlns:xlink="http://www.w3.org/1999/xlink"
    x="0px"
    y="0px"
    width="69.964px"
    height="11.535px"
    viewBox="0 0 69.964 11.535"
    enable-background="new 0 0 69.964 11.535"
    xml:space="preserve"
>
    <g>
        <path
            fill="#ffffff"
            d="M32.988,8.093c-0.587,0-1.062,0.478-1.062,1.06c0,0.586,0.475,1.059,1.062,1.059c0.584,0,1.058-0.473,1.058-1.059C34.045,8.57,33.572,8.093,32.988,8.093z"
        />
    </g>
</svg>

Results in the following SVG when being run through svgo -i file.svg:

<svg xmlns="http://www.w3.org/2000/svg" width="69.964" height="11.535" viewBox="0 0 69.964 11.535"><path fill="#fff" d="M32.988 8.093a1.06 1.06 0 1 0 0 0z"/></svg>

If I run it through svgo -p 4 -i file.svg instead, I get similar to original output (with no loss of the shape):

<svg xmlns="http://www.w3.org/2000/svg" width="69.964" height="11.535" viewBox="0 0 69.964 11.535"><path fill="#fff" d="M32.988 8.093c-.587 0-1.062.478-1.062 1.06 0 .586.475 1.059 1.062 1.059.584 0 1.058-.473 1.058-1.059-.001-.583-.474-1.06-1.058-1.06z"/></svg>

Before version 0.6.0 (latest being 0.5.6) svgo would not mangle the shape without explicitly setting the precision argument to something higher than the default. I can tell that some changes have happened in between 0.5.6 and 0.6.0 with relation to the precision argument. I admittedly don't understand the full consequence by the changes, but it worries me that the default options on the CLI tool is not lossless but will actually do destructive optimizations to the image.

The original SVG is similar to the second image example, but there's no background color on it, so I added the green one to make the changes more obvious.

Hmm, there should be two arcs making a circle. Something went wrong there.

@GreLI Sorry, I didn't quite understand if you refer to something going wrong in svgo or in my original svg :smile:

@GreLI ping :-) Do you need anything for me, or is this a bug in svgo which you have all the information necessary to dig into?

Sorry, haven't dig into it yet, rather busy lately.

I was about to complain about the rather abrupt closing of this issue...
But I went on to verify if this bug is still there, and I am happy to report that the latest version (2.2.0) will not mangle the circle.

Oops, sorry. Forgot to write that I could not reproduce this issue 😅


Add fake implementation of os.path.samefile

This is used in the implementation of shutil.
See #189.

I'm shocked that this does not already exist.  This is important.

Yes - I thought I implemented this one, but it turned out that was only in fake_pathlib.


Adding software, prepping .NET Core cache, and accepting EULA for sta…

…ndard image

@thechrisjohnson, you should update ubuntu/versioned/Dockerfile.template. Once you do that and run update.sh, it will automatically fix up the TFS2017 Dockerfiles.

@stepro thanks! I added the TFVC license acceptance code the the versioned Dockerfile template as well.

Looks good. I'll rebuild and push the images.


Disable triple cmd (sticky window)

It would be great if you could disable the triple cmd sticky window in preferences.
I find it triggers too easily when I'm thinking / hesitating and have my thumb on the cmd key and as there's no native way to remap the trigger I'd like to disable it if possible?

Sure, I would include possibility to disable it.

Got the latests update, that's awesome! Thank you so much!


vanilla-latest still points to 1.3

I was so confused :)

Yes, and give an error with an 1.4 map!
i'm using right now the specific version tag, like vanilla-<IP_ADDRESS> until this is fixed

@juanrodgil Does it work for you ? Im getting some errors with my docker-compose after
`Settling liquids 49%
Settling liquids 50%
Terraria Server v<IP_ADDRESS>
Listening on port 7777
Type 'help' for a list of commands.
: ================
05/17/2020 09:00:15: Unhandled Exception
Thread: 6 [Server Input Thread]
Culture:
Exception: System.NullReferenceException: Object reference not set to an instance of an object
at Terraria.Main.startDedInputCallBack () [0x00017] in <4ca1e6c0de0846809f149ffc19998e7b>:0
at System.Threading.ThreadHelper.ThreadStart_Context (System.Object state) [0x00017] in <1170fb08300b4d0db71c1d1db8555768>:0
at System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x0008d] in <1170fb08300b4d0db71c1d1db8555768>:0
at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x00000] in <1170fb08300b4d0db71c1d1db8555768>:0
at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state) [0x00031] in <1170fb08300b4d0db71c1d1db8555768>:0
at System.Threading.ThreadHelper.ThreadStart () [0x0000b] in <1170fb08300b4d0db71c1d1db8555768>:0
[ERROR] FATAL UNHANDLED EXCEPTION: System.NullReferenceException: Object reference not set to an instance of an object
at Terraria.Main.startDedInputCallBack () [0x00017] in <4ca1e6c0de0846809f149ffc19998e7b>:0
at System.Threading.ThreadHelper.ThreadStart_Context (System.Object state) [0x00017] in <1170fb08300b4d0db71c1d1db8555768>:0
at System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x0008d] in <1170fb08300b4d0db71c1d1db8555768>:0
at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) [0x00000] in <1170fb08300b4d0db71c1d1db8555768>:0
at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state) [0x00031] in <1170fb08300b4d0db71c1d1db8555768>:0
at System.Threading.ThreadHelper.ThreadStart () [0x0000b] in <1170fb08300b4d0db71c1d1db8555768>:0 `

This works for me:
`version: '3.5'
services:
terraria:
container_name: terraria
restart: unless-stopped
image: beardedio/terraria:vanilla-<IP_ADDRESS>
environment:
world: LizardWizard.wld
stdin_open: true
tty: true
ports:
- 7777:7777
volumes:
- ${CONFIG_PATH}/terraria:/config
networks:
- default
networks:
default:
driver: bridge`
I dont know if the stdin and tty are really necessary, but i see them on other docker compose examples for terraria

@juanrodgil You are the best, tty and stdin_open did the trick, thanks !!

My docker compose works with @juanrodgil's example as well

Sorry about that. My build script was missing the -lastest. This has been fixed and pushed to docker hub. Thanks!


Support for Desktop Bridge for light up icon

This will be handy when discard changes dialog will appear, as the dialog cannot appear when app is not in foreground (I suppose uwp limitation).
Here a sample from original Notepad

Not sure what you mean. What are the exact steps to trigger that behavior in the classic Notepad?

Closing Notepad from the taskbar while using another app

I can't reproduce this behavior on 1903. I have seen this behavior in some Win32 apps but considering that Microsoft has been making changes to Notepad, it's possible it was changed.

It seems this feature has been removed and in the overall it doesn't worth to implement Desktop Bridge just for this small feature


Updated the repos.

This is a great resource for anyone looking to prepare for an Azure Data Factory interview! It will be a great addition to this repos for learners.

Thanks for submitting this PR @letscode-17, and for working to help us make the docs better!  Since this is an external link from the Microsoft site, we can't add it to our public docs, but I do think it's a great resource, and you can add it to the Stack Overflow ADF Questions page.  Since we can't merge it, I will go ahead and close this one out, but please continue to submit PRs anytime you see opportunities to improve the docs!
#please-close


Always reinstalling Winget and User prompt issue

I'm encountering an issue where every time I use Winutil, it reinstalls winget, even if I keep it open and just install another program. Despite being on version 1.8.1911, it keeps indicating that winget is outdated.

also the most annoying thing is that for every program that I try to install it asks for a User credential prompt even tho I'm opening it as admin, how do I get rid of this user credential prompt?

Hey @Sozachiago
The user promt is not because it needs to run as admin, quite the other way around.
Some applications do not like being installed as admin, which is why this is a "fallback" installs the respective application not in an admin promt. hope this clears up the confustion.
Still the main issue here is not addressed. The Winget Update does not seem to work correctly and that might also be the reason why the installs do not work as expected and want to try it using a non admin shell.
Just from looking at the image it looks like it does not know what the latest winutil release is, you have v1.2.10691 and "latest one" is v1.8.1911.

Hey @Sozachiago
The user promt is not because it needs to run as admin, quite the other way around. Some applications do not like being installed as admin, which is why this is a "fallback" installs the respective application not in an admin promt. hope this clears up the confustion.
Still the main issue here is not addressed. The Winget Update does not seem to work correctly and that might also be the reason why the installs do not work as expected and want to try it using a non admin shell.
Just from looking at the image it looks like it does not know what the latest winutil release is, you have v1.2.10691 and "latest one" is v1.8.1911.

Turns out it was the Winget version causing the issue, like you said. It was showing 1.2.10691, but I thought WinUtil had updated it since it kept trying every time I opened it. I manually updated it to 1.8.1911, and now it's working. Thanks a lot for the help <3 and sorry for taking up your time


check for inheritance in list_objects_recursive

following the check in rootls, check not only for exact class matches but also inheritance.
This catches the corner cases of e.g. TNtupleD

Coverage remained the same at 99.693% when pulling 9b68706a9ec1745791f8e06d5d021c3c2689d913 on pseyfert:check_inheritance into d8155933a33f69b1d682f3fd39722f1dd89acf98 on rootpy:master.

Coverage remained the same at 99.693% when pulling 4c6265eee48905919fc0d2c13e3e59165c9fd112 on pseyfert:check_inheritance into d8155933a33f69b1d682f3fd39722f1dd89acf98 on rootpy:master.

Thanks for pointing this out. I've got a cython-level implementation in #249

fine with me. then close this, and go for #249.


Add Forgot password in Login If user Forgot

Issue:
Currently, there is no option for users to reset their password if they forget it, which can lead to frustration and hinder access to their accounts.
Proposed Solution:
Implement a "Forgot Password" link on the login page. This link should direct users to a password recovery process, where they can enter their registered email address to receive instructions for resetting their password. This enhancement will improve user accessibility and overall experience.

Do you want to fix it ?

Hi! I'd love to take on this issue and implement the "Forgot Password" feature. I have a solution in mind where users can reset their password via email by entering their registered email address on the login page. Could you please assign this issue to me? Looking forward to contributing!

Hi! I’d like to work on the "Forgot Password" feature. I'll use Nodemailer to send a password reset link to users when they enter their registered email. This will make it easier for users to reset their passwords. Could you assign this issue to me? Looking forward to helping!

Currently only google login is working properly
The backend for the sign up and login exists but is not connected properly
would you like to fix that as well ?

Yeah ! I can go with that also..

@Mdraza78 @akash-inft1905 do you both wanna work together ?
Or @Mdraza78  should I give it to @akash-inft1905  ?

I’d prefer to work on this alone, but if @Mdraza78  would like to collaborate, I’m open to teaming up.
Let me know how you'd like to proceed!

@Mdraza78 what do you want ?

Currently only google login is working properly The backend for the sign up and login exists but is not connected properly would you like to fix that as well ?

yes!! for sure

I’d prefer to work on this alone, but if @Mdraza78 would like to collaborate, I’m open to teaming up. Let me know how you'd like to proceed!

ok bro no problem you can do by your own

@Vineetchelani1304 Please list down all the features you will like to add, then I'll assign you.

Fix Sign-Up and Login:
Debug and connect the frontend with the existing backend for proper sign-up and login functionality.
Forgot Password Feature:
Implement password recovery using NodeMailer for users to reset their passwords via email.

hey @MinavKaria , I'll do all the things like     1)Debug and connect the frontend with existing backend for credential based signup and login,     2)changing the password in case of forgot password,    today itself, assign it to me

@ArthKulkarni93 I will assign the task to @Vineetchelani1304 Right now because he asked first and to be fair. If in few days he doesn't send the PR , I'll assign it to @ArthKulkarni93

@Vineetchelani1304 Keep in mind that both Google and normal auth should work

@Vineetchelani1304 any output ?

hello sir, is this issue free? if yes i would like to work on it. my idea:

working on backend bugs as mentioned not connected properly
writing backend routes for forgot password
implementing otp based forgot password functionality
user enters email
otp sent to users email
user enters otp
otp is verified on backend
user enter new password, password updates
designing frontend for forgot password


Aspire Redis Commander setup issue

Describe the issue or suggestion
Hi
I was exploring Aspire by following a guide at https://github.com/csharpfritz/BigBadBlog/blob/main/docs/1-Introduction.md. However, when I try to add Redis Commander to Aspire, it seems like the port setup to Redis on the commander container is not correct? It refuses to connect to the Redis instance if I query inside the Commander.
Screenshot below:

Logs:

See https://github.com/dotnet/aspire/issues/5584


⚠ GitLab Server has degraded performance

In 19d98c3, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6344 ms

Resolved: GitLab Server performance has improved in 4540787.

Resolved: GitLab Server performance has improved in 4540787.


Unable to download simplified-Chinese model

The link in readme is useless. And I can't find  simplified-Chinese model  in NLPL Vectors Repository.
Thx.

We also provided simplified-Chinese ELMo. It was trained on xinhua proportion of Chinese gigawords-v5, which is different from the Wikipedia for traditional Chinese ELMo.

我这也是下载不了的

有哪位好心人先前曾经下载过这个简体中文模型的，可以放到云端给我们下载么？版主似乎很久都没有再回复这里了......。
Thanks！


safety-cli: 2.2.1 -> 2.3.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/pyupio/safety/releases.
meta.description for safety-cli is: "Checks installed dependencies for known vulnerabilities"
meta.homepage for safety-cli is: "https://github.com/pyupio/safety"
meta.changelog for safety-cli is: "https://github.com/pyupio/safety/blob/2.3.1/CHANGELOG.md"
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.3.1 with grep in /nix/store/qdnc875hgcl5n3p4c98lyv6sv3cgr8sk-python3.10-safety-2.3.1
found 2.3.1 in filename of file in /nix/store/qdnc875hgcl5n3p4c98lyv6sv3cgr8sk-python3.10-safety-2.3.1

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.safety
python39Packages.safety
safety-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qdnc875hgcl5n3p4c98lyv6sv3cgr8sk-python3.10-safety-2.3.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qdnc875hgcl5n3p4c98lyv6sv3cgr8sk-python3.10-safety-2.3.1
ls -la /nix/store/qdnc875hgcl5n3p4c98lyv6sv3cgr8sk-python3.10-safety-2.3.1/bin

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

2 packages built:

safety-cli (python310Packages.safety)
python39Packages.safety

Maintainer pings
cc @thomasdesr @dotlambda for testing.

nixpkgs-review fails on it
error: build log of '/nix/store/p74513g499pmsa3ca0xdwxpgf023nlfs-python3.10-safety-2.3.1.drv' is not available
error: build log of '/nix/store/fba051lr1ilvrpcdmzxd2n4gvgxk7cw6-python3.9-safety-2.3.1.drv' is not available

However I can build both packages from the PR checkout.

done in https://github.com/NixOS/nixpkgs/pull/200901


Static Webpack Server as part of ui.frontend

Enhancement
This is an enhancement but it would be nice to include a static webpack dev server as part of the ui.frontend module. This would allow a front end developer to develop CSS/JS against static markup, allowing the FED to be further de-coupled from AEM if desired.

fixed in https://github.com/adobe/aem-project-archetype/pull/266


sfrax apr not showing

sfrax apr not showing on OP https://op.beets.fi/pool/0x2feb76966459d7841fa8a7ed0aa4bf574d6111bf00020000000000000000011d
Maybe other added YB aprs are also not working?

Tracked it down to the token price being 0.
Main difficulty in debugging is the circular dependency in the config, but at least we've got a specific case for refactoring.
Calling:
const service = new YbTokensAprService(...)
service.updateAprForPools(pools);

results in:
TypeError: yb_tokens_apr_service_1.YbTokensAprService is not a constructor
    at Object.<anonymous> (/Users/jareknowotka/Developer/balancer/backend/modules/network/mainnet.ts:368:9)
    at Module._compile (node:internal/modules/cjs/loader:1256:14)

thanks for checking! I believe they are in the process of listing on coingecko


Updated mismatched ymal title and header in cmdlet in StorageSubsystem

#2078

Docs Build status updates of commit c70afc6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/storage/Get-StorageSubSystem.md
:white_check_mark:Succeeded

docset/windows/storage/Set-StorageSubSystem.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/storage/Get-StorageSubSystem.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/storage/Set-StorageSubSystem.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Package request: python35Packages.i3ipc

Issue description
i3ipc python package exists in pypy but not in nixpackages.
It would be great if we could use it.
Link for reference: https://pypi.python.org/pypi/i3ipc
Steps to reproduce
    nix-shell -p python35.i3ipc --run "python3 -c 'import i3ipc'"

Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 4.9.80, NixOS, 17.09.2955.76614894f34 (Hummingbird)`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 1.11.16`
 - channels(cypher): `""`
 - channels(root): `"nixos-17.09.2955.76614894f34"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs`

Thanks.
I'm probably up for writing the nix package definition if someone can point me in the right direction,

nix-shell -p python35.i3ipc --run "python3 -c 'import i3ipc'"

There are two mistakes in that line. But, your intention is clear ;)

@FRidh for future reference, what mistakes did I make? It's my first week of nix (and I'm loving it).

nix-shell -p python35.i3ipc --run "python3 -c 'import i3ipc'"

Python packages are part of the pkgs attribute of the interpreter, so python35.pkgs.i3ipc is the reference to the package.
While it is now possible to use
nix-shell -p python35.pkgs.i3ipc --run "python3 -c 'import i3ipc'"

the supported solution is
nix-shell -p 'python35.withPackages(ps: with ps; [ i3ipc ])' --run "python3 -c 'import i3ipc'"

The other one typically works, but not always because python is not always propagated and thus unavailable. Also, withPackages guarantees that you will have the Python that is "connected" to the package, and not accidentally the one that was in your PATH.

It's my first week of nix (and I'm loving it).

Good to hear!

Thanks! Interesting to see the differences.

Just wanted to let you know that I finally got my i3 setup working near perfectly on nixos!
Loving it (Thank you all)


Right panel close button styling

Apply similar styling to right panel close button as suggested in #35

Can do in next release.


wasmWorker example: Add option to receive empty response when QR not found

Issue #, if available:
N/A
Description of changes:
In my own app, I was unable to send messages to the worker too frequently, lest I send messages while it's still working, leading to the worker stalling over a pile of work that it can never catch up on, delaying the time it takes to actually register a successful scan.
In my use case, I was able to remedy this by having the worker respond with data/type undefined, and then once the app receives word that the worker is no longer busy, it's free to send another snapshot from the camera. This significantly improved the performance of my scanner, because I can essentially always keep the worker busy & not be afraid of piling it with more work than it can handle.
I added it under the optional event.data.alwaysRespond, because this is a change in the API behavior, and therefore the programmer must update the way they call the worker in order to take advantage of this change. (i.e., if you request alwaysRespond: true, then you can no longer assume that data is present when receiving a message from the worker. So it makes sense to me to have this an "opt-in" change.
Side note: I noticed that public/jsQrWorker.js has a console.log statement where wasmWorker doesn't. I didn't include a change to it in this PR cuz it's outs of the scope of this issue, but I thought I'd mention it if you didn't realize it was there.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Commit https://github.com/maslick/koder/pull/87/commits/112272897de530fde93333d21278dcf681dc0de3 was because I saw a diff on the last line of jsQrWorker.js, so I thought it was because i accidentally deleted a newline at the end of the file, but the diff is still there. Not sure what the deal is with that.

Hi, why can't you adjust the scan rate (see https://github.com/maslick/koder/blob/master/src/index.js#L8)? The default is 250 ms, meaning the app sends 4 images / sec to the Web worker. From my experience (mostly with iPhones 11 Pro, 13 Pro, 14 Plus), koder is capable to process images at 10 ms (100 images / sec). I find 250 ms is optimal from the performance / usability, anything below 100 ms is just a waste of resources.
Hey there, why can't you adjust the scan rate (see https://github.com/maslick/koder/blob/master/src/index.js#L8)? It's set to 250 ms by default, resulting in 4 images per second sent to the Web worker. In my experience, especially with iPhones 11 Pro, 13 Pro, and 14 Plus, koder can handle image processing at 10 ms (100 images per second) without any problem. I personally find 250 ms optimal for balancing performance and usability, anything below 100 ms seems like a bit of a resource overkill. What are your thoughts on adjusting the scan rate?

The app I'm working on may be run on a wide variety of devices, including older and slower phones. Also, sometimes the device/browser may give a video stream that's a bit higher resolution than expected, which takes longer to parse. For me personally, "blindly" adjusting the scan rate without the main thread receiving any concrete feedback on how long the parsing really takes makes me really uncomfortable. When I was playing with different video resolutions, I was encountering issues where the web worker was overloaded with requests as described above, and I didn't want that to happen again, so I turned it up to 500 ms but then found that the scanning experience was terrible and inconsistent. For me, being able to know for sure when the web worker has stopped scanning the last frame gives me the peace of mind necessary to push it further with more frames.
That being said, you're probably right about it being a bit overkill to crank up the frequency too high, so I might cap the scan rate anyways instead of requesting a new scan as soon as it's possible. I'll do some tests to see where the point of diminishing returns is, and cap it there.
About the PR: I've copied the WASM and JS into the project's source (with my suggested changes) so I'm not dependent on the PR being accepted & uploaded to a new version on NPM. I just figured I'd submit it as PR because it's an upgrade IMO, in case others would benefit from it too. If you don't want to merge it, that's alright!


Use master as dependencies on the example

Nice, I'll probably apply the same pattern to react-native-modal-datetime-picker.
Thanks!


Failed to load addins error when using RDP file which contains drivetoredirect:s

When connecting to a Windows machine using an RDP file that has been generated to use in Windows environment and has the option drivestoredirect:s: with drives in Windows format (ie, drivestoredirect:s:c), Freerdp exits with error:
[ERROR][com.freerdp.client.common] - [freerdp_client_load_channels]: Failed to load addins [00000000]
Steps to reproduce the behavior:

Create an RDP file with the line drivestoredirect:s:c
Execute xfreerdp <rdp_file.rdp>
See error

Expected behavior
A better error message to point out that the format of drivestoredirect:s is not compatible with Linux, as the current error message points to a missing or broken configuration (as in https://github.com/FreeRDP/FreeRDP/issues/9591)
or
Instead of an error and exit, issue a warning, ignore the drivetoredirects:s line in the rdp file and continue to connect to remote machine
Application details

FreeRDP version xfreerdp all versions
Command line used: xfreerdp <rdp_file.rdp>
Output of xfreerdp /buildconfig:  Any build configuration will fail on this
OS version connecting to (server side): Any Windows version

Environment (please complete the following information):

OS: Linux
Version/Distribution: Any distro will fail
Architecture: amd64

Downstream bug: https://bugs.gentoo.org/926879


Support for Linux 20.04

Hello there,
When are you planning to support Linux 20.04 and ros2 Foxy, ros1 noetic?

Any update on support?

Any update on support?

Hi  Ranjitkathiriya,
It's on the way, we will release a new version at the end of this month.
Please be patient, thanks.

Hello there,
When are you planning to support Linux 20.04 and ros2 Foxy, ros1 noetic?

Thanks for this comment, I hope the new version will be easy to install and every step of the installation will be well documented.
Thanks, once again for helping.

https://github.com/lips-hci/ae400-realsense-sdk/releases/tag/AE400_v<IP_ADDRESS>_rs2.36.0
This is working fine in ubuntu 20.04.


🚑 adapt to FB data change

FB changed some data format, spotted by @kartol but not fixed by him :(

removed thread_type which we used to distinguish between groupchat and regular chat (current fix assumes regular chat == 2 participants, groupchat != 2)
removed type from message (so I removed call duration count, we didn't use it anyway)

Stopped indexing chats with deleted accounts - since you don't know the other participant, it doesn't really make sense to show it (and it also causes some weird behavior which I can avoid fixing 😄)


NR filter_chain_not_found

Bug Description
I am facing error while adding domain name in Gateway host, it is working for wildcard, but for domain name it is showing error: filter_chain_not_found
Configuration:
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
name: jenkins-gw
namespace: jenkins
annotations:
ingress.kubernetes.io/force-ssl-redirect: "true"
kubernetes.io/ingress.class: "istio-gateway"
spec:
selector:
istio: ingressgateway
servers:
- port:
number: 443
name: https
protocol: HTTPS
hosts:
- "jenkins.shared.huru.co"
tls:
mode: ISTIO_MUTUAL

apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
name: jenkins-vs
namespace: jenkins
spec:
gateways:
- jenkins-gw
hosts:
- jenkins.shared.huru.co
http:
- match:
- uri:
prefix: /
route:
- destination:
host: jenkins.jenkins.svc.cluster.local
port:
number: 8080
ERROR:
Istio-gateway-log:
2022-12-27T22:07:16.522359Z	info	Envoy proxy is ready
2022-12-27T22:07:16.558143Z	info	envoy upstream	cds: add 21 cluster(s), remove 5 cluster(s)
2022-12-27T22:07:16.560269Z	info	envoy upstream	cds: added/updated 0 cluster(s), skipped 21 unmodified cluster(s)
[2022-12-27T22:08:14.102Z] "- - -" 0 NR filter_chain_not_found - "-" 0 0 0 - "-" "-" "-" "-" "-" - - <IP_ADDRESS>:443 <IP_ADDRESS>:1544 - -
[2022-12-27T22:08:14.258Z] "- - -" 0 NR filter_chain_not_found - "-" 0 0 0 - "-" "-" "-" "-" "-" - - <IP_ADDRESS>:443 <IP_ADDRESS>:30120 - -
Version
version: 1.16.1

Additional Information
istioctl bug-report -n jenkins
Target cluster context: arn:aws:eks:me-central-1:365834278172:cluster/shared-eks
Running with the following config:
istio-namespace: istio-system
full-secrets: false
timeout (mins): 30
include: {  }
exclude: { Namespaces: kube-node-lease,kube-public,kube-system,local-path-storage }
end-time: 2022-12-27 22:15:43.943685601 +0000 UTC
Cluster endpoint: https://3ED77DBA7E1D7564F9661FB88AD66EDC.gr7.me-central-1.eks.amazonaws.com
CLI version:
version.BuildInfo{Version:"1.16.0", GitRevision:"8f2e2dc5d57f6f1f7a453e03ec96ca72b2205783", GolangVersion:"go1.19.3", BuildStatus:"Clean", GitTag:"1.16.0"}
The following Istio control plane revisions/versions were found in the cluster:
Revision default:
&version.MeshInfo{
{
Component: "pilot",
Info:      version.BuildInfo{Version:"1.16.1", GitRevision:"tetratefips-v0-bcb41746650e2eb3d8d0aa7624a3a1ca80b6fec0", GolangVersion:"", BuildStatus:"Clean", GitTag:"1.16.1"},
},
}
The following proxy revisions/versions were found in the cluster:
Revision default: Versions {1.16.1-tetratefips-v0, 1.15.3-tetratefips-v0}
Fetching proxy logs for the following containers:
cert-manager//cert-manager-csi-driver-9xtgt/cert-manager-csi-driver
cert-manager//cert-manager-csi-driver-9xtgt/liveness-probe
cert-manager//cert-manager-csi-driver-9xtgt/node-driver-registrar
cert-manager//cert-manager-csi-driver-z6wkg/cert-manager-csi-driver
cert-manager//cert-manager-csi-driver-z6wkg/liveness-probe
cert-manager//cert-manager-csi-driver-z6wkg/node-driver-registrar
cert-manager/cert-manager-cainjector/cert-manager-cainjector-86f7f4749-g4zqk/cert-manager-cainjector
cert-manager/cert-manager-webhook/cert-manager-webhook-66c85f8577-zt5bz/cert-manager-webhook
cert-manager/cert-manager/cert-manager-7d57b6576b-4ltk4/cert-manager-controller
istio-system//istio-cni-node-7qvw5/install-cni
istio-system//istio-cni-node-qn775/install-cni
istio-system/istio-ingressgateway/istio-ingressgateway-685fcc78c-2h5bj/istio-proxy
istio-system/istiod/istiod-54bcb75b55-gzh7d/discovery
jenkins//jenkins-0/config-reload
jenkins//jenkins-0/istio-proxy
jenkins//jenkins-0/jenkins
jfrog//jfrog-artifactory-jcr-0/artifactory
jfrog//jfrog-artifactory-jcr-0/istio-proxy
jfrog//jfrog-artifactory-oss-0/artifactory
jfrog//jfrog-artifactory-oss-0/istio-proxy
Fetching Istio control plane information from cluster.
Running istio analyze on all namespaces and report as below:
Analysis Report:
Warning [IST0105] (Pod jfrog/jfrog-artifactory-jcr-0) The image of the Istio proxy running on the pod does not match the image defined in the injection configuration (pod image: containers.istio.tetratelabs.com/proxyv2:1.15.3-tetratefips-v0; injection configuration image: containers.istio.tetratelabs.com/proxyv2:1.16.1-tetratefips-v0). This often happens after upgrading the Istio control-plane and can be fixed by redeploying the pod.
Warning [IST0105] (Pod jfrog/jfrog-artifactory-oss-0) The image of the Istio proxy running on the pod does not match the image defined in the injection configuration (pod image: containers.istio.tetratelabs.com/proxyv2:1.15.3-tetratefips-v0; injection configuration image: containers.istio.tetratelabs.com/proxyv2:1.16.1-tetratefips-v0). This often happens after upgrading the Istio control-plane and can be fixed by redeploying the pod.
Info [IST0102] (Namespace cert-manager) The namespace is not enabled for Istio injection. Run 'kubectl label namespace cert-manager istio-injection=enabled' to enable it, or 'kubectl label namespace cert-manager istio-injection=disabled' to explicitly mark it as not needing injection.
Info [IST0102] (Namespace default) The namespace is not enabled for Istio injection. Run 'kubectl label namespace default istio-injection=enabled' to enable it, or 'kubectl label namespace default istio-injection=disabled' to explicitly mark it as not needing injection.
Info [IST0118] (Service jenkins/jenkins-agent) Port name agent-listener (port: 50000, targetPort: 50000) doesn't follow the naming convention of Istio port.
Creating an archive at /home/ec2-user/git/infra/terraform/bug-report.tar.gz.
Cleaning up temporary files in /tmp/bug-report.
Done.
[ec2-user@ip-10-1-1-140 terraform]$ istioctl bug-report -n jenkins
Target cluster context: arn:aws:eks:me-central-1:365834278172:cluster/shared-eks
Running with the following config:
istio-namespace: istio-system
full-secrets: false
timeout (mins): 30
include: {  }
exclude: { Namespaces: kube-node-lease,kube-public,kube-system,local-path-storage }
end-time: 2022-12-27 22:17:22.184852012 +0000 UTC
Cluster endpoint: https://3ED77DBA7E1D7564F9661FB88AD66EDC.gr7.me-central-1.eks.amazonaws.com
CLI version:
version.BuildInfo{Version:"1.16.0", GitRevision:"8f2e2dc5d57f6f1f7a453e03ec96ca72b2205783", GolangVersion:"go1.19.3", BuildStatus:"Clean", GitTag:"1.16.0"}
The following Istio control plane revisions/versions were found in the cluster:
Revision default:
&version.MeshInfo{
{
Component: "pilot",
Info:      version.BuildInfo{Version:"1.16.1", GitRevision:"tetratefips-v0-bcb41746650e2eb3d8d0aa7624a3a1ca80b6fec0", GolangVersion:"", BuildStatus:"Clean", GitTag:"1.16.1"},
},
}
The following proxy revisions/versions were found in the cluster:
Revision default: Versions {1.16.1-tetratefips-v0}
Fetching proxy logs for the following containers:
cert-manager//cert-manager-csi-driver-9xtgt/cert-manager-csi-driver
cert-manager//cert-manager-csi-driver-9xtgt/liveness-probe
cert-manager//cert-manager-csi-driver-9xtgt/node-driver-registrar
cert-manager//cert-manager-csi-driver-z6wkg/cert-manager-csi-driver
cert-manager//cert-manager-csi-driver-z6wkg/liveness-probe
cert-manager//cert-manager-csi-driver-z6wkg/node-driver-registrar
cert-manager/cert-manager-cainjector/cert-manager-cainjector-86f7f4749-g4zqk/cert-manager-cainjector
cert-manager/cert-manager-webhook/cert-manager-webhook-66c85f8577-zt5bz/cert-manager-webhook
cert-manager/cert-manager/cert-manager-7d57b6576b-4ltk4/cert-manager-controller
istio-system//istio-cni-node-7qvw5/install-cni
istio-system//istio-cni-node-qn775/install-cni
istio-system/istio-ingressgateway/istio-ingressgateway-685fcc78c-2h5bj/istio-proxy
istio-system/istiod/istiod-54bcb75b55-gzh7d/discovery
jenkins//jenkins-0/config-reload
jenkins//jenkins-0/istio-proxy
jenkins//jenkins-0/jenkins
jfrog//jfrog-artifactory-jcr-0/artifactory
jfrog//jfrog-artifactory-jcr-0/istio-proxy
jfrog//jfrog-artifactory-oss-0/artifactory
jfrog//jfrog-artifactory-oss-0/istio-proxy
Fetching Istio control plane information from cluster.
Running istio analyze on all namespaces and report as below:
Analysis Report:
Info [IST0102] (Namespace cert-manager) The namespace is not enabled for Istio injection. Run 'kubectl label namespace cert-manager istio-injection=enabled' to enable it, or 'kubectl label namespace cert-manager istio-injection=disabled' to explicitly mark it as not needing injection.
Info [IST0102] (Namespace default) The namespace is not enabled for Istio injection. Run 'kubectl label namespace default istio-injection=enabled' to enable it, or 'kubectl label namespace default istio-injection=disabled' to explicitly mark it as not needing injection.
Info [IST0118] (Service jenkins/jenkins-agent) Port name agent-listener (port: 50000, targetPort: 50000) doesn't follow the naming convention of Istio port.
Creating an archive at /home/ec2-user/git/infra/terraform/bug-report.tar.gz.
Cleaning up temporary files in /tmp/bug-report.
Done.
ISTIO namespace log:
istioctl bug-report -n istio-system
Target cluster context: arn:aws:eks:me-central-1:365834278172:cluster/shared-eks
Running with the following config:
istio-namespace: istio-system
full-secrets: false
timeout (mins): 30
include: {  }
exclude: { Namespaces: kube-node-lease,kube-public,kube-system,local-path-storage }
end-time: 2022-12-27 22:18:56.495281432 +0000 UTC
Cluster endpoint: https://3ED77DBA7E1D7564F9661FB88AD66EDC.gr7.me-central-1.eks.amazonaws.com
CLI version:
version.BuildInfo{Version:"1.16.0", GitRevision:"8f2e2dc5d57f6f1f7a453e03ec96ca72b2205783", GolangVersion:"go1.19.3", BuildStatus:"Clean", GitTag:"1.16.0"}
The following Istio control plane revisions/versions were found in the cluster:
Revision default:
&version.MeshInfo{
{
Component: "pilot",
Info:      version.BuildInfo{Version:"1.16.1", GitRevision:"tetratefips-v0-bcb41746650e2eb3d8d0aa7624a3a1ca80b6fec0", GolangVersion:"", BuildStatus:"Clean", GitTag:"1.16.1"},
},
}
The following proxy revisions/versions were found in the cluster:
Revision default: Versions {1.16.1-tetratefips-v0}
Fetching proxy logs for the following containers:
cert-manager//cert-manager-csi-driver-9xtgt/cert-manager-csi-driver
cert-manager//cert-manager-csi-driver-9xtgt/liveness-probe
cert-manager//cert-manager-csi-driver-9xtgt/node-driver-registrar
cert-manager//cert-manager-csi-driver-z6wkg/cert-manager-csi-driver
cert-manager//cert-manager-csi-driver-z6wkg/liveness-probe
cert-manager//cert-manager-csi-driver-z6wkg/node-driver-registrar
cert-manager/cert-manager-cainjector/cert-manager-cainjector-86f7f4749-g4zqk/cert-manager-cainjector
cert-manager/cert-manager-webhook/cert-manager-webhook-66c85f8577-zt5bz/cert-manager-webhook
cert-manager/cert-manager/cert-manager-7d57b6576b-4ltk4/cert-manager-controller
istio-system//istio-cni-node-7qvw5/install-cni
istio-system//istio-cni-node-qn775/install-cni
istio-system/istio-ingressgateway/istio-ingressgateway-685fcc78c-2h5bj/istio-proxy
istio-system/istiod/istiod-54bcb75b55-gzh7d/discovery
jenkins//jenkins-0/config-reload
jenkins//jenkins-0/istio-proxy
jenkins//jenkins-0/jenkins
jfrog//jfrog-artifactory-jcr-0/artifactory
jfrog//jfrog-artifactory-jcr-0/istio-proxy
jfrog//jfrog-artifactory-oss-0/artifactory
jfrog//jfrog-artifactory-oss-0/istio-proxy
Fetching Istio control plane information from cluster.
Running istio analyze on all namespaces and report as below:
Analysis Report:
Info [IST0102] (Namespace cert-manager) The namespace is not enabled for Istio injection. Run 'kubectl label namespace cert-manager istio-injection=enabled' to enable it, or 'kubectl label namespace cert-manager istio-injection=disabled' to explicitly mark it as not needing injection.
Info [IST0102] (Namespace default) The namespace is not enabled for Istio injection. Run 'kubectl label namespace default istio-injection=enabled' to enable it, or 'kubectl label namespace default istio-injection=disabled' to explicitly mark it as not needing injection.
Info [IST0118] (Service jenkins/jenkins-agent) Port name agent-listener (port: 50000, targetPort: 50000) doesn't follow the naming convention of Istio port.
Creating an archive at /home/ec2-user/git/infra/terraform/bug-report.tar.gz.
Cleaning up temporary files in /tmp/bug-report.
Done.

complete version check

@kushalg-1212 :
version: 1.16.1

https://istio.io/latest/docs/ops/common-problems/network-issues/#configuring-sni-routing-when-not-sending-sni is likely

Answered above and no followups so I'll close this for now


Playing sound with volume specified in config does not affect volume

When playing a sound using the latest MPF (0.8 I think) and GMC, the volume has no effect on the volume that the sound plays. I have tried various values ranging from -80 to -1 and from 0.0 to 1.0 (and the default of 0.5) but none seem to have an effect.
sound_player:
  play_song1:
    song1:
      action: play
      volume: -80 # have tried other values such as 0.1, 1.0 (as per description)

My original use case is to "duck" the music that is playing in the background whilst in the bonus mode and then to increase the volume once the bonus mode has been disabled and the players next ball has started. Since I couldn't get ducking to work (even though the logs in Godot indicate that it has ducked for a specific duration at a specific attenuation level), it does not appear to work hence why I am trying to get just volume working to start with whilst I troubleshoot.
Hopefully that makes sense.

See this discussion for some commentary related to this;
https://github.com/orgs/missionpinball/discussions/147

Submitted pull request: https://github.com/missionpinball/mpf-gmc/pull/14

Pull request approved and merged, closing this issue! 🎉


Issue 41981: Inject filterGroups from query xmls defined in fk definition of a column in query-importData.view

Rationale
FK definition in query xml has child element filters that can be utilized to filter column values during insert and update and it was not getting applied as filterArray in query-importData.view because it is named filterGroups. This PR converts them to filterArray in Utils.js.
Related Pull Requests

https://github.com/LabKey/tnprc_ehr/pull/397

@labkey-nicka Can you comment on this? For example, can we use const and let in these JS files? How do we synchronize these changes (if at all) with the versioned JS API? Anything else?


[capi] docs for experiment unarchive

Fixes CON-148

#1579  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @brian-statsig and the rest of your teammates on  Graphite

Merge activity

May 15, 2:01 PM EDT: @brian-statsig merged this pull request with Graphite.

🚀 Deployed on https://6644f8c3d89973590dba01e5--cozy-fox-0defec.netlify.app


Streaming Agent Error -----> TypeError: 'function' object is not iterable

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.

Example Code
from flask import Blueprint, request, Response, current_app, jsonify, stream_with_context
from flask_jwt_extended import jwt_required, current_user
from project import db
from langchain_community.llms import Ollama
from langchain_community.vectorstores import Chroma
from langchain_community.document_loaders.csv_loader import CSVLoader
from langchain_community.embeddings import OllamaEmbeddings
from langchain_core.prompts import ChatPromptTemplate, MessagesPlaceholder, PromptTemplate
from langchain_community.chat_message_histories import MongoDBChatMessageHistory
from langchain.agents import tool, AgentExecutor, create_react_agent
from langchain.tools.retriever import create_retriever_tool
from langchain.agents import AgentExecutor
from langchain_openai import OpenAI
from langchain_openai import OpenAIEmbeddings
import json
from langchain_core.utils.function_calling import convert_to_openai_tool
from langchain_experimental.llms.ollama_functions import OllamaFunctions
from langchain_openai import ChatOpenAI
from langchain.callbacks.streaming_stdout_final_only import FinalStreamingStdOutCallbackHandler
import time
import asyncio
from random import choice

chat_bot_2_Blueprint = Blueprint('chat_bot_2', __name__)
@chat_bot_2_Blueprint.route('/api/ai/chat_bot_2/', methods=['GET', 'POST']) # <- from '/'
@jwt_required()
async def chat_bot_2():

    user_ID = current_user.user_ID
    chat_input = request.json.get("chat_input", None)
    chat_ID = request.json.get("live_chat_ID", None)

    path_to_chroma_db = "./project/api/ai/chroma_db/"
    path_to_csv = "/love_qa.csv"

    loader = CSVLoader(file_path=path_to_csv)
    data = loader.load()

    vector_collection = "dr_love_4"
    model_type = "open_ai"

    if model_type == "open_ai":
        embedding_model = OpenAIEmbeddings(openai_api_key=current_app.config['OPENAI_SECRET'])
        llm = ChatOpenAI(
            openai_api_key=current_app.config['OPENAI_SECRET'],
            model_name="gpt-3.5-turbo-0125",
            streaming=True,
            max_tokens=2000,
            callbacks=[FinalStreamingStdOutCallbackHandler(answer_prefix_tokens=['Final', ' Answer', ':'])],)
    else:
        model_type = OllamaEmbeddings(model="llama2")
        llm = OllamaFunctions(model="llama2")
        print('LLAMA')

    react_template = get_react_template()
    custom_react_prompt = PromptTemplate.from_template(react_template)

    # save to disk
    #vectorstore = Chroma.from_documents(documents=data, embedding=embedding_model, persist_directory=path_to_chroma_db + vector_collection)

    # load from disk
    vectorstore = Chroma(persist_directory=path_to_chroma_db + vector_collection, embedding_function=embedding_model)
    retriever = vectorstore.as_retriever(search_type="similarity", search_kwargs={"k": 2})

    retreiver_title = "hormone_doctor_medical_questions_and_answers"
    retreiver_description = "Searches and returns a series of questions and answers on hormones and medical advice and also personal questions"
    retriever_tool = create_retriever_tool(retriever, retreiver_title, retreiver_description)

    tools = [confetti, get_word_length, retriever_tool]

    message_history = MongoDBChatMessageHistory(session_id=chat_ID, connection_string=current_app.config['MONGO_CLI'], database_name=current_app.config['MONGO_DB'], collection_name="chat_histories")

    agent = create_react_agent(llm=llm, tools=tools, prompt=custom_react_prompt)
    agent_executor = AgentExecutor(agent=agent, tools=tools, verbose=True, max_iterations=5, max_execution_time=100, return_intermediate_steps=True, handle_parsing_errors=True).with_config({"run_name": "Agent"})

    async def stream_output(chat_input):

        # async for chunk in agent_executor.stream({"input": chat_input, "chat_history": message_history}):
        #     print(chunk)
        #     if "output" in chunk:
        #         # print(f'Final Output: {chunk["output"]}')
        #         yield chunk["output"]

        async for event in agent_executor.astream_events({"input": chat_input, "chat_history": message_history}, version="v1"):
            kind = event["event"]
            if kind == "on_chain_start":
                if (event["name"] == "Agent"):  # Was assigned when creating the agent with `.with_config({"run_name": "Agent"})`
                    print(f"Starting agent: {event['name']} with input: {event['data'].get('input')}")
            elif kind == "on_chat_model_stream":
                content = event["data"]["chunk"].content
                if content:
                    print(content, end="|")
                    yield content

        update_chat_history(user_ID, chat_ID, chat_input)

    return Response(stream_with_context(stream_output(chat_input)), mimetype='text/event-stream')

def update_chat_history(user_ID, chat_ID, chat_input):
    preview = None
    chats = current_user.chats
    # Assume chat_ID and user_ID are defined earlier in your code
    chat_exists = any(chat_ID == item.get('chat_ID') for item in chats)
    if not chat_exists:
        db.credentials.update_one({"user_ID": user_ID}, {"$push": {'chats': {'chat_ID': chat_ID, 'preview': chat_input}}})
    else:
        #Move the items to the front of the list
        #get the preview
        for item in chats:
            if item['chat_ID'] == chat_ID:
                preview = item['preview']

    # First, pull the item if it exists
    db.credentials.update_one({"user_ID": user_ID, "chats.chat_ID": chat_ID}, {"$pull": {"chats": {"chat_ID": chat_ID}}})
        # Then, push the item to the array
    db.credentials.update_one({"user_ID": user_ID}, {"$push": {"chats": {"chat_ID": chat_ID, "preview": preview}}})

def get_react_template():

    react_template="""
                Assistant is a large language model trained by OpenAI.

                Assistant is designed to be able to assist with a wide range of tasks, from answering simple questions to providing in-depth explanations and discussions on a wide range of topics. As a language model, Assistant is able to generate human-like text based on the input it receives, allowing it to engage in natural-sounding conversations and provide responses that are coherent and relevant to the topic at hand.

                Assistant is constantly learning and improving, and its capabilities are constantly evolving. It is able to process and understand large amounts of text, and can use this knowledge to provide accurate and informative responses to a wide range of questions. Additionally, Assistant is able to generate its own text based on the input it receives, allowing it to engage in discussions and provide explanations and descriptions on a wide range of topics.

                Overall, Assistant is a powerful tool that can help with a wide range of tasks and provide valuable insights and information on a wide range of topics. Whether you need help with a specific question or just want to have a conversation about a particular topic, Assistant is here to assist.

                TOOLS:
                ------

                Assistant has access to the following tools:

                {tools}

                To use a tool, please use the following format:

                Thought: Do I need to use a tool? Yes
                Action: the action to take, should be one of [{tool_names}]
                Action Input: the input to the action
                Observation: the result of the action

                When you have a response to say to the Human, or if you do not need to use a tool, you MUST use the format:

                Thought: Do I need to use a tool? No
                Final Answer: [your response here]

                Begin!

                Previous conversation history:
                {chat_history}

                Question: {input}

                New input:
                {agent_scratchpad}
        """
    return react_template

@tool
async def confetti(string: str) -> str:
    """Adds a random word to the string"""
    return "confetti " + string

@tool
async def get_word_length(word: str) -> int:
    """Returns the length of a word."""
    return len(word)

@tool
async def where_cat_is_hiding() -> str:
    """Where is the cat hiding right now?"""
    return choice(["under the bed", "on the shelf", "in your heart"])

Error Message and Stack Trace (if applicable)
Debugging middleware caught exception in streamed response at a point where response headers were already sent.
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/werkzeug/wsgi.py", line 500, in next
return self._next()
^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/werkzeug/wrappers/response.py", line 50, in _iter_encoded
for item in iterable:
TypeError: 'function' object is not iterable
Description
I am trying to stream the output from an OpenAI agent using astream or astream_events. Neither work.
I have tried many different potential solutions and would appreciate some guidance.
Thank You.
System Info
System Information

OS:  Darwin
OS Version:  Darwin Kernel Version 23.3.0: Wed Dec 20 21:33:31 PST 2023; root:xnu-10002.81.5~7/RELEASE_ARM64_T8112
Python Version:  3.11.2 (main, Mar 24 2023, 00:31:37) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]

Package Information

langchain_core: 0.1.23
langchain: 0.1.7
langchain_community: 0.0.20
langsmith: 0.0.87
langchain_cli: 0.0.21
langchain_experimental: 0.0.51
langchain_openai: 0.0.6
langchainhub: 0.1.14
langserve: 0.0.41

Packages not installed (Not Necessarily a Problem)
The following packages were not found:

langgraph

hey! were you able to resolve this?


[VM] Fix --no-wait option on availability-set create

Fix #4207.

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[X] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[N/A] Each command and parameter has a meaningful description.
[N/A] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #4243 into master will decrease coverage by <.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master    #4243      +/-   ##
==========================================
- Coverage   70.05%   70.04%   -0.01%
==========================================
  Files         475      475
  Lines       29603    29607       +4
  Branches     4539     4540       +1
==========================================
  Hits        20739    20739
- Misses       7417     7420       +3
- Partials     1447     1448       +1

Impacted Files
Coverage Δ

...zure-cli-vm/azure/cli/command_modules/vm/custom.py
74.39% <0%> (-0.14%)
:arrow_down:

...re-cli-vm/azure/cli/command_modules/vm/commands.py
81.09% <100%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
69.73% <0%> (-0.93%)
:arrow_down:

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 3a8412b...4b3298e. Read the comment docs.


[BUG]

Describe the bug
Whenever I am on the PalWorld map, it does not show me the locations of the Pals that are in the Pro version.
To Reproduce
Steps to reproduce the behavior:

Go to
Click on 'Any Pal except the first 6 which are free.'
See that it does not work.

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
Not working :

And this is how it looks when it works (this one is just free) :

Desktop (please complete the following information):

OS: Windows 11
Browser: Brave and Chrome
Version : v1.62.162(brave) and 121.0.6167.161(chrome)

Additional context
Add any other context about the problem here.

heatmaps are not supported in v2 yet. But I found a way to unlock the feature and will be available in v3.


changes for merged repos

Modifications to iter8 necessary to support merged etc3/iter8ctl/handler repos.
Signed-off-by: Michael Kalantar<EMAIL_ADDRESS>

@sriumcp I don't think we envisioned this to be merged with master. Rather it was meant as proof of concept; that the etc3 changes would work once they were merged. This PR will need to be modified to refer to the tag once we merge and create.
Could you have a look at this in terms of whether or not there is anything missing or if this is sufficient motivation to move forward with the etc3 changes.

@kalantar All checks have passed which IMO is sufficient motivation to move forward with etc3 changes.

@sriumcp one thing I noticed is that we need to update the etc3 version in each test file. There are more places to make sure it is right. Is there a way we can simplify this?  Perhaps a manually executed wf to create PR or something?

You mean due to go get iter8ctl?
I think we may be able to drop the version specification altogether and simply go with the latest. Please create an issue in anycase. I feel we don't need yet another workflow (I would like to get rid of a few of them if possible).


Are single-operand instructions R or I-type?

<Moved from originally misposting in #1257>
In the unpriv-isa-asciidoc_20240411, page 554 or page 563 defines R-type and I-type formats, which differ only in that instr[31:20] = {funct7, rs2} for R-type and imm12 for I-type.
Many instructions such as fsqrt, fcvt, cpop, and zip use a single register, and contain constants in the instr[24:20] that serve as an extended function code rather than an immediate.  It appears that the unprivileged spec is silent about their type. Is there a collective understanding of type for these instructions?
Is it best to classify these instructions as I-type even though the immediate is not used as a source operand?  Or better to call them R-type with rs2 hardwired to a funct5 value?
Sorry this question is pendantic, but I’m a pedant.
Thank you,
David
Andrew Waterman replied:
...But I’ll briefly comment that I-type would also have been a poor classification because there’s no immediate operand. We use instruction types as classification tools, not as strictures. We’d have several dozen distinct types if we used that tool with precision—-and at that point, it wouldn’t be doing anyone any service.

The reason I'm asking is that I've been preparing a RISC-V quick reference summary table similar to the old MIPS green card. Classifying instruction types helps provide a dense layout.  It doesn't need perfect precision because it is quick reference, not an ISA spec.
The following instructions have one register source and no immediate, and I will classify them either as R or I type for the sake of the table. It's the only classification issue common enough to present an issue for the table. I'd prefer to follow a convention if there is any consensus in the community; otherwise I'll make a somewhat arbitrary choice between two imperfect options for the sake of a dense table.
F extension      fsqrt, fclass, fcvt, fmv
Zb* extension  orc.b, zext, clz{w}, ctz{w}, cpop{w}, sext, rev8
A extension       lr
Zfa Extension   fli. fround{nx}, fmvh, fcvtmod
Zbk*/Zkn*        brev8, zip, unzip, aes64im, sha256*, sha512{sig/sum}{0/1}

After rereading Andrew's answer that he doesn't like I-type classification, I think I'll call these instructions R-type with rs2 tied to a constant, and close the issue.

Just to be clear, I don't think calling them I-type is less accurate than calling them R-type--and if you're of the mindset that the register specifiers are the most important feature being classified, calling them I-type might indeed be preferable.  (By contrast, the argument for classifying fcvt and fadd together is that we use bits 31..25 in the same manner for both kinds of instruction, and depending on the value of that field, we know that we can use rs2 as an ancillary opcode field.)  The best choice probably depends on the pedagogical intent.


Allow players to edit select details

I'd like to allow my players to be a bit more self selecting with quests, however if I hand over the ability for them to edit quests then they have full reign to do whatever.
I'd appreciate there being a middleground where they could accept from and perhaps junk the available (sub)quests on their own.

Currently there are two settings:

[Module Configuration] "Allow Players to Accept Quests", with this, players can accept any available quests
[Manage Quest tab] "Allow players edit Details" with this, players can edit everything on Details tab of the quest, they can't add/remove subquests, they can't access Manage Quest tab or see GM Notes.

Currently these 2 are only options. More detailed permission management is planned

Oh, my bad.


Changed the Sections order, make additional changes to the Demographic and Academic Section

Changed the Section orders

Made addtional changes to the wording in the Demographic and Academic Section
Added an additional graphs

Let me know if you think certain sections should be in a different order

Once https://github.com/sexxis/classprofile/pull/41 is merged, I'll update the entrance average graph so it also uses the multi series box plot


Include support for Gentoo distribution

Thanks!

https://github.com/inode64/inode64-overlay/blob/1ca794a272048d8832fa9435e3f23ef2201495f9/media-video/go2rtc/go2rtc-1.8.2.ebuild#L47
what is build.go ?

Thanks , fix in version 1.8.3


Update sbt-sonatype to 3.9.2

Updates org.xerial.sbt:sbt-sonatype from 3.8.1 to 3.9.2.
GitHub Release Notes - Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.xerial.sbt", artifactId = "sbt-sonatype" } ]

labels: sbt-plugin-update, semver-minor

WIll be upgraded in pull request from @Atry


chore(main): release 0.5.1

:robot: I have created a release beep boop
0.5.1 (2023-01-23)
Bug Fixes

fix flaky test (8d591df)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/herdstat/herdstat/releases/tag/v0.5.1 :sunflower:


Add golangci-lint

Add golangci-lint and fix a couple issues it found.  I used the config file currently in hostd.  I think the linters are a bit nitpicky (they say having banned and Banned on EphemeralPeerStore is confusing) but I didn't disagree enough to make an exception with //nolint.

Snuck this in in #44


Add Eclipse Microprofile GraphQL 1.1 integration

The integration should be on-par with the graphql-java integration. We are currently blocked by https://github.com/eclipse/microprofile-graphql/issues/127 though which is why we will only do this when version 1.1 is out.

Since not even SmallRye GraphQL can do what we need, this will take even longe. Also see https://github.com/Blazebit/blaze-persistence/issues/1212


DOC-6840 DB Console stale capacity

Addresses: DOC-6840

Adds brief mention that unavailable nodes will show stale (last-known) values for capacity on the Cluster Overview page.

Staging
ui-cluster-overview-page.md

Hi @iAaronBuck  , friendly ping on this review when you get a chance --- or feel free to voluntell a fellow engineer of your choosing! Thank you.

This looks great! Thank you for updating this and doing an awesome job!

Thanks for the review!

I'll merge this simple change that still merges cleanly.


Make ObjectPermission a Changelogged model

As ...
Patti - Platform Admin
I want ...
I want ObjectPermission records to support the standard Nautobot changelog feature set
So that ...
I have a standard way to track changes to this data
I know this is done when...

ObjectPermission model support changelog integration
I can view ObjectPermission instances in the global changelog view and API

There is no expectation of changing the detail view implementation of ObjectPermission to view changelog records from the perspective of a single ObjectPermission instance. We will revisit this later, with moving all models out of the admin area.
Optional - Feature groups this request pertains to.

Security (Secrets, etc)

Database Changes
No response
External Dependencies
No response

Hey team! Please add your planning poker estimate with Zenhub @gsnider2195 @HanlinMiao @glennmatthews

This will target 2.1.X


-bash: drive: command not found

After attempting installation (both via git and pip),
entering any drive command returns:
-bash: drive: command not found

I was also getting this after a pip install on linux. It wasn't on my path and after adding it the app was found.
export PATH=~/.local/bin:$PATH


Css preprocessor

Add less4j and sass css compilers

Mixing two PRs together makes it hard to consider & review them independently.
Would you separate them?  It would also be nice to squash before opening the PR for less noise.

OK, I will close it and open a new PR from a clean branch. I didn't know that I should use separate branches to split PR.

Thanks.  It makes it easier for us to review.  Conceptually, this would be a nice addition.
It would be worthwhile to consider how such integration should operate in PROD & TEST mode vs DEV mode.


Greatly improve the efficiency of lidar

What changes do you make in this PR?

Please describe why you create this PR

I reduce the time for lidar to 10%!
close #379
Profile with: pgdrive/pgdrive/envs/marl_envs/marl_inout_roundabout.py
Before:

After:

Checklist

[x] I have merged the latest main branch into current branch.
[x] I have run bash scripts/format.sh before merging.
Please use "squash and merge" mode.

Wait! Many tests break!


Consumed utility based CMA SDK object in exports module

@netrajpatel, @antonyagustine changes seems to have formatting  changes in many file, any reasons ?. Are we using the prettier ?. Seems existing code is not formatted accordingly. Please confirm


Swap Alternate Elements

please accept my pull request

give this repo a star and follow me for follow back😊


What is explain_instance() "text" argument supposed to be?

According to the official documents, explain_instance() takes "raw text string to be explained" as the input of "text" argument.
Intuitively, it should be an single instance of raw text, but I just encountered a similar error reported in #35.
I followed some of the mentioned solutions in that relevant issue by dynamically changing "num_samples" to my data samples number, but I still could not make my code work ----- it reports "TypeError expected string or bytes-like object"  in lime/lime_text.py  at line 114.
It seems like there exist a contradiction that explain_instance() only takes raw string object as input; however, plain string object is not compatible with any "num_samples" except 1.
I am quite confused. May I get a demo that clarify what text supposed to be.

Having the same issue as you. Did you manage to figure out what was the problem?

Having the same issue as you. Did you manage to figure out what was the problem?

Remain unsolved.


Update model.stub remove type declaration

This pull request removes the type declaration from the model.stub file

@mirmovsum-abasov Oh god! I don't know how I missed your PR!
Thank you. It is merged it and I'll do a release soon 💪