Control D not exiting alacritty

This seems to be a regression caused by https://github.com/jwilm/alacritty/pull/419. Before my PR I could type Control + D and alacritty would happily quit. Now it just hangs and waits for any event to quit. The event could be anything, like lose focus, key press, mouse click, etc.

I can reproduce.

Getting this as well

@michelboaventura Your Glutin patch included this:
diff --git a/src/api/x11/window.rs b/src/api/x11/window.rs
index 48c022f..45af18f 100644
--- a/src/api/x11/window.rs
+++ b/src/api/x11/window.rs
@@ -240,7 +240,7 @@ impl<'a> Iterator for PollEventsIterator<'a> {
                     return Some(Refresh);
                 },
-                ffi::KeyPress | ffi::KeyRelease => {
+                ffi::KeyPress => {
                     let mut event: &mut ffi::XKeyEvent = unsafe { mem::transmute(&mut xev) };
                     let events = self.window.input_handler.lock().unwrap().translate_key_event(&mut event);
                     for event in events {

This looks like it will prevent KeyRelease events from ever being sent. I think translate_key_event should still be called in this case, but is should just not attempt to run Xutf8LookupString. I've pushed a patch to my glutin fork which implements this. Would you mind checking that dead keys are still only printed once?
Incidentally, this doesn't actually fix the bug where the shell exiting doesn't immediately close Alacritty. The bug is worked around by receiving a KeyRelease event. In any case, I think this implementation might be more correct for glutin/winit.

Dead keys still work. Thanks!
@jwilm, is there a way to compile alacritty faster? If i change a single line of code and run cargo build it takes 147 seconds to compile!


No support for selectors

Because Dispatch doesn't return anything, we cannot dispatch a getValueFromState event.

Possible solution?  https://github.com/johnathanhowell/gredux/pull/5


#1. Ubuntu/Debian Quick Start Guide issue: ClickHouse does not start …

…with no password.
This is a suggested fix for #1

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Closing: a wrong repository was used.


non recursive version

If you have time, can you write astar non recursive version?

Well, I don't have time. ;)


Support .Net 6

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] Build (fake build or .\build.cmd) on local branch was successful
build -t NuGet works

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
Doesn't support .Net 6.
What is the new behavior?
Supports .Net 6
Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

I think this should be changed to 2022. The 2019 image on Appveyor does not have .NET 6 and you'd need to install it manually.
https://github.com/Shmew/Fable.SignalR/blob/dab3b002c00cd298d34f6d3c292c735b1fec16ce/appveyor.yml#L3

Hello, @Shmew, are you around? This would be cool to have.

@Shmew what's preventing this from being merged? Anything we can help with?

@PierreYvesR thanks for getting this across the line!
I noticed in the paket.dependencies the Net6 group both Saturn and Giraffe need to be updated to their latest versions, if you want them pinned they need to be updated to 0.16.1 and 6.0.0


Any way to ignore some directory ?

I want the code which is not in the src/ directory not to be counted in the coverage report, is there any way to do that?

Yeah, for sure.  If you register a custom loader, you can set the exclude option.  Have a look at https://github.com/benbria/coffee-coverage/blob/master/docs/HOWTO-istanbul.md#writing-a-custom-loader.  (It would be nice if there was an include, so you could say "only include /src", but right now there's only an exclude.)


Add initial manage admins functionality via GitHub API

Feature description
We will start interacting with the GitHub API to manage Admins and add users to teams.
This will help keep the experience in the console instead of having users go out to GitHub to manage their users.
Feature design or visual mockup

Additional context

Duplicate of #141


allow setting of legacy pattern for charuco

App doesnt seem to detect any points on my charuco board. Board wasn't created by this app but follows 4x4_250 charuco dictionary with A0 dimensions and 10x14 grid. I've tried with two different cameras, just to make sure that video file or colors weren't the issue. No points appear on the frames, clicking "add grid" button results in "Not enough points....grid not added..." error in the console. Autocalibration throws "Waiting for sufficient calibration boards to become populated at port 1" for each frame. I've verified that the board and video files are fine by writing my own simple test program, corners and markers are detected without any issues. What might be the issue here?

Check the config.toml where the charuco board is defined. By default the dictionary is the 4x4_50. Hopefully if you update that parameter in the charuco definition things should flow through. Please let me know if this resolves the issue!
[charuco]
columns = 4
rows = 5
board_height = 11.0
board_width = 8.5
dictionary = "DICT_4X4_50"
units = "inch"
aruco_scale = 0.75
square_size_overide_cm = 5.4
inverted = true

I have already modified the config appropriately. This is its full contents:
`camera_count = 1
creation_date = 2024-10-20T22:19:10.125729
fps_sync_stream_processing = 100
save_tracked_points_video = true
[cam_1]
port = 1
rotation_count = 3
error = "null"
matrix = "null"
distortions = "null"
translation = "null"
rotation = "null"
exposure = "null"
grid_count = "null"
size = [4608, 2592]
[charuco]
columns = 14
rows = 10
board_height = 84.1
board_width = 99.99
dictionary = "DICT_4X4_250"
units = "cm"
aruco_scale = 0.75
square_size_overide_cm = 8.0
inverted = false`
With another problem I've encountered, board_width is capped to 99.99 (not enough for 1189mm of a0 format). Dimensions shouldn't be a problem when it comes to just detecting points, should them?

Thank you for bringing the value cap issue to my attention. I've updated a separate issue related to that input box so that I can address this at the same time: https://github.com/mprib/caliscope/issues/641
My recollection is that the board dimensions should not be influencing the point detection. That is there primarily to aid with printing. The square size override is there to help with the world scale.
Back to your Issue....
One other thought is that it is the aruco_scale may be throwing things off. In the raw charuco board definition, the scale is used to set the aruco_length parameter based on the square_length (inferred from the board dimensions or the override):
https://github.com/mprib/caliscope/blob/9748eab9bf1d064e888bc9d737d561add53b5620/caliscope/calibration/charuco.py#L82
I hope to have more time tomorrow/later this week to look at this. If you are comfortable sharing one of your calibration video files it would be helpful for my troubleshooting/iterating.

@mssc89,
I just merged in some changes and published to PyPI (v0.5.2). I have made some updates that should hopefully resolve the board size restrictions you encountered and make the GUI run a little more smoothly.
I continue to suspect that the aruco_scale parameter is the thing that needs to be modified in the config to get the board recognized. This value is just the aruco_length/square_length from the cv2.aruco.CharucoBoard call that was used to create the board you are using.
Long ago I had played around with changing the relative size of the aruco marker relative to the solid squares and settled on a 0.75 ratio as a sensible default that seemed to have good recognition. I'm curious to learn what your board is using.
Please let me know if this resolves your issue. If not, I'm still open to troubleshooting it, though it would be easier even with just a single frame or screenshot of the charuco you are using.

Thanks for quick reply. When it comes to aruco_scale, as my aruco markers are 60mm and squares are 80mm, 0.75 should work fine (another problem I've encountered, aruco_scale value set in config gets overwritten with 0.75 when launching the app, so I can't test other values). I've tried playing around with other config values but no luck.
I attach 9 seconds of my calibration video:
https://github.com/user-attachments/assets/df3cdd74-0861-48f1-93aa-72eed401ee2a

Thank you for the video file! When I use the configuration above (10 rows, 14 columns, DICT_4X4_250, aruco_scale = 0.75) this is the board generated by opencv that Caliscope is looking for (and doesn't match the board in the video):

Could you share the code that you used to define the board such that the points were successfully identified in your test script?
Just to provide some additional context, the board dictionary name in the config is ultimately used to map to a cv2 dictionary as shown in the snippet from charuco.py:
ARUCO_DICTIONARIES = {
    "DICT_4X4_50": cv2.aruco.DICT_4X4_50,
    "DICT_4X4_100": cv2.aruco.DICT_4X4_100,
    "DICT_4X4_250": cv2.aruco.DICT_4X4_250,
    "DICT_4X4_1000": cv2.aruco.DICT_4X4_1000,
    "DICT_5X5_50": cv2.aruco.DICT_5X5_50,
    "DICT_5X5_100": cv2.aruco.DICT_5X5_100,
    "DICT_5X5_250": cv2.aruco.DICT_5X5_250,
    "DICT_5X5_1000": cv2.aruco.DICT_5X5_1000,
    "DICT_6X6_50": cv2.aruco.DICT_6X6_50,
    "DICT_6X6_100": cv2.aruco.DICT_6X6_100,
    "DICT_6X6_250": cv2.aruco.DICT_6X6_250,
    "DICT_6X6_1000": cv2.aruco.DICT_6X6_1000,
    "DICT_7X7_50": cv2.aruco.DICT_7X7_50,
    "DICT_7X7_100": cv2.aruco.DICT_7X7_100,
    "DICT_7X7_250": cv2.aruco.DICT_7X7_250,
    "DICT_7X7_1000": cv2.aruco.DICT_7X7_1000,
    "DICT_ARUCO_ORIGINAL": cv2.aruco.DICT_ARUCO_ORIGINAL,
    "DICT_APRILTAG_16h5": cv2.aruco.DICT_APRILTAG_16h5,
    "DICT_APRILTAG_25h9": cv2.aruco.DICT_APRILTAG_25h9,
    "DICT_APRILTAG_36h10": cv2.aruco.DICT_APRILTAG_36h10,
    "DICT_APRILTAG_36h11": cv2.aruco.DICT_APRILTAG_36h11,
}

This is the dictionary that is ultimately used to create the charuco board in charuco.py:
        return cv2.aruco.CharucoBoard(size=(self.columns, self.rows),
                                      squareLength= square_length,
                                      markerLength= aruco_length,
                                      dictionary= self.dictionary_object,
        )

Thank you!

I see, I've generated my board using calib.io, not opencv own functions. Seems like calib.io is using the "legacy" generator which starts board with a marker instead of a chessboard when row count is even, as opencv did until release 4.6.0. they hid this functionality behind setLegacyPattern() (https://github.com/opencv/opencv/issues/23152). Is there any chance to implement support for these "legacy" boards into caliscope?

I'm using PyPI

Cool. I think I have it working.  The latest version is now published to PyPI as 0.5.3, though with the way these things go, I probably broke something else while I was at it.
At the very least, your board is recognized:

Note that you need to add legacy_pattern=true to the charuco parameters in the config.toml. Also, note that this board is 10 columns and 14 rows (not 14 rows and 10 columns):
[charuco]
columns = 10
rows = 14
board_height = 60.1
board_width = 99.99
dictionary = "DICT_4X4_250"
units = "cm"
aruco_scale = 0.75
square_size_overide_cm = 8.0
inverted = false
legacy_pattern = true

Please let me know how it goes!

Thank you, I’ve tested the new version and it works as expected. I have two additional questions for you, I don’t want to open additional issues for them:

What are your hands-on recommendations for calibration videos? I’ve followed scarce tutorials found on internet and my results are suboptimal (reprojection error is between 5-20 depending on the video).
Did you consider using deepcharuco instead of opencv implementation? From little testing I did, it shown to be more robust in difficult lighting and angle conditions.

Glad to hear it's working!

What are your hands-on recommendations for calibration videos? I’ve followed scarce tutorials found on internet and my results are suboptimal (reprojection error is between 5-20 depending on the video).

I'd have two recommendations after viewing your sample calibration video. First, have the board shown from multiple distances. Second, have the board shown from multiple angles. You want the foreshortening to be quite apparent.
You may also benefit from having the camera fixed and slowly moving the board in front of it (held at an angle and back and forth through a range of distances). Motion blur is your enemy.

Did you consider using deepcharuco instead of opencv implementation? From little testing I did, it shown to be more robust in difficult lighting and angle conditions.

I wasn't aware of this project and am glad that you've put me in the loop. I definitely see the value in this for allowing lower exposure/less blurred calibration images. If there was a simple plug and play solution for this I'd definitely try to integrate it in the near term. Is there a specific implementation you would recommend?

The video I attached here was just a quick test recording, I've refined my methods since. Currently I have the camera fixed and try to move the board around with varying distance and angles - these "refined" videos result in pretty high RMSE/reprojection error, even for cameras without significant distortions. I will try to re-record them with your tips in mind, do you by any chance have a good calibration video at hand?
When it comes to deepcharuco, I've briefly tested this implementation as it seems to be the only publicly available one. it also supports MPS on mac systems out of the box and has pretty good inference instruction straight in the readme.


Customisable theming

Clear and concise description of the problem
It could be rather nice to be able to switch out the theme used by Woodpecker.
Suggested solution
It would be great if one could provide an alternative set of colours/styles (via a theme.css file) to be used.
Alternative
No response
Additional context
This issue has been spun off from https://github.com/woodpecker-ci/woodpecker/issues/468#issuecomment-1207185270
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

All colors should be extracted to CSS variables on the :root node. This will make designing custom color themes a breeze.

I would also suggest that an index of sorts be made available for themes, similar to plugins.


indent title contact form

indent to the left title

Contact for heading "Company Information" is now centered with the contact form.

*form heading


chore: bdd-sbhg-1588128923 to 0.0.1

chore: Promote bdd-sbhg-1588128923 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


removeEdge leaves traces in Vertex

I hava a very simple relationship

Player --> plays --> Team

When I remove the relationship (edge) between class Player and class Team using the following java code
Iterator<Edge> team_edges = player.getEdges(Direction.OUT, "plays").iterator();
while(team_edges.hasNext()) {
    final Edge edge = team_edges.next();
    graph.removeEdge(edge);
}

The edges are all successfully removed, but when I check the OrientStudio I can still see the link to the edge in both the Player and the Team as out and in respectively.
If I try to remove the occurence I get a 404 not found error. At first I though that was a bug in the studio, but when I used the countEdges method on the Player object I got the exact same amount of edges the studio reported.
If i count the steps of the above iterator or print the edge before the remove action I only get the remaining edges as expected.
As a side note, I have the same problem for every relationship in my application, this was just the simple case.
I am developing a pure java application using the Java Graph API and the above problem occurs in both 2.0.6 and 2.0.8 versions.

find out the root cause, if have the edge class with a different case of the label it fail to remove the links, easy to reproduce with:
graph.createEdgeType("Plays");
player.addEdge("plays", team);
//and here the remove

@tglman I think this issue is somehow related to #3980
In my opinion it's not clear how to use labels and class names together for an edge. As reported in the other issue, if label and className are synonyms the API is misleading: In
https://github.com/orientechnologies/orientdb/blob/master/graphdb/src/main/java/com/tinkerpop/blueprints/impls/orient/OrientVertex.java#L850-L944
if you specify both label and className only the label is used. I would like to have some clarification on this.

@fabriziofortino yes the className is there only for backward compatibility, before remove it could be better deprecate it, i'm going to do some fix to make the use of the labels an classes consistent.

@ctampak  by the way as work around you can check that all the label have the same case of the related Edge class

found a work around  to fix this specific use case, closing

I just tried the proposed solution and I can confirm that it works!
I also agree with @fabriziofortino that the documentation should clearly state the situation.
@tglman I will now close the bug and mark it fixed.
Thank you all for your time

Fixed


README-fr.md

Hi everybody.
This is the french translation of the Readme.
Translation for state/dependency/route management soon.
Cheers

Great work!

Great work!

Thank you @eduardoflorence.

Thank you @eduardoflorence.

Thanks for your contribution!

Thanks for your contribution!

My pleasure @jonataslaw.

My pleasure @jonataslaw.

Thanks again for your work. Is this version ready for merge?

Hi @jonataslaw . Yes, it's all done now. I've  just added the links for the files in french inside the spanish and chinese READMEs too.

Ok, LGTM, I'll merge


XREAL Light PacketTimeout when setting display mode on latest firmware

I'm having trouble putting my Lights into Stereo mode.
They're running firmware version <IP_ADDRESS>_20230518, which appears to be the latest.
I added the following to NrealLight::new_common to print the firmware version:
let res = result.run_command(Packet {
    category: b'3',
    cmd_id: b'5',
    ..Default::default()
})?;
println!("firmware version={}", String::from_utf8(res).map_err(|_| Error::Other("Firmware version was not utf-8")).unwrap());

Could you check what firmware version you're running on your glasses?
The following firmwares are available in the latest APK:
<IP_ADDRESS>_20200922.bin
<IP_ADDRESS>_20200922.bin
<IP_ADDRESS>_20200922.bin
<IP_ADDRESS>_20220523.bin
<IP_ADDRESS>_20230317.bin
<IP_ADDRESS>_20230518.bin
I intend to use the sources available here https://ota.xreal.com/en/light-activation.html to downgrade my glasses to whatever version you're using.
A Rust port should be easy enough and could be included in this repo if you'd like.

Hi,
I'm using <IP_ADDRESS>_20221114.bin, but instead of you downgrading, the timeout bug should be fixed instead IMO. Have you tried increasing the command timeout? That might be enough. If that's not enough, maybe we should use some alternative display mode switching command.
A firmware update functionality would be pretty great actually, so I'd really appreciate that.
By the way, if you screw up the firmware update, there's an emergency boot mode on the glasses if you long press one of the buttons while powering it up. It then comes up as a serial console so you can unbrick it. This blogpost may or may not be related: https://voidcomputing.hu/blog/crc-reversing/ :)

I did try increasing the timeout to no avail. It's possible there's a new command that needs used but my reverse engineering skills are zero!
I've managed to get a firmware flasher put together based on the sources on that site. It works great! Here it is: https://github.com/luveti/xreal-light-firmware-flasher
I did actually fail to flash an image part way through (wasn't reading from the serial port, so it would hang). It appears the glasses just boot into flashing mode when an update fails. Which is awesome to see! This may not always be the case, so it's great to know you can force it with the button press.
But... the firmware update didn't help. It looks like the reason has nothing to with the firmware, but is instead due to a this change.
The MCU is expecting the display mode as ASCII numbers, but this commit changed it to use decimals numbers.
Reverting that change fixes the issue! It works on both your firmware and the latest firmware. So no changes are required on that end.

Damn, that's some big regression right there! I need to test changes more, but it gets difficult with this many glasses :)
But at least now I know about your firmware flasher project, it will be quite handy.


Make option to contribute more conspicuous

I really want to encourage new contributions to this site. I propose replacing the contribution text and link at the bottom of the landing page with a "Contribute" option in the horizontal nav bar, at the end. We can make room for this by dumping the "Home" option, which I think is redundant.
Thoughts?
cc  @EssamWisam

I really want to encourage new contributions to this site.

Same here and the skills needed surely exist in the community.

by dumping the "Home" option, which I think is redundant.

Indeed, it is, as the logo does equivalent functionality.
The question becomes do you prefer that we:

Make it link to the For Developer section in the README.

OR replicate that in a new page following the the theme of DataScienceTutorials.

The question becomes do you prefer that we:

Make it link to the For Developer > section in the README.

OR replicate that in a new page following the the theme of DataScienceTutorials.

I vote for option 1, but I don't have a strong opinion.
Maybe we could also add there an explicit link to an existing tutorial in _literate that could be used as a template.


fix: some v3 branch fixes

@JamesMessinger
Hi!
I was testing out the v3 branch as a dependency in my repo https://github.com/mustafar/parrot/
I noticed that next() was not being invoked in some validate-request middleware; this was causing the middleware chain to stall so fixing that here.
I also saw that the isOpenApiRequest predicate was not guaranteed to be a bool. fixing that too.
Let me know if there's anything else I need to do to get this accepted and merged. Thanks!

Pull Request Test Coverage Report for Build 289

10 of 40 (25.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 41.122%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/middleware/validate-request/validate-request-body.js
2
4
50.0%

lib/middleware/validate-request/validate-content-length.js
3
11
27.27%

lib/middleware/validate-request/validate-params.js
4
24
16.67%

Totals

Change from base Build 284:
-0.04%

Covered Lines:
784

Relevant Lines:
1717

💛  - Coveralls


Add preview images to the RS website (metadata)

Problem statement
Images have been created in a previous ticket by Audrey. Need to be added to RS website.
What you need to know

#12889

Acceptance criteria

[ ] Preview image(s) show up when linking to RS website
[ ] Refresh LinkedIn's cache for this issue (not currently working on LinkedIn)

Closing. Duplicate of #12891


Add the ocm-policies namespace to the uninstall

ConfigurationPolicies in that namespace should be checked for finalizers so that the uninstall can complete. It should not be checked when the addon is in hosted mode, since in that case the namespace might be used by other clusters.
Refs:

https://issues.redhat.com/browse/ACM-14707

This flag was added to config-policy-controller in https://github.com/open-cluster-management-io/config-policy-controller/pull/303


Refactor SiteInfo class to use attrs dataclass

Refactor the SiteInfo class in site_info.py to use the attrs library dataclass and clean up the __init__ function.

How do test fixes fit in with this issue? It will take some time to get those working again. I did some of that in my branch


There is a issue when trying to 'go get'  the repo

c.client.config.Config.Consumer.Offsets.Retention undefined (type struct { CommitInterval time.Duration; Initial int64 } has no field or method Retention)

thats the error that its throwing when trying to get the repo

my b had a old version of sarama


toDataURL("image/png") failed to recognize buffer as png

Hello, cannot compress screenshot I make in my app with toDataURL, it is silently returned failing isPng check. I tried to remove part of string before comma as well but have no effect. Will try to use file but really would be nice to be able to work with buffer.

 Working well, sorry for disturbance.


No issue but question: Stopping

Hi,
thanks a lot for your tool. I started some downloads that happen to be live for way longer than thought or needed. Now I wanted to stop the recording to have the recording until that point.
I pressed Ctrl + C, I think after that it asked something if I want to stop, I did and then it had an error message and it deleted the 2 .ts files.
That was still when used the python-Script. Does the exe-File has some check before deleting the .ts-files?

I realized I added this in commit 1b80152a1cafd3a5045397b6df102b38b08a6e11
Closing.


macOS TextField: deleted letters reappears in "pre-confirmed" text entry in Japanese

Intro to the issue
A language like Japanese requires constant 'translation' of entered texts (with the spacebar usually) between its own different alphabet families (like Kana vs Kanji). By default pressing spacebar goes through a list of suggested translations, or arrow keys (up/down) should do the same. Once we find a specific translation we want, we confirm with the return key. Example: Kana style "あい" can be translated into many different forms in Kanji style, like "愛", "藍" or others.
The issue faced
TextField indeed accepts translation attempt with spacebar, however, there are following issues associated to this.

Cycling through the list of suggested translations with pressing spacebar works as intended, but not with up/down arrow keys. This suggestion list pop up is an OS level component where the same pop up is shown in any other app. Seems like spacebar press event is captured correctly by the OS, but not up/down arrow keys' press events. Instead, up/down arrow key event seems to be captured by TextField's TextEditingController to move the cursor, which definitely makes the TextField unusable.
(This issue is even more severe) When we make a typo and use the backspace key to delete letters, it seems like those deleted letters are still cached in the controller or somewhere and reappears in the next character typing. Example: "あいう" -> (delete う) -> "あい" -> (type え) -> "あいうえ" is what we get.

Steps to Reproduce
Run this with macOS as a target. (Only happens with macOS)
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Container(
          padding: EdgeInsets.all(20),
          child: Center(
            child: TextField(),
          ),
        ),
      ),
    );
  }
}

For issue 1

Type "あい"
Hit the spacebar for translation till you see a pop up translation suggestions
Try to cycle through the suggestion list with up/down arrow key. Shouldn't work...

For issue 2

Type "あいう" - Make sure you don't hit enter here
Assuming "う" was a typo, delete it with backspace key. Current entered text should look "あい"
Assuming "え" was what you intended to type, type "え"
If you were able to reproduce the error, you should be seeing "あいうえ" where "う" shouldn't be included. It should have been "あいえ" instead.

Expected results:
Since it never works on flutter macos app, just a sample reference.
Issue1: arrow key up/down is able to cycle through the suggested translations
https://user-images.githubusercontent.com/48747894/142588870-f310e962-0841-42b9-b47b-d87e306b86fb.mov
Issue2:
https://user-images.githubusercontent.com/48747894/142589017-ec203504-9d5b-4c22-a885-3b915267e5f9.mov
Actual results:
Issue 1: Notice only the cursor is moving around as I hit up/down arrow keys
https://user-images.githubusercontent.com/48747894/142584167-2a9df5b1-c9ce-4fca-9903-6d03fd21f205.mov
Issue 2:
https://user-images.githubusercontent.com/48747894/142584200-0f714275-5953-4526-b51e-8ba6e5fca58c.mov

Logs

*flutter run --verbose* output was too long and going over the max characters... So below.

flutter_run_--verbose.txt

Analyzing jp_textfield_test...

   info • Prefer const with constant constructors • lib/main.dart:15:20 • prefer_const_constructors
   info • Prefer const with constant constructors • lib/main.dart:16:18 • prefer_const_constructors
   info • Prefer const with constant constructors • lib/main.dart:17:20 • prefer_const_constructors

3 issues found. (ran in 1.8s)

[✓] Flutter (Channel stable, 2.5.3, on macOS 11.6 20G165 darwin-arm, locale en-JP)
    • Flutter version 2.5.3 at /Users/kazuki/Developer/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (5 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

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

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] Connected device (3 available)
    • iPhone 12 mini (mobile) • 3C6E03A6-8778-4258-891E-B589390B5EF5 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-4 (simulator)
    • macOS (desktop)         • macos                                • darwin-arm64   • macOS 11.6 20G165 darwin-arm
    • Chrome (web)            • chrome                               • web-javascript • Google Chrome 96.0.4664.55

! Doctor found issues in 1 category.

This issue #52718 is not the same but can be related.

Hi @K-Maron,  Thanks for filing the issue.  The first issue is a duplicate of https://github.com/flutter/flutter/issues/85328,  I was able to reproduce the second issue too.
Second Issue
https://user-images.githubusercontent.com/31410839/142627287-7ba90a2f-b300-4b06-8e29-42f19f3bd205.mov
Other issues I could notice are

keyboard shortcuts do not work cmd+c, cmd + a, cmd+v etc.
Entering some text enters a different text on the left of the text
When the Desktop window loses focus the suggestion menu still stays open at the same location (on top of other apps).

flutter doctor -v
[✓] Flutter (Channel stable, 2.5.3, on macOS 12.0.1 21A559 darwin-arm, locale en-GB)
    • Flutter version 2.5.3 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (5 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.1, Build version 13A1030d
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.61.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.28.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.0.1 21A559 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 96.0.4664.55

• No issues found!

[✓] Flutter (Channel master, 2.6.0-12.0.pre.757, on macOS 12.0.1 21A559 darwin-arm,
    locale en-GB)
    • Flutter version 2.6.0-12.0.pre.757 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 1ba81373ee (34 hours ago), 2021-11-17 22:01:34 -0500
    • Engine revision a3ef1d5351
    • Dart version 2.16.0 (build 2.16.0-20.0.dev)
    • DevTools version 2.8.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.61.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.28.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.0.1 21A559 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 96.0.4664.55

• No issues found!

Hi @K-Maron,  Thanks for filing the issue.  The first issue is a duplicate of https://github.com/flutter/flutter/issues/85328,  I was able to reproduce the second issue too.
Second Issue
https://user-images.githubusercontent.com/31410839/142627287-7ba90a2f-b300-4b06-8e29-42f19f3bd205.mov
Other issues I could notice are

keyboard shortcuts do not work cmd+c, cmd + a, cmd+v etc.
Entering some text enters a different text on the left of the text
When the Desktop window loses focus the suggestion menu still stays open at the same location (on top of other apps).

flutter doctor -v
[✓] Flutter (Channel stable, 2.5.3, on macOS 12.0.1 21A559 darwin-arm, locale en-GB)
    • Flutter version 2.5.3 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (5 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.1, Build version 13A1030d
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.61.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.28.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.0.1 21A559 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 96.0.4664.55

• No issues found!

[✓] Flutter (Channel master, 2.6.0-12.0.pre.757, on macOS 12.0.1 21A559 darwin-arm,
    locale en-GB)
    • Flutter version 2.6.0-12.0.pre.757 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 1ba81373ee (34 hours ago), 2021-11-17 22:01:34 -0500
    • Engine revision a3ef1d5351
    • Dart version 2.16.0 (build 2.16.0-20.0.dev)
    • DevTools version 2.8.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.61.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.28.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.0.1 21A559 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 96.0.4664.55

• No issues found!

CC @cbracken

/cc @justinmc @LongCatIsLooong since both of them have been looking at IME input on Mac recently.

duplicate of https://github.com/flutter/flutter/issues/78061?

How embarrassing... yep, I filed it myself...

Alright, since both parts of this are dupes, I'm closing this one.


Type mismatch between mz_uint64 and std::uint64_t

In line 518, mz_uint64_t is defined as unsigned long long.
However, std::uint64_t on my platform (Linux 64-bit Debian 11) is defined as unsigned long.
This leads to errors on multiple lines.

and when I correct the type, I get the error multiple definition of  almost everything.
Not sure what am I doing wrong there

Same thing happens over here on Ubuntu 22.04. Anyone found a solution?


RuntimeError: The size of tensor a (4) must match the size of tensor b (64) at non-singleton dimension 1

Hello, thank you very much for your open-source code! However, I encountered a problem when running MobileNetV4HybridMedium and MobileNetV4HybridLarge.
The error message is
File "/model/mobilenetv4.py", line 187, in forward
attn_score = torch.matmul(q, k) / (self.head_dim ** 0.5)
RuntimeError: The size of tensor a (4) must match the size of tensor b (64) at non-singleton dimension 1

At this time, my model is MobileNetV4HybridMedium, with batch_size=64. The printed information is "Size of q tensor:([64, 4, 196, 64])
Size of k tensor:([64, 64, 49])".

Hi @ponyling,
Thank you for pointing this bug. The latest commit should fixed this problem, kindly have a look. Cheers!


[FEATURE] import-images from file

It would be great to add to a flag  to the import-images argument to specify a local file. Atm in my CircleCI script when I restore an image from a local cache (as docker cache is a premium option), I have to load the docker image on disk into docker, then use k3d to import it. If I had such an option I would do it in one step and it would be much quicker.
So what I envision is something like
k3d import-images -f saved-image.tgz or gunzip -c saved-image.tgz|k3d import-images -f -
Describe alternatives you've considered
At the moment, I'm doing
docker load < saved-image.tgz
k3d import-images name-of-my-image

Hi @cscetbon , thanks for opening this issue :)
I agree that loading images from a tarball (or tar stream) would be nice to have.
However, k3d v1.x is in maintenance mode now and no new features will land there.
Luckily, loading images from tarball/-stream is already planned (and started) for the upcoming v3.0.0 release :+1:

Apparently, it's not trivial to load from stdin and brings in quite a lot edge cases.
I hope that loading from file is good enough for now :+1:

@iwilltry42 this is niiiiice. Thanks !

@iwilltry42 I've tested the last version and it only supports tar. It would be great if it could support tar.gz as well

@cscetbon , yes unfortunately that's the current limitation. The thing is that

docker save produces a tar stream
ctr image import takes a tar stream
so it would be additional effort on k3d side to decompress the gzip to have the plain tar archive for ctr to unfurl :thinking:

Yep was asking cause I don't know how much effort we're talking about


Publish some NeuroML models from the Neurocomputinglab (ErasmusMC)

The Neurocomputinglab has quite some nml models in active use. It would be nice to publish these to e.g. https://neuroml-db.org. This strengthens the nml ecosystem and gives nmlcc an even better reason to exist/be used.
Assign to @llandsmeer

Good idea. More things to be seen are always nice.

One question though: in what capacity is this a feature/issue with nmlcc? Do you propose to make these published models part of this repo?

We had a small brainstorm yesterday on possible projects for @llandsmeer, and perhaps for a student of his who'll start in november. All models in neuroml-db.org were seeded with a one-pass conversion in 2018, and outside of Padraigs group, the lab at Erasmus is to our knowledge the only place actually using the format in the course of science and training of students. We'll get back from @llandsmeer on how much effort Mario and Christos want to put on building an nml-ecosystem, but if that is nonzero, having some of their models as the first contributions to neuroml-db.org could be a good start for them, and for us because we'd make sure the models would be completely supported by nmlcc and Arbor.


Support for more databases

Based on a discussion on [hackernews] (https://news.ycombinator.com/item?id=31266460)it might make sense for us to support multiple storages.
Even though CockroachDB will be the primary one
Acceptance criteria

[x] #3898
[ ] Discuss what more we can support

I like to hear your thoughts on this @adlerhurst

IMO before we begin we should discuss how we will test other databases. They might need different indexes. We can assume that we have to create the same indexes and index types as for crdb.
To not break the configuration files structure for v2 we have to add a database type, this must be done before the release.

Added task for database type: #3600

I did some research around the offerings from the hyperscalers. It turns out that >80% of them has some form of pure Postgresql offering. Based on this I think we should allow and test for plain Postgresql as well besides cockroachDB.
What I currently would ignore are their proprietary offerings e.g CosmosDB, Amazon RDS, Google Spanner. Even though some of them provide Postgresql as dialect

We are interested in using zitadel and being able to use postgres is the largest hurdle for adoption on our side.

@Sytten
We are currently valuating to support plain postgresql besides cockroachDB. This because multiple customers are interested in using the managed postgresql offerings.
I think we should be able to given an update to this in the next 2-4 weeks.

Nice! Let me know if you need beta testers, willing to give you good feedback since I want to implement our integration with v2 as soon as possible 😀

Support for mysql was requested here: https://github.com/zitadel/zitadel/issues/3600#issuecomment-1161246978

@hifabienne Considering you set priority to low, I'm guessing this this statement from two months ago is no longer applicable ?

Probably we will have a look at this in 1-2 months.

Thanks

At the moment we are doing the finishing touches for Postgres which should be ready really soon to be released.
After this we will consider what additional databases might make sense.

Oh, very nice, thanks !

What DB are you interested in?

Postgres is fine, I'll try it as soon as it's released.
Although MySQL is my primary database.

hi @Sytten and @KaKi87
We have just released postgresql support of ZITADEL. If you want to check it out you can find the release notes here: https://github.com/zitadel/zitadel/releases/tag/v2.3.0
If you have any feedback we are happy to answer :-)

Since PostgreSQL support of Zitadel is limited to the latest major version, I cannot use it, therefore I look forward to MySQL support. Thanks

Since PostgreSQL support of Zitadel is limited to the latest major version, I cannot use it, therefore I look forward to MySQL support. Thanks

I think it will be a while until we support mysql. Can you shine some light why it is a problem to use Postgresql 14?

Since PostgreSQL support of Zitadel is limited to the latest major version, I cannot use it, therefore I look forward to MySQL support. Thanks

I think it will be a while until we support mysql. Can you shine some light why it is a problem to use Postgresql 14?

question was answered here: https://discord.com/channels/927474939156643850/1014503578439794688/1015302346449883176

If you have any feedback we are happy to answer :-)

I'm not the ones, but give feedback :-) Also wanted to switch to Postgres after seeing Cockroach RAM usage. In Keycloak the app is the memory hog, in Zitadel it's Cockroach.
This works:
ZITADEL_DATABASE_COCKROACH_HOST=cockroach

... and this doesn't work:
ZITADEL_DATABASE_POSTGRES_HOST=postgres
ZITADEL_DATABASE_POSTGRES_HOST=5432
ZITADEL_DATABASE_POSTGRES_DATABASE=zitadel
ZITADEL_DATABASE_POSTGRES_USER_USERNAME=zitadel
ZITADEL_DATABASE_POSTGRES_USER_PASSWORD=password

postgres is the container name of Postgres.

hi @helmut72
I think there is some configuration missing. If you use postgres you have to set all the fields for the connection. The defaults of zitadel are only configured for cockroachdb:
here is an example for disabled ssl on postgres:
ZITADEL_DATABASE_POSTGRES_HOST=postgres
ZITADEL_DATABASE_POSTGRES_PORT=5432
ZITADEL_DATABASE_POSTGRES_DATABASE=zitadel

ZITADEL_DATABASE_POSTGRES_USER_USERNAME=zitadel
ZITADEL_DATABASE_POSTGRES_USER_PASSWORD=password
ZITADEL_DATABASE_POSTGRES_USER_SSL_MODE=disable

if you init the database also with zitadel you have to set the admin user additionaly:
ZITADEL_DATABASE_POSTGRES_ADMIN_USERNAME=root
ZITADEL_DATABASE_POSTGRES_ADMIN_PASSWORD=root
ZITADEL_DATABASE_POSTGRES_ADMIN_SSL_MODE=disable

@adlerhurst
Thanks, have added SSL Mode and tried with disable, disabled, false and 0. But no luck. It still wants to connect to a Cockroach db. Which is unset:
level=fatal msg="unable to initialize the database" caller="/home/runner/work/zitadel/zitadel/cmd/initialise/init.go:47" error="dial tcp <IP_ADDRESS>:26257: connect: connection refused"

We are currently working on ram usage of crdb.

I hope Postgres doesn't take this much RAM too... ;) Nevertheless I prefer Postgres for my home setup.

Update. Now it works. First I've pulled latest :stable release. My image was from end of last week. Still got an error:
level=fatal msg="unable to initialize the database" caller="/home/runner/work/zitadel/zitadel/cmd/initialise/init.go:65" error="ID=DATAB-0pIWD Message=Errors.Database.Connection.Failed Parent=(cannot parse `host=postgres port=5432 user= application_name=zitadel sslmode= sslrootcert=`: failed to configure TLS (sslmode is invalid))"

Then I've also set the Postgres admin variables:
ZITADEL_DATABASE_POSTGRES_ADMIN_USERNAME=zitadel
ZITADEL_DATABASE_POSTGRES_ADMIN_PASSWORD=zitadel
ZITADEL_DATABASE_POSTGRES_ADMIN_SSL_MODE=disable

But even then it doesn't worked:
level=fatal msg="unable to initialize the database" caller="/home/runner/work/zitadel/zitadel/cmd/initialise/init.go:65" error="ID=DATAB-0pIWD Message=Errors.Database.Connection.Failed Parent=(failed to connect to `host=postgres user=db_user database=`: server error (FATAL: database \"db_user\" does not exist (SQLSTATE 3D000)))"

Look at the database name. Same as Postgres user variable. But I have set a different db name. My examples here are other names for security reason.
Then I have tried with zitadel everywhere. Adminuser, Username, DB-Name and Password. Then it worked. It looks like the ENV parser need a fix with ZITADEL_DATABASE_POSTGRES_DATABASE option?
All at all it would be cool if the ENV parser use default values if any option is unset. I have this experience from other projects.

I hope Postgres doesn't take this much RAM too... ;)

Can answer my own question. Postgres use 38MB. For now. That's great. :)

@adlerhurst @fforootd Should we close this issue for now? We have added Postgres and I think for now we do not have the intention to add any other? Or  how do you see it?

MySQL ?

I would leave it open that we can track the state and put it at the end of the backlog.
For now we focus to improve the currently implemented sql dialects (cockroach, postgresql). Additional dialects add an additional layer of complexity for each change and maintenance overhead. To add more dialects there must be made ground work to simplify the addition of dialects.

I could use yugabytedb support. Based on postgresql, so should be 'relatively easy'.

Why not.
Although, I think MySQL is the most urgent one, as it's the most-used database.

That's right:
Black = mysql
Orange = postgres
Yellow = cockroachdb
Green = yugabytedb

I would leave it open for tracking the state and put it at the end of the backlog.
For now we focus to improve the currently implemented sql dialects (cockroach, postgresql). Additional dialects add an additional layer of complexity for each change and maintenance overhead. To add more dialects there must be made ground work to simplify the addition of dialects.

So maybe use an ORM like sqlboiler? But that is only a shot as I am not familiar with the code. Just interested on your opinion on this.

SQLite support would be great for compact setups on the edge, but also for testing locally.
Together with solutions like LiteFS SQLite is a great alternative for the "regular" databases.

I don't know if anyone need this, but I was able to run Zitadel with Postgres using this docker-compose.yaml:
version: '3.8'

services:
  zitadel:
    restart: 'always'
    networks:
      - 'zitadel'
    image: 'ghcr.io/zitadel/zitadel:stable'
    command: 'start-from-init --masterkey "MasterkeyNeedsToHave32Characters" --tlsMode disabled'
    environment:
      - 'ZITADEL_EXTERNALSECURE=false'
      - 'ZITADEL_DATABASE_POSTGRES_HOST=pgdb'
      - 'ZITADEL_DATABASE_POSTGRES_PORT=5432'
      - 'ZITADEL_DATABASE_POSTGRES_DATABASE=zitadeldb'
      - 'ZITADEL_DATABASE_POSTGRES_ADMIN_USERNAME=zitadeladmin'
      - 'ZITADEL_DATABASE_POSTGRES_ADMIN_PASSWORD=zitadeladmin'
      - 'ZITADEL_DATABASE_POSTGRES_ADMIN_SSL_MODE=disable'
      - 'ZITADEL_DATABASE_POSTGRES_USER_USERNAME=zitadeluser'
      - 'ZITADEL_DATABASE_POSTGRES_USER_PASSWORD=zitadeluser'
      - 'ZITADEL_DATABASE_POSTGRES_USER_SSL_MODE=disable'
    depends_on:
      pgdb:
        condition: 'service_healthy'
    ports:
      - '8080:8080'

  pgdb:
    restart: 'always'
    networks:
      - 'zitadel'
    image: 'postgres:16beta1'
    environment:
      - POSTGRES_USER=zitadeladmin
      - POSTGRES_PASSWORD=zitadeladmin
      - POSTGRES_DB=zitadeldb
    healthcheck:
      test: ["CMD-SHELL", "pg_isready -U postgres"]
      interval: '10s'
      timeout: '30s'
      retries: 5
      start_period: '20s'
    ports:
      - '9090:8080'
      - '5432:5432'

networks:
  zitadel:

I'd like to also leave a mark here as I stumbled across this issue and have a different viewpoint onto the problem as well.
Our company operates within the Financial and Insurance Sector. This sectors comes with some limitations that do limit a persons choice of a database drastically (and is probably the reason why you cleverly packaged it into Enterprise 🙈 ).

They do not especially like self-hosted databases
They nearly all use Microsoft (few on premise, more and more in the cloud, private or public)
They audit you/us/the database and the architecture around it

They like backup and restore
They like if its not self-made but from Microsoft and the kind
They like if its automated
They like encryption in transit and at rest
They like desaster recovery

Now, certainly there are services where you get Cockroach as a Service, but also this generally FIS customers do not really like, especially when the companies have 🇺🇸 -background (🇨🇭 bancs and 🇺🇸 have a special relation).
In the end, we are nearly forced, pushed / deeply motivated to take Azure-hosted databases as FIS customers trust Microsoft just more than any other startup/company that cooks their own soup.
This mindset, that large customers (and large customers bring large money) bring their own database (Cosmos, MySQL, Maria, Postgres or whatever Microsoft hosts) should be also kept in mind when evaluating this issue 🤔
Sadly, that is also why most of them and us can and won't use Zitadel Cloud. They want it in their own Azure Tenant.
Visionary thought: Given that you are 🇨🇭 based and maybe one day want to bark up the FIS sector as well (just dreaming) you would kind of start to fall into the Landing Zone Concept like we do and there, it would be (so I assume) highly encourage that you use one Microsoft DB's 🚀

That's a closed-mindedness issue, i.e. invalid from my point of view.

I'd like to also leave a mark here as I stumbled across this issue and have a different viewpoint onto the problem as well.
Our company operates within the Financial and Insurance Sector. This sectors comes with some limitations that do limit a persons choice of a database drastically (and is probably the reason why you cleverly packaged it into Enterprise 🙈 ).

They do not especially like self-hosted databases

They nearly all use Microsoft (few on premise, more and more in the cloud, private or public)

They audit you/us/the database and the architecture around it

They like backup and restore

They like if its not self-made but from Microsoft and the kind

They like if its automated

They like encryption in transit and at rest

They like desaster recovery

Now, certainly there are services where you get Cockroach as a Service, but also this generally FIS customers do not really like, especially when the companies have 🇺🇸 -background (🇨🇭 bancs and 🇺🇸 have a special relation).
In the end, we are nearly forced, pushed / deeply motivated to take Azure-hosted databases as FIS customers trust Microsoft just more than any other startup/company that cooks their own soup.
This mindset, that large customers (and large customers bring large money) bring their own database (Cosmos, MySQL, Maria, Postgres or whatever Microsoft hosts) should be also kept in mind when evaluating this issue 🤔
Sadly, that is also why most of them and us can and won't use Zitadel Cloud. They want it in their own Azure Tenant.
Visionary thought: Given that you are 🇨🇭 based and maybe one day want to bark up the FIS sector as well (just dreaming) you would kind of start to fall into the Landing Zone Concept like we do and there, it would be (so I assume) highly encourage that you use one Microsoft DB's 🚀

Thank you for this insightful write up 🙏.
Allow me a question, would it not be sufficient for most to run zitadel inside their azure tenant and use one of the managed Postgres offerings from azure? Or do these customers want a managed zitadel in their own tenant?
Because this is what we see people doing instead of using our cloud service.
(Postgres works in the OSS version already, just yesterday I deployed a zitadel with Google cloud SQL)
As a side note CRDB soon will offer Azure as well, but I do not know the details 😅.

That's a closed-mindedness issue, i.e. invalid from my point of view.

I am not sure I am getting this.
What is closed-minded?

We do not want that, our customers want that.

I know, but you should be convincing them to open themselves to what the rest of the world can offer, despite the additional efforts required, and eventually help them doing these.
Otherwise, we'll never stop GAFAM from ruling us.

🤝

Allow me a question, would it not be sufficient for most to run zitadel inside their azure tenant and use one of the managed Postgres offerings from azure? Or do these customers want a managed zitadel in their own tenant?

We see both. We have one customer asking for a fully managed service (where we actually struggle to find a good service-level agreement kind of because Privileged and Conditional Access to now their tenant). But we also have one customer that wants our solution deployed over a LZ but then manages it mostly themselves. We are also still finding our way through Microsoft and their LZ/Marketplace visions and papers but in the end the customers really just care about: Where is the data, who sees/processes it when and for how long and are they in charge of all the encryption keys or not.

Because this is what we see people doing instead of using our cloud service.

Yeah, like us.

Because this is what we see people doing instead of using our cloud service.

I am right now running terraform code to provision an Azure Flexible PSQL to do exactly that 🚀

As a side note CRDB soon will offer Azure as well, but I do not know the details 😅

This would be very interesting if they allow customer-managed encryption keys 🤔
@KaKi87 We do not want that, our customers want that. And they are regulated by authorities and the law so you get edged into a 🗳 in these situations. Of course you can self-host but then you have to answer every audit why exactly you chose GPG as your favourite key and who exactly can access the key you made on your local machine, when did you rotate it last (show the evidence you did) etc. etc. etc.
Update: You can have a look at this document from an expert in 🇨🇭 about that Does a cloud solution meet the requirements of a Swiss bank? to get an impression of the situation.

Yeah there will always be a need for regulated customers to take more control over their infra and data.
We had the idea once that we could deploy zitadel into customers infra through our customer portal (so basically we deploy to your tenant and provision all the things we need). However we postponed that since it not an easy task to achieve.
An alternative could be the marketplaces, however that is not easy/cheap either 😁
This is why we currently think having postgres and good support for k8s/knative is a great approach.

This is why we currently think having postgres and good support for k8s/knative is a great approach.

Sure! I just left my thoughts as notes in case you revisit the issue one day 🚀 And since it works since yesterday with AKS Flex PSQL we can cover most of the audited world ourselves already 💪

Thank you for this insightful write up 🙏.
Allow me a question, would it not be sufficient for most to run zitadel inside their azure tenant and use one of the managed Postgres offerings from azure? Or do these customers want a managed zitadel in their own tenant?

Short addition here @fforootd: We are one of those customers and currently cannot switch to Zitadel because of the missing MySQL support. I'd love to, but our existing infrastructure is all MySQL, including backups, security agreements, SLAs, legal stuff, management/escalation agreements and so on. Our team is too small/busy to get into longer conversations with the legal department/compliance department/etc  why we can't use existing resources, who will maintain Postgres in case of disaster, etc..
The hurdles to take might be out of control of the dev department and sometimes there is a lot of enterprise overhead in bigger structures. As I'd assume supporting MySQL should be just about adding some existing driver, I'd be really happy to see this feature and start switching over to Zitadel.

Any news on MySQL/TiDB support?

Any news on MySQL/TiDB support?

At the moment we do not have any news regarding support of mysql or any other databases.
A new database always brings additional complexity into the product, as not every DB works exactly the same. We can already see that with cockcroach and PostgreSQL.
Right now, our focus is on improving the current integrations, not adding more. However, this might change in the future. So we still hold the issue open to get all your feedback and opinion.

At the moment we do not have any news regarding support of mysql or any other databases. A new database always brings additional complexity into the product, as not every DB works exactly the same. We can already see that with cockcroach and PostgreSQL. Right now, our focus is on improving the current integrations, not adding more. However, this might change in the future. So we still hold the issue open to get all your feedback and opinion.

Maybe im missing something but why not to use some ORM?

We are a big enterprise company with several sister companies and only uses MYSQL, any plans on supporting this anytime soon as it is the only reason preventing us from using your solution?

We are a big enterprise company with several sister companies and only uses MYSQL

That doesn't make any sense.
It's like saying "we only use SOAP APIs powered by Java running on Ubuntu Server" : sure it's common but still a ridiculous meaningless limitation. 🤷

I think that cloud spanner would also be a good choice.
Of course especially for the google cloud users ;)
We are currently using spanner to store permission on a global scale.
We would like to do the same with our user identities ...
Spanner should behave quite similar to Cockroach so maybe porting isn't that hard compared to other types of databases.
Thanks

