cloud-aws plugin does not put snapshot to s3-ceph API

This is my config in elasticsearch.yml for connecting to aws:
cloud:
    aws:
        access_key: my-key
        secret_key: my-secret-key
        protocol: http
        s3:
            endpoint: "http://s3.mydomain.net"
            bucket: "esrepo"
            signer: "S3SignerType"
        signer: "S3SignerType"
repositories:
    s3:
        endpoint: "http://s3.mydomain.net"
        bucket: "esrepo"
        access_key: my-key
        secret_key: my-secret-key
        protocol: http

When I start elasticsearch with enabling debug log, it notice:
[2015-12-03 09:38:34,117][DEBUG][cloud.aws                ] [Alkhema] using AWS AP I signer [S3SignerType]
[2015-12-03 09:38:34,122][WARN ][cloud.aws                ] [S3SignerType] may not  be supported

However, with this signer, I can connect to s3-ceph and verify snapshot repo with this command:
curl -XPUT 'http://localhost:9200/_snapshot/mysnap?verify=false' -d '{"type":"s3","settings":{"access_key":"my-key","secret_key":"my-secret-key","bucket":"esrepo","endpoint":"s3.mydomain.net","protocol":"http"}}'
elasticsearch log
[2015-12-03 09:41:02,589][DEBUG][repositories.s3          ] [Alkhema] using bucket [esrepo], region [null], endpoint [s3.mydomain.net], protocol [http], chunk_size [100mb], server_side_encryption [false], buffer_size [null], max_retries [3], cannedACL [null]
[2015-12-03 09:41:02,590][DEBUG][com.amazonaws.request    ] Sending Request: HEAD http://esrepo.s3.mydomain.net / Headers: (User-Agent: aws-sdk-java/1.10.33 Linux/2.6.32-573.3.1.el6.x86_64 Java_HotSpot(TM)_64-Bit_Server_VM/25.65-b01/1.8.0_65, Content-Type: application/x-www-form-urlencoded; charset=utf-8, )
[2015-12-03 09:41:02,590][DEBUG][com.amazonaws.services.s3.internal.S3Signer] Calculated string to sign:
"HEAD

application/x-www-form-urlencoded; charset=utf-8
Thu, 03 Dec 2015 02:41:02 GMT
/esrepo/"
[2015-12-03 09:41:02,593][DEBUG][com.amazonaws.http.impl.client.SdkHttpClient] Attempt 1 to execute request
[2015-12-03 09:41:02,604][DEBUG][com.amazonaws.requestId  ] x-amzn-RequestId: not available
[2015-12-03 09:41:02,604][DEBUG][com.amazonaws.request    ] Received successful response: 200, AWS Request ID: null
[2015-12-03 09:41:02,604][DEBUG][com.amazonaws.requestId  ] AWS Request ID: not available
[2015-12-03 09:41:02,605][INFO ][repositories             ] [Alkhema] update repository [mysnap]

--------------CEPH access -------------------------------
<IP_ADDRESS> - - [03/Dec/2015:09:42:23 +0700] "HEAD / HTTP/1.1" 200 242 "-" "aws-sdk-java/1.10.33 Linux/2.6.32-573.3.1.el6.x86_64 Java_HotSpot(TM)_64-Bit_Server_VM/25.65-b01/1.8.0_65"
-----------------------------------------------------------------------------------------------------------------------

It's connected okey but I can't put anything to repo, it time out and hang on:
--------------------------log------------------------------
[2015-12-03 09:58:33,509][DEBUG][com.amazonaws.http.impl.client.SdkHttpClient] Attempt 1 to execute request
[2015-12-03 09:59:26,564][DEBUG][com.amazonaws.http.impl.client.SdkHttpClient] Closing the connection.
[2015-12-03 09:59:26,566][INFO ][com.amazonaws.http.AmazonHttpClient] Unable to execute HTTP request: Read timed out
java.net.SocketTimeoutException: Read timed out
        at java.net.SocketInputStream.socketRead0(Native Method)
        at java.net.SocketInputStream.socketRead(SocketInputStream.java:116)
        at java.net.SocketInputStream.read(SocketInputStream.java:170)
        at java.net.SocketInputStream.read(SocketInputStream.java:141)
        at org.apache.http.impl.io.AbstractSessionInputBuffer.fillBuffer(AbstractSessionInputBuffer.java:160)
        at org.apache.http.impl.io.SocketInputBuffer.fillBuffer(SocketInputBuffer.java:84)
        at org.apache.http.impl.io.AbstractSessionInputBuffer.readLine(AbstractSessionInputBuffer.java:273)
        at org.apache.http.impl.conn.DefaultHttpResponseParser.parseHead(DefaultHttpResponseParser.java:140)
        at org.apache.http.impl.conn.DefaultHttpResponseParser.parseHead(DefaultHttpResponseParser.java:57)
        at org.apache.http.impl.io.AbstractMessageParser.parse(AbstractMessageParser.java:260)
        at org.apache.http.impl.AbstractHttpClientConnection.receiveResponseHeader(AbstractHttpClientConnection.java:283)
        at org.apache.http.impl.conn.DefaultClientConnection.receiveResponseHeader(DefaultClientConnection.java:251)
        at org.apache.http.impl.conn.ManagedClientConnectionImpl.receiveResponseHeader(ManagedClientConnectionImpl.java:197)
        at org.apache.http.protocol.HttpRequestExecutor.doReceiveResponse(HttpRequestExecutor.java:271)
        at com.amazonaws.http.protocol.SdkHttpRequestExecutor.doReceiveResponse(SdkHttpRequestExecutor.java:66)
        at org.apache.http.protocol.HttpRequestExecutor.execute(HttpRequestExecutor.java:123)
        at org.apache.http.impl.client.DefaultRequestDirector.tryExecute(DefaultRequestDirector.java:685)
        at org.apache.http.impl.client.DefaultRequestDirector.execute(DefaultRequestDirector.java:487)
        at org.apache.http.impl.client.AbstractHttpClient.doExecute(AbstractHttpClient.java:863)
        at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:82)
        at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:57)
        at com.amazonaws.http.AmazonHttpClient.executeOneRequest(AmazonHttpClient.java:769)
        at com.amazonaws.http.AmazonHttpClient.executeHelper(AmazonHttpClient.java:506)
        at com.amazonaws.http.AmazonHttpClient.execute(AmazonHttpClient.java:318)
        at com.amazonaws.services.s3.AmazonS3Client.invoke(AmazonS3Client.java:3595)
        at com.amazonaws.services.s3.AmazonS3Client.putObject(AmazonS3Client.java:1382)
        at org.elasticsearch.cloud.aws.blobstore.DefaultS3OutputStream.doUpload(DefaultS3OutputStream.java:143)
        at org.elasticsearch.cloud.aws.blobstore.DefaultS3OutputStream.upload(DefaultS3OutputStream.java:108)
        at org.elasticsearch.cloud.aws.blobstore.DefaultS3OutputStream.flush(DefaultS3OutputStream.java:90)
        at org.elasticsearch.cloud.aws.blobstore.S3OutputStream.flushBuffer(S3OutputStream.java:71)
        at org.elasticsearch.cloud.aws.blobstore.S3OutputStream.close(S3OutputStream.java:89)
        at org.elasticsearch.common.blobstore.support.AbstractLegacyBlobContainer.writeBlob(AbstractLegacyBlobContainer.java:76)
        at org.elasticsearch.repositories.blobstore.BlobStoreRepository.startVerification(BlobStoreRepository.java:644)
        at org.elasticsearch.repositories.RepositoriesService.verifyRepository(RepositoriesService.java:211)
        at org.elasticsearch.repositories.RepositoriesService$VerifyingRegisterRepositoryListener.onResponse(RepositoriesService.java:436)
        at org.elasticsearch.repositories.RepositoriesService$VerifyingRegisterRepositoryListener.onResponse(RepositoriesService.java:421)
        at org.elasticsearch.cluster.AckedClusterStateUpdateTask.onAllNodesAcked(AckedClusterStateUpdateTask.java:58)
        at org.elasticsearch.cluster.service.InternalClusterService$AckCountDownListener.onNodeAck(InternalClusterService.java:772)
        at org.elasticsearch.cluster.service.InternalClusterService$UpdateTask.run(InternalClusterService.java:521)
        at org.elasticsearch.common.util.concurrent.PrioritizedEsThreadPoolExecutor$TieBreakingPrioritizedRunnable.runAndClean(PrioritizedEsThreadPoolExecutor.java:231)
        at org.elasticsearch.common.util.concurrent.PrioritizedEsThreadPoolExecutor$TieBreakingPrioritizedRunnable.run(PrioritizedEsThreadPoolExecutor.java:194)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
        at java.lang.Thread.run(Thread.java:745)

And when I restart elasticsearch with changing in API type from S3SignerType to AWS3SignerType
---------------eslasticsearch log when restart---------------------------

[2015-12-03 09:54:16,868][DEBUG][com.amazonaws.request    ] Received error response: com.amazonaws.services.s3.model.AmazonS3Exception: Forbidden (Service: null; Status Code: 403; Error Code: 403 Forbidden; Request ID: null), S3 Extended Request ID: null
-----------------------------------------------------------------

I use 1 key for 2 case but AWS3SignerType cannot to access to S3-ceph API so I cannot do anything
Please help me, if u need to more log, inform me
I run this system with es 2.1, latest cloud-aws plugin, jdk 8 (java version "1.8.0_65")

Update output when run verify repo with S3SignerType API:
{"error":{"root_cause":[{"type":"repository_verification_exception","reason":"[esrepo] path  is not accessible on master node"}],"type":"repository_verification_exception","reason":"[esrepo] path  is not accessible on master node","caused_by":{"type":"i_o_exception","reason":"Unable to upload object tests-mVhkDiISRfuIJ66t-pwM3Q/master.dat-temp","caused_by":{"type":"amazon_client_exception","reason":"Unable to execute HTTP request: Read timed out","caused_by":{"type":"socket_timeout_exception","reason":"Read timed out"}}}},"status":500}

