widget: slider: Add stepping functionality

A new builder method with_step(f64) is added to set the stepping value.
Signed-off-by: Christopher N. Hesse<EMAIL_ADDRESS>

What should a 0.0 step value represent? Smooth (no step at all) or a single step aka binary min/max range?

No step at all is reasonable. It makes logical sense to me, you have infinite steps.
Maybe its more rust-like use an option and disallow 0.0, but I am the wrong person to ask that.

This is good to go from my point of view now.
Paging @cmyr since he responded to my Zulip topic a few days ago.

Huh, the stepping is 10. I can see there being debate on of that should be in the form of min+n*stepping, but I'm confused how a stepping of 10 should create fractional results. (when min isn't a fraction)
With this rounding formula a stepping if 11 would create the same result as with 10.

I'm confused how a stepping of 10 should create fractional results. (when min isn't a fraction)

because self.max was used in calculation. and self.max / step is fractional

different implementation treats max as a separate step if it is not "well aligned"(min + n * step)


Documentation for Power BI excludes GCC High specific details

The above documentation for Power BI for US Government excludes details regarding the URL to access the service for GCC High customers. This URL is: https://app.high.powerbigov.us
Can we get this documentation updated to reflect both tenant types?

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

ID: d017bc96-49d4-7b63-9394-1739cdf7528d
Version Independent ID: 1e70d724-121d-feea-471e-307c2f27dd24
Content: Power BI for United States Government customers - Overview - Power BI
Content Source: powerbi-docs/service-govus-overview.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment @patrickabel. I've assigned this issue to the author for review.

Thanks for your feedback @patrickabel  - I've added that URL to the FAQ section, it'll update in the next 24 hours. Thanks!


Cydia crashes when I open it

all the tweaks work fine but i cant open cydia
i’ve tried the “reinstall cydia“button but it still crashes. i am on the dummy version of substrate and both unc0ver 2.02and 2.1 have the issue
it is really werid

Delete the following folders:
1- var/mobile/library/cydia/metadata.cb0
2- var/mobile/library/caches/com.saurik.Cydia (Whole folder)
now you can open cydia again

So ive been jailbroken for months now with no cydia issues what so ever on my ipad pro 12.1.1 using uncover jailbreak and this morning when i tried to open cydia it would start to load but always crash...i tried everything reccomended and nothing worked until i read this post..i made an account just to personally thank you for posting this solution...it was killing me that i couldnt figure out how to fix my problem especially when i havent installed any new tweaks in over a month....thanks eeesliao!!


Setup Automated with Gitpod

Hey @GokuMohandas ,
I would like to add two config files through this repo which can automate the task of setting up a workspace on Gitpod. Gitpod is an online IDE for GitHub. This PR might help the contributors as well as the learners. The workspace is equipped with anaconda so everything is installed by default and you can edit and runt the jupyter notebooks with the Gitpod IDE.
You can give it a try with my fork of this repo

When you launch the workspace for the first time you will be asked to setup a password for the jupyter notebook through the command line and you can use the same password to login to jupyter notebooks for the future sessions.
I hope this PR proves to be a great help for the contributors and learners.
Thanks,
Anudeep Reddy :)

hey @anudeepreddy , thanks for suggesting this. I definitely agree that adding an automated workspace will make it seamless for even more people to jump into the code. All the notebooks work on Google Colab but for students in China, Iran and other countries that ban google, having this workspace is important.
I've been using Binder which has been the standard for the community for the last couple years and it's completely free. My one concern with GitPod is that it's free for open source but capped at 100 hrs/month. Is there a massive benefit that it offers compared to Binder?

Hey @GokuMohandas , Sorry for the delay. I had my semester ending exams so i could't reply any faster.
I have discussed with the team of Gitpod regarding the same and here is the response

I want to give an overview over the current state of JNB support in Gitpod.
We have several options:

Run native Jupyter in Gitpod.
Run the Python VS Code Extension with support for JNB in Gitpod. This is what I spoke about in the past.
Custom integration of Jupyter. TypeFox has done similar work for https://r-brain.io/ in the past.

About Option 1:

This is robust, since it is the original Jupyter Notebook implementation. See screenshot1 below. You can try it yourself by opening https://gitpod.io/#https://github.com/meysholdt/demos/tree/jupyter. The notebook will open automatically. It’s a bit slow right now, but that can be easily improved by pre-installing everything in the docker image.
Jupyter runs inside the Gitpod workspace: If you edit the notebook, the changes are visible in the Gitpod workspace and can be committed via git.
The Notebook can run in Gitpod’s Preview or on another browser window/tab.
The user experience could be more streamlined: Jupiter has it’s own main menu, file browser, “save” action, etc., which is all redundant to Theia.

About Option 2:

The plan is to make all features documented here work in Gitpod: https://code.visualstudio.com/docs/python/jupyter-support
The Python extension with the JNB support is already installed in Gitpod by default.
Code Cells are already highlighted in Gitpod, see Screenshot2. You can see the “Run Cell”, “Run Above” grey links in the python code.
The “Interactive View” does not work yet in Gitpod, there are still some bugs.
.jpynb files can be automatically converted to python files with code blocks and back to .jpynb.

About Option 3:

This is about resolving my criticism in Option 1: Integrate Jupyter Notebooks in Theia to provide a more streamlined user experience: Use the Theia menu, allow opening notebooks from the Explorer, display Notebooks like editors.
My colleagues have been discussion this for a while without committing to any schedule.
In case this is interesting for you, we can discuss what the low-hanging fruits and their effort would be.

If you want to give it a try then do let me know. I will be happy to work on adding Gitpod support to your repository.


blend mode state is not properly url encoded

I am using ndviz with render and my render web service has a default value set of:
WEB_SERVICE_MAX_TILE_SPECS_TO_RENDER=20
to speed rendering by rendering only bounding boxes when the FOV is larger than 20.
When I have a multi-layer view, and am zoomed in sufficiently, I can see both layers rendered. My URL looks like:
http://em-131snb1:8001/#!{'layers':{'12_section_montage_split0':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split0'}_'split1':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split1/'_'color':2}}_'navigation':{'pose':{'position':{'voxelSize':[1_1_1]_'voxelCoordinates':[23398.046875_249166.5_1028.5]}}_'zoomFactor':27.11263892065789}_'blend':'additive'}
Now, if I'd like to zoom out, and want to render more tiles, I manually add in maxTilesSpecsToRenderto the URL like this:
http://em-131snb1:8001/#!{'layers':{'12_section_montage_split0':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split0?maxTileSpecsToRender=500'}_'split1':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split1?maxTileSpecsToRender=500/'_'color':2}}_'navigation':{'pose':{'position':{'voxelSize':[1_1_1]_'voxelCoordinates':[23398.046875_249166.5_1028.5]}}_'zoomFactor':27.11263892065789}_'blend':'additive'}
Once I do this, the 2 layers can't be seen at the same time anymore. I.e. the blending isn't working anymore.
On the other hand, if, when I add the 2nd layer, and add ?maxTileSpecsToRender=500 in the dialog html text box before pressing "add layer", then I do get the desired behavior, though, the URL looks basically the same:
http://em-131snb1:8001/#!{'layers':{'12_section_montage_split0':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split0?maxTileSpecsToRender=500'}_'split1':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split1/?maxTileSpecsToRender=500'_'color':2}}_'navigation':{'pose':{'position':{'voxelSize':[1_1_1]_'voxelCoordinates':[62600.31640625_295237.78125_1028.5]}}_'zoomFactor':90.01713130052188}_'blend':'additive'}
so,it seems like the URL is not fully capturing the blending state, and I can't manually tweak the URL and maintain the correct blending.

It looks like the issue is that in the manually edited url there is a /
character at the end, rather than before the ? character.
By the way, in recent versions of neuroglancer there is a built in editor
for editing the URL state as json --- it is the curly brace icon in the top
right.
On Mon, May 7, 2018, 16:23 Dan Kapner<EMAIL_ADDRESS>wrote:

I am using ndviz with render and my render web service has a default value
set of:
WEB_SERVICE_MAX_TILE_SPECS_TO_RENDER=20
to speed rendering by rendering only bounding boxes when the FOV is larger
than 20.
When I have a multi-layer view, and am zoomed in sufficiently, I can see
both layers rendered. My URL looks like:
http://em-131snb1:8001/#!{'layers':{'12_section_montage_split0':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split0'}_'split1':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split1/'_'color':2}}'navigation':{'pose':{'position':{'voxelSize':[1_1_1]'voxelCoordinates':[23398.046875_249166.5_1028.5]}}'zoomFactor':27.11263892065789}'blend':'additive
'}
Now, if I'd like to zoom out, and want to render more tiles, I manually
add in maxTilesSpecsToRenderto the URL like this:
http://em-131snb1:8001/#!{'layers':{'12_section_montage_split0':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split0?maxTileSpecsToRender=500'}_'split1':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split1?maxTileSpecsToRender=500/'_'color':2}}'navigation':{'pose':{'position':{'voxelSize':[1_1_1]'voxelCoordinates':[23398.046875_249166.5_1028.5]}}'zoomFactor':27.11263892065789}'blend':'additive
'}
Once I do this, the 2 layers can't be seen at the same time anymore. I.e.
the blending isn't working anymore.
On the other hand, if, when I add the 2nd layer, and add
?maxTileSpecsToRender=500 in the dialog html text box before pressing
"add layer", then I do get the desired behavior, though, the URL looks
basically the same:
http://em-131snb1:8001/#!{'layers':{'12_section_montage_split0':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split0?maxTileSpecsToRender=500'}_'split1':{'type':'image'_'source':'render://http://em-131snb1:8080/danielk/MM2/12_section_montage_split1/?maxTileSpecsToRender=500'_'color':2}}'navigation':{'pose':{'position':{'voxelSize':[1_1_1]'voxelCoordinates':[62600.31640625_295237.78125_1028.5]}}'zoomFactor':90.01713130052188}'blend':'additive
'}
so,it seems like the URL is not fully capturing the blending state, and I
can't manually tweak the URL and maintain the correct blending.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/neuroglancer/issues/86, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEBE6gjkEjU2ZTbaIQLgoWe_mol3QM8lks5twNeJgaJpZM4T1you
.

