Update BVMF holidays to account for 2022's changes

Starting in 2022, B3 will operate normally on municipal and state holidays, namely Aniversário de São Paulo, Revolução Constitucionalista, and Consciência Negra.
Here's a circular letter issued by B3 on 2020 anticipating some of these changes:

https://www.b3.com.br/data/files/55/37/81/D0/038D5710700BFA57AC094EA8/OC 150-2020 PRE -  2021_2022 Calendar_Public Holidays_ING.pdf (see Section 2.2).

And here are some external sources (only in portuguese, sorry):

https://www.infomoney.com.br/mercados/bolsa-divulga-calendario-e-informa-que-tera-pregao-em-feriados-municipais-de-sp-a-partir-de-2022-veja-as-datas/
https://economia.uol.com.br/noticias/estadao-conteudo/2020/11/18/b3-tera-pregao-em-feriados-municipais-de-sp-a-partir-de-2022.htm
https://guru.com.vc/noticias/feriado-de-25-de-janeiro-b3-vai-abrir-vai-funcionar-o-pregao/
https://www.spacemoney.com.br/geral/b3-funcionara-normalmente-no-aniversario-da-cidade-de-sao-paulo/176793/

I believe it can be fixed easily by adding the end_date parameter to each of the aforementioned holiday instances.
Since this seems like a good first issue, I could tackle it as my first contribution.

@almeidaxan I've fixed the holiday for Aniversário de São Paulo in !162
As you mentioned it is enough to add end_date and then regenerate the csv files used for testing via
python etc/make_exchange_calendar_test_csv.py BVMF

This issue was only partially implemented - it is still necessary to update two more holidays: Revolução Constitucionalista and Consciência Negra.
@gerrymanoim Could you please reopen it?

Apologies - reopened!

No worries - I'll try to work on it later today.


add shorthand colorspace descriptions

Adds a few special values for the colorspace description strings: sRGB, DisplayP3, Rec2100PQ, and Rec2100HLG. These are probably the most commonly used and relevant color spaces right now and in the foreseeable future.
This makes it somewhat more convenient to manually specify these colorspaces when e.g. using cjxl on a ppm or pfm input, that is, you can now also do -x color_space=sRGB instead of having to do -x color_space=RGB_D65_SRG_Per_SRG which I find hard to remember.
It also has the effect of making the description string a little shorter (and arguably easier to understand) in the corresponding synthesized ICC profiles.

These are probably the most commonly used and relevant color spaces right now and in the foreseeable future.

P3 HLG (and P3 PQ?) options might be handy as well - I think this is what Adobe Camera Raw exports in AVIF, and what Apple use as well.

These are probably the most commonly used and relevant color spaces right now and in the foreseeable future.

P3 HLG (and P3 PQ?) options might be handy as well - I think this is what Adobe Camera Raw exports in AVIF, and what Apple use as well.

Adobe Camera Raw gives a choice between "HDR sRGB (Rec.709)", "HDR P3", and "HDR Rec.2020". It looks like these all use the PQ transfer function, and "HDR Rec.2020" corresponds to Rec2100PQ.
And yes, it looks like Apple uses P3 HLG.
Would it make sense to add DisplayP3_HLG and DisplayP3_PQ? (or without the underscore, or with a dash instead?) You can already use the longer description string RGB_D65_DCI_Rel_HLG / RGB_D65_DCI_Rel_PeQ for these though.
I want to keep the list of spaces with a shorthand name relatively short, because you could add spaces like Adobe98 and ProPhoto that are also still relatively common but at some point we're turning this into a colorspace name registry and that's not the intention :)

I think I would have them without “Display”, because Display P3 is already “P3 primaries + sRGB white point and transfer function”, so replacing the transfer function again arguably makes it not Display P3 anymore.

Would it make sense to add DisplayP3_HLG and DisplayP3_PQ? (or without the underscore, or with a dash instead?) You can already use the longer description string RGB_D65_DCI_Rel_HLG / RGB_D65_DCI_Rel_PeQ for these though.

I guess you're right - it's easier to add more shorthands later if really needed (i.e. frequently requested) than overthink it now...

I think I would have them without “Display”, because Display P3 is already “P3 primaries + sRGB white point and transfer function”, so replacing the transfer function again arguably makes it not Display P3 anymore.

I agree, and also shorter is better. We could go both ways: Adobe uses the string "P3 PQ" (in the ICC description field) while Apple uses the string "displayP3_HLG". I would go for P3PQ and P3HLG, maybe. But for now, let's start with what's in this PR and more can be added later.


create your own primitive for matrix valued function

Is it possible to create a primitive for a matrix valued function, ie func(x) = y where x is a vector and y is a matrix
For example, if I have matrix, A:
A = [
[x[0], x[1]].
[x[3], x[4]] ]
I would like to give autograd the jacobian dA/dx as:
dA/dx = [
[[1,0,0,0]. [0,1,0,0]],
[[0,0,1,0], [0,0,0,1]]]
If possible, can you please provide an example for this case? I have been using defvjp and having dimensional mismatch issues.

Autograd doesn't (directly) compute the jacobian of intermediate operations. Instead what you have to define is the vector-jacobian product vjp. So instead of telling autograd how to compute dA/dx, you have to tell it (using defvjp) how to compute the product np.dot(dfdA, dAdx), where df/dA is some gradient "flow" coming from the subsequent operations which used A on the forward pass. For your particular example, I think something like this should work, as long as x always has shape (4, ) :
def A_vjp(A, x):
    dAdx = np.array([[[1,0,0,0], [0,1,0,0]], [[0,0,1,0], [0,0,0,1]]])
    return lambda v: np.dot(v, dAdx)
defvjp(A, A_vjp)

Thanks for your quick response. Unfortunately the simple np.dot did not work, I still get a dimension mismatch error in my case.
However, by considering the dimension of the forward pass, as you mention, I was able to fix my problem by using the np.tensordot function. Something like this:
def A_vjp(A, x):
dAdx = np.array([[[1,0,0,0], [0,1,0,0]], [[0,0,1,0], [0,0,0,1]]])
return lambda v: np.tensordot(v, dAdx)
defvjp(A, A_vjp)
I matched the output of A_vjp (ie product of dfdA and dAdx) to be the same dimension as dfdx.


[WIP][Paying for College] Remove jQuery from Disclosures app

This PR will remove jQuery from the P4C "Dynamic Disclosures" app.
Originally, the intent was to add a bunch of cypress tests to verify the app's functionality, then remove all the jQuery, but after a long time trying to figure out why GHA failed all the cypress tests that passed locally, I finally found that it was the jQuery itself that was causing the GHA cypress failures, so we couldn't really write a PR with passing tests while the jQuery was in there.
Thus, this PR adds the cypress tests while also removing the jQuery. Not ideal, but it is easy enough to first verify the cypress tests pass with the existing code, then check again that they pass with this branch. See "How to test this PR" for a quick command to copy/paste to test the cypress in our main branch locally.

Additions

Add cypress tests for Dynamic Disclosures
Add dollar-sign.js which is a vanilla replacement for some jQuery functions

Removals

removed jQuery!

Changes

Change some jQuery calls to vanilla JS calls

How to test this PR

Pull down the disclosures branch
From your main branch, you can checkout just the cypress tests:
git checkout disclosures -- test/cypress/fixtures/paying-for-college/ test/cypress/integration/paying-for-college/disclosures/
Run the cypress tests (under paying-for-college/disclosures in your cypress interface) against the main branch, where jQuery still reigns supreme. (Make sure your JS files are built in this branch before testing!)
Then go to the disclosures branch, rebuild your JS files, and then try the cypress tests again!
Check out the general functionality of the disclosures app. Here's a test URL: test url for localhost
When you're done, don't forget to clean up your main branch if you did the git checkout. - switching to main and doing a git clean -i will get it done!

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code follows the standards laid out in the CFPB development guidelines
[ ] Future todos are captured in comments and/or tickets
[ ] Project documentation has been updated, potentially one or more of:

I think you've mentioned this before, but I'm noticing some weird overlap of the dollar-sign with the "Updating" text as seen below:

Yep! I figured that out last week - it was because my siblings method was bad, and the change I just pushed should fix that.


version 0.13.2 missing class in the jar file?

Honestly this is just after upgrading to 5.0 API for like 10 plugins. But the error seemed specific enough to post here.
[01:02:22 ERROR] [Sponge]: Could not pass GameInitializationEvent$Impl to Plugin{id=pjp, name=ProjectPortals, version=0.13.2, description=One
stop shop for all things portal, url=http://github.com/TrenTech/ProjectPortals, authors=[TrenTech], source=mods/projectportals-5.0.0-0.13.2.ja
r}
java.lang.NoClassDefFoundError: com/gmail/trentech/helpme/help/Usage
        at com.gmail.trentech.pjp.Main.onInitialization(Main.java:94) ~[Main.class:?]
        at org.spongepowered.common.event.listener.GameInitializationEventListener_Main_onInitialization40.handle(Unknown Source) ~[?:?]
        at org.spongepowered.common.event.RegisteredListener.handle(RegisteredListener.java:95) ~[RegisteredListener.class:1.10.2-6.0.0-BETA-1
49]

09a515e9fa258afdbf6d4505c71b193e5c302505


How to show YUV image with correct colors?

Hi,
My test image:
Image_YUYV_1920x1200
I have a YUYV format image and I want to show the image in PixelViewer.
But It showed a image with incorrect color when I selected YUYV whatever I changed setting.

I expect the correct image like below:

I currently set L16 to show mono image to check image but I hope I can get color image with PixelViewer.

Thanks!

