subsys: storage: flash_map - power management feature

flash_map module provide flash_area_open and flash_area_close API. Is it possibile that original (so MyNewt) intention for these API call was to provide some kind of power management regards memory devices.
This discusion started at convert FCB to use sectors PR
flash power management future:
@ccollins476ad wrote:
The changes look good to me. I just have one possible concern. I noticed the flash area gets opened once in fcb_init() (via a call to flash_area_open()), and never gets closed. For internal flash, this is not a problem, but I believe the expectation was that flash areas would be opened when accessed, and closed when the access is complete.
I guess there was a thought that keeping an external flash device open is problematic (perhaps for power usage reasons).
So I'm not sure if this is something that should be changed or not. Like I said, it should work fine with internal flash.
Maybe it is best to keep your changes as is. This seems like an issue that is best addressed when needed (if ever).
my answer to this:
so far calling flash_area_open each time only increasing the latency. flash_area_close dose nothing. Maybe it was an idea of power management basing on this, but so fare this add only unwanted latency. My proposition is to modify flash_map API instead: Add f. flash_area_get(int flash_area_id, flash_area **fap) - it will do what current flash_area_open' does so it will return proper flash_area pointer basing on flash_area_id. Change 'int flash_area_open(int flash_area_id, flash_area **fap)' to 'int flash_area_open(flash_area *fap)' and limit its functionality to "flash power activation features" - so it will be empty for now. We will keepflash_area_close` as is - but we coment that it is dedicated to functionality of "flash power deactivation future".
@mbolivar - I need your opinion on this, I'm not sure what idea was behind flash_area_open/close API i mynewt.
^^
@OpenSourceFoundries

cc @ccollins476ad @mbolivar

@nvlsianpu I have often wondered the same thing.
In the middle of last year, I inserted some debugging calls into mcuboot's loader.c file to track opening and closing, and it was full of bugs. There were a lot of times that things were getting opened but never closed, and the newt implementations for those functions were empty IIRC.
I fixed the mcuboot bugs as best I could (with few exceptions, we now have that boot_go() basically just opens everything in the beginning and closes it in the end), and then left it there.
Maybe the newt people can shed more light on this -- though as I've said before I don't know how much sense it makes to duplicate newt internal APIs in Zephyr. It seems like a "shortest path" approach, but I don't like it in general.

This has already been discussed here: https://github.com/zephyrproject-rtos/zephyr/pull/60709.
TL;DR:  open/close functions are supposed only to get ownership and access to given flash area only and special power management functions should be provided instead.
Subsystems open areas and never release them, which is intended, and this would make device work at elevated power level.


Docker: Preinstall git in released docker image

This is a fix for 4f7d5bc which installed git in the wrong image (head).
See #2615

Why not in both?

Is git really needed in the head image? If so, I'll happily add it.
BTW, what is head actually used for?

I don't even know what the release image is really used for, but I imagine similar things for both.

I'm trying to use the release image (built by Dockerfile.release and push to crystallang/crystal:0.x.x) on our GitLab CI. I think that's a common use-case. Here, git is useful to use shards.
I suppose that the head images (built by Dockerfile) is used to build crystal itself??
If so, you don't need git there.
Just guessing... 8-)

The official release packages are built from crystal-lang/omnibus-crystal. Travis CI support uses the debian packages built by omnibus, as well as the nightly packages built by my fork of the omnibus project. Crystals own Travis CI setup uses the images from jhass/crystal-build-docker.

@jhass Thank you for the explanations!
Last question, how are images on docker.io built?
https://hub.docker.com/r/crystallang/crystal/~/dockerfile/ states a Dockerfile which includes llvm.
 Dockerfile.release build release images 0.x.x.
I hope I'm right %-)

@splattael That's exactly right. I think head is built automatically. We manually use Dockerfile.release to create images after each release. But I don't know much about all this, I just run some scripts @waj gave me 😊 (he used to do that, but since I do the part related to omnibus it's better if I do it too so we don't forget to do it). I guess it's OK for git to be in both images because you'll likely want to use shards.

@jhass @asterite Ok, got it!
I've added git to Dockerfile.release, so git is now available in head and 0.x.x docker images :dancer:
PR is rebased.

Thanks!

:green_heart:


fix compile errors w/improved flow typing

Darnit @lucaswerkmeister !!!

I like how we had the exact same fix :D

haha


Trying to setup constraints and indexes

Hey there I'm trying to create a new project on Django and to connect it to Neo4J.
I'm a kind of nooby in Neo4J, had some expirience with Django :)
I created two types of nodes Person and Country and tried to add the constraints or indexes need to be added to Neo4j
I'm not sure that I ran the correct commend
neomodel_install_labels neomodel_init.py someapp.models --db bolt://neo4j:
Traceback (most recent call last):
File "/home/dan/Python_Environments/django_neomodel/bin/neomodel_install_labels", line 67, in
main()
File "/home/dan/Python_Environments/django_neomodel/bin/neomodel_install_labels", line 58, in main
load_python_module_or_file(app)
File "/home/dan/Python_Environments/django_neomodel/bin/neomodel_install_labels", line 30, in load_python_module_or_file
import_module(module_name, package=pkg)
File "/home/dan/Python_Environments/django_neomodel/lib/python3.5/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 986, in _gcd_import
File "", line 969, in _find_and_load
File "", line 956, in _find_and_load_unlocked
ImportError: No module named 'neomodel_init'

Have you tried the following command?
python manage.py install_labels


Add editor config folders to gitignore

Editors generally create config files when in projects, .idea/, .vscode/ for example. It wouId be great to have the gitignore cover as many of the popular editors as possible so these files dont get into PRs and repos. So VsCode, Pycharm, Sublime, etc. I was going to do this myself but thought it could be a good first issue for someone.

I would suggest against it
https://blog.martinhujer.cz/dont-put-idea-vscode-directories-to-projects-gitignore/

This is a valid point, and while I do in fact have a global .gitignore set up, not everyone does, and having these files ignored on the project level is much more convenient for new contributors who do not have have a global .gitignore

I can take this. I wasn't aware of global .gitignores so I can do this and help my own repos by cleaning them up a bit.

hi, can I work on this as part of Hacktoberfest?

This is a valid point, and while I do in fact have a global .gitignore set up, not everyone does, and having these files ignored on the project level is much more convenient for new contributors who do not have have a global .gitignore

This is not the best option for me, as you read. I also think new contributors (to external repositories I mean) are often aware of what must be ignored and not pushed, and eventually I'd prefer fixing the PR than keeping this redundant entries in all the .gitignore

Hey @georgemunyoro,
I would like to work on this issue.
Please assign it to me.


Invalid flow specified error while adding integration

Hello,
I've successfully build the ESP32 device and added all shades and groups. Now I wanted to add the Home Assistant integration from HACS. I've added the custom repository and installed the integration. Then I restartet HA and it successfully found the device in the network. But when I try to add it (using username and password) it fails, showing the error: "Invalid flow specified".
Restarted multiple times, no success.
ESPSomfy RTS firmware: V 2.3.0
HA version: 2023.10.5
Running on Docker container at Synology NAS
Any idea, where this is coming from?

Try changing your language to English for the configuration if it is not.  Also try to omit the username and password.  The api key generated out of HA may override it.  If it doesn't then select it and edit the username and password under configure after the hub is added.

Try changing your language to English for the configuration if it is not. Also try to omit the username and password. The api key generated out of HA may override it. If it doesn't then select it and edit the username and password under configure after the hub is added.

I've changed the language to English but that was not the solution.
Then I removed the security options in ESPSomfyRTS. After restarting HA again, I was able to add the integration (without Username and PW). Now I activated the security option again in ESPSomfyRTS. And without changing the configuration of the integration I'm still able to control my shades. So it seems there is something like an API key integrated into the integration in HA.
Would it make sense to add this as a hint to the readme of the installation steps?

The plugin now properly detects invalid login credentials and does fall into the Invalid flow anymore.  ESPSomfy RTS will also ignore credentials if they have not been set in firmware.


kolor和flux什么时候能用啊，期待

@zyhwyl this is not managed repo

We will release v1.1 model for SDXL by the end of September. Additionally, we are currently training PuLID for FLUX, which will be released once it is ready. We appreciate everyone's patience

PuLID for FLUX sounding amazing

We will release v1.1 model for SDXL by the end of September. Additionally, we are currently training PuLID for FLUX, which will be released once it is ready. We appreciate everyone's patience

Thx for your great works... It is just tedious to train a LoRA...

We will release v1.1 model for SDXL by the end of September. Additionally, we are currently training PuLID for FLUX, which will be released once it is ready. We appreciate everyone's patience

Please please I've been waiting all this time! :pray: :smiley:
Rooting for you!

PuLID for FLUX is released now, please enjoy and have fun with it!

PuLID for FLUX is released now, please enjoy and have fun with it!

😍😍😍😍😍


Add search box to web fundamentals

Because we can't add JS to the site, I took a look at what is need to get search on our site.  It looks like the following simple HTML will get us 90% of the way to what we need.
<form class="devsite-search-form" action="/s/results" method="GET" id="top-search">

<input placeholder="Search" type="text" class="devsite-search-field devsite-search-query" name="q" value="" autocomplete="off">
<input type="hidden" name="p" id="search_project" value="/android" data-project-name="Google APIs for Web" data-project-path="/web" data-query-match="">

</form>

cc @gauntface @petele - I see this as a P0, I get asked about it all the time and it is hard to navigate the site without it (even in the new layout)

Should be done in https://github.com/google/WebFundamentals/commit/29ccf2610868330c80e1d61c4a3011ef987b82e1
Going to close this and add an item to the checklist to test before launch.


Fix actor context

Description
When using an asynchronous method in the Receive method, exceptions that are thrown are ignored.
Purpose
This pull request is a:

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[X] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

I see that this IsCompleted property is used as well in Futures.cs and DefaultClusterContext.cs, maybe it's worth checking.

Good catch!


Gitea Mastodon Account

Hi all,
As Gitea gains more traction the desire for a representative figure begins to become necessary. I was wondering if there were any plans to make a Mastodon account for either posting about updates and/or providing quick support for those who don't need to make a full-on issue here.
I figure it would be a good idea considering it would help lighten the load on the issues queue and would allow more pressing issues to get the attention they need.
I'd be willing to manage such an account but I think there are others who would be much more qualified to be such person. I'm looking forward to getting some feedback and also some suggestions on what exactly could/should be done with a Mastodon account.
I also propose making the account on Fosstodon.org as it's an open-source based instance. For full disclosure, it's important that I mention that I do have an account on Fosstodon.org and I really like the instance.

