Consider moving chat to Gitter

A number of open source projects now use Gitter as their chat hangout over the more traditional IRC. I'm not crazy about moving from an open platform to a closed one but I think it may be worth the ideological sacrifice. I'll list some pros and cons:
Pros

Easy access to chat history from when you were away
Easy access on mobile (their app is pretty good)
We could probably move our existing HipChat integration to use their API
Uses our existing GitHub permissions
Lower barrier to entry
I personally already hang out in other Gitters ;)

Cons

Gitter is a closed platform :disappointed:

Thoughts? Especially @jonafato @spicyj and @divad12

:+1:

TL;DR: I'm fine with IRC, but I won't argue against Gitter much either.
Posting the comments I made in IRC (with some additions and modifications) here for better visibility:
The last couple of times I used Gitter, I wasn't a big fan. Overall, I found the app to be kind of sluggish (particularly when navigating between channels). Things may have improved since then, so I can't really speak to the current state of the app. As @captbaritone  notes above, the lack of openness is also a concern of mine.
That said, I recognize that it has a lower barrier to entry than IRC for many people, and there aren't a ton of other easy options. I'm optimistic about the direction of https://matrix.org/, and I like the concept of Dropbox's https://www.zulip.org/, but I haven't used either enough to recommend them as an alternative. I'm not opposed to using gitter in any principled sense, though, so I'm happy to lurk there if requested.
Ultimately, based on experience from other projects, very few people outside of core contributors will use the chat room. This means that it's best to pick whatever solution fits into the core team's workflow best and optimize for other factors secondarily and once they become relevant (i.e. avoid premature optimization). IRC is part of my normal development workflow (including ZNC to avoid missing messages), so I'm not itching to move off of it, and one extra channel is minimal effort for me. For others, this may not be the case, and it's probably easier to get started with Gitter than IRC. Ease of getting started is a really important factor here; if no one is going to put out the effort to sign into chat, all other arguments become moot.

@jonafato Thanks for your thoughts. All valid!
I've updated the Readme and made Gitter our main hangout with the understanding that if it fails to meet our needs we can easily move somewhere else.
https://github.com/divad12/vim-awesome/commit/d440d9970450b4def6c560d28d7094e373377211
Join me! https://gitter.im/divad12/vim-awesome

Great, thanks for setting that up @captbaritone! Let's see how it goes.
I'm open to either — I don't use either right now! Been too busy with other things to do much in open source land. And I'm handing this off to you and others who can spend the proper time and care to contribute and maintain to this project, so I leave it up to y'all!


BuildStream doesn't hit the cache when building foreign arches

See original issue on GitLab
In GitLab by [Gitlab user @bochecha] on Jul 27, 2018, 16:39
Summary
I'm building org.freedesktop.Sdk/arm/unstable on my x86_64 laptop. The exact same Git revision has already been built in the CI.
Yet BuildStream doesn't pull from the cache, it rebuilds everything instead.
Steps to reproduce

Set up qemu/binfmt on your x86_64 machine, so that building for ARM just works

git clone https://gitlab.com/freedesktop-sdk/freedesktop-sdk

cd freedesktop-sdk

git checkout bochecha/qemu-bash
this is the branch I'm trying to build, I won't force-push so that it remains stable, which will help test more easily than on a moving "master"

cd sdk

bst -o bootstrap_build_arch aarch64 -o target_arch arm build all.bst

What is the current bug behavior?
BuildStream will pull some things, and then proceed to rebuilding pretty much everything.
What is the expected correct behavior?
BuildStream should have pulled everything from the shared cache, since everything has already been built by our CI.
Relevant logs and/or screenshots
Here is an extract from the CI build logs (just a few random lines) :
fetch needed 40d8f66436c808e0d2803207cb611944d6bfa92df3e2149cf4ea8b4a6c2c8c68 bootstrap-junction.bst:coreutils.bst
     waiting 21ecc5bf231e2a2e4a8332f418729b413feb12c70d10e54df949beccff270767 bootstrap-junction.bst:diffutils.bst
fetch needed 2ada674ab2eb535bf757c75b883a73258b062c34a7eae357556cd01b257db414 bootstrap-junction.bst:files-stage1.bst
     waiting 459339a2b4c1db0948e7f3d9168f0bfd9dc06cea0f5d3a6d3929189cc6251973 bootstrap-junction.bst:zlib.bst
fetch needed 1a28b1601c89b3ffe1c106a25046b8dc66360ec48c9be47943e65c4917121fbe bootstrap-junction.bst:file.bst
     waiting f711791e4f17914f7abe0da876627d09bfca8b43f7d64195cbe4386b57a1a5e1 bootstrap-junction.bst:findutils.bst
fetch needed b1af27320610e2738c0ca6f99c71672bf8d8395003f549fe185ce63e5227384a bootstrap-junction.bst:gawk.bst
     waiting e92e604dbff0f1a6006775b3faed944c6f2a2b00d649c8b4f8c186f86e34cc64 bootstrap-junction.bst:gcc.bst

And here is an extract (the same 8 lines) from what I get on my laptop right now, running bst -o bootstrap_build_arch aarch64 -o target_arch arm build all.bst:
     waiting 495f1f280a527df0b1b0fa2a83b887a31228bb5efc941112eb9b9150dffd8c00 bootstrap-junction.bst:coreutils.bst
      cached ddb03abe3928bd9d7e2dc9d4e40d39ad0390aef293cebf6857be65a972c69872 bootstrap-junction.bst:diffutils.bst
     waiting 05000dd93ec34d64b6cad910a7d12c2ed6fb2f38c9e7417b4652634cfca93794 bootstrap-junction.bst:files-stage1.bst
      cached 887e19680fd9568f3387438e6720c0cbd2f5be08ba44c2b8023919c6b9193262 bootstrap-junction.bst:zlib.bst
     waiting 805f803d01a8151b1bb2259482d1a25e22b8343fdb09117c8b74f5aae1648ab7 bootstrap-junction.bst:file.bst
     waiting 375a0032b05c89619f022783960f6def87597de540a38f03fd52927d8421c814 bootstrap-junction.bst:findutils.bst
     waiting b9f24859a032a4faaada15c55762feed3e644caeea23910713911633fdaab2b3 bootstrap-junction.bst:gawk.bst
     waiting a77694f1bd40ada7fbf416dac1597aa49c1db65339081d1770798dd3fae140d4 bootstrap-junction.bst:gcc.bst

As you can see, the cache keys are different.
I can't figure out what difference between my laptop and the CI could cause this though. They have the same project.conf and buildstream.conf, the same version of bst and bst-external.
The only difference seems to be the arch of the host, but I don't think that should be considered when computing the cache key.

