[Enhancement] Don't ignore text layers prefixed with '-' where the layer name is the text value

Whenever I like to use a negative value in a text for example -25% and then try to add a padding to it, the padding jumps away because of the Ignoring specific layers mode. Of course it can be solved by simply removing the minus from the layer name.

Thanks for this @tikshow!
That's a really good point. I hadn't considered something like this. What do you think would be a good solution for this?
Perhaps layers need to be prefixed with a space after the dash? e.g. '- ' instead of '-'
Although, you'd still have the case of people writing text values that may begin with '- ' 🤔
This might be a tricky one to solve without breaking what other people already have set up too.

I'd propose the solution to be auto renaming any existing text layers starting with a - in the name to have a * prefix so it will be *-25% in your case, which will not be ignored by paddy.

This should now be fixed v1.0.7 – if you find it still doesn't work; then please re-open this issue.


Regenerator Runtime is Not Defined

I am seeing this error in the firefox console when trying to claim an account. Maybe this is a problem with the hosted package of Dsteem?

Thank you for reporting. I downgraded dsteem to v0.10.1. Its seems working but I don't have a big SP account to test. Would you please check again and confirm?


VM console is not available

FAILURE
REQUEST
{
"org.zstack.header.console.APIRequestConsoleAccessMsg": {
"vmInstanceUuid": "93ace80c440147b693bc8137c02d3ea7",
"session": {
"uuid": "918e72fce64b4466b6743d9466825971"
}
}
}
RESPONSE
{
"org.zstack.header.console.APIRequestConsoleAccessEvent": {
"success": false,
"apiId": "4f2690b3a2bd4036848d5b6aebec48b1",
"headers": {
"schema": {}
},
"error": {
"code": "SYS.1006",
"description": "An operation failed",
"details": "the console agent is not connected; it's mostly like the management node just starts, please wait for the console agent connected."
},
"id": "d3833858682640ec947b2f7aa2211e7a",
"creatingTime":<PHONE_NUMBER>645
}
}

This is an expected behavior for current ZStack. Every time the management node boots, it takes some time to deploy the console proxy agent; during the period, you will get the error above when you try to connect to a console because the console proxy is not ready.
The console proxy is written in Python, which will compile every time during deployment. We will replace it with a golang version that will significantly the deploying process.

Peter, our QQ group is 410185063 which has lots of users discussing issues there. If you speak Chinese, welcome to join !


⚠ TROJAN 🇺🇸 United States USO 1 has degraded performance

