Support more customization for the dialogs

Just going to think out loud in here.
Okay, so the material dialogs library offers a great deal of customization. Ideally, I'd like the morphing animation to work well with all of it.
Duplicating the API is hardly a good solution, might work as a quick hack right now, but in the long run a better architected solution should obviously be preferred.

Can we somehow just transmit the MaterialDialog root view from the calling activity into the MorphDialogActivity? Well, won't that be nice. But I think not possible without leaks, I'll look into this first.
But I think my best bet is to somehow save the "state" of the MaterialDialog.Builder to be reconstructed in the MorphDialogActivity. Perhaps by extending and implementing Parcelable? Or something like that...

P.S: Let me know if you'd like this library to have more functionality and customization. If there's demand, I'll get to work 😄


detailsToggle fast follow

this pr updates the code i the following sections because the ux was a little weird if you immediately clicked on the toggle after it expanded:

handleToggleClick flips the current open/close state (!current), and ensures hover is always active.
handleMouseEnterchecks if the content is already open (isOn) before enabling hover
handleMouseLeave enables so that the content closes only if it's not already open due to a click.

awesome! thank you sooo much @JKarlavige !

awesome! thank you sooo much @JKarlavige !

For sure, nice work on the fix!!

adding this slack convo for more context and explaination


Add delete button for k8s solutions

Add delete button for k8s solutions

done

done

Done
Branch
development_vdc_kube
PR
https://github.com/threefoldtech/js-sdk/pull/2447

Verified in #2225


Update index.html

Works now also without javascript 😬(~ redirects to a slim version if it detects that js is disabled)
Not sure about the whole single-page thing since a lot of portfolios are kind of single page approaches.
What is your opinion about it?
...Well I'll give it another shot. ✌

Hi Florian,
Sorry but this counts as a single pager as none of the content is hosted on the site itself and only redirects outside, sorry.
Nice work making it work without javascript tho :)


feat(zone-ingress/summary-view): add config

Adds the YAML config to the ZoneIngressSummaryView.
I've also removed some of the custom types that are added in the data layer to move further to the point of inferring the type from the return value.
Part of #2922

I've also removed some of the custom types that are added in the data layer to move further to the point of inferring the type from the return value.

Thanks for doing this ☝ 🎉

Separately, something I forgot to note before. This YAML is showing the ZoneIngressOverview which isn't an actual resource, its an HTTP API only thing.
Ideally here we want to show the config for the ZoneIngress I think, so if the aim is to show the copyable config of the thing.
It would be the equivalent of:

I thought this little weirdness only happened for Dataplanes, but just realizing myself now that because ZoneIngress/Egresses are just Envoy instances (i.e. Dataplanes) we also have this weirdness here.

Ideally here we want to show the config for the ZoneIngress I think, so if the aim is to show the copyable config of the thing.

Ah, thank you 👍 🙂

@johncowen I've reworked the part in the data layer and now include the config which is derived from item.zoneIngress and some more item metadata. Doing this gives the same result as and saves us making a request to /zone-ingresses/:name in the summary view. Please let me know what you think 🙂

Looks good to me, I think we just need to lose the availableServices which I think are added in the backend and therefore not part of the config.

Actually, we need to be intentional about what we are showing here, and then keep it consistent. I suppose the question is are we showing:

The config or;
The "result of what your config has done".

Now I think about it I'm not totally sure which we want to do.

Just to note down what we chatted through offline:

For the moment at least we decided that we would go with showing:

The config  that is currently applied, including all the stuff that gets added by the backend (and potentially k8s)

Reasons being:

I think this is what we currently do everywhere else
Chances are the user already has the config because they just applied it (although there is a usecase of using the GUI to share the config)
The interesting thing is the representation of what that config has done, which the user doesn't have

We can always come back and revisit if we decide different, but we should try to be consistent as to what these panels are showing.


Add tests

Added unit tests for the ProfileController and the TalkController.

@BackEndTea Thanks so much for your contribution!


When panel is enabled, only /index.html can be directly visited

Describe the bug
When the panel is enabled --enable-panel, the logic that handles the panel endpoints will only serve /index.html and any assets that are discovered under client/dist when building.
The logic that handles this is at: https://github.com/vmware-labs/wasm-workers-server/blob/549387b83be50b1b2ec7e04782c629a2c2e38965/crates/panel/src/handlers/panel.rs#L18-L29
There are other virtual paths such as /_panel/workers that are not covered by any of this cases.
Ideally, to avoid duplicating routing in multiple places, this logic should:

If path is empty, default to index.html, as per the current logic.
If an asset exists, serve it, as per the current logic.
Otherwise, serve a static page, that will perform an AJAX request to the current path and will forward the response to the client.

This way, we keep the current Single Page Application (SPA) approach, while making arbitrary paths directly visitable (e.g. reload endpoint, write path in location bar and hit enter...)
Reproduction steps
Run wws with --enable-panel. Visit /_panel/workers. The server will return a 404  HTTP status code.
Expected behavior
Visiting /_panel/workers directly should be equivalent to visiting / and navigating through the UI to the list of workers.
Additional context
No response

I reproduced the problem, except I don't know what it is now.
But it looks like the odds are that the front-end file isn't receiving the route forwarding.
Because I tried to change the path to /_panel/workers  and I can't access it either.
It only works if you visit /panel first, go to the UI and then click on the sidebar.

I tried to fix this bug and was able to access /_panel/workers through my browser.
I mainly utilized the fact that the pages I couldn't find were using the front end to forward requests.
I'm not sure if this is the desired result, can you help me out?
/// Find the static assets of the administration panel
#[actix_web::get("/_panel{_:.*}")]
pub async fn handle_static_panel(path: web::Path<String>) -> impl Responder {
    let path = if path.is_empty() {
        "index.html"
    } else {
        path.as_str().strip_prefix('/').unwrap()
    };

    let (content_type, content_data) = Asset::get(path)
        .map(|content| {
            (
                from_path(path).first_or_octet_stream().to_string(),
                content.data.into_owned(),
            )
        })
        .unwrap_or_else(|| {
            let default_content = Asset::get("index.html").unwrap();
            (
                "text/html; charset=utf-8".to_string(),
                default_content.data.into_owned(),
            )
        });

    HttpResponse::Ok()
        .content_type(content_type)
        .body(content_data)
}

Hello @dierbei,
Thank you for taking the issue and submit the PR to fix it! 👏

I tried to fix this bug and was able to access /_panel/workers through my browser.
I mainly utilized the fact that the pages I couldn't find were using the front end to forward requests.
I'm not sure if this is the desired result, can you help me out?

Yes, this is the expected result. The administration panel is a Single Page Application (SPA), so the routing happens in the browser. If you access the panel on / and then navigate to /workers through a link, it works. However, if you reload the site, it will return a 404 error as the admin panel dist package doesn't include any workers.html file.
The solution you proposed fix this error by returning the index.html file when there's no file in the admin panel dist folder that matches it.

But it looks like the odds are that the front-end file isn't receiving the route forwarding.
Because I tried to change the path to /_panel/workers  and I can't access it either.

This is the reason why you were getting these errors. The server didn't find any workers.html file.
Thank you!!


[6.0🍒] NCGenerics reverse condfail fix + bonus content!

Explanation:

Fixes an issue with the ASTPrinter where an incorrect swiftinterface can be generated (rdar://130179698).
Fixes a reverse condfail where the new compiler can't read a slightly older stdlib's swiftinterface (rdar://130424971).
In addition, there are NFC / documentation improvements which these patches depend on or are mostly harmless.

Scope: Bug fixes with modules.
Issue: rdar://130179698&130450351&130424971
Original PR:

https://github.com/swiftlang/swift/pull/74672 (reviewed by @jckarter)
https://github.com/swiftlang/swift/pull/74743 (review pending)

Risk: Low; bug fixes for cases that hadn't come up before.
Testing: Swift CI + new tests
Reviewer: see above

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test


[AUDIT][SPARK-41248][SQL] Add "spark.sql.json.enablePartialResults" to enable/disable JSON partial results

https://github.com/apache/spark/commit/5b5083484c
This PR adds a SQL config spark.sql.json.enablePartialResults to control SPARK-40646 change. This allows us to fall back to the behaviour before the change.
It's related to https://github.com/NVIDIA/spark-rapids/issues/7143.

We should test the behavior of the JSON parser in RAPIDS Spark and see whether it matches the behavior described in SPARK-40646.  Then we can determine next steps.


Critical vulnerability!!

Hey there,
I found a very critical vulnerability in satt wallet, Where I'm able to take over anyone's satt account. Please reply here so we can discuss.

hello,
Tell me about it

Report : 1
Issue : Account takeover
Severity : Critical!!
Description:
Response manipulation is a technique where attackers make changes in the response before it reaches the browser.
Steps to reproduce :

Go to your forget page and enter any random value.

2)Turn on the burp and click on the button

Capture this request in Burp .

the original request will be like this,

HTTP/1.1 401 Unauthorized
Date: Thu, 18 Aug 2022 14:27:25 GMT
Server: Apache/2.4.38 (Debian)
Access-Control-Allow-Origin: *
Content-Type: application/json; charset=utf-8
Content-Length: 33
ETag: W/"21-tKprQWr1DQE9uyH3PKvHNM6XwnI"
Set-Cookie: connect.sid=s%3A7PxtY7V6Hjexo7oO3pgHiFfEPzh8PJQD.ZQ8mTmk74GzntqqHVLc8DlJj3yr2QaokfLWey9RKq0g; Path=/; HttpOnly
Connection: close
{"code":401,"error":"wrong code"}

Now change the response to : {"code":200,"message":"code is matched","data":true}

Edited response would be like this,

HTTP/1.1 200 OK
Date: Thu, 18 Aug 2022 14:27:25 GMT
Server: Apache/2.4.38 (Debian)
Access-Control-Allow-Origin: *
Content-Type: application/json; charset=utf-8
Content-Length: 52
ETag: W/"34-/NTGnLO8ibhFg+ElJJGcN6zY48c"
Set-Cookie: connect.sid=s%3AYFY_skfYOd5rl1d4-c48-LScO4vXuHyG.%2BXI%2F%2BCYF00StHsWy8NaFJMZOqdSap5b54hGy72i8iko; Path=/; HttpOnly
Connection: close
{"code":200,"message":"code is matched","data":true}

Now forward the request.

You will be able to change anyone's password and directly login into their account.

Proof attached, check for reference!
impact:
Mass Account takeover : By this method an attacker can login anyones account password which leads to ;
Loss of funds;
Identity theft;
Privacy violation;
Fraud;
Reputation damage;
External policy violation (PCI); or Material loss.
proof of concept : https://drive.google.com/file/d/1UqV4f4Xfkv6dyQNstJNNBvs_8OSOu5xJ/view?usp=sharing
Best,
Yesh

@HamdiBenK Hi, Any updates on this?

Hello, any updates on this?


Cursor not showing on editor, can't select text.

Hello, I installed this editor on an angular 8 project. It seems to be working fine, but the cursor is not showing. I also can't select text. It makes it difficult to use the editor correctly.
I tried both the master branch and the typescript one, with the same results.
I installed it using npm and imported the json to the styles.css file. Do I need to do anything else? Thank you

Hello @Jorge0117
Did you find a solution for your Angular integration?

I experienced the same issue when trying to setup EasyMDE using Webpack (Elixir Phoenix project). Notably, the "sizing" string (a string of x characters at the top of the textarea) was visible, and the text wasn't properly contained by the boundary of the editor.
The problem ended up being my CSS import. I originally had this:
@import '../node_modules/easymde/src/css/easymde.css';

However, that only imports the CSS specific to EasyMDE; the styling isn't complete because it doesn't contain the CodeMirror CSS. This is what I needed instead:
@import '../node_modules/easymde/dist/easymde.min.css';

That will provide you with the distribution CSS which includes the dependency styles.

Please add a full simple example to reproduce your behavior

My case was exactly the same as @DavidAntaramian (Webpack on Phoenix project).
Is it expected that I import the CSS file separately or is this a bug? Shouldn't import EasyMDE from 'easymde' be enough?

You'll need to import the CSS file as well.

Alright, thanks!
Maybe adding this info to the README would prevent confusion in the future.


Test scaling in kuttl tests

This extend the existing kuttl tests and validate scale up/down of individual services.

/test heat-operator-build-deploy-kuttl


check pack_message parameters early on api level

Signed-off-by: Axel Nennker<EMAIL_ADDRESS>

@AxelNennker please take a look on CI failures. Seems like java and python wrappers have to be updated
https://build.sovrin.org/job/indy-sdk/job/indy-sdk-ci/view/change-requests/job/PR-1594/3/execution/node/614/log/
https://build.sovrin.org/job/indy-sdk/job/indy-sdk-ci/view/change-requests/job/PR-1594/3/execution/node/674/log/

@jovfer thanks for providing the links to the logs


Consider adding new DefaultHttpContext() to Mock<DefaultHttpContext> alternatives

https://docs.microsoft.com/en-us/dotnet/core/compatibility/2.2-3.0#http-defaulthttpcontext-extensibility-removed

If your unit tests use Mock<DefaultHttpContext>, use Mock<HttpContext> instead.

After some research, it seems new DefaultHttpContext() is a valid way way to mock HttpContext in unit tests. Should this be added to the above-quoted line? Otherwise, it could be questioned whether newing a DefaultHttpContext() is a Microsoft-accepted practice or Mock<HttpContext> is the only recommended approach.
Supporting comments:
@JamesNK (https://github.com/dotnet/aspnetcore/pull/6504#issuecomment-452526331):

The difference is HttpContext requires you to mock out all the behavior that you need, while DefaultHttpContext lets you take the defaults and override only what you care about.

@davidfowl (https://github.com/dotnet/aspnetcore/pull/6504#issuecomment-452567003):

Most of the tests I see, new up the DefaultHttpContext (it was actually intentional to make it POCO like and newable).

@anurse (https://stackoverflow.com/questions/45959605/inspect-defaulthttpcontext-body-in-unit-test-situation#comment78887946_45960766)

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

ID: df2066cb-244d-0b1b-cf46-428b198e8601
Version Independent ID: e52631de-28de-77c2-082f-e79b89b17f04
Content: Breaking changes, version 2.2 to 3.0 - .NET Core
Content Source: docs/core/compatibility/2.2-3.0.md
Product: dotnet-core
GitHub Login: @gewarren
Microsoft Alias: gewarren

After some research, it seems new DefaultHttpContext() is a valid way way to mock HttpContext in unit tests.

Yep, it's a totally reasonable option. Since most of the properties are settable and easy to override it means you often don't even need a mock at all! I'm in favor of adding it as another alternative to that breaking change note.


Help wanted with getting access to other s3 buckets and resources

I have a bucket that I'd like to use apart from the default Metaflow bucket.
I also have ec2 instance with mlflow server that stores metrics and model artifacts in s3.
I am not an advanced user of AWS at all.

As far as I understand I should give Metaflow access to these resources in Cloud Formation - is that correct?
I don't know whether it's a proper place but could someone give me a hint on how to do that?

Thanks

In IAM Manager I added  AdministratorAccess for batch and ecs-tasks. I will look for a better-defined policies


NarrowPublicClassMethodParamTypeRuleTest: Skip array-generics failling test

failling test for https://github.com/rectorphp/type-perfect/issues/34

Going through stale issues and closing "the ideas"/WIPs ones to keep focus on PRs and change.
Feel free to propose a PR with fix 👍
Thanks for understanding


caffe2 installs Python library to wrong location relative to CMAKE_INSTALL_PREFIX

I set CMAKE_INSTALL_PREFIX=$PREFIX and caffe2 dumped the Python modules in $PREFIX and not in $PREFIX/lib/python2.7 (or perhaps site-packages) where they should be placed. Subsequently, I had to add $PREFIX to my $PYTHONPATH to get caffe2 to find it.
This was tested on ae9af51496a4485437cf704bc7566eebe5e7f4d6
I used this conda build.sh and meta.yaml to do the build:
#!/bin/bash
set -e
if [ -z "$CONDA_CUDA_HOME" ]; then
  CUDA_ARGS=""
else
  CUDA_ARGS="-DCUDA_TOOLKIT_ROOT_DIR=$(printf %q "$CONDA_CUDA_HOME")"
fi
if [ -z "$PREFIX" ]; then
  PREFIX="$CONDA_PREFIX"
fi
rm -rf build || true
PYTHON_ARGS="$(python ./scripts/get_python_cmake_flags.py)"
mkdir -p build
cd build
cmake3 -DCMAKE_INSTALL_PREFIX="$PREFIX" $CUDA_ARGS $PYTHON_ARGS ..
make -j20
make install/fast

and
{% set version = "0.8.dev" %}

package:
  name: caffe2
  version: {{ version }}

source:
  path: ..

build:
  number: 1
  skip: True  # [win]

requirements:
  build:
    - future
    - glog
    - numpy
    - six
    - eigen
    - python <3
  run:
#   - future
#   - glog
#   - numpy
#   - six
#   - eigen
#   - python ==2.*

about:
  home: https://caffe2.ai/
  license: BSD
  summary: deep learning library

extra:
  recipe-maintainers:
    - ezyang

It seems like CMake should be able to automatically infer the correct installation prefix based on the base installation prefix, but as far as I can tell it can't.
One person solved this issue by using setuptools:

I realized that I could just call the standard Python installer
from CMake.  This led to a more reliable install process and also a vastly
cleaner CMakeLists.txt file.  My line for installing the Python code is
simply:
install(CODE "execute_process(COMMAND python setup.py install -f
--prefix=${CMAKE_INSTALL_PREFIX} WORKING_DIRECTORY
../source/libmoleculizer-1.1.2/python-src/language_parser)")
http://public.kitware.com/pipermail/cmake/2013-March/054141.html

I wish CMake had something like GNUInstallDirs, but for Python: https://github.com/BVLC/caffe/pull/4237
/cc @CDLuminate

Yes, I agree that using setuptools is a reasonable way to solve the problem. Caffe2 doesn't have a setup.py yet but it ought not be hard to write.


"Permission denied" when deleting a (large) file being served

Perhaps not directly related to #384, but it is afaik Windows 10 related.
I can't remove a (large audio) file while http-server is serving it. Presumably that's due to some locking that happens during the streaming/reading process. Once I killed http-server all was well again.
I'm curious if others have this same issue on other platforms.

I'm not getting the same on Linux Mint 18.2. My first guess would also be a locking mechanism, possibly just on Windows.

Pretty sure this isn't actually something we can solve...sadly...


⚠ Melody Beta has degraded performance

In 8b23531, Melody Beta (https://canary.melody.api.triza.dev) experienced degraded performance:

HTTP code: 200
Response time: 26412 ms

Resolved: Melody Beta performance has improved in 19e8816.


[optimize] Optimize spark load/broker load reading parquet format file

Please see the description in #3877 .

Great job

@xy720 Hi, Please add the UT for new BufferedReader class, Thanks.


[worker-relay] Add AND filter compatability

Adds AND filter capability re: https://github.com/nostr-protocol/nips/pull/1365
{ "&t": [ "4k", "movie" ] }

These PR's ready?

This one definitely is.


Feature request: Auto on/off for selected networks

Auto toggle on for cell networks and off for wifi so you don't have to activate/deactivate manually every time

Why do you need to disable Intra on your Wifi network?

I use OpenDNS on the wifi network where I have content filtering on my account. Since I loose content filtering and malicious domain blocking on the cell network with OpenDNS, I use Quad9 on that so I can at least have some security filtering.
So basically I have to manually toggle Intra on when not on the cell network and off when on wifi, of course forgetting many times to do so.


From 2.7.2 upgrade to 2.8.0,  there are continously http request not respond without any error.

Describe the bug
After upgrading pulsar 2.8.0,  there are continously http request not respond without any error.
And then the client timeout in 60 seconds.
Downgrade to 2.7.2, everything works well.
To Reproduce
The continuously test calls these http api

list tenants (20 per second)
create partition topics (20 per second)
list paritition topics (20 per second)
delete partition topics (20 per second)

Expected behavior
The http request get respond immediately.
Screenshots

Desktop (please complete the following information):

OS: Huawei EulerOS

I tried to rollback jetty and Jersey version， doesnot works

Can you provide the name of the API ? @Shoothzj

We're experiencing a similar issue after upgrading from 2.6.3 to 2.8.0. It seems to be particularly problematic when creating a producer using the Java client. "2 request timedout after ms 30000".

We're experiencing a similar issue after upgrading from 2.6.3 to 2.8.0. It seems to be particularly problematic when creating a producer using the Java client. "2 request timedout after ms 30000".

You can try to revert the commit  https://github.com/apache/pulsar/pull/9338
But I still don't know why

The issue had no activity for 30 days, mark with Stale label.


bug - while screen sharing

Hello,
Screen sharing is working fine, but it is not working with the following scenario.

Select window and start screen share (Working fine)
Close(remove) the selected window previously
Stop the content share
Not able to share screen again (its automatically get resize when start to screen share)

i thinks this the bug, handle window remove event while screen share.
can you please provice the solution.
thanks.

Hi @kalpeshims, thanks for pointing this out!
We are working on this, will get back to you when the fix is merged.


Expose a method to compute gravity for any physics body

Implements https://github.com/godotengine/godot-proposals/issues/8054 see the proposal for the justification.
Minimal test project: character_gravity.zip
I split this PR into 3 commits for easy reviewing:

First commit: In 3D we have both rigid and soft bodies, but currently a lot of the logic is duplicated between these two. I made a new class GodotSolidObject3D that can hold the logic shared by both. For 2D we don't have soft bodies yet, but I made the same change for consistency.

In just this commit, the classes are empty. More shared code can be moved here in the future.
This commit also renames the AreaCMP classes to Area2DCMP and Area3DCMP, which is necessary to fix a name conflict after moving the 2D AreaCMP to godot_area_2d.h for consistency with 3D.

Second commit: Move the shared code for computing gravity into the GodotSolidObject classes.
Third commit: Expose the compute gravity method on the physics nodes and physics server. Due to both 2D and 3D physics and all the different physics server classes for Godot physics, GDExtension, etc, this commit touches a lot of files, but it's only a few lines per file so it's not a lot of code.

This commit also uses the new methods in the script templates, which replaces the long var gravity: float = ProjectSettings.get_setting("physics/3d/default_gravity") line with just compute_gravity(), and now the code no longer needs to care which direction the gravity is.

Superseded by https://github.com/godotengine/godot/pull/84640.


Add resource requests/limits

Please, let me know if you don't think these limits are reasonable.

Can one of the admins verify this patch?

Hard to tell. We haven't really seen it under any form of load. :)

Can we do any load testing? Or is that too artificial?

We cannot got to prod without limits...

@aslakknutsen I've updated this - removed requests and lifted the limits a bit. Please review


Fallback to read RequestMethods from controller RequestMapping if api…

… method has none defined

Closing, as unit tests are needed.  Please re-open with unit tests if you're still interested.


Deformation field close to zero.

Hi Junyu,
Sorry to bother you again. Last time you suggested that I look at the diffeomorphic variants. Lately, I have been specifically looking at VoxelMorph-diff among the baseline models you included. However, I have observed that after the training converges, the resulted deformation fields are very close to zero, i.e. the intensity at each voxel falls under the range of around 0.001 to 0.02. When I visualize the deformation field in ITK-SNAP (see attached image), the grid lines seem straight which is drastically different from the deformation fields produced by non-diffeomorphic variants. I am wondering if this is normal.

At first, I suspected that I am saving the stationary velocity field (SVF) instead of the deformation field but that is not the case. I also suspected that there is something wrong with the "scaling and squaring" integrating step. However, the deformation field is still close to zero even after I replace the for-loop with the VecInt function here: https://github.com/voxelmorph/voxelmorph/blob/a746f77098962da1be9e6a03dacc3ef6c90d5244/voxelmorph/torch/layers.py#L51-L68.
After all these attempts to understand what causes such behavior of the model, I assume that the diffeomorphic variants such as VoxelMorph-diff tend to produce close to zero deformation fields. I would really appreciate it if you could provide some insight on this or confirm my observation.
Again, I enjoyed reading your paper and your code!

Hi @kvttt ,
Apologies for the delayed response. I borrowed the VoxelMorph-diff implementation from https://github.com/uncbiag/easyreg. The network output deformation field is actually normalized:
https://github.com/junyuchen245/TransMorph_Transformer_for_Medical_Image_Registration/blob/47517d0cc5b7b8b56c77b6b3efa60d59abba3432/IXI/Baseline_registration_methods/VoxelMorph-diff/models.py#L493-L494
To visualize the deformation before normalization, you can output the field from this line:
https://github.com/junyuchen245/TransMorph_Transformer_for_Medical_Image_Registration/blob/47517d0cc5b7b8b56c77b6b3efa60d59abba3432/IXI/Baseline_registration_methods/VoxelMorph-diff/models.py#L493
Junyu

Thanks for the clear explanation! In addition, should I also use the unnormalized deformation when calculating losses (e.g., Grad3d loss)?

Hi @kvttt ,
You don't need to. All the loss functions were calculated using the normalized field (the same training framework in  https://github.com/uncbiag/easyreg). Although it is feasible to use the unnormalized field for computing losses, you just have to tune the hyperparameters. So I don't recommend you to do it.
Junyu

Hi @junyuchen245,
Thank you for the quick reply. I just tried visualizing the un-normalized deformation and it works! As for my question regarding calculating loss on the un-normalized deformations, I think adjusting hyperparameters would definitely work. The trick I am currently using is to compensate for the difference between the normalized and unnormalized deformations by utilizing the loss_mult argument here:
https://github.com/junyuchen245/TransMorph_Transformer_for_Medical_Image_Registration/blob/dfa24a47a564a000aa9b4eea95a6e83a24568359/TransMorph/losses.py#L546-L548
And I think both approaches would essentially yield the same result. Again, thank you so much for the explanation!


Connect a Frontend to a Backend Using Services

service/access/backend-service.yaml
targetPort: http

Related Doc - https://kubernetes.io/docs/tasks/access-application-cluster/connecting-frontend-backend/
/sig docs
/language en

Hi.
What's wrong? The http port is port 80; that example manifest looks right to me.
/triage needs-information

I'm assuming you may be expecting a port number in front of targetPort, are you?

hey @lemonguge can you provide some more info on what the issue here is? I would like to work on this if possible

Sorry, my mistake


Get Query for Retrieving all Keys

What is the equivalent KV client query for performing the following operation to retrieve all keys?
etcdctl get "" --prefix=true

I tried the following query, it didn't work.
 CompletableFuture<GetResponse> futureResponse =
      client.getKVClient().get(ByteSequence.fromString(""),
          GetOption.newBuilder().withPrefix(ByteSequence.fromString("true")).build());

 GetResponse response = futureResponse.get();

etcdctl (in v3 mode) appears to be special-casing the empty string arg for getPrefix as 0x00 then using WithFromKey instead of WithPrefix.  I see you are using withPrefix.
jetcd has withRange which I am guessing is equivalent to WithFromKey in etcdctl.
So for the jetcd equivalent, try using withRange(\0).
See https://github.com/coreos/jetcd/blob/112956b907e7ea09fd5daf4e40dd99cf3f27dd0e/jetcd-core/src/main/java/com/coreos/jetcd/options/GetOption.java#L140

It did the trick. Thanks, Sam!

we probably need to document this somewhere, or provide a const to make it clear.

how to use withRange ?

Here is a complete example using withRange:
@Override
public Map<String, String> findAllKeys() {
    try {
        ByteSequence key = ByteSequence.fromString("\0");
        GetOption option = GetOption.newBuilder()
                .withSortField(GetOption.SortTarget.KEY)
                .withSortOrder(GetOption.SortOrder.DESCEND)
                .withRange(key)
                .build();

        CompletableFuture<GetResponse> futureResponse =
                client.getKVClient().get(key, option);

        GetResponse response = futureResponse.get();
        if (response.getKvs().isEmpty()) {
            log.info("Failed to retrieve any keys.");
            return null;
        }

        Map<String, String> keyValueMap = new HashMap<>();
        for (KeyValue kv : response.getKvs()) {
            keyValueMap.put(kv.getKey().toStringUtf8(),
                    kv.getValue().toStringUtf8());
        }

        log.info("Retrieved " + response.getKvs().size() + " keys.");

        return keyValueMap;

    } catch (Exception e) {
        throw new DatabaseException(
                "Failed to retrieve any keys.", e);
    }
}

thks


Warning due to recent lunr changes

Can we do something about this?

Presumably due to #943

I don't think we can prevent the overwriting of an "existing registered function", but we can overwrite the lunr warn function to do nothing, which I'm not convinced is a great idea.


Refactor requirements.txt into multiple files

I recently found out that we can include other requirements files from a requirements file. Something like this:
file1.txt:
package1
package2

file2.txt:
-r file1.txt
package3

The path of the child file is relative to the directory where the parent file resides. This means we can pip install from any directory.
We can use this to refactor our requirements into multiple files. We can have different requirements file for production and development and maybe different for python 2 and python 3.
This will help in https://github.com/zulip/zulip/issues/717#issuecomment-219829003

The package pipdeptree can help in finding out which packages are dependencies and which ones we directly need. We'll then be able to move packages to appropriate files.
I suggest that we make a directory named 'requirements' and put all our requirements files there.

Yeah, that refactoring concept sounds great!

Now we have split requirements.txt into multiple files in requirements/. However, we need to move development dependencies from requirements/common.txt to requirements/dev.txt.

Yep.  Here are some things that can be easily moved:

All the things tagged with # Dependency of moto s3 mock can move to dev.txt
transifex-client and its dependencies can move to dev.txt
ipython can at least be its own file (but we probably want it in prod)
pyflakes isn't needed in production
I'll just do these in a quick branch, and we'll see where that leaves us.

Opened #1074.

We can copy the comments on why we need things from:
puppet/zulip/manifests/app_frontend_base.pp

@timabbott
common.txt and py3k.txt both have six==1.9.0. While running pip install -r requirements/dev.txt, it is causing Double requirement error. Ideally it should be only in the common.txt.
Also mypy.txt has six==1.10.0.
Also if you are not working on that comments thing, let me know I will do it.

You can claim the comments thing; I'm not working on it.
The six issue is interesting and problematic for this overall strategy.  py3k.txt, ideally should be installable independently of dev.txt, since it is used in tools/travis/setup-py3k...

@sharmaeklavya2 any ideas for how to solve this problem?

@staranjeet, your're getting that error because you're using an old version of pip. Doing pip install --upgrade pip will fix that.

@sharmaeklavya2 do you know what the minimum version of pip required for this to work properly is?  We may need to update provision.py and our manual installation instructions to make sure a new enough pip is available.

mypy will only be installed on python 3. Some dependencies in common.txt are not python 3 compatible, so I didn't expect pip to install dependencies from common.txt or dev.txt. Therefore there will currently be no problems with six having different versions in mypy.txt and in common.txt. We'll eventually have to make the six versions same someday.

I don't know the minimum version of pip required to make this work. But provision.py currently runs pip install --upgrade pip before pip install -r requirements/dev.txt. We will need to update our manual install instructions though.

cool, that makes sense.

@staranjeet for doing the comments project, I'd probably recommend adding the comments on the lines before the requirements.txt entries, rather than the same lines as the versions are specified; I think that's a cleaner way to do that.  E.g.
# We need foo because it's the best!
foo==1.0

not
foo==1.0 # We need foo because it's the best!

I think we've basically split the files out in a pretty reasonable way at this point, so I think once we've copied the comments over, it'd be reasonable to declare this resolved.

Ok i will work on this and update accordingly

@timabbott : submitted a PR #1113 . The PR adds comments for most of the packages. Some packages have no comments.

@timabbott : I think this issue should be closed now.

Agreed, closed.  Thanks @sharmaeklavya2 and @staranjeet!


Add Azure Pipeline and Remove Jenkins builds

[X] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Remove the build line from the Jenkinsfile
Add azure-pipeline.yaml to point to main archive location
Remove build.toml
Add a post-build step to Core projects to move the PPL outputs to the correct directory (previously this was done by Jenkins)

Why should this Pull Request be merged?
Jenkins will not be used soon.
What testing has been done?
PR Build outputs should be in the right places.

[ ] build.toml is not actually deleted it appears
[ ] Let's go ahead and bump the version to 23.5


Web App Content Review

Hi @soroushojagh - I reviewed the web app content and here are some thoughts and suggestions below. Overall it's looking amazing - nice work!
Home page

Under "What is LDAR-Sim?" we say "multiple different technologies." I would delete "different" and say "LDAR programs may consist of multiple technologies."
Need a period at the end of the last sentence under the "What is LDAR-Sim?" section.
I think the link to the education page is super cool!
Under "How does the web app work" the word credential should be plural. On the setup and run part I would say "programs are simulated on our servers." On the view results part I would say "View and download results from simulations to understand how LDAR programs performed."
Under the "Key features" section for configurable programs you could write "Designing custom LDAR programs requires flexibility. Users can adopt default parameters or configure highly customized programs." Partnerships should be plural in the built together part. Estimate is spelled wrong in the designed for everyone box.

About page

In the second paragraph, I think the acronym for IM3S should be expanded to demonstrate what it stands for the first time it's mentioned. A semi-colon can also be used after the acronym before you say "a collaboration...".
There is a y on Alberta Energy Regulator that should be dropped.
There is no link provided to GitHub at the end where it says "click here." But I'm sure this is intentional right now ;).

Education

The clickable preview boxes look really good! Can we take screenshots of the publications that we're going to host on this page? That way users could see an Environmental Science & Technology paper or Environmental Research Letters paper as the preview. I think this would look cool!

How to use:

Should there be a mix of text and videos? For example, the steps could be written out in order with supporting videos below each step. It could be presented as a flowchart or something.

Setup - programs

The drop down menu looks great.
When I clicked "modify" and hovered over number of simulations and process with my cursor, a message appeared that said "you cannot change this," but I think these parameters are changeable.
When I modified a program and clicked save, the app would keep me in the general settings tab - it wouldn't return me to the initial page where all parameters are listed for that program unless I selected a different program from the drop down menu and then re-selected the program I just edited. I'm sure you're actively working on this though!
In the general settings tab, "number of process" should be "number of processes."
The program reports are sweet!!!

Other notes:

The status bars for registering or making changes to programs look awesome!
Communication from the web app with output messages to the reader is helpful.
Visuals are clean and navigation is easy.

The app is looking awesome @soroushojagh! Let me know if you have any questions about the feedback - I hope it's helpful!

I've looked it over as well and only have one minor edit to add to Coleman's list:
On the setup, run, and results pages, when not logged in, a message says “without registration or logging in , we are unable to display setup menus. Please Sign Up here or Login here”. This might be simpler as: “LDAR-Sim cannot be used without an account. Please Sign Up here or Login here.”. There is currently a link for logging in but no link for signing up.
This looks amazing - really great work.

Login and sign up - sign up button is missing if browser window screen is narrow and tall.
Terms are missing.
Couldn't login - network error.
Apologies I could not provide more feedback - I can go at this again when I can register.


Add 	 an interface to strftime to the time package

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.10
Does this issue reproduce with the latest release?
N/A
What operating system and processor architecture are you using (go env)?
Linux/amd64
What did you do?
N/A
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
N/A
What did you expect to see?
N/A
What did you see instead?
N/A
I think an interface to strftime would be a useful addition to the time package, making easy formatting of dates

Duplicate of #8390

Thank's for the quick reply. Does that mean there is no interest in adding
this? I have working implementation, it's a direct call to the function in
the C standard library, which I would have thought would have helped in
porting to go.
Regards
Andrew alcolm
On Tue, 8 May 2018 at 19:39, Tim Cooper<EMAIL_ADDRESS>wrote:

Duplicate of #8390 https://github.com/golang/go/issues/8390
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/25294#issuecomment-387500586, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Aj2RoQplMh2ZBZ17_Vje3djUZDsAFfTEks5tweZKgaJpZM4T3FWu
.

Thanks, but we aren't going to add this to the standard library.  It works just as well outside the standard library, and the standard library already has a way to format dates.  https://golang.org/doc/faq#x_in_std .


[BUG] [VSCode Extension] Adding functions to an LLM tool, adds the same functions to all LLM tools in flow.

Describe the bug
The LLM tool provides a button to add functions:

When clicking that button, a new json file opens and a (list of) functions can be added.
When saving/closing this file:
EXPECTED: the functions array is added to the one that was selected.
ACTUAL: it is added to all nodes of type LLM in the flow.
SCREENSHOTS:
Added to the selected node:

Added to another LLM node:

Screenshots

Environment Information

Promptflow Package Version: [1.1.0]
Operating System: [DevContainer based on mcr.microsoft.com/devcontainers/python:3.11]
Python Version: [python==3.11.7]
VS Code version: [1.86.0-insiders]
Prompt Flow extension version: [1.9.2]

I have also encountered similar problems. Sometimes it happens. You can temporarily bypass this issue by directly editing the YAML file.

@roel4ez @chenyugithub365 Thanks for your reports. We've figured out the root cause and it should be fixed in next release.


Finish mobile setup

-Added more html and css styling
-Added more about me and what I do

Hi, I have used all my extra reviews and I need 1 more to complete my
project , can I please get 1
On Sat, Jan 14, 2023 at 5:14 PM Amel Khiri @.***> wrote:

@.**** requested changes on this pull request.
STATUS: CHANGES REQUESTED ♻ Hello @Munyabelden
https://github.com/Munyabelden ,
✋🏼 Nice done I am impressed. Your project is great. There are just some
small changes to add.
HIGHLIGHTS

Your Project looks great ✔
No Linter errors ✔
You have a descriptive PR ✔

Required changes :

Please, check the comments under the review.

As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form. https://airtable.com/shrQAqnBwek5a0O0s
If anything seems to be unclear to you, reach out and ask anything by
commenting on this Pull Request and mention me as Algerina
https://github.com/algerina, I will receive a notification and will
respond.
GitHub https://github.com/algerina | LinkedIn
https://linkedin.com/in/amel-khiri | Twitter
https://twitter.com/AmelQahwadji
Reviewed by Amel Khiri https://github.com/algerina
In style.css
https://github.com/Munyabelden/PortfolioMobilePage/pull/2#discussion_r1070298886
:

+.features li div {

overflow: hidden;

+}

+.features li i {

color: #abc7c3;

font-size: 20px;

line-height: 24px;

+}

+.feature-items li {

display: flex;

flex-direction: row;

align-items: center;

width: 303px;

height: 66px;

background: #f7f7f9;

border-radius: 8px;

color: #344563;

overflow: hidden;

padding-inline: 25px;

justify-content: space-between;

You did a great job emplementing the changes required but your
elements are not positioned correctly, you can fix the issue by deleting
the width in the .feature-items li class and adding a little margin.
[image: mob2]
https://user-images.githubusercontent.com/53347966/212478509-0e63f78c-a82a-483f-8753-eaf68766a192.JPG

—
Reply to this email directly, view it on GitHub
https://github.com/Munyabelden/PortfolioMobilePage/pull/2#pullrequestreview-1248817552,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A32THZFBWKJHHGES3JNLE6TWSK7GTANCNFSM6AAAAAAT2YTMXE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Munyabelden  I think that the only thing that you can do If you think that the code review was not fair, is to request a second opinion using this form.

Hello @Munyabelden
This is an additional review as a result of the second opinion request.

The issue is not mentioned by @IngDavidTM, the project should be approved and the issue should be optional. It appears that your project has been approved.If you require further clarification on any of the reviews, please reply to this message and tag me.

Cheers and Happy coding! 💻 👏

GitHub | LinkedIn | Twitter
Reviewed by Ahmet Bozacı


Add E2E test for TableOfContentsPlugin

@karam-qaoud Does this fix the page freezing issue

@thegreatercurve Yes, packages/lexical-playground/src/plugins/TableOfContentsPlugin/index.tsx has the fix.


worked example of default export?

I have read https://github.com/cidgoh/DataHarmonizer/wiki/DataHarmonizer-Templates#export_-fields but still don't understand how to use it.
I know there the templates bundled with this repo have exports enabled, but I believe they all use custom methods. Are they any examples of enabling exports without any custom JS programming?
For example, I added an EXPORT_MIxS column to my template's data.tsv and populated that with the the values that I want to appear as column headers in the export.
What else do I have to do on top of that? The documentation says to list the "export format keyword" in the TEMPLATES dictionary in main.js, but the dictionary entries look like
'CanCOGeN Covid-19': {'folder': 'canada_covid19', 'status': 'published'}
Would I enter EXPORT_MIxS or MIxS as they key? What should I enter for the folder and status?

Hi,
The TEMPLATES dictionary in main.js serves as a menu system of the templates only.  In each template folder the export.js code provides the list of possible export formats for that template in the EXPORT_FORMATS dictionary. I see I toasted the simplest example because it wasn't in use a number of months ago but here it is, so if you have this in your export.js with appropriate name change it should work, i.e. swap out "EXPORT_MIxS" for "Generic":
/**

Download secondary headers and grid data.
@param {String} baseName Basename of downloaded file.
@param {Object} hot Handonstable grid instance.
@param {Object} data See data.js.
@param {Object} xlsx SheetJS variable.
*/
var exportGeneric = (baseName, hot, data, xlsx, fileType) => {
// Create an export table with template's headers (2nd row) and remaining rows of data
const matrix = [getFlatHeaders(data)[1], ...getTrimmedData(hot)];
runBehindLoadingScreen(exportFile, [matrix, baseName, fileType, xlsx]);
};

// A list of the above functions keyed by the Export menu name they should appear as:
var EXPORT_FORMATS = {
"Generic": {'method': exportGeneric, 'fileType': 'tsv', 'status': 'published'},
//etc
};
There is just a default few lines of code that have to be provided for a generic export; otherwise it is a more complicated process if field values or ordering have to be adjusted as part of the export.

Progress! The export process starts now and I get a download, but there's no header and I'm getting the following error message. I'll remove my creation of the empty 'unknown section' and experiment some more.

Failed to map:
The template for the loaded file has a configuration error:
unknown section
This is a field that has no parent, or a section that has no fields.

The export looks good now. Thanks.

Thanks
https://turbomam.github.io/DataHarmonizer/main.html https://turbomam.github.io/DataHarmonizer/main.html
I built that with my new general purpose LinkML to DH converter. Here’s the relevant lines form the Makefile, as a starting point. I haven't documented anything yet, including the dependency on Python Poetry.
https://github.com/turbomam/linkml-round-trips/blob/b3067266d337afb777b8826068475445e191e774/Makefile#L30-L34 https://github.com/turbomam/linkml-round-trips/blob/b3067266d337afb777b8826068475445e191e774/Makefile#L30-L34

On Nov 23, 2021, at 5:59 PM, Damion Dooley @.***> wrote:
Hi,
The TEMPLATES dictionary in main.js serves as a menu system of the templates only. In each template folder the export.js code provides the list of possible export formats for that template in the EXPORT_FORMATS dictionary. I see I toasted the simplest example because it wasn't in use a number of months ago but here it is, so if you have this in your export.js with appropriate name change it should work, i.e. swap out "EXPORT_MIxS" for "Generic":
/**
Download secondary headers and grid data.
@param https://github.com/param {String} baseName Basename of downloaded file.
@param https://github.com/param {Object} hot Handonstable grid instance.
@param https://github.com/param {Object} data See data.js.
@param https://github.com/param {Object} xlsx SheetJS variable.
*/
var exportGeneric = (baseName, hot, data, xlsx, fileType) => {
// Create an export table with template's headers (2nd row) and remaining rows of data
const matrix = [getFlatHeaders(data)[1], ...getTrimmedData(hot)];
runBehindLoadingScreen(exportFile, [matrix, baseName, fileType, xlsx]);
};
// A list of the above functions keyed by the Export menu name they should appear as:
var EXPORT_FORMATS = {
"Generic": {'method': exportGeneric, 'fileType': 'tsv', 'status': 'published'},
//etc
};
There is just a default few lines of code that have to be provided for a generic export; otherwise it is a more complicated process if field values or ordering have to be adjusted as part of the export.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/cidgoh/DataHarmonizer/issues/236#issuecomment-977250872, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGAEX6IKJEW3TSZCBXD355DUNQMETANCNFSM5IURUI3A.

Hi, so if I get this right, the round trip is a good test of what is preserved in a linkml spec in DH, and what is lost at moment, e.g. I presume LinkML fields which are the combo decimal + unit are probably converted to xsd:token fields at moment. I'd still need you to describe to me what your code examples are doing i/o wise.
I'm just turning to more DH work today on smaller items (having validate normalize capitalization of entries to match selection menus for example); next week I aim to finesse the native DH LinkML model, and hopefully tackle the number+unit issue.


Does not work on OS X

Looks like because of posix realtime extensions it does not work on OSX.
See https://gist.github.com/jbenet/1087739 for workarounds.
It fails with:
luajit: lib/ljio/core/timer.lua:5: dlopen(librt.dylib, 5): image not found
stack traceback:
	[C]: in function 'load'
	lib/ljio/core/timer.lua:5: in main chunk

Thanks for reminding.
In fact, the luajit.io only supports Linux, because it depends on a lot of linux specific features, e.g. epoll, as well as linux-specific libc.

I would recommend making it clear on your homepage that you only work on linux.  This will help set expectations.  Trust me, it's never good when people think your project can do more than it really can.
That said, I think this is a really good idea.  I went with using libuv for my project (luvit.io) because cross-platform is a big deal to me, but I also love the minimal simplicity of this project.

Thanks for your advice.
I would improve the homepage later.


Assert that det != 0 when inverting matrices

Fixes two TODOs that keep bugging me. Hopefully that's what @abarth
meant by them.

Sure.  Can we actually handle exceptions at these points?  If so, LGTM.

Define "handle exceptions".
I mean, it's not going to be good for the user. But usually calls to these functions are called from code in a high-level try/catch (e.g. the pointer router one) which will report it to the console. In practice in release mode this is going to do crazy things (looks like Matrix4 returns the input matrix without inverting it in these cases) and in checked mode it'll get a dart crash.
It wasn't clear to me if the comment was meant to imply that the transforms should never be degenerate (which is what this patch assumes), or whether it was meant to imply that we should do something useful when they occur (e.g. assume that the hit test didn't hit, return 0,0 as the point, or whatever).
Happy to switch it from asserts to the alternative approach if you prefer that.

Usually the right behavior is to act as if it isn't hit.  For example, consider a transform with the zero matrix.  There's no way to hit the thing inside the transform.

Ok, that's easy to do for the hit test case.
What should localToGlobal / globalToLocal do?

What should localToGlobal / globalToLocal do?

Probably either return null or throw an exception...
It probably should be legal to have degenerate transforms in the tree.  For example, if you want to animate a scale to zero, on the last frame of the animation, you'll have a degenerate subtree.  That implies that it's not an error to have an non-invertable transform, which makes me think that localToGlobal / globalToLocal should return null and callers will need to handle that case.

Seems reasonable.

I'm going to close this pull request since this approach isn't the one we want to do.
Filed https://github.com/flutter/flutter/issues/1161 to trace further work on this.


Share bitcoin test

Make fake bitcoin test fixture reusable across libs

Codecov Report
Base: 65.39% // Head: 65.71% // Increases project coverage by +0.31% :tada:

Coverage data is based on head (af07469) compared to base (5e4cd69).
Patch coverage: 97.72% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1003      +/-   ##
==========================================
+ Coverage   65.39%   65.71%   +0.31%
==========================================
  Files         116      117       +1
  Lines       17404    17536     +132
==========================================
+ Hits        11382    11524     +142
+ Misses       6022     6012      -10

Impacted Files
Coverage Δ

client/client-lib/src/wallet/mod.rs
69.84% <ø> (ø)

fedimint-testing/src/bitcoin/bitcoind.rs
100.00% <ø> (ø)

fedimint-testing/src/lib.rs
91.91% <ø> (ø)

fedimint-testing/src/bitcoin/fixtures.rs
97.72% <97.72%> (ø)

fedimint-server/src/lib.rs
81.49% <0.00%> (+1.02%)
:arrow_up:

fedimint-server/src/net/peers.rs
91.17% <0.00%> (+2.05%)
:arrow_up:

fedimint-server/src/multiplexed.rs
91.25% <0.00%> (+2.50%)
:arrow_up:

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.

Sorry, needs a rebase now, really need a bot to tell us which PRs are compatible.

Would it be less complicated if we just had gateway-testing depend on integration-test?  I image there will be other code that needs to be shared such as FakeLightningTest, and perhaps we also want to run gateway tests with real bitcoind when mocks are disabled.

Would it be less complicated if we just had gateway-testing depend on integration-test?

No, but both gateway-tests and integration-tests can depend on fedimint-testing. This scales nicely if you want to build bespoke test suites per crate and per crate groups. fedimint-testing holds all base test artifacts like bitcoin / lightning mocks like you mention

LGTM, eventually we might want to put the testing crates into their own directory, it's getting kinda crowded in the root dir.


Fix #3631

Fixes the logic problem that was causing too many atoms to be considered for potential ring-stereo involvement.
Also does a minor bit of refactoring/cleanup of isAtomPotentialChiralCenter(), which was just too much of a mess.

I think this code is ok, and it is consistent with other things we do, but I can't help but think that we might be wrong about not allowing N in cycles being chiral. Unless I'm wrong,
https://cipvalidationsuite.github.io/ValidationSuite/?search=VS132 should be an example of that.

I think we can safely add an exception for bridgehead Ns as part of an enhancement for the next release cycle. It's still worth pointing out that InChI does not recognize those.


i18n(pt-br): add reference/plugins.md

What does this PR change?

Changes or translations of Starlight docs site contents.

Description
This PR adds the Portuguese version of the reference/plugins.md page.

Btw, I forgot to welcome you. Welcome! :D. Thank you very much for putting this in. Make sure you are on Astro discord  to receive a warm welcome when this or the other PR gets in.

Thanks for the welcome @vasfvitor and @Fryuni!
I'm starting using Astro on some projects, and I came to Starlight from their documentation.
There is nothing better to dive into a project than digging into the documentation and working with some translations.

GitBug attacks again 😆


updated readme

What kind of change are you introducing

[ ] Breaking change
[ ] New framework implementation
[x] Misc. lib improvement

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added Grafana dashboard for subnets

currently only showing Spaces and WAGMI on Fuji
not installed by the monitoring installer script (currently too niche)

love this thank you


feat: adding pre-commit

PR checklist:

[ ] Github Issue associated with branch.
[ ] Code passes linting through autopep8.
[ ] PR branch is up-to-date with latest master branch.

Codecov Report

Merging #4 into master will not change coverage by %.
The diff coverage is 29.62%.

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   59.51%   59.51%
=======================================
  Files          10       10
  Lines         788      788
=======================================
  Hits          469      469
  Misses        319      319

Impacted Files
Coverage Δ

src/algorithms/lstm_classifier/lstm_classifier.py
32.00% <24.00%> (ø)

tests/sanity.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[i18n] Move xx.toml to each language subdirectory

Describe your changes
This PR is to make GitHub request reviews to localization approvers of each language when a PR that touches i18n/xx.toml gets opened.
Related issue number or link (ex: resolves #issue-number)
resolves #2199
Checklist before opening this PR (put x in the checkboxes)

[X] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[X] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Note: if we are to adopt this, then we need to manage & update i18n/data/xx/CODEOWNERS files when the root CODEOWNERS file gets changed, to make them synced well.
When merging this PR, please merge using 'Squash and merge' method.

@seokho-son Good point! I couldn't come up with that.
After seeing your suggestion, this might be able to be achieved by not moving xx.toml files and creating symlinks, but just adding lines to existing CODEOWNERS file like:
# Approvers for Arabic contents
/content/ar/ @TarekMSayed @same7ammar @AShabana @arezk84
+/i18n/ar.toml @TarekMSayed @same7ammar @AShabana @arezk84

# Approvers for Bengali contents
/content/bn/ @Arindam200 @Imtiaz1234 @mitul3737 @sajibAdhi
+/i18n/bn.toml @Arindam200 @Imtiaz1234 @mitul3737 @sajibAdhi

@seokho-son How does this seem to you?
I can open a pilot PR to see whether GitHub says that a proposed CODEOWNERS file is okay, like:

@jihoon-seo
Validation for CODEOWNERS file works for me as well. I think you don't have to open another pilot PR.
I think updating the main CODEOWNERS will pass the validation since existing approvers will be duplicated.

LGTM,The verification process has confirmed that the Approvers teams across different language locales i18n and content are consistent and identical.


Update bitflags to 0.7

bitflags 0.6 and 0.7 are not compatible with Rust <1.8, maybe I should use a version range there?

If they are API compatible (at least for the part used by wayland-kbd) it'd be best, yes.

0.5-0.7 are. Will amend the patch soon.

Amended!

Thanks!


Implement In operator for string and string

Implemented In operator for string and string
This will allow expressions like
'a' in 'abc'

We already have this one:
'abc' contains 'a'


Configure: add -fkeep-inline-functions to --strict-warnings

The reason for doing this is that there are compilers (such as SUNPRO
CC) that do this by default, and this has consequences when header
files like safestack.h and lhash.h are included (possibly via other
header files) but the resulting program isn't linked with libcrypto.
Incidently, we have a test program that gets safestack.h included and
doesn't link with libcrypto (it shouldn't need to, since all it does
is to verify that it builds without macro clash), and is thereby the
perfect test victim.
Related to #8102

NOTES:

I fully expect this PR to fail.  That is a success, there will be a follow up PR that fixes the problem.
I'm making this PR for 1.1.1, and will do the same with the follow up PR, because that's the branch that this has been reported for.  Of course, this and the follow up PR will be up-ported to master.

Note that I've been working on a solution, and there are actually two, both quite displeasing but from different points of view.

Hi levitte,
I get the below warning during make on Solaris
Warning: Option -fkeep-inline-functions passed to ld, if ld is invoked, ignored otherwise

Hi levitte,
I get the below warning during make on Solaris
Warning: Option -fkeep-inline-functions passed to ld, if ld is invoked, ignored otherwise

@levitte  can you help

Ignore that warning.

Ignore that warning.

@levitte
along with this warning i an getting the below error as well.

Ignore that warning.

@levitte
along with this warning i an getting the below error as well.

do you have any idea

Ignore that warning.

@levitte
along with this warning i an getting the below error as well.

do you have any idea

@levitte

I'm going to ask you to be a bit more patient than pinging me twice within the same minute...

I think we need a better solution for inline functions than just keeping them.

Note that this is for --strict-warnings, and is done specifically to emulate what other compilers (such as some Solaris CC that I forget the exact name of) do by default.  The intent is to create problems, because we have a problem with those compilers, and our current "solution" to the problem is pretty bad.
However, the error that @sunnysaurabh93 finally showed says that we can't have that flag in CFLAGS because the compiler can't filter it out when calling ld (because GNU ld also accepts -f, but with a completely different and incompatible meaning)  [sigh]

I suggest that we fix the current solution.
We can make API breaking changes in 3.0...

We can make API breaking changes in 3.0...

... but only so it affects a maximum of 2%...

I doubt that that many folks use the inline functions in our headers directly....

The mere fact that we do call DEFINE_SPECIAL_STACK_OF and variants thereof in a public header file is enough, it only comes down to which compiler is used.  The problem comes when they are not being used, or very specifically, when they don't link with libcrypto (even indirectly)

... so all that's needed to create havoc is for an application to include something that in turn includes openssl/safestack.h or openssl/lhash.h...

Removing the DEFINE_* from the headers will prevent the compiler problems and I doubt it will impact many.  If we're worried about impacting anyone, we could declare the functions in the header and define them as exports from libcrypto.  There is no necessity to inline any of them.
Using the inline functions and not linking against libcrypto won't work -- they just call through to the generic flavours.

I.e. I think that there is a better solution to this one.
I can make a PR to illustrate at some point.  No idea when.

I did actually work on something along those lines at some point...  it probably sits in a branch somewhere.  I can have a look later today

I.e. I think that there is a better solution to this one.
I can make a PR to illustrate at some point. No idea when.

Hi @paulidale  @levitte  Did you guys get any chance to look into this. What should I do to resolve this issue. I am unable to proceed. Do you guys have any temporary solution for this? That will be a great help

I want to do something about this, I've not had a chance yet :(
Like @levitte, I've got a plan.

I want to do something about this, I've not had a chance yet :(
Like @levitte, I've got a plan.

@paulidale
Ok.. But what I wanted to know that how people are using OpenSSL 111c on solaris. There are multiple products which uses openssl on solaris. Is there any temporary solution. I was able to use OpenSSL 102r as part of our product on solaris, Now having trouble uplifting the same to 111c on solaris.

Should this PR actually be in WIP since it cannot be merged until the underlying problems from #8102 are solved.

Is this relevant any more since #8102 is now closed and the stack and hash implementations no longer use static inline functions?

Is this relevant any more since #8102 is now closed and the stack and hash implementations no longer use static inline functions?

I dunno any longer.  I'd be happy to close this.  If we find a need later on, I suppose this will re-emerge in a new PR.
So, closing.


[sidecar] support port re-use with unique bind IPs

Allow users to configure sidecars to listen on the same port but with unique IP addresses
I'll be able to test this in our production environment this coming week which consists of multiple control planes and VM's. I will report back.
\cc @yskopets
Closes #37487.

Hi @dmichel1. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

just confirmed it works on my VM with this patch.
[root@lb-dev ~]# netstat -nltp | grep 127 | grep 8080
tcp        0      0 <IP_ADDRESS>:8080         <IP_ADDRESS>:*               LISTEN      30264/envoy
tcp        0      0 <IP_ADDRESS>:8080          <IP_ADDRESS>:*               LISTEN      30264/envoy

/test integ-pilot-multicluster_istio

@dmichel1: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
cd60ecf27487783b7a047994d3b6b6961f62d48e
link
true
/test release-notes_istio

integ-pilot-multicluster_istio
cd60ecf27487783b7a047994d3b6b6961f62d48e
link
true
/test integ-pilot-multicluster_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@dmichel1: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
cd60ecf27487783b7a047994d3b6b6961f62d48e
link
true
/test release-notes_istio

integ-pilot-multicluster_istio
cd60ecf27487783b7a047994d3b6b6961f62d48e
link
true
/test integ-pilot-multicluster_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Customize the order of jib filter，Jib filter extensions created layer order is not good

normal,jib create layers for:
libs
classes
resources

this is good layers for thin docker.but,Libs is too big, and there are some jar packages in it that may change frequently, so it is necessary to separate the layers to reflect the role of thin packages;
filter jib extensions is good job,but order is wrong,like this:
libs
classes
resources
new-layer-1
new-layer-2

if we only change some classes and resources.that change very small.but,new layers all need download again.that not good idea.
maybe,we can insert new layer  like this
libs
new-layer-1
new-layer-2
classes
resources

use jib extensions like this:
pluginExtensions {
            pluginExtension {
                implementation = 'com.google.cloud.tools.jib.gradle.extension.layerfilter.JibLayerFilterExtension'
                configuration {
                    filters {
                        filter {
                            glob = '/app/libs/google-*.jar'
                            toLayer = 'google dependencies'
                            after/before = "xxxx"
                        }
                    }
                }
            }
        }

Hi @SamPeng87,
Thanks for the feedback.

filter jib extensions is good job,but order is wrong,like this:
libs
classes
resources
new-layer-1
new-layer-2

if we only change some classes and resources.that change very small.but,new layers all need download again.that not good idea.

I'd like point out that this is not true. The order of layers doesn't matter. The order matters only when using Dockerfile and the Docker CLI to create and cache layers. With Jib, changing classes and resources will only rebuild the "classes" and "resources" layers. Also you'll only pull and push the "classes" and "resources" layers from/to registries.

Hi @chanseokoh
thanks for your reply,I research docker pull workflow again,you are right.

@chanseokoh , @SamPeng87  I have a similar situation and in my case "docker pull" seems always to download the layers "after" the changed one again.  So, in the example above, I changed resources, and docker pull did download new-layer-1 and new-layer-2.
When looking into the image with help of https://github.com/wagoodman/dive,  I see that the digests for new-layer-1 and new-layer-2 are still the same, but their IDs have changed.
For resources layer, digest and ID hs changed (of course).
I have no idea what is going wrong, but it seems, docker pull looks at the changed IDs and not on the stable digests in my case.
Docker Version: 20.10.21  (and Docker Desktop on Windows)

I am having an issue with this too. A good chunk of OCI/Docker compatible container stores rely on previous hash just as much.


Add Socrata distribution README

A quick README for distributing to Socrata. @fvankrieken @sf-dcp @damonmcc let me know what information would be useful in order to get started automating socrata distributions.
Also, if you have any suggestions for formatting, etc. I like the idea of having README's at relevant levels in dcpy, then perhaps we can link them together from a top-level README

Do you have an example revision that you've pushed, but not published? Just to see more practically how to interact with it in socrata

Do you have an example revision that you've pushed, but not published? Just to see more practically how to interact with it in socrata

@fvankrieken Let me add an example of that whole flow


Storage swagger

There are two commits in this PR that might be easier to review individually.
The first changes to the official versions of the Storage swagger and adds README.md transformations address any deltas.  I've regenerated which resulted in some code changes, but I think that's all generator updates.  Please review this carefully to make sure all the changes are acceptable.
The second commit updates blobs to support STG69 features.

Can one of the admins verify this patch?

I would prefer merging all the changes to the preview3 branch.
https://github.com/Azure/azure-sdk-for-python/tree/storage-preview3

I was pretty sure I was missing a parameter or two.  😄   Thanks @annatisch

All good - for some reason all the XML context for serialization/deserialization has been stripped out of the models and I can't figure out why....
So I my add a few more tweaks once I figure it out!

Try adding enable-xml: true as a setting?  Java uses that.

omg - yup that was it! Thanks!

To fix the analyze step, the following "known content issues" need to be add at https://github.com/Azure/azure-sdk-for-python/blob/master/.docsettings.yml#L19
  - ['/sdk/storage/azure-storage-file/swagger/README.md', '#4554']
  - ['/sdk/storage/azure-storage-queue/swagger/README.md', '#4554']
  - ['/sdk/storage/azure-storage-blob/swagger/README.md', '#4554']


Missing css classes

Hi
The inlined template is missing the following css class:
preheader, wrapper

Hi @rabol
The preheader and wrapper styles are inlined. The wrapper has embedded responsive styles. Could you point out what exactly you're not seeing that you expected?
Here are the lines:

wrapper CSS https://github.com/leemunroe/responsive-html-email-template/blob/master/email-inlined.html#L25
wrapper HTML https://github.com/leemunroe/responsive-html-email-template/blob/master/email-inlined.html#L103
preheader HTML https://github.com/leemunroe/responsive-html-email-template/blob/master/email-inlined.html#L98


Update README.md file of TFLite Android Example

Please check. @perfinion

@kyscg please check now.

@arp95 could you please resolve the conflicts? Thanks!

@arp95 Did you get a chance to check build failures? Please let us know on the update. Thanks!

I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen!


Unexpected time range behavior when querying data [2.7.1 OSS]

Steps to reproduce:

Create a new bucket named "test"

Write specific data to bucket on specific date (it could be any day) (2023-09-01). (line protocol, presicion: seconds):
admin,config=configTest,type=testingType,farm=testingFarm,shed=testingShed testingType=20000<PHONE_NUMBER>

Write new some data with more recent date (This case 2023-10-10) with a .flux file query (influx-cli or notebook)
import "array"
import "timezone"

option location = timezone.fixed(offset: -5h)

array.from(
    rows: [
        {
            _time: 2023-10-10T00:00:00Z,
            _measurement: "admin",
            _field: "testingType",
            _value: 0,
            config: "configTest",
            type: "testingType",
            farm: "testingFarm",
            shed: "testingShed"
        },
    ],
) |>to(bucket: "test")

Query data with time range started at 2023-09-01 and ends at 2023-10-11:
from(bucket: "test")
  |> range(start: 2023-09-01, stop: 2023-10-11)
  |> filter(fn: (r) => r["_measurement"] == "admin")
  |> filter(fn: (r) => r["_field"] == "testingType")

Expected behaviour:
The query should return to me a series with 2 points at 2023-09-01 with _value: 20000, and another at 2023-10-10 with _value: 0 like:

_time
_measurement
_field
_value
config
type
farm
shed

2023-09-01 00:00:00 UTC
admin
testingType
20000
configTest
testingType
testingFarm
testingShed

2023-10-10 00:00:00 UTC
admin
testingType
0
configTest
testingType
testingFarm
testingShed

Actual behaviour:
Only returns to me the first point:

_time
_measurement
_field
_value
config
type
farm
shed

2023-09-01 00:00:00 UTC
admin
testingType
20000
configTest
testingType
testingFarm
testingShed

Unexpected behaviour:

If modify the flux query adding |> last() at end returns to me the second point (actually the last point):
from(bucket: "test")
  |> range(start: 2023-09-01, stop: 2023-10-11)
  |> filter(fn: (r) => r["_measurement"] == "admin")
  |> filter(fn: (r) => r["_field"] == "testingType")
  |> last()

_time
_measurement
_field
_value
config
type
farm
shed

2023-10-10 00:00:00 UTC
admin
testingType
0
configTest
testingType
testingFarm
testingShed

If modify the range start to 2023-09-02 returns nothing (It should return point at 2023-10-10):
from(bucket: "test")
  |> range(start: 2023-09-02, stop: 2023-10-11)
  |> filter(fn: (r) => r["_measurement"] == "admin")
  |> filter(fn: (r) => r["_field"] == "testingType")

_time
_measurement
_field
_value
config
type
farm
shed

If modify the range start to 2023-09-04 (and any recent date after that day) returns the point at 2023-10-10:
from(bucket: "test")
  |> range(start: 2023-09-04, stop: 2023-10-11)
  |> filter(fn: (r) => r["_measurement"] == "admin")
  |> filter(fn: (r) => r["_field"] == "testingType")

_time
_measurement
_field
_value
config
type
farm
shed

2023-10-10 00:00:00 UTC
admin
testingType
0
configTest
testingType
testingFarm
testingShed

Environment info:
InfluxDB 2.7.1 OSS running on windows 11 x86_64  VM  (8 vcores 8 GB RAM).
InfluxDB 2.7.1 OSS running on Ubuntu 20.04LTS VM (8 vcores 8 GB RAM).
Logs:
If you need logs, how can I generate them in Windows?
Notebook Result:

I have a similar issue w.r.t. relative time ranges using relative time ranges.
Observed behavior:

Expected behavior (from Grafana):

=>

It even respects the timeshift from CEST to CET happened two weeks ago :sunglasses:
Similar error occurs using -1y
Observed behavior:

Expected behavior (from Grafana):

When I use timeshifts smaller than 1mo, it kinda works except that InfluxDB still does not respect the timeshift correctly like Grafana does.
The following query also works, yet still not respecting the timeshift:

I'm not sure if this is the same or a different issue. Let me know if I should create a separate one.

@friebi I thing is different behaviour, but i thing that is related to timezone. I dont't sure, but Grafana time range windows respect your local timezone (you can check administrator/user/organization configurtation -> timezone). Otherwise influx always use UTC timezone as default

timezone.utc is the default location with a completely linear clock and no offset. It is used as the default for location-related options.

To fix this you can explicit especify the timezone query with timezone package from flux with fixed or location
Something like this should be enought:
import "timezone"

option location = timezone.location(name: "TZ_Identifier like America/LosAngeles")

from(bucket: "power_meter")
   ...

Read the timezone documentation from flux and TZ Indetifier of your country.
Let me know if this solve your problem :)

Thanks @daguirrem for your reply. Indeed setting the timezone to my local one fixes the second, but smaller issue.
So using -1mo it now looks like this:

So still a faulty offset of 14 hours.
Using date.sub(d: 1mo, from: now()) it is now fully correct:

The error apparent to be an datatype conflict, If change the step 2 of "steps to reproduce" to write an int (long) like:
admin,config=configTest,type=testingType,farm=testingFarm,shed=testingShed testingType=20000i<PHONE_NUMBER>
The error is solved.
Also if keep the step 2 to write a float, and change step 3 to explicit write a float the error also is solved.


FS-1773: Pull language from accept header regardless of casing

In scenarios where the users Accept-Language header is in a different casing, locale sniffing won't work.
This PR should address that by searching for a header that matches regardless of casing.

red herring


New Feature: allow SDK to perform Agent direct request

Example:
    lola.request_chat_completion("123", "Hello there!")

Where "123" is any user id available. And "Hello there!" is input text for the Agent.
The Agent response is async, and it's capture using an event decorator such as:
@lola.on_event('onAgentResponse')
def handle_new_conversation(session, ctx: LolaContext, msg):
    print(f'Got Agent Response message!!!!!')
    print(msg)
    ctx.messanger.send_text_message('>> Agent: ' + msg, blend=False, appendToHistory=False)


Merge Pester tests back into Debug-ContainerHost.ps1

Per feedback from #414 - it was hard to download all of the files needed to run the tests without cloning the full repo. I looked into various solutions and agree with @StefanScherer and @taylorb-microsoft that it's probably easiest to just publish a single file.
This merges everything back into one file, and doesn't change any code.

Ping @jterry75 - any concerns?

This makes it super easy to get the latest version of that check. Just tried it in Azure:

@StefanScherer Thanks! Did those errors cause any problems? I don't recall seeing the Resolver... error before.

@PatrickLang After creating the Azure VM and RDPing into it, a docker version only showed the client version. The docker deamon was not running. I just ran start-service docker and everything is fine since then, even after a reboot.

Anything you want to investigate?

@StefanScherer I see it too. I'll take that up with the right team.
PS C:\Users\patrick> wget https://raw.githubusercontent.com/Microsoft/Virtualization-Documentation/5429b3fa25a95cac7afb1
a52cabc5a6d7eef757a/windows-server-container-tools/Debug-ContainerHost/Debug-ContainerHost.ps1 -OutFile Debug-ContainerH
ost.ps1
PS C:\Users\patrick> .\Debug-ContainerHost.ps1
Checking for common problems
Describing Windows Version and Prerequisites
 [+] Is Windows 10 Anniversary Update or Windows Server 2016 818ms
 [+] Has KB3192366, KB3194496, or later installed if running Windows build 14393 206ms
 [+] Is not a build with blocking issues 33ms
Describing Docker is installed
 [+] A Docker service is installed - 'Docker' or 'com.Docker.Service'  107ms
 [-] Service is running 206ms
   Expected: {Running}
   But was:  {Stopped}
   28:         $service.Status | Should Be Running
   at <ScriptBlock>, C:\Users\patrick\Debug-ContainerHost.ps1: line 28
 [+] Docker.exe is in path 1.44s
Describing User has permissions to use Docker daemon
 [+] docker.exe should not return access denied 59ms
Describing Windows container settings are correct
 [+] Do not have DisableVSmbOplock set 54ms
 [+] Do not have zz values set 35ms
Describing The right container base images are installed
error during connect: Get http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.25/images/json: open //./pipe/docker_engine: The syst
em cannot find the file specified.
 [-] At least one of 'microsoft/windowsservercore' or 'microsoft/nanoserver' should be installed 179ms
   ValidationMetadataException: The argument is null or empty. Provide an argument that is not null or empty, and then t
ry the command again.
   ParameterBindingValidationException: Cannot validate argument on parameter 'Property'. The argument is null or empty.
 Provide an argument that is not null or empty, and then try the command again.
   at <ScriptBlock>, C:\Users\patrick\Debug-ContainerHost.ps1: line 76
Warnings & errors from the last 24 hours

   ProviderName: docker

TimeCreated                     Id LevelDisplayName Message
-----------                     -- ---------------- -------
10/12/2016 8:57:05 PM            4 Error            fatal: Error starting daemon: Error initializing network control...

Talked to Justin, he thinks it seems reasonable


python38Packages.google-cloud-iot: 2.1.0 -> 2.2.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.google-cloud-iot is: "Cloud IoT API API client library"
meta.homepage for python38Packages.google-cloud-iot is: "https://github.com/googleapis/python-iot"
meta.changelog for python38Packages.google-cloud-iot is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

/nix/store/zlk86b0gc333ijc326xsxgh2zakva3i3-python3.8-google-cloud-iot-2.2.1/bin/fixup_iot_v1_keywords.py passed the binary check.

/nix/store/zlk86b0gc333ijc326xsxgh2zakva3i3-python3.8-google-cloud-iot-2.2.1/bin/.fixup_iot_v1_keywords.py-wrapped passed the binary check.

2 of 2 passed binary check by having a zero exit code.

0 of 2 passed binary check by having the new version present in output.

found 2.2.1 with grep in /nix/store/zlk86b0gc333ijc326xsxgh2zakva3i3-python3.8-google-cloud-iot-2.2.1

found 2.2.1 in filename of file in /nix/store/zlk86b0gc333ijc326xsxgh2zakva3i3-python3.8-google-cloud-iot-2.2.1

directory tree listing: https://gist.github.com/81ff1bf13c06d7de06eba3712141b396

du listing: https://gist.github.com/abe2f23b6045d2eed8cd085dd9b34d0b

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

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.google-cloud-iot
python39Packages.google-cloud-iot

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zlk86b0gc333ijc326xsxgh2zakva3i3-python3.8-google-cloud-iot-2.2.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zlk86b0gc333ijc326xsxgh2zakva3i3-python3.8-google-cloud-iot-2.2.1
ls -la /nix/store/zlk86b0gc333ijc326xsxgh2zakva3i3-python3.8-google-cloud-iot-2.2.1/bin

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

2 packages built:

python38Packages.google-cloud-iot
python39Packages.google-cloud-iot

Maintainer pings
cc @SuperSandro2000 for testing.

Result of nixpkgs-review pr 133822 at 2368e780 run on aarch64-linux 1

2 packages built successfully:

python38Packages.google-cloud-iot
python39Packages.google-cloud-iot


Map server config in docker run command

Either
a) map the entire target's config dir to the container, or
b) map all of the individual files from the local target's config dir to the container's config dir.
Prefer (b) in case the user's Dockerfile has copied some content to the config dir which is not part of the Maven/Gradle build, so we don't want to remove those content.

Based on the UFO meeting on May 12 and the follow up discussion, it was decided it may be best to go with option (a) and override the entire container config directory. Need to follow up with Sebastian to understand potential cons with this method.

If no Dockerfile is provided, merge the project's config files with those in the container so that any files from the base image will still be there.  This will be covered by #817
If a Dockerfile is provided, do not include the project's config files in container.  The user should configure their Dockerfile to copy over whatever is needed into their image.


[datadog] Bump kube-state-metrics version

What this PR does / why we need it:
Bump kube-state-metrics version in Datadog chart.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped

/lgtm


Error getting when try to create entity in front end

You don't seem to have a generator with the name “jhipster-ionic:entity” installed.

Our app is 1.5 years old so we are creating firstly entity in microservice and then in frontend.
Tried this yo jhipster-ionic entity TestScreen
But getting another issue now
D:\Project_Upgrade\kubs-client>yo jhipster-ionic entity TestScreen
Error jhipster-ionic entity TestScreen
This is a JHipster blueprint and should be used only like jhipster --blueprints ionic
D:\Project_Upgrade\kubs-client>

Is your Ionic app 1.5 years old or your JHipster app? If it's Ionic, you shouldn't use the latest version of this blueprint to generate entities. You should use the Ionic for JHipster generator that you created your app with.

Closing due to lack of response. Please re-open and provide steps to reproduce if you're still having an issue.


Fix Travis CI builds

Locking grunt-jscs to the last version that considers current Gruntfile.js style valid, which is 1.2.0.

+1 this needs to be merged


Feature Request (new-data event)

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux

Shell

[x] zsh

Problem / Steps to reproduce
Just a request to add a new event to trigger an action when additional data comes in from a stream.
Thanks!

What are you planning to do with it?

In my particular use case I'm using fzf as a log filter, so data is being piped in from a tail, but the new data is being shown at the bottom of the list.  I'm not sure if it is re-applying the filter on new data received, but is not scrolling to the bottom.  I have bound the change event to scroll to the bottom so it just takes a keypress to see the new data.
Here are the options I'm using:
FZF_OPTS="--no-sort --height=100% --exact --ansi --no-expect --tabstop=4 --layout=reverse-list --bind 'change:last' --bind 'esc:clear-query+last' --bind 'enter:last'";

Have you considered --tac?

I've tried that, but then it reverses the log output, causing it to go forward in time up the screen.

what about piping in through head with the tac optoon or genuinely piping through the tac binary. before fzf handles it?

or swotching your bindings to first and your layout to defailt or.. this last one is a little wild you can use coproc (woild help to know specifically the commands your using do you need a purley shell solution?

This is a streaming log; how would tac handle it properly?
All I'm really trying to do is effectively let fzf be a searchable tail command.  So newest events at the bottom, input on the bottom, filterable.
When new events come in, it doesn't scroll down to the end of the stream.  (my workaround right now is any key will scroll to the bottom)

for i in {1..1000}; do echo $i; sleep 0.1; done | fzf --tac

I see now, I had not played around with trying to set --layout with --tac.  I believe this will work fine:
for i in {1..1000}; do echo $i; sleep 0.1; done | fzf --tac --layout=default

Just to follow-up on this for future forensic purposes.
I did get my original use to work for it with --tac --layout=default  I also had to clear out my bindings because last in --tac logically means the top entry.
The final command line that works as needed is:
fzf --tac --layout=default --height=100% --no-sort --no-expect --exact --ansi --tabstop=4 --bind 'enter:ignore' --bind 'esc:cancel';

There is still some buffering going on with what I'm piping into fzf, but given the for i in line I can see it's not fzf.  Thanks for sticking with this despite my misunderstanding.


deprecate onnx runtime

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

are there docs on why this is being deprecated?

#1792

Thanks for the PR, generally it's fine.

We are not actually deprecating onnx model support,  it is the model server onnxruntime server which is deprecated. Once https://github.com/kserve/kserve/pull/1901 is merged, we should switch to use triton inference server to serve the onnx model.


Add Article for Hello World in Nim

Code exists #67! Just need to article it up.

Added link to README: https://github.com/jrg94/sample-programs/commit/00235ee6841f2f3e71ac8f1a735cff03378009e7


fix: should update node's incomings and outgoings when edge was removed

Description

Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Enhancement (changes that improvement of current feature or performance)
[ ] Refactoring (changes that neither fixes a bug nor adds a feature)
[ ] Test Case (changes that add missing tests or correct existing tests)
[ ] Code style optimization (changes that do not affect the meaning of the code)
[ ] Docs (changes that only update documentation)
[ ] Chore (changes that don't modify src or test files)

Self Check before Merge

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Pull Request Test Coverage Report for Build 266633705

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build 266436720:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls

这个 x6-bot 怎么搞的，有点可爱。

这个 x6-bot 怎么搞的，有点可爱。

整了个自定义机器人，将 actions 中要用到 token 的地方使用机器人的 token。


[modify] disallow edit ignore syntax check

coverage: 83.118% (+0.008%) from 83.11% when pulling ce580d705285d292abf95b73e7a6cf19382ec192 on pr-syntax-checker into 7ca700966a61249896a191eecbd0a61cd383d6d1 on master.


UI in snapshot is broken

Issue Description
When creating UI, it should be available, and
Deeplearning4j UI server started at: http://localhost:9000
should be in the log.
In current snapshot, UI is not available, and log entry is not generated.
Version Information
Current snapshot.
Additional Information
2020-02-22 10:51:41.190 - SEVERE: The FreeMarker version requested by "incompatibleImprovements" was 2.3.29, but the installed FreeMarker version is only 2.3.23. You may need to upgrade FreeMarker in your project. java.lang.IllegalArgumentException: The FreeMarker version requested by "incompatibleImprovements" was 2.3.29, but the installed FreeMarker version is only 2.3.23. You may need to upgrade FreeMarker in your project.         at freemarker.template._TemplateAPI.checkVersionNotNullAndSupported(_TemplateAPI.java:49)         at freemarker.core.Configurable.<init>(Configurable.java:309)         at freemarker.template.Configuration.<init>(Configuration.java:634)         at org.deeplearning4j.ui.module.train.TrainModule.<init>(TrainModule.java:140)         at org.deeplearning4j.ui.VertxUIServer.start(VertxUIServer.java:197)         at io.vertx.core.AbstractVerticle.start(AbstractVerticle.java:106)         at io.vertx.core.Verticle.start(Verticle.java:66)         at io.vertx.core.impl.DeploymentManager.lambda$doDeploy$8(DeploymentManager.java:556)         at io.vertx.core.impl.ContextImpl.executeTask(ContextImpl.java:369)         at io.vertx.core.impl.EventLoopContext.lambda$executeAsync$0(EventLoopContext.java:38)         at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:163)         at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:510)         at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:518)         at io.netty.util.concurrent.SingleThreadEventExecutor$6.run(SingleThreadEventExecutor.java:1044)         at io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74)         at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)         at java.base/java.lang.Thread.run(Thread.java:834)
Forcing FreeMarker version to 2.3.29 in application pom.xml seem to be workaround.

You are probably affected by a variant of https://github.com/eclipse/deeplearning4j/issues/8724
please share your pom.xml file, and the output of mvn dependency:tree so we can help you find a workaround.

I have already workaround for this, so no need to find one.
I think problem is already visible in conflicting pom definitions; the 2 poms below should specify same version.
 deeplearning4j/deeplearning4j/deeplearning4j-ui-parent/deeplearning4j-ui-components/pom.xml  <properties>    <freemarker.version>2.3.23</freemarker.version> </properties>
and
deeplearning4j/deeplearning4j-ui-parent/deeplearning4j-vertx/pom.xml <dependency>   <groupId>org.freemarker</groupId>   <artifactId>freemarker</artifactId>   <version>2.3.29</version> </dependency>

For future reference - this issue would also be fixed by resolving this issue, currently blocked however - https://github.com/eclipse/deeplearning4j/issues/8750#issuecomment-604244356