I think support is spread thin as it is and having a separate site/account to interact with issue reports wouldn't really help lighten the load at the moment but makes it more work (there are already many support questions in the forums that go unanswered because it is in a separate location, discord, etc...)
We could revisit if there is ever maintainer energy/support for this, but it seems that is not the case at the moment. Closing in the same spirit of lightening the issue queue to make pressing issues more clear.

Understood :) Thanks for the explanation


Destination Folder: Network

Hi,
is it possible to change your directory folder to a network folder?
I try it, but it not work. Do you have any idea?

my solution for the network root directory is, that you have to allow the ISS-User to read on your network folder. In the Webconfig you have to write in the rootdirectory exemplary: \Server\rootfolder


ArrayList return procedures don't work

I get the same error in both versions:
error: incompatible types: List  cannot be converted to ArrayList
To replicate: simply make an ArrayList in a procedure and have it return itself.

the issue can be fixed by doing this:


PlatformException(InputImageConverterError, java.lang.IllegalArgumentException, null, null)

Describe your issue. If applicable, add screenshots to help explain your problem.
The error occurs whenever I call the processImage(InputImage inputImage) function.
The last time it worked was May 27, 2024 with these plugin versions:
camera: ^0.11.0
google_mlkit_text_recognition: ^0.13.0
After I updated to these versions:
camera: ^0.11.0+2
google_mlkit_text_recognition: ^0.14.0
Then I tried to revert back to previous versions and the problem still persists.
Steps to reproduce.
The error occurs whenever I call the processImage(InputImage inputImage) function.
What is the expected result?
processImage(InputImage inputImage) function should work
Did you try our example app?
No
Is it reproducible in the example app?
No
Reproducible in which OS?
Android
Flutter/Dart Version?
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.4, on Microsoft Windows [versÆo 10.0.22631.4317], locale pt-BR)
[✓] Windows Version (Installed version of Windows is version 10 or higher)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Chrome - develop for the web
[✓] Visual Studio - develop Windows apps (Ferramentas de Build do Visual Studio 2019 16.11.29)
[✓] Android Studio (version 2024.1)
[✓] VS Code (version 1.94.2)
[✓] Connected device (4 available)
[✓] Network resources
Plugin Version?
camera: ^0.11.0+2
google_mlkit_text_recognition: ^0.14.0

Could be related to #684

Same here, after updating to
camera: ^0.11.0+2
google_mlkit_text_recognition: ^0.14.0
Same dependencies. No issue running on Apple devices.

Related to #700 . You can use my solution as temporarily.

Any update?

Related to #700 . You can use my solution as temporarily.

