IGListKit how to support waterfall？

as title

Sorry we need a lot more detail before can be of any use.
Sent with GitHawk

Maybe you're talking about the common waterfall style collection view layout?
https://github.com/chiahsien/CHTCollectionViewWaterfallLayout (for example)
If so there's another discussion on the matter at #877 - Otherwise, as Ryan said, please give a little bit more information so we can help you better :)


Use configuration in TaskAttemptContext

What changes were proposed in this pull request?
ContextUtil.getConfiguration(job) is configuration in driver
context.getConfiguration is the correct configuration for executor
How was this patch tested?
Unit test

LGTM

I'll merge it by today


enhancement: Info Request

Hey @ziflex ,
Sometimes we are billed per request on the hosting of an API;
it would be possible to have an option on the post / to retrieve the info in the process?
{
  data: ...
  info: ...
}

I have a quick look for PR .. I have to dig a little GO before 😅
It's my last step to align all my containers on your worker and trash my fork 🤞

Hey, not sure I understand what you want.
Could you elaborate more on that?

@ziflex, Yep, sorry for the lack of precision.
Some providers will charge per request to the worker, the number of requests and its execution time etc.
If we want to retrieve the information for each triggered scrap, we go to two requests per scrap ( POST / & GET /info )
My idea was to include a new parameter on the post request.
Query {
    text: String!
    params: Map<string, any>
    info: Boolean?
}

and if boolean is true return something like
{
  data: post request content []
  info: get /info request content []
}

Hmm, I do not like the idea of putting payload from another endpoint.
What particular information are you interested in ? We could use response headers for that and set up the behavior on a worker launch.

Hmm, I understand, and you are right. All the information, the goal would be to have the scrap and information about its execution in a single request (ip / chrome / ferret).
Perhaps a more straightforward solution would be to add a route running the other two? something like POST / debug (This is what I wanted PR to separate the use-case well but too little knowledge of GO to do it)

We could add this information to response headers.
What exactly do you need from /info? Everything?

hey, yep 🙂
the idea is to get all necessary informations to debug


Add missing closing parenthesis

Codecov Report

Merging #112 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #112   +/-   ##
=======================================
  Coverage   80.54%   80.54%
=======================================
  Files           2        2
  Lines         185      185
=======================================
  Hits          149      149
  Misses         36       36

Impacted Files
Coverage Δ

src/HttpServer.jl
79.71% <ø> (ø)
:white_check_mark:

Continue to review full report at Codecov.

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


Message-ID contain quoted-printable encoded words

This is not strictly a WebClient bug report, but rather a backend bug report.
One of my email's recipients recently noticed ProtonMail is sending quoted-printable encoded Message-ID headers 1. It looks like this:
Message-ID: =?utf-8?q?=3CwJ6Gixv2LKcZ9Gsu3OPG3UQYBNFbsSoJC79LXjCmEEV1bmhxI74?=
 =?utf-8?q?hqauM26AMFfIaih1MrukUrJhF---575PI5q4J6Eq5UiVHHfIYBuCB8SA=3D=40eme?=
 =?utf-8?q?rsion=2Efr=3E?=

However per RFC 5322 this is incorrect. The grammar specifies:
message-id      =   "Message-ID:" msg-id CRLF
msg-id          =   [CFWS] "<" id-left "@" id-right ">" [CFWS]

RFC 2047 section 6.1 states conditions in which a header field can contain encoded words, and the Message-ID field does not qualify.
This causes issues with other email clients, which ignore the Message-ID field, thus the In-Reply-To field is not set when they reply and that breaks the threading on their end.

I can't reproduce this, can you? My theory is that the quoting is being done by some mailing list software somewhere before it reaches the final recipient.

Maintainer of said mailing list here. Looking into that possibility now.

Yep. It does a round-trip through the Python email parser and I needed an extra flag to prevent the header from being mangled. Fixed on my end.

Thanks for taking the time to investigate, and sorry for this bug report!


RSDK-8566 Send gRPC heartbeats from signaling server to answerer

https://viam.atlassian.net/browse/RSDK-8566
Sends heartbeats from signaling server to signaling answerer.
I'll add tests if reviewers agree with the general design.
(cc @npmenard)

Code changes look perfectly fine to me. Just want to leave as much context in the form of code documentation on how we think these messages help for future readers.


Monkey-patched functions called by unmangled functions preserve old definition, unless called by a mangled function

The title's a little long, but this is an oddly specific issue:
def test(msg)
    puts "Old definition! " + msg
end

fun handler_fun() : Void
    test "Testing 123"
end

def handler_def()
    test "Testing 123"
end

def test(msg)
	puts "New definition! " + msg
end

handler_fun
#handler_def

results in
Old definition! Testing 123

Uncomment the last line and this happens:
New definition! Testing 123
New definition! Testing 123

Note that the use of arguments in the monkey-patched function is important (otherwise both function calls use the new definition, as expected).
My original hypothesis was that the fun was being instantiated at the definition site, while the def was being instantiated at the call site, but this doesn't account for the behaviour seen above.

Duplicate of #1104 . We need to define what to do with method definitions and code that uses them.


Automatically build bundled libsecp256k1 during install

This PR adds the ability to install this wrapper without the need
to have a prior installation of libsecp256k1 on one's system.
This works as follows:

During 'sdist':
If the directory 'libsecp256k1' doesn't exist in the
source directory it is downloaded from the location specified
by the LIB_TARBALL_URL constant in setup.py and extracted into
a directory called 'libsecp256k1'
To upgrade to a newer version of the bundled libsecp256k1 source
simply delete the 'libsecp256k1' directory and update the
LIB_TARBALL_URL to point to a newer commit.

During 'install':
If an existing (system) installation of libsecp256k1 is found
(either in the default library locations or in the location pointed
to by the environment variable LIB_DIR) it is used as before.
Due to the way the way cffi modules are implemented it is necessary
to perform this detection in the cffi build module
'_cffi_build/build.py' as well as in 'setup.py'. For that reason
some utility functions have been moved into a 'setup_support.py'
module which is imported from both.
If however no existing installation can be found the bundled
source code is used to build a library locally that will be
statically linked into the CFFI extension.
By default only the 'recovery' module will be enabled in this bundled
version as it is the only one not considered to be 'experimental' by
the libsecp256k1 authors. It is possible to override this and enable
all modules by setting the environment variable
SECP_BUNDLED_EXPERIMENTAL.

Additionally there are some small other optimizations:

The source code has been moved into a package so that site-packages
top-level doesn't get cluttered with the cffi generated source files.

The cffi build script has been moved to a '_cffi_build' directory
since it is not needed at runtime.

The C-definitions used by cffi have been extracted into their own
files inside the '_cffi_build' directory to make the build script more
readable.

+1

+1

Does it work on Windows?

+1

@ludbb any chance we get this merged? a lot of work went into this.

Looking at this now, thanks for all the effort.

Looks great, you should put your name on the setup.py as this is a very great contribution.

Thanks for merging :)
I have two small changes I'd like to add (lowering the minimum required setuptools version and a check for pkg-config). I'll open a separate PR for those.


Create Listener - Windows Permissions - prestage the computer account for a listener virtual network name - missing step