@jigoku1206 YUYV format is not currently supported by PixelViewer. We will add support with next preview release.

Seems the image is YVYU instead of YUYV. Both formats will be support in next preview release.

@jigoku1206 YUYV and YVYU are now supported in <IP_ADDRESS>8.

Thanks for your help!


Support pet packaging with cide

This allows for building in docker with cide.
This PR also supports packaging the code: cide is configured to use script/build when building a package, which creates a deployable pet package.

Oops I merged by accident, thinking it was this PR you approved @jameshfisher. Let me know if there's any problems with it, but otherwise I'm assuming it's fine.

@WillSewell sorry, yes, LGTM, comparing with other cide packages (I didn't test it though)

do you guys want to take over the cide project? happy to transfer it over if you want

If you'd rather not look over it, then I'd be happy to have it transferred over to us

pusher/cide already exists and You don’t have the permission to create repositories on pusher
I made you both collaborators in the mean time


VSC doesn't let me click in random rows - my windows updated today, can this cause this bug?

Issue Type: Bug
I inserted for example a predefined built-in snippet "form" and i can't click in the highlighted 3 rows.

every extension is disabled (I reactivated 1 extension after final test, no extensiont were active and bug still remained)

every custom json setting is removed

I run it as administrator

pc restarted

if I restart VSC it works again and I can click them

UNTIL I add another "form" using the built in snippet

Do you have ever seen something like this?
VS Code version: Code 1.49.0 (e790b931385d72cf5669fcefc51cdf65990efa5d, 2020-09-10T13:22:08.892Z)
OS version: Windows_NT x64 10.0.19041

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4200U CPU @ 1.60GHz (4 x 2295)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.89GB (3.23GB free)

Process Argv
--crash-reporter-id 3dc3553f-51a8-40a9-a2de-a839d6028789

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

snippet-creator
rya
0.1.3

![bug](https://user-images.githubusercontent.com/63600657/93376566-16b61c00-f85a-11ea-8185-b5cfd4fd6845.jpg)

Update: seems reinstalling vsc solved the problem, when I installed again and activated "custom css and js loader" then vsc showed this error again, so I guess it was the extension which modified files which couldn't be reverted by only a disable

/needsMoreInfo
Is this still an issue then? I have also trouble following your steps.


Add xresources color support

This will use the current terminals foreground and background colors as set in the currently used xresources.
New order of colors is then:
ARG -> ENV -> XRES -> Fallback(black and white)
This does not add any additional dependencies.
NOTE: This uses currently active window for getting the colors! As such this could fail.
Example if used with sleep
sleep 5; fontpreview-ueberzug

Well, for me I use *.{fore,back}ground in my Xresources file, in which case the method in the PR does not work. And there are some terminal emulators that does not use Xresources but their own config files. So I am afraid the PR could only cover a small part of all cases.
The users might not mind to add one or two lines of env var definitions in their shell rc file if they want to achieve a more blend-in look.


Failover not working on Rancher Kube cluster

Failover Question:
I've got a Rancher-installed Kubernetes cluster on Ubuntu 18.04, standing up the cluster with no customization except that I added heketi/gluster storage for storage redundancy.  Maybe I'm missing some step here, but my sample app containers do not fail over to a working node when the node they're on is down.  I've waited 15 minutes with no action being taken by the cluster except it shows that the desired number of containers is not available.  Once I turn the down node back on, the containers there resume.  I'd expect this for daemonsets but not for regular apps like mysql.
I've looked at this case https://github.com/rancher/rancher/issues/14712 referencing setting tighter time limits for down containers before they are recovered, and in following the advice I've found that the cluster's yaml file doesn't even have an active kube_api or kube_controller section.  I do have this, but as you can see, it is commented out:

So am I missing some major step, or does swarm really kick Kubernetes' butt in re-scheduling down containers elasewhere?
Environment information

Rancher version: v2.1.1
Installation option (single install/HA): Single install
Network: flannel

Cluster information

Cluster type: Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory): VM, 4 vcpu's, 16 GB RAM
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.0", GitCommit:"ddf47ac13c1a9483ea035a79cd7c10005ff21a6d", GitTreeState:"clean", BuildDate:"2018-12-03T21:04:45Z", GoVersion:"go1.11.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.3", GitCommit:"a4529464e4629c21224b3d52edfe0ea91b072862", GitTreeState:"clean", BuildDate:"2018-09-09T17:53:03Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

Docker (use docker info):

Containers: 106
 Running: 95
 Paused: 0
 Stopped: 11
Images: 42
Server Version: 18.09.0
Storage Driver: overlay2
 Backing Filesystem: extfs
 Supports d_type: true
 Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
 Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: c4446665cb9c30056f4998ed953e6d4ff22c7c39
runc version: 4fc53a81fb7c994640722ac585fa9ca548971871
init version: fec3683
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.15.0-42-generic
Operating System: Ubuntu 18.04.1 LTS
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 15.65GiB
Name: vva-er-k8s0
ID: GBFE:IF3Z:ZQTI:7SIW:MN55:HM33:QKMA:4G6D:PZGN:PPQU:PMAP:XIFQ
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false
Product License: Community Engine

WARNING: No swap limit support

If there are no services defined, the defaults are used. You can add/override parameters as described in https://rancher.com/docs/rancher/v2.x/en/cluster-provisioning/rke-clusters/options/#config-file and we have a FAQ entry describing the default Kubernetes behavior: https://rancher.com/docs/rancher/v2.x/en/faq/technical/#why-does-it-take-5-minutes-for-a-pod-to-be-rescheduled-when-a-node-has-failed
By default, it takes around 5 and a half minutes to reschedule a pod. If this doesn't happen, please describe steps to reproduce and the output from the kube-controller-manager container and kubectl describe output from the resource you created.

OK... my cluster config yaml file contains the following service section:
services:
etcd:
extra_args:
election-timeout: "5000"
heartbeat-interval: "500"
snapshot: false
kube-api:
pod_security_policy: false
service_node_port_range: "30000-32767"
kube-controller:
extra_args:
node-monitor-grace-period: "16s"
node-monitor-period: "2s"
pod-eviction-timeout: "30s"
kubelet:
extra_args:
node-status-update-frequency: "5s"
fail_swap_on: false
I'm still not seeing any failover.

How many nodes with what roles do you have in the cluster? Did you confirm the settings were applied (docker inspect kube-controller-manager) on the hosts? What are you deploying and what is the logging as requested in https://github.com/rancher/rancher/issues/17142#issuecomment-448001465

I have 3 master nodes with all roles assigned, one worker node with just worker assigned.
I do see the kube-controller-manager container, and the age of it matches when I added the kube-controller args to the cluster yaml.
(docker ps output):
468f26f3de88        rancher/hyperkube:v1.11.3-rancher1   "/opt/rke-tools/entr…"   18 hours ago        Up 18 hours                             kube-controller-manager
However, if I "kubectl get service --all-namespaces" I don't see it listed as a service.  Same with pods.

The last lines of the container logs:
I1217 22:08:52.603563       1 controllermanager.go:122] Version: v1.11.3
W1217 22:08:52.604052       1 authentication.go:55] Authentication is disabled
I1217 22:08:52.604164       1 insecure_serving.go:49] Serving insecurely on [::]:10252
I1217 22:08:52.605217       1 leaderelection.go:185] attempting to acquire leader lease  kube-system/kube-controller-manager...

Let me know if this still causes issues, this should be default Kubernetes logic. The logs requested are from all kube-controller-manager containers as there is only one active which will log the actions.


Roll Flutter Engine from 60c4e61416cd to 36730dd4e4b1 (1 revision)

