NameError: Field name "X" shadows an attribute in parent "Event"; you might want to use a different field name with "alias='X'"

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
No idea what's happening. Here's the trace:
.venv/lib/python3.10/site-packages/_pytest/runner.py:341: in from_call
    result: Optional[TResult] = func()
.venv/lib/python3.10/site-packages/_pytest/runner.py:372: in <lambda>
    call = CallInfo.from_call(lambda: list(collector.collect()), "collect")
.venv/lib/python3.10/site-packages/_pytest/python.py:531: in collect
    self._inject_setup_module_fixture()
.venv/lib/python3.10/site-packages/_pytest/python.py:545: in _inject_setup_module_fixture
    self.obj, ("setUpModule", "setup_module")
.venv/lib/python3.10/site-packages/_pytest/python.py:310: in obj
    self._obj = obj = self._getobj()
.venv/lib/python3.10/site-packages/_pytest/python.py:528: in _getobj
    return self._importtestmodule()
.venv/lib/python3.10/site-packages/_pytest/python.py:617: in _importtestmodule
    mod = import_path(self.path, mode=importmode, root=self.config.rootpath)
.venv/lib/python3.10/site-packages/_pytest/pathlib.py:565: in import_path
    importlib.import_module(module_name)
../../../../.local/homebrew/Cellar/python@3.10/3.10.7/Frameworks/Python.framework/Versions/3.10/lib/python3.10/importlib/__init__.py:126: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
<frozen importlib._bootstrap>:1050: in _gcd_import
    ???
<frozen importlib._bootstrap>:1027: in _find_and_load
    ???
<frozen importlib._bootstrap>:1006: in _find_and_load_unlocked
    ???
<frozen importlib._bootstrap>:688: in _load_unlocked
    ???
.venv/lib/python3.10/site-packages/_pytest/assertion/rewrite.py:178: in exec_module
    exec(co, module.__dict__)
tests/unit/test_events.py:10: in <module>
    class TestEventV1(events.Event):
.venv/lib/python3.10/site-packages/pydantic/_internal/_model_construction.py:172: in __new__
    set_model_fields(cls, bases, config_wrapper, types_namespace)
.venv/lib/python3.10/site-packages/pydantic/_internal/_model_construction.py:400: in set_model_fields
    fields, class_vars = collect_model_fields(cls, bases, config_wrapper, types_namespace, typevars_map=typevars_map)
.venv/lib/python3.10/site-packages/pydantic/_internal/_fields.py:154: in collect_model_fields
    raise NameError(
E   NameError: Field name "dataschema" shadows an attribute in parent "Event"; you might want to use a different field name with "alias='dataschema'".

Removing dataschema as an attribute complains about specversion
Something to do with typing.ClassVar with defaults?
Thank you for any help !
Example Code
# skm/event.py
from __future__ import annotations

import datetime
import typing
import uuid

import pydantic
import uuid_extensions  # type: ignore[import]

class Event(pydantic.BaseModel):
    specversion: typing.ClassVar[String] = "1.0"
    dataschema: typing.ClassVar[Uri | None] = None
    type: typing.ClassVar[String]
    source: typing.ClassVar[UriReference]

    id: String = pydantic.Field(default_factory=lambda: str(uuid_extensions.uuid7()))
    subject: String | None = None
    time: Timestamp = pydantic.Field(default_factory=lambda: datetime.datetime.now(tz=datetime.timezone.utc))
    data: Data | None = None

    class Config:
        frozen: bool = True

# tests/unit/test_event.py
from __future__ import annotations

import typing

import pytest

from skm import events

class TestEventV1(events.Event):
    source: typing.ClassVar[str] = "skm.api"
    type: typing.ClassVar[str] = "skm.api.test.created.v1"
    data: Data

class Data(events.Data):
    test_value: str

test_event = TestEventV1(data=Data(test_value="test"))

Python, Pydantic & OS Version
pydantic version: 2.1.1
        pydantic-core version: 2.4.0
          pydantic-core build: profile=release pgo=false mimalloc=true
                 install path: /Users/Eimantas/git/skm/api/skm-python/.venv/lib/python3.10/site-packages/pydantic
               python version: 3.10.7 (main, Mar 13 2023, 10:00:01) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
                     platform: macOS-13.4.1-arm64-arm-64bit
     optional deps. installed: ['typing-extensions']

Selected Assignee: @davidhewitt

@sidekick-eimantas can you please share the definitions of type aliases so that I can reproduce your original report?

This is by design. We don't allow you to shadow parent attributes.

I stumbled across this fact by accident, too. While I can sort of guess the motivation behind this decision, I would appreciate it, if you @adriangb or someone else from the team could elaborate on that.
https://github.com/pydantic/pydantic/blob/d5a42210e70516b4b0984db3faf94367e7c08425/pydantic/_internal/_fields.py#L149-L157
Why exactly am I not permitted to shadow any parent class' attribute? I get that there are special attributes that should never be overwritten by a field, but a blanked ban for this is strange to me.
I understand that the collect_model_fields function (called only during class/model creation) removes attributes that represent fields from the class namespace. This means they do not trigger this error, when the model is being subclassed. But there are use cases, when I do want to overwrite a non-field attribute in a parent with a field in the subclass.
If the reason for this blanked ban is documented or if a discussion about it exists, would you be so kind and link it here?

I found the same error appears, while the property on the parent class is an abstractmethod. Do you consider this a bug or design as well?

Similar question, why UserWarning?. In my case, I'm trying to define an interface of a kind(since abstract classes don't seem to work). Here's a small example:
class Packet(BaseModel):
    @property
    def metadata(self) -> SpecialMeta | None:

    @property
    def payload(self) -> Payload | None:

class OnePacketSource(BaseModel):
    metadata: SpecialMeta = Field(alias="some_meta", default=None)
    payload = None

This way if someone needs to write a new implementation of the packet parsing the see the interface they need to implement. Now, of course, I'm getting the UserWarning about shadowing attrigbutes.

I think both cases are bugs; in case in the OP the attribute being shadowed is a ClassVar. I think that's valid Python and not something Pydantic should care about.
The case of @OlegZv / @TiansuYu, I think that it's probably also fine to have an abstract property in the base class which becomes a real field in the child class.
PRs are welcome to help with both cases 👍


⚠ Bofrost.de has degraded performance

In 8b40137, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1510 ms

Resolved: Bofrost.de performance has improved in f345b8f after 6 minutes.


Azure CLI 2.0 commands fail when running in CloudShell

If I try to run an Azure CLI 2.0 command in SimDem, it fails:

$ $ echo "hello world"
hello world

$ az vm list -d -o table
/home/aleksandar/simdem/simdem-env/bin/python: Error while finding spec for 'azure.cli' (ImportError: No module named 'azure')

$ az account list
/home/aleksandar/simdem/simdem-env/bin/python: Error while finding spec for 'azure.cli' (ImportError: No module named 'azure')

(simdem-env) aleksandar@Azure:~/bin$

Hmmmm.... I've done some testing of this and it looks like we need to configure the Azure CLI inside the Virtualenv created for SimDem.
I added the virtualenv because Cloudshell would not allow the installation of some packages in the default virtualenv.
The question is, do we see if we can make Simdem install inside the default env or do we see if we can make Azure CLI work within the SimDem virtual env... This requires some research
In the meantime I recommend running in a Docker container in some other environment. It runs just fine in Docker for Windows.

@jluk Any thoughts on this issue as it relates to Cloudshell and the AZ CLI.

closed in error

My primary interest right now is to run Simdem in the Cloud Shell because I want to use it next week at the conference.
I would prefer if both, Azure CLI and Simdev, could run in the default virtualenv. With Simdev virtualenv, we might expect to have some configuration issue with other preinstalled tools in Cloud Shell.

OK. Well this worked before I added the flask requirement. This requirement is not used in Cloudshell. Therefore we can identify a point in git history that you can use safely.
I'm on my phone right now so can't do it immediately, but I'll find it and tag it early tomorrow.

I've put a temporary workaround in place for you and will fix this more permanently when I'm able to. Do this to take advantage:
git pull
git checkout tags/CloudShell
pip install -r requirements.txt # don't worry about "already satisfied" this is just a precaution
mkdir -p ~/bin/simdem-dev # probablu already exists from earlier install attempts
cp -r * ~/bin/simdem-dev
chmod +x ~/bin/simdem-dev/main.py
ln -s ~/bin/simdem-dev/main.py ~/bin/simdem # will probably complain you already have this from previous installs

I hope this unblocks you for your upcoming demo.

Now that we have a workaround in place we need to fix the larger issue which is recorded in #19

I'd have to look deeper into the virtualenv issue when I get a moment, it would seem cleaner to get simdem to work properly within Cloud Shell instead of pairing CLI within simdem's env if possible.
Adding @yugangw-msft in case he has any insight into CLI working within a virtualenv.

Closing while we refactor SimDem which will change the nature of this issue, and maybe even solve it.


feature(pu): add muzero

Description

add board_games env for muzero, including tictactoe, gomoku, chess, go, atari.
add muzero config for atari and tictactoe, 为EfficientZero适配atari and tictactoe env.
在atari和tictactoe env上，game buffer和game history 的单元测试
add eval() and _forward_eval() method for MuZero policy, add base_serial_evaluator_muzero.py

Related Issue
TODO

game history和game buffer连接起来的单元测试
优化muzero policy collect train
muzero policy 的单元测试
atari 整体流程跑通:collect  train eval

Check List

[ ] merge the latest version source branch/repo, and resolve all the conflicts
[ ] pass style check
[ ] pass all the tests

We split this pr into 2 separate self-consistent pr https://github.com/opendilab/DI-engine/pull/356 and https://github.com/opendilab/DI-engine/pull/358


Configuring search extension before deployment

Hi!
I would like to configure the bing search extension globally on my deployment of azurechat.
Is that possible? I couldn't find any information on it in the docs.
Thanks,

Make sure you are logged in with the user account that is set as the "admin" for your instance then you should see the publish option to allow everyone to see it :).

Thanks, that worked like a charm!

Sorry for necroing, but how did you specifically do this? 😊

@AbasBicep what part specifically are you stuck at?

@AbasBicep
For running locally, this will be entered in the env.local (check the env.example file for more info)
For hosting in Azure, check the app service (web app) in azure. There will be an Environment Variable
called ADMIN_EMAIL_ADDRESS that you can edit to add your 365 email address to promote you to an admin.

@bwitzig-zen Wow this worked wonders.
I am just curious; how did you find this out? I did not have that env variable in my deployment, but after adding it, it worked like a charm.

https://github.com/microsoft/azurechat/blob/main/docs/5-add-identity.md#configure-an-admin-user


How make dropdownlist  in editForm