BuildStream version affected: bc5a40e3dccbe1cb8066171fc2c0a4a5abeaa31e to 1.2.0 at least (I haven't tried with master)

[Gitlab user @jjardon]

In GitLab by [Gitlab user @valentindavid] on Jul 31, 2018, 16:27
It seems SandboxConfig uses os.uname to add host architecture to the cache keys.

In GitLab by [Gitlab user @bochecha] on Sep 13, 2018, 09:23

It seems SandboxConfig uses os.uname to add host architecture to the cache keys.

I tried hardcoding that to 'aarch64', but bst still didn't hit the cache.

In GitLab by [Gitlab user @tristanvb] on Sep 13, 2018, 14:12
[Gitlab user @bochecha] it is indeed confusing to me that hard coding that did not produce the same key.
Do you think this might be a symptom of #641 ? I.e. max-jobs variable being used directly in build instructions somewhere ?
That said, the comment in the referred code of SandboxConfig is pretty telling. BuildStream does not support cross building, at least not yet.
It is intended that to support that, we need:

SandboxConfig to ask for the correct machine architecture
BuildStream to find out if there is a sandbox implementation available at runtime for the given architecture; and to either use that implementation or bail out with an error.

In GitLab by [Gitlab user @bochecha] on Sep 14, 2018, 07:22
changed the description

In GitLab by [Gitlab user @bochecha] on Sep 28, 2018, 09:42
I tried again with Buildstream 1.2.2.

Do you think this might be a symptom of #641 ? I.e. max-jobs variable being used directly in build instructions somewhere ?

That could have been the case in the past, but we don't use max-jobs any more thanks to a contribution by [Gitlab user @tacgomes] (https://gitlab.com/freedesktop-sdk/freedesktop-sdk/merge_requests/513)
The latest FDO Sdk 18.08 uses max-jobs in exactly one place:
environment-nocache:
  - MAXJOBS

environment:
  MAXJOBS: "%{max-jobs}"

It shouldn't cause any problem, since it's an environment-nocache variable?
In any case, the native Aarch64 build in the CI had the following key for all.bst:

647a23a5c192fa48ab910a0939c61ccbde89ebebcdb8c43cd1e13bd0dd897988

On my x86_64 machine, if I run bst -o bootstrap_build_arch aarch64 -o target_arch aarch64 show all.bst I get:

edbfd7783c2af03e97f7bc4b8b58bcedad8a63079846c5f2c8f7f38f2ed87ffb

If I modify buildstream/sandbox/_config.py, hardcoding 'aarch64' as the arch, I get:

bb9fb09fa0ad9fb51fd2bc7b5a03bec8cf7d45e6c666ade872b13b90e2ebe45c

So hardcoding the arch in that file does have an impact on the cache key.
But there must be something else that also does, somewhere?

In GitLab by [Gitlab user @abderrahimk] on Oct 13, 2018, 20:18
[Gitlab user @bochecha]: This is weird. Hardcoding works for me. (using freedesktop-sdk/freedesktop-sdk[Gitlab user @e6939dcbaa31058434ca5b025db23733d11b05af], I get 364ee5a998a83e70d988d32e2dbd3220366214ae332e9c37ea55890cd93f0c40, which is the same as the CI). Are you sure you are modifying the bst-1.2 branch?
Another case I'd like to use this for is setting a 32-bit personality (using setarch) to build i686 on x86_64 without the buildscripts thinking they are cross compiling.
I did some tests, and here is what I get (on my x86_64). I ran uname -m with different architectures for the uname binary:

if the binary is x86_64 or x86, I get x86_64;
if the binary is x86_64 or x86, and I'm setting a 32-bit personality (using linux32), I get i686;
if the binary is aarch64 (running through qemu-user), I get aarch64 (regardless of whether I run under linux32).

So the thing is we cannot know for sure what running uname -m in the sandbox will give before we actually run it.

It is intended that to support that, we need:

SandboxConfig to ask for the correct machine architecture
BuildStream to find out if there is a sandbox implementation available at runtime for the given architecture; and to either use that implementation or bail out with an error.

So IIUC, we need an option to pass the sandbox architecture to buildstream, and this will be used instead of os.uname() (in both the cache key, and as a default for options of type arch).
Then, we need to check whether we can run the uname binary in the sandbox, and whether it returns the correct value.
And to support my additional requirement, we also need to call bwrap while setting a personality using setarch.
Does this sound good?

In GitLab by [Gitlab user @abderrahimk] on Nov 4, 2018, 09:08
Today, I found this nifty tool https://github.com/kilobyte/arch-test. It can tell whether your machine can run binaries of the given arch (or list all the arches your machine can run binaries of).
The problem is that it gives architectures as Debian names them (amd64 instead of x86_64 for example), but I think this shouldn't be a problem.

In GitLab by [Gitlab user @juergbi] on Nov 4, 2018, 09:35

So IIUC, we need an option to pass the sandbox architecture to buildstream, and this will be used instead of os.uname() (in both the cache key, and as a default for options of type arch).

Yes, see https://mail.gnome.org/archives/buildstream-list/2018-November/msg00003.html for my proposal how to configure this.

And to support my additional requirement, we also need to call bwrap while setting a personality using setarch.

Agreed, we should set the personality as well.

In GitLab by [Gitlab user @valentindavid] on Nov 22, 2018, 11:36
marked this issue as related to #775

In GitLab by [Gitlab user @abderrahimk] on May 16, 2020, 19:13
mentioned in merge request !1934

In GitLab by [Gitlab user @tristanvb] on Nov 6, 2020, 08:35
I think that we can close this now that we have Platform.canonicalize_arch() used to contribute to cache keys.
Is this still an issue in master ?

In GitLab by [Gitlab user @juergbi] on Nov 6, 2020, 08:41
This was fixed in !969.


the "order" layer-creation parameter is not parsed as expected

🐛 Bug
napari.view_image does not parse the order parameter how I expecte.  order=(2,1,0) will return a viewer where viewer.dims.order is (0,2,1).
Using an array of shape (100,100,10) and order=(2,1,0), I get the YZ plane:

Setting the order with viewer.dims.order = (2,1,0) works:

To Reproduce
Steps to reproduce the behavior:
import napari
import numpy as np

data = np.full(shape=(100,100,10), dtype=np.uint8, fill_value=255)
viewer = napari.view_image(data, order=(2,1,0))
print(viewer.dims.order)       # output (0, 2, 1) ???
#viewer.dims.order = (2,1,0)   # Uncomment to correctly draw the 100x100 layer
napari.run()

Expected behavior

Passing in order as a layer-creation parameters should behave identically to setting the dims.order property.
Environment

Please copy and paste the information at napari info option in help menubar here:

napari: 0.4.11
Platform: macOS-11.5.2-x86_64-i386-64bit
System: MacOS 11.5.2
Python: 3.9.6 (default, Jun 29 2021, 05:25:02) [Clang 12.0.5 (clang-1<IP_ADDRESS>)]
Qt: 5.15.2
PyQt5: 5.15.4
NumPy: 1.21.0
SciPy: 1.7.0
Dask: 2021.06.2
VisPy: 0.8.1
OpenGL:

GL version: 2.1 ATI-4.6.20
MAX_TEXTURE_SIZE: 16384

Screens:

screen 1: resolution 1792x1120, scale 2.0

Plugins:

console: 0.0.4

scikit-image: 0.4.11

svg: 0.1.5

Any other relevant information:

Additional context

I've looked a bit into this and it's happening because the default ndim for the Viewer on creation is 2, and this block of code rearranges the passed order of (2, 1, 0) to just be (1, 0) - which is still technically correct.
But then when we add a new layer, the dims.ndim gets updated to 3 and we rebuild the order incorrectly, because line 115 here assumes that the first dimensions will be first in the order.
I'm not really sure what we should do here - maybe if we get passed an order on __init__ we should set ndim to that? Or at the very least we should warn the user when we reduce order? @sofroniewn you are probably more familiar with this

eek! good detective work. ok i'm not sure what we should do. maybe once we have better axes labels figured out we can improve this. @andy-sweet might have ideas too

Thanks @DragaDoncila for finding the cause!
I would have a slight preference for having ndim updated rather than a warning. napari.view_image(...) is touted in many examples as the method to view data, and I doubt I'm the only one who is trying to load data with a warped dimension order.  (Why did I not transpose the array? Well, the actual data I was passing in was not a true numpy array...)
In any case, I definitely agree with the extremely low severity label. 😄

Happy to give some thoughts, but I don't think there going to help fix these in the short term!
I've made a little progress on axes labels (#906) and ran into a similar (though not as confusing) issue/cause, as axis_labels are also trimmed/padded to have a length that corresponds to the default value of ndim=2. So you get behavior like the following.
viewer_model = ViewerModel(axis_labels=('channel', 'time', 'z', 'y', 'x'))
viewer_model.dims.axis_labels
# ('y', 'x')
viewer_model.dims.ndim = 3
viewer_model.dims.axis_labels
# ('0', 'y', 'x')

As #906 is about giving each layer its own axis_labels, I am trying to understand if ViewerModel needs to have its own axis_labels because the set of axis labels can be derived from the union of all the layers' labels. One way that ViewerModel.axis_labels could be useful is to specify an ordering over that set of labels, so maybe that's one way we could specify the desired view order. Something like the following.
viewer_model = ViewerModel()
viewer_model.add_image(..., axis_labels=('time', 'y', 'x'))
viewer_model.axis_labels
# ('time', 'y', 'x')
viewer_model.add_image(..., axis_labels=('x', 'y', 'z')
viewer_model.axis_labels
# ('z', 'time', 'y', 'x') - 'z' is new, so prepend it
viewer_model.axis_labels = ('time', 'z', 'y', 'x')
# user doesn't like that ordering so specifies a new one

Of course, this works nicely if axis labels were specified, but by default they probably won't be, so we should also accept something like viewer_model.axis_labels = (2, 3, 1, 0). If axis_labels are given that are not compatible with the current layers, then I think we should error.
In the long term, I'm thinking that axis_labels, order, and maybe other things are not parameters that you should pass to ViewerModel at construction time when there are no layers, data, or dimensions yet, but only once you have added some data.

In the long term, I'm thinking that axis_labels, order, and maybe other things are not parameters that you should pass to ViewerModel at construction time when there are no layers, data, or dimensions yet, but only once you have added some data.

Yes, agreed - maybe answer here is to not allow these to be passed to Viewer or a view_* method directly. Will take a bit of time to get there, but I like that direction.
Thoughts on all this @jni?

I agree with @andy-sweet that there are some attributes on dims that only make sense once you have added some data - order being one of them.

Yes, agreed - maybe answer here is to not allow these to be passed to Viewer or a view_* method directly. Will take a bit of time to get there, but I like that direction.

I kind of don't agree with this simply because in the particular case of the view_* methods, we are getting data, and so any parameters which may not make sense on Viewer creation would definitely make sense by the end of the function call because we've added a layer.
We could therefore (in the short term) move some of these Viewer arguments passed to view_* methods to be set on the viewer after we've added the layer  - assuming we have a concrete list of which arguments need this - maybe anything on dims?


ExpandDate + ExpandTime = ExpandDateTime

I love using the ExpandDate and ExpandTime methods to extract helpful columns quickly. It would be neat if there were an "all-in-one" method to do both date and time attributes.
import ibisml as ml
from ibis.interactive import *

wowah_data = ex.wowah_data_raw.fetch()

recipe = ml.Recipe(ml.ExpandDate("timestamp"), ml.ExpandTime("timestamp"))

recipe.fit(wowah_data).transform(wowah_data).table

That seems like a good plan to me!


[TASK] Compute numerical chain_id (u32) for cosmos chains

@shekohex @drewstone
I think we already had some discussion about this topic - how to convert string chain_id into numeric.
I remember @shekohex suggested a solution like:

Hash the chain_id of blockchain
eg: keccak256("juno-1")
Slice the last last 4 bytes & convert to u32 numeric value

@shekohex
can you please confirm the solution & document it in the current webb-proposal doc?


Provide admission server builds for CPU architectures other than amd64

What would you like to be added:
Admission server builds are currently only available for amd64 architecture.
We should provide builds for other architectures as well.
Why is this needed:
Our users could be running k8s clusters on other CPU architectures.

For some context, #621 was the first attempt to get an image built and published. We decided to take an incremental approach to building out our images. There is no blocker for building docker images for other CPU architectures.

/good-first-issue
Outstanding work here is to update our webhook admission build to some other architectures, linux/arm64 for certain, and to make sure we can add more if needed.

/assign

Quick question -

what all arch do we want to add?
All arch images need to use to repo that has arch information in its name, earlier we were using ${REGISTRY}/admission-server:${VERSION_TAG}. Now it should be ${REGISTRY}/admission-server-${ARCH}:${VERSION_TAG}. So do we need to create new repo or is it handled by infra team.

I think that amd64 and arm64 are probably okay for now, we can add more later if required.
I didn't know that was a requirement, maybe we can check with some other things like ingress-nginx as to how they handle this (I don't think that other Kubernetes projects have the arch in the repo name?)

We now have a multi-arch build with new arm64 support:
https://console.cloud.google.com/gcr/images/k8s-staging-gateway-api/global/admission-server@sha256:5dd8c42658daeb0a26ba0e4ac5e33c3bba2f1f4cd0f41685f837dd4a3684d2f0/details?tab=pull


Raise minimum FFmpeg version to 3.2

See discussion in #386.
We will do this for the next major release.

I've dropped the old ffmpegs from Travis in #386

Aaaand they are merged.


Touch of Corruption not applying cruelty

Describe the bug
They fail the save and gain no status effect from any cruelty
To Reproduce
Steps to reproduce the behavior:

Attack with cruelty
Check to see if they failed the save
Check status of enemy
See that there is no status effect applied

Expected behavior
Feared, sickened or other status effect applied

As an example I tried using sicken on this and it doesn't have immunity to sicken. Ive tried all my cruelties out including fear which nothing should be immune to.

Example of me trying it with fear in another playthrough without the conquerer archtype.

Not even working on a basic cultist

The silly thing is that I have fixed this before, I don't know what changed and won't be able to get time for a look for a day or two.
Any chance it worked in the past and broke suddenly, or has it never worked in your game?

No I tried it again a few more times today and tried turning off other mods with no luck. Thanks for looking into it. Will just put these play throughs on hold till you get a chance to find the issue. Love your mod btw and am excited to play with it more.

Fixed! One of the last updates broke every single one, so I won't be able to update until I have more the the 20 or so min today to sort them all out.
Thanks for your testing it was a big help, if there is anything you want from the tabletop that I might be able to add let me know.
I'll leave this ticket open until the update is out.

Happy to help! Content mods like yours make this game way more replayable and its always a pleasure to see new ones coming up.
" if there is anything you want from the tabletop that I might be able to add let me know."
I know it would be a big ask, but I've been desperate for some psychic class/ archtypes from pnp. I loved playing it with "Call of the Wild" in kingmaker, and was sad to see no one adopt the class/ psychic archtypes yet for Wotr. Other than that more gish archtypes are always fun like the Blade Adept Archanist :)

Updates out, one day I might be brave enough to try the psychic stuff but not yet.
I believe another modder is doing the blade adept archetype so I'll hold off on that for now.
Once again, thanks for the help!


az eventgrid event-subscription create command error

Related command
az eventgrid system-topic event-subscription create
az eventgrid event-subscription create
Extension name (the extension in question)
az eventgrid
Description of issue (in as much detail as possible)
In Azure Portal, we can create an event subscription for system topic with enabling dead-lettering as shown below: (doc reference)

However, az eventgrid system-topic event-subscription create doesn't have parameters deadletter-identity and deadletter-identity-endpoint in CLI and running results in unrecognized arguments: --deadletter-identity-endpoint /subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Storage/storageAccounts/xxx/blobServices/default/containers/egcontainer --deadletter-identity systemassigned error.
So followed az eventgrid topic event-subscription create doc and ran the following command and it also resulted in error.
Command:
az eventgrid event-subscription create --name xxx --source-resource-id "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.EventGrid/systemTopics/xxx" --endpoint-type storagequeue --endpoint "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Storage/storageAccounts/xxx/queueservices/default/queues/xxx" --deadletter-identity-endpoint "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Storage/storageAccounts/xxx/blobServices/default/containers/egcontainer" --deadletter-identity systemassigned
Output:
Code: InvalidRequest
Message: This event subscription operation is not supported using this API call. Please use the proper API and try again.
Any plan for supporting deadletter-identity for system topic? These parameters are available in both ARM template as well as Rest API.

Thank you for opening this issue, we will look into it.

@mumurug-MSFT Thanks for the feedback! We will review and get back to you.


[BEAM-1925] Remove deprecated context param from DoFn

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @chamikaramj @robertwb PTAL

Travis seems happy for Python so should be safe to review

Changes Unknown when pulling 0e540a7fad7016ef7e57bd804e1dd6fcafe4a4f8 on sb2nov:BEAM-1925-remove-deprecated-context-param into ** on apache:master**.

Coverage decreased (-0.003%) to 69.681% when pulling ed12b00b5ff8414f9f966baccd2661be22bd693d on sb2nov:BEAM-1925-remove-deprecated-context-param into a8a04a2905e1ff639f403be7d9132acb77365e84 on apache:master.

LGTM (lemme know if I should merge)

Please merge
On Fri, Apr 28, 2017 at 12:49 PM Chamikara Jayalath <
<EMAIL_ADDRESS>wrote:

LGTM (lemme know if I should merge)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/2712#issuecomment-298090754, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABco0z_ETnSG8rKtPuIJvDodi_2TZE0Fks5r0kKygaJpZM4NJX9O
.

@robertwb PTAL

I'll wait for Robert's LGTM.

LGTM, are there any docs we should be updating?

I checked the beam-site and the only mentions of ContextParam seem to be inside the pydoc.

Sg. Merging.


Add project type tag to project cards

Also move tags to bottom of cards to improve visibility of the titles

Closed by #151


"No issues", but it's a lie!

"No issues", but it's a lie!
In a nutshell:
I'm using archlunux and fedora 25. Linter-gcc isn't working. It just shows me "No issues" all the time.
Before i had Ubuntu 16.04 and linter-gcc was working perfect.

Please do the following:

Enable debugging messages in the linter-gcc configuration page
Press Ctrl+Shift+I to enable Developer Mode, and click 'Console'
Perform a save on your C/C++ source file to trigger linter-gcc
Copy the linter-gcc command that appears, paste it into a terminal. Then copy the output and paste it here.

where is the c/c++ source file?  haha~


UI.renderPage not documented

It seems PDFJSAnnotate.render is where metadata is set, but PDFJSAnnotate.UI.renderPage is what the main example is using, which does not in turn use PDFJSAnnotate.render. UI.renderPage however is not documented.

UI.renderPage does indeed use PDFJSAnnotate.render https://github.com/mzabriskie/pdf-annotate.js/blob/master/src/UI/page.js#L70
Though you are correct that the docs are missing. Will update.

Docs are updated https://github.com/mzabriskie/pdf-annotate.js/blob/master/docs/api/UI.md#createpage


ASP.NET MVC with EF Core Tutorial: Implement inheritance

In the SchoolContext.cs file, it is necessary to remove the Student and Instructor entities from the OnModelCreating method; otherwise, an error is generated, "Invalid object name '[Entity Name]'. at Microsoft.Data.SqlClient.SqlCommand." when either the Students or Instructors navigation link is clicked.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 55345ff6-3c81-969e-3c92-29c24d78d2ca
Version Independent ID: aa826512-74a5-752a-0ad0-0f8133f6c39e
Content: Tutorial: Implement inheritance - ASP.NET MVC with EF Core
Content Source: aspnetcore/data/ef-mvc/inheritance.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

duplicate of #17465


PullToRefreshViewAndroid is deprecated

PullToRefreshViewAndroid is no longer necessary; it has been replaced by RefreshControl which works on both platforms.
As of 0.22 (in RC now), PullToRefreshViewAndroid will be deprecated.

@FaridSafi, I'm planning to make a PR for this. Would you review it? A small downside of RefreshControl is that you lose some customizing options, but I think it's worth it.

thank you, new version has been published


1.44 TFT LCD ST7735S is not working as expected

Hi! I have this 1.44" SPI 128x128  TFT LCD Display (RED PCB V1.1) with following specs:
conf/st7735.conf:
--display=st7735
--interface=spi
--spi-bus-speed=16000000
--gpio-reset=24
--gpio-data-command=23
--gpio-backlight=18
--width=128
--height=128

When running an example  sudo python examples/3d_box.py -f conf/st7735.conf , there is nothing on the screen! Even the backlight is not enabled.

After I connect gpio-backlight pin directly to 3.3V source - it works fine

Notes:

It seems that code can't reach/control some of the required GPIO or numbering is wrong. (I've used BCM numbering)