Your solution did not work for me :(

Bump, I have the same problem. Rolled back for now.

`InputImage? _inputImageFromCameraImage(CameraImage cameraImage) {
if (_controller == null) return null;
final camera = _cameras[_cameraIndex];
final sensorOrientation = camera.sensorOrientation;
InputImageRotation? rotation;
if (Platform.isIOS) {
rotation = InputImageRotationValue.fromRawValue(sensorOrientation);
} else if (Platform.isAndroid) {
var rotationCompensation =
_orientations[_controller!.value.deviceOrientation];
if (rotationCompensation == null) return null;
if (camera.lensDirection == CameraLensDirection.front) {
rotationCompensation = (sensorOrientation + rotationCompensation) % 360;
} else {
rotationCompensation =
(sensorOrientation - rotationCompensation + 360) % 360;
}
rotation = InputImageRotationValue.fromRawValue(rotationCompensation);
}
if (rotation == null) return null;
const format = InputImageFormat.nv21;
final plane = cameraImage.planes.first;
final WriteBuffer allBytes = WriteBuffer();
for (final Plane plane in cameraImage.planes) {
  allBytes.putUint8List(plane.bytes);
}

return InputImage.fromBytes(
  bytes: allBytes.done().buffer.asUint8List(),
  metadata: InputImageMetadata(
    size: Size(cameraImage.width.toDouble(), cameraImage.height.toDouble()),
    rotation: rotation, // used only in Android
    format: format, // used only in iOS
    bytesPerRow: plane.bytesPerRow, // used only in iOS
  ),
);

}`

`const format = InputImageFormat.nv21;
final plane = cameraImage.planes.first;
final WriteBuffer allBytes = WriteBuffer();
for (final Plane plane in cameraImage.planes) {
  allBytes.putUint8List(plane.bytes);
}

return InputImage.fromBytes(
  bytes: allBytes.done().buffer.asUint8List(),
  metadata: InputImageMetadata(
    size: Size(cameraImage.width.toDouble(), cameraImage.height.toDouble()),
    rotation: rotation, // used only in Android
    format: format, // used only in iOS
    bytesPerRow: plane.bytesPerRow, // used only in iOS
  ),
);`

Try this. Most important thing is
final WriteBuffer allBytes = WriteBuffer(); for (final Plane plane in cameraImage.planes) {   allBytes.putUint8List(plane.bytes); }


[Grid]  not  support Spread syntax

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

Expected Behavior 🤔

Steps to Reproduce 🕹

import * as React from "react";
import Grid from "@material-ui/core/Grid";

const response = {
  xs:12,
  sm:6,
  md:4,
  lg:3,
  xl:3,
};

export default function CSSGrid() {
  return (
    <Grid container spacing={3}>
      <Grid item {...response}>

1

      </Grid>
    </Grid>
  );
}

https://codesandbox.io/s/cssgrid-material-demo-forked-bedf8?file=/demo.tsx
Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
5.0.0-alpha.14

React
16.13.1

Browser
chrome lastest

TypeScript
4.0.2

etc.

The following solve this issue but I'm not sure if there is a more pragmatic approach:
import * as React from "react";
import Grid from "@material-ui/core/Grid";

const response = {
- xs: 12,
+ xs: 12 as 12,
};

export default function CSSGrid() {
  return (
    <Grid container spacing={3}>
      <Grid item {...response}>

1

      </Grid>
    </Grid>
  );
}

const response = {
xs: 12,
sm: 2
} as const

@oliviertassinari @xiaoyu-tamu
thank you very much

const response = { xs: 12, sm: 2 } as const
works


KBFS client wasn't found

I'm seeing the following when I launch Keybase:
Error: KBFS client wasn't found at t.Error (native) at new t (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/index.bundle.js:11:5254) at a (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/index.bundle.js:11:4885) at o (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/index.bundle.js:11:4810) at file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/index.bundle.js:5:23565 at file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/index.bundle.js:53:23817 at e.t.Deferrals.e._call (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/index.bundle.js:35:1893) at file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/index.bundle.js:35:2044 at c (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/index.bundle.js:35:1671) at e.t.Deferrals.e._fulfill (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/index.bundle.js:35:2003)
I've performed an uninstall and reinstallation and continue to see it. I submitted a log earlier, ID: ef631dd85228e292506ce71c
Thanks

Same issue. My log ID:<PHONE_NUMBER>f4cfe59297fd1c

Any fix to this?

@murshid1988 should all be fixed, can you send a keybase log send?

@maxtaco I just sent.

b4dabd54d1c5bc2757e2e71c

#7385
I guess if you run upd.exe check it should be fixed now.


fixes and refactoring

Add metrics for initial deployment
Add ObservedGeneration when checking if PVs have been restored
Add PVUploader for unit test (this has not been tested)
Add the Required field "Reason" to the setVRGInitialCondition
Refactor PV Restore and add annotation to help determine the PVs restored by Ramen as opposed to PVs that are not.

Closing this PR as it is already included in PR #220


Add a 2-pages example

Hey, I'm using your template and would find this to be very cool. When I tried introducing a \newpage and then adding another \cvcontent, the compilation complains with \endcsname and \rc@extract@page errors, but the page appeared and looks correct.
Any ideas what I'm doing wrong?

Hello, thank you for your feedback! Glad other people are using this package!
I will try to reproduce and fix your issue.

I haven't been able to reproduce your issue. I've added a second page (a74ea58) to cv.tex.
Can you provide me an example of your code?

That's exactly what I did. Thanks to your hint I realized the problem had nothing to do with adding new pages.
It has to do with using \cvlink within a \cvitem and presumably their friends instead.

Could you open a new issue? I'm closing this one.


Showing POC of using uplink to access acapy

So uplink is a clever library for building web api clients...
It is inspired by a java project called retrofit
Included in this pull request is a demonstration of how to use uplink to access acapy.
The key thing to note is that there is a vast amount of code reuse, so now only is the less code to write, there is also less code to go wrong.

Closing this so it doesn't stay int he list of open PR's... won't delete the the branch


Add write permissions to workflows

Github changed the default permissions for github actions, we need to specify what permissions we need in the workflow. I'm doing this for all repos, however, I realized I already committed it, so I'm closing the PRs.


Invalid .NET version

Describe the bug
.NET framework version value in "az functionapp config set" generates warning/error
Related command
az functionapp config set
Errors
Invalid version: for runtime dotnet-isolated and os windows. Supported versions for runtime dotnet-isolated and os windows are: ['.4.8', '8', '7', '6']. Run 'az functionapp list-runtimes' for more details on supported runtimes.
Issue script & Debug output
az functionapp config set name "appname" --resource-group "groupname" --slot "slot" --net-framework-version v7.0
Expected behavior
It should not produce any error because I've tried using "7.0", "7", "v7" and "v7.0" and they all produce the same message.
Environment Summary
azure-cli                         2.57.0
core                              2.57.0
telemetry                          1.1.0
Extensions:
ai-examples                        0.2.5
ml                                2.22.0
ssh                                2.0.2
Dependencies:
msal                              1.26.0
azure-mgmt-resource             23.1.0b2
Additional context
This warning occurs when we run our ADO pipeline, thus causing the pipeline to fail.  This was a new issue since nothing is changed on our pipeline since the last successful run.

Thank you for opening this issue, we will look into it.

@yonzhan  anything new here?


🐛 [Cloud-Firestore] Loading bundle failed with Invalid argument on Flutter Web

Bug report
When trying to load a bundle into Firestore Cache an error occurs.
Looking in the console the below is shown:
Uncaught (in promise) unimplemented
@firebase/firestore: Firestore (8.10.0): SyncEngine Loading bundle failed with Invalid argument: 0
Uncaught (in promise) Invalid argument: 0
    at Object.b (https://contracts-plus.web.app/main.dart.js:3873:3)
    at Object.v (https://contracts-plus.web.app/main.dart.js:3880:14)
    at Object.BO (https://contracts-plus.web.app/main.dart.js:3724:25)
    at Object.fm (https://contracts-plus.web.app/main.dart.js:6031:17)
    at abi.$1 (https://contracts-plus.web.app/main.dart.js:44599:57)
    at Object.aZ4 (https://contracts-plus.web.app/main.dart.js:3828:69)
    at Object.aHl (https://contracts-plus.web.app/main.dart.js:6028:19)
    at b1a (https://contracts-plus.web.app/main.dart.js:7412:19)
    at Object.next (https://contracts-plus.web.app/main.dart.js:7408:42)
    at Ap._failWith (https://www.gstatic.com/firebasejs/8.10.0/firebase-firestore.js:1:279244)

On the Chrome Simulator via VSCode, it works fine. Only when deployed to Firebase Hosting the error occurs as above.
Steps to reproduce
I am following the instructions from here:
https://firebase.flutter.dev/docs/firestore/usage/#data-bundles
// Use a package like 'http' to retrieve bundle.
import 'package:http/http.dart' as http;
final response = await http.get(url);

// Convert the 'bundle.txt' string in the response to an Uint8List instance.
Uint8List buffer = Uint8List.fromList(response.body.codeUnits);

// Load bundle into cache.
LoadBundleTask task = FirebaseFirestore.instance.loadBundle(buffer);

// Use .stream API to expose a stream which listens for LoadBundleTaskSnapshot events.
task.stream.listen((taskStateProgress) {
  if(taskStateProgress.taskState == LoadBundleTaskState.success){
    //bundle is loaded into app cache!
  }
});

// If you do not wish to .listen() to the stream, but simply want to know when the bundle has been loaded. Use .last API:
await task.stream.last;

// Once bundle is loaded into cache, you may query for data by using the GetOptions() to specify data retrieval from cache.
QuerySnapshot<Map<String, Object?>> snapshot = await FirebaseFirestore.instance
  .collection('cached-data')
  .get(const GetOptions(source: Source.cache));

I also have to mention that LoadBundleTaskState is marked by VSCode as an undefined name. So there seems to be anyways something missing in the implementation.

Expected behavior
I would expect the bundle to be loaded and served via Firestore Cache which is working fine in the Chrome Emulator.
Sample project
The failing version is uploaded here where the console output can be checked.
https://contracts-plus.web.app/#/
Here is the sample I use via Flutter in VSCode:
  // Loading Bundle
  loadBundle() async {
    // Use a package like 'http' to retrieve bundle.
    Uri url = Uri.parse("https://firebasestorage.googleapis.com/v0/b/contracts-plus.appspot.com/o/exports%2Fbundle.txt?alt=media&token=28c306d9-98c5-44d6-8e3f-0a50b286f398");
    final response = await http.get(url);

    // Convert the 'bundle.txt' string in the response to an Uint8List instance.
    Uint8List buffer = Uint8List.fromList(response.body.codeUnits);

    // Load bundle into cache.
    LoadBundleTask task = FirebaseFirestore.instance.loadBundle(buffer);

    // Use .stream API to expose a stream which listens for LoadBundleTaskSnapshot events.
    task.stream.listen((taskStateProgress) {
      //if (taskStateProgress.taskState == LoadBundleTaskState.success) {
        //bundle is loaded into app cache!
        //LoadBundleTaskState is not implemented!!!!
      //}
    });

    // If you do not wish to .listen() to the stream, but simply want to know when the bundle has been loaded. Use .last API:
    await task.stream.last;

    await FirebaseFirestore.instance
        .collection('languages')
        .doc('en')
        .get(const GetOptions(source: Source.cache))
        .then((snap) {
          Map dic = snap?.data();
          print(dic);
    });
  }

Additional context
As already mentioned it works fine in the Emulator but fails when uploaded to a webserver like Firebase Hosting.
Below is another example, that works fine on the Emulator but fails when uploaded to a webserver like Firebase Hosting.
Further, I am using the fact that FlutterFire now supports initialization directly from Dart which I have used.
Here I am using Firebase Storage to fetch the data bundle.
      var buffer = await _storage.ref('exports/bundle.txt').getData();
      // Load bundle into cache.
      LoadBundleTask task = FirebaseFirestore.instance.loadBundle(buffer);
      // Wait until all info has been loaded into cache!
      await task.stream.last;

    await FirebaseFirestore.instance
        .collection('languages')
        .doc('en')
        .get(const GetOptions(source: Source.cache))
        .then((snap) {
          Map dic = snap?.data();
          print(dic);
    });

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[✓] Flutter (Channel stable, 2.8.0, on macOS 12.0.1 21A559 darwin-arm, locale
    de)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.63.2)
[✓] Connected device (1 available)

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
firebase_core: ^1.10.6
cloud_firestore: ^3.1.5
firebase_storage: ^10.2.4

at Ap._failWith (https://www.gstatic.com/firebasejs/8.10.0/firebase-firestore.js:1:279244)

Seems you are using 8.10.0 version of the firebasejs. I looked at the manual installation part of the document and it shows the version as 8.6.0, may be you can try with that version and see if it helps ? https://firebase.flutter.dev/docs/manual-installation/web#add-firebase-sdks

at Ap._failWith (https://www.gstatic.com/firebasejs/8.10.0/firebase-firestore.js:1:279244)

Seems you are using 8.10.0 version of the firebasejs. I looked at the manual installation part of the document and it shows the version as 8.6.0, may be you can try with that version and see if it helps ? https://firebase.flutter.dev/docs/manual-installation/web#add-firebase-sdks

I have also tested the manual installation with various versions including 8.6.0 with the same result.

Since this works on Android but fails while loading on web server, it may have to do with the firebase js version. Keeping it open for further investigation and labeling.
/cc @Salakar

Since this works on Android but fails while loading on web server, it may have to do with the firebase js version. Keeping it open for further investigation and labeling.
/cc @Salakar

Most weird thing is that running it locally in the emulator it works as expected with 8.10.0 and 8.6.0. I agree that it may something wrong with the firebase js version.

Is this still an issue, @halloway? We're now on Firebase web JS SDK version 9.11.0.

Is this still an issue, @halloway? We're now on Firebase web JS SDK version 9.11.0.

I need to test it on the latest version.

Having the same issue.


SearchAnchor should dispose created FocusNode and SearchController.

Description

Fixes https://github.com/flutter/flutter/issues/136117;
Adds isAttached property to the SearchController.

Tests

Updates material/search_anchor_test.dart to use testWidgetsWithLeakTracking;
Adds test which checks that FocusNode of SearchBar can be swapped;
Adds test which checks that SearchController of SearchAnchor can be swapped.

Pre-launch Checklist

[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[X] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[X] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[X] I signed the [CLA].
[X] I listed at least one issue that this PR fixes in the description above.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is [test-exempt].
[X] All existing and new tests are passing.

cc @polina-c

@Piinks , I do not have enough context. Would you mind to review?

@polina-c @Piinks @QuncCccccc @Renzo-Olivares, thanks a lot for the review!


Failed to import linked account Fanatical games details

After buying some games recently the extension started throwing an exception during import.
12-04 10:33:40.179|INFO |FanaticalLibrary#FanaticalAccountClient:Current Playnite.SDK, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null does not support Javascript Event Messaging. Using old way unoptimized strategy
12-04 10:33:40.779|INFO |FanaticalLibrary#FanaticalAccountClient:Current authentication token is valid.
12-04 10:33:41.647|ERROR|FanaticalLibrary#FanaticalLibrary:Failed to import linked account Fanatical games details.
Newtonsoft.Json.JsonReaderException: Unexpected character encountered while parsing value: {. Path '[0].order.gift_order', line 1, position 973.
   at Newtonsoft.Json.JsonTextReader.ReadStringValue(ReadType readType)
   at Newtonsoft.Json.JsonTextReader.ReadAsString()
   at Newtonsoft.Json.JsonReader.ReadForType(JsonContract contract, Boolean hasConverter)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateDictionary(IDictionary dictionary, JsonReader reader, JsonDictionaryContract contract, JsonProperty containerProperty, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetPropertyValue(JsonProperty property, JsonConverter propertyConverter, JsonContainerContract containerContract, JsonProperty containerProperty, JsonReader reader, Object target)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateList(IList list, JsonReader reader, JsonArrayContract contract, JsonProperty containerProperty, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateList(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, Object existingValue, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
   at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
   at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value, JsonSerializerSettings settings)
   at Playnite.Common.Serialization.FromJson[T](String json) in C:\projects\playnite\source\Playnite\Common\Serialization.cs:line 336
   at Playnite.Common.DataSerializer.FromJson[T](String json) in C:\projects\playnite\source\Playnite\Common\Serialization.cs:line 80
   at Playnite.SDK.Data.Serialization.FromJson[T](String json) in C:\projects\playnite\source\PlayniteSDK\Data\DataSerialization.cs:line 406
   at FanaticalLibrary.Services.FanaticalAccountClient.GetLibraryItems() in D:\Develop\GitProjects\playnite-fanatical-plugin\source\Services\FanaticalAccountClient.cs:line 475
   at FanaticalLibrary.FanaticalLibrary.GetLibraryItems() in D:\Develop\GitProjects\playnite-fanatical-plugin\source\FanaticalLibrary.cs:line 44
   at FanaticalLibrary.FanaticalLibrary.GetGames(LibraryGetGamesArgs args) in D:\Develop\GitProjects\playnite-fanatical-plugin\source\FanaticalLibrary.cs:line 61

Will try to get the actual JSON if I can at a later time.

I have the exact same problem., with the same Exception in the Log File.

Hi,
It seems some of the games you purchased have some unexpected value in the "gift_order" attribute that broke the JSON syntax expected from the Newtonsoft JSON library.
To have a better check we need the JSON as you suggest (and possibly some detail about the game that perhaps broke the syntax).
You can get the JSON a bit more easily using GOG Galaxy with the Playnite library connector: there are more debugging options in that, and if I remember correctly you can also extract the JSON (hoping it's extracted "before the parsing" if it's a parsing problem).

Do you know how I can get the JSON in Playnite. If I know, where I can get it, I could provide it... Which game(s) make the problem, I don't know... I have quite a lot Keys in my Fanatical account.

Sadly I confirm that the GOG plugin's first login was broken by some of the last Fanatical site releases... let me check and possibly solve that problem first: I do not remember any simple OOTB way to have the JSON in the Playnite plugin. Hopefully, I'll come back to this thread when solved the other one!

Ok, it seems I have to play a bit with the deserializer, will use this JSON (that seems to be perfectly valid) as an example...

Released a new, hopefully fixed, version, let me know if it's ok!

Released a new, hopefully fixed, version, let me know if it's ok!

It's working now! No Errors and Keys are imported. Thank you :-)

One thing: Is it possible to automatically delete the Fanatical entries in Playnite after I redeemed the keys? I only want to have Fanatical game entries in my library, if I don't have the key redeemed. At the moment, after the key was imported, it remains in my library - if I don't want that, I have to delete it manually.
It would be great, if that would be possible (maybe optinal).

No more errors on my side either. Great job!

@Rondrian82, the Plugung Library lacks support for this. I asked Josef Nemes himself, owner of the Pynite project, if can be added, getting a formal no-go ... see https://github.com/JosefNemec/Playnite/issues/2677.
You can try adding a comment to that thread if you want to support the request.
I will close this thread asap, being the issue solved.


Opencart <IP_ADDRESS>

After installing when I go to the edit button, where I will be entering the bitpay details I get the following error :
Fatal error: Call to undefined method Loader::library() in /home/mydomain.com/public_html/admin/controller/payment/bitpay.php on line 25
    $this->load->library('bitpay');

I think this is associated with #43

Fixed in release v2.2


Change the framework switches to the new syntax

This PR updates the syntax of the framework-specific code samples. With this new syntax, you'll be able to:

have paragraphs of text be framework-specific instead of just code samples
have support for Flax code samples if you want.

This should be merged after https://github.com/huggingface/doc-builder/pull/63 and https://github.com/huggingface/doc-builder/pull/130

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


:sparkles: Add container exec possibility

Fix #8

There is still a problem with the screen refresh when coming back to the container list.


The code is Flickering

Hello V S Code
There is a bug in the application.
he code is flickering, not able to code.

/duplicate https://github.com/microsoft/vscode/issues/99346


Crash when opening large file

I tried to open minified and bundled .js file, but it is crashed. File size is 2.4MB.
Here is the error

@elgunhuseynli Can you provide the bundled.js for reproducing? Thanks.

@yuezk And here is the zip file.
jsbundle.zip

@elgunhuseynli Thanks for the test objects, but I couldn't reproduce this issue with the jsbundle you provided. Does this issue happen on the source code?

@yuezk Thanks for testing. No this issue doesn't happen on source.
I can still reproduce the issue. I deleted .vimrc file and all other plugins except vim-jsx-pretty and error still occur.
I thought maybe this issue occur on my Windows 10, so I tried to reproduce on linux environment and yeap it doesn't give error on linux. Also, are you using linux?

@elgunhuseynli  I'm using macOS and it works well. It's a bit of hard for me to debug it with the bundled js on Windows 10. Since it works well with the source code, I am no longer spend time on this.
Thanks for your feedback again, and if it happens on the source code later, feel free to let me know.


Add the Ability to Reflect Doc Strings

What problem does this solve or what need does it fill?
When using Reflect to power an automatically generated scripting API, it would be useful to have access to the type and field documentation strings, in addition to the type information.
That would allow auto-generated, typed bindings such as TypeScript bindings to include useful documentation on the meaning of the fields and types.
What solution would you like?
Concretely, I just want the ability to get the docstring related to types and fields. For instance, with this struct:
/// My struct is awesome
struct MyStruct {
    /// This is somebody's name
    name: String,
}

If I have a Box<dyn Reflect> containing this struct, then I should be able to get the doc string "My struct is awesome", possibly by calling a new method on the Reflect trait, such as:
fn doc_string(&self) -> Option<&str>;

Additionally, I should be able to get the documentation for the name field, for instance, if I use reflect_ref() to find out that it is a Struct. We could probably add a method to get the docstring of fields in a struct:
fn doc_string_at(&self, index: usize);

What alternative(s) have you considered?
Instead of adding more methods to Reflect and Struct traits, maybe we could add a ReflectDocs type that could be gotten as associated type data in a TypeRegistry, but I'm not sure if that would be as ergonomic, and might not work as well for getting the doc strings for specific fields.
Additional context
None yet.

This could add to the size of the objects and code gen significantly, and won't be useful for many use cases. We should definitely make this an extra, optional trait.

If it was optional, then that would make it very easy to leave out the extra derive, and leave a lot of docs out in many Bevy plugins, etc. I've already noticed that a little bit in the case of reflecting Default, which is easy to forget to do, even when types already implement default, and then scripts are left unable to create the type.
It's understandable and fine if that ends up the case, but it'd be nice if people didn't have to manually opt-in.
I wonder if it could be a compile time feature flag or something like that. So that the derive macro would automatically include the docs for for everything, without the user having to opt-in, but people who want to compile it out because it's not used could easily do so.
Maybe could even be disabled by default, because even in a scripting scenario, we only really need it for type generation, unless we're building an editor.  I'm not a big fan in general of adding cargo features for things that don't really need it, but this might be a scenario where it makes sense, for something so often un-needed.

As far as object size, though, does adding another method increase the size of the trait object much? The string will be static for any Rust type, and the trait object just needs a pointer to the method that returns the static string, so it shouldn't be copied around with each Box<dyn Reflect>, if I'm understanding that right. ( It's a little fuzzy in my head right now, so I might have that wrong. )

I was thinking about this recently as well. Imo it should be hidden behind a feature flag. If people need it (and most probably don't), they can choose to enable it

Feature flag that's off by default sounds great to me.

For the record: you're right about object size; I was thinking executable size but misspoke.

I think one modification I'd make to your suggestion is to supply the doc strings on the TypeInfo types (e.g. StructInfo, NamedField, etc.) instead of Reflect as this is pretty much what TypeInfo is for. And it could very easily be added to that system.
However, my question regarding using TypeInfo for this: would this work for scripting purposes? I know there are cases where static compile-time info can be problematic for scripting which is inherently dynamic.
If it is a problem to register this info statically, I think we should go with the type data approach. I just don't feel that doc string info should be accessible directly from Reflect. It could but it just doesn't belong there imo.

I'm still building an intuition for how the Reflect API is designed, but what you're saying seems to make sense to me.
As far as the static-ness of data is concerned, static values can still be created fine at runtime, as long as you "leak" them and make sure they are never dropped ( like a lazy_static or static OnceCell, etc. ), so I don't think that's an issue.

Sounds good. Are you planning on working on this @zicklag? I can maybe take it on if you're not planning to.

I'm quite busy at the moment, and this is really a bonus for me, so if you can get to it that'd be great. :D


Add utility function to split string into component values

When parsing component values passed as a string to rangeStart, rangeEnd and inset we have previously used str.split(), either with a single whitespace or with a regex with various issues:

'calc(2 * 10px) auto'.split(' ') returns ['calc(2', '*', '10px)', 'auto'], breaking apart functions
'calc(2 * 10px) auto'.split(/(?<!\([^\)]*)\s(?![^\(]*\))/) returns ['calc(2 * 10px)', 'auto'] as expected, but causes an error in browsers without support for lookahead

This PR adds a utility function splitIntoComponentValues(input) that splits a string with multiple component values into an array of individual component value strings. These can then be processed either as keywords or be further parsed using CSSNumericValue.parse().
splitIntoComponentValues('calc(2 * 10px) calc(2 * 5px)');
// [('calc(2 * 10px)', 'calc(2 * 5px)']

splitIntoComponentValues('10px 10px').map(val => CSSNumbericValue.parse(val));
// [new CSSUnitValue(10, 'px'), new CSSUnitValue(10, 'px')]

@bramus Rebased to account for changes from https://github.com/flackr/scroll-timeline/pull/153

Overall structure looks good. A few remarks to address.

Thanks for the review @bramus 🙏
I think I've addressed all your remarks: https://github.com/flackr/scroll-timeline/compare/1afa09f2c7a3c39efc4a348f0940fd6fb6b03085..8e1896c3c547119a6bafed25ffd764f0efd786b4

Just came to the realization one can perfectly author something like animation-range: calc(0% + 50px)calc(100% - 50px) – with no spaces between the start and end – which isn’t handled by splitIntoComponentValues().
Now, I don’t think anyone would write such a declaration and consider this be an extreme edge case, so I’m fine with merging this as is.
/ping @flackr for an opinion on this.

Just came to the realization one can perfectly author something like animation-range: calc(0% + 50px)calc(100% - 50px) – with no spaces between the start and end – which isn’t handled by splitIntoComponentValues().

We could easily handle that in splitIntoComponentValues() when we consume the end parenthesis.

Updated the PR to support calc(...)calc(...):
https://github.com/flackr/scroll-timeline/compare/8e1896c3c547119a6bafed25ffd764f0efd786b4..14cde5bf6812b437282c49603ec79b10b8d702e4


Adafruit BusDevice error when reading anything

Looks like Adafruit I2C Device class no longer supports stop in write functions.  Needs to be removed.  Will look at a pull request as soon as testing is successful.

Ran into this today while testing a Qwiic Twist with CircuitPython.
>>> qwiic_rot.version
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "sparkfun_qwiictwist.py", line 122, in version
  File "sparkfun_qwiictwist.py", line 363, in _read_register16
TypeError: unexpected keyword argument 'stop'

This should be pretty easy to fix by changing _read_register8 and _read_register16 to use write_then_readinto. There are lots of examples of this kind of change in the mentions on this issue:
https://github.com/adafruit/circuitpython/issues/2082
I've got some time today so I'll take a stab at it and see how it goes.

I guess it went pretty well! The PR should fix things. :D


Acceptance tests do not test real world data

When using om to configure-director, we used the documentation as a reference, and this was incorrect. Could om add end-to-end tests to validate suggested data?

@fredwangwang Can you tell us a bit more about where you're seeing the incorrect documentation? Is this for a specific infrastructure?

As the configure-director method is really a wrapper for an api endpoint of opsman, the method implementation itself has no problem. However, because of the json payload format mismatch between your doc and opsman's doc, we were unable to configure the director correctly. A suggestion would be pointing your doc to opsman's doc and we can find the payload format there, or just keeping the doc up-to-date with opsman's. But it's your choice.

It looks like our README refers users to OpsMan docs and does say that some commands accept JSON and should refer to the OpsMan docs.
Regarding the acceptance tests using fake data, our goal isn't to specify and test actual data. It's would be difficult for us to constantly update our acceptance tests if OpsMan changes what they require.
We're going to close this issue. If this doesn't address the issue, feel free to reopen this issue.

I think we do cover this to some degree in our om CI pipeline as well:
https://releng.ci.cf-app.com/teams/main/pipelines/om
One thing we could (should?) consider doing in the future is bringing the om CI configuration into the om repo and/or extracting some of the pieces into standalone tests here.


Accumulate's recursive function does not actually test recursive mapping

The test here:
https://github.com/exercism/xelixir/blob/master/exercises/accumulate/accumulate_test.exs#L35
Looks like its attempting to check that the accumulate/2 function will continue mapping if it encounters a list as one of the items in the list. This however is not true since the 2nd level of lists is just created in the anonymous function.
The test IMO should be more like:
chars = ~w(a b c)
nums  = ~w(1 2 3)
list = for c <- chars, num <- nums, do: c <> num
expected = [["A1", "A2", "A3"], ["B1", "B2", "B3"], ["C1", "C2", "C3"]]
assert Accumulate.accumulate(chars, &String.upcase/1) == expected

@jeg2 I belive you submitted this exercism

Uh, I don't remember ever submitting an exercism.  😄

Is that not you at the bottom of the readme?? http://exercism.io/exercises/elixir/accumulate/readme

That's me.  Wild.  I wonder what I said???

I believe you misread the actual code from the test. each char is being mapped with the function given, which returns a list with the char and [1,2,3] as strings. the accumulate function does nothing special if the list passed in has a list as an element
I do agree that this test is confusingly named though

So I understood the test name to mean that the map function is applied recursively if a list element is encountered while iterating.
Thats why I put a recursive call in my implementation:
http://exercism.io/submissions/b907c6a12d6e4b0fae4bb664b3b8adbf#L22
Which I found out that the test suite passes without that line since the passed in anonymous function performs the second level "map".
I'm not sure what its trying to test if it doesn't require recursive mapping.

so yes, I think the test name is confusing and should be changed here

Yes, this looks confusing. Would someone mind submitting a PR with a better test description?

Is there a way you can reach this last test and have it fail? I don't see what it drives out, and the test itself is adding non-essential complexity (passing in a function that closes over a list), that has nothing to do with the function tested.
In a code base of my own, I'd remove it. If we change the name, I'd suggest "accumulate lists". Still pretty ambiguous, but perhaps less likely to lead someone to implement a deep map.

@martinsvalin I was suggesting, as you are, that we change the test (leaving the name) to have the user implement a deeply recursive map function. However useless it might be in real life, the exercise would be a good addition.

That's me. Wild. I wonder what I said???

@JEG2 I think this was during the Rogues retreat, and we were casually chatting about mentoring people, and you mentioned that you had gotten some of your mentees to reimplement enumerable methods.

Ah, could be!

I would suggest using accumulate itself instead of for and call it "nested accumulate":
  test "nested accumulate" do
    chars = ~w(a b c)
    nums  = ~w(1 2 3)
    fun = fn(c) -> Accumulate.accumulate(nums, &(c <> &1)) end
    expected = [["a1", "a2", "a3"], ["b1", "b2", "b3"], ["c1", "c2", "c3"]]
    assert Accumulate.accumulate(chars, fun) == expected
  end

I'm scratching my head on this. It looks to me like the original thought was to drive out deep mapping with something like:
test "accumulate recursively" do
    values = [[1,2,3], [4, 5, 6]]
    fun = fn(num) -> num * 2 end
    expected = [[2, 4, 6], [8, 10, 12]]
    assert Accumulate.accumulate(values, fun) == expected
  end

@tejasbubane the change in your PR illustrates that Accumulate.accumulate/2 can take a function that allows it to compute the permutations of two lists. However, I don't see it driving out any additional behavior. This test, as @martinsvalin points out, has the non-essential complexity of closing over num in the passed function.
My recommendation is that we either take this test out or make it drive out additional behavior.

@parkerl Agreed. It does not drive any additional behaviour. My idea was to give a better description to the test (nested instead of recursive) and since we already have accumulate use that itself instead of for. I am perfectly fine with removing the test entirely since it does not test anything new which the rest of the tests don't.


Module name clashing with node-semver port

Hello,
I have a problem of module name clashing between python-semver and https://pypi.org/project/node-semver/ package.
The problem is that it seems that the other version of package is used by conan (C++ package manager) which is quite widely used for C++ projects making python-semver unusable in that environment.
This creates quite some issues as currently both packages cannot coexist together.
Do you have any advice how to overcome to problem?
There is also link to issue in https://github.com/podhmo/python-semver/issues/39 node port.

Hi @MaciejPatro,
thanks for the heads up about this. That's a really interesting problem. I don't know of any method to rename a module later.
In theory, you could patch one of the setup.py files, rename the library, and install it under a different name. However, that's probably cumbersome.
I've did some research and found https://github.com/pypa/packaging-problems/issues/114#issuecomment-348048622 which would not solve the issue, but give some background information. It seems, the problem is known.

@MaciejPatro Maciej, just a theoretical question: would it help if you could download this the wheel, let's say, from GitHub instead of PyPI?

The node-semver library was renamed/will be renamed in version 0.9.0, see https://github.com/podhmo/python-node-semver/issues/39#issuecomment-919706124
I think we can close this issue now.


Added Color for the Squirrel Language

Picked the color from the Squirrel Language's website.
The color is from the Squirrel's logo (Dark Red)

Thanks @iFarbod. This will be live on GitHub in the next few days.


Fix workflows with caching

There is a crash when running update_cache.py even if tuxsuite does not run:
  Reading builds.json
  Traceback (most recent call last):
    File "update_cache.py", line 88, in <module>
      main()
    File "update_cache.py", line 58, in main
      raw = builds_json.read_text(encoding='utf-8')
    File "/usr/local/lib/python3.8/pathlib.py", line 1236, in read_text
      with self.open(mode='r', encoding=encoding, errors=errors) as f:
    File "/usr/local/lib/python3.8/pathlib.py", line 1222, in open
      return io.open(self, mode, buffering, encoding, errors, newline,
    File "/usr/local/lib/python3.8/pathlib.py", line 1078, in _opener
      return self._accessor.open(self, flags, mode)
  FileNotFoundError: [Errno 2] No such file or directory: 'builds.json'

None of the steps after tuxsuite need to run, so apply the same condition to all steps.

So if tuxsuite fails to run there is no builds.json is this causing the crash? @nathanchance


[BUG] Rich traceback is not working with @inject from " ets-labs / python-dependency-injector"

[x] I've checked docs and closed issues for possible solutions.
[x] I can't find my issue in the FAQ.

Describe the bug
Hi I opened the same issue here also https://github.com/ets-labs/python-dependency-injector/issues/667 as I don't know which one must fix this issue.
It happens when I use the @inject decorator
Platform

Click to expand
What platform (Win/Linux/Mac) are you running on? What terminal software are you using?
I may ask you to copy and paste the output of the following commands. It may save some time if you do it now.
If you're using Rich in a terminal:
python -m rich.diagnose
pip freeze | grep rich

pip freeze | grep rich

rich==13.3.1

If you're using Rich in a Jupyter Notebook, run the following snippet in a cell
and paste the output in your bug report.
from rich.diagnose import report
report()

I think I found the issue while I was reading the error messages and I tried to do some modification in the dependency injection library (dependency-injector = "^4.41.0") but it did not work, but then I tried to do modifications in the rich library (rich = "^13.3.1") and it solved the issue.
In the file rich/syntax.py I've done this modification just to see what it will happen:
I added try..catch and break and I'm getting the current token
                def tokens_to_spans() -> Iterable[Tuple[str, Optional[Style]]]:
                    """Convert tokens to spans."""
                    tokens = iter(line_tokenize())
                    line_no = 0
                    _line_start = line_start - 1 if line_start else 0

                    # Skip over tokens until line start
                    while line_no < _line_start:
                        try:
                            current_value = next(tokens)
                            print(f"-------------------->>>>>>>>>> current value: {current_value}")
                            _token_type, token = current_value
                        except StopIteration as err:
                            print(f"-------------------->>>>>>>>>>>>>Error: err ({err})")
                            break

                        yield (token, None)
                        if token.endswith("\n"):
                            line_no += 1

# other code is omitted

Then in the shell I saw that it was empty:
-------------------->>>>>>>>>> current value: (Token.Comment.Single, '# no possibility to compile a first-type citizen coroutine in Cython.')
-------------------->>>>>>>>>> current value: (Token.Text.Whitespace, '\n')
-------------------->>>>>>>>>> current value: (Token.Text.Whitespace, '\n')
-------------------->>>>>>>>>>>>>Error: err ()
-------------------->>>>>>>>>> current value: (Token.Literal.String.Doc, '"""\n')
-------------------->>>>>>>>>> current value: (Token.Literal.String.Doc, '"""')
-------------------->>>>>>>>>> current value: (Token.Text.Whitespace, '\n')
-------------------->>>>>>>>>> current value: (Token.Text.Whitespace, '\n')
-------------------->>>>>>>>>> current value: (Token.Keyword.Namespace, 'from')
-------------------->>>>>>>>>> current value: (Token.Text, ' ')
-------------------->>>>>>>>>> current value: (Token.Name.Namespace, 'uuid')
-------------------->>>>>>>>>> current value: (Token.Text, ' ')
-------------------->>>>>>>>>> current value: (Token.Keyword.Namespace, 'import')
-------------------->>>>>>>>>> current value: (Token.Text, ' ')
-------------------->>>>>>>>>> current value: (Token.Name, 'UUID')
-------------------->>>>>>>>>> current value: (Token.Text.Whitespace, '\n')
-------------------->>>>>>>>>> current value: (Token.Text.Whitespace, '\n')
-------------------->>>>>>>>>> current value: (Token.Keyword.Namespace, 'from')

And because of that it was throwing exception from inside the library.
But now it's working, and I don't know right now if there is some other side effects of using this solution. I just wanted to mention it here as an idea

This looks like it's caused by a change in the read_code function in #2754. Previously, if the file couldn't be read, that would cause an exception inside rich which would be caught rendering that stack frame. Now it uses stdlib linecache.getline(), which is documented: "This function will never raise an exception — it will return '' on errors." Then the rendering code tries to skip some number of lines in '' and crashes.
That said, it seems possible that we can read the file, but the length has changed, so I think catching StopIteration along the lines that @arkanmgerges suggested would be a better fix.

Re-open if not fixed in v13.3.2

Looks like it works; our regression test passes with the latest version of rich. Thank you for the fix (and for the library in general)!


New package: YOLOv7 v0.1.0

Registering package: YOLOv7
Repository: https://github.com/gabrielpreviato/YOLOv7.jl
Created by: @gabrielpreviato
Version: v0.1.0
Commit: 7feff1cd6888d28ca8529cb895118af86dad7e1d
Reviewed by: @gabrielpreviato
Reference: https://github.com/gabrielpreviato/YOLOv7.jl/issues/4#issuecomment-1442241355

Can be deleted


Warning of unused files during ng build. Even with the proposed configuration. Which also deactivates autocompletion.

🐞 Bug report
I know there are some similar issues. I searched this repo and the web. And nothing helped. I think it's not a duplicate. Because in short: I followed the recommended config (no ts-files in "include", entry "files" only). But this does not fully work and also disables intellisense. ...
Command

[x] build
[x] serve

Is this a regression?
I think this is since Angular 9, 10, 11.
Description
I get some warnings about unused ts-files. Even if I remove all "included" of ts-files and only use entry "files".
And over this, I need the includes for the autocompletion / auto-import suggestion / IntelliSense.
Without **/*.ts in include of ts.config, there are less warnings. But for some files like the prod.ts the problem persists.
And without this include, I have no autocomplete anymore. This is no solution for me.
What is this warning? What's the problem to have this "unused" files. Those files are included in the build?
Anyway, I cannot remove all warnings about it and cannot use autocomplete if includes are not set. ...
🔬 Minimal Reproduction

Build a monorepo (in my case).
Call ng build.
Get the warning/s about unused ts-files.
Checkout the tsconfig files. Ensure the there is no **/*.ts as suggested. Use entry files only.
Go to apps\demo-app\src\app\pages\dashboard\dashboard.page.ts and  test the autocompletion with sayHello.
Find out that the autocompletion only works when src/**/*.ts and ../../shared/**/*.ts are set in tsconfig.app.ts.

It does not matter if the include of ts-files is correct or not:

You cannot remove all warnings with this suggestion config.
You drop the autocompletion if you follow the suggestion.

Download Demo-Project
🔥 Exception or Error

Warning: ...\test-demo\apps\demo-app\src\environments\environment.prod.ts is part of the TypeScript compilation but it's unused.
Add only entry points to the 'files' or 'include' properties in your tsconfig.

The file environment.prod.ts is not the only one in the warnings, on a more complex project.
tsconfig.json
{
  "files": ["src/main.ts", "src/polyfills.ts"],
  "include": ["src/**/*d.ts", "../../shared/**/*d.ts"],
  "exclude": ["src/**/*.spec.ts", "../../shared/**/*.spec.ts"]
}

Down-side: No autocompletion. To enable it, I have to include the ts-files:
{
  "files": ["src/main.ts", "src/polyfills.ts"],
  "include": ["src/**/*.ts", "src/**/*d.ts", "../../shared/**/*.ts", "../../shared/**/*d.ts"],
  "exclude": ["src/**/*.spec.ts", "../../shared/**/*.spec.ts"]
}

How to disable this warning? (If nothing helps)
🌍 Your Environment

Angular CLI: 11.0.0-next.7
Node: 10.22.0
OS: win32 x64

Angular: 11.0.0-next.6
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Ivy Workspace: Yes

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1100.0-next.7
@angular-devkit/build-angular   0.1100.0-next.7
@angular-devkit/core            11.0.0-next.7
@angular-devkit/schematics      11.0.0-next.7
@angular/cli                    11.0.0-next.7
@schematics/angular             11.0.0-next.7
@schematics/update              0.1100.0-next.7
rxjs                            6.6.3
typescript                      3.9.7

Hi @infacto,
The problem here looks like it is caused because you updated from 10.0.x to 11.0.0-next.x or from 10.0.x to 10.1.x without using ng update. This means that migrations have not been executed during the update, which leaves you in a "broken" state.
In this case you'll need to run the the migrations using the below command.
ng update @angular/cli --migrate-only --from 10 --to 11

it also appears the the globs you added in tsconfig.app.json are not doing what you expect, which infant rightfully are causing the warnings mentioned above.
The below glob will not include only type definitions, but also will include any file which ends with d.ts, ex: environment.prod.ts
  "include": [
    "src/**/*d.ts",
    "../../shared/**/*d.ts"
  ],

The above should be updated to:
  "include": [
    "src/**/*.d.ts",
    "../../shared/**/*.d.ts"
  ],

@alan-agius4 Argh, I missed the dot. 🤦♂ Thanks, the migration worked. Down-side now: Now you can import stuff from every app. For example, in AppA you can import a class from AppB. It's suggested in IntelliSense. This is a unwanted behavior.
Because tsconfig.app.json is not uses by VS Code. Therefore is the references array in tsconfig.json. Otherwise the config inside tsconfig.app.json like alias paths, excludes, etc. only works for Angular, but not the IDE. But with this references-config, it's impossible to include other folders like "shared". The initial issue. A dilemma. I have no perfect solution yet. Just fyi, but I would be glad if you have a spontaneous idea.
Expected behavior:

In "shared", you can only import stuff from "shared".
In an app, you can only import stuff from this app and "shared".

Current behavior:

You can import anything criss-cross.
Settings in tsconfig.app.ts has no effect. Until references is set in tsconfig.json. But then you cannot include anything because the Warning in Anglular (see initial post).

Hi @infacto,
Unfortunately, the above is expected since in version 10.1 following a meeting with the TypeScript team we removed solutions style tsconfig from Angular projects (ie: tsconfigs with references) due to a number of reasons.
More information: https://github.com/microsoft/TypeScript/issues/39632


Standard_NV24ms_v3 not documented

I cant find the documentation for standard_NV24ms_v3 on your page.

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

ID: f0ffeb72-66b6-6d6e-413c-aa4b8354a2a3
Version Independent ID: f21d752c-a983-fe90-c48f-238d11d6c3c0
Content: NVv3-series - Azure Virtual Machines
Content Source: articles/virtual-machines/nvv3-series.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @vikancha-MSFT
Microsoft Alias: jushiman

Thanks for brining this to our attention. We will investigate it further and update you shortly.

@SvenSowa Can you please reach out to me at<EMAIL_ADDRESS>

@vikancha-MSFT - Thank you for responding to the query.
@SvenSowa - Please send an email to @vikancha-MSFT so that your issue can addressed.
At this moment, Since there is no doc update required, We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Docker Images - Missing

Hi,
I was trying to launch this in docker and ran into these Image Repository links to be empty. Is there a reason they were taken down?
Thanks.
"mszarlinski/spring-petclinic-admin-server" and all other images.

I have pulled images, please try again.

Awesome, thank you @mszarlinski !


Prompt with color

Hello, you will have a problem with coloring prompt, because when using add_history() it will fckd up cause readling will count every color code as char and add it to length.
If you test 2 cmds, and then try to go to history WITH an actual prompt (someting in prompt) it will fckd up!
I was stuck on that problem for like 2 weeks, hope it'll help you <3
GLHF

hi, first of all, thanks for checking out that kind of thing :)
secondly, i'm not exactly getting what error you're refering to? just for context, the prompt here is the part that isn't saved in the history, readline makes it possible to add some

missclick, as I was writing, prompt is just used to add in front of the user added command line
could you show me how to reproduce the bug?

For sure!
Looks my code:

It's my func for create the prompt:

It's similar tou yours, cause of colors.
If you try to run your minishell, with the add_history, it will bug.
To reproduce:

run minishell (with a history cmd)
run 3 or +  differents cmd (like "ls", "env" or "ls -l")
press up arrow key, then space, then up again

Then you will see the "bug"!
Of course I consider that the minishell is working ahahah.
If you want to try, on my account I've almost finished my minishell, and you can clone it to see. I just put in comment de "create_prompt" function with color (and there is one without color under, which is the one I use)
Hope it helped!

can't seem to reproduce at home / don't see the bug if it is present, will try to reproduce once in campus, thanks for the insight, will keep informed once I manage, or not, to recreate the bug

It’s a bug only on mac, I don’t know why, but on linux it seems to be ok…JordanLe 17 oct. 2023 à 09:47, Liam Vincent @.***> a écrit :
can't seem to reproduce at home / don't see the bug if it is present, will try to reproduce once in campus, thanks for the insight, will keep informed once I manage, or not, to recreate the bug
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

well, my campus runs only on linux, so should not be a problem, at least for now (ps: je viens de remarquer mais on est tout deux français il semblerait :p)


Add ability to unlock betterlockscreen from CLI

loginctl lock-session can be used to lock the screen by internally calling betterlockscreen --lock
Similarly is there a way to unlock the screen using maybe something like betterlockscreen --unlock which can be linked to loginctl unlock-session

Yup, seems correct. I'll probably open an issue on i3lock-color.


maven-assembly-plugin does not use source or target in its configuration

Description
Remove incompatible  and  tags from the maven-assembly-plugin configuration
Issues Resolved
Resolves issue #13
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

ran checks - all OK

You approved this pull request and then closed it without merging it. I'm confused.

Eh? Did i miss something? Sorry for that.
Please advise.
Rob Verschoor
AWS | Principal Database Engineer, Aurora/PostgreSQL
<EMAIL_ADDRESS>|  T +31-655-772131tel:+31-655-772131 | twitter: @rob_verschoor
On 1 Nov 2021, at 18:18, brtrvn @.***> wrote:

You approved this pull request and then closed it without merging it. I'm confused.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/babelfish-for-postgresql/babelfish_compass/pull/14#issuecomment-956424578, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVYPZMDCNDELJXY5JNSAOA3UJ3DUPANCNFSM5G5VSWJA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
Amazon Development Center (Netherlands) B.V., Johanna Westerdijkplein 1, 2521 EN The Hague, Registration No. Chamber of Commerce 56869649, VAT: NL 852339859B01


Future Enhancement: Stories Carousel block custom titles

As a future enhancement to the Stories Carousel block, the content editor desires the ability to apply a custom title to be displayed instead of automatically pulling in the Title from the metadata block.

Added requirement in #78


POLL: Old / New Crawl button?

We want your opinion and have started a small poll:
How do you want the Crawl button to look like in one of the upcoming versions?
Please vote by pressing the thumbs up  under one of the options.
Thanks for your participation!

keep it

change 1

change 2

change 3

download is better wording

Thanks to all who participated.
We'll change it to version "change 2".


ImageIO initialization issue

we have the following exception with commit d3bd0ac :
nested exception is java.lang.ExceptionInInitializerError] with root cause
java.lang.IllegalArgumentException: provider is null!
	at javax.imageio.spi.ServiceRegistry.setOrdering(ServiceRegistry.java:590)
	at de.digitalcollections.turbojpeg.imageio.TurboJpegImageReaderSpi.lambda$onRegistration$0(TurboJpegImageReaderSpi.java:84)
	at java.util.Spliterators$ArraySpliterator.forEachRemaining(Spliterators.java:948)
	at java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:647)
	at de.digitalcollections.turbojpeg.imageio.TurboJpegImageReaderSpi.onRegistration(TurboJpegImageReaderSpi.java:79)
	at javax.imageio.spi.SubRegistry.registerServiceProvider(ServiceRegistry.java:731)
	at javax.imageio.spi.ServiceRegistry.registerServiceProvider(ServiceRegistry.java:310)
	at javax.imageio.spi.IIORegistry.registerApplicationClasspathSpis(IIORegistry.java:211)
	at javax.imageio.spi.IIORegistry.<init>(IIORegistry.java:138)
	at javax.imageio.spi.IIORegistry.getDefaultInstance(IIORegistry.java:159)
	at javax.imageio.ImageIO.<clinit>(ImageIO.java:66)
	at io.bdrc.iiif.image.service.ReadImageProcess.getReader(ReadImageProcess.java:150)

while it's ok with commit 6ee3b5a

Fixed as of commit 68b2c0c
The reason is unclear and very weird but this fixed it:
https://github.com/buda-base/buda-iiif-server/commit/68b2c0c43dce90a27f6dfaf76785a119750b3e02#diff-46f1c1ad4c2aa5d8a2b75ec6a1fc869aL21

Still, the problem is occurring. Rolling back to  commit 6ee3b5a

More:
Having this issue, I decided to restart the server using: sudo systemctl restart iiiifserv
That didn't change anything. However, when I did :
sudo bash update.sh (with the same commit) then it fixed the issue.
It is like something is well set on install (so ImageIO is found) then lost during operations...(ImageIO class cannot be found anymore...)

Once again, when I installed the new commit through update.sh, nothing worked correctly and I got the same exception as above...
When I re-ran update.sh, it fixed the issue... I just don't get it !

More on that issue:
if I restart the server and request (jpg):
https://iiif-dev.bdrc.io/bdr:V4CZ5369_I1KG9127::I1KG91270004.jpg/full/1250,/0/default.jpg
it fails
then I request (tif)
https://iiif-dev.bdrc.io/bdr:V4CZ5369_I1KG9127::I1KG91270004.jpg/full/1250,/0/default.tif
and it works
then I retry (jpg)
https://iiif-dev.bdrc.io/bdr:V4CZ5369_I1KG9127::I1KG91270004.jpg/full/1250,/0/default.jpg
and it works
So it looks like ImageIO initialization is working when done through a *.tif image request and not with a *.jpg - Once initialized, everything looks fine.

The workaround described above is not a valid one. I struggled today with that issue and finally it started to work after I re-ran update.sh
I received exclusively this exception:
[2020-05-19 18:45:40,178 ERROR] cat].[localhost].[/].[dispatcherServlet]: 182 [http-nio-13580-exec-3] - Servlet.service() for servlet [dispatcherServlet] in context with path [] threw exception [Handler dispatch failed; nested exception is java.lang.NoClassDefFoundError: Could not initialize class javax.imageio.ImageIO] with root cause
java.lang.NoClassDefFoundError: Could not initialize class javax.imageio.ImageIO
	at io.bdrc.iiif.image.service.ReadImageProcess.getReader(ReadImageProcess.java:150)
	at io.bdrc.iiif.image.service.ReadImageProcess.readImageInfo(ReadImageProcess.java:118)
	at io.bdrc.iiif.controllers.IIIFImageApiController.getImageRepresentation(IIIFImageApiController.java:230)

I am thinking of installing OracleJDK 8 and give it a try...

the best would be to install openjdk on your machine I think

Done, now it fails on my machine as well

Instead of moving to oracle Jdk 8, we actually moved to openJdk11 on both environements (buda1 and buda3) as openJdk 11 will be the official version of java for debian 10.
This solved the problem.


Adds RabbitMQ trace support for pull API and Spring-amqp rabbit

This adds trace support for pull API (Channel.basicGet) and trace propagation for Spring-amqp rabbit configured rabbitmq client applications.
related to #1412

Codecov Report

Merging #3887 into master will decrease coverage by 0.24%.
The diff coverage is 1.21%.

@@            Coverage Diff             @@
##           master    #3887      +/-   ##
==========================================
- Coverage   40.96%   40.72%   -0.25%
==========================================
  Files        2191     2200       +9
  Lines       71016    71460     +444
  Branches     9742     9841      +99
==========================================
+ Hits        29094    29100       +6
- Misses      39165    39604     +439
+ Partials     2757     2756       -1

Impacted Files
Coverage Δ

...nterceptor/MQExternalClientHandlerInterceptor.java
0% <ø> (ø)
:arrow_up:

...t/plugin/rabbitmq/client/RabbitMQClientPlugin.java
0% <0%> (ø)

...in/rabbitmq/client/RabbitMQClientPluginConfig.java
0% <0%> (ø)

...nsumerHandleCompleteInboundCommandInterceptor.java
0% <0%> (ø)

...ceptor/SocketFrameHandlerConstructInterceptor.java
0% <0%> (ø)

...interceptor/QueueingConsumerHandleInterceptor.java
0% <0%> (ø)

...terceptor/RabbitMQConsumerDispatchInterceptor.java
0% <0%> (ø)

...SocketChannelFrameHandlerConstructInterceptor.java
0% <0%> (ø)

...interceptor/ConsumerHandleDeliveryInterceptor.java
0% <0%> (ø)

...interceptor/QueueingConsumerOnNextInterceptor.java
0% <0%> (ø)

... and 29 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 5cc2d0b...7104247. Read the comment docs.


Helm sometimes fails to delete post-install/post-upgrade job

Similar to #1769 we sometimes cannot upgrade charts because helm complains that a post-install/post-upgrade job already exists:
ERROR: Failed to install helm chart on namespace: xxxxx
Reason: Error: UPGRADE FAILED: post-upgrade hooks failed: warning: Hook post-upgrade build/charts/minio/templates/post-install-create-bucket-job.yaml failed: jobs.batch "minio-make-bucket-job" already exists

Chart used: https://github.com/helm/charts/blob/master/stable/minio/templates/post-install-create-bucket-job.yaml:
apiVersion: batch/v1
kind: Job
metadata:
  # [...]
  annotations:
    "helm.sh/hook": post-install,post-upgrade
    "helm.sh/hook-delete-policy": hook-succeeded
  # [...]

The job successfully ran though but we get the error above on update:
$ kubectl describe job minio-make-bucket-job -n xxxxx
Name:           minio-make-bucket-job
Namespace:      xxxxx
Selector:       controller-uid=23a684cc-7601-4bf9-971e-d5c9ef2d3784
Labels:         app=minio-make-bucket-job
                chart=minio-3.0.7
                heritage=Helm
                release=xxxxx
Annotations:    helm.sh/hook: post-install,post-upgrade
                helm.sh/hook-delete-policy: hook-succeeded
Parallelism:    1
Completions:    1
Start Time:     Mon, 11 May 2020 15:28:52 +0200
Completed At:   Mon, 11 May 2020 15:40:46 +0200
Duration:       11m
Pods Statuses:  0 Running / 1 Succeeded / 0 Failed
Pod Template:
  Labels:  app=minio
           controller-uid=23a684cc-7601-4bf9-971e-d5c9ef2d3784
           job-name=minio-make-bucket-job
           release=xxxxx

There is no running pod for that job. It is just the job which exists in the cluster.
Output of helm version:
version.BuildInfo{Version:"v3.2.0", GitCommit:"e11b7ce3b12db2941e90399e874513fbd24bcb71", GitTreeState:"clean", GoVersion:"go1.13.10"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.0", GitCommit:"70132b0f130acc0bed193d9ba59dd186f0e634cf", GitTreeState:"clean", BuildDate:"2019-12-13T11:52:47Z", GoVersion:"go1.13.4", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.10", GitCommit:"1bea6c00a7055edef03f1d4bb58b773fa8917f11", GitTreeState:"clean", BuildDate:"2020-02-11T20:05:26Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
Kubernetes 1.15.10 installed using KOPs on AWS
Is there a workaround for this except manually deleting the job?

Hooks are considered un-managed by Helm. Once a hook is created, it is up to the cluster administrator to clean those up.
That being said, there are hook deletion policies available to help assist in some regards. Have a look at the documentation for more options. https://helm.sh/docs/topics/charts_hooks/#hook-deletion-policies
Hope this helps.

The deletion policy is set inside the chart. It just does not always work in helm 3. It definitely did work fine in helm 2. We got this bug repeatedly every other day


Upgrading to 0.98.0 from 0.97.1 fails meteor app

Description
I know based on research that upgrades usually break meteor component but is there anyway to fix this asap.

What is broken? Can you provide some kind of error log?

=> Started proxy.
=> Meteor <IP_ADDRESS> is available. Update this project with 'meteor update'.
=> Started MongoDB.
=> Errors prevented startup:

   While loading package<EMAIL_ADDRESS>   error: Command failed:
   /root/.meteor/packages/meteor-tool/.1.4.4_1.4figcf++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/bin/npm
   rebuild --update-binary
   node-pre-gyp ERR! Tried to download:
   https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.0.15/fse-v1.0.15-node-v46-linux-x64.tar.gz
   node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and
<EMAIL_ADDRESS>(node-v46 ABI) (falling back to source compile with node-gyp)
   Error extracting archive
   Phantom installation failed { [Error: Command failed: tar jxf
   /tmp/phantomjs/phantomjs-1.9.8-linux-x86_64.tar.bz2
   tar (child): bzip2: Cannot exec: No such file or directory
   tar (child): Error is not recoverable: exiting now
   tar: Child returned status 2
   tar: Error is not recoverable: exiting now
   ]
   killed: false,
   code: 2,
   signal: null,
   cmd: 'tar jxf /tmp/phantomjs/phantomjs-1.9.8-linux-x86_64.tar.bz2' } Error:
   Command failed: tar jxf /tmp/phantomjs/phantomjs-1.9.8-linux-x86_64.tar.bz2
   tar (child): bzip2: Cannot exec: No such file or directory
   tar (child): Error is not recoverable: exiting now
   tar: Child returned status 2
   tar: Error is not recoverable: exiting now

   at ChildProcess.exithandler (child_process.js:200:12)
   at emitTwo (events.js:87:13)
   at ChildProcess.emit (events.js:172:7)
   at maybeClose (internal/child_process.js:862:16)
   at Socket.<anonymous> (internal/child_process.js:338:11)
   at emitOne (events.js:77:13)
   at Socket.emit (events.js:169:7)
   at Pipe._onclose (net.js:490:12)
   npm ERR! code ELIFECYCLE
   npm ERR! errno 1
   npm ERR<EMAIL_ADDRESS>install: `node install.js`
   npm ERR! Exit status 1
   npm ERR!
   npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node install.js'.
   npm ERR! Make sure you have the latest version of node.js and npm installed.
   npm ERR! If you do, this is most likely a problem with the phantomjs package,
   npm ERR! not with npm itself.
   npm ERR! Tell the author that this fails on your system:
   npm ERR!     node install.js
   npm ERR! You can get information on how to open an issue for this project
   with:
   npm ERR!     npm bugs phantomjs
   npm ERR! Or if that isn't available, you can get their info via:
   npm ERR!     npm owner ls phantomjs
   npm ERR! There is likely additional logging output above.

   npm ERR!  Tried to download:
   https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.0.15/fse-v1.0.15-node-v46-linux-x64.tar.gz
   node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and
<EMAIL_ADDRESS>(node-v46 ABI) (falling back to source compile with node-gyp)
   Error extracting archive
   Phantom installation failed { [Error: Command failed: tar jxf
   /tmp/phantomjs/phantomjs-1.9.8-linux-x86_64.tar.bz2
   tar (child): bzip2: Cannot exec: No such file or directory
   tar (child): Error is not recoverable: exiting now
   tar: Child returned status 2
   tar: Error is not recoverable: exiting now
   ]
   killed: false,
   code: 2,
   signal: null,
   cmd: 'tar jxf /tmp/phantomjs/phantomjs-1.9.8-linux-x86_64.tar.bz2' } Error:
   Command failed: tar jxf /tmp/phantomjs/phantomjs-1.9.8-linux-x86_64.tar.bz2
   tar (child): bzip2: Cannot exec: No such file or directory
   tar (child): Error is not recoverable: exiting now
   tar: Child returned status 2
   tar: Error is not recoverable: exiting now

   at ChildProcess.exithandler (child_process.js:200:12)
   at emitTwo (events.js:87:13)
   at ChildProcess.emit (events.js:172:7)
   at maybeClose (internal/child_process.js:862:16)
   at Socket.<anonymous> (internal/child_process.js:338:11)
   at emitOne (events.js:77:13)
   at Socket.emit (events.js:169:7)
   at Pipe._onclose (net.js:490:12)
   npm ERR! code ELIFECYCLE
   npm ERR! errno 1
   npm ERR<EMAIL_ADDRESS>install: `node install.js`
   npm ERR! Exit status 1
   npm ERR!
   npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node install.js'.
   npm ERR! Make sure you have the latest version of node.js and npm installed.
   npm ERR! If you do, this is most likely a problem with the phantomjs package,
   npm ERR! not with npm itself.
   npm ERR! Tell the author that this fails on your system:
   npm ERR!     node install.js
   npm ERR! You can get information on how to open an issue for this project
   with:
   npm ERR!     npm bugs phantomjs
   npm ERR! Or if that isn't available, you can get their info via:
   npm ERR!     npm owner ls phantomjs
   npm ERR! There is likely additional logging output above.

   npm ERR!  Waiting for file change.


Add a way to blacklist names that isn't hardcoding into source control

Currently we blacklist a few project names just by hardcoding those names into the actual source of the upload view. Ideally the list of blacklisted names would just be more data in the database.

Fixed in #2404.


infra: remove target from ovirt_storage_connection

The reason for this is that the storage domain connection can be created with multiple targets, the issue comes when there is another connection with a different target than the user has specified by the storage in the YAML file.
https://bugzilla.redhat.com/show_bug.cgi?id=1943221
same as:
https://github.com/oVirt/ovirt-ansible-collection/pull/252

verified in:
https://code.engineering.redhat.com/gerrit/c/rhevm-qe-automation/ansible-playbooks/+/316502


added certificate-list.md file which contains the list of certificate…

… use in gardener
How to categorize this PR?

/area TODO
/kind TODO
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Ammar seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

As discussed via mail, let's not continue with this PR due to the reasons mentioned above.
/close


Psalm 6 Warning⚠ :  "findUnusedBaselineEntry" will be defaulted to "true" in Psalm 6.

What steps will reproduce the problem? Recent use of Psalm 5.6
What is the expected result? No warnings
What do you get instead? The below warning
Warning: "findUnusedBaselineEntry" will be defaulted to "true" in Psalm 6. You should explicitly enable or disable this setting.
Warning: "findUnusedCode" will be defaulted to "true" in Psalm 6. You should explicitly enable or disable this setting.
Target PHP version: 8.1 (inferred from composer.json) Enabled extensions:  (unsupported extensions: pdo_sqlite).
Additional info

Q
A

Version
1.0.?

PHP version
8.2.1

Operating system
Windows 10

Psalm Level 1 Testing completed with above settings included in psalm.xml file.
findUnusedBaselineEntry="true"
findUnusedCode="false"


Internal affairs: expect_warn_deprecation matches any message when there's fail in the example

Subject of the issue
expect_warn_deprecation matches any message when there's raise_error(RSpec::Expectations::ExpectationNotMetError) in the example.
Might affect other rspec-support deprecation matcher helpers.
Your environment

Ruby version: 2.7.1
rspec-expectations version: d1239222 (main)

Steps to reproduce
Original code:
  it 'prints a deprecation warning when given a value and negated' do
    expect_warn_deprecation(/complete nonsense/)
    expect { expect(3).not_to matcher }.to fail
  end

Simplification:
  it do
    expect_warn_deprecation(/foo/)
    expect {
      RSpec.configuration.reporter.deprecation(message: 'bar')
      expect(1).to change { 1 }
    }.to raise_error(RSpec::Expectations::ExpectationNotMetError)
  end

Expected behavior
       expected StandardError, got #<RSpec::Expectations::ExpectationNotMetError: expected "bar" to match /foo/
       Diff:
       @@ -1 +1 @@
       -/foo/
       +"bar"

or
       expected "bar" to match /foo/
       Diff:
       @@ -1 +1 @@
       -/foo/
       +"bar"

Actual behavior
  is expected to raise RSpec::Expectations::ExpectationNotMetError

1 example, 0 failures

:aggregate_failures fixes the issue with the first expectation, it is now printed, but now the inner failure is propagated.

The use of non-nested expectation resolves the issue:
    expect(RSpec.configuration.reporter).to
      receive(:deprecation).with(include(message: match(/Tahe/)))

just for the reference, original implementation:
  def expect_warn_deprecation(snippet=//)
    expect(RSpec.configuration.reporter).to receive(:deprecation) do |options|
      message = options[:message]
      expect(message).to match(snippet)
    end
  end

That approach looks reasonable, is there a PR?

There are no tests for the whole file, it's tricky to write, takes me a while.
Since we're nearing that moment when deprecation messages are becoming extremely important to properly cover, I'll handle this ASAP.


etcd v3.2.5: missing grpc_server_* metrics when scraping with Prometheus

We are scraping etcd (v3.2.5) instances with Prometheus and not getting any of the grpc_* metrics, all other etcd_* are collected correctly.
Am I missing some configuration? my understanding is that these should be included by default and only the histogram metrics are optional (enabled with –metrics extensive)
Any suggestions? thoughts?

grpc_* won't show up until initial requests come in, as far as I know. Can you try sending test requests to your cluster and try again?

This particular etcd cluster is part of K8s control plane, the clients in this case are ectd proxies deployed on the K8s worker nodes (these are actually running etcd v2).
I tried executing a bunch of etcdctl command (not sure if it's enough)
But not seeing any of the gprc metrics.

I tried executing a bunch of etcdctl command (not sure if it's enough)

What commands are you sending to etcd cluster? If v2, it won't send any gRPC requests.

Works fine:
$ bin/etcd &
$ curl localhost:2379/metrics 2>&1 | grep -v "#" | grep grpc
etcd_grpc_proxy_cache_hits_total 0
etcd_grpc_proxy_cache_keys_total 0
etcd_grpc_proxy_cache_misses_total 0
etcd_grpc_proxy_events_coalescing_total 0
etcd_grpc_proxy_watchers_coalescing_total 0
etcd_network_client_grpc_received_bytes_total 0
etcd_network_client_grpc_sent_bytes_total 0
$ ETCDCTL_API=3 ./bin/etcdctl put abc def
OK
$ curl localhost:2379/metrics 2>&1 | grep -v "#" | grep grpc
etcd_grpc_proxy_cache_hits_total 0
etcd_grpc_proxy_cache_keys_total 0
etcd_grpc_proxy_cache_misses_total 0
etcd_grpc_proxy_events_coalescing_total 0
etcd_grpc_proxy_watchers_coalescing_total 0
etcd_network_client_grpc_received_bytes_total 10
etcd_network_client_grpc_sent_bytes_total 29
grpc_server_handled_total{grpc_code="OK",grpc_method="Put",grpc_service="etcdserverpb.KV",grpc_type="unary"} 1
grpc_server_msg_received_total{grpc_method="Put",grpc_service="etcdserverpb.KV",grpc_type="unary"} 1
grpc_server_msg_sent_total{grpc_method="Put",grpc_service="etcdserverpb.KV",grpc_type="unary"} 1
grpc_server_started_total{grpc_method="Put",grpc_service="etcdserverpb.KV",grpc_type="unary"} 1

Thank you both, indeed all clients (including etcdctl) that we have are v2 and as described these won't generate grpc requests.


Appropriately marking test suites pass/fail

Hi Experts,
I wanted to know what is the best way to handle on how to mark a test suite pass/fail based on the test case status
Attached has two test cases, however the Open Connection fails due to connectivity issues to the server. When I run this and I can see the Test Suite is marked as pass.
I am sure there is a better way to write this test suite, would like to get your views
Thank you
health.txt

This issue tracker is for bug reports and enhancement requests. General questions should be sent to robotframework-users mailing list, our super active Slack, or to other such support forums listed at robotframework.org.