[web]: Blurry rendering at some resolutions

The following page renders extremely blurry when in fullscreen on my 1080p monitor. This is fixed when I use Chrome Dev Tools responsive mode.
Running on Chromium.

import 'package:flutter_web/material.dart';

class ConfirmPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Container(
          constraints: BoxConstraints(maxWidth: 500.0),
          child: Card(
            child: Column(
              mainAxisSize: MainAxisSize.min,
              children: <Widget>[
                DataTable(
                  columns: [
                    DataColumn(label: Text("Services")),
                    DataColumn(label: Text("Quantity")),
                    DataColumn(label: Text("Price")),
                    DataColumn(label: Text("Amount")),
                  ],
                  rows: [
                    DataRow(
                      cells: [
                        DataCell(Text("Consulting")),
                        DataCell(Text("1 hours")),
                        DataCell(Text("£30.00")),
                        DataCell(Text("£30.00")),
                      ],
                    ),
                    DataRow(
                      cells: [
                        DataCell(Text("Balance Due")),
                        DataCell(Text("")),
                        DataCell(Text("")),
                        DataCell(Text("£30.00")),
                      ],
                    )
                  ],
                ),
                RaisedButton(
                  child: Text("Continue"),
                  onPressed: () {},
                )
              ],
            ),
          ),
        ),
      ),
    );
  }
}

Any update on this? It's clearly a visual bug/glitch. I'm still having it with the latest Flutter build on master channel. I get it often on FlatButtons' text and icon but not all the time. It's like the position as decimals and the pixel alignment makes the blurry effet, like it's between 2 pixels.

Hi Nicolas, I'm getting the exact same issue as you on this one. For some reason all my icon and text widgets are appearing blurry like yours.

Having the same issue in a custom dialog, I found that if I round the position of the dialog itself, it fixes the problem! It's annoying to manage thought but it's a workaround.

Having the same issue in a custom dialog, I found that if I round the position of the dialog itself, it fixes the problem! It's annoying to manage thought but it's a workaround.

It's renders ok only on firefox for me.

We need fix for this

Fresh install of flutter for web and same issue here. Worth noting that I had flutter for web running a while ago and didn't notice this issue.

on flutter 1.10.15-pre.356 this problem not occur;
on the actual master this problem occurs;

Had the same issue with unsharp/blurry text rendering in chrome.
Even web support is still in tech preview, this is the only issue which prevents me of using flutter web in a production project.
The reason are decimal values in transform property of css style which results in blurry text:

So I created this workaround:
  class _AppState extends State<GeorefApp> {
MutationObserver _mutationObserver;

@override
initState() {
  _mutationObserver = MutationObserver((List<dynamic> mutations, MutationObserver observer) => _translateHack());

  WidgetsBinding.instance.addPostFrameCallback((_) => _translateHack());

  super.initState();
}

void _translateHack() async {
  List<Node> nodes = window.document.getElementsByTagName("*");

  for (Node node in nodes) {
    _mutationObserver.observe(node, childList: true);
    html.Element el = node as html.Element;
    if (el.style.transform.isEmpty) continue;
    if (!el.style.transform.contains("\.")) continue;
    //print(el.style.transform + " --> " + _normalizeTranslate(el.style.transform));
    el.style.transform = _normalizeTranslate(el.style.transform);
  }
}

String _normalizeTranslate(String value) {
  if (value.length > 12) {
    if (value.substring(0, 10) == "translate(") {
      String p = value.replaceFirst("translate(", "").replaceFirst(")", "").replaceAll("px", "");
      List<String> m = p.split(", ");
      return "translate(" + (double.parse(m[0]).toInt()).toString() + "px, " + (double.parse(m[1]).toInt()).toString() + "px)";
    } else if (value.substring(0, 12) == "translate3d(") {
      String p = value.replaceFirst("translate3d(", "").replaceFirst(")", "").replaceAll("px", "");
      List<String> m = p.split(", ");
      return "translate3d(" + (double.parse(m[0]).toInt()).toString() + "px, " + (double.parse(m[1]).toInt()).toString() + "px, " + double.parse(m[2]).toInt().toString() + "px)";
    }
  }
  return value;
}

what it does:
at every change of DOM it loops though it and removes decimal values in the transform prop of tags, so that only integer values remains.
Even though this solution is very very dirty I feel like a real hacker today :-)

I've tested the Bottom Navigation section on https://flutter.github.io/samples/#/ on different browsers, and it seems to be an issue specific to Chrome.
Edge

Firefox

Chrome

If you look at the "Comments" text on Chrome and compare it to other browsers, it is quite fuzzy compared to other browsers.
I'm surprised this issue is this open, as it seems like quite a severe issue to me, and that we have now recently gone Beta with Flutter Web, while this was reported in May!
Can we please get this fixed Flutter Team :-)

I've tested the Bottom Navigation section on https://flutter.github.io/samples/#/ on different browsers, and it seems to be an issue specific to Chrome.
Edge

Firefox

Chrome

If you look at the "Comments" text on Chrome and compare it to other browsers, it is quite fuzzy compared to other browsers.
I'm surprised this issue is this open, as it seems like quite a severe issue to me, and that we have now recently gone Beta with Flutter Web, while this was reported in May!
Can we please get this fixed Flutter Team :-)

another solution (only for profile and release builds) is to use
--dart-define=FLUTTER_WEB_USE_SKIA=true
unfortunately this does not work for debug builds

@moodstubos' observation is correct. This is caused by a misalignment of bitmap pixels w.r.t. physical screen pixels.

We  have the similar issue in our Flutter Web project and I also noticed that texts in Flutter Web/Chrome seems have grayscale anti-aliasing.
I suppose this is happening because canvas context used in Flutter for web supports transparency and being retrieved without alpha: false parameter.
As I see dart::html has only context2D property on Canvas class and does not allow to pass contextAttributes while retrieving context.
So I tried to make a tricky workaround for this by directly modifying dart2js output by adding alpha:false and got cleartype antialiasing for text but got many black rectangles on the screen too :)
@yjbanov, do you have any plans to support cleartype antialiasing in Flutter for web, maybe in skia backend?...

Same issue here 🙃

Me also facing same issue.

I am also facing the same issue.

@GioPan04 / @Jayrajghoniya / @DevarshRanpara: if you are also affected by this, please use the 👍emoji to give this issue more priority instead of adding a +1 / me too comment. Everyone subscribed to the issue will get notified with a rather useless message otherwise. Check https://github.com/flutter/flutter/wiki/Issue-hygiene#thumbs-up-reactions on how issues are prioritized.
Thanks and have a great day :)

another solution (only for profile and release builds) is to use
--dart-define=FLUTTER_WEB_USE_SKIA=true
unfortunately this does not work for debug builds

After doing this, it seems that the Chinese characters cannot be displayed.

another solution (only for profile and release builds) is to use
--dart-define=FLUTTER_WEB_USE_SKIA=true
unfortunately this does not work for debug builds

I assume SKIA means the Skia Graphics Library .While this remove the blur in chrome, It also disable the Unicode support. Skia Graphics Library in default has some issues with the Unicode characters

I had the issue as well and just tried again. On Mac: Chrome 79.0.3945.88 and Flutter 1.12.13+hotfix.6 (beta).
Font is no longer blurry. No idea if this is now correctly fixed.

I had the issue as well and just tried again. On Mac: Chrome 79.0.3945.88 and Flutter 1.12.13+hotfix.6 (beta).
Font is no longer blurry. No idea if this is now correctly fixed.

Same versions here . Still chrome is blurry

After doing this, it seems that the Chinese characters cannot be displayed.

@aiqinxuancai
Skia backend is highly experimental
currently, it is not a solution for the blurry font rendering at all.

@aiqinxuancai The issue with fonts you are seeing is tracked in https://github.com/flutter/flutter/issues/45507.