[Enter feedback here]
Hi,
I've gone through this process:
https://docs.microsoft.com/en-us/sql/database-engine/availability-groups/windows/create-or-configure-an-availability-group-listener-sql-server?view=sql-server-ver15#WinPermissions
I've prestaged the computer account for a listener virtual network name (we can't let the computer have the create computer premission) , configured dns - and when i tried to create the listener i've got the following error:
"The WSFC cluster could not bring the Network Name resource with DNS name ‘’ online. The DNS name may have been taken or have a conflict with existing name services, or the WSFC cluster service may not be running or may be inaccessible. Use a different DNS name to resolve name conflicts, or check the WSFC cluster log for more information.
The attempt to create the network name and IP address for the listener failed. If this is a WSFC availability group, the WSFC service may not be running or may be inaccessible in its current state, or the values provided for the network name and IP address may be incorrect. Check the state of the WSFC cluster and validate the network name and IP address with the network administrator. Otherwise, contact your primary support provider. (Microsoft SQL Server, Error: 19471)"
I've been able to resolve the issue by following this link:
https://stevethompsonmvp.wordpress.com/2018/10/17/failed-to-create-availability-group-listener/
The Steps:
On the computer account for a listener virtual network name , i've added on the Security tab The cluster object name (CNO) with Full Premission.
After that I managed to create the listener :)
Please add this step on this web page or if there is already documentation about this step (I haven't found it) please reference it

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

ID: b679be86-06b0-b425-cd4f-d7b3ea104693
Version Independent ID: aec2cf5b-5bf9-647e-6b0f-9fab6fca3a79
Content: Configure availability group listener - SQL Server Always On
Content Source: docs/database-engine/availability-groups/windows/create-or-configure-an-availability-group-listener-sql-server.md
Product: sql
Technology: availability-groups
GitHub Login: @cawrites
Microsoft Alias: chadam

@orbiton21 -- Or, thank you for your thorough feedback.
@cawrites -- Cheryl, please look into this issue.

@orbiton21 -- Or, thank you for your thorough feedback.
@cawrites -- Cheryl, please look into this issue.

@orbiton21 we are reviewing the information and we will provide an update when it is available.


Add profiling

Replaces (outdated) CodeMarkers with EventSource. Also adds a few new locations for profiling in LazyItemEvaluation.

This needs validation before being merged.

/azp run

/azp run


Bug when setting alpha values for a sprite inside container

Version 3.19 and lower

Phaser Version: 3.19

Operating system: Windows 10x64

Browser: Chrome

Description
Setting the alpha topLeft, topRight, botLeft, botRight alpha values on a sprite inside a container will cause the command being ignored, so, calling setAlpha(1, 1, 0, 0) on these sprites will do nothing

Example Test Code ->  (Code Took and modified from phaser3 examples container alph)

   var image0 = this.add.image(0, 0, 'lemming');
    var image1 = this.add.image(-100, -100, 'lemming');
    var image2 = this.add.image(100, -100, 'lemming');
    var image3 = this.add.image(100, 100, 'lemming');
    var image4 = this.add.image(-100, 100, 'lemming');
    container = this.add.container(400, 300, [ image0, image1, image2, image3, image4 ]);
    image1.setAlpha(0, 0, 1, 1);

Additional Information
When calling with the four parameters, it will just make the sprite opaque

Yes, this is a known limitation of Containers. I have no immediate plans to change this, although I would happily accept a non-invasive PR that did so.

What would be a non invasive pr? I could fix this but I don't know how where the render call really happens, it is quite hidden you know, how can I find the render call
PS: can update on the documentation that this don't work? I lost some time trying to find if it were bug on my code

The WebGL render call for a Container is made in the not-so-hidden file ContainerWebGLRenderer.
The issue is where the Container alpha is multiplied by the Child alpha, so that changing the alpha of a container impacts children. It uses the global alpha value, not the per-vertex one. A non-invasive PR means one that replicates this functionality unless per-vertex alpha has been used.

I made the new pull request

Thanks for opening this issue, and for submitting a PR to fix it. We have merged your PR into the master branch and attributed the work to you in the Change Log. If you need to tweak the code for whatever reason please submit a new PR.


map bytes to Uint8array

Uint8Array is a js native representation for an array of unsigned bytes, the current Array mapping has bad performance characteristics, especially when working with large buffers. I'd rather leave the Array conversion to the user than making it part of the generator.

I ran yarn build and yarn test locally to confirm this doesn't break anything, but noticed that I had a bunch of changed files not sure why. LMK if i should reset some of them, or if you want to merge them.

I ran yarn build, but the npm package doesn't work on my platform (darwin/arm64):
Only x86_64 / Linux distributed in NPM package right now.
Trying globally installed anchor.

Ah that makes sense. It ran your globally installed version of anchor which is < anchor 0.25 and that's why the docs aren't generated.
No worries, I'll build it before merging.

Alright, looks good overall! I'll finish it up from here with the build. Thanks for the PR!


Routing outbound traffic from pods on node with iptables not working

I am trying to get something like kube2iam up and running with microk8s.
Basically the goal is to hijack traffic for all pods on a node that are making calls to http://<IP_ADDRESS>/latest/meta-data/iam/security-credentials and route them to a special endpoint in a container to turn annotations into AWS IAM credentials.
The service runs the following command at boot time.
iptables -t nat -A PREROUTING -p tcp -d ${metadataAddress} --dport 80 -j DNAT --to-destination ${hostIp}:${appPort} -i ${hostInterface}

example
iptables -t nat -A PREROUTING -p tcp -d <IP_ADDRESS> --dport 80 -j DNAT --to-destination <IP_ADDRESS>:8181 -i docker0

This works on Minikube but doesn't work on Microk8s
MiniKube works
iptables -t nat -v -L PREROUTING -n --line-number
Chain PREROUTING (policy ACCEPT 15 packets, 1691 bytes)
num   pkts bytes target     prot opt in     out     source               destination
3        0     0 DNAT       tcp  --  docker0 *       <IP_ADDRESS>/0            <IP_ADDRESS>      tcp dpt:80 <IP_ADDRESS>:8181

microk8s
iptables -t nat -v -L PREROUTING -n --line-number
Chain PREROUTING (policy ACCEPT 15 packets, 1691 bytes)
num   pkts bytes target     prot opt in     out     source               destination
3        0     0 DNAT       tcp  --  docker0 *       <IP_ADDRESS>/0            <IP_ADDRESS>      tcp dpt:80 to:<IP_ADDRESS>:8181

Does something about mircrok8s and snaps make this impossible or is there a special configuration?

I am not sure exactly what may be wrong but here are some thoughts. Comparing the minikube and microk8s cases, what is the <IP_ADDRESS>:8181 and why is this different than the <IP_ADDRESS>:8181? The pods are on <IP_ADDRESS>/16 and microk8s does not use docker0 I suspect the  <IP_ADDRESS> is where the translation service is?
Could you please use the edge channel of microk8s? We recently addressed a routing issue described in https://github.com/ubuntu/microk8s/issues/1139

What is the <IP_ADDRESS>:8181 and why is this different than the <IP_ADDRESS>:8181?

Running k get pod kube2iam-96d4d86db-b6gl6 -o yaml on minikube
...
status:
  ...
  hostIP: <IP_ADDRESS>
  podIP: <IP_ADDRESS>
  podIPs:
  - ip: <IP_ADDRESS>

So its the IP of the virtual image where as on microk8s its the ip of the physical server (<IP_ADDRESS>) that I am running ubuntu 20.04 on.
The 8181 comes from the daemon set manifest
...
          ports:
            - containerPort: 8181
              hostPort: 8181
              name: http
...

The pods are on <IP_ADDRESS>/16 and microk8s does not use docker0 I suspect the <IP_ADDRESS> is where the translation service is?

I can override this with a flag:
fs.StringVar(&s.HostInterface, "host-interface", "docker0", "Host interface for proxying AWS metadata")

Which is used here
	return ipt.AppendUnique(
		"nat", "PREROUTING", "-p", "tcp", "-d", metadataAddress, "--dport", "80",
		"-j", "DNAT", "--to-destination", hostIP+":"+appPort, "-i", hostInterface,
	)

Could you please use the edge channel of microk8s? We recently addressed a routing issue described in #1139

I gave latest/edge a go and it didn't work, but I wonder if its because I am using docker0 and I need to try a different value there.

Digging into this more, I found this:
https://askubuntu.com/a/579540

I think this might be my issue here, let me do a little spike and adjust the command and report back.

Okay, running
iptables -t nat -A OUTPUT -p tcp -d <IP_ADDRESS> --dport 80 -j DNAT --to-destination <IP_ADDRESS>:8181

Does have the effect I want, where as with Minikube you can use PREROUTING


RuntimeError: module compiled against API version 0xe but this version of numpy is 0xd

I'm building JAX from source on a Macbook Pro M1. The compilation finishes but after installing the wheel in a virtualenv I get a RuntimeError when trying to import JAX along with an ImportError:
RuntimeError: module compiled against API version 0xe but this version of numpy is 0xd
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/ben/devel/jax/jax/__init__.py", line 37, in <module>
    from . import config as _config_module
  File "/Users/ben/devel/jax/jax/config.py", line 18, in <module>
    from jax._src.config import config
  File "/Users/ben/devel/jax/jax/_src/config.py", line 27, in <module>
    from jax._src import lib
  File "/Users/ben/devel/jax/jax/_src/lib/__init__.py", line 75, in <module>
    import jaxlib.lapack as lapack
  File "/Users/ben/.pyenv/versions/run_jax_1/lib/python3.9/site-packages/jaxlib/lapack.py", line 21, in <module>
    from . import _lapack
ImportError: ImportError: numpy.core.multiarray failed to import

I've built JAX in a virtualenv (python 3.97) with numpy==1.19.5 since that's required by tensorflow_macos. The bazel version is 4.10, installed via homebrew. Bazel also prints that the numpy version is 1.19.5 so I assume that's what's being used.
Is this version of numpy just fundamentally not compatible with the head of JAX (despite the minimum version mentioned in setup.py being 1.18.0)? Or do I have to specify the numpy version somewhere else as well? Thanks for any pointers!

I think this means that you have built your jaxlib against NumPy 1.20. Version 0xe is 1.20:
https://github.com/numpy/numpy/blob/f8e87f8d5892dd5fe11ddd05470ae83a0104d417/numpy/core/setup_common.py#L44
Please double-check that you built jaxlib against a NumPy version that is at least as old as the version you wish to use it with.
By the way, we do have (lightly tested) Mac ARM jaxlib wheels available, which you should be able to install with:
pip install jaxlib -f https://storage.googleapis.com/jax-releases/jax_releases.html

We don't yet release the wheels to pypi because we have no CI testing for them.
Feel free to reopen this issue if the suggestions above don't work for you.

Thanks for the quick reply! Do you know where I can define what version of numpy should be used? I was assuming it would use whatever is in the path. Since Bazel prints the following I assumed it was using 1.19.5 to build jaxlib:
Bazel binary path: /opt/homebrew/bin/bazel
Bazel version: 4.1.0
Python binary path: /Users/ben/.pyenv/versions/build_jax/bin/python
Python version: 3.9
NumPy version: 1.19.5
MKL-DNN enabled: yes
Target CPU: arm64
Target CPU features: release
CUDA enabled: no
TPU enabled: no
ROCm enabled: no

I've also seen the wheels for ARM but unfortunately I think they are all built against a newer numpy version as well so I get the same error. The only reason I'd need to use 1.19.5 is tensorflow_macos' requirement, and unfortunately there's no way to build it yourself.


Remove upgrade text from AWS doc

Fixes #582

Nice. It would be good to link the IPIP section in master/reference/calicoctl/resources/ippool to the new IP-in-IP page.  I think the paragraph at the bottom of the resources page can be removed if that link is added too.
I have a PR #592 up from a support issue I was reviewing which I think would better fit on the IP-in-IP page you've created. Feel free to add something like it or I can rebase my PR after this is in.

What's your plan for general pool configuration docs? One option could be renaming this new ip-in-ip.md to be configuring-pools.md and adding in the FAQ section on how to enable generic IPIP and nat-outgoing

Just want to say that I think you've nailed the organization and location of this info. Just going to be syntax & wording nit's from here on out mostly

Oh yeah don't forget to apply these changes to the v2.1 section too.

LGTM, hit the button @robbrockbank


namespaced events and states can be confused

This is particularly confusing with multiple state_machines.  An event called by #canceled_payment in the payment namespace should not be possibly confused with canceled in another, though both would be logged as such in the database.
It appears that #state_name (with no namespace) and #payment_state_name (in a payment namespaced state_machine) both yield canceled instead of a full state name like you would expect from the generated event method name.

The best options appears to be to add the namespace field to the state transition.

:+1:

what about machine ?

Well, since the attribute on machine is namespace, I figured it might be best to label it as such.
 state_machine :alarm_state, initial: :active, namespace: :'alarm' do

So our field/value is namespace: 'alarm'

:shipit:

resolved with commit 61548c0bfb4ed49291f06d412b264b8a8c188708 and released as 1.0.1


Resizing all widgets

Is there any way to resize all widgets on the grid??

@billkim84 You can loop through each widget and call resize - https://github.com/troolee/gridstack.js/tree/develop/doc#resizeel-width-height

Thanks for replay. I wonder is there any way user can do it not by programmatically

@billkim84 You mean something like selecting multiple (or all) widgets and resize all selected widgets?

@profesDevelopment yes

@billkim84 No, there is no way to do this


CANT DIE in the weaponless level.

 I already got damaged at least 200 times and nothing more happens.

FIXED, i changed the names haha


[api-minor] Add support for relative URLs, in both annotations and the outline, by adding a docBaseUrl parameter to PDFJS.getDocument (bug 766086)

Please refer to the individual commit messages.
Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=766086.
(Note: This PR attempts to implement https://github.com/mozilla/pdf.js/pull/7212#issuecomment-216678039, hence it supersedes that one.)

This change is

/botio unittest

/botio unittest

/botio-linux preview

/botio unittest

I like that we can pass base URI of the document. I wonder if make page object referring its document and annotation object referring its page will allow us avoid passing base URI to the other API functions, e.g. annotation.page.document.baseUrl might be more easier to maintain.

I like that we can pass base URI of the document. I wonder if make page object referring its document and annotation object referring its page will allow us avoid passing base URI to the other API functions, e.g. annotation.page.document.baseUrl might be more easier to maintain.

Thank you for the feedback!
I'm not sure that I completely understand your preferred changes, but after looking through the code a bit it seems to me that adding docBaseUrl (only) to PDFManager may very well be the cleanest solution.
Doing this removes the need to pass a single parameter around, and we already have access to a PDFManager instance in both PDFDocument and Page, which should reduce the maintenance burden.
As far as I can see, we still need to pass something in when creating the Annotations, but with this now being a PDFManager instance it should at least be better than previously.
I'm hoping that the updated PR is more what you had in mind, but please let me know otherwise!
Note: I added a new WIP commit with the above changes, and I'll fold it into the previous commit once the PR is ready to land.

Comments have been addressed, thank you for the review!
Besides updated/improved unit-tests, the only functional changes were made in the second commit, and to simplify auditing them I'm including the diff below:
diff --git a/src/core/obj.js b/src/core/obj.js
index 2e0b607..bbbfe23 100644
--- a/src/core/obj.js
+++ b/src/core/obj.js
@@ -594,7 +594,7 @@ var Catalog = (function CatalogClosure() {
   Catalog.parseDestDictionary = function Catalog_parseDestDictionary(params) {
     // Lets URLs beginning with 'www.' default to using the 'http://' protocol.
     function addDefaultProtocolToUrl(url) {
-      if (isString(url) && url.indexOf('www.') === 0) {
+      if (url.indexOf('www.') === 0) {
         return ('http://' + url);
       }
       return url;
@@ -629,7 +629,7 @@ var Catalog = (function CatalogClosure() {
           if (isName(url)) {
             // Some bad PDFs do not put parentheses around relative URLs.
             url = '/' + url.name;
-          } else if (url) {
+          } else if (isString(url)) {
             url = addDefaultProtocolToUrl(url);
           }
           // TODO: pdf spec mentions urls can be relative to a Base
@@ -646,7 +646,7 @@ var Catalog = (function CatalogClosure() {
             // We assume that we found a FileSpec dictionary
             // and fetch the URL without checking any further.
             url = urlDict.get('F') || null;
-          } else if (urlDict) {
+          } else if (isString(urlDict)) {
             url = urlDict;
           }

@@ -686,12 +686,13 @@ var Catalog = (function CatalogClosure() {
         default:
           warn('Catalog_parseDestDictionary: Unrecognized link type "' +
                linkType + '".');
+          break;
       }
     } else if (destDict.has('Dest')) { // Simple destination link.
       dest = destDict.get('Dest');
     }

-    if (url) {
+    if (isString(url)) {
       url = tryConvertUrlEncoding(url);
       var absoluteUrl;
       try {
@@ -700,12 +701,16 @@ var Catalog = (function CatalogClosure() {

       if (isValidUrl(absoluteUrl, /* allowRelative = */ false)) {
         resultObj.url = absoluteUrl;
-      } else {
-        resultObj.unsafeUrl = url.toString();
       }
+      resultObj.unsafeUrl = url;
     }
     if (dest) {
-      resultObj.dest = isName(dest) ? dest.name : dest;
+      if (isName(dest)) {
+        dest = dest.name;
+      }
+      if (isString(dest) || isArray(dest)) {
+        resultObj.dest = dest;
+      }
     }
   };

/botio unittest


Extend CLI help message

I'm not sure if this is something we want, but wanted to at least discuss it.
$ cookiecutter --help
Usage: cookiecutter [OPTIONS] TEMPLATE [EXTRA_CONTEXT]...

  Create a project from a Cookiecutter project template (TEMPLATE).

  Cookiecutter is free and open source software, developed and managed by
  volunteers. If you would like to help out or fund the project, please get
  in touch at https://github.com/audreyr/cookiecutter.

Options:
  -V, --version              Show the version and exit.
  --no-input                 Do not prompt for parameters and only use
                             cookiecutter.json file content
  -c, --checkout TEXT        branch, tag or commit to checkout after git clone
  -v, --verbose              Print debug information
  --replay                   Do not prompt for parameters and only use
                             information entered previously
  -f, --overwrite-if-exists  Overwrite the contents of the output directory if
                             it already exists
  -o, --output-dir PATH      Where to output the generated project dir into
  --config-file PATH         User configuration file
  --default-config           Do not load a config file. Use the defaults
                             instead
  --debug-file PATH          File to be used as a stream for DEBUG logging
  -h, --help                 Show this message and exit.

Current coverage is 100% (diff: 100%)

Merging #859 into master will not change coverage

@@           master   #859   diff @@
====================================
  Files          16     16
  Lines         658    658
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
  Hits          658    658
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 1bdf78e...cf2f284

:heart: from me.

@audreyr @pydanny? 😄

Huge ❤  from me. 😄

Awesome! Thanks everybody 🍪


containers: add support for interface "io.edgehog.devicemanager.apps.AvailableDeployments"

ref: #614
interface here

Closed by #638


Make internal ClusterInfo::getPlan() wait for initial plan load from agency

Scope & Purpose
Otherwise the maintenance, fired off by the heartbeat thread, may not wait for the initial data to arrive from the agency, which can cause Plan and Current to be empty when maintenance starts.
No backport for 3.6 is needed, as 3.6 has blocking agency communication here in case the Plan is not there.

[x] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Backports:

[x] Backports required for: 3.7 (https://github.com/arangodb/arangodb/pull/13291)

Testing & Verification

[x] The behavior in this PR was manually tested
[x] This change is already covered by existing tests, such as mini chaos.

Link to Jenkins PR run:
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13367/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13370/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13374/

Tests blue

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13382/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13395/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13462/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13462/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13465/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13465/

Tests blue

Tests blue


Documented applyMixins doesn't compile since TypeScript 2.7

The following snippet from the handbook doesn't compile anymore:
function applyMixins(derivedCtor: any, baseCtors: any[]) {
    baseCtors.forEach(baseCtor => {
        Object.getOwnPropertyNames(baseCtor.prototype).forEach(name => {
            Object.defineProperty(derivedCtor.prototype, name, Object.getOwnPropertyDescriptor(baseCtor.prototype, name));
        });
    });
}

2.7.2 Playground
2.4.1 Playground
Any suggestions on how to fix it?

Bump


Apply changes to requirements from upstream

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

I was a little to fast. This change will be for new releases > v3.20.1. Closed.


CI: Conformance AKS  - check-log-errors - EndpointPolicyVisibilityEvent event failed

CI failure
Hit on https://github.com/cilium/cilium/pull/30998
Link: https://github.com/cilium/cilium/actions/runs/8062340743/job/22021880556
Sysdump: cilium-sysdump-final-1.27-eastus2-3-true.zip
❌ Found 2 logs in cilium-cilium-8062340743-1-3/kube-system/cilium-mq98p (cilium-agent) matching list of errors that must be investigated:
level=error msg="EndpointNoTrackEvent event failed" ciliumEndpointName=kube-system/hubble-relay-d5b87984f-qhh46 containerID=69963737e3 containerInterface=eth0 datapathPolicyRevision=0 desiredPolicyRevision=0 endpointID=710 error="pod.core \"hubble-relay-d5b87984f-qhh46\" not found" identity=445 ipv4=<IP_ADDRESS> ipv6= k8sPodName=kube-system/hubble-relay-d5b87984f-qhh46 subsys=endpoint (1 occurrences)
level=error msg="EndpointPolicyVisibilityEvent event failed" ciliumEndpointName=kube-system/hubble-relay-d5b87984f-qhh46 containerID=69963737e3 containerInterface=eth0 datapathPolicyRevision=0 desiredPolicyRevision=0 endpointID=710 error="pod.core \"hubble-relay-d5b87984f-qhh46\" not found" identity=445 ipv4=<IP_ADDRESS> ipv6= k8sPodName=kube-system/hubble-relay-d5b87984f-qhh46 subsys=endpoint (1 occurrences)

There's a couple similar such EKS (+maybe AKS?) issues going on.  I believe they're caused by incomplete cleanup between cilium installs.  Waiting for a cilium-cli release to fix with this: https://github.com/cilium/cilium-cli/pull/2344

Haven't seen this failure in recent runs, closing as done for now unless it comes up again.


BUG FulltextFilter requires table identifiers in match query

See https://github.com/silverstripe/silverstripe-framework/pull/3554#issuecomment-127612834
We should use MATCHES("Table"."Column") syntax rather than matching against only column, as we could run into ambiguous column error.

Closed via 95ae107c4c39f06d409310e5ee07f0a3def81cf4


ebpf: printk patcher: Skip http2 program

What does this PR do?
Skipping patching http2 programs as it generate errors while adding bpf_printk calls

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Apply the following diff:
diff --git a/pkg/network/ebpf/c/protocols/http2/decoding.h b/pkg/network/ebpf/c/protocols/http2/decoding.h
index 648beebda4..f13cb24b95 100644
--- a/pkg/network/ebpf/c/protocols/http2/decoding.h
+++ b/pkg/network/ebpf/c/protocols/http2/decoding.h
@@ -595,6 +595,7 @@ static __always_inline bool find_relevant_frames(struct __sk_buff *skb, skb_info

 SEC("socket/http2_handle_first_frame")
 int socket__http2_handle_first_frame(struct __sk_buff *skb) {
+    bpf_printk("test");
     const __u32 zero = 0;
     http2_frame_t current_frame = {};

enable http2 monitoring and run the agent
verify the following line does not appear
usm initialization failed: could not initialize USM: error initializing ebpf program: co-re load failed: Could not find stack offset instruction for bpf_trace_printk call 3 in socket__http2_handle_first_frame

covered by https://github.com/DataDog/datadog-agent/pull/23432


support building for and from ARM

Is your feature request related to a problem? Please describe.

Currently none of the build commands (like bazel build //build:test-targets) work when run from ARM based machines (M1 Macs). This not only makes it impossible to build images but also makes cert-manager development a bit tough.
Describe the solution you'd like

Support ARM architecture natively. If not possible then a workaround with Rosetta 2 might be nice too. Shifting from Bazel to Make might be a step in this direction.
Describe alternatives you've considered

I tried running the terminal with Rosetta 2 and trying the build commands but that failed too.
Additional context

See slack discussion
/kind feature
cc @irbekrm

Hey @RinkiyaKeDad ,
Thank you for opening the issue and sorry you're having issues - we should definitely make sure cert-manager can be built/tested on darwin/arm,  linux/arm .
Could you please add some extra info`:

what version of Bazel are you running? I am guessing that you might need a version of Bazel that includes this PR ?

I am wondering if this is a problem with just the Bazel setup for cert-manager - are you able to build targets in other repos (i.e if you clone gazelle and run bazel build //:gazelle_local inside that repo- does that work?)

output of something like uname -a on your machine

error logs could be helpful- I have read that Slack discussion, but perhaps someone else sees this issue and could chime in 😄

I am currently looking at upgrading a bunch of deps including Bazel go_rules, Gazelle and Kazel.
Unfortunately we don't have any darwin/arm machines to test this on- if anyone does and can chime in with some advice that would be much appreciated 🙏🏼 !

Hi @irbekrm!

I'm running bazel 4.0.0-homebrew. That is the latest release and was released on 21st Jan but the link you mentioned is of 27th Jan so I'm not sure if that PR is included or not. I will try the instructions mentioned in this comment and get back to you.
I cloned the repo and ran that command and it worked :) This is the output I got:

bazel build //:gazelle_local                                       ✔
Starting local Bazel server and connecting to it...
INFO: SHA256 (https://golang.org/dl/?mode=json&include=all) = 64a61556e291c1695f7597037faec67fd48d87feced12541fff23f3a8400cf72
INFO: Analyzed target //:gazelle_local (76 packages loaded, 7696 targets configured).
INFO: Found 1 target...
Target //:gazelle_local up-to-date:
  bazel-bin/gazelle_local_/gazelle_local
INFO: Elapsed time: 73.751s, Critical Path: 13.38s
INFO: 95 processes: 11 internal, 84 darwin-sandbox.
INFO: Build completed successfully, 95 total actions

This is weird though. Since I don't have that commit of 27 Jan I'm not entirely sure why this worked 🤔

name -a output:

Darwin Arshs-MacBook-Air.local 20.3.0 Darwin Kernel Version 20.3.0: Thu Jan 21 00:06:51 PST 2021; root:xnu-7195.81.3~1/RELEASE_ARM64_T8101 arm64

Here are the error logs from the Slack conversation:

ERROR: /Users/arsh/Projects/Forks/cert-manager/build/BUILD.bazel:92:10: Configurable attribute "srcs" doesn't match this configuration (would a default condition help?).
Conditions checked:
 @io_bazel_rules_go//go/platform:linux_amd64
 @io_bazel_rules_go//go/platform:darwin_amd64
ERROR: Analysis of target '//build:test-targets' failed; build aborted: /Users/arsh/Projects/Forks/cert-manager/build/BUILD.bazel:92:10: Configurable attribute "srcs" doesn't match this configuration (would a default condition help?).

    Fetching @bazel_gazelle_go_repository_config; Restarting.
    Fetching @local_config_cc_toolchains; fetching
    Fetching @bazel_gazelle_go_repository_cache; Restarting.
make: *** [verify] Error 1

I can run any tests you need to perform on my machine so please feel free to let me know 😄

Thank you so much for working on this!

Thank you for the logs!
I think these logs are from running a test target which probably would not work without an explicit platform flag since we only support amd64 test platforms.
Perhaps you could include the logs for running something like bazel build //cmd/ctl/... - that I though would have worked on darwin/arm too.
Thank you very much for the offer to run tests 🙏🏼

Logs from running bazel build //cmd/ctl/...:
ERROR: While resolving toolchains for target //cmd/ctl:ctl: no matching toolchains found for types @io_bazel_rules_go//go:toolchain
ERROR: Analysis of target '//cmd/ctl:ctl' failed; build aborted: no matching toolchains found for types @io_bazel_rules_go//go:toolchain
INFO: Elapsed time: 1.346s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (30 packages loaded, 331 targets configured)

Logs from running bazel build --platforms=@io_bazel_rules_go//go/toolchain:darwin_amd64 //cmd/ctl/...:
INFO: Build option --platforms has changed, discarding analysis cache.
ERROR: While resolving toolchains for target //cmd/ctl/pkg/approve:go_default_library: no matching toolchains found for types @io_bazel_rules_go//go:toolchain
ERROR: Analysis of target '//cmd/ctl/pkg/approve:go_default_library' failed; build aborted: no matching toolchains found for types @io_bazel_rules_go//go:toolchain
INFO: Elapsed time: 0.151s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded, 292 targets configured)

Yaaaay @irbekrm!
bazel build //cmd/ctl/...  worked:
INFO: SHA256 (https://golang.org/dl/?mode=json&include=all) = 4bb9f5f7d71d826c10b10c65bcaf53847a038f646101c25de10e52d8ffae7a7e
DEBUG: /private/var/tmp/_bazel_arsh/5e27204bfc1c3de748fd0630d361c7e8/external/bazel_gazelle/internal/go_repository.bzl:189:18: com_github_emicklei_go_restful: gazelle: finding module path for import github.com/mjibson/appstats: go get: module github.com/mjibson/appstats: git ls-remote -q origin in /private/var/tmp/_bazel_arsh/5e27204bfc1c3de748fd0630d361c7e8/external/bazel_gazelle_go_repository_cache/pkg/mod/cache/vcs/444aab7ea271b6130ec26c66b91d9ad882ee59ce867f2107d31a0875343c64c2: exit status 128:
        remote: Repository not found.
        fatal: repository 'https://github.com/mjibson/appstats/' not found
INFO: Analyzed 52 targets (447 packages loaded, 11964 targets configured).
INFO: Found 52 targets...
INFO: Elapsed time: 132.449s, Critical Path: 35.97s
INFO: 1024 processes: 16 internal, 1008 darwin-sandbox.
INFO: Build completed successfully, 1024 total actions

Thanks for fixing this!

The strange thing is that bazel build //build:test-targets is still failing 🤔
Let me know if you need me to test something out 🙂

Hey, really glad we're closer to fixing this!
bazel build //build:test-targets will fail because we have a hardcoded set of test platforms that doesn't include arm, but I will create an issue and add that.
Out of interest, does bazel test --platforms=@io_bazel_rules_go//go/toolchain:darwin_amd64 //build:test-targets fail too?

Let me know if you need me to test something out
Thanks for all the help so far! Once I add the arm targets, it would be great if you could try if it works 😄

Out of interest, does bazel test --platforms=@io_bazel_rules_go//go/toolchain:darwin_amd64 //build:test-targets fail too?

Yup that works too. It says "Build completed successfully" after running that command but I do see some weird messages in the output:
remote: Repository not found.
        fatal: repository 'https://github.com/onsi/B/' not found
gazelle: finding module path for import github.com/onsi/C: go get: module github.com/onsi/C: git ls-remote -q origin in /private/var/tmp/_bazel_arsh/5e27204bfc1c3de748fd0630d361c7e8/external/bazel_gazelle_go_repository_cache/pkg/mod/cache/vcs/1ff2d3d933f213cc968d43422e39ca0f1e3925d1e9711e8d6938d480160380fc: exit status 128:
        remote: Repository not found.
        fatal: repository 'https://github.com/onsi/C/' not found
gazelle: finding module path for import github.com/onsi/C: go get: module github.com/onsi/C: git ls-remote -q origin in /private/var/tmp/_bazel_arsh/5e27204bfc1c3de748fd0630d361c7e8/external/bazel_gazelle_go_repository_cache/pkg/mod/cache/vcs/1ff2d3d933f213cc968d43422e39ca0f1e3925d1e9711e8d6938d480160380fc: exit status 128:
        remote: Repository not found.
        fatal: repository 'https://github.com/onsi/C/' not found

Although I don't think so they have something to do with this issue.

I do see some weird messages in the output

Those are currently expected, see i.e this successful e2e test run on prow, it's not an error.

Yup that works too. It says "Build completed successfully" after running that command

That's really interesting, so you can run code compiled for amd64 on your M1 Mac! I guess it's the Rossetta kicking in automatically? In any case, glad it works!

No no, as far as I know, Rosetta doesn't kick in automatically and I for one didn't launch VS Code or the terminal with it so it is definitely running natively. Thanks to your PRs :)

I've tried adding arm64 to test platforms in #3871
@RinkiyaKeDad if you at some point had a chance to checkout my branch for that PR to see if you can now run bazel test //...  without a platform flag that would be much appreciated 😄

Hey @irbekrm! I tested the changes but they failed :(
Here are the logs
Starting local Bazel server and connecting to it...
ERROR: /Users/arsh/Projects/Forks/cert-manager/hack/bin/BUILD.bazel:83:8: Configurable attribute "srcs" doesn't match this configuration (would a default condition help?).
Conditions checked:
 //hack/bin:k8
INFO: Repository kustomize_osx instantiated at:
  /Users/arsh/Projects/Forks/cert-manager/WORKSPACE:99:17: in <toplevel>
  /Users/arsh/Projects/Forks/cert-manager/hack/bin/deps.bzl:28:22: in install
  /Users/arsh/Projects/Forks/cert-manager/hack/bin/deps.bzl:89:17: in install_kustomize
Repository rule http_archive defined at:
  /private/var/tmp/_bazel_arsh/5e27204bfc1c3de748fd0630d361c7e8/external/bazel_tools/tools/build_defs/repo/http.bzl:336:31: in <toplevel>
INFO: Repository jq_osx instantiated at:
  /Users/arsh/Projects/Forks/cert-manager/WORKSPACE:99:17: in <toplevel>
  /Users/arsh/Projects/Forks/cert-manager/hack/bin/deps.bzl:20:17: in install
  /Users/arsh/Projects/Forks/cert-manager/hack/bin/deps.bzl:112:14: in install_misc
Repository rule http_file defined at:
  /private/var/tmp/_bazel_arsh/5e27204bfc1c3de748fd0630d361c7e8/external/bazel_tools/tools/build_defs/repo/http.bzl:426:28: in <toplevel>
INFO: Repository buildozer_darwin instantiated at:
  /Users/arsh/Projects/Forks/cert-manager/WORKSPACE:99:17: in <toplevel>
  /Users/arsh/Projects/Forks/cert-manager/hack/bin/deps.bzl:22:24: in install
  /Users/arsh/Projects/Forks/cert-manager/hack/bin/deps.bzl:168:14: in install_bazel_tools
Repository rule http_file defined at:
  /private/var/tmp/_bazel_arsh/5e27204bfc1c3de748fd0630d361c7e8/external/bazel_tools/tools/build_defs/repo/http.bzl:426:28: in <toplevel>
INFO: Repository kube-apiserver_darwin_amd64 instantiated at:
  /Users/arsh/Projects/Forks/cert-manager/WORKSPACE:99:17: in <toplevel>
  /Users/arsh/Projects/Forks/cert-manager/hack/bin/deps.bzl:21:42: in install
  /Users/arsh/Projects/Forks/cert-manager/hack/bin/deps.bzl:121:14: in install_integration_test_dependencies
Repository rule http_file defined at:
  /private/var/tmp/_bazel_arsh/5e27204bfc1c3de748fd0630d361c7e8/external/bazel_tools/tools/build_defs/repo/http.bzl:426:28: in <toplevel>
ERROR: Analysis of target '//devel/cluster:create-openshift3' failed; build aborted: /Users/arsh/Projects/Forks/cert-manager/hack/bin/BUILD.bazel:83:8: Configurable attribute "srcs" doesn't match this configuration (would a default condition help?).
Conditions checked:
 //hack/bin:k8
INFO: Elapsed time: 20.066s
INFO: 0 processes.


Feature request: search command history

Would be nice to have a function that searches the user's command history. It looks like there is an option to search available commands, but not the command history.

Checkout the new command FuzzyCmdHistory. I haven't tested it throughly yet. Please let me know if you run into any bug.


frawk: init at 0.4.6

Description of changes
frawk is a small programming language for writing short programs processing textual data.
https://github.com/ezrosent/frawk

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 192964 run on x86_64-linux 1

1 package built:

frawk

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1212

any updates? @SuperSandro2000

Then we do it without tests for now.


Handle jobs that fail to delocalize on the cluster

Currently, orchestrator.make_output_DF() assumes that the keys of self.job_spec are identical to those of the outputs parameter. The outputs parameter is almost always generated by running localizer.delocalize(), so assuming that self.job_spec.keys() == outputs.keys() implicitly assumes that every single job properly delocalized.
This is not necessarily a safe assumption; for example, a long-running task that runs out of preemption attempts will never get a chance to run delocalization.py
This PR will do two things:

Attempt to run delocalization.py on the controller, so that we can at least recover stdout/stderr
For jobs that were totally lost, fill in values for any missing keys in outputs to {}

Coverage increased (+0.002%) to 58.94% when pulling c340922d8f6f0d09c3b6574a4772249663bb291c on julianhess:missing_outputs into 41d062ca4b40cee1dc2000593c4d34ec99a75e07 on broadinstitute:master.


How to make FromJsonAttribute for GET request ?

In past version of ASP.NET MVC, I use FromJsonAttribute for my query objects, that use stateless queries from queryString
When I start writing this in vnext version i confused. How to do it ? Fortmatters use FromBody attribute.
[HttpGet]
public async Task<IActionResult> List([FromJsonUri]QueryObject query)
{
}

@ZOXEXIVO neither MVC 5 nor Web API 3 contain a FromJsonAttribute or FromJsonUriAtttribute class. Are you using an extension such as that discussed on StackOverflow: http://stackoverflow.com/questions/9846158/knockout-and-mvc3-posting-json-to-action-serializing-twice-cant-convert-to-c ?

Yes, but my question about using vnext notation for CustomModelBinderAttribute
It use System.Web.Http.ModelBinding, but I want use pure MVC vnext conception

@ZOXEXIVO System.Web.Http.ModelBinding doesn't mix with MVC 6. So you're on the right track.
In MVC 6, use [ModelBinder(BinderType = typeof(JsonModelBinder))].

Sorry, but it is not usable. We need attribute with GetBinder method, please (like in System.Wen.Http)
And what about access to IServiceCollection in binder ?

Can you please also clarify what your scenario looks like? What does the request look like and what do you want to action method to look like?

We use MVC as WebAPI host Single Page Application with AngularJs.
We have many methods for display stateless grids with difficult many-object dynamic filters with select2, sub-objects e.t.c
All filters store in angular query object, that we serialize and request server with it. We need get requests like this and we happy with this experience,
Yes, we can use [ModelBinder(BinderType = typeof(JsonModelBinder))]. but its looks like bad.
http://ovolime.com/transactions/list?query={q: 'abc', organization: { id: 'd34efwe....', name: 'sadasd'}, category: { id: 3}}
[HttpGet]
public async Task List([FromJsonUri]TransactionQueryObject query)
{
}
Some spec object for each grid
public class TransactionQueryObject : QueryObject
{
public ClassifierDto Category { get; set; }
...........
}
Thank you

@ZOXEXIVO just copy the code from https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Core/ModelBinderAttribute.cs into your own attribute type, but explicitly set the BinderType property to your custom model binder type.

Eilon, Thank you!


[Suggestion] Discord OAuth

Add the option to link your Discord account to your zipline account with Discord OAuth after registration in the account page

In a more broad term allowing account connections like discord, or any other oauth

Already planned


docs, README: add reference to official Docker containers

@mpeters the greetings check seems to be broken. Can you look into why?

The greeting is broken globally for GH actions: https://github.com/actions/first-interaction/issues/101  workaround incoming from our side.


Ability to move todo item to other categories

Dragging and dropping would be a better solution

Suggested by Zachary


Increase wait timeout for meets page to 5 seconds

It was previously 3 seconds which was changed in the last commit, but this wasn't enough sometimes apparently, so let's see if 5 seconds is any more consistent.
Change-type: patch

@balena-ci I self-certify!


pathPrefix parameter is missing in Client#pullChanges()

One Core API feature that seems to be missing in dropbox.js is the path_prefix parameter for the /delta endpoint.
I created a pull request for this a while back – is there any timescale for when this could be merged into master?
Thanks!

+1 for this!

:+1:


Adding LaunchDarkly example

This adds a LaunchDarkly example under the typescript section. Please let me know if everything is in the format expected.

Hi @remotesynth!
Thanks for your contribution. We'll get it into our review pipeline and will comment on this PR if there are any changes to discuss.

Thanks for all the feedback. I have addressed each of the individual items.
Regarding the test event, this would require a larger change. The current code does not look for identity information passed via JWT or in the event. As noted in the comments, it uses an anonymous user context, which could still be useful for experimentation or gradual rollouts.
To create tests that would target users by their identity would probably require an additional example with additional targeting code as well as test code that could further delay completing this example.

Thanks for all the feedback. I have addressed each of the individual items.
Regarding the test event, this would require a larger change. The current code does not look for identity information passed via JWT or in the event. As noted in the comments, it uses an anonymous user context, which could still be useful for experimentation or gradual rollouts.
To create tests that would target users by their identity would probably require an additional example with additional targeting code as well as test code that could further delay completing this example.

Thanks @remotesynth ! I'll take another review pass.
Regarding adding tests, would you be comfortable with someone from our serverless testing initiative taking a 2nd pass (after we merge this) to add them?

Regarding adding tests, would you be comfortable with someone from our serverless testing initiative taking a 2nd pass (after we merge this) to add them?

Sure. I am happy to work with them on it and even come up with an additional example. I think it could be helpful.

Ok, great! I don't necessarily want to block this PR on it, but I do think it would be a really valuable addition.

@brnkrygs I made the readme change and added the .gitkeep file in. I believe that resolves all of the open issues. Thanks again!


Updates build to JDK 11

log4j12 tests fail like this
[ERROR]   MDCCurrentTraceContextTest>CurrentTraceContextTest.attachesSpanInCallable_canClear:189->verifyImplicitContext:39 expected:<"ffffffffffffffff0000000000000001"> but was:<null>

looking into it.

log4j 1.2 org.apache.log4j.helpers.Loader's static initializer accidentally interprets "java.version" of "11" as Java 1, disabling functionality.

@shakuzen finally whacked all the moles I think..


Rescue database error for Heroku pre-deploy step

This snippet was copied from closure_tree:
https://github.com/mceachen/closure_tree/blob/5ad0d58a0ee3744c2c80dab3156802c778fce566/lib/closure_tree/has_closure_tree.rb#L32

Thanks!


Please update HongKong country code (+852)

Thanks👍🏻

Thanks for the contribution.
I just added the Hong Kong country code. It will be available in the next release.


lfs: 2.2.0 -> 2.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Canop/lfs/releases.
meta.description for lfs is: "Get information on your mounted disks"
meta.homepage for lfs is: "https://github.com/Canop/lfs"
meta.changelog for lfs is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/fq0vq1chbg4gm460f9bn2sjwqjxd90q0-lfs-2.4.0/bin/lfs had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 2.4.0 with grep in /nix/store/fq0vq1chbg4gm460f9bn2sjwqjxd90q0-lfs-2.4.0
found 2.4.0 in filename of file in /nix/store/fq0vq1chbg4gm460f9bn2sjwqjxd90q0-lfs-2.4.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
lfs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fq0vq1chbg4gm460f9bn2sjwqjxd90q0-lfs-2.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A lfs https://github.com/r-ryantm/nixpkgs/archive/1bdeae3546415664370cb81a816c1d73bcfb89c5.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fq0vq1chbg4gm460f9bn2sjwqjxd90q0-lfs-2.4.0
ls -la /nix/store/fq0vq1chbg4gm460f9bn2sjwqjxd90q0-lfs-2.4.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

lfs

Maintainer pings
cc @k0ral for testing.

Tested on my side, looks good to me.


Manual install of HRW not possible with MMA 10.20.18001.0

The AzureAutomation folder is no longer present in the MMA folders.

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

ID: 7b29372c-7bd9-7da2-4cff-9afbb432bccf
Version Independent ID: 66ce101d-d21b-3fdf-be70-7f9cadc1570e
Content: Azure Automation Windows Hybrid Runbook Worker
Content Source: articles/automation/automation-windows-hrw-install.md
Service: automation
Sub-service: process-automation
GitHub Login: @bobbytreed
Microsoft Alias: robreed

Thanks for your comment. We are actively investigating and will get back to you shortly. Thanks for your patience.

Please disregard this issue, if directions are followed (adding the Automation solution) the folder is present and the manual installation can proceed as it was in the past. Sorry for the bother. (Didn't RTFM!)


Add an error message for SAS token permission errors

Great, thanks!  Would be even better if it had the command to run, but maybe it's hard to tell what that is here.

Great, thanks!  Would be even better if it had the command to run, but maybe it's hard to tell what that is here.

It's also hard to tell who should be running the command. This came up when someone was trying to copy something from someone else's storage account to their own, but they had only been granted container level permissions.

It's also hard to tell who should be running the command. This came up when someone was trying to copy something from someone else's storage account to their own, but they had only been granted container level permissions.


Wrong version number in CHANGELOG.md

Version 0.5.1 is described as 0.5.0+1.

Turns out that the notification was already a github issue, so this is a duplicate.


Remove service environment variables injection

Description
This PR removes the logic injecting the environment variables concerning services information from the virtual-kubelet, as automatically added in the target cluster by standard logic.
Fixes #(issue)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[x] Existing
[x] Manual

/test

/test

/test

/merge


wrong initial time with timezone.

Describe the bug
If I set the input DateTime to 2020-02-17T15:32:00+01:00 which means the DatePicker should show 16:32 as the time in the input, instead it show 15:32.
Also when I manually select 16:32 it shows 16:32 but it returns 16:32+01:00 instead of 15:32+01:00
To Reproduce
const date = Date("2020-02-17T15:32:00+01:00")
<DatePicker
selected={date}
onChange={date => {
console.log(date);
}}
showTimeSelect
timeFormat="HH:mm"
timeIntervals={30}
dateFormat="dd. MMMM yyyy, HH:mm"
locale={"sk"}
/>
Now the input will show 15:32 (not 16:32), if you change the time to for example 17:30 it will return 17:30+01:00 instead of 16:30+01:00 (which is for some weird reason ok because PHP modifies the date to 16:30+01:00), so technically the problem is that display time that is displayed on the initial render is wrong.

I'm having the same issue.
Also there's nothing about timezones in the official documentation and in the repository.

Hm. I also am having the same issue

Hm. I also am having the same issue


Trading TWS session is connected from a different IP address

Hi!
I wanted to try this library but when I try the sample code I just receive this error:
{'error': 'Trading TWS session is connected from a different IP address'}
Any idea about if this is a bug or theres something wrong?

No, this is weird in fact (I assumed was a but from IB). I'm using the Client Portal Gateway... but I get this error. I don't have even installed the TWS

Hmm, that's interesting.
What do you get when you type the following command in the shell terminal?
curl -X GET "https://localhost:5000/v1/api/one/user" -k

I found the reason: I was authenticated using the paper account. No idea why it says "TWS session" because I have never used it, it must be a minor issue in their API. Thanks! Good work!

Cool, thanks. Just want to note that paper account should work fine as well, but maybe you need to initialize it first from your main account.

Just for anyone having this error, I experienced this error when my account was logged in my web browser at the same time.

Hey im getting this same error, but im not logged in the browser at the same time.


bump version to 0.27.1

Platforms Impacted

[x] iOS
[ ] visionOS
[ ] macOS

Changes since 0.27.0

#1994

Binary change
N/ A
Verification
Build pipeline passes
Pull request checklist
This PR has considered:

[ ] Light and Dark appearances
[ ] iOS supported versions (all major versions greater than or equal current target deployment version)
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[ ] iPad Pointer interaction
[ ] SwiftUI consumption (validation or new demo scenarios needed)
[ ] Objective-C exposure (provide it only if needed)

Microsoft Reviewers: Open in CodeFlow

could you update the pr description to be easier format to read of what is included in this version bump? similar to #1991 . clearly indicator no changes in macOs but few changes in ios. it makes it easier to copy paste and refer back to release notes

Done!


Support DNS based Service Discovery

This PR add new two DiscoveryClient implementations that depend on native Service Discovery supported by Cloud Foundry.

CloudFoundryAppServiceDiscoveryClient uses cf-java-client and find container's index-based route (e.g. 0.billing.apps.internal, 1.billing.apps.internal, 2.billing.apps.internal).
SimpleDnsBasedDiscoveryClient simply uses DNS and resolve container's ip (e.g. <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>). does not depend on cf-java-client

By default, existing CloudFoundryDiscoveryClient is chosen.
Configuring spring.cloud.cloudfoundry.discovery.use-dns=true enables CloudFoundryAppServiceDiscoveryClient.
Configuring spring.cloud.cloudfoundry.discovery.use-dns=true and spring.cloud.cloudfoundry.discovery.use-container-ip=trueenables SimpleDnsBasedDiscoveryClient.
This DiscoveryClient eliminates the limitation of Tubrbine on Cloud Foundry and does not need Turbine Stream.

In some environments (such as in a PaaS setting), the classic Turbine model of pulling metrics from all the distributed Hystrix commands does not work.

#19

@making can you rebase?
@scottfrederick can you review?

@making @scottfrederick ^^^

rebased

ping @scottfrederick

@spencergibb @ryanjbaxter I keep coming back to this and looking at it, and I'm having a hard time deciding what we should do with it.
Having two competing options implemented feels like overkill to me, and I think it would be better to pick just one.
The CloudFoundryAppServiceDiscoveryClient approach that relies on the apps.internal domain seems like a better way to go with the latest abilities in CF. It's possible without any Spring Cloud support for one app to refer to another app using the apps.internal domain and let CF take care of routing to an individual instance; this change moves the routing decision to the client side. It's hard to say how much value this adds because AFAIK no one is asking for this kind of discovery and routing ability in Spring Cloud (even a year after this ability was proposed).
As interesting as the technical approach is, I think we should close this PR pending real-world use cases that point to one approach over another.
@making If you have more info to share on use cases you've seen for this, feel free to make a case for pulling this in.

I agree the new approach seems better, but for backward compatibilities sake, we need to keep the old one. We can certainly mark it deprecated. I'd like to get this merged if you agree the new one is fine @scottfrederick.

I don't have any concerns with the implementation. It's possible with some versions of CF to override the default app.internal domain to a custom value, but it would be better to add that additional configuration later than to hold up this PR for that change.

It's possible with some versions of CF to override the default app.internal domain to a custom value

Yes it's configurable since PAS 2.3.

Squashed and merged via c9ff45fc5ad6e20899dba1238b003ec388bd4e3d


On-Chain bitcoin:Send Max is not working.

Nothing happens when I click on Send Max in Send screen and Send max disappears after 2/3 clicks.

Send max is removed for now

Verified this issue on dev app v2.0.70(438)


Sticky Spells

I know i have mentioned this before but i seem to have an issue where spells are getting stuck all the time. I thought maybe an issue with spellqueue and while i put this down to 0 to fix most of it, it's still happening so i grabbed a GCD dump.
Not sure if you would be able to see what might be happening here to cause it. I have changed random speed for the keypresses, i have tried random sequences (not too bad) but sequential seems to cause this all the time for me.
stuck spell issues.txt
In here is the macro i am trying to use. Happens on all. From line 77 to 248 i am stuck casting Shadow Bolt while GCD is free too.

Just to add context to it, i have uploaded a video...
https://youtu.be/2Rjh7DirQtI

There’s not much I can do here.  GSE is not actually executing the action.  It just tells wow to execute a macro command.  If that command is not executing and it’s skipping to a specific spell then this is because something else that i haven’t been able to track is causing this.
TimothyLuke

On 29 Apr 2018, at 4:24 pm, Cymiryc<EMAIL_ADDRESS>wrote:
Just to add context to it, i have uploaded a video...
https://youtu.be/2Rjh7DirQtI
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

While this might not help for the time being, just a small update
I had a look into more sticky spell stuff and where i am getting stuck does seem to be around the instants like @EnixLHQ mentioned previously. I was building a Fury sequence and i get stuck a lot on Furious Slash which was at the bottom of my priority sequence. I noticed if then instant isat the bottom of a macro or around 2nd last then it just loops on that spell. Might also be why i get stuck between Shadow Bolt and Demonic Empowerment since DE can cause a free SB
I can mostly alleviate it by moving the instant a little more up the list
I will check your other loop stuff that was mentioned

I'll add what I know about this to try to narrow it down. This might get long.
First, with my DH Havok macro. I have two versions. The first one is the older version, where the body of the macro is 23 lines. All of the spells repeat multiple times, one each line. In the KeyPress section is a GCD ability with a CD modded behind the ALT key. In the KeyRelease sections is a NON-GCD ability with a CD not locked behind a mod key at all.
The second version is the newer version where I refined the macro. The body is only 7 lines, though 5 of them are castsequences consisting of two spells each. In KeyPress there are 3 spells all on the GCD, all behind some sort of lockout. One is a castsequence that ends in null, the second is an ALT mod, and the last is a SHIFT mod. KeyRelease is a NON-GCD ability with a CD that is not locked out at all.
Most users are reporting that the second macro produces considerable gains and not a lot of issues. However, myself and at least two others do mention that it will sometimes hang during combat, acting as if it has totally forgotten most of the spells. These aren't the spells that change during Metamorphosis, in fact those appear to remain working. But ones that are skipped are ones on the GCD and have CDs.
Even though the same skills appear in the first macro, it does not have this problem. It is as though the sheer number of times an ability appears (8 times for one, 5 for another, both of which are ones that drop in the second macro) make up for one or two of them no longer registering. No one has ever reported the first macro hanging or getting stuck on a spell or set of spells.
Is it possible that something is causing a spell or set of spells to become temporarily blacklisted or otherwise blocked? Where if you had multiple lines of that spell the others could still get through?
The other issue is with my Shadow Priest macro. This one is likely a different occurrence. The macro has Vampiric Touch, a spell on the GCD that has a cast time, in two places in the KeyPress section. The first mention it is locked behind an ALT mod. The second a reset=target and null at the end of the cast sequence. These are the only places it exists, the rest of the macro doesn't mention Vampiric Touch at all. However, when used in real combat, not a test dummy, Vampiric Touch can lock itself to being the only ability spammed. Unlike the DH macro this can be fixed just by waiting for one GCD and then starting again, even though combat never ended. But, recently there seems to be a higher occurrence of getting stuck.
KeyPress
/castsequence [mod:alt] Vampiric Touch
/castsequence  reset=target  Vampiric Touch, null
Since with this macro this is the only area it hangs, and the only instance of this spell in the macro, it's easy to narrow it down to these lines. Both lines have something that should prevent it from casting. The second line has two. But it'll act like VT is the only spell that exists.

I am seeing problems where people put this in KeyPress - /castsequence reset=target Vampiric Touch, null
The null starts locking up the rest of the macro at random times.

In my inexperienced and uneducated guess, it would be that null isn't a
proper termination command. It's just a broken command. Maybe the API or
how GSE holds macros in memory since the code change means that terminators
like null are now not total locks on the castsequence anymore.
Is it possible to add one? Maybe make null a true stop to the line?
-Written from my phone-
Brian R. Luedtke
AB LUCEM
fiction and writing theory
On Sat, Oct 6, 2018, 1:02 PM Timothy Minahan<EMAIL_ADDRESS>wrote:

I am seeing problems where people put this in KeyPress - /castsequence
reset=target Vampiric Touch, null
The null starts locking up the rest of the macro at random times.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/395#issuecomment-427602329,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADtS2_3K2pYllTWKNnqPMaFmGvFOfKYGks5uiQxLgaJpZM4TrXng
.

null was never a proper termination. It was just something used to stop the advancement of the sequence because wow couldn't recognise and parse it.
Anything that wow cannot use could be swapped out with the same effect as null such as /castsequence reset=target Vampiric Touch, nil or /castsequence reset=target Vampiric Touch, SweatyHobbitFeet would all have the same effect as null... just null makes more sense and easier to use.
but since it's LUA "technically" it should be /castsequence reset=target Vampiric Touch, nil
It's been in use since ~2011 maybe earlier as a way to stop a castsequence from proceeding but "shouldn't" stop lockup a whole macro after that, it's been used for affliction for a long time

Not sure if this should remain open. There are other users/cases of where this happens. I know it can and does happen in the wows own macro system where certain lines are not progressed at all as in
/cast Shadow Bolt
/cast Demonbolt
as it always resets back to Shadow Bolt after the GCD is over
Don't think this can be fully investigated even with all the debugging switched on as other spells are free to cast and not in GCD but sticks on 1 skill only for around 7 casts before resuming... There is nothing repeatable to cause it to happen 100% of the time

I know, Cy, I was suggesting this be converted into an enhancement request
to code for an actual null command in GSE to operate how we expect it to.
But I don't know if that would fix anything, or even if that's possible.
https://ablucem.com
Brian R. Luedtke
AB LUCEM https://ablucem.com

fiction and writing theory*

On Mon, Oct 8, 2018 at 8:44 AM Cymiryc<EMAIL_ADDRESS>wrote:

Not sure if this should remain open. There are other users/cases of where
this happens. I know it can and does happen in the wows own macro system
where certain lines are not progressed at all as in
/cast Shadow Bolt
/cast Demonbolt
as it always resets back to Shadow Bolt after the GCD is over
Don't think this can be fully investigated even with all the debugging
switched on as other spells are free to cast and not in GCD but sticks on 1
skill only for around 7 casts before resuming... There is nothing
repeatable to cause it to happen 100% of the time
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/395#issuecomment-427883721,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADtS2_IOrmdZTcv0XOO1nIrltEARxo24ks5ui3K8gaJpZM4TrXng
.


StringValue reappears in input when mouse hovers over the up/down button after clicking away

Encountered a bug where when a user types in anything in the numeric box (incl special characters: $#@as), then click away from the box, the value gets validated and cleaned up. But when the mouse hovers over the up/down button, the string value reappears until you click inside the numeric textbox.

im getting the same issue

+1


Top stories

Hi, it'd be great if Ghost was able to provide minimal blog stats for "top stories".
I'm not talking about all the details we can get from Google Analytics. Rather just a widget of some sort that can connect to a GA account and display "x" top stories (e.g.  3, 5, or 10) somewhere in the theme.
To get a better idea of what I mean, have a look at the top right section on Medium home page.
Is there any plan to provide this feature in the near future? In the meantime, is there any plugin or widget that could do? Anytthing that works in connection with Google Analytics, but only display the most popular posts by some parameter (e.g. weekly, or monthly)?
I just published a question on Quora too.
Looking forward to hear your thoughts!
Cheers

Hey @pensierinmusica,
Something like this I have no doubt will be created some day, but it will most likely be once Apps are available. @PaulAdamDavis is working on the dashboard right now, dont know if there is a timeline for that, but once that is out, and then apps are out, this should be able to be created as an app and then installed onto the dashboard.
I know that the plan is to have some built in stats for the dashboard, but any tie ins to google analytics will prob be app territory.
Hope that helps,
David


Deb package files installed with user as owner instead of root

Hello,
the Debian packages available under releases install the files with the system user as owner instead of root.
$ ls -lah /etc/vulkan/icd.d/
total 12K
drwxr-xr-x 2 root      root      4,0K apr 20 10:17 .
drwxr-xr-x 5 root      root      4,0K apr 10  2020 ..
-rw-rw-r-- 1 user      user  645 apr  7 13:13 amd_icd64.json

$ ls -lah /usr/lib/x86_64-linux-gnu/amdvlk64.so
-rwxrwxr-x 1 user user 56M apr  7 13:13 /usr/lib/x86_64-linux-gnu/amdvlk64.so

Operating system: Ubuntu 18.04
Steps to reproduce:
wget https://github.com/GPUOpen-Drivers/AMDVLK/releases/download/v-2021.Q2.1/amdvlk_2021.Q2.1_amd64.deb
sudo dpkg -i amdvlk_2021.Q2.1_amd64.deb

Expected behavior:
Files installed in the /etc and /usr folder should be owned by root not a regular system user.
Needless to say, this poses a security issue as any process from this regular user can modify these system files that will be used by any vulkan context.

fixed


undefined: New

Hello,
I am quite new to go.. but I get this error:
.\main.go:6:5: imported and not used: "github.com/mxpv/nvml-go" as nvml
.\main.go:318:15: undefined: New
what may I be missing?

Hello. Try
api := nvml.New("")


update APIC chart

Discarding this new chart version since what I wanted to implement with it can be done through overrides. It is not only it can be done but also the preferred method. Hence, closing this PR


Screenshot for advanced skill while offline training shows login screen

If you advance in skill while offline training, lyzer will actually screenshot the login screen instead of waiting for you to be logged in to screen shot it.

I can't really solve this issue, the logs are simply loaded into memory before the character finishes logging in. I don't really think anyone cares about "offline training" screenshots anyway, so I'm not planning on fixing this issue either.


Tokio test might fail on ubuntu

Sometimes the tokio test fails:
error: test failed, to rerun pass '--test tokio'
##[error]test failed, to rerun pass '--test tokio'
Caused by:
  process didn't exit successfully: `/home/runner/work/foundationdb-rs/foundationdb-rs/target/debug/deps/tokio-c51e2bb533a18041` (signal: 6, SIGABRT: process abort signal)
##[error]The process '/usr/share/rust/.cargo/bin/cargo' failed with exit code 101
##[error]Node run failed with exit code 1

This was with v0.2.10.
Seems like recent builds (v0.2.11) aren't affected.

Nevermind, still failing sometimes: https://github.com/Clikengo/foundationdb-rs/pull/173/checks?check_run_id=422465156

This is probably caused due to this foundation spec:
fdb_stop_network:
Signals the event loop invoked by fdb_run_network() to terminate. You must call this function and wait for fdb_run_network() to return before allowing your program to exit, or else the behavior is undefined.

yep, this is the cause, fdb_stop_network MUST be called before the program exits.

Internally I traced the segfault to a data race between the network thread that is still running and the __do_global_dtors__aux call in libfdb_c.so.
main thread:
    frame #22: 0x00007ffff7cb8ddf libfdb_c.so`TraceLog::~TraceLog(this=0x00007ffff7faa980) at Trace.cpp:496:14
    frame #23: 0x00007ffff74432de libc.so.6`__cxa_finalize + 206
    frame #24: 0x00007ffff77f5f47 libfdb_c.so`__do_global_dtors_aux + 39
    frame #25: 0x00007ffff7fe13fb ld-2.30.so`___lldb_unnamed_symbol59$$ld-2.30.so + 523
    frame #26: 0x00007ffff7442ba7 libc.so.6`___lldb_unnamed_symbol193$$libc.so.6 + 247
    frame #27: 0x00007ffff7442d60 libc.so.6`exit + 32
    frame #28: 0x00007ffff74201ea libc.so.6`__libc_start_main + 250
    frame #29: 0x000055555556c17e tokio-8ff57d870675dfc8`_start + 46

Fixing this undefined behavior will requires a breaking change :(

NetworkAutoStop, NetworkWait, NetworkStop cannot stay public
Only one #[test] will be possible per test file.
NetworkBuilder::boot<T>(self, f: impl FnOnce() -> T) -> T
unsafe NetworkBuilder::build


How to specify a different format date dd/mm/yyyy for Excel export?

I tried this:
var ws = XLSX.utils.table_to_sheet(document.getElementById(id), {dateNF:'d/m/yy'});
instead of:
var ws = XLSX.utils.table_to_sheet(document.getElementById(id));
No effect. date result is still inversed (formatted in html table with dd/mm/yyyy)
04/10/2017 in html table is written 10/04/2017 in excel file.
It seems to does not support french format...
Do you have any idea to resolv my issue?

dateNF:'mm/dd/yyyy;@' seems to resolv my problem but I don't understand why cell format is mixed with Standard and date format. When date format can not be read with format dd/mm/yyyy or mm/dd/yyyy (ex: 31/01/2017), it's standard format and when it can read in both ways dd/mm/yyyy or mm/dd/yyyy (ex: 04/10/2017), it's date format.
Please help, I'm lost!

And cellDate has no effect
var ws = XLSX.utils.table_to_sheet(document.getElementById(id), {dateNF:'mm/dd/yyyy;@',cellDates:true });

@laurentauthier can you share a small HTML table and the result of XLSX.utils.table_to_sheet(document.getElementById(id)); ?  Wondering whether it is an ingress or egress issue

@SheetJSDev with dateNF:'mm/dd/yyyy;@' it seems to be ok: 04/10/2017 (4 october) is written 04/10/2017 in xlsx file with date format.
But 31/01/2017 is written with standard format with spaces before and after (due to comments in html I suppose). I don't understand why.
I'm french.

@laurentauthier the ingress part (which takes the cell text and generates a date) uses the browser Date constructor to parse the date string: https://github.com/SheetJS/js-xlsx/blob/master/bits/20_jsutils.js#L129 .  I suspect that the browser parsing implicitly assumes the format is "m/d/y".  Here's what I get on Windows 7 Chrome under French (France) Locale:

Can you check the two commands against your browser (new Date("04/10/2017") and new Date("31/01/2017")) and see whether you get the same results?

@SheetJSDev export as csv:
,ADM - administratif 2016,-,       31/01/2017           , ...
,ADM - Administratif 2017,-,04/10/2017,                               ,  ...
when there is some whitespaces, it's not formatted.
How to specify to trim values?

In fact,I do not understand why specify dateNF:'mm/dd/yyyy;@' to format dd/mm/yyyy in excel file... How to do?

i have problem to:

this.viewExportar = false;
const ws: XLSX.WorkSheet=XLSX.utils.table_to_sheet(this.table.nativeElement, {dateNF:'mm/dd/yyyy;@',cellDates:true});
const wb: XLSX.WorkBook = XLSX.utils.book_new();
XLSX.utils.book_append_sheet(wb, ws, 'Sheet1');
/* save to file */
XLSX.writeFile(wb, "relatorio_CP-"+moment().format()+".xlsx");

Solved with: const ws: XLSX.WorkSheet=XLSX.utils.table_to_sheet(this.table.nativeElement, {dateNF:'mm/dd/yyyy;@',cellDates:true, raw: true});

Muchas gracias danillo10, ¿De dónde sacaste la referencia mil disculpas?, me interesaría profundizar más sobre esta librería, muchas gracias

Thanks a lot @danillo10 your answer solved my same problem.

@danillo10 thank you so much. it works!

Overriding single cell date formats is described in the docs https://github.com/SheetJS/sheetjs#number-formats by setting the z property.
As @danillo10 wrote, pass the option cellDates: true to recover date cells.

What about this code?
function setExcelDateFormat(theWS,theCols,frmtTxt){
for(h=0;h<theCols.length;h++){
var colNum = XLSX.utils.decode_col(theCols[h]);
var range = XLSX.utils.decode_range(theWS['!ref']);
for(i=range.s.r+1;i<range.e.r+1;i++){
var ref = XLSX.utils.encode_cell({r:i, c:colNum});
if(!theWS[ref]) continue;
theWS[ref].v = theWS[ref].v.toString().replace(/( смн)[.]/gi,'');
if(!/^[0-9,.-]+$/gi.test(theWS[ref].v)){
}
else{
theWS[ref].v = theWS[ref].v;
theWS[ref].t = 'd';
theWS[ref].z = frmtTxt;
}
}
}
return theWS;
}

i answer in stackOverflow
https://stackoverflow.com/questions/53163552/format-date-with-sheetjs/76894466#76894466
const element = document.getElementById('excel-table');
const ws: XLSX.WorkSheet = XLSX.utils.table_to_sheet(element);
in "ws" date format like a this json
{
"t": "n",
"v": 44933.000555555554,
"z": "m/d/yy"
}
so we know date have type (t) number (n) and z is "m/d/yy"
Object.keys(ws).forEach(cell => {
const cellValue = ws[cell];
  if (cellValue.t === 'n' && cellValue.z === 'm/d/yy') {
    const excelDate = cellValue.v;
    const jsDate = new Date((excelDate - (25567 + 1)) * 86400 * 1000);
    const formattedDate = `${String(jsDate.getDate()).padStart(2, '0')}.${String(jsDate.getMonth() + 1).padStart(2, '0')}.${jsDate.getFullYear()}`;
    ws[cell].v = formattedDate;
    ws[cell].t = 's';
  }
});
and do const wb: XLSX.WorkBook = XLSX.utils.book_new();...


AlmMomentTime pipe refactoring and unit tests

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@michaelkleinhenz @nimishamukherjee can you please start the Jenkins job?

[test]

[test]

Current coverage is 51.43% (diff: 100%)

Merging #688 into master will increase coverage by 0.39%

@@             master       #688   diff @@
==========================================
  Files            41         41
  Lines          2167       2166     -1
  Methods         416        416
  Messages          0          0
  Branches        423        423
==========================================
+ Hits           1106       1114     +8
+ Misses         1061       1052     -9
  Partials          0          0

Powered by Codecov. Last update abd9f64...480b536


tree-sitter v0.20.8 causes several tests to fail

The offending commit is https://github.com/tree-sitter/tree-sitter/commit/d223a81b5064587af3a5f61f52d519670ba8995f with the following failures:
3 failures:

expected / actual

  1. Simple glsl function no end:

    (file
      (value_declaration
      (ERROR
        (function_declaration_left
          (lower_case_identifier))
        (eq)
        (glsl_code_expr
          (glsl_content)
          (MISSING _glsl_end))))
        (glsl_content)))

  2. Incomplete model access:

    (file
      (value_declaration
        (function_declaration_left
          (lower_case_identifier)
          (lower_pattern
            (lower_case_identifier)))
        (eq)
        (function_call_expr
          (value_expr
            (value_qid
              (lower_case_identifier)))
          (list_expr)
          (list_expr
            (ERROR
            (field_access_expr
              (value_expr
                (value_qid
                  (lower_case_identifier)))
              (dot))))))
              (dot)
              (MISSING lower_case_identifier))))))

  3. Incomplete model access:

    (file
      (value_declaration
        (function_declaration_left
          (lower_case_identifier)
          (lower_pattern
            (lower_case_identifier)))
        (eq)
        (function_call_expr
          (value_expr
            (value_qid
              (lower_case_identifier)))
          (list_expr)
          (list_expr
            (ERROR
            (field_access_expr
              (value_expr
                (value_qid
                  (lower_case_identifier)))
              (dot))))))
              (dot)
              (MISSING lower_case_identifier))))))

This is likely causing a freeze downstream in Helix per helix-editor/helix#2997

How did you get those tests to fail?

With a tree-sitter of lib version 0.28.0, tree-sitter generate -> tree-sitter test.
Here's the full process in a temp directory (assumes you have cargo, so nix-shell -p cargo or the equivalent for non-Nix):
git clone https://github.com/tree-sitter/tree-sitter
cd tree-sitter
make
cargo build --release
cd ..
git clone https://github.com/elm-tooling/tree-sitter-elm
cp ./tree-sitter/target/release/tree-sitter ./tree-sitter-elm/
cd tree-sitter-elm
./tree-sitter generate
./tree-sitter test

My current feeling is, that it might be a lsp server problem. It doesn't expect the new tree's shape and that might cause our code to run in a loop?

The freeze in helix is caused by that linked change in tree-sitter. There was a similar freeze with the org parser which has a very similar scanner implementation: https://github.com/milisims/tree-sitter-org/issues/34, and their fix: https://github.com/milisims/tree-sitter-org/commit/698bb1a34331e68f83fc24bdd1b6f97016bb30de
It looks the failing tests are fixed by applying a similar patch to the scanner here. I haven't been able to reproduce the hang but I suspect it would be fixed by that patch too.


SCOPING: WORKING LIST OF BINDINGS FOR SWIFTUI

Active list of all available bindings we can map. From:
List of Bindings
LIVEVIEW BINDINGS FOR SWIFTUI
Click Events

[x] phx-click liveview-native/liveview-client-swiftui#1096

Focus Events

[x] phx-blur liveview-native/liveview-client-swiftui#1097
[x] phx-focus liveview-native/liveview-client-swiftui#1098
[x] phx-window-blur liveview-native/liveview-client-swiftui#1099
[x] phx-window-focus liveview-native/liveview-client-swiftui#1100

Key Events

[ ] phx-keydown liveview-native/liveview-client-swiftui#1101
[ ] phx-window-keydown liveview-native/liveview-client-swiftui#1102

Posting for my own attention: this work may require an update to the "event-bindings" guide when merged.


Improve FBSimulator Inflation

Currently, FBSimulator instances are inflated on demand and then discarded (if they aren't referenced). It should be pretty easy to figure out the processIdentifier of running Simulators as well rather than having it set to -1. Using the Notifiers can help keep the instances up-to-date.

Some work on this done in #21 but the caching of Simulators and their expensive properties (like the construction of Simulator processes) needs some more work.


WIP: HWKAPM-779 | opentracing jaxrs fraction

Please do not review yet!

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

on hold until implemented in main repo


Installer Error "Critical environment variable 'DATABASE_URL' missing!\n"

I try to install shopware 6 and I get
"Critical environment variable 'DATABASE_URL' missing!\n"
url
/recovery/install/database-import/

Im closing this in favor of https://github.com/shopware/platform/issues/192 and https://jira.shopware.com/browse/NEXT-4939.

so.. are there any fixes / workarounds for this that I can use to install Shopware?


How to disable style inline on element swiper-wrapper and swiper-slide

I have noticed this problem when my slide display not nice. Maybe some style conflicts but i want to know if possible to disable styles by javascript apply on swiper-wrapper and swiper-slide. I have checked
I expected slide will have 100% size width but it fixed width by inline styles you can see in below
You can see in image below

I am using swiper v3. Effected in Slide mode, Loop false autoHeight true

I agree with this. The feature would be needed because the width of the slides is wrong at first loading, until you resize the page manually.


SyntaxError: Cannot use import statement outside a module since version 2.1.1

Describe the bug
I'm getting the following build time error when attempting to use the package in a locally running node app:
/node_modules/link-preview-js/build/index.js:10
import cheerio from "cheerio-without-node-native";
^^^^^^

SyntaxError: Cannot use import statement outside a module

Initially I thought it was my config but rolling back to version 2.1.0 (the one with the security issues) works fine. This leads me to believe that its something to do with the internal usage of Cheerio.

ah, I changed the transpilation target to ES6 since most node environments support ES6 syntax, which version of Node are you running? also bear in mind if you are not using 12+ you are probably going to run into troubles soon since it is reaching end of support

Version 2.1.3 is released, I've reverted the compilation target to ES5 and you should not face troubles with modules anymore


As a CIF developer, I want to store reporting requirements in the database, so I can build a frontend for it

Description:
Building the database side for reporting requirements
Acceptance Criteria:
Given I am a CIF developer,
When I store reporting requirements in the database
Then I have the (reporting_requirement, report_type, budget_item, payment) tables available
Development Checklist:

[ ] reporting_requirement table

due_date timestamptz
submitted_date timestamptz
status varchar? enum? check with @suhafa for a list of status
comments varchar(10000)
certified_by varchar? can maybe become a contact in the future
certified_by_professional_designation varchar , can probably be an enum once we know a list of accepted values

[ ] report_type table

name varchar(1000)
form_schema jsonb - we'll need to confirm whether we use this or not, this is a larger discussion to be consistent with the rest of the application

[ ] budget_item table

amount numeric
category enum? right now the 2 values would be "general" and "materials, supplies, equipment and other items"
description varchar(10000)
is_confirmed boolean
is_tentative boolean

[ ] payment table (this is a bit of a placeholder right now)

amount numeric
date_issued timestamptz
date_paid timestamptz
comment varchar(10000)
transaction_id varchar(1000)
status varchar(1000)? enum?

[ ] Meets the DOD

Notes:
Fields with question marks can be implemented either way, and changed at a later time

Hey team! Please add your planning poker estimate with ZenHub @BCerki @gurjmatharu @matthieu-foucault @pbastia @Sepehr-Sobhani

I'm assuming this is done as it is in the approved by po/dev column? I don't think there is anything I can really test on this one, so moving to closed unless there are any objections.


PyPy3?

Any interest in maintaining a PyPy3 copy of this?  If not I might find time to fork and handle it but it seems like it might be a quick thing to do.
Cheers,
-kyle

@altendky I may take a stab at it if you don't want to

@dhermes, I haven't popped down my stack of other tickets and PRs back to here yet so... Go for it :]  not that I know chocolatey but feel free to ping me if you want another set of eyes on something.

https://chocolatey.org/packages/python.pypy3/6.0.0 pending, source at https://github.com/dhermes/python.pypy3
I added an AppVeyor configuration to build and verify the *.nupkg file (my only Windows install is on a partition on my work machine and it's no fun to shut down to run Windows commands).

@altendky I'm happy to add you as a collaborator there if you like.

@dhermes, thanks for setting that up. Now I have to figure out what's wrong with virtualenv.
https://ci.appveyor.com/project/ebroecker/canmatrix/build/job/osmldoit63ujycye

For posterity / anyone else who comes across this, the package is in https://chocolatey.org/packages/pypy3 (i.e. the name changed from python.pypy3 to just pypy3)


support(bulk-create): add raise_error flag when bulk-create empty models

add raise_error flag #843

Codecov Report

Merging #853 (acab758) into master (816c3ce) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff            @@
##            master      #853   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          192       188    -4
  Lines        16057     16041   -16
=========================================
- Hits         16057     16041   -16

Impacted Files
Coverage Δ

ormar/queryset/queryset.py
100.00% <100.00%> (ø)

tests/test_queries/test_queryset_level_methods.py
100.00% <100.00%> (ø)

ormar/signals/__init__.py

ormar/relations/__init__.py

ormar/models/__init__.py

ormar/fields/__init__.py

@ponytailer We can do it inline with bulk_update and simply raise and exception on empty list (and release as 0.12.0 since it's breaking). Thanks!

**collerek ** commented 2分钟前

Yes, if you will have the big version recently,  raise exception simply when it's empty.

Thanks!


Sort output

This PR orders the results. By default, it's ordered by chrom, start, and id. Custom ordering can be provided using order_by.
If subsets of columns are selected, then the columns are sorted with available columns and finally by id.

Codecov Report
Attention: Patch coverage is 95.23810% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 93.01%. Comparing base (ef33d87) to head (a5cdb15).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #64      +/-   ##
==========================================
- Coverage   93.09%   93.01%   -0.08%
==========================================
  Files           6        6
  Lines         333      358      +25
==========================================
+ Hits          310      333      +23
- Misses         23       25       +2

Files
Coverage Δ

src/genomic_features/ensembl/ensembldb.py
93.02% <95.23%> (-0.14%)
:arrow_down:


Collaborative wallet: Textual change.

Need to add text around sharing of wallet after wallet is created.

Text added in the modal now
Section:
https://www.figma.com/design/m5eMFTKD391AiJAXaEzgxB/Bitcoin-Keeper?node-id=6667-11585&t=38cIy5OgiDA0ky4S-1
S.S:

Verified this issue on keeper app v 1.3.1(430)


Update Settings “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for gentle-cat-bc6937 processing.

Name
Link

🔨 Latest commit
333835dde54c896172fe2ea86e8bd6762712264d

🔍 Latest deploy log
https://app.netlify.com/sites/gentle-cat-bc6937/deploys/64dc7dd2f812e30008429821