Intermittent segfault for big cascaded-if

Steps to reproduce the issue
The following crashes intermittently:
yosys -p proc test.v
module mux_if_33_8 #(parameter N=33, parameter W=8) (input [N*W-1:0] i, input [$clog2(N)-1:0] s, output reg [W-1:0] o);
always @*
    if (s == 0) o <= i[0+:W];
    else if (s == 1) o <= i[1*W+:W];
    else if (s == 2) o <= i[2*W+:W];
    else if (s == 3) o <= i[3*W+:W];
    else if (s == 4) o <= i[4*W+:W];
    else if (s == 5) o <= i[5*W+:W];
    else if (s == 6) o <= i[6*W+:W];
    else if (s == 7) o <= i[7*W+:W];
    else if (s == 8) o <= i[8*W+:W];
    else if (s == 9) o <= i[9*W+:W];
    else if (s == 10) o <= i[10*W+:W];
    else if (s == 11) o <= i[11*W+:W];
    else if (s == 12) o <= i[12*W+:W];
    else if (s == 13) o <= i[13*W+:W];
    else if (s == 14) o <= i[14*W+:W];
    else if (s == 15) o <= i[15*W+:W];
    else if (s == 16) o <= i[16*W+:W];
    else if (s == 17) o <= i[17*W+:W];
    else if (s == 18) o <= i[18*W+:W];
    else if (s == 19) o <= i[19*W+:W];
    else if (s == 20) o <= i[20*W+:W];
    else if (s == 21) o <= i[21*W+:W];
    else if (s == 22) o <= i[22*W+:W];
    else if (s == 23) o <= i[23*W+:W];
    else if (s == 24) o <= i[24*W+:W];
    else if (s == 25) o <= i[25*W+:W];
    else if (s == 26) o <= i[26*W+:W];
    else if (s == 27) o <= i[27*W+:W];
    else if (s == 28) o <= i[28*W+:W];
    else if (s == 29) o <= i[29*W+:W];
    else if (s == 30) o <= i[30*W+:W];
    else if (s == 31) o <= i[31*W+:W];
    else if (s == 32) o <= i[32*W+:W];
    else o <= {W{1'bx}};
endmodule

Expected behavior
No SIGSEGV.
Actual behavior
(gdb) bt
#0  0x0000000000620d58 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=<optimized out>) at frontends/ast/genrtlil.cc:551
#1  0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfdd140) at frontends/ast/genrtlil.cc:438
#2  0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfd1150) at frontends/ast/genrtlil.cc:513
#3  0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfd1020) at frontends/ast/genrtlil.cc:438
#4  0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfc70a0) at frontends/ast/genrtlil.cc:513
#5  0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfc5330) at frontends/ast/genrtlil.cc:438
#6  0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfb8490) at frontends/ast/genrtlil.cc:513
#7  0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfb8360) at frontends/ast/genrtlil.cc:438
#8  0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfac890) at frontends/ast/genrtlil.cc:513
#9  0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfac760) at frontends/ast/genrtlil.cc:438
#10 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xf8fa70) at frontends/ast/genrtlil.cc:513
#11 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfdcd00) at frontends/ast/genrtlil.cc:438
#12 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfcea10) at frontends/ast/genrtlil.cc:513
#13 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfccb70) at frontends/ast/genrtlil.cc:438
#14 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfbdd20) at frontends/ast/genrtlil.cc:513
#15 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfbbe80) at frontends/ast/genrtlil.cc:438
#16 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfb0280) at frontends/ast/genrtlil.cc:513
#17 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfb0170) at frontends/ast/genrtlil.cc:438
#18 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfa6210) at frontends/ast/genrtlil.cc:513
#19 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xf936a0) at frontends/ast/genrtlil.cc:438
#20 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfde180) at frontends/ast/genrtlil.cc:513
#21 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfdcab0) at frontends/ast/genrtlil.cc:438
#22 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfd3320) at frontends/ast/genrtlil.cc:513
#23 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfcd650) at frontends/ast/genrtlil.cc:438
#24 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfc1b40) at frontends/ast/genrtlil.cc:513
#25 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfc37e0) at frontends/ast/genrtlil.cc:438
#26 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfb87c0) at frontends/ast/genrtlil.cc:513
#27 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfb7cd0) at frontends/ast/genrtlil.cc:438
#28 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfaea60) at frontends/ast/genrtlil.cc:513
#29 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfadf70) at frontends/ast/genrtlil.cc:438
#30 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xf920c0) at frontends/ast/genrtlil.cc:513
#31 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xf93e30) at frontends/ast/genrtlil.cc:438
#32 0x0000000000620946 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xf89ed0) at frontends/ast/genrtlil.cc:513
#33 0x0000000000620570 in Yosys::AST_INTERNAL::ProcessGenerator::processAst (this=0x7fffffffc1f0, ast=0xfde070) at frontends/ast/genrtlil.cc:438
#34 0x000000000061e830 in Yosys::AST_INTERNAL::ProcessGenerator::ProcessGenerator (this=<optimized out>, always=<optimized out>, initSyncSignalsArg=...)
    at frontends/ast/genrtlil.cc:279
#35 0x0000000000615929 in Yosys::AST::AstNode::genRTLIL (this=0xf8ac10, width_hint=<optimized out>, sign_hint=<optimized out>) at frontends/ast/genrtlil.cc:1533
#36 0x00000000005f5336 in Yosys::process_module (ast=0xf88d10, defer=<optimized out>, original_ast=<optimized out>) at frontends/ast/ast.cc:1065
#37 0x00000000005f43d3 in Yosys::AST::process (design=<optimized out>, ast=<optimized out>, dump_ast1=<optimized out>, dump_ast2=<optimized out>,
    no_dump_ptr=<optimized out>, dump_vlog1=<optimized out>, dump_vlog2=<optimized out>, dump_rtlil=<optimized out>, nolatches=<optimized out>,
    nomeminit=<optimized out>, nomem2reg=<optimized out>, mem2reg=<optimized out>, noblackbox=<optimized out>, lib=<optimized out>, nowb=<optimized out>,
---Type <return> to continue, or q <return> to quit---

Could this be a stack space issue? (But this stack trace isn't so deep....)

Finding it really hard to reproduce reliably, outside of running my new yosys-bench equiv_opt script: https://github.com/YosysHQ/yosys-bench/compare/eddie/equiv

Finding it really hard to reproduce reliably, outside of running my new yosys-bench equiv_opt script

What commands do I need to run exactly in order to reproduce this?

Can't seem to repro at all now, even on my branch. Let's close this for now and I'll re-open it if I run into it again. Sorry for the noise.


VRTK_BodyPhysics.SweepCollision() CapsuleCast calculations fail when PlayArea is rotated.

In VRTK_BodyPhysics.SweepCollision(), the calculation to find the starting points for the CapsuleCast does not take into account rotations of the PlayArea, and thus behave unexpectedly when the PlayArea is rotated.
I've resolved this on my Fork by changing:

Vector3 point1 = (bodyCollider.transform.position + (bodyCollider.center)) + (Vector3.up * ((bodyCollider.height * 0.5f) - bodyCollider.radius)); Vector3 point2 = (bodyCollider.transform.position + (bodyCollider.center)) - (Vector3.up * ((bodyCollider.height * 0.5f) - bodyCollider.radius));
to:
Vector3 point1 = bodyCollider.transform.parent.TransformPoint(bodyCollider.transform.localPosition + (bodyCollider.center)) + (Vector3.up * ((bodyCollider.height * 0.5f) - bodyCollider.radius)); Vector3 point2 = bodyCollider.transform.parent.TransformPoint(bodyCollider.transform.localPosition + (bodyCollider.center)) - (Vector3.up * ((bodyCollider.height * 0.5f) - bodyCollider.radius));

Environment

Source of VRTK: GitHub
Version of VRTK: 3.3.0-alpha
Version of Unity3D: Unity 5.62p4
Hardware used: Vive
SDK used: SteamVR

Steps to reproduce

Open Example Scene "042_CameraRig_MoveInPlace"
Rotate "[VRTK_SDKManager]" (or your SDK Setup)  90 degrees.
Walk around the inside of the "room". You should notice that you'll stop suddenly when NOT near a wall.

Expected behavior
Player should stop moving only when walking into a wall.
Current behavior
When the PlayArea is rotated, the player stops when not walking into a wall, since the CapsuleCast start from the wrong position.

It would be good if you could raise a pull request with the proposed fix.

I recently ran into this myself. Thanks so much for fixing this Michaelwolf95 .

PR with fix https://github.com/thestonefox/VRTK/pull/1408

@thestonefox
Sorry for not raising the PR, I've been absolutely swamped with multiple projects, and I just hadn't had a chance. Thanks for doing it for me!


Grapes can't be right-click harvested with Quark and grape seeds can't be composted

I can't use Quark's right-click Simple Harvest to harvest grapes, and I can't compost grape seeds in the vanilla composter. Also, less relevant but worth mentioning is that when punching a grape plant, I only ever get a single seed back, so I have to find all my grape seeds in the wild. Corn and eggplants are working fine; it's only grapes I'm having a problem with.
1.18.2
Forge 40.1.80
culturaldelights-1.18.2-0.14.jar

Cultral Delights doesn’t add Grapes
and I don’t know any FD addons that do
🤔

Dang, my bad. It was Italian Delight that adds them. Sorry about that.


[11.0 stable] Implement propagation of IP routes into apps via DHCP

It was strongly requested to make the propagation of static/connected IP routes available in the EVE 11.0 LTS version. This PR must therefore break the rule of not backporting new features (hopefully a one-time exception).
Original PR with the description of the feature: https://github.com/lf-edge/eve/pull/3690

@uncleDecart I had to update EVE APIs in 11.0 and it brought also some patch envelopes changes. Hopefully it does not break anything.

Can you check whether there are other API additions which arrive with this and in particular whether there was some api_capability value defined between 11.0.X and the route propagation one?

Luckily there were no API capabilities added between what 11.0 has and route propagation.
API update brought few patch envelopes additions (metrics). But as I understand it, it has no effect on what is implemented in 11.0 for this feature.


Avatar doesn't show Background or Color

Trying to use the Component in a fresh Next JS project getting the result below. Text + Icons are showing but background and colors are missing.
Code:

Result:

Looks like it's happening due to server components, Goober has no support for it so we need to revert it back to something else. I'll look at what I can do.

Trying to use the Component in a fresh Next JS project getting the result below. Text + Icons are showing but background and colors are missing.
Code:
Result:

One thing that helped me to sort this out was to create a client componente such as
"use client";

import Avvvatars from "avvvatars-react";

export default function Avatar({ placeholder }: { placeholder: string }) {
  return (
    <Avvvatars
      value={placeholder}
      size={64}
      style="character"
      radius={50}
      border
      borderSize={3}
    />
  );
}

Then used this in my page.tsx
import { redirect } from "next/navigation"

import { createClient } from "@/utils/supabase/server"
import Avatar from "@/components/Avatar";

export default async function HomePage() {
  const supabase = await createClient();

  const { data, error } = await supabase.auth.getUser();
  if (error || !data?.user) {
    redirect("/login");
  }
  return (
    <main className="h-screen w-screen z-10">
    <nav className="fixed z-20 top-0 h-[100px] w-full flex justify-center items-center bg-black shadow-md text-white">
      Bienvenido...
     <Avatar
     placeholder={data.user.email || data.user.user_metadata.name || 'User'}>

     </Avatar>
    </nav>
  </main>
  );
}

That works, still I'm going to remove goober because it's unnecessary at the moment. and I'm going to expose a class in case people want to manipulate from outside, at the moment it's quite hacky to achieve that.


Introspection ofType.field failed

Component(s)
router
Component version
v0.99.1
wgc version
0.58.5
controlplane version

router version
v0.99.1
What happened?
Description
When introspection is requested, it falls on ofType.field
Steps to Reproduce
Use the schema I sent you in the mail
https://github.com/wundergraph/cosmo/issues/959#issuecomment-2258868305
Run Query:
query MutationsSchema {
  __schema {
    mutationType {
      fields {
        description
        name
        type {
          ofType {
            name
            fields {
              name
              description
            }
          }
        }
      }
    }
  }
}

Expected Result
new cosmo
{
  "errors": [
    {
      "message": "Failed to fetch from Subgraph 'introspection__type__fields' at Path<EMAIL_ADDRESS>Reason: no data or errors in response."
    }
  ]
}

old cosmo
{
  "errors": [
    {
      "message": "Unexpected end of JSON input",
      "stack": "SyntaxError: Unexpected end of JSON input"
    }
  ]
}

Actual Result
Work Query
Environment information
Environment
OS: (e.g., "Ubuntu 20.04") macOS
Package Manager: pnpm, npm, yarn, etc pnpm
Compiler(if manually compiled): (e.g., "go 14.2") go version go1.22.5 darwin/arm64
Router configuration
version: "1"

# General router options
graph:
  name: "production"
  token: ""

log_level: "info"
listen_addr: "localhost:3002"
playground_enabled: true
introspection_enabled: true
json_log: true
shutdown_delay: 15s
grace_period: 20s
poll_interval: 10s
health_check_path: "/health"
readiness_check_path: "/health/ready"
liveness_check_path: "/health/live"
router_config_path: "config.json"

cors:
  allow_origins: ["*"]
  allow_methods:
    - HEAD
    - GET
    - POST
  allow_headers:
    - Origin
    - Content-Length
    - Content-Type
  allow_credentials: true
  max_age_minutes: 5m

# Config for custom modules
# See "https://cosmo-docs.wundergraph.com/router/custom-modules" for more information
modules:
  myModule:
    # Arbitrary values, unmarshalled by the module
    value: 1

Router execution config
No response
Log output
No response
Additional context
No response

Hi @flymedllva,
This does indeed appear to be an issue with the introspection datasource. However, this does not appear to be affecting any of our existing customers, who we must prioritise. We will attempt to address it when we have the capacity.
If this is urgent for you, we would encourage you to create a Pull Request that adequately fixes (with tests) this issue in the engine v2 package:
https://github.com/wundergraph/graphql-go-tools/
https://github.com/wundergraph/graphql-go-tools/tree/master/v2/pkg/introspection
Thanks!
The WunderGraph Team


Operator GUI window attributes are not being restored when a session file is opened

Describe the bug
I have a session file with a pseudo color plot with, for example a Threshold operator, though it is the active plot and operator, the operator GUI window attributes are not being restored when a session file is opened. The plot is correct and the attributes for the pseudo color plot GUI window are restored. Just not the operator.
For both the plot and operator GUI they are saved as being opened in the session file.
I have verified this issue with the threshold, slice, and extrude operators. As such, suspected it is operator wide.
To Reproduce
Steps to reproduce the behavior. For example:

Create a pseudo color plot with a threshold operator.
Set various attributes and apply
Open both the plot and operator GUI windows.
Save the session file
Restore the session file
The operator GUI window will be the defaults even though the plot and operator are selected in the plot window.
The plot GUI window will be correct and have the current attributes.

Expected behavior
I would expect the attributes to be updated in the operator GUI not be the defaults.
Example
Screen shot of the plot window after the session is restored. Note the plot and operator are selected.

Screen shot of the slice operator window after the session is restored. These are the default values.

Screen shot of the slice operator window after the session is restored and "reopening" the source. These are the correct attribute value.

Additional information
If I select another plot (which my example does not have) or another operator (which my example does not have) then as normal attributes are updated.
If I change an attribute in the operator GUI then apply. One get the defaults modulo the attribute changed.

Resolved by #19874, #19875


Add PlatformConstants.ServerHost for iOS to support react-devtools on device

Description
As described in this issue, react-devtools works great for iOS and Android in simulator/emulator, but only works on device on Android. I believe this is because PlatformConstants.ServerHost is not implemented on iOS, so it always falls back to attempting to connect to localhost, and therefore from your device the react-devtools are unreachable.
Reproduction Steps and Sample Code
Try to run react-devtools and connect with an iOS device
Solution
Add PlatformConstants.ServerHost to RCTPlatform, it should be a string constant that returns the ip:port string that points to the packager. With this change in place, react-devtools should work out of the box on device on iOS.
Additional Information

React Native version: 0.45
Platform: iOS
Development Operating System: macOS

We’re going to look at this with @javache.

In some places we use getDevserverUrl. It sounds like we should merge that with ServerHost, which is only exposed on Android right now.

Any news on this? Would be very nice.

See also https://github.com/facebook/react-native/pull/15547 for a different approach.

Is anyone working on this? if not I can pick it up and do it.

@Eyesonly88 - I don't think so! would love to see a PR for it

@Eyesonly88 are you already doing this? If NO, I can change #15547 to support PlatformConstants.ServerHost for iOS.
I missed this issue. 😢

This or #15547 would be greatly appreciated! I'm teaching a course on React Native and telling students to manually patch the setupDevtools function is proving to be difficult.

@jhen0409 sorry, I didn't get around to fixing this issue and stopped working on it.

This should definitely remain open! Without the proposed fixes in this thread, we can't use the remote inspector on an iOS device, which IMHO should absolutely work.

@jhen0409 Looks like you have a fix for this figured out. Would you be able to submit a PR to fix this? https://github.com/facebook/react-native/pull/15547/files

Should remain open. Still not possible to use react native devtools on iOS devices.

This can be closed, it works fine now.


Remote video is frozen after returning from background

Frequently after extended time outside of the app, when returning to app remote video view shows a frozen frame. Audio is unaffected. This has been observed on many devices with latest SDK.

@etown Thanks for reaching out. In order to debug this problem, can you provide following information ?

The Room SID (TVIRoom.sid)
The Device and iOS version
Video SDK version

Also, is it possible to for you to share the debug logs with us?
Best,
Piyush

I am getting the same issue often. Did you get to solve this?

@vidhyasri2106 Thanks for reaching out. Unfortunately we haven't been able to reproduce the problem yet. Is it possible for you to provide following when the problem occers -
The Room SID (TVIRoom.sid)
The Device and iOS version
Video SDK version
Best,
Piyush

Room SID - RMc3454c17f2333f2556061c52356ebba6
Device – iPhone X
iOS version – 12.1.4
Video SDK Version - '2.10'
Also it is automatically getting unsubscribed from Video and Audio track often. Can you please help?
Regards,
Vidhya Sri
Product Developer – Channel Apps Digital & Analytics
HCL Technologies Ltd.,
602/3, ELCOT, SEZ Unit – 1, Sholinganallur,
Chennai – 600 119
Direct: +91<PHONE_NUMBER>
https://www.hcltech.com/digital-analytics-serviceshttps://apac01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.hcltech.com%2Fdigital-analytics-services&data=02|01|vidhyasri.so%40hcl.com|ded1b32f653f478f30d108d5a386b5e5|189de737c93a4f5a8b686f4ca9941912|0|0|636594721364915827&sdata=vPWPmJ8otwLyTlRPb5r2UGD%2F9%2FAuwESf%2F15p9gP50lQ%3D&reserved=0
<EMAIL_ADDRESS>From: Piyush Tank<EMAIL_ADDRESS>Reply-To: twilio/video-quickstart-swift<EMAIL_ADDRESS>Date: Friday, 24 May 2019 at 12:21 PM
To: twilio/video-quickstart-swift<EMAIL_ADDRESS>Cc: Vidhya Sri Soundararajan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [twilio/video-quickstart-swift] Remote video is frozen after returning from background (#388)
@vidhyasri2106https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fvidhyasri2106&data=02|01|vidhyasri.so%40hcl.com|f5b77aa763c44649d26008d6e0143dcb|189de737c93a4f5a8b686f4ca9941912|0|0|636942774874402864&sdata=zxu1ISgpqRjSE7wMdhxtT8SSSYErqTJmeO6SqgJ%2BB54%3D&reserved=0 Thanks for reaching out. Unfortunately we haven't been able to reproduce the problem yet. Is it possible for you to provide following when the problem occers -
The Room SID (TVIRoom.sid)
The Device and iOS version
Video SDK version
Best,
Piyush
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftwilio%2Fvideo-quickstart-swift%2Fissues%2F388%3Femail_source%3Dnotifications%26email_token%3DAGZT2TOHMZIPLQHYPIZIECDPW6F6XA5CNFSM4HMWKKI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWEKGVI%23issuecomment-495493973&data=02|01|vidhyasri.so%40hcl.com|f5b77aa763c44649d26008d6e0143dcb|189de737c93a4f5a8b686f4ca9941912|0|0|636942774874412859&sdata=%2FdfMDu5uy8DXnVrDzgbW5dMXmZNagYppO2sNjnpufzU%3D&reserved=0, or mute the threadhttps://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGZT2TP2OMKXNVGUKXXGRIDPW6F6XANCNFSM4HMWKKIQ&data=02|01|vidhyasri.so%40hcl.com|f5b77aa763c44649d26008d6e0143dcb|189de737c93a4f5a8b686f4ca9941912|0|0|636942774874412859&sdata=D%2F2VSvDgeMxlAmZswxI6qWw0kixFHoGd64zx8qeH%2FyA%3D&reserved=0.
::DISCLAIMER::
The contents of this e-mail and any attachment(s) are confidential and intended for the named recipient(s) only. E-mail transmission is not guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or may contain viruses in transmission. The e mail and its contents (with or without referred errors) shall therefore not attach any liability on the originator or HCL or its affiliates. Views or opinions, if any, presented in this email are solely those of the author and may not necessarily reflect the views or opinions of HCL or its affiliates. Any form of reproduction, dissemination, copying, disclosure, modification, distribution and / or publication of this message without the prior written consent of authorized representative of HCL is strictly prohibited. If you have received this email in error please delete it and notify the sender immediately. Before opening any email and/or attachments, please check them for viruses and other defects.

Any update on this?

@vidhyasri2106 Thanks for the information and we are investigation the problem. We have observed an issue in our SDK's  network manager module where Wifi to LTE results into one way media, we think the root cause could be the same for the issue that you are observing.
We are working on the issue and I will get back to you as soon as I have the update.
Thanks again,
Piyush

Thanks. I will wait for your update.

Hello,
We have found a bug in our infrastructure (media server). It is 100 % reproducible in following network handoff scenario :

Alice connects to a Room with Wifi and LTE on.
Bob connects to the Room.
Alice switches off Wifi
After network handoff, Bob can see and hear Alice, but Alice's can not see or hear Bob. Alice observes frozen screen on her phone.

The fix for this problem has been tested and it is going to be deployed to our media server in a few days. We believe the media server fix should resolve the issue reported by @vidhyasri2106 and @etown on this ticket.
I am sorry for the inconvenience caused by this bug. I will post the update on this ticket once the fix is deployed.
Best,
Piyush

Hi Piyush,
But I am not doing network handoff at all. There is no sim in both devices. Both the devices are connected only with wifi.

Also, I am not getting this issue after returning from the background. I am getting this when the app is in the foreground itself.

@vidhyasri2106 This ticket is specific to network handover and backgrounding scenario. I have deleted your comments to avoid confusion for readers.
Let use this ticket for the ARKit freeze issue you are facing without backgrounding and without network handover.

Hello,
Our media server team has deployed the fix for this ticket. The discussed Wifi+LTE -> LTE network handoff scenario should work as expected now.
Try it out, and Let us know if you have any questions.
Best,
Piyush

Yes @piyushtank I am also facing this issue on remote view, how I will fix this?

@piyushtank I am also facing this issue with latest SDK. What is the solution for this? re-enabling the video track when app becomes active as opposed to app entering foreground?

@NaiyerAghaz Apologies for the delay. Can you provide the detailed scenario and the RoomSID when you observed the issue?
@Horatiu Can you provide the detailed scenario and the RoomSID when you observed the issue?

@piyushtank I have fixed it by enabling the video track after applicationDidBecomeActive notification fires as opposed to applicationWillMoveToForeground notification. The way I was able to reproduce was this:

In applicationDidEnterBackground notification handler I would set videoTrack?.isEnabled = false and this would disable the video
In applicationWillEnterForeground notification handler I would set videoTrack?.isEnabled = true

I would notice that the videoTrack would freeze even though at the other end I got the didEnableVideoTrack callback. The reason I need to enable and disable when user moves between app states is because I want a specific UI at the remote participant end and I would implement didEnableVideoTrack/didDisableVideoTrack there to achieve the wanted UX.
To fix it I removed enabling the video track in applicationWillEnterForeground and instead I am enabling it in applicationDidBecomeActive notification handler and that seems to work.
This issue is with the iOS SDK.

@Horatiu You should be able to enable/disable a video track while going to foreground/background.  We would like to fix the issue in SDK but when I tried to reproduce, I couldn't. So, I have created a branch on our VideoQuickstart sample app to reproduce the bug. Can you try it out and let me know.

actually the above described 'fix' did not work. If I background one or both participants for 2-3 minutes when I foreground either of them the local track will be 'freezed' even though callbacks that video track was enabled would be received at the other end. I'll try to repro on your branch a little later today.

@piyushtank Upon further investigation I am finding that when moving from background to foreground I get cameraSourceDidFail with error message from AVFoundation: "The operation could not be completed" and that is when the call is placed in this state.
Trying now to recover the call by rebuilding the cameraSource and localVideoTrack from scratch and republishing it when this happens. Will let you know if I manage to fix it that way.

@piyushtank I'm also experiencing the issue that @etown mentioned. I've filed a Twilio support ticket #4302109 with a call SID, debug logs and a video of it slowly recovering. This seems different than the issue Horatiu is mentioning, since the frozen video is the remote one for me and etown. We can reproduce this every time we background the app for 5+ minutes (but backgrounding the app briefly is fine). After foregrounding, the remote video feed will be frozen for a minute or two before rendering a few frames and freezing again. And each time it freezes, it freezes for a slightly shorter duration until it fully recovers around 5 minutes.

@raylillywhite Thanks. Followed up on the twilio support ticket.

Any news on a fix here ?

@halldorlogi We haven't been able to reproduce the problem and our team is busy upgrading the webrtc to latest release. If you have room sid and steps to reproduce, we can take a look. Also reproduction with quickstart will be helpful.

Closing the ticket due to no activity. Feel free to reopen if you run into issue or open a new ticket.


commit

Added the run.sh file used to run the main python file
After updating the permissions in terminal with:
cd /home/warg/computer-vision-python
chmod +x run.sh
The to test use ./run.sh to run the python file

Next time, please use a branch in the WARG organization rather than a fork.

Ok will do.
On Mon, Jul 3, 2023 at 9:25 PM Xierumeng @.***> wrote:

Next time, please use a branch in the WARG organization rather than a fork.
—
Reply to this email directly, view it on GitHub
https://github.com/UWARG/computer-vision-python/pull/133#issuecomment-1619328796,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJLU7BO3P7LNHZTT3GSYFMDXONWIDANCNFSM6AAAAAAZ5AAZXI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Insert performance degrades over time, recovers after restart

When inserting many individual entries, the insert performance degrades noticeably over time. After many 10k inserts, it takes about an order of magnitude longer for the same number of writes compared to the beginning. When the insert process is paused, and cottontail is restarted, the performance recovers substantially. This might be related to garbage collection processes performed by xodus. If so, such cleanup should be triggered more often in order to avoid substantial performance degradation.

This problem should be (partially) adressed in 0.16.0. One reason was the rather inefficient handling auf the auto-incrementing tupleIds.


change x509 case

Please note that X509FindType must be written x509FindType. It is case-sensitive.
Title
The attribute X509FindType  is not accepted. It must be x509FindType. The synthax is case-sensitive.
Summary
The attribute X509FindType  is not accepted. It must be x509FindType. The synthax is case-sensitive.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Increase Build Speed (Remove Cypress)

Build speed appears to be mostly a function of CPU availability on netlify??

QA Pass


v.035.0 <-- WRONG! should be v0.35.0

The latest version release was set to "v.035.0", shouldn't it be "v0.35.0"?
It's important to follow the naming convention, due to the fact that the support docs reference the latter!

Yes, it was a typo in the release name. It's now been fixed.


Category search for entity spawner

I want to be able to type "food" or "ammo" and it lists all the food/ammo in the game.
I envision that either we can
A. Add a yaml tag to individual entities (tedious)
B. Specify all entities in a yaml file as a certain category (might be trickier with certain yaml files)
Either way I think this would be an extremely useful feature and make map editing, admining and developing much easier.

Most entities already have components or parents that could be used for this, for example ammo has "BallisticMagazine" as a component, and foods have "FoodBase" as parent. Implementing this in the entity spawner wouldn't require editing yaml files at all

Please don't add folders. That sounds like a nightmare to navigate.
I think the ideal is just a way to add keywords to the entity yml for the spawn panel similar to how the suffix field works but without displaying.


Deploy some components by default

Description
There are some components, which we want to ship in every installation, like metrics-server. For that, lokoctl should support list of default components, which will be deployed without any configuration.
The alternative would be to put them into bootkube, but that does not play well with customization etc.
We should also decide, if we want to have an option to disable default components, if some users don't want them.
Initial task breakdown

[ ] Decide if we should be able to disable default components
[ ] Define a way to mark components as default

Acceptance Criteria

[ ] Lokoctl component can be marked as default and will be installed on default cluster installation

This is kind of done via examples configuration.

@invidian Shall we discuss this in the next sprint planning or close this ?
Based on the components that we have only metrics-server is something is is platform agnostic and doesn't require a configuration. If its just a single component that gets installed as part of the default installation. IMO it doesn't provide much value.

@ipochi the idea is to deploy components like MetalLB automatically when creating a cluster e.g. on Packet or Bare Metal, so we provide LB solution out of the box.


Update howto-registration-mfa-sspr-combined.md

Working through the document on World Password(less) day, and found that the user feature previews has recently updated to reflect the new option name "Users can use the combined security information registration experience".

@cgill : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for the contribution, @cgill
#sign-off


can't run unless the repository is writable

Bug: filegator can't run unless the repository is writable
Comment: this should not  be necessary if the repository is intended as read-only for downloads only
Risk: making it rwx-rwx-rwx opens it up for attacks e.g. wordpress vulnerability attacks, which I get quite a lot of.
Workaround: none
Proposal: make it a config option to have a read-only repo, and if writable is needed, provide a tmp/ directory to write temp files into, e.g.  if you need to zip things.

Hi,
What do you mean by can't run? Do you get an error or something?
The actual demo is read-only https://demo.filegator.io and you can see how it is configured in Dockerfile: https://github.com/filegator/demo

hi. Meaning it stops and returns an error saying "repository is unwritable" and goes no further.
Dr John Ostrowick
HdipEd MA PhD
Cell -<PHONE_NUMBER>
Web - www.ostrowick.co.za
On 13 Oct 2023, 11:36 +0200, Milos Stojanovic @.***>, wrote:

Hi,
What do you mean by can't run? Do you get an error or something?
The actual demo is read-only https://demo.filegator.io and you can see how it is configured in Dockerfile: https://github.com/filegator/demo
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Oh yes, I see it now.
It is hard-coded to check for ../repository folder but I've replaced that in configuration file with /var/www/demorepo/ and that's why it works for me.
Do, yes, this check should be removed I guess.

You can try with the latest version released just now.

Thank you
On Fri, Oct 13 2023 at 03:18:38 -0700, Milos Stojanovic
@.***> wrote:

You can try with the latest version released just now.
—
Reply to this email directly, view it on GitHub
https://github.com/filegator/filegator/issues/423#issuecomment-1761273512,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ43ZI2B6SK33SFB2V7LQV3X7EIP5ANCNFSM6AAAAAA56YCH7Y.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi there. Thanks for this fix. I see it still has the same complaint for private/logs. On my site I can't have anything writable because I get wordpress attacks dumped into anything writable. Can you put a config flag to say "log errors" or somesuch and if the flag is set to false, tell it to not look for a writable logdir or to write to apache's normal log file? Thanks


Provide COM_INTERFACE macro to support COM interop

This implements the macro for "classic" COM interop as discussed in #80.
The implementation of the macro mostly closely mirrors RT_INTERFACE!; I started with a copy of that macro, removed support for WinRT-specific features (generics, statics, IInspectable special case) and Rt* trait impls, then added the method impls directly in the macro expansion since we can't rely on generated code here. The other major difference for the user is that those method impls are all marked unsafe and expose an unrefined direct translation of the ABI with raw pointers, exposed HRESULTs, out pointers for returns, PascalCasing, etc.. The idea is that our COM interop support is intended to be purely a low-level FFI rather than a higher-level "projection," reflecting that projections are mostly a WinRT-specific concept.
Naming/structure: I considered a more precise name like COMBASE_INTERFACE, COM_INTEROP_INTERFACE, etc., but decided on plain COM_INTERFACE even though com_rs exposes a similar macro with the same name. This was based on my guess that few projects would ever want to use more than one COM macro, and any that did could qualify with winrt::COM_INTERFACE, which seems like a good name that reflects our macro's purpose and focus: a COM interface macro designed for interop with WinRT. It's in a new file called interop.rs because I felt COM interop support is a specialized feature that should be clearly labeled and delineated from other features. (I was thinking in the future this might expand into an interop folder, especially if we start adding direct support for individual COM interop interfaces)
Dependencies: I'm not a Rust macro expert but my understanding is that all the identifiers referenced from a macro expansion need to be accessible at the point of macro invocation. This unfortunately means that to get this macro to work from outside the crate I had to publically export ComPtr, ComAbi, and as_abi(); since using the new macro requires defining an IID I exposed DEFINE_IID! with macro_export as well.
It seems to me that it would be nice if there were a way for crates to better encapsulate implementation details of exported macros - so that macros could better fit into proper API design. I started a Reddit thread asking about this (where I threw out the off-the-cuff idea of providing a pub(crate_macros) accessibility option for this) but it didn't draw any interest. https://www.reddit.com/r/rust/comments/cg6bln/encapsulating_implementation_details_of_exported/
Testing: I ported my example to use this branch and this macro, and it works (On My Machine). Unfortunately this example isn't quite ready to be published or added to the winrt-rust examples folder because I'm not quite done disentangling its dependencies. If you feel you need to see a working test case for yourself before merging it'll probably be another week or two before I can provide it. 😟
As you can see, I went back and forth on some of these decisions, so please let me know if you have disagreements or concerns. Thanks!

I've posted a sample that uses the macro in this branch to support interop between a Win32 window and WinUI composition:
https://github.com/contextfree/rust-winui-experiments/


Enable P1NoAssignee tool

P1 means that the bug should be fixed for the next release;
so depending of the date of the next release, it will help to "nag" people to either find someone to fix the bug or change the priority.

@marco-c it'd be nice if you could make a dry-run with this PR locally to check that everything is fine.
The nagging strategy here depends of the number of days before the next release:
https://github.com/mozilla/relman-auto-nag/blob/master/auto_nag/scripts/configs/tools.json#L391-L411
And people will get a ni + mail to the manager (n+1 in the config), but the manager will only have one email. I think it make sense to double nag for p1 bugs (which are normally for very high priority bugs) but some people can be kinda pissed of by it.

Could we enable just the email for now, without the needinfo?

Thanks!


Explain CodeEditor's parameters within the doctring

Issue Report Checklist

[x] Searched the issues page for similar reports

Problem Description
There is not an explanation of the multiple parameters that a CodeEditor instance can receive
Following suggestion at https://github.com/spyder-ide/spyder/pull/9897#discussion_r320016309

I would like to contribute on this if possible.

Thanks @Akashtyagi08 for wanting to help us with this!
@dalthviz, please guide @Akashtyagi08 on how to do approach this task.

Hi @Akashtyagi08 you should add a docstring for the setup_editor method of the CodeEditor. The contents could be something like:
"""
Set-up configuration for the CodeEditor instance.

Usually the parameters here are related with a configurable preference in the Preference Dialog and Editor configurations:

param_1: explanation/description
param_2: explanation/description
...

"""

For the params explanation/description an initial text could be:

linenumbers:  Enable/Disable line number panel. Default True
language: Set editor language for example python. Default None
markers: Enable/Disable markers panel. Used to show elements like Code Analysis. Default False
font: Base font for the Editor to use. Default None
color_scheme: Initial color scheme for the Editor to use. Default None
wrap: Enable/Disable line wrap. Default False
tab_mode: Enable/Disable using Tab as delimiter between word, Default True
strip_mode: Default False
intelligent_backspace: Default True
automatic_completions: Enable/Disable automatic completions. The behavior of the trigger of this the completions can be stablished with the two following kwargs. Default True
automatic_completions_after_chars: Number of charts to type to trigger an automatic completion. Default 3
automatic_completions_after_ms: Number of milliseconds to pass before an autocompletion is triggered. Default 300
completions_hint: Enable/Disable documentation hints for completions. Default True,
completions_hint_after_ms: Number of milliseconds over a completion item to show the documentation hint. Default 500
hover_hints: Enable/Disable documentation hover hints. Default True
code_snippets: Enable/Disable code snippets completions. Default True
highlight_current_line: Enable/Disable current line highlighting. Default True
highlight_current_cell: Enable/Disable current cell highlighting. Default True
occurrence_highlighting: Default True
scrollflagarea: Default True
edge_line: Enable/Disable vertical line to show max number of characters per line. Customizable number of columns in the following kwarg. Default True
edge_line_columns: Number of columns/characters where the editor horizontal edge line will show. Default (79,)
show_blanks: Enable/Disable blanks highlighting. Default False
underline_errors: Enable/Disable showing and underline to highlight errors. Default False
close_parentheses: Enable/Disable automatic parentheses closing insertion. Default True
close_quotes: Enable/Disable automatic closing of quotes . Default False
add_colons: Enable/Disable automatic addition of colons. Default True
auto_unindent: Default True
indent_chars: Characters to use for indentation. Default " "*4
tab_stop_width_spaces: Default 4
cloned_from: Editor instance used as template to instantiate this CodeEditor instance. Default None
filename: Initial filename to show. Default None
occurrence_timeout: Default 1500
show_class_func_dropdown: Enable/Disable a Matlab like widget to show classes and functions available in the current file. Default False
indent_guides: Enable/Disable highlighting of code indentation. Default False
scroll_past_end: Enable/Disable possibility to scroll file passed its end. Default False
debug_panel: Enable/Disable debug panel. Default True
folding: Enable/Disable code folding. Default True

There are some options which explanation/description missing, but maybe @ccordoba12 @goanpeca or @andfoy could help (pinging them to check the list to complete it, and possible corrections/improvements to the ones I did).

There are some options which explanation/description missing

What options @dalthviz?

@ccordoba12 the ones I just put Default <value> like:

occurence_timeout
tab_stop_width_spaces
auto_unindent
scrollflagarea
strip_mode -> probably related with tab_mode?
occurrence_highlighting
intelligent_backspace

@dalthviz, please dig into the code and find what the missing parameters do (I don't have time to do that myself).

@Akashtyagi08 the missing initial descriptions:

occurrence_timeout : Timeout in milliseconds to start highlighting matches/occurrences for the current word under the cursor. Default 1500 ms.
tab_stop_width_spaces: Enable/Disable using tabs for indentation. Default True
auto_unindent: Enable/Disable automatically unindentation before else, elif, finally or except statements. Default True
scrollflagarea : Enable/Disable flag area that shows at the left of the scroll bar. Default True
strip_mode: Enable/Disable striping trailing spaces when modifying the file. Default False
occurrence_highlighting: Enable/Disable highlighting of current word occurrence in the file
intelligent_backspace: Enable/Disable automatically unindenting inserted text (unindenting happens if the leading text length of the line isn't module of the length of indentation chars being use). Default True

Working on it.

tab_stop_width_spaces: Enable/Disable using tabs for indentation. Default True

@dalthviz This seems to be a numerical parameter rather than Boolean.

Totally right @Akashtyagi08 ! Actually for that it should be something like:

tab_stop_width_spaces: Number of spaces a tab is equivalent. Default 4


.caret has line down its center in iOS Safari

I created a button using the following code:

  <button class="btn btn-default btn-lg dropdown-toggle" type="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
    Large button
  </button>

    ...

I use bootstrap version 3.3.5
On Safari iOS, looks like: http://i.stack.imgur.com/xi8XQ.png

That looks like iOS Chrome, which we don't support.
Also, we'd need a JS Fiddle or JS Bin that demonstrates the problem.

The fonts also fail on Firefox 38.2.0, once anything is changed in the HTML and the page is forced to reload.

@fennec15 Please open a separate issue.

I solved issue, read my answer from here: http://stackoverflow.com/questions/32460263/bootstrap-fonts-seem-to-be-striped-on-safari-ios

@cvrebert you can see this on the getbootstrap.com site. As far as I can tell this was caused by 119ce267a60732ba98513e5ed329baa3d866168a .

@Sonic3R Thanks, this helped me out.
This should be fixed asap.

X-Ref: https://github.com/zurb/foundation-sites/blob/8ff8ba68f1d1b0b4c6299e0e1e6fee2b5999fcbe/scss/util/_mixins.scss#L14

Bootstrap 3 is no longer being officially developed or supported.
All work has moved onto our next major release, v4. As such, this issue or pull request is being closed as a "won't fix." For additional help and support, we recommend utilizing our community resources. Thanks for your understanding, and see you on the other side of v4!
<3,
@mdo and team


Collection deserialisation needs to check for null tokens

Sql query results are composed of an array of values, with elements that may be null.
Fixes #3813

Spoke with @codebrain about this change. Will update this PR with a different approach

Integration test currently failing: Machine Learning - PreviewDatafeedApi

Check for deserialisation of response in the event of an error


Add a script that will list all UFS files that break spatch

Gives us the following list of:
../ufs/dirhash.h
../ufs/extattr.h
../ufs/inode.h
../ufs/quota.h
../ufs/ufs_dirhash.c
../ufs/ufs_quota.c
../ffs/ffs_extern.h
../ffs/ffs_snapshot.c
../ffs/ffs_softdep.c
../ffs/ffs_vfsops.c
../ffs/softdep.h
The script can be adapted for future ports.
Signed-off-by: Graham MacDonald<EMAIL_ADDRESS>

I have to give this a try.

The script fails for me. What shell is it?
sh bla bla gives me
[[: not found

Ah, must be a bash thing.  Do we mind if we use bash, or only sh?
I can change either way.

if it's a bash thing then the #! needs to be /bin/bash, not /bin/sh

The problem seems that sh is a symlink to dash in many linuxes and a bourne shell in BSDs. We should do shell independent or check env:
#!/usr/bin/env bash

Also, when harvey was built through a bash script in early days, "for" loops were warned by Ron as a future generator of troubles. One more time, as usually, he's in right (man, the first time you were wrong I'll do a party).


population filters: has/not bachelor/master

This will be done bigger and better with extent_codes from oodi api


Alowing the use of GDSFactory programmatic layout design

Some of the PDK's draw_* functions do not currently return instances of gf.Component. This difficults working directly with GDSFactory to, for instance, design an inverter without depending on klayout.
This happens because some of these functions, such as draw_nfet, generates a temporary GDS and then it's registered in the layout, removing all hierarchies and abstractions that GDSFactory uses.
This Pull Request moves that operation into the produce_impl method of the Pcell.
Changes introduced:

The "layout" attribute of certain draw_* functions is not used.
All the draw_* functions return an instance of gf.Component.
The gf_to_pya() function is defined to perform the registration in klayout. All PCells occupy it.
The efuse and bjt cells are not modified because their structure is different. That can be another pull request.

Things that can be a problem:

Some functions, like draw_nfet, use named cells. The documentation don't recommend it
because it can create duplicated cells. https://gdsfactory.github.io/gdsfactory/notebooks/11_best_practices.html
I have not executed any test case.

@akiles-esta-usado Could you please cancel this PR and create a PR on separate branch in this repo? I need to run the test cases to see if they pass. Could you do that?


Enumerate the search results

About this line: https://github.com/skttl/umbraco-fulltextsearch8/blob/7b32850ba662bcaeeea6aba04da38eacad8b59e7/src/Our.Umbraco.FullTextSearch/Services/SearchService.cs#L49
I would add a ToList() at the end, so the collection gets enumerated and this prevents multiple enumerations of the same collection.

Is it getting enumerated further in Full Text Search code? Or is it when you implement the package?

Is it getting enumerated further in Full Text Search code? Or is it when you implement the package?

In my implementation. I accessed the Results also for displaying the count, instead of using the TotalResults property (which I use now).
So in my case the Results were enumerated 3 times, instead of just once. It's an easy fix to prevent this from happening imo and I can't think of a downside, but maybe you have one?

In my implementation. I accessed the Results also for displaying the count, instead of using the TotalResults property (which I use now).
So in my case the Results were enumerated 3 times, instead of just once. It's an easy fix to prevent this from happening imo and I can't think of a downside, but maybe you have one?

Couldn't there be a potential memory issue if the enumeration is done too early?
I admit, this is something I don't have any expertise in, so any advice is welcome :)

Couldn't there be a potential memory issue if the enumeration is done too early?
I admit, this is something I don't have any expertise in, so any advice is welcome :)

I don't see how this can lead to a memory issue. It just makes sure the enumeration is done once, which I think is good in this case.
But as I said I might miss something ;-)

I don't see how this can lead to a memory issue. It just makes sure the enumeration is done once, which I think is good in this case.
But as I said I might miss something ;-)

But doesn't that load the whole list of results into memory? (basing on this https://stackoverflow.com/questions/3628425/ienumerable-vs-list-what-to-use-how-do-they-work)
Again - no expertise :)

But doesn't that load the whole list of results into memory? (basing on this https://stackoverflow.com/questions/3628425/ienumerable-vs-list-what-to-use-how-do-they-work)
Again - no expertise :)

AFAIK it's this:

Whenever I'm "stacking" LINQ expressions, I use IEnumerable, because by only specifying the behavior I give LINQ a chance to defer evaluation and possibly optimize the program.

Versus:

So is it faster to use List over IEnumerable? Only if you want to prevent a query from being executed more than once.

When doing DB work one should always use IEnumerable, until all LINQ queries are applied and the result is complete.
In the end, when one iterates an IEnumerable, the same amount of memory is used.
This quote is nice imo:

There are pros and cons to both. If you call ToList, you may remove some mystery as to when the query gets executed. If you stick to IEnumerable, you get the advantage that the program doesn't do any work until it's actually required.

AFAIK it's this:

Whenever I'm "stacking" LINQ expressions, I use IEnumerable, because by only specifying the behavior I give LINQ a chance to defer evaluation and possibly optimize the program.

Versus:

So is it faster to use List over IEnumerable? Only if you want to prevent a query from being executed more than once.

When doing DB work one should always use IEnumerable, until all LINQ queries are applied and the result is complete.
In the end, when one iterates an IEnumerable, the same amount of memory is used.
This quote is nice imo:

There are pros and cons to both. If you call ToList, you may remove some mystery as to when the query gets executed. If you stick to IEnumerable, you get the advantage that the program doesn't do any work until it's actually required.

But doesn't that load the whole list of results into memory?

Yes but your search results shouldn't be 1000000s of results, they should always be limited to a max... and better yet paged.
IEnumerable is an implementation, it's not the same everywhere. The IEnumerable on ISearchResult in Examine is  a custom enumerator. Once you start enumerating it, that's when the search will be executed. Same thing with the built-in Skip.
Rules:

Don't re-enumerate an IEnumerable

This is easy to accidentally do with all linq things like .Any(), .Count(), .First(), .Last(), Single() etc... all of these re-iterate. Count, Last, Single are especially bad since they enumerate the entire collection from start to end. This ok if you are only enumerating the collection once

If you need to iterate more than once, resolve it into a List. Yes this is stored in memory but that is much faster than the alternative and memory gets cleared ... so long as you're not pinning this to a static reference. The search result set shouldn't be huge (but that means it should be limited!)
Don't use Linq when there are much better options available. When you have a real ICollection like List, array, etc... these already have .Count, .Length properties = use them. Don't rely on Linq .Any() for things this is huge overhead

All of these rules apply to using IPublishedContent. This is the biggest problem in sites is too much enumeration and using Linq across collections a ton of times over and over again. I've seen some razor pages that re-iterate an IEnumerable in IPublishedContent 10-100s of thousands of times on each request 😱  that's due to using .Count(), .Any() all over the place without understanding what it does.

Cool - thanks for elaborating @Shazwazza!

@stefankip I guess the ToList() should come before Select(GetFullTextSearchResultItem), right?

It should be the last thing you do, I think. So after the Select call.

Yeah, but as @Shazwazza writes

IEnumerable is an implementation, it's not the same everywhere. The IEnumerable on ISearchResult in Examine is a custom enumerator. Once you start enumerating it, that's when the search will be executed. Same thing with the built-in Skip.

Wouldn't it execute the search both on Skip and Select?

AFAIK Skip and Select only adds an instruction to the IEnumerable and doesn't materialize it, hence it returning another IEnumerable. The call to ToList at the end would be the only thing materializing the IEnumerable.
But I might be wrong here 😏

Correct 👍


Used a namespaced argo instance for the site applications

Pros

namespaced argos have reduced capabilities
better isolation between sites

Cons

can't see all applications in one place
need to look up the URL and password for each argo instance

After consultation, we agree that isolation is a good thing, and that Argo is not and should not try to be the one-stop dashboard for patterns.  Therefor we will merge this PR.


Log info duplicated

Log info is duplicated between Builds and BuildFlows, i.e. Build logs contain all the lines of its BuildFlow logs.

Nope, I was wrong! Build logs are indeed separate.


View initialization uses stale data.

I have a view whose state data has nothing at all to do with the parent which creates the view. So I tried to initialize vm.data in my view constructor:
function View(vm) {
    vm.data={};
    ...
    return {  init: viewInit, render: viewRender };
    }

But when I tried to access the state in init, I found it was passed as undefined:
function viewInit(vm,sta) {
    /**/console.log("[**] init vm",vm); // vm.data is there
    /**/console.log("[**] init sta",sta); // sta is null!
    ...
    }

This happens because DOMVM uses the original data argument to invoke init, not the current vm.data value:
function ViewModel(view, data, key, opts) {
    ...

    var out = isPlainObj(view) ? view : view.call(vm, vm, data, key, opts);
    // vm.data is now {}

    ...

    // but this uses data arg, not vm.data
    vm.init && vm.init.call(vm, vm, data, key, opts);
}

The same argument could be used for key, which appears to be populated into vm before the view factory is called, and which might be overridden by it, particularly if the view's identity is self-realized.
I believe the init call should be:
vm.init && vm.init.call(vm, vm, vm.data, vm.key, opts);

I believe the init call should be: vm.init && vm.init.call(vm, vm, vm.data, vm.key, opts);

i made this change, though i must say that your use pattern is rather strange. vm.data is mostly an implementation detail. data's purpose is specifically to inject external dependencies (eg from parent). as states in https://github.com/leeoniya/domvm/issues/151, i've reserved vm.state and vm.api for the purpose of maintaining externally-accessible view state and methods.
while you could use data to hold view state, you now have to know whether or not data and vm.data was internally-set state or externally set state on a per-component basis, which is a definite footgun IMO. i would stick to setting and using vm.state in your case.

How is it that all this time I've been mentally oblivious to vm.state. What you say makes perfect sense and I'll alter my code forthwith. I've been up for about 60 hours. Sorry!

no worries. get some sleep!


#10936: Enable llama tg unit test

Ticket
Link to Github Issue
What's changed
Enables llama matmul2d prefill shape unit test after referenced issue was fixed.
Checklist

[ ] TG Frequent pipeline: https://github.com/tenstorrent/tt-metal/actions/runs/10719360302

fyi @bbradelTT


[BUG] Output in 0.7.3 is MUCH hotter than in 0.7.2 (especially in Right Channel)

The output level in 0.7.3 is MUCH hotter than in 0.7.2 especially on the right channel.
(In my situation, have not tried changing things around yet). Details on my particular setup at the end of this write-up.
To Reproduce
Steps to reproduce the behavior:
Simply load a previous project that was using 0.7.2 after upgrading to 0.7.3, the difference in levels should be noticeable immediately. See also screenshot below.
Expected behavior
I was expecting the output level to not be affected, I guess :) Even though the entire signal chain is actually mono, the right channel seems MUCH more affected than the left. See also screenshot.
Screenshots

Desktop (please complete the following information):

OS Windows 10 x86_64
Plugin version: 0.7.3
VST3
DAW: Reaper 6.79

Additional context
The FX Chain I was using:

DI Track of guitar 48kHz 16bit wav recording
ReaTune (as a tuner, does no processing, just used as a visual aid for tuning the guitar)
NAM Instance 1 (using the Xotic EP Booster profile named "GP EP Booster 100 Vintage.nam". Settings: Input 0.0 / Gate ON @ -80db, EQ ON with B/M/T  @ 5 (noon), Output 0.0, Normalize OFF, IR Loader OFF
NAM Instance 2 (using the JCM800 Zakk Wylde profile named "MARSHALL JCM800 ZW - HIGH SENSITIVITY - G3". Settings: Input 0.0, Gate OFF, EQ OFF, Output 3.8, Normalize OFF, IR Loader OFF
Ignite NadIR
ReEQ

Signal is already way too hot before NadIR and ReEQ.
Initial attempts at isolating the issue

Even only 1 instance (not 2 chained as in the original setup) is still hotter in 0.7.3
The output level setting is not the culprit, even if left at 0.0 the newer version is still a lot hotter.
Removing the ReaTune, NadIR and ReEQ have no effect, they don't seem to influence the problem at all.

Perhaps there is a channel division missing here:
https://github.com/sdatkinson/NeuralAmpModelerPlugin/blob/56a3a3228432cb9668af677c2bbc8f153ef0c7a1/NeuralAmpModeler/NeuralAmpModeler.cpp#L925
const double gain = pow(10.0, GetParam(kOutputLevel)->Value() / 20.0);

Should (maybe?) be:
#ifdef APP_API
  const double gain = pow(10.0, GetParam(kOutputLevel)->Value() / 20.0);
#else
  const double gain = pow(10.0, GetParam(kOutputLevel)->Value() / 20.0) / (float)nChansIn;
#endif

I'll look into this.
@vvasilikos it shouldn't be in the output. The previous behavior was to apply the gain then copy to all of the connected output channels; this release doesn't change that.

First test: an instance of the plugin with no models correctly outputs the same as its input.
Bypassed:

Active but no model loaded, no IR loaded, noise gate, EQ, and normalize disabled, input/output at 0:

Only plugin on the signal chain, input track is mono (first few seconds of v1_1_1.wav).

Test 2: Using DR Pedal Platform.
Installed v0.7.2 VST3 and rendered:

Installed v0.7.3 and:

So, it's replicated.
Lemme pop these into Python and have a closer look

Left channel appears to match

But this is different from what you're seeing--visually, it looks like the left channel is different (not only where there's clipping).

Inspecting with the debugger while playing the standalone, things seem correct: output is the same for both channels.
(Still running the DR pedal platform model)

Gotta say this is odd--I could've sworn I've spent a fair amount of time playing the plugin in a state that should have been functionally equivalent to what I'm looking at right now and never noticed this.
Unfortunately, I'm currently away from home until Sunday and don't have an interface nor even a pair of headphones, so I'm limited in what I can do. I'm going to put up a "help wanted" on this Issue and see if I can't get a hand from the community. 0.7.3 is a really exciting release and I'd love for folks to not have to downgrade in the meantime 💔

Right channel, v0.7.3 seems exactly twice as loud (unless it clips obv)
Plotting around the first spike in v1_1_1.wav

Think I figured it out:
Can you check out the Plug-in pin connector?
I opened it up and found an extra conenctions from Output 1 to L:

Unchecking it like so...

...results in something that nulls out perfectly as expected:

Let me know if that's what happened for you too

That seems to be it indeed!
That is so weird, I never messed with that configuration. Here's what it looks like if I have 0.7.2 loaded:

It's already different than what I would expect to be the default and Reaper will put it back to normal if you hit Reset there. I wonder how this setting got this configuration...
This is what it looks like after upgrading to 0.7.3:

The problem I assume was because Reaper somehow tried to "upgrade" the previous config to Stereo, and this resulted in this clipping or something?
Anyway, I think it's fixed now, I just have to go back to the normal full mix projects and reset the Plugin Connector...


dpg doc, add caution section to spread doc

Adding a caution section about the spread / anonymous model.
It is not always best to do this spread, as it affect SDK signature. For some languages, having lots of method arguments is not a good design.

.NET and Java have guideline on "simple methods" that should not have more than 6 arguments.
Python and TypeScript/JavaScript does not have such guideline (AFAIK). Python has keyword argument. TypeScript does not put optional arguments to method.

No changes needing a change description found.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/722/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/722/


Add missing prometheus metrics

What does this PR do?

Adding prometheus support for some metrics available through dogstatsd
Metadata update in #9388
Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Should those metrics be added to metadata.csv?


fix: articles/api-management/api-management-api-templates.md

Replace code fence languages

@nschonni : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Temporary folder of scripted tests changed in sbt 1.4

When running scripted tests in sbt 1.4+, the files of a project are now copied directly into the generated sbt temp directory.
Before sbt 1.4 sbt would create a subfolder with the project name (or id?, whatever)
For example:
The sbt project name is my_project:

sbt <1.4: /tmp/sbt_e205d35d/my_project/<files>
sbt 1.4+: /tmp/sbt_e205d35d/<files>

Why is this a problem?
I have a scripted test where it's important that some files reside outside of the project's base folder. Using file commands like copy and copy-file that's easy, I can just copy the files out of the base project folder to the parent folder.
Before sbt 1.4 however it was guaranteed that after the scripted test finished (or failed) that these copied files would be removed along with the sbt temp folder (because of course the files are within that folder).
Now that files will not be deleted automatically anymore, the just hang around in the /tmp/ folder...
Can you please again create that subfolder? Thanks!

@mkurz I don't recall changing that explicitly, but maybe it changed when we started batching the scripted tests. Would the following make a difference?
scriptedBatchExecution := false

@mkurz I don't recall changing that explicitly, but maybe it changed when we started batching the scripted tests. Would the following make a difference?
scriptedBatchExecution := false


Pre-Processor Test Plan jsessionid is blank in each request

Hello,
I'm following your book and got up to the chapter about the pre-processors specifically the HTTP URL Re-writing Modifier I start the web application on localhost and start the test.
Then I check the Order History as described at the end of the steps of section 8.1 of the book. No history is recorded. I check the Jetty server log and noticed:
session id --> null
After that I've taken a closer look to the requests sent by JMeter and I notice the jsessionid= parameter is sent blank. For example:
http://localhost:8080/order/placeOrder;jsessionid=
I'm using JMeter 3.1

Izaac,
Certain Web Servers do not support sessions in Cookies. They expect the sessionid as a part of URL. Only in such cases you would need URL rewriting.
This example in section 8.1 HTTP URL Re-writing Modifier expects the URL to have sessionid. The dt application needs to be run with a special environment variable SESSION_IN_URL set to true
Please look at the Appendix B.
B.1.2. Start With URL ReWriting Enabled
$ export SESSION_IN_URL=true
$ ./startDt.sh

Let me know if this solved your issue.
regards,
Sai Matam

@saimatam thank you, this resolved the issue I was having.
I didn't see any reference to Appendix B in Chapter 8 or I might've missed it somehow.
Regards,
izaac

Izaac,
Thank you for the feedback. When we correct the book next time, we will
put a special note and ask readers to refer Appendix B.
I would appreciate if you can write a review on Amazon or LeanPub - wherever
you bought the book.
regards,
Sai Matam


Orphaned attributes not always hidden

When an attribute which existed before stops existing on the malcolm server (e.g. after a reconnect), it should be hidden on the UI; this does not always appear to be happening

This is a particular issue when exports are removed from the table; they should immediately be removed from the details pane (currently only happens on refresh)


Crash when render 3d object GLB

Requirements:
Please go through this checklist before opening a new issue

[x] Review the documentation
[x] Search for existing issues in: viromedia/viro & ViroCommunity/viro
[] Use the latest ViroReact release

Environment
Please provide the following information about your environment:

Development OS: Mac
Device OS & Version: Android 12
Version: ViroReact: 2.22.0 and React Native version 0.63.4
Device(s): Pixel 4

Description
Viro React  made crash when render GLB file, It only happen in some of GLB file. And when I check logcat in Android Studio, it show this error
Failed to create an anchored node at position [0.0, 0.0, 0.0]
This is some off my code

    <ViroNode>
      <ViroSpotLight
        innerAngle={5}
        outerAngle={45}
        direction={[0, -1, -0.2]}
        position={[0, 3, 0]}
        color="#ffffff"
        castsShadow={true}
        influenceBitMask={2}
        shadowMapSize={2048}
        shadowNearZ={2}
        shadowFarZ={5}
        shadowOpacity={0.7}
      />
      <Viro3DObject
        source={require("./pet.glb")}
        position={randomPosition}
        scale={[0.02, 0.02, 0.02]}
        type="GLB"
        lightReceivingBitMask={3}
        shadowCastingBitMask={2}
      />
      <ViroQuad
        rotation={[-90, 0, 0]}
        width={0.5}
        height={0.5}
        arShadowReceiver={true}
        lightReceivingBitMask={2}
      />
    </ViroNode>

Remove ViroNode doesn't seem to help, I believe it only happen with some off my glb 3d object
Can any one help me with this?

are you using typescript ? if so, you need to add the extension files that are not common to the compiler

Can you provide the GLB 3D object you're using?


Fix #84

<!-- Make sure you have read http://napalm.readthedocs.io/en/latest/contributing/index.html --!>

Coverage decreased (-0.1%) to 86.212% when pulling dbe8110d8b7814395901fcb6e9a24b9c9e060b25 on mirceaulinic:FIX-84 into 2fcee2831c1e03c3d7a970884ec45a7e1246e76a on napalm-automation:develop.

Coverage decreased (-0.2%) to 86.187% when pulling 08abe5c38f6ceec9da037373596583fb93a3e8c5 on mirceaulinic:FIX-84 into 2fcee2831c1e03c3d7a970884ec45a7e1246e76a on napalm-automation:develop.

+1


Error or warning if XLIFF translations are missing.

I have already searched all existing topics. Unfortunately I could not find a similar question on this subject. So far the English XLIFF files are created automatically. Now it can happen that you have already created several additional language files, and now make a change to the code. And here it is very easy to forget to adapt the other languages. In my view, this is actually a mistake, but at least a warning. How can you notice as a developer that you have to add something here if you don't know how and where. I think it would be enough to display a message here, where in the way like the error message if a key already exists in the file. This would ensure that all translations have been entered. That in itself would only be a small thing, but it could make your life much easier.

Hi @FSharpCSharp,
Thanks for the feedback!
We have been hearing this from other people as well and it is on the backlog but I don't have any time frame for you regarding when it will be there.

@thpeder
Thank you very much for the information! I'm glad to hear you've already grasped the subject. Then I'm curious, and I'm looking forward to it.

Hi,
Please go ahead and post this to our Ideas forum at https://aka.ms/BusinessCentralideas, or vote up the idea if its already there. We're constantly monitoring top Ideas and will consider them for a future release.


APIref.md HTML formatting in sites.get REST API link

https://github.com/tableau/server-client-python/blob/master/docs/docs/api-ref.md#sitesget
REST API: Query Sites{:target="_blank"}

resolves on github.io


GOPATH environment variable is ignored when using interpreter

The following program sample.go triggers an unexpected result
package main

import (
	"fmt"

	"github.com/traefik/yaegi/interp"
)

var script = `package main

import (
	"fmt"
)

func main() {
	fmt.Println("hello there")
}
`

func main() {
	i := interp.New(interp.Options{})

	_, err := i.Eval(script)
	if err != nil {
		fmt.Println("Error:", err)
	}
}

Expected result
$ env GOPATH=~/go/ go run main.go
// hello there

Got
$ env GOPATH=~/go/ go run main.go
// Error: 4:2: import "fmt" error: unable to find source related to: "fmt". Either the GOPATH environment variable, or the Interpreter.Options.GoPath needs to be set

Yaegi Version
v0.15.0
Additional Notes
Yaegi states that the GOPATH environment variable is used for finding packages, except it never is. This only seems to work for the Yaegi binary, which explicitly passes it to the interpreter options.

Yes, the GOPATH environment variable is used by default for the command line executable (github.com/traefik/yaegi/cmd/yaegi), but not by the library package (github.com/traefik/yaegi/interp) that you use in your example. This is on purpose, for security reason when the interpreter is embedded in another host process and that we require the full control of the GOPATH, possibly fully separated from the default. We decided to make the management of GOPATH explicit, with an empty default.
You can still pass the GOPATH when creating the interpreter as in the following:
I := interp.New(interp.Options{GoPath: build.Default.GOPATH})

Could it be a good idea to change the error message as it's (almost) never applicable and thus not accurate?

@karelorigin yes you're right. I haven't noticed that the error message is inaccurate and needs to be changed. Thanks.


Time saver removed the zipping and un-zipping of the nuttx-export

Since the build is done in build_xxxx there is no reason to zip and unzip the nuttx export to a new location. We simply build insitu.

@dagar
On the subject of not zipping/unzipping the export.....How does this look to you?

Looks good. I readded the export stamp. I don't think dependencies on a folder work.
What's the plan with Nuttx patches? Are we going to keep all modifications as patches and not touch the repo?
How are you generating the patches? If I do it with git diff there are additional lines.

@dagar

What's the plan with Nuttx patches? Are we going to keep all modifications as patches and not touch the repo?

yep - we will only have patches. It is a lot easier to grasp and maintain all the PX4 deltas that will not be accepted upstream this way.

How are you generating the patches? If I do it with git diff there are additional lines.

cd NuttX
git diff --no-prefix --dst-prefix=NuttX/nuttx/ --src-prefix=NuttX/nuttx/   >../../nuttx-patches/\<patch name\>.patch

@dagar - Thank you


use timestamp or processing time as fallback for activity.id

Fixes #1122
Description
Use outgoing activity.timestamp or current processing time in TranscriptLoggerMiddleware to populate id field in activities being logged by the middleware. In some cases, ResourceResponses from Teams does not have a id value.
Testing
Unit tests added.

Pull Request Test Coverage Report for Build 77636

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 88.621%

Totals

Change from base Build 77617:
0.03%

Covered Lines:
4432

Relevant Lines:
4808

💛  - Coveralls

I'm considering generating a fake activity.id BotFrameworkAdapter.sendActivities for InvokeResponses. This change would need to be picked up in C# and would solve the original problem this PR was attempting to address.


fix: can't connect to server (websocket connection fails)

🐛 The bug
Devtools doesn't work on my machine, it's stuck on Connecting to server and then shows WebSocket disconnected. I use base Nuxt 3 starter. The interesting thing is that when I run the starter template on Stackblitz it works, so it probably has to be connected to my machine, but really don't know how. Do you have any tips on how to debug it?

jankaderabek@Jan-MacBook-Pro test-devtools % node -v
v18.18.0

The issue is same in every browser I have
🛠 To reproduce
https://stackblitz.com/github/nuxt/starter/tree/v3/
🌈 Expected behavior
It should connect to devtools websocket server
ℹ Additional context
No response

can you make sure that you are using the latest version of DevTools?

Yep, I checked that, Nuxt version 3.7.4 and devtools 0.8.5.

The devtools in the screenshot seems to be pretty old. Can you check .nuxtrc file under your home directory? There there is nuxt/devtools related entries, remove it and try restart your nuxt app.

I have this exact same issue.  There isn't a .nuxtrc in any of my projects.  It just started to do this after the last update.  The only difference is I have an M1 mbp.
Here are some screenshots maybe they will give you some clues.

t/devtools/assets/85001306/dbd3b76e-d202-4677-9b76-8807562a0bd0">

Ok I removed ~/.nuxtrc and it started to work, I just noticed that in the console is a message about Nuxt devtools, now I can see correctly version 0.8.5. I was checking the version just in package-lock and node_modules before.
jankaderabek@Jan-MacBook-Pro drone-analysations % npm run dev

> dev
> nuxt dev

Nuxt 3.7.4 with Nitro 2.6.3                                                                                9:20:55 AM
                                                                                9:20:55 AM
  ➜ Local:    http://localhost:3000/
  ➜ Network:  use --host to expose

ℹ Using default Tailwind CSS file                                                                                nuxt:tailwindcss 9:20:56 AM
✔ Nuxt DevTools enabled (v0.8.5), press Shift + Option + D in app to open  (experimental)                                                                            9:20:57 AM
ℹ Tailwind Viewer: http://localhost:3000/_tailwind/                                                                                nuxt:tailwindcss 9:20:57 AM
✔ Nitro built in 847 ms                                                                                nitro 9:21:00 AM
ℹ Vite client warmed up in 4091ms

Before it was:
jankaderabek@Jan-MacBook-Pro drone-analysations % npm run dev

> dev
> nuxt dev

Nuxt 3.7.4 with Nitro 2.6.3                                                                                6:31:06 PM
                                                                                6:31:07 PM
  ➜ Local:    http://localhost:3000/
  ➜ Network:  use --host to expose

ℹ Using default Tailwind CSS file                                                                                nuxt:tailwindcss 6:31:08 PM
✔ Nuxt Devtools is enabled v0.2.5[global] (experimental)                                                                                18:31:08
ℹ Tailwind Viewer: http://localhost:3000/_tailwind/                                                                                nuxt:tailwindcss 6:31:08 PM
ℹ Vite client warmed up in 3826ms                                                                                6:31:13 PM
✔ Nitro built in 681 ms

Everything work smoothly now 🎉 , thank you for this amazing module @antfu 👍

@daver987 I can see in your console screenshot that you have devtools in version 0.4.2 (current is 0.8.5), try to find out why is that.

@daver987 I can see in your console screenshot that you have devtools in version 0.4.2 (current is 0.8.5), try to find out why is that.

That's a good question, it always said it was up to date, I'll see if I can find out why.


Revert "Bump actions/checkout from 3 to 4"

Reverts caioselias/skills-introduction-to-github#1

.


[Bug] Data races between initDecoder and initEncoder operations

In v0.10.4, A data race condition has been detected when running tests with -race flag.

I suspect the following is occurring from this PR (#490 ):

runtime.AnalyzeTypeAddr() is being called in both initDecoder and initEncoder.
Although using sync.Once, since initEncoderOnce and initDecoderOnce are separate variables, concurrent calls to initEncoder and initDecoder result in race conditions in runtime.AnalyzeTypeAddr() operations.

In our project, tests that simultaneously use json.Marshal and json.Unmarshal have become flaky.

I also have the same issue

same

@toddtreece It seems that your PR caused a regression. I'm currently unable to allocate time to address this, so could you please handle it ?

@goccy yes, sorry about missing that. i will open a PR to address this tomorrow morning


Feat: Transform - Execute a function that returns className #114

Description

Execute a function at css()
Related Issue

#114

Summary by CodeRabbit

New Features

Enhanced the transform function to process arrays containing both class names and functions that return class names.
Expanded the ComplexCSSItem type to support function-based class name generation.

Bug Fixes

Updated the test suite with new cases to validate the handling of function inputs and complex CSS rules.

Additional context

Checklist

/fast-forward


Propagating build script error codes to the top level script

This is to address #167.
@sharwell, I noticed you added a NuGet package reference for the WiX toolset and updated CodeTools BuildMSM.xml to refer to the correct toolset (which is great). However, NuGet packages from the top-level nuget.config are not restored prior to building CodeTools. As a result, running buildCC on a freshly checked out copy fails.
I added a line to restore NuGet packages right at the top of buildCC.bat. Let me know if there is a better way to handle this.

I added a line to restore NuGet packages right at the top of buildCC.bat. Let me know if there is a better way to handle this.

I would just add a line here like the following:
rem Also need to restore packages from the top-level solution before building CodeTools
..\..\.nuget\NuGet.exe restore ..\..\CodeContracts.sln

I changed it as you suggested.


[DEV-182920] Fixing Number Format bug exception

This PR changes a peace of code from Int to Long, to prevent NumberFormatException

LGTM 👍


Compression Algorithm

We should choose a proper compression algorithm.
Will edit these with possible options.

Will go with brotli but will make it trait so everyone can easily arrange.


getMinValue and getMaxValue crash the program

Describe the bug
The function getMaxValue and getMinValue have a bug. When called with an empty vector they crash the program.
This is caused by the lines below.
https://github.com/tier4/scenario_simulator_v2/blob/173a8c1d5e16b6e594de58d78cdb81a1d518ad82/common/math/geometry/src/polygon/polygon.cpp#L55
https://github.com/tier4/scenario_simulator_v2/blob/173a8c1d5e16b6e594de58d78cdb81a1d518ad82/common/math/geometry/src/polygon/polygon.cpp#L61
std::min_element and std::max_element, when called with two identical iterators (which happens when we call begin() and end() on an empty vector), return this iterator. De-referencing an end() iterator causes the program to crash.
This case should be considered e.g. with an early return that returns nan when the vector is empty, or throwing an exception.
Context:
I am adding unit tests for the geometry package and wanted to add tests for the aforementioned functions, but calling them with an empty vector causes the whole program to crash.
To Reproduce
Steps to reproduce the behavior:

Edit any existing source code file or add a new one
Call getMinValue or getMaxValue with an empty vector as argument
Compile and execute
See error

Expected behavior
Some controlled handling of the empty vector situation like return of nan or throwing an exception, but not program crash.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]: Ubuntu
Browser [e.g. chrome, safari]: Firefox
Version [e.g. 22]: 22
ROS 2 version: Humble
DDS: CycloneDDS

Thanks, please use THROW_SIMULATION_ERROR macro when the length of the points are zero.
And, if the length of the points are one, return the first points.

#1139 was merged


fluffychat: fix sqlite3 library missing

There were errors about libsqlite3.so missing, which I fixed by adding it to runtime dependencies
// cc @tanja-4732
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg eval

I was going to update it but failed (a vendored dep fetched by dart fetching some vendored dep in cmake). I think it's fine to fix it first.


[BEAM-3761] Define cmp() in Python 3

Signed-off-by: cclauss<EMAIL_ADDRESS>Please add a meaningful description for your change here
Define cmp() function that was removed in Python 3.  A more straight ahead approach to #4774 that should be easier to review.  Fixes the following issues:
flake8 testing of https://github.com/apache/beam on Python 3.6.3
$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics
./sdks/python/apache_beam/io/vcfio_test.py:77:14: F821 undefined name 'cmp'
      return cmp(v1.end, v2.end)
             ^
./sdks/python/apache_beam/io/vcfio_test.py:78:12: F821 undefined name 'cmp'
    return cmp(v1.start, v2.start)
           ^
./sdks/python/apache_beam/io/vcfio_test.py:79:10: F821 undefined name 'cmp'
  return cmp(v1.reference_name, v2.reference_name)
         ^
./sdks/python/apache_beam/io/gcp/datastore/v1/helper.py:93:12: F821 undefined name 'cmp'
  result = cmp(p1.kind, p2.kind)
           ^
./sdks/python/apache_beam/io/gcp/datastore/v1/helper.py:101:12: F821 undefined name 'cmp'
    return cmp(p1.id, p2.id)
           ^
./sdks/python/apache_beam/io/gcp/datastore/v1/helper.py:106:10: F821 undefined name 'cmp'
  return cmp(p1.name, p2.name)
         ^
./sdks/python/apache_beam/transforms/window.py:195:12: F821 undefined name 'cmp'
    return cmp(self.end, other.end) or cmp(hash(self), hash(other))
           ^
./sdks/python/apache_beam/transforms/window.py:195:40: F821 undefined name 'cmp'
    return cmp(self.end, other.end) or cmp(hash(self), hash(other))
                                       ^
./sdks/python/apache_beam/transforms/window.py:250:14: F821 undefined name 'cmp'
      return cmp(type(self), type(other))
             ^
./sdks/python/apache_beam/transforms/window.py:251:12: F821 undefined name 'cmp'
    return cmp((self.value, self.timestamp), (other.value, other.timestamp))
           ^

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
  When I thought about centralizing it but the three files are not near each other in the hierarchy (io, testing, transforms) but I suppose we could do something like from from apache_beam.utils.compat import cmp  It also stuck me as a bit weird (in legacy Python) to import a builtin from another file so that made the try/except block essential which then means that one one line per file is actually saved.  I will make the change if that seems like the best approach.

It seems that there were failures in Cython: https://builds.apache.org/job/beam_PreCommit_Python_Commit/166/console

You can run this test locally via tox -e py27-cython from within sdks/python/

I'm curious, why does Cython fail specifically on Python 2.7? I've tried locally with Python 3.6 and 0.28.1, and the following imports fine:
from past.builtins import unicode

print(unicode)

On 2.7 however it produces:
ImportError: No module named past.builtins

You could try updating tox.ini to explicitly mention that as a dep?

The current approach is safer...
We only have 18 more months to get this done.

There are no Jenkins build issues using my approach  ;-)

Isn't https://builds.apache.org/job/beam_PreCommit_Python_Commit/166/console from your approach or am I looking at the wrong build logs?

The build ran once more after d9d8689 was pushed. See: https://builds.apache.org/job/beam_PreCommit_Python_Commit/199/

Yes.  The commits were squashed into a single commit.

Ah ok, sorry for my confusion then :)

There are no Jenkins build issues using my approach ;-)

