get cookie value

Is there a proc that returns the value of a cookie? Something like:
proc getCookie(req: Request, val: string): string =
  let
    c = req.headers.table.getOrDefault("Cookie", @[""])
    cookies = c[0].parseCookies()
  result = cookies.getOrDefault(val)

I did a search but didn't see a proc for this. I used the std lib for the parseCookies proc.

yeah, haven't implement such a api , I think you can do let c:string = req.headers["Cookie"]; let cookies = c.parseCookies(),  import std/cookies first.

fixed in https://github.com/bung87/scorper/commit/25316365c7f70ee782d6a9b61ade07fba503eac7 v1.0.18


Automate the deployment of the database.

As a program i want to be able to open up the database by my self, to enable all that use me to acces the database.
Acceptence critiria:
The program can deploy the database by it self.

We deploy a new api version, when we create a new release.


[ADnD_2E_Revised] New Feature / Rogue section v3

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
All changes can be seen on the forum: https://app.roll20.net/forum/permalink/11502391/

Character Sheet Info Roll20 Internal Use only.


Fix memory leak if the client fails to register with the server

registration_data_t is not feed if the server does not respond to the connection message or the user decides to quit the (Wakaama) client before the server respond.
See: https://github.com/eclipse/wakaama/issues/693

closing this PR and reopening #696 to keep things simple.


You should be able to add arbitrary class bindings to md-header-cell elements.

You should be able to add arbitrary class bindings to md-header-cell elements.
Originally posted by @jelbourn in https://github.com/angular/components/issues/6470#issuecomment-322621918

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate.
(this is a canned response)


Revert "Expose an HTTP socket to receive app container logs"

Reverts GoogleCloudPlatform/appengine-sidecars-docker#15
Due to possible stability regressions from routing Docker logs directly to fluentd, we're rolling this feature back. It's not important that this rollback be picked up in a timely fashion because the business logic is elsewhere; this is just cleanup of a now unexposed feature.
R: @apolito
CC: @isdal

@andrewsg FYI, you should set your git config to use your corp e-mail address. See here.

I did! The problem with the CLA check here was that I edited it in the github interface initially. I had to amend the commit manually (and the push -f) to fix it.

LGTM


Add power operator

This issue was originally filed by @ahmetaa

A minor, and possibly breaking "familiarity" principle feature request.
Could it be possible to add Python's power (**) operator for representing Math.power(x,y)? Such as, x**y = Math.pow(x,y)
http://docs.python.org/release/2.5.2/ref/power.html

I don't think an extra operator that is only really useful for numbers, is worth the syntactic footprint it requires.
A function like power could just become a member on num, so you could do 4.pow(5). Having it as external, and still working on both doubles and integers, seems unnecessary.


tpacpi-bat: init at 3.0

Motivation for this change
Allow setting battery thresholds on newer Thinkpad models that don't support the tp_smapi kernel module.
Things done

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

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Pushed fixes some other things I found. You should use a version in the derivation name and fetchFromGitHub only fetches the tarball provided by Github instead of the git repository.

Squashed our commits & merged. Also fixed the package name typo in the commit message.

Thanks! :beers:

@orbekk is this the same tpacpi-bat which comes bundled with tlp ( /nix/store/*-tlp-0.8/lib/tlp-pm/tpacpi-bat )? If it is package is cooler than the tlp bundled tpacpi-bat then it might be a good idea to replace it when building tlp :)

@makefu Thanks for pointing that out. I'll send a pull request to replace the one that comes bundled with TLP.
@fpletz fetchFromGithub is neat. I'll keep it in mind for the future.

@makefu Just took another look at this. The tpacpi-bat that comes with tlp is only used internally, and I think replacing it is unnecessary and might break something.

@makefu Just took another look at this. The tpacpi-bat that comes with tlp is only used internally, and I think replacing it is unnecessary and might break something.

@orbekk thanks for the heads-up :+1: let's just have two separate packages then


remove gdspy dependency from gdsfactory

gdsfactory is built on top of gdspy
The main issues are:

extra dependencies makes the package harder to install and sometimes hard to follow and document
gdspy is no longer developed (only in maintenance mode). gdstk is supposed to be the successor of gdspy
gdspy requires conda install for windows. Now you can install gdsfactory through mamba/conda but it would be nice to only depend on one package manager such as pip

We have some options
1. switch to klayout API

Klayout has faster booleans
Klayout builds wheels for windows pip install klayout. So no more need to conda install gdsfactory for windows users.
Klayout is already an optional dependency
Klayout API is not super well documented

See @amccaugh work on the phidl klayout backed
@sebastian-goeldi
@thomaslima
See zeropdk
2. switch to gdstk

gdstk is an improved version of gdspy

3. switch to rust
https://github.com/dan-fritchman/Layout21/
What do you think?
@thomasdorch
@tvt173
@flaport
@basnijholt

I'm in favor of switching to the KLayout backend

I obviously support the klayout option but disclaimer is I don't use gdsfactory yet as I focus on zeropdk. That said, next 0.27 version will support static type checking with mypy and docstring inspection from vscode, which should be helpful for you guys. See https://github.com/KLayout/klayout/pull/1125

I would be in favor of migrating to the gdstk backend. My experience working with both gdspy and gdstk is that the APIs are similar, give the same layout results within ~1nm (so folks existing silicon is back compatible) and the compile times are much faster in gdstk.

I agree with @alexsludds . I think a migration to gdstk will be much more straightforward and there should be limited breaking changes because of the similarity between the two. i'm a big fan of klayout also and appreciative of the work from @thomaslima to make it easier to use the python API! but I'm not sure it makes sense to try and migrate gdsfactory over

Hi @sebastian-goeldi I am curious about your comment about merging partial gds. I have to deal with this too and need to manually check for cell duplicates. Would you mind sharing an example and an expected behavior? I can try to implement a patch, or at least give a better error message.
BTW Here's my code for loading a specific cell from a gds file: https://github.com/lightwave-lab/zeropdk/blob/master/zeropdk/klayout_extend/layout.py

Hi @thomaslima . I don't have an example directly at hand. I will try to get a minimal working example running when I get back from vacation ;). But I think the proper solution would be to run either a full strm2gds (I think that's the name of the routine) on the partial layouts or run the python equivalent with the LayoutDiff. With the diff object we can just throw errors/warnings if there's a difference between the two. If we don't care about names of cells (which I believe many do, including me), we could also just use conflict_resolution rename.

Hey folks,
I posted this on a relevant thread on the phidl package, but I thought it important to put here too.
As part of converting gdsfactory away from gdspy I think it's important to move phidl away from gdspy to gdstk. I made a brief first pass at converting the phidl backend from gdspy to gdstk:
phidl with gdstk backend
It's not bug free, but it does generate some basic geometry. I need folks help with handling dependencies on gds/oas generation.
In particular, if you run this example code (borrowed from the phidl examples):
example code
You will see it tries to create three waveguides of different shapes. However, in the layout all waveguides have the same shape, equal to the last waveguide that was placed.
This is because of how I tried to implement the write_gds function:
write_gds function
I spent a few hours trying different ways to implement phidl's write_gds and have it be compatible with gdstk, but I can't get the dependency handling correct. I'm hoping this might be really obvious to someone else.
Best,
-Alex

thank you Alex, it looks good, I fixed some more tests
Another alternative is to create an intermediate format (such as shapely) and use klayout to read and write gds files
this is what Helge does in gdshelpers
@HelgeGehring
@jenshnielsen
@Dominik-Vogel

For example, the VLSIR project uses protocol buffers as an intermediate representation, and it has a rust package from converting to procol buffer to GDS
https://github.com/dan-fritchman/Layout21/issues/22

After the gdstk segmentation issues
Here is an MVP with klayout backend
@growly is also looking into using VLSIR as a backend

We were able to get rid of segmentation faults in gdstk by avoiding inheritance from gdstk.Polygon and gdstk.Label
gdsfactory 6.0.0 will be released soon using gdstk backend :tada:
we also have an MVP of a klayout_backend on a branch that we will further investigate


contract file parsing and solc errors

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
latest
What command(s) is the bug in?
forge build
Operating System
macOS (amd)
Describe the bug
Creating an issue to dump the relevant problems as I find them.
The issue appears to be in OS (like macos) that is case insensitive, thus capitalized and non-capitalized smart contracts are the same. The problem arises when the file appears to be capitalised, but for some reason solc sees it uncapitalized.
In the following example, we see solc showing the same file with 2 different versions (capitalised, not capitalised).

can you push a branch where I can see the content of those files?

Ping @odyslam - can you provide an example we can check out? :smile:

think we can close this


added morgan for request logging

This is a trivial change. I did it as a basic middleware test/because I thought morgan was cool. Could do with some more tweaked morgan config, but that can come later.


[UI] Update Configuration on Cluster Monitoring Page Results in 404/Fail

What kind of request is this (question/bug/enhancement/feature request):
BUG
Steps to reproduce (least amount of steps as possible):
Ensure Cluster Monitoring is enabled, then go back to the Cluster Monitor and click on Update Configuration.
Result:
404/Fail
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.2.0-alpha6
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Imported HA Cluster
Machine type (cloud/VM/metal) and specifications (CPU/memory): Meta 16GB/4 Core
Kubernetes version (use kubectl version): 1.12.1

Close in favor of https://github.com/rancher/rancher/issues/18004
Users don't have to go to the catalog page to edit it


Feature: "read more" suggestions at the end

Possible feature: suggest other articles at the end of an article.
Casper for Ghost does this well.

Hi,
First of all thanks for the great work porting the casper theme to hugo.
I reworked this theme slightly last week to support "readnexts":
Sorry I don't have enough time in the short term to make a proper pull request.
Here are the changes:

New partial readnext.html

{{ $next := or .NextInSection .Next . }}
{{ $prev := or .PrevInSection .Prev . }}

{{with $next}}

{{.Title}}

{{.Description}}

{{end}}
{{with $prev}}

{{.Title}}

{{.Description}}

{{end}}

At the end of _default/single.html I added before the footer:

{{ partial "readnext.html" . }}

I picked the current casper css, modified to use this theme's icon font.

Best,
Arthur

Thanks ! I'll implement this

I implemented something similar to @arthur-flam's suggestion in on my blog. Happy to do a PR.

@sungwoncho Just going through this now - how should this be implemented/turned on? We may want to suggest an update to the readme for this.

@anthonylavado The feature is turned on automatically. If I remember correctly, the implementation is as follows:

if inside a post page, the twitter card will use the post title, summary, and photo
if outside, it will use the site title, summary, and photo.

I agree that a doc might be helpful.

@sungwoncho Oh, I was referring to the suggestions. At any rate though, I wonder about updating the README.
@vjeantet, I can try to create a draft including this if you would like?

@sungwoncho I figured out what's happening. The changes you've made are in the /layouts/page directory, but there are also template pages in the /layouts/_default/ directory.
The _default page takes precedence, and it doesn't have the include for the suggestions at the bottom.


Support kyuubi procedures

Why are the changes needed?

Support kyuubi defined procedures, refer Iceberg-Spark procedures(https://iceberg.apache.org/spark-procedures/)
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

cc @ulysses-you since this PR rename lots of package name of classes.

cc @ulysses-you since this PR renames lots of package name of classes.

Some class is only accessible in package org.apache.spark.sql, so I rename the package for convenience.
And I think the extension is only for spark-sql, so, it makes senses.

But it seems that it also rename class  KyuubiSparkSQLExtension.
So, let me make it compatible with before.

cc @ulysses-you since this PR renames lots of package name of classes.

revert back and just add a util class for some classes that only can be accessed in package org.apache.spark.sql.

Codecov Report

Merging #1301 (eae5931) into master (c6710ae) will decrease coverage by 0.18%.
The diff coverage is 72.47%.

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

@@             Coverage Diff              @@
##             master    #1301      +/-   ##
============================================
- Coverage     78.50%   78.31%   -0.19%
- Complexity      110      155      +45
============================================
  Files           178      189      +11
  Lines          7215     7421     +206
  Branches        880      904      +24
============================================
+ Hits           5664     5812     +148
- Misses         1067     1101      +34
- Partials        484      508      +24

Impacted Files
Coverage Δ

...ala/org/apache/kyuubi/sql/command/statements.scala
0.00% <0.00%> (ø)

...ql/zorder/ZorderSparkSqlExtensionsParserBase.scala
69.44% <ø> (-3.48%)
:arrow_down:

...cala/org/apache/spark/sql/KyuubiSparkSqlUtil.scala
0.00% <0.00%> (ø)

...e/kyuubi/sql/command/ResolveKyuubiProcedures.scala
49.27% <49.27%> (ø)

...che/kyuubi/sql/KyuubiSparkSQLCommonExtension.scala
84.61% <50.00%> (-6.30%)
:arrow_down:

...mmand/KyuubiCommandsSparkSqlExtensionsParser.scala
68.57% <68.57%> (ø)

.../main/scala/org/apache/kyuubi/sql/ParserBase.scala
83.33% <83.33%> (ø)

...apache/kyuubi/sql/command/ProcedureParameter.scala
83.33% <83.33%> (ø)

...he/kyuubi/sql/command/KyuubiDefinedProcedure.scala
87.50% <87.50%> (ø)

...org/apache/kyuubi/sql/command/KyuubiCommands.scala
88.88% <88.88%> (ø)

... and 3 more

Continue to review full report at Codecov.

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

I think we can combine this grammar (EXEC ... ?) and zorder, adding a new .g4 and something else seems not friendly for maintain.

I think we can combine this grammar (EXEC ... ?) and zorder, adding a new .g4 and something else seems not friendly for maintain.
sure

if it is needed, will reopen it


Port to Chromium

Any chance to port Copy ShortURL to Chromium?
This is just a question- if you don't have such plans close this issue.

I think that's a great idea. It should be relatively easy though I have never tried it! Will have to be over Christmas break when I have some time to code!

@fwenzel
Thank you. I had been Firefox user for years but recently I switched to a Chromium fork. Now I use Firefox mostly for testing add-ons. Happy holidays and happy coding!


UX functional tests to demonstrate validating state between XAML and UIKit/CoreAnimation

Demonstrating a simple pattern to validate the XAML and UIKit properties using the FunctionalTest framework

CALayerAppearanceTests.mm - 2 tests - validating opacity and backgroundColor between the UIKit and XAML layers
UIButtonsTests.mm - 1 test - re-use XAMLCatalog's buttonViewController to validate the state of a button backgroundColor property

CHANGES:

Moved all UIKit Tests under its own folder/filter
Created a CoreAnimation test suite for CALayer tests

TODO:

ARC-enabled these UX tests where possible
Formalize the relationship between UIButton tests that reach into existing VC (e.g. XAMLCatalog)
Jared doesn't like NSCondition - explore if there are other options that work better :-)

This change is

Think I can squash it first and compare against github/develop. Opening a new PR soon.


Can't reorder columns with custom column header template

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior

When a custom column header template is set, the rows can't be reordered anymore.
Expected behavior

When reorder is activated, it should be possible to reorder all columns.
Reproduction of the problem
The behavior can be seen in the example: https://swimlane.github.io/ngx-datatable/#inline. Only age can be moved.
What is the motivation / use case for changing the behavior?
Better options for customization
Please tell us about your environment:

Table version: 10.0.5

Angular version: 4.3.4

Browser: Firefox 55.0.2

Language:

I have found the solution. Is already working as it should but ngx-datatable-header-template and datatable-header-cell-wrapper is needed.
    <ng-template #searchTpl let-column="column" let-sort="sortFn" ngx-datatable-header-template>

          {{column.name}}

    </ng-template>

Ah ok, I see now. For any one looking at my example, you basically just have to add the classes mentioned by @JackArlington - so my example ends up looking like this:
  <ngx-datatable-column prop="description">
    <ng-template let-column="column" let-sort="sortFn" ngx-datatable-header-template>

        {{ 'description' | translate }}

    </ng-template>
  </ngx-datatable-column>

The documentation really should be updated to make this more clear


improve contribution guide

at least a stub security bug section, this works as a policy most likely
until we can improve this (TODO we need an email for this!)
adds info in CONTRIBUTING.md for creating helpful normal issues for us, this
is much the same info as the template
coding style section. This was lacking, and led to wishy washy reviews. now
we have an official reference in place to point at for 'do this this way
please' and not just random opinions. everyone should read this if they
haven't! I have it bookmarked...
info on creating useful commit messages and commit formatting, like code
reviews, its nice to have this in the contrib guide to reference when asking
people to do this so that it's not just a one off opinion

tried to make this pretty lax, the last thing i/we want is for the
contributing process to be overbearing, I do think the contribution guide
serves the dual purpose of best practice enforcement as well as helping people
to maneuver the process to make it easier for all of us (them included).
open to idears. this is a convergence of a few guides from popular repos

closes #1075

gonna wait on security email to big green button, no rush. thanks

lgtm


North Charleston, SC - letter to Mayor and City Council Members

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Defunding the North Charleston Police Department
Message:Dear Mayor Summey and North Charleston City Council Members,
My name is [YOUR NAME] and I am a resident of [NEIGHBORHOOD/CITY].I am demanding you reallocate funds from the North Charleston City Police department and invest them into social services that would directly improve the well-being of North Charleston citizens.The budget for 2019 was $35,882,389, and the budget for 2020 is $37,834,665. This is almost a 6% increase from 2019 to 2020 and an almost 7% increase from 2018 to 2020. That money could be better spent on supporting affordable housing, educational opportunities, healthcare, and community outreach programs that are more successful at promoting safe and stable communities than law enforcement. I demand more aggressive financial support be directed to those areas.
I am demanding long term and sustainable change. I am demanding that the city of North Charleston’s budget be better spent on quality of life for all. In particular for those in our Black and Brown communities, who are more than likely to be directly affected by police brutality and violence. I also urge the North Charleston City Council to enact legislation that holds police accountable and to overturn policies that allow police to engage in unlawful behavior with impunity.
People are marching in the streets because they know that these actions will result in a healthier, more just society. I implore you to please listen to the needs of your constituents and take immediate action to address their concerns. Can I count on you to consider an alternative budget that puts a focus on social service programs?
Thank you,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

I’m reviewing

doing this one


K3s support for cilium-cli

Proposal / RFE
Is your feature request related to a problem?
yes - we use k3s for an on-prem install and we'd like to use cilium as well. The documentation for installing cilium shows the CLI as the recommended way to install cilium, however the tool does not K3s as supported.
Describe the solution you'd like
Update documentation to reflect support. If needed, a note could be added that K3S is supported, but not tested.

I may have jumped the gun here, but it seems like cilium-cli might work just fine.
For some reason cilium-cli wasn't finding my cluster. I implemented a work-around using this.
mkdir ~/.kube
kubectl config view --raw=true > ~/.kube/config
chmod 0600 ~/.kube/config
# - comment out certificate-authority-data
# - add `    insecure-skip-tls-verify: true` just under `server: https://<IP_ADDRESS>:6443`

Yup, this was just my mistake - When I originally explored this, I had my KUBECONFIG set, but not exported, so cilium was never seeing it.
Is k3s officially supported? Maybe the main page of this project just needs to be updated?

Thanks for the report. I assume you followed the getting started guide from the Cilium docs: https://docs.cilium.io/en/v1.11/gettingstarted/k8s-install-default/?
Is there anything in the guide that you think needs correction or clarification?
As far as cilium-cli is concerned, k3s is supported but currently not covered by CI. Same for Cilium itself. We usually test some of these k8s distributions/platforms (e.g. k3s) manually before a new release. It seems installation and single node tests worked on k3s, but there was an issue in connectivity tests with two nodes.

Is k3s officially supported? Maybe the main page of this project just needs to be updated?

I think we can just update the README of this project to mention k3s as supported. Happy to take a PR in case you want to send one.

I got it working a little a bit before you responded, ty for taking a look.

@jamesharr How exactly did you get it to work? I have been trying to install cilium via the cli on my k3d cluster the following way:
docker exec k3d-server-0 sh -c "mount bpffs /sys/fs/bpf -t bpf && mount --make-shared /sys/fs/bpf"
kubectl taint nodes k3d-server-0 node.cilium.io/agent-not-ready=true:NoSchedule --overwrite=true
cilium install --context k3d --wait false --config wait-bpf-mount=false --wait-duration 0s

However, the cilium pod is stuck in init phase with
sh: mount: not found
Mounting eBPF filesystem...
sh: mount: not found

On the server node fs i can see that the mount point in mount exists and is ready to be used
$ mount
none on /sys/fs/bpf type bpf (rw,relatime)

@Demonsthere , I was running k3s on top of Alma Linux 8.5 (CentOS clone) in a VM; not inside a container. Alma had bpfs mounted already.
I had tried installing Cilium on top of Alma inside an LXC container (via ProxMox VE) and something about BPFS inside a nested container just wasn't getting along. Unfortunately I didn't resolve it and didn't keep and notes because I figured that was probably a long shot working in the first place.
As a side note, I did need to turn off FirewallD. When it was enabled, it caused several problems inside the overlay. And the work around most listed in forum posts of enabling IP masq through firewall-cmd made things work, but really screwed up any NetworkPolicy -- from the pod's perspective, any traffic that came in from another node was NATd to the local node's address (10.42.x.1) so any firewall rule went out the window. That didn't show up on a tcpdump inside the pod, so I spent some time questioning my sanity until I noticed the source address when adding -v to netcat while in listen mode.
If you do get it to work inside of k3d, please do post somewhere because I'd be interested

I actually managed to get cilium working on my k3d cluster, but there are some rough corners:

Create the cluster

k3d cluster create --image ${K3SIMAGE} --registry-use k3d-localhost -p "9090:30080@server:0" \
   		-p "9091:30443@server:0" foobar \
--k3s-arg=--kube-apiserver-arg="enable-admission-plugins=PodSecurityPolicy,NodeRestriction,ServiceAccount@server:0" \
--k3s-arg="--disable=traefik@server:0" \
--k3s-arg="--disable-network-policy@server:0" \
--k3s-arg="--flannel-backend=none@server:0" \
--k3s-arg=feature-gates="NamespaceDefaultLabelName=true@server:0"

Create the bpffs ahead of installing cilium:

docker exec k3d-foobar-server-0 sh -c "mount bpffs /sys/fs/bpf -t bpf && mount --make-shared /sys/fs/bpf"

Taint nodes

kubectl taint nodes k3d-foobar-server-0 node.cilium.io/agent-not-ready=true:NoSchedule --overwrite=true

The funny part. Install cilium using helm with the following values, then wait for it to fail due to missing cgroupv2 mount and create that mount. We cannot create it beforehand, because cilium is expecting it to be available in /run

setValues:
  externalIPs.enabled: true
  nodePort.enabled: true
  hostPort.enabled: true
  hubble.relay.enabled: true
  hubble.ui.enabled: true

		until bash -c "kubectl get pods -n kube-system -l k8s-app=cilium | grep 'Init:CreateContainerError'"; do \
			kubectl get pods -n kube-system -l k8s-app=cilium | grep "Running"; \
			is_running=$${?}; \
			if test $${is_running} -eq 0; then \
				break; \
			else \
				sleep 5; \
			fi; \
		done
		docker exec k3d-foobar-server-0 sh -c "mount --make-shared /run/cilium/cgroupv2"

With this I was able to get a working setup running, however:

on my Mac and docker4mac this is working as expected
a colleague, running ubuntu reported problems with not being able to recreate the setting (startup, teardown, startup again)
our CI, also running ubuntu was not able to curl any service exposed by the cluster (without cilium it worked fine)
don't let me get started to GKE dataplane v2...

I have to say, this is quite the adventure...

I'm seeing the same issue running Cilium in a k3d cluster, where the agent's ebpf-mount init container fails with:
sh: mount: not found

I managed to log into that init container, and found that indeed, /usr/bin/mount is absent. In fact, /usr is rather empty:
# ls /usr/
libexec

This is very surprising, because if I launch the exact same image with Docker, /usr/bin/mount works fine (and /usr is quite full):
❯ docker run --rm -ti quay.io/cilium/cilium:v1.11.2@sha256:4332428fbb528bda32fffe124454458c9b716c86211266d1a03c4ddf695d7f60 mount
overlay on / type overlay (rw,relatime,lowerdir=/var/lib/docker/overlay2/l/RVNMOVON4PZLH2T63BJUPCUORR:/var/lib/docker/overlay2/l/GLCILOFUKT34HJRNJJ4FAYNV64:/var/lib/docker/overlay2/l/U6PZOYWZQQVQQPKBDGCS56B2AQ:/var/lib/docker/overlay2/l/E6DNCGQPYXIC6SQC2IKZQYKW5B:/var/lib/docker/overlay2/l/CDB77LLWEEOWCDO2OXS4M4HGFK:/var/lib/docker/overlay2/l/YEHF4Q26ZKSJLY5W4ZT7SMPCLI:/var/lib/docker/overlay2/l/KJLUA5FBPCGC43VS5TRSKMGCLD:/var/lib/docker/overlay2/l/IGTN5GBKPXRSZK73YPW5J5THKK,upperdir=/var/lib/docker/overlay2/c5d14eb98be1489facb9eb11c5a0e900e5e863547ebb5acc3a0ac4bd51990f66/diff,workdir=/var/lib/docker/overlay2/c5d14eb98be1489facb9eb11c5a0e900e5e863547ebb5acc3a0ac4bd51990f66/work)
proc on /proc type proc (rw,nosuid,nodev,noexec,relatime)
tmpfs on /dev type tmpfs (rw,nosuid,size=65536k,mode=755,inode64)
devpts on /dev/pts type devpts (rw,nosuid,noexec,relatime,gid=5,mode=620,ptmxmode=666)
sysfs on /sys type sysfs (ro,nosuid,nodev,noexec,relatime)
cgroup on /sys/fs/cgroup type cgroup2 (ro,nosuid,nodev,noexec,relatime,nsdelegate,memory_recursiveprot)
mqueue on /dev/mqueue type mqueue (rw,nosuid,nodev,noexec,relatime)
shm on /dev/shm type tmpfs (rw,nosuid,nodev,noexec,relatime,size=65536k,inode64)
/dev/mapper/vgubuntu-root on /etc/resolv.conf type ext4 (rw,relatime,errors=remount-ro)
/dev/mapper/vgubuntu-root on /etc/hostname type ext4 (rw,relatime,errors=remount-ro)
/dev/mapper/vgubuntu-root on /etc/hosts type ext4 (rw,relatime,errors=remount-ro)
devpts on /dev/console type devpts (rw,nosuid,noexec,relatime,gid=5,mode=620,ptmxmode=666)
proc on /proc/bus type proc (ro,nosuid,nodev,noexec,relatime)
proc on /proc/fs type proc (ro,nosuid,nodev,noexec,relatime)
proc on /proc/irq type proc (ro,nosuid,nodev,noexec,relatime)
proc on /proc/sys type proc (ro,nosuid,nodev,noexec,relatime)
proc on /proc/sysrq-trigger type proc (ro,nosuid,nodev,noexec,relatime)
tmpfs on /proc/asound type tmpfs (ro,relatime,inode64)
tmpfs on /proc/acpi type tmpfs (ro,relatime,inode64)
tmpfs on /proc/kcore type tmpfs (rw,nosuid,size=65536k,mode=755,inode64)
tmpfs on /proc/keys type tmpfs (rw,nosuid,size=65536k,mode=755,inode64)
tmpfs on /proc/timer_list type tmpfs (rw,nosuid,size=65536k,mode=755,inode64)
tmpfs on /proc/scsi type tmpfs (ro,relatime,inode64)
tmpfs on /sys/firmware type tmpfs (ro,relatime,inode64)

I can't find what causes this /usr directory to be empty, as it doesn't seem to come from the Pod spec.


Update NuGet packages to contain XML documentation

Already done with Microsoft.Build.Framework and Microsoft.Build.Utilities, still need to do Microsoft.Build.
@rainersigwald @AndyGerlicher @cdmihai I'll probably skip Microsoft.Build.Tasks because most people don't reference it or need intellisense from it right?

Seems fine to deprioritize or skip Tasks, yeah.

Some tasks are meant to be subclassed right?

@cdmihai from my quick look at the classes in Tasks, most of them are subclasses for other tasks in the Tasks assembly.
The stuff that our general audience would consume are in Utilities (ie Task, ToolTask, etc).  Does that seem correct to you?

I was thinking of the case where people want to extend actual tasks like Copy. But I have no idea how prevalent that is in the MSBuild community :)


2.3.4 cant play catchup anymore

2.3.4 cant play catchup anymore nor see that option in any of the channels also can u tell me if anyone planning to add Hungama tamil channel to the list?
Much appreciated for your help / response and time.. thanks a lot

plugin.video.jiotv.zip
try this zip. If it is working. Krreet can cross check code and do necessary correction. Now it don;t required to import inputstream.adaptive so can be removed from addon.xml

Catchup is playing fine for krreet/tobalan v2.3.4
The Catchup option appears as context menu on long press/right click/options menu button on the Channel Name.

However for easy access to select channels Catchup by filter on home screen, I tried this and it works great.
Step 1 - Catchup Option

Step 2 - Catchup Channels

Step 3 - Catchup Shows List

Negative TV as the same issue as mentioned above no catch up unless I am doing something wrong pls help

I only see the following right click or long press

This is TV Channels section of KODI.
Catchup video should play on clicking program name in the program guide (EPG).
Another option is in the Add-ons section of KODI, in JioTV addon open any channel with Catchup by Genre or Language and right-click/long press on the channel name. Catchup appears as an option in the context menu which when clicked gives catchup shows day-time list.

doing something wrong pls help

Enable Catchup in the IPTV Simple Client addon Settings/Configuration.
Restart KODI and then check again.

Did you mean here coz  i dont see an option for catch up?

Check all Tabs for Catchup Enabled option...I think its Timeshift tab

I only see the following right click or long press

You are getting only total 1024 channels. Now 1030 channels are there in Channel List API v1.4 and 1.3
Turn OFF the API 3.0 switch in JioTV addon.
Turn ON the Generate Playlist on Startup.
Restart KODI and check to see 1030 channels in all.

Tried but still 1024 channels listed any other way

Did you mean here coz i dont see an option for catch up?

Got it. Its the Catchup Tab at the end.
KODI Skin : Estuary

KODI Skin : Quartz

As soon as I enable and click ok error pop's up saying IPTV simple client connection lost same issue i reported earlier
IPTV simple client now shows connection lost #761

error pop's up saying IPTV simple client connection lost

It is because you have more than one instances of JioTV playlist created by Kodi. Try to keep only one instance.

reinstall then? please advise but I have done this reinstall many times earlier as well

No. No need to reinstall, just remove extra instances.

trying to enable catchup comes with the same error again IPTV simple client connection lost

Is playlist.m3u of JioTV add-on in any of the instances? Just keep it.
Are you doing Setup PVR ? Turn OFF Generate Playlist on Startup in add-on Settings.
It is plugin's programmatic access to IPTV Simple Client that is causing all this "connection lost" issues.

trying to enable catchup comes with the same error again IPTV simple client connection lost

Is playlist.m3u of JioTV add-on in any of the instances? Just keep it. Are you doing Setup PVR ? Turn OFF Generate Playlist on Startup in add-on Settings. It is plugin's programmatic access to IPTV Simple Client that is causing all this "connection lost" issues.

Hi @technofreak24, you are a genius and here is what i did on my desktop and smart TV removed Migrated Add-on Config and edited the highlighted instead of adding another see below

Here i could enable catch up and all if fine working but stil have 1024 channels

API 3.0 is turned off and how do i set channels list to 1.4 dont see any settings

API 3.0 is turned off and how do i set channels list to 1.4 dont see any settings

Turning OFF API 3.0 automatically makes Channel List 1.4 active.
Do Clear Cache in JioTV Addon settings.
Playlist has to be generated on startup once, by turning it ON and Restarting KODI.

Perfect API 3.0 is turned off and cleared cache restarted Kodi and playlist generated turned off generate playlist applied settings now I see zee ganga 129th channel still missing one channel then?

I can live with it unless u have anything? if not thanks a lot again @technofreak24

Catchup is playing fine for tobalan's plugin v2.3.4 or krreet's plugin v2.3.4. The Catchup option appears as context menu on long press/right click/options menu button on the Channel Name.  However for easy access to select channels Catchup by filter on home screen, I tried this and it works great. Step 1 - Catchup Option  Step 2 - Catchup Channels  Step 3 - Catchup Shows List

Which skin is this?

Which skin is this?

I am using Quartz KODI  skin. Quartz is a light-weight skin with a intuitive user interface and highly customizable home screen.
https://kodi.tv/addons/nexus/skin.quartz
https://kodi.tv/addons/matrix/skin.quartz
https://github.com/nonJon/skin.quartz

Which skin is this?

I am using Quartz KODI skin. Quartz is a light-weight skin with a intuitive user interface and highly customizable home screen. https://kodi.tv/addons/nexus/skin.quartz
https://kodi.tv/addons/matrix/skin.quartz
https://github.com/nonJon/skin.quartz

Thanks


Add link to OFL page to exported fixtures

A link to the fixture being exported should be included in the exported fixtures, either via an XML comment, or (in e:cue's case) in the comment field.
This helps us reach new users and the users to find the nice overview page and possibly updated versions.

Closed by #199


Make refresh interval configurable

Is your feature request related to a problem? Please describe.
30 seconds between refreshes is too high for my use case as sometimes my first layer takes less than that, rendering my automations based on that fail
Describe the solution you'd like
I have adjusted manually in the code for 5 seconds but I would like to either have it lower hardcoded or configurable.

Related to https://github.com/marcolivierarsenault/moonraker-home-assistant/issues/168
(will be tracking the progress over there)


Trouble running/installing.

So somehow I've managed to mess this up even though the install instructions are like 3 items long. Figures. I'm running linux mint if that makes any difference.
The problems begin on the third step/running the installation script. After entering ./install.sh I'm presented with the following message: cp: cannot stat '/home/xxxx/squash/test_themes/*': No such file or directory.
Any ideas?

I have updated the installation instructions and removed in the install script. This new system should make it easier for them to adapt squash to their own system.

You will need to edit squash to point to your themes and desired colors directory. You can do this by changing the first couple lines in squash. They default to the following:

THEMES_DIR=~/.config/squash/themes
CACHE_DIR=~/.cache/squash
OB_DIR=~/.local/share/themes/generic/openbox-3

After these have been set, you can change your other applications to point to these directories. Examples of these are provided in the Usage section.


Added Netflix Logo

Description
Please provide a 1 line description of your art
Fixes #issue-number
Insert the image of the art below
Checklist:
To fill the checklists, put x inside the brackets. [x]

[] My code follows the guidelines of this project
[] I have tested my code and it works properly
[] I have added just one folder that includes my Art files (jsx and css)
[] I have not edited App.js file
[] I have commented my code, particularly in hard-to-understand areas
[] My changes generate no new warnings
[] This is my original code and I have NOT copied the code from anywhere else.

Please merge my PR #41

@shahbazalam07, I'm closing this PR. Feel free to open a fresh PR whenever you make the changes.


feature: grpc API

@mudler I had to jump through a ton of hoops to get this working, but I thought it might be helpful: https://github.com/openai/openai-openapi/pull/41. If not it was still a great excuse to learn about Earthly and gRPC.


Discussion: handle slow peers (including HTTP peers)

Hi :)
Original issue from https://github.com/Chocobozzz/PeerTube/issues/2047
If we try to download a segment from a P2P peer or from a CDN, but if the remote peer is slow, it's difficult to detect, abort the download and try another peer. I don't think adding another variable to timeout a download (P2P download or HTTP download) would be a good idea, because maybe it's just the client that is slow (and we would abort all its downloads).
I don't really know how we could detect that a remote peer is slow, but I have some clues:

Compute the average bandwidth per peer and store the max:

If this maximum average bandwidth is greater than the segment bitrate then abort the downloads when the remote peer has an average bandwidth < segment bitrate
It the maximum average bandwidth is not greater than the segment bitrates then add a percentage API options on which we abort a download regarding the max average bandwidth available. Example: I download a segment from a peer (1.example.com) at 2MB/s. If I set the option to 50%, and I download a segment from another peer (2.example.com) but the average bandwidth is 700KB/s then I abort the download (1MB/s would be the threshold). We could also use this setting even if the maximum average bandwidth is not greater than the segment bitrates, to eject too slow remote peers

Blacklist remote ejected P2P peers

What do you think? Do you have a better idea?

Hi! Thanks for opening this issue. I'm wondering if there's a way to specifically prevent video from pausing, by retrying downloads far enough in advance in order to prevent playback from stalling. This would address the main symptom I'm observing. There might also be ways to change how the different peers are weighted relative to each other but I don't know how this currently works :)

There is requiredSegmentsPriority configuration parameter. If a segment is close enough to playhead and still not downloaded - P2P download, if any, will be canceled and HTTP download started.

@mrlika In our use case a peer could be a P2P or a HTTP peer, so this configuration parameter does not help.

What is an HTTP "peer"?
Do you mean a peer using HTTP as the source, and the HTTP download itself is slow?

Do you mean a peer using HTTP as the source, and the HTTP download itself is slow?

Yes, a HTTP peer is for example a CDN, or a static mirror. Our use case could be simplified like that: imagine you have a video file and x URLs that serve this video file. We use segmentUrlBuilder so p2p-media-loader downloads segments from different endpoints. But sometimes, an endpoint is slow and we need to know that, to abort the HTTP download and try with another URL.

Ah, so multiple HTTP sources. I believe in that case an extra parameter would be needed, allowing for the "client" to judge in realtime which "servers" or "http peers" have good performance and rate them as such. Bandwidth could be one metric but perhaps something else as well.

If we encounter a lot of slow video loading with webtorrent, do you think we have to disable the tracker in the static configuration while we find a solution here ?

@maxdrash is your problem related to this issue?
If "slow video loading in webtorrent" you mean static videos shared via webtorrent this isn't the right place.
I'm also unable to understand why disabling tracker would help?
If you mean the p2p peers are slow and so cannot watch the video, you can increase the probability of HTTP download.

If P2P is slow it switches to HTTP. It should not download slowly than from HTTP.

If P2P is slow it switches to HTTP. It should not download slowly than from HTTP.

Unfortunately, it could be slow from HTTP too. Not everybody pay a CDN. You could also have a network of mirrors, where some of them are slow.

so...how to change the segment url (to another server) when http download is too slow?

It is not a standard feature of HLS/MPEG-DASH video streaming but I think it can be implemented  using the configuration parameter:

segmentUrlBuilder
Function
undefined
Allow to modify the segment URL before HTTP request. If handled, expected a function of one argument of type Segment that returns a string - generated segment URL.

https://github.com/Novage/p2p-media-loader/tree/master/p2p-media-loader-core#hybridloader

Hello,
This issue is still useful in our use case


Allow the use of jsconsole behind https

Small fix that changes the way origin is computed to not force 'http://' but instead keep the protocol and whole path until the last '/'.
This way, jsconsole can be used behind reverse proxies that work with https (just as I do).

Looks like this was fixed by https://github.com/remy/jsconsole/pull/78


The Omniverse platform can't load the Cesium extension version 7 or higher

Hi,
I have the latest version of Omniverse installed, and tried all extension versions from the highest one up to version 6, and only version 6 is working for now.
I would really appreciate if you could tell me how can I use the latest version of Cesium in omniverse.
If there is any alternative way to have the latest extension run on omniverse, please let me know.
Thank you

@nzariei we now have a version table in the getting started guide. Hopefully that helps with any future issues:
https://cesium.com/learn/omniverse/omniverse-quickstart/#step-1-add-the-cesium-for-omniverse-extension-to-omniverse-usd-composer


Run the ruby/spec suite with the spec-runner on every commit

This PR adds a GitHub Actions workflow. This workflow will run Artichoke's ruby/spec harness on every commit to the trunk branch. The workflow will generate JSON reports that tags failing specs and lists exceptions logged when running the spec suite. The reports will be pushed to https://github.com/artichoke/spec-state.
Eventually, tooling in https://github.com/artichoke/spec-state will generate a static GitHub Pages website to present the information about historical spec compliance.
This workflow directly pushes to the spec-state repository with an SSH deploy key which is exposed to this repository as an organization secret.
Followup to #1322, #1088.
Fixes #142.
Fixes #101.
This workflow won't get triggered until it is merged to master. I may have to followup to this PR to fix any failures.

Oh whoops. The spec runner exits with an error code for the tagger and yaml formatters that needs to be suppressed.

The workflow succeeded! https://github.com/artichoke/spec-state/commit/782835724217930a5758cf3f60f5cb2d8f73319d
But this change has been reverted since the commit metadata was not properly interpolated: https://github.com/artichoke/spec-state/commit/fa3928c8e74e0173a6c5c9254471c5d4ed903885.


Fix a false positive for Lint/BinaryOperatorWithIdenticalOperands cop

This PR fixes a false positive for Lint/BinaryOperatorWithIdenticalOperands cop when using * operator with same variable operand.
Below is the case I encountered.
% cd path/to/github.com/teamcapybara/capybara
% bundle exec rubocop --only Lint/BinaryOperatorWithIdenticalOperands
(snip)

Offenses:

lib/capybara/queries/selector_query.rb:656:30: W:
Lint/BinaryOperatorWithIdenticalOperands: Binary operator * has
identical operands.
          cap_d = (a * c) - (b * b)
                             ^^^^^

256 files inspected, 1 offense detected

Maybe + operator has a similar false positive, first this PR only targets * operator that is the above pragmatic issue.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Technically speaking, that's a problem with all the arithmetic operators - you can write code like a + a or b * b, but there's also the question why would you do it, if you can change it to 2 * a or b ** 2.


⚠ Bofrost.es has degraded performance

In 0933195, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1378 ms

Resolved: Bofrost.es performance has improved in 5147ca7 after 10 minutes.


Vision Sensor Testing - Validate new Algorithm

Test W1 mark version of code and check for fatal errors on drivetrain Triball tracking
`#region VEXcode Generated Robot Configuration
from vex import *
import urandom
Brain should be defined by default
brain=Brain()
Robot configuration code
wait for rotation sensor to fully initialize
wait(30, MSEC)
def play_vexcode_sound(sound_name):
# Helper to make playing sounds from the V5 in VEXcode easier and
# keeps the code cleaner by making it clear what is happening.
print("VEXPlaySound:" + sound_name)
wait(5, MSEC)
add a small delay to make sure we don't print in the middle of the REPL header
wait(200, MSEC)
clear the console to make sure we don't have the REPL in the console
print("\033[2J")
#endregion VEXcode Generated Robot Configuration
------------------------------------------

Project: Auton Drive Test #1
Author: Shreyas Sharma
Created: Jan-11-2023
Configuration: Standard Team Motor Setuop

------------------------------------------
Library imports
from vex import *
import cv2
Begin project code
def pre_autonomous():
# actions to do when the program starts
brain.screen.clear_screen()
brain.screen.print("Auton code ( PRE) StandingBy")
wait(1, SECONDS)
def autonomous():
brain.screen.clear_screen()
brain.screen.print("autonomous code")
# Move the robot forward for 30 CM
Right.start(50)  # Adjust the motor speed as needed
Left.start(50)  # Adjust the motor speed as needed
wait(3000)  # Adjust the duration as needed
Stop the robot
Right.stop()
Left.stop()
Do a full slow 360-degree turn
inertial_9.calibrate()
inertial_9.reset_rotation()
Right.start(10)  # Adjust the motor speed as needed
Left.start(-10)  # Adjust the motor speed as needed
while inertial_9.rotation() < 360:
pass
Right.stop()
Left.stop()
Find the closest triball with the vision sensor
closest_triball = None
closest_distance = float('inf')
for obj in vision_10.objects():
if obj.signature == 1 and obj.distance < closest_distance:
closest_triball = obj
closest_distance = obj.distance
Move closer to the triball
while closest_triball and closest_triball.centerX != 50:
error = 50 - closest_triball.centerX
# Adjust the motor speeds based on the error
Right.start(50 + error)
Left.start(50 - error)
closest_triball = vision_10.objects()[0]  # Update the closest triball object
wait(10)
Activate the drivetrain
Right.start(50)  # Adjust the motor speed as needed
Left.start(50)  # Adjust the motor speed as needed
# place automonous code here

def user_control():
brain.screen.clear_screen()
# place driver control in this while loop
while True:
wait(20, MSEC)
create competition instance
comp = Competition(user_control, autonomous)
pre_autonomous()
`

Validated, and working


Added Linux Support to Project

This commit rewrites a few functions and adds linux support to the project. If approved I also ask that you tag this and the Dollar project so that they can both be used on a linux system.

Thanks for the PR and will create a release once this is merged. Can you address my comment https://github.com/ankurp/Cent/pull/18/files/5c19b62077f28212e565122e05b8f053de198889#r84699225

@ankurp In response to your comment, I thought it was causing a separate issue. It turns out to be perfectly fine the way it was.

I squashed your commits and merging it as part of this PR. Will create release soon. https://github.com/ankurp/Cent/pull/19

@Bored0ne This is now published and tagged as version 6.0.2


🐛 BUG: Unable to put an object with a key containing '...' to R2

Which Cloudflare product(s) does this pertain to?
R2
What version of Wrangler are you using?
3.1.1
What operating system are you using?
Linux
Describe the Bug
After I exec wrangler r2 object put 'blog/assets/_...slug_.abcd1234.css' -f ./dist/assets/_...slug_.abcd1234.css, it returned Failed to fetch /accounts//r2/buckets//objects/assets/...slug.abcd1234.css - 403: Forbidden).
But when I changed the key into blog/assets/_.slug_.abcd1234.css, which has no consecutive '.', it just uploaded without any error.
I think it could be a bug of path handling because the original file (whose key contains three consecutive '.') could be uploaded in the R2 webpage.

I think the interpolated strings (specifically objectName) in the endpoint pathname need to be escaped here https://github.com/cloudflare/workers-sdk/blob/c4e6f1565ac6ef38929c72d37ec27d158ec4f4ee/packages/wrangler/src/r2.ts#L111

I assume this is related, but a similar issue happens when a key contains a forwardslash (eg "log/2023-09-21/15:00.json").
The PUT is successful because I can see it in the sqlite database with the correct name.  If I call SELECT * FROM _mf_objects, I get:

log/2023-09-21/15:00.json|67880c83ecd213cd99d85d84b741a73d079ab83996b6ae30250c85f36706615a0000018ab843be1d|b1547edcb91979d2dd5cd2c4916b26a8|1838611|ae7b047273edae8603a505929a60de79|1695308561994|{}|{}|{}

However, when I try to GET the file, it fails.
Call: wrangler r2 object get mybucket/log/2023-09-21/15:00.json --local

⛅ wrangler 3.9.0
Downloading "log/2023-09-21/15:00.json" from "mybucket".
/usr/src/app/node_modules/wrangler/wrangler-dist/cli.js:30947
throw a;
^
Error: ENOENT: no such file or directory, open 'log/2023-09-21/15:00.json'
Emitted 'error' event on WriteStream instance at:
at emitErrorNT (node:internal/streams/destroy:151:8)
at emitErrorCloseNT (node:internal/streams/destroy:116:3)
at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
errno: -2,
code: 'ENOENT',
syscall: 'open',
path: 'log/2023-09-21/15:00.json'
}
Node.js v18.18.0

I think the interpolated strings (specifically objectName) in the endpoint pathname need to be escaped here
https://github.com/cloudflare/workers-sdk/blob/c4e6f1565ac6ef38929c72d37ec27d158ec4f4ee/packages/wrangler/src/r2.ts#L111

@RamIdeas The variables inside the ${} are automatically escaped, so there's no need to manually escape the interpolated strings. Could be we need to stringify the values to make sure variables don't contain any characters that are not allowed in a URL, we could just force it into a string.

I think the interpolated strings (specifically objectName) in the endpoint pathname need to be escaped here
https://github.com/cloudflare/workers-sdk/blob/c4e6f1565ac6ef38929c72d37ec27d158ec4f4ee/packages/wrangler/src/r2.ts#L111

@RamIdeas The variables inside the ${} are automatically escaped, so there's no need to manually escape the interpolated strings. Could be we need to stringify the values to make sure variables don't contain any characters that are not allowed in a URL, we could just force it into a string.

I am pretty confident that there is no escaping going on here. The objectName is not escaped at the time it is passed to cfetch(), which in turn passes it unchanged into unidici's fetch(), which I believe will also not do any escaping?
That being said I don't know if escaping would be a full solution, unless our API is designed to unescape the string when it reads it in... we'd need to check. I guess the fact that it was possible to upload the key via the R2 webpage implies that escaping would be fine.


[FEQ] Jim/FEQ-2490/add transparent red color

Changes

Added the colour #ff444f14 for the primary variant on hover

Screenshots

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

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
223

Relevant Lines:
343

💛  - Coveralls


Show right messages as soon as possible

Flashed messages should be flushed in every single page, otherwise they will be accumulated and shown in the wrong page, making the users extremely confused.
The error messages of API calls should be shown immediately, and should not flash any message, because it will always be shown in the wrong page.

Coverage decreased (-0.2%) to 81.772% when pulling 4ed64750ecd355ea869bad85059a82b6044644a6 on x4base:ajax_error_messages into 04388a7b9b2aff026f46a4412aff48558f9150ec on airbnb:master.

Coverage decreased (-0.03%) to 81.91% when pulling f1ce77e2c715da628657a821eb7c380b66d846a6 on x4base:ajax_error_messages into 04388a7b9b2aff026f46a4412aff48558f9150ec on airbnb:master.

I've just found out it shouldn't be in list.html, otherwise the script will be included multiple times if there are multiple inline list views. For example, in the datasource edit page, there are column list tab and metric list tab, so the script will be included twice. I think it should be in base.html, so I named it common.entry.js

Coverage increased (+0.08%) to 82.019% when pulling 2b92e3a8004472ec15ecfac4d0cde89bba288dc3 on x4base:ajax_error_messages into 04388a7b9b2aff026f46a4412aff48558f9150ec on airbnb:master.


chore(deps): update terraform github.com/terraform-ibm-modules/terraform-ibm-landing-zone to v5.25.2

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-landing-zone
module
patch
v5.25.1 -> v5.25.2

Release Notes

terraform-ibm-modules/terraform-ibm-landing-zone (github.com/terraform-ibm-modules/terraform-ibm-landing-zone)
v5.25.2
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/landing-zone-vpc/ibm to v7.19.0 (#839) (f4b278b)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This issue has been resolved in version 1.4.15 :tada:
The release is available on:

GitHub release
v1.4.15

Your semantic-release bot :package::rocket:


Regulations page

Closes https://ethyca.atlassian.net/browse/PROD-1494
Description Of Changes
https://www.loom.com/share/5a3b8c49537749f59da1ecaa97a16da9?sid=486f6b01-9aea-4eb6-a0f3-16893698f2ed
Code Changes

[ ] list your code changes here

Steps to Confirm

[ ] list any manual steps for reviewers to confirm the changes

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

Looking great @allisonking , just a couple of minor enough points :

For the sub group titles, it looks like we're using the same font size and weight as the main "continent" title. The group title should be smaller so that there's some typographic hierarchy from continent to group to locations. So, all that said, can we make sure we're using 18px for the continent title and 16px for the sub group titles, weight should be semibold.
I've added a recommendation for the modal copy into the design if you have a chance to update that.

Otherwise all good. Thank you.

Thanks @simonkeane33 , I've made those changes! They're actually on the branch this one is branched off of, but I've merged it into this branch as well.


Calling get on Model::newQuery returns generic Model type

Larastan Version: 2.6.3
--level used: 8

Description
When having model assigned to property and then when getting result from newQuery it returns for example collection with generic model type, instead of expected one.
Laravel code where the issue was found
class CSVOrderImportProcessRepository
{
    private CSVOrderImportProcess $model;

    public function __construct()
    {
        $this->model = new CSVOrderImportProcess();
    }

    /** @return Collection<int, CSVOrderImportProcess> */
    public function getByShop(int $shopId): Collection
    {
        return $this->model->setConnection(SecondaryConnection::NEAREST_CONNECTION_NAME)->newQuery()
            ->where(CSVOrderImportProcess::SHOP_ID, $shopId)
            ->get();
    }
}