In 60758a5, TROJAN 🇺🇸 United States USO 1 (https://ust-2.opensvr.net/) experienced degraded performance:

HTTP code: 200
Response time: 17504 ms

Resolved: TROJAN 🇺🇸 United States USO 1 performance has improved in 2ed3f4b.


remove errant comma

Coverage remained the same at 100.0% when pulling 07978d5962e1a7acbee3099803063d91ef402055 on grahamu-patch-1 into 9b9cf91bdc599386c1349d68b68ee5ac50ed7599 on master.


3rd Party Entity lookup API

Documentation: https://docs.google.com/document/d/13NGa46a_WWno-XYfe8mQrglQdtOYMFVZtxkPKXDC3ac/edit#heading=h.m0btedxhv6ao
Author: @leonerd
Date: 21/07/2016
PRs: #1353

Ftr: #1414 is amending #1353 (under review now).

this got merged


feat: Add configuration validation

If some fields in the config file are missing or wrong, the backend expects them to exist and correct anyways, making it crash.
And because of issue #19 the enduser doesnt know why. Those two things need to be fixed.

That is true. But people removing the webhook table for example, or having a wrong turtle as string, results in a crash. the tab just closing is because of the crash by backend. You can always recheck that by running /src/init.lua and reproduce the error again, to see what went wrong. I today had 2 or 3 people having that issue with different configuration errors.

Solved with d3e6719991a46949c67a1b62934c46861886d3d4
config validation got moved to init.lua and makes sure that (hopefully) all required variables are correct. If not, it will log a error message with more details.


"Digest" image update strategy by tag deployed.

Is your feature request related to a problem? Please describe.
I'm deploying some complex stuff by using an ApplicationSet which based in a values.yaml file that an user push, it deploys an app of apps.
The problem is if I hardcode the version in the image list annotations of the app of apps, if the user change the version, it's not going to be automatically updated for a new digest.
Describe the solution you'd like
I would like to be able of update automatically by "digest" strategy the tag that has been already deployed without having to hardcode it again in the image-list annotation.
Describe alternatives you've considered
Same as we define annotations to specify which is the tag parameter that must be overwriting: argocd-image-updater.argoproj.io/test.helm.image-tag: test.image.tag would be nice to be able of use this also in the image list like:
argocd-image-updater.argoproj.io/image-list: test=repo/test:test.image.tag somehow
As I'm working with Helm, I thought about add the annotations using values templated. But the app of apps is created using a range loop, so I can't add them by just adding the annotation + the helm value like: argocd-image-updater.argoproj.io/image-list: test=repo/test:{.Values.test.image.tag}
Additional context
Any suggestion or way to solve this problem?

Why is the current digest update strategy not adequate for you? Once you decide on the name of the mutable tag, say latest, you can juse it in image-list annotation with digest update strategy, and image updater should be able to pick up any latest versions.  See https://argocd-image-updater.readthedocs.io/en/latest/basics/update-strategies/#strategy-digest

Why is the current digest update strategy not adequate for you? Once you decide on the name of the mutable tag, say latest, you can juse it in image-list annotation with digest update strategy, and image updater should be able to pick up any latest versions. See https://argocd-image-updater.readthedocs.io/en/latest/basics/update-strategies/#strategy-digest

Because is the user who specifies the tag depending on the valuesObject definition. If I add latest, it will get the latest, but maybe the user only wants the newest digest of tag 1.0.0 and other user wants the newest digest of tag 1.0.1.
Most of the values of the "app of apps" is defined by the user, and can differ a lot from one user to an other. I've can preset some default values and some annotations adding the image list with the version, but if the user decides to use a different version from the one I added by default, it must also be updated when a new digest is builded for that specific tag.

So you want the digest of the running image's tag?

So you want the digest of the running image's tag?

@jannfis yes, I want to update by digest strategy the running imag's tag.

What if there are two or more tags of the same image running for a given app?

well I will expect to update them all to the latest digest, otherwise, maybe can be added an exclusion annotation.

Any updates on this? I would be glad to participate if needed to add this feature, if you're okay with it


Dark Mode Broken

i broke dark mode :/

the solution'd prolly be height: 100% width: 100%

There isn't even a light mode anymore so...


Typo on Voice Control Template

In the whitespace section there is ironically a lack of whitespace between the words whitespace and on, so it reads: whitespaceon

Fixed in Commit 3b61b6a


Leaked credentials risk event

Based on the docs: "When the service acquires username / password pairs, they are checked against AAD users current valid credentials. When a match is found, it means that a user's password has been compromised, and a leaked credentials risk event is created"
I've seen cases where user identities are Federated and Password Hash Sync is not enabled, and the risk event is still raised. Can you please further clarify in which cases the event is raised?
Many thanks.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 198f0e69-af2a-8c82-041a-c5ca4126f02f
Version Independent ID: 4be57788-f1c2-d6d4-b94a-a59350a618a7
Content: Azure Active Directory risk events
Content Source: articles/active-directory/reports-monitoring/concept-risk-events.md
Service: active-directory
Sub-service: report-monitor
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@antede Thanks for your comment. We are actively investigating and will get back to you shortly.

@antede Federated users (without PHS) should not fall under this category. Do you have any events reported currently that I can use to validate?

@antede We have not heard from you in a while. We will now proceed to close this thread. If you have further questions, please tag me in the comments and I will gladly continue the conversation.

This is because sometimes we see attack patterns for the sign in's so we classified it as leaked credentials.

@AhmadNYasin that makes sense, but it is confusing because it is not described in the documentation.


investigate poor benchmark results

round 21 of techempower's benchmarks' results are out
json serialisation
single query
multiple queries
fortunes
data update
plain text
angel3 seems to be much worse than plain dart,
maybe there is some bottleneck worth investigating

@iapicca Thanks for the update on the performance benchmark. Version 4 is basically the first release after we migrated the entire codebase to null safety whereby the primary focus was to get everything working correctly first. Now that is pretty much settled, we can now look into ways to crank up the performance.


feat: Dynamic indexing of non-homogenous slices

Description
Problem*
Resolves #2752
Summary*
This PR enables dynamic indexing of non-homogenous slices. There are some extra caveats for dynamically indexing slices vs. arrays as arrays have a known size at compile-time while slices do not. In https://github.com/noir-lang/noir/pull/2703 where dynamic indexing of nested arrays was added, in order to accurately index the nested array we have to compute the true flat index of a nested array from its type information. As slices do not contain length information as part of the type we must use the SSA array values to determine a slice's flattened size and indices.
Slices have a dynamically tracked length, but the actual value representing a slice (and thus its capacity) is dependent the program witness. For example if we have an if statement based upon witness values where in the if-branch a slice's size is increased by 5 but in the else-branch the slice's size is increased by 10, both slices will now have a capacity of 10, while the dynamically tracked length is used to restrict out of bounds accesses. This can lead to some complications for dynamically indexing a nested slice, as we need to account for this placeholder data that may be greater than the slice itself.
Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[X] I have tested the changes locally.
[X] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Is this referring to slices in Noir? / Is this expected to be made use of by Noir devs (hence requires docs)?


feat: #27 add group actions to proposal create flow

closes: #27
closes: #48

@erika you can test this on https://deploy-preview-98--regen-groups-ui.netlify.app/

I wonder if we should add a tooltip here to explain the create proposal button? https://monosnap.com/file/NGx0eDkewXD9mF3HchJdBZ4XCf5VRM Or maybe change the button to "create proposal to save"? Or maybe both, wdyt?

Yeah I can add that quickly

Also, this could be a separate task but we should probably warn the user if they navigate away without clicking "create proposal"

Yes probably worth a separate task since 1. this PR is already quite large and 2. we might want the same behavior too even when the policy is not the admin

I wonder if we should add a tooltip here to explain the create proposal button? https://monosnap.com/file/NGx0eDkewXD9mF3HchJdBZ4XCf5VRM Or maybe change the button to "create proposal to save"? Or maybe both, wdyt?

Yeah I can add the tooltip quickly, but afraid that if we change the wording, it might result in a too long button visually and also by creating a proposal, it won't actually be saved, until the proposal passes and is executed, so it could be a bit misleading IMO

Yes that's true, let's just do the tooltip for now

Also, this could be a separate task but we should probably warn the user if they navigate away without clicking "create proposal"

Yes probably worth a separate task since 1. this PR is already quite large and 2. we might want the same behavior too even when the policy is not the admin

Ok cool I'll add a task for this.

@erikalogie can you give the final ok on this? Does the tooltip look good to you?

This looks good!


Dimensions in svg data

Hi!
thanks for plugin. It fast and solve my problem.
I used svg icons => svg sass data uri
And in genered file .scss i have widht/height for each icon.
Can you generate file without this dimensions, it will be less by size. For example add some option.
Thnx

I understand...
Need only change mst template

Sorry for the late answer.
Yes, you need only to create a custom template and remove the lines that relate to height and width.
And also call the custom template with the options.templateFileSvg option.
But consider that the svg is added as background, so without dimensions the container could be invisible, because it would not take space.

Thanks!
I paste here my template
%iconz {
  background-repeat: no-repeat;
  background-size: contain;
}

{{#items}}
.{{className}} {
  @extend %iconz;
  background-image: url('data:image/svg+xml;charset=utf-8{{#isBase64}};base64{{/isBase64}},{{encoded}}');
}
{{/items}}

Good.
But be careful: the background-size is not supported by ie8.
This is one of the reasons why i set width and height in the original template


Vaccination: Reporting and tracking adverse side-effects in the app

Feature description
Add  a box below "Wurden Sie getestet?": "Impfung" that includes a two buttons:

Register Vaccination
Register adverse side-effect

1. Button: Register Vaccination

Question: How would you like to enter the details?
a) Scan a QR-Code via the camera (when two doses are required this option should remain open after a first vaccination)
b) Enter the Information manually, possibly with additional verification
Produces an overview over the vaccination
Follow up: Would you like to share this with RKI and PEI to help?

if yes: possibly additional information such as age bracket and risk factors to offer additional metrics on how the vaccination effort is progressing

2. Button: Register adverse side-effects

Button only in active state after registering a vaccination
Auto-fill with the information about the used vaccine
Implement online reporting form in the app; possibly with additional questions.

Enable uploading documents such as lab reports, etc.

Button: Share with PEI
Overview over report including case number
Possible: Additional information about the reported side-effects ans possible remediation

In a dialogue between the Ministry of Health, PEI and representatives for doctors a similar and easy way of reporting directly should be set up additionally.

Problem and motivation
Current way of reporting
Doctors report adverse side effects (both Impfnebenwirkungen and Impfkomplikationen) to their local Health Authority (Gesundheitsamt). These processes are typically paper-based and use personell resources that could better be used for contact-tracing. With the high number of vaccinations it is questionable if this is a good use of doctor's time. They should be able to access a quicker method of reporting as well.
Consumers may report adverse side-effects direct to Paul-Ehrlich-Institut (PEI) both digitally as well as with a paper form.
Accessing and filling these forms will be a deterrence for many consumers and doctors. The number and quality of reports sent to PEI and in turn EudraVigilance will be smaller than is desirable.
This will not allow the national and international authorities to react quickly enough to specific vaccinations` adverse side-effects, thus not protecting public health as well as is possible.
Background
Several new vaccinations are currently in trials to test their safety and efficacy for immunization against SARS-CoV-2. Their development and approval is being sped up considerably, leading to shortened clinical trials in phase III. These will likely include only the required and smaller than usual number of 3000 participants that are required to finde side-effects that occur in at least 0,1% of all cases.
This means that phase IV - after approval and general use - is very important to monitor for unknown adverse side-effects of the different vaccinations. Safety of these vaccines is critical for trust in them and a high rate of vaccination to reach enough immunity throughout the population for the transmission of the SARS-CoV-2 to slow down dramatically.
As of today (Oct. 23rd 2020) some Local Health Authorities have become overwhelmed with cases and cannot trace individual case's contacts anymore. If Corona Warn App had more functionality for keeping a manual diary of one's contacts and possible super-spreading events already, this might alleviate some strain. The coming month's will likely remain challenging and public trust is vital for succeeding in the immunization effort.
Challenges
Privacy concerns
Corona Warn App is specifically designed not to contain personally identifiable data. With an implementation such as above users will be de-anonomized once they submit a report about an adverse side-effect.
It will need to be clearly communicated that this data isn't linked to contact-tracing data and will not allow authorities to create a profile of a user's geolocations.
Lower quality of reports
Self-reporting by consumers will generally be less precise that a doctor's report. With a high enough number of reports it should allow trends to emerge and be followed up by PEI. Allowing users to attach documents such as lab reports might further mitigate this issue.
Additionally, a diary of symptoms and structured questionnaires should mitigate this issue furthermore.
Abuse
Much like registering an infection within Corona Warn App there is potential for abuse; a similar system that (a) registers that a vaccination has been made and (b) filtering reports for plausibility might address this.
Is this something you're interested in working on
I am happy to help with the implementation; my coding skills do not allow me to productively participate in coding though.

Internal Tracking ID: EXPOSUREAPP-3429

Hey @NilsHempel ,
Thank you for the (very detailed) suggestion. I have created an internal Jira ticket () for your request. The feature will now be discussed by the developers and the RKI who will then decide when and how to implement it. We will post the status on the progress here, if we get any updates regarding your issue.
Regards,
CH

Corona-Warn-App Open Source Team

Like in #283 mentioned a message on wich date you have to get your second vaccination would be also great.

(sorry for my englisch)
As i wrote in #283 in addition of the reporting and tracking i want to do the next step with the App and make it to an "digitaler Corona Impfpass"
I think it will be possible that companies(Airlines) or concerts or in Stadion you only can get in if you have an vaccination. So How to Check this. There are two ways. 1.Oldscool or of i have no Smartphone i habe to carry a copy of my Impfpass. I think that is Not secure becouse i can copy it severel Times and give it to my friends who have no vaccination
The second way: use the App. On entrance of a concert, Bundesliga, airline,.. I go in my cwa app on menu "Impfpass" and the Smartphone Shows up an qr Code which can be scanned or acivates the nfc (Not every Smartphone has an nfc so the qr Code way needes to be implemented too, but with nfc i can use in future my smartwatch too)
The qr Code can be secured (Short validation timespan, secureid,...) that make it harder to fake (Screenshot that i send to my friends with no vaccination)
On my opinion this way has a great advantage: fast! (fastline on Bundesligastadion, so the security has more time to Check the Papers from the Persons who use the oldscool way) and secure. (i dont have to write All my datas to an paper like it is for now on Restaurants) and at least the Datenschutz is given to, i only send my vaccination Status, Not my Adress (oldscool Impfpass or on the Papers on the kontaktverfolgung in Restaurants i always wrote my Adress visible to All)

Are there any updates yet?
I heard about the SafeVac-App from the PEI (Paul Ehrlich-Institut). Is there the possibility to integrate this in the cwa?

Hi @NilsHempel, @jucktnich, thanks again for proposal and contributing here. We will raise this issue again in one of the next meetings. Stand by for feedback. Best, DS

Corona-Warn-App Open Source Team

@jucktnich

I heard about the SafeVac-App from the PEI (Paul Ehrlich-Institut).

https://www.zusammengegencorona.de/impfen/wirksamkeit-und-sicherheit/#faqitem=80f1f47f-b2b9-54d7-90b6-e518e0c5a731 says
Wo kann ich den Verdacht einer Nebenwirkung melden?
Sollten Sie den Verdacht einer Nebenwirkung haben, können Sie dies sowohl Ihrer Ärztin/Ihrem Arzt als auch beim Paul-Ehrlich-Institut (PEI) als oberster Bundesbehörde, Ihrem örtlichen Gesundheitsamt oder online melden. Weiterhin können Sie Nebenwirkungen in der "SafeVac"-App des PEIs melden.

Stand: 23.12.2020

Die Frage ist ja schon, wo will das RKI mit der Corona App hin?

B) Die App sollte nur Kontakte aufzeichnen.  Und auch während der Impfung einen Qr Code zu scannen um wesentliche Daten zu meiner Impfung zu erfassen finde ich nicht schlimm.
Falls B das ziehl ist, könnte ja in der App ein Link zur PEI App eingebaut werden.
 Das Problem daran. Niemand kennt diese.

Let me suggest to split the ticket into two. Whether CWA would trace vaccination side effects or link to a separate facility as it is today provided by PEI can be discussed separately from the question whether contact tracing should include information on vaccination or not.
The news[1] currently report that there is currently one study conducted by Pfizer on whether immunization protected people can effectively transmit the desease, and it might take months to give results.
Thus in preparation to a possible positive outcome of this and related study and as a means to start adjusting the severity algorithm for contacts the moment the results are there (with enough time on its sociological consequences) distribution of the vacciation state of contacts should be expidited.
[1] Harro Albrecht, Andrea Böhm et. al., Eine Ampulle Normalität (07.01.2021). Hamburg, Germany: DIE ZEIT Politik

Let me suggest to split the ticket into two. Whether CWA would trace vaccination side effects or link to a separate facility as it is today provided by PEI can be discussed separately from the question whether contact tracing should include information on vaccination or not.
The news[1] currently report that there is currently one study conducted by Pfizer on whether immunization protected people can effectively transmit the desease, and it might take months to give results.
Thus in preparation to a possible positive outcome of this and related study and as a means to start adjusting the severity algorithm for contacts the moment the results are there (with enough time on its sociological consequences) distribution of the vacciation state of contacts should be expidited.
[1] Harro Albrecht, Andrea Böhm et. al., Eine Ampulle Normalität (07.01.2021). Hamburg, Germany: DIE ZEIT Politik


Fixes smart quotes

Fixes smart quotes in Podfile in example project to make pod install stop complaining.

Thanks!


Aerial states "Your Mac does not support Night Shift" since Sonoma

General troubleshooting tips
Before logging an issue please check that:

[x] You have the latest version installed (There may be a beta version that fixes your issue), see here for the latest releases and bug fixes: https://github.com/JohnCoates/Aerial/releases
[x] Your issue isn't already mentioned in our issues. You may find a workaround there or a similar request already made.
[x] Your problem isn't mentioned in the troubleshooting page.

If none of this fixes your issue, tell us about the problem you are experiencing or the feature you'd like to request.
Required information
In order to help us sort your issue, we ask that you provide the following information:

[x] Mac model: MacBook Pro 16" 2021, M1 Pro
[x] macOS version: macOS Sonoma 14.0
[x] Monitor setup: None/single external monitor (not relevant to issue)

If appropriate, please enable Debug mode and Log to disk in Advanced tab and replicate your bug, then attach the AerialLog.txt file (You can access this file through the Advanced tab).
Description of issue / Feature request

After updating my Mac to Sonoma, I noticed that in the Aerial Time settings (edit: in macOS System Settings), Aerial now states incorrectly "Your Mac does not support Night Shift" and the option is greyed out; this was not an issue while on Ventura, and I had recently seen/checked this in the past week also. (System-wide) Night Shift definitely still works on my Mac now in Sonoma, and it's still set to Sunset to Sunrise as before.

I have just further noticed however, that somehow Aerial settings within System Settings has this issue, versus when I open Aerial Settings from within Aerial Companion it does correctly show Today's sunrise & sunset timings from Night Shift. Aerial v3.3.1, using Aerial Companion 1.5.1
(I have switched to Use my location instead as a workaround in the meantime.)

Not sure if it's related to the above issue, but I have just noticed also that Use my location seems to push the Aerial sunrise videos window one hour early (e.g. 5.43-6.43am) and/but sunset videos one hour later (e.g. 7.07-8.07pm), as compared to Use macOS Night Shift for calculations (6.55-7.55am and 5.54-6.54pm respectively). Not sure which is correct/the actual intention.

Hey there!

After updating my Mac to Sonoma, I noticed that in the Aerial Time settings (edit: in macOS System Settings), Aerial now states incorrectly "Your Mac does not support Night Shift" and the option is greyed out; this was not an issue while on Ventura

I knew I had forgotten something, it's one of these things I saw during the beta but never investigated. I'll have to double check. The night shift check might be blocked by the system, or Apple changed the format and Aerial can't read it. I'll have a look.

I have just further noticed however, that somehow Aerial settings within System Settings has this issue, versus when I open Aerial Settings from within Aerial Companion it does correctly show Today's sunrise & sunset timings from Night Shift. Aerial v3.3.1, using Aerial Companion 1.5.1

Yeah it's one of those things, Apple puts extra restrictions all the time when the screensaver runs from System Settings, as a screen saver, or via companion. It's a never ending game of cat and mouse to try and figure out what still works, what I can workaround, and what I should drop...

Not sure if it's related to the above issue, but I have just noticed also that Use my location seems to push the Aerial sunrise videos window one hour early (e.g. 5.43-6.43am) and/but sunset videos one hour later (e.g. 7.07-8.07pm), as compared to Use macOS Night Shift for calculations (6.55-7.55am and 5.54-6.54pm respectively). Not sure which is correct/the actual intention.

I'll have to check but I used to have different calculation modes available (there's multiple definitions of sunset/sunrise depending on what you want). Could be the settings is hidden somehow or I removed it for some reason ? I'll check tomorrow or around the week end and get back to you!

Ok quick follow up :

A screen saver is no longer able to read night shift data, as I feared, in Sonoma. It works under companion because we don't live under the same restrictions
The same issue makes the ability to run a shell script in the message overlay broken too
Regarding calculations, may I suggest you change Sunset/Sunrise window to 2h, this will give you something closer to what you had with nightshift

Thanks @glouel for the follow up on this. Thankfully (at least for me) none of this is show-stopping/there are the workarounds. Will leave it to you to consolidate whatever changes you decide will be best going forward at your time, completely understand the limitations and nature of this beast. Thanks again for all your time and effort on the whole project!

There will be a fix in next version, the workaround does require Aerial Companion to run. Same goes for location if you need updates to your location (like on a macbook).

@bryandmlee
Please try and install : https://github.com/AerialScreensaver/AerialCompanion/releases/download/v1.5.2/Aerial.Companion.zip
Then when prompted update Aerial to 3.3.4. Afterwards, Night shift should work back again. You need to have companion running for it to work.

Thanks @glouel. Have installed Companion v1.5.2 and Aerial 3.3.4 now, will check on the Night Shift behaviour come sunset/sunrise.
Since Companion needs to be running (in that sense all the time), which is the correct setting to use/what exactly is the behaviour of each option, in Companion settings > Launch Companion > At startup/In the background? (I hope I'm asking the right question)

@bryandmlee let me know how it goes for you !
And no worries, that's a good question I should have explained it more, you need "At Startup" which will have it in your menubar. Since Apple is blocking the screen saver from accessing it, I have the saver "ask" Companion to fetch and send back the data instead. For this to work it needs to be launched.
I will probably remove the "in the background" option at some point because I fear many features will require that workaround.

I think this is fixed now, so closing, feel free to reopen if you need more help.


Unable to install Salt native minion on vQFX: malformed package

Salt has recently open-sourced their Junos native minion (https://saltstack.gitlab.io/open/docs/salt-install-guide/topics/install/native/juniper.html) but it does not install on vQFX10k running 20.2R1.10:
root@vqfx-re> request system software add /var/tmp/salt-junos-x86-64-20201026-235317.tgz
malformed package: /var/tmp/salt-junos-x86-64-20201026-235317.tgz

{master:0}

root@vqfx-re> show log messages | match mgd
Jun  2 18:26:35  vqfx-re mgd[3217]: /usr/libexec/ui/package -X update /var/tmp/salt-junos-x86-64-20201026-235317.tgz
Jun  2 18:26:40  vqfx-re mgd[3217]: UI_CHILD_EXITED: Child exited: PID 4404, status 9, command '/usr/libexec/ui/package'

For comparison, it runs just fine on an MX204 running 18.3R1.9:
root@mx204# run request system software add /var/tmp/salt-junos-x86-64-20201026-235317.tgz
Verified salt-junos-x86-64-20201026-235317 signed by saltstack-packaging-commercial-1 method RSA2048+SHA1
Backing up salt-junos to /config/SaltBackup/salt-junos.tgz
Removing old /var/db/scripts/commit/salt.slax
Copying /packages/mnt/salt-junos/var/db/scripts/commit/salt.slax to /var/db/scripts/commit/salt.slax
Removing old /var/db/scripts/op/salt_dualrengine.slax
Copying /packages/mnt/salt-junos/var/db/scripts/op/salt_dualrengine.slax to /var/db/scripts/op/salt_dualrengine.slax
Removing old /var/db/scripts/event/salt_event.py
Copying /packages/mnt/salt-junos/var/db/scripts/event/salt_event.py to /var/db/scripts/event/salt_event.py
Removing old /var/db/scripts/event/salt_log.slax
Copying /packages/mnt/salt-junos/var/db/scripts/event/salt_log.slax to /var/db/scripts/event/salt_log.slax
root@mx204> edit
Entering configuration mode

[edit]
root@mx204# set system scripts op file salt_dualrengine.slax

[edit]
root@mx204# commit
commit complete

[edit]
root@mx204# exit
Exiting configuration mode

root@mx204>
root@mx204> edit
Entering configuration mode

[edit]
root@mx204# set system login user saltstack uid 2001

[edit]
root@mx204# set system login user saltstack class super-user

[edit]
root@mx204# set system login user saltstack authentication encrypted-password "salt@123"

[edit]
root@mx204# set system scripts commit file salt.slax

[edit]
root@mx204# set system scripts language python

[edit]
root@mx204# set event-options generate-event E1 time-interval 60

[edit]
root@mx204# set event-options policy SALT_POLICY events E1

[edit]
root@mx204# set event-options policy SALT_POLICY then event-script salt_event.py

[edit]
root@mx204# set event-options event-script file salt_event.py python-script-user saltstack

[edit]
root@mx204# deactivate event-options policy SALT_POLICY

[edit]
root@mx204# set event-options policy SALT_LOG_POLICY events SALT_LOG_ROTATE

[edit]
root@mx204# set event-options policy SALT_LOG_POLICY then event-script salt_log.slax

[edit]
root@mx204# set event-options event-script file salt_log.slax

[edit]
root@mx204# set event-options generate-event SALT_LOG_ROTATE time-interval 180

[edit]
root@mx204# commit
commit complete

FYI the code was developed / tested / released with the use of a QFX 5120 running v18.4.  Also it was additionally tested by Juniper too as part of a co-development.

18.4R2-S2.3


[turborepo] .jsonc support (Json with comments)

What version of Turborepo are you using?
v1.7.2
What package manager are you using / does the bug impact?
Yarn v2/v3 (node_modules linker only)
What operating system are you using?
Mac
Describe the Bug
renaming turbo.json to turbo.jsonc
Expected Behavior
search for turbo.jsonc for fallback
To Reproduce
rename .json to .jsonc
Reproduction Repo
No response

@softmarshmallow for what it's worth, you can have comments in turbo.json. I know it doesn't help with things like IDE formatting, but you should at least be able to write comments. Previously discussed here also: https://github.com/vercel/turbo/issues/1470

Looks like in VSCode at least you can associate a file with a language, I've done that for this repo as well in #3812. Does that work for you @softmarshmallow?

This is supported behavior! Woohoo!

@anthonyshew is it? I tried turbo.jsonc with 1.12.3 and I get an error Could not find turbo.json.

I suppose the specific implementation described in this issue isn't supported. I was doing some cleanup of old issues and saw the title + not much description so went on the goal, not the implementation. Apologies!
Given that the goal is "add comments to turbo.json", I'd figure we could call the intent of this issue satisfied since turbo,.json supports comments?
@mehulkar Or, alternatively, we could leave it open as a label:good-first-issue?

i don’t feel that supporting an alternate file name is valuable, but since the issue was closed for a slightly different reason I wanted to give @ari-becker a chance to reply. I’m in favor of closing unless we discover a compelling reason.

@mehulkar it's not even my issue originally, I just watched it for updates since I'm affected by it too 😅
Please note that editors (e.g. WebStorm) can and do identify the comments as an error:

This registers as a false-positive in IDEs and negatively impacts attempts to try to keep the number of problems in a project at zero.
Maybe you think that's "not valuable", but IMO not supporting a .jsonc extension is bad developer experience and that's reason enough to keep this issue open - @anthonyshew 's suggestion to keep it as label:good-first-issue sounds good to me.

I agree that native support would be ideal. "valuable" was the wrong choice of words, my apologies. I mean that in my opinion, it doesn't seem worth tradeoff of maintaining that feature. I'll let @anthonyshew make the final call though, as I could be in the minority here.
Webstorm can also configure this file type associations for what it's worth, and I imagine other editors can too.
We configure for VSCode in our repo as well: https://github.com/vercel/turbo/blob/178a2948eeb795ba925838b2380bd5c532a2cd5f/.vscode/settings.json#L12

We configure for VSCode in our repo as well:
https://github.com/vercel/turbo/blob/178a2948eeb795ba925838b2380bd5c532a2cd5f/.vscode/settings.json#L12

@mehulkar you comment is so valuable! It should definitely be added to the official docs!

This approach doesn't work 100% for users of Jetbrains IDEs (Webstorm, IDEA, etc). You can associate turbo.json with JSON5 but not JSONC, but JSON5 includes other features (like non-quoted keys) that turbo.json can't handle.
To use JSON5 as a partial workaround you can right click on the file and Override file type. Alternatively you can set up a general rule in Settings > Editor > File types where you associate turbo.json with JSON5.
It's also possible to 'fake' comments in JSON with a key like "// 1": "the comment" but then this falls foul of the Turbo JSON schema validation, which expects things like tasks to be objects.

For now, I'm using a ln -s symbolic link “turbo.json -> turbo.jsonc” because the suggested VSCode config solution isn't great (e.g. prettier doesn't like it + how to make a generic config for a large dev team who each use a different IDE).
I don't think it would be complicated to do what renovate does, which uses a list of possible paths ordered by precedence:

(https://github.com/renovatebot/renovate/blob/main/lib/config/app-strings.ts)

Workaround for VS Code, add the following config in the workspace/folder config file.
{
  "settings": {
    "files.associations": {
      "turbo.json": "jsonc"
    }
  }
}


Ocamlformat 0 25 1

Description of changes

Added ocamlformat.0.25.1 and ocamlformat-lib.0.25.1
Added myself as a maintainer for ocamlformat

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

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

[x] Fits CONTRIBUTING.md.

@Zimmi48 @marsam Is it OK for me to be added to the maintainers ?

Please fix the commit messages according to this and this, thanks!

Fixed.

Result of nixpkgs-review pr 227229 run on x86_64-linux 1

8 packages built:

ocamlformat (ocamlformat_0_25_1)
ocamlformat_0_19_0
ocamlformat_0_20_0
ocamlformat_0_20_1
ocamlformat_0_21_0
ocamlformat_0_22_4
ocamlformat_0_23_0
ocamlformat_0_24_1

Result of nixpkgs-review pr 227229 run on x86_64-linux 1

8 packages built:

ocamlformat (ocamlformat_0_25_1)
ocamlformat_0_19_0
ocamlformat_0_20_0
ocamlformat_0_20_1
ocamlformat_0_21_0
ocamlformat_0_22_4
ocamlformat_0_23_0
ocamlformat_0_24_1

Needs to be rebased.

Thanks, rebased!

Are more reviews needed ?

No, sorry for the long wait!


Unable to get a full Swagger JSON file

I'm not sure if I'm missing something or if there is a bug. Is there a way to get a full Swagger JSON file beside the Swagger-UI?
This is how my generated /documentation looks like:

And this is what I get when I enter /docs:
{"apiVersion":"0.0.0","swaggerVersion":"1.2","basePath":"http://localhost:5000/docs?path=","apis":[{"path":"account-applications"},{"path":"approval-requests"},{"path":"clients"},{"path":"partners"},{"path":"user"}],"info":{"title":"API","description":"API is used to access selected systems and data."}}
How can I get the full Swagger JSON file describing my API?

Hi
So swagger 1.2 breaks up its JSON. The file /docs: points to the JSON endpoints with the API descriptions. This is done with a querystring called path.  From the JSON you posted:
{
    "apiVersion": "0.0.0",
    "swaggerVersion": "1.2",
    "basePath": "http://localhost:5000/docs?path=",
    "apis": [
        { "path": "account-applications" },
        { "path": "approval-requests"},
        { "path": "clients" },
        { "path": "partners"},
        { "path": "user" }
    ],
    "info": {
        "title": "API",
        "description": "API is used to access selected systems and data."
    }
}

The URLs would be:
http://localhost:5000/docs?path=account-applications
http://localhost:5000/docs?path=approval-requests
http://localhost:5000/docs?path=clients
http://localhost:5000/docs?path=partners
http://localhost:5000/docs?path=user
You should be able to see these network request in your browser dev tool when viewing the Swagger UI.
In the latest version of Swagger 2.x all the information is usually in one JSON document, but hapi-swagger is still using 1.2
Hope that helps

Thank you. Is there an update to 2.x planned?

I have been wanting to do this for last 6 months. There was a technical reason why it was not worth doing, but I think that is now sorted. Its just comes down to time, the project needs a complete rewrite and I am busy with other stuff.  My next clear block of time is at the end of the summer.


Bump STS for release branch

https://github.com/microsoft/sqltoolsservice/compare/<IP_ADDRESS>...<IP_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.004%) to 41.574%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/dialog/configurePython/configurePathPage.ts
1
57.02%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
30787

Relevant Lines:
69356

💛  - Coveralls


Multi-threading causes continuous physics_process time increase until crash

Godot 3.4.3-stable
Ubuntu Linux
I don't know if this is an implementation issue or an engine bug, but the multi-threading in the example scenes causes a continuous rise in physics process time, while generating tiles, until Godot eventually crashes.  This is despite nodes being properly freed, with the total node count staying constant once it hits the terrain tile limit, as shown in the screenshot below.  This was only a short test, but in a longer test of moving the player and generating tiles for about five minutes, Godot crashed after reaching about 50ms physics process time.  Somehow the physics engine is not working in sync with the multithreading.

I tested both Godot physics engines and they exhibited the same issue.
I removed multithreading from the example scene and repeated my testing, and the issue disappeared: physics process stayed constant, as you would expect with generating/removing a constant number of tiles:

You do no use mutex locking in your example, which I would expect to see.  However, I added mutex locking in my testing and it made no difference to this issue.

Confirmed. I have tested it but the results were different. However, I've also noticed a constant increase in the physics process as well. What's interesting is when I stop the physics process drops below 1ms. I'll try to do some more testing to find out what is going on

Turns out, this is due to having the signals.  If you disconnect the signals in the editor (or just don't emit them), the physics_process time remains constant, as expected, instead of continuously climbing.
I thought it might be related to how you send the meshinstance reference with the chunk_spawned signal -- but never use the reference.  So I moved signal wiring into code, removed the meshInstance reference from the spawned signal -- e.g. signal chunk_spawned(chunkIndex) -- but this made no difference and the physics_process time continued to climb.  Only removing (or not emitting) the signals altogether stopped the physics_process time accumulation.
I haven't tested a build to see if this is an Editor-only bug.

Further to above, the problem is directly related to updating the ChunkDebug label -- which I have cached the reference in the example below.  If you comment out the label.text lines, which update the label on screen, the physics_process problem goes away.  Or if you change it to add an empty string (i.e. do nothing) -- label.text += "" -- the problem goes away.
func chunk_spawned(chunkIndex, chunkNode):
	label.text += "\nChunk spawned: " +str(chunkIndex)
	debugLines += 1
	skip_lines()

func chunk_removed(chunkIndex):
	label.text += ". Chunk removed: " +str(chunkIndex)

I tried moving the emits to next main-thread frame with call_deferred and that did not remedy the physics_process accumulation.
I don't understand yet why updating that label -- from the thread or the main thread -- causes this physics issue.  Labels by themselves have no physics component.

That's weird. I think it either might be a problem with passing arguments to a chunk_spawned() function or a bug with updating ChunkDebug label

It gets even weirder.
As shown below, if you change both the label-update lines from:
label.text += "\nChunk...
to:
label.text = "\nChunk...
then there's no constant increase in physics_process time.  Somehow, doing simple string concatenation on that label text is bugging out the physics!


Extracted interface Leashable.

Only golems and animals are leashable in vanilla, However other entities are still able to be leashed if forced.
I propose that an entity other then what is leashable in vanilla is an implementation specific detail, and should not be included in the API.
However, due to leashable being it's own interface, anyone that wishes to may attempt to cast an not listed as leashable in the API as leashable in order to test whether it happens to be leashable as an implementation detail. This would allow for mods/other server implementations to make anything leashable, whilst leaving room for the sponge mod to refuse to support entities which are currently leashable but not listed as such in the API at any time.

Leashable should extend Living at the least.

Through gameplay it is not possible to leash the other entities, however it is still possible to leash them using other methods. As all living creatures /can/ be leashed, I don't see why they shouldn't be Leashable.

I had tested leashing some entities to each other using commandblocks and the results were weird. not rendering correctly but still following the leash behaviour.

@SpongePowered/developers
I'd like a consensus on this one.

@me4502 to me it's the distinction between saying that all living entities will always be leashable because it's in the API, vs saying that specific entities will always be leashable.
If someone created a LivingEntity that wasn't leashable, what would happen then? Or would it be by definition no longer a LivingEntity because it can't be leashed?

I don't think LivingEntity is defined by the ability to be leashed, that
wouldn't make much sense, it's too narrow a trait for a class already
fulfilling a defined role (living entities).


Orphan installation namespace for the addon deploy ManifestWorks

The addon controller is deployed in the same namespace (defaults to "submariner-operator") as the Submariner components on the spoke cluster. The ManifestWorks deploys the Namespace resource as well to ensure it's created when deploying the other resources however we don't want the ManifestWorks to delete the Namespace on uninstall to avoid a race condition where the Submariner operator pod is deleted before it is able to run cleanup and remove its finalizer from the Submariner resource.
Set the addon.open-cluster-management.io/deletion-orphan annotation on the Namespace resource so the addon framework configures the ManifestWorks to orphan it.
See https://issues.redhat.com/browse/ACM-15538 for more details.

/cc @dislbenn

/approve

/cherry-pick release-2.12


[TCL] 插入大 key 会卡主不动

Is this a regression?
Yes
Description
如下 TCL 测试会卡主过不了：
    test {Very big payload in GET/SET} {
        set buf [string repeat "abcd" 1000000]
        r set foo $buf
        r get foo
    } [string repeat "abcd" 1000000]

Please provide a link to a minimal reproduction of the bug
No response
Screenshots or videos

Please provide the version you discovered this bug in (check about page for version information)
No response
Anything else?
No response

@shimingxuan @Samuel size


Test with Boost 1.65

Boost 1.65 has been released. Bond needs to be tested with this, and we need to add it to the CI matrix for Travis and AppVeyor if possible.

[x] Build Boost 1.65 archive for Travis CI. Added in commit 6821db9.

[x] Test Boost 1.65 with recent Clang on Linux {x86, x64}
[x] Test Boost 1.65 with a recent GCC on Linux {x86, x64}

[ ] Add Boost 1.65 to AppVeyor matrix

[ ] Test Boost 1.65 with Visual C++ 2015 on Windows {x86, x64}

[x] Manually test Boost 1.65 with Visual C++ 2017 on Windows {x86, x64}

Boost 1.65 is outside of the range of Boost that Bond supports now. Closing this in favor of https://github.com/microsoft/bond/issues/855


File picker site web support

Q
A

New feature?
[ X]

Related issues?
partially #999

What's in this Pull Request?
SiteFilePickerTab already had webAbsoluteUrl Support, but on the Startpage, RecentFilesTab didn't support this property

Thanks @petkir  - the PR has been merged manually.


stddev missmatches postgres'

Problem Statement
Context
In standard deviation calculation there is the Bessel's correction this mean that there are 'two ways' of calculating a deviation: population standard deviation and sample standard deviation, they are very similar but the formula and result varies a little bit.
population stddev
$\sqrt{Σxi - u}\over{N}$
sample stddev
$\sqrt{Σxi - u}\over{N-1}$
The problem.
Our stddev is a population stddev but postgres's is sample stddev so crate and postgres cannot equate stddev results.
you can quickly check this in here and running
crate:
select stddev(a) from unnest([1.0, 2.0, 3.0, 4.0, 5.0]) as a; -> 1.4142135623730951
postgres:
select stddev(a) from unnest(array[1.0, 2.0, 3.0, 4.0, 5.0]) as a; -> 1.5811388300841897
Possible Solutions
Change the implementation of stddev to match postgres'  or keep it the same and add stddev_pop and stddev_samp? so we can at least polyfill/overload stddev calls.
Most databases seem to implement both functions in one way or another:
postgres -> stddev (default sample), stddev_pop and stddev_samp
mongodb -> stdDevPop and stdDevSamp
mysql -> sstdev (default pop), stddev_pop, stddev_samp
influxdb -> stddev(..., mode: 'sample' or 'population' = 'sample')
Considered Alternatives
No response

This is pretty much also the case with variance, the formula varies depending on sample vs population.
crate:
select variance(a) from unnest([1, 2, 3, 4, 5]) as a; -> 2
postgres:
select variance(a) from unnest(array[1.0, 2.0, 3.0, 4.0, 5.0]) as a; -> 2.5


Some changes

Fixes #[Add issue number here. If you do not solve the issue entirely, please change the message e.g. "First steps for issue" #IssueNumber]
Changes: [Add here what changes were made in this pull request and if possible provide links showcasing the changes.]
Screenshots of the change: [Add screenshots depicting the changes.]

Some changes


VueJS "other input" bug

Are you requesting a feature, reporting a bug or ask a question?
bug
What is the current behavior?
we see many "other" fields
What is the expected behavior?
only one "other" field
How would you reproduce the current behavior (if this is a bug)?
choose other checkbox
Provide the test code and the tested page URL (if applicable)
http://surveyjs.org/examples/vue/service-oneperuser.html
Specify your

browser: chrome
browser version: last
surveyjs platform (angular or react or jquery or knockout or vue): vue
surveyjs version: 0.12.6

not reproduced in new build


Customize the amount of storage associated to Ceph

We recently hit issues in CI because this pod is evicted with the following message:
Usage of EmptyDir volume "data" exceeds the limit "500Mi"

This patch exposes the DATA_SIZE parameter that can be used to customize the amount
of space reserved to the toy Ceph cluster (especially for CI purposes).

Tested locally (make ceph DATA_SIZE=2Gi TIMEOUT=90) and see the value being updated:
Volumes:
  data:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  2Gi
  run:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  2Gi
  log:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  2Gi

/lgtm


[Bug]: ManagedPeripheral, on iOS, notifications fails to automatically re-subscribe after the device disconnected and connected again.

Component/Nuget
BluetoothLE Client (Shiny.BluetoothLE)
What operating system(s) are effected?

[ ] iOS 12
[ ] iOS 13
[ ] iOS 14
[X] iOS 15
[ ] Mac Catalyst
[ ] Android 8
[ ] Android 9
[ ] Android 10
[ ] Android 11
[ ] Android 12

What platform(s) are effected?

[ ] MAUI
[ ] Xamarin Native
[X] Xamarin Forms

Steps To Reproduce
1 - Create a ManagedPeripheral from a Device and Connect to it
2 - Subscribe to a notification of this device:
ManagedPeripheral.WhenNotificationReceived("97d8ccb6-b6b6-4bc8-a7ba-5b4ecb9e4f62","875607c1-1f96-449c-b4d5-14dab172d5e6").Subscribe(result => Process(result));
4 - Test it, and see the notifications being received as expected.
5 - Turn off the Device
6 - Turn on the Device again
7 - The device will automatically reconnect but will show an error, and the Notification will no longer be received.
System.ArgumentException: No characteristic found - 97d8ccb6-b6b6-4bc8-a7ba-5b4ecb9e4f62/875607c1-1f96-449c-b4d5-14dab172d5e6
  at Shiny.BluetoothLE.Extensions+<>c__DisplayClass15_0.<Assert>b__0 (Shiny.BluetoothLE.IGattCharacteristic ch) [0x00028] in <1c688311e7f04d34b71919a7f9529315>:0
  at System.Reactive.Linq.ObservableImpl.Do`1+OnNext+_[TSource].OnNext (TSource value) [0x00000] in <3ad0cc868be642ac8d0c22906bb12ddb>:0

Everything works as expected on Android. The problem only happens on iOS.
Expected Behavior
The device should automatically re-subscribe to the notification.
Actual Behavior
An error occurs and the notification is not re-subscribed.
Exception or Log output
No response
Code Sample
No response
Code of Conduct

[X] My reproducible sample is NOT FROM THE SHINY SAMPLES REPOSITORY!  If you file an issue that says 'use shiny sample', YOU WILL BE BANNED FROM THE SHINY REPOS.  Read the code of conduct for why we care about this!
[X] I am using the latest stable version from nuget (sponsors can discuss previews in discord)
[X] I agree to follow this project's Code of Conduct

I'm testing with another device.  It seems to be some problem with Discover Services and Characteristics on iOS. On these another device, I cannot event read manually values from some Characteristics that I know it exists.
var rawValue = await ManagedPeripheral.Read("000030ff-0000-1000-8000-00805f9b34fb", "0003000-0000-1000-8000-00805f9b34fb").ToTask(); ;
It shows me the error :
No characteristic found - 30ff/00003000-0000-1000-8000-00805f9b34fb

Works fine for me.  I just tried it with my code without issue.  Since no reproducible sample has been added, I'm closing this since it does not meet the terms of filing an issue


SSFO: BG Job commits unchanged objects randomly

Hi.
I'm having this situation where my nighly job commits strange things that haven't been touched since years.
it only happens to (some, but the same) SSFO objects and it looks like that:

first analysis looks like the language selection is unsorted.
it starts in FB: SSF_READ_FORM line 45 /59 where all texts are read into t_objt.

and follows in FB SSF_READ_OBJ_TEXT where it is returned to whoever wants the available texts.
It seems to be only occurring on big forms combined with lots of supported languages, our t_objt has more than 10.000 entries, so I assume it is natural that the order of all those is not identically when not explicitly requested ).
I remember that we had to do some work back when we migrated to HANA and manually added the sort command to all DB selects that were not sorted yet, as HANA does not sort by default. I assume SAP should have done that as well.
Now.. what would you recommend..
Is that a fix that would implemented in your ZCL_ABAPGIT_OBJECT_SSFO
or
should we try SAP changing the FBs, adding a sort
or
do you have another approach to this?
Thanks
PhiL

