Particle quantity not working

The quantity option of the playeffect command doesn't effect the count of the particles.
- playeffect <player.location> effect:FLAME quantity:100 targets:<player> returns ONE single particle.
I'm using Denizen build 1589 and Spigot build 6d0ae89-a03743b.

The default offset is 0.5 I wouldn't be shocked if it /looked/ like one particle. Try setting something like offset:2 and see how that effects it.

That's not the problem! It also doesn't work with higher offset values.

You're running the latest dev build, right?

Build 1589 when I type /denizen version.

This is fixed in the latest developmental build.
http://ci.mineconomy.org/job/Denizen_Developmental/lastSuccessfulBuild/

You're several releases behind, and that's a release build, not a dev build.
The latest dev build is this: http://ci.mineconomy.org/job/Denizen_Developmental/lastSuccessfulBuild/

This version doesn't work. Do I have to run it with the latest Spigot version?

For example the on player joins event doesn't work.

Yeah you need latest Spigot.

It works now! Thanks! :smile_cat:


[Windows] Expose and Rewrite forceRefresh

Force refresh the window or the flutter view by resizing the window by 1 pixel and then restoring the size.
Might fix #155 (need confirmation)

I don't think this method should be exposed to the user


Cmake compile always fails

Scanning dependencies of target opencc_dict
[ 54%] Building CXX object src/tools/CMakeFiles/opencc_dict.dir/DictConverter.cpp.obj
[ 57%] Linking CXX executable opencc_dict.exe
CMakeFiles/opencc_dict.dir/objects.a(DictConverter.cpp.obj):DictConverter.cpp:(.text+0x94c): undefined reference to `opencc::ConvertDictionary(std::__cxx11::basic_string<char, std::char_traits, std::allocator >, std::__cxx11::basic_string<char, std::char_traits, std::allocator >, std::__cxx11::basic_string<char, std::char_traits, std::allocator >, std::__cxx11::basic_string<char, std::char_traits, std::allocator >)'
collect2.exe: error: ld returned 1 exit status
make[2]: *** [src/tools/opencc_dict.exe] Error 1
make[1]: *** [src/tools/CMakeFiles/opencc_dict.dir/all] Error 2
make: *** [all] Error 2

i run into this bug too, but it has been fixed: https://github.com/BYVoid/OpenCC/commit/76caac3c6a350e5dbeff6b43802803af46812b74


Update typeAcquisition.ts: detect dynamic imports

See changes between:
https://regex101.com/r/hdEpzO/6
https://regex101.com/r/hdEpzO/8
This allows for dynamic imports (e.g.: let x.= await import("mod")

Cool, looks good to me 👍🏻

Cool, looks good to me 👍🏻

Ironically I think this fixes a bug where strings like this would be matched incorrectly as imports:
"DELETE /repos/{owner}/{repo}/import": Operation<"/repos/{owner}/{repo}/import", "delete">;

This string (and many others) come from @octokit/types - you can see the errors the old regex is causing in the "Automatic Type Acquisition" example in the playaround, which has these errors in the console:

It seems that Deno is using the same regex, as it suffers from the same bug.

Ironically I think this fixes a bug where strings like this would be matched incorrectly as imports:
"DELETE /repos/{owner}/{repo}/import": Operation<"/repos/{owner}/{repo}/import", "delete">;

This string (and many others) come from @octokit/types - you can see the errors the old regex is causing in the "Automatic Type Acquisition" example in the playaround, which has these errors in the console:

It seems that Deno is using the same regex, as it suffers from the same bug.


[TACHYON-679] Put shared test utils in an independent module

https://tachyon.atlassian.net/browse/TACHYON-679

Jenkins, test this please

@apc999, please review this PR

@uronce-cc , thanks for the patch! We think the benefit of having an additional module with only two classes is kinda marginal. Instead of adding this module but still to clean up the dependency on test-jar of tachyon-servers, we may want to split the two test util files and refactor them into other parts:
(1) for TestUtils.java,  getIncreasingByteArray and other buffer stuff can go to BufferUtils? uniqPath can may go to pathUtils and writeBufferToFile can may go to fileUtils
(2) for TachyonFSTestUtils.java, they seem to me legitimate to be a part of src of tachyon-client
In this way, we can probably still clean up the dependency :)

@apc999 thx for suggestion. I came up with the solution, but I thought maybe these utils are only for tests, and not so appropriate to put in main code base. hmm, yup, they may be useful for other parts too. I'll close and open another PR.


'utf-8' codec can't decode byte 0xff in position 14: invalid start byte

Hi. I get issue if ping from server to client.
I'm getting an error like the one in the title, how do I fix it?
I'm using 1 server and 2 clients, and I'm getting an error like the title when I have 2 clients.

i made a maintained fork : https://github.com/LopeKinz/Python-Botnet


Page resembles apartheid South African Flag

The new cc webpage, shown in the screenshot resembles the flag of the apartheid regime in South Africa.
Screensjot

Previous South African flag

The resemblance is not just a strip of orange and then white and then blue but also by having other logos on the middle strip.  The previous South African flag is associated with the discredited apartheid system by millions of South Africans. The resemblance of the webpage to that flag will have unfortunate, probably unconscious resonance not only for South Africans but many others.

@ryanmerkley @peterspdx

Current design is now orange and yellow.


Create and assign an InvocationSpanContext for every invocation

@fhanau ... please consider this one carefully. What I'm not sure about is whether the InvocationSpanContext should be held by the IncomingRequest or something internal to the RequestObserver.
Eventually I'd like to abstract out the WorkerTracer here and actually hide that inside the RequestObserver and put it entirely behind SpanBuilder but that'll come later. Accordingly, however, I don't want to put InvocationSpanContext inside the WorkerTracer.
The other thing to keep in mind is that we'll need to be able to propagate the InvocationSpanContext across subrequests of various kinds so we'll need easy access to it to place it into things like the control header.
How is this expected to be used? Good question! When streaming tail workers is fully operational, the invocation span context will be recorded with each mark event. Every child span created during the invocation of a worker will become a child off this root and should have its own kj::Rc<InvocationSpanContext> and there should always be a "current invocation span context" at any given time during an invocation that is propagated via the async context just like the current SpanParent. The fact that the user SpanBuilder would need to have access to this suggests to me that maybe the root InvocationSpanContext should be owned by the RequestObserver but I'm not entirely sure.

@fhanau ... please consider this one carefully. What I'm not sure about is whether the InvocationSpanContext should be held by the IncomingRequest or something internal to the RequestObserver.

The thoughts above seem reasonable, I don't have much to add. Intuitively, having the InvocationSpanContext be owned by RequestObserver in some capacity feels more sensible/easier to support, but I could also see it work with IncomingRequest. Having it in/around RequestObserver would be more in line with the current model, but I guess we're making big changes anyway.

Ok, well, we can keep it here for now and can always move it later if we discover it makes more sense in RequestObserver


Avoid intern short Strings

In case the string is less than 8 bytes, will storing the string and not a pointer will save some space?
Also in some of the cases interning the IString doesn't worth it since the value is unique and in fact we just add overhead.

It's possible this could work on 64-bit platforms. But it would add a fair amount of complexity. Could maybe limit it to 7 characters which would allow the low bits to still contain the tag value for string, and the length could also be stuffed into that byte in the non-tag space.


Getting error in base.js

Hey,
I just followed the given steps and whenever I am trying to build handler with
handler = Gmaps.build('Google');

I am getting this error in javascript console
Uncaught ReferenceError: _ is not defined

And it is coming from line nuber 8 of gmaps/base.js
model = _.isFunction(options.handler) ? options.handler : Gmaps.Objects.Handler;

The whole content of gmaps/base.js is
(function() {
  this.Gmaps = {
    build: function(type, options) {
      var model;
      if (options == null) {
        options = {};
      }
      model = _.isFunction(options.handler) ? options.handler : Gmaps.Objects.Handler;
      return new model(type, options);
    },
    Builders: {},
    Objects: {},
    Google: {
      Objects: {},
      Builders: {}
    }
  };

}).call(this);

Did I missed or messed something?

Hi! I get the same error. I am not sure how to fix it? How do you get underscore.js to work?


Draw diamond and horizontal hexagon shields mathematically

Adds functions to draw diamond shields and horizontal hexagon shields.

They're needed; I just factored it into ShieldText.calculateTextWidth().


Feat: Create an ' Contributing to Open Source' Page

Category

[X] Documentation
[X] Resource Addition
[ ] Codebase
[X] User Interface
[ ] Feature Request

Description
Resource :
Name: Github
Link: https://opensource.guide/how-to-contribute/
Name: Stack Overflow
Link: https://stackoverflow.blog/2020/08/03/getting-started-with-contributing-to-open-source/
And all the resources mentioned in your readme page.
Please assign me under JWOC 2024, if approved.

@Surajit0573 You're assigned.

@Sriparno08 pls assign this issue to me under JWOC

@vishapraj Another Mentee @Surajit0573 is already assigned to this issue.

@Sriparno08 I would love to work on this issue under JWOC. Please assign this issue to me

please Assign it to me

@Sriparno08 As the assignee passed the 4 day limit and hasn't pushed any Pull Request.
Can you assign this issue to me under JWOC.

@Sriparno08 As the assignee passed the 4 day limit and hasn't pushed any Pull Request. Can you assign this issue to me under JWOC.

I haven't "passed the 4 day limit". Today is the 4th day; if I can't make any PR by the end of the day, you can take it from tomorrow.


Add option to disable manual resizing of the window

Thanks for this, but what would the use case of this be? I think by exposing all the BrowserWindow options via the cli as suggested in #35 would actually be a better solution.

yep #35 would be the better soloution.


Add actions

Branch 279-add-actions
Actions doess not work on components with functionality in storybook.
OnChange can't be an action and have a function handleOnchange at the same time as it is coded now

Eksample of  onChange as action but also eventhandler: https://6195b518b76f57003aa69b4c-ynczzfqqyq.chromatic.com/?path=/docs/form-checkboxgroup--default.
Github code to this example: https://github.com/phork/phorkit/blob/develop/src/components/Form/Checkbox/stories/CheckboxGroup.stories.tsx


Concurrency issues

This library is great, it is only a small adjustment and it somehow makes a complex library like SQLAlchemy compatible with Trio and Postgresql! I had difficulties getting anything else to work.
I did notice however that if I have multiple coroutines making a connection that I would get errors originating from trio_asyncio about asyncio futures being active. I am specifically sharing a singleton engine and singleton async_sessionmaker for all those coroutines. Now I had to wrap any use of the async_sessionmaker with a lock to prevent concurrent access to the database.
Am I doing something wrong here?

do you mean that you're having difficulty running concurrent tasks using the same session object? if so, that pattern is not supported by SQLAlchemy - every tasks requires its own database session.
if that's not what you mean, could you share a code snippet?

do you mean that you're having difficulty running concurrent tasks using the same session object? if so, that pattern is not supported by SQLAlchemy - every task requires its own database session.
if that's not what you mean, could you share a code snippet?

I think not although I have to double check my code to be sure. But I exclusively use the async sessionmaker to create new sessions and I'm not passing any session objects around. I think that the sessionmaker makes new sessions only so I don't think I'm concurrently using the same session in different tasks.

The error & stacktrace I am getting:
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/ext/asyncio/session.py", line 574, in scalars
    |     result = await self.execute(
    |              ^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/ext/asyncio/session.py", line 461, in execute
    |     result = await greenlet_spawn(
    |              ^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/util/_concurrency_py3k.py", line 201, in greenlet_spawn
    |     result = context.throw(*sys.exc_info())
    |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/orm/session.py", line 2351, in execute
    |     return self._execute_internal(
    |            ^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/orm/session.py", line 2226, in _execute_internal
    |     conn = self._connection_for_bind(bind)
    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/orm/session.py", line 2095, in _connection_for_bind
    |     return trans._connection_for_bind(engine, execution_options)
    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "<string>", line 2, in _connection_for_bind
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/orm/state_changes.py", line 139, in _go
    |     ret_value = fn(self, *arg, **kw)
    |                 ^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/orm/session.py", line 1189, in _connection_for_bind
    |     conn = bind.connect()
    |            ^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/engine/base.py", line 3276, in connect
    |     return self._connection_cls(self)
    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/engine/base.py", line 146, in __init__
    |     self._dbapi_connection = engine.raw_connection()
    |                              ^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/engine/base.py", line 3300, in raw_connection
    |     return self.pool.connect()
    |            ^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/pool/base.py", line 449, in connect
    |     return _ConnectionFairy._checkout(self)
    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/pool/base.py", line 1263, in _checkout
    |     fairy = _ConnectionRecord.checkout(pool)
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/pool/base.py", line 712, in checkout
    |     rec = pool._do_get()
    |           ^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/pool/impl.py", line 179, in _do_get
    |     with util.safe_reraise():
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/util/langhelpers.py", line 146, in __exit__
    |     raise exc_value.with_traceback(exc_tb)
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/pool/impl.py", line 177, in _do_get
    |     return self._create_connection()
    |            ^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/pool/base.py", line 390, in _create_connection
    |     return _ConnectionRecord(self)
    |            ^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/pool/base.py", line 674, in __init__
    |     self.__connect()
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/pool/base.py", line 914, in __connect
    |     )._exec_w_sync_on_first_run(self.dbapi_connection, self)
    |       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/event/attr.py", line 481, in _exec_w_sync_on_first_run
    |     with self._exec_once_mutex:
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/util/_concurrency_py3k.py", line 224, in __enter__
    |     return await_fallback(self.mutex.acquire())
    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/util/_concurrency_py3k.py", line 164, in await_fallback
    |     return current.parent.switch(awaitable)  # type: ignore[no-any-return,attr-defined] # noqa: E501
    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |   File "/home/laurens/projects/news-archive/venv/lib/python3.11/site-packages/sqlalchemy/util/_concurrency_py3k.py", line 196, in greenlet_spawn
    |     value = await result
    |             ^^^^^^^^^^^^
    |   File "/usr/local/lib/python3.11/asyncio/locks.py", line 114, in acquire
    |     await fut
    | TypeError: trio.run received unrecognized yield message <Future pending>. Are you trying to use a library written for some other framework like asyncio? That won't work without some kind of compatibility shim.

if you're using session maker everywhere, then things should be working fine
would you mind providing a small example to reproduce the behavior? the snippet you provided, without the lock, should be functional...

if you're using session maker everywhere, then things should be working fine
would you mind providing a small example to reproduce the behavior? the snippet you provided, without the lock, should be functional...

Haven't gotten around to it yet but I plan to produce a minimal working example soon

I was able to reduce the code to a minimal working example in which the error still occurs.
from __future__ import annotations

import contextlib
import datetime
import sys

import trio
from loguru import logger
from sqlalchemy import DateTime
from sqlalchemy import select
from sqlalchemy.dialects import registry
from sqlalchemy.ext.asyncio import AsyncAttrs, async_sessionmaker, create_async_engine
from sqlalchemy.orm import (
    DeclarativeBase,
    Mapped,
    mapped_column,
)
from sqlalchemy.sql.expression import func
from trio_asyncio import run

registry.register("triopg", "sqlalchemy_triopg.triopg", "TrioPGDialect")

engine = create_async_engine("triopg://my-server", echo=False)

async_session = async_sessionmaker(engine, expire_on_commit=False)

class Base(AsyncAttrs, DeclarativeBase):
    __abstract__ = True

class Article(Base):
    __tablename__ = "articles"

    id: Mapped[int] = mapped_column(primary_key=True)
    url: Mapped[str]
    title: Mapped[str]
    on_archive: Mapped[int]
    on_disk: Mapped[int]
    lastmod: Mapped[datetime.datetime] = mapped_column(DateTime(timezone=True))
    site: Mapped[int]
    publish_date: Mapped[datetime.datetime] = mapped_column(DateTime(timezone=True))
    record_date: Mapped[datetime.datetime] = mapped_column(DateTime(timezone=True))
    skip: Mapped[int] = mapped_column(default=0)
    attempts: Mapped[int]
    last_attempt_msg: Mapped[str]
    last_attempt_ts: Mapped[datetime.datetime] = mapped_column(DateTime(timezone=True))

async def load_from_db() -> list[Article]:
    async with async_session() as session:
        stmt = select(Article)

        return (await session.scalars(stmt)).all()

async def main():
    async with trio.open_nursery() as nrs:
        nrs.start_soon(load_from_db)
        nrs.start_soon(load_from_db)

if __name__ == "__main__":
    run(main)

This is my pip environment (linux, ubuntu server, python 3.11.4):
ago==0.0.95
aio-databases==0.15.0
aiobotocore==2.5.4
aiohttp==3.8.5
aioitertools==0.11.0
aiopg==1.4.0
aiosignal==1.3.1
annotated-types==0.6.0
anyio==4.0.0
async-generator==1.10
async-timeout==4.0.3
asyncpg==0.29.0
attrs==23.1.0
Automat==22.10.0
beautifulsoup4==4.12.2
boto3==1.28.52
botocore==1.31.17
bs4==0.0.1
cachetools==5.3.3
certifi==2024.6.2
cffi==1.15.1
chardet==5.2.0
charset-normalizer==3.2.0
click==8.1.7
cloudpickle==3.0.0
connectorx==0.3.3
constantly==15.1.0
courlan==0.9.4
cryptography==41.0.4
cssselect==1.2.0
curl_cffi==0.7.0b4
dateparser==1.1.8
deltalake==0.18.2
dotmap==1.3.30
duckdb==1.0.0
elastic-transport==8.4.0
elasticsearch==8.9.0
expiring-dict==1.1.0
faust-cchardet==2.1.19
feedfinder2==0.0.4
feedparser==6.0.10
filelock==3.12.4
frozenlist==1.4.0
fsspec==2023.9.2
greenlet==3.0.3
h11==0.14.0
hjson==3.1.0
htmldate==1.5.1
httpcore==0.17.3
httpx==0.24.1
hurry.filesize==0.9
hyperlink==21.0.0
idna==3.4
incremental==22.10.0
iniconfig==2.0.0
itemadapter==0.8.0
itemloaders==1.1.0
jieba3k==0.35.1
jmespath==1.0.1
joblib==1.3.2
jusText==3.0.0
langcodes==3.3.0
langdetect==1.0.9
loguru==0.7.1
lxml==4.9.3
multidict==6.0.4
mypy==1.8.0
mypy-extensions==1.0.0
nltk==3.8.1
numpy==1.25.2
outcome==1.2.0
packaging==23.1
pandas==2.1.4
parsel==1.8.1
peewee==3.17.6
peewee-aio==1.7.4
Pillow==10.0.1
plac==1.4.0
pluggy==1.3.0
polars==1.2.1
Protego==0.3.0
psycopg2-binary==2.9.7
pyarrow==17.0.0
pyarrow-hotfix==0.6
pyasn1==0.5.0
pyasn1-modules==0.3.0
pycparser==2.21
pydantic==2.6.1
pydantic_core==2.16.2
PyDispatcher==2.0.7
pyee==11.1.0
PyMySQL==1.1.0rc2
pyOpenSSL==23.2.0
PySocks==1.7.1
pytest==7.4.2
pytest-pycharm==0.7.0
pytest-trio==0.8.0
pythainlp==4.0.2
python-dateutil==2.8.2
pytz==2023.3.post1
PyYAML==6.0.1
queuelib==1.6.2
readability-lxml==0.8.1
regex==2023.8.8
requests==2.31.0
requests-file==1.5.1
s3fs==2023.9.2
s3transfer==0.6.2
Scrapy==2.11.0
selenium==4.22.0
service-identity==23.1.0
sgmllib3k==1.0.0
six==1.16.0
sniffio==1.3.0
socksio==1.0.0
sortedcontainers==2.4.0
soupsieve==2.5
SQLAlchemy==2.0.32
sqlalchemy-triopg==1.0.0
tblib==3.0.0
tinysegmenter==0.3
tld==0.13
tldextract==3.6.0
tqdm==4.66.1
trafilatura==1.6.2
trio==0.22.2
trio-asyncio==0.12.0
trio-mysql==1.0.3
trio-parallel==1.2.1
trio-util==0.7.0
trio-websocket==0.11.1
triopg==0.6.0
Twisted==22.10.0
typing_extensions==4.12.2
tzdata==2023.3
tzlocal==5.0.1
urllib3==1.26.16
w3lib==2.1.2
warcio==1.7.4
websocket-client==1.8.0
wrapt==1.15.0
wsproto==1.2.0
yarl==1.9.2
zope.interface==6.0

Upon further investigation I found that a asyncio Lock object is the issue. I was able to solve the error by monkey patching the lock being used by SQLAlchemy:
from sqlalchemy.util.langhelpers import memoized_property
from sqlalchemy.util._concurrency_py3k import await_fallback
from typing import Any
import sqlalchemy

class AsyncAdaptedLock:
    @memoized_property
    def mutex(self) -> trio.Lock:
        # there should not be a race here for coroutines creating the
        # new lock as we are not using await, so therefore no concurrency
        return trio.Lock()

    def __enter__(self) -> bool:
        # await is used to acquire the lock only after the first calling
        # coroutine has created the mutex.
        return await_fallback(self.mutex.acquire())

    def __exit__(self, *arg: Any, **kw: Any) -> None:
        self.mutex.release()

if __name__ == "__main__":
    from unittest.mock import patch

    sqlalchemy.event.attr.AsyncAdaptedLock = AsyncAdaptedLock
    run(main)

hi @iiLaurens: well, this is a fascinating problem! i can reproduce this error, but it seems to come up only in certain circumstances. my existing unit tests always drop and recreate the database tables before every test. if i do that, this error does not appear. if i let the tables persist between tests, though, I run into this error.
strange indeed
however, your proposal for patching sqlalchemy's lock is seemingly the best option. i'll play around with it a little more, and release a new version that includes whatever changes are necessary. thanks!

this should be resolved in v1.0.1.

@iiLaurens does the new version work for you?


[Health Tracker] Heart rate monitoring sometimes turns screen backlight on (?)

Affected hardware version
Bangle 2
Your firmware version
2v12
The bug
I turned on heart rate tracking in the health tracking app, and it seems to cause the watch's backlight to turn on occasionally. This is very annoying at night time.
It looks like the screen turning on is sync'd with the 3 minute period, but I haven't observed it enough times to be sure. It also appears to have stopped happening since I turned off heart rate tracking, which is why I think it is caused by heart rate tracking.
I have not been able to reproduce it with Bangle.setHRMPower(1) in the console, and I've also had the watch running now with heart rate tracking switched back on for the last 10 minutes and haven't seen the screen turn on, so I don't think it happens every time, but I don't know what else could be making the backlight come on in the middle of the night.
I wasn't wearing it and it wasn't charging. It was just sitting on a table.

Could it be the bluetooth connection dropping in and out? That is sometimes a cause.
It seems extremely unlikely it's heart rate monitoring - I've never seen that happening, and the HRM code internally never touches the screen

Thanks, it could well be! I'll close this on the assumption that it is that, and I'll reopen if I discover otherwise.


Fix #388: off by one bug where wrong command is returned

Keep track of whether the last command was valid or not

Nuances:

if valid, we should return the current command (i.e. the latest command entered), since on the UI the text field is reset to be empty after entering the command
if invalid, we should return the previous command since the UI will display the invalid command in red to show that it is invalid

Everytime the navigation goes to the oldest or most recent command, a double key press is required in order to retrieve the next comand.
e.g.
When counter is at 0 (oldest command), when getNextCommand() is executed, it will return the command again at index 0 before incrementing the counter.

Codecov Report

Merging #389 (1c7928f) into master (7190c83) will decrease coverage by 1.41%.
The diff coverage is 36.14%.

@@             Coverage Diff              @@
##             master     #389      +/-   ##
============================================
- Coverage     60.98%   59.56%   -1.42%
+ Complexity      542      541       -1
============================================
  Files           101      100       -1
  Lines          2040     2075      +35
  Branches        209      225      +16
============================================
- Hits           1244     1236       -8
- Misses          708      747      +39
- Partials         88       92       +4

Impacted Files
Coverage Δ

src/main/java/seedu/programmer/MainApp.java
0.00% <0.00%> (ø)

...main/java/seedu/programmer/logic/LogicManager.java
46.87% <0.00%> (+2.75%)
:arrow_up:

...seedu/programmer/logic/commands/AddLabCommand.java
62.50% <0.00%> (ø)

...du/programmer/logic/commands/DeleteLabCommand.java
69.56% <0.00%> (ø)

...edu/programmer/logic/parser/EditCommandParser.java
64.70% <0.00%> (-18.63%)
:arrow_down:

src/main/java/seedu/programmer/model/Model.java
100.00% <ø> (ø)

...ain/java/seedu/programmer/model/student/Email.java
80.00% <ø> (ø)

...n/java/seedu/programmer/model/student/Student.java
81.17% <ø> (ø)

...java/seedu/programmer/model/student/StudentId.java
80.00% <ø> (ø)

.../student/exceptions/DuplicateStudentException.java
100.00% <ø> (ø)

... and 35 more

Continue to review full report at Codecov.

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


Microsoft Edge Issues

Been getting some emails and messages that when we push an update, it breaks the site on Microsoft Edge. The issue seems to be that Edge is holding on to the old cache. Is there any way we can force the browser to reset cache after pushing a change?
Here's what Edge users see when this happens, so this is not a graceful break.

There seem to be several solutions.
https://stackoverflow.com/questions/1922910/force-browser-to-clear-cache

changing title as it seems this affects other browsers as well

+1 to adding a query string version number. This is used by Wikipedia, for example.
It will however require a version number increment commit each time you deploy a new asset.
<script src="foo.js?27">

We might want to have some form of automation for this?

Happy to do it manually as well if it's simpler. Right now it's nice (at least from my perspective) that we don't need to compile or build anything.
So if I understand this correctly, all that's need is to add a ?number to the end of all the updated files linked in index.html? i.e. linked javascript and css files? Let's try to do this on future updates.

Closing. Adopting the approach were we add a ?number to updated linked .css & .js files


feat(deps): use RHDH themes in the backstage app and dev pages

Fixes: https://issues.redhat.com/browse/RHIDP-1171
Use the RHDH theme (extracted as npm package) also in the backstage app and the dev pages.
For now it shows different themes, incl. backstage, RHDH 1.0, 1.1, and 1.2.

https://github.com/redhat-developer/red-hat-developer-hub-theme
https://npmjs.com/package/@redhat-developer/red-hat-developer-hub-theme

/retest

@jerolimov the test jobs seem to fail due to link issues
       @redhat-developer/red-hat-developer-hub-theme must be declared in devDependencies of plugins/analytics-module-matomo/package.json, run 'yarn --cwd plugins/analytics-module-matomo add --dev @redhat-developer/red-hat-developer-hub-theme' from the project root

       plugins/analytics-module-matomo/dev/index.tsx:5:1
       3 | import { createDevApp } from '@backstage/dev-utils';
       4 |
     > 5 | import { createDevAppThemes } from '@redhat-developer/red-hat-developer-hub-theme';
         | ^
       6 |
       7 | import { analyticsModuleMatomoPlugin } from '../src';
       8 | import { Playground } from './Playground';

  ✘ 1 problem (1 error, 0 warnings)

https://github.com/janus-idp/backstage-plugins/actions/runs/8833415969/job/24371757247?pr=1480#step:3:512

Hi @jerolimov , the theme works perfectly on plugins. Just the theme options aren't scrollable.

@jerolimov the test jobs seem to fail due to link issues
       @redhat-developer/red-hat-developer-hub-theme must be declared in devDependencies of plugins/analytics-module-matomo/package.json, run 'yarn --cwd plugins/analytics-module-matomo add --dev @redhat-developer/red-hat-developer-hub-theme' from the project root

       plugins/analytics-module-matomo/dev/index.tsx:5:1
       3 | import { createDevApp } from '@backstage/dev-utils';
       4 |
     > 5 | import { createDevAppThemes } from '@redhat-developer/red-hat-developer-hub-theme';
         | ^
       6 |
       7 | import { analyticsModuleMatomoPlugin } from '../src';
       8 | import { Playground } from './Playground';

  ✘ 1 problem (1 error, 0 warnings)

https://github.com/janus-idp/backstage-plugins/actions/runs/8833415969/job/24371757247?pr=1480#step:3:512

I fixed this kind of build errors. I believe that the other test errors aren't related to this PR. :man_shrugging:

/retest

LGTM

I haven't seen any issues with the Orchestrator plugin UX.
My only observation is that overflowX for Backstage page root is scroll not auto and kinda light mode (see the bottom bar in the screenshot).

Tested Argocd plugin and Quay plugin, both looks good and no issues found! 🎉

In Tekton plugin, RHDH Light (latest) and RHDH Dark (latest) is showing -- under the collapsable rows . Other parts in tekton like logs, output dashboards etc looks good.

@karthikjeeyar @ciiay I've updated to include 3 merged PRs from @ciiay in the theme repo:

https://github.com/redhat-developer/red-hat-developer-hub-theme/pull/8
https://github.com/redhat-developer/red-hat-developer-hub-theme/pull/10
https://github.com/redhat-developer/red-hat-developer-hub-theme/pull/11
PR 8 fixes https://issues.redhat.com/browse/RHIDP-2289 - [UI] Hide scrollbar when not scrollable
PR 10 fixes https://issues.redhat.com/browse/RHIDP-2290 - [UI] Elements located by "makeStyles-" classname doesn't get applied to elements on ci builds
PR 11 fixes https://issues.redhat.com/browse/RHIDP-2300 - [UI] Remove double dash on unexpected components

Can you please take a look and try to get this in? And, of course, we will update the theme later regularly.

/approve

/lgtm


linux: make RejitReason.cpp build

Forward declaring enums is not allowed by the C++ standard (I assume
this just happens to work in MSVC). To fix the build error declare the
storage type of the enum in the forward declaration and its
definition. This is allowed since C++0x.

Hi @xanlpz, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Pushed another commit making DateUtilities.cpp build too.

Hrm, not building, so I guess the way I added this to pal/ is not OK. I guess S_INT64 does not actually exist anywhere?

@xanlpz Please try "int64" instead. We have a typedef in CommonTypedefs.h.

Right, the thing is S_INT64 is something I added in pal/ to allow for safe addition/multiplication of (signed) int64 types. But it seems it's not building on Windows. I'm guessing it does not actually exist on Windows, so I can probably just use ClrSafeInt directly?

Right, the thing is S_INT64 is something I added in pal/ to allow for safe addition/multiplication of (signed) int64 types. But it seems it's not building on Windows. I'm guessing it does not actually exist on Windows, so I can probably just use ClrSafeInt directly?

@xanlpz The issue here is that ClrSafeInt is from pal, and we don't use pal on Windows. We only use pal to fill in things missing on unix.
I noticed Int64Add/Int64Mul are missing from pal. One alternative here is to change Int64Add/Int64Mul code to use Int64Math.

You are right, I misunderstood one comment and thought this was a standard class. Pushed a new version using Int64Math, let's see if this one builds.


Add blackbox exporter dashboard and add probes from WC to SC

Is your feature request related to a problem? Please describe.
It would be of great interest to add a dashboard for blackbox exporter to easily see the performance of probes within an environment, both in terms of DNS resolve and TCP connect.
Furthermore it would be nice with probes going from the WC to public SC endpoints to monitor how well it can communicate with it.
Describe the solution you'd like

Added dashboard for blackbox exporter
Added probes in WC against public target in SC

Definition of done:
Both points added above.

See https://grafana.com/grafana/dashboards/7587-prometheus-blackbox-exporter/

Furthermore it would be nice with probes going from the WC to public SC endpoints to monitor how well it can communicate with it.

Should this be a separate issue?

I don't think it needs to be since there shouldn't be too much work to adapt the current probes to be checking from both clusters. But a separate PR would be preferred.

So what's a good target URL to hit?

You can reuse the one defined for SC in https://github.com/elastisys/compliantkubernetes-apps/blob/main/helmfile/values/prometheus-blackbox-exporter-sc.yaml.gotmpl
Skipping the in cluster addresses.


Getting the where clause

I was wondering if it's possible to get the WHERE clause that makes up the statement from the query?  Would like to be able to get what the WHERE clause that was used in the execution of the query.
Try this call but no results:
$query->getQuery('criteriaonly');
calling
$query->getQuery();  - Returns the entire SQL Query that is to get executed.
any possible way of accomplishing what is required?

Sorry $query->getQuery('criteriaonly'); seems buggy. Have to fix but can't give any ETA.

I need that too. It can be very useful, this way we can reuse a criteria for different tasks such as deleting,selecting or updating.
Can't you just speed this one up? I think it would be very simple I am building a framework on top of pixie and without this feature I can't continue.

I use something like this:
$statements = $query->getStatements();
$wheres = $statements['wheres'];
This gives me an array of the where conditions of the query.


Many threads creation by SyncIdleConnectionMonitorThread

I have to work with a large number of Unirest instances and I had a problem of creating new threads.
The problem is the kong.unirest.apache.SyncIdleConnectionMonitorThread class.
For each instance of Uniest, a new thread of this class is created.
And in my case, jvm crashes after creating ~ 16kk threads.
To fix this problem, I think need to use thread pools.
For example, Executors.newCachedThreadPool ().
Of course I close all instances.
At the same time, usually no more than 1k instances work.
I understand that problem is quite specific :)
=>0x00000000819cf000 JavaThread "Thread-16135980" daemon [_thread_in_vm, id=636100, stack(0x000000007f0d0000,0x000000007f1d0000)]
Register to memory mapping:

RAX=0x000000007f1cc2c0 is pointing into the stack for thread: 0x00000000819cf000
RBX=0x0000000000000004 is an unknown value
RCX=0x0000000000003000 is an unknown value
RDX=0x0000000000000006 is an unknown value
RSP=0x000000007f1cf2c0 is pointing into the stack for thread: 0x00000000819cf000
RBP=0x0000000000000000 is an unknown value
RSI=0x000000007f1cf2c0 is pointing into the stack for thread: 0x00000000819cf000
RDI=0x0000000000000004 is an unknown value
R8 =0x0000000000000006 is an unknown value
R9 =0x000000003b702f50 is an unknown value
R10={method} {0x000000004d7eccf8} 'run' '()V' in 'kong/unirest/apache/SyncIdleConnectionMonitorThread'
R11=0x000000007f0da170 is pointing into the stack for thread: 0x00000000819cf000
R12=0x000000003b702f50 is an unknown value
R13=0x00000000819cf000 is a thread
R14=0x000000007f1cf388 is pointing into the stack for thread: 0x00000000819cf000
R15={method} {0x000000004d7eccf8} 'run' '()V' in 'kong/unirest/apache/SyncIdleConnectionMonitorThread'

Stack: [0x000000007f0d0000,0x000000007f1d0000],  sp=0x000000007f1cf2c0,  free space=1020k
Native frames: (J=compiled Java code, j=interpreted, Vv=VM code, C=native code)

Why do  you have so many configured instances of Unirest? It's designed such that you should have relatively few in  a particular JVM. What is stopping you from re-using them?

Why do you have so many configured instances of Unirest? It's designed such that you should have relatively few in a particular JVM. What is stopping you from re-using them?

At a minimum, I need to clear cookies and I don’t know how to do this.
But also I have no idea what each instance stores in itself, maybe it accumulates some information.

You can disable cookie tracking with Unirest.config().enableCookieManagement(false); the rest of  what's "tracked" is just security stuff. I couple possible enable not even  having the thread, but  that might have other weird side effects.

You can disable cookie tracking with Unirest.config().enableCookieManagement(false);

I need cookie management. But I need to clear the cookies. Now I am doing this by creating new instances. And there is no other way, as it seemed to me.

the rest of what's "tracked" is just security stuff.

We saw that there was a thread at number 16kk. About 30 requests are executed for one thread.
And if at least a little data is collect there, it seems to me that this is just a memory leak (in my case).

I couple possible enable not even having the thread, but that might have other weird side effects.

I did not understand this sentence.
Do you mean that you have a way to create instances without a monitor ? If so, this is most likely my option.

You can certainly populate Unirest with your own configured instance of Apache Http Client that doesn't use a threadmonitor and has whatever you need as far as cookies go:
https://github.com/Kong/unirest-java/blob/master/unirest/src/test/java/BehaviorTests/CustomClientTest.java#L57-L81

You can certainly populate Unirest with your own configured instance of Apache Http Client that doesn't use a threadmonitor and has whatever you need as far as cookies go:
https://github.com/Kong/unirest-java/blob/master/unirest/src/test/java/BehaviorTests/CustomClientTest.java#L57-L81

At first I thought: “What can I change if the client is created immediately, when building the config”.
And then I checked - a very interesting solution, building a client on demand.
Well then you can consider the problem settled. Thanks for the help.


"Recommend to all" tagged topics not showing up

After I answer the questionnaire, I expect to see 1.1 Topic: "Manitoba Start & Settlement agencies" and 1.2 Topic "General information" first as they are tagged as "recommendToAll" topics. Instead, they do not show up.
Acceptance Criteria

[ ]  aforementioned topics appear no matter what answers are selected on the questionnaire

Steps for QA:

Fill out the questionnaire
Look for aforementioned topics

now showing up! yay


React error tab

https://www.figma.com/file/MxA95ZHK5NqGX4ptFQljRJ/⭐-Design-System-3.0?type=design&node-id=4193-103791&mode=design&t=gmTvUCAiuOPmhgMi-4

Task scope updates:

Errors tab is not visible in the Home scope
Pagination in the error list view is outside of this task
Environment list will not be visible on the card
Affected services will not be visible on the card
Sceletons will not be visible Related https://github.com/digma-ai/digma-ui/pull/845


0.28.4 release

Bugfix release: https://github.com/describo/crate-builder-component/releases/tag/v0.28.4

Done https://www.npmjs.com/package/@describo/crate-builder-component-react


[23.11] presenterm: 0.2.1 -> 0.4.1; enable sixel support

Description of changes
Backports: #269697, #272441, #274748, and #276073
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thank you @NickCao (:


Cannot call value of non-function type 'NSHTTPURLResponse?'

I'm getting this error when trying to compile after migrating from 2.3 to swift 3.
This is pointing to Request+AlamofireImage.swift

Any idea why this is happening? It looks like I got the latest code from AlamoFire (version 4) and AlamofireImage (version 3)

Also happening here when trying to compile after migrating from 2.3 to swift 3.

Duplicated issue #186

Make sure you installed using the latest CocoaPods or Carthage.


Explore Collaboration Opportunities with Marimba Repository

We’ve noticed that the Marimba repository shares some overlapping features with our project. I believe this presents a valuable opportunity for collaboration. It would be great to connect with the Marimba team to discuss how we can support each other and potentially work together.

Create a Marimba and Paidiverpy Integration Example


IBM Dynamical Decoupling Tutorial

IBM Dynamical decoupling tutorial that builds on the introductory material in #464

Nice, lgtm!


Feature/migrations and database

Learning objectives

Understand what ORM is.
Use database migration files to maintain database schema.

Hi @Thi-Ponce ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


download speed

is an option available to show the current download speed? all I see currently is the elapsed time and a progress bar.

Not currently, but it should be easy to add in the progress bar area.

yeah that is must needed. currently I am using FFmpeg or YT-DLP which both have it.

Implemented in v0.2.10.


TimeDistributed model compatibility

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 and Ubuntu 18.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.10.0a20220604 (nightly)
Python version: 3.8.10

Describe the problem.
Just came across something rather strange. Seems like some architectures in keras.applications does not work directly with TimeDistributed.
For an example, you have the three architectures MobileNetV2, *V3, and a ConvNeXt-S architecture. If you use MobileNetV2 it works, but for the two others it does not.
AFAIK, eager mode is enabled by default, so I don't see why this fails here? I also have not compiled the model yet or run it, where I would then do run_eagerly=True, if necessary.
Describe the current behavior.
Fails for some models. MobileNetV3 also fails in TF 2.8.0, so it has apparently been an issue a while.
Describe the expected behavior.
Should expect the same behaviour for all model applications in this scenario.
Potential solution.
I think this has to do with how the output_shape is dynamically fetched, which may not work as intended for some layers? Perhaps layers like TFOpLambda and LayerScale require that the compute_output_shape methods are implemented to work as expected?
Standalone code to reproduce the issue.
Works for MobileNetV2 and not for the two other.
import keras
from keras.applications import MobileNetV2, MobileNetV3Small, ConvNeXtSmall

input_ = keras.layers.Input(shape=(8, 224, 224, 3))

# base_model  = MobileNetV2(include_top=True, input_shape=(224, 224, 3))
base_model = MobileNetV3Small(include_top=True, input_shape=(224, 224, 3))
# base_model = ConvNeXtSmall(include_top=True, input_shape=(224, 224, 3))

output = keras.layers.TimeDistributed(base_model)(input_)
model = keras.Model(inputs=input_, outputs=output)

Source code / logs.
Error prompt for MobileNetV3:
Traceback (most recent call last):
File “.\test_timedistributed.py”, line 12, in
output = keras.layers.TimeDistributed(base_model)(input_)
File “C:\Users\47955\workspace\sandbox\venv\lib\site-packages\keras\utils\traceback_utils.py”, line 70, in error_handler
raise e.with_traceback(filtered_tb) from None
File “C:\Users\47955\workspace\sandbox\venv\lib\site-packages\keras\engine\base_layer.py”, line 879, in compute_output_shape
raise NotImplementedError(
NotImplementedError: Exception encountered when calling layer “time_distributed” (type TimeDistributed).

Please run in eager mode or implement the compute_output_shape method on your layer (TFOpLambda).

Call arguments received by layer “time_distributed” (type TimeDistributed):
• inputs=tf.Tensor(shape=(None, 8, 224, 224, 3), dtype=float32)
• training=False
• mask=None

and for ConvNeXtSmall:
Traceback (most recent call last):
File “.\test_timedistributed.py”, line 10, in
output = keras.layers.TimeDistributed(base_model)(input_)
File “C:\Users\47955\workspace\sandbox\venv\lib\site-packages\keras\utils\traceback_utils.py”, line 70, in error_handler
raise e.with_traceback(filtered_tb) from None
File “C:\Users\47955\workspace\sandbox\venv\lib\site-packages\keras\engine\base_layer.py”, line 879, in compute_output_shape
raise NotImplementedError(
NotImplementedError: Exception encountered when calling layer “time_distributed” (type TimeDistributed).

Please run in eager mode or implement the compute_output_shape method on your layer (LayerScale).

Call arguments received by layer “time_distributed” (type TimeDistributed):
• inputs=tf.Tensor(shape=(None, 8, 224, 224, 3), dtype=float32)
• training=None
• mask=None`

