GitHub Workflows security hardening

This PR adds explicit permissions section to workflows. This is a security best practice because by default workflows run with extended set of permissions (except from on: pull_request from external forks). By specifying any permission explicitly all others are set to none. By using the principle of least privilege the damage a compromised workflow can do (because of an injection or compromised third party tool or action) is restricted.
It is recommended to have most strict permissions on the top level and grant write permissions on job level case by case.

Codecov Report

Merging #10518 (7a310c4) into 3.1 (09582dd) will decrease coverage by 0.42%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                3.1   #10518      +/-   ##
============================================
- Coverage     65.65%   65.22%   -0.43%
+ Complexity      319      317       -2
============================================
  Files          1286     1286
  Lines         55222    55186      -36
  Branches       8269     8240      -29
============================================
- Hits          36255    35997     -258
- Misses        15039    15237     +198
- Partials       3928     3952      +24

Impacted Files
Coverage Δ

...mmon/serialize/fastjson2/FastJson2ObjectInput.java
0.00% <0.00%> (-62.50%)
:arrow_down:

...mon/serialize/fastjson2/FastJson2ObjectOutput.java
0.00% <0.00%> (-56.25%)
:arrow_down:

...dubbo/remoting/zookeeper/ZookeeperTransporter.java
50.00% <0.00%> (-33.34%)
:arrow_down:

...on/serialize/fastjson2/FastJson2Serialization.java
57.14% <0.00%> (-28.58%)
:arrow_down:

...ng/zookeeper/curator5/Curator5ZookeeperClient.java
43.86% <0.00%> (-26.42%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
41.66% <0.00%> (-25.00%)
:arrow_down:

...nt/metadata/ServiceInstanceHostPortCustomizer.java
65.78% <0.00%> (-21.06%)
:arrow_down:

...n/serialize/fastjson2/Fastjson2CreatorManager.java
46.66% <0.00%> (-20.00%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

...dubbo/remoting/exchange/support/DefaultFuture.java
80.35% <0.00%> (-11.89%)
:arrow_down:

... and 62 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[BUG] Old SimpleReport Page with PDF Appears in Top Search Result

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When I google ReportStream, the top result is the old page for SimpleReport with the ReportStream PDF vs. the link to the ReportStream site.
Here is the URL that appears in the top of the search result
https://simplereport.gov/resources/getting-started/public-health-departments/about-prime-reportstream/

Steps To Reproduce

Using Chrome to google "reportstream"
Screen shot of search result is below:

When click on the result, the URL is https://simplereport.gov/resources/getting-started/public-health-departments/about-prime-reportstream/

Expected Behavior
When I search for ReportStream, I expect to see the most current page which is:
https://simplereport.gov/getting-started/public-health-departments/about-prime-reportstream/
Additional context
No response
Does this issue have a priority label?

[X] I have added a priority label

Has issue been assigned to a team?

[X] I have added a team label

@nickclyde I know we are doing a full audit, but can we fix this one ASAP.


chore: Bump library dependency in app

This is needed following PR https://github.com/googlemaps/android-maps-compose/pull/109, otherwise users will get a crash when tapping the "Toggle Map" button (caused by https://github.com/googlemaps/android-maps-compose/issues/106, which is fixed in v2.1.1 release).

Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open a GitHub issue as a bug/feature request before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

@barbeau Please take a moment to fill out this short survey. Thank you!
This is an automated message, feel free to ignore.


chore: package name placeholder

I guess the problem is that [name] can be a valid name in package.json

I guess the problem is that [name] can't be a valid name in package.json

Does is matter? I suppose the first thing most people do after cloning this repo is to replace [name] globally.🤔

You see, eslint is not happy with it :/


RuntimeError: NetCDF: Invalid dimension ID or name

What happened?
I got a long error message and the RuntimeError: NetCDF: Invalid dimension ID or name error when trying to open a nc file with xarray. I did a bit of digging around, but could not find a workaround (the issues with similar problems I found were old and supposed to be solved).
What did you expect to happen?
No response
Minimal Complete Verifiable Example
See the jupyter notebook: https://github.com/jerabaul29/public_bug_reports/blob/main/xarray/2023_08_15/illustrate_issue_xr.ipynb .
Copying the commands (note that it should be run as a notebook):
import os
import xarray as xr
xr.show_versions()
# get the file
!wget https://arcticdata.io/metacat/d1/mn/v2/object/urn%3Auuid%3Ad5f179a3-76a8-4e4f-b45f-8e8d85960ba6
# rename the file
!mv urn\:uuid\:d5f179a3-76a8-4e4f-b45f-8e8d85960ba6 bar_BSO3_a_rcm7_2008_09.nc
# the file is not xarray-friendly, modify and rename
!ncrename -v six_con,six_con_dim bar_BSO3_a_rcm7_2008_09.nc bar_BSO3_a_rcm7_2008_09_xr.nc
xr.open_dataset("./bar_BSO3_a_rcm7_2008_09_xr.nc")

MVCE confirmation

[X] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[X] Complete example — the example is self-contained, including all data and the text of any traceback.
[X] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.

Relevant log output
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
File ~/miniconda3/envs/pytmd/lib/python3.11/site-packages/xarray/backends/file_manager.py:211, in CachingFileManager._acquire_with_cache_info(self, needs_lock)
    210 try:
--> 211     file = self._cache[self._key]
    212 except KeyError:

File ~/miniconda3/envs/pytmd/lib/python3.11/site-packages/xarray/backends/lru_cache.py:56, in LRUCache.__getitem__(self, key)
     55 with self._lock:
---> 56     value = self._cache[key]
     57     self._cache.move_to_end(key)

KeyError: [<class 'netCDF4._netCDF4.Dataset'>, ('/home/jrmet/Desktop/Git/public_bug_reports/xarray/2023_08_15/bar_BSO3_a_rcm7_2008_09_xr.nc',), 'r', (('clobber', True), ('diskless', False), ('format', 'NETCDF4'), ('persist', False)), 'b771184f-a474-4093-b5f1-784542ef9ce6']

During handling of the above exception, another exception occurred:

RuntimeError                              Traceback (most recent call last)
Cell In[6], line 1
----> 1 xr.open_dataset("./bar_BSO3_a_rcm7_2008_09_xr.nc")

File ~/miniconda3/envs/pytmd/lib/python3.11/site-packages/xarray/backends/api.py:570, in open_dataset(filename_or_obj, engine, chunks, cache, decode_cf, mask_and_scale, decode_times, decode_timedelta, use_cftime, concat_characters, decode_coords, drop_variables, inline_array, chunked_array_type, from_array_kwargs, backend_kwargs, **kwargs)
    558 decoders = _resolve_decoders_kwargs(
    559     decode_cf,
    560     open_backend_dataset_parameters=backend.open_dataset_parameters,
   (...)
    566     decode_coords=decode_coords,
    567 )
    569 overwrite_encoded_chunks = kwargs.pop("overwrite_encoded_chunks", None)
--> 570 backend_ds = backend.open_dataset(
    571     filename_or_obj,
    572     drop_variables=drop_variables,
    573     **decoders,
    574     **kwargs,
    575 )
    576 ds = _dataset_from_backend_dataset(
    577     backend_ds,
    578     filename_or_obj,
   (...)
    588     **kwargs,
    589 )
    590 return ds

File ~/miniconda3/envs/pytmd/lib/python3.11/site-packages/xarray/backends/netCDF4_.py:602, in NetCDF4BackendEntrypoint.open_dataset(self, filename_or_obj, mask_and_scale, decode_times, concat_characters, decode_coords, drop_variables, use_cftime, decode_timedelta, group, mode, format, clobber, diskless, persist, lock, autoclose)
    581 def open_dataset(  # type: ignore[override]  # allow LSP violation, not supporting **kwargs
    582     self,
    583     filename_or_obj: str | os.PathLike[Any] | BufferedIOBase | AbstractDataStore,
   (...)
    599     autoclose=False,
    600 ) -> Dataset:
    601     filename_or_obj = _normalize_path(filename_or_obj)
--> 602     store = NetCDF4DataStore.open(
    603         filename_or_obj,
    604         mode=mode,
    605         format=format,
    606         group=group,
    607         clobber=clobber,
    608         diskless=diskless,
    609         persist=persist,
    610         lock=lock,
    611         autoclose=autoclose,
    612     )
    614     store_entrypoint = StoreBackendEntrypoint()
    615     with close_on_error(store):

File ~/miniconda3/envs/pytmd/lib/python3.11/site-packages/xarray/backends/netCDF4_.py:400, in NetCDF4DataStore.open(cls, filename, mode, format, group, clobber, diskless, persist, lock, lock_maker, autoclose)
    394 kwargs = dict(
    395     clobber=clobber, diskless=diskless, persist=persist, format=format
    396 )
    397 manager = CachingFileManager(
    398     netCDF4.Dataset, filename, mode=mode, kwargs=kwargs
    399 )
--> 400 return cls(manager, group=group, mode=mode, lock=lock, autoclose=autoclose)

File ~/miniconda3/envs/pytmd/lib/python3.11/site-packages/xarray/backends/netCDF4_.py:347, in NetCDF4DataStore.__init__(self, manager, group, mode, lock, autoclose)
    345 self._group = group
    346 self._mode = mode
--> 347 self.format = self.ds.data_model
    348 self._filename = self.ds.filepath()
    349 self.is_remote = is_remote_uri(self._filename)

File ~/miniconda3/envs/pytmd/lib/python3.11/site-packages/xarray/backends/netCDF4_.py:409, in NetCDF4DataStore.ds(self)
    407 @property
    408 def ds(self):
--> 409     return self._acquire()

File ~/miniconda3/envs/pytmd/lib/python3.11/site-packages/xarray/backends/netCDF4_.py:403, in NetCDF4DataStore._acquire(self, needs_lock)
    402 def _acquire(self, needs_lock=True):
--> 403     with self._manager.acquire_context(needs_lock) as root:
    404         ds = _nc4_require_group(root, self._group, self._mode)
    405     return ds

File ~/miniconda3/envs/pytmd/lib/python3.11/contextlib.py:137, in _GeneratorContextManager.__enter__(self)
    135 del self.args, self.kwds, self.func
    136 try:
--> 137     return next(self.gen)
    138 except StopIteration:
    139     raise RuntimeError("generator didn't yield") from None

File ~/miniconda3/envs/pytmd/lib/python3.11/site-packages/xarray/backends/file_manager.py:199, in CachingFileManager.acquire_context(self, needs_lock)
    196 @contextlib.contextmanager
    197 def acquire_context(self, needs_lock=True):
    198     """Context manager for acquiring a file."""
--> 199     file, cached = self._acquire_with_cache_info(needs_lock)
    200     try:
    201         yield file

File ~/miniconda3/envs/pytmd/lib/python3.11/site-packages/xarray/backends/file_manager.py:217, in CachingFileManager._acquire_with_cache_info(self, needs_lock)
    215     kwargs = kwargs.copy()
    216     kwargs["mode"] = self._mode
--> 217 file = self._opener(*self._args, **kwargs)
    218 if self._mode == "w":
    219     # ensure file doesn't get overridden when opened again
    220     self._mode = "a"

File src/netCDF4/_netCDF4.pyx:2485, in netCDF4._netCDF4.Dataset.__init__()

File src/netCDF4/_netCDF4.pyx:1863, in netCDF4._netCDF4._get_dims()

File src/netCDF4/_netCDF4.pyx:2029, in netCDF4._netCDF4._ensure_nc_success()

RuntimeError: NetCDF: Invalid dimension ID or name

### Anything else we need to know?

 I think that xarray is just being careful / doing some additional checks and refusing to "take chances", but the initial file looked good to me when I looked into it. But I would like to fully switch my workflows from netCDF4 to xarray, hence my report hoping this can help :) .
Ok, I did not investigate the health of the file following ncrename use. Do you think my call is valid (if so, should I open an issue at ncrename?), if not any pointers? :) .
Wondering if it could be doable to make xarray less "careful" about what it accepts to open, so that I could open the file without the issue that comes up when not doing the ncrename?
(for reference: see the updated notebook at the link above:
xr.open_dataset("./bar_BSO3_a_rcm7_2008_09.nc")

produces:
---------------------------------------------------------------------------
MissingDimensionsError                    Traceback (most recent call last)
<ipython-input-8-6a9791186661> in <module>
----> 1 xr.open_dataset("./bar_BSO3_a_rcm7_2008_09.nc")

~/.local/lib/python3.8/site-packages/xarray/backends/api.py in open_dataset(filename_or_obj, engine, chunks, cache, decode_cf, mask_and_scale, decode_times, decode_timedelta, use_cftime, concat_characters, decode_coords, drop_variables, inline_array, backend_kwargs, **kwargs)
    539
    540     overwrite_encoded_chunks = kwargs.pop("overwrite_encoded_chunks", None)
--> 541     backend_ds = backend.open_dataset(
    542         filename_or_obj,
    543         drop_variables=drop_variables,

~/.local/lib/python3.8/site-packages/xarray/backends/netCDF4_.py in open_dataset(self, filename_or_obj, mask_and_scale, decode_times, concat_characters, decode_coords, drop_variables, use_cftime, decode_timedelta, group, mode, format, clobber, diskless, persist, lock, autoclose)
    590         store_entrypoint = StoreBackendEntrypoint()
    591         with close_on_error(store):
--> 592             ds = store_entrypoint.open_dataset(
    593                 store,
    594                 mask_and_scale=mask_and_scale,

~/.local/lib/python3.8/site-packages/xarray/backends/store.py in open_dataset(self, store, mask_and_scale, decode_times, concat_characters, decode_coords, drop_variables, use_cftime, decode_timedelta)
     45         )
     46
---> 47         ds = Dataset(vars, attrs=attrs)
     48         ds = ds.set_coords(coord_names.intersection(vars))
     49         ds.set_close(store.close)

~/.local/lib/python3.8/site-packages/xarray/core/dataset.py in __init__(self, data_vars, coords, attrs)
    611             coords = coords.variables
    612
--> 613         variables, coord_names, dims, indexes, _ = merge_data_and_coords(
    614             data_vars, coords, compat="broadcast_equals"
    615         )

~/.local/lib/python3.8/site-packages/xarray/core/merge.py in merge_data_and_coords(data_vars, coords, compat, join)
    573     objects = [data_vars, coords]
    574     explicit_coords = coords.keys()
--> 575     return merge_core(
    576         objects,
    577         compat,

~/.local/lib/python3.8/site-packages/xarray/core/merge.py in merge_core(objects, compat, join, combine_attrs, priority_arg, explicit_coords, indexes, fill_value)
    753         coerced, join=join, copy=False, indexes=indexes, fill_value=fill_value
    754     )
--> 755     collected = collect_variables_and_indexes(aligned, indexes=indexes)
    756     prioritized = _get_priority_vars_and_indexes(aligned, priority_arg, compat=compat)
    757     variables, out_indexes = merge_collected(

~/.local/lib/python3.8/site-packages/xarray/core/merge.py in collect_variables_and_indexes(list_of_mappings, indexes)
    363                 append_all(coords_, indexes_)
    364
--> 365             variable = as_variable(variable, name=name)
    366             if name in indexes:
    367                 append(name, variable, indexes[name])

~/.local/lib/python3.8/site-packages/xarray/core/variable.py in as_variable(obj, name)
    160         # convert the Variable into an Index
    161         if obj.ndim != 1:
--> 162             raise MissingDimensionsError(
    163                 f"{name!r} has more than 1-dimension and the same name as one of its "
    164                 f"dimensions {obj.dims!r}. xarray disallows such variables because they "

MissingDimensionsError: 'six_con' has more than 1-dimension and the same name as one of its dimensions ('nchar', 'six_con'). xarray disallows such variables because they conflict with the coordinates used to label dimensions.

)

it appears we recently relaxed this: see #7989 (I totally missed that PR, hence my confusion). That means that after we release the next version you will be able to open the file as-is.
To rename the dimension, I think you would need to use ncrename -d six_con,six_con_dim old_file new_file. However, that also creates a corrupted netcdf file for me, so I'm not sure what to recommend. Even
import netcdf4 as nc

with nc.Dataset("bar_BSO3_a_rcm7_2008_09.nc", "r+") as ds:
    ds.renameDimension("six_con", "six_con_dim")
    ds.renameDimension("all_con", "all_con_dim")

does not appear to work, so I'm not sure what is wrong with that.

I can open the file using the netcdf4 engine even without the ncrename call (when did we relax the "cannot open files with self-referential coordinates" constraint?

Yes! https://github.com/pydata/xarray/pull/7989

Ok, so if I understand well, the issue with the initial dataset without ncrename should disappear (which would be enough to more or less close this issue), but there is an issue with ncrename, right? I think this is where this function is developed:
https://github.com/nco/nco/blob/master/src/nco/ncrename.c
Do you think I should report this issue on their repo at https://github.com/nco/nco for them to look into? :)

I don't know a lot about nco, but given that netcdf-python also seems to be affected this might be a bug in netcdf-c. In any case, opening an issue on the nco repository should help figuring out what exactly is going wrong.

So nco confirmed this is a bug with the netcdf library, and offered a workaround. Looks like the maintainers of the netcdf library do not have the resources to fix the issue at the moment.
Then I guess that this answer from nco, + the fact that a recent pull request fixes this, means that we can close this issue, right? Or am I forgetting something? :)

Thanks for following up on the NCO repository @jerabaul29

the fact that a recent pull request fixes this, means that we can close this issue, right? Or am I forgetting something? :)

Yes.


Add another answer regarding why data is missing from a server in app…

… insights.

@MarkIannucci : Thanks for your contribution to the Azure documentation! The author, @mrbullwinkle, has been notified to review your proposed change.

#sign-off


Disable failing OSM tests until can be properly addressed

See #2677

Frontend tests were OK :+1: (details)


adding action create_if_missing

need the ability to create crons in dev, but not manage them afterwards so developers can make changes without fear or chef overwriting.
also updates spec to have full coverage along with testing  action :delete

paging @someara
can you kindly pull this in for me? :grin:

It's probably time to remove the updated resource mess in the LWRP and just do use_inline_resources.  It would greatly simplify the code and it would just do the right thing for users.

agreed but right now this was a 15 min cleanup versus a rewrite.
i just noticed issue PR #64 has already converted it to the new resource style. I will rebase my changes off of that.

Looks like this broke deletion as well as the cron.d emulation.  I'm going to back out part of the template changes.

test-kitchen checks out fine and it has a spec for action :delete
do you have more details on this?

I've converted the LWRP to inline resources with a fix for the issue and expanded the test kitchen setup for the create_if_missing action https://github.com/chef-cookbooks/cron/pull/66

This was the reported issue btw: https://github.com/chef-cookbooks/chef-client/issues/352

thanks for fixing it up and leaving the action in. surprised that error cropped up and was valid previously.


HSV input

Hi,
I'm a bit stuck on the input side for a HSV object.
I pushing in an object with
msg.Hue
msg.Value
msg.Saturation
but I always get the error
"no output, normally means bad input"
Could you point where I'm wrong?
Thanks!

From the README

For the first 3 input modes the node will attempt to determine the input format of msg.payload from the 3 following:
...
a object with keys hue,saturation,lightness or value,brightness with values between 0-360 for hue and 0-100 for the others.

The input must be msg.payload.hue, msg.payload.saturation and msg.payload.value


[Documentation] tutorial on how to deploy a dream web server on a hosted service

Do you have any tips on how to deploy a dream web server for real, on the web, easily?
Do you personally use AWS? google cloud? Or something like Linode? How do you scale it?

No tips at the moment. I added a TODO item for working out AWS deployment after the forum discussion. However, I also took a minor break, so I haven't gotten around to it yet. I plan to add it before release, and will definitely ping this issue, a well.
I don't have direct experience with scaling beyond a couple Digital Ocean machines. Whenever I worked on a large enough project, we had an ops team doing that.
The project that motivated Dream hasn't reached that scale yet, not even close :) However, outside that project, I intend to run at least some synthetic tests on a fake large Dream deployment.

@aryx I've added several deployment examples:

Heroku (live), with a CI script for deploying from GitHub Actions. Heroku has built-in scaling support, at the price of putting constraints on your app's environment.
Digital Ocean with Docker.
Digital Ocean, as a systemd daemon.

In addition, the new playground is itself deployed to Digital Ocean as a systemd daemon, and can serve as a small but real example.
I will probably add an AWS example later on in development, as Dream becomes more mature. Heroku and Digital Ocean should be sufficient for medium-traffic applications, but AWS (or Azure, Google Cloud) will be necessary for huge deployments.

Fantastic!


the nmp install barfs over a missing bzip2

This might no longer be needed, but as a data point, the npm install was failing unable to unpact a bzip2 archieve.

This change is

LGTM


fix(react-router): prefer pendingComponent over defaultPendingComponent when wrapping Outlet in Suspense

This would resolve https://github.com/TanStack/router/issues/2030 yes? Since the user would now be able to add their own pendingComponent on the root route.

it maybe solves https://github.com/TanStack/router/issues/2026

it maybe solves #2026

Whoops, my bad, linked the wrong issue.

@tannerlinsley please have a look if omitting this suspense is ok, check #2026  for the motivation.
one thing I dislike is that there is no way to opt out of suspense here if defaultPendingComponent is set.

@SeanCassiere @schiller-manuel @tannerlinsley Is there any chance of getting at least the first commit merged?
That would allow me to work around this bug without having to patch the package itself.


Confusing and ambiguous error (FS0368) when implementing an interface with overloads

If you implement only one member of an interface with overloads without using specific typing you may well receive a very dubious error claiming your override has the wrong number of type parameters:

error FS0368: The member 'Bar<'a0> : 'a0 -> int' does not have the correct number of method type parameters. The required signature is 'Bar : double -> int'.

Repro steps
To see this in action, copy the following snippet into any F# project:
type IOverload =
    abstract member Bar : int -> int
    abstract member Bar : double -> int

type Overload =
    interface IOverload with
        override __.Bar _ = 1    // FS0368

Expected behavior
An error that explains what's going on. There are no type parameters here. Suggested text for error:

Ambiguous match: the generic method Bar matches multiple overloads of the same method, candidates are 'Bar : double -> int', 'Bar : int -> int'. Use a more specific type annotation for overriding the proper method.

Actual behavior
Actual behavior is that you see the following in the editor:

If you compile the project, you get a little more information, but it is still not covering the actual error:

error FS0368: The member 'Bar<'a0> : 'a0 -> int' does not have the correct number of method type parameters. The required signature is 'Bar : double -> int'.

error FS0368: The member 'Bar<'a0> : 'a0 -> int' does not have the correct number of method type parameters. The required signature is 'Bar : int -> int'.

error FS0017: The member 'Bar<'a0> : 'a0 -> int' does not have the correct type to override any given virtual method

Known workarounds
None (unless of course: learning what this error may mean and act on it).
Related information
Seen with:

Windows 7
.NET Runtime 4
VS 2025
F# 4.0

would that help?

That looks excellent, yes, that would certainly help!
Awe, so fast :))))