We are a big enterprise company with several sister companies and only uses MYSQL, any plans on supporting this anytime soon as it is the only reason preventing us from using your solution?

Nice to hear from you.
At the moment we have not planned mysql compatibility but I would encourage you to use a postgres/cockroachdb since we have a lot of experience with these storage layers.
The main reason to (not yet) support other storage layers is that we currently lack the engineering capacity to do this in a high quality 😁 I hope you understand this.

I think that cloud spanner would also be a good choice. Of course especially for the google cloud users ;)
We are currently using spanner to store permission on a global scale. We would like to do the same with our user identities ...
Spanner should behave quite similar to Cockroach so maybe porting isn't that hard compared to other types of databases.
Thanks

@adlerhurst and I where playing around with the postgres driver of spanner in the past but never finished it 😁
But certainly a postgres spanner optimization would be an interesting thing.

Indeed SQLite support or an other embedded database would be really great for smaller deployments.

Agree, if you were to implement sqlite support, I think you'd get a lot of traction.  A lightweight db would be much better for CI/CD for infrastructure.

@ext4xfs if I get 15 thumbs up I do it if they agree that they would accept a PR 😊

Indeed SQLite support or an other embedded database would be really great for smaller deployments.

+1 on this, even though my primary use-case would be simplified testing/setup in CI/CD.