I m currently seeing this error again in 2.16.1

Hi @amadou1998 , I ran the code from the initial issue report with 2.16.1 with both Keras 3 and tf-keras (Keras 2) and I did not get any error message. Can you provide specific repro steps?

`import tensorflow as tf
from tensorflow.keras.layers import Input, Embedding, LSTM, Concatenate, Dense, TimeDistributed
from tensorflow.keras.models import Model
tf.version, tf.keras.version
('2.16.1', '3.3.3')
input_icd = Input(shape=(10,))
embedding_icd = Embedding(input_dim = 100+1, output_dim = 16, mask_zero=False)(input_icd)
lstm_icd1 = LSTM(units=32, return_sequences=True)(embedding_icd)
lstm_icd2 = LSTM(units=32, return_sequences=False)(lstm_icd1)
model = Model(inputs=input_icd, outputs=lstm_icd2)
input_full = Input(shape=(10,10))
td = TimeDistributed(model)(input_full)
model2 = Model(inputs=input_full,outputs=td)`
You do this, to get the error.
In my case, tf = 2.16.1 and tf.keras = 3.3.3

@vikkubaligas ,
Please upgrade to tf 2.17.0 and keras 3.5.0. The problem is resolved with those versions.
Thanks.


Remove SCP option in favor of a 'finished' action.

