Binary Protocol queries are prohibitively slow

The test program for my nim-orient project takes over a minute to get and display all of the records in the GratefulDeadConcerts sample database. After banging my head against the wall and trying various profiling strategies, I've found out that 99.8% of the test program's time is spent waiting on data to come into the socket. A Wireshark dump confirms that orientdb sends data to the socket at an incredibly slow trickle. This slowness prohibits OrientDB from being used in a production setting.

So, the problem isn't with Nim, bit with the OrientDB software?

Correct. The OrientDB server is slow at sending things down the socket.

When you say slow, what do you mean? how many seconds? Did you try with OrientDB console (that uses Java client)?

@lvca i tried with orientdb console (2.1-rc5 and 2.2-SNAPSHOT) and i get similar response time (>900 seconds) to retrieve 809 records for both versions:
orientdb {db=GratefulDeadConcerts}> select from v limit -1 fetchplan *:-1

809 item(s) found. Query executed in 925.719 sec(s).

@lvca When I say slow, I mean that running a query over the binary protocol is many times slower than running the same query in the OrientDB console. Here are some numbers:

Running nim-orient test program:

$ git checkout prerelease
$ cd orient/tests
$ nim c --gc:none -d:release test_all

$ ./test_all
(This makes sure that OrientDB has already loaded the dataset from the disk.)
...

$ time ./test_all
...
63.93 real         0.05 user         0.01 sys

Running the same query in the OrientDB console:

$ ./bin/console.sh
orientdb> CONNECT remote:localhost/GratefulDeadConcerts admin admin
Connecting to database [remote:localhost/GratefulDeadConcerts] with user 'admin'...OK
orientdb {db=GratefulDeadConcerts}> SET limit -1
Previous value was: 20
limit = -1

orientdb {db=GratefulDeadConcerts}> SELECT FROM V LIMIT -1
...
809 item(s) found. Query executed in 0.251 sec(s).

orientdb {db=GratefulDeadConcerts}> SELECT FROM V LIMIT -1
...
809 item(s) found. Query executed in 0.098 sec(s).

@lvca Interestingly enough, if I use the @nagarajasr's command in the OrientDB console, which specifies the fetch plan explicitly as "*:-1" (like nim-orient's test program does), the command fails to complete and appears to repeat the following indefinitely:
orientdb {db=GratefulDeadConcerts}> SELECT FROM V LIMIT -1 FETCHPLAN *:-1
2015-07-12 01:42:27:168 WARNING Caught I/O errors from /<IP_ADDRESS>:2424 (local socket=/<IP_ADDRESS>:49549), trying to reconnect (error: java.net.SocketTimeoutException: Read timed out) [OStorageRemote]
2015-07-12 01:42:27:169 SEVERE Removing disconnected network channel '<IP_ADDRESS>:2424/GratefulDeadConcerts'... [OStorageRemote]
2015-07-12 01:42:27:186 WARNING Connection re-acquired transparently after 17ms and 1 retries to server '<IP_ADDRESS>:2424/GratefulDeadConcerts': no errors will be thrown at application level [OStorageRemote]
2015-07-12 01:43:14:080 WARNING Caught I/O errors from /<IP_ADDRESS>:2424 (local socket=/<IP_ADDRESS>:49550), trying to reconnect (error: java.net.SocketTimeoutException: Read timed out) [OStorageRemote]
2015-07-12 01:43:14:081 SEVERE Removing disconnected network channel '<IP_ADDRESS>:2424/GratefulDeadConcerts'... [OStorageRemote]
2015-07-12 01:43:14:089 WARNING Connection re-acquired transparently after 8ms and 1 retries to server '<IP_ADDRESS>:2424/GratefulDeadConcerts': no errors will be thrown at application level [OStorageRemote]
2015-07-12 01:44:00:703 WARNING Caught I/O errors from /<IP_ADDRESS>:2424 (local socket=/<IP_ADDRESS>:49551), trying to reconnect (error: java.net.SocketTimeoutException: Read timed out) [OStorageRemote]
2015-07-12 01:44:00:704 SEVERE Removing disconnected network channel '<IP_ADDRESS>:2424/GratefulDeadConcerts'... [OStorageRemote]
2015-07-12 01:44:00:709 WARNING Connection re-acquired transparently after 5ms and 1 retries to server '<IP_ADDRESS>:2424/GratefulDeadConcerts': no errors will be thrown at application level [OStorageRemote]

Moving the entire graph as result of each record could cause memory problem and timeouts like you reported. Whats' the goal of returning the entire graph? This is what *:-1 does.

What fetch plan should I be using? The documentation says that *:-1 is valid.

It depends on the level of depth you want to have.
http://orientdb.com/docs/last/Fetching-Strategies.html
Scott

For the test program, what I want to do is get all of the vertices in the GratefulDeadConcerts database and iterate through them. Since I want all of the data for in all vertices, it seems like "*:-1" is what I want. Is this correct?
Also, I guess I don't really get fetch plans.

The query:
SELECT FROM V LIMIT -1

With default fetchplan (*:0) Already loads all the vertices.

@lvca Thanks! If *:0 is the default fetch plan, then I will use it for nim-orient's test program and add it as a default.

Good news! After changing the test program to use the *:0 fetch plan and -1 as the limit, nim-orient's time becomes comparable with the OrientDB console!
$ nim c -d:release test_all
$ time ./test_all
...
        0.07 real         0.04 user         0.00 sys

$ ./bin/console.sh
...
orientdb {db=GratefulDeadConcerts}> SELECT FROM V LIMIT -1 FETCHPLAN *:0
...
809 item(s) found. Query executed in 0.029 sec(s).

Great!


Feature: Select Folder type of Dialog

Feature Request
Currently, when using the Dialog method, we can use the DialogTypeEnum to determine if we want a dialog for:

Message - a simple message
Error - an error message
Open - open a file
Save - save a file

Well, it turns out that sometimes we want the user to also select a folder instead of a file. So I would like to propose a new DialogType:

Folder - select a folder