@fforootd Any plans to support SQLite in the near future?

@fforootd would you accept a community PR for SQLite support?

@fforootd would you accept a community PR for SQLite support?

I can clarify with them team, what I want to make clear though this is not something we likely tend to support as part of our offering 😁
@adlerhurst what is your take, do you think it is even feasible?

We are currently working on database changes which could influence an implementation of another database quite heavily therefore i would wait with that feature until we gained some clarity there.
What i am concerned about is the maintenance overhead after the implementation. It's already time consuming maintaining crdb and postgres.

As an avid self-hoster, I would vouch for SQLite and even consider working on community PR. Postgres is a popular database but hard to maintain, as you mentioned in the docs. But getting crdb into self-hosted stack is even more challenging for those without prior knowledge.
On the other hand, with today's hardware, SQLite is more than enough for enthusiast projects and requires no maintenance at all. It even moves toward native horizontal scaling 1, so the possible higher availability scenario in 1-2 years can be just WAL-replicated SQLite.

so the possible higher availability scenario in 1-2 years can be just WAL-replicated SQLite.

A REPLICA is readonly. I don’t think that it works so easy.

Hi, guys! So no update on this?
I think the implementation of a lighter db like sqlite would make Zitadel an even more attractive option for home lab and smart home users, or for those who wanted a more contained choice than CRDB or PG.
I understand that it is not easy already to maintain two DBs, but I hope you can consider the idea.
Thank you for your valuable time, in any case