I'd suggest fixing it in https://github.com/abapGit/abapGit/blob/main/src/objects/zcl_abapgit_object_ssfo.clas.abap
it might be a bit of a mess, as it got the ixml object
pull requests welcome

what is the XML path down to <T_CAPTION>? and does the problem only occur under <T_CAPTION> ?

it occurs here and there:
/abapGit/sf:SMARTFORM/sf:VARHEADER/sf:item/sf:PAGETREE/sf:NODE/sf:SUCC/sf:item[1]/sf:NODE/sf:SUCC/sf:item[8]/sf:NODE/sf:OBJ/sf:WINDOW/sf:PROC_CTRL/sf:NODE/sf:SUCC/sf:item[6]/sf:NODE/sf:OBJ/sf:SECTION/T_CAPTION
/abapGit/sf:SMARTFORM/sf:VARHEADER/sf:item/sf:PAGETREE/sf:NODE/sf:SUCC/sf:item[1]/sf:NODE/sf:SUCC/sf:item[8]/sf:NODE/sf:OBJ/sf:WINDOW/sf:PROC_CTRL/sf:NODE/sf:SUCC/sf:item[4]/sf:NODE/sf:SUCC/sf:item[1]/sf:NODE/sf:OUTATTR/sf:OUTATTR/T_CAPTION
/abapGit/sf:SMARTFORM/sf:VARHEADER/sf:item/sf:PAGETREE/sf:NODE/sf:SUCC/sf:item[1]/sf:NODE/sf:SUCC/sf:item[8]/sf:NODE/sf:OBJ/sf:WINDOW/sf:PROC_CTRL/sf:NODE/sf:SUCC/sf:item[6]/sf:NODE/sf:SUCC/sf:item/sf:NODE/sf:OBJ/sf:SECTION/T_CAPTION
and many more.
but yes, it really does seem to only occur under <T_CAPTION>
I'm sorry, I havened tried to solve it yet my self.

