[Hub Generated] Review request for Microsoft.RecoveryServices to add version stable/2021-06-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[x] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@asmaskar , I don't see the changes for swagger on top of the base version.

Hi @raych1  The version is added for s360 completeness. There is no change in this version. Its just we had to add to version to avoid being flagged

Agree, there are no changes. @raych1, this should not have been flagged for ARM review. Does your team own the automation for that, or do you know who should look at this issue?

@raych1 Can you please merge it if things are fine

Agree, there are no changes. @raych1, this should not have been flagged for ARM review. Does your team own the automation for that, or do you know who should look at this issue?

@mentat9 , waitForARMFeedback label was added due to the checkbox is checked in ARM API Review Checklist.


EPMLABSBRN-331: remove commit-msg git hook

EPMLABSBRN-331
It's time to get rid of blocking tools and put responsibility on developers and reviewers.
Not sure if got rid of Husky correctly. Feel free to make any comments.

@Mikhail-Sosnin why you removing frontend hooks?

@Mikhail-Sosnin any concerns, why you want to remove commit message stylings?


Custom theme variables not working - 14.0.0

🐛 The bug
The module doesn't recognizes custom theme variables on css files, downgrading to v0.13.1 works.
🛠 To reproduce
https://stackblitz.com/edit/nuxt-starter-qx18gq?file=nuxt.config.ts
🌈 Expected behaviour
Expects the module recognizes and apply ionic variables overwritten in css files.
ℹ Additional context
No response

We should investigate whether this is an upstream issue with Ionic, as this module has never done anything with the CSS variables, just documented how to set them.

Updating, apparently with Vue/Vite it continues to work, looking at module code now...
https://stackblitz.com/edit/github-gxs7ou-myj5vx?file=src%2Ftheme%2Fvariables.css

Thank you ❤


Segment-Quantcast integration

Hey team, we would like to enable a free marketing service called Quantcast - this should include adding a library to the eigen project and enabling the integration on the segment site: link to Segment integration docs.
cc @jessekedy @katarinabatina

@anipetrov could you tell us more about the service?

@jessekedy will be able to provide the most context

This is a service that give us aggregate (not individual) demographic info on our visitors (regardless of logged state), such as:


Adding Twemoji to sites like Tumblr

Is there any way of me adding Twemoji to sites like my Tumblr blog?
I'm wondering this, because if I use emojis on my blogposts, they appear as white boxes, rather than emojis. I was unsure on whether there was a way of implementing Twemoji onto sites like Tumblr, so that this doesn't happen.
If so, what do I need to put into my code and where?
Thanks.

can you modify the HTML template of your blog? If so, you can add the following on the header
<script defer src="//twemoji.maxcdn.com/twemoji.min.js"></script>
then you can add the following CSS style
<style>
img.emoji {
   height: 1em;
   width: 1em;
   margin: 0 .05em 0 .1em;
   vertical-align: -0.1em;
}
</style>

then you can add a script like the following one
<script>
document.addEventListener(
  'DOMContentLoaded',
  function DCL(e) {
    document.removeEventListener(e.type, DCL, false);
    twemoji.parse(document.body);
  },
  false
);
</script>

You can eventually be more specific during the parsing so that you'll grab only posts and/or comments instead of the entire body, resulting in probably better performance.
<script>
document.addEventListener(
  'DOMContentLoaded',
  function DCL(e) {
    document.removeEventListener(e.type, DCL, false);
    for (var el = document.querySelectorAll('div.post-content'), i = el.length; i--; twemoji.parse(el[i]));
  },
  false
);
</script>


Explore providing a rayon::ThreadPool to milli::update::IndexerConfig

I am assuming, by default, milli does all indexing on the same thread. If we pass IndexerConfig a ThreadPool, I am assuming indexing speed will increase several times over, which is huge, as indexing speed for a large number of documents is a big pain point currently.

We can use the exercises_test.dart file to benchmark before and after.

Didn't see any noticeable improvements over supplying our own ThreadPool. Also, I realized that what I perceived as slow was a result of compiling for debug. Compiling for release has no issue (I got 10x faster in my benchmark!).


Adding Local Promise Store

Supports IndexedDB
Local Promise Store

If I am not mistaken, the read modify write logic is not wrapped in indexed transactions, potentially leading to missed writes


Viewing contacts does not update the main viewing pane

The command in question: view in/c index/ANY_VALID_INDEX


command to list secrets for a worker

wrangler should support a command that allows the user to view a list of secret bindings configured on their worker.
Syntax
$ wrangler secret:list

Depends on
add endpoint definition to cloudflare-rs for calling the GET script bindings endpoint

Maybe add a note on whether env scoping will be supported (I think it's overkill for list)

Maybe add a note on whether env scoping will be supported (I think it's overkill for list)

actually env scoping will be necessary, since every env is its own script


add comma in pt.json

Could I get you to submit another PR with new translations? See: https://github.com/primefaces/primelocale/blob/main/en.json


Upstream TCP connection buffer and read buffer limits (#150).

As with fd58242, but on the upstream cluster side.

Sorry my approval is premature, I clicked the wrong button.

@htuch +1 looks good, but I think there are merge conflicts now.


Develop

Related issues
#2265 Create translation for Chinese #2265
Short description and why it's useful
Add Chinese lanuage Translate.I Tested it on Default themes.
Switch to Chinese,Modify the local.json or default.json as this:
"i18n": {
"defaultCountry": "CN",
"defaultLanguage": "CN",
"availableLocale": ["en-US","de-DE","fr-FR","es-ES","nl-NL", "jp-JP", "ru-RU", "it-IT", "pt-BR", "pl-PL", "cs-CZ","zh-cn"],
"defaultLocale": "zh-cn",
"currencyCode": "RMB",
"currencySign": "Y",
"currencySignPlacement": "preppend",
"dateFormat": "HH:mm D/M/YYYY",
"fullCountryName": "China",
"fullLanguageName": "Chinese",
"bundleAllStoreviewLanguages": true
},
Screenshots of visual changes before/after (if there are any)
NO
Screenshot of passed e2e tests (if you are using our standard setup as a backend)
Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility)
[ ] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

Contribution and currently important rules acceptance

[x] I read and followed contribution rules
[x] I read the TypeScript Action Plan and adjusted my PR according to it
[x] I read about Vue Storefront Modules and I am aware that every new feature should be a module

That is my pleasure


Fix duplicate URI escape

Some Time ago when I adopted 2FA for our forum I noticed that it was listed as "TripleA%20Forum" instead of "TripleA Forum".
the replace(/\s/, '%20') part replaces all spaces with %20 which is then encoded by encodeURIComponent to %2520, so that is being removed

@julianlam Interestingly enough this issue still persists after this change.
Could it be the case that meta.config.title already stores the title URI encoded?

Does this happen with certain devices? All devices?
The site title (with a space) is captured correctly on my device (FreeOTP on Android)

I'm using Google Authenticator, but I checked the encoded URI directly and it still contains the %25 part, so the problem should occur regardless of the app.
Is there a way I could check the stored forum title in the database for comparison?

I checked the encoded URI directly and it still contains the %25 part

What did you mean by this, that you looked at the text contained in the QR code?

this, that you looked at the text c

What did you mean by this, that you looked at the text contained in the QR code?

Yes.
Will check the db thingy later this day

Yes.

I think that's to be expected, since a QR code is essentially a URI (at least, for as far as the OTP apps are concerned), so it should be escaped.

A %20 is expected but not a %2520

Indeed! So you'll need to help me to identify where the %2520 (or %20) is coming from

After connecting to the db db.objects.find({ _key: 'config' }).pretty() revealed that title is indeed TripleA Forum:
"title" : "TripleA Forum",

However I realized that in the dashboard it says "Forum was last restarted 6 months ago", even though I know I restarted it a lot of times since then.
Looking at the forum logs I noticed the recent restart attempts were interrupted by a permission denied error because some folder belonged to root instead of the nodebb user.
So I fixed the permission problem which solved this issue and also fixed https://github.com/NodeBB/nodebb-plugin-write-api/issues/134#issuecomment-725884950
Nothing more to discuss here :)


Agenix integration

AFAICT This is mostly a documentation thing. But with divnix/devlib#2 we can now add agenix to the devshell.

I did testing of secrets between agenix and sops-nix.
Maybe those commands will help users to use agenix or sops easily.
https://github.com/GTrunSec/nixops-infra-land/blob/b1fb9983f9ce6a58691e87410827876167fcfc35/devshell.toml#L16
Default setting of ssh:

https://github.com/GTrunSec/nixops-infra-land/blob/b1fb9983f9ce6a58691e87410827876167fcfc35/profiles/core/core-services.nix#L4

what do you think of this suggestion?

Thanks for the suggestions!

https://github.com/GTrunSec/nixops-infra-land/blob/b1fb9983f9ce6a58691e87410827876167fcfc35/devshell.toml#L16

That looks useful, but for agenix I don't know if its warranted to add an entire command in the template to get a hosts public key. Its not something most people would do often. Although more pressing, this version of the template doesn't allow as much devshell customization. Something I would like to fix eventually.

I think we can do this here too
bors try

https://github.com/GTrunSec/nixops-infra-land/blob/b1fb9983f9ce6a58691e87410827876167fcfc35/devshell.toml#L16

I'm thinking about this again. And I realize the command is not agenix specific and generally pretty useful. I think it would be a good addition to the shell and can be done in in the library itself. Would you mind making a PR there to add it?

I think it would be a good addition to the shell and can be done in in the library itself. Would you mind making a PR there to add it?

Sure, by the way, Do you have a plan to add the sops-nix module for extending secrets?

Sure, by the way, Do you have a plan to add the sops-nix module for extending secrets?

Not currently, I also don't entirely understand how to use sops-nix. I opted for agenix here due to simplicity.
For the most part secrets management is documentation, so I'm sure a doc addition to explain sops-nix usage with devos would be very welcome. We could discuss adding sops-nix to the template by default with such a change.

When comparing sops-nix and agenix, I cannot see how sops-nix could outperform agenix or even complement with additional features.
Maybe there is a complementary element which I havn't been able to recognize, yet.
As such, I would think we can relatively safely limit our attention to agenix. No?

Maybe there is a complementary element which I havn't been able to recognize, yet.

This is what I was thinking. And it looks like @GTrunSec's repo shows that. They used agenix and sops-nix together. At the very least I think documentation on how to use sops-nix with devos would be good. And I would be open to discussion on how it can be integrated in the template along with agenix.
Also I know we are considering multiple templates in the future. So maybe one of those could have sops-nix.

Agree, while let's remember:
As long as we maintain clear, and well-argumented preferences, any documented alternative still is pareto-dominated, as it doesn't burden an unfeasible choice especially on our newcomer user persona (let's him "Novatus" from now on).