@adlerhurst @fforootd Could you inform us if the changes were made and if you could clarify if a community PR for SQLite would be accepted?


Bounds checking in Python <=3.5

There are a slew of xfail tests in both the rustapi_module and test_datetime.rs because the Python C API only started doing bounds checking on inputs in Python 3.6.
It may be that an exception is set in Python <=3.5 and it's just never actually raised, but I think more likely you get a malformed date/time object.
If no exception is set, I believe we should fix this by doing bounds checking ourselves on the Rust side (objects/datetime.rs), probably with a macro under a conditional compilation flag.

As we're set to drop Python 3.5 support soon (either 0.12 or 0.13) I'm marking this as wontfix and closing.


BG Process

Windows Terminal version (or Windows build number)
<IP_ADDRESS>
Other Software
Windows 11
Steps to reproduce
Update the Software
Expected Behavior
Need increase the PAyload.
Actual Behavior
Slow down in Load time if more request comes in.

HI


No module in unelevated prompt.

Syntax highlighting is not working on a normal PowerShell console, but working on an elevated one. Both point to same profile(which doesn't exist btw), there are no differences in task parameters.
Also, in the results of Get-Module, PSReadLine is only available in elevated console, not in default one.
I'm using ConEmu on Windows 10 Build 10586.
Anyone know why is this happening?

I can think of a couple possible explanations:

Your unelevated prompt is x86.  This is a known issue on Win10
Your unelevated prompt has an incorrect PSModulePath


Xboinc broken by updated Tracker structure

Should be solved by monifications just pushed to xboinc/main.
@davidedicroce don't hesitate to reopen in case you still see issues


Option to update the PR comment – improvements

I've rebased the PR https://github.com/zgosalvez/github-actions-report-lcov/pull/28 created by @oacik and applied some improvements to properly handle minimum-coverage check.
I've tested the changes in a test repository – https://github.com/bartosz347/github-actions-flutter-workflows/pull/1. The build is failing intentionally – to test minimum coverage requirement. I've tested the following scenarios:

Current version, minimum coverage requirement satisfied
Current version, minimum coverage requirement not satisfied
Updated version, minimum coverage requirement satisfied
Updated version, minimum coverage requirement not satisfied
Updated version, minimum coverage requirement satisfied, update-comment: false (regression)

In both 3. and 4. the comment https://github.com/bartosz347/github-actions-flutter-workflows/pull/1#issuecomment-1325483698 has been updated. In 5. new comment has been created.
If there's any better way to contribute to the previous PR, please let me know. Thanks @oacik for your work!

Hi @bartosz347! Curious why you didn't simply contribute to the original PR #28?

Hey @zgosalvez, that's what I've wanted to do at first, but I think I can't contribute to the existing PR – I'd have to push to its author's repository (I don't have RW access). Only maintainers are allowed to add new commits. Or maybe I'm wrong – please tell me how can I do this :)