https://github.com/abapGit/abapGit/pull/6609 should fix it. please test


[12/10/2022]: Backend API (feat) - Project structure

Summary
Setup NodeJS MVC app structure for the BACKEND API for MOOCs platform issue #5
.....
backend_API/
  -- node_modules/
  -- src/
       -- controllers/
       -- routes/
           -- routes_handler.js
       -- models/
       -- db/
       -- test/
       -- utils/
             -- async_wrapper.js
             -- config.js
       -- .env
       -- app.js
       -- server.js
package-lock.json
package.json

Fixes issue #13
List of changes proposed in this PR (pull-request)

Added project structure for backend API
Added database connection
Setup MVC app structure

What should a reviewer concentrate their feedback on?

[x] Everything looks ok?

@BatoolMM  this is  the pull request to add the Backend API project structure as explained in issue #5.
Please review and merge to dev branch so others can pull and start working on the tasks

Thank you @RealRichi3 for your hard work!
Just to double-check, I just merged PR #14 for dev branch. Would you like to have your work at the same dev branch or I can create another one if you are working in different groups?

Thanks @BatoolMM, i would be better to work on a different branch, but we were hoping to discuss more on that tomorrow

I created a new branch called dev_team2 just in case you want to use it.

Perfect, but will the zoom session for tomorrow still hold?

