Expression: cannot dereference end map/set iterator on Windows

Checklist

[X] I read the troubleshooting guide before raising this issue
[X] I made sure that the issue I am raising doesn't already exist

Current bug behaviour
Audio cannot be played on Windows due to a MVC++ error occuring.
Expected behaviour
Audio should play without any problem.
Steps to reproduce

Clone the repository and extract the flame_audio example project.
Run flutter create . in order to create the Windows project.
flutter run.

Code sample

Code sample
import 'package:flame/components.dart';
import 'package:flame/extensions.dart';
import 'package:flame/game.dart';
import 'package:flame/input.dart';
import 'package:flame/palette.dart';
import 'package:flame_audio/flame_audio.dart';
import 'package:flutter/widgets.dart' hide Animation;

void main() {
  runApp(GameWidget(game: AudioGame()));
}

/// This example game showcases three possible use cases:
///
/// 1. Use the static FlameAudio class to easily fire a sfx using the default
/// configs for the button tap.
/// 2. Uses a custom AudioPool for extremely efficient audio loading and pooling
/// for tapping elsewhere.
/// 3. Uses the Bgm utility for background music.
class AudioGame extends FlameGame with TapDetector {
  static Paint black = BasicPalette.black.paint();
  static Paint gray = const PaletteEntry(Color(0xFFCCCCCC)).paint();
  static TextPaint text = TextPaint(
    style: TextStyle(color: BasicPalette.white.color),
  );

  late AudioPool pool;

  @override
  Future<void> onLoad() async {
    pool = await FlameAudio.createPool(
      'sfx/fire_2.mp3',
      minPlayers: 3,
      maxPlayers: 4,
    );
    startBgmMusic();
  }

  Rect get button => Rect.fromLTWH(20, size.y - 300, size.x - 40, 200);

  void startBgmMusic() {
    FlameAudio.bgm.initialize();
    FlameAudio.bgm.play('music/bg_music.ogg');
  }

  void fireOne() {
    FlameAudio.play('sfx/fire_1.mp3');
  }

  void fireTwo() {
    pool.start();
  }

  @override
  void render(Canvas canvas) {
    super.render(canvas);
    canvas.drawRect(size.toRect(), black);

    text.render(
      canvas,
      '(click anywhere for 1)',
      Vector2(size.x / 2, 200),
      anchor: Anchor.topCenter,
    );

    canvas.drawRect(button, gray);

    text.render(
      canvas,
      'click here for 2',
      Vector2(size.x / 2, size.y - 200),
      anchor: Anchor.bottomCenter,
    );
  }

  @override
  void onTapDown(TapDownInfo info) {
    if (button.containsPoint(info.eventPosition.game)) {
      fireTwo();
    } else {
      fireOne();
    }
  }
}

(as you can see, it's the sample project of flame_audio, which is available here).
Affected platforms
Windows
Platform details
Processor : AMD Ryzen 5 5600H with Radeon Graphics            3.30 GHz
RAM :	16,0 Go (15,3 Go usable)
Edition :	Windows 11 Famille
Version :	22H2
Installed on : 05/10/2022
OS Build :	22621.1555
Experience :	Windows Feature Experience Pack 1000.22640.1000.0
AudioPlayers Version
4.0.1
Build mode
debug
Audio Files/URLs/Sources
bg_music.ogg
Screenshots

Logs
No log as it's a native error. See screenshot above.
Related issues / more information
No response
Working on PR
Why not, but I have no C++ experience.

Good catch @Skyost. The following happens: Dispose is called on bgm in FlameAudio, which should not be the case as it makes the player obsolete, should be replaced with  release() (cc @spydon).
But it should also be tested and fixed in AP, as a player could be disposed, without ever playing any source.
#1450
#1478
Further .ogg is not supported by Windows, but that was not the problem for crashing here :)

Good ! Can I close this one @Gustl22 ?

I'll close, when #1480 is merged ;)


TimeoutException doesn't have a type to check what kind of timeout it was

Describe the bug
There are different types of timeout errors when running code. As described in the TimeoutException class:
class TimeoutException(SandboxException):
    """
    Raised when a timeout occurs.

    The [unavailable] exception type is caused by sandbox timeout.\n
    The [canceled] exception type is caused by exceeding request timeout.\n
    The [deadline_exceeded] exception type is caused by exceeding the timeout for process, watch, etc.\n
    The [unknown] exception type is sometimes caused by the sandbox timeout when the request is not processed correctly.\n
    """

However, when catching one of these exceptions you have to parse the message to infer which of these it is. This is error-prone as you evolve the E2B SDK.
I need a way to check what kind of timeout to then decide what to do with it.
Currently, a sandbox Idle timeout message is:

<title>502 Bad Gateway</title>

502 Bad Gateway

nginx/1.27.0

: This error is likely due to sandbox timeout. You can modify the sandbox timeout by passing 'timeout' when starting the sandbox or calling '.set_timeout' on the sandbox with the desired timeout.

While an execution timeout is "Execution timed out — the 'timeout' option can be used to increase this timeout"
To Reproduce
Steps to reproduce the behavior:
Set very short timeouts of the different kinds and see the TimeoutExceptions being thrown.
Expected behavior
A clear and concise description of what you expected to happen.
Should either raise different TimeoutException sub-classes, or the SandboxExceptions should have fields to make it easy to identify exactly what the issue was so the client can handle it properly without having to parse the message.
Additional context
E2B Py SDK v1.01

Hey @antonioalegria, we will be improving the timeout exceptions and adding more granular ones.

Thanks!


Add a CloudFile/FileLink category

As a part of bug 1531595 I want to add a "Find more providers…"-type link to the preferences tab. I wish we could decide what we call the feature though, I know of at least three names.

I think category addition works by adding to src/olympia/constants/categories.py and also creating an olympia.addons.models.Category db object. But I could be wrong.


Node version error for Node-red UI table for nodered-sil.sh script

Describe the bug
I tried to follow the tutorial for getting setup on everest quickly (I followed this a couple of months ago with success), but I'm encountering errors with the nodered-sil.sh script. I believe the new version of node-red needs Node version >=16 however the docker image that this script is pulling from uses version ~14.