Oh yeah, I forgot. 😅 Well, I did take a look at your PR and it appears to be somewhat unrelated to #28, right? #28 is simply about providing the option to update a comment instead of creating a new one. Whereas your PR here is an improvement on minimum coverage, correct?

Not exactly, #28 has a return statement which omits isFailure check (see https://github.com/zgosalvez/github-actions-report-lcov/pull/28/files#r935143815) – I believe minimum code coverage condition does not work properly in #28.

Hey @zgosalvez looks like something's wrong with Build & Test pipeline, it can't check out code from fork. Would you be able to fix this and merge my PR?

Would love to see this addition and fixes for v3!

Addressed in #85 🚀


ERR operation not permitted with "auth" command

Hi,
I use authentication on my Redis box and when I try to reach it using $redis->auth("My passowrd") it crashes with an Exception
Here is the code to reproduce
<?php
require("autoload.php");
$redis = new Predis\Client(array(
     'host'     => '<IP_ADDRESS>',
     'port'     => 6379,
     'database' => 15));
$result = $redis->auth("my password");
?>

And here is the result

PHP Warning:  Uncaught exception 'Predis\ServerException' with message 'ERR operation not permitted' in /usr/share/nginx/www/Core/predis/lib/Predis/Network/StreamConnection.php:257
Stack trace:
#0 /usr/share/nginx/www/Core/predis/lib/Predis/Network/ConnectionBase.php(142): Predis\Network\StreamConnection->read()
#1 /usr/share/nginx/www/Core/predis/lib/Predis/Network/StreamConnection.php(160): Predis\Network\ConnectionBase->readResponse(Object(Predis\Commands\ConnectionSelect))
#2 /usr/share/nginx/www/Core/predis/lib/Predis/Network/StreamConnection.php(135): Predis\Network\StreamConnection->sendInitializationCommands()
#3 /usr/share/nginx/www/Core/predis/lib/Predis/Network/ConnectionBase.php(197): Predis\Network\StreamConnection->connect()
#4 /usr/share/nginx/www/Core/predis/lib/Predis/Network/StreamConnection.php(172): Predis\Network\ConnectionBase->getResource()
#5 /usr/share/nginx/www/Core/predis/lib/Predis/Network/StreamConnection.php(285): Predis\Network\StreamConnection->writeBytes('*2??$4??AUTH??$...')
#6 /usr/share/ in /usr/share/nginx/www/Core/predis/lib/Predis/Network/StreamConnection.php on line 257