The SCP option can be generalized by just allowing a script to run after the conversion was successful.
The user can make their own script that does scp and even much more.
Come to think of it, things like notifications could also be done this way.

PostCmd can already do this.
We should improve it so stdout and stderr are visible to the user.

Done in this commit


How do I change the error message?

Please tell me how to display an error message inside the  field, as shown in the example?

	  Name
	  Error message:

Thank you
Test case: https://jsfiddle.net/cferdinandi/yhqepa3x/

I'm unable to provide support for custom modifications to the core code like this right now. Sorry!


lxml twitterscraper

INFO: {'User-Agent': 'Mozilla/5.0 (Windows; U; Windows NT 6.1; x64; fr; rv:<IP_ADDRESS>) Gecko/20101203 Firebird/3.6.13'}
Traceback (most recent call last):
File "/home/mist/.local/bin/twitterscraper", line 5, in
from twitterscraper.main import main
File "/home/mist/.local/lib/python3.7/site-packages/twitterscraper/init.py", line 13, in
from twitterscraper.query import query_tweets
File "/home/mist/.local/lib/python3.7/site-packages/twitterscraper/query.py", line 73, in
proxies = get_proxies()
File "/home/mist/.local/lib/python3.7/site-packages/twitterscraper/query.py", line 44, in get_proxies
soup = BeautifulSoup(response.text, 'lxml')
File "/usr/lib/python3/dist-packages/bs4/init.py", line 156, in init
% ",".join(features))
bs4.FeatureNotFound: Couldn't find a tree builder with the features you requested: lxml. Do you need to install a parser library?