Hi @RealRichi3, I am very sorry that this will be postponed till Tuesday at the same time. This was outside our control, but we are happy to support you and answer questions you have. I think the best way to resolve this PR is to create another one directed toward a dev_team2 branch. Let us know if you want to proceed using a different approach, @alswajiab and I will help you with it 🌻

Hi @RealRichi3, I am very sorry that this will be postponed till Tuesday at the same time. This was outside our control, but we are happy to support you and answer questions you have. I think the best way to resolve this PR is to create another one directed toward a dev_team2 branch. Let us know if you want to proceed using a different approach, @alswajiab and I will help you with it 🌻

Apologies, it was closed by mistake, it is re-opened now!

Hi @RealRichi3, I am very sorry that this will be postponed till Tuesday at the same time. This was outside our control, but we are happy to support you and answer questions you have. I think the best way to resolve this PR is to create another one directed toward a dev_team2 branch. Let us know if you want to proceed using a different approach, @alswajiab and I will help you with it 🌻

Alright, i guess we'll go with it. The issue i had was that, since the project plan has been set in issue #5, every pull request to the branch has to be reviewed so it'll be inline with the project plan, i'd like to suggest that i should review all pull request made to dev_team2, if it's ready for merge then i can tag you saying 'reviewed and ready to merge'.

