expected most recent version of archive (October) but got April

`(base) katjaseltmann@unreg-dhcp-67 bif % preston head --anchor hash://sha256/37bdd8ddb12df4ee02978ca59b695afd651f94398c0fe2e1f8b182849a876bb2 --remote https://linker.bio
[https://linker.bio/hash:...a2e9d93ae86d82415c56e1e] 100.0% of 78 bytes at 0.00 MB/s completed in < 1 minute
[https://linker.bio/hash:...b2bc1e023e0f9782cc1af77] 100.0% of 78 bytes at 0.04 MB/s completed in < 1 minute
[https://linker.bio/hash:...2215a5e14edff81320311a0] 100.0% of 78 bytes at 0.04 MB/s completed in < 1 minute
[https://linker.bio/hash:...57f406a8f87d4858b6f1efd] 100.0% of 78 bytes at 0.07 MB/s completed in < 1 minute
[https://linker.bio/hash:...50e1efe31ff6ce7323a0bb8] 100.0% of 78 bytes at 0.07 MB/s completed in < 1 minute
[https://linker.bio/hash:...5198d64208c52475e9c1dc8] 437 MB at 6.10 MB/s completed in 1 minute(s)
hash://sha256/37bdd8ddb12df4ee02978ca59b695afd651f94398c0fe2e1f8b182849a876bb2%                                                     (((b(base(b((((((((((base) katjaseltmann@unreg-dhcp-67 bif % preston cat hash://sha256/37bdd8ddb12df4ee02978ca59b695afd651f94398c0fe2e1f8b182849a876bb2 | head -10
<https://preston.guoda.bio> <http://www.w3.org/1999/02/22-rdf-syntax-ns#type> <http://www.w3.org/ns/prov#SoftwareAgent> <urn:uuid:0781f840-2a52-4245-8974-154c68787255> .
<https://preston.guoda.bio> <http://www.w3.org/1999/02/22-rdf-syntax-ns#type> <http://www.w3.org/ns/prov#Agent> <urn:uuid:0781f840-2a52-4245-8974-154c68787255> .
<https://preston.guoda.bio> <http://purl.org/dc/terms/description> "Preston is a software program that finds, archives and provides access to biodiversity datasets."@en <urn:uuid:0781f840-2a52-4245-8974-154c68787255> .
<urn:uuid:0781f840-2a52-4245-8974-154c68787255> <http://www.w3.org/1999/02/22-rdf-syntax-ns#type> <http://www.w3.org/ns/prov#Activity> <urn:uuid:0781f840-2a52-4245-8974-154c68787255> .
<urn:uuid:0781f840-2a52-4245-8974-154c68787255> <http://purl.org/dc/terms/description> "A crawl event that discovers biodiversity archives."@en <urn:uuid:0781f840-2a52-4245-8974-154c68787255> .
<urn:uuid:0781f840-2a52-4245-8974-154c68787255> <http://www.w3.org/ns/prov#startedAtTime> "2024-04-01T20:31:04.492Z"^^<http://www.w3.org/2001/XMLSchema#dateTime> <urn:uuid:0781f840-2a52-4245-8974-154c68787255> .
<urn:uuid:0781f840-2a52-4245-8974-154c68787255> <http://www.w3.org/ns/prov#wasStartedBy> <https://preston.guoda.bio> <urn:uuid:0781f840-2a52-4245-8974-154c68787255> .
<https://doi.org/10.5281/zenodo.1410543> <http://www.w3.org/ns/prov#usedBy> <urn:uuid:0781f840-2a52-4245-8974-154c68787255> <urn:uuid:0781f840-2a52-4245-8974-154c68787255> .
<https://doi.org/10.5281/zenodo.1410543> <http://www.w3.org/1999/02/22-rdf-syntax-ns#type> <http://purl.org/dc/dcmitype/Software> <urn:uuid:0781f840-2a52-4245-8974-154c68787255> .
<https://doi.org/10.5281/zenodo.1410543> <http://purl.org/dc/terms/bibliographicCitation> "Jorrit Poelen, Icaro Alzuru, & Michael Elliott. 2021. Preston: a biodiversity dataset tracker (Version 0.8.4) [Software]. Zenodo. https://doi.org/10.5281/zenodo.1410543"@en <urn:uuid:0781f840-2a52-4245-8974-154c68787255> .
(base) katjaseltmann@unreg-dhcp-67 bif % `

The expected Oct hash is (at 2024-10-18) -
hash://sha256/fc6a81d2968272005c947df326ac3e99fd4db9b585198d64208c52475e9c1dc8

disabling the cloudflare cache did not appear to resolve the issue, continuing investigation . . .

Note that the file did cause a proxy cache hit when requested from linker.bio
curl -I https://linker.bio/hash://sha256/fc6a81d2968272005c947df326ac3e99fd4db9b585198d64208c52475e9c1dc8

produced:
HTTP/2 200
date: Fri, 18 Oct 2024 19:24:51 GMT
access-control-allow-origin: *
access-control-allow-methods: GET, OPTIONS
access-control-allow-headers: DNT,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type,Range
access-control-expose-headers: Content-Length,Content-Range
x-proxy-cache: HIT
...

In comparing the file served through linker.bio and the original file, the first 10 lines are identifical, as determined by matching sha256 signatures.
Please note, however that the file served through linker.bio is smaller 438MB than the original file 536MB .
and the last three lines of the file served through linker.bio appears truncated:
cat cached-fc6a81d2968272005c947df326ac3e99fd4db9b585198d64208c52475e9c1dc8 | nl | tail -n3

2265212	<urn:uuid:f2b6f13b-d763-440f-89f8-e7570ad3a758> <http://www.w3.org/ns/prov#wasInformedBy> <urn:uuid:8294d597-360c-468d-bdbb-674b30ae0c4f> <urn:uuid:f2b6f13b-d763-440f-89f8-e7570ad3a758> .
2265213	<urn:uuid:f2b6f13b-d763-440f-89f8-e7570ad3a758> <http://www.w3.org/ns/prov#used> <https://tb.plazi.org/GgServer/dwca/8A361103FFB56B311C45FFB0912BFF8A.zip> <urn:uuid:f2b6f13b-d763-440f-89f8-e7570ad3a758> .
2265214	<https://tb.plazi.org/GgServer/dwca/8A361103FFB56B311C45FFB0912BFF8A.zip> <http://

with line 2265214 appearing to be truncated.
The same line in the original file is:
cat src-fc6a81d2968272005c947df326ac3e99fd4db9b585198d64208c52475e9c1dc8\
 | nl\
 | tail -n+2265212\
 | head -n3

being:
2265212	<urn:uuid:f2b6f13b-d763-440f-89f8-e7570ad3a758> <http://www.w3.org/ns/prov#wasInformedBy> <urn:uuid:8294d597-360c-468d-bdbb-674b30ae0c4f> <urn:uuid:f2b6f13b-d763-440f-89f8-e7570ad3a758> .
2265213	<urn:uuid:f2b6f13b-d763-440f-89f8-e7570ad3a758> <http://www.w3.org/ns/prov#used> <https://tb.plazi.org/GgServer/dwca/8A361103FFB56B311C45FFB0912BFF8A.zip> <urn:uuid:f2b6f13b-d763-440f-89f8-e7570ad3a758> .
2265214	<https://tb.plazi.org/GgServer/dwca/8A361103FFB56B311C45FFB0912BFF8A.zip> <http://purl.org/pav/hasVersion> <hash://sha256/c693574536b83bb54cdda22bbe09985f85ea79bbf09c7cef3e5741ece6946dfe> <urn:uuid:f2b6f13b-d763-440f-89f8-e7570ad3a758> .

Which suggests that somehow, the cached version of the original got truncated as it was being cached.

Looks like the nginx cache is used and keeps 64GB around.

I found the matching truncated file in the cache
cat /var/cache/prov/content/9/9b/6da0da46cf191e66cd2f6922d436c9b9 | tail -n+7 | sha256sum
481634cee082dad6f5b75aa6d88e7f304ffc4b10dacab13435580bc2365fe111  -

which sha256 sum and size matches the one served through linker.bio .

Note the key in the cache was:
KEY: xxxxx/hash://sha256/fc6a81d2968272005c947df326ac3e99fd4db9b585198d64208c52475e9c1dc8
HTTP/1.1 200 OK
Date: Fri, 18 Oct 2024 15:27:10 GMT

showing that the requested hash://sha256/fc6a81d2968272005c947df326ac3e99fd4db9b585198d64208c52475e9c1dc8 was linked to the truncated content.

turning off the cache for now.

