How to use sanitize?

I need sanitize a email a username field.
in email have<EMAIL_ADDRESS>and i want<EMAIL_ADDRESS>username, I have "  'asda'sd'  " and i want "asdasd".
I don't know how to use that function, Help me please
Sorry for my bad english, very thanks.

The list of sanitizers available can be found at chriso/validator.js.
For instance, there are multiple ways to achieve what you're looking for. One would be to utilise the blacklist sanitizer:
req.sanitize('email')
    .blacklist('{}')    // this removes any '{' or '}' characters found in 'email'
    .normalizeEmail({lowercase: true}) // optionally, you can use this to normalize the email format (read the docs)

Same goes for username.

Now I have this
req.sanitize('email').whitelist(['abcdefghijklmnñopqrstuvwxyz','@']);
and works correctly, but there any way to simplify this using whitelist method?(i mean abcdefghijklmnñopqrstuvwxyz)

@jopcode, can you check whether normalizeEmail suffices, as suggested above by @IOAyman?

Closing due to lack of response.


Switch SecretBinding controller to controller-runtime

How to categorize this PR?

/area dev-productivity scalability
/kind enhancement
What this PR does / why we need it:
Refactor the SecretBinding controller to controller-runtime.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/4251
Special notes for your reviewer:
Generally, we want to follow this cookbook while refactoring existing controllers:

Add documentation
Add integration test based on envtest
Switch controller to controller-runtime

Release note:

NONE

/lgtm

/skip


Parametric: add tests for headers precedence with conflicting context

Description
When a tracer receives headers for multiple propagation styles that is supports, it should use the precedence of the styles to select the primary context, then if other contexts are extracted but do not match, the tracer should generate the  the proper span links and add them to the span.
Motivation

Workflow

⚠⚠ Create your PR as draft
Follow the style guidelines of this project (See how to easily lint the code)
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Once your PR is reviewed, you can merge it! :heart:
Reviewer checklist

[ ] Check what scenarios are modified. If needed, add the relevant label (run-parametric-scenario, run-profiling-scenario...). If this PR modifies any system-tests internal, then add the run-all-scenarios label (more info).
[ ] CI is green

[ ] If not, failing jobs are not related to this change (and you are 100% sure about this statement)

[ ] if any of build-some-image label is present

is the image labl have been updated ?
just before merging, locally build and push the image to hub.docker.com

[ ] if a scenario is added (or removed), add (or remove) it in system-test-dasboard nightly

I don't think all of the other tracers should be failing on this by not having span_links on the span. Is there something enabled for python/python_http that hits a newer testagent version  or endpoint?

This feature should be functional on all tracers...

@robertomonteromiguel I'm having trouble seeing the error that's occurring in PHP, now that I'm skipping the test for the other libraries, and how it's related to the changes I've made. Any insight?


Obj-c

Is it compatible with obj-c ?

Yes, it should be compatible with Objective-C. Write use_frameworks! in podfile.
Also maybe this answer would help you if The “Swift Language Version” (SWIFT_VERSION) build setting must be set to a supported value for targets which use Swift. This setting can be set in the build settings editor. would appear.


Csv export

Does anyone know how to add the average ownership to the cvs export? I see in the code, Budget and Fppg. If I add Ownership would it automatically add it too?

yes

yes

I tried that but maybe I was in the wrong spot. What file is the actual code under

it works for me if ive edited my csv to have #s in the ownership columns and
including this line
optimizer.add_players_group(PlayersGroup( players=[player for player in optimizer.players if player.projected_ownership <= 10],min_from_group=1))

He wanted it on the export csv  not in the import  larry
On Thu, Jun 23, 2022, 7:36 AM lightninglarry @.***>
wrote:

it works for me if ive edited my csv to have #s in the ownership columns
and
including this line
optimizer.add_players_group(PlayersGroup( players=[player for player in
optimizer.players if player.projected_ownership <= 10],min_from_group=1))
—
Reply to this email directly, view it on GitHub
https://github.com/DimaKudosh/pydfs-lineup-optimizer/issues/368#issuecomment-1164299969,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANBWO7RGDIICUHI4UYK5QUDVQRD4VANCNFSM5ZAJNZRQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

That's actually pretty easy


Add support for snake case naming convention

@ejsmith You've got some failing tests in this against the JsonLocatorField. Did something get missed?

Looks like it. I will get it updated.

Looks like there may be some flakey tests? These tests pass locally for me.


Compile errors on windows 11 / vs2022

Hello, I was trying to use your stopwatch; but there were some compiler errors. Code maid / MSVC++ found a couple includes that were transitively included. I was trying to figure it out; but for my purposes, it was a little too much anyway. I just needed some help deciphering the std::chrono namespace... so, I have a working timer that I was able to build in a few lines of code...
Anyway, I liked your implementation; and look forward to seeing it compile :)

I just tried building and running the tests on Windows 11 / VS 2022 and it worked just fine. Is your project C++17? It's possible it might have some problems on a different language version.
I did find that my test project had some random header files referenced that have nothing to do with this project, but that should not affect the functionality or anything.


DS: vkQueueSubmit checks Event state too early when vkSetEvent is used

I originally submitted this as part of #378, but this part of the problem wasn't fixed yet, so I'm opening a new issue.
Consider the following situation:
// recording a CB
vkCmdWaitEvents(cb, 1u, &event, VK_PIPELINE_STAGE_HOST_BIT, VK_PIPELINE_STAGE_TOP_OF_PIPE_BIT, 0u, NULL, 0u, NULL, 0u, NULL);

// some time later
vkQueueSubmit(queue, 1u, &submitInfo, NULL);

vkSetEvent(device, event);

Inside a command buffer I'm waiting for an event that is signaled on the host. Note that source flags are specified correctly, i.e. have VK_PIPELINE_STAGE_HOST_BIT. Still, I'm getting this error:

DS(ERROR): object: 0 type: 0 location: 8532 msgCode: 17: Submitting cmdbuffer with call to VkCmdWaitEvents using srcStageMask 0x4000 which must be the bitwise OR of the stageMask parameters used in calls to vkCmdSetEvent and VK_PIPELINE_STAGE_HOST_BIT if used with vkSetEvent.

If I use vkSetEvent before vkQueueSubmit, all is good and no errors appear. If I use wrong flags, I'm getting an error as expected. So I think internal state that is being updated in vkSetEvent is fine, but the checks are done upfront in vkQueueSubmit before it happens.
Also, if vkCmdWaitEvents is part of a secondary CB executing indirectly through a primary CB, then srcStageMask is not checked at all.

Should be fixed. Let me know if it continues to be an issue.

Hi, no I don't think this is working yet. I'm observing two problems:
[1] If vkQueueSubmit comes before vkSetEvent (in API call order), then stageMask observed by validateEventStageMask in core_validation.cpp is 0.
If this order is reversed, then a correct mask will be used.
E.g.:
vkBeginCommandBuffer()
vkCmdWaitEvents()
vkEndCommandBuffer()

vkQueueSubmit()       // error here

vkSetEvent()

But this will work fine:
vkBeginCommandBuffer()
vkCmdWaitEvents()
vkEndCommandBuffer()

vkSetEvent()

vkQueueSubmit()       // OK

[2] The condition to check flags seems too strict to me. It expects an exact mask, rather than presence of some bits:
if (sourceStageMask != stageMask && sourceStageMask != (stageMask | VK_PIPELINE_STAGE_HOST_BIT)) {

If we wait for two events with different flags, then this can't work by definition. E.g. imagine srcStageMask = VK_PIPELINE_STAGE_HOST_BIT | VK_PIPELINE_STAGE_VERTEX_SHADER_BIT and signal from vkSetEvent. I don't think this use is prohibited by the spec, so it shouldn't be an error.

For [1], we have a test case called WaitEventThenSet which does what you are describing. If your case still fails can you please describe what the differences between that case and yours are or provide a repro case? For [2] it expects an exact mask where the mask is the bitwise or of each event that is being waited on which should be correct according the the spec. This means that in your case both flags would be logged in stageMask. Also, the spec says:
"srcStageMask must be the bitwise OR of the stageMask parameter used in previous calls to vkCmdSetEvent with any of the members of pEvents"
which doesn't seem to allow for a only a subset being checked.

Hi, I've reconsidered this definition and I think you're right. Given this the problematic test used the stage masks incorrectly.
I've corrected the arguments and I'm no longer getting any errors. The original issue you speak in [1] is fixed. Thanks!


WIP - virtualized lists

Closes #95
Still in draft mode, need feedback

Primary issues I'm facing in this PR right now:

scrolling is buggy
List doesn't expand to the full height of the screen

List doesn't expand to the full height of the screen

Need to wrap the whole thing and flex it. Like here.

scrolling is buggy

Scrolling is always a bit like that with windowing. Probably just need to tweak the overscan options.

Awesome so the flex wrapping worked for problem #1.
Scrolling is still a little iffy though. I upped the overscan rows to 100 before I was able to make it feel seamless. But 100 rows is way too much and on top of that, scrolling backward yields jagged/weird results. I can't help but think there's something else missing here.

I'm going to wait for #153 to be merged in and then pick this up.


Links on page are missing

Touch - https://docs.microsoft.com/en-us/windows/uwp/design/input/keyboard-interactions#touch-keyboard
Focus visuals - https://docs.microsoft.com/en-us/windows/uwp/design/input/keyboard-interactions#focus-visual
UI Navigations - https://docs.microsoft.com/en-us/windows/uwp/design/input/keyboard-interactions#navigation

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

ID: 7538142a-74f2-63fe-5ffc-b205a69f9493
Version Independent ID: c382db31-6f36-b071-423a-72b61b77aed1
Content: Keyboard interactions - UWP app developer
Content Source: windows-apps-src/design/input/keyboard-interactions.md
Product: windows
GitHub Login: @Karl-Bridge-Microsoft
Microsoft Alias: kbridge

Hi @vadobre. Thanks for the feedback. These links should be fixed now.


Error: no access key set for instance

Hi,
I am trying to login to the instances under a tmux session which is already having the keys loaded.
awless ssh username@instance-id
Error: no access key set for instance-id
Gives me the following error. Do you know wht do i need to configure to set the location to the private key?
Regards,
Kevin

If you list your instances with awless list instances, you should see that the "ACCESS KEY" property for your instance is empty. That's why awless ssh do not automatically detect the key it should use (which must be stored in the ~/.awless/keys folder).
For now, you cannot specify a specific key to use to connect via awless ssh, but there is an opened issue, #54, that aims at doing that.

Hi.
It still doesnt work. I have the key named test-thor.pem under ~/.awless/keys folder and still says
Error: no access key set for instance
Regards,
Kevin

When you list instances with awless list instances. Do you have something in the ACCESS KEY column, for the instance you try to SSH in ?

Nothing.

OK. So this is a normal behavior.  For now, awless do not test all keys in ~/.awless/keys as a classic SSH client does. It only takes the key referenced in the ACCESS KEY property, if it exists.

Ok. So how do i fix this? Also i have few instances with ACCESS KEY property set and it gives me another error Error: ssh: cannot decode encrypted private keys. I have the private keys loaded with ssh-agent

Are these access keys either RSA, DSA or ECDSA keys ?


Servo motor functions

Hello,
I would like to suggest adding a way to control servo motors with this library.
Usually, with an Arduino, one would simply use the "Servo.h" library to control a servo motor. However, when using this library, one must recreate the whole PWM functions since a servo motor is expecting a 40Hz frequency which isn't really obtainable by using analogWrite commands.
I beleive it could help greatly to have such a function :) (if there already is one and I simply couldn't find it, I'm sorry and could you please point me in the right direction?)