I did notice that / character was different, but moving it did not change the behavior I'm reporting.
Thanks for the tip about the json edit. I'm not seeing any curly bracket, I'll check what version I'm running

another easy way to see that it is not encoded entirely is to build up a multi-layer view with blending (no manual URL twiddling), then copy/paste the URL into another browser tab, blending doesn't follow.

The JSON editor may not have been merged into ndviz yet.
I noticed that 'blend': 'additive' is being specified in the wrong place.  It should be a per-layer property rather than a global property.  Not sure if this is a neuroglancer/ndviz difference.

ok, I wasn't sure if this was a neuroglanceror ndviz issue. Sounds like ndviz, I'll post over there

@jbms I think the issue here is that the blend mode is not getting attached to the url state of the layer when new layers are getting added. The blend_mode at the top level does successfully cause new layers to get additive blending, but then that isn't encoded in the url, so when dan adds the query parameter to the url, and the browser goes to this 'new' page, the blend mode state is lost.

The top-level blend mode is an ndviz-only feature, though.

ah i see... i'm talking to @alexbaden and @perlman about fixing this for ndviz.  Is this feature portable to neuroglancer?  I think the general use case of having a visualization routine which involves changing the default options for new layers is something worth considering how to handle.  for example, having a different default shader for each layer, the blend mode, etc.

fixed in ndviz https://github.com/neurodata/ndviz/issues/181


Upgrade to Node.js v6.9.0

Node.js v6 enters LTS today.

@GoogleCloudPlatform/node-team PTAL

LGTM! We need to sync with @jmdobry to make sure all of our samples will run though

For posterity:

v6.x LTS accounce<EMAIL_ADDRESS>Changelog for v6.9.0: https://github.com/nodejs/node/blob/master/doc/changelogs/CHANGELOG_V6.md#6.9.0


Operations Dashboard Content Items

Operations Dashboard applications do not display the 'View App' options which appear beside the favorite button and when you hover over the item thumbnail.
This means that the end-user has to open the item page and then launch from there instead of launching directly from the gallery.
It would be great if this functionality could be expanded to Operations Dashboard items.

FYI this is logged in My Esri as BUG-000125866.

I want to add that this also occurs with Insights items.


Add auto bumper to create PRs in light hotspot software repo

As per the full miner software, add repository dispatch action to auto add a pr to nation light hotspot software repo on update of this container

Closing as this has been added to main helium-miner-software repo already as part of #35


imgpdf improvements

You can specify:

Number of cores in which the pages will be rendered in parallel
Scale the pages are rendered (1 = 72 dpi, 0.2 = 360 dpi)
Whether the render has antialias or not

Parallelizing the rendering saves some time (even not as much as one
would hope) since the saving back to pdf is still done one page after
the other.
Using antialias allows for decreasing the dpi while still getting a
reasonable enough quality.

This is absolutely fantastic, thank you! Do I dare to ask you also to upgrade README.md to document the new parameters and the changes. Let's say version number after this upgrade would be 3.1.0.

README.MD updated

Hi there. Any updates on merging this?

@tpisto Any chance of merging this?

@tsdgeos I tried to try out your fork, but I guess you would also need to update the installation instructions. I'm on mac and installing your fork via npm fails because QMutex class is not available

Thank you for your pull request! What's your name? I would like to add it to the list of contributors. I'm very pleased to see that so many has been willing to push this library forward!

That is Albert Astals Cid, Poppler's maintainer.

@tsdgeos thank you, i prepended QtCore to the includes you added but installation still fails. I followed the recommended installation routine with homebrew

2017-12-20 12:11:25.117 xcodebuild[71177:791382] [MT] PluginLoading: Required plug-in compatibility UUID E0A62D1F-3C18-4D74-BFE5-A4167D643966 for plug-in at path '~/Library/Application Support/Developer/Shared/Xcode/Plug-ins/Unity4XC.xcplugin' not present in DVTPlugInCompatibilityUUIDs
2017-12-20 12:11:25.118 xcodebuild[71177:791382] Failed to load plugin at: /Users/floriannorbertbepunkt/Library/Application Support/Developer/Shared/Xcode/Plug-ins/Unity4XC.xcplugin, skipping.  Reason for failure: *** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[0]
  CXX(target) Release/obj.target/pdf_fill_form/src/pdf-fill-form.o
  CXX(target) Release/obj.target/pdf_fill_form/src/NodePopplerAsync.o
  CXX(target) Release/obj.target/pdf_fill_form/src/NodePoppler.o
In file included from ../src/NodePoppler.cc:21:
In file included from /usr/local/include/QtCore/QThreadPool:1:
In file included from /usr/local/include/QtCore/qthreadpool.h:45:
/usr/local/include/QtCore/qthread.h:163:43: error: 'future<void>' is unavailable: introduced in macOS 10.8
    static QThread *createThreadImpl(std::future<void> &&future);
                                          ^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/future:1279:52: note: 'future<void>' has
      been explicitly marked unavailable here
class _LIBCPP_TYPE_VIS _LIBCPP_AVAILABILITY_FUTURE future<void>
                                                   ^
