Hammer PAN event is not working in IE 11 touch device in windows 8

Hello Team,
With any version of hammer.js ,The PAN event in IE 11 in windows 8 device its not working.
Kindly look into that.can u suggest?

Please provide a working replication of this, because we do test IE11 and it is known to work correctly.

Hello Team,
Thank you so much  for reply.
Have a great time to you!!!
I am replying late because your mail came into spam.
For div element the PAN event is working in IE 11 touch windows device but
its not working for SVG element.
OS: Windows 8
Browser: IE11
Element: SVG Element
Device:  Windows touch tablet
Event: PAN event is not working
I will provide the replication of issue soon.
Have A Good Day!!
Regards,
Vikash Singh
Software developer Siemens
Mob:+91<PHONE_NUMBER>
On Tue, Jul 12, 2016 at 8:22 PM, Chris Thoburn<EMAIL_ADDRESS>wrote:

Please provide a working replication of this, because we do test IE11 and
it is known to work correctly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hammerjs/hammer.js/issues/996#issuecomment-232072551,
or mute the thread
https://github.com/notifications/unsubscribe/ATOsS1IZWcEWosv381-exUv06vIu52XWks5qU6o4gaJpZM4JAx6J
.

Hello ,
Kindly find recorded  replication of the  issue
VID_20160715_155014.mp4
https://drive.google.com/file/d/0B0YnDWTNQ_vxOTRzbDVneGZ5M2c/view?usp=drive_web

Regards,
Vikash
On Fri, Jul 15, 2016 at 1:03 PM, Vikash Singh<EMAIL_ADDRESS>wrote:

Hello Team,
Thank you so much  for reply.
Have a great time to you!!!
I am replying late because your mail came into spam.
For div element the PAN event is working in IE 11 touch windows device but
its not working for SVG element.
OS: Windows 8
Browser: IE11
Element: SVG Element
Device:  Windows touch tablet
Event: PAN event is not working
I will provide the replication of issue soon.
Have A Good Day!!
Regards,
Vikash Singh
Software developer Siemens
Mob:+91<PHONE_NUMBER>
On Tue, Jul 12, 2016 at 8:22 PM, Chris Thoburn<EMAIL_ADDRESS>wrote:

Please provide a working replication of this, because we do test IE11 and
it is known to work correctly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hammerjs/hammer.js/issues/996#issuecomment-232072551,
or mute the thread
https://github.com/notifications/unsubscribe/ATOsS1IZWcEWosv381-exUv06vIu52XWks5qU6o4gaJpZM4JAx6J
.

Hello,
Kindly update on the issue.
Regards,
Vikash
On Fri, Jul 15, 2016 at 5:18 PM, Vikash Singh<EMAIL_ADDRESS>wrote:

Hello ,
Kindly find recorded  replication of the  issue
VID_20160715_155014.mp4
https://drive.google.com/file/d/0B0YnDWTNQ_vxOTRzbDVneGZ5M2c/view?usp=drive_web

Regards,
Vikash
On Fri, Jul 15, 2016 at 1:03 PM, Vikash Singh<EMAIL_ADDRESS>wrote:

Hello Team,
Thank you so much  for reply.
Have a great time to you!!!
I am replying late because your mail came into spam.
For div element the PAN event is working in IE 11 touch windows device
but its not working for SVG element.
OS: Windows 8
Browser: IE11
Element: SVG Element
Device:  Windows touch tablet
Event: PAN event is not working
I will provide the replication of issue soon.
Have A Good Day!!
Regards,
Vikash Singh
Software developer Siemens
Mob:+91<PHONE_NUMBER>
On Tue, Jul 12, 2016 at 8:22 PM, Chris Thoburn<EMAIL_ADDRESS>wrote:

Please provide a working replication of this, because we do test IE11
and it is known to work correctly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hammerjs/hammer.js/issues/996#issuecomment-232072551,
or mute the thread
https://github.com/notifications/unsubscribe/ATOsS1IZWcEWosv381-exUv06vIu52XWks5qU6o4gaJpZM4JAx6J
.

While the video is nice, without demo code it's not a valid replication, any number of other things could be producing this error.
Based on your further feedback on this being specific to svg, it also seems likely that this is either a duplication of https://github.com/hammerjs/hammer.js/issues/906 (which has a simple fix but needs an implementor).

Hello ,
Thanks for reply!!
I am sharing you same source code of demo replication ,for that replication
video was created.
I think the issue #906 is different from this.

SVG
sample!  // Attaching event with SVG Text Element.

Kindly find in attachment.
Have A Good time !!
With best regards,
Vikash Singh
Mob:<PHONE_NUMBER>
On Fri, Aug 19, 2016 at 2:43 AM, Chris Thoburn<EMAIL_ADDRESS>wrote:

While the video is nice, without demo code it's not a valid replication,
any number of other things could be producing this error.
Based on your further feedback on this being specific to svg, it also
seems likely that this is either a duplication of #906
https://github.com/hammerjs/hammer.js/issues/906 (which has a simple
fix but needs an implementor).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hammerjs/hammer.js/issues/996#issuecomment-240858595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATOsS5p9Z_V4O2Cdl4PPkQAOj_E20GLGks5qhMrmgaJpZM4JAx6J
.

Hello,
Kindly update about this issue , Are you going to fix in next version.
Regards,
Vikash
On Mon, Aug 22, 2016 at 11:48 AM, Vikash Singh<EMAIL_ADDRESS>wrote:

Hello ,
Thanks for reply!!
I am sharing you same source code of demo replication ,for that
replication video was created.
I think the issue #906 is different from this.

SVG
sample!  // Attaching event with SVG Text Element.

Kindly find in attachment.
Have A Good time !!
With best regards,
Vikash Singh
Mob:<PHONE_NUMBER>
On Fri, Aug 19, 2016 at 2:43 AM, Chris Thoburn<EMAIL_ADDRESS>wrote:

While the video is nice, without demo code it's not a valid replication,
any number of other things could be producing this error.
Based on your further feedback on this being specific to svg, it also
seems likely that this is either a duplication of #906
https://github.com/hammerjs/hammer.js/issues/906 (which has a simple
fix but needs an implementor).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hammerjs/hammer.js/issues/996#issuecomment-240858595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATOsS5p9Z_V4O2Cdl4PPkQAOj_E20GLGks5qhMrmgaJpZM4JAx6J
.

@arschmitz I do think that this is likely an IE11 representation of the Firefox issue seen in #906
I put his example here but lack a test device atm.
https://jsfiddle.net/9ufvvgds/7/

Hello Team,
Kindly update, when this can be fixed from your side.
Similar issue id #46 has been raised.
Regards,
Vikash

This issue also seems to be causing a problem in vis.js when you click the vertical scroll bar in some versions of IE. You can't let go after that.


Facebook Sharing on Android - no URL or Image (or Message)

On Android 4.4 I've tried sharing to Facebook via .share, .shareViaFacebook, and . shareViaFacebookWithPasteMessageHint - with no image and a url. My url never gets added. Maybe this has something to do with the splitting out of Messenger functionality?
On iOS this works fine.

@EddyVerbruggen Hi, I am using Social Sharing plugin in one of my Ionic2 projects. Earlier evrything was working fine but from past 3 days I am not being able to share anything using shareViaWhatsApp() function. Whenever I try to share anything it says: "Attempt to invoke virtual method 'android.content.res.XmlResourceParserandroid.content.pm.ProviderInfo.loadXmlMetaData(android.content.pm.PackageManager, java.lang.String)' on a null object reference."
Is there anything you can help me with?

Do we know why this happens only on Android? Is the Android Facebook or Messenger App doing something after a cordova/ionic sends the correct URL, but not on iOS?
Or can we update some native SDK/code inside this plugin, that could allow for this?

@replysam2009 See my comment  https://github.com/EddyVerbruggen/SocialSharing-PhoneGap-Plugin/issues/774#issuecomment-369211159 into your issue about loadXmlMetada() error.

It seems that .edu also doesn't work?


XAML STYLER

Issue type
enhancement
Which version of the app?
WinUI 3 Gallery
Description
Just wanted to create an issue in order to have a handle to let XAML Styler run over a few folders.
Additional context
Do we need a Settings.XamlStyler file ?
### Tasks
- [ ] ❔ Settings.XamlStyler file
- [ ] prepare <elements> with `space=preserve`
- [ ] let XamlStyler do its magic
- [ ] check content of <elements> - think manual line endings and LineBreaks

### Folders 📁
- [ ] ControlPages
- [ ] Controls
- [ ] Navigation
- [ ] SamplePages
- [ ] TabViewPages
- [ ] WinUIGallery

[!NOTE]
Need to set xml:space="preserve" in elements first, in order to prevent the following:
In a paragraph of text, a <Hyperlink> or a <Bold> will be stripped of a preceding space. By contrast, a period following a </Hyperlink> or a </Bold> wil be placed on a new line. This will insert a [space] in the UI.

https://github.com/Xavalon/XamlStyler/wiki/Getting-Started#whitespace

I can do this 😇

Or... Let the workflow do this.
Like https://github.com/files-community/Files/issues/13396

@chingucoding any thoughts?

Thanks so much for tackling this @Jay-o-Way! As you know already cleaning up the XAML is in the backlog as part of the efforts to clean up the codebase.
I like the idea of a Settings.XamlStyler file should we have future customizations, and so it's easier to reference implemented rules.

Thanks for the positive comment @karkarl :) Imma need a bit more specific answers to adress this, mainly the choice between a one-time (manual) execution or the workflow solution. (easy versus not-so-easy, resp.)

I think having a config for this is also necessary to avoid mixed formatting based on who is doing it.

Well, the default settings seem fine to start with. I think it's more a matter of: is the plug-in installed and being used in Visual Studio? But then, if it's done via a central workflow, this is a non-issue, because the changes are not made by people. Right? :)

is there a way to work around that formatting of text?

You mean the unintended changes with spaces? I believe setting the space=preserve is the solution.

@niels9001 Currently, there are a lot of RichTextBlock + Paragraph constructions. Mostly used for formatting text and including Hyperlinks. Adjusting them to add space=preserve, is a huge p.i.t.a. and the resulting xaml is not pretty. I think it would be a lot more efficient to use a MarkdownTextblock. wdyt?

Also, cases like the following might be too much XAML for simple text
<RichTextBlock
 x:Name="ErrorTextBlock"
 Grid.Row="1"
 Margin="0,0,0,12"
 Foreground="{ThemeResource SystemFillColorCriticalBrush}"
 IsTextSelectionEnabled="True"
 Visibility="Collapsed">
 <Paragraph xml:space="preserve">
 This sample requires a later version of Windows to be fully functional.
 </Paragraph>
