Add fuse cuts and selections

@HenningSE Thanks a lot!
Just a confirmation, are below correct?

Since we are using the clustered info, we are not merging the energy deposit outside TPC
Also, the evt_id is the EventID in Geant4

@HenningSE Thanks a lot! Just a confirmation, are below correct?

Since we are using the clustered info, we are not merging the energy deposit outside TPC
Also, the evt_id is the EventID in Geant4

Hi @MasatoshiKobayashi, yes both of them are correct.

As default only energy deposits in LXe are stored in Geant4. The clustering distance is quite small compared to the detector dimensions, so it should be more or less impossible to form a cluster of in and out of TPC interactions.
We renamed evtid  in one of the recent PRs to eventid to make it consistent throughout the full simulation. It is taken from the Geant4 input here.

Sure, thanks. then the energy selection part looks fine for me.
Are other files, like detector_volume or apply_selection are just copied?
I am not sure if I have to review also for them, or just energy cut part is fine.

Thanks Masatoshi, the detector volumes are a little different different with this PR to better integrate with the energy selection. I will ask @ramirezdiego to take a look at this part.

@HenningSE
Sorry, one more question:
the energy used for cut should be stored in truth as well. it looks there's already the total energy deposit of the event --https://github.com/XENONnT/fuse/blob/1dbaac1e03acb28ac92a103f5f240f95e6c6e0d1/fuse/plugins/truth_information/event_truth.py#L57.
what's the difference between this and the energy used for cut?

Hi @MasatoshiKobayashi these two energies are not identical and depend on the definition of an event. In EventTruth an event is what strax(en) considers to be one event. This must not always be the same what we have as an event from Geant4. The energy range cut in this PR uses the Geant4 event definition as we do not know at this stage how straxen will build the events.

Hi @HenningSE , thanks.

In EventTruth an event is what strax(en) considers to be one event. This must not always be the same what we have as an event from Geant4.
yes, this I  understood -- we may have event without s1 or s2 due to CIV, below cathode, misidentification etc.  However, once the event is recorded as an event, the truth energy should be more or less the same, no?

I am not the expert of fuse yet, but is "clustering" process also eliminates the cluster outside the sensitive volume as epix does? (just for confirmation... otherwise the energy cut includes the e_dep outside TPC, which we should not do.)

However, once the event is recorded as an event, the truth energy should be more or less the same, no?

This depends on the Geant4 simulation setting. In most cases I should be like this but you could also have e.g. a full U decay chain in one single Geant4 event.  This would then lead to multiple straxen events that are ages apart from each other.

I am not the expert of fuse yet, but is "clustering" process also eliminates the cluster outside the sensitive volume as epix does? otherwise the energy cut includes the e_dep outside TPC, which we should not do.)

Hmm good point. I think with the method that is in this PR right now The energy deposits outside would still be included in the energy cut. There I would need to change the plugins in a way that we first evaluate the detector volumes and then the energy cut.

@HenningSE
Thank you for the reply!!
Now I understood -- I was talking about the energy cut at the epix level (time and volume clustered) while it's not the case for fuse.
As you say, in that case we need to apply the cut after volume (and time if needed) selection.


Enable walking network and horse riding in Germany

There are several walking networks from the Netherlands that pass the German border.
Now these networks have problems with verification as the analysis said there are no walking networks in Germany.
Actually the networks of Twente and Vechtdal have already problems with the expected_rwn_route_relations tag. Several rwn_ref's are reported that they have a wrong number of routes but in fact they have the right number, but some routes are in Germany.
Also the network Bargerveen have problems
The  same problem will come in the very close future for horse riding networks.
The network of Twente will pass the German border and in the Grafschaft Bentheim there is a horse riding network already
Please enable walking networks and horse riding networks in Germany.
Thanks in advance
Regards,
Dick van den Hoven

The code has been updated to add support for walking networks and horse riding networks in Germany.


8.0.0

8.0.0

updated analyzer, source_gen, build, build_test dependency ranges
fix nnbd issues in generator
set sdk range lower bound to 2.12.0

https://travis-ci.org/github/davidmarne/built_redux


Lack of proper support for Facebook Canvas Apps?

The codebase is having some canvas app support (apparently inherited from django-social-auth), but it doesnt seem to be usable anymore. Or at least I am misunderstanding how it should be done :)
With regular oauth login buttons, button is redirecting to /login/<backend>, which usually redirects to oauth provider page, then return to /complete/<backend> and then redirects to / or whatever is configured.
With FB Canvas apps it looks like PSA is requiring /login/facebook-app and /complete/facebook-app to be called in proper moments, but FB dashboard is allowing to set only single canvas URL. So, if I understand correctly, I have following options:

Set canvas url to /login/facebook-app, and provide html template like the one in docs, with some redirect URI. But:
if the redirect URI is my canvas URL, I'm getting infinite login loop
I'm not sure if I can redirect to some URL other than my canvas URL (not checked yet), and trick FB to load that in its canvas iframe (after all I'm overriding window.top). Maybe yes.
Regardless, even if the above will work, I will have extra 2 redirects for users who are already authenticated and just want to re-visit my app. Not cool.
Set canvas_url to /complete/facebook-app:
this works well if user already allowed to share his info with the app, so complete might create user record and use it
but of course will work only once, and of course there is no place where user can share profile data with the app in the first place, since in this scenario nobody displays permission dialog
I can try to write my own backend (basically copied from FacebookAppOAuth2) and inside having some smarter complete which optionally tries to redirect to login, if necessary. Yet, after short code inspection it seems that it will be extremely hacky at best.
Provide my own /login/fb kind of view, which can check whether we need to ask for permission or not, and then call login or complete from social auth. Yet, that is extremely tricky, since we have POST data which basically means that we cannot redirect, or we lose the data. So, apparently importing the view methods from PSA django app and calling them directly would be the way...
Something else?

FB Canvas app and its associated POST is done in such way that if user is already authenticated, it allows app to continue with zero redirects. Just the response to initial POST will contain HTML for the specific user. I hope that with PSA it is possible to reach the same situation, with no redirects. If user had not shared personal info with the app yet, single redirect to permission dialog and back is unavoidable. In this case, however, PSA requires third redirect after /complete/.., it would be great to be able to avoid it as well.
Now, am I missing something obvious? Is there a way to get current code cooperate properly with canvas app? Or I'm first user in years who is trying to do so, only to discover that the code is badly outdated? Any hints appreciated ;)

+1

I've raised https://github.com/python-social-auth/social-core/issues/8 to properly track this issue in the new components.


Uncaught TypeError: Cannot read property 'offsetParent' of undefined when draggableColumns is enabled

Expected Behavior

Draggable columns can work properly
Current Behavior

Get error Uncaught TypeError: Cannot read property 'offsetParent' of undefined
Steps to Reproduce (for bugs)

Open https://dashboard-preview.web.app/timah, please wait until all charts and tables has been loaded.
Click inspect element, then open console tab.
Select table you want to drag its column, e.g. Tender Amount by Vendor.
Drag one column inside that table.
When that column is dragging, check the console that displays Uncaught TypeError: Cannot read property 'offsetParent' of undefined.

Your Environment

Tech
Version

@material-ui/core
4.10.2

mui-datatables
3.1.0

react
16.13.1

browser
Google Chrome on Linux (Ubuntu 20.04)

Yikes, that's a bad bug :/. It only happens when selectableRows: "none".  I've got a fix in development. I'm going to do a little more testing and hopefully push a 3.1.1 version a little later tonight.

thanks for your quick response, @patorjk .

No problem, thank you for the quick report! I've push out a 3.1.1 patch that has draggable columns working when selectableRows="none". If you see anything else please let me know.

I have updated mui-datatables to 3.1.1. I have forgot that the website use iframe. Drag and drop feature still doesn't work for me. Only display warning both [Violation] 'dragstart' and [Violation] 'drop' handler into console tab.

Can you show me an example? I can't seem to re-create this.

I'm aiming to do another patch release tonight. I'm not sure if this will solve your problem, but if you have a moment can you try this beta version:
npm install<EMAIL_ADDRESS>