Here I am getting error like:
 ------ ------------------------------------------------------------------------------------------------------
  26     Method App\DIS\Models\Mongo\CSVOrderImportProcessRepository::getByShop() should return
         Illuminate\Database\Eloquent\Collection<int, App\DIS\Models\Mongo\CSVOrderImportProcess> but returns
         Illuminate\Database\Eloquent\Collection<int, Illuminate\Database\Eloquent\Model>.

What I expect - no error

Thank you for your report.
What is the return type of setConnection?
\PHPStan\dumpType($this->model->setConnection(SecondaryConnection::NEAREST_CONNECTION_NAME));

\PHPStan\dumpType($this->model->setConnection(SecondaryConnection::NEAREST_CONNECTION_NAME));
Dumped type: App\DIS\Models\Mongo\CSVOrderImportProcess
\PHPStan\dumpType($this->model->setConnection(SecondaryConnection::NEAREST_CONNECTION_NAME)->newQuery())
Dumped type: Illuminate\Database\Eloquent\Builder

Looks like the issue is with setConnection if anyone wants to investigate.

@wslawski-printify Can't reproduce your issue.
Can you check https://github.com/sanfair/larastan-playground/pull/3 and point out what I'm missing here?
PHPStan did not found any issues here https://github.com/sanfair/larastan-playground/actions/runs/5655529029