</RichTextBlock>

Linking this to this task in backlog #1279

Example of pr: https://github.com/files-community/Files/pull/15115

@Jay-o-Way Were you still planning to look at this? 100% agree we should do this, because not having this makes reviewing XAML PRs a pain :(

Not really. Sometimes things seem easy at first glance, but are just insanely complex when you really dig into it. I honestly gave up a long time ago.


Ignore the fieldValue for Signature annotations, since there're currently unsupported (issue 10374)

Given that Signature (Widget) annotations are currently not supported, since they cannot be validated, simply ignoring the fieldValue seems OK for now considering that attempting to blindly include unparsed/unvalidated data isn't very useful.
Fixes #10347.

/botio lint
/botio unittest

/botio unittest

/botio lint

/botio unittest

I think this is fine too. Thank you for fixing this!


blog: document Code generated convention in the Generating code blog post

As the Go community has reached consensus on the convention to use for distinguishing machine-generated code (i.e. https://golang.org/s/generatedcode), it would be good if the Generating code article on the Go blog was updated to reflect this. Preferably a link could also be added from the Generating code article to https://golang.org/s/generatedcode
In particular, the Generating code article is still using the convention // generated by stringer -type Pill pill.go; DO NOT EDIT. Since the blog post is the first hit (for me at least) on Google which searching for generate code golang, the article should in my opinion be updated so that new Gophers learn the newly established convention.

Sounds good. Feel free to send.
But I wouldn't link to https://golang.org/s/generatedcode, at least until we have a better target URL for that shortlink.

Ok, I'll skip adding the link, but will send a CL for
-// generated by stringer -type Pill pill.go; DO NOT EDIT
+// Code generated by stringer -type Pill pill.go; DO NOT EDIT.

@bradfitz How far away is maintner from being able to convert GitHub pull requests to Gerrit CLs? I'd be happy to beta-test ;)

Far.

@mewmew - PRs work now. Want to send one ?

@mewmew - PRs work now. Want to send one ?

@agnivade Done :)


Problem with Shapely module?

Following instructions from https://developmentseed.org/blog/2018/01/19/sagemaker-label-maker-case/
Running label-maker labels --dest vietnam_building --config vietnam.json
got this error:
Retiling QA Tiles to zoom level 17 (takes a bit)
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/label_maker/stream_filter.py", line 4, in <module>
    from shapely.geometry import shape, box
ImportError: No module named shapely.geometry

Shapely==1.6.3 is installed
I'm on Macos
python -V
Python 3.6.4

@moonflash can you print the result of running pip freeze? Also are you running this from a virtual environment?

@drewbo thanks for feedback.
I'm new to Python (Ruby developer originally) and obviously didn't know enough about the environment.
Now when i've properly used pyenv and installed label-maker under python 3.6.3 it all works OK 👍


Strip the tag off of the images in the stack CR

Per comments in #386, we should strip the tags off of any images we read from the stack index yaml.  Having the tag there is confusing when the tag is only two digits (if there are two versions of the same collection active, and they both say for example "my-stack:0.2", then obviously one of them is wrong (they can't both be using the same image).

I am thinking the way I'd like to see this implemented, is:

when the Kabanero CR reads the stack index, and creates Stack CRs, the images should have their tag removed at that point.  So the images in the spec section of the Stack CR do not have tags.
For manually created Stack CRs, the stack mutating webhook should strip the tag off of the image.
When we copy the image from the Stack CR spec to the Stack CR status we should again strip the tag off.

Hopefully we can write a utility function that can do this in all three places.  I guess technically we should not have to do 1) since 2) will do it for us, and if not then 3) will do it for us.  I am just thinking it would be clearer for the user who may be paying attention, to be consistent across the board.


fix(qwikloader): stopPropagation for any async handlers

sync$ may not be async function and e.stopPropagation() won't work unless it's fired immediately
fixes https://github.com/QwikDev/qwik/issues/5322
used this repo with the fix in a custom qwik loader
https://github.com/PatrickJS/bug-qwik-loader-stopPropagation

ok so this really should fix this problem if you see here only the first sync fn with e.stopPropagation() will make it work correctly
https://qwik.dev/playground/#f=7VdNS8NAEP0rSw7tBtvQeJLgIh4F8SB6kh7UxNpLLdKDEPa%2F%2B2Znd5LdtEURtEKDtelkdna%2B9r1Jr2lOy7Nepbr6TxSdXnzhL7BvyHqWVilLI795qxtE3m8nkCuEMOs5BPld3TMD2R0JwuzhA%2FH8TCp84chj%2F0qsFQ75jAEd09YXKivXHwoYuqzVE8gqU1VftHhvmlU2EXPrx7pG8StVzoLQWn9D%2FD8lq1Paw7R067YLCjIKtA9i0CVP60YiCxd4sCkwdqyvwDn1EuHjcGP2YbBBebao7tMYUJ8GddCmHW2TZKJmeS4rbd6Fzp4yVmvUhweA3X7vc6Y%2Fe7GdAp%2FLDU4W5sdGj%2BNcjvPUQGgJLUmONH4r2L%2BP9KAqe%2FjBzvksBtwbnlGWPNM8B7xXo5EYiR4UGOK189KliXGIr3OHafSIIcAK9pg23FmJ1LThznZwDPjN2SF%2Br%2FJ4jE%2BMnndApwF6biD8GnqS5v6IfhgL1XfoNTFP7LQ45l%2F5vGVoBUbR3IhUKeCzB2Q37VfKIbHbJhiQkI2CdddGXTMnTOBscpPJE36NMKngRJWkGG1FOYRm5B7ReXDRZ7dSAfK7ExW01XXzEvS3r4nXDdamBhIj8%2F4T2%2F0Quci2%2BHa7XLzucC5d735jaOhNgq5BKUWmpf%2FfaR%2FfOem4huY5DpL%2Fb5D8BA

this fix wasn't the fix I'm looking into qwikloader

this might be a chrome bug
// Add a click event listener to the document body
document.body.addEventListener('click', function(e) {
  // Stop propagation
  e.stopPropagation();

  // Log cancelBubble value immediately after stopping propagation
  console.log('Immediate cancelBubble:', e.cancelBubble);

  // Set a timeout
  setTimeout(function() {
    // Log cancelBubble value after the timeout
    console.log('After setTimeout cancelBubble:', e.cancelBubble);
    // e.cancelBubble <--- should be true but it's false
  }, 1000);
});

LGTM 🙌 . Merging


Unable to use iframes (VegaLite) behind HTTPS proxy

Environment

Elixir & Erlang/OTP versions (elixir --version): Elixir 1.13.2 / Erlang/OTP 24
Operating system: x86_64-pc-linux-gnu
How have you started Livebook (mix phx.server, livebook CLI, Docker, etc): docker-compose build
Livebook version (use git rev-parse HEAD if running with mix): 70ccaafa14fc736e8c7c08ee29ba40250ef8f125
Browsers that reproduce this bug (the more the merrier): Safari
Include what is logged in the browser console: Refused to load https://livebook.space/iframe/v3.html because it does not appear in the child-src directive of the Content Security Policy.
Include what is logged to the server console: n/a

Current behavior
Prior to this commit: e5e13d86c25695f14f11e090efe21533066f9288, I was able to run LiveBook behind an HTTPS proxy server and utilize VegaLite (and other web-based libraries).
Expected behavior
I would expect to be able to define an origin source (that is internal to our network) for establishing the iframe in an HTTPS environment (see: https://github.com/livebook-dev/livebook/blob/62140eb8d75dd5345d1e425551a93df11257eeba/assets/js/hooks/js_view/iframe.js#L45).

We will gladly accept an option to make it customizable, but it means you will have to run the iframe yourself. The source code is in this repository too under iframe.

Similar iframe issue on localhost. Initilized by escripts, livebook 0.6
Unhandled Promise Rejection: Error: The loaded iframe content doesn't have the expected checksum
(anonymous function) — app.js:2:3477146

@Albert481 what operating system are you at?

macOS

Does it happen in other browsers or incognito mode?

@jonatanklosko this error can happen if the assets get cleaned I think, due to the macOS tmp directory behaviour. Perhaps we should do a new release with the fix?

@Albert481 can you please try running from main to see if it works? You can run: mix escript.install github livebook-dev/livebook

@josevalim the error is The loaded iframe content doesn't have the expected checksum, so it's about the iframe content, before we load the assets.

Ah, true.

both Safari and Google Chrome

I am already using the 0.6.1 ver actually. Still occuring.

And does it also happen when installing the escript from main? Or running directly from GitHub? Also, do you have anything running on 8001 by chance?

Both escripts and github. and no running processes on the default livebook iframe port (default 8081)

And if you go directly to http://localhost:8081/iframe/v3.html does it return a file or 404?
Also, please clone the repo and check the checksum with
cat iframe/priv/static/iframe/v3.html | openssl dgst -sha256 -binary | openssl base64 -A

And if you go directly to http://localhost:8081/iframe/v3.html does it return a file or 404?
Also, please clone the repo and check the checksum with
cat iframe/priv/static/iframe/v3.html | openssl dgst -sha256 -binary | openssl base64 -A

Shows a blank page, no console errors.

Shows a blank page, no console errors.

Interesting, a blank page indicates the iframe is actually served.
@Albert481 could you also check the checksum? :)

I'm thinking the smallest fix would be to deprecate the LIVEBOOK_IFRAME_PORT configuration setting in lieu of LIVEBOOK_IFRAME_URI.
The LIVEBOOK_IFRAME_URI setting would allow you to provide a full path to the v3.html file.  If defined, it would be returned -- if not, then the existing behavior would happen:
function getIframeUrl(iframeUri) {
  if (iframeUri) return iframeUri;

  return window.location.protocol === "https:"
    ? "https://livebook.space/iframe/v3.html"
    : `http://${window.location.hostname}:8081/iframe/v3.html`;
}

I'm not sure what the stance is on deprecating configuration settings.  If backwards compatibility is needed, we could add "iframeUri" to the initializeIframeSource function and just skip the call to getIframeUrl if defined.
Happy to submit a PR for either option (or if there's another option that I'm not thinking of).

I think we can have both options. LIVEBOOK_IFRAME_PORT is used to configure the local port for running the iframe. LIVEBOOK_IFRAME_URL is the URL to access. So LIVEBOOK_IFRAME_PORT should set LIVEBOOK_IFRAME_URL (if none is set).
A PR is welcome!