Hi volkanceylan.
I have a "encoding" column in a table , the type is string .  I want to add some value such as UTF-8,GBK to the editForm  ,  But not use enumType  ,
Is there have a attribute can I add them.

https://github.com/volkanceylan/Serenity/issues/436

[Editor]
public class XYZEditor : Select2Editor<object, object>
{
public XYZEditor(jQueryObject hidden)
: base(hidden, null)
{
AddItem("AC", "AC");
AddItem("DC", "DC");
}
}
sorry!
I want to know that where should I code them?

If you mean where to actually create this file? You should create the file in the entity folder under ~/Modules directory where you plan to use the editor. Create the file XYZEditor.cs.

I also meet the problem
#752
the following code is my EncodingEditor.  is there something wrong?
@Serenity.Decorators.registerEditor()
export class EncodingEditor extends Serenity.Select2Editor<Object, Object> {
constructor(div: JQuery) {
super(div, Option);
super.addItem("gb2312", "gb2312");
super.addItem("UTF-8", "UTF-8");
super.addItem("GBK", "GBK");
}


Issues with MySQL test cases

[ ] RMLTC0018a-MySQL The test assumes that MySQL is ANSI SQL compliant, but it is not. MySQL does trim trailing spaces. Hence the test will wrongly accuse of engines not to produce correct RDF. I would suggest removing RMLTC0018a-MySQL.
[ ] RMLTC0015a-MySQL The JDBC URL in the mapping does not set the sql-mode to ANSI_SQL. This test case thus assumes that this SQL mode is enabled on the server. This is not only be default not the case, but the community encourages not to do so as it can be specified at connection level. So options here are: remove the test, add the sql mode in the jdbc connection url, or explicitely state all assumptions of the MySQL server.
[x] RMLTC0002f-MySQL seems to be a valid mapping. Everything matches (table name, and column names).
[x] RMLTC0003b-MySQL needs to return an error (and thus no output file with triples) as the source contains both a table name and a query.

Yes. Either remove the tests for MySQL. Or have something that would pass for them and write a note. RML engine should try and support PostgreSQL (which is more ANSI compliant).
I'm pretty sure 2f is not wrong
I proposed a fix for 3b. The output of 3b was not empty. All other failing mappings had empty output files. So, I believe that the additional rml:tableName was a copy-paste error. :-) It makes sense, as the reference relied on a derived attribute (concatenate). In other words, 3b is fixed.

Some other issues:
rml:SQL2008TableName is used vs rml:SQL2008Table (in io spec)
rml:tableName used but not described io spec
rml:sqlVersion used but described io spec
Actions:

[ ] replace rml:SQL2008TableName with  rml:SQL2008Table
[ ] remove rml:tableName and rml:sqlVersion

2f should use delimited identifiers as in https://www.w3.org/2001/sw/rdb2rdf/test-cases/#R2RMLTC0002f

[ ] RMLTC0012b-MySQL missing reference formulation

I'm leaning towards dropping RMLTC0018a entirely due to the different behavior of databases and this rather niche case. Agreed @chrdebru @DylanVanAssche ?

+1 for dropping 18a

Agree on dropping 18a.

agree on dropping 18a too. But we probably want an IO test to cover "Description: Generation of triples by using CHAR datatype column, resulting RDF literal is space-padded. (specification reference)" at a later stage. Tracked at https://github.com/kg-construct/rml-io/issues/41 so this issue can be closed after stuff is fixed


How to stop BarcodeReaderView or release the camera resource.?

After barcode is scanned, software should release the camera resource.
Otherwise, the camera can't be used by the rest of the app.
Do you know how to release the resource? Thank you very much.

This project is dead

It's also not possible to use NFC service in the rest of the app after scanning.

I have the same problem on iOS

You release the camera resource by disposing of the parent Shell class. Create a new Shell class and add a ShellContent whose ContentTemplate points to the ContentPage with the Zxing component. To access this scanner page, use these two lines of code:
Application.Current.MainPage = new YourScannerShellClass();
await Shell.Current.GoToAsync("yourscannerroute");

To navigate back to the main shell from the scanner page,
Application.Current.MainPage = new AppShell();
await Shell.Current.GoToAsync("yourlastmainshellroute");

I have same problem. Is this project really dead? Is there another alternative?

I found this: https://www.youtube.com/watch?v=FuvFrIS9wm0

I solved this. that not bug.
我将代码clone下来研究了一下，是因为DisconnectHandler（MAUI用于清理使用资源的方法）没有被触发，在MAUI中，Handler中的DisconnectHandler是需要手动触发的。
你需要在ContentPage的构造函数中：
        Unloaded += (sender, e) =>         {             cameraBarcodeReaderView.Handler.DisconnectHandler();         };
ref: https://learn.microsoft.com/zh-cn/dotnet/maui/user-interface/handlers/create?view=net-maui-7.0#native-view-cleanup

I solved this. that not bug.
我将代码clone下来研究了一下，是因为DisconnectHandler（MAUI用于清理使用资源的方法）没有被触发，在MAUI中，Handler中的DisconnectHandler是需要手动触发的。 你需要在ContentPage的构造函数中：
 Unloaded += (sender, e) => { cameraBarcodeReaderView.Handler.DisconnectHandler(); };
记住在XAML页面增加控件的标识名称：
 <zxing:CameraBarcodeReaderView x:Name="cameraBarcodeReaderView" x:FieldModifier="public" />
其实说不是一个BUG的话，我感觉也是属于设计不合理，应该在控件订阅Unloaded方法自动调用DisconnectHandler，不需要使用者手动去调用的！
ref: https://learn.microsoft.com/zh-cn/dotnet/maui/user-interface/handlers/create?view=net-maui-7.0#native-view-cleanup

I have one page that can turn off the camera using this solution, but another page with the same config except setting set to QRCode => using this way it still display camera is running. Any help?, please? @silmooninc

I had a same problem and solved by https://github.com/Redth/ZXing.Net.Maui/issues/118#issuecomment-1540628217. Thank you!

Here's a translation of @silmooninc's post:
I cloned the code and researched it because the DisconnectHandler (the method used by MAUI to clean up resources) was not triggered. In MAUI, the DisconnectHandler in the Handler needs to be triggered manually.
You need in the constructor of ContentPage:

         Unloaded += (sender, e) => { cameraBarcodeReaderView.Handler.DisconnectHandler(); };

Remember to add the identification name of the control to the XAML page:

        <zxing:CameraBarcodeReaderView x:Name="cameraBarcodeReaderView" x:FieldModifier="public" />

In fact, if it is not a BUG, I feel that it is also an unreasonable design. The DisconnectHandler should be automatically called in the control subscription Unloaded method, and the user does not need to call it manually!

If i do this, the camera will be released, but once in a while it throws "VirtualView cannot be null here".

Here's a translation of @silmooninc's post:

I cloned the code and researched it because the DisconnectHandler (the method used by MAUI to clean up resources) was not triggered. In MAUI, the DisconnectHandler in the Handler needs to be triggered manually.

You need in the constructor of ContentPage:

         Unloaded += (sender, e) => { cameraBarcodeReaderView.Handler.DisconnectHandler(); };

Remember to add the identification name of the control to the XAML page:

        <zxing:CameraBarcodeReaderView x:Name="cameraBarcodeReaderView" x:FieldModifier="public" />

In fact, if it is not a BUG, I feel that it is also an unreasonable design. The DisconnectHandler should be automatically called in the control subscription Unloaded method, and the user does not need to call it manually!

If i do this, the camera will be released, but once in a while it throws "VirtualView cannot be null here".

根据我给的提示是没问题的，如果有问题找一下自己的其他原因，不过我现在已经弃用这个项目了，正在使用新的实现：https://youtu.be/FuvFrIS9wm0


Fix 3951

[x] Closes #3951
[x] Tests added
[x] Passes isort -rc . && black . && mypy . && flake8
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API

It looks
dataframe.set_index(index).index == index is not always true.
Added a workaround for this...

Hello @fujiisoup! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file xarray/core/dataset.py:

Line 4606:1: W293 blank line contains whitespace

In the file xarray/tests/test_dataarray.py:

Line 3520:60: W291 trailing whitespace
Line 3521:64: W291 trailing whitespace

Thanks a lot @fujiisoup


Broken path parsing for install spec

$ uv --version
uv 0.2.35
$ uv pip install --editable '/home/phil/Dev/Python/Packaging, Docs/session-info2'
error: Failed to parse: `/home/phil/Dev/Python/Packaging, Docs/session-info2`
  Caused by: Expected path (`/home/phil/Dev/Python/Packaging,`) to end in a supported file extension: `.whl`, `.zip`, `.tar.gz`, `.tar.bz2`, `.tar.xz`, or `.tar.zst`
/home/phil/Dev/Python/Packaging, Docs/session-info2
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Thanks!
cc @charliermarsh might a regression from #5888

I can take a look.

Ah, sorry, I should have said that there is also an error with older versions before that PR (e.g. 0.2.32)
The underlying issue of something splitting the path at the space is older.

I think we currently require that you use a backslash here.

(But of course it should work as-is.)

I wasn’t aware that escape sequences in dependency specs are a thing at all!

Yeah it's a bit tricky. Like this doesn't work:
-e /Users/crmarsh/workspace/uv/foo bar/black_editable

But it does work if you quote it, or pass it directly on the command-line etc.

I am quoting it, see initial comment.

Yeah I'm describing what the behavior should be, not what we do today.

I see! Thank you for clarifying!


timeout

store task and implement timeout, avoiding that (and possible memory leak):
[1 workers; 1 clients; 3 topics; 0 tasks]

ATM we build a lazy check because recv is blocking the thread.
I have to dig tokyo then

timeout should look after tasks without workers too

look at
ZMQ > 3.0 allows you to set the socket's RCVTIMEO option, which will cause its recv() to raise a timeout error, without the need of a Poller object.

https://stackoverflow.com/questions/16157518/reconnecting-to-zmq-feed-after-disconnect


Update CMap Tips

Added a GIF for changing arrowheads style

Please do not put a 2 MB binary file into the git repo

I was able to get it down to 500k using Camtasia. OK? Also fixed some broken links.


Missing ModularVisualization in rainfall example?

Describe the bug
Cloned the repo and run the coverage tests:
$ pytest --cov=mesa_geo tests/
>   from mesa_geo.visualization.ModularVisualization import ModularServer
E   ModuleNotFoundError: No module named 'mesa_geo.visualization.ModularVisualization'
examples/rainfall/rainfall/server.py:5: ModuleNotFoundError

Expected behavior
Running the test suite should not give any errors
Indeed I don't see the module anymore in the visualization folder of mesa-geo
Maybe I'm getting confused from https://github.com/projectmesa/mesa-geo/issues/76
Or I simply messed up the dev installation?

