Examples: Where is err being defined in the Running tasks in series example?

In the Running tasks in series, i.e. Task Dependency example, where is err being defined in this code block?
var gulp = require('gulp');

// takes in a callback so the engine knows when it'll be done
gulp.task('one', function(cb) {
    // do stuff -- async or otherwise
    cb(err); // if err is not null and not undefined, the orchestration will stop, and 'two' will not run
});

// identifies a dependent task must be complete before this one begins
gulp.task('two', ['one'], function() {
    // task 'one' is done now
});

gulp.task('default', ['one', 'two']);
// alternatively: gulp.task('default', ['two']);

I'm new to Gulp, so please excuse me if I am missing something obvious, but looking at the example it's not clear to me.

If you perform some operation in the task that may result in an error you could pass that to the callback, e.g.:
try {
  something();
}
catch (e) { cb(e); }

Or if you perform some async operation upon which success or failure of the task hinges and that takes a standard node-style callback, you can just pass the callback to that operation. E.g.:
gulp.task('one', function (cb) {
    fs.writeFile('filename', 'data', cb);
});

That makes sense. The example just omits that in kind of a strange way.
When I get a chance I'll put a suggested change in a PR. Maybe something like:
// takes in a callback so the engine knows when it'll be done
gulp.task('one', function(cb) {
    try {
        // do stuff -- async or otherwise
    } catch (err) {
        cb(err); // if err is not null and not undefined, the orchestration will stop, and 'two' will not run
   }
});

@JamesMcMahon we'd be down for a PR but try-catch doesn't work in an async context so the example you gave doesn't make complete sense

Sorry for the late response, I've been busy.
Does err ever make sense in an async context? Would it be better to just remove err from the examples?

@JamesMcMahon

Does err ever make sense in an async context?

Yes, usually.
gulp.task('one', function (cb) {
  fs.writeFile('filename', 'data', opts, function (err) {
    if (err) return cb(err);
    console.log("Success!");
    cb(null);
  });
});

Closed by #1169


creating blob in the readium (because of this ,facning data migration)

hi , we have found that there is blob creation in the readium library and assigning that value to the iframe src , and for the data-src  assigning differently is there any particular reason for doing that, please consider this one as priority

please find this attachment

in the above attachment there are two different values for the src and data-src

Blobs URI are used when some pre-processing is required on EPUB resources, or when the EPUB container itself is zipped instead of "exploded" (i.e. unzipped).
For example, drag-and-dropping an EPUB file onto the web/cloud reader triggers a code path that stores the EPUB in memory, and extracts / inflates resources from the EPUB container dynamically, whenever the webview requests them (i.e. byte ranges / binary buffers => Blob URIs).
Another example is when loading an exploded EPUB (i.e. not zipped) from an HTTP server, and when some resources are "encrypted" (e.g. obfuscated fonts). In this case, a wholesale replacement of all font references (i.e. URLs) is necessary, and unfortunately this is a "viral" behaviour that impacts all associated resources: fonts faces are declared in CSS, so the stylesheets must be "patched" and consequently served as Blob URIs ... but stylesheets are referenced from HTML files, so they too must be converted to binary blobs.
Finally, there is the base case of the top-level HTML file (i.e. the one passed to its host iframe), which needs to be injected "ahead of time" with custom CSS or MathJax (this is from the top of my head, I do not remember exactly). When this is needed, the HTML URL becomes a Blob URI, and the <head><base href="..."> construct is used so that resources linked from the HTML document can simply resolve with their normal URL (i.e. not Blob).


ensure infrastructureName quoted

our automated testing uses travis build numbers and without the quotes this fails. An example travis build number is 52872630

/lgtm
/approve
hopefully this will be replaced soon :)


Adjust visual elements on front page

This includes mostly the Ilmastoroolikysely and Oppimisvisa cards along with "welcome" header and subheader of site.

[ ] On mobile
[ ] On desktop
[x] Add pictures
[ ] Add new texts

Pull request #489 adds new images, added also new texts to roleSurvey and factQuiz cards on front page. Still need to add the front page header ("welcome" text)  and its subheader. Further visual adjustments need to be made towards mobile view especially, which are next on my task list.

Pull request #494 fixes issues for card content scaling on mobile screens. Still need to adjust header and sub-header size or even font. Furthermore we should add padding between elements as the elements especially hug the mobile screen on sides.


sort includes

My IDE keeps complaining about these unsorted includes so I sorted them in alphabetic order.

Makes it more readable.
Thanks. Merged.


Typos and formatting on EntitySetMapping Class documentation

The documentation for the EntitySetMapping Class is below the normal excellent standard for Microsoft .NET documentation.

Typo "EnitytSet" instead of "EntitySet" (presumably)
Example is hard to follow due to formatting, sentence structure and strange capitalization. Could also be clearer and/or elaborated.

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

ID: 21f3953c-3ecb-b9de-854a-ada5ef742d7c
Version Independent ID: d70f18c9-67b9-be07-5f6e-0b6dbab741f1
Content: EntitySetMapping Class (System.Data.Entity.Core.Mapping)
Content Source: EF/xml/System.Data.Entity.Core.Mapping/EntitySetMapping.xml
Product: entity-framework
Technology: entity-framework-6
GitHub Login: @dotnet-bot
Microsoft Alias: divega

Thank you for requesting a change to the EF6 documentation. We are prioritizing improvements to the EF Core documentation and at this time we are not making changes to the EF6 documentation.
However, we will consider well-written community pull requests to improve the EF6 documentation. The documentation source is located in the EntityFramework.Docs repository on GitHub. To submit a pull request:

Find the file to update and click on it to open

Click on the edit button (✏) to edit in your web browser

When done, add a title and description, then use the "Create a new branch and start a pull request." option.

(Note that you can also clone the docs repo and do a GitHub PR in the normal way.)


add announcing_plugins_and_cloud_security_with_falco.md file

This submission adds an announcement in the content / zh / blog / directory of the project_ plugins_ and_ cloud_ security_ with_ falco.md Chinese documents are convenient for readers to get a better reading experience

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

c00c31a add announcing_plugins_and_cloud_security_with_falco.md file, Bring better reading experience to readers
50d8fa7 Update announcing_plugins_and_cloud_security_with_falco.md
8c3acda Update announcing_plugins_and_cloud_security_with_falco.md
7ff449b Update announcing_plugins_and_cloud_security_with_falco.md

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: qinzhiben11
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @qinzhiben11! It looks like this is your first PR to falcosecurity/falco-website 🎉

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

c00c31a add announcing_plugins_and_cloud_security_with_falco.md file, Bring better reading experience to readers
50d8fa7 Update announcing_plugins_and_cloud_security_with_falco.md
8c3acda Update announcing_plugins_and_cloud_security_with_falco.md
7ff449b Update announcing_plugins_and_cloud_security_with_falco.md
89f80ae add announcing_plugins_and_cloud_security_with_falco.md file, Bring better reading experience to readers
452bbac Merge branch 'master' of github.com:qinzhiben11/falco-website

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: qinzhiben11, radhikapc
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [radhikapc]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: d8c6bc784f0f245849cf66b76af4c5e9f9cac1df

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale


Correct styling of GitHub

I readily admit this is not very important but it bothers me every time I visit the site (and am not logged in) :P

Haha! Hey! All improvements are important!


CodeNav shows interfaces methods that are implemented in parent

Hi,
There is a strange issue on VS2017: CodeNav shows methods from parent interface, even if they are not implemented in the class. So, it shows methods that not exists :
Here, the region "IViewModel", this methods are implemented in parent and not in the current class:

Exemple:
Interface1 : Interface2
class A : B, Interface1 {}
CodeNav shows methods from Interface2 in class A.
And it shows methods from Interface1 even if they are implemented in B.
Result: the navigation don't work well in this case, and sometimes causes a VS crash.
NB: I'm new to github, I don't know how to make this issue a "bug".
Thanks

Thanks for reporting!
Will look into this :)

Thanks for including the small example, that was really helpful in reproducing the bug.
Grab the new release! 🚀

So quickly !
Thanks !
There's still maybe a small behavior, this time concerning WPF XAML controls, and methods that are defined in *.g.i.cs files, like InitializeComponent().

I know this is difficult to determine if it's a "bug" or not, because it's a partial class, and the methods are in the class, but not in the file. I think that CodeNav should only show methods that are in files, not in class. Or if it shows this methods, it maybe should coloring them differently ?
Like this :

I let you choose :)
Thanks again

Again thanks for reporting!
Will look into it :)

Grab the new release! :rocket:

Thanks !
Updated to 6.3.362, closed & reopened VS, but I don't see any change ?

Hmm that is weird. It is working on my side, testing with a custom XAML control.
Please verify the installed version of CodeNav, maybe the update failed?
Could you provide a small example piece of code to reproduce?

Just have uninstalled / reinstalled CodeNav, the version is 6.3.362:

There is still the behavior on partial classes (xaml views, InitializeComposent method for example):

Actually, the same issue seems happen on partial classes that implements multiples interfaces in different files:
File 1: partial class A: I1
File 2: partial class A: I2...
Exemple, CodeNav seems to show every methods that are in the class:

Maybe the issue concerns partial classes globally, not only with interfaced ones ? (removing the interface still showing methods that are not in the file)
Or maybe some kind of refreshing issue ?
Please let me know if you need something else.
Thanks again

Thanks again!
Able to reproduce the problem, looking into it :)

Let's try it again, Grab the new release (v6.3.363)! 🚀

Fast & efficient ! Problem solved !
Thanks a lot


package ThriveGmbH.BottomNavigationBar.XF 1.0.2 is not compatible with netstandard1.4

any solution?