So LIVEBOOK_IFRAME_PORT should set LIVEBOOK_IFRAME_URL

We shouldn't set LIVEBOOK_IFRAME_URL to anything by default, because it is protocol dependent. We can just make the URL take precedence if configured, for completeness we should probably rewrite the protocol part to make sure it matches.


How to retrict or validate when input into textbox?

I want to only allow user input some characters. Ex: /^[A-Za-z0-9\s]$/. How to show message error or retrict input.
thanks!

Sorry, no such feature.


Support subscriptions

Looks like subscriptions made it into the spec.
While most users will implement subscription logic outside juniper (like how Facebook interfaces with a MQTT server handling the actual delivery and connections) it might be useful to add some integrations as optional features.

Wow, I hadn't seen that happen. It certainly complicates things :)
I'll have to look and see how different implementations solve this, but it seems that there are (at least) three parts to this feature:

Storing/managing active subscriptions.
An API to push messages into Juniper.
An API to get transformed messages from Juniper out to clients.

We could probably use Stream from the futures crate to represent the two latter points, while the first one is just a trait that you have to implement for the backing storage you're interested in.
For convenience, it would be nice to have some reasonable implementations for these points behind feature flags just like the Iron and Rocket integrations. What it'll be specifically depends on the state of respective crates - I haven't looked at any of the AMQP/MQTT or websockets crates that are available.

That sounds about right to me! The AMQP and MQTT crates look pretty early days sadly...

In graphene they just went with a consumer type interface where on next the user decides how the message is transported.

I've just prototyped subscriptions interface for my app. The basic idea is to rewrite subscription to a query using grapqhql-parser and execute the query each time underlying data is updated. This works fine except serializing request and parsing it again is surely quite costly.
Another finding is that subscriptions-transport-ws protocol fits serde model quite fine with some annotations (at least parts of the proto I needed so far).
So in the first iteration, I would say that API for subscriptions may look like this:
let (data, sub) = execute_with_subscription(...);

Where the function works similarly to execute, and data is result of a query. Except it's unclear what sub here means.
In the first iteration, it might even be true if request contains subscriptions. Meaning that you must manually inspect AST to find out events you need to subscribe to.

@tailhook hello I recently implemented subscriptions in python with observables,
here a sub could be an observable Stream that can both consume and send messages to the on_next method of the observable .. but the state of the rust observables library does not inspire confidence, it hasn't been touched in years.

I'm definitely of the opinion that subscriptions should be implemented as part of, or following, the effort to cut this system over to a futures based API. Doing subscriptions without futures seems like we would be asking for major performance problems.

Subscriptions is the feature I want to see implemented the most. Polling is not really ideal for most of my use cases of GraphQL. I agree with @thedodd and with async-await semantics on the way, it will be much easier to create a futures based API.

Really like rust, but to bad there is no support for subscriptions yet. For Clojure there is https://github.com/walmartlabs/lacinia which has subscriptions for some time now. But that was a lot easier, having already a good async web-socket implementation.

So after investigating, as of October 25 2018, it looks like the current status of subscriptions in juniper is that it is blocked by:

#2 (Interop with async) to use futures, blocked by:
futures crate reaching v0.3.0, (currently at  0.3.0-alpha.9), tracked/blocked by:
futures 0.3.0 milestone, blocked by:
(...various issues, some of which depend on other crates)

A lot of the futures 0.3 milestone stuff hasn't been discussed since March (though I didn't investigate commits in various forks/branches that might be working on it). It seems to me that we might be waiting a while for futures 0.3.0 to come out.
Is the best path forward for this project to start a branch with futures in their current 0.3.0-alpha.9 state to have a good forward-compatible base for implementing usable GraphQL Subscriptions support?
A reason this is huge to me is that it allows a complete GraphQL implementation in a resource-constrained embedded systems Linux environment with a good language like Rust, which would be awesome!

Here is the interface Apollo uses to keep the lib delivery agnostic yet allow people to implement subscriptions for various 3rd party protocols:
https://github.com/apollographql/graphql-subscriptions/blob/master/src/pubsub-engine.ts

Is there an ETA for this feature? What can we do to help get it unblocked?

@mtyrrell See https://areweasyncyet.rs/ for the blockers on the ergonomic language features that can help power subscriptions in the future.

We can totally implement subscriptions without tokio support though.
I will probably tackle this soon as I need it myself.

Any updates on this? Would the asynchronous changes needed for subscription support be able to be reused in moving us towards @live directive support as well?

The asynchronous syntax for rust is about to be locked down, final proposal
https://boats.gitlab.io/blog/post/await-decision/

Any updates on this? Adding support for this would be killer. Really like the work you guys have done with this package,
I have a apollo service talking to some iot devices and weve been thinking about switching it over to either the excellent gqlgen lib in go but also weighing graphql options in rust, having subscription support would deff be a big factor lending in rusts favor

+1 on subscription support.
Loving the implementation of juniper thus far.