1 error generated.
make: *** [Release/

Okay, quick follow up in case anyone else comes across the same problem. This is an issue with Qt5.10 on mac (at least on el capitan). Installing an older version and prepending the new include as stated by @tsdgeos solves the issue.

Just add /usr/local/include/QtCore to the end of the CPATH environment variable, assuming a brew link --force qt. I have CPATH, LD_LIBRARY_PATH, and PKG_CONFIG_PATH set in my .bashrc file.
function update_path_variable()
{
  local paths="${!1}"
  local new_paths="$2"

  if [[ -z "${paths}" ]]; then
    paths="${new_paths}"
  else
    paths="${new_paths}:${paths}"
  fi

  export "$1=${paths}"
}

update_path_variable CPATH "${HOME}/include:/usr/local/include/QtCore"
#update_path_variable LIBRARY_PATH "${HOME}/lib:/usr/X11/lib:/usr/local/lib:/lib:/usr/lib"
update_path_variable LD_LIBRARY_PATH "${HOME}/lib:/usr/X11/lib:/usr/local/lib:/lib:/usr/lib"
#update_path_variable DYLD_LIBRARY_PATH "${LIBRARY_PATH}"
update_path_variable DYLD_FALLBACK_LIBRARY_PATH "${LIBRARY_PATH}"
update_path_variable MANPATH "${HOME}/share/man:${HOME}/man:/usr/bin/man:/usr/X11/share/man:/usr/local/share/man:/usr/share/man"
update_path_variable PATH "${HOME}/bin:/usr/local/opt/openssl/bin:${HOME}/projects/chromium/tools/depot_tools"
update_path_variable PKG_CONFIG_PATH "${HOME}/lib/pkgconfig:/usr/X11/lib/pkgconfig:/usr/local/lib/pkgconfig:/lib/pkgconfig:/usr/lib/pkgconfig"


path/filepath: WalkDir does not treat "." as equivalent to os.Getwd()

What version of Go are you using (go version)?
go1.18-16d6a5233a
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
linux/amd64 via play.golang.org.
What did you do?
(https://go.dev/play/p/OLEED5lI02l?v=gotip)
With PWD set to a path ending in a symlink to the actual working directory, compare the behavior of filepath.WalkDir with "." and with the directory reported by os.Getwd().
What did you expect to see?
Equivalent behavior for "." and os.Getwd(). Specifically, I expected filepath.WalkDir to report the current working directory as a symlink in both cases.
What did you see instead?
os.Chdir accepts the symlink as valid and changes to the underlying directory. With a matching PWD environment variable, os.Getwd() reports the symlink as the current working directory (as intended since #8400).
filepath.WalkDir(cwd, …) treats the current directory as the symlink itself, and does not list its contents.
However, filepath.WalkDir(".", …) treats "." as the underlying directory and walks its contents.

This may be as close to “working as intended” as is feasible given #8400. The equivalent C program reports "." as a non-link, although on Linux getcwd also reports the current working directory as a non-link.

#include <limits.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/stat.h>
#include <sys/types.h>
#include <unistd.h>

int main() {
  size_t n = PATH_MAX;
  char *buf = malloc(n);
  char *cwd = NULL;
  while (!(cwd = getcwd(buf, n))) {
    n *= 2;
    buf = realloc(buf, n);
  }
  printf("cwd is %s\n", cwd);
  free(buf);

  struct stat statbuf;
  memset(&statbuf, 0, sizeof(statbuf));
	if (lstat(".", &statbuf) != 0) {
    perror(NULL);
    return 1;
  }

  if (S_ISLNK(statbuf.st_mode)) {
    printf(". is a link\n");
  } else {
    printf(". is not a link\n");
  }

  if (fflush(stdout) != 0) {
    perror(NULL);
    return 1;
  }

  return 0;
}

~/tmp/link$ gcc main.c && ./a.out
cwd is /usr/local/google/home/bcmills/tmp/dir
. is not a link

I don't think this can be fixed without breaking compatibility, and — given that it can be worked around with a wrapper that reimplements the first layer of the walk — I don't think it's worth breaking compatibility to fix.

This is closely related to #26033.


Crashed on windows 10 with 1050Ti

Example code is https://github.com/Remering/three_dart_playground, may be I don't know how to use this library, please help me. Thanks a lot!

Hi @Remering,
Not sure if you have solved your issue yet, but I noticed in your viewer.dart code you have
    if (kIsWeb) {
      await glPlugin.prepareContext();
    }

please make this
    Future.delayed(const Duration(milliseconds: 100), () async {
      await glPlugin.prepareContext();
    });

this might fix the crashing issue since prepareContext is needed for all platforms.
Hope this helps.


Clan Announcements

I believe putting announcements such as level ups, etc should be formatted in bold or something to easier tell. that or you could put something like this in front of it.

I've added icons for the different type of broadcasts, they are toggleable via a new setting, going to see how the icons fit in to see if they need to be changed.

I've added icons for the different type of broadcasts, they are toggleable via a new setting, going to see how the icons fit in to see if they need to be changed.

This new feature is broken since you cant disable it, i think that on server level you dont have correct naming as the name that is being checked/required.
ai this makes it so you can really update any change at all.

I've added icons for the different type of broadcasts, they are toggleable via a new setting, going to see how the icons fit in to see if they need to be changed.

This new feature is broken since you cant disable it, i think that on server level you dont have correct naming as the name that is being checked/required.
ai this makes it so you can really update any change at all.

Just pushed a fix, just a typo between clan_icons and clain_icons in validation logic. Let me know if this fixes it :)

I've added icons for the different type of broadcasts, they are toggleable via a new setting, going to see how the icons fit in to see if they need to be changed.

This new feature is broken since you cant disable it, i think that on server level you dont have correct naming as the name that is being checked/required.
ai this makes it so you can really update any change at all.

Just pushed a fix, just a typo between clan_icons and clain_icons in validation logic. Let me know if this fixes it :)

Yep, perfect, fixed, thank you

Just got wind of this, how exciting! I will let you know if I have further suggestions.


Handle not wrapped exceptions same way as all other

Closes #1536
First of all I want to note that this change is breaking one as far as I can see.
While most of my motivation is in #1536 and I will put an overview and some extra thoughts here:
Hystrix had that nice idea that BadRequestException was designed to bypass fallbacks. It was a really nice feature that allowed users of Hystrix to make a difference between execution errors and some conditional (so called "expected") errors e.g. validation.
#1414 #1503 introduced that nice concept of exception unwrapping which is really cool and I'm grateful to authors to have this feature. It has only one flaw - it has broken an idea of BadRequestException. Now any exception that implements ExceptionNotWrappedByHystrix leaves no choice to user if he wants to have fallbacks available - not to use it.
This PR solves this issue.
I hope I haven't broken a lot.
So the consequences of this PR:

users need to review their logic as all fallbacks will start to work in case of exceptions marked by ExceptionNotWrappedByHystrix

Thanks @gagoman .
Let me restate the cases to check that I understand the implications:

exception thrown neither extends HystrixBadRequestException nor implements ExceptionNotWrappedByHystrix.  It's eligible for fallback and the exception thrown to caller is wrapped in HystrixRuntimeException.
exception thrown extends HystrixBadRequestException but does not implement ExceptionNotWrappedInHystrix.  It's not eligible for fallback and the original exception (which extends HystrixBadRequestException) is thrown to the caller
exception thrown implements ExceptionNotWrappedInHystrix and does not extend HystrixBadRequestException.  It's eligible for fallback and the unwrapped exception is thrown to the caller in the case the fallback is not implements or otherwise fails
exception thrown both implements ExceptionNotWrappedInHystrix and extends HystrixBadRequestException.   It's not eligible for fallback and the original exception (which extends from HystrixBadRequestException) is thrown to the caller.

Those sound like the correct semantics, and what your PR implements.  The piece which is broken in master is the 3rd case - it does not attempt to fallback.
Did I get that right?

The piece which is broken in master is the 3rd case - it does not attempt to fallback
Yes, the goal of PR is to fix this.

Answers:

exception thrown neither extends HystrixBadRequestException nor implements ExceptionNotWrappedByHystrix. It's eligible for fallback and the exception thrown to caller is wrapped in HystrixRuntimeException.

tests are: testExecutionFailureWithNoFallback and testExecutionFailureWithFallbackFailure

exception thrown extends HystrixBadRequestException but does not implement ExceptionNotWrappedInHystrix. It's not eligible for fallback and the original exception (which extends HystrixBadRequestException) is thrown to the caller

There are no tests that cover extends case. Only available are: testExecutionHookSemaphoreBadRequestException and testExecutionHookSemaphoreBadRequestException

exception thrown implements ExceptionNotWrappedInHystrix and does not extend HystrixBadRequestException. It's eligible for fallback and the unwrapped exception is thrown to the caller in the case the fallback is not implements or otherwise fails

tests are: testNotWrappedExceptionWithFallback and testNotWrappedExceptionWithNoFallback

exception thrown both implements ExceptionNotWrappedInHystrix and extends HystrixBadRequestException. It's not eligible for fallback and the original exception (which extends from HystrixBadRequestException) is thrown to the caller.

as in #2 there are no tests that cover extends case while tests that have exception wrapped in HystrixBadRequestException are available:
testNotWrappedBadRequestWithFallback and testNotWrappedBadRequestWithNoFallback

@mattrjacobs Any update on this? Should I add/change something? Thank you.

Thanks @gagoman !

Hi,
in my project implemented a custom exception HttpBaseException which extends HystrixBadRequestException and implements ExceptionNotWrappedByHystrix. My command throws the HttpBaseException, I expected to receive no fallback and the exception propagated, but I received the fallback instead. What I missed to obtain this behaviour? Thank you.

Hello.
I'm not sure about your case. Could you please provide an example?
Is this your case?
private static class BadAndUnwrapped extends HystrixBadRequestException implements ExceptionNotWrappedByHystrix {
    public BadAndUnwrapped(String message) {
        super(message);
    }
}

private static class Sample extends HystrixCommand<String> {

    protected Sample() {
        super(asKey("sample"));
    }

    @Override
    protected String run() throws Exception {
        throw new BadAndUnwrapped("expected one");
    }

    @Override
    protected String getFallback() {
        throw new RuntimeException("not expected");
    }
}

public static void main(String[] args) throws Exception {
    new Sample().run();
}

generates next output:
Exception in thread "main" hystrix.HystrixSample$BadAndUnwrapped: expected one
	at hystrix.HystrixSample$Sample.run(HystrixSample.java:82)
	at hystrix.HystrixSample.main(HystrixSample.java:92)

Hystrix version: 1.5.12

Hi @gagoman, thank you for your response. Yes, it was exactly my case, I've not tried directly the run method, but the execute one on HystrixCommand.
I've the same Hystrix version: 1.5.12. Now I changed my configuration, with a custom workaround implemented, but calling the execute I've the issue described in my previous comment.

You are right about execute. Still I can not reproduce, even by using execute. Could you please provide an example?
P.S. I would suggest to debug executeCommandAndObserve and especially final Func1<Throwable, Observable<R>> handleFallback part to see how your Exception is treated.


Fix for nested Leap 15.4 VM

What does this PR change?
This will add the official Leap 15.4 repositories and install e.g. the venv-salt-minion package in order to not have to sync the complete Leap 15.4 repository in the test suite CI servers to be able to bootstrap this nested VM. Furthermore I added nss-mdns and  qemu-guest-agent like we do in sumaform.

Is the virthost state only used in the test suite or would this change apply to my sumaform deployment for development?


将URL中的默认值使用三目运算符表示。

What is the purpose of the change
将URL类中的默认值，修改为用三目运算符使用。减少冗余代码！
Brief changelog
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #4301 into master will increase coverage by 0.01%.
The diff coverage is 30.43%.

@@             Coverage Diff              @@
##             master    #4301      +/-   ##
============================================
+ Coverage     62.94%   62.95%   +0.01%
+ Complexity      506      504       -2
============================================
  Files           769      769
  Lines         33041    32995      -46
  Branches       5215     5215
============================================
- Hits          20798    20773      -25
+ Misses         9850     9821      -29
- Partials       2393     2401       +8

Impacted Files
Coverage Δ
Complexity Δ

...mon/src/main/java/org/apache/dubbo/common/URL.java
49.4% <30.43%> (-0.6%)
0 <0> (ø)

...pache/dubbo/registry/support/AbstractRegistry.java
80.07% <0%> (-1.54%)
0% <0%> (ø)

...ache/dubbo/registry/multiple/MultipleRegistry.java
62.67% <0%> (-1.41%)
20% <0%> (-2%)

...pache/dubbo/remoting/transport/AbstractServer.java
47.91% <0%> (-1.05%)
0% <0%> (ø)

...g/apache/dubbo/registry/consul/ConsulRegistry.java
59.6% <0%> (-0.67%)
27% <0%> (ø)

...ting/zookeeper/curator/CuratorZookeeperClient.java
64.15% <0%> (+1.25%)
0% <0%> (ø)
:arrow_down:

...he/dubbo/registry/multicast/MulticastRegistry.java
69.23% <0%> (+1.35%)
0% <0%> (ø)
:arrow_down:

.../apache/dubbo/remoting/transport/AbstractPeer.java
67.39% <0%> (+2.17%)
0% <0%> (ø)
:arrow_down:

.../remoting/transport/netty4/NettyServerHandler.java
71.42% <0%> (+9.52%)
0% <0%> (ø)
:arrow_down:

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 3acf5b8...f5b7909. Read the comment docs.


Profiling method?

First of all, I love the project you are doing!
So my question is: How do you profile the memory (and maybe storage) the model consumes?
I really appreciate any help you can provide.
Rodo

Thanks a lot for the detailed answer! That's more than sufficient for my discovery.
Rodo

Glad to hear it helps. I will close this issue since is solved.


Fix cairo constructor auto-import bug

I couldn't find a reasonable way to do it. I tried the following approaches:

modifying precompilation output — the information about constructor is spread in various places, which modification triggers asserts
modifying source code on the fly — logic error — test suites do not have constructor

Instead of fixing this problem, I'll make Protostar print a nice error.

Sounds good to me I will take a look on that if I have some spare time (probably not very soon)


Add style to headings

As Tailwind unstyled headings (see: Headings are unstyled ), it make a significant difference between standard and tailwind backend UIs.
Standart Winter backend UI

This plugin

Wouldn't it be possible to add some substance to the titles, or the plugin devs have to add their owns styles ?

I'm fine with any PRs you want to make so long as they have before / after screenshots for me to review :)

It is the responsibility of this plugin to provide a fully styled backend skin, it's safe to assume that if the default skin styles something then this plugin should also style said element, albeit with more of a modern look.

@LukeTowers That answer my question.
So no need to style the headings from the Tailwinf-UI plugin.

Well this plugin should at the very least provide the same visual distinction between titles and regular text that the default backend skin does, and I'm fine if we want to go above and beyond to make the backend feel more modern and consistent.


Sharp in strings

Hello, just opening an issue to signal that the # character contained inside a string delimited by quotes triggers the colouring into comment colour for the rest of the line.
This can cause issues while writing headers for output files. For example consider the following:
fix     BDH all ave/histo 1 1 1 0 1 100 c_Mycompute ave running file bd.hist &
start 500 title1 "# Bond distrance histogram" mode vector
The bold part will be considered as a comment while "mode vector" is actually an input for the fix.
Thanks for considering the issue if you have time.
Regards.

Hi @Bibobu !
Sorry for taking so long to reply, somehow I did not see this message. I stopped using Atom (and LAMMPS) altogether soon after making this package. I cannot recall well how Atom packages work and I have no interested in learning it again.
If you have some time you feel like trying play with the regex rules in lammps.cson (that uses Oniguruma) and follow any TextMate tutorial about language support.
Best wishes and sorry again for a negative reply,
Bruno


fix: A bug fix

Overview
fix #5708

What is it?

[x] Feature / enhancement
[x] Bug
[x] Docs / tests / types / typos

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Use cases and why

One use case

Another use case

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] Added new tests to cover the fix / functionality