My PR solves this issue with ceph.

See my comment on the PR.  This PR last year should have fixed the issue: #13360. However, that is marked as 2.1, but I'm not sure it was actually backported. I can't find it anywhere in 2.x. @xuzha Can you check this?

Oh actually it looks like there was a backport, but it was not exactly the same as master. I think the warning you saw is a red herring, because it is trying to register an already registered type (on master it skips S3SignerType because it has statically registered it).

In 2.x is slightly different. Taking a look, but I think the S3SignerType is the right type to use for ceph .

I agree it is the right type. But with the 2.x change, it looks like it will still try to register S3SignerType even though it is always registered (which gets the IAE and prints the warning).

See here, https://github.com/elastic/elasticsearch/blob/2.1/plugins/cloud-aws/src/main/java/org/elasticsearch/cloud/aws/AwsSigner.java#L53-L60
The warning is actually don't stop user to setup the S3SignerType, we just overwrite the Signer by calling configuration.setSignerOverride(signer). It is not register it, right ?

true, aws sdk java 1.10.33 (which is used now) is registering it.

As you explain above, I cannot use s3 ceph to store my snapshot with this s3 repository plugin?
I've upgraded to the latest version of elasticsearch, 2.2

Same question here. Anyone have backup to cloud working with S3SignerType with ES 2.2.1 or 2.3.5?
In both I have "[S3SignerType] may not be supported" and amazonaws.services.s3.model.AmazonS3Exception: Gateway Time-out (Service: Amazon S3; Status Code: 504; Error Code: 504 Gateway Time-out; Request ID: null), S3 Extended Request ID: null

I'm currently using ES 2.4.1 with cloud-aws plugin version:
_cat/plugins?v

name     component     version     type
node-1   cloud-aws     2.4.1       j

After some digging it seems that the correct signer is: AWS3SignerType (at least with this current configuration)
I'm able create the repository but it always fails verification with the following error:
{
   "error": {
      "root_cause": [
         {
            "type": "repository_verification_exception",
            "reason": "[ceph_repo1] path [es-snapshots] is not accessible on master node"
         }
      ],
      "type": "repository_verification_exception",
      "reason": "[ceph_repo1] path [es-snapshots] is not accessible on master node",
      "caused_by": {
         "type": "i_o_exception",
         "reason": "Unable to upload object es-snapshots/tests-ZG71SQBeTWmAgce_0GNF8Q/master.dat-temp",
         "caused_by": {
            "type": "amazon_s3_exception",
            "reason": "null (Service: Amazon S3; Status Code: 403; Error Code: AccessDenied; Request ID: tx00000000000000006b8fe-00583f3b24-6c2f8-default)"
         }
      }
   },
   "status": 500
}

The bucket exist check succeeds with response 200 however the PUT fails with a 403.
Is this a plugin issue or is related to permissions?
I'm able to create directories/buckets using s3cmd and the same access_key and secret_key configured in the elasticsearch.yml.
Does anyone have any ideas?

As @jogura confirmed (and thanks a lot for the update), it seems that CEPH can be used with S3 plugin.
So I'm closing this issue.


AnimationPlayer method call bug

So this is the Node with the scripts i made

And these are the only methods that can be called via animation player

Is this a bug? or am I just missing something like a format?

We have an issue template for a reason, there's nothing we can do to answer this or help you with so little details.

nvm. i fixed the issue by restarting the engine.


feat: add env configuration + jsdom custom conf

This PR adds environmentConfiguration and jsdom configuration.
I need some help for happy-dom.

@userquin is it just to add happy-dom file and config?

@dammy001 I mean if this makes sense in happy-dom (it is just an example)?


Update Node.js to v18.20.2

This PR contains the following updates:

Package
Type
Update
Change

cimg/node
docker
minor
18.16.0 -> 18.20.2

Configuration
📅 Schedule: Branch creation - "before 4am on the first day of the month" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Self-Hosted Renovate Bot. Find all logs here.

 You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Rpm Failure

Mar 28 14:15:36 prvlopsdba01 puppet-agent[8370]: Could not update: Execution of '/usr/bin/yum -d 0 -e 0 -y list false' returned 1: Error: No matching Packages to list
Mar 28 14:15:36 prvlopsdba01 puppet-agent[8370]: (/Stage[main]/Rsyslog::Install/Package[false]/ensure) change from absent to latest failed: Could not update: Execution of '/usr/bin/yum -d 0 -e 0 -y list false' returned 1: Error: No matching Packages to list
Mar 28 14:15:36 prvlopsdba01 puppet-agent[8370]: (/Stage[main]/Rsyslog::Config/File[/etc/sysconfig/rsyslog]) Dependency Package[false] has failures: true
Mar 28 14:15:36 prvlopsdba01 puppet-agent[8370]: (/Stage[main]/Rsyslog::Config/File[/etc/sysconfig/rsyslog]) Skipping because of failed dependencies
Mar 28 14:15:36 prvlopsdba01 puppet-agent[8370]: Finished catalog run in 4.79 seconds

Please try the newest release and open this issue again, if the problem still exists.


Add "numberOfImagesToPreLoad" prop for smooth multi-image transitions

Feature
Solution for https://github.com/xiaolin/react-image-gallery/issues/428
Adds extra "pre-load" divs to the end of the carousel.
Before:

After:

Setting numberOfImagesToPreLoad to 3 actually makes the scrolling even smoother.

Tech debt story to add unit tests: https://rentpath.atlassian.net/browse/INVP-588
Note: the project currently does not use babel, so this will take some work.


RequestPatternBuilder has ability to clear/replace underlying Collection types.

Proposal
Currently, RequestPatternBuilder provides a like method that allows for instantiating a builder with all of the fields set similarly to a provide RequestPattern.
The utility of this is limited because all of the collections fields on RequestPatternBuilder either:

put (in the case of Maps, formParams, queryParams, etc.) or
add (in the case of lists, bodyPatterns, multiparts etc.)

There is no support to replace the underlying collection.  For example, if I wanted a RequestPattern like another, except with a disjoint set of bodyPatterns, that's currently impossible with the builder, this forces you to call the 14 parameter RequestPattern constructor directly.
We could solve this by expanding the RequestPatternBuilder API to either:

Add replacing* methods.  E.g. replacingRequestBodies(Collection<ContentPattern> contentPatterns) which will add the provided ContentPatterns to a new empty List set this.bodyPatterns
Change the semantics of with (which would be treated as replace) and supplement the API with add methods which would exhibit the current behavior of with methods.

(1) is probably less invasive and breaking, so I'd bias in that direction.  Naming is debatable, maybe replacingRequestBodiesWith is more fluent?
References
No response

Ran into this myself recently. This limitation forces you to do the following if wanting to swap only a collection:
val requestMapping = RequestPattern(
  originalRequest.scheme,
  originalRequest.host,
  originalRequest.port,
  originalRequest.urlMatcher,
  originalRequest.method,
  originalRequest.headers,
  originalRequest.pathParameters,
  originalRequest.queryParameters,
  originalRequest.formParameters,
  originalRequest.cookies,
  originalRequest.basicAuthCredentials,
  bodyPatterns,
  null,
  null,
  originalRequest.multipartPatterns
)

Thoughts on this change @tomakehurst / @oleg-nenashev ?  I'm willing to make it if you're willing to pull it.

I've actually been thinking about going back to full mutability with defensive copying in this context.
I feel like the immutability + builders approach has resulted in an awkward programming model without a huge amount of benefit. Plus it's not fully one or the other at the moment - some bits are still mutable.

Just to be clear, you're suggesting:

Remove withRequestBody(ContentPattern contentPattern)
Add withRequestBodies(Collection<ContentPattern> contentPatterns) and taking a defensive copy (i.e. ImmutableList.copyOf(contentPatterns) in the Builder setter.
We'd make this change for the full set of Builder field that modify an underlying collection:

  private Map<String, MultiValuePattern> headers = new LinkedHashMap<>();
  private Map<String, MultiValuePattern> queryParams = new LinkedHashMap<>();
  private Map<String, MultiValuePattern> formParams = new LinkedHashMap<>();
  private Map<String, StringValuePattern> pathParams = new LinkedHashMap<>();
  private List<ContentPattern<?>> bodyPatterns = new ArrayList<>();
  private Map<String, StringValuePattern> cookies = new LinkedHashMap<>();

I think this would be an improved pattern.
If the goal is to ensure that RequestPattern is deeply immutable, another option would be making the defensive copies in the constructor of RequestPattern, that way whether RequestPattern is created via RequestPattern.Builder or constructor Immutability is delivered.


window splitting not working

Hello and thank you for creating this! It looks like an awesome tool!
I have one issue:
My ~/.config/dmux/dmux.conf.toml looks like this:
dir = "~/work"
session_name = "work"
number_of_panes = 2
commands = ["nvim", "zsh"]

but when I run dmux <project> or even dmux -c nvim zsh -p 2 <project> inside my tmux session, I can see the new window being created, but it is not split in two. There is only one panel with zsh.
Is this a bug or am I doing something wrong?

huh, that's really interesting, let me look into it a bit

Running it locally I'm not seeing the same behavior, what system are you on?

There could be an issue with your nvim closing immediately, if you change it the dmux -c zsh zsh -p 2 <project> does that create 2 windows?

Thanks for the reply! Using zsh 2x did not change anything. At first I thought it was to do with my key bindings, but I checked the code and it seems you're using a proper tmux client.
I'm on macOS, Alacritty, tmux with zsh and neovim 0.5. I'd be happy to help debug this locally, if you can give me some pointers where to look. I'm super excited about this tool!

Hmm, I'll try to think of something to try debugging wise, what happens if you try dmux -p 3 . from inside the directory you want to open?

Also just opens a single panel.

Have the same problem. Running dmux -p 3 . from inside target directory produce the following:
 RUST_BACKTRACE=1 dmux -p 3 .
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', src/tmux.rs:279:40
stack backtrace:
   0: <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt
   1: core::fmt::write
   2: std::io::Write::write_fmt
   3: std::panicking::default_hook::{{closure}}
   4: std::panicking::default_hook
   5: std::panicking::rust_panic_with_hook
   6: rust_begin_unwind
   7: core::panicking::panic_fmt
   8: core::panicking::panic
   9: dmux::tmux::Window::all_in_session
  10: dmux::tmux::Session::all_sessions
  11: dmux::open_selected_dir
  12: dmux::run_command_from_args
  13: dmux::main
  14: std::rt::lang_start::{{closure}}
  15: std::panicking::try::do_call
  16: __rust_maybe_catch_panic
  17: std::rt::lang_start_internal
  18: main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

tmux version: 3.2a


Micro swallows error

I had a sytax error in my file, but all I got was:
micro: In order for `async` & `await` to work, you need to use at least Node.js 8!
micro: https://err.sh/micro/old-node-version

because of this line: https://github.com/zeit/micro/blob/master/lib/handler.js#L29
It was a good guess, but misleading and hard to debug.
(I'm aware of micro-dev, but I don't need all the devtools)

install Node V8 and run...

Oh, right - it only happens if your Node.js version doesn't support async and await.

I think @markmarijnissen is saying that he received the async/await message as false-negative. The actual problem was a JS syntax error in his code and then he got the misleading error message.

To wit:
$ node -v
v6.10.3

$ cat server.js
module.exports = async (req, res) => {
  (1+
}

$ micro server.js
micro: In order for `async` & `await` to work, you need to use at least Node.js 8!
micro: https://err.sh/micro/old-node-version

Yes, but the error is will only get triggered if: A syntax error occured PLUS your Node.js version doens't support async/await. So if after doing what the error says (upgrading) you see the actual error. Isn't that what we want? Or should we allow people to write APIs without async/await?

OP wasn't originally using async though. Sorry, my first example was bad since it used async.
For example:
$ node -v
v6.10.3

$ cat server.js
module.exports = function (req, res) {
  return (1+
}

$ micro server.js
micro: In order for `async` & `await` to work, you need to use at least Node.js 8!
micro: https://err.sh/micro/old-node-version

Yes, errors show never be misleading. If I want to use micro without async /await, it should be possible.
Also, if I use a async-to-gen, I do have support, but still the misleading error would be triggered. (your async detection cannot detect async to Gen in all cases)

My use case is aws lambda, which is still stuck at 6. 10.


Cannot login

Describe the bug
I have already logged in using my KU account, but it still asks me to log in.
To Reproduce
Steps to reproduce the behavior:

Go to home page
Click on login
Log in using a KU account.
After logging in, the message "Please login. Profile creation is needed for the user to be authenticated" still appears.

Expected behavior
Should be redirected to an authenticated page after logging in.
Screenshots

Desktop (please complete the following information):

OS: IOS
Browser: safari
Version: 16.5

Mac user here. Because of safari/Apple security it's cause our app won't work properly. To make our web work properly you can go to toolbar on the top then go to Safari > Settings > Privacy > Unchecked Prevent cross-site tracking. This will make our website work properly. But if you're uncomfortable to do that you may need to use other browser such as Chrome, Firefox instead. Sorry for inconvenience.


Tweak ClrArray

Reverts #787 - it did nothing.
Deprecates GetStructValue(int, bool) - in ClrMD you simply can't get a non-interior struct value.
Adds multi-dimensional GetStructValue.
Fixes multi-dimensional GetObjectValue.
Resolves code duplication.
Reduces generic instantiation.

@NextTurn A while back someone suggested adding a NuGet package which generates a public API .txt file so that it's easy to see when someone makes a breaking change to the surface area of the library through diffs.  Do you happen to remember that at all?  I can't seem to find the issue or comment that brought it up and it seems like a good time to add it to the library.

A while back someone suggested adding a NuGet package which generates a public API .txt file so that it's easy to see when someone makes a breaking change to the surface area of the library through diffs. Do you happen to remember that at all? I can't seem to find the issue or comment that brought it up and it seems like a good time to add it to the library.

It was suggested by @drewnoakes in https://github.com/microsoft/clrmd/issues/624#issuecomment-623246855.

@chrisnas Shipped here:  https://www.nuget.org/packages/Microsoft.Diagnostics.Runtime/2.0.145301

@NextTurn A while back someone suggested adding a NuGet package which generates a public API .txt file so that it's easy to see when someone makes a breaking change to the surface area of the library through diffs.

It was suggested by @drewnoakes in #624 (comment).

Implemented in #826.


python3Packages.streamz: 0.6.2 -> 0.6.3

Motivation for this change
Update to latest upstream release 0.6.3

Switch to pytestCheckHook

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.streamz
python39Packages.streamz


Display spinning/loading animation while an LLM response message is still streaming

Problem
Currently, the UI makes it impossible to tell if a model has finished streaming its response back to the user, or if it is still underway and is just taking a long time to calculate the response. This is especially prevalent on slower machines without AVX-512 or CUDA, and even more of a problem when the message being returned by the model is very very long.
Proposed fix
To improve the user experience, a small spinning loading wheel (or some other kind of animation) should be added either beneath or next to the latest message in the chat view, which will spin or animate in some way that makes it obvious that the model is still generating a streaming a response. Then, once the message streaming has completed, the spinning animation would end and set itself to non-visible.
Additional improvements
Alternatively, that spinning animation could change to a tiny checkmark icon or something similar, in order to indicate that the message had been fully streamed from the model.
In addition, this small loading animation could also be displayed while a model is being loaded in the background, which again can take quite a long time (20+ seconds) on slower machines. This would help inform the user that something is indeed still happening in the background, rather than leaving the user confused while the UI does nothing for dozens of seconds.
Implementation starter ideas
The wasmedge frontend/backend does make it obvious when a message has completed being streamed. You can observe this while looking at the console log:
[2024-08-06T21:56:32Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:33Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:33Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:33Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:34Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:34Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:35Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:35Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:35Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:36Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:36Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06T21:56:36Z INFO  llama-core] Get output buffer generated by the model named Meta-Llama-3-8B-Instruct-f16.gguf in the stream mode.
[2024-08-06 14:56:36.967] [info] [WASI-NN] GGML backend: EOS token found

The last statement indicates that the stream has ended. Sometimes you may also observer other forms of stream ending, most of which are currently handled at the Rust level via the StopReason enum. Search for both StopReason and finish_reason to help you get started.

Here's an example illustrating the need for this feature. In the screenshot below, the model has actually finished sending its response message, but it doesn't seem like it's finished (based on the strange content where it just kind of ... trails off without finishing the sentence).
This is confusing because there's no definitive way for the user to know whether the model is done responding, or if it's just taking a long time to calculate the next words.


Create manual trigger for gh-pages deployment

Is your feature request related to a problem? Please describe.
Only admin commits/merges into dev branch trigger a gh-pages deployment due to permissions.
Describe the solution you'd like
Add a manual trigger for gh-pages deployment by adding a workflow_dispatch flag to the github action workflow configuration.

Looks good:

Manual deployment successful run: https://github.com/CityOfDetroit/COD-Design-System/actions/runs/9372684970
Generate gh-pages deployment successful: https://github.com/CityOfDetroit/COD-Design-System/actions/runs/9372703345


suggestion to add more exchanges

if it developed to add other exchanges like KuCoin etc.. with wss real-time live streaming data .. that's will be insane.. thanks in advance.

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.


[3.x] Need to provide dynamic connectionPayload to SubscriptionWsProtocol

Is your feature request related to a problem? Please describe.
Authentication token needs to be provided for SubscriptionWsProtocol via payload. When token expires and client re-connects new token needs to be used.
Describe the solution you'd like
Currently SubscriptionWsProtocol only accepts value when client is constructed. It would be nice if it would accept connectionPayload : value|lambda|suspendable lambda as it is implemented in apollo js counterpart

Hi @audkar ! Thanks for the feature request. Can you detail that part:

When token expires and client re-connects new token needs to be used.

I think it's pretty well established that a 401 error requires a new token using plain HTTP but I'm unsure about websocket. Especially since there's no HTTP code like 401? What should trigger the refresh?

This is my first time working with graphql so I am not an authority by any means :smile:
But in Apollo server documentation is written that connection is rejected

In case of an authentication error, the Promise will be rejected, and the client's connection will be rejected as well.

I'll try to dig into this. What I'm trying to understand is how much this is a "specified thing" vs "each server will have a different implementation and we'll need to add APIs to customize that".
We can start with your use case. What server are you using? Is it available publicly by any chance?

We are using Hasura which uses same authentication as Apollo server

I see, thanks for the feedback! Maybe it's simply a matter of making
class SubscriptionWsProtocol(
    private val connectionPayload: Map<String, Any?>? = null,
    override val frameType: WsFrameType = WsFrameType.Binary
)

instead of
class SubscriptionWsProtocol(
    private val connectionPayload: () -> Map<String, Any?>? = {null},
    override val frameType: WsFrameType = WsFrameType.Binary
)

Could you try with your backend and see if that works?

That helps to retrieve payload on connectionInit(). But:

It would be nice to have suspend lambda
I don't see actual logic which reconnects when server rejects connection.

@audkar that should be in the latest  SNAPSHOTs, I'll push a tag soon.

3.0.0-alpha01 now has a suspend fun callback for subscriptionPayload. I'm closing this issue.  Feel free to open a new one for reconnections if needed


Use command-color for shebang recipes

The recent command-color flag works nicely for regular just recipes, but it's not used for shebang recipes (e.g. those which start with #!).
So if I do just --command-color purple foo, it will echo recipe lines in purple if foo is a regular recipe, but will still use white if foo is a shebang recipe.
Unless anyone thinks this is the desired behavior, I can probably work on a fix for this.

Using the --command-color for shebang recipes seems reasonable to me!


add download count to navbar

count updates when page is reloaded if app has been installed by a team.

the count I implemented (which right now is bugged) counts the number of times the app has translated the time. We could also count the slack groups that have installed the app with a .length of the database

got it. will make a note to reflect that on the navbar. we can add the #of slack groups that have installed later

I think we can close this issue, since it's on production now.


Duplicate Cities

Many cities exist twice. Causing a pretty serious problem in React because I'm using the city name as the key.

Hey @rahulakrishna
Two states might have a city with the same name. There's a city called Rudrapur near Nainital and another one near Gorakhpur. So, I'd suggest use the iteration ID, or the combination of city and state as a key.
If there are some city names that are repeated with the same state, let me know :smile:


Comment Flag issue

Feature request-
When the user has flagged a comment then the flag button disables, but still we are able to click on it and it is redirecting to user profile
Is your feature request related to a problem? Please describe. (optional)
A clear and concise description of what the problem is.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Describe how this feature will be useful to our readers.
Users may become frustrated, if they are trying to flag a comment but they are getting redirected to user profile
Additional context
Add any other context or screenshots about the feature request here.

Note:

Do mention if your are working under GSSoC'21.
If you want to work on an issue, you should check if it has already been assigned to anyone. If the issue is free you can comment /assign to get the issue assigned to you.
If you are raising a new issue and want to work on it then also you should comment /assign under the issue to get it auto assigned.
Please refrain from adding labels to your issue/pull-request on your own. It is the job of the Project Admin and the Mentors to review your issue/pull-request and add labels accordingly.
According to the level of the issue, respective time will be allotted under which the task needs to be completed/resolved by the assigned person. If the issue is not resolved within the given time, it will be assigned to the next person. In case of any discrepancies, the decision of the maintainer will be final.

level - 0: 2 days
level - 1: 3 days
level - 2: 4 days
level - 3: 5 days

On clicking on flag button, it is redirecting to profile

/assign


ht32f5xxxx RAM limit increase

Crate name: ht32f5xxxx
Build failure link: https://docs.rs/crate/ht32f5xxxx/0.1.0/builds/254183
Requested RAM limit: Not 100% sure how much it takes...though it seems like the current limits aren't enough
Why your crate needs the resource increases:
It's a PAC crate for embedded Rust, hence it consumes bigger amounts of memory during docs generation as the amount of automatically generated code is quite huge. Judging from the log line
[INFO] [stderr] WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap.

I'm guessing it's an issue with the RAM limits.

As the other crate, the error is:
[INFO] [stderr] error: Package `ht32f5xxxx v0.1.0 (/opt/rustwide/workdir)` does not have these features: `þ32f52344_54`


FormData example?

How do I perform a POST with formdata? I see mentions of formData in the code, but it's left abstract and it's not clear from the docs how I would construct a formData object to pass into a makeWithFormData call.

You'd have to provide your own bindings to do this. There's a PR in #14 which was going to add this, but I'm not sure what state it is in now. This seems like a good reminder to follow up on that.
In the meantime, you might just poach those bindings.


Static checker does not find that (a+b) != (a+b+1)

    static int Sum(int a, int b)
    {
        Contract.Ensures(Contract.Result<int>() == a + b);
        return a + b + 1;
    }

Should this not warn? This indeed fails at runtime with a CC exception. For example:
    static void Main()
    {
        Sum(1, 2);
    }

I configured the static checker aggressively and so far it has found other problems already so it's basically working. I also waited for cccheck.exe to exit.
I understand that the static checker is not omniscient but this seems to be a rather simple case.
Here it also does not find any problem:
    static int Sum(int a, int b)
    {
        Contract.Ensures(Contract.Result<int>() == a + b);
        return 0;
    }

Only calls to Sum are marked as errors. But it's not the call that is at fault. It's Sum. The message is:

This does not make sense to me. There is no such precondition.

What happens if you disable "Infer requires"?

If I turn off "Infer requires" the ensures is proven false as desired. But what possible requires could lead to the ensures not being false? It's false for all inputs. Was the requires inferred as false? :)

No idea, I don't use Infer Requires. Suggest Requires seems to be more valuable.
If Yu run with Suggest Requires, does it tell you what it inferred, or does it just stay silent?

Nothing is suggested in that case. Appears to be an inference bug then.
For now I'm not using the infer options per your suggestion.


fix: (try to) convert the type of the Equals comparator's value to attribute type

@ayliao raised an issue where his filter was failing to find a match on an alarm's status attribute. This was happening because the value provided to the comparator (a string) did not have the same type as the target attribute (AlarmStatusValue).
To address this in a general way, we try to convert the type of the value passed into the comparator to that of the attribute's type being filtered on.

actually I was about to ask why not using to.ConvertibleTo(u Type) instead of recovering the panic?

I confirm that the following code passes all the tests and is way more elegant
func safeConvert(to reflect.Type, fromV reflect.Value) (value reflect.Value) {

	if !fromV.Type().ConvertibleTo(to) {
		return fromV
	}

	return fromV.Convert(to)
}

@primalmotion that looks good! Nice find, let me just look at the implementation a bit more to see if there are any 💥cases 👌


Can not remove or add GlobalParameter using CSV file ?

Hi,
I´m trying to remove and add  global parameters from ADF Target, using de config csv file.
Syntaxes i have tried to REMOVE/DROP a global parameter called "STORAGE_CLIENT_ID"
factory,stix-bi-qa-datafactory-tst,"$.properties.globalParameters.'STORAGE_CLIENT_ID'.value","ABC"
factory,stix-bi-qa-datafactory-tst,"$.properties.globalParameters.'STORAGE_CLIENT_ID'.value",""
factory,stix-bi-qa-datafactory-tst,-"$.properties.globalParameters.'STORAGE_CLIENT_ID'.value",""
factory,stix-bi-qa-datafactory-tst,-"$.properties.globalParameters.'STORAGE_CLIENT_ID'.value",
factory,stix-bi-qa-datafactory-tst,"-$.properties.globalParameters.'STORAGE_CLIENT_ID'.value",""
factory,stix-bi-qa-datafactory-tst,"-$.properties.globalParameters.'STORAGE_CLIENT_ID'.value",

Syntaxes i have tried to a ADD a new global parameter called "EXAMPLE_PARAMETER"
factory,stix-bi-qa-datafactory-tst,+"$.properties.globalParameters.'NEW_EXAMPLE_PARAMETER'.value","123"
factory,stix-bi-qa-datafactory-tst,"+$.properties.globalParameters.'NEW_EXAMPLE_PARAMETER'.value","123"

For all these examples, the CSV file was not correct interpreted. I tried to follow examples on link "https://marketplace.visualstudio.com/items?itemName=SQLPlayer.DataFactoryTools"
Could someone provide me the correct syntaxes to add and remove parameters by CSV file?
Best regards,
Sergio Coutinho

Can you try the following syntax:
factory,stix-bi-qa-datafactory-tst,-$.properties.globalParameters.STORAGE_CLIENT_ID.value,""

NowinskiK,
Thank you very for your orientation. I have tried these two options:
factory,stix-bi-qa-datafactory-tst,-$.properties.globalParameters.STORAGE_CLIENT_ID.value,
factory,stix-bi-qa-datafactory-tst,-$.properties.globalParameters.STORAGE_CLIENT_ID.value,""

But unfortunately, for both options i had the same error:
2022-02-22T23:03:26.9234050Z ##[error]'Value' cannot be null.
Is there another syntax option to remove the global parameter?
Best regards,
Sergio
Best regards,
Sergio Coutinho

NowinskiK,
Finaly I could remove the global parameter using this syntax:
factory,stix-bi-dv-datafactory,-$.properties.globalParameters.STORAGE_CLIENT_ID,
As I am trying to remove, so i didn´t need to inform ".value" or ".type" in this operation.]
Best regards,
Sergio Coutinho
as

Great. Thanks for letting me know! It will help others once they face a similar issue.


New Hive defaults to st 10

When adjusting step size, and then creating a new hive, the new hive defaults to st 10.  If you slide the bar again, it re-adjusts correctly.    New hive should pick up the modified step size.

I'll look into it!


Socket IO failing to clone

Tried building TermKit today but the Socket IO submodule fails when cloning.
git submodule update --init
Cloning into Node/node_modules/socket.io-client...
fatal: https://github.com/LearnBoost/Socket.IO.git/info/refs not found: did you run git update-server-info on the server?
Clone of 'https://github.com/LearnBoost/Socket.IO.git' into submodule path 'Node/node_modules/socket.io-client' failed

Can someone at least get this project into a usable state? I want to experiment and contribute to it, but even getting it running is a PITA

@unconed let me know if you need any help on my end
Adding a package.json pointing to the right socket.io and socket.io-client would fix this easily.


Adding logic to the constructor for wellness encounters.

#133
Wellness encounters will be only on weekdays, during business hours.
This is a first pass -- the referenced issue talks about wanting a more flexibility around setup. For this pass, I've set all WELLNESS encounters to business days during business hours.
I'm opening this up to start the discussion of what the community wants to control this (configuration? and if so, how configurable?)
Thanks!

@adamdavis40208 Thank you for the pull request!
I love it. However, it needs a little bit of work still.
A few issues:

When I looked at the FHIR output, one of the first examples I looked at had a wellness encounter that started at 8:21 AM. Fine... it must have changed the hours to be 8 AM as designed.

It would be nice if it picked a start time uniformly distributed during business hours.

Except the wellness encounter now ends at what I believe is the original time, which appears to be 11:21 AM. See period.start and period.end in the example below. So the wellness encounter, which should take 15 minutes, somehow took 3 hours.
My guess is that this end time of 11:21 AM has something to do with the EncounterEnd state which is using the original unaltered time to end the encounter, but I haven't verified this.
Next, all the associated resource activity in this encounter starts at 11:21 AM -- which appears to be the original unaltered time of the encounter. See the Observation included in the abridged example below.

Example:
      "fullUrl": "urn:uuid:6672cb24-ae67-4c09-934f-5384bffcc837",
      "resource": {
        "resourceType": "Encounter",
        "id": "6672cb24-ae67-4c09-934f-5384bffcc837",
        "status": "finished",
        "class": {
          "code": "WELLNESS"
        },
        "type": [
          {
            "coding": [
              {
                "system": "http://snomed.info/sct",
                "code": "185349003",
                "display": "Encounter for check up (procedure)"
              }
            ],
            "text": "Encounter for check up (procedure)"
          }
        ],
        "subject": {
          "reference": "urn:uuid:dc3c317c-9c41-48c4-b5bc-e501d71f0e40"
        },
        "participant": [
          {
            "individual": {
              "reference": "urn:uuid:f93083bd-9b54-4996-bac1-fdba8c21c62a"
            }
          }
        ],
        "period": {
          "start": "2009-07-07T08:21:23-04:00",
          "end": "2009-07-07T11:21:23-04:00"
        },
        "serviceProvider": {
          "reference": "urn:uuid:2bf83548-7fde-411b-aa35-45a6e3d47b77"
        }
      },
      "request": {
        "method": "POST",
        "url": "Encounter"
      }
    },
    {
      "fullUrl": "urn:uuid:2803def9-8ac7-440e-9776-32adffc710f0",
      "resource": {
        "resourceType": "Observation",
        "id": "2803def9-8ac7-440e-9776-32adffc710f0",
        "status": "final",
        "category": [
          {
            "coding": [
              {
                "system": "http://hl7.org/fhir/observation-category",
                "code": "vital-signs",
                "display": "vital-signs"
              }
            ]
          }
        ],
        "code": {
          "coding": [
            {
              "system": "http://loinc.org",
              "code": "8302-2",
              "display": "Body Height"
            }
          ],
          "text": "Body Height"
        },
        "subject": {
          "reference": "urn:uuid:dc3c317c-9c41-48c4-b5bc-e501d71f0e40"
        },
        "context": {
          "reference": "urn:uuid:6672cb24-ae67-4c09-934f-5384bffcc837"
        },
        "effectiveDateTime": "2009-07-07T11:21:23-04:00",
        "issued": "2009-07-07T11:21:23.264-04:00",
        "valueQuantity": {
          "value":<PHONE_NUMBER>238728,
          "unit": "cm",
          "system": "http://unitsofmeasure.org",
          "code": "cm"
        }
      },
      "request": {
        "method": "POST",
        "url": "Observation"
      }
    },

It would be nice if it picked a start time uniformly distributed during business hours.

It would be nice if it picked a start time uniformly distributed during business hours.
That one is easy, I'll just pick random hours -- do you mind if the end time is past business hours?

And i'll look into the 11:21 issue, nice catch! I'll try to add a test for start/end times.

Another thing I'm seeing: I'm in PST (UTC -8) and the current "isBusinessHours" function is assuming UTC. Want me to use the person's timezone? (or are people just using the system timezone?)
Still digging into the encounter period end, it's very all over the place. Most people have 15 minutes, sometimes the same person has 15 minutes or hours, for their wellness encounters. Something weird going on there :|

Thank you for the pull request, but it is being closed due to conflicts and inactivity.


Figure out February scheduling for Robyn and Matt

@rmarow
There's both a budget limitation and time limitation (I have a hard stop for another project in March, and AON has their own time constraints) for this work.
We need to talk a little bit about how, based on what work is remaining at the start of the new year, we want to schedule ourselves. I think we should meet for 30 minutes or so to go over the remaining priorities. What things should be done solo, what should be parallel, and what should be paired?
We definitely should reserve some pairing time for architecture stuff and/or anything that's important to cross-train on. But we should also consider AON's budget constraints (the analogy I often hear is "9 people can't make a pregnancy take 1 month") and only use the time for pairing that we absolutely need.

Our top priority is to maximize our overlap.


static NavigationOptions lost the props when use Decorator@

If I use decorators, the navigator does not detect the static navigationOptions.
Is there a better way to solute it?
This does not work:
@NetInfoDecorator
export default class ManagerView extends Component {
static navigationOptions = { headerTitle: 'manager' };
}
No title !!!!!
I can show the title when I don't use the decorator
This does not work:
export default class ManagerView extends Component {
static navigationOptions = { headerTitle: 'manager' };
}

Closing because you haven't followed the issue template; please open a new one that follows the template.
Without that, it's nearly impossible to understand & investigate the issue. I suggest you also try (in the new issue) to answer/explain in a way that fits these questions:

have you tried updating to the latest release?
does it happen only on Android/iOS?
does it happen on both Debug and Release modes?
does it happen with Debugger On or Off? Both?
Can you provide a repo or a snack to reproduce this issue?

same problem. fixed it?


WIP refractor

Notes:

The stylesheet parsing thing is brittle, mostly hacked together something for it to work. I'm also using this for react-native, so I care less than I should about really retaining the styles.
Test are broken but just need new snapshots. Didn't really want to fix that until I was confident about the styles.
No automatic style detection support in refractor.
Obviously feel free to remove new package definition and minor license changes &/or pull this in as its own branch.

@maxmcd thanks for the info, goal 1 will be having them both work, then trying to make more friendly slash seeing if i can styles less brittle

@maxmcd still some work to do but basic version working here -- https://github.com/conorhastings/react-syntax-highlighter/pull/89 -- to use refractor on that branch you just require 'react-syntax-highlighter/prism'


change font family to IBM Plex Sans

What would you like to be added:
Update global font family to IBM Plex Sans https://fonts.google.com/specimen/IBM+Plex+Sans to be consistent with design
Why is this needed:

/triage accepted
/area ui-ux core
/assign @thedoublejay
/priority urgent-now


Fix for lower case cell reference reading ("a1" vs "A1").

Forces upper case on read reference value.
If the reference value is lower case ParseReference would always return false and 0 for the column.
LowerCaseReferenceExample..xlsx

Thanks!
Could you add the example as a test case?


Create mark/unmark DG documentation

Codecov Report

Merging #126 (14514fd) into master (16b6656) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #126   +/-   ##
=========================================
  Coverage     39.65%   39.65%
  Complexity      406      406
=========================================
  Files           119      119
  Lines          2290     2290
  Branches        258      258
=========================================
  Hits            908      908
  Misses         1323     1323
  Partials         59       59

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 16b6656...14514fd. Read the comment docs.


failed to migrate with new database using installer

Bug report
Title
failed to migrate with new database
Issue Description
migration is failed when setup with new database
Preconditions
1. framework Version- master

Steps to reproduce
1. step1- upgrade code with latest master
2. step2- add new database from web installer

Actual result

fixed


Black Boxes in the distance

Describe the bug
I see black boxes in the distance. They follow me when i move.
To Reproduce
Steps to reproduce the behavior:
Apply the shader. that's it.
Expected behavior
A clear and concise description of what you expected to happen.
I though everything would be ok, but no.
Screenshots
If applicable, add screenshots to help explain your problem.

System Information
I'm using 1.21.4, shader version 2.13.6. I'm using an intel UHD Graphics 610, with Linux Mint 22.
Mods list : (note the file is too big so i took screenshots of the mods)

Additional context
Add any other context about the problem here.
I will provide the log file. (Note: I used other shaders to see if it was only with miniature shader and yes it is) :
latest.log

The log has a bunch of errors that are impossible to occur normally, so it's probably a mod conflict. See if this happens with just Iris or Optifine, otherwise it'll probably be fixed by the culprit eventually.

fixed, it was with "raise the cloud"


Add a godot-mono config (C#)

Added a very basic Godot Mono (C#) config
Unable to test it because of unifree bugs on my machine (yaml is valid however)
Needs to be reviewed (not sure about all the flags in target and source/csharp)

@FlooferLand config looks reasonable, have you tried running it?

@FlooferLand config looks reasonable, have you tried running it?

Unfortunately can't get unifree to run at all on my system. Hence the "Unable to test it because of unifree bugs on my machine"

@FlooferLand I really should read comments better :)
Let me merge it none the less.


Docker commands inconsistency

Filing a bug based upon the tweet at https://twitter.com/agoncal/status/786187878195863552
docker volume ls / docker service ls / docker network ls. So why docker images and not docker image ls ? #Docker #Consistency
Thanks @agoncal!

Based on the other Docker commands, I would expect the image command to be:
docker image inspect
docker image ls
docker image rm
...

cc @bfirsh

See  #26025, should be in 1.13

Ah as that issue is merged, will close this one.


Does it use nnunetv1 or nnunetv2?

Please read the following resources before posting issues:
Common questions:
https://github.com/MIC-DKFZ/nnUNet/blob/master/documentation/common_questions.md
Common Problems and their solutions:
https://github.com/MIC-DKFZ/nnUNet/blob/master/documentation/common_problems_and_solutions.md
Expected epoch times and tips on how to identify bottlenecks:
https://github.com/MIC-DKFZ/nnUNet/blob/master/documentation/expected_epoch_times.md

Hello @tangqishi,
This repo used nnunetv1.

Thanks so much! I found that if using Docker, the input folder needs to contain files that have been preprocessed. However, if I want to use the data preprocessing function of nnUNet within Docker as well, how should I proceed?

Thank you very much. I would like to use a Docker image. Could you please tell me what should be stored in the input folder? As shown in the picture, these are files obtained from preprocessing with nnU-Net. Should I place these files into it?


Investigate the use of UpperCut

Based on Rob's suggestion, take a look at UpperCut for controlling the build pipeline of ChocolateyGUI

Use Cake instead. 👍
And let's get UppercuT using Cake.

Yup yup. At least for now, the build scripts work, and work well (thanks @gep13!) so I'm not going to touch them. But cake will def be preference to move to in the future.

@ferventcoder @RichiCoder1 Cake would get my vote too.

I'm biased to I'll just :+1:  :wink:


Interactive window stdout stops working after 1-2 days of usage

Applies To

[ ] Notebooks (.ipynb files)
[X] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
After running multiple Python files in interactive mode for a couple of days in a row, the stdout stops working.

More puzzling, after clicking around in the UI to find the extension version et al while submitting this bug, the output reappeared.

VS Code Version
[Window Title] Visual Studio Code  [Main Instruction] Visual Studio Code  [Content]  Version: 1.69.1 (user setup) Commit: b06ae3b2d2dbfe28bca3134cc6be65935cdfea6a Date: 2022-07-12T08:21:24.514Z (6 mos ago) Electron: 18.3.5 Chromium: 100.0.4896.160 Node.js: 16.13.2 V8: <IP_ADDRESS>-electron.0 OS: Windows_NT x64 10.0.14393  [OK] [Copy]
Jupyter Extension Version
v2022.6.1201981810
Jupyter logs
info 5:39:30.659: Attempting to start a server because of preload conditions ...
info 5:39:42.400: Execute Cell 320 Interactive-2.interactive
info 5:39:42.449: Cell 320 executed with state Success
info 5:46:7.295: Executing silently Code (idle) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info
info 5:46:7.328: Executing silently Code (completed) = def _VSCODE_getVariable(what_to_get, is_debugging, *args):\n# Query Jupyter server for the info with 1 output(s)

Coding Language and Runtime Version
Python v3.9.7
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

Sorry we didn't get around to investigating this. I haven't seen this myself or any similar issues, so I'll close for now, but let us know if this is still occurring.


Need to add domainName property to Thycotic.PowerShell.FolderPermissions.Permission class

Verified issue does not already exist?
Yes
What error did you receive
| Cannot convert value "" to type "Thycotic.PowerShell.FolderPermissions.Permission". Error: "Cannot
| convert the "" value of type "System.Management.Automation.PSCustomObject" to type
| "Thycotic.PowerShell.FolderPermissions.Permission"."
Please run the command using -Verbose
| Cannot convert value "" to type "Thycotic.PowerShell.FolderPermissions.Permission". Error: "Cannot
| convert the "" value of type "System.Management.Automation.PSCustomObject" to type
| "Thycotic.PowerShell.FolderPermissions.Permission"."
Provide a test case or steps to reproduce
search-tssfolderpermission
Expected behavior
returns valid object
What Edition of Secret Server?
Platinum
What version of Secret Server
Secret Server Cloud
What PowerShell host was used when producing this error
PowerShell Core (pwsh)
PowerShell Host Version
7

src/Thycotic.SecretServer/classes/folder-permissions/Permission.cs
add string value


Need Query Parser Support

Hi,
We have a use case where we want to know the tables/schemas dependencies of a Presto Query.
The package provides a way to query the presto db with an actual query which would actually execute the query, but instead I'm looking for query parser support. We have such support in java but not in typescript.

When implementing the SQL Client of the Presto UI, I tried several SQL parsers from the NPM repo, but they don't work well with Presto SQL. Presto SQL is compliant with ANSI SQL, however, it still has some unique formats in some entities, like a fully qualified table name is using <catalog>.<scheme>.<table name> format, and most parsers don't support it.
So my solution is to generate the SQL Parser from the ANTLR grammar file that Presto is using: https://github.com/prestodb/presto/blob/master/presto-parser/src/main/antlr4/com/facebook/presto/sql/parser/SqlBase.g4
You can check details here: https://github.com/prestodb/presto/tree/master/presto-main/src/main/resources/webapp/src/sql-parser and antlr4 supports TypeScript too. You just need to get familiar with the Lexer, Listener, and Parser concepts in ANTLR to use the parser it generates.
If people do need a js SQL parser for Presto SQL, it would be good to automate the process to:

generate the ANTLR Lexer, Listener, and Parser for TypeScript from Presto SQL grammar
publish the files as a presto-sql-parser (like @prestodb/presto-sql-parser) NPM module

Once there is any change in the grammar file, the new parser will be generated and published.

Thanks for pointing out a solution, I'll update the thread once I try it out


fix-query-params-builder

User data should not have a value for BPT but main function should.

I'd like to ask you if you feel comfortable with the logic of including / removing BPTs? Especially for building joins where you never join with BPT, it's confusing why it's passed as a maxAmountIn in the first place. IMO it shouldn't be, but rather filled in, in case it's needed. To make it easier to use I think the interface needs to be more intuitive. Params like amounts are better to be passed and returned as an object:
{ [token: string]: BigInt }
That way it should be straightforward to build joins and to understand what is returned from the query.

I'd like to ask you if you feel comfortable with the logic of including / removing BPTs? Especially for building joins where you never join with BPT, it's confusing why it's passed as a maxAmountIn in the first place. IMO it shouldn't be, but rather filled in, in case it's needed. To make it easier to use I think the interface needs to be more intuitive. Params like amounts are better to be passed and returned as an object:
{ [token: string]: BigInt }
That way it should be straightforward to build joins and to understand what is returned from the query.

Totally agree here. It didn't feel intuitive when I was working on it. In this case I was just focusing on an immediate fix for the issue as it just wasn't working (and interface already seemed to expect amount for BPT). I think it can definitely improve, I'll make a ticket in Asana so someone can pick it up.


Sharing CSS with app

I would like to use rails_admin for my app. Unfortunately I spent a lot of time designing custom CSS for the app and I just need to make it automatically available to the admin pages. I do not want to have to replicate... is this supported? ( i used to do it with gem administrate). thanks much for your help.

Pulling RailsAdmin's CSS in your app should be enough.
https://github.com/sferik/rails_admin/blob/0a5a45757e730b12de2b76b72e8b711ade6874ee/app/views/layouts/rails_admin/application.html.haml#L9


Update UX of source entry

Adding a new source flow. Once a URL has been added in the bulk upload or new line list entry screen and select + a new screen appears. Current flow is confusing in 2 areas:

The URL field goes into the name field, the 2 should be separate and not run into each other
The cursor should automatically move to name when the dialogue pops up so the name can be entered. Currently the cursor remains in the URL field when dialogue opens, this is confusing and results in additions being made to the url by mistake.

Attached screenshot

Update: Do not trigger a dialog for a new source. Add a new field under URL titled "Source name" that is required.

Latest UX:

Suggest we add the copy for the source reference name @shroothless how do you see this appearing the UI
Enter the name of the entity that has provided the data e.g. Brazil Ministry of Health


Store last read message for every user per channel

Sounds like we need a new table for that. Yay complexity. But it would be worth since logging in shows all channels as no unread messages which is wrong. The frontend could then ask the backend which channels and servers have unread messages.

BLOCKED BY https://github.com/discord-irc/discord-irc/issues/39


DOC: Add .zenodo.json for author association

Based on https://github.com/nipy/nipype/blob/8fe05afcd7c716b52044837c96749ef420154fbb/.zenodo.json
This mechanism explicitly associates authors with their
ORCID.
This commit seeds the configuration. Additional authors should
self-identify by adding their ORCID to this file.
h/t @satra
We may want to add a note to CONTRIBUTING.md after merged.

After initialization, I will post to Discourse requesting contributors add their ORCID.

@thewtex - in nipype we have some custom auto inclusion based on commits + management.
https://github.com/nipy/nipype/blob/master/tools/update_zenodo.py
just a note that no system of contributors for a large system is good enough for everyone !

@satra awesome! I'll try the script...

This is gold, thanks @thewtex.
A related question, does somebody know what is the criteria of Google Scholar to parse Zenodo? It seems that some DOI's are integrated, but not all of them.

Awesome @thewtex !
Adding a note to CONTRIBUTING.md looks fair. I think Nibabel has already some comment about it in their developper's guide, but the website seems to be down now.
To complete what @satra has said, a few related thoughts in an earlier issue:
https://github.com/InsightSoftwareConsortium/ITK/issues/456#issuecomment-476200207
If the script to parse authors based on the commits+management is finally found to fit our needs, we may want to add it to the Utilities/Maintenance folder, and add a section or note to the Release.md document.

A related question, does somebody know what is the criteria of Google Scholar to parse Zenodo? It seems that some DOI's are integrated, but not all of them.

No idea @phcerdan. Sorry.

Adding a note to CONTRIBUTING.md looks fair. I think Nibabel has already some comment about it in their developper's guide, but the website seems to be down now.

Ah, sorry, it was Nypipe that had a complete web page and details about how to cite the toolkit and inviting developers to open a PR to modify the .zenodo.json file with their data when contributing for the first time:
https://nipype.readthedocs.io/en/latest/about.html
They also cross-reference the GitHub contributors section.
At the end of the page, they also have a section dedicated to acknowledging funding parties: might be an inspiration for #454.

@thewtex FYI:  These errors are similar to the ones I was encountering on a previous version of the #1471 branch yesterday.

https://github.com/nipy/nipype/blob/master/tools/update_zenodo.py

I added a script based on this and a version in nibabel @jhlegarreta referenced. Running this script is documented in the release process notes so .zenodo.json is automatically updated.

Adding a note to CONTRIBUTING.md looks fair.

Done.

we may want to add it to the Utilities/Maintenance folder, and add a section or note to the Release.md document.

Done.

These errors are similar to the ones I was encountering on a previous version of the #1471 branch yesterday.

The source of this is not clear or consistent -- I will exclude them from the CI to avoid false positives.

FYI:  https://orcid.org/orcid-search/quick-search/?searchQuery=Luis Ibanez

ORCID
First
Last
_
Affiliations

https://orcid.org/0000-0003-2712-8584
Luis
Ibanez

Google Inc, Kitware Inc, University of North Carolina at Chapel Hill, Université de Rennes 1, Universidad Industrial de Santander

https://orcid.org/0000-0002-9546-9703
Dženan
Zukić

Kitware Inc, University of Siegen

https://orcid.org/0000-0002-6498-6273
Daniel
Blezek

Mayo Clinic, Mayo Clinic Graduate School for Biomedical Sciences

https://orcid.org/0000-0002-9020-3770
Julien
Jomier

Kitware SAS, Kitware Inc

https://orcid.org/0000-0002-7499-3514
Lydia
Ng

Allen Institute for Brain Science

https://orcid.org/0000-0002-7862-8856
Stephen
Aylward

https://orcid.org/0000-0002-4212-3362
Brian
Avants

https://orcid.org/0000-0001-9418-5103
NICHOLAS
TUSTISON

University of California Irvine, University of Virginia, Washington University in Saint Louis, Brigham Young University

https://orcid.org/0000-0002-9688-8950
Jean-Christophe
Fillion-Robin

Kitware Inc, École Polytechnique Fédérale de Lausanne, Ecole Supérieure Chimie Physique Electronique de Lyon

https://orcid.org/0000-0001-8639-5354
Vincent
Magnotta

https://orcid.org/0000-0001-8543-4016
Paul
Yushkevich

University of Pennsylvania

https://orcid.org/0000-0002-8858-1254
Ali
Ghayoor

Just an idea, but for credit in the zendo file, should we have a "long-tail truncation" where at least N (2, 3, 10, 50) commits are necessary in order to be listed?   I'm always concerned that a long tail listing dilutes the benefits of being listed.   My knee jerk reaction would be some number that includes approximately the top 50 developers.  Just an idea.

Just an idea, but for credit in the zendo file, should we have a "long-tail truncation" where at least N (2, 3, 10, 50) commits are necessary in order to be listed? I'm always concerned that a long tail listing dilutes the benefits of being listed. My knee jerk reaction would be some number that includes approximately the top 50 developers. Just an idea.

I am in two minds about this. I think giving credit to all contributors, regardless of the number of commits, makes the community a more welcoming place. GitHub lists all contributors. and the list can be ordered in different ways, always showing the top contributors by number of commits or additions/deletions. And there are other sites, if I am not mistaken, that list the top contributors according to different criteria.
Remote module developers would not be credited by the current script either. May be this is also controversial, since they would not actually appear as having made a commit to ITK itself. Not sure if when the remote makes into ITK proper, the remote's history also makes it into ITK. If not, I assume they would not get their information into zenodo.
Sorry Hans, more questions than answers.

Just an idea, but for credit in the zendo file, should we have a "long-tail truncation" where at least N (2, 3, 10, 50) commits are necessary in order to be listed? I'm always concerned that a long tail listing dilutes the benefits of being listed. My knee jerk reaction would be some number that includes approximately the top 50 developers. Just an idea.

I agree with Hans, long-tails dilutes the benefits indeed, NCommits>70(?), or NModules >= 1?

Just an idea, but for credit in the zendo file, should we have a "long-tail truncation" where at least N (2, 3, 10, 50) commits are necessary in order to be listed?

In d9a3ede a requirement for 10 commits is added. I think 50 is too high -- at 10, a contributor is often also contributing code reviews. This may also be helpful in reducing the burden for new contributors by not need to worry about adding an affiliation and an ORCID iD to .zenodo.json.

Remote module developers would not be credited by the current script either.

This is ok -- remote modules have their own repository, which can have a Zenodo record.

Merging -- @hjmjohnson if you create a separate PR for the ORCID's found, the contributors can review them.


There is no root package in cargo metadata

Which operating system does the issue occur on?
Linux blade-stealth 5.0.0-16-generic #17-Ubuntu SMP Wed May 15 10:52:21 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
If on linux, are you using X11 or Wayland?
X11
Hey guys .. im trying to install the master version on ubuntu using cargo deb  and I get the following output
alacritty master > cargo deb --install
cargo-deb: There is no root package in cargo metadata

Is this an issue of cargo deb or of alacritty?
rustc 1.36.0-beta.5 (07a9daea9 2019-06-07)
cargo 1.36.0-beta (c4fcfb725 2019-05-15)
cargo deb --version 1.18.0

Please see the INSTALL.md for instructions on how to install the Alacritty deb package.

Sorry my bad I didnt saw the install method was changes  ... Thanks you

I am getting the same error following steps from:
https://github.com/jwilm/alacritty/blob/master/INSTALL.md#debianubuntu-1

@rkj Are you sure you're in the correct directory when executing the command? Please note that these commands need to be executed in the Alacritty root directory and just cargo deb --install is not enough.

my bad, sorry for noise! I was executing from the parent directory. What confused me is that Cargo.toml exists in the root of the repo, I thought I should point to that file.
Thanks for working on this awesome terminal!


I Am Unable To Sign In On The Piskel Website.

Whenever I open Piskel, it does not give me an option to sign in. I am using the web browser version, and the sign in button used to show up, but as of a couple months ago it vanished and never reappeared.
I am using windows 11 on the newest version, and I have tried in multiple browsers, including Google, Google Chrome, Microsoft Edge, and Microsoft Bing.

Piskel removed account support in January.

Where are my piskel files with hours of work though?

Piskel removed account support in January.

In January 2022, a year ago.

Where are my piskel files with hours of work though?

They are gone.
There was a note on the accounts page for more than a year that explained the phased shutdown:
Archived website: https://web.archive.org/web/20220322102238/https://www.piskelapp.com/accounts

Shutting down accounts on Piskel
When will this happen?
This will happen in 2 steps:

August 1st 2021: new accounts can no longer be created
January 1st 2022: existing accounts and sprites will be removed. There will be no way to recover account or sprite data after that date

How can I backup my sprites?
The export page is still available until January 31st 2022.
To open your export page, you first need to sign in. You will find an "Export page" item in the User menu on the top right.
If you have an issue to download one of your sprites, send an email at support @ piskelapp.com
What was removed after January 1st 2022?
After January 1st 2022, we removed

the Sign in buttons
the Gallery pages
the Piskel pages

But the editor is still available. The offline applications can still be downloaded. The online editor can still be used directly in your browser. Only two features were removed from the online Piskel editor:

the "Save to your gallery" button in the SAVE panel
the "Upload" button in the EXPORT GIF panel

You can still draw pixel art, create animations, save them locally in your browser or download them as .piskel files or spritesheets.
Alternatives
Thankfully there are many other tools available for spriting and pixel art. Aseprite is a well known desktop software, and https://www.pixilart.com/ is another very powerful online sprite editor, with a lot more community-oriented features.
Also, the Piskel editor will still be available here, only accounts will be removed.
Finally, Piskel is fully open source, both the editor, and the website. Anyone who wants to duplicate the project and run another instance of Piskelapp could do it.
Thank you
Thank you to everyone who used Piskel, hope you will still enjoy using the editor even without the accounts.
And a big thank you to everyone who sent support messages since I announced the removal of accounts. I was not expecting such kindness, and I'm really thankful for it. Thank you so much.

Oof. I had no idea, I lost a bunch of work too.  They clearly waited half an year for all the users to transition. If you truly cared about your files, you should have backed them up before hand even if they are storing it in the cloud. And if you are angry to have your files deleted, you should have also checked up on it few years ago. There is no point in trash talking about something that has been done years ago.

There's also no point in replying to a year old comment.

There's also no point in replying to a year old comment.

I did not expect you to be active still. You are correct; but human tendencies you know. :)


Incorrect number of results on large graph and homomorphic matching

Hi,
Thanks for this project, it looks super interesting and influential!
I have been experimenting with the LDBC data set and some simple subgraph queries and ran into a potential bug.
I use graph instances based on the LDBC SF3 and SF10 data sets:

https://www.dropbox.com/s/i0rmhzc7etkmh8a/ldbc-3.graph.zst?dl=0
https://www.dropbox.com/s/ckm5k3cq5ur88gw/ldbc-10.graph.zst?dl=0

I changed two flags before compiling RapidMatch: https://github.com/szarnyasg/RapidMatch/commit/c3cab52c78f1d1f137deed870a9db23731c23ca3
#define RELATION_STRUCTURE 2
#define HOMOMORPHISM

My query graph is as follows:
t 4 4
v 0 0 2
v 1 0 2
v 2 5 2
v 3 6 2
e 0 1
e 1 2
e 2 3
e 3 0

The query is based on the LDBC SNB queries. Its pattern looks as follows:

I cross-validated the implementation of this query against two other (closed-source) systems, an RDBMS and a graph analytics engine.
On SF3, I consistently get 3340333 from all systems.
On SF10, the expected result of 11407928 (returned by the two other systems). RapidMatch returns different results on each run.
For example, I get these results:
Load graphs time (seconds): 76.772033
Query time (seconds): 62.493771
#Embeddings: 5346976
End.

Load graphs time (seconds): 79.109461
Query time (seconds): 62.303299
#Embeddings: 5851058
End.

Load graphs time (seconds): 79.178901
Query time (seconds): 62.080454
#Embeddings: 5730862
End.

The machine I'm running on has an Intel Xeon 8272CL CPU, 378GiB RAM, and runs Ubuntu 20.04 with GCC 9.3.0.

Hi,
Thanks for this project, it looks super interesting and influential!
I have been experimenting with the LDBC data set and some simple subgraph queries and ran into a potential bug.
I use graph instances based on the LDBC SF3 and SF10 data sets:

https://www.dropbox.com/s/i0rmhzc7etkmh8a/ldbc-3.graph.zst?dl=0
https://www.dropbox.com/s/ckm5k3cq5ur88gw/ldbc-10.graph.zst?dl=0

I changed two flags before compiling RapidMatch: szarnyasg@c3cab52
#define RELATION_STRUCTURE 2
#define HOMOMORPHISM

My query graph is as follows:
t 4 4
v 0 0 2
v 1 0 2
v 2 5 2
v 3 6 2
e 0 1
e 1 2
e 2 3
e 3 0

The query is based on the LDBC SNB queries. Its pattern looks as follows:

I cross-validated the implementation of this query against two other (closed-source) systems, an RDBMS and a graph analytics engine.
On SF3, I consistently get 3340333 from all systems.
On SF10, the expected result of 11407928 (returned by the two other systems). RapidMatch returns different results on each run.
For example, I get these results:
Load graphs time (seconds): 76.772033
Query time (seconds): 62.493771
#Embeddings: 5346976
End.

Load graphs time (seconds): 79.109461
Query time (seconds): 62.303299
#Embeddings: 5851058
End.

Load graphs time (seconds): 79.178901
Query time (seconds): 62.080454
#Embeddings: 5730862
End.

The machine I'm running on has an Intel Xeon 8272CL CPU, 378GiB RAM, and runs Ubuntu 20.04 with GCC 9.3.0.

Hi,
Thanks for your interest in our project.
To find homomorphisms, you need to disable the failing set pruning technique by commenting out "FAILING_SET_PRUNING" in config.h because this optimization is designed for enumerating isomorphisms. Moreover, you need to increase the time limit for the query because the program will terminate the query if it cannot be completed within the time limit. In my machine, the query that you provide takes around 70 seconds. So you can set the time limit to 120 seconds by "-time_limit 120".
The following is the log.
Load graphs time (seconds): 39.103812
Query time (seconds): 70.279089
#Embeddings: 11407928
End.

Hope this helps. I will update the README file to clarify the problem.
Finally, thanks for your question.

@shixuansun Thanks for the prompt reply! With the change in the configuration and by increasing the time limit, I could derive the correct answer.


Do not allow any filtering

LGTM (let's go to market) with this. I endorse these changes for CE.

@mhausenblas

@pyronicide

This needs some love still. @sargun
We need to list TCP, UDP and ICMP to make sure folks understand what "unfettered" actually means.


Integration Test - Release - 2019/02/04 17:00:25

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@bharney-msft : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


Conflicts with Vagrant nfs mounts

I work on projects that uses Vagrant with Virtual Box and Docker with Docker Machine on Mac OS X.
Every time I use Vagrant, and then I switch to my projects with Docker, the docker-machine-nfs gives me an error saying it's conflicting with the current /etc/exports.
The workaround till now is:

Clear the content in /etc/exports.
Run the docker-machine-nfs again.

The problem is that I'm having to do almost the same thing when have to go back to my other projects using Vagrant.
I took a look at the other issues talking about the /etc/exports usage by this command, though I didn't find any solution.
Help? :)

Hi Sebastian!
Could you please do the following:

Clear exports file
Run your Vagrant project with nfs mount, so it will create exports entry
Post contents of exports file here

I want to see vagrants configuration, but don't have it installed

Hey @sebas5384 ,
Do you know what the default ip vagrant is using? Because exports conflict can happen without vagrant too if you use multiple docker-machine boxes. The thing with docker-machine is that it doesn't lock the ip address to a box (see https://github.com/docker/machine/issues/1709). So it could be that when you start vagrant and later machine (and vice-versa) the ip changes constantly.
What I do to prevent this is following work around (https://github.com/docker/machine/issues/1709#issuecomment-161026484).
Try this and let us know.
That being all said, the management of the /etc/export in docker-machine-nfs is very basic, so it can be improved to avoid such things. But for for our (Adlogix/Adsdaq) needs it's not high priority.
Cheers

@krasilich and @tonivdv thanks your quick response!!
@krasilich here's my /etc/exports file when I have my vagrant's projects running.
$ cat /etc/exports
# VAGRANT-BEGIN: 501 6991d57f-1e23-42a1-a13a-d14d18659ec0
"/Users/sebas/Virtual/projects/myprojectwithvagrant" <IP_ADDRESS> -alldirs -mapall=501:20
# VAGRANT-END: 501 6991d57f-1e23-42a1-a13a-d14d18659ec0

@tonivdv I'll try to do that and I'll let you know.
Again thanks.

I tried to create a new machine with different CIDR, "<IP_ADDRESS>/24" instead of "<IP_ADDRESS>/24" like my vagrant's projects do:
$ docker-machine create -d virtualbox --virtualbox-hostonly-cidr "<IP_ADDRESS>/24" default98
$ docker-machine start default98
(default98) Starting VM...
Started machines may have new IP addresses. You may need to re-run the `docker-machine env` command.

$ eval "$(docker-machine env default98)"
$ docker-machine-nfs default98
[INFO] Configuration:

  - Machine Name: default98
  - Shared Folder: /Users
  - Force: false

[INFO] machine presence ...       OK
[INFO] machine running ...      OK
[INFO] Lookup mandatory properties ...    OK

  - Machine IP: <IP_ADDRESS>
  - Network ID: vboxnet8
  - NFSHost IP: <IP_ADDRESS>

[INFO] Configure NFS ...

 !!! Sudo will be necessary for editing /etc/exports !!!
Password:
exports:5: /Users conflicts with existing export /Users/sebas/Virtual/projects/myprojectwithvagrant
exports:6: /Users conflicts with existing export /Users/sebas/Virtual/projects/myprojectwithvagrant

but still same error, maybe I'm doing something wrong :/

Hey @sebas5384 ,
I could be wrong but it could be related to the fact that one machine tries to share /Users/sebas/Virtual/... and the other /Users only. I would have to test this to confirm.

@tonivdv yep! that was the problem :(
So now I just comment the Vagrant mount whenever I want to use the docker-machine with nfs.
I can't figure it out which project should deal with this, but I feel like Vagrant isn't removing the Vagrant mount when the virtual machine is down. That could resolve this issue, ... I think.
Thanks for the help!

It's Local that should be using a more specific path, their path overlaps everything.


Compiler tag

the check-build script does not create a modulefile specific to the compiler and version of mpi used. There is a different module for each version, but there should also be a different version for the compilers used.
E.g.

openmpi/1.8.8-gcc-4.8.4
openmpi/1.8.8-gcc-5.2.2

OR

openmpi-gcc5/1.8.8
openmpi-gcc4/1.8.8

etc.

I've edited, for each build,

openmpi/1.8.8-gcc-4.9.2
openmpi/1.8.8-gcc-5.2.0
modulefiles will be created and tested.

fixed in build 14


Parsing failure

I recently try to add sajson to nativejson-benchmark in this branch.
It was able to parse two test json data, but failed for twitter.json. The error message generated is
Verifying sajson (C++) ... Error (1:1): illegal unprintable codepoint in string

with this code:
class SajsonParseResult : public ParseResultBase {
public:
    SajsonParseResult(document&& d_) : d(std::move(d_)) {}
    document d;
};

// ...
    virtual ParseResultBase* Parse(const char* json, size_t length) const {
        (void)length;
        SajsonParseResult* pr = new SajsonParseResult(parse(literal(json)));
        if (!pr->d.is_valid()) {
            std::cout << "Error (" << pr->d.get_error_line() << ":" << pr->d.get_error_column() << "): " << pr->d.get_error_message() << std::endl;
        }
    	return pr;
    }

I am pretty sure the json is valid, as it is obtained from twitter's API, and also passed more than a dozens of parsers.
BTW, I am the author of RapidJSON :)

Hi!  Thanks for the bug report.  :)  Indeed, it was valid.  The latest sajson fixes the bug.
Thanks for RapidJSON, by the way.  We use it at IMVU, and while I have a few things I don't particularly like about the API, you set a new high bar for everyone.

I have verified the fix in the benchmark. Thank you.
If you have suggestions on RapidJSON, please feel free to drop an issue.

I just run with the both latest versions:
Benchmarking RapidJSON (C++)
          Parse canada.json          ...  9.763 ms  219.888 MB/s
          Parse citm_catalog.json    ...  5.638 ms  292.159 MB/s
          Parse twitter.json         ...  4.225 ms  142.546 MB/s

Benchmarking sajson (C++)
          Parse canada.json          ... 12.538 ms  171.221 MB/s
          Parse citm_catalog.json    ...  8.717 ms  188.963 MB/s
          Parse twitter.json         ...  2.661 ms  226.328 MB/s

It seems I need to figure out how to improve the performance for twitter.json. :smile:

Hah!  On my machine / benchmark, (an older version of) rapidjson outperformed sajson with twitter.json.  :)  I'd post numbers but I don't have access to that VM at the moment.
I've found they tend to go back and forth depending on the CPU and workload...  which probably means there's room to improve both parsers.  :)

One of my intention for making the benchmark is to learn from others and make improvements together.
I just learn about __builtin_expect from your code. Is it really beneficial?

It depends on the CPU (branch predictor and whatever), compiler, and data set, but sometimes it can be very beneficial!
In particular, in this particular context in a different project ( https://github.com/chadaustin/buffer-builder/blob/master/cbits/buffer.cpp#L82 ) it was a substantial win.
I suspect it won because the compiler could then optimize the register usage / spillage for the hot path.  You could clearly see in the generated assembly that clang was trying to make the hot path a single sequence of instructions, with only forward jumps for the cold paths.  (I believe, by default, forward jumps are predicted not taken, and backward jumps are predicted taken.)