User Story
As a user,
I want to be able to select a folder,
In order to select the destiny of a generated file, for example.
As a developer,
I want to be able to popup a Dialog to request a folder destiny to the user,
In order to get the path I should use as destiny of a generated file, for example.
(I'm no good at writing user stories, but well, I tried)

Well, it seems to be necessary a pull request for this in https://github.com/parro-it/libui-node and https://github.com/andlabs/libui :/
I've already take a look at the files, it would be necessary to modify these files:
darwin: https://github.com/andlabs/libui/blob/86c4485578d6d58aff6528b27558bc1ccc6aadbe/darwin/stddialogs.m
windows: https://github.com/andlabs/libui/blob/86c4485578d6d58aff6528b27558bc1ccc6aadbe/windows/stddialogs.cpp
unix: https://github.com/andlabs/libui/blob/86c4485578d6d58aff6528b27558bc1ccc6aadbe/unix/stddialogs.c

Probably two new types are needed:

OpenFolder and
SaveFolder (this would also make it clearer that there are related to their Open/Save counterparts)

because on all platforms, you have to select one type (i.e. NSSavePanel or NSOpenPanel).

There was already a pull request for openFolder (closed): https://github.com/andlabs/libui/pull/190

@mischnic nice to know this, maybe I can build a PR up on this, but I didn't found it in the master branch if I can recall (I need to take a look again..). Maybe it was removed later?

I didn't reject the PR. I decided to hold off on merging it, because the current APIs for dialogs isn't ideal. It all has to do with event handling and synchronicity. I'll use pseudocode in JavaScript to explain what I mean (this will probably be easier than explaining it in C in the main repo, to be honest...).
On Windows, the system dialog API only gives me one option:
var selectedItem = runDialog(parentWindow);

You'll notice this function does not return until the dialog is dismissed. You'll notice that a parent window must be specified. Clicking on that window while the dialog is open will cause that window to flash.
Other windows can still be interacted with, but they will NOT respond to things like tab navigation. This is because tab navigation on Windows is intrinsically tied to dialog boxes! Specifically, the IsDialogMessage() function requires you to pass a handle to the top-level window (I'm not sure if it's the dialog box or the parent window), and only handles tab navigation for the window you passed in. libui works around this for the outermost main loop by passing the currently active window into this function, but in the runDialog() function above I have no such control. Likewise, I have no such control over what happens inside the main loop body, so no custom main loops (uiMainSteps()/uiMainStep()).
I could also disable all the windows during the dialog's lifetime, but then clicking on those other windows won't flash the dialog window (and I don't think I have a way to do this myself at all...).
Now I could theoretically use a dialog message hook to try to work around this, but I'm not sure if this solution might be flaky (I would have to write it). I have tried to deduce what the correct programming model on Windows is based on all this information, even going so far as to ask outright, but I can't get an answer out.
Failure to provide a parent window will cause Windows to not flash the dialog window when clicking other windows.
GTK+ is the nicest of the bunch, as I have two options:
var selectedItem = runDialog(parentWindow);

openDialog(parentWindow).then((selectedItem) => {
    // ...
});

The former will prevent interaction with any window during the dialog's lifetime, but in a weird some-of-the-events-still-go-through way. I also can't do any custom main loops with this option.
The latter will let you interact with everything else in the program while the dialog is running. I don't remember if there were caveats, but notice you still need to provide a window to disable while the dialog is running.
In both cases, failure to provide a parent window will do nothing but cause GTK+ to spit the dreaded GtkDialog mapped without a transient parent on stderr.
macOS is only slightly less nice:
var selectedItem = runDialog();

openDialog(parentWindow).then((selectedItem) => {
    // ...
});

In the former case, the dialog is global to the application, and all window interaction is blocked until the dialog is dismissed. I also can't do any custom main loop stuff.
In the latter case, the dialog is modal to to a specific window, and appears as a "sheet" that pulls down from the titlebar. You can still interact just fine with other windows, under certain circumstances. While custom main loops are still possible, this approach probably tells NSApplication to run in a different main loop run mode which might affect things. I need to read more on how NSApplication and sheets work together before I can make a verdict here. Also, you must provide a parent window in this case.
So hopefully that explains the dilemma.

New issue tracking progress: https://github.com/andlabs/libui/issues/314


Docs: Use emojis instead of images to indicate fixable/recommended rules

In the rule page, I wanted to search for all auto-fixable rules but the icon was an image instead of unicode, so I couldn't do a ctrl-f. So this PR changes chechmarks and wrenches to unicode to allow searching, which should make it easier for people to find the rules they're looking for.
Screenshot of how it looks below. Chechmarks could be ✅ or some other things instead if people prefer it.

before
after

Interesting idea. I feel like the wrench icon is a little bit difficult to decipher visually, though. Is there another that could be used instead?

I'd be in favor of switching to emojis.
In many projects I've worked on, we use 🔧 for fixable, and ✅ for recommended (I've also seen ✔ for recommended).

https://github.com/sindresorhus/eslint-plugin-unicorn#rules
https://github.com/not-an-aardvark/eslint-plugin-eslint-plugin#supported-rules
https://github.com/platinumazure/eslint-plugin-qunit#rules
https://github.com/ember-cli/eslint-plugin-ember#components
https://github.com/ember-template-lint/ember-template-lint#rules

Okay, I don't feel very strongly about this. I kind of like the dark checkmark as opposed to the green, but if anyone wants to mock that up for a comparison, we can take a look.

Ok I signed CLA and changed commit message so I think this is ready if yall are happy with it

Could you include both before and after screenshots in the PR description?
| Before | After |
| --- | --- |
| before.jpg | after.jpg |

And ideally, the individual rule docs would use the same emojis in their fixable/recommended notices.

Updated description. Rule docs are already changed.

Netlify automatically generates a preview site each time the PR is updated. You can see this page here:
https://deploy-preview-858--eslint.netlify.app/docs/rules/
I believe what @bmish is looking for is to use the same icons on the individual rule pages, which you can see an example of here:
https://deploy-preview-858--eslint.netlify.app/docs/rules/no-extra-semi
I'm also 👍 to using the same emojis on the individual rule pages.

Video of usage:
https://user-images.githubusercontent.com/10591373/126547217-b700182b-8764-463a-bc17-04a2a4c639a3.mov
With the styling on the wrench & checkmark right now, they're not selectable when they float left of the paragraph. Is that fine? They do look the same everywhere now though.

Agree with @bmish.


Not working in docker

java.lang.UnsatisfiedLinkError: Unable to load library '/tmp/io.woo.htmltopdf/wkhtmltox/0.12.4/libwkhtmltox.so': Native library (tmp/io.woo.htmltopdf/wkhtmltox/0.12.4/libwkhtmltox.so) not found in resource path

It seems to work for me in a simple test with a stock docker container. It appears that for some reason there's a failure to copy wkhtmltopdf library resources. Are you using version 1.0.2?

We have the following Dockerfile:
FROM frolvlad/alpine-oraclejdk8:slim

VOLUME /tmp
ADD project-1.0-SNAPSHOT.jar app.jar
RUN sh -c 'touch /app.jar'
ENV JAVA_OPTS=""
ENTRYPOINT [ "sh", "-c", "java $JAVA_OPTS -Djava.security.egd=file:/dev/./urandom -jar /app.jar" ]

We are building docker image in Jenkins, then saving image in AWS

@lgabeskiria it looks like the library is being copied to /tmp and JNA is attempting to load it, the (rather misleading) error comes from that some dependencies are missing in the alpine image:
Error loading shared library libXrender.so.1: No such file or directory (needed by /tmp/io.woo.htmltopdf/wkhtmltox/0.12.4/libwkhtmltox.so)
Error loading shared library libfontconfig.so.1: No such file or directory (needed by /tmp/io.woo.htmltopdf/wkhtmltox/0.12.4/libwkhtmltox.so)
Error loading shared library libfreetype.so.6: No such file or directory (needed by /tmp/io.woo.htmltopdf/wkhtmltox/0.12.4/libwkhtmltox.so)
Error loading shared library libXext.so.6: No such file or directory (needed by /tmp/io.woo.htmltopdf/wkhtmltox/0.12.4/libwkhtmltox.so)
Error loading shared library libX11.so.6: No such file or directory (needed by /tmp/io.woo.htmltopdf/wkhtmltox/0.12.4/libwkhtmltox.so)
Error loading shared library libstdc++.so.6: No such file or directory (needed by /tmp/io.woo.htmltopdf/wkhtmltox/0.12.4/libwkhtmltox.so)

If you could find a way to install these libraries into your docker image, htmltopdf should work fine.

If you add the following to your docker build script that should solve it:
RUN sh -c 'apk update && apk add libx11 libxext libxrender libstdc++ freetype fontconfig libssl1.0'

@benbarkay libc6-compat is missing too. I was able to get working with:
RUN sh -c 'apk --update add libc6-compat libx11 libxext libxrender libstdc++ freetype fontconfig libssl1.0'

@benbarkay  Thanks that worked for us (except fonts).
But there is some strange behaviour: When calling API for first time it generates pdf, but second time it just goes to timeout.
I think the converter should be singleton.

@miguelbaldi his docker image has glibc2 installed on it so he did not need it. But that's good to know.
@lgabeskiria could you open another issue with a snippet that would reproduce the behavior?

@benbarkay https://github.com/wooio/htmltopdf-java/issues/8

In my case (Ubuntu 18.04), the package libpng12-0 was missing:

Exact reason for this issue is that libpng12is no longer available in the Ubuntu repository archives. In-fact, Ubuntu dropped it from Ubuntu 16.10 too.

Possible solution is to install this package manually:
wget http://security.ubuntu.com/ubuntu/pool/main/libp/libpng/libpng12-0_1.2.54-1ubuntu1.1_amd64.deb && sudo dpkg -i libpng12-0_1.2.54-1ubuntu1.1_amd64.deb && sudo rm -f libpng12-0_1.2.54-1ubuntu1.1_amd64.deb

java.util.concurrent.ExecutionException: java.lang.UnsatisfiedLinkError: Unable to load library '/tmp/io.woo.htmltopdf/wkhtmltox/0.12.5/libwkhtmltox.so': Native library (tmp/io.woo.htmltopdf/wkhtmltox/0.12.5/libwkhtmltox.so) not found in resource path ([file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/classes/, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/ant-1.9.7.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/ant-launcher-1.9.7.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/classmate-1.3.4.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/commons-beanutils-1.9.3.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/commons-codec-1.10.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/commons-collections-3.2.2.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/commons-lang3-3.9.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/curator-client-2.12.0.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/curator-framework-2.12.0.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/dubbo-2.6.2.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/dubbo-spring-boot-actuator-0.1.1.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/dubbo-spring-boot-autoconfigure-0.1.1.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/dubbo-spring-boot-starter-0.1.1.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/fastjson-1.2.60.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/flying-saucer-core-9.0.3.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/flying-saucer-pdf-itext5-9.0.3.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/greenline-guser-auth-share-1.0.0.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/groovy-2.4.15.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/guahao-specification-data-1.0.3.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/guahao-specification-service-1.0.2-SNAPSHOT.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/guahao-user-model-share-3.4.48.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/guahao-user-service-share-3.4.52.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/guava-19.0.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/hessian-4.0.38.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/hibernate-validator-5.3.6.Final.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/htmltopdf-1.0.8.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/httpclient-4.5.4.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/httpcore-4.4.9.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/httpmime-4.5.2.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/itext-asian-5.2.0.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/itextpdf-<IP_ADDRESS>.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/jackson-annotations-2.8.0.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/jackson-core-2.8.11.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/jackson-databind-<IP_ADDRESS>.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/javassist-3.21.0-GA.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/javax.el-2.2.6.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/javax.el-api-2.2.5.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/jboss-logging-3.3.2.Final.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/jcl-over-slf4j-1.7.25.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/jline-0.9.94.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/jna-4.2.2.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/jsqlparser-1.0.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/jul-to-slf4j-1.7.25.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/log4j-over-slf4j-1.7.25.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/logback-classic-1.1.11.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/logback-core-1.1.11.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/lombok-1.16.20.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/mybatis-3.4.6.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/mybatis-spring-1.3.2.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/mybatis-spring-boot-autoconfigure-1.3.2.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/mybatis-spring-boot-starter-1.3.2.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/mysql-connector-java-5.1.46.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/nekohtml-1.9.22.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/netty-3.10.6.Final.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/ognl-3.0.8.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/owasp-java-html-sanitizer-20190610.2-GH-SNAPSHOT.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/pagehelper-5.1.2.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/pagehelper-spring-boot-autoconfigure-1.2.3.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/pagehelper-spring-boot-starter-1.2.3.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/slf4j-api-1.7.25.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/snakeyaml-1.17.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-aop-4.3.17.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-beans-4.3.17.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-boot-1.5.13.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-boot-autoconfigure-1.5.13.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-boot-starter-1.5.13.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-boot-starter-jdbc-1.5.13.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-boot-starter-logging-1.5.13.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-boot-starter-thymeleaf-1.5.13.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-boot-starter-web-1.5.13.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-context-4.3.17.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-core-4.3.17.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-expression-4.3.17.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-jdbc-4.3.17.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-tx-4.3.17.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-web-4.3.17.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/spring-webmvc-4.3.17.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/thymeleaf-2.1.6.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/thymeleaf-layout-dialect-1.4.0.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/thymeleaf-spring4-2.1.6.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/tomcat-jdbc-8.5.31.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/unbescape-1.1.0.RELEASE.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/util-core-1.0.9.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/validation-api-1.1.0.Final.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/xercesImpl-2.11.0.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/xml-apis-1.4.01.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war--any-3574633427262141141.dir/webapp/WEB-INF/lib/xmlworker-5.5.9.jar, file:/tmp/jetty-0_0_0_0-7040-diagnostic_war-_-any-3574633427262141141.dir/webapp/WEB-INF/lib/zookeeper-3.4.9.jar])==========={hospitalId=1, numcard=1, studyIdApply=8596564634}

在Linux 物理机上部署 不会成功

same problem, any solutions?

same problem, any solutions?

me too, so sad

I got the same error.


 If the error directory doesn't exist the application will finish as soon as there's an error. Workaround is to just use a directory that exists :)