This is not the right way to tackle this. The reason that in dev mode the decorators don't work is that the qwik plugin (which understands them) isn't running.
So the options are:

making qwik plugin work during dev mode (might be difficult with HMR but worth trying)
add very light-weight support for decorators (this excludes babel plugins) during dev only

This is not the right way to tackle this. The reason that in dev mode the decorators don't work is that the qwik plugin (which understands them) isn't running.
So the options are:

making qwik plugin work during dev mode (might be difficult with HMR but worth trying)
add very light-weight support for decorators (this excludes babel plugins) during dev only

thank for your tips. it's useful for me, especially as a new one.
let me dig into it deeply,

I'd like to add that it's fine to use this as a standalone vite plugin, but for qwik it needs to be very light weight because not many people use decorators in qwik projects.

I'd like to add that it's fine to use this as a standalone vite plugin,

Could you clarify the conception about qwik plugin?

such as isn't qwikCity a Vite plugin? So which is qwik plugin?

but for inclusion in qwik core it needs to be very light weight because not many people use decorators in qwik projects.

I totally a agree, babel is such heavy, I'll find out another

Actually I've just tried to figure out why the decorator transform doesn't run in Feb mode.
Here's the rust plugin enabling it: https://github.com/BuilderIO/qwik/blob/6b6b3acfa698b449a6debe932d2a8b9dd33e0f81/packages/qwik/src/optimizer/core/src/parse.rs#L522
And when I look at the qwik optimizer plugin.ts, it only changes mode to be "dev" but it calls the optimizer in the same way in the transform() hook.
So maybe something else is erroring in dev mode, running before the Qwik optimizer?

Actually I've just tried to figure out why the decorator transform doesn't run in dev mode.
Here's the rust plugin enabling it:
https://github.com/BuilderIO/qwik/blob/6b6b3acfa698b449a6debe932d2a8b9dd33e0f81/packages/qwik/src/optimizer/core/src/parse.rs#L522
And when I look at the qwik optimizer plugin.ts, it only changes mode to be "dev" but it calls the optimizer in the same way in the transform() hook.
So maybe something else is erroring in dev mode, running before the Qwik optimizer?

it;


[suggestion] adding ollie to the android orm benchmark

AndroidDatabaseLibraryComparison is a project that compares the performance of different android orm/db libraries.
The results can be found here
At the moment, it covers greendao, sugar, activeandroid, sprinkles and dbflow (and ormlite is on the way)
Adding Ollie into the mix would be interesting.
What do you think?

Would be very interesting.

:+1:

@PerfectCarl @pardom This can probably be closed now that it has been merged.


OpenSuse Tumbleweed - azure-cli-telemetry doesn't support python311-portalocker in version 3 and above

Describe the bug
When I'm trying to update my system via sudo zypper dup I'm getting an error with python311-portalocker version issue.
Please update azure-cli-telemetry to work with azure-cli-telemetry version 3 and above.
Related command
sudo zypper dup
Loading repository data...
Reading installed packages...
Warning: You are about to do a distribution upgrade with all enabled repositories. Make sure these repositories are compatible before you continue. See 'man zypper' for more information about this command.
Computing distribution upgrade...
Problem: 1: the installed azure-cli-telemetry-1.1.0-3.1.noarch requires 'python311-portalocker < 3.0', but this requirement cannot be provided
deleted providers: python311-portalocker-2.10.1-1.2.noarch
Solution 1: Following actions will be done:
deinstallation of azure-cli-telemetry-1.1.0-3.1.noarch
deinstallation of azure-cli-core-2.66.0-1.1.noarch
deinstallation of azure-cli-2.66.0-1.1.noarch
Solution 2: keep obsolete python311-portalocker-2.10.1-1.2.noarch
Solution 3: break azure-cli-telemetry-1.1.0-3.1.noarch by ignoring some of its dependencies
Errors
Problem: 1: the installed azure-cli-telemetry-1.1.0-3.1.noarch requires 'python311-portalocker < 3.0', but this requirement cannot be provided
deleted providers: python311-portalocker-2.10.1-1.2.noarch
Issue script & Debug output
sudo zypper dup
Loading repository data...
Reading installed packages...
Warning: You are about to do a distribution upgrade with all enabled repositories. Make sure these repositories are compatible before you continue. See 'man zypper' for more information about this command.
Computing distribution upgrade...
Problem: 1: the installed azure-cli-telemetry-1.1.0-3.1.noarch requires 'python311-portalocker < 3.0', but this requirement cannot be provided
deleted providers: python311-portalocker-2.10.1-1.2.noarch
Solution 1: Following actions will be done:
deinstallation of azure-cli-telemetry-1.1.0-3.1.noarch
deinstallation of azure-cli-core-2.66.0-1.1.noarch
deinstallation of azure-cli-2.66.0-1.1.noarch
Solution 2: keep obsolete python311-portalocker-2.10.1-1.2.noarch
Solution 3: break azure-cli-telemetry-1.1.0-3.1.noarch by ignoring some of its dependencies
Expected behavior
Resolving the package version conflict.
Environment Summary
az --version
azure-cli                         2.66.0 *
core                              2.66.0 *
telemetry                          1.1.0
Dependencies:
msal                              1.31.0
azure-mgmt-resource               23.2.0
Python location '/usr/bin/python3.11'
Extensions directory '/home/bliker/.azure/cliextensions'
Python (Linux) 3.11.10 (main, Sep 09 2024, 17:03:08) [GCC]
Legal docs and information: aka.ms/AzureCliLegal
Additional context
No response

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

cc @jiasl@rayluo for awareness. MSAL is also restricting portalocker<3

I've used https://github.com/AzureAD/microsoft-authentication-extensions-for-python/pull/117 in CLI on SAW 2.55.0 and it works.

I've used AzureAD/microsoft-authentication-extensions-for-python#117 in CLI on SAW 2.55.0 and it works.

@bebound Thanks for the promising feedback! You also brought up a good point on SAW. I am not even sure whether the portalocker was allowed on SAW, and if not, how the previous versions of AzureCLI + MsalExtensions + Portalocker combination worked (or, did they?). Regardless, this becomes yet another good reason to trim down that dependency.
@jiasli , do we want this change to be in one of upcoming Azure CLI releases?

I've used this PR's msal-extension without portalocker since the initial release of SAW version 2.40.0.

@bebound, when was that Azure CLI 2.40.0 released? Was it in September 2022?
Sorry I don't quite remember our meeting. Did you mean you used/tested that portalocker-less msal-extension for 2+ years already?

CLI on SAW is an internal version. It's been released for two years, and we haven't received any bug report related to msal. At least it works for the SAW version.
I assume dropping portalocker causes issues in high concurrency environments?

I assume dropping portalocker causes issues in high concurrency environments?

No. Because that PR does not simply drop portalocker. It also uses a replacement locking mechanism without external dependencies.

CLI on SAW is an internal version. It's been released for two years, and we haven't received any bug report related to msal. At least it works for the SAW version.

Thanks for the much-awaited confirmation! I'll merge that PR in, then.


Add a small delay after the page scroll is idle to calculate the progress bar position.

Towards #1975

Waits a little bit after the task view has changed in order to recalculate the progress bar position, which is based on the top of the action buttons container. This is is in order to allow the action buttons to settle into their position after the keyboard has moved out of the way, since the callbacks are called too quickly.
Also replaces deprecated Java systemWindowInsetTop property (sorry, I used Bard/Gemini for that).

[x] Waits a ~100ms delay before attempting to set the progress bar position.
[x] Replaces deprecated  systemWindowInsetTop with WindowInsetsCompat API.

Clicked on a text task, wait for the keyboard to show up, then moved to the next and previous tasks while the action buttons are pushed up.

@gino-m  PTAL!

Thanks @sufyanAbbasi ! Note that the progress bar still isn't visible when the keyboard is active. Also, ideally we wouldn't need the delay. I've experimented with embedding the progress bar in the task view in https://github.com/google/ground-android/pull/2287; the layout works well, but the progress state isn't set correcting on each page change.


IL2CPP from Microsoft

