Added new 'Wonkey' programming language

Wonkey is a fork of Monkey2 programming language.

[x] I am adding a new language.

[x] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?q=extension%3Awx+NOT+nothack

[x] I have included a real-world usage sample for all extensions added in this PR:
[x] I have included a syntax highlighting grammar: https://github-lightshow.herokuapp.com/

Wonkey is a fork of Monkey2 programming language.

🤔 and it appears to have only been forked within the last week which would suggest it would be no where near meeting the currently documented minimum requirement for inclusion:

We try only to add new extensions once they have some usage on GitHub. In most cases we prefer that extensions be in use in hundreds of repositories before supporting them in Linguist.

Many of the files returned by your search don't look to be "Wonkey" and with a little refinement of the search, I think every single "Wonkey" file can be attributed to a single repo: https://github.com/wonkey-coders/wonkey
Unfortunately, this makes it far too early to even consider adding support to Linguist.

Wonkey is a fork of Monkey2 programming language.

🤔 and it appears to have only been forked within the last week which would suggest it would be no where near meeting the currently documented minimum requirement for inclusion:

We try only to add new extensions once they have some usage on GitHub. In most cases we prefer that extensions be in use in hundreds of repositories before supporting them in Linguist.

Many of the files returned by your search don't look to be "Wonkey" and with a little refinement of the search, I think every single "Wonkey" file can be attributed to a single repo: https://github.com/wonkey-coders/wonkey
Unfortunately, this makes it far too early to even consider adding support to Linguist.

Wonkey is a fork of Monkey2 programming language.

🤔 and it appears to have only been forked within the last week which would suggest it would be no where near meeting the currently documented minimum requirement for inclusion:

It's a fork with extension and grammar changes.

We try only to add new extensions once they have some usage on GitHub. In most cases we prefer that extensions be in use in hundreds of repositories before supporting them in Linguist.

Many of the files returned by your search don't look to be "Wonkey" and with a little refinement of the search, I think every single "Wonkey" file can be attributed to a single repo: https://github.com/wonkey-coders/wonkey

I targeted search into Wonkey repo

Unfortunately, this makes it far too early to even consider adding support to Linguist.

I need syntax highlighting on Github :)

Wonkey is a fork of Monkey2 programming language.

🤔 and it appears to have only been forked within the last week which would suggest it would be no where near meeting the currently documented minimum requirement for inclusion:

It's a fork with extension and grammar changes.

We try only to add new extensions once they have some usage on GitHub. In most cases we prefer that extensions be in use in hundreds of repositories before supporting them in Linguist.

Many of the files returned by your search don't look to be "Wonkey" and with a little refinement of the search, I think every single "Wonkey" file can be attributed to a single repo: https://github.com/wonkey-coders/wonkey

I targeted search into Wonkey repo

Unfortunately, this makes it far too early to even consider adding support to Linguist.

I need syntax highlighting on Github :)

I targeted search into Wonkey repo

That's the problem... we need popularity demonstrated across the whole of GitHub. Your repo appears to be the only one using this language. One user is not sufficient to qualify for inclusion.
This PR will not be accepted until there is wide-spread usage across the whole of GitHub.

I targeted search into Wonkey repo

That's the problem... we need popularity demonstrated across the whole of GitHub. Your repo appears to be the only one using this language. One user is not sufficient to qualify for inclusion.
This PR will not be accepted until there is wide-spread usage across the whole of GitHub.

This PR should be closed as being far off being able to be added.


[tou.tv] Unable to download JSON metadata: HTTP Error 426: Upgrade Required

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Canada
Provide a description that is worded well enough to be understood
Whenever I try to download any video from tou.tv (I have tried several), it gives me the HTTP 426 error, saying an upgrade is required.  I noticed that there were a few issues with CBC Gem and the 426 error, but I could not find any currently-open issues for tou.tv that mention the 426 error.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['https://ici.tou.tv/vieillir-autrement/s01e01', '-o', 'D:\\Downloaded Audio-Video Tracks\\ViaYouTubeDL\\tou.tv\\Custom\\%(title)s-%(id)s.%(ext)s', '--get-url', '-vU']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out cp1252 (No VT), error cp1252 (No VT), screen cp1252 (No VT)
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [12b248ce6] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.22621-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 6.0-essentials_build-www.gyan.dev (setts), ffprobe 6.0-essentials_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.02.02, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.2, sqlite3-3.35.5, urllib3-2.2.1, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1820 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[debug] Using fake IP <IP_ADDRESS> (CA) as X-Forwarded-For
[tou.tv] Extracting URL: https://ici.tou.tv/vieillir-autrement/s01e01
[tou.tv] vieillir-autrement: Downloading JSON metadata
[tou.tv] vieillir-autrement: Unable to download JSON metadata: HTTP Error 426: Upgrade Required (caused by <HTTPError 426: Upgrade Required>)
  File "yt_dlp\extractor\common.py", line 734, in extract
  File "yt_dlp\extractor\toutv.py", line 63, in _real_extract
  File "yt_dlp\extractor\common.py", line 1133, in download_content
  File "yt_dlp\extractor\common.py", line 1093, in download_handle
  File "yt_dlp\extractor\common.py", line 954, in _download_webpage_handle
  File "yt_dlp\extractor\common.py", line 903, in _request_webpage
  File "yt_dlp\extractor\common.py", line 890, in _request_webpage
  File "yt_dlp\YoutubeDL.py", line 4142, in urlopen
  File "yt_dlp\networking\common.py", line 117, in send
  File "yt_dlp\networking\_helper.py", line 208, in wrapper
  File "yt_dlp\networking\common.py", line 337, in send
  File "yt_dlp\networking\_requests.py", line 366, in _send
yt_dlp.networking.exceptions.HTTPHTTP Error 426: Upgrade Required
An error occured

Devs: see https://github.com/yt-dlp/yt-dlp/issues/6999#issuecomment-1974844578


[go_router] Handle errorBuilder with Error Type

copied from https://github.com/csells/go_router/issues/355
Description
When an error occurs when navigation eg the route does not exist or eg, queryParameter or params is invalid, or also problems about the assertion can be distinguished when in errorBuilder.
So when the error handle in errorBuilder I can render a specific page according to the error type.
Currently

For the current version I only get exception info when there is an error when navigating, and I don't know what type of exception it is and what the error is
Expected
The following might help an example of implementing an error handle:
enum GoRouterErrorType { notFound, paramInvalid, queryParamInvalid, ... }

 errorBuilder: (context, state) {
      // `state.error` the type is `GoRouterErrorType`
      // `state.exception` the type is `GoRouterException` (custom exception)
 },

is there a way to handle this exception? so I could perform a custom action after catching it.

is there a way to handle this exception? so I could perform a custom action after catching it.

OP clarified in this comment that they would like GoRouter to use more specific Exception and Error types, such as GoRouterException and GoRouterNotFoundException


close opened kubectl processes when extension is deactivated