Basically, the ui-table element is non-usable which breaks the node-red ui entirely.
EVerest Domain
Simulation
Affected EVerest Module
docker image: everest-nodered
script: /everest/everest-core/build/run-scripts/nodered-sil.sh
To Reproduce
follow the instructions to get setup on everest: (https://everest.github.io/latest/general/03_quick_start_guide.html)
This happens when running:
{EVerest Workspace Directory}/everest-core/build/run-scripts/nodered-sil.sh
Anything else?
I believe you just need to update the Node version in the docker image as the ui table works fine for the other nodered image.

Was there a fix to this?

I also saw the same issue. Following the guide leaves me with an empty ui sub-page.

While executing ./user_workspace/everest-core/build/run-scripts/nodered-sil.sh I also get the warning about

notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":">=16"} (current: {"node":"14.19.3","npm":"6.14.17"})

ENV NODE_VERSION=14.19.3 is set in the node-red container

Related to https://github.com/EVerest/everest-core/pull/895.
Adding RUN npm install<EMAIL_ADDRESS>on the build Dockerfile at everest-core/build/dist/share/everest/docker/Dockerfile made the dashboard work ^^


onLoad On img Firing Almost Never

What version of Remix are you using?
1.13
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
Create a new Remix project and put the following in index.jsx:
export default function Index() {
  return (
    <img
      src="Rockafella.jpg"
      alt="NYC"
      onLoad={function () {
        console.log("loaded!");
      }}
    />
  );
}

Expected Behavior
After loading the image, I expect to see a console log informing us.
Actual Behavior
I could see the console log in very few cases, not after refresh, not after hard refresh, not after reopening the tab.  I couldn't see the pattern, but most of the time (like 99/100) I didn't see anything.

Hmmm, now it seems to work again.  Werid.


FundMe.sol Error: not found @chainlink/contracts/src/v0.8/vendor/SafeMathChainlink.sol

Hello,
I am trying to run this code in the terminal but I get this Error: not found @chainlink/contracts/src/v0.8/vendor/SafeMathChainlink.sol.
I updated the version for solidity.
I understand that SafeMath is no longer necessary for solidity 0.8 version, but I don't know how to modify the code so that it can work.
Thank you for help.
// SPDX-License-Identifier: MIT
pragma solidity 0.8.0;
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>contract FundMe {
using SafeMathChainlink for uint256;

mapping(address => uint256) public addressToAmountFunded;
address[] public funders;
address public owner;

  constructor() public {
    owner = msg.sender;
}

function fund() public payable {
    uint256 minimumUSD = 50 * 10**18;
    require(
        getConversionRate(msg.value) >= minimumUSD,
        "You need to spend more ETH!"
    );

    addressToAmountFunded[msg.sender] += msg.value;
    funders.push(msg.sender);
}

  function getVersion() public view returns (uint256) {
    AggregatorV3Interface priceFeed = AggregatorV3Interface(
        0xD4a33860578De61DBAbDc8BFdb98FD742fA7028e
    );
    return priceFeed.version();
}

function getPrice() public view returns (uint256) {
    AggregatorV3Interface priceFeed = AggregatorV3Interface(
        0xD4a33860578De61DBAbDc8BFdb98FD742fA7028e
    );
    (, int256 answer, , , ) = priceFeed.latestRoundData();
         return uint256(answer *<PHONE_NUMBER>0);
}

function getConversionRate(uint256 ethAmount)
    public
    view
    returns (uint256)
{
    uint256 ethPrice = getPrice();
    uint256 ethAmountInUsd = (ethPrice * ethAmount) /<PHONE_NUMBER>000000000;
         return ethAmountInUsd;
}

 modifier onlyOwner() {
          require(msg.sender == owner);

    _;
}

function withdraw() public payable onlyOwner {
    msg.sender.transfer(address(this).balance);

         for (
        uint256 funderIndex = 0;
        funderIndex < funders.length;
        funderIndex++
    ) {
        address funder = funders[funderIndex];
        addressToAmountFunded[funder] = 0;
    }
         funders = new address[](0);
}

}

Hi .. check if you install chainlink to your code .

Hello @ellyblueeyes
As you want to use the solidity version 0.8, you don't need the safeMath import.
Try deleting it.

@cromewar is correct. Additionally, the import is not found because it doesn't not exist at the given path. If you go to that file path on GitHub 'SafeMathChainlink.sol' does not exist. It stops after 'v0.7' due to lack of need with Solidity v0.8.0

Thank you!


create storages once

this PR focuses following items:

change config.json storage type default to "mem", so to facilitate local development and test;
keep all backend storage object in global cache, so to avoid creating on every op, by store name (not by host:port)

It is preliminary change for the PR introducing dummy database type (which is coming soon on top of this).
Verification it works
Verifying is relatively easy: simply start rkv (be default it uses mem database), and using curl to insert/read k-v pairs.

in the description, I'd suggest add some example curl commands just to keep it around.


Use ControlledGate instead of having ControlledXXYYGate, etc.

Also they can be simulated twice as fast, since you only need to affect half of the state vector, though this probably works best for qubits that represent areas of memory that are far apart.

They can inherit from ControlledGate and still override default_decompose() later.


refactor-decorators

Codecov Report

Merging #61 into master will decrease coverage by 6.64%.
The diff coverage is 91.47%.

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
- Coverage    97.5%   90.85%   -6.65%
==========================================
  Files          13       14       +1
  Lines         160      175      +15
  Branches       44       35       -9
==========================================
+ Hits          156      159       +3
- Misses          0        6       +6
- Partials        4       10       +6

Impacted Files
Coverage Δ

...ts/rx-form-mapper/src/lib/rx-form-mapper.module.ts
100% <ø> (ø)
:arrow_up:

...pper/src/lib/bind/reflect-metadata-design-types.ts
100% <ø> (ø)

projects/rx-form-mapper/src/lib/bind/field-type.ts
100% <100%> (ø)

...apper/src/lib/decorators/form-control.decorator.ts
100% <100%> (ø)
:arrow_up:

...-mapper/src/lib/decorators/form-group.decorator.ts
100% <100%> (ø)
:arrow_up:

projects/rx-form-mapper/src/lib/utils.ts
100% <100%> (ø)
:arrow_up:

...-mapper/src/lib/services/rx-form-mapper.service.ts
40% <50%> (-60%)
:arrow_down:

...r/src/lib/decorators/async-validators.decorator.ts
66.66% <66.66%> (ø)

...-mapper/src/lib/decorators/validators.decorator.ts
66.66% <66.66%> (ø)

...-mapper/src/lib/decorators/form-array.decorator.ts
75% <75%> (ø)

... and 8 more

Continue to review full report at Codecov.

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


Added map-pin icon

I have read the CONTRIBUTING.md file.
YES
What kind of change does this PR introduce?
Added map-pin icon
Demo
https://github.com/user-attachments/assets/8f786c1c-57e8-4d4d-8d0e-53864e0df4a5
Additional context
Add any other context or screenshots.

could you make the animation on hover less sharp, cause it jumps on hover? maybe you should just rotate a little on hover from right to left as a circle-help

could you make the animation on hover less sharp, cause it jumps on hover? maybe you should just rotate a little on hover from right to left as a circle-help

Made some changes for a more pleasing animation
https://github.com/user-attachments/assets/5c255c3e-bcd0-4636-9875-087a1acc0fd3


Remove einops dep

We shouldn't add a new dependency for just two reshape calls. We can do this the less readable way for now.

The bug is that this dependency is imported unconditionally, but not a actual pip depedency of the package. But I think the right solution is to avoid the new depedency for now. This is not important enough of a usage to warrant a new library dep.


Quantized

First off, this model is pretty good and I'm enjoying it almost as much as ChatGPT.
Would like to ask if it is possible to quantize the model using Optimum? The following code throws me an Onnx Error.
[ONNXRuntimeError] : 1 : FAIL : Deserialize tensor onnx::MatMul_5230 failed.tensorprotoutils.cc:640 onnxruntime::utils::TensorProtoToTensor External initializer: onnx::MatMul_5230 offset: 0 size to read: 41943040 given file_length: 16777216 are out of bounds or can not be read in full.
model_id="declare-lab/flan-alpaca-xl"
model = ORTModelForSeq2SeqLM.from_pretrained(model_id, from_transformers=True)

Thank you!

Exporting to ONNX and quantizing worked fine for me using Optimum. You can simply use optimum-cli like this:
optimum-cli export onnx --model declare-lab/flan-alpaca-base flan-alpaca-base_onnx/
optimum-cli onnxruntime quantize --onnx_model flan-alpaca-base_onnx/ --avx2

But keep in mind that by default only MatMul and Add operators are quantized, you need to pass a custom config if you want all the operators to be quantized. You can find the base model exported and fully quantized in my HF repo: https://huggingface.co/GarciaLnk/onnx-models-quantized/tree/main/flan-alpaca-base


Hashable prep

@tbkka @allevato I need your thoughts on this one.
I feel like this is a good thing before fully taking on SE0206 because it gets us completely using the conditional conformances added to collections.  But…
It comes at an API cost.  I have changed the signature of some of the Visitor methods.  I see two ways out of this:

Just accept it, and maybe bump the library to 1.1.x because most folks should be the signatures that are public that we did change.
Add methods with the old and new signatures and a default impl of the new back to the old to now break any code.

Option 1 seems better because it avoids the shims, and I think it remains safe because nothing changes for the generated code, so it doesn't require folks to regenerate, etc.
So, specific thoughts on:

Signature changes ok?  We really only have to make the signature change on the repeated methods, the single ones don't have to have it, but I figured for completeness it might be better to keep them in sync.
How to handing the api changes?

This kind of makes me want to just revisit our whole protocol hierarchy.
We've gone out of our way to provide a Message protocol without any associated types or Self constraints so that people can create heterogeneous collections of messages. Is this desired so frequently that we're making our lives harder to support it?
What if we just bite the bullet and do what Swift does in the standard library when it comes to wanting these kinds of collections? Let Message declare its Equatable and Hashable requirements and create a type-erasing AnyMessage box if someone really wants to handle them in a polymorphic way. I haven't completely thought this through yet so there could be rough edges, but AnyMessage could probably still conform to Message and forward traverse to the boxed message (just as AnyHashable conforms to Hashable and forwards its operations), so users could still serialize such collections without extra overhead at the call site. They would just have to unbox the underlying message explicitly instead of merely casting it (which they already have to do today anyway).

I'll admit, as I look at doing SE0206, the fact that we defined hashValue already worries me as we sorta want to kill that off also for the new method, but this also has me revisiting some other things…
We used protocols default impls to provide Equatable and Hashable and did them in such a way that they can be hooked.  With SE0206, I'm guessing we can do the same things with hash(into:), but I also wonder if we're going out of our way to support something that might not be used? And…
The recent Swift Forums thread Retroactive Conformances vs. Swift-in-the-OS also got me thinking about this.  As the thread mentions, there is potential for extension collisions not just with new system libraries, but also with multiple frameworks.  So as Swift reaching binary stability, are these customization points also opening the door for future issues for developers?  If a third party framework exposes Messages as part of their interface, and some code on top of that framework hooks hashing/equality for something they do with the messages; what happens when the framework is updated and they provide a hashing override also?  Does that suddenly break to code on top because it gets a different hashing/equality behavior?  Are we better off not exposing these hooks and if someone wants to override hashing/equality they just have to box the type like the forums thread talks about?
Anyway, this may be larger than a PR discussion.  Should open an issue for tracking this without it being tied to a PR?  Should we open the issue and continue the discussion, but push this PR ahead to go after full SE0206 support to see if it impacts the discussion and just make sure we resolve this larger question before we try to cut a release?  (or maybe we just cut 1.1.x and then 1.2.x/2.x if we do revisit this hierarchy)

We've gone out of our way to provide a Message protocol without any associated types or Self constraints so that people can create heterogeneous collections of messages. ...

The real motivation for me was to allow Message to be used as a function argument or return type.

The real motivation for me was to allow Message to be used as a function argument or return type.

What's your thoughts then on how we also try to make things hookable given the approaching binary stability work and potential issues when frameworks and multiple attempts to replace the hooks?

Dropping this.  It spawning some useful discussion and got us looking at things, but some of what was raised needs to be sorted out before deciding to go forward with this or the other things.

This would be a breaking API change, so we should maybe revisit for a future V2 effort.


Add tags to blog posts

More pages for SEO. It will also help separate out the newsletters which are kinda taking over the list of posts.

@Xantier that needs another +1 please since I rebased.

@Xantier that needs another +1 please since I rebased.


Add TSQL suffix to F1 keywords

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

Ok, I'll do that. I was trying to match the syntax from https://github.com/MicrosoftDocs/sql-docs/blob/live/docs/relational-databases/system-catalog-views/sys-query-store-query-transact-sql.md

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

Thanks @zikato!

Ok, I'll do that. I was trying to match the syntax from https://github.com/MicrosoftDocs/sql-docs/blob/live/docs/relational-databases/system-catalog-views/sys-query-store-query-transact-sql.md

We are reviewing F1 keywords across all our docset so I'll make a note of this as well. Thanks again.

#sign-off


Update chooser.py

As of wagtail 2.3 wagtail.admin.forms SearchForm has be deprecated and moved to wagtail.admin.forms.search SearchForm.
If this isn't the best thing to do for backwards compatibility I would suggest checking the wagtail version before importing SearchForm.

as of today wagtailnews is incompatible with current version of Wagtail is 2.5 without this changes.


Crash just after presenting picker view. if coredata concurrency debug option enabled.

My app crashes after the picker view is presented. I have enabled the coredata concurrency debugging option with the following in build arguments: -com.apple.CoreData.ConcurrencyDebug 1.
Seems like its using NSManagedObjectContext the wrong way. Although i don't know if this library uses it or its just ALAssetsLibrary messing up.

Thanks for submitting an issue, but I need more info :)
Which version of image picker?
Which iOS version?
Device?
Crash log?
BSImagePicker doesn't use core data

I'm closing this one as I didn't receive any more info and couldn't reproduce it myself.


Screen Shots for Formatting Reports Need to be Updated

Module: Design Reports in Power BI Desktop
Lab:  06-Design-Report-in-Power-BI-Desktop and 07-Design-Report-in-Power-BI-Desktop-enhanced
Task: Exercise 1, Task 4
Step: 17
Please see 1st screenshot. This is from the lab instructions for  06-Design-Report-in-Power-BI-Desktop  but applies to any lab instructions that ask for report formatting.
Since the lab instructions were developed, MS has updated PBI Desktop and changed the icon for the format pane. The lab (Go Deploy hosted) is using the February 2022 build of PBI Dekstop.  All the lab instructions still show the "Paintbrush roller" icon instead of this new icon. See 2nd screenshot. That is one issue.
The more challenging issue is that the PBI Desktop seems to have come up with different categorizations of the properties. For example, the 3rd screenshot is from Exercise 1, Task 4, Step 18 on formatting a slicer. It shows General and Selection controls.
But, as you can see from the 4th screenshot, the PBI Desktop is now calling these controls "Visual" and "General".  Except for using the search capabiity built into the Visualization Format pane, someone is not likely to be able to follow the lab instructions if they are trying to locate a Visualation Format property unless they Search for that property.
Anyway, someone needs to go through the lab instructions that concern formatting Visualizations and update them for the newer release of the PBI Desktop.
The following article from MS on the new Format pane might be of help: https://docs.microsoft.com/en-us/power-bi/fundamentals/desktop-format-pane
Thank you.
Repro steps:

1

.

This is one of the manifestations of https://github.com/MicrosoftLearning/PL-300-Microsoft-Power-BI-Data-Analyst/issues/1

@VitalyMCT Vitaly, thank you for the update.
For the record, I am using ALH Go Deploy which is using the February 2022 build of the PBI Desktop. But, as is already well documented in this forum, the Lab instructions on GitHub and, thus in Go Deploy, need  to be updated for the Data Model and the Report formatting.
I appreciate all your comments @VitalyMCT. Am very new to using GitHub so appreciate what you and your colleagues are saying on PL-300.
Thank you.

After my original post and reading the comments, I went back and looked again the MS documentation on the New Format Pane as given in the URL of my original post.
See the screenshot on How to turn off the New Format Pane preview feature.

Just tired it out in the Go Deploy lab .. and it works. The lab instruction screen shots now "match" what is in the PBI Desktop.
So, this seems to be a workaround until MS "updates" the GitHub instructions to match the New Format Pane.
Thank you.

MOC Labs Review confirms this issue.

All visuals have been updated.


BIT page navbar and footer part

navbar and footer is not matching with main pages.

Yes, pls fix it!


Add Json Scalar

Add JsonType to Hotchocolate.Types
Closes #3661

@pedro-coelho-ff The scalar needs to only support a JsonDocument type.

is this one ready for review?

is this one ready for review?

Yes, I believe it is 😅

on it.

I think I will take a swing at this so we get it into the release.

I think I will take a swing at this so we get it into the release.

I really appreciate it if you or any other that is more comfortable than me taking this one and putting it in the next release. 😅

@michaelstaib let me know if you want me to take it

@wonbyte and @michaelstaib sorry to bother you with this pull request.
Just a quick note, I had some performance issues with this approach, the solution was to use the AnyType but instead of JsonObject we created a custom JsonConverter and convert it to an object.


About original CenterNet performance on DOTA

Hello, Thanks for your nice work again!
I also want to adopt CenterNet by just adding an extra angle prediction branch as which mentioned in your paper. For data, I used the same data as BBAVecotrs, except for negative images that do not contain any objects. I changed the backbone from hourglass104 into ResNet101 and upsampling layers as the same as BBAVectors. The losses are just followed the original CenterNet and the weight of angle loss is 0.1. After training for 120 epochs, I got the mAP of 0.62, which is lower than your result of 0.6887.
Did I miss some key steps? Or the negative images are very important to improve the performance?

I didn't remove the negative images from the output of image split algorithms. For center+wh+theta,  I used the same training skills and architectures except for the output branches for a fair comparison.

I also wonder how to adopt CenterNet by adding an extra angle prediction branch, could you mention the code of this part?
Thanks @ yijingru @Tarazed