Hello!
I know that you know about IL2CPP by Unity Technologies (because Microsoft is Unity’s partner).
I think that .NET Native is a very good tool but it would be even better if you could wrap .NET in pure C++ and make MSBuild convert .NET assemblies to C++ or ASM so developers can build them standalone.
Best,
Andrey

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

ID: de0709f4-087a-4192-01d1-61cc8665b6ba
Version Independent ID: 1492eb8c-d2c9-55e6-dea8-11d7c057cec1
Content: Optimize your .NET Desktop apps with native images - UWP app developer
Content Source: windows-apps-src/porting/desktop-to-uwp-r2r.md
Product: windows
GitHub Login: @rido-min
Microsoft Alias: normesta

Thank you for your suggestion. Better interop with other languages is definitely on the .NET roadmap, for all platforms, not just UWP.
https://github.com/dotnet/coreclr/issues/19566, https://github.com/dotnet/corert/pull/5703 or https://github.com/mono/embeddinator-4000 are discussions and prototypes related to this that you may find interesting.


Added support for docker volume API (for Docker 1.9)

This adds docker volume basic commands (ALL, GET, CREATE and DELETE). It enabled to use docker volumes command.
First time writing ruby and first time using rake so i'll happily read your comments and thoughts

@cbalducci awesome, I've got a PR that I'm looking to merge real soon that'll remove all the VCR files.  After that, can you rebase your branch and remove any additional files you've got there? That way we can really analyze these additions :) Thanks!

@tlunter sure, i suppose you're talking about PR https://github.com/swipely/docker-api/pull/340, right?

That's the one!

done

Finished in 7 minutes 51 seconds (files took 0.2551 seconds to load)
173 examples, 0 failures, 3 pending

Hey @tlunter, what's blocking this from getting merged? let me know if i can help :)

@cbalducci you'll need to rebase before we can merge. will review after that.

tried to rebase, it's a mess (mainly because i'm not very good at git)
Since cbalducci/add-volume can merge with swipely/master it's ok if we close this PR and open another one?

That's what this PR is attempting to do. I'll see if I can do a rebase later on today.

this would make what I'm working on soooo much easier.
+1 for getting this merged and will help if possible

Fixed up and released in v1.26.0.  Thanks @cbalducci

@tlunter awesome! thank you and sorry for the mess :smile:

This pr could probably be closed??
I think we could also add some info on the new functionality in the README.md?


unifi: 5.6.30 -> 5.6.36

Release Notes
(cherry picked from commit 754911847e2a5ab3bebb252facd222ae4f87c3c9)
Motivation for this change
Follow up of https://github.com/NixOS/nixpkgs/pull/37351
/cc @peterhoeg
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Thanks @badi!


Handle FS using VFS POSIX API

Is POSIX style VFS API supported? I want to use this library in conjunction with SQLite 3 database for ESP32, all running in PSRAM (allocations for SQLite and the database files itself).
By the way, awesome project 😎

hello @ianwillianb and thanks for your feedback :+1:
PSRamFS vfs compliance has been tested with rrdtools and should work as is with SQLiteEsp32.
Please let me know how if you face any issue.

Hello @tobozo, thanks for the quick reply.
I'll definitely try it out 🙂


Remove flushability of streams when stream window size is set to zero.

Motivation:
When setting the stream window size for a stream which had data but
couldn't write it (because of a zero window size) the stream would be
marked as flushable. Setting the window size to zero again would not
mark it as unflushable. On the following window (non-zero) resize it
would be marked as flushable again. However, since this is preceeded by
an assertion that it wasn't already marked as flushable it would cause a
crash.
Modifications:

Mark the stream as unflushable if it previously had data which it
couldn't write.

Result:
Fewer crashes

@Lukasa I had to build this against NIO commit<PHONE_NUMBER>3ec8a0b36e67108dd352b2fe1c37e6 since there are breaking changes after that which haven't been resolved in nio-http2 yet.

@swift-nio-bot add to whitelist


Suggestion for optional parameter with type 'never'

The type never currently just mean the function return that will never arrive or the unexpected value after type inference.
I think the type never can be more powerful.
optional parameter: using type never
Since undefined has two meaning: not existed and existed but no value, it would be better to split it out to have the clearest meaning.

It currently uses the ? operator to define a optional parameter:
interface IExample {
  a?: string;
}
const test1: IExample = {
  a: undefined, // expected error, but passed
};

function funcExample(a?: string) {}
funcExample(undefined); // expected error, but passed

It is expected to be error, but it is passed because it was resolved as the following code in strictNullChecks mode:
interface IExample {
  a?: string | undefined
}
function funcExample(a?: string | undefined): void

where the undefined should not appear since it just can be string or void, so I think it should be resolved as:
interface Example {
  a: string | never
}
function funcExample(a: string | never) {}

and expect its behavior as:
const example1: Example = {
  a: undefined; // show error message
};
const example2: Example = {}; // expected passed, but error now

funcExample(undefined); // show error message

For more about never with function optional parameter:
function neverExample(a: never) {}
neverExample(); // expected passed, but error now

Why dont I just use the following code?
function neverExample() {}

Consider the following generic class:
abstract class Matcher<T> {
  public abstract rawMatch(target: T): boolean;
  public match(target: T): boolean {
    return this.rawMatch(target);
  }
}
class TestMatcher extends Matcher<never> {
  constructor(public bool: boolean) {
    super();
  }
  public rawMatch(): boolean {
    return this.bool;
  }
}
const result = new TestMatcher(true).match(); // expected passed, but error now

It will be more convenient if we can use never with function in generic class.
The only way to solve this problem now is to override the method:
class TestMatcher extends Matcher<null> {
  constructor(public bool: boolean) {
    super();
  }
  public rawMatch(): boolean {
    return this.bool;
  }
  public match(): boolean {
    return super.rawMatch(null);
  }
}

alias for { [key: string]: T; }
{ [key: string]: T } is a type that we use it frequently, but it looks not comfortable like number, object, etc.
I'd like to suggest to have an alias for it globally, for example:
type dictionary<T> = { [key: string]: T; };

function setData(data: dictionary<any>) { ... }
function setData(data: { [key: string]: any }) { ... }

function setData(data1: dictionary<any>, data2: dictionary<any>, something: any) { ... }
function setData(data1: { [key: string]: any }, data2: { [key: string]: any }, something: any) { ... }

disabling rules with inline comments
I don't know if this is duplicate ( issue 11051 ) or not, since the following code is 100% error in type-checking but 100% correct in logic.
Type inference is powerful, but sometimes it'll block some convenient usage.
Given the following class, the generic is consider to be a state of it:
type MapCallback<Input, Output> = (value: Input) => Output;

abstract class Querier<Target, Result> {

  public mapCallback: MapCallback<any, any>;

  public abstract rawQuery(target: Target): Result[];
  public query(target: Target): Result[] {
    const results = this.rawQuery(target);
    return results.map(this.mapCallback);
  }

  public map<MappedResult>(callback: MapCallback<Result, MappedResult>): Querier<Target, MappedResult> {
    this.mapCallback = callback;
    return <Querier<Target, MappedResult>>this; // error: cannot be converted
  }

}

The map method will change its state, for example:
class SomeQuerier extends Querier<object, number> { ... }
const someResults =
  new SomeQuerier() // Querier<object, number>
    .map((num: number) => num.toString()) // Querier<object, string>
    .query(someObject); // string[]

So I think is there  a way to provide some inline comments to disable rules, just like eslint and tslint did?
public map<MappedResult>(callback: MapCallback<Result, MappedResult>): Querier<Target, MappedResult> {
    this.mapCallback = callback;
    // tsc:disable-next-line type-convert-checking
    return <Querier<Target, MappedResult>>this;
  }

Edit: I found the following part is duplicate ( issue 10727 ), just ignore it.
Relative complement operator
We have union operator ( | ) now, from myType1 to myType2:
type myType1 = string | number;
type myType2 = myType1 | boolean;
// myType2 = string | number | boolean

But we can't do the opposite way from myType2 to myType1:
type myType2 = string | number | boolean;
type myType1 = myType2 - boolean; // we cannot do this now
// expected myType1 = string | number;

If we can have the complement operator ( - ), it will be easily to use in some Partial case (e.g. options ):
interface PartialOptions {
  a?: string;
  b?: number;
  c?: {
    c1: string;
    c2: number;
  }
}

If we want to get types within c, it is impossible to do this now since c can be undefined:
type cnames = keyof PartialOptions['c'];
// cnames = never, since PartialOptions['c'] could be undefined
type ctypes = PartialOptions['c'][cnames]; // error: Type 'never' cannot be used as an index type

The only way to get types now is to copy it manually. but we have to copy it again while PartialOptions changed:
type ctypes = string | number;

If we can use complement operator:
type EntireOptions = {
  [T in keyof PartialOptions]: PartialOptions[T] - undefined;
}
// expected EntireExmaple = {
//   a: string;
//   b: number;
//   c: {
//     c1: string;
//     c2: number;
//   }
// }
type cnames = keyof EntireOptions['c'];
// expected cnames = 'c1' | 'c2'
type ctypes = EntireOptions['c'][cnames];
// expected ctypes = string | number

never is the bottom type, it is assignable to everything. For the distinction between undefined/missing I do not think never is the right type.
You need a type that is for read is undefined and for write is a subtype of undefined. Issue https://github.com/Microsoft/TypeScript/issues/13195 tracks doing so.


[ISSUE-89] Completed list of all javax.servlet.* that should be transformed

Fixes https://github.com/wildfly-extras/batavia/issues/89

Sorry, I didn't see this pr before.  I'll create a pr for comparison with my changes after partial merge with OpenLiberty.

https://github.com/wildfly-extras/batavia/pull/91 brings the same changes in and some more I think.  It also allows us to sync with https://github.com/OpenLiberty/open-liberty/blob/integration/dev/wlp-jakartaee-transform/rules/jakarta-direct.properties as it updates to follow the same organization for properties.


Implement a __main__.py with the functionality of blinding a SACC file

