chore: give verbose names to form args "e"

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

This update does not quite change or add any functionality, but aims at improving the readability of the form.ts file from the default create-svelte app. I noticed e was used as the event argument, and then later in the same handler as an error variable. To me this was a bit confusing to read and it does not appear to hold significant value with a shorter variable name. For newcomers this can be off-putting or confusing reading multiple variables with the same name in the same function (despite block scoping)
Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

thanks 👍


Log the current replication lag every 30 seconds

Logging replication lag is very useful for determining how mongo-connector is performing.
This change adds a thread per replica set that periodically logs the current replication lag (either in seconds or number of oplog entries).
If the current thread is up to date the log message will be:
2016-09-30 15:26:46,616 [INFO] mongo_connector.oplog_manager:69 - OplogThread for replica set 'source' is up to date with the oplog.
If the current thread is behind the log message will be:
2016-09-30 15:18:45,114 [INFO] mongo_connector.oplog_manager:59 - OplogThread for replica set 'source' is 81 seconds behind the oplog.
or:
2016-09-30 15:18:15,014 [INFO] mongo_connector.oplog_manager:65 - OplogThread for replica set 'source' is 36 entries behind the oplog.
Possible improvements:

Make the logging interval a configuration option.
Clear up the logging statements.
Add more info: include the raw timestamps?

great! It will really help to understand what's going on and what's the status of mongo-connector.


Various improvements/refactoring to modernize the project

This should contain all the content from #20 rebased to thematic commits. Extensive discussion is captured on the #20 PR page.

Hey @vvasuki, am I good to merge this?

Sure - @rramphal can take @psvenk 's approval as sufficient condition for merging and vice versa.


Nginx config

Added a sample nginx proxy configuration with instructions for hosting HTTPS over port 443. #901 inspired me to solidify my config which is basically this. Using this configuration yields an A+ with SSLLabs. Yay for needless metrics!

What's with all the merge commits? Try rebasing off dev. Not perse needed but a good practice of git Foo.
Does streaming events work? I've had some reports of people having caching turned on and it broke streaming

I'll fix the commit history. Also, yes event streaming does work. It is the proxy_buffering that caches the responses from HA into a buffer. This is what breaks HA if it is enabled.

I think I may have forked up my fork. I'll close my PRs and reopen.


✨ Add example1 variant with single git clone

Summary
This PR introduces example1-local.md that, when executed, uses the local content rather than fetching from github.  Thus, this executable doc can be used for CI and can be used by a developer to test before pushing and even before committing.
Related issue(s)
Making all the executable docs work like this would resolve #720

Agreed with @clubanderson that the better approach is to change the existing documents to not do nested fetches.


Use add-zle-hook-widget zle-line-pre-redraw

Fixes #245, q.v..
Should be rebased/squashed before merging.

Merging onto current master (51614ca2c994486dc0a41ec0737a97f0af16a505), the three tests trigger the following warnings:

highlighters/brackets/test-data/cursor-matchingbracket.zsh
highlighters/main/test-data/alias-assignment1.zsh
highlighters/pattern/test-data/rm-rf.zsh

% make -s quiet-test ZSH=$prefix/zsh/bin/zsh
Running test brackets
add-zle-hook-widget:21: job table full or recursion limit exceeded
_zsh_highlight:39: command not found: _zsh_highlight_highlighter_brackets_predicate
Bail out! output on stderr
# cursor-matchingbracket
1..1
not ok 1 - [9,9] «)» - expected "cursor-matchingbracket", observed "".
Running test main
add-zle-hook-widget:21: job table full or recursion limit exceeded
_zsh_highlight:39: command not found: _zsh_highlight_highlighter_main_predicate
Bail out! output on stderr
# alias-assignment1
1..1
not ok 1 - [1,3] «x=y» - expected "unknown-token", observed "".
Running test pattern
add-zle-hook-widget:21: job table full or recursion limit exceeded
_zsh_highlight:39: command not found: _zsh_highlight_highlighter_pattern_predicate
Bail out! output on stderr
# rm-rf
1..1
not ok 1 - [1,8] «rm -rf /» - expected "fg=white,bold,bg=red", observed "".
Makefile:38: recipe for target 'test' failed
make[1]: *** [test] Error 2
Makefile:49: recipe for target 'quiet-test' failed
make: *** [quiet-test] Error 2
zsh: exit 2     make -s quiet-test ZSH=$prefix/zsh/bin/zsh

Adding zmodload zsh/zle to the top of tests/test-highlighting.zsh resolves this.  This appears to be an infinite loop in upstream's add-zle-hook-widget; will report upstream.

This appears to be an infinite loop in upstream's add-zle-hook-widget; will report upstream.

workers/39128

Upstream will fix this; however, a fix on this end is needed too:
diff --git tests/test-highlighting.zsh tests/test-highlighting.zsh
index 1d169d6..d164590 100755
--- tests/test-highlighting.zsh
+++ tests/test-highlighting.zsh
@@ -47,6 +47,9 @@
   exit 2
 }

+# Required for add-zle-hook-widget.
+zmodload zsh/zle
+
 # Load the main script.
 . ${0:h:h}/zsh-syntax-highlighting.zsh

diff --git zsh-syntax-highlighting.zsh zsh-syntax-highlighting.zsh
index 1c3e8e4..035f3ac 100644
--- zsh-syntax-highlighting.zsh
+++ zsh-syntax-highlighting.zsh
@@ -330,8 +330,8 @@ if (( $zsh_highlight_use_redrawhook )); then
     } "$@"
   }
   _zsh_highlight_bind_widgets(){}
-  add-zle-hook-widget zle-line-pre-redraw _zsh_highlight
-  add-zle-hook-widget zle-line-finish _zsh_highlight__zle-line-finish
+  add-zle-hook-widget zle-line-pre-redraw _zsh_highlight || return 42
+  add-zle-hook-widget zle-line-finish _zsh_highlight__zle-line-finish || return 42
 fi

 # Load highlighters from directory.

Will flesh this out and add it to the branch.