Update: after looking at this it turns out we have blurriness issues in plain HTML too, not just the bitmap canvas.
Here's a small repro case: https://jsfiddle.net/yjbanov/9tjv5fuL/13/embedded/result
I only see this in Chrome. Firefox does not look blurry. I filed a Chrome bug (https://bugs.chromium.org/p/chromium/issues/detail?id=1040222, for some reason it was marked as non-public though, dunno why).
In the meantime, I will try to implement a workaround specifically for Chrome running on device pixel ratio 1.0 screens.

@yjbanov The same problem happened to me, and without Flutter too.
Here is the site: https://nogravitysound.altervista.org/music.php?id=1
On Chrome when you click the download icon, a popup opens, but the text is blurry. On Firefox, it's all normal.

@yjbanov I am still seeing this for custom fonts; should I create a new issue?

What's the real solution?

I think the issue is connected with Flutter engine flexibility. You can easily draw a line or a text at decimal coordinates (e.g 0.5) and you'll get the blurry line/text. Indirect positioning widgets like Center could easily place widgets at decimal coordinates. Flutter should provide mechanism of snapping/rounding coordinates to int values on demand. On mobile devices with high pixel density it's not an issue but on web/desktop it will always be. Microsoft faced the same issue in UWP apps at the beginning but they introduced some sort of snapping and eliminated the problem.

chromium Version 84.0.4147.105 (Official Build) Arch Linux (64-bit) :

chromium --force-device-scale-factor=1

a lot better.

The following workaround/hack seems to improve font/image sharpness for my own case and the original issue at the top.
Add this CSS rule to your html:
flt-glass-pane {
    image-rendering: pixelated;
}

The following workaround/hack seems to improve font/image sharpness for my own case and the original issue at the top.
Add this CSS rule to your html:
flt-glass-pane {
    image-rendering: pixelated;
}

worked best for me on the p element, thanks for the hack.

wrapping Text, TextField, SelectibleText and even the Icon widget with Material widget makes them look decent but still the something is off

Any news on that issue?

The following workaround/hack seems to improve font/image sharpness for my own case and the original issue at the top.
Add this CSS rule to your html:
flt-glass-pane {
    image-rendering: pixelated;
}

worked best for me on the p element, thanks for the hack.
(note this is just a short-term solution if you need to quickly polish something)

where did you add this? I add in web/index.html following snip

but didn't work

The following workaround/hack seems to improve font/image sharpness for my own case and the original issue at the top.
Add this CSS rule to your html:
flt-glass-pane {
    image-rendering: pixelated;
}

worked best for me on the p element, thanks for the hack.
(note this is just a short-term solution if you need to quickly polish something)

where did you add this? I add in web/index.html following snip

but didn't work

Well it does work well for most of the rendering, texts and lines mostly, it's a loss for images as you lose the anti-aliasing (bluring) when scaling an image. So for me, it was better for displaying images without the fix since I use bigger images that I down-scale and even one smaller image that is up-scaled (I know it's bad but don't really have control on that one!). So a real fix would be better in my opinion...

The issue is closed but even using the latest stable build, I still have the issue. Is it supposed to be fixed in 1.22.4?

where did you add this? I add in web/index.html following snip

but didn't work

The snippet should be in the <head> block. If it doesn't work, for some reason, you might have different element names. Perhaps a version changed some things, but you can adapt the style rule to match different elements.
How i figured out the element was by opening the website in Chrome, opening dev tools, then selecting the canvas element where flutter renders its output. Then I tried adding the style image-rendering: pixelated on it and a few elements surrounding it. Multiple elements worked for me, while some didn't.

If you only want text to have image-rendering: pixelated, you can also use:
<style> flt-glass-pane * p { image-rendering: pixelated; } </style>

in your header.

So is this issue fixed yet?

@Mehul-Bisht There's been a fix but as I commented before, it may fix text rendering but not image displayed. To me, it's not fixed right now.

To me it's also not fixed. Still blurry/non-sharp images.

You need to use SKIA engine. flutter build web --dart-define=FLUTTER_WEB_USE_SKIA=true

You need to use the SKIA engine. flutter build web --dart-define=FLUTTER_WEB_USE_SKIA=true

Unfortunately it didn't fix it for me.

It works only in the release mode (It won't work in the debug mode). Flutter is using the SKIA rendering on mobile by default but you can also enable it on the web.

It works only in the release mode (It won't work in the debug mode). Flutter is using the SKIA rendering on mobile by default but you can also enable it on the web.

Is there a way to check if the website is actually using the SKIA engine?
Unfortunately I still haven't been lucky. I also tried the tips from Stackoverflow (I did clean the workspace using flutter) which didn't work either. On mobile it's totally fine. Just the desktop version is affected.

@cocos2dbeginner You can check the network tab in the developer console. If CanvasKit is downloaded, you know Skia is used.

@creativecreatorormaybenot @michalkubizna Canvaskit is being used, but it's still not rendering correctly :/


Mobilenet-NNIE

Hisense Nie, transferred to mobilenet V2, reported an error in the optimization stage, did not understand, and RESNET did not report an error,Who succeeded? Let's talk about it
谁成功了，交流一下海思NNIE 部署mobilenet v2

您好，您的邮件我已收到

我已收到你的邮件，谢谢


room.Send() should throw an error if the send failed, rather than returning silently

Hi @rnd256, thanks for reporting as always.
Forgive my ignorance but in which situation would room.Send() fail? I can only think of operating on a disconnected WebSocket, if that's what you're referring to we should make it throw an error indeed!

Hi @endel, thanks for the fast response as always :)
That's exactly what I'm referring to! Here are the two spots where I think an issue could occur:

https://github.com/colyseus/colyseus-unity3d/blob/4472b4db8ba39a7705d001b96b86d18cee3218ea/Assets/Colyseus/Runtime/WebSocket/WebSocket.cs#L485
https://github.com/colyseus/colyseus-unity3d/blob/4472b4db8ba39a7705d001b96b86d18cee3218ea/Assets/Colyseus/Runtime/WebSocket/WebSocket.cs#L492

+1
I don't know if a disconnection detected by the Colyseus client will always be detected by the server, but having a confirmation that a disconnection did happen client-side will help with problem diagnosis greatly, and identify such cases.


ARROW-3383: [Gandiva][Java] Fix java build

set arrow as parent project, use same style as arrow java
fix lots of checkctyle violations
add java build/tests to CI

Just rebased, so the build should pass now (unless we hit ARROW-3343)

Does this have any conflict with https://github.com/apache/arrow/pull/2717? I should probably merge that first

No conflict with #2717 but it’s likely that will show up checkstyle errors (this PR switches to arrow checkstyle) in the new java code. Please merge that one first. I’ll fix the checkstyle errors.

OK, merged #2717

I had forgotten to push the Gandiva commits to apache/master. Should be good now

@wesm can you please merge now ?

Yep! merging now


fix : Unclear test output use-array.map-to-dynamically-render-elements

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #38170

Hi, @ojeytonwilliams @raisedadead @RandellDawson I think I didn't have correct forked codebase and it's very complex to analyze this because of my work on the master branch so, I have created this new pull request hope it will work. Please close this Pr https://github.com/freeCodeCamp/freeCodeCamp/pull/38172

@kundan59 You still need to resolve the issue mentioned in the comment of the other PR.  I closed the other since you deleted the branch.

Hi @raisedadead Travis support clear that of my flagged account. could you please push force to tigger travis.

Great, thanks for confirming. I think you should be able to force push yourself to that the CI will work correctly:
git checkout <you pr branch name>
git pull --rebase upstream master
git push --force

Hi, @raisedadead I am a bit confused locally npm run lint successful. but at Travis ignore this. Do you have any idea?

So, Travis is testing the Arabic challenges (which fail) since it's missing repository environment variables.  I pushed a copy to my fork and saw:

Setting environment variables from repository settings
$ export LOCALE=english
$ export TEST_CHALLENGES_FOR_LANGS=english

but that doesn't appear on @kundan59's.
Compare my copy of this branch https://travis-ci.com/ojeytonwilliams/freeCodeCamp/jobs/286248778
to https://travis-ci.com/kundan59/freeCodeCamp/jobs/286231989
I'm not sure why this is happening, but that seems to be the problem.

@ojeytonwilliams right these two environment variables of repository settings are missing. But how this happened?

Hi, @raisedadead @ojeytonwilliams when I provide these environment variables externally, Travis successfully build this job.

@kundan59
That is build that you have on your fork. It's optional and not required by us. Those variables do not affect this PR. The CI checks are done on our own account, but your user account was flagged by Travis for some reason, which prevented us from running the builds.
Please follow the instructions that I gave you in the previous comment and rebase. That should be enough from your side.

@raisedadead I am done with push force as you mention above let see whether it work or not.

@kundan59 I think you account is still flagged by Travis for some reason:
List of build requests from freeCodeCamp:

Corresponding commit:

OK - I have sent a request to travis support. I will update on the status.


Creating data file for code.

Dear Dr. Dan,
I read your Ph.D. thesis “Towards a Universal Modeling and Control Framework for Soft Robots”, inspired by your work and podcast “ How to do grad school”, I wanted to do similar work for my better understanding of Koopman Operator and how to implement the algorithms you have mentioned in your thesis for system identification.
I am using data from a soft robot and the data file looks similar to your “arm-3link-markers-noload-50trials_train-10_val-5.mat” file but the issue I am running into is the model prediction is not accurate. So I was wondering what possibly could I be doing wrong.
Given below is the structure of my data file:
Training data:
X = [  x( : , : ) , y( : , :) , z( : , : )  ] % x,y,z coordinates of points along the robots body
Y = [ x( : , end ) , y( : , end ) , z( : , end ) ] % trying to track the tip of the robot.
U = all the torques that are acting on the robot ( torques are constant during the simulation for different points )
T = time .
Similarly, I create a data set for validation using new data the model was not trained before. I am not sure where I am going wrong?
Also when I identify the linear system by setting a delay of 1, I get ud1 , ud2 , ud3 ..... as the basis function. During the simulation of the identified linear model, what will be the initial condition corresponding to these delay input functions ?
Given below are the setup parameters that I am using :
Ksysid = Ksysid( data4sysid ,...
'model_type' , 'linear' ,...    % model type (linear, bilinear, or nonlinear)
'time_type' , 'discrete' , ...  % 'discrete' or 'continuous'
'obs_type' , { 'poly' } ,...    % type of basis functions
'obs_degree' , [1] ,...       % "degree" of basis functions
'snapshots' , Inf ,...          % Number of snapshot pairs
'lasso' , [ Inf ] ,...            % L1 regularization term (Inf for least-squares sol.)
'delays' , 1 ,...               % Numer of state/input delays
'loaded' , false ,...           % Does system include external loads?
'dim_red' , false);             % Should dimensional reduction be performed?

Thank you for the response, Dr. Dan. Your suggestions were very helpful and they did improve the accuracy of the model a lot.
Can you please confirm if the variable z0 at line no. 1666 of Ksysid.m is the initial condition for the identified linear model and can be used when as an initial condition when simulating the identified linear model ?

Yes z0 should be the initial condition for the system in the lifted space.
Best,
Dan

Thank you for the confirmation Dr. Dan


Show gateway type next to the gateway name when adding a new payment provider

This is confusing. Ideally we should also change the name but just in case we should also display the unique type like square_up or stripe_connect.

Released in v1.27.0


Network errors when installing hellow world

If I try
cargo run -r  -- install -r hello_world

I get a timeout:
        ERROR run_install:bake:bake_inner:run_bake: brioche_core::bake: error=Request error: error sending request for url (https://registry.brioche.dev/v0/blobs/85b898d9dda3158822f5a0c25e71bc5dceb83c06620b4a0f7bf40370a362556f.zst?brioche=0.1.0): error sending request for url (https://registry.brioche.dev/v0/blobs/85b898d9dda3158822f5a0c25e71bc5dceb83c06620b4a0f7bf40370a362556f.zst?brioche=0.1.0): operation timed out scope=Project { project_hash: ProjectHash(Hash("d0aeff98c15b839e2d99e04c023e84414c48e249eb6e2c695696685c42e3d355")), export: "default" } recipe_hash=39cf103288f270b8c774b4d9c5d5b4d10fff20c5da3faf8a0e2721257cc4773e recipe_kind=Process recipe_hash=39cf103288f270b8c774b4d9c5d5b4d10fff20c5da3faf8a0e2721257cc4773e recipe_kind=Process

If I manually bump in-code timeouts to 60s from 10s, I then get some error about "temporary DNS failure" (sadly, lost the exact text of error somewhere in the git history)

Actually, now that I think about it, I only have Fly.io instances near Seattle, maybe I just need to expand to more Fly.io regions...

I am in Lisbon. I can curl the file manually, but it feels very slow for 4 kilobytes (about 10 seconds actually).

Okay, I've come up with a few possible explanations, and I've also made a few changes to try and address them. Let me try and summarize:

One theory I had was that the roundtrip time to the Seattle Fly.io instances was the problem. I've reduced the total number of instances but distributed them over more regions, so if that was the cause, then that should hopefully help
Another theory was that the roundtrip time to Cloudflare R2 on the US west coast was the problem. I needed to do some reconfiguration, but the bucket itself is now sitting behind Cloudflare's cache, so that should help too
My final theory is just that the Fly.io startup time takes so long that requests will time out. I currently have all the instances set to scale to 0, and they can take a few seconds to spin back up. I'm currently paying the bill out of pocket, so if this is the problem I think the best option would be to just increase the timeouts, unfortunately...

Could you try curling the URL from before again, ideally twice in a row? (the first time will likely be a cold boot, and the second should then hit the already-running instance)

If it seems like things have improved, then the final test would be trying the hello world installation from scratch again (also with some extra debugging for good measure):

Run chmod -R +w ~/.local/share/brioche && rm -rf ~/.local/share/brioche to remove all the locally-stored files
Re-run the install command with env vars: BRIOCHE_LOG_OUTPUT='./brioche.log' BRIOCHE_LOG_DEBUG='[]=debug' brioche install -r hello_world

If that still fails, then brioche.log should at least give some insights

Could you try curling the URL from before again, ideally twice in a row? (the first time will likely be a cold boot, and the second should then hit the already-running instance)

Yup, the first time around it took a minute, the second one was fast. I guess it might make sense to bump default timeouts to something like 120 seconds, rather than just 10? 10 is a reasonable number for the steady state, but with cold boots, network topology changes and what not, I think P100 could go higher than that.

Oh yeah, it sounds like this was resolved by #54


Failed to create file appender on UWP

Hi!
I found a little bug on FileTarget.
NLog version: 4.5.11
Platform: UniversalWindows (1803/17134, Windows10 64bit)
Current NLog config: I did it by programable way just below.
            FileTarget fileTarget = new FileTarget("fileTarget")
            {
                Encoding = Encoding.UTF8,
                Layout = "${date:format=yyyy-MM-dd HH\\:mm\\:ss.fff}\t${level}\t${logger}\t${message}\t${exception:format=toString,Data:maxInnerExceptionLevel=10}",
                FileName = @"${var:LogPath}\XXX\log-${date:format=yyyy-MM-dd}.log",
                WriteBom = false,
                LineEnding = LineEndingMode.CRLF,
                KeepFileOpen = true,
                ConcurrentWrites = true,
                OptimizeBufferReuse = true,
                CreateDirs = true
            };
            config.AddTarget(fileTarget);

What is the current result?

No Log is logged. After making InnerLogger on, I found exceptions.

What is the expected result?

Make it run and log.

Did you checked the Internal log?

Yes, I did.

Please post full exception details (message, stacktrace, inner exceptions)

NLog: 2018-12-28 10:18:40.1169 Warn Failed to create file appender: C:\Users\sakamoto\AppData\Local\Packages\~~~~~~~\LocalState\log-2018-12-28.log Exception: System.UnauthorizedAccessException: Access to the path 'Global\NLog-FileFileLock-c:_users_sakamoto_appdata_local_packages_~~~~~_localstate_log-2018-12-28.log' is denied.
   at System.IO.__Error.WinIOError(Int32 errorCode, String str)
   at System.Threading.Mutex.MutexTryCodeHelper.MutexTryCode(Object userData)
   at System.Runtime.CompilerServices.RuntimeHelpers.ExecuteCodeWithGuaranteedCleanup(TryCode code, CleanupCode backoutCode, Object userData)
   at System.Threading.Mutex.CreateMutexWithGuaranteedCleanup(Boolean initiallyOwned, String name, Boolean& createdNew, SECURITY_ATTRIBUTES secAttrs)
   at System.Threading.Mutex..ctor(Boolean initiallyOwned, String name, Boolean& createdNew, MutexSecurity mutexSecurity)
   at NLog.Internal.FileAppenders.BaseMutexFileAppender.CreateSharableMutex(String mutexNamePrefix)
   at NLog.Internal.FileAppenders.MutexMultiProcessFileAppender..ctor(String fileName, ICreateFileParameters parameters)
   at NLog.Internal.FileAppenders.MutexMultiProcessFileAppender.Factory.NLog.Internal.FileAppenders.IFileAppenderFactory.Open(String fileName, ICreateFileParameters parameters)
   at NLog.Internal.FileAppenders.FileAppenderCache.CreateAppender(String fileName, Int32 freeSpot)

Are there any workarounds? yes

After few hours of try-and error,
Just turn off 'ConcurrentWrites', exception is gone and logs are logged.
And I read source code of BaseMutexFileAppender.cs, I fownd some strings that appears in inner target log.
https://github.com/NLog/NLog/blob/dev/src/NLog/Internal/FileAppenders/BaseMutexFileAppender.cs
        private string GetMutexName(string mutexNamePrefix)
        {
            const string mutexNameFormatString = @"Global\NLog-File{0}-{1}";
            const int maxMutexNameLength = 260;

As far as my concerned, on UWP, Mutex path must be on permitted directory path.
But under 'ConcurrentWrites = true' configuration, it is not configuable.
And not permitted (C:)Global\NLog-File~ to write.

Is there a version in which it did work?

I have no idea.

Can you help us by writing an unit test?

Sorry, I can't.
I hope this poor english report may be understood.

Glad you found the issue. Have create a PR for changing the default value of ConcurrentWrites for NetStandard: #3048

After few hours of try-and error,
Just turn off 'ConcurrentWrites', exception is gone and logs are logged.

As a small recap, this isn't working in UWP due to limitations.

resolved with #3079

FYI, NLog 4.6 is online which includes a fix fir this


Fp 88 fe complete redux setup for main data sources

Description
State was restructured with Redux Toolkit to more efficiently store and access state. Slices of state were added for clients and for the current user.
Loom Video
Please paste a link here of a quick loom video walking through what you did in your PR .
Type of change

[X] New feature (non-breaking change which adds functionality)
[X] This change requires a documentation update

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have removed unnecessary comments/console logs from my code
[ ] I have made corresponding changes to the documentation if necessary (optional)
[X] My changes generate no new warnings
[X] I have checked my code and corrected any misspellings
[X] No duplicate code left within changed files
[X] Size of pull request kept to a minimum
[X] Pull request description clearly describes changes made & motivations for said changes

For anyone needing documentation on how we're using Redux Toolkit, use this video: https://www.youtube.com/watch?v=bbkBuqC1rU4&t=2019s&ab_channel=freeCodeCamp.org


levelTimeout is 5s not 500ms

class SimpleDHT {
protected:
    long levelTimeout = 5000000; // 500ms

I just noticed that the comment in the file SimpleDHT.h says "500ms" for the variable long levelTimeout = 5000000;. The number works out to be 5 seconds, not 500ms. I don't know what value is better, so either the comment should be changed to "5 seconds" or the value should be changed to 500000.
Even better, the value should be customizable using another method, like setTimeout, similar to other Arduino libraries.

I think 500ms is enough for a timeout.


Test actions installing missing dependencies

Ref - https://github.com/Microsoft/vscode/issues/70546

[x] AnyOS @chrisdias
[x] AnyOS @kieferrm

Complexity: 2

Install any extension that has dependencies for example ms-azuretools.vscode-cosmosdb
Open extensions folder and delete the extension it depends on ms-vscode.azure-account
Reload the window and open Azure Explorer in the side bar
Make sure that an error with an action Install and Reload pops up with missing dependency message. Clicking on Install and Reload action should install the missing dependency and reloads the window.
There is also a command Extensions: Install Missing Dependencies, you can run to install all missing dependencies. Test running this command after step 3 and make sure missing dependencies are installed.

Tested on macOS and Windows.

Tested on Linux, overall looks good.


Not compatible with the latest version of serverless

There are lot's of breaking changes in the new version of serverless released yesterday which is what is causing this.
Maybe the CORS-functionality in this plugin could be achieved with the new templates introduced in serverless 0.1.0?
https://github.com/serverless/serverless/releases
/User/X/.node/lib/node_modules/serverless/node_modules/bluebird/js/main/async.js:43
        fn = function () { throw arg; };
                           ^

TypeError: Cannot read property 'region' of undefined
    at ServerlessCors.addPreflightRequests (/User/X/Code/chatter/node_modules/serverless-cors-plugin/src/index.js:68:29)
From previous event:
    at /User/X/.node/lib/node_modules/serverless/lib/Serverless.js:177:25
    at Array.reduce (native)
    at Serverless._execute (/User/X/.node/lib/node_modules/serverless/lib/Serverless.js:176:26)
    at Object.actions.(anonymous function) [as endpointDeploy] (/User/X/.node/lib/node_modules/serverless/lib/Serverless.js:378:20)
    at /User/X/.node/lib/node_modules/serverless/lib/actions/DashDeploy.js:288:34
From previous event:
    at DashDeploy._deploy (/User/X/.node/lib/node_modules/serverless/lib/actions/DashDeploy.js:283:10)
From previous event:
    at DashDeploy.dashDeploy (/User/X/.node/lib/node_modules/serverless/lib/actions/DashDeploy.js:132:10)
From previous event:
    at /User/X/.node/lib/node_modules/serverless/lib/Serverless.js:152:29
    at Array.reduce (native)
    at /User/X/.node/lib/node_modules/serverless/lib/Serverless.js:151:30
    at processImmediate [as _immediateCallback] (timers.js:384:17)
From previous event:
    at Serverless._execute (/User/X/.node/lib/node_modules/serverless/lib/Serverless.js:149:10)
    at Serverless.actions.(anonymous function) (/User/X/.node/lib/node_modules/serverless/lib/Serverless.js:378:20)
    at Serverless.command (/User/X/.node/lib/node_modules/serverless/lib/Serverless.js:347:38)
    at Object.<anonymous> (/User/X/.node/lib/node_modules/serverless/bin/serverless:16:12)
    at Module._compile (module.js:399:26)
    at Object.Module._extensions..js (module.js:406:10)
    at Module.load (module.js:345:32)
    at Function.Module._load (module.js:302:12)
    at Function.Module.runMain (module.js:431:10)
    at startup (node.js:141:18)
    at node.js:977:3

Thanx for reporting! I'll be releasing a new version of the plugin in a few days!

The plugin is now compatible with the brand new release of Serverless (v0.2)!

Thanks for updating this!


FileHandle: fix readInBackgroundAndNotify for Win32

DispatchIO.read(fromFileDescriptor:maxLength:runningHandlerOn:) is not
viable on Windows as the fileDescriptor is not available as we cannot
convert the handle to a file descriptor.  Extract the operation into a
local closure and dispatch from the right function in DispatchIO.

@swift-ci please test


add unit test cases for convertAssignRows

ref: https://github.com/cectc/dbpack/issues/
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #84 (1cfc983) into dev (1eeb6d2) will increase coverage by 2.70%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev      #84      +/-   ##
==========================================
+ Coverage   38.70%   41.40%   +2.70%
==========================================
  Files          47       47
  Lines        5774     5774
==========================================
+ Hits         2235     2391     +156
+ Misses       3339     3163     -176
- Partials      200      220      +20

Impacted Files
Coverage Δ

pkg/misc/utils.go
69.18% <0.00%> (+20.28%)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM

It looks more elegant if put private functions after public ones.  😺

LGTM

It looks more elegant if put private functions after public ones. 😺

it's done


One line summary of state of gamedev in Rust

As per https://www.reddit.com/r/rust/comments/519m7r/arewegameyet_is_online/d7bhjhj

Thanks to @norru for the one liner


Create a Virtual Server Resource

Problem: A Virtual Server class is required to manage BIG-IP Virtual
Server resources.
Solution: Create a Virtual Server class and unit tests.

I like how simple this is.


Performance: pause / resume rendering

At the moment, the renderer will always draw incoming data (manipulate the dom), no matter if the terminal is actually visible to the user. Bigger applications like hyper or vscode, that may render multiple terminals in tabs would benefit a lot (performance and battery drain) if they could suspend the rendering while the terminal element is visually hidden. I think we should offer an API that allows the implementor to pause and resume the renderer.
term.pauseRenderer();
term.resumeRenderer();
term.rendererPaused;

Once the renderer is resumed, it has to immediately redraw to catch up the new state.
Any thoughts? I would be tempted to provide a PR 😉

This would be amazing for my use case. I'm ready doing something similar for the resizing: i resize just the focused terminal, and when the user switches to another tab it updates . Thia would be an awesome addition to improve it! :tada:
Maybe this could be an addon as not all use cases would benefit/need this?
+1

@LucaT1 I think the API has to go into core because it needs to touch the internals of the renderer - but we could write a plugin that uses the IntersectionObserver to automatically pause and resume the renderer using the new API 🤓.

I don't think this is as big of a problem as it first sounds like as in general, at least for me, background tabs aren't doing much and therefore aren't refreshing much. Seems like a good area for investigation though.
I'm not much of a fan of the low level API, but it would be really cool if we just handled everything if the browser supports IntersectionObserver. I wonder if registered IntersectionObservers doing a whole bunch of work whenever scrolling happens that could negatively impact performance as well?

@Tyriar In my case, I manage a lot of servers, having many tabs (ssh connections) open to them, and when troubleshooting I have to do a lot of tail -f logfile in multiple tabs.
With a fullscreen terminal and five fullscreen tabs running tail -f busy-log-file.log you can feel performance degrading quite a lot, which made me come to this idea. Note that there is quite a difference in performance between running five terminals split on the same screen, or if they were running all in separate tabs. Because on the same screen they still only occupy all the pixels on the screen once, but with one tab active and four fullscreen tabs in the background, they occupy 5* screen size (and cols and rows).
There was a nice post from one of the authors of the atom editor a while ago, that describes how they moved to use IntersectionObserver and ResizeObserver (here are the few lines of code that do it: https://github.com/atom/atom/blob/4b2a26e74f8290f7206e6ca5706fb2a51bc04200/src/text-editor-component.js#L1344). Btw. the ResizeObserver is now enabled in stable Chrome 60 by default and we could also use it for the fit addon!
Performance wise I can only guess, but since it is somewhat baked into the browser's rendering pipeline I would assume it should be a really minimal impact.
Last but not least, xterm this: telnet mapscii.me

This is quite interesting.
I think that before deciding to move on with this, we should have some benchmarks or at least some proof that such a change will have a performance or energy consumption impact.
Now as far as considering the API, I would be happy if we exposed a limited public renderer API.
Example:
let term = new Terminal(...);

term.renderer.status;
term.renderer.pause();
term.renderer.resume();

My biggest implementation question is how do we store what is the last state that got rendered from the renderer and from where should we start rendering again.

My biggest implementation question is how do we store what is the last state that got rendered from the renderer and from where should we start rendering again.

The last state of the renderer will still be in the DOM, all this change would be in its simplest form is some boolean flag which is checked by the renderer and the renderer does nothing. resume would flip the switch back and trigger a full refresh.

I'll create a proof-of-concept PR with a modified demo that creates multiple terminals that get filled with data continuously and an option to enable / disable the renderer for every single terminal. That way we should be able to inspect if performance is positively impacted.

@mofux I'd love to have a proper set of benchmarks checked in that we could run from the command line. I had a go at it earlier using phantomjs https://github.com/Tyriar/xterm.js/tree/phantomjs but I didn't end up polishing it and then phantomjs was abandoned in favor of headless chrome. If you want to explore that, it's an area that needs help too 😃


Selecting a monster and going back, immediately pushes the same monster detail again.

Thanks for reporting! I can’t reproduce this unfortunately. What branch/app version and OS version are you using?

App Store version (latest, presumably); iOS 14.0

Thanks. This is one of the things that broke on iOS 14, I have yet to push an Xcode 12 build to the App Store. main is much better on iOS 14 but needs a few last refinements. You could try the latest TestFlight build if this issue is a deal breaker.

I believe this has been resolved with version 1.1, which was released yesterday. Feel free to reopen the issue if it persists.

Yup, it's fixed!


Update azure-netapp-files-create-volumes-smb.md

Somehow "Share permissions" section was missing from the document and it should remain the same on this page as they need to map the share to a drive to give NTFS permissions.

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

#label:"aq-pr-triaged"

@KazTamai - "Shared Permissions" subsection was intentionally removed. We are not planning to add it back.  Per our SME, ANF shows NTFS permissions of the mapped drive, not the share permissions.  Please see the following ANF FAQ

Can the SMB share permissions be changed?
No, the share permissions cannot be changed. However, the NTFS permissions of the root volume can be changed using the NTFS file and folder permissions procedure.

If you have further questions, please reach out through<EMAIL_ADDRESS>and I can loop you in for a discussion with the SMEs.

Please closing this PR.  Thanks.
#please-close


workspace.applyEdit allows 'concurrent' calls to succeed

In Stable, when calling applyEdit multiple times before the first one is complete, subsequent calls are rejected, until the first edit is completes. This allows you to make the assertion that if an applyEdit succeeds, there should be one, and only one, onDidChangeTextDocument change event containing the edits that were applied.
This is not true in Insiders -- both applyEdit calls will succeed, and there will be multiple onDidChangeTextDocument events raised.
Attached is an example extension that has one command -- minibash.concurrentBash that will show the repro.
Expected Debug console output w/ Stable:
1: Applying Edit: 0,0, text: 'O'
2: Applying Edit: 0,0, text: 'DRENG'
Doc changed with 1 changes
1: Applied Edits
2: Applied Edits
2: Failed to apply edits
Failed

Output w/ Insiders:
1: Applying Edit: 0,0, text: 'M'
2: Applying Edit: 0,0, text: 'RRGAOCOQ'
Doc changed with 1 changes
Doc changed with 1 changes
1: Applied Edits
2: Applied Edits
saw multiple edits
saw multiple edits
Failed

To run this:

open the extension sources, and F5
create a new empty document (or open existing one)
Run Minibash: Concurrently Bash
Minibash.zip

Note, that if you remove the return false; on line 69 in doEditsUntilBad, you'll see that Insiders will happly apply 1000 edits. Stable does not; it immediately fails on the second edit (As expected)

@grork Thank you for the repro.
I can reproduce. This issue was introduced with https://github.com/Microsoft/vscode/commit/54a6ec8f588c766349e5d36891d0bd13b8673462#diff-ffd7342f51e8e5d64bad8622e6f5a48e
The gist of it is that the version id is checked and then asynchronously the edits are applied. We should check the version id right before applying the edits...

Thanks @alexandrudima


Default item, bundle types?

Since we are providing a pre-trained model that uses PlanetScope imagery (item type = PSScene), specifically from the 4-band surface relfectance product bundle (bundle_type = analytic_sr_udm2), I am proposing that we make these the default options in the search and download functions.
This will mean that for a user that just wants to do our standard SCA mapping, they don't have to think about providing these details. But allows for advanced users to use our library to search and download for different products if they're developing their own model.

bundle type, from which we want to select the four-band surface relfectance bundle analytic_sr_udm2
item type, specifying spacecraft and/or processing level. For the SCA mapping model, we will need PlanetScope imagery, which is identified with an item type of PSScene

This has been addressed with #36


Improve Homepage Design and Content

The current homepage of our Travel Guide website requires some enhancements to improve user experience and engagement. Below are some areas that need attention:

Header Section:

The introductory text needs to be more captivating and concise to immediately grab the user's attention.
Consider adding visually appealing elements such as images or videos to make the header more engaging.

Services/Features Section:

The services/features section lacks visual hierarchy and could benefit from better organization.
Explore ways to highlight key services and features more effectively to attract user interest.

Testimonials Section:

We need to gather more testimonials from satisfied customers to showcase the credibility and reliability of our platform.
Consider reaching out to recent customers for feedback and testimonials to include on the homepage.

Famous Places/Blogs/Articles Section:

Currently, the section is placeholder text. We need to populate it with actual content such as famous travel destinations, blog posts, or articles.
Identify relevant content that aligns with our target audience's interests and provides value to visitors.

Footer Section:

The social media links in the footer are static. We should dynamically fetch the latest posts or updates from our social media profiles to keep users engaged.
Review the navigation links and ensure they are accurate and lead users to relevant pages within our website.

@jaiyankargupta #2 is same issue, I'm closing itm just add comment on issue #2 if their are any updates.


test wake can fail in GPU

Example.
        # check loss went down
        print("target loss: ", target_loss)
        print("initial loss: ", init_loss)
        print("trained loss: ", trained_loss)
        diff0 = init_loss - target_loss
        diff1 = trained_loss - target_loss
        if torch.cuda.is_available():
>           assert diff1 < (diff0 * 0.5)
E           AssertionError: assert tensor(7.4708, device='cuda:0', grad_fn=<SubBackward0>) < (tensor(5.3381, device='cuda:0', grad_fn=<SubBackward0>) * 0.5)

test_wake.py:114: AssertionError

Let's revisit this once we work with wake phase again (probably after first paper)


Can we help?

We (the e-commerce team at InfluxData) wrote a ConfigCat client for Elixir a few months ago.  It's currently embedded within our one of our (closed-source) apps, but I was literally just about to create a new repo to extract and open source our work when I found this repo.
I don't see any code here yet, but we're wondering how far along you are with development. Can we combine forces to make a single, official Elixir client for ConfigCat?

Hey,
I just created the repository, we don't have any code. We don't have any experiences with Elixir, but we could help you if you have any questions. It would be awesome if you could push your code so we could review and check it. I gave you write permissions for this repo.

Sounds good! It'll probably be a couple of days before I can get the code extracted and pushed up, but I will get it done.  We have a fully-functioning client that we've been running in production for a couple of months now; there's a few features missing that we didn't need, but we were planning to add some of those as part of our open-sourcing effort.

I'm deeply sorry, I missed the part that you are from the InfluxData team.
Actually one of our other customers told us that they would help us in creating this SDK, and I thought they wrote to us here.
I'm glad that both of you could help us. Let's combine the forces and make a great official Elixir SDK!
I've created a elixir-sdk channel on our community slack channel. I think there we could talk about the details:
https://join.slack.com/t/configcat-community/shared_invite/enQtMzkwNDg3MjQ3MTUyLWY1ZmE0NjY5NjA0YjAzMWU3MDg3ODhkMGI2ZjdmZjZmYTY4YTg1NmQ0YzBhMTVhOGE5YmNiYTI5OTg4NmNjYTQ

Hey guys! That's great!
We (Drover) were organizing ourselves to develop a new one based on the Ruby SDK. If you guys at InfluxData already have something already running on production would be could if we could start from where you guys are at the moment 👍

This is what we had planned:

Not sure how far you guys are at the moment. We based the work on the guidelines @laliconfigcat provided via e-mail a few days back.

I've given you write permissions too for this repo @igorescobar.
Please send me who should I give write permissions from your sides.


Error if specified locale attribute

Hi @indrimuska ,
I tried to specify the locale attribute in my input but it gives me this error.
TypeError: Cannot read property 'longDateFormat' of null
ng-model="vm.scheduleDate" moment-picker="vm.scheduleDate" these are the relevant parameters of the input. Notice that if I do not specify the locale attribute, the error isn't fired, but then if I change moment locale with moment.locale("it"), it changes only the name of the months and not the name of the week days.
Why? Am I missing something?
Simone

Hi @simonemazzoni, maybe there's something wrong in your settings 'cause it seems to work properly to me.
<input moment-picker="myDate" format="YYYY-MM-DD" locale="en">

Can you please share a working example of your configuration?
Thanks :)

I have exactly the same problem, locale="en" works fine but "dk", "sv", "it" doesn't work for me.

I'm going to close this issue since no answers has been provided.

@indrimuska yes, I'm sorry about this, but I hadn't time to investigate and I had to change library because of this bug and another issue about the responsiveness of the layout of the component.

The issue should be reopened, after 2 days of investigation wondering why the behavior of your component was buggy i finally nailed it !
There is a bug with the double binding on ng-model when locale is set to anything OTHER THAN "en" you can try for example "fr" !
Moreover you need to remove the format="YYYY-MM-DD" property that some how forces back the component to locale="en" thus preventing the error.
Very easy to reproduce :

open your plunker https://embed.plnkr.co/hs10SM
change locale="en" to locale="fr"
remove format="YYYY-MM-DD"
click on "set to today" you'll see that it set date to undefined or null

This makes the plugin impossible to use for any other language than English, and i hope that with this steps to reproduce it you'll be able to patch it (I tryed to look at your code but didn't manage to find any hint of solution).
Since the issue is closed and pretty old i'm going to open a new one on this


Update typedoc to 0.20.X

An update regarding this issue -
the new typedoc requires the source and the associated tsconfig file in order to compile the documentation correctly. This prevents us from compiling the documentation pages directly in this repository.
Will keep the current version to stay consistent, but in the future we will need to compile the API documentation as part of the core repository's CI process.


Document that KeyCode::Capital is the Caps Lock key

Problem

I was trying to find what the KeyCode for Caps Lock was and was very confused

Solution

changed the documentation of KeyCode::Capital

This will be obsoleted by the winit migration. Good changes but I'm closing this out, sorry!


"fastlane run resign" do not use the new cert id but the old cert id to resign the ipa

New Issue Checklist

[ ] Updated fastlane to the latest version
[X] I read the Contribution Guidelines
[X] I read docs.fastlane.tools
[X] I searched for existing GitHub issues

Issue Description
Since our enterprise distribution certificate will be expired soon, we installed a new enterprise distribution certificate with the same name into Mac.  After that, we have 2 distribution certificates in the machine.
When we use "fastlane run resign" with cert name as signing-identity name, the old enterprise distribution certificate id is used to resign ipa but not using the new one. It caused that the ipa cannot be installed by end user.
Command executed
fastlane run cert
fastlane run sigh
** fastlane run resign **
Complete output when running fastlane, including the stack trace and command used

Environment

Fastlane version: 2.206.2
Mac OS version: Big Sur 11.6.7
Xcode version: 12.1, 12.4, 13.1 and 13.2.1

Fastlane ENV
🚫 fastlane environment 🚫
Stack

Key
Value

OS
11.6.7

Ruby
2.6.3

Bundler?
false

Git
git version 2.27.0

Installation Source
/usr/local/bin/fastlane

Host
macOS 11.6.7 (20G630)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib

OpenSSL Version
LibreSSL 2.8.3

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode11_7.app/Contents/Developer/

Xcode Version
11.7

Swift Version
5.2.4

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.206.2
🚫 Update available

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.6.1

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.8.0

terminal-table
1.8.0

plist
3.6.0

addressable
2.8.0

multipart-post
2.0.0

word_wrap
1.0.0

optparse
0.1.1

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

artifactory
3.0.15

babosa
1.0.4

colored
1.2

highline
2.0.3

commander
4.6.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.4

faraday-cookie_jar
0.0.7

faraday-em_http
1.0.0

faraday-em_synchrony
1.0.0

faraday-excon
1.1.0

faraday-httpclient
1.0.1

faraday-net_http
1.0.1

faraday-net_http_persistent
1.2.0

faraday-patron
1.0.0

faraday-rack
1.0.0

ruby2_keywords
0.0.5

faraday
1.8.0

faraday_middleware
1.2.0

fastimage
2.2.6

gh_inspector
1.1.3

json
2.1.0

mini_magick
4.11.0

naturally
2.2.1

rubyzip
2.3.2

security
0.1.3

xcpretty-travis-formatter
1.0.1

dotenv
2.7.6

simctl
1.6.8

emoji_regex
3.2.3

rexml
3.2.5

nanaimo
0.3.0

colored2
3.1.2

claide
1.1.0

CFPropertyList
3.0.5

atomos
0.1.3

xcodeproj
1.22.0

public_suffix
4.0.7

excon
0.92.3

bundler
2.3.16

jwt
2.4.1

webrick
1.7.0

httpclient
2.8.3

multi_json
1.15.0

signet
0.16.1

os
1.1.4

memoist
0.16.2

googleauth
1.1.3

mini_mime
1.1.2

retriable
3.1.2

trailblazer-option
0.1.2

declarative
0.0.20

uber
0.1.0

representable
3.2.0

google-apis-core
0.6.0

google-apis-playcustomapp_v1
0.9.0

google-apis-androidpublisher_v3
0.22.0

rake
13.0.6

digest-crc
0.6.4

google-apis-storage_v1
0.15.0

google-apis-iamcredentials_v1
0.12.0

google-cloud-errors
1.2.0

google-cloud-env
1.6.0

google-cloud-core
1.6.0

google-cloud-storage
1.36.2

aws-eventstream
1.2.0

aws-sigv4
1.5.0

aws-partitions
1.600.0

jmespath
1.6.1

aws-sdk-core
3.131.2

aws-sdk-kms
1.57.0

aws-sdk-s3
1.114.0

forwardable
1.3.2

logger
1.5.1

date
2.0.0

stringio
0.0.2

openssl
2.1.2

ipaddr
1.2.4

zlib
1.0.0

mutex_m
0.1.1

connection_pool
2.2.5

net-http-persistent
4.0.1

ostruct
0.5.5

strscan
1.0.0

io-console
0.4.7

fileutils
1.6.0

libxml-ruby
3.2.1

psych
3.1.0

Is there any update on this?
I think this is is unexpected behavior.
Since "fastlane run sigh" used the new code signing id to generate the provisioning profile, "fastlane run resign" used the old code signing id to resign the ipa.

We are facing the same issue. Is there any update on this?

Same problem still present.
When creating a new certificate and creating the new provisioning from it, fastlane seems to still take the old certificate and the app stops working on the date the old provisioning would have expired.
The strange thing is that when using the ProvisionQL plugin, the app appears to be correctly renewed and expiring on the new date.
Currently to avoid the problem just delete the old certificate from the Mac before resigning the .ipa.

This issue still happens on latest version of fastlane.  I have to either...

Delete the old certificate (after backing up to file)
Move the old certificate to a backup keychain, then hide said keychain (as fastlane will still try and use it in that one)

Basically it appears that the --keychain_path flag doesn't do anything


Remove unused Dockerfile and add Snyk dependency

Use concourse docker file for local development
We were using the old Jenkins Dockerfile before, which had a very old version of Node so gave different results when installing dependencies when compared to the Concourse Dockerfile.
Add Snyk dependency and prepare command
This cherry-picks the commits from https://github.com/alphagov/notifications-node-client/pull/117 but adds an extra commit to fix the Dockerfile and package.json so that they work.

What would be a good way to review this PR / check the changes work as expected?

To test that nothing breaks you can try building the Dockerfile locally - that should be fine though because I have tried that locally. I think the main thing I'd like an opinion on is the approach taken in the last commit.

Hey, nothing breaks, I do not know enough to comment on the command change. But if it works, it's something, right?
Also, there are those two commits test and wip, and the latter has some commands to execute as sudo, is this temporary?

I decided to take another look at this, which is why there are currently a couple of temporary commits at the end (I've added [WIP] to the title to make that clear now). The original PR did work but is a bit hacky - I'd like to find a nicer way of getting it to work

Ahh, ok, sounds good, ganbatte! 💖


Fix yarn run dev

Pull Request Test Coverage Report for Build 1907

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.844%

Totals

Change from base Build 2257:
0.0%

Covered Lines:
4162

Relevant Lines:
4811

💛  - Coveralls


Use a static ConnectionMultiplexer instead of creating one each time

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

#sign-off


Change title of main banner

The title of the banner on every page now is Service status. We would like to change that to something else. Can that be possible in the settings?

You mean the one next to the logo right? Right now it is not possible but this is a great idea and I'm going to do it soon.


change PAGE_SIZE of pagination in contract tabs

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind ui-change
/kind api-change
/kind backend-change
kind ui-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
to let the three tabs to be same PAGE_SIZE:15
Which issue(s) this PR fixes:
issue #1255

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

 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.


Key reform

This fixes #184 because of which we couldn't access certain keys. The KeyState struct is renamed to Key and has fields code and printable that correspond to libtcod's vk and c.
This also adds two convenience fields: alt and ctrl that are true if the left or right variant is pressed.
This is a breaking change. To update your code, convert it to match on the Key struct. I.e. replace this:
match keypress.key {
    Special(Enter) if keypress.left_alt => { // fullscreen }
    Special(Escape) => { // exit game }
    Key::Printable('>') => { // descend }
}

with this:
match keypress {
    Key { code: Enter, alt: true, .. } => { // fullscreen }
    Key { code: Escape, .. } => { // exit game }
    Key { printable: '>', .. } => { // descend }
}

I like it. My only complaint with the current design is that matching an event is ridiculous:
match event {
    Some(_, Event::Key(Key { code: Enter, .. })) => {},
}

That' s a lot of noise for nothing, but I don't think it's solvable in current Rust. Ideally I'd like to be able to do something like this:
match event {
    Some(_, Event::Key { code: Enter, .. }) => {},
}

This could be done if we declared Key inside the Event struct, but unfortunately it's used in console as well (Which is the recommended method of getting events, console or input?).

Yeah, that is a bit silly. I wonder if the upcoming struct & enum unification could help here (haven't followed it too thoroughly yet).
If you're just getting keyboard input, console is the place to go, if you want mouse or both, then input. At least that's how I understand libtcod. It being a roguelike toolkit, mouse is sidelined a bit... Especially since it's more painful to match generic events instead of just the key ones.

Well, other than that it seems good to me. Once it's merged I'll go and see if returning a dedicated Event type would make matching a bit easier (instead of the current tuple of (EventFlags, Event).


Cursor implementation

Hi mybatis dev team,
Here's an up to date version of a minimalist Cursor implementation. This PR replaces the previous implementation I made in PR 52 (https://github.com/mybatis/mybatis-3/pull/52).
I bumped the version to 3.4.0-SNAPSHOT as SqlSession gets new methods.
This PR adds the Cursor feature which is needed to implement a MyBatisCursorItemReader for Spring Batch.
Features

Added new SqlSession methods selectCursor()
Cursors extends Iterable to be easily used with foreach Statement and implements Closeable to be used with try-with-resources in Java 7+
Cursors are automatically closed when fully fetched or on SqlSession close

A Cursor does not retain any fetched element, so it is a perfect choice for iterating over a ResultSet with several millions rows, without exhausting memory.
Usage :
XML mapping :
    <select id="getEmployeeNestedCursor" resultMap="results" resultOrdered="true">
       select id,name,salary,skill from employees order by id
    </select>

Java code :
// Get a list which can have millions of rows
Cursor<Employee> employees = sqlSession.selectCursor("getEmployeeNestedCursor");

// Get an iterator on employees
Iterator<Employee> iter = employees.iterator();

List<Employee> smallChunk = new ArrayList(10);
while (iter.hasNext()) {
    // Fetch next 10 employees
    for(int i = 0; i<10 && iter.hasNext(); i++) {
        smallChunk.add(iter.next());
    }
    doSomethingWithAlreadyFetchedEmployees(smallChunk);
    smallChunk.clear();
}

Best regards,
Guillaume

Its been a long time @gdarmont but thank you very much for looking again into this.

Hi @emacarron  @gdarmont,
https://github.com/mybatis/mybatis-3/blob/master/src/main/java/org/apache/ibatis/cursor/defaults/DefaultCursor.java#L102-L104
    protected T fetchNextUsingRowBound() {
        T result = fetchNextObjectFromDatabase();
        while (currentIndex < rowBounds.getOffset()) {
            result = fetchNextObjectFromDatabase();
        }
        return result;
    }

Logic to skip records until offset is this ?
I think this logic(way to skip) should be discuss to improve.
How do everyone think ?

What happens when you call again?
Iterator<?> it = cursor.iterator();

cursor.close(); // first call
cursor.close(); // ??? second call

it.hasNext(); // ??? after cursor close()

Nose smell that fetchNextObjectFromDatabase method does not check the value of the attribute "opened".

Hi all,
Thanks for the feedback. I admit I did not make tests to handle corner cases, and as you mentioned there may be problems (several close() calls, not enough rows etc...).
This part can easily be fixed. I will push fixes and corresponding tests ASAP.
I think it is worth explaining the origin of Cursor :
The need of a MyBatis Cursor comes from the need of having a SpringBatch ItemReader that can handle several millions items with only one request, without exhausting memory. Currently the MyBatisPagingItemReader make one request per chunk (this can be very time consuming for requests on large datasets) and can get wrong results if paging is done on request with nested resultmaps (see tests in https://github.com/gdarmont/mybatis-spring/blob/943797d2fd036efa293fdb65a8b76794d5c2105d/src/test/java/org/mybatis/spring/batch/SpringBatchTest.java)
Requirement : SQL requests used with selectCursor() needs to be ordered (resultOrdered="true" in mapping), to allow intermediate result objects to be released (https://github.com/mybatis/mybatis-3/blob/master/src/main/java/org/apache/ibatis/executor/resultset/DefaultResultSetHandler.java#L771-L773), otherwise it wouldn't be very memory efficient for very large resultset. Result ordering is the only constraint that makes selectCursor() different from selectList().
RowBound part : The only reason I integrated a selectCursor() method taking a RowBound is to have a one to one Cursor equivalent to selectList().
But the semantic is not the same, and as such, it maybe a better solution to simply remove the selectCursor(String,Object,RowBound) method for now. Or introduce a new type of RowBound.
Currently a RowBound unit (= value of 1) in MyBatis is mapped to a row in a ResultSet (a database row).
The RowBound unit used in Cursor is mapped to a "toplevel" object (a List or Cursor element, mapped from 1 or more database rows). That'w why we may need a new object (ObjectBound ?)
The loop part is used to mimic the offset advance used in DefaultResultSetHandler : https://github.com/mybatis/mybatis-3/blob/master/src/main/java/org/apache/ibatis/executor/resultset/DefaultResultSetHandler.java#L348-350 . And since MyBatis can transform several rows into one "toplevel" object, I don't currently see how it could be more efficient. Of course, this triggers a lot of garbage since objects are created but never used.
But it seems to me that RowBound in general is only here to ease life on few situations (since it can lead to wrong results, as seen above when we use nested collections mappings). A developer with performance concerns will craft the SQL request to only compute the needed data, and so, won't have the need of the RowBound feature.
To sum up, here's a small todo list :

Fixes and tests for the fixes
Discuss about the RowBound pertinence on selectCursor() and if a new RowBound (ObjectBound ? name is open to discussion) type is needed. IMO, I think it would be better to introduce an ObjectBound. This feature could be then extended to other parts of MyBatis.

@alexey-su For the nested SQL queries, the current code simply rely on the core mapping features of MyBatis and as such, should handle them seamlessly. Do you have a reproduce test so that I can check on my side ?
Hope this helps.
Guillaume

我用的3.4.2 cursor好像不起啥作用


Adjust the order of events and record elapsed time when async invoke.

Motivation:
Adjust the order of events and record elapsed time when async invoke.
Modification:

ClientEndInvokeEvent will not post before async invoke return real response.
Adjust the order of events when async invoke.
Record elapsed time when async invoke.

Result:
Fixes #125

Codecov Report

Merging #126 into master will increase coverage by 0.21%.
The diff coverage is 76.08%.

@@             Coverage Diff              @@
##             master     #126      +/-   ##
============================================
+ Coverage     65.12%   65.33%   +0.21%
- Complexity      652      656       +4
============================================
  Files           271      272       +1
  Lines         11618    11620       +2
  Branches       1957     1955       -2
============================================
+ Hits           7566     7592      +26
+ Misses         3080     3066      -14
+ Partials        972      962      -10

Impacted Files
Coverage Δ
Complexity Δ

.../java/com/alipay/sofa/rpc/common/RpcConstants.java
66.66% <ø> (ø)
0 <0> (ø)
:arrow_down:

...y/sofa/rpc/transport/bolt/BoltClientTransport.java
69.42% <ø> (-0.2%)
31 <0> (ø)

...va/com/alipay/sofa/rpc/client/AbstractCluster.java
65.14% <100%> (+0.22%)
0 <0> (ø)
:arrow_down:

...om/alipay/sofa/rpc/context/RpcInternalContext.java
81.57% <100%> (+18.1%)
0 <0> (ø)
:arrow_down:

...com/alipay/sofa/rpc/client/ClientProxyInvoker.java
83.33% <33.33%> (-2.88%)
0 <0> (ø)

...pay/sofa/rpc/message/bolt/BoltInvokerCallback.java
43.63% <37.5%> (-4.06%)
4 <1> (ø)

...ofa/rpc/message/bolt/BoltFutureInvokeCallback.java
64.15% <71.42%> (-2.52%)
4 <1> (-1)

.../sofa/rpc/message/bolt/AbstractInvokeCallback.java
90.47% <90.47%> (ø)
6 <6> (?)

...m/alipay/sofa/rpc/module/FaultToleranceModule.java
66.66% <0%> (-26.67%)
4% <0%> (-1%)

... and 6 more

Continue to review full report at Codecov.

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


[BUG] memory usage grows exponentially when there are lots of CRDs

Hey folks,
I am running flux on a AKS cluster Server Version: v1.27.3 with:

60 HelmReleases
108 CRDs

I have been experiencing a memory issue with the helm controller, to the point where I faced OOMKilled a couple of times a day.
I have followed the advanced debugging instructions to profile the controller and got some interesting results:
(pprof) top10
Showing nodes accounting for 668.27MB, 89.98% of 742.69MB total
Dropped 280 nodes (cum <= 3.71MB)
Showing top 10 nodes out of 113
      flat  flat%   sum%        cum   cum%
  335.59MB 45.19% 45.19%   335.59MB 45.19%  reflect.New
  112.06MB 15.09% 60.28%   112.06MB 15.09%  google.golang.org/protobuf/internal/impl.consumeStringValidateUTF8
   82.07MB 11.05% 71.33%    82.07MB 11.05%  io.ReadAll
   41.01MB  5.52% 76.85%   105.53MB 14.21%  k8s.io/kube-openapi/pkg/util/proto.(*Definitions).parseKind
   20.50MB  2.76% 79.61%       29MB  3.91%  k8s.io/kube-openapi/pkg/util/proto.(*Definitions).parsePrimitive
   18.01MB  2.43% 82.03%    18.01MB  2.43%  github.com/go-openapi/swag.(*NameProvider).GetJSONNames
   17.50MB  2.36% 84.39%    33.01MB  4.44%  k8s.io/kube-openapi/pkg/util/proto.VendorExtensionToMap
      15MB  2.02% 86.41%       15MB  2.02%  google.golang.org/protobuf/internal/impl.consumeStringSliceValidateUTF8
   14.01MB  1.89% 88.30%    54.03MB  7.27%  k8s.io/kube-openapi/pkg/validation/spec.(*Schema).UnmarshalNextJSON
   12.51MB  1.68% 89.98%    12.51MB  1.68%  reflect.mapassign0

After posting this on the Slack channel, @stefanprodan suggested that it is related to the amount of CRDs (or their size), since Helm SDK uses all of the CRDs for discovery purposes and there's no way to disable that.
To test this issue, I have created https://github.com/fluxcd/flux-benchmark/pull/4 which automatically installs N CRDs on a k8s cluster and runs the controller against it. While running this on my M2, I tried 500 CRDs with 100 HR and at some point I think I crossed 1 CPU. Managed to catch this screenshot:

I also ran this on a Azure AMD D2as_v5 node.

The yellow line shows a test with HR=100 and CRD=100, while limits where set to 1 CPU, 2Gi memory.
The next lines are from the same experiment, using HR=100 and CRD=150

Once I did this, I tried to increase the limits to 2 cpu and 2Gi memory, moved the helm-controller to a more powerful node (4vcpu/16Gb mem) and also made sure that the helm-controller doesn't share the same node as Prometheus/Grafana/cert-manager, the restarts count decreased but were still happening:

Currently I managed to stop the restarts by increasing the limits again using 2cpu and 3Gi memory.
While I think that removing the cpu limit might help, the origin of this issue is directly related to @stefanprodan suggestion regarding the Helm SDK and the way it uses the installed CRDs and how GC works.

Extra info

2vcpu/8gb memory node
helm-controller limits:

limits:
  cpu: 1000m
  memory: 2Gi

Flux's extra arguments

--concurrent=10
--requeue-dependency=5s

Flux version

$ flux version
flux: v2.2.3
distribution: flux-v2.2.3
helm-controller: v0.37.4
kustomize-controller: v1.2.2
notification-controller: v1.2.4
source-controller: v1.2.4

So i did digging in Helm source code. The culprit seems to be the getCapabilities which invalidates the CRD cache and then queries the API to get all CRDs, so basically all CRDs are loaded twice into memory for each upgrade. Not sure how this can avoided and still have the Helm Capabilities feature working. I see that we could pass our own Capabilities, so maybe we could cache them globally in helm-controller and only refresh them when we install CRDs, but CRDs can also be in templates so we risk breaking Helm Capabilities.
@Shaked to validate my assumptions, you could modify helm-controller to load the Capabilities at startup only, then run your test and see of the memory usage drops.

Some good new 🎉 A combination of improvements in Flux 2.3 and Kubernetes API 1.29/1.30 make this issue less impactful.
Compared to Flux 2.2 and Kubernetes 1.28 where large number of CRDs would drive helm-controller into OOM, in Flux 2.3 and Kubernetes 1.29, even with 500 CRDs, helm-controller reconciles 1K HelmReleases in under 9 minutes when configured with concurrent 10, 2CPU and 1GB RAM limits. Benchmark results here: https://github.com/fluxcd/flux-benchmark/pull/6


83 add hpc testing framework

Adds cmake flags to turn off all of the tests that use google test until that can be debugged

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.01 :tada:

Comparison is base (46bc42b) 92.24% compared to head (36af4ac) 92.26%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #84      +/-   ##
==========================================
+ Coverage   92.24%   92.26%   +0.01%
==========================================
  Files          17       17
  Lines         967      969       +2
==========================================
+ Hits          892      894       +2
  Misses         75       75

Impacted Files
Coverage Δ

include/micm/system/system.hpp
90.90% <ø> (ø)

include/micm/util/sparse_matrix.hpp
98.64% <ø> (ø)

include/micm/process/arrhenius_rate_constant.hpp
100.00% <100.00%> (ø)

include/micm/process/photolysis_rate_constant.hpp
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Feature Request] Export Shareable Configs

Similar to other ESLint plugins like this (flowtype, react, etc.) it would be really helpful if this plugin exported shareable configs. Perhaps configs aiming for parity with TSLint's built in configs would be the most appropriate.
Edit: I'd be happy to submit a Pull Request implementing this if it interests you.

BTW, https://www.npmjs.com/package/eslint-config-standard-with-typescript

+1 here, would love to see plugin:typescript/recommended or something similar in my configuration:
"extends": [
  "standard",
  "plugin:node/recommended",
  "plugin:promise/recommended",
  "plugin:import/recommended",
  "plugin:flowtype/recommended",
  "prettier",
  "prettier/standard",
  "prettier/flowtype"
],

Closing in favour of #144


Add quotes for echo messages

Refer to this link for build results (access rights to CI server needed):
http://jenkins.socketplane.io/job/socketplane-verify/72/
Test PASSed.

Hi @tangkun75 thanks for your contribution.
I'm a little confused behind the motivation for this change. Is it purely a style thing or is it something functional?


[Request] Allow to delete tag from multiple images with one click

I want to mention that I can filter out images with the same tag, but still need to fix only specific images.
If you select 10 images with the same tag, you have to click the "Delete" button 10 times.
This takes time, so a "delete all identical" button would be best.

Looks like such button already exist.. "Remove selected 'All tags' from image tags"
So I need just select specific tag and specific images.

It seems that such feature is still needed.
If I use multiple filtered tags, other tags cannot be removed.
It seems that a separate table/section is needed that allows visual filtering.

Once you've filtered the list using several chosen tags in the All tags list, then you can click on other tags without that filtered set changing. This way you can remove any number of tags from the filtered set, which is the feature you're asking for if I'm interpreting what you wrote correctly.

I filtered 'blurry' and 'food' from "All tags" list.
And 3 filtered images appeared.
Now I want to fast delete 'fruit' from all or selected filtered images.
If I select 1 image I can use:
Find tag in 'All Tags' and click on Remove selected tags for selected image
If I select 2 or more images, above function not working.
I want to quickly delete any tag listed within the 'Image Tags' section from all filtered or selected images.

Ah, I see. I haven't tried that particular combination, but I can see that this may be a case the app doesn't support.
My workflow for that situation (workaround I guess) is to:

Filter a result set, just like you described.
Select the tag in the All-tags (right-hand list) I want to remove from a subset of the filtered images shown in the left list, and click the Remove from Filtered button.
For the filtered set of images still in the left list I actually want the tag applied to,
I just click those images one-by-one and use the keyboard shortcut to re-add the chosen tag to the selected image (Shift+W by default). This way no mouse-clicking is needed. Just UP/DOWN key to navigate the filtered images, and Shift+W to add the tag to the images that should have it.

It's not exactly as streamlined as what you're asking for, but it'll get the job done. unless the tag you wanted to remove from some of the filtered images are only tagged on those images. In that corner case once the tag is removed from the filtered set, then no image contains the tag, and it will vanish from the All-tags view as well, making step 3 above impossible. In that particular situation, I just re-add the tag to one of the images manually in a step "2.5" and then the process above works.


[youtube] Added 2 test cases in YoutubeIE

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[ ] Searched the bugtracker for similar pull requests
[ ] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature
[x] New test case(s)

Description of your pull request and other information
I've found out youtube-dl didn't have a Members Only and Youtube Music Premium test cases, so I added it

I've also used some trick to make a video become members only, this can be sure they won't be removed in the future

Are you sure the first test case (https://www.youtube.com/watch?v=rmeKRQXjfZA) you added is only for YouTube Music Premium users?
I seem to be able to access it just fine:

(It even works in incognito mode - i.e., not logged in)

Are you sure the first test case (https://www.youtube.com/watch?v=rmeKRQXjfZA) you added is only for YouTube Music Premium users?
I seem to be able to access it just fine:

(It even works in incognito mode - i.e., not logged in)

Oh sorry, it seems like I've entered the wrong video id, I'll correct it

After rechecking the music, seems like the tag has been removed, I'll remove that check


ci: fixed detox tests.

Close: https://github.com/jhipster/generator-jhipster-react-native/issues/889

Will update again in the weekend.

Still something wrong on e2e:

Update: Failed on the real machine (Mac OS 14.1.2 (23B92)):

Just some interesting findings:
Circle CI works for now (Xcode v15.1.0 (latest) - green):

but e2e CI failed in github actions (Xcode v15.0.1 (latest) - timeout)
@mraible @mshima PTAL

I will download the artifacts to see what’s happened in the tests, and will update the tests in the weekend if possible.

Hi @mraible @mshima
Could you please take a look if the pull request makes sense? :eyes:
Although there's something break in the e2e test files, eg, some issues I have found:

Album access permission (or the tests will hang out due to the prompt)

Expand to see the screenshot

Strange Hard-coded element

IMO the CI works now (see OAuth App).

It's already an improvement.
But but tests should pass so we can add as requirement to pass otherwise dependabot can break it again.
For now I recommend to skip failing tests.

You should use describe.skip instead of filter test.

Hi @mshima @mraible
I have upgraded Xcode from 14 to 15.1 (latest version currently) because:

all of tests are passed on my machine using XCode 15.1.0 and IOS simulator iPhone 15+.
we always maintain dependencies but not for e2e test enviroment, so I handled it in this pull request.

Please let me know if you have any questions!
Merry Chrismas! 🎄🧑🎄

and seems that Circle CI is more reliable than Github Actions.

@zhyd1997 Can you please fix the conflicts in this PR?

I thought I had merged this PR =\


Update to the @next

It will be nice if plug-in check which ver. of web UI is installed and if we already are on beta and next beta is available then show update button direct from GUI. Right now we need always run command.
Thanks

v4.29.0 adds the ability to install be beta/test version of any plugin, including the Homebridge core itself, from the UI.
The update notifications are still based on the "latest" tag however.


Incorrect Response Structure documentation for Jobs

This Bug Report affects these Traffic Control components:

Documentation

Current behavior:
The documentation for the APIv4 (and ostensibly APIv5) /jobs endpoint lists a description of the fields in a response to a GET request as being identical to those seen in APIv3. The response examples reveal that this is in error.
Expected behavior:
The documentation should be accurate.
Steps to reproduce:
Read the APIv4/5 GET /jobs documentation, create and fetch "Jobs" to confirm the response descriptions are incorrect.

Hi can I work on this issue :)


address issue where dispatch_io_read cal be called with a data argume…

Hello. I ran into a crash in the block passed to dispatch_io_read and from what I could see it seemed to be caused by data being set to nil. This is my attempt to address that crash by setting dataSize to 0 which just ignores the data for the rest of the block. This seemed to be a random one time situation and I have not been able to reproduce the issue since.

FYI, the  done flag was true when this crash was hit. The stack trace looked like this:

thread #7, queue = 'com.apple.libdispatch-io.opq', stop reason = EXC_BAD_ACCESS (code=1, address=0x30)
frame #0: 0x000000018142c480 libdispatch.dylib`dispatch_data_get_size
frame #1: 0x00000001010af560 Aria__53-[PTProtocol readPayloadOfSize:overChannel:callback:]_block_invoke(.block_descriptor=0x00000002820588a0, done=true, data=0x0000000000000000, error=57) at PTProtocol.m:247:23 frame #2: 0x000000018142be44 libdispatch.dylib___dispatch_operation_deliver_data_block_invoke + 148


Support buildkit build without dependency on kubeconfig

Is your feature request related to a problem?
I am attempting to use devspace in my Azure Pipelines CI for image builds rather than the usual sequential pipeline task, one per image.  Using devspace would allow me to take advantage of parallel builds and the pipelines for creating a single base image for all other images to use.
I'm having a similar problem to #1769  only the related merged PR doesn't solve it for me as I do use buildkit in order to use --secrets to support passing of a token to the docker build - necessary for the purpose of accessing a private package feed (in Azure).
So basically I need buildkit enabled and have no kubernetes cluster so no kubeconfig and no namespace.
Which solution do you suggest?
Would it be possible to introduce a flag to prevent the devspace build or build_images in the pipeline needing the kubeconfig rather than relying on if buildkit is used or not?
Which alternative solutions exist?
None for me.  I need buildkit if I'm to use my yaml in CI for image builds.
Additional context

/kind feature

I should add that I'm focused (fixated) on the v6 alpha versions :-) due to the pipelines feature, so looking for something to hopefully help in v6.

@markphillips100 thanks for creating this issue! We'll see what we can do there, but generally this should be possible

Thank you @FabianKramm .  I really appreciate it.

@FabianKramm do you have any expectation on a possible solution for this issue?  No pressure, just wanted to do some planning of my own is all.

Any updates, or whether this change would make it into v6 @FabianKramm ?

@markphillips100 I believe this should work with the latest DevSpace version. Is it currently still a problem?

Yep all good.  Just shows a warning about a missing kubeconfig but no longer prevents the image builds.  Thank you @FabianKramm

Hello!
Thank you @markphillips100 for raising this issue up!
We're glad your use case was successful with our updated version. So, I will close this issue as there are no clarifications left.
If you have any further questions or concerns, we'd be happy to support!


Profiling: reset_and_return_previous translates into api only to translate right back

If we cleanup the builder slightly, we can avoid this unnecessary roundtrip

Actually, it looks like this is needed to force re-insertion of the strings into the string table.


Secret Hashing

When a Secret is used as part of a non-human Client, it's effectively a password that needs to be protected. One of my go-to resources for security is the OWASP website and they recommend using a "salt" value when storing passwords:
https://www.owasp.org/index.php/Password_Storage_Cheat_Sheet
The ASP.NET Core documentation suggests, and provides examples for, a similar technique:
https://docs.asp.net/en/latest/security/data-protection/consumer-apis/password-hashing.html
Do you think it would be worthwhile added a "Salt" property to the Secret then modifying the HashedSharedSecretValidator to handle this?
I'm happy to submit a PR for this kinda change when you're happy with the proposal. It might also be worthwhile providing an ICryptoGenerator via Dependency Injection to complement the HashExtensions class for generating Salt values.

A salt is to thwart dictionary-style attacks. Given that we're using high entropy secrets, a salt isn't really necessary.

Sorry -- I should have added that if you disagree, please feel free to express your concerns.

Right - the recommendation for client secrets is to use random numbers - which already have a high enough entropy.

Maybe I'm just old school but without a salt, if two clients used the same secret, then they'd end up with the same "Sha256" value. Correct?

True. But secrets should be random keys to start with.
Sent from my iPhone

On 12.09.2016, at 14:29, dpbevin<EMAIL_ADDRESS>wrote:
Maybe I'm just old school but without a salt, if two clients used the same secret, then they'd end up with the same "Sha256" value. Correct?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Also, you should be generating these from your client registration. Look into this API: https://github.com/IdentityModel/IdentityModelv2/blob/dev/src/IdentityModel/CryptoRandom.cs#L34

Thanks for the tip

I share the same concern as @dpbevin, though I admit that I am very new to IdentityServer/OAuth/OpenId Connect. It seems that we should be using salts and a more secure algorithm to hash the secrets, not because the secrets aren't high enough entropy, but in order to prevent brute force attacks if, say, the database storing the secrets were to become compromised. Am I thinking about this incorrectly or missing something?

A 128 bit entropy secret hashed once should be sufficient. If you have anything to show this is not sufficient, please share.

I have nothing to share at this time that would prove this to be insufficient. I suppose that I'm just a bit paranoid of the situation in which vulnerabilities or implementation errors do happen to be found with Sha256. While salts may not necessarily be a cure-all depending on the type of vulnerability, I would imagine that they could end up helping to minimize the attack surface and overall exposure at a relatively low cost if such a situation were to arise, no?

If Sha256 is vulnerable we'll have bigger problems than our client secrets ;)
Anyways - you can use Sha512 as well - just make sure the secrets have a good entropy to start with - or implement your own secret validator with your preferred hashing algorithm.

Good point.. I appreciate your feedback! Thanks for everything you guys do!

Adding Salt property in the client object will make Databases that store passwords/secrets compliant. Not supporting this functionality creates a lot of compliance issues.


chore: migrate to owl bot

@SurferJeffAtGoogle anything I can do to help this PR get in? Is there a particular weird setup in this repo I can help clarify? Thanks!


Shared element transition

Overview (Required)

We can do shared element transition when going to the speaker screen.

Sorry for delay.
I will send PR this Weekend.


Problems in data streaming using Brainflow using muse2

Discussed in https://github.com/NeuroTechX/eeg-notebooks/discussions/198

Originally posted by Parvfect July 26, 2022
I am trying to run the N170 experiment, and it seemed to be working perfectly before and I have some good data from then. I am not sure what I changed, but the instruction screen seems to come up twice, the second time after running the main experiment for a few seconds.  About two images get displayed when the same screen comes up for the second time. The code used is the following
from eegnb.devices.eeg import EEG
from eegnb.experiments.visual_n170 import n170

board_name = 'muse2'

record_duration = 120
eeg_device = EEG(device=board_name)
n170.present(duration=record_duration, eeg=eeg_device)

And the runtime error is the following,

I am not very clear why this is happening and would really apprectiate some help. I was testing the ExperimentSubClass2 branch when this happened and switched to Master and got the same issue.

Can you repeat and show screen outputs for device = muse2_bfn ?

I get a key-error when I run with muse2_bfn as the device. Am I doing something wrong?

Yes something not right if you're using the above script and getting a key error.
We need to get back to step 1.
Please try:

fresh clone of eeg-notebooks (rename as '_temp' or '_v2' or whatev)
go in to folder, create new environment, activate environment, pip install -e .
Turn on muse
On command line eegnb checksigqual -ed muse2_bfn
If not working: repeat a few times; make sure bluetooth is on
If you get errors from this, please share screenshot here.

So it works with muse2 (the signal quality check), but if I do muse2_bfn, I get this

Do you mean it works with muse2 when bluemuse is running?
Try one more time: close conda terminal, close bluemuse, open new conda terminal, run _bfn signal check.
(Brainflow can get confused if there's been other recent communication with the device by currently running programs)

Yeah so it works with bluemuse and muse2. On closing bluemuse and independently running brainflow via the _bfn run check, I get this error as before

https://github.com/brainflow-dev/brainflow/issues/531
Issue opened in Brainflow
Unable to prepare the streaming session after finding Muse2 using direct calls to Brainflow.
Code:
from brainflow.board_shim import BoardShim, BoardIds, BrainFlowInputParams

params = BrainFlowInputParams()
board_id = BoardIds.MUSE_2_BOARD

board = BoardShim(board_id, params)
board.enable_dev_board_logger()

board.prepare_session()

Screenshot of Error

Original problem with respect to data streaming has been solved by using BlueMuse backend. The problem was that my file had an open call to streaming which was fixed by adding the following,

if __name__ == '__main__':
    # Do the Experiment

However, setting up a streaming session using Brainflow is still unresolved and listed above.

Summary:

Suggestion from linked brainflow thread https://github.com/brainflow-dev/brainflow/issues/531 was to update to 3.10


JSON CustomField Type unable to serialize Dictionary

Nautobot: 1.3.9 -1.4.1
pynautobot: 1.1.2
Python: 3.8
Observed Behavior:

In nautobot 1.3 JSON field type was introduced as an option for CustomFields. This field is set as a pythonic representation of JSON in the UI view.
If a custom field is created as a JSON type AND the value of the custom field is a dictionary a ValueError is raised for trying to unpack value as a key in the dictionary when calling serialize from PyNautobot on an instance of a device object.

Expected Behavior:

PyNautobot able to serialize a JSON type custom field.

Example with dictionary
>>> device = nautobot.dcim.devices.all()[0]
>>> device.custom_fields
{'json_test': {'bar': 'baz', 'foo': 1}}
>>> device.serialize()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/whitej6/Desktop/.venv/lib/python3.9/site-packages/pynautobot/core/response.py", line 335, in serialize
    ret[i] = flatten_custom(current_val)
  File "/Users/whitej6/Desktop/.venv/lib/python3.9/site-packages/pynautobot/core/response.py", line 63, in flatten_custom
    return {k: v if not isinstance(v, dict) else v["value"] for k, v in custom_dict.items()}
  File "/Users/whitej6/Desktop/.venv/lib/python3.9/site-packages/pynautobot/core/response.py", line 63, in <dictcomp>
    return {k: v if not isinstance(v, dict) else v["value"] for k, v in custom_dict.items()}
KeyError: 'value'

Example with list
>>> device = nautobot.dcim.devices.all()[1]
>>> device.custom_fields
{'json_test': [1, 2, 3]}
>>> device.serialize()
{'id': '7cc62d17-8b67-41fc-9233-03f10b9b3c49', 'display': 'HOU-WAN00', 'url': 'https://nautobot.whitej6.com/api/dcim/devices/7cc62d17-8b67-41fc-9233-03f10b9b3c49/', 'name': 'HOU-WAN00', 'device_type': '5795540f-e9c1-492a-bbc5-46d001c93207', 'device_role': '637f1db1-1656-4ebd-813c-0cf0f22d1747', 'tenant': None, 'platform': 'cceb8e56-1e1f-4b32-a4c3-1fa26b9681bb', 'serial': '', 'asset_tag': None, 'site': 'c09f95cf-d37e-4fec-8312-06e8ff340011', 'location': None, 'rack': None, 'position': None, 'face': None, 'parent_device': None, 'status': 'active', 'primary_ip': None, 'primary_ip4': None, 'primary_ip6': None, 'secrets_group': None, 'cluster': None, 'virtual_chassis': None, 'vc_position': None, 'vc_priority': None, 'comments': '', 'local_context_schema': None, 'local_context_data': None, 'config_context': {}, 'created': '2022-08-29', 'last_updated': '2022-08-29T18:30:19.674675Z', 'tags': [], 'notes_url': 'https://nautobot.whitej6.com/api/dcim/devices/7cc62d17-8b67-41fc-9233-03f10b9b3c49/notes/', 'custom_fields': {'json_test': [1, 2, 3]}}
>>>

https://github.com/nautobot/pynautobot/blob/develop/pynautobot/core/response.py#L62
Offending code

Resolved by #72


hover effect on navbar items

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
There is no proper hovering effect in the nav bar . It needs an improvement ...
kindly assign me this issue as I already created the nav bar for website like amazon.
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

please assign me

i working on issue