Another idea.
Since the backend and frontend are part of the main plan in issue #2, we can just use one branch, then two separate folders in it, the front end work will be done in the frontend folder, then the code for backend will de done in backend. I think this will be good so we won't need to open so many branches.
This way it'll be one branch per proposal
For this proposal @sandyguide can review all PR related to the frontend issue #10
And I can review all PR related to backend api issue #5
Once reviewed we can just comment reviewed and ready to merge then you can merge. This way it'll be inline with the main proposal in #2 and collaboration for #10 and #5 would be more effective

I love the way you have been managing and organising the protject @RealRichi3. I created a new tag ready to merge. Feel free to use it and either me or @alswajiab will merge your PR. Let us know if you need any other thing!

Another idea.
Since the backend and frontend are part of the main plan in issue #2, we can just use one branch, then two separate folders in it, the front end work will be done in the frontend folder, then the code for backend will de done in backend. I think this will be good so we won't need to open so many branches. This way it'll be one branch per proposal
For this proposal @sandyguide can review all PR related to the frontend issue #10 And I can review all PR related to backend api issue #5
Once reviewed we can just comment reviewed and ready to merge then you can merge. This way it'll be inline with the main proposal in #2 and collaboration for #10 and #5 would be more effective

I agree with this, @BatoolMM, please merge the backend project set up PR to the dev  branch , so we commenced from there.

@RealRichi3 just to be sure, are we using the dev branch or dev_team branch

@sandygudie let's go for the dev_team

I love the way you have been managing and organising the protject @RealRichi3. I created a new tag ready to merge. Feel free to use it and either me or @alswajiab will merge your PR. Let us know if you need any other thing!

Thanks @BatoolMM, i Will make the PR to dev_team so we can start contributing

Also then, i will set up a frontend structure  for dev_team branch

Sorry it's dev_team2

@sandygudie it's dev_team2 not dev_team

@BatoolMM it's ready to merge now

I love the way you have been managing and organising the protject @RealRichi3. I created a new tag ready to merge. Feel free to use it and either me or @alswajiab will merge your PR. Let us know if you need any other thing!

@BatoolMM  how do we add the ready to merge tag?

Brilliant, it's merged now!

@RealRichi3 , let me know if you still can't access ready to merge tag!


Clarifies the order and avoid the vendor start with One instead of Zero

I put a tiny detail where we explicitly say that the parameter starts with one instead of zero.

Thanks, updated.


Some Python Implementations Short-Circuit Lines Causing Incorrectly Marked Missed Lines

Describe the bug
Very specific code constructions can cause lines to be missed by coverage, when they are fully executed.  This may be an underlying Python optimization that's causing the lines to be skipped.
To Reproduce
The following code, on some versions of Python, cause the return line to be marked as missing.  Playing around with this shows that any changes like getting rid of the try/except block, returning anything other than None, and adding a print statement before the return, cause the return line to be correctly marked.
import unittest
from typing import Callable, Optional

def func_under_test(*conditions: Callable[[], bool]) -> Optional[str]:
    """Function that skips lines."""
    try:
        for condition in conditions:
            if not condition():
                return 'Failed'
        return None
    except BaseException as err:
        return 'Failed with error: ' + repr(err)

class RunFuncTest(unittest.TestCase):
    """Run the test."""

    def test_first_true(self) -> None:
        """Test with just one argument."""
        self.assertIsNone(func_under_test(lambda: True))

    def test_first_false(self) -> None:
        self.assertEqual(
            'Failed',
            func_under_test(lambda: False),
        )

    def test_multiple_true(self) -> None:
        self.assertIsNone(func_under_test(lambda: True, lambda: True, lambda: True))

    def test_multiple_false(self) -> None:
        self.assertEqual(
            'Failed',
            func_under_test(lambda: True, lambda: False),
        )

    def test_error(self) -> None:
        def raise_err() -> bool:
            raise OSError('my failure')
        self.assertEqual(
            "Failed with error: OSError('my failure')",
            func_under_test(raise_err),
        )

Running this on Python 3.8.5 on Windows generates:
> python -m coverage run -m unittest coverage_test.py
.....
----------------------------------------------------------------------
Ran 5 tests in 0.001s

> python -m coverage report -m
Name               Stmts   Miss  Cover   Missing
------------------------------------------------
coverage_test.py      23      1    96%   12

Running this on Python 3.8.5 on Linux generates:
$ python3 -m coverage run -m unittest coverage_test.py
.....
----------------------------------------------------------------------
Ran 5 tests in 0.001s

OK

$ python3 -m coverage report -m
Name               Stmts   Miss  Cover   Missing
------------------------------------------------
coverage_test.py      23      0   100%

In my two environments, coverage debug sys generates:
Windows:
-- sys -------------------------------------------------------
                        version: 5.2.1
                       coverage: C:\dev\project\.venv\windows\lib\site-packages\coverage\__init__.py
                         tracer: -none-
                        CTracer: available
           plugins.file_tracers: -none-
            plugins.configurers: -none-
      plugins.context_switchers: -none-
              configs_attempted: .coveragerc
                                 setup.cfg
                                 tox.ini
                                 pyproject.toml
                   configs_read: -none-
                    config_file: None
                config_contents: -none-
                      data_file: -none-
                         python: 3.8.5 (tags/v3.8.5:580fbb0, Jul 20 2020, 15:57:54) [MSC v.1924 64 bit (AMD64)]
                       platform: Windows-10-10.0.20197-SP0
                 implementation: CPython
                     executable: C:\dev\project\.venv\windows\Scripts\python.exe
                   def_encoding: utf-8
                    fs_encoding: utf-8
                            pid: 17768
                            cwd: C:\dev\project
                           path: C:\dev\project
                                 C:\dev\Python38.5-x64\python38.zip
                                 C:\dev\Python38.5-x64\DLLs
                                 C:\dev\Python38.5-x64\lib
                                 C:\dev\Python38.5-x64
                                 C:\dev\project\.venv\windows
                                 C:\dev\project\.venv\windows\lib\site-packages
                    environment: -none-
                   command_line: C:\dev\project\.venv\windows\lib\site-packages\coverage\__main__.py debug sys
                sqlite3_version: 2.6.0
         sqlite3_sqlite_version: 3.31.1
             sqlite3_temp_store: 0
        sqlite3_compile_options: COMPILER=msvc-1924
                                 ENABLE_FTS4
                                 ENABLE_FTS5
                                 THREADSAFE=1

Linux:
-- sys -------------------------------------------------------
                        version: 5.1
                       coverage: /home/my/project/.venv/linux/lib/python3.8/site-packages/coverage/__init__.py
                         tracer: -none-
                        CTracer: available
           plugins.file_tracers: -none-
            plugins.configurers: -none-
      plugins.context_switchers: -none-
              configs_attempted: .coveragerc
                                 setup.cfg
                                 tox.ini
                                 pyproject.toml
                   configs_read: -none-
                    config_file: None
                config_contents: -none-
                      data_file: -none-
                         python: 3.8.5 (default, Jul 27 2020, 08:42:51) [GCC 10.1.0]
                       platform: Linux-4.19.121-microsoft-standard-x86_64-with-glibc2.2.5
                 implementation: CPython
                     executable: /home/my/project/.venv/linux/bin/python3
                   def_encoding: utf-8
                    fs_encoding: utf-8
                            pid: 20474
                            cwd: /home/my/project
                           path: /home/my/project
                                 /usr/lib/python38.zip
                                 /usr/lib/python3.8
                                 /usr/lib/python3.8/lib-dynload
                                 /home/my/project/.venv/linux/lib/python3.8/site-packages

I don't think I've ever seen the same version of Python optimize programs differently under different operating systems, so I'm not sure what's going on there.  Coverage.py tries to understand the optimizations that could happen, but it doesn't always get it right.  A "# pragma: no cover" comment might be the best way to deal with this.

The pretty code side of my brain was looking for a way to not stick no-cover lines over code just because the python runtime likes to optimize.
I was hoping to find some flag, environment variable, or other setting to force the Python runtime to not optimize, and instead force it to run over every line.  So far, I haven't found anything that forces the runtime to run this specific line.

I repeated this on:
(.venv) PS C:\Users\tagra\covpy> python --version
Python 3.9.0rc1
(.venv) PS C:\Users\tagra\covpy> coverage --version
Coverage.py, version 5.2.1 with C extension
Full documentation is at https://coverage.readthedocs.io

-- sys -------------------------------------------------------
                        version: 5.2.1
                       coverage: c:\users\tagra\covpy\.venv\lib\site-packages\coverage\__init__.py
                         tracer: -none-
                        CTracer: available
           plugins.file_tracers: -none-
            plugins.configurers: -none-
      plugins.context_switchers: -none-
              configs_attempted: .coveragerc
                                 setup.cfg
                                 tox.ini
                                 pyproject.toml
                   configs_read: -none-
                    config_file: None
                config_contents: -none-
                      data_file: -none-
                         python: 3.9.0rc1 (tags/v3.9.0rc1:439c93d, Aug 11 2020, 19:19:43) [MSC v.1924 64 bit (AMD64)]
                       platform: Windows-10-10.0.19041-SP0
                 implementation: CPython
                     executable: c:\users\tagra\covpy\.venv\scripts\python.exe
                   def_encoding: utf-8
                    fs_encoding: utf-8
                            pid: 16040
                            cwd: C:\Users\tagra\covpy
                           path: C:\Users\tagra\covpy\.venv\Scripts\coverage.exe
                                 C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_<IP_ADDRESS>_x64__qbz5n2kfra8p0\python39.zip
                                 C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_<IP_ADDRESS>_x64__qbz5n2kfra8p0\DLLs
                                 C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_<IP_ADDRESS>_x64__qbz5n2kfra8p0\lib
                                 C:\Users\tagra\AppData\Local\Microsoft\WindowsApps\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0
                                 c:\users\tagra\covpy\.venv
                                 c:\users\tagra\covpy\.venv\lib\site-packages
                    environment: PYTHONUSERBASE = C:\Users\tagra\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages
                   command_line: C:\Users\tagra\covpy\.venv\Scripts\coverage debug sys
                sqlite3_version: 2.6.0
         sqlite3_sqlite_version: 3.32.3
             sqlite3_temp_store: 0
        sqlite3_compile_options: COMPILER=msvc-1924
                                 ENABLE_FTS4
                                 ENABLE_FTS5
                                 ENABLE_JSON1
                                 THREADSAFE=1