No, but your approach introduces boilerplate. I think this is worth fixing given that a) there seem to be no issue with importing from past on Cython, and b) a past import is no less safe than try-except NameError.

OK.  Can you please create a new PR that solves the problem and when it is checked in then I will gladly close this one.  I have zero pride of ownership on these fixes...  I merely want them resolved.

Sure, will do! I actually have a PR open which addresses similar patterns in the code (see #5869).

@charlesccychen Your review please?

Run Python PostCommit

Thanks!  This LGTM.  Will merge once tests pass.

Can someone please help me to decipher what went wrong?

https://builds.apache.org/job/beam_PostCommit_Python_Verify_PR/23/

Strange, looks like it crashed / timed out in the middle.

Run Python SDK PostCommit

Run Python Dataflow ValidatesRunner

I am even more confused.

This looks fine, and the ValidatesRunner tests pass, so I'm going to merge even though Jenkins seems to buggy right now for the PostSubmit run.

Thanks all!


Update benefits.md

SSL has been deprecated, TLS is the current encryption we use.

@KumarVivek Could you merge this?


Add maxFileCount option and feature

Thanks for library!  I needed to limit the number of uploaded files, so I added some quick code to enable the feature.
Originally saw issue https://github.com/promosis/file-upload-with-preview/issues/11, but it was for an older version and had a clunky delay in it.  It didn't look like the limit feature was included in the rewrite.

Hello! Thank you for this pull request - I just tested it out and it worked great!
It seems like there are some eslint issues though, which I had to fix before it would run.
But don't worry! What I'll do is accept this pull request so that you get credit for this code, and then I'll touch the errors up and push a new version. Just keep this in mind for the future, thank you. I'll update here when I'm done and the new version is pushed!

Ok great - this is up at v4.0.3 - thanks @DancesWithCode!


OAuth based workspace is unable to acquire token

Expected Behaviour
OAuth based clients should be able to acquire access token
Actual Behaviour
OAuth based clients are not able to get access token
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Create a OAuth based workspace and call any operation which need a token for example: getProviders() API
Platform and Version
Java 11 + AEM + AEM Guides
Sample Code that illustrates the problem
Logs taken while reproducing problem
feign.FeignException$BadRequest: [400 Bad Request] during [POST] to [https://ims-na1-stg1.adobelogin.com/ims/token/v3?client_id=xxxx] [ImsApi#getOAuthAccessToken(String,String,String)]:
[
    {
        "error": "invalid_target_scope",
        "error_description":"None of the requested scopes are both on the client and the binding."
    }
]

PR: https://github.com/adobe/aio-lib-java/pull/211


The important parts of C++23

stacktrace
<print>
C++23: flat_map and flat_set
C++23: Multidimensional Subscript
C++23: <ranges> Upgrades
C++23: constexpr <cmath> (missing trig functions)
C++23: <expected>
C++23: <generator>
C++23: <md_span>
C++23: explicit this (aka deducing this)
size literal suffixes
start_lifetime_as

https://compiler-explorer.com/z/b4bqGPe4x

Coming in 419


Free Code Camp study group

https://www.facebook.com/groups/free.code.camp.kirksville/

@thonghdang  Hi, can you confirm the state - Missouri ? thanks


question about epoch-counting

In trainval_net.py:
By default it only trained for (max_epoch - 1) epochs.
It seems that epoch should mean "the number of done epoch", thus start_epoch's default value should be 0, and some details should be modified in the training loop?
btw, is "epoch % (args.lr_decay_step + 1) = 0" a typo that "+1"?

OK, thanks for your great work.

closing this issue


Implement notification pipeline/buss (New Relic or APM)

We should have an internal log system/pipeline (maybe Redis queue or sth) for noticing errors, bottlenecks etc.
We can use New Relic custom alerts or figure out APM (https://www.elastic.co/solutions/apm). We should monitor all internal problems, timeouts etc

Related to: #1651


Send unsolicited messages from server to client?

Hi,
I would like to send messages from the server to the client. For example, the client registers itself with a POST message to some URI. Then the server keeps the address of that client around and periodically sends COAP messages to that client informing it of updates.
Is there a way to do this? Creating a CoapClient for each incoming connection does not seem to work, and I don't see any other way to do queries on the client.
Thank you!

Not sure if I got you right but I guess the observe option is your friend... (see RFC 7641).