Thank you @sanfair for the reproduction. Closing this as we can't reproduce it. Please open a new issue with more information if this continues to happen.


Added option to split PDF into multiple parts and merge into one PDF

Description
Added the option to merge into one PDF in the function of splitting a PDF into multiple parts to facilitate the work of splitting a single-page PDF in the middle and then merging it.
Closes #(issue_number)
Checklist:

[*] I have read the Contribution Guidelines
[*] I have performed a self-review of my own code
[*] I have commented my code, particularly in hard-to-understand areas
[*] My changes generate no new warnings

Contributor License Agreement
By submitting this pull request, I acknowledge and agree that my contributions will be included in Stirling-PDF and that they can be relicensed in the future under the MPL 2.0 (Mozilla Public License Version 2.0) license.
(This does not change the general open-source nature of Stirling-PDF, simply moving from one license to another license)

What's the difference between this and the one from MergeController? Does it stitch together multiple pages into one (like 1x1+1x1=1x2)?

For example, split a 10-page PDF horizontally and merge it into a 20-page PDF. (like 1x1+1x1=1x2+1x2)
Because for the convenience of printing, some PDFs will merge the original two pages of PDF into one page to facilitate cropping.

Would splitting it into sections and then merging it using the other feature be the same?

Yes, it's the same
For example
example.pdf：
page1: AB
page2: CD
Split Horizontally and merge：
example_output.pdf:
page1: A
page2: B
page3: C
page4: D
If not merged：
example_output.zip:
pdf1,pdf2,pdf3,pdf4