Implement a main function that can be called from the terminal and blinds a  SACC file provided by the user.

This is going to be challenging because the build_likelihood() methods read automatically a sacc file.
The firecrown likelihood families demand that .read() method is called instantly and this method actually somehow filters the required data-points/binning/etc.. the likelihood "knows" how to get the theory-vector in the same shape of the (internally) selected data-vector. But there's no way of extracting that information from the likelihood so that it can be applied to an external data-vector, loaded outside the likelihood.
This really complicates things for blinding. The solution found so far would only work for a data-vector that is already being loaded into the firecrown likelihood.

This is somewhat related to https://github.com/LSSTDESC/firecrown/issues/394#issuecomment-1960949419

@tilmantroester pointed out that jsonargparse module could be super useful for this!
https://jsonargparse.readthedocs.io/en/stable/#


能否更换为自有TG账户的API?

这个图床能否更换为自己的TG账户api，并且保存至收藏夹？

同问

他好像是把图片上传到 https://telegra.ph/

This repo is none of Telegram's business. Even TelegraPh is a sub company of Telegram, they are two different services too. Telegram file link is only a hash code and download url will expire without login session (including bot login), so function should login every time that needs more time to process or cache login session that not safe for your account. I suggest to close this issue as no plan.


MDE behaves differently when submitted via mouse click vs keyBinding

Package
filament/forms
Package Version
3.2.80
Laravel Version
11.7.0
Livewire Version
3.4.12
PHP Version
8.2.18
Problem description
The behavior of clicking on submit vs pressing the registered keyBinding() is different.
Clicking will clear the form. (correct)
Pressing the shortcut key does not. (incorrect)
Expected behavior
Pressing the shortcut key should also clear the form.
Steps to reproduce

Type something into the markdown editor at /admin/issue
Press ctrl + enter

Reproduction repository
https://github.com/bilogic/filament-issue/tree/click-different-from-keyBindings
Relevant log output
https://github.com/filamentphp/filament/assets/946010/fb144bf9-802f-4003-8dcd-ec4c2148e742
No response

It's because of this line: https://github.com/filamentphp/filament/blob/98249512b3135475e26bf14b14103280f927096d/packages/forms/resources/js/components/markdown-editor.js#L208
When the editor is focused, the state is not changed.
@danharrin if you think this is a bug, I can create a PR.

Its deliberate, to prevent an infinite loop of state updates when the editor is being typed into

I guess. Still looking into it.

Perhaps ctrl + enter can be made to steal the focus first before submitting?

Thank you @bilogic. After checking other solutions, this one seemed to be the best. Let's see if ok for merging.

Thanks for the fix!

I'm concerned about the new UX. I don't think you should be defocused from the input when you save, imagine you are writing a long paragraph and want to save as you go with a keyboard shortcut, you don't want to lose your place necessarily

How about having another parameter on ->keyBindings(['mod+enter'], stealFocus: true); to steal focus with default set to false?
Or is there a way to provide a closure to ->keyBindings() that steals focus first? Then we won't have to change anything in filament.


[helm]support config override in starrocksFESpec/starrocksBeSpec/starrocksCnSpec

Describe the current behavior
Currently all the config in starrocks helm chart are taken as a block, see the code here. As a result, partial update is not supported.
Describe the enhancement
Support partial update in values.yaml to simplify configuration changes.
Additional context
cc @dengliu @yandongxiao

The "partial update" you mentioned refers to adding or updating some additional configurations in other values.yaml files on top of a certain base values.yaml configuration?

The "partial update" you mentioned refers to adding or updating some additional configurations in other values.yaml files on top of a certain base values.yaml configuration?

Yes,

This is a bit challenging because the current type of config is a string. We need to parse this string in _helper, convert it into the format of a map, and finally merge it with the default values. I tried to write it, but failed.


Proposal logo/Icon

hi, i want to contribute about this great project. I'm not saying your logo is ugly. if you allow me to propose a new logo / icon for you. thank you

go ahead, as long as you made it yourself, and not stolen from flaticons like ive already exposed two other people doing and got them banned from utopian.io :)

of course, I will design a new logo without changing the old logo construction, just a little improvement to look more tidy. hope you like it.😄

@amirdesaingrafis sorry if me saying that seemed a bit confronting lol, im just tired of having to deal with people who just steal icons from websites (and say they made them themselves, which really drives me up walls) to try to "contribute" to open source projects for money (not to mention potential legal problems, as the icons are licensed), and then lash out at me when i call them out for it. usually they also have shitty grammar as well, so its pretty easy to tell who's fake and who isn't


Enabling Dialog Presentation Over Native Screens

If user has performed some action or failed some action, I want to a show dialog on the view I open. How can I do that? I can listen the handling events but I can't show the dialog use with context, it stays under the view.
https://docs.adapty.io/docs/paywall-builder-events-flutter

Hi @ilknurimamhaliloglu, thank you for reaching out. At the moment, we don't support these options. If you plan to present anything above the standard native view, you'll need to implement it yourself, which I believe is only feasible with native code.
But, I think your request is quite a good enhancement, and we will consider adding this option in future releases.

Thanks for your answer, I will be waiting for the enhancement.

Hey @ilknurimamhaliloglu! Good news – we've added the requested feature to our new version 2.1.0. Simply use the .showDialog method of the AdaptyUIView, and you're good to go! Please consider updating.

I want to redirect to a bottom sheet or any other page within the openUrl section of the paywall, but I'm unable to do so because all such attempts stay behind the paywall. How can I resolve this issue?


chore: bdd-nh-import-1585970338 to 0.0.1

chore: Promote bdd-nh-import-1585970338 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


Task naming for general chunkmanagers

What is your issue?
(Follow-up to #7019)
When you create a dask graph of xarray operations, the tasks in the graph get useful names according the name of the DataArray they operate on, or whether they represent an open_dataset call.
Currently for cubed this doesn't work, for example this graph from https://github.com/pangeo-data/distributed-array-examples/issues/2#issuecomment-1533852877:

cc @tomwhite @dcherian

If you hover over a node in the SVG representation you'll get a tooltip that shows the call stack and the line number of the top-level user function that invoked the computation. Does that help at all? (That said, I'm open to changing the way it is displayed, or how tasks are named in general.)
BTW should this be moved to a cubed issue?

If you hover over a node in the SVG representation you'll get a tooltip that shows the call stack and the line number of the top-level user function that invoked the computation. Does that help at all?

That's neat!

When you create a dask graph of xarray operations, the tasks in the graph get useful names according the name of the DataArray they operate on

I realise now that this is not true - but can we make it true for cubed in xarray? Using cubed with xarray creates array's with names like array-002, but couldn't we use the dataarray's .name attribute to give this node of the graph the name "U" for example?

BTW should this be moved to a cubed issue?

I raised it here because it relates to an unfinished part of #7019 - where there is still dask-specific logic for naming individual tasks. I think that to solve this we will need to alter xarray code to allow ChunkManager objects to decide how they want to name their tasks, but using information passed from xarray (i.e. DataArray.name).


persistent notification if exposure logging is active

Current Implementation
The user has to open the app in order to see if the exposure logging is active.
Suggested Enhancement
Add a persistent notification wich tells the user if exposure logging is active. If the user wants this notification should be adjustable in the settings.
Expected Benefits
The user can easily see if the logging is active and doesn´t need to open the app.

Maybe a separate issue, but should be solved as well in this context: According to FAQ activating location is as bluetooth prerequisite for risk identification on android devices. When turning bluetooth off the cwa risk identification changes to off as well. But when setting loation to off the cwa risk identification stays on. Thus for the user location seems not be necessary.

@Spirit100 this is already a known issue but thanks for mentioning!

Dear @ChillerSargant ,
thank you very much for your proposal.
To give the community a space for discussing new feature ideas or enhancements that might not be taken up immediately by the development teams, we created the cwa-wishlist repository.
This idea is one of these feature ideas or enhancements. Consequently, we will move this issue to this repository and allow the community and you to further refine it, discuss pros and cons, and evaluate alternatives.
The issue will not be closed in this repository to ensure long-term visibility. All issues in the cwa-wishlist repository will be discussed with our contracting entities.  As soon as we have any updates, we will let you know about the details.
Best regards,
TK
Corona-Warn-App Open Source Team


Fix - Creating an address fails with message "address space plan '' not found"

Type of change

Bugfix

Description

Handled null check for fetching queries to get address types and plans in create Address configuration view.
Fix - #4279

Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests testcase=*ConsoleTest

:heavy_check_mark: Test Summary :heavy_check_mark:
TEST_PROFILE: pr
TEST_CASE: *ConsoleTest
TOTAL: 77
PASS: 71
FAIL: 0
SKIP: 6
BUILD_NUMBER: 388


Google Pagespeed Insights Says Browser Cache isn't Being Leveraged

If you feel the issue is an EasyEngine core specific issue, please attach the output of the following commands.
System Information
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 18.04.1 LTS
Release: 18.04
Codename: bionic
EE 4.0.0-rc.1
+-------------------+----------------------------------------------------------+
| OS | Linux 4.15.0-23-generic #25-Ubuntu SMP Wed May 23 18:02: |
| | 16 UTC 2018 x86_64 |
| Shell | /bin/bash |
| PHP binary | /usr/bin/php7.2 |
| PHP version | 7.2.11-3+ubuntu18.04.1+deb.sury.org+1 |
| php.ini used | /etc/php/7.2/cli/php.ini |
| EE root dir | phar://ee.phar |
| EE vendor dir | phar://ee.phar/vendor |
| EE phar path | /root |
| EE packages dir | |
| EE global config | /opt/easyengine/config/config.yml |
| EE project config | |
| EE version | 4.0.0-rc.1 |
+-------------------+----------------------------------------------------------+
wp: command not found

I was asked to submit a new issue for this as instructed here

https://github.com/EasyEngine/easyengine/issues/1272#issuecomment-435753043