What's holing this back? I think it is a very useful improvement.

I think its ready to go, feel free to merge. I think devshell additions and any other agenix features can be added in separate PR's.

I'm putting on my "Novatus" cap, while I start to review our documentation.

I think more extensive doc changes should target develop. A little to prevent merge conflicts, but also to improve the review process of doc changes.

bors r+

bors r+

Looking at it from https://github.com/divnix/digga/pull/20 angle, we probably wanted to implement some of the devshell enhanceents through https://github.com/divnix/digga/pull/11 for agenix. :stuck_out_tongue_winking_eye:

@GTrunSec's comments were one of my main motivations for divnix/digga#11. I would be for adding add a devshell.toml to this repo.


[Bug] Crash due to assertion in firebase::rest::RequestJson::UpdatePostFields

[REQUIRED] Please fill in the following fields:

Pre-built SDK from the website or open-source from this repo: self-built
Firebase C++ SDK version: 8.8.0
Problematic Firebase Component: Remote Config
Other Firebase Components in use: Auth
Platform you are using the C++ SDK on: Windows
Platform you are targeting: desktop

[REQUIRED] Please describe the issue here:
We have one user for whom the app crashes on launch due to the assertion in request_json.h:68 here:
https://github.com/firebase/firebase-cpp-sdk/blob/f54d7c08cfcd17c26a3da6abc847adb8cc013962/app/rest/request_json.h#L64-L68
Following is the full stack trace:
OS Version: Windows 10.0.19044 (1586)
Report Version: 104

Crashed Thread: 11440

Application Specific Information:
Fatal Error: 0x40000015 / 0x00000001

Thread 11440 Crashed:
0   Redacted.exe                    0x7ff732b7fcb6      std::vector<T>::_Tidy
1   ucrtbase.dll                    0x7fff01171880      raise
2   ucrtbase.dll                    0x7fff01172850      abort
3   Redacted.exe                    0x7ff732af9e01      firebase::DefaultLogCallback (log.cc:81)
4   Redacted.exe                    0x7ff732af9fbb      firebase::LogMessageWithCallbackV (log.cc:145)
5   Redacted.exe                    0x7ff732af9ee9      firebase::LogAssert (log.cc:196)
6   Redacted.exe                    0x7ff732c21c98      firebase::rest::RequestJson<T>::UpdatePostFields (request_json.h:68)
7   Redacted.exe                    0x7ff732b9c408      [inlined] firebase::remote_config::internal::RemoteConfigRequest::UpdatePost (remote_config_request.h:90)
8   Redacted.exe                    0x7ff732b9c408      firebase::remote_config::internal::RemoteConfigREST::SetupRestRequest (rest.cc:141)
9   Redacted.exe                    0x7ff732b9bb1a      firebase::remote_config::internal::RemoteConfigREST::Fetch (rest.cc:71)
10  Redacted.exe                    0x7ff732b38fc8      firebase::remote_config::internal::RemoteConfigInternal::FetchInternal (remote_config_desktop.cc:587)
11  Redacted.exe                    0x7ff732b39425      <lambda>::<T> (remote_config_desktop.cc:174)
12  Redacted.exe                    0x7ff732b3a121      firebase::callback::CallbackVariadic<T>::RunInternal (callback.h:341)
13  Redacted.exe                    0x7ff732b2c84d      firebase::scheduler::Scheduler::TriggerCallback (scheduler.cc:189)
14  Redacted.exe                    0x7ff732b2c5e3      [inlined] bool (shared_ptr.h:128)
15  Redacted.exe                    0x7ff732b2c5e3      firebase::scheduler::Scheduler::WorkerThreadRoutine (scheduler.cc:169)
16  Redacted.exe                    0x7ff732b3460e      [inlined] std::invoke (type_traits:1488)
17  Redacted.exe                    0x7ff732b3460e      std::thread::_Invoke<T> (thread:55)
18  ucrtbase.dll                    0x7fff01121bb1      thread_start<T>
19  KERNEL32.DLL                    0x7fff02397033      BaseThreadInitThunk
20  ntdll.dll                       0x7fff033a2650      RtlUserThreadStart

It seems like some flatbuffer structure is invalid, causing GenerateText() and thereby the assertion to fail. I’d appreciate any input on how to debug this further.
Steps to reproduce:

Set system language to French:

Open Windows 10 settings
Select "Time & language" section
Click "Add Language" and select "Français (France)" / "French (France)"
Make French the primary language by moving it to the top of the list of preferred languages
Reboot

Initialize Firebase Remote Config

Hi @triplef,
do you have any information about the data payload that is being transfered by RemoteConfig?
Thanks!

Unfortunately we don't, since this is on a customer's machine that we're not able to debug directly. Is there any way to eg. enable logging of data payloads to debug this further?