sorry, that beta version still doesn't fix this issue.

Can you show me an example? I'm unable to reproduce the problem.

@patorjk
I think I need to pass window to use it in iframe
ref :

https://react-dnd.github.io/react-dnd/examples/dustbin/iframe
https://codesandbox.io/s/github/react-dnd/react-dnd/tree/gh-pages/examples_hooks_js/01-dustbin/single-target-in-iframe?from-embed=&file=/src/Container.jsx:335-354


WriteToCSV

The docs page for writetocsv shows write( workbook, filepath [, overwrite [, delimiter ] ] ) as the function call.
Also, would it be possible/easy to add a parameter that defaults true for include header row?  Sounds odd, but I have to generate files that cannot include a header row and takes time to remove them.  Not a huge deal if not, but would keep my code cleaner if possible.

Thanks for spotting the docs error, John. Fixed.
Happy to add includeHeaderRow as an option to writeToCsv().


"unused" option in uglifyjs webpack prod config behaves unexpectedly

Is this a bug report?
Yes
Steps to Reproduce

Install app
run dev server
build app
run build

Expected Behavior
Both the dev code & the built code should behave the same.
Actual Behavior
Some of the built code behaved incorrectly.
More specifically, some WebGL code generated with Emscripten did not behave as expected in the built code.
In my app, I expected to see this :

Instead, I saw this :

Setting the "unused" flag to "false" fixes this problem.
Demo
A demo app can be found at https://github.com/jslegers/luciadria-react-app.
Unfortunately, this demo requires LuciadRIA as a local dependency, and unfortunately I'm not allowed to share this code as it is proprietary. However, it should give an idea of what I'm trying to achieve.
Considering the behavior of the "unused" flag is unreliable, it makes sense to set it's default value to "false", like the "comparisons" flag.

Would it be more appropriate to file this with Uglify?