Did I miss something or is there a bug?
My password is long and complicated but even with a password like 12345 the issue is the same.
I tried to authenticate using
array(
	'host'     => '<IP_ADDRESS>',
	'port'     => 6379,
	'database' => 15,
	'password' => "my password")

and it works, nonetheless, I can't use auth as the first command just after a plain connection.
Thank you.

It may act weird but it's not a bug. If you look at the code that handles the password and database parameters you will notice that they are respectively translated to AUTH and SELECT commands, with AUTH having the precedence. Both commands are sent to the server transparently to the user as soon as the connection to Redis is opened.
If you omit password but specify database, Predis will send SELECT as the first command making it fail when connecting to a Redis instance protected by a password (which requires AUTH with a valid password as the first command).
Using password and database in the connection parameters or AUTH and SELECT as plain Redis commands is basically the same thing, but you shouldn't mix the two approaches:
$client = new Predis\Client(array("database" => 15, "password" => "my password"));

or
$client = new Predis\Client();
$client->auth("my password");
$client->select(15);

This makes perfectly sense.
Thank you for your time.
Sorry for the inconvenience.

Goods Job


Add Icons To Footer

Describe the feature
Hello Project Admin
I would like to add icons to Footer to make website look attractive
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@sailaja-adapa What all icons you want to add??

