Assistance in setting up

Describe in more detail how to set up the relationship with a sonoff or other relay to turn on the valve

Hope this helps..

No :(, pls rec video

?

Not getting the time to create a video but do try this resource which can help in understanding how Eve automations can created..

@Sergbmw how is your irrigation system being controlled? What model is your irrigation controller and how are you able to control it using Homebridge outside of this plugin?

@Sergbmw - don't know if you figured this out yet.. else if you are on discord could guide you over chat sometime..

@Sergbmw - don't know if you figured this out yet.. else if you are on discord could guide you over chat sometime..

Good day. I still did not understand how to tie automation to the irrigation relay

I’m having the same difficulty setting this up, I’m using a raw HomeKit setup and want to use an exposed smart switch as a trigger. Nothing exposed seems to be actionable against automation.  This would allow us to use the standard HomeKit automation workflow.

I’m having the same difficulty setting this up, I’m using a raw HomeKit setup and want to use an exposed smart switch as a trigger. Nothing exposed seems to be actionable against automation.  This would allow us to use the standard HomeKit automation workflow.

Please understand that Homekit is an ecosystem and Apple's Home App is just one of the ways of interacting with it. Other apps such as Home+ or Controller for Homekit or Eve do not alter or modify the basic Homekit ecosystem itself but offer a greater degree of control and automation using Homekit - and everything that Homekit offers! Essentially the Home App only exposes a subset of what Homekit itself offers. So to modify your statement - "Nothing exposed seems to be actionable against automation - from within Apple's Home App". Homekit does permit it and that is how many other apps such as Eve achieve it - this is the reason why an automation created by them would survive within your Homekit hub even if you were to uninstall the apps themselves after creating the automation! Its the Home app's limitation, not Homekit - and the design decision for this plugin(like many others) was not to be limited by the Home app - and utilise what Homekit offers. Surely the Home app will grow up further one day!🙂
That being said, the benefits of exposing irrigation valves far exceeds those of exposing switches - there is no way for a switch to show a countdown timer(within Homekit!) like a valve indicating when it will shut off! There would also be no way of showing remaining duration on the Irrigation system. Going further in the direction of aggregating total water consumption in Eve would also be non-intuitive using switches.
Please check out any of the apps referred above to unleash a little more of what Homekit offers compared to the Home app✌

If so many questions arise, maybe it is worth making a simpler instruction? Shoot a video of creating all automation using the example of one irrigation zone

@Sergbmw I’m willing to try to help with a video or write up, could you answer the questions I had from above? A generic video may not be of any use for you.
“how is your irrigation system being controlled? What model is your irrigation controller and how are you able to control it using Homebridge or Homekit outside of this plugin?”

I'm going to and create a video for a generic smart socket today..

@subcritical71 - thanks so much for offering to help with this!
@Sergbmw - have tried to do a video grab on my phone to show how I setup to trigger a smart socket with the zone-1 using the Eve app - you will probably need to keep a finger on "pause" as you watch it to make sure you are getting the steps right..
[Edit: seems the video file is too large(32MB) to be posted here so please check again if you are able to understand from the below instructions along with the screenshots above!]
The video starts by showing that the socket is not yet following the state of the valve in absence of the automations. This is followed by going into the Eve app to create a rule for which the triggers are "zone valve is running(on) and in use" - no conditions - to activate the scene where the smart socket is turned on. Similarly a rule is created to be triggered by the zone valve being "off" - no conditions - to activate the scene where the smart socket is turned off. Thats it - when the zone turns on, the rule activates the "on" scene and when the zone vale turns off it activates the scene where the socket is off. The video ends with the demonstration that the rules are working and the socket is being turned on and off following the state of the plugin exposed valve.
Hope this helps...

@subcritical71 - thanks so much for offering to help with this!
@Sergbmw - have tried to do a video grab on my phone to show how I setup to trigger a smart socket with the zone-1 using the Eve app - you will probably need to keep a finger on "pause" as you watch it to make sure you are getting the steps right..
[Edit: seems the video file is too large(32MB) to be posted here so please check again if you are able to understand from the below instructions along with the screenshots above!]
The video starts by showing that the socket is not yet following the state of the valve in absence of the automations. This is followed by going into the Eve app to create a rule for which the triggers are "zone valve is running(on) and in use" - no conditions - to activate the scene where the smart socket is turned on. Similarly a rule is created to be triggered by the zone valve being "off" - no conditions - to activate the scene where the smart socket is turned off. Thats it - when the zone turns on, the rule activates the "on" scene and when the zone vale turns off it activates the scene where the socket is off. The video ends with the demonstration that the rules are working and the socket is being turned on and off following the state of the plugin exposed valve.
Hope this helps...

Where is the video

@subcritical71 - thanks so much for offering to help with this!
@Sergbmw - have tried to do a video grab on my phone to show how I setup to trigger a smart socket with the zone-1 using the Eve app - you will probably need to keep a finger on "pause" as you watch it to make sure you are getting the steps right..
[Edit: seems the video file is too large(32MB) to be posted here so please check again if you are able to understand from the below instructions along with the screenshots above!]
The video starts by showing that the socket is not yet following the state of the valve in absence of the automations. This is followed by going into the Eve app to create a rule for which the triggers are "zone valve is running(on) and in use" - no conditions - to activate the scene where the smart socket is turned on. Similarly a rule is created to be triggered by the zone valve being "off" - no conditions - to activate the scene where the smart socket is turned off. Thats it - when the zone turns on, the rule activates the "on" scene and when the zone vale turns off it activates the scene where the socket is off. The video ends with the demonstration that the rules are working and the socket is being turned on and off following the state of the plugin exposed valve.
Hope this helps...

Where is the video

Like I said in the edit...

[Edit: seems the video file is too large(32MB) to be posted here so please check again if you are able to understand from the below instructions along with the screenshots above!]

add youtube

sorry.. the video is too crude. Please get on Discord and I can possibly help you there...

Did you manage to get it going?


Added deployment command to rebuild docs

I am setting up some proactive connections from upstream documentation repos to the central Mapzen repository. This change will cause CircleCI to trigger rebuilds on that repository, after merge of PR #233.

Also referenced from https://github.com/mapzen/mapzen-docs-generator/issues/224.

so whenever we release, docs is going to get rebuilt?

Yes, that is correct.


chore: helm chart release for driver v1.2.0

Signed-off-by: Anish Ramasekar<EMAIL_ADDRESS>
Reason for Change:

Requirements

[ ] squashed commits
[ ] included documentation
[ ] added unit tests and e2e tests (if applicable).

Issue Fixed:

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

[ ] Yes
[ ] No

If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers:

/azp run pr-e2e-azure

@aramase Could we please hold this PR until ARO changes are in?
Ref: #920

@aramase We are good with proceeding on this PR. Let's rerun chart release to get ARO changes as well.

Codecov Report

Merging #923 (7022c8c) into master (2eee8b7) will not change coverage.
The diff coverage is n/a.


Add ability to add grouped options in a select field.

Is your feature request related to a problem? Please describe.
Would like to have grouped categories in a select field but I cannot do so, so I guess it's a problem.
Describe the solution you'd like
Allow an option, such as  which houses  items.
Describe alternatives you've considered
Additional context
Add any other context or screenshots about the feature request here.

An example.

It's a good idea to see if it can be added for a next update, thank you very much for the contribution

The group item functionality has been added in the latest version, thank you very much for the contribution and the wait
https://lusaxweb.github.io/vuesax/components/selects.html#is-selected-item


Update chaos-studio-tutorial-aks-portal.md

Added the limitation that we have in Chaos Mesh fault.

@prasha-microsoft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@johnkemnetz
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Document enlarging Particles visibility AABB when they cast shadows

Note: I opened this directly against the 3.2 branch as GPU particles are temporarily absent from the master branch.

This closes https://github.com/godotengine/godot/issues/17267. Adjusting the visibility AABB to take shadows into account is probably too difficult (and maybe inefficient), so documenting this behavior should work well enough.

Thanks!


TypeScript class extending Set returns error.

When trying to extend a set to allow unique custom objects, I found an error stating:
Set: 'this' is not a Set object
Recreation:
https://jsfiddle.net/9xknubyj/

I just tried running that jsfiddle, and I didn't get any errors. If I add the lines
var m = new MySet<string>();
m.add("foo");

then I do get an error, but the error is to do with the use of super (or more specifically, _super once converted to javascript) in the add method, which looks correct to me. Where were you seeing the error you report? What version of Edge are you using, and does the error show up in the console when running the exact jsfiddle you link to?

I got the error when running the exact piece of code in a separate project, or when running it through the typescript playground(http://www.typescriptlang.org/play/). The error doesn't show up on IE11, which is why I posted the issue here.
Edge version is 15, the error you got is showing up in the console of the Jsfiddle, but I'm not sure what kind of compilers they are using.

Ah I see; using the typescript playground I do get the error that you report. However, I also get a similar bug if I try to repeat the same thing using v8 (using node 10.6.0 as a host):

TypeError: Constructor Set requires 'new'

Do you see this code working as you expect anywhere? This may be a bug in the way typescript is converting to javascript?

Also to clarify: you say that it works in ie11, but I believe that will be because ie11 doesn't have full ES6 support, and so ie11 doesn't have the notion of classes. Edge does have ES6 support which changes the behavior of things like Set and Map, and that may be what is causing the discrepancy here.

The same error occurs when using the TypeScript Playground in Chrome.  The behavior is unfortunately correct per the ECMAScript specification.
The issue appears to be that the transpiled constructor does:
_super.apply(this, arguments)

This is not legal for class constructors (they must be called using either new or Reflect.construct()).
Setting the TS compiler to target ES6 or higher would probably fix it.  Sadly the Playground doesn't seem to have an option to do that.
Long story short: ES5-ified classes cannot extend native ES6 classes.
See also: https://github.com/babel/babel/issues/6299 (this issue bit me in Babel too)

If I run the code sample using tsc set to es5 mode then I get the following build errors:
> npx tsc
src/index.ts:16:40 - error TS2304: Cannot find name 'Set'.

16 class MySet<T extends SetItem> extends Set<T> {
                                          ~~~

src/index.ts:19:14 - error TS2339: Property 'forEach' does not exist on type 'MySet<T>'.

19         this.forEach(item => {
                ~~~~~~~

src/index.ts:19:22 - error TS7006: Parameter 'item' implicitly has an 'any' type.

19         this.forEach(item => {
                        ~~~~

If I switch my tsconfig.json to es2015 then I get correct output which uses class syntax.  Since Set wasn't introduced until ES2015, you'll likely need to take another approach.


chore: upgrade internals to PNPM 9

This PR upgrades to PNPM 9

Implemented in #57 (no idea why this one caused an Nx issue)


Update imagestreams

Use groupified apiVersion
Add openj9 tags to ppc64le imagestreams
Add openjdk-17-ubi8 tags to imagestreams
Add java-runtime imagestream

Thanks for submitting your Pull Request!
Please make sure your PR meets the following requirements:

[ ] Pull Request title is properly formatted: [CLOUD-XYA] Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Attached commits represent units of work and are properly formatted
[x] You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
[x] Every commit contains Signed-off-by: Your Name<EMAIL_ADDRESS>- use git commit -s

Thanks @yselkowitz! This will collide with #248 to some extent so we'll figure out how best to address that.

Hey @jhuttana : whilst we were working on #248 , this PR has arrived. I think it covers much of what you were working on in #248 but there might be some things missing, there is also extra things we need (such as the apiVersion changes). Could you give it a review please? My hunch is this is likely good and if there are template changes we need to make not in this PR, it might make sense to leave those in #248 and rebase that PR on top of this one. WDYT?

Small notes from a quick review (more to come)

It would be nice to have a ubi8-openjdk-17 imageStream in  community-image-streams.
and ubi8-openjdk-17-runtime in runtime-image-streams

@yselkowitz is there any plan to automatically consume the runtime imagestreams somewhere? I think we might prefer they are not displayed by default in a fresh OpenShift install, to avoid cluttering it up too much. I think we (jdk team) need to review what shows up in what context at some point

@yselkowitz oh rats I just realised this was targetting develop. Openshift/library consume from the release branch. Would you mind raising a PR against release please?


getVariancesWorker slow when comparing large types

Bug Report
I'm often seeing slowdowns in type checking when getVariancesWorker is involved. In some cases, I can trick TypeScript into avoiding getVariancesWorker and type checking is significantly faster in those cases. It seems to be related to the size of the type, but I have not been able to figure out the exact issue or the reason why getVariancesWorker is called.
🔎 Search Terms
getVariancesWorker inferFromTypes inferFromTypeArgs checkExpression
🕗 Version & Regression Information
Types slowed down for us since 4.3, see https://github.com/microsoft/TypeScript/issues/47052 for more info.
⏯ Playground Link
Slow example. 134ms
trace-01.zip
Fast example. 41ms
trace-02.zip
This is the diff:

Note that I've not been able to eliminate all calls to getVariancesWorker in the second example - this is because of the estypes namespace which serves as a stub. The actual example on my laptop doesn't need this stub, and there is no (significant) work being done in getVariancesWorker in that case.
I've also seen it happen when awaiting Promise.all (vs awaiting a single promise).
🙁 Actual behavior
Type checking is significantly slower in some cases even though the code is equivalent.
🙂 Expected behavior
Type checking should be roughly as fast in both cases.

Doing some more digging, it looks like structuredTypeRelatedTo is also doing a lot of work in cases where it's not necessarily being called by getVariancesWorker. Fwiw, I think I disabled spans for structuredTypeRelatedTo in the attached traces to eliminate the possibility of those spans adding CPU overhead that would not be there in normal usage.

Spending a bunch of time in getVariancesWorker / structuredTypeRelatedTo is a routine finding in perf investigations; a very large proportion of the work that needs to be done to check a program is represented by these functions. Without an abnormal finding or some clearly-identifiable skippable thing, this isn't indicative of a defect or even an opportunity.

@RyanCavanaugh what I don't get is why that work disappears if I define a type with a statically known property to pick from.

@RyanCavanaugh can you clarify what info you need? I've attached traces and playground examples that demonstrate a performance slowdown which IMHO is unexpected given the very minimal difference in the types.

@dgieselaar it's always going to be possible to construct two programs a small edit distance apart where once checks much more quickly than the other, since any work-skipping mechanism we have is going to be defeatable one way or another. If you can construct a small example where there's an "obviously wrong" amount of extra work happening we can investigate that, but as-is this isn't really actionable -- different inputs will always potentially have different performance characteristics.

@RyanCavanaugh I'm not sure if I can provide more info than what's already given. If that's not enough, feel free to close it. I will say I'm a bit stumped why the provided example is much faster when wrapping a mapped type in a type with a statically known property and why that does not warrant an investigation :)

why that does not warrant an investigation :)

With infinite time, absolutely I'd love to look at it. We just have to prioritize where we spend investigatory time.


AssertionError: Torch not compiled with CUDA enabled

Pretty required to have this capability as at least a warning. I know HOW to do this, but it should at least warn. The demo isn't very verbose nor is it very helpful in terms of messaging, so I had to probe a bit before I figured out why it wasn't rendering up to speed.
When rendering only with the cpu it should automatically warn the user and there should definitely be a progress bar.

As a quick fix, install the PyTorch version that matches your graphics card driver. Here is a quick guide:
Find the Proper CUDA Version
Open Command Prompt and type nvidia-smi.
It should display something like:
NVIDIA-SMI <your-smi-version>    Driver Version: <your_driver_version>  CUDA Version: <your_cuda_version>
in the first line.
Install PyTorch withh CUDA
Next, install PyTorch with the correct CUDA version. Visit https://pytorch.org/get-started/locally/ and copy the command that matches your CUDA version from the previous step.
Verify that CUDA is Supported by PyTorch:
import torch
print(torch.cuda.is_available())  # Should return True
print(torch.cuda.get_device_name(0))  # Should return 'your graphics card name

Hope that helps!


Added Documentation for Auto Key Cipher

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Added Documentation for Auto Key Cipher Algorithm and added code in C++ and Java.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes: #481

@ankitaggarwal23  Can I know the reason..?

You don't even tag any of the mentors for review and at the end, you are saying for review. The mistake is from your side. GSSoc period is now over. I will review this Don't worry.

Thanks for the review. @ankitaggarwal23
Just would like to mention that i'm not contributing as a part of GSSOC. It's a normal contribution.


Initial api reference commit

This creates an api reference on livepeer.com and adds all the reference copy. And API reference menu item should be at the top of the /docs part of the site.
I think everything is formatted right, and I didn't break anything. But it would be good for an engineer to make sure.

One "bug" that exists on the guides section and now also exists on the API reference is that you can't access what's in the sidebar menu on mobile. I don't know how to fix this myself.

One "bug" that exists on the guides section and now also exists on the API reference is that you can't access what's in the sidebar menu on mobile. I don't know how to fix this myself.

ah yeah there used to be a dropdown in the header on mobile. It might be commented out. I'll fix.

@wohlner I've fixed the mobile situation. That can be reviewed now 👍

Looks like the caret dropdown loads the /docs menu (just api, guides) instead of each sections' menu...the all the pages in guides and all the pages in api reference.
Also, can you please make the upside down caret purple and/or bold to help draw the eye to it.

@wohlner can you do a hard refresh? You should see a dropdown with all the pages in the side bar.

Also, can you please make the upside down caret purple and/or bold to help draw the eye to it.

@wohlner I pushed up a change that makes the carrot bigger and more bold to draw more attention to it.

Looks great, @adamsoffer. Can you remove the caret next to the logo? It repeats what is in the hamburger menu. That is what i was looking at with my previous comment and it got me confused.

Can you remove the caret next to the logo?
@wohlner All set

@adamsoffer let's get this deployed to production (stage first of course). Everything looks great!


'specify' option refers to stylesheet location in sass directory

I hope somebody here can enlighten me in any way regarding the 'specify:' option.
What I'm trying to achieve is the following:
I want to create a separate task, to only compile a subset of my .sass files.
The directory structure is as follows:
project-directory
|- sass
    |- subfolder a
    |- subfolder b
        |- modules
            |- …
        |- screen.sass
        |- all-old-ie.sass

When I'm developing, I just want to compile the screen.sass, not the all-old-ie.sass
Therefore, my gruntfile contains the following:
// compile sass code
compass: {
    // general options
    options: {
        bundleExec: true,
        cssDir: project.codeDir + 'css',
        fontsPath: project.codeDir + 'fonts',
        httpPath: '/',
        imagesPath: project.codeDir + 'img',
        javascriptsPath: project.codeDir + 'js',
        sassDir: project.sourceDir + 'sass',
        raw: 'preferred_syntax=:sass\n'
    },
    dev: {
        options: {
            sourcemap: true,
            environment: 'development',
            outputStyle: 'expanded'
        }
    },
    'dev-no-ie': {
        options: {
            sourcemap: true,
            environment: 'development',
            outputStyle: 'expanded',
            specify: '**/screen.sass'
        }
    },
    dist: {
        options: {
            force: true,
            environment: 'production',
            outputStyle: 'compressed'
        }
    }
}

the error message:
when trying to compile It only states:

Individual stylesheets must be in the sass directory.

I tried different patterns in the specify option, but as of now, it seems that files are found matching the pattern, but the error message does not enlighten me as much as I would hope.
Because, the screen.sass resides in a subfolder of the sassDir.

Anyway, I hope someone can help me or point me to a solution, or tell me if it's actually a bug. Thanks a lot for your time, any help is greatly appreciated.

Duplicate of https://github.com/gruntjs/grunt-contrib-compass/issues/223


fix persist txn record of single replica load and ccr ingestion

Otherwise txn would be dropped when a be reboots.
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Updated daniel.csv

Screwed up, wanted to merge daniel.csv commit to main and demo.html to test. How to unscrew?? :)


Added two more LeetCode problems

Added the  Filename as
#231-Power_Of_Two.cpp
#342-Power_Of_Four.cpp
Changes Proposed  in the PR. Describe
Added the solutions to problems #231 and #342.
Runtime and memory usage is mentioned in the commits.
I am assuring that the file is not redundant and making the Pull Request

[x] Yes

Do let me know if there are any issues.

Please write full code with main functions too with an explanation.

Hey @mansikagrawal, LeetCode solutions do not require the main function as it is already provided by the IDE. One is just required to write the solution inside the Solution class provided by the LeetCode IDE. The other solutions that have been submitted to the repo follow a similar suite.

I've done the requested changes. Do check and let me know.

Works. Please squash the commits .

Commits have been squashed.


Smoke Android layers

This series allows us to package validation layers with Smoke and enable them on Android.
Since Smoke only exists in this repo for validating that layers are working correctly, the layers are packaged into the APK unconditionally.
This change is primarily driven by downstream needs for using Smoke to verify that persistently mapped buffer support is working with vktrace.
The biggest change is a direct port of arg parsing from Hologram that didn't make the original cutoff.
There should be no change in desktop functionality.

I don't know of any documentation Courtney.  Here's a link to the parsing code though.
https://github.com/KhronosGroup/Vulkan-LoaderAndValidationLayers/blob/master/demos/smoke/Game.h#L113
I've only tested --validate, but the other arguments should get parsed in the same way.
I'll add a note to BUILD.md showing how to invoke Smoke with --validate (just like Cube).

cherry-picked to master


Follower Thank You Page, backend

After clicking on the Call button, follower is taken to a thank you page that has the message from the influencer and opportunity to share their call details on FB/Twitter
BE needs to return:

the Thank You message from the influencer
Pre-prepped post/tweet templates for sharing

@kate-rose should social media call to action on the thank you page be MVP or nah?

Hi there -
Just checking in because I saw these notifications. I worked on hellogov
back in 2016. Is someone picking it up again :)?
On Mon, Jul 1, 2019 at 8:55 PM Sarah Port<EMAIL_ADDRESS>wrote:

@kate-rose https://github.com/kate-rose should social media call to
action on the thank you page be MVP or nah?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/helloGov/webapp/issues/30?email_source=notifications&email_token=AAOXLJK2VQLA5J5B5CFBT3LP5LGS7A5CNFSM4C5JA37KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY77LGA#issuecomment-507508120,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOXLJN63OCLRF6RRLKD2BLP5LGS7ANCNFSM4C5JA37A
.

--
Bill Kee

@billkee Sorry for the suuuuper delayed reply. This project has been under development at Hack For LA for several months now. We're working to get it stood up as an MVP that orgs can start using soon! Let me or @kate-rose know if you want more information about any of this :)


LoadSupport should provide a means of detecting 'stale' loads

It would be nice if LoadSupport could detect if a call to doLoadAsync()  was the latest call made, or if there was a more recent call made, so applications can check whether they should throw away the results of whatever asynchronous requests they may have made in the doLoadAsync()
In discussing with Lee, LoadSupport could keep track of the most recent loadSpec that was passed to a doLoadAsync and potentially install a function on the loadSpec which would make it easy to check if the current closure is the latest call or not.

Thinking a flag of the form loadSpec.isObsolete or loadSpec.isCurrent that could be checked by async processes to early out, if another load has been requested.

Thinking a flag of the form loadSpec.isObsolete or loadSpec.isCurrent that could be checked by async processes to early out, if another load has been requested.

Seems like you almost want to know if
a) a later load has been kicked off or
b) a later load has completed.
In the former case, you might want to stop your loading.  In the latter case, you definitely want to stop your loading!

Seems like you almost want to know if
a) a later load has been kicked off or
b) a later load has completed.
In the former case, you might want to stop your loading.  In the latter case, you definitely want to stop your loading!