@gaeron :
I can't really tell which of the underlying components is responsible. Is it webpack-contrib/uglifyjs-webpack-plugin? Or maybe mishoo/UglifyJS2? I'm not sure how to tell.
I can, however, tell that setting the "unused" flag in the "webpack.config.prod.js" config file of "create-react-app" fixes the issue.
And considering setting the "unused" flag to "true" can actually break an app and this behavior is unpredictable, it seems reasonable to me to set the default to "false" for a boilerplate project like "create-react-app", as is already the case for the "comparisons" flag (see https://github.com/mishoo/UglifyJS2/issues/2011).
You don't agree?


[PREVIEW] Feature/snl 1809 internal ssl comms

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

No longer required


Task/api token prefix

Summary | Résumé
ADR to document implementing API token prefixes, as well as mechanisms to enforce them and verify they don't land in public repos.

Looks good to me and ready to merge!


Umlauts/encoding in the web version of the diagram converter

Describe the bug
When uploading a BPMN diagram with umlauts in the XML, the encoding is not set correctly. As a result the output contains question marks.

Steps to reproduce

Create a BPMN diagram with umlauts in the XML (e.g., as part of a label, documentation etc.).
Upload the BPMN file to the diagram converter.
Click on "Convert and download".
Open the result in the Camunda Desktop Modeler.

Expected Behaviour
The converted diagrams should be processed correctly, with no question marks replacing the umlauts.
Library version
0.6.2

The cause for this issue is the LANG env variable not being defined properly.

@hkupitz is fixed on live system, a patch will be made to build the image correctly by default


Fix RFC 2047 spaces encoded as underscores

Fixes MQpeng/eml-parse-js#3:

Hi, I have an email containing the subject line =?UTF-8?q?Off-The-Beaten-Path_Trails_You've_Never_Heard_Of!__=F0=9F=8C=8F?=. This is decoded by my email client (Mail.app on macOS), as well as by the npm library rfc2047 as

Off-The-Beaten-Path Trails You've Never Heard Of! 🌏

However, this library instead returns

Off-The-Beaten-Path_Trails_You've_Never_Heard_Of!__🌏

i.e. underscores are not successfully decoded into spaces. The relevant section of the standard is RFC-2047, section 4.2.2:

Encodings
4.2. The "Q" encoding
(2) The 8-bit hexadecimal value 20 (e.g., ISO-8859-1 SPACE) may be represented as "_" (underscore, ASCII 95.).  (This character may not pass through some internetwork mail gateways, but its use will greatly enhance readability of "Q" encoded data with mail readers that do not support this encoding.)  Note that the "_" always represents hexadecimal 20, even if the SPACE character occupies a different code position in the character set in use.

Fix
I believe a fix would involve updating mimeDecode as follows:
         if (chr === '=' && hex && /[\da-fA-F]{2}/.test(hex)) {
             buffer[bufferPos++] = parseInt(hex, 16);
             i += 2;
         }
+        else if (chr === '_') {
+            buffer[bufferPos++] = 20; // space character
+        }
         else {
             buffer[bufferPos++] = chr.charCodeAt(0);
         }

Thank you @MQpeng! When will the next NPM release be?

Thank you @MQpeng! When will the next NPM release be?
Thank you for PR @c-harding!
The pkg will be built and released by GitHub-CI.


Could not find the "brew" executable in /usr/local/bin or anywhere on the path for arm64(M1)

:speaking_head: Foreword
Tried the latest version 5.3.0 which support arm64 install, but got path issue for brew
:ghost: Brief Description
Tried the latest version 5.3.0 which support arm64 install, but got path issue for brew
:pancakes: Cookbook version
5.3.0
:woman_cook:  Chef-Infra Version
17.7.29
:tophat: Platform details
macOS Monterey Version 12.1
Mac mini(M1, 2020)
Steps To Reproduce
just tried the example in the cookbook:
homebrew_cask 'google-chrome'
homebrew_cask "Let's remove google-chrome" do
cask_name 'google-chrome'
install_cask false
action :remove
end
:police_car: Expected behavior
got this Error:
Running handlers:
[2021-12-22T20:34:33-08:00] ERROR: Running exception handlers
Running handlers complete
[2021-12-22T20:34:33-08:00] ERROR: Exception handlers complete
Infra Phase failed. 1 resources updated in 11 seconds
[2021-12-22T20:34:34-08:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
[2021-12-22T20:34:34-08:00] FATAL: ---------------------------------------------------------------------------------------
[2021-12-22T20:34:34-08:00] FATAL: PLEASE PROVIDE THE CONTENTS OF THE stacktrace.out FILE (above) IF YOU FILE A BUG REPORT
[2021-12-22T20:34:34-08:00] FATAL: ---------------------------------------------------------------------------------------
[2021-12-22T20:34:34-08:00] FATAL: Chef::Exceptions::CannotDetermineHomebrewOwner: Could not find the "brew" executable in /usr/local/bin or anywhere on the path.
:heavy_plus_sign: Additional context
Add any other context about the problem here. e.g. related issues or existing pull requests.

this issue is related to chef client 17.9.18

@chadjiang so how did you resolve the issue? Running into it myself using chef client 17.9.26

We're experiencing the same issue, using Chef client 17.9.26, though its release notes mention that it fixes exactly this issue (present in 17.9.18).
The release notes thank @jweyer8 for fixing it. What might we be doing wrong, Jared?

@RockLobster @chadjiang
Running Chef client 17.9.26 seems to have fixed the issue for us.
The homebrew_tap resource was trying to reference the 'full' property which was removed in v17.9.18 #12346. Below is the line that was causing the issue:
shell_out!("#{new_resource.homebrew_path} tap #{new_resource.full ? "--full" : ""} #{new_resource.tap_name} #{new_resource.url || ""}",
The fix was as simple as removing the reference to 'new_resource.full' which was causing the no method error due to the fact that the property was removed (The full property was removed by Chef because the --full option was depreciated by Homebrew).

@RockLobster @jweyer8 @phirk
Chef client 17.9.26 fixed --full issue.  but the issue i got before I cannot do brew installl package for ARM64 (m1) machine,  the brew installation path in the node is /opt/homebrew,
in the node , /opt/chef/embedded/lib/ruby/gems/3.0.0/gems/chef-17.9.26/lib/chef:
resource/homebrew_tap.rb
resource/homebrew_cask.rb
mixin/homebrew_user.rb.
provider/package/homebrew.rb
those file  use  /usr/local/Homebrew  for brew.   I have to update them in the node to make installation work.

@chadjiang
Have you tried specifying the install path with the homebrew_path property?
To use your example, would the following fix your issue:
homebrew_cask 'Install Chrome' do
cask_name 'google-chrome'
homebrew_path '/opt/homebrew/bin/brew'
end

@jweyer8
ha, good to know. I did not realize this :homebrew_path parameter. Thanks a lot!

@chadjiang
No problem!

oh, that looks interesting, will try on Monday if that saved our issue!

I needed to apply two changes:

even though homebrew got installed, it didn't get added to the path, leading homebrew_user.rb to fail.
I fixed it by adding the following lines to my default.rb:

include_recipe 'homebrew'
ENV['PATH']="#{ENV['PATH']}:/opt/homebrew/bin:/opt/homebrew/sbin"

Even though it's now part of the path, all homebrew_cask, homebrew_tap require setting homebrew_path.

Note: Just doing 2.) is not enough because homebrew_user.rb will still fail.

Weirdly enough, homebrew_package / package does not have a homebrew_path and is working just fine


Things (packets/models/ect)

Decided to leave the item/iteminstance change out of this, ill do it after we get more votes.

Idk merge time


React Suspense support

React's future fetcher/Suspense API will have ramifications for React Request. It's too early to say anything with certainty right now, but once that gets more fleshed out I should make a plan here.
At the end of the day, the resolution of this issue will likely be an update to the documentation.
Based on the future fetcher preview, my current thoughts are:

future fetcher does not implement deduping, so a wrapper or alternative solution may still be necessary. Maybe something like React Request could add that
future fetcher may conflate the two loading states ( #144 ). It the final API does this, then it may make sense for there to be a wrapper for it
I am pretty sure that the entire future fetcher demo can be polyfilled right now using the existing React API + render props. Once the API is more finalized, I might make a polyfill for it, and that polyfill may supply additional features (like deduping and splitting out loading states)
A Promise-based API may make more sense in the future than one that tries to map fetch's API. This is something I could do now, but I want to wait because future fetcher may implement the features differently/better. I don't want to write something just to rewrite it or throw it out in a few months 🙂

I'm closing this as revisit. We won't know more until React Suspense is closer to being finalized.
Feel free to leave your comments/ideas in this issue, though!

Reopening. Apollo's React Client has a solution that acts as a prop on their Query component. I can use that as motivation on how to implement it here.

Can you provide a link pointing at that?

I won’t have time to do that for you


Quick fix on TS80006 got not equivalent codes

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.72.0
OS Version: Windows 10

Steps to Reproduce:
Consider these codes:
function makeRequest() {
    try {
        const promise = fetchApi();
        // not async/await style, got indicates TS80006: This may be converted to an async function
        return promise.catch(error => {
            console.log('catch1');
            throw new Error('wrap error');
        });
    } catch (error) {
      console.warn('catch2: this line should never be reached');
      return Promise.resolve('wrong return');
    }
}

async function fetchApi() {
    throw new Error('network error')
}

By quick fix, we expect:
async function makeRequest() {
    try {
        const promise = fetchApi();
        return (async () => {
            try {
                return await promise;
            } catch (error) {
                console.log('catch1');
                throw new Error('wrap error');
            }
        })();
    } catch (error) {
      console.warn('catch2: this line should never be reached');
      return Promise.resolve('wrong return');
    }
}

But actually we got:
async function makeRequest() {
    try {
        const promise = fetchApi();
        try {
            return await promise;
        } catch (error) {
            console.log('catch1');
            throw new Error('wrap error');
        }
    } catch (error) {
      console.warn('catch2: this line should never be reached');
      return Promise.resolve('wrong return');
    }
}

The second catch statement will be executed incorrectly.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.72.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!


⚠ Bofrost.at has degraded performance

In a73a0a2, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2074 ms

Resolved: Bofrost.at performance has improved in 92852a3 after 16 minutes.


Rely on with-current-buffer in traad-open

I tried to run (traad-open default-directory) in my python-mode-hook
to automatically open projects, and found that it switched buffers to
the proc buffer. My elisp is a bit rusty but I think this PR fixes it.

Thanks! That's certainly an improvement.


Move Remove Symbolizer Button to Tablist in Symbolizer Window

Feature Request
Is your feature request related to a problem? Please describe.
The "Add Symbolizer" button is currently on the top row next to the tab list of symbolizers. The "Remove Symbolizer" button, however, is at the bottom of the input fields, which is often not directly visible and users have to scroll to find it.
Describe the solution you'd like
I think we should move the "Remove Symbolizer" button from to bottom to the actual tab item (just like it is done with browser tabs).

Describe alternatives you've considered
Additional context

Maybe we could also add an extra Tab with a + instead of the add button.


MYSQL Plugin

Summary
Adds a plugin for MySQL DB. Implementation is very similar to MariaDB plugin, except:

init_hook is changed to use mysqld --initialize-insecure
removed unnecessary symlink wrappers from flake.nix
renamed services

How was it tested?
devbox shell using the example in this PR, then:

devbox services up -- should show mysql + mysql_logs
devbox run mysql -- should give you a mysql console

Some thing weird happened with this, let me update and re-push


Update dependencies

merged manually - merge_all script failed in https://github.com/LD4P/sinopia_api/pull/51


External shoutcast v2 server with multiple sids

For some reason I'm unable to pull stats from an external shoutcast stream if I append the SID at the end of the url. For example,

http://serverip:port/index.html?sid=1 - AzuraCast doesn't pull stats for sid1. For some reason if I use http://serverip:port (without sid=1) it starts to work.

I want to be able to monitor the other streams on the same server so i figured the appropriate URL would be for the other two external stations

http://serverip:port/index.html?sid=2
http://serverip:port/index.html?sid=3

Any help would be great!
Thanks again.

@kbysfm Sorry for the delay in getting back to you.
This is definitely a valid issue, and is likely caused by the fact that we append ?sid=1 to any URL that doesn't have it already, so that we can get valid XML data behind the scenes.
I'll push an update to the code as soon as I get a chance. I've been busy with full-time work lately, and that's led to me having less time to devote to AzuraCast. :(

@SlvrEagle23 I decided to start from scratch and try the docker version of AzuraCast. For some reason when adding an external shoutcast v2 server I noticed that /index.html?sid=2 and /index.html?sid=3 do not play music or show stats. The base url http://serverip:port works fine as advertised.
Thanks for hard work!


'Request ID' & 'Status of Request' should be displayed in ‘Registered Requests’ Page

Subject of the issue
'Request ID' & 'Status of Request' should be displayed in ‘Registered Requests’ Page .
"Request ID" column helps to identify, report & clean repeated/redundant issue details
"Status" column (OPEN, INPROGRESS& CLOSED).) helps the public to address ‘OPEN’ issues .
So please update ‘Registered Requests’ page with  ‘Request ID’ & ‘Status’
Your environment

version of your python:Python 3.7.0
which browser and its version : Chrome 68.0.3440.106
OS Type (iOS/Android/Linux/Windows): WINDOWS 10

Steps to reproduce

Enter following URL  Browser : https://keralarescue.in/

Navigate to ‘Registered Requests’ Screen

Display in contents with Necessary info of Registered Requests
https://keralarescue.in/requests/?district=

Expected behaviour
Registered Requests’ Screen  will be displayed with  “request ID” & “Status” of request  along with “District”, “location” ,”Requestee”, “Phone” &”Date” .
Actual behaviour
“Registered  Requests” Screen has District, location ,Requestee, Phone & date .
But it does not have unique “Request ID” & “Status” of request(OPEN, INPROGRESS& CLOSED).
More Details
Request ID helps to identify, report & clean repeated/redundant issue details
Status helps the public to address ‘OPEN’ issues .
So please update ‘Registered Requests’ page with  ‘Request ID’ & ‘Status’

Sir though i am not much experienced with contributing in open source , but i think we need to add new request id and status in the original schema.?
Can you tell me how to progress with it?

@AkkoliPottammal Is there a mechanism by which we know if the request is served (CLOSED) or is being served (INPROGRESS)? If not, then we'll need to have that first in place and then inclusion of Status column on the requests page shall make sense. Thoughts?

@AdityaGiri11 Start with the project setup if not completed already.

@AkkoliPottammal Sorry went through the code now. It seems it can be implemented

while setting up in the step 4 , while running the command it shows error. Can you help me with it?

@AkkoliPottammal Raised a PR. Can you please review #568  and test it?

python3 manage.py makemigrations --merge
python3 manage.py migrate
these command worked for me.
Now i am looking forward to test and help with whatever i can.
Thank you.

@AdityaGiri11 Can you please review the PR #568 and test it on your local machine?
It has the changes requested in this issue #549

@darkdefender27 Sure


Dedupe multiple destinations on key lookups after failed proxy

Destinations were not deduped properly after a proxied request failed and thus would not be retried even if the lookup resulted in a single destination (w/ the same value)
@markyen @Raynos

LGTM

lgtm.


Document adding no_args_is_help to Command for 7.1

In #1167, a parameter was added to the Command class, changing the documentation accordingly. However, it was not specified that it was a change for an unreleased version. This change indicates that the no_args_is_help parameter is added in 7.1.
Tests are passing. Closes #1259.

Sorry for the confusion that this generated!
This looks good. Merging.


Rule0015PermissionSetCoverage runtime error

Hi, I've got this error, and I don't know why!:
[{
"resource": "",
"owner": "generated_diagnostic_collection_name#0",
"code": "AD0001",
"severity": 4,
"message": "Analyzer 'BusinessCentral.LinterCop.Design.Rule0015PermissionSetCoverage' threw an exception of type 'System.MissingFieldException' with message 'System.MissingFieldException: Field not found: 'Microsoft.Dynamics.Nav.Analyzers.Common.Constants.PermissionNodeXPath'.\r\n   at BusinessCentral.LinterCop.Design.Rule0015PermissionSetCoverage.XmlPermissionExistsForObject(IEnumerable1 permissionSetDocuments, PermissionObjectKind objectType, Int32 objectId)\r\n   at BusinessCentral.LinterCop.Design.Rule0015PermissionSetCoverage.CheckPermissionSetCoverage(SymbolAnalysisContext context)\r\n   at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass49_2.<ExecuteSymbolActionsCore>b__0() in D:\\a\\_work\\1\\s\\source\\Prod\\Microsoft.Dynamics.Nav.CodeAnalysis\\DiagnosticAnalyzer\\AnalyzerExecutor.cs:line 658\r\n   at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable1 info) in D:\a\_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1102'",
"source": "AL",
"startLineNumber": 1,
"startColumn": 1,
"endLineNumber": 1,
"endColumn": 1
}]

which version of the AL Language extension are you using?
Is it possible to reproduce this in a minimal al project?

@f4n0, we need more details to try to reproduce this error. Do you have the requested information that @StefanMaron had mentioned here?

sorry for ghosting this issue!
unfortunately, I wasn't able to reproduce this issue in any way!
I suppose there is something strange in my code!
If I'm the only one with this, you could close this issue until further notice!

Thank you for sharing the update and good to know the issue seems to be resolved.


Optimization idea: avoid copy-on-write if all other refs are never read

Not sure how to phrase the issue title, but the core of the issue is that performance in some cases could be greatly improved if large data are not copied if they will never be read from. I suppose usually that situation would be a developer mistake, but bear with me 🙂
For example, insert is practically instantaneous on a large map. But if there is another reference laying around, then it makes a copy of it (slow) before inserting (so that the old references do not get modified). If the old ref is never looked at (which static analysis should be able to determine) then the copy operation was wasteful.
Example
# Experimental!

map ⊃⇡⊚ 1e7
&p ⍜now insert 0 ⚂    # modify in place
&p ⍜now insert 0 ⚂ .  # make modified copy
&p ⍜now insert 0 ⚂ ⊙◌ # modify in place
&p ⍜now insert 0 ⚂ .  # make modified copy
◌◌

Here's a simple demo, but I can provide a recursive example too where it would be impossible to eliminate "extra" references without changing the algorithm to be iterative.

Can you provide this?

I updated the main post with a recursive example.

Sorry, do you have a minimal example?
Also, in case you are not aware, there is a memo function.

Well, 2 things.

When trying to create a minimal recursive example, it seems it... works fine 🎉. I probably should have profiled before filing the issue (are there docs for profiling?); sorry for the bother about that. I've modified the main post to not mention recursion. What remains is philosophical, I guess.
I did not know about memo, that's awesome, thanks!


Add convert support for Vitest

Adds support for Vitest in the convert command.

@ynahmany Let me know if there is anything else 🙂


Error when comments in the middle of a command MOVE.

Hi guys,
I have this reproduceable issue where the cleaner cannot handle code that has comments in the middle, similar to https://github.com/SAP/abap-cleaner/issues/88
this works:
REPORT zvzxphu_1.
DATA lv_werks TYPE werks_d.
MOVE 5 TO
lv_werks.

this does not:
REPORT zvzxphu_1.
DATA lv_werks TYPE werks_d.
MOVE 5 TO "test
lv_werks.

error in the log reads:
20231013_165445 , line 12: Failed referential integrity test on command starting at source line 12!

Hi openPhil,
thanks for reporting this! Yes, comments, pragmas or colons in unexpected places are annoying… but this should be fixed in the next release.
To identify more of these issues, I now implemented a "stress test" for both the unit tests of ABAP cleaner and the >30 MB of real-life ABAP code on which I run ABAP cleaner before every release. With the stress test, each code document is now additionally tested 128 times, and every time,

either a) " line-end comments, or b) * comment lines, or c) ## pragmas, or d) : colons are introduced after
each 1st / 2nd / 3rd / … 32nd token (if syntactically allowed) of each ABAP statement.

Then all cleanup rules are executed on the code, and the test checks whether referential integrity is kept. So you could say, the stress test now cleans approx. 4000 MB of code with lots of comments, pragmas, and colons in all sorts of weird places.
This did find a few more issues which will all be fixed in the next release; and the stress test will of course check regression for further development efforts.
Kind regards,
Jörg-Michael

Hi,
I think this is a great approach to generalize the issue!
One thought is left though.. Your approach (at least what I understand) assumes that it is enough to have 1 (as in single / as in once) element of 1. for each of 2.
technically, we can have them all combined.
concatenate ##needed
*put advertisement here:
lv_a ##needed lv_b into: ##needed "really
data(lv_c), data(lv_x).

(this beautiful code doesn't break anything (yet), it is just to explain me :)
There might be edge cases that have multiple pragmas and comments combined in some lines.

Hi openPhiL,
this should now be fixed with version 1.7.2, which was just released!
And you're right, it would be a nice enhancement of the stress test to introduce multiple elements into the same statement. Of course, to some extent, this already happens, because the element added by the stress test comes on top of what comments, pragmas, and colons are already there. But I'll think of something here! Since there are too many possible combinations, there may be a stress test variant that adds multiple elements in random places (just have to think how to persist changes in case of errors in order to make them reproducible).
Kind regards,
Jörg-Michael
P.S.: I absolutely like *put advertisement here 😉: code is indeed among the very few places in our lives that are still advertisement-free, so there's certainly great potential here!


Opentracing 1.6.0 compatibility

Hi. It looks like v0.1.1 of lua-bridge-tracer doesn't work with opentracing-cpp 1.6.0.
It's been a while since I've done any C++ coding, but it looks like by just implement the 3 missing methods it compiles and works OK. I've updated and tested the hello-world example and the CI pipeline with lua 5.1, 5.2 and 5.3.
As jaeger-client-cpp still doesn't have a release with Opentracing 1.6.0 compatibility I had to compile the master branch manually in the hello-world example. This slows down the Docker container build a fair bit...

Thank you for doing this, I ran into the same problem. The PR addressed my issues as it allowed 1.6.0 to build the library.


Add in-person intervention for SMS sign-up/notifications

CUTGroup testers might have trouble with text messaging or the questions asked. I would like there to be a way for testers to text "HELP" during the conversation chain either when signing up or when notified of a test/screening questions. Once a tester texts "HELP" the automatic Twilio conversation will end and I will receive a notification (preferably by email) that lets me know where in the Twilio conversation the tester texted help. I would then like to be able to text back from the same Twilio phone number and continue a more personal conversation with the tester.
This does relate to #93 but I see it being more useful when SMS test notification system is implemented.

We will have to use something different from "HELP." HELP is a keyword used by Twilio to respond with unsubscribe information


Planner sends unnecessary plan updates to controlplane nodes when utility operations are performed.

When taking an etcd snapshot, the planner will trigger a reconciliation of all controlplane nodes as the cluster spec will technically change.
We can fix this by simply not storing utility fields when we save a copy of the cluster spec, as these are never used in etcd restore anyway.

The easiest way to test this is to provision a cluster with a dedicated controlplane node, perform an etcd snapshot create, and observe that the plan on the dedicated controlplane node does not change during this procedure.

On Rancher v2.6-head 42ef7aea7415212d4b96c73fdff33766011b6f93:

Provisioning a cluster with 3 etcd, 2cp, and 2 worker nodes.
Took an ectd snapshot.
Observed that the control plane machines changed status to Reconciling indicating that the plan did change.
SSH'd to the nodes to see that multiple plans were applied by rancher-system-agent

On Rancher v2.6-head cac23eae92ec00c93fcaf3e44f53cafa44453bec:

Provisioning a cluster with 3 etcd, 2cp, and 2 worker nodes.
Took an ectd snapshot.
Observed that the control plane machines never changed status indicating that the plan did NOT change
SSH'd to the nodes to see that only one plan was applied by rancher-system-agent


kubernetes/azure-cloud-provider: Enable annotation to specify subnet to create Azure load-balancer

Is this a request for help?: yes

Is this an ISSUE or FEATURE REQUEST? (choose one): Feature

What version of acs-engine?: All

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm) Kubernetes
What happened:
Currently all Azure Load balancers are provisioned in the same subnet as the master VMs.
What you expected to happen:
Allow an annotation to specify which subnet you wish the Azure loadbalancer to be created in
How to reproduce it (as minimally and precisely as possible):
Create a Kubernetes service resource of type load balancer on a cluster that has master and agent nodes in different subnets. The Azure LB will be provisioned in the master subnet
Anything else we need to know:

cc @itowlson @slack @jdumars

I believe there is a further restriction that LBs can only be provisioned in the primaryAvailabilitySet which is hard-coded in the cloud provider config. Would be nice to tackle that restriction at the same time.

@slack We may have to - I've got the subnet annotation working, but if I try to deploy two internal LBs on different subnets, I get errors from Azure about "Network interface [nic_id] references more than one load balancer of the same type (internal or public): ... Only one internal and one public load balancer are allowed per availability set."  So if we don't support multiple availability sets then the feature is limited to to "you can control which subnet the internal LB is in but you can't have internal LBs in different subnets."  @lachie83 I assume that isn't an acceptable limitation?

@slack It looks like there is a problem with availability sets anyway - if I have multiple agent pools, and create a deployment on the secondary agent pool, I can't expose that as a service.  It gets assigned an IP address and port on the load balancer, but the LB's backend pool is the primary agent pool, so requests to the LB never get routed to the deployment.  It seems like we should be allowing one internal and one external LB per agent pool (we can't allow more because of Azure restrictions it seems): the front end IPs can be controlled as to subnet, but controlling that at the service level feels dubious -- if I move the agentpool1 internal LB from subnet1 to subnet2 then that affects all services in agentpool1.  cc @lachie83 @jdumars - do we need a wider discussion about the desired behaviour here?


Do not construct a permutation in createFromSends if sending only to the same rank

One build did not build container, otherwise good.


Create qodana.yml

:white_check_mark: Build bitcointrade-sdk-dotnet 1.0.249 completed (commit https://github.com/guibranco/bitcointrade-sdk-dotnet/commit/8c64acb10f by @guibranco)

:white_check_mark: Build bitcointrade-sdk-dotnet 1.0.252 completed (commit https://github.com/guibranco/bitcointrade-sdk-dotnet/commit/8c64acb10f by @guibranco)


Add note about is_int portability.

Reproduced in zsh 5.7.1-1 under a Debian 10.1.0-amd64 install, (though updated recently).

I was able to reproduce.
@spiralofhope is you sh not symlinked to zsh? Here is how I reproduced:
$ echo 'printf %d "$1" >/dev/null 2>&1' >test.sh
$ sh ./test.sh not_number
! # <-- correct exit code
$ ln -s $(which zsh) ./sh
$ ./sh ./test.sh "not_number"
$ # exit code of 0

You noted an exit code of !, did you mean 1 ?
My sh is symlinked to dash <IP_ADDRESS>-5
echo 'printf %d "$1" >/dev/null 2>&1' >test.sh
# Actually dash:
sh ./test.sh not_number ; echo $?
1
ln -s $(which zsh) ./sh
./sh ./test.sh "not_number"; echo $?
0

At no point in my tests does the printf display anything.

oh yeah my bad - the error I was trying to highlight was the exit code, I don't print anything either, of course.


Error: could not find a ready tiller pod

Output of helm version:
Client: &version.Version{SemVer:"v2.10.0-rc.3", GitCommit:"9ad53aac42165a5fadc6c87be0dea6b115f93090", GitTreeState:"clean"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.2", GitCommit:"bb9ffb1654d4a729bb4cec18ff088eacc153c239", GitTreeState:"clean", BuildDate:"2018-08-07T23:17:28Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.2", GitCommit:"bb9ffb1654d4a729bb4cec18ff088eacc153c239", GitTreeState:"clean", BuildDate:"2018-08-07T23:08:19Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}
OS version:
CentOS 7.4
Issue:
My tiller pod is pending forever:
NAME                                           READY     STATUS    RESTARTS   AGE
tiller-deploy-6fd7f5b55-vm4rj                  0/1       Pending   0          18m
There is no log in the pod...and I can't figure it out how to debug it...

I found this:
● kubelet.service - kubelet: The Kubernetes Node Agent
Loaded: loaded (/etc/systemd/system/kubelet.service; enabled; vendor preset: disabled)
Drop-In: /etc/systemd/system/kubelet.service.d
└─10-kubeadm.conf
Active: active (running) since Qua 2018-08-22 16:24:46 -03; 1h 0min ago
Docs: https://kubernetes.io/docs/
Main PID: 5424 (kubelet)
CGroup: /system.slice/kubelet.service
└─5424 /usr/bin/kubelet --bootstrap-kubeconfig=/etc/kubernetes/bootstrap-kubelet.conf --kubeconfig=/etc/kubernetes/kubelet.conf --config=/var/lib/kubelet/config.yaml --cgroup-driver=systemd --cni-bin-dir=/opt/cni/bin --cni-conf-dir=/etc/cni/net.d --network-plugin=cni
Ago 22 17:24:51 s3797.corenmg.gov.br kubelet[5424]: E0822 17:24:51.845317    5424 summary.go:102] Failed to get system container stats for "/system.slice/kubelet.service": failed to get cgroup stats for "/system.slice/kubelet.service": failed to get container info for "/system.slice/kubelet.service": unknown container "/system.slice/kubelet.service"
Ago 22 17:24:51 s3797.corenmg.gov.br kubelet[5424]: E0822 17:24:51.845352    5424 summary.go:102] Failed to get system container stats for "/system.slice/docker.service": failed to get cgroup stats for "/system.slice/docker.service": failed to get container info for "/system.slice/docker.service": unknown container "/system.slice/docker.service"
Ago 22 17:25:01 s3797.corenmg.gov.br kubelet[5424]: E0822 17:25:01.865406    5424 summary.go:102] Failed to get system container stats for "/system.slice/kubelet.service": failed to get cgroup stats for "/system.slice/kubelet.service": failed to get container info for "/system.slice/kubelet.service": unknown container "/system.slice/kubelet.service"
Ago 22 17:25:01 s3797.corenmg.gov.br kubelet[5424]: E0822 17:25:01.865448    5424 summary.go:102] Failed to get system container stats for "/system.slice/docker.service": failed to get cgroup stats for "/system.slice/docker.service": failed to get container info for "/system.slice/docker.service": unknown container "/system.slice/docker.service"
Ago 22 17:25:11 s3797.corenmg.gov.br kubelet[5424]: E0822 17:25:11.883351    5424 summary.go:102] Failed to get system container stats for "/system.slice/docker.service": failed to get cgroup stats for "/system.slice/docker.service": failed to get container info for "/system.slice/docker.service": unknown container "/system.slice/docker.service"
Ago 22 17:25:11 s3797.corenmg.gov.br kubelet[5424]: E0822 17:25:11.885240    5424 summary.go:102] Failed to get system container stats for "/system.slice/kubelet.service": failed to get cgroup stats for "/system.slice/kubelet.service": failed to get container info for "/system.slice/kubelet.service": unknown container "/system.slice/kubelet.service"
Ago 22 17:25:21 s3797.corenmg.gov.br kubelet[5424]: E0822 17:25:21.899992    5424 summary.go:102] Failed to get system container stats for "/system.slice/docker.service": failed to get cgroup stats for "/system.slice/docker.service": failed to get container info for "/system.slice/docker.service": unknown container "/system.slice/docker.service"
Ago 22 17:25:21 s3797.corenmg.gov.br kubelet[5424]: E0822 17:25:21.900900    5424 summary.go:102] Failed to get system container stats for "/system.slice/kubelet.service": failed to get cgroup stats for "/system.slice/kubelet.service": failed to get container info for "/system.slice/kubelet.service": unknown container "/system.slice/kubelet.service"
Ago 22 17:25:31 s3797.corenmg.gov.br kubelet[5424]: E0822 17:25:31.917327    5424 summary.go:102] Failed to get system container stats for "/system.slice/kubelet.service": failed to get cgroup stats for "/system.slice/kubelet.service": failed to get container info for "/system.slice/kubelet.service": unknown container "/system.slice/kubelet.service"
Ago 22 17:25:31 s3797.corenmg.gov.br kubelet[5424]: E0822 17:25:31.917367    5424 summary.go:102] Failed to get system container stats for "/system.slice/docker.service": failed to get cgroup stats for "/system.slice/docker.service": failed to get container info for "/system.slice/docker.service": unknown container "/system.slice/docker.service"