Hi,
I also plan to use servo.
I hope that ToneRequest or something similar will do the trick.
Sounds like pwn command...
Please let me know, if there is better way for low level servo handling for now.
Thanks,
Dima
var driver = new ArduinoDriver(ArduinoModel.UnoR3, true);
driver.Send(new ToneRequest(8, 200, 1000));

Found much better solution :)
Connecting full function secondary arduino allows any usage.
First arduino connected by USB to pc provides control signal and secondary handle complex hardware with all libraries!
This is more flexible, as one cannot duplicate all the arduino libraries in c#.
Hope it helps

i'm to trying to do the same think. I don't know how i can make servo control through arduinodriver.

i'm trying to do the same think. I don't know how i can make servo control through arduinodriver.

have you find a way to control the servo motor?


lsusb is not showing any output

I am trying to read usb in ubuntu 18.04 bionic through WSL.
I am using $ lsusb but it is not showing any output.

Kind of related to #825 #2195 #1077

/dupe #2195


SPFx Application Customizer is causing redirect to spfxsinglesignon.aspx on sub webs

Category

[x] Question
[ ] Typo
[ ] Additional article idea

Question
Is this the expected behaviour? If so, how to overcome this issue?
Environment details
Windows 10
SharePoint Online
Node 10.23.1
Yo 3.1.1
TypeScript 4.1.2
VS Code
We have a simple SPFx Application Customizer which renders a global navigation to all sites. We are using pnp to fetch items from a config list. The component is running fine. Each time we navigate to a sub-web (new browser-session or adding a tab to the browser), the page is loading twice. We can see that the title "SPFx Single Sign On Redirect Page" is rendered on the tab when the page loads the second time.

From the network tab, I found some entries with requests and referrer to /_forms/spfxsinglesignon.aspx
{        "name": "referer",        "value": "https://xxx.sharepoint.com/_forms/spfxsinglesignon.aspx"  }
Please note, that this issue only occurs on sub-webs, no matter if this is a team site or communication site.
Additional findings: This does not occur, if we navigate from websitecontent to the sub-web (which also is adding a new tab to the browser)

Thanks @EA12 - Not quite sure if I got this 100%, so let's confirm.

You have app customers build with SPFx (v1.11?) which is deployed to all sites in SharePoint Online using tenant wide deployment option
App customizer renders properly in root site of site collections in modern pages - like https://contoso.sharepoint.com/sites/fuufaa
When you move to a sub site, page is rendered twice and you see references to spfxsinglesinon.aspx in the network - URL in the browser for example https://contoso.sharepoint.com/sites/fuufaa/sub
This does not happen for sub sites if you do not have the app catalog deployed or?

Is that accurate?

Hi @VesaJuvonen,
yes, with the latest SPfx, using tenant wide deployment option.
Root site rendering is OK. Rendering on pages of root site is also OK.
This happens to sub-webs only, if we start a new browser-session or adding a tab to the browser and pasting the url to a sub-web.
We are just using the global app catalog on this tenant, no site app catalog is activated.
Now clearer?

Thanks @EA12
good information but we can't seem to repro with the following setup. Is this exactly what you are doing?

Deploy tenant wide deployed app extention
Include tenant scoped activation of app extension
Ensure that is rendered properly in root site - https://contoso.sharepoint.com/sites/Comms - works
Create modern sub site and confirm status - https://contoso.sharepoint.com/sites/Comms/modern

So - not quite sure what's going on. Are you embedding this to Teams or doing something else on the model?