I would like to add icons like this

@sailaja-adapa Issue assigned to you

Ok Thanks @SUGAM-ARORA


Suggested Readme edits

Hi, in the readme, conda env create –name=BONITA --file platform_BONITA.yaml should be --name. I also noticed that the YAML file currently has windows line endings and has a user-specific prefix set, so it might be good to remove that prefix (or mention that the YAML might need to be edited).

Noted - will fix!

The readme was edited and a new version of platform_BONITA.yaml uploaded. Thanks @adam-cornwell !


Default values for struct fields

strawman syntax:
struct Foo {
    a: String,
    b: i32 = 42,
}

To instantiate:
let f = Foo { a: "Hello".to_owned() };  // b gets default value of 42
let f = Foo { a: "Hello".to_owned(), .. };  // alternative syntax to make default use explicit
let f = Foo { a: "Hello".to_owned(), b: 51 };  // b is 'overridden'

cc https://internals.rust-lang.org/t/struct-field-defaults/3412

I would much rather introduce .. as a sugar:
MyStruct {
    a: 2,
    .. // means:   .. Default::default()
}

Would the default be an arbitrary expression, executed at Foo creation time, or a constant expression?  The latter would make this a lot less useful than default().

At the very least the syntax for using the default should be explicit (IE: using ..), otherwise this feature could become slightly confusing.

Hi, I think .. should not be required to enable backward compatibility. Assume a library now has a struct with some fields, it would be possible to add another field with a default value without breaking client code that initializes only the two old fields.

There was a bit of discussion on this about a year ago here: https://www.reddit.com/r/rust/comments/2m99ly/idea_for_default_struct_fields/
Probably the most important thing about this proposal is that's it's providing more flexibility than just Default, as you can have default fields for a struct that isn't neccessarily default altogether.

It would be cool to provide both a const initializer (via default values) and a non-const one (via the Default). This is actually already possible via const functions, but it is a bit verbose IMHO. I think this would eliminate a lot of uses of lazy_static.
Example:
// In some library
pub struct Bar {
    ... // internals
}

// In my code
struct Foo {
    i: u64,
    b: Bar,
}

static f: Foo = Foo {
   i: 34,
   b: Default::default(), // Ooops! Cannot do this because `default` is non-const
};

In many cases, this leads to the use of a lazy_static, which may still be the right solution in a lot of cases. But the fact that we cannot use Default for static initializers is a bit unfortunate.

Also useful in a case like:
struct Point {
    pub x: f64,
    pub y: f64,
}

When later on you want to add a new field:
struct Point {
    pub x: f64,
    pub y: f64,
    norm: f64 = NAN,
}

impl Point {
   pub fn get_norm(&self) -> f64 {
        if(self.norm.is_nan()) {
            ...
        } else {
           self.norm
        }
   }
}

you don't have to change code across the whole project to add norm into every Point.

@kindlychung I think that mess with the goal of explicitness.

Whether support private fields？ I think  like this   code Foo { a: "Hello".to_owned(), .. }  will be  good way to initialize private fields.
Now， must do like this
Struct Foo {
          ...，
         pub __hide，
}
Foo { a: "Hello".to_owned(), ..Foo::default() };

Closing in favor of https://github.com/rust-lang/rfcs/pull/1806 for tracking purposes.

Hey what ended up happening to this? Would be super handle for the use case I have right now. Doesnt seam to have make it into code?

This RFC was closed in favour of 1806, which was closed as postponed, which means the lang team did not want to add the feature to the language at the time. I'm no longer on the lang team, but I believe this is not a current priority. It may be at some point.

There is a new pre-RFC at IRLO.