Change heads in the main.py, and also change the ground-truth accordingly (wh is the width and height of the box, angle is theta in Fig1a.  I used SmoothL1 loss for angle regression. And box decoding parts.
    heads = {'hm': num_classes[args.dataset],              'wh': 2,              'reg': 2,              'angle': 1 }

Change heads in the main.py:
    heads = {'hm': num_classes[args.dataset],
             'wh': 2,
             'reg': 2,
             'angle': 1
             }

and change the ground-truth accordingly (wh is the width and height of the box, angle \in [-90,0) is theta in Fig1a, note ground-truth theta is already calculated in datasets/base.py. I used SmoothL1 loss for angle regression.

I have found a strange phenomenon: when I used the 3rd channel of 'wh' head to predict angle, I got 0.636 of mAP, but when I  used an extra branch to predict angle as you said, I got 0.530 mAP... I set the random seed to 0, the performance of which is a little bit lower than random seed. I'm confusing as extra branch should be better.

Maybe training loss matters. I used weighted loss asloss =  hm_loss + wh_loss + off_loss + 0.1*angle_loss.

Maybe training loss matters. I used weighted loss asloss = hm_loss + wh_loss + off_loss + 0.1*angle_loss. The multi-tasks are hard to optimize sometimes. But we do see the angle is hard to learn for the network.

That's awesome! The loss weight of multi-task can significantly affect the results! After changed the weight of theta_loss, I got 0.68mAP! Thank you very much!

Maybe training loss matters. I used weighted loss asloss = hm_loss + wh_loss + off_loss + 0.1*angle_loss. The multi-tasks are hard to optimize sometimes. But we do see the angle is hard to learn for the network.

That's awesome! The loss weight of multi-task can significantly affect the results! After changed the weight of angle_loss, I got 0.68mAP! Thank you very much!

My pleasure.

i also want to know the loss weight of multi-task of 0.68mAP, can you share it? thanks very much! @Tarazed

i also want to know the loss weight of multi-task of 0.68mAP, can you share it? thanks very much! @Tarazed

just the same as what the author replied, loss = hm_loss + wh_loss + off_loss + 0.1*angle_loss

@Tarazed 请问用原始的centernet加一个额外的角度，除了main.py，还需要修改哪里嘛

@Tarazed 请问用原始的centernet加一个额外的角度，除了main.py，还需要修改哪里嘛

Change the ground-truth accordingly in base.py


1.10 Only latest draft form data available on frontend

All drafts are showing in the Drafts tabs, but only the most recent draft created, populates data. Drafts saved from edit page will also persist their data.
steps to recreate:

Start a new 1.10 form and enter some data and save the draft.
Create a second 1.10 form, enter some different data, and save the draft.
Go to drafts tab and try to view both forms.
Only one will have data.
There is also a chance that the data from the one form that persisted the data does not populate until a refresh.

Don has a demo at 4pm today to show the union the new form. He is the one that found the bug as he was pulling up one of his saved drafts.

@morezooomm @Andrew-Vargas-bcgov
This should be fixed on TEST. Could you please test it out after a hard-refresh on your browser to make sure you have the latest version of the TOOL? Thank you

Iman,
How do I do a “hard refresh” of my browser?
Don Smallwood, Director of Investigations & Intake
Employee Relations Division | BC Public Service Agency
167 Lorne Street | Kamloops BC | V2C 1V9
Office:<PHONE_NUMBER> | Mobile:<PHONE_NUMBER>
Living with gratitude and respect on the unceded territory of the Splatsin
Government confidentiality and privilege requirements apply to this message and any attachments.  If you are not the intended recipient, you are hereby notified that any review, retransmission, conversion to hard copy, copying, circulation or other use is strictly prohibited. If you are not the intended recipient, please notify the sender immediately, and delete this message and any attachments from both your inbox and deleted items folder.
@.***
From: iman-jamali-fw @.>
Sent: Tuesday, February 21, 2023 12:10 PM
To: bcgov/digital-journeys @.>
Cc: Smallwood, Don PSA:EX @.>; Mention @.>
Subject: Re: [bcgov/digital-journeys] 1.10 Only latest draft form data available on frontend (Issue #936)
[EXTERNAL] This email came from an external source. Only open attachments or links that you are expecting from a known sender.
@morezooommhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmorezooomm&data=05|01|Don.Smallwood%40gov.bc.ca|d438237354224c4ed98c08db1447a068|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126070076922507|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=89P51hMRs8262pql4Qm76EE4lim%2Firylc3nVUAas4ho%3D&reserved=0 @Andrew-Vargas-bcgovhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAndrew-Vargas-bcgov&data=05|01|Don.Smallwood%40gov.bc.ca|d438237354224c4ed98c08db1447a068|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126070076922507|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=2%2FJcrHH4JnC712OvpPF%2FTN4EoaW%2BxW9r3pMq3mRcd%2F4%3D&reserved=0
This should be fixed on TEST. Could you please test it out after a hard-refresh on your browser to make sure you have the latest version of the TOOL? Thank you
—
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbcgov%2Fdigital-journeys%2Fissues%2F936%23issuecomment-1439033023&data=05|01|Don.Smallwood%40gov.bc.ca|d438237354224c4ed98c08db1447a068|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126070076922507|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=RFxS5SVEwAAOj9RYdik9DIDR9PSAx%2Fj46nYoptNRLOU%3D&reserved=0, or unsubscribehttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA37NTNOKTUZYPEKR2AU5GNLWYUOJ5ANCNFSM6AAAAAAU7W6XDQ&data=05|01|Don.Smallwood%40gov.bc.ca|d438237354224c4ed98c08db1447a068|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126070076922507|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=4RdfxwZcDeWfcS2OihBR%2BTem2srALNKjYWabEbC365o%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@morezooomm
Please either follow the instruction below or simply open a private/incognito page on your browser and then go to the TEST.

Sorry, I don’t see any “instructions below” on how to do a hard refresh of my browser.
I have also never opened a private/incognito page on my browser, so not sure how to do that either.
I did test the site with the link you provided below and the problem is still there for me.  The moment I click on a draft form, it jumps me to a new form.
Don Smallwood, Director of Investigations & Intake
Employee Relations Division | BC Public Service Agency
167 Lorne Street | Kamloops BC | V2C 1V9
Office:<PHONE_NUMBER> | Mobile:<PHONE_NUMBER>
Living with gratitude and respect on the unceded territory of the Splatsin
Government confidentiality and privilege requirements apply to this message and any attachments.  If you are not the intended recipient, you are hereby notified that any review, retransmission, conversion to hard copy, copying, circulation or other use is strictly prohibited. If you are not the intended recipient, please notify the sender immediately, and delete this message and any attachments from both your inbox and deleted items folder.
@.***
From: iman-jamali-fw @.>
Sent: Tuesday, February 21, 2023 12:33 PM
To: bcgov/digital-journeys @.>
Cc: Smallwood, Don PSA:EX @.>; Mention @.>
Subject: Re: [bcgov/digital-journeys] 1.10 Only latest draft form data available on frontend (Issue #936)
[EXTERNAL] This email came from an external source. Only open attachments or links that you are expecting from a known sender.
@morezooommhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmorezooomm&data=05|01|Don.Smallwood%40gov.bc.ca|1ba29f2003ac4d61265708db144ad819|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126083899413513|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=VU4QCG9demzByep2BoVUhzDGYnMr7QQXejvBR1hvO90%3D&reserved=0
Please either follow the instruction below or simply open a private/incognito page on your browser and then go to the TESThttps://digital-journeys-test.apps.silver.devops.gov.bc.ca/.
[cid:~WRD0002.jpg]https://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fcamo.githubusercontent.com%2F58f1f90557db28ab1d72b15889a65a44b09d176e56af5aa65a95f1b6d4443d59%2F68747470733a2f2f696d616765732e7a656e68756275736572636f6e74656e742e636f6d2f3631646463343835343766343736656433663335636132382f64343166333636362d316631302d346435312d616162622d396266653664343438386235&data=05|01|Don.Smallwood%40gov.bc.ca|1ba29f2003ac4d61265708db144ad819|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126083899413513|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=%2BN0xmLpr5hu0MVJaWHyYnDCBAzLgFR0BRlnpmOE1lzo%3D&reserved=0
—
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbcgov%2Fdigital-journeys%2Fissues%2F936%23issuecomment-1439056212&data=05|01|Don.Smallwood%40gov.bc.ca|1ba29f2003ac4d61265708db144ad819|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126083899413513|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=7HV30SzPmZfCiEBnWD5paWXe1FqAJBDhzYg9w19hBtk%3D&reserved=0, or unsubscribehttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA37NTNLW7JAOV2SLIMQ4TM3WYURAJANCNFSM6AAAAAAU7W6XDQ&data=05|01|Don.Smallwood%40gov.bc.ca|1ba29f2003ac4d61265708db144ad819|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126083899413513|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=vZS3UW4%2BHf3%2B4gw%2B365q0wgB7B3UhCZeoL0KYMOSvUc%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@morezooomm
No problem, I think the instruction which was an image didn't load on your side.
Please let me know which browser and OS (Windows or Mac) are you using.

I am using Microsoft Edge on Windows.
Don Smallwood, Director of Investigations & Intake
Employee Relations Division | BC Public Service Agency
167 Lorne Street | Kamloops BC | V2C 1V9
Office:<PHONE_NUMBER> | Mobile:<PHONE_NUMBER>
Living with gratitude and respect on the unceded territory of the Splatsin
Government confidentiality and privilege requirements apply to this message and any attachments.  If you are not the intended recipient, you are hereby notified that any review, retransmission, conversion to hard copy, copying, circulation or other use is strictly prohibited. If you are not the intended recipient, please notify the sender immediately, and delete this message and any attachments from both your inbox and deleted items folder.
@.***
From: iman-jamali-fw @.>
Sent: Tuesday, February 21, 2023 12:53 PM
To: bcgov/digital-journeys @.>
Cc: Smallwood, Don PSA:EX @.>; Mention @.>
Subject: Re: [bcgov/digital-journeys] 1.10 Only latest draft form data available on frontend (Issue #936)
[EXTERNAL] This email came from an external source. Only open attachments or links that you are expecting from a known sender.
@morezooommhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmorezooomm&data=05|01|Don.Smallwood%40gov.bc.ca|2d7d94c03de1409f66df08db144d9a95|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126095751728418|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=jrlt%2BcQDtX4UFbRsSB4IW7G8HMSoc1Ofj29GxfYm2JQ%3D&reserved=0
No problem, I think the instruction which was an image didn't load on your side.
Please let me know which browser and OS (Windows or Mac) are you using.
—
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbcgov%2Fdigital-journeys%2Fissues%2F936%23issuecomment-1439076686&data=05|01|Don.Smallwood%40gov.bc.ca|2d7d94c03de1409f66df08db144d9a95|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126095751728418|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=04qeePFeMk9x%2FrQA7yO6qzUrLFAPY%2BEyV2QhZWoRKWs%3D&reserved=0, or unsubscribehttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA37NTNJ3UZG7434SCELVMXLWYUTKLANCNFSM6AAAAAAU7W6XDQ&data=05|01|Don.Smallwood%40gov.bc.ca|2d7d94c03de1409f66df08db144d9a95|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126095751728418|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=GWPC3BPwifuqKblv7No6SKKLXlSBBXpqRpgP3lV8Fnw%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@morezooomm
To hard refresh Edge on Windows:

Hold the Ctrl key and click the Refresh button.
Or, hold the Ctrl key and press the F5 key.

OR to open the TEST in a InPrivate window in Edge, please select and hold (right-click) TEST LINK and select Open link in InPrivate window.

I did a refresh of the browser but the same problem still exists.
I followed the directions below to launch a private browser, but didn’t get that option available.
Don Smallwood, Director of Investigations & Intake
Employee Relations Division | BC Public Service Agency
167 Lorne Street | Kamloops BC | V2C 1V9
Office:<PHONE_NUMBER> | Mobile:<PHONE_NUMBER>
Living with gratitude and respect on the unceded territory of the Splatsin
Government confidentiality and privilege requirements apply to this message and any attachments.  If you are not the intended recipient, you are hereby notified that any review, retransmission, conversion to hard copy, copying, circulation or other use is strictly prohibited. If you are not the intended recipient, please notify the sender immediately, and delete this message and any attachments from both your inbox and deleted items folder.
@.***
From: iman-jamali-fw @.>
Sent: Tuesday, February 21, 2023 1:32 PM
To: bcgov/digital-journeys @.>
Cc: Smallwood, Don PSA:EX @.>; Mention @.>
Subject: Re: [bcgov/digital-journeys] 1.10 Only latest draft form data available on frontend (Issue #936)
[EXTERNAL] This email came from an external source. Only open attachments or links that you are expecting from a known sender.
@morezooommhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmorezooomm&data=05|01|Don.Smallwood%40gov.bc.ca|9458d4a60cc24c8c122f08db14530381|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126118987406321|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=v2cXuWvlYDiON1s%2BuBKLiMpw%2Fr7cSTIdOfRyYDFOloY%3D&reserved=0
To hard refresh Edge on Windows:

Hold the Ctrl key and click the Refresh button.
Or, hold the Ctrl key and press the F5 key.

OR to open the TEST in a InPrivate window in Edge, please select and hold (right-click) TEST LINKhttps://digital-journeys-test.apps.silver.devops.gov.bc.ca/form and select Open link in InPrivate window.
—
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbcgov%2Fdigital-journeys%2Fissues%2F936%23issuecomment-1439118525&data=05|01|Don.Smallwood%40gov.bc.ca|9458d4a60cc24c8c122f08db14530381|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126118987406321|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=6Ftsow0lLyYvJUDEgGhdKzIf08VGOwacaDqPM85Nv2c%3D&reserved=0, or unsubscribehttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA37NTNI2TDGVY4YDUHP5KWTWYUX3TANCNFSM6AAAAAAU7W6XDQ&data=05|01|Don.Smallwood%40gov.bc.ca|9458d4a60cc24c8c122f08db14530381|6fdb52003d0d4a8ab036d3685e359adc|0|0|638126118987406321|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=dikJGbpTDwdF5t3TNjoJ%2FXZkNZj135mRQyE8lOgsz5w%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Not on issue, Iman & Don connected on Feb 22 and did not encounter this issue

Agreed, we can close this one.
Don Smallwood, Director of Investigations & Intake
Employee Relations Division | BC Public Service Agency
167 Lorne Street | Kamloops BC | V2C 1V9
Office:<PHONE_NUMBER> | Mobile:<PHONE_NUMBER>
Living with gratitude and respect on the unceded territory of the Splatsin
Government confidentiality and privilege requirements apply to this message and any attachments.  If you are not the intended recipient, you are hereby notified that any review, retransmission, conversion to hard copy, copying, circulation or other use is strictly prohibited. If you are not the intended recipient, please notify the sender immediately, and delete this message and any attachments from both your inbox and deleted items folder.
@.***
From: fazil-ey @.>
Sent: Thursday, February 23, 2023 10:39 AM
To: bcgov/digital-journeys @.>
Cc: Smallwood, Don PSA:EX @.>; Mention @.>
Subject: Re: [bcgov/digital-journeys] 1.10 Only latest draft form data available on frontend (Issue #936)
[EXTERNAL] This email came from an external source. Only open attachments or links that you are expecting from a known sender.
Not on issue, Iman & Don connected on Feb 22 and did not encounter this issue
—
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbcgov%2Fdigital-journeys%2Fissues%2F936%23issuecomment-1442253047&data=05|01|Don.Smallwood%40gov.bc.ca|65700877f8f342b6863208db15cd4397|6fdb52003d0d4a8ab036d3685e359adc|0|0|638127743566779854|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=9BoiyLm9djeQYV4tcqnqBsU%2BONMOMMq3zf8kUHGwG8g%3D&reserved=0, or unsubscribehttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA37NTNM5IR73YUVW2HFKCVDWY6VFFANCNFSM6AAAAAAU7W6XDQ&data=05|01|Don.Smallwood%40gov.bc.ca|65700877f8f342b6863208db15cd4397|6fdb52003d0d4a8ab036d3685e359adc|0|0|638127743566779854|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=cnHJMlAf5o4fQhBHQoDd3OCDgophcWDInogjPyd3hwY%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


Create a GitLab token page

What type of suggestions?
Missing Information
What page is concerned?
No response
What is the issue?
Create a GitLab token page like the GitHub one
Code of Conduct

[X] I agree to follow this project's Code of Conduct

The scope is api for now.

It was created https://docs.kubefirst.io/common/gitAuth?git_provider=gitlab


Add Contract Pack: The K-Files

line 14 needs your closed square bracket ] and a comma

Can't see anything obviously wrong - may be the AVC file though, I'll look at that later.

Huh. I presume this is the problem:
2251 [1] ERROR CKAN.RelationshipResolver (null) - Dependency on ContractConfigurator found, but nothing provides it.
ContractConfigurator is definitely listed (and my other contract packs are indexed fine)

Closing and re-opening to force a new Jenkins build.

It took me a long while to figure out what was up, lol.
You've specified a minimum ContractConfigurator version of 1.8.0. 1.8.0 of CC is only compatible with ksp_version 1.0.5. Meanwhile, the ksp_version compatibility in your .version file indicates your mod is only compatible up to 1.0.4.
2 potential fixes are 1) drop the version requirement for CC 2) create a new release of your mod with your up-to-date .version file

I'll do both, as I'll have to make a new release for CKAN anyway for 1.0.5

This passes and works for 1.0.4. I'll note that the compatibility for 0.90.0 can't be fulfilled (because of the min_version of CC set) but that's not a significant issue. Merging this with my thanks.

Oh yeah, that's just a leftover as I copied KSS's netkan file. Cheers for your help!


bnxt_en version number confusing in the cilium documentation

Is there an existing issue for this?

[X] I have searched the existing issues

Version
equal or higher than v1.14.13 and lower than v1.15.0
What happened?
We're running cilium in a bare metal K8s cluster:

k8s version: v1.21.14
cilium version: v1.11.4
os: Linux n62-250-235.sin2a 5.10.135.bsk.3-amd64 #5.10.135.bsk.3 SMP Debian 5.10.135.bsk.3 Thu Mar 9 03:06:59 UTC x86_64 GNU/Linux
cilium is running with kubeproxy replacement strict mode.

the cilium agent pod failed to start with:
level=fatal msg="Failed to compile XDP program" error="program cil_xdp_root: attaching XDP program to interface eth0: cannot allocate memory" subsys=datapath-loader
And the NIC driver info is:
> ethtool -i eth0
driver: bnxt_en
version: 1.10.2-[<IP_ADDRESS>](http://<IP_ADDRESS>/)
firmware-version: [<IP_ADDRESS>/pkg](http://<IP_ADDRESS>/pkg) [<IP_ADDRESS>](http://<IP_ADDRESS>/)
expansion-rom-version:
bus-info: 0000:17:00.0
supports-statistics: yes
supports-test: yes
supports-eeprom-access: yes
supports-register-dump: yes
supports-priv-flags: yes

The version is 1.10.2-[<IP_ADDRESS>].
Per cilium documentation here: https://docs.cilium.io/en/latest/bpf/progtypes/ , it required broadcom bnxt_en >= 4.11 in order to support XDP.
However, when we checked bnxt_en drivers, there is no such a format of bnxt_en version like 4.11 or higher.
for example, here is an output from apt:
apt list bnxt-en-dkms -a
Listing... Done
bnxt-en-dkms/buster-private,now <IP_ADDRESS>.4.8.0 all [installed]
bnxt-en-dkms/buster-private <IP_ADDRESS>.0.142.0 all
bnxt-en-dkms/buster-private <IP_ADDRESS>.1.236.1 all
bnxt-en-dkms/buster-private <IP_ADDRESS>.1.88.0 all

also here:
https://www.broadcom.com/products/ethernet-connectivity/network-adapters/n425g
https://www.broadcom.com/products/ethernet-connectivity/network-adapters/p425g
How can we reproduce the issue?
In the environment above, the cilium agent failed every time with the exact same error.
Cilium Version
v1.11.4
Kernel Version
Linux n62-250-235.sin2a 5.10.135.bsk.3-amd64 #5.10.135.bsk.3 SMP Debian 5.10.135.bsk.3 Thu Mar 9 03:06:59 UTC x86_64 GNU/Linux
Kubernetes Version
v1.21.14
Regression
No response
Sysdump
No response
Relevant log output
level=fatal msg="Failed to compile XDP program" error="program cil_xdp_root: attaching XDP program to interface eth0: cannot allocate memory" subsys=datapath-loader

### Anything else?

_No response_

### Cilium Users Document

- [ ] Are you a user of Cilium? Please add yourself to the [Users doc](https://github.com/cilium/cilium/blob/main/USERS.md)

### Code of Conduct

- [X] I agree to follow this project's Code of Conduct

Thanks for logging this issue @rockc2020.
Part of the reason that this discrepancy exists is that the documentation you were looking at is for the latest version of Cilium - which is the currently-in-development 1.17 release, not for the 1.11 version that you are using.
In fact, the 1.11 Cilium release has been unsupported for nearly a year, and the 1.21 Kubernetes version for even longer. I'd recommend upggrading both as soon as possible.

Thanks for logging this issue @rockc2020.
Part of the reason that this discrepancy exists is that the documentation you were looking at is for the latest version of Cilium - which is the currently-in-development 1.17 release, not for the 1.11 version that you are using.
In fact, the 1.11 Cilium release has been unsupported for nearly a year, and the 1.21 Kubernetes version for even longer. I'd recommend upggrading both as soon as possible.

Thank you for the reply. Yeah, both versions are quite old and upgrade is a bit complicated on bare metal :( I'd also check doc for 1.11 for some references.

Per cilium documentation here: https://docs.cilium.io/en/latest/bpf/progtypes/ , it required broadcom bnxt_en >= 4.11 in order to support XDP.

The documentation refers to the minimum kernel version that includes a broadcom driver with XDP support. So you being on a 5.10 based kernel should be fine.
Let's please revisit this once you're on a current Cilium release.


Upgrade Bootstrap lib to v4+

Upgrade bootstrap npm package to latest version

Until this happens... is there any workaround to using this in a Bootstrap 4 application? The display is mangled for me and I can't figure out how to fix without conflicting with Bootstrap 4.

You need to override some css classes used in the library. I think the easiest method is to import the Bootstrap 3.x css file provided by the v3 npm package in your specific component. This is ugly, I know.

Shouldn't conflict if you're creating a wrapper component just for the cron-editor and import (or maybe copy-paste?) v3 css file into that single component.
Unfortunately I don't have a timeline for v4 upgrade.

I see there is a Pull Request for the upgrade for Bootstrap 4. Any timeline when this can get implemented?


Test processors

There are currently minimal tests for processors (see napari/_qt/_qapp_model/_tests/test_processors.py
They should at the very least be run.

Hi there, just in case, I think the napari/_qt/_qapp_model/_tests/test_processors.py test file is currently being run. For example:
https://github.com/napari/napari/actions/runs/10906488388/job/30270858900?pr=7269#step:13:349
Is there something else to be done here or maybe this could be closed or maybe I missing something related with the test file execution on CI?

@dalthviz the file is being run, but there's only one processor being tested in there (_add_layer_data_to_viewer). Ideally all of the processors in napari/_qt/_qapp_model/injection/_qprocessors.py would be tested individually.
It might be nice to actually check that as well as the functions themselves being tested, we also test a round trip where we execute an app-model action that returns a value that one of these processors should respond to, and that the processor is called (even with a mock). Maybe that's moving too far into testing app-model itself though so not sure about that last bit.

Maybe that's moving too far into testing app-model itself though so not sure about that last bit.

Maybe you are right but it may help with picking up things happening downstream that affects us, e.g., https://github.com/napari/napari/issues/7129 ? Certainly, a test would have prevented a lot of debugging time in that issue! :sweat_smile:


More reliable extensions

Based on the problem with #989, it makes sense to add better error handling. The code in csharpier probably needs to use a try/catch around the call to format a file. If the work to get GRPC working is done, it would be easier to report the failure back to the extension.
Maybe it is also possible to detect the crashing/hung csharpier process better on the extension side. I can re-introduce the problem and try to come up with a better way to deal with a similar situation in the future.

Also some logging on the csharpier end for what is happing with either GRPC or the piping files mode is a good idea.

Also ensure that if the extensions detect a hung csharpier they don't retry indefinitely.

Also deal with the new version string of 0.26.1+3d5768a543259cd498064d1ff364f4c741905ab6

Some exceptions I saw around the time Rider finally hung for me
2023-11-18 10:28:21,937 [   8786]   WARN - #c.i.c.CSharpierLogger - Exception while running 'dotnet csharpier --version' in C:\Users\bela\AppData\Local\CSharpier\0.26.0
java.lang.NullPointerException: Cannot invoke "String.equals(Object)" because "output" is null
	at com.intellij.csharpier.CustomPathInstaller.ensureVersionInstalled(CustomPathInstaller.java:30)
	at com.intellij.csharpier.CSharpierProcessProvider.setupCSharpierProcess(CSharpierProcessProvider.java:203)
	at com.intellij.csharpier.CSharpierProcessProvider.findAndWarmProcess(CSharpierProcessProvider.java:92)
	at com.intellij.csharpier.CSharpierProcessProvider.<init>(CSharpierProcessProvider.java:45)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:499)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:480)

2023-11-18 10:28:21,930 [   8779] SEVERE - #c.i.c.CSharpierLogger - Cannot run program "C:\Users\bela\AppData\Local\CSharpier\0.26.0\dotnet-csharpier" (in directory "C:\Users\bela\AppData\Local\CSharpier\0.26.0"): CreateProcess error=2, The system cannot find the file specified
java.lang.Throwable: Cannot run program "C:\Users\bela\AppData\Local\CSharpier\0.26.0\dotnet-csharpier" (in directory "C:\Users\bela\AppData\Local\CSharpier\0.26.0"): CreateProcess error=2, The system cannot find the file specified
	at com.intellij.openapi.diagnostic.Logger.error(Logger.java:370)
	at com.intellij.csharpier.ProcessHelper.ExecuteCommand(ProcessHelper.java:53)
	at com.intellij.csharpier.CustomPathInstaller.ensureVersionInstalled(CustomPathInstaller.java:26)
	at com.intellij.csharpier.CSharpierProcessProvider.setupCSharpierProcess(CSharpierProcessProvider.java:203)
	at com.intellij.csharpier.CSharpierProcessProvider.findAndWarmProcess(CSharpierProcessProvider.java:92)
	at com.intellij.csharpier.CSharpierProcessProvider.<init>(CSharpierProcessProvider.java:45)

On my old SSD, rider experiences the hangs. VSCode gets the loading forever thing, and VS gets some weird exceptions while trying to setup csharpier.

VSCode needs a link to read more

Look at #1048 a bit

I also experience the issue with "System.UnauthorizedAccessException: Access to the path 'CSharpier.dll' is denied.".
When I open the first solution csharpier works fine, but i get this error as soon as I open a second solution. Formatting in the first solution still works fine.
I use visual studio 17.8.3 on windows and csharpier 0.26.5 installed globally.


Improve Check for multiple instantiations

Currently the check for multiple instantiations is done via the static bool property _metricsBuilt. Which will throw the an InvalidOperationException if that bool is set to true when someone calls one of the WebHostBuilderExtension methods. This works fine in the 'normal' use case when you would expect only one instance of AppMetrics in that program. However if you attempt to test your website/api with Microsoft.AspNetCore.TestHost.TestServer its possible that you would want to standup and teardown multiple different configurations under test. In this use case the static bool will be set to true after the first TestServer has been created and then error for any other instances even though the appmetrics services haven't been registered for that instance.
Can we change the way this check is done, the ASP.NET MVC project use a MvcMarkerService that they check for inside the IServiceCollection to determine if they already registered their services.
I have done a rough pass at these changes to help illustrate the idea on https://github.com/countrywide/AspNetCore/commit/afdcf967dd03d9f5b48c4a136fca17a3c3e35727
We are able to work around this by having one single instance that we share between our tests but we are unable to test changes to our startup configuration due to this design.
Thanks

@Decad thanks for the details. I would prefer not to force IServiceProvider to be built more than once.
I'll have a think about your scenario.

I'd prefer to leave this as is, as mentioned I don't think its a good idea to have App Metrics build the IServiceProvider without the user knowing.
I do use the Marker Service approach in other cases when the service provider is already built, in this particular case it is before the fact, ASP.NET Hosting also uses a static variable to ensure only one instance of IWebHost, see here.
In your case re "setup and tear down multiple different configurations under test", could you take a similar approach to that used in App.Metrics.AspNetCore.Integration.Facts where IClassFixtures are used?
I would rather have an exception thrown if multiple instances of IMetrics  are accidentally attempted to be registered through App Metrics configuration since allowing it could cause run-time confusion.
Thoughts?


[multikueue] add support for custom multikueue controller

What type of PR is this?
/kind feature

What this PR does / why we need it:
Provide a way to pass custom controller to multikueue admission check controller.
Which issue(s) this PR fixes:

Fixes #2349
Special notes for your reviewer:
Consider base branch:  #2373
Does this PR introduce a user-facing change?

NONE

/assign @trasc

With the refactor and documentation PRs merged, can you know propose an update to the KEP to add custom jobs support?

With the refactor and documentation PRs merged, can you now propose an update to the KEP to add custom jobs support?

Yes, I am working on KEP now.

/retest

/ok-to-test
/test-all

/retest

/onhold


mvn autodeploy

take secure vars from env

Hi @serious6, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

ok finally it works


Bundle version 3.5.0 not available?

I see version 3.5.0 has been released, but I still get version 3.3.0 when starting my function app. Seems like version 3.5.0 isn't available?
I get this output when starting my function app:
Fetching information on versions of extension bundle Microsoft.Azure.Functions.ExtensionBundle available on https://functionscdn.azureedge.net/public/ExtensionBundles/Microsoft.Azure.Functions.ExtensionBundle/index.json
The list of versions on that URL does not include 3.5.0.

/cc @soninaren
Anyone reading these issues? It's kind of urgent for us to get this issue of ours fixed.

/cc @soninaren
Anyone reading these issues? It's kind of urgent for us to get this issue of ours fixed.

Apologies for a delayed response here. We are in the process of releasing bundle version 3.5.0. I think one of the release steps was ordered incorrectly which created the release of GitHub much sooner than expected. We have updated the release process since then. Going to remove the release from GitHub until it is actually released as well.


Support more sqlsrv options

The sqlsrv driver for PHP supports a long list of options, but relatively few are supported by CodeIgniter's sqlsrv driver.
Can the driver be extended to support the full range of options?
One approach would be to define a bunch of optional database parameters. However, since the SQL Server driver may add new options at any time, and since most of these new options would only have meaning for the SQL Server driver, adding an optional dbdriver_params setting makes sense to me. For example:
$db['default'] = [
    'hostname' => 'sql-server',
    'username' => 'dbuser',
    'password' => 'secret',
    'database' => 'dbname',
    'dbdriver' => 'sqlsrv',
    'dbdriver_params' => [
        'LoginTimeout' => 10,
        'MultiSubnetFailover' => 'yes',
        'ApplicationIntent' => 'ReadOnly',
    ]
];

I can submit a PR if there's interest (and if this approach is acceptable).

Some of these are integrated to match our own, universal settings and others (like Scrollable) we enforce, so adding something like dbdriver_params would create possible conflicts between two sets of configurations.
I suppose we could blacklist the ones that could conflict, but then there's an even bigger issue - doing it for sqlsrv means doing it for all drivers and all kinds of platform-specific settings, often altering behavior and making our Query Builder inconsistent, beating its entire purpose.
So that's a No by default, but if you have some particular use case or otherwise strong reasoning for including a particular feature, we could work on that.

Thanks for the quick reply.
So, thinking out loud, I think I'd argue that there are already a few ways to break encapsulation and/or configure conflicting settings: directly access the conn_id and use database APIs to put it in a state that CodeIgniter doesn't expect, or use PDO with a DSN that CodeIgniter doesn't parse. Since there's already some possibility of a developer making problems for themselves by doing something CodeIgniter doesn't expect, and since it would be potentially very hard for a web framework to fully support every possible database option, it seems valuable to me to offer a standard "escape hatch" of sorts to pass through arbitrary options, with the documented understanding that some options could cause problems.
But I've obviously a lot less familiar with the CodeIgniter database layer's design and tradeoffs than you are, so I could be wrong.
The particular SQL Server setting that we're missing is MultiSubnetFailover.
After opening this issue, I noticed that CI_DB_pdo_sqlsrv_driver does support several SQL Server options (as individual config settings, rather than via a generic catch-all dbdriver_params).  However, it doesn't support MultiSubnetFailover, and CI_DB_sqlsrv_driver doesn't support most SQL Server options.  Therefore, if you don't want to support generic dbdriver_params, would something like this work?

Add MultiSubnetFailover (and other options as appropriate; I could review the SQL Server docs to see if anything else looks like it should be added) to the list of supported options within CI_DB_pdo_sqlsrv_driver.
Combine and consolidate the list of supported options between CI_DB_pdo_sqlsrv_driver and CI_DB_sqlsrv_driver: instead of duplicating code, have a string array giving supported config settings, and update both classes to refer to that.  (Would it work to make this a static propery of CI_DB_sqlsrv_driver and have CI_DB_pdo_sqlsrv_driver access that?)

So, thinking out loud, I think I'd argue that there are already a few ways to break encapsulation and/or configure conflicting settings: directly access the conn_id and use database APIs to put it in a state that CodeIgniter doesn't expect, or use PDO with a DSN that CodeIgniter doesn't parse.

Yes, but that's not deliberate. It's just not worth the effort necessary to prevent these hacks, and you really have to know what you're doing in order to utilize them.

Since there's already some possibility of a developer making problems for themselves by doing something CodeIgniter doesn't expect, and since it would be potentially very hard for a web framework to fully support every possible database option, it seems valuable to me to offer a standard "escape hatch" of sorts to pass through arbitrary options, with the documented understanding that some options could cause problems.

Well, it seems valuable to you because you want it. :)
In my experience, documenting that something can have negative side effects isn't sufficient. The moment we introduce a feature, any kind of feature, people start expecting it to be fully supported and seamlessly integrated with the rest of the framework. Unfortunately, in some cases that's just not possible.
I wish that such DB options were limited to an extent where they didn't change SQL behavior, in which case I'd have nothing against something like dbdriver_params, but alas that's not the case.

The particular SQL Server setting that we're missing is MultiSubnetFailover.
After opening this issue, I noticed that CI_DB_pdo_sqlsrv_driver does support several SQL Server options (as individual config settings, rather than via a generic catch-all dbdriver_params). However, it doesn't support MultiSubnetFailover, and CI_DB_sqlsrv_driver doesn't support most SQL Server options. Therefore, if you don't want to support generic dbdriver_params, would something like this work?

Add MultiSubnetFailover (and other options as appropriate; I could review the SQL Server docs to see if anything else looks like it should be added) to the list of supported options within CI_DB_pdo_sqlsrv_driver.
Combine and consolidate the list of supported options between CI_DB_pdo_sqlsrv_driver and CI_DB_sqlsrv_driver: instead of duplicating code, have a string array giving supported config settings, and update both classes to refer to that. (Would it work to make this a static propery of CI_DB_sqlsrv_driver and have CI_DB_pdo_sqlsrv_driver access that?)

I guess the first thing can work, yes, but not the second one ... it's not a matter of just looking up whether an option exists or not. sqlsrv_connect() expects an array of options, while PDO_SQLSRV needs them written into the connection string; it's also possible that values need to be sent in different formats and/or even that the same option exists under different names.

Fair enough. I do think I'd make a different trade-off, but I really appreciate the explanation.
Regarding CI_DB_pdo_sqlsrv_driver versus CI_DB_sqlsrv_driver: It seems like it ought to work. sqlsrv_connect and PDO_SQLSRV are documented as taking the same options. I could imagine data type handling for a DSN string versus an array being a problem - except that CI_DB_pdo_sqlsrv_driver currently simply concatenates strings together to make its DSN, so any questions of data type handling do not appear to be a problem in practice.
And, if you don't want to worry about this right now, I do have a workaround - we're using the PDO driver with a DSN, and that lets us pass in whatever parameters we want. It just seemed odd to me that CodeIgniter has three different ways to connect to SQL Server (PDO with DSN, PDO with individual database config options, and sqlsrv), and each way of connecting has a different set of supported SQL Server options (totally flexible with DSN, somewhat complete with PDO with individual options, and minimal with sqlsrv).
Thanks.

I could imagine data type handling for a DSN string versus an array being a problem - except that CI_DB_pdo_sqlsrv_driver currently simply concatenates strings together to make its DSN, so any questions of data type handling do not appear to be a problem in practice.

Well, yes - that's what I meant. Serializing key/value pairs from array into a string isn't the same mechanism as simply passing an array of options.
Anyway, if you can provide me with an assorted list of options that you think should be added (and where), we'll review and eventually implement them. A PR would be even better.


Allow IdleReplicaCount less than or equal to MinReplicaCount

Proposal
Keda doesn't allow IdleReplicaCount to be equal to MinReplicaCount. Therefore, I cannot keep my service at the minimum resource.

Here is an example:
I cannot configure the ScaledObject like this:
# invalid configuration
idleReplicaCount: 1
minReplicaCount: 1
maxReplicaCount: 5

I have to set it as:
# valid configuration 1
idleReplicaCount: 0
minReplicaCount: 1
maxReplicaCount: 5

or
# valid configuration 2
idleReplicaCount: 1
minReplicaCount: 2
maxReplicaCount: 5

The first approach doesn't make the service always available
The second approach creates the redundant replica. It could be a big deal for the expensive GPU machine

Use-Case
No response
Anything else?
No response

Hi @zroubalik
Thank you for your reply. I'll close the issue


VRP validation of DebtorAccount is owned by the user (trusted beneficiary logic)

The VRP payment flow supports payments with SCA Exemption, in this flow the TPP needs to provide the Debtor Account from which the PSU will make the VRP payment.
See: https://standards.openbanking.org.uk/customer-experience-guidelines/payment-initiation-services/vrp-payments-with-an-sca-exemption/latest/
We need to validate that the Debtor Account sent in the VRP Consent is an account which belongs to the PSU. Currently, any account can be specified (with any name) and this data will be rendered in the consent ui.
As we only support the SCA Exemption flow, then we should reject consents which do not supply a Debtor Account. In the spec the Debtor Account is optional, as for SCA flows the account would be selected in the consent ui.
Acceptance criteria

If the the debtor account does not belong to the PSU reject the VRP payment request in a way that conforms with the OB Specifications and User Experience guidelines
Add functional tests to check correct errors are raised under these circumstances

Hey team! Please add your planning poker estimate with Zenhub @BohoCode @dbadham-fr @mariantiris

Fix covered by: https://github.com/secureapigateway/secureapigateway/issues/731


Drag gets priority over overlapping elements

Hi, thank you for an awesome package!
I have an issue that I thought was similar to #288 but it doesn't seem to be fixed with the latest version 2.0.7. I have an overlapping button element that I want to be able to click and collapse a drawer. However, the draggable border gets priority even though the Z-index on the button is above.
This can be partially solved by moving the button outside of the resizable components (although it makes the positioning of the button troublesome), but when the button contains only an icon and not text, the drag gets priority again. The button is although still clickable.
If the button contains text and is outside of the resizable components it works as expected, but the positioning of the button is again cumbersome.
It's difficult to explain, so I reproduced a simple example it in the sandbox below:
https://codesandbox.io/p/sandbox/react-resizable-panels-forked-s5jzpg

The behavior you're describing is controlled by this logic:
https://github.com/bvaughn/react-resizable-panels/blob/cd32d1c3d7c57bbb1f6430812aff7f0d75ee1d13/packages/react-resizable-panels/src/PanelResizeHandleRegistry.ts#L168-L204
This Replay shows what's happening: 8c5e2948-00d4-4dd0-a7f7-c11f660ee59f
Nearest I can tell, it looks like a bug in the stacking-order package, because it's returning a value indicating that your buttons are not "above" the drag-handle. I added a comment where I Think that library is broken, but I don't have time to dig into it more.
You could try reducing this test case even more and filing an issue in that project's GitHub? Or if you'd like to dig in yourself and submit a PR I'll try to find time to review it. I'm pretty swamped right now though with work and don't have time to dig into this more.

Thank you for the reply, I'll see what I kind find in that repo, in the meantime I'll try to use a workaround though.

I had a few minutes this morning, and tried reproducing this in an isolated project. For some reason, I was unable to reproduce the behavior. I wonder what it is about the Code Sandbox you attached that is different...

Ah, I think it's somehow being caused by the flex layout! That is promising.

I had a few minutes this morning so I made a smaller repro and filed https://github.com/Rich-Harris/stacking-order/issues/3

Wow thank you so much for taking the time to look at it! Very glad the problem could be located. Looking forward to their response and hopefully a fix since my workaround wasn't so good 😊.

No problem! Think I found a fix @ https://github.com/Rich-Harris/stacking-order/pull/4

Fixed published in 2.0.9

❤ → ☕ givebrian.coffee

Amazing, thanks!


Add set negative button text and positive button text

Allows user to customize negative button and positive button text of the dialog

@adrianhartanto004 Merged. I will publish the new version for you tonight.


feat: support AbortSignal in page.waitForRequest/Response/NetworkIdle/Frame

Issue #9709

Removed backticks from the title to avoid potential release-please issues


drone: 2.0.0 -> 2.0.3

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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.

Result of nixpkgs-review pr 130072 at 6f0f3940 run on aarch64-linux 1

2 packages built successfully:

drone
drone-oss


Time padding is incorrect

Expected behavior

It should look like this
Actual behavior

But it looks like this
Steps to reprodcue
So i looked in to it and their is no padding:0 in it so he takes the default ul.

I have the same issue.

A temporary fix is to add this CSS:
ul.react-datepicker__time-list {
    padding: 0;
}

It seems to be the ul.react-datepicker__time-list element, which inherits from the base 'user agent stylesheet' (in chrome at least) with this class:
ul, menu, dir {    padding-inline-start:40px; }
Guess we need to reset this to 0px, as mentioned above...


Periodic Cloud SQL IAM service account authentication failed errors

Question
Periodically, we see the following error:

server error (FATAL: Cloud SQL IAM service account authentication failed for user<EMAIL_ADDRESS>(SQLSTATE 28000))

This happens in background jobs that are in the process of being shut down. Do you have a suggestion on how to avoid this situation in GKE?
We've never experienced this specific issue when using passwords, so I presume there's some interplay between the app, its environment (GKE), and how IAM is handled by the pods when it receives a shutdown signal.
We're setting up the token sources as described in WithIAMAuthNTokenSources and then cloudsqlconn.WithPrivateIP() along with the instance's connection name.
Code
No response
Additional Details
cloud.google.com/go/cloudsqlconn v1.9.0

A couple of questions:

How long do your background jobs typically run for?
Why are your jobs connecting to the database during shutdown?

As for suggestions:

The WithIAMAuthNTokenSources is really meant as a convenience for people who want IAM AuthN in a docker container that isn't running in a Google-managed environment. Instead, I'd suggest using Workload Identity which will remove the need to set any auth options at all and have the Go Connector automatically pick up credentials from the metadata server running in GKE.
Probably not related, but FYI there is github.com/GoogleCloudPlatform/cloud-sql-proxy/issues/2212 which is for a similar issue.

When using WithIAMAUthNTokenSources, the Go Connector won't be able to refresh the token. So once it expires, I would expect you'd see errors like the one above. So it's best to use Workload Identity.

How long do your background jobs typically run for?

They are periodic jobs run every 10 min, for ~2-3 seconds.

Why are your jobs connecting to the database during shutdown?

 Even though we handle the context gracefully, there's always a chance a new job might start just before the server is signalled to shut down. We won't initiate any new jobs, but some may be on their "last run" and may require a database connection.

When using WithIAMAUthNTokenSources, the Go Connector won't be able to refresh the token. So once it expires, I would expect you'd see errors like the one above.

I think this is root of the problem, I presume when a pod is signalled to terminate the token cannot be refreshed?

If it just so happens that the token is expired (and a refresh is necessary), why would this fail?

The Go Connector uses a refresh ahead cache internally that every 56 minutes refreshes the client cert by performing two SQL Admin API calls. If the token that authorizes the client to make those calls expires, the calls fail and the client certificate (and OAuth2 token within the cert) eventually expires, causing the authentication errors.

I think this is root of the problem, I presume when a pod is signalled to terminate the token cannot be refreshed?

Correct. This is why we recommend using a static token source for one-off operations and otherwise recommend application default credentials with workload identity as the default. With Workload Identity, the Go Connector's underlying auth library can refresh the OAuth2 token for you automatically.

Fwiw this happens fairly infrequently. My goal is to understand why the token cannot be refreshed when using WithIAMAUthNTokenSources. Also, is the token cached for 1 hour?

Could you show me how you're configuring your token source? In some cases (like with a static token source), I've seen the auth library not refresh the token and eventually expire. I assume that's what's happening here, but need to see how you're configuring a token source to say for certain.

Alternatively, could this be an intermittent token refresh issue and less an issue with pod termination? If so, is there any verbose logging and/or debug we could enable to track this down?

You could enable debug logging -- that will report on the refresh operations and we'll be able to see for certain if the hypothesis about the token expiring is in fact true.

For debug logging, use cloudsqlconn.WithContextLogger.

I enabled debug logging, so hopefully this helps us track down specific errors for the token refresh.

I've seen the auth library not refresh the token and eventually expire. I assume that's what's happening here, but need to see how you're configuring a token source to say for certain.

Fwiw just today I noticed this happen in a pod that was not terminated and had an uptime of many hours (>4h). It appears the token refresh did succeed since traffic continued to be served by that pod (and app), so it's definitely intermittent but I'm a bit surprised.
Here's an example of what this looks like under the hood (most of this is adapted from the examples and tests):

options := []cloudsqlconn.Option{
	cloudsqlconn.WithIAMAuthN(),
	cloudsqlconn.WithContextDebugLogger(...),
}

sqlTS, err := impersonate.CredentialsTokenSource(
	ctx,
	impersonate.CredentialsConfig{
		TargetPrincipal: config.Impersonate,
		Scopes:          []string{"https://www.googleapis.com/auth/sqlservice.admin"},
	},
)
if err != nil {
	return nil, nil, err
}
loginTS, err := impersonate.CredentialsTokenSource(
	ctx,
	impersonate.CredentialsConfig{
		TargetPrincipal: config.Impersonate,
		Scopes:          []string{"https://www.googleapis.com/auth/sqlservice.login"},
	},
)
if err != nil {
	return nil, nil, err
}
options = append(options, cloudsqlconn.WithIAMAuthNTokenSources(sqlTS, loginTS))

dialer, err := cloudsqlconn.NewDialer(ctx, options...)
if err != nil {
	return nil, nil, err
}
pgconnDialFunc := func(ctx context.Context, network, address string) (net.Conn, error) {
	return dialer.Dial(ctx, config.CloudSQLInstance, cloudsqlconn.WithPrivateIP())
}

Your setup looks perfectly correct.

so it's definitely intermittent but I'm a bit surprised.

Same here. It can happen that the backend call to verify the IAM principal fails, but it should be uncommon.
Debug logging will at least confirm whether the background refresh is working as intended. If it is, then we'll need to get the backend team to look at this. If you have a support contract, you might open a case and reference this thread.

Minor update, it does appear to be working as intended. We'll continue to monitor this. But I think we should keep this issue open because I suspect this will happen again and hopefully we'll get more details on the underlying error.
Thank you for your support.
Connection info refresh operation scheduled at 2024-05-16T13:20:46Z (now + 56m0s)
Connection info refresh operation complete
Connection info refresh operation started
Connection info added to cache
Connection info refresh operation scheduled at 2024-05-16T13:20:25Z (now + 56m0s)
Connection info refresh operation complete
Connection info refresh operation started
Connection info added to cache

We got a few alerts last night, I couldn't find any errors though. That pod is still running and serving traffic, so my initial observation that this issue may be related to pod termination was incorrect.
It appears to be an intermittent issue that happens (granted rarely) across all our clusters.

server error (FATAL: Cloud SQL IAM service account authentication failed for user<EMAIL_ADDRESS>(SQLSTATE 28000))

Fail #1
2024-05-21T01:31:51.637573838Z [spanId: b6ada0833320d66f] Now = 2024-05-21T01:31:51Z, Current cert expiration = 2024-05-21T02:10:49Z
2024-05-21T01:31:51.646609058Z [spanId: b6ada0833320d66f] Dialing <ip>:<port>
2024-05-21T01:31:51.646604178Z [spanId: b6ada0833320d66f] Cert is valid = true
2024-05-21T01:31:51.646583547Z [spanId: b6ada0833320d66f] Now = 2024-05-21T01:31:51Z, Current cert expiration = 2024-05-21T02:10:49Z
2024-05-21T01:31:51.637610658Z [spanId: b6ada0833320d66f] Dialing <ip>:<port>
2024-05-21T01:31:51.637605138Z [spanId: b6ada0833320d66f] Cert is valid = true

No errors
2024-05-21T01:32:01.677066925Z [spanId: 1c895344a9319de7] Dialing <ip>:<port>
2024-05-21T01:32:01.677058585Z [spanId: 1c895344a9319de7] Cert is valid = true
2024-05-21T01:32:01.677033825Z [spanId: 1c895344a9319de7] Now = 2024-05-21T01:32:01Z, Current cert expiration = 2024-05-21T02:11:49Z
2024-05-21T01:32:01.668646365Z [spanId: 1c895344a9319de7] Dialing <ip>:<port>
2024-05-21T01:32:01.668640315Z [spanId: 1c895344a9319de7] Cert is valid = true
2024-05-21T01:32:01.668582635Z [spanId: 1c895344a9319de7] Now = 2024-05-21T01:32:01Z, Current cert expiration = 2024-05-21T02:11:49Z
2024-05-21T01:32:01.594015583Z [spanId: 7e32753d96ce643a] Dialing <ip>:<port>
2024-05-21T01:32:01.594009073Z [spanId: 7e32753d96ce643a] Cert is valid = true
2024-05-21T01:32:01.593984683Z [spanId: 7e32753d96ce643a] Now = 2024-05-21T01:32:01Z, Current cert expiration = 2024-05-21T02:11:49Z
2024-05-21T01:32:01.587584384Z [spanId: 7e32753d96ce643a] Dialing <ip>:<port>
2024-05-21T01:32:01.587578124Z [spanId: 7e32753d96ce643a] Cert is valid = true
2024-05-21T01:32:01.587530024Z [spanId: 7e32753d96ce643a] Now = 2024-05-21T01:32:01Z, Current cert expiration = 2024-05-21T02:11:49Z

Fail #2
2024-05-21T01:36:51.648763983Z [spanId: 08d132b6aea91544] Dialing <ip>:<port>
2024-05-21T01:36:51.648753223Z [spanId: 08d132b6aea91544] Cert is valid = true
2024-05-21T01:36:51.648736833Z [spanId: 08d132b6aea91544] Now = 2024-05-21T01:36:51Z, Current cert expiration = 2024-05-21T02:10:49Z
2024-05-21T01:36:51.648265153Z [spanId: 08d132b6aea91544] Dialing <ip>:<port>
2024-05-21T01:36:51.648259623Z [spanId: 08d132b6aea91544] Cert is valid = true
2024-05-21T01:36:51.648233583Z [spanId: 08d132b6aea91544] Now = 2024-05-21T01:36:51Z, Current cert expiration = 2024-05-21T02:10:49Z
2024-05-21T01:36:51.646296843Z [spanId: 08d132b6aea91544] Dialing <ip>:<port>
2024-05-21T01:36:51.646291933Z [spanId: 08d132b6aea91544] Cert is valid = true
2024-05-21T01:36:51.646270343Z [spanId: 08d132b6aea91544] Now = 2024-05-21T01:36:51Z, Current cert expiration = 2024-05-21T02:10:49Z
2024-05-21T01:36:51.637149304Z [spanId: 08d132b6aea91544] Dialing <ip>:<port>
2024-05-21T01:36:51.637144773Z [spanId: 08d132b6aea91544] Cert is valid = true
2024-05-21T01:36:51.637136533Z [spanId: 08d132b6aea91544] Now = 2024-05-21T01:36:51Z, Current cert expiration = 2024-05-21T02:10:49Z
2024-05-21T01:36:51.637077044Z [spanId: 08d132b6aea91544] Dialing <ip>:<port>
2024-05-21T01:36:51.637061024Z [spanId: 08d132b6aea91544] Cert is valid = true
2024-05-21T01:36:51.636883953Z [spanId: 08d132b6aea91544] Now = 2024-05-21T01:36:51Z, Current cert expiration = 2024-05-21T02:10:49Z
2024-05-21T01:36:51.636756704Z [spanId: 08d132b6aea91544] Dialing <ip>:<port>
2024-05-21T01:36:51.636751653Z [spanId: 08d132b6aea91544] Cert is valid = true
2024-05-21T01:36:51.636739744Z [spanId: 08d132b6aea91544] Now = 2024-05-21T01:36:51Z, Current cert expiration = 2024-05-21T02:10:49Z

Thanks @mfridman. This at least shows us the background refresh isn't doing anything wrong.
Based on past conversations with the backend team, I understand some small percentage of IAM authentication calls may fail. Do you have a ball part estimate for what you're seeing?

I'd categorize it as very low, but I don't have a ballpark figure.
Depending on the error, is it worth retrying the operation? Maybe a few retries (say 3) within a short duration (say 2s) would make it slightly more resilient against backend failures? Given the choice between an error and additional latency, I'd pick the former. Could also make this an opt-in so the caller gets to choose.

Right now the Go Connector doesn't understand the database protocol and just returns a connected socket to the driver.  If you're using pgx directly, I wonder if you could retry calls to pool.Acquire to hide these auth errors.

We were using pgx/v5 as the driver but using *sql.DB for connection pooling. We recently switched to pgxpool and maybe it's anecdotal, but haven't seen these issues. Or maybe there was a fix by your backend team :P.
Up to you whether you'd like to keep this issue open/closed @enocom. I don't think there's anything immediate here on either side outside of recording this as a data point and user experience.

Interesting. I don't know of any change, but this might have been a difference in how sql.DB initiates connections vs pgxpool? I'll close this but appreciate all the data. This will be a good reference if the issue pops up again.

For brevity, we ended up catching SQLSTATE 28000 (invalid_authorization_specification) errors and retried them with a backoff when attempting to begin a transaction.
Pseudo:
if pgErr := (*pgconn.PgError)(nil); errors.As(err, &pgErr) {
	if pgErr.Code == pgerrcode.InvalidAuthorizationSpecification{
		// This is an invalid authorization error. Retry.
	}
	// Handle all other errors.
}

Thanks for the info. It's too bad we can't hide that in the Connector here, but looks like a working solution.


make easier to integrate external portable software(ex. portable Git or clang) in order make vscode a fully portable modular IDE

I'm not sure what you're asking for, please be more specific


ClusterServiceVersion minKubeVersion is not enforced

Bug Report
What did you do?

Create a CSV and bundle image  with a single operator with spec.minKubeVersion: 1.20.0
Add the CSV to a catalog index image
Create a CatalogSource
Create a Subscription to the operator in the CatalogSource.

What did you expect to see?
I expected the operator to be excluded from the update graph based on the constraint, preventing the operator from being selectable as an install option.
What did you see instead? Under which circumstances?
The operator is installable and successfully installed regardless of the minKubeVersion being detected.
In the status section of the applied CSV, I do see the following:
  - group: operators.coreos.com
      kind: ClusterServiceVersion
      message: >-
        CSV version requirement not met: minKubeVersion (1.20.0) > server
        version (1.19.0+7070803)
      name: testoperatora.v1.0.0
      status: PresentNotSatisfied
      version: v1alpha1

Environment

operator-lifecycle-manager version:
OpenShift 4.6.9

Kubernetes version information:

oc version
Client Version: 4.6.8
Server Version: 4.6.9
Kubernetes Version: v1.19.0+7070803

Kubernetes cluster kind:
UPI

Possible Solution
None
Additional context
None

The operator is installable and successfully installed regardless of the minKubeVersion being detected.

The operator should be in the Pending state - do you see the CSV in the Succeeded state?
This is more of a known limitation of minKubeVersion (+ a set of other things we only detect once we have the CSV applied) and was part of the reason for the resolver revamp last year and the declarative config this year.  I would reframe this as a feature request ("olm shouldn't resolve operators that would fail to run, if that can be detected by metadata"), though it's a feature request that we're finally in a position to do something about.

If the CSV is in Pending state, then this is expected behavior. The minKubeVersion doesn't prevent the operator to be installed in the cluster. It only prevents the operator from reaching the Succeeded state and ready to serve.

Sorry, yes, the CSV Is indeed in the Pending state (I'm used to my test pod failing all the time and didn't look closely).
I'm concerned that as products move forward and start to leverage APIs as Kubernetes starts removing them, this will be one of those values that developers will want to leverage to avoid auto-updates from trying to install now-incompatible versions.
In this particular case the operator developer was using minKubeVersion to allow using the new v1 CRD, which isn't available on OCP 4.4.
Maybe the right way to go is to use spec.apiservicedefinitions.required instead?  Presumably this is used by the sat solver like spec.customeresourcedefinitionsrequired ...
I'll bring this up for discussion on the olm-dev call on 2/11/2021 to discuss the right way to handle cases where APIs go away.


Use istio as keyword for Makefile.

@gyliu513: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
fda2d63171e303edfd2adad66cb37f3fcaccb6fc
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@gyliu513: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
fda2d63171e303edfd2adad66cb37f3fcaccb6fc
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/cc @mandarjog @linsun
Comments for this?

@gyliu513: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
fda2d63171e303edfd2adad66cb37f3fcaccb6fc
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Does this work with geospatial queries?

Hey there,
Thank you for the awesome work here.
My application heavily depends on mongoose's geoNear() function to query for nearby places. Does your plugin mongoose-paginate have any support for that geospatial query function?
Thanks in advanced

Have you tried it?

Tried and it didn't work. I'm just paginating manually, this plugin isn't
doing much after all. Uses skip&limit just like I would do.
On Sep 29, 2016 10:05 AM, "niftylettuce"<EMAIL_ADDRESS>wrote:

Have you tried it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/edwardhotchkiss/mongoose-paginate/issues/90#issuecomment-250474733,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANJ8dOqOPUT2TiVzt5vCdpOra-jXSEUyks5qu8TwgaJpZM4JA0oC
.

@ihadeed check out this cool example for the Koa version of express-paginate I made https://github.com/koajs/ctx-paginate#example


Primary key issue loading from gazetteer.Place.json

Once modified to adhere to issue #8, loading json to geodjango database for gazetteer works in the following order:

gazetteer.AdminCounty.json
gazetteer.HistoricCounty.json
gazetteer.Country.json

But fails loading gazetteer.Place.json:
  File "/usr/local/lib/python3.11/site-packages/django/core/management/commands/loaddata.py", line 102, in handle
    self.loaddata(fixture_labels)
  File "/usr/local/lib/python3.11/site-packages/django/core/management/commands/loaddata.py", line 163, in loaddata
    self.load_label(fixture_label)
  File "/usr/local/lib/python3.11/site-packages/django/core/management/commands/loaddata.py", line 251, in load_label
    for obj in objects:
  File "/usr/local/lib/python3.11/site-packages/django/core/serializers/json.py", line 70, in Deserializer
    yield from PythonDeserializer(objects, **options)
  File "/usr/local/lib/python3.11/site-packages/django/core/serializers/python.py", line 163, in Deserializer
    raise base.DeserializationError.WithData(
django.core.serializers.base.DeserializationError: Problem installing fixture '/app/gazetteer/fixtures/gazetteer.Place.json': ['“” value must be an integer.']: (gazetteer.place:pk=1) field_value was ''

The fixture file does include pk fields though (first entry for example):
[{"pk": 1, "model": "gazetteer.place", "fields": {"wikidata_id": "Q1000115", "label": "Faversham", "latitude": 51.3177, "longitude": 0.8928, "geonames_ids": "['2649    622', '7297621']", "historic_county": 1.0, "admin_county": "", "country": 1.0}}]

I think this fixes it...? https://github.com/Living-with-machines/alto2txt2fixture/commit/b86be8734830f2b3fde67c8167e54b6f31a2fdb7

This is a dupe of #6.


Update design picture/file upload

Update the picture/file upload components to match this design. Remove "Save" and add "Attach" and keep the same functionality.

So basically rename the button and move it up? Can I do it?


something wrong with ssh key in docker install

can test with
python3 /sandbox/code/github/threefoldtech/jumpscaleX/install/install.py -3 --portrange=3 --name=test -d -y -c

cannot login over ssh after install

fixed