Here are some design thoughts we are about to try to implement on top of async-await branch:
If resolving query/mutation results in a value/future, then resolving subscription represents a serie of values/futures, so results in an iterator/stream.
First, we need to add subscription support to GraphQLType and ability to have our own resolvers (and probably update macros, or introduce new #[juniper::subscription] macro).
Then, high-level processing logic on subscription request should look like that:
(stream may be replaced by iterator in sync environment)
got subscription request as `GraphQLRequest` --->
subscribe() --->
   request processed --->
   subscription handler -> Result<Stream, Error> {
         manager.get_stream("key")
   } --->
return element when got some from stream --->
server handles how element is returned --->
end subscription when stream ends (`Async::Ready(None)`).

request.subscribe() always returns Stream. If subscription handler returns something that is not Stream it will be internally wrapped into a trivial Stream.
And RootNode will now have to be like that: RootNode<Queries, Mutations, Subscriptions> (breaking change).

Seems fine. Would the stream be supported by a web socket, or are there other ideas to implement that part? In lucinia, a Clojure GraphQL server implementation a subscription is backed up with web sockets. It's for example possible to consume some data stream, and then send it by websocket to all the clients that subscribed to certain data. How would something like that work with juniper?

Just like the rest of Juniper, subscriptions should be agnostic over the transport medium.
It should only produce streams of messages. The support libraries for the different servers will handle implementing the actual transport, with websockets/ H2 server side events, long polling fallback, ....

@gklijs
We're using actix-web and pairing its WebSockets with such juniper subscription abstraction is quite a convenient. As WebSocket connection represents an Actor itself, all we need is just to inject returned Stream into Actor's Context and then we're able to handle it just like regular actix Messages.
I suppose other web-framework will propose near the same "abstraction glue".

It's for example possible to consume some data stream, and then send it by websocket to all the clients that subscribed to certain data. How would something like that work with juniper?

This thing, however, I suppose should be implemented on the other side of juniper, where actual subscription handlers are declared.
I mean, the decision to reuse some channel for all subscribers or do not do that is just an implementation detail and should not be hardcoded into juniper abstractions.

@theduke what do you think about design thoughts described above? Should we start poking with it to make some MVP at least?

+1 to transport agnostic subscriptions. I use WebRTC data channels to send
GraphQL subscriptions and a subscribe function that returns a stream would
be perfect for me.
I'd add that the message format that encapsulates the GraphQL responses
(Apollo calls this GRAPHQL_WS if memory serves and we should aim to be
compatible with this defacto standard) should also be implemented as a
transport agnostic stream (perhaps it consumes and wraps the unencoded
GraphQL stream like a Tokio Codec?).
On Tue, Sep 24, 2019, 7:41 AM Kai Ren<EMAIL_ADDRESS>wrote:

@theduke https://github.com/theduke what do you think about design
thoughts described above? Should we start poking with it to make some MVP
at least?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/graphql-rust/juniper/issues/54?email_source=notifications&email_token=AABDOIF5RNLYIJC2CEPN42TQLH4HFA5CNFSM4DPUQQP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7OB6OI#issuecomment-534519609,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABDOIAFRAM3UJDTR75BDDLQLH4HFANCNFSM4DPUQQPQ
.

Sounds entirely valid, library I was referring to can by used without middleware, but also has a 'default' implementation. Reason I'm asking all this is I would really like to compare my Clojure implementation with one in Rust.

I think a trait returning a Stream (and maybe optionally an AsyncStream from tokio?) would be a good way to start prototyping and playing around @tyranon.
Though would we need to support things like timeouts, backpressure, etc? If so it starts to look more like a Tower service.

@LegNeato

Though would we need to support things like timeouts, backpressure, etc? If so it starts to look more like a Tower service.

I don't think so. juniper is all about GraphQL schema definition and resolving. Timeouts and backpressure are just implementation details on both sides of GraphQL schema (either select(request.execute(schema, context), timeout) when receiving HTTP request, or backpressure baked into Stream returning on resolving GraphQL subscription).
While on-top-of-juniper crates like (juniper_rocket, etc) may provide such capabilities/abstractions if necessary/useful.

This has landed on master! Please check it out and give feedback on usage. Thanks a ton  @nWacky and @tyranron ! 🍻 🍾 ❤


support all post foxy distros

rolling would be the best place to start

leaving it open, until testing it on the hardware

Tested on Rolling and humble works fine!


Improve Paper component

Closes https://github.com/mui/mui-toolpad/issues/993

Allow building layouts inside Paper component with the new layout slot type
Force Paper to take full-width of its container to keep resizing logic predictable (as it would be in the page itself)

I made Paper full-width because the resizing breaks easily and becomes very finicky if the Paper width changes when new components are added inside. To be able to resize widths inside an inner layout with content-dependent width would add a lot of edge cases to the resizing logic and make it quite more complicated, so not sure if it's worth the effort of looking into, it might be a very difficult problem to solve or may not even have a good solution. I could try if we find it really important, but I feel like it would take significant time and effort.
https://user-images.githubusercontent.com/10789765/197014848-2538e6b6-e9c8-4859-a49d-0b95fb8de28f.mov

I think this is good now, it was only pending because of a bug we've fixed in the meantime.

Hey @apedroferreira I tested this around and I think I found a weird glitch 🤔 seems that everything is working as long as you are adding items, but I tried removing item and after that I could put anything into the paper component anymore 👇
https://user-images.githubusercontent.com/437214/197123767-bf13b990-cab2-4fb1-a670-1425ae64279f.mov

Hey @apedroferreira I tested this around and I think I found a weird glitch 🤔 seems that everything is working as long as you are adding items, but I tried removing item and after that I could put anything into the paper component anymore 👇
Screen.Recording.2022-10-21.at.09.01.25.mov

There must be a few more cases where the logic depends on something being a page but now I need to account for inner layouts too.
I'll figure it out, thanks for finding this!

Hey @apedroferreira I tested this around and I think I found a weird glitch

I've added a fix, seems good now


chore: use stable Jest 29 in tests

Cleaner than using an alpha 🙂

Thanks a lot!
Planning to release<EMAIL_ADDRESS>on monday when I'm back from travelling and have access to my laptop with the GPG key for commit signing again.

Awesome, looking forward to it!


mjx.ncon removed as of MuJoCo 3.1.5

Hi friends,
It appears that MuJoCo officially removed mjx.ncon (https://mujoco.readthedocs.io/en/latest/changelog.html#version-3-1-5-may-7-2024) and hence this line is broken:
https://github.com/google/brax/blob/b45760cfa6ba0c3567ae36f988766a8299853c49/brax/contact.py#L38
Should be a quick fix; perhaps L38 should be replaced by
mujoco.MjData(sys.mj_model).ncon

or something cleverer.

Hi @namheegordonkim , we have a release in the backlog that we'll push out shortly! Apologies

See https://github.com/google/brax/releases/tag/v0.10.4 for the latest release. Thanks for the ping @namheegordonkim


Invalid toml value errors are fairly verbose

This is a regression compared to toml-rs as found by cargo's test suite
---- build::cargo_compile_with_invalid_manifest2 stdout ----
running `/home/epage/src/personal/cargo/target/debug/cargo build`
thread 'build::cargo_compile_with_invalid_manifest2' panicked at '
test failed running `/home/epage/src/personal/cargo/target/debug/cargo build`
error: stderr did not match:
1   1     error: failed to parse manifest at `[..]`
2   2
3   3     Caused by:
4   4       could not parse input as TOML
5   5
6   6     Caused by:
7        -  invalid TOML value, did you mean to use a quoted string? at line 3 column 23
    7    +  TOML parse error at line 3, column 23
    8    +    |
    9    +  3 |                 foo = bar
    10   +    |                       ^
    11   +  Unexpected `b`
    12   +  Expected `-`, `+`, `inf`, `nan`, `0x`, `0o` or `0b`
    13   +  expected 4 more elements
    14   +  expected 2 more elements
    15   +  While parsing a Time
    16   +  While parsing a hexadecimal Integer
    17   +  While parsing a octal Integer
    18   +  While parsing a binary Integer
    19   +  While parsing an Integer
    20   +  While parsing a Date-Time
    21   +  While parsing a Float

#279 fixed this specific case.


PT-1686 | feat: move "Enrol to queue" button under the occurrences table

Description :sparkles:
feat: move "Enrol to queue" button under the occurrences table
refs PT-1686
Issues :bug:
Closes :no_good_woman:
PT-1686
Related :handshake:
Testing :alembic:
https://kultus-ui-pr287.dev.hel.ninja/
Automated tests :gear:
Manual testing :construction_worker_man:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

It's not so simple, to just move it, because there are different kind of event types and states.
I can make another pull request out of that, since you had issues to setup the local env.

Ok, thanks. I was quite frustrated with all of the environments (local & testing & review) being broken earlier. I'll continue on LIIKUNTA-522 and you can take the ball on this one.

It's not so simple, to just move it, because there are different kind of event types and states.
I can make another pull request out of that, since you had issues to setup the local env.

Ok, thanks. I was quite frustrated with all of the environments (local & testing & review) being broken earlier. I'll continue on LIIKUNTA-522 and you can take the ball on this one.

the new PR to refactor this #288.


[HFFS] Hanlde "refs/convert/parquet" and "refs/pr/(\d)+" (PRs) revision correctly

Mentioned in https://github.com/huggingface/huggingface_hub/issues/1707#issuecomment-1746431572.
In HfFileSystem, revision with a "/" doesn't work properly if the / are not url-encoded properly.
Example
<EMAIL_ADDRESS>
is not a valid path to file ./default/train/000.parquet in revision refs/convert/parquet.
The correct path is
<EMAIL_ADDRESS>
Since refs/convert/parquet is a quite special revision, let's handle it separately (same for PR revisions like refs/pr/1).
Note: this will make repos with a file refs/convert/parquet/default/train/000.parquet less convenient to request. However, this is very unlikely to happen compared to the normal use case of revision.
cc @mariosasko  @Hakimovich99

Note that you can use the ~parquet alias:
dd.read_parquet("hf://datasets/jamescalam/llama-2-arxiv-papers-chunked@~parquet")

Resolved by https://github.com/huggingface/huggingface_hub/pull/1712.


Evaluation for R

Hi Scott @swo and Ed @eschrom, this branch is inherited from fy_eval with R files only. No need to review the code as it was the same as in fy-eval, and the suggested changes have been made in the python branch fy_eval_py.

Close in favor of #15


feat(clustering/rpc): add support for cert details

Summary
KAG-6084
This PR follows #14046.
Checklist

[ ] The Pull Request has tests
[ ] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix #[issue number]

Should we update the spec file as well?

No, we did not change rpc spec.


High memory usage, may have memory leak

panamax version: 1.0.6
Operating system: Ubuntu 20.04 LTS in WSL2
I start panamax on a new directory, synced around 150k crates out of 588629 crates,
then I interrupt it.
When I run panamax sync on that directory again,
the memory usage keeps increasing, reaching 30GB+,
and OOM my PC, so Windows OS kills the WSL2
It seems to have a memory leak issue.
I do not think panamax needs so much memory
Contents of mirror.html
# This is a Panamax mirror. It is a self-contained directory made to be easily copied
# to an offline network or machine via rsync, USB, or another method.

# When offline, Panamax also includes a "serve" command that can be used to serve
# rustup and cargo clients from the mirror. This will also give setup instructions
# on the homepage.

[mirror]
# Global mirror settings.

# Number of download retries before giving up.
retries = 5

# Contact information for the user agent.
# This is entirely optional, and is not required for the crates.io CDN.
# You may want to set this if you are mirroring from somewhere else.
# contact =<EMAIL_ADDRESS>

[rustup]
# These are the configuration parameters for the rustup half of the mirror.
# This will download the rustup-init files, as well as all components needed
# to run Rust on a machine.

# Perform rustup synchronization. Set this to false if you only want to mirror crates.io.
sync = false

# Whether to mirror XZ archives. These archives are more efficiently compressed
# than the GZ archives, and rustup uses them by default.
download_xz = true
# Whether to mirror GZ archives, for further backwards compatibility with rustup.
download_gz = false

# Number of downloads that can be ran in parallel.
download_threads = 128

# Where to download rustup files from.
source = "https://rsproxy.cn"

# How many historical versions of Rust to keep.
# Setting these to 1 will keep only the latest version.
# Setting these to 2 or higher will keep the latest version, as well as historical versions.
# Setting these to 0 will stop Panamax from downloading the release entirely.
# Removing the line will keep all release versions.
keep_latest_stables = 1
keep_latest_betas = 1
keep_latest_nightlies = 1

# Pinned versions of Rust to download and keep alongside latest stable/beta/nightly
# Version specifiers should be in the rustup toolchain format:
#
# <channel>[-<date>][-<host>]
#
# <channel>       = stable|beta|nightly|<major.minor>|<major.minor.patch>
# <date>          = YYYY-MM-DD
# <host>          = <target-triple>
#
# e.g. valid versions could be "1.42", "1.42.0", and "nightly-2014-12-18"
# Uncomment the following lines to pin extra rust versions:

#pinned_rust_versions = [
#    "1.42"
#]

# UNIX platforms to include in the mirror
# Uncomment the following lines to limit which platforms get downloaded.
# This affects both rustup-inits and components.

# platforms_unix = [
#     "arm-unknown-linux-gnueabi",
#     "x86_64-unknown-linux-gnu",
#     "x86_64-unknown-linux-musl",
# ]

# Windows platforms to include in the mirror
# Uncomment the following lines to limit which platforms get downloaded.
# This affects both rustup-inits and components.

# platforms_windows = [
#     "x86_64-pc-windows-gnu",
#     "x86_64-pc-windows-msvc",
# ]

# Whether to download the rustc-dev component.
# This component isn't always needed, so setting this to false can save lots of space.
download_dev = false

[crates]
# These are the configuration parameters for the crates.io half of the mirror.
# This will download the crates.io-index, as well as the crates themselves.
# Once downloaded, it will then (optionally) rewrite the config.json to point to your mirror.

# Perform crates synchronization. Set this to false if you only want to mirror rustup.
sync = true

# Number of downloads that can be ran in parallel.
download_threads = 64

# Where to download the crates from.
# The default, "https://crates.io/api/v1/crates", will actually instead use the corresponding
# url at https://static.crates.io in order to avoid a redirect and rate limiting
source = "https://rsproxy.cn/api/v1/crates"

# Where to clone the crates.io-index repository from.
source_index = "http://rsproxy.cn/crates.io-index"

# URL where this mirror's crates directory can be accessed from.
# Used for rewriting crates.io-index's config.json.
# Remove this parameter to perform no rewriting.
# If removed, the `panamax rewrite` command can be used later.
base_url = "http://<IP_ADDRESS>:3000/crates"

Oh no, that's not good. Thanks for bringing this up - I'll look into it.

Do you still have this issue on the latest version? I'm running on Linux and my memory usage stays around 450MB on a new directory.

This could be specific in WSL2
I checked the memory usage in /proc/pid/status, the VmRSS is around 450MB
but the "vmmem" process memory usage showed by the Task manager in Windows > 20GB

Oh! Yes, that is very WSL2-specific.
Basically, wsl2's virtual machine is set to use a maximum amount of ram (I think by default, it's half the available ram), and it will grow until it reaches the max. On windows, WSL2 is little more than a Hyper-V VM.
On the Linux side however, that ram usage will increase when Linux fills up its file caches, and the windows side can't tell the difference.
https://www.aleksandrhovhannisyan.com/blog/limiting-memory-usage-in-wsl-2/ this is a good resource for limiting the ram that WSL2 uses.


Include Windows UCRT in Windows build

Older versions of Windows needs to have the Windows Universal C++ Runtime packaged with the binary to work. This PR adds to the build_wine.sh build script to download the DLLs for the UCRT and package them with the Windows build.
Fixes #150

tACK on win7. (Tested as I tested the binary you provided here: https://github.com/bitcoin-core/HWI/issues/150#issuecomment-485052324)
Also tACK that it doesn't ruin win10.


Roles

Users can have different roles in an event.
Add a form to match a user to a role (from a predefined list of roles) in the context of an edition.
You can add or edit a role from the edition's page.
The roles are listed both on the edition page and on a users profile page.
Maria:

event creator
event coordinator
member of the organising team
participant

Marius - I would go for:

organiser - makes sure the event stays on track
teacher - presents something to the participants
mentor - work side by side with the participants
participant

@maria what do you think ? I would like to ask Alex to handle this issue, on CDL, this week.

Maria:

event creator this is a default, all events have creators. it's not necessary a role.
event coordinator == organiser
member of the organising team == teacher + mentor + anything else
participant

I would go with:

lead organiser
organiser
mentor
mente

I don't see why we should track the creator (inventor) of an event. The first two roles (lead organiser and organiser) and the next two (mentor and mentee) sound too much the same and it's hard to follow them. They seem to define the same category of people, so I would stick to this 'divisions' and to their initial names.

I don't see why we should track the creator (inventor) of an event.

I meant that now we show the person who created the event , and I think we should separate this person from the actual roles in the projects. For example I'm creating events even though I don't have a role in them.

If we are speaking about events. I think we should add roles per events, not per editions. but we should listen to other opinions. @fullvlad @Aleks23 ?


khal fails to install with pip >= 6.0.7

Either of the recommended installation methods aren't working for me:
$ pip install --user khal
Collecting khal
  Using cached khal-0.3.1.tar.gz
    wrote khal/version.py with version=0.3.1
    error: Invalid distribution name or version syntax: khal-0.3.1
    Complete output from command python setup.py egg_info:
    wrote khal/version.py with version=0.3.1

    running egg_info

    error: Invalid distribution name or version syntax: khal-0.3.1

    ----------------------------------------
    Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-DuLtVo/khal

Or via git:
$ pip install --user git+git://github.com/geier/khal.git
Collecting git+git://github.com/geier/khal.git
  Cloning git://github.com/geier/khal.git to /tmp/pip-GAut25-build
    local/lib/python2.7/site-packages/setuptools/dist.py:298: UserWarning: The version specified ('0.4.0.dev-145-gd81298d') is an invalid version, this may not work as expected with newer versions of setuptools, pip, and PyPI. Please see PEP 440 for more details.
      "details." % self.metadata.version
    wrote khal/version.py with version=0.4.0.dev-145-gd81298d
    error: Invalid distribution name or version syntax: khal-0.4.0.dev-145-gd81298d
    Complete output from command python setup.py egg_info:
    lib/python2.7/site-packages/setuptools/dist.py:298: UserWarning: The version specified ('0.4.0.dev-145-gd81298d') is an invalid version, this may not work as expected with newer versions of setuptools, pip, and PyPI. Please see PEP 440 for more details.

      "details." % self.metadata.version

    wrote khal/version.py with version=0.4.0.dev-145-gd81298d

    running egg_info

    error: Invalid distribution name or version syntax: khal-0.4.0.dev-145-gd81298d

    ----------------------------------------
    Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-GAut25-build

Am I doing something wrong?

Hmm, I'll really have to have a look at that pep and then fix the version number creation.
For now you might be able to install from the git version after you delete the .git directory.

I would recommend removing version.py, giving up on dev versions.

strangely, it is working for me with
% pip --version
pip 6.0.7 from /home/cg/.virtualenvs/khal_pip/local/lib/python2.7/site-packages (python 2.7)
% pip install git+git://github.com/geier/khal.git

...

Successfully installed atomicwrites-0.1.0 click-3.3 configobj-5.0.6 icalendar-3.8.4 khal-0.4.0.dev-146-g459751d lxml-3.4.1 python-dateutil-2.4.0 pytz-2014.10 pyxdg-0.25 requests-2.5.1 requests-toolbelt-0.3.1 six-1.9.0 tzlocal-1.1.2 urwid-1.3.0 vdirsyncer-0.4.2

What version of python are you on?

@untitaker I believe there only ever should be exactly one commit with release number x.y.z. So while the hash part of the dev version might have to go, something to make development versions recognizable will stay.

@geier Actually, now it just works, be it in a virtualenv or with the --user switch. I don't know what went wrong before. Closing this.
My configuration for reference (should this pop up again).
$ python -V
Python 2.7.8
$ apt-cache policy python
python:
  Installed: 2.7.8-1
  Candidate: 2.7.8-1
  Version table:
 *** 2.7.8-1 0
        500 http://archive.ubuntu.com/ubuntu/ utopic/main amd64 Packages
        100 /var/lib/dpkg/status
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 14.10
Release:	14.10
Codename:	utopic


[iOS] Receiving virtual file may be doing unnecessary file copy

Reader should try to open the file in place if possible.

Closed via https://github.com/superlistapp/super_native_extensions/pull/55


Factory::create assign empty string instead of null

In a project I had a foreign key who should be null are a valid id, I notice TestDummy doesn't assign a valid null when creating but assign an empty string ("").
I reproduce this issues on a simple exemple:
Migration
Schema::create('posts', function(Blueprint $table)
{
	$table->increments('id');
	$table->string('title')->nullable();
	$table->timestamps();
});

Model
class Post extends \Eloquent {
	protected $fillable = ['title'];
}

UnitTest
public function testBasicExample()
{
     $post = TestDummy::create("Post",['title'=>null]);
     $this->assertNull($post->title);
}

Result
1) ExampleTest::testBasicExample
Failed asserting that '' is null.

Having the same issue... Found a solution?

Can someone submit a PR for this?

I just try with Laravel 5 and last testdummy build and now it works fine.
factories.php
$factory('App\Post', [
'title' => 'whatever'
]);
ExampleTest.php
$post = Factory::create("App\Post",[
'title' => null
]);
$this->assertNull($post->title);
Result
OK (1 test, 1 assertion)


Adding support for short version of intrinsic functions missing from goformation

Missing support for functions in our intrinsicsolver module:

!Ref
!GetAtt
!ImportValue

The first approximation was proposed in #82, where those functions are treated as not yet solvable and thus omited by parser. If those functions would be fixed someday inside goformation library, this approach prevents us from the coding crossfire.


how to defined ScrollMagic server side

How can we defined scroll magic at server side level
https://findmy-web.herokuapp.com/work here is  the site
scrollscene used in work page for tiles animation but on page reload/refresh all tiles are hide due to there is no scrollscene SSR
note:- Have tried with scrollmagic also but its shown "window is not defined"
how can we fixed this
Code
import { gsap, TimelineMax, TweenMax, Power1, Power4, Power0 } from 'gsap'; import { ScrollScene } from 'scrollscene';

ScrollMagic is a client-side based library. You can't run this on the server side. You have to run it on the client.
I gave examples for this in the README: https://github.com/jonkwheeler/ScrollScene#using-react


Added cover

Pull Request Test Coverage Report for Build 817123315

163 of 163 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 99.064%

Totals

Change from base Build 816661853:
0.7%

Covered Lines:
438

Relevant Lines:
442

💛  - Coveralls


Added quest requirement to 'Client of Kourend'

Quest 'Client of Kourend' requires completion of 'X Marks the Spot'.
Source: https://oldschool.runescape.wiki/w/Client_of_Kourend (also confirmed in-game)

I like, I like 😉

Thanks, Dad 😊


promisify: pad arguments with undefined if few are passed

This fixes an issue with getStats().
If you call getStats() with no arguments, promisify will mistakenly call getStats with the successFunction as the track and failureFunction as successFunction. By padding out the argument list this issue is avoided.

@zxcpoiu PTAL.

sure, will take a look

@zxcpoiu Why wait for M69? It's not even stable yet. I've been testing it.

Oh, I saw you are actively updating things and thought you might want to avoid to mess around the code base for merge's sake. :smile:


[Snyk] Security upgrade netlify-cli from 15.9.1 to 16.9.3

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

package.json

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.

⚠ Warning
```
Failed to update the yarn.lock, please update manually before merging.
```

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Regular Expression Denial of Service (ReDoS) SNYK-JS-ASYNC-7414156
  631

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

👷 Deploy Preview for resonant-biscochitos-ee6470 processing.

Name
Link

🔨 Latest commit
3752a3487ef719e78dc33261f8a235f10bf8876d

🔍 Latest deploy log
https://app.netlify.com/sites/resonant-biscochitos-ee6470/deploys/6684e2b8e3c19b0008a3ff09


Remove casks and recipes not listed

Removes casks and recipes that are not listed

I don't want to do this, in case I've temporarily installed a cask before adding it to my dotfiles!


Install software icon request

🎨 Icon Request
📌 Request Background
Systems application :: Users will click install software button to initiate the new, streamlined software installation flows
🧪 Research
I kinda like the simplicity of this one on the noun project:

🥅 Acceptance Criteria

[ ] Icon name approved by PO and owners
[ ] Icon reviewed by design team
[ ] Icon available in Nimble_Iconography (link)
[ ] Icon published in Nimble Storybook (link)

@TJ-G @jattasNI
My first instinct is we should be using our standard download icon rather than creating a new one.  What are we trying to convey with a new metaphor?  Especially looking at this inspiration icon, it looks like someone's unique interpretation of the standard download icon and doesn't provide any additional value about specifics of the download through its visual design.
If we aren't using this in tandem with another option to download differently, what are the thoughts behind needing a new one?

@TJ-G @jattasNI
Traditionally we've used the icons download and install interchangeably, but I'm happy to update them so they are unique.

Here's an elegant solution that works well with our existing assets.  The subtle difference you'll see as a standard between download and install (when not drawn exactly the same) is install will be a package or abstractly drawn as a box.

@jattasNI @fredvisser @m-akinc
Please add these two icons into the token library.
arrow-down-box_16x16
Tags:  install, rectangle, arrow, down, drive, package

arrow-up-box_16x16
Tags:  uninstall, rectangle, arrow, up, drive, package

@TJ-G these icons are available in Nimble now


Use items as strings if displayProperty is an empty string.

Example
  <tags-input ng-model="book.tags" display-property="">
    <auto-complete source="searchTags($query)"></auto-complete>
  </tags-input>

If searchTags($query) return strings and we select item, then get in book.tags {text: "..."}, but need "..."
PS Sorry for bad English.

Can this issue be closed?

It is not always convenient. The requirement for a specific structure of the model is not very good.
But currently the problem for me is not relevant.

There will always be a requirement for some specific structure, and it will never be convenient for every one. In fact, the first version of ngTagsInput worked only with array of strings, but it quickly became clear that that was too restrictive. You can easily map an array of objects into an array of strings (as I showed above), but not the other way around.
Anyway, I'm closing this issue since you said it's not important to you right now.

I thing, it is an importtant features. I also wasted whole day finding out the solution.


Implement Key base class

Description:
This PR implements the Key base class and updates processing for classes that derive from it.
This contains work from #311, so that should be reviewed first.
Related issue(s):
Fixes #312
Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

Here are changes in more than 90 files. I will continue my review once PR #311 has been merged or rebased with the main branch.


Remove unused enable_post_deploy property

The default for the enable_post_deploy property was changed from false to true in commit 8fbf0457d48fbf6ab1223ef5e9134585cbf43f57. However, as part of that commit, the property was removed from the director.yml config file template. The value also defaulted to true in the Config class. With these two factors combined, enable_post_deploy has been unconfigurable and always true.
This commit removes the property entirely rather than add it back to the config file template, since it hasn't been configurable in several years without complaint.
[#187358662]

Thanks! @ystros


Kudo vs. IaC doc

Adds a comparison to IaC tools such as Terraform and Ansible.

Closes #132

fixed


chore: release 1.0.2

:robot: I have created a release *beep* *boop*
1.0.2 (2019-06-14)
Bug Fixes

docs: move to new client docs URL (#39) (debea92)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-containeranalysis/releases/tag/v1.0.2


make marmotta host configurable in collection service deployment

It could be useful to have the ability to expose the collection service's LDP implementation on an external port. Hostname and port for the linked marmotta container should be configurable.

port is configurable. decided host isn't necessary, as it will be a local linked container in the test bed setup.


Get the text in tiles evenly spaced vartically

This css issue is singled out as this is something that Colin mentioned specifically.

I think this might have to wait for Rui's updated design?

On Rui design these are not evenly spaced!


Rust SDK

[ ] Create a Rust SDK that allows developers to easily integrate Chimoney's payment processing capabilities into their Rust applications.
[ ] Ensure compatibility with various versions of the Rust compiler and runtime.
[ ] Provide support for popular Rust frameworks and libraries, such as Actix and Rocket.
[ ] Implement comprehensive error handling and exception management mechanisms.
[ ] Optimize SDK performance for web applications and real-time data processing.
[ ] Develop a user-friendly interface for integrating Chimoney's global payout capabilities into Rust applications

Hey @phyleria
please assign me to this task


need help

```hi new to linux and just wondering if it possible to get the steps needed to install linux. i am not sure if would download code a and just downlad the zip file or there some other file i need

thanks

First read this
When crouton has been installed, try this:
sudo CROUTON_BRANCH=silence crouton -r xenial - xfce
The issues tab is not for the purpose of teaching newbies how to use Linux. It is for reporting errors in crouton itself. You could try the discussions page. Also, read everything you can about Linux in general.

@newuser3456, @CroutonIsFun,
Here's a slight change to the above command, it needs the -t option for the 'target':

sudo CROUTON_BRANCH=silence crouton -r xenial -t xfce

-DennisLfromGA


Scoop doesn't install

So i have looked at #118 and #885 before opening this issue. After installing scoop if i run scoop help or anything else it says "scoop : The term 'scoop' is not recognized as the name of a cmdlet, function,
script file, or operable program. Check the spelling of the name, or if a path
was included, verify that the path is correct and try again."
I tried running this script that was posted in the other issue
iex (new-object net.webclient).downloadstring('https://gist.githubusercontent.com/lukesampson/210853bdace8e381ff1cb27ef37c8be9/raw/585d44db18b76537e532f16b6e8928587954c5dc/scoop-reinstall.ps1')
It runs trough fine but same problem as before.
I tried looking for the scoop folder in C:\Users****\appdata\local> and it is not there.

You should find it in C:\Users\<name>\scoop (%USERPROFILE%) and not in %LOCALAPPDATA%.

I had the same issue. I followed the tutorial on the Scoop page and it gave me the same error.
Then I tried to just open the scoop.cmd file through my normal command prompt (that is without using powershell) and it worked. It gave me the same error, but was still working properly afterwards though.

I had the same issue.
I did the following:

I deleted the contents of the scoop directory,
Enabled powershell for current user : set-executionpolicy remotesigned -s currentuser,
Installed scoop : iex (new-object net.webclient).downloadstring('https://get.scoop.sh')

Just like it is mentioned in the Readme for this repository. The important thing is deleting files that were created due to install without enabling powershell for current user before installing again.

This appears to be resolved. If not, let us know. Thanks.


Fixed transparent navbar issue

Signed-off-by: Nikhil-Ladha<EMAIL_ADDRESS>Description
This PR fixes #326 , fixes #325
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

It works in my tests on a mobile phone.

It works in my tests on a mobile phone.

The fix or the current site?

It works in my tests on a mobile phone.

The fix or the current site?

Sorry, I meant to say the Fixx Works. Whereas the current site menu does not work well on mobile


add updated paleo timings files

NZNSHM_paleotimings_GEOLOGICsliprates_all_2feb.txt
NZNSHM_paleotimings_GEODETICGEOLOGICPRIORsliprates_all_2feb.txt
NZNSHM_paleotimings_GEODETICsliprates_all_2feb.txt
as provided by Chris Rollins

@voj one more file coming FYI

@chrisbc files in issue have been updated


Confusing optional dependency when altering columns

Laravel Version: 5.6
PHP Version: 7.2
Database Driver & Version: PostgreSQL 10.x (unrelated)

Description:
If the user tries to write a migration following their guts and IDE auto-completion, they might stumble upon a "weird" dependency issue that's not solvable by running composer install again ("who knows, maybe the other package I included caused some unsolved conflict?).
According to a small section in the migrations documentation, you need to require doctrine/dbal for changes in columns to work. And that seems to be mentioned during initial installation as well (probably, together with a lot of other package recommendations). However, that's simply not enough for a couple of cases, such as:

entering a project where that dependency wasn't needed/included before (my case)
simply overlooking the list of package suggestions

Then, the user is confronted with a generic exception stating something is not where it should be.
My suggestion is to create a try..catch for such cases that would return a more comprehensive exception, stating the issue is probably caused by a missing optional dependency called doctrine/dbal. Bonus points if there was a link to the missed doc section.
It's fairly easy to find a bunch of developers stumbling upon the same issue: #785, #3116, SO 1, SO 2, Laracasts Forum 1, Laracasts Forum 2
Steps To Reproduce:

Install a fresh Laravel project
Create a migration that rename a column (say, rename users.name to users.full_name
Try to run migration
Receive a generic error: Symfony\Component\Debug\Exception\FatalThrowableError  : Class 'Doctrine\DBAL\Driver\PDOPgSql\Driver' not found

feel free to open a PR.


public interface update

List of related issues or pull requests

no reimports (refs #226)
removed cli from public interface (refs #230)
added readme (refs #240)

Describe the changes made in this pull request
changes to public interface
Instructions to review the pull request
rm -r howfairis.egg-info

# (virtual env activated)
pip install --no-cache-dir --editable .[dev]

# to verify:
bash clitests/script.sh
pytest
pytest livetests/

livetests status on this branch

Tests that ran before hitting rate limit are all green.

Been there :)


LD-531 - Adds correct margin/padding to last carousel image

https://artsyproduct.atlassian.net/browse/LD-531

Fails

:no_entry_sign:
TypeScript hasn't passed, see below for full logs

TypeScript Fails

  Generated by :no_entry_sign: dangerJS

Our TypeScript error-parsing is broken (see @orta's comment on #1553), but here's the problem:

src/lib/Scenes/Show/Components/ShowHeader/Components/Carousel.tsx(55,3): error TS1005: ',' expected.


BUG: Coerce GenericSeq path attributes to str

Hoping this will fix CI failures

Codecov Report
Patch and project coverage have no change.

Comparison is base (970be2f) 94.10% compared to head (c1ef7a5) 94.10%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #237   +/-   ##
=======================================
  Coverage   94.10%   94.10%
=======================================
  Files          41       41
  Lines        1510     1510
=======================================
  Hits         1421     1421
  Misses         89       89

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

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


Scrape prometheus application metrics with mtls enabled

Add istio certs into prometheus deployment so prometheus tls config can use them
Add  pod job to scrape istio/mtls pods securely
Update existing pod job to exclude pods contained in #2, and scrape normally.

Note: Since prometheus does not support istio secure naming, prometheus client is unable to verify server identity. This configuration enables metrics collection on the same port, however it is not suitable for true secure metrics scraping. That will follow in other PRs.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: sdake
If they are not already assigned, you can assign the PR to them by writing /assign @sdake in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

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

Addresses #5673 and partially addresses #7352
Uses label planned in #8601 and partially addresses

/assign @douglas-reid

@douglas-reid ping !


Update email styling to work on narrow devices.

Gmail web client screenshots.

Gmail mobile with narrow screen (this is a device that had the formatting problem originally reported).


"doas" support under OpenBSD

Description of the problem or steps to reproduce
When saving file without appropriate permissions
Specifications
enable save with "doas" instead of "sudo"

Is this just the case on OpenBSD or does it also apply to other flavors of BSD?

On OpenBSD doas is the default.
On FreeBSD and NetBSD not. There is su as default right after installation and sudo can be installed later.

On NetBSD su(1) can work like sudo.. depends on PAM settings.


[DO NOT MERGE] Auto PR to update dependencies of pilot

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
019e38ab7a842893065a8ee549cbf9c19c08cdbd
link
/test pilot-presubmit

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


Fix and normalize express permission names

Trying to install the last develop I get this error:
Whoops\Exception\ErrorException:
  Call to a member function setPermissionObject() on a non-object
  in file concrete/src/Tree/Type/ExpressEntryResults.php on line 69

That's because of wrong names of an express permission (sometimes they are used in singular form and sometimes they are used in plural form): let's normalize the names by always using the singular form.

The names are actually supposed to be plural. I have a fix for this in another branch, I will commit once I run tests on it.


Some issues report

Hello there, it's me again :)
After a week of usage, I've discovered some unexpected errors, so I'm back to report them.
1. Fails to handle the event keyword correctly during reorganization
As shown below, this is the original code:
using System;

public class Test
{
    public event Action action;
}

After reorganization:
using System;

public class Test
{
 action
}

As you can see, all keywords except the field itself have disappeared. This issue also occurs with Func or any custom delegates.
2. Nested classes are moved out of their container classes
As shown below, this is the original code:
public class Container
{
    public class Nested { }
}

After reorganization:
public class Container { }

public class Nested { }

3. Partial classes are merged
I'm not sure if this is an error or a feature, but I personally prefer to keep static members in a separate partial class, so the class merger is troublesome for me.
public partial class PartialClass
{
    public int fieldInSourceClass;
}

partial class PartialClass
{
    public int fieldInPartialClass;
}

After reorganization:
public partial class PartialClass
{
    public int fieldInPartialClass;
    public int fieldInSourceClass;
}

4. If a file defines one or more namespace, classes outside are merged into the namespace above them
namespace MyNamespace
{
    public class InNamespaceClass { }
}

public class OutNamespaceClass { }

namespace MyNamespace2
{
    public class InNamespaceClass { }
}

public class OutNamespaceClass2 { }

After reorganization:
namespace MyNamespace
{
    public class InNamespaceClass { }

    public class OutNamespaceClass { }
}

namespace MyNamespace2
{
    public class InNamespaceClass { }

    public class OutNamespaceClass2 { }
}

Although this situation shouldn't occur in normal programming workflows, I discovered it while writing a report, so I'm listing it here as well.
5. Classes not in a namespace seem to use the namespaceLevel settings during reorganization
Maybe my understanding is different, but I believe that even if not in a namespace, class members should belong to typeLevel.
Here I assume the following settings are used:
"csharpen.namespaceLevelOrganization": {
	"ordering": [
		"access",
		"type",
	],
},
"csharpen.typeLevelOrganization": {
	"ordering": [
		"type",
		"access",
	],
},

And here's the original code:
public class MyClass
{
    public int a;

    private int b;

    public void Method() { }
}

In my expectation, it should use the typeLevel settings, so after reorganization it should be:
public class MyClass
{
    public int a;
    private int b;

    public void Method() { }
}

But the actual result of sorting is as follows: (using the namespaceLevel settings)
public class MyClass
{
    public int a;

    public void Method() { }

    private int b;
}

btw, I sometimes get this error while reorganizing:

CSharpen: Maximum call stack size exceeded

It seems that the reorganization process is exceeding some call limit, but my code is correct and executable. Is there a way to adjust this maximum value?

I've seen the event issue before, just hadn't resolved it. I'll take a look at the issues you've provided. Thanks.

I should get to this this weekend.

As for issue 1 regarding event declarations: v0.0.6 has been released which fixes this.

I cannot reproduce issue 2. What are your settings?

As for issue 3 regarding partial class in same file being combined, VSCode reports the document symbols as one. Because the extension uses what is reported, this is how it is handled—neither a feature or a bug per se. But, I understand the desire to keep things separate as you explained. Unfortunately, for now at least, I won't address this as it will require a new mechanism of how the file is parsed.

Issue 5 regarding namespaceLevel handling classes outside a namespace: This is by design and expected. The terms I use of namespaceLevel and typeLevel can be deceiving, I know.
See "§14.6 Namespace member declarations" of https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/language-specification/grammar#a2-lexical-grammar.
Because a namespace_member_declaration can have both a namespace_declaration and type_declaration, I treat a top-level class (i.e. class not in a namespace), as I do a namespace in that namespace_member_declaration realm (sort of speak). Hopefully that makes sense.

Issue 4 regarding outside-of-namespace classes being included in namespaces: Fixed with just-released v0.0.7

I believe all have been address or otherwise by design.

For issue 2, here is my setting:

settings.json
"csharpen.namespaceLevelOrganization": {
		"ordering": [
			"access",
			"type",
			// "name"
		],
		"typeSort": [
			"delegate",
			"enum",
			"interface",
			"struct",
			"record",
			"record struct",
			"class",
		],
		"regionThreshold": 2
	},
	"csharpen.typeLevelOrganization": {
		"ordering": [
			"type",
			"access",
			// don't sort by name for unity
			// "name"
		],
		"typeSort": [
			"static constructor",
			"constructor",
			"finalizer",
			"constant",
			"field",
			"indexer",
			"property",
			"event",
			"method",
			"operator",
			// inner structures
			"delegate",
			"enum",
			"interface",
			"record struct",
			"struct",
			"record",
			"class",
		],
		"regionalization": [
			"type",
		],
		"regionThreshold": 2,
		"onlyRegionalizeMembersOf": [
			"record struct",
			"struct",
			"record",
			"class",
		],
		"onlyRegionalizeTypes": [
			"constructor",
			"constant",
			"field",
			"property",
			"method",
		]
	},

Btw, when verifying, I found that the latest version (v0.0.8) will deletes the 'p' from public. This might be an error caused by recent changes.
public class Nested
{
}

ublic class Container
{
}

Regarding Issue 5, I fully understand your intention to adhere to Microsoft's C# syntax specifications.
However, more than the potential misleading nature of naming, I'm concerned about the completeness of functionality.
I'm no expert, but here's my superficial thought:
Settings should serve functionality, like:

When I want to organize a class -> I define how to organize my types in the typeOrganization settings -> Done

When I want to organize all types within a namespace -> I define how to organize types within the namespace in the namespaceOrganization settings -> Done

Rather than being constrained by syntax specifications, leading to:

When I want to organize a class -> I define how to organize my types in the typeOrganization settings -> Because I also have types not in a namespace, I have to define how to organize my classes in the namespaceOrganization settings as well -> Done

When I want to organize all types within a namespace -> I want to define how to organize types within the namespace in the namespaceOrganization settings -> Then I realize this conflicts with the above situation, and I must choose to give up one or the other -> Failure

This is similar to CodeMaid's approach, where you set the organization logic once, and it applies the same principles to reorganize types whether they're inside or outside a namespace.
I'm not quite sure how to describe my thoughts, but I hope this example adequately expresses my idea.

Well shoot. I saw that trimming issue but thought I addressed it.
You described your thoughts clearly. I'll take a look at it. I agree the logical would make sense trading a class as a class whether on the namespace level or within.

Hmm, I can't reproduce the dropped p issue.

I've double-checked, and this error indeed occurs on my device.
The csharpen version and settings remain unchanged from what was previously mentioned, and command identifier is kokoabim.csharpen.sharpen-file.
This is my VS Code version:
CopyVersion: 1.91.1 (user setup)
Commit: f1e16e1e6214d7c44d078b1f0607b2388f29d729
Date: 2024-07-09T22:06:49.809Z
Electron: 29.4.0
ElectronBuildId: 9728852
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Windows_NT x64 10.0.22631

It seems to be a general error rather than one limited to Nested Classes. Here's a screenshot of the error:

If you need any additional information, please let me know. I'd be glad to provide it.

What version of the extension do you have?

By chance did this go away? I've never been able to reproduce this since v0.0.8, only prior versions.

I've done some more testing:
If other content gets reorganized (like adding an empty line at the beginning), the first reorganization works fine (just removes the empty line). But if you try to reorganize again after that, it deletes the 'p' of the public.
I also noticed that if the script has any namespace or using statements, the error doesn't happen.
Considering these case cover most scenarios, maybe it's a good idea to put this issue on hold for now if we can't consistently reproduce the error?
Let me know if you want me to try anything else or if you have any thoughts on this.

Stepped away for a bit and will revisit this. a PR has been submit but something else that may resolve this.


PERF: prepare single geometry input to contains_xy and intersects_xy

Closes #1734
See https://github.com/shapely/shapely/issues/1726#issuecomment-1399434147 for example of performance impact
Still need to update the docstring, if we want to do this.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 87.882%

Totals

Change from base Build<PHONE_NUMBER>2:
0.3%

Covered Lines:
2618

Relevant Lines:
2979

💛  - Coveralls


PowerShell script doesn't create the "MIMInstall" required user account

The PowerShell script on this page don't create the required "MIMInstall" user which is referred to later in the guide. The first script block should be updated to include:
New-ADUser –SamAccountName MIMINSTALL –name MIMINSTALL
Set-ADAccountPassword –identity MIMINSTALL –NewPassword $sp
Set-ADUser –identity MIMINSTALL –Enabled 1 –PasswordNeverExpires 1
Then, the second script block should be updated to include:
Add-ADGroupmember -identity MIMSyncAdmins -Members MIMInstall

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

ID: 44b72229-7157-8f7a-a937-d9e1035af4c2
Version Independent ID: 22ba54c4-069d-5c50-cade-edc27edd8581
Content: Set up a gMSAs for Microsoft Identity Manager 2016
Content Source: MIMDocs/preparing-domain-gmsa.md
Product: microsoft-identity-manager
GitHub Login: @EugeneSergeev
Microsoft Alias: esergeev

The following issue is being closed due to the age. If this is still an issue, please re-submit this. Thank you.
#please-close


Implement missing /source commands and tests

This API is used to list and add custom sources. It is not fully implemented and tested yet.
Get as close as possible to Lorax’ implementation. At the least, managing custom source in the UI and all composer-cli sources commands must work. Start by implementing the tests for lorax, so that we know what to aim for in the osbuild implementation.
API documentation

The missing routes are:
POST /api/v0/projects/source/new
DELETE /api/v0/projects/source/delete/<source-name>

This needs two things:

As @jkozol mentioned, sources have to be stored in the store now. Only store the user-added ones there, though (not the "base" one).
The ability to give multiple repositories to dnf-json. This is implemented in https://github.com/osbuild/osbuild-composer/pull/74, but most places in osbuild-composer still only pass one repository around.

Per #77 I added functionality for sources but don't currently use the sources to fetch our package list. I should add the packages we fetch to the store instead of generating them on each call for the package list correct? Should this list be only for the user defined sources or include the base repo's sources?

This has been completed with #103


[Bug] Running bash update.sh not fetching new icons/version.

The following error is output by terminal when running the script that auto-generates the new code & fetches the icons.
cp: temp_icons/icons/*.svg: No such file or directory
sed: Sources/Iconoir/Assets.xcassets/*.imageset/*.svg: No such file or directory
Processed *

@sammarks has anything changed recently that might cause this? Had directory structure changed in the main repository?

v6.11.0 vs v7.0.0 ,see the icons folder ，since v7.0.0， icons folder has some sub-folders cause we has different style icons regular+ solid,so the update.sh failed.
Maybe we should handle this changes,and upgrade the build result for SPM.

Ahh that's good to know. I'll take a gander tomorrow and see if I can get it updated.


Fix typo in /docs/01-start.md

traier -> trainer

Thanks!

When the branch will be merged
Asking because other merge commits may arise conflicts on this branch.


Confusing naming conventions

I find myself confused about the naming conventions/comments and you should probably be more precise here. If I understood it correctly, the channel_mean is the mean used in Channel-Wise RSC and computes the mean across the spatial domain while the spatial_mean is the mean used in Spatial-Wise RSC and computes the mean across the channel domain.
This is why we need F.interpolate in the Spatial-Wise RSC since it doesn't make sense for architectures with average pooling afterwards like mentioned here: #3. By the way: This isn't mentioned in the paper, what is the intuition behind that even?

Hi, different convolutional layers should be able to locate discriminative spatial information consistently. The high-level convolutional layer tends to capture more semantic concepts while the low-level layer tends to capture more details. That's why spatial interpolation can be used in a convolutional network with average pooling.


Update build.sh

Moved the hardcoded directory to at least C;\temp so the chance of overwriting existing files is reduced.  Perhaps a check for existing directores before wiping data would be advised.

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

Did you confirm that the path was short enough to build qt using vcpkg?

Can't confirm as can't complete build due to other build errors.
Perhaps a better PR would be a simple if [ -d "/path/to/directory" ]; then     echo "Directory exists. Delete or choose another buildtree"     break


add a method to return the created_at value from a signature request

When doing audits on outstanding signature requests via the API it would be useful to access the created_at value.
The added method returns a java.time.Instant as that would be the best way to return the seconds since epoch and makes it easier for the developer to convert it to a relevant time on their end using the standard java date/time library.
I would be happy to convert the other methods that currently return the old java.util.Date to the modern java.time.Instant

Thanks for the PR. I like the idea of using Instant however I don't want a one-off where we use instant when we're using Date elsewhere. See AbstractResource::getDate for how this is being used.
Thanks for the contribution, I've made a similar change in https://github.com/hellosign/hellosign-java-sdk/pull/179 to provide the same sort of support.

@jspaetzel Maybe this is something that can be done with a full version rev and convert everything away from the legacy Date API and to the standard modern one.


fix(superset-frontend): update Iran ISO 3166-2 codes

SUMMARY
The official ISO 3166-2 codes for the provinces of Iran have been updated. You can find the latest codes on the ISO website: https://www.iso.org/obp/ui/#iso:code:3166:IR.
This pull request updates the codes used in the chart country map plugin to reflect these changes.
Changes:

Update province codes.
Sort provinces by new ISO 3166-2 codes (IR-00 to IR-30).

We should make these changes in the Jupyter notebook, but not until another couple of PRs get merged first... this notebook doesn't always play nicely with version control.

No need to close the PR, as long as you're willing to rebase in a little bit and edit the notebook :)


Optimize path and scene

[ ] Optimize path and scene?

[ ] sometimes, we go the middle of triangles and that's really confusing for VSLAM

(kinda fixed)

[ ] add more triangles in the distance, currently it's just too weird on the edges
[ ] we go out of bounds at some point, no reason to do it

done!


${version} is not replaced in manifest.json files

Expected Behavior
When building my webapp, string "${version}" should be replace in manifest.json
Current Behavior
ui5-builder creates a
Steps to reproduce the issue

Checkout the official sample app https://github.com/SAP/openui5-sample-app/tree/ui5-tooling

Edit manifest.json file to:
"sap.app": {
"id": "sap.ui.demo.todo",
"type": "application",
"applicationVersion": {
"version": "${version}"
}
},

Run $ui5 build

Check the built version and open manifest.json file.

Context

OS/Platform: *
Node.js Version: *
npm Version: *
Other information: ui5-builder v0.2.1

Affected components

[x] ui5-builder
[ ] ui5-server
[ ] ui5-cli
[ ] ui5-fs
[ ] ui5-project
[ ] ui5-logger

Pending release

This has been fixed with ui5-builder release 0.2.3.
You can update a globally installed UI5 CLI to use the new version of UI5 Builder by executing npm install --global @ui5/cli. Please let us know if there are any issues.

@RandomByte thanks for that... Just one more thing, you haven't published it to NPM. Latest version there is v0.2.1
https://www.npmjs.com/package/@ui5/cli

You're right, thanks. @ui5/cli v0.2.2 just got released: https://github.com/SAP/ui5-cli/releases/tag/v0.2.2


Simplify logic to insert canvas as first element

Instead of conditionally checking if the .cavnasWrapper already has a child element and then inserting the canvas before it, we can use .prepend which always injects the new element as the first child.
https://developer.mozilla.org/en-US/docs/Web/API/Element/prepend

/botio integrationtest


the number of success and failure dot in scatter chart different from…

… success and failure dot count . #1732

@jaehong-kim review please :)


Loqed battery percentage in homey

It would be nice if we could get the battery percentage in Homey. This is so we can trigger if the battery is getting low.

You can already trigger low battery warning, but a viewable percentage would be even better

This is fixed in the new version. https://homey.app/a/com.loqed.touch-smart-lock/test/


Guidance on reducing heap usage of large typings?

I maintain google/schema-dts which generates typings for schema.org JSON-LD. The generated typings are ~10k loc and involves a lot of type unions (e.g. Thing can be Organization or CreativeWork or Person or ...), and these can get pretty nested.
Compiling these works for me and the vast majority of users. But I get constant reports of JavaScript heap OOM errors from users consuming the typings. See google/schema-dts#34. I haven't been able to repro this at all, but I'm still hoping for some guidance on getting a better memory footprint for the type checker.
FYI, the .d.ts file looks something like this<EMAIL_ADDRESS>Questions (I'd take answers to any of these):

Are there better strategies for writing these in a more memory friendly way? e.g. naming intermediate types, or declaring all properties in one huge object and defining respective types in terms of Pick<>?
Are there any profiling tools that anyone would recommend using to track this down?
any other advice is appreciated.

Unfortunately splitting the types up isn't an option (at least not obviously) because the types are deeply self referential.

Hey @Eyas, I saw your post on https://twitter.com/EyasSH/status/1257373760216281091. We do have some docs on https://github.com/microsoft/TypeScript/wiki/Performance, and @amcasey has a branch to diagnose which line of code is causing lots of types to be generated internally (https://github.com/microsoft/TypeScript/pull/37785). We're trying to understand what profiling tools would be useful for these situations.
I think one thing that is reasonable is to tell your users in the meantime to increase their heap sizes when running TypeScript. I know that in VS Code this can be done with "typescript.tsserver.maxTsServerMemory", and on Node I think this is done with --max-old-space-size.

Note that you'd want to run that instrumented build on the consumer of the .d.ts file to help figure out which parts are the most expensive for them.  In my experience, profiling the producer tends to reveal few, if any issues.

@Eyas I'm probably missing something obvious, but the npm scripts I see produce a bunch of separate .d.ts files instead of one big, consolidated one, like you linked above.  How can I produce one of those locally?

The packaged .d.ts is here: https://www.npmjs.com/package/schema-dts

If you're trying it from source, running "npm run pkg" will create the single .d.ts in "/dist/schema".

The multiple files you're seeing are the .d.ts files of the generator CLI. It makes an http request to an arbitrary .NTriples schema declaration (by default latest schema.org schema). That final output (TS defs of schema.org schema) is what contains huge unions that seem to be causing people to run into problems

@Eyas Thanks!  Yes, I'd like to be able to build it locally, so I can play around a bit.
We've definitely had some problems with huge unions, especially when they're operated on by mapped types.  Things should be somewhat better in 3.9 (now in RC).  The easiest way to play with the latest version is to install the TS Nightly extension.
It's something of an art, but I've definitely seen some impressive perf improvements from naming intermediate types (because it can short-circuit structural type comparison, which is very expensive).

Helpful to know about naming intermediate types! That also gives me a few avenues to try

Here's one I thought was particularly amazing (and for which I can take no credit): https://github.com/mui-org/material-ui/pull/19996

As for identifying where to apply this sort of optimization, we're still working on that, but https://github.com/microsoft/TypeScript/pull/37785 is our current best attempt.  (Looking now, I see that the bot silently failed to create a standalone build - I'll investigate.)

Probably worth mentioning: I have heard reports in other issues that the node version in use can heavily affect this. 8-10 predate v8 pointer compression (and other optimizations), notably, and can OOM in places where node 12+ would not. So you should also ask what node your users are running, if possible; it could be critical to the repro.

Got it. Thanks. I've seen reports on Node 10 and 12, although a lot of reports are missing Node version.

I ran #37785 on your example and it revealed precisely nothing.  So either this is a different class of slowdown than we were expecting or the tool is useless on .d.ts files.  I'll have to investigate more to figure out which.

@amcasey if it helps/narrows anything down, some users are reporting that TypeDoc is also OOMing with the linked library. Not sure if TypeDoc uses any of the typescript lib when running through types.


Understand why we need to set up spec.ports in the ingress controller when it is already being set by the ConfigMap

The approach to open the ports is:
https://kubernetes.github.io/ingress-nginx/user-guide/exposing-tcp-udp-services/

Matt found this link and I just exactly followed the steps in this link :)
https://stackoverflow.com/a/67146133

Closing after discussing with @taojing2002 and @mbjones. Explanation:
This issue originally arose from testing with metacat outside and the indexer inside the cluster, which meant we had to expose solr and rabbitmq ports through the ingress.
However, since we now have a helm chart for Metacat that includes the indexer as a sub-chart, the two most likely deployment scenarios will now be "everything inside the cluster" or "not kubernetes-based" (i.e. a legacy deployment). This, in turn, means that we no longer need to expose indexer-related ports externally, thus rendering this Issue (and Issue #36) moot.


Update one FAQ with the 2010 census tract language

Describe the bug
Add the following to the second question's answer
Census tracts are small units of geography. Census tract boundaries for statistical areas are determined by the U.S. Census Bureau once every ten years. The tool utilizes the census tract boundaries from 2010. This was chosen primarily because many of the data sources in the tool currently use the 2010 census boundaries.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Yes, but let's add.
The tool uses census tracts. Census tracts are small units of geography. Census tract boundaries for statistical areas are determined by the U.S. Census Bureau once every ten years. The tool utilizes the census tract boundaries from 2010. This was chosen primarily because many of the data sources in the tool currently use the 2010 census boundaries.


ARROW-4556: [Rust] Preserve JSON field order when inferring schema

Uses the preserve-order feature in serde_json, which is required to guarantee that we iterate through JSON records' keys in their provided order.
Adds crate indexmap, which serde_json already uses.

Travis failures are unrelated.