I'll do this as soon as the latest Hoist v38 refactor is in.

I'll do this as soon as the latest Hoist v38 refactor is in.


Form  label is present,  no content

(Also on homepage)

Empty form label
What It Means
A form label is present, but does not contain any content.
Why It Matters
A  element that is associated to a form control but does not contain text will not present any information about the form control to the user.
How to Fix It
Ensure that the form label contains text that describes the function of the associated form control. > Labels are not required for image, submit, reset, button, or hidden form controls. If a label is not necessary visually, a descriptive title attribute may be added to the form control.

This is the toggle button for the mobile navigation. To make the hamburger animated, it was made with three stacked spans. I'm not quite sure how we could incorporate text here that is only visible to screen readers... 🤔

You could use aria-hidden=true this hides the element for a screen reader

@shiffman  @vimkat  This is issue has been resolved and can be closed.
The button has appropriate aria-label and aria-role. When tested with VoiceOver/NVDA it announces "Open navigation menu"
<button id="menu-button" class="mobilenav-toggle" tabindex="0" aria-haspopup="true" aria-controls="global-nav" aria-label="Open navigation menu">    </button>

Thank you!!!


Unable to activate microservices in microk8s cluster

Describe the bug
A clear and concise description of what the bug is.
Unable to start any of the micro services within kubernetes microk8s cluster. Pods appear to be running and identified in the front end, however toggling micro services on within the admin -> system page does not work.
Affected Component(s)