is that node operational?

The messages in log seems to be related with https://github.com/kubernetes/kubernetes/issues/56850#issue-279360383

Try this command, tiller pod came to running state when i executed this command.
kubectl taint nodes --all node-role.kubernetes.io/master-

Having the same error on a fresh machine, opened a question about here.


Expander Header Stretch is not Exposed

Describe the bug
This bug was inherited from the Toolkit style, see our issue describing the problem here: https://github.com/windows-toolkit/WindowsCommunityToolkit/issues/3070 This should be resolved before Expander ships stable.
We found in general that having the ToggleButton within the Expander control made re-styling the control very difficult for our developers. I had missed this on the original PR, I would have suggested exploring an alternative approach...
Expected behavior
Able to stretch the content of an Expander's Header
Additional context
Also there's a duplicate of ExpanderHeaderUpStyle both in the Theme Resources and the main Style for Expander.

@ranjeshj FYI

🦙 Also see #4391 and #4390 which should be resolved before shipping stable as well.


Add support for log formatters in Tesla Middleware

There is a single fixed log format for the Tesla.Middleware.MetaLogger (and in fact not well-documented).
Request
[{tag}] {HTTP method} {URL} {headers}
[{tag}] {body}

Response:
[{tag}] {HTTP status code} {headers}
[{tag}] {body}