When using horizontal splitting, multiple PDF files will be generated. This function only adds an option to directly merge them into a single PDF file after splitting.
The scenario is: in order to facilitate cropping after printing, the PDF exported by some systems will merge the document that should have been two pages into one page. This way you can fold it in half and cut it directly. However, for other processing needs, these PDF files need to be split horizontally and merged into one file. This feature also appears in other PDF tools.

So this just simplifies a 2 step process into 1 step. I see what you mean now.

Thanks!


Subgroup participants

Identify clusters of similarly behaving individuals so that their data can be treated together. Provide greater personalization by only including behavior of other participants if it is similar to one's own behavior.

Same as #23 and #24


Input Field confused by Bluetooth keyboard

Describe the bug
The Input field works great with the soft keyboard for Hololens 2, but when I have a Bluetooth keyboard connected to the Hololens 2, the input field still activates the soft keyboard as opposed to reading input from just the Bluetooth keyboard. The input field recognizes both the soft keyboard and the bluetooth keyboard at different times, but is unable to prioritize between the two. When I have a bluetooth keyboard connected, I don't need a soft keyboard to pop up. After the input field has been activated and the soft keyboard appears, the soft keyboard can then be closed and the bluetooth keyboard can be used to enter input into the field briefly. I have disabled the Soft keyboard in the unity editor to test, and when I do this, the input field cannot be activated at all on the hololens 2 device, therefore rendering it unable to receive input from the bluetooth keyboard.
To reproduce
Steps to reproduce the behavior:

Run the MRTK Keyboard Test scene on Hololens with a bluetooth keyboard connected to the Hololens.
Select an input field
Type with bluetooth keyboard. For a brief second, the input field will receive input from the bluetooth keyboard, but then the soft keyboard will appear and the input field will cease receiving input from the bluetooth keyboard.
Close the soft keyboard, and continue typing with the bluetooth keyboard.

Expected behavior
If a bluetooth keyboard is connected to the hololens 2, the input field will not spawn a soft keyboard when selected, but instead, just read input from the bluetooth keyboard. However, if there is no bluetooth keyboard connected, the input field will spawn the soft keyboard and read input from it.
Your setup (please complete the following information)

Unity Version [e.g. 2021.2.8f1]

Target platform (please complete the following information)

HoloLens 2

Additional context
I am imagining a scenario where the input field reads input devices and activates the soft keyboard only if there is no bluetooth keyboard connected.

Hi @PennoGT9, after some discussion the team believes the issue you are encountering is a Unity bug as Unity ships the Input Field script. Could you file a bug with Unity by going to Help (in the Unity editor menu bar) → Report a bug... and post a link to the Unity Issue Tracker page once it's created so that the community can get updates on the bug? Thanks!

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


How to play default notification sound

I don't want to add any extra sound file, I just want to play notification sound available with RingtoneManager.TYPE_NOTIFICATION. how should I do this with react-native sound

any update on this?

any update?

Update ??


delete go/vt/concurrency/resource_constraint.go

resource_constraint.go is not used by other packages, delete it.

LGTM


NEW Feature: Ability to save progress and continue from where things were left at last checkpoint

Seeking the ability to go through the address space sequentially and save progress, every so many keys... Allowing to continue from where things were left off. A checkpoint, perhaps.
Curious to know how long it would take to do the whole address space. I think this would be quicker than other methods using CUDA or openCL.
Testing in the cloud I'm getting about.
Starting a random thread!
crackBTC > stats
AES256 block operations<PHONE_NUMBER>232054016/s
Total op<PHONE_NUMBER>160270336, seconds 5
That's 16384 threads.

do you know some cloud computing for free or just very cheap to try? thanks :-)


[Hoyo r4.4 r5.2] push.py 修正, おもにDiscordがmarkdownのheader（#で始まるやつ）に対応していないのでそれの対応

半角空白（スペース）2個挿入により強制的に改行させる

remote: Enumerating objects: 22, done.
remote: Counting objects: 100% (22/22), done.
remote: Compressing objects: 100% (8/8), done.
remote: Total 17 (delta 14), reused 11 (delta 9), pack-reused 0
Unpacking objects: 100% (17/17), 2.40 KiB | 273.00 KiB/s, done.
From github.com:n138-kz/oshiire-bot
   9a29aea..05002d9  main       -> origin/main
Updating 9a29aea..05002d9
Fast-forward
 dev/push.py |  8 ++++++++
 prd/push.py | 12 ++++++++++--
 2 files changed, 18 insertions(+), 2 deletions(-)
[+] Building 1.8s (13/13) FINISHED     docker:default
 => [internal] load build definition from Docke  0.0s
 => => transferring dockerfile: 235B             0.0s
 => [internal] load .dockerignore                0.0s
 => => transferring context: 2B                  0.0s
 => [internal] load metadata for docker.io/libr  1.6s
 => [auth] library/python:pull token for regist  0.0s
 => [internal] load build context                0.0s
 => => transferring context: 11.39kB             0.0s
 => [1/7] FROM docker.io/library/python:3@sha25  0.0s
 => CACHED [2/7] WORKDIR /tmp                    0.0s
 => CACHED [3/7] RUN set -x &&  pip install req  0.0s
 => CACHED [4/7] RUN set -x &&  pip install pyy  0.0s
 => [5/7] COPY push.py /tmp/                     0.0s
 => [6/7] COPY secret.* /tmp/                    0.0s
 => [7/7] COPY images_*.png /tmp/                0.0s
 => exporting to image                           0.0s
 => => exporting layers                          0.0s
 => => writing image sha256:444dcdfd9a0e9d2cec9  0.0s
 => => naming to docker.io/n138-kz/oshiire-bot:  0.0s
[+] Building 0.4s (12/12) FINISHED     docker:default
 => [internal] load .dockerignore                0.0s
 => => transferring context: 2B                  0.0s
 => [internal] load build definition from Docke  0.0s
 => => transferring dockerfile: 235B             0.0s
 => [internal] load metadata for docker.io/libr  0.3s
 => [1/7] FROM docker.io/library/python:3@sha25  0.0s
 => [internal] load build context                0.0s
 => => transferring context: 11.42kB             0.0s
 => CACHED [2/7] WORKDIR /tmp                    0.0s
 => CACHED [3/7] RUN set -x &&  pip install req  0.0s
 => CACHED [4/7] RUN set -x &&  pip install pyy  0.0s
 => [5/7] COPY push.py /tmp/                     0.0s
 => [6/7] COPY secret.* /tmp/                    0.0s
 => [7/7] COPY images_*.png /tmp/                0.0s
 => exporting to image                           0.0s
 => => exporting layers                          0.0s
 => => writing image sha256:04dacf0bac7b870c402  0.0s
 => => naming to docker.io/n138-kz/oshiire-bot:  0.0s
[+] Building 0.3s (12/12) FINISHED     docker:default
 => [internal] load build definition from Docke  0.0s
 => => transferring dockerfile: 235B             0.0s
 => [internal] load .dockerignore                0.0s
 => => transferring context: 2B                  0.0s
 => [internal] load metadata for docker.io/libr  0.3s
 => [1/7] FROM docker.io/library/python:3@sha25  0.0s
 => [internal] load build context                0.0s
 => => transferring context: 94B                 0.0s
 => CACHED [2/7] WORKDIR /tmp                    0.0s
 => CACHED [3/7] RUN set -x &&  pip install req  0.0s
 => CACHED [4/7] RUN set -x &&  pip install pyy  0.0s
 => CACHED [5/7] COPY push.py /tmp/              0.0s
 => CACHED [6/7] COPY secret.* /tmp/             0.0s
 => CACHED [7/7] COPY images_*.png /tmp/         0.0s
 => exporting to image                           0.0s
 => => exporting layers                          0.0s
 => => writing image sha256:04dacf0bac7b870c402  0.0s
 => => naming to docker.io/n138-kz/oshiire-bot   0.0s
$


Ensure that all processes associated with group are terminated

On Unix this involves finding the process group ID of the process
and sending a signal to kill the inverse of the that ID
On windows, this involves using the program TASKKILL and passing
the /T flag to indicate that the entire process tree should terminate.

@mrmeku, wow, great PR! Out of curiosity does this mean that if you SIGKILL the iBazel task that the process group is now linked since you're setting the process group ID?

@mrmeku anything I can do to help this across the finish line?


[BUG] Microsoft Wubi IME (A Chinese IME) appear a bug in a list when using Enter key to clear the input code

Microsoft Wubi IME (A Chinese IME) appear a bug in a list when using Enter key to clear the input code
Steps to reproduce
This is hard to explain, because it concerned with a Chinese IME. To reproduce it, you may need to enable the Wubi IME (五笔输入法) :

Go to windows Settings App
Time & Language
Language
Add a language and choose Chinese (Simplified, China)
In to the Option of Chinese (Simplified, China)
Add a keyboard
add Microsoft Wubi
Into the Option of Microsoft Wubi
Set the Enter Key selection list to Clear Wubi coding

Open Obsidian editor, switch to Microsoft Wubi IME:

type in a then Space Key, the 工 will appear in the editor
type in a then Enter Key, the a code will be cleared

So far, everything is normal.
Then you create a list, like:
- 123
- 456

and put the cursor into the list. Type in a then Enter Key
Expected result
The Wubi code a should be cleared, and the cursor stay at where it was.
Actual result
The a code is not cleared, but saved in the editor, and the cursor moved to the next line!
Of course you can use other Wubi codes, they end the same.
Environment