[X] Back-End (Database, Micro services, Containers, etc)
[ ] Front-End (Flask, API, etc)
[ ] Docs
[ ] Build System

To Reproduce
Steps to reproduce the behavior:

Go to 'Admin -> System page'
Click on 'Attempt to enable one of the bgp micro services'
Nothing occurs, micro service remains in-active

Expected behavior
Micro service enables and feed begins to ingest
Screenshots

System (please complete the following information):

OS: Ubuntu 20.04
Browser: Firefox
Version 94.0.2

Additional context
Logs do not show any errors related to starting the micro services.

@Surphix have you tried enabling the detection service to see if that works? Also, have you properly configured RIPE RIS in the conf file? If not configured, the base service is running but it does not actually consume data (and appears off in the UI). Note that in general unconfigured monitor microservices appear as off until they are both configured and enabled in the UI.
@CuriouzK0d3r do you have a local deployment to check if e.g., detection or mitigation services can be properly enabled and run? The autostarter microservice is the one that takes care of enabling/disabling the actual service workers of other services: https://github.com/FORTH-ICS-INSPIRE/artemis/blob/5b12a9d79a2785d53182244b2eec0d30f87aad96/backend-services/autostarter/core/autostarter.py#L262

@Surphix saw that the issue is closed, was it a missing configuration problem? Did you active the microservices after enabling them on the conf file?

@vkotronis I uninstalled and reinstalled the chart, updated the config and it seemed to work after that. Thanks!

@vkotronis I uninstalled and reinstalled the chart, updated the config and it seemed to work after that. Thanks!

Great, thanks for reporting this @Surphix and I am glad that it works as expected.


[Feat.] Allow Remote Config Backup+Restore

Description of the new feature/enhancement

I use many different machines on a daily basis, for example my work laptop, my development laptop, my testing laptop, my private laptop, my private Tower etc.
On each Device I need to import the full Config of Windows Terminal. A way to synchronise all the Terminal Config files would be insanely great and would really hype up this terminal soo much more!!!
PowerToys already has a Backup option, could  similar thing be done for windows-terminal?
I'd even consider github integration a nice idea, as this would allow for easy fetching from a raw.githubusercontent link.
(I wouldn't mind the config being public).
Also something like fully remote fetching would be nice, for example the machine fetches the config from remote github repository and on no internet it fallbacks to a cached version.
Proposed technical implementation details (optional)
The Github Integration to the initial Backup + Restore would be like a nice-to-have addon to keep in mine, but could also be done through scripts.

Add a way to Backup and restore the Config

Backup to a specfic Path
Restore from Path

Github Integration would be nice-to-have

User can enter a URL to a github file, where config is fetched from.
User can connect windows terminal to github account, select a repo and the accoring file, where windows terminal pushes changes to.

Thanks for filing! Marking this as a /dup of #2933. I copied over your post to https://github.com/microsoft/terminal/issues/2933#issuecomment-2207424906 too for easier tracking 😊

Thanks for filing! Marking this as a /dup of #2933. I copied over your post to https://github.com/microsoft/terminal/issues/2933#issuecomment-2207424906 too for easier tracking 😊

Hi Carlos
Thanks for the kind response 😊
Apologies for filing a dup, but I could not find that report beforehand.

Apologies for filing a dup, but I could not find that report beforehand.

Oh that’s not a problem! With 17,000 issues in our haystack it wouldn’t be fair for us to expect everyone to always find the needle :)


bytes: TrimSpace returns non-nil byte slice when given all-space input

What version of Go are you using (go version)?

$ go version
go version devel +501632339f Mon Mar 25 18:16:51 2019 +0000 linux/amd64

Does this issue reproduce with the latest release?
No. This is a regression from go1.12.1 and earlier.
package main

import (
	"bytes"
	"fmt"
)

func main() {
	fmt.Println(bytes.TrimSpace([]byte("   ")) == nil)
}

On tip (1.13) it outputs false while on previous versions it prints true. This was caused by  https://golang.org/cl/152917 because it no longer uses bytes.TrimFunc(s, unicode.IsSpace) which calls bytes.TrimLeftFunc on an empty string, causing it to return nil due to indexFunc returning -1:
// TrimFunc returns a subslice of s by slicing off all leading and trailing
// UTF-8-encoded code points c that satisfy f(c).
func TrimFunc(s []byte, f func(r rune) bool) []byte {
	return TrimRightFunc(TrimLeftFunc(s, f), f)
}

// TrimLeftFunc treats s as UTF-8-encoded bytes and returns a subslice of s by slicing off
// all leading UTF-8-encoded code points c that satisfy f(c).
func TrimLeftFunc(s []byte, f func(r rune) bool) []byte {
	i := indexFunc(s, f, false)
	if i == -1 {
		return nil
	}
	return s[i:]
}

// TrimRightFunc returns a subslice of s by slicing off all trailing
// UTF-8-encoded code points c that satisfy f(c).
func TrimRightFunc(s []byte, f func(r rune) bool) []byte {
	i := lastIndexFunc(s, f, false)
	if i >= 0 && s[i] >= utf8.RuneSelf {
		_, wid := utf8.DecodeRune(s[i:])
		i += wid
	} else {
		i++
	}
	return s[0:i]
}

It seems weird that TrimRightFunc returns a non-nil slice every time, while TrimLeftFunc returns a nil slice if it can't find a rune that satisfies f(c), but I imagine changing that behavior would cause more breakage in weird places.
@benhoyt @josharian

This is unfortunate, because "returns a subslice of s" to me suggests it returns an empty slice, not nil, on an all-space input. (Might not be worth breaking, though.)

This is unfortunate, indeed, but we should try to preserve the existing behavior. From a glance, looks like we just need to replace return s[start:stop] with r := s[start:stop]; if len(r) == 0 { r = nil }; return r. @benhoyt are you interested in following up on this? I.e. investigate more deeply, write some tests, and send a CL?

@josharian Yes, I'm interested in following up. What timeline are we looking at? I can probably do this in the next 7-10 days.
Are we definitely sure we want to preserve backwards compatibility here, and the weirdness? Or move to always returning a non-nil slice?

Sooner is better of course, but we have at least a month worst case. If it is remotely reasonable, let’s remain perfectly backwards compatible, weird or no.

Spoke with @ianlancetaylor about this and while it is odd behavior and people shouldn't be relying on these functions returning a nil slice in practice, it's not worth the breakage of code that does.

Submitted. While I was in there, I noticed there were no tests for TrimLeftFunc or TrimRightFunc (though there are a couple of examples for each). It seems like there should be tests for these, including the nil-returning case, so if you folks agree I can submit another CL for that (should I open a new issue as well?).

