Deploy ArgoCD to staging

https://phabricator.wikimedia.org/T360874
This PR just adjusts the environment condition, which now evals to true for local and staging but not production.

This is a bit twisted, but if I understand the ticket (as well as @tarrow) correctly, we want to deploy Argo to the production cluster only, while skipping staging. The idea behind it is that having a single argo deployment for both environments will allow us to use autopromotion at some point.

This is a bit twisted, but if I understand the ticket (as well as @tarrow) correctly, we want to deploy Argo to the production cluster only, while skipping staging. The idea behind it is that having a single argo deployment for both environments will allow us to use autopromotion at some point.

I see, thanks for pointing that out. Seems like I missed a lot of discussion around this topic

Seems like I missed a lot of discussion around this topic

I only just understood this myself yesterday.

Shouldn't we still first deploy this thing to staging to figure out how to configure & use it?


Update help text.

By default, in practice and looking at the code, it seems to only be listening in localhost so we should update the help text so we don't confuse users.

🤔 I don't think that helps in anyway, it majorly disproves whats happening under.

What do you mean? Isn't serving in localhost:5000 the default behaviour?

Serving on localhost:5000 is different from <IP_ADDRESS>:5000.
localhost listens only on the loopback interface(logical), <IP_ADDRESS> listens on all network interfaces(IPv4 - physical and logical). It's through what interface the server can be contacted from the outside world(or from within the host).
Node listens on <IP_ADDRESS> or :: by default.

Closing. The above comment is correct.


Doesn't work, I got blank page

Hi,
I tired to replace react-error-guard with default react ErrorBoundary, but it doesn't show anything when error happens. (ErrorBoundary works as expected btw)
I also got this in browser console:

React will try to recreate this component tree from scratch using the error boundary you provided, ErrorBoundaryComponent.

I  use:

React v16.5.0
Webpack v4.18.0

Would you help me on this?

Sure, can you provide some code snippets?

Well, without no code snippet I can give you a checklist of things to look for:

the error boundary component has to be within the subtree of the component who crashed (in case you don't wrap the whole app in just one.
if the error thrown is caught it won't show the error component
if you cause a re-render/refresh somehow after the component crashed it could also prevent the error message to be rendered.

Also, does this happens both with the Production and Developer error boundary components?

I will re-check your checklist. thanks for that.

Also, does this happens both with the Production and Developer error boundary components?
I didn't test, I'll and report the result.

I have very simple react starter kit and by "simple hello world react app" I mean this starter kit.
Actually I used this for my main project too and that's why I have no idea why your plugin doesn't work in it.

I found out what was wrong, it was my configuration not your plugin. sorry about that.


Minimal Ktlint actions for IdeaVim

I use IdeaVim and tend to hook up formatting actions to keystrokes. Running :actionlist using IdeaVim shows me that there aren't any actions available to bind. Would it be possible to present some to users so that we can do stuff like the following?
" binds ctrl+shift+i to the action `KtlistFormatFile`
nnoremap <C-S-i> :action KtlintFormatFile<CR>

Great idea for a plugin, thank you! Gonna try to incorporate it into my workflow 😄

Great idea! I'll take a look at how IdeaVim searches for actions 😁

So until this point, all the "actions" available were intention actions, which are separate from standard actions. I've added a format action under the Reformat menu. It now shows up under :actionlist, and I think it should allow keybinding in general too :)

Woot! Thank you! 🙌


Pump dependencies

There should be a way for greenkeeper to pump these peerDep and optionalDep for us.

Current coverage is 67.57%

Merging #177 into next will not affect coverage as of 7ebdd9f

@@             next    #177   diff @@
=====================================
  Files          37      37
  Stmts         475     475
  Branches        0       0
  Methods         0       0
=====================================
  Hit           321     321
  Partial         0       0
  Missed        154     154

Review entire Coverage Diff as of 7ebdd9f

Powered by Codecov. Updated on successful CI builds.

I think it should be "Bump dependencies" :)
Thanks!


BGP state doesn't resync correctly when adding a BGP session via node labels

Is this a bug report or a feature request?:
Bug, thanks to @jpiper for reporting on slack.
What happened:
If you cause a BGP session to be added to a node by editing node labels, the session comes up, but the existing advertised IPs don't get propagated on that BGP session.
How to reproduce it (as minimally and precisely as possible):
Add a BGP peer to MetalLB's config, with a node selector.
Make the selector match some nodes, so that BGP sessions establish.
Set up a service, verify that speaker is advertising it.
Edit node labels so selector no longer matches. Verify speaker tore down the BGP session.
Edit node labels so selector applies again. Verify speaker sets up BGP session, but does not readvertise IPs.
Environment:

MetalLB version: 0.4.1

Fixed by 025ada85c.


Increase Aruba exit_timeout value for TruffleRuby

To allow enough time for subprocesses to complete when using TruffleRuby.

Thanks!


when creating the ghost element for drag effect, copy the canvas content as well

Fixes issues #330.

+1
@clauderic any idea if and when this is going to be merged?

@pritam1994 thanks for approving this. who is responsible for merging?

@clauderic can you please merge this?

+1

@clauderic any chance of merging this soon?

Integrated this fix in this commit https://github.com/clauderic/react-sortable-hoc/commit/43ad122bbed56b9ece79e48ff2d08d30a6503554, which shipped with<EMAIL_ADDRESS>


P1 Fix Silent bad context results of Cody Enterprize customers

Version
1.42(But this is present in other versions too)
Areas for Improvement

[X] UI/UX
[x] Onboarding
[ ] Docs
[x] Chat
[ ] Commands
[x] Context
[ ] Response Quality
[ ] Other

What needs to be improved? Please describe how this affects the user experience.
There is a very subtle silent failure modes of Cody that I am quite positive a ton of customers have faced and they DON'T even know that they are facing them. This failure mode exists in both Vscode and Jetbrains extensions.
It goes like this:

User gets on a repo
User assumes that because the repo is indexed(because Admin said so)
They click the enhanced context and assume the jobs done. They get a very very bizarre answer that looks like it might be useful but for the most part is completely frivolous.

Classic example(See photo). In this I query message provider as it is an important class in the cody codebase but when the repo is not connected then the results are just bizarre claude hallucinations. The user still doesn't know that Cody didn't pick up any context. Why? Because other than the response of the LLM the user has literally no way to know if the context was even gotten to Cody at all or not? The user keeps using cody and probably it takes them a while to figure out this ain't working.
No context(The error case )

With context

Related Thread

Describe the solution you'd like to see
@toolmantim has already solved the design part of this problem, just needs to be implemented. This should be super easy(see comment)
Describe any alternatives that could be considered
No response
Additional context
No response

@toolmantim , WDYT about popping the enhanced context selector open if we can't detect ANY repositories?

What does it mean to not detect any repo?

I think he means this

Sorry for the lack of clarity. There's a two step process:

Given a workspace, find a git repo.
Given that git repo, ask the remote if it is indexed.

By "not detect any repo" I meant fail in step 1.

from EPD grooming:

add a better way to signal to enterprise users that they have no context available
add telemetry to understand how many users are affected

open questions:

what is the happy path for the user to get their local repo added to remote

Here's the "spec" of what the product does today for enterprise.
A Chat's Repo Set
When chatting, your set of repos is formed this way:

If the chat has an "automatic repo" (see below), include it in the repo set.
If you set repos with the repo picker, add those ones.
Otherwise, if the chat had repos saved with it, add those ones.
Otherwise, do "Workspace Repo Detection" (see below.)

Note, when we save a chat, its repos are saved with it.
Background: Repo Identification
To get enterprise search results, you need a set of repo IDs. This is a fundamental operation in repo set building so I'll explain it in detail here and then refer to it briefly after that. The software follows a chain like:
File → git repo → git remote → repo "name" → repo ID
File → git repo: If the file is in the workspace, walk up the directory tree to find a git repo. We do walk out of the workspace root. For example, say Bob adds /Users/bob/sourcegraph/client to the workspace and the workspace root is "client" and we scrutinize file /Users/bob/sourcegraph/client/foo.ts then we will detect information from git in /Users/bob/sourcegraph.
git repo → git remote: Roughly, git remote -vv and find the first (push) URL.
git remote → repo "name": A Sourcegraph name mangling heuristic. For example, "git@github.com:sourcegraph/sourcegraph" is translated to "github.com/sourcegraph/sourcegraph".
repo "name" → repo ID: A GraphQL query that looks in a database on the Sourcegraph instance.
Any one of these steps can fail in the obvious ways you'd imagine. For example, file is in Perforce not git. Repo is not indexed on the Sourcegraph instance. etc.
Workspace Repo Detection
In the background, establish a "default repo set" this way:

Continuously watch the list of folders in the workspace.
When the list of folders change, for each root folder (can be more than one; we only consider the roots and not sub-folders), do the repo identification procedure outlined earlier.
If the folder has an ID, it goes in the default repo set.

When any chats are open, continuously attend to the "automatic repo name" this way:
When an editor gets focus, if there are any folders in the workspace, then:

If the cody.codebase setting is set, use that repo name.
Otherwise, if the file is in a workspace folder, do the repo identification procedure.

When a chat session is started, set the chat session's repo set to the repos in the current default repo set.
When a chat session is restored, and that chat session does not have a set of repos associated with it, set the chat's repo set to the repos in the current default repo set.
Likely Causes of Empty Repo Sets

VSCode workspace has no roots.
Workspace roots aren't in git repositories.
git repositories don't have remotes.
git repositories have remotes, but the first remote's push URL doesn't fit the Sourcegraph heuristic.
Repository is not indexed on the Sourcegraph instance.

Likely Causes of "Wrong" Repo Sets

Open file is not in the workspace.
Open file is in the workspace, but that part of the workspace ... problems outlined above: not in a git repository, repository doesn't have remotes, first remote has a push URL name our heuristic doesn't match, repository is not indexed.

Likely Bugs

Repo Identification depends on the network, so YMMV under variable network conditions.

My $0.02, why don't we...