When I use

pip install lxml

Result is

Requirement already satisfied: lxml in /usr/lib/python3/dist-packages (3.5.0)

Found that problem, what should I do? Thankyou

I am also having this specific issue

@NicholasDow you may have multiple conflicting python installations. I recommend

trying this fix https://stackoverflow.com/a/52259845

trying to create a virtualenv and install twitterscraper in there


optimizer: void always-false filter's upstream

Is your feature request related to a problem? Please describe.
Although LogicalScan with always-false predicates has been specifically optimized, another case where the downstream is not a scan but e.g. a table function ( select * from generate_series(1,<PHONE_NUMBER>, 1) where 1= 0;) shall also be optimized such that no real physical scan being executed.
Describe the solution you'd like
return empty value immediately
Describe alternatives you've considered
No response
Additional context
No response

Supported by https://github.com/risingwavelabs/risingwave/pull/8629.


Include root hidden files

I'm trying to deploy a wordpress project with capistrano and capistrano/copy . Wordpress contains the .htaccess file in the root project directory but capistrano is not deploying it. Any way to deploy this file properly?

Same issue for me. Any update on this?

It works by naming explicitly the file in the :include_dir directive.
set :include_dir, [".", ".htaccess"]


"Parse" the whole init config left-to-right

This will let us incrementally create initialization tree later.
The tree-descending code can then be used to work with sample
parameters.
The use of field name as a map key is regrettable. This can be fixed
later.
The RHS being arbitrary string is also regrettable. We'll migrate
slowly.