Thanks @VesaJuvonen ,
yes, that is how we do it.
What we do:
From the OnInit in the applicationcustomizer.ts, we determine the culture:
this.culture = this.context.pageContext.cultureInfo.currentUICultureName.toUpperCase().substr(0, 2);
To render the top placeholder, we provide the context as prop to our component.
if (this._topPlaceholder.domElement) {         const element: React.ReactElement<{}> = React.createElement(           GlobalNav,           {             context:this.context,             lc: this.culture,             filterFieldName: this.FilterFieldName           }         );         ReactDom.render(element, this._topPlaceholder.domElement);       }
The component itself instantiate a provider in componentWillMount():
public componentWillMount(): void {     this.globalNavProvider = new GlobalNavProvider(this.props.context, this.props.lc, this.props.filterFieldName);   }
The provider will setup pnp and assigns locale variables for the context and so on:
constructor(context: any, lc: string, filterFieldName: string) {     this.spfxContext = context;     sp.setup({       sp: {         headers: {           Accept: "application/json;odata=verbose"         },         baseUrl: ${window.location.protocol}//${window.location.hostname}               }     });     this.culture = lc;      this.FilterFieldName = filterFieldName; }
The component is calling the provider to get the data:
public componentDidMount(): void {     this.globalNavProvider.isSiteIsPrivate().then((res: string): void => {       if(res == "Private") {         this.setState({           doNotRender: true         });       }       else {         this.globalNavProvider         .getGlobalNavigation()         .then(           (result: IGlobalNavItem[]): void => {             this.setState({               globalNavItems: result             });           }         ) 	  } 	}); }
First, the provider will identify, if the current site is a private group, because, we will not render anything in a private group
`public async isSiteIsPrivate(): Promise {
return new Promise((resolve, reject) => {
const web = Web(this.spfxContext.pageContext.web.absoluteUrl);
  web.select("AllProperties").expand("AllProperties").get().then(w => {
    const jsonString = JSON.stringify(w);
    const pop = JSON.parse(jsonString).AllProperties.GroupType;
    resolve(pop);
  });
});

}`
If the method isSiteIsPrivate() does not return "Private", we fetch the items to build the global navigation
`public async getGlobalNavigation(): Promise<IGlobalNavItem[]> {
var self = this;
const results = await sp.web.lists
  .getByTitle(ListNames.GlobalNavigation)
  .items.select("*,"+FieldNames.ZGNParentTitle).top(2000)
  .expand(FieldNames.ZGNParent)
  .orderBy(FieldNames.ZGNOrder)
  .orderBy(self.FilterFieldName)
  .get();
return this.parseGlobalNavigationNodes(results);

}`
The rest is just building the UI with React, based on the resultset.
So, nothing special here.
Any Idea what is causing the reload?
Please, open a new Browser-Tab and paste the url to a sub-web to the address-bar, then you will see the page loading twice.
Sorry, but the 'Insert code' toolbar action does not always work.

Nobody?

I have same problem, but without ApplicationCustomizer, just some custom webpart in page.
Recently, I have this constant reload on admin SP page (the last load finish on spfxsinglesignon.aspx)
I have the problem on several tenants.

I have same problem that occurs on iPhones and iPads but not for every user. We have extension and web parts in the solution.

The problem disappeared by itself!

@EA12 - nothing stands out from your description that should cause spfxSingleSignOn to be called.  Can you capture a network trace and see what is triggering the redirect to it?  Maybe pnp.js is doing something atypical here?  You might also try asking in their issues list - https://github.com/pnp/pnpjs/issues
@PooLP - the admin pages are outside the scope of this issue list (as it is not extensible).

The issue concerns also the spfxsinglesignon.aspx page, I found it relevant to also report this problem. Sorry, I did not want to destabilize you.


Basic documentation page to merge

work in progress at this moment in time, trying to implement a clickable tile

Approved


fix the archiver.rs: delete println

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Files with Coverage Reduction
New Missed Lines
%

src/archiver/zip.rs
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
24

Relevant Lines:
229

💛  - Coveralls


gauge custom scale not applying style

I am using a DAQ gauge as following
daq.Gauge(
            color={"gradient": True, "ranges": {"green": [0, 5], "yellow": [5, 7], "red": [7, 9], "black": [9, 10]}},
            value=2,
            max=10,
            min=0,
            scale={
                "custom": {
                    0.14: {"style": {"color": "red"}, "label": "Hello"},
                    4.14: {"style": {"color": "red"}, "label": "Foo"},
                    4.95: {"style": {"color": "red"}, "label": "Baz"},
                    8.12: {"style": {"color": "red"}, "label": "Jeez
bom"},
                }
            },
        ),

and I have the issues:

style instruction is in the dom but has no effect (ie, text is not red)
<text x="118.58641410998341" y="64.01665418691996" text-anchor="middle" style="color: red;">Baz</text>
the  is escaped

If style did work, would it be possible to have the text of the scale outside the gauge instead of inside ?

I ran into this issue too and just came across this post.  After some playing around, I figured out you need "fill" instead of "color" in your custom scale styles.

as mentioned by @jdyerrhg, this will be your solution. I have modified some colors
Output

Code
daq.Gauge(
            color={"gradient": True, "ranges": {"green": [0, 5], "yellow": [5, 7], "red": [7, 9], "black": [9, 10]}},
            value=2,
            max=10,
            min=0,
            scale={
                "custom": {
                    0.14: {"style": {"fill": "red"}, "label": "Hello"},
                    4.14: {"style": {"fill": "turquoise"}, "label": "Foo"},
                    4.95: {"style": {"fill": "blue"}, "label": "Baz"},
                    8.12: {"style": {"fill": "red"}, "label": "Jeez
bom"},
                }
            },
        ),


Await stable value function

Hi!
One feature that I would like to have in the UVVM utility library is a modified version of await_stable where I can specify which value I want to wait for.
What I want to do is to check that a signal has a given value, and that the signal keeps this value over a time period.
Today I often write this over several lines of code:
check_value(X, '0', "Check that X is zero")
timestamp = now;
wait for 1 ms;
check_value(X, '0', "Check that X is still zero")
check_stable(X, (now-timestamp), "Check that X was stable over the given time")

It would be nice to instead have something like:
await_stable_value(X, '0', 0 ns, 1 ms, "Checking that X is zero and remains zero for 1 ms");
Even better, it would be a good idea to have a non-blocking version that can run in the background:
await_stable_value(X, '0', 0 ns, 1 ms, NON_BLOCKING, "Checking that X is zero and remains zero for 1 ms while I do something else");
Thanks and best regards,
Andre

Hi Andre,
Could you create a pull-request for this feature?
Br,
UVVM Team

Hi Andre,

and thanks for the pull request :-)
I understand your point about the name, but 'await_*' is used to indicate that the procedure is time consuming and that you (potenitally) have to wait for it to finish, so I think the name should start with 'await_', or otherwise we break the "rule".

I have updated the pull request and replaced the name check_stable with await_stable_value.


[LLC] Arbitrary Client Options Value

Describe the issue or request
There is no way today to create an arbitrary client options pointing to a version we don't know about.
Example:
https://github.com/Azure/autorest.csharp/blob/feature/v3/samples/Azure.AI.DocumentTranslation/Generated/AzureAIDocumentTranslationClientOptions.cs#L28
Describe your ideas for solutions
Make Version string virtual so a client could create a derived class and return a custom arbitrary string.

Way for ppl to say "I'm using version of API you don't know about" in ServiceVersion enum.
We expect it's a one-line change to make it virtual. => bring estimate down.

Virtual property won't prevent a constructor from throwing an exception.
However, since type is partial, user can simply add a new constructor, so I'm not sure if we need to do anything at all:
    public partial class AzureAIDocumentTranslationClientOptions : ClientOptions
    {
        /// <summary> Initializes new instance of AzureAIDocumentTranslationClientOptions. </summary>
        public AzureAIDocumentTranslationClientOptions(string version)
        {
            Version = version;
        }
    }

Since a user picking an API version that the client doesn't know about is a pretty off the beaten path scenario and I worry about adding such an easy way to put you into an invalid state.
Today we generate code that looks like this for ClientOptions for an LLC:
namespace Azure.Analytics.Purview.Account
{
    /// <summary> Client options for PurviewAccountClient. </summary>
    public partial class PurviewAccountClientOptions : ClientOptions
    {
        private const ServiceVersion LatestVersion = ServiceVersion.V2019_11_01_preview;

        /// <summary> The version of the service to use. </summary>
        public enum ServiceVersion
        {
            /// <summary> Service version "2019-11-01-preview". </summary>
            V2019_11_01_preview = 1,
        }

        internal string Version { get; }

        /// <summary> Initializes new instance of PurviewAccountClientOptions. </summary>
        public PurviewAccountClientOptions(ServiceVersion version = LatestVersion)
        {
            Version = version switch
            {
                ServiceVersion.V2019_11_01_preview => "2019-11-01-preview",
                _ => throw new NotSupportedException()
            };
        }
    }
}

If we make the Version property public and virtual (while it remaining read-only):
namespace Azure.Analytics.Purview.Account
{
    /// <summary> Client options for PurviewAccountClient. </summary>
    public partial class PurviewAccountClientOptions : ClientOptions
    {
        private const ServiceVersion LatestVersion = ServiceVersion.V2019_11_01_preview;

        /// <summary> The version of the service to use. </summary>
        public enum ServiceVersion
        {
            /// <summary> Service version "2019-11-01-preview". </summary>
            V2019_11_01_preview = 1,
        }

        public virtual string Version { get; }

        protected PurviewAccountClientOptions() { }

        /// <summary> Initializes new instance of PurviewAccountClientOptions. </summary>
        public PurviewAccountClientOptions(ServiceVersion version = LatestVersion)
        {
            Version = version switch
            {
                ServiceVersion.V2019_11_01_preview => "2019-11-01-preview",
                _ => throw new NotSupportedException()
            };
        }
    }
}

Then a user could subclass this ClientOptions type and override Version to return whatever arbitrary string they wanted to return, but it would be far less likely that you naturally fall into a pit of failure.
I think this is the pattern that @KrzysztofCwalina was imagining.

I worry about adding such an easy way to put you into an invalid state.

@ellismg , isn't @AlexanderSher saying that a customer can add this constructor without subclassing the <Service>ClientOptions type?  That does feel like a lighter-weight solution than requiring the subclass.  I don't think @AlexanderSher's approach leads the customer toward the pit of failure, because it's not obvious and it's already possible.  Or are you saying that if the Azure SDK API developer does this this way, the SDK customer could easily throw it into the wrong state?  I see that.  (I sometimes get confused who we're talking about when we say customer because there are so many levels of this.)
Ok, I have convinced myself that @ellismg's is the better approach for the SDK customer (not the SDK author).  Is there any way to prevent the addition of the constructor, or is that something left to API reviews?

Postpone this to RC1.1.


Breaking change in splat 7.3.6

just an FYI, MemoizingMRUCache  now has an extra required runtime parameter https://github.com/reactiveui/splat/commit/161eeda1e321d038cf1e6b814b8bb7fcf9971e0a

which means your consumers will get this exception at runtime
System.MissingMethodException
Method not found: 'Void Splat.MemoizingMRUCache`2..ctor(System.Func`3<!0,System.Object,!1>, Int32, System.Action`1<!1>)'.
   at ReactiveUI.ReactiveNotifyPropertyChangedMixin..cctor()

updating to the newest splat should fix this. let me know if u want a PR

We use the LibraryRange feature for NuGet so should automatically fix itself when we go to next version on NuGet. I am adding those constructors back though for other packages that rely on Splat.

https://developercommunity.visualstudio.com/content/problem/606218/error-xa5300-the-java-sdk-directory-could-not-be-f.html

@glennawatson thanks, and no rush. i only noticed due to a dependabot auto update

Hi, Is there a workaround for this in the meantime?  The latest release version of splat is 7.3.6, @SimonCropp did you mean use the pre-release version of splat 8.0.0-dotnet3-preview.8 ?

Just explicitly use the old version for the moment.

I would of released a fix by now but azure DevOps broke today unfortunately

Fixed by https://github.com/reactiveui/splat/pull/345

@glennawatson thanks :)


May I create a GUI client?

I greatly appreciate your work, fabulous job @nathan-alden! The new terminal in windows10 is also awesome, but I think there are some drawbacks in using it connected with such useful application (oh yeah, I do not imagine live without CLI), I think it will be nice to create a simple, eye-catching GUI based on WPF/Universal Windows framework (C#). I have some experience in developing C# apps and I'd like to help you promote this project instead of erm.. the root, which is... well, bad managed.

+1 for this. IMHO, good robust tools should have two things: A simple GUI and a robust CLI.

Are you asking to contribute to this repository? If so, the I have only one request: the actual logic that does the work must be in a single place (DRY). I'd prefer if the PowerShell script retains the logic with the GUI simply making calls to the script. Do you think that's doable?

Sure it is able to do, but unfortunately the logic of powershell will be needed to be divided in small parts (function/procedures) - just like additional parameters passed inline during cmd session. During my .psl file investigation it is almost done. You can pass parameter to only disable or delete or even enable. So yes - consider it done...

Thanks for your interest, piqus. :+1: I added you as a collaborator. Also, please take a look at the new README and folder structure. I made some significant changes to their organization to better allow you to get started.

Yay! I really appreciate that, thank you! I am goting to get into developing things asap.


@[example] macro within  tag is rendered outside of

Describe the bug
@[example] macro within <details> tag is rendered outside of <details> section, showing up even if the <details> section is collapsed.
To Reproduce
Multi-Qubit Measurements kata, exercise Multi-Qubit Measurement Outcome Probabilities II, Code-Based Solution - the code included in it is rendered outside the <details>. You can see it both in the playground and in the website.
Expected behavior
Exercise rendered within <details> tag and collapsed together with it, showing the code only when expanded.

After an offline discussion we concluded that this is caused by the fact that all macros are treated as top-level entries, and they split the code between them into separate HTML sections, so the <\details> tag ends up in a separate section from the opening tag. This is challenging to fix, and this is used only this once (I can't think of another occurrence of this scenario in the upcoming katas either), so we agreed to do a workaround by modifying the text to discuss the code outside of the <details> tag: https://github.com/microsoft/qsharp/pull/1631.


Question about S* compartment in SIRF model

Summary of question
For SIRF model, what exactly is the S* compartment?
If I understand correctly is this the people who are actually carriers of the disease without anyone (including themselves) knowing about it, who either die and they are confirmed positive after death, while some others are moved to infected after being confirmed? Essentially you need to separate the two death situations and insert a probability factor of {a, 1-a}, right?

S* is not an actual variable. S* means this case was confirmed with COVID-19, not considering died or not.
S* cannot exist in the real world, but I introduced this variable to create an ODE model. This is because, in this dataset, we know the number of cases who were confirmed with COVID-19, but we do not know the number of died cases who died without COVID-19.

Your understanding appears correct.

Is it necessary to documemnt this in https://lisphilar.github.io/covid19-sir/usage_theoretical.html and the kaggle Notebook?

I think yes it would be helpful for clarification.

I intend to add the following comments in SIR-F description. Please review it.
S* means this case was confirmed with COVID-19, not considering died or not.
S* cannot exist in the real world, but S* was introduced to create an SIR-F model. This is because, in JHU-style dataset, we know the number of cases who were confirmed with COVID-19, but we do not know the number of died cases who died without COVID-19.

For S* please add as well my clarification, "... describes the people who are actually carriers of the disease without anyone (including themselves) knowing about it, who either die and they are confirmed positive after death, while some others are moved to infected after being confirmed. Essentially it serves as an auxiliary compartment to separate the two death situations and insert a probability factor of {a, 1-a}."
Also add a tau interpretation/definition.

Thank you. I will add the following sentences to my Kaggle Notebook and https://lisphilar.github.io/covid19-sir/usage_theoretical.html#SIR-F-model

\mathrm{S}^\ast describes the cases who are actually carriers of the disease without anyone (including themselves) knowing about it, who either die and they are confirmed positive after death, while some others are moved to infected after being confirmed.
In JHU-style dataset, we know the number of cases who were confirmed with COVID-19, but we do not know the number of died cases who died without COVID-19.
Essentially \mathrm{S}^\ast serves as an auxiliary compartment in SIR-F model to separate the two death situations and insert a probability factor of {$\alpha$, $1 - \alpha$}.

Tau interpretation/definition is discussed in #392.

With #403, documentation was updated. This will be uploaded to the GitHub site in several hours.
I also updated my Kaggle Notebook.

Should TERM.md be also updated with S* definition as well?

Because S* is related to specific model (SIR-F model), it is expected to use https://lisphilar.github.io/covid19-sir/usage_theoretical.html and the Kaggle Notebook rather than the term list.
The update with #413 was deployed, but I found syntax error in the markdown file. This will be solved with #415 and deployed in several hours.


ci(levm): integration CI

This work is currently stopped because it is not a priority atm
Motivation

Check in CI: Running L2 and sending a transaction successfully

Description

Closes #issue_number

Closing this as we've agreed on having only the EF tests running.


fix(#3896): Fix dependency inspector supporting property placeholders

Do not raise errors during dependency inspection when using property placeholders in toURI. Fixes http-sink Kamelet
Fixes #3896

FYI @tadayosi

Thanks a lot Christoph for taking care of this issue!
As already commented in the review, I don't think checking only : in uri can cover all edge cases. For example:
https://github.com/apache/camel-kamelets/blob/main/kamelets/elasticsearch-search-source.kamelet.yaml#L110
            uri: "{{local-es}}:{{clusterName}}"

This kamelet has uri that contains : but still uses a property placeholder. A better logic should be:

Try to decode the uri.
Check the returned scheme; if it's nil, check the uri on whether a placeholder is used at the place of scheme.
If a placeholder is used return silently; otherwise throw an error.

Thinking a bit further, what should we really do with the use of Kamelet placeholders in terms of the dependency resolution?  The original intention why we've added a stricter component dependency validation is to improve UX in case of wrong components used in the user's integration, so that Camel K can give users early feedback on possible misconfigurations in user side.
Since it is related only to Kamelets and Kamelets already have an explicitly defined set of dependencies, so dependency resolution is not a problem in this case, right?  Is it just fine to skip it silently and everything should work fine?

I wonder if throwing an error at this state when the uri does not resolve to a component in the Catalog is the right thing to do in general. Using placeholders in the uri may be only one scenario where throwing an error because of this is inappropriate.
What about uri that for some other reason does not resolve to a component that we know. We also support jitpack dependencies and custom Maven repositories that might ship libraries with custom components that are not known to the Catalog. All these will raise the error here or am I missing something?

back from PTO 😄 so I enhanced the PR to also cover URIs like {{scheme}}:{{resource}}. The inspector will now ignore URIs that are not resolvable due to a scheme using a placeholder. Added some unit tests to cover some scenarios.
@tadayosi @squakez please have another look

The CI job failing because of TestLocalBuildWithInvalidDependency but the error message says this is due to a test that has been running before:
panic: Fail in goroutine after TestLocalBuildWithTrait has completed

Does that sound familiar to somebody?
Could someone please rerun the failing CI job so we make sure this has been a temp hiccup

Re-running it.

@squakez thank you!
YAY! all green!


yaml_cpp_0_3: 0.3.1-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository yaml_cpp_0_3 to 0.3.1-0:

upstream repository: https://github.com/stonier/yaml_cpp_0_3.git
release repository: https://github.com/yujinrobot-release/yaml_cpp_0_3-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.5.23
previous version for package: null

yaml_cpp_0_3
* refactored the actual library namespace (previously just renamed the library)

While it looks like ubuntu now has support for the old version, it doesn't let you develop with both versions simultaneously so this is still useful packaged as a ros 3rd party dependency, with appropriate namespacing for the headers.


ListView rendering crashes when bound to @Component object list member

This is the  view that crashes:
<StackLayout>
    <ListView [items]="heroes">
       <template let-item="hero">
           <StackLayout>
               <Label [text]="hero.name"></Label>
           </StackLayout>
       </template>
    </ListView>
</StackLayout>

And this is the component:

import { Component } from '@wwwalkerrun/nativescript-ngx-magic';
import { OnInit } from '@angular/core';
import { Hero } from './model/Hero';
import { HeroesService } from './services/heroes.service';

@Component({
  moduleId: module.id,
  selector: 'app-root',
  templateUrl: './views/app/app.component.html',
  styleUrls: ['./views/app/app.component.css'],
  providers: [HeroesService]
})
export class AppComponent implements OnInit {
  title = 'app works!';
  heroes: Hero[] = [];

  constructor(private heroesService: HeroesService) {}

  ngOnInit(): void {
      this.heroesService.getHeroes().subscribe(heroesResult => this.heroes = heroesResult);
  }
}

(The component works in a web browser)
I basically tried what is in this video: https://egghead.io/lessons/angular-2-display-lists-using-nativescript-s-listview-component
The error is:

(copy to clipboard does not work)
Calling js method getView failed at .../ui/list-view/list-view.js line 200 column 28

@sebastienratez you have a syntax error in your HTML file when declaring the let for your item.
Where you have  it should be
 <template let-hero="item">

@NickIliev my good holiness I can't believe it... thank you


pull_db.sh fails in certain case

Specifically, pull_db and likely other scripts fail if the remote db password begins with ">".

In that case, you'll need to quote the password command... or change the password on the db.
I would have it quoting the password by default, but I found that failed with certain versions of MySQL/MariaDB


fix(deno-embed): windows support

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#378  👈
#377
#376
#375
#349
#374
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Add note on multilingual docs to README

Thanks for the suggestion! I've migrated the new content to its own section :)


Add first subscription metric

[x] subscriptions.total.sum

@imsickofmaps Please review.

@gsvr re-reviewed this and thought about all the other metrics we're going to be adding and I think we need to re-look at the logging sender. Because there's no payload checking on responses it feels pretty shady relying on the total metrics fired when we really care about the correctness of the data. Thoughts?

Couple of tweaks then :+1: when travis is happy


Gevent support in Python 3

Gevent 1.2 supports 3.4+ according to their documentation (3.3+ less officially), so theoretically this library shouldn't have any problems supporting thriftpy_gevent in 3.4+ (3.3+). I propose, then, that it should (and if it does, the README should reflect that)

#50 targets this.

...and merged!


Adjust guard logic on amazon fwds

Proposed change

Rearrange guard logic on amazon fwds check

Type of change

[ ] Dependency upgrade
[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR is related to issue: n/a

Codecov Report

Merging #324 (40812ca) into 0.3.0 (85c43cf) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##            0.3.0     #324      +/-   ##
==========================================
- Coverage   97.44%   97.42%   -0.03%
==========================================
  Files           3        3
  Lines         745      737       -8
==========================================
- Hits          726      718       -8
  Misses         19       19

Impacted Files
Coverage Δ

custom_components/mail_and_packages/helpers.py
96.76% <100.00%> (+<0.01%)
:arrow_up:

custom_components/mail_and_packages/sensor.py
98.21% <0.00%> (-0.23%)
:arrow_down:

custom_components/mail_and_packages/config_flow.py
100.00% <0.00%> (ø)

Codecov Report

Merging #324 (40812ca) into 0.3.0 (85c43cf) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##            0.3.0     #324      +/-   ##
==========================================
- Coverage   97.44%   97.42%   -0.03%
==========================================
  Files           3        3
  Lines         745      737       -8
==========================================
- Hits          726      718       -8
  Misses         19       19

Impacted Files
Coverage Δ

custom_components/mail_and_packages/helpers.py
96.76% <100.00%> (+<0.01%)
:arrow_up:

custom_components/mail_and_packages/sensor.py
98.21% <0.00%> (-0.23%)
:arrow_down:

custom_components/mail_and_packages/config_flow.py
100.00% <0.00%> (ø)


Automated Package Upgrade

:tada: This PR is included in version 7.7.229 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add fully type-erased event triggering

Objective
Use case: A socket receives packets that are disambiguated via a packet ID and the current client state. I need to trigger these packets as events, but I don't statically know the type when triggering the events. EmitDynamicTrigger currently exists to try to fulfill this need, but I don't have a type that can stand-in to be passed to EmitDynamicTrigger, so it feels a bit implicitly unsafe.
We should be able to trigger events with a ComponentId and type-erased PtrMut to the event data, instead.
Solution
Added:
impl World {
    pub unsafe fn trigger_ref_dynamic<T: Traversal>(
        &mut self,
        event_type: ComponentId,
        event_data: PtrMut<'_>,
        auto_propagate: bool,
    );

    pub unsafe fn trigger_targets_ref_dynamic<T: Traversal>(
        &mut self,
        event_type: ComponentId,
        event_data: PtrMut<'_>,
        targets: impl TriggerTargets,
        auto_propagate: bool,
    );
}

Note: I have not added a Command variant of these because PtrMut<'a> is not 'static, and it's not important for my use case.
Box<dyn Any> would probably work, though.
Testing

Added two tests.

Well, actually I just figured out a better solution involving storing fn(Commands, RawPacket) instead, so this isn't necessary for me anymore. 😅
If anyone else finds a use for it, I'll reopen.


Applicant Dashboard

Figma link with the application designs: dashboards and forms. https://www.figma.com/file/nhOquOVmWD1P3BxEY0KIh5/CCBC-Intake?node-id=0%3A1

@zoeyli-46 , please, add the link to the prototype in the description.

See also #31 which is the dashboard after an application has been submitted


Documentation: "engineVersion: v2" required even when using apiVersion v1beta1

Describe the solution you'd like
The "Advanced Templating/v2" section of the documentation (https://external-secrets.io/v0.5.7/guides-templating/) states that "If you are still using v1alpha1, You have to opt-in to use the new engine version by specifying template.engineVersion=v2". From my own use case it appears that it is necessary to include this engine specification even when "apiVersion: external-secrets.io/v1beta1" is specified. Without it, values from the data section referenced with dot notation are not successfully pulled. Unless I am mistaken this should be corrected.

Hi @EthanGahm ! We force the default of v1beta1 externalsecrets api version to be v2 (see https://github.com/external-secrets/external-secrets/blob/main/apis/externalsecrets/v1beta1/externalsecret_types.go#L93). Unless your externalSecret got converted from a v1alpha1 manifest, you do not need to specify it.
You can check what is the loaded value from your manifests by inspecting kubectl get es <your-es-name> -o yaml. It should be stated if it is using engineVersion v1 or v2 there.

If you can share your manifests, we can take a look and understand what's going on there, but you shouldn't need to specify the engineVersion whatsoever.

Thanks for the context. I think we may have been misunderstanding the problem.


Arbitrary Return Types for Register Modification Closures

Discussed in #859.
Added

write_and
write_with_zero_and
modify_and

These functions return a T. The writer cannot be returned outside of the closure because the lifetime is constrained to remain within the function body.
Motivation
My reasons for this change are to allow strong type-state validity:
let state = rb.reg().write_and(|w| State::set(w.field()));

Thanks @burrbull!

These functions return a T

Return a generic T which corresponds to type returned by passed closure.
Just a note.

Instead of suffix it can be prefix.
What about external_write? ext_write

Hey great minds think alike, I was just thinking the same thing.
I was thinking from_write (but that's a bit close to From) and with_write.

from_write sounds good

Ok well I guess we can tentatively go with from_* then. Thanks!

On a side note, to reduce duplicate code, I think the original methods could invoke the new ones with the generic T filled in. Will look into it later.

cc @Emilgardis any comments?

Would it make sense to pit this behind a setting? I think not, but somethibg to maybe consider


fix object assignments

This patch allows the following code to work:
var foo = {}
foo.bar = css`.foo { color: blue }`

Codecov Report

Merging #123 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
+ Coverage   97.18%   97.22%   +0.03%
==========================================
  Files           2        2
  Lines         142      144       +2
==========================================
+ Hits          138      140       +2
  Misses          4        4

Impacted Files
Coverage Δ

transform.js
98.76% <100%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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

v6.0.2


Added config file to handle default directories

Figuring out configurable file for the brainglobe directory. Also:

downloading bar when retrieving the atlas;
name fixes for the atlases

This should now be fixed.
The way it currently works: There is a bg_config.conf file in the repo. Its position is kept in the variable CONFIG_PATH here: https://github.com/brainglobe/brainatlas-api/blob/8f60bb9c0d41f42bf91e80c20d561ba9aa1ef865/brainatlas_api/config.py#L6
If you can suggest a better way of referencing to the package location, I am happy to change this.
There is no "template" or "dafault" config in the repo, which I can imagine being a pain for version control of we want to avoid weird file duplications. It is written from a standard configuration dictionary here:
https://github.com/brainglobe/brainatlas-api/blob/8f60bb9c0d41f42bf91e80c20d561ba9aa1ef865/brainatlas_api/config.py#L10
so that it is initialised with the valid user root folder. The folder is created in the BGAtlas class the first time we actually need it:
https://github.com/brainglobe/brainatlas-api/blob/8f60bb9c0d41f42bf91e80c20d561ba9aa1ef865/brainatlas_api/bg_atlas.py#L44
To change this location, there's 3 ways:

manually editing the bg_config.conf file;
using brainatlas_api.config.write_config_value(brainglobe_dir, new_path);
using a little command line utility:

 > brainatlas_config -k brainglobe_dir -v /Users/luigipetrucco/.brainglobe

In the future the configuration could be used to handle other values.
Let me know what you think about all of this, also eg for variable names etc, it would be important to keep this part of the code stable in the future.

Nice one, it looks really good.

using a little command line utility:

What's the best way to add tests for this kind of stuff?
Currently bg_atlas accepts a brainglobe_path argument:
https://github.com/brainglobe/brainatlas-api/blob/8442627ee7b9df35958e7d8989737c956d0d9503/brainatlas_api/bg_atlas.py#L34
How does it interact with the path in the config stuff?

Nice one, it looks really good.

using a little command line utility:

What's the best way to add tests for this kind of stuff?

If we agree that we like this scheme I will try to figure out tests, click has some testing modules so it should not be difficult.

Currently bg_atlas accepts a brainglobe_path argument:
https://github.com/brainglobe/brainatlas-api/blob/8442627ee7b9df35958e7d8989737c956d0d9503/brainatlas_api/bg_atlas.py#L34
How does it interact with the path in the config stuff?

Object arguments have full priority over default ones!

Looks good.

If you can suggest a better way of referencing to the package location, I am happy to change this.

I'm not sure if it's a "better" way to reference the package location, but you can also use something like this, that would not be dependent on the location of the file calling the function:
from pkg_resources import resource_filename
CONFIG_PATH =  Path(resource_filename("brainatlas_api", "bg_config.conf"))

resource_filename

Great, I like this more! Then I'll write tests and merge this by today.


cannot install extension from vsix

Issue Type: Bug
Selecting "Extensions > Install from VSIX" results in an error message "your extension policy does not allow installing extensions. please change your extension policy and try again."
Azure Data Studio version: azuredatastudio 1.40.1 (3a32679ab590c1023ab4de6f191769e7e127d25b, 2022-11-22T18:22:25.529Z)
OS version: Darwin arm64 22.3.0
Restricted Mode: No
Preview Features: Enabled

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
5, 4, 3

Memory (System)
16.00GB (0.56GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (12)

Extension
Author (truncated)
Version

query-editor-boost
dre
0.4.1

ssmskeymap
kev
1.1.0

azuredatastudio-oracle
Mic
0.1.2

dacpac
Mic
1.11.0

dsct-oracle-to-ms-sql
Mic
0.2.0

import
Mic
1.5.4

net-6-runtime
Mic
1.1.0

profiler
Mic
0.11.0

query-history
Mic
0.5.3

schema-compare
Mic
1.17.0

sql-database-projects
Mic
0.20.0

azdata-sanddance
msr
4.0.1

(1 theme extensions excluded)

workaround: based on https://github.com/microsoft/azuredatastudio/blob/6998c8787f5c7bf4378bd41092e218ed74e092a4/src/vs/platform/extensions/common/extensions.ts#L18, updated settings for "Extensions: Extensions Policy" to allowAll.  Was blank in my settings

@dzsquared when you say it was blank, do you mean in the setting editor or in the plain settings.json? The default value is allowAll, you won't see it in the settings.json if it is the default value.

@alanrenmsft the settings.json included:
 "extensions.extensionsPolicy": "",

last time we had the similar issue was because the setting contribution is removed accidentally: https://github.com/microsoft/azuredatastudio/issues/6456, I checked the code and it is still there, I also couldn't reproduce the problem on Windows or macOS. I suspect that you might have accidentally modified the setting.


Fix wrappers.Request.application docs

The request is given as the last argument, not the first.
wchargin-branch: wrappers-application-docs

This is sort of misleading too, as all "last" really means in this case is "after self if the application is a method", but in effect that's the first real argument.

But it’s not just “after self”; it can be used with
functools.partial:
$ cat /tmp/test.py
from __future__ import print_function

import functools
import werkzeug

@werkzeug.wrappers.Request.application
def foo(x, y, request):
  return werkzeug.Response("{} + {} = {}".format(x, y, x + y))

foo_one_two = functools.partial(foo, 1, 2)

print(list(foo_one_two({"REQUEST_METHOD": "GET"}, print)))
$ python /tmp/test.py
200 OK [('Content-Type', 'text/plain; charset=utf-8'), ('Content-Length', '9')]
['1 + 2 = 3']

If I’m reading the code correctly—
resp = f(*args[:-2] + (request,))

—it looks pretty clear that request is actually the last argument, and
that any arguments prior to the WSGI environ and start_response are
passed through to the underlying function, which makes both self and
other bound arguments work.

Ah, you're right, the change is fine then.

Thank you!


nmstatectl: Show propries in the insertion order.

nmstatectl should show the name of link-aggregration ports first in the output, for example:
- name: ovs-br-lagg0
  type: ovs-bridge
  state: up
  bridge:
    port:
    - name: veth0_ep
    - name: ovs-lagg0
      link-aggregation:
        mode: balance-slb
        slaves:
        - name: veth1_ep
        - name: veth2_ep

This needs adjustments to order the state recursively (or at least specificially for ovs-bridges):
https://github.com/nmstate/nmstate/blob/master/libnmstate/prettystate.py#L91

If you fix this generically, it would be great to test it for team ports as well. But it could also be fixed in a different PR/issue if needed.
Example:
cat team0.yaml
interfaces:
- name: team0
  type: team
  state: up
  team:
    ports:
    - name: vethteam0
    - name: vethteam0_ep
    runner:
      name: roundrobin

ip link add vethteam0 type veth peer name vethteam0_ep
nmcli dev set vethteam0 managed yes
nmcli dev set vethteam0_ep managed yes
nmstatectl set team0.yaml
nmstatectl show team0

Actual results:
  team:
    - link_watch:
        name: ethtool
      name: vethteam0
    - link_watch:
        name: ethtool
      name: vethteam0_ep
    runner:
      name: roundrobin

Expected results:
  team:
    ports:
    - name: vethteam0
      link_watch:
        name: ethtool
    - name: vethteam0_ep
      link_watch:
        name: ethtool
    runner:
      name: roundrobin

Since python 3.6 the dict will use insertion ordered.
With yaml.dump(sort_keys=False), the nmstatectl could show the properties with the order of properties insertion.
But the sort_keys=False only available in pyyaml 5.1+ which does not exists in RHEL/CentOS 8.
We could:
try:
    yaml.dump(state, default_flow_style=False, sort_keys=False)
except TypeError:
    # some hack like yaml.add_representer()

https://stackoverflow.com/questions/16782112/can-pyyaml-dump-dict-items-in-non-alphabetical-order

I will link all the property ordering issues here.
If you think fixing this in the generic way is too hard for good first issue, please let us know.

I think this is too much for a good first issue IMO. AFAIK, @elvgarrui have not started to work on this.

No I have not started yet. I will unassign myself. Thanks for the heads up!!

Can I dig into it?

No sorry, I am going to take this. As I think this could be too much for a good first issue. Thank you!

This is fixing:

https://bugzilla.redhat.com/1806474
https://bugzilla.redhat.com/1809038


RenderAction should be independent of http method

When using @{ Html.RenderAction("ActionName", "ControllerName") } MVC should return the specified action independent of the current http method.
See http://stackoverflow.com/questions/4227930/renderaction-calls-wrong-action-method and http://stackoverflow.com/questions/16941317/a-public-action-method-was-not-found-on-controller for more information

RenderAction is gone in ASP.NET Core so this doesn't apply. Should this be an MVC bug?

Am I in the wrong MVC?

@dotnetchris you likely want MVC version 5.x. That's available at https://aspnetwebstack.codeplex.com/ This repo is for ASP.NET 5 MVC, a rewrite that works on.NET Core.

Assuming that there is no comparable situation here. (And my god does Microsoft not know how to name anything)


hadoop: qualify all relative path

Codecov Report

Merging #2709 (b1e1f47) into main (cdccfe8) will decrease coverage by 0.01%.
The diff coverage is n/a.

:exclamation: Current head b1e1f47 differs from pull request most recent head 3a2994f. Consider uploading reports for the commit 3a2994f to get more accurate results

@@            Coverage Diff             @@
##             main    #2709      +/-   ##
==========================================
- Coverage   59.58%   59.56%   -0.02%
==========================================
  Files         138      138
  Lines       23626    23626
==========================================
- Hits        14077    14073       -4
- Misses       7716     7720       +4
  Partials     1833     1833

Impacted Files
Coverage Δ

pkg/meta/tkv_mem.go
86.61% <0.00%> (-0.71%)
:arrow_down:

pkg/meta/sql.go
63.71% <0.00%> (-0.25%)
:arrow_down:

pkg/chunk/cached_store.go
74.02% <0.00%> (-0.19%)
:arrow_down:

pkg/chunk/disk_cache.go
78.20% <0.00%> (ø)

pkg/chunk/utils_unix.go
91.66% <0.00%> (+25.00%)
:arrow_up:

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


Update system blueprint

Need

As a Developer
I want a blueprint of how data flows through the system
so that I can visualize what currently exists/what is planned

Approach

[x] use lucid chart to show architecture plans
[ ] update lucid chart to show reality

https://www.lucidchart.com/invitations/accept/29d63295-dc9d-43e5-82ea-7f9e2b7afc49

Add sync and async read path

@MDunitz add this to the Github developer README.md.


[Question] Control animation speed

The css animations go from 0 to 50 to 100%, so they are fast by default.
So is it possible to control the speed, to make the animations slower?

Using the la-timer as an example...changing the following properties will increase (with lower number) or decrease (with higher number) the speed of the seconds hand.
.la-timer>div:before,
.la-timer>div:after {
-webkit-animation: timer-loader 12ms infinite linear;
-moz-animation: timer-loader 12ms infinite linear;
-o-animation: timer-loader 12ms infinite linear;
animation: timer-loader 12ms infinite linear;
}
And changing the the following will increase (with lower number) or decrease (with higher number) the speed of the minutes hand.
.la-timer>div:after {
-webkit-animation-duration: 1s;
-moz-animation-duration: 1s;
-o-animation-duration: 1s;
animation-duration: 1s;
}
Here is an example you can play around with:
https://jsfiddle.net/bschreck86/obvwfj0p/9/


argocd-operator Service Account Missing RBAC for v1.HorizontalPodAutoscaler when installing via OLM

The argocd-operator Service Account Missing is missing RBAC for v1.HorizontalPodAutoscaler when installing via Operator Lifecycle Manager ( OLM ). An initial install works ( I believe ), but subsequent reconciles are blocked from the error. I think this has been added on master https://github.com/argoproj-labs/argocd-operator/blob/master/deploy/role.yaml#L50 , and it's just a version skew with what's deployed for OLM.
I followed the directions from the docs site for installing the argocd operator, as well as OLM ( 0.14.1 )
You can add a new role or patch an existing role bound to the SA to resolve the issue.
kind: Role
metadata:
  name: argocd-operator-hack
rules:
  - apiGroups:
      - autoscaling
    resources:
      - horizontalpodautoscalers
    verbs:
      - "*"
---
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  name: argocd-operator-hack
subjects:
  - kind: ServiceAccount
    name: argocd-operator
roleRef:
  kind: Role
  name: argocd-operator-hack
  apiGroup: rbac.authorization.k8s.io

Here is the log message below from the k logs -f argocd-operator-cbd8b45cf-wxstt -n argocd

E0217 23:54:22.476979       1 reflector.go:125] pkg/mod/k8s.io/client-go@v0.0.0-20190918200256-06eb1244587a/tools/cache/reflector.go:98: Failed to list *v1.HorizontalPodAutoscaler: horizontalpodautoscalers.autoscaling is forbidden: User "system:serviceaccount:argocd:argocd-operator" cannot list resource "horizontalpodautoscalers" in API group "autoscaling" in the namespace "argocd"

And once you apply the hack for the service account

1 reflector.go:125] pkg/mod/k8s.io/client-go@v0.0.0-20190918200256-06eb1244587a/tools/cache/reflector.go:98: Failed to list *v1.HorizontalPodAutoscaler: horizontalpodautoscalers.autoscaling is forbidden: User "system:serviceaccount:argocd:argocd-operator" cannot list resource "horizontalpodautoscalers" in API group "autoscaling" in the namespace "argocd"
{"level":"info","ts":1581985470.2491558,"logger":"controller_argocd","msg":"reconciling ingresses"}
{"level":"info","ts":1581985470.2693412,"logger":"controller_argocd","msg":"Reconciling ArgoCD","namespace":"argocd","name":"example-argocd"}
{"level":"info","ts":1581985470.2694004,"logger":"controller_argocd","msg":"reconciling service accounts"}
{"level":"info","ts":1581985470.2694175,"logger":"controller_argocd","msg":"reconciling certificate authority"}
{"level":"info","ts":1581985470.2699878,"logger":"controller_argocd","msg":"reconciling CA secret"}
{"level":"info","ts":1581985470.2700505,"logger":"controller_argocd","msg":"reconciling CA config map"}
{"level":"info","ts":1581985470.2704508,"logger":"controller_argocd","msg":"reconciling secrets"}

System Info:
minikube version: v1.7.2
commit: 50d543b5fcb0e1c0d7c27b1398a9a9790df09dfb
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.5", GitCommit:"20c265fef0741dd71a66480e35bd69f18351daea", GitTreeState:"clean", BuildDate:"2019-10-15T19:16:51Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:22:30Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}

Thank you for reporting this @pmcjury . Will add a PR today to fix this.

Released as part of v0.0.5.


Reinstated save_png method of figure

@SylvainCorlay Brought back the save_png function for the Figure, @stonebig asked for a stable beta, I think this should probably be included

The issue is that the save_png makes each view save the png.

As in all views or the current view? Because the latter seems to be what we want right?
----- Original Message -----
From: Sylvain Corlay<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC: DHRUV MADEKA<EMAIL_ADDRESS>At: 17-Jun-2016 15:42:08
The issue is that the save_png makes each view save the png.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

All the views... It is a kernel function that sends a message to all the
views...
That is why we need to make the toolbar a child widget and have the save
button only save the one of the view of the parent figure...
On Fri, Jun 17, 2016 at 5:27 PM, Dhruv Madeka<EMAIL_ADDRESS>wrote:

As in all views or the current view? Because the latter seems to be what
we want right?
----- Original Message -----
From: Sylvain Corlay<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC: DHRUV MADEKA<EMAIL_ADDRESS>At: 17-Jun-2016 15:42:08
The issue is that the save_png makes each view save the png.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bloomberg/bqplot/pull/166#issuecomment-226885705, or mute
the thread
https://github.com/notifications/unsubscribe/ACSXFr5BRQNiFiXKBnRihKgpL3KVH7uYks5qMxE5gaJpZM4I4mAz
.

Alternately, we could have a little button on the Figure itself right? A controllable mini-toolbar thats displayed or not and clicking that could save the Figure. Because right now to save, you have to create a Toolbar instance, plug it to the Figure and then display it and click save. There's no Pythonic way or simple way to do it

Actually, the direction I wanted to take was to move up the toolbar widget
as a child widget and have the figure instantiate a default toolbar. This
would make it more easily extensible ...
Sylvain
On Sat, Jun 18, 2016 at 5:10 PM, Dhruv Madeka<EMAIL_ADDRESS>wrote:

Alternately, we could have a little button on the Figure itself right? A
controllable mini-toolbar thats displayed or not and clicking that could
save the Figure. Because right now to save, you have to create a Toolbar
instance, plug it to the Figure and then display it and click save.
There's no Pythonic way or simple way to do it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bloomberg/bqplot/pull/166#issuecomment-226947359, or mute
the thread
https://github.com/notifications/unsubscribe/ACSXFjeIFu0MEPgdjq9XKZp_YTyinQb3ks5qNApZgaJpZM4I4mAz
.

Well. Ideally, the Toolbar should allow you to configure any properties of the Figure right? So that's the right way to go I guess

btw @SylvainCorlay will the Toolbar as a child solve the issue that we aren't able to programatically save a png?

No, I am still ok with your PR!
On Mon, Jun 20, 2016 at 4:59 PM, Dhruv Madeka<EMAIL_ADDRESS>wrote:

btw @SylvainCorlay https://github.com/SylvainCorlay will the Toolbar as
a child solve the issue that we aren't able to programatically save a png?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bloomberg/bqplot/pull/166#issuecomment-227267789, or mute
the thread
https://github.com/notifications/unsubscribe/ACSXFnxG1zezSE9r91LZUkKQqQHgFGliks5qNv85gaJpZM4I4mAz
.

okay @SylvainCorlay yours to merge then

Thanks


Thank you

Hey @ayrat555 and @pxp9,
I hope you're both doing well -- I just wanted to say thanks for this wonderful crate. It's because of efforts like this that the rust ecosystem can continue to grow and see more adoption.

Thank you so much @Wulf

Hey @Wulf We are back !!!
This summer we are planning to implement some more features.
More backends
SQLite Backend
InMemory Backend
Maybe also
SurrealDb backend.

Great to hear. I’ll be sure to lookout for the updates :)
We’re using fang in create-react-app but only if end users choose postgres as their backend. Would love to see SQLite support! 🙌

@pxp9 Can you tell more about the in-memory backend. I was sent here from a reddit suggestion.

@pxp9 Can you tell more about the in-memory backend. I was sent here from a reddit suggestion.

Sure, In memory backend will be supported when SQLite is supported.
SQLite has a mode that it is in memory.
As you can see #141 pr , SQLite is implemented for asynk module.
There is not support yet for blocking module.
We are working to have a multiple queues support and ensure that performance and reliability are good.


Use TimeSpan for break period

Fixes #7826

@amnguye @seanmcc-msft I would like to merge this in by EOD Wednesday, if possible. Please if you can take a look, thanks!

/azp run net - storage - ci


Update datafusion to v43

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

I guess we'll have to wait for datafusion-python to be released before we could update ballista-python

closing as duplicate of #1125 (#1124)


Provide a list of recommendations to reduce memory allocations.

Description
I am migrating my asp.net core app to .net 8 and in the spirit of this migration I've stumbled across the fact that parsing a 250 kb with my standard approach consumes around 5.3 MB of RAM.
I've started experimenting with the benchmark and discovered several potential improvements to my code.
To provide some context:

using AngleSharp;
using AngleSharp.Dom;
using AngleSharp.Html.Parser;
using BenchmarkDotNet.Attributes;
using BenchmarkDotNet.Configs;
using BenchmarkDotNet.Environments;
using BenchmarkDotNet.Jobs;

namespace ConsoleApp7;

[Config(typeof(Config))]
[MemoryDiagnoser]
public class AngleSharpBenchmarks
{
    private class Config : ManualConfig
    {
        public Config()
        {
            AddJob(Job.ShortRun.WithRuntime(CoreRuntime.Core80));
        }
    }

    private static readonly HtmlParserOptions Options = new HtmlParserOptions()
    {
        IsEmbedded = false,
        IsScripting = false,
        IsStrictMode = false,
        IsNotSupportingFrames = true,
        IsKeepingSourceReferences = false,
        IsPreservingAttributeNames = false,
        IsSupportingProcessingInstructions = false,
        IsNotConsumingCharacterReferences = true,
        IsAcceptingCustomElementsEverywhere = true,
    };
    private static readonly BrowsingContext Context = new BrowsingContext();
    private static readonly HtmlParser parser = new HtmlParser(Options, Context);

    [Benchmark(Description = "Custom options + no dispose + unwrap linq + cached parser")]
    public async Task<string?> Allocate_Cached()
    {
        await using var file = File.OpenRead("home.html");
        var document = await parser.ParseDocumentAsync(file);

        string? found = null;
        foreach (var scriptElement in document.Scripts)
        {
            var data = scriptElement.ChildNodes.OfType<IText>().FirstOrDefault()?.Data;
            var contains = data?.Contains("var dimensionsArray = [];");
            if (contains == true)
            {
                found = data;
                break;
            }
        }

        return found;
    }

    [Benchmark(Description = "parse stream + no dispose + unwrap linq")]
    public async Task<string?> Allocate_Realistic()
    {
        await using var file = File.OpenRead("home.html");
        var parser = new HtmlParser();
        var document = await parser.ParseDocumentAsync(file);
        string? found = null;
        foreach (var scriptElement in document.Scripts)
        {
            var data = scriptElement.ChildNodes.OfType<IText>().FirstOrDefault()?.Data;
            var contains = data?.Contains("var dimensionsArray = [];");
            if (contains == true)
            {
                found = data;
                break;
            }
        }

        return found;
    }

    [Benchmark(Description = "parse stream + no dispose")]
    public async Task<string?> Allocate()
    {
        await using var file = File.OpenRead("home.html");
        var parser = new HtmlParser();
        var document = await parser.ParseDocumentAsync(file);
        var interestingScript = document.Scripts.FirstOrDefault(it => it.GetDescendants().OfType<IText>().FirstOrDefault()?.Data.Contains("var dimensionsArray = [];") == true);
        return interestingScript?.Text;
    }

    [Benchmark(Description = "parse stream + dispose doc")]
    public async Task<string?> Allocate4()
    {
        await using var file = File.OpenRead("home.html");
        var parser = new HtmlParser();
        using var document = await parser.ParseDocumentAsync(file);
        var interestingScript = document.Scripts.FirstOrDefault(it => it.Text.Contains("var dimensionsArray = [];"));
        return interestingScript?.Text;
    }

    [Benchmark(Description = "parse string", Baseline = true)]
    public async Task<string?> Allocate5()
    {
        var file = await File.ReadAllTextAsync("home.html");
        var parser = new HtmlParser();
        var document = parser.ParseDocument(file);
        var interestingScript = document.Scripts.FirstOrDefault(it => it.Text.Contains("var dimensionsArray = [];"));
        return interestingScript?.Text;
    }

    [Benchmark(Description = "parse string + using on doc")]
    public async Task<string?> Allocate6()
    {
        var file = await File.ReadAllTextAsync("home.html");
        var parser = new HtmlParser();
        using var document = parser.ParseDocument(file);
        var interestingScript = document.Scripts.FirstOrDefault(it => it.Text.Contains("var dimensionsArray = [];"));
        return interestingScript?.Text;
    }
}

and results:
| Method                                            | Mean     | Error     | StdDev    | Ratio | RatioSD | Gen0     | Gen1     | Gen2     | Allocated | Alloc Ratio |
|-------------------------------------------------- |---------:|----------:|----------:|------:|--------:|---------:|---------:|---------:|----------:|------------:|
| 'Custom options + no using + unwrap linq'         | 5.896 ms | 2.4853 ms | 0.1362 ms |  0.68 |    0.03 | 476.5625 | 265.6250 | 125.0000 |   3.65 MB |        0.76 |
| 'Custom options + no using'                       | 5.890 ms | 0.1473 ms | 0.0081 ms |  0.68 |    0.01 | 476.5625 | 320.3125 | 156.2500 |   3.76 MB |        0.78 |
| 'Default options'                                 | 5.947 ms | 0.1298 ms | 0.0071 ms |  0.69 |    0.01 | 492.1875 | 328.1250 | 328.1250 |   4.49 MB |        0.94 |
| 'Cached options + cached context'                 | 5.973 ms | 0.7051 ms | 0.0386 ms |  0.69 |    0.01 | 492.1875 | 328.1250 | 328.1250 |   4.43 MB |        0.92 |
| 'Cached options + cached context + cached parser' | 5.957 ms | 1.0465 ms | 0.0574 ms |  0.69 |    0.01 | 492.1875 | 328.1250 | 328.1250 |   4.43 MB |        0.92 |
| string                                            | 8.666 ms | 2.2499 ms | 0.1233 ms |  1.00 |    0.00 | 812.5000 | 796.8750 | 421.8750 |    4.8 MB |        1.00 |
| 'string + using on doc'                           | 7.019 ms | 1.5742 ms | 0.0863 ms |  0.81 |    0.00 | 906.2500 | 820.3125 | 531.2500 |   5.28 MB |        1.10 |

Disposing a documents results in an 10% of memory consumption.
Accessing text content might be pricey
Caching parser + settings yields a minor memory consumption win, but it is unclear if I can cache it. Could it be accumulating circular references, idk.
Would appreciate if someone can come up with answer to the following questions:

Is it safe to not dispose IDocument instances?
Can I reuse parser instances during the lifetime of an app? Is there a sample on how to pool them maybe?
Is there a cheap way or maybe an overload which will allow to get text content of a script without extra allocations?
What options on HtmlParserOptions and BrowsingContext can/should I use to reduce memory consumption if I am only interested in parsing html (no scripting and css) without preserving context.

Background
No response
Specification
No response

Is it safe to not dispose IDocument instances?

It should be pretty safe, but disposing it should be a bit more reliable. It would be great if somebody does some tests on this.

Can I reuse parser instances during the lifetime of an app? Is there a sample on how to pool them maybe?

The parser yes - the underlying DOM builder no (but I guess you refer to the parser itself, which you can definitely re-use). You can even use a single parser for everything - as the parser is just an abstraction and spawns new DOM builders there would be no side-effect. However, it is not thread-safe and it is bound to a given browsing context and options. So, if you want to change either one of them then don't reuse it.

Is there a cheap way or maybe an overload which will allow to get text content of a script without extra allocations?

No - the parser always constructs the full model and thus you will always get the allocations for everything in the DOM. Presumably, we could maybe add something to not allocate / immediately throw elements away, but the use-case for this is little and the effort would be high.

What options on HtmlParserOptions and BrowsingContext can/should I use to reduce memory consumption if I am only interested in parsing html (no scripting and css) without preserving context.

The standard options should be as good as it can be. For instance, switching IsKeepingSourceReferences to true would certainly have a bad impact, so don't do that unless you need to keep the original data around (with additional pointers from essentially every element).
Likewise, the Configuration.Default should be quite good, but if you want to be even better then throw out everything related to CSS selectors / parsing. Nevertheless, here I also don't see much saving potential.

I've been using AngleSharp for a several years already, and was not disposing parsed documents, just started doing it recently for reasons. I've deployed a version to production env without Dispose. Last time it was causing issues it was due to GC not removing parsed docs due to circular references probably, (#45). To be fair I was not noticing any memory related issues with it, but will keep an eye on it. Common symptom is memory working set grows indefinitely?
Due to parser reuse. If, lets say, I was always using HtmlParser with default config I can potentially declare a factory with a ObjectPool<HtmlParser> implementation and return rented parsers there, guarantying no multithreaded access to it I should be fine? It won't accumulate state over time somehow? (sorry for maybe asking same question twice but just to be sure)


WFCORE-2192 Change deployment restart on missing dependency to happen…

… in the phase stop()
For now this only takes effect if the system property 'org.jboss.msc.directionalExecutor' is set
Previously this happened on next start, however this could lead to patalogical situations
if optional dependencies were in use where deployments would continually restart. After
this change when a deployment has missing dependencies it will immediatly attempt to
restart, and will pause at the point that it requires the missing dependencies.

@ppalaga @gaol This change needs to be paired with an MSC upgrade. So it should not be merged into EAP branches without the related upgrade.

@bstansberry you probably mean the upgrade to MSC 1.2.7.SP1 tracked by https://issues.jboss.org/browse/WFCORE-2208 that incorporates just https://issues.jboss.org/browse/MSC-159, right?
MSC-159 does not seem to have a PR yet. Who is working on it?

@ppalaga MSC-159 is resolved; 1.2.7.SP1 is now released, as is a 1.1.7.SP1 suitable for EAP 6.4.x. I just sent up a PR for the WildFly Core upgrade to 1.2.7.SP1.
I'm also going to rework https://github.com/wildfly/wildfly-core/pull/2062 a bit to move the test to testsuite/manualmode as having these fixes take effect requires setting a system property as server launch. So a manual launch.

Thanks for the info, @bstansberry

Superceded by #2099 which is merged.
@ppalaga FYI https://github.com/jbossas/jboss-eap/commit/a5054ce17142981cfdf31cd19cd742b7c459704c is a variant of the 3rd commit on #2099, but ported to 6.4.x. But I don't have time to do a backport of the 4th commit, which moves the test to the manualmode testsuite and tweaks things to ensure the server is started with the org.jboss.msc.directionalExecutor system property. The way manualmode works in 6.4.x is different so it's not just a cherry-pick backport.
I don't think backporting the test is really required; it's far from a 100% reproducer of the problem anyway.


Fallback to response body if text undefined

Thank you -- if it's not too much trouble could you also publish a new version of the package https://www.npmjs.com/package/expressjs-openwhisk as well?

@shazron done!


Add typst-action

Adding typst-action, a GitHub action to build Typst files.
Useful for CI/CD and release of fresh PDFs on push of new commits.

What an excellent addition to the world of typst! Thanks for the awesome contribution @lvignoli


TP2000-749: Add quota step to multiple measures edit form

TP2000-749: Add quota step to multiple measures edit form

Why

Users need to be able to edit the quota order number on multiple measures

What

Adds an extra step to the multiple measures edit wizard to allow editing of quota order number

Codecov Report
Base: 91.86% // Head: 91.87% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (e1b1dec) compared to base (8d307ef).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head e1b1dec differs from pull request most recent head e6c5a9d. Consider uploading reports for the commit e6c5a9d to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #831      +/-   ##
==========================================
+ Coverage   91.86%   91.87%   +0.01%
==========================================
  Files         399      399
  Lines       26233    26280      +47
  Branches     1988     1993       +5
==========================================
+ Hits        24099    24145      +46
  Misses       1750     1750
- Partials      384      385       +1

Impacted Files
Coverage Δ

measures/conditions.py
100.00% <100.00%> (ø)

measures/constants.py
100.00% <100.00%> (ø)

measures/tests/test_views.py
100.00% <100.00%> (ø)

measures/views.py
97.47% <100.00%> (+<0.01%)
:arrow_up:

measures/forms.py
97.67% <0.00%> (-0.20%)
:arrow_down:

measures/tests/test_forms.py
100.00% <0.00%> (ø)

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

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


Can't clear search history

Best kodi addon ever works perfect. Just need a clear search feature if possible. Thanks

well reboot and in setting clear it

The only setting I'm aware if is clear cache and that doesn't work

The only setting I'm aware of is clear cache and that doesn't work

basically in Quasar you can clear your search, strange its not here

Compyy, Which file did you remove from the user data folder please?

Please, try with 0.0.49.
There is a context menu item on each search query, which removes it from the history.
Total cleanup with "one click" will be added later.

Thank you Elgatito

On 18 Jun 2018, at 2:31 pm, Denis<EMAIL_ADDRESS>wrote:
Please, try with 0.0.49.
There is a context menu item on each search query, which removes it from the history.
Total cleanup with "one click" will be added later.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Closing this as a partly-implemented.
There will be total cleanup handler in plugin Settings later.


Electricity bill Calculator

Electricity bill Calculator using C language

hey @yash-jagtap1706, currently we're accepting contributions in the form of projects, even a simple single-page application would work but a low code contribution as this won't be accepted.


Cannot delete an environment

Which Category is your question related to?
Delete an environment. I want to completely remove an existing environment. Through console, I chose Delete  that resulted int th following error:
Deletion failed ValidationException - An error occurred while processing your request: null
Amplify CLI Version
6.3.1
What AWS Services are you utilizing?
Amplify console
Provide additional details e.g. code snippets

Hey @OlgaMaslova :wave: thanks for raising this! I'm going to transfer this to https://github.com/aws-amplify/amplify-console to get more eyes on it


Firebase

[Memo] After the app has been developed to some extent, the backend is transferred to use Gin.

Setup complete.

c31b44621c17013bd81f91bcf14610a677e58ea2


Windows with NetKVM has a little chance hang during shutdown

If the NetKVM has not initialized yet (or reset), QEMU will ignore
the control message. It will cause driver hanging in infinite while loop
for waiting response (which leads to windows hang in shutdown). In our
test, there is a little chance to receive an OID for setting multicast list
after device reset. To make symptom more clear, I have added some log.
Below is the log and call stack when this situation happens:
Logs:
[ParaNdis_OnShutdown]=>
Common [ParaNdis_ResetVirtIONetDevice] Done
[ParaNdis_OnOidSetMulticastList] Go
[ParaNdis_UpdateDeviceFilters] Do
[CParaNdisCX::SendControlMessage] enter: 4277 - cls: 0, cmd: 0
CParaNdisCX::SendControlMessage - ERROR: in while
Assertion: *** DPC watchdog timeout
This is NOT a break in update time
This is most likely a BUG in an ISR
Perform a stack trace to find the culprit
The period will be doubled on continuation
Use gh to continue!!
Call stack:
kd> k
ChildEBP RetAddr
8b582530 83ab50be nt!KeAccumulateTicks+0x3c5
8b582570 83ab4f6b nt!KeUpdateRunTime+0x145
8b5825c8 83ab9c17 nt!KeUpdateSystemTime+0x613
8b5825c8 83a10122 nt!KeUpdateSystemTimeAssist+0x13
8b582680 8e16db70 hal!HalpTscStallExecutionProcessor+0xc0
WARNING: Stack unwind information not available. Following frames may be wrong.
8b582700 8e16b216 netkvm+0x4b70
8b582730 8e16c418 netkvm+0x2216
8b582740 8e16f219 netkvm+0x3418
8b582750 8e17740f netkvm+0x6219
8b5827a8 880fc32b netkvm+0xe40f
8b5827f0 880a8ce3 ndis!ndisMDoOidRequest+0x3a1
8b582818 880a87ff ndis!ndisFQueueRequestOnNext+0x21e
8b582880 8df21086 ndis!NdisFOidRequest+0x181
8b5828a0 880fc961 pacer!PcFilterRequest+0x78
8b582900 880a8c5c ndis!ndisFDoOidRequest+0x1a2
8b582928 880a87ff ndis!ndisFQueueRequestOnNext+0x197
8b582990 8df15b22 ndis!NdisFOidRequest+0x181
8b5829b0 880fc961 wfplwf!FilterOidRequest+0x50
8b582a10 880a7799 ndis!ndisFDoOidRequest+0x1a2
8b582a3c 8810d602 ndis!ndisQueueRequestOnTop+0x224
8b582ab0 8810d686 ndis!ndisMOidRequest+0x1dd
8b582ac0 882a4646 ndis!NdisOidRequest+0xf
8b582b7c 88254d1d tcpip!FlpNdisRequestUnderReference+0x7d
8b582bb8 8825523b tcpip!FlpMulticastListSet+0xa1
8b582bc4 88251a2e tcpip!FlpSerializedNdisDeleteGroupWorker+0xd
8b582bec 83c4f460 tcpip!FlpSerializedNdisRequestWorkerRoutine+0x41
8b582c00 83ab7aab nt!IopProcessWorkItem+0x23
8b582c50 83c43f5e nt!ExpWorkerThread+0x10d
8b582c90 83aeb219 nt!PspSystemThreadStartup+0x9e
00000000 00000000 nt!KiThreadStartup+0x19
I proposed two approaches to solve this problem:

apply the following patch to simply prevent infinite waiting.

commit bf5ba3319594cef72d279a0d82137141fcff6581
Author: yuchenlin<EMAIL_ADDRESS>Date:   Fri Nov 30 13:50:34 2018 +0800

    NetKVM: Prevent hanging in SendControlMessage

    Windows with NetKVM has a little chance hang during shutdown.

    If the NetKVM has not initialized yet (or reset), QEMU will ignore
    the control message. It will cause driver hanging in infinite while loop
    for waiting response (which leads to windows hang in shutdown). In our
    test, there is a little chance to receive an OID for setting multicast list
    after device reset. To make symptom more clear, I have added some log.
    Below is the log and call stack when this situation happens:

    Logs:

    [ParaNdis_OnShutdown]=>
    Common [ParaNdis_ResetVirtIONetDevice] Done
    [ParaNdis_OnOidSetMulticastList] Go
    [ParaNdis_UpdateDeviceFilters] Do
    [CParaNdisCX::SendControlMessage] enter: 4277 - cls: 0, cmd: 0
    CParaNdisCX::SendControlMessage - ERROR: in while
    Assertion: *** DPC watchdog timeout
        This is NOT a break in update time
        This is most likely a BUG in an ISR
        Perform a stack trace to find the culprit
        The period will be doubled on continuation
        Use gh to continue!!

    Call stack:

    kd> k
    ChildEBP RetAddr
    8b582530 83ab50be nt!KeAccumulateTicks+0x3c5
    8b582570 83ab4f6b nt!KeUpdateRunTime+0x145
    8b5825c8 83ab9c17 nt!KeUpdateSystemTime+0x613
    8b5825c8 83a10122 nt!KeUpdateSystemTimeAssist+0x13
    8b582680 8e16db70 hal!HalpTscStallExecutionProcessor+0xc0
    WARNING: Stack unwind information not available. Following frames may be
    wrong.
    8b582700 8e16b216 netkvm+0x4b70
    8b582730 8e16c418 netkvm+0x2216
    8b582740 8e16f219 netkvm+0x3418
    8b582750 8e17740f netkvm+0x6219
    8b5827a8 880fc32b netkvm+0xe40f
    8b5827f0 880a8ce3 ndis!ndisMDoOidRequest+0x3a1
    8b582818 880a87ff ndis!ndisFQueueRequestOnNext+0x21e
    8b582880 8df21086 ndis!NdisFOidRequest+0x181
    8b5828a0 880fc961 pacer!PcFilterRequest+0x78
    8b582900 880a8c5c ndis!ndisFDoOidRequest+0x1a2
    8b582928 880a87ff ndis!ndisFQueueRequestOnNext+0x197
    8b582990 8df15b22 ndis!NdisFOidRequest+0x181
    8b5829b0 880fc961 wfplwf!FilterOidRequest+0x50
    8b582a10 880a7799 ndis!ndisFDoOidRequest+0x1a2
    8b582a3c 8810d602 ndis!ndisQueueRequestOnTop+0x224
    8b582ab0 8810d686 ndis!ndisMOidRequest+0x1dd
    8b582ac0 882a4646 ndis!NdisOidRequest+0xf
    8b582b7c 88254d1d tcpip!FlpNdisRequestUnderReference+0x7d
    8b582bb8 8825523b tcpip!FlpMulticastListSet+0xa1
    8b582bc4 88251a2e tcpip!FlpSerializedNdisDeleteGroupWorker+0xd
    8b582bec 83c4f460 tcpip!FlpSerializedNdisRequestWorkerRoutine+0x41
    8b582c00 83ab7aab nt!IopProcessWorkItem+0x23
    8b582c50 83c43f5e nt!ExpWorkerThread+0x10d
    8b582c90 83aeb219 nt!PspSystemThreadStartup+0x9e
    00000000 00000000 nt!KiThreadStartup+0x19

    Signed-off-by: yuchenlin<EMAIL_ADDRESS>
diff --git a/NetKVM/Common/ParaNdis-CX.cpp b/NetKVM/Common/ParaNdis-CX.cpp
index 69516dbb..68f98435 100644
--- a/NetKVM/Common/ParaNdis-CX.cpp
+++ b/NetKVM/Common/ParaNdis-CX.cpp
@@ -102,17 +102,24 @@ BOOLEAN CParaNdisCX::SendControlMessage(
         if (0 <= m_VirtQueue.AddBuf(sg, nOut, 1, (PVOID)1, NULL, 0))
         {
             UINT len;
+            void *p;
             m_Context->m_CxStateMachine.RegisterOutstandingItem();

             m_VirtQueue.Kick();
-            while (!m_VirtQueue.GetBuf(&len))
+            p = m_VirtQueue.GetBuf(&len);
+            for (int i = 0; i < 10 && !p; ++i)
             {
                 UINT interval = 1;
                 NdisStallExecution(interval);
+                p = m_VirtQueue.GetBuf(&len);
             }
             m_Context->m_CxStateMachine.UnregisterOutstandingItem();

-            if (len != sizeof(virtio_net_ctrl_ack))
+            if (!p)
+            {
+                DPrintf(0, "%s - ERROR: get_buf failed\n", __FUNCTION__);
+            }
+            else if (len != sizeof(virtio_net_ctrl_ack))
             {
                 DPrintf(0, "%s - ERROR: wrong len %d\n", __FUNCTION__, len);
             }

Implement a fine-tuned synchronous mechanism to prevent sending any event to QEMU when the device is not ready. For example: before SendControlMessage, we should check driver state by checking  pContext->bDeviceInitialized. However, if it does not have a fine-tuned synchronous mechanism, checking and setting bDeviceInitialized may run in any order. It still cause the problem. But, I don't know how many effort to do this.

I want to discuss this problem and the solution.
Thanks,
Yu-Chen Lin

Related #206
Include @sameehj @ladipro
Thanks,
Yu-Chen Lin

Hi, Yan
Sure, see #351
Thanks,
Yu-Chen Lin


Robot name exercise

here we go...

The stabby lambdas are not awesome where they are, but otherwise all is well.

agreed on the stabby lambdas... I wanted to just get rid of them, but decided to leave them in and move on.


(GH-264) Scaffold PRMlib

This deviates from the class diagram. I'd like to review this with folks on a call alongside the initial design.

Design doc updated


recognize CFBridgingRelease() when analyzing memory leaks

The following generates an erroneous memory leak warning:
NSMutableString *s = CFBridgingRelease(CFStringCreateMutableCopy(NULL, 0, CFSTR("foo")));

Hi,
I cannot reproduce your issue. I tried the following little example and I don't get the bug.
#import <Foundation/Foundation.h>
@interface A : NSObject
@end
@implementation	A
-(void) test {
NSMutableString *s = CFBridgingRelease(CFStringCreateMutableCopy(NULL, 0, CFSTR("foo")));
}
@end
Could you give me more context about how you called Infer?
Thanks,
Dulma

By checking this, it seems unlikely that that line produces a Memory Leak. We think that it may be an issue with line numbers. Sometimes they are wrong and we are working to fix them.


Artist object can't read foreign_id, even if requested

Looking through the code, I'm pretty sure that data must be coming back. It bounces an error if you try to limit without specifying an idspace in buckets arg. Maybe it's being filtered out by the "fix" function?

Juuuuuuust kidding. It's evidently filed under a node called "cache" rather than the root object


bluetooth not detect devices

hi, i've read the other topic about bluetooth detecting problems, but i still can't resolve it, i'm new in nativescript and mobile programming, this is my discovering function, it works until the enable  function, then the
onDiscovered: function (peripheral) {                                 console.log("Periperhal found with UUID: " + peripheral.UUID);                             }
don't return nothing,
where am i wrong?
exports.bt = function () {     bluetooth.isBluetoothEnabled().then(         function (enabled) {             console.log("Enabled? " + enabled);             if (!enabled) {                 bluetooth.enable().then(                     function(enabled) {                         console.log("attivato "+ enabled);                         bluetooth.startScanning({                             serviceUUIDs: [],                             seconds: 10,                             onDiscovered: function (peripheral) {                                 console.log("Periperhal found with UUID: " + peripheral.UUID);                             }                         }).then(function () {                             console.log("scanning complete ");                             console.log("Periperhal found with UUID: " + peripheral.UUID);                         }, function (err) {                             console.log("error while scanning: " + err);                         });                     }                 );             }         }     ); };
thank you for your help

sorry, now i've seen that this library is for BLE, my devices are BT standard, could this be the problem?

+1 is this library only for BLE? and Why?

Correct. This plugin is for BLE devices.

Is there any way I can modify this to work with Bluetooth standard devices?