This PR should fix the problem that we have with "all" kubectl processes that are left opened if vscode is shutted down before those processes are closed by the user.
The only issue i wasn't able to fix and i don't know why is related to the observable that are opened when following pod logs.
What i tried to do was to use an abortController and put the abortSignal inside the shell execOptions (as described in the documentation https://nodejs.org/api/globals.html#class-abortsignal). Ideally when the signal is sent, the process should be killed. It didn't work.
So i tried another way which is less elegant, https://github.com/Azure/vscode-kubernetes-tools/pull/1001/files#diff-68da841fbea6534a636dc451cf3178e52d0d0c0beab917207420fd7127ed071bR284 , but it doesn't always work. Sometimes it doesn't close all processes.
@itowlson Can you please try following logs of 3-4 pods at the same time, closing vscode and see the kubectl process that are not closed? Do you have any? Of course if you know the reason or have a fix feel free to edit the PR yourself. After 1 week trying it, i'm giving up for some days :laughing:

Thanks heaps for looking into this.  From what you're seeing, are we hitting the code to kill the processes?

I'll have a tinker but if you know which of these is the issue then that would help narrow it down.
Also, do we know if this issue happens on all operating systems or only a particular one?  (Again, if you don't know offhand then I can have a play.)

Thanks heaps for looking into this. From what you're seeing, are we hitting the code to kill the processes?

I'll have a tinker but if you know which of these is the issue then that would help narrow it down.
Also, do we know if this issue happens on all operating systems or only a particular one? (Again, if you don't know offhand then I can have a play.)

We completely missed the code to kill processes. This patch adds it but it doesn't always work. Atleast when we don't have a direct reference to a process. If we are the ones who create that process, it's easy, i kill it directly, i have its id.
But, there are cases when we only have access to the stdout of a process because it is spawned inside a library. I tried to send an abortSignal within the execute options and, ideally, this should be good enough to tell that process to abort when i want. Sadly, it doesn't work.
I think it is a problem related to all OSes but not 100% sure.

@itowlson Removed the abort controller which wasn't working as expected. Now it should work fine. I still keep it on draft as i don't have much time to test it extensively today but you can give it a look.

@itowlson tested it. It looks like all leaked kubectl processes are killed properly now. Please give it a look. Thanks!!


setup a model and schema - REDO

This is great! Did you get a chance to test this and see if you could get any data back from your DB?


Fix a bunch of issues

The mysterium transformer wasn't being initiated which meant that its weird patches (wow this could use a mixin conversion) were never applied. Instead of having 2 coremods I merged the mysterium fix loader doohickey into the Backhand coremod and added the two missing transformers.
This fixed the majority of the problems and
Fixes https://github.com/GTNewHorizons/Backhand/issues/4, fixes https://github.com/GTNewHorizons/Backhand/issues/8, fixes https://github.com/GTNewHorizons/Backhand/issues/5 and this also fixes the problem mentioned in https://github.com/GTNewHorizons/Backhand/issues/2
I also fixed some incorrect interactions with EFR's totem of undying, tipped arrows & fireworks which
Fixes https://github.com/GTNewHorizons/Backhand/issues/9, fixes https://github.com/GTNewHorizons/Backhand/issues/6, fixes https://github.com/GTNewHorizons/Backhand/issues/7
I also ditched the access transformer class because it was both pointing to the wrong file and not necessary

Have you considered forking the 1.4.1 version?

No

We are actually based on the 1.4.1 version if you check the commit history

We are actually based on the 1.4.1 version if you check the commit history

but 1.4.1 doesn't have most of the aforementioned issues..


db_reader writer speed issue

Annie reported db_reader writer speed issue when processing thousands of notes. I think it due to the single document handling, it would be a lot faster if reading and writing in batches. I will be even faster if asynchronizing the IO and the NLP processing. For scaling use, it might be easier to port the pipeline into pyspark framework instead. So for now, I think implement a batch IO would help for lightweight experiment.

done
https://github.com/medspacy/medspacy/pull/301


Add update methods for timeseries and metadata

Make it easy to update a timeseries or metadata dictionary with new data.

Closed by #40


Can't connect to CRM online managed APIs

When creating Logic App in Nort Europe , and trying to add CRM Online connector I'm Getting this error:
<<MissingRegistrationForType. Failed to create connection for connection id '/subscriptions/500dea4d-3a9c-4290-8379-54274cf6ed3d/providers/Microsoft.Web/locations/northeurope/managedApis/dynamicscrmonline/connections/dynamicscrmonline'. >>
In North America its trying to redirect to : " https://ema1.exp.azure.com/ema/default/auth" , but the web windows in hang on Redirecting...

All connectors to services not singing. For examle I try "Office 365 Outlook - Create connect", and press "sign in", window (https://ema1.exp.azure.com/ema/default/auth) opened in hang on Redirecting... In main window error "Cannot set property message of [object Object] which has only a getter"


[BUG] Simple bot gets stuck after a day or two

Describe the bug.
Hiya! I've built a pretty simple bot using interactions.py, and it works extremely well for my use case, but I'm having trouble getting the bot to stay online.
I've turned on some logging and it seems to get stuck after messages like this:
2022-07-22T14:52:35.058970301Z app[web.1]: 07/22/2022 02:52:35 PM [DEBUG] (gateway) {"op": 1, "d": 1}
2022-07-22T14:52:35.059016619Z app[web.1]: 07/22/2022 02:52:35 PM [DEBUG] (gateway) HEARTBEAT
2022-07-22T14:52:35.087019104Z app[web.1]: 07/22/2022 02:52:35 PM [DEBUG] (gateway) None
2022-07-22T14:52:35.087030590Z app[web.1]: 07/22/2022 02:52:35 PM [DEBUG] (gateway) HEARTBEAT_ACK
2022-07-22T14:53:16.351026755Z app[web.1]: 07/22/2022 02:53:16 PM [DEBUG] (gateway) {"op": 1, "d": 1}
2022-07-22T14:53:16.351077315Z app[web.1]: 07/22/2022 02:53:16 PM [DEBUG] (gateway) HEARTBEAT
2022-07-22T14:53:16.390594024Z app[web.1]: 07/22/2022 02:53:16 PM [DEBUG] (gateway) None
2022-07-22T14:53:16.390606161Z app[web.1]: 07/22/2022 02:53:16 PM [DEBUG] (gateway) HEARTBEAT_ACK
2022-07-22T14:53:57.642993887Z app[web.1]: 07/22/2022 02:53:57 PM [DEBUG] (gateway) {"op": 1, "d": 1}
2022-07-22T14:53:57.643029579Z app[web.1]: 07/22/2022 02:53:57 PM [DEBUG] (gateway) HEARTBEAT
2022-07-22T14:54:38.927076841Z app[web.1]: 07/22/2022 02:54:38 PM [DEBUG] (gateway) HEARTBEAT_ACK missing, reconnecting...

I don't get messages after that, and restarting the bot seems to fix it for a day or two and then it is back to that.
My bot is really simple, so here is my usage:
@bot.command(
    name="print",
    description="Prints a message to the Paper Crumpler 🖨",
    scope=discord_guild_id,
    options=[
        interactions.Option(
            name="text",
            description="Something cool you want printed!",
            type=interactions.OptionType.STRING,
            required=True,
        ),
    ],
)
async def print_message(ctx: interactions.CommandContext, text: str):
    nick = ctx.author.nick or ctx.author.user.username
    logging.info(f"Printing message: [{nick}@discord] {text}")
    send_print_server_message(text, nick)  # Updates a record in a Firebase DB
    await ctx.send(f"Printing message: `[{nick}@discord] {text}`")

and then I'm just kicking off the bot with bot.start().
Currently using discord-py-interactions==4.2.1, but it's been an issue on a couple of older versions and unstable as well. Running inside a Docker container FROM python:3.10.4-slim-bullseye.
List the steps.

Create a bot that can respond to a single command
Run bot using bot.start()
Wait a day or so
Eventually the bot stops responding and the last logged message is (gateway) HEARTBEAT_ACK missing, reconnecting...

What you expected.
The application appears to continue to run but never successfully reconnects. If it crashed or otherwise exited, I'd be able to easily restart the application, but since it gets stuck like that, it's hard to detect.

What version of the library did you use?
release
Version specification
4.2.1
Code of Conduct

[X] I agree to follow the contribution requirements.

Tried using git+https://github.com/interactions-py/library.git@unstable yesterday at the suggestion of Toricane in the Discord help channel, but it got stuck again today.

https://github.com/interactions-py/library/blob/e636d4e7d24d60604c371c5d92a054df269c1c13/interactions/api/gateway/client.py#L150-L153
This can be seen here where we decide to log it before the action, as recommended in Python logging nature.
https://github.com/interactions-py/library/blob/e636d4e7d24d60604c371c5d92a054df269c1c13/interactions/api/gateway/client.py#L155-L161
Our restarting method is simply cleaning up the heartbeater loop to avoid accidental misfires and to wipe out the WebSocket connection for potential socket leaks.
https://github.com/interactions-py/library/blob/e636d4e7d24d60604c371c5d92a054df269c1c13/interactions/api/gateway/client.py#L182-L195
Here with our connection establisher method, you can see that we'll just attempt to recursively reconnect to something if there's existent data supplied to it. There's zero logging, and we only raise an Exception if we've detected a non-1000 code that is client error-bound, not from the server.

Thanks for detailed info! I wonder if it's getting stuck in that while not self._closed: loop then? My bot never reconnects, unless I restart the application. I can try to attach a debugger to see if that's where it's stuck, but I might have to restart the Docker container to do that.

interactions.py doesn't properly close the yielded WS connection when it ends, which is supposed to be done to initiate an exiting call to its context manager. Since we're hard wiping it out to be None, we're essentially not telling aiohttp to cache the fact that it's gone, so nothing is changing. We've rewritten a value on our end without informing the module in control, which proves to be problematic over time.
I believe the fix to this would unironically be rewriting the way Gateway connections are established and reconnected. This is a lot of old code I admittedly wrote to work in a very jank manner. Here's a comparison to another project I am working on that imposes very similar sequence patterns as interactions.py:
            case _GatewayOpCode.INVALID_SESSION:
                logger.info(
                    "Our Gateway connection has suddenly invalidated. Checking reconnection status."
                )
                await self._dispatch("INVALID_SESSION", InvalidSession, payload.data)

                if bool(payload.data):
                    logger.debug(
                        "The Gateway has told us to reconnect. Resuming last known connection."
                    )
                    await self._resume()
                else:
                    logger.debug(
                        "The given connection cannot be reconnected to. Starting new connection."
                    )
                    self._meta.session_id = None
                    await self._conn.aclose()
                    await self.reconnect()
            case _GatewayOpCode.RECONNECT:
                logger.info("The Gateway has told us to reconnect. Resuming last known connection.")
                await self._dispatch("RECONNECT", Reconnect)
                await self._resume()

This is a more structurally sound manner to checking. _conn here is what is a yielded WebSocketConnection given to us after a connection is established. This properly makes an asynchronous closing call to the module is informed that the connection has ended. Additionally, in contrast to interactions.py, proper checks are made to see if things can actually be reconnected to or not instead of purely relying on the client error-bound closing codes.
    async def connect(self):
        """Connects to the Gateway and initiates a WebSocket state."""
        self._stopped = False
        ...

        async with open_websocket_url(
            f"{__gateway_url__}?v={self._meta.version}&encoding={self._meta.encoding}"
            f"{'' if self._meta.compress is None else f'&compress={self._meta.compress}'}"
        ) as self._conn:
            self._closed = self._conn.closed

            if self._stopped:
                await self._conn.aclose()
            if self._closed:
                await self._conn.aclose()
                await self.reconnect()

            while not self._closed:
                data = await self._receive()

                if data:
                    await self._track(data)

    async def reconnect(self):
        """Reconnects to the Gateway and reinitiates a WebSocket state."""
        self._closed = True
        self._heartbeat_ack = False

        if self._closed:
            await self.connect()
        else:
            logger.info("Told to reconnect, but did not need to.")

While somewhat more complicated, this will make the proper calls for resetting attribute fields where needed.
The Gateway code that interactions.py uses for connection was written all the way back when 4.0.0-alpha was in early stages, which would be around September-October 2021. I believe the best course of action for this library, as already mentioned, is a rewrite of the logic.

@i0bs Thanks for taking the time to write the long detailed explanation of what you believe is going on.  In case you were wondering, this issue is easily reproducible and explainable. It looks like 5.x is a complete rewrite so this is probably pointless to mention.
Steps to reproduce (occurs on both 4.3.4 & 4.4.0):

run client
wait for client to connect and heartbeats to start
disconnect internet (ifdown/unplug, etc)
reconnect internet
will hang forever

the line where the client seems to be hanging is within the gateway.client.__reconnect method:
https://github.com/interactions-py/interactions.py/blob/8bc3e1d28f9ee1956ee2e8201bf4335a0e2071db/interactions/api/gateway/client.py#L942
self.options is hardcoded and not configurable:
https://github.com/interactions-py/interactions.py/blob/8bc3e1d28f9ee1956ee2e8201bf4335a0e2071db/interactions/api/gateway/client.py#L182-L190
This corresponds to aiohttp client, reference here: https://docs.aiohttp.org/en/stable/client_quickstart.html#aiohttp-client-timeouts
In particular, the default is for the connect timeout to be disabled, which is unfortunate because this connection will never connect if the internet connection is dropped. Even after the internet connection is re-established, the client remains stuck here.
The only solution for me was to rely on the newly added in on_disconnect event.  Hopefully this helps anyone else that may be experiencing the same issue.

I suggest migrating to version 5 of this library which properly handles internet connection being lost.
https://github.com/interactions-py/interactions.py/tree/5.x


open /dev/urandom when entropy context is initialized and keep open

Proposed patch for #322. The user simply calls entropy_init before entering chroot.

Automatic CI verification build not done, please verify manually.

At first glance, looks good to me.
In order to accept your contribution, we'll request that you sign our contributor license agreement. Can you please get in touch with @sbutcher-arm or @pjbakker regarding the details? Thanks!

Happy to do the CLA but I'm not sure how to get in touch (neither Sam nor Paul have their email addresses public). Either of them can contact me at the address on my profile. Thanks.

I just sent an email to the three of you.

@jkarneges Thank you for your contribution! I've made a new pull request #1545 which builds on your work, fixes the case of getrandom() available at build time but not at runtime, and makes a few related improvements.

This PR hasn't been active in a long time, and development work has been pursued in PR #1545. I am therefore closing this PR, and further work can happen there.
@jkarneges - if you'd like to pick this up again, please raise a new PR or push to the other PR #1545.


Select Parent as individul node don't select all child element when parent is selected

I want that parent should be selected as separate node don't select all child when parent is selected. Is there any props available like this?

Hi @Ajay-Bacancy, it is a nice feature. Do you mean that every node should be independent? If we check the parent node - children will not be selected, so if you check all children - the parent node will not be selected as well? Did I understand correctly? If it works this way, I can implement it.

This is something I would like too as an optional configuration option, as you just described in your comment @dipson88

@masiorama I will try to include this feature in the next release.

I was also searching on a way to do something similar. In our application, we allow parent nodes to be assigned to different entities. So, in order to filter by the parent node, I need to find parent node myself.
But if I can just select the parent node without selecting the children nodes automatically then it would solve the problem.
When is the next release by the way? Is there anything planned?

Now I can't say the exact date of the next release. But I've investigated this feature. This leads to more complicated code, but if people need this, I will set higher priority to this feature.

For us it would be a mandatory addition because some clients need this.
+1 high priority for me, thanks

+1 high priority for me, i need the same things.

An option to include/exclude the parent in the searchable items would be a very cool feature.

Thanks! I will check it out asap and give a feedback!


add to_bytes, for consuming uuid

Adds a method to consume Uuid and return inner bytes; alternative to referencing as_bytes. Thought it might be useful

Perhaps the as_bytes method could be changed to return &[u8; 16]? That way you could directly load from that (as it's a Copy type) to get the underlying bytes.

I don't know if that's considered a breaking change. I wanted a way to store the uuid as bytes without copying the bytes themselves, so just move semantics, it seemed like a good idea to just consume the uuid in this case. I'm not sure if that fits the library because it's awfully close to as_bytes, it's okay if it doesn't but thought I'd PR anyways :)

Due to coercions it's likely to not be a breaking change, but yeah there are subtle cases it could perhaps cause problems. If this should specifically be a consuming conversion then we probably want to call it into_bytes rather than just to_bytes, but the Uuid type is Copy anyway so you can't really consume an instance of it (it's just always copied)

Ok, with https://github.com/rust-lang-nursery/uuid/pull/56 on the horizon as a breaking change I'm gonna go ahead and change this to return a &[u8; 16] in as_bytes as another breaking change. Thanks for the PR though!


Unable to find vcvarsall.bat

When running pip install -r requirements.txt in the p309 env, I get the following error:
          raise DistutilsPlatformError("Unable to find vcvarsall.bat")
      distutils.errors.DistutilsPlatformError: Unable to find vcvarsall.bat

When I search online for related issues, I get posts related to Visual Studio. So, I'm guessing that my Build Tools is of the wrong version.
I tried to match my Visual Studio install to the python version (3.9). So, I'm running Visual Studio 2019. However, when I try to install Build Tools from the link, I'm automatically directed to Build Tools for 2022. I can't find a link that lets me get Build Tools for 2019. Moreover, online, they say that 2022 should be backward compatible.
Any suggestions?

Hi,
That’s weird. Can you install the Microsoft Visual C++ 2019 Redistributable from here


fixed subscript, superscript, italics, bold tags for elsevier

https://github.com/adsabs/ADSIngestParser/issues/99

@seasidesparrow please note that the changes to base.py did modify the parsing for other parsers like jats. I haven't updated all of them yet, because I'd like your feedback to see if this is working well. Once we settle on a solution, I can modify other files.


Config file support

There are many config options like image size, aspect ratio, camera angles etc, which can be supplied from a config file for simpler handling of the program.

This is addressed with #8


add DirFS option to filter xattrs

allow consumers the option to filter certain xattrs that may be irrelevant for various reasons (such as OS compatibility)
this materializes downstream in something like melange (through apko) as:
❕ aarch64   | buildImage failed: installing apk packages: installing packages: installing ca-certificates-bundle: unable to install files for pkg ca-certificates-bundle: error setting xattr com.apple.provenance on etc/ssl/cert.pem: file does not exist

follow up PRs will be on apko and melange to pull this change in that will essentially just ignore macos specific xattrs: strings.HasPrefix("com.apple.")

I'm not attached either way to where the filters should be attached to dirFS or memFS (or both?)
let me know and I'll move things around accordingly!

This seems unnecessary in practice, further investigation showed the actual problem was with symlinks.  See #101.


Maintain dirtiness and use retained engine layers

For #21756

Github code review is broken for this PR...


Feature Request: Custom Model Managers

It'd be cool to be able to subclass ModelManager so that I can write custom ones (I'm working on a project with a custom model class). While it's easy to subclass right now, there's no easy way to add a custom ModelManager to ModelStore. So, some method like model_store.register_model_manager(my_custom_manager) would be nice.
Hmm actually after writing the above, I realize I could just append my custom class to model_store._managers which is a bit hacky but feasible. Anywho, I'll leave this issue up as food for thought!

That's a cool idea! I've added it to my list of things to look into. Is this for an in-house library, or an ML library that is just not supported by modelstore yet?
One thing that modelstore already has is a way of saving "models" that are just paths (PosixPath or str - and just files, not directories). Something like:
# Your own code
model = train_my_custom_model()
model_path = save_custom_model(model)

# Upload by pointing to the file
meta_data = model_store.upload(_DOMAIN_NAME, model=model_path)

But model_store.load() cannot work with this approach 😕 , so registering a custom manager could be a great alternative

Yeah, this is for an in-house library. Uploading the serialized model file is a great solution. While load() doesn't work, download() does which is perfectly fine for now!


resolves race condition in shared variable node.segment

the race condition occured when while a segment was being proved and encoded, other goroutines modify the node.segment, appending it with replicas.
simplifying logic by assuming that the segment length is 1 (other parts of bsp-agent also work on such an assumption)

@sudeepdino008 please add the linter recommendations

@noslav done


Integration with hosted JIRA

Hi,
There is any way for integration with hosted JIRA edition?
Thanks

Yes. Configuration is the same, but the URL should point to your internal instance.


tests: kernel: Add a test to verify early sleep

Add a test to verify that k_sleep() can be used to put
the calling thread to sleep for a specified number of
ticks during system initialization.
This test is inspired from legacy test_early_sleep test
from v1.7.0 release.
Signed-off-by: Ramakrishna Pallala<EMAIL_ADDRESS>

Codecov Report

Merging #6440 into master will increase coverage by 0.03%.
The diff coverage is 96.87%.

@@            Coverage Diff            @@
##           master   #6440      +/-   ##
=========================================
+ Coverage   53.06%   53.1%   +0.03%
=========================================
  Files         430     431       +1
  Lines       41071   41103      +32
  Branches     7911    7911
=========================================
+ Hits        21795   21826      +31
- Misses      16052   16053       +1
  Partials     3224    3224

Impacted Files
Coverage Δ

tests/kernel/early_sleep/src/early_sleep.c
96.87% <96.87%> (ø)

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 42e94c7...b60bc43. Read the comment docs.


Finding out why MetaTextBlockComponent tests fail

The MetaTextBlockComponent is now used in three places in motions (as of #4120). However, angular tests with these fails, and thus are commented out, which is not ideal. It is probably something asynchronously messing with each other, as the tests run fine individually

done


Page Navigation Functionality Suggestion

Some sort of means to navigate "back" to the main menu from viewing a membership page.

Template issue - please make part of 0.1 milestone.

With the current navigation bar, I consider this resolved.


Error: connect ECONNREFUSED <IP_ADDRESS>:3306 as AWS EC2 with nodes

Hi.
I have Error and.. I look around all of web site. but all of solution didn't work.
My setting is below information.

AWS : EC2(linux), route53, load balancer
nodJs with express
express listen port : 80
mysql : port 3306 (also, in EC2 security group, I'm authorize RDS server using RDS endpoint).

in my case, a little odd..
because of connecting domain, I should use port 80 in my EC2. but problem is if I set the port 80, then the linux block.
so, I use 'sudo', and then it's work!
but, the mysql to call the function give to me error like that Error: connect ECONNREFUSED <IP_ADDRESS>:3306
if I change the express listen port to 3000 then mysql is good work!
but another way, If I set the port at express as 3000 and use keyword 'sudo', then It will be same error
So, I think problem is related to use keyword 'sudo'
do you know what happen???
I attach my source code... than you
const app = express();
const port = 3000;
app.use('/', express.static(__dirname + '/../public'));

app.post('/Master', (req, res) => {

    var name = req.body.name;
    var pwd = req.body.pwd;

    db.getMasterid(function(err, results){
        if(err) {
            res.send(500, "Server get  Error from Masterid");
            return;
        }

var pool = db.createPool({
    connectionLimit : 10,
    host            : process.env.MYSQL_HOST,
    user            : process.env.MYSQL_USER,
    password        : process.env.MYSQL_SECRET,
    database        : process.env.MYSQL_DB,
    port            : '3306',
    ssl             : "Amazon RDS",
})

exports.getMasterid = function(callback){
    var sql = 'select * from masterid';
    pool.getConnection((err, connection) => {
        if(err){
            console.log(err);
            callback(true);
            return;
        }
        connection.query(sql, (err, results, field) => {
            connection.release();
            if(err){
                console.log(err);
                callback(true);
                return;
            }
            callback(false, results);
        });
    });
};

Error Message
{ Error: connect ECONNREFUSED <IP_ADDRESS>:3306
    at Object.exports._errnoException (util.js:1026:11)
    at exports._exceptionWithHostPort (util.js:1049:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1081:14)
    --------------------
    at Protocol._enqueue (/home/ec2-user/mjlens/node_modules/mysql/lib/protocol/Protocol.js:141:48)
    at Protocol.handshake (/home/ec2-user/mjlens/node_modules/mysql/lib/protocol/Protocol.js:52:41)
    at PoolConnection.connect (/home/ec2-user/mjlens/node_modules/mysql/lib/Connection.js:130:18)
    at Pool.getConnection (/home/ec2-user/mjlens/node_modules/mysql/lib/Pool.js:48:16)
    at Object.exports.getMasterid (/home/ec2-user/mjlens/build/mysql_db/reservation_db.js:23:10)
    at /home/ec2-user/mjlens/build/main.js:111:30
    at Layer.handle [as handle_request] (/home/ec2-user/mjlens/node_modules/express/lib/router/layer.js:95:5)
    at next (/home/ec2-user/mjlens/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/home/ec2-user/mjlens/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/home/ec2-user/mjlens/node_modules/express/lib/router/layer.js:95:5)
  code: 'ECONNREFUSED',
  errno: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 3306,
  fatal: true }

hi @XOneto9
you mentioned you are using RDS mysql but error is ECONNREFUSED <IP_ADDRESS>:3306. Do you set MYSQL_HOST variable to RDS host?

@sidorares Hi,
I'm setting MYSQ_HOST as RDS instance end point

MYSQ_HOST or MYSQL_HOST ? try to log what's actually passed to db.createPool. If undefined it's trying to connect to localhost

@sidorares
Sorry, My miss typo. MYSQL_HOST
how to check log on db.createPool ??

console.log({
    host            : process.env.MYSQL_HOST,
    user            : process.env.MYSQL_USER,
    password        : process.env.MYSQL_SECRET,
    database        : process.env.MYSQL_DB
 })

@sidorares Thank you!

@sidorares Yes,
your right....
I get a result...........all of....T^T OMG....
{ host: undefined,
user: undefined,
password: undefined,
database: undefined }

you probably set variables for current user. With sudo you het different set of env variables. Try to pass directly when you start it - MYSQL_HOST=x.y.z node server.js ( maybe sudo su before that)

@sidorares Thank you!!!


nee my bru

nee my bru
dis hierdie mod breaks my fokin speletjie.
what the poes? is jy dom?
nis nie good nie

no my bro
Ok.

this mod breaks my fokin game.
How do you know that?  And nobody else in the development department as perfection is an impossibility.

not good
Cool.

There seems to be an issue with LethalLib preventing enemy spawns. It's known as part of this list https://docs.google.com/spreadsheets/d/1c9rZIyxcOS7ZYjfyBsjFaYKWkbozkOVZJ1wDk2yE3Tc/edit#gid=0
Otherwise LGU is working fine ;-)
@zabbazebra fokof go have a braai

NEE! JULLE FOK!
EK IS NIE GELUKKIG NIE!

Closing this issue due to lack of information to help improve this mod so it can satisfy as many people as possible without jeopardizing others.

NEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEE

Is this with the 3.5.3 release?
If yes, post me the full log file.
Otherwise, use that and see if the issue persists.

Closing due to lack of response so assuming it's working as expected.


added MEMORY_LIMIT environment variable to desiredAppWorkload

Is there a related GitHub Issue?
https://github.com/cloudfoundry/korifi/issues/3081
What is this change about?
Add MEMORY_LIMIT env to cfprocess generation to ensure compatibility with classic CF behavior
Does this PR introduce a breaking change?
not expected
Acceptance Steps
kubectl get appworkload  -n  -o yaml
Check, that spec.env has a variable with Name: "MEMORY_LIMIT"
Tag your pair, your PM, and/or team
@danail-branekov

Looking good @marsteg
Just make sure you sign the CLA above so that we can get your change through

@marsteg could you please squash the two commits into a single one?


fl rmsd_upper_bound(): Assertion a.size() == b.size() failed

While doing large scale virtual screening using Vina-GPU, many molecules fail because of the assertion error:
CUDA_VISIBLE_DEVICES=1 ./Vina-GPU --receptor protein.pdbqt --ligand 1.pdbqt --num_modes 1
--center_x 7.2395000000000005 --center_y -0.6745000000000001 --center_z 31.628500000000003
--size_x 9.583 --size_y 19.809 --size_z 12.779 --thread 10000 --out res.pdbqt:

Vina-GPU: ./lib/coords:31 fl rmsd_upper_bound(const vecv&, const vecv&): Assertion `a.size() == b.size()` failed

@Glinttsd Do you have any idea of this?  Is there any configuration that can prevent it or does retry work?
Thanks in advance!

Hi, this could be caused by any problem that interrupts the docking procudure performed on GPU. You can try to run the problem ligand manually and see if the program die immediately. Also, you can try to decease thread to see what will happend


The last jobs will be done before the firsts ...

How did you notice this behavior?
I made a test-case with jMeter in which I start 100 print jobs in 10 seconds, poll the status and then request the report. Below you can see the order in which the finished reports where requested (for example request 4 was finished before request 1):
4
1
5
2
3
9
6
10
7
8
14
11
15
12
13
19
16
20
17
18
24
21
...
84
81
85
82
83
86
89
90
87
88
94
91
95
92
93
99
96
100
97
98

From this order I can not really observe that last jobs are prioritized. For sure you don't have the exact order in which the jobs were created as this is up to the scheduling of the operating-system.

I don't know what's wrong,but on http://geomapfish.demo-camptocamp.com/sbrunner/theme/Transport?debug
If I click 6 times on print the down order is systematically:
0, 1, 5, 4, 3, 2 !
Notes that I have only one concurrent printing job.

Notes that I have only one concurrent printing job.

Are you saying that you changed maxNumberOfRunningPrintJobs to 1?

Yes

Why?

Out of memory errors ...

There really was a problem with the comparator of the queue (see #225), but I would recommend you to increase the heap size of Tomcat so that you can run more jobs in parallel.

Effectively but actually it's not a goal to have more jobs in parallel, we will probably on near feature increase it to 2 but probably never to the default value, it create too many server load ...

A print job ideally should not create such a great load if this is the case
then we need to profile the application and optimize.  It is critical that
the system can handle hundreds of requests in parallel without causing
problems.
Are the problems memory or CPU related?
Jesse
On Wed, Mar 25, 2015 at 5:50 PM, Stéphane Brunner<EMAIL_ADDRESS>wrote:

Effectively but actually it's not a goal to have more jobs in parallel, we
will probably on near feature increase it to 2 but probably never to the
default value, it create too many server load ...
—
Reply to this email directly or view it on GitHub
https://github.com/mapfish/mapfish-print/issues/217#issuecomment-86118172
.


settings: fcb-backend: how to assign storage partition properly

**The problem we have **
Currently it is possible via kconfig to assign any partition for FCB using its number.
Partitions flash_area_id becomes non predefined (are auto-generated since #13081).
So it is possible only to ques which number will be signed to certain area.
Unfortunately it is not possible to transfer FLASH_AREA_XXX_ID label via Kconfig. It is possible to make some run-time binding operation - but this seams like using a tank to kill an ant.
solution
Let assign settings to the storage partition and remove SETTINGS_FCB_FLASH_AREA property from settings Kconfig. Optionally we can decide for use a dedicated partition name.
In mine opinion each flash memory user module should use dedicated partition. Optionally some partition might group into one biger partition (I think it is not forbidden to declare such flash_map - but this is independent topic, not important for this one)

@galak @anangl @carlescufi @pdunaj @Laczen @cvinayak : your opinions.

My honest opinion: try to remove flash_map.
Flash_map serves two purposes:

Getting a unique mapping between id and location in flash.
Protecting against writing/reading the wrong location given 1.

If 1. is not possible anymore, because of how it is generated, 2. has no meaning at all.
But if you really need to keep flash_map (think about how you will handle multiple images), it should be done at runtime.

@Laczen
If you wana't to operate on more than one device then you need always to know which device should you use to operate on certain flash-area. So you need a way to describe relations. So flash map is an answer to that. Otherwise you still need something else.
Flash_map is also generic component - otherwise we need dedicated solution for any user module.
There is also PR pending which introduce API for get flash device form flash_map - so it will be possible to use flash map as only information source (https://github.com/zephyrproject-rtos/zephyr/pull/13303).
Predefined ID was lost - but it was not needed - we still can use area we want because each area has very deterministic area ID (like DT_FLASH_AREA_IMAGE_SCRATCH_ID. We can provide a way to assign ID number manually (using DTS) at some point - but this is not needed right now, and I believe not very soon (that's why I didn't protest against).
I believe you maybe have got better idea than flash_map is. We can discuss such a idea in other topic.

@Laczen
If you wana't to operate on more than one device then you need always to know which device should you use to operate on certain flash-area. So you need a way to describe relations. So flash map is an answer to that. Otherwise you still need something else.

No you don't, this information is already available as you are using it to define the default flash_map. If each dts entry generates the device name there is no need to put it in a array for looking it up.

Flash_map is also generic component - otherwise we need dedicated solution for any user module.
There is also PR pending which introduce API for get flash device form flash_map - so it will be possible to use flash map as only information source (#13303).

This is definitively a bad idea, lets force everyone into using a unnecessary intermediate level.

Predefined ID was lost - but it was not needed - we still can use area we want because each area has very deterministic area ID (like DT_FLASH_AREA_IMAGE_SCRATCH_ID. We can provide a way to assign ID number manually (using DTS) at some point - but this is not needed right now, and I believe not very soon (that's why I didn't protest against).
I believe you maybe have got better idea than flash_map is. We can discuss such a idea in other topic.

Flash_map only makes sense if this is generated at the same time as the bootloader, the dts would not need any definition of regions whatsoever, the flash_map routines would read this from flash. A change in how your flash_map would look like would be a upgrade of the flash_map definition in flash.

cc @lemrey

@carlescufi, @pdunaj Any thought?

cc @SebastianBoe

I support the idea of removing SETTINGS_FCB_FLASH_AREA from the options. The point is, Kconfig is able to set it correctly and automatically; the user doesn't have a simple way of doing that.

Like @lemrey, I also Agree with the proposed solution as a first step. @nvlsianpu could you propose a PR?

@nvlsianpu, @carlescufi, yesterday I created a PR #13954 that generates the DEV name for each of the parts in the dts definition. This would allow you to create a default_flash_map easily without using the area_id. The definitions that are generated are:
DT_FLASH_AREA_IMAGE_0_DEV="NRF_FLASH_DRV_NAME"
Using these definitions you can generate a default flash_map that has known definitions (linking the id again to the definition). This also allows the removal of flash_map completely but this is something else.


Completed simulator test teaks

All simulator tests now runnable via npm test with the exception of the executive tests which had a problem with the custom jasmine matcher.
c:\github\gareththegeek\corewar>npm test

><EMAIL_ADDRESS>test c:\github\gareththegeek\corewar
> jasmine-ts

Started
................................................................

64 specs, 0 failures
Finished in 0.184 seconds

LOL, sorry, just auto merged this one as I wasnt paying attention :S


blog for nav and dashboard

product blog for nav+dashboard work.
fixes #1573
do not merge until changes are live in prod
url is blog/new-improved-pulumi-service-user-experience/

we lost a bunch of image references and the entire blog section on the dashboard in https://github.com/pulumi/pulumi-hugo/pull/1722/commits/bd4aebff392ee28de1aaa5b8dc479c9c81208ac2
def always rec pulling down changes from the remote before editing locally when working from a shared branch
do let me know if you want help fixing this up


Modal memory leak

🐛 Bug Report
Warning: Cannot update during an existing state transition (such as withinrender). Render methods should be a pure function of props and state.

To Reproduce
I use pictures as the video list and modal as the window for video playback. After switching videos several times, an error is reported.
Link to runnable example
snack
UI Kitten and Eva version

Package
Version

@eva-design/eva
^2.0.0

@ui-kitten/components
^5.0.0

Environment information
Type: [Simulator]
Device: [Genymotion]
OS: [Android9.0]
"expo-video-player": "^1.6.1",
Expo version (in 40)
Expo CLI version (4.0.13)

Please see https://github.com/akveo/react-native-ui-kitten/issues/1094#issuecomment-630141254


[kueuectl] Stop/resume cluster queue

What type of PR is this?
/kind feature

What this PR does / why we need it:
Adds kueuectl stop clusterqueue, kueuectl resume clusterqueue commands
Which issue(s) this PR fixes:

Fixes #2136 and #2137
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Stop/Resume ClusterQueue (`kueuectl stop clusterqueue`, `kueuectl resume clusterqueue`) commands are available

leaving lgtm on @trasc

/release-note-edit
CLI: Add commands to stop and Resume a ClusterQueue


Add suport for Debian 'the Debian way'

This enables deployments using as much debian-vodo as possible and compatible with systemd

+1
As a side note - as of 15.04 Ubuntu comes with systemd too.

+1


[Metrics] Kubernetes with a kubectl proxy and Prometheus do not work together

As in https://github.com/coreos/prometheus-operator/issues/2392 and https://github.com/prometheus/prometheus/issues/1583#issuecomment-214661546 and https://github.com/coreos/prometheus-operator/issues/287 if we run Clipper and use the kubectl proxy, Prometheus will be available at the expected address but no variables or graphs can be created using the web GUI. I will check if I can fix that on my cloud with the extra argument that is mentioned in the threads.

So good news is that using a LoadBalancer works, but I have not been successful in crating graphs on the NodePort metrics - adding the external url argument did not work for me.

@jacekwachowiak Did you figure out anything else? Seems like it should be a bug.

It seems like a Prometheus bug. The problem is when you have to use the kubectl proxy and you don't have direct acccess to the master node - NodePort service is useless then, the LoadBalancer type is the solution. Anyway, it seems that this is a Prometheus design problem. I am running the default Prometheus and an external Grafana and everything is fine too.

Hmm that's weird. Let's keep tracking this issue and see if we can fix this later.


Continue [Masthead] Clean up design spec, add Cloud menu designs

Clean up

Clean up design specs to represent the latest decisions
Archive the old...

Cloud alignment

Leaving the 16px & 14px untouched
Identify any new component needed by Cloud masthead

I hope at the end of this cleanup, we will have an understanding on the number & type of components masthead has today, and would potentially need for future. We'd like to target the future components that already exist in Cloud masthead to be absorbed eventually, so Cloud masthead can be taken out of experimental and live harmoniously with dotcom masthead.

This work is a continuation of the work managed under issue (#5708)
Acceptance criteria

[ ] Once analysis is finished, create the issues needed for future alignment

@shixiedesign maybe we could review together early next week, or if you have time to look alone and lmk if you see anything glaring.

Reviewed questions about L1 behavior with Shixie, Lila and James 5/25
Decisions:

L1 ecosystem name / topic should be visible on medium & small breakpoints
Keep options for selected state of ecosystem name on medium & small breakpoints
within the hamburger menu, the ecosystem name will only be visible on the first level of navigation.


Update quick-create-terraform.md

change Linux to Windows. Maybe we need to say it differently?

@ju-shim
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


fix missing state in disabled rich editor component

[x] Changes have been thoroughly tested to not break existing functionality.
[not applicable] New functionality has been documented or existing documentation has been updated to reflect changes.
[not applicable] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Rich Editor component in disabled state showed undefined and threw errors on console due to the alpine variable state not being set. Added x-data attribute to entangle the state. No more errors after that and the value is shown correctly.

Fixes #7641


Missing treesitter symbols for javascript

Language: javascript
AerialInfo:
Aerial Info
-----------
Filetype: javascript
Configured backends:
  treesitter (supported) (attached)
  lsp (supported)
  markdown (not supported) [Filetype is not markdown]
  man (not supported) [Filetype is not man]
Show symbols: Class, Constructor, Enum, Function, Interface, Module, Method, Struct

The minimal amount of code that demonstrates the missing symbol:
const utils = {};

utils.bot = {};
utils.bot.status_updates_channels = {};
utils.bot.admin_commands_channels = {};
utils.bot.logs_channels = {};

utils.bot.status_updates_channels.broadcast = async (message) => {
  for (const id of status_updates_channels) {
    await bot.channels.cache.get(id).send(message);
  }
};

utils.bot.admin_commands_channels.broadcast = async (message) => {
  for (const id of admin_commands_channels) {
    await bot.channels.cache.get(id).send(message);
  }
};

utils.bot.logs_channels.broadcast = async (message) => {
  for (const id of logs_channels) {
    await bot.channels.cache.get(id).send(message);
  }
};

export default utils;

Expected symbol:
󰊕  broadcast(function)
󰊕  broadcast(function)
󰊕  broadcast(function)

First of all thank you for the plugin. I really found it useful.
To give more info about backends, I changed the backend for the plugin to use lsp first and it worked properly. This issue only occurs with treesitter.
Hope this additional info helps.


Does the "keys" really work?

Read the quick start guide to create a template resource config file as below.
$ cat /etc/confd/conf.d/myconfig.toml
...
keys = [
    "/myapp/database/url",
    "/myapp/database/user",
]
...
$ cat /etc/confd/templates/myconfig.conf.tmpl

[myconfig]
database_url = {{getv "/myapp/database/url"}}
database_user = {{getv "/myapp/database/user"}}

As my understanding is, confd will replace with real secrets for all keys list in /etc/confd/conf.d/myconfig.toml .
But in fact, it still overrides all keys. If I remove one key from /etc/confd/conf.d/myconfig.toml,
$ cat /etc/confd/conf.d/myconfig.toml
...
keys = [
    "/myapp/database/user",
]
...

the output file should only be replaced by one key /myapp/database/user. But I found all confd key {{ getv ....}} are replaced with secrets.

Which backend is used?

Good question. @hubo1016
I used the un-merged feature (https://github.com/kelseyhightower/confd/pull/697) and saw this issue.
Will test with other backend later.

I believe this will happen if you remove the key an a reference to it still exists in the template.
ERROR template: myconfig.conf.tmpl:2:17: executing "myconfig.conf.tmpl" at <getv "/myapp/databas...>: error calling getv: key does not exist: /myapp/database/url
At the moment Secrets Manager does not do that however.

@hubo1016
Get some helps from @jrhoward , but seems this is the usage in confd I update my request in another way to explain what I need:
My purpose is, I need convert secrets for different environments.
$ cat /etc/confd/conf.d/myconfig-dev.toml
[template]
src = "myconfig.conf.tmpl"
dest = "/tmp/myconfig.conf"
keys = [
    "/myapp/database/dev/user",
]

$ cat /etc/confd/templates/myconfig.conf.tmpl
[dev]
database_user = {{getv "/myapp/database/dev/user"}}
[staging]
database_user = {{getv "/myapp/database/staging/user"}}

So each time, I only need convert several secrets for nominated environment which defined in /etc/confd/conf.d/myconfig-dev.toml, whatever all keys are marked by {{ getv ...}}.
Because in aws dev account, there is not key of /myapp/database/staging/user
Not sure how to do that.

how about this?
{{if exists "/myapp/database/dev/user" }}
[dev]
database_user = {{getv "/myapp/database/dev/user"}}
{{else if exists "/myapp/database/staging/user" }}
[staging]
database_user = {{getv "/myapp/database/staging/user"}}
{{end}}

Thanks, @jrhoward
I have more than 20 secrets for each environment, this will be big job for me .

@ozbillwang I suggest you to use a different prefix for different environments, then use the -prefix option of confd to get only the keys that are relevant for the given environment. Environment value can be injected from the environment variables like that -prefix /myapp/database/$ENVIRONMENT. You can then reference the keys in confd without the prefix part, which is very convenient.
Also, I suggest you to prefix all of the keys with the /$SERVICE/$ENVIRONMENT string (SERVICE=myapp ENVIRONMENT=staging), which then can be used as a prefix in confd.

Can you set the keys with prefix?
[template]
src = "myconfig.conf.tmpl"
dest = "/tmp/myconfig.conf"
keys = [
    "/myapp/database",
]

then
{{- range gets "/myapp/database/*/user" }}
[{{- .Key | dir | base }}]
database_user = {{ .Value }}
{{- end }}

I got same problem using ssm backend. We have multiple services that can be deployed in different flavor, and then there is configuration that is for flavor and then for flavor & service combo.
So I have 2 step process where first replace keys from prefix "/<flavor>/<service>" and then use result of that with prefix "/flavor/"
But unfortunately it fails on first step because it can't find the key because that key was supposed to be replaced in next step. <getv "/db/database">: error calling getv: key does not exist: /db/database


⚠ Cioos National CKAN has degraded performance

In 63ee1b9, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 12710 ms

Resolved: Cioos National CKAN performance has improved in 394222c after 14 minutes.


[opengl] Very incomplete start of multiview support for opengl3 driver

This is a draft because it's barely even started! Rendering is one of my weaker areas, so we'll see how far I'm able to actually get with this...

So far, all this does is attempt to load the correct extensions and allocate the textures and buffers (and it probably isn't even working right :-)). The bigger changes will probably be in the shaders (taking into account the gl_ViewID_OVR variable), but I haven't started on that at all yet.

This relates to PR #65227, in that how overriding the textures on render targets works in opengl will depend on if multiview is supported.

This is finally starting to get somewhere! It appears to work on desktop with OpenGL (yay!), but it doesn't work in WebGL yet. I'm still trying to figure out how to load and use the multiview extension on the web platform.

I think I've got a way to access the multiview extension with WebGL, and I think it is using it the way I'm intending it to be used. But it isn't working with WebXR. I think it's because I'm doing something somewhere that WebGL isn't happy about, probably something around scene.glsl that I didn't change right.
When running in my desktop browser with the WebXR emulator I get this error:
GL_INVALID_OPERATION: The number of views in the active program and draw framebuffer does not match.

This sounds like there is some case where the scene.glsl shader is drawing to a normal 2D texture, rather than a 2D texture array, but so far I haven't been able to figure that out.
When I try in the Oculus browser (since at this point, I'm kinda suspect of the WebXR emulator), I get an error about no texture being bound to texture unit 14, which in this case is the radiance map (the quest has 16 texture units). Looking at the code, it does, in fact, seem like it's not binding a radiance map - but I'm not sure why that didn't break anything in the non-multiview testing I was doing? Or, maybe this got broken by some other change that I pulled in from master?
Anyway, I've got to put this down for now. :-)

Just giving a brief read through, this looks very promising. Amazing how much overlap there is with the Vulkan implementation though that is to be expected.
Mismatches between multiview and non-multiview plagued me too, it's very easy to overlook a place where we're using the wrong framebuffer or something like that, while often desktop drivers just shrug and do the next best thing and you're not aware there is a bug. With Vulkan turning on Vulkan Validation Layers were a life saver in finding these areas. I'm not sure if we have something similar for OpenGL..

Also keep in mind I'm trying to get #65227 merged, this will remove the old external texture implementation in favor of a new texture override feature. In that PR however I only implement that for Vulkan and clear out the old logic from OpenGL (was mostly commented out anyway).

This is now working in WebXR on the Oculus Quest 2!!!
This still needs a bunch of clean-up, though. In particular, part of the key to getting this working in WebGL, is that the #extension GL_OVR_multiview2 : require line needs to come before any non-preprocessor code in the shaders, but ShaderGLES3 is automatically injecting the precision highp ... lines before any of the code in the *.glsl files.
So, we end up with:
# ... [other preprocess lines] ...

# These lines are injected by ShaderGLES3. But WebGL doesn't like that these are above the extension loading code belowe.
precision highp float;
precision highp int;
precision highp sampler2D;
precision highp samplerCube;
precision highp sampler2DArray;

# The code we need for multiview that's at the top of the *.glsl files that need to be multiview aware.
#ifdef USE_MULTIVIEW
#if defined(GL_OVR_multiview2)
#extension GL_OVR_multiview2 : require
#elif defined(GL_OVR_multiview)
#extension GL_OVR_multiview : require
#endif
layout(num_views=2) in;
#define ViewIndex gl_ViewID_OVR
#else
#define ViewIndex 0
#endif

This works fine with OpenGL on the desktop, but on WebGL, it won't compile the shader. :-(
So, currently, in the PR, I've removed the lines in ShaderGLES3 that add those precision highp ... lines, and copied them into all the *.glsl files BELOW the code we need to load the multiview extension. This isn't great - we don't want to have to manually copy those lines to every shader! But I'm not sure what would be a good way to handle this.

On RocketChat @clayjohn suggested putting the code that loads the multiview extension into ShaderGLES3::_build_variant_code(), which is what the latest code here does. I had to change the signature of that function a little bit, because we needed the code to be a little different between the vertex and fragment shaders.

This now also includes the changes from PR #66780, because it's much easier to test in the WebXR emulator, rather than putting on a headset :-)

The changes I just pushed address all the previous review:

The multiview data is now passed to scene.glsl via a multiview-specific UBO
When loading the glFramebufferTextureMultiviewOVR function pointer via EGL, it's only loading it once on Config, and then it's shared where needed
Multiview is now a specialization of scene.glsl, instead of a variant

On this last point, specializations weren't completely wired up in rasterizer_scene_gles3.cpp. Basically, some base flags were being passed into RasterizerSceneGLES3::_render_list_template() but weren't actually given to the shader, meaning the default of 32 (USE_RADIANCE_MAP) was always used.
So, when you see this code in the PR:
	// @todo Temporarily hardcode this for now.
	//uint32_t base_spec_constants = p_params->spec_constant_base_flags;
	uint32_t base_spec_constants = 1 << SPEC_CONSTANT_USE_RADIANCE_MAP;

It isn't newly hardcoding this - it was already hardcoded to this value! - it's just assigning it to a variable, so that we can add in the flag for multiview when needed. I'm assuming that @clayjohn is planning to fully implement the rest of the specializations at some point!
On RocketChat, @BastiaanOlij brought up using the OpenGL renderer with OpenXR, which I'm going to look into next. But that's the last thing on my personal TODO list for this PR!

Looks great so far! And yeah, I need to do some work on the shader compiler so that specializations are compiled async and then used once available.
Is Multiview working by force enabling the specialization constant?

Thanks!

Is Multiview working by force enabling the specialization constant?

It's enabling the specialization if the render target has more than one view. Here's the relevant code:
	// @todo Temporarily hardcode this for now.
	//uint32_t base_spec_constants = p_params->spec_constant_base_flags;
	uint32_t base_spec_constants = 1 << SPEC_CONSTANT_USE_RADIANCE_MAP;

	if (p_render_data->view_count > 1) {
		base_spec_constants |= 1 << SPEC_CONSTANT_USE_MULTIVIEW;
	}

Previously, it wasn't passing any specialization into the shader class at all, so that change comes a little later as well:
@@ -2128,7 +2155,7 @@ void RasterizerSceneGLES3::_render_list_template(RenderListParameters *p_params,
 		}

 		if (prev_shader != shader || prev_variant != instance_variant) {
-			material_storage->shaders.scene_shader.version_bind_shader(shader->version, instance_variant);
+			material_storage->shaders.scene_shader.version_bind_shader(shader->version, instance_variant, base_spec_constants);
 			float opaque_prepass_threshold = 0.0;
 			if constexpr (p_pass_mode == PASS_MODE_DEPTH) {
 				opaque_prepass_threshold = 0.99;

Hopefully, that's all OK? I'm doing a lot of guessing as to how this is meant to work. :-) If it should be done differently, please let me know and I'll fix it up!

Thanks!

Is Multiview working by force enabling the specialization constant?

It's enabling the specialization if the render target has more than one view. Here's the relevant code:
	// @todo Temporarily hardcode this for now.
	//uint32_t base_spec_constants = p_params->spec_constant_base_flags;
	uint32_t base_spec_constants = 1 << SPEC_CONSTANT_USE_RADIANCE_MAP;

	if (p_render_data->view_count > 1) {
		base_spec_constants |= 1 << SPEC_CONSTANT_USE_MULTIVIEW;
	}

Previously, it wasn't passing any specialization into the shader class at all, so that change comes a little later as well:
@@ -2128,7 +2155,7 @@ void RasterizerSceneGLES3::_render_list_template(RenderListParameters *p_params,
 		}

 		if (prev_shader != shader || prev_variant != instance_variant) {
-			material_storage->shaders.scene_shader.version_bind_shader(shader->version, instance_variant);
+			material_storage->shaders.scene_shader.version_bind_shader(shader->version, instance_variant, base_spec_constants);
 			float opaque_prepass_threshold = 0.0;
 			if constexpr (p_pass_mode == PASS_MODE_DEPTH) {
 				opaque_prepass_threshold = 0.99;

Hopefully, that's all OK? I'm doing a lot of guessing as to how this is meant to work. :-) If it should be done differently, please let me know and I'll fix it up!

That is exactly right! The pending changes to the shader compiler will just make things even more efficient.

On RocketChat, @BastiaanOlij brought up using the OpenGL renderer with OpenXR, which I'm going to look into next. But that's the last thing on my personal TODO list for this PR!

I've been working on this, but this is turning out to be a bigger and slower project than I had hoped it would be. I think I'm going to do that in a follow-up PR, which I'll open soon.
I just rebased and retested this PR, which I think is solid and ready for a full review!

@dsnopek glUniform1i isn't called in many places right now. Its used when setting a single integer uniform with shader.version_set_uniform(), but I don't see any places where you do that so it means the issue likely comes from the other place it is calledshader._compile_specialization(). Specifically the code that maps textures to their binding points
https://github.com/godotengine/godot/blob/72b845b28773dd40adf6f55b226fb732910cbf14/drivers/gles3/shader_gles3.cpp#L376-L398
I would try removing the specialization constant that disables the radiance map and see if that helps.

@clayjohn:

I would try removing the line that sets the specialization constant that disables the radiance map and see if that helps.

Thanks for the suggestion!
Unfortunately, that didn't work. :-( I went so far as commenting out every occurrence of glUniform1i() in the code, and it still ran fine on the desktop (both OpenGL and the WebXR emulator in Chrome), however, on Quest 2 the error is gone, but the scene still doesn't render.
Currently, I've only found that these changes (removing the multiview-specific UBO and putting the data on the scene data UBO) will fix it on the Quest 2:
https://gist.github.com/dsnopek/cffa8a46cc8537560e22975ec38ae0e1
Is it possible that the glUniform1i() is error is just a symptom of the shader getting totally messed up? Maybe I'm doing something wrong with the multiview UBO that's just completely breaking the shader, so WebGL isn't seeing any of the uniforms, leading to the glUniform1i() error?
But if so, I'm not sure what I'm doing wrong... :-/

I solved the problem on Quest 2!
The issue turned out to be that I needed to pass base_spec_constants to version_set_uniform(), now that we aren't just using the default specializations anymore. Now that I know this was the issue, I'm actually more surprised that it worked on my desktop computer, than the fact that it failed on Quest 2. :-)
@clayjohn @BastiaanOlij: Anyway, this is really, actually ready for full review now!

I solved the problem on Quest 2!
The issue turned out to be that I needed to pass base_spec_constants to version_set_uniform(), now that we aren't just using the default specializations anymore. Now that I know this was the issue, I'm actually more surprised that it worked on my desktop computer, than the fact that it failed on Quest 2. :-)

Oof, sorry I didn't catch that. Certain drivers must be really lenient with how they set uniforms because technically it should not have worked.

@clayjohn Thanks for the review! I've just pushed all the requested changes :-) Please let me know what you think!

yeah this looks good to me guys, great work @dsnopek !

I'm going to go ahead and merge this as is. Great work David!


🐛 (v5) MemberAutoData and ClassAutoData override explicit null in data source

Describe the Bug
When using MemberAutoData or ClassAutoData from the the 5.0.0-preview0011 package, explicit null values in the data source are replaced with values generated by Auto Fixture. I searched issues and the discussions but didn't see anything that explains this change in behavior.
Scenario
In the setup below, all three instances of the theory fail because the second parameter has been substituted with a value from AutoFixture
public static IEnumerable<object?[]> TestData
{
  get
  {
    yield return [ null, null ];
    yield return [ "", null ];
    yield return [ " ", null ];
  }
}

public class TestObject {  public int X { get; set; } }

[Theory]
[MemberAutoData(nameof(TestData))]
public void SomeTest(string? value, string? value2, TestObject input)
{
  Assert.Null(value2);
}

The above yields:
Xunit.Sdk.NullException
Assert.Null() Failure: Value is not null
Expected: null
Actual:   "value2a6acd719-0956-4484-ad3a-461d3f91967b"

The following with ClassAutoData also results in the same:
public class TestData : TheoryData<string?, string?>
{
  public TestData()
  {
    Add(null, null);
    Add("", null);
    Add(" ", null);
  }
}

public class TestObject {  public int X { get; set; } }

[Theory]
[ClassAutoData(typeof(TestData))]
public void SomeTest(string? value, string? value2, TestObject input)
{
  Assert.Null(value2);
}

Expected Behavior
Autofixture should not substitute explicit null values in the source data source. This is different behavior from v4
Tasks

[ ] Updated all AutoFixture and related libraries
[ ] Confirmed in support forums if behavior is expected

More Information
InlineAutoData continues to work as expected. In other words, this works as expected:
[Theory]
[InlineAutoData(null, null), InlineAutoData("", null), InlineAutoData(" ", null)]
public void SomeTest(string? value, string? value2, TestObject input)
{
    Assert.Null(value2);
}

Thanks!

Hey @aivascu no problem! This is xUnit not NUnit


A full instructions for starting from scratch.

easy-matrix-wechaty is a simple and fast temporary deployment solution that provides a way to experience the main functions of matrix-appservice-wechaty. After some simple steps you can deploy a synapse server and a matrix-appservice-wechaty server. At present, in order to have a better experience, we need a domain name and a computer with a public IP address. Fortunately, we are designing a full-featured deployment plan for unpublic IP and domain names, and it is expected to be released this year(If it's necessary. ^_^).
I test it with a wechaty account name "疯帽子"(mad hatter, id: little-printf). After a period of testing, I will try to merge into this project (if appropriate). Welcome anyone to point out the shortcomings of it.

@zrn-fight How is the progress of the project deployment this week?

I have deployed the matrix project according to your deployment method. I changed the domain name in export host = localhost to my own domain name fontweb.asia, but after I entered fontweb.asia: 8008 in the address bar of the browser, the server that cannot connect to fontweb.asia is displayed.
It's the first time I've heard of docker. Will it be often used in later projects? If it is often used, I need to learn it.

@545641826 I have deployed the matrix project according to your deployment method. I changed the domain name in export host = localhost to my own domain name fontweb.asia, but after I entered fontweb.asia: 8008 in the address bar of the browser, the server that cannot connect to fontweb.asia is displayed.
It's the first time I've heard of docker. Will it be often used in later projects? If it is often used, I need to learn it.

Oh, you have misunderstood the purpose of the project.There is no domain name and registered server are required. You can run it without any modify.

I deployed it on Tencent ECs. Do you mean that I only need to deploy it on my own computer? OK, I'll deploy locally. Thank you

And, did the original deployment plan encounter any problems?
We can discuss any problems you encounter during the deployment process here. (Sometimes you need to explain your problems in other new issues。)

@zrn-fight Please speak freely here. I hope you can evaluate these deployment scripts from the perspective of a new user as a reference for my next modification. If you encounter any trouble, you can give feedback here, and we can analyze the reasons together. This will be much faster.

OK.After I typed docker-compose run --rm -e SYNAPSE_SERVER_NAME=$HOST -e SYNAPSE_REPORT_STATS=yes synapse generate on the virtual machine,there was an error that I haven't solved.Errors are reported as follows:
【ERROR: Couldn't connect to Docker daemon at http+docker://localunixsocket - is it running?If it's at a non-standard location, specify the URL with the DOCKER_HOST environment variable.】
I restart docker and add the current user to the docker group, but still report an error.

https://techoverflow.net/2019/03/16/how-to-fix-error-couldnt-connect-to-docker-daemon-at-httpdocker-localhost-is-it-running/#:~:text=ERROR%3A Couldn't connect to Docker daemon at http%2Bdocker%3A%2F%2Flocalhost,does not have the permissions to access docker.

OK.After I typed docker-compose run --rm -e SYNAPSE_SERVER_NAME=$HOST -e SYNAPSE_REPORT_STATS=yes synapse generate on the virtual machine,there was an error that I haven't solved.Errors are reported as follows:
【ERROR: Couldn't connect to Docker daemon at http+docker://localunixsocket - is it running?If it's at a non-standard location, specify the URL with the DOCKER_HOST environment variable.】
I restart docker and add the current user to the docker group, but still report an error.

Yes, this seems to be a problem that docker beginners are prone to encounter. Your question can help others to solve the problem quickly. see: solution

After I typed sudo sed -e 's/#app_service_config_files/app_service_config_files/' -e '/app_service_config_files/a\  - /data/wechaty-registration.yaml' -i files/homeserver.yaml,an error appears as shown in the picture, 【sed: unable to read files / homeserver.yaml: there is no file or directory】

Would you please run ls or tree -L 3 command on your work directory to provide more information?

After I typed sudo sed -e 's/#app_service_config_files/app_service_config_files/' -e '/app_service_config_files/a\ - /data/wechaty-registration.yaml' -i files/homeserver.yaml,an error appears as shown in the picture.【sed: unable to read files / homeserver.yaml: there is no file or directory】

The files/homeserver.yaml is the output of the command docker-compose run --rm -e SYNAPSE_SERVER_NAME=$HOST -e SYNAPSE_REPORT_STATS=yes synapse generate. If you have ran the command successfully?

I guess the admin user is not allowed to execute docker commands by default, like the root in centos. In this script, all commands that do not start with sudo should be executed by ordinary docker group users.

I think I should build a docker image to run the script.

@zrn-fight OK, I write a new version for you, please try as soon as possible and give me a reply. see: easy-matrix-wechaty

I tried to use the new version you wrote, but an error was reported after entering Chmod + X. / run.sh. I tried the first version of easy matrix wechaty again, and it ran successfully.

I tried to use the new version you wrote, but an error was reported after entering Chmod + X. / run.sh. I tried the first version of easy matrix wechaty again, and it ran successfully.

What's wrong with it？And why the first version can run now?

【chmod: : unable to read files / homeserver.yaml: there is no file or directory】
I'm also curious why the first version can run. I just reentered the command in the first version after the new version was wrong, and then it worked well. after I entered curl http://localhost:8008/_matrix/static/ on the terminal,an HTML file is returned.I'm now looking at how to install a browser on a virtual machine.

【chmod: : unable to read files / homeserver.yaml: there is no file or directory】
I'm also curious why the first version can run. I just reentered the command in the first version after the new version was wrong, and then it worked well. after I entered curl http://localhost:8008/_matrix/static/ on the terminal,an HTML file is returned.I'm now looking at how to install a browser on a virtual machine.

You do not need to install a browser. Just install the client according to the readme file.

Your machine is too weird. Why run chmod +x ./run.sh will throw such error about homeserver.yaml? And this command is not  necessary if you can run the ./run.sh directly.

ok,I will try to install a client.

OK, There is to version for you: v0.1 and v0.2.

@yswtrue Are you interesting to be next user of easy-matrix-wechaty. It's easy for you. Your evaluation will be a valuable reference for further development.

Is the service started by running this command [docker-compose -f "docker-compose.yml" up -d]? The service cannot start after I run it

Is the service started by running this command [docker-compose -f "docker-compose.yml" up -d]? The service cannot start after I run it

please attach the log, let me see see.

【docker-compose exec synapse register_new_matrix_user http://localhost:8008 -c /data/homeserver.yaml -u $SYNAPSE_USER_NAME -p $SYNAPSE_USER_PASSWD --no-admin】
ERROR: No container found for synapse_1

【docker-compose -f "docker-compose.yml" up -d】
easymatrixwechaty_matrix-appservice-wechaty_1 is up-to-date Starting easymatrixwechaty_synapse_1

Which mean that you have started the synapse successfully.

【docker-compose exec synapse register_new_matrix_user http://localhost:8008 -c /data/homeserver.yaml -u $SYNAPSE_USER_NAME -p $SYNAPSE_USER_PASSWD --no-admin】
ERROR: No container found for synapse_1

Registering users are only the installation process, not the startup process.
And in the new version of easy-matrix-wechaty, a script to run the server has been added.


Isochrone coarseness/smoothness cannot be set

According to ORS isochropne API (https://openrouteservice.org/dev/#/api-docs/v2/isochrones/{profile}/post) it is possible to use the optional field smoothing to specify the coarseness of the isochrone (in terms of how short a segment of the output polygon can be). It looks like that the default is 25. There is no GUI widget (or any other way ?) to specify the coarseness of the isochrones.
Expectation is that there should be a way to specify how coarse the isochrones should be.

fixed in #176 and the corresponding #199


Remove version: Alibaba.aDrive version 4.9.8

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

URL: https://cdn.aliyundrive.net/downloads/apps/desktop/update/4.9.8/win32/ia32/aDrive-4.9.8.exe
Status Code: 200
(Automated message - build 870)


Update ultrastar txt with musicbrainz - Artist, Title, Year, Genre, Cover

Update the tags without pitching and the other stuff

Add MBID manually to avoid odd search


Error about  claim with universally-quantified var on RHS must show var names

One of Haskell backend errors looks like this:
Error:
  module 'GEN-B9B4CAC69636F5AA489AA9CCD4116C5719559854D98BBDCEAC41C2EDC89C0D03-TO-7ABB79081A802ABFBA809B6B97DA08A1BDD820D0A54525EE76FE0EC90D4542E0-SPEC':
  claim declaration:
    Found claim with universally-quantified variables appearing only on the right-hand side

It would be preferred if error also mentions variable name and the claim location.
~Also, for all backend crashes like this we would prefer different exit codes for easy debugging and tool integration. At very least, backend crash due to out of memory must give a different exit code than a functional error.~

kore-exec-flopper-kick1.tar.gz

flopper-kick-spec.k

We can talk about this at the meeting.
@JKTKops I could still use some help from the backend, since I have no idea what went wrong here just by looking at the claim.
@denis-bogdanas in order to be able to debug this in the front end I will need the entire definition. If this is part of the erc20-verification repo, then just details about how to set it up.

The backend needs to give more information. After that is done, transfer issue to the frontend.

Probably is a syntax validation error https://github.com/runtimeverification/haskell-backend/tree/master/kore/src/Kore/Validate

Can you post the steps you took to reproduce it?
It would be helpful to fix the issue in the front-end as well.

I just ran ./kore-exec.sh from the attached archive and got the same error as in the issue description:
λ ./kore-exec.sh
kore-exec: [1738164] Error (ErrorVerify):
    Error:
      module 'GEN-B9B4CAC69636F5AA489AA9CCD4116C5719559854D98BBDCEAC41C2EDC89C0D03-TO-7ABB79081A802ABFBA809B6B97DA08A1BDD820D0A54525EE76FE0EC90D4542E0-SPEC':
      claim declaration:
        Found claim with universally-quantified variables appearing only on the right-hand side
Created bug report: kore-exec.tar.gz

The error still has no variable name or claim location information.
I'm just going through needs-reproducing issues and trying to reproduce them to find which ones are outdated and which ones are still relevant.
Sorry, I think I can not provide any useful information on this issue.

That's good enough. Thank you.


Default code coloring change to distinguish variables from other foreground tokens like braces, operators

With this PR, default code coloring in Brackets ( Default Light and Dark) being changed to distinguish between variables and other tokens colored as fore ground. Consider the following snippet -

With this change set, the same snippet will be rendered as -

Along with this change we might have to implement some more changes to distinguish between call expressions and simple variable access. As suggested by @marijnh in CM issue 4906, I am looking into extending our JavaScript mode with google-modes

@petetnt Modified cm-variable colors to improve the contrast ratio to acceptable limit. PSB, the screenshot from http://webaim.org/resources/contrastchecker/
For DarkTheme

For LightTheme

@petetnt That was really quick. Thanks a lot 👍


Fix clear -> clean in README

Coverage remained the same at 89.5% when pulling 5fec3a58ea8d4c8af499f4b074292f091908aae5 on freemanoid:patch-1 into 1d47c0679a66ad9be3c388d0dafa03661da2eb48 on gmontard:master.

Coverage remained the same at 89.5% when pulling 5fec3a58ea8d4c8af499f4b074292f091908aae5 on freemanoid:patch-1 into 1d47c0679a66ad9be3c388d0dafa03661da2eb48 on gmontard:master.

Thanks for the correction!


Feature/chap 838 new table components

@anynines-johannchopin I wanted to finish all the requirements given, so I already implemented the sorting feature with it. Good news is, 'sortable' feature is now optional. If you don't want to implement it, then just don't pass any 'onSort' function or 'sortable' prop to cells. That's it.


Too strict peerDependency

Hello,
I found that this plugin defines an exact react-native peerDependency, currently 0.71.6 as 5.3.2 which prevents me to upgrade to 0.71.7 without changing every npm command to use legacy peer deps or force.
Is there any specific reason to not use like ^0.71 for peer dependency?
Thanks

New version is released and now it should work.
I'm going to close the issue.


Fix repeating dependency verification exception bug

If an AMD module includes the same dependency twice but the module ids are not identical, then when development mode and verify dependencies are both enabled, a DependencyVerificationException response will occur that does not resolve when dependencies are verified.
Example:
define(['a/foo', './foo'], function(foo) {
});

where both module ids resolve to the same module.

@ddumont please review

lgtm


Enable clicking multiline messages + other tweaks

Now:

Before:

@steelbrain Two remaining issues:

The first line of a multiline is now inconsistent with the normal line text when wrapping occurs. Personally, I would like to make the tweak to make the link for single line text to also have links flow inline with the text properly (I was previously trying to avoid that because of the previous issues opened, but I think it's preferable).
Why would someone use the multiline flag? Is it used? As it is, it's a little broken as we assume it is certainly multiline if they say it is, but when it's not the experience will break. In contrast, we are already checking if it's multiline. So the fix here for that issue would be to instead of the check, which results in multiline being a redundant flag. Does that make sense?

About Point 1, do whatever pleases you. About point 2 ide-hasksell does I think, the newline check is just a test in case they missed it. but I get your point, maybe we should depreciate that flag?

@steelbrain Made the changes for 1. For 2, yeah, I think it should be deprecated until it's properly thought through. If it's going to be deprecated, I'll make another PR for that and we remove some of the duplicate code.

Tested it locally, looks good to me :clap:


HdrHistogram

Fork https://github.com/HdrHistogram/HdrHistogram.NET and port to .NET Standard
https://github.com/HdrHistogram/HdrHistogram.NET/issues/17
This can replace the current HdrHistogram implmentation

HdrHistogram.NET .NET Standard https://github.com/HdrHistogram/HdrHistogram.NET/issues/17

After benchmarking the comparing HdrHistogram and the current implementation, the performance of the current implementation is much better although not as flexible. Leaving the current implementation as a result, the current implementation was 13 times quicker, after some performance improvements on HdrHistogram it was down to 8 times slower but requires too much rework.


ReferenceError: document is not defined

When I try to server render using Node.js + Express + ReactRouter, I get this:
C:\Projects\Git\gearz\node_modules\react-datepicker\node_modules\tether\dist\js\tether.js:406
var el = document.createElement('div');
         ^
ReferenceError: document is not defined

You are assuming document exists, which isn't true when you are server rendering.
The most curious part is that I'm copying my example from the server rendering example. This is how my component looks like:
    var Example = React.createClass({
        getInitialState: function () {
            return {date: moment()};
        },
        onChange: function () {},
        render: function () {
            return React.createElement(
                DatePicker,
                {
                    selected: this.state.date,
                    onChange: this.handleChange
                }
            );
        }
    });

Am I doing anything wrong?

:+1:

Getting this same issue still in 0.11..

@martijnrusschen and I will dive into some IE testing tomorrow. Please stay put!

I'm not really that well versed in the SSR bit, but shouldn't #113 have fixed this?

I'm having the same issue... (node ssr)  If I require the datepicker on the server without even using it it shows that error. A workaround I found is that if I require it just before I open the picker I don't see it, in my case I will never render the picker on the server side, so it works.
I tried to locate the issue and seems to be with tether itself,
see:

var transformKey = (function () {
  var el = document.createElement('div');
  var transforms = ['transform', 'webkitTransform', 'OTransform', 'MozTransform', 'msTransform'];
  for (var i = 0; i < transforms.length; ++i) {
    var key = transforms[i];
    if (el.style[key] !== undefined) {
      return key;
    }
  }
})();

That code gets executed when you include tether. Does this have to do anything with the way the datepicker is being bundled with webpack? Sorry I have to do something else so I cant keep digging. If I have time later I'll do it.

Right, I think I know where this issue is coming from. I was a bit confused, as we clearly only load Tether when document is defined, but then I realised that we just figured out that the webpack externals feature has a strange side-effect: if you say you expect a dependency to be global, it will make it global if it isn't already.
I'll have to dig into this, we noticed this @Hacker0x01 when we tried to upgrade and suddenly our underscore was replaced by lodash in the global namespace.

Has there been any progress on this? I'm still getting "document is not defined" when upgrading to 0.11.4, and have to revert to 0.8.0.

Getting same error in 0.12. Are there any workarounds?

Well, I'm still using it, I just made a fugly workaround, fortunately the component that contains the datepicker has to re-render after some data gets loaded into the page so what I did is if I canUseDom I render the datepicker, otherwise I just render a simple input.
import { canUseDOM } from 'react/lib/ExecutionEnvironment';

...

dateField = function(label, minDate, date, onChange){
    if(canUseDOM){
      var DatePicker = loadDatePicker();
      return <DatePicker
        dateFormat={'MM/DD/YY'}
        minDate={minDate}
        selected={date}
        maxDate={moment().add(1, 'y')}
        placeholderText={label}
        onChange={onChange}
        />;
    }
    return <input
      type='text'
      placeholder={label}
      groupClassName='input-group-sm'/>;
  },
  loadDatePicker = function(){
    if(DP == null){
      DP = require('react-datepicker');
    }
    return DP;
  };

...

<div className='date-field'>
     {dateField('from', moment(), f.getFromDate(), onFromChange)}
</div>

Just out of curiosity... Do you get any error for inconsistency between
whats been rendered on the client and on the server?
Em Qui, 27 de ago de 2015 11:17, Gabriel Espinola<EMAIL_ADDRESS>escreveu:

Well, I'm still using it, I just made a fugly workaround, fortunately the
component that contains the datepicker has to re-render after some data
gets loaded into the page so what I did is if I canUseDom I render the
datepicker, otherwise I just render a simple input.
import { canUseDOM } from 'react/lib/ExecutionEnvironment';
...
dateField = function(label, minDate, date, onChange){
if(canUseDOM){
var DatePicker = loadDatePicker();
return <DatePicker
dateFormat={'MM/DD/YY'}
minDate={minDate}
selected={date}
maxDate={moment().add(1, 'y')}
placeholderText={label}
onChange={onChange}
/>;
}
return ;
},
loadDatePicker = function(){
if(DP == null){
DP = require('react-datepicker');
}
return DP;
};
...

     {dateField('from', moment(), f.getFromDate(), onFromChange)}
—
Reply to this email directly or view it on GitHub
https://github.com/Hacker0x01/react-datepicker/issues/137#issuecomment-135448688
.

Same error
import DatePicker from 'react-datepicker';

with
          <DatePicker />

Trace:
  [1] [piping] error given was: ReferenceError: document is not defined
  [1]     at /Users/a/b/c/node_modules/react-datepicker/node_modules/tether/dist/js/tether.js:406:12
  [1]     at /Users/a/b/c/node_modules/react-datepicker/node_modules/tether/dist/js/tether.js:415:3
  [1]     at i (/Users/a/b/c/node_modules/react-datepicker/node_modules/tether/dist/js/tether.js:7:22)
[1]     at Object.<anonymous> (/Users/a/b/c/node_modules/react-datepicker/node_modules/tether/dist/js/tether.js:11:2)
[1]     at Module._compile (module.js:460:26)
[1]     at Module._extensions..js (module.js:478:10)
[1]     at Object.require.extensions.(anonymous function) [as .js] (/Users/a/b/c/node_modules/babel-core/lib/api/register/node.js:214:7)
[1]     at Module.load (module.js:355:32)
[1]     at Module._load (module.js:310:12)
[1]     at Function.module._load (/Users/a/b/c/node_modules/piping/lib/launcher.js:32:16)

To anyone interested I created a small wrapper, that you can import instead of the DatePicker.
It just renders an input field serverside and on the client it passes the props along using ES7 spread syntax.
I just
import {default as DatePicker} from './IsomorphicDatePicker';

And it seems to work fine.
https://gist.github.com/Hagbarth/2eeb59bcbc303313a10e

Thanks!, It looks good, I like to use canUseDom instead of checking for document, my2c
import { canUseDOM } from 'react/lib/ExecutionEnvironment';

I also had this problem when running mocha tests, as the import of Datepicker fails because it tries to access 'document' before jsdom() is called in the test (which is what makes 'document' available). I fixed this by moving the require of Datepicker into the render() method of the component that uses it rather than having it as an import at the top.

Fixed.


Car Riding Off Into The Sunset - Daksh

I present to you, a scuffed f1 inspired car. The grass/wheels/sun and spoiler are randomly done.
I had a lot of fun building this. :D

[x] I have read the steps to getting a blot
[x] I am submitting art that...

[x] is algorithmically generated (will meaningfully change each time the program is run)
[x] is your own work that is not copied from somewhere else
[x] doesn't call Math.random() (See the documentation on randomness)
[x] is drawable on Blot itself (doesn't have lines overlap more than 5 times)

[x] Doesn't overlap lines more than 5 times
[x] Fits in the working area of 125x125mm
[x] Doesn't rely on colour to look good

If you used based your art on something else, used a tutorial, or remixed it from something else, please link it here:

will do. give me some time

@qcoral, updated the design.. removed colours :D

This could use some more work! I would like to see more put in the art, different car sizes, different landscapes, etc. Also, some grass seems to be generating outside of the border! Please create a new PR once you have updated your art. I closing this due to a new management system for the PRs.