https://github.com/flutter/engine/compare/60c4e61416cd...36730dd4e4b1
2023-04-25<EMAIL_ADDRESS>Roll Dart SDK from 337365e42613 to 170137fc759b (2 revisions) (flutter/engine#41483)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jsimmons@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


[Bug Report] Should "v-model" be always included in "el-input"?

Element UI version
2.8.2
OS/Browsers version
Chrome
Vue version
2.6.10
Reproduction Link
https://jsfiddle.net/nh0627/mj41w2ac/3
Steps to reproduce
If I type anything into el-input without v-model, el-input itself cannot contain any letter.
It seems like el-input always requires a user to declare v-model to use it.
What is Expected?
A user can use "el-input" without "v-model", in order to available to enter any words even without "v-model".
What is actually happening?
If there is no "v-model" in "el-input", "el-input" cannot contain any letter.

Why does it make sense to use el-input without value binding?

@georgyfarniev  I mean, I have to render an element with createElement()(https://vuejs.org/v2/guide/render-function.html#createElement-Arguments), but in this case, I cannot use "v-model" as a property of the method.
Rather than v-model="foo", I have to use other version of syntax, v-bind:value="foo" v-on:input="e => foo = e.target.value", with createElement().
However, el-input seems it does not support v-bind:value="foo" v-on:input="e => foo = e.target.value".
https://jsfiddle.net/nh0627/mj41w2ac/39/

1, I have a special reason to use.

https://jsfiddle.net/pg4zn0qt/1/
like this?

@liaoyinglong
Thank you for your answer!
So now, in Element UI, an event($event) of "el-input" is "$event.target.value" in a normal vue app?
Also, why do I have to check a type of input value? Is there any other case than "string" as input value?

@nh0627
no, just string
see the source code
https://github.com/ElemeFE/element/blob/dev/packages/input/src/input.vue#L352
https://github.com/ElemeFE/element/blob/dev/packages/input/src/input.vue#L389
and the HTMLInputElement value is also string
see  https://github.com/microsoft/TypeScript/blob/master/lib/lib.dom.d.ts#L7011

@liaoyinglong Thanks 🙌

@input event arg is a string. it differs from native HTML Input Event.
v-model="value" is equivalent to :value="value" @input="handleInput"

Element UI version
2.8.2
OS/Browsers version
Chrome
Vue version
2.6.10
`

`
Unable to enter


WIP: kn-channel does conflict with the service from IMC

Signed-off-by: Matthias Wessendorf<EMAIL_ADDRESS>Fixes #

Proposed Changes

Release Note

Docs

/close


add go 1.3.3 formula

Need this to build some software that runs on go 1.3.
/cc @mikemcquaid

Thanks for your contribution to Homebrew!  You rock!
For future reference the preferred commit message format for simple version updates is go13: update 1.3.3 .


Semantic-release should also create GitHub Releases besides the pub.dev package release

Yes, I have. It also needs to configure GITHUB_TOKEN and CREDENTIAL_JSON.


Add initial public pointer API.

The various backends are still using mouse events, with a compatiblity shim bridging the gap.

I wasn't able to completely avoid touching the backends, because the name of the wheel method didn't change. Also, I punted on the timestamp.

Are you looking for anybody to give this a review, or somebody in particular? Happy to give it a look, but not sure how code review has worked on glazier for larger PRs historically.

Happy to give it a look, but not sure how code review has worked

Anybody is welcome to review PRs!
Regarding the size, if you only have time for a partial review or you're not confident in your thoroughness, you can just state that with your review and we can combo several reviews together.

This is really exciting work.

Yea, meant to mention that, but super excited about this too!

Thanks for the reviews! I agree that it isn't clear if Angle is worth it, but we can revisit that later. I'm hoping that the addition of methods like Angle::cos will make it relatively low-boilerplate.


Add official support of Couchdb  for Ubuntu 18.04

Hello,
Please add this task in your 2.2.0 milestone.
https://github.com/apache/couchdb/issues/1314 (Duplicate of this but it's closed and task is still open)

@codeyash The convenience binaries for CouchDB are only semi-official.
Whether that ticket is open or not, the work is ongoing.

Any expected date please?

Hello,
I cannot install the couchdb 2.1.1 in Ubuntu 18.04. I use this url:
https://apache.bintray.com/couchdb-deb/dists/bionic/main/
But it returns an error. Any suggestions?  Thank you.

Please see https://github.com/apache/couchdb/issues/1314


Final testing of move generation

There exist some bug inside the move generation function, which is known from the beginning of the project, the logs are inside the .devnotes file. It probably does not affect the overall performance significantly, but it should be investigated as soon as possible.

Add exec that follows game path and checks generated moves on each step

The found bug is preventing some el passant moves from being generated. Not yet resolved


Cydia won't launch iPhone 6+

Hey, I tried with the latest master from yesterday and the latest master from an hour ago. The jailbreak succeeds fine every time, cydia just crashes on launch. I am able to ssh into the iPhone so I think there is just something wrong with cydia on 6+. I tried copying and running "tar" from the source code again (to reinstall cydia) but this did not help.
Can you point me to the direction of the logs for this? I can attach them here, I'm just unsure where to look...

Stop deleting issues that can possibly pertain to coding issues thank you

@skimaskngun if kpwn is deleting them, then they are not code issues

How is this unrelated? I see people having success on iPhone 6's but I haven't heard anything from 6+. So I am sharing what is happening. I don't think it is unrelated and I'm willing to try and help. The problem would need to be solved before a public release anyway?

@GlennMurphy the cydia issue is not related to the issue with the jailbreak. there have been many of these. they were all closed due to being unrelated to yalu.

I'm on 6+ here too, my cydia doesnt install at all and i can't SSH

@FrodoToto #162 and #64

@beanjo55 Why? Still trying to help, to get the jb stable.

@FrodoToto if you looked at the issues that i linked, it would explain that your issue is not related to the code of the jailbreak, rather its related to something else.

I believe this is a Cydia issue I've encountered myself. Try backing up the /var/mobile/Library/Cydia/metadata.cb0 file, then removing the original, then running Cydia.
If it solved it, please give me the backup.

@FrodoToto just wait for kpwn to weigh in. If he closes this then he doesn't want to deal with it. I just want to make sure he's aware there is an issue on 6+ as he is using a 6.

Hi
me too I have this Problem
u can install Filza on ur phone with Cydia impacter
Always need jailbreak working from yalu102
open filza /var/mobile/library/preferences/
and delete com.suarik.cydia.plist

This removes that error? Thank you was gonna restore wheeew

metadata.zip @kpwn You're a god man. That solved it! I've attached the old metadata file. Seems live garbage to me but hopefully it helps.

@masbog no i dont have cydia at all

How did you do it ? @GlennMurphy


git not found error when building k6 via xk6 docker image and including sql extension

Get the following error when building using the xk6 docker image:
go: errors parsing go.mod:
go.mod:5: git init --bare in /go/pkg/mod/cache/vcs/7f51e6d9fcff52165040e226b11ec63a94d49f901bb4063780ffab7eaff6a1ce: exec: "git": executable file not found in $PATH

using the command
docker run --rm -it -e GOOS=darwin -u "$(id -u):$(id -g)" -v "${PWD}:/xk6" \
  grafana/xk6 build v0.51.0 \
  --with<EMAIL_ADDRESS>
able to successfully build k6 without the sql extension using
docker run --rm -it -e GOOS=darwin -u "$(id -u):$(id -g)" -v "${PWD}:/xk6" \
  grafana/xk6 build v0.51.0

Docker version 26.0.0, build 2ae903e
MacOS Sonoma Version 14.4.1 (23E224) - M1 chip

@dennislefever  The problem seems to be produced because the extension github.com/szkiba/xk6-sql does not exist. The following command works:
docker run --rm -it -e GOOS=darwin -u "$(id -u):$(id -g)" -v "${PWD}:/xk6" \
  grafana/xk6 build v0.51.0 \
  --with<EMAIL_ADDRESS>


level cherry pick solution does not match problem.

answer should be cherry-pick C3 C5 C7, solution shows C3 C4 C7

Hi :)
I found a bug in the learnGitBranching project. Here are the details:
Description:
In lesson 1 level 3(main)  program except answer C4 in final task, I expected answer C5. The program did not skip to the next lesson
Expected Behavior:
Answer C5 is correct
Actual Behavior:
Answer C4 is correct
Screenshots/Logs:

Thanks for looking into this!
Peter

The solution is indeed correct, and this is false. Let me explain. So currently, the HEAD is checked out at c1 with main, so anything cherry-picked will continue the branch, as you can see. If you use git cherry-pick c4 it will add just c4 to the branch. So in this case, when you check the goal image with the Show Goal button, or the goal command, on the far right you can see the main branch is continued, and what branches to commit. You see at the top, c3, which means that is the first value you enter. Then you see c4, and then c7. This means the solution to this issue would be git cherry-pick c3 c4 c7 which is what the solution says.
In short, this issue should be marked as resolved, and the solution is correct as far as I can observe. The solution, according to the code itself, is in the following: "solutionCommand": "git cherry-pick C3 C4 C7", which is correct.

Thanks @Conmandouglas for taking a look!


MinetestForFun Logo

Great news, thanks to the work of Obani, we are going to have our own Logo !

But it's still Work in Progress, we need a few more Logos because after, we want to organize a survey to let our players choose our Logo ! :D

So, we're waiting your last logos Obani, keep up the good work !

Ooooops... Forgot it :p . I try to work on it again :)

J'aime bien les formes de tes logos mais tu devrais essayé d'en faire un avec des couleurs moins foncées juste pour voir ce que ca donne :p

Yes @Ataron, more lighter colors can be a good idea ! :)

I made a new concept for the logo... What do you think of it?

The problem is I'm really fond of black and white mix :p

My choice is the second logo (red) !

@ObaniGemini
The new concept is interrresting and beautiful, but we really need colors (website color chart for exemple)
@paly2
Thank you for your answer, but the survey isn't on this topic, we will create a big/good survey on the website (section event/événements) when we have more choice for logos !
(so, the players decided, not only the devs ;))

Even more concepts !!!! And with colors !!!

Yeah, very good Obani !

Right This one is beautiful.

@ObaniGemini
Can you add the text MinetestForFun to your logos ?
I will create a survey as soon as i have your different logos with our name ! :)
The best is to palce the "MinetestForFun" text to the right of the logo, like this :
https://images.duckduckgo.com/iu/?u=http%3A%2F%2Fwww.minimall.fr%2F4264-thickbox%2Ftatouage-rond-dcer.jpg&f=1 MinetestForFun

I want a try too. please wait unitl tomorrow.

Ok @Cyberpangolin, I will wait your logo(s ?) :)

Here is a try:

WTFPL

Interresting concepts, my favorite is the second you show us @Cyberpangolin, the bucket of the third try disturbs me, the first has just too many details compacted for be pleasant to read.
But i think i need say you 2 or 3 things :

The logo needs to respect the chart design of the site (colors),
your logo is very interresting but i have trouble to read correctly the "MINETESTFORFUN" text,

Your logo is interresting because it looks really attractive and we immediately understand that is a game logo, it's a good point !

I hope you like them more.
The dark background is the color already used on the site.
I assume that the lightened background is slightly on yellow-green, because full white kills my eyes. It can be changed anyway, and I will when I am back home with my true color screen.
Maybe you can see a difference between the reds, the goal is to have a good contrast without pain in the eyes. It stays in the colors we use.
Uh. maybe we should keep a dark banner for the website and a lightened for the ingame menus (like in the news).