(github doesn't point out 2cf6a48, which was a sync-merge from master.)

(I can't edit the log messages of earlier commits on the branch, so remarking here instead for posterity:)  The reference to .match-bracket in ef41c9232d62e1ae7cf2cadb9ce8a6372341dddf is a bug.  That test was meant to detect zsh≥5.3, but it was incorrect since that widget name never existed in zsh.

Merged and pushed to the feature/redrawhook branch.  Posted a call for testing to<EMAIL_ADDRESS>

(This issue should be reopened.)

Also fixes #507.

https://travis-ci.org/zsh-users/zsh-syntax-highlighting/builds/441045121
Seems that 4.3.11 and 4.3.12 do not support passing arguments to an anonymous function. Perhaps we should drop the "$@"in _zsh_highlight__zle-line-finish() (since they aren't used anyway) to retain compatibility to 4.3.11.

https://travis-ci.org/zsh-users/zsh-syntax-highlighting/builds/441045121
Seems that 4.3.11 and 4.3.12 do not support passing arguments to an
anonymous function. Perhaps we should drop the "$@"in
_zsh_highlight__zle-line-finish() (since they aren't used anyway) to
retain compatibility to 4.3.11.

"aren't used"?  Couldn't someone's zshrc be calling zle zle-line-finish -- foo bar baz from some custom widget or another?
Formally we require 4.3.17+ since before I had commit access, but I also remember a bug report from a 4.3.9 user.
The next version, 4.3.13, was released on 2011-11-30, i.e., seven years ago.
How about doing here what we do with grep -q elsewhere: use the new idiom, and revert to the old one if someone complains that their system doesn't support the new idiom?  Or we could just name the anonymous function.

Summary of IRC:
:A is in 4.3.10 (thanks @nicoulaj), so supporting 4.3.11+ shouldn't be an issue there. I'm a fan of keeping support for 4.3.11 until CentOS 6 is no longer supported (Nov 2020) (Debian wheezy is on 4.3.17 and seems to already be EOL).
Since _zsh_highlight does not use its arguments, it should be safe to drop "$@" from the anonymous function and the _zsh_highlight invocation.


why helm can't be delete?

use "kubectl delete deployment tiller-deploy --namespace kube-system, or more concisely helm reset"
but tiller service and pod still exist, so why?
Is there other method to delete or uninstall helm?

I can't reproduce this on v2.3.1:
><> helm init
$HELM_HOME has been configured at /home/bacongobbler/.helm.

Tiller (the helm server side component) has been installed into your Kubernetes Cluster.
Happy Helming!
><> helm version
Client: &version.Version{SemVer:"v2.3.1", GitCommit:"32562a3040bb5ca690339b9840b6f60f8ce25da4", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.3.1", GitCommit:"32562a3040bb5ca690339b9840b6f60f8ce25da4", GitTreeState:"clean"}
><> helm reset
Tiller (the helm server side component) has been uninstalled from your Kubernetes Cluster.
><> kk get po
NAME                             READY     STATUS        RESTARTS   AGE
kube-addon-manager-minikube      1/1       Running       0          17m
kube-dns-v20-4b0ll               3/3       Running       0          17m
kubernetes-dashboard-xs3qx       1/1       Running       0          17m
tiller-deploy-1332173772-19hzx   0/1       Terminating   0          1m

The pod goes into Terminating status and is eventually removed from the cluster.
Can you provide more backstory/logs on how we can reproduce your issue? It's not clear to me what the issue is (there is no tiller service deployed on helm init).

my helm is abnormal, when I reset it, can't remove:
kubectl get pods --namespace kube-system
NAME                             READY     STATUS             RESTARTS   AGE
tiller-deploy-1332173772-jjnb8   0/1       ImagePullBackOff   0          54m

I'm guessing this is related to #2309 then? Once that issue is resolved you should be able to run helm reset, though I'm curious if there's a reason we can't just make an API call to delete the deployment even if we can't connect to it.

now it can be deleted with kubectl cmd, but it is can't yesterday, thx very much:)

issue occurs again:
root@ubuntu:/home/zte# kubectl delete deployment tiller-deploy --namespace kube-system
deployment "tiller-deploy" deleted
root@ubuntu:/home/zte# kubectl get service --namespace kube-system
NAME            CLUSTER-IP   EXTERNAL-IP   PORT(S)     AGE
tiller-deploy   <IP_ADDRESS>           44134/TCP   21h
root@ubuntu:/home/zte# helm reset
Error: could not find a ready tiller pod
root@ubuntu:/home/zte# kubectl get service --namespace kube-system
NAME            CLUSTER-IP   EXTERNAL-IP   PORT(S)     AGE
tiller-deploy   <IP_ADDRESS>           44134/TCP   21h

I would phrase it:
helm delete && helm reset do not delete a non-finished deployment (while IMHO it should).
I expect that helm delete && helm reset would do the equivalent of:
kubectl delete deployment --namespace=kube-system tiller-deploy, but in the event of non-completed/stuck deployment it doesn't/
[root@machine0 ~]# kubectl get -o wide --namespace kube-system pods | tail -n +2
etcd-machine0                      1/1       Running             0          26m       <IP_ADDRESS>   machine0
kube-apiserver-machine0            1/1       Running             0          26m       <IP_ADDRESS>   machine0
kube-controller-manager-machine0   1/1       Running             0          26m       <IP_ADDRESS>   machine0
kube-dns-2245654080-nhq7l            3/3       Running             0          26m       <IP_ADDRESS>     machine0
kube-flannel-ds-27wzd                2/2       Running             0          26m       <IP_ADDRESS>   machine0
kube-flannel-ds-kjk3x                2/2       Running             0          25m       <IP_ADDRESS>   machine1
kube-flannel-ds-kkm60                2/2       Running             0          25m       <IP_ADDRESS>   machine2
kube-proxy-4ms64                     1/1       Running             0          25m       <IP_ADDRESS>   machine2
kube-proxy-h7152                     1/1       Running             0          25m       <IP_ADDRESS>   machine1
kube-proxy-tdlbr                     1/1       Running             0          26m       <IP_ADDRESS>   machine0
kube-scheduler-machine0            1/1       Running             0          26m       <IP_ADDRESS>   machine0
tiller-deploy-1332173772-rwrnw       0/1       ContainerCreating   0          4s        <none>          machine1
[root@machine0 ~]# helm reset
Error: could not find a ready tiller pod
[root@machine0 ~]# helm delete
Error: could not find a ready tiller pod
[root@machine0 ~]# kubectl get -o wide --namespace kube-system pods | tail -n +2
etcd-machine0                      1/1       Running             0          27m       <IP_ADDRESS>   machine0
kube-apiserver-machine0            1/1       Running             0          27m       <IP_ADDRESS>   machine0
kube-controller-manager-machine0   1/1       Running             0          27m       <IP_ADDRESS>   machine0
kube-dns-2245654080-nhq7l            3/3       Running             0          26m       <IP_ADDRESS>     machine0
kube-flannel-ds-27wzd                2/2       Running             0          26m       <IP_ADDRESS>   machine0
kube-flannel-ds-kjk3x                2/2       Running             0          25m       <IP_ADDRESS>   machine1
kube-flannel-ds-kkm60                2/2       Running             0          25m       <IP_ADDRESS>   machine2
kube-proxy-4ms64                     1/1       Running             0          25m       <IP_ADDRESS>   machine2
kube-proxy-h7152                     1/1       Running             0          25m       <IP_ADDRESS>   machine1
kube-proxy-tdlbr                     1/1       Running             0          26m       <IP_ADDRESS>   machine0
kube-scheduler-machine0            1/1       Running             0          26m       <IP_ADDRESS>   machine0
tiller-deploy-1332173772-rwrnw       0/1       ContainerCreating   0          20s       <none>          machine1
[root@machine0 ~]# kubectl delete deployment --namespace=kube-system tiller-deploy
deployment "tiller-deploy" deleted
[root@machine0 ~]# kubectl get -o wide --namespace kube-system pods | tail -n +2
etcd-machine0                      1/1       Running       0          27m       <IP_ADDRESS>   machine0
kube-apiserver-machine0            1/1       Running       0          27m       <IP_ADDRESS>   machine0
kube-controller-manager-machine0   1/1       Running       0          27m       <IP_ADDRESS>   machine0
kube-dns-2245654080-nhq7l            3/3       Running       0          26m       <IP_ADDRESS>     machine0
kube-flannel-ds-27wzd                2/2       Running       0          26m       <IP_ADDRESS>   machine0
kube-flannel-ds-kjk3x                2/2       Running       0          25m       <IP_ADDRESS>   machine1
kube-flannel-ds-kkm60                2/2       Running       0          25m       <IP_ADDRESS>   machine2
kube-proxy-4ms64                     1/1       Running       0          25m       <IP_ADDRESS>   machine2
kube-proxy-h7152                     1/1       Running       0          25m       <IP_ADDRESS>   machine1
kube-proxy-tdlbr                     1/1       Running       0          26m       <IP_ADDRESS>   machine0
kube-scheduler-machine0            1/1       Running       0          27m       <IP_ADDRESS>   machine0
tiller-deploy-1332173772-rwrnw       0/1       Terminating   0          33s       <none>          machine1

Thank you for articulating what I was trying to point out, @ReSearchITEng. That's exactly what I was trying to say. :)

with "kubectl delete deployment", the pod is still terminating after several minutes.

The deletion logic should get fixed by 2.4 simply because we switched to the newer Kubernetes library.


Downgrade "reference to captured var" to warn in preconcurrency code.

Fixes rdar://97228088.

@swift-ci please smoke test

Spurious failure, trying again

@swift-ci please smoke test macOS


Resizing of thumbs in tilesGrid

Hi,
I try to search in documentation, but I did not find it.
I have set gallery_width and height to 100%, which is OK, but thumbnails are scaled down to 180 x 150px.
How I can change width and height of tilesgrid thumbnails (my source images have 250 x 208px) ?

HI. please show me the gallery so I could look. I can't understand what you
want to do
On Mon, Feb 8, 2016 at 5:02 AM, Peter74<EMAIL_ADDRESS>wrote:

Hi,
I try to search in documentation, but I did not find it
I have set gallery_width and height to 100%, which is OK, but thumbnails
are scaled down
How I can change width and height of tilesgrid thumbnails?
—
Reply to this email directly or view it on GitHub
https://github.com/vvvmax/unitegallery/issues/56.

Hi,
I send you e-mail with link.

Hi, please did you receive e-mail from me?

..now i see :) you include options in each example - that is really useful and solve this issue for me.
Really thanks!


Remove unnecessary and erroneous case in LineParser

@viktorklang misinterpreted the previous implementation – which indeed is much harder to comprehend than the Klangonian one – of parseLines and added an unnecessary case:
case CR if buffer(math.max(at - 1, 0)) == LF => // Ignore first CR after a CRLF sequence
  parseLines(buffer, at + 1, at + 1, parsedLines)

Actually a CR after a CRLF is absolutely valid – even if it doesn't make a lot of sense for the domain – and hence the above case not only unnecessary but even erroneous.

That makes for an even simpler implementation :)
--
Cheers,
√
On Jan 27, 2016 6:09 PM, "Heiko Seeberger"<EMAIL_ADDRESS>wrote:

@viktorklang https://github.com/viktorklang misinterpreted the previous
implementation – which indeed is much harder to comprehend than the
Klangonian one – of parseLines and added an unnecessary case:
case CR if buffer(math.max(at - 1, 0)) == LF => // Ignore first CR after a CRLF sequence
parseLines(buffer, at + 1, at + 1, parsedLines)
Actually a CR after a CRLF is absolutely valid – even if it doesn't make a
lot of sense for the domain – and hence the above case not only unnecessary
but even erroneous.
—
Reply to this email directly or view it on GitHub
https://github.com/hseeberger/akka-sse/issues/78.


Move the verification code into a lazy-loaded file

This will help improve load time for the library while reducing load-time side-effects.  In particular, this will avoid forcing ActionController::Base to load in Rails applications during initialization.
Proposed as a potential solution to #880.

@pvande Apologies I didn't see this PR sooner.
https://github.com/rollbar/rollbar-gem/pull/882 includes updated rspec/travis for Rails 6.x, and since the reported issue has behavior specific to Rails 6.x, I wanted to get those updates in as well.
Thank you for the work on this, especially the very complete analysis.

@waltjones No worries!  I just figured that since I'd already done the legwork I might as well put together a PR too.  Cheers!


Conflict with html2canvas - ReferenceError: CanvasRenderer is not defined

Using version "1.4.1"
html2canvas version v1.0.0-alpha.12
When both libraries are connected, the html2canvas library breaks:
ReferenceError: CanvasRenderer is not defined
The cause of conflict - jsPDF library.

Hmmm. Probably because jsPDF 1.4.1 contains the old html2canvas. But in the next release it will not be integrated into jsPDF. So yeah, probably will be solved soon.

not reproducable, probably will be solved in next release.


[Bug] getCheckoutInfoQuery error when converting into the Checkout model.

When a checkout info query is made on a completed order, the Gql returns an 'Order' typename within the Checkout typename like below. (I obscured the key)
order: {__typename: Order, id: gid://shopify/Order/5848740561236?key=20397cf872......e1075132807}
This is causing the Checkout.fromJson to throw an error, which I stacked traced below:
stackTrace: #0      _$$_OrderFromJson
order.g.dart:11
#1      new _$_Order.fromJson
order.freezed.dart:481
#2      _$OrderFromJson
order.freezed.dart:18
#3      new Order.fromJson
order.dart:68
#4      _$$_CheckoutFromJson
checkout.g.dart:33
#5      new _$_Checkout.fromJson
checkout.freezed.dart:515
#6      _$CheckoutFromJson
checkout.freezed.dart:18
#7      new Checkout.fromJson
checkout.dart:48
#8      ShopifyCheckout.getCheckoutInfoQuery
shopify_checkout.dart:77
I believe it is looking for a full Order object, whereas the Gql document returns just the id within the Order typename.

Are you using your own custom gql?
if not, getCheckoutInfoQuery result always contains the Order object or null.

Closing the issue since there is no update from @jschaeken


Reference instance swap components by name

Currently, "instance swap" properties are referenced as "links" to other components. It would be great to be able to reference them by name, and get a string value when mapping back to the React component. Ideally, we should also be able to manipulate the strings when mapping (for example, to convert from snake_case to camelCase).

For icons for example, we don't have a React component for each individual icon. Instead, we use a single <Icon> component, and we use a string prop to reference icons by id.
<Icon id="icChef" />

When connecting this to the Figma component, we get a "link" to the instance-swap icon, but we don't have a connected React component for every icon in Figma.

figma.connect(
  Icon,
  "https://www.figma.com/file/...[Icon component with an instance swap]",
  {
    props: {
      iconId: figma.instance("Icon ID"),
    },
    example: (props) => <Icon id={props.iconId} />
  },
);

So ideally, we should be able to do something like this, where the instance has a name attribute we can access.
figma.connect(
  Icon,
  "https://www.figma.com/file/...[Icon component]",
  {
    props: {
      iconId: figma.instance("Icon ID"),
    },
    example: (props) => <Icon id={props.iconId.name} />
  },
);

Additionally, with a custom mapper function, we could also manipulate the instance at will:
    props: {
      // just get the `name` as a string
      iconId: figma.instance("Icon ID", (instance) => instance.name),
    },
    example: (props) => <Icon id={props.iconId} />

or even change it how we see fit:
  iconId: figma.instance("Icon ID", (instance) => toCamelCase(instance.name)),

Hey @andreiduca, so right now the way we suggest you solve this is that you create a connection for every icon. You can have multiple connections for the same React component, so you could have a file icons.figma.tsx with something like:
// Chef icon
figma.connect(Icon, "https://figma.com/file/123?node_id=1:1", {
  example: () => <Icon id="icChef" />
}

// Plate icon
figma.connect(Icon, "https://figma.com/file/123?node_id=1:2", {
  example: () => <Icon id="icPlate" />
}

// Fork icon
figma.connect(Icon, "https://figma.com/file/123?node_id=1:3", {
  example: () => <Icon id="icFork" />
}

// etc...

We talk a bit about this in the README, although I think we could probably make this clearer and show some examples.
There's a template script which you can use a basis for automatically generate such a file from Figma. If you have any questions or comments about implementing this, please let us know and we'd be happy to help out.
That said, we have also talked about concepts like you describe, where e.g. strings can be manipulated. Right now, we don't have concrete plans on whether we'll support this or when, but your input is really helpful and I'll share it with the team for discussion. We might follow up with further questions if we do look at implementing something like this.

Connecting all individual icon components in Figma is exactly what I am trying to avoid. It's a scalability issue to have to "connect" every time a new icon gets added to the library.
In Figma, we have 250+ icons as individual components. So we created a separate component named Icon as a wrapper around them, which uses an instance swap named Icon ID to swap the actual icon component being rendered inside.
Similarly, in React, we only have a single <Icon> component, and we reference individual icons by a string id prop.
We can easily connect these two, only once, since the APIs are identical, and then it shouldn't matter how many new icons get added to the library – the "connection" remains the same: an Icon component on both sides with an icon-id property, which is an instance swap in Figma, but a string in React.
So mapping the swap instance to a string, like I tried to explain in my original comment, and referencing the swapped instance by its name should solve this problem.

I got similar case with passing component as an icon. Example:
import { IconPlay } from 'icon-library'
import { BrandIcon } from 'ui-library'

function BrandIcon({ icon: Icon }) {
  return <Icon className="brand-icon" />
}

function View() {
  return (
    <div>
      <BrandIcon icon={IconPlay} /> Title
    </div>
  )
}

In this example I use icon for dependency injection. Of course I could pass icon={<IconPlay />}, but in this case I need to use cloneElement (what is not common case https://react.dev/reference/react/cloneElement) or prop render function. But it's more complex in this case. Passing component itself instead of instance is a common case for UI libraries:

component props in MUI https://mui.com/material-ui/api/button/#button-prop-component
elementType in react-aria https://react-spectrum.adobe.com/react-aria/useButton.html#api
asChild in @radix-ui https://www.radix-ui.com/primitives/docs/components/tabs#root
as prop in Chakra UI https://v2.chakra-ui.com/docs/components/box/usage#usage

Hey @andreiduca, we've just released some improvements in v1.2.0 that make it easier to work with icons. We do still have the limitation of requiring each icon to be individually connected (which can be done via the script); right now this is the only way to ensure that your icons are using the correct names that match what you have in code. We're planning to add improved ways of automating Code Connect for icons and keeping them in sync via the CLI, which we're hoping will make this more maintainable.
You can now return a string from a connected component, meaning that string can be used downstream in props to other components:

figma.connect("icon", {
  example: () => "my-icon"
})

// `icon` will render a string instead of a JSX element
figma.connect("parent", {
  props: {
    icon: figma.instance<string>("IconInstance")
  },
  example: ({ icon }) => <Box icon={icon} />
})

More info in the docs here: https://github.com/figma/code-connect/blob/main/docs/react.md#icons


Failed to exec method /usr/lib/apt/methods/http

Installing Debian packages via rosdep install fails (reminder: we are using the QEMU hack)
Failed to exec method /usr/lib/apt/methods/http
0 upgraded, 1 newly installed, 0 to remove and 16 not upgraded.
E: Method http has died unexpectedly!
E: Sub-process http returned an error code (100)
E: Method /usr/lib/apt/methods/http did not start correctly
ERROR: the following rosdeps failed to install
  apt: command [sudo -H apt-get install -y python-mock] failed

As always, installing Debian packages directly works just fine, so the temporary solution is to manually install all rosdeps:
# HACK: rosdep would fail to install these on its own using the QEMU hack
RUN apt-get install -yq --no-install-recommends \
    python-yaml python-mock python-empy python-netifaces python-numpy \
    python-imaging python-nose python-coverage python-paramiko \
    cmake \
    libboost-all-dev libtinyxml-dev sbcl liblz4-dev libapr1-dev \
    libconsole-bridge-dev libaprutil1-dev libgtest-dev liblog4cxx10-dev

I was able to replicate the issue without the involvement of rosdep, just apt-get install:
Failed to exec method /usr/lib/apt/methods/http
0 upgraded, 147 newly installed, 0 to remove and 20 not upgraded.
E: Method http has died unexpectedly!
E: Sub-process http returned an error code (100)
E: Method /usr/lib/apt/methods/http did not start correctly
The command '/bin/sh -c sudo -H apt-get install -y     python-yaml python-mock python-empy python-netifaces python-numpy     python-imaging python-nose python-coverage python-paramiko     cmake uuid-dev sbcl     libboost-all-dev libtinyxml-dev liblz4-dev libapr1-dev libpoco-dev     libconsole-bridge-dev libaprutil1-dev libgtest-dev liblog4cxx10-dev' returned a non-zero code: 100

This is how I invoked the command that resulted in this error:
RUN sudo -H apt-get install -y

Fixed! executing command [apt-get install -y python-yaml]


Update Humble development branch for image_transport_plugins

Related to breaking change being introduced in https://github.com/ros-perception/image_transport_plugins/pull/106

Related release repository change: https://github.com/ros2-gbp/image_transport_plugins-release/pull/4


[DO NOT MERGE - Integration Tests] Update opentelemetry-operations-collector submodule

Description
This PR updates the OpenTelemetry Operations Collector submodule to receive the new OTel v0.100.0 updates.
Related issue
b/339060391
How has this been tested?
These integration tests :)
Checklist:

Unit tests

[ ] Unit tests do not apply.
[ ] Unit tests have been added/modified and passed for this PR.

Integration tests

[ ] Integration tests do not apply.
[ ] Integration tests have been added/modified and passed for this PR.

Documentation

[ ] This PR introduces no user visible changes.
[ ] This PR introduces user visible changes and the corresponding documentation change has been made.

Minor version bump

[ ] This PR introduces no new features.
[ ] This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
[ ] This PR bumps the version.

Obsoleted by OTel upgrade to v0.102.0


Insert table command

Can you make a insert/edit table command?

Can you write one and paste it here?

I want to help, but I do not know.
I looked at the source code of Medium Editor's tables extension, but I do not understand the code.

Ok - this will certainly never become part of the core pell code, but is a good idea for an extension for V2.0


Update STYLE_GUIDE.md

From publisher requirements (more details in PR #868)

Job d2l-zh/PR-889/2 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-889/

Job d2l-zh/PR-889/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-889/

Job d2l-zh/PR-889/3 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-889/


As a writer I need to know what format to offer the short descriptions in, so we can start the migration

Acceptance Criteria

[x] Format documented
[ ] VS Code has accepted format
[ ] Fx Devtools has accepted format

Acceptance Criteria

[ ] Format documented
[ ] VS Code has accepted format
[ ] Fx Devtools has accepted format

@atopal @wbamberg Can this be closed?

I think so, yes.

The acceptance criteria haven't been met yet as far as I can tell. @wbamberg do we think those are not relevant anymore, or are we tracking them somewhere else?

VS Code has accepted the format but we haven't asked for feedback from Fx Devtools. I suppose we should. I'll do this.

Should this be moved forward to an upcoming sprint, since its AC are not yet met?

@wbamberg shall I ask devtools for a review of the format? Is this still relevant, since we are moving ahead? Close it?

Closing this since work has moved ahead on short descriptions, regardless of what Fx DevTools may think.


PublicPath is not working in Production Mode

I am specifying publicPath as /js/ so that the output of worker goes to the js folder along with my other build files. However, it's not going there. Also, I'm getting an error in console saying
excelReaderWorker.js:2 GET http://<IP_ADDRESS>:8887/cpoupload/cbfd9ad385e82500bb79.worker.js 404 (Not Found)
{
test: /.worker.js$/,
exclude: /node_modules/,
loader: 'worker-loader',
options: { inline: true, fallback: false , publicPath: '/js/' , name: 'excelReader.[hash].worker.js'}
}
I'm able to get webworker working in my localhost. This only happens when i create build.

i have the same question

Is someone looking into this?

You need to set fallback: true, I will improve this, it doesn't have sense when you use fallback: false together with publicPath or name


Enable async delegation

This feature looks to enable async and impure node delegation as long as Features are enabled for it.
Additionally, removes the ICustomDelegationFunction interface in favor of overriding the delegation validation strategy in some functions to always delegate async calls of the function for appropriate functions (i.e. current behavior of AsType or User function in Power Apps).

✅ No public API change.


Introduce CVSS v4 in OSIDB

This PR introduces CVSS v4 in OSIDB (tested in BZ stage). The NVD collector was not updated because NVD does not support v4 yet.
Related to OSIDB-528

@osoukup I tested the code against the BZ stage, even though this change only extends the CVSSVersion enum by a new string and nothing more. I was able to test it only manually via curl. For some reason, none of the integration tests worked for me. I think it would be better to discuss it tomorrow.

@osoukup I tested the code against the BZ stage, even though this change only extends the CVSSVersion enum by a new string and nothing more. I was able to test it only manually via curl. For some reason, none of the integration tests worked for me. I think it would be better to discuss it tomorrow.

Stating that only the enum is being changed is an underrating of this PR. I see both BBSync and bzimport are being modified too. Based on a number of recent negative experience I would like us to be more mindful about the integration testing.


ModuleNotFoundError: No module named 'pcseg.version'

Hi, how can I find the package pcseg.version? I meet the Error: ModuleNotFoundError: No module named 'pcseg.version' in the file pcseg/__init__.py  when runing scrips/dist_train.sh. Thanks!

Hi, have you install pcseg following: https://github.com/CVMI-Lab/PLA/blob/main/docs/INSTALL.md?

Hi, have you install pcseg following: https://github.com/CVMI-Lab/PLA/blob/main/docs/INSTALL.md?

Yes，I have followed the INSTALL.md to build the extension. But there is no version folder or file under pcseg.

Hi, have you install pcseg following: https://github.com/CVMI-Lab/PLA/blob/main/docs/INSTALL.md?

Or could you release the version file or folder? I can try to download and put it under pcseg folder. Thanks!

this is quite strange. I don't think you should download it. Can you check whether this line in setup.py has been ran?
https://github.com/CVMI-Lab/PLA/blob/2e4b307f99a146ceaf9e430d4fcb9ebe635cfb71/setup.py#L32

this is quite strange. I don't think you should download it. Can you check whether this line in setup.py has been ran?
https://github.com/CVMI-Lab/PLA/blob/2e4b307f99a146ceaf9e430d4fcb9ebe635cfb71/setup.py#L32

I have re-installed pcseg and solved this problem. Thanks!


Allow more types for indexes in addVars

Currently, the indexes in the addVars can be ints or Iterable[_Scalar], where _Scalar = Union[int, float, str]. Can we allow Iterable[Hashable] instead?

Closing as duplicate of #6


Attributes MUST be last in expression

The parser should fail the following message.
{{1 :number @attr=true signDisplay=always}}

Attributes MUST be defined last in the expression, they are not part of the annotation.
Expected syntax
literal-expression    = "{" [s] literal [s annotation] *(s attribute) [s] "}"
variable-expression   = "{" [s] variable [s annotation] *(s attribute) [s] "}"
annotation-expression = "{" [s] annotation *(s attribute) [s] "}"

It works as expected. The example is pattern, not expression.


Use rel="icon" instead of rel="shortcut icon"

<link rel="shortcut icon" is deprecated and <link rel="icon" should be used.
(Also, remove unneeded (?!\*) from the lang regex in prism-file-highlight.js.)

Hi! Thanks for contributing again.
But please don't include both changes in a single PR, instead make two PRs, one for each issue.

New PRs: #1398 and #1399.


Detect exit request with CTRL-D in Input

Feature Request
Feature Request
Hi, thank you for the lovely library.
I am using it in a project here https://github.com/fortify500/stepsman
Would appreciate if you can add me to the list.
Currently I am using Input and not Run because I have no apparent way to enter the OptionInitialBufferText on every input (which i need).
However, the biggest problem is that in Input I don't know if a CTRL-D was pressed to exit. I tried serval tricks like KeyBind (which is not triggered for some reason) but to no avail.
Would appreciate it if it was possible to somehow either detect CTRL-D after the Input finished like an err returned in addition to the string or a callback or simply enter "exit" string.
Thanks!

I would add that I tried the exitchecker but CTRL-D exits before getting to it, not that it would help as it is, it always provides breakline false (at least on my build) for some reason.

Managed to do something like
prompt.OptionAddKeyBind(prompt.KeyBind{
Key: prompt.ControlM,
Fn: func(buffer *prompt.Buffer) {
wasEnter = true
},
})
if the returned input is "" and wasEnter=false then i am guessing CTRL-D was pressed. Not sure if it is valid ...

If I understand correctly, you implemented your own exit command, yes? If so, you can use the executor to detect this:
var exitCalled bool

func(in string) {
    if in == "exit" {
        exitCalled = true
}

Somewhere else in your program:
func foo() {
prompt.Run()
if exitCalled {
        fmt.Println("exited with `exit` cmd")
} else {
        fmt.Println("exited with `Ctrl-D`")

If that's the only other method of exiting, then that should work.

@mpokryva thank you for the reply.
However, please note I am using the prompt.Input().
Run() does not have that issue. I am using the Input() to have full control.
Thanks.

Whoa, sorry, I have no idea how I missed that. That was dumb. I think your approach is valid. I'm using v0.2.3, the latest tag, but it looks like master behaves the way https://github.com/c-bata/go-prompt/blob/56c224c88aba4d7326e30211b5a29d816a5cd42d/prompt.go#L123. If Enter, ControlJ, ControlM are pressed, then Input() returns. I think your approach will be more comprehensive if you add those keys as well.


Fixed Bug related defaultCommitMessage

Hey, I just made a Pull Request!

closes #19844
:heavy_check_mark: Checklist

[X] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[X] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[X] All your commits have a Signed-off-by line in the message. (more info)

@awanlin have to look

@benjdlambert why duplicate code is there any reason behind it?
gitHelpers
helpers

@npiyush97, aww no that's a mistake from when I did refactoring. You can go ahead and delete one of them :)

good for merge  @benjdlambert

@npiyush97 looks like theres some tests failing, you can ignore the E2E workflows, but the CI verify and CI Test workflows need fixing before we can merge this.

The verify one you can ignore as I just fixed that separately. But the CI / Test workflow failures are definitely related to the contents of this PR :)

@benjdlambert Oh but this is because of commit test I are written wrong providing defaultmessage
scaffolder: {
        defaultCommitMessage: 'Test commit message',
      }

but expects initial commit no worries I will fix this I didn't run all tests before committing.

@benjdlambert fixed but no idea why verify 20.x failed.


Writing a factory for a method that belongs_to two different instance of the same model and requires its id's

I have a User model that has_many accounts, each Account belongs_to one user and has_many transactions, each Transactions to two different accounts (sender_account and recipient_account), the original belongs_to method that adds the factory (like account) will not work since I need a sender_account and recipient_account.
If I manually set the ID with sender_account 1 and run rpsec I get:
ActiveRecord::AssociationTypeMismatch:
       Account(#5570447856500) expected, got 1 which is an instance of Integer(#5570414320800)

Even when the ID's are stored as Integer.
How can I add an association when my model belongs_to to different instances of the same model?
I already have my Account and User factory defined and they work as expected.

Thanks to thelp of Igor Drozdov I was able to figure out what was wrong (in case someone gets to this Issue with the same problem).
I defined my associations with:
association :sender_account, factory: :account, strategy: :build
association :recipient_account, factory: :account, strategy: :build
When the strategy was not needed as it didn't save the accounts and therefore the tests were failing.


CI: test cp313-dev

Start testing on cp313 in the run up to its release. Py_UNICODE is deprecated in cp313, which necessitated one amendment.

As far as I can tell the fail is unrelated.

It is unrelated. PR that fixes it will merge soon.

Looks great, thanks for this @andyfaff!


Bot Deployment for Bot Framework Composer

"Bot Framework Composer" can be deployed on premises or is mandatory to deploy on Azure only ?
If yes could you please provide any reference for that.
Thanks

@ahmedmahmoudR You can host Composer on premises, it is not mandatory to deploy to Azure. I will see if I can find any pertinent documentation for you.

Hello @dmvtech
Any Update

Hi @ahmedmahmoudR
Apologies for the delay. There are no specific directions for hosting on premises (or even in Azure).
The basic instructions for running Composer with a local build (as opposed to running the Electron client on a machine) can be found here:
https://github.com/microsoft/BotFramework-composer#build-composer-locally


[RFC] "Life of a Request" Doc

I'd appreciate comments on docs/life-of-a-request.md especially.

Automated message from Dropbox CLA bot
@acrefoot, it looks like you've already signed the Dropbox CLA. Thanks!

Broad comment: consider going out of your way to overdefine/explain for new developers (new to Zulip or to web development or to Python or to programming entirely) where it wouldn't disrupt or bog down the explanation too much for more experienced developers. Where it would be disruptive to define inline, consider linking out to other documentation.
For example, I'd probably link out to nginx, tornado, and django when they're first mentioned. (Similarly but probably less urgently, I'd link directly to things like "the HTTP RFC" for the curious.)

Also on Life of a Request: I think it might also be helpful to motivate the discussion with a specific example request rather than (as far as I can tell) the current use of a generic request. Pick a specific call with a specific endpoint and trace that through (with some digression/expansion as necessary to clarify alternative paths other kinds of requests take). It looks like you mostly stick to "creating a user" in the places you choose to show code, so maybe call that out at the beginning of the document with a "Specifically, we'll highlight a POST request to /api/v1/users." Alternatively, it may be worthwhile to pick an example that also works well with the "PUT vs POST vs PATCH" section to avoid switching into emoji and back out.

Most of my inline comments are nits/personal style preferences, feel free to adopt or ignore at your discretion.

Another document-wide comment for Life of a Request (I promise this is the last one for the night, and that I'll do a better job of aggregating all of these before posting next time I'm reviewing one of those :) ): it feels like the use of "we" vs "you" and "I" is a little inconsistent between sections. It's probably worth reviewing that and nailing it down. (For what it's worth, we start with "Let us" in the first paragraph and I like the idea of following through with "we" in the rest of the document to treat this as a collaborative exploration with the reader.)
Overall, though, I think this is going to be a great addition to the docs! Thanks for putting it together :)

@showell: good to see your recent work, and thanks for the comments!
@jdherg: I really appreciate the high-level and detailed-oriented comments. Sorry for the typos and writing inconsistency--I wanted to get some early feedback on whether this was a good document to have, or whether there were better places in the docs to incorporate this knowledge.
I'm working on addressing these concerns now!

Cool, this is looking great!  I posted a few comments, the other thing I'd do is a bit of liberal sprinkling of links to the Django docs when we talk about standard concepts (e.g. urls.py), similar to how we do it in directory-structure.md.


Replace @JonasFranzDEV with @jonasfranz

This pull should NOT be merged. As previously stated, this repository is NOT for general source changes. Changes in this repository should be left up to someone that is familiar with Debian packaging.

@MTecknology sorry, this PR was created automatically. I will close this PR.


Update package.json

Description

A clear and concise summary of the change and which issue (if any) it fixes. Should also include relevant motivation and context.
Resolved or fixed issue:
Affirmation

[ ] My code follows the CONTRIBUTING.md guidelines

:no_entry: Snyk checks have failed. 2 issues have been found so far.

Icon
Severity
Issues

Critical
0

High
1

Medium
1

Low
0

:no_entry: security/snyk check is complete. 2 issues have been found. (View Details)
:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)


Deploying theme to server has blank page

In development server (localhost) everything works fine.
But after npm run build get folder that I deploy to wordrpess server.
It appears in theme section I can choose it, but it appears only blank page with some errors:

I saw you closed, but just wanted to be sure you got it working. I'm guessing you had different root paths in Dev and Prod...? And you found the instructions...?
Dealing With Differing Paths Between DEV and PROD

Yes , I got it working, I had different root path in Dev and Prod.
Thanks.


import error when using datapackage installed by pip

Hi,
currently i am testing the datapackage module (for use in our research data management enviroment). First i tried your examples https://github.com/frictionlessdata/datapackage-py/blob/master/README.md. But
i got an ImportError like this one:

Traceback (most recent call last):
File "create-datapackage.py", line 3, in
from datapackage import Package
File "/home/RRZN/kaps/Dokumente/Projekte/dataset-examples/venv/lib/python3.5/site-packages/datapackage/init.py", line 10, in
from .package import Package
File "/home/RRZN/kaps/Dokumente/Projekte/dataset-examples/venv/lib/python3.5/site-packages/datapackage/package.py", line 19, in
from tableschema import Storage
File "/home/RRZN/kaps/Dokumente/Projekte/dataset-examples/venv/lib/python3.5/site-packages/tableschema/init.py", line 10, in
from .schema import Schema
File "/home/RRZN/kaps/Dokumente/Projekte/dataset-examples/venv/lib/python3.5/site-packages/tableschema/schema.py", line 12, in
from .field import Field
File "/home/RRZN/kaps/Dokumente/Projekte/dataset-examples/venv/lib/python3.5/site-packages/tableschema/field.py", line 13, in
from . import types
File "/home/RRZN/kaps/Dokumente/Projekte/dataset-examples/venv/lib/python3.5/site-packages/tableschema/types/init.py", line 21, in
from .string import cast_string
File "/home/RRZN/kaps/Dokumente/Projekte/dataset-examples/venv/lib/python3.5/site-packages/tableschema/types/string.py", line 12, in
import rfc3986.validators
ImportError: No module named 'rfc3986.validators'

Is it a problem of my installation or a problem of the pip-package / python module? And how can i fix it?
Many thanks for your work, Reiko

@reikkaps
Hi, probably it's a problem with your installation because I can't reproduce it on a fresh one.

@roll many thanks. i will investigate it further. And btw a fresh source installation from the github repo was sucessful.

@reikkaps
Great! Related to the pip installation - the following should work:
mkdir test
cd test
virtualenv .python -p python3
source .python/bin/activate
pip install datapackage

Closing for now. Please re-open if needed.


Feature/methods

What does this PR do?
Allows you to configure the allowed methods for each proxy.

👍


ci: Fix creating GCP instances

Motivation

Close #8151.
We're about to release Zebra 1.6, and a necessary precondition is to: "Make sure there has been at least one successful full sync test since the last state change, [...]", and there's no such test.

PR Author Checklist
Check before marking the PR as ready for review:

[x] Will the PR name make sense to users?
[x] Does the PR have a priority label?
[ ] Have you added or updated tests?
[x] Is the documentation up to date?

Solution

Remove the DISK_OPTION env var, which causes this error: https://github.com/ZcashFoundation/zebra/actions/runs/7930252459/job/21652230260#step:11:76. The problem is that the variable is set in another job with id get-disk-name, and evaluates to an empty string in other jobs.

Reviewer Checklist
Check before approving the PR:

[ ] Does the PR scope match the ticket?
[ ] Are there enough tests to make sure it works? Do the tests cover the PR motivation?
[ ] Are all the PR blockers dealt with?
PR blockers can be dealt with in new tickets or PRs.

And check the PR Author checklist is complete.
Follow Up Work
Refactor the cached disk selection. TODO: Replace the previous sentence with an issue number.

Further Details
Removing the variable effectively turns off using cached disks, but I don't think that worked before because we had a syntactic error in the --create-disk flag. We were missing a comma, which was added in 516fe1c (#8005). Adding the comma likely started causing the current issue since the --create-disk flag now evaluates to --create-disk=,, which seems to be the bug. Without the comma, DISK_OPTION evaluated to an empty string, and the name property directly followed, so the --create-disk flag was syntactically correct, but the image=$CACHED_DISK_NAME propery, set in DISK_OPTION, was missing.

This PR didn't address the underlying issue, so I'm closing it. We should address #8302 instead.


Add support for WinUI 3 projects on .NET 9 Previews

Ability to create WinUI 3 projects on .NET 9 Previews

This feature is now available in the following release:

VijayAnand.WinUITemplates - v3.0.0


fix github actions

the gh actions runner is failing to run the binary, it exits with a segmentation fault for some reason.
In my fork, I made the action upload the executable, I put it in /usr/bin and it worked perfectly fine even with box64:
ubuntu@pi-builder:~/Downloads/artifact $ ls
temp
ubuntu@pi-builder:~/Downloads/artifact $ file temp
temp: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, BuildID[sha1]=6c2d20add7e6550f86605bc08fe92689f4b3c867, for GNU/Linux 3.2.0, not stripped
ubuntu@pi-builder:~/Downloads/artifact $ sudo mv temp /usr/bin/\$
ubuntu@pi-builder:~/Downloads/artifact $ $ echo test 123
Box64 with Dynarec v0.1.3 2976bf5 built on Jul 12 2021 13:53:42
Using default BOX64_LD_LIBRARY_PATH: ./:lib/:lib64/:x86_64/:bin64/:libs64/
Using default BOX64_PATH: ./:bin/
Counted 28 Env var
Looking for /usr/bin/$
argv[1]="echo"
argv[2]="test"
argv[3]="123"
Using native(wrapped) libc.so.6
Using native(wrapped) ld-linux-x86-64.so.2
Using native(wrapped) libpthread.so.0
Using native(wrapped) librt.so.1
test 123
ubuntu@pi-builder:~/Downloads/artifact $

but when running directly, it fails:
ubuntu@pi-builder:~/Downloads/artifact $ ls
temp
ubuntu@pi-builder:~/Downloads/artifact $ ./temp $ echo 123
Box64 with Dynarec v0.1.3 2976bf5 built on Jul 12 2021 13:53:42
Using default BOX64_LD_LIBRARY_PATH: ./:lib/:lib64/:x86_64/:bin64/:libs64/
Using default BOX64_PATH: ./:bin/
Counted 28 Env var
Looking for ./temp
argv[1]="$"
argv[2]="echo"
argv[3]="123"
Using native(wrapped) libc.so.6
Using native(wrapped) ld-linux-x86-64.so.2
Using native(wrapped) libpthread.so.0
Using native(wrapped) librt.so.1
/bin/bash: $: command not found
ubuntu@pi-builder:~/Downloads/artifact $

doing it natively does work:
pi@TwisterPi4:~/Documents/git/DollarSkip $ make
gcc dollarskip.c -o temp
pi@TwisterPi4:~/Documents/git/DollarSkip $ ./temp $ echo 123
123
pi@TwisterPi4:~/Documents/git/DollarSkip $

I tried making the action run it by putting the executable in ~/.local/bin/$ and adding that to $PATH, but it still didn't work.
The changes I made to the workflow

rename main.yml -> build.yml
remove the test part
make it upload the executable so we can test it later if we want

I have a feeling it is something to do with it being in a script.
Rather than being on the bash command line, as if it were copying commands, the .yml seems to become a .sh file and be executed.
Where is the binary put?

pi@TwisterPi4:~/Documents/git/DollarSkip $ ./temp $ echo 123

the ./temp is the $ so you are effectively executing.
 $ $ echo 123 so I don't know how that isn't breaking

pi@TwisterPi4:~/Documents/git/DollarSkip $ ./temp $ echo 123

the ./temp is the $ so you are effectively executing.
 $ $ echo 123 so I don't know how that isn't breaking

:facepalm:

Where is the binary put?

without moving it, its in ~/DollarSkip I think.

Leave the branches.


Repurpose "no files" Files screen content as first-launch or help material

Describe the bug
Enhancement recommendation*
From a user experience aspect, the below image (welcome information) would be good as a landing page for the app rather than seeing it after clicking onto a menu option.
To Reproduce
Steps to reproduce the behavior:

Go to the Files page
View the Information that's displayed on the initial page

Expected behavior
This information is really useful and is lost once you upload a file. If this information was the landing page for the app then it wouldn't be lost and would be in a more useful position
Screenshots

Desktop (please complete the following information):

OS: Windows 10 (Laptop)

Hi @davenes8148 -- this is great feedback. Thanks!
Right now that material appears if the user doesn't have any files uploaded (as you noticed), but you're right: It's more general information than files-specific info, and would be a lot more useful as welcome/introductory content, either at first launch or as help tour content.
Slotting this into our queue of UX examples for further prioritization and concept work. If anyone's reading who would be interested in contributing, this is more than welcome!

Let's also roll the suggestion from https://github.com/ipfs-shipyard/ipfs-webui/issues/1406 into this effort: incorporating some kind of prominent link to Awesome IPFS as an example of cool things new users can be inspired by.


Bug: Change box sizer to grid sizer and validator varnames get changed

Description:

Changing a box sizer to a grid sizer changes all validator variable names in any children of the form (it adds a number at the end). This can be reproduced in the PreferencesDlg in wxUiEditor.

Probably the same underlying cause as #1452


Is this line [assembly: XmlConfigurator(Watch = true)] required?

I found that once I set up my logging I didn't see any log messages. After some googling I saw a mention of adding this line somewhere in my project:
[assembly: XmlConfigurator(Watch = true)]

I see no mention of this in your own documentation. But I have noticed that without this line, my logging stops. So I just wanted to clarify that it is required or I'm doing something else wrong.

I haven't seen that before. The only thing I can think of is that sometimes I needed to make sure that my App/Web.config would need to copy to the output directory before log4net would pick it up. Possible that it is related.

Odd that you haven't seen it. Here's a SO thread: http://stackoverflow.com/questions/10204171/configure-log4net-in-web-application
Logging works fine for me with the assembly attribute above and an old version of Logentries (le_log4net version 2.3.7). No luck at all getting Logentries (2.6.3) working in a web app (with or without this XmlConfigurator).
By the way the sample console app you pointed me to in #38 works fine locally. Not sure what is different about my web app. Like I said, everything is fine with the old version of logentries (except I can't get ssl to work).
With version 2.6.3 I'm seeing this in my debug output (if I set XmlConfigurator, if I don't set that attribute then I see nothing in my debug window).
LE: LE - Adding Line: line
LE: LE - Adding Line: line
LE: LE - Adding Line: line
LE: LE - Adding Line: line

This would seem to indicate to me that Logentries is working. But nothing shows up in my log.

The only way I got the current version to work is to checkout the source code add LogentriesAppender to my own project and compile locally. Literally, without changing anything else in my project or web.config (except the assembly name), now it works.

I finally got plain text and ssl logging to work. I had to ensure I used the exact same versions of log4net, logentries.core, and logentries.log4net that you did.
<packages>
  <package id="log4net" version="2.0.3" targetFramework="net45" />
  <package id="logentries.core" version="2.7.1" targetFramework="net45" />
  <package id="logentries.log4net" version="2.6.3" targetFramework="net45" />
  <package id="Microsoft.WindowsAzure.ConfigurationManager" version="3.1.0" targetFramework="net45" />
</packages>


Travis: Enable retries for unit tests as well.

At this point, they are too flaky and it hinders productivity too much to bother with that.
@alainjobart

LGTM


VSCode on Macos no longer lets me open a directory from commandline

VSCode Version: Version 1.34.0 (1.34.0)
OS Version: 10.14.4 (18E226)

Steps to Reproduce:

cd path/to/project
open -a "/Applications/Visual Studio Code.app" .

Error:
LSOpenURLsWithRole() failed for the application /Applications/Visual Studio Code.app with error -600 for the file /Users/user/go/src/test/thing.

Does this issue occur when all extensions are disabled?: Yes
Up until I updated this morning this worked perfectly, I could spin up a vscode instance in any project directory, now I cannot.
(nb, I normally use an alias alias vsc='open -a "Visual Studio Code"' but it doesn't work, same error)

In macOS, simply run F1, Shell Command: Install 'code' command in PATH and remove your alias. Then you can simply run code.


Material UI button effect

Note: return new wrapper with events and add additional elements with filter effects (overlay or something)


All but 3 build configurations are successful

I ran the AdoptOpenJDK/build-jdk@v1 GitHub action against all combinations of jdk versions 11,12,13,15, platforms ubuntu-latest,macos-latest,windows-latest, and jdk implementations hotspot,openj9.
All but 3 of them were successful. See: this github workflow run

Updated jdk15 to jdk16  now 3 builds failed. Open the issue https://github.com/AdoptOpenJDK/build-jdk/issues/27, close this one.


[KYUUBI #5884][Bug] PVM should inherit MultiInstance and wrap with new exprId

:mag: Description
Issue References 🔗

This pull request fixes #5884
Describe Your Solution 🔧
We meet a case that we create one temp view from a PVM, and setting spark.sql.legacy.storeAnalyzedPlanForView=true
CREATE OR REPLACE TEMPORARY VIEW tmp_view AS
SELECT * FROM persist_view

Then we crease two new view based on tmp_view then join them, this cause a column exprId conflict on join's left and right side.
This is because in spark side, to avoid view will cause a code exprId conflict, it will wrap a project with new exprId to view's child

Before PVM is UnaryNode,  this behavior still can work on PVM's child, now it's LeafNode then won't work.
Like HiveTableRelation, PVM also need to inherit MultiInstanceRelation  and wrap child with new ExprId to avid below issue.
This change works fine in our prod.
Types of changes :bookmark:

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Added UT
Behavior Without This Pull Request :coffin:
Failed since vies's exprId conflict.
Caused by: org.apache.spark.sql.AnalysisException: cannot resolve '`a.id`' given input columns: [a.id, a.scope]; line 5 pos 3;
'Project [ArrayBuffer(a).*, 'b.scope AS new_scope#22]
+- 'Join Inner, ('a.id = 'b.id)
   :- SubqueryAlias a
   :  +- SubqueryAlias view2
   :     +- Project [id#18, scope#19]
   :        +- Filter (scope#19 < 10)
   :           +- SubqueryAlias view1
   :              +- Project [id#18, scope#19]
   :                 +- Filter (id#18 > 10)
   :                    +- SubqueryAlias spark_catalog.default.perm_view
   :                       +- PermanentViewMarker
   :                             +- View (`default`.`perm_view`, [id#18,scope#19])
   :                                +- Project [id#20, scope#21]
   :                                   +- SubqueryAlias spark_catalog.default.table1
   :                                      +- HiveTableRelation [`default`.`table1`, org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe, Data Cols: [id#20, scope#21], Partition Cols: []]
   +- SubqueryAlias b
      +- SubqueryAlias view3
         +- Project [id#18, scope#19]
            +- Filter isnotnull(scope#19)
               +- SubqueryAlias view1
                  +- Project [id#18, scope#19]
                     +- Filter (id#18 > 10)
                        +- SubqueryAlias spark_catalog.default.perm_view
                           +- PermanentViewMarker
                                 +- View (`default`.`perm_view`, [id#18,scope#19])
                                    +- Project [id#20, scope#21]
                                       +- SubqueryAlias spark_catalog.default.table1
                                          +- HiveTableRelation [`default`.`table1`, org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe, Data Cols: [id#20, scope#21], Partition Cols: []]

Behavior With This Pull Request :tada:
Can work well
Project [id#18, scope#19, scope#24 AS new_scope#22]
+- Join Inner, (id#18 = id#23)
   :- Filter ((id#18 > 10) AND (scope#19 < 10))
   :  +- PermanentViewMarker
   :        +- View (`default`.`perm_view`, [id#18,scope#19])
   :           +- Project [id#20, scope#21]
   :              +- SubqueryAlias spark_catalog.default.table1
   :                 +- HiveTableRelation [`default`.`table1`, org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe, Data Cols: [id#20, scope#21], Partition Cols: []]
   +- Filter ((id#23 > 10) AND isnotnull(scope#24))
      +- PermanentViewMarker
            +- Project [cast(id#18 as int) AS id#23, cast(scope#19 as int) AS scope#24]
               +- View (`default`.`perm_view`, [id#18,scope#19])
                  +- Project [id#20, scope#21]
                     +- SubqueryAlias spark_catalog.default.table1
                        +- HiveTableRelation [`default`.`table1`, org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe, Data Cols: [id#20, scope#21], Partition Cols: []]

Related Unit Tests

Checklists
📝 Author Self Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] This patch was not authored or co-authored using Generative Tooling

📝 Committer Pre-Merge Checklist

[ ] Pull request title is okay.
[ ] No license issues.
[ ] Milestone correctly set?
[ ] Test coverage is ok
[ ] Assignees are selected.
[ ] Minimum number of approvals
[ ] No changes are requested

Be nice. Be informative.

ping @yaooqinn

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (e2d9a8e) 61.62% compared to head (6814a83) 61.50%.

Files
Patch %
Lines

...authz/rule/permanentview/PermanentViewMarker.scala
20.00%
3 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5885      +/-   ##
============================================
- Coverage     61.62%   61.50%   -0.13%
  Complexity       23       23
============================================
  Files           613      613
  Lines         36327    36331       +4
  Branches       4974     4974
============================================
- Hits          22388    22346      -42
- Misses        11546    11576      +30
- Partials       2393     2409      +16

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Would you mind to provide a Unit test?

Then we crease two new view based on tmp_view then join them, this cause a column exprId conflict on join's left and right side.
What would happen in this case, the query failed in analysis or mismatched the policies?

Then we crease two new view based on tmp_view then join them, this cause a column exprId conflict on join's left and right side.
What would happen in this case, the query failed in analysis or mismatched the policies?

Failed in analysis

Would you mind providing a Unit test?

DOne

Then we crease two new view based on tmp_view then join them, this cause a column exprId conflict on join's left and right side.
What would happen in this case, the query failed in analysis or mismatched the policies?

Can check the new PR desc. Shows the analysis exception

cc @bowenliang123, any more concerns?

Thanks, merged to master

@AngersZhuuuu
Brother, I have a question that's been bothering me.
If PermanentViewMarker inherits LeafNode and the RuleApplyPermanentViewMarker rule is registered during the resolution phase. If the PermanentViewMarker LogicalPlan is generated during the resolution phase with an empty children property, will it affect other rules that read the children property? For example, RuleApplyRowFilter.

@AngersZhuuuu
Brother, I have a question that's been bothering me. If PermanentViewMarker inherits LeafNode and the RuleApplyPermanentViewMarker rule is registered during the resolution phase. If the PermanentViewMarker LogicalPlan is generated during the resolution phase with an empty children property, will it affect other rules that read the children property? For example, RuleApplyRowFilter.

@yaooqinn @pan3793 @bowenliang123


treewide: unpin most LLVM-14s in all-packages.nix

packages marked broken are unpinned by default (poac, codon)

built all other packages

cdrtools still pinned on darwin

ikos still pinned as it is a static-analyzer which requires llvm-14

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Nice!
poac should be fixed by https://github.com/NixOS/nixpkgs/pull/342790; maybe worth dropping from this PR to avoid merge conflicts.


Add end() method and mode change validation

Hi, we've been doing some more development, though it's worth sharing :)
I've implemented the STC3100dd::end() for setting sensor standby mode in a single call and modified start() to explicitly call setModeOperateRun() so then _operate reflects the current sensor state.
Also added mode register readout after writing inside updateModeReg() so that both start() and end() can verify the run bit actually flipped and that is reflected in their return code.

Thanks for the updates


pull request for e2e test at 01/03/2017 14:26:08

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Intent to Search K9 Broken

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.800
Where did you get the app from?
F-Droid
Android version
14
Device model
Pixel 8 Pro
Steps to reproduce

Have app that uses an intent to open K9 to search all messages

This code used to work:
                                   ```
                                    thelastterm = foo
                                    val mapIntent = Intent(Intent.ACTION_SEARCH)
                                    mapIntent.setPackage("com.fsck.k9")
                                    mapIntent.putExtra("query", thelastterm)
                                    mapIntent.flags = Intent.FLAG_ACTIVITY_NEW_TASK
                                    startActivity(mapIntent)

### Expected behavior

K9 used to display the results

### Actual behavior

Crash

### Logs

FATAL EXCEPTION: main
Process: com.example999.searchparty, PID: 30613
android.content.ActivityNotFoundException: No Activity found to handle Intent { act=android.intent.action.SEARCH flg=0x10000000 pkg=com.fsck.k9 (has extras) }
	at android.app.Instrumentation.checkStartActivityResult(Instrumentation.java:2363)
	at android.app.Instrumentation.execStartActivity(Instrumentation.java:1949)
	at android.app.Activity.startActivityForResult(Activity.java:5743)
	at a.o.startActivityForResult(SourceFile:2)
	at android.app.Activity.startActivityForResult(Activity.java:5701)
	at a.o.startActivityForResult(SourceFile:1)
	at android.app.Activity.startActivity(Activity.java:6199)
	at android.app.Activity.startActivity(Activity.java:6166)
	at i0.B.onClick(Unknown Source:305)
	at android.view.View.performClick(View.java:7729)
	at com.google.android.material.button.MaterialButton.performClick(Unknown Source:9)
	at android.view.View.performClickInternal(View.java:7706)
	at android.view.View.-$$Nest$mperformClickInternal(Unknown Source:0)
	at android.view.View$PerformClick.run(View.java:30484)
	at android.os.Handler.handleCallback(Handler.java:959)
	at android.os.Handler.dispatchMessage(Handler.java:100)
	at android.os.Looper.loopOnce(Looper.java:232)
	at android.os.Looper.loop(Looper.java:317)
	at android.app.ActivityThread.main(ActivityThread.java:8501)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:552)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:878)

We did remove support for this intent action in #7593. However, it was never meant to be a public API. So we don't consider this to be a bug.
Can you provide a use case for such an API?

We did remove support for this intent action in #7593. However, it was never meant to be a public API. So we don't consider this to be a bug.
Can you provide a use case for such an API?

I used it for a personal app as it was handy for searching through all my years of emails.


New version: Maxthon.Maxthon version <IP_ADDRESS>01

Created by 🥟 Dumplings.
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335141
2024-06-16 13:29:27.826 [CLI ] Unable to locate nested installer at: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\bronya0.ES-King.0.11\extracted\ES-King\ES-King-v0.11.exe

Automated error analysis suggests -1978335141 may mean Unable to locate nested installer. Does the RelativeFilePath have the previous version number in it?
(Automated response - build 886.)

Automatic Validation ended with:
(Automated response - build 886.)


Overlay API exploration

I've personally not invested a lot of time updating omnisharp-atom because all the cool features I wanted we're depending on being able to do overlays in the editor easy
The Atom folks added a pretty sweet API https://github.com/atom/atom/pull/4144 for all of this which landed in Atom 2 weeks ago so I/we (but mainly I :)) need to get my ass back in gear.

[X] Reimplement autocomplete using overlay API (see #44 )
[x]  alt+enter refactorings, omnisharp allows you to get a list of available refactorings at current position, this should be smoother then relying on the command launcher (cmd+shift+p) which visually works better for global actions.

This issue serves as a scratchpad while we fletch out the basis for both.

We have this now with the nice code actions window from @nosami !


Resolve Failing Update_Pr

Swapping to bot that won't have this push issue. azure-sdk

I assume the pipeline has already been updated to point to this new variable. BTW I think there are 2 pipelines one for the public repo and one for the private repo that might need updating.

Yes, one public, one private. Both are updated.

I'm merging this and re-enabling the job. @zikalino update_pr should pass after this.


Pretty print

Using this file:
$ cat x.json
{"name":{"first":"Tom","last":"Smith"}}

JQ can pretty print a block:
$ jq .name x.json
{
  "first": "Tom",
  "last": "Smith"
}

JSONed seems to have no way to do that:
$ jsoned -i x.json name
{"first":"Tom","last":"Smith"}

https://github.com/tidwall/jsoned/issues/4

I just added a -p flag that will now pretty print. Also the keypath is optional when the -p flag is specified.

confirmed fixed - damn that was fast - thanks

Thanks! I'll close these issues. Feel free to reopen if you run into any problems.


DEV-35268 feat: adding demo for docx checking support

Hi @Thilo-Acrolinx,
I see that this PR is still blocked, so what is the next step here?


Nitro DevProxy ignores target basePath

Environment
NitroPack
nitropack@latest:
  version "2.6.1"

Nitro Config
export default defineNitroConfig({
    devProxy: {
        "/api": {
           target: "http://localhost:3001/api"
        }
    }
});

Node
v18.12.1

Yarn
1.22.19

Reproduction
Sample is provided in https://github.com/cs8898/nitro_devproxy_mre
Describe the bug
Problem
Nitro devProxy drops basepath of proxied server (only accepts Protocol, Host and Port of URL)
Expected
http://nitro:3000/api/foo => http://server:3001/api/foo

Observed
http://nitro:3000/api/foo => http://server:3001/foo

Additional context
No response
Logs
No response

Possible Relevant File:
https://github.com/unjs/nitro/blob/main/src/dev/server.ts#L170C33-L170C33

Actually httpxy is ignoring the BaseURL
Nitro just drops the Route Segment of the defined Router
https://github.com/cs8898/nitro_devproxy_mre/commit/f15a35fd636d37b490affd053a79c04c9e446eb0

FWIW, I tried looking into this and also found the bug to be in httpxy. When setting the target path, it is using a path property on an URL object. This property does not exist, and changing it to pathname seems to have solved the issue: https://github.com/jonasolesen/httpxy/commit/0919f9d0be5b4749bb9d40c6f9557e0137712818

that seems to help Thanks
https://developer.mozilla.org/en-US/docs/Web/API/URL/pathname

@jonasolesen Works also for me, i can confirm, that solved the Problem.
I will close that here, if you could create an PR that would be great

@cs8898 That's great! I've submitted an issue and a PR- hoping the fix is as simple as it seemed 🙂


Remove Arcs server targets from travis config.

Travis is slow, code here is stale. Removing from CI for the time being. To return from the ashes like a phoenix but different in the not so distant future.

This has been implemented in #4008


[VMAX Driver] Update Resource status based on common model

Issue/Feature Description:
Resource status model needs to be finalized and map all resource status according to SODA resource model
Why this issue to fixed / feature is needed(give scenarios or use cases):
To make model generic
How to reproduce, in case of a bug:
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)

/move sim


cuda 11.7.*  does not exist (perhaps a missing channel)

conda install pytorch==1.13.1 torchvision==0.14.1 torchaudio==0.13.1 pytorch-cuda=11.7 -c pytorch -c nvidia
Channels:

pytorch
nvidia
defaults
Platform: osx-arm64
Collecting package metadata (repodata.json): done
Solving environment: failed

LibMambaUnsatisfiableError: Encountered problems while solving:

nothing provides cuda 11.7.* needed by pytorch-cuda-11.7-h67b0de4_0

Could not solve for environment specs
The following package could not be installed
└─ pytorch-cuda 11.7**  is not installable because it requires
└─ cuda 11.7.* , which does not exist (perhaps a missing channel).

Looks like you are on a mac, won't be possible

@JuvenileLocksmith i've already solved

conda install pytorch::pytorch-cuda
conda install pytorch==1.13.1 torchvision==0.14.1 torchaudio==0.13.1 -c pytorch -c nvidia

I am a Macbook M2 user, and here is my workaround:

you don't need to install the pytorch-cuda. Simply remove that entry from the conda install command:

conda install pytorch==1.13.1 torchvision==0.14.1 torchaudio==0.13.1 -c pytorch -c nvidia

in the jupyter notebook, replace device = 'cuda:0' with device = 'cpu' in the Initialization step

It works like a charm.


[QUESTION] got a error when run twint

Issue Template
Please use this template!
Description of Issue
Hello, sorry to disturb
I got an error run Twint as module with a test .py script, the log shows can not connect to twitter.com:80, but I have set  proxy host in my test.py.
and the status of proxy is ok, I can access twitter.com with chrome broswer
could you pls tell me where shoule I add or fix? thanks a lot

#this is my proxy

#this is the error log
CRITICAL:root:twint.get:User:Cannot connect to host twitter.com:80 ssl:False [Connect call failed ('<IP_ADDRESS>', 80)]
Traceback (most recent call last):
File "D:\Program Files\Python37\lib\site-packages\aiohttp\connector.py", line 924, in _wrap_create_connection
await self._loop.create_connection(*args, **kwargs))
File "D:\Program Files\Python37\Lib\asyncio\base_events.py", line 943, in create_connection
raise exceptions[0]
File "D:\Program Files\Python37\Lib\asyncio\base_events.py", line 930, in create_connection
await self.sock_connect(sock, address)
File "D:\Program Files\Python37\Lib\asyncio\selector_events.py", line 475, in sock_connect
return await fut
File "D:\Program Files\Python37\Lib\asyncio\selector_events.py", line 505, in _sock_connect_cb
raise OSError(err, f'Connect call failed {address}')
TimeoutError: [Errno 10060] Connect call failed ('<IP_ADDRESS>', 80)
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "D:/Program Files/PycharmProjects/Python_Project/twint-master/test_2.py", line 12, in
twint.run.Search(c)
File "D:\Program Files\PycharmProjects\Python_Project\twint-master\twint\run.py", line 271, in Search
run(config, callback)
File "D:\Program Files\PycharmProjects\Python_Project\twint-master\twint\run.py", line 200, in run
get_event_loop().run_until_complete(Twint(config).main(callback))
File "D:\Program Files\Python37\Lib\asyncio\base_events.py", line 568, in run_until_complete
return future.result()
File "D:\Program Files\PycharmProjects\Python_Project\twint-master\twint\run.py", line 141, in main
await task
File "D:\Program Files\PycharmProjects\Python_Project\twint-master\twint\run.py", line 185, in run
await self.tweets()
File "D:\Program Files\PycharmProjects\Python_Project\twint-master\twint\run.py", line 124, in tweets
await self.Feed()
File "D:\Program Files\PycharmProjects\Python_Project\twint-master\twint\run.py", line 50, in Feed
response = await get.RequestUrl(self.config, self.init, headers=[("User-Agent", self.user_agent)])
File "D:\Program Files\PycharmProjects\Python_Project\twint-master\twint\get.py", line 80, in RequestUrl
response = await Request(_url, params=params, connector=_connector, headers=headers)
File "D:\Program Files\PycharmProjects\Python_Project\twint-master\twint\get.py", line 130, in Request
return await Response(session, url, params)
File "D:\Program Files\PycharmProjects\Python_Project\twint-master\twint\get.py", line 135, in Response
async with session.get(url, ssl=False, params=params, proxy=httpproxy) as response:
File "D:\Program Files\Python37\lib\site-packages\aiohttp\client.py", line 1005, in aenter
self._resp = await self._coro
File "D:\Program Files\Python37\lib\site-packages\aiohttp\client.py", line 476, in _request
timeout=real_timeout
File "D:\Program Files\Python37\lib\site-packages\aiohttp\connector.py", line 522, in connect
proto = await self._create_connection(req, traces, timeout)
File "D:\Program Files\Python37\lib\site-packages\aiohttp\connector.py", line 854, in _create_connection
req, traces, timeout)
File "D:\Program Files\Python37\lib\site-packages\aiohttp\connector.py", line 992, in _create_direct_connection
raise last_exc
File "D:\Program Files\Python37\lib\site-packages\aiohttp\connector.py", line 974, in _create_direct_connection
req=req, client_error=client_error)
File "D:\Program Files\Python37\lib\site-packages\aiohttp\connector.py", line 931, in _wrap_create_connection
raise client_error(req.connection_key, exc) from exc
aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host twitter.com:80 ssl:False [Connect call failed ('<IP_ADDRESS>', 80)]
Environment Details

Windows 7 and Python 3.7

Hi, Can you tell me how to solve the problem?

It's a network issue, please be sure that your connection is working as wanted

It's a network issue, please be sure that your connection is working as wanted

Thank you. I will check it.


Adding tools for scanning

Adding tools for scanning

Docs Build status updates of commit 9dea3ce:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/98828/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add parsable directive to TCP_CHECK and HTTP_GET

Keepalived can parse those syntaxes:
:
    TCP_CHECK {
        :
        retry 3
        delay_before_retry 3
        warmup 1
        :
    }
:

and
:
    HTTP_GET {
        :
        warmup 1
        :
    }
:

It is undocumented in keepalived.conf(5).
But I checked those directives run correctly.
And you can check those syntaxes is parsable by reading source code.

https://github.com/acassen/keepalived/blob/951e72059b776f55b25d9b59966d7e1f7139a239/keepalived/check/check_tcp.c#L89-L98
https://github.com/acassen/keepalived/blob/951e72059b776f55b25d9b59966d7e1f7139a239/keepalived/check/check_http.c#L173
https://github.com/acassen/keepalived/blob/951e72059b776f55b25d9b59966d7e1f7139a239/keepalived/check/check_http.c#L192

So I added the codes of parse those syntaxes.

LGTM! Thanks.

Thank you for quick merge :innocent:

I have released v0.2.0! https://github.com/yuuki1/gokc/releases/tag/v0.2.0


More typing on context

I had started this a while ago and then forgot about it. Just some more type annotations.

Seemed like it should be easy, but there is a mypy error. Will investigate:
src/opendp/context.py:273: error: Incompatible return value type (got "tuple[Measure, float | None]", expected "tuple[Measure, float | tuple[float, float]]")  [return-value]
src/opendp/context.py:277: error: Incompatible return value type (got "tuple[Measure, tuple[float | None, float]]", expected "tuple[Measure, float | tuple[float, float]]")  [return-value]
Found 2 errors in 1 file (checked 75 source files)


Add ACL patches

Here are the patches, rebased.

CI: https://ci.nodejs.org/job/libuv-test-commit/2059/


Conversions of Any arrays

Calls like lines(Any[1,2,3]) currently error. It would be nice if they didn't, since you end up with Any arrays by doing
data = []
for step in steps
    # do some work
    push!(data, result)
end

I think we could handle this with something like identity.(arr) in a single-argument conversion call. If the array type afterwards is not Any or can otherwise be handled, it would work


pacman -> pellet eating

When a ghost is very close to pacman while it is consuming pellets, it will cause some of the ghost to be "left behind" in video memory. Pellet eating involves wiping the video memory at the location rather than performing an xor with the current video memory, which means that part of the ghost is wiped and on the next ghost xor instead of removing the previous ghost it removes the part that wasn't wiped and draws back in the part that was wiped.

Fixed


Consider add a "ng add" schematic?

Would you like to make a ng add schematics for this project? That will be much easier upgrade existing Angular CLI project from Protractor to WebDriverIO. Thanks!

Sure. Just give me some time.

I'm interested in this enhancement too. Have you made any progress on this? Can you point me to any references on how to manually do this while we're waiting for this enhancement to be complete?


"name contains" queries seem very flaky...

At first we thought it was only matching on a prefix, but after doing a few tests I found it even more confusing. It seems like name contains really isn't reliable?
Environment details

OS type and version: OS X, Linux
Python version: 3.7.6
pip version: 20.0.2
google-api-python-client version: 1.8.0

Steps to reproduce

Put a bunch of files in a drive folder.
Try to search for them based on stuff in their names.

Code example
# example
folder_id = "...[omitted]..."

# make some test files
for n in (
    "ASDAS_casfa-1341_asdas.csv",
    "123_asdasdas-SDASDA-xcbvcbv.xlsx",
    "fdsfs_asdasd-1231_asda.csv"
):
    with drive.put_file(drive_service, folder_id, n, binary=False):
        ...

for q in ('ASDAS', 'SDASDA', 'asda', 'sd-1231'):
    query = (
        f"'{folder_id}' in parents"
        f" AND trashed = false"
        f" AND name contains '{q}'"
    )
    print(
        drive_service.files()
        .list(q=query, pageSize=4)
        .execute()["files"]
    )

Results
Generates the output:
python
[{'kind': 'drive#file', 'id': '1YbHpV4QMJrn0eFuyo7jE4nwTwFvFoqZl', 'name': 'ASDAS_casfa-1341_asdas.csv', 'mimeType': 'text/csv'}]
[{'kind': 'drive#file', 'id': '10pwge2EK3rl40XPe-_WlEIeKbrGaDbUM', 'name': '123_asdasdas-SDASDA-xcbvcbv.xlsx', 'mimeType': 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet'}]
[{'kind': 'drive#file', 'id': '1YbHpV4QMJrn0eFuyo7jE4nwTwFvFoqZl', 'name': 'ASDAS_casfa-1341_asdas.csv', 'mimeType': 'text/csv'}]
[]

Note that the third query should match all the files, while the last should return one.

Hi!
It looks like this is feedback related to the Drive API itself. Please follow the instructions here to file an issue with the Drive API.
Thanks!


Fix: strip spaces from names and email in Person, ProfileUpdateRequest

Spaces are stripped upon saving the object.
This fixes #612, #619.

:+1:.


Added md for new adaptor jixie

Also submitted jixie adaptor implementation and test code in the Prebid.js repo (pull request 5751 in that repo) just now

Link to PBJS PR:
https://github.com/prebid/Prebid.js/pull/5751

adapter files merged in prebid.


Simplify build

This change was to move functionality found in build.sh files and Grunt configuration into the npm build with the goal of simplifying the build process by reducing the number of tools required.

Coverage remained the same at 39.537% when pulling 607897b46cdd035f183e52359b41e5b4a5e20e28 on jsamsa:simplify-build into 230549a2c88c0ff95edcecd794914bf4124bc5dd on UW-Madison-DoIT:master.

I like it.  Does anything change in our documentation here: https://github.com/UW-Madison-DoIT/uw-frame/blob/master/README.md or  https://github.com/UW-Madison-DoIT/uw-frame/blob/master/docs/markdown/quickstart.md which results in the io page here being updated: http://uw-madison-doit.github.io/uw-frame/latest/#/md/quickstart
?

Fixed missing rimraf issue
Updated documentation
Modified Dockerfile

Coverage decreased (-0.4%) to 39.157% when pulling 201d67e618a6d172127f9105d5f79aa42f31de7a on jsamsa:simplify-build into 230549a2c88c0ff95edcecd794914bf4124bc5dd on UW-Madison-DoIT:master.

:+1: less is more!


Fix problem in Firefox webextension content script

For anyone that finds their way here from crashes in a Firefox extension environment, I've merged this branch with 1.3.0 here:
https://github.com/conoremclaughlin/reflect-metadata/commit/9a5223f018c7dde66278f0a30a9fb4901e7aa7d4
You can fork it and use it for solving any reflect-metadata problems you may have in Firefox. Good luck!

I'm not comfortable taking this PR as I'm not familiar with what window.wrappedJSObject is, and how this change might affect other consumers of the library. The test seems fairly fragile as well, as indicated by @dianlight's comment.
I'd like to understand more about what the specific issue is to determine whether this change is acceptable, or if there is another change that could be made in its place. If this is still a concern, please file or provide a link to an issue regarding the underlying cause for further discussion.


Fatal error on make transactions method on 2.0.0 version

Hello, Thank's for this wonderful tool!
I'm testing the new release 2.0.0 version and getting fatal error on the make transaction method on LoggerStore class. The error is : Not enough bits to represent the passed value on transaction.negotiatedTLSCipherSuite variable, as shown in the image.
I tested before the 2.0.0 beta 2 version and the error did not occur.

Thanks for the report. It looks like I need to use a larger fix-width type.

Thank you!


Out of Memory exception after starting and stopping the camera several times.

I noticed that if you start and stop the camera several times, the memory fills up until you get an OOM exception. After some investigating with the memory profiler I noticed that there's a huge hash map that keeps getting bigger, namely mBytesToByteBuffer.
The reason for that is that mBytesToByteBuffer is never cleared when CameraSource is paused, but when createCamera() is called, 4 new fairly large byte buffers are created.
My fix is to call mBytesToByteBuffer.clear() in createCamera() right before the 4 calls to camera.addCallbackBuffer(mPreviewSize);

Yes, this issue was recently discovered by another developer.  See the pending change below.  I'll try to get this committed today.
https://github.com/googlesamples/android-vision/pull/35#issuecomment-146794685

Whoops, alright, I guess this can be closed then. Thank you.


WFLY-5771 fix SerializablePermission("enableSubclassImplementation") …

…requirement for subclasses of org.omg.CORBA_2_3.portable.Output/InputStream
CDRInputStream constructors has to be invoked inside of privileged block.
Details in
https://issues.jboss.org/browse/WFLY-5771

@tadamski could you review please?

@istudens ok, will look at it today

:thumbsup:
everything ok from my perspective


Pep Rally Tomorrow

Everything is looking nice. Just wish we had more team photos. Other than that, I think everything is great. Even after announcing tomorrow at the Pep Rally, we can apply changes and add stuff whenever.
@theseanconn What is the status on the cards with the website bar code? Are we handing those out? Also, who is speaking tomorrow? If we need speaker(s) I am willing (not afraid to talk to large crowds).

Answering in order:
We have printed out ~ 1,200 individual QR code "index" cards to give to the 8th period teachers to give out to students when they arrive back from the pep rally. We have also printed out 90 flyers for teachers to hang in their rooms and also around the school.
I am not sure who is speaking tomorrow but we do have 2 speakers already and there is a script written. I appreciate your willingness to do it.
@Ernie3


feat: added Wishlist page

Wishlist page
#93
Changes
Here's a new feature introduced that enables the user to add & remove products from the Wishlist page .
The user can simply click on the Wishlist option , and do check their wishlisted products.
https://github.com/user-attachments/assets/8a977862-c15b-489e-a4ca-d16f6f4ff3c2

Hey @Saloni1707. Few things

Do fill the PR description and related fields, It's to be populated not to be left blank.
Make sure to include proof of work/screenshots of what you did, since this is a UI PR.
Please change the PR title to pass the tests. feat: added wishlist page.

Looks good @Saloni1707. I can see the user is not signed in when you added the item in wishlist, so there would be no sense to have a list of loved items if the users can't track what they actually saved. Here is what I suggest:

Make this option available only after signin.
For each user, store the items they have wishlisted (you'll need to make a db change).
When the user fetches the list, they should be able to see the complete info of their products, not just the name.


Respect user input aborts

Fixes #12
The callback of ui.input returns nil if a users aborts the dialog. This should be respected.
With the changes, the decision to abort an input with e.g., Ctrl+c (or any keymap a user has set to cancel a DressinInput dialog)  will correctly not add a bookmark. Adding an empty bookmark will still be possible when hitting Enter in the empty input dialog as it is not registered as abort.

Thanks for your help!~

I thank You!