I had to add the classes manually to the android project and forms project. The only thing that doesn`t work is the tab color change.

@n0tc0ding  But it's compatible with netstandard?

@angelru , Is it the same problem with the  xamarin version? I could not install the nugget in my xamarin forms project so I had to add all the classes manually. It worked fine except the tab colors.

@n0tc0ding  but can you used PCL or NET Standard Class Library? thanks a lot!

@angelru yes it works on PCL


extension/ext: Add missing length-rounding in cmd_set_sample_mask()

As stated in the vulkan specification for vkCmdSetSampleMaskEXT: https://registry.khronos.org/vulkan/specs/1.3-extensions/man/html/vkCmdSetSampleMaskEXT.html
sample_mask must be an array of size ceil(samples/32). However right now the ash wrapper for this function is checking that it is of size floor(samples/32), making this wrapper unusable for any sample count less than 32.

Ah, div_ceil was only stabilized in Rust 1.73. Do you want me to rewrite this function the classic way ((samples.as_raw() + 31) / 32 as usize), or let it be this way as I'm seeing you are planning to increase the MSRV anyway?

That MSRV bump has already happened, to 1.69 which the CI is enforcing. We don't have a strict (written down) MSRV policy other than trying to not bump too eagerly (i.e. at least 6 months) and keeping them to semver-breaking releases.
Having div_ceil over hand-written rounding in two instances isn't enough to warrant an MSRV bump. On the other hand the October 5th release of Rust 1.73 is nearing that same 6-month mark, and could allow us to use more features added in those 4 releases in future patch-releases that we don't yet know about.

I see, I changed the content of this PR so it doesn't use div_ceil anymore.

We can make a separate PR to preemptively bump the MSRV if deemed necessary.
This PR also reminded me of an overwritten setter for VkPipelineMultisampleStateCreateInfo::pSampleMask in:
https://github.com/ash-rs/ash/blob/71387e942ff769cfefe824179c70b87e537f05ba/generator/src/lib.rs#L2014-L2031
This is to treat nullability properly (where Rust actually has a non-NULL pointer for &[] with length 0) and we might use that same override to also set rasterization_samples: vk::SampleCountFlags with similar validation to these commands, but not a requirement.
Also, just to make sure, unlike VkPipelineMultisampleStateCreateInfo, neither of these two cmd_set_sample_mask() functions allow pSampleMask to be NULL to "treat the mask as if all bits were set to 1".

Also, just to make sure, unlike VkPipelineMultisampleStateCreateInfo, neither of these two cmd_set_sample_mask() functions allow pSampleMask to be NULL to "treat the mask as if all bits were set to 1".

Yes, the fact that pSampleMask has slightly different semantics in VkPipelineMultisampleStateCreateInfo compared to vkCmdSetSampleMaskEXT is a bit counter-intuitive (although at least the validation layer should be able to detect easily any misuse).


How to play with examples

Maybe I missed it, but I have a suggestion to the examples: Can somewhere in the frist examples stated how to build and execute the examples? There are some build.sh scripts, but one can also give arguments like the executor for example ./simple-solver "omp", if it is correct. This would make it easier for any new students without large programming knowledge to play with the tutorials.

There is an option GINKGO_BUILD_EXAMPLES, and its default value is ON. After building the code, there will be a directory named examples in the build directory, where the built results (or executable binary files) are kept. Before running these files, export a certain environment variable(s) for the sake of performance or device selection.
I also agree with you that a few individual examples should be provided for learning how to use the derived Ginkgo libraries in real applications and to build the code, because existing examples are built along with the Ginkgo libraries.


Global Settings -Feature Flags is out of sync between Dashboard and Cluster Management UI

bug on v2.5-head 87006de
We can activate or deactivate features on both the dashboard and cluster management UI, but the states of features are not synced between those two UIs.
For example, deactivating the istio-virtual-service-ui on the dashboard does not change its state in cluster management UI.
This happens to all three existing feature flags.

I can't reproduce this issue - the states do stay in sync - note that in the Ember UI,they show as disabled where as in the new UI they show as inactive - I have changed this so that the new UI also shows disabled.

Hi @nwmac  I see  inactive became disabled in Dashboard UI, but I still see the bug in my setup.
the following screen is after I disabled istio-virtual-service-ui in the Ember UI

the following screen is after I disabled istio-virtual-service-ui in the Dashboard UI

In both cases, I have refreshed both pages multiple times to make sure it gets the latest values from the server


[Refactor] Migrate the entire ProgressAD codebase to Typescript with strict type checking.

It's Finally Time.
I'd like to add new/better features to the application, but using plain JavaScript is not effective or efficient as the application grows.
I can't explain the amount of "property doesn't exist on undefined" and other lack-of-type-based issues we've gotten throughout the early development process.
So, ProgressAD will be migrating to TypeScript.
This will be incremental, so I'll be doing the backend first, and then hitting the frontend.

Splitting this into two PR's to decouple the changes into what makes the most sense.
The first PR will be for the backend, under the Backend-TypeScript-Migration branch.
The second will be the Frontend-TypeScript-Migration branch.


Migrated from original Tachiyomi, trying to install Hitomi extension results in odd failure loop

Steps to reproduce

Have Extension repo installed
Go to 'browse' -> 'extensions'
Scroll past installed extensions, down to Hitomi extension (v.1.4.33)
Hit the 'install arrow'

Expected behavior

Go to 'browse' -> 'extensions'
& Scroll past installed extensions, down to Hitomi extension (v.1.4.33)
Hit the 'install arrow'
Appears to go through install process
Extension appears in 'Installed' list, works like every other extension.

Actual behavior

Go to 'browse' -> 'extensions'
Scroll past installed extensions, down to Hitomi extension (v.1.4.33)
Hit the 'install arrow'
Appears to go through install process
Extension not appear in installed extensions list
Try to install again
Now asks if I want to update extension, choose 'yes'
Appears to go through update process
Does not appear in installed extensions list
Retry with Legacy, PackageInstaller and Shizuku Installers, same result

Crash logs
`App version: 1.10.5 (standard, 6c563d7, 65, 2024-03-02T20:42Z)
Preview build: 0
Android version: 12 (SDK 31; build SP1A.210812.016.G975U1UEU9IXE1)
Device brand: samsung
Device manufacturer: samsung
Device name: beyond2q (beyond2qlteue)
Device model: SM-G975U1
WebView: Android System WebView 128.0.6613.127
--------- beginning of main
09-13 11:20:15.367 5054 5054 E de.tachiyomi.s: Not starting debugger since process cannot load the jdwp agent.
09-13 11:20:15.368 5054 5054 E USNET : USNET: appName: eu.kanade.tachiyomi.sy
09-13 11:20:15.655 5054 5097 E de.tachiyomi.s: No package ID 78 found for ID 0x780b000f.
09-13 11:20:15.655 5054 5097 E de.tachiyomi.s: No package ID 78 found for ID 0x780b000f.
09-13 11:20:16.117 5054 5054 E de.tachiyomi.s: Invalid ID 0x00000000.
09-13 11:21:25.320 6067 6067 E de.tachiyomi.s: Not starting debugger since process cannot load the jdwp agent.
09-13 11:21:25.321 6067 6067 E USNET : USNET: appName: eu.kanade.tachiyomi.sy
09-13 11:21:25.501 6067 6125 E de.tachiyomi.s: No package ID 78 found for ID 0x780b000f.
09-13 11:21:25.502 6067 6125 E de.tachiyomi.s: No package ID 78 found for ID 0x780b000f.
09-13 11:21:25.859 6067 6067 E de.tachiyomi.s: Invalid ID 0x00000000.
09-13 11:29:02.442 10967 10967 E de.tachiyomi.s: Not starting debugger since process cannot load the jdwp agent.
09-13 11:29:02.443 10967 10967 E USNET : USNET: appName: eu.kanade.tachiyomi.sy
09-13 11:29:02.624 10967 11009 E de.tachiyomi.s: No package ID 78 found for ID 0x780b000f.
09-13 11:29:02.624 10967 11009 E de.tachiyomi.s: No package ID 78 found for ID 0x780b000f.
09-13 11:29:03.014 10967 10967 E de.tachiyomi.s: Invalid ID 0x00000000.
09-13 11:29:03.593 10967 10967 E ViewRootImpl@30859d1[MainActivity]: Surface is not valid.
09-13 11:29:03.615 10967 10967 E ViewRootImpl@30859d1[MainActivity]: Surface is not valid.
09-13 11:30:41.214 10967 11120 E StandaloneCoroutine: Failed to install extension 11621 content://eu.kanade.tachiyomi.sy.provider/ext_files/Download/tachiyomi-all.hitomi-v1.4.33.apk
09-13 11:30:41.214 10967 11120 E StandaloneCoroutine: java.lang.RuntimeException: Failed to commit install session 905788419
09-13 11:30:41.214 10967 11120 E StandaloneCoroutine: at eu.kanade.tachiyomi.extension.installer.ShizukuInstaller$processEntry$1.invokeSuspend(SourceFile:243)
09-13 11:30:41.214 10967 11120 E StandaloneCoroutine: at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(SourceFile:12)
09-13 11:30:41.214 10967 11120 E StandaloneCoroutine: at kotlinx.coroutines.DispatchedTask.run(SourceFile:112)
09-13 11:30:41.214 10967 11120 E StandaloneCoroutine: at kotlinx.coroutines.internal.LimitedDispatcher$Worker.run(SourceFile:4)
09-13 11:30:41.214 10967 11120 E StandaloneCoroutine: at kotlinx.coroutines.scheduling.TaskImpl.run(SourceFile:3)
09-13 11:30:41.214 10967 11120 E StandaloneCoroutine: at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(SourceFile:1)
09-13 11:30:41.214 10967 11120 E StandaloneCoroutine: at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(SourceFile:63)
09-13 11:30:51.432 10967 11174 E StandaloneCoroutine: Failed to install extension 11622 content://eu.kanade.tachiyomi.sy.provider/ext_files/Download/tachiyomi-all.hitomi-v1.4.33.apk
09-13 11:30:51.432 10967 11174 E StandaloneCoroutine: java.lang.RuntimeException: Failed to commit install session 67319604
09-13 11:30:51.432 10967 11174 E StandaloneCoroutine: at eu.kanade.tachiyomi.extension.installer.ShizukuInstaller$processEntry$1.invokeSuspend(SourceFile:243)
09-13 11:30:51.432 10967 11174 E StandaloneCoroutine: at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(SourceFile:12)
09-13 11:30:51.432 10967 11174 E StandaloneCoroutine: at kotlinx.coroutines.DispatchedTask.run(SourceFile:112)
09-13 11:30:51.432 10967 11174 E StandaloneCoroutine: at kotlinx.coroutines.internal.LimitedDispatcher$Worker.run(SourceFile:4)
09-13 11:30:51.432 10967 11174 E StandaloneCoroutine: at kotlinx.coroutines.scheduling.TaskImpl.run(SourceFile:3)
09-13 11:30:51.432 10967 11174 E StandaloneCoroutine: at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(SourceFile:1)
09-13 11:30:51.432 10967 11174 E StandaloneCoroutine: at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(SourceFile:63)
09-13 11:33:35.431 14834 14834 E de.tachiyomi.s: Not starting debugger since process cannot load the jdwp agent.`
TachiyomiSY version
1.10.5
Android version
Android 12
Device
Samsung Galaxy S10+
Other details
Every other extension installs wonderfully, including those restored from OG Tachiyomi (after un/reinstalling them) - did not have Hitomi extension on original Tachiyomi for ages, since it had been broken for a long time.
Everything looks normal, except after the installation finishes, the extension isn't there. Trying it over results in it trying to update the non-existent extension, but still results in nothing, multiple attempts made.
Tried to find where extensions are stored on device in hope of deleting any potential corrupted version of it that might be causing the issue, but could not locate.
All I can do in-app is hit install, be asked if I want to update the extension, say yes, and repeat at infinitum.
There is clearly some issue shown in the log, but I am not quite sure what exactly it is, and have no clue as to how I would fix it even if I did.
My hope is that someone smarter than me will understand this, and, ideally, even figure out a solution. My deepest gratitude in advance to whomever that may be 🙇♂
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 1.10.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

There's a version of that extension still installed on your phone, check your Phone Settings > App list, or check your Samsung Secure Folder application (this is not your File Manager). Also, change it back to the PackageInstaller or whatever


NFT cards explore + landing

landing page:

missing bottom stroke on the img

explore:

network non caps (only first letter)
network to 12px, as on landing page

👋

👋

can we please make the img fit crop, so there are no white lines. Since we are displaying it like this in gallery item and we will soon also provide full screen option.

👋

can we please make the img fit crop, so there are no white lines. Since we are displaying it like this in gallery item and we will soon also provide full screen option.

Will check it


update dashboards in init SQL script

@gmacario @arol-varesi please review this PR when you have a moment. Thank you very much!


Multi-binding causes infinite reconciliation loop

Hi, I'm testing the controller using the multi-binding feature.
It works perfectly but if I check the logs of the controller it seems like it is continuously reconciling, and printing logs.
Can someone check if this is the correct behavior?
I had a similar problem with the service binding operator of redhat but that one wasn't supposed to support multi binding.
It seems like the controller is in an infinite loop.
As an addition: if I remove completely the label multi-binding it behaves the same
This is the service binding I created:
---
apiVersion: servicebinding.io/v1
kind: ServiceBinding
metadata:
  name: multi-binding-1
  namespace: customer1-workload
  labels:
    multi-binding: "true"
spec:
  # direct Secret reference is used for compatibility, but not recommended for dynamically provisioned services
  service:
    apiVersion: v1
    kind: Secret
    name: multi-binding-1
  workload:
    apiVersion: apps/v1
    kind: Deployment
    name: podinfo
  env:
  - name: MULTI_BINDING_1
    key: number

---
apiVersion: servicebinding.io/v1
kind: ServiceBinding
metadata:
  name: multi-binding-2
  namespace: customer1-workload
  labels:
    multi-binding: "true"
spec:
  # direct Secret reference is used for compatibility, but not recommended for dynamically provisioned services
  service:
    apiVersion: v1
    kind: Secret
    name: multi-binding-2
  workload:
    apiVersion: apps/v1
    kind: Deployment
    name: podinfo
  env:
  - name: MULTI_BINDING_2
    key: number

manager 2024-04-19T13:55:12Z    INFO    admission    mutating resource    {"object": {"name":"podinfo","namespace":"customer1-workload"}, "namespace": "customer1-workload", "name": "podinfo", "resource": {"group":"apps","version":"v1","resource":"deployments"}, "user": "system:serviceaccount:servicebinding-system:servicebinding-controller-manager", "requestID": "7d157ced-9eec-40cd-a86e-79623651c8ca", "UID": "7d157ced-9eec-40cd-a86e-79623651c8ca", "kind": "apps/v1, Kind=Deployment", "resource": {"group":"apps","version":"v1","resource":"deployments"}, "operation": "UPDATE", "patch": [{"op":"remove","path":"/spec/template/spec/volumes/1/projected/defaultMode"},{"op":"remove","path":"/spec/template/spec/volumes/2/projected/defaultMode"}]}
manager 2024-04-19T13:55:12Z    INFO    ServiceBindingResourceReconciler.WithFinalizer.3.PatchWorkloads    updated resource    {"controller": "servicebinding", "controllerGroup": "servicebinding.io", "controllerKind": "ServiceBinding", "ServiceBinding": {"name":"multi-binding-1","namespace":"customer1-workload"}, "namespace": "customer1-workload", "name": "multi-binding-1", "reconcileID": "f1c3a403-fb3e-4272-bacb-6014c8b8c94c", "resourceType": "servicebinding.io/v1, Kind=ServiceBinding"}
manager 2024-04-19T13:55:12Z    DEBUG    events    Updated Deployment "podinfo"    {"type": "Normal", "object": {"kind":"ServiceBinding","namespace":"customer1-workload","name":"multi-binding-1","uid":"def0c814-98cd-4542-b1ed-0f696f5a0674","apiVersion":"servicebinding.io/v1","resourceVersion":"206043"}, "reason": "Updated"}

I was able to reproduce this with the multi-binding sample. A partial fix is available in #413.


Update example projects

This updates the example projects listed on the homepage for each license.

add a 3rd example (.NET Core) to MIT for consistency
replace SVN & NuGet with more more modern and well-known examples
(Swift, Docker) for Apache
Add a GPLv3 project to examples

There are not currently guidelines for how we choose the projects. We (@amateurhuman and I) tried to choose projects that:

are more likely to be known by people in most ecosystems (sorry NuGet)
represent different types of projects, like programming languages, libraries, and utilities

Seems an improvement to me. I'd suggest Android rather than Docker for Apache, just because former is a mass consumer brand.
I was eventually going to suggest removing the examples altogether. I don't see how they help a newcomer understand the licenses. Only people who follow projects closely have any idea of the reasons why projects use particular licenses, and often the reasons are highly idiosyncratic anyway.

Nice. :+1:

I was eventually going to suggest removing the examples altogether. I don't see how they help a newcomer understand the licenses.

I think they're helpful for newcomers to be able to relate to their community. If I'm working on an open source developer tool, I might want to consider using the same license as something like Docker since that may be what my community is comfortable with. I'd say Android developers likely want to release code under the same license as Android (some do it for complex technical reasons around license compatibility, others do it because they want to fit with the community). Either way, I think they guide users in a helpful way.


Add Python 3 aiohttp TLS client

The default license is OK for me.

Thanks for the submission!
Running on Fedora 24 (Python 3.5.1 + whatever aiohttp you get with dnf and the latest one from pip) I get:
[pp@the python3-aiohttp]$ python3 run.py wrong.host.badssl.com 443
Exception in callback None                                                      handle:                                                       Traceback (most recent call last):                                                File "/usr/lib64/python3.5/asyncio/events.py", line 125, in _run                  self._callback(self._args)                                                   File "/usr/lib64/python3.5/asyncio/selector_events.py", line 671, in _read_ready                                                                                self._protocol.data_received(data)                                            File "/usr/lib64/python3.5/asyncio/sslproto.py", line 492, in data_received       ssldata, appdata = self._sslpipe.feed_ssldata(data)                           File "/usr/lib64/python3.5/asyncio/sslproto.py", line 200, in feed_ssldata
self._sslobj.do_handshake()
File "/usr/lib64/python3.5/ssl.py", line 633, in do_handshake                     match_hostname(self.getpeercert(), self.server_hostname)
File "/usr/lib64/python3.5/ssl.py", line 296, in match_hostname                   % (hostname, ', '.join(map(repr, dnsnames))))
ssl.CertificateError: hostname 'wrong.host.badssl.com' doesn't match either of '.badssl.com', 'badssl.com'
REJECT
which is ok vs. the stub spec, but could be a bit cleaner? E.g. just the last line of the traceback.

@ppietikainen, I stumbled upon this too, but I don't quite understand what's going on.
My code doesn't do anything to print this traceback. It's asyncio that logs it for some reason.
I suspect it's a bug in aiohttp or asyncio.

According to the aiohttp maintainer, this is asyncio bug:
https://github.com/python/asyncio/issues/404


Incorrect AtMostOne cardinality inference on unless conflict...else

Follow up from discord

Filing here instead of on JS because this is believed to be a cardinality inference issue not a JS issue.

EdgeDB Version: 3.2
EdgeDB CLI Version: 3.4.0+4d95a2c
OS Version: macos 10.13

Steps to Reproduce:

Create an upsert query

  insert A {
    a:=<str>$a,
    b:=<str>$b
  } unless conflict on (.a) else (
    update A set {
      b:=<str>$b
    }
  )

Use the typescript query generator to create typescript files, type is inferred as | null, which is unexpected as the insert case should obviously always return an object, and the conflict case by definition should also have an object.

export type TestReturns = {
  "id": string;
} | null;

Schema:
type A {
  property a: str {
    constraint exclusive;
  }
  property b: str;
}

Interestingly, while the QB style EdgeQL of this query also has the issue, QB itself does not.
e.insert(e.A, {
  b: "b",
  a: "a",
}).unlessConflict(conflictingA => ({
  on: conflictingA.a,
  else: e.update(conflictingA, () => ({
    set: { b: "b" },
  })),
}));

correctly infers to
{
    id: string;
}

underlying EdgeQL,
INSERT default::A {
  b := "b",
  a := "a"
}
UNLESS CONFLICT ON default::A.a
ELSE ((WITH
  __scope_0_defaultA := default::A
UPDATE __scope_0_defaultA SET {
  b := "b"
}))

underlying inferred type (via query files),
{
  "id": string;
} | null;

There is the case where an access policy prevents you from seeing the existing object, but then the insert throws an error, not {}.

Confirmed this is still occuring in 4.0.


bug(ssim/ms-ssim): MS-SSIM feature was fixed to provide valid values.

Closes #39 and #13.

Rebased onto current master. I suggest to merge this PR first, then #38.

As new tests were introduced to compare our implementation with official TensorFlow, the following bug was detected. The previous version of MS-SSIM appears to estimate wrong values due to incorrect evaluation of SSIM at each scale and final aggregation of the results. More details are available at #39.
This goals of this PR are to fix the MS-SSIM functionality and to provide test scenarios.


Two problems

Why does the out file say it doesnt exist - of course it doesnt exist ... ;)
> recast --from yaml --to json --out 80c5a4ad-78a0-4a01-83e1-c4145cbf9946.json 80c5a4ad-78a0-4a01-83e1-c4145cbf9946.yaml
Error:
   0: No such file or directory (os error 2)

Note: 80c5a4ad-78a0-4a01-83e1-c4145cbf9946.json

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.

Error mentions both TOML and YAML at the same time?
> recast --from yaml --to json  80c5a4ad-78a0-4a01-83e1-c4145cbf9946.yaml > 80c5a4ad-78a0-4a01-83e1-c4145cbf9946.json
Error:
   0: foo.bar.baz: invalid type: unit value, expected any valid TOML value at line 14 column 14

Note: cannot deserialize as yaml

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.

Why does the out file say it doesnt exist - of course it doesnt exist ... ;)

My bad. Fixed on 223bc72a0021fea026073af5c39e5c9198658333 (master). Please run cargo install --force recast or cargo install --git https://github.com/Altair-Bueno/recast to update

Error mentions both TOML and YAML at the same time?

Internally, recast uses serde_toml::Value, as serde_yaml::Value and serde_json::Value cannot be consistently deserialized into TOML (toml requires key/values to be sorted so subtables appear last). Something similar happens if you try to deserialize null, as TOML doesn't have null values. Haven't found time to write my own Value alternative that solves these issues yet.

Thank you Altair!


copy_kvp_client.sh Connection refused

[13:11:27.578]+ ../../../common/copy_kvp_client.sh
[13:11:27.578]--2023-11-04 13:11:29--  https://raw.githubusercontent.com/microsoft/lis-test/master/WS2012R2/lisa/tools/KVP/kvp_client.c
[13:11:27.578]Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, ::
[13:11:27.578]Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... failed: Connection refused.
[13:11:27.578]Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|::|:443... failed: Connection refused.
[13:11:58.577]root@rjz-MSI:~/azhpc-images/ubuntu/ubuntu-20.x/ubuntu-20.04-hpc# ping raw.githubusercontent.com
[13:11:58.577]PING raw.githubusercontent.com (<IP_ADDRESS>) 56(84) bytes of data.
[13:11:58.577]64 bytes from localhost (<IP_ADDRESS>): icmp_seq=1 ttl=64 time=0.031 ms
[13:11:59.605]64 bytes from localhost (<IP_ADDRESS>): icmp_seq=2 ttl=64 time=0.030 ms

@githubtjl this is the case when the machine is not connected to the internet. Please re-check


Contributor Console - CLA Group banner covers wording.

Summary
When a contributor selects "corporate" and then their company, and the company has not added them to the approved list - the name of the cla group is covering the message at the top of the screen.
https://contributor.v1.easycla.lfx.linuxfoundation.org/#/cla/project/7ddba198-d8ff-4bde-a997-1d772d1f43d4/user/1a75f09e-c97e-4392-b438-c9ac606f20b6
You can reproduce from https://github.com/CLATestOrg/adetest/pull/26 "not covered" message and select corporate, and "Demo Company" from the list.
You'll see this screen:

This happens on both my large monitor and my smaller laptop screen.

Verified this is fixed in production


Nothing is saved in bin

Hello,
I have connected my 25lc080 to my arduino and i ran the C program and i am not getting any eeprom data saved to the bin file.
In arduino serial console. if i press random letters i get some dump data.

I had a quick glance at the datasheet of your EEPROM. It seems to use 16 bit addresses, while my code is written for 24 bit addresses.
Please test again after deleting this line:
https://github.com/andre-richter/arduino-spi-read-eeprom/blob/master/spi_read_eeprom.ino#L27

Thanks. It worked. :) Even without removing it.
Problem was with the serial port. I ran the program and tried sending some data using serial terminal to initialize the data dump!


Feature/tooltips improvements

As per @jakesyl, creating a new PR pointing at dev

We've recently announced a feature complete status for v1.x and will only be accepting bug fixes for v1.x. Be sure to take a look at v2.0-dev and it's new features that are either finished or currently under development.  If you see your feature is still not implemented, feel free to submit another PR to the v2.0-dev (or later) branch for your feature.


Add MMRL Loyalty Token

Hi,
When we can expect merge?
Regards

Address not active

Dear sir, We are still waiting for adding our MMRL Logo !Best regards,Marius Jedlickij Mob/ WhatsApp +971 508828056 Sent from my iPhoneOn 13 Sep 2023, at 13:37, MRabenda @.***> wrote:
Closed #1165.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Hi,Kindly add our MMRL logo & address will come active. Regards,Marius Jedlickij Mob/WhatsApp +971508828056Elite Icon Group @.) • Instagram photos and videosinstagram.comSent from my iPhoneOn 13 Sep 2023, at 13:37, MRabenda @.> wrote:
Address not active
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


fix(DockerBuilder): when doing RUN abort after an amount of time?

Currently, we wait forever for the RUN instruction to finish: https://github.com/stencila/dockter/blob/v0.2.6/src/DockerBuilder.ts#L208
Probably should have some sort of maximum time limit.

Difficult to cover all scenarios, but it can be annoying to have a build being interrupted over an arbitrary (default) time limit; in my experience RUN commands map to installations that use the network (downloading packages or resolving a long list of dependencies) and hence can be very slow in some situations due to limited bandwidth.
If RUN is being used to run a package manager, it would have its own timeouts that would raise an error and interrupt execution when reached; those can be overridden with the standard primitives of that package manager rather than with an additional Dockter-specific notation.
There is also the possibility of wrapping dangerous instructions in timeout commands which I've seen commonly used, but at runtime more than during build phases.

@giorgiosironi: yeah, I think you are right - it would be annoying. This was maybe a kneejerk suggestion to avoid getting stuck in that while loop forever. But since most of the time this would be run from the CLI, the user can always press Ctrl+C. I'll close this issue.


make react internals usage optional to support preact

Hi there,
I have a project where I'm using URQL with Preact, which works really well together apart from an issue with URQL devtools exchange.
It breaks as it relies on __SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED to read the name of the initiator component when a GraphQL request is sent, but unfortunately that internal variable is not available with Preact.
As I see this is the only usage and in the current implementation it's optional as well, as the getDisplayName function already checks the presence of ReactCurrentOwner.owner.
With these minor adjustments the devtools works fine with Preact, apart from displaying Component as fallback for the component name in the Events tab.

Hey @dios-david thanks for the issue!
We recently addressed this (https://github.com/FormidableLabs/urql-devtools-exchange/issues/25 and https://github.com/FormidableLabs/urql-devtools-exchange/pull/28). Expect a new release in the coming week!


Beautify doesn't beautify when my Gemfile specifies ruby version

Description
When I specify a ruby version in my Gemfile, beautifier stops beautifying.
I'm using rvm and my Gemfile has the following line
...
ruby '2.3.1'
...

Whenever I open terminal, i get the following message:
RVM used your Gemfile for selecting Ruby, it is all fine - Heroku does that too,
you can ignore these warnings with 'rvm rvmrc warning ignore /Users/gilani/Sandbox/mailman/Gemfile'.
To ignore the warning for all files run 'rvm rvmrc warning ignore allGemfiles'.

When I edit files Rubocop shows me the errors, and I can fix them manually, but when I try beautifying, it spins up, and does nothing.
Expected Results
The code should have been beautified. It does so if I remove ruby '2.3.1' from my Gemfile, but with it, it does nothing.
Steps to Reproduce

Add code to Atom editor
Generate a Rails project
Add ruby '2.3.1' at the top, right after source 'https://rubygems.org'
Write some ugly code
Run command Atom Beautify: Beautify Editor
The code does not beautify!

Debug
Here is a link to the debug.md Gist: https://gist.github.com/35567eaba444214176a4cddfe4a9df87
Checklist

[N] I have tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[Y] I have reloaded (or restarted) Atom to ensure it is not a caching issue
[Y] Searched for existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[Y] Generated debugging information and added link for debug.md Gist to this issue

Disregard the first debug.md gist, use this instead:
https://gist.github.com/ffdca00b244f48c8eb92600fa4dd6d99
This one was created with the ruby '2.3.1' line added in the Gemfile, the last one was created without it.

No idea what happened. It was so long ago. Everything works me.


yii\bootstrap\Nav method isItemActive

/**
     * Checks whether a menu item is active.
     * This is done by checking if [[route]] and [[params]] match that specified in the `url` option of the menu item.
     * When the `url` option of a menu item is specified in terms of an array, its first element is treated
     * as the route for the item and the rest of the elements are the associated parameters.
     * Only when its route and parameters match [[route]] and [[params]], respectively, will a menu item
     * be considered active.
     * @param array $item the menu item to be checked
     * @return boolean whether the menu item is active
     */
    protected function isItemActive($item)
    {
        if (isset($item['url']) && is_array($item['url']) && isset($item['url'][0])) {
            $route = $item['url'][0];
            if ($route[0] !== '/' && Yii::$app->controller) {
！！！！！！！！！！！！//  here when $route is only an actionId will not return true
                $route = Yii::$app->controller->module->getUniqueId() . '/' . $route;
            }
            if (ltrim($route, '/') !== $this->route) {
                return false;
            }
            unset($item['url']['#']);
            if (count($item['url']) > 1) {
                $params = $item['url'];
                unset($params[0]);
                foreach ($params as $name => $value) {
                    if ($value !== null && (!isset($this->params[$name]) || $this->params[$name] != $value)) {
                        return false;
                    }
                }
            }

            return true;
        }

        return false;
    }

@yidashi why the $route is only an actionId? It should be a complete route!

@rossoneri because yii\helpers\Url::toRoute support an actionId,bug here does not support.
see in yii\helpers\BaseUrl
if (strpos($route, '/') === false) {
            // empty or an action ID
            return $route === '' ? Yii::$app->controller->getRoute() : Yii::$app->controller->getUniqueId() . '/' . $route;
        } else {
            // relative to module
            return ltrim(Yii::$app->controller->module->getUniqueId() . '/' . $route, '/');
        }

Shouldn't it be the same?

the same Nav is supposed to be used with multiple controllers so a menu entry will point to different places dependent on the current controller. this does not seem to be correct to me. What is your use case?

I used it in the same current module and controller, because I just need a small nav.
Even in different module and controllers,when only an actionId,shoule it also mean current module and controller?
sorry,my english is not good.

your english is fine, this case has just not been considered when creating this class. We might add support for this. In the mean time you can use explicit controller and module in the route.

thanks very much

@yidashi 哥们，我的理解是Yii的Nav组件里的路由应该是以/开始的完整路由地址，如果你用完整的路由能解决你现在的问题吗？

how about a NavItemobject auto created with the $item array?
then you can just do
$navWidget->getActiveItem()->equals($item)

The route without action, "/quarter", doesn't work with isItemActive. Maybe it's possible to substitute /index in the nav widget and spare rules settings like 'url' => '/url/index' in the  in UrlManager


Feature/pre train subreddit

This enables to pre-train the network on different output labels for the same input data. Specifically to use subreddits as lables for pre-training.
In main.py the a new layer is added after adding secondary output layer, this is so that the network can generelise from the learnt prameters and when tested showed better preformance. Where in the network the secondary output should be placed should be treated as a hyperparameter.

Also, I just realised that the concatenated subreddit input is still used when pre-training on subreddits. Maybe this should not be concatenated when doing that? Seems a bit biased. At the same time... The dimensions of the weights depend the size of the input.

Use softmax for subreddit predictions.
Output layers should be last


 Unable to add a source with url git@gitlab.alibaba-inc.com:alipods/specs.git named alibaba-inc-specs.
You can try adding it manually in ~/.cocoapods/repos or via pod repo add.

I'm pretty sure I installed cocoapods successfully, is that issue about git@gitlab.alibaba-inc.com:alipods/specs.git, this impacts me to use weexpack, could anyone help me on this?

Got the same issue on weexpack 0.3.2.  My workaround is downgrading to 0.2.5.
FYI: https://github.com/weexteam/weex-pack/issues/56

Upgraded weexpax to 0.3.3, works fine.


Get_Pressure_From_DE undefined

In ppmc_cuda and plmc_cuda, the function Get_Pressure_From_DE is undefined (it only gets called when running with DE). It appears to now be defined in the hydro utilities instead of in hydro_cuda.cu, so this should be a quick fix.

Fixed with PR #164


Adding function instrumentation back

When the Hyperload issue occured, instrumentation was removed from the
makefile, which disabled the backtracing. Adding it back, but setting
L0_LowLevel folder as an exclusion zone for instrumentation.

Coverage decreased (-0.05%) to 91.26% when pulling 70bb6091eab79481ae790043448c17b7a542c37c on fixing-instrumentation into ba5cc4f5071d9e3883d07e7c423f723da176c6fe on master.


Support all three official Unity Render pipelines

This package should work with all official render pipelines:

[x] Built-in Render Pipeline
[ ] High Definition Render Pipeline
[ ] Universal Render Pipeline

I tried this already but always failed with this question: How is it possible for three versions to live in one package and conditionally compile for a certain render pipeline? Or do we need three different package versions?

The completion of #9 will hopefully make this a whole lot easier. Using assembly definitions should make it possible to create package modules that conditionally compile for each render pipeline.

https://docs.unity3d.com/Manual/cus-asmdef.html
https://docs.unity3d.com/Manual/ScriptCompilationAssemblyDefinitionFiles.html

Work is in progress #14

Merged #14


[Codegen] Extract throwIfUnsupportedFunctionParamTypeAnnotationParserError function

Summary

This PR is a task from https://github.com/facebook/react-native/issues/34872:

Extract the UnsupportedFunctionParamTypeAnnotationParserError in its own throwing function (if it does not exists already) and reuse that function passing a proper type. Then, refactor the code using a dictionary and avoiding the three different ifs in both parsers.

Changelog

[Internal] [Changed] - Extract the UnsupportedFunctionParamTypeAnnotationParserError in its own throwing function in error-utils
Test Plan

@cipolleschi I still need to do the second part of the task ("Then, refactor the code using a dictionary and avoiding the three different ifs in both parsers."), but I have some troubles with Flow.
Flow doesn't understand anymore that paramTypeAnnotation can't be of type VoidTypeAnnotation nor PromiseTypeAnnotation.

I tried to use predicate functions but I didn't manage to make it work in this case. I don't like the idea of adding a $FlowFixMe comment.
Do you have any idea?

I searched if there were other places where we encountered the same problem. I believe it is the case in this PR https://github.com/facebook/react-native/pull/34917, and I see that the if(XXX) is not in the throwIfXXX function, but I feel like it's losing the point of extracting the function.

Hi @AntoineDoubovetzky, thanks for the comment.
I understand what you are trying to say and I completely agree. However, I think we have two orders of problems:

We have the repetition of the if (type == 'something') then throw for each of the invalid types
The logic you highlighted not in the throwIfXXX function.

To keep things as simple as possible, I tried to split the problem in two steps: the PR you linked was addressing only the first one and I think it's fine: we can have another task for the second step.
Does this reasoning make sense to you?
For your task, you can tackle both points together, if you feel it! 😊


Update Help

It appears that the information related to updating the help for the ConfigurationManger module is no longer valid.
Version                     : 5.00.8968.1000

PS SFO:\> Update-Help -Module ConfigurationManager
Update-Help : Failed to update Help for the module(s) :
'ConfigurationManager'
The Update-Help command failed because the specified module does not support updatable help. Use Get-Help -Online or
look online for help for the commands in this module.
At line:1 char:1
+ Update-Help -Module ConfigurationManager
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (HelpInfoUri:Uri) [Update-Help], Exception
    + FullyQualifiedErrorId : HelpInfoUriNotFound,Microsoft.PowerShell.Commands.UpdateHelpCommand

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

ID: 7d3cfeb8-c773-1ae5-0975-ea48e20e4c3a
Version Independent ID: 7794c3e5-20ad-e389-6e18-c70628a3505a
Content: Configuration Manager PowerShell cmdlets - Configuration Manager
Content Source: docs-conceptual/overview.md
Product: configuration-manager
Technology: configmgr-sdk
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Hi @Krytical13 thanks for the feedback. It's been a known issue for a while, due to a backend bug in the module and where the update help content was stored. I'm actively working with the ConfigMgr engineering team, PowerShell product team, and PowerShell doc team to test a new help pipeline that will synchronize this doc content for the module to a location for update-help to access. I don't have a specific timeframe yet, but keep an eye on the upcoming technical preview releases for an announcement when we have it ready to evaluate.


YAML support

Pass YAML filepath instead of JSON.
Improves lazy-loadability; probably fastest path to enable #4.

Closing for now, since I think data IO is probably out-of-scope for this repo.


vSphere CPI and CSI configuration is not visible on v2.6-head rancher.

Rancher Server Setup

Rancher version: v2.6-head (11a7451)
Installation option (Docker install/Helm Chart): Docker

Information about the Cluster

Kubernetes version:
Cluster Type (Local/Downstream):

If downstream, what type of cluster? Downstream RKE2.

Describe the bug

vSphere CPI and CSI configuration is not visible on v2.6-head rancher.
To Reproduce

Install v2.6-head rancher.
Proceed to create RKE2 vSphere cluster.
Input correct vSphere credentials.
Input the required pool, instance options.
In Cluster Configuration, select Cloud Provider as vSphere. You will be prompted to configure vSphere CPI and CSI configuration in Add-On Config tab.

Important: Configure the vSphere Cloud Provider and Storage Provider options in the Add-On Config tab.

Click on the Add-On Config.

Result
A form to enter the vSphere CPI Configuration and vSphere CSI Configuration is not visible.

Expected Result

A form to enter the vSphere CPI Configuration and vSphere CSI Configuration should be visible just below the Calico Configuration.

Given the issue is just the form appearing and not appearing we believe this is a dashboard issue.

I just pulled the latest of v2.6-head and I'm not seeing this issue. Would you mind checking to see if you can still reproduce this?

Tested this issue on vSphere Environment 7 on latest v2.6-head(39befda) rancher. Issue is not active on latest rancher v2.6-head.

However, the issue still active on v2.6.4-rc4.

@codyrancher Not sure if I should close the issue.

Since the v2.6-head is ahead of the v2.6.4-rc4. I'm considering the issue as fixed. Hence, closing the issue.

Re-opening for another validation on the latest rc.

Blocked by this issue - https://github.com/rancher/dashboard/issues/5462

Issue is active on rancher v2.6.4-rc11.

This is a backend issue and duplicate of https://github.com/rancher/rancher/issues/36987


RequestParser type bool incompatibility between json and values

parser = reqparse.RequestParser()
parser.add_argument('checked', type=bool)
args = parser.parse_args()
This works fine if you send json w. location as 'json' but if the location is 'values' then regardless of what you send as an argument ('false', 0, 'False') args['checked'] is always True
I realise why this happens but it's kind of a big gotcha if you switch between location types carelessly (which is what happened to me :))

see #397


Add docs for deployment env vars

Description & motivation
Added docs for the new read-only env vars available in dbt Cloud runs
To-do before merge

Feature live in prod / GA

Pre-release docs
Is this change related to an unreleased version of dbt?

[x] No: please ensure the base branch is current

Checklist
If you added new pages (delete if not applicable):

[x] The page has been added to website/sidebars.js
[x] The new page has a unique filename

cc @barryaron ! This is the place where we update the docs for dbt Cloud. You can check out the staging preview of the site by clicking the deploy/netlify "Details" link above, or by navigating here
One quick note because it's easy to miss - this repo is public, so no client-specific info or sensitive info in this repo pls :)

Thanks drew!


[MM-18746] Fix channels autocomplete both for interactive dialog and menu

Summary
Fix channels autocomplete both for interactive dialog and menu.  This was introduced by the recent migration which I missed to check during code review.
Can be tested using mattermost demo plugin (/dialog) or running Cypress tests with "/interactive_dialog" and "/interactive_menu" folders
Ticket Link
Jira ticket - MM-18746

@enahum This is for 5.18.  Related actions migrations is also on master/5.18.  Thanks for asking.


rancher-nfs silently fails to create volume if a local volume of the same name already exists

Rancher Versions:
Server: 1.3.1
healthcheck: 0.2.0
ipsec: 0.0.2
network-services: 0.0.8
scheduler: 0.3.0
Docker Version:
1.12.3
OS and where are the hosts located? (cloud, bare metal, etc):
RancherOS
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single Node
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Create a local volume (either directly or via a container)
Create an nfs volume of the same name
Start a container that uses the nfs volume

Results:

nfs volume is created in rancher but remains in Inactive state
container will actually use the local volume
nfs storage is not used

The main issue is visibility. It was not very clear why all my nfs volumes were working except one. A left over local volume of the same name was sitting on the host and had not been cleaned up yet. Once I realized the volume name was already taken I removed the old local volume (docker volume rm ...) and my nfs volume was then created after restarting the owning container.

I had same issue, it is really difficult to find a cause, especially when "rancher-compose" ignores "rancher-nfs" during running environment "up" and then we have "locked" state, without possibility to upgrade containers to nfs...

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Use code-block for online docs in metadata.py

Looking at the API docs online, it looks like some of the docs are in markdown versus restructured text:

It's probably a good idea to find these and update, as well as fixup some of the other issues: https://github.com/latchbio/latch/blob/7baa4791c2ac4998d8a27a48575eae04ef7b1b0d/latch/types/directory.py#L68C7-L69

Hi Nils! Thanks for the suggestion - I ended up having to modify what you had to get this to work, and since I can't push to your branch I just pushed it directly to main. Thanks for helping improve our API docs!


Censored Facebook videos

[x ] I'm reporting a site feature request
[ x] I've verified that I'm running youtube-dl version 2019.09.01
[ x] I've searched the bugtracker for similar site feature requests including closed ones

Facebook videos that are censored with "This video may show graphic or violent content." for which you need to click a "Uncover video" graphic/button.
When a valid name & password a response such as below is provided:
% youtube-dl --username "NAME" --password "PASSWORD" "https://www.facebook.com/syafaat.muzakki/videos/2145011278957932/"
WARNING: Assuming --restrict-filenames since file system encoding cannot encode all characters. Set the LC_ALL environment variable to fix this.
[facebook] Downloading login page
[facebook] Logging in
WARNING: unable to log in: bad username/password, or exceeded login rate limit (~3/min). Check credentials or wait.
[facebook]<PHONE_NUMBER>957932: Downloading webpage
[facebook]<PHONE_NUMBER>957932: Downloading webpage
ERROR: This video is only available for registered users. Use --username and --password or --netrc to provide account credentials.
When adding the "--cookies" option I receive the follow:
andrew@certhas:~/Videos % youtube-dl --username "NAME" --password "PASSWORD" --cookies ~/.config/APPLICATION/profiles/default/Cookies "https://www.facebook.com/syafaat.muzakki/videos/2145011278957932/"
WARNING: Assuming --restrict-filenames since file system encoding cannot encode all characters. Set the LC_ALL environment variable to fix this.
Traceback (most recent call last):
File "/usr/local/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/local/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/usr/local/bin/youtube-dl/main.py", line 19, in
File "/usr/local/bin/youtube-dl/youtube_dl/init.py", line 469, in main
File "/usr/local/bin/youtube-dl/youtube_dl/init.py", line 440, in _real_main
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 418, in init
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2318, in _setup_opener
File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 2748, in load
File "/usr/local/lib/python3.6/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0x8d in position 26: ordinal not in range(128)

https://github.com/ytdl-org/youtube-dl/issues/30681


Fix remaining go vet errors. Resolves #1858

Description

This fixes the remaining go vet errors as identified by running go vet ./... | grep -v '^vendor\/' | grep -v '^exit\ status\ 1'
Motivation and Context

Fixes #1858
How Has This Been Tested?

Ran make checks. No new tests as there is no new functionality.
Checklist:

[x] I have added a Signed-off-by
[x] Either no new documentation is required by this change, OR I added new documentation
[x] Either no new tests are required by this change, OR I added new tests
[x] I have run goimports, go vet, and golint. I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives.  The goal is clean, consistent, and readable code.

Signed-off-by: Sheehan Anderson<EMAIL_ADDRESS>

LGTM
Reviewed-by: Greg Haskins<EMAIL_ADDRESS>


This language is not supported

I tried the easiest example:
a hello world program
and then command+shift+i
result: this language is not supported.
vscode is in english.

Hi everyone!
I hope you're all doing well. Please take a moment to update to the latest version here: Visual Studio Marketplace.
Thank you!


Add hash of pages to avoid in full pass if already generated?

can keep body of transformed HTML (outside of layout) in cache.
or just check if nothing has changed and then just do nothing

done


Typeclasses cleanup and domain refactoring

Introduce again explicit subclasses for all concrete instances of type {1D,2D,3D}. This makes the implementation more straight-forward and reduces typeclass related clutter.
Along the way we had to refactor the domain concept, as the class hierarchy as it was did not allow for these changes.

@Ghazi-Bouabene  Thanks for the review. I implemented your proposals for improvements and added an additional unit test.


node 13.10.0+: Error: No "exports" main resolved

My Setup

Operating System: Gnu/Linux

App Version: 2.17.9

Installation type: git checkout, yarn start

[x] I have tested with the latest version

[x] I can simulate the issue easily

Description
When executing yarn start while using node 13.10.0 (or higher, including 14.3.0), the following error occurs:
src/main.js → app/main.js...
[!] (plugin babel) Error: No "exports" main resolved in<EMAIL_ADDRESS>src/main.js
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: No "exports" main resolved in<EMAIL_ADDRESS>    at applyExports (internal/modules/cjs/loader.js:524:9)
    at resolveExports (internal/modules/cjs/loader.js:541:12)
    at Function.Module._findPath (internal/modules/cjs/loader.js:661:22)
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:963:27)
    at Function.Module._load (internal/modules/cjs/loader.js:859:27)
    at Module.require (internal/modules/cjs/loader.js:1036:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Object.<anonymous> (/home/ole/projects/github/Rocket.Chat.Electron/node_modules/@babel/preset-env/lib/debug.js:8:33)
    at Module._compile (internal/modules/cjs/loader.js:1147:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1167:10)

With 13.9.0 it does not happen.
Current Behavior
The application does not start.
Expected Behavior
package.json names compatibility with node as "node": ">=10.16.x", so I expext node 13.10.0+ to work.

@neurolabs I fixed this issue by removing the node_modules directory and package-lock.json file then run yarn   to fresh install all packages then the issue was fixed.

@franckadil thanks, this worked for me as well.

@neurolabs you're welcome! I was also struggling with it.

Please test if the latest release did solve this. If not please reopen the issue or create a new one and tell us. I will be closing this one for now.
Thank you!


Download failure leaves invalid file

I just ran into an issue where Github failed to serve up a file properly, and that prevented all future runs of my cookbook from converging successfully.
The github_asset resource downloads directly into the destination file (/var/chef/cache/github_assets/org/repo/version/foo-version.tar.gz). If the download fails it leaves a zero-length archive behind with that name, and future cookbook runs don't re-download the file because the zero-length file exists. See https://github.com/reset/github-cookbook/blob/master/libraries/github_asset.rb#L94.
Suggested solution: download to a temporary file and rename it if the download completes successfully. I suggest downloading to a temporary name in the same directory so that the rename step doesn't also require a copy, if (for example) the /tmp directory is on a different volume than the /var/chef/cache/github_assets directory.

I put in a fix for this in #11
Example:
https://github.com/hightail/github-cookbook/blob/d61fcebc6636876bb51c54dfcfcc05fa0c60ccbf/libraries/github_asset.rb#L118


Clarification needed about new default style creation flows

@afabiani I'm trying to figure out where a call to the GS REST API can be put to set some custom metadata file (it's for the automatic configuration for the editing of default styles in visual mode).
It looks like different paths are followed depending if the we're doing an upload or an ingestion (and I hope we don't have further flows).
In broad terms, In the case of an upload I see that:

the upload module calls the resource managet set_style method
geoserver_create_style calls get_sld_for which in turn calls set_dataset_style and, finally, gs_catalog.create_style is called

On the other hand the ResourceManager.ingest method:

uses import_dataset which, after obtaining the import session, calls the set_styles method, which calls set_style and, in the end, esecutes gs_catalog.create_style

Can we harmonize the flows? Having two completely different sets of methods and flows is hard to reason and maintain
In this moment, if we want to have an additional step right after the creation of a new layer default style (not for user uploaded styles) where would you place it? I guess we will have to do it twice, once for flow.

FYI the request to be executed to set the custom metadata would be the following:
gs_catalog.http_request(
    f'{gs_catalog.service_url}/workspaces/{style.workspace}/styles/{style.name}.json',
    method='put',
    data=json.dumps({'style': {'metadata': {'msEditorType':'visual'}}}),
    headers={'Content-Type': 'application/json'}
)

@giohappy here is the full stack of method calls:
helpers.save_style(gs_style, layer):
 # This utility function:
 # 1. makes a copy of the 'gs_style' into the default workspace if it does not exist already
 # 2. creates the GeoNode 'Style' object and makes a copy of the 'gs_style' body into it
 - gs_catalog.create_style: if not gs_style.workspace
 - Style.objects.get_or_create

helpers.set_styles(layer, gs_catalog):
 # This utility function:
 # 1. set the default style after having made a copy of it into the workspace (see save_style)
 # 2. stores the alternative styles into the workspace and GeoNode if any (see save_style)
 # 3. creates the legend links on GeoNode
 -> save_style(default_style, layer): layer.default_style
 - alt_styles
 - create legend_url

helpers.set_dataset_style(saved_dataset, title, sld, base_file=None):
 # This utility function:
 # 1. creates a new style on GeoServer if no one matching exists
 # 2. updates the default style
 # 3. purges the old styles (the ones with no workspace)
 # 4. updates the legend links on GeoNode
 # 5. invokes the 'set_styles' function
 - gs_catalog.create_style: if no match on gs_catalog
 - updates default_style to the layer workspace
 - deletes the old styles
 - updates the legend_url
 -> set_styles(saved_dataset, gs_catalog)

helpers.get_sld_for:
 # This utility function:
 # 1. invokes the 'set_dataset_style' above on the dataset default style (if any)
 # 2. fetches the dataset geometry/type, creates a template sld and returns it (currently no one uses this sld later on)
 -> set_dataset_style: if default_style
 - fetch a style template accordingly to the gs_dataset geometry

helpers.sync_instance_with_geoserver:
 # Basically invokes the 'set_styles' utility
 -> set_styles(instance, gs_catalog)

geoserver.manager.create:
 # Basically invokes the 'sync_instance_with_geoserver' function
 -> sync_instance_with_geoserver

geoserver.manager.update:
 # Basically invokes the 'sync_instance_with_geoserver' function
 -> sync_instance_with_geoserver

geoserver.manager.import_dataset:
 # Basically invokes the 'set_styles' utility (only when creating a new dataset)
 -> set_styles(instance.get_real_instance(), gs_catalog): @create time only

geoserver.manager.set_style:
 -> geoserver_set_style: if sld_file provided
 -> geoserver_create_style: if no sld_file provided

geoserver.tasks.geoserver_set_style:
# Basically invokes the 'set_dataset_style'
 -> set_dataset_style: if sld_file provided

geoserver.tasks.geoserver_create_style:
 # This task seems to be a bit redundant
 # 1. if an 'sld_file' has been provided, it creates a new SLD on GeoServer and set it as 'default' one
 # 2. invokes the utility function 'set_styles' in order to store the Style on GeoNode and create the legend Link
 # 3. invokes the 'get_sld_for' in order to move the sld into the workspace if needed and finalize the alt styles setup
 - gs_catalog.create_style: if sld_file provided
 - gs_dataset.default_style = style: if sld_file provided
 - gs_catalog.save(gs_dataset)
 -> set_styles(instance, gs_catalog)
 -> get_sld_for

geoserver.tasks.geoserver_post_save_datasets:
 # Basically invokes the 'sync_instance_with_geoserver'
 -> sync_instance_with_geoserver

geoserver.management.commands.sync_geonode_datasets:
 # Basically invokes the 'sync_instance_with_geoserver'
 -> sync_instance_with_geoserver: updatebbox

geoserver.view.dataset_style_manage:
 # Basically invokes the utility function 'set_styles' in order to store the Style on GeoNode and create the legend Link
 -> set_styles(layer, cat)

https://docs.google.com/drawings/d/1SVkVJXfPyFSYmeNB3dcTW8mhlWRJFpK9VFLw9K-FaWY/edit?usp=sharing

Can we harmonize the flows?
We can simplify this set of methods, remove some redundant logic on the geoserver.tasks.geoserver_create_style method and maybe provide some sort of utility class that we can use to perform any kind of generic operation on the styles, centralizing the functions above with a more OO approach.
In this moment, if we want to have an additional step right after the creation of a new layer default style (not for user uploaded styles) where would you place it?
That should probably go into the helpers.set_styles function which seems to be invoked by all the other methods at a certain point.
Nevertheless, accordingly to the point 1, I would go for a class able to handle the styles in general and eventually update/modify them if needed.
We could use the same approach used by the resource_manager, a generic class which handles the Styles and Legends on GeoNode and a concrete class which handles the styles on GeoServer.


Getting errors on XHTML declaration

I have valid XHTML files and I get three warnings in the first two lines. They are:

It seems that the rest of the wornings work the same if I am in XHTML or HTML. It works well.
It would be good if I could not get these warnings. Right now, I just ignore the warnings.
Is there something in configuration that could resolve this issue? I have looked through the seetings, but do not find anything that applies.
FYI I am blind and using a screen reader, so I may be missing something. Also fairly new to VS Code and I am loving it.

Hi George! Thanks for reporting this! I think you had intended to include a code snippet or screenshot but there isn't one. What are the three warnings in the first two lines you mentioned? Once you let me know about that I'll be able to advise. Look forward to hearing back from you.

Here are the three errors followed by the first three lines of the XHTML file.
level 1  3 problems in file introduction.xhtml of folder C:\Users\kersc\OneDrive - DAISY Consortium\work\EPUB always up-to-date\Fundamental-Accessibility-Tests-Non-Visual-Reading-v2.0.0-unpacked\EPUB\xhtml  not selected  expanded  1 of 1
level 2  Warning: Special characters must be escaped : [ < ]. at line 1 and character 1.  1 of 3
Warning: Doctype must be declared first. at line 1 and character 1.  2 of 3  level 2
Warning: Special characters must be escaped : [ > ]. at line 1 and character 38.  3 of 3  level 2
Here are the first few lines of the file:

Avove are the first three lines of the file.
I also edit HTML and would not want to impact that functionality. Would:
In any case, if you are using XHTML then you will likely need the following in your .htmlhintrc config.
"doctype-first": false,
"doctype-html5": false,
Sould that impact my HTML editing?

That will not impact your HTML editing at all.
I use HTMLHint in many different kinds of projects and always have a custom .htmlhintrc config in the root of my project with rules for the type of project. It's best to disabled the rules which don't apply to you so they don't get in the way when editing in VS Code.

First, thank you for the help.
The doctype set to false removed one error. I still have two in the first line which is:

The warnings
Warning: Special characters must be escaped : [ < ]. at line 1 and character 1.
Warning: Special characters must be escaped : [ > ]. at line 1 and character 38
I have spent a lot of time going through documentation and cannot figure out what to do to make this warning go away.
I would think that this declaration would be a fairly common request.
Any suggestions welcome.

For that error you can use this in the HTMLHint config:
"spec-char-escape": false,

It seems this would allow < dangling anywhere. I don't think we want that. I am just trying to get the first line of an XHTML file to be without errors.
XML is the file format used in EPUB, which is being used by publishers all over the world. Publishers freek out when they see warnings in documents they are working on. I would love to see an elegant solution.

You haven't shared the code that was problematic with me yet. If you can do that then that would help me to suggest a solution.

Hi,
I am just looking for a rule or two that will allow the first line of an XHTML file not to throw errors. That is:

The rest of HTMLHINT rules work fine.

Ah, I realized what was happening.. the code you had pasted in your reply was hidden (invisible) in the GitHub comments UI so I didn't see the <?xml version="1.0" encoding="utf-8"?> you had previously pasted. Sorry about that - I just discovered it by editing your post to see the raw markdown code!
The code you provided in your first post was:
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE html>
<html xmlns="http://www.w3.org/1999/xhtml" xmlns:epub="http://www.idpf.org/2007/ops" lang="en">

However, that doesn't look right for XHTML. I think it should be:
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" lang="en">

The <!DOCTYPE html> is for HTML5, not XHTML.
I don't think the XML declaration (<?xml version="1.0" encoding="UTF-8"?>) is ever needed for XHTML... I suggest removing that.


feat: allow configuring hosted_zone_id

As requested in https://github.com/libdns/route53/pull/261

@aymanbagabas Here's the PR you requested. I pushed the go.mod update as a second commit since https://github.com/libdns/route53 doesn't have a release that contains the commit. Happy to wait to update to a tagged release, or use this, as you prefer.

@andrew-d Thank you for the PR. I've released<EMAIL_ADDRESS>


Add support for major ERC20 icons

@simibac - nice work on the bounties submissions. Were there any specific token icons that you'd want prioritized?

ETH, DAI, BTC would make sense to prioritise imo.

Great, thanks for the feedback!
On Tue, Oct 15, 2019 at 12:13 PM Simon Bachmann<EMAIL_ADDRESS>wrote:

ETH, DAI, BTC would make sense to prioritise imo.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ConsenSys/rimble-ui/issues/374?email_source=notifications&email_token=AAGCU2TB7MWPOE4WMATNFWLQOUYOLA5CNFSM4I6PS4Y2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBHINGY#issuecomment-542017179,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGCU2VVYZ32SZU74DL2WKLQOUYOLANCNFSM4I6PS4YQ
.


update readme, fix small grammar issues

just some small grammar changes :D

Hi @hisRoyalty, while the development team appreciates your interest in autoMod, it has been decided amongst us that we will no longer be accepting PRs focusing on the readme.md file, or spelling | grammar that has no impact on the program itself (ergo doesn't cause logic errors).
Sorry about this, @ThatsLiamS.


add topics

I suggest adding the topics aerodynamics, cfd, computational-fluid-dynamics in the About section.

Thanks for pointing that out. I had neglected to update those.


Add Plex request cache to backend

We should periodically cache results from Plex on the backend to reduce latency. The backend as a Plex proxy endpoint that we forward requests through.
Alternatively we could do this on the frontend with react-query and use the plex 'key' as part of the query key.

This exists as an experimental feature


Kustomization Templating

Each KustomizationRef can have substitute values:
. Values is a map[string]string
. ValuesFrom references ConfigMap/Secret instances whose Data sections contain values.
These key-value pairs can optionally leverage Go templates for further processing. During deployment, Sveltos iterates through the key-value pairs. If a key-value pair's value is a template, Sveltos evaluates the template using data available in the context (e.g., cluster information). This allows to dynamically construct values based on other resources or variables.
Finally, Sveltos uses the processed key-value pairs to replace placeholder values (identified using {{ .VariableName}} ) within the output generated by the Kustomize SDK.
Let's take an example.
Values:
Region:  {{ index .Cluster.metadata.labels "region" }},
Version: v1.2.0,
And then the output of Kustomize SDK can have a deployment
apiVersion: apps/v1
kind: Deployment
metadata:
  name: myapp-deployment
  namespace:  test
  labels:
    region: {{ default "west" .Region }}
spec:
  ...
  image: nginx:{{ .Version }}

What Sveltos will do in this case:

instantiate Values. So Sveltos will take the Cluster instance representing the cluster where Sveltos is about to deploy the Kustomize resources, it will take the Cluster's labels. if Cluster has labels: region: east  then instantiated Values will be:
Region:  east,
Version: v1.2.0,

Sveltos will take the Kustomize resources and will use the substitute Values, so:

apiVersion: apps/v1
kind: Deployment
metadata:
  name: myapp-deployment
  namespace:  test
  labels:
    region: east
spec:
  ...
  image: nginx:v1.2.0

Sveltos will deploy resources in the managed cluster

Fixes this

tested this with Flux.
Flux is syncing https://github.com/gianlucam76/kustomize
apiVersion: source.toolkit.fluxcd.io/v1
kind: GitRepository
metadata:
  name: flux-system
  namespace: flux-system
  ...
spec:
  interval: 1m0s
  ref:
    branch: main
  secretRef:
    name: flux-system
  timeout: 60s
  url: https://github.com/gianlucam76/kustomize.git

ClusterProfile is referencing this GitRepository and using Values:
apiVersion: config.projectsveltos.io/v1alpha1
kind: ClusterProfile
metadata:
  name: flux-system
  ...
spec:
  clusterSelector: env=fv
  kustomizationRefs:
  - deploymentType: Remote
    kind: GitRepository
    name: flux-system
    namespace: flux-system
    path: ./template/helloWorld/
    targetNamespace: eng
    values:
      Region: '{{ index .Cluster.metadata.labels "region" }}'
      Version: v1.2.0
  reloader: false
  stopMatchingBehavior: WithdrawPolicies
  syncMode: Continuous

Cluster instance has label: region: east
ConfigMap, Deployment and Service are all correctly deployed.
sveltosctl show addons
+-----------------------------+-----------------+-----------+----------------+---------+--------------------------------+----------------------------+
|           CLUSTER           |  RESOURCE TYPE  | NAMESPACE |      NAME      | VERSION |              TIME              |          PROFILES          |
+-----------------------------+-----------------+-----------+----------------+---------+--------------------------------+----------------------------+
| default/clusterapi-workload | apps:Deployment | eng       | the-deployment | N/A     | 2024-05-01 11:43:54 +0200 CEST | ClusterProfile/flux-system |
| default/clusterapi-workload | :Service        | eng       | the-service    | N/A     | 2024-05-01 11:43:54 +0200 CEST | ClusterProfile/flux-system |
| default/clusterapi-workload | :ConfigMap      | eng       | the-map        | N/A     | 2024-05-01 11:43:54 +0200 CEST | ClusterProfile/flux-system |
+-----------------------------+-----------------+-----------+----------------+---------+--------------------------------+----------------------------+

ConfigMap in the Kustomize repo was defined as
apiVersion: v1
kind: ConfigMap
metadata:
  name: the-map
  labels:
    region:  '{{ default "west" .Region }}'
data:
  altGreeting: "Good Morning!"
  enableRisky: "false"

has the label region set to east
apiVersion: v1
data:
  altGreeting: Good Morning!
  enableRisky: "false"
kind: ConfigMap
metadata:
  labels:
    app: hello-v1.2.0
    region: east
    ...
  name: the-map
  namespace: eng

and the deployment which in the kustomize repo had
  selector:
    matchLabels:
      deployment: hello-{{ .Version }}

has
  spec:
    selector:
      matchLabels:
        app: hello-v1.2.0
        deployment: hello-v1.2.0


Acknowledge multithreading capabilities of Scala Native

Adjusts the Scala Native variant of codebase to support multithreading capability introduced in Scala Native 0.5.
Tests have been adjusted to use async tests.
1 test was temporarily disabled for Scala Native variant - ConductorSuite - runaway threads will cause a test to be timed out - it was deadlocking becouse of a bug, would be fixed in a next release (0.5.7) - https://github.com/scala-native/scala-native/pull/4119

@cla-bot[bot] check


Affected by E0034 in from old crate dependency

Describe the bug
cargo building in some use cases, for example with the conjunction of the blsttc crate, of this crate causes rust error E0034 caused by a dependency.
To Reproduce
The code:

use near_sdk::borsh::{self, BorshDeserialize, BorshSerialize};
use near_sdk::near_bindgen;
use blsttc::{Signature, PublicKey};

#[near_bindgen]
#[derive(Default, BorshDeserialize, BorshSerialize)]
pub struct VerifySignature;
#[near_bindgen]
impl VerifySignature {
    #[payable]
    pub fn verify_sig(&mut self, message: Vec<u8>, signature: Vec<u8>, public_key: Vec<u8>) -> bool {
        let signature: [u8; 96] = signature.try_into().expect("invalid signature");
        let public_key: [u8; 48] = public_key.try_into().expect("invalid pk");
        let sig = Signature::from_bytes(signature).expect("couldn't read signature");
        let pk = PublicKey::from_bytes(public_key).expect("couldn't read pk");
        pk.verify(&sig, message)

    }
}

#[cfg(not(target_arch = "wasm32"))]
#[cfg(test)]
mod tests {
    use super::*;
    use near_sdk::test_utils::VMContextBuilder;
    use near_sdk::{testing_env, VMContext};

    use blsttc::{PublicKey, SecretKey};

    fn get_context(is_view: bool) -> VMContext {
        VMContextBuilder::new()
            .signer_account_id("bob_near".parse().unwrap())
            .is_view(is_view)
            .build()
    }

    #[test]
    fn verify_msg() {
        let context = get_context(false);
        testing_env!(context);
        let mut contract = VerifySignature::default();

        let sk = SecretKey::random();
        let pk = sk.public_key();
        let message = "hello".try_to_vec().unwrap();
        let sig = sk.sign(message.clone());

        println!("sk: {:?}", sk);
        println!("pk: {:?}", pk);
        println!("sig: {:?}", sig);

        println!("pk_bytes: {:?}", pk.as_bytes());
        println!("sig_bytes: {:?}", sig.as_bytes());
        println!("sig_bytes: {:?}", sig.as_bytes());

        let sig_bytes = sig.to_bytes().to_vec();
        let pk_bytes= pk.to_bytes().to_vec();
        let verified = contract.verify_sig(message, sig_bytes, pk_bytes);

        println!("verified: {:?}", verified);

    }

}```