Please get a try on use the red color of the website for the text and, the green "title article color" for the background of the banner, this for respect the chart design of the website
For the moment, we prefer (ayame and me) the third and the fourth you made i think it's because of the writing design of the text.
But i don't see any other players leave their comments about your logo... :(
Oh and a last thing, maybe it's too much to ask but, the vine you place around the background, it's a good idea, but it's maybe too much repetitive, maybe add some trees ? or flowers ? or aynthing other that is different :D

Okay, here is a new try.
You already know 3 and 4
5 is with the website's background green
6 is 5 with the website's text color.

I had already tried with these red and green, I didn't think they are that good and did not present them. To be - really - honest, I think 5 is ugly and 6 awful.
About the red: the only place you see it is on the logo that will replace it. --> Not that important I think.
About the wines: I may change, but with the green you can almost not see them... And removing is not a solution, it looks empty.

After discussion on IRC with Ombriride;

Added a floating island around the logo.
slightly moved the axe on the shield
changed the wines with saplings, flowers, etc.

My point of view : the island isn't very beautiful
Maybe it can help you ?

And I hope this can please you :)

Great ! On the website right now :)
But i think the mini-lgoo needs to be more bigger (try 300*60 px)

300x60 is not possible. I've done a 300x50.

Ok, i think it's fine

The logos are integrated to the wesbite/forum.
I think i can closed the issue, thank you for your time & work, you can re-opened the issue if you need it.

 O.o

Just read the entire issue and you will understand :D (also visit www.minetestforfun.com)

Question réthorique

Obani, comme je te le disais, si tu souhaites l'améliorer ou tester des choses et nous les montrer, tu as à ta disposition le .svp, libre a toi de nous monrter ton travail :)

Je ne pourrais pas avant plusieurs jours

Obani, qu'est il advenu de ton idée d'améliorer le logo actuel ?

On ne peut pas faire une belle image à partir d'une image ratée. On ne peut qu'affiner une image belle à l'origine.

@ObaniGemini Il te faut donc repartir de zéro si l'on en suit ta loguique, tu te plaint beaucoup de nombreuses textures, mais je ne te vois pas en proposer... :)

@Ombridride : Juste pour te rappeler, @ObaniGemini a proposé pas mal de textures dans ce ticket, et je ne suis pas sûr qu'il ait accès en ce moment ou dans un futur très proche à tout ses outils graphiques.
Sois just patient.

À vrai dire j'ai ouvert mon ordi pour la première fois depuis 2 semaines pour des raisons que vous ignorez surement pour cause d'oubli de vous en avertir. La première et dernière chose que j'y ait faite fut de lire les notifications sur Github et de poster des commentaires.
Donc non, je n'ai pas eu accès à mes outils.
De plus, je suis en train de tenter une approche du monde extérieur plus approfondie et d'ainsi m'éloigner du monde vidéo-ludique (cela me permettrait par exemple de ne pas briller au soleil).
De ce fait, je continuerais à travailler sur de la musique et de l'image, mais en étant beaucoup moins présent qu'à l'usuelle, quitte à m'ennuyer. Cela me permettra aussi de reprendre réellement la musique et d'avoir des rapports sociaux avec des gens dont je peux voir le facies en Retina. Si j'échoue dans ma tache, je serais alors surement plus présent et apte à me connecter durablement sur IRC et in-game ^^.
Mais je pense tout de même qu'un retour au vrai monde réel de la réalité véritablement véridique me permettrait de m'améliorer dans les matières sociales et artistiques, étant donné que la programmation a du mal à m'attirer et que à part améliorer mes squillzes sur des jeux vidéos ou parler avec d'éventuels kikoos, je ne fais pas grand chose sur mon petit pécé.
Voila voila, sur ce, bisous virtuels virtuoses !

Up, i will closed this issue in one week, if you don't send us a logo soon :)

Actually the first one of this page was great. The rest was just to have fun.

We need to see "MinetestForFun" on the logo, with a pixel policy if it's possible, that's a must have.
Second, we need two banners, one small (left top of the website, news/irc/... ingame background) and a big one (first banner of the website and forum presentation) the size need to be rectangular, but easy to tweak if you add "MinetestForFun" i thing...

The one I made was in circle, so easy to put in a square or circle form like in the website experiment of davedevils. For the MInetestForFun font, I remember you that I'm not on my usual computer and that I will have more difficulties to do things I would do on my normal computer. Also, I don't understand why we should take a font and why it should be pixelized. Most part of the fonts are boring or bad for logos.

Closed this issue.
No answer since 1 month.


Need to update renewIssueEtc when renewing cookie

cc @Tratcher

Hi @HaoK, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Fixes https://github.com/aspnet/Security/issues/628
I'm not sure if we need to handle sliding expiry as well here...

I think you need to handle sliding refresh.
https://github.com/aspnet/Security/blob/haok/1-6-issue/src/Microsoft.AspNet.Authentication.Cookies/CookieAuthenticationHandler.cs#L85-L86
If they access the site every 30 seconds and issuedUtc is updated each time, then timeElapsed will always be 30s and timeRemaining will keep shrinking. The sliding expiration won't be renewed until you're within 30s, and if you miss a request in that window then your cookie suddenly expires.

Yeah I thought as much, this cookie refresh/expires code is pretty complicated as its written :/

I think I will try to refactor the cookie refresh/renew logic into some sane methods so this is actually readable and sane...

Updated:

Moved logic for refresh into two helpers
We request refresh if explicitly asked for via context.ShouldRenew, or sliding check
If we renew, we ALWAYS set expires/issues UTC (is that correct/safe?)

:shipit: and consider updating the naming.

5bcc1bfc260978b0bea9b981056dd0beb31df400


Atom hangs after startup

When starting Atom on Windows7 - 65 bit without having a windows I can see the icon in my task bar. Unfortunately I cannot open the Atom-Window. INstead of this I see the waiting cursor from Windows and nothing happens.
Preview-Windows from window when hovering over task-icon shows me the preview of the Atom-window.
We are using a proxy, but I haven't configured it right now. Maybe Atom waits for any kind of timeout?
What can I do? I am using version <IP_ADDRESS>.

If you mean version 0.223.0, you're using an extremely old version of Atom - please update to the latest version (1.0.2). Feel free to reopen this if you still experience this after successfully updating to 1.0.2.

I used the latest version 1.0.2, but when taking a look into the pocesses with the Process-Explorer is seems to show a deprecated version of atom ( the one I noted above <IP_ADDRESS> ).
I checked the sources but I am not sure if this version is set only for the installer or the installer is using a version string rom the atom sources.

I am not allowed to reopen the issue, but it is still there with the latest version. Let me know if I can help you investigating this issue.

For some reason your process explorer seems to be detecting the Atom Shell version instead of the Atom version.

Hil kimkulling are you still experiencing this problem?

No response, closing issue. Please re-open if issue still exists.


Update related to bypass CloudFlare and change README

I added a User-Agent in Axios headers, it works.
Hope it will do the same for you, however I haven't any idea how many requests you can do before getting blocked.

Thanks! I'll add my own changes as well to support commonjs and not force others to use +ES5 syntax so you can easily run your scripts without ESM package!

published to npmjs as well
so now you can just do npm i steamdb-js again and it works!


No multi-part upload

The comments for uploading mention utilization of the multi-part upload capability, but no usage of the defined AWS SDK API (https://docs.aws.amazon.com/AmazonS3/latest/dev/HLuploadFileDotNet.html?shortFooter=true) is seen in the code that I can find.

Then you didn't look that hard 😄


Add 404 page

https://www.11ty.io/docs/quicktips/not-found/
Make sure permalink is set to 404.html
---
permalink: "404.html"
---

Maybe grab landing image from undraw
Maybe add search function with original url cleaned up

Can't seem to get netlify cli to run local 404 redirects
Local redirects seem possible by adding browsersync config, but probably not enough value to dev environment to add just of testing 404s


Confluence

Describe the task
Team has confluence access to store notes
Acceptance Criteria

[ ] All team has access to Confluence

Request to NRIDs is in.


Laravel 10.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 9.x.
Before merging, you need to:

Checkout the l10-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-sign-in-with-apple.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "genealabs/laravel-sign-in-with-apple": "dev-l10-compatibility",
    }
}

Finally, run: composer update

@mikebronner would you mind merging this is it's all correct? We wish to upgrade our project with this great package to Laravel 10


nginx: [emerg] http_auth_ldap: parse_require in /etc/nginx/conf.d/ldap.conf

I just updated both nginx (to 1.9.2) and nginx-auth-ldap and I got the following messages when I run nginx -t.
I'm not sure if it's just a message or if it's an error. I think I didn't have this in the past.
bbigras@ubuntu:~/nginx$ sudo nginx -t
nginx: [emerg] http_auth_ldap: parse_require in /etc/nginx/conf.d/ldap.conf:5
nginx: [emerg] http_auth_ldap: Setting group in /etc/nginx/conf.d/ldap.conf:5
nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
nginx: configuration file /etc/nginx/nginx.conf test is successful

ldap.conf
ldap_server server1 {
        url ldaps://host:636/ou=something,o=NET?uid?sub?(objectClass=person);
        group_attribute member;
        group_attribute_is_dn on;
        require group "cn=test,ou=something,o=NET";
        binddn "cn=test,ou=something,o=NET";
        binddn_passwd "pass";
}

I have the same issue. It is introduced in #85.
The ngx_conf_log_error is used to debug, instead of ngx_log_debug.

just ran into this myself trying to use nginx-auth-ldap for the first time

I have the same issue. Please help!...

same here.

Can this be reverted or fixed ? #85 is clearly wrong, regular users will think their config is wrong..

See #95

See #95

