ext_authz: Pass full downstream certificate to the external service

Title: ext_authz: Pass full downstream certificate to the external service
Description:
This is a feature request.
Today the ext_authz filter sends the principle of the downstream X.509 certificate in the Peer AttributeContext.
In our authorization system we require validation of additional parts of the X.509 certificate.
We propose to send the full downstream client X.509 certificate as a label in the source Peer AttributeContext.

I have uploaded #8327 to aid in explanation of the change that we propose.

Hi @sel-vcc , definitely this is an amazing feature! I have one quick question regarding to passing the peer certificate to the ext auth service. Not sure if you know this.
I found that only if the validation_context is configured in the yaml can we access the peer certificate in the ext auth service.
Is this expected?
BTW my yaml config file is as followed:
static_resources:
  listeners:
  - name: listener_0
    address: {socket_address: {address: ::, port_value: 443}}
    filter_chains:
    - filters:
      - name: envoy.filters.network.http_connection_manager
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
          stat_prefix: ingress_http
          route_config:
            virtual_hosts:
            - name: default
              domains: ["*"]
              routes:
              - match: {prefix: "/"}
                route:
                  cluster: some_service
     http_filters:
        name: envoy.filters.http.ext_authz
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.filters.http.ext_authz.v3.ExtAuthz
          grpc_service:
            envoy_grpc:
              cluster_name: ext-authz
          with_request_body:
            max_request_bytes: 1024
            allow_partial_message: true
            pack_as_bytes: true
      transport_socket:
        name: envoy.transport_sockets.tls
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.DownstreamTlsContext
          common_tls_context:
            tls_certificates:
            - certificate_chain: {filename: "certs/servercert1.pem"}
              private_key: {filename: "certs/serverkey1.pem"}
            validation_context: // enabled here!!!!!!!
              match_subject_alt_names:
  		contains: ""
  clusters:
  - name: some_service
    type: STATIC
    lb_policy: ROUND_ROBIN
    load_assignment:
      cluster_name: some_service
      endpoints:
      - lb_endpoints:
        - endpoint:
            address:
              socket_address:
                address: <IP_ADDRESS>
                port_value: 1234
    transport_socket:
      name: envoy.transport_sockets.tls
      typed_config:
        "@type": type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.UpstreamTlsContext
        common_tls_context:
          tls_certificates:
          - certificate_chain: {"filename": "certs/servercert2.pem"}
            private_key: {"filename": "certs/serverkey2.pem"}
  - name: ext-authz
     ...


@anandaroop => [ArtistSeries] Allow more series shelf to show up to 50 items

Force side of https://github.com/artsy/gravity/pull/13480
Pretty...simple! 😅
I mean, it's not a great UI as a substitution for a listings of landing page of all artist series (by an artist), but it'll do! (for now?)

Merging. The test:mocha job is failing, yet should be unrelated to these changes.


Error on websocket close

When a websocket connection is closed, I get the following error:
ERROR 2017-03-16 19:26:54,394 [sanic:72][MainThread] Traceback (most recent call last):
  File "/Users/dfee/code/spark/env/lib/python3.6/site-packages/sanic/app.py", line 470, in handle_request
    _response = await _response
  File "/Users/dfee/code/spark/env/lib/python3.6/site-packages/sanic_compress/__init__.py", line 27, in compress_response
    return (await self._compress_response(request, response))
  File "/Users/dfee/code/spark/env/lib/python3.6/site-packages/sanic_compress/__init__.py", line 31, in _compress_response
    content_length = len(response.body)
AttributeError: 'NoneType' object has no attribute 'body'

websocket is now supported in the HEAD of sanic, by the way!

Hrm I will take a look, but you might want to avoid gzipping your websocket routes regardless.


chore: 🤖 update react

:tada: This PR is included in version 1.4.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


how to predict 3D tracks using Hypernerf?

Hi, thanks for your great work.
In 3d tracking comparison, you show the result of Hypernerf. I wonder that how to get the tracking result since hypernerf builds a backward warping field, which computes the motion vector of any 3d point from arbitrary time to canonical time .It's difficult to get the motion vector from the canonical time to arbitrary time.

@wefvdge Hi there, I have the same question here, did you manage to solve this? I also did not find any code relates to tracking in theHyperNeRF repo.


[BUG] Eggs Publish Link 404s

Describe the bug
When you run eggs publish it generates the following output:
You can now find your package on our registry at<EMAIL_ADDRESS>
The link<EMAIL_ADDRESS>will 404.
To Reproduce
Run eggs publish.
Expected behavior
The link generated by the eggs publish command should resolve, AKA return 200.
Screenshots

Desktop (please complete the following information):

OS: Docker Ubuntu 18.04
nest.land Version 0.1.2

Smartphone (please complete the following information):

Desktop

@RobDWaller Similar issue at #133

Well should the link<EMAIL_ADDRESS>resolve?
Also I ran eggs upgrade but still get 0.1.2 when I run eggs -V, I'm assuming 0.1.2 is the latest version?

@RobDWaller I'm really sorry to hear that. This is a bug with eggs upgrade. The latest version of eggs is 0.1.5/0.1.6 (they are identical). Try installing it with this: deno install -A -f --unstable -n eggs<EMAIL_ADDRESS>This should fix your original publish link 404 issue as well.


Crash in DataFrame.reindex

Not sure if it's a bug or just me doing something wrong, but the code below crashes after about two seconds of 100% CPU utilization:
from pandas import DataFrame
df = DataFrame({'a': [1, 2, 3], 'b': [4, 5, 6]})
df2 = DataFrame({'c': [2, 2], 'd': [3, 3]})
gp2 = df2.groupby('c')['d']
df.reindex(gp2.index)

ERROR: An unexpected error occurred while tokenizing input
The following traceback may be corrupted or invalid
The error message is: ('EOF in multi-line statement', (2494, 0))
[...]
TypeError: unhashable type: 'numpy.ndarray'