Example:
[MyApp.Client.Http] POST https://test.xxx.com/2.4/itineraries?token=123pim567pim&param1=value1 [{"Authorization", "[FILTERED]"}, {"Customer-Ip", "<IP_ADDRESS>"}, {"Accept", "application/json"}, {"Accept-Encoding", "gzip"}, {"Content-Type", "application/json"}, {"User-Agent", "BoFH.Providers.EPS/0.83.0"}
[MyApp.Client.Http] {"some":"body","work":"my body","customer_ip":"<IP_ADDRESS>","sensitive":"[FILTERED]"}
[MyApp.Client.Http] 201 [{"content-type", "application/json"}, {"vary", "Accept-Encoding,Accept-Encoding"}, {"location", "/path/7657854242219?token=567pim123pim"}, {"content-encoding", "gzip"}, {"content-length", "1720"}, {"date", "Thu, 25 Nov 2021 09:22:57 GMT"}, {"connection", "keep-alive"}]
[MyApp.Client.Http] {"some_id":"7657854242219","links":{"test":{"method":"GET","href":"/path/7657854242219?token=567pim123pim"}}}

Supporting log formatters, the default logger formatter could be the current one and open the flexibility to have as many as we want. For example, a log formatter for Datadog JSON or a curl output for the request.

And maybe could be a good idea to move the Tesla.Middleware.MetaLogger to a separate library.