Awesome tool by the way, sadly I didn't find it before importing my takeout to Apple Photos and now my photo library is pretty crippled, with thousands of pictures with date today.

Thanks for reporting this @citricut. I've pushed a fix, so now it'll do the same check for --errorDir as it does for --outputDir. If the directory doesn't exist it'll create it automatically, if it does exist then it'll proceed if it's empty or throw an error if the directory is not empty (to avoid accidentally writing into any folders that have files in already)


Purging package cache macos_python.yml

To fix the error:
ERROR: THESE PACKAGES DO NOT MATCH THE HASHES FROM THE REQUIREMENTS FILE. If you have updated the package versions, please update the hashes. Otherwise, examine the package contents carefully; someone may have tampered with them.

Fixed all failing tests! Merging.


added DE translationsof groups

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)

Proposed changes
added german transalations of groups in editor-client/locales/de/editor.json

Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[ ] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 69.392% when pulling e57251f415b9705fecade4409154dcbfc3d69536 on heikokue:i18n-de/group into 9b1c114c3f5336e8f6b410a96f8f9c61a944f437 on node-red:dev.

this PR is obsolete and I merged the "group" changes to branch new entries and node-red-terms


make install error

kustomize build config/crd | kubectl apply -f -
Error: must build at directory: not a valid directory: evalsymlink failure on 'config/crd' : lstat /Users/config/crd: no such file or directory

@jacobxy Thanks for raising this issue! I think this typically has to do with the operator-sdk create api ... subcommand not being run before attempting to run make install.
Would you mind sharing the exact steps you took until you got this error?


operator dell-csm-operator (0.2.0)

Signed-off-by: Rensy Thomas<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[x] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

Hi @J0zi and @mvalarh, please help check why image pull failed for kiwi tests. Please let me know if I am missing something here

@rensyct can you specify registry name like docker.io in image dellemc/dell-csm-operator:v0.2.0

/merge possible

@rensyct can you specify registry name like docker.io in image dellemc/dell-csm-operator:v0.2.0

Thank you @mvalarh, tests passed after specifying the registry name in image . Please help merge this PR if all ok

/merge possible


Add Meeting Minutes for Labs Meeting on 2015-12-11

Hey @zachlatta, is this fine?
Let me know if anything needs tweaking before we merge :smile:

@MaxWofford We made it! Sorry for all my mistakes with the formatting haha, but I think next time I should have it down.

Absolutely no worries re: formatting! It's super difficult to get started with and we all learn with time :-).

@JevinSidhu Nice work! The tests are passing. The last step is to rebase it so we don't have extra commits in the pull request.

Yay! Thanks @zachlatta :100:

Nice work @JevinSidhu! Thanks for adding the notes :)


Add support for ARM64 Installation Target for Windows ARM.

Requires updating VSSDK.BuildTools and Community.VisualStudio.Toolkit.17. Also updated to latest markdig package.

Thank  you!


Rendering issues for inline diagnostics

New user here trying out lite-xl and the plugins it has. I ran into this issue that is very much similar to what #48 describes. I have tried rust-analyzer and pyright setups on my end, diagnostics do get reported correctly and viewable from status bar. Running Lsp: Toggle Diagnostics twice (first it disables and then enables again) renders them in the document but continues to either stay statically or disappears until I run the command twice again. I'm not seeing any errors in the logs as well.

All plugins were installed through lpm.
OS: Windows 10
Lite-xl version: 2.1.3
user init.lua and plugin hierarchy:

Seems to work here with pyright installed via the plugin lsp_python:
Video del 2024-03-27 16-37-33.webm
Am I missing something about this issue?
Could you test with lsp_python too? Maybe even in an lpm bottle (lpm run --ephemeral lsp_python).

I tried lsp_rust and lsp_python in the system install before reporting here too, but it turned out be same case.
Bottles are affected as well on my end apparently: lpm run --ephemeral lsp_python
https://github.com/lite-xl/lite-xl-lsp/assets/71869007/47ab71ae-b34e-475f-a3ea-bf9470979bf1
PS: dont worry about the unresolved dependencies in the video, it supposed to be in a poetry managed env but it is not setup on this box.

Apologies for not giving any updates but I'm gonna close this issue here as I was simply trying out lite-xl out of curiosity. Could've helped more if I was more comfortable with Lua :(

Thank you for reaching out back to me again! I can confirm it does work flawlessly now on Windows. I'll re-close this issue as completed then. Thanks again for your time.

Thank @Callmore for finding the root cause!


remove shh and git volumes

The git config and ssh keys are mounted into the build container to
grant access to private repositories for go build. This is no longer
necessary after the open-sourcing of Stackrox. Go dependencies can
simply be pulled via the default https git config and no ssh keys are
needed.
Checklist

[x] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on stackrox/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
make image on macOS using a colima Kubernetes setup.

@porridge Since I know you use Linux, could you give it a try?

 Compiling with -linkmode=external
Compiling Go source in ./central to bin/linux/central
go: downloading golang.org/x/net v0.0.0-20220325170049-de3da57026de
go: downloading github.com/stackrox/scanner v0.0.0-20220408151911-460993206ee4
go: downloading google.golang.org/genproto v0.0.0-20220405205423-9d709892a2bf
go: downloading cloud.google.com/go/storage v1.22.0
go: downloading google.golang.org/api v0.74.0
go: downloading k8s.io/kubelet v0.22.8
go: downloading golang.org/x/sys v0.0.0-20220328115105-d36c6a25d886
go: downloading github.com/PagerDuty/go-pagerduty v1.5.0
go: downloading google.golang.org/protobuf v1.28.0
go: downloading github.com/stretchr/testify v1.7.1
go: downloading cloud.google.com/go/iam v0.3.0
go: downloading github.com/googleapis/gax-go/v2 v2.2.0
go: downloading github.com/googleapis/go-type-adapters v1.0.0
go: downloading github.com/spf13/cobra v1.4.0
go: downloading k8s.io/apiserver v0.22.8
go: downloading github.com/magiconair/properties v1.8.6
Unable to connect to the server: dial tcp: lookup api.jephilli-4-11-04-21-0636.devcluster.openshift.com on <IP_ADDRESS>:53: no such host
Unable to connect to the server: dial tcp: lookup api.jephilli-4-11-04-21-0636.devcluster.openshift.com on <IP_ADDRESS>:53: no such host
make: *** [Makefile:180: fast-central] Error 1

Ah, it must be the pod killing script that runs after the build. So LGTM


feat: autosuggest error handling

Description
Added logic to set error handling when clicking or tabing outside of the component. Created logic to allow consumer to use freeform functionality. Created option values and seperate display versus data value logic.
Deploy Preview
Include a direct link to your changes in this PR's deploy preview here (e.g., a specific component page).
Merge Checklist

[ ] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[ ] Does your change adhere to the documented style conventions?
[ ] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[ ] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[ ] Were your changes tested in the example app?
[ ] Is there adequate test coverage for your changes?
[ ] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

I'm going to close this now that https://github.com/openedx/paragon/pull/2899 is in, but it was absolutely helpful to reference when implementing that! Thanks so much for the PR!


Altair Facetchart rendering issue

shinywidgets version: 0.3.1
Python version: 3.10.13
Operating System: Windows

Description
Hi team,
When we generate a facet chart, an error occurs while rendering the altair facet chart.
error message
Facetchart object has no attribute 'width'
I'd really appreciate your help. many thanks!
What I Did
minimal example
from shiny.express import input, ui
from shinywidgets import render_altair

ui.input_selectize("var", "Select variable", choices=["bill_length_mm", "body_mass_g"])

@render_altair
def hist():
    import altair as alt
    from palmerpenguins import load_penguins

    df = load_penguins()
    return (
        alt.Chart(df)
        .mark_bar()
        .encode(x=alt.X(f"{input.var()}:Q", bin=True), y="count()")
    ).facet(column='species:N')

Thanks! I was quite surprised to learn that vega doesn't support specifying height/width on a FacetChart, so the sizing will be awkward/non-responsive, but @render_altair won't error on it anymore.


Add vulkano-derive with custom derive for Vertex

[ ] Added an entry to CHANGELOG_VULKANO.md or CHANGELOG_VK_SYS.md if knowledge of this change could be valuable to users
[ ] Updated documentation to reflect any user-facing changes - in this repository
[ ] Updated documentation to reflect any user-facing changes - PR to the guide that fixes existing documentation invalidated by this PR.

WIP. Currently it does the same as impl_vertex! but as a derive macro. There is a lot of room left to improve the code, and even use features only available to procedural macros.

Right now an implementation of Default is required for the vertex struct, in order to make the macro work. It cannot be removed AFAIK because we use a dummy instance of it to calculate offsets for each of the members, which is impossible to do right now without an existing value of the struct (there are attempts out there using raw pointers and uninitialized data, but right now they rely on undefined behavior more or less).
I'd like to remove that restriction in the future, but right now we'll have to live with it.

Aside from the mismatched version there are some issues with adding another crate. Version publishing is automatically handled by travis ci, so that'll need to be updated via the travis config to publish that also.

Perhaps, the goal if this PR is already at least partially covered by my other PR https://github.com/vulkano-rs/vulkano/pull/1460 that now allows custom derives in existing macro(from vulkano-shaders crate).
Also to mention, that now we(maintainers) have capabilities to maintain crates.io publishings directly(thanks to Tomaka, the owner of the project), so the issue with the side crates is probably not that big deal now, however I would suggest to avoid of adding additional crates to the project ecosystem whenever it possible. It might complicate maintainability.
There were no updates in the Pull Request nor comments about the development progress from the author( @jmi2k ) quite a while. It seems like the development was abandoned. As such I would like to close the PR for now. If the development continues please reopen the Pull Request again. We are glad to accept any infrastructure improvements that could benefit Vulkano user community experience.


Update SE-0253: introducing callables.

Update: direct references to call members are no longer supported.

Add "direct references" section under "alternatives considered".

Expand on function-type conversion future direction.
Start filling in ABI stability/resilience sections.

@rxwei: could you please review?
Let's try to merge afterwards, to make incremental progress.

Is this ready to merge?  Just let me know and I'll do so.

Is this ready to merge? Just let me know and I'll do so.

@lattner Ready to merge now, thanks!

Thanks!  Please let the thread know

Thanks! Please let the thread know

Done.

@dan-zheng  Missed one:

But it is more similar to a func declaration in that:

It does not allow get and set declarations inside the body.
When a parameter has a name, the name is treated as the argument label.
It can be referenced directly by name, e.g. foo.call.

@dan-zheng Missed one:

It can be referenced directly by name, e.g. foo.call.

Thanks! Fixed in https://github.com/apple/swift-evolution/pull/1021.

Ah sorry there was one more, I edited the comment.

Ah sorry there was one more, I edited the comment.

Thanks! Done in https://github.com/apple/swift-evolution/pull/1021.
Please leave any extra comments on the new PR :)