Implement @toolmantim 's UX improvement in #issuecomment-1949261885
Put a red badge on the enhanced context selector if we have 0 repos to draw some attention to the problem.
Put a diagnostic about which phase of the process broke down... somewhere. Tooltip?
Add logging to work out the prevalence of the issue.

Product team could look into...

What's the current practice for customers with other SCMs (p4, etc.) Is it just a name mangle to pick these up?


Memory Leak

I have noticed an issue with the App lately that if I leave it running for extended periods, even with my MacBook Pro in sleep mode, the Messenger app will end up consuming huge amounts of RAM. At one point after I had left my computer in sleep mode for a few days it was consuming 11.5GB and brought my machine to its knees.

I have been using messenger.com in a safari tab for the last couple of weeks to try and see if it was not specific to this application, but that tab is fine even after all this time and I haven't had to close and reopen it.

I've got a similar problem, it's been getting up in the multiple GB range every few days.

+1

+1

This is unfortunately not an issue with this app (I've spent time with instruments trying to find leaks and it all points to webkit) We could work around the issues though by force-restarting the web view once in a while, but that's a hack and would lead to a reduction in user-experience quality.
If anyone has time, please do attempt to instrument the app and investigate where memory is allocated (but never free'd.)
I'm not going to do anything about this until we know what's actually causing the issue.

Just a thought, you could force restart the web view when refreshing the page. Refreshing the page did not make the memory usage go down at all - and actually went up
I encountered a similar problem in Safari in another project. https://github.com/weirongxu/plantuml-previewer.vim/issues/6
The problem that caused a memory leak was that a single image file was being repeatedly loaded, and Safari kept each copy in memory, which led to lots of memory usage. (Interestingly, chrome did not have a memory leak.) Safari wouldn't even garbage collect the old images when refreshing the page.

@rsms Running the XCode analyzer gave the following warnings:

If you've seen this and the memory leak is not related to any of this I would like to know what can I do, specifically, to help resolve this issue.
I use the app on a daily basis and it just slows down my computer a lot 😞

fwiw I encounter pretty bad memory leaks with messenger.com running in a Chrome tab, I have to hard-reload to release the 2-3 GB it hoards after a couple days.

You can run a launchd script to monitor memory usage and restart app. For example: https://github.com/longngn/fb-mac-messenger-monitor


Roll Flutter Engine from b2855e074deb to 6b53e4672ed9 (2 revisions)

https://github.com/flutter/engine/compare/b2855e074deb...6b53e4672ed9
2023-03-31<EMAIL_ADDRESS>Add ui_web to embedder.yaml so that the analyzer knows about it. (flutter/engine#40827)
2023-03-31<EMAIL_ADDRESS>Roll Skia from 2b86c6d364d0 to 33f80c07a09c (1 revision) (flutter/engine#40826)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


How do I force Youtube-DL to download the best quality MP4-DASH video?

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

How do I make Youtube-DL actually pick the highest quality MP4-DASH video/audio streams? I know I can do default which selects what it thinks are the best quality DASH streams regardless of format (it will download webm audio/video DASH streams if it thinks those are best). I can force it to download MP4 video by using "-f mp4" but this only works for forcing it to download the best quality single-file MP4 video. Unfortunately on Youtube this is limited to 720p. If 1080p (or better) is available it will ONLY be available via DASH streams. I need to know how to force Youtube-DL to download the best quality MP4-DASH streams (AAC for audio, and h264 for video). And one other thing, I want it to ONLY use h264 codec for video, and ignore any streams using the AV1 codec, as not all video player software supports this codec yet. Is there a way to force it to use best quality MP4-h264-DASH for the video stream and best quality AAC-DASH for the audio stream?

Read the manual section Format Selection.
However, run yt-dl with -F to list the available formats; then either select a listed format or pair of formats (eg -f 137+140) or use some selection criteria; how you formulate the criteria will depend on whether you have a lower limit for the resolution you'll accept. Eg:
-f 'bestvideo[ext=mp4][vcodec!^=av0][vcodec!^=av1]+bestaudio[ext=m4a]/mp4/best' --recode-video mp4

Also, almost a duplicate of #29794, and see also #29783.

hi

i make Telegram Bot in Python @FasterSaveeBot


Let user choose it's own Full Node

Hi guys, I was wondering if in the next version you'll let to choose your own full node working in the same network?
At this moment I had to fork your project and make own build pointing at my already functioning full node.
Are you planning to do such thing in near future?
Thanks for info!

Ok i think it has been poorly documented that you can run Stratis.Core with a parameter --daemonip anyways it just waits for the node to initialize ? Even though my node is fully synchronized?
Atm:
======Node stats====== 07/09/2019 17:48:30
Headers.Height:      1379238  Headers.Hash:     6780b33b6fa2b875e06f596396ff0d6b9f2e2a8495816d9f149b8fd580993f91
Consensus.Height:    1379238  Consensus.Hash:   6780b33b6fa2b875e06f596396ff0d6b9f2e2a8495816d9f149b8fd580993f91
BlockStore.Height:   1379238  BlockStore.Hash:  6780b33b6fa2b875e06f596396ff0d6b9f2e2a8495816d9f149b8fd580993f91
Wallet.Height:       1379238  Wallet.Hash:      6780b33b6fa2b875e06f596396ff0d6b9f2e2a8495816d9f149b8fd580993f91

Here's the screenshot when running with  --daemonip=192.xxx.xxx.xxx  entered (ip masked for purpose)

Thanks

Ammm, okay guys we cool I think it's because I'm running <IP_ADDRESS> instead of <IP_ADDRESS> node. Gonna update when it builds.

Ok guys, everything is fine - that's the exact functionality i was looking for.
Please add proper documentation for commandline arguments and we are good.
Thanks!

Hey @pablos91, good that you've solved your own issue :). We'll work on adding the appropriate documentation.


add public directory

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

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

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added necessary documentation (if appropriate)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

I think the public directory has to be inside templates/web folder and not the project root.
You can also test if everything is working fine by running following command.
npm run build
node build/bin/run.js sample

My bad 🤦♂ Thanks

Closing, since already added (discussed on discord)


KEP update: Allow replacement pods in groups of pods

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Which issue(s) this PR fixes:

Part of #976
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

cc @achernevskii

/hold
I'm rethinking whether all Pods owning the Workload is the best idea.

@tenzen-y @ahaysx any additional feedback?

/assign

lgtm, thanks Aldo

/hold cancel

@tenzen-y anything to add before merging?


3D renders: Cairo has weird rendering

For some reason there is a tiny difference to the images, although the helm is empty (at these parts):
# `compare` comes with ImageMagick
compare \
  "477be35554684c28bdeee4cf11c591d3c88afb77e0b98da893fd7bc318c65184-6-body.png" \
  "477be35554684c28bdeee4cf11c591d3c88afb77e0b98da893fd7bc318c65184-6-bodyhelm.png" \
  "diff.png"

The difference is highlighted in red:

The two lines on the chin and cheek don't seem to resize on larger images though.

There is more discussion in #26. At the moment it seems to be a bug with Cairo.

IIRC we cut out, rotate, and transform images with a single canvas function call.
I wonder if it makes a difference if we split that into a few functions to cut out the different parts of the skin (as we do with avatars) and then pass them to Cairo for rotation etc.
Maybe that fixes those weird renders. maybe. these glitches are really annoying

Just ignore this for now, I want to redo the renders in the future and make them look better, really simple atm.

Maybe we could also try to reimplement this using WebGL (if we find that for node).
Just throwing in my ideas here...

Was planning on doing that, considering how bad the renders look with all the issues :stuck_out_tongue:


Official docker builds

Can we get official docker builds? There are many out of date/unofficial docker builds on docker hub that do not have the fix that recently went in. An official build would be very helpful.

#761 tracks publishing official docker images. I had another go at it recently but it really needs one of the core Zalando folks to step up. It's a shame there's no jitpack.io for docker images!
Good to know that there are build on docker hub and people are trying to use them though.


Update change log for ODL 7.12.2

Docs Build status updates of commit f299262:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Odata-docs/changelog/odatalib-7x.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 095768e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Odata-docs/changelog/odatalib-7x.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Update index.html

Updated incorrect "numeric" datatype to "numerical".

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


fixbug/09-sast-semgrep add end path to nginx location alias

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.


Documentation on working with other layers

While attempting to test this repo I encountered a couple of hiccups. First, I wasn't seeing the node version updated when checking inside my Lambda runtime. After some testing I realized that my runtime in my Lambda template was still set to "nodejs14.x" which was overriding any changes the Lambda made. This setting would be useful to highlight in the readMe.
Second, I'm having issues getting other layers to work when using in conjunction with this repo. For my use case it seems that using a node shebang declaration in my other layer fails. This layer works when using AWS node runtimes so it seems that the layer has changed other parts of the environment outside of node version. This likely just requires some tweaking on my end but it would be helpful to include some documentation on how running an everynode layer differs from the standard AWS node runtime.
#!/usr/bin/env node

/usr/bin/env: node: No such file or directory

Appreciate the work here. Very interesting feature!

@cjlludwig When using the everynode Lambda layers, the standard Node.js runtimes from AWS are not used at all. You will notice that the instructions in the readme use the provided runtime as opposed to nodejs14.x:
aws lambda create-function --function-name hello17 \
  --layers $LAYER \
  --region us-west-1 \
  --zip-file fileb://function.zip \
  --handler function.handler \
  --runtime provided \
  --role {iam-role-arn}

The implication is that the Node.js executable normally provided by the AWS runtime is absent. Instead, the executable of the selected version of Node.js is placed in /opt/node/bin/node by the layer.
One thing you can try to make the composition with other layers work is to provide a PATH environment variable when creating your Lambda function that will include the /opt/node/bin directory -- I have not tested it though.

@jlarmstrongiv How are you launching those scripts from your code? The custom runtime bundled in the layer sets the process.env.PATH to include the location of the node executable (https://github.com/fusebit/everynode/blob/main/src/bootstrap#L1-L3). As long as you launch the script in a way that passes process.env as the environment, your shebang should work


[6.x] Add selectConcat() builder method and related DB grammar

This PR adds a new method to the query builder, selectConcat, which provides a way to add concatenated values as an alias in the query, to be used later in the query, or returned to the models. This happens external to the normal select method and defined columns.
This work was originally done for October CMS (https://github.com/octobercms/library/pull/434) however, we felt it would be a useful addition to Laravel and so have put it forward for consideration.
The method accepts an array parameter $parts which make up the parts of the concatenated string. If a value looks to be a column name, it is wrapped with column quotes. For all other values, it wrap them in string quotes. If a column name or operator is to be used, it can be double-quoted in order to force it to be considered as a string. The second parameter is a string parameter that defines the alias name of the concatenated value.
The concatenated alias is populated as an attribute in the model after the query is run, allowing models to use the values as well.
I'm happy to take advice on any improvements or changes required.

Related: https://github.com/laravel/framework/issues/5676

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

@taylorotwell this PR adds support for the selectConcat() abstraction to the QueryBuilder so that developers looking to use database level concatenation no longer have to lock into one DB syntax through the use of DB::raw("CONCAT(first_name, ' ', last_name)") if using MySQL or DB::raw("(first_name || ' ' || last_name)") if using any other SQL driver. This simplifies the development experience and makes code that is using DB level concatenation more portable between different database drivers.
What else would you like to see before merging this in? We could also add it to the documentation here: https://laravel.com/docs/6.x/queries#selects

@taylorotwell In addition to @LukeTowers' comments above, the other benefit this method provides is being able to use concatenations to manipulate other parts of the query. For example, the values could be used as groupings or for ordering. Whilst this could be done through Eloquent, that would require all the data to be retrieved first and then manipulated - this allows us to do it before it ever reaches PHP code.

@themsaid could you review this please?

@themsaid ping


Winlator dx9 regression at 2.5.x builds

System information

GPU: Adreno 735
Driver: Turnip 25.0
Wine version: 9.2
DXVK version: 2.5.1
Box64: c30a392 ( action build artifact )
App: Simple d3d9 test environment

DXVK 0.4 ~ 2.4.1

DXVK 2.5 ~ 2.5.1

What happened? :'( wheres my cute 4 digit fps.

Duplicate of #4484

Yeah someone worked out that this is almost certainly caused by turnip having to emulate timeline semaphores when running on top of the proprietary Qualcomm kernel driver. It's not really viable for us to revert back to the old path, so future versions of DXVK will remain slow.

Yeah someone worked out that this is almost certainly caused by turnip having to emulate timeline semaphores when running on top of the proprietary Qualcomm kernel driver. It's not really viable for us to revert back to the old path, so future versions of DXVK will remain slow on this platform.

what about an environment variable or an option in config file to disable that timeline semaphores?

what about an environment variable or an option in config file to disable that timeline semaphores?

Too much maintenance overhead to keep both paths around.


Data Updates for /post/[slug], But Not For /post

Describe the bug
If you use the same server load function for a root route and a slug route, the root route loses the ability to navigate to slug routes once navigated to.
Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-wbh3iw?file=src%2Froutes%2Fpost%2F[[slug]]%2F%2Bpage.server.js
If you click on "first post" and then "posts" in the header, it works fine.
If you click "posts" then "first post", you'll see how the content does not update when you click on "first post" this time.
The other routes work as expected.

It is noteworthy that the file structure in the example is:
/src
  /lib
    /routes
      /post
        /[[slug]]
          +page.server.js
          +page.svelte

But this bug persists for different variations that lead to the same routing structure. For example:
/src
  /lib
    /routes
      /post
        +layout.server.js
        +layout.svelte
        +page.svelte
        /[slug]
          +page.svelte

Etc.
Logs
There is no output to either the logs in the terminal on http://localhost:5173/ or in the browser's console.
System Info
System:
    OS: macOS 14.2
    CPU: (8) arm64 Apple M1
    Memory: 56.66 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.2.0 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.6.6 - /opt/homebrew/bin/npm
    pnpm: 7.8.0 - /opt/homebrew/bin/pnpm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 120.0.6099.129
    Edge: 120.0.2210.61
    Safari: 17.2
  npmPackages:
    @sveltejs/adapter-auto: ^3.0.0 => 3.1.0
    @sveltejs/kit: ^2.0.0 => 2.0.6
    @sveltejs/vite-plugin-svelte: ^3.0.0 => 3.0.1
    svelte: ^4.2.7 => 4.2.8
    vite: ^5.0.3 => 5.0.11

Severity
annoyance
Additional Information
Discord Post

Hi,
Yes invalidation is completely broken in SvelteKit, buckle up :
https://github.com/sveltejs/kit/issues/11635
https://github.com/sveltejs/kit/issues/11696
https://github.com/sveltejs/kit/issues/11663
https://github.com/sveltejs/kit/issues/11446
https://github.com/sveltejs/kit/issues/10209
https://github.com/sveltejs/kit/issues/10123
https://github.com/sveltejs/kit/issues/10457
https://github.com/sveltejs/kit/issues/10359
Unfortunately the team will read your issue and pretend it doesn't exist just like the other ones.
Meanwhile, features like generating random images of gooses receive a lot of care and attention.

Hi,
Yes invalidation is completely broken in SvelteKit, buckle up :
#11635 #11696 #11663 #11446 #10209 #10123 #10457 #10359
Unfortunately the team will read your issue and pretend it doesn't exist just like the other ones.
Meanwhile, features like generating random images of gooses receive a lot of care and attention.

Sorry this flew under the radar. I've just forwarded these issues to the team. Thank you for helping to compile a list of these issues.
Personally, I'll try to have a look at these this week when I can, usually after work and other commitments. Recently, I've been giving myself a bit of a break as I've been feeling burnt out from looking at code at work then looking at more code after work so I haven't been looking at any issues in depth.

Someone suggested on discord some workarounds for this particular error -- for example:
export const load = async ({ params: {slug} }) => {
  const post = posts.find((post) => post.slug === slug);
  return { post: post ?? posts[0] };
};

I tried it and it does seem to work! As the author points out, it "seems like kit just doesn't see the dependency on slug without explicit reference".
I will leave this ticket open though, as the original issue is still true and it seems to be symptomatic of deeper issues. Hopefully some of this will help with debugging.

@gterras , could invalidation also be causing this issue https://github.com/sveltejs/kit/issues/6315 ?

@gterras , could invalidation also be causing this issue #6315 ?

Not sure what you mean this looks like a purely conceptual issue?

I'm not able to reproduce this issue by following the instructions. The pages load fine in both orders of navigation. Can you re-confirm if you're still seeing this issue?

https://github.com/sveltejs/kit/pull/11870 just fixed a lot of the issues with invalidation, so can you test to see if you're still having issues with 2.5.1?


groups: Ensure student 'Delete Group' button has correct path

Motivation and Context

The current code for the student interface Delete Group button calls a path helper as follows<EMAIL_ADDRESS>@assignment.id). This omits an :id argument, which is automatically filled in as the @assignment.id. For example, DELETE courses/1/assignments/10/groups/10.
The GroupsController ignores the :id parameter so this previously wasn't an issue, but now that we're doing stricter URL checking and model associations, this :id is used to look up a grouping, which doesn't necessarily belong to the assignment, or even part of the same course, resulting in a 404 error.
Your Changes

Description: Fix the path helper.
Type of change (select all that apply):

[x] Bug fix (non-breaking change which fixes an issue)

Testing

Tested manually in the UI (the bug is in the view file).
Questions and Comments (if applicable)

Even though I fixed the :id parameter, it's still not being used in the controller method. We can change this if we move the method into a Groupings controller in the future.
Checklist

[x] I have performed a self-review of my own code.
[x] I have verified that the pre-commit.ci checks have passed.
[ ] I have verified that the CI tests have passed.
[ ] I have reviewed the test coverage changes reported on Coveralls.
[x] I have updated the Changelog.md file.

We really should have a different grouping controller but that's a different conversation

@mishaschwartz I see what you're saying and did consider this, but I disagree here. I prefer sticking with the Rails default, which is to treat destroy as a member route. My preference is to modify the controller method itself, but I didn't do so in this pull request to keep the change small.
With the remove_group method, it would also make sense to move this to a GroupingsController (which I mentioned in the PR description), and turn it into a member route. It already takes in a grouping_id parameter.


fix(catalog): ShrinkingCatalog alarm tracks major versions instead of packages

Our ShrinkingCatalog alarm currently tracks the total number of major versions in the catalog.
This is a bit confusing since its counter intuitive, also the description of this alarm in our runbook doesn't mention major versions, but speaks in package terms.
I think tracking packages is more appropriate because we have no control over how many major versions a package can have, which makes it difficult to define a threshold for.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Still need to deploy and test this


MultiSelectPropertyValue Typescript Bug

The MultiSelectPropertyValue type has a bug where it expects the multi_select to be a single SelectOption instead of an array of SelectOption[]
export interface MultiSelectPropertyValue extends PropertyValueBase {
    type: 'multi_select';
    multi_select: {
        id: string;
        name: string;
        color: Color;
    };
}

should fixed in v0.1.5 - let me know if you have any issues!


Static files browser caching

Just running a site in production through google page speed insights and getting expiration not specified errors for static files. Is there a way in bone to specify

Setting an expiry date or a maximum age in the HTTP headers for static resources instructs the browser to load previously downloaded resources from local disk rather than over the network.

Code in question is:
	r.Mux.Handle("/public/", http.StripPrefix("/public/", http.FileServer(http.Dir("./public/"))))
	r.Mux.Handle("/attachments/", http.StripPrefix("/attachments/", http.FileServer(http.Dir("./attachments/"))))

As I write this I am starting to think its not an issue with bone, but I am still unsure of what to do. Any help / point to a feature I should be using would be hugely appreciated.

func staticHandler(w http.ResponseWriter, r *http.Request) {
    w.Header().Set("Cache-Control", "max-age=x")
}

You can set header while responding back Cache-Control: max-age=x, where x is in seconds.
Better option will be to serve static files directly from your load balancer and having header rules at that level.


Add defaults + split SignIn/SignOut

Move SignIn/Signout into IAuthenticationHandlerSign[In\Out]Handler
Add tests for AuthenticationService
add overloads for SignOut/Forbid to use new default

@Tratcher

Fixes https://github.com/aspnet/Security/issues/1244 and https://github.com/aspnet/Security/issues/1264
Need to update Security/Hosts (to remove SignIn/Signout impls only) as a result of this PR as well

I'm going to combine the events changes we want for https://github.com/aspnet/Security/pull/1266 into this PR as well.  So the update to AuthenticationResult + rename of None => Ignore will go in this PR as well

So much sorcery in this PR... :cry:
It would be much better to throw an InvalidOperationException when there's more than 1 scheme to encourage the developers to read the related documentation, so they can set the different schemes themselves.

Related post: https://github.com/aspnet/Security/pull/1267#discussion_r122566994.

Updated the default sign/signout to enforce the handlers implement the interface, otherwise they return null (should this throw instead?)

We don't throw if the default scheme doesn't exist today, so it feels weird to throw

We'll also make SignIn derive from SignOut which dervies in Auth

Updated, SignInHandler is now the old uber AuthenticationHandler that has all methods implemented.  Renamed None() => NoResult() with the bool property Nothing => None (done for Security)

271faf11bbd5b05cd758f1c7e83eb59d45b6db59


feat: add support for initialization error endpoint

Thanks for this great project! This PR adds support for the initialization error endpoint at /2018-06-01/runtime/init/error, which terminates the invocation immediately.
I didn't found any tests regarding those endpoints so this PR doesn't add any.

Fixed the Clippy errors!

Seems like a flaky test: https://github.com/cargo-lambda/cargo-lambda/actions/runs/7493006725/job/20398803280?pr=591#step:6:1164


ctrl + shift +x map become useless in goneovim

I set
noremap <C-H> :BufferLineCyclePrev<CR>
noremap <C-L> :BufferLineCycleNext<CR>

in my init and they work fine in a plain neovim. However, goneovim seems don't know what I just entered and result nothing when I "C-H"

Hi @fecet.  (Just another Goneovim user here.)  Just thought I'd see if I could reproduce the problem.  I don't have your buffer line plugin, but it shouldn't matter.
The title of the issue said "ctrl + shift +x".  I thought that <C-H>, etc. meant the same thing as <C-h>.  Is that not always correct?
I tried defining mappings in plain nvim and in Goneovim, and there was a difference, but not exactly what you are describing here.  What I found was that I can define different mappings for Ctrl-Shift-H vs. Ctrl-H in Goneovim, but not in plain nvim.  I'm using <C-S-H> for the first mapping, and <C-H> for the second mapping.
(However, I am on MacOS, and you might be on a different OS.  And I think that the reason that Ctrl-Shift-H and Ctrl-H can't be distinguished in plain nvim might have something to do with my terminal app (iTerm2) or the terminal settings.)
Maybe none of that is relevant to your problem, but since I don't have the same problem, it might matter which OS version of Goneovim this is.  (Also, you have probably done this, but if not maybe it's worth defining <C-H> and <C-L> to something other than the :BufferLineCycle* commands to see what happens, in case the problem is not with mappings but with how the buffer line plugin interacts with Goneovim.)

Thanks for your detailed explanation @akiyosi , map to "C-S-H" save my life!

That's great.  I'm happy I could help!

@fecet @mars0i
hi, sorry for late reply.
Generally, @mars0i's comments are the same as my perception. Thank you @mars0i
This issue has been fixed in reference to changes made to neovim and neovim-qt. Please refer to the following issue for details.
https://github.com/neovim/neovim/issues/12037
https://github.com/equalsraf/neovim-qt/issues/817


Could not resolve .jsx file when import from other .jsx

Current Behavior
Encounter an error
Error: Could not resolve './ComponentC' from src/ComponentB.jsx

When import JSX file from JSX file with than 2 levels deep, eg
index.ts -> A.jsx -> B.jsx -> C.jsx
# Error: Could not resolve './C' from src/B.jsx

index.ts -> A.js -> B.jsx
# Error: Could not resolve './B' from src/A.js

index.js -> A.jsx
#  Error: Could not resolve './A' from src/index.js

Below is working fine
index.ts -> A.jsx -> B.jsx

# or

index.ts -> A.jsx -> B.jsx -> C.jsx
index.ts -> B.jsx

Repo for reproducing

https://github.com/n3tr/tsdx-import-jsx-test

Expected behavior
The build command should be successfully
Suggested solution(s)
I don't really know what is the root cause of this, but trying to set  extensions: ['.mjs', '.js', '.jsx', '.json', '.node'] to rollup-plugin-typescript2 seems to solve the issue.
https://github.com/jaredpalmer/tsdx/blob/3a6d42fb767150233c651056aabda9dd91bd048d/src/createRollupConfig.ts#L105-L111
Additional context

Your environment

Software
Version(s)

TSDX
0.12.3

TypeScript
3.8.2

Browser

npm/Yarn

Node
v12.4.0

Operating System

Weird that it works sometimes but not others... 🤔🤔
(Also you've got a typo here, you wrote rollup-plugin-typescript2 instead of @rollup/plugin-node-resolve)


Travis CI is dropped

Given the poor handling of the recent security incident (https://arstechnica.com/information-technology/2021/09/travis-ci-flaw-exposed-secrets-for-thousands-of-open-source-projects/) I'm suspending Travis CI integration indefinitely.
Let's move on to GitHub Actions.

https://github.com/sbt/sbt-osgi/pull/88


ENH: Autodetect README file and its MIME type

Check through a small list of likely candidates for README filenames:
['README.rst', 'README.md', 'README.txt', 'README']
and use the first one which exists.
We then infer the long_description_content_type argument from the file extension of the readme file.

I'm using a try except block to handle the files which may or may not exist because its the best way of handling a file which may or may not exist. https://stackoverflow.com/a/82852
Before I was using FileNotFoundError, but have now realised it was introduced in Python 3. So I've changed it to IOError which is on both Python 2 and 3.

too complicated

thank you though!


add note pointing contributors to DefaultDockerClientUnitTest

Document that we would prefer to have more unit tests than integration
tests for new functionality going forward, now that
DefaultDockerClientUnitTest uses a mock web server where we can capture
and stub HTTP requests/responses.

@davidxia

Codecov Report

Merging #760 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #760   +/-   ##
=========================================
  Coverage     65.56%   65.56%
  Complexity      655      655
=========================================
  Files           153      153
  Lines          2878     2878
  Branches        335      335
=========================================
  Hits           1887     1887
  Misses          844      844
  Partials        147      147


separate user for each team member

Hi,
The security department asked me to have a separate user for each team member or each employee accessing orchestrator even for read only (we all like security right ? :-) ) , is that possible ?
Is there a way to use "Google Sign in" to manage access to orchestrator or any other external authenticator like AD ? if not please add it :-) .
Br

Hi,
I'd suggest using LDAP. orchestrator itself has no built-in support for ldap/google-sign-in/oautch/etc.
Instead, what I've done in different places:

bind orchestrator to <IP_ADDRESS>:3000 (pick your port) ; this will ensure it cannot serve external requests
run nginx or other reverse proxy on same server
have nginx authenticate users via ldap
have nginx send HTTP header to orchestrator with authenticated username.
pizza

see https://github.com/github/orchestrator/blob/master/docs/security.md, "https://github.com/github/orchestrator/blob/master/docs/security.md, "Headers authentication".


Flutter_Quill_Extention-> Image-> If to use the zoom option from the image embed menu, we get the red screen Quill Provider is null

Is there an existing issue for this?

[X] I have searched the existing issues

Steps to reproduce
Use the Example Project in this repository
Once the application is loaded

Add an image form Gallery is not present in Quill Editor
Click on the image (you will see a popup)
Click on the Zoom button

you will see the following screen.

Expected results
Application Should not crash
Actual results
Application is Crashing
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output (optional)
Doctor output
[Paste your output here]

Hello there, on behalf the Flutter Quill Team I would like to thank you for opening your first issue. Your inputs and insights are valuable in shaping a stable and reliable version for all our users. Thank you for being part of the open-source community!

Thank you for the report, it looks like you have used the master branch which might have things that haven't been released and tested yet, we will take a look at it soon.


Preview drawing

Hello,
Again, thanks for creating this package. It has helped my project a lot!
I have an issue regarding drawing circles or triangles over an image. When I do not use a image, I can see a sort of preview (see screenshot).

When using an image to draw over, this circular preview disappears. How do I solve this issue?

you mean when you set background image the guide is missing?

I publish beta version of 1.0.13. You can try it or test it on this sandbox.
Let me know if you found any bugs.

Your sandbox version solved the issue!
Can you let me know when this will be released?

Closing this issue with the release of 1.0.13


HttpRequest constructor should accept a "json" kwargs

Which is mostly syntactic sugar for set_json_body.
This issue implies adding "Content-Type: application/json" to set_json_body.
Beware of headers + json as kwargs.

addressed this in azure.core.rest. Removed azure.core and client labels to avoid confusion with GA azure core


Fix HK2 service reification failure

The jaxrs_client_utils tests used to dump lots of warnings of the format:
WARNING: HK2 service reification failed for [org.glassfish.jersey.message.internal.DataSourceProvider] with an exception:
MultiException stack 1 of 2
java.lang.NoClassDefFoundError: javax/activation/DataSource
	at java.base/java.lang.Class.getDeclaredConstructors0(Native Method)

This adds them to the test scope. It's not entirely clear to me whether 1. another scope would be better (provided I presume), or 2. whether there are other tests that need similar treatment.

👍I assume that those deps are provided in all runtime scenarios, at least inside a Jdisc container, so that's probably why you don't see those warnings there. Test scope is probably fine.

it's compile scope in jdisc_core, which I found a bit surprising, but it might be enough depending on which bundle that reflection stuff is happening at run time. And the *-dependencies-enforcer has lists e.g. com.sun.activation as provided!?

jdisc_core classes are provided from the classpath.


"Dictionary required" when referencing class members  returned from autoloaded functions inside functions

Steps to reproduce

Put the following in ~/.vim/autoload/autoloaded_script.vim:

vim9script

export class My_class
	this.hello: string
endclass

export def Return_my_class(): My_class
	var r = My_class.new('From my_test_autoloaded_script')
	return r
enddef

Source the following in Vim:

vim9script

import autoload 'autoloaded_script.vim'

def My_func()
	var whatever = autoloaded_script.Return_my_class()
	echo whatever.hello
enddef

My_func()

You'll see E715: Dictionary required. This doesn't happen if you move the code outside the function; it doesn't happen if you declare the class inside the script you're sourcing. It doesn't happen when you call the class's constructure inside your test file.
I'm guessing that this issue stems from the same space as that those mentioned in #11822 and  #12024.
Expected behaviour
You'd expect it to print "From my_test_autoloaded_script".
Version of Vim
9.0.1427
Environment
Fedora 37
Kitty
xterm-kitty
Zsh
Logs and stack traces
No response

These issues, along with #12196, may originate from the same reason: Vim rely on the typing information it knows at the point when the code is compiled to generate proper instructions, and for member access with any/unknown type as its lhs, Vim would generate STRINGMEMBER instruction to index the member by its name; the problem is that the instruction only accept dictionaries as lhs, thus the error is raised.
So as a workaround now, give a more precise explicit typing can solve this: it forces Vim to generate proper instructions to access the correct member. This is an example:
a.vim:
vim9script

export class T
    this.n: string
    def F()
        echom "F:" this.n
    enddef
endclass
export def RT(): T
    return T.new("QWQ")
enddef

b.vim:
vim9script

import autoload "./a.vim"

def F()
    var v: a.T = a.RT()
    echo v.n
    v.F()
enddef

F()

a: a.T forces Vim to generate the correct instruction.
BTW, since the use of any is inevitable now, I think maybe the only solution is to extend STRINGMEMBER instruction, allowing it to access class members and functions.

Just an FYI, this has nothing to do with autoload. Putting the two files in the same directory, with a primary file that looks like the following also fails.
vim9script

import './f2.vim'

def My_func()
	echo f2.Return_my_class().hello
enddef

My_func()


text/v2: use embedded bitmap data when possible

Operating System

[ ] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] PlayStation 5
[ ] Xbox
[ ] Web Browsers

What feature would you like to be added?
Some TrueType/OpenType fonts include bitmap data for small glyphs. These provide better user experience for low-resolution environment.
This feature is requested by @kettek .
Why is this needed?
Better UX

We just have to treat bitmap data at https://github.com/hajimehoshi/ebiten/blob/6eee7ccc5fffb7183a53f0f069486334ff7ab939/text/v2/gotextfacesource.go#L228-L231
See also https://pkg.go.dev/github.com/go-text/typesetting/opentype/api#GlyphBitmap

https://github.com/go-text/typesetting/issues/160


Prevent Label from disappearing upon scrolling

When the TextView is limited to a certain height, the label disappears when the user scrolls beyond the viewport of the TextView. How can this be disabled? Thanks

I think it's functioning as intended, but I'll see if I can add this feature.

Awesome! Let me know how I can help.
Sent from my iPhone

On Jul 1, 2015, at 7:55 PM, Arthur Ariel Sabintsev<EMAIL_ADDRESS>wrote:
I think it's functioning as intended, but I'll see if I can add this feature.
—
Reply to this email directly or view it on GitHub.

I thought about this feature. The point of the floating label is to remind the user the name/purpose of the field/view after they've filled it out, not while they're filling it out. Therefore, I will not be adding this feature.
Sorry for taking so long to respond - I had to think about it for a bit.


Timeshifting in HLS ?

I have created the following config for hls in application 'live':
hls on;
hls_path /hls;
hls_fragment 10s;
hls_playlist_length 18h;
hls_sync 100ms;
hls_fragment_naming system;
hls_continuous on;
hls_nested on;

pull rtmp://localhost:1935/live/test.stream name=test.stream live=1 static;

Now the index.m3u8 contains timestamp-named chunks and plays fine.
My question is - is it possible to create a timeshifted version of the index.m3u8, which would play the stream shifted back in time ?
One idea is to implement it as an hls option, e.g.:
hls_timeshift_index name=index_5.m3u8 timeshift=5h;
hls_timeshift_index name=index_3.m3u8 timeshift=3h;

Another is to implement it as an argument in HTTP request (is it possible ?):
http://server/live/test.stream/index.m3u8?timeshift=5h

Thanks !

Anyone who can help me with this as well? I'm trying to achieve the same thing, arut said something about ngx_rtmp_hls_module.c. And that it's simple anyone who can provide help? Would be much appreciated.

Has anyone got the answer as arut stated as I am looking to do this too

i'm looking the same

same question

Here's my setting, but it doesn't really play from the beginning:
            application hls {
                    live on;
                    hls on;
                    hls_fragment_naming system;
                    hls_fragment_slicing aligned;

                    hls_path /var/hls;
                    hls_nested on;
                    hls_continuous on;

                    hls_type event;
                    hls_fragment 2s;
                    hls_cleanup off;
                    hls_playlist_length 10m;
            }

any one having luck with nginx rtmp timeshifting? I am not sure which is the best forked project on this?

Need to know as well


migrate to pre-commit for linting+formatting

using pre-commit has some advantages

you can't commit code to the repo doesn't meet your quality gates (unless you use git's --no-verify flag)
there are numerous hooks available for improving quality
dependencies are installed in an isolated environment

that last point is especially important for this project, as its dependency resolution seems quite brittle

Thanks for the PR.
As I don't  have any experiences with pre-commit hook, I will test it a little bit today.
But I like the idea and the PR looks valid for me.


Update Alloc to take the type it allocates instead of pointer type it creates

closing for now to clean up the PR list, not sure if you are working on this


Turkish localization

Hello,
I am developing a computer-assisted translation tool in Django/Electron and stumbled upon django-helpdesk a few days ago, while looking for a ticket platform to handle user queries and whatnot. Firstly, great job with the Django app!
I see that there are 500+ strings that have not been localized into Turkish. As a translator/opensource-enthusiast, I would like to offer help. I've already submitted a request to join the Turkish team on Transifex. My username on Transifex is csengor. If the django-helpdesk account on Transifex is no longer managed, I can, alternatively, translate the .po file using the CAT tool I'm developing and create a pull request.
Best,

Hello @csengor! Thank you for offering to update the Turkish translations! I'm sorry I didn't see this sooner, for some reason I did not receive an email notification so didn't know you submitted a request. I just approved you on Transifex so please feel free to use it and open a pull request when you're ready to submit. Thanks again!


[ConstraintSystem] Detect and fix extraneous use of &

Diagnose extraneous use of address of (&) which could only be
associated with arguments to inout parameters e.g.
struct S {}

var a: S = ...
var b: S = ...

a = &b

@swift-ci please test

Test case?

@slavapestov We have a lot in the suite already, I'm just trying to port more diagnostics associated with assignment before I can port "contextual mismatch" ones...

@xedin Gotcha, thanks!

No problem!


[TUBEMQ-199] Support TubeMQ sink function for flink

Support TubeMQ sink function for flink
Jira link: https://issues.apache.org/jira/browse/TUBEMQ-199

+1


List/Dict arguments defaults to "body" instead of "form"

If a view argument is annotated as a dict or list, it is considered a "body argument":
https://github.com/encode/apistar/blob/master/apistar/components/dependency.py#L589
However, body arguments return the entire request data object. "Form" arguments, in contrast, take the request data object and attempt to extract a specific keyword. Given that there is already a builtin component for accessing the entire request data object, and that users are specifically naming these arguments in the function definition, I think keywords should default to form arguments instead.
Consider:
def fn(a: dict, b: list, c: apistar.http.RequestData):
    return [a, b, c]

# with "body" arguments:
result == [<dict(RequestData)>, <list(RequestData)>, <RequestData>]

# with "form" arguments
result == [<dict(RequestData['a'])>, <list(RequestData['b']), <RequestData>]

I don't think the "body" case is nearly as useful as the "form" one, especially for POST requests.
(Note that #291 makes this behavior overridable, but I suggest this change to the default)

This’ll be obsoleted by #400


Updated mocha version

Summary
Updated mocha to its current version (6.1.4).
Resolves #125

Running npm install then npm test resulted in all tests successfully passing.


revert code and use new method to fix publish_module_bundle crash

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)

reference https://github.com/move-language/move/blob/aptos/language/move-binary-format/src/normalized.rs#L280


Update distribution.yaml

Add IG_exploration and exploration_planning while are used in autonomous environment exploration.
I'd like IG_exploration and exploration_planning to be indexed and documented on ros.org

I'd like IG_exploration and exploration_planning to be indexed and documented on ros.org

this branch has conflicts, can you please rebase it ?

@ZhuangYanDLUT friendly ping

@ZhuangYanDLUT I'm closing this due to no response. Feel free to comment here or to open a new pull request once this branch is rebased


Fixed batch gradient warning

Summary
Fixed warning in TestTorchConnector.test_circuit_qnn_batch_gradients
Details and comments
I found that the dataset for TestTorchConnector.test_circuit_qnn_batch_gradients was not correct. I corrected the output_size parameter in this test and added warnings to circuit_qnn.py to help guide the user with output_size.

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

Thanks for the feedback. I have updated the docstring in circuit_qnn to add the constructor rules, and have explained each rule in what I have considered the most relevant argument (most of them are in output_shape). As a guidance, here is a compilation of all the rules that I have noticed and the corresponding argument where I have added them (1.1 was already clarified in the docs, but I added it to the list to have a full overview of how the constructor works):
Constructor Rules for CircuitQNN:

If sampling==True:
1.1. The output will always be DENSE (sparse==False), independently
of the value provided for the 'sparse' argument (it will be overridden). --> [already in docs] sampling argument
1.2. If sampling==True and interpret==None, the output shape will be automatically
calculated as (num_samples,1). --> [new addition] output_shape argument
1.3. If sampling==True and interpret!=None, the output shape will be automatically
inferred from the interpret function as (num_samples, interpret_dim).  --> [new addition] output_shape argument
1.4. 'statevector_simulator' is not a valid backend for 'quantum_instance'. --> [new addition] quantum_instance argument

If sampling==False:
2.1. If sampling==False and interpret==False, the output shape will be automatically
calculated as 2^num_qubits, independently of whether the output is dense
(sparse==True) or sparse (sparse==False), or whether a custom shape has been
provided through the 'output_shape' argument. --> [new addition] output_shape argument
2.2. If sampling==False and interpret==True, the output shape MUST be provided
through the 'output_shape' argument, independently of whether the output is dense
(sparse==True) or sparse (sparse==False).** --> [updated with sampling condition] output_shape argument**

If anything isn't clear, please let me know. Thank you!


update fork from master

Update fork from main repository
CATEGORY/PROJECT_NAME - PROJECT_LINK
PROJECT_DESCRIPTION

update fork from master

(screwed the update of my fork/copy of the repository - please forgive me ! :p)


Usermanager.GetuserAsync and User.IsInRole don't work with v. 2.0.0-xxxxxx

Hi,
I don't know if I'm in right repo to speak about that but with 2.0.0-preview1-final
my web mvc app with identity individual
 services.AddIdentityServiceAuthentication();
with 1,1.0, my controller or view used a method:
await _userManager.GetUserAsync(User); to get user properties
with 2.0.xxx
please find attached the result in the capture:

What am I missing  or the all model to get access to the stuff changed ?
Txs
-Vince

@HaoK @javiercn

@danroth27

This issue was moved to aspnet/Identity#1243


Fix composer cache typo in 7.1-cli Dockerfile

Thanks for the spot.  It's in the FPM image too.  Can you change it there as well please?

Thanks!


AppBar can't function in a container that's less than 64 dp wide

See: https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/app_bar.dart#L61

Yeah, this is part of https://github.com/flutter/flutter/issues/6537

Fixed (https://github.com/flutter/flutter/pull/6894)


Chang htarg/ftarg to dicts internally

Closes #15

Coverage decreased (-4.6%) to 95.349% when pulling edaa648d0293286c6c2e39ab38273a6d13722373 on htarg-ftarg into 7529a8ef8098310dd64e8b52af6899aab789469b on v2-dev.


[sysrst_ctrl,lint] Hush an unused signal warning for nc_intg_err_o

This has a FIXME next to it already; rename the signal to avoid the
linters warning about it as well.

Yes. Tim notified me about it. I had a pull request for it.
https://github.com/lowRISC/opentitan/pull/6134/checks

Oh, great! I'll close this one, thanks.


No login after enabeling 2fa

Hello,
sorry to bother you :-(
This seems not to be a hard issue (for me) but i don't understand it yet.
I've done the following:

create a new user as admin
login as the new user
scan the barcode in google autheticator as the new user
enter the current password, check "enable 2fa" and click save
the app asks for the google authenticator code
the new user enters the code and the app says: Successfully updated settings".
open the login page and try to login as the new user by providing username & password. That produces the following error:

It seems that the reason for that behavior is, that after a user enables 2fa the field "password_enabled" in table "users" is set to "0".
After i set that manually in the table to 1, a normal login with 2fa is possible again.
Best wishes
Sascha

Maybe do you have unchecked the Enabled Password Auth on your profile?

hmmm, when i am logged in as a normal user, there is no such option in the profile page...

Even when i try to edit the user as admin, there is no such option

mmm, ok, the option is only shown if you have certificate auth enabled. Let me check.

fyi: I've disabled the "certificate_auth_enabled" param in .env as i don't want to show the "login with certificate" button on the login page. Maybe that is the problem here...???

Fixed.
password_enabled attribute on user is set to false on profile update when auth.certificate.enabled = false.
Thanks!


CORE-19319 - Fix bug caused by using wrong db namespace

There was a bug in the app-simulator script, caused by using the wrong db namespace for the one way load generation.

Closing this and will only fix in 5.3.


[WIP] Refactor sourceMgr

This PR refactors the sourceMgr in several areas:

Fixes race conditions/concurrency issues - #170
Incorporates context.Context - #159
Make ProjectAnalyzer a method parameter - #195

The changes here also pave the way for persistent caching - #130.

#202 is quite related to this, but I'm gonna leave it out of this PR to kinda sorta avoid scope creep just a teensy bit.

Noting this here because I'm not sure if it'll remain an issue or not, so I don't want to make it a separate issue until this is merged. But, this PR as currently written has a problematic case. (The problem exists on master, too, but is even less organized in how it deals with the issue)
Say there's a ProjectRoot, github.com/foo/bar. When attempting to get the source handler for that project, the default deduction and maybeSource path will end up pointing to https://github.com/foo/bar. That's the first scheme it'll try, anyway.
The role of the maybeSource is interesting. It's the way that the root deduction system abstracts over which exact upstream URL (scheme, mostly) might be used to fulfill an import. maybeSource.try() needs to balance two things: returning as quickly as possible, but only returning if it's reasonably sure the source it's deduced actually exists.
There's two definitions we could use for "a source exists": it exists upstream, or it exists in the cache. The latter, being local, is faster to check, but potentially wrong about the state of the world - even if the source exists in local cache, the upstream could have moved, been deleted, become inaccessible due to expired credentials, etc.
For now, I think I'm going to just force the remote check to always happen. However, it seems likely that we may need to revisit this, and rely only on the local check initially. If/when we make that shift, we'll need to add a mechanism by which to reach back into the sourceCoordinator and change its various maps to correspond to the new URLs. This could present issues for the current logic in getSourceGatewayFor(), as it's all carefully crafted around the idea that those maps are essentially append-only.

Thanks for the review! This is getting there...slowly... 😛

OK, I think #170 is done at this point. There's a little more cruft to clean out in the vcs sources (like, we don't need the *repo substruct anymore, can just move that up top onto the baseVCSSource), but...basically, the internals here look pretty good.
One thing I'm punting on, though, is proper handling of windows processes. This is a gnarly area in general. Reading through some threads like golang/go#17608 and make me think that we should maybe/probably have specialized implementations - this might be a starting point.
https://groups.google.com/forum/#!topic/golang-nuts/XoQ3RhFBJl8  also suggests that...well, maybe we need some special linux handling, too.

(as coverage suggests, a buncha tests still needed)

As-is, this is very likely to cause a perceived performance regression. It will end up serializing calls from the solver that previously had been parallelized, as it reverses the change made in 4815cd5d.
The fundamental problem is, we can't avoid that regression while maintaining the invariant I described as valuable earlier.
However, this PR has been open for far too long, and diverged far too much as-is. Because it's accomplished its main purpose, I'm going to merge it now, and then we can revise and modify more incrementally.


Add typings definition that can be used in typescript

It would be nice to have the typings definition of this script or you could submit your typings definition to: 'https://github.com/DefinitelyTyped/DefinitelyTyped' so that it can be added to the packages as @types/element-resize-detector.

See my answer in #76.


Function for getting state of units

Nice API. I found no function how to get the state of the unit.. Do you realise to get this data by another way or the function is just missing?

Thanks!
I have moved on to https://github.com/hellqvio86/aiocasambi (asynchronous version of this API), they send the state over the virtual wire (websocket).
I am using that in this project https://github.com/hellqvio86/home_assistant_casambi (home assistant integration for casambi).

I have raised a feature request (getting state over the rest api), but they have not yet implemented it (they probably never will). So now you need to listen on the websocket for incoming messages and they usually send all the state over the socket when you connect. So one way of doing it is to connect over websocket wait for the state, then disconnect, return result etc.

Ok I did not saw that you have also another library ;-) I will try to check the async out, next time. I assume the normal casambi library is kind of deprecated now :-)


Can't create new receiving address

Hi,
I'm running the Windows wallet as just a normal user and I'm trying to create a new receiving address.  I enter a Label and I'm prompted for my password.  I enter the correct password; however, I get the error Could not unlock wallet.
Sometimes, closing the wallet and restarting can help and allow the address to be created.

Just unlocked the wallet, rather than unlocked the wall for staking only, and I was able to create a new receiving address.


feat: added direct link provider sdk support

PR summary

Fixes: <! -- link to issue -->
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

@KumarGanesanIBM Can you please check the failures for SSL certificate. I tried it 4-5 times but they are continuously failing.
Thanks

@KumarGanesanIBM Can you please check the failures for SSL certificate. I tried it 4-5 times but they are continuously failing.
Thanks

@ajay-malhotra1
Hi Ajay,
The Given SSL certificate as env variable, got expired.
Updated it and started travis build.
https://travis-ci.com/github/IBM/networking-python-sdk/jobs/386802209
Please give it some time to finish and rebase your code and start your build process
Thanks...

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


add extensions methods to schema

Add createExtension, createExtensionIfNotExists, dropExtension, dropExtensionIfExists methods.

Coverage decreased (-0.6%) to 96.818% when pulling a5564a2980cf282930c8a6b9f14377421fb398ca on enniel:add-new-methods-to-schema into 63bad5701569787c4d6b30ff968b71a334daac6b on adonisjs:develop.

Ohhh I never knew about it https://github.com/tgriesser/knex/issues/1318

Can u add the documentation too here https://github.com/adonisjs/docs/tree/4.0. The branch is 4.0

Coverage decreased (-0.6%) to 96.818% when pulling 510f0603b3cb0e328403650c98b3640bec7fdb52 on enniel:add-new-methods-to-schema into 63bad5701569787c4d6b30ff968b71a334daac6b on adonisjs:develop.

Coverage decreased (-0.6%) to 96.818% when pulling 9825448b4d3995dd4169e7541ee83ef7fbaab738 on enniel:add-new-methods-to-schema into 63bad5701569787c4d6b30ff968b71a334daac6b on adonisjs:develop.

docs added https://github.com/adonisjs/docs/pull/143

Thanks 😄


Make available on Bower

BBH v0.1.0 is available via NPM. It would be useful to have BBH available on Bower as well.
This should just require adding a bower.json and publishing to Bower via bower register.

The file required for this was committed by PR #8, but I'm not going to close it until the Bower registration has actually happened.

BBH is now available via bower install babel-browser-harness.


Yarn Build - removing required field from bool type

closes #661

As per my conversation with @bradystroud , we have identified that some modification is made by Tina that is causing the issue during the execution of the yarn build process. It appears that in Tina's schema, when a variable is of boolean type and marked as required, it should be set to true. Otherwise, it will be considered null, resulting in GraphQL being unable to retrieve its value accurately.


Feature-test moz-chunked-arraybuffer before use

Resubmission of https://github.com/mozilla/pdf.js/pull/5457, rebased to master. The previous one was closed with the following comment:

Closing for now. #5448 has landed, so the issue itself should be fixed with less code than in this PR. Optimizions for the landed fix can be proposed in a new (rebased) PR.

This PR indeed has a bit more code, but that is caused by the fact that this PR uses feature detection instead of try-catch.
The other diff is just some re-indention after moving the Mozilla-specific APIs between preprocessor conditionals.

@yurydelendik Could you merge this? Since Chrome 44.0.2360.0, the console prints the following warning:

The provided value 'moz-chunked-arraybuffer' is not a valid enum value of type XMLHttpRequestResponseType.

I'd still like @yurydelendik to sign off on the patch, but let's check that the tests pass.
/botio test

@yurydelendik Gentle ping to get this PR merged. I want to get rid of console spam (https://github.com/mozilla/pdf.js/pull/5531#issuecomment-109447710) (the log spam already appears in the current version of Chrome, 43.0.2357.81).

Looks good. Good to land after testing.
/botio test

/botio-linux preview

node mode generic fails with:
### Creating generic viewer
Could not evalute line '!(CHROMIUM || FIREFOX || MOZCENTRAL)' at /mnt/botio-files-pdfjs/private/7ee6b6f9ef78a3b/build/pdf.worker.js:1674

/mnt/botio-files-pdfjs/private/7ee6b6f9ef78a3b/external/builder/builder.js:72
            throw e;
                  ^
ReferenceError: CHROMIUM is not defined
    at evalmachine.<anonymous>:1:3
    at preprocess (/mnt/botio-files-pdfjs/private/7ee6b6f9ef78a3b/external/builder/builder.js:68:24)
    at /mnt/botio-files-pdfjs/private/7ee6b6f9ef78a3b/external/builder/builder.js:221:7
    at Array.forEach (native)
    at /mnt/botio-files-pdfjs/private/7ee6b6f9ef78a3b/external/builder/builder.js:215:13
    at Array.forEach (native)
    at Object.build (/mnt/botio-files-pdfjs/private/7ee6b6f9ef78a3b/external/builder/builder.js:210:20)
    at Function.target.generic (/mnt/botio-files-pdfjs/private/7ee6b6f9ef78a3b/make.js:152:11)
    at Object.global.target.(anonymous function) [as generic] (/mnt/botio-files-pdfjs/private/7ee6b6f9ef78a3b/node_modules/shelljs/make.js:28:26)
    at Function.target.web (/mnt/botio-files-pdfjs/private/7ee6b6f9ef78a3b/make.js:224:10)
``

CHROMIUM should be CHROME everywhere according to the targets at https://github.com/mozilla/pdf.js/blob/master/make.js#L74

/botio-windows test
/botio-linux preview

/botio test

Thank you for the patch


Fix gimp and hex-fiend with a+rx

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

GIMP does not correctly set 'group' and 'other' permissions and will crash if a non-owner runs it. The buggy files are fixed by updating their 'group' and 'other' permissions to reflect the 'user' permissions: chmod a+rx. Before patch:
$ ls -l /Applications/GIMP.app/Contents/MacOS
total 16120
-rwxr--r--  1 Administrator  staff     2920 Dec 13  2015 GIMP
      ^~~~ lolwhoops
-rwxr-xr-x  1 Administrator  staff  8229340 Dec 13  2015 GIMP-bin
-rwxr-xr-x  1 Administrator  staff    13088 Dec 13  2015 python

Hex Fiend was previously patched with chmod og=u, which would give everyone write permission, as pointed out in this issue comment: https://github.com/ridiculousfish/HexFiend/issues/52#issuecomment-151438511. This commit also changes Hex Fiend to be patched with chmod a+rx.

Thank you. But please separate these in two commits.

Separated!

Thank you.


Update release notes + API documentation for CP 175 release.

Update release notes + API documentation for CP 175 release.

Merged per request from @hansthen.


Airline support

Thanks for the colour schemes. Could you please add vim-airline support?

Sorry, I don’t use vim-airline. Would you know how I should go about creating one, i.e. if there are any best practices and such?


Crowdin badges!

:clipboard: Description

It would be awesome to have few Crowdin badges! Like:

overall localized %
number of languages (every, 100% translated?, 100% proofread?)
localized % for a given language

:link: Data
I didn't find any API endpoint for stuff like that, but maybe they have some, they look like cool guys and may warmly reply to an email :)
If not the API, scrapping is also an option. The projects reside on /project/<project-name>, for example https://crowdin.com/project/fast-shopping which is not disallowed for scrapping: https://crowdin.com/robots.txt

:microphone: Motivation

I want to have cool shields in my open-source app's README!

Love Shields? Please consider donating $10 to sustain our activities:
👉  https://opencollective.com/shields

I love them! I donated a little :)

I love them! I donated a little :)

thanks :heart: going to close this as a duplicate of #4906 though


IOU - Missing Hold request, Replace and Delete receipt option when there is one IOU request

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.59-3
Reproducible in staging?: Y
Reproducible in production?: N
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Go to staging.new.expensify.com
Go to 1:1 DM with user that has no unsettled IOU.
Create a manual request with receipt/
Click on the IOU preview.
Click 3-dot menu.
Open the receipt.
Click 3-dot menu.

Expected Result:
Hold request, Replace and Delete receipt option will be present.
Actual Result:
Hold request, Replace and Delete receipt option are missing when there is only one IOU request.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/8818b21e-5e30-416f-8d7c-bfbea6ff46e7

View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Hold request, Replace and Delete receipt option are missing when there is only one IOU request.
What is the root cause of that problem?
We don't add hold or uphold option into three-dot menu items of MoneyReportHeader
https://github.com/Expensify/App/blob/ebc8de8229fa700e5a7d08f4589db934a79d4f1d/src/components/MoneyReportHeader.tsx#L164
For three-dot in receipt, this is dupe of https://github.com/Expensify/App/issues/39436#issuecomment-2033854684
What changes do you think we should make in order to solve the problem?
We should use the same approach in MoneyRequestHeader to add hold or unhold option to three-dot menu item.
https://github.com/Expensify/App/blob/ebc8de8229fa700e5a7d08f4589db934a79d4f1d/src/components/MoneyRequestHeader.tsx#L125
https://github.com/Expensify/App/blob/ebc8de8229fa700e5a7d08f4589db934a79d4f1d/src/components/MoneyRequestHeader.tsx#L118
https://github.com/Expensify/App/blob/ebc8de8229fa700e5a7d08f4589db934a79d4f1d/src/components/MoneyReportHeader.tsx#L164
What alternative solutions did you explore? (Optional)
NA

@stephanieelliott I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think this issue might be related to the #collect project.

I wouldn't expect Hold request to be in the 3-dot menu for the receipt image. I would expect Replace and Delete though. If we are going to address Replace and Delete in https://github.com/Expensify/App/issues/39436, then I think we can close this out.

Reopening the issue for a second chance. It seems to occur only when there is a single request.
https://github.com/Expensify/App/assets/93399543/7cc5acd7-219a-4b0b-8f88-e05a555c6d29

Reopening the issue for a second chance. It seems to occur only when there is a single request.

This seems to be for the three dot menu on the expense. I'm not sure if that behavior is expected or not I'll ask internally

Discussion about this is ongoing here and here. I'm gonna put a hold on this since there's no UI to show which expenses are held.

Unassigning since there's no more action to take for now

Moving to Weekly while we're on hold

Still held

I'm gonna put a hold on this since there's no UI to show which expenses are held.

Still held

Still holding

We have a design for the UI pattern now, this scenario has been incorporated into the design: https://github.com/Expensify/App/issues/39989


M3-1172 Correct balance display on MakeAPayment

Purpose
It was found that balance display was swapped, showing credits with a negative symbol and in the color red while showing owned balance as a positive value in green.

Coverage remained the same at 26.241% when pulling 3ad692fbbc5ff1344f87d55cd6938f8ddcd0fa4a on rjstires:M3-1172 into 8a63f6b4c464a2d67012401f7455c9150cfbdb13 on linode:release-0.31.5.


[Feature Request] Reset Workflow

Hi. Is there any way/code snippet to reset workflows in python sdk?
like this question
https://community.temporal.io/t/how-do-i-restart-an-opened-workflow/1044/2
https://docs.temporal.io/tctl-v1/workflow#reset

There is not in any of our SDKs because this is an advanced concept. If you don't want to use the CLI, technically you can call await my_client.workflow_service.reset_workflow_execution(temporalio.api.workflowservice.v1.ResetWorkflowExecutionRequest(... which would be calling the raw API with a raw API request. But that is a bit of an advanced API.
Resets are meant to be rarely used, not necessarily part of normal code, so we may not provide a common code sample for this.

Closing as answered

Thanks for your help. I found that there is a "reset_type" in tctl (that can be LastContinuedAsNew) but it's not supported in ResetWorkflowExecutionRequest structure. what is reason of this mismatch? (background conversion in tctl or a missing field in proto or ...)

That is used to crawl the history to find a finishing task ID to reset to. See https://github.com/temporalio/tctl/blob/d124bd56ba6bce7d6b6b90decb5093d4c007c653/cli_curr/workflowCommands.go#L1438-L1490 and https://github.com/temporalio/tctl/blob/d124bd56ba6bce7d6b6b90decb5093d4c007c653/cli_curr/workflowCommands.go#L1796-L1824 and associated code.


Latest update from master

This is to get the updated packages

They haven't made a release since 2017 and the last commit on master in 2019, and omnipay/common is bumped up to 3 but it only works with their master branch

They haven't made a release since 2017 and the last commit on master in 2019, and omnipay/common is bumped up to 3 but it only works with their master branch


Send event when a Reorg happens

Background
It could be useful for some backends receiving tx service events to know when a reorg is detected on the chain
Proposed structure
{
  "type": "REORG_DETECTED",
  "blockNumber": "<integer>",
  "chainId": "<stringified-int>"
}

@Uxio0 Should the event being published when the REORG is detected or the REORG is fixed?
I could see several times that from REORG detected until fixed it takes time.

When Reorg is detected

Tested and verified! Thanks for the help testing it 😄


Fix broken links in Production Security Guidelines

Description:
Following link is broken in the docs[1]
Mitigating Cross Site Scripting Attacks: https://apim.docs.wso2.com/en/latest/administer/product-security/mitigating-cross-site-scripting-attacks/
[1] https://apim.docs.wso2.com/en/latest/install-and-setup/setup/deployment-best-practices/security-guidelines-for-production-deployment/
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

@tgtshanika , why have the following pages been removed from the 3.1.0 docs.

https://apim.docs.wso2.com/en/3.0.0/administer/product-security/mitigating-cross-site-request-forgery-attacks/

https://apim.docs.wso2.com/en/3.0.0/administer/product-security/mitigating-cross-site-scripting-attacks/

The XSS input validation valve is already enabled in latest releases of APIM (3.1.0) therefore, the following documents have been removed from APIM 3.1.0. Therefore, I will be removing this row with info in [3]. @tgtshanika , please let me know if you have any concerns.

[1] https://apim.docs.wso2.com/en/3.0.0/administer/product-security/mitigating-cross-site-request-forgery-attacks/
[2] https://apim.docs.wso2.com/en/3.0.0/administer/product-security/mitigating-cross-site-scripting-attacks/
[3] https://apim.docs.wso2.com/en/latest/install-and-setup/setup/deployment-best-practices/security-guidelines-for-production-deployment/


Update CommunityPlugins.ts

PR done as discussed here : https://github.com/patak-dev/vite-rollup-plugins/pull/20#issuecomment-1880928531
but new question : should we update the status ? Seems incorrect to me to call it Covered as the link provided is not a Vite feature or a Vite official plugin.

I think we should update the status info description, so Covered would mean that is provided by Vite or there is an equivalent plugin from the community. Or we could add a new status for these, but I'm having a hard time finding a good word for it. Equivalent? Community?. Feel free to update the wording here or make this PR ready and you could send another one if you'd like (or I'll get to it at some point)

I'm going to archive this resource. I think each rollup plugin should start documenting their compatibility with Vite in its own docs.

yep, I understand. Thanks for your work anyway !


Fix conflicts with default-branch-button

LINKED ISSUES:
Closes #3449

TEST URLS:
https://github.com/sindresorhus/refined-github/tree/incremental-tag-changelog-link/media?rgh-link-date=2020-08-10T10%3A00%3A55Z

SCREENSHOT:

Excellent! Love the bugfixes


Obscene Burn Damage Needs a Solution

Description

This recently happened on a Salamander shift where a sentient artifact managed to get inside the walls and spew tritium gas on demand. This turned into a major plasma fire that ended up killing a lot of the crew, most patients were at over 2k burn damage and rising shortly after the fire.
Reproduction

Start a plasma or tritium fire and stand within with minimal/no protection, you will crit rapidly and then continue to burn to death. After death your body temperature will likely stay absurdly high and continue to apply heat damage. This effectively round removes you from the game but also doesn't. Med usually can't deal with this without absurd amounts of necrosol or cloning.
Additional context

We were discussing this on Discord and suggested that excessive heat/shock damage turns a player to ash once exceeding a certain large value, such as over 500 heat. Additionally there needs to be a more effective mechanic to reduce body heat of an afflicted individual besides just dumping them into cyro which is extremely ineffective.

There is leporazine for an easy temp fix. That's probably the easiest. Other chemicals will cool you, but they're more finicky.

I don't think that completely dusting a player is a good option with high heat damage. just port husks from ss13 when you have over 300+ burn damage and require an unhusking cryo chem

not to mention 5k burn victims can be borged, ash just round removes them

Instead of ashing burn victims, why not reduce them to a skeleton? It's more comedic and would let you preserve the brain inside the skull (crack them open with the circular saw or something). I do hope we eventually get more gibbing alternatives, not just for burn damage but for other damage types too. Take enough radiation damage and you melt into a puddle of flesh and organs.

We have support for this in a way.
Destruction conditions are used to not spawn debris if something sustains massive damage at once. Presume you can set it per damage type for heat, radiation, etc and have them turn to goo. Preserving the brain and character might need something else though.

see also exterminator uses this to have special gib/burn damage and popups

The main thing I want to bring attention to this issue is that burn damage will functionally never stop and med simply can't deal with it.

Another fun example of this issue with everyone's favorite fox, but on the other end of the temperature spectrum.

If excessive heat turns your corpse to ash/husk, excessive cold should turn your corpse into an ice block
Both should be capped idk

- type: TransferMindOnGib
 - type: Destructible
   thresholds:
   - trigger:
       !type:DamageTypeTrigger
       damageType: Heat
       damage: 200
     behaviors:
     - !type:GibBehavior

Then you'll want to add a spawn behaviour in with that gib behaviour for your husk as well as this to your husk entity:
    tags:
    - MindTransferTarget```
Only issue is I'm not sure how it'll transfer all your stuff over.

So first thing is you can use the above for gibbing if you don't want them to be in the round anymore. But an actual solution would just be using mob visualiser states for being too burnt or frozen in a block of ice, and setting a limit to the amount of damage someone can take per type / subtype. So they cap out at like -250 damage or gib at that point. Depends what people want for balance whether they want it to gib or just cap.

I think capping the damage and having husk/ice block is preferable, since it offers a way to actually counterplay spaced bodies should you find them, or excessively burnt bodies, otherwise it just sucks.

This may or may not have been resolved by #26971


[DOC] Document cookie split configuration parameter

What do you want to do?

[x] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
We are shipping an update to our session cookies where we split them and stitch them back together to circumvent cookie size limitations. As part of this effort, we are creating a new configuration parameter to specify the number of "chunks" we allow. See details of the change (and question in particular about this behavior) here. We are targeting 2.7 for this change (hope it is small enough that we can fit the doc changes in, and sorry for the late notice!).
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

Monitoring for 2.7.


Changes to AKSequencer and AKMusicTrack: Set Time Signature, remove MetaEvents

Changes:

Added method for setting time signature to AKSequencer's tempo track
Added method for clearing existing time signature events from AKSequencer's tempo track
Added generalised method for iterating thru CoreMIDI MusicTracks to improve readability and prevent future code repetition (I use this in the clearTimeSignature methods, and intend to use it for future methods that involve iterating thru AKMusicTrack's internalTrack)
Made sure that AKSequencer's tempo track is wiped of any residual MetaEvents or ExtendedTempoEvents before a new MIDI file is loaded.

Thanks, keep 'em coming.


Nextjs - Optimized Light video preview using Image component or something like that

Current Behavior
Workin with Nextjs, light preview image is loading at the beginning of page load like a normal image but not caching or when it's on the viewport as expected using next/image.
Expected Behavior
Maybe add an Image component to boolean | string
Steps to Reproduce
Im not able to reproduce it sorry
https://nextjs.org/docs/basic-features/image-optimization

Yeah
I have the same issue, need to help with this.

I'd love to have this feature as well. Maybe the ability to add a custom component to the light prop. Something like:
light={<Image src={video.thumbnail_url} />}

Looks like this might be possible, just not documented: https://github.com/cookpete/react-player/blob/master/src/Preview.js#L62


FWREZNETxxx Modelling - publish interface not working for newly created interfaces

when creating an interface, checking the publish box and saving, the interface is not published.
can be fixed by editing and saving.

fixed in https://github.com/CactuseSecurity/firewall-orchestrator/pull/2739


Dynamic theme material you

Premise: I know that this feature is not strictly about components but is more about browsers but I have not been able to find any other information and this seems to me the closest place to the feature
Is your feature request related to a problem? Please describe.
Inability to use Android 12 dynamically generated colors in my web app
Describe the solution you'd like
I don't know if browsers currently expose (and/or plan to) CSS variables to access the dynamically created theme colors from Android. If not, the ideal would be to have CSS variables. It would then be great if the default values of the components were aligned with those of the system
Describe alternatives you've considered
If not system colors, it would be great if the components were at least aligned with the variables generated by the Material Theme Builder tool
Additional context
I tried the dynamic theme on my Pixel 6 Pro and I must say that the consistency and union between the various apps given by the theme greatly improves the experience and it would be a shame not to be able to extend it to websites and PWAs
PS:
If someone should be aware of issues or tickets, even of other projects (e.g. chrome, chromium, etc.), related to the feature the links would be welcome

Hi! This is a pretty broad topic for how we want to handle this on the web and coordinate with platforms like Chrome to handle dynamic color (aka Material You).
Dynamic theming is possible with the --md-sys-color-* custom properties, but how this API will look like for end users is still being discussed.
I'm going to leave this open to track dynamic color support on the web for Material 3 since it's slightly different than https://github.com/material-components/material-web/issues/3050 for M3 availability.

hey @asyncLiz, can you enable discussions in this repo? I feel this is more of a discussion than it's an issue.

Closing since discussions are enabled and we have basic theming docs


Look for non-deleted containers on packfile endpoint

Review Checklist

Tests were added to cover all code changes
Documentation was added / updated
Code and tests follow standards in CONTRIBUTING.md

Codecov Report

Merging #1090 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1090      +/-   ##
==========================================
+ Coverage    90.8%   90.81%   +<.01%
==========================================
  Files          50       50
  Lines        7029     7032       +3
==========================================
+ Hits         6383     6386       +3
  Misses        646      646