Blizzard: Overwatch stats not working without any competitive played.

When requesting the Overwatch stats for my account (Naichoryss#1808) I get the following error:

ERROR red on_command_error 364: Exception in command 'overwatch stats'
Traceback (most recent call last):
File "lib\discord\ext\commands\core.py", line 50, in wrapped
ret = yield from coro(*args, **kwargs)
File "botdir\cogs\blizzard.py", line 190, in _stats_overwatch
thumb_url = comp['overall_stats']['avatar']
TypeError: 'NoneType' object is not subscriptable

Obviously I've never played competitive.

Both issues acknowledged. @Naichoryss, your issue is probably because the battletag you checked has never played a competitive match. @HelmicNewciv, as you identified, the 3rd party API I am using probably pulls the previous season competitive stats if you haven't played this season. I will address both issues when I overhaul this cog.

I'm not a programmer, so I probably made a few formatting mistakes, and there's probably a better way to do this.
But here's what I changed to get it working for me. Just a few If Else.
    @overwatch.command(name="stats", pass_context=True)
    async def _stats_overwatch(self, ctx, tag: str=None, region: str=None):
        """Overwatch stats for your battletag (case sensitive and PC only!).
        If battletag is ommitted, bot will use your battletag if stored.
        Region is optional and will autodetect with this priority: kr>eu>us

        Example: [p]overwatch stats CoolDude#1234 kr
        """

        uid = ctx.message.author.id
        # Little hack to detect if region was entered, but not battletag
        if (tag in ['kr', 'eu', 'us']) and (region is None):
            region = tag
            tag = None

        if tag is None:
            if uid in self.settings['battletags']:
                tag = self.settings['battletags'][uid]
            else:
                await self.bot.say('You did not provide a battletag '
                                   'and I do not have one stored for you.')
                return

        tag = tag.replace("#", "-")
        url = 'https://owapi.net/api/v3/u/' + tag + '/stats'
        async with aiohttp.ClientSession(headers=self.header) as session:
            async with session.get(url) as resp:
                stats = await resp.json()

        if 'error' in stats:
            await self.bot.say('Could not fetch your statistics. '
                               'Battletags are case sensitive '
                               'and require a 4 or 5-digit identifier '
                               '(e.g. CoolDude#1234)'
                               'Or, you may have an invalid tag '
                               'on file.')
            return

        if region is None:
            if stats['kr']:
                region = 'kr'
                region_full = 'Asia'
            elif stats['eu']:
                region = 'eu'
                region_full = 'Europe'
            elif stats['us']:
                region = 'us'
                region_full = 'US'
            else:
                await self.bot.say('That battletag has no stats in any region.')
                return

        if stats[region] is None:
            await self.bot.say('That battletag exists, but I could not '
                               'find stats for the region specified. '
                               'Try a different region '
                               '<us/eu/kr> or leave that field blank '
                               'so I can autodetect the region.')
            return

        url = 'https://playoverwatch.com/en-us/career/pc/' + region + '/' + tag
        tag = tag.replace("-", "#")
        comp = stats[region]['stats']['competitive']
        qplay = stats[region]['stats']['quickplay']
        if comp == None:
            thumb_url = qplay['overall_stats']['avatar']
        else:
            thumb_url = comp['overall_stats']['avatar']

        if comp == None:
            icon_url = 'http://i.imgur.com/xU2iv6U.png'
        else:
            tier = comp['overall_stats']['tier']
            if tier == 'bronze':
                icon_url = 'http://i.imgur.com/B4IR72H.png'
            elif tier == 'silver':
                icon_url = 'http://i.imgur.com/1mOpjRc.png'
            elif tier == 'gold':
                icon_url = 'http://i.imgur.com/lCTsNwo.png'
            elif tier == 'platinum':
                icon_url = 'http://i.imgur.com/nDVHAbp.png'
            elif tier == 'diamond':
                icon_url = 'http://i.imgur.com/fLmIC70.png'
            elif tier == 'master':
                icon_url = 'http://i.imgur.com/wjf0lEc.png'
            elif tier == 'grandmaster':
                icon_url = 'http://i.imgur.com/5ApGiZs.png'
            else:
                icon_url = 'http://i.imgur.com/xU2iv6U.png'

        if comp == None:
            qplay_stats = ''.join(['**Wins:** ', str(int(round(qplay['game_stats']['games_won']))),
								   '\n**Avg Elim:** ', str(int(round(qplay['average_stats']['eliminations_avg']))),
								   '\n**Avg Death:** ', str(int(round(qplay['average_stats']['deaths_avg']))),
								   '\n**Avg Dmg:** ', str(int(round(qplay['average_stats']['damage_done_avg']))),
								   '\n**Avg Heal:** ', str(int(round(qplay['average_stats']['healing_done_avg'])))])
        else:
            comp_stats = ''.join(['**Wins:** ', str(int(round(comp['game_stats']['games_won']))),
								  '\n**Avg Elim:** ', str(int(round(comp['average_stats']['eliminations_avg']))),
								  '\n**Avg Death:** ', str(int(round(comp['average_stats']['deaths_avg']))),
								  '\n**Avg Dmg:** ', str(int(round(comp['average_stats']['damage_done_avg']))),
								  '\n**Avg Heal:** ', str(int(round(comp['average_stats']['healing_done_avg'])))])

            qplay_stats = ''.join(['**Wins:** ', str(int(round(qplay['game_stats']['games_won']))),
								   '\n**Avg Elim:** ', str(int(round(qplay['average_stats']['eliminations_avg']))),
								   '\n**Avg Death:** ', str(int(round(qplay['average_stats']['deaths_avg']))),
								   '\n**Avg Dmg:** ', str(int(round(qplay['average_stats']['damage_done_avg']))),
								   '\n**Avg Heal:** ', str(int(round(qplay['average_stats']['healing_done_avg'])))])

        embed = discord.Embed(title='Overwatch Stats (PC-' + region_full + ')', color=0xFAA02E)
        embed.set_author(name=tag, url=url, icon_url=icon_url)
        embed.set_thumbnail(url=thumb_url)
        if comp == None:
            embed.add_field(name='__Quick Play__', value=qplay_stats, inline=True)
        else:
            embed.add_field(name='__Competitive__', value=comp_stats, inline=True)
            embed.add_field(name='__Quick Play__', value=qplay_stats, inline=True)
        await self.bot.say(embed=embed)

Thanks so much for sharing. This is on the right track. The API gives a null value for competitive stats if no games are played. I'm working on this issue tonight, and I will use a similar method.
On a related note, does anybody have a confirmed example of a profile that has still not played in the current competitive season, but did play in the past one? That problem is proving a little more difficult to solve because I'm not seeing a season number in the API response.

As for someone who did play a previous season (don't know which) but not current season have a look at Ratman#2844
As far as I know he's not playing anymore.

This issue should be fixed with the latest update 👌 . The stats embed will now show that you haven't played any competitive games.


Error estimation method need to be updated for high count rate cases

Currently, errors are estimated assuming Poisson statistics. However, if there is significant saturation, the errors could be underestimated. See
N. P. M. Kuin, S. R. Rosen, The measurement errors in the Swift-UVOT and XMM-OM, Monthly Notices of the Royal Astronomical Society, Volume 383, Issue 1, 1 January 2008, Pages 383–386, https://doi.org/10.1111/j.1365-2966.2007.12549.x

Simple fix: apply the saturation correction.


Update pre-commit

coverage: 95.666% (+0.5%) from 95.161%
when pulling ce587bf6a61c1d97ed51b09809de8e44cc6ca84e on update-precommit
into acc1ba0a6998e7b991a6a42d0c67c4a56e3b3a17 on main.


Support for projects

Group tasks under projects, similar to the tasking manager workflow.
Some things to consider:

Project and task creation workflow. Should this be similar to tasking manager? A single geojson cut by a shape file or split by tile boundaries?
Integration with mapping tickets on GitHub.
Aggregating activity and statistics at project level.

cc: @aaronlidman @Rub21 @batpad

I think we should start doing this on the backend side.


Add setup detail to readme

Hello rutan.
In my opinion, the following document is need to setup this application.
How do you think about this?

Oh, I had forgotten to fix readme. Thanks :+1:


sqlite: update to 3.26.0

SQLite < 3.26 is vulnerable to the Magellan attack.
Release notes: https://www.sqlite.org/releaselog/3_26_0.html

See also https://news.ycombinator.com/item?id=18685296 for discussion

master has it (I guess it was added when #51352 got merged), in #52141 there is a discussion whether 3.25.3 actually has enough of the fix. The expectation is to merge the bump unless someone raises a substantial obkection. (which is why I closed #52225 when I found out I missed #52141)

Dup;l

Closed with d683610


 Decision is to document this as a limitation.
Q28 reported as bug: https://github.com/spiceai/spiceai/issues/3315


Sometimes execResult is undefined.

Hello,
Sometimes, execResult return is undefined. In this case, we just have to return.
Note: Commit updated.
Regards,

@luin Could you please merge this PR? 🙏

@rimiti Thank you for the pull request. I'm wondering in which case the pipeline will return void?

It occurred when I had a transaction with some del actions. I can’t share you a snipped code because it’s from my client software and the logic is too complicated to extract you an example.
Otherwise the fix doesn’t break the CI. It cant be safely merged.
Note: if you’re looking for a maintainer, I can help you.

I just did some tests on my side, but the results seem to always be arrays no matter what the commands were in the transaction. Here's a sample:
redis.multi().del('foo1').del('foo2').exec(console.log)

The output is:
null [ [ null, 1 ], [ null, 1 ] ]

Could you create a minimum reproducible sample for this fix?

I've re-run the tests with adding a console.log() when execResult is undefined:
if (typeof execResult === 'undefined') {
    console.log(pipeline);
    return;
}

Output:
tests    |        ...
tests    |        [ Command {
tests    |            name: 'multi',
tests    |            transformed: true,
tests    |            isCustomCommand: false,
tests    |            replyEncoding: 'utf8',
tests    |            errorStack: undefined,
tests    |            args: [],
tests    |            callback: undefined,
tests    |            resolve: [Function],
tests    |            reject: [Function],
tests    |            promise: [Promise] },
tests    |          Command {
tests    |            name: 'srem',
tests    |            transformed: true,
tests    |            isCustomCommand: false,
tests    |            replyEncoding: 'utf8',
tests    |            errorStack: undefined,
tests    |            args: [Array],
tests    |            callback: undefined,
tests    |            resolve: [Function],
tests    |            reject: [Function],
tests    |            promise: [Promise] },
tests    |          Command {
tests    |            name: 'srem',
tests    |            transformed: true,
tests    |            isCustomCommand: false,
tests    |            replyEncoding: 'utf8',
tests    |            errorStack: undefined,
tests    |            args: [Array],
tests    |            callback: undefined,
tests    |            resolve: [Function],
tests    |            reject: [Function],
tests    |            promise: [Promise] },
tests    |          Command {
tests    |            name: 'del',
tests    |            transformed: true,
tests    |            isCustomCommand: false,
tests    |            replyEncoding: 'utf8',
tests    |            errorStack: undefined,
tests    |            args: [Array],
tests    |            callback: undefined,
tests    |            resolve: [Function],
tests    |            reject: [Function],
tests    |            promise: [Promise] },
tests    |          Command {
tests    |            name: 'del',
tests    |            transformed: true,
tests    |            isCustomCommand: false,
tests    |            replyEncoding: 'utf8',
tests    |            errorStack: undefined,
tests    |            args: [Array],
tests    |            callback: undefined,
tests    |            resolve: [Function],
tests    |            reject: [Function],
tests    |            promise: [Promise] },
tests    |          Command {
tests    |            name: 'srem',
tests    |            transformed: true,
tests    |            isCustomCommand: false,
tests    |            replyEncoding: 'utf8',
tests    |            errorStack: undefined,
tests    |            args: [Array],
tests    |            callback: undefined,
tests    |            resolve: [Function],
tests    |            reject: [Function],
tests    |            promise: [Promise] },
tests    |          Command {
tests    |            name: 'srem',
tests    |            transformed: true,
tests    |            isCustomCommand: false,
tests    |            replyEncoding: 'utf8',
tests    |            errorStack: undefined,
tests    |            args: [Array],
tests    |            callback: undefined,
tests    |            resolve: [Function],
tests    |            reject: [Function],
tests    |            promise: [Promise] },
tests    |          Command {
tests    |            name: 'exec',
tests    |            transformed: true,
tests    |            isCustomCommand: false,
tests    |            replyEncoding: null,
tests    |            errorStack: undefined,
tests    |            args: [],
tests    |            callback: undefined,
tests    |            resolve: [Function],
tests    |            reject: [Function],
tests    |            promise: [Promise] },
tests    |          Command {
tests    |            name: 'del',
tests    |            transformed: true,
tests    |            isCustomCommand: false,
tests    |            replyEncoding: 'utf8',
tests    |            errorStack: undefined,
tests    |            args: [Array],
tests    |            callback: undefined,
tests    |            resolve: [Function],
tests    |            reject: [Function],
tests    |            promise: [Promise] },
tests    |          Command {
tests    |            name: 'del',
tests    |            transformed: true,
tests    |            isCustomCommand: false,
tests    |            replyEncoding: 'utf8',
tests    |            errorStack: undefined,
tests    |            args: [Array],
tests    |            callback: undefined,
tests    |            resolve: [Function],
tests    |            reject: [Function],
tests    |            promise: [Promise] } ],
tests    |       _result:
tests    |        [ [ null, 'OK' ],
tests    |          [ null, 'QUEUED' ],
tests    |          [ null, 'QUEUED' ],
tests    |          [ null, 'QUEUED' ],
tests    |          [ null, 'QUEUED' ],
tests    |          [ null, 'QUEUED' ],
tests    |          [ null, 'QUEUED' ],
tests    |          [ null, [Array] ],
tests    |          undefined,
tests    |          undefined ],
tests    |       _transactions: 0,
tests    |       _shaToScript: {},
tests    |       resolve: [Function],
tests    |       reject: [Function],
tests    |       promise:
tests    |        Promise {
tests    |          [ [Array],
tests    |            [Array],
tests    |            [Array],
tests    |            [Array],
tests    |            [Array],
tests    |            [Array],
tests    |            [Array],
tests    |            [Array],
tests    |            undefined,
tests    |            undefined ] },
tests    |       exec: [Function],
tests    |       execBuffer: [Function],
tests    |       nodeifiedPromise: true,
tests    |       replyPending: 0 }".
tests    |
tests    |       at BufferedConsole.log (node_modules/@jest/console/build/BufferedConsole.js:199:10)
tests    |       at node_modules/ioredis/built/transaction.js:47:25

I hope it can help you.

Thanks for the logs. I can finally reproduce the issue. The key point is to send other commands after exec():
const pipeline = redis.multi().del('foo1')
pipeline.exec(console.log) // TypeError: Cannot read property '0' of undefined
pipeline.del('foo3') // pipeline shouldn't be used to send any commands when the `exec()` has been called on it.

I think we should catch the exception (like this pr do) but re-throw it as a more meaningful error with a message like "Transaction resolved failed. Did you call any commands in a transaction after exec() had been called on that transaction?"
What do you think?

I think we should catch the exception (like this pr do) but re-throw it as a more meaningful error with a message like "Transaction resolved failed. Did you call any commands in a transaction after exec() had been called on that transaction?"

It think you're right, it's the better way to fix this case 👍

I think we should catch the exception (like this pr do) but re-throw it as a more meaningful error with a message like "Transaction resolved failed. Did you call any commands in a transaction after exec() had been called on that transaction?"

It think there are two solution:

First: Throwing an error when we are in this case
Second: Adding commands sent after exec() into the pipeline array.

I prefer the second solution because, it always works.

First: Throwing an error when we are in this case.

This is the quickest solution for this issue.

Second: Adding commands sent after exec() into the pipeline array.

This breaks BC since in the past commands invoked after exec() will not be sent to Redis. So even we chose this way, it have to be happened in the next major release.
Another cons I can think of is allowing commands after exec() makes things obscure: is these command will be included in the transaction, or be excluded from the transaction?

Another cons I can think of is allowing commands after exec() makes things obscure: Will these command be included in the transaction, or be excluded from the transaction?

I think, if it's obscure, it's because this is not the right way to do that. After reflection, it'll be more clear to just throw an error instead of trying to bypass transaction logic.
What disturbs me, if I run the same tests with redis package, it's works and without error.
It may be interesting to check how to the redis package treat this case ?

Just tried node_redis. The following code：
const m = r.multi().del('foo1')
m.exec(console.log)
m.del('foo2')

Both node_redis & ioredis will simply ignore del(foo2). node_redis returns [0, undefined] corresponding to del(foo1) and del(foo2). The result of del(foo2) being undefined is because the command was not sent to Redis.
I think in this case, simply throwing errors to let developers know that there are commands not being sent would be a fair solution.

Both node_redis & ioredis will simply ignore del(foo2). node_redis returns [0, undefined] corresponding to del(foo1) and del(foo2). The result of del(foo2) being undefined is because the command was not sent to Redis.

Oh, it's so a silent error... 😕
Totally agreed with you, throwing an error is best solution.

@luin Do you want I throw an error like that instead of the return?
Before
        if (typeof execResult === 'undefined') {
          return
        }

After
        if (typeof execResult === 'undefined') {
          throw new Error('Pipeline cannot be used to send any commands when the `exec()` has been called on it.');
        }

What do you think?

Looks good to me!

It can be merged. 🎉

@rimiti The new version has been published automatically when merged. I just add the 4.9.3 to the latest tag.


feat(ses): Add Compartment load function

This change surfaces the load function so compartments can be used to create archives without executing any of the assembled modules.

This PR is a delta on the ses-load branch. But is there a PR for the ses-load branch? I could not find one.

This PR is a delta on the ses-load branch. But is there a PR for the ses-load branch? I could not find one.

This PR should contain one commit that can be cherry-picked on master. This PR tracks the kris/ses-load branch and its differences atop the kris/endo-zip branch.

@erights This change is now at the head of the Endo queue and rebased so it should only show one commit again. (The artifact you saw was the product of the squash merge of another change in the queue; I need to promptly rebase the stack each time I land a change so you don’t see these intermediate states).


[Bug] 开发环境下，form渲染组件，控制台有 formMode warning提示，比较影响日志调试

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of DripForm、DripFormTheme.
[ ] I have read the console error message carefully (if applicable).

Description

Steps to reproduce

form渲染组件
控制台有 formMode warning提示

Current behavior
/
Expected behavior
/
Your environment

DripForm version used: 0.9.0-alpha.0
DripFormTheme used: 0.9.0-alpha.0
DripFormTheme version used: 0.9.0-alpha.0

Reproducible demo
No response
Self-service

[ ] I'd be willing to fix this bug myself.

Duplicate of #226


fix: make update-schema.ps1 more resilient

Description
Fixes some issues when files weren't present before trying to move or remove them.
Task Item
refs #3733

Coverage remained the same at 57.352% when pulling eb6fd2a7b19aa77014de0f117bafe7d53130ab83 on abrown/fix/schema-script into 501a81cbbd04ff9695133da27cda6376fd7a381b on main.


PointZilla has case-sensitive parameter matching

Had some unexpected failures when a parameter name was not perfectly matching the case.
HG had DisplayID="Gage height", but was able to create the time-series with "Gage Height".
$ PointZilla.exe -server=aqts-ora "Gage Height.Sine@DougDelete" -createmode=basic
10:22:24.606 INFO  - PointZilla v1.0.109
10:22:24.625 INFO  - Generated 1440 SineWave points.
10:22:24.628 INFO  - Connecting to aqts-ora ...
10:22:24.811 INFO  - Connected to aqts-ora (2<IP_ADDRESS>)
10:22:26.188 INFO  - Creating 'Gage Height.Sine@DougDelete' time-series ...
10:22:27.007 ERROR - System.Net.WebException: The remote server returned an error: (400) Bad Request.
   at System.Net.HttpWebRequest.GetResponse()
   at ServiceStack.ServiceClientBase.Send[TResponse](String httpMethod, String relativeOrAbsoluteUrl, Object request)
10:22:27.038 ERROR - Unhandled exception
400 IdenticalParameterAndLabelException
Code: IdenticalParameterAndLabelException, Message: A time-series already exists with the same label and parameter at this location.

$ PointZilla.exe -server=aqts-ora "Gage height.Sine@DougDelete" -createmode=basic
10:23:32.619 INFO  - PointZilla v1.0.109
10:23:32.638 INFO  - Generated 1440 SineWave points.
10:23:32.641 INFO  - Connecting to aqts-ora ...
10:23:33.329 INFO  - Connected to aqts-ora (2<IP_ADDRESS>)
10:23:35.368 INFO  - Appending 1440 points [2019-10-18T17:23:32Z to 2019-10-19T17:22:32Z] to Gage height.Sine@DougDelete (ProcessorBasic) ...
10:24:02.400 INFO  - Appended 1440 points (deleting 0 points) in 27.0 seconds.

Fixed by #69


Update access_tags.md

Remove text referring to the tag policy in Organizations, since there no way to enforce tag keys within tag policies, only tag values.
Issue #, if available:
Description of changes: Small change - removing reference to Tag Policies, since they're not relevant here.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for submitting this pull request.
The existing text is correct. AWS Organizations tag policies can be used to require specific tag key case for services and resource types that support enforcement.


Why does onLayout take so long to complete?

For Discussion
I built a Data Table that has both sticky rows and sticky columns as well as has the ability to resize the columns as well. It's all done and working but I have to pre render all the cells in order to get sizing and positioning information to facilitate the requirements. This pre render is getting hosed because onLayout takes forever to complete its process. It is almost as if there is a setTimeout somewhere under the hood hosing the entire process not to mention onLayout completes after componentDidMount so I have to keep track of how many cells are rendered just to determine when it finally finishes its process. Can anyone explain what the hold up with onLayout is and why this process cannot be sped up or at the least fired async? Am I missing something here? Should I be running a pre renderer component hidden in the background? Seems a bit much but my only alternative at this point in order to truly get infinite scroll and still have dynamically sized rows/columns....
I propose making it to where at least the componentDidMount fires after onLayout finishes. At best make it to where the onLayout calls are fired all at once so they finish asynchronous. Thoughts? Suggestions?

Hey, the Discussion template is meant for long-form discussions about React Native. This looks like a request for help. Please take a look at http://facebook.github.io/react-native/en/help.html for a list of help resources.


⚠ GitLab Server has degraded performance

In ff4e7cd, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6135 ms

Resolved: GitLab Server performance has improved in ca022f4.


`Failed to find C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\ckpts\lllyasviel/Annotators\150_16_swin_l_oneformer_coco_100ep.pth.

Hi,
I'm trying to use this workflow: https://civitai.com/models/420317?modelVersionId=468375
All Missing Nodes successfully installed via. ComfyUI Manager
I still got this Error.
Failed to find C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\ckpts\lllyasviel/Annotators\150_16_swin_l_oneformer_coco_100ep.pth.  Downloading from huggingface.co cacher folder is C:\Users\Gianluca\AppData\Local\Temp, you can change it by custom_tmp_path in config.yaml C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\python_embeded\Lib\site-packages\huggingface_hub\file_download.py:1132: FutureWarning: resume_downloadis deprecated and will be removed in version 1.0.0. Downloads always resume when possible. If you want to force a new download, useforce_download=True.   warnings.warn( C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\python_embeded\Lib\site-packages\huggingface_hub\file_download.py:1194: UserWarning: local_dir_use_symlinks parameter is deprecated and will be ignored. The process to download files to a local folder has been updated and do not rely on symlinks anymore. You only need to pass a destination folder aslocal_dir. For more details, check out https://huggingface.co/docs/huggingface_hub/main/en/guides/download#download-files-to-local-folder.   warnings.warn( !!! Exception during processing!!! [Errno 2] No such file or directory: 'C:\\Users\\Gianluca\\Downloads\\ComfyUI_windows_portable\\ComfyUI\\custom_nodes\\comfyui_controlnet_aux\\ckpts\\lllyasviel\\Annotators\\.huggingface\\download\\150_16_swin_l_oneformer_coco_100ep.pth.0684dfc39720c772b81d43639c3ae1896b5c15aa9ee9a76f4c593b19dfa33855.incomplete' Traceback (most recent call last):   File "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\ComfyUI\execution.py", line 151, in recursive_execute     output_data, output_ui = get_output_data(obj, input_data_all)                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\ComfyUI\execution.py", line 81, in get_output_data     return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\ComfyUI\execution.py", line 74, in map_node_over_list     results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\node_wrappers\oneformer.py", line 17, in semantic_segmentate     model = OneformerSegmentor.from_pretrained(filename="150_16_swin_l_oneformer_coco_100ep.pth")             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\oneformer\__init__.py", line 32, in from_pretrained     model_path = custom_hf_download(pretrained_model_or_path, filename)                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\src\controlnet_aux\util.py", line 324, in custom_hf_download     model_path = hf_hub_download(repo_id=pretrained_model_or_path,                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\python_embeded\Lib\site-packages\huggingface_hub\utils\_validators.py", line 114, in _inner_fn     return fn(*args, **kwargs)            ^^^^^^^^^^^^^^^^^^^   File "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\python_embeded\Lib\site-packages\huggingface_hub\file_download.py", line 1202, in hf_hub_download     return _hf_hub_download_to_local_dir(            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\python_embeded\Lib\site-packages\huggingface_hub\file_download.py", line 1487, in _hf_hub_download_to_local_dir     _download_to_tmp_and_move(   File "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\python_embeded\Lib\site-packages\huggingface_hub\file_download.py", line 1872, in _download_to_tmp_and_move     with incomplete_path.open("ab") as f:          ^^^^^^^^^^^^^^^^^^^^^^^^^^   File "pathlib.py", line 1044, in open FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\Gianluca\\Downloads\\ComfyUI_windows_portable\\ComfyUI\\custom_nodes\\comfyui_controlnet_aux\\ckpts\\lllyasviel\\Annotators\\.huggingface\\download\\150_16_swin_l_oneformer_coco_100ep.pth.0684dfc39720c772b81d43639c3ae1896b5c15aa9ee9a76f4c593b19dfa33855.incomplete'
I manually downloaded https://huggingface.co/lllyasviel/Annotators/blob/main/150_16_swin_l_oneformer_coco_100ep.pth and put it in ComfyUI\custom_nodes\comfyui_controlnet_aux\ckpts\lllyasviel\Annotators\.huggingface\download\ (since it was empty), but it still doenst work.
I tried a lot random stuff with chatGPT, but still not working.
Re-installing and repeating only the most necessary steps -> still not working.
THank you.
BR
Gianluca

Hi, this repo is for diffusers, you need to report this is the correct comfyui one.

I think I fixed it, thanks to chatgpt.
This file  https://huggingface.co/lllyasviel/Annotators/blob/main/150_16_swin_l_oneformer_coco_100ep.pth
shall be put into "\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\ckpts\lllyasviel\Annotators\150_16_swin_l_oneformer_coco_100ep.pth"
and NOT into
"C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\ckpts\lllyasviel\Annotators.huggingface\download\150_16_swin_l_oneformer_coco_100ep.pth"
also adjust config.yaml in confyui_controlnet_aux
annotator_ckpts_path: "C:\Users\Gianluca\Downloads\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui_controlnet_aux\ckpts"


Module5: Kanban board

https://github.com/users/derrick1451/projects/1/views/1
2 members

https://github.com/users/derrick1451/projects/1/views/1
2 members

Hi @derrick1451 and @kayjeee,
While you made a great effort in this project, I cannot proceed to review your code.
Invalid Code Review Request
Your Kanban link is broken or the project is private and I cannot access it.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi Team,
Good job so far, indeed you have spent time on this project!
There are some issues that you still need to work on to go to the next project but you are almost there!

Highlights

Good use of Kanban board ✔
All tasks added ✔

Required Changes ♻
Check the comments under the review.

[ ] I can see you have a "No Status" category, I will encourage you to remove it and add all to the Todo category since that is what is to be done.
[ ] I will also encourage you to distribute the tasks fairly as I can see from the time assigned that one is significantly more than the other.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

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

Hi @derrick1451 and @kayjeee,
APPROVED 🟢
Highlights
✔ Assigned tasks
Great implementation of all the requirements in this milestone. 👏
You have done an amazing job 👍
Your project is complete!  There is nothing else to say other than... it's time to merge it. :shipit:
Congratulations! 🎉
CHEERS AND HAPPY CODING!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

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


Custom routes in app/router.options.ts don't work

Environment

Operating System: Windows_NT
Node Version:     v16.17.0
Nuxt Version:     3.0.0-rc.10
Nitro Version:    0.5.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
Minimum Reproduction: https://stackblitz.com/edit/github-fkskej
Describe the bug
Hi,
it looks like the routes overriding feature introduced in nuxt/framework#7129 (included in RC 10) doesn't work. After starting a completely new project & copy-pasting an example from the documentation, I get the error:
500: Cannot read properties of undefined (reading 'replace')
at Module.generateRouteKey (../nuxt-app/node_modules/nuxt/dist/pages/runtime/utils.mjs:8:73)
at default (../nuxt-app/node_modules/nuxt/dist/pages/runtime/page.mjs:47:45)
at ..\nuxt-app\node_modules\@vue\runtime-core\dist\runtime-core.cjs.js:4166:35
at renderFnWithContext ..\nuxt-app\node_modules\@vue\runtime-core\dist\runtime-core.cjs.js:842:21)
at normalizeSlot (../nuxt-app/node_modules/vue-router/dist/vue-router.mjs:2455:25)
at Proxy. (../nuxt-app/node_modules/vue-router/dist/vue-router.mjs:2447:13)
at renderComponentRoot (..\nuxt-app\node_modules\@vue\runtime-core\dist\runtime-core.cjs.js:885:44)
at renderComponentSubTree (..\nuxt-app\node_modules\@vue\server-renderer\dist\server-renderer.cjs.js:263:51)
at renderComponentVNode (..\nuxt-app\node_modules\@vue\server-renderer\dist\server-renderer.cjs.js:188:16)

Additional context
No response
Logs
No response

when i deploy my nuxt 3 on the server specially chrome it gives me blank white page but when i remove the router.options.ts it works but just in home page any help please @danielroe

@hoSaMShoukry  please create a new issue with a reproduction and i'll happily help!


Primary website directus-extensions.codihaus.io not working

Hi,
I am unable to find any information on using this plugin due to no README.md and the primary link of the site returning a cloudflare bad gateway error.
If there's an alternate URL I'd appreciate a response.
Thank you

Is this repo maintained? It feels a bit dead, the idea however is really good!


Renyi DP

Closes #1168

#2027  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Shoeboxam and the rest of your teammates on  Graphite


int() rounds negative numbers in the wrong direction

int(-2.5) # produces -2, should be -3

int() should always round down, but currently appears to round toward zero.

No actually this behaviour is correct, int should truncate towards zero and not round down. So rounding towards zero is right for python.

Oops, looks like I was mistaken about this one. Thanks, I'll close the issue.


Pledge Taken

Thanks for your PR @G1Joshi . You come from a beautiful place. Welcome to the community.

Thank you.
It was my very first successful pull request and 1st step towards open source contribution.

Oh great! have a great way ahead! keep doing it. Goodluck!


Chore: Cleaned up types

regex replace
MapFunction<(.*?)> = function\(source(:.+)\)(:.+) \{$
MapFunction<$1> = function(source) {

Is the idea here just that this typing was redundant?

@space-age-pete exactly, just adds noise

@space-age-pete I believe at one point either because of typescript or eslint (maybe both) the types were necessary. But something got updated and now they're not necessary so nice to be able to remove them.
thanks @stackoverfloweth !


Add AUR package to README installation instructions

Closes #1313
Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Add the AUR package to README

See https://github.com/yt-dlp/yt-dlp/issues/1313#issuecomment-945153465


Update transitions.md

More correct description for 'v-leave'

I'm not sure this is accurate actually. If it was removed one frame after the element was removed, wouldn't that mean it's never removed? There'd be no element to remove it from at that point, right?


installer_mac issue with file permission issue.

jdk11u-mac-x64-openj9
https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-mac-x64-openj9/622/console
04:47:56  Starting building: build-scripts » release » create_installer_mac #3352
[Pipeline] copyArtifacts
[Pipeline] }
[Pipeline] // stage
[Pipeline] }
[Pipeline] // node
[Pipeline] echo
05:04:38  Execution error: Unable to get file permissions
[Pipeline] }
looks like something went wrong in overwriting file ?, there is no other error or explanation I can see. Might require cleaning machine if it is not done auto clearing before build.

Its odd, as the stage view says the installer job has failed, but the status page of that installer job says its passed https://ci.adoptopenjdk.net/job/build-scripts/job/release/job/create_installer_mac/3352/

Ahh, the Copy artifacts from another project part of the installer stage failed.  The failure log isnt helpful

https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-mac-x64-openj9/623/
On the same machine, the latest build does not fail, so im putting this down to an intermittent failure.
Will keep an eye on it

@karianna I'm still seeing this on a few builds except the installer are passing this time (in the case of the windows job below anyway, I don't have the perms to view the mac ones). Do the installers need the same change you made with George yesterday?
Only the artifacts are failing to be copied across for JRE job. FYI @douph1

https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-mac-x64-hotspot/613/console
https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk14u/job/jdk14u-mac-x64-openj9-macosXL/101/console
https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk14u/job/jdk14u-windows-x86-32-hotspot/98/console

23:18:39  Scheduling project: build-scripts » release » create_installer_windows
23:18:48  Starting building: build-scripts » release » create_installer_windows #4502
[Pipeline] copyArtifacts
[Pipeline] }
[Pipeline] // stage
[Pipeline] }
[Pipeline] // node
[Pipeline] echo
23:22:10  Execution error: Unable to get file permissions

@karianna I'm still seeing this on a few builds except the installer are passing this time (in the case of the windows job below anyway, I don't have the perms to view the mac ones). Do the installers need the same change you made with George yesterday?
Only the artifacts are failing to be copied across for JRE job. FYI @douph1

https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-mac-x64-hotspot/613/console
https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk14u/job/jdk14u-mac-x64-openj9-macosXL/101/console
https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk14u/job/jdk14u-windows-x86-32-hotspot/98/console

23:18:39  Scheduling project: build-scripts » release » create_installer_windows
23:18:48  Starting building: build-scripts » release » create_installer_windows #4502
[Pipeline] copyArtifacts
[Pipeline] }
[Pipeline] // stage
[Pipeline] }
[Pipeline] // node
[Pipeline] echo
23:22:10  Execution error: Unable to get file permissions

We updated all of the jobs that were impacted (mac install, win installer, and sign release).  Will need to do some more digging on this one.  Let's meet up on slack

Seen again on https://ci.adoptopenjdk.net/view/Failing Builds/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-windows-x64-openj9/630/console
@Haroon-Khel @archanaNogriya Can this be reopened?
@karianna We never managed to meet up the other day to discuss this. Is today better for you?

From https://plugins.jenkins.io/copyartifact/

Specify projects who can copy artifacts

Scripted pipeline

properties([
    copyArtifactPermission('job1,job2,...'),
]);
node {
    ...
}

You can specify multiple jobs separated by commas.
You can use wildcard character ('*') to specify name patterns.

After a quick search, we only seem to use,
copyArtifactPermissionProperty {
  projectNames('*')
}

not copyArtifactPermission. Is it possible that copyArtifactPermission is deprecated now that we've switched to Production mode @karianna ?

@M-Davies  - It could well be, worth trying I reckon if you're happy to submit a PR?


Remove a spurious inclusion of the sparse array header file

This pull request is ready to merge

Merged, thanks.


Scroll causes component with useThree hook to rerender

I specifically noticed this issue when using react-spring to animate a property of a react-three-fiber component.
If the useThree hook is used anywhere in the tree, scrolling will cause the size value to recalculate in order to update the bottom and top values. If a spring going to a chained value (an array of values) is used that spring chain will occur again every time the size is recalculated.
Here is an example of what I am referring to https://codesandbox.io/s/stoic-wozniak-z58lm.

size changes because the canvas is in a nested scroll area, it must recalculate or else you wouldnt be able to interact/raycast. later react-spring (all the canary versions) have update dependencies:
  const spring = useSpring(
    toggle
      ? { to: [{ rotation: [0, 1, 0] }, { rotation: [0, 2, 0] }],  from: { rotation: [0, 0, 0] }}
      : { to: { rotation: [0, 0, 0] } }
  , [toggle])

Ok, thanks for the response.
By the way, I really like the idea of update dependencies for react-spring!

I've run into this issue, but I cannot turn off scroll do to click events. However my component in its current state is unusable due to textures flickering from re-rendering on scroll. How can I get around this?
Working example: https://dev-joshkincheloe.herokuapp.com/
Might be a little slow to load since I'm using a free dino for hosting my dev site.
`export const Model = memo(({ animating, handleActive, images }) => {
const loader = new THREE.TextureLoader();
const meshRef = useRef(null);
const mouse = new THREE.Vector2();
const props = useSpring({
scale: animating ? [1, 1, 1] : [3.5, 3.5, 3.5]
});
const textures = images.map(image => loader.load(image));
const raycaster = new THREE.Raycaster();
const { camera } = useThree();
function handleClick(e) {
const rect = document.getElementById('cubeCanvas').getBoundingClientRect();
mouse.x = ((e.clientX - rect.left) / rect.width) * 2 - 1;
mouse.y = -((e.clientY - rect.top) / rect.height) * 2 + 1;
raycaster.setFromCamera(mouse, camera);

const intersects = raycaster.intersectObject(meshRef.current);

if (intersects.length > 0) {
  const index = Math.floor(intersects[0].faceIndex / 2);
  handleActive(index);
}

}
return (
<a.mesh onClick={handleClick} ref={meshRef} scale={props.scale}>
<boxBufferGeometry attach="geometry" args={[1, 1, 1]} />
{textures.map((texture, i) => (

))}
</a.mesh>
);
});`

even if it re-renders, it shouldn't flicker, that points to an error. some component re-mounts, some texture gets re-created, some object re-instanciated, something like that.
i think it may be this:
const textures = images.map(image => loader.load(image));

that's a side effect, inside your render function. it will re-load textures every time model renders. i would suggest you fix that - either use useState/useEffect to handle effects, or, i think that's better, useLoader and suspense.

Ahhh this makes sense now that I know useThree() is meant to re-render. Immediately fixed the flickering thank you!
And oh my goodness you're right about not needing the raycaster. Didn't think that a regular click event would have that!
Appreciate it! Everything is working and performing very well now!


bug(workflow): error during startup because of missing service

Publishing DB state to k8s...
Traceback (most recent call last):
  File "/app/manage.py", line 12, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/lib/python3.5/site-packages/django/core/management/__init__.py", line 353, in execute_from_command_line
    utility.execute()
  File "/usr/lib/python3.5/site-packages/django/core/management/__init__.py", line 345, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/lib/python3.5/site-packages/django/core/management/base.py", line 348, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/lib/python3.5/site-packages/django/core/management/base.py", line 399, in execute
    output = self.handle(*args, **options)
  File "/app/api/management/commands/load_db_state_to_k8s.py", line 15, in handle
    obj.save()
  File "/app/api/models/domain.py", line 23, in save
    config = self._load_service_config(app, 'router')
  File "/app/api/models/__init__.py", line 71, in _load_service_config
    svc = self._fetch_service_config(app)
  File "/app/api/models/__init__.py", line 61, in _fetch_service_config
    svc = self._scheduler._get_service(app, app).json()
  File "/app/scheduler/__init__.py", line 1054, in _get_service
    error(response, 'get Service "{}" in Namespace "{}"', name, namespace)
  File "/app/scheduler/__init__.py", line 281, in error
    raise KubeHTTPException(errmsg, response=response)
scheduler.KubeHTTPException: failed to get Service "app-name" in Namespace "app-name": 404 Not Found
{'status': 'Failure', 'metadata': {}, 'message': 'services "app-name" not found', 'reason': 'NotFound', 'kind': 'Status', 'apiVersion': 'v1', 'code': 404, 'details': {'kind': 'services', 'name': 'app-name'}}

We have the same issue with missing/conflicting Namespace and RC as well.
Did your particular situation happen due to a failed deploy?

@helgi I removed the service just to see if is recreated using scale (I saw the same thing with ns and rc)

Ah, that's a known failure scenario. Right now it is all pretty dumb but I have a few things in the works to ease that up with the eventual plan of being able to fill in the blanks.
I'll create a meta ticket to track all the various connected open tickets revolving around this.

Talking to @aledbf and he ran the command again with the same result but it looks like the Domain model is potentially ran before App is ever invoked

#482 got merged with the fixed and verified by @aledbf


Features Request: Mark All Read

Please consider adding an option to mark all read in the RSS feeds.

I dont see any compelling reason to add "mark all as read" because when new RSS list was provided by the source old RSS will be removed.

I didn't know they automatically are deleted. If I see new bold RSS feeds I will just long press to make them read for now.

Mark as read doesn't automatically delete RSS. And doesn't related with the feature request that you want. If you think you have encounter a bug then log a new issue and explain it properly with step by step of how you encounter the issue because As both user and a developer I really can't understand what you are trying to describe and explain.

And as per what I stated there is no compelling reason, and you are not providing more reason of why the feature should be added. I will close this issue

Well, if someone does have lots of items in a feed, it just would take more time to mark them all read. I will just hold down on my phone in the app until all are unread.


Searching with predicates

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


Checking out my develop env broke after merging/switching to master env

Note: If your issue/bug is regarding the AWS Amplify Console service, please log it in the
Amplify Console GitHub Issue Tracker

Describe the bug
After checking out the master branch and fixing issues setting up the master env based on the develop environment, I switched back to the develop env to find a bunch of errors stopping me from checking out the env.

✦ ➜ amplify env checkout develop
Current environment cannot be determined
Use 'amplify init' in the root of your app directory to initialize your project with Amplify
UndeterminedEnvironmentError: Current environment cannot be determined
Use 'amplify init' in the root of your app directory to initialize your project with Amplify
    at getEnvInfo (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/lib/extensions/amplify-helpers/get-env-info.js:18:15)
    at Object.run (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/lib/commands/env/checkout.js:17:30)
    at Object.run (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/lib/commands/env.js:25:37)
    at Object.executeAmplifyCommand (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/lib/index.js:110:25)
    at executePluginModuleCommand (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/lib/execution-manager.js:131:32)
    at async Object.executeCommand (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/lib/execution-manager.js:16:9)
    at async Object.run (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/lib/index.js:54:9)

app on  develop [⇡$] is 📦 v0.1.0 via ⬢ v12.11.1
✦ ➜ amplify status
| Category                 | Resource name                          | Operation | Provider plugin   |
| ------------------------ | -------------------------------------- | --------- | ----------------- |
| Auth                     | xxxxxxxxxxx                 | Create    | awscloudformation |
| Auth                     | userPoolGroups                         | Create    | awscloudformation |
| Function                 | xxxxxxxxxxx | Create    | awscloudformation |
| Function                 | xxxxxxxxxxx                   | Create    | awscloudformation |
| Function                 | xxxxxxxxxxx                | Create    | awscloudformation |
| Function                 | xxxxxxxxxxx                | Create    | awscloudformation |
| Function                 | xxxxxxxxxxx                          | Create    | awscloudformation |
| Function                 | xxxxxxxxxxx               | Create    | awscloudformation |
| Function                 | xxxxxxxxxxx           | Create    | awscloudformation |
| Function                 | xxxxxxxxxxx                | Create    | awscloudformation |
| Function                 | xxxxxxxxxxx                      | Create    | awscloudformation |
| Storage                  | xxxxxxxxxxx                             | Create    | awscloudformation |
| Hosting                  | S3AndCloudFront                        | Create    | awscloudformation |
| Api                      | xxxxxxxxxxx                         | Create    | awscloudformation |
| Api                      | xxxxxxxxxxx                              | Create    | awscloudformation |
| Analytics                | xxxxxxxxxxx                | Create    | awscloudformation |
| Lambdatriggerpermissions | xxxxxxxxxxx            | Create    | awscloudformation |

ENOENT: no such file or directory, open '/home/acidburn/Documents/xxxxxxxxxxx.app/xxxxxxxxxxx.app/amplify/backend/amplify-meta.json'
Error: ENOENT: no such file or directory, open '/home/acidburn/Documents/xxxxxxxxxxx.app/xxxxxxxxxxx.app/amplify/backend/amplify-meta.json'
    at Object.openSync (fs.js:451:3)
    at Object.readFileSync (fs.js:353:35)
    at readJsonFile (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/src/extensions/amplify-helpers/read-json-file.js:14:33)
    at AmplifyToolkit.getProjectMeta [as _getProjectMeta] (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/src/extensions/amplify-helpers/get-project-meta.js:6:23)
    at showPinpointURL (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/lib/display-helpful-urls.js:20:41)
    at displayHelpfulURLs (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/lib/display-helpful-urls.js:6:3)
    at Object.showHelpfulLinks (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/index.js:83:10)
    at /home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/src/extensions/amplify-helpers/show-helpful-provider-links.js:14:40
    at Array.forEach (<anonymous>)
    at AmplifyToolkit.showHelpfulProviderLinks [as _showHelpfulProviderLinks] (/home/acidburn/.nvm/versions/node/v12.11.1/lib/node_modules/@aws-amplify/cli/src/extensions/amplify-helpers/show-helpful-provider-links.js:12:13)
acidburn@gibson

So then I added back in the amplify-meta.json file and then also the local-env-info.json file that another issue on here mentioned to exclude via .gitignore to stop merge conflicts and then run
amplify env checkout develop again and all seems well. What am I doing wrong here? I would like to be able to go between environments quickly without fixing merge conflicts every time and having to add back in gitignored files
Amplify CLI Version
4.21.0
To Reproduce
As above
Expected behavior
I would like to be able to go between environments quickly without fixing merge conflicts every time and having to add back in gitignored files
Desktop (please complete the following information):

OS: Ubuntu 18.04
12.11.1

Thanks in advance,

@kylekirkby I see that something went off for sure in your case, and currently also gathering more information around team workflow issues in #4587 if you can help provide complete repro steps including git and amplify commands that would enable is to see if it is a workflow problem on your end or an actual code problem.
Also share any feedback about the merge conflicts, what files are usually affected any specific sections? How can we make documentation better.

@attilah I'm going to be merging some changes over into my production branch / env this weekend.
I'll let you know if I come across the same / other issues surrounding multi environments. Do you have the list of files that should be in the .gitignore listed some place for reference? Not entirely sure why I didn't keep the .gitignore they way it was when I ran amplify init 4 months ago! Just checked my team-provider-info.json file and I've only got the develop env currently; shouldn't this have the master env too?

Hi @kylekirkby ,

Do you have the list of files that should be in the .gitignore listed some place for reference?

You can find the reference to Amplify CLI's .gitignore here


fix scroll preventDefault

After update React version, out team suffer from a issue about "passive event listener invocation".
So I remove preventDefault Methods, and invoke this when event is not passive one.
Related to https://github.com/facebook/react/pull/19654/files

Can we get this merged @adcentury? I'm getting tons of console log errors. "Unable to preventDefault inside passive event listener invocation."

Can we please get this merged @adcentury? I'm getting tons of console log errors. "Unable to preventDefault inside passive event listener invocation."
(And I'm unable to get values from the input anymore).

@pius712 Thanks for this contribution, new version (v0.1.13) with these changes has been published to npm. cc @Paxatax

@pius712 Thanks for this contribution, new version (v0.1.13) with these changes has been published to npm. cc @Paxatax

Could you change contribution id if possible? I committed with previous github id.. pius712 <--- my private github id

@pius712 Thanks for this contribution, new version (v0.1.13) with these changes has been published to npm. cc @Paxatax

Could you change contribution id if possible? I committed with previous github id.. pius712 <--- my private github id

Sorry I can't. The contribution info is auto-generated by GitHub, and it relates to your git config, so I can not change it.

@adcentury it seems v0.1.13 does not include this change correctly. I can see it in the source (src/index), but not in built main file (lib/react-mobile-picker.js)


VCPKG_FORCE_SYSTEM_BINARIES needs to be set during vcpkg bootstrap

On a Mac M1 (arm64) the feature build fails because VCPKG_FORCE_SYSTEM_BINARIES has not been set.
This one-line change fixed it:
VCPKG_FORCE_SYSTEM_BINARIES=1 "${VCPKG_ROOT}"/bootstrap-vcpkg.sh

which I brought into my dev container locally as a workaround, but it would be nice to have it fixed at source.

On a Mac M1 (arm64) the feature build fails because VCPKG_FORCE_SYSTEM_BINARIES has not been set.
This one-line change fixed it:
VCPKG_FORCE_SYSTEM_BINARIES=1 "${VCPKG_ROOT}"/bootstrap-vcpkg.sh

which I brought into my dev container locally as a workaround, but it would be nice to have it fixed at source.

@0xg0nz0 Thanks for your suggestion. I will fix it later.

@msclock this appears to be still a problem, but only for vcpkg-test linking (is that even needed?):
  [2024-04-20T14:57:03.427Z] #42 1434.2 [216/216] Linking CXX executable vcpkg-test
  [2024-04-20T14:57:03.535Z] #42 1434.3 Environment variable VCPKG_FORCE_SYSTEM_BINARIES must be set on arm, s390x, ppc64le and riscv platforms.

FYI, full failed CI build when I remove my workaround to export VCPKG_FORCE_SYSTEM_BINARIES=1:
https://github.com/iggy-rs/iggy-cpp-client/actions/runs/8765694062/job/24056959525

Reading the relevant scripts, I have a hunch this line in main.sh is the issue:
"${VCPKG_ROOT}"/bootstrap-vcpkg.sh

The failure is happening during the build of the vcpkg binary, and it looks like you added this in your updaterc function. It may be that this has not yet taken effect by the time you bootstrap. Maybe try this?
VCPKG_FORCE_SYSTEM_BINARIES=1 "${VCPKG_ROOT}"/bootstrap-vcpkg.sh

@0xg0nz0 Ah, I have missed it. And now it works after #21 merged.

Thanks @msclock ! You can close this, I can confirm it's working my end with the latest dev container build. Thanks for contributing this feature!

you're welcome:)


remove number_of_workers workaround, add acr_use_managed_identity_cre…

The original pull request was accidentally closed.
1011
PR Checklist

[X] I have added example(s) inside the [./examples/] folder
[ ] I have added the example(s) to the integration test list for normal (~30 minutes) or long runner >30 minutes
[X] I have checked the coding conventions as per the wiki
[X] I have checked to ensure there aren't other open Pull Requests for the same update/change?

Description

Does this introduce a breaking change

[ ] YES
[X] NO

Testing

What happend to this pull request???


Boolean values should not be quoted in output

When outputting to YAML or JSON, boolean values should not have quotes around them. An output file will produce E3012 errors when ran through cfn-lint because of this. See: https://github.com/awslabs/cfn-python-lint/issues/260 for more details on why its not a warning.
Changing the validators.boolean function to return bool values, rather than strings would correct this issue.  I can make a PR for this, but I wanted to discuss any possible repercussions first, since all these  types would need to be verified.

this is related https://github.com/cloudtools/troposphere/pull/1136

This is fixed in #1409


Doc fix at io.netty.util.concurrent.AbstractEventExecutor#lazyExecute

LazyRunnable already move to AbstractEventExecutor.
Fixes #11021

@horizonzy thanks a lot!


symfony 3.3 issues

I use Laravel and have Symfony v3.3 dependecies installed by the framework.
composer CurrencyCloud/currencycloud-php is giving me an error:

Installation request for currency-cloud/client ^0.10.0 -> satisfiable by currency-cloud/client[v0.10.0].
Conclusion: remove symfony/event-dispatcher v3.3.12
Conclusion: don't install symfony/event-dispatcher v3.3.12
Installation failed, reverting ./composer.json to its original content.

Is there any particular reason for this? What is your advice? I don't think Laravel lets me roll back to ^2.8
Thanks!
Steve
This is from the laravel/framework composer.json:
 "require": {
        "php": ">=7.0",
        "ext-mbstring": "*",
        "ext-openssl": "*",
        "doctrine/inflector": "~1.1",
        "erusev/parsedown": "~1.6",
        "league/flysystem": "~1.0",
        "monolog/monolog": "~1.12",
        "mtdowling/cron-expression": "~1.0",
        "nesbot/carbon": "~1.20",
        "psr/container": "~1.0",
        "psr/simple-cache": "^1.0",
        "ramsey/uuid": "~3.0",
        "swiftmailer/swiftmailer": "~6.0",
        "symfony/console": "~3.3",
        "symfony/debug": "~3.3",
        "symfony/finder": "~3.3",
        "symfony/http-foundation": "~3.3",
        "symfony/http-kernel": "~3.3",
        "symfony/process": "~3.3",
        "symfony/routing": "~3.3",
        "symfony/var-dumper": "~3.3",
        "tijsverkoyen/css-to-inline-styles": "~2.2",
        "vlucas/phpdotenv": "~2.2"
    },

Fixed in v1.0.1


Edit supported platforms page

Our supported platforms page has a few issues.

Some grammatical issues.
IE11 listed as supported when it's not.
Safari version listed twice when it's the same value.
We use >= to show "X or above" but may not be clear to all readers.
Browser support in a table but only has two rows—not ideal for mobile.

Content to edit the page and tweak the above issues.

Yep 0 support for IE


Error: spawnSync /bin/sh E2BIG

fails to execute ffmpeg because of too long command line.
We have a lot of recorded lecture meetings with a bunch of slides and a huge amount of cursor movements and drawings.
I have seen e.g. about 20000 drawboxes and about 3000 pencil pngs. This results in a command line of more than MAX_ARGS=2097152 bytes which are allowed an execve system call.
Is there a possibility to split the ffmpeg calls into separate calls where the command line is below the MAX_ARGS limit?
I can provide logs.

@christmart This problem is already being worked on. See #6

And this issue should be fixed. There's still a bug regarding desksharing, which I'll fix today or latest tomorrow.


The html rendering result is different from the html output result when tbody is added inside unclosed th

When we have a tbody tag inside a unclosed th then the output is different from the way chrome renders it.
While closes the entire thead, HAP adds the tbody tag inside the th.
HTML:

HAP Result:

Chrome Result:

This is the test I used:
var html = @"

";
var doc = new HtmlDocument();
doc.LoadHtml(html);
var newHtml = doc.DocumentNode.OuterHtml;  //

var res = @"

"; //chrome or edge rendering results
var b = newHtml == res; //false

Thank you for reporting.
The fix has been done and will be available the next time we deploy Html Agility Pack
Best Regards,
Jon

Hello @pauloortins ,
The v1.11.58 has been released.
The issue you reported should now be fixed.
Best Regards,
Jon

Thank you Jon!
On Mon, Jan 29, 2024 at 2:51 AM Jonathan Magnan @.***>
wrote:

Hello @pauloortins https://github.com/pauloortins ,
The v1.11.58 has been released.
The issue you reported should now be fixed.
Best Regards,
Jon
—
Reply to this email directly, view it on GitHub
https://github.com/zzzprojects/html-agility-pack/issues/530#issuecomment-1914007007,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIXWK6HG75NIQY5LFJ4MJ3YQ42FJAVCNFSM6AAAAABBOUWTY2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJUGAYDOMBQG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Paulo Ortins - www.pauloortins.com
Tel.: (71) 9354-4577


Add command to explicitly delete cached colorscheme

Thank you for your wonderful script.
I've been using
Invoke-WebRequest -Uri https://source.unsplash.com/random -OutFile C:\Users\jayma\Documents\wallpaper.jpg
to download a wallpaper and save it to the same file every time. On Linux I'd use wal -c & wal -i path/to/wallpaper.jpeg to resolve this.
I'm new to PowerShell scripting. How could I regenerate the colour scheme every time?

Ideally I want this to run automatically on a wallpaper change, how could I got about doing that? power automate?

I'm not familiar with triggering on wallpaper change. I think for your use case you can do something like:
Invoke-WebRequest -Uri https://source.unsplash.com/random -OutFile C:\Users\jayma\Documents\wallpaper.jpg
Update-WalTheme -Image C:\Users\jayma\Documents\wallpaper.jpg

For clearing the cache, you can still use pywal wal -c since this is just wrapping around pywal.
pywal doesn't perform it's actions correctly on Windows, so the PowerShell is doing that after using pywal to generate the cache.


add doc comments to Vec

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6037  👈
#6035
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite


Fix problem creating nested directories with no parent

I was getting this error before my fixing:
- Running electron-builder 2.2.0
- Ouput directory ´/Users/maxcnunes/Development/sqlectron/sqlectron-gui/dist/installers/osx/´ does not exist
fs.js:747
  return binding.mkdir(pathModule._makeLong(path),
                 ^
Error: ENOENT, no such file or directory '/Users/maxcnunes/Development/sqlectron/sqlectron-gui/dist/installers/osx/'
    at Error (native)
    at Object.fs.mkdirSync (fs.js:747:18)
    at Object.Builder.build (/Users/maxcnunes/Development/sqlectron/sqlectron-gui/node_modules/electron-builder/index.js:45:10)
    at Object.<anonymous> (/Users/maxcnunes/Development/sqlectron/sqlectron-gui/node_modules/electron-builder/cli.js:36:9)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)

Thanks. LGTM. :+1:


fix: wrong variable using in NodeVisitor

Since $class is a ClassReflection object, the if statement would never
be true. According to the context, change $class to $node

Thank you @feiyuanqiu.


Add optional parameter for name

I constantly find myself trying to remember the correct syntax for sending a mail to a raw email/name combination. Digging in the framework shows you may pass an array with a key/value pair. However, you also have to remember to pass this as a nested array.
This PR adds an optional second parameter for name, to allow conveniently sending mail to a email/name.
Before:
Mail::to([['email' =><EMAIL_ADDRESS>'name' => 'JMac']])->send($mailable);

After:
<EMAIL_ADDRESS>'JMac')->send($mailable);

Note, this parameter is only used when set and the first parameter is a string.

What about cc and bcc methods? Do the PendingMail methods allow this syntax?

Yes, the underlying methods in Mailable all have a signature of $address, $name = null. I am glad to make all of these methods (to, cc, and bbc)symmetrical on the Mailer if you like.

Done!


Twitter login attempts fail

3.0.2 beta on Mac. The request seems to fail when inspecting, see below:

It seems that upon re-launching the app I'm logged in if not using login verification. If the account uses login verification, I get texted the verification code, but I can never get redirected to the page that asks you to input it.

@swrobel I think it's also related to #39

This could be a possible solution: https://github.com/electron/electron/issues/3471#issuecomment-236368409

@swrobel It works 👍


can't install this with laravel 5.5

Problem 1
    - Conclusion: don't install barryvdh/laravel-ide-helper v2.3.2
    - Conclusion: don't install barryvdh/laravel-ide-helper v2.3.1
    - Conclusion: don't install barryvdh/laravel-ide-helper v2.3.0
    - Conclusion: remove laravel/framework 5.5.x-dev
    - Installation request for barryvdh/laravel-ide-helper ^2.3 -> satisfiable by barryvdh/laravel-ide-helper[2.3.x-dev, v2.3.0, v2.3.1, v2.3.2].
    - Conclusion: don't install laravel/framework 5.5.x-dev
    - barryvdh/laravel-ide-helper 2.3.x-dev requires illuminate/support ^5.0,<5.5 -> satisfiable by laravel/framework[5.4.x-dev], illuminate/support[5.0.x-dev, 5.1.x-dev, 5.2.x-dev, 5.3.x-dev, 5.4.x-dev, v5.0.0, v5.0.22, v5.0.25, v5.0.26, v5.0.28, v5.0.33, v5.0.4, v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.25, v5.1.28, v5.1.30, v5.1.31, v5.1.41, v5.1.6, v5.1.8, v5.2.0, v5.2.19, v5.2.21, v5.2.24, v5.2.25, v5.2.26, v5.2.27, v5.2.28, v5.2.31, v5.2.32, v5.2.37, v5.2.43, v5.2.45, v5.2.6, v5.2.7, v5.3.0, v5.3.16, v5.3.23, v5.3.4, v5.4.0, v5.4.13, v5.4.17, v5.4.19, v5.4.9].
    - Can only install one of: laravel/framework[5.4.x-dev, 5.5.x-dev].
    - don't install illuminate/support 5.1.x-dev|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support 5.2.x-dev|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support 5.3.x-dev|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support 5.4.x-dev|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.1|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.13|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.16|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.2|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.20|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.22|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.25|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.28|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.30|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.31|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.41|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.6|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.1.8|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.0|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.19|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.21|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.24|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.25|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.26|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.27|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.28|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.31|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.32|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.37|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.43|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.45|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.6|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.2.7|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.3.0|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.3.16|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.3.23|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.3.4|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.4.0|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.4.13|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.4.17|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.4.19|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.4.9|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support 5.0.x-dev|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.0.0|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.0.22|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.0.25|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.0.26|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.0.28|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.0.33|don't install laravel/framework 5.5.x-dev
    - don't install illuminate/support v5.0.4|don't install laravel/framework 5.5.x-dev
    - Installation request for laravel/framework 5.5.* -> satisfiable by laravel/framework[5.5.x-dev].

Can you submit a pr to allow 5.5?

should now work as of #525 @kasperpeulen

how to resovle it this  problem

I did a 5.5 installation today and it worked, see also https://github.com/barryvdh/laravel-ide-helper/issues/539#issuecomment-649519846
@barryvdh issue can be closed :}


Full tuple unpacking

The following example gives the type ((String, PExpr), PExpr), we would like instead (String, PExpr, PExpr).
let_expr = let_kw let_binding in_kw expression
let_binding = identifier bind_op expression

I think it's better to leave this unchanged. I'd prefer to use an explicit tuple unpacking operator, if anything at all.

Really? I believe that your AST will seldom use a type of the form (A, (B,C)) but either (A, B, C) or a custom type. The use case was as follow:
factor = let_expr > let_in_expr
let_expr = let_kw let_binding in_kw expression
let_binding = identifier bind_op expression

fn let_in_expr(let_binding: (String, PExpr), expr: PExpr) -> PExpr {
  Box::new(LetIn(let_binding.0, let_binding.1, expr))
}

In this example, I feel it is more clear and convenient to unpack so the function becomes:
fn let_in_expr(identifier: String, expr: PExpr, body: PExpr) -> PExpr {
  Box::new(LetIn(identifier, expr, body))
}

Can you provide a use case for which tuple unpacking is not convenient?

Ah, thanks for clarification. I'm thinking about situations where the tuple is directly included in a sequence or in the parse result, rather than consumed by a function. Then, the current behavior follows WYSIWG in the simplest way. I'm writing a parser generator that takes a slightly different approach to typed parsing expressions.

Keep in mind that the following works: (even though it's unreadable, admittely)
fn let_in_expr((identifier, expr): (String, PExpr), body: PExpr) -> PExpr {
  Box::new(LetIn(identifier, expr, body))
}

Thanks for your feedback. Actually, I didn't know pattern matching directly work in function's parameters. I'm curious about your approach to write a parser generator, do you want to discuss more about it?

I can explain my approach to making a parser generator. However, I would rather write an exhaustive blog post about it. (I don't have a blog yet.) The most interesting stuff is still at a concept stage, anyway.
I think there are two important areas to creating a parser generator: a) the set and character of fundamental features it provides, and b) the ways users are allowed to describe grammars.


Port to Swift 3.0

Done!

I installed HDF5Kit (0.2.0) via CocoaPods to work on Deep Learning on iPhone. When I opened xcworkspace file, it said "Convert to Current Swift Syntax?". After the conversion, there were a lot of errors. Are you sure this works for Swift 3.0?

I just tried master with Xcode 8.1 and everything works fine. Can you post one of the errors you get? You can also try pointing the Podfile to master.

Tried again with pointing the Podfile to master. Just got a typical error "Convert to Current Swift Sytax?"

The target “HDF5Kit” contains source code developed with an earlier version of Swift.
Choose “Convert” to update the source code in this target to the latest SDKs. You will be given the choice to use Swift 2.3 syntax or update to Swift 3.
This action can be performed later using “Convert to Current Swift Syntax” in the Edit menu.

My Podfile is like this.
# Uncomment this line to define a global platform for your project
# platform :ios, '9.0'

target 'HDF5Kit-test' do
  # Comment this line if you're not using Swift and don't want to use dynamic frameworks
  use_frameworks!

  # Pods for HDF5Kit-test
  pod 'HDF5Kit', :git => 'https://github.com/aleph7/HDF5Kit.git'

end

See how it's done in https://github.com/venturemedia/intune-features Swift3 branch. Works there.

Sorry, but I'm still not sure. Which branch and which file are you talking about?

This finally works for me.
# Uncomment this line to define a global platform for your project
platform :ios, '10.0'
source 'https://github.com/CocoaPods/Specs.git'
use_frameworks!

target ‘hdf5’ do
  # Comment this line if you're not using Swift and don't want to use dynamic frameworks

  # Pods for HDF5Kit
  pod ‘HDF5Kit’

end

post_install do |installer|
    installer.pods_project.targets.each do |target|
        target.build_configurations.each do |config|
            config.build_settings['SWIFT_VERSION'] = '3.0'
        end
    end
end

If you update to CocoaPods 1.1.0 Swift 3 is the default.


[Enhancement] Pick how many games to show in the "Recents Achievements" view

As far as I can tell, the Recents Achievements view shows your recent achievements from your last 3 games
This causes quite a lot of wasted space when you use it as a column:
https://i.imgur.com/5Wpt8xu.jpg
It would be nice to be able to have an option menu on the Recents Achievements view to pick:

how many games to show
how many achievements to show max per game

sorry just found the option under general settings m(_  _)m


Unable to "Drop" my public PGP Key (revoke)

Attempted to revoke from the laptop (log id: f7a81c36522d731196bb151c) and from the phone.  Both attempts resulted in a DB Error.

also, unable to drop the key from the windows app

I read something about this. Try to revoke the key from the keybase web site first. Then from the terminal on your computer.

Thanks for that.  That was the workflow I attempted first.  The site
redirects me to revoke the key from a listed-device in my sigchain.  The
windows app, the command prompt and my phone all point to a DB Error.
On Wed, Jul 5, 2017 at 1:19 PM, Louis Dupont<EMAIL_ADDRESS>wrote:

I read something about this. Try to revoke the key from the keybase web
site first. Then from the terminal on your computer.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/7626#issuecomment-312987078,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ9TfK3xjzz8tZmtbo-D0SgqpRfNC7y3ks5sKwCtgaJpZM4ONKMc
.

This bug should be fixed now (the DB error that prevented your revoke from
happening).  Sorry for the bug and the inconvenience
On Tue, Jul 4, 2017 at 11:25 PM, JTS<EMAIL_ADDRESS>wrote:

Thanks for that. That was the workflow I attempted first. The site
redirects me to revoke the key from a listed-device in my sigchain. The
windows app, the command prompt and my phone all point to a DB Error.
On Wed, Jul 5, 2017 at 1:19 PM, Louis Dupont<EMAIL_ADDRESS>wrote:

I read something about this. Try to revoke the key from the keybase web
site first. Then from the terminal on your computer.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/7626#issuecomment-312987078,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AJ9TfK3xjzz8tZmtbo-
D0SgqpRfNC7y3ks5sKwCtgaJpZM4ONKMc>

.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/7626#issuecomment-312987612,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA05_zzxV0qhl8d_8uLPfyHS9UsfyD6Lks5sKwIMgaJpZM4ONKMc
.

that fixed it.... now a new ticket for a new issue....


Issue with smart contract payableAmount/amount during to/from bytes

Description
Encountering some strange behaviour regarding the amount/payableAmount of a ContractExecute.
Steps to reproduce

Follow the "how to deploy smart contracts" tutorial
Add a payable amount to the code in the tutorial:

Do toBytes() on it, and then do fromBytes()
you can see the difference here:

Additional context
No response
Hedera network
testnet
Version
2.7.1
Operating system
No response

I believe this is now fixed: https://github.com/hashgraph/hedera-sdk-js/commit/4316bf9593510b88f06037e34cb0d27fd1deda0d


Images not loading in web view

When accessing the documentation via web: https://forklift-docs.konveyor.io/documentation/doc-Migration_Toolkit_for_Virtualization/master/index.html
... Images do not load. For example this diagram:

Which is pointing to this URL:
https://forklift-docs.konveyor.io/documentation/doc-Migration_Toolkit_for_Virtualization/master/modules/images/136_Upstream_Migration_Toolkit_0121_mtv-workflow.svg
... that returns a 404

Thanks. The images directory in the config_yml file was no longer working for upstream builds. I fixed the path and now the images are loading.


Add error on undefined action type

First, thanks for a great development tool!
I like the helpful errors in Redux, e.g. for undefined action.type. Of course you should have unit test catching this, but it would be nice if redux-devtools also had this message in case the action type is undefined.
I ported the behaviour from here:
https://github.com/rackt/redux/blob/5b94a18f560eff60ce37ae01d1b16387bcfb8e3a/src/createStore.js#L110
Let me know what you think. : )

Wouldn't Redux throw this anyway? Is logging it just before throwing beneficial?

No, this is not caught by Redux as all the actions are wrapped.

Oh, this is a great catch. I think it should be a hard error, then.

Yes, it is somewhat hidden that devtools wraps the action and hides these errors. : )
Ok, I will make it a hard error.

@gaearon please have a look now.


Allow modification of initial prompt

Would be great to separate the device info from the rest of the initial prompt to allow for some modification, e.g., role and personality. I am thinking of everything in italics below. I suggest the custom message should be specified in the config file.

You are Command Line App ShellGPT, a programming and system administration assistant.\nYou are managing Linux/Ubuntu 22.04.2 LTS operating system with bash shell.\nProvide only plain text without Markdown formatting.\nDo not show any warnings or information regarding your capabilities.\nIf you need to store any data, assume it will be stored in the chat.

The initial prompt unintentionally limits GPT's capabilities. For example, it should be able to complete this task successfully:
curl -s https://www.gutenberg.org/cache/epub/feeds/today.rss | sgpt --no-cache 'convert to json'

but it responds with

Sorry, I am not capable of converting XML to JSON. However, there are many online tools available that can perform this conversion

If I manually remove the default initial prompt (by editing the make-prompt.py code in site-packages), It converts the RSS XML to JSON without any issue.

@brianpeiris, I had a similar issue, I asked it to translate something and said something to the effect of "sorry, I can't do that, I'm a command line tool". I tried to rephrase slightly and it ran fine after that.
I just tried running your example prompt half a dozen times and got the same result you did every time, but when I tweaked the prompt a tiny bit
curl -s https://www.gutenberg.org/cache/epub/feeds/today.rss | sgpt --no-cache 'convert to the following data json'

I got the desired result
curl -s https://www.gutenberg.org/cache/epub/feeds/today.rss | sgpt --no-cache 'convert to the following data json'
{
  "rss": {
    "version": "0.91",
    "channel": {
      "title": "Project Gutenberg Recently Posted or Updated EBooks",
      "link": "https://www.gutenberg.org",
      "description": "EBooks posted or updated today on Project Gutenberg.\n       This feed is regenerated every night.",
      "language": "en-us",
      "webMaster"<EMAIL_ADDRESS>      "pubDate": "Tue, 11 Apr 2023 02:21:45 -0400",
      "lastBuildDate": "Tue, 11 Apr 2023 02:21:45 -0400",
      "item": [
        {
          "title": "The Complete Works of William Shakespeare by William Shakespeare",
          "link": "https://www.gutenberg.org/ebooks/100",
          "description": "Language: English"
        },
        {
          "title": "The Taming of the Shrew by William Shakespeare",
          "link": "https://www.gutenberg.org/ebooks/1508",
          "description": "Language: English"
        },
        {
          "title": "Middlemarch by George Eliot",
          "link": "https://www.gutenberg.org/ebooks/145",
          "description": "Language: English"
        },
        {
          "title": "Unspecialist by Murray F. Yaco",
          "link": "https://www.gutenberg.org/ebooks/23443",
          "description": "Language: English"
        },
        {
          "title": "De muis, of De gestoorde nachtrust by [pseud.] Braga Jr. and Wilhelm Busch",
          "link": "https://www.gutenberg.org/ebooks/17637",
          "description": "Language: Dutch"
        },
        {
          "title": "For the freedom of the seas by Ralph Henry Barbour",
          "link": "https://www.gutenberg.org/ebooks/70522",
          "description": "Language: English"
        },
        {
          "title": "A Pair of Blue Eyes by Thomas Hardy",
          "link": "https://www.gutenberg.org/ebooks/224",
          "description": "Language: English"
        },
        {
          "title": "Thus Spake Zarathustra: A Book for All and None by Friedrich Wilhelm Nietzsche",
          "link": "https://www.gutenberg.org/ebooks/1998",
          "description": "Language: English"
        },
        {
          "title": "Heldensagen en Legenden van de Serviërs by Vuk Stefanović Karadžić and Woislav M. Petrovitch",
          "link": "https://www.gutenberg.org/ebooks/18363",
          "description": "Language: Dutch"
        },
        {
          "title": "Les trois mousquetaires by Alexandre Dumas",
          "link": "https://www.gutenberg.org/ebooks/13951",
          "description": "Language: French"
        },
        {
          "title": "Remarks on the speeches of William Paul Clerk, and John Hall of Otterburn, Esq : by Anonymous",
          "link": "https://www.gutenberg.org/ebooks/70523",
          "description": "Language: English"
        },
        {
          "title": "A csókok éjszakája és egyéb elbeszélések by Ferenc Molnár",
          "link": "https://www.gutenberg.org/ebooks/70519",
          "description": "Language: Hungarian"
        },
        {
          "title": "The glue book : by J. A. Taggart",
          "link": "https://www.gutenberg.org/ebooks/70517",
          "description": "Language: English"
        },
        {
          "title": "Nippon-ország naposkertje : by Ignác Kúnos",
          "link": "https://www.gutenberg.org/ebooks/70518",
          "description": "Language: Hungarian"
        },
        {
          "title": "A true & exact history of the island of Barbados by Richard Ligon",
          "link": "https://www.gutenberg.org/ebooks/70489",
          "description": "Language: "
        },
        {
          "title": "The hermit's Christmas by David de Forest Burrell",
          "link": "https://www.gutenberg.org/ebooks/70521",
          "description": "Language: English"
        },
        {
          "title": "Naomi : by Mrs. J. B. Webb",
          "link": "https://www.gutenberg.org/ebooks/70520",
          "description": "Language: English"
        }
      ]
    }
  }
}

GPT3 can take some persuading, as I'm sure you've probably found through using ChatGPT directly.
Definitely would be nice to be able to modify the initial system message though in the future :)

Ironically, it fails if you use proper english
curl -s https://www.gutenberg.org/cache/epub/feeds/today.rss | sgpt --no-cache 'convert the following data to json'

but works if you add "now" at the end (!)
curl -s https://www.gutenberg.org/cache/epub/feeds/today.rss | sgpt --no-cache 'convert the following data to json now'

I would still just like a flag to disable the initial prompt, since it apparently confuses GPT easily for these basic tasks.

If the maintainers would accept this as a solution, I could look into adding a flag to disable the initial prompt.

Custom roles will be released soon https://github.com/TheR1D/shell_gpt/pull/183


open.vscode.dev no longer avaliable

Does this issue occur when all extensions are disabled?: N/A

VS Code Version: N/A
OS Version: N/A

Steps to Reproduce:

Visit a relevant URL: https://open.vscode.dev/meshtastic/Meshtastic-device
Should not redirect you to vscode.dev.

@joaomoreno are there any alternate services that offer the same functionality, this was great for getting users up and running


Removed cluster/saltbase/salt/_states/container_bridge.py since it is…

… useless now.
@thockin @zmerlynn

LGTM

@thockin @roberthbailey for rubber stamps

LGTM
On Tue, Jun 30, 2015 at 11:24 AM, Zach Loafman<EMAIL_ADDRESS>wrote:

@thockin https://github.com/thockin @roberthbailey
https://github.com/roberthbailey for rubber stamps
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/10558#issuecomment-117290774
.

LGTM.

Kicking Shippable, though I can't imagine it mattering...

Actually, I like to live "dangerously". The pull builder passed, there's no way Shippable matters on this PR. :P


DSWx-S1 Beta SAS Acceptance Test

Ticket to track acceptance testing for the DSWx-S1 SAS Beta delivery.
The container tarball for the delivery can be found in Artifactory-FN under the path general/gov/nasa/jpl/opera/adt/dswx_s1/r2.1/beta/dockerimg_dswx_s1_beta_0.2.1.tar, test inputs and expected outputs can be found under general/gov/nasa/jpl/opera/adt/dswx_s1/r2.1/beta/sample_data.zip. Documentation for the delivery is under general/gov/nasa/jpl/opera/adt/dswx_s1/r2.1/beta/documents/
A new wiki page needs to be created to capture the Acceptance Testing procedure for the Beta delivery. The procedure for the previous Interface delivery may be used as a starting point. However, unlike the previous Interface delivery, a script to compare the results of the Acceptance Test with the expected output products (within sample_data.zip) is now available, and its use needs to be incorporated into the testing procedure.
A copy of the DSWx-S1 comparison script can be obtained here: https://raw.githubusercontent.com/opera-adt/DSWX-SAR/main/src/dswx_sar/dswx_comparison.py
It is up to the developer to decide whether the dswx_comparison.py script should be executed from within the container (mounting the Acceptance Test outputs and expected outputs into the container), or from outside the container as a standalone Python process.

Closed by #387