When I follow 'Python usage' instructions nothing happens when I try 'draw.rectangle' or 'draw.text'

When I try to get the 'device' object details in runtime (run with -f conf/st7735.conf), I see the unexpected arguments for screen size (my LCD has width=128, height=128):

luma.lcd.device.st7735 at 0x765da3b0 (
  bounding_box = (0, 0, 159, 127),
  framebuffer = luma.core.framebuffer.diff_to_previous at 0x765da7f0 (
      bounding_box = (0, 0, 160, 128),
      image = PIL.Image.Image at 0x76359b90 (
          category = 0,
          format = None,
          format_description = None,
          height = 128,
          im = ImagingCore at 0x76d6afa0 (
              bands = 3,
              id = 22136808,
              mode = 'RGB',
              ptr = PyCapsule at 0x7655f020 (),
              size = (160, 128),
              unsafe_ptrs = ((...), (...), (...), (...), (...), ..., (...), (...), (...), (...), (...))
            ),
          ...,
          palette = None,
          pyaccess = None,
          readonly = 0,
          size = (160, 128),
          width = 160
        )
    ),
  height = 128,
  mode = 'RGB',
  rotate = 0,
  size = (160, 128),
  width = 160
)

Do you have any suggestions?

See the comment in #39 - i took a decision (wrongly) to hardcode the resolution to 160x128 thinking that was the only resolution. I will push a change so it supports 128x128 but it won't be for another few days.
As to the issue with the backlight, i don't know. I have bought one of these devices but haven't had the chance to plug it in. I will update the ticket with my findings

@mnrozhkov there is a new version that supports 128x128 variants of this device. If you upgrade with:
$ sudo -H pip install --upgrade luma.core luma.lcd

The backlight was originally written for the PCD8544 which is enabled as active-low, whereas the ST7735 enables the backlight as active-high. It is now configurable, so supports either setting.
Lastly, if you then pull from master for the luma.examples repo, and you should find a conf/st7735_128x128.conf file with the correct settings (well, this works on the 128x128 device I have).
Let me know how you get on,

@mnrozhkov any news?

@rm-hull thanks a lot! It works well. I've updated library and changed conf.
I use LCD to build eyes for my robot. I'll share with you how it will word as soon as I finish this. Your library is priceless help for our project!
Best wishes!


mount/shallow: restructure what a wrapper is, and add .root()