WIP PR at https://github.com/Microsoft/visualfsharp/pull/1907

As a belated follow-up (and to reassure anybody getting here through googling or other means), I can confirm this works in the current VS RTM, for both F# 4.0 and F# 4.1, and in the latest nightlies + previews:

Thanks again!

@abelbraaksma Thanks for checking!!


DBZ-2554 Edit topic routing/event flattening SMTs format/style/annotations/conditionalization

This is for DBZ-2554.
There are edits to topic-routing.adoc and event-flattening.adoc to make this doc consistent with the doc for other SMTs:

Change references to the SMT to plain text
Remove conditionalization and add nebel split annotations in comments
Ensure that headings contain "Debezium" for optimum search success
Standardize on "option" in place of "property"
Add some introductory sentences
Insert {zwsp} as needed.

These changes will be needed for the downstream 1.3 release. They are NOT needed for the 2020-Q3 (1.2) release.

Thanks @TovaCohen

@Naros  This PR needs to be backported to 1.3. After, this PR is backported, then PR-1839 will also need to be (merged and then) backported to 1.3.  Let me know if I messed up somewhere. Hope this all good. Thanks!

@TovaCohen, I'm confused. This PR has been sent against master (=1.3) in the first place? Why is it you say it should be backported to that one then?

@gunnarmorling  Here's what I see:

In the Debezium community doc, when Version = master, the heading in the topic routing configuration options table looks like this, with the error in the third column: https://debezium.io/documentation/reference/configuration/topic-routing.html#configuration-options

Also in the Debezium community doc, when Version = 1.3, you do not see that error:
https://debezium.io/documentation/reference/1.3/configuration/topic-routing.html#configuration-options

Why not?
And how do I make sure that the error gets fixed?

Oook, I see what's happening. The 1.3 branch didn't get updated during today's release, as it should have. I'll look into it, no need for you to take any action.

Ok, upstream/1.3 is updated and will reflect this change now (will take a bit until it's live). Before, upstream/1.3 still was pointing to 1.3.0.Beta2, which is why you didn't see any changes there coming after this Beta2, like the commit which apparently introduced this table rendering problem.


TA Review (VnV Report): Formatting and spelling issues

@samm82
Summary
Minor formatting issues (e.g., extra horizontal lines in LaTeX tables, inconsistent capitalization, LaTeX quotation marks) and spelling (e.g., it should be "iPhone", "iOS", and "walkthrough" or "walk-through"). "TBD" should be explained, either generally or on a case-by-case basis (like in Test-UH6).
Deadline
Apr 4, 2024
Estimated Work Hours
1.0

Completed in commit id: 48154ac


[Request] Bower package

Hello! Can you please consider adding this repo to bower?
Best

Sorry for the delayed response, i have been so busy lately,
you can easily install a git repo as a bower component
bower install https://github.com/shakee93/vue-toasted.git

this will install the latest version of the package. if you need install a specific version
check the versions here : https://github.com/shakee93/vue-toasted/releases
bower install https://github.com/shakee93/vue-toasted.git#v1.1.20

Hope that helps

Omg, sorry, never thought of this option.
Anyway, vue-toasted is now availableat cdnjs >> https://github.com/cdnjs/cdnjs/issues/12017
Ty for this repo!

Thanks buddy 🤞


add memory statistics

just peak memory for different parts of the block proving process

Does it collect sub-thread memory data? Just wonder sgx prover is arose by system command in raiko-host, is it memory also included??

Does it collect sub-process memory data? Just wonder sgx prover is arose by system command in raiko-host, is it memory also included??

ChatGPT: no

Processes and Memory Isolation
When you start a new process from your Rust application (for example, by using std::process::Command), the new process runs in its own separate memory space. This is a fundamental aspect of how modern operating systems handle processes. Each process has its own virtual address space, and the memory allocations within one process are completely isolated from those in another process.

Global Allocator Scope
The #[global_allocator] attribute in Rust allows you to specify a custom allocator that will be used for all dynamic memory allocations within the current Rust process. This includes allocations made by the Rust standard library types like Vec, Box, String, and so on. However, this allocator is only applicable within the context of the Rust process where it is defined. It does not influence how memory is allocated in other processes, including those launched from the Rust process.


Error "external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list"

Bugs should be filed for issues encountered whilst operating cert-manager.
You should first attempt to resolve your issues through the community support
channels, e.g. Slack, in order to rule out individual configuration errors.
Please provide as much detail as possible.

Describe the bug:
Hi
we have an error on several GKE clusters. We don't have this error everytime we re-install cert-manager and sometimes, everything works for several days/weeks and then it doesn't work anymore.
We didn't find a way to reproduce this bug.
Expected behaviour:
We install cert-manager with a remote base for kustomize :

first  : CRD install
second : cert-manager code

Steps to reproduce the bug:
Steps to reproduce the bug should be clear and easily reproducible to help people
gain an understanding of the problem.
Anything else we need to know?:
We get this error : to resolve this error, we need to delete cert-manager and cert-manager-cainjector pod and to re-create them in random order, and stop when it works.
E0428 11:53:29.744894       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1alpha2.CertificateRequest: certificaterequests.cert-manager.io is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "certificaterequests" in API group "cert-manager.io" at the cluster scope
E0428 11:53:29.753951       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1alpha2.Issuer: issuers.cert-manager.io is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "issuers" in API group "cert-manager.io" at the cluster scope
E0428 11:53:29.763561       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1.Service: services is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "services" in API group "" at the cluster scope
E0428 11:53:30.055311       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1alpha2.Order: orders.acme.cert-manager.io is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "orders" in API group "acme.cert-manager.io" at the cluster scope
E0428 11:53:30.055782       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1.Pod: pods is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "pods" in API group "" at the cluster scope
I0428 11:53:30.191037       1 controller.go:411] cert-manager/controller/webhook-bootstrap "level"=0 "msg"="existing Secret does not exist, creating new empty secret" "resource_kind"="Secret" "resource_name"="cert-manager-webhook-ca" "resource_namespace"="cert-manager"
E0428 11:53:30.195625       1 controller.go:438] cert-manager/controller/webhook-bootstrap "msg"="failed to create new empty Secret" "error"="secrets is forbidden: User \"system:serviceaccount:certificates:cert-manager-cainjector\" cannot create resource \"secrets\" in API group \"\" in the namespace \"cert-manager\"" "resource_kind"="Secret" "resource_name"="cert-manager-webhook-ca" "resource_namespace"="cert-manager"
I0428 11:53:30.196142       1 controller.go:411] cert-manager/controller/webhook-bootstrap "level"=0 "msg"="existing Secret does not exist, creating new empty secret" "resource_kind"="Secret" "resource_name"="cert-manager-webhook-tls" "resource_namespace"="cert-manager"
E0428 11:53:30.199745       1 controller.go:438] cert-manager/controller/webhook-bootstrap "msg"="failed to create new empty Secret" "error"="secrets is forbidden: User \"system:serviceaccount:certificates:cert-manager-cainjector\" cannot create resource \"secrets\" in API group \"\" in the namespace \"cert-manager\"" "resource_kind"="Secret" "resource_name"="cert-manager-webhook-tls" "resource_namespace"="cert-manager"
E0428 11:53:30.310532       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1alpha2.Challenge: challenges.acme.cert-manager.io is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "challenges" in API group "acme.cert-manager.io" at the cluster scope
E0428 11:53:30.535814       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1beta1.Ingress: ingresses.extensions is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "ingresses" in API group "extensions" at the cluster scope
E0428 11:53:30.666832       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1alpha2.ClusterIssuer: clusterissuers.cert-manager.io is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "clusterissuers" in API group "cert-manager.io" at the cluster scope
E0428 11:53:30.769740       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1.Service: services is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "services" in API group "" at the cluster scope
E0428 11:53:30.775933       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1alpha2.Issuer: issuers.cert-manager.io is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "issuers" in API group "cert-manager.io" at the cluster scope
E0428 11:53:30.793003       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1alpha2.CertificateRequest: certificaterequests.cert-manager.io is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "certificaterequests" in API group "cert-manager.io" at the cluster scope
E0428 11:53:31.067646       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1.Pod: pods is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "pods" in API group "" at the cluster scope
E0428 11:53:31.068080       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1alpha2.Order: orders.acme.cert-manager.io is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "orders" in API group "acme.cert-manager.io" at the cluster scope