And maybe could be a good idea to move the Tesla.Middleware.MetaLogger to a separate library. 💡

I think the code is too small and simple to be extracted now, not worth the effort of a major release.


Add environment variable names to starctl options

We currently document the command line options for starctl [1], but need the environment variable equivalents. E.g. --dc is DATACENTER_NAME.
[1] https://stargate.io/docs/stargate/0.1/developers-guide/starctl.html

I think most value would be for docker compose usage, so possibly close to docker images.
IMO This would be most convenient for the reader.

Maybe we need a separate topic on using docker compose.


VersionError

hi,
with version 1.17 i get Docker::Error::VersionError: Expected API Version 1.15
if i downgrade the docker-api gem to 1.15, i get Docker::Error::VersionError: Expected API Version 1.12
can you help me?
btw. im on mac osx with boot2docker version 1.4.1, ruby 2.1.1 and rails 4.2.0

Sorry about the confusing error message.  I think docker is complaining that the endpoint version that our gem is hitting, is behind the latest version supported by docker.  The error seems to imply that you should change the version of docker-api, but that's not the case!  I haven't seen this error yet myself, can you provide example code that'll hit this error?  Also, what is the output of docker version for the server?

i created a new rails app, added your gem and called Docker.validate_version! in the console.
all other methods get a timeout.

Have a look on this https://github.com/swipely/docker-api/pull/249


Provide C++20 branch with simpler comparison

prepare for Better Software talk.

Unfortunately, comparison gets almost more complicated in C++20... since defaulting doesn't work.
But everything (including some concepts) is available in the c++20 branch.

Unfortunately, comparison gets almost more complicated in C++20... since defaulting doesn't work.
But everything (including some concepts) is available in the c++20 branch.


Support predefined goodvoxels mask in init_bold_fsLR_resampling_wf

What would you like to see added in fMRIPrep?
I am currently using fMRIPrep as a dependency for ASLPrep, and, while the currently version of init_bold_fsLR_resampling_wf allows me to convert preprocessed ASL files to CIFTIs, I can't use the goodvoxels mask generated from the ASL file when converting derived CBF maps to CIFTIs.
It might even be worth it to move init_bold_fsLR_resampling_wf to niworkflows, since it's useful beyond just BOLD data.
Do you have any interest in helping implement the feature?
Yes
Additional information / screenshots
This might be useful for PET as well (I'm not sure what PET processing looks like).

Yes - there will be a PET use for resampling volumes to fsLR grayordinates. But no wrinkles :-)


Stage Variables

Describe your problem here.
How do you find if a song's stage is a pixel stage in a lua script?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Nah

getProperty('isPixelStage')

getPropertyFromClass('PlayState','isPixelStage')


QA Shares all Works in System to Users

When I select the Works Shared with Me tab, I see 176 pages of works shared with me. I'm not that popular.

Verify if this is admin only or if regular users also see all items in the shared.  It is either a nice to have if it is admin only, or a blocker if all users see it.


Exclude classes/modules in Sorbet::Private:: from hidden.rbi.