#95 was merged a day ago. Now the messages are labeled as [notice] instead of [emerg].
@kvspb
You close the dupicate issues #98 and #103

I compiled nginx-1.13.5 with the @kvspb patch.  When I use the following config file, I get the parse error.
http {
    include mime.types;
    default_type application/octet-stream;

    client_body_timeout 12;
    client_header_timeout 12;
    keepalive_timeout 15;
    send_timeout 10;

    gzip on;

    auth_ldap_cache_enabled on;
    auth_ldap_cache_expiration_time 10000;
    auth_ldap_cache_size 1000;

    ldap_server adds {
         url "ldaps://<IP_ADDRESS>/dc=example,dc=com?sAMAccountName?sub?";
         binddn "cn=admin,dc=example,dc=com";
         binddn_passwd "password";
         require valid_user;
         satisfy any;
    }

    server {
        listen       80;
        server_name  localhost;

        auth_ldap "Forbidden";
        auth_ldap_servers adds;

        location / {
            root   html;
            index  index.html index.htm;
        }
    }
}

It passes the configtest:
ubuntu@ip-10-0-0-171:~$ sudo nginx -t
nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
nginx: configuration file /etc/nginx/nginx.conf test is successful

Looks like you have not provided proxy_pass directives?

The location should probably be like the one below to get rid of the error:
location / {
proxy_pass http://localhost
proxy_set_header  Host $host
auth_ldap_servers   test1;
}


Drop logrus and replace it with logr

Dedup loggers and unify into logr
Signed-off-by: Dinar Valeev<EMAIL_ADDRESS>

👍  awsome!


Fixing server group styling

New Server Group Styling:

Adds server group color to connections
Adds back guide which was removed in #7119 (@Charles-Gagnon I did not understand the issue due to which it was removed #7117)
This is based off on new design suggested by UX.
Mockup:

Design:

TBH, I'm not really a fan of the beveled edges. What exactly is this "fixing"? It seemed fine to me beforehand - was there specific feedback you're addressing here?

is this recommended by the designer?

@Charles-Gagnon , do you know why tree guides were removed in the first place.

@Charles-Gagnon ,

Yes, I was able to change the behavior of indent guides using the settings.
Yes, it re-renders the child connections as well and changes the color for them.
Yes, that's just the screenshot cutting it off.

To confirm - we  are implementing to get some feedback from users, correct?  The contrast is better for highlighting, but it feels very busy.

@Charles-Gagnon and @erinstellato-ms
Adding gif for the indent guides


minor fixes in logic adapters & terminal example

Just a couple of minor changes, including:

Add additional logic adapters to the terminal example adding functionality
Possible bug fix in evaluate_mathematically (not really sure, but this makes sure no bug will ever arise)
Sometimes, the time logic adapter would find a statement similar to "what is 100 * 10?" as something it should process. By adding the additional piece of training data, the time adapter does not incorrectly process math statements as often

:+1: Changes look good.
Also, I'm currently looking into #111. Should have some sort of feedback shortly.

Awesome! Thank you!
Also, on a side note, GCI ended today, so I hope to be able to spend some more time working on chatterbot now. Thank you!


Installer Validation Failed

The installer installs Cesium just fine, and it still works, but when it tries to run Cesium after the installation is finished, it throws an "Error: Validation" error.
I can still run Cesium from the apps menu and everything else works perfectly, but the installer is incapable of running Cesium (and yes, even I can tell this is nitpicking a little bit :P).
Also, is there any way you can have the installer delete Cesium before installing instead of just telling me it can't install the new version? Thanks.

Well that's odd. You may have to clear out the archive, sounds like something made it get corrupted.

It was my OS.
Upgrading to OS 5.4 fixed it.


TensorBoard.dev is shutting down - find an alternative?

By December 31st 2023, TensorBoard.dev will shutdown.
What alternatives are there?
See:

--
Some alternative options:

TensorBoard live in Google Colab - https://colab.research.google.com/github/tensorflow/tensorboard/blob/master/docs/tensorboard_in_notebooks.ipynb
Weights and Biases - https://docs.wandb.ai/guides/integrations/tensorboard

Also referenced in #610

Also referenced in #610

Hi dbourke
Please updata your tensorflow deeplearning section4 code on github (due to TensorBoard has been shut down)

Hi @MohammadH8086, did you find an alternative solution?


The result of a patch metadata is a union rather than replacement.

It looks like patch operations do not work well with Map attributes (such as metadata) and instead of replacing the map content (which is backed by repeated message) it merges it.

@aozarov can you add in the description the code snippet that reproduces this for us ?

Here is the bug for gcloud-node: https://github.com/GoogleCloudPlatform/gcloud-node/issues/597
We haven't decided whether this is just something we document, or whether we find a way to make setMetadata delete the entries and re-add them, or something entirely different....

The issue is in the Apiary level and there is a plan to fix that (so once fix current code will work as expected).

Here is a code snippet to reproduce the problem:
StorageServiceOptions options = StorageServiceOptions.defaultInstance();
StorageService storage = StorageServiceFactory.instance().get(options);
Blob blob = storage.get("bucket", "blobName");
Map<String, String> metadata = new HashMap<>();
metadata.put("name", "value");
storage.update(blob.toBuilder().metadata(metadata).build());

I double checked and this "problem" persists.
gcloud-node guys ended up documenting that the setMetadata method actually merges metadata.

Merge the given metadata with the current remote file's metadata. This will set metadata if it was previously unset or update previously set metadata. To unset previously set metadata, set its value to null.

We could do something similar for update. Notice that in our case setting metadata to null does not unset remote metadata.

@aozarov @jgeewax Sorry for pinging, you have any thoughts on this?

Might want to take a look at this discussion we had over in gcloud-node: https://github.com/GoogleCloudPlatform/gcloud-node/issues/597

I actually checked, this is why I was proposing to document this behavior as it has been done for gcloud-node.

The internal bug for apiary is still open. There are various ways that we can deal with it but I would rather avoid implementing the patch logic ourselves (via get, merge and set if not modified).
I am not sure that in Java we have an option to null/clear the metadata property and still use the  patch operation.
Some options:

Replace patch with update and document that update operation will replace all the blob's metadata with values from the given BlobInfo.

Provide to the user both patch and update options (documenting the differences) and
also mentioning that if a new metadata is given its values will be merged with the previous version
and if that is not desired update should be used instead.

Option (2) is actually an add-on to option (1).

BTW, @BrandonY suggests that setting the metadata to null does work for him (and clears it).

@mziccard I think that the reason you were not able to set metadata to null was because we need to change the BlobInfo.Builder.metadata function to use Data.nullOf when user explicitly sets it to null (otherwise for PATCH null means keep-previous-value).
If that is indeed the case than from the API perspective I would suggest a different approach from the one we took in #276. Basically I would suggest that we only change the documentation to mention that an update of metadata will be merged with any previously set metadata and setting it to null will clear it. That means for replacing metadata with new content the workaround would be an update with null following another update with the new map.
If we decide to go with this route, than we will need to create our own "ImmutableEmptyMap" class (public and public empty constructor) to be used with Data.nullOf.

Issue was fixed with #279.
Users now have the option of removing individual metadata properties (by setting their value to null) or removing all metadata properties (by setting metadata to null) and possibly resetting it with a new map.
Also, #279 add some details and a code sample to the javadoc.


Hourly vs. Minute-to-Minute Price Updates for sensor.zonneplan_current_electricity_tariff?

Hey, thanks for this addon. Just a quick question on which I could not find the answer online. When I review sensor.zonneplan_current_electricity_tariff I observe different prices every few minutes, but in the app, I see different prices per hour.
I expected the prices to change in home-assistant-zonneplan-one on an hourly basis as well.
Is home-assistant-zonneplan-one more specific with these minute-to-minute prices, or am I looking at some sort of misconfiguration.
HomeAssistant with Zonneplan-one:

Zonneplan app:

Perhaps it was because I was running 0.0.17". Updated just now, fingers crossed.

Just checked my own stats and I only see 1 update each hour.
Did you hover the graph and check the times the of each change.

Thanks @fsaris I see I was zoomed out a lot on that first screenshot. Closing


Support Ubuntu 18.04

Required Info

Camera Model
D415

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 18.04

Kernel Version (Linux Only)
4.15.0-20-generic

Platform
PC

SDK Version
2.11.0

Issue Description
Hello, the issue i am having that i am unable to update the kernel drivers to 4.15. Running the script gives me an error that kernel version 4.15 isn't supported. Since this is a LTS release it would be worth adding this kernel version in my opinion. It's also not possible for me to update the firmware because the debian repositories are down. Thanks!

Hello @feliwir ,
We are working to add kernel 4.15 support for Ubuntu 18 and it is scheduled for the next version release.
We will also look into updating the debian repositories due to transition from xenial to bionic.
In the meanwhile you're welcomed to use the kernel patch for Ubuntu 18 from WIP branch
https://github.com/ev-mp/librealsense/tree/kernel_4.15

[Realsense Customer Engineering Team Comment]
Hi @feliwir,
Is the issue resolved? You can also refer to below for Ubuntu 18 installation instruction.
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation.md

@RealSense-Customer-Engineering
Hello, according to the installation instructions this still doesn’t seem possible with Kernel version 4.15, which is the default to Ubuntu 18.04

Hello @feliwir ,
Ubuntu 18 with kernel 4.15 is supported. It was not listed in one of the preparation steps (updated now), so it could confuse.
Please follow the guide to build kernel patches and update if you have issues.