We thinks that we have the right CRD :
$ kubectl get crds | grep cert-mana
certificaterequests.cert-manager.io            2020-03-03T23:14:24Z
certificates.cert-manager.io                   2020-03-03T23:14:24Z
challenges.acme.cert-manager.io                2020-03-03T23:14:25Z
clusterissuers.cert-manager.io                 2020-03-03T23:14:25Z
issuers.cert-manager.io                        2020-03-03T23:14:25Z
orders.acme.cert-manager.io                    2020-03-03T23:14:26Z

Environment details::

Kubernetes version (e.g. v1.10.2): 1.14.10-gke.32
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): GKE
cert-manager version (e.g. v0.4.0): quay.io/jetstack/cert-manager-controller:v0.12.0
Install method (e.g. helm or static manifests): kustomize (can provide source code if helpful)

Sorry, we can't help with more details or debug path or logs. We searched a lot about theses errors and the current way to solve it, is to delete everything and to retry until it works :(
If you need us to try several cmds or install/reinstall, we can try on a new cluster  or we can wait to find a current cluster which is not working anymore.
Thanks !
/kind bug

It seems your RBAC is missing, could you share more with us about your installation?
/triage support

@tpoindessous have you got any more info on this?

Hi !
Sorry for the delay, we are experiencing, again, this problem.
My co-worker @nlamirault will look at this.
Thanks !

We have this error :
E0706 14:30:45.583990       1 reflector.go:123] external/io_k8s_client_go/tools/cache/reflector.go:96: Failed to list *v1alpha2.ClusterIssuer: clusterissuers.cert-manager.io is forbidden: User "system:serviceaccount:certificates:cert-manager-cainjector" cannot list resource "clusterissuers" in API group "cert-manager.io" at the cluster scope

And the RBAC :
kubectl describe clusterrolebindings.rbac.authorization.k8s.io cert-manager-cainjector
Name:         cert-manager-cainjector
Labels:       app.kubernetes.io/component=certificates
              app.kubernetes.io/instance=certmanager
              app.kubernetes.io/managed-by=kustomize
              app.kubernetes.io/name=certmanager
              app.kubernetes.io/version=v0.12.0
              env=dev
              k8s-app=cainjector
Annotations:  kubectl.kubernetes.io/last-applied-configuration:
                {"apiVersion":"rbac.authorization.k8s.io/v1beta1","kind":"ClusterRoleBinding","metadata":{"annotations":{},"labels":{"app.kubernetes.io/co...
Role:
  Kind:  ClusterRole
  Name:  cert-manager-cainjector
Subjects:
  Kind            Name                     Namespace
  ----            ----                     ---------
  ServiceAccount  cert-manager-cainjector  certificates

$ kubectl describe clusterrolebindings.rbac.authorization.k8s.io cert-manager-cainjector
Name:         cert-manager-cainjector
Labels:       app.kubernetes.io/component=certificates
              app.kubernetes.io/instance=certmanager
              app.kubernetes.io/managed-by=kustomize
              app.kubernetes.io/name=certmanager
              app.kubernetes.io/version=v0.12.0
              env=dev
              k8s-app=cainjector
Annotations:  kubectl.kubernetes.io/last-applied-configuration:
                {"apiVersion":"rbac.authorization.k8s.io/v1beta1","kind":"ClusterRoleBinding","metadata":{"annotations":{},"labels":{"app.kubernetes.io/co...
Role:
  Kind:  ClusterRole
  Name:  cert-manager-cainjector
Subjects:
  Kind            Name                     Namespace
  ----            ----                     ---------
  ServiceAccount  cert-manager-cainjector  certificates

kubectl describe clusterrole cert-manager-cainjector cert-manager-cainjector
Name:         cert-manager-cainjector
Labels:       app.kubernetes.io/component=certificates
              app.kubernetes.io/instance=certmanager
              app.kubernetes.io/managed-by=kustomize
              app.kubernetes.io/name=certmanager
              app.kubernetes.io/version=v0.12.0
              env=dev
              k8s-app=cainjector
Annotations:  kubectl.kubernetes.io/last-applied-configuration:
                {"apiVersion":"rbac.authorization.k8s.io/v1beta1","kind":"ClusterRole","metadata":{"annotations":{},"labels":{"app.kubernetes.io/component...
PolicyRule:
  Resources                                                     Non-Resource URLs  Resource Names  Verbs
  ---------                                                     -----------------  --------------  -----
  events                                                        []                 []              [get create update patch]
  mutatingwebhookconfigurations.admissionregistration.k8s.io    []                 []              [get list watch update]
  validatingwebhookconfigurations.admissionregistration.k8s.io  []                 []              [get list watch update]
  customresourcedefinitions.apiextensions.k8s.io                []                 []              [get list watch update]
  apiservices.apiregistration.k8s.io                            []                 []              [get list watch update]
  secrets                                                       []                 []              [get list watch]
  certificates.cert-manager.io                                  []                 []              [get list watch]

Name:         cert-manager-cainjector
Labels:       app.kubernetes.io/component=certificates
              app.kubernetes.io/instance=certmanager
              app.kubernetes.io/managed-by=kustomize
              app.kubernetes.io/name=certmanager
              app.kubernetes.io/version=v0.12.0
              env=dev
              k8s-app=cainjector
Annotations:  kubectl.kubernetes.io/last-applied-configuration:
                {"apiVersion":"rbac.authorization.k8s.io/v1beta1","kind":"ClusterRole","metadata":{"annotations":{},"labels":{"app.kubernetes.io/component...
PolicyRule:
  Resources                                                     Non-Resource URLs  Resource Names  Verbs
  ---------                                                     -----------------  --------------  -----
  events                                                        []                 []              [get create update patch]
  mutatingwebhookconfigurations.admissionregistration.k8s.io    []                 []              [get list watch update]
  validatingwebhookconfigurations.admissionregistration.k8s.io  []                 []              [get list watch update]
  customresourcedefinitions.apiextensions.k8s.io                []                 []              [get list watch update]
  apiservices.apiregistration.k8s.io                            []                 []              [get list watch update]
  secrets                                                       []                 []              [get list watch]
  certificates.cert-manager.io                                  []                 []              [get list watch]

The yaml output :
kubectl get clusterrole cert-manager-cainjector -o yaml
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"rbac.authorization.k8s.io/v1beta1","kind":"ClusterRole","metadata":{"annotations":{},"labels":{"app.kubernetes.io/component":"certificates","app.kubernetes.io/instance":"certmanager","app.kub
ernetes.io/managed-by":"kustomize","app.kubernetes.io/name":"certmanager","app.kubernetes.io/version":"v0.12.0","env":"dev","k8s-app":"cainjector"},"name":"cert-manager-cainjector"},"rules":[{"apiGroups":["cert-m
anager.io"],"resources":["certificates"],"verbs":["get","list","watch"]},{"apiGroups":[""],"resources":["secrets"],"verbs":["get","list","watch"]},{"apiGroups":[""],"resources":["events"],"verbs":["get","create",
"update","patch"]},{"apiGroups":["admissionregistration.k8s.io"],"resources":["validatingwebhookconfigurations","mutatingwebhookconfigurations"],"verbs":["get","list","watch","update"]},{"apiGroups":["apiregistra
tion.k8s.io"],"resources":["apiservices"],"verbs":["get","list","watch","update"]},{"apiGroups":["apiextensions.k8s.io"],"resources":["customresourcedefinitions"],"verbs":["get","list","watch","update"]}]}
  creationTimestamp: "2020-04-27T17:03:00Z"
  labels:
    app.kubernetes.io/component: certificates
    app.kubernetes.io/instance: certmanager
    app.kubernetes.io/managed-by: kustomize
    app.kubernetes.io/name: certmanager
    app.kubernetes.io/version: v0.12.0
    env: dev
    k8s-app: cainjector
  name: cert-manager-cainjector
  resourceVersion: "200645276"
  selfLink: /apis/rbac.authorization.k8s.io/v1/clusterroles/cert-manager-cainjector
  uid: f313d97e-88a8-11ea-b4a9-42010a84002a
rules:
- apiGroups:
  - cert-manager.io
  resources:
  - certificates
  verbs:
  - get
  - list
  - watch
- apiGroups:
  - ""
  resources:
  - secrets
  verbs:
  - get
  - list
  - watch
- apiGroups:
  - ""
  resources:
  - events
  verbs:
  - get
  - create
  - update
  - patch
- apiGroups:
  - admissionregistration.k8s.io
  resources:
  - validatingwebhookconfigurations
  - mutatingwebhookconfigurations
  verbs:
  - get
  - list
  - watch
  - update
- apiGroups:
  - apiregistration.k8s.io
  resources:
  - apiservices
  verbs:
  - get
  - list
  - watch
  - update
- apiGroups:
  - apiextensions.k8s.io
  resources:
  - customresourcedefinitions
  verbs:
  - get
  - list
  - watch
  - update

I check also the release manifest file (v0.12.0. It is the same ClusterRole .
Any idea ?

Strange logs :
I0706 17:14:29.452158       1 leaderelection.go:241] attempting to acquire leader lease  certificates/cert-manager-controller...
I0707 09:35:15.736749       1 leaderelection.go:251] successfully acquired lease certificates/cert-manager-controller
I0707 09:35:15.738231       1 controller.go:172] cert-manager/controller/certificaterequests "level"=0 "msg"="new certificate request controller registered"  "type"="ca"
I0707 09:35:15.738505       1 controller.go:172] cert-manager/controller/certificaterequests "level"=0 "msg"="new certificate request controller registered"  "type"="selfsigned"
I0707 09:35:15.738650       1 controller.go:172] cert-manager/controller/certificaterequests "level"=0 "msg"="new certificate request controller registered"  "type"="vault"
I0707 09:35:15.738652       1 controller.go:101] cert-manager/controller "level"=0 "msg"="starting controller" "controller"="certificaterequests-issuer-selfsigned"

After these lines ... everything works

So it seems to work now?

Looks like this issue is resolved
/close

Looks like this issue is resolved
/close


Motion Sense is always precise

Despite there being plenty of instances where its not.
Version 6.0.1

Again you are correct! It is by definition precise.

Doesn't seem to be by definition, only by example. Out of curiosity to the two of you:

@stwlam Where is the definition that its precise? I can't find this definition anywhere (I looked)
@MrVauxs what are the other instances? It might be that it has different behavior for PCs, so I'd need to know those instances.

@MrVauxs what are the other instances? You said there's multiple.

Ultimately ended up being two after examining every instance of the word in the books more thoroughly. The feat I linked above and Oozeform Chair. That is of course excluding homebrew (and thus Summoners+).
In either case if its only precise for creatures lacking vision, then the PR fixing this still should be merged so it can be an imprecise for creatures that do have it.


[TFL] Fix unbounded activation clip range.

Fixes #45312. For floating point with unbounded activation such as relu and none, set max/min to positive/negative infinity in case infinity is clipped. The change only affects floating point as std::numeric_limits<T>::has_infinity is always false for non-floating point. Doing min(max(x, -inf), inf) is safe because positive infinity is larger than all values except itself and NaN and negative infinity is smaller than all values expect itself and NaN. With the change, TFLite is the same with TF for relu and none activation when input is positive or negative infinity.
https://www.gnu.org/software/libc/manual/html_node/Infinity-and-NaN.html
https://en.cppreference.com/w/cpp/types/numeric_limits/has_infinity

This PR will be hold until we can invest to make a thorough investigation about how current ops are handling inf value to ensure there is no compatibility issues.


Get Toyota safety param from DBC

For toyota, we pass a torque scaling factor to panda which is defined in the DBC. Instead of duplicating the scale factor in selfdrive/car/toyota/interface.py, we should get it from the DBC.

Not needed after #23635


java.util.ImmutableCollections$List12 is not in the allowlist

Describe the bug
We use Spring Authorization Server with some fixed users in our Project in the Test Environment
. There it runs in a Docker container.
I think, since the last version, is get a http 500 when i try to login, after the Server has run for more than 24h.
I am sorry to not be able to provide a better description. But i will attach the log.
I solve the problem by restaring the Pod, then all works fine.
ms-backend-test-authorization-server-mock.txt

Closing this as a duplicate. See this comment for details.


Cant build project

Hello!
In vercel and locally, I am getting these errors. What can I do to fix this?
branch: marketing-starter
03:48:30.840
info  - Generating static pages (0/3)
03:48:31.577
03:48:31.577
Error occurred prerendering page "/". Read more: https://nextjs.org/docs/messages/prerender-error
03:48:31.577
TypeError: Cannot read property 'modules' of null
03:48:31.577
    at Object.Home (/vercel/path0/.next/server/pages/index.js:2799:28)
03:48:31.577
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2523)
03:48:31.577
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2667)
03:48:31.577
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2667)
03:48:31.577
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:1576)
03:48:31.577
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:1767)
03:48:31.577
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2667)
03:48:31.578
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:1576)
03:48:31.578
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2667)
03:48:31.578
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2667)
03:48:31.600
03:48:31.600
Error occurred prerendering page "/404". Read more: https://nextjs.org/docs/messages/prerender-error
03:48:31.600
TypeError: Cannot read property 'seo' of null
03:48:31.600
    at Object.ErrorPage (/vercel/path0/.next/server/pages/404.js:5151:17)
03:48:31.600
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2523)
03:48:31.600
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2667)
03:48:31.601
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2667)
03:48:31.602
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:1576)
03:48:31.602
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:1767)
03:48:31.602
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2667)
03:48:31.602
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:1576)
03:48:31.602
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2667)
03:48:31.602
    at m (/vercel/path0/node_modules/preact-render-to-string/dist/commonjs.js:1:2667)
03:48:31.602
03:48:31.602
> Build error occurred
03:48:31.602
Error: Export encountered errors on following paths:
03:48:31.602
	/
03:48:31.602
	/404
03:48:31.602
    at /vercel/path0/node_modules/next/dist/export/index.js:31:1106
03:48:31.602
    at async Span.traceAsyncFn (/vercel/path0/node_modules/next/dist/telemetry/trace/trace.js:5:584)
03:48:31.602
    at async /vercel/path0/node_modules/next/dist/build/index.js:43:49
03:48:31.602
    at async Span.traceAsyncFn (/vercel/path0/node_modules/next/dist/telemetry/trace/trace.js:5:584)
03:48:31.602
    at async /vercel/path0/node_modules/next/dist/build/index.js:25:1475
03:48:31.603
    at async Span.traceAsyncFn (/vercel/path0/node_modules/next/dist/telemetry/trace/trace.js:5:584)
03:48:31.603
info  - Generating static pages (3/3)
03:48:31.651
npm ERR! code ELIFECYCLE
03:48:31.651
npm ERR! errno 1
03:48:31.654
npm ERR<EMAIL_ADDRESS>build: `next build`
03:48:31.654
npm ERR! Exit status 1
03:48:31.654
npm ERR!
03:48:31.654
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
03:48:31.654
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
03:48:31.660
03:48:31.660
npm ERR!  It sounds like you haven't published any of the required documents in Sanity to properly generate the pages of your site. Make sure you have a connected sanity project with (at least) initial documents created for each singleton (all settings).

Closing as I think this has been addressed.


Make ALERT_TRIGGERED_OR_RESOLVED placeholder values configurable

We're currently using Gatus for triggering health checks and having an internal dashboard. In order to unify with our public status reporting we'd like to use Gatus to set the status of components on our statuspage.io status page.
I've configured a custom alerting provider which works great when a service goes down. But I don't see a way to report it as up again at the moment. Statuspage.io requires different strings than TRIGGERED and RESOLVED to distinguish the statuses. Currently they are hard coded here ALERT_TRIGGERED_OR_RESOLVED:
https://github.com/TwinProduction/gatus/blob/master/alerting/provider/custom/custom.go#L48
The configuration is as follows. I use the ALERT_DESCRIPTION to identify the service name at statuspage.io.
alerting:
  custom:
    url: "https://api.statuspage.io/v1/pages/XXX/components/[ALERT_DESCRIPTION]"
    method: "PATCH"
    headers:
      Authorization: "OAuth YYY"
    body: |
      { "component": { "status": "operational" } }

services:
  - name: foo
    url: "https://this.is.down"
    interval: 1m
    conditions:
      - "[STATUS] == 200"
    alerts:
      - type: custom
        enabled: true
        send-on-resolved: true
        description: a9gbjb11q8gy

Status of a component could be any of
"operational" "under_maintenance" "degraded_performance" "partial_outage" "major_outage" ""

(see https://developer.statuspage.io/#operation/patchPagesPageIdComponentsComponentId).
It would be great if the two strings were configurable on alerting.custom level, making the custom alert feature even more flexible. Maybe like this, defaulting to the current hardcoded-values:
alerting:
  custom:
    url: "https://api.statuspage.io/v1/pages/XXX/components/[ALERT_DESCRIPTION]"
    method: "PATCH"
    triggered_message: "partial_outage"
    resolved_message: "operational"
    headers:
      Authorization: "OAuth YYY"
    body: |
      { "component": { "status": "[ALERT_TRIGGERED_OR_RESOLVED]" } }

Gatus is great and keeps getting better. I wanted to say thanks for providing this neat tool and actively maintaining it!

Fascinating - so you're using Gatus to push data onto a different status page?
I was originally afraid people wouldn't get creative enough with the custom alerting provider, but it looks like my worries were unnecessary.
But yeah, this seems like a good feature to add. My only concern is how to expose these configurations.
Off the top of my head, I think something like this would be acceptable:
alerting:
  custom:
    url: "https://api.statuspage.io/v1/pages/XXX/components/[ALERT_DESCRIPTION]"
    method: "PATCH"
    placeholders:                        <--
      ALERT_TRIGGERED_OR_RESOLVED:       <--
        triggered: "partial_outage"      <--
        resolved: "operational"          <--
    headers:
      Authorization: "OAuth YYY"
    body: |
      { "component": { "status": "[ALERT_TRIGGERED_OR_RESOLVED]" } }

Fascinating - so you're using Gatus to push data onto a different status page?
I was originally afraid people wouldn't get creative enough with the custom alerting provider, but it looks like my worries were unnecessary.
But yeah, this seems like a good feature to add. My only concern is how to expose these configurations.
Off the top of my head, I think something like this would be acceptable:
alerting:
  custom:
    url: "https://api.statuspage.io/v1/pages/XXX/components/[ALERT_DESCRIPTION]"
    method: "PATCH"
    placeholders:                        <--
      ALERT_TRIGGERED_OR_RESOLVED:       <--
        triggered: "partial_outage"      <--
        resolved: "operational"          <--
    headers:
      Authorization: "OAuth YYY"
    body: |
      { "component": { "status": "[ALERT_TRIGGERED_OR_RESOLVED]" } }

Yes, as the two tools serve slightly different purposes, we want to leverage the combination of both. Gatus is able to do the actual health check and also monitor all internal services, the public status page is used for displaying some of the result to the public, incident communication and notification subscription.
Regarding your proposal for the configuration syntax: That is definitely better than my initial idea :+1:
A question on a slightly different topic that just came up: If I understand the current configuration syntax of alerts correctly it is possible to use Slack and a custom alerting method at the same time but not two different custom alerts, correct? Just out of curiosity if we ever wanted to add a second custom alert.

Yes, as the two tools serve slightly different purposes, we want to leverage the combination of both. Gatus is able to do the actual health check and also monitor all internal services, the public status page is used for displaying some of the result to the public, incident communication and notification subscription.
Regarding your proposal for the configuration syntax: That is definitely better than my initial idea :+1:
A question on a slightly different topic that just came up: If I understand the current configuration syntax of alerts correctly it is possible to use Slack and a custom alerting method at the same time but not two different custom alerts, correct? Just out of curiosity if we ever wanted to add a second custom alert.

Yes, as the two tools serve slightly different purposes, we want to leverage the combination of both. Gatus is able to do the actual health check and also monitor all internal services, the public status page is used for displaying some of the result to the public, incident communication and notification subscription.
Regarding your proposal for the configuration syntax: That is definitely better than my initial idea :+1:
A question on a slightly different topic that just came up: If I understand the current configuration syntax of alerts correctly it is possible to use Slack and a custom alerting method at the same time but not two different custom alerts, correct? Just out of curiosity if we ever wanted to add a second custom alert.

Yes, as the two tools serve slightly different purposes, we want to leverage the combination of both. Gatus is able to do the actual health check and also monitor all internal services, the public status page is used for displaying some of the result to the public, incident communication and notification subscription.
Regarding your proposal for the configuration syntax: That is definitely better than my initial idea :+1:
A question on a slightly different topic that just came up: If I understand the current configuration syntax of alerts correctly it is possible to use Slack and a custom alerting method at the same time but not two different custom alerts, correct? Just out of curiosity if we ever wanted to add a second custom alert.

@GeorgFleig That's correct, Gatus currently only supports one of each alerting providers, so at most, you could have 5 providers configured (1 slack, 1 pagerduty, 1 mattermost, 1 twilio and 1 custom).

@GeorgFleig That's correct, Gatus currently only supports one of each alerting providers, so at most, you could have 5 providers configured (1 slack, 1 pagerduty, 1 mattermost, 1 twilio and 1 custom).

FYI, I made some changes and the syntax is now:
alerting:
  custom:
    placeholders:
      ALERT_TRIGGERED_OR_RESOLVED:
        TRIGGERED: "partial_outage"
        RESOLVED: "operational"

Released in v2.2.0


Flat oak package needs update - square symbols under fedora

I confirm (by marking "x" in the [ ] below: [x]):

[x] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/.
[x] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[x] I have read contributing guidelines.

Summary

Square symbols are back even in flatpack package
All other electron app are working fine under flatpack wether it's signal element or others.
So according to my previous issue about square in snap package, your version of electron is not the up to date one.
Thanks to update the app with the latest electron builder.
Environment

Operating System: fedora 34
Mattermost Desktop App version: 4.7.0 commit b757742
Mattermost Server version: X.X.X

Steps to reproduce
Expected behavior
No more square symbol in the system sub menu in gnome-shell interaction menu right click on your icon
Observed behavior

Square symbol

Possible fixes
Update electron builder as suggested in the numerous threads in snapcraft forum which has basically the same problem than flatpack because of electron, and my other issues here.

Hello and thanks for the report, I'm afraid we are not the ones submitting the app to flathub, you should probably submit an issue in the submitters repo or even to flatpack itself. as you mention this affects multiple applications based on electron and either electron or flatpack should consider what is needed for keeping compatibility.
For an update on the version, I'm afraid you'll need to wait until the next major version, we don't usually upgrade the version on dot releases unless it is really critical. The reason for that is that moving to the next one usually has breaking changes and might have unexpected behaviour.
I'm closing the issue but feel free to reopen if you think there is anything we can really do to help with this without an upgrade. Although as I mentioned this should be handled by either electron or flatpack.


[WIP] [CSL-2298] Implement tests for duplicate VssCerts

This PR adds tests for CT-100. Though, it only adds one check instead of required two (because I'm not sure how the second one should be implemented). Though, at least test pass for me locally.

Tests pass locally for me. If everything is ok (and if I understood task properly) I can then squash commits to make commit history cleaner.


[DO NOT MERGE] Added GX OSS

Description
Added GX OSS examples
Related to: Neptune+GX Colab
Any expected test failures?

Add a [X] to relevant checklist items
❔ This change

[X] adds a new feature
[ ] fixes breaking code
[ ] is cosmetic (refactoring/reformatting)

✔ Pre-merge checklist

[X] Refactored code (sourcery)
[X] Tested code locally
[X] Precommit installed and run before pushing changes
[X] Added code to GitHub tests (notebooks, scripts)
[X] Updated GitHub README
[ ] Updated the projects overview page on Notion

🧪 Test Configuration

OS: Windows11
Python version: 3.10.11
Neptune version: 1.10.4
Affected libraries with version: great_expectationn==0.18.16

It seems some of Edyta's items were resolved, but without the changes being applied. If not applying
a suggestion, let's maybe keep the thread open until there's some agreement or the reviewer resolves it?

It seems some of Edyta's items were resolved, but without the changes being applied. If not applying a suggestion, let's maybe keep the thread open until there's some agreement or the reviewer resolves it?

Looks like the changes were applied in https://github.com/neptune-ai/examples/pull/328/commits/057fe476ff7312c5cfe46b2ca6e0e947ab35b9d0 and threads automatically marked as resolved, but the next commit to fix the notebook format https://github.com/neptune-ai/examples/pull/328/commits/c9f10d493ebd38fe7b07a38ac2399d1c6dce5d2a was not based on these changes and overwrote them.
Let me fix this


xxd: Fix that color was disabled on Cygwin

"windows.h" was unintentionally included on Cygwin since 9.0.1834. This accidentally disabled coloring on Cygwin.
Stop including "windows.h" on Cygwin.

thanks!


nil-pointer in getTestSigChan

After upgrading to the latest digest I am seeing:
panic: interface conversion: interface {} is nil, not <-chan os.Signal

goroutine 108 [running]:
github.com/oklog/run.getTestSigChan(...)
	/go/pkg/mod/github.com/oklog/run@v1.1.1-0.20231129054641-daf88ede3394/actors.go:51
github.com/oklog/run.SignalHandler.func1()
	/go/pkg/mod/github.com/oklog/run@v1.1.1-0.20231129054641-daf88ede3394/actors.go:31 +0x355
github.com/oklog/run.(*Group).Run.func1({0xc0006a1710?, 0xc000690a50?})
	/go/pkg/mod/github.com/oklog/run@v1.1.1-0.20231129054641-daf88ede3394/group.go:38 +0x29
created by github.com/oklog/run.(*Group).Run in goroutine 1
	/go/pkg/mod/github.com/oklog/run@v1.1.1-0.20231129054641-daf88ede3394/group.go:37 +0x67

getTestSigChan is only used within tests, but causes an nil-pointer if used outside.
I was able to reproduce the nil-pointer: https://go.dev/play/p/JFXak3UIrB6
From what I can tell there needs to be a nil check on the value retrieved from the context before the type assertion.

How are you calling getTestSigChan in the first place?


IsListNullOrEmptyConverter documentation

Documentation added for .NET MAUI Community Toolkit - [Proposal] ListIsNullOrEmpty Converters https://github.com/CommunityToolkit/Maui/issues/69

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

File Change Percent
This PR contains file(s) with more than 20% file change.

Let's make sure to update the TOC to include this doc too.
Maybe something like this?
|    Setup
└─── Converters
|    └─── IsListNullOrEmptyConverter

Let's make sure to update the TOC to include this doc too.
Maybe something like this?
└─── Setup
└─── Converters
|    └─── IsListNullOrEmptyConverter

Good catch thanks!

Docs Build status updates of commit 1d430b3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/converters/is-list-not-null-or-empty-converter.md
:white_check_mark:Succeeded
View

docs/maui/converters/is-list-null-or-empty-converter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

Added File(s)
This PR contains added files. New files require human review.

Yaml File(s)
This PR includes changes to .yml file(s) owned by another author.

File Change Percent
This PR contains file(s) with more than 20% file change.

Thanks Shaun! Let's make sure to include C# Examples as well for folks like me who prefer C# over XAML 😊
I think it's best to recommend CommunityToolkit.Maui.Markup and include it in the C# sample since Maui.Markup makes using Converters much easier, and it will also help raise awareness for the  CommunityToolkit.Maui.Markup NuGet Package.
Here's an example using the sample from the Proposal:
Syntax
XAML
// XAML example
C#

Note: This C# Syntax leverages the CommunityToolkit.Maui.Markup NuGet Package

using CommunityToolkit.Maui.Markup;

class MyPage : ContentPage
{
  public MyPage()
  {
    Content = new Label().Bind(Label.IsVisible, nameof(ViewMode.MyList), converter: new IsListNullOrEmptyConverter();
  }
}

On the topic of Markup, I assume it should have its own folder and contents in the docs/TOC? Then once that is there this reference to it could link to that parts of the docs? If so I might knock up a quick Markup page first so I can get the link right

Great idea! Let's do it!

[ ] Add Banner for 'preview' information

Docs Build status updates of commit 12b34f9:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

docs/maui/converters/is-list-not-null-or-empty-converter.md
:x:Error

Details

docs/maui/converters/is-list-null-or-empty-converter.md
:x:Error

Details

docs/maui/converters/is-list-not-null-or-empty-converter.md

Line 10, Column 1: [Error: include-not-found] Invalid include link: 'includes/preview-note.md'.

docs/maui/converters/is-list-null-or-empty-converter.md

Line 10, Column 1: [Error: include-not-found] Invalid include link: 'includes/preview-note.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 05b6865:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 09ef8c8:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/maui/TOC.yml
:warning:Warning
View
Details

docs/maui/converters/is-list-not-null-or-empty-converter.md
:warning:Warning
View
Details

docs/maui/converters/is-list-null-or-empty-converter.md
:warning:Warning
View
Details

docs/maui/TOC.yml

Line 13, Column 13: [Warning: file-not-found - See documentation] Invalid file link: 'converters/is-list-null-or-empty-converter.m'.

docs/maui/converters/is-list-not-null-or-empty-converter.md

Line 16, Column 78: [Warning: file-not-found - See documentation] Invalid file link: '../is-list-null-or-empty-converter.md'.
Line 65, Column 5: [Warning: file-not-found - See documentation] Invalid file link: 'markup/markup.md'.

docs/maui/converters/is-list-null-or-empty-converter.md

Line 16, Column 78: [Warning: file-not-found - See documentation] Invalid file link: '../is-list-not-null-or-empty-converter.md'.
Line 64, Column 5: [Warning: file-not-found - See documentation] Invalid file link: 'markup/markup.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit de9016b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/maui/converters/is-list-not-null-or-empty-converter.md
:warning:Warning
View
Details

docs/maui/converters/is-list-null-or-empty-converter.md
:warning:Warning
View
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/converters/is-list-not-null-or-empty-converter.md

Line 65, Column 5: [Warning: file-not-found - See documentation] Invalid file link: '../markup/markup.md'.

docs/maui/converters/is-list-null-or-empty-converter.md

Line 64, Column 5: [Warning: file-not-found - See documentation] Invalid file link: '../markup/markup.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Current warnings will be resolved by #13

Docs Build status updates of commit 262b95b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/TOC.yml
:white_check_mark:Succeeded
View

docs/maui/converters/is-list-not-null-or-empty-converter.md
:white_check_mark:Succeeded
View

docs/maui/converters/is-list-null-or-empty-converter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


firing api request on decline button in terminated state not working

please using 2.0.1 and check example https://github.com/hiennguyen92/flutter_callkit_incoming/blob/master/example/ios/Runner/AppDelegate.swift


Converting Word to PDF failed and garbled characters appeared.

I have added LANGS correctly.
but still failed

Is it because of insufficient font support? In the Word document I used italics.

can you show your container startup logs and also run the command
apk list --installed | grep font

can you show your container startup logs and also run the command
apk list --installed | grep font

I also have the same situation, and it’s still the same after running this command

Can you show the output of the command *

Issue still ongoing?


Fixed graph to work on MS edge

Fixed minor issue with ampersand in html
Changed scoping of "latest" variable to make Edge happy
Commented out unused THRESHOLD_RANGE_VALUES as flatMap isn't supported in Edge
Removed localStorage - not supported by Edge when loading from a file, and never actually used on the web site.

Thank you for the changes, I've merged them in. I didn't merge the PR directly, because I wanted to keep the local file cache behavior. From what you said, this shouldn't effect Edge at all, because most users won't be loading from a file:// URL, right?
I have it there for 2 reasons:

When working on development, the page doesn't keep hitting the AirRater endpoint every time I reload.
I eventually want to use that "cache" to avoid hitting AirRater for users who refresh the page multiple times per hour.

Does 42de0f85 capture what you were going for?


leaks when using with addView/removeView

I'm attempting to use a NucleusLayout as the base class for my view.  Because the view life cycle is not tied to the activity life cycle, Nucleus is creating a new Presenter each time addView() is used, but not destroying it when removeView() is used.  Due to the way PresenterHelper and NucleusLayour are implemented I am unsure how to fix this issue without duplicating the classes in their entirety.

It should not. I'll check this today.

More specifically, the issue is caused by PresenterHelper only choosing to destroy a presenter in the case that activity.isFinishing() is true.
The fact that the presenter is not destroyed is actually a potentially desirable behavior, in the instance that I want to keep certain presenters around even when their related views are pushed/popped off of the stack. Right now this is not possible because PresenterHelper always creates a new instance of the presenter due to "requestPresenter(view.getClass(), null);"
I would imagine there are a few ways to solve this.  Originally I did it by adding an annotation to the presenter class, and then copying the code from DefaultPresenterManager to a custom version that returned the (leaked) instance when providing a presenter of that type.  This seems less than ideal.
The biggest issue I tend to run into is that everything in nucleus is locked away.  NucleusLayout has a PresenterHelper, but it's private so I can't replace it with my own subclass.

Sorry, I can not reproduce your issue. Here is the test:
https://github.com/konmik/nucleus/blob/master/nucleus/src/androidTest/java/nucleus/view/NucleusLayoutTest.java#L45
Probably you have an issue somewhere in another place. Can you provide me an example that reproduces the issue?

The PresenterHelper creates a new presenter instance only if it has been destroyed.

I don't have an example I can put up at the moment, but I'll try to explain what I think causes it:

Activity is created
NucleusLayout view is created, which creates a Presenter and a PresenterHelper
NucleusLayout is added to Activity with addView()
NucleusLayout is removed from Activity with removeView()

At this point the NucleusLayout is destroyed, which means that PresenterHelper is also destroyed.  However, because the main Activity has not been destroyed, the check on this line:
https://github.com/konmik/nucleus/blob/master/nucleus/src/main/java/nucleus/view/PresenterHelper.java#L56
... is not true, so the Presenter is not destroyed.

Ok, I see. This behavior is intentional.
There is NucleusLayout.destroyPresenter() method. https://github.com/konmik/nucleus/blob/master/nucleus/src/main/java/nucleus/view/NucleusLayout.java#L79
It is public for cases like this. When you're detaching a view that has a lifecycle that is different to Activity, you need to call destroyPresenter() to destroy the presenter. A view does not know will it be attached again or not. So it can not just destroy presenter, and this decision is left to the caller.
I have no idea what is the best solution for this case. In my early releases I had enum OnDetachedAction {NONE, DESTROY_PRESENTER, DESTROY_IF_FINISHING} inside of NucleusLayout, but I've never used it.
I think I need to highlight this moment in the introduction article.
Thank you for your contribution. :)

Argh... just when I made a bug case too...
https://github.com/scriptacus/Nucleus-Leak
So here's the thing, I'd actually like the keep that Presenter around for use the next time that view is created.  Ideally I could just retain the PresenterHelper, or subclass it in some way.

If you use view's save/restore, your presenter should be re-attached to a view that is restored from that bundle. A pure miracle. ;)

Sorry this is turning into an android QA thread.
Is there a way to do that without the PresenterHelper creating a new instance of the Presenter?  It seems that addView calls onAttachedToWindow() which calls helper.takeView(), which calls requestPresenter with a null bundle.  I can probably override onAttachedToWindow() in my view class, but then I would need to pass some sort of state to the view telling it to wait for an onRestoreInstanceState() instead of calling super.onAttachedToWindow().

You can call view's restore yourself. Here is an extract from one of my projects:
public Parcelable freeze(View view) {
    SparseArray<Parcelable> states = new SparseArray<>();
    view.saveHierarchyState(states);
    Bundle bundle = new Bundle();
    bundle.putSparseParcelableArray(STATE_KEY, states);
    return bundle;
}

public View unfreeze(Parcelable parcelable) {
    Bundle bundle = (Bundle)parcelable;
    View view = inflate(...);
    view.restoreHierarchyState(bundle.getSparseParcelableArray(STATE_KEY));
    return view;
}

P.S. I'm happy to provide support - you're helping to make this library better. :)

The introduction article has been enhanced with "View recycling and view stack" section.
http://konmik.github.io/introduction-to-model-view-presenter-on-android.html

Can you provide any details on how to implement this sort of save/restore with StackLayout + Nucleus?    I've been able to do it by restoring the view state (saved in a hashmap on the activity) in each View's onAttachedToWindow, but that seems a bit hacky.

Try this:
stackLayout.set().setOnDestroyViewListener(new StackLayout.OnDestroyViewListener() {
@Override
public void onExitView(View view) {
if (view instanceof NucleusLayout)
((NucleusLayout)view).destroyPresenter();
}
        @Override
        public void onFreezeView(View view) {
            if (view instanceof NucleusLayout)
                ((NucleusLayout)view).destroyPresenter();
        }
    }).apply();


Demo is broken

... and the Glitch is gone.
Made a Glitch with A-Frame 1.4 here (edit).
Thanks for the component, Kyle!
Cheers, Dirk

Thanks for the info, will update link.
https://a-super-sky.glitch.me/ has always been up, guess I forgot to update here? will check.


Boolean filter is shown as check box

This is code:
class SomeFilter(BaseModel):
    is_foo: bool | None = Field(default=None, json_schema_extra={"placeholder": "Filter by Foo..."})

What I actually want is this:

but it render as this:

Workaround is to use Literal["true","false"] on filter, and then switch it to boolean on table view. But I'm questioning if bool should be always checkbox or not? i'd leave that to user tbh. sometimes it doesn't make sense.
Another bug would be that the json schema extra doesn't even show up.
and the last bug is that It also doesn't submit search/filter form when clicking on it, which is another bug in its state.

Adding this to the v2.6.0 milestone - not sure we'll implement this change, but definitely something we'll consider!

Just circling back - I think we'd want to add support for both!


Make where recursive

I'd like for @buzzdecafe to weigh in on this one, as I think where has a special place in his heart.  If nothing else, it was a focus of the very first article on Ramda, which he published nearly two years back.
But I'm happy to see this.  It adds a bit of both power and complexity to this function, but make the library as a whole more consistent.
:herb:

i'm happier with whereEq and where as is. The current definitions are simple, respectively: specify equality constraints as values on object properties; and specify arbitrary constraints as functions on object properties. This was an improvement over the original where because it removed the run-time type-checking: whereEq is a String -> value mapping, where is String -> (Any -> Boolean). Simple.
i expect (although i have no data) that those two functions cover the overwhelming majority of uses. i certainly have not  heard any rumbling of discontent that the function are insufficient.
Cases where recursive descent is desired can be implemented by the user, possibly nesting where* within the predicate. I strongly prefer that to adding complexity to the implementation and the overhead of type-checking all the keys of the spec.

@buzzdecafe:
I suspected that might be your reaction.  I should have also pointed to the motivation in #1592.
Perhaps we should be looking at restricting evolve instead.
What do others think of this?  @davidchambers, you're our strongest advocate for library-wide consistency.  How do you think this should play out?  (Or is evolve simply off your radar since lenses should cover it anyway?)

I agree with @buzzdecafe.
Ad hoc polymorphism provides convenience at the cost of simplicity. Adding a special case for one particular data type inevitably leads to difficult questions as to whether other data types deserve special treatment as well. In this case I can imagine someone requesting a special case for Array, so one could write R.where({address: {street: [R.equals('123 Main Street')]}}).


Frontend: allow process to appear multiple times

DEUS and Reliability-Service need to be called mutliple times. Accomodate that in frontend.

Atm, our Frontend creates a graph of processes and products. It arranges them linearly using toposort. However, a topological order is only possible iff the graph is a DAG. This is problematic, since having a process be run multiple times introduces circles.

option 1 is to just duplicate the process with another id.
option 2 would be to include the information that every edge in the graph may only be passed through once. (Never done that before, will do research on this.)

Maybe a simple solution: Do not use the name of the process implementation for the graph node, but instead use a description of the goal that should be achieved.

Really, I was being silly here. What we're looking for here is a (hamiltonian) walk through the graph, not an order of the nodes.


Request for Explosive category for Attack Animations

It might be cool to add another category in the default settings for ranged light explosives. It would search for things with blast and limited ammo weapon qualities and apply the animation and audio when playing those ranged light weapons. Almost all explosives have those two qualities. (minus glop grenades and a handful of others).

If you can find a reasonable (and open-source) animation for it, I'll include it.  Otherwise, I think this is covered by permitting users to set it on a per-item basis.

I requested a grenade toss one in the JB2A site. In the meantime perhaps you can use the Magic Missile one with an Explosion Orange at the end from here?
https://www.jb2a.com/Library_Preview/
modules/JB2A_DnD5e/Library/1st_Level/Magic_Missile/MagicMissile_01_Regular_Purple_30ft_01_1600x400.webm
modules/JB2A_DnD5e/Library/Generic/Explosion/Explosion_01_Orange_400x400.webm

I've added it for items with the word "grenade" in their name. Thanks for the suggestion!
Will be released in 0.2.1.


Introduce WorkerProcessor in QueryEngine

Description
This PR introduce basic WorkerProcessor framework in Trino to help develop future Window Functions and UDF in Table Model.

Add these files in LICENSE https://github.com/apache/iotdb/blob/master/LICENSE

PTAL~


Does not generate correct SHA or show correct status on merge commits

Working example: https://github.com/FundingCircle/us-direct-borrower/pull/628
I've made two commits to this PR. On each commit, I opened up the status checks window, clicked 'details' next to the shipment tracker check to create a feature review, and everything worked as expected

However, since the thing we're most interested in tracking is the unit of work, this is best represented by the merge commit. When we view the details on a merge commit, we can see the shipment tracker status check

Two issues occur:

the 'details' link takes me to a feature review for the latest commit in this PR, and not the merge commit itself
the status check erroneously says 'no feature review found' when the details link points to an existing FR

We believe that the correct behaviour is that the details link for the merge commit should point to an FR with the SHA of the merge commit, and that the status check should accurately represent the presence or absence of that FR.
In the mean time, a workaround for the first issue is to manually create the FR from the merge commit SHA, either through the UI or through manual URL manipulation.

Hey Tim,

the 'details' link takes me to a feature review for the latest commit in this PR, and not the merge commit itself

Feature Reviews are for a specific software version, which is the commit SHA. This could also be the SHA of a merge commit, but you would have to manually prepare such a FR. For example, this one is for a merge commit.
The Pull Request Status is for convenience, and it always refers to the head of the branch. Note that the Status is tied to a specific commit, it's really the status of the head of the branch, not the status of the PR.
It should be possible to create a Status for merge commits, but that Status won't show up in the PR because the commit lives on the master branch. But even if it did show, the Status panel becomes hidden when the PR is merged.
Many teams have gates pre-merge, and in this case the Shipment Tracker PR Status works very well. But as mentioned above, you could theoretically have FRs for merge commits, the workflow just won't be as convenient.

When we view the details on a merge commit, we can see the shipment tracker status check

The GitHub Statuses are for the head of the topic branch. If you merge the branch, the details in the Pull Request's Status panel still refer to the last commit on the topic branch.

the status check erroneously says 'no feature review found' when the details link points to an existing FR

Thanks for reporting this. The Status hasn't been updated for whatever reason. We'll investigate.

The Pull Request Status is for convenience, and it always refers to the head of the branch. Note that the Status is tied to a specific commit, it's really the status of the head of the branch, not the status of the PR.

It should be possible to create a Status for merge commits, but that Status won't show up in the PR because the commit lives on the master branch. But even if it did show, the Status panel becomes hidden when the PR is merged.

Many teams have gates pre-merge, and in this case the Shipment Tracker PR Status works very well. But as mentioned above, you could theoretically have FRs for merge commits, the workflow just won't be as convenient.

I hear you, and this makes sense. I think we're encountering a pain point centered around a difference between US and UK process. Thanks for clarifying this for me; I was unsure if this was a bug or just how it works.


FR: Provide a way to inspect what version of electron vscode is using

I'd like to know what version of electron visual studio code is using, mainly to address bugs related to docker containers such as this: electron/electron#10345

F1 > developer tools > in console type
process.versions.electron


Make icon customizable

Feature request: add a config option to choose custom image for rpc logo

This should now be possible with #7 by using your own Discord App ID and uploading your own images to the Discord App Dashboard, but do you want to define your own asset names as well (instead of using logo and nowplaying)?

Just FYI:
https://github.com/nekowinston/mpd-rich-presence/blob/cfc78627eba04d946cac226bfc2ce69c2f8eb618/main.go#L352-L353
We could also pass in arbitrary URLs here, they will show up in Discord; this is how things like Neovim Rich Presence work: https://github.com/nekowinston/neovim.drv/blob/96d46ac5979cc762aa49716a957326f450711e7d/plugins/presence.lua#L52-L84

Hmm, if we pass in a URL, I imagine we'll have to cache it with ttlcache as well? How would we change the config to fit this feature?

Closing this as not planned since I'm archiving this repo.


Openshift knative release 0.13 fix image

Image used in Dockerfile during https://github.com/openshift-knative/eventing-operator/pull/3 fails in https://github.com/openshift/release/pull/7812
docke...hel8/go-toolset not found: does not exist or no pull access

Using the image we previously used now.

cc @matzew

/lgtm

/approve

/just-fucking-merge-already


Add KidSunday to Contributors List

Added my name based on the contributors file instructions at:
https://github.com/Roshanjossey/first-contributions/blob/master/Contributors.md

@KidSunday, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?
We’ve added a tweet feature to our web app recently. Can you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Tag the lib please

I had to tag the library (0.0.1) in my fork to make my composer require it.
Could you add a tag, please?
So that I don't have to use a forked repository to load it.

Added a tag (0.0.1). Thanks for reminding!

You are great - thanks! :+1:


DevMode generates invalid schema: Invalid schema change (UPLOAD): destructive schema change

Goals
Create a synced schema based on a current local realm schema. The schema contains relationships
Expected Results
When running MongoDb Realm in devmode, the schema should be generated correctly.
Actual Results
Sync fails since it says that it is an invalid schema change.
"Sync: Connection[2]: Session[2]: Received: ERROR "Invalid schema change (UPLOAD): destructive schema change: cannot change field "deletedPropertyName" in table "AnObject" from "string" to a relationship" (error_code=225, try_again=0)"
It looks like all relationships in the generated schema is of type "string" instead of relationship and this will cause sync to fail even though I am running the app in dev mode with the goal of generating a schema
Steps for others to Reproduce
1: Create an realm schema containing a relationship
2: Run MongoDb Realm in devmode
Version of Realm and Tooling
Realm framework version: 10.7.5

Invalid issue. I am closing this. The issue is that the "relationship" in the local database becomes a "string" in the generated schema. Will create a new issue for this instead.

Looks like all relationships in my old schema gets converted to strings in MongoDb Realm. Why is this? According to the docs here https://docs.mongodb.com/realm/sdk/react-native/fundamentals/relationships-and-embedded-objects/ it should be possible to use relationships in MongoDb Realm. The way it works now is unintuitive. Do I need to change the schema manually?

I think I figured out what is happening here. All "relationships" in embedded objects are automatically changed from a relationship to a string in the Atlas schema when using devmode. This makes sync fail with "Invalid schema change (UPLOAD): destructive schema change". I assume this is a bug? That the relationship that exists in the models should generate a relationship in Atlas as well?
Models I used are the same as the ones I posed in the sample for https://github.com/realm/realm-cocoa/issues/7145

More weirdness. The schema in atlas actually contains all the relationships, but the generated swift classes have converted all  relationships in EmbeddedObject classes to plain strings. Not sure what is going on here.

Hi @sipersso Can you please share your Atlas schema definition, so I can help you with this issue?

@dianaafanador3 I am not able to share it here, but I have uploaded it to the support portal (ticket 00771573) and asked them to forward it to you and added contact details if you need any more input.

Hello @sipersso , I got everything I needed for testing it out, I'll let you know if I'm able to solve this as soon as possible.

Hmm... I actually managed to remove the property that caused the invalid schema. It was not necessary to have this in the sync schema and now the sync has actually started working. I am still not sure why this has started working, but it does.

Hi @sipersso given your last comment, are you ok to now close this issue?

Yes, I have verified that the sync work after turning dev mode off. Closing the issue.


回收PreparedStatement时，不仅clearParameters，还要clearBatch，因为Query对象的batchedArgs里的批量数据需要清除

回收PreparedStatement时，不仅clearParameters，还要clearBatch，因为Query对象的batchedArgs里的批量数据需要清除。

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


Newer gems and pip packages in docs build

From this Dockerfile.  Mainly updating software versions.

Hi @mclow could this be merged? Now is a good time in between boost release cycles.
A summary of test results: After upgrading everything, 99% of the boost library documentation continued to build correctly. The only problems were a few formatting issues in gil, hof, and python. They use Sphinx, and the newer version of Sphinx was not entirely compatible. I sent pull requests to those libraries. Gil and Hof have merged the changes. Python is the remaining holdout. At the time of this writing boostorg/python has 29 open pull requests. In that case though with Python, the documentation still compiles, and is mostly fine. Shouldn't be a blocker.