Motivation
Resolves #1488. This adds a filter for classes and modules in the Sorbet::Private namespace, so they don't show up in hidden.rbi.
Test plan
See included automated tests.

I think I've fixed the issue that was causing the test failures.


rename ModList.txt.new ModList.txt: Access is denied.

Replace ModList.txt instead of renaming

Fixed in v0.1.5


How to use component according to async data?

There are some trouble to me, when I use component after async data ready.
Say that, I fetch data in router's data hook, and need use other components according to the data I fetched. Before I fetched the data, I don't know which component need I use, or what components orders.
so, I've no idea what the template is, until I fetched the data. And I need push component and it's params to the template according to the data.
How should I do?
Thanks for answering!

Please do not use issues to ask questions. Use either Gitter or the forum.

got it, sorry.


Test Publishing 5.0 Capabilities

Changes Proposed
Adds tests for Publishing 5.0 Capabilities

compute file actions

Checklist

[ ] unit tests added and/or verified that tests pass
[ ] I have considered any possible security implications of this change
[ ] I have considered deployment issues.

merging without review as this has only test cases


iOS Safe zone support for new Material Design

The last Podfile had safe zone support but this one don't.

Look @alexandruantonica

{
"description": "NativeScript Application",
"license": "SEE LICENSE IN ",
"readme": "NativeScript Application",
"repository": "",
"nativescript": {
"id": "org.nativescript.NewFuckingApp",
"tns-ios": {
"version": "3.4.1"
}
},
"dependencies": {
"@angular/animations": "~5.2.0",
"@angular/common": "~5.2.0",
"@angular/compiler": "~5.2.0",
"@angular/core": "~5.2.0",
"@angular/forms": "~5.2.0",
"@angular/http": "~5.2.0",
"@angular/platform-browser": "~5.2.0",
"@angular/platform-browser-dynamic": "~5.2.0",
"@angular/router": "~5.2.0",
"nativescript-angular": "~5.2.0",
"nativescript-bottom-navigation": "^1.4.1",
"nativescript-theme-core": "~1.0.4",
"reflect-metadata": "~0.1.10",
"rxjs": "~5.5.5",
"tns-core-modules": "3.4.1",
"zone.js": "~0.8.18"
},
"devDependencies": {
"nativescript-dev-typescript": "~0.6.0",
"nativescript-dev-webpack": "~0.9.1",
"typescript": "~2.6.2",
"webpack": "~3.10.0",
"webpack-bundle-analyzer": "^2.9.1",
"webpack-sources": "~1.1.0",
"copy-webpack-plugin": "~4.3.0",
"raw-loader": "~0.5.1",
"css-loader": "~0.28.7",
"nativescript-worker-loader": "~0.8.1",
"resolve-url-loader": "~2.2.1",
"extract-text-webpack-plugin": "~3.0.2",
"uglifyjs-webpack-plugin": "~1.1.6",
"@angular/compiler-cli": "~5.2.0",
"@ngtools/webpack": "~1.9.4"
}
}

update your tns-ios version to 4.2.0


Unexpected error: unable to parse target intrinsic snippet: CheckAccessFullyMapped

I see that CheckAccessFullyMapped was added to stdlib in #1712, but I'm getting this when attempting to use it in the shader below:
hlsl.meta.slang(6600): error 29000: unable to parse target intrinsic snippet: CheckAccessFullyMapped
hlsl.meta.slang(6600): error 29000: unable to parse target intrinsic snippet: CheckAccessFullyMapped
hlsl.meta.slang(6600): error 29000: unable to parse target intrinsic snippet: CheckAccessFullyMapped

This syntax is accepted by DXC and seems like it should work. I looked at hlsl.meta.slang and all the types seem to line up.
Shader
SamplerState      gSampler  : register(s5);
Texture2D<float4> t         : register(t1);

float4 main(int2 offset: A) : SV_Target {
    uint status;
    float clamp;
    float4 val = t.Sample(gSampler, float2(0.1, 0.2), 1, clamp, status);

    bool success = CheckAccessFullyMapped(status);
    return success ? (float4)1 : (float4)0;
}

HLSL spec can be found from the following URL,
https://learn.microsoft.com/en-us/windows/win32/direct3dhlsl/checkaccessfullymapped
bool CheckAccessFullyMapped(
  in uint_only status
);


Add support for fingerprint login

This is a port of https://github.com/kercre123/ly, hence he is added as co-author.
This PR adds support for fingerprint login, and fixes #227.
I tested this in a single-user setup, by patching this into the arch package. logging in as a non-default user was not tested.

There turned out to be a couple of issues, one was a fast blinking cursor, which has not been fixed, and one issue where after logging in using fingerprint auth, then logging out, logging back in results in an inability to start sway. I tried reproducing the normal auth flow (not pushed) but nothing really fixes. I might have some time to fix this, but some help would be appreciated!

I fixed the issue! I also split the normal auth flow in 2, just like the fingerprint auth flow. This removes a lot of duplicate code, and keeps the 2 flows very much similar.

I'm sorry, github does not seem to be sending me emails about anything anymore, im subscribed to this thread but I just didn't get any emails!! (I even searched all my emails, nothing).

@JAicewizard

This is true, but I think that restarting a PAM login every few minutes is not so bad. We could also increase the timeout to an hour so that there should be basically no wasted cycles (without removing the feature)

I'm not a huge fan of adding such a long artificial timeout, plus, with the link you provided, wouldn't this only apply to the pam_fprintd.so module?

We are also not doing any expensive computations, the most power-hungry part is probably activating the fingerprint reader, but that is simply the cost of the user enabling it, not something we can do anything about

Yes, but think about a laptop for example: even if it's not doing anything expensive, if even just one core is pegged at 100%, the fans may spin really fast, which makes you wonder what Ly is even doing in the background... and it's not great.

I don't feel like putting in the effort if you do not want the feature, but if you have the intention of merging this if it gets into a better state I will clean it up.

I do, in fact, want to merge this PR! So if you can improve this PR, then do it! :D

Yes, but think about a laptop for example: even if it's not doing anything expensive, if even just one core is pegged at 100%, the fans may spin really fast, which makes you wonder what Ly is even doing in the background... and it's not great.

I understand the concern, but while in PAM land the CPU should be idle. pam_fprintd.so offloads to fprintd which waits for a finger to be detected and only then starts to actually perform any computations. This waiting is done using some kind of low-level communication with the reader, fprind shows 0% usage when waiting for a finger for me, I hope that alleviates these concerns.

I do, in fact, want to merge this PR! So if you can improve this PR, then do it! :D

Then I will start working on them when I have time :) I will also do a rebase, just so you know

I am not sure why github is saying there are merge conflicts, this branch should be fully up-to-date with upstream except one commit which is just about translations, but I think I resolved most issues you pointed out.

@AnErrupTion , would be great if work on this can continue, I can volunteer as tester, I think this will make Ly the best login manager for most linux and specifically wayland users right now.

Any updates on this?

I am still waiting on response, I can rebase this but I would first like some kind of interaction from @AnErrupTion before I put in a lot of effort into it

Lemurs can do this with pam rules, however it hasn't been developed in years


Return BlockHash from BlockHeader::validate_pow

Validating a block's proof-of-work involves computing the block hash. Returning it from BlockHeader::validate_pow avoids having callers recompute the block hash if it is needed.
Helpful when validating the chain in https://github.com/rust-bitcoin/rust-lightning/pull/791#discussion_r575705113

Based on current master and two acks. Merging.


Can't remove override if the directory no longer exists

Create a directory.
Call "rustup override set nightly".
Remove a directory.

There is no way to specify path for "unset" command. If the directory does not exist and you can't "cd" to it, then the override cannot be removed.
Possible solution: implement "rustup override unset for [path]" or "rustup override unset --path=[path]".

Maybe rustup override list could check each of the directories for existence, mark those that no longer exist, show a warning and offer a command to easily remove all overrides for such paths?

Thanks for the report!
rustup override unset --path= makes sense to me. Help wanted. For bonus points, @Boddlnagg 's suggestion also sounds good to me.

I think this can be closed now that https://github.com/rust-lang-nursery/rustup.rs/pull/650 has landed.