Replace cloudinary_dart with cloudinary_url_gen

This PR replaces the old cloudinary_dart with the new cloudinary_url_gen

thank you @adimiz1!

@all-contributors please add @adimiz1 for code


Implement code completion from functions from our library.

You could also add some documentation in web page. Generate JS documentation.


Help new users build Crisp

These simple changes address minor issues in the experience of a new user building Crisp.

I fixed the mixed up names in a couple of the readmes, thanks! I can't and won't list all indirect dependencies of baseview and viziai there. If you somehow don't have them installed however (almost any Linux install will have the xcb extensions for iccm, xfixes, etc. installed), then you would get an error message telling you about it during compilation.


Adding nock.cleanAll()

I have recently opened a PR adding cleanAll to a stop function in mockJWKS. Let me explain.
When we create a nock scope we are able to hit the request once and receive the mocked response.
However, if I hit the request second time, I get an error:
const scope = nock('https://google.com').get('/.well-known/jwks.json').reply(201)

// First request
const r1 = await request('https://google.com/.well-known/jwks.json')
expect(r1.statusCode).to.equal(201) // This will pass

// Second
const r2 = await request('https://google.com/.well-known/jwks.json')
expect(r2.statusCode).to.equal(404)

/*
Nock: No match for request {
  "method": "GET",
  "url": "https://google.com/.well-known/jwks.json"
}
*/

The way nock works is that it sets the scope for the base URL. Each base URL can have multiple interceptors.
When it receives the request to a base URL it tries to match the request with the right interceptor.
The second request is not matched because the first request removed the interceptor (by default interceptor is for a single hit) but left the base URL matching.
The same problem happens using mock-jwks. Once mocked the jwksOrigin + jwksPath  with the persist, we remove the interceptor but jwksOrigin stays as a base URL for nock and it matches the requests. Meaning, that all the tests are written and run after the use of mock-jwks can't be nocked again since it will be failing at mock-jwks/nock matchers before reaching the other nock.
I have created an example repo demonstrating how it affects the tests that are being run after mock-jwks. 3.test.js tries to nock again the same jwksOrigin but the tests are hitting the first initiated nock from mock-jwks:
https://github.com/edvinasbartkus/mock-jwks-example
Just by running different sequence of tests it fails and it passes:
    "test:fails": "lab ./test/1.test.js ./test/2.test.js ./test/3.test.js",
    "test:works": "lab ./test/1.test.js ./test/3.test.js ./test/2.test.js"

In order to remove the matching of base URL nock.cleanAll() should be called.
Since mock-jwks is using nested nock version it should be free to call nock.cleanAll() and affect only it's used nock version. The same way calling nock.cleanAll() from the project tests we are not able to reach mock-jwks/node_modules/nock.
There is an alternative. https://github.com/Levino/mock-jwks/blob/master/index.ts#L25 can be replaced with:
-     jwksUrlNock = nock(jwksOrigin)
+     jwksUrlNock = nock(jwksOrigin, {allowUnmocked: true})

This would result that once interceptor is removed, the base URL will not be matched for the future request.
Hopefully, I have managed to explain the scenario well :)
Let me know if the alternative approach could be right?
@Levino

Please push a failing test to your repo. Is it not possible to just add an automated test that shows your issue? Code says more than 1 Mio words.

I still want one test which fails. One test that starts the mock, stops the mocks and then does something which does not work as expected. I do not understand why you have three tests instead of just one. Please reduce your code to one test.

@Levino updated to include just a single file:
https://github.com/edvinasbartkus/mock-jwks-example/blob/master/test/mock.test.js

Using mock-jwks in a project with nock@^10 is causing this issue. The proposed solution might "solve the issue" but does not grab the problem by the root. Closing.

@edvinasbartkus I encountered a similar issue like yours. Using<EMAIL_ADDRESS>solves the issue for me. Please confirm.


Add button/visual indicator for un-hiding navbar

Description
Preface: this might just be a personal problem, but keen to hear your opinion.
But, when hiding the sidebar, it took me quite a while to figure out that you can un-hide the sidebar by simply clicking on the GitLight logo.
Maybe the existing << icon that is used for collapsing the sidebar can be used to un-hide it as well?

If it is unclear for you, it probably is also unclear for other people (or maybe everyone 😓), so I will make a change
Thanks!


Space between Navigation Bar and FlatList

Current Behavior

I am implementing a FlatList with StackNavigator, and I am getting a space between the Navigation Bar and the View.

Expected Behavior

I can't seem to get rid of the space, and when I did it with ListView previously it worked correctly.

Your Environment

software
version

react-navigation

react-native

node

npm or yarn