I created a test website with EE4 RC1 2 or 3 days ago and wanted to know if I needed any extra caching plugins to compliment what I had already setup.
My setup command was. . .

ee site create shaylahleigh.com --type=wp --cache --admin-user=user--admin-pass=pass --ssl=le

But when I go to Google Pagespeed Insights, I am told that I am not leveraging Browser Cache and am told the following files are being ignored in the caching process. . .

https://shaylahleigh.com/…ntyseventeen/assets/js/global.js?ver=1.0 (24 hours)
https://shaylahleigh.com/…n/assets/js/jquery.scrollTo.js?ver=2.1.2 (24 hours)
https://shaylahleigh.com/…assets/js/skip-link-focus-fix.js?ver=1.0 (24 hours)
https://shaylahleigh.com/…emes/twentyseventeen/style.css?ver=4.9.8 (24 hours)
https://shaylahleigh.com/…s/jquery/jquery-migrate.min.js?ver=1.4.1 (24 hours)
https://shaylahleigh.com/…-includes/js/jquery/jquery.js?ver=1.12.4 (24 hours)
https://shaylahleigh.com/…wp-includes/js/wp-embed.min.js?ver=4.9.8 (24 hours)
https://shaylahleigh.com/…des/js/wp-emoji-release.min.js?ver=4.9.8 (24 hours)

So I was curious if I needed to use a caching program like WP Rocket, W3 Total Cache, Super Cache or any other plugin to properly cache the files Google is complaining about with the default 2018 WP theme

If you remove query strings from your js and css files you should get rid of this issue pretty easily. You can use a free plugin like Machete (which has an option to do this, but also quite a few other options to remove unnecessary features from your header). There's also other free plugins called query strings remover, and Remove Query Strings From Static Resources.
Then your js and css files will be cached like your other files, according to your settings. I've found that this drastically decreases load times on my website, down to ~1.4 seconds now under heavy load

If you remove query strings from your js and css files you should get rid of this issue pretty easily. You can use a free plugin like Machete (which has an option to do this, but also quite a few other options to remove unnecessary features from your header). There's also other free plugins called query strings remover, and Remove Query Strings From Static Resources. The latter two don't need you to even tick any boxes to enable the feature, just activate the plugin and flush your cache.
Then your js and css files will be cached like your other files, according to your settings. I've found that removing query strings is a big part in drastically decreasing load times on my website, down to ~1.4 seconds now under heavy load. If I leave query strings in, my load times are around 1.5 seconds longer. This is especially important if you use a 3rd party theme / plugins.
It would be cool if EE installed and enabled one of these plugins when you first set up a wp website with the --cache flag imo.

I am a fan of Autoptimize and it has a feature with the description. . .

Removing query strings (or more specifically the ver parameter) will not improve load time, but might improve performance scores.

I will give that a shot. And if it fails then I'll try the Machete plugin that you suggested.

I tried Machete and Autoptimize, they can remove the query strings from the static resources, but pagespeed keeps complaining about leveraging browser cache.
There's a year and half old post about it on the rtcamp forum about it: https://community.easyengine.io/t/how-do-i-enable-browser-caching/8440
I noticed there is also a custom folder with a user.conf file for the sites created: /var/lib/docker/volumes/tacensicom_config_nginx/_data/custom/user.conf
I tried inserting those rules to leverage the browser cache but the user.conf doesn't seem to be included in the nginx.conf file.
So I wonder if someone could point me on how to include it.
Also: would not be better to have it on by default?

I tried inserting those rules to leverage the browser cache but the user.conf doesn't seem to be included in the nginx.conf file.

@tacensi Did you try ee site restart/reload example.com after doing it?

@kirtangajjar
Yes indeed.
Here is the headers for one of the scripts googlespeed is complaining about:
curl -sD - -o /dev/null https://tacensi.com/wp-includes/js/wp-emoji-release.min.js
HTTP/2 200
server: nginx
date: Thu, 08 Nov 2018 17:10:53 GMT
content-type: application/javascript
content-length: 12026
last-modified: Tue, 06 Nov 2018 18:16:52 GMT
vary: Accept-Encoding
etag: "5be1da94-2efa"
expires: Fri, 09 Nov 2018 17:10:53 GMT
cache-control: max-age=86400
cache-control: public, must-revalidate
accept-ranges: bytes

Is there a way to check if it is getting loaded?

@kirtangajjar
I took a time to look at it a little more. Actually, user.conf is being loaded, but the problem is all the cache directives are in the main.conf file, so user.conf cannot overwrite them.
Images and media already have expires max, but scripts and styles have expires 1d. Woudn't it be better to change to 1M?

@tacensi @Endda Did you end up figuring out how to change the expiry date on EE4 RC1? Having the same issue on my end

Ok I've found the proper directories to edit the right nginx conf file for my sites. I've changed my cache expiry to 30d and flushed the caches but everything still comes up as 1d. No idea why it's set as 1d by default -- you'd expect js and css files to have a very long expiry time as they are rarely changed.

@kirtangajjar

So as @tacensi and @b167er have discovered, is there a reason why the cache is being defaulted to 1 day and not allowed to be set to any longer?

Google Pagespeed Insights are very important for a stable web stack and if Google is recommending that the browser cache needs to be leveraged better, then it should be that way by default. Or at least let us change it.
I get it, it's still RC1 and all, but is this issue on the radar and set to be fixed?

Yes. We're tending to all the issues and this is set to be fixed before the stable release.

@Endda Closing this issue as it has been fixed in v4.0.


ci: doc-build: change Doxygen download URL

It looks like versions >= 1.9.2 have disappeared from the Sourceforge website. While problem persists, use the Doxygen domain download URL (only hosts latest).
Ref. https://github.com/doxygen/doxygen/issues/9801
Signed-off-by: Gerard Marull-Paretas<EMAIL_ADDRESS>

DNM: looks like they've moved to GH releases, will use that instead.


nexus-publish-plugin + gradle-nexus-staging-plugin = WrongNumberOfRepositories

I'm trying to set up publishing according to this article.
Here's my repo: https://github.com/bozaro/git-lfs-java/tree/6464e5d36cb4f7c8f1579a46b740e66e44a11d9f
Travis calls gradle publishToNexus closeAndReleaseRepository: https://github.com/bozaro/git-lfs-java/blob/6464e5d36cb4f7c8f1579a46b740e66e44a11d9f/.travis.yml#L43
nexus-publish-plugin is set up for child projects: https://github.com/bozaro/git-lfs-java/blob/6464e5d36cb4f7c8f1579a46b740e66e44a11d9f/build.gradle.kts#L43
nexus-staging-plugin is set up for root project: https://github.com/bozaro/git-lfs-java/blob/6464e5d36cb4f7c8f1579a46b740e66e44a11d9f/build.gradle.kts#L154
But still, I'm getting:
DEPRECATION WARNING. The staging repository ID is not provided. The fallback mode may impact release reliability and is deprecated. Please consult the project FAQ how it can be fixed.
Requested operation wasn't successful in first try. Retrying maximum 20 times with 2 seconds delay between attempts.

Followed by build failure:
WrongNumberOfRepositories was thrown with message 'Wrong number of received repositories in state 'open'. Expected 1, received 7'. Giving up. Configure longer timeout if necessary.

Full build log: https://travis-ci.org/bozaro/git-lfs-java/jobs/533836201
What I am missing?

Okay, I think I know what's the problem. I'm trying to use all this machinery for SNAPSHOT build.

Aaaargh. I cannot enable staging for SNAPSHOTs because sonatype doesn't support staging for snapshots. So, seems like the only option is to write conditional logic that will call closeAndReleaseRepository depending on whether we're building a SNAPSHOT or not.

@slonopotamus Definitely. Staging repositories doesn't make much sense for SNAPSHOTs.
I would just use onlyIf { isSnapshot() } where isSnapshot is a method which (most likely) checks if version ends with -SNAPSHOT (I believe I have seen that checks in the Gradle itself, but maybe now there is some better option to write that if).

That Travis build after your change looks somehow strange:
Attempt 10/61 failed. RepositoryInTransitionException was thrown with message 'Repository 'ru[secure]-1032' (in 'open' state) is in transition. Check again later.'. Waiting 10000 ms before next retry.
Getting repository 'ru[secure]-1032'
Requested operation was executed successfully in attempt 11 (maximum allowed 61)
> Task :closeRepository FAILED
:closeRepository (Thread[Daemon worker,5,main]) completed. Took 2 mins 52.871 secs.
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':closeRepository'.
> Wrong 'ru[secure]-1032' repository state 'open' after transition (expected [closed]). Possible staging rules violation. Check repository status using Nexus UI.

Can you check that repository in the Nexus UI. There could be some validation errors presented.

I'm not quite sure what stuff in gradle script you're suggesting to guard with onlyIf.
I could imagine something like: create new task like publishProperly, make it depend on publish and for releases make it also depend on closeAndReleaseRepository.
I disagree with "Staging repositories doesn't make much sense for SNAPSHOTs". For multimodule projects, I want atomic deploys to repo, no matter if it is snapshot or release. Only staging procedure gives such guarantees.

Or you mean that I always call closeAndReleaseRepository but in build.gradle do this?
tasks.closeRepository.configure {
    onlyIf { !project.version.toString().endsWith("-SNAPSHOT") }
}

tasks.releaseRepository.configure {
    onlyIf { !project.version.toString().endsWith("-SNAPSHOT") }
}

If yes, maybe this logic should go to nexus-publish plugin?

nexus-staging in general is upload/publish agnostic. You could run it even for the artifacts published earlier (e.g. just by passing the staging repository ID). The plugin doesn't know what is in that repository.
However, in general it most of the cases where nexus-publish integration is enable the current project version determines if it is a snapshot or not.

Btw, in Groovy you could write something like that:
[tasks.closreRepository, tasks.releaseRepository]. configure {
    onlyIf { !project.version.toString().endsWith("-SNAPSHOT") }
}