Per discussion:
In enzyme v2, a shallow wrapper is what a component renders, but a mount and a render wrapper need .children().eq(0) to get to "what the component renders". (this impacts what .props(), .instance(), etc point to; which is inconsistent across the APIs). Mount in particular is this way because you need a wrapper around the component itself to be able to set props and state and whatnot, however, the more common case is writing assertions about what a component renders.
Thus, in v3, all three of shallow, mount, and render will be wrappers around what a component renders, and both shallow and mount will have a .root() method that will return a wrapper around the component itself. Wrapper methods will continue to throw when it makes sense (render won't have .root(), for example), but now all wrapper methods will point to "the thing(s) that are being wrapped", and the initial output of all three APIs will now conceptually be the same thing.

As discussed, we are no longer doing this for v3.


Translation Missing for Edit Message buttons

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to Sidebar click on Preference
Change the language to Spanish
Come back to Chat Messages and right-click to Edit Comment
You'll see the button texts still shows "Cancel" and "Save Changes

Expected Result:
Should show the button texts in Spanish
Actual Result:
Shows the button texts in English only

Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @lschurr (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Proposal
Basically the button texts are hardcoded with english content. Need to pick these from translations.
In ReportActionItemMessageEdit.js,
// Change button texts with translate function

<Button
    style={[styles.mr2]}
    onPress={this.deleteDraft}
    text={this.props.translate('common.cancel')}
/>
<Button
    success
    style={[styles.mr2]}
    onPress={this.publishDraft}
    text={this.props.translate('common.saveChanges')}
/>

// Add references for localization

export default compose(
	withLocalize, withWindowDimensions
)(ReportActionItemMessageEdit);

and add translation for saveChanges key in en.js and es.js
en:
	common: {
		saveChanges: 'Save Changes'
	}

es:
	common: {
		saveChanges: 'Guardar cambios'
	}

Is this different from this one? https://github.com/Expensify/App/issues/4218

@lschurr Yes. This one handles the button texts - Edit and Cancel (highlighted in red rectangle in the screenshot above).
#4218 deals with the timestamp itself not showing up when language is switched. (Line below the red rectangle).

Triggered auto assignment to @Jag96 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Proposed Solution:
Add missing keywords in languae file and update the English text with function.
For language files add keys,
en -> common -> saveChange: 'Save changes'
es -> common -> saveChange: 'Guardar cambios'
Key for cancel already available in language file.
Update the below block in ReportActionItemMessageEdit

with
text="Cancel"  -> text={this.props.translate('common.cancel')}
text="Save Changes" -> text={this.props.translate('common.saveChanges')}
Currently, the class doesn't have localize hoc. Need to add that too.
export default compose(
	withLocalize,
	withWindowDimensions
)(ReportActionItemMessageEdit);

and props ...withLocalizePropTypes

@mananjadhav your proposal looks good and straightforward, I've invited you to the job, feel free to start on the PR once accepted.

PR raised.

Paid + bonus!


Add Service Provider Factory support

Copied implementation from aspnet/Hosting. Allows overriding of service provider implementation.

Thanks for the contribution! Could you please also add some tests to verify the feature works? Also, out of curiosity, have you been successful using any 3rd party IoC containers using this change?

@danroth27 I've added a unit test using autofac, I know its not ideal to add that as a test dependency, however its hard to properly test it without using a different DI framework.  I can however revert this change, and implement a unit test that instead just checks whether the GetService method is called on a custom service provider (wraps the default ServiceProvider) atleast once, however due to how ServiceProvider is implemented, it means the host.Services type won't be the custom provider type, and the only easy way to achieve this was through using autofac.
I sadly haven't had any luck with Autofac, there seems to be some mono issue blocking by the looks.
System.NullReferenceException: Object reference not set to an instance of an object.
  at System.Linq.Expressions.Interpreter.LightLambda.MakeRunDelegateCtor (System.Type delegateType) <0x1c452e0 + 0x00294> in <6276484b63204edaa16fa5b759c1a011>:0

https://pastebin.com/B1Y6ex1g
It seems to be a similar issue to #640

@danroth27 I assume the AppVeyor build failing can be safely ignored, seeing as travis and blazor-ci both build and the tests pass?

@csnewman Nice job on this PR, very surgical and neat. Are you able to use the Startup.ConfigureContainer(ContainerBuilder builder) method after your changes? Or must you pass a configureAction lamda to the AddAutofac() method to be able to do Autofac-specific container configuration?
I wonder if it wouldn't be a good idea to also add support for the Startup.ConfigureServices(IServiceCollection services) returning an IServiceProvider directly, to maintain similarity with aspnet/Hosting? I'm guessing many devs coming from ASP.NET Core would expect that to work.

Currently neither are supported. This is the simpler IServiceProviderFactory<IServiceCollection> support as seen in aspnet/Hosting. To use autofac, you have to create a custom IServiceProviderFactory<IServiceCollection>, in which you can do any customisations you wish, including creating the autofac service provider.
I have a planned followup PR that will implement the ConfigureContainer and ConfigureServices support, however those will be much larger pull requests. I think its worth getting this more manual IServiceProviderFactory<IServiceCollection> added first so people can atleast start to use custom DI frameworks and then adding the shortcut methods later in the followup PR. Especially seeing as any code implemented with IServiceProviderFactory<IServiceCollection>, will still work post my followup PR.

@csnewman Makes sense to separate these two things, indeed.
Just to clarify though - you say we need to create a custom IServiceProviderFactory<IServiceCollection>, but already with this current PR, isn't it as simple as this?
var host = BlazorWebAssemblyHost
    .CreateDefaultBuilder()
    .ConfigureServices(services =>
    {
        services.AddAutofac(builder =>
        {
            // Interact directly with ContainerBuilder here...
            builder.RegisterModule<CoreModule>();
        });
    })
    .UseBlazorStartup<Startup>()
    .Build();

host.Run();

As far as I understand, the AddAutofac() extension method adds the service provider factory.

@DaRosenberg Unfortunately not. As I am following the same principals as aspnet/Hosting, it doesn't work this way.
AddAutofac registers a IServiceProviderFactory<ContainerBuilder>, with ContainerBuilder being a custom Autofac type. In aspnet/Hosting, they check the parameter type of ConfigureContainer(T sometype), and use this to instantiate the correct IServiceProviderFactory<T>. If there is no ConfigureContainer method, they then fallback to IServiceProviderFactory<IServiceCollection>. For AddAutofac to work, it will need the ConfigureContainer method support, to allow us to instantiate the correct type using DI.
Example ServiceProviderFactory for Autofac
private class MyServiceProviderFactory : IServiceProviderFactory<IServiceCollection>
{
    public IServiceCollection CreateBuilder(IServiceCollection services)
    {
        return services;
    }

    public IServiceProvider CreateServiceProvider(IServiceCollection serviceCollection)
    {
        var containerBuilder = new ContainerBuilder();
        containerBuilder.Populate(serviceCollection);
        // TODO: Call your configure action here
        var container = containerBuilder.Build();
        return new AutofacServiceProvider(container);
    }
}

@csnewman Ah, got it. Thanks for explaining that. @danroth27 this PR would unblock our development too, so having it merged would be super helpful.

@danroth27 I have fixed the merge conflicts, if you wish to review the commit.

Thanks very much for this, @csnewman! It's now moved to https://github.com/aspnet/AspNetCore/pull/4785/files, where it will be merged as soon as it passes automated verification.
I did tweak the test to eliminate the Autofac dependency (though I do understand why it was a bit awkward and why you chose to use Autofac in the test originally). Mainly I wanted to avoid the external dependency so as to make the test even more explicit about what interactions we're having with other units of code. Hope that's OK!


ImproveJob internal interface provides immediate scheduling

Delete meaningless job status
System scheduling is executed in the time wheel
Optimize window calculation code
Proposed changes
Issue Number: close #xxx

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

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.44 seconds
stream load tsv:          536 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>8 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.5 seconds
stream load tsv:          535 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.2 seconds inserted 10000000 Rows, about 342K ops/s
storage size:<PHONE_NUMBER>4 Bytes


TextField with value via parents props latency between typings

When I use TextField with value from parents props there is a latency between typing a letter and displaying it on the screen. I see this issue on my project and at official page for TextField component (I talk about the third TextField from the first colunm).

I've seen this too.  I think because the state object on the parent component is changing, this is causing every field to re-render.
In house, we're wrapping field, and have a pure-render-mixin applied - however we had to suppose valueLink, and some deepness in models - which the React.addons.PureRenderMixin doesn't.
@hai-cea - is something like this too opinionated to include in the core?
(Excuse the coffeescript)
_ = require 'lodash'

module.exports =
  shouldComponentUpdate: (nextProps, nextState) ->
    for key of nextProps
      if key is 'onChange'
        continue
      if key is 'valueLink'
        return true if nextProps.valueLink.value isnt @props.valueLink.value
      else if not _.eq(nextProps?[key], this.props?[key])
        return true
      else if not _.eq(nextState?[key], this.state?[key])
        return true
    return false

@chrismcv  I think we need to address shouldComponentUpdate for most if not all of our components. What about children though?

_.eq is deep - which should deal with children - i'm sure it's not mega efficient though... but at the same time has yielded a performance improvement for me

Is there any way to prevent the others component to re-render when I typing into TextField? One more question this will be fixed in a new material-ui version?

Typing in textfields is indeed extremely slow, Firefox and Chrome. If you type really fast, it may even lock up for 2-3 seconds.

Encountered the same problem. I'm currently using onBlur, getValue and setValue as a workaround and handle the updates only after input lost the focus.

@hai-cea the pull request I just opened should fix the textfield lag in the demo page significantly. Belle had a similar issue in their docs

@mstamos @CoDEmanX @Kagami can you test again? I don't see this issue today on the docs website.. We've since implemented the PureRenderMixin so I think it has taken care of the issue

Seems to work very smooth now, only tested chrome but I think that fixed it. Great job!

Still getting this issue, I'll post a new issue with flamegraphs.

Encountered the same problem. I'm currently using onBlur, getValue and setValue as a workaround and handle the updates only after input lost the focus.

how can i do this, im experiencing the same issue?


Support validation of patternProperties

A schema like the following seems to just create an additionalProperties<String, Object> map with none of the validation in the schema.
{
    "type": "object",
    "patternProperties": {
        "^[a-zA-Z0-9-_.]+$": {
            "type": "array",
            "items": {
                "type": "object",
                "properties": {
                    "version": {"type": "integer"},
                    "addr": {
                        "type": "string",
                        "anyOf": [
                            {"format": "ipv4"},
                            {"format": "ipv6"}
                        ]
                    }
                },
                "required": ["version", "addr"]
            }
        }
    }
}

The regex should be used to validate the names of the keys in the object, and the values have a schema which needs validating too

I think this is a place where generating JsonDeserializers for Jackson and TypeAdapters for GSON would allow the project to support these JSON Schema constructs, along with some method signatures like:
public T getAdditionalProperty( String name, Class<T> type );
public T get( String name, Class<T> type );

that throw ClassCastException when the wrong type is used.

the class above is generated and name is known, so we can just use it in the code.

Would it be an option to skip the validation of the key itself and just generate Map<String, Type>?

I'd love to see patternProperties to be supported with the two constraints proposed by @joelittlejohn

patternProperties is only supported when "additionalProperties" : false is present

and

only one patternProperties pattern is supported, all others are ignored

It may even only be available through explicit configuration. patternProperties helps defining rules of Map-keys, which we would like to restrict.

Is this supported by any chance?

Likewise- not looking for validation, just looking for a generic Map to hold the values that come in. In this case, I'm using a JSON schema inside a YAML editor in VS Code, so the editor is enforcing the constraint/regex well enough with a red squiggle (or my application code can). But now that I turned off additionalProperties, I have nowhere to track those additional fields that are coming in.


Ignoring "Command Prompt" in terminal.integrated.defaultProfile.windows during build.

Issue Type: Bug
VS Code is no longer recognizing "terminal.integrated.defaultProfile.windows": "Command Prompt" when I run a build while building an Azure Function dotnet core 3.1 project.
As a workaround I'm setting "terminal.integrated.automationShell.windows": "C:\Windows\System32\cmd.exe". This started happening this week after an update.
"Command Prompt" = "C:\Windows\System32\cmd.exe" in profiles.
VS Code version: Code 1.60.0 (e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff, 2021-09-01T10:41:52.311Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No
Extensions (27)

Extension
Author (truncated)
Version

vscode-sql-formatter
adp
1.4.4

azurite
Azu
3.14.2

vscode-markdownlint
Dav
0.43.2

vscode-diff
fab
1.4.2

code-runner
for
0.11.5

csharpextensions
jch
1.3.1

vscode-nuget-package-manager
jmr
1.1.6

minifyall
jos
2.9.2

json2csv
kha
0.0.1

l13-diff
L13
1.0.3

vscode-azurefunctions
ms-
1.5.0

vscode-azureresourcegroups
ms-
0.4.0

vscode-azurestorage
ms-
0.12.1

vscode-asa
ms-
1.0.1

csharp
ms-
1.23.15

vscode-dotnet-runtime
ms-
1.2.0

mssql
ms-
1.10.1

python
ms-
2021.9.1191016588

vscode-pylance
ms-
2021.9.1

jupyter
ms-
2021.8.2041215044

jupyter-keymap
ms-
1.0.0

azure-account
ms-
0.9.8

notepadplusplus-keybindings
ms-
1.0.7

azurerm-vscode-tools
msa
0.15.2

elastic
ria
0.13.3

azure-event-hub-explorer
Sum
0.1.1

change-case
wma
1.0.0

Please try latest Insiders build and let devs know if it's still broken.
https://code.visualstudio.com/insiders/

The insiders build worked! Thank you.

/duplicate of #132150


feat(system_launch): add group tag around include

Signed-off-by: h-ohta<EMAIL_ADDRESS>PR Type

Bug Fix

Related Links
https://github.com/ros2/launch/issues/620
https://tier4.atlassian.net/browse/T4PB-18074
Description

The <include> tag does not scope, so same <arg> tags are overwritten when including multiple launch files.
To avoid this, I set a scope limitation to add <group> tag around <include> tag.

Review Procedure

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Code follows coding guidelines
[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code follows coding guidelines
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets
[ ] Write release notes

CI Checks

Build and test for PR: Required to pass before the merge.
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

I'll merge.


Can we replace deprecated dex helm chart with the official one

What should be investigated.
Can we replace stable/dex be replaces with dex/dex?
What artifacts should this produce.

Either a new issue replacing the helm chart
or a document with fixes that are needed so that @cristiklein can decide wheather or not we should contribute them or just wait for 'em to be fixed.

FYI we've created a new (official) chart that uses an untagged version of Dex: https://github.com/dexidp/helm-charts/tree/master/charts/dex
We should be able to reach 0.1.0 in the coming weeks which is going to be considered the first stable(ish) version. Until then, feedback is welcome.


training only support batch=1?

I read through the code. It seems that training only support batch = 1 images? In caffe we can accumulate the gradient for several images (e.g. batch=2) before we update the parameters. It is supported in tensorflow? Thanks.

I also opened an issue here at https://github.com/smallcorgi/Faster-RCNN_TF/issues/54, but not much progress

Hi @hengck23 , interested in collaborating on implementing this for kaggle dsb17 lung data?


Unable to style DataLabels

Setting
Series.FontSize = 20;

Should change the font style of the data labels in the series, this feature is not working correctly in the latest version.

@RambaaZambaa Thanks for the report, this is already fixed and will be included in next release

I think the problem still persists, I use the latest version from nuget 0.7.10
The FontSize property is also ignored on AxisX and AxisY.
I use for example the following code:
CC_Chart1.AxisX.Add(new Axis
            {
                LabelFormatter = val => TimeSpan.FromSeconds(val).ToString(@"mm\:ss"),
                //FontStyle = System.Windows.FontStyles.Italic,
                //FontWeight = System.Windows.FontWeights.ExtraBold,
                FontSize = 30,
                Separator = new Separator
                {
                    Step = 5
                }
FontStyle, FontWeight, FontFamily are set as expected.

@Showmaster3000 you are right, it was being ignored for axes, last commit fixes it, it will be included in next release 0.7.11


Drop libvirt-daemon-driver-storage for libvirt v6.9.0+

Is this a BUG REPORT or FEATURE REQUEST?:
/kind enhancement
What happened:
As of libvirt 6.9.0, the QEMU driver no longer unnecessarily depends on the storage driver, and so it should be possible to stop including the libvirt-daemon-driver-storage-core package in containers once a newer libvirt version is imported.
Anything else we need to know?:
Original issue: https://github.com/kubevirt/libvirt/issues/61
libvirt fix: https://gitlab.com/libvirt/libvirt/-/commit/accdc0e7730739f398e392c23bc8380d3574a878

AFAIK we are currently using libvirt 6.6. Keeping this issue as a tracker.

AFAIK we are currently using libvirt 6.6. Keeping this issue as a tracker.

AFAIK we are currently using libvirt 6.6. Keeping this issue as a tracker.

Yup, that was the intention :)

AFAIK we are currently using libvirt 6.6. Keeping this issue as a tracker.

Yup, that was the intention :)

/remove-lifecycle stale


how to  modify the code and make sure it can train on cpu

RuntimeError: CUDA out of memory
Environment:

OS: [e.g. Ubuntu 18.04]
GPU: [e.g. GTX 960M]
CUDA Version [e.g. 10.2]

You can try removing all .cuda() and change all torch.cuda to torch.  However, you might expect the training on the CPU quite  slow.  Thank you!

Hi xI-820815641,
Did you finally achieve training on CPU ?
I am also trying to do so, because my GPU is AMD Ryzen which is not CUDA compatible.
I'd be happy if you could share some tips or steps to folllow.
Best,
Anthony


Typo in typescript remark

This line in the typescript code is wrong:
// Render the cell value in green when the value is blue than $100,000 and red otherwise
The correct text should be
// Render the cell value in blue when the value is more than $100,000 and red otherwise

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

ID: 225f1305-7aea-3fdd-9a46-e0f258c670ff
Version Independent ID: 225f1305-7aea-3fdd-9a46-e0f258c670ff
Content: Customized editable grid - Power Apps
Content Source: powerapps-docs/developer/component-framework/sample-controls/customized-editable-grid-control.md
Service: powerapps
Sub-service: pcf
GitHub Login: @jasongre
Microsoft Alias: jasongre

@Drout
Thanks for your feedback.
I've fixed this.


Generated types and functions lack documentation

Luau LSP's doc comments do not work in definition files. Instead, you are asked to create documentationFiles for your definitionFiles' types, however the only piece of written information I could find on how to write one is from an old release changelog. Most of the syntax is clear but the changelog doesn't clarify what scope types should be put under. (@roblox/global/whatever)
This could probably be solved with an issue in JohnnyMorganz' repository but I'm putting it off until later.

Fortunately, somewhere in time I abandoned this project and now, I did find out how documentationFiles worked. Unfortunately, I don't remember. Still open.

After some more searching I found it, and I also found with it this link to the scheme used by luau-lsp for parsing documentation. The only step now is to write/generate the documentation itself.


Fix #11573 PageChoosers locale filter

Fixes #11573
Now the page choosers locale filter is working correctly, but still I am not sure if this is the correct the approach. I bind the widget and the form so the widget can access the instance or parent id. When I pass the instance id (during EditView), it behaves as the parents id and the chooser can only choose child pages. Should I only pass the parent id and never pass the instances id. Any feedback is appreciated for the way to approach this.
Tests are not added yet.
Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

Need feedback to know how to continue with this

I'll investigate this fully when I get the chance, but patching AdminPageChooser to dig into the values of other fields in the form is definitely not an approach we want to take. Form widgets should always be able to work independently without any knowledge of the form they're contained in. If there's some extra information that they need to do their job, that information should be explicitly passed to it when setting up the form.

I'll draft this PR and try get a better solution for this. It is a bit tricky passing the Parent Page to the widget, but I will figure a better way. Thank you for your feedback.


Support order overrides via frontmatter for pages

Currently you are only able to specify order for patterns. It would be nice for things like indexes to also have more options than alphabetical order.

Closed via #88
We may revisit this again if we decide that using that explicit ordering via {{pages orderby=""}} is insufficient.


Feature/wgu pdvd

Add gen and g4 stages of fhicls for ProtoDUNE-VD

These three PRs are made at the same time:
https://github.com/DUNE/dunecore/pull/24
https://github.com/DUNE/dunesim/pull/19
https://github.com/DUNE/dunesw/pull/32  (this PR)


Improve Documentation

IMO #648 improves on documentation rigor significantly, but there are still probably weak points like:

[ ] It would be nice to have a static site #59
[ ] We still fall back on saying "it's like apollo" sometimes #544
[ ] The readmes are kind of weak as "getting started" guides tbh
[ ] Embedded runnable examples
[ ] Update policies section
[ ] Add explanation of reasoning behind immutability #633
[ ] Add more info on context + example of how to set headers in context per-request #633
[ ] Add docs on rebroadcast behavior and why subscription rebroadcasting doesn't make sense

Examples:

[ ] Add android INTERNET permissions to examples so they work in release mode

I want to consolidate these other issues here to minimize broad/ill defined "better docs plz" issues

#608 has some docs discussion / links as well

consolidating #657

Embedded runnable examples would be dope. We'll need [embedded api docs with library support]
(https://github.com/rrousselGit/flutter_hooks/issues/118) though, which I'm willing to wait for official support for

consolidating #748 as it is smol

The problem with rebroadcasting subscription results when the cache changes is that subscriptions are already a stream, so semantically each should be treated as a separate event. A rebroadcast would look like a new event, and thus break the the contract.
Theoretically, if RebroadcastPolicy is made its own concept, then allowing the user to explicitly set subscriptions to rebroadcast would be a fine feature to implement. Probably will be some time before that is implemented though.
This problem is similar to the problem of #774 – mutations should not do a cache round-trip either by default.

The problem with rebroadcasting subscription results when the cache changes is that subscriptions are already a stream, so semantically each should be treated as a separate event. A rebroadcast would look like a new event, and thus break the the contract.
Theoretically, if RebroadcastPolicy is made its own concept, then allowing the user to explicitly set subscriptions to rebroadcast would be a fine feature to implement. Probably will be some time before that is implemented though.
This problem is similar to the problem of #774 – mutations should not do a cache round-trip either by default.


Adding AnyChart licence key

Hi,
Is there an example for best practice in adding the AnyChart licence key to an angular 4/5 project?

Yes, you can specify the license key in an Angular 4/5 app.
Basically, you can specify it in ngOnInit() function like this:
ngOnInit() {
anychart.licenseKey('YOUR-LICENSE-KEY');
//creating chart code
}

Thanks.
Do you need to do this in every component that uses AnyChart? Can you not specify it in AppModule or somewhere global?

Hmm. Let me think about it and get back to you. It makes sense to do it on another level, I'll ask developers to assist.

@ImranAhmed
You can specify your license key globally in the Angular application.
You should add anychart.licenseKey('YOUR-LICENSE-KEY'); in main.ts file after AnyChart js imports.
Like this:
//main.ts
import 'anychart/dist/js/anychart-base.min.js';
import 'anychart/dist/js/anychart-ui.min.js';
anychart.licenseKey('YOUR-LICENSE-KEY');


Add GPT Visual support in testing suite

Backend: https://github.com/coronasafe/ayushma/pull/372
Add GPT Visual support

@Ashesh3 can we have a small preview next to the attachment?

Member

I tried that, but couldn't fit it in the design, for now you can click the file to preview it. If you can think of a good design for the previews, do let me know!


[question] how to add defaultFormatter

in vscode I have this configuration and I want to pass everything to nvim but it's not working
// .vscode/settings.json

{
  "editor.formatOnSave": true,
  "editor.defaultFormatter": "esbenp.prettier-vscode",
  "[typescript]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "[yaml]": {
    "editor.defaultFormatter": "redhat.vscode-yaml"
  },
  "editor.codeActionsOnSave": {
    "source.fixAll": true,
    "source.organizeImports": true
  },
  "typescript.preferences.importModuleSpecifier": "non-relative",
  "js/ts.implicitProjectConfig.experimentalDecorators": true,
  "files.eol": "\n",
  "editor.tabSize": 2
}

on nvim i'm having this problem

but there shouldn't be a space between braces

my coc-settings.json file

any tips?
I thought it might be something in my prettier settings but it's working in vscode so it can't be
I even opened this problem,
https://github.com/prettier/eslint-plugin-prettier/issues/473
but if it solves here I'll close it and point to this one

Seems you need prettier to format your document, but it's not working, checkout output by :CocCommand workspace.showOutput Prettier


lsp: Enable formatting functionality

In https://github.com/StyraInc/regal/pull/641 support was added to handle textDocument/formatting requests from clients. However, the functionality was disabled for fear that users would see a confusing message:
https://github.com/StyraInc/regal/issues/638#issuecomment-2048459623
In https://github.com/open-policy-agent/vscode-opa/pull/148 the regal formatter will be used if the user has enabled regal and opa fmt will be used if not. This means that there is only ever one formatter available for rego and the user will see no message.

Fixes https://github.com/StyraInc/regal/issues/638


fitToCoordinates zoom level works incorrect

Hi.
I tried dynamically change my map padding by fitToCoordinates function and it works perfect.
But the problem come on when I want to change the zoom level of coordinate. Unfortunately, latitudeDelta and longitudeDelta they both aren't affect on zoom level.
_fitToCoordinates = (destSnapPoint) => {
        this.mapRef.current._component.fitToCoordinates(
            [{
                latitude: 59.978315,
                longitude: 30.314519,
                latitudeDelta: 0.02,
                longitudeDelta: 0.02,
            }],
            {
                edgePadding: {
                    top: 0,
                    right: 0,
                    bottom: destSnapPoint,
                    left: 0,
                },
                animated: true,
            },
        );
    };

Any ideas ?

@rborn when using fitToCoordinates on a single coordinate, it gets zoomed really aggressively. It is a corner case happening for a single coordinate only. Is there a way to control the zoom level in this case?

@jacek213 i was having this issue with fitToCoordinates too but i couldnt get it to work, what did work is animateToRegion, like this
                mapRef.animateToRegion(
                  {
                    latitude,
                    longitude,
                    latitudeDelta: 0.01,
                    longitudeDelta: 0.01
                  },
                  2000
                );

for anyone who might run into this in future.

I was facing this issue too. try maxZoomLevel It worked for me.

I was facing this issue too. try maxZoomLevel It worked for me.

But maxZoomLevel won't allow to zoom even if the user wanted to. What I want is to control the level of zoom 'fitToCoordinates" gives

I was facing the same issue with fitToCoordinates, wrapping the call to that function inside of a requestAnimationFrame worked for me.
requestAnimationFrame(() =>
  this.mapRef.fitToCoordinates(
    [
      {
        latitude: eastMostPoint + latDelta / 3,
        longitude: northMostPoint + lngDelta / 3
      },
      {
        latitude: westMostPoint - latDelta / 3,
        longitude: southMostPoint - lngDelta / 3
      }
    ],
    {
      edgePadding: { top: 10, right: 10, bottom: 10, left: 10 },
      animated: false
    }
  )
)

I was facing the same issue with fitToCoordinates, wrapping the call to that function inside of a requestAnimationFrame worked for me.
requestAnimationFrame(() =>
  this.mapRef.fitToCoordinates(
    [
      {
        latitude: eastMostPoint + latDelta / 3,
        longitude: northMostPoint + lngDelta / 3
      },
      {
        latitude: westMostPoint - latDelta / 3,
        longitude: southMostPoint - lngDelta / 3
      }
    ],
    {
      edgePadding: { top: 10, right: 10, bottom: 10, left: 10 },
      animated: false
    }
  )
)


add spy report reading

add the possibility to read spy reports based on fleet id

Refer To:
[(https://github.com/alaingilbert/pyogame/pull/46)]
Ogames Spy Messages dosent store a Fleet id. Even with an Fleet ID your spy Satellites would have come back in this time so the fleet id becomes obsolete. You could just remember the coordinates or the name and search for your spy report that way.

implementet in new version


Delete locks in database

We have an application that processes incoming files and we are using the LockingTaskExecutor to prevent files from being processed on mutliple nodes. The name of the lock is the filename being processed and since this filename contains a timestamp, a new lock is created for each file. This leads to a large number of locks in the database as these locks are never deleted.
I would like to a have a method  to delete these logs safely from the database. Obviously I can delete old locks manually, but since this discouraged in the documentation I was looking for a better solution.

Hi, feel free to delete the locks rows in the database, if you are sure you will never need them again (after a day/week). If you want to be super-sure, you can call clearCache() on the lock provider right after the clean-up.

Ok, thank you for your reply.


Support named ports

Coverage increased (+0.09%) to 23.233% when pulling bc572ee564e1f15efa770eb2eea6653aa0030a39 on 472-named-ports into f80ccc06e2793a729dc240af6df5e211af974b2a on master.


TorrentFreaks Top-10 by KrazyKayaker

Addon Manifest URL
https://stremio-torrentfreak.krazykayaker.com/manifest.json
Addon Description
Provides an up-to-date list of TorrentFreak's "Top 10 Most Pirated Movies of the Week". It contains all the Top-10's from today until 2020.  Visit https://stremio-torrentfreak.krazykayaker.com for details.
Language of Content
None
Choose Labels

[ ] anime
[ ] asian drama
[ ] bollywood
[ ] cartoons
[ ] debrid support
[ ] http streams
[ ] live tv
[ ] metadata
[ ] misc
[X] movies
[ ] music
[ ] nsfw
[ ] podcasts
[ ] radios
[ ] subtitles
[ ] torrents
[ ] tv shows
[ ] video games

Great addon ,but on android tv version doesnt show the posters,pictures of the movie,on android mobile is showing

Thanks for the feedback @enko2974 .
I also noticed some inconsistancy with the documentation vs the various stremio platform.  The documentation clearly says:

You can use any unique string for the id. In this case we use the corresponding IMDB ID. Stremio features an system add-on called Cinemeta. This add-on provides detailed metadata for any movie or TV show that matches a valid IMDB ID.

This addon only returns IMDB IDs.  Although, it's a possibility to explicitly enrich it, I was reluctant to start to scrape for IMDB content within the addon... mainly because Cinemeta was suppose to do that (per the docs)... and scraping the IMDB content wasn't going to be easy and I'm certain the results would never be as good as Cinemeta.
You're right on Android (v1.5.9), everything is perfect.  Windows Desktop v4.4.149 sort of works, Cinemeta shows the artwork/poster, but the content itself is missing.  However, Desktop v4.4.160 (the latest version) is perfect.  This addon shows the exact artwork and metadata as everything else.
So, my guess is that the various Stremio platforms are moving towards having Cinemeta generate the content, but some platforms are further along than others.
Would using https://web.stremio.com/ be an option for Android TV?


Can't build with iOs...!  NOT WORKING

can't found any solution.
Please resolve this asap so I wrap-up app

Guys please review that. I need an urgent response on that , client is waiting for next build

Hi @faisalrajput ,
I have the same problem as you. How did you solve it?

This should be fixed now. Please download version 1.1.0 and follow the new linking instruction:
# RN >= 0.60
cd ios && pod install

# RN < 0.60
react-native link react-native-tus-client


Fosshub downloads

Scoop is trying to get
https://www.fosshub.com/IrfanView.html/iview451_plugins.zip or
https://www.fosshub.com/IrfanView.html/iview451_plugins_x64.zip
the links at fosshub points to
https://www.fosshub.com/IrfanView.html?dwl=iview451_plugins.zip or
https://www.fosshub.com/IrfanView.html?dwl=iview451_plugins_x64.zip
Error message while trying to install irfanview via scoop:
~ $ scoop install irfanview -a 32bit
Installing 'irfanview' (4.51) [32bit]
Loading iview451.zip from cache
Checking hash of iview451.zip ... ok.
Der Remoteserver hat einen Fehler zurückgegeben: (404) Nicht gefunden.
URL https://www.fosshub.com/IrfanView.html/iview451_plugins.zip is not valid

Same with kitty.json here in extras. Originial cause should be in commits 14a4612 and 803525a.

Bump: just ran into this issue as well.

Hm, the real problem here: Irfanview plugins does not support direkt download anymore, so scoop might not be able to download the ZIP files. I did not find another place to get the plugin ZIPs as a direct download.

Fixed in #1449

@rasa - kitty is still through Fosshub and will continue to fail.

Good point. Duh.


While uploading contract  file for the project the UI view is misaligned

Describe the bug
When contract file for the project  is uploaded, in the Upload field the name of contract file is mislaigned in UI view
Refer screenshot

To Reproduce
Steps to reproduce the behavior:

Go to CRM
Click on Projects
Click on any Project
Click on Edit button to upload a contract file for the project

Expected behavior
Contract file name should be alligned in UI view


Deprecated notice using ContentRangeUploadHandler.php

What steps will reproduce the problem?
Enable deprecated notice log on php
What is the expected result?
No deprecated
What do you get instead?
preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated in /var/www/instream-project-test.impactify.dev/build-9d9fc6b6/vendor/pion/laravel-chunk-upload/src/Handler/ContentRangeUploadHandler.php on line 113
Additional info
To resolv issue please change in
/vendor/pion/laravel-chunk-upload/src/Handler/ContentRangeUploadHandler.php
Line 70 :
$contentRange = $this->request->header(self::CONTENT_RANGE_INDEX);
To :  $contentRange = $this->request->header(self::CONTENT_RANGE_INDEX, '');

Q
A

This Package Version
1.5.2

Laravel Framework Version
10.9.0

PHP version
8.1.5

Operating system
Ubuntu / Debian

no news ?

Thanks to @RTippin is is completed. Thank you.
https://github.com/pionl/laravel-chunk-upload/releases/tag/v1.5.4


Can I make yakui look whatever I want it to look like?

Can I make yakui UI look whatever I want it to?
For example like pygui

nuclear's buttons

etc etc?

Yeah! Right now the styling available with the built-in widgets is pretty limited, but custom widgets are easy to implement. The goal is that games generally will use the built-in widgets at first, and then eventually create custom widgets that fit their game's visual style.
The yakui-widgets crate is designed to be forkable, so you can use its widgets as a starting point. Eventually, we'll have a better slate of primitives for building very custom widgets.

@LPGhatguy I see mate thank you :)


use standard date picker for the dashboard/analytics page

FRONTEND PULL REQUEST
Related Issue

The date picker used from the uswds is not accessible and the standard input is preferred.

Changes Proposed

Replace the date picker with the input object on the /dashboard page.
Moved loading message from being the entire page to just the data portion of the page. This was done to prevent all of the inputs from reloading as well in the middle of typing.

Additional Information

From my experience, trying the date input value is very finicky requiring a specific data format so when possible I would avoid setting that value.

Testing

Run locally (since there are no available environments to publish to at the time of writing this), Go to the dashboard, select custom range from the drop down and set dates and expect the correct values to appear in the data portion below
Ensure there are no accessibility issues with the new date pickers.

Screenshots / Demos
https://user-images.githubusercontent.com/10108172/191363270-4a757722-0032-4541-a530-b758f1403092.mp4

This makes sense to me Boban! I did notice the validation is a little pushy while typing the date in (not via date picker). It might be less disruptive if it only validated after moving off the date field, but based on what you mentioned about this implementation being tricky, I think it's ok.
Screen.Recording.2022-09-21.at.9.28.01.AM.mov

Woops! forgot to clarify this change was only applied on the dashboard analytics page, there are still a few places in the code that use the other one though!

ooh gotcha! my bad for not making sure I was on the correct tab 😆

@BobanL One thing I'm noticing is that on the custom date range on the Dashboard page, I can't tab into the datepicker like I could on the other component used on the Results page, so the date picker is not accessible I think. Is that intentional?

also fwiw, the little datepicker thingy doesn't show up in firefox 🤔 but we can still enter a date by typing so I think it's fine 😁

@BobanL One thing I'm noticing is that on the custom date range on the Dashboard page, I can't tab into the datepicker like I could on the other component used on the Results page, so the date picker is not accessible I think. Is that intentional?
edit: screen capture. should I be testing this differently for accessibility?

also fwiw, the little datepicker thingy doesn't show up in firefox 🤔 but we can still enter a date by typing so I think it's fine 😁

Good call on the tabbing check! I forgot about it :). So from my understanding, the input element we are using now is just the native html element and each browser makes it own decisions on how to implement it, that's why the dropdown may look different in each browser. That button/datepicker that appears is totally rendered by the browser and is not part of our code. I do not believe there is anything we can really do about it to change it.
Chrome:

Firefox:

Safari:

@BobanL One thing I'm noticing is that on the custom date range on the Dashboard page, I can't tab into the datepicker like I could on the other component used on the Results page, so the date picker is not accessible I think. Is that intentional?
edit: screen capture. should I be testing this differently for accessibility?

also fwiw, the little datepicker thingy doesn't show up in firefox 🤔 but we can still enter a date by typing so I think it's fine 😁

Good call on the tabbing check! I forgot about it :). So from my understanding, the input element we are using now is just the native html element and each browser makes it own decisions on how to implement it, that's why the dropdown may look different in each browser. That button/datepicker that appears is totally rendered by the browser and is not part of our code. I do not believe there is anything we can really do about it to change it.

Thanks for explaining that! I think the important thing is it's still possible to enter a date by tabbing through ✅


update QC pdf definition

Coverage decreased (-0.09%) to 64.392% when pulling 4e7e1ee36ce3c7cfad89cfba2bc2dd48d1e2038d on leslieaula:dev into f045ba4b46304c8cf3d6d1ec72d1063202aad3d2 on danliris:dev.


Upgrade to 3.10

Description
Please include a summary of the change and which issue is fixed. Be sure to reference the ISSUE that this PR addresses which should have been approved by an earthpy maintainer.
Fixes #96 and #94
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Built a docker container with the new environment, see https://hub.docker.com/repository/docker/earthlab/earth-analytics-python-env/general
Docker container successfully runs using key packages (rioxarray, geopandas) in GitHub Actions ubuntu-latest, and GitHub Codespaces

Checklist:

[ ] I have already submitted an issue and i was invited to submit a pr by an earthdatascience.org maintainer
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

The merge conflicts come from an updated Action. The action in questions was based on a similar action used by Cyverse for their geospatial containers (i.e. it activates the conda environment robustly and installs some jupyterlab extensions.

Note the following potential problems, which I have not been able to reproduce:

I've gotten reports of the rasterstats package failing with a message about an incompatible version of numpy.
I've eliminated the pins on nbconvert and jupyter_contrib, even though the coroutine error that inspires those has not been fixed in the jupyter notebook extension (nor do there seem to be plans to do so). Please use jupyter lab instead, which runs nbgrader commands without incident.


Error in bower.json file bootstrap4v2

When I try to run the command "bower i", I see this error
bower bootstrap#^v4.0.0-alpha.6       EMALFORMED Failed to read C:\Users\MURON\AppData\Local\Temp\DESKTOP-C4FMOVN-MURON\bower\ca4c50b905dc21ea17a10549a6f5944f-12648-A0s3u3\bower.json
Additional error details:
Unexpected token @ in JSON at position 0
If I use git://github.com/twbs/bootstrap.git#v4.0.0-alpha.6 (without ^), all good, but if I use git://github.com/twbs/bootstrap.git#v^4.0.0-alpha.6 and try loaded last version I see this error
http://c2n.me/3OL911r

They dropped bower support for beta.2, I am trying to find out how to migrate to NPM. https://stackoverflow.com/questions/46854634/moving-from-bower-to-npm-and-using-main-bower-files

Bower is dead, so we've removed it from v4.
duplicate of #24451

@myorangeca let me know if you have a challenge migrating to NPM.

@jgacuca567 :( Struggling for the last 3 days on it. I ended up posting a question on stackoverflow (link above). Love to get your support or a hint on the right direction. :)

@myorangeca
I had the same issue. I resolved it using following dependencies in package.json (NPM):
"popper.js": "1.12.3",
"bootstrap": "4.0.0-beta.2"
It creates the folder under node_modules\bootstrap\
Be aware using nodejs version >= 6. In Visual Studio 2017 I had to change environment variable priority.
([https://stackoverflow.com/questions/43849585/update-node-version-in-visual-studio-2017](Stackoverflow Update Node Version in Visual Studio 2017)
Hope this solves your migrating problem.

OK, I just published an article on Medium - with step-by-step instructions on how to add React to your ASP.Net core projects. Check out the article here (includes support for Bootstrap 4.0 Beta 2):
https://medium.com/@csharpsolutionsuk/how-to-use-webpack-in-asp-net-core-projects-a-basic-react-template-sample-25a3681a5fc2
Github repo: https://github.com/phantom2017-Nov/AspNetCore-ReactTemplate

@Johann-S  if Bower is dead then why is Bootstrap 4.0.0-beta.2 in the bower package manager?

You won't be able to pull Bootstrap beta 2 from Bower 😉


Update syndication.md to fix typos

This fixes two typos or word-choice problems describing what syndication packages are for.

Thanks!


Error sweep: fix error messages for timing out Runs

Changes
This commit fixes the user facing error messages for timing out Runs as well as the controller logs.
/kind misc

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[ ] Has Tests included if any functionality added or changed
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including functionality, content, code)
[ ] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/lgtm


Region plugin content going out of bounds

Sure, we could do that.
I think the part of the plugin that renders a table is better handled on the application level, but I can see value in parsing the ELAN XML into JSON and rendering annotations as regions.
Originally posted by @katspaugh in https://github.com/katspaugh/wavesurfer.js/discussions/3072#discussioncomment-6623638
In an attempt to replicate this I believe I have uncovered a bug where the annotations in the regions are being pushed off of the screen after n amount of regions are rendered
see attached photo

Oof, that's an ugly one :)
Thank you!

I have been attempting to band-aid fix it with this:

But I am having no joy with the selectors.
Any insight on this?
Happy to help, such a marvelous library!

You almost got it right, but it should be without :hover.

Ah yes, the hover was a reminant from my testing - the waveform was the same ID used but alas it did not seem to work

solved my issue with the selectors not workign by using ::ng-deep in angular

::ng-deep #waveform ::part(region-content) {   margin-top: 0px !important; }


move loki usage guide into handbook

Towards: https://github.com/giantswarm/giantswarm/issues/28020
This PR move the Loki usage guide into our public handbook.

Copy Loki usage from intranet
Some rewording

Please make sure you add the front-matter tags as described here: https://intranet.giantswarm.io/docs/security/isms/operations/publish-intranet-docs/#initial-assessment

Please make sure you add the front-matter tags as described here: https://intranet.giantswarm.io/docs/security/isms/operations/publish-intranet-docs/#initial-assessment

That's done


Migrate ms-output to k8s testbed

Impact of the new feature
The ms-output microservice will be disabled in the VM testbed and will start to run in the k8s testbed environment.
Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Create required deployment and k8s service configuration changes to disable ms-output in the VM testbed and enable ms-output in the k8s testbed. Work with Imran to schedule the migration of the service.
Describe alternatives you've considered
None
Additional context
Requires changes to how mongodb is handled in the k8s environment, see issue: https://github.com/dmwm/WMCore/issues/10180

ms-output and associated mongodb pod are running in k8s testbed as of Jan. 13. Closing this issue.

ms-output and associated mongodb pod are running in k8s testbed as of Jan. 13. Closing this issue.


[bug] [0.3.0-1.21]option descriptions persist after hovering off the option

video shows how to achieve the bug

Hi @krispyking24, this seams to be related to the ModMenu mod, because is only on ModMenu and not on Achievement page


WSL - wine cannot access to the host path DLL files

Windows Version
Microsoft Windows [Version 10.0.22621.1265]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
Linux version <IP_ADDRESS>-microsoft-standard-WSL2 (oe-user@oe-host) (x86_64-msft-linux-gcc (GCC) 9.3.0, GNU ld (GNU Binutils) <IP_ADDRESS>00220)
Distro Version
Arch Linux latest
Other Software
wine-8.4
Repro Steps
Hello,
I'm using a WSL image of Arch Linux in Windows 11 and, since the WSL can access to host folders, the PATH env variable on Arch WSL image has also System32 path. Usually, in WSL images, the C drive is accessed by /mnt/c directory.
I'm trying to use wine for running Remote Desktop Service of the Windows host (so, the mstsc.exe executable.
I noted that, when I try to run the Remote Desktop Service by "mstsc.exe" or "wine mstsc.exe" I get the following output:
0084:fixme:hid:handle_IRP_MN_QUERY_ID Unhandled type 00000005
0084:fixme:hid:handle_IRP_MN_QUERY_ID Unhandled type 00000005
0084:fixme:hid:handle_IRP_MN_QUERY_ID Unhandled type 00000005
0084:fixme:hid:handle_IRP_MN_QUERY_ID Unhandled type 00000005
0108:err:module:import_dll Library api-ms-win-core-commandlinetoargv-l1-1-0.dll (which is needed by L"Z:\\mnt\\c\\Windows\\system32\\shcore.dll") not found
0108:err:module:import_dll Library shcore.dll (which is needed by L"Z:\\mnt\\c\\Windows\\system32\\SHLWAPI.dll") not found
0108:err:module:import_dll Library SHLWAPI.dll (which is needed by L"Z:\\mnt\\c\\Windows\\system32\\mstsc.exe") not found
0108:err:module:import_dll Library api-ms-win-core-commandlinetoargv-l1-1-0.dll (which is needed by L"Z:\\mnt\\c\\Windows\\system32\\SHCORE.dll") not found
0108:err:module:import_dll Library SHCORE.dll (which is needed by L"Z:\\mnt\\c\\Windows\\system32\\mstsc.exe") not found
0108:err:module:LdrInitializeThunk Importing dlls for L"Z:\\mnt\\c\\Windows\\system32\\mstsc.exe" failed, status c0000135

despite these dll files exist:
ls /mnt/c/Windows/System32/shlwapi.dll
 /mnt/c/Windows/System32/shlwapi.dll

ls /mnt/c/Windows/System32/SHCore.dll
 /mnt/c/Windows/System32/SHCore.dll

I tried also to delete $HOME/.wine and run env WINEPREFIX=$HOME/.wine wine /mnt/c/Windows/System32/mstsc.exe but I get the same error.
Of course, on Windows host, I can run "mstsc.exe" by CMD with no issues.
It seems that on a WSL container, Wine cannot have access to the dependencies/DLLs on the Windows host by the Windows paths mapped on WSL.
For reproducing this:

Install an Arch Linux WSL image
Install wine
Run wine mstsc.exe

Expected Behavior
wine mstsc.exe should show Remote Desktop Service Windows application.
Actual Behavior
Receive the following output:
0084:fixme:hid:handle_IRP_MN_QUERY_ID Unhandled type 00000005
0084:fixme:hid:handle_IRP_MN_QUERY_ID Unhandled type 00000005
0084:fixme:hid:handle_IRP_MN_QUERY_ID Unhandled type 00000005
0084:fixme:hid:handle_IRP_MN_QUERY_ID Unhandled type 00000005
0108:err:module:import_dll Library api-ms-win-core-commandlinetoargv-l1-1-0.dll (which is needed by L"Z:\\mnt\\c\\Windows\\system32\\shcore.dll") not found
0108:err:module:import_dll Library shcore.dll (which is needed by L"Z:\\mnt\\c\\Windows\\system32\\SHLWAPI.dll") not found
0108:err:module:import_dll Library SHLWAPI.dll (which is needed by L"Z:\\mnt\\c\\Windows\\system32\\mstsc.exe") not found
0108:err:module:import_dll Library api-ms-win-core-commandlinetoargv-l1-1-0.dll (which is needed by L"Z:\\mnt\\c\\Windows\\system32\\SHCORE.dll") not found
0108:err:module:import_dll Library SHCORE.dll (which is needed by L"Z:\\mnt\\c\\Windows\\system32\\mstsc.exe") not found
0108:err:module:LdrInitializeThunk Importing dlls for L"Z:\\mnt\\c\\Windows\\system32\\mstsc.exe" failed, status c0000135

Diagnostic Logs
No response

I'm entirely befuddled. Why would you want to use wine instead of interop?

I'm entirely befuddled. Why would you want to use wine instead of interop?

How can I use interop for calling mstsc.exe?

If the software you've installed hasn't changed binfmt registrations then just mstsc.exe should work. Else it can be done explicitly like /init /mnt/c/WINDOWS/system32/mstsc.exe mstsc.exe

If the software you've installed hasn't changed binfmt registrations then just mstsc.exe should work. Else it can be done explicitly like /init /mnt/c/WINDOWS/system32/mstsc.exe mstsc.exe

I didn't do any changes on binfmt registrations. If I run just mstsc.exe I get:
Cannot open assembly '/mnt/c/Windows/system32/mstsc.exe': File does not contain a valid CIL image.

Like I said: "If the software you've installed..."
The software you've installed has. Please see: https://github.com/microsoft/WSL/discussions/9755


nixos/stage-1: use readlink -e in builder

The builder has this convoluted while loop which just replicates
readlink -e. I'm sure there was a reason at one point, because the
loop has been there since time immemorial. It kept getting copied
around, I suspect because nobody bothered to understand what it actually
did.
Incidentally, this fixes #17513, but I have no idea why.

Tested in a VM, looks good.

It may be a fix because of this line I think. We have set -e so when file does actually exist this will fail. It should have been || true or, as you've done, an if.

Oh, I did not realize set -e did that. Even after reading man bash I still would not have understood that without your explanation. Thanks!


Account import unexpected keyword in call to add_account

When trying to import a key:
ubuntu@ubuntu-VirtualBox:~/tools/pyethapp$ pyethapp account import ../pyethereum/zkey.hex
No handlers could be found for logger "eth.pow"
INFO:app	using data in path=/home/ubuntu/.config/pyethapp
INFO:config	loading config path=/home/ubuntu/.config/pyethapp
CRITICAL:app	Move to EthApp.default_config
WARNING:accounts	keystore directory does not exist directory=/home/ubuntu/.config/pyethapp/keystore
WARNING:accounts	no accounts found
INFO:app	registering service service=accounts
Password to encrypt private key:
Repeat for confirmation:
Traceback (most recent call last):
  File "/usr/local/bin/pyethapp", line 9, in <module>
    load_entry_point('pyethapp==1.0.1', 'console_scripts', 'pyethapp')()
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 700, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 680, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 1027, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 1027, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 873, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 508, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/click/decorators.py", line 16, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/ubuntu/tools/pyethapp/pyethapp/app.py", line 500, in import_account
    app.services.accounts.add_account(account, path=account.address.encode('hex'))
TypeError: add_account() got an unexpected keyword argument 'path'

I already have a PR from my account, so I'll just paste the fix here :
ubuntu@ubuntu-VirtualBox:~/tools/pyethapp$ git diff
diff --git a/pyethapp/app.py b/pyethapp/app.py
index 4542583..b8cbc1f 100644
--- a/pyethapp/app.py
+++ b/pyethapp/app.py
@@ -496,8 +496,11 @@ def import_account(ctx, f, uuid):
         password = click.prompt('Password to encrypt private key', default='', hide_input=True,
                                 confirmation_prompt=True, show_default=False)
     account = Account.new(password, privkey, uuid=id_)
+    account.path = os.path.join(os.path.abspath(ctx.obj['config']['data_dir']),
+                                ctx.obj['config']['accounts']['keystore_dir'],
+                                account.address.encode('hex'))
     try:
-        app.services.accounts.add_account(account, path=account.address.encode('hex'))
+        app.services.accounts.add_account(account)
     except IOError:
         click.echo('Could not write keystore file. Make sure you have write permission in the '
                    'configured directory and check the log for further information.')

Thank you!
Fixed in a7788d6cbe32d45f50b630ce001765e1d616ef9c (requires latest commit of pyethereum)


Clarify hostname prerequisites especially for Windows downstream devices

A few IcMs and Github issues have been filed recently related to configuring a Windows downstream device to connect to an Edge transparent gateway device using IoT Edge 1.2.  In each of these cases, the customer saw TLS authorization errors in EdgeHub logs because they failed to meet the prerequisites documented in the following article: https://docs.microsoft.com/en-us/azure/iot-edge/how-to-authenticate-downstream-device?view=iotedge-2020-11#prerequisites
Specifically, the customers had the following configuration errors on their downstream devices:

Misconfiguration of the Gateway Host Name in the connection string. The customer used the FQDN but needed to use the non-FQDN.
Missing entry in the hosts file to map the gateway device's non-FQDN to its IP address.

These steps should be made more clear and explicit in the public documentation.

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

ID: 3512ec24-1146-0dc8-4529-80d4770e1ec2
Version Independent ID: 8abd57ee-a785-4d47-c0e5-6ac713367279
Content: Authenticate downstream devices - Azure IoT Edge
Content Source: articles/iot-edge/how-to-authenticate-downstream-device.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

Hello @nlcamp Thanks for the feedback.
Our SME will review this feedback and get back soon on this.

Thanks for the feedback, a new section was added to the documentation in the prerequisites section:
"This article refers to the gateway hostname at several points. The gateway hostname is declared in the hostname parameter of the config file on the IoT Edge gateway device. It's referred to in the connection string of the downstream device. The gateway hostname needs to be resolvable to an IP Address, either using DNS or a host file entry on the downstream device."
This should warn readers about the hostname specifications. Closing this issue, but feel free to reopen if you need to continue the discussion.
#please-close


how to clear the existing modal contents before modal.load()

My use case is this:
I have a get() request each time a modal is loaded but I can´t seem to clear existing content without a hack.
Any ideas?

There are two hooks:  onBeforeClose / onClose
That may potentially help you out there by allowing you to script in a destory/clear/sanetize depending on what you would need to achieve.
A full list of options are available on here :)
https://github.com/thephuse/vanilla-modal

Thanks for replying so quickly.
Seems that those hooks don´t call the defined function:
var clearModalContent = function() {
  console.log('clear content');
};

const modal = new VanillaModal({
  onClose: clearModalContent()
});

Opps, sorry mate. Needed an anonymous function there :-)
This works:
var clearModalContent = function() {
  console.log('clear content');
};

const modal = new VanillaModal({
  onClose: function() {
    clearModalContent()
  }
});

All good! Re open if you need :)