Some tests would be nice, yes, in a separate CL.  No need to open an issue.  Thanks.


Local development requires internet connection

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I am not sure if it is just not Supported anymore but local development is no longer possible if not connected to a network.
In an older supabase version it was still possible, I worked several times traveling without a connection.
To Reproduce
https://github.com/supabase/cli/assets/52098754/8c9bce78-af38-44c3-a07e-905e3defecd2
Expected behavior
Development should also be possible without internet connection
Screenshots

System information
I am on the latest supabase version 1.58.1

Having the same issue here for Monacoeditor

Also having this issue. I can't even visit the local dashboard at http://localhost:54323/ without an active internet connection.

This should have been fixed in latest release. Feel free to reopen if you are still unable to load the dashboard offline.


Updated code from 1.18.1 to 18.2

I have updated my structure generation from 1.18.1 to 18.2 according to this github.
I am using fabric.
Now none of my structures generate, and the console does not throw any notable exceptions, does anyone know what the issue could be?

My code: https://pastebin.com/xfU2CEei

Does /locate command show the structures? Are the structures spawning at y = 0 or high up in the sky? Does the log show any errors at all? (Not exceptions. Lines that say any json file was failed to be parsed). And of course, did you make sure there’s no typos in the json files and where you call the start pool?

Ah, found my mistake >-<
Thanks for helping me realize that I did not see that large parts of the code is now json files. I totally forgot to add most of them.
thank you for your time, and your tutorial is helping a lot!


[FRONTEND][AOT][IMPROVEMENT]

[FRONTEND][AOT][IMPROVEMENT]
Add automatic AOT generation of kernels by tracing JITFunctions

Description
Currently AOT requires manual specification of kernel signature and params.  The proposed additions will automatically generate AOT kernels by extending the existing triton compilation pipeline (invoked upon triton.runtime.jit.JITFunction.run) to allow for kernel "tracing" and aot kernel generation.
All a user needs to do is to pass an additional trace kwarg when calling a jitted function.  Under the hood, the extended triton.runtime.jit.JITFunction.run will run the aot pipeline to generate kernel headers / sources and linked dispatcher library after the existing jit pipeline runs.
If a user wants to generate multiple kernel configurations for different dtypes / specializations, a KernelTracer interface is provided that enables generation of multiple kernels and a linked dispatcher library similar to the existing triton.tools.link script.
In summary, these tools aim to increase adoption of triton by easing the generation of tt-optimized kernels that can be used for downstream deployment / binding to other frameworks & languages through a unified interface.
Usage
The 3 primary abstractions are AOTCompiler, AOTLinker, and KernelTracer located in the following submodules of the newly introduced triton.tools.aot module.

triton.tools.aot.compiler

This module encapsulates the functionality of triton.tools.compile, split across  codegen, template, and compiler sub-modules.

codegen.py: types and functions for generating kernel headers and sources from a jitted function.

Notably AOTCompilerParamsBuilder is an abstraction for building the params needed to hydrate the compiler templates in templates.
AOT_C_CUDA_ParamsBuilder is a concrete instantiation for the existing aot compile.{h,c} templates specific for CUDA.

compiler.py contains the AOT_Compiler abstract class and the concrete AOT_C_CUDA_Compiler, which delegates to AOT_C_CUDA_ParamsBuilder to generate kernel headers and sources given a jitted function and arguments typically passed to triton.runtime.jit.JITFunction.run.
Usage:

  from triton.runtime.jit import JITFunction
  from triton.tools.aot.compiler import AOT_C_CUDA_Compiler as AOTCompiler, AOTCompilationResult, JITCompileArgs

  kernel_path = "path/to/kernel.py"
  kernel_name = "name of kernel in kernel.py"

  jit_args = JITCompileArgs(
      signature=signature,
      constants=constants,
      configs=configs,
      **kwargs,
  )
  jit_fn = JITFunction(kernel_path)

  compiler = AOTCompiler(
      kernel_name=kernel_name,
      jit_args=jit_args,
      jit_fn=jit_fn,
      save_dir=codegen_dir,
  )

  compiled_result: AOTCompilationResult = compiler.generate()

jit_args is a dataclass that packs the arguments to JITFunction.run such as signature, constants, configs, etc.  In the traced path, these args are automatically constructed by the existing jit pipeline and passed to the AOTCompiler after the normal jit pipeline runs.
AOTCompilationResult is a dataclass that packs the results of the compilation   pipeline: kernel headers, sources, and the compiler params used to generate them.

Note that JITFunction can now take a path as an argument in addition to the python function object (per usual path).  Under the hood, JITFunction now checks for the type of argument and dynamically loads the kernel (using importlib) if a path is passed.

triton.tools.aot.linker

Similar to the structure and intent of triton.aot.compiler, the triton.tools.link module has submodules codegen, templates, and linker that implement the functionality of triton.tools.link.
codegen.py contains the HeaderGenerator and SourceGenerator interfaces for generating the header and source files for the linked dispatcher library.  C_CUDA_HeaderGenerator and C_CUDA_SourceGenerator are concrete implementations specific to the C CUDA template per triton.tools.link.
linker.py contains the AOTLinker abstract class and AOT_C_CUDA_LINKER concrete implementation comprised of C_CUDA_HeaderGenerator and C_CUDA_SourceGenerator, as well as a C_CUDA_HeaderParser for generating the metadata needed for header and source generation from the compiled kernels.
Usage:

 from triton.tools.aot.linker import AOT_C_CUDA_Linkeer as AOTLinker, AOTLinkerResult

 headers = ["path/to/kernel1.h", "path/to/kernel2.h"]

 linker = AOTLinker(
      kernel_name=kernel_name,
      headers=headers,
      save_dir=codegen_dir,
  )

  linker_result: AOTLinkerResult = linker.generate()

headers are the headers generated from the previous AOTCompiler step; AOTLinkerResult is a simple dataclass with the generated header and source.

triton.tools.aot.tracing

KernelTracer is the core abstraction for tracing jitted functions composed of the prior AOTCompiler and AOTLinker abstractions.

A user would need to implement a concrete KernelTracer class specific for the kernel of interest. MatMulKernelTracer is one such implementation, tailored to the matmul kernel tested in triton.test.unit.tools.test_aot.py with some minor modifications (remove additional matrix multiplication at the end of the kernel).

Usage:

  from triton.tools.aot.tracing import TraceConfig, MatMulConfig, MatMulKernelTracer

  BLOCK_M = BLOCK_N = BLOCK_K = 16

  matmul_config = MatMulConfig(
              dtype_in=dtype_in,
              dtype_out=dtype_out,
              BLOCK_M=BLOCK_M,
              BLOCK_N=BLOCK_N,
              BLOCK_K=BLOCK_K,
          )

  do_not_specialize = ["stride_cm", "stride_am"]

  trace_config = TraceConfig(
      do_not_specialize=do_not_specialize,
      num_warps=1,
      trace_dir="path/to/output/dir", # where to save the generated kernels
      trace_grid=("M / 16", "N / 16", 1), # dynamic grid to use for the traced kernels
  )

  kernel_dir = "path/to/kernels"

  matmul_config2 = ...
  trace_config2 = ...

  matmul_tracer = MatMulKernelTracer(kernel_dir)

  traces, *_ = matmul_tracer.trace(
      kernel_configs=matmul_configs, trace_configs=trace_configs
  )