to reduce duplication. Probably in Kotlin it is also possible.

For multimodule projects, I want atomic deploys to repo, no matter if it is snapshot or release.

Then you can just resign from using -SNAPSHOT and just bump a version number by one on every commit/push or CI build.

Anyway. I managed to resolve my problem (by using onlyIf like you suggested). If you believe there is nothing to improve on nexus-publish/nexus-staging side for projects that deploy both snapshots and releases, feel free to close this issue.

I think this should work and remove the duplicated logic:
tasks {
	closeRepository {
    	onlyIf { nexusPublishing.useStaging.get() }
	}
	releaseRepository{
    	onlyIf { nexusPublishing.useStaging.get() }
	}
}

	closeRepository {
    	onlyIf { nexusPublishing.useStaging.get() }
	}

This is pretty good, but won't work in multi-project setups where parent project is not published, so it doesn't have nexus-publish plugin and thus doesn't have nexusPublishing section.


Need code to create cross account alarms in monitoring account.

Need code to create alarms in monitoring account from the metrics defined in another accounts.cross account sharing role is already enabled.

Answered in https://github.com/gruntwork-io/knowledge-base/discussions/139


quick POC at non-Long numerics (encoding and decoding)

A quick stab at showing how non-Long numbers could be encoded and decoded.
#163
This is a very quick draft, I haven't spent a lot of time on it. I'm just sharing because I think the general approach is good.
This approach is based off the Kotlinx Serialization approach to handling JSON numbers
https://github.com/Kotlin/kotlinx.serialization/blob/dc950f5098162a3f8dd742d04b95f9a0405470e1/formats/json/commonMain/src/kotlinx/serialization/json/internal/StreamingJsonDecoder.kt#L281-L290
First the numbers are parsed as a Long or Double, as is already the case. Then ktoml will try to manually convert these numbers to the required type, but first verifying that the numbers are in range.
If you wanted to verify based on the raw string content, you could perhaps implement a function based on how KxS parses numbers: https://github.com/Kotlin/kotlinx.serialization/blob/dc950f5098162a3f8dd742d04b95f9a0405470e1/formats/json/commonMain/src/kotlinx/serialization/json/internal/lexer/AbstractJsonLexer.kt#L533-L584

@aSemy I will merge your part into #171 , I have made a simple part, could you please review?

@aSemy I will merge your part into #171 , I have made a simple part, could you please review?

Of course! Thanks for asking

POC completed - will be finalised in https://github.com/akuleshov7/ktoml/pull/171

no-no, let's not close it! I will merge it to get tests as a separate PR :)

Ahh okay, I misunderstood 👍


[BUG] Syntax class does not display all content

Describe the bug
When nesting a Syntax class (with line_numbers and word_wrap enabled) inside a Panel inside a Layout, only the first line is displayed.
To Reproduce
print(Layout(Panel(Syntax('import stuff\nwhile True:\n  print("hello!!!!")\n','python',line_numbers=True, word_wrap=True))))
Platform
Linux

Please try 9.12.0

Looks good to me! Thanks a ton, and awesome project!


System tests are failing / flaking

Was taking a look at this PR:
https://github.com/googleapis/nodejs-pubsub/pull/898
And noticed the system tests are failing when they probably shouldn't be:
https://source.cloud.google.com/results/invocations/ef1f2dcb-8913-4642-b5a1-4cafb02d0ae9/targets/cloud-devrel%2Fclient-libraries%2Fnodejs%2Fpresubmit%2Fgoogleapis%2Fnodejs-pubsub%2Fnode10%2Fsystem-test/log

  38 passing (4m)
  1 pending
  1 failing

  1) pubsub
       Topic
         should publish a message:
     Uncaught AssertionError [ERR_ASSERTION]: ifError got unwanted exception: Retry total timeout exceeded before any response was received
      at topic.publishMessage (system-test/pubsub.ts:228:16)
      at callbacks.forEach (src/publisher/message-queues.ts:87:44)
      at Array.forEach (<anonymous>)
      at Immediate.topic.request [as _onImmediate] (src/publisher/message-queues.ts:87:19)
      at repeat (node_modules/google-gax/src/normalCalls/retries.ts:83:23)
      at Timeout.setTimeout (node_modules/google-gax/src/normalCalls/retries.ts:124:13)

Related: https://source.cloud.google.com/results/invocations/c1b2cd19-3d88-4c98-a6e6-0fec30efe73e/targets
(via this PR: https://github.com/googleapis/nodejs-pubsub/pull/917)

These haven't really had problems lately, so at Justin's inquiry, I'm going to go ahead and close this.


Django 1.8 introduced the use of argparse - this switches to use it.

Management commands break in Django 1.10 due to the argparse switch; optparse (the
old way) was deprecated in 1.8 in favour of argparse. In 1.10, optparse is removed.
This switch removes support for Django 1.7, which as of Django 1.9's release is no
longer a supported Django version. (LaxOptionParser is no longer used.)

Perfect, thanks. I merged this #75, which wants a requires_system_checks = False option on the command class. Correct?

@miracle2k I'd like to see this commit released on PyPI, can you do one? Thx.


Typescript type declarations in the work?

Is there a separate project in the works for type declarations?  If not, I was thinking of starting one.

Hello @do917.
No there are no plans for typescript support at the moment.
So feel free to create it! :)


⚠ Account has degraded performance

In 4542bcf, Account (https://account.wrk.com/health) experienced degraded performance:

HTTP code: 200
Response time: 186 ms

Resolved: Account performance has improved in 407c2c0.


Dynamic center zoom bug

There is a bug when zooming after been setting center dynamic.
Map is grayed out or moved to a different location.
You can test it in on your own site: http://ngmap.github.io/#/map_with_dynamic_center.html

fixed. Thanks for reporting this bug.


IDisposable guidance for Transient and shared instances

Fixes #17766
Fixes #17775
Internal Review Topic (links to new IDisposable guidance for Transient and shared instances section)
The new section appears twice ... 2.x and 3.x parts of the doc ... so you only need to review the first instance of the section. See :point_right: Lines 454-487.
Thanks @brandondahler! 🎷

@mkArtakMSFT ... if it helps move this one along, David gave the original remarks a :+1: at https://github.com/dotnet/extensions/issues/1785#issuecomment-532972451.

@javiercn given that you're driving a related conversation, do we want to make this update now, or wait till you come up with some resolution?

Ping @javiercn :point_up:

@mjrousos ... I'm going to go ahead and get this in. If someone wants to float sample code, I'll drop it in later. I'm 🏃😅 trying to get caught up on Blazor issues for the next month ....... or two! 🏃🏃


[milvus-4.1.18] Enabling High Availability for Milvus

What this PR does / why we need it:
By enabling this feature, Milvus Instances will be resilient to failures, ensuring continuous availability and reliability for users. The implementation includes fault-tolerant mechanisms and redundancy configurations to mitigate downtime and main uninterrupted service in the event of node failures or network issues. Overall, this enhancement enhances the robustness and performance of Milvus, providing a more resilient infrastructure for data storage and retrieval.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [mychartname])
[x] PR only contains changes for one chart

/assign @loveeachday
@haorenfsa please validate.

context: #67

Hi @Rachit-Chaudhary11, a little advice, better continue the work on the same PR, so that we could tract the context of the problem directly.
For example, your previous work is on branch rollingupgrade, then you refine it on the branch deploymentStrategy.
So after you finish your work, you can checkout rollingupgrade again, and git reset deploymentStrategy --hard to make this branch the same as deploymentStrategy. Then you can use git push upstream rollingupgrade -f the force update your rollingupgrade branch on github.
In this way, we can now continue the progress on the previous PR opened with branch rollingupgrade.

/lgtm

/cc @LoveEachDay please help review

@haorenfsa

Hi @Rachit-Chaudhary11, a little advice, better continue the work on the same PR, so that we could tract the context of the problem directly.
For example, your previous work is on branch rollingupgrade, then you refine it on the branch deploymentStrategy. So after you finish your work, you can checkout rollingupgrade again, and git reset deploymentStrategy --hard to make this branch the same as deploymentStrategy. Then you can use git push upstream rollingupgrade -f the force update your rollingupgrade branch on github. In this way, we can now continue the progress on the previous PR opened with branch rollingupgrade.

I have done this to keep the alternative approaches in 2 separate branches as the other approach we are planning to use for some other feature. I hope you understand. Thanks

Hi @Rachit-Chaudhary11, a little advice, better continue the work on the same PR, so that we could tract the context of the problem directly.

@haorenfsa
I have done this to keep the alternative approaches in 2 separate branches as the other approach we are planning to use for some other feature. I hope you understand. Thanks

@Rachit-Chaudhary11 Only if we have enabled activeStandby for milvus coordinator components, we can use rollingUpdate strategy, otherwise the pod will crash when rolling update.
Could you also check whether rootCoordinator.avtiveStandby.enabled is true before change the deployment strategy?

@LoveEachDay kindly validate now.

@LoveEachDay @haorenfsa kindly validate now.

@LoveEachDay Thank you for the suggestion. I have the necessary changes, kindly validate.
cc: @haorenfsa

@haorenfsa @LoveEachDay kindly validate the changes.

@haorenfsa @LoveEachDay kindly let me know if there is anything else needed from my end.

/lgtm

@Rachit-Chaudhary11  Looks good. Thank you for the contribution!


🐛 Index of Bestiary not generating

When generating the sources, the Index of Bestiary only generates Planar Incarnate (planar-incarnate-mpp.md), therefore the the index only shows one entry.

Expected to have the index of the bestiary listed just as if one clicks on the dragon folder and the Index of Dragons are listed

Only one line generate ("Planar Incarnate
Configuration

OS Version: Windows
CLI version:  2.3.3
CLI type: Native zip, exe/command

Sources:

Majority of them
custom templates (attach if possible) - WIP