cc @beccca

@vchudnov-g PTAL

Codecov Report

Merging #2172 into master will increase coverage by 0.02%.
The diff coverage is 86.27%.

@@             Coverage Diff              @@
##             master    #2172      +/-   ##
============================================
+ Coverage     86.78%   86.81%   +0.02%
- Complexity     5116     5117       +1
============================================
  Files           454      454
  Lines         20282    20289       +7
  Branches       2176     2174       -2
============================================
+ Hits          17602    17614      +12
+ Misses         1908     1905       -3
+ Partials        772      770       -2

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com/google/api/codegen/util/Scanner.java
85.36% <100%> (+0.18%)
32 <1> (+1)
:arrow_up:

...gle/api/codegen/transformer/go/GoSurfaceNamer.java
91.03% <100%> (+0.18%)
59 <1> (ø)
:arrow_down:

...m/google/api/codegen/metacode/InitFieldConfig.java
100% <100%> (+20.68%)
11 <1> (-5)
:arrow_down:

...gle/api/codegen/metacode/FieldStructureParser.java
88.09% <84.09%> (+3.34%)
21 <14> (+5)
:arrow_up:

Continue to review full report at Codecov.

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


Loss function and method for importance

For LOCO, CPI and PI, the classes have a loss function as a parameter to evaluate the importance of the variables.
The various loss functions must be used with a specific estimator “method” (“predict”, “predict_proba”, “decision_function” and “transform”).
For the time being, the user needs to know which loss functions are associated with which “method”.
We have two options:

Stay as we are and add documentation to indicate which method is to be used with which loss functions.
Remove the possibility for users to choose any loss function and provide a list of loss functions with their associated methods.

This question remains open.

This question will evolve with the integration of p-value methods #48 .
The definition of p-value is connected to the definition of the loss function and the estimator "method".

I would accept all estimators in sklearn ; (Do they need to have a predict_proba method ?)
And document (and test ! ) choices that are suitable.
People may want to use other estimators, but it is at their own risk.


[GOBBLIN-1226] remove lombok from runtime classpath

JIRA
- https://issues.apache.org/jira/browse/GOBBLIN-1226