We managed to reproduce the issue on a French system and found that builder->json had the following truncated value:
{
  appInstanceId: "<redacted>",
  appInstanceIdToken: "<redacted>",
  appId: "1:<redacted>:ios:<redacted>",
  languageCode: "fr_FR",
  platformVersion: "2",
  timeZone: "Paris, Madrid (heure d

The full time zone name should be Paris, Madrid (heure d'été), so it looks like the string is getting incorrectly truncated at the single quote.

Maybe the GetTimezone() function should always return the time zone in English, which can be done by first calling SetThreadUILanguage() as suggested here.
Also I noticed that this method currently returns the Windows time zone name. If any callers expect an IANA identifier it should be converted as discussed here.

So the steps to reproduce this are very straightforward:

Set system language to French:

Open Windows 10 settings
Select "Time & language" section
Click "Add Language" and select "Français (France)" / "French (France)"
Make French the primary language by moving it to the top of the list of preferred languages
Reboot

Initialize Firebase Remote Config

As far as we can see, this currently blocks using Remote Config on any system set to French (or other languages that might have similarly localized time zone names). I’d appreciate if this could be looked into.

Thank you very much. I can confirm that the crash is no longer reproducible with v11.2.0.

Unfortunately I spoke too soon. We’re seeing this crash #1367 on various machines in our testing with v11.2.0.


Implementation of PredefinedSectionComponent class that creates a part of a report

Class is to implement in guide.en.py module that creates a part (reporting component - formally child of ReportComponent class) of guide for reporting module.

Class was checked by the following call:
from dynpy.utilities.components.guides.en import PredefinedSectionComponent
PredefinedSectionComponent(None);


Added: Tags (Categories) for journal + Filter Function

This PR closes #9
It adds the following:
Assign tags to journals :

Journals can have multi tags now assigned to them.
Both back-ends have support for them
Create/Edit journals prompt has a new text field in it to add the tags as a comma-separated list
Tags appears in the journal list in extra lines below the date
The change in SQLite back-end isn't backward compatible. Once a database file is opened used the new version, it can't be open again using the older ones

Filter Function:

New filter prompt is added with the keybinding f while journals list is in focus.
In filter prompt user can select one or many tags to filter the journals using them.
In addition to that, user can set the matching logic for the filter (If all criteria should be met or of any one of them) with keybinding r inside the filter prompt
Filter can be reset with the keybinding x
All edge cases should covered here. Like if no entries have a tag used inside the filter, it should be removed automatically from the filter, and the filter will be reset if it doesn't have tags any more

@kevinmatthes I'll add merge this to master but I wont create a new release for it on crates.io until I spend sometime using it to make sure it has nor problems anymore and has some extra enhancements I'm planning to add to it next.
If you want to use the nightly version, please make sure to back-up your Back-ends before applying the new version to them to avoid any data-lose in case something went wrong


Initialize Game on Cutscene Skip

Developer Tavi implemented an option to skip cutscenes by pressing the "Enter" key. While this does skip the cutscene, none of the runners spawn and the game does not initialize till the full duration of the cutscene passes. This is due to some hard coded timers in the code preventing initialization till the timer ends (which corresponds to the duration of the cutscene). This story will fix this discrepancy by correctly initializing and spawning AI and the player.

[ ] #305
[ ] #306

Overall, this story was pretty simple to complete after doing research of understanding the existing methods. Fortunately with my experience with spawning work, I knew where the timers lie so it was easy to intercept them and apply the fix to both relevant maps. Total time spent on this was about 1.5 hours


API Reference

The APIs are getting more and more different from the JS version. It'd be great to have some format of documentation. What are the best options? ocamldoc?
cc @chenglou @jordwalke

Would be nice to have the docs generated with https://github.com/jaredly/docre

yes, absolutely. docre works for the most part so thankfully it's not the issue of tooling anymore.

What is the current status of this? At Rebolt, for time being, we have created a documentation covering most of the APIs by hand. While I agree it would be beneficial to have automatically generated documentation, it would be great to still have a hand-made section containing guides and getting started samples.
Let me know where we are here and how we can help.

It depends how far we want to go with this. It'd be best to generate documentation using odoc because it's super good. That said, even though odoc does support Reason, there's no bucklescript runner. If we want to have proper docs for future projects, I'd say, let's try to run odoc with bucklescript. For now though, the docs generated by hand are good enough.

cc @ryyppy

@wokalski Yeah, I would suggest bear with us a little bit longer until we get the odoc story a little bit more polished / documented ... at some point we would like to try to generate the API docs of reasonml.github.io with odoc as well
cc @rizo

I think we are a good testing ground. Even if you have something sorta-kinda-hacky working, I'd love to take a look at it.

@wokalski okay, let's keep this issue open while we are investigating. I will try to move the documentation from Rebolt onto here in the meantime.

awesome.

@ostera has added BuckleScript support for odoc. That's a great news!

Interesting! That sounds like an ideal plan for remote hackathon one day? :fire:

I'm reopening since the module reference is not accessible from the main docs page.

Oh there's Status page link from the main docs -> https://reasonml-community.github.io/bs-react-native/BsReactNative/status.html
Links itself don't work due to issues with namespacing - no easy fix for now, unless we do it manually (which is probably what I am going to do in this case).

Ok, is there an issue for that already? If not, we can create a new one or use this one.

Not a dedicated one tho. There's #260 PR, but I am wondering whether we
should do manual links vs manual namespace for now.
Don't think it makes sense to change the workflow and have added
development effort for the sake of automatically generated documentation.
I'd rather fix it in there.
On Mon, 1 Oct 2018 at 11:51 Wojtek Czekalski<EMAIL_ADDRESS>wrote:

Ok, is there an issue for that already? If not, we can create a new one or
use this one.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/reasonml-community/bs-react-native/issues/14#issuecomment-425850690,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWcxsYQk33yOY4upqegNBKXqRElD2sOks5ugeW_gaJpZM4NWX8B
.

I'll do it right now manually and open up a new PR that closes this issue.

Sounds great!


FEATURE - Added an option for ZPClass description add custom information

This would be useful in some classes like leech zombie, so you will no longer have to update manually the classe description (500 HP)

I will do this later after the gamemode update release


Support resuming of image/diff pulls

If the engine is pulling an image or diff, and the connection is interrupted, the download will restart from the beginning instead of resuming from the last chunk of data that was downloaded successfully.
Steps to reproduce:

Setup any device with OS v2.80.0 or older
Deploy an application release large enough to not download immediately, such as 1 GB or more
As the device is downloading the image, disable its connection to the internet. If it's setup to use a Linux machine as a router, you can do this with sysctl -w net.ipv4.ip_forward=0
Wait a few seconds, and re-enable the connection. The logs will show that the connection was reset, as well as: Delta failed with Error: failed to register layer: Error processing tar file(exit status 1): unexpected EOF

Layer diffs reduce the download bandwidth of deploying releases, but large diffs on unreliable networks can still be onerous and expensive to deploy, as any interruptions force it to start from the beginning. Instead, the engine should download the image tarball in chunks, and resume the download from the last successful chunk when possible.

Related: https://github.com/distribution/distribution/issues/762
https://github.com/moby/moby/pull/28348

@jakogut Do you have some estimations as to when we can expect this to be deployed?

@lmbarros Is working on this, my understanding is that he's working on some tests to ensure there are no regressions.

@joehounsham Resuming image downloads after network failures were implemented by this PR: https://github.com/balena-os/balena-engine/pull/253. We might need to adjust things like the number of times we retry and the amount of time we wait between retries, but otherwise I hope this to be a good improvement.
It's available in balenaEngine 19.03.23 and balenaOS 2.82.7. I am not sure this OS version is available in production for any device type, but I know we released it on staging for Xavier NX SD-CARD. I am aware of one customer trying it currently on staging and AFAIK, so far so good.

This has already been released so closing.


Error while pulling image: Get [...] authenticationrequired

Rancher Version:
?
Docker Version:
1.10.3
OS and where are the hosts located? (cloud, bare metal, etc):
RHEL7 as underlying system (for now)
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Hosts
Add Hosts
vSphere

OS ISO URL: Internal Link to boot2docker iso
Network: Custom Network
Data Store: Custom Data Store
Also Added Environment Variables (HTTP_PROXY & HTTPS_PROXY)

Results:
Error while pulling image: Get https://index.docker.io/v1/repositories/rancher/agent/images: authenticationrequired
Expected:
I expected the hosts to come up alive.

can your hosts do a docker pull without authenticating? You might have a proxy in your environment preventing connecting directly to dockerhub?

Docker pull works fine but we upgraded to the new ESXi, creating everything again and facing new issues now. I'm not able to reproduce the issue myself for now, so i guess this can be closed.


Refresh

Specify BSD license type.
Use pypi.io to get source.
Switch to sha256.
Re-render with conda-smithy 2.3.2.
Bump build number to 1.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Proposal for recursive constraints on associated types

This isn't 100% ready to go, but I figure reviewing a pull request might be more convenient than comments through email and directly on commits. I've incorporated a round of feedback from @dabrahams, @DougGregor, and @erica.
Once this is finished I'll squash the commits and fix the commit message.
TODO: fill out the missing stdlib changes.

Hi Austin – do you think you'll be able to cover the requested changes? Would be good to get this into review if we can resolve these.

Sorry @airspeedswift, I've been horribly busy these past few days but will definitely carve out some time tomorrow or Saturday to address your points. Apologies for the delays (again).

Does this protocol allow us to require that an associated type is equal to Self? For example, the FloatingPoint protocol should perhaps require that Stride == Self.

Thank you, all, for your patience. I've addressed the comments.
I believe the primary outstanding work now is adding more standard library APIs that need to be changed in response to the new features. Is that correct?

Does this protocol allow us to require that an associated type is equal to Self?

I'm not sure I understand the point of doing this, @natecook1000. Naively, I would think you'd just use Self anywhere you would otherwise use an associated type constrained to be equal to Self. Am I missing something?

Just wanted to let @airspeedswift know that this has been updated. There is no rush on my part to get it in, so feel free to review it whenever you have time.

Taking.

If there are any further changes you'd like me to make, I'd be happy to make them and submit a follow-up PR (in a timely manner).

Unless it's editorial, or so serious that we should halt and re-start the review, it should probably just get discussed on the list.


CMake Library Search Paths

Using $ENV{PROGRAMFILES(X86)} causes warnings on newer versions of CMake because of the parantheses.

Should be fixed, please let me know if it works! We can close if it does.

Seems to be resolved. I don't get any warnings about it anymore.


Reapply pull request #1221 - but leave the Window.prototype.location reference

Leave the Window.prototype.location reference, but add a comment as to why it has to be there.

Actually there were some breakages with history as well. Do you mind just doing the extra comment in this PR? I will change history to a var internally and rerun the tests and see if I can land it without much work.
(That will avoid another external rollback if I can't.)

Done. As an FYI, it was the history reference that originally prompted this - it's the one I actually need.


Add Protocol Selector Dropdown

PR Description
This PR adds a drop-down to select the protocol. Any further protocol support can be added by modifying the ProtocolType enum in consts.dart.
Related Issues

Related Issue #307
Closes #307

Checklist

[x] I have gone through the contributing guide
[x] I have updated my branch and synced it with project main branch before making this PR
[x] I have run the tests (flutter test) and all tests are passing

Added/updated tests?
We encourage you to add relevant test cases.

[x] Yes
[ ] No

Is not visible in light mode.
The dropdown should be sleek. Its height is too big.

I have made the changes and the dropdown looks like this:
Light Mode:

Dark Mode:

Let me know if this looks good to go!

Closing this PR due to major refactoring of the request model.


Inherited workspace dependencies

Several dependencies are copied in multiple Cargo.toml package files in the hotshot workspace. Example:
jf-primitives = { git = "https://github.com/EspressoSystems/jellyfish", branch = 'hotshot-compat'} # rev = "470a833" for branch = 'hotshot-compat'

Perhaps it's better to use the dependency inheritance feature of cargo to allow us to specify such dependencies in only one place. According to The Cargo Book all inherited dependencies are put into a new [workspace.dependencies] section of the root-level Cargo.toml.
Any reason not to do this?

+1 +1 +1


Glance component buttons should use a single blade file

Glance component buttons should use a single blade file

Fixed.


issue with codesign on build

Hi,
on command: flutter build ios --release
i get a weird error. I already created the cert via xcode UI based on the descriptions i got on my first error (No valid code signing certificates were found). After fixing it, the build looks better but fails anyway:
`Building de.okaycloud.billdonkey for device (ios-release)...
Automatically signing iOS for device deployment using specified development team in Xcode project: SQW44MM593
Starting Xcode build...
├─Building Dart code...                              2,1s
├─Generating dSYM file...                            0,3s
├─Stripping debug symbols...                         0,0s
├─Assembling Flutter resources...                    1,0s
└─Compiling, linking and signing...                  1,6s
Xcode build done.                                            6,3s
Failed to build iOS app
Error output from Xcode build:
↳
** BUILD FAILED **
Xcode's output:
↳
=== BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Release ===
The use of Swift 3 @objc inference in Swift 4 mode is deprecated. Please address deprecated @objc inference warnings, test your code with “Use of deprecated Swift 3 @objc inference” logging
enabled, and then disable inference by changing the "Swift 3 @objc Inference" build setting to "Default" for the "Runner" target.
The use of Swift 3 @objc inference in Swift 4 mode is deprecated. Please address deprecated @objc inference warnings, test your code with “Use of deprecated Swift 3 @objc inference” logging
enabled, and then disable inference by changing the "Swift 3 @objc Inference" build setting to "Default" for the "Runner" target.
=== BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Release ===
/Users/ml/development/projects/flutter/billdonkey/build/ios/Release-iphoneos/Runner.app/Frameworks/Flutter.framework: errSecInternalComponent
Command /usr/bin/codesign failed with exit code 1
/Users/ml/development/projects/flutter/billdonkey/build/ios/Release-iphoneos/Runner.app/Frameworks/App.framework: errSecInternalComponent
Command /usr/bin/codesign failed with exit code 1
Encountered error while building for device.       `
The first time i run it, a password screen for my keychain popped on the screen. Since i lost my keychain login pw, i needed to recreate the login keychain. But now i wont get asked and this error appears. Of course on first run i said "dont allow" to exit the process. Perhaps the build process remembers my last "dont allow" selection and now fails?
Just wild guessing here.

ok. It has nothing to do with remembering stuff. The build asked me for my keychain pw again and i put it in correctly. My keychain is open and the cert is in there. But it fails anyway.

Ok... i solved it but it was so weird, that it cant be the solution. First i changed my certificate access rights via keychain. Then i restarted the build but before i opened the login keychain. Then within the build i got asked if i should grant permission where i said "always grant" (dont know the english term, i got it in german). And then suddenly it worked.
To be honest... this is so crazy that i dont have any explanation for that.


Balder/summary.numthreads 2 backgroundnumthreads rebased 1

@toregge or @geirst PR

I will merge this one now. If we find a better name, I will update after the weekend. Documentation will also be updated then.

@baldersheim please have a look at my comments for next PR


refactor: hoist Docker build context concerns + fix: missing ecosystem config

Description
https://community.rosetta-api.org/t/improvement-proposal-embrace-native-remote-build-contexts-to-improve-dockerfile-design/204
I was intending to wait for feedback, but am hitting significant complexity with implementing a robust CI process to establish a merge preview context, which is awkward given it's there inherently. The original intent was to manage a dev image, but it would require two cache images to provide the CI build with a source, which in addition to the added weight, is only testing an approximation.
Also, config handling was changed in 42a1db1, removing the previous default assignments for PORT and BIND_ADDRESS that was relied up by the the Dockerfile ecosystem config.
Proposed Solution

remove dev.Dockerfile and setup script
COPY from context, rather than git clone
update example build commands to demonstrate same outcome
refactor automation to use locally managed versions,
maintained in source control, except for pre-release
add cache_from to the docker-compose.yml
Specify the missing ENVs

Important Changes Introduced

The suggested build commands have changed, as documented.
There is no longer an internal version to maintain in the Dockerfile.

Testing
This PR is covered by defined tests including

Automated CI
Manual trigger of the pre-release workflow: https://github.com/input-output-hk/cardano-rosetta/actions/runs/237666935

You've changed from inputs.cardano-rosetta-version to inputs.tag in the Dockerfiles, but the actions still use inputs.cardano-rosetta-version. Do they need to be changed too?
https://github.com/input-output-hk/cardano-rosetta/blob/4ba82ad59084397772dbe31be62d84994c880469/.github/actions/build_cardano_rosetta/action.yml
https://github.com/input-output-hk/cardano-rosetta/blob/4ba82ad59084397772dbe31be62d84994c880469/.github/actions/smoke_test_cardano_rosetta/action.yml


Relaxing crossterm dependency version

This is a follow-up to #153.
Since crossterm 0.9 version development is still in progress (as an example, see this bug), forcing to use one specific version is a little bit excessive, so this PR relaxes crossterm version to ^0.9.
@TimonPost confirmed that crossterm adheres to semantic versioning, therefore version can be relaxed safely.
As an addition, I removed the crossterm_backend feature because of two reasons:

crossterm feature is exists already (because crossterm as a depedency is optional)
Code that introduces it was merged into master 6 hours ago (at the time of creating this PR) and was not published yet, therefore possibility of breaking something is really small.

P.S. I have personal request also - can there be new version of tui published after merge of this PR? This is a only thing which blocks me.

Merged =). I was about to revert the crossterm_backend feature myself so that made perfect sense to me. I've also published a new version (v0.6.0).


Invalid URI format

I am using windows 8.1 x64 and node 6.x when I tried to pack the quick start electron example I have the following error.
Invalid URI: The format of the URI could not be determined.
at ChildProcess.exithandler (child_process.js:202:12)
at emitTwo (events.js:106:13)
at ChildProcess.emit (events.js:191:7)
at maybeClose (internal/child_process.js:852:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:215:5)

i have same problem. i am using windows 10

i resolve that.
u should add iconUrl in config params. and note that most be url.


Unable to unpin a pinned cask (wrong number of arguments)

Hi,
When I try to unpin a pinned cask, an error occurs: wrong number of arguments (given 2, expected 1).

Does anyone have a suggestion? Thank you in advance!

me too...

Same issue over here.  I took a quick look at the codebase, it looks like there is a typo that was introduced in pin_remove.rb (default parameter for quiet was removed in :18 and :26).
However, running a "manual" upgrade for the cask in question appears to clear the PINNED status.  e.g.:
brew upgrade siyuan
I don't have the cycles ATM to get this environment up and running to issue a PR, hopefully someone can get to it before me!

Should be working now 🙂  thx @su-thomas for the contribution 🙂


Dev Home Feature Exploration: Dev Home machine configuration generates a configuration file

Suggested new feature or improvement
Please Provide Feedback & Ideas!
Please let us know what you think about this feature by commenting on this issue! We'd love to hear your ideas and feedback!
Let us know:

What do you like about the feature?
What do you think is missing / what could improve the experience?

Overview
This issue outlines the proposed experience to generate a Winget configuration file at the end of the end to end machine configuration flow based on the details specified in the flow. This would allow a user to easily re-create their set up on another machine, or share it with others.
The Problem
Today a user can specify details for setting up their machine in the end to end configuration flow, but they don't have a way to easily repeat that set up. Winget configuration files make it easy and fast to configure a machine like the end to end flow, but there isn't a way to generate a configuration file easily today.
Scenario

#
Scenarios

1
As a user, I go through the end to end set up flow, and I specify repos to clone, dev drive creation, and applications to install. At the end of the flow, in addition to setting up my machine, I am able to generate and 'save as' a winget configuration file so that I can repeat this set up process on another machine or share my set up with others.

2
As a user, I am able to view the configuration file and edit it before I create and save it.

Additional details
No response

This could be a great feature!
As a developer, I'd like to create a configuration and share it with other members of the team, either via the GitHub repo requiring that specific configuration, or just sending the yaml file manually to users saying "it works on my machine" 😄
This could be interpreted like a "share a working setup" feature, in order to allow maintainers and users to recreate the same environment.

This seems like a good way to do it! Thanks! 😍

The winget part is easy, to be frank, to even replicate somewhat manually. That is - I can chain some winget commands and install all the required packages and versions. The interesting part, and specifically what I refered to in #721, is the ability to export the proper machine configuration - winget is part of that, but also get me all the dev settings (e.g., dashboard configuration, Git connections, repository links, registry settings enabled for development, etc.)
Without that functionality, the winget export alone is not really a huge value prop for my own scenarios.

Maybe add some sort of .ps1 module that sources authorized configuration files
where you can browse through a list of them and the features ( as a widget ? )


preferred data sources to have a connection point

http://www.bafg.de/GRDC/EN/Home/homepage_node.html

This is excellent!
I will try to create a system to get data from this source.

Actually, it appears that the GRDC only provides data to researchers, and does not allow us to share with the general public. I am going to check to see if any of the original data sources provide their data in a way that we can use.

This discussion lists several data sources by country:
https://www.researchgate.net/post/Can_anyone_tell_me_of_a_free_portal_to_download_river_discharge_data

This site provides access to UK daily data as a csv file: http://nrfa.ceh.ac.uk/
Unfortunately, data sharing is not allowed.

Good point of being allowed... Depends of the license information provided with the data provider. Check the description here: http://www.canadensys.net/2012/why-we-should-publish-our-data-under-cc0


CATS DOC tool additions

What version of CPACS are you using?
"CPACS 2.0"

Please specifiy further information for your issue:

Proposed addition of CATS DOC (Direct Operating Cost) input and output nodes,
see sample XML attached.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 2 Aug 2012 at 8:25
Attachments:

DOC_sample_adds.xml

Currently, there is no activity on this issue foreseeable, and hence, it will be closed


Add request throttling to nuget.org service APIs

The NuGet.org service API needs some configurable and enforceable upper limits on acceptable API usage.
Initial planned throttling limits are based on the maximum load we've seen on these endpoints to date.
V1 OData API
Throttling Type: based on IP

Api
Throttling Values
Main use case

GET /api/v1/Packages
1000 / minute
Query NuGet package metadata via v1 OData Packages collection

GET /api/v1/Packages/$count
100 / minute
Query NuGet package count via v1 OData Packages collection

GET /api/v1/Search()?searchTerm=&targetFramework=&includePrerelease=
3000 / minute
Search for NuGet packages via v1 Search endpoint

V2 OData API
Throttling Type: based on IP

Api
Throttling Values
Main use case

GET /api/v2/Packages
20000 / minute
Query NuGet package metadata via v2 OData Packages collection

GET /api/v2/Packages/$count
100 / minute
Query NuGet package count via v2 OData Packages collection

GET /api/v2/Search()?searchTerm=&targetFramework=&includePrerelease=
3000 / minute
Search for NuGet packages via v2 Search endpoint

PUT /api/v2/package
100 / minute
Upload a new NuGet package (version) via v2 push endpoint

DELETE /api/v2/package/{id}/{version}
100 / minute
Unlist a NuGet package (version) via v2 endpoint

@xavierdecoster , Can you also add the following details:

Error code and message o the author submitting packages more than the threshold?
Add a documentation plan - IMO this should go into a page in the API section as "Throttling limits"
Add some text to recommend on using V3 APIs.


in graph the new line starts on same level

Now every time a new dot is created on the graph the line starts on the bottom and moves up. the line needs to start on the same level than the one before or fast to the right place.

Wow! Super opgeschreven Alex!


Refactor temporal arithmetic

Which issue does this PR close?

Closes #6155.
Rationale for this change

https://github.com/apache/arrow-datafusion/pull/6196 PR has solved the issue of temporal scalars cannot appear as the LHS argument in an arithmetic operation. But there is an unoptimized solution such as creating an array from a scalar value as the length of the RHS array, and apply array vs array operations. This PR solves this issue in a similar way of LHS:Array and RHS:Scalar arithmetic.
What changes are included in this PR?

In the datetime.rs, there are two functions that handle arithmetic operations between arrays and arrays, and between scalars and arrays. To determine the appropriate operation to perform, a flag is used to hold information about the left-hand side (LHS) and right-hand side (RHS) operands.
In the kernel_arrow.rs, type matching is performed to identify the specific operation to execute. Finally, the actual computation is carried out in the corresponding function within the binary.rs.
Are these changes tested?

Yes.
Are there any user-facing changes?

I plan to review this PR tomorrow. Perhaps @tustvold  is interested as I believe he is planning some work to improve the temporal kernel support in arrow-rs

Thank you @berkaysynnada -- in general this code is looking quite good. I think it is structured nicely and is improving
My one big concern is that the new code seems to be mostly be untested -- specifically there are a lot of new functions and code that looks similar to me such as ts_ms_interval_mdn_op and ts_us_interval_mdn_op
Do you know if this code covered by existing tests?
I realize it might be repetitive to write tests for this code, but I think it is is really important to avoid regressions in the future. Perhaps there is some way to reduce the code replication (and therefore also the testing burden) 🤔

Thank you for the review ☺ I have applied your suggestions. I have added some tests that compares the results of a(ScalarValue) vs b(another ScalarValue) and a vs c(Array having one element and the same value of b). I thought this would suffice, as the ScalarValue tests are wide-ranging and testing edge cases. I have also tried to reduce arithmetic functions as much as possible, but I would appreciate it if you let me know if anything else catches your eye.

Thank you @berkaysynnada -- in general this code is looking quite good. I think it is structured nicely and is improving
My one big concern is that the new code seems to be mostly be untested -- specifically there are a lot of new functions and code that looks similar to me such as ts_ms_interval_mdn_op and ts_us_interval_mdn_op
Do you know if this code covered by existing tests?
I realize it might be repetitive to write tests for this code, but I think it is is really important to avoid regressions in the future. Perhaps there is some way to reduce the code replication (and therefore also the testing burden) 🤔

Thank you for the review ☺ I have applied your suggestions. I have added some tests that compare the results of a(ScalarValue) vs b(another ScalarValue) and a vs c(Array having one element and the same value of b). I thought this would suffice, as the ScalarValue tests are wide-ranging and testing edge cases. I have also tried to reduce arithmetic functions as much as possible, but I would appreciate it if you let me know if anything else catches your eye.

Thanks everyone!


fix issues(isolation policy, pjs and stale files).

Description:
fix issues.

Testing done:
Done

Affected area:

Functional Area

New Functionality
[ ]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[ ]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Is this a breaking change? no

Has documentation corresponding to this change been updated in the fsm-docs repo (if applicable)? no

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 3.06122% with 95 lines in your changes missing coverage. Please review.

Please upload report for BASE (release/v1.4@9656b7e). Learn more about missing BASE report.

Files with missing lines
Patch %
Lines

pkg/repo/client.go
0.00%
29 Missing :warning:

pkg/gateway/processor/v2/config.go
0.00%
25 Missing :warning:

pkg/sidecar/providers/pipy/repo/jobs.go
0.00%
11 Missing :warning:

pkg/repo/types.go
0.00%
10 Missing :warning:

pkg/catalog/mock_catalog_generated.go
0.00%
8 Missing :warning:

pkg/sidecar/providers/pipy/proxy.go
0.00%
7 Missing :warning:

pkg/catalog/catalog.go
60.00%
2 Missing :warning:

pkg/catalog/isolation.go
0.00%
2 Missing :warning:

pkg/gateway/processor/v2/utils.go
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@               Coverage Diff               @@
##             release/v1.4     #393   +/-   ##
===============================================
  Coverage                ?   14.27%
===============================================
  Files                   ?      819
  Lines                   ?    64684
  Branches                ?        0
===============================================
  Hits                    ?     9232
  Misses                  ?    55279
  Partials                ?      173

Flag
Coverage Δ

unittests
14.27% <3.06%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Provide a list of supported Terraform Provider.

Is your feature request related to a problem? Please describe.
I wanted to try Yor out for personal projects and found out, that the Terraform provider I use are as it seems not supported to run Yor against it.
For example I have a K3S Cluster at Civo or some development machines at Scaleway.
As it seems DigitalOcean is also not supported - at least I don't receive any errors or recommended tags by Yor.
Describe the solution you'd like
I would like to have some kind of overview about which Providers are supported or even in detail which terraform resources for the providers are supported. Maybe there is a way to add this to the Website or this GitHub Repository.
Describe alternatives you've considered
An additional idea would be to maybe autodetect if the current Terraform provider is support by Yor, so the CLI would just spit out "Yeah sorry I found that you want to create a DigitalOcean resource, but we don't support DigitalOcean yet".
Additional context
Here you can see, that my GCP project is the only one which finds resources which can be updated.
Output of a GCP project:
terraform providers && yor tag -d .

Providers required by configuration:
.
├── provider[registry.terraform.io/hashicorp/random]
└── provider[registry.terraform.io/hashicorp/google] 3.75.0

  __    __
  \ \  / /
   \ \/ /___  _  ____
    \  /  _ \| |/  __|
    | |  |_| |   /
    |_|\____/|__|v0.1.85
 Yor Findings Summary
 Scanned Resources:       7
 New Resources Traced:    2
 Updated Resources:       0

New Resources Traced (2):
[...]

Output of a Scaleway project:
terraform providers && yor tag -d .

Providers required by configuration:
.
├── provider[registry.terraform.io/scaleway/scaleway] ~> 2.1
└── provider[registry.terraform.io/integrations/github] ~> 4.0

Providers required by state:

    provider[registry.terraform.io/integrations/github]

    provider[registry.terraform.io/scaleway/scaleway]

2021/07/28 12:08:55 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type scaleway_account_ssh_key
2021/07/28 12:08:55 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type scaleway_vpc_private_network
2021/07/28 12:08:55 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type scaleway_instance_ip
2021/07/28 12:08:55 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type scaleway_instance_ip_reverse_dns
2021/07/28 12:08:55 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type scaleway_instance_security_group
2021/07/28 12:08:55 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type scaleway_instance_security_group_rules
2021/07/28 12:08:55 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type scaleway_instance_server
  __    __
  \ \  / /
   \ \/ /___  _  ____
    \  /  _ \| |/  __|
    | |  |_| |   /
    |_|\____/|__|v0.1.85
 Yor Findings Summary
 Scanned Resources:       9
 New Resources Traced:    0
 Updated Resources:       0

Output of a DigitalOcean project:
terraform providers && yor tag -d .

Providers required by configuration:
.
└── provider[registry.terraform.io/digitalocean/digitalocean] 2.6.0

2021/07/28 12:04:08 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type digitalocean_droplet
2021/07/28 12:04:08 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type digitalocean_firewall
  __    __
  \ \  / /
   \ \/ /___  _  ____
    \  /  _ \| |/  __|
    | |  |_| |   /
    |_|\____/|__|v0.1.85
 Yor Findings Summary
 Scanned Resources:       5
 New Resources Traced:    0
 Updated Resources:       0

Hi @dirsigler !
We support the 5 official cloud providers terraform supports - AWS, GCP, Azure, OCI & AliCloud

Thank you for the response!
I already looked into the code but couldn't figure out where to add support for other cloud provider. Else I would raise something myself😞

Cool, we'd love a contribution on this!
https://github.com/bridgecrewio/yor/blob/caec4ffddd722adeaa57e98ce892f0520a29435e/src/terraform/structure/terraform_parser.go#L28

I just tried yor and ran into the same problem.
I have a repo with a mix of supported and unsupported resources. Unfortunately, yor does not provide tags for the supported resources in this case:
yor tag -d . --dry-run
2023/07/10 11:27:30 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type kubernetes_cluster_role_binding_v1
2023/07/10 11:27:30 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type kubernetes_secret_v1
2023/07/10 11:27:30 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type kubernetes_namespace_v1
2023/07/10 11:27:30 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type helm_release
2023/07/10 11:27:30 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type kubernetes_namespace_v1
2023/07/10 11:27:30 [WARNING] failed to parse terraform block because failed to find tags attribute name for resource type helm_release
  __    __
  \ \  / /
   \ \/ /___  _  ____
    \  /  _ \| |/  __|
    | |  |_| |   /
    |_|\____/|__|v
 Yor Findings Summary
 Scanned Resources:	  109
 New Resources Traced: 	  0
 Updated Resources:	  0

But the module contains for example an azurerm_resource_group which gets tagged in a pure azurerm module.
Is there something I can do to make it work for the supported resources?
And I have the same question about contributing: Is it just a matter of adding a terraform provider to label name mapping in the code shown in https://github.com/bridgecrewio/yor/issues/169#issuecomment-897386640 ?


Bug: Docs broken in Firefox

I tested in Chromium and FF. Chromium works fine, but in FF I get this when I click any menu item (on the left):

Works for me on firefox 39.0.

Hmm.. interesting. I just opened a private window and there it worked too. Don't know the exact cause, but obviously has to do with my system/browser. Closed.

This is obvious so I apologise, but have you tried restarting firefox?
Or perhaps it's an addon? You could try disabling all addons too and trying again. Or maybe disable each addon one by one.

Actually, I believe private mode still has addons enabled, so it's probably not an addon if it works in private mode. I guess if it still persists in normal firefox, you could try clearing the cache (with care).

I think I found the issue: I did a search once for "channel" and nothing was found, that's why it says "channel" and no menu item is displayed. Everytime I click the menu the search is apparently rerun and as such the menu disappears. I was able to replicate this behaviour in Chromium too.

Okay I think I get what you are doing:

Go to http://crystal-lang.org/docs/

Click the search icon to pull down the search text box and type in "channel" here, and notice there are no results, and the menu becomes empty.

Click the search icon again to get rid of the search text box.

Click on any menu item and observe that the search box is pulled down again with "channel" still there.

So this means the menu is now empty again when it probably shouldn't be?

I did this on firefox 39.0.
Clearing "channel" from the search text box just before step 3 obviously fixes this.
And I guess the fix, if this is actually considered a bug, would be to somehow determine if toggling the search box will hide it (instead of making it appear), and then clearing the contents of the search text box.

Yup, that sums it up pretty well. I agree with your fix proposal. When the search is closed, it shouldn't be active until one deliberately reactivates it again. Also the field content should be cleared right after closing the search in any case.

We are using GitBook.
I was able to reproduce this in a random gitbook.
Could you report the bug there, if it doesn't already exist? Thanks!

@asterite Ah right of course, I completely forgot! It seems to already exist: https://github.com/GitbookIO/gitbook/issues/783.

Alright, I'm going to close it then. Thx everyone!


Simulation workspace builds with warnings

Branch: ros2
colcon build
Starting >>> xacro
Starting >>> sagemaker_rl_agent
Finished <<< xacro [1.08s]
Starting >>> turtlebot3_description_reduced_mesh
Finished <<< turtlebot3_description_reduced_mesh [0.42s]
Starting >>> object_tracker_simulation
--- stderr: sagemaker_rl_agent
zip_safe flag not set; analyzing archive contents...
markov.__pycache__.single_machine_training_worker.cpython-36: module references __file__
---
Finished <<< sagemaker_rl_agent [1.58s]
--- stderr: object_tracker_simulation
warning: install_lib: '/home/ubuntu/environment/aws-robomaker-sample-application-objecttracker/simulation_ws/build/object_tracker_simulation/build/lib' does not exist -- no Python modules to install

(Copied from B9-5975).

I am no longer seeing this warning when building right now (HEAD of ros2 branch is commit 9b5e09e) using an up-to-date Ubuntu Bionic / ROS2 Dashing environment.
It does not show up in our Travis cron builds either (for example: https://api.travis-ci.org/v3/job/646634591/log.txt).
Closing this issue now.


Timestamp still shows UTC time instead of local time

Changed /admin to my local time from UTC. Alerts still have UTC time stamp:

Hi, can you check your user settings at /user/ID? It's possible an override of UTC is defined there. I did a quick test and I do get non-UTC timestamps.

Sure. I'll test.

You were right. It was defined there as well. Changed it to my time zone and everything looks great now. Thanks!


Markdown tables should use alternative CSS in description

From bugbash: tables in comments display very different than in description. @SamPeck supplied alternative values for the main container (see below).

I've also included the fix for #4108 here to avoid merge conflicts if they were done separately.


New Crowdin translations

Enjoy!

@megastep @analogcode What's the status of this pull request?

It's an automated PR from CrowdIn. I merge it manually whenever new translations are completed. We should leave it open though because they'll just create a new one if we close it.

@megastep Thanks for the clarification.

Great, that's just going to trigger a new one being open ;)


[Enhancement] Abort executing compactions on error

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:disappointed: fail : 0 / 53 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/lake/compaction/CompactionJob.java
0
6
00.00%
[49, 53, 54, 74, 102, 103]

:large_blue_circle:
com/starrocks/lake/compaction/CompactionScheduler.java
0
22
00.00%
[134, 135, 137, 139, 140, 141, 142, 143, 144, 145, 146, 147, 150, 153, 155, 160, 201, 202, 203, 204, 205, 376]

:large_blue_circle:
com/starrocks/lake/compaction/CompactionTask.java
0
25
00.00%
[68, 78, 79, 82, 83, 84, 85, 86, 87, 88, 93, 95, 96, 97, 98, 99, 101, 103, 104, 115, 116, 118, 119, 120, 121]


mockRouter() not working, but vueRouter() does

I have a component hat is using the following:
  import { useRoute } from "vue-router";

  const route = useRoute();

My preview.ts looks like this:

import { Preview, setup } from "@storybook/vue3";
import { createPinia } from "pinia";
import { Quasar } from "quasar";
import { vueRouter } from "storybook-vue3-router";
import { i18n } from "../src/boot/i18n";

setup((app) => {
  app.use(Quasar, {});
  app.use(createPinia());
  app.use(i18n);
});

const preview: Preview = {
  // global decorators for every story, see https://storybook.js.org/docs/vue/writing-stories/decorators#global-decorators for reference
  decorators: [vueRouter([])],
  parameters: {
    actions: { argTypesRegex: "^on[A-Z].*" },
    controls: {
      matchers: {
        color: /(background|color)$/i,
        date: /Date$/i,
      },
    },
    docs: {
      toc: true, // generate table of contents for component documentation pages
    },
  },
};

export default preview;

Using the vueRouter() decorator works.
When using  mockRouter() as the global decorator for all stories, I get the following error:
import { mockRouter } from "storybook-vue3-router";
// ...

decorators: [mockRouter({})],

The documentation states that mockRouter() should be fine for my usecase (https://github.com/NickMcBurney/storybook-vue3-router#mock-router). This seems to be wrong. Can you explain to me what I am missing?

@Walnussbaer its because your using const route = useRoute() mock router just 'mocks' this.$route and this.$router, as useRoute() is a composible from vue-router it requires the full router setup.
Is there a reason you are trying to use mocked router and not full router?

The documentation of useRoute() states that it is the same as calling $route in the template. But okay, thanks for the explanation.
The reason I wanted to use the mocked router is that your documentation states the following:
The full vue-router is not always needed - for example if you don't have components using  or  then using the mockRouter export may cover your needs (and reduce the imports being used in your project).
As I stated in my first sentence, my expectation was that $route and useRoute() from the composable refer to the same global variable. They probably do, but as you explained, the composable does not work if the full router is not set up correctly.

@Walnussbaer I've improved documentation on the back of this issue. Thanks for your feedback.

@NickMcBurney I saw it, great stuff. Thank you for your effort and have a nice weekend! :)


__WD_Sleep .. SetError($_WD_ERROR_UserAbort)

__WS_Sleep() should  set $_WD_ERROR_UserAbort in case of error instead $_WD_ERROR_Timeout

now I think this is all fine
As so far I do not plan other changes in this PR

in
Refactoring __WD_Sleep() usage in functions STEP2
I ony use TIDY nothing more.

Why do you use $v_Parameter for custom _My_Sleep function? Why not $iDelay?
Fixed

So if $iDelay=100 why there was:
__WD_Sleep(1000)
It's a bit weird when I say wait a little second $iDelay=100 and then the program waits 10 times longer.
btw.
FunctionHeader says
$iDelay              - [optional] Milliseconds to wait before checking status
looking in the code and I see that status is checked multiple time in a While loop.

It's a bit weird when I say wait a little second $iDelay=100 and then the program waits 10 times longer.

It's an optional parameter that allows the user to not start the process of checking for the desired element for a specific length of time. In it's default setting, the $iDelay value equals 0, so there isn't a delay. If the user passes in something like 5000, then there is an initial delay before entering the loop.
The __WD_Sleep(1000) is simply a pause during the looping process. It could be any other value (ie: 250, 500), but I didn't see an issue with a 1 sec delay per loop cycle

FunctionHeader says

Yes, the header could be clearer. Could be as simple as adding the word "initially" before "checking status".

looking in the code and I see that status is checked multiple time in a While loop.

Correct. What issue do you have with that?

It's a bit weird when I say wait a little second $iDelay=100 and then the program waits 10 times longer.

It's an optional parameter that allows the user to not start the process of checking for the desired element for a specific length of time. In it's default setting, the $iDelay value equals 0, so there isn't a delay. If the user passes in something like 5000, then there is an initial delay before entering the loop.
The __WD_Sleep(1000) is simply a pause during the looping process. It could be any other value (ie: 250, 500), but I didn't see an issue with a 1 sec delay per loop cycle

FunctionHeader says

Yes, the header could be clearer. Could be as simple as adding the word "initially" before "checking status".

looking in the code and I see that status is checked multiple time in a While loop.

Correct. What issue do you have with that?

Will answer in next few days

FunctionHeader says

Yes, the header could be clearer. Could be as simple as adding the word "initially" before "checking status".

done

The __WD_Sleep(1000) is simply a pause during the looping process. It could be any other value (ie: 250, 500), but I didn't see an issue with a 1 sec delay per loop cycle

looking in the code and I see that status is checked multiple time in a While loop.

Correct. What issue do you have with that?

Working on example .....

`
#include <GuiComboBoxEx.au3>
#include <GUIConstantsEx.au3>
#include <ButtonConstants.au3>
#include <MsgBoxConstants.au3>
#include <WindowsConstants.au3>
#include "wd_helper.au3"
;~ #include "WebDriver-master\wd_helper.au3"
Global $sSession
_Example()
If @error Then ConsoleWrite("! ---> @error=" & @error & "  @extended=" & @extended & _
" : Error" & @CRLF)
Func _Example()
SetupChrome()
_WD_Navigate($sSession, 'https://google.com')
If @error Then Return SetError(@ScriptLineNumber)
MsgBox($MB_OK + $MB_TOPMOST + $MB_ICONINFORMATION, "Information #" & @ScriptLineNumber, "Accept the POP UP message in Google Chrome")

Local $hTimer = TimerInit() ; Begin the timer and store the handle in a variable.
For $i = 1 To 10
	_Search()
	If @error Then Return SetError(@error)
Next
Local $fDiff = TimerDiff($hTimer) ; Find the difference in time from the previous call of TimerInit. The variable we stored the TimerInit handlem is passed as the "handle" to TimerDiff.
MsgBox($MB_SYSTEMMODAL, "Time Difference", $fDiff / 1000 & ' Seconds')

EndFunc   ;==>_Example
Func _Search()
_WD_Navigate($sSession, 'https://google.com')
If @error Then Return SetError(@ScriptLineNumber)
_WD_LoadWait($sSession)
If @error Then Return SetError(@ScriptLineNumber)

Local $sElement_Input = _WD_WaitElement($sSession, $_WD_LOCATOR_ByXPath, '//*[@name="q"]')
If @error Then Return SetError(@ScriptLineNumber)

_WD_SetElementValue($sSession, $sElement_Input, 'AutoIt', $_WD_OPTION_Advanced)
If @error Then Return SetError(@ScriptLineNumber)

Sleep(10)
_WD_ElementAction($sSession, $sElement_Input, 'click')

Local $sElement_Search = _WD_WaitElement($sSession, $_WD_LOCATOR_ByXPath, '//input[@name="btnK"]', Default, Default, $_WD_OPTION_Enabled)
If @error Then Return SetError(@ScriptLineNumber)

Do
	Sleep(10)
	_WD_ElementAction($sSession, $sElement_Search, 'click')
Until Not @error

Local $sElement_PaginatorNext = _WD_WaitElement($sSession, $_WD_LOCATOR_ByXPath, '//*[@id="pnnext"]')
If @error Then Return SetError(@ScriptLineNumber)

_WD_ElementAction($sSession, $sElement_PaginatorNext, 'click')
If @error Then Return SetError(@ScriptLineNumber)

_WD_LoadWait($sSession)
If @error Then Return SetError(@ScriptLineNumber)

EndFunc   ;==>_Search
Func SetupChrome()
_WD_UpdateDriver('chrome')
_WD_Option('Driver', @ScriptDir & '\chromedriver.exe')
_WD_Option('Port', 9515)
_WD_Option('DriverParams', '--verbose --log-path="' & @ScriptDir & '\chrome.log"')
_WD_Startup()
Local $sDesiredCapabilities = '{"capabilities": {"alwaysMatch": {"goog:chromeOptions": {"w3c": true, "excludeSwitches": [ "enable-automation"]}}}}'
$sSession = _WD_CreateSession($sDesiredCapabilities)
If @error <> $_WD_ERROR_Success Then Exit

EndFunc   ;==>SetupChrome
`

Ok. So now I have proper explantion which show why I do not want to use:
__WD_Sleep(1000)
Check this following example with and without this 1000 ms sleep.
wd_testing_Sleep.zip

The __WD_Sleep(1000) is simply a pause during the looping process. It could be any other value (ie: 250, 500), but I didn't see an issue with a 1 sec delay per loop cycle

My example shows that after each proper finding there is added 1 sec waiting, where there is no need to wait so long.
I could understand to add:
__WD_Sleep(10) which will not overheat CPU, and will keep interaction (script responsivity) thanks to using:
_WD_Option('Sleep', _MyResponsiveSleepFunction)


Swift: allow passing a read stream to uploadFile()

By this you can pass a function directly to uploadFile():
storage.uploadFile('container', 'filename.jpg', () => myReadStream, (err) => {

});

Thank you again for your contribution, it is included on the latest version just published:
https://github.com/carboneio/tiny-storage-client/blob/v2.2.5/CHANGELOG.md


Add sense support for all algorithms

Currently sense is only supported for a subset of algorithms. This breaks the common interface as some solvers solve min vs max. Furthermore, since sense is a kwarg for OptimizationProblem and not solve, the interface and docs give the impression that it is supported for all algorithms.
using Optimization, OptimizationBBO

J(x,p) = x

F = OptimizationFunction(J)
prob = Optimization.OptimizationProblem(F, 0.0; lb = -10, ub =10, sense = MaxSense)
solve(prob, BBO_adaptive_de_rand_1_bin_radiuslimited()).objective # -10.0

What we'd like to do here is instead of changing it in the solvers handle it in the instantiate_function method and add the negative sign to the objective function evaluation and derivatives when MaxSense is passed


Make registering components idempotent

I'd like to be able to have systems in my library automatically register any components they use upon initialisation, so that clients don't need to do that separately. The consequence of this is that multiple systems would attempt to register the same component type.
World::register can't be used in that way directly, by the look of it because it doesn't keep track of what components are registered before blindly "re-initialising" the storage for the component type.
I thought of checking this myself using something like TypeMap, then it occurred to me that you're effectively tracking the information necessary for this internally in Specs anyway, so it wouldn't be much of a stretch to make register idempotent`.
Any objection to this, or reason I've missed that it shouldn't be idempotent? I'm happy to implement it and PR if no objections.

@jeffparsons It sounds sensible to me. How about a separate register_maybe variant that returns a bool?

@kvark How's this? https://github.com/slide-rs/specs/pull/98

Replaced #98 with https://github.com/slide-rs/specs/pull/99.

I think we can call this one done. :)


Debug and updates

This PR fixes the unaccepted answer infinite loop bug, does some house keeping with llama-server flags and finally, updates the README local llm section (fixes the recommended llama.cpp to a specific commit and recommend a more recent llm download).

Looks good. I will make some prompt changes after accepting.


Move onAccessAttempt to TinyStaticProperty

@samreid and I found this to be only used here, and it was causing weird and buggy logic in tying to support it in TinyForwardingProperty. We thought the best path forward was to move onAccessAttempt to TinyStaticProperty.

Moved above. @jonathanolson will you please review and also note https://github.com/phetsims/scenery/issues/1103.

Reviewed, sounds and looks great to me.


tuareg-mode integration (particularly merlin-mode setup) issues

I am using esy-mode by loading in the .el in my doom config, which works fine for the major reason-mode (with hooks for merlin-mode and esy-mode). I have not added a hook for esy-mode, but have found it to be on (but dormant, as the esy-mode comments say it should be in a project that doesn't look like an Esy one) in ocaml buffers anyway. Seems like it is globally active by default now that I'm looking.
System:

Emacs 28.0.50 (Doom)
Manjaro

package.json
{
  "name": "bonsai_revery",
  "version": "0.0.1",
  "description": "Bonsai Revery",
  "license": "MIT",
  "homepage": "https://github.com/osener/bonsai_revery",
  "esy": {
    "build": "dune build -p #{self.name}",
    "buildsInSource": "_build"
  },
  "scripts": {
    "format": "esy dune build @fmt --auto-promote",
    "run": "esy x examples"
  },
  "dependencies": {
    "@opam/bonsai": "v0.14.0",
    "revery": "*"
  },
  "devDependencies": {
    "ocaml": "~4.9.0",
    "@opam/dune": ">= 2.5.0",
    "revery": "revery-ui/revery#8e0479e",
    "@opam/merlin": "*"
  },
  "resolutions": {
    "esy-sdl2": "2.0.10008"
  }
}

Despite being "on" in an Esy project with a package.json, the merlin-command variable is not set (remains as default = "opam") when ocaml buffers are loaded, and merlin version mismatches result.
merlin error (cursor over a module)
Error from syntax checker ocaml-merlin: Wrong type argument: characterp, nil
eldoc error: (error merlin: [ ... ]/bonsai_revery/_build/default/src/.bonsai_revery.objs/byte/bonsai_revery.cmi
seems to be compiled with OCaml 4.09.
But this instance of Merlin handles OCaml 4.11.)

describe-variable, after opening tuareg-mode ocaml buffer.
merlin-command is a variable defined in ‘merlin.el’.
Its value is ‘opam’

describe-variable, after off/on esy-mode.
merlin-command is a variable defined in ‘merlin.el’.
Its value is
"/home/geoff/.esy/3____________________________________________________________________/i/opam__s__merlin-opam__c__3.4.1-533ba774/bin/ocamlmerlin"
Original value was ‘opam’

Workaround:
Turning off esy-mode then turning it back on does correctly set merlin-command. Subsequent refresh of tuareg-mode with M-x tuareg-mode gets things working.

Thanks for reporting this

Pull request with fix here:
https://github.com/ManasJayanth/esy-mode/pull/9

Fixed by #9


enhance login prompt

replace https://github.com/containerd/nerdctl/pull/641 ( thanks @simorgh1 )
Signed-off-by: fahed dorgaa<EMAIL_ADDRESS>

Not entering the username raises the following error
FATA[0002] error reading username: unexpected newline
instead of the validation error: username is required
I had the same issue when assigning the username,
options.username = username
By not entering username, the value will be a newline and option.username raises the above error.

Please squash


passing an array

Hi
Am new to meteor and javascript.
Am having two sets of array ,
1.array which contains a table head.
ex head: ["Column 1", "Column 2"]
2.nested array contains table data.
ex data: [["column 1 value", column 2 value"] , ["column 1 value", column 2 value"]  ...]
how can i pass the title and data using this array
Suggest me how to solve this problem?

This is a question for datatables itself, not for the reactive wrapper. I personally don't know how to do what you're asking, I've always used this with objects rather than arrays. I'd recommend you go to the datatables website and look at their documentation and examples


Fix action manager memory leak.

Don't use references in callbacks as they may keep layers around. Should close napari/napari#2027
Fixes/Closes
Closes #2027
References
You can use the following test script to see the object graph before/after this PR.
import gc
import objgraph
import weakref
import sys
import napari

viewer = napari.Viewer()
r = weakref.ref(viewer.add_points())

viewer.layers.pop()
print(gc.collect(), "unreachable objects after collection")

objgraph.show_backrefs(
    [r()], filename='sample-backref-graph.pdf', max_depth=10
)

Type of change

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

This will fail as now some layers are weakrefs, they are becomming None in some lined_layers callback. I need to fix to unbind an deletion.


Lab1: Peer review by Krzysztof Kleist s296450

It's good and productive that you implemented the new TupleSet() class to make your approach more individual. Also it's well described in README.md file.
Unfortunately you didn't provide the solution in the notebook, it could be way more readable, there are a lot of useful comments though.
Another good thing is the check if the problem is solvable.
Unfortunatelly you managed to solve only Breadth-first search but your solution can be considered as complete because you also managed to provide solutions for N > 20 what was problematic.

Thanks for your feedback!


Regenerate package lock

Fixes #209.
/cc @es128

Based on my interactions with @es128, I think https://github.com/strongloop/fsevents/pull/211 is a better option.


Support inline LaTeX formula in znai

The current way to generate an image from a latex formula is to use a fenced block.
A user wants to line a latex formula within a paragraph.

@jyeakel is this something your guys will PR?

Yes, we'll take care of it!

closed by #602


Document Envoy error messages when traffic is blocked via an SMI policy

We need to compose this document after #705 is finished - it will change these error messages.

Different error codes emitted by the control plane are now well documented at https://docs.openservicemesh.io/docs/guides/troubleshooting/control_plane_error_codes/.


KyuubiSessionEvent shall always have Id

Why are the changes needed?

session id is a val
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #1623 (23fc3b2) into master (fbccba9) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master    #1623   +/-   ##
=========================================
  Coverage     58.28%   58.29%
  Complexity      164      164
=========================================
  Files           258      258
  Lines         12823    12834   +11
  Branches       1608     1610    +2
=========================================
+ Hits           7474     7481    +7
- Misses         4718     4722    +4
  Partials        631      631

Impacted Files
Coverage Δ

.../org/apache/kyuubi/events/KyuubiSessionEvent.scala
93.10% <100.00%> (+0.51%)
:arrow_up:

.../org/apache/kyuubi/session/KyuubiSessionImpl.scala
83.92% <100.00%> (-0.56%)
:arrow_down:

...in/scala/org/apache/kyuubi/server/api/v1/dto.scala
68.75% <0.00%> (-1.47%)
:arrow_down:

...ache/kyuubi/server/api/v1/OperationsResource.scala
65.30% <0.00%> (+1.20%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fbccba9...23fc3b2. Read the comment docs.

thanks, merged to master


Substitution of deprecated grep command/add_config_line for reading from/writing to glidein_config

Until now, grep was being used to read values from/write values to glidein_config when running system scripts during the glidein startup process. In 3.9.6, add_config_line.source was updated to include two utilities primarily to replace the grep-based logic in these scripts. The two functions, gconfig_get (for reading) and gconfig_add (for writing) will be used from release 3.9.7 onwards and therefore, the use of grep, for the purposes of reading and writing to glidein_config will be deprecated hereafter.
Note that, the add_config_line.source will need to be imported inside the script before either of the gconfig_xxx utilities can be invoked/used. It is only for this that grep and cut commands will continue to be used like before.

Generally when sourcing add_config_line:

Make sure that the $config_file is quoted in the grep
Add the shellcheck line:
prefer . to source.
Make sure that the file name is quoted.

add_config_line_source=$(grep -m1 '^ADD_CONFIG_LINE_SOURCE ' "$glidein_config" | awk '{print $2}')
# shellcheck source=./add_config_line.source
. "$add_config_line_source"

All the requested changes have been completed and the comments have been marked as resolved.


Retraining explanation with tf-hub

Trying to go through this tutorial below to retrain ssd_inception_v2_coco_11_06_2017 to fine tune and retrain on coco + additional set of images:
https://www.tensorflow.org/tutorials/image_retraining#creating_a_set_of_training_images
This tutorial seems to be different from others I've seen where labelling images and creation of tfrecord are necessary steps. However, this tutorial seems to accept a directory of images and retrains the model with the image dir.
Question: Is the tf-hub automating the tfrecord creation process using the directory name. If so, is it using the full image size and not tagging the bounding boxes of contained boxes?

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 29 days with no activity and the awaiting response label was assigned. Is this still an issue?

The example is for classification not detection

Is this still an open issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 29 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 44 days with no activity and the awaiting response label was assigned. Is this still an issue?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Upgrade Migaloo to v4.1.0

https://ping.pub/Migaloo/gov/24
https://ping.pub/Migaloo/block/5689150

Team is requesting that validators vote no on the prop and wait for another one - will edit this based on the correct new prop / height when released.


Specify StorageClass with AnyPath

Is there any way to specify a target Storage Class when using AnyPath
Specifically I'm using AnyPath to create S3 objects and I want to be able to have them created in the Intelligent Tier storage class.
I can tackle this by using life cycles to move the object in S3 but it would be easier and cheaper to be able to create the object in the desired class from the get go.

Thank you again, that pointed me at the way to fix what I wanted.
Instead of setting the client as the default I was able to pass it to AnyPath with the client option since I only want to us this option on some calls to AnyPath.


[3.5] - Change auto to default in cursor css settings.

A replacement for #67964 which duplicated a fix rather than cherry-picking it.
will close #67925
cherry-picking from #67044

There was a temporary CI issue which broke the iOS build, so I went ahead and just cherry-picked the original PR myself for my next batch. Thanks anyway!


Make priority values categorical

Make priority values categorical (one of low, high, max). This simplifies things for queue status metrics and enables setting the queue selection probability at runtime.
If no priority header is specified, priority defaults to max. If an invalid header is specified, it defaults to low.
The config.properties parameter nPriorities is now no longer needed, but there is a new float parameter high_prio_prob, which determines the probability of a worker choosing a job from the high priority queue (low probability is 1 - high_prio_prob, and max jobs are always executed first).

One thing I am a bit unsure here, is why we wrap a concurrent hash map inside - if we would use the re entrant lock for all public methods, then we cannot get any race condition, right?

I think my knowledge of how the ReentrantLock actually works is not large enough to judge the effects of that. I was using the ReentrantLock mainly for the Condition functionality, I'm not sure to what extent it prevents race conditions.
I can investigate this and make a separate PR to make the switch to a non-concurrent hash map :+1:

I tested it locally with all changes and it still seems to work :tada:


Update hackage release to support base 4.9 & ghc-8

The current release on hackage doesn't work with base-4.9, or ghc-8.

Thanks for reporting; I'll try to get to this at the weekend.  Haven't had to look into this code since I released it . . .

This should be fixed with <IP_ADDRESS> on Hackage.  Travis is now testing it against GHC HEAD and 8.0.2.  (Interestingly the only bound that needed to be adjusted was that of the pipes package, not base!)  Please reopen if you have any trouble.


No syntax highlighting for .env.

Can anyone let me know if i can get syntax highlighting for custom environment names

You can associate custom file naming schemes through the vscode file association settings. See https://github.com/mikestead/vscode-dotenv#associate-files for an example.

In particular the setting that you seem to want is this:
"files.associations": {
  ".env.*": "dotenv"
},


Add example on how to do actions on item click

Now no of the examples show what you should do when the user clicks on one item, and all the items in all the examples are only for looks.
Show what I should do attach an action to a item. Eg, when I select "Format -> Bold" from a menu, it actually bolds a text. Or something similar.

Added this to #38, let's collect all the demo improvements in one issue for simplicity.


[Hand4Whole]I would like to ask a question about the Agora dataset.

Hello Dr. Moon.
I would like to ask if you would mind sharing the yolov5 detector that you finetune on agora. I would like to use it to try the detection effect on the synthetic data set of Agora.

sorry I do not have it, but you can easily fine-tune it.

Thanks for your reply, nevermind, I fine-tuned one myself.
Sorry, I have another question to ask you, that is, the Agora data set contains some labels of kids. The betas of these data contain 11 parameters. Did you only take the first 10 parameters of the betas for training?

I think so. I did not take care much about kids...

Thank you very much for your patient reply!


Relative URLs incorrectly prepend /wp/ when running acorn optimize

Version
1.0.1
What did you expect to happen?
When enabling relative URLs I expected them to be enqueued with the correct URL, e.g.
/app/themes/sage10/public/css/app.1259cf.css
What actually happens?
Scripts and styles are enqueued with /wp/ prepended, e.g.
/wp/app/themes/sage10/public/css/app.1259cf.css
Steps to reproduce

Set 'enabled' => true in the prettify config
Run wp acorn optimize
Note theme scripts and styles aren't loading correctly.

See roots/acorn#226 for more details.
System info
macOS 14.5 M3 chip
Lima as local env
Also same when deployed with acorn optimize as a build-after task to Ubuntu 22.04.
Both envs php8.2
Log output
No response
Please confirm this isn't a support request.
Yes

@Log1x Tested, working. Legend 💪


Include SAIL snippets into the specification

This includes the snippets into the instruction listings as well as the capability description chapter.
Currently waiting for a publicly available, up to date SAIL model being released.

Can we (and should we) pick up a dependency to the Sail model for that? Would it be possible to automatically pull in the relevant lines from the Sail model? @Timmmm @PeterRugg

I did set this up when the spec was in Codasip but we removed it when it was moved to GitHub since the Sail code wasn't open at the time. I'll see if I can resurrect it.
The big question is how do you get the JSON file. It's very big so I don't think you just want to add it to Git. But adding Sail compilation to the build process is also not great (although it's a lot better since we made the Sail binary release). In Codasip we had a cache on a web server which worked pretty well. How do people feel about using GitHub releases for that?
So every push (or maybe tag) in the Sail repo would generate the JSON and add it to a GitHub release, and then this repo would just point to the URL and download it.
We'll have to answer this question for riscv-isa-manual too and I think this repo could serve as a good demonstrator.

For my testing purposes on riscv-isa-manual I've been generating the index at the top of tree of the sail repository and adding it directly into the repo and referencing it with an AsciiDoc attribute in the document header (riscv-unprivileged.adoc) with :sail-doc: src/sail-index/riscv_RV64.json.

Yeah the issue is it's 4.5 MB (3.1 MB minified). Feels a bit much for a file that we'd continually update? Though I guess it probably does diff pretty well at least.

Ok I have update the Sail repo to have a Github workflow to generate the JSON and I have made a release of it (making a release is a manual process currently but I can automate it in future).
Next step is to update this repo to curl https://github.com/CHERI-Alliance/sail-cheri-riscv/releases/download/2024-12-19/riscv_RV64.json and add the asciidoctor-sail stuff. I'll have a go tomorrow.

PR here: #498
(on top of #497)


Dw/fix heading hashtag positions

Fix misaligned h2 anchor link.
Remove broken community project link.

Old
New

Looks great, thanks @donmckenna!


Can’t use Cyrillic letters in console since 1.74

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

When I disable all user extensions manually and restart the editor, it still occurs.
But when I launch it with code --disable-extensions it doesn’t occur.

Maybe this launch option affects something else, some built-in extension perhaps.

What is the output of echo $LANG in your case?

en_DE.UTF-8


0763 - Partition Labels

Description of the Problem
A string S of lowercase English letters is given. We want to partition this string into as many parts as possible so that each letter appears in at most one part, and return a list of integers representing the size of these parts.
Example 1:
Input: S = "ababcbacadefegdehijhklij"
Output: [9,7,8]
Explanation:
The partition is "ababcbaca", "defegde", "hijhklij".
This is a partition so that each letter appears in at most one part.
A partition like "ababcbacadefegde", "hijhklij" is incorrect, because it splits S into less parts.

Note:

S will have length in range [1, 500].
S will consist of lowercase English letters ('a' to 'z') only.

Code
class Solution:
    def partitionLabels(self, S: str) -> List[int]:

Link To The LeetCode Problem
LeetCode

It seems like an interesting problem. Please assign it to me.

which language do you want the solution in?


Support string uidType

PR that adds 'string' as an allowable relationship uidType.
Regarding: https://github.com/jagql/framework/issues/83

awesome!!

you can ignore the travis failure, i'll update the dependencies

Codecov Report

Merging #84 into master will increase coverage by <.01%.
The diff coverage is 40%.

@@            Coverage Diff             @@
##           master      #84      +/-   ##
==========================================
+ Coverage   94.94%   94.95%   +<.01%
==========================================
  Files          64       64
  Lines        1782     1783       +1
==========================================
+ Hits         1692     1693       +1
  Misses         90       90

Impacted Files
Coverage Δ

lib/ourJoi.js
87.5% <40%> (+0.22%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40181e5...f5ad566. Read the comment docs.


t411.li changed to t411.ai

Expected behaviour:
thanks for the change (api, etc.)
Actual behaviour:
Steps to reproduce:

Step 1: ...

Config:
paste relevant config
use paste service if config is too long

Log:
paste log output
use paste service if log is too long

Additional information:

Flexget Version:
Python Version:
Installation method:
OS and version:
Link to crash log:

Ignoring the template will just get your issue ignored.

sorry but it's too complicated, i'm not a developer. thanks