BUG: SciPy functions raise an error given lists of extreme integers

Describe your issue.
The example below leads to a TypeError since version 1.13.0 of scipy. Up to version 1.12.0, the output was ShapiroResult(statistic=0.9446643968771752, pvalue=0.6829615285403758), which (I believe) is the expected result.
As a first step toward debugging it: The change in behavior has been introduced by https://github.com/scipy/scipy/pull/19680
Reproducing Code Example
from scipy import stats

data = [10**20, -10**20, 10**30, -10**30] # Extreme values
r = stats.shapiro(data)
print(r)

Error message
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/scipy/scipy/stats/_axis_nan_policy.py", line 544, in axis_nan_policy_wrapper
    contains_nan = [_contains_nan(sample, nan_policy)[0]
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/scipy/scipy/_lib/_util.py", line 745, in _contains_nan
    if np.issubdtype(type(el), np.number) and np.isnan(el):
                                              ^^^^^^^^^^^^
TypeError: ufunc 'isnan' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

SciPy/NumPy/Python version and system information
1.15.0.dev0+1043.bbe993d 1.26.4 sys.version_info(major=3, minor=12, micro=3, releaselevel='final', serial=0)
Build Dependencies:
  blas:
    detection method: pkgconfig
    found: true
    include directory: /root/conda/envs/scipy-dev/include
    lib directory: /root/conda/envs/scipy-dev/lib
    name: openblas
    openblas configuration: USE_64BITINT=0 DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK=0 NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP=0 PRESCOTT MAX_THREADS=128
    pc file directory: /root/conda/envs/scipy-dev/lib/pkgconfig
    version: 0.3.27
  lapack:
    detection method: pkgconfig
    found: true
    include directory: /root/conda/envs/scipy-dev/include
    lib directory: /root/conda/envs/scipy-dev/lib
    name: openblas
    openblas configuration: USE_64BITINT=0 DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK=0 NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP=0 PRESCOTT MAX_THREADS=128
    pc file directory: /root/conda/envs/scipy-dev/lib/pkgconfig
    version: 0.3.27
  pybind11:
    detection method: pkgconfig
    include directory: /root/conda/envs/scipy-dev/include
    name: pybind11
    version: 2.12.0
Compilers:
  c:
    args: -march=nocona, -mtune=haswell, -ftree-vectorize, -fPIC, -fstack-protector-strong,
      -fno-plt, -O2, -ffunction-sections, -pipe, -isystem, /root/conda/envs/scipy-dev/include,
      -DNDEBUG, -D_FORTIFY_SOURCE=2, -O2, -isystem, /root/conda/envs/scipy-dev/include
    commands: /root/conda/envs/scipy-dev/bin/x86_64-conda-linux-gnu-cc
    linker: ld.bfd
    linker args: -Wl,-O2, -Wl,--sort-common, -Wl,--as-needed, -Wl,-z,relro, -Wl,-z,now,
      -Wl,--disable-new-dtags, -Wl,--gc-sections, -Wl,--allow-shlib-undefined, -Wl,-rpath,/root/conda/envs/scipy-dev/lib,
      -Wl,-rpath-link,/root/conda/envs/scipy-dev/lib, -L/root/conda/envs/scipy-dev/lib,
      -march=nocona, -mtune=haswell, -ftree-vectorize, -fPIC, -fstack-protector-strong,
      -fno-plt, -O2, -ffunction-sections, -pipe, -isystem, /root/conda/envs/scipy-dev/include,
      -DNDEBUG, -D_FORTIFY_SOURCE=2, -O2, -isystem, /root/conda/envs/scipy-dev/include
    name: gcc
    version: 12.3.0
  c++:
    args: -fvisibility-inlines-hidden, -fmessage-length=0, -march=nocona, -mtune=haswell,
      -ftree-vectorize, -fPIC, -fstack-protector-strong, -fno-plt, -O2, -ffunction-sections,
      -pipe, -isystem, /root/conda/envs/scipy-dev/include, -DNDEBUG, -D_FORTIFY_SOURCE=2,
      -O2, -isystem, /root/conda/envs/scipy-dev/include
    commands: /root/conda/envs/scipy-dev/bin/x86_64-conda-linux-gnu-c++
    linker: ld.bfd
    linker args: -Wl,-O2, -Wl,--sort-common, -Wl,--as-needed, -Wl,-z,relro, -Wl,-z,now,
      -Wl,--disable-new-dtags, -Wl,--gc-sections, -Wl,--allow-shlib-undefined, -Wl,-rpath,/root/conda/envs/scipy-dev/lib,
      -Wl,-rpath-link,/root/conda/envs/scipy-dev/lib, -L/root/conda/envs/scipy-dev/lib,
      -fvisibility-inlines-hidden, -fmessage-length=0, -march=nocona, -mtune=haswell,
      -ftree-vectorize, -fPIC, -fstack-protector-strong, -fno-plt, -O2, -ffunction-sections,
      -pipe, -isystem, /root/conda/envs/scipy-dev/include, -DNDEBUG, -D_FORTIFY_SOURCE=2,
      -O2, -isystem, /root/conda/envs/scipy-dev/include
    name: gcc
    version: 12.3.0
  cython:
    commands: cython
    linker: cython
    name: cython
    version: 3.0.10
  fortran:
    args: -march=nocona, -mtune=haswell, -ftree-vectorize, -fPIC, -fstack-protector-strong,
      -fno-plt, -O2, -ffunction-sections, -pipe, -isystem, /root/conda/envs/scipy-dev/include
    commands: /root/conda/envs/scipy-dev/bin/x86_64-conda-linux-gnu-gfortran
    linker: ld.bfd
    linker args: -Wl,-O2, -Wl,--sort-common, -Wl,--as-needed, -Wl,-z,relro, -Wl,-z,now,
      -Wl,--disable-new-dtags, -Wl,--gc-sections, -Wl,--allow-shlib-undefined, -Wl,-rpath,/root/conda/envs/scipy-dev/lib,
      -Wl,-rpath-link,/root/conda/envs/scipy-dev/lib, -L/root/conda/envs/scipy-dev/lib,
      -march=nocona, -mtune=haswell, -ftree-vectorize, -fPIC, -fstack-protector-strong,
      -fno-plt, -O2, -ffunction-sections, -pipe, -isystem, /root/conda/envs/scipy-dev/include
    name: gcc
    version: 12.3.0
  pythran:
    include directory: ../../../root/conda/envs/scipy-dev/lib/python3.12/site-packages/pythran
    version: 0.15.0
Machine Information:
  build:
    cpu: x86_64
    endian: little
    family: x86_64
    system: linux
  cross-compiled: false
  host:
    cpu: x86_64
    endian: little
    family: x86_64
    system: linux
Python Information:
  path: /root/conda/envs/scipy-dev/bin/python3.12
  version: '3.12'

np.asarray([10**20, -10**20, 10**30, -10**30]) results in an object array. I suspect this happens in a few other functions.

One possibility could be to cast the input to float64 before further processing. This may be appropriate for special functions, which always work with floating-point numbers anyway.  (And this could also be done by the ufunc machinery so that no code duplication is needed.)
For stats such casting would disallow certain usage with non-floating input, e.g. it appears some users run a statistical test using input data with unit attached and expect the output to have unit attached. It seems to me impossible to ensure type safety and support duck typing at the same time.

Thanks @michaelpradel, but since this behavior is not necessarily wrong (not all functions are supposed to be defined for object arrays, and lists of very large ints are converted to object arrays), I don't think we can do much here unless NumPy sets a stronger precedent.
import numpy as np
x = [10**20]
np.sin(x)
# TypeError: loop of ufunc does not support argument 0 of type int which has no callable sin method

If they change the behavior for ufuncs, that would carry over to SciPy in many cases, and then it might be worth taking a look at the other cases. In the meantime, users can very easily resolve the problem by adding a period after any one of the numbers if this should occur in practice and if they intend for the argument to be treated as a floating point dtype. All things considered, this is probably a bit better than SciPy preemptively changing every function to which this issue applies.


Do not crash when reading a Scala 2 macro definition.

It contains an annotation with a subtree that is an IDENTtree without prefix nor symbol, with the name macro. That is then Applyed and TypeApplyed in ways that make no semantic sense.
We define a special symbol in Definitions, which we attach to such fake identifiers.

Only for things that we can find in the standard library. We don't have a setup for other weird stuff that we only find elsewhere, unfortunately.

What about adding a Scala 2 test project like this one?
https://github.com/scalacenter/tasty-query/blob/917d4d6111d49820b070eb784ba2eb601ef6937d/build.sbt#L64C1-L72C4

All right, I added a scala2TestSources project to be able to test targeted Scala 2 stuff, and I added a specific test for the fix in this PR.


Refactor standalone e2e and add support for Windows runner

Signed-off-by: Shubham Sharma<EMAIL_ADDRESS>Description
This PR achieves two things majorly

Refactor the e2e test suite for standalone mode - it decouples the tests and allows tests to be run independently while locally running them. It also covers scenarios for slim mode and Windows operating system.
Introduces Windows tests - Uses --slim mode to initialize Dapr without having to use containers (LXCs are unsupported on Windows GitHub agent). This will prevent manual Windows testing with each release.

I have also added some minor checks to the tests while refactoring them.
Issue reference
Please reference the issue this PR will close: #932
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[ ] Extended the documentation

@shubham1172 how are you guaranteeing the order of execution ie the publish test runs after init etc ...

@mukundansundar I am not. The tests should not be tightly coupled. Today if I make changed to Publish tests, I need to run the entire Standalone test suite. This change decouples the tests. There is a function call in Publish test to ensure that Dapr is installed. This ensures that locally anyone can selectively run tests.

Also a lot more tests are there in standalone init that needs to be added ...

Yes, this is a WIP PR, will add them shortly.

Are there any functionalities that do not get tested in the slim mode? VMs will be much heavier (might significantly increase e2e test time and cost).

The main reason I wanted a windows VM was to be able to run docker in the VM and test dapr init with docker in Windows.


Enable RTC functionality

This issue is a container for making use of the RTC.
Situation statement
A first step is to allow setting the datetime (and potentially the timezone) and having a secure way to determine a proper datetime is set. This is not a given as there is some tinkering involved to install an RTC battery (see this video by @rquellet  for details).
Potential use cases

DVR recordings can contain metadata with the recording datetime #116
DVR files could potentially contain a timestamp #42
The OSD menu could show the time (not sure about the value)

Implementation suggestion

Provide a settings entry to set time

This should allow setting

year, month and date
hours and minutes
Optionally set a time zone (otherwise the metadata will only be correct if the set datetime is given in UTC)

The setting should have an asterik (*) like the WiFi setting currently has. The footer should contain the warning

Time will be reset on reboot unless RTC battery is installed!

If there is a simple way to check whether an RTC battery is installed, the asterik could be shown conditionally with a message like

Time will be reset on reboot because no RTC battery is installed!

Currently that menu entry fits probably best in Record Options (already quite crowded) or Firmware (this could be renamed to System to be a little more generic)

Features that want to use the datetime should check for the following precondition (pseudo code) to ensure a plausible datetime is configured DateTime.now() >= thresholdDateTime

thresholdDateTime has to fulfil the following conditions (2023-01-01T00:00:00Z might match them)

The lower bound for thresholdDateTime should be a date older than the day this feature ships
The upper bound for thresholdDateTime should be at least some days after the datetime the RTC starts at after loosing power
The lower and upper bound together ensure that metadata is only set if a plausible datetime is configured

Lovely! Thank you.
i like that there is pre-planning for informing the user. feels like a good compromise to me.
Do you think you would be able to put forward a PR implementing this?

Here's an interesting workaround: what if we use ELRS backpack to push the RTC timestamp from radio to hdzero goggles?

Here's an interesting workaround: what if we use ELRS backpack to push the RTC timestamp from radio to hdzero goggles?

does not change much in actuality, as this is not something that we can rely on being preset.
so all the checks outlined above would still need to be in-place.
manual entry is just one possible way of obtaining the timestamp, elrs, wifi, etc are all possible inputs to this system.

I really like the layout of this plan.  My first goal was to have the RTC driver implemented prior to tackling this proposal.  I would like to break it down into a few phases.
Phase 1: RTC Driver Support (see pull request):
https://github.com/hd-zero/hdzero-goggle/pull/191
Phase 2: RTC Driver Enhancements:
New method which will generate a filesystem safe naming convention similar to ISO 8601:
YYYYMMDDTHHMMSS
20230310T011735
New method which will generate a pretty naming convention for use in the OSD:
Fri Mar 10 01:17:35 2023   or    2023/03/10 - 01:17:35
New method which can report the detection of altered RTC clock:
return year != 1970 && month != 1 && day != 1;
Phase 3: Display date/time within OSD menu via a new submenu:
Create new submenu for manipulating RTC date/time
Have the option to display the date/time if the RTC clock contains an non EPOCH date onto the OSD. (Use * to denote why it maybe disabled)
Provide the ability to modify the RTC Clock via this same menu: 6 fields containing their respective attributes for year/month/day/hour/min/sec
Phase 4: Alter naming convention of DRV recordings:
If the RTC clock contains an non EPOCH date, then automatically reference the filesystem safe naming convention method and prefix the formatted timestamp onto the name of the recorded file.
Phase 5 Investigate the use of metadata for timestamping and provide possible solutions... like the one described above.

@SumolX Happy to hear that you are working on this. And I like your implementation sketch overall 👍🏻.
Nevertheless I have some suggestions/remarks  (I hope that this is welcome):
General stuff
Time zones

I suspect the RTC hardware does not care for time zones and the OS would have to take care of this (then the RTC should contain UTC).
So if timezones should be considered it is a concern from Phase 2 onwards
Why would timezones be important:

the timestamps in video metadata contains timezone (for CreationDate) and expected to be UTC for all other fields (see Stack Exchange and the linked docs)
For the file timestamp it technically does not matter as FAT is used:

The FAT file system stores time values based on the local time of the computer.
from Microsoft's docs

For anyone in a timezone significantly away form GMT, setting the clock to UTC is no nice workaround I guess

due to it being shown in the UI
and it being not that intuitive

Order of Phases
I personally would moved some of the changes to a later stage:

Showing the clock in the OSD is something I would have tackled after Phase 5. I personally believe that this is not that important for most. If one wants to see the time one can use the Betaflight OSD ¯_(ツ)_/¯.
This would save an  (arguably simple) method in Phase 2 as well
I would implement the UI based date/time config to after Phase 4. Adding the txt file based on https://github.com/hd-zero/hdzero-goggle/pull/191 seems feasible for most who care enough (at least temporarily).
And I guess that modification menu item is the most complex part of the whole plan and not absolutely necessary to start

Remarks regarding specific phases
Phase 2
filesystem safe naming convention
I would use a slightly different filesystem naming convention: YYYY-MM-DDTHH_MM_SS I have two reasons for that:

my proposition is a lot more glanceable for a human
if someone needs to (for whatever reason) extract the timestamp from the filename, a simple sed 's/_/:/g' will produce an easy to parse ISO 8601 timestamp

pretty naming convention
If you choose the second option, I would make it a little more ISO-8601-like and loose the - as the date-time-separator:
2023-03-10 01:17:35 This is exactly the format Betaflight uses in the OSD
My motivation for this is internationalization. As long as there are no different localizations there will be just one type of "pretty" date. An ISO-8601 date would reduce ambiguity for anyone outside the US 🙈
Your first option Fri Mar 10 01:17:35 2023 seems okay as well, but:

In these Goggles, do we really need the day of the week?
The format uses a lot of space without adding any "real" benefit
If the Goggle menus would ever become localized, this function would need to change, with the other suggestion that would be optional

Phase 5

See Stack Exchange and the linked docs for the relevant tags
I think I saw ffmpeg being used. Based on Stack Overflow it can take care of setting all relevant date/time info using one parameter: https://stackoverflow.com/a/60534938

Thank you for the feedback.  Feel free to reach out to me on discord.  This will allow us hash this out in real-time vs going back and forth via comments.

@SumolX asked me to do some research on setting metadata for the recordings.
Unfortunately I failed to create a firmware which sets the desired timestamps with a dummy timestamp.
But I will share my results, so someone with any experience in C can have a look:

Using the ffmpeg cli the command to set the desired metadata is really simple:
 ffmpeg -i  hdz_009.mp4 -c copy -map 0 -metadata creation_time="2023-02-03 04:05:06"  test.mp4

successfully sets all relevant metadata timestamps to the specified time

exiftool (and to some extend ffprobe) can be used to check on the available metadata:
# Add the commented out stuff to only see the relevant 6 metadata fields
exiftool -g -sort hdz_009.mp4 # -G | grep Date

# shows the `creation_time` metadata element
ffprobe hdz_009.mp4

My attempt to prove the functionality in the goggles was based on this stackoverflow
diff --git a/src/record/ffpack.c b/src/record/ffpack.c
index 757f7bc..2a04c00 100755
--- a/src/record/ffpack.c
+++ b/src/record/ffpack.c
@@ -323,6 +323,8 @@ int ffpack_start(FFPack_t* ff)
     int ret = 0;

     //ffpack_dumpTimebase(ff);
+    av_dict_set(&ff->ofmtContext->metadata, "creation_time", "2023-02-03 04:05:06", 0);
+    av_dict_set(&ff->ofmtContext->metadata, "title", "testtest", 0);

     ret = avformat_write_header(ff->ofmtContext, NULL);
     if (ret < 0)

and I also tried the solution of that stackoverflow:
diff --git a/src/record/ffpack.c b/src/record/ffpack.c
index 757f7bc..0e41876 100755
--- a/src/record/ffpack.c
+++ b/src/record/ffpack.c
@@ -324,6 +324,11 @@ int ffpack_start(FFPack_t* ff)

     //ffpack_dumpTimebase(ff);

+    AVDictionary *pMetaData = NULL;
+    av_dict_set(&pMetaData, "creation_time", "2023-02-03 04:05:06", 0);
+    av_dict_set(&pMetaData, "title", "testtest", 0);
+    ff->ofmtContext = pMetaData;
+
     ret = avformat_write_header(ff->ofmtContext, NULL);
     if (ret < 0)
     {

In both cases the files get written as expected, but no extra metadata is written (not even the title)
My theory for failing is one of

I made a mistake using C (I have last worked with pointers and memory allocation ~10 years ago…)
This comment applies. That documentation might be relevant in that case
I failed to make the change in the right location

I am happy to answer further questions I can help with :)
And I am glad if someone can point out the mistake(s) I made 🙊

Besides https://github.com/hd-zero/hdzero-goggle/issues/116 RTC functionality is available


Update package "geograph"

This PR restricts Python to versions that work with the dependencies.
Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender


Angular material compatibility

Current version is not compatible with angular material, because of it's overlay layout strategy for selects, autocomplete, datepicker, etc. When they want show overlay, they changed body style, making it's position fixed, and 'top' style to negative value of current scrollY position. On finish they recover all settings back.
So this behavior breaks stickyLine variable value and sticky element won't unstick.
Codepen example:
https://codepen.io/mesteno/pen/qZpNEe
Reproduce bug:

scroll to second list
open datepicker
close datepicker
scroll top

Possible dirty fix:
Fix scrollbarYPos to ignore body negative top:
function scrollbarYPos() {
      var position;
      if (typeof scrollbar.scrollTop !== 'undefined') {
          position = scrollbar.scrollTop;
      } else if (typeof scrollbar.pageYOffset !== 'undefined') {
          position = scrollbar.pageYOffset;
      } else {
          position = document.documentElement.scrollTop;
      }
      var bodyTop = $body.css('top').replace('px', '');
      if (bodyTop < 0) {
          position -= bodyTop;
      }

      return position;
}

@d-oliveros, any plan to include this? You can check on @dlorenzo-plx implementation and maybe get it through a pull request. We'd like to have support for material with the offiicial bower package you created.
Cheers

+1

+1

With this new release, sticky elements disappear while material popup is open and reapper after it has been closed (tested with a material select control).
The issue with elements not ever unsticking seems to be fixed though. All in all, incompatibility between Angular Material and ngSticky still persists.

I would +1 this again. This would be rad to use with my angular-material project. Wouldn't want to have to roll it myself since your solution seems really well thought-out.


Update things to work with newer versions of Django

Greetings! I'm trying to work on a few election related things up in my new home county just north of Milwaukee. I was hoping to incorporate some of the opencivicdata models into the project and hit a bit of a roadblock since it seems that it's been a while since anyone has updated the Django version here. I'm not entirely sure where this project left off in terms of Django versions but it would seem that anything newer than perhaps Django 3.0 won't work because of some of the ways in which they've modified how models and index creation work. Specifically:

JSONField is now supported for all DB backends This means that the imports are different there, basically
index_together is deprecated in favor of indexes Rather than passing a list of fields you'd like to create and index for, you pass a list of models.Index objects that include the fields you'd like to index.

I updated the models and migrations to reflect these changes and tested it out against the latest Django release. I'm not sure how necessary it might be to support versions between 3.0 and 4.2. If that's something that we'll want to do here, I can break these changes out into two different PRs so that it's simpler to cut two different versions.
Anyways, I really hope someone is out there listening still! I notice that @fgregg has been in here in the past year or so which gives me hope!

hi eric! the james mckinney just fixed the jsonfield thing (like this week). can you rebase off of that?

Ha! I just did.

Sorry about the noise here. This is one of those things I thought was gonna be simple.

no problem. looks like you have some linting issues left?


Clicking the "Close" button on the upper right corner of the Flutter Windows desktop cannot close or exit the app

I use the SDK in the latest master, and the desktop app generated by running Flutter in Windows version 11 cannot be closed and exited after clicking the" "x" "button. I need to go to Windows Process Management to exit it. Note that I did not customize the menu bar.

Hi @xiaorui-23
Do you also see the issue on the newly created project? After clicking the Close button, are you able to interact with app's components (clicking a button for eg)?
Is there any special log from the console log output?
Also, please provide the output of flutter doctor -v. Thanks!

I directly executed 'flutter create demo' to create a new project named 'demo', and then I ran it under Windows 11 ,  clicked the 'x' close 'button to not close it. I need to end the run or end the process from Windows Process Management to exit. SDK version: The latest version on the master channel. I have placed the output log of my execution of the 'flutter doctor -v' command in this' txt 'file.
The output log does not report any errors or other exceptions, and it is all OK.
log.txt

Is this only happening in a release build?
Maybe related to https://github.com/flutter/flutter/issues/119415

这只发生在发行版中吗？可能与#119415 有关

No, this problem exists in both debug and release

我直接执行了‘flutter create demo’新建了一个项目，名字叫‘demo’，然后我在Windows 11下运行，点了‘x’close’按钮不关闭。我需要结束运行或结束 Windows 进程管理中的进程才能退出。SDK版本：主频道最新版本。我已将执行“flutter doctor -v”命令的输出日志放在这个“txt”文件中。
输出日志没有报错，也没有其他异常，一切OK。 日志.txt

我直接执行了‘flutter create demo’新建了一个项目，名字叫‘demo’，然后我在Windows 11下运行，点了‘x’close’按钮不关闭。我需要结束运行或结束 Windows 进程管理中的进程才能退出。SDK版本：主频道最新版本。我已将执行“flutter doctor -v”命令的输出日志放在这个“txt”文件中。
输出日志没有报错，也没有其他异常，一切OK。
日志.txt

Hi @xiaorui-23 Do you also see the issue on the newly created project? After clicking the Close button, are you able to interact with app's components (clicking a button for eg)? Is there any special log from the console log output?
Also, please provide the output of flutter doctor -v. Thanks!

I executed 'flutter create demo' to create a new project named 'demo', Then I ran it under Windows 11 and couldn't close it by clicking the 'x' close button. I need to end running or end a process in Windows Process Management to exit. SDK Version: The latest version of the main channel. I have placed the output log of executing the "flutter doctor - v" command in this "txt" file.
The output log did not report any errors or other exceptions, and everything was OK.
log.txt

Is this only happening in a release build? Maybe related to #119415

No, this problem exists in both debug and release

@xiaorui-23  I can not reproduce this issue from my end. Can you please check if this issue happens on lower Flutter SDK versions? (stable or 3.3.10 if you have it in your environment).

@xiaorui-23 I can not reproduce this issue from my end. Can you please check if this issue happens on lower Flutter SDK versions? (stable or 3.3.10 if you have it in your environment).

In the latest stable version of Flutter, version 3.7.9 and below, it is normal, and there is no issue I raised. Currently, I only encounter this issue on the channel master.

@xiaorui-23 Thanks for checking that. Can you try to debug the app on Visual Studio to see if it has any error stack trace there? (Please check Compiling with Visual Studio for reference).

@xiaorui-23 Thanks for checking that. Can you try to debug the app on Visual Studio to see if it has any error stack trace there? (Please check Compiling with Visual Studio for reference).

Well, I guess I couldn't debug with Visual Studio because I couldn't find a way to fix it, so I did a downgrade and switched to the stable version. But it's worth noting that I used Visual Studio Code for debugging runs without any errors.

@xiaorui-23 The issue is a regression as I see and it may be caused by a commit between the current stable 3.7.9 and the latest master version.
Unfortunately, my colleague with his Windows 11 can not reproduce this issue. It would be helpful for us if you can help to debug it with Visual Studio and provide more information if any (stack trace/exception for eg), so that the issue is actionable for the next steps to handle this from our side. Thank you!
Also, I'm not sure if this is a case mentioned in https://github.com/flutter/flutter/issues/123654.
cc @gspencergoog to help to verify this if this issue is related to #123654 since I see is also a regression issue.

@xiaorui-23 The issue is a regression as I see and it may be caused by a commit between the current stable 3.7.9 and the latest master version.
Unfortunately, my colleague with his Windows 11 can not reproduce this issue. It would be helpful for us if you can help to debug it with Visual Studio and provide more information if any (stack trace/exception for eg), so that the issue is actionable for the next steps to handle this from our side. Thank you!
Also, I'm not sure if this is a case mentioned in #123654. cc @gspencergoog to help to verify this if this issue is related to #123654 since I see is also a regression issue.

Sorry, I am temporarily unable to use Visual Studio to run debugging and provide any error (stack trace/exception for eg). I have seen the issue https://github.com/flutter/flutter/issues/123654, and I think this issue may have caused this error for me. You can try running a newly created Flutter Windows Desktop project using the latest version of Windows 11, and then check on Windows desktop project to see if it can be closed.

I tested this on my Windows 11 machine and was not able to reproduce it.
It would be helpful to collect flutter doctor -v output from a machine that is exhibiting this problem so that we can narrow down what version or environment is causing it.

flutter doctor -v

on Microsoft Windows 11 Pro for Workstations
Flutter 3.9.0-20.0.pre.96 • channel master • git@github.com:gspencergoog/flutter.git
Framework • revision 0046a25e39 (6 minutes ago) • 2023-04-03 12:56:56 -0400
Engine • revision 01aa040721
Tools • Dart 3.0.0 (build 3.0.0-402.0.dev) • DevTools 2.22.2
(code/flutter:master) -- flutter > flutter doctor -v
[!] Flutter (Channel master, 3.9.0-20.0.pre.96, on Microsoft Windows [Version 10.0.22621.1413], locale en-US)
    • Flutter version 3.9.0-20.0.pre.96 on channel master at C:\Users\gspencer\code\flutter
    ! Upstream repository git@github.com:gspencergoog/flutter.git is not a standard remote.
      Set environment variable "FLUTTER_GIT_URL" to git@github.com:gspencergoog/flutter.git to dismiss this error.
    • Framework revision 0046a25e39 (6 minutes ago), 2023-04-03 12:56:56 -0400
    • Engine revision 01aa040721
    • Dart version 3.0.0 (build 3.0.0-402.0.dev)
    • DevTools version 2.22.2
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.
[√] Windows Version (Installed version of Windows is version 10 or higher)
[X] Android toolchain - develop for Android devices
X ANDROID_HOME = C:/Users/gspencer/Android/Sdk
but Android SDK not found at this location.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop for Windows (Visual Studio Professional 2022 17.1.4)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Professional
• Visual Studio Professional 2022 version 17.1.32407.343
• Windows 10 SDK version 10.0.22000.0
[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
[√] IntelliJ IDEA Community Edition (version 2022.2)
• IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2022.1
• Flutter plugin version 70.2.5
• Dart plugin version 222.4345.14
[√] VS Code, 64-bit edition (version 1.74.3)
• VS Code at C:\Program Files\Microsoft VS Code
• Flutter extension version 3.56.0
[√] Connected device (3 available)
• Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22621.1413]
• Chrome (web)      • chrome  • web-javascript • Google Chrome 111.0.5563.147
• Edge (web)        • edge    • web-javascript • Microsoft Edge 100.0.1185.39
[√] Network resources
• All expected network resources are available.
! Doctor found issues in 3 categories.

Also, I tried switching to the Flutter version reported in https://github.com/flutter/flutter/issues/123761#issuecomment-1491282420 (91311156e12232466f78ce6ba96a0404c4f9790e), and also was unable to reproduce the problem.

Since I can not reproduce this issue from my end (and my colleague with his Windows 11 too), I label this issue for further insights.
Preliminary summary

Windows app can not be closed by clicking on Close (X) button
The issue occurs event with the newly created Flutter project
There is no stack trace from the output log
Maybe a case mentioned in https://github.com/flutter/flutter/issues/123654

For anyone also facing this issue, please help to debug this with Visual Studio (Compiling with Visual Studio) and provide more information if any (stack trace/exception for eg). Thanks!

Hello, we recently landed https://github.com/flutter/engine/pull/40802. Could you try updating to the latest master channel and check if this issue is now fixed?
flutter channel master
flutter upgrade

Please let us know if you can still reproduce this issue while on the master channel! /cc @yaakovschectman

I am also unable to repro on Windows 11 on both master and stable, which leads me to wonder if this issue is still up to date. Please check as Loic suggested. Thanks.


circleci-cli: Rename executable to circleci

Description of changes
This is the official name of the tool, so it matches the docs and usages out there.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 191178 run on x86_64-linux 1

1 package built:

circleci-cli

@ofborg eval

Result of nixpkgs-review pr 191178 run on x86_64-linux 1

1 package built:

circleci-cli

shell completion is missing. Can you add it?

shell completion is missing. Can you add it?

Good idea! Done.

Result of nixpkgs-review pr 191178 run on x86_64-linux 1

1 package built:

circleci-cli


[AutoRelease] t2-resource-2023-02-22-36666(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3833
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1025
BuildTargetingString
azure-mgmt-resource
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/3833


Add time import option

The import option "time" is missing from the available options.

Looks good. Thanks for the contribution!


Check if jQuery is available.

This enable us to use vanilla js.

And I think it's better to use 4 (or 2) spaces indent and lf newline?


Fixed uninstall command to take the version in kubernetes mode to uninstall

(cherry picked from commit da5e4ce7b814a93f2e7b4b3e5dfc7e94cfbcd7fa)
Description

Fixed uninstall command to take version in kubernetes mode. It is needed when installed through specific version, and it helps testing release candidate builds.
Added --runtime-version flag to dapr uninstall
Added -k shorthand to dapr uninstall

Issue reference
closes #395
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[x] Extended the documentation

@shalabhms I merged all new changes from master to release so we do not need this.

@shalabhms I merged all new changes from master to release so we do not need this.

sounds good.


Support declare(strict_types=1)

When I start a file with:
<?php

declare(strict_types=1);

<<<CONFIG
packages:
    - .....
CONFIG;

Then I get the following error in console (after melody run)
In ResourceParser.php line 23:

  Impossible to parse the content of the document.

  The php script should look like:

  <?php
  <<<CONFIG
  packages:
      - "symfony/finder: ~2.5"
  CONFIG;

  $finder = Symfony\Component\Finder\Finder::create();
run [--no-cache] [--prefer-source] [-t|--trust] [--] <script> [<arguments>]...

https://gist.github.com/pimolo/d06bb4d2e803369a66758095c007f2d1 can be used in order to reproduce.

Melody's parse does not allow extra string before the <<<CONFIG header (see https://github.com/sensiolabs/melody/blob/ccf10f654ccfa4c90cbac96320aaff1f927594ce/src/SensioLabs/Melody/Resource/ResourceParser.php#L16)
PR welcome!

@jderusse I'm actually already working on it 😄 🔥

I'm going to archive this project now to make it clear that the project is not maintained anymore. Anyone can fork it.


Set locale to en_US.utf-8

Changes the locale from POSIX to en_US.utf-8 which is expected by OMERO.web, by setting the LANG  environment variable. See this thread on the image.sc forum for reference
See https://github.com/ome/omero-server-docker/pull/76#issue-1562133645 for the corresponding pull request for the OMERO.server docker.

Thanks, @MicheleBortol!


Documentation on compiling a static library

It seems like add_library only allows shared library version: https://github.com/westerndigitalcorporation/sweet-b/blob/master/CMakeLists.txt#L126
I haven't found anywhere in documentation on how to build a static version. Is it supported? If not, is there any documentation on why not?
Thanks a lot, great library!!

Thanks for asking for this! PR #6 makes it easier to build a static version of Sweet B by using standard CMake features. To compile a static version of the library, add -DBUILD_SHARED_LIBS=0 when configuring the project with CMake. As an example, to build a static version:
mkdir cmake-build-static
cd cmake-build-static
cmake .. -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=0
make

This will produce a static library named libsweet_b.a.
You can also append -DSB_TEST=0 to skip building the test harness as well.


Test coverage

We have 100% test coverage (except for two lines which can intentionally never be reached).
What are your thoughts on adding Codecov, so we can put a "100% tested" badge on the readme?

Codecov fails all the time and is just annoying. Maybe we just add c8 locally and then add a static badge:
https://img.shields.io/badge/coverage-100%25-green

This works. :+1:


[Bug]: Ensure user is not created as admin first before the update

Plugin Version
4.2.1
PHP Version
8.1
Shopware Version
<IP_ADDRESS>
Installation method
Composer
Identity provider
Google Cloud
What happened?
The shopware user provisioner creates a user as admin by default. Therefore, the user is first created as admin - as nothing else is specified.
It would probably be better to initially create the user as non admin there and only use updateUser to set admin to true, if specified accordingly.
Relevant log output
No response

I like the approach following least privilege first :) we check whether we will release it as an update or take care of it in our already started 6.5 refactoring process.

I have not checked the code but is this still an issue with 6.0 or has this actually been fixed?

In the current setup shown in the screenshot where I used plugin version 7, the Shopware user provisioner creates a user without any roles by default. This means that every new user is initially assigned no roles because no other specification is provided.

As a result, users encounter missing permissions errors, such as “locale” and “language”, which indicate that they do not have the necessary permissions for certain actions.

To prevent these errors and ensure that users have the required permissions to perform their tasks, it is advisable to assign a default role that does not include admin rights. This default role should grant the necessary permissions, such as “locale” and “language”, so users can operate without encountering these errors.

@JoshuaBehrens Any news when this might be fixed? As currently one could use password forget if the update fails and the user has full admin permissions.
it should also be a simple fix, just add 'admin' => false to https://github.com/HEPTACOM/HeptacomShopwarePlatformAdminOpenAuth/blob/main/src/Service/UserResolver.php#L66

@AndreasA as @wimwenigerkind already mentioned: in the version 7 of the plugin this is fixed by having a different approach on managing roles at all. So this could be the way for you to work with it.

Hi @JoshuaBehrens the issue still exists in theory:

If you don't find a user, one is created here https://github.com/HEPTACOM/HeptacomShopwarePlatformAdminOpenAuth/blob/7.0.0/src/Service/UserResolver.php#L66

This adds a user to the database with admin = true.

In postUpdates the permissions are adjusted accordingly (rules etc.) https://github.com/HEPTACOM/HeptacomShopwarePlatformAdminOpenAuth/blob/7.0.0/src/Service/UserResolver.php#L71
However, if for whatever reason the process fails before the update call, e.g. memory limit or whatever, the following would be possible:

User can request a password forgotten mail.
User logs in with the new password.
User has full admin permissions.

As mentioned it is a bit of an edge case but the fix is quite simple, just call
 $this->userProvisioner->provision($user->primaryEmail, $password, ['admin' => false, 'email' => $user->primaryEmail]);

instead of
 $this->userProvisioner->provision($user->primaryEmail, $password, ['email' => $user->primaryEmail]);

I can create a PR for this. Would also like to see that change in 6.x for extended support Shopware versions.


Update package.json

Update jsdom library to fix vulnerabilities of Regular Expression Denial of Service warning

Closed by https://github.com/domchristie/turndown/commit/bc23db7129bac65bebdedc5bfb152b5823111882

Great 🎉

Closed by bc23db7

Great 🎉


Investigate SET AXIS/NAME=  for renaming coordinate axes

Reported by ansley b manke on 22 Apr 2009 16:37 UTC
Needed by Roland for F-TDS in the THREDDS server - handling multiple datasets having duplicate names for coordinate variables.
We have SET VAR/NAME=  for variables coming in from NetCDF datasets. Should be able to do a parallel thing for coordinate axes. The new setting LIST/KEEP_AXISNAMES  (see ticket #1634) could be used to restore the original axis name when writing data.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1656

Comment by ansley.b.manke on 22 Apr 2009 21:08 UTC
Lowering the priority and severity of this ticket.  It'd still be a useful capability to have, but not needed for F-TDS to work correctly.
Roland reports that in F-TDS the automatic renaming of coordinate axes works fine to prevent conflicts. The coordinate variable names don't have to be given anywhere specifically, so the existing behavior is exactly what's needed: Rename an axis if its name is the same but its definition is different from an existing axis.

Comment by ansley.b.manke on 5 Jan 2011 21:45 UTC
Closing this. The change is not needed.


Settings JSON - Add sort by keys button/link

I realize the focus has shifted to settings GUI, yet the json file is still just as important as it was before and sorting it with keys would be a quality improvement.

It's not likely that we'll invest a lot more in the settings JSON editor but an extension could do this.


Avoid that checking-statement produces an ansible change state

The ansible task "Check if bootstrap is needed" is a pure read-only
task and therefor should not cause a pseudo "change" in the summary.

This CI error seems not to be something introduced by this commits:
/bin/sh scripts/premoderator.sh
User does not have permissions to start CI run

ci check this

@guenhter I will approve CI for your jobs manually.

Strange. Gitlab is green, but here it states "failed"


Publish assets for 2.4.x on release page

Before you start, we need a little bit more information from you:
Use Case: Looking to prototype portainer for work.
Have you reviewed our technical documentation and knowledge base? Yes/No
Yes

Question:
Portainer missing new release assets on release page. When it will be published?

Hi
is it about CE or BE release. if its CE  which hasn't been released yet. and it  will be CE 2.5

thanks for assets


Example: Window closes when ESC is pressed

Description
The example window can now be closed when pressing ESC. Makes it a bit more pleasant to go though all the examples.

Should be platform independent now

Thanks!


Same testing framework

[x] Change the unittests to pytests so we have the same thing across the board (or the other way round if we need anything specific from unittest).
[x] Make sure they are running as part of the rest of the tests in the CI here (no change required if we are just changing to pytest).

This is a legacy from a separate profiler repository.


Is there a way to detect a connection reset by the pool?

Hi,
I've got a question:
Is there a reliable way of detecting  a connection is reset and return to the pool?
I would use this trigger for my jdbc wrapper to reset some statistics as soon as the connection is logically closed and returned to the pool.

Sorry, I was confused. I realized that I'm wrapping HikariCP proxy objects and not vice versa. I can hook on the Connection.close() method.


try to remove deep add save/restore on top of main

copying over a comment by @yoavkatz :
yoavkatz 17 hours ago
@dafnapension , @elronbandel - I don't think it's right to expect the user to save and restore values in metrics. Metrics pre-process should only be involved with the logic of the metic.
It has to be done in the generic pipeline code.


Add infra availability policy to hypershift deployment

Signed-off-by: Sai Sindhur Malleni<EMAIL_ADDRESS>Description
Fixes

It needs a rebase.


[Question] Firebase is not detected when building the app in Android

Unity editor version: 2019.4.21f1
Firebase Unity SDK version: 26.1.1
Source you installed the SDK: Unity Hub
Problematic Firebase Component: Auth
Other Firebase Components in use: RealTime Database
Platform you are using the Unity editor on: Windows
Platform you are targeting: Android (iOS, Android, and/or desktop)
Scripting Runtime: IL2CPP (Mono, and/or IL2CPP)

Hello there! First of all, excuse me if I get any mistakes in my question, but it is my first time.
I have developed a videogame in Unity which connects with Firebase in order to get authenticated or to get some data from the RealTime Database.
In editor mode, when I start the game and when I build the app for Windows as target, all works perfectly. I can register new users, login and save/retrieve data from RealTime Database.
The problem is that when I build the videogame for Android, my variable auth is null.
This is how I initialize Firebase and get my auth variable:
public Firebase.DependencyStatus dependencyStatus;
public Firebase.Auth.FirebaseAuth auth;
 public Firebase.Database.DatabaseReference DBReference;

void Awake()
{
  Firebase.FirebaseApp.CheckAndFixDependenciesAsync().ContinueWith(task =>
          {
              dependencyStatus = task.Result;
              if (dependencyStatus == Firebase.DependencyStatus.Available)
              {
                  // If available, initialize Firebase
                  InitializeFirebase();
              }
              else
              {
                  Debug.LogError("Could not resolve all Firebase dependencies: " + dependencyStatus);
              }
          });
}
private void InitializeFirebase()
    {
        Debug.Log("Setting up Firebase Auth");
        // Set the authentication instance object
        auth = Firebase.Auth.FirebaseAuth.DefaultInstance;
        DBReference = Firebase.Database.FirebaseDatabase.DefaultInstance.RootReference;
    }

I read that you have to write the complete dependency, because Android has problem resolving them.
The Awake function works fine, and it gets to InitializeFirebase function, but is then when the auth variable is null.
These are my Build Settings:

These are my Player Settings -> Other Settings:

These are the Publishing Settings, where I put my keystore:

I created the keystore using the Keystore Manager, and then I used the command keytool -list -v -keystore "C:\Users\happy\Documents\PFG Data\user.keystore"
Then I copied the SHA-1 code in Firebase, as you can see here:

And when I install the app via APK it's not working. I debugged the APK with the tablet connected to my PC, and I realized that the auth variable was null.

Hi @dizcampeador,
In order to identify the cause of your issue, could you provide a minimal, reproducible example of your project along with the debug logs when your app is running?

Hi @dizcampeador,
In order to identify the cause of your issue, could you provide a minimal, reproducible example of your project along with the debug logs when your app is running?

Hello! I don't know how to create a reproducible example, since the code works in Editor but not in my tablet. The only log I receive when I click the login button to do the Login function is that the auth variable is null... If I can do anything more for giving you more info I will do it, but I don't know how to provide you more info...

Hi @dizcampeador,
It's possible that your implementation may work on the editor but not on my device on my end. With that you may upload using a private repository and invite me as a collaborator. If you're unable to create a minimal, reproducible example, you may provide the debug logs along with the complete steps you've taken before facing the issue.


I cant Deserialize a length more than 32 array

for example,i have a strct field ” m_weather_forecast_samples: [WeatherForecastSample; 32] “,then when i build it,it will give me a error:
the trait bound `[WeatherForecastSample; 33]: Deserialize<'_>` is not satisfied
    --> src\eventModel.rs:226:33
     |
226  |     m_weather_forecast_samples: [WeatherForecastSample; 33],   // Array of weather forecast samples
     |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `Deserialize<'_>` is not implemented for `[WeatherForecastSample; 33]`
     |
     = help: the following other types implement trait `Deserialize<'de>`:
               [T; 0]
               [T; 1]
               [T; 2]
               [T; 3]
               [T; 4]
               [T; 5]
               [T; 6]
               [T; 7]
             and 26 others
note: required by a bound in `next_element`
    --> C:\Users\lc\.cargo\registry\src\index.crates.io-6f17d22bba15001f\serde-1.0.197\src\de\mod.rs:1726:12
     |
1724 |     fn next_element<T>(&mut self) -> Result<Option<T>, Self::Error>
     |        ------------ required by a bound in this associated function
1725 |     where
1726 |         T: Deserialize<'de>,
     |            ^^^^^^^^^^^^^^^^ required by this bound in `SeqAccess::next_element`

what happen?

We only implement the serialization traits for arrays up to length 32.
Supporting all array lengths is not easy, see https://github.com/serde-rs/serde/issues/1937 for details


Update gems

Ran bundle update to use latest versions of gems. Addresses security alert for Rack 2.0.5

@dana-stripe - would you be up for using Dependabot to automatically create dependency update PRs for this repo (maybe just for security issues)? I spoke to a couple of stripes at GitHub Universe and they seemed keen.


Nothing appears after loading a script on mac

I'm not sure if I'm doing this right, but I clicked load a script and was greeted with another blank screen. Tried with multiple scripts
https://streamable.com/ay3vi

You're doing everything right, it's just that this application is completely untested on Mac. I'm honestly surprised it runs at all.
If you've got a log file at ~/Library/Logs/Unity/Player.log, it would be greatly appreciated if you could paste its contents here.

https://pastebin.com/RJXT7TLJ

Honestly, I have no idea why this happens and how to fix it. This will take more investigation. Essentially, what's happening is that the method I use to allow a user to select a file yields a completely nonsensical path, leading to an error.


Implementation of the Library doesn't works

I just got amazed by this library but found that it is no longer getting installed in my android app from the   implementation 'com.github.p32929:MaterialOfficeAbout:1.0.5' line.
Could the developers please have a look at it!

What's the error?

When I am adding the implementation line the file is not getting downloaded properly I guess because the methods could not be implemented in the Java File

Thanks for the issue. Otherwise, I might have not known it.
Probably it was a problem with the hosting ( jitpack ). So, I updated the readme again. It will work now. Try with this:
implementation 'com.github.p32929:MaterialOfficeAbout:1.0.6'

Great Work Pal!


FIX #4731: update condition for selectedRowIndices when table widget drag and drop

Description

By dragging and dropping the Table widget 'The Value at the selected row is invalid' error occurs in the debugger

Fixes #4731
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Tested Locally.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
No changes to code coverage between the base branch and the head branch

re-running, tests are failing

Tested this issue in https://deploy-preview-4864--appsmith-editor-ce.netlify.app/applications and it is working as expected

Rerunning jobs as a flaky test failed

@vibhandikyash need to merge the latest release here.

/ok-to-test sha=2e0f307

@vibhandikyash latest merge required here for fix failing tests.


Add showcase demo games and demo data

Add some very basic demo games and demo data to radix as well as a target to build them.

Are demo games supposed to show what the engine is capable of (as in, its best features), or is it supposed to show just the basic things of the engine? Guess what I'm asking is how complex these games will, and are supposed to be. Because if they are more complex it might be a good idea to split them up in their own projects..

@Geertiebear I have some other issues for demo games that could be their own project for example #68. The ones I am referring to here should showcase engine features. I am changing the title of this issue accordingly. The main point is that if a radix user wants to build an FPS there is already a demo that shows how he can do it. We probably should start with an FPS demo and we will probably add new demos as we develop new features. So #67 is probably a sub-task to this.


Add oc alias fn to append compliance reason

Adds an ergonomic shell expansion that allows this:
oc get po -A --reason 'compliance'
to be expanded to this:
`ocm backplane elevate 'compliance' -- get po -A
Video:
Screencast from 2023-04-21 17-43-20.webm
To test:
Run make build-ocm-container in https://github.com/openshift/backplane-cli to use the latest version. Make sure that .config/backplane/config.json is set correctly (reach out to the backplane team for how to do this).
Run the above command. The Impersonate-Extra-Reason header should get added to your oc request.
Current drawbacks:
Only works if --reason is appended to the end of a request.

[ ] Requires https://github.com/openshift/ocm-container/pull/174 to be merged.

Thanks this is really great to that splunk can handle the reason!
imho it would be best to update the oc_elevate flow (currently in gitlab) that every time you run oc-elevate get po -A inside the container, you get a prompt to enter a reason before command is executed
/hold

Given this needs the new backplane it may be best to merge this after #174.
/hold

I'm personally a -1 on this, because it's not portable and for the users who don't know better they won't understand why oc --reason won't work outside of OCM Container.
I'm not sure what specific oc_elevate flow is in gitlab, but I could get behind some kind of plugin for oc that accepts --reason as a parameter.  In fact I think this would be pretty easy to do with the same code we have here - I think we'd just need to rename the file something like oc_elevate and put it in the PATH and it would automatically be found.
If an individual would still want to accomplish something like this then adding this to their personalizations file would be appropriate, I just dislike the idea of bulk-changing upstream functionality without being explicit as to who's changing that functionality.

Totally fine with me @iamkirkbater - could we explore something like an oc-reason script that gets put into ocm-container, so it doesn't conflict with anything that's already there? The main thing is just that it's awkward to use ocm backplane elevate --reason "" to attach the reason to the elevation. There's definitely space for something more ergonomic, I think.

100% - I'm totally on board with that.


Review usage of unicode_literals

See https://github.com/geier/khal/pull/187

I'm with hobarrera on this, I don't have a problem with unicode_literals and haven't seen them brake something yet.

From PythonCharmers/python-future#22:

Only a very low number of people are using Python 3.2 or older

I don't agree with this. Python 2.7 is still quite popular, probably the default for > 50% of the users.

While I don't really like how the code looks after these changes, I guess functionality is more important than aesthetics, so I won't argue on this further. :)

This is only talking about Python 3 versions.

I'm not a fan of the look either :( I'd love to drop python 2 support all along.

I'm not a fan of the look either :( I'd love to drop python 2 support all along.

+1 on this. Also getting rid of compat+six. However, we really need to know if it's possible for downstream packages to update to py3 (maybe the entire dependency tree is missing from many distributions?).
We could open a separate issue and open this for discussion with downstream packagers and users.

It's unlikely that Debian or some other LTS distros ship with 3.3+ as their
Python 3 package. However, khal depends on vdirsyncer, which requires 3.3+ or
2.7+.
Options:

Remove vdirsyncer as dependency (good idea for other reasons too)
Vdirsyncer drops Py2 support simultaneously.

On Thu, Nov 19, 2015 at 12:10:08PM -0800, Hugo Osvaldo Barrera wrote:

I'm not a fan of the look either :( I'd love to drop python 2 support all along.

+1 on this. Also getting rid of compat+six. However, we really need to know if it's possible for downstream packages to update to py3 (maybe the entire dependency tree is missing from many distributions?).
We could open a separate issue and open this for discussion with downstream packagers and users.

Reply to this email directly or view it on GitHub:
https://github.com/geier/khal/issues/189#issuecomment-158183413

Debian seems to include python3.3 on stable+ (source):

python3.4: stable, testing
python3.5: testing, unstable

Only oldstable seems to include python3 < 3.3 (it has 3.1, according to the above link). I really don't think we should limit ourselves because of Debian-oldstable.
However, the issue is if all other dependencies are available (otherwise the burden of packaging+maintaining would fall on the current khal maintainer, most likely).
Also: Does khal currently work with python3<3.3?

Also: Does khal currently work with python3<3.3?

Definetly doesn't, because of u"".

Also bug-wise it's not ideal to use 3.3, a lot of stuff got ironed out in later versions.

https://github.com/geier/khal/issues/302


Upgrade Subtitle Master to v2.01

The homepage redirects to their Github project, and the old download link was broken.

Merged as 2683aabce11f0608160eca22ac6d1a04e370b2d8.


MB-11671 Change 'Shipment Weight' wording for NTSR

Jira ticket for this change
Summary
This PR changes Shipment Weight to Previous Recorded Weight on the edit shipment form and shipment display cards for NTSR shipments .
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps

Access the office app
Login as a TOO
Look at NTSSUB
Verify that the NTSR shipment card shows Previous Recorded Weight
Edit the NTSR shipment and verify that the edit form shows Previous Recorded Weight

Verification Steps for Author
These are to be checked by the author.

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output

Screenshots

Messages

:book:
:link: MB-11671

  Generated by :no_entry_sign: dangerJS against 13ba372f6694da558e85b5116771214ceeac588e


[bash] kill completion not working

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux

Shell

[x] bash

Problem / Steps to reproduce
This problem has been persistent for quite a few months now. kill used to work in following manner:
a) in shell, type kill    (ie "kill" followed by a space)
b) press TAB
c) processes were listed and selectable via fzf
Now the c) step produces following error:
 kill sed: couldn't write 246 items to stdout: Broken pipe
sed: couldn't write 284 items to stdout: Broken pipe

Display all 461 possibilities? (y or n)

It still works as expected if input in a) step is replaced by kill **

Is this expected?
Config/additional info
My ~/.bashrc contains line [ -f ~/.fzf.bash ] && source ~/.fzf.bash
~/.fzf.bash contents:
if [[ ! "$PATH" == */home/laur/.fzf/bin* ]]; then
  PATH="${PATH:+${PATH}:}/home/laur/.fzf/bin"
fi

# Auto-completion
# ---------------
[[ $- == *i* ]] && source "/home/laur/.fzf/shell/completion.bash" 2> /dev/null

# Key bindings
# ------------
source "/home/laur/.fzf/shell/key-bindings.bash"

$ fzf --version
0.35.1 (b55f555)

$ bash --version
GNU bash, version 5.2.2(1)-release (x86_64-pc-linux-gnu)

Possibly related:

#1941
#1866
https://github.com/junegunn/fzf/issues/2716#issuecomment-1238301776

hi laur89.
Did you tweak you pipe maximum buffer size on your system recently?
Here's the value I have on my Linux station for example:
cat /proc/sys/fs/pipe-max-size
1048576

Have the exact same value.

It still works as expected if input in a) step is replaced by kill **
Is this expected?

Yes. See https://github.com/junegunn/fzf/commit/52594355bfa63ce7d579c7961f4f2fb30b486101. kill completion without trigger sequence is no longer supported.

 kill sed: couldn't write 246 items to stdout: Broken pipe
sed: couldn't write 284 items to stdout: Broken pipe

However, this is not expected, and I can't reproduce the problem. You might want to examine what's going on by doing it again after set +x.

Will do later when I'm back at the computer. In the mean time, would you be kind and reply in the #2716 thread whether there's a sane rollback for kill cmd

However, this is not expected, and I can't reproduce the problem. You might want to examine what's going on by doing it again after set +x.

Where should the debug be enabled? Doing it in the completion.bash, key-bindings.bash and complete shell itself didn't provide any extra logging.
Also something I just noticed - this sed error is only returned if I'm in a non-tmux shell. In tmux the output is:
kill <TAB>
Display all 502 possibilities? (y or n)

Where should the debug be enabled?

In your interactive shell.

Where should the debug be enabled?

In your interactive shell. Type in set +x and the subsequent actions will print the commands that are run.


Private endpoint supportability in Zone to Zone DR using Azure Site recovery

[Enter feedback here]
Hi Team,
Could you please confirm if we can use the Zone to Zone Azure site recovery using Private endpoint. i am not able to find any article for that. We want to configure RSV and Cache storage account with Private endpoint and Protect the VMs.

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

ID: e4bd4d9f-e7a7-782c-53fb-e5023ef39640
Version Independent ID: f6488b68-615b-ce4f-188e-392099e93272
Content: Enable zone-to-zone disaster recovery for Azure virtual machines - Azure Site Recovery
Content Source: articles/site-recovery/azure-to-azure-how-to-enable-zone-to-zone-disaster-recovery.md
Service: site-recovery
GitHub Login: @ankitaduttaMSFT
Microsoft Alias: ankitadutta

@Gouravkala
Thanks for your feedback! We will investigate and update as appropriate.

HI Team , any update on this please

hi @Gouravkala
The following articles could help:

https://learn.microsoft.com/en-us/azure/site-recovery/azure-to-azure-how-to-enable-replication-private-endpoints
https://learn.microsoft.com/en-us/azure/site-recovery/hybrid-how-to-enable-replication-private-endpoints

You can comment back on this thread if this doesn't solve your query.

HI could you please does this apply for zone to Zone DR as well ?


Return ConfigurationResponse in the ConfigurationHandler rather than in the WebhookHandler

@rutgervandeelen
Please return the ConfigurationResponse after the new webhoook is registered.  This allows the client to give user a feedback with the Webhook URL he/she can then use to registry it on a Webhook trigger (e.g. on GitHub).
You seem to return the ConfigurationResponse in the WebhookHandler, which I don't think is necessary as I don't think the Webhook trigger (such as GitHub) will be able to process this response.
https://github.com/Donders-Institute/hpc-qaas/blob/9bb5d69688f42c2e565113340027bb9cdacca184/internal/server/handlers.go#L113

Good catch. Now the ConfigurationResponse in is returned in the ConfigurationHandler instead of in the WebhookHandler. See commit eb40578123cc2d35d7a542b4121b93ae751a3371.


Support for HT-H6550WM

Hi,
how to find out if my Samsung HT-H6550WM Blu Ray player is supported with this plugin?

i googled your model number and found out:

It is an H-series (first character)
It is from around 2014
Both these facts suggest it would not work. But the best way to determine that is just to install the plugin and try it.

Closing as no feedback received


Dispatch CLI - formalize down sampling feature

Tool
Name: Dispatch
Version: tool version
OS: os version
Describe the bug

Expose this feature
Update documentation

To Reproduce
Steps to reproduce the behavior:

...
...
...
...

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

Added an auto active learning feature that can down sample a large intent/utterance set exceeding LUIS
's 15000 limit. Auto active learning will scan through a large training set and intelligently down sample it before creating a LUIS app.


Error in building from the source code

Description
ERROR in ./src/web_modules/metadata.js
Module build failed: Error: null
at ChildProcess.onExit (/home/ryan/Downloads/livestreamer-twitch-gui-0.15.2/src/web_loaders/metadata-loader.js:52:14)
at emitTwo (events.js:106:13)
at ChildProcess.emit (events.js:191:7)
at Process.ChildProcess._handle.onexit (internal/child_process.js:204:12)
@ ./src/app/controllers/AboutController.js 11:16-35
...
Expected / Actual behavior
Should build
...
Reproduction steps

Download the source and build
Follow the build steps in the readme

Environment details (operating system, etc.)
KUbuntu 16
...
Comments, logs, screenshots, etc.

...

Download the source

You need to clone the git repo and not just download the archive of the head of the master branch that github provides. The metadata webpack-loader builds an array of all git contributors. See here
https://github.com/bastimeyer/livestreamer-twitch-gui/blob/v0.15.2/src/web_loaders/metadata-loader.js#L42


Fix black spots appearing due to NANs when SSAO is enabled

Objective
Fixes https://github.com/bevyengine/bevy/issues/8925
Solution
Clamp the bad values.
More Info
The issue is that NdotV is sometimes very slightly greater than 1 (maybe FP rounding issues?), which caused F_Schlick() to return NANs in pow(1.0 - NdotV, 5.0) (call stack looked likepbr() -> directional_light() -> Fd_Burley() -> F_Schlick())

Another option that fixes it is to wrap prepass_normal() here https://github.com/bevyengine/bevy/blob/bb59509d44d2979e4b91bd0f998c047f16b89dd0/crates/bevy_pbr/src/render/pbr.wgsl#L121 with normalize()

I changed it to just normalize the prepass_normal() return value to avoid any potential loss of data from clamping.
We do this for apply_normal_mapping() when the prepass normals aren't used here https://github.com/bevyengine/bevy/blob/c6170d48f91357b5ed8ebb167b33d3cfa9688a6d/crates/bevy_pbr/src/render/pbr_functions.wgsl#L107

@superdump sorry, I forgot to update the PR description.
I removed all the clamps, and this just normalizes the prepass_normal() call.

We do this for apply_normal_mapping() when the prepass normals aren't used here

Unless something else changed, the prepass already calls apply_normal_mapping() so the results should already be normalized. If they aren't it might mean there's a missing shader def in the prepass or something.

Also, I think this shouldn't be done in prepass_normals() but should be done at the callsite. The normal texture can be generated by a custom material and normalizing there might not be what user expects. So I'd just do it directly in pbr.wgsl.

It was pointed out on discord that this happens because of precision issues even if the normals are normalized in the prepass. So it makes sense to do it like this. So you can ignore my comments.

@superdump are you okay if we merge this?

The saturate method is defined as:

Returns clamp(e, 0.0, 1.0). Component-wise when T is a vector.

So it doesn't seem to make much sense to do clamp(saturate(...), 0.0, 1.0)...


oh-my-zsh messes with zsh process substitution

#!/bin/bash
foo() { cat <(cat "$@"); }; foo <(echo bar);
bar
#!/bin/zsh
foo() { cat <(cat "$@"); }; foo <(echo bar);
cat: /proc/self/fd/11: No such file or directory
This is a bug from zsh ([BUG] process substitution breaks when nested or traverses a function) that has been fixed in version 5.5
HOWEVER, when running that in a zsh shell that uses oh-my-zsh, the bug is still there!
I tried disabling all plugins and the problem persists.
I don't expect you guys to be able to fix this, but please try to give me some insight on what is exactly is triggering this behaviour, because the zsh guys don't have enough info to work on this. So if we can narrow it down to something like, "autoloading a function that does x and y and uses this modules leaves zsh unable to do proper process substitution", will be a lot of help.
Thanks in advance

My bad, problem still happens with vanilla zsh


Demo gives error

Followed the instructions as per readme, this is the result. Saw that the settings.json points to a dead link for the image (container.imageLocation).

Thanks for trying it! SharedArrayBuffer on github.dev seems to return errors sometimes. Could you reload the editor page sometimes (possible with clearing cache)?
Please see also: https://github.com/ktock/vscode-container-wasm-gcc-example?tab=readme-ov-file#sharedarraybuffer-is-not-defined-error-occurs-when-launching-the-container

After reload it seems to work.


feat: Add notifications for issues.

UI for https://github.com/qdrant/qdrant/issues/3471
Layout section

[x] Notification for error(issues) with count.

Modal  to view issues.

[x]  Layout of modal
[x] Action button

@kartik-gupta-ij That looks fantastic!
I do think I prefer a overlaying sidebar, just like the green commands button on the bottom left. It has more room for issues, and keeps the UI consistent. Wdyt?
And we should probably render the request result a bit better, but I don't have a direct suggestion there.

Screen.Recording.2024-05-23.at.8.02.01.PM.mov
Feedbacks @timvisee , @coszio ?

Uploading Screen Recording 2024-05-23 at 8.02.01 PM.mov…
Minor updates!

The displaying of issues is looking better 👍
Another reason for having a dedicated page is that sometimes it is helpful to clear the issues with a button that calls DELETE /issues. Use-case: just playing around / developing with filters, so that using a wrong key name will trigger the issue for it, and it doesn't mean that an index should be created.


feat: handle window change event

[x] nvim_set_current_win
[x] output panel
[x] notebook cells
[x] tests

Resolves #1241
Should wait for #1414 and #1416 to be merged

Wow, you're on a role! I very much appreciate your contributions. I'll merge this once I have the chance to test.
I'm starting school this week so my ability to work on vscode-neovim is greatly reduced.


Initial review

This is great!
Some notes from looking at the action:

the branch is pushed to test-this-pr/$NUMBER. If a PR is force-updated and then the test comment is issued again, I think this push will fail unless it's done with --force
I believe there is already  a merge ref available so you can avoid fetching both full repos and doing the merge yourself: git fetch origin pull/$NUMBER/merge:test-this-pr/$NUMBER will fetch the existing merge ref and create the local branch test-this-pr/$NUMBER.
The comment at the end provides a link to the tree view of the new branch. Since this is for testing, it might make sense to link directly to the Actions page for that branch, e.g. https://github.com/binderhub-test-org/pr-test/actions?query=branch%3Amore-tests
Should the test-this-pr branch be deleted automatically on any particular event? E.g. merge/close of the parent PR?

I still don't know if it will work best to 'just' test the branch, or open local-branch PRs. WDYT so far after trying it out?
My initial thoughts:

pro PR: you get all the nice checks UI, easier view of success/failure
pro just branch: two PRs for the same change might get confusing. Which one do we merge? What happens when they update? If they are in-sync, merging either one should close both, but it's not easy to see if/when they are out of sync.
pro just branch: It seems like the checks/test conditions to avoid too many redundant tests might be easier for this case? I'm not sure.

Thank you for this! Great stuff to work on here.

Should the test-this-pr branch be deleted automatically on any particular event? E.g. merge/close of the parent PR?

This to me sounds like an optional feature so that the project maintainers can use their judgment. Although looking at the API docs, I can't easily find anything that'll close a PR, only merge one. So that might throw a spanner in the works this feature, and would be a point in favour for the "just branch" approach.

not sure I easily see an API endpoint to delete a branch

Weirdly it seems you have to use the git API to delete branches. So maybe if you have push access already, deleting via git push will be simplest: git push origin :test-this-pr/number

Should the test-this-pr branch be deleted automatically on any particular event?

On reflection I'm thinking if we do implement this, it wouldn't be in this repo built into this action. I think it's a whole other action that has probably already been implemented somewhere else.
For example in mybinder.org-deploy, we could have another job that looks something like the below.
jobs:
  ...
  delete-test-branch:
    # Only runs on pushes to test-this-pr/** branches
    if: (github.event_name == 'push') && (contains(github.ref, 'test-this-pr'))
    # Requires the deploy to staging job to succeed before this job runs
    needs: staging-deploy
    runs-on: ubuntu-latest
    steps:
      - run: |
          # Some code to delete the branch

Or there are some options that, e.g., sweep and delete merged and stale branches https://github.com/marketplace/actions/git-sweep-merged-and-stale


Switch url on fail

I am using ioredis against a Keydb master-master architecture. Since my specific scenario asks for one single connection a time, I would need to connect to let's say host1 and switch to host2 in case host1 is down/unresponsive. Is there any official way to do it? I cannot use sentinel or cluster.
Thank you for every hint!

Was looking for something else but saw you got no answer here. Use HaProxy. Doing the same here :)

@thenitai I ended using ha-Roxy as well :)


implement facet search

Power tool component of client requires more sophisticated search capabilities.
Current strategy for dealing these searches:
There are 3 facets to orient search around:

location
client isp
transit (server) isp

Each facet allows for searching of all 3 entities but with different constraints.

location facet allows for searching of client isp and transit isp limiting the results of these searches to only include client isps and transit isps found in this location

client isp likewise allows searching for location and transit isp with locations only listing locations the selected client isps are active in. And only transit isps with data coming from the selected client isp(s)

transit isp allows searching for location and client isp. This is the location from data from   which client isps come from that terminate with this  transit isp. So this is still client location. Also, client isps are filtered by showing only client isps that have data terminating with this transit isp.

Filters
With the facets Here are the different entity/filtering combinations we have:

location with no filters

location filter by client isp

location filter by transit isp

client isp with no filters

client isp filter by location

client isp filter by transit isp

transit isp with no filters

transit isp filter by location

transit isp filter by client isp

Possible Implementation
We have an existing solution to filtering by one entity in place:
client_location_client_asn is a table with a compound key of 2 values:

location_key
client_asn_number

This provides a solution for filtering a client isp by a location.
If we expand this solution to include the additional filtering we need, it would involve making these tables:

client_location_search ✔

keys: reverse_location_key

client_location_client_asn ✔

keys: location_key, client_asn_num

client_location_transit_asn

keys: location_key, transit_asn_num

client_asn_search ✔

keys: client_asn_name_lookup

client_asn_client_loctation

keys: client_asn_num, client_location

client_asn_transit_asn

transit_asn_search

transit_asn_client_location

transit_asn_client_asn

where ✔ indicates already existing tables.
Our current implementation of location_search only provides starts with name matching - due to bigtable query capabilities. This is a limitation of client_asn_search This would probably be a limitation for transit_asn_search as well.
For filtering. the keys are all built from asn keys and location keys. So the easiest thing to do would be to pass all results matching a filter (for example, all transit isps with client location of 'new york') to the api. If the resulting dataset was large, further filtering could occur at the api level (example: only pull out values with name containing 'lev').
The UI has a need for allowing multiple values in the initial filter. We will use these as union values for the filterable dependent entities.
For example. On facet by location, new york and london are selected. Now the user is selecting a client_isp. The API in this scenario must return client isps that appear in new york or london.
This could be implemented at the API level by acquiring both client isp lists from bigtable, and then performing the union there.

Thanks for writing this up Jim, looks good to me :+1:

done.


Allow the creation of a DataTarget just by a simple Stream, not by filename

Hey,
after looking into the code a bit, I've noticed that only the isTinyDump in Minidump with MemoryMappedFile.CreateFromFile requires a real FileStream, all other segments are using a normal Stream.
Would it be possible to remove the FileStream restriction and File.Exists(crashDump) from the code in favor of a new additional factory like DataTarget.LoadFromStream(string displayName, Stream stream, CacheOptions? cacheOptions = null)?

Ideally LoadDump could accept a Stream, an ImmutableArray<Byte> or even a Byte* just like what PEReader does.


Software rendering problems with multiple domains.

These issues were reported by Matthew Wheeler at AWE. email: ======Id like to report a couple of VisIt issues that one of our users came across recently. There appear to be two problems:Initially we were not sure if they were related but we now suspect them to be different problems. (?)(The user was seeing both problems at the same time) I have now managed to reproduce them in much simpler form with smaller and public domain datasets.(See notes further down). Hopefully these will help someone who understands that area better than me to pin it down. 1) Visible white lines/breaks, randomly between some domains when using Parallel Software-Rendering. User was running ClientServer, Software Rendering with a large 2D AMR dataset and trying to display a Pseudocolor Plot with a slightly transparent Mesh Plot superimposed. White lines appear between some domains as you zoom in and out but disappear if you switch to Hardware Rendering or force the problem to run on a single processor (np 1). The resultant images can look very bad with a large complicated dataset. I have shown that the dataset does not have to be AMR other 2D multi-domain datasets also seems to exhibit the same problem if you look carefully but they dont always look as bad. The problem seems similar to the reported VisIt Bug #1054 VisIt generates plots with gaps in between domainsAlso possibly similar to Bug #1350 although this seems to have a different cause (and has been fixed?). I think I have now managed to reproduce the problem with a much simpler dataset so hopefully this may help in debugging - see later. 2) AMR levels appearing coarser than the levels defined in the dataset, i.e. not all the finer levels being shown. Sometimes switching levels off and on again seems to help but not always. Again switching to Non-Scalable Rendering (=Hardware Rendering) mode produces the correct result.Plotting the Mesh isnt necessary to reproduce this problem.See test datasets and example pictures. We have tried experimenting with a whole bunch of factors .e.g.:Whether SR is on or off Before you start the Plot.Sometimes the first time you produce the Plot it is different to subsequent renderingsAdding extra Operators doesnt seem to affect the problem.ReOpening the dataset ? Not sure if this affects itOccurs in different versions of VisIt discovered in 2.6.1 but seems to be present in all the other versions that we still have installed ~ 2.4.* 2.7.0b- Difficult to see what the common factors are. User has a short term workaround to use Hardware Rendering for both problems but there must come a point when the datasets will become too large to display in this manner. Test DatasetsI have used two datasets to illustrate the problems which you should be able to get at: multi_curv2d.silo - illustrates problem 1, A 2D Curvilinear Mesh database part of VisIt test suite (?) Consists of 5 domains/blocks in an arc. amr2d_wmrgtree.silo - illustrates problems 1 and 2A 2D AMR dataset downloaded from https://wci.llnl.gov/codes/silo/examples.html(This is not actually as simple a dataset as it could be Id be interested if you can find a more straightforward AMR dataset that we could both gain access to) To Reproduce- Problem 1 Domain breaks: In Parallel Software Rendering mode, running over multiple processors (np > 1) Open multi_curv2d.silo (or amr2d_wmrgtree.silo) Do a Pseudocolor Plot of an appropriate variable (say u or Density) Add a Mesh PlotNow turn the Transparency of the Mesh lines down below 100% e.g. 16% Observe the flickering white lines between domains as you zoom in and out. Note that if you switch to Hardware Rendering (which would be the default for small datasets!) you would get quite a pleasing result the Mesh would appear as a subtle hint superimposed on top of the colored surface.This must say something about where the problem is occurring it would suggest that the data is being read-in and manipulated correctly the problem would appear to be further down the rendering pipeline. Problem 2 - Coarse AMR levels: Just try to plot a Pseudocolor Plot of the AMR dataset while in Multi-Processor, Software Rendering mode.Note the chunky representation which is resolved when you switch to Hardware Rendering. I am sending you some images illustrating the problems.They are in pairs with a good/OK (HW) and a BAD (SR) version but otherwise using the same setup Let me know if I can provide more information or need to clarify anything.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1487
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: Software rendering problems with multiple domains.
Assigned to: -
Category: -
Target version: -
Author: Brad Whitlock
Start: 06/05/2013
Due date:
% Done: 0%
Estimated time:
Created: 06/05/2013 06:02 pm
Updated: 06/26/2014 01:51 pm
Likelihood: 3 - Occasional
Severity: 3 - Major Irritation
Found in version: 2.6.2
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
These issues were reported by Matthew Wheeler at AWE. email: ======Id like to report a couple of VisIt issues that one of our users came across recently. There appear to be two problems:Initially we were not sure if they were related but we now suspect them to be different problems. (?)(The user was seeing both problems at the same time) I have now managed to reproduce them in much simpler form with smaller and public domain datasets.(See notes further down). Hopefully these will help someone who understands that area better than me to pin it down. 1) Visible white lines/breaks, randomly between some domains when using Parallel Software-Rendering. User was running ClientServer, Software Rendering with a large 2D AMR dataset and trying to display a Pseudocolor Plot with a slightly transparent Mesh Plot superimposed. White lines appear between some domains as you zoom in and out but disappear if you switch to Hardware Rendering or force the problem to run on a single processor (np 1). The resultant images can look very bad with a large complicated dataset. I have shown that the dataset does not have to be AMR other 2D multi-domain datasets also seems to exhibit the same problem if you look carefully but they dont always look as bad. The problem seems similar to the reported VisIt Bug #1054 VisIt generates plots with gaps in between domainsAlso possibly similar to Bug #1350 although this seems to have a different cause (and has been fixed?). I think I have now managed to reproduce the problem with a much simpler dataset so hopefully this may help in debugging - see later. 2) AMR levels appearing coarser than the levels defined in the dataset, i.e. not all the finer levels being shown. Sometimes switching levels off and on again seems to help but not always. Again switching to Non-Scalable Rendering (=Hardware Rendering) mode produces the correct result.Plotting the Mesh isnt necessary to reproduce this problem.See test datasets and example pictures. We have tried experimenting with a whole bunch of factors .e.g.:Whether SR is on or off Before you start the Plot.Sometimes the first time you produce the Plot it is different to subsequent renderingsAdding extra Operators doesnt seem to affect the problem.ReOpening the dataset ? Not sure if this affects itOccurs in different versions of VisIt discovered in 2.6.1 but seems to be present in all the other versions that we still have installed ~ 2.4.* 2.7.0b- Difficult to see what the common factors are. User has a short term workaround to use Hardware Rendering for both problems but there must come a point when the datasets will become too large to display in this manner. Test DatasetsI have used two datasets to illustrate the problems which you should be able to get at: multi_curv2d.silo - illustrates problem 1, A 2D Curvilinear Mesh database part of VisIt test suite (?) Consists of 5 domains/blocks in an arc. amr2d_wmrgtree.silo - illustrates problems 1 and 2A 2D AMR dataset downloaded from https://wci.llnl.gov/codes/silo/examples.html(This is not actually as simple a dataset as it could be Id be interested if you can find a more straightforward AMR dataset that we could both gain access to) To Reproduce- Problem 1 Domain breaks: In Parallel Software Rendering mode, running over multiple processors (np > 1) Open multi_curv2d.silo (or amr2d_wmrgtree.silo) Do a Pseudocolor Plot of an appropriate variable (say u or Density) Add a Mesh PlotNow turn the Transparency of the Mesh lines down below 100% e.g. 16% Observe the flickering white lines between domains as you zoom in and out. Note that if you switch to Hardware Rendering (which would be the default for small datasets!) you would get quite a pleasing result the Mesh would appear as a subtle hint superimposed on top of the colored surface.This must say something about where the problem is occurring it would suggest that the data is being read-in and manipulated correctly the problem would appear to be further down the rendering pipeline. Problem 2 - Coarse AMR levels: Just try to plot a Pseudocolor Plot of the AMR dataset while in Multi-Processor, Software Rendering mode.Note the chunky representation which is resolved when you switch to Hardware Rendering. I am sending you some images illustrating the problems.They are in pairs with a good/OK (HW) and a BAD (SR) version but otherwise using the same setup Let me know if I can provide more information or need to clarify anything.
Comments:
I think this is the 2D case of #1860.


Add support for tone-based surface colors

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
[Part of #91605]
Material 3 has introduced tone-based surface colors:
https://m3.material.io/styles/color/overview#fb46760e-a96a-4d05-9ca6-2b6754ddfb73
These changes appear to be part of a wider set of updates to the color system, see this.
Updates
Added
The following color roles have been added. See: https://m3.material.io/styles/color/the-color-system/color-roles:

Color Role
Tone: Light
Tone: Dark

Surface Dim
N-87
N-6

Surface Bright
N-98
N-24

Surface Container Lowest
N-100
N-4

Surface Container Low
N-96
N-10

Surface Container
N-94
N-12

Surface Container High
N-92
N-17

Surface Container Highest
N-90
N-22

New fixed accent colors have also been introduced (see #137683).
Changed

Tones for the "Surface" color role have been updated:
Old

Color Role
Tone: Light
Tone: Dark

Surface
N-99
N-10

New

Color Role
Tone: Light
Tone: Dark

Surface
N-98
N-6

The chroma for the neutral palette has been increased from 4 to 6.

Deprecated
The following color roles are marked as legacy/deprecated:

Background
On background
Surface variant

As stated here:

Note: Background is a legacy color role. It is recommended to use Surface instead of Background.

The "On background" and "Surface Variant" roles still exist in the list of tokens but are not shown as part of the color scheme and possibly will be deprecated soon.
Proposal
Update the ColorScheme implementation to add color roles for tone-based surface colors. Many components also use the new surface color roles as part of their specification. Update the components to use the new color roles.
Implementation
Currently, Flutter generates the Material 3 color schemes using ColorScheme.fromSeed which, in turn, uses the Scheme class of the Material Color Utilities package.
However, it appears that the Scheme class will be deprecated soon in favor of DynamicScheme. The make_schemes.md file in the Material Color Utilities GitHub repository has instructions for migrating from Scheme.
The ColorScheme class will also have to be updated to include the new color roles.

Thanks for filing a detailed issue. However, I see there's an open issue addressing the case you described https://github.com/flutter/flutter/issues/115912.
Closing this issue as a duplicate, so please leave your comment there that will help consolidate the issue. Thank you!

What is the specific API that I couldn't find in the theme

@peter100u Are you referring to the Scheme or DynamicScheme class?

@peter100u你指的是SchemeorDynamicScheme类吗？

I mean how to obtain different height APIs

@peter100u Flutter's ColorScheme class does not provide access to the tonal palette directly. It uses the Material Color Utilities package to generate the palettes. If you want access to the tonal palettes directly, you have to add this package and generate a CorePalette :
Color seed = yourColor;
CorePalette palette = CorePalette.of(seed);

In the above image "N" stands for the neutral palette. For example, "Surface Dim" in light theme is "N-87". This can be accessed as:
Color surfaceDim = Color(palette.neutral.get(87));

@peter100uFlutter 的ColorScheme类不提供直接访问色调调色板的功能。它使用Material Color Utilities 包来生成调色板。如果您想直接访问色调调色板，则必须添加此包并生成CorePalette：
Color seed = yourColor;
CorePalette palette = CorePalette.of(seed);

在上图中，“ N ”代表中性调色板。例如，浅色主题中的“Surface Dim”是“N-87”。可以通过以下方式访问：
Color surfaceDim = Color(palette.neutral.get(87));

Thank you for your reply. It would be great if different tones could be obtained from the theme or colorschme, as our project theme is dynamic and not fixed, making it difficult to obtain them from a single color. We will obtain them from the main color of the current theme, etc


Keystroke delay

There should be a option to set a custom keystroke delay when using secret agent. The default one is very slow
In puppeteer/playwright I can do this by adding {delay: delay} to the type command
ex:
~ await agent.type('user', {delay: 5});

The best way to do this right now is to modify the default-human-emulator. You can load your own default human emulator if you want to override the provided example. We're going to eventually add variables to tweak some of the settings in the human emulator to adjust the speed it types and moves. You can also just ignore any human-like behavior by having a HumanEmulator plugin that just clears out the human emulator behavior
// core side (needs to be in same process as core - see (interact.test.ts)[https://github.com/ulixee/secret-agent/blob/e8e400b2d23092ccbcb652349f5053dd8fd5074d/full-client/test/interact.test.ts] for examples)
Core.use(class BasicHumanEmulator extends HumanEmulator {
  static id = 'basic';
})

// client side needs to then create an agent with the emulator: new Agent({ humanEmulatorId: 'basic', connectionToCore: { host .. })


Allow errors on rmtree

Getting errors on uploads from GHA:
Error:  Trouble reading metadata from C:\Miniconda3\conda-bld\win-64\conda-23.5.0.post1-py39h178df7a_1.conda. Is this a valid conda package?
Error:
Traceback (most recent call last):
  File "C:\Miniconda3\Scripts\anaconda-script.py", line 9, in <module>
    sys.exit(main())
  File "C:\Miniconda3\lib\site-packages\binstar_client\scripts\cli.py", line 154, in main
    binstar_main(command_module, args, _exit,
  File "C:\Miniconda3\lib\site-packages\binstar_client\scripts\cli.py", line 136, in binstar_main
    return args.main(args)
  File "C:\Miniconda3\lib\site-packages\binstar_client\commands\upload.py", line 339, in main
    package_info = upload_package(
  File "C:\Miniconda3\lib\site-packages\binstar_client\commands\upload.py", line 206, in upload_package
    package_attrs, release_attrs, file_attrs = get_attrs(package_type, filename, parser_args=args)
  File "C:\Miniconda3\lib\site-packages\binstar_client\utils\detect.py", line 153, in get_attrs
    return inspector(filename, fileobj, *args, **kwargs)
  File "C:\Miniconda3\lib\site-packages\binstar_client\inspect_package\conda.py", line 159, in inspect_conda_package
    rmtree(tmpdir)
  File "C:\Miniconda3\lib\shutil.py", line 759, in rmtree
    return _rmtree_unsafe(path, onerror)
  File "C:\Miniconda3\lib\shutil.py", line 624, in _rmtree_unsafe
    _rmtree_unsafe(fullname, onerror)
  File "C:\Miniconda3\lib\shutil.py", line 629, in _rmtree_unsafe
    onerror(os.unlink, fullname, sys.exc_info())
  File "C:\Miniconda3\lib\shutil.py", line 627, in _rmtree_unsafe
    os.unlink(fullname)
PermissionError: [WinError 5] Access is denied: 'D:\\a\\_temp\\tmplxka0ri1\\info\\about.json'
Failed to upload due to Command '['anaconda', '--quiet', '--show-traceback', '-t', 'D:\\a\\_temp\\XXXX\\XXXXX.token', 'upload', 'C:\\Miniconda3\\conda-bld\\win-64\\conda-23.5.0.post1-py39h178df7a_1.conda', '--user=napari', '--channel=bundle_tools_3', '--force-metadata-update']' returned non-zero exit status 1.. Trying again in 28.722900390625 seconds

Some files have bad permissions in conda packages; yikes. However https://github.com/anaconda/anaconda-client/pull/724

@jaimergp do you have a reproducible command I could confirm with the change in #724? I'm hoping to have a RC build we could try to confirm the fix works for your issue.

I'm afraid I no longer have access to the repro conditions. It was a CI workflow building conda packages for a conda fork we used in the napari project for a bit, but we have since then upstreamed the fork changes so those CI workflows and repos are now archived.


Do we have consistency and canonical forms?

I can't find the theorems for them.

For pi type, https://github.com/Beluga-lang/McLTT/blob/main/theories/Core/Semantic/Consequences.v#L48
But for others, I don't think we have one.

I suppose we don't have consistency? Is there one?

Now we have.