Hey @ev-mp ,
when running the patch script i need to click through a lot of choices with enter myself. After selecting all the options i get the following:
Replacingvideodev : 	Module videodev  is used by uvcvideo 	Unloading dependency uvcvideo 	Module is resident, unloading ...  succeeded.  Applying the patched module ... modprobe: ERROR: could not insert 'videodev': Exec format error Failed to insert the patched module. Operation is aborted, the original module is restored Verify that the current kernel version is aligned to the patched module version The original  videodev  module was reloaded

Please post the kernel log (dmesg) and the script log for review

@ev-mp i guess we meant the output of the patch script. The issue is that i can't capture that with the > operator because i always need to press enter manually. However here is the dmesg log:
dmesg.txt

@feliwir , thanks for the log.
It seem as if the kernel you're running was built without "Meltdown"-required patches. Seem a little strange as it was just one of the latest features both introduced and retrofitted into all the latest LTS kernels.
Are you using Ubuntu 18.04 or Ubuntu 18.10 distribution?
Additionally, please check and provide host build and toolchain info:
gcc -v
cat /sys/devices/system/cpu/vulnerabilities/spectre_v2
'zgrep CONFIG_RETPOLINE /boot/config* /proc/config.gz'

Hey @ev-mp ,
I am using Ubuntu 18.04. I downloaded the .iso from the official website after it was officially released (so no beta version whatsoever)
gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/8/lto-wrapper
OFFLOAD_TARGET_NAMES=nvptx-none
OFFLOAD_TARGET_DEFAULT=1
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 8-20180414-1ubuntu2' --with-bugurl=file:///usr/share/doc/gcc-8/README.Bugs --enable-languages=c,ada,c++,go,brig,d,fortran,objc,obj-c++ --prefix=/usr --with-gcc-major-version-only --with-as=/usr/bin/x86_64-linux-gnu-as --with-ld=/usr/bin/x86_64-linux-gnu-ld --program-suffix=-8 --program-prefix=x86_64-linux-gnu- --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-vtable-verify --enable-libmpx --enable-plugin --enable-default-pie --with-system-zlib --with-target-system-zlib --enable-objc-gc=auto --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-offload-targets=nvptx-none --without-cuda-driver --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 8.0.1 20180414 (experimental) [trunk revision 259383] (Ubuntu 8-20180414-1ubuntu2)

cat /sys/devices/system/cpu/vulnerabilities/spectre_v2
Mitigation: Full AMD retpoline, IBPB

grep CONFIG_RETPOLINE /boot/config* /proc/config.gz
/boot/config-4.15.0-20-generic:CONFIG_RETPOLINE=y
/boot/config-4.15.0-22-generic:CONFIG_RETPOLINE=y
gzip: /proc/config.gz: No such file or directory

I see recognize some differences:
1.The GCC versions differ

gcc version 8.0.1 20180414 (experimental) [trunk revision 259383] (Ubuntu 8-20180414-1ubuntu2)

while I have gcc version 7.3.0 (Ubuntu 7.3.0-16ubuntu3).
2 Additionally

Mitigation: Full AMD retpoline, IBPB

vs (local version)
Mitigation: Full generic retpoline
We will look into this to understand what are the requirements/constrain  to use gcc8-based tool-chain.
In the meantime, can you reconfigure the toolchain rebuild the kernel patches using gcc 7  ? (this link might be helpful)
Also, can you send a link to iso that you're using ?

Hey @ev-mp,
i downloaded the 64-bit .iso from here: http://releases.ubuntu.com/bionic/ (desktop of course). I installed gcc 8 via apt install, but not sure if i can switch the gcc version since this computer is used for development in work.

I can't recall where, but I remember finding instructions online for how to
install multiple versions of gcc side by side. Could that be a solution for
you?

Also tried with gcc 7.3 which made no difference in the result

Per the instructions contained in https://github.com/IntelRealSense/librealsense/blob/development/doc/installation.md
It says to run the script: ./scripts/patch-realsense-ubuntu-lts.sh
Starting on line 34 of the script, we have:

ubuntu_codename=. /etc/os-release; echo ${UBUNTU_CODENAME/*, /}
if [ -z "$UBUNTU_CODENAME" ];
then
# Trusty Tahr shall use xenial code base
ubuntu_codename="xenial"
retpoline_retrofit=1
fi
kernel_name="ubuntu-${ubuntu_codename}-$kernel_branch"
#Distribution-specific packages
if [ ${ubuntu_codename} == "bionic" ];
then
require_package libelf-dev
require_package elfutils
fi

The line first line correctly evaluates to bionic. The second line however evaluates to true as long as $UBUNTU_CODENAME isn't emtpy, and then it resets $ubuntu_codename to xenial. Then down on line 54, it pulls the xenial kernel rather than the bionic kernel from git://kernel.ubuntu.com.
For me, after commenting out that if block (lines 35 to 40),  the patch runs successfully. I have not yet finished the installation process to see if everything else works.

Hello @jecyrd , it is not directly related to the original issue, but

The second line however evaluates to true as long as $UBUNTU_CODENAME isn't emtpy...

According to multiple resources (e.g.  1, 2 ) the operator if [ -z "$var" ]  evaluates to true when the variable is unset or empty.
Can you share the execution log for lines 34-40 when you run unmodified script ? (enable log by replacing set -e with set -ex)

Sorry. Suppose I didn't read the thread close enough. Though I was having the same modprobe error as feliwir and was avoiding starting a duplicate.

According to multiple resources (e.g. 1, 2 ) the operator if [ -z "$var" ] evaluates to true when the variable is unset or empty

You are correct, I got it turned around. The behavior is as I described, just for the wrong reasons (see script output below). I think the issue is that the condition needs to [ -z "$ubuntu_codename"] rather than [ -z "$UBUNTU_CODENAME"]. Which also worked for me, as an alternative to commenting out the if block.
Output of unmodified script:
$ ./scripts/patch-realsense-ubuntu-lts.sh
+ exec
+ exec
+ '[' 1 -ne 0 ']'
+ echo -e '\e[32m'

+ read -p 'Remove all RealSense cameras attached. Hit any key when ready'
Remove all RealSense cameras attached. Hit any key when ready
+ echo -e '\e[0m'

+ source ./scripts/patch-utils.sh
+ sudo apt-get install linux-headers-generic build-essential git
Reading package lists... Done
Building dependency tree
Reading state information... Done
build-essential is already the newest version (12.4ubuntu1).
git is already the newest version (1:2.17.0-1ubuntu1).
linux-headers-generic is already the newest version (<IP_ADDRESS>.23).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
+ require_package libusb-1.0-0-dev
+ package_name=libusb-1.0-0-dev
+ printf '\e[32mPackage required %s: \e[0m' libusb-1.0-0-dev
Package required libusb-1.0-0-dev: + exec
+ exec
+ set -e
+ '[' 1 -eq 0 ']'
+ echo -e '\e[32m - found\e[0m'
 - found
+ require_package libssl-dev
+ package_name=libssl-dev
+ printf '\e[32mPackage required %s: \e[0m' libssl-dev
Package required libssl-dev: + exec
+ exec
+ set -e
+ '[' 1 -eq 0 ']'
+ echo -e '\e[32m - found\e[0m'
 - found
+ retpoline_retrofit=0
++ uname -r
+ LINUX_BRANCH=4.15.0-22-generic
++ choose_kernel_branch 4.15.0-22-generic
++ IFS=.
++ read -a kernel_version
++ case "${kernel_version[1]}" in
++ echo master
+ kernel_branch=master
++ . /etc/os-release
+++ NAME=Ubuntu
+++ VERSION='18.04 LTS (Bionic Beaver)'
+++ ID=ubuntu
+++ ID_LIKE=debian
+++ PRETTY_NAME='Ubuntu 18.04 LTS'
+++ VERSION_ID=18.04
+++ HOME_URL=https://www.ubuntu.com/
+++ SUPPORT_URL=https://help.ubuntu.com/
+++ BUG_REPORT_URL=https://bugs.launchpad.net/ubuntu/
+++ PRIVACY_POLICY_URL=https://www.ubuntu.com/legal/terms-and-policies/privacy-policy
+++ VERSION_CODENAME=bionic
+++ UBUNTU_CODENAME=bionic
++ echo bionic
+ ubuntu_codename=bionic
+ '[' -z '' ']'
+ ubuntu_codename=xenial
+ retpoline_retrofit=1
+ kernel_name=ubuntu-xenial-master
+ '[' xenial == bionic ']'
+ '[' '!' -d ubuntu-xenial-master ']'
+ git clone -b master git://kernel.ubuntu.com/ubuntu/ubuntu-xenial.git --depth 1 ./ubuntu-xenial-master
Cloning into './ubuntu-xenial-master'...

Hello @jecyrd ,
thanks for the log - indeed the condition is not evaluated properly resulting in the version being overwritten with xenial:

'[' -z '' ']'

ubuntu_codename=xenial
retpoline_retrofit=1

I'm away from Ubuntu18, but the variable should probably be encapsulated with curly braces for robustness [ -z "${UBUNTU_CODENAME}"] - can you share the bash version for reproduction ? (echo $BASH_VERSION)

I ran into the same modprobe error on PopOS 18.04.
Replacingvideodev :
	Module videodev  is used by uvcvideo
	Unloading dependency uvcvideo
	Module is resident, unloading ...  succeeded.
Applying the patched module ... modprobe: ERROR: could not insert 'videodev': Exec format error
Failed to insert the patched module. Operation is aborted, the original module is restored
Verify that the current kernel version is aligned to the patched module version
The original  videodev  module was reloaded

For me UBUNTU_CODENAME on line 34 didn't evaluate to anything. I tried just evaluating it by itself  echo ${UBUNTU_CODENAME} on my 18.04 PopOS, as well as inside a ubuntu:18.04 docker container - neither worked.
My solution was simply setting an environment in the shell before executing the script:
export UBUNTU_CODENAME=bionic
./scripts/patch-realsense-ubuntu-lts.sh

Then it ran through smoothly.

The solution from @compiaffe worked for me!

@RealSense-Customer-Engineering
There still is no package for the Ubuntu 18.04 if i am not mistaken. Until this isn’t done i wouldn’t close this issue.

[Realsense Customer Engineering Team Comment]
Hi @feliwir,
You're right. The DKMS package is not there yet. Will further check internally about its availability.

[Realsense Customer Engineering Team Comment]
Please refer to below instructions and DKMS package ready for kernel 4.15.
https://github.com/IntelRealSense/librealsense/blob/development/doc/distribution_linux.md

[Realsense Customer Engineering Team Comment]
Hi @feliwir,
If no other question for this issue, will close this first. If you still can re-open or create another ticket.


Drop-down menu closes when mouse pointer wanders off

In most user interfaces a menu that you click open stays open until you select an option or click outside of the menu.
However in Scale Workshop drop-downs such as the New scale menu close when the mouse pointer moves outside the menu.
It makes the interface feel flimsy to me and makes it hard to select options at the bottom of the menus.

It was already implemented (click the dropdown header), I was just bad.


Spinner can't be added directly from the python code

Description of the Bug
I tried to solve with a pull request, but my approach wasn't good enough. Before trying to pull another one, I prefer to set this as an Issue.
Whenever you try to add the MDSpinner as a class (self.add_widget(MDSpinner()), you get an IndexError back (see log).
Code and Logs
from kivy.app import App
from kivy.lang import Builder
from kivy.uix.boxlayout import BoxLayout
from kivymd.uix.spinner import MDSpinner

kv = """
<Testing>:
        orientation: "vertical"
        size_hint: 0.5, 0.5
        pos_hint: {"center_x": 0.5, "center_y": 0.5}
"""
class Testing(BoxLayout):
    def __init__(self, **kwargs):
        super().__init__(**kwargs)
        s = MDSpinner()
        self.add_widget(s)

class MainApp(App):
    def build(self):
        self.root = Builder.load_string(kv)
        return Testing()

if __name__ == '__main__':
    MainApp().run()

Log
 ...
       9:        Color:
      10:            rgba: self.color
 >>   11:            a: self._alpha
      12:        SmoothLine:
      13:            circle: self.center_x, self.center_y, self.width / 2,            self._angle_start, self._angle_end
 ...
 IndexError: list index out of range

I tried removing the line  >>   11:            a: self._alpha and it worked, so I think this is another solution I propose, but I don't really know if removing that line will cause any problem to the rest of the code.
I'd really like to help solving this issue, so if you need something, here I am.
Versions

OS: not depending on OS
Python: latest
Kivy: latest
KivyMD: latest

See the last commit
https://github.com/HeaTTheatR/KivyMD/commit/49bd42a6a697c831437dae322d7e7b969b5dac09

See the last commit
49bd42a

I'm glad one of the approaches worked. Thanks!


[ENH] Improve conditional join for range indices

improve range_indices significantly, for scenarios where numba is not used
fix output when indicator is used and multiindex is returned

with this PR, numba  might not be required anymore. I think it's good to keep it though.
This PR improves conditional_join.
performance test
url = "https://raw.githubusercontent.com/samukweku/data-wrangling-blog/master/notebooks/Data_files/results.csv"
events = pd.read_csv(url, parse_dates=['start', 'end']).iloc[:, 1:]

# dev
In [25]: %%timeit
    ...: (events
    ...: .conditional_join(
    ...:     events,
    ...:     ('start', 'end', '<='),
    ...:     ('end', 'start', '>='),
    ...:     ('id', 'id', '!='),
    ...:     use_numba = False,
    ...:     df_columns = ['id', 'start', 'end'],
    ...:     right_columns = ['id', 'start', 'end'])
    ...: )
    ...:
    ...:
823 ms ± 2.35 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [26]: %%timeit
    ...: (events
    ...: .conditional_join(
    ...:     events,
    ...:     ('start', 'end', '<='),
    ...:     ('end', 'start', '>='),
    ...:     ('id', 'id', '!='),
    ...:     use_numba = True,
    ...:     df_columns = ['id', 'start', 'end'],
    ...:     right_columns = ['id', 'start', 'end'])
    ...: )
    ...:
    ...:
23.9 ms ± 567 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

# this PR

# massive performance improvement when use_numba=False
In [20]: %%timeit
    ...: (events
    ...: .conditional_join(
    ...:     events,
    ...:     ('start', 'end', '<='),
    ...:     ('end', 'start', '>='),
    ...:     ('id', 'id', '!='),
    ...:     use_numba = False,
    ...:     df_columns = ['id', 'start', 'end'],
    ...:     right_columns = ['id', 'start', 'end'])
    ...: )
    ...:
    ...:
20.1 ms ± 109 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

In [21]: %%timeit
    ...: (events
    ...: .conditional_join(
    ...:     events,
    ...:     ('start', 'end', '<='),
    ...:     ('end', 'start', '>='),
    ...:     ('id', 'id', '!='),
    ...:     use_numba = True,
    ...:     df_columns = ['id', 'start', 'end'],
    ...:     right_columns = ['id', 'start', 'end'])
    ...: )
    ...:
    ...:
24 ms ± 955 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

Test on a larger dataset:
Another test on a 10 million row; data also from DuckDB

employees = pd.read_csv("../employees.csv")

# dev
In [27]: %%timeit
    ...: (employees
    ...: .conditional_join(
    ...:     employees,
    ...:     ('salary', 'salary', '<'),
    ...:     ('tax', 'tax', '>'),
    ...:     use_numba = True)
    ...: )
    ...:
    ...:
4.35 s ± 34.1 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

# this PR
# massive performance improvement,
# as this could not be executed in dev
In [18]: %%timeit
    ...: (employees
    ...: .conditional_join(
    ...:     employees,
    ...:     ('salary', 'salary', '<'),
    ...:     ('tax', 'tax', '>'),
    ...:     use_numba = False)
    ...: )
    ...:
    ...:
1.79 s ± 3.16 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [19]: %%timeit
    ...: (employees
    ...: .conditional_join(
    ...:     employees,
    ...:     ('salary', 'salary', '<'),
    ...:     ('tax', 'tax', '>'),
    ...:     use_numba = True)
    ...: )
    ...:
    ...:
4.34 s ± 62.4 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

PR Checklist

Please ensure that you have done the following:

[ ] PR in from a fork off your branch. Do not PR from <your_username>:dev, but rather from <your_username>:<feature-branch_name>.

[ ] If you're not on the contributors list, add yourself to AUTHORS.md.

[ ] Add a line to CHANGELOG.md under the latest version header (i.e. the one that is "on deck") describing the contribution.

Do use some discretion here; if there are multiple PRs that are related, keep them in a single line.

Automatic checks
There will be automatic checks run on the PR. These include:

Building a preview of the docs on Netlify
Automatically linting the code
Making sure the code is documented
Making sure that all tests are passed
Making sure that code coverage doesn't go down.

Relevant Reviewers

Please tag maintainers to review.

@ericmjl

🚀 Deployed on https://deploy-preview-1256--pyjanitor.netlify.app

@samukweku, I'm going to merge. There are tons of great improvements accumulating here, let's get them shipped out soon with a release party!


any chance to config the model local foler?

so we could save the model download time and use our own custom model bin file. Thanks

You can download the model seperatly to any destination you like and then point the endpoint-server to the directory, i.e. python main.py --pretrained="../my_model"

Thanks, yes seems it's not working, I'll try more ways and let you know the result.

It seems you've done what you want, if there's still any problem, you can reopen it.


SmartContractGasInjector fails with recursive methods.

The SmartContractGasInjector throws System.InvalidOperationException: 'Operation is not valid due to the current state of the object.' when injecting gas into a recursive method.
Simple, albeit contrived example:
    public void ThrowException()
    {
        ThrowException();
    }

We should have a test for a recursive example, like above, in GasInjectorTests.cs, and have a fix so this is resolved.
Current Bounty: 10 STRAT

Yes, on both. I will DM you on Twitter if that's ok.

Sounds good!


remove unnecessary warning about dnszone not getting deleted

Revert 9ad718b.
Only remove the unnecessary warning: Revert the rest of the superfluous changes.

/lgtm


Documentation about SuggestedAction & History

Is there any documentation about what the SuggestedActions & History object can do? And how to use it?

Hey Sammy. I apologize that this question wasn't addressed earlier. Hopefully you've got an answer before now.
To quickly answer:
"Suggested actions enable your bot to present buttons that the user can tap to provide input. Suggested actions appear close to the composer and enhance user experience by enabling the user to answer a question or make a selection with a simple tap of a button, rather than having to type a response with a keyboard. Unlike buttons that appear within rich cards (which remain visible and accessible to the user even after being tapped), buttons that appear within the suggested actions pane will disappear after the user makes a selection. This prevents the user from tapping stale buttons within a conversation and simplifies bot development (since you will not need to account for that scenario)."
You can find a sample here (this is a v4 sample):
https://github.com/Microsoft/BotBuilder-Samples/tree/master/samples/csharp_dotnetcore/08.suggested-actions
Additionally, in the future; these types of "how to" questions are best addressed in Stack Overflow


Support the import from "License Classification Providers"

While ORT itself should stay neutral about how licenses are classified, we should make it easy for the user to choose and use a classification. For this, we should have something like a LicenseClassificationProvider similar to like we have a PackageCurationProvider. The existing license-classifications.yml could then be used via a FileLicenseClassificationProvider (compare FilePackageCurationProvider). Potential other providers could be (in no specific order):

OSADL Open Source License Checklists
FSF License Metadata API
OSI license information (API)
TLDRLegal (Developer API)
Choose a license (appendix table, project page)
ifrOSS

Should be done after https://github.com/oss-review-toolkit/ort/issues/2624.

Note that Gitter.im has been sold to the makers to Matrix meanwhile. But I'll stick with Slack for the foreseeable time.

@tsteenbe  @sschuberth @mnonnenmacher shall we really add the complexity of requesting licenses at runtime?
I believe it suffices to provide means to export the license classifications from the above APIs to a licenses.yml.
Having this file makes it much clearer what actual licenses classifications have been used.
IMO this is adding complexity without need.
Apart from that, re-computing reports locally can be optimized by not recomputing evaluation results if no configuration was changed. Whether the configuration has changed will become harder / impossible to detect if license classifications are obtained at runtime.

shall we really add the complexity of requesting licenses at runtime?

I'd argue it does not really add that much (code) complexity, unless you actually do add a (difficult to handle) remote license provider. This issue is only about making it possible to add such providers, not necessarily already about adding such providers. And just abstracting away the current implementation behind a FileBasedLicenseClassificationProvider is really not that complex, esp. as we have the same concept already elsewhere.

IMO this is adding complexity without need.

In the larger (Open Chain) community we had several asks over the years to add support for TLDR legal, and also @tsteenbe has filed two related issued about this.

Whether the configuration has changed will become harder / impossible to detect if license classifications are obtained at runtime.

Certainly not impossible. On a low level OkHttp already takes care of not re-downloading unmodified resources again, and if really needed we could easily add our own hash-based caching mechanism on top of that.

Certainly not impossible. On a low level OkHttp already takes care of not re-downloading unmodified resources again, and if really needed we could easily add our own hash-based caching mechanism on top of that.

The use case I had in mind was: downloading an OrtResult produced on CI and re-doing the evaluator step locally using the exact same license classifications as was used when the original OrtResult was produced. I don't see how this would be working without adding quite a bit of complexity.

After giving this some thought again, I'd now also prefer to realize this via "importers" that create license-classifications.yml and rules.kts files which can serve as a template / starting point for custom rules, and be used "offline" without being dependent on third-party infrastructure.

I'm deprecating this idea in favor of recommending the use of license classifications generated by LDBcollector as mentioned in our license classifications docs.


Types do not exist

I'll probably be able to make these but it will take a while
Issue: TypeScript hates the module due to no typings

@DevAX1T

Yeah I'm rewriting this module in TypeScript, but I've been quite busy with life and procrastination

Yeah I'm rewriting this module in TypeScript, but I've been quite busy with life and procrastination

Alright gl with that then


Fail when compiling project with custom paths (wrong location for tsconfig.json)

OS?
MacOS Sierra
Versions.
angular-cli: 1.0.0-beta.26
node: 7.4.0
os: darwin x64
angular/*: 2.4.4
Repro steps.
just use ng new to create a project and add paths into tsconfig.json like:
  "baseUrl": ".",
  "paths": {
      "@app-root/*": ["src/app/*"],
      "@app-core/*": ["src/app/core/*"],
      "@app-shared/*": ["src/app/shared/*"],
      "@app-features/*": ["src/app/features/*"]

Create some sample feature module like home.module
The log given by the failure.
ng serve failed at compiling project (same for --aot and for running test)
ERROR in ./src/app/app.module.ts
Module build failed: Error: /Users/***/test/src/app/app.module.ts (7,28): Cannot find module<EMAIL_ADDRESS>    at _checkDiagnostics (/Users/***/test/node_modules/@ngtools/webpack/src/loader.js:145:15)
    at /Users/***/test/node_modules/@ngtools/webpack/src/loader.js:172:17
 @ ./src/$$_gendir/app/app.module.ngfactory.ts 13:0-48
 @ ./src/main.ts
 @ multi webpack-dev-server/client?http://localhost:4200/ ./src/main.ts

Mention any other details that might be useful.
I think the problem is place where tsconfig.json is located (currently /src/tsconfig.json). Also it is hard for IDEs to recognize/adapt changes inside tsconfig.json since he's not in the projects root (mentioned paths were not recognized by Visual Studio Code).
When I moved it into project root folder and edited angular-cli.json file with
"tsconfig": "../tsconfig.json"

everything works fine.

@elgibit I think you might need to leave out the src part from the paths in your paths key.

@beeman You're right! Thanks 🥇
I didn't catch that, also VSC is reacting to it correctly.

Awesome! Thanks for the gold @elgibit 😎


External projects common dependencies not handled correctly

In GitLab by @tpaluszk on Apr 2, 2019, 13:27
Currently CMake script doesn't correctly handle external projects, which are dependencies of several plugins (e.g. Paraconf, Config validator and FlowVR depend on YAML, but only Paraconf adds YAML external project, this leads to error when USE_PARACONF=SYSTEM and USE_YAML=EMBEDDED).
One possible solution would be to add a macro/function, which contains all dependencies common to plugins:
macro(add_external name)
  if("${name}" STREQ "YAML")
    ExternalProject_Add(YAML ...)
    set(YAML_ADDED TRUE)
  elseif(...)
  ...
  else()
    message(FATAL_ERROR "Invalid project name")
  endif()
endmacro()

And use it when adding plugin ExternalProject:
...
if(NOT "${USE_YAML}" STREQUAL SYSTEM)
  if (NOT ${YAML_ADDED})
    add_external(YAML)
  endif()
  add_dependency(... YAML)
endif()

closed via merge request !214

https://guix-hpc.bordeaux.inria.fr/about/

created merge request !214 to address this issue

We should investigate using guix-HPC instead of cmake for the PDI distribution (but keep cmake as buildsystem for PDI proper!)


artifactory_ldap_setting -> user_dn_pattern Should be optional

Describe the bug
user_dn_pattern required when should be optional.
│ Error: Missing required argument
│
│   on main.tf line 61, in resource "artifactory_ldap_setting" "ldap":
│   61: resource "artifactory_ldap_setting" "ldap" {
│
│ The argument "user_dn_pattern" is required, but no definition was found.
Requirements for and issue
Configure Artifactory LDAP setting

[ ] A fully functioning terraform snippet that can be copy&pasted (no outside files or ENV vars unless that's part of the issue)

resource "artifactory_ldap_setting" "ldap" {
  key              = "ldap"
  ldap_url         = "ldaps://ldap.com"
}

[ ] Your version of artifactory (you can curl it at $host/artifactory/api/system/version

7.0

[ ] Your version of terraform
1.14

Expected behavior
user_dn_pattern should be optional or have a default of ""

This was closed with #355 last week. You don't have repeatable HCL here that shows us what version of the provider you're using

https://github.com/jfrog/terraform-provider-artifactory/blob/master/pkg/artifactory/resource_artifactory_ldap_setting.go#L69-L73
Released with v2.22.2


MenuFlyout does not update correctly on theme change

Describe the bug
Changing the system theme from Light to Dark or from Dark to Light will not update the Theme of a MenuFlyout on runtime like the rest of the controls.
This happens also in WinUI 3 Gallery app, but randomly, as I can't reproduce it everytime. On my app, on the otherhand, I can reproduce it everytime.
The only way it will display the correct theme is to reopen the app.
It might also be related to #3373 which was for WinUI 2.
Steps to reproduce the bug

Create a blank WASDK app, packaged.
Add a MenuFlyout on the MainWindow.

<StackPanel Orientation="Horizontal" HorizontalAlignment="Center" VerticalAlignment="Center">
    <AppBarButton Icon="Sort" IsCompact="True">
        <AppBarButton.Flyout>
            <MenuFlyout>
                <MenuFlyoutItem Text="By rating" Tag="rating"/>
                <MenuFlyoutItem Text="By match" Tag="match"/>
                <MenuFlyoutItem Text="By distance" Tag="distance"/>
            </MenuFlyout>
        </AppBarButton.Flyout>
    </AppBarButton>
</StackPanel>

Open the MenuFlyout.
Step 3 is important, because if you don't open it first, and you change the theme, then open, the correct theme will be applied.

Change the system theme from the Settings App > Personalization > Colors : Chose Color Mode - Light or Dark

Open the MenuFlyout.

Observe the strange color state of the Flyout, it's not Light or Dark theme, more of a broken dimmed or washed out state.

Expected behavior
The Flyout should display the correct theme based on what theme was selected by the OS at runtime, without the need to reopen the app or to not open the MenuFlyout first.
Screenshots

Light Theme normal state

Light Theme switched state

Dark Theme normal state

Dark Theme switched state
NuGet package version
WinUI 3 - Windows App SDK 1.2.2: 1.2.221209.1
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

We were able to repro this simple case, looks like a theming issue on the background, thanks for filing.

@gabbybilka I'm also hitting this with a Flyout with custom content. It seems that if the flyout isn't shown prior to the theme change it works correctly, but after, the style seems to be cached. Is there a workaround until this is fixed to notify the flyout to recreate the content or update its style references?

This also affects content dialogs.

A fix for this was checked in for WinAppSDK 1.6, and was included in 1.6-experimental1.


Consider having shipit.local promise reject with the same object as the resolve

Currently a string error trace gets returned on a rejection. This string is very messy to use for controlling code flow.  I find it a lot easier to read the stderr string. It would be great if an object was returned that gave us options so we could use the value the best suites the situation.

Sounds reasonable. PRs welcome, otherwise I'll try to get to it.