after disabling the cache, the expected result was observed:
preston head --anchor hash://sha256/37bdd8ddb12df4ee02978ca59b695afd651f94398c0fe2e1f8b182849a876bb2 --remote https://linker.bio
[https://linker.bio/hash:...5198d64208c52475e9c1dc8] 535 MB at 24.74 MB/s completed in < 1 minute
hash://sha256/fc6a81d2968272005c947df326ac3e99fd4db9b585198d64208c52475e9c1dc8

yay for content-based addressed content. It'd be hard to find these kind of caching issues with content of unknown hash/version.


remove dependencies-of-dependencies

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Merging a 3 year old dead nixpkgs branch does not make sense to me, and no comment or explanation is provided. Closing.


cannot work updateFilterValue with built in filters

Describe the bug
I want to use updateFilterValue when click to clear all filter button, but it does not work with the built in filters:
https://stackblitz.com/edit/clear-allfilter?file=src/app/app.component.ts
Versions
Clarity project:

[ ] Clarity Core
[x] Clarity Angular/UI

Clarity version:

[ ] v3.x
[ ] v4.x
[x] v5.x

Framework:

[x] Angular
[ ] React
[ ] Vue
[ ] Other:

Framework version:
Angular 13

It does work with built-in filters. Here's your StackBlitz from the previous bug, updated:
https://stackblitz.com/edit/clear-allfilter-ur3tzc?file=src%2Fapp%2Fapp.component.ts
The difference with the new StackBlitz is that it features some custom filters, which are themselves not working in the linked demo above, see your HTML template.  Probably left-over from some attempts to make it work with custom filters?

@Jinnie thank you for the response. in fact, i want to use the Clear Filter option on the custom filter. There is any solutions maybe, how can i do that?

@kitodev I'd ask to start with a working reproduction. The above StackBlitz does not really work, so I can't say or debug now at what level the functionality gets broken.
I'll reopen the issue and mark it as needing reproduction.


Enforce maven minimum version

Why:
We have plugins that require >= 3.5.0

D'oh didn't check the existing output, the maven output is fine as is


feature request:  update to new integration method

https://www.hashicorp.com/blog/nomad-1-7-improves-vault-and-consul-integrations-adds-numa-support?mkt_tok=ODQ1LVpMRi0xOTEAAAGPci1d3czmyelPP9EadD3j0z0v6hIwygs8UOZH0H8Wzk44i8lRtXsQQgC-zpChdxBTsDXAr8b11KnXlTESrHOUmzctggvJo-S50FrXaOfbmCd3NDg

I will look into this, once this goes GA.
Quickly looking at the docs, from an automation standpoint it looks like the nomad setup command will be an interactive command, not sure if it will accept input arguments.


Pubsub: single-consumer -> multi-consumer code sample

I found the single-consumer code for aioredis.pubsub.Receiver to be quite limiting. I just don't have the use case for one task being responsible for my entire subscription interface to redis. Probably, that's normal.
So I ended up doing a couple of inversions and ended up with this code sample that allows for usage like this:
async def handler(channel):
    async for message in pubsub.subscribe(channel):
        print(message)

Much better. The code handles multiple consumers. But what is that pubsub object? Well, it's probably what Receiver should be; it's not a mpsc (multi-producer single-consumer) but instead a mpmc (multi-producer multi-consumer).
Here's the code sample:
import asyncio
import logging
import pickle
import typing

import aioredis
import async_generator

logger = logging.getLogger(__name__)

class PubSub:
    # pylint: disable=R0902, too-many-instance-attributes
    _TERMINATE = object()

    def __init__(
            self,
            redis: aioredis.Redis,
            serializer: typing.Callable[[typing.Any], typing.Any] = \
                pickle.dumps,
            deserializer: typing.Callable[[typing.Any], typing.Any] = \
                pickle.loads,
        ) -> None:
        self.redis = redis
        self.serializer = serializer
        self.deserializer = deserializer

        self._lock = asyncio.Lock()
        self._mpsc = aioredis.pubsub.Receiver()
        self._reader_fut: typing.Optional[asyncio.Future] = None
        self._registry: typing.Dict[
            aioredis.abc.AbcChannel,
            typing.Set[asyncio.Queue],
        ] = {}

    async def start(self) -> None:
        self._reader_fut = asyncio.ensure_future(self.reader())

    async def stop(self) -> None:
        if not self._reader_fut:
            raise RuntimeError(f'{type(self).__name__} is not running.')

        self._reader_fut.cancel()
        cancellation_fut = asyncio.gather(*[
            subscription.put(self._TERMINATE)
            for subscriptions in self._registry.values()
            for subscription in subscriptions
        ])
        cancellation_fut.add_done_callback(lambda fut: self.stop)

    async def reader(self):
        async for channel, produced in self._mpsc.iter():
            channel = typing.cast(aioredis.abc.AbcChannel, channel)

            try:
                message = self.deserializer(produced[1]) \
                    if channel.is_pattern \
                    else self.deserializer(produced)
            except Exception: # pylint: disable=W0703, broad-except
                continue

            asyncio.gather(*[
                sub.put(message)
                for sub in self._registry.get(channel, [])
            ])

    async def publish(self, channel, message):
        # pylint: disable=E1102, not-callable
        serialized = self.serializer(message)
        await self.redis.publish(channel, serialized)
        logger.info('Sent messsage on %s: %s', channel, message)

    @async_generator.asynccontextmanager
    async def _subscribe(
            self,
            channel: str,
            is_pattern: bool,
        ) -> typing.AsyncGenerator[asyncio.Queue, None]:
        """
        Async context manager that provides a multi-consumer proxy for aioredis'
        pubsub single-consumer.
        """
        async with self._lock:
            registration = self._mpsc.pattern(channel) \
                if is_pattern \
                else self._mpsc.channel(channel)
            subscription: asyncio.Queue = asyncio.Queue()

            if registration not in self._registry:
                if is_pattern:
                    await self.redis.psubscribe(registration)
                    logger.info('Subscribed to pattern: %s', channel)
                else:
                    await self.redis.subscribe(registration)
                    logger.info('Subscribed to channel: %s', channel)
                self._registry[registration] = set()
            self._registry[registration].add(subscription)

        exception = None
        try:
            yield subscription
        except asyncio.CancelledError as exc:
            exception = exc

        async with self._lock:
            self._registry[registration].remove(subscription)
            if not self._registry[registration]:
                if is_pattern:
                    await self.redis.punsubscribe(channel)
                    logger.info('Unsubscribed from pattern: %s', channel)
                else:
                    await self.redis.unsubscribe(channel)
                    logger.info('Unsubscribed from channel: %s', channel)
                del self._registry[registration]

        if exception: # re-raise (propagate) CancelledError
            raise exception # pylint: disable=E0702, raising-bad-type

    async def subscribe(
            self,
            channel: str,
            is_pattern: bool = False,
        ) -> typing.AsyncGenerator[typing.Any, None]:
        # pylint: disable=E1701, not-async-context-manager
        async with self._subscribe(channel, is_pattern) as subscription:
            while True:
                value = await subscription.get()
                if value is self._TERMINATE:
                    break
                else:
                    yield value

I think this code, or something like this code, should be a part of aioredis, but I don't mind maintaining this in my personal library of snippets anyway.
By the way, it's exciting that I finally got to use the new code for url parsing

This looks great. Would be wonderful if it could be added to aioredis.
@popravich would you accept this enhancement?

Dam, someone beat me to it.
I have made a similar improvement, but with a totally different implementation (mince looks a lot simpler to me)
class ChannelReader:
    """
    Wrapper against aioredis.Channel used to allow multiple readers
    on the same subscription

    Example::

        channels = await redis_pool.subscribe('subscribeme')
        reader = ChannelReader(channels[0])

        for message in reader:
            print(message)

    """
    loops = weakref.WeakKeyDictionary()

    # time to wait before failing when readers get stuck
    ack_wait_time = 1.0

    # subscribers

    def __init__(self, channel):
        self.channel = channel

        # if channel was not previously seen, create a new reader task
        if channel not in self.loops:
            weakset = weakref.WeakSet()

            self.loops[channel] = weakset
            asyncio.ensure_future(self._main_loop())

        self.subscribers = self.loops[channel]

    async def __aiter__(self):
        """
        Iterate over subscription messages
        """
        future = asyncio.Future()
        self.subscribers.add(future)

        while True:
            value = await future

            # NOTICE: previous future object is removed from the weakset
            # when this new reference is created
            future = asyncio.Future()
            self.subscribers.add(future)

            yield value

    async def _main_loop(self):
        """
        Task reading the actual channel messages and re-dispatching them
        to the different readers

        """
        time = asyncio.get_running_loop().time

        async for message in self.channel.iter():
            for subscriber in list(self.subscribers):
                subscriber.set_result(message)

            # Deleting reference to the last weakset entry
            # so it is correctly removed from the set when consumed
            del subscriber

            start = time()
            while True:
                # Wait for all done tasks to be acknowledged
                if not any(fut.done() for fut in self.subscribers):
                    break
                await asyncio.sleep(0)

                # … but don't wait too long (should not happen ©)
                if start + 1 < self.ack_wait_time:
                    raise RuntimeError('Task result not consumed')

@romuald Would you (or anyone above) be willing to put this snippet on GitHub with some license like MIT? That way we can track any changes better. If you create a bare repo with the code pasted into a file, I'd be happy to add the setup.py etc. to get it usable.

Dam, someone beat me to it.
I have made a similar improvement, but with a totally different implementation (mince looks a lot simpler to me)
class ChannelReader:
    """
    Wrapper against aioredis.Channel used to allow multiple readers
    on the same subscription

    Example::

        channels = await redis_pool.subscribe('subscribeme')
        reader = ChannelReader(channels[0])

        for message in reader:
            print(message)

    """
    loops = weakref.WeakKeyDictionary()

    # time to wait before failing when readers get stuck
    ack_wait_time = 1.0

    # subscribers

    def __init__(self, channel):
        self.channel = channel

        # if channel was not previously seen, create a new reader task
        if channel not in self.loops:
            weakset = weakref.WeakSet()

            self.loops[channel] = weakset
            asyncio.ensure_future(self._main_loop())

        self.subscribers = self.loops[channel]

    async def __aiter__(self):
        """
        Iterate over subscription messages
        """
        future = asyncio.Future()
        self.subscribers.add(future)

        while True:
            value = await future

            # NOTICE: previous future object is removed from the weakset
            # when this new reference is created
            future = asyncio.Future()
            self.subscribers.add(future)

            yield value

    async def _main_loop(self):
        """
        Task reading the actual channel messages and re-dispatching them
        to the different readers

        """
        time = asyncio.get_running_loop().time

        async for message in self.channel.iter():
            for subscriber in list(self.subscribers):
                subscriber.set_result(message)

            # Deleting reference to the last weakset entry
            # so it is correctly removed from the set when consumed
            del subscriber

            start = time()
            while True:
                # Wait for all done tasks to be acknowledged
                if not any(fut.done() for fut in self.subscribers):
                    break
                await asyncio.sleep(0)

                # … but don't wait too long (should not happen ©)
                if start + 1 < self.ack_wait_time:
                    raise RuntimeError('Task result not consumed')

For anyone else using this we got local variable 'subscriber' referenced before assignment error in _main_loop, it happens at del subscriber when self.subsribers is empty and the for-loop never happens. Fixed it by initializing subscriber = None before the for-loop


1 adding dash analytics

Adding the initial commit for dash_analytics, a package intended to only be used within Dash tooling to unify analytics across different Dash tools in Google Analytics 4.

I made a pass through with some comments. I generally didn't look at the library implementation; has this already been reviewed? And or, should somebody else review the impl?

Yes, the most part, the implementation has been reviewed at my repo github.com/eliasyishak/dash_analytics – do we need to preserve those conversations somehow in this repo?

Yes, for the most part, the implementation has been reviewed at my repo github.com/eliasyishak/dash_analytics – do we need to preserve those conversations somehow in this repo?

I don't think that's necessary. We could retain the commit history if we wanted to; that would be more useful if there had been multiple authors over time say (I don't think we need to figure out the git commands for that in this case).


Problem Run flutter On Emulator

hello guys please i Encountered  This Error when running my application. please i need help
C:\flutterApp\fluttertutorial>flutter run
Using hardware rendering with device Android SDK built for x86. If you get graphics artifacts, consider enabling
software rendering with "--enable-software-rendering".
Launching lib/main.dart on Android SDK built for x86 in debug mode...
Initializing gradle...                                                 /
Oops; flutter has exited unexpectedly.
Sending crash report to Google.
Crash report sent (report ID: 74e719945ec1c5a4)
Initializing gradle...
Crash report written to C:\flutterApp\fluttertutorial\flutter_02.log;
please let us know at https://github.com/flutter/flutter/issues.
Secondly  i Ran it using the Android Studio And this was the error i got . please  i need help and solutions
Launching lib\main.dart on Android SDK built for x86 in debug mode...
Initializing gradle...
Finished with error: ProcessException: Process "C:\flutterApp\fluttertutorial\android\gradlew.bat" exited abnormally:
Unzipping C:\Users\ONUMBA COLLINS.gradle\wrapper\dists\gradle-4.10.2-all\9fahxiiecdb76a5g3aw9oi8rv\gradle-4.10.2-all.zip to C:\Users\ONUMBA COLLINS.gradle\wrapper\dists\gradle-4.10.2-all\9fahxiiecdb76a5g3aw9oi8rv
Exception in thread "main" java.util.zip.ZipException: error in opening zip file
at java.util.zip.ZipFile.open(Native Method)
at java.util.zip.ZipFile.(ZipFile.java:219)
at java.util.zip.ZipFile.(ZipFile.java:149)
at java.util.zip.ZipFile.(ZipFile.java:163)
at org.gradle.wrapper.Install.unzip(Install.java:214)
at org.gradle.wrapper.Install.access$600(Install.java:27)
at org.gradle.wrapper.Install$1.call(Install.java:74)
at org.gradle.wrapper.Install$1.call(Install.java:48)
at org.gradle.wrapper.ExclusiveFileAccessManager.access(ExclusiveFileAccessManager.java:65)
at org.gradle.wrapper.Install.createDist(Install.java:48)
at org.gradle.wrapper.WrapperExecutor.execute(WrapperExecutor.java:128)
at org.gradle.wrapper.GradleWrapperMain.main(GradleWrapperMain.java:61)
Command: C:\flutterApp\fluttertutorial\android\gradlew.bat -v

PLEASE  HELP

try run
flutter run --enable-software-rendering
as mentioned in the error.

Thanks for the reply but don't realm know how to enable (software rendering ) ,do you have the step on how to do so ?

simply run the command  flutter run --enable-software-rendering and it will take care of it.

After running  flutter run --enable-software-rendering this was the error i encountered. Any suggestions
C:\flutterApp\fluttertutorial>flutter run --enable-software-rendering
Using software rendering with device Android SDK built for x86. You may get better performance with hardware mode by
configuring hardware rendering for your device.
Launching lib/main.dart on Android SDK built for x86 in debug mode...
Initializing gradle...                                                 -
Oops; flutter has exited unexpectedly.
Sending crash report to Google.
Failed to send crash report due to a network error: SocketException: Failed host lookup: 'clients2.google.com' (OS
Error: No such host is known.
, errno = 11001)
Initializing gradle...
Crash report written to C:\flutterApp\fluttertutorial\flutter_03.log;
please let us know at https://github.com/flutter/flutter/issues.

simply run the command flutter run --enable-software-rendering and it will take care of it.

@collinspro Can you provide that flutter_03.log? Because in this case it may be totally different issue.

Duplicate of https://github.com/flutter/flutter/issues/40517


Compilation Fail - Throws Error Badly packaged Library

Issue description
While compiling the app it throws below error:
<EMAIL_ADDRESS>is missing from the TypeScript compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
The missing file seems to be part of a third party library. TS files in published libraries are often a sign of a badly packaged library. Please open an issue in the library repository to alert its author and ask them to package the library using the Angular Package Format (https://goo.gl/jB3GVv).

Steps to reproduce and a minimal demo of the problem
Update your project to:
Angular 7.0.0-beta.6
Angular Cli 7.0.0-beta.4
@angular-devkit/build-angular 0.9.0-beta.4
Then try to build your project.
Current behavior
It throws error
Expected/desired behavior
It should compile successfully.
angular2 & angular-google-maps version
`"@agm/core": "1.0.0-beta.4"

@naveedahmed1 I don't think its a general problem. Please provide a repo on github so I can look at it

@SebastianM  Looks like this error is thrown when app is using @angular-devkit/build-angular version 0.9.*  as explained in #1515, #1500 is related as well.
I can confirm that when you downgrade to 0.8.4 version of build-angular the build passes, but looks like it's an error in library itself.


Issue70 add all option to create-project

Adds an all option for create-project, sub-directories are created with the suffixes Api, Admin, and Storefront
Solves https://github.com/reactioncommerce/cli/issues/70

@breakerbar This looks correct but the commit message does not match the semantic release format so it's not going to trigger a release. You should add the feat: prefix

force pushed with feat: prefix

:tada: This PR is included in version 1.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Make bidirectional Dijkstra traversing 'more balanced'

Minor improvement which should lead to faster execution and better conditions for things like alternative route calculations

Although this is a good approach for CH and normal weightings etc the problem starts if weighting is very asymmetric like when one wants to really avoid highways and force that very extreme. Then several millions of nodes instead of hundred thousands are explored and speed can be factor of 10 to 30 slower for none-CH.
E.g. the destination is placed on a highway, then the node exploration stops for the 'to'-shortest-path-tree as the weights are very high and only the 'from'-shortest-path-tree is able to go forward but in some situations won't directly reach the destination and will only reach it if the some of the weight of all these edges is bigger than this very high weight for the single highway.

Also the more symmetric routing is roughly 5% slower for the none-CH case.


how to use Username and password authetication

Is there have an example of connect on SFTP username and password?

This crate only provides the sftp client and the ssh client is provided by openssh.
I will transfer this issue to openssh


@see links in JSDoc comments are broken by VSCode

TS Template added by @mjbvz
TypeScript Version: 4.3.0-dev.20210420
Search Terms

quick info
IntelliSense

Issue Type: Bug
When you have a JSDoc comment with an @see line above a function or type and you hover over said function or type, the link will be automatically split after https, making it impossible to Ctrl + click to go to the linked website. The link should display like this: https ://
Example:

VS Code version: Code 1.55.2 (3c4e3df9e89829dce27b7b5c24508306b151f30d, 2021-04-13T09:35:57.887Z)
OS version: Windows_NT x64 10.0.19042

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 x 3192)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
19.89GB (10.13GB free)

Process Argv
D:\Users\rodri\Documents\GitHub\hypixel-translators-bot --crash-reporter-id 1aa51578-c79e-468f-b062-c24334e21918

Screen Reader
no

VM
0%

Extensions (23)

Extension
Author (truncated)
Version

gitlens
eam
11.4.1

EditorConfig
Edi
0.16.4

prettier-vscode
esb
6.3.2

vscode-lombok
Gab
1.0.1

vscode-graphql
Gra
0.3.15

discord-vscode
icr
5.6.3

vscode-env
Iro
0.1.0

csharp
ms-
1.23.11

python
ms-
2021.4.765268190

jupyter
ms-
2021.5.745244803

vsliveshare
ms-
1.0.4131

vsliveshare-pack
ms-
0.4.0

gradle-language
nac
0.2.3

vetur
oct
0.33.1

java
red
0.77.0

vscode-yaml
red
0.18.0

vscodeintellicode
Vis
1.2.12

vscode-java-debug
vsc
0.32.1

vscode-java-dependency
vsc
0.18.3

vscode-java-pack
vsc
0.14.0

vscode-java-test
vsc
0.28.1

vscode-maven
vsc
0.29.0

minecraft-lang-colorizer
zz5
1.0.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspyt653:30270858
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstry244:30276681
pythonvsdeb440:30248342
pythonvsded773:30248341
pythonvspyt875:30259475
pythonvsnew554cf:30291488
pythontb:30283811
vspre833cf:30267465
pythonptprofiler:30281270
vshan820cf:30276953
pythondataviewer:30285071
vscus158cf:30286554

@sandersn Looks like @see https://example.com is being broken into:
 "name": "see",
            "text": [
                {
                    "text": "https",
                    "kind": "text"
                },
                {
                    "text": " ",
                    "kind": "space"
                },
                {
                    "text": "://example.com",
                    "kind": "text"
                }
            ]

should be fixed by https://github.com/microsoft/TypeScript/pull/43800

Is there a way for me to test before the next release is out?

@ImRodry maybe this extension? https://marketplace.visualstudio.com/items?itemName=ms-vscode.vscode-typescript-next
I don't use this but I think @mjbvz made it so if I'm right he'll know more.

Oh I had seen that extension and installed it but forgot to restart vscode. The issue is indeed fixed so I'll close this. Thank you!


Separate command blocks in dev README

What this PR does / why we need it:

Which issue(s) this PR fixes:

Does this PR require a test?

NONE
Does this PR require a release note?

NONE

Does this PR require documentation?

NONE

Merging as this is only a README change.


az account show - Python sdk equivalent

Hi there,
I want to get the logged in user information, which I would otherwise do with the Azure CLI command az account show but I am struggling to find how.
Is there any possible way or is it a feature that is missing?
Kind regards,
Stamatis Katsaounis

Hi @skatsaounis
There is no concept of "currently logged user" in SDK, this is a CLI concept. I think we should take a step back and try to understand what are you trying to achieve. If you wish to run a SDK script with CLI currently logged user, there is several solution like AzureCliCredential (see https://pypi.org/project/azure-identity/). This will ask the CLI what is the current logged user and import the credentials to run your SDK with them.
If this is not what you are trying to do, could you be more precise?
Thanks,

Hi @lmazuel ,
The intention behind this is that I want to get some information from the CLI signed_in_user to consume it in my application. For example: email, display_name etc.
I finally managed to do it but the solution is very ugly and I am wondering if it can be made simpler than that. Please take a look at my snippet:
from azure.identity import DefaultAzureCredential
from azure.mgmt.resource import ResourceManagementClient
from azure.mgmt.resource.subscriptions import SubscriptionClient
from azure.graphrbac import GraphRbacManagementClient

from my_project_modue import AzureIdentityCredentialAdapter

env_creds = not(os.environ.get('AZURE_CLIENT_ID') and os.environ.get(
    'AZURE_CLIENT_SECRET') and os.environ.get('AZURE_TENANT_ID'))

credentials = DefaultAzureCredential(
    exclude_interactive_browser_credential=False,
    exclude_shared_token_cache_credential=True,
    exclude_visual_studio_code_credential=True,
    exclude_managed_identity_credential=True,
    exclude_environment_credential=env_creds)

resources_credentials = AzureIdentityCredentialAdapter(
    credential=credentials)

subscriptions_client = SubscriptionClient(
    credentials=resources_credentials)

sub_list = subscriptions_client.subscriptions.list()
current_sub = list(sub_list)
tenant_id = current_sub[0].tenant_id

resources_credentials = AzureIdentityCredentialAdapter(
    credential=credentials, resource_id="https://graph.windows.net")
graphrbac_client = GraphRbacManagementClient(
    credentials=resources_credentials, tenant_id=tenant_id)

user_obj = AzureClient.graphrbac_client.signed_in_user.get()
print(user_obj.display_name, user_obj.mail_nickname)

My adapter is an object essential since the different implementations of credentials between clients

You could build your own credential chain instead of configuring DefaultAzureCredential:
from azure.identity import ChainedTokenCredential, EnvironmentCredential, InteractiveBrowserCredential

credential = ChainedTokenCredential(EnvironmentCredential(), InteractiveBrowserCredential())

That would check environment variables first, and fall back to interactively authenticating a user.

@chlowell Thanks for the tip! Regarding the initial question. Is there any simpler way to be able to get user display_name or is this the only one? Supposing that the credential here is created the way you are recommending.

Sorry for the late response; I was on vacation. I don't know a simpler way to get a user's display name than from Graph, as you're doing.

No worries. In that case let's close this ticket as it served its purpose. 🙂

I'll close this issue then, please open another if you encounter another problem.


Add two e2e tests for mixer check cache

Covers ingress and service sidecar cases. Part of fix for #2160

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
24e7d070b61a5c7b5e07e2d725905ba3e4ed08d4
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
24e7d070b61a5c7b5e07e2d725905ba3e4ed08d4
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
24e7d070b61a5c7b5e07e2d725905ba3e4ed08d4
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
24e7d070b61a5c7b5e07e2d725905ba3e4ed08d4
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
24e7d070b61a5c7b5e07e2d725905ba3e4ed08d4
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
24e7d070b61a5c7b5e07e2d725905ba3e4ed08d4
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
24e7d070b61a5c7b5e07e2d725905ba3e4ed08d4
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
24e7d070b61a5c7b5e07e2d725905ba3e4ed08d4
link
/test e2e-bookInfo

prow/istio-unit-tests.sh
22739aec85291c26ede03465d35079242baecd7e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
22739aec85291c26ede03465d35079242baecd7e
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
22739aec85291c26ede03465d35079242baecd7e
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
d6f428c18a34241fcce54a8ca5d61c42cab89a1e
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

ptal.

/test istio-pilot-e2e

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: douglas-reid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS [douglas-reid]

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

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
d6f428c18a34241fcce54a8ca5d61c42cab89a1e
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
d6f428c18a34241fcce54a8ca5d61c42cab89a1e
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Generate build targets for benchmark presets

Create a build target for each benchmark preset, indicated by the presets field in the benchmarks. Makes it easier to build only the needed benchmarks.
When we added large models to the benchmark suites, there was an attempt to classify benchmarks with tags and selectively build and run benchmarks. IMO what we want is to label each benchmarks with the list of presets, later the build system and benchmark tools read the preset list to know what to do.
This change:

Annotates architecture-based presets on all benchmarks
Creates a CMake build target for each preset
Update benchmark tools (export_benchmark_config.py) to read the presets field.

The build targets for all presets are:
# Meta benchmark presets based on size
iree-benchmark-suites-default
iree-benchmark-suites-large
# Architecture-based benchmark presets
iree-benchmark-suites-android-cpu
iree-benchmark-suites-android-gpu
iree-benchmark-suites-cuda
iree-benchmark-suites-cuda-large
iree-benchmark-suites-riscv
iree-benchmark-suites-vulkan-nvidia
iree-benchmark-suites-x86_64
iree-benchmark-suites-x86_64-large
# Compilation benchmark presets
iree-benchmark-suites-comp-stats
iree-benchmark-suites-comp-stats-large

The legacy but intuitive iree-benchmark-suites is now an alias to iree-benchmark-suites-default. The build targets of compilation benchmarks iree-e2e-compile-stats-suites* is renamed to iree-benchmark-suites-comp-stats* for consistency.
Fix #13683
benchmark-extra: x86_64-large

Kindly ping


Having Dreo ChefMaker crashes plugin

Dreo ChefMaker is pretty neat device but crashes the plugin - probably no support for it:
[18/11/2023, 8:25:52 pm] [Dreo Platform] Adding new accessory: ChefMaker
TypeError: Cannot read properties of undefined (reading '1')
    at new FanAccessory (/var/lib/homebridge/node_modules/homebridge-dreo/src/FanAccessory.ts:38:75)
    at DreoPlatform.discoverDevices (/var/lib/homebridge/node_modules/homebridge-dreo/src/platform.ts:150:9)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

This should be fixed once #30 is merged.


Factorio 0 14 x

Update info.json file for 0.14 compatibility

@anoutsider Time for a mod portal release before this maybe?

Yes will be pushing a 0.3.1 release to the portal today, personally i prefer to keep separate version numbering for each factorio major release, so for 0.14 it'd be 0.4.x.
Would also move the 0.13 updates to a 0.13 branch and keep the master for 0.14/0.4.


Updated OpenBSD changes

Took into account feedback on previous pull request

@bceverly You should have a look at the FindXXX.cmake files and fix the search paths there. This way it should not be necessary to add a global link directory. As for execinfo libraries should be linked by the library or executable actually using them and not globally. Still missing the revert for the #ifdef that is not necessary if properly adding the include path for the libraries detected.

I believe my latest push addresses everything.  Turns out we didn't need
the library on OpenBSD so I made its inclusion for FreeBSD only.
Thanks,
Bryan
On Mon, Sep 14, 2015 at 3:17 AM, akallabeth<EMAIL_ADDRESS>wrote:

@bceverly https://github.com/bceverly You should have a look at the
FindXXX.cmake files and fix the search paths there. This way it should
not be necessary to add a global link directory. As for execinfo
libraries should be linked by the library or executable actually using them
and not globally. Still missing the revert for the #ifdef that is not
necessary if properly adding the include path for the libraries detected.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeRDP/FreeRDP/pull/2883#issuecomment-139982100.

@bceverly if possible try not to have merges within your pull request.

@freerdp-bot test

Ok, looks good now ;)

LGTM too, so I'm merging it too fast like the last time ;)


Fix multiple buildpack invocation collision

Context
If the go buildpack is used by multiple users in the same environment, users other than the first user are unable to succesfully run compile. Running compile results in an exit code of "1", with no output other than this, which only occurs on the first run:
-----> Fetching jq... done

After the first run, no output appears (because jq is cached), and the exit code is still "1"
This occurs because a recent change to the buildpack replaced the old mktemp behaviour with a predefined path to a directory to use for temporary storage here.
Solution
Go back to using mktemp to create a temporary directory
Presumably there's a reason to support the TMPDIR variable (docker layer caching maybe? I don't know.), so this pull request adds logic to preserve the existing behaviour that uses $TMPDIR/heroku-buildpack-go if that variable is set, but otherwise mktemp -d is used.

Nevermind my comments (since deleted), apparently my coffee hadn't kicked in yet.


Slightly improve the usage of OutOfRangeError

Holy f**k, another case of an overly complicated error type stealing time and brain power from me.
The OutOfRangeError is re-used to mean multiple things, it is complex enough that we wrote a kani test that used it incorrectly, and it took me an hour or so to debug it - awesome.

Document the retarded usage of the is_signed field.
Fix the broken kani test.
Add a unit test to check no-one patches OutOfRangeError::negative function when they notice it looks insane.

Patch does not make the error type less complex because the whole amount module is overly complicated and a real PITA to work on.
PR does not need backporting, its just a fix to the kani verification. As such it counts as CI only and can be one ack merged.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 83.137%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
19203

Relevant Lines:
23098

💛  - Coveralls

Lol. These changes are fine but FWIW I found the existing fields and functions perfectly clear.

Lol. These changes are fine but FWIW I found the existing fields and functions perfectly clear.

Did you? So it just me being simple - damn.

ooooh, I see where my confusion is. We have a variable is_negative that represents if the value being parsed is negative not to be confused with is_signed which means Amount/SignedAmount ... I still think the errors are too complicated, that took me over 20 minutes to work out.

We should've split the error into two (possibly with a generic) rather than having one error with a boolean inside. But that would've made the API noiser. So it's not an obvious tradeoff. Maybe we made the wrong one.

I've inverted the is_greater_than_max to is_negative, and added comments. At least the is_signed vs is_negative thing is right there now. And there is no "greater-than-max" relating to a u64 which is confusing.

Force push includes just the change to is_below_min.

In 5b686deb5e7aef9ebb07f845ccb7c83fcaaf6d96:
I'm tempted to just ACK this, but I'm not entirely convinced it's an improvement. Yes, it would have made our broken Kani test more obviously broken. But the fact that we now have accessors whose names clearly match the old field name, and which now return the value of a completely-differently-named field, suggests that the old name had merit.

Closing in favour of #2700 since this PR is not satisfactory for reviewer or for author.


Fix nl tax code

Discovered that the NL tax code validation was not working for all examples (as far as I can gather) from a change in 2020 that meant codes need to be checked using either a mod11 or mod97 algorithm. We only had mod11.

I'll merge this directly. But you might find it interesting @dstotijn that we weren't correctly able to verify your NL Tax ID :-)


Experimental results cannot reach the results of the paper

Hello researcher, thank you very much for such an amazing paper and idea.
I am very amazed by this result and I would like to try to reproduce your result for this reason.
I set up a brand new conda environment and followed your experimental requirements to the letter.
But using python main.py --saved-model-name matchmaker.h5 --train-test-mode 1
I can only get a score of 88 for mse, which is very puzzling to me. Can you share your experimental environment and other tricks?

Hi,
It could be due to early stopping since you are training the model. Can retry your training after increasing the earlyStop_patience value to 1000 in here

Thank you for your reply!
I guessed before that this might be the problem. But the model trained 2000 epoch. but still can't reach the accuracy.
Also, I used a completely consistent environment.


[Feature Request] Support array of introMessage

Allow to multiple introMessage, currently i am having a usecase of 2 intro message from ai, 1st one is for greeting, 2nd one is for giving example question the user can ask
Ref: https://deepchat.dev/docs/messages/#introMessage
Thank you!

Hi @michaelleungbowtie.
I have added this feature and it is now available in deep-chat-dev and deep-chat-react-dev version 9.0.190.
The new introMessage interface has been changed to the following:
{text?: string, html?: string} | {text?: string, html?: string}[]

It accepts a single message or an array of messages.
I will push this change in the next release. Let me know if you face any issues with it. Thanks!

Thank you for the fix @OvidijusParsiunas
I noticed a few issue when checking it out (sorry i am testing a few of issue i submitted together, let me know if that is confusing):

When i am having 2 introMessage the ai avatar is displayed on both of the message
The clearMessages function now triggers an error of Cannot read properties of undefined (reading 'innerContainer') from the hideRoleElements function

Minimum reproducible example: codesandbox

@michaelleungbowtie Thankyou for letting me know about these bugs! I will take a look at them ASAP! ❤

thank you! just a supplement, i think the same issue happened in the deep-chat-react: 2.0.1 too

that works for me, thank you!


Allow grantee withdraw unbonded value via KEEP token dashboard

Depends on: #1834 keep-network/keep-ecdsa#470
Closes: #1838
This PR allows the grantee to withdraw unbonded value via dapp.

Currently Withdraw button is available for authorizer of the delegation. The withdraw transaction is rejected by the contract as this type of role is not allowed to perform the operation.
KeepBonding contract allows only token owner, operator, and grantee to withdraw funds.

I tested this change with the following scenarios:
I. Bonding withdraw without delegation - https://github.com/keep-network/keep-ecdsa/pull/478
II. Bonding withdraw with a delegation
a. Delegated by Owner
token owner:	0x3d373D872B7BA29d92Ed47CAA8605b4DD6Ec84eF
operator: 	0xF0002dFf3113400f953DFc2F341129f8D4F40650
beneficiary: 	0xD4aF650b6664059C55D1abdE5156b08884318c06
authorizer: 	0xdf42641326140B45e23E178aE6FAaFF08E48cdEa
withdraw by:

token owner - ✅
operator - ✅

b. Delegated by grantee (Grant - TokenGrant.receiveApproval)
token owner: 				0x3d373D872B7BA29d92Ed47CAA8605b4DD6Ec84eF
grantee: 			0xaaB0ca8E2C3d86120b5A840E0006d18E6ea84000
operator: 		0xC21cB9C99C316d1863142F7dD86dd5496D81A8D6
beneficiary: 	0x86AD7e828F7DDd39D4Ac9Ec2050FA1578418aDC3
authorizer: 	0x68ad60CC5e8f3B7cC53beaB321cf0e6036962dBc
withdraw by:

grantee - ✅
operator - ✅
token owner - not allowed

c. Delegated by grantee (ManagedGrant - ManagedGrantFactory.createManagedGrant)
token owner: 				0x3d373D872B7BA29d92Ed47CAA8605b4DD6Ec84eF
grantee: 			0xfe04e65Ae59BAbdf862EcA1ADc287F2B1C3075e7
operator: 		0x45B756a5f662c4f99Bef0397a55f94DDDeF4C62E
beneficiary: 	0xfd42E065d5b128a65d752F11A1dA089f74DBf49A
authorizer: 	0x1C948F07dF428E79c5821D6215b66269c286aAaA
withdraw by:

managed grantee - ✅
operator - ✅
token owner - not allowed


Missing entities?

Hi there,
Hope you can help - I hadn't actually completed these dashboards which is why they are in several bits, but I think this happened whilst updating; I seem to have lost a lot of entities? I was hoping to be able to run an automation where i set it to use electric as a source and then kick off hearing the tank to a set charge and these options no longer seem to be available? Any ideas or suggestions welcome, thanks :)

My Mixergy entities have also gone unavailable after an update that I took yesterday.  It has been working fine, but I fear something got broken perhaps.  I tried re-installing, but to no avail.

I think the entity names have changed e.g. you reference sensor.target_charge, but this is now called sensor.mixergy_target_charge.
Check the integration itself to be sure all the entities are present.
I don't remember when I did that, so sorry about that!

I removed and downloaded the integration again, and that seems to have cleared up my issues.  I am guessing that there was some corruption over the update.
Thanks for your work, and your reply - I really appreciate it.  And btw, my entity is still sensor.target_charge

Hi there, I can report this is all working as expected for me again too thank you. I did a few things that may have helped, namely updating HA and my NAS drive, which required suspening and restarting my HA VM. All entities are back now - so all good thanks. Switch it off and on again :)

Great!


fix formatting of md documents

*.md files in doc/man directory have some issues:

some options are not correctly marked by **
kube-proxy.md is missing space between # and NAME heading

@eparis

ok to test

@ingvagabund I would love if you could find a way to un-revert the removal of the .1 man pages.  But this PR itself looks good.

In order to generate man pages from md, go-md2man has to be installed on a system. go-md2man is not a dependency of kubernetes. Man pages need to be generated in a pre commit hook script which has to either install go-md2man on a system and run transformation. Or run transformation in a container that requires containerization tools. I don't want to replicate Mike Danese's job. Plus there is no assurance that format of md files is not going to change (e.g. options items) as Mike is going to update md files manually. At the moment additional script with 3 seds is neeed to transform md files into a form suitable for transformation. The script can change over time so it is more suitable to have it as a part of distro's spec file until Mike comes with its solution.
@mikedanese, when do you think man pages will be generated? Btw. can you keep formatting of md files as they are now? Without changing a structure of a list of options? The current format is:
*[[:space:]]***--option=value**:[[:space:]]DESCRIPTION_OF_OPTION

I have a branch that autogens with cobra. I'm aiming to get this in for v1.0 (so next week or so). I REALLY want this to be autogenerated during run-gendoc.sh. If you have a different solution that generates everything (markdown and man) reasonably quickly during ./hack/run-gendoc.sh, I will happily review and get it into v1.0

The doc changes look fine. @mikedanese - Do we want to take manual changes or just wait for auto-changes?

Let's LGTM this and we can see which get's in first.

Fyi #10594


use of custom model throws an error

Hi, Thanks for your work. I am trying to use "Activation maximization" notebook with my own custom model. The difference is that my model takes an input of (56,56,24). The gradient ascent function accepts my model without an error. I perform my own  transformation on a random generated input and the input shape is torch.Size([1, 24, 56, 56]).
When I call visualize function giving it any intermediate layer and filter, it throws below error:

TypeError                                 Traceback (most recent call last)
 in
----> 1 g_ascent.visualize(layer2_0_conv2, title='layer1_0_conv2')
~/.local/lib/python3.6/site-packages/flashtorch/activmax/gradient_ascent.py in visualize(self, layer, filter_idxs, lr, num_iter, num_subplots, figsize, title, return_output)
210                                     num_iter,
211                                     len(filter_idxs),
--> 212                                     title=title)
213
214         if return_output:
~/.local/lib/python3.6/site-packages/flashtorch/activmax/gradient_ascent.py in _visualize_filters(self, layer, filter_idxs, num_iter, num_subplots, title)
347                 standardize_and_clip(output[-1],
348                                      saturation=0.15,
--> 349                                      brightness=0.7)))
350
351         plt.subplots_adjust(wspace=0, hspace=0); # noqa
~/.local/lib/python3.6/site-packages/matplotlib/init.py in inner(ax, data, *args, **kwargs)
1563     def inner(ax, *args, data=None, **kwargs):
1564         if data is None:
-> 1565             return func(ax, *map(sanitize_sequence, args), **kwargs)
1566
1567         bound = new_sig.bind(ax, *args, **kwargs)
~/.local/lib/python3.6/site-packages/matplotlib/cbook/deprecation.py in wrapper(*args, **kwargs)
356                 f"%(removal)s.  If any parameter follows {name!r}, they "
357                 f"should be pass as keyword, not positionally.")
--> 358         return func(*args, **kwargs)
359
360     return wrapper
~/.local/lib/python3.6/site-packages/matplotlib/cbook/deprecation.py in wrapper(*args, **kwargs)
356                 f"%(removal)s.  If any parameter follows {name!r}, they "
357                 f"should be pass as keyword, not positionally.")
--> 358         return func(*args, **kwargs)
359
360     return wrapper
~/.local/lib/python3.6/site-packages/matplotlib/axes/_axes.py in imshow(self, X, cmap, norm, aspect, interpolation, alpha, vmin, vmax, origin, extent, shape, filternorm, filterrad, imlim, resample, url, **kwargs)
5624                               resample=resample, **kwargs)
5625
-> 5626         im.set_data(X)
5627         im.set_alpha(alpha)
5628         if im.get_clip_path() is None:
~/.local/lib/python3.6/site-packages/matplotlib/image.py in set_data(self, A)
697                 or self._A.ndim == 3 and self._A.shape[-1] in [3, 4]):
698             raise TypeError("Invalid shape {} for image data"
--> 699                             .format(self._A.shape))
700
701         if self._A.ndim == 3:
TypeError: Invalid shape (56, 56, 24) for image data
Can anyone help me with this please.

Hi @SarfarazHabib,
Thanks for your question.
The reason why it's giving an error is that currently the visualize method is passing the output of the gradient ascent to matplotlib which expects the input to be an RBG image.
It is on my todo list to make the API more generalisable to various custom inputs and not just RGB or grayscale images.
As a quick solution meanwhile, I suggest below:

Delete this line so the conditional becomes just

if isinstance(module, nn.modules.conv.Conv2d):

Use optimize api, rather than visualize, which returns raw gradients accumulated over the number of iterations as a list

Plot the gradients/channels of your interest

This is untested, but hope this helps - let me know how it goes.

@SarfarazHabib please feel free to reopen the issue or open a new one if required.


[WIP] Foldable devices with .NET MAUI

Initial proposal for the Foldable NuGet docs.

[x] What is the right location in the TOC? Currently in Fundamentals but could belong in User Interface > Views ?
[ ] ~Whether to include DualScreenInfo?~
[ ] Whether to include Triggers?
[ ] More samples required?

Screwed up the branch, moved to #1057


Invalid xiVIEW csv exported for ambigous crosslinks

Format seems to be incorrect when exporting ambigous crosslinks to xiVIEW.
Related issue: https://github.com/Rappsilber-Laboratory/xiView_container/issues/94

no, it's a bug in the software reading the file, you can close this


chore: keeping track of latest seen position in different state stores

Previously KAFKA-13480 was adding position tracking to the RocksDB and InMemory KV store. This PR adds other stores such as CachingKVStore, and all Session and Windowed variants.

I think the test build failed before because of that bug on trunk. I kicked it off again.

The test failures were known-flaky, unrelated tests.


Cells on the edge of the grid causes a crash

java.lang.ArrayIndexOutOfBoundsException: 31
	at virus$Cell.pushOut(virus.java:1054)
	at virus$Cell.doAction(virus.java:911)
	at virus$Cell.iterate(virus.java:883)
	at virus.iterate(virus.java:185)
	at virus.draw(virus.java:116)
	at processing.core.PApplet.handleDraw(PApplet.java:2482)
	at processing.awt.PSurfaceAWT$12.callDraw(PSurfaceAWT.java:1547)
	at processing.core.PSurfaceNone$AnimationThread.run(PSurfaceNone.java:313)

I was able to fix by setting all cells on the edge to empty (type 0) in getTypeFromXY
int getTypeFromXY(int preX, int preY){
  //set all cells on the edge to  emtpy
  if ( preX >= WORLD_SIZE - 1 || preY >= WORLD_SIZE - 1 ) {
    return 0;
  }
...

I mentionned that in my issue #11 the assumptions the program is making about the cell arrangements is unclear and makes experimenting with it 1000% harder.

If this helps, I'm pretty sure this happens when running the "pushOut" function in Cell.pde, which ejects particles from inside the cell to outside, and is only ran when the cell runs the "REMOVE WASTE" or "REMOVE FOOD" codons. The cell checks if there are any obstacles in it's way before ejecting waste. These obstacles would be other cells or walls, and it checks them by looking in [x + 1, y], [x - 1, y], [x, y + 1], [x, y - 1] indexes, which is the equivalent to right, left, up, and down. However, when a cell is against the wall and tries to eject a particle, it will check if a cell is in a position which is out of bounds of the cells array. This causes a ArrayIndexOutOfBoundsException error. A solution is to check if the x and y the cell is checking is bigger than the dimensions of the cells array, or the world size. Hopes this gets fixed!

Just fixed it! Let me know if it doesn't work, thanks!
Replace pushOut function in Cell.pde with this,
  public void pushOut(Particle waste){     int[][] dire = {{0,1},{0,-1},{1,0},{-1,0}};     boolean canPushOut = false;     for(int i = 0; i < 4; i++) {       if(!(y+dire[i][1] > WORLD_SIZE - 1 || y+dire[i][1] <= 0 || x+dire[i][0] > WORLD_SIZE - 1 || x+dire[i][0] <= 0 || cells[y+dire[i][1]][x+dire[i][0]].type != 0)) {         canPushOut = true;       }     }     if(canPushOut) {       int chosen = -1;       while(chosen == -1 || y+dire[chosen][1] > WORLD_SIZE - 1 || y+dire[chosen][1] <= 0 || x+dire[chosen][0] > WORLD_SIZE - 1 || x+dire[chosen][0] <= 0 || cells[y+dire[chosen][1]][x+dire[chosen][0]].type != 0){         chosen = (int)random(0,4);       }     double[] oldCoor = waste.copyCoor();     for(int dim = 0; dim < 2; dim++){       if(dire[chosen][dim] == -1){         waste.coor[dim] = Math.floor(waste.coor[dim])-EPS;         waste.velo[dim] = -Math.abs(waste.velo[dim]);       }else if(dire[chosen][dim] == 1){         waste.coor[dim] = Math.ceil(waste.coor[dim])+EPS;         waste.velo[dim] = Math.abs(waste.velo[dim]);       }       waste.loopCoor(dim);     }     Cell p_cell = getCellAt(oldCoor,true);     p_cell.removeParticleFromCell(waste);     Cell n_cell = getCellAt(waste.coor,true);     n_cell.addParticleToCell(waste);     laserT = frameCount;     laserTarget = waste;     }     else{       println("i'm stuck");     }   }


Use MAX_LENGTH constant to check buffer size

Replace hard-coded constant for max buffer size check with Buffer MAX_LENGTH when available.

I actually prefer to use my own copy of the constant, which is documented explicitly in the API. If node later decides to increase the limit, I want my API to remain the same. The limit in yazl is based on node's limit, but it is now yazl's limit. If node were to reduce the limit, that would break my assumptions, but I don't think that's ever going to happen.
Thanks for the contribution though! And sorry for the delayed response.

@thejoshwolfe sorry I should have been more explicit in the description: the motivation was that 64 bit versions of node were capable of creating buffers up to 4GB, so valid buffers are triggering this error and I assumed the limit you have here was related to the maximum buffer size in Node. The value of MAX_LENGTH is different on different architectures, so that's why I used it, rather than just changing the constant value here. Since the PR the limit has again been lifted to 8 PiB on 64 bit architectures in v23, which means buffers are effectively limited only by available memory now. Have a look at the API docs to confirm. If this check is indeed related to max buffer size then it might be worth reconsidering since the current limit is below what's possible in Node.
Rereading this some time later though I'm actually not totally clear on what the purpose of the check is? yazl can add files to a zip larger than 1GB via a stream, and it's not possible to allocate a Buffer larger than MAX_LENGTH as Node throws an error. Do you need the check at all?

Huh. Now I'm also not totally clear on what the purpose of the check is. I don't like the idea of zlib compression throwing a runtime error when the compressed buffer would exceed the max size; that would be a pretty bad user experience. But limiting the input is only one of many ways to avoid that problem. (And the current limit doesn't even avoid that problem, I think?) It could also be solved by streaming the compression into the output instead of keeping the compressed buffer in ram. :thinking: (This also relates to #54 .)
It does seem compelling to allow the bigger buffers when they're available and to simply document that the size limit is imposed by node, and link to the API docs you gave. Those docs are pretty clear and include the change history and everything.

Yes I think it makes sense that the principle behind doing the check is to try to avoid the output buffer getting too big by not allowing in huge inputs. Having thought about it, since it's checking an existing buffer that the user is providing, by definition it can't possibly be bigger than MAX_BUFFER, so checking length against that might actually be redundant anyway?
For now, might it be worth removing instead of modifying the check here since it sounds like it won't cause new  problems and would allow bigger buffers to be added?
I'm not across the library enough to talk about the compressed buffer bit and I'm unclear on whether Node is going to throw automatically in that code, but it might already throw a RangeError that could be caught and wrapped with a friendlier message?


Monophony

Linux app for streaming music from YouTube.

Metadata

homepage URL: https://gitlab.com/zehkira/monophony
source URL: https://gitlab.com/zehkira/monophony
license: mit
platforms: linux

I'd be interested in doing this if its not too hard, I'll look into it.

The package is almost done, had some complications including unpackaged dependencies, but it should be good to go within the next few days.

Just waiting for a merge now.


PP-5366 - use inputmode and pattern attribute on numeric fields

The Design System team have done some accessibility testing on input type="number" and deemed them problematic. They have suggested using
input type="text" inputmode="numeric" pattern="[0-9.]*"
instead as documented here https://github.com/alphagov/govuk-frontend/issues/1449

The pattern is [0-9.] so decimal points are allowed it brings up this keyboard in iOS 12.2

But does using pattern mean one can experience the built-in browser validation error messages? If so, it might be better to just not include pattern at all.


Clean up errors.

- Summary
This PR streamlines the implementation of errors.

add BaseError to use class name as .name property to eliminate the additional const error descriptions.
rename EditorialWorkflowError to NotUnderEditorialWorkflowError to eliminate the single property conditional which was effectively unused.
document APIError and EditorialWorkflowError

- Test plan
There are unit tests in place that confirm the error classes work.  The two code paths that would trigger these errors would be trying to edit a post not under EditorialWorkflow or triggering a github error with a 504 response.  I'm not quite sure how to do that.  From reading the code the behavior should ostensibly be the same.

oh please don't merge this one yet. I have more tests to finish.
On Fri, Sep 29, 2017 at 3:28 PM, Shawn Erquhart<EMAIL_ADDRESS>wrote:

@erquhart approved this pull request.
Added one nitpick comment, otherwise LGTM.
In src/valueObjects/errors/tests/BaseError.spec.js
https://github.com/netlify/netlify-cms/pull/631#discussion_r141948756:

@@ -0,0 +1,22 @@
+import BaseError from '../BaseError';

+describe('valueObjects', () => {
Nitpick: these outer two describes can be combined since we're inside of
both the valueObjects and errors directories and any tests here would
only apply to both.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/netlify/netlify-cms/pull/631#pullrequestreview-66248768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXqOGNHkhW3ORwsdt5gtUigYwa0SaW0ks5snUTfgaJpZM4Pjb1J
.

--
Darrel O'Pry
The Spry Group, LLC.
http://www.spry-group.com
718-355-9767 x101

@dopry okay to merge?

Yep.
On Oct 13, 2017 1:10 PM, "Shawn Erquhart"<EMAIL_ADDRESS>wrote:

@dopry https://github.com/dopry okay to merge?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/netlify/netlify-cms/pull/631#issuecomment-336512171,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXqOCcpCr19q7UsCZfRDd-qXCxpelPdks5sr5mBgaJpZM4Pjb1J
.

@Benaiah @tech4him1 if either of you approve, merge away.

@Benaiah go ahead and make that a change request, looks like a few of us are in agreement with your thoughts BaseError.

Look don't worry 'bout it. It's easy enough to close the issue if you don't like the implementation.


doc: fix format

Commit Message:
Massage the doc build complain at alien dev machines.
Additional Description:
I may have a weird env so docs/build.sh always complains.
This fix should pass universally.
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]

@lambdai just so I understand, how is this failing for you? Everything should be in a venv by pip? cc @htuch

Yeah, changes LGTM, but I don't get why this is not hermetic.


paho-mqtt-c: 1.3.12 → 1.3.13

Description of changes
Changelog
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux (cross)
[x] x86_64-darwin
[ ] aarch64-darwin

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 262055 run on x86_64-darwin 1

2 packages built:

paho-mqtt-c
paho-mqtt-cpp


This VS microsoft version

Hi, do you have a recommendation whether to use this extensions versus the microsoft terraform extension?
I saw that you work for MS. Did you work on MS's terraform ADO extension?
BTW, this extension's TerraformInstall task and the one from MS have identical names and having both extensions causes a conflict.

Hey @gjcampbell, to be honest i was not aware of the other extension. It looks like that was created only a few months ago. I would recommend evaluating which suites your needs best. It appears that the other supports deployments to other cloud providers than azure. If you need to deploy to AWS or Google, that may be a good reason to use the other.
As a side note, I am trying to work on transitioning this extension to a microsoft dev team. I am hoping this transition will happen before the end of the year. If that is successful then this should have a full time dev team. I would imagine these would eventually be converged after that point.
Hopefully that helps. Let me know if you have any other questions.


VertxSliceServer.start() returns port the server started to listen on

Closes issue #11
VertxSliceServer may start on random port if 0 value is passed as port argument and will return actual port it listens on as result to call of start() method. Useful in tests

@g4s8 PLease assign to me, I'm available

This pull request #10 is assigned to @vatavuk/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @g4s8/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be a monetary reward for this job

@olegmoz please reference issue in the PR description.

@rultor release, tag=0.2.2

@rultor release, tag=0.2.2

@g4s8 OK, I will release it now. Please check the progress here

@rultor release, tag=0.2.2

@g4s8 Done! FYI, the full log is here (took me 18min)

@0crat status

@0crat status (here)

@Vatavuk This is what I know about this job in CT2E6TK9B, as in §32:

The job #10 is in scope for 4days
The role is REV
The job is assigned to @vatavuk/z for 3days
There is a monetary reward attached
@vatavuk/z will get 25% of their hourly rate for this job
The job doesn't have any impediments
The budget is 15 minutes/points
These users are banned and won't be assigned:

@olegmoz/z: This user reported the ticket

Job footprint (restricted area)

Job audit: performer @vatavuk/z/z didn't make CR comments


release note: V3.3.0: STM32 and shields update

@erwango There is a documentation build failure:
/home/runner/work/zephyr/zephyr/doc/_build/src/releases/release-notes-3.3.rst:314:undefined label: 'counter'

@erwango This needs a rebase.

@erwango Need another rebase :(


Light sensor component on FunHouse does not work

Addressed in https://github.com/espressif/arduino-esp32/pull/5711, master includes this fix.
Blocking on https://github.com/adafruit/io-issues/issues/391

Fixed by IO Issues #391.


fzf.fish keybindings no longer work with update

Describe the bug
I had some problems with fzf.fish, so I tried removing it with fisher, and reinstalling. Now no keybindings are in fish_variables.
To Reproduce

Uninstall fzf.fish with fisher, reinstall. Doesn't work, keybindings missing.

Expected behavior
It to work.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Fish: 3.1.2
fzf.fish: v5.5
plugin manager: fisher 4.2.0
terminal: alacritty
OS: arch linux

Additional context
Is there any way to test that it works correctly, even without keybindings?

Nevermind, fisher has a weird behavior on my system where it won't install the function files unless I start a fresh term.

Feel free to open an issue in the Fisher repository so we can look into it. It could be something in your config.fish.

Glad you figured it out!


os/kola: send summary of failures to slack

If any tests fail in a kola run posts a slack message with a list of
failed tests. Parses the list of failed test from the json output via
jq.

Do we want to add the BUILD_ID to the slack messages and/or a link?

Looks like you added the link, yes? It would also be pretty slick to add a link to the _kola_temp.tar.xz as well.

@ajeddeloh I did add the BUILD_URL, I'll look at ways of constructing a link to the _kola_temp.tar.xz

I take it take points to the job status page (e.g. https://jenkins-url.example/job/os/job/kola/job/packet/443/)

Yes (according to their documentation) https://wiki.jenkins.io/display/JENKINS/Building+a+software+project#Buildingasoftwareproject-belowJenkinsSetEnvironmentVariables

@ajeddeloh updated to include a link to both the URL & the _kola_temp (and shortened them so the line is readable instead of just a massive blob of hyperlinks before the failed tests)

Fixed link, dropped extraneous strings & compacted the lines.


add ionicons (https://ionicframework.com/docs/ionicons/)

I'm not sure If I did everything correctly but I tried to prepare the ionicons font for Fontello.
See: https://github.com/publicarray/ionicons.font
The search parameters are missing but I can add them if requested. Also should the css for the font have a icon- prefix? I feel like the ios- and md- prefixes are enough.
I followed https://github.com/fontello/awesome-uni.font so hope everything is fine.
Feel free to fork or otherwise criticise thanks

@puzrin ping

@puzrin ping

This issue is stale. So I'm closing it.


WIP - OCPBUGS-20164: Capability Annotation for Build CRD

Add missing capability.openshift.io/name annotation to the Build cluster configuration CRD. This will ensure that the config object for the Build sub-system is not created when the Build capability is not enabled.
JIRA ID: OCPBUGS-20164

Related implentation PR: https://github.com/openshift/cluster-config-operator/pull/360

/approve
i know you're still working on this but making sure you're not blocked on the api approval portion anyway.

/jira refresh

/jira refresh

/lgtm

Trying to shake Tide loose.
/skip
/retest-required

/close

/reopen

luckily i think we can consider this PR moot since we now need the rename PR anyway.


fixed type error in search component

Issue Number: N/A
Issue Description: N/A
Summary of solution:
In its calls to the backend API, the search component was sending this.props.position.lat and this.props.position.lng for lat and lng fields respectively.  However, if you look at the constructor it is storing the position object in this.state.  Consequently, type errors were showing up in the console preventing the backed even hitting the google api to get results.
Can this issue be closed?
yes
Should any new issues be added as a result of this solution?
No
Have you named your branch in a descriptive way? Remember to name your branch in a unique and descriptive manner in order to properly reflect the issue or feature.
Yes search_component_type_err

@jkwening  sorry about that.  Yeah, the state object having a position key confused me.  I reverted the change i made and removed position from the search component's state.

@Euklidian-Space no problem, there was an error and it is now been resolved. Thanks for contributing!


utils: Fix build-script-impl with distcc.

@swift-ci Please smoke test and merge

This is not necessary anymore because I reverted the offending patch and will incorporate this fix into my next patch.

I'm still not getting any distcc action.

I reverted all my patches to the build-script and build-script-impl. FYI, I can only get distcc to work with user@machine syntax in the .distcc/hosts file instead of just the machine name.

This patch is only useful if the other patches are applied.
2b923f38461390db3bd68d0bcb4bf55108171399
f3b74853697cfac23526cfc904c3c226b7873dcc
e480227d82299c0ba76242b9178e4c0df7cc75f2


Get browser sync

[EDIT from original] Server code and client code can trigger a rebuild/restart of the server, but the browser isn't refreshing. Would like to appease my laziness and make the computer do the work of restarting my browser.

Made use of reload. Really simple to get working.


#89 is not fixed

Hi @DavertMik!
The issue #89 is still not fixed. I tried it once again. I am currently using version 0.3.3 and the tests are not failing when they should!
I have a very simple test once again:
'use strict';
const config = require('../../config.js');
const expect = require('chai').expect;

Feature('Navigation');

Before((I) => {
  I.amOnPage(config.baseUrl);
});
Scenario.only( 'Test', (I) => {
    expect(true).to.be.eql(false);
});

Here is my terminal output:
fabio@fabio-XPS-15-9530:~/Documents/Project/git/core-feature-tests$ node ./node_modules/.bin/codeceptjs run --steps
CodeceptJS v0.3.3
Test root is assumed to be /home/fabio/Documents/Project/git/core-feature-tests

Navigation --
Test
 • I am on page "https://www.test.com"
fabio@fabio-XPS-15-9530:~/Documents/Shore/git/core-feature-tests$

as you can see the tests don't fail they just finish
Here is the output if I change the assertion (expect(true).to.be.eql(true)):
Navigation --
Test
• I am on page "https://www.test.com"
 ✓ OK in 1ms

UPDATE:
This also happens with the CodeceptJS assertions. Like I.see('asdfasdf') when 'asdfasdf' is not on the page, the test suite does not fail properly

Hi @fabioel,
if you run node ./node_modules/.bin/codeceptjs run --steps; echo $?
i guess there is a '0' at the end of your output. So I think my bug (#98) is a duplicate of yours (or v.v.)
Is it?
Cheers
tri

I had the same problem until I manually created the output folder. I thought it was a windows-related problem (#126) since I only could reproduce the problem on a windows machine but it turned out that the output folder was missing when I tried on the windows machine and once I created the folder it starts working again.

Same issue for me on OSx ... it seems like the fix in #94 fixes the reporting, but breaks screenshots. Also in all cases, also the browser isn't shut down correctly on failure. The existence of the output folder only seems to effect recording of the screenshots on the current HEAD for me.
current HEAD

without the output folder existing, no screenshot, tests don't fail properly
with the output folder existing, screenshot recorded, tests don't fail properly

with PR #94 applied

without the output folder existing, no screenshot, tests fail properly
with the output folder existing, no screenshot, tests fail properly

As well as the output folder not existing I also encountered this being caused by helper classes not having a _failed function by default. I've added a no-op version it to the abstract Helper class so people don't need to provide it by themselves.
@fabioel maybe that is what your hitting?
However I've also noticed that it looks like browser.end() doesn't get called when there is a failure in a before block. Meaning you can end up with dangling browser sessions. I might raise that in a different issue.

Calling browser.end in _failed in my helper fixed this for me. Hope it helps anyone else

Fixed in https://github.com/Codeception/CodeceptJS/pull/162


#927 refactor notification state

Work done:

[x] Refactor NotificationsStore from mobx to using state atoms and hooks.
[x] Implement state and logic using valtio over jotai. Reasoning will follow below.
[x] Refactor existing jotai usage to valtio.
[x] Remove unused NavigationStore and ExchangeStore from Store

Why I chose valtio over jotai:

They have very similar apis so refactoring was quite simple.
Valtio supports mutating state outside of react so implementing coroutines that mutate state is much easier. Like the new implementation of the timer logic in the new state/notifications module.
Since we can subscribe to and set state outside of react with valtio its easier to incrementally refactor existing mobx stores when they are mutually reactive/coupled.

Fixes #927
Fixes #938

There is this warning appearing in terminal after I run yarn dev
Warning: useLayoutEffect does nothing on the server, because its effect cannot be encoded into the server renderer's output format. This will lead to a mismatch between the initial, non-hydrated UI and the intended UI. To avoid this, useLayoutEffect should only be used in components that render exclusively on the client. See https://reactjs.org/link/uselayouteffect-ssr for common fixes.

It doesn't appear on staging. Do you know where does that come from?


[release-2.2] upgrade openssl to 1.1.1j and remove nginx-fips

upgrade openssl to 1.1.1j to fix below issues
https://github.com/open-cluster-management/backlog/issues/10245
https://github.com/open-cluster-management/backlog/issues/10243
Remove nginx-fips as fips-2.0.16 doesn't support openssl 1.1.1j  and nginx-fips mode is not used so far.

/lgtm
/approved

/lgtm


Check if the given token is a string

Some model use bytes as their tokens, such as Qwen (see: https://huggingface.co/Qwen/Qwen-7B/blob/ef3c5c9c57b252f3149c1408daf4d649ec8b6c85/tokenization_qwen.py#L136 )

Thank you! Is this solving an issue you encountered? Also would you mind running pre-commit to fix code style?

Thank you! Is this solving an issue you encountered? Also would you mind running pre-commit to fix code style?

Yeap, this solves my issue when I am using the Qwen model family, such as https://huggingface.co/Qwen/Qwen-1_8B.
Here is a code snip:
import vllm
from outlines.serve.vllm import RegexLogitsProcessor
from transformers import AutoTokenizer

tokenizer = AutoTokenizer.from_pretrained('Qwen/Qwen-1_8B', trust_remote_code=True)
prompt = "Answer with A, B, C, or D."

llm = vllm.LLM(model="Qwen/Qwen-1_8B", trust_remote_code=True)
logits_processor = RegexLogitsProcessor(regex_string=r'[ABCD]', llm=llm)
outputs = llm.generate([prompt],
            sampling_params=vllm.SamplingParams(
                max_tokens=5, logits_processors=[logits_processor]
            ),
        )

I have also found out that the Qwen model family use a non-standard tokenizer implementation, i.e., they return byte sequences as their tokens, not strings. However, I think it is always benifical to consider more corner cases.


Doesn't [^}^{] match on any character that is not '{}^'?

https://github.com/mkaranasou/pyaml_env/blob/1ce85018818d5a553c3c499347195a1ec22d47f2/src/pyaml_env/parse_config.py#L37
I'm new to python, but not programming or regexes. I'm confused by this construct:
default_sep_pattern = r'(' + default_sep + '[^}^{]+)?'

Doesn't that last part allow for any character except '{', '}', or '^'?
If not, what is it doing?
If so, wouldn't '[^}]' be a better construct?

Hi, yes, it matches all characters but  {, }, or ^.
In terms of having more flexible default values, since [^}] would mean all characters but }, I think it would indeed be a better construct. Trying to think whether allowing { would backfire somehow :thinking:
Do you have a case where { or ^ needs to be in the default value?

While I can come up with some scenarios, I don't have a specific use case. I was just trying to figure out where my understanding and python differ. There have been a few gotchas, so I'm not assuming anything. :)
Allowing '^' shouldn't be a problem, but the only scenarios I can think of that would allow '{' would have to allow for an escaped '}'. Probably more work than is needed, at least for a 'what if'.


Update requirements.txt

Added utils module dependecy in requirements.txt

Do we need olefile module as a dependency?

@HarshKhatore I don't know, the person who created the requirements.txt first had kept it, so I guess its required. Anyway, you know that your contributions for KWoC will be counted after 1st December, right? I'm merging this anyway.

Yes, I am aware of that.
Yes, please merge this.


Add mvn versions note to DEVELOPER

+1

Applied this separately from the PR since my fork was deleted since I opened this.


Always on top doesn't work properly with mulitple screen

Microsoft PowerToys version
"0.58"
Running as admin

[X] Yes

Area(s) with issue?
Always on Top
Steps to reproduce
If you are using always on top with multiple screen
When you switch the window who's supposed to stay AOT from one screen to another, it lost the AOT effect and became like a normal window ( but it keep the border)
✔ Expected Behavior
The window to stay on top
❌ Actual Behavior
The window lost top focus when i clicked anywhere
Other Software
No response

Does it happen with some specific software? It's a known problem for OneNote.
/needinfo


[Vendor] update macaron related

gitea.com/macaron/session to a177a270
gitea.com/macaron/macaron to 0db5d458
gitea.com/macaron/cache to 905232fb
gitea.com/macaron/i18n to 4ca3dd0c
gitea.com/macaron/gzip to efa5e847
gitea.com/macaron/captcha to e8597820

Codecov Report

Merging #13409 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13409      +/-   ##
==========================================
+ Coverage   42.11%   42.13%   +0.01%
==========================================
  Files         691      691
  Lines       76002    76002
==========================================
+ Hits        32008    32020      +12
+ Misses      38744    38732      -12
  Partials     5250     5250

Impacted Files
Coverage Δ

modules/indexer/stats/db.go
52.17% <0.00%> (-8.70%)
:arrow_down:

modules/git/utils.go
73.77% <0.00%> (-3.28%)
:arrow_down:

modules/log/event.go
59.90% <0.00%> (-1.89%)
:arrow_down:

modules/git/repo.go
46.19% <0.00%> (-0.51%)
:arrow_down:

services/pull/pull.go
41.27% <0.00%> (+0.49%)
:arrow_up:

modules/queue/workerpool.go
60.00% <0.00%> (+1.22%)
:arrow_up:

services/pull/patch.go
55.68% <0.00%> (+1.70%)
:arrow_up:

models/unit.go
49.31% <0.00%> (+2.73%)
:arrow_up:

services/pull/check.go
51.82% <0.00%> (+2.91%)
:arrow_up:

services/pull/temp_repo.go
29.78% <0.00%> (+3.19%)
:arrow_up:

... and 1 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 b687707...e09613c. Read the comment docs.


Imagery Layers Sandcastle fails to load Bing Maps Road

Go to https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/?src=Imagery Layers Manipulation.html
Click "Bing maps road" from the drop down. Observe the blue globe. Observe:
An error occurred in "BingMapsImageryProvider": An error occurred while accessing https://dev.virtualearth.net/REST/v1/Imagery/Metadata/Road?jsonp=loadJsonp622672&incl=ImageryProviders&key=undefined.

I think that Sandcastle just needs to be updated since it works fine if you go to any Sandcastle and select Bing Maps Roads from the imagery widget?

Hi @OmarShehata I can work on this issue, where is the file located?

@abuDarda97 this can be another easy fix - just go here:
https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Apps/Sandcastle/gallery/Imagery Layers Manipulation.html#L118-L121
Replace that section with ion imagery, which can be created using createWorldImagery. An example of that here: https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/?src=Imagery Layers.html
Then just double check it looks the same as the default imagery in Cesium, and open the PR!

Cam up again here: https://github.com/CesiumGS/cesium/issues/7128#issuecomment-629031364
Our examples should be the last place we let trivial bugs linger, it would be good if someone opened a PR to fix this.

In my case the key expired, try generate a new one


[VS Code] Joining collaboration session: The specified value isn’t a valid Live Share URL. Please check the link provided by the host and try again.

Error:
Joining collaboration session: The specified value isn’t a valid Live Share URL. Please check the link provided by the host and try again.
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.1071

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.28.2

platformName
Linux

platformVersion
4.10.0-42-generic

Do you know what was the link you pasted and how you got it? Some email or chat apps (like Hangouts) may inject a redirect into a URL that causes it to no longer be recognizable to Live Share.


[Security, dependency] Usage of golang.org/x/net with version v0.0.0-20220722155237-a158d28d115b which is marked as CVE-2022-27664

Hi,
when I run trivy on my project which is using client-go in k8s.io/client-go v0.25.0 I got notification about https://avd.aquasec.com/nvd/2022/cve-2022-27664/
Best to upgrade to 0.0.0-20220906165146-f3363e06e74c

Hey! I can no longer find that revision 0.0.0-20220906165146-f3363e06e74c and I can still see the CVE in 0.25.2. Is there a plan to fix this as part of 0.25.3? I've scanned 0.25.3-rc0 and it is also present there

latest released tags for 1.23 - 1.26 reference a version of the golang.org/x/net dependency that addresses this CVE


Add simple Pythom examples

Thank you @chezou, very good feedback. (Had to accept individual suggestions, the “add suggestion to a batch” button was grayed out)
If you want me to merge these commits, let me know.

Looks good to me with notification around py> operator is not GA yet.

@chezou Added a disclaimer + squashed commits.

@kiyoto ship it 👍

Thank you!


Can you use Compose for desktop to achieve the effect in the picture?

Can you use Compose for desktop to achieve the effect in the picture? Each part is a separate exe. The ToolBar above occupies the upper part of the computer desktop, and other application icons automatically move down to make way for them.

But building a single application with all these separated windows is possible. There's a flag that makes window bars transparent to give you control over how bars are styled , you just need to position and size the windows, This will take some time but accomplishable
You could build each window in a separate program, Have a common launcher that launches all these windows. That launcher would need to know location of all the executables. So you should be able to accomplish this, Take some time to read the docs though

Are you referring to the setting of the transparent attribute? I set the property to true, but the part that is set to transparent will block the operation.
https://github.com/JetBrains/compose-multiplatform/assets/19502034/00941824-868e-49ff-abc8-be68154fbc1b

What do you mean block the operation, I haven't tried this, If there's a bug this you should change the title to reflect a bug is the issue

Can you try and create multiple windows in the application block

Thank you for your suggestion. I will try to use a multi-window program to achieve the above requirements.

@changhexuefei I think I understand what you mean by "blocking the operation".
Potentially, the window you are creating is not "wrapping" its content, e.g. the size of the window is bigger than its content, making it capture the clicks on it, even if you can't see (again, because it's transparent). I can see it from a frame of your video where your mouse passes where one of the borders of the window should be and it transforms to a resize cursor:

Try creating the window with WindowState(size=DpSize.Unspecified), and that should resize the window to be exactly the content's size, achieving the desired effect.

I set the WindowState(size=DpSize.Unspecified) property, clicked the exit button, and when I needed to pop up the query box, it became like this.

I think you should be able to create the dialog outside of the parent Window.
Something like this (this one uses a preview API but you should be able to simply use a Dialog composable for doing the same):
@OptIn(ExperimentalMaterialApi::class)
fun main() = application {
    var showDialog by remember { mutableStateOf(false) }

    Window(onCloseRequest = ::exitApplication, state = rememberWindowState(size = DpSize.Unspecified)) {
        App(onClick = {
            showDialog = true
        })
    }

    if (showDialog) {
        AlertDialog(
            onDismissRequest = { showDialog = false },
            dialogProvider = UndecoratedWindowAlertDialogProvider,
            text = { Text("Yay or nay?") },
            confirmButton = {
                Button(onClick = { showDialog = false }) { Text("Yay") }
            },
            dismissButton = {
                Button(onClick = { showDialog = false }) { Text("Nay") }
            }
        )
    }
}

https://github.com/JetBrains/compose-multiplatform/assets/4479527/cfa846b4-08a7-46e3-9eb5-1a8617a05e81
As you can see, the dialog is outside of my main window, created as its own [borderless] window.

Also, for more information about Dialogs, here it is a useful page https://github.com/JetBrains/compose-multiplatform/tree/master/tutorials/Window_API_new#dialogs

ok，thank you

Hello, would you like to ask, what are the attributes that are not circled in AlertDialog? Which version of AlertDialog are you using?

Why did AlertDialog in compose.version=1.5.0 remove "dialogProvider=UndecoratedWindowAlertDialogProvider"?

In Compose multiplatform 1.5.0 and above, DialogWindow is used to replace the "dialogProvider=UndecoratedWindowAlertDialogProvider" attribute in AlertDialog to achieve this requirement.

@changhexuefei Can we close this issue as resolved now?

yes

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Updating the Meteor Package

Could the meteor package be kept up to date with the current release of Bootstrap?
This is a very popular package in the Meteor Community and Bootstrap 5 looks like it would gain a following among devs using Meteor.
https://atmospherejs.com/twbs/bootstrap

@mdo: do you have access yourself? If so we need to add this in the release checklist.

I'm sorry, Im not sure I'm following the question? do i have access to what?

My message was directed to @mdo, not you @BrianMulhall.


Classes instead of colors

On every hover or an another action, the color changes via "style"-tag. This is nice, but there should be an additional class, so that the styling it could be overwritten via css.

@iCodr8
I have an option [theme] to customise the color of the picker. If there was an additional class to overwritten the style, it would not have any effect as the javascript styling has a higher priority.
Are you suggesting that I should not customise the picker color via javascript?
Cheers.

The problem is that the theme option is not enough for me. I need more flexibility than to only change the color. So it would be nice to add an additional class to that parts where the style tag is used.
For example:
.picker-day is on every day, but on today and the current selected item is not an extra class. Only the style tag is different.
So it would be nice to have for example the classes picker-day picker-day-today and picker-day picker-day-current on the class tag.

@iCodr8
I got what you mean.
Thanks for the suggestion. I will look at it in next couple days.

@iCodr8
I added picker-day-today for today, picker-day-selected for the day you selected, picker-month-current for the month displayed, picker-year-current for the year displayed.
What else classes should I add?

I think that's enough.

@iCodr8
Update to version 4.1.2, it should be alright.