Blind shot...
Speaking look is as expected with ListView, I would blame FlatList instead of react-navigation. Have you tried `<FlatList style={{ margin: 0 }} />?

@Andreyco yeah I tried that :/

Removing the <List> -component surrounding the FlatList fixed this issue for me.

Yeah if using the <List /> from react-native-elements will cause this. Closing as that should be a solution.


Implement GSLB_Monitor_External

Is your feature request related to a problem? Please describe.
Please implement External monitor type for GSLB.
Describe the solution you'd like
Just like Monitor_External but for GSLB_Monitor_External.
Describe alternatives you've considered
There are none
Additional context
External monitors solve many customer corner cases but in this case it also would help solving a very relevant problem: the HTTPS monitor for GSLB doesn't support SNI and the only workaround is to use curl through an external monitor.

+1
I have a customer in the UK that needs this feature to provide GSLB for apps deployed across multiple k8s clusters, that use a shared ingress model.

This looks to be a duplicate of https://github.com/F5Networks/f5-appsvcs-extension/issues/259

Closing as duplicate of #259.


OPTIONS not functional - how to proceed?

OPTIONS not functional - how to proceed?
I'm using a devops-server which obviously doesn't support "OPTIONS" as http-get method. When I try to access the server via curl, I get the following behavior...
curl -I --request GET --url 'https://SERVER/collectionkfurt/automotivekfurt/_apis' --header 'authorization: Basic TOKENB64'
HTTP/2 200
[...]

In contrast.
curl -I --request OPTIONS --url 'https://SERVER/collectionkfurt/automotivekfurt/_apis' --header 'authorization: Basic TOKENB64'
HTTP/2 404
[...]

I see that in the client.py module OPTIONS is used to establish a first connection. Is there a way around this?
For testing I use the following code:
ORGURL = "https://SERVER/collectionkfurt/automotivekfurt"
credential = BasicAuthentication(USERNAME, PAT)
connection = Connection(base_url=ORGURL, creds=self.credential)
core = self.connection.clients.get_core_client();

The error occurs at the line self.connection.clients.get_core_client();, with ...
raise AzureDevOpsClientRequestError(full_message_format.format(error_message=error_message,
azure.devops.exceptions.AzureDevOpsClientRequestError: Page not found.  Operation returned a 404 status code.

Try:
ORGURL = "https://SERVER/collectionkfurt"

Hi @tedchamb ,

Try:
ORGURL = "https://SERVER/collectionkfurt"

Yeees great - its working. But why? I didn't see any OPTIONS request anymore.

If the OPTIONS call wasn't made, that means you have the options data already cached for that url.  Once the cache expires it will make the OPTIONS call again.
The cache file is in the following location.
os.path.expanduser(os.path.join('~', '.azure-devops', 'python-sdk', 'cache')

You can delete the cache folder or file if you want to see the OPTIONS call made again.

Thanks for your help. Its working fine now.


fixtypo: misspell in runtime package

Signed-off-by: Wei Fu<EMAIL_ADDRESS>

Codecov Report

Merging #2730 into master will increase coverage by 3.67%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2730      +/-   ##
==========================================
+ Coverage   43.75%   47.43%   +3.67%
==========================================
  Files         100       91       -9
  Lines       10725     8408    -2317
==========================================
- Hits         4693     3988     -705
+ Misses       5301     3696    -1605
+ Partials      731      724       -7

Flag
Coverage Δ

#linux
47.43% <ø> (ø)
:arrow_up:

#windows
?

Impacted Files
Coverage Δ

remotes/docker/auth.go
63.82% <0%> (-3.97%)
:arrow_down:

remotes/docker/status.go
21.42% <0%> (-3.58%)
:arrow_down:

platforms/cpuinfo.go
3.77% <0%> (-0.85%)
:arrow_down:

log/context.go
36.84% <0%> (-0.66%)
:arrow_down:

namespaces/context.go
55% <0%> (-0.56%)
:arrow_down:

errdefs/grpc.go
74.6% <0%> (-0.4%)
:arrow_down:

oci/spec_opts.go
20.6% <0%> (-0.18%)
:arrow_down:

remotes/docker/pusher.go
0% <0%> (ø)
:arrow_up:

content/local/locks.go
100% <0%> (ø)
:arrow_up:

labels/validate.go
100% <0%> (ø)
:arrow_up:

... and 55 more

Continue to review full report at Codecov.

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


Internal modifier for namespaces

Hi,
Why don't implemented internal class in typescript?
Element.ts:
namespace Library {
    export class Element {
        //....
    }
}

SelectElement.ts:
namespace Library{
    export class SelectElement extends Element {
        //....
    }
}

Program.ts:
namespace Library{
    export class Program {
        var selectElement = new Library.SelectElement();
        var element = new Library.Element();
    }
}

If i put to export keyword before Element class, other developer can access to Element class, but i remove export keyword i don't use Element class in Library namespace. Does internal class will be able to use now or next version?

might be related #321

indeed duplicate of #321

Where is related issues? :scream:


Enum isHash

Adds a new enum modifier flag, EnumIsHash that forces MakeCode to generate the enum values as the hash16 of the name. This is particularly useful for radio-broadcast or any other named based broadcast API where users might want to write different programs talking the same protocol.

LGTM. I guess this one doesn't really need tests

I'm not completely sure how the .blocks files are supposed to be generated.

It gets printed to console with ?dbg=1
I wasn't hinting at anything, there really isn't anything to test here :smile:


JSDoc unexpected leading whitespace behaviour with @example and

Bug Report

🔎 Search Terms

#15749 matches, but was last updated in June 2020 -- was informed to make a new issue
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about JSDoc and leading whitespace

⏯ Playground Link

Playground link with relevant code
💻 Code

// In the code block below, `sum(answer, 5)` is NOT indented, but it should be.
// `sum2(answer, 5)` is indented.

/**
 * Adds two numbers
 *
 * @param a number one
 * @param b number two
 * @returns sum of number one and number two
 *
 * @example
Add 1 and 3. If answer is >0, sums result and 5.

 * ```
 * const answer = sum(1, 3); // returns 4
 *
 * if (answer > 0) {
 *   sum(answer, 5)
 * }
 * ```
 */
function sum(a: number, b: number): number {
  return a + b
}

/**
 * Adds two numbers
 *
 * @param a number one
 * @param b number two
 * @returns sum of number one and number two
 *
 * @example
Add 1 and 3. If answer is >0, sums result and 5.

 * ```
 *          const answer = sum2(1, 3); // returns 4
 *
 *          if (answer > 0) {
 *            sum2(answer, 5)
 *          }
 * ```
 */
function sum2(a: number, b: number): number {
  return a + b
}

🙁 Actual behavior

The JSDoc produced for sum() has the example without any indentation, despite there being preceding whitespace.
This only happens when there is a
 tag present.

🙂 Expected behavior

The result should have the same indentation as present in the comment, preferably determined from the indentation of the code block backticks.

I came across this working on a custom elements library. I'd like to be able to document examples for custom element classes, but without indentation it's not as helpful as it could be.
https://astexplorer.net/#/gist/5f17f411fcd556f8292a46b66ec952be/latest
for example, after I convert the typescript source to a custom-elements.json, then load that manifest into a page, I get this:

Where the source (abbreviated) is
/**
 * @element polymer-apollo-query
 *
 * `<polymer-apollo-query>` fires Polymer-style prop-changed events
 * when its `data`, `error`, `loading` or `networkStatus`
 * properties change.
 *
 * See [ApolloQueryInterface](/api/core/interfaces/query/) for more information.
 *
 * @example Querying for Data
 * ```html
 * <polymer-apollo-query
 *     data="{{data}}"
 *     variables="[[variables]]"
 *     query="[[UserQuery]]"
 * ></polymer-apollo-query>
 *
 * <paper-icon-item>
 *   <iron-image slot="item-icon">[[data.user.picture]]</iron-image>
 *   [[data.user.name]]
 * </paper-icon-item>
 * ```
 *
 * @fires data-changed
 * @fires error-changed
 * @fires errors-changed
 * @fires loading-changed
 * @fires network-status-changed
 */
export class PolymerApolloQuery<D, V> extends Super {/*...*/}

Eek, this is still an issue. My current workaround for this is:
/**
 * @example
 *
Adding a new child to a discussion list item title

 *
 * ```
 * extend(DiscussionListItem.prototype, 'view', function (vnode) {
 *   findFirstVdomChild(vnode, '.DiscussionListItem-title', (vnode) => {
 *     // ...
 *   });
 * });
 * ```
 */

Which does render correctly inside the doc pop-ups in VS Code.

Experiencing this still, but none of the workarounds work currently.

The above workaround will only work if you do not have a language specified.
This works, sort of, and it will let you add indentations. But its not preferred as now the entire codeblock starts after one indentation.
`/**

@description Using blockquote to fix indentation of code examples

@example Correct Usage:

type SourceType = { required: string; optional1: string; optional2: string; omit: string };

// Require and Omit keys from a SourceType:

const requiredAndOptional: PickRequiredOmit<SourceType, 'required', 'omit'> = {

required: 'This value is required.',

optional1: 'This value is optional.',

// omit: 'This value cannot be provided.'

};

// Omit specified keys from SourceType while making all others optional. ( = undefined).

const omitAndOptional: PickRequiredOmit<SourceType, undefined, 'omit'> = {

optional1: 'This value is optional.',

optional2: 'This value is also optional',

// required: 'This value is technically optional, too.'

// omit: 'This value cannot be provided.'

}

*/
`

Let me know if this is off-topic or should go in another issue, but I experience this without @example:
export class Heap<T> {
  arr: T[];
  compareFn: (first: T, second: T) => number;

  /**
   * @constructor
   * @param arr An array of type T
   * @param compareFn A comparator function. Examples:
   *
   * ```ts
   * // Min Comparator
   * [5, 4, 3, 2, 1].sort((a, b) => {
   *     return a - b;
   * }) === [1, 2, 3, 4, 5]
   *
   * // Max Comparator
   * [1, 2, 3, 4, 5].sort((a, b) => {
   *     return b - a;
   * }) === [5, 4, 3, 2, 1]
   * ```
   */
  constructor(arr: T[], compareFn: (first: T, second: T) => number) {
    this.arr = arr.sort(compareFn);
    this.compareFn = compareFn;
  }
// ...

When I hover over the constructor keyword:

related:

https://github.com/microsoft/TypeScript/issues/56583


[Feature Request] - Add Debounce /Throttle in pagingController.refresh() method

When implement search / filtering. I would be helpful if we can add debounce when calling refresh method. Moreover we call it  in onChanged callback.

Hi @ariefwijaya .
That is outside of the package's scope. You should handle the debouncing yourself.
There are multiple ways for you to do that. The example project, for example (:P), handles it on the input.


bump to version 0.2.0

Cutting a new release with xarray support (see #13 and #15). @eyaltrabelsi @zbarry and @ericmjl.

Thanks @Zsailer! :smile:


Expose rate of incoming transactions (lambda) in tangelwidthcollector

This is because width/lambda should be a constant

Not needed


Elm-format not working when installed as a package

Apologies Dillon - my elm-format patch is breaking prod at the moment.
The issue is that .bin doesn't get populated when graphqelm gets installed as a module.
Currently looking into a fix for this.

For a global installation, it looks like it might be as simple as moving elm-format from devDependencies to dependencies in package.json.
For local installs, .bin is one level up. Currently looking into a way to get around this.

I opened a PR to fix this: https://github.com/dillonkearns/graphqelm/pull/11

Merged 👍


Need way to specify latest browser versions without releasing new code

At the moment, the various languages each hard-code browser versions like "Firefox 67.0" and "Chrome 75.0" right there in the code. Understanding that this was not meant to be permanent, it's time to start thinking about how to make these frameworks more self-sufficient.
Initial requirements:

All the language bindings use the same mechanism, and ideally read from the same "cache"
Browser versions come from config, not from code, likely either yaml or JSON that's generated by a query to a web service
Poll the Platform Configurator (or some other tool) to generate the currently-supported versions of each browser
The code will read this cache file, taking the most recent browser as "latest" and running that by default
Users will need to be able to override the version
The file can't be generated every time you run a test. It would either be at the beginning of a build, or the Web Service needs to indicate, probably via CRV check, whether or not the info has changed
This call to the web service has to be very tolerant of failure--if it fails, there must be some way to use another resource, either a JSON/yaml that we package with each version (which we'll try to keep up-to-date), or some set of fall-backs. This call CANNOT prohibit the execution of tests, nor can it create any impediment to starting a test

There are probably more, but the main idea is that we have to get out of the business of hand-maintaining browser versions. This is NOT a requirement for a beta, but I'll be watching closely to make sure such a feature can be incorporated into the various bindings.

I'm cool with this idea. What I want to make sure doesn't happen is that we open up possibilities for users to start passing strings again. I want zero interactions with the Platform Configurator and I want the IDE to guide the user into the correct available platform capabilities. So, almost everything statically typed and avoiding strings.  That even as an SA, I never have any idea about what strings to pass to what properties to create a proper capabilities object. I've dealt with this pain for years with Sauce and Browser Stack and I'd like it to stop 🛑 :) W3C is a perfect example. I have no clue what works and what doesn't until I try the code and debug for hours. Even our docs are useless. But our code could be the source of truth 😊

Can't we just use 'latest' to get this functionality? And set that to be the default?

That said, I have a spike in the Ruby code that creates cache from API, looks for requested info from it and if it can't find it, does an API call in case the cache is outdated. Wasn't fully functional, but I think that's the same idea as here.

Yeah, it's the same idea. I think there's a way to greatly improve things, and also keep users from having to guess or use the Platform Configurator... we just haven't thrown much effort behind that yet.

Using latest will work across all supported browsers as a "reasonable default". There is some quirkiness with Safari/Mac but this will work.

Since we do have a way of specifying latest browser, I'm opening a new ticket #151 that represents the other pieces of this conversation and am closing this one.


Which step of export_fig antialiasing causes this gap in polar presentation?

Code which causes an artifact most likely caused by antialiasing, observed in polar coordinates after conversion from Cartesian img
[img, alpha] = export_fig(filename, '-png', '-native', '-q101', '-a1', '-m1', '-RGB', '-nofontswap', '-dpng', hax);

The example code to make the cartesian image, store it and map it to polar coordinates is in the thread below.
Matlab: 2016a
OS: Debian 8.5 64 bit
Linux kernel: 4.6 of backports
Hardware: Asus Zenbook UX303UA
Related: Why is there Gap in this Matlab Polar presentation?

This is something strange in Matlab 2016a, not in the function itself.

However, the fix may be related to export_fig because you need to make the adjustment img=img(2:end-1,2:end-1) to fix it in the Cartesian, since I think you cannot pass img directly to export_fig.

It looks like this may be a bug, but I am not sure as I am inexperienced with export_fig. The previously linked stackoverflow question (and answer) describes it nicely. It looks like after the saving, there is an extra border of [240 240 240] (which is the default MATLAB plot background color) saved in the image.


Packaging cmake fails on Windows with mambabuild

With conda mambabuild, we are able to perform the cmake build. However when packaging the installed files together, packaging fails. See this log (also attached). Note this only happens with Windows (both macOS and Linux succeed). However when switching to conda-build this issue goes away. See this log (also attached). Not entirely sure why this issue occurs with mambabuild (or why only Windows), but hopefully this information is helpful in figuring that out.

Thanks @jakirkham we have a similar symptom with Orange3. It's pretty weird. I'll have to debug that.

Thanks Wolf! 😀
Yeah saw conda-build in the logs before the error as well so that sounds plausible. Though am confused by why conda-build didn't see the same error

OK, I have a lead ... pylief and mamba both use pybind11. However, pylief uses a vendored pybind11 and not the one from conda-forge and it's probably outdated.
When inspecting the binaries of CMake, pylief is throwing some exception and it could be that e.g. the exception handling code of pybind11 is ABI incompatible somehow, leading to this mysterious segfault.
Since conda-build doesn't use pybind11 it's not exposed to this problem.
I'll see if I can fix up the pylief recipe to use the proper pybind11 version :)

Nice debugging! Thanks Wolf 😀

It looks like pybind11 and these others were not related. I've just tried the latest py-lief (0.12.1) with mambabuild.
I can see the lief process print a Load commands are corrupted which comes from Mach-O parsing in LIEF. The file that is parsed is Library\share\cmake-3.22\Modules\Internal\CPack\CPack.OSXScriptLauncher.in
This also happens with the version coming from conda-forge (I tested in the pkgs folder).
I wonder if that could be the problem although it is still strange that it would only affect mambabuild and not regular condag-build...


Add README

This needs a license header, or needs to be ignored in the license check: https://travis-ci.org/kaazing/gateway/builds/171497285

run mvn com.mycila:license-maven-plugin:2.11:format to add it

I excluded README.txt from the license check.


shouldResubscribe option

The problem
The main reason for this PR is I have a couple of cases where compose ends up in an infinite loop. It goes something like this:

On first call to my onPropsChange I start an async data fetch and fire onData which shows a contained component in one style
Later ,once data arrives, onData is fired again and the component changes look drastically
This seems to cause React to fire componentWillReceiveProps with the same prop values as before
react-komposer then calls my onPropsChange, and so we go back to step one
Repeat forever grinding that browser tab to a halt

See the note in the React docs of componentWillRecieveProps for an explanation that this being fired with the same props is expected.
At a higher level, sometimes (often) the lifecycle of the component has to be taken into account in onPropsChange. Many users will expect it to only fire once per component.
Previously there was no way to detect that onPropsChange is being called with the same props and the expensive / async operation does not need to be repeated.
What this PR does

The current !pure and !shallowEqual check for the props is done in componentWillRecieveProps before onPropsChange is fired, not just in shouldComponentUpdate. This will stop a lot of existing redundent calls to onPropsChange but might somehow cause a backwards incompatible change if current react-komposer users have impure onProposChange functions but are not setting pure: false (not very likely IMHO).
The above check can be overriden with a custom function passed in the options as shouldResubscribe. This allows the user to check for a change in only the props that affect the onPropsChange function. This is useful, for example, when there are multiple layers of compose components and props coming down from outer components then don't need to fire onProposChange on unrelated inner components.

Other implementation paths not taken
I thought about binding this to the component so cache check info could be kept on the component and checked by onPropsChange but that breaks the pure function principle that this package aims for. Same with having the same cache info in a variable outside the onPropsChange function (that also has bugs in certain scenarios if mutliple components are used).
I also thought about passing the old props into onPropsChange but that function already has enough parameters and it didn't seem good separation of concerns.
Tests / docs
Plenty of tests and API doc added.
Also added env: "mocha" to .eslintrc otherwise ESlint bawked on beforeEach().

Just committed a further addition that allows you to pass a fixed value (treated as truthy) for shouldResubscribe, instead of a function.

You pass in nextProps and context but you also need to pass this.context to the shouldResubscribe function, just like you do with this.props.

Agree about mocha in .eslintrc. Should have done that.
I thought about passing the old/current context into the function, but does componentWillReceiveProps get called when the context changes? Because if not then this.context and the context param will always be the same. That was my assumption.

I'm pretty sure it gets called when context changes because otherwise context would never update.

I'll add an enzyme test and see
On Wed, 6 Jul 2016 at 10:25 Curtis Layne<EMAIL_ADDRESS>wrote:

I'm pretty sure it gets called when context changes because otherwise
context would never update.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kadirahq/react-komposer/pull/99#issuecomment-230620925,
or mute the thread
https://github.com/notifications/unsubscribe/ABk9r1s0TQhOBEHAk1z8jNOYiJVQ5gSTks5qStn2gaJpZM4JEqcX
.

Now also allows checking of changes to context. Tests added and API doc updated.

I think you should squash your commits and rebase off of master so that this is ready to get pulled in. Make it as easy as possible for @arunoda.
I can't wait for this to make it in, performance will be WAY better.

@clayne11 This is my first PR so still figuring my way through. Will give it a go today.

Let me know if you get stuck and I can point you in the right direction.

@clayne11 I think the best way to learn is to muddle your way through. Each time I understand git a bit more and a bit less at the same time :)
Squashed and rebased. Ready to merge.

Looks good to me. Hopefully @arunoda will pull this in soon!
Fixes #4 & #2

Hey @arunoda, I think this PR will solve a few different issues. There's thorough testing. Any chance of a merge?

I've been using it in production and it's definitely fantastic. Please merge.

The ticket at https://github.com/mobxjs/mobx/issues/84#issuecomment-235652897 explains a good use-case for merging this in...

We really need this merged @arunoda. I'm currently using @damonmaria 's fork in production because I absolutely need this functionality.

Any movement on this @arunoda? This is a critical feature that we need to have pulled in.

Yep. This is pretty needed.
I'm going to start fresh with the 2.x targeting this problem with a cleaner API.
Releasing pretty soon (This week)

Can you please just merge this and release a minor version with it? It's a
critical feature and this PR has been sitting here for over a month.
On Tue, Sep 27, 2016, 02:16 Arunoda Susiripala<EMAIL_ADDRESS>wrote:

Yep. This is pretty needed.
I'm going to start fresh with the 2.x targeting this problem with a
cleaner API.
Releasing pretty soon (This week)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kadirahq/react-komposer/pull/99#issuecomment-249775866,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACs0VHmj3SnB4M5s9678gj3f4hQtBR9rks5quLTFgaJpZM4JEqcX
.

With v2, we've a similar API.
Check this.


Is there a way to add alt, placeholder to the Label tag?  Also is there a way to add angular directive to the elements?

Describe the bug
which template:

[ ] MaterialDesignFrameworkModule — Material Design
[ ] Bootstrap3FrameworkModule — Bootstrap 3
[ ] Bootstrap4FrameworkModule — Bootstrap 4
[ ] NoFrameworkModule — plain HTML
[ ] Other (please specify below)

A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

...

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Log output covering before error and any error statements
Insert log hereCopy

Additional context
Add any other context about the problem here.

Detailed Description

Context

Possible Implementation

You can add the key placeholder & give it a value in the json schema. As for the other options the library doesn't support it.


Case sensitive path issue: BugRegression -> bugregression

The folder that the test files are in is called 'bugregression', but the tests refer to it as 'BugRegression'. This causes it to fail to find the files on Linux.

Hi @Numpsy - if you can sign the CLA we can look at getting this merged

Signed

Working around a CLA webhook issue by closing & re-opening this pull request. Thanks for your contribution.

@Numpsy Thank you for contributing this fix!


drivers: pwm_nrfx: Fix the driver to allow changing period on the fly

Previously it was not possible to change the PWM period, even if only
a single channel was in use, without first stopping the peripheral,
i.e. setting pulse cycles for the channel to 0. This patch corrects
this behavior.
Fixes #15323.

@Mierunski and @kl-cruz could you please review this PR?


Re-Open a force closed Ticket

Is it Possible?

Closed tickets can be re-opened.
Force close differs from close only in who initiates the action. The standard workflow assumes the creator/owner closes the ticket. Force close is used by staff to closes an issue, rather having the owner do. They generate different activity messages, but the ticket's status is the same either way.

First of all thankyou for the quick answer, Stephen, it looks like the button isn´t displaying at all, I leave as evidence that i changed the language of everything in spanish, including buttons, may that be the issue?

That seems odd. This TD 2.5 or 2.1?

it's 2.1, any suggestions?

I'll need to dust off a copy and check into it for you.; it's been a while since I worked with it.

Ok, went back to re-familiarize myself with this feature in TD 2.1. There are no security checks controlling if or when the re-open button appears. As long as the ticket is resolved or closed, then there should be a re-open button. Force close doesn't do anything special other than close the ticket.
I can confirm that running the stock TD 2.1 code, closing, resolving, and force close all work as expected, and all result in displaying the re-open button.

Well i must have messed up something when translating to spanish, and it's been so long since i've looked into this project that i dont really know where to look for it ahahah, but, thank you very much for your concern and and answers.

Take a look at this view:  https://github.com/NullDesk/TicketDesk/blob/ticketdesk-v2.1/TicketDesk-2/TicketDesk.Web.Client/Views/TicketEditor/Controls/ActivityButtons.ascx
If you messed something up in translation, the if statements that control each button in this view are the most likely place to start.

So, I was checking what you told me and i found something disturbing xD, i have a dev project and the one already in production, both pointing to the same DB for this purpose, and i noticed that tickets that are resolved appear like this in production (wrong):

and like this in my local project (right i guess):

if you don't understand Spanish "Re-Abrir" stands for "Re open", the button i'm missing in production.
Maybe it isn't updating the ticket status correctly?

It certainly looks like production thinks that ticket is still active. I think it might depend on which application resolved the ticket. TD 2 was not designed for web farms at all, and each instance of the application tends to reuse ticket repository and EF contexts a lot; so you are probably just getting an the previously loaded ticket data, not fresh data from the DB.  You can confirm that by restarting production... it should then look just like your local when you go back to that ticket.

What do you mean by restarting it? what is the impact that that may have on the data that's already there in production?

I mean to just restart the web site. It wont impact any data stored in the DB, just force TD to re-query for fresh data... it's effectively doing the same thing as a system that uses an in-memory cache, though the mechanics are a bit different.

i've done that a few times before, when the site has crashed because of server issues, but i have never seen that that action brought up the missing button, i took a print of the settings table on SQL, maybe there's something wrong there, i also noticed that the version states as 2.0 there:

That version flag looks correct to me.
If you have two instances talking to the same DB, and one behaves differently from the other, it's very unlikely that the problem is the data, or the database. More probable that there is a runtime difference in state between the two running application instances, or the code itself isn't the same between the two instances.


Repository not found

Following official installation guide produces errors during installation.
lsb_release -a  No LSB modules are available. Distributor ID:	Ubuntu Description:	Ubuntu 18.10 Release:	18.10 Codename:	cosmic
sudo apt-get update Hit:1 http://archive.ubuntu.com/ubuntu cosmic InRelease Hit:2 http://archive.ubuntu.com/ubuntu cosmic-updates InRelease                                         Hit:3 http://archive.ubuntu.com/ubuntu cosmic-backports InRelease                                       Hit:4 http://archive.ubuntu.com/ubuntu cosmic-security InRelease                                        Ign:5 https://packages.microsoft.com/repos/azure-cli cosmic InRelease                                   Err:6 https://packages.microsoft.com/repos/azure-cli cosmic Release   404  Not Found [IP: <IP_ADDRESS> 443] Reading package lists... Done E: The repository 'https://packages.microsoft.com/repos/azure-cli cosmic Release' does not have a Release file. N: Updating from such a repository can't be done securely, and is therefore disabled by default. N: See apt-secure(8) manpage for repository creation and user configuration details.
Manual installation via bash script works. However, I cannot update the system anymore because of this error.

We are releasing the Ubuntu package that supports Cosmic later today.

Dupe of #7776. Closing.

We are releasing the Ubuntu package that supports Cosmic later today.

@tjprescott Just tried an update again and the same error is produced.

Did you run apt-get update first to refresh your repositories?

@tjprescott
This is output from today:
Last login: Wed Nov 21 14:13:29 2018 from <IP_ADDRESS>
worker@worker:~$ sudo apt-get update
[sudo] password for worker:
Hit:1 http://archive.ubuntu.com/ubuntu cosmic InRelease
Get:2 http://archive.ubuntu.com/ubuntu cosmic-updates InRelease [83.2 kB]
Ign:3 https://packages.microsoft.com/repos/azure-cli cosmic InRelease
Err:4 https://packages.microsoft.com/repos/azure-cli cosmic Release
404  Not Found [IP: <IP_ADDRESS> 443]
Hit:5 http://archive.ubuntu.com/ubuntu cosmic-backports InRelease
Get:6 http://archive.ubuntu.com/ubuntu cosmic-security InRelease [83.2 kB]
Get:7 http://archive.ubuntu.com/ubuntu cosmic-updates/main amd64 Packages [82.7 kB]
Get:8 http://archive.ubuntu.com/ubuntu cosmic-updates/universe amd64 Packages [40.6 kB]
Reading package lists... Done
E: The repository 'https://packages.microsoft.com/repos/azure-cli cosmic Release' does not have a Release file.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

Hi @nicolaibaralmueller yes, we have to open a new repo for these builds. Please keep an eye on #7776 which is tracking this issue.

I can confirm that update is working again.


remove Omayeli's bio and abstract :(

lgtm -  i very much prefer that emoji in the title removed though

I'm very okay with expressing sadness that one of the speakers we really wanted to come speak can't make it, but sure


Image attachment via "Attach" button doesn't work in v.19.2 - 19.4

This is a regression, I should had created a Test Case for this earlier...
The bug is caused by a changed behavior of the Message Editor. Now, as soon as Activity get paused (e.g. when an image selection dialog opens), the Editor is hidden and current draft is being saved in a background. After image selection is made, control returns to the Activity, but there is no open Editor there, and this is why selected image is ignored.
After Activity's return from a paused state the last draft is being loaded back to the Message Editor automatically, again, in a background, but this is too late: information on selected image is discarded already.
I think that now you are starting to understand the tricky asynchronous logic of the Message Editor?!
https://loadaverage.org/notice/7234537
PS: Sharing an image via AndStatus works,  so this allows to send posts with attachments even now.

Fixed in v.19.5


virtctl create vm cleanups

What this PR does
Before this PR:
After this PR:

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer
Merge after #13072

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

None

/retest-required

/cherry-pick release-1.4

/retest

/lgtm

Great job!
/approve


Cleanup auto_super_init

Constructors have a long history of attempted specifications, therefore a lot of complex cases and code was still present to avoid breaking thing.
Here, we simplify the code of auto_super_init that deal with injecting implicit call to various semantic of super in a constructor.
Now, there is only two remaining cases:

the constructor is a redefinition, then a genuine call to super (polymorphic and linearized) injected
the constructor is the introduction of a named constructor, then a call to the auto init is injected iff there is no conflict and no arguments.

reroll: rebase and adapt src to new code (this branch was quite old)

+1


Clear package.json

The actual devDependency field in our package.json is the one generated from the yeoman generator, we might want to remove some libraries but we have to find out which ones are useless.

The author field is generated too and no package.lock are commited (npm 5+)


Missing population denominator for vaccination quota calculations

Hi,
The Malta government recently claimed that Malta reached 70% of the population with first doses of vaccine, but there is no mention of the population denominator value for this. If one were to use the population value from the PCR tests data as shown in the README, one would come up with a percentage of ~60%.
There have been suggestions that the claim is based on the adult population, but it is not so clear. Could you point us to an exact value to be used, as is shown for the PCR tests, or to a source for such data?
Some other datasets for vaccinations in other countries include the quotas/percentages of first doses and full doses in their dataset, and would be a welcome addition in Malta's vaccination dataset.
Thanks in advance and regards.

Thank you for your observations and sorry for the delay in replying.  The COVID Public Health Data Management team is only involved in the publication of the numerators, as communicated by personnel in charge of the Vaccination Registration System.


replace buffer/difference geom subdividing with shapely split

resolves #479

Coverage decreased (-0.009%) to 92.834% when pulling b137916a62dd302e2ec65e69464bc59663cc1328 on fix into f64460278bf2504e1fc3d53aeb9be0f1f1e5f897 on master.


How do I change how much XP is required to level up

Character.level will be called within one of the processEndOfCombat functions (processEndOfMudCombat). There it will be, on default, playerlevel * 1000.

Thanks!

Oh wait that is going to bit added or it is there already?

leveling is not in; you can leave this open.

Ok cool!


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/47744

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/52202

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/52214

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.


[docs-infra] Fix dark/light mode

Summary 💡
The dark/light mode on http://mui.com/ feels all broken.

Open https://mui.com/material-ui/getting-started/
Toggle dark mode

https://github.com/mui/material-ui/assets/3165635/f4e10160-e33f-411a-89e8-13e7bc212f3d
Motivation 🔦
Noticed in https://github.com/mui/material-ui/pull/38792, this should fix https://mui.com/material-ui/experimental-api/css-theme-variables/usage/#getting-started.

Does it include this bug or should I open a new issue?
Any time you refresh the page while in dark mode it flashes white and then goes back to black. I installed the latest version of MUI and does the same thing in my local env.
https://github.com/mui/material-ui/assets/35123596/8f0538df-c644-40ba-8340-71c86ea11074

Duplicate of #38575


Window resize listener?

Tried
$(window).on('load resize orientationchange', function () {
	if (window.matchMedia("(min-width: 768px)").matches) {
		var stickyEl = new Sticksy('.s-cont > .sticky');
		stickyEl.hardRefresh()
	}else{
		Sticksy.disableAll(); ///seems do not clean the markup
	}
});

But after that i see a LOT of sticksy hidden nodes and element mispositioned till the page reload.
How to refresh state after window resize and clear the markup when sticky functionality is not needed?

Looks like you initialize Sticksy instance each time you match the media query.
if (window.matchMedia("(min-width: 768px)").matches) {
		var stickyEl = new Sticksy('.s-cont > .sticky');

You need to initialize sticky element once and then use the instance methods when you match the query.
Fixed version:
var stickyEl = new Sticksy('.s-cont > .sticky');
$(window).on('load resize orientationchange', function () {
	if (window.matchMedia("(min-width: 768px)").matches) {
		stickyEl.hardRefresh()
	} else{
		stickyEl.disable();
	}
});

@kovart Looks like once it inited it works without any listeners to window resize


Allow group read on cookie_secret_file

The test for bad permissions of the cookie_secret_file in app.py is not ideal for operating in an environment where you want to preven the process from overwriting the file. I'd like to separate concerns: the Hub process does not need to modify its configuration so it shouldn't be allowed to for security reasons. It only needs group read access. The owner of the file can then be a different user.
I think this particular test could be testing for world write, allowing normal use of group permissions to protect data:
if perm & 0o007:
    self.log.error("Bad permissions on %s", secret_file)

Note that this test is not sufficient to protect against security errors, since an admin can set cookie_secret  in the main config file.

That check makes sense to me, if you'd like to make a PR.


dashboards: Add Kong Connection Distribution.

Adds a dashboard which shows the distribution of active Kong connections across pods, nodes and zones.
Towards https://github.com/giantswarm/giantswarm/issues/21426.

Checklist

[x] Update changelog in CHANGELOG.md in an end-user friendly language.


Faster disableable CSS features

Closes https://github.com/refined-github/refined-github/issues/7944

All verified in both states.


lenghty FlatList (ListView) colliding with the motion animation

Hi. Your example works fine, but there is an issue when rendering the list rows. I´ve supplied a list of 24 items. But they are not rendered all. Sometimes only the first 9 items are rendered, other times only the first 20 items are rendered. The number is different. But never did I succeed in rendering all 24 items. I´ve found out that all the items (24) are available on the state but they are rendered up to the point when the onMoveToDestinationWillStart will fire as is apparent from the attachment. After this life-cycle fires, no more items get rendered. Is it, somehow, expected behavior?

Hey David 👋 Not sure if I understand the problem 🤔 FlatList has got some performance improvements so it doesn't render everything in once (as ListView did). Is that what you think?

Hey David  Not sure if I understand the problem  FlatList has got some performance improvements so it doesn't render everything in once (as ListView did). Is that what you think?

Sorry if I was not clear enough. The issue is that even though all 24 items are available on the state, they are not rendered all even if I scroll to the very bottom of the list. The onMoveToDestinationWillStart seems to somehow stop the flat list from rendering all items. If I reach the bottom of the list, the remaining items are not rendered.

Why is the onMoveToDestinationWillStart fired when you're just scrolling down?

Why is the onMoveToDestinationWillStart fired when you're just scrolling down?

It gets fired only once , ie. when navigating from the main list to detail. In fact, the issue is easily reproducible. Just make sure the data file returns more than 20 rows for the FlatList inside the Screens/Detail/Detail.js and you will probably not be able to see all list items even with scrolling. The issue is gone once the animation-making wrapper is commented out:
//<TranslateYAndOpacity isHidden={phase !== 'phase-2'} delay={56 * delay}>
        <View style={styles.itemContainer}>
          <Row style={styles.rowContainer}>
            <View style={styles.titleContainer}>
              <Text style={styles.titleText}>{item.hour}</Text>
            </View>
            <Text style={styles.amountText}>{item.money.EUR}</Text>
          </Row>
          <Text style={styles.vatText}>
            {`${"foo"} X1 (Including VAT 10%)`}
          </Text>
        </View>
//</TranslateYAndOpacity>


get all keys in generated schema

I can't remember why I thought this would be useful.
I don't see what the branch has to do with #1


pushbots returns valid with invalid secret

Yeah, it does. Might have to switch to a different end point. I guess pushbots doesn't care if the secret is invalid for stats.


A trigger difined as "FOR EACH STATEMENT" is shown as "FOR EACH ROW"

#9835 ## System information:

Operating system (distribution) and version: Ubuntu 20.10
DBeaver version: 7.2.2

Connection specification:

Database name and version: PostgreSql 12.2
Driver name: PostgreSql 42.2.18
Do you use tunnels or proxies (SSH, SOCKS, etc)? No

Describe the problem you're observing:
A trigger defined as FOR EACH STATEMENT is displayed in the definition as FOR EACH ROW
Steps to reproduce, if exist:

If you execute the following script, it shows the correct SQL:
select pg_get_triggerdef(oid)
from pg_trigger
where tgname = '__XXX_trigger_on_insert_position';
CREATE TRIGGER __XXX_trigger_on_insert_position AFTER INSERT ON "XXXXXXX".posicion REFERENCING NEW TABLE AS new_positions FOR EACH STATEMENT EXECUTE FUNCTION __XXX_on_insert_position()

Should be fixed in 7.2.3.


Implement Strict Mode Version 1

Pull Request (PR) description
Implemented strict mode when running all DSC resource unit tests.
A few changes was necessary in many or the resources to be able to successfully run with strict mode. Some of the resources had bugs that prevented them from working as intended in certain situations.
Implementation of strict mode version 2 is the goal, but it requires many more modifications in order to execute successfully.
This Pull Request (PR) fixes the following issues

Partly implements #901 (only strict mode version 1)

Task list

[ ] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md in the resource folder.
[ ] Resource parameter descriptions added/updated in schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[ ] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #999 into dev will increase coverage by <1%.
The diff coverage is 93%.

@@          Coverage Diff           @@
##             dev    #999    +/-   ##
======================================
+ Coverage     92%     92%   +<1%
======================================
  Files        116     116
  Lines      11977   11977
  Branches       1       1
======================================
+ Hits       11123   11125     +2
+ Misses       853     851     -2
  Partials       1       1