Add Arrays library with unit tests (#1209)

prepared due to snapshot token requirements
add library with method to find upper bound
add unit test for basic and edge cases

Blocks PR #1331
🚀 Description

Arrays library which is the requirement for ERC20 token with snapshots. Blocks PR #1331.

[+] 📘 I've reviewed the OpenZeppelin Contributor Guidelines
[+] ✅ I've added tests where applicable to test my new functionality.
[+] 📖 I've made sure that my contracts are well-documented.
[+] 🎨 I've run the JS/Solidity linters and fixed any issues (npm run lint:fix).

@frangio @nventuro hey guys! I'm not sure if does it stick to general concept but I tried. Let me know if you have any comments are ideas for missing test cases.

Nice, just want to thanks again for your help here 😄 I will go for #1331 now.


suggestion: Behance

https://www.behance.net/dev/api/brand#downloads

Added ffb8581f7f0230213f293040e61d7d342141a51f


Install Fails -<EMAIL_ADDRESS>is no longer on npm

One of the dependencies in react-native (currently looking into which one - will post back here when I figure it out) depends on<EMAIL_ADDRESS>15 minutes ago (as of the writing of this issue) left-pad updated to 0.0.9 and they seem to have removed 0.0.3 from npm: https://www.npmjs.com/package/left-pad
This results in the following error when simply running npm i react-native:
npm ERR! No compatible version found<EMAIL_ADDRESS>npm ERR! Valid install targets:
npm ERR! 0.0.9

:disappointed:

just to be clear this also means reactive-native init <projectName> fails as well

damn!!! the same issue

+1

+1

+1

@kauffecup Running into this issue w React 0.14, so it's something not specific to react-native

+1

+1

Kinda ironic considering the entire package is:
module.exports = leftpad;

function leftpad (str, len, ch) {
  str = String(str);
  var i = -1;
  ch || (ch = ' ');
  len = len - str.length;

  while (++i < len) {
    str = ch + str;
  }

  return str;
}

Makes you wonder what was in versions 0.0.4-0.0.8...

Kinda ironic this thing isn't important at all... Very funny, string padding destroyed the whole infrastructure. LoL. Does someone tried to downgrade their npm version?

It's such a simple library.

This issue is also discussed here, but a workaround is to add this to your package.json:
"left-pad": "git://github.com/azer/left-pad.git#bff80e3ef0db0bfaba7698606c4f623433d14355"

@dimitriwalters this should be fixed now by re-installing babel if you are on version 6

+1

+1

I ran npm-remote-ls react-native to print out the entire dependency tree... and it isn't in the resulting print. I'm not going to paste it here because it's massive, but ran for yourself and see what I mean. It's fairly confusing.

For everyone giving +1: https://github.com/blog/2119-add-reactions-to-pull-requests-issues-and-comments

+1

+1

+1

+1

+1

+1

npm install<EMAIL_ADDRESS>npm ERR! No compatible version found<EMAIL_ADDRESS>npm ERR! Valid install targets:
npm ERR! 0.0.9

Im getting this with react not react-native (If react has anything to do with the error).  Just updated npm/node this morning and was going to update all my deps

Author has pulled his packages from NPM. See<EMAIL_ADDRESS>

+1

Tracing backwards through the dependencies, Babel indirectly depends on left-pad. So basically that guy broke a ton of the JS ecosystem by indirectly breaking Babel (people reading this -- take note, probably not the best idea).
Once Babel republishes their packages as either a patch or minor version update, npm will automatically get the latest version of Babel with the fix.
Shrinkwrap wouldn't have helped because the author outright deleted the package from npm so, there's not much we could have done in this situation other than to statically ship the dependencies with react-native. Perhaps npm will come up with a better story around this -- ex: making it so that packages stay up for 24 hours and print a big warning about a pending removal if you try to install an unpublished version.

This hilarious :laughing: , but not.  :sob:

User @azer mentions in his post that he would be glad to hand over ownership of a module—I'm assuming that means it's also OK to republish the module on NPM. Since this is such an extremely simple module, perhaps someone could take it over and republish 0.0.3? That might help lots of people avoid the extra work to fix this.

I've published Babel 6.7.2 and 5.8.38. The ranges in package.json are fine but the shrinkwrap will need to be updated.

FYI: The library in question fits in a tweet (https://twitter.com/the_kenny/status/712414574658588672). Maybe people should finally start to think about what deserves to be an external dependency and what not.

npm republished that version: https://twitter.com/seldo/status/712414400808755200


The autocomplete misaligned when the width is limited

Describe the bug

When the terminal width is less than the width of the auto-completed command, the cursor gets misaligned.
Reproduce
Sorry for the vague description, I tried to convey the idea the best I can but this one is very difficult to explain. Please take a look at this GIF:

If I disable the autocomplete plugin, the problem goes away. Can someone help, please? Thanks.
Desktop

OS + distribution: macOS 10.14.6
Zsh version: 5.3
Plugin version: tag: v0.6.4

After reading this https://github.com/vercel/hyper/issues/283, I noticed my $PROMPT variable is invalid (missing some important format characters). I corrected it and the issue went away.


Add copyright to keras.py

Hi @popojargo
can you please add your name in a end copyright line after mine for copyright in this script: https://github.com/PhilipMay/mltb/blob/master/mltb/keras.py
So it looks like this:
# Copyright (c) 2020 by Philip May
# Copyright (c) 2020 by Alexis Côté
# This software is distributed under the terms of the BSD 2-Clause License.
# For details see the LICENSE file in the root directory.

If the contribution was not private but in context of your company please add
, <company> after your name.
Many thanks
Philip

done


GPT4 Support

It would be nice to be able to specify GPT4 for the model

openai_pipe version 0.1.3 installed fresh from gem
Anything I query I get the error Unable to fetch response: The model text-davinci-003 has been deprecated, learn more here: https://platform.openai.com/docs/deprecations
I see that this repo was updated 2 weeks ago but from what I can tell the current version in the master branch is also 0.1.3. Is this no longer functional or am I doing something wrong. If not, can the model be updated to gpt-4 or at least gpt-3.5-turbo? Better yet, can the model be configurable? Perhaps through an environment variable? That would be ideal. Just confused as this issue has been open almost a year and at least gpt-3.5 has been favored for quite a while. I don't know ruby or I would try to upgrade the model and make a PR.

openai_pipe version 0.1.3 installed fresh from gem Anything I query I get the error Unable to fetch response: The model text-davinci-003 has been deprecated, learn more here: https://platform.openai.com/docs/deprecations
I see that this repo was updated 2 weeks ago but from what I can tell the current version in the master branch is also 0.1.3. Is this no longer functional or am I doing something wrong. If not, can the model be updated to gpt-4 or at least gpt-3.5-turbo? Better yet, can the model be configurable? Perhaps through an environment variable? That would be ideal. Just confused as this issue has been open almost a year and at least gpt-3.5 has been favored for quite a while. I don't know ruby or I would try to upgrade the model and make a PR.

I will take a look at this, in the mean time try gem update quick_openai


FEATURE(designer): Implements a designer view to enter the cloud credentials

This commit adds a view to enter the credentials. It also contains code to generate a topology from a DAM, and to inject the credentials into a DAM. Thanks to @rosogon for the Credential Editor. This closes #183
Dependencies:
<script src="static/lib/angular-topology-editor/graph.js"></script>
<script src="static/lib/angular-topology-editor/canvas.js"></script>
<script src="static/lib/angular-topology-editor/types.js"></script>
<script src="static/lib/angular-topology-editor/forms.js"></script>
<script src="static/lib/angular-topology-editor/editor.js"></script>
<script src="static/lib/angular-topology-editor/status.js"></script>
<script src="static/lib/angular-topology-editor/credentials.js"></script>
<script src="static/lib/angular-topology-editor/angular-topology-editor.js"></script>

Usage:
<credentials-editor  bind="damFile"></credentials-editor>

Current coverage is 31.84%

Merging #212 into master will increase coverage by +0.01% as of c35b0ff

@@            master    #212   diff @@
======================================
  Files          329     329
  Stmts        12126   12126
  Branches      1534    1534
  Methods          0       0
======================================
+ Hit           3860    3862     +2
  Partial        406     406
+ Missed        7860    7858     -2

Review entire Coverage Diff as of c35b0ff

Powered by Codecov. Updated on successful CI builds.

Tested and working. Merging


"Coordinated replication action test" in testThaliReplicationPeerActionCoordinated.js does the  work even after it's finished

Our thali tape should not care about completed tests. We should be properly unbinded from socket after finish.
We need fix tests like testThaliReplicationPeerActionCoordinated in order to block any activity after teardown.

I am still working on this issue.
We have a problem: socket.io-client >= 1.4.6 just hangs, socket.io-client >= 1.4.0 <= 1.4.5 hangs after random period of time. socket.io-client <= 1.3.7 works fine.
Maybe this is related to jxcore, but I am not sure. I will try to run it with recent node later. For now we can use 1.3.7


Illuminate\Database\Query\Builder::get() stub

[ ] Added or updated tests
Documented user facing changes none

Changes
Adds stub function for Illuminate\Database\Query\Builder::get() to make the return type better: 'Collection of objects' instead of just 'Collection (of mixed)'.
Breaking changes
No breaking changes.

Tests?

This kind of fix should really be PR'd to the framework first to fix it at the source

Does Laravel accept list<string> specificity (instead of just array)? Is that general phpdoc enough? Or psalm-like enough?
(Wow, that's a lot of tests! 😆)

Should be fine, but you could also just use array<int, string>

Hi,
This looks good but it needs to have tests. You can add it in query-builder.php Also the stub needs to be added to other versions.

In Laravel soon 😮 https://github.com/laravel/framework/pull/53567 Good advice @calebdw

For this kind of changes we can use stub files. That way we can have the changes faster and also in other Laravel versions too (where it makes sense) I added it to the stubs. Thank you for the PR for the Laravel.


OnNavigatedToAsync receives empty parameter

I'm adding dynamically buttons to the MyHamburger inside Shell.xaml, based on user's input.
Basically the user inserts the name of a category and hits the add button, having the new category shown on the hamburger.
This new button should navigate to the detail page, unfortunately the OnNavigatedToAsync receives a new and dempty category as parameter and I don't understant why.
Here's some code:
Shell.xaml - addNewCategory method
var sp = new StackPanel
{
    Orientation = Orientation.Horizontal
};
sp.Children.Add(
    new SymbolIcon
    {
        Width = 48,
        Height = 48,
        Symbol = Symbol.Clock
    }
    );
sp.Children.Add(
    new TextBlock
    {
        Margin = new Thickness(12, 0, 0, 0),
        VerticalAlignment = VerticalAlignment.Center,
        Text = e.Name
    }
    );
var newButton = new HamburgerButtonInfo
{
    ClearHistory = false,
    PageType = typeof (CategoryPage),
	// Here e is 100% fine, category name is a valid name
    PageParameter = e,
    Content = sp
};
MyHamburgerMenu.IsOpen = true;
MyHamburgerMenu.PrimaryButtons.Add(newButton);

CategoryPageViewModel
public override async Task OnNavigatedToAsync(object parameter, NavigationMode mode, IDictionary<string, object> suspensionState)
{
    // Here parametere is a Category with Name = null as if it was a newly created object
    Category = suspensionState.ContainsKey(nameof(Category))
        ? (OtpCategory) suspensionState[nameof(Category)]
        : (OtpCategory) parameter;
    // Close the hamburger pane
    Shell.HamburgerMenu.IsOpen = false;
    await Task.CompletedTask;
}

Category model
public class Category
{
    #region Equals

    protected bool Equals(Category other)
    {
        return string.Equals(Name, other.Name);
    }

    public override bool Equals(object obj)
    {
        if (ReferenceEquals(null, obj)) return false;
        if (ReferenceEquals(this, obj)) return true;
        return obj.GetType() == GetType() && Equals((Category) obj);
    }

    public override int GetHashCode()
    {
        return Name?.GetHashCode() ?? 0;
    }

    public static bool operator ==(Category left, Category right)
    {
        return Equals(left, right);
    }

    public static bool operator !=(Category left, Category right)
    {
        return !Equals(left, right);
    }

    #endregion

    #region Properties

    /// <summary>
    ///     Category name
    /// </summary>
    public string Name { get; }

    /// <summary>
    ///     List of all the entries associated to this category.
    /// </summary>
    public List<Entry> Entries;

    #endregion

    #region ctor

    /// <summary>
    ///     Empty ctor for serialization
    /// </summary>
    public Category()
    {
    }

    /// <summary>
    ///     Basic ctor
    /// </summary>
    /// <param name="name"></param>
    public Category(string name)
    {
        Name = name;
        Entries = new List<Entry>();
    }

    #endregion
}

What am I doing wrong?
EDIT:
For some reasons I tried removing the empty constructor and it now works, is this the correct behavior?
EDIT 2:
After removing that constructor it works, but the other constructor gets called a huge number of times during navigation, and everytime it resets the Entries list which will always be empty when I load the page.
I feel like I'm missing the reason why the ctor is called so many times during navigation.

Sub-classing anything should have a constructor like:
class MyClass: MyBase {
    public MyClass() : base() { }
}

I'm sorry but I don't know why you're talking about sub-classing since Category doesn't extend anything.
By the way I've added that kind of constructor and now both Name and Entries are null after navigation, as if instead of passing the object instance as reference it calls the empty constructor everytime the page loads.
So I removed the empty constructor from both Category and Entry to see what happens, and it leads to

System.Exception: Your parameter must be serializable. If it isn't, then use SessionState.

Next step was to add DataContract to both Category and Entry to make them serializable, but now it won't even navigate from the hamburger and I really have no clue on what's going on :(


Correct corresponding files of canterbury and silesia corpus.

NFC: "plrabn12.txt" and "ptt5" belong to canterbury corpus.

Oops.  Good catch, thanks.


Bump version number to 1.0.0

Current coverage is 76.44% (diff: 100%)

Merging #49 into master will not change coverage

@@             master        #49   diff @@
==========================================
  Files            25         25
  Lines           972        972
  Methods           0          0
  Messages          0          0
  Branches        135        135
==========================================
  Hits            743        743
  Misses          185        185
  Partials         44         44

Powered by Codecov. Last update ee774ca...a124d7f


Sample: Issues with Antiforgery + Form + File Uploads

So, given the work to add size limits to form (HttpAbstractions#618) we're going to want the ability to configure and bypass these limits per-action/controller.
There's a few scenarios we have in mind here:

I have a restrictive limit set app wide to limit things by default to really small inputs. But, I want to increase the limit for a specific action. (maybe for a big form post or for file uploads)
I have an action that's going to accept a really large file upload - usually as an ajax action. I want to completely bypass the default buffering behavior of FormFeature and stream this directly to my storage.

We need a few specific things from MVC to support everything here we want to do:
[ ] - Make value providers lazy Mvc#4531 (prevent MVC from reading the form if you have no model binding)
[ ] - Create a filter/sample that removes the form value provider (prevent MVC from reading the form at all)
[ ] - Create a filter/sample that sets the limits on FormFeature (prevent antiforgery from reading the form while a 'low' limit is still in effect)

E2E Sample
Furthermore we want an end-to-end sample of using MVC to stream a large file upload. This will include a combination of the above, and using MultipartReader + Antiforgery Header Token which are already supported.

/cc @Tratcher

86147eda46a476b8acc409df0eaa9b0eca730e94


Support SoLiD POD

SoLiD is a new personal storage protocol that raised by Prof. Tim Berners-Lee, inventor of the World Wide Web.
Can draw.io adapt SoLiD POD in its storage layer?

Happy to accept PR to review it.