Operating system: Windows 10

Obsidian version: 0.12.4

Additional information](<### Steps to reproduce
This is hard to explain, because it concerned with a Chinese IME. To reproduce it, you need to enable the Wubi IME (五笔输入法)  in Windows 10:

Go to windows Settings App
Time & Language
Language
Add a language and choose Chinese (Simplified, China)
In to the Option of Chinese (Simplified, China)
Add a keyboard
add Microsoft Wubi
Into the Option of Microsoft Wubi
Set the Enter Key selection list to Clear Wubi coding

Open Obsidian editor, switch to Microsoft Wubi IME:

type in a then Space Key, the 工 will appear in the editor
type in a then Enter Key, the a code will be cleared

So far, everything is normal.
Then you create a list, like:
- 123
- 456

and put the cursor into the list. Type in a then Enter Key
Expected result
The Wubi code a should be cleared, and the cursor stay at where it was.
Actual result
The a code is not cleared, but saved in the editor, and the cursor moved to the next line!
Of course you can use other Wubi codes, they end the same.
Environment

Operating system: Windows 10

Obsidian version: 0.12.4

Additional information

Found out it is caused by the Enhance the Enter Key feature.

@HaujetZhao Thank you for your contribution! Should be fixed in https://github.com/vslinko/obsidian-outliner/releases/tag/1.2.6


Renaming via select breaks preceding sorting

When a field that's used for sorting gets renamed using select, things break:
from x
derive somefield = "something"
sort [somefield]
select [renamed = somefield]

SELECT
  'something' AS renamed
FROM
  x
ORDER BY
  somefield

This will error in the database – the ORDER BY should now be using 'something' instead (or however that derived field is defined – it's probably also referencing other columns):
Query 1 ERROR: ERROR:  column "somefield" does not exist
LINE 6:   somefield
          ^

FYI: managed to change our queries so this isn't blocking us for now, though I suspect it'll sneakily pop up at some point in the future


chore: update consensus version (0.3)

we forgot to update consensus version for rc.0

Afaik we didn't introduce any consensus-breaking changes? Ideally we'd bump on every such change, just like with the API version.

Afaik we didn't introduce any consensus-breaking changes?

I did not introduce any

but u32::MAX needs to be replaced for sure

Oh right, we should probably also fix this on master now that normally everything should be compatible (I wonder why version negotiation even works in compatibility tests rn …).
I think we should just set it all to 2.0, I think any updates we will make in the near future will be "soft forks", so would only bump the minor consensus version. E.g. if we introduced new consensus items that would be 2.1, 2.2, …. Only if we removed stuff that would be breaking and lead to a 3.0 bump.

Oh right, we should probably also fix this on master now that normally everything should be compatible

Oh, I thought that normally we would keep it at INT_MAX on master, just to mark it as incompatible with anything (potentially).

Oh, I thought that normally we would keep it at INT_MAX on master, just to mark it as incompatible with anything (potentially).

That was the reasoning when I didn't think we'd aim for compatibility anytime soon :sweat_smile: I think by now we are good enough at reviewing for breaking changes that

I kind of dislike INT_MAX […] especially the fact that we need to remember about changing it

I'm coming to the same conclusion :laughing:


ÅÄÖ (and equally weird characters, in strings) get borked when formatting code.

From @Southgrove on November 11, 2016 7:46

VSCode Version: Code 1.7.1 (02611b40b24c9df2726ad8b33f5ef5f67ac30b44, 2016-11-03T13:46:00.226Z)
OS Version: Darwin x64 16.0.0
Extensions:

Extension
Author
Version

unity-debug
Unity
1.0.3

Theme-8-Colour-Dark
gerane
0.0.2

Theme-red
gerane
0.0.2

Theme-x3-alpha
gerane
0.0.2

omnisharp
ms-vscode
0.3.3

red
red-auto
0.0.4

Steps to Reproduce:

Copied from original issue: Microsoft/vscode#15332

For what language is that?

From @Southgrove on November 11, 2016 20:43
Oh, sorry. C#.
On Fri, 11 Nov 2016 at 21:29, Martin Aeschlimann<EMAIL_ADDRESS>wrote:

For what language is that?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/15332#issuecomment-260048531,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcyU7OiCWvtw07QEMF7r0r3iNCIvNjcks5q9NBFgaJpZM4KvgTy
.

@Southgrove: Is this still an issue? Do you have any repro steps?

I seem to recall that there was a formatting issue introduced in VS Code awhile back that broke formatting in C# for VS Code as well. I suspect this may have been the issue here.

Ah, sorry I didn't respond. Yes it seems to be fixed.
On Tue, 7 Feb 2017 at 20:46 Dustin Campbell<EMAIL_ADDRESS>wrote:

Closed #916 https://github.com/OmniSharp/omnisharp-vscode/issues/916.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OmniSharp/omnisharp-vscode/issues/916#event-952688063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcyU-MdwFBtZ_hkQSigm5kT1ZdETO3kks5raMoHgaJpZM4KwITU
.

Great -- thanks for confirming!


SPI Slave won't handle CS Interrupt if routed through GPIO Matrix whereas the data lines are on the IO MUX (IDFGH-6768)

There is a bug in the SPI Slave driver, which only occurs if the SPI Slave uses a GPIO port for the CS line which needs to use the GPIO matrix (that means CS GPIO != 15 for ESP32) whereas the other lines (MISO/MOSI/SCLK) use the default lines and do not need the GPIO matrix.
In this case host->flags&SPICOMMON_BUSFLAG_IOMUX_PINS does not get set, as bus_uses_iomux_pins in spi_common.c only returns false if either the SCLK, QUADWP, QUADHP, MOSI or MISO line uses the GPIO matrix.
I would highly suggest not to add the CS line to this check (which would be one solution) as the CS line is not a high-speed / data line and using using the GPIO matrix for all data lines just because the CS is using the GPIO matrix would reduce the maximum performance to the half.
The initial handling of the CS signal in spi_common.c (spicommon_cs_initialize()) works as the check here is performed not against the flag SPICOMMON_BUSFLAG_IOMUX_PINS but instead it is checked if the CS GPIO is the default or needs routing through the GPIO matrix.
So my suggestion (and this push-request) is not to use the function bus_is_iomux() in restore_cs() (as this returns false even though the CS signal needs the GPIO matrix) but instead does the same as is done in spicommon_cs_initialize and check the routing of the CS signal separately in restore_cs() in spi_slave.c (which doesn’t take longer and thus does not increase the execution time of the interrupt handler).

Hi @fm1234 ,
On ESP32, there is a delay if the input signals go through GPIO matrix. For this reason, all input signals should be using/not using IOMUX together. Otherwise there are synchronization issue. For example, due to the delay on CS line, first clock will not be seen and first bit will not be latched.
In practice, if you can ensure the setup/hold time of CS line is long enough, it's not harmful if user want to route data/clock via IOMUX, while CS via GPIO matrix. But this is dangerous, so only expert should do like this. This is the reason why I don't want to merge this PR.
TBH I understand the restore thing will prevent some flexibility. For your requirement, you can use some tricks to do it:

Initialize the driver with GPIO matrix, so that the restore feature will always put CS to the GPIO matrix.
Reconfigure the GPIO matrix input signals of MOSI, CLK to bypass_gpio matrix. This way, those signals will be placed to IOMUX to get higher frequency performance.

Hope this will help you.

Hi Michael,
thank you very much for your comments on this.
I see that there can be a timing issue when mixing the signals between IO MUX and the GPIO Matrix, so my pull request is probably not the best way to do it.
But I think this issue needs to be addressed somehow. Currently when wanting to use the "correct" (IO-Mux) GPIOs for MISO, MOSI and SCLK and only the GPIO for the CS line is a different one, then the SPI slave driver does not work as the restore_cs() function currently tries to route the CS directly over IO MUX bypassing the GPIO Matrix as well.
The problem is that the checks performed in bus_uses_iomux_pins() (spi_common.c line 361 and subsequent) are not performed for the CS line (because the struct spi_slave_t is not passed to bus_uses_iomux_pins as this routine is independent of SPI master/slave). If this would be possible then this function would then (correctly) return false even if only the CS pin is using the GPIO Matrix.
This would lead to the parameter force_gpio_matrix to be set to true when calling spicommon_cs_initialize() (spi_common.c line 570 and subsequent). Then all communication would be done over the GPIO Matrix if one of the involved pins are routed over the GPIO Matrix.
It would involve some architectural (function prototype) changes though....
What do you think?

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

Hi @fm1234
I find now that this issue already fixed in same way, cs pin and signal pins are bound  to check route through iomux or not.
another issue you mentioned that as soon as one device ,,, may not the issue in slave driver.
so if no other issues, you can close this issue. :+1:


MY_ACCESS_TOKEN gives 401 if collected under Firefox

Hey, just wanted to add this here. I kept getting the help message and was thinking somehow my ENV variable wasn't taking, so I put it directly in the code. It was unclear to me that the help message also shows up under a 401 as the error message looks similar to what's being printed in help itself.
I stopped getting an issue once I got the token from Chromium instead (although I believe I ran into another issue because it wanted me to verify by text message and I got blocked by accidentally requesting too many resends, will check back about this if it doesn't resolve).

I updated the script to hopefully make the error message clear by spitting out after the help.  Also tried to make the error that spits out something you could copy paste into an issue so it's easier to debug.  I'm assuming you got this working eventually once you did the text message verification step.  If not, feel free to reopen and paste in your error message.
FWIW the token I got from firefox (my primary browser) works the same for me as the one I got from Chrome.  I'm GUESSING you copied different portions depending on which dev tools you used.


Lack of escaping in ILIKE (mode: 'insensitive' )

Bug description
const email = '%@gmail_com';
prisma.users.findFirst({where: {
    email: {
        equals: email,
        mode: 'insensitive'
    }
}})

Generates  "email" ILIKE $1 , $1 is %@gmail_com
% matches anything or nothing, and _ matches any character.
Same stuff with startsWith/endsWith.
Here is what I wrote to escape these characters:
export function escapeLike(str) {
    return str.replace(/[_%\\]/g, match => '\\' + match)
}

How to reproduce

Expected behavior
No response
Prisma information

Environment & setup

OS: Mac OS
Database: PostgreSQL
Node.js version: 17

Prisma Version
prisma                  : 3.6.0
@prisma/client          : 3.6.0

Thanks for reporting this. I can confirm the behaviour here and I would like to wait for the team for input on whether we should escape this or not.

Any news?

Can't get findFirst nor findUnique working with mode: 'insensitive'
prisma.user.findFirst({ where: { name: { equals: name, mode: "insensitive" } } })

which throws Unknown arg mode in where.name.mode for type StringFilter
findUnique I kinda understand accepts only strings for exact matches, because the uniqueness is only guaranteed per case?
I'd like to be able to get by user.name which is unique but can contain any casing, alternatively I could store a always lower cased version in a separate field and query that instead 🤔


Add benchmarks for initializing Characters from literals.

These benchmarks test both small characters (no more than 63 bits in UTF-8 encoding, packed in an integer) and large characters (more than 63 bits, requiring a retained StringBuffer). They will be submitted first as a baseline against which to compare #6850, which implements performance improvements for initializing characters from literals.

@swift-ci Please benchmark

@swift-ci Please benchmark

@swift-ci Please smoke test

The benchmark job will fail (bug with CI when we add new benchmarks) but can check the log to confirm the benchmark ran, it's just the comparison with previous that fails.

@airspeedswift @shahmishal Perhaps we should add a special benchmark job that just runs the tests. I think we actually might have a preset that does that.

I see no reason why the benchmark job shouldn't just handle newly-added benchmarks.

I agree, the job should handle new tests and just put N/A for the speedup on the new tests.
Benchmark logs show the new tests are running OK so I think this is OK to merge.


Docs don’t line up to my experience

When sharing to a work account from another Office365 subscription the user was unable to signin with its work account, instead it was treated as a user requiring a email code each time.
When sharing with a Microsoft Account, the same as above.
When sharing to a gmail account, the same as above.

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

ID: 6cadc974-5434-4647-6480-996c1e40a2f8
Version Independent ID: 7953c446-e5ac-6a46-4ab4-7acd4b75f3b0
Content: External sharing overview
Content Source: SharePoint/SharePointOnline/external-sharing-overview.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@officedocsbot assign @scanum

@ajhstn Thank you for reaching out and helping improve MS Docs.  I have the team looking into this.  I'll have a reply for you soonest.

@ajhstn Thanks for contacting us. I just recently updated this article and added screen shots. When you share with someone who has a gmail address, the person will always have to use a code. When you share with someone who has a Microsoft account (hotmail.com, live.com, outlook.com), the person should be able to sign in and be added to the directory. When you share with someone who has a work or school account in Office 365 for another organization, it varies. They will get a code the first time. They might be able to sign in after that, or they might have to use a code each time. The team is working on this.

The term "Microsoft Account" is kind of misleading. In my experience sharing without a code is ony possible when using a work/school Office 365 account or with users with at hotmail / live / outlook.com account which is not the same as a Microsoft account since ayone can register for a Microsoft account using his private e-mail address at https://account.microsoft.com.
However user experience for those Microsoft accounts remains same: you need to login with the code and do not get the same functionality as when logged in with the work/school/hotmail account (e.g. setting alerts).
I would suggest to refer to hotmail/live/outlook when using the term Microsoft account in the docs


String-based API plus nice cookie API

In https://github.com/bsittler/async-cookies-api/issues/1#issuecomment-192504735 @bsittler mentioned just a simple API for async setting/getting of the cookie string, like document.cookie except it can be used in workers and is async.
We chatted about this a bit and then he had a brilliant idea: "why not both?"
That is: we expand this current proposal with promise-returning getAsString() and setAsString(s) objects, which do exactly what document.cookie does, just async and works-in-workers. UAs could implement this small subset almost immediately! In the meantime, we could work on making a nicer cookie API, like this repo already has, as additional methods (get/getAll/set/etc.).
This also neatly resolves #7. We can make the nice API a bit more restrictive, with the escape hatch of {get,set}AsString to take care of the crazy esoteric cases like no-value cookies for anyone who really needs them.

Better bikeshed color: readAsString() and writeAsString() to avoid confusion with get = "get from the multimap by key" and similarly for set.

readAllAsString().then(jar => { ...}) perhaps?

What is jar in that example? A string or a CookieJar?

A string, exactly as from a (synchronous) evaluation of document.cookie in rvalue context.

Specifically, a string like "NAME1=VALUE1; NAME2=VALUE2; NAME1=VALUE3; ... etc." exactly as would be seen in the HTTP "Cookie" header, but with HttpOnly pairs filtered out.

OK cool. Yeah that name seems reasonable. I just got confused by your variable name.

What are the use cases for this? Is there anything script can (or should) do with this API other than wrap it with a custom (i.e. fragile/buggy...) parser/serializer?
It would make for easier interop with existing code (i.e. custom cookie APIs with parsers/serializers)  and be easy for UAs to implement, but IMHO those - on their own - don't seem to justify the inclusion.
I could imagine a use case for getting/setting the whole shebang at once, and/or having a pre-serialized form (e.g. to store/send elsewhere?) but I haven't heard of those. Are those real or just my fevered imagination?

I think it's mainly useful as a "minimum viable API" for access to cookies from workers (and other async contexts where blocking on any sort of cookie mutex/disk access is unacceptable.) It also directly exposes the only cookie abstraction currently visible to scripts and web servers, so library and framework support for this should be a lot less difficult than adapting to a completely new higher-level API with its own abstractions.
Finally, a higher-level API can be implemented in terms of this one, so it may be possible to reach implementer consensus early on the "minimum" API with broad availability, then iterate rapidly on polyfills of the higher-level API separately without blocking those worker-based web applications which are in dire need of cookie access today.

After further though I think the Set-Cookie/Cookie-like API presented by document.cookie is bad enough (especially in terms of defaults) that I do not want to perpetuate it. So instead the new API proposed in the explainer only offers a higher-level abstraction. I would like to see how far we get with that, and whether an escape-hatch is actually needed, before committing to maintaining one indefinitely as part of the new API. What do you think?

It sounds reasonable, but we really need to be sure that all applications can migrate to the new API.

I mostly agree. I think the vast majority of applications need to work with
the new API without any major change to how they use cookies.
However, I'm sure there are unusual edge cases where an application only
works in a subset of modern browsers, or uses and relies on differences
cookie behavior in different browsers, and I'm not convinced those need to
keep working without modifications.
On Wed, Jul 27, 2016, 00:05 Anne van Kesteren<EMAIL_ADDRESS>wrote:

It sounds reasonable, but we really need to be sure that all applications
can migrate to the new API.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bsittler/async-cookies-api/issues/8#issuecomment-235503878,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD3R3eM_tkbaPhX_AQ02nQpVdnzaFNEks5qZwMzgaJpZM4HpzB2
.

Yeah, on reflection it's worth starting out small (as I said elsewhere in the past I noticed), but make sure we can grow to the full feature set which is always possible by adopting the string-based document.cookie elsewhere.

Fixed explainer link

String-based API (class AsyncCookieJar_) is there now and used internally in CookieStore but is not exported. Re-open if you think it should be exported (and explain why, please!)


Edited .html to reflect basic framework and containers on the main ap…

…p page where API fetches will be displayed to. Created submission form to be used as well.

You are so good!


[WIP] Add Facebook and Twitter logins

Fixes #44  #45
Proposed Changes

Implementing login with Facebook
Implementing login with Twitter

Pull Request Test Coverage Report for Build 29

0 of 2 (0.0%) changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.8%) to 65.517%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/spender_web/controllers/auth_controller.ex
0
1
0.0%

lib/spender_web/controllers/welcome_controller.ex
0
1
0.0%

Totals

Change from base Build 27:
-0.8%

Covered Lines:
57

Relevant Lines:
87

💛  - Coveralls


drivername not found or defined

I don't know if this is a config issue or a bug at this point.
Flast-SQLAlchemy is used as part of mead project.
** My setup is in a virtual environment:
** python version
(mead_dev)seth@dharma:~/Code/python/mead_dev/mead$ python -V
Python 2.7.2+
** Please take a look at the stack dump below.  Object name seem not to be defined.
Looked @ the code and don't have much context.
_EngineConnector.get_uri() does some binding, which fails to be defined properly and is not 'sane'.
Hence an invalid uri is returned.  Also digging further it seems that the binding to a db-engine is not defined.  So maybe this is a config issues?
** Maybe you can shed some light on this?
[stack trace]
(mead_dev)seth@dharma:~/Code/python/mead_dev/mead$ ./run_server.py
Traceback (most recent call last):
File "./run_server.py", line 4, in
from mead import app
File "/home/seth/Code/python/mead_dev/mead/mead/init.py", line 66, in
import mead.core.views.content
File "/home/seth/Code/python/mead_dev/mead/mead/core/views/content.py", line 10, in
from available import defaults
File "/home/seth/Code/python/mead_dev/mead/mead/core/views/available.py", line 22, in
defaults.update(dict(pages = Page.query.all()))
File "/home/seth/Code/python/mead_dev/local/lib/python2.7/site-packages/Flask_SQLAlchemy-0.15-py2.7.egg/flaskext/sqlalchemy.py", line 386, in get
return type.query_class(mapper, session=self.sa.session())
File "/home/seth/Code/python/mead_dev/local/lib/python2.7/site-packages/SQLAlchemy-0.7.3-py2.7.egg/sqlalchemy/orm/scoping.py", line 53, in call
return self.registry()
File "/home/seth/Code/python/mead_dev/local/lib/python2.7/site-packages/SQLAlchemy-0.7.3-py2.7.egg/sqlalchemy/util/_collections.py", line 877, in call
val = self.registry.value = self.createfunc()
File "/home/seth/Code/python/mead_dev/local/lib/python2.7/site-packages/Flask_SQLAlchemy-0.15-py2.7.egg/flaskext/sqlalchemy.py", line 191, in init
bind=db.engine,
File "/home/seth/Code/python/mead_dev/local/lib/python2.7/site-packages/Flask_SQLAlchemy-0.15-py2.7.egg/flaskext/sqlalchemy.py", line 719, in engine
return self.get_engine(self.get_app())
File "/home/seth/Code/python/mead_dev/local/lib/python2.7/site-packages/Flask_SQLAlchemy-0.15-py2.7.egg/flaskext/sqlalchemy.py", line 736, in get_engine
return connector.get_engine()
File "/home/seth/Code/python/mead_dev/local/lib/python2.7/site-packages/Flask_SQLAlchemy-0.15-py2.7.egg/flaskext/sqlalchemy.py", line 428, in get_engine
self._sa.apply_driver_hacks(self._app, info, options)
File "/home/seth/Code/python/mead_dev/local/lib/python2.7/site-packages/Flask_SQLAlchemy-0.15-py2.7.egg/flaskext/sqlalchemy.py", line 679, in apply_driver_hacks
if info.drivername == 'mysql':
AttributeError: 'NoneType' object has no attribute 'drivername'

Your database connection is not defined, or, if it is defined, you don't have the appropriate database 'driver' defined.

This is not an issue.. it's a config problem on my side.
SQLAlchemy needs a uri for db to bind to.

Why does init_app(my_app) leave self._app as None ... and why did you change self.app to self._app to thwart all the people who were doing it manually?

@joranbeasley ummm? This issue is 6 years old...

lol ignore me :P its my fault :)

hey i have the same problem how did you solve it a detail explabation would do
Thanks