Simplifies Using a Different Client

Changes the Google Translate URL to use client=p instead of client=t, which already returns a valid JSON.

Thanks!

 Your project's source code helped me a lot finishing a small weekend project, it seems fair to contribute back with my improvements :)


Loading clusters from a KubeConfig file is broken

A user has reported that they couldn't load a cluster from a kubeconfig file from  the UI but could do it via the env var.
I took a look and turns out that since the stateless clusters, we have an exception being raised in the client code since we assume there to be an answer when the cluster is successfully set up but now there is none.
Here is a kubeconfig that illustrates the issue:
apiVersion: v1
clusters:
- cluster:
    server: https://k8s-api.dev-use1-100.domain.com/
  name: dev-use1-100
contexts:
- context:
    cluster: dev-use1-100
    user: cdenneen-dev-use1-100
  name: cdenneen-dev-use1-100
current-context: cdenneen-dev-use1-100
kind: Config
preferences: {}
users:
- name: cdenneen-dev-use1-100
  user:
    token: REDACTED

Closing in favor of #1831


Add um::objidl::IPersistFile and um::shobjidl_core::IShellLink

This adds the IPersistFile and IShellLink COM interfaces. I'm not exactly an expert in COM but I can confirm that this works for creating shell links.

PR has merge conflicts.

Should be fixed now


Implement App version checking

[x] distribute app via IPNS or DNS
[x] make app check for new versions and go back to older versions
[x] provide a redirect to the latest version of the app

Done in 8a690d7c95e8dded4fc18334d35498e7738a212b


Why there is always a property "x"?

Whenever I define an object, there's an "x" property.

the letter X denotes any variable

Yes this is by design. It indicates that the object is open, you can add any additional property to it or lookup any additional property on it. This is not always true, but it is the case when dealing with object literals in javascript


Bug - ISIIS ROIs displayed twice when sorting by size

Jonathan Nash testing/using PHOTIC with ISIIS ROIs has found that he is seeing double. Some of the ROIs will appear twice, not even right next to each other.
When sorting by ROI number, this problem does not show up.

How can I reproduce this issue?

could be related to #146

@joefutrelle I think we determined previously that this more about the loading of the ROIs than the UI, correct? The duplicates that I had found had very slight differences in the heights, and the IDs were different even though the image "looked" the same


Add benchmarks

Codecov Report
Base: 93.50% // Head: 91.39% // Decreases project coverage by -2.10% :warning:

Coverage data is based on head (b78684c) compared to base (ffc2176).
Patch coverage: 66.66% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
- Coverage   93.50%   91.39%   -2.11%
==========================================
  Files           1        1
  Lines          77       93      +16
==========================================
+ Hits           72       85      +13
- Misses          4        6       +2
- Partials        1        2       +1

Impacted Files
Coverage Δ

builq.go
91.39% <66.66%> (-2.11%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


correct the log

Describe what this PR does / why we need it
correct the log print wrong content
Does this pull request fix one issue?
Fixes alibaba#1114
Describe how you did it
just change fallbackProperties.getResponseStatus() to fallbackProperties.getResponseBody()
Describe how to verify it
define spring.cloud.sentinel.scg.fallback in yml file , when app starts , will see logs in console
Special notes for reviews

 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.

Hi, could you please sign the CLA?


Sourcegraph badge

As seen in pgrok, sourcegraph badge can be added into README.
[![Sourcegraph](https://img.shields.io/badge/view%20on-Sourcegraph-brightgreen.svg?style=for-the-badge&logo=sourcegraph)](https://sourcegraph.com/github.com/sadikkuzu/squiral)

will do

Love this. Let me know when it is up, and @sourcegraph will tweet a link to this project

cool. However not assigned yet

Thank you so much for your first contribution!
Without people like you submitting PRs, open source projects couldn't be run.
You rock, @ayushmaanshrotriya 🚀

great. thanks!

Tweet queued up. Thanks @sadikkuzu @ayushmaanshrotriya

Wow @jdorfman 🤩
I really appreciate 🙏🏼 ✨
https://twitter.com/sourcegraph/status/1635346932104323076

@sadikkuzu I told you :)

Yeah, you definitely did:)) @jdorfman 🚀

really cool

For readers reading these messages, you can join @sourcegraph discord server: https://discord.gg/s2qDtYGnAE


Restore or remove the --depend flag

rustc claims to support a --depend flag that lists the .rs files in a crate. There is support in front::eval for this, but it is not wired up to the driver. Either finish the implementation or remove its vestiges.

Removed.


Renaming of BuildEnvs

the Function struct's BuildEnvs member should be renamed simply "Envs" (along with its yaml serialized name).
The reasoning for this is not just because it is redundant, but because it is a hint to people manually editing the func.yaml that it should be defined at the wrong level.  for example, if you only look at the names of the members, it would be logical to assume the correct func.yaml structure to be:
build:
  pvcSize: 256Mi
buildEnvs:
 - name: MY_ENV
    value: myvalue

When in reality the correct structure is the unnecessarily redundant
build:
  pvcSize: 256Mi
  buildEnvs:
  - name: MY_ENV
    value: myvalue

(see Slack question)

Hey @lkingland ,
I would like to take up this good-first-issue.
I am new to this repository but would love to learn and solve this issue on the go.

Hello @aroramrinaal ; Please go right ahead.  As any questions you have

@aroramrinaal I think that @JanaganSaravanan was looking into this too.

@lkingland , I'm planning to address the issue of renaming the BuildEnvs member in the Function struct to "Envs" and updating its YAML serialized name. My approach involves a few steps. Firstly, I'll locate the file that contains the definition of the Function struct, which is usually a Go source code file. Inside the struct, I'll rename the BuildEnvs member to Envs and make sure to update its corresponding YAML serialized name as well. Secondly, I'll search for any references to BuildEnvs throughout the codebase and replace them with the new name Envs. Lastly, I'll ensure that the updated member name and serialized name match the desired structure in the func.yaml file. By following these steps, I believe I'll successfully resolve the issue by renaming the member and aligning it with the correct structure in the codebase.

@JanaganSaravanan thank you and sorry for the confusion.  I see you already had some work done in this direction, but were not assigned to the issue
@aroramrinaal , the closed PR by @JanaganSaravanan might be a good starting point

Hello @lkingland, I see that this issue has been closed as it hasn't been contributed to in the last 120 days.  Would it be possible to work on this as a good-first-issue? I am currently working with a team who would like to contribute to func over the course of the next year and we were hoping for something we could start off on.


remove python dependency

python2 is EOL. Not sure why you need python anyway, it seems to build and test fine.

python2 is EOL. Not sure why you need python anyway, it seems to build and test fine.

Python is needed for building native dependencies in node via node-gyp https://github.com/nodejs/node-gyp#on-unix
Recently the latest sqlite3 prebuilt package was (maybe still is?) not available on s3, causing npm to try to build the dependency natively. I was having trouble getting python3 recognized by tooling to get things to build.
I can see if I can get the python 3 version working.
Should note that python, gcc and make are not included in the final docker image. It's only used during the first stage build.

Python is needed for building native dependencies in node via node-gyp https://github.com/nodejs/node-gyp#on-unix
Recently the latest sqlite3 prebuilt package was (maybe still is?) not available on s3, causing npm to try to build the dependency natively. I was having trouble getting python3 recognized by tooling to get things to build.
I can see if I can get the python 3 version working.
Should note that python, gcc and make are not included in the final docker image. It's only used during the first stage build.

Got python3 working in #937

Got python3 working in #937


updated OpenMMEngine docstring, adresses #769

Changes to the docstring to reflect current implementation.

Please note that we have changed to the OPS license from LGPL (2.1 or later) to MIT. For any pull request to OPS that was started while the license was still LGPL, I need an explicit confirmation that you approve of the license change. Please add a comment with something like "The changes in this pull request are licensed under the MIT license."

The changes in this pull request are licensed under the MIT license.


Scrolling position doesn't get reset on page changes

Need to fix this, it's kind of composing how the page jumps to the middle on switching.

 I needed to edit their theme anyway. Thanks