Also, as a general thought or question, my intention for doing the above was this (and perhaps you can suggest a better way of getting there): I have a dataset with two columns used for identification of results (say a "place" and a "time"), one column with booleans for success/failure, plus some more columns. I'm trying to extract data for all "time" and "place" combinations that have at least one recorded successful result. My steps so far look like this:
groups = df[df['success] == True].groupby(["time", "place"])['success'].agg(len)

Now i would take the index from 'groups' and reindex the original data with it to get the result i want (this is where the above crash occurs, at least with the data in the example). But for that result, the last two steps (column selection and the '.add' call) are not necessary, i just want the index. Is the index already assembled after ".groupby()", so that it could be used without running the aggregation first? Or is there any other process that is preferable for this task?

The heavy CPU utilization I believe is caused by a Cython bug that seems to manifest in IPython. Referenced here:
https://github.com/ipython/ipython/issues/1317
I verified that the behavior you're seeing doesn't occur with the latest Cython (I built the source tarball using a development version of Cython before)
Will respond to your other comment a bit later.

There are a couple options. Probably your best is to replace ['success'].agg(len) with .size() which will return a Series with group sizes and the index you're after. If you look at the source code for size you can see what's going on

In [15]: gp2.size??
Type:       instancemethod
Base Class: <type 'instancemethod'>
String Form:<bound method SeriesGroupBy.size of <pandas.core.groupby.SeriesGroupBy object at 0x350f410>>
Namespace:  Interactive
File:       /home/wesm/code/pandas/pandas/core/groupby.py
Definition: gp2.size(self)
Source:
    def size(self):
        """
        Compute group sizes
        """
        result = sorted((k, len(v)) for k, v in self.groups.iteritems())
        keys, values = zip(*result)

        if len(self.groupings) > 1:
            names = [ping.name for ping in self.groupings]
            index = MultiIndex.from_tuples(keys, names=names)
        else:
            index = Index(keys, name=self.groupings[0].name)

        return Series(values, index=index)

Great, thank you very much!

I should also add, that bug will be gone in the official 0.7.0 release


Releasing pre-trained RetroMAE checkpoints

Hi!
Would it be possible to get the checkpoints of the pre-trained RetroMAE models? In particular, the model after this pre-training stage described here https://github.com/FlagOpen/FlagEmbedding/blob/master/FlagEmbedding/baai_general_embedding/README.md#1-pre-train , before the contrastive fine-tuning . I'm particularly interested in the pre-trained checkpoint (RetroMAE only) for BAAI/bge-base-en-v1.5 .
I've seen a few papers that indicate that if you want to fine-tune for your own task, it's generally better to start from this base model, as opposed to a model that's already been contrastively trained. If your experience differs from this, please let me know!

Hi, RetroMAE provides a good initialization for embedding model, and we fine-tune it on massive pairs to learn similarity between sentences. For the information retrieval task, I think that it's better to finetune the model which has learned knowledge from lots of text pairs.


How to Change Grid View Navigation Title

Hi, i need to change grid view's navigation title. I have tried:
self.title = "NewTitle"
self.navigationController.title = "NewTitle"

But these don't work. Any suggestions?

Anyone?

Maybe you should look over the updateNavigation method in MWPhotoBrowser.m.

In MWPhotoBrowser.m
search for method:

(void)updateNavigation
comment this
self.title = [NSString stringWithFormat:@"%lu %@", (unsigned long)numberOfPhotos, photosText];

and add
self.title = @"Photos";


(setup.py develop) is deprecated.

General

pyfar version: main branch
Python version: 3.12
Operating System: windoof

Description
(setup.py develop) is deprecated.
What I Did
pip install -e .
  DEPRECATION: Legacy editable install of pyfar==0.6.8 from file:///D:/git/_pyfar/pyfar (setup.py develop) is deprecated. pip 25.0 will enforce this behaviour change. A possible replacement is to add a pyproject.toml or enable --use-pep517, and use setuptools >= 64. If the resulting installation is not behaving as expected, try using --config-settings editable_mode=compat.
Please consult the setuptools documentation for more information. Discussion can be found at https://github.com/pypa/pip/issues/11457
  Running setup.py develop for pyfar

https://github.com/c4urself/bump2version?tab=readme-ov-file
bump2version is also deprecated


Use scala-cli for local compilation

Mmmh, turns out my changes need some more work (issues with coursier bootstrap class path leaking in the failing Windows job here). Just opened https://github.com/VirtusLab/scala-cli/pull/537 with only the first commit (rebased), that we can merge as soon as the CI is green.

Let's close this one for now not to confuse reviewers and we can re-review it once it is ready.


Add protection info to breakage reports

Task/Issue URL: https://app.asana.com/0/1202279501986195/1202345990054913/f
Description
Full context in this task
Incidentally added some unit tests that were missing before.
Steps to test this PR
Test report metadata

fresh install from this branch
launch app and enable AppTP
filter logcay by breakage_report
report breakage for any app
[ ] verify the breakageMetadata contains the field overridenDefaultProtection set to false under the vpnTrackerLists object
override protection for any app and then report breakage for it
[ ] verify the breakageMetadata contains the field overridenDefaultProtection set to true under the vpnTrackerLists object

Current dependencies on/for this PR:

develop

PR #1967   👈

This comment was auto-generated by Graphite.


Advise about the payment feature

Describe the feature you'd like
If reader want to read, for "book" or "chapter" or "page"  or part of the "content", the reader must be the vip of the website owner or must type the password or must pay some money.
The admin can set at the control panel about which he want to control. "for "book" or "chapter" or "page"  or part of the "content"" also the admin can use short code to set those
Describe the benefits this would bring to existing BookStack users
The BookStack users can use these earn money to write books.
Can the goal of this request already be achieved via other means?
the best way is add features and can be set  at the control panel
the easiest way is make a short code
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundamental request

How long have you been using BookStack?
Not using yet, just scoping
Additional context
the payment can be stripe, 2checkout or alipay

Thanks for the suggestion @zhanggaoxuan but I'd consider any kind of payment integration as out of scope for the core platform we're building/maintaining here.
It may be possible to hack something relevant in yourself using our methods of hacking/extension: https://www.bookstackapp.com/docs/admin/hacking-bookstack/
Otherwise I'm going to close this off as out of scope.


Add bulk correlation figure from plate 4 analysis

Preliminary figures visualizing correlation of bulk profiles in the plate 4 NF1 siRNA experiment.
This PR is pending several developments:
✅ @jenna-tomkinson: WayScience/nf1_cellpainting_data#21
✅ @MattsonCam refactor bulk data processing to use pycytominer (Jenna's contribution above)
✅ @MattsonCam add correlations to scramble for negative control
✅ @gwaybio rerun this notebook to incorporate the new profiles and new scramble correlations

Thanks for the review @MattsonCam - Sorry that I missed it and am returning to it so late.
Thanks specifically for the figure recommendations. Given that @jenna-tomkinson will be regenerating this figure to be publication ready, I will save time by ignoring these suggestions for now. Jenna, please take these into consideration when generating the official figure. Thanks!
Merging now!


Add new String storage class and replace read-only views.

_SwiftStringStorage<CodeUnit> is a new generic class with tail-allocated storage; it replaces _StringBuffer and NativeString. It includes methods for mutating the storage inline, but it isn’t designed for read-only access — use _UnmanagedString for that.
_UnmanagedString<CodeUnit> is a new generic struct providing type-safe read-only view into a contiguous string; it replaces UnsafeString.
_UnmanagedOpaqueString is a new struct replacing OpaqueCocoaString.
The transient structs FatalErrorString, SmallCocoaString and NonTaggedCocoaString are replaced by _StringGuts properties providing direct access to their fields.
Notes

This is still quite an enormous change -- I tried spinning off the UnsafeString -> _UnmanagedString<T> change into a separate PR, but debugging the in-between state would just have been too much work. If we have type-safe native storage, it seems pointless to keep raw pointers in unmanaged views, anyway.
This PR keeps the original _StringGuts layout -- I backed out caching the count for Cocoa strings.
The function withUnmanagedVariant and its associated enum from the previous version is gone. Instead, we have a three-way if ladder as the basic read-only access pattern:if _guts.isASCII {
  let ascii = _guts._unmanagedASCIIView
  ...
} else if _guts._isContiguous {
  let utf16 = _guts._unmanagedUTF16View
  ...
} else {
  let opaque = _guts.asOpaque()
  ...
}

For mutations, we have withMutable{ASCII,UTF16}Storage(unusedCapacity:,_:). It takes a closure and feeds it an unmanaged reference to a uniquely referenced native String storage instance of the specified encoding and with the given extra capacity._guts.withMutableUTF16Storage(unusedCapacity: 1) { storage in
  storage._value._appendInline("A")
}

The core reallocation logic is implemented by the function allocationParametersForMutableStorage(of:unusuedCapacity:). It is also used by replaceRange(_:with:) to eliminate an extra copy of the replaced range and the parts following it.
I had to comment out the _unmanagedContiguousImpl function for now, as I found it difficult to update for type-safe unmanaged views. To revive it, we probably need to split it into two stages -- one for detecting the code unit, and another to retrieve the corresponding UnmanagedString instance.

I’ll review in a little bit. Thanks for splitting out some of the unrelated changes!


Survey cover page

Survey Cover Page

Changes

Added new route at /cover-page with associated template
Styled various elements
Re-factored loading json into a helper function which can be exposed to a template (feel free to test this, it's not used yet though)

How to test

Do the usual npm install npm run compile dance.
Visit /cover-page
Do that thing where you resize the browser window to check if something is responsive

This now seems to be missing the cover-page route, possibly a rebase issue.

LGTM :+1:


chore: eslint setup

What kind of change does this PR introduce?
Chore: setup eslint to have a consistent coding style and avoid unnecessary bugs.

Coverage remained the same at 100.0% when pulling 309238ff790656a0a7794eb07aecb08e6381dbcc on eslint-setup into 9379a53bcbcbc160cec1de87f3206105db3928f5 on master.

Coverage remained the same at 100.0% when pulling 309238ff790656a0a7794eb07aecb08e6381dbcc on eslint-setup into 9379a53bcbcbc160cec1de87f3206105db3928f5 on master.

Hi @Meemaw!
In my solo projects I removed eslint to stay with Prettier only and I definitely have no regrets :)
BTW let's give pedantic eslint another try! I might adjust the configuration in future if it bothers too much. Just a few point, thought:

I tried to do a fairly un-opinionated setup and I think the changes do make sense (unused-imports, prefer-const, no-ts-ignore,...). Some rules can be annoying, but it's easy to turn them off.

does concurrently bring an actual benefit to the project?

It makes things much faster on my machine and in CI as well for some of the projects I work on.

may we update renovate configuration to update all eslint related stuff in a single go? (this the actual change request)

Yep, will do.

Not related, I saw that mac testing environments usually takes way longer to complete. Do you think we might consider dropping macos tests relying on the hope (😄 ) that a Node app that breaks on macos should also break on linux?

Well it takes around 2-3 minutes and thats nothing in my opinion (and its free 🙂 ). I'm used to CIs running for ~1 hour.

Hi @Meemaw!
In my solo projects I removed eslint to stay with Prettier only and I definitely have no regrets :)
BTW let's give pedantic eslint another try! I might adjust the configuration in future if it bothers too much. Just a few point, thought:

I tried to do a fairly un-opinionated setup and I think the changes do make sense (unused-imports, prefer-const, no-ts-ignore,...). Some rules can be annoying, but it's easy to turn them off.

does concurrently bring an actual benefit to the project?

It makes things much faster on my machine and in CI as well for some of the projects I work on.

may we update renovate configuration to update all eslint related stuff in a single go? (this the actual change request)

Yep, will do.

Not related, I saw that mac testing environments usually takes way longer to complete. Do you think we might consider dropping macos tests relying on the hope (😄 ) that a Node app that breaks on macos should also break on linux?

Well it takes around 2-3 minutes and thats nothing in my opinion (and its free 🙂 ). I'm used to CIs running for ~1 hour.

@toomuchdesign if you feel comfortable with this going forward I would merge it as soon as possible to avoid merge conflicts that will probably start to appear.

@toomuchdesign if you feel comfortable with this going forward I would merge it as soon as possible to avoid merge conflicts that will probably start to appear.


Fix recovery path for searchable snapshots

Replicas are bootstrapped using the recovery path, as opposed to the restore path used for creating the primary shard. This has been broken in the initial implementation of searchable snapshots. The fix here is to put in the appropriate checks to avoid failing during recovery.
Signed-off-by: Andrew Ross<EMAIL_ADDRESS>Issues Resolved
#4778
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #4813 (e096c0e) into main (e44158d) will increase coverage by 0.02%.
The diff coverage is 42.85%.

@@             Coverage Diff              @@
##               main    #4813      +/-   ##
============================================
+ Coverage     70.77%   70.80%   +0.02%
- Complexity    57785    57794       +9
============================================
  Files          4687     4687
  Lines        277293   277297       +4
  Branches      40376    40378       +2
============================================
+ Hits         196264   196335      +71
+ Misses        64733    64649      -84
- Partials      16296    16313      +17

Impacted Files
Coverage Δ

...ch/indices/recovery/PeerRecoveryTargetService.java
52.14% <25.00%> (-0.68%)
:arrow_down:

...rg/opensearch/indices/recovery/RecoveryTarget.java
73.88% <66.66%> (-0.31%)
:arrow_down:

...opensearch/index/mapper/MockFieldFilterPlugin.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../index/shard/IndexShardNotRecoveringException.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/opensearch/action/NoSuchNodeException.java
0.00% <0.00%> (-50.00%)
:arrow_down:

.../opensearch/client/indices/CloseIndexResponse.java
42.50% <0.00%> (-48.75%)
:arrow_down:

...opensearch/persistent/PersistentTasksExecutor.java
22.22% <0.00%> (-40.75%)
:arrow_down:

...r/src/main/java/org/opensearch/http/HttpUtils.java
16.66% <0.00%> (-33.34%)
:arrow_down:

...search/aggregations/pipeline/HoltWintersModel.java
21.47% <0.00%> (-32.22%)
:arrow_down:

...n/java/org/opensearch/test/rest/yaml/Features.java
60.00% <0.00%> (-20.00%)
:arrow_down:

... and 476 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Bukhtawar This change uses the boolean flag introduced in #4318 which complicates the backport to 2.x unless or until that work is backported. Any reason not to do the backporting for the remote translog work now?


Update Japanese language code for Gender

JP is the country code.
JA is the language code.
Introduce by https://github.com/ffaker/ffaker/pull/374

Just realize even the merge was 2 days ago, it has already been released. Might not be great to change it.

Sorry it is my bad. This is a somewhat complicated issue.
Like an address, apparently belonging to a country, uses a country code.
So if the country code came first, gender might have been added as a country code.
GenderKR is that case.
I thought that would be the case in GenderJP as well. but it was JA.
in this repository, suffix could be:

country code
language code
or both

I would like to solve this problem in the next major release, but I think it would be better to align it with JA now.

On the semver side, however, it is not advisable that the released module disappear, so it would be better to move the logic to GenderJA and modify GenderJP to use GenderJA.

close with 20602520c04ce67da2c3c0513b72f9f05b85ae14


Change the default FFT window

Currently the default FFT window is the Hamming window.
This default is set in the class SpectrogramSettings at line 29:
public string WindowFunction { get; set; } = WindowFunctions.HAMMING.ToString();
Recent experience indicates that this window is not as good as the Hanning window.
The default should be change d to Hanning. However, because the FFT window is used early in the signal processing sequence, this change may cause multiple tests to fail.

Can you show the effect of the different window function please? Add a couple of pictures into the response.
I believe you said resampling in combination with hamming causes side band effects?
We'll go ahead with this change but after .Net Core branch is merged in

Here is an example of the Hamming window with detection of Plains Wanderer calls.

Here is an example of the Hann

ing window with detection of Plains Wanderer calls.
The Hanning window offers a cleaner spectrogram and therefore better detection of Plains Wanderer whistles.

I have run both Hanning and Hamming windows on a frequently used test recording from Hendra Brisbane and can see no difference between the resulting spectrograms. So it would appear that the major difference arises only for recordings that have been resampled either up or down. Due to the significant difference in the resampled situation, I therefore propose that we switch to HANNING window as default. This may or may not result in a number of breaking unit tests depending on how many use a resampled signal.


How did you do it?

Hi,
I'm trying to run python scripts from C# scripts associated with a Unity project. But I am not able to. The system gives me the following errors -
"DllNotFoundException: Could not load Assets/Plugins/Python.Runtime.dll with flags RTLD_NOW | RTLD_GLOBAL: dlopen(Assets/Plugins/Python.Runtime.dll, 0x000A): tried: 'Assets/Plugins/Python.Runtime.dll' (not a mach-o file), ...."
Did you have the same problems? Could you tell me how you set up pythonnet on Unity? I would appreciate any help.
Best regards.

@nmital21
Please refer to the following scripts and folders for initialization and runtime of Python.Net.

https://github.com/shiena/Unity-PythonNet/blob/main/Assets/Scripts/PythonLifeCycle.cs
https://github.com/shiena/Unity-PythonNet/tree/main/Assets/StreamingAssets

Although it is in Japanese, the procedure for building the environment is written here.


WEAVE_DOCKER_ARGS cause "invalid reference format" error

Symptom:
When attempting to pass additional Docker run arguments to weave launch, got an error "invalid reference format".
Found in version : Weave version2.4.0
Steps to repro:
Just execute the code below to add a  custom Docker label "com.abc.com.container.ignore"
WEAVE_DEBUG= WEAVE_DOCKER_ARGS="-l com.abc.com.container.ignore"  weave launch
unable to create container: API error (500): invalid reference format
Note: When WEAVE_DOCKER_ARGS="", it works.

Hi, could you run WEAVE_DEBUG=true WEAVE_DOCKER_ARGS="-l com.abc.com.container.ignore" weave launch and post the full output please

WEAVE_DEBUG=true  WEAVE_DOCKER_ARGS="-l com.abc.com.container.ignore"  weave launch
/usr/local/bin/weave: 6: /usr/local/bin/weave: Bad substitution

WEAVE_DOCKER_ARGS="-l com.abc.com.container.ignore"  weave launch
unable to create container: API error (500): invalid reference format

Fixed by #3371.


Investigate "stateless" approach

First, I don't mean to pile on because I think the work you've done here is terrific Tom. Thank you for making such a valuable contribution to the developer community. And please let me  know if my next comments would be more appropriate elsewhere.
I have seen the same issue reported by Slamich and I independently arrived at the same solution (ie. "openssl.exe"). I am fine with leaving it at that.
"The fact is that there were no such problems with the previous ASMESharp module."
This is my real concern.
I am sure there are many others who migrated to "ACME-PS" from "ASMESharp" (who are also very grateful for your efforts Tom). With that migration in mind I have been wondering about something else.
One nice feature of "ASMESharp" was its "stateless" implementation (ie. it saved 100% of its own state to disk). That approach allowed its various commands to be run independently in multiple (ie. separate) Powershell sessions (one after another) without concern for losing state.
"ACME-PS" does not allow this. While "ACME-PS" saves some of its state to disk, the balance requires the use of temporary in-memory PS variables. That means users must complete the entire process in a single Powershell session.
Can you please explain that design choice Tom? Is there an easy way to arrive at a similar "stateless" approach (ie. similar to how "ASMESharp" manages 100% of its own state itself) ?
Originally posted by @GeorgeSchiro in https://github.com/PKISharp/ACMESharpCore-PowerShell/issues/45#issuecomment-558203023

I see the problem, that this is kind of an successor to ACMESharp, but the focus is a little bit different indeed. The ACMESharp module has not been stateless, but it hid away the state from you, by managing it in the background in a fully automatic fashion. IIRC it also was able to manage IIS-Sites automatically and such.
While such an approach hides some of the complexity, it'll make it difficult to reach into the process and get intermediate results. The approach here is more to provide a protocol handler, and let the specifics to the user / administrator / script implementer, e.g. I'll provide you with everything neccessary to fulfill the AuthZ, but you know your environment the best and just need some lines of PS to put the challange-files where they need to be or to modify your DNS.
About the scope of the module:
The hard part of ACME is to have something speaking the protocol and correctly answering the authorizations. The module will help with the protocol and provide the means to answer the authorization. It tries not to assume your environment and thus, it's made to be used in your scripts - not to replace them.
If there are possibilities to make the module more beginner friendly and/or usefull, I'm all ears,
Also I understand, that migrating from ACMESharp might be steep. Perhaps I - or we? - can provide guidance how to migrate from common scenarios.
Oh and btw:

It's Thomas ;)
I don't feel "piled", but the message from #44, which you also cited seemed borderline pushing / demanding.

Thanks for the quick response Thomas.
Naturally, everyone has his or her own style. My style includes a preference
for flexibility. Its seems from your response (eg. "let the specifics to the
user / administrator / script implementer") that flexibility would appear to
be your goal as well.
That said, I don't really understand why you would save some state to disk
but not all of it. That choice makes the tool less flexible. You could have
chosen the "all state to disk" approach (which is what I really meant and
why I used quotes: "stateless") and still allowed the movement of the same
data into and out of in-memory variables. If you had, the result would be
exactly the same as the current implementation (ie. current single-session
tool users would see no difference) PLUS the various register / order / auth
commands could all be run in a single session or each could be run
separately in multiple independent sessions, whichever the tool user
prefers.
IMHO, that's flexibility having little to do with hiding complexity (it's
not pandering to beginners).
-----Original Message-----
From: Thomas Glatzer<EMAIL_ADDRESS>Sent: Monday, November 25, 2019 11:35 AM
To: PKISharp/ACMESharpCore-PowerShell
Cc: George Schiro; Mention
Subject: Re: [PKISharp/ACMESharpCore-PowerShell] Investigate stateless
approach (#46)
I see the problem, that this is kind of an successor to ACMESharp, but the
focus is a little bit different indeed. The ACMESharp module has not been
stateless, but it hid away the state from you, by managing it in the
background in a fully automatic fashion. IIRC it also was able to manage
IIS-Sites automatically and such.
While such an approach hides some of the complexity, it'll make it difficult
to reach into the process and get intermediate results. The approach here is
more to provide a protocol handler, and let the specifics to the user /
administrator / script implementer, e.g. I'll provide you with everything
neccessary to fulfill the AuthZ, but you know your environment the best and
just need some lines of PS to put the challange-files where they need to be
or to modify your DNS.
About the scope of the module:
The hard part of ACME is to have something speaking the protocol and
correctly answering the authorizations. The module will help with the
protocol and provide the means to answer the authorization. It tries not to
assume your environment and thus, it's made to be used in your scripts - not
to replace them.
If there are possibilities to make the module more beginner friendly and/or
usefull, I'm all ears,
Also I understand, that migrating from ACMESharp might be steep. Perhaps I -
or we? - can provide guidance how to migrate from common scenarios.
Oh and btw:
a.. It's Thomas ;)
b.. I don't feel "piled", but the message from #44, which you also cited
seemed borderline pushing / demanding.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Essentially every command until certificate key generation and exporting the completed certificate will use the State-Object and will be able to make use of it.
That's probably something which could be added, so the CertKey will be stored along the order - probably removing a pain point for "multi-session"-users.
Also an automatic update of the order from HDD might be feasable - same for other objects like account-key (which is most likely not needed to be updated frequently).
The state itself could be made automatically available to running cmdlets by leveraging the $PSDefaultParameterValue-Hashset (at least as an opt-in).
On top of that, there could be a single-line-initialization for the state, using some default path as well as a default-path for the state.
What do you think are other actionable items?

I would avoid automatic updates, but I would also suggest some new syntax.
Old syntax:
$order | Update-ACMEOrder $state -PassThru;

New syntax (to be available in addition to the "old" syntax):
Update-ACMEOrder -Path "path to ACME working folder";

where "-Path" has the same meaning as it does for "New-ACMEState" and
"Get-ACMEState".
"single-line-initialization for the state, using some default path"
You already have state initialization with "New-ACMEState". In my
experience, a default path will only make it harder for newbies to find the
state stuff on the disk (everyone wants to browse through it). I wouldn't
use it.
The tool already works great as is (excepting multi-session support). All
that's really needed, IMHO, is making the passing of the various in-memory
objects optional. Here's the entire set of proposed new syntax:
New-ACMEState            -Path "path to ACME working folder";
Get-ACMEServiceDirectory -Path "path to ACME working folder" -ServiceName
"ACME service name"
New-ACMENonce            -Path "path to ACME working folder";
New-ACMEAccountKey       -Path "path to ACME working folder";
New-ACMEAccount          -Path "path to ACME working folder";
New-ACMEOrder            -Path "path to ACME working folder" -Identifiers
$identifiers;
Get-ACMEAuthorization    -Path "path to ACME working folder";
beg loop thru SAN names
Get-ACMEChallenge      -Path "path to ACME working folder" -Identifier
$identifier -Type "http-01";
Complete-ACMEChallenge -Path "path to ACME working folder" -Identifier
$identifier;
end loop thru SAN names
Update-ACMEOrder         -Path "path to ACME working folder";
New-ACMECertificateKey   -Path "path to ACME working folder";
Complete-ACMEOrder       -Path "path to ACME working folder";
Update-ACMEOrder         -Path "path to ACME working folder";
Export-ACMECertificate   -Path "path to ACME working folder" -ExportPath
"{CertificatePathFile}";
Note: "* loop thru SAN names" is not part of the proposed new syntax, it

just shows where SAN activity happens.
I suggest that the only new "automatic" behavior should perhaps be the
calling of "New-ACMECertificateKey" from within "Complete-ACMEOrder" (if the
key doesn't already exist in the state object).
FYI, I think there was a question some time ago about SAN support. I've
confirmed that the code already supports a SAN list - and it works
perfectly.
-----Original Message-----
From: Thomas Glatzer<EMAIL_ADDRESS>Sent: Monday, November 25, 2019 2:31 PM
To: PKISharp/ACMESharpCore-PowerShell
Cc: George Schiro; Mention
Subject: Re: [PKISharp/ACMESharpCore-PowerShell] Investigate "stateless"
approach (#46)
Essentially every command until certificate key generation and exporting the
completed certificate will use the State-Object and will be able to make use
of it.
That's probably something which could be added, so the CertKey will be
stored along the order - probably removing a pain point for
"multi-session"-users.
Also an automatic update of the order from HDD might be feasable - same for
other objects like account-key (which is most likely not needed to be
updated frequently).
The state itself could be made automatically available to running cmdlets by
leveraging the $PSDefaultParameterValue-Hashset (at least as an opt-in).
On top of that, there could be a single-line-initialization for the state,
using some default path as well as a default-path for the state.
What do you think are other actionable items?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

So - to allow this the following has to happen:
[] Include CertificateKey in State or Order, since it currently is not saved alongside those objects
[] Allow Complete-Order to call New-CertificateKey, if neccessary (probably opt-in)
[] Change implementation of State to load certain objects ad-hoc (e.g. next-nonce) instead of one-time
[] Add -StatePath Parameter to all cmdlets supporting -State
I don't see any donwsides to this approach currently.
Changing the state-object needs caution, due to probability of breaking changes, but else ...

I know that's a lot to think about (let alone do). To obviate the need for always having to use a single session, an alternate syntax for every command would be the ideal. But perhaps there is an intermediate step that would serve the same purpose.
Consider defining 4 new commands to write state to disk:
Set-ACMEAuthorization $authZ     -Path "path to ACME working folder";
Set-ACMEChallenge     $challenge -Path "path to ACME working folder";
Set-ACMEOrder         $order     -Path "path to ACME working folder";
Set-ACMEState         $state     -Path "path to ACME working folder";

Plus 4 corresponding commands to read state back from disk:
$authZ     = Get-ACMEAuthorization -Path "path to ACME working folder";
$challenge = Get-ACMEChallenge     -Path "path to ACME working folder";
$order     = Get-ACMEOrder         -Path "path to ACME working folder";
$state     = Get-ACMEState         -Path "path to ACME working folder";

Does that make more sense Thomas? Would that cover all multi-session scenarios?

Hi George,
I'd always use -StatePath, to make the purpose clear - but that's only a side-note.
PowerShell allows to define ParameterSets, which allows to disambiguate between different modes of CmdLets, thus I'll probably define "overloads" of the given commands which accept will accept either State or StatePath. Besides that there needs to be an Update-ACMEWhatever to allow for reading it from the LE-Servers again.
So essentially, the Get-ACME Commands will be extended, to allow those scenarios and Update-ACME will be provided, where Re-Reading is necessary (as already done with the Order).
To make all that "multi-session" enabled, the State-Object itself needs to read everything on demand instead of reading on creation and thus will need some changes (especially regarding the order objects). This is especially true for concurrent multi-sessions. But I don't like to have surprises, if users (for whatever reason) switch between multiple sessions more or less simultaneously.
So the first thing I'll be doing is making the State not doing everything in Memory, but directly interacting with the underlying storage - especially for loading.

I started workin on it - but it'll probably take some time.
I began with splitting the AcmeState Object into multiple ones, all implementing the same "interface" making the code simpler for in-memory state vs persisted state. The latter one will always load from disk and not store anything in memory.
If that's finished, I'll look into storing the certificate-key alongside the order.
When that's done, it'll be fairly simple to enhance all cmdlets to allow getting a statepath instead of a state.

Please let me know if there is anything I can do to help test or anything
else.
Thank you for taking this request seriously Thomas. Your efforts are greatly
appreciated.
-----Original Message-----
From: Thomas Glatzer<EMAIL_ADDRESS>Sent: Monday, December 02, 2019 5:49 AM
To: PKISharp/ACMESharpCore-PowerShell
Cc: George Schiro; Mention
Subject: Re: [PKISharp/ACMESharpCore-PowerShell] Investigate "stateless"
approach (#46)
I started workin on it - but it'll probably take some time.
I began with splitting the AcmeState Object into multiple ones, all
implementing the same "interface" making the code simpler for in-memory
state vs persisted state. The latter one will always load from disk and not
store anything in memory.
If that's finished, I'll look into storing the certificate-key alongside the
order.
When that's done, it'll be fairly simple to enhance all cmdlets to allow
getting a statepath instead of a state.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Happy New Year Thomas!
I've been wondering if there has been anything new on this topic. I noticed a new branch named "deep-state" (cute).
Any bearing?

Essentially its done in the aforementioned branch - except one thing:
I'd like to use the -State Parameter, that exists everywhere and define an automatic conversion FROM String. Currently I could not get that running and I have no clue what the problem is.
I have yet to reach out to StackOverflow and/or the PowerShell-Gurus, I know to get that figured out.
Or I have to make the Parameter "dynamic" and accept string as well as AcmeState, which would then allow to convert manually - but I'd like not to.
Happy new year.

Fantastic!
Is "deep-state" at a point I can test? Or do you need to resolve the "-State Parameter" issue first?

The last state I commited, should be usable, if you use Get-AcmeState as before.
I already issued some test certificates with that branch, so if anything goes wrong, it'll probably be a minor problem.

Ah sorry - the automatic certificate key management is missing - I forgot about that.
Nevertheless - the "heavy lifiting" (still except the TypeConversion) is done - the rest should be easily doable.

Ok, I will check in later. Thanks Thomas!


Install app name is wrong

Describe the bug
By the readme Install app name is wrong
To Reproduce
Steps to reproduce the behavior:
npx create-nodejs-ts --no --app=bot
it will create a folder named my-app
Expected behavior
a folder created named bot
Screenshots
Environment (please complete the following information):

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

Additional context
i checked the source, it should use --name  instead of --app arg

Thank you @zhaoxiaohai!


Support iOS 18 and Swift 6

Support iOS 18 and Swift 6
Fix bug:  #16

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 86.04651% with 6 lines in your changes missing coverage. Please review.

Project coverage is 95.99%. Comparing base (963dedd) to head (38870c0).

Files with missing lines
Patch %
Lines

Sources/sRouting/Models/TimeIdentifier.swift
60.00%
4 Missing :warning:

Sources/sRouting/Viewing/NavigatorView.swift
83.33%
1 Missing :warning:

Sources/sRouting/Viewing/SRNavigationStack.swift
80.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
- Coverage   96.38%   95.99%   -0.40%
==========================================
  Files          38       38
  Lines        1772     1746      -26
==========================================
- Hits         1708     1676      -32
- Misses         64       70       +6

Files with missing lines
Coverage Δ

Sources/sRouting/Models/AnyRoute.swift
100.00% <100.00%> (ø)

Sources/sRouting/Models/SRDismissAllEmitter.swift
100.00% <ø> (ø)

Sources/sRouting/Models/SRNavigationPath.swift
100.00% <ø> (ø)

Sources/sRouting/Models/SRTabarSelection.swift
100.00% <100.00%> (ø)

Sources/sRouting/Models/SRTransition.swift
100.00% <100.00%> (ø)

Sources/sRouting/Models/SRWindowTransition.swift
100.00% <100.00%> (ø)

Sources/sRouting/Prototype/SRRoute.swift
63.63% <ø> (ø)

Sources/sRouting/Viewing/SRRootView.swift
100.00% <ø> (ø)

Sources/sRoutingMacros/ContextMacro.swift
99.49% <100.00%> (+<0.01%)
:arrow_up:

Sources/sRoutingMacros/RouterMacro.swift
100.00% <100.00%> (ø)

... and 5 more


Integrate visualfsharp/OOB --> master

Integrate the big file renaming into "master" of fsharp/fsharp

The Travis Linux build is failing building
/home/travis/build/fsharp/fsharp/tests/projects/Sample_VS2012_FSharp_Portable_Library/Sample_VS2012_FSharp_Portable_Library.fsproj

Will investigate


CLOUD-48 restart services before decommission

@keyki pls review
We need to restart the services before downscale

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1527/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1531/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1533/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1535/


Drop legacy genconfig2 cli tool

The tool is deprecated, it also had a deprecation notice for a long
time.

I created this PR for visibility. I would like to merge when a major release is required and not now


-Fixed issue with Spanish -> English transition

English->Spanish and Spanish->English transition button. (Code updated) ✔ Working
All links.  ✔Working
joinStudyLink (at line 306). ❌Not Working
Conclusion:

Everything is working except the join study link (nrelopenpath://join_study?label=uprm-civic&source=github). This is something that we never touched nor modified. We believe that the problem is not the code, but its possible that the URL is written incorrectly.
This is the link:

@dcolondavila that link will work if you have the app installed on your phone and open it from the phone.
See the instructions above:

After you have installed the NREL OpenPATH app on your phone, join the study by scanning the QR code or visiting the link below using your phone.

It is designed to handle the use case that people open the join page on their phone. Then they cannot scan the QR code, so they will click on the link.
You should test that use case too, but you can wait until we deploy to production since you have not changed that code.
Have you installed the app on your personal phones? I still don't see any data in the public dashboard...

I have not installed it yet. I will do it right now and will also re-test the link

So I installed the app, but the QR codes that you provided us are not working. Non of them are working. Could you send us a new QR code for the UPRM CIVIC study?

This are the QR codes that we tried and non of them are working: nrelop_uprm_civic_test.zip

Also, how did the new merge go? Is everything working perfectly as intended?

I have deployed the new merge to staging, but will have to sign off now. Will only be able to push to production later tonight.
What do you mean by "but the QR codes that you provided us are not working"? What are you trying to do with them?

When I try to scan the QR code through the app so that I can access the study, nothing happens. I tried all of them and non of them work.

I tested the QR code that's located in the join page and that one works, but that one is not for the UPRM-CIVIC study.

there is a UPRM civic join page that I have sent out multiple links to
please read the instructions on it carefully

I got it working.

This is the new error that im getting right now.

which token did you use?

This one:

@dcolondavila please read the instructions and don't randomly scan QR codes

Ok so I got it working but this is what I did to get it to work. First, I installed the openPATH app. Then, I scanned the UPRM-CIVIC opcode that's on the join page through my phone's app camera, not through the openPATH application. And now everything works just fine. Don't know why it was giving me the user registration error.


AttributeError: 'str' object has no attribute 'decode' when trying to run simulation server with GPU

Hey guys, great job on the repo so far. Here's a simple error I encountered when trying to start the servers with a GPU:
deployments-webots-1  | Traceback (most recent call last):
deployments-webots-1  |   File "/usr/local/lib/python3.8/dist-packages/tornado/web.py", line 1711, in _execute
deployments-webots-1  |     result = method(*self.path_args, **self.path_kwargs)
deployments-webots-1  |   File "simulation_server.py", line 633, in get
deployments-webots-1  |     gpu = nvmlDeviceGetName(nvmlHandle).decode('utf-8')
deployments-webots-1  | AttributeError: 'str' object has no attribute 'decode'

The fix is simple -- I just removed the call to decode, since nvmlDeviceGetName returns a str already.
Not sure if this is reproducible on other platforms: I'm running this in a Docker container on Windows 10 (x86_64) with GPU passthrough enabled via Docker Desktop on the WSL 2 engine.
If you want to make line simulation_server.py:633 more resilient, I'd suggest adding a check for a byte or str before attempting to decode.
Thanks!

This problem was probably caused by a very recent change (5 days ago) in the pynvml module:
https://github.com/gpuopenanalytics/pynvml/pull/46/files#diff-c9c6160b3107b52ed919117b3cba3bd81f8539baa056cf1e8693329a26ba5bc7R2009

Thank you!
I fixed it in #23


docs(vue-query): vue2, nuxt3 examples

Codecov Report
Base: 96.36% // Head: 92.27% // Decreases project coverage by -4.08% :warning:

Coverage data is based on head (6d25b29) compared to base (eab6e2c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4384      +/-   ##
==========================================
- Coverage   96.36%   92.27%   -4.09%
==========================================
  Files          45       87      +42
  Lines        2281     3562    +1281
  Branches      640      902     +262
==========================================
+ Hits         2198     3287    +1089
- Misses         80      259     +179
- Partials        3       16      +13

Impacted Files
Coverage Δ

src/core/queryObserver.ts

src/core/onlineManager.ts

src/react/setBatchUpdatesFn.ts

src/react/tests/utils.tsx

src/devtools/theme.tsx

src/devtools/Explorer.tsx

src/react/Hydrate.tsx

src/react/useQuery.ts

src/core/infiniteQueryBehavior.ts

src/react/QueryClientProvider.tsx

... and 122 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[FEAT] Allow sorting on a related table field using supabase dataprovider

Is your feature request related to a problem? Please describe.
Right now a table can only be sorted on its own fields, it would be useful to allow sorting on a related table field.
Describe the solution you'd like
Using the metaData arguments we could add to the getList method something like this right before line 118
metaData?.sort?.map((item) => {
    query.order(item.field, { foreignTable: item.foreignTable, ascending: item.order === "asc" });
});

This would allow the user to pass an object like this to useTable to obtain the table correctly sorted
{
    metaData: {
        select: '*, categories(*)'
        sort: [{
            field: 'title',
            foreignTable: 'categories',
            order: 'asc'
        }]
    }
}

Describe alternatives you've considered
An alternative solutions could be to allow the use of a field foreignTable inside initialSorter, permanentSorter ecc.. and update lines 118-120 of the getList method to this
sort?.map((item) => {
    query.order(item.field, { ascending: item.order === "asc", foreignTable: item.foreignTable });
});

Additional context
In both of these solutions the user would still be required to pass a custom select operation to metaData, as illustrated above,  since the related table must be present inside the select clause for supabase to not give an error

Hey @icomad ,
so how about use like this,
setSorter([
     {
         field: "categories.title",
         order: "desc",
     },
]);

On the data provider side, the sort field can be split according to . and the first part can be evaluated as foreignTable. Thus, it also works with the Tables of frameworks such as Ant Design and Material UI.

It seems to be a great solution!

Hey @icomad ,
We added this feature but it doesn't work on the Supabase side. We'll create a PR next week, but you'll have to wait for it to get fixed on Supabase to start using it.

https://github.com/supabase/postgrest-js/issues/198
https://github.com/supabase/supabase/discussions/4255

Hey @icomad ,
We've released<EMAIL_ADDRESS>🎁


fix(stdlib): don't force newline after printf

C's printf does not add newlines, but c.php's standard printf function does. Change c.php to match the C standard behavior.

I noticed this patch doesn't have any tests.  Please make sure to add tests for this patch

@strager looks good to me! :+1: Thank you for the contribution!


Dashing

main branch is base on foxy version.
Is there any reason to merge a dashing branch into main?


Support Hang Up when Withdraw or accidental "non-repeated" quest

Currently, when running with "Withdraw" option turned on, if a Withdraw happens, the app will get stuck on selecting a support. This is caused by the App only following a loop expecting the "repeat" option to be pressed. When the repeat option is pressed, a click on the support you want is all you need to progress to load the stage, but when you choose from the main menu, after selecting a support, a click on the "start quest" button is required. When you Withdraw, the app still follows the process that is used for "repeat" and gets hung up without pressing "start quest" button. This problem also arises from manually clicking to "not repeat" the quest, but still having the app run. When it gets to the main menu, it will click on the node as usual, but only click on the support and not click start quest. The suggestion for fixing this is to create a loop back to main menu clicking if "repeat" is never clicked. You can do this a number of ways, but the simplest would be using a boolean variable (true/false) that is checked in Support window and is set to true when "repeat" is clicked and reset to false when clicking the node in main menu.
Don't really think the device matters for this particular problem, but here's some of the info.

Device model: Samsung S9
Android Version: Android 10

Good catch. I was 3T farming all this time so didn't notice 😅

https://github.com/MathewSachin/Fate-Grand-Automata/blob/fa872020697d990b38f7e57aeaee02b9b1abd684/scripts/src/main/java/com/mathewsachin/fategrandautomata/scripts/entrypoints/AutoBattle.kt#L153
After looking through the code, I saw you used the "isContinuing" boolean I made for the repeat option when it first came out, you can fix this right after this line by setting it to "false" here

Did just what you said for the fix

Cool, not sure how releases work on GPlay store, but I assume this isn't live on GPlay store yet?

It's not live there yet, but you can get a test APK from GitHub Actions.
https://discordapp.com/api/webhooks/755765515770789941/xeUUK3Q8ulYoQ9ZxgDRDe-CluH0quKRJnCRUw8mP-uekmf1LGHHxKv_eHfWz8Vrjs-aH
It's still building right now

I see, thanks much.

Oh sorry wrong link. 😓


incorrect logging of messages in app.logger.get_messages

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
pyaedt_icepak.logger.get_messages works in non-graphical mode if aedt_messages = True
e.g. =
message_list = self._pyaedt_icepak.logger.get_messages(self._pyaedt_icepak.project_name, self._pyaedt_icepak.design_name, severity, aedt_messages=True)

The above command shows different behavior in graphical and in nongraphical mode
In graphical mode, design messages are logged to the --> message_list.design_level
But in the non-graphical mode, design messages are logged to the --> message_list.global_level

I think this is not the desired behavior. Please comment if I am wrong.
Thanks,
Nitin
Steps To Reproduce
run any dummy model in graphical and non-graphical. Try to get messages.
Which Operating System are you using?
Windows
Which Python version are you using?
3.10
Installed packages
anyio==3.6.2
argon2-cffi==21.3.0
argon2-cffi-bindings==21.2.0
astroid==2.11.7
asttokens==2.1.0
attrs==22.1.0
backcall==0.2.0
beautifulsoup4==4.11.1
bleach==5.0.1
certifi==2022.12.7
cffi==1.15.1
charset-normalizer==3.1.0
click==8.1.3
click-plugins==1.1.1
cligj==0.7.2
clr-loader==0.2.5
colorama==0.4.5
contourpy==1.0.7
cycler==0.11.0
debugpy==1.6.3
decorator==5.1.1
defusedxml==0.7.1
dill==<IP_ADDRESS>
entrypoints==0.4
et-xmlfile==1.1.0
executing==1.2.0
fastjsonschema==2.16.2
Fiona==1.9.3
fonttools==4.39.3
geopandas==0.12.2
idna==3.4
imageio==2.28.0
ipykernel==6.17.0
ipython==8.6.0
ipython-genutils==0.2.0
ipywidgets==8.0.2
isort==5.10.1
jedi==0.18.1
Jinja2==3.1.2
jsonschema==4.16.0
jupyter==1.0.0
jupyter-console==6.4.4
jupyter-server==1.21.0
jupyter_client==7.4.4
jupyter_core==4.11.2
jupyterlab-pygments==0.2.2
jupyterlab-widgets==3.0.3
kiwisolver==1.4.4
lazy-object-proxy==1.7.1
MarkupSafe==2.1.1
matplotlib==3.7.1
matplotlib-inline==0.1.6
mccabe==0.7.0
mistune==2.0.4
munch==2.5.0
nbclassic==0.4.7
nbclient==0.7.0
nbconvert==7.2.3
nbformat==5.7.0
nest-asyncio==1.5.6
networkx==3.1
notebook==6.5.2
notebook_shim==0.2.0
numpy==1.24.1
openpyxl==3.0.10
osmnx==1.3.0
packaging==21.3
pandas==1.5.3
pandocfilters==1.5.0
parso==0.8.3
pickleshare==0.7.5
Pillow==9.5.0
platformdirs==2.5.2
plumbum==1.8.1
pooch==1.7.0
prometheus-client==0.15.0
prompt-toolkit==3.0.31
psutil==5.9.3
pure-eval==0.2.2
pyaedt==0.6.87
pycparser==2.21
Pygments==2.13.0
pylint==2.14.5
pyparsing==3.0.9
pyproj==3.5.0
pyrsistent==0.19.1
python-dateutil==2.8.2
pythonnet==3.0.1
pytz==2023.3
pyvista==0.37.0
pywin32==304
pywinpty==2.0.9
pyzmq==24.0.1
qtconsole==5.3.2
QtPy==2.2.1
requests==2.29.0
rpyc==5.3.1
scikit-rf==0.26.0
scipy==1.10.1
scooby==0.7.1
Send2Trash==1.8.0
shapely==2.0.1
six==1.16.0
sniffio==1.3.0
soupsieve==2.3.2.post1
SRTM.py==0.3.7
stack-data==0.6.0
terminado==0.17.0
tinycss2==1.2.1
tomli==2.0.1
tomlkit==0.11.4
tornado==6.2
traitlets==5.5.0
urllib3==1.26.15
utm==0.7.0
vtk==9.2.6
wcwidth==0.2.5
webencodings==0.5.1
websocket-client==1.4.1
widgetsnbextension==4.0.3
wrapt==1.14.1

Hi @nnetake ,
Could you please provide a script to reproduce this error? otherwise we are not able to reproduce it.
Also could you please change the title?
Thanks

Hi @nnetake, thanks for your feedback.
I've tried to reproduce the bug you mentioned using your requirements, with EDT 2023.R2, and the following script:
from pyaedt import Icepak, generate_unique_folder_name, downloads, settings
import os

# Download needed files
temp_folder = generate_unique_folder_name()
package_temp_name, qfp_temp_name = downloads.download_icepak_3d_component(temp_folder)

ipk = Icepak(projectname=os.path.join(temp_folder, "Heatsink.aedt"), specified_version="2023.2", non_graphical=False,
             close_on_exit=True, new_desktop_session=True)

msg = ipk.logger.get_messages(ipk.project_name, ipk.design_name, aedt_messages=True)

assert not msg.design_level
assert msg.global_level

However, I had no values in design_level while global_level had content. Could you please provide me a specific example that could help me reproduce the behavior you mentionned ?
Thanks,
Sébastien

Hi @SMoraisAnsys, sorry for the late response.
Yes to have some content on the design level you need to add messages at the design level.
You can do by following:
ipk.odesktop.AddMessage(
            ipk.project_name, ipk.design_name, 0, "This is a test message"
        )

Once you add this message try to retrieve it in graphical and non-graphical mode. Please let me know if you have any further questions.

@nnetake Thanks for your answer :)
I'll look at it as soon as possible !

Hi @nnetake!
Thanks for your reply! :)
However in the snippet you posted you are using the native API.
Have you tried using add_message in the aedt_logger.py?
Giulia

@gmalinve add_messages does not work in non-graphical mode, that's why I have shared native API calls.
Here our concern is about get_messages. If we have messages at the design level, get_messages log those messages at a global level in a non-graphical model however it works correctly in graphical mode.

@nnetake which version of AEDT are you using?

@gmalinve I am using 22R2.

@nnetake have you tried with at least 2023.2 to see if you have the same issue? because we are aware of some logging problems with 2022.2.

@gmalinve I tested this with 23R2, it is able to log messages to design levels in nongraphical mode, however, I don't see any global messages. I think pyaedt does not log any global messages in non-graphical mode right?

Hi @nnetake. I just created a new branch to fix the logging of global messages in non graph though it won't apply to version 2022.2 because this was a bug in the native api but if you test with versions > 2023.1 it should work as expected.


Re-arrange nodes position to look better view

Hi there,
is there any function to re-arrange the nodes x,y position after user drag/drop in the canvas.
So that there will be no overlapping nodes each other
second question, how to automatically adjust the zoom in /out of the nodes in the canvas so that it will fit into it
Note: just started exploring the angular2 with Drawflow...awesome
Thanks.

View:

#685
#721
#383

Form move canvas to node:

#88

Thanks jerosoler ! We will check on this and get back to you.
Another quick question - is there any css to change the node shape - circle, hexagon or any others.
And i dnt want all are same shapes, onlye certain nodes is required in that shapes.

View this comment

https://github.com/jerosoler/Drawflow/issues/454#issuecomment-1154762914

thanks for the reply. im trying with the circle for nodes.

found that it is break the css. also lagging. i want similar to the other node. like border circle and inside circle div different color.
Also specific nodes required the connector different color.
example 5 main nodes connector is orange. 3 subnode connector with purple color.
So that it will be easier to differentiate which is main and subnode.

Example with circle node:
<!DOCTYPE html>

  <meta charset="UTF-8" />
  <meta name="viewport" content="width=device-width, initial-scale=1.0" />
  <title>Document</title>
  <link rel="stylesheet" href="https://cdn.jsdelivr.net/gh/jerosoler/Drawflow/dist/drawflow.min.css" />

Thank you so much ! great app and great support!

hi jerosoler,
i have implemented theme changes accordingly for user taste. the themes can be changed and the nodes colors are changing. however the node connector seems not joined properly whenever change the themes.
i need to drag or move the node upper a bit or down to make it connect correctly.
Please let me know how to handle this situation.
after theme changed

after moved a bit the nodes, the connector sit in correcti position.

anyone?

Force update conections with:
editor.updateConnectionNodes('node-4');

yes above is what i have done initially. but no different.
`theme(styleName: any): void {
const head = this.document.getElementsByTagName('head')[0] as HTMLHeadElement;
let themeLink = this.document.getElementById('workflow-theme') as HTMLLinkElement;
let cssFileNm = styleName.toLowerCase();
    if (themeLink) {
        themeLink.href = `assets/workflow/css/${cssFileNm}.css`;
    } else {
        const style = this.document.createElement('link') as HTMLLinkElement;
        style.id = 'workflow-theme';
        style.rel = 'stylesheet';
        style.type = 'text/css';
        style.href = `assets/workflow/css/${cssFileNm}.css`;
        head.appendChild(style);
     }

    this.editor.updateConnectionNodes("node-1");
    this.editor.updateConnectionNodes("node-2");
    this.editor.updateConnectionNodes("node-3");
    this.editor.updateConnectionNodes("node-4");

    this.currentTheme = styleName;
}`

Try with a settimeout, the new style may not be loaded yet.
Others problems with style:

https://github.com/jerosoler/Drawflow/issues/708


fix: fix import of lcm function

Fix import of lcm function. #114

Hey, unfortunately this is the tip of the iceberg for the code that needs to change for things to match up with the new way comfy handles conditioning (for sliding context window), so I'm currently working on code that will work properly. I won't accept the PR since my in-progress code already has this change, but I appreciate it nonetheless!

I've now pushed a fix to match the ComfyUI changes - controlnets and prompt travel works with sliding contexts as expected.


Implements HartreeFock and VSCF initial states a BlueprintCircuit

Summary
Fixes #881
Details and comments
I have tried to preserve the original code as best as I can rather than a complete overhaul, however I'm unsure about one thing in this regard; should the checks for unsupported mappers remain in __init__ or be moved to _check_configuration for both initial states?

should the checks for unsupported mappers remain in init or be moved to _check_configuration for both initial states?

These need to be moved to _check_configuration or even the qubit_converter.setter to disallow it early :+1:
Please also leave the files in qiskit_nature.circuit unchanged and instead apply your changes to the files in qiskit_nature.second_q.circuit


Ci comment

SUMMARY
Trying to add comments on this PR programatically.
ISSUE TYPE

Bug, Docs Fix or other nominal change

test

test

CI Report 2

CI Report 3

CI Tests PASSED!

CI Tests PASSED! Thanks to Victor & Meg

CI Tests PASSED! Thanks to Victor & Meg


Tapping quickly with 2 fingers could cause an exception in the BaseTapGestureRecognizer class

With the following very simple widget tree:
class TagGestureHomePage extends StatefulWidget {
  _TagGestureHomePageState createState() => _TagGestureHomePageState();
}

class _TagGestureHomePageState extends State<TagGestureHomePage> {

  @override
  Widget build(BuildContext context) {

    return Scaffold(
        body: Container(
            color: Colors.green,
            child: GestureDetector(
              behavior: HitTestBehavior.opaque,
              onTapDown: (details) => print("outer tap down"),
              onTapCancel: () => print("outer tap cancel"),
              onTap: () => print("outer tap"),
              child: Center(
                  child: Container(
                    width: 300,
                    height: 500,
                    color: Colors.blue,
                    child: GestureDetector(
                      onTap: () => print("inner tap"),
                    )
                  )
              ),
            ),
        )
    );
  }
}

We could get an exception thrown by the TapRecognizer class with the following careful repro:
Tap on the outer container, and immediately tap on the inner container. 2 fingers are required in order to tap quickly enough. When the timing is perfect, the inner tap is recorded, but loses, however it is called back and exceeds its deadline and calls _checkDown anyway.
I/flutter ( 6942): outer tap down
I/flutter ( 6942): outer tap
E/flutter ( 6942): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: NoSuchMethodError: The getter 'position' was called on null.
E/flutter ( 6942): Receiver: null
E/flutter ( 6942): Tried calling: position
E/flutter ( 6942): #0      Object.noSuchMethod (dart:core-patch/object_patch.dart:53:5)
E/flutter ( 6942): #1      TapGestureRecognizer.handleTapDown (package:flutter/src/gestures/tap.dart:456:28)
E/flutter ( 6942): #2      BaseTapGestureRecognizer._checkDown (package:flutter/src/gestures/tap.dart:256:5)
E/flutter ( 6942): #3      BaseTapGestureRecognizer.didExceedDeadline (package:flutter/src/gestures/tap.dart:227:5)
E/flutter ( 6942): #4      PrimaryPointerGestureRecognizer.didExceedDeadlineWithEvent (package:flutter/src/gestures/recognizer.dart:493:5)
E/flutter ( 6942): #5      PrimaryPointerGestureRecognizer.addAllowedPointer.<anonymous closure> (package:flutter/src/gestures/recognizer.dart:446:40)
E/flutter ( 6942): #6      _rootRun (dart:async/zone.dart:1122:38)
E/flutter ( 6942): #7      _CustomZone.run (dart:async/zone.dart:1023:19)
E/flutter ( 6942): #8      _CustomZone.runGuarded (dart:async/zone.dart:925:7)
E/flutter ( 6942): #9      _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:965:23)
E/flutter ( 6942): #10     _rootRun (dart:async/zone.dart:1126:13)
E/flutter ( 6942): #11     _CustomZone.run (dart:async/zone.dart:1023:19)
E/flutter ( 6942): #12     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:949:23)
E/flutter ( 6942): #13     Timer._createTimer.<anonymous closure> (dart:async-patch/timer_patch.dart:23:15)
E/flutter ( 6942): #14     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:384:19)
E/flutter ( 6942): #15     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:418:5)
E/flutter ( 6942): #16     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)
E/flutter ( 6942):
I/flutter ( 6942): inner tap

The very simple fix for this is to do the following in gestures/tap.dart
change
void _checkDown() {
    if (_sentTapDown) {
      return;
    }
    handleTapDown(down: _down);
    _sentTapDown = true;
  }

to
void _checkDown() {
    if (_sentTapDown || _down == null) {
      return;
    }
    handleTapDown(down: _down);
    _sentTapDown = true;
  }

I've confirmed this fixes the issue and doesn't introduce any regressions that I can tell.

@justinmc FYI.
Let me know your thoughts about the fix. Thanks!

Thanks for the issue. Can you run flutter doctor -v and paste the output here, just so I know which version you're on?
A similar issue was supposed to be fixed by https://github.com/flutter/flutter/pull/45943.

[✓] Flutter (Channel beta, v1.12.13+hotfix.6, on Mac OS X 10.14.6 18G3020, locale en-US)
    • Flutter version 1.12.13+hotfix.6 at /Users/danny.valente/Library/flutter/1.12.13+hotfix.6-beta
    • Framework revision 18cd7a3601 (9 weeks ago), 2019-12-11 06:35:39 -0800
    • Engine revision 2994f7e1e6
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/danny.valente/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.0, Build version 11A420a
    • CocoaPods version 1.7.5

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 43.0.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

Thanks @dkwingsmt The fix to the issue you linked seems like it could very well fix this issue, and I doubt it's in v1.12, right?

@dkwingsmt I can confirm that when applying the fix you linked, I no longer see an exception in this scenario. Thanks!

Sounds like this can be closed!

@justinmc  why did you close this issue? I am using flutter ]
Flutter (Channel beta, 1.19.0-4.3.pre, on Microsoft Windows [Version 10.0.18362.1082], locale en-MU)
and the issue still persist

I closed it because the author said it was fixed, but I suppose it could be back as a regression.  Could you try again after updating Flutter?  Or maybe even using the master channel.
If the bug still persists, can you open a new issue with a minimal app to reproduce it and tag me?


Restrict Deformation Bug?

Hi,
I happened to glance at the restrict deformation code out of curiosity. Shouldn't https://github.com/stnava/ANTs/blob/master/ImageRegistration/itkANTSImageRegistrationOptimizer.cxx#L788
read:
updateFieldInv->SetPixel(index, tempInv);
instead of:
updateFieldInv->SetPixel(index, temp);
Also - can't the SetPixel calls go after the for loops, rather than being called multiple times?
If I'm right I can put together a PR, if I'm wrong sorry to waste your time - I'm guessing at how the algorithm works here.

PR would be good  -  we dont maintain that function.   Recommend using
antsRegistration.
brian
On Thu, Jun 8, 2017 at 3:32 AM, Tobias Wood<EMAIL_ADDRESS>wrote:

Hi,
I happened to glance at the restrict deformation code out of curiosity.
Shouldn't https://github.com/stnava/ANTs/blob/master/ImageRegistration/
itkANTSImageRegistrationOptimizer.cxx#L788
read:
updateFieldInv->SetPixel(index, tempInv);
instead of:
updateFieldInv->SetPixel(index, temp);
Also - can't the SetPixel calls go after the for loops, rather than being
called multiple times?
If I'm right I can put together a PR, if I'm wrong sorry to waste your
time - I'm guessing at how the algorithm works here.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/456, or mute the thread
https://github.com/notifications/unsubscribe-auth/AATyfh4tbxv2OLclvkdsNb05T3Tc-3Xjks5sB6N_gaJpZM4NzriP
.

Ah - I see now. I am using antsRegistration. I was grepping through my ANTs directory to find the file where restrict-deformation is processed, and I stopped at the first one I found without realising that was old ANTs.
The actual problem that I was trying to diagnose was that I was trying to restrict only the SyN stage of my registration, and it didn't seem to be working. Now I've looked at the code, I understand that all stages up to the one you want have to have the option specified, even if you specify e.g. 1x1x1x1x1x1 so they aren't actually restricted.
I am wondering if a different PR, to change the wording of the option to something like:
          This option allows the user to restrict the optimization of the displacement
          field, translation, rigid or affine transform on a per-component basis. For
          example, if one wants to limit the deformation or rotation of 3-D volume to the
          first two dimensions, this is possible by specifying a weight vector of '1x1x0'
          for a deformation field or '1x1x0x1x1x0' for a rigid transformation.
          Low-dimensional restriction only works if there are no preceding
          transformations. All stages up to and including the desired stage must have
          this option specified, even if they should not be restricted (in which case
          specify 1x1x1...)

Sure.

Done - see #457
Thanks!


doc/server: bring back doc for net.core.bpf_jit_limit

This setting was removed in
https://github.com/lxc/lxd/commit/6655cb19b6b55683dd11068d25f908d1a39cccc3 but it makes sense to keep it in the docs for older kernels.
Fixes #11312
Signed-off-by: Ruth Fuchss<EMAIL_ADDRESS>

Looks like we need a new acronym defining for spellchecker.


Evolving the Schema

a solution to unfreeze the HCA metadata standard to enable us to evolve the standard to capture the information needed to distribute, discover and interpret these new data types, such as but not limited to new assays, cell type annotations and expression matrices.
See more in a doc describing this feature.

List of things needed to unblock schema evolution:

[ ] Identify and modify documentation regarding the new procedures (Mostly stays the same, but there are some additions like e.g. generating the test data and metadata and maintenance)

[ ] Set up the GH repository

[ ] Set up generation of test data/metadata

[ ] Set up testing internally

[ ] Determine what is the placeholder for data/ This can be related to a previous issue we had before about being able to hit "submit" without relying on external approval.

[ ] What are the first set of changes we want to push? (Probably best in the doc and not as a ticket)

I'll add here as I think of more points but I think this is a good start to create the user stories

Above points are great. Just want to check we haven't skipped a step. I think the goals are:

To be able to easily create test datasets whenever the schema changes (so that we and the other components can test the effects of these changes)
Store these test datasets somewhere that is easy for us to deposit and easy for the other components to use them

I'd also add
3. For ingest to be able to test metadata updates

Implications of schema changes of graph validator?

Had a chat with @ESapenaVentura @Wkt8 on the flow of how we propagate schema changes and let other teams test data with the latest schema. https://docs.google.com/document/d/1lof-UiCTlrY53W1rR-AtDXU58rISalIOG7p3KX82-po/edit


Hydrogen install fails on OS 10.11.5

When try to install Hydrogen from the Atom Install packages menu it fails as follows:
Installing<EMAIL_ADDRESS>failed.Hide output…

<EMAIL_ADDRESS>install /private/var/folders/jj/x_wp_0257_nfp529qm60f90h0000gn/T/apm-install-dir-11652-53864-yjbq9p/node_modules/Hydrogen/node_modules/jmp/node_modules/zmq
node-gyp rebuild

make: *** No rule to make target Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/addon.gypi', needed by Makefile'.  Stop.
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at ChildProcess.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:820:12)
gyp ERR! System Darwin 15.5.0
gyp ERR! command "node" "/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /private/var/folders/jj/x_wp_0257_nfp529qm60f90h0000gn/T/apm-install-dir-11652-53864-yjbq9p/node_modules/Hydrogen/node_modules/jmp/node_modules/zmq
gyp ERR! node -v v0.10.40
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
npm ERR! Darwin 15.5.0
npm ERR! argv "/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/bin/node" "/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/Users/robw/.atom/.apm/.apmrc" "--userconfig" "/Users/robw/.atom/.apmrc" "install" "/private/var/folders/jj/x_wp_0257_nfp529qm60f90h0000gn/T/d-11652-53864-1p4v9op/package.tgz" "--target=0.36.8" "--arch=x64"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the zmq package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls zmq
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /private/var/folders/jj/x_wp_0257_nfp529qm60f90h0000gn/T/apm-install-dir-11652-53864-yjbq9p/npm-debug.log
My $PATH value from a Terminal is: /Users/roxx/anaconda/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/usr/local/git/bin:/usr/texbin:/Developer/NVIDIA/CUDA-7.5/bin
I don't know if there are any commonalities with nbviewer code but it works fine in my Atom installation

Have you tried installing hydrogen via terminal with apm install hydrogen?

Here is the result of running  apm install hydrogen
untitled
[robw]$ apm install hydrogen
Installing hydrogen to /Users/robw/.atom/packages ✗

<EMAIL_ADDRESS>install /private/var/folders/jj/x_wp_0257_nfp529qm60f90h0000gn/T/apm-install-dir-11653-60949-1029y09/node_modules/Hydrogen/node_modules/jmp/node_modules/zmq
node-gyp rebuild

make: *** No rule to make target Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/addon.gypi', needed by Makefile'.  Stop.
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at ChildProcess.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:820:12)
gyp ERR! System Darwin 15.5.0
gyp ERR! command "node" "/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /private/var/folders/jj/x_wp_0257_nfp529qm60f90h0000gn/T/apm-install-dir-11653-60949-1029y09/node_modules/Hydrogen/node_modules/jmp/node_modules/zmq
gyp ERR! node -v v0.10.40
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
npm ERR! Darwin 15.5.0
npm ERR! argv "/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/bin/node" "/Applications/ Codingware/Atom.app/Contents/Resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/Users/robw/.atom/.apm/.apmrc" "--userconfig" "/Users/robw/.atom/.apmrc" "install" "/private/var/folders/jj/x_wp_0257_nfp529qm60f90h0000gn/T/d-11653-60949-mbzavo/package.tgz" "--target=0.36.8" "--arch=x64"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the zmq package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls zmq
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /private/var/folders/jj/x_wp_0257_nfp529qm60f90h0000gn/T/apm-install-dir-11653-60949-1029y09/npm-debug.log
[robw]$
Regards
Rob

On 2/06/2016, at 21:09, Lukas Geiger<EMAIL_ADDRESS>wrote:
install hydrogen

Just to make shure, have you installed pkg-config and zeromq and used python2 for installing hydrogen? (https://github.com/nteract/hydrogen/blob/master/README.md#homebrew-on-os-x)
Which version of atom are you using?

The log message reports an error on node-gyp rebuild. This suggests Atom in your machine is unable to build native modules.
@rgbkrk , are the OS installation instructions here complete? Is it enough to install homebrew, zeromq and pkg-config?

@n-riesco Thanks for taking a look!
On travis the installation works with homebrew, zeromq and pkg-config.

@lgeiger I've been quiet for a while, but I'm still working on Hydrogen and IJavascript. At the moment, my priorities for these projects are: get out the next release of IJavascript, develop a javascript-only kernel launcher and solve (hopefully) the situation with native modules.

For my OS X machine, it's been enough to install homebrew, zmq, and pkg-config. This is assuming the node environment is set to use Python2 for node-gyp.

Attached are the answers to your questions taken from a Terminal session:
Last login: Sat Jun  4 17:06:27 on ttys000
Current PATH is: /Users/robw/anaconda/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/usr/local/git/bin:/usr/texbin:/Developer/NVIDIA/CUDA-7.5/bin:/Users/robw/torch/install/bin:/Users/robw/.local/bin:usr/local/Cellar/llvm38/3.8.0/bin
[robw]$ brew install pkg-config
Warning: pkg-config-0.29.1 already installed
[robw]$ brew install zeromq
Warning: zeromq-4.1.4 already installed
[robw]$ jupyter
usage: jupyter [-h] [--version] [--config-dir] [--data-dir] [--runtime-dir]
[--paths] [--json]
[subcommand]
jupyter: error: one of the arguments --version subcommand --config-dir --data-dir --runtime-dir --paths is required
[robw]$ jupyter --version
4.1.0
[robw]$ jupyter —paths
config:
/Users/robw/.jupyter
/Users/robw/anaconda/etc/jupyter
/usr/local/etc/jupyter
/etc/jupyter
data:
/Users/robw/Library/Jupyter
/Users/robw/anaconda/share/jupyter
/usr/local/share/jupyter
/usr/share/jupyter
runtime:
/Users/robw/Library/Jupyter/runtime
[robw]$ atom --version
Atom    : 1.7.4
Electron: 0.36.8
Chrome  : 47.0.2526.110
Node    : 5.1.1
robw]$ python --version
Python 2.7.11 :: Anaconda custom (x86_64)
Regards
Rob

On 3/06/2016, at 20:29, Lukas Geiger<EMAIL_ADDRESS>wrote:
Just to make shure, have you installed pkg-config and zeromq and used python2 for installing hydrogen? (https://github.com/nteract/hydrogen/blob/master/README.md#homebrew-on-os-x https://github.com/nteract/hydrogen/blob/master/README.md#homebrew-on-os-x)
Which version of atom are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/nteract/hydrogen/issues/279#issuecomment-223521235, or mute the thread https://github.com/notifications/unsubscribe/AGLYJoTi8-c1vfjYuqNxbiy280d5HtAlks5qH-XvgaJpZM4IsKjc.

Following up on your query about node environment I updated from node 6.1 to node 6.2. No improvement.
Then found this link : https://github.com/npm/npm/issues/823 and followed its instructions to “  add $HOME/.homebrew/lib/node/ to my NODE_PATH.” The file .homebrew did not exist, and adding it made no difference anyway..
However finding that NODE_PATH is empty makes me wonder. Should it be set? and if so what to?
My whole Python working environment is 2.7 so I assume that is what node-gyp is using too. How do I check?
I did find that node-gyp.js is tucked away in usr/local/lib/node-modules/npm/node-modules/node-gyp/lib so I added usr/local/lib to $PATH and usr/local/lib/node-modules/npm/ to $NODE_PATH as a test.
That did not work either.
I hope this negative information is helpful
Regards
Rob

On 4/06/2016, at 01:18, Kyle Kelley<EMAIL_ADDRESS>wrote:
For my OS X machine, it's been enough to install homebrew, zmq, and pkg-config. This is assuming the node environment is set to use Python2 for node-gyp.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/nteract/hydrogen/issues/279#issuecomment-223576236, or mute the thread https://github.com/notifications/unsubscribe/AGLYJsgdUVlUVvKi6Oi6w02tsk-1y8Dgks5qICmvgaJpZM4IsKjc.

On my machine $NODE_PATH is empty.
Try removing ~/.atom/packages/Hydrogen/ if anything is in there. Maybe this will help.

lgeiger's advice to remove ~/.atom/packages/Hydrogen/ did not help because there were no packages to find.
However I found and deleted 4 identical copies of the unpacked Hydrogen  package in  /private/var/folders/jj/x_wp_0257_nfp529qm60f90h0000gn/T/npm-39677-c351c005/unpack-d14c24747d76
There was no apm-install-dir or npm-debug file.
Further exploration of the build problem led me here:
https://github.com/JustinTulloss/zeromq.node/issues/490
where it says   "2.15.1 introduced Node 6 compatibility”
and
https://github.com/JustinTulloss/zeromq.node/wiki/Installation where it says:
"Then for every node.js project you have, you need to change to that directory and run
npm install zmq
(For installation on linux without root access, see Installation on Linux / bsd without root access )
Note that you can't do a global install. In node.js, once you require something, it needs to be in that directory (AFAIK), so don't try the '-g' global flag"
It seems that these links on the zmq developer's github site probably explain my problem.
But I still do not know enough about what  to do to compile Hydrogen.

Following advice received on issue 318 this issue was resolved.
I replied there as follows:
n-riesco's suggestion worked.
"On MacOS 10.11.5 all I had to do was remove the space from the file path and do:
export PKG_CONFIG_PATH=/usr/local/lib/pkgconfig to provide the link to libzmq.
Hydrogen installed without any further problems.
Thank you all"


Release staging

Notable changes:

@erictleung 	Add tests Get Geo-location Data challenge
@hangaebal 	Fix Claim Certificates - Challenges - Text Rendering Issues
@mriyaz Fixed the Ordered List Issue
@arun1595 Change number of arguments in sorted-union challenge's function to 1
@sarbbottam fix(a11y): using button instead of div
@pranayberry Fixed increment and decrement issue
@Bouncey Fix/ btn-block Bootstrap challenge

Thanks for contributing!

How do you get free codes,and for what they are used for??


Scope and time scheduling of the meeting (issue #196 related)

The goal is to execute a meeting covering mechanical systems package

closed as finished


Necklace is missing

There's no Necklace in the editor. It's displayed as Nothing if loaded from a file
Save.sav

uty-save-editor.netlify.app

Thanks! These should be fixed in https://github.com/KockaAdmiralac/uty-save-editor/commit/2e35360cda3726fd415d17445a7983baa8795583 now.


Improve select by indices performance on list array

Problem Statement
The current implementation is FileReader read the full column between indices[0],  indices[-1] in memory first, and apply arrow::compute::Function("take") to filter out the data.
Desired Behavior
We can push the selection further down to reduce the amount of data read from S3 / disk

Done with sparse read


There is a memory leak defect at line 3409 in the file /vim/src/autocmd.c.

Steps to reproduce
1.A pointer named pat is defined at line 3307 in the file /vim/src/autocmd.c. This pointer is assigned the return value of the function dict_get_string at line 3372, as shown in the following illustration:
https://github.com/LuMingYinDetect/vim_defects/blob/main/vim_20.png
2.A pointer named s is defined at line 690 in the function dict_get_string. This pointer is assigned the return value of the function vim_strsave at line 697 and returned at line 698, as shown in the following illustration:
https://github.com/LuMingYinDetect/vim_defects/blob/main/vim_21.png
3.Dynamic memory allocation is performed using the alloc function at line 27 in the vim_strsave function. This allocated memory block is then returned at line 30, as depicted in the following illustration:
https://github.com/LuMingYinDetect/vim_defects/blob/main/vim_22.png
4.After the execution of the dict_get_string function, pat is assigned a dynamically allocated memory block. When the if statement at line 3373 returns false, it confirms that the allocation of the dynamic memory block pointed to by pat was successful. However, when the if statement at line 3407 returns true, the program returns at line 3409, thus skipping the release operation of the dynamic memory block pointed to by the pat pointer at line 3429. This situation constitutes a memory leak, as illustrated in the following diagram:
https://github.com/LuMingYinDetect/vim_defects/blob/main/vim_23.png
5.The if statement at line 3411 also suffers from the same issue.
Expected behaviour
If the memory leak is not a false positive, it is advisable to fix it.
Version of Vim
9.1.0296
Environment
Ubuntu 20.04
Logs and stack traces
No response

I think this was fixed


reformat tables, grammar and style issues in documentation-files

What type of PR is this?
Add one of the following kinds:

cleanup

What this PR does / why we need it:
This PR is a fixing of some style and reference-issues
Which issue(s) this PR fixes:
Fixes #232

@PedroDiez updated your typo-hint and resolve the merge conflicts.
Please review again :)

Merging as commonly approved (the last commits resolved conficts)


Maintainance/finish integration with bigbio

some tests fail due to file staging:
[74/deffc2] Submitted process > NFCORE_QUANTMS:QUANTMS:CREATE_INPUT_CHANNEL:SDRFPARSING (PXD026600.sdrf.tsv)
WARN: Unable to stage foreign file: ftp://massive.ucsd.edu/MSV000087597/raw/RD139_Raw_files_DIA_Narrow/RD139_Narrow_UPS1_0_1fmol_inj2.raw (try 1) -- Cause: sun.net.ftp.FtpProtocolException: CWD MSV000087597:550 Failed to change directory.

WARN: Unable to stage foreign file: ftp://massive.ucsd.edu/MSV000087597/raw/RD139_Raw_files_DIA_Narrow/RD139_Narrow_UPS1_0_1fmol_inj1.raw (try 1) -- Cause: sun.net.ftp.FtpProtocolException: CWD MSV000087597:550 Failed to change directory.

WARN: Unable to stage foreign file: ftp://massive.ucsd.edu/MSV000087597/raw/RD139_Raw_files_DIA_Narrow/RD139_Narrow_UPS1_0_1fmol_inj1.raw (try 2) -- Cause: sun.net.ftp.FtpProtocolException: CWD MSV000087597:550 Failed to change directory.

WARN: Unable to stage foreign file: ftp://massive.ucsd.edu/MSV000087597/raw/RD139_Raw_files_DIA_Narrow/RD139_Narrow_UPS1_0_1fmol_inj2.raw (try 2) -- Cause: sun.net.ftp.FtpProtocolException: CWD MSV000087597:550 Failed to change directory.

WARN: Unable to stage foreign file: ftp://massive.ucsd.edu/MSV000087597/raw/RD139_Raw_files_DIA_Narrow/RD139_Narrow_UPS1_0_1fmol_inj2.raw (try 3) -- Cause: sun.net.ftp.FtpProtocolException: CWD MSV000087597:550 Failed to change directory.

WARN: Unable to stage foreign file: ftp://massive.ucsd.edu/MSV000087597/raw/RD139_Raw_files_DIA_Narrow/RD139_Narrow_UPS1_0_1fmol_inj1.raw (try 3) -- Cause: sun.net.ftp.FtpProtocolException: CWD MSV000087597:550 Failed to change directory.

Staging foreign file: ftp://massive.ucsd.edu/MSV000087597/raw/RD139_Raw_files_DIA_Narrow/RD139_Narrow_UPS1_0_1fmol_inj2.raw
Staging foreign file: ftp://massive.ucsd.edu/MSV000087597/raw/RD139_Raw_files_DIA_Narrow/RD139_Narrow_UPS1_0_1fmol_inj1.raw
[null] NOTE: Can't stage file ftp://massive.ucsd.edu/MSV000087597/raw/RD139_Raw_files_DIA_Narrow/RD139_Narrow_UPS1_0_1fmol_inj2.raw -- reason: sun.net.ftp.FtpProtocolException: CWD MSV000087597:550 Failed to change directory.
 -- Execution is retried (1)
ERROR ~ Unable to re-submit task `NFCORE_QUANTMS:QUANTMS:FILE_PREPARATION:THERMORAWFILEPARSER (2)`

and some fail due to mamba
Creating env using micromamba: openms::openms-thirdparty=3.1.0 [cache /home/runner/work/quantms/quantms/work/conda/env-1ebbd668b091d8a21d85f361d04f7768]
ERROR ~ Error executing process > 'NFCORE_QUANTMS:QUANTMS:LFQ:ID:DATABASESEARCHENGINES:SEARCHENGINECOMET (2)'

Caused by:
  Failed to create Conda environment
  command: micromamba create --yes --quiet --prefix /home/runner/work/quantms/quantms/work/conda/env-fc7fcf3cfd1984f30a2d9888b40928bf bioconda::openms-thirdparty=2.9.1
  status : 1
  message:
    critical libmamba Failed to create dir 'lib'
    warning  libmamba Invalid package cache, file '/home/runner/micromamba/pkgs/mono-<IP_ADDRESS>-he1b5a44_1001/include/mono-2.0/mono/jit/jit.h' is missing
    error    libmamba Cannot find a valid extracted directory cache for 'mono-<IP_ADDRESS>-he1b5a44_1001.tar.bz2'
    critical libmamba Package cache error.

This is not within the scope of what I am addressing so I will merge anyway!