Hmm this is interesting. But I do not have this issue on my side.
Could you try to use a clean virtual environment from scratch, and install dependencies and Mesa-Geo via pip install -e ".[dev]"?

Hey, thanks for the quick answer. I'm testing it now on my home pc (ubuntu)
This is what I tried;
git clone https://github.com/projectmesa/mesa-geo.git
cd mesa-geo/
mamba create -n testmg python
mamba activate testmg
pip install -e ".[dev]"
pytest --cov=mesa_geo tests/
[...]
    import mesa
>   from mesa_geo.visualization.ModularVisualization import ModularServer
E   ModuleNotFoundError: No module named 'mesa_geo.visualization.ModularVisualization'

examples/geo_schelling/server.py:2: ModuleNotFoundError

Here the full output of the pytest errors and the mamba package list:
pytest_errors.txt
mamba_list.txt
Thanks for your help!

I see. Looks like Mesa-Geo is not properly installed for some reason.
We do not have a mesa_geo.visualization.ModularVisualization module in our code base; it is actually copied over from Mesa during setup. See: https://github.com/projectmesa/mesa-geo/blob/a3f01ad47a454246c2da05d0d6c7ccc27bcd963b/setup.py#L43
How about pip install -e . or install from PyPI directly, i.e., pip install mesa-geo? I am not very sure why this is happening.

okay so this is interesting: it seems that (at least on my machine) there is a subtle difference in the way setuptools treats command classes between install and install_editable with pip.
I start from a fresh clone (so the ModularVisualization.py file is not there) then:

If I install with pip install . the build command is properly called and the file is copied. Everything work and this is the expected behaviour
This is also the case, if I manually run python setup.py install/develop
Now, if I install with pip install -e . the develop command is not called and I don't get the visualization/ModularVisualization.py. Tests then obviously fail.
This is also the case for pip install -e .[dev]

So the question is, why --editable is not calling neither the build nor the develop subcommands?
Actually pip install . calls the custom build command, as we can see from this log ("INSIDE BUILD") is my debug text
Building wheels for collected packages: Mesa-Geo
  Created temporary directory: /tmp/pip-wheel-uy4v8atq
  Destination directory: /tmp/pip-wheel-uy4v8atq
  Running command Building wheel for Mesa-Geo (pyproject.toml)
  /tmp/pip-build-env-6f4grtxt/overlay/lib/python3.10/site-packages/setuptools/config/setupcfg.py:508: SetuptoolsDeprecationWarning: The license_file parameter is deprecated, use license_files instead.
    warnings.warn(msg, warning_class)
  running bdist_wheel
  running build
  INSIDE BUILD
  Downloading the leaflet.js dependency from the internet...
  Downloading the leaflet.css dependency from the internet...
  running build_py