traces are the AOTCompilationResult per above that contain the generated kernel headers, sources, and compiler params (the remaining objects returned by trace are optional checks).
The next step is to then pass the resulting headers to AOTLinker to generate the linked dispatcher library.
Since jit pipeline bind the launch grid to the function call args (resulting in a static grid for the traced kernel), users can pass an additional kwarg trace_grid   to specify a dynamic grid in the same form as the aot scripts.
E.g., if a user were to pass
grid = lambda meta: (triton.cdiv(M / meta["BLOCK_SIZE"],
triton.cdiv(N / meta["BLOCK_SIZE"], 1)

then the traced kernels will be generated with a grid bound to the values of M and N at the function call (assuming BLOCK_SIZE is a constexpr).  If M = N = BLOCK_SIZE = 16, then the grid will be bound as (1, 1, 1) in the generated kernel trace, meaning the launch params for the kernel template would be filled in as gX = 1, gY = 1, gZ = 1.
Instead with trace_grid, a grid can be specified as
trace_grid = ("M / 16", "N / 16", 1)

The generated kernels will have now have gX = M / 16, gY = N / 16, gZ = 1, which allows for various grid configurations for different values of M and N.

Note that trace_grid is an additional kwarg that can be passed in at JITFunction invocation -- this does not interfere with existing grid handling or the rest of the jit pipeline.  It is only used if tracing is enabled when running JITFunction.run and passed to the AOTCompiler, which runs in addition / separate from the existing jit pipeline.

Examples
Specific examples of usage can be seen in triton.test.unit.tools.test_aot_tracing.py.

Of particular relevance are the MatMulCodegen and MatMulTrace test suites, which test the code generation and end-to-end tracing pipeline, respectively, for the matmul signatures and kernel configs in test_aot.py.
MatMulCodegen: tests the AOTCompiler and AOTLinker components, both demonstrating usage as well as checking that kernel headers / source and linked headers / source match line-by-line those generated by existing aot scripts when fed the same parameters.
MatMulTrace: tests MatMulKernelTracer which implements the end-to-end pipeline of tracing a jitted function and generating kernels and linked code from the traced function using the AOTCompiler and AOTLinker components.

Since the aot scripts only handle divisible_by_16 and equal_to_1 specializations, whereas the jit pipeline contains additional hints such as  divisible_by_8 and ids_of_folded_args, particular care is taken to ensure that the tests of the kernel headers / sources and linked headers / sources take this into account when comparing against kernels generated using the original scripts.
Any of the codegen'ed header / source lines that depend on these specializations such as kernel_name and triton_kernel_name won't align with those from the script.
See the inline documentation of MatMulTrace for more details.

Sorry, but I don't think we should merge this. There is already a massive amount of extra complexity in Triton right now, and adding another 2500+ lines is going to add a substantial maintenance burden. I recommend you try to make it work out-of-tree so people can still benefit from this utility without it being maintained in the core repository.


Release Google.Cloud.Iap.V1 version 2.6.0

Changes in this release:
New features

Change netstandard2.1 target to netstandard2.0 (commit 82bea85)

Created release for Google.Cloud.Iap.V1-2.6.0


Add stack trace to all current suspensions on early exit

Example output: https://gist.github.com/kelunik/eb5649f872755960a4f53b312d8f00fd

This is a great idea, thanks for this.

So to be clear, this only happens when the event loop ends while there are still fibers around with potential work? And this visualise each stack trace for those fibers?

Yup


Branch with a lighter frontend

React makes the frontend installation very slow on VM and makes testing awkward.
Make a branch with a lighter frontend setup to speed up.

For the deployment to the VM, this was solved by running npm install and npm run build on the localhost and copying the build folder to the VM.


Document external requirements thoroughly

As shown in #51, Blizzardry could do with some thorough documentation on what external requirements are needed and how to set these up on Mac OSX, Linux and Windows.

Covered by cafb0c340b20563afbf7c7e458ed17916c75c0a9.


Update package.json

Update node-sass to 5.0.0, to support Node 15.

Thanks all, close this to focus efforts on #802 which is more rounded at this point in time.


Huge bandwidth problem of current implementation and Possible solution

Problem:

In the current implementation, each block needs to do election, so confirm ack messages from about  30 representatives  are rebroadcast and thare are about 1000 nodes. So about 1800 (30 * 2 * 1000^0.5 )messages need to be sent for each block election, bandwidth cost is about 4Mbps (1800 * 256byte per block).

The network will become more decentralized in the future, representatives need to be rebroadcast will obviously increase (eg. 30 -> 300), bandwidth cost is about 40Mbps per block, that`s terrible.

Solution:
1.If the block not sent to my account, just do election when fork found(new fork notify message need to be rebroadcast immediately, but current election message(confirm ack message) need to send slowly for bandwidth saving, and if account is in election , all the following blocks from the account are dropped)
2.If the block is sent to my account, do P2P confirm requests to representatives myself only before receive.
3.There is another benefit from the solution, because for representatives, it is no need to do re-signature, just rebroadcast blocks immediately when first see. I think transactions (send and receive) will complete in less than 3 seconds.

Unfortunately we have plans that will require votes for all blocks even when there isn't a fork. One of those is sentinel mode, where blocks will need confirmation before entering the ledger.
The recent enormous bandwidth increase was due to a bug and has sense been fixed. There are other options to decrease bandwidth. A simple one is to separate publish and confirm_ack. More complicated ideas are being worked on too.

Thanks for reply, can i ask the purpose of election on all blocks, for prunning?

I think it could be for users to have finality in their transactions

This issue should be solved by vote stapling and/or active graph.  I am going to close this issue so that those specific improvements can be tracked directly.  Let me know if you have additional questions !


Adding a new rebus

Associated Issue: N/A
Summary of Changes

Adding a new rebus

Pull Request Test Coverage Report for Build 557

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.143%

Totals

Change from base Build 514:
0.0%

Covered Lines:
158

Relevant Lines:
180

💛  - Coveralls


Silent unmount in DokanNetMirror during operation

TL;DR;
I launch DotNetMirror to mirror Original into Mirror folder. It shows in Explorer as a mount, I can open it.
Launch a 3rd party game with its data path set to a subfolder of Mirror folder.
By the time the game finishes loading, the mount disappears together with the NTFS directory where it was mounted to. Later game as part of its functioning recreates the directory as a regular NTFS directory (I suspect it just sees mount point gone).
A potential clue from the game is

22.750 Warning FileUtil.cpp:447: Permission denied trying to remove C:\Users\lost\Projects\Borg\Debug\DummySaves\Factorio\temp...lock. Will set write permissions and try again
22.844 Error FileUtil.cpp:469: Couldn't remove C:\Users\lost\Projects\Borg\Debug\DummySaves\Factorio\temp...lock: The process cannot access the file because it is being used by another process.

Repro steps

Download and install Factorio Demo. Full game should also work.
Create a directory Original anywhere on existing NTFS volume.
Copy this file to Original: config.txt
Rename it to config.ini
Edit the file and change write-data= to point to the full path to Mirror\UserData (might need to precreate Original\UserData)
Launch DokanNetMirror to mirror Original into Mirror (I did NOT use unsafe)
Check that Mirror has config.ini with write-data= pointing to C:\Path\To\Mirror\UserData
Launch factorio.exe --config C:\Path\To\Mirror\config.ini
Wait and watch the contents of Mirror directory

What happens
As mentioned above, at some point the mount just disappears, and the only clue is the error from the game console output.
The game will shortly recreate it as a regular directory, but the Original will no longer be updated.
DokanNetMirror continues to work at this point, and no errors are displayed. Actually, there are no more logs from it from this point on. The log just stops until I Ctrl+C it at which point it prints Success and quits as if everything is OK.
This one is from DokanNetMirror when I do above steps (instead of UserData I use Factorio but that should not matter):
mirror.log

Hi @lostmsu ,
are you able to reproduce with the native C mirror ?
Usually when the mount disappears it means a userland request was taking too long to process a request (did not response) and the kernel force unmount the drive to avoid possibly hanging the system too.
But in that case you should also see the mirror unmounting and not thinking it is still alive which is very weird.

Hi @Liryna , just tried with the native mirror, and it does not reproduce.

Interesting, are you able to attach a debugger and see the running threads just after the device unmount ? It would be great if it shows the hanging thread / request.

@Liryna it does not look like timeout. In the game log, the permission denied message appears slightly over 1 second after a message about successful initialization.
Unfortunately I don't have game source code, so I can not pinpoint the moment when the game hits the error.

Even funnier thing. Turns out when I close the game after the mount already disappeared from Explorer, the game is still able to write log file to Original through Mirror I am guessing via previously open handle.

I just thought to look into Event Viewer/System and found a few entries from dokan:

Warning: no matching IRPs found for reply.

FYI I originally experienced similar behavior with another IDokanOperations implementation, but switched to reproducing in Mirror as that's what other issues requested.

Is there something I could do short of debugging the driver?
Perhaps any guesses, like an unhandled exception?

By using Procmon you could see the status returned to the system for all the requests and find  the one returned with a timeout. Then the next step would be  to trace it on user land and see what it is happening

Still investigating, but heads up.
This is from native mirror (that works):

This is from dotnet one (that does not):

The insufficient resources is most likely the query that timeout on dotnet side. Should find out if it is received and why not processed/ returned with a status in time.

I am debugging it, and I see that the driver returns that error very quickly.
Here's what I did:
I opened the solution, and wrapped the contents of ZwCreateFileProxy in Stopwatch. E.g.
var time = Stopwatch.StartNew();

try { ... }
finally
{
  time.Stop();
  logger.Debug("CreateFileProxy : {0} Time : {1}ms",
               rawFileName, time.ElapsedMilliseconds);
}

And the maximum value I see for that directory is 17ms.

Also, the directory in question as well as couple of others gets a call to SetDispositionInformationEx(FILE_DISPOSITION_POSIX_SEMANTICS | FILE_DISPOSITION_DELETE).

OK, I figured it out. The issue has nothing to do with the driver, hangs, or instability. Factorio actually deletes the directory for some reason when it encounters a certain error.
Now the error has to do with magic "." and ".." entries in FindFiles that native mirror returns, and .NET mirror does not.
I feel like either .NET version needs to make a breaking change and add them automatically for non-root directories, or the proxy should check the list of returned files for presence of "." and ".." and if they don't appear, make some loud warning so that developers would see that their implementation is incorrect.

Looks like https://github.com/dokan-dev/dokan-dotnet/commit/7fa8ca4dd709d93155ac0915aefbd7977fbd413e got lost at some point.

Nice catch!
It was moved to the native library to avoid having all the wrappers introducing the samelogic https://github.com/dokan-dev/dokany/blob/c4014d846c4092735afff21b1fd8eefc04d44ef8/dokan/directory.c#L465
Maybe the issue is in C# mirror that adds them for the root ? https://github.com/dokan-dev/dokany/blob/c4014d846c4092735afff21b1fd8eefc04d44ef8/samples/dokan_mirror/mirror.c#L824-L831
Does EnumerateFileSystemInfos includes them ? If yes, we should remove them like native mirror does https://github.com/dokan-dev/dokan-dotnet/blob/137dbdd6162286b31f89eaa949fdcc2c5d82be26/sample/DokanNetMirror/Mirror.cs#L727

No, EnumerateFileSystemInfos does not return the magics.
Is it possible that the native library bundled with .NET lib is old and does not include these items?
Another thought that I had after a quick skim is maybe that code path in the native library is not actually getting invoked in certain scenarios: it seems suspicious to me, that Procmon shows Result NO MORE FILES and the only call to AddMissingCurrentAndParentFolder is guarded by if (Status == STATUS_SUCCESS).

If the native code you mentioned would be working, then my explicit addition of . and .. would be a no-op, however it does resolve the issue with Factorio.

And thanks for finding the source of the issue and the solution!
It is true that the Win32 can return NO_MORE_FILES but in our case as we don't have a buffer limit, we can return the whole listing without having to split them by batch. Therefore an implementation returning NO_MORE_FILES is invalid for me and this is not the case as C# Mirror only return success.
https://github.com/dokan-dev/dokan-dotnet/blob/master/sample/DokanNetMirror/Mirror.cs#L408
https://github.com/dokan-dev/dokan-dotnet/blob/master/sample/DokanNetMirror/Mirror.cs#L747
I believe what is wrong is that the . and .. are append at the end of the listing which could confuse the app as they are normally always at front. (This can be reproduced with cmd.exe dir, the . .. are currently last).
https://github.com/dokan-dev/dokany/commit/b41d0226c8137dc4051c31e9f54323c59ff5bc82
Are you cappable to just rebuild the native library and use it in C# to confirm it fixed the problem ?
Otherwise I believe you can reproduce the issue with the native sample by changing
https://github.com/dokan-dev/dokany/blob/c4014d846c4092735afff21b1fd8eefc04d44ef8/samples/dokan_mirror/mirror.c#L825 to BOOLEAN rootFolder = TRUE;

BOOLEAN rootFolder = TRUE; breaks it in native mirror, and https://github.com/dokan-dev/dokany/commit/b41d0226c8137dc4051c31e9f54323c59ff5bc82 then fixes it

Awesome, Thanks for the confirmation! 🏆  I believe this can be closed now and waiting for a next release.


Release for Angular 13 support

Hi, I've noticed that in #79 the problem with the changed Schema was fixed. However, this fix was merged, it was never released.
Can we organize a new release?

Hello @Bjeaurn,
Thanks for reminding us of this. We will do it.

<EMAIL_ADDRESS>is published now, thank you for your patience.


Changing the Code Syntax Highlighting Theme

Hi everybody,
Thank you very much for the reveal.js menu plugin - its absolutely great and is exactly what I need - thanks.
Apologies if this has been answered elsewhere, but is there any way to change the code syntax highlighting theme when selecting a different theme in the theme panel? In particular, I would like to use monokai.css when using the 'Black' theme (i.e. the default) and then switch to (for example) Xcode.css when using 'League'.
Many thanks and have a good day,
Jim

This is currently not supported. When I get some time I'll have a look at adding it.

Great - thank you very much - much appreciated.
Thanks again and have a good day.

I just released v2.1.0 to add support for changing the highlight theme. You'll need to provide your specific theme setup in the menu configuration in your presentations. Let me know if you have any issues with it.

Great - thank you very much - that's excellent - thanks - I'll give it a go.
Thanks again and take care.


The second video  in the gstreamer-send example

How to write a corrent pipeline string for the second video in the gstreamer-send example? any hint?

Can you provide more information? What video are you referring to - do you mean the second example usage in the README?

Hi @mobistor
gstreamer-send only has one video pipeline, but you can send the same video to multiple tracks. If you want to start two video pipelines you will need to call gst.CreatePipeline again.
I am closing this for inactivity, if you have further questions please re-open!


Increment versions for billingbenefits releases

Increment package versions for billingbenefits releases

API change check
API changes are not detected in this pull request.


l2geth/dtl changes to avoid monotonicity violations

I've re-structured some of the interactions between l2geth and the data-translation-layer to fix one source of the timestamp monotonicity violations which showed up when running the integration stress tests against my local system. Please try it out and see if it works in other environments.

I've found a potential problem with this PR. Please don't merge it into the develop branch or deploy it to Rinkeby until I've had time to look at it. It is still useful to try this branch on a local development system and see if it helps (or hurts) the reliability under load.

I'm closing this PR and will open a new one from a different branch (based against a newer 'develop' and with additional fixes).


doesnt always detect login form properly

example: twitter https://twitter.com/i/flow/login

also: google, possibly since they now implemented a two-page login: first page is username, second is password


[WIP] Use awscc Resources, Where Possible

include awscc user_agent
leverage awscc resources where possible
open issues for resources that cannot be converted yet
examples specify var.profile default as null to allow users credential chain to control
terraform-docs for README.md, still lacks some content and a strong header

Remaining:

go test is failing
README should be have better descriptions on usage patterns

closing in favor of rewrite to vpc module #32


Limit emulated machine values to the q35 family

What this PR does / why we need it:
Limit emulated machine values to the q35 family at the API-level.
Which issue(s) this PR fixes:
Fixes #1230
Release note:
Machine type is now limited to 'q35' or one of its derivatives.

CI failures seem unrelated:
--- PASS: TestValidatingWebhook (0.05s)

@cynepco3hahue I missed that somehow, thanks!

Right, this is something that was discussed in the context of supporting q35 in oVirt as well - the ability to use SATA instead of IDE should enable that.

@rmohr IIRC, windows image already have virtio drivers, so it will not cover our case(but worth to double check it).

@cynepco3hahue so they use sata disks right now, I saw it in the tests. If you did not install extra sata drivers (which I would find unbelievable) we should be good.

ci test please

@ahadas I see you need to run make generate and commit again.

@cynepco3hahue oh, right, I didn't do that after adding the last two commits - done.

ci test please

I got the request to make this filter configurable.
Could this be extended to take the filter pattern - or a list of allowed values from the kubevirt-config configmap?

why do we even want to validate anything? It should be a configuration, not even changeable normally.
Also q35 is not valid for non x86 architectures

Currently we make silent assumptions about the primary machine typ, i.e. we set e1000e by default, which is only applicable to q35.
Validating this field is only the safety net for these kind implicit assumptions.
We need to know the baseline we operate on.
I'm not saying that we should prevent any other machine type altogether, I am merely saying that we should add some validation here and there to give the user some guarantees. And to also know what users are looking for.
IOW: A whitelist of machine types instead of full freedom.
We will need this moving forward in order to give guarantees about guest support.

Could this be extended to take the filter pattern - or a list of allowed values from the kubevirt-config configmap?

Sure, looks like an over-specification at the moment but may be handy in the future.

With https://github.com/fabiand/common-templates handling that I wonder how far you want to go in kubevirt. It doesn't make sense to maintain it in two places

Would all VMs be based on those templates? What about VMs that are converted from VMware or oVirt? VMs that are created from scratch?

(just use this on ppc64le and you're screwed - there's no q35* machine type, no e1000e at all)

Right, we would need to extend that filtering. We would probably need to code more things in order to support additional architectures though.

(as much as we want Windows to work as well, I believe we should favor performance and compatibility with Linux)

So shouldn't it be OS-specific setting?

It should allow whatever we can theoretically support because sooner or later someone always finds some use case for it

But is there a reason to think that we would be able to support a new chipset type by just allow setting it instead of q35? Our past experience in oVirt shows that supporting q35 while you only support i440fx (on x86_64) was not trivial to achieve. Even if we won't have that many things that are incompatible with the new chipset as we had in oVirt, the fact that we need to convert some VM configuration to the domain XML suggests that some coding would probably be needed.

The reason why we should have this is the testing matrix and implicit assumptions.
Right now our default machien type is q35. And for simplicity we want to focus around this machine type.
It is true that this will (initially!) prevent certain VMs which require non-q35 machien type to not be able to run on KubeVirt. however, there is work going on in v2v to allow migrating between machien types.
After all, we have the opportunity to start with a rather fresh (but 10yrs old) machien type (q35) and avoid to drag all legacy from i440 into KubeVirt.
Thus - For now let's limit to q35, this will help us to focus on this and make running VMs on this solid.
We'll need to update test cases to ensure that q35 is used as well.

It should allow whatever we can theoretically support because sooner or later someone always finds some use case for it

But is there a reason to think that we would be able to support a new chipset type by just allow setting it instead of q35? Our past experience in oVirt shows that supporting q35 while you only support i440fx (on x86_64) was not trivial to achieve. Even if we won't have that many things that are incompatible with the new chipset as we had in oVirt, the fact that we need to convert some VM configuration to the domain XML suggests that some coding would probably be needed.

I'm fully on your side. The only reason why I am not totally against making this configurable is that we can try stuff and allow people to gather experience.

The change itself actually shows how it impacts certain configurations of the VM
If we do not limit us to q35, then the testing matrix is much larger.
Creating device topologies will be much larger. etc.
Btw - I consider this a API breaking change, thus something for #1576.

See above.
/hold

The supported emulated machines are now configurable via the kubevirt-config configmap. I guess we should consider unifying the module introduced in this PR with feature-gaps as they both read that configmap at some point

Not sure what happened but I probably updated my master branch (yeah, shouldn't use the master branch for this PR..) - I'll post a new PR

Posted #1718


Add “mix check.unused_locked_deps” task

We want to avoid keeping unused dependencies in mix.lock. This pull request adds a new check.unused_locked_deps that is used in the CI build.
I also renamed our other check erlang.check_version to check.erlang_version in order to keep things consistent 😄
Also, there were three unused deps in mix.lock 🙄

🦀 Requesting reviewers for this pull request:

@simonprev (contributor with 4 commits in the last 90 days and 16 commits overall)
@olivierbuffon (expert reviewer for the elixir stack)


MN-469 Fixing portrait camera + prevent zoom/wide camera

Also now that I think about it : when I wrote the code for the Camera in the refacto, I thought it was a good idea to let the user select the device manually using a deviceId in the constraints given to the Camera component. But after Deepak's work on the camera, it appears that this is a mistake : we should always handle the device selection ourselves to prevent users from using zoomed cameras by inadvertance.
So could you take the time to remove this option in the code please? Remove every option to select the deviceId in the camera constraints (everywhere in the code : in the Camera props, in the getMediaConstraints, in the type definitions, in the doc etc.)


Missing Brackets while Squaring Expressions

Related to #273 but keeping it separate for organization. The following is what I would like printed.

This is what I typed for the first expression in the equation.
((C slopeHght)-(C slipHght )):^(Int 2):/(Int 2) :* (C satWeight) +
((C waterHght)-(C slopeHght)):^(Int 2) :* (C satWeight)

And this is what was generated.

To point out the issue, brackets are missing around the (y_u,i - y_slip,i)^2 part.

(C intNormForce)*((Int 1)-(C SM.poissnsR)) /
(((Int 1)+(C SM.poissnsR)) :* ((Int 1) :- (Int 2):*(C SM.poissnsR) :+ (C baseWthX)))

Another case of missing brackets, this time with :*

I've updated this as of 45dc8f132871d51dcc2e998bac423748bfc6de00 can you please check if everything has been updated correctly (and double check that no new bugs have arisen)?
If it all looks good, we can close this issue.

All the brackets look good. Not sure if this is new or not but this came up. The original document has "0.01fraction" but the way this is laid out looks like its is "1.0e minus 2*fraction"

@szymczdm

Are they mathematically equivalent?

0.01*fraction is equal to 1.0e-2 * fraction. The problem is that is looks like (1.0*e) - (2*fraction)

I could not parse that correctly, which is why I was confused.  Now that I parse it as intended, it is indeed obvious that they are mathematically equivalent, but it looks all wrong!  Please file a separate issue for this.  I think the issue which started this is fixed.


Parthenon Vibe on GPUs

Hi Dan,
Just wondering do you have any update on this?
Thanks,
Peng
From: Shipman, Galen M<EMAIL_ADDRESS>Sent: Friday, July 28, 2023 7:51 AM
To: Peng Wang<EMAIL_ADDRESS>Ian Karlin<EMAIL_ADDRESS>Cc: Scot Halverson<EMAIL_ADDRESS>Giridhar Chukkapalli<EMAIL_ADDRESS>Magee, Daniel J<EMAIL_ADDRESS>Paul Cook<EMAIL_ADDRESS>Subject: RE: [EXTERNAL] RE: Parthenon-VIBE Questions
Dan,
Can you check this? Try to reproduce the results on Darwin a100 first.
Thanks
Galen
From: Peng Wang<EMAIL_ADDRESS>Date: Thursday, Jul 27, 2023 at 5:00 PM
To: Shipman, Galen M<EMAIL_ADDRESS>Ian Karlin<EMAIL_ADDRESS>Cc: Scot Halverson<EMAIL_ADDRESS>Giridhar Chukkapalli<EMAIL_ADDRESS>Magee, Daniel J<EMAIL_ADDRESS>Paul Cook<EMAIL_ADDRESS>Subject: RE: [EXTERNAL] RE: Parthenon-VIBE Questions
Hi Galen, Dan,
Thanks for the info. The input file (burgers.pin) in the repo has meshblocks=16. However, I couldn’t reproduce the A100 performance in Table 4.4 of https://lanl.github.io/benchmarks/3_vibe/vibe.html: my measured A100 perf is ~ 4X lower than Table 4.4 with meshblocks=16. However, if I use meshblocks=32, I would get numbers much closer to Table 4.4.
So I want to double check did you use meshblocks=16 or 32 for the result in Table 4.4?
Thanks,
Peng
From: Shipman, Galen M<EMAIL_ADDRESS>Sent: Thursday, July 27, 2023 4:27 PM
To: Ian Karlin<EMAIL_ADDRESS>Cc: Scot Halverson<EMAIL_ADDRESS>Giridhar Chukkapalli<EMAIL_ADDRESS>Magee, Daniel J<EMAIL_ADDRESS>Paul Cook<EMAIL_ADDRESS>Peng Wang<EMAIL_ADDRESS>Subject: Re: [EXTERNAL] RE: Parthenon-VIBE Questions
Ian, (Looping in Dan Magee)
For Parthenon-VIBE only the parthenon/mesh/nx* parameters may be changed (to adjust memory footprint).
We would like to see 20%/40%/60% memory footprint results.
Thanks!
Galen
From: Ian Karlin<EMAIL_ADDRESS>Sent: Wednesday, July 12, 2023 2:57 PM
To: Shipman, Galen M (EXTERNAL<EMAIL_ADDRESS>Cc: Giridhar Chukkapalli<EMAIL_ADDRESS>Scot Halverson<EMAIL_ADDRESS>Paul Cook<EMAIL_ADDRESS>Peng Wang<EMAIL_ADDRESS>Subject: Parthenon-VIBE Questions
Hi Galen,
A couple of questions about Parthenon-VIBE that would help us with our initial modeling of this.
Can you please document the settings used in your runs so we can recreate the results and understand the baseline?
Then, can you document the settings for the benchmark problem that can be changed and those that can’t be changed from the baseline runs.  In particular, meshblocks, numlevels, nlim and tlim?
Thanks and let us know if you have any questions,
Ian

@bvillasen  We are cleaning up older issues as we prepare for a final version of the benchmarks, can you confirm is the perf variability issue described above still an issue for you? Thanks, Galen

I believe this issue is resolved. Closing, please ping if still an issue.


doc/instances: Improves proxy docs

Signed-off-by: Thomas Parrott<EMAIL_ADDRESS>

Had a few repeat questions/issues with proxy device recently (https://discuss.linuxcontainers.org/tag/proxy) so thought this could help as a reference.

Testsuite passed


add:rabin

Description
[Include a brief description of the changes you've made]
Related Issues
[Include any related issues or pull requests that this PR addresses or is related to. Use GitHub's shorthand syntax to link to them, like #1234.]
Changes Proposed
[Describe the changes you've made in detail. Be specific and include any relevant code snippets.]
Checklist

[ ] I have read and followed the Contribution Guidelines.
[ ] All new and existing tests passed.
[ ] I have updated the documentation to reflect the changes I've made.
[ ] My code follows the code style of this project.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots
Note to reviewers

@rabinprajapati Can you resolve the conflicts?

@rabinprajapati Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and Star the repo to help us grow even further!


unwanted association belongs_to :plan in model Phase?

Please complete the following fields as applicable:
Expected behaviour:
Actual behaviour:
Model Phase claims to belong to one single Plan,
which would mean that once a Phase is claimed by
one Plan, others could not claim this anymore.
But on the other hand, there is no attribute "plan_id"
in the table "phases". So this is a typo, and can be removed?
Steps to reproduce:
See https://github.com/DMPRoadmap/roadmap/blob/development/app/models/phase.rb#L46

agreed. This looks like legacy code that we have not cleaned up.
We should make sure the load_for_phase class method on the Plan model still functions as expected after removing the association (I suspect it will since it is going through the hierarchy properly plan.template.phases)
@nicolasfranck, we happily review and accept PRs if you are so inclined and have bandwidth/time to address this :)


trigger doesn't work

I'm getting the following error when it triggering a job, and it doesn't trigger the jenkins-job:

{"alert":"ALERT","context":"runner","details":{"job":"ilai-run-gor-staging-1"},"duration":"712.840438ms","event_id":"899d","executor":"Jenkins","level":"error","msg":"runner got executing task #1/1 error, stopping group: Could not invoke job ","result":"exec_error_without_block","rule":"JenkinsAutofix","tasks":[{"alert":"ALERT","details":{"job":"ilai-run-gor-staging-1"},"event_id":"899d","executor":"Jenkins","rule":"JenkinsAutofix"}],"time":"2019-04-17T15:55:01Z"}
{"context":"runner","level":"debug","msg":"runner finished executing group","tasks":[{"alert":"ALERT","details":{"job":"ilai-run-gor-staging-1"},"event_id":"899d","executor":"Jenkins","rule":"JenkinsAutofix"}],"time":"2019-04-17T15:55:01Z"}

Hello, @mosespx
Please show me your config (hide password).

Hi @krpn this is the config file:
# cache size for blocked tasks
# 50 * 1024 * 1024 = 50 MB
block_cache_size: 52428800

# pool size for new tasks
# locks webhook if overflow
pool_size: 100

# runners count for parallel actions execute
runners: 10

# remote config refresh interval
# rules refreshed only
remote_config_refresh_interval: 60s

# common parameters for actions
common_parameters:
  jenkins_credentials:
    endpoint: https://jenkins-url.corp.com/
    login: prometheus
    password: <PASSWORD>
    state_refresh_delay: 5s

# list of rules to check and act
rules:
- name: JenkinsAutofix
  conditions:
    labels:
      #webhooker_jenkins_autofix: enabled # auto fix enabled
      #jenkins_job: (.*?)                 # jenkins job is set
      intent: jenkins-webhook
  actions:
  - executor: jenkins
    common_parameters: jenkins_credentials
    parameters:
      #job: ${ANNOTATION_JENKINS_JOB} # job name from annotation jenkins_job
      job: ilai-run-gor-staging-1 # job name from annotation jenkins_job
#    block: 10m

Is job available on link https://jenkins-url.corp.com/job/ilai-run-gor-staging-1 for user prometheus using password <PASSWORD>?

I checked again, yes

This error threw from gojenkins library when REST API request got bad HTTP status code (not 200 or 201). Possible reasons:

Jenkins endpoint URL is not reachable (from webhooker server), status code: 5**
Jenkins endpoint URL redirects to another URL (redirects are not supported), status code: 302/307
Wrong credentials, status code: 403
Wrong job name, status code: 404

As I see in log job passed correctly. Config is also correct. Please check reasons above.

@krpn Thanks
Seems like it was I managed to make it work, but the triggering is very slow, it can take a few minutes from the alert firing to the job triggering. any idea why?

First of all check webhooker logs. You need to compare duration between payload is received (webhook triggered) and runner starts executing group (start of Jenkins job in this case). Webhooker works fine If duration is small. There is no any special delays for this.
Second you need to check alertmanager config. There are some settings that can make delays. For example: group_wait.

it seems like an alertmanager issue.
thanks!


can not add group info for user

hi team,
I have an instance that is updated from an old version, and now it is 1.503.0. I found the user info page lost many fields, is there any control option to effect this? or some fields in the database are missing?

@jump2cn the community here only provides support for the latest version (aka v1.602.0 for now). So either you upgrade to latest version and update your wording, or you find a third-party which provides old version support.


Error while executing yarn view:local minimal

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues: https://github.com/wowchemy/wowchemy-hugo-themes/issues?q=is%3Aissue
[X] This issue is not a question, feature request, or anything other than a bug report directly related to Wowchemy. Please post those things on Discord: https://discord.gg/z8wNYzb

Description
I followed the instructions about contributing and cloned a fork of the repository. After that I used the command yarn view:local minimal but got that error:
Error: command error: failed to load modules: module "github.com/wowchemy/wowchemy-hugo-themes/modules/wowchemy-seo" not found in "/Users/Konafets/Sites/wowchemy-hugo-themes/starters/minimal/themes/github.com/wowchemy/wowchemy-hugo-themes/modules/wowchemy-seo"; either add it as a Hugo Module or store it in "/Users/Konafets/Sites/wowchemy-hugo-themes/starters/minimal/themes".: module does not exist
error Command failed with exit code 1.

Reproduction Link
https://github.com/Konafets/wowchemy-hugo-themes
Steps to Reproduce

git clone git clone https://github.com/Konafets/wowchemy-hugo-themes.git
cd wowchemy-hugo-themes
yarn view:local minimal

Expected Result
I can open the theme in my browser.
Actual Result
Error: command error: failed to load modules: module "github.com/wowchemy/wowchemy-hugo-themes/modules/wowchemy-seo" not found in "/Users/Konafets/Sites/wowchemy-hugo-themes/starters/minimal/themes/github.com/wowchemy/wowchemy-hugo-themes/modules/wowchemy-seo"; either add it as a Hugo Module or store it in "/Users/Konafets/Sites/wowchemy-hugo-themes/starters/minimal/themes".: module does not exist
error Command failed with exit code 1.

What Hugo Module versions does your site use?
cat starters/minimal/go.mod
module github.com/wowchemy/hugo-minimal-theme

go 1.15

require (
	github.com/wowchemy/wowchemy-hugo-themes/modules/wowchemy-core v0.2.0 // indirect
	github.com/wowchemy/wowchemy-hugo-themes/modules/wowchemy-plugin-netlify v1.0.0 // indirect
	github.com/wowchemy/wowchemy-hugo-themes/modules/wowchemy-plugin-netlify-cms v1.0.0 // indirect
	github.com/wowchemy/wowchemy-hugo-themes/modules/wowchemy-seo v0.1.0 // indirect
	github.com/wowchemy/wowchemy-hugo-themes/modules/wowchemy/v5 v5.7.1-0.20221024214304-ad466232017f
)

What operating system(s) are you seeing the problem on?
macOS
What browser(s) are you seeing the problem on?
Chrome
Which Wowchemy template are you using?

What version of Hugo are you using?
v0.115.4+extended darwin/arm64 BuildDate=unknown

LOL, this issue is not closed.


tooltip positions failes on flex wrapper

Describe the bug
I made a quick video that shows the problem:
https://user-images.githubusercontent.com/84905165/178108568-a72b5bb4-6470-419a-a54c-d2a9225eb6b6.mp4
After hovering the tooltip element the third time the position seems to be fine. The first two hovers are placing the tooltip on different positions.
To Reproduce
Steps to reproduce the behavior:

Put a wrapper with flex around your tooltip
move cursor on tooltip
leave tooltip
move cursor again on tooltip

Code samples

	<Tooltip content={$LL.components.navigation.meta.helpIcon.tooltip()} placement="auto">
		test
	</Tooltip>

Expected behavior
The position should be always the same when hovering the tooltip
Screenshots
see video
Desktop (please complete the following information):

Browser: chrome
Version: 102.0.5005.115

Thank you for the issue.
I'm not able to reproduce the problem.
REPL
Am I missing something here?

Thank you for the issue. I'm not able to reproduce the problem.
REPL
Am I missing something here?

You are right - the problem seems to be somehow related to another part of my app. I will investigate a bit more and give you feedback. Thanks a lot!

I close the issue for now.
Thank you for your contribution.


Handle OpenTracing SetTag with "error"

Many OpenTracing users use span.SetTag("error", error), this passes the exception through to the AO span.Error() or span.Err() methods if an error provided.

Codecov Report

Merging #149 into master will increase coverage by 0.04%.
The diff coverage is 95.00%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
+ Coverage   81.98%   82.03%   +0.04%
==========================================
  Files          37       37
  Lines        4585     4603      +18
==========================================
+ Hits         3759     3776      +17
- Misses        703      704       +1
  Partials      123      123

Impacted Files
Coverage Δ

v1/ao/opentracing/tracer.go
99.13% <95.00%> (-0.87%)
:arrow_down:

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 0e4a1e2...4a107e4. Read the comment docs.


Add introduction section docs

Closes #133
Adds introduction section:

Introduction.
Getting started.
Alternatives and scope.
Install.

Also adds a more section with links to Github and releases.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #146   +/-   ##
=======================================
  Coverage   75.42%   75.42%
=======================================
  Files          26       26
  Lines        1184     1184
=======================================
  Hits          893      893
  Misses        216      216
  Partials       75       75

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 5db9f13...51b0ffc. Read the comment docs.


[Sql] DatabasesOperationsExtensions.DeleteAsync() should throw if not exists

DatabasesOperationsExtensions.DeleteAsync() should throw an exception when the database does not exist. Currently you can await it and if the database doesn't exist it will simply carry on. This is in contrast to say attempting to delete a Dns record set which will throw if it does not exist.

Sorry for late response. SQL team might have some insights to share. Thanks.

Thank you for reporting this feedback!
The current behaviour is correct according to ARM RPC.
https://github.com/Azure/azure-resource-manager-rpc/blob/master/v1.0/resource-api-reference.md#delete-resource

The resource provider can return 200 (OK) or 204 (NoContent) to indicate that the operation completed successfully. A 200 (OK) should be returned if the object exists and was deleted successfully; and a 204 (NoContent) should be used if the resource does not exist and the request is well formed.
202 (Accepted) can be returned to indicate that the operation will complete asynchronously.
If the resource group does not exist, 404 (NotFound) will be returned by the proxy layer and will not reach the resource provider. 412 (PreconditionFailed) and other normal REST codes are acceptable as long as they match the REST guidelines.

If deleting dns record set throws when it doesn't exist, then that's a bug in the DNS API ;)


send_message api giving error ""Missing 'num_before' argument""

i am was just trying to test locally running zulip server by sending message to a stream using bot credentials  with the following code:
`import zulip
import pprint as pp
client = zulip.Client(config_file=".zuliprc", insecure=True)
result = client.get_user_by_id(9)
print(result)
Send a stream message
request = {
"type": "stream",
"to": "klasa_integration",
"subject": "test",
"content": "this is a test message"
}
result = client.send_message(request)
pprint.pprint(result)`
and this is giving the following error:
Insecure mode enabled. The server's SSL/TLS certificate will not be validated, making the HTTPS connection potentially insecure /Users/user1/.virtualenvs/disana/lib/python3.8/site-packages/urllib3/connectionpool.py:997: InsecureRequestWarning: Unverified HTTPS request is being made to host '<IP_ADDRESS>'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings   warnings.warn( {'result': 'error', 'msg': "Missing 'num_before' argument", 'var_name': 'num_before', 'code': 'REQUEST_VARIABLE_MISSING'}

I would close this as not a bug.  But I wonder if we should be doing something to disable redirect handling with how we're calling requests -- probably worth keeping this open until we've investigated that.

Had the same issue white trying to use the API
client = zulip.Client(site='http://localhost'<EMAIL_ADDRESS>api_key='REDACTED', insecure=True, verbose=True)
It helped to enable some debug logs
import http
http.client.HTTPConnection.debuglevel = 1

after that I got the following output while using client.send_message
send: b'POST /api/v1/messages HTTP/1.1\r\nHost: localhost\r\nUser-agent: ZulipPython/0.8.2 (Ubuntu; 22.04)\r\nAccept-Encoding: gzip, deflate\r\nAccept: */*\r\nConnection: keep-alive\r\nContent-Length: 12\r\nContent-Type: application/x-www-form-urlencoded\r\nAuthorization: Basic REDACTED\r\n\r\n'
send: b'type=streami'
reply: 'HTTP/1.1 301 Moved Permanently\r\n'
header: Server: nginx/1.18.0 (Ubuntu)
header: Date: Thu, 02 Mar 2023 00:23:59 GMT
header: Content-Type: text/html
header: Content-Length: 178
header: Connection: keep-alive
header: Location: https://localhost/api/v1/messages
/usr/lib/python3/dist-packages/urllib3/connectionpool.py:1015: InsecureRequestWarning: Unverified HTTPS request is being made to host 'localhost'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
  warnings.warn(
send: b'GET /api/v1/messages HTTP/1.1\r\nHost: localhost\r\nUser-agent: ZulipPython/0.8.2 (Ubuntu; 22.04)\r\nAccept-Encoding: gzip, deflate\r\nAccept: */*\r\nConnection: keep-alive\r\nAuthorization: Basic REDACTED\r\n\r\n'
reply: 'HTTP/1.1 400 Bad Request\r\n'
header: Server: nginx/1.18.0 (Ubuntu)
header: Date: Thu, 02 Mar 2023 00:23:59 GMT
header: Content-Type: application/json
header: Content-Length: 115
header: Connection: keep-alive
header: Vary: Accept-Language
header: Content-Language: en
header: X-RateLimit-Limit: 200
header: X-RateLimit-Remaining: 197
header: X-RateLimit-Reset:<PHONE_NUMBER>
header: Strict-Transport-Security: max-age=15768000
header: X-Frame-Options: DENY
header: Access-Control-Allow-Origin: *
header: Access-Control-Allow-Headers: Authorization
header: Access-Control-Allow-Methods: GET, POST, DELETE, PUT, PATCH, HEAD
Out[9]:
{'result': 'error',
 'msg': "Missing 'num_before' argument",
 'var_name': 'num_before',
 'code': 'REQUEST_VARIABLE_MISSING'}

So @timabbott 's hunch was write, looks like something redirects to the GET request.
What helped me was pointing the client to use site='https://localhost' (instead of http://). After that it ended up as a single POST request, and succeeded.
Perhaps NGINX inside the container redirecting http:// to https:// or something like that? Not sure what's a proper solution here, but perhaps the client could warn if the endpoint is http and suggest to switch to https?


Add hal+json as a supported media type option

Add application/hal+json as a Content-Type option to the dropdown for requests, treating the actual request body identically to application/json.  See #88

Hey @davidyscott,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 9adb8d30-c9ee-11e9-a673-e3b5aa05dde7

Please provide a description to PR for testing purposes

Can you please mention at least one mock/test API which has application/hal+json response? This is to run few tests so that I could verify and merge this PR into master.

Rebase and pull again

With apologies, I may not have time to get back to this for several days.  If someone else is able to step in, go ahead.  Otherwise, I haven't forgotten!

#129 solved this, hence closing.


coq: default to version 8.11

Motivation for this change
Coq 8.9 (current default) is quite old
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build framac satallax why3


Cannot use my own port

I am using port 8888 instead of 22.
I tried to add the port in my shipitfile.js but not work:
  shipit.initConfig({
    default: {
      workspace: '/tmp/git',
      deployTo: '/myfolder',
      repositoryUrl<EMAIL_ADDRESS>      ignores: ['.git', 'node_modules'],
      key: '/mykey',
      port: 8888,
      shallowClone: true
    },
    staging: {
      servers<EMAIL_ADDRESS>    }
  });

Use myuser@staging.myweb.com:8888.

Thanks @neoziro!

This doesn't seem to work aye,
live: {
        servers: 'myserver:2222',
    }

I can see shipit trying to execute the following
ssh deploy@myserver:2222
When it needs to be doing
ssh -p2222 deploy@myserver


suitable prune rule

When training, there are something different between your code and @Lam1360 code, such as:
in Lam1360 code, the training is:
1, python train.py --model_def config/yolov3-hand.cfg -sr --s 0.01
but in yours, missing the back value.
2，for step2, in Lam1360 code, he didn't show the suitable prune rule, but in your code, it is

python test_prune.py

no suitable prune rule following.
I was wonder whether you have already test and the result is fine, or you missing some key point.

if the auther has already test the result, can you upload the result, and there are more information can discuss which can lead to the results we want.

in @talebalone code, the prune rule is:

python prune.py --cfg yolov3.cfg --weights checkpoints/yolov3_sparsity_100.weights --percent 0.3
the --percent 0.3 is following, I wonder whether this it the suitable value, or more test result.

It is the same...... This parameter is " --s 0.01" is the default parameter in the code.
The prune rule is (1)避免剪掉所有channel的最高阈值(每个BN层的gamma的最大值的最小值即为阈值上限)
(2)剪枝的时候会先对 bias 做处理，用其后的卷积层或者 BN 层来吸收这个参数，然后再将权重赋值给 compact model，
这样做的话就能保证 pruned_model （只对 weight 置 0 的模型）和 Compact_model 对相同的输入有相同的输出
The prune ratio mainly depend on the application.


docs(CONTRIBUTING): add warning about forced push

Add warning about the possible consequences of a forced push
docs(CONTRIBUTING): add warning about forced push
Replacing https://github.com/angular/angular.js/pull/13719

Instead, I think we need to change the wording to be...
If we suggest changes then:

* Make the required updates.
* Re-run the Angular test suite to ensure tests are still passing.
* Commit your changes to the branch
* Push to your GitHub repository (this will update your Pull Request):

If the PR gets too outdated we may ask you to rebase and force push to update the PR:

git rebase master -i
git push origin my-fix-branch -f

So no warning?
What if a maintainer will ask to force push forgetting the consequences?
With an old PR especially this may lead to loss of people's work.

Without squashing, there shouldn't be any loss, right ?

squashing is part of git rebase master -i

OK, so you can add the warning as a line after what I wrote :-)

Thanks for the changes - they are nearly right. I will fix up and merge.