I get 100% coverage on ubuntu with py3.9
-- sys -------------------------------------------------------
                        version: 5.2.1
                       coverage: /home/graingert/.virtualenvs/covpy/lib/python3.9/site-packages/coverage/__init__.py
                         tracer: -none-
                        CTracer: available
           plugins.file_tracers: -none-
            plugins.configurers: -none-
      plugins.context_switchers: -none-
              configs_attempted: .coveragerc
                                 setup.cfg
                                 tox.ini
                                 pyproject.toml
                   configs_read: -none-
                    config_file: None
                config_contents: -none-
                      data_file: -none-
                         python: 3.9.0rc1 (default, Aug 11 2020, 22:46:54) [GCC 9.3.0]
                       platform: Linux-4.4.0-19041-Microsoft-x86_64-with-glibc2.31
                 implementation: CPython
                     executable: /home/graingert/.virtualenvs/covpy/bin/python3.9
                   def_encoding: utf-8
                    fs_encoding: utf-8
                            pid: 1193
                            cwd: /mnt/c/Users/tagra/covpy
                           path: /home/graingert/.virtualenvs/covpy/bin
                                 /usr/lib/python39.zip
                                 /usr/lib/python3.9
                                 /usr/lib/python3.9/lib-dynload
                                 /home/graingert/.virtualenvs/covpy/lib/python3.9/site-packages
                    environment: -none-
                   command_line: /home/graingert/.virtualenvs/covpy/bin/coverage debug sys
                sqlite3_version: 2.6.0
         sqlite3_sqlite_version: 3.31.1
             sqlite3_temp_store: 0
        sqlite3_compile_options: COMPILER=gcc-9.3.0
                                 ENABLE_COLUMN_METADATA
                                 ENABLE_DBSTAT_VTAB
                                 ENABLE_FTS3
                                 ENABLE_FTS3_PARENTHESIS
                                 ENABLE_FTS3_TOKENIZER
                                 ENABLE_FTS4
                                 ENABLE_FTS5
                                 ENABLE_JSON1
                                 ENABLE_LOAD_EXTENSION
                                 ENABLE_PREUPDATE_HOOK
                                 ENABLE_RTREE
                                 ENABLE_SESSION
                                 ENABLE_STMTVTAB
                                 ENABLE_UNLOCK_NOTIFY
                                 ENABLE_UPDATE_DELETE_LIMIT
                                 HAVE_ISNAN
                                 LIKE_DOESNT_MATCH_BLOBS
                                 MAX_SCHEMA_RETRY=25
                                 MAX_VARIABLE_NUMBER=250000
                                 OMIT_LOOKASIDE
                                 SECURE_DELETE
                                 SOUNDEX
                                 TEMP_STORE=1
                                 THREADSAFE=1
                                 USE_URI

for a workaround, try this:
try:
    if all(f() for f in conditions):
        return None
except BaseException as err:
    return f'Failed with {err=}'
return "Failed"

I don't think I've ever seen the same version of Python optimize programs differently under different operating systems

I need to do a better job: this is a dup of #959. We still don't know why it is happening, or whether coverage.py should take it into account.


[5.2] Dry up managers resolvers

Dry up managers resolvers.

Nothing functional was changed.

Just got one review comment. :)

+1 for avoiding these inline assignments.

It was working before and it's so close to release I don't want to dig around making formatting type changes in case we break something.


Drop inline and forwardRef prop from DOM

Solved on project level https://github.com/taskany-inc/issues/pull/1849


a question about ODIoU loss

Great works! I got a question when reading paper:
how do you calculate IoU between Bp and Bg in ODIoU loss? Do you calculate axis-aligned IoU or rotated IoU (the backward seems complicated)?

Indeed, it's complex due to the gradient back-propagation, and IoU are calculated as non-axis-aligned. The code will be released in a few weeks and reveal all details.


Missing "description" field from JSON schema

[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2319bae2-e023-07fd-28e1-b0b10bf33a22
Version Independent ID: 0a00e889-20d4-9ef4-5935-15dc67059d61
Content: Lead management with an HTTPS endpoint - Microsoft commercial marketplace
Content Source: articles/marketplace/partner-center-portal/commercial-marketplace-lead-management-instructions-https.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @keferna
Microsoft Alias: keferna

@bondaq Thanks for the comment. Appreciate if you could provide more details on your ask and point to the document section for any changes required.

Hello.
In first JSON schema missing field “description”. As you can see, in JSON example we can see “description” of request, but in main schema it missing. It means what we can’t get argument “description” in Flow and transfer it in another instance.
Regards,
Anton.
From: SadiqhAhmed-MSFT<EMAIL_ADDRESS>Sent: Thursday, October 22, 2020 12:39 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Антон Алексенко<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Missing "description" field from JSON schema (#64808)
@bondaqhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbondaq&data=04|01|antona%40allware.ru|5a4977b3d23e40b6032108d8766e5b48|8e25bf6080db48469418ca9c8bac0232|0|1|637389563637588394|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|2000&sdata=eIjsGOOIyWJP7SK0qXjirx2jDEQ3bfo8RE47p6loPJM%3D&reserved=0 Thanks for the comment. Appreciate if you could provide more details on your ask and point to the document section for any changes required.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F64808%23issuecomment-714368168&data=04|01|antona%40allware.ru|5a4977b3d23e40b6032108d8766e5b48|8e25bf6080db48469418ca9c8bac0232|0|1|637389563637588394|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|2000&sdata=H0MZV2YzCeu2hNB6PjiR0qmzCGy0qLSrR8vqw0CsvFM%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FARPCJTGYT5OLDER44LCYIATSL74UTANCNFSM4S22734A&data=04|01|antona%40allware.ru|5a4977b3d23e40b6032108d8766e5b48|8e25bf6080db48469418ca9c8bac0232|0|1|637389563637598382|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|2000&sdata=ivO59vqyN5V2fspK5treHqlGN52EQJMZm1r54TAy2vs%3D&reserved=0.

@keferna Please review and updates as appropriate.

#reassign:trkeya

We've added additional code to address the Description field. It should post after 10:00 am PST today. Thank you for your patience and for contributing! #please-close

We've added additional code to address the Description field. It should post after 10:00 am PST today. Thank you for your patience and for contributing! #please-close


WIP: Revamping the structure of the repository.

This branch restructures the repository into more meaningful and appropriate directories and file names.

Appropriate directories have been made to contain the relevant files: Pages, Scripts, Styles, and Documentation.

Included basic CONTRIBUTING and CONVENTIONS documentation. Both documents will be expanded later on.

Removed the previous files, as the front-end is done from scratch.

The direction of the merge needs to be reversed.
Note:
The conflict was resolved by adding a blank README to your branch.


Command lines for keyboard shortcuts

Is there somewhere I can find how to execute actions such as clipboard copying via commands in the cmd prompt, instead of via keyboard shortcuts? I would like to write a short batch file to automate the copying process, but I would need the command lines to execute.

No, this is currently not possible.
I'm closing as duplicate of #399 (I guess).


Middleware and context

How would you access userId inside handler function?
https://github.com/senacor/azure-function-middleware/blob/ac1586b0a72aa6325358f0cf4eb9b50dbf50e729/example/src/test-jwt-authorization-function.ts#L6-L9

Hi @yozik04,
you can access the parsed JWT claims in the handler via context.extraInputs.get('jwt'). I have extended the example test-jwt-authorization-function.ts accordingly.
I hope this helps you.


Remove SFML submodule path and update README

-SFML submodule path is removed from .gitmodules
-README now includes instructions to clone ad build

Looks good!


Allow users to specify delimiter when copying to csv file

Currently, .to_csv() creates a comma-delimited file. But PostgreSQL's COPY TO statement allows users to specify any one-byte character (e.g., tab) via the DELIMITER parameter. We might add this option, but keep , as the default delimiter.
TODO:

[ ] Add a delimiter kwarg to .to_csv()
[ ] Write tests to confirm default behavior and example of specifying delimiter
[ ] Update docs

This is done!


Create site not happening through Chef but working through manual method

I am automating the creation of ArcGIS servers through Chef and it is not able to create the site and it fails at creating the directories in the network share path, but when I create the site through the browser and give the network path everything works out and it gets created, this is the error I get through chef:
[2018-02-02T02:54:07-05:00] FATAL: RuntimeError: arcgis_enterprise_server[Create ArcGIS Server site] (arcgis-enterprise::server line 115) had an error: RuntimeError: Failed to create the site. The specified configuration store location is not accessible. Ensure that the ArcGIS Server account has read and write access to the location.

The ArcGIS server account has privileges to create files on the network share and the user is a domain user and I am using the user name in this format: mydomain\username

arcgis-enterprise::server recipe does not create directories on  network shares.
To create and share the server directories  run arcgis-enterprise::fileserver recipe on the file server machine.

I am encountering an issue when I used the fileserver recipe:
default['arcgis']['fileserver'].tap do |fileserver|
   fileserver['shares'] = ['\\\\_ip_\\_share_name_\\arcgisportal\\']
end

The above is my atrribute file.
I am creating the files on the network share directly I am not mounting them anywhere:
    Mixlib::ShellOut::ShellCommandFailed
    ------------------------------------
    Expected process to exit with [0], but received '1'
    ---- Begin output of "C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NonInteractive -NoProfile -ExecutionPolicy Bypass -InputFormat None -File "C:/Users/PIDARC~1/AppData/Local/Temp/chef-script20180203-4824-12fbaym.ps1" ----
    STDOUT:
    STDERR: The syntax of this command is:
    NET SHARE
    sharename
              sharename=drive:path [/GRANT:user,[READ | CHANGE | FULL]]
                                   [/USERS:number | /UNLIMITED]
                                   [/REMARK:"text"]
                                   [/CACHE:Manual | Documents| Programs | BranchCache | None]
              sharename [/USERS:number | /UNLIMITED]
                        [/REMARK:"text"]
                        [/CACHE:Manual | Documents | Programs | BranchCache | None]
              {sharename | devicename | drive:path} /DELETE
              sharename \\computername /DELETE
    ---- End output of "C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NonInteractive -NoProfile -ExecutionPolicy Bypass -InputFormat None -File "C:/Users/PIDARC~1/AppData/Local/Temp/chef-script20180203-4824-12fbaym.ps1" ----
    Ran "C:\Windows\system32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NonInteractive -NoProfile -ExecutionPolicy Bypass -InputFormat None -File "C:/Users/PIDARC~1/AppData/Local/Temp/chef-script20180203-4824-12fbaym.ps1" returned 1

Try to set the attributes for fileserver recipe like this:
{
  "arcgis": {
    "run_as_user": "arcgis",
    "run_as_password": "XXXXXXXXXXXXX",
    "fileserver": {
      "shares": ["C:\\arcgisserver", "C:\\arcgisdatastore", "C:\\arcgisportal"]
    },
    "server": {
      "local_directories_root": "C:\\arcgisserver"
    },
    "portal": {
      "local_content_dir": "C:\\arcgisportal\\content"
    },
    "data_store": {
      "local_backup_dir": "C:\\arcgisdatastore\\backup"
    }
  },
  "run_list": [
    "recipe[arcgis-enterprise::fileserver]"
  ]
}

Thanks Pavel.
But the thing is the shared drives are already created and the UNC path is given to me --> \<IP_ADDRESS>\Volume1 and on this I want to create the config stores and directories.
Why is that no permissions error coming in Chef but not when I manually create the site and pass the UNC for my shared drive and directories?

RuntimeError: Failed to create the site. The specified configuration store location is not accessible. Ensure that the ArcGIS Server account has read and write access to the location.

For some reason the specified configuration store is not accessible form the domain user account.
If the config store path is the same in both cases, there should be no difference between the manual site creation and the one called by Chef.

I did this for portal it create the site doesn't throw the error for that. The domain user has been given access to use that network share.

Set chef-client's log level to debug (-l debug) when you run chef-client.
With debug logging enabled, Chef will log all the requests to ArcGIS Server Admin API.
Compare the request to the manual one and try to create all the directories used in the createSite request.

I did this I used a directory resource to create the necessary directories on to the shared file and then let the node converge. Will this approach not help?

Yes. That should help. You just need to know the exact paths of the directories used by createSite request.

Alright thanks! And also the new server recipe cookbook is causing this weird error when authorizing the servers. In the run list I have mentioned the authorization_file_version is 10.5:
C:\Program Files (x86)\ESRI\License10.6\\keycodes not available Retrying software authorization.'

I ran into the same issue today when I tried to test my wrapper against the v3.2 cookbook.  It seems like you will have to override over node.['arcgis']['server']['keycodes'] with the correct path for 10.5.x:
node.default['arcgis']['server']['keycodes'] = ENV['ProgramW6432'] + "\\ESRI\\License#{node['your_wrapper']['arcgis']['version'].to_f}\\sysgen\\keycodes"
I am pretty new to Chef so my understanding is probably wrong.  But it seems like the the precedence is preventing the attribute from being set correctly, so it will always default to 10.6.

@kawenglou - Thanks the above method you suggested worked well!
@pbobov - How do I compare the request to the manual one and try to create all the directories used in the createSite request? I did the debug level but the request was encoded in format that's not understandable, is there a documentation I can refer to for the request?


Missing translations with current Mongoid

Since Mongoid 2.3 or 2.4, I'm seeing messages like translation missing: en.mongoid.errors.models.user.attributes.full_name.presence instead of can't be blank etc. At first I thought it's a Mongoid issue but with client_side_validations disabled the correct messages are shown.

Fantastic

I've set aside some time to tackle a 3.2 release this weekend. This will break out the mongoid functionality in the client_side_validations-mongoid gem. This will allow be better maintenance and faster updates when stuff like this happens.

Thx for the update. If you think I can help, I'm willing.
May I have your email address? -Ken
On Fri, Jan 20, 2012 at 12:07 AM, Brian Cardarella <
<EMAIL_ADDRESS>
wrote:

I've set aside some time to tackle a 3.2 release this weekend. This will
break out the mongoid functionality in the client_side_validations-mongoid
gem. This will allow be better maintenance and faster updates when stuff
like this happens.

Reply to this email directly or view it on GitHub:
https://github.com/bcardarella/client_side_validations/issues/253#issuecomment-3580328

gem "client_side_validations-mongoid", :git => "
https://github.com/dockyard/client_side_validations-mongoid.git"
ActionView::Template::Error (couldn't find file 'rails.validations'
I did not re-run
rails g client_side_validations:install
Is this necessary?
On Sun, Jan 22, 2012 at 3:47 AM, Brian Cardarella <
<EMAIL_ADDRESS>
wrote:

This is now fixed in the beta version of client_side_validations-mongoid
https://github.com/dockyard/client_side_validations-mongoid

Reply to this email directly or view it on GitHub:
https://github.com/bcardarella/client_side_validations/issues/253#issuecomment-3602481

Nevermind--I see I need to retain:
gem client_side_validations
On Sun, Jan 22, 2012 at 9:12 AM, ken keller<EMAIL_ADDRESS>wrote:

gem "client_side_validations-mongoid", :git => "
https://github.com/dockyard/client_side_validations-mongoid.git"
ActionView::Template::Error (couldn't find file 'rails.validations'
I did not re-run
rails g client_side_validations:install
Is this necessary?
On Sun, Jan 22, 2012 at 3:47 AM, Brian Cardarella <
<EMAIL_ADDRESS>
wrote:

This is now fixed in the beta version of client_side_validations-mongoid
https://github.com/dockyard/client_side_validations-mongoid

Reply to this email directly or view it on GitHub:
https://github.com/bcardarella/client_side_validations/issues/253#issuecomment-3602481

Currently the Mongoid plugin requires the beta of version 3.2.0. Use the following in your Gemfile
gem 'mongoid`
gem 'client_side_validations', '~> 3.2.0.beta.1'
gem 'client_side_validations-mongoid', '~> 2.4.0.beta.2'

This only supports Rails 3.1 and greater.

Works!
BTW, my background is at www.kellerapps.com
I'm looking for a great Rails engr to join me in my new startup. -Ken
On Sun, Jan 22, 2012 at 9:16 AM, Brian Cardarella <
<EMAIL_ADDRESS>
wrote:

gem 'client_side_validations', '~> 3.2.0.beta.1'
gem 'client_side_validations-mongoid', '~> 2.4.0.beta.2'


GIF image with same uri do not rendered sometimes.

FastImage just saved my life, the performance is very good.
However, i found that GIF image do not rendered sometimes.
Please noted that the same image were used in both testing.
First Test:

Render the same GIF image with RCTImage, FastImage in the same page.
FastImage component renders placeholder which equals to the image
FastImage component do not renders the GIF image. (Although there is no any warning during the loading, I am sure that it is not caused by the image file size because i was waiting for the loading for 5 minutes and RCTImage just successfully rendered the gif image within 5 seconds)

Second Test:

Render the same GIF image with RCTImage, FastImage in the same page.
FastImage component renders placeholder which equals to the image
FastImage component renders the GIF image

@tikkichan4 Thanks for the bug report.
I'd need more information to be able to look into it. Can you reproduce this issue? Is there any gif in particular that causes this issue?


Log HTTP Responses

As part of supporting debugging we would like to have HTTP Response Logged

Is this something we can enable/disable via CLI?

I think that for info level, we should output the http response status code. Also, I think that we should add a info log level for every record read:
https://github.com/infinyon/http-sink-connector/blob/674d331e02b8a9c435e4b2e48ab58d9a9f3c2df7/crates/http-sink/src/main.rs#L19 Maybe we can print there the record size.

With regards to toggling logs on the fly I think we can use this approach from the same crate we use for logging:
https://docs.rs/tracing-subscriber/latest/tracing_subscriber/reload/index.html
We just have to design how the logging is switched from the CLI

I think that for that, we should create a mechanism to set environment variables on connectors. Maybe that way we can just set RUST_LOG with the value we want in the rust connectors

We wouldn't want to add Info level logs for high frequency message, as it will impact performance and it will burry important messages such as errors. Instead we should have a way to add debug, and implement stats for counters such as incoming records, outgoing records, and whatever else is relevant for troubleshooting.

Even a http servers typically uses an info log for reporting status code of each request, I think that it should not impact performance

Log level should be up to the user since it will impact performance and incur additional costs.


Youtube plugin breaks on the wrong user input URL and doesn't recover

Hi,
My needs are pretty simple, to allow user to preview youtube video by url.
Everything is good until user puts some incorrect url. More precise not youtube url but just some junk.
The player gets NaN in all timers etc., which I can live with, but when user puts the correct youtube url the player does not recovers and just stays in the error state.
Any suggestions?

Can you open this issue in the Youtube plugin repository?
https://github.com/NamPNQ/bower-videogular-youtube/issues
Thanks


[BUG] New version of SDK did not using Maven relocation for changed Maven coordinates

Describe the bug
New version of SDK did not using Maven relocation for changed Maven coordinates
Expected behavior
For a user of the previous version of the SDK it is unclear the newer version of the SDK has been relocated. Maven supplies a mechanism to do so, see https://maven.apache.org/guides/mini/guide-relocation.html
Please perform the steps in the guide mentioned above so users get a relocation message

Hi @mnriem Can you say explicitly which version you're referring to and which SDK? We have a number of libraries both in preview and released and I want to make sure I understand the request here.

Hi @loarabia,
I used the following dependency for the new Cosmos SDK
<dependency>
  <groupId>com.microsoft.azure</groupId>
  <artifactId>azure-cosmos</artifactId>
  <version>3.0.0</version>
  <scope>compile</scope>
</dependency>

And it was previously available using:
<dependency>
    <groupId>com.microsoft.azure</groupId>
    <artifactId>azure-cosmosdb</artifactId>
    <version>3.0.0-beta-3</version>
</dependency>

And no relocation was used there.
To facilitate relocation one should release an additional POM file at the old coordinates as follows:
<project>
  <modelVersion>4.0.0</modelVersion>
  <groupId>com.microsoft.azure</groupId>
  <artifactId>azure-cosmosdb</artifactId>
  <version>3.0.0</version>
  <distributionManagement>
    <relocation>
      <groupId>com.microsoft.azure</groupId>
      <artifactId>azure-cosmos</artifactId>
    </relocation>
  </distributionManagement>
</project>

If any other new SDK has been released with a new groupId or artifactId it would suffer from the same problem. If this gets fixed folks get warned automatically. Note once you have done the relocation for this 3.0.0 version the next version after that obviously would not need the relocation.

Thanks @mnriem  for the update. I've tagged this so the right part of the team should take a look.


Adds ability to use a custom date format

A date format string can be entered as the prop dateFormat to the DatePicker element. If you do not enter this prop, it will default to YYYY-MM-DD, which was previously the only valid format allowed. The requirements of one of my current projects was that I needed a different format, so I put in the issue https://github.com/Hacker0x01/react-datepicker/issues/45. Please let me know if you have any questions. Thanks!

Closing this since #48 is already merged.


Admin Login Page

Hey , I came across this issue and would love to work on it.
Could you confirm if it's still open? If available, could you please assign it to me?