Deps:
```toml
near-sdk = "4.0.0"
blsttc = {version = "7.1.0"}
getrandom = {version = "0.2.8", features = ["js"]}

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Versions
near-sdk: 4.0
blsttc: 7.1.0
Happens on linux and windows.
Additional context
Seems to be a known bug  that affects older versions of the bitvec crate. I think since both crates import different versions somewhere down the line the issue is exposed when compiling.
Ideally, both crates should update their deps, but I know it's in sub-dep hell for both. But the latest version of bitvec no longer has this bug.

Have you tried pinning the funty dependency as suggested in this comment?
I've seen that work for similar situations before.
Otherwise, you should just be able to disable the unit-testing feature that's enabled by default as a workaround and just write e2e tests with https://github.com/near/workspaces-rs instead of unit tests

Pinning, unfortunately, does not work, and even with default-features = false for near-sdk the same errors from the original post occur when trying to build.

Pinning, unfortunately, does not work, and even with default-features = false for near-sdk the same errors from the original post occur when trying to build.

ah, those deps were gated behind releases in 4.1.0-pre.0 and above versions, that's why that dep is still being pulled in.

@austinabell thanks, setting the version to 4.1.0-pre.0 does work with default-features = false. Hopefully, in the future, we can update those dependencies better.

Digging further it seems near-sdk depends on near-crypto version 0.13.0 but even the latest version of near-crypto uses primitives-types version 0.10.0 while upgrading this crate there would then update to a version of bitvec that no longer has this bug.
There's a similar issue in the bsttc crate of them not updating a single dependency and therefore having a buggy version of bitvec.

Yea, we can work around it for now with that feature flag, but would be ideal to see the dep updates eventually! So I think moving the issue is a good call :). Much appreciated!


[Snyk] Security upgrade io.streamnative:pulsar-broker from <IP_ADDRESS> to v2.8.0-rc-202106091140

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

568/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 3.5
Arbitrary Code Execution SNYK-JAVA-ORGECLIPSEJETTY-5903003
io.streamnative:pulsar-broker:  <IP_ADDRESS> -> v2.8.0-rc-202106091140
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

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


Add documentation for which references we're using for implementation

SOCKS4: https://www.openssh.com/txt/socks4.protocol
SOCKS4A: https://www.openssh.com/txt/socks4a.protocol
SOCKS5: https://www.ietf.org/rfc/rfc1928.txt

Covered in #2


Fix git commit-hook label and code snippet doc

Used bash instead of shell script because the web page didn't render it correctly.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2232   +/-   ##
=========================================
  Coverage     81.35%   81.35%
  Complexity     2067     2067
=========================================
  Files           341      341
  Lines          5968     5968
  Branches       1086     1086
=========================================
  Hits           4855     4855
  Misses          532      532
  Partials        581      581

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 43a0982...0f43c8e. Read the comment docs.