Description
Lombok is a cool tool (although IMHO not worth it's troubles, considering generated code is harder to debug, incompatibilities in bytecode, line version mismatch in debugger, etc, etc) which promises to have zero runtime dependencies.
It is also strongly advised by its developers that it should be present on the classpath only during compilation/code generation and not at all, during runtime. (I don't have an exact reference at hand but it can be searched in the lombok google forum).
Lombok in runtime's classpath is considered not only a bad practice but wrong usage of lombok.
The issue it caused me was that it brings an older version of guava into path.
I propose the compile ...lombok in build.gradle be changed to compileOnly. This has zero effects on any current project using gobblin.

+1 LGTM,
@hkoosha , It would be also good to place this dependencies at parent level, as a core feature available for all modules in Gobblin.
@sv2000 can u pls help review this.

@jhsenjaliya where would it go? in globalDependencies.gradle?

@hkoosha you r right.  but if you want to do it. we can probably add dependencies section in globalDependencies.gradle that should apply/add dependecies for all modules.

updated :heavy_check_mark:

+1, LGTM
@sv2000 , @shirshanka , can u pls take a look at this.

@hkoosha Thanks for this valuable contribution. Hope, this triggers a more comprehensive discussion about recommended usage of Lombok. To that end, it would be great if you can cite references/enumerate specific issues associated with usage of Lombok as a runtime dependency. This should be educational for the wider Gobblin developer community.

@autumnust @htran1 What are your thoughts/opinions on removing Lombok as a runtime dependency?

Well, one problem could be that lombok may liberally use any dependency as they wish, no matter how outdated. Then including lombok in Gobblin's runtime means needlessly pulling those transitive dependencies into classpath.
Why needlessly? because one of lombok promises is: zero runtime dependency. No code path within lombok is accessed during runtime. No other third-party library is used within lombok's generated code either. The generated code is just pure vanilla java.
All those lombok annotations we add to the code, and lombok.* package references are all removed during AST transformation.
So why include it's jar and its transitive dependencies at all?

+1 on this PR. @hkoosha thanks.
@sv2000 I think that makes sense.  Reference:
https://projectlombok.org/mavenrepo/

+1. Thanks @hkoosha @jhsenjaliya for this PR!


Fixed #31306 -- Raise ValidationError only on first failure validator.

Changed fields validators model validation error occurs when first validator failed.

Closing per ticket.


RUF008/9 for non-dataclasses as well

Closes #4053.
This is my first PR and I didn't know if discussion in #4053 ended with a tacit approval of this change or not, so I didn't put this up fully polished just yet. I'm hoping for early feedback since this is a pretty simple PR, but happy to go back and polish first if that's preferred.
Makes new rules for non-dataclasses, which might lead to some confusion since there is such high overlap. Still, I didn't want to break backwards compatibility, even for very new rules.

I don't know what's going on with the test failure. It doesn't fail locally on my mac. Did I except some constant number of rules or something?

Closing in favor of #4390.


Use sed to find-replace PORT envvar in nginx conf

Nginx does not allow using environment variables in server blocks[1], so instead we use sed to find instances of the template variable {PORT} and replace with the value of the envvar.
We do this at runtime rather than at image build time, as the PORT envvar is usually set by the Cloud Foundry runtime[2].

Yup this'll do it. Point of note though - until a month or so ago, before I fiddled with this, we had been completely fine just setting it as 80. And it does look like we'll never be able to use $PORT on the APIs becuase of the simultaneous-80-and-8080 thing and the need to still serve the metrics on 80...

I've now tested this using buyer-frontend on preview


[Bug]

Before Reporting

[X] I found no existing issues matching my bug
[X] My issue is not caused by a plugin or theme

Describe the Bug
It seems that some latest updates on discord canary channel completely broke up better discord - after installing better discord over the latest discord canary it became unusable, lacking all the better discord features and I had to re-login to the already logged discord. After uninstalling better discord the vanilla discord canary becomes normal as it should be without any issues, but if I'm trying to reinstall better discord the described bug happens again and again. Please check it up and update core modules if possible.
Thanks!
Expected Behavior
Better discord should be working without the issues that I've described above
Steps To Reproduce
Install better discord 1.9.7 over any latest discord canary version
Operating System
Windows
Discord Version
Canary
BetterDiscord Version
261973 (954e1a5) (discord canary's version) better discord 1.9.7
Anything else?
No response

Canary is on a newer version of Electron at this point in time. BD does not yet account for this and misfires some things. This is a known issue that has been discussed within the Discord server already.
Wait for the BetterDiscord team to add compatibility for the new electron version and try again. Use Discord stable for the time being instead.

Canary is on a newer version of Electron at this point in time. BD does not yet account for this and misfires some things. This is a known issue that has been discussed within the Discord server already. Wait for the BetterDiscord team to add compatibility for the new electron version and try again. Use Discord stable for the time being instead.

Thanks for the quick response. I'll switch to the stable version and stick with it until the BD is updated with a newer version.


Add language server functionality, implement "Jump to definition"

You may want to add ESLint like this, (I just did it yesterday 😅)
It will help clean the code :)
npm install --save-dev lint-staged husky eslint @typescript-eslint/parser @typescript-eslint/eslint-plugin eslint-config-prettier eslint-plugin-prettier

.eslintrc.js
module.exports = {
    root: true,
    parser: '@typescript-eslint/parser',
    plugins: ['@typescript-eslint'],
    env: {
        node: true,
        es6: true,
    },
    extends: [
        'eslint:recommended',
    ],
    parserOptions: {
        ecmaVersion: 2020,
        sourceType: 'module',
        project: 'tsconfig.json',
    },
    overrides: [
        // {
        //   files: ['*.js'],
        //   rules: {},
        // },
        {
            files: ['*.ts'],
            extends: [
                'eslint:recommended',
                'plugin:@typescript-eslint/eslint-recommended',
                'plugin:@typescript-eslint/recommended',
                'plugin:@typescript-eslint/recommended-requiring-type-checking',
                'prettier/@typescript-eslint',
            ],
            rules: {
                // '@typescript-eslint/no-use-before-define': 'off',
                // 'no-useless-escape': 'off',
            },
        },
    ],
}

package.json
  "husky": {
    "hooks": {
      "pre-commit": "lint-staged"
    }
  },
  "lint-staged": {
    "*.{js,ts,css,json,md}": [
      "prettier --write",
      "git add"
    ],
    "*.{js,ts}": [
      "eslint"
    ]
  }

I have added some E2E tests for jumpt to def as well, do you have any CI to integrate them into?

do you have any CI to integrate them into?

We usually use GitHub Actions as it is just available here and works pretty well.
Adding that might be better handled in a separate PR though, which means you possibly also want to split out the tests themselves - your call.

do you have any CI to integrate them into?

We usually use GitHub Actions as it is just available here and works pretty well.
Adding that might be better handled in a separate PR though, which means you possibly also want to split out the tests themselves - your call.

Ok good. I'll handle all things regarding automated testing in a separate branch and remove those from this PR then.


Fixing bucket alignment for group by

This addresses issue #2005
It makes sure the buckets land on natural boundaries.  If the minimum time specified falls in the middle of a bucket, it returns bucket, but only data after the minimum time.

Thanks @jnutzmann -- the test suite failed however. Can you dig into that? You can find the test output on the "Details" link above, and you can also run the tests manually too, of course.

Yup.  Already have a fix that I am working on.  Seems like there are two issues.  First in how it calculates the number of points to return and also in the expected result (should return a bucket with null).  I'll update my branch shortly.

Great.
You must also sign the CLA, if you have not already done so.
@pauldix

I signed the CLA this morning.
As far as the change to the test goes, I believe the previous expected result was incorrect, but I want to make sure you agree.  The previous result placed the the point occurring at "2000-01-01T00:00:10Z" in the "2000-01-01T00:00:00Z" bucket when grouped by 10s.  This should rather be in the "2000-01-01T00:00:10Z" bucket.  We get null for the "2000-01-01T00:00:00Z" bucket, since the previous point occurs at "2000-01-01T00:00:00Z" and we limit to "2000-01-01T00:00:05Z" in the query.
Seem correct?

@jnutzmann I'm check on this now. Your logic makes sense - I'll verify and let you know.

Unit test changes look good to me -- thanks @jnutzmann. +1 on this change.


Notification Player UI

Hello,
I have used this package to handles videos and sounds on my app, it works very well, but is there a way to make a Notification Player UI like Spotify or else?

Hi @anastely, closing as this is currently not supported by react-native-video. You may want to look at https://github.com/tanguyantoine/react-native-music-control which works well with react-native-video.


Disable name violation on converters

Disable name violation on FloatConverter and DoubleConverter

I figured it'd be better to actually fix the violation, not just suppress the suggestion.


Reaching out!

Hey Yamabuki, my name's Jamie. I'm reaching out because I've seen you've been forking a lot in your free time with today's (mostly diffusion-based) biology models! I'd love to talk to you about your experience using them and what you're doing with them.
I've tried looking for another way to contact you, but I can't find any. Could you send me an email address or a LinkedIn or Discord profile or somewhere else I can reach you? :-)

I'm super-busy now and I may not be able to respond if you offer me something.
Sorry for the inconvenience.


Only download Mathematica/Wolfram Engine once per workflow

The problem
Due to tests parallelization #578, we are now downloading the docker image with Mathematica 4 times, which is redundant. We should figure out how to only download it once thus saving 3 minutes' worth of credits. If we download it prior to starting the testing job, we will also reduce the workflow duration by 1 minute.
Possible solution

Take WL out of the Docker image and put it into the Workspace for all nodes to use (tricky because it needs to stay activated).
We can then add another wolfram-engine-download job which is going to put it into the workspace.
The build/test jobs will then just need to put symbolic links/copy relevant files which should be much faster.

Alternative solutions
Circle CI has docker layer caching which might be useful here, however, it is not available on the free plan, and even if it was, it's quite expensive costing more credits than the entire WL testing job.

Documentation for attach_workspace states that:

The full contents of the workspace are downloaded and copied into the directory the workspace is being attached at.

Therefore, if we persist the wolfram installation and attach on our four parallel jobs, it will download the wolfram installation on each one. DLC looks like it could be promising though.

Documentation for attach_workspace states that:

The full contents of the workspace are downloaded and copied into the directory the workspace is being attached at.

Therefore, if we persist the wolfram installation and attach on our four parallel jobs, it will download the wolfram installation on each one. DLC looks like it could be promising though.

We should even be able to cache it between builds similar to how you cache Python modules.

We should even be able to cache it between builds similar to how you cache Python modules.


Fix issue #853 add warning for @all / @everyone

The warning contains a count of the number of people in the stream.
An error appears if the warning is ignored and the user tries to send the message anyway.
The message cannot be sent until the warning is acknowledged.
This only applies to stream messages and not private messages.

@timabbott This is a new branch after rebasing and squashing the changes from https://github.com/zulip/zulip/pull/904 . I wasn't able to push after rebasing the other branch. I can remove it after this is approved since it has some comments and history that might be relevant.

Cool!  I'll take a look soon.  Just FYI, you could have updated #904 by using a git force-push, via either git push -f or git push origin +localbranch:remotebranch, and that's how one usually updates an open PR after rebasing the relevant branch.

Awesome, this is looking really good @knowcliu!  I posted a few small issues on the code above, and also I noticed one bug, which is that if you remove the @-all from the message, it still won't let you send:

I think we discussed this and that the best solution was probably to just add a check in validate_stream_message for whether the word @all or @everyone still appears in the message, and if not, let the message through.  Does that sound right to you @knowcliu?

While debugging the case for removing the error when @all is removed, I noticed that @all and @all are not the same. That is, I need to also handle when the user does not select @all from the typeahead but instead hits escape. They won't be warned.

Ahh! all and all

Ahh, yeah, that makes sense.  I would also be fine with the approach of removing support for @all and only supporting @**all**...

This should fix the funny logic when the user removes @all they are allowed to send the message.
This also improves the regex to include both @**all** and @all.
Still todo:

refactor test code
localization / internationalization of strings
accessibility of tabbing to the acknowledgement button

Ok, I think this is ready to go.

Awesome, this looks great, merged!  I'm super excited, this is a feature we've wanted for a long time.
I think there's still an open question whether we should limit the warning to only large streams; I imagine we can do that iteratively based on feedback from users.
Thanks @knowcliu for building this!


Migrate e2e tests to Playwright

We decided after #3035 to migrate the end-to-end tests from Cypress to Playwright.
From that issue:

Cypress has several limitations relating to things like keyboard navigation which is preventing us from writing comprehensive and realistic tests.

Previous spike: https://github.com/defencedigital/mod-uk-design-system/pull/3037


there are some lint issue i faced when i tried to update logo

Signed-off-by: gunjanmimo<EMAIL_ADDRESS>

@leecalcote sir, I made all those changes which are required to solve those error


Changed highlight color and border radii of nav items

Main Nav Item - Gray
Secondary Nav Items - Light Gray
Secondary Nav Items (Highlighted) - Light Green

Fixes #5

what a king 👑


Ecs provider fix

Description

Fixes #2055

The ECS 'describe tasks' SDK call now specifies the cluster explicitly.
ECS monitoring poll interval is now correctly converted into seconds.

Purpose
This pull request is a:

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update documentation

rocWMMA  documentation update as per comments in [https://github.com/ROCmSoftwarePlatform/rocWMMA/pull/248/files]

There is some inconsistency here as to whether the copyright contains the (c) symbol:
Copyright (c) 2021-2024

vs
Copyright 2021-2024

@mkarunan  Can you please check with Roopa for which is preferred? (Or if it matters?)


fix: composer distribution file exclusions

sorted the file alphabetically
fixed phpunit.dist.xml reference, file is actually called phpunit.xml.dist
fixed package-lock.json reference - missing export-ignore
added new exclusions for phpmd.xml

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


Changing Fullscreen mode does not change Fullscreen mode

Changing the option and hitting apply or okay or both does not change the mode.
(Looking in the Config File, no change is made)

I just noticed that it made the change after the launcher timed out and closed. Maybe we need to make it able to write out the changes before close?

When the game itself launches seems the best time?

Well, when the start button is pressed, ye.

Why don't make it save after hitting OK or Apply?

Hmm that's a bug from my code. The new configuration is supposed to be written to a file before the application gets started. I somehow missed it to add. I don't have access to a computer atm so if someone wants fix this it's the same code as line 66-69 in Launcher.vb but instead of LauncherConfig there should be OpenRCT2Config. Just add it after Line 69 and it should work.


Resized Version of EK100

Hi,
Thank you for this amazing work!
I saw "(Recommended) The raw videos are huge (~1 TB). As an alternative, please check out a resized version" in DATASET readme, but the link to resized version seems to be pointing back to the readme file. I wonder if there is a specific download link to get "video_ht256px".
Thanks again!

Hi @SJTUwxz ,
Thanks for spotting this! Please check out the latest commit for the fix.
Best,
Yue

Hi @SJTUwxz ,
Thanks for spotting this! Please check out the latest commit for the fix.
Best, Yue

I wanna know ， Is the shortest side 256 p? thx

Hi @SJTUwxz ,
Thanks for spotting this! Please check out the latest commit for the fix.
Best, Yue

I wanna know ， Is the shortest side 256 p? thx

@wolfworld6  Yes it is.


Add support for different camera urls

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
Flashforge has pushed out a new firmware for the 5M/5MPro and the camera now streams on http://{IP}:8080/?action=stream instead of http://{IP}:8080/stream_simple.html
Describe the solution you'd like
Since it is firmware dependent, you can't just base it on whether it's a 5M/Pro or not so, can you have it be either a text box for entering the camera url or have radio button for which it should be?
Describe alternatives you've considered
I could fork the code and do it but I figured it would make sense keep it in one place instead.
Additional context
Let me know what you think.

Did you mean that in the new firmware the url is http://{IP}:8080/stream_simple.html  ??
I have a AD4 and can't confirm. Right now the /?action=stream url is used.

The 5M was streaming the camera at shttp://{IP}:8080/stream_simple.html and the plugin was working. They pushed new firmware to the 5M series which moves the camera to the what I guess is the same URL as the AD4 now...let me try removing and re-adding the printer in the plugin to see if it gets the camera URL right.

Since I still had the code from the last time I was tinkering with this I looked...that's really weird that it was working...the camera was not exposed on the action=stream url in the past so the fact that it was working makes no sense unless it was and just wasn't told to anyone that it was actually working that way. I guess there's something else going on with this, and you don't need to deal with this now, so I'll just close this.


View isn't sync in combined usage of mixin & provide & render fn & computed (no explicit reason)

Version
3.0.4
Reproduction link
https://codesandbox.io/s/priceless-matsumoto-7i99q?file=/src/index.vue
Steps to reproduce
follow the link
What is expected?
The theme in theme-consumer should equal to parent's and child's.
View doesn't sync

What is actually happening?
After the first correct mutation, the view doesn't sync anymore.

Some factors effect the demo.
If you

don't pass anything to child's slot, the behavior will be correct
remove the mixin inside child, the behavior will be correct
write parent in .vue sfc, the behavior will be correct
...

It works in 3.0.2, fails after it.

It's really complicated too.like #2768

It's really complicated too.like #2768

Yes, I think it may be caused by the same reason (due to the version in which it fails). However I have no time for debugging it😂. Hope it can be fixed soon.

Same as #2768, like @edison1105  said, it caused by af95604 .


Only Strings are supported

Current;y, only String events are supported due to
store(new String(delivery.getBody))  // in RabbitMQInputDStream.scala
it's better to provide generic ByteArray wrapper to give end user a chance to identify object type.
Because of, non string objects can't be transferred now

Thanks, we will update the receiver asap.

Here's a link to a gist with a possible implementation:
https://gist.github.com/koen-dejonghe/940e0d5f3fc6c2b2d39c
It uses a separate message consumer (extension of DefaultConsumer), which may be possible way of solving.

Thank you very much Koen. We will take a look to your code

Hello, has a byteArray reader for RabbitMQ for Spark been added to the code base yet by any chance? Thanks.

Hi,
this issue is solved in the PR #41 with generic type, additionally are been implemented the DStream creation with String and Array[Byte] types.
Thanks for all.


MM-17042 Add support for using metadata in cypress it function

Summary
Adds support for using the 2nd argument of it as an object of metadata.
You can add any additional data to this metadata object. The only thing that this current code will use is the tags property, and if set as an environment variable it will only run those tagged tests.
This sets us up thought to add test Id's to the tests, and use them to report to a test case management tool.
Overwrites the global it function with our custom one, but supports keeping the normal 2nd arg of it as the test function to run.
Example:
it('this is my test', { testId: 'MM-123', tags: ['foo', 'bar'] }, () => {
    // test code
})

Ticket Link
https://mattermost.atlassian.net/browse/MM-17042

I did some test on this and found some issues:

error when excluding/including test with it.skip/it.only
with npm run cypress:run, testing specific tag does not work

set tags in cypress.json as "env": {"tags": ["foo"]}
add metadata tags as it('test name', {tags: ['foo']}, testFn)
initiate npm run cypress:run and observe

with npm run cypress:open, it with other tag gets an error of actualTestFn is not a function

set tags in cypress.json as "env": {"tags": ["foo"]}
add different metadata tags to  to two it --> one as foo and other as bar
initiate npm run cypress:open, test the spec file and observe

if I add tags at cypress.json and set some it with metadata same tags, initiating npm run cypress:run still run everything where I expect to runset tags only.

Other items that I think should consider:

CYPRESS_tags=["foo"] npm cypress:run should work the same as expected.
Maybe let's make tags env to be more forgiving by allowing string so we can do CYPRESS_tags="foo", CYPRESS_tags="foo,bar", or {"env":{"tags":"foo,bar"}} at cypress.json
metadataIt will be critical to our testing.  I suggest to add unit test so this is protected from future change.

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

Not needed anymore. We will handle tagging via directories, and metadata can be added via
it('my test', function() {
    this.test.metadata = 'foo'
    // test code...
})


Add docs for additional aggregate pushdown functions for sql server

Docs for #5299

Merged, although  --> https://github.com/prestosql/presto/pull/5299#issuecomment-699125000


Scope and time scheduling of the meeting (issue #326 related)

Aim of the issue is to schedule a meeting.

Scope proposed: Presentation of latest development achievements in modules: reporting and solvers.
Date and time: 07.12.23 at 19:45

𝓓𝓸𝓷𝓮


Can't move task for other projects with lower ID

Hello,
I'm having trouble to move some tasks to another project: I have two projects, one with ID 1 and another with ID 2. I can move tasks from project ID 1 to project ID 2, but there is no way I can move from project ID 2 to project ID 1 - none exception is throw, but there is no projects to move to when I click in "move to another project".
Thanks in advance and thanks for the good work ;)

I found out what it was: the table 'project_has_users' has no entry for the project ID 1, so what I did to fix it is manually add something and voala, I can now move tasks to project ID 1 :)

The dropdown on the page "move to another project" show projects where you are member. You have to be member of the project event if you are admin.


Puppet 8x support

Hi,
I have tested the module on puppet 8x and we have there errors :
Error: Facter: Error while resolving custom fact fact='local_groups', resolution='<anonymous>': undefined method exists?' for File:Class Error: Facter: Error while resolving custom fact fact='local_users', resolution='<anonymous>': undefined method exists?' for File:Class
Is it possible to fix it with a new Tag ?
File.exists? => File.exist?
Thanks,

Thank you for your report. It appears this is tied to your underlying version of Ruby, as this method was renamed and removed in 2.3.0. Would you please share your OS as well?

Hi,
Yes that's fine.
I tested it on CentOS 7 but after I plan to reinstall the platform maybe on Ubuntu 22.04.
Thanks.

It looks like we could make a modest attempt by replacing occurrences of File.exists? with File.exist? as a first step.

If this works for you, I am tempted to close this as done. We can have a separate enhancement request to support Puppet 8.x with a Splunk idea. I have gone ahead and filed it for you: https://ideas.splunk.com/ideas/SFXIMMID-I-485
We can continue the conversation over there and I will close this ticket. Thanks!

Hello,
We have this type of error too in Puppet 8x :

Unknown variable: '::osfamily'.

@flepoutre we're moving to Splunk ideas to prioritize enhancements, would you please upvote and report that it's not working?


Test Time Augmentation for Multiclass images

horizontal flips
vertical flips
weighting

3 aspects to this

what type of augmentation (e.g. flips see above)
what logic to combine augmented outputs? e.g. type of weighting? (equal weighting? weighting proportional to degree of transformation?)
how to preserve raw outputs so any logic (above) can be reversed/changed in post-processing

The last point is another reason for storing model outputs and metadata as npz or similar

test-time aug for multiclass outputs would now need to be implemented for both single-model seg_images_in_folder and ensemble model scripts, with a common do_seg function

there are now two pred scripts ... seg_images_in_folder and ensemble_seg_images_in_folder - they are identical except for the number of models employed, and now use common functions defined in prediction_imports

Sounds good, thanks

[ ] add to do_seg fn in prediction_imports.py
[ ] add new input to do_seg (TESTTIMEAUG)
[ ] put new input is in all calls of do_seg
[ ] if user does not specify TTA, it should be false

[ ] add details to readme

hard voting is not implemented


Saved images has sometimes a line.

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{  D400 }

Firmware Version
(<IP_ADDRESS>)

Operating System & Version
{ Linux (Ubuntu 14/16/17)

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PNVIDIA Jetson

SDK Version

Language
{C }

Segment
{Robot}

Issue Description
<Describe your issue / question / feature request / etc..>

Here is an example image.
I am confused why the line in the image was happened.
I am using a Jetson nano and saving data streaming into a .bag file.
I cannot see the line in the Laptop. It might be Jetson problem or anything I can do?
Thanks.

It does it look like it is just a line across the screen, as the top section of the image above the line does not align with the rest of the image beneath and has different color shading in places.
It appears as though the RGB is "buffering" , like when an internet video stutters or a television has a bad signal reception.  It looks as though the top of the image is a second ahead of the rest of the image, like a video of someone walking down a corridor where the top of the video is showing the position in the corridor that the person is currently at and the rest of the picture shows where they were at a second ago before they reached their current position.
If the image is okay on the laptop then it is probably not the florescent ceiling strip lights affecting the image and it is instead a consequence of recording to a bag with the Nano's lesser hardware capabilities.  You could try reducing the resolution and / or the FPS speed to put less processing demands on the Nano.

Thanks.
Updating SDK and reducing resolution was helpful.
2020년 3월 6일 (금) 오후 4:23, MartyG-RealSense<EMAIL_ADDRESS>작성:

It does it look like it is just a line across the screen, as the top
section of the image above the line does not align with the rest of the
image beneath and has different color shading in places.
It appears as though the RGB is "buffering" , like when an internet video
stutters or a television has a bad signal reception. It looks as though the
top of the image is a second ahead of the rest of the image, like a video
of someone walking down a corridor where the top of the video is showing
the position in the corridor that the person is currently at and the rest
of the picture shows where they were at a second ago before they reached
their current position.
If the image is okay on the laptop then it is probably not the florescent
ceiling strip lights affecting the image and it is instead a consequence of
recording to a bag with the Nano's lesser hardware capabilities. You could
try reducing the resolution and / or the FPS speed to put less processing
demands on the Nano.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/5986?email_source=notifications&email_token=AFY4LCM53ZXX72TJVT24ASLRGFSUTA5CNFSM4LDDAYDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOC4YPA#issuecomment-595971132,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFY4LCIXBSBZCOF2ZKMVPK3RGFSUTANCNFSM4LDDAYDA
.

I'm glad the advice helped.  Thanks for the update!

@hojunson   Do you require further assistance or is it okay to close this case now, please?   Thanks!

No. Thanks
On Thu, Mar 12, 2020, 9:02 AM MartyG-RealSense<EMAIL_ADDRESS>wrote:

@hojunson https://github.com/hojunson Do you require further assistance
or is it okay to close this case now, please? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/5986#issuecomment-598172708,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFY4LCOBRORJYCFIGYRG5ALRHDMO5ANCNFSM4LDDAYDA
.

Ok, thanks so much!


Undo recipe

Hello
I was looking at the Undo recipe shown on the documentation website of redux-saga.
I love the idea, it is plain and simple. However, I started wondering whether there are issues with the provided code.
This is what got me thinking (so, please correct me if I'm wrong, which would not surprise me, since I'm not an expert, yet :)
When the main saga starts, a takeEvery (which is non-blocking) will listen for all ARCHIVE_THREAD actions and immediately continue to the next take(UNDO) action, which is blocking. Now, when the user performs an ’archive thread’, an onArchive is forked. When the user performs an UNDO, the take(UNDO) gets unblocked, the next statement will cancel all forks created by the takeEvery task and the while(true) gets restarted. All is good.
The same applies when the user does not UNDO the archive (this will just keep the takeEvery waiting for the next archive action).
But, what happens when the user triggers two ’archive threads’ in rapid succession? Probably two ’undo UI elements’ will be shown. But when the user now does an UNDO on any of the ’undo UI elements’, both archive requests will be undone. Or, maybe, the UI is smart enough to combine both UI elements into one (this depends on the requirements of the application). Then it would make more sense.
Another issue I see with this solution is when the user triggers an UNDO at the exact time where the code is calling the updateThreadApi method. This method is probably a Fetch request (or similar).
So, the HTTP request has been started, and now the archiveTask gets cancelled, forcing the onArchive to execute the finally block and thus marking the thread as not archived. And this results in an invalid state (the thread is archived on the server, but the client thinks it is not archived)
All of this made me wonder why this Undo recipe was not provided using a race condition? In my personal opinion, using a race would make the code much more understandable. So, I tried to make a solution (small disclaimer: I did not test this code, I've written it in Notes on my phone :-s

function* onArchive(action) {
  const { threadId } = action
  const undoId = `UNDO_ARCHIVE_${threadId}`

  const thread = { id: threadId,archived: true }
  // show undo UI element, and provide a key to communicate
  yield put(actions.showUndo(undoId))
  // optimistically mark the thread as `archived`
  yield put(actions.updateThread(thread))
  // allow user time to active the undo action
  const { undo, timedout } = yield race({
    undo: take(action => action.type === UNDO && action.undoId === undoId),
    timedout: call(delay, 5000)
  })

  // hide undo UI element, in any case
  yield put(actions.hideUndo(undoId))

  if (undo) {
    // revert thread to previous state
    yield put(actions.updateThread({ id: threadId, archived: false }))
  } else if (timedout) {
    // make the API call to apply the changes remotely
    yield call(updateThreadApi, thread)
  }
}

function* main() {
  while (true) {

    const action = take(`ARCHIVE_THREAD`)
    // use spawn to execute non-blocking and prevent cancelation when main saga gets cancelled. this is needed to keep all state in sync (server and client)
    spawn(onArchive, action)
  }
}

What do you think?
Regards
Hans

You are right that this recipe is not ideal and your snippet fixes its issues.
Would you maybe like to make pull request out of it?


feat: added docs and handling of aborting event into basic crawler

Added new event aborting for handling gracefully aborted run from Apify platform.

aborting event should trigger persistState event

Sorry guys, I totally missed the review here. I updated feat under 1.3.0 release. Regarding @metalwarrior665 question, we are emitting events persistState and aborting on backend together, so we do not need to re-call it in SDK again.


Attempting to add a data node to cluster returns: add-data: operation exited with error: open : no such file or directory

URL for relevant page?
https://docs.influxdata.com/enterprise_influxdb/v1.6/install-and-deploy/production_installation/data_node_installation/#join-the-data-nodes-to-the-cluster
What products and version are you using?
Influx Enterprise 1.6.3
Where did you look before opening the issue?
Google, which pointed to some issues in github
I was able to use add-meta commands using the same license-key.

@dbernbeck The license-path or license-key is specified in the config file used for your data nodes. Meta nodes use a separate config file. You likely just need to updatedyour data node config files with either the license-path or license-key, restart the influxd process, then try to add the node(s) again.

Thanks @sanderson I do have a license-key value in place, and the error I receive provides the license-key value instead of '<license-key." (I just didn't add it to this issue in case it is publically accessible.)
I went back to the config file, ensured it has the license key value I was given, stopped and started the data node service.  Same error on the meta node when I try the add-data command

I found the problem. on the meta node I'm using to add the data nodes to the cluster, I had errantly add the key to the license-path parameter. Once I corrected the meta node, the add-data command worked.


1.20.x support?

Are there any plans to update this mod to support 1.20.x or is it safe to assume this mod is abandoned?

Plan for it eventually, but idk when I’m next gonna work on the mod


Consider renaming the lastConnectionState from the CorePublisher state

I'm a little wary of this synthesis as it infers that it provides value to the runtime during operation, but the reality is that you only ever access state and errorInformation on these state change events.
Also, the last prefix is pretty much meaningless given the context of a State encapsulation... by definition this is a snapshot of state so it's implicitly "current" / "currently known".
Originally posted by @QuintinWillison in https://github.com/ably/ablyassettracking-android/pull/252#discussion_r603247958

The corresponding Jira issue is https://ably.atlassian.net/browse/SDK-791


Error when upload extension on production environment

Describe the bug
I'm trying to upload an extension on production environment(https://deonline.bc.dynamics.com) but i received and error when i'm doing that. Error : "Object reference not set to an instance of an object. Page View - Extension Deployment Status has to close". What could be the cause ?
To Reproduce

Go to Extension Management.
Upload Extension.
Deployment Status
Error

Expected behavior
I expected that the extension to be uploaded.

system information error.docx
Versions:

AL Langauge: 3.0.115791
Business Central: 13.3.27437.0-NL

I attached the error from system information in that word document and also the app.json and launch.json files

Let's get the issue to a team who will fix and backport it, as this repository is only for issues related to the latest developer preview environment for Dynamics 365 Business Central. We suggest that you open a support case or file a bug in Collaborate, to ensure that all Business Central users benefit from your catch as soon as possible.
To open a support case, you can:

Open a support request to CSS through the PartnerSource portal
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company

If you file the bug in Collaborate, remember to include steps to reproduce the issue, and the Business Central build number and country version you're using.


Add heading elements to dialog headers to improve accessibility

Closes #998.

Looks like a related e2e failure.