On the other side, pip install --editable .  calls editable_wheel and not develop
Building wheels for collected packages: Mesa-Geo
  Created temporary directory: /tmp/pip-wheel-0upylsq5
  Destination directory: /tmp/pip-wheel-0upylsq5
  Running command Building editable for Mesa-Geo (pyproject.toml)
  /tmp/pip-build-env-atgt3u8t/overlay/lib/python3.10/site-packages/setuptools/config/setupcfg.py:508: SetuptoolsDeprecationWarning: The license_file parameter is deprecated, use license_files instead.
    warnings.warn(msg, warning_class)
  running editable_wheel
  creating /tmp/pip-wheel-0upylsq5/tmpjw6p7snr/Mesa_Geo.egg-info
  writing /tmp/pip-wheel-0upylsq5/tmpjw6p7snr/Mesa_Geo.egg-info/PKG-INFO
  writing dependency_links to /tmp/pip-wheel-0upylsq5/tmpjw6p7snr/Mesa_Geo.egg-info/dependency_links.txt
  writing requirements to /tmp/pip-wheel-0upylsq5/tmpjw6p7snr/Mesa_Geo.egg-info/requires.txt
  writing top-level names to /tmp/pip-wheel-0upylsq5/tmpjw6p7snr/Mesa_Geo.egg-info/top_level.txt
  writing manifest file '/tmp/pip-wheel-0upylsq5/tmpjw6p7snr/Mesa_Geo.egg-info/SOURCES.txt'
  reading manifest file '/tmp/pip-wheel-0upylsq5/tmpjw6p7snr/Mesa_Geo.egg-info/SOURCES.txt'
  adding license file 'LICENSE'
  writing manifest file '/tmp/pip-wheel-0upylsq5/tmpjw6p7snr/Mesa_Geo.egg-info/SOURCES.txt'
  creating '/tmp/pip-wheel-0upylsq5/tmpjw6p7snr/Mesa_Geo-0.3.0.dist-info'
  creating /tmp/pip-wheel-0upylsq5/tmpjw6p7snr/Mesa_Geo-0.3.0.dist-info/WHEEL
  /tmp/pip-build-env-atgt3u8t/overlay/lib/python3.10/site-packages/setuptools/command/install.py:34: SetuptoolsDeprecationWarning: setup.py install is deprecated. Use build and pip and other standards-based tools.
    warnings.warn(
  running build_py
  running egg_info

Thus no file is copied...

As a proof, if I change the setup.py accordingly:
--- a/setup.py
+++ b/setup.py
@@ -12,6 +12,7 @@ from distutils.command.build import build

 from setuptools import setup
 from setuptools.command.develop import develop
+from setuptools.command.editable_wheel import editable_wheel

 def get_version_from_package() -> str:
@@ -22,10 +23,22 @@ def get_version_from_package() -> str:
     return version

+class EdiWheelCommand(editable_wheel):
+    """Installation for development mode."""
+
+    def run(self):
+        print("INSIDE EDIWHEEL")
+        get_mesa_viz_files()
+        get_frontend_dep()
+        editable_wheel.run(self)

 if __name__ == "__main__":
+
     setup(
         name="Mesa-Geo",
         version=get_version_from_package(),
         cmdclass={
+            "editable_wheel": EdiWheelCommand,
             "develop": DevelopCommand,
:

Building wheels for collected packages: Mesa-Geo
  Created temporary directory: /tmp/pip-wheel-556vurov
  Destination directory: /tmp/pip-wheel-556vurov
  Running command Building editable for Mesa-Geo (pyproject.toml)
  /tmp/pip-build-env-ysf_0t7s/overlay/lib/python3.10/site-packages/setuptools/config/setupcfg.py:508: SetuptoolsDeprecationWarning: The license_file parameter is deprecated, use license_files instead.
    warnings.warn(msg, warning_class)
  running editable_wheel
  INSIDE EDIWHEEL

And the ModularVisualization.py file is present again!

We should replace the get_mesa_viz_files mechanism. This pattern seems to be non-standard in Python package setups.

Yes, if you can confirm this, we could:

Leave it like this (nobody a part me seem to have had the same problem)
Extend the build_py command instead of build and develop as this seems to be always called
Maybe consider using the recent sub_command extension from setuptools to add additional build steps

And additional question: is there any reason why build is imported from distutils and the rest from setuptools?

@mrceresa If get_mesa_viz_files() is not called as expected, how about get_frontend_dep()? Do the Leaflet files (js and css) get downloaded?
If not then we may still need to address this issue for the Leaflet dependency.

I got the same problem when I installed mesa-geo from git. I reinstall this package by using 'pip install mesa-geo' and it works.

@zlfdodo Sorry about that. Could you share which operating system and python version were used?

@wang-boyu Thanks for the prompt reply. I am using Windows system and conda python 3.10.6. The mesa-geo package only works under this python version on my computer. When I use any other python version in Anaconda and run any model/run.py file in the example folder, it will return "ImportError: DLL load failed while importing_version."


Not able to start Watir (6.14) remote with chrome headless

I am either using the wrong syntax, or there something not compatible:
I tried both:
browser = Watir::Browser.new :chrome, {headless: true,timeout: 120, url: selenium_server}
browser = Watir::Browser.new :chrome, :switches => %w[--ignore-certificate-errors --disable-popup-blocking --disable-translate --disable-notifications --start-maximized --disable-gpu --headless]
driver.driver.manage.timeouts.implicit_wait = 100 # seconds
server starts ok but in regular mode.
I have latest version of chromedriver,chrome,zalenium,selenium and watir

Both of your attempts work fine for me with (with 6.14.0) using a local selenium server. Can you try the latest and provide debug output from your server?

I've been trying to verify this issue, but I cannot reproduce it. Both local and on a simple selenium-grid chrome starts in headless mode just fine
Just verified that it works fine on:

Windows 10
Chrome 74
Ruby 2.5.5
selenium-webdriver 3.142.3
watir 6.16.5


Inconsistency of markers for SAXS and XPCS

There's also repeat of markers for g2.
Would be nice to make the markers larger for easier recongnition

Problem fixed, closing the issue


[oracle] Fix empty lock payloads

What does this PR do?
Fixes empty lock payloads.
Motivation
dbm-metrics-processor was sending empty series to metrics intake. Meanwhile, a workaround was implemented on dbm-metrics-processor but the agent shouldn't be sending empty payloads.
Additional Notes
No changelog, since the bug wasn't customer facing.
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Use of app->mail() crashes on php 8.1

This is caused by calling the default newInstance() method. In the version of Swift Mailer used by this framework, newInstance() calls to strtolower(null) which crashes in php 8.1. In a later version of Swift Mailer, they fix this bug with strtolower(null ?? '').

Thank you!
What exactly is “the version of Swift Mailer used by this framework”? This framework can actually be used with multiple versions of any dependency – unless you are referring to major versions (e.g. 1.x.x), which are often limited.
Can you go to the root folder of your project, open the CLI there and run composer show? The output should tell us exactly which version of Swift Mailer you are using.
Can you also point to the bug report or issue for Swift Mailer, or name the first version of Swift Mailer where that strtolower behavior is fixed?

You can inspect how this works by looking at the SmtpTransport.php file in the Swift Mailer project

Thanks!
Will use the workaround you described, at least in the very short term, unless we can include the fix via a simple update of the dependency.
In order to get the version of Swift Mailer, we really need the way I described before. Looking at composer.json is not enough, because ^5.4 is just a constraint, and, say, version 5.9.2 also fulfills that constraint. So you’d need to look either in composer.lock, or, more easily, just run composer show.

Ah, I understand now (sorry). Looks like composer finds version 5.4.12.


match_re is not working while configuring reciever

Hi Team ,
I configured to use alert name to send alerts to slack,but it's not working :
 - receiver: 'fdi-slack'
    group_by: '["alertname","oci_region","dbname","prd_pod","instance","resource_name","resource_type"]'
    match_re:
      alertname: "fusion:instance:status.*"

I can see alert is getting fired in prometheus , I can see slack URL is also correct.
I am not sure if I can use alertname for match or not.
Please help.

As a general rule, you'll be more likely to find help on either the prometheus-users mailing list or the IRC/matrix chat.
From the snippet is seems like you might have a yaml formatting issue in your config. Your match_re looks reasonable, though match_re is deprecated, consider moving to the matcher field.


Ticking player & ticking Entities crashes

https://pastebin.com/E0gAdgcq
https://pastebin.com/WbDwMtqX
I was playing survival and i went through lava in the nether with my jet pack on and crashed a few seconds later. trying to replicate the issue in creative world making the nether portal my game crashed as well.

update to 3.1.1


test: add a program that crashes the runtime

Add a cairo program that compiles but throws a runtime error to verify that we correctly catch them.

Hey @greged93, I would like to work on it


Allow Selection of Display Timezone

Right now DBeaver seems to convert TIMESTAMPTZ columns to the local timezone of the machine on which it is being run and there is not option to override this timezone in the client itself. I've seen this stackoverflow:
https://stackoverflow.com/questions/45323552/how-to-change-dbeaver-timezone-how-to-stop-dbeaver-from-converting-date-and-ti
But adding command line options is not a very user friendly way to adjust this. In addition, without running it from the command line I'm unsure how you would set this in MacOS.
Can we get an option added to preferences that allows you to set an offset that TIMESTAMPTZ columns are converted to please?

Agreed. Also check #238

Is this a "UI setting" for not having to add -Duser.timezone=UTC to the ini file after each upgrade?

Already implemented.
Timezone can be set in Preferences

@uslss how do I add that? 21.1.3


Not working in 4.5.3

Operating system
Windows
qBittorrent version
v4.5.3
Python version
Python 3.8.10
Is magnet links support enabled?

[ ] Magnet links support enabled

Output
Traceback (most recent call last):
  File "engines/rutracker.py", line 77, in <module>
    class RuTrackerBase(object):
  File "engines/rutracker.py", line 214, in RuTrackerBase
    def _open_url(self, url: str, post_params: dict[str, str]=None, log_errors: bool=True) -> bytes:
TypeError: 'type' object is not subscriptable

Additional information
QBittorrent will not even load the plugin. There was no output from the first command.

Seems Qbittorrent installed an old version of Python.  Version 3.11 works fine.


When using Gorouter for navigation, the state of Riverpod does not get updated.

Describe the bug
my provider code
@riverpod
class SelectTask extends _$SelectTask {
  @override
  Task? build() {
    return null;
  }

  setTask(Task task) {
    state = task.copyWith();
  }
}

my set action
                onPressed: () {
                  ref.read(selectTaskProvider.notifier).setTask(Task(
                      duration: duration.value, detail: detailController.text));
                  logger.d(
                      "set task successfully ${duration.value}, ${detailController.text}");
                  context.go("/startpage");
                },

my read page code
class StartScreen extends HookConsumerWidget {
  const StartScreen({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context, WidgetRef ref) {
    final task = ref.watch(selectTaskProvider);
    logger.d(task);
   ....

when i update selectTaskProvider,
the log is set successful
I/flutter (32476): │ 🐛 {
I/flutter (32476): │ 🐛   provider: selectTaskProvider,
I/flutter (32476): │ 🐛   oldValue: null,
I/flutter (32476): │ 🐛   newValue: Task(duration: 0:10:00.000000, detail: ss)
I/flutter (32476): │ 🐛 }
I/flutter (32476): │ 🐛
but in startpage, i also read a null value from selectTaskProvider.  Is there a problem?

I assume your provider got disposed because it isn't listened anywhere

I don't quite understand. How can I  watch the latest state?

i must add code    "ref.watch(selectTaskProvider)"  on NewTaskScreen.   it can updated on StartScreen.
It's a bit confusing to me. 🤔

I have the same issue. When navigating from Screen A to Screen B and there is a class-type notifier that I want to use in Screen B, the problem is how to initialize that notifier.
The first solution is to make SelectedTask with @Riverpod(keepAlive: true). However, SelectedTask is not a state that is used globally across the app, but only within a single screen. Furthermore, I expect that this SelectedTask will often be modified within that screen. In my experience, adding keepAlive in such cases tends to increase the possibility of bugs.
Another solution I have considered is creating an additional class-type notifier for initial value.
@Riverpod(keepAlive: true)
class SelectedTaskInitialValue extends _$SelectedTaskInitialValue {
  Task build() => ...
  void set(Task task) { state = task }
}

@riverpod
class SelectedTask extends _$SelectedTask {
  Task? build() => ref.watch(selectedTaskInitialValueProvider)
}

A more simplified version would be to create a late variable for the initial value like this:
late Task selectedTaskInitialValue;

@riverpod
class SelectedTask extends _$SelectedTask {
  Task? build() => selectedTaskInitialValue
}

I have chosen the last solution.
Discussions, opinions, and critiques are welcome.

extension AutoDisposeRefExtension<T> on AutoDisposeRef<T> {
  /// Keep listening to an autoDispose provider until last listener is removed.
  ///
  /// This is primarily used to initialize and preserve the state of the provider
  /// when navigating to a route until that route is popped off.
  void keepAliveUntilNoListeners() {
    keepAlive();
    onCancel(invalidateSelf);
  }
}

@riverpod
class SelectTask extends _$SelectTask {
  @override
  Task? build() {
    ref.keepAliveUntilNoListeners();
    return null;
  }

  setTask(Task task) {
    state = task.copyWith();
  }
}

extension AutoDisposeRefExtension<T> on AutoDisposeRef<T> {
  /// Keep listening to an autoDispose provider until last listener is removed.
  ///
  /// This is primarily used to initialize and preserve the state of the provider
  /// when navigating to a route until that route is popped off.
  void keepAliveUntilNoListeners() {
    keepAlive();
    onCancel(invalidateSelf);
  }
}

@riverpod
class SelectTask extends _$SelectTask {
  @override
  Task? build() {
    ref.keepAliveUntilNoListeners();
    return null;
  }

  setTask(Task task) {
    state = task.copyWith();
  }
}

This will mark your provider as a keep-Alive but will also auto-dispose it when you leave Screen-B

It is cool

This is pretty useful, thanks, it will let me get rid of my very own dirty workaround.
However, I still consider this a workaround for the very common use case, I don't get why this is so hard and not part of Riverpod proper.

A solution was given. This is not really a bug, so closing :)


useLocalStorage - globally reactivity

Hello! Is it possible to achieve reactivity of localStorage across different components when the data is updated in one of them?
Something like in this example: https://stackblitz.com/edit/react-starter-typescript-3cjkkm?file=App.tsx

I think react-use is no longer actively being developed.
A good alternative is @react-hookz/web. They cover this exact use case in useLocalStorageValue.


Update engine_v2 recipes to publish completed results to pub/sub

https://flutter-review.googlesource.com/c/recipes/+/43360 is ready, just waiting for the code freeze to be finished.


Consider adding ^ in package.json to allow minor version bumps in dependencies

This ask came from #315.
The original intent of version lock was to prevent any chance of unexpected dependency issues. But if it deals more pain by default, a case (the above referenced issue) can be made to relax the restriction.
The alternative is to have mono-repos and locking our dependency versions between projects. But this approach will not help with externally managed projects like ion-tools in which case we will need to fork.

Some examples of why we shouldn't be locking to exact versions in our SDK consumed by other users:
2 versions of @nobile/ed25519, 3 versions of @noble/secp256k1,

2 versions of secp256k1 (and isnt this already a duplicate of the above secp256k1 package?)

a whopping 3 versions of readable stream, which was already 220kb. this is 1.3mb worth of readable-stream

3 versions of multiformats worth a whopping 3.6 megabytes:

and this is the best case scenario since i ran this in a prototype. in an end user's package they'll have even more duplicates since they'll be using other packages from the community

locking to exact versions makes sense for a product but does not make sense for a library which depends on other libraries which recursively depend on other libraries

picking this up

@Petsamuel Assigned 🥳 good luck!

We've decided that version ranges aren't worth it for us right now. The potential benefits of slimming package size don't outweigh the risk of introducing breaking changes in upstream deps or causing inconsistent issues that result from local package versions being different depending on when someone runs npm install. In the future, when we revisit package size optimization, we'll revisit this issue.


Can you provide a build generated with transform-runtime?

My situation is this:

I'm running my tests with karma + babel + babel-runtime
I'm not using polyfills because I want to be closer to my live environment
I'm not using transform-runtime for the same reason

It'd be great if there was a version of flow-runtime built/distributed with transform-runtime, which loads in things like regenerator and WeakMap specifically for the flow-runtime module only. Then I can continue to exclude polyfills from my tests, and know that my tests will fail if I try to use WeakMap or something not available in older browsers I have to support.
Thoughts on this?

We're already producing 3 builds (umd, es2015 and commonjs), I don't really want to include another - if you're using webpack you could always produce another target which only includes flow-runtime and the various transforms / polyfills it needs, so your normal code won't be affected

Hi @phpnode. Totally understand you're reasoning, there are a lot of builds already.
One thing I need to figure out is, right now I'm running flow-runtime as a babel plugin, and I'm not sure how I'd apply polyfills or transform-runtime to just this module and ignore the rest of my code. Do you have any tips there?
Second quick question -- is there a reason the existing 3 builds couldn't be passed through transform-runtime, without creating an additional build? That way the builds would be es5-ready out of the box.
Appreciate your time, and really love flow-runtime!

I would do something like this:
cat node_modules/flow-runtime/dist/flow-runtime.js | babel --no-babelrc --presets=env,stage-0,react --plugins=transform-decorators-legacy,transform-runtime > rt.js

I get your point about wanting an ES5 build, the issue is that with all of the dependencies the lib becomes huge, and if they're included directly within flow-runtime then any other code which uses symbols or weakmaps or any other feature will need to duplicate the polyfill code - for this reason I think we have to ship modern code and let the consumer deal with this as appropriate for their environment.

Hey again,
Totally get your reasoning. I'd say the duplication is less of an issue since I'm imagining (?) most people are using this in their testing env rather than for distributed code.
Pre-transcompiling flow-runtime.js makes perfect sense -- but then how do I get my pre-compiled version to be picked up by the flow-runtime babel plugin?

Pre-transcompiling
😄

how do I get my pre-compiled version to be picked up by the flow-runtime babel plugin?

Yeah, that's true, now you'd need to do some rewiring. babel-plugin-flow-runtime should accept a libName configuration option to point to the right file or module (we'd need this even if we do end up shipping a batteries included build too)

Sounds good to me!

I caved and now regenerator-runtime comes bundled as of 0.11.1, thanks to @ForbesLindesay


env1_lab.py: Incorrect lane detection when computing reward

Bug Description
The lane detection condition in function compute_collision_reward is incorrect, which leads to negative rewards for lateral collisions even if no lane changing is done.

Unprocessed direction action was sent to to compute_reward method. Should be correct now.


Temperature rate of change

This is the proposed renaming of TemperatureChange (introduced by @ksheffer-opti) to TemperatureRateOfChange as discussed in issue #42. It is indeed breaking, but note that conversion rates and symbols had been fixed already in 5696b64dcc18854b2fcb45ad412e83b276e2493b.
The proposed change also brings two tests on this quantity: operators and conversions.
I haven't touched the README.md nor CHANGELOG.md - I don't know at which state this should be done.

I would like the unreleased section in the change log to be updated every time a feature is added or a bug is fixed. Note that we have not set up proper semver and unstable nuget builds for this project yet. I'll add an issue!


README and IMPLEMENTATION_COVERAGE mismatch for run_instances

In the README.md, section another-example, there's an exemple of moto mocking and testing the usage of boto3.client('ec2', region_name='us-west-1').run_instances(...), yet in master's IMPLEMENTATION_COVERAGE.md that method is not marked as implemented.
But it seems to be the case here, and should be marked as such.
I created a pull request here

pull request https://github.com/spulec/moto/pull/3342 merged and closed, closing issue.


8/31/2021 Publish

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

File
Status
Preview URL
Details

docs-ref-services/latest/resourcemanager-iothub-readme.md
:warning:Warning
View (azure-java-stable)
Details

docs-ref-services/latest/security-keyvault-jca-readme.md
:warning:Warning
View (azure-java-stable)
Details

docs-ref-services/preview/resourcemanager-apimanagement-readme.md
:warning:Warning
View (azure-java-preview)
Details

docs-ref-services/preview/resourcemanager-postgresqlflexibleserver-readme.md
:warning:Warning
View (azure-java-preview)
Details

docs-ref-services/preview/resourcemanager-servicefabric-readme.md
:warning:Warning
View (azure-java-preview)
Details

docs-ref-services/preview/sdk-template-readme.md
:warning:Warning
View (azure-java-preview)
Details

docs-ref-mapping/reference-latest.yml
:white_check_mark:Succeeded

docs-ref-mapping/reference-unified.yml
:white_check_mark:Succeeded

docs-ref-services/latest/resourcemanager-iothub-readme.md

Line 2, Column 1: [Warning-ms-prod-and-service] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 11, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'iothub'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 5, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'magpint' is not a valid Microsoft alias.
Line 39, Column 3: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/java/azure/jdk/' will be broken in isolated environments. Replace with a relative link.

docs-ref-services/latest/security-keyvault-jca-readme.md

Line 2, Column 1: [Warning-ms-prod-and-service] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 11, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'keyvault'.
Line 132, Column 35: [Warning-file-not-found] Invalid file link: 'resources/SignToPrincipal.png'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 5, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'magpint' is not a valid Microsoft alias.
Line 24, Column 49: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/key-vault/' will be broken in isolated environments. Replace with a relative link.
Line 41, Column 5: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/java/azure/jdk/?view=azure-java-stable' will be broken in isolated environments. Replace with a relative link.
Line 41, Column 5: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: https://docs.microsoft.com/java/azure/jdk/?view=azure-java-stable. Otherwise, remove the view parameter.
Line 43, Column 15: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/key-vault/keys/quick-create-portal' will be broken in isolated environments. Replace with a relative link.
Line 128, Column 32: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/key-vault/certificates/about-certificates#exportable-or-non-exportable-key' will be broken in isolated environments. Replace with a relative link.

docs-ref-services/preview/resourcemanager-apimanagement-readme.md

Line 2, Column 1: [Warning-ms-prod-and-service] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 11, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'apimanagement'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 5, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'magpint' is not a valid Microsoft alias.
Line 39, Column 3: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/java/azure/jdk/' will be broken in isolated environments. Replace with a relative link.

docs-ref-services/preview/resourcemanager-postgresqlflexibleserver-readme.md

Line 2, Column 1: [Warning-ms-prod-and-service] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 2, Column 8: [Warning-duplicate-titles] Title 'Azure Resource Manager PostgreSql client library for Java' is duplicated in: 'docs-ref-services/preview/resourcemanager-postgresqlflexibleserver-readme.md(2,8), docs-ref-services/preview/resourcemanager-postgresql-readme.md(2,8)'. Titles must be unique within a docset.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 5, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'magpint' is not a valid Microsoft alias.
Line 39, Column 3: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/java/azure/jdk/' will be broken in isolated environments. Replace with a relative link.

docs-ref-services/preview/resourcemanager-servicefabric-readme.md

Line 2, Column 1: [Warning-ms-prod-and-service] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 11, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'servicefabric'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 5, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'magpint' is not a valid Microsoft alias.
Line 39, Column 3: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/java/azure/jdk/' will be broken in isolated environments. Replace with a relative link.

docs-ref-services/preview/sdk-template-readme.md

Line 2, Column 1: [Warning-ms-prod-and-service] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 11, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'templatetemplate'.
Line 74, Column 78: [Warning-bookmark-not-found] Cannot find bookmark '#introduction' in 'docs-ref-services/preview/sdk-template-readme.md'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 5, Column 12: [Suggestion-ms-author-invalid] Invalid value for 'ms.author', 'magpint' is not a valid Microsoft alias.
Line 23, Column 94: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/style-guide/capitalization' will be broken in isolated environments. Replace with a relative link.
Line 31, Column 22: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/' will be broken in isolated environments. Replace with a relative link.
Line 54, Column 94: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/cosmos-db/account-overview' will be broken in isolated environments. Replace with a relative link.
Line 54, Column 190: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/java/azure/jdk/?view=azure-java-stable' will be broken in isolated environments. Replace with a relative link.
Line 54, Column 190: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: https://docs.microsoft.com/java/azure/jdk/?view=azure-java-stable. Otherwise, remove the view parameter.
Line 148, Column 18: [Suggestion-disallowed-html-tag] HTML tag 'enlistmentroot' isn't allowed. Disallowed HTML poses a security risk and must be replaced with approved Docs Markdown syntax.

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


CAMEL-9477 - add simple lines to indicate removal of CamelSpringJUnit…

…4ClassRunnerPlainTest for deprecation and update the link with the right test case & also fix some document links which are .html to .adoc
As mentioned in the issue, I agree that there seems to be much work required to do on the documentation to keep the documents, website and code in sync.
This PR only covers what's complained in the issue.

Thanks for the PR it has been merged. Do you mind closing this?

thanks


Force batch nested processing bug fixed

Typo fixed --> in innerCalls in endExclusiveToSkipInternalEvents we need to filter by ItemEvents, not by CompletionEvents. (inner events are processed with ItemEvents, current call find bounds with CompletionEvents)

Before:
error.log
After(with additional prints):
after.log


Add support to markdown task lists

Following github's markdown parser, add support to use task lists items.
Basically it converts a [ ] or [x] inside a list item to a read-only
checkbox.
Eg.
- [x] Foo
- [ ] Bar

Renders something like:

<input type="checkbox" disabled checked> Foo

<input type="checkbox" disabled> Bar

This is only available on unordered lists
I would appreciate some code-review / feedback on this, I know there is a lot to improve, still it is a proposal and a work in progress.

I actually have to -1 this. I don't think it makes sense to copy GFMD for the general purpose markdown renderer shipped in a language standard library. It's also not part of the CommonMark spec, which is something I'd like to see implemented by our markdown renderer personally.

@jhass good point! Let's close it for now. I'll try to put my effort on making the renderer more close to the CommonMark spec then!
Thanks!

Having a CommonMark compliant implementation of Markdown would be awesome!

I'm biased since I use - [ ] aaaall the time in markdown: I don't think anyone will use that sequence of character anytime unless actually meaning "checkbox", so personally I'd like to see it supported. /2c.


Shell integration (testing waters for a PR)?

I've been curious enough about shell integration for fzy that I've started studying the integration fzf has, to see what's possible for making something comparable. What I want to know is:

Would you be open to a PR contributing such an entrancement, pending further discussion on actual features associated?
We're would this go in the repo? The contrib folder?

Might be worth stating what exactly you would like to bring over from fzf if you want to know what might be accepted in a PR. contrib seems like a good spot it though 👍

@casr: The most notable features would be:

Binding of shortcuts to certain invocations of fzy (i.e., CTRL+T for something like find | fzy) whose output then get put into the current command being written.
Completions for fzy options in bash and zsh

I'm actually not sure what's already present, which is also part of the reason I wanted to open a dialogue here.

Just wanted to ping this issue, since I haven't gotten an answer yet!


Stopping Strings

stop_markers:
- '### End of Transcript ###<|endoftext|>'
- <|endoftext|>
-
stopping_strings: [</s>]

OUTPUT containts the stopping marker/string
Neuromorphic chips are a type of artificial intelligence chip designed to mimic the functioning of biological neural networks found in the human brain. They are typically used for tasks such as image recognition, natural language processing, and decision-making. To simulate these chips on a GPU (Graphics Processing Unit), you can use deep learning frameworks like TensorFlow or PyTorch. These frameworks provide tools for building and training neural networks, which can then be deployed onto the GPU for faster computation.</s>

Thanks for the info!  Currently the stopping_string will only work when it's on a line by itself.  In this example it looks like it's at the end of a line.
I'll work on making it work more generally.

This should be fixed in https://github.com/chrisrude/oobabot/commit/f5021703b9f892b2dece98c3572b675a2791bf54
When we get a stop token in the middle of a line, return all text up until the token, then stop responding.
Also, we now automatically add anything in "stopping_strings" to the list of stop tokens we're looking for.
@sidonsoft if you feel like / want to test this from main, go ahead, if not it should show up in the 0.1.8 release.


ConfigUpdateRequest should include only updated CType

Category: Bug fix
Github Issue # or road-map entry, if available: #879
Description of changes: include only modified config CType into ConfigUpdateRequest
Why these changes are required? it is not necessary to reload configs not modified as part of PATCH request
What is the old behavior before changes and new behavior after changes? all config CTypes are reloaded (even ones that were not modified/updated)
Testing done: UT
TO-DOs, if any:
Is it backport from master branch? (If yes, please add backport PR # and commits #)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

How are dependent CTypes handled in this implementation?

How are dependent CTypes handled in this implementation?

How are dependent CTypes handled in this implementation?

The dependencies are handled by DynamicConfigFactory.onChange() that loads all CType configurations from ConfigurationRepository cache. ConfigurationRepository cache is updated in reloadConfiguration() call.

How are dependent CTypes handled in this implementation?

The dependencies are handled by DynamicConfigFactory.onChange() that loads all CType configurations from ConfigurationRepository cache. ConfigurationRepository cache is updated in reloadConfiguration() call.


Fixed htmlValidator for synchronous usage, and expanded tests to chec…

…k both synchronous and asynchronous usage.

perfect :+1:


Fix ONNX export for MobileViT for segmentation

What does this PR do?

Fixes #1127. In the TasksManager, image-segmentation should also be associated to AutoModelForSemanticSegmentation (see https://github.com/huggingface/transformers/blob/2c977e4a9031e3961ef185b40ff4b7f54eda363c/src/transformers/pipelines/init.py#L345).
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Can confirm the models exported correctly! 👍

https://huggingface.co/Xenova/deeplabv3-mobilevit-small
https://huggingface.co/Xenova/deeplabv3-mobilevit-x-small
https://huggingface.co/Xenova/deeplabv3-mobilevit-xx-small

To me it feels more like the models on the Hub are wrongly labeled as image-segmentation.

To me it feels more like the models on the Hub are wrongly labeled as image-segmentation.

Well, semantic segmentation is one type of image segmentation so it kind of make sense, even though I don't know why there are no fine-grained labels for segmentation. But the fact is that there is only one segmentation task on the hub when searching models and it is image segmentation: https://huggingface.co/models?pipeline_tag=image-segmentation&sort=downloads
That means that all semantic segmentation models are tagged that way. Changing this would entail changes in Transformers and maybe other libs, so that's not really an option.


[Request] Snipaste

Request:
Snipaste
Description:
Snipaste offers powerful yet easy-to-use snipping, pasting and image annotating functionalities. It is highly customizable to fit all your needs.
The portable (non Windows Store) version is free, and has all the useful features available.

https://scoop-docs.now.sh/apps/?query=snipaste
https://github.com/lukesampson/scoop-extras/blob/master/bucket/snipaste.json

https://scoop-docs.now.sh/apps/?query=snipaste
https://github.com/lukesampson/scoop-extras/blob/master/bucket/snipaste.json

Thanks! But the scoop version is outdated. How can I fix it?


Rejected shares even after setting the coin target time correctly.

I am running Feather coin MPOS pool, I have set the coin target time as 2.5min that is 150 seconds, even after this, I found Rejected shares after +-5minutes.

How you solved this problem?

This is not an MPOS issue, please refer to the appropriate channels (git or chat or forums) on how to setup your Stratum properly. This is a issue tracker for this software, not all dependencies or a community exchange.


Add stacked events

What feature do you want to get added on the base engine?
Idea
I recommend you make it so multiple events can be stacked onto one event note.
Why?
Without this, you can't sync multiple events on the same step, which is really annoying when you (for example) want to change gf's speed and swap one of the characters at the same time.

it is possible to stack events, with the strum time for notes, but its pretty much all we can do until we get more space for events

(also strum time applies with selected event notes too)

or just zoom in (press x, can go up to x24) and the notes will be so squished together that they will be basically one note

or just zoom in (press x, can go up to x24) and the notes will be so squished together that they will be basically one note

zooming in only zooms in for the chart editor and doesnt change the actual chart

or just zoom in (press x, can go up to x24) and the notes will be so squished together that they will be basically one note

zooming in only zooms in for the chart editor and doesnt change the actual chart

thats just wrong you can test it yourself in the chart editor

0.5 is out.


gradle build file for project.

No lightadmin specific repository(i.e, http://lightadmin.org/nexus/content/repositories/... ) is added into this build file, because when writing and testing the build file, the repository was not working(50x error)
spring-boot's version is upgraded to 1.2.5-RELEASE.
lightadmin is using the 1.2.0.RC1, which is available on maven central repository at the moment.
Build env is as following:

lightadmin-springboot jerry$ gradle -version

------------------------------------------------------------
Gradle 2.5
------------------------------------------------------------

Build time:   2015-07-08 07:38:37 UTC
Build number: none
Revision:     093765bccd3ee722ed5310583e5ed140688a8c2b

Groovy:       2.3.10
Ant:          Apache Ant(TM) version 1.9.3 compiled on December 23 2013
JVM:          1.8.0_51 (Oracle Corporation 25.51-b03)
OS:           Mac OS X 10.10.4 x86_64

Thanks for your help, @jerryrt )


[Bug]:

What happened?
tabs are too high compared to everything else, and is glitched. I've redownloaded and deleted it but the same issue arrises.

Steps to reproduce

Install theme
Install custom theme
restart firefox
Tabs are glitched

Additional info (Optional)
No response

Edit: Issue has been fixed by adding --tabstrip-min-height: 0 !important to the custom_example.css, and then renaming it to custom.css


Describe the bug ✍
Currently, the README.md file of the Arcade_Game project includes a link instead of an embedded screenshot, which could be improved for better user experience and presentation. It would be beneficial to replace this link with an actual screenshot of the game.
Screenshot of the bug:

@kunjgit please assign me this issue for better user experience.
Provide step by step information reproduce the bug 📄

Steps to Reproduce the Bug
Clone Repository
Navigate to Repository Directory
Open README.md
Identifying the Issue
Locate Screenshot Link
Steps to Fix the Issue
Capture Game Screenshot
Add Screenshot to Repository
Update README.md
Finalizing Changes
Save and Commit

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Bindusree1515 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @Bindusree1515 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


feat(secretsmanager-secret): properly support deleting secret replicas

This allows replica secrets to be removed. Depending on how you look at it, it's either a fix or a feature. Previously this would just fail flat with no recourse.

:tada: This PR is included in version 3.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Adding electron ARM and ARM64 tasks for cross-compiling

This follows on from pull request #52119.
Adding these tasks will allow for explicitly downloading ARM and ARM64 versions of Electron when it is not the native architecture of the build machine (this is helpful for cross-compilation scenarios).

Thanks! :beers:

Thanks for the fast turnaround 👍


backward hooks "i" and "o" using tuples instead of tensors

class BackwardTupleCallback(HookCallback):
    def __init__(self,**kwargs):
        super().__init__(is_forward=False, **kwargs)

    def begin_fit(self):
        super().begin_fit()
        self.stats = L()

    def hook(self, m, i, o): return {o: type(o), i:type(i)}

    def after_step(self):
        if self.training:
            self.stats.append(self.hooks.stored)
            raise CancelFitException

Anticipated behavior is that backward hooks use tensors like forward hooks do

That's just the format that PyTorch uses. The PyTorch docs mention that backward hooks can be tuples. I don't think we should change the PyTorch format.


Gitlens annotation  in middle of line

I know this is basically an old issue, but I've been noticing weird cases more recently. Here it gets totally stuck
https://user-images.githubusercontent.com/323878/110731026-66a4e880-81d6-11eb-8b00-a1275695e35f.mp4
This is shortly after copy/pasting this block, then making a few small edits

me too


Fraud DB for reimbursement

Add new data to Fraud DB for reimbursement reports

[ ] Fraud db and mapping description specs
[ ] Data Mart Developing
[ ] Data Mart test

[ ] medication_requests - without verification_code


runserver => api

The readme needs an update:
go run cmd/bun/main.go -env=dev runserver shoud be go run cmd/bun/main.go -env=dev api

@vmihailenco ping

Fixed - thanks :+1:


Tagged Functions

Main page shows for tagged elements:
edn.unknown('my/foo', [3, 4])

Think this should be
edn.generic('my/foo', [3, 4])

Hey @motosota, thanks for the feedback. But this project is now abandoned.


[2011-05-30] is_cli returns false when cron jobs run with cgi-fcgi

created by: burnbright
created at: 2011-05-30
original ticket: http://open.silverstripe.org/ticket/6649

The is_cli function on Director.php only functions correctly when php_sapi_name() returns 'cli', however some setups run cron jobs with 'cgi-fcgi', causing the function to return false.
The execution will therefore act as a web-based rather than command-line, resulting in html output of the secuirty login page being returned to the cron output.

Don't think we should change this.

Hi All . Please go through my blog post related the same issue and i hope , you guys will get the fix.
http://lakinmohapatra.blogspot.in/2016/08/solve-if-php-file-being-executed-by.html

Could we declare a constant in cli-script.php that would ensure any subsequent checks for "is_cli" to true? Maybe just after the PHP_SAPI check?

in cli-script.php
if(PHP_SAPI != "cli" && PHP_SAPI  != "cgi" && PHP_SAPI != "cgi-fcgi") {
	echo "cli-script.php can't be run from a web request, you have to run it on the command-line.";
	die();
}
define('SS_ENVIRONMENT_CLI', true);

In Director.php
public static function is_cli() {
	return defined('SS_ENVIRONMENT_CLI') || php_sapi_name() == "cli";
}

:S If our CLI script has this check, why isn't the same one used by Director? We could add the constant, but those are pseudo "environment vars" and being on the CLI is not a property of the environment... so it feels nasty.

@tractorcow  , For resolving this issue , we dont need to  change core files because basically cron jobs should be run by cli .
To fix this, we need to find the exact path to the correct php binary by typing this:
"which php"
This should give you a path like /usr/bin/php. You can go one  step further and check if this is actually a "symbolic link" pointing to a different filename or not:
ls -l $(which php)
Output will be like /usr/bin/php -> /usr/bin/php5-cli / may be some other binary
Then add this full path to the PHP executable and use that in your crontab entry, so it looks something like this:

*/usr/bin/php5-cli   /home/path/to/artisan schedule:run 1>> /dev/null 2>&1

I hope , it will solve your issue without hacking core files.

changing core files is not "hacking" if we're fixing a bug... we are maintaining the core files here.
I don't have a problem with cron module identifying itself as running not on the CLI - what OS / PHP versions do you have installed?

@dhensby , it will be good if you will write common function for identifying cli jobs.
But i was telling from user prospective.
Possible solutions :
if (array_key_exists('REQUEST_METHOD', $_SERVER)) die();
if (php_sapi_name() === 'cli' OR defined('STDIN')) return true;
But make sure , your cli defending system is covering all scenarios.

@lakinmohapatra ah, ok. @tractorcow was suggesting a fix that could be used in core, rather than suggesting you amend core files in your project (which of course would be bad).

@dhensby , Thanks for understanding my points.
Please review again his codes and see if it's working for all sapi names  being mentioned in php.net

One more point  -
There are lots of advantages of using php-cli instead of other sapi names.
It has unlimited execution time .
It does not write headers to output.
The CLI SAPI  does not change the current directory to the directory of the executed script!
When using the CGI  version, the output is:
$ pwd
/tmp
$ php -q another_directory/test.php
/tmp/another_directory
This clearly shows that PHP changes its current directory to the one of the executed script.
Using the CLI SAPI  yields:
$ pwd
/tmp
$ php -f another_directory/test.php
/tmp
This allows greater flexibility when writing shell tools in PHP.
So i think , it has lots of advantages over other sapi names.
So better to stick with cli check only.
@dhensby  please let me know your feedback on it.
http://www.softpanorama.org/Scripting/Phprama/command_line_php.shtml

:S If our CLI script has this check, why isn't the same one used by Director? We could add the constant, but those are pseudo "environment vars" and being on the CLI is not a property of the environment... so it feels nasty.

Because it doesn't check if the environment IS cli, it checks if the environment is definitely NOT cli.
cgi-fcgi is still ambiguous.

Yeah this is a wonfix, I think.

We could potentially introduce the if(defined('SS_ENVIRONMENT_CLI')) check but leave it up to project developers to create a script where that is defined. They can ensure that the script that defines it is outside of the webroot or similar.
cli-script.php will end up getting exposed to some webservers by a hapless installer and I don't want to see a highway-like security bypass created in those situations.

It sounds like we can do better at detecting cli, but @sminnee raises a good point about just assuming cli script is run by cli

Agreed, #wontfix :)

I've run into this problem myself and the resolution is to run the cron using php-cli instead of php binary...
Seems like a fairly simple fix and perhaps something sake should be looking for?


Encoding error during log generation (character maps to )

Hi!
I'm getting an encoding error during log generation (probably because of cyrillic characters).
I tried setting the sysdefaultencoding to UTF-8 but it does not work because of python 3+.

Python: 3.7.4
robotmetrics: 3.1.4 (pulled from github)
OS: Win10

Could you please share your output.xml to my mail id (go through git profile for mail id) to debug issue

I've made a dummy test case that produces this error. It seems like it fails on keywords containing UTF8 characters. I sent the output.xml to your email.


[3.x] CanvasItemEditor: Fix snapping grid misalignment

Cherry-picks changes from #70428 to Godot 3.X.
Grid alignment is horrible broken in Godot 3.X and needs to be fixed asap, especially for pixelart games in which grid alignment is crucial.

Could this not be cherry-picked automatically? We normally cherry pick to older branches in batches when appropriate

Cherry picking as in git cherry-pick
But see the comment on the original PR for this https://github.com/godotengine/godot/pull/70428#issuecomment-1911197964
You also have to credit the original author, see adding co-authors

I had a little look at this yesterday with the view of moving this along, i.e. how to address the regression.
I'm not super familiar with the area so had to read some of the previous topics.
imo this PR should have a lot more information describing the original problem, why the PR doing the regression was introduced, and then more consideration in the PR "fixing" the regression (which this attempts to cherry pick).
It is understandable to some extent given that this is a cherry pick not written by the original author @phnix-dev . But I would like some more input from the original author / other 2d contributors and possibly @timothyqiu who cherry picked as he may be more familiar.
If I am correct in assessing the situation:

#65101 fixed a minor bug in slider ranges
It was unforeseen that this could create regressions in existing sliders (especially in the editor)
An obvious regression was seen in #70186 which is "fixed" (or possibly patched over?) by this PR

I can see that this has been the pattern in master, but we are generally a lot more conservative in 3.x as backward compatibility is paramount.
My concerns are that #65101 could potentially have created more than just 1 regression, in not just the editor, but potentially also in user projects, and we could end up in a "whack-a-mole" situation trying to fix these. This is feasible in 4.x, which has lots of testing and rapid change, but not so feasible in 3.x, which is in LTS and people are expecting existing code to work "as is".
So rather than just merging this fix, we have a potential choice between this, and simply reverting the original PR #65101 and living with the pre-existing bug (which I'm assuming has existed for years without a report, so is not major). These regressions do appear to be more major.
With a revert, we then have the opportunity for a new PR re-applying, but also fully considering all possible regressions, which ideally should have been the situation before merging the first PR.
Anyway, I'm not an expert here, just speaking from the responsibility of trying to keep the branch as stable as possible, and it may be that this can be guaranteed to be the only regression, but I think we should hear that from the author @phnix-dev .

I think the intention is to only allow input to be min + step * X.
The original 0.01 min only allows values like 0.01, 1.01, 2.01, 3.01, etc after #65101. That's the cause of the issue/regression.
There is always an ambiguity when the min in the property hint is not divisible by step. For example, should the property hint 2,12,4 allow the use of 4, 8, 12 or 2, 6, 10? Both seem to make sense. The current behavior is more functional and can express both sets of values, while the previous behavior could not express the latter.

But in turn this means that fractional minimums are no longer viable in a lot of cases.
Say you wanted to prevent divide by zero, but have otherwise a grid every 4th, previously you could use:
min 0.01, step 4
Now as I understand it, that is no longer possible?
So you are getting some flexibility in exchange for losing it elsewhere?
The other major concern is that this is not backward compatible, not just in the engine, but in third party code.

But in turn this means that fractional minimums are no longer viable in a lot of cases. Say you wanted to prevent divide by zero, but have otherwise a grid every 4th, previously you could use:
min 0.01, step 4
Now as I understand it, that is no longer possible?

For property hints, using 0.01 as a minimum in order to prevent the property being zero is a hack. It's not a feature of SpinBox / Range.
The number is arbitrarily chosen and only works in the inspector. To avoid zero seriously, it should be checked in the setter, then the inspector is zero-free automatically.

Update: I guess reverting that PR might be easier than updating related hints indeed.

Yes, I do worry that this could be a rat's nest and difficult to find all cases, and then we have third party addons / modules etc to consider.
And the "primary directive" for 3.x is as an LTS version - we should try and introduce new features / enhancements in a non-breaking way wherever possible.
I have no objections to adding it in a backward compatible way, but maybe that would be more appropriate as a new PR starting from the old working state.

Hello 👋, I am the one who made this PR (https://github.com/godotengine/godot/pull/65101)
I sincerely apologize as I do not have the time to help fix these bugs. As @lawnjelly rightly said, Godot 3 is an LTS version and it is better if fixes do not introduce regressions. I think keeping this fix in Godot 4 and reverting it to Godot 3 might be a good solution.


Refactor diagnostics framework?

Instead of defining both a Diagnostic struct and a run_diagnostic(::Model, ::Diagnostic) function, maybe there's a way of being able to just define a run_diagnostic function.
Also look into implementing generic callback. See: https://github.com/climate-machine/CLIMA/blob/master/src/ClimaAtmos/Dycore/src/GenericCallbacks.jl
Also see: https://fluxml.ai/Flux.jl/stable/training/training/#Callbacks-1

I think this is pretty vague so I'm closing this issue. It'll become clear what we want in our diagnostics once we start trying to do fast on-the-fly diagnostics.


cilium: Add nodeID object with reference counts for OOO events

Handle OOO pod/kvstore/node events.
Assumptions and Requirements:
We require same event types for single objects to be ordered. For example:
NodeAdd(1) -> NodeDelete(1)
is valid. But we would not handle this order currently,
NodeDelete(1) -> NodeAdd(1)
I'm assuming the API server and watchers can at least maintain this constraint. We don't however require any ordering between Node events and Pod events nor between different objects. So these are valid,
PodAdd(hostA.IP) -> NodeAdd(hostA) -> NodeDelete(hostA.IP) -> PodDelete(hostA.IP)
NodeAdd(hostA) -> NodeDelete(hostA) -> PodAdd(hostA.IP) -> PodDelete(hostA.IP)
Also noteworthy we require the Delete events to eventually reach us to be eventually consistent. We also require the delete events to reach us before the system reuses the IP. If the system were to try to reuse the hostIP before we received a delete for the same IP we wouldn't be able to identify if the next delete event was for the old object or the latest object.
NodeID Issue
The following flow can currently cause an orphaned nodeID, meaning  a nodeID is associated with an IP in the nodeID table but it maps to a stale IP.
NodeDelete(hostA) -> Pod{Add|Delete}(hostA.IP)
the reason is nodeIDs are removed from the nodeID map on Node Delete. But, in order to handle the case where pod or kvstore events are received before the NodeAdd ipcacheUpdates will also create an id if no id is found. So we get,
     Event                                              Mapping
 ---------------------------------------------------------------
     NodeAdd(hostA)                           id:1, hostA.IP
     NodeDelete(hostA)                      --- // empty now
     PodAdd(hostA.IP)                          id:2, hostA.IP
     PodDelete(hostA.IP)                     id.2, hostA.IP

After the last PodDelete we have a stale entry and no future node delete to clean up the table. Couple things might happen now. We can leak nodeIds until nodeID cache runs out. If the IPAM reuses an IP we can collide and use the old id which would be incorrect.
Fix
The first naive attempt is to just delete the id on PodDelete() but this trivially breaks the expected case,
NodeAdd() -> PodAdd() -> PodDelete() /// broken -> NodeDelete()
So we can't simply remove on PodDelete. This series instead adds refcnt and delete from pod flow so that the broken case is resolved as follows,
     Event                                              Mapping                     RefCnt:
 ----------------------------------------------------------------------------------------------
     NodeAdd(hostA)                           id:1, hostA.IP                 1
     NodeDelete(hostA)                      --- // empty now            0
     PodAdd(hostA.IP)                          id:2, hostA.IP                1
     PodDelete(hostA.IP)                    --- // empty now             0

And the good case continues to work as expected,
     Event                                              Mapping                     RefCnt:
 ----------------------------------------------------------------------------------------------
     NodeAdd(hostA)                           id:1, hostA.IP                 1
     PodAdd(hostA.IP)                          id:2, hostA.IP                2
     PodDelete(hostA.IP)                     id:2, hostA.IP                 1
     NodeDelete(hostA)                      --- // empty now             0

Nice to do
I didn't do all the nice to have things in this series couple items on the todo list,

Lift nodeID logic into its own pkg so we can fake the map updates and write unit tests
Lift nodeID allocation out of ipsec and encap specific logic and use in all cases

Testing
I wrote a set of unit tests to handle the cases attached here. But, unfortunately it requires commenting out the bpf map updates because there is no way to "fake" those from this layer easily. Also I didn't catch if there was a way to fake the GetLocalIP calls they were panic'ing because I didn't setup those layers correctly from the unit tests.
Then I tested by hand in an EKS ENI setup with 30nodes. Where I jumped down/up between nodes and had a stress test nginx server that ran on all nodes as they came online and would terminate and restart every few minutes. This let me get lots of interesting order on events posted for node add/delete and pod add/delete while having traffic in-flight which would trigger warnings/bugs when hitting nodeid mismatches. Also doing this shows that we get IP reuse so that we get collisions. With this PR I've managed to run this scenario for 24hrs or so without any further issues.
To test if we have "bad" id mappings I check the clusters ipcache for any entries that are not local objects without a nodeID or with a nodeID not in the nodeID list. And then check the other way to ensure the nodeID list only has IP addresses that exist. Before the PR the nodeID list would grow over long running tests as well. Now it correctly stays roughly aligned with number of nodes in the system
Note I didn't test this PR exactly but the backport onto released version.

/test

For reference the primary blocking issue that resulted in this patch is how to restore the map on agent restart. We restore the nodeID<->IP mappings from the bpf map, but that currently doesn't include a reference count which would be necessary to work correctly. Then we would need to do some other work to recover if we missed any pod events and correct the refcnt. At minimum it seems the refcnt would have to be part of the BPF map and this convolutes agent concerns and datapath concerns. Overall I don't like it and Paul's solution to just use the bpf nodeid map feels better to me.

This PR was superseded by https://github.com/cilium/cilium/pull/27029, right? Can it be closed or is there some other case that this would be addressing?

Correct closing. superseded by #27029


[stable/cloudserver] bump docker image

What this PR does / why we need it:
Bumps the Cloudserver docker image to 8.1.5 which contains bug fixes and minor capability tweaks
Special notes for your reviewer:
Added myself to the OWNERS file as well since I am already listed as a maintainer.
cc @giacomoguiulfo
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/lgtm


Hover coordinates should be in lat/lon rather than projected coordinate system

import os
import xarray as xr
import holoviews as hv
import geoviews as gv
import holoplot.xarray
from cartopy import crs

hv.extension('bokeh')

img_opts_dict = dict(
    projection=crs.Robinson(),
    colorbar=True,
    show_grid=True
)

os.system(
    'wget -nc '
    'ftp://ftp.cdc.noaa.gov/'
    'Datasets/ncep.reanalysis.dailyavgs/'
    'surface/'
    'air.sig995.2018.nc'
)
coast = gv.feature.coastline()

obs_ds = xr.open_dataset('air.sig995.2018.nc')
img = obs_ds.holoplot.image(x='lon', y='lat', z='air', groupby='time', geo=True).options(**img_opts_dict)

from bokeh import models

obs_ds
hover = models.HoverTool(tooltips=[
    ("lon", "@lon"),
    ("lat", "@lat"),
])

img.options(tools=[hover])

Hy guys, is this issue solved?

Which part of the issue? The variable should show up correctly. However, if you project it, the lat / lon hover tooltip won't be in degrees; we're not sure on the right approach to fix that yet.

That part of the issue. Without calling HoverTool but just defining tool=['hover'] inside opts i get degree. The problem is also the variable in vdims i get ??? also for the variable, any idea?
xr_dataset = gv.Dataset(dataset, kdims = ['lon', 'lat','time'],vdims=['variable'])
ho=HoverTool(tooltips = [("variable", "@variable"),("(lon, lat)", "(@lon, @lat)")])

I tried :
import matplotlib.ticker as tick
yfmt =tick.FormatStrFormatter('%2.2e')
gview_CAI=xr_dataset.to(gv.Image, ['lon', 'lat'],vdims=[hv.Dimension('variable', value_format=yfmt)])

but it seems to do nothing.

Still confused on what That part of the issue is

Still confused on what That part of the issue is

the question marks instead of the values.

Try
hover = models.HoverTool(tooltips=[
    ("lon", "xy"),
    ("lat", "@y"),
])

Try
hover = models.HoverTool(tooltips=[
    ("lon", "$x"),
    ("lat", "$y"),
])

but it won't be in degrees

Thank you  very much for the answer, it works but as you said not in degrees.

any idea on how to show it in degrees?
is there any way to call the var value in vdims, i got ??? for that too?
hview=xr_dataset.to(hv.Image, ['lon', 'lat'],vdims=[var])

You can checkout this branch to get it in degrees:
https://github.com/pyviz/holoviews/pull/3692
What is your var name?
if
var = 'tmp'
hover = models.HoverTool(tooltips=[
    ("lon", "$x"),
    ("lat", "$y"),
     ("tmp", "@tmp"),
])

You can checkout this branch to get it in degrees:
pyviz/holoviews#3692
What is your var name?
if
var = 'tmp'
hover = models.HoverTool(tooltips=[
    ("lon", "$x"),
    ("lat", "$y"),
     ("tmp", "@tmp"),
])

Still get ??? , var is a float.64 Datavariable in xarray
var (y, x) float64 0.7663 0.7811 0.7663 0.7426 ... 0.8205 0.8103 0.7727
could be this the problem?

Sorry, it was renamed too
hover = models.HoverTool(tooltips=[
    ("lon", "$x"),
    ("lat", "$y"),
     ("tmp", "@image"),
])

hover = models.HoverTool(tooltips=[
("lon", "$x"),
("lat", "$y"),
("tmp", "@image"),
])

Sorry,

but what if there are more than one vdims?
is there any way not to show NaN values?

After the first vdim, just do @var where var is your second vdim's name.
import xarray as xr
import hvplot.xarray
from bokeh.models import HoverTool

ds = xr.tutorial.open_dataset('air_temperature').isel(time=0)

ds['airx2'] = ds['air'] * 2

import holoviews as hv
hv.extension('bokeh')

hv.Dataset(ds).to.image(['lon', 'lat'], ['air', 'airx2']).opts(tools=['hover'])

[('Longitude (degrees_east)', '$x'), ('Latitude (degrees_north)', '$y'), ('4xDaily Air temperature at sigma level 995 (degK)', '@image'), ('airx2', '@airx2')]

@ahuang11 for me it does not work.
I Have an RGB image object (with hv.RGB)  with values betwwen [0,1] and if i take @image I get a number of this order like: 418116853

Do the RGB values mean something (give you useful info) if you hover over it though? Maybe related to https://github.com/pyviz/hvplot/issues/204

@ahuang11  No nothing useful and wrong maybe I would say. I wil give you an example of what I get so you could see what i mean.
#########################################
import numpy as np
import xarray as xr
import holoviews as hv
import geoviews as gv
import cartopy.crs as crs
from bokeh.models import HoverTool, CustomJSHover
hv.extension('bokeh','matplotlib')
##########################################
x,y = np.mgrid[-50:51, -50:51] * 0.1
r = 0.5*np.sin(np.pi  +3*x**2+y**2)+0.5
g = 0.5*np.sin(x**2+2*y**2)+0.5
b = 0.5*np.sin(np.pi/2+x**2+y**2)+0.5
coords=np.arange(0,101)
#THIS WIL
custom=CustomJSHover(code="""
                     return value + " tot"
                     """)
tooltips=[
  ("value","@image{1.1}"),
  ("value","@image{custom}"),
  ("G","@G"),
  ("B","@B"),
]
hover = HoverTool(tooltips=tooltips, formatters={'image' : custom})
ds = xr.Dataset({'R': (['x', 'y'],r),'G': (['x', 'y'],g),
                 'B': (['x', 'y'],b)},
                coords={'x': (['x'], coords),
                        'y': (['y'], coords)})
gv.RGB((ds['x'],ds['y'],ds.R,ds.G, ds.B),vdims=['R','G','B'], crs=crs.PlateCarree()).opts(tools=[hover])

@ahuang11 with this example lat and lon are in degreea but for my case not well projected. Do you know how to change projection type  inside the JS code?
from bokeh.models import CustomJSHover
lat_custom = CustomJSHover(code="""
    var projections = require("core/util/projections");
    var x = special_vars.x
    var y = special_vars.y
    var coords = projections.wgs84_mercator.inverse([x, y])
    return "" + coords[1]
""")
lon_custom = CustomJSHover(code="""
    var projections = require("core/util/projections");
    var x = special_vars.x
    var y = special_vars.y
    var coords = projections.wgs84_mercator.inverse([x, y])
    return "" + coords[0]
""")

hover=HoverTool(
    tooltips=[( '(lon,lat)','(@x{custom},@y{custom})' )],
    formatters=dict(x=lon_custom,y=lat_custom)
)

Sorry, not too sure... this JS code calls
https://github.com/bokeh/bokeh/blob/13baccbb2835f7d5831c22b5cf68214e89c02618/bokehjs/src/lib/core/util/projections.ts

@emryslda

I Have an RGB image object (with hv.RGB) with values betwwen [0,1] and if i take @image I get a number of this order like: 418116853 for the others (@ G,@ B) i get ???

I just encountered this and was reminded that you had the same problem. So my current workaround is also plotting a transparent hv.Image with hover cols set to R, G, B

@ahuang11  good idea, thank you!


autotest: Create test suite and integrate with TravisCI

This also involves unifying some Markus settings and markus-autotesting defaults to work out of the box


Confusing usage of the word "credentials"

README says:
"This application should give you a ready-made starting point for writing your own messaging apps with Twilio IP Messaging. Before we begin, we need to collect all the credentials we need to run the application"
The table following that sentence has more than just credentials in it though.

I see you Jake Sully. Updated README


Best way of making Pagination Footer disappear when paging=False

I would like to completely remove the pagination component and the space it takes out.
When paging = False the Table Pagination Component keeps appearing but empty.
Any suggestion on the best way to do this?

My mistake


Provide TZ as environment variable

What feature would you like added?
It would be useful to be able to set the timezone so that log date time stamps align to the given region.
If this could be defined on the environment collection, that would be ideal.
Example
environment:
- TZ=Europe/London
Version
1.1.2
Log output
No response

This already works because the Docker container is based off of node:18, but it's not in the README. I'll update it to include this.


explore Janis as a framework for achieving this

https://janis.readthedocs.io/en/latest/
Their recent release supports reading in many CWL files and converting to WDL or Nextflow, according to https://docs.google.com/presentation/d/1JTR_PVtcc5cg_qa0blv4XCsuBVie8F2bvKnTizuNioU/mobilepresent#slide=id.gd316bc3748_0_107
I believe they are working on parsing WDL next, so perhaps we can assist them..

https://github.com/PMCC-BioinformaticsCore/janis-core/pull/69 is their work in progress for WDL import ; see the limitations section for more information


How to send get request for the list of urls....

I have below urls in the array list.   How to sent get request  to all of the urls...
http://www.parallec.io/docs/track-status/
https://stackoverflow.com/questions/3142915/how-do-you-create-an-asynchronous-http-request-in-java

Hi @satheessh
Thanks for trying Parallec! For these 2 urls on parallec.io and stackoverflow.com, I assume they belong to the category of "Different requests to different target hosts"
Could you please follow
http://www.parallec.io/docs/submit-task/#apis-on-variable-replacement-for-heterogeneous-requests
https://github.com/eBay/parallec-samples/blob/master/sample-apps/src/main/java/io/parallec/sample/app/http/HttpDiffRequestsDiffServersApp.java
thanks,

close it for now