"cargo install" doesn't read the "rustflags" setting in .cargo/config

Problem

Cargo install command doesn't read the rustflags setting in .cargo/config
Steps

In order to fix a linker issue on macOS (https://github.com/alexcrichton/git2-rs/issues/180), i have to use something like: RUSTFLAGS='-L /usr/lib' cargo build.
And I put this flag inside {project-folder}/.cargo/config file:
[build]
rustflags = ["-L", "/usr/lib"]

This works for cargo build, but not cargo install
It looks like cargo install ignores that setting.
I have to use RUSTFLAGS='-L /usr/lib' cargo install --path . to make it work again.
i also tried something like:
[targer.x86_64-apple-darwin]
rustflags = ["-L", "/usr/lib"]

No luck either.
Notes
Output of cargo version:
$ cargo version
cargo 1.31.0 (339d9f9c8 2018-11-16)

Operating System: macOS High Sierra, 10.13.6
I have edition = "2018" in Cargo.toml

As suggested in https://github.com/rust-lang/cargo/pull/6026#issuecomment-421519652 cargo install --path . should read .cargo/config in my, and Mark's, opinion.

Thanks for the pointer @Eh2406 .
Yes, I agree with @dwijnand  and @Mark-Simulacrum , that there should be a way to provide "local configuration"
Just created a feature request (#6498) to keep it tracked.

Yes @dwijnand , closing this one.


NXP: Build error : ModuleNotFoundError: No module named 'elftools'

Hello,
I am trying to build sample application on for mimxrt1064_evk on windows 10. I seem to have installed everything, but i still get this error:
C:\Users\mbajaj\zephyrproject\zephyr>west build -p auto -b mimxrt1064_evk samples/hello_world
[1/130] Generating include/generated/kobj-types-enum.h, include/generated/otype-to-str.h, include/generated/otype-to-size.h
FAILED: zephyr/include/generated/kobj-types-enum.h zephyr/include/generated/otype-to-str.h zephyr/include/generated/otype-to-size.h
cmd.exe /C "cd /D C:\Users\mbajaj\zephyrproject\zephyr\build\zephyr && C:\Python36\python.exe C:/Users/mbajaj/zephyrproject/zephyr/scripts/gen_kobject_list.py --kobj-types-output C:/Users/mbajaj/zephyrproject/zephyr/build/zephyr/include/generated/kobj-types-enum.h --kobj-otype-output C:/Users/mbajaj/zephyrproject/zephyr/build/zephyr/include/generated/otype-to-str.h --kobj-size-output C:/Users/mbajaj/zephyrproject/zephyr/build/zephyr/include/generated/otype-to-size.h --include C:/Users/mbajaj/zephyrproject/zephyr/build/zephyr/misc/generated/struct_tags.json "
Traceback (most recent call last):
File "C:/Users/mbajaj/zephyrproject/zephyr/scripts/gen_kobject_list.py", line 62, in
import elftools
ModuleNotFoundError: No module named 'elftools'
[2/130] Generating include/generated/driver-validation.h
FAILED: zephyr/include/generated/driver-validation.h
cmd.exe /C "cd /D C:\Users\mbajaj\zephyrproject\zephyr\build\zephyr && C:\Python36\python.exe C:/Users/mbajaj/zephyrproject/zephyr/scripts/gen_kobject_list.py --validation-output C:/Users/mbajaj/zephyrproject/zephyr/build/zephyr/include/generated/driver-validation.h --include C:/Users/mbajaj/zephyrproject/zephyr/build/zephyr/misc/generated/struct_tags.json "
Traceback (most recent call last):
File "C:/Users/mbajaj/zephyrproject/zephyr/scripts/gen_kobject_list.py", line 62, in
import elftools
ModuleNotFoundError: No module named 'elftools'
ninja: build stopped: subcommand failed.
FATAL ERROR: command exited with status 1: 'C:\Program Files\CMake\bin\cmake.EXE' --build 'C:\Users\mbajaj\zephyrproject\zephyr\build'
And elftools is installed as verified by:
pip show pyelftools
Name: pyelftools
Version: 0.27
Summary: Library for analyzing ELF files and DWARF debugging information
Home-page: https://github.com/eliben/pyelftools
Author: Eli Bendersky
Author-email<EMAIL_ADDRESS>License: Public domain
Location: c:\python39\lib\site-packages
Requires:
Required-by: pyocd

You need to pip3 install pyelftools or, even better, pip3 install zephyr/scripts/requirements.txt

Reopening since I missed some of the input

I tried this locally and works fine on a Windows 10 machine. It might be that you have multiple Python 3 installations and CMake is picking one where you don't have pyelftools installed.


Add a formatting script for Travis CI

Please check if the PR fulfills these requirements:

[x] The commit message follows our guidelines.
For bug fixes and features:

[ ] You tested the changes.
[ ] You updated the docs or changelog.

Related issue (if applicable): #

What kind of change does this PR introduce?
This is a simple script which automatically checks and corrects file formatting to comply with POSIX standards. This repo already has all files complying with this script, but I noticed that there were some issues with files missing newlines in the feature/code-rewrite branch, and if this script was present, those issues could've been avoided.
To run on Travis, this repo needs to be added on the Travis CI website. The formatting script can also be run locally to fix formatting.
Starting with Godot 3.2, Godot will automatically ensure files end in newlines with its internal editor, but this script can help with fixing up old files, and ensuring this is done with all text files.
Does this PR introduce a breaking change?
No.

Thanks but we already use gdformat to do that. The branch in question is just a save of razvan's work until we have time to merge it, it hasn't been formatted yet. The issue will be fix once we format the branch's code


update doc

Pull Request Test Coverage Report for Build 1474

0 of 0   changed or added relevant lines in 0 files are covered.
13 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.2%) to 63.88%

Files with Coverage Reduction
New Missed Lines
%

core/rest_invoker.go
2
91.84%

metrics/metric.go
3
59.26%

tracing/zipkin/collector.go
3
72.73%

core/router/cse/translate.go
5
33.8%

Totals

Change from base Build 1473:
0.2%

Covered Lines:
4959

Relevant Lines:
7763

💛  - Coveralls


⚠ Bofrost.com has degraded performance

In 358e397, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 425 ms

Resolved: Bofrost.com performance has improved in 6431e3b after 14 hours, 2 minutes.


clamp the tooltip position on the x axes and flip it on the y axes if needed

This isn't ready for review is it? The Y flipping doesn't work, and there's no padding when clamping to screen (looks pretty bad)

super-seeded by https://github.com/ppy/osu-framework/pull/785


Axios Template generate Typescript Client with error

When using NSwagStudio (v<IP_ADDRESS>) to generate a TypeScript client with template set to Axios and Dto Type Style set to Interface, it generates the api with an error.

If I switch the  template to Fetch it generates it correctly. So how can I resolve this issue (I tried to google about it and search in the issues but found nothing that might help me).

Fixed with v13.2.5. Please try to use the latest version


myjobs server : doesn't change state of running job from new to running

I create job which it function  while  loop job which wait for 100 sec ,
at the beginning the state of process was new then ok , and never converted to running although it was running for 100 sec.
test.py file
import time
def just1():
    count = 0
    while(1):
        time.sleep(20)
        count=count+1
        if count == 5:
            print(count)
            break

j.servers.myjobs.schedule(test.just1)
in the beginning and through 100 sec

after 100 sec

Did you start any worker?

Yes four workers

i can't reproduce this issue

Never changed to running.. just went from new to ok

closing this issue because behavior is updated


uri related code cleanup

I am working to clean up and pull together code belonging to UUri, specifically:

make issues during serialization visible to the user - use TryFrom and Result<> instead of empty default objects and Option<>
move the serialization/deserialization code directly into the TryFrom implementations, this gets rid of all the *serializer stuff while making the whole thing more rust-like
general little cleanups and simplifications

This work will base on #18, so depends on that being merged.

I found an issue with ambiguity in the usage of id vs up when we're serializing / desrializing to and from Micro URI form.
So #18 is waiting on this up-core-api #102 change to get merged, so that the validation logic and I guess serialization and deserialization logic in general can be reflected.
So I guess #18 may do a bit of what you're suggesting, but very lightly. Just more or less trying to get the Micro URI validation in. Lemme know if you have any concerns on overlap over on that PR when I can update it.

LongUri cleanup done via #49  -  Will start on MicroUri soon

Hi @AnotherDaniel
@PLeVasseur and I discussed about the serialization of micro Uri here.
We think it would be good if up-rust can support the API to serialize UAuthority alone.
The requirement is that we need to extract UAuthority to generate the Zenoh key for the uStreamer scenario.
Do you think it can be also included in your cleanup? Thank you!

@evshary sure! Will include that - only for micro serialization, correct?

@evshary sure! Will include that - only for micro serialization, correct?

Yes, I think so.

Microuri cleanup here: #54

#54 merged, so this is done


Remove "Waiting for .. to become available.." and explain it briefly

remove the message from console output
mention that it could occur and briefly note down what it means

related to the discussion at #1753

I'm not quite sure if the result for calling is_action_server_available is the same as for calling service_server_is_available because the inconsistency would then still be there.
Both call rcl_service_server_is_available for all services but rcl_action_server_is_available also checks the number of publishers while service_server_is_available doesn't. I assume that the number_of_publishers will be zero if no node interacts with the action server.

I assume that the number_of_publishers will be zero if no node interacts with the action server.

I think in this case feedback_subscription and status_subscription are subscriptions on the action client's side, i.e. the publishers are on the action server's side. Checking if there are publishers for those topics is part of checking if there is an available action server.

In this case, I agree with @christophebedard that if you follow the tutorial as it is written, you won't get the "Waiting for service" message.  Also, we always get the "Waiting for an action server to become available" right now, so we shouldn't get rid of that.
Overall, I think we don't need this, so I'm going to close this out.  Feel free to reopen if you disagree.


Production build takes average 12 seconds to load App.

I created a build with npm run build and deployed the static files on the production server.

Here we see the size of the main.js os  around 270K Gzipped. but when I see in Network tab its takes much time to load.

Any ideas on how to get around with this.
I already got rid of the sourcemaps.

I already got rid of the sourcemaps.

Here we see the size of the main.js os around 270K Gzipped. but when I see in Network tab its takes much time to load.

It looks like your server isn't configured to do gzip then? I don't know what you use for hosting but I encourage you to consult the appropriate resources for enabling compression on your server.
You can also use code splitting to further reduce the size of the initial bundle.
But enabling gzip is the first thing you should do.

Thank you so much Dan. That was my initial guess. .. :) I will the Server guys to look into that .


Pixart Homework

Level of Completeness 3
Level of Comfort 3

I didn't follow the commits in the instructions. I was figuring a few things out as I went along.

Good job getting the input to change the background color. There's a lot more to this exercise and I encourage you to finish it when you get a chance. That practice will be extremely important getting comfortable with DOM manipulation.


Part extracts that include backbones

In BP011, @GC-repeat noticed that our terminology is currently awkward with respect to a digestion product that includes a backbone. Although it is technically covered by the "part extract" term, that feels wrong from the perspective of terminology.
We therefore propose to expand the terminology of digestions products to include distinctions with respect to their contents as follows:

If there is a SubComponent that includes an engineered_insert, but no SubComponent that includes a backbone, then it is a "part digest"
If there is a SubComponent that includes a backbone, but no SubComponent that includes an engineered_insert, then it is an "opened backbone"
If there is both a backbone and an engineered_insert, then it is an "opened part in backbone"

This is backward compatible with the current BP011, which is underspecified for "opened backbone" and "opened part in backbone"
Examples:

A BioBricks digest with XbaI and EcoRI produces an opened part-in-backbone that is ready to ligate a new part into on the 5' side of the current part
A digest on a MoClo Level 1 destination vector (backbone) produces an opened backbone ready to ligate with a set of parts to produce a Level 1 part-in-backbone

Hi, I agree with the idea in general.
I used the term open_backbone in the implementation of it so it makes sense to save that information.
I reached almost the same term for  "opened backbone" , but "opened part in backbone" dont feel right to me. The name is too long and is less intuitive that  "opened backbone".
I would like to suggest the use of "sticky part" and "sticky backbone" given that both have sticky ends when opened or digested and they communicate that they are to ligate to other parts. In thi way the name will include information of their state (sticky) rather than an action that happened to them (opened).
Also, I would consider the use of "sticky part" and "open/ed backbone".

I don't have a strong preference between "open backbone" and "opened backbone."
I would prefer not to use the "sticky" terms though, because:

that would replace "part extract", which got a lot of support,
that's referring only to the ends, and
some methods (though not our current targets) might turn out to involve blunt cuts

Per discussion between myself and @Gonza10V, the proposal is to change to "open" from "opened" (matching the case of "extract" as opposed to "extracted"), and to keep the distinction of "open backbone" and "open part in backbone" due to the representational distinction.
Bottom line:

If there is a SubComponent that includes an engineered_insert, but no SubComponent that includes a backbone, then it is a "part digest"
If there is a SubComponent that includes a backbone, but no SubComponent that includes an engineered_insert, then it is an "open backbone"
If there is both a backbone and an engineered_insert, then it is an "open part in backbone"


Add more helpers

This project uses semver, please check the scope of this pr:

[ ] #none# - documentation fixes and/or test additions
[ ] #patch# - backwards-compatible bug fix
[x] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Added more test setup helpers to support other common modules that utilizes the factory (services, adapters, helpers)

👍


Week days color.

Need to add parameter for weekdays. and also for header full name.

Thanks. It will be in coming updates.


chore: Move existing technology resources from aggregate file to separate markdown files

Overview
Problem
I'm modernizing this repository and setting it up for future additions of resources. We are currently constrained by a single large technology resources file in markdown which makes it very hard to parse through what resources you want to read.
Solution
Divide up the content in the technology-resources.md file into separate content files linked to a table of contents in the root README.md.


Server error (500) on "Upload Your Own Recipe" button

"Leaderboard" button in the home page return Sever Error (500)

To reproduce the problem: Click on "Leaderboard" button in the home page

Thank you for your report. I apologize for it not working correctly at first. Now that we have resolved it.


Fixed compilation when using Clang on Windows

Clang on Windows defines _MSC_VER so using it for the definition of CLAY_PACKED_ENUM makes it use __pragma with Clang, which it cannot understand. This PR fixes it by replacing #ifdef _MSC_VER with #if defined(_MSC_VER) && !defined(__clang__).

Thank you for this, nice catch! I don't use Clang on windows so I haven't tested this myself, but we can always fix it again if it's not quite right 🙂


[Help] Some problems about customizing my own dataset for training/testing GCA model

I really appreciate this interesting work, which nicely bridges up two seemingly different areas. I'm currently interested in applying the GCA (not cGCA) model to a new dataset (actually, it's ShapeNet with more categories). It would be great if you can give me some help regarding how to custom a new dataset. I've tried to read the source codes of TransitionShapenetDataset, but I still feel a bit confused.

It seems that the dataset requires embeddings to train the GCA model, and the read context is then used to output embedding_coord, embedding_feat, input_pc, state_coord, respectively. What are the default values for embedding['coord'], embedding['feat'] and embedding['translation']? I think it's important because in the training script, it takes the embedding information as input, and therefore we have to give them some default values.
Besides, I think the current released version may still have some room for small improvements, including but not limited to, the actual dataset folder is named cgca_shapenet rather than shapenet_sdf; config file configs/gca-shapenet-vox=64.yaml provides a None value for the embedding_root: key, which certainly causes problem when trying to compute the self.embedding_root for TransitionShapenetDataset.
Many thanks!!

Thank you for the interest!
Regarding the features of GCA, we just use features of 1, so for the input with N x 3 coordinates size, the output of the DataLoader __getitem__() should have dictionary values of N x 1 (with all having values of 1) for key state_feat. Instead of using the if statement, which I have to change accordingly for all the datasets, I just use a python decorator change_feat() defined in link, which can be applied to all the datasets easily. The best way to check this is by creating a breakpoint in train.py, the data['state_feat'] will be a list of features having all values of 1.
For the config values of the embedding_root, I should change like as you said, soon. Thanks for the comments

Very appreciate for the detailed reply! May I have another question?
I also found that the following codes try to generate random sampled points that are within ball radius self.sphere_radius from center points sphere_centers, I would recommand using pytorch_cluster for speedup.
https://github.com/96lives/gca/blob/8571c4a54ef95d5192842dc0970d57c01516858b/datasets/transition_dataset.py#L229-L239
And it seems that the above codes are only for training a cGCA model (if we want to complete a point cloud), not for training a GCA model? If we want to change the dataset to train a GCA model, maybe we should remove the above codes and rewrite the codes which produces point_coord that is the partially observed point cloud, not the uniformly ball-query downsampled one?

I don't think there's a bottleneck in creating the partial pointcloud.
If you see below, there is a quantizing function after the removal. The code will work properly with GCA as well.

Yeah, I got your point that point cloud do need to be voxelized.
Since I'm interested in the completion task, and therefore wonder if for GCA's input, If I understand correctly, during training stage, it should be a partially observed cloud (shown below at the right) instead of the downsampled one (shown below at the middle)?

It seems that the current codes only implement the middle case, which is for cGCA only. So do I understand correctly, and need to change the codes to the right case?

Thanks very much!!
I think I've totally understand the shape and dtype, and I post the resuts here to help anybody who wants to build their own dataset.
input_pc ==> (1024, 3)             ==> partially observed pc
state_coord ==> (678, 3)                          ==> after quantized     int32,    voxelized partially observed voxel
state_feat ==> (678, 1) ==> all ones       ==> after quantized
embedding_coord ==> (5642, 3)  ==> int32, max(30), min(-27)  ==> complete voxel (served as GT)
embedding_feat ==> (5642, 32)


Add Rithm School to code_schools.yml

Feature
Why is this feature being added?

name: Rithm School
url: rithmschool.com?utm_source=OperationCode&utm_medium=logo&utm_campaign=root
logo: [SEE ATTACHMENT]
full_time: true
hardware_included: false
has_online: false
online_only: false
locations:
- va_accepted: false
address1: 3338 17th St.
address2: #100
city: San Francisco
state: CA
zip: 94110

I can do this.
Is there any opposition to alphabetizing the code_schools.yml file too?

@rickr Has this been taken? If not can I make a PR for this?


Management groups spanning from EA to PAYG?

The illustration in this page is confusing to me - how would a management group under an organization's EA tenancy include an Azure PAYG subscription? If it is possible, I'd love to have more details around the ILLO to exactly describe how the IT Management Group seems to span an impossible boundary as I understand it to this point. Thanks!

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

ID: 1527b80e-5270-b5f0-9034-747c6f92ae4e
Version Independent ID: 50b9882c-dcb0-ecc1-241b-1361ad2fff1e
Content: Organize your resources with Azure Management Groups | Microsoft Docs
Content Source: articles/azure-resource-manager/management-groups-overview.md
Service: azure-resource-manager
GitHub Login: @rthorn17
Microsoft Alias: rithorn

@fswgm Thank you for the valuable feedback,we are investigating the issue.

@fswgm Thanks for the feedback.  There are situations where some developers have their own subscriptions and then they are added to an Enterprise.  When the developer accepts the invite and is added to the Enrollment, the subscription is then pulled into the enterprise enrollment.  This is not ideal and is not recommended, but my goal in this diagram was to show it doesn't matter what type of subscription you might have.  Management Groups is a subscription type agnostic feature that does not limit what offer type you might have.
Sorry for the confusion.

Given that this not recommended, I am thinking there is no documentation change needed right now. If more people are confused by this diagram, we might want to add some clarification.
@fswgm - are you okay with closing the issue?

Thanks for asking - I get it, given clarity. It's fine to close.


Release/2.16.1

Set QUAY_EXPIRATION in build action for non release branches
Add quay.expires-after label to build# tagged images that were lacking it (but not static/base image tags, which get replaced)
Update OPM to v4.15.0
Update certification pipeline script to v1.0.131
Disable default provenance for docker buildx builds. The additional provenance registry data/manifest layer can't be labeled with quay.expires-after and breaks expiration
update goretryablehttp
fix test timing issue with recreation of marketplaceconfig
add OpenShift v4.16 support

need to fix catalogsource with new opm


Remove 'perspective' declarations from explorer CSS

Fixes #2549
I'm guessing these rules were in place to enable hardware acceleration, but removing them doesn't visibly affect the rendering. Either way, I figure that making Wagtail usable again on Firefox is more important than the performance boost...

Hm... I'm not sure that perspective is a reason. I can reproduce this bug on v1.4.3 tag, but can't reproduce on master (now contains perspective declarations).
Mac OS 10.11.4, FireFox 46.0.1.

Interesting. Applying of your patch on v1.4.3 solves the issue, but looks like in master this issue was solved in another way.

This changes fixes it for me on Firefox 46.0.1 & Firefox 48.0a2 ("developer edition") on OSX 10.11.3. Tested in Chrome  50.0.2661.94 & Safari 9.0.3 and there are no adverse effects there either.
The animations on the menu use transform and opacity properties which browsers are very good at optimising by default. We shouldnt need perspective for performance (will-change would be the preferred method these days if we did). I suspect this might have been left over from a more complex animation in the past (swinging out from the sidebar "in 3d" perhaps).

It seems that @alexgleason's fix in #2531 (796239b7d991322632da6f9f63fd1ccd172d3b58) fixed this too. (side note: using git bisect to find out when something fixed itself is really confusing, because you have to mentally switch around bad and good :-) )
Since it was a fairly arbitrary choice not to merge that fix into 1.4.x, I'll go ahead and backport it now instead of this one, so that both branches have a consistent fix.

Fix 796239b7d991322632da6f9f63fd1ccd172d3b58 backported to stable/1.4.x in 3ac9c457eb7f6077ed56fa9ecd8b402dba740775.

wow... really? What a strange phenomenon. Fixing problems I didn't even know exist with code that's inconceivably related.

@gasman: git bisect recently learned some new flags that might help with this case: --term-old and --term-new:
$ git bisect start --term-old broken --term-new fixed
# Set the bounds of the bisect range
$ git bisect broken <some-old-commit>
$ git bisect fixed master
# Test the commit, etc, see if the CSS is still broken or not
$ git bisect broken # The CSS is still borked in this commit
$ git bisect fixed # Someone has accidentally fixed it in this commit

@gasman I think this code should still be removed. It's not currently doing anything and is likely to cause bugs or confusion down the line.

@jonnyscholes OK, merged in db449bff2aab5243eb79d193f40442e649362846.

@gasman thanks :)


Powerapps Cli LIbrary

Any plans on publishing the pac library as a separate npm package?

no, not as npm package, since the pac CLI is .NET code. Wrapping it into a npm tarball is technically possible, but weird...
Note that the pac CLI windows installer (http://aka.ms/PowerAppsCLI) doesn't contain the CLI itself, but instead is a installer front end over a set of nuget packages, one of them: Microsoft.PowerApps.CLI
Directly using the nuget package standalone is an advanced scenario; the supported way to install the CLI is via the Windows installer I mentioned initially: it will not only unpack and lay down the CLI nuget and PCF packages, but also add them to the PATH, and also add automatic update support.

This is brilliant. Thank you so much for the info, super helpful. This will definitely come in handy while orchestrating inner dev loop tasks. By any chance, do you or someone you know happen to know if PAC will support import solution command any time soon?


Deprecate and remove methods that exist in Android KTX?

I'm proposing we deprecate and then remove all symbols that are duplicated by the official Android KTX library. That would simplify code completion when using both libraries and ensure the community doesn't split. What do you think?

this is surely a must do I think, will increase productivity of both libraries, as both libraries are great and must be simple to use together, and re implementing same thing with two libraries doesnt make any sense :)

Yes, most of anko-commons will be deprecated at some point in favor of Android KTX. Probably closer to Android KTX release. If you wish to mark something already present in Android KTX with @Deprecated feel free to send pull request.

@4u7 Sweet, see #617. I'd also love it if you could take a look at #616 since I rebased #617 out of it.


Store API: remove distinction between id and value in endorsements

Remove distinction between id and value in endorsements
Abtract SwComponents and TrustAnchors into a more regular Endorsement type
Remove (for now) the unused Get{SwComponent,TrustAnchor} provisioning interfaces
Also fix a couple of lint warnings in the provisioning area

Fixes #122

@setrofim the updated internal interfaces look like this:

PSA
TPM-EnactTrust


[Candidate 1v1 Entropy's Empire ZH v2] Map name contains apostrophe symbol

Issue Description
"Entropy's Empire ZH v2" contains apostrophe symbol.
Expected behavior
Map name does not contain apostrophe symbol. Should contain alpha numeric only (apart from [] for [RANK]).

Please remove that symbol


[WIP] Allow arbitrary secrets for TLS config

@kflynn I had some questions regarding the approach -

I understand that the entire secrets logic need to be taken out of kubewatch.py, and moved into config.py. Is that correct?
Right now, since we support only one secret, we store that one information at /etc/certs/tls.{crt,key} and /etc/cacert/tls.crt, and then update the restarter. How do you suggest we do this with multiple secrets? One way would be to store the secrets at `/etc/certs/server_name/tls.{crt,key} and /etc/cacert/server_name/tls.crt. Another ways could be to store in the cluster itself as another secret or configMap. What do you think?
Currently, once the secrets are found, we update in tls_mod like -

                tls_mod = {
                    "apiVersion": "ambassador/v0",
                    "kind": "Module",
                    "name": "tls-from-ambassador-certs",
                    "ambassador_id": ambassador_id,
                    "config": {
                        "server": {
                            "enabled": True,
                            "cert_chain_file": "/etc/certs/tls.crt",
                            "private_key_file": "/etc/certs/tls.key"
                        }
                    }
                }

and then trigger the restarter update. How do we trigger the restarter update now? Should we make config.py trigger the restarter update?
4. Right now, we watch for updates at startup using https://github.com/datawire/ambassador/blob/95969cbf4cf085684ecdfbed5556f1e711161d79/ambassador/kubewatch.py#L331, how do we watch for updates now with multiple certs?
5. From what I could find online, we'll need to configure SNI in envoy to enable this - https://www.envoyproxy.io/docs/envoy/latest/faq/sni#faq-how-to-setup-sni, is that correct? I will have to update the envoy.j2 to accommodate this, right?
Not very familiar with the codebase right now, so all these questions. Would appreciate any help :)

These are good questions that may or may not have good answers. ;)
We have conflicting pressures here on Config and kubewatch (especially in light of the discussion in #454):

Config should really understand the entire Ambassador configuration, but
Config really should not understand the interface with Kubernetes (because Config needs to work when we're running standalone for tests, or in Docker for the quickstart). Finally,
Config really should not be responsible for restarting Envoy. That goes double as we think ahead to ADS.

Right now, kubewatch understands a bit more of the Ambassador config than it really should, because it's kubewatch that understands how to take a Secret or a ConfigMap and generate Ambassador configuration resources from them. On the other hand, cf #454: we're going to be splitting kubewatch in two anyway, most likely. So we can start thinking in terms of configmgr and kubewatch.
In that light, it feels like this is the sequence we're after:

Is there stuff present on the filesystem?

if so, read each file and send its resources to configmgr.

Is an Ambassador ConfigMap present?

if so, read its resources and send them to configmgr.

Are any secrets present?

if so, read their resources and send them to configmgr.

Are there services present?

if so, read their resources and send them to configmgr.

Finally, watch for changes to services or secrets.

when they happen, send the new resources or a DELETE to configmgr.

(The first four steps can possibly be a new program entirely, rather than a mode of kubewatch, actually.)
In the short term, we can probably move this direction by separating the initialization and parsing steps in the Config class, and giving Config an incremental add method that accepts a new resource, of whatever type, so that you work with things by:
config = Config()

for resource in random_resource_list:
    config.add(resource)

config.generate_envoy_config()

or the like.
@containscafeine, we should talk this over further once @plombardi89 and @rhs and I talk over #454 -- so hopefully tomorrow will be good for that.

Testing:

kubectl create secret tls ambassador-certs --cert old-cert.pem --key old-key.pem
No other TLS Module mapping present
kubectl delete pods -l service=ambassador
Cert loaded which points to old.minikube.<IP_ADDRESS>.xip.io - :green_heart:, everything works fine

kubectl delete secret ambassador-certs
kubectl create secret tls new-secret --cert new-cert.pem --key new-key.pem
added mapping to service -

      ---
      apiVersion: ambassador/v0
      kind:  Module
      name:  tls
      config:
        server:
          enabled: True
          secret: new-secret

kubectl delete pods -l service=ambassador
New secret loaded, cert points to new.minikube.<IP_ADDRESS>.xip.io - :green_heart:, everything works fine

kubectl create secret tls ambassador-certs --cert old-cert.pem --key old-key.pem

$ kubectl get secrets
NAME                     TYPE                                  DATA      AGE
ambassador-certs         kubernetes.io/tls                     2         16s
new-secret               kubernetes.io/tls                     2         2m

The TLS Module mapping exists
kubectl delete pods -l service=ambassador
New secret ignored, ambassador-certs loaded, points to old.minikube.<IP_ADDRESS>.xip.io - :green_heart:

kubectl create secret generic ambassador-cacert --from-file=tls.crt=/home/concaf/ambassador/old-cert.pem --from-literal=cert_required=true
kubectl delete pods -l service=ambassador
contents of old-cert.pem and /etc/cacert/tls.crt inside the pod are the same :green_heart:

kubectl delete secret ambassador-cacert
kubectl create secret generic new-cacert --from-file=tls.crt=/home/concaf/ambassador/new-cert.pem --from-literal=cert_required=true

      ---
      apiVersion: ambassador/v0
      kind:  Module
      name:  tls
      config:
        server:
          enabled: True
          secret: new-secret
        client:
          enabled: True
          secret: new-cacert

kubectl delete pods -l service=ambassador
Contents of new-cert.pem and /etc/cacert/tls.crt inside the pod are the same :green_heart:

kubectl create secret generic ambassador-cacert --from-file=tls.crt=/home/concaf/ambassador/old-cert.pem --from-literal=cert_required=true
new-cacert and ambassador-cacert both secrets exist
kubectl delete pods -l service=ambassador
Contents of old-cert.pem and /etc/cacert/tls.crt are the same, new-cacert has been ignored despite the mapping :green_heart:

Taking into another shared/develop.


fix: updated deps

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


⚠ Bofrost.fr has degraded performance

In 2256ab7, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1908 ms

Resolved: Bofrost.fr performance has improved in 1fd722f after 18 minutes.


Allocate Resource Improper Exception Handling

`
    try {

        for (VirtualDeploymentUnit vdu : virtualNetworkFunctionRecord.getVdu()) {
            VimInstance vimInstance = vims.get(vdu.getId());

            if (vimInstance == null)
                throw new NullPointerException("Our algorithms are too complex, even for us, this is what abnormal IQ means :(");
            try {
                if(vdu.getCreate_and_attach_volumes()!=null){
                    for (CreateVolume createVolume : vdu.getCreate_and_attach_volumes()) {
                        vdu.getVolumes_id().add(vimManagement.createVolume(vimInstance.getId(), createVolume, vdu.getProjectId()).getExtId());
                    }
                }
                ids.add(resourceManagement.allocate(vdu, virtualNetworkFunctionRecord, vimInstance, vdu.getUser_data()));

            }
            catch (VimException e) {
                e.printStackTrace();
                log.error(e.getMessage());
                LifecycleEvent lifecycleEvent = new LifecycleEvent();
                lifecycleEvent.setEvent(Event.ERROR);
                VNFCInstance vnfcInstance = e.getVnfcInstance();

                if (vnfcInstance != null) {
                    log.info("The VM was not correctly deployed. ExtId is: " + vnfcInstance.getVc_id());
                    log.debug("Details are: " + vnfcInstance);
                    vdu.getVnfc_instance().add(vnfcInstance);
                }
                virtualNetworkFunctionRecord.getLifecycle_event_history().add(lifecycleEvent);
                saveVirtualNetworkFunctionRecord();
                OrVnfmErrorMessage nfvMessage = new OrVnfmErrorMessage(virtualNetworkFunctionRecord, e.getMessage());
                return nfvMessage;
            }
        }

    } catch (VimDriverException e) {
        e.printStackTrace();
        log.error(e.getMessage());
        LifecycleEvent lifecycleEvent = new LifecycleEvent();
        lifecycleEvent.setEvent(Event.ERROR);
        virtualNetworkFunctionRecord.getLifecycle_event_history().add(lifecycleEvent);
        virtualNetworkFunctionRecord.setStatus(Status.ERROR);
        saveVirtualNetworkFunctionRecord();
        OrVnfmErrorMessage nfvMessage = new OrVnfmErrorMessage(virtualNetworkFunctionRecord, e.getMessage());
        return nfvMessage;
    }`

These Catch blocks never run, because "resourceManagement.allocate" is an Async method and exception thrown from Async is caught by Spring. Spring then places the exception in "cause" property and throws "ExecutionException". (Right now it is caught in AbstractTask Class)

Catch block should not return, as in case of mutiple VDUs throwing exception only first one will save the "vnfcInstance" and return statement will stop the execution for successive VDUs (making them undeletable from the VIM)

Thanks @Rizwan-Qamar, we fixed this issue in our develop branch


Instructions to use massif for personal media collections

Say I have cloned massif and have a bunch of .epubs or subtitles files and want to use them for indexed sentences, how I'd do it?
I guess this is the most important part of the code for this: https://github.com/rsimmons/massif/tree/master/backend/intake
But there are no instructions so I'm not really sure

Thanks for your interest in Massif.
Unfortunately it would be very difficult to run sentence scoring on your own, which is the key feature of Massif. Also, part of the reason that Massif works (sorta) well is that it draws on a very large set of sentences, around 30 million.
I made the code public in case people were curious how something worked, but I never intended for people to get it running on their own.
I'm curious, why do you want to sort your own sentences by quality/score? What is the overall learning approach you have in mind?

Thanks for your interest in Massif.
Unfortunately it would be very difficult to run sentence scoring on your own, which is the key feature of Massif. Also, part of the reason that Massif works (sorta) well is that it draws on a very large set of sentences, around 30 million.
I made the code public in case people were curious how something worked, but I never intended for people to get it running on their own.
I'm curious, why do you want to sort your own sentences by quality/score? What is the overall learning approach you have in mind?

Massif seems to be biased towards fantasy novels. Being able to use your own personal library would be cool and you'd be able to use it with other media (like visual novels) or more recent books.

In my experience, sorting sentences by quality/score is not useful unless you have a very large corpus. I'm not sure a typical personal library would be big enough. Also, the way I've been scoring sentences (using AI) would be prohibitively expensive for most people.
But I agree it would be good to expand the Massif corpus to include sources beyond fantasy novels. I may get around to that at some point in the future.


Authenticate to provision a user with OAuth with sufficient privileges fails

Steps to reproduce

Add a secondary JDBC userstore - e.g. TEST

Create a new role in TEST. e.g. TEST/role1 and do not assign any permissions.

Create a new user in TEST domain and assign the above role to him. e.g. TEST/user1

Create a OAuth service provider and obtain an access token for TEST/user1.

Now try to provision a new user using SCIM2 by authorizing with the above access token.
curl -v -k -H "Authorization:Bearer 9eb61d5e-4df6-383d-b2b0-47a153bc9c16" --data '{"schemas":[],"name":{"familyName":"jayawardana","givenName":"vindula"},"userName":"vindula4","password":"admin","emails":[{"primary":true,"value":"vindula_home.com","type":"home"},{"value":"vindula_work.com","type":"work"}]}' --header "Content-Type:application/json" https://localhost:9443/scim2/Users

This fails with 403 forbidden error as expected.

Next update the TEST/role1 with following permission.

/permission/admin/manage/identity/usermgt/create

Now obtain a new access token for the same user.
Now if we try to provision a new user using SCIM2 by authorizing with this new access token, it is failing with 403 unauthorized error even though required permission is assigned to the role.

in user.core debug logs we can see the following
[2019-04-17 16:23:36,155] DEBUG {org.wso2.carbon.user.core.authorization.JDBCAuthorizationManager} -  Authorization cache hit. TEST/user1 user is not Authorized to perform ui.execute on /permission/admin/manage/identity/usermgt/create

Changing the severity for further review

Here we have authentication flow and authorization flow. Token validation service is used at authentication and username is set by the validator. However, when "Use user store domain in local subject identifier" is not enabled in Local & Outbound Authentication Configuration usernames of secondary user-store users are set without the domain parameter.
Since the user realm is configured depending on the tenant name when authorizing secondary user store users at Authorization Valve suffering from a caching issue which resulted in permission update ignorance.
This issue is not affected to tenant users as the tenant is explicitly appended to the username at the time of authentication.
This issue can be overcome by enabling "Use user store domain in local subject identifier" in Local & Outbound Authentication Configuration

With the way presently this is designed we have a mechanism to overcome this respective problem as mentioned by @IsurangaPerera
Yet, the design of this handler is conceptually wrong. Even here the access token is used to authenticate the user.
IMO, to keep the present design on, we can call user info endpoint and get user realm and proceed with present design rather than depending on the subject returned from the introspection endpoint.
Or the handler should be designed under the authorization aspect.
@IsurangaPerera , can you please create a separate issue to address this

We need to document the fact that to get for the respective service provider, user store domain in subject identifier needs to be enabled.
@IsurangaPerera , can you please create a doc issue to track this

Further noted that this happens when below config in identity.xml is set to 'true'.
<BuildSubjectIdentifierFromSPConfig>true</BuildSubjectIdentifierFromSPConfig>
There is a problem on identifying the user realm information with this config
From the authorization rest valve perspective we can of course ask to enable "Use user store domain in local subject identifier" that @IsurangaPerera mentioned above.
Yet, there is a concern on how user realm information can be relayed from introspection.
Requirements, concerns around this needs to be thought through .

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


ModFS: return ModFSFileInfo objects list in GetAllFilesInFolder

This allows us to see the relative path of a ModFS file, not just its absolute path.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

26 of 26 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 92.835%

Totals

Change from base Build<PHONE_NUMBER>9:
0.02%

Covered Lines:
3443

Relevant Lines:
3666

💛  - Coveralls


The end of line ($) in a regex, is replaced by

Example:

<template>

{{pattern}}

</template>

After formatting, it becomes:

<template>

{{ pattern }}

</template>

If I add $ to the end of the string separately:
const pattern = new RegExp(`/user/d+` + '$');

When formatting, Prettier throws an error "SyntaxError: Unexpected closing tag 'script'. It may happen when the tag has already been closed by another tag." Without the plugin, all errors disappear.
.prettierrc
{
  "plugins": ["@trivago/prettier-plugin-sort-imports"],
  "importOrder": [],
  "importOrderSeparation": true,
  "importOrderSortSpecifiers": true
}

Packages:

prettier: 3.3.3,
@trivago/prettier-plugin-sort-imports: 4.3.0

I think I have come across the same issue, PR here https://github.com/trivago/prettier-plugin-sort-imports/pull/283

It seems to have nothing to do with regex and setup in Vue.

It becomes:


Input shape Error, bidirectional rnn

System Info:
- OS: Windows 7
- Cude: 9, 9.1
- Tensorflow: 1.7.0
- GPU Mode
I have some problems with bidirectional rnn inputs. After my anderstanging, a bidirectional  rnn needs a input from the shape "[batch_size, input_size]".
May input "hidden" has first the shape (?, 512), I then reshape is to (1, 512) and expande the dims to
(1, 1, 512). I treid to pass all three versions to the bidirectional  rnn (and many more).  I have already used the last shap successful, with a normal rnn.
Error:
Traceback (most recent call last):
  File "X:/PyProjects/Domm_Lerner/A3C_Arthur_Julianis_Doom.py", line 572, in <module>
    master_network = AC_Network(s_size, a_size, cnn_size, 'global', None)  # Generate global network
  File "X:/PyProjects/Domm_Lerner/A3C_Arthur_Julianis_Doom.py", line 159, in __init__
    lstm_1_outputs, lstm_1_state_fw, lstm_1_state_bw = tf.nn.static_bidirectional_rnn(lstm_cell, lstm_cell, rnn_in,         initial_state_fw=state_1_in_fw, initial_state_bw=state_1_in_bw, sequence_length=step_size)
  File "X:\PyProjects\Domm_Lerner\venv\lib\site-packages\tensorflow\python\ops\rnn.py", line 1478, in static_bidirectional_rnn
    raise TypeError("inputs must be a sequence")
TypeError: inputs must be a sequence

Code:
            hidden = tf.reshape(hidden, shape=[1, cnn_size])
            rnn_in = tf.expand_dims(hidden, [0])

            step_size = tf.shape(self.imageIn)[:1]

            lstm_1_outputs, lstm_1_state_fw, lstm_1_state_bw = tf.nn.static_bidirectional_rnn(lstm_cell, lstm_cell, rnn_in,         initial_state_fw=state_1_in_fw, initial_state_bw=state_1_in_bw, sequence_length=step_size)
            lstm_2_outputs, lstm_2_state_fw, lstm_2_state_bw = tf.nn.static_bidirectional_rnn(lstm_cell, lstm_cell, lstm_1_outputs, initial_state_fw=state_2_in_fw, initial_state_bw=state_2_in_bw, sequence_length=step_size)
            lstm_3_outputs, lstm_3_state_fw, lstm_3_state_bw = tf.nn.static_bidirectional_rnn(lstm_cell, lstm_cell, lstm_2_outputs, initial_state_fw=state_3_in_fw, initial_state_bw=state_3_in_bw, sequence_length=step_size)
            lstm_4_outputs, lstm_4_state_fw, lstm_4_state_bw = tf.nn.static_bidirectional_rnn(lstm_cell, lstm_cell, lstm_3_outputs, initial_state_fw=state_4_in_fw, initial_state_bw=state_4_in_bw, sequence_length=step_size)
            lstm_5_outputs, lstm_5_state_fw, lstm_5_state_bw = tf.nn.static_bidirectional_rnn(lstm_cell, lstm_cell, lstm_4_outputs, initial_state_fw=state_5_in_fw, initial_state_bw=state_5_in_bw, sequence_length=step_size)
            lstm_6_outputs, lstm_6_state_fw, lstm_6_state_bw = tf.nn.static_bidirectional_rnn(lstm_cell, lstm_cell, lstm_5_outputs, initial_state_fw=state_6_in_fw, initial_state_bw=state_6_in_bw, sequence_length=step_size)

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 7 6.1.7601 Service Pack 1 Build 7601
TensorFlow installed from (source or binary): pip install
TensorFlow version (use command below): 1.7.0
Python version: 3.6
Bazel version (if compiling from source): n/a
GCC/Compiler version (if compiling from source): n/a
CUDA/cuDNN version: 9
GPU model and memory: GTX 750 Ti 2GB and GTX 1080 Ti 11GB
Exact command to reproduce:

Nagging Assignee @rohan100jain: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 46 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 61 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 76 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 91 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 106 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 106 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I think inputs needs to be a list more than the appropriate shape.

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 35 days with no activity and the awaiting response label was assigned. Is this still an issue?

We are closing this issue for now due to lack of activity. Please comment if this is still an issue for you. Thanks!


Generic Flair MultiLabelPool class

Are you asking pool request? I would like to help maintain the repo if you are open to it. I think this project is super cool and relevant to the research that I do. Making a generic Flair MultiLabelPool class would give access to all the sequence label datasets available in the flair repo.

Yes, that is a good idea. We could also think about exposing sequence tagging datasets in hugging face datasets repo. Regarding the maintenance, sure feel free to support and contribute. I have been following your active learning research, would also like to hear in what context you would like to use nlp_gym :)


XML parsing

This feed http://www.heise.de/open/news/news-atom.xml renders void entries in elfeed, although content is defined.

I'm surprised this issue hasn't come up yet, since this bug has probably
been around since the beginning. The issue is that this feed uses
embedded XHTML (allowed by Atom). This gets parsed with the rest of the
feed document by the XML parser. When Elfeed builds the entry, it needs
to convert the data structure back into XML so that it can be reparsed
later by shr in elfeed-show.


TF 2.0 not support

Platform (like ubuntu 16.04/win10):
Python version:
Source framework with version (like Tensorflow 1.4.1 with GPU):
Destination framework with version (like CNTK 2.3 with GPU):
Pre-trained model path (webpath or webdisk path):
Running scripts:

@duguqiankun , thank you very much for the feedback. TensorFlow 2.0 is not officially released yet now. We will support it in future.

Hi! This library looks promising. As TF2.0 has been officially released for some time, do you have concrete plans regarding the support?

I am just wondering when the support of tf2.0 will be released.

I successfully converted mxnet into TF2.0 model, the main idea is:

firstly convert mxnet into IR
generate keras code based on IR file
Debug and modify the keras file, enbale tf2 running. (It is worth to note that TF2 comes with a native keras support, so you don't need to do so much modifications, just use the tf's keras and it will be ready.)

I have a blog which is in Chinese[中文]，maybe can help some guy. Here is the link: https://firstai.blog.csdn.net/article/details/108628300

Hi, I forked this repository and added the tf-keras backend/framework. This is simply a port of keras to tf.keras and works with weights in HDF5 format. It seems to be working fine in TensorFlow 2.0, at least for my needs.
Example usage:
from tensorflow.keras import layers, Model
from tensorflow.keras.models import save_model
inp = layers.Input([10, 10, 3])
x = layers.Conv2D(5, 3)(inp)
x = layers.Flatten()(x)
x = layers.Dense(5)(x)
x = layers.Softmax()(x)
m = Model(inp, x)
save_model(m, 'model.h5', save_format='h5')  # cannot be in frozen graph format

To convert:
mmconvert -sf tf-keras -iw model.h5 -df pytorch -om model.pt

This yields the files model.pt and model.py.
Loading:
import torch
import imp
import numpy as np

MainModel = imp.load_source('MainModel', 'model.py')
model = torch.load('model.pt')

Eval:
>>> model.eval()
KitModel(
  (conv2d): Conv2d(3, 5, kernel_size=(3, 3), stride=(1, 1))
  (dense): Linear(in_features=320, out_features=5, bias=True)
)

My Versions:
>>> import tensorflow as tf; import torch
>>> tf.__version__
'2.1.3'
>>> torch.__version__
'1.3.1'

craymichael/MMdnn


DROOLS-1277 Remove detailed info by process instance, ...

... keeping statistics by process (aggregated) only

@etirelli for your review please.

+1


Bls signer blockchain methods

What is this PR doing?
This PR is concerned with adding/tesing the signer blockchain methods.
getBalance()
getChainId()
getGasPrice()
getTransactionCount()
call()
estimateGas()
resolveName()

Notes:
estimateGas - We are currently not getting an accurate value here, so I just added tests to ensure the method does not error. We have an issue in the backlog to more accurately estimate gas for transactions. See #410
resolveName - ENS records are not supported by hardhat so we have decided to omit an override and test that the underlying ethers logic throws the correct errors. ENS are working on L2 support that allows ENS records to be stored on L2s while retaining the ability to be resolved, so we should keep an eye on these developments.
How can these changes be manually tested?

run local hardhat node
Spin up a local aggregator (Make sure to delete bundle table in aggregator db if you haven't already)
in the ./contracts directory, run yarn test-clients

Does this PR resolve or contribute to any issues?
#374
Checklist

[x] I have manually tested these changes
[x] Post a link to the PR in the group chat

Guidelines

If your PR is not ready, mark it as a draft
The resolve conversation button is for reviewers, not authors

(But add a 'done' comment or similar)

Commit history is a bit...much given we squashed the last one and this includes some of those old commits. You may be able to git cherry-pick your commits/changes for this PR into a clean state from main. I can help with that offline.

Yeah I tried (and failed) rebasing. I'll give cherry picking a go and if I get nowhere, help offline sounds good :)


Import-*HostingServiceCert examples have two typos

These two examples must be fixed, the current form is invalid:
"-ComputeName" should be "-ComputerName"
and
"[Byte[]] $HostingCertContent" should be "[Byte[]]$HostingCertContent"
Additionally (the current version is valid syntax):
[Byte[]]$HostingCertContent = [Byte[]](Get-Content c:\certificate\myhostingcertificate.pfx  -Encoding Byte)
could be shortened to:
[Byte[]]$HostingCertContent = Get-Content c:\certificate\myhostingcertificate.pfx -Encoding Byte
Note: "  -Encoding" currently has two leading spaces and only needs one.

@johnerskine : Thanks for your contribution! The author, @mattbriggs, has been notified to review your proposed change.


[Java] Ed25519 long first verification call

Describe the bug:
When verifying a Ed25519 signature, the first verification takes significantly longer (100x) than subsequent verifications.
What was the expected behavior?
All verifications take less than 1ms
How can we reproduce the bug?
I modified the Java tutorial and ran 10 verifications
import com.google.crypto.tink.CleartextKeysetHandle;
import com.google.crypto.tink.JsonKeysetReader;
import com.google.crypto.tink.KeysetHandle;
import com.google.crypto.tink.PublicKeyVerify;
import com.google.crypto.tink.signature.SignatureConfig;

import java.io.IOException;
import java.nio.charset.StandardCharsets;
import java.security.GeneralSecurityException;
import java.time.Instant;
import java.time.temporal.ChronoUnit;

public class Main {
  public static void main(String[] args) throws GeneralSecurityException, IOException {
    SignatureConfig.register();
    byte[] hello = "hello".getBytes(StandardCharsets.UTF_8);
    byte[] signatureOfHello =
        new byte[] {
          1, 80, -65, 99, -116, -52, 123, -52, 93, 58, 22, 62, 19, -77, 88, -61, -25, -102, -76,
          -109, -5, 100, -96, 17, 107, -81, 22, 80, 107, 34, -51, -46, -82, 96, 2, -93, -82, -8,
          -22, 5, -78, -69, 98, 34, -36, -57, -37, 126, -91, -1, -98, 112, 127, -113, -1, -31, -51,
          63, 36, 9, -85, -14, 17, -30, -71, 28, 40, 82, 14
        };

    // VERIFYING

    // 1. Obtain a handle for the public key material.
    String publicKey =
        "{\"primaryKeyId\":1354720140,\"key\":[{\"keyData\":{\"typeUrl\":\"type.googleapis.com/google.crypto.tink.Ed25519PublicKey\",\"value\":\"EiBBGA30gT9m79ETMRXZRxie4idJftl0UkZR81wAaaE49w==\",\"keyMaterialType\":\"ASYMMETRIC_PUBLIC\"},\"status\":\"ENABLED\",\"keyId\":1354720140,\"outputPrefixType\":\"TINK\"}]}";
    KeysetHandle publicKeysetHandle =
        CleartextKeysetHandle.read(JsonKeysetReader.withString(publicKey));

    // 2. Get the primitive.
    PublicKeyVerify verifier = publicKeysetHandle.getPrimitive(PublicKeyVerify.class);

    // 4. Use the primitive to verify.
    for (int i = 0; i < 10; i++) {
      Instant start = Instant.now();
      verifier.verify(signatureOfHello, hello);
      System.out.printf(
          "[%d] Verify took %dms\n", i, ChronoUnit.MILLIS.between(start, Instant.now()));
    }
  }
}

Do you have any debugging information?
[0] Verify took 164ms
[1] Verify took 1ms
[2] Verify took 1ms
[3] Verify took 2ms
[4] Verify took 1ms
[5] Verify took 1ms
[6] Verify took 1ms
[7] Verify took 1ms
[8] Verify took 1ms
[9] Verify took 2ms

Provide your version information:

Language: Java
Version: 1.7.0
Environment: JDK 8

You are right, it takes much longer for the first call. The reason seems to be that when the first time Ed25519Constants is used, this class is loaded and some constants are precomputed. And that takes some time.
Is this an issue for you? If it is, I guess you could verify something at startup of your binary, so that everything is loaded when you need to verify the first signature.

You are right, it takes much longer for the first call. The reason seems to be that when the first time Ed25519Constants is used, this class is loaded and some constants are precomputed. And that takes some time.
Is this an issue for you? If it is, I guess you could verify something at startup of your binary, so that everything is loaded when you need to verify the first signature.

I am doing a dummy sign and verify in my binary's startup to get around this. So not a huge deal, but maybe worth noting in docs?

I think I will just add some code that make sure that Ed25519Constants is initialized when the Ed25519 PublicKeyVerify primitive is created.

I have now added this. When I run you code now, the first call is almost as fast as the following calls.

See here: https://github.com/tink-crypto/tink-java/commit/d8a064702f09c586fa983c6cc867a1dcc2ae79f5


pacquet install with dependencies that require color-convert fail to parse: Missing integrity field

The package.json that cause error:
{
  "dependencies": {
    "@tsfun/all": "0.0.37",
    "@babel/core": "^7.22.9",
    "@changesets/cli": "^2.26.2",
    "@types/jest": "^29.5.3",
    "@types/node": "^16.18.39",
    "c8": "^7.14.0",
    "concurrently": "8.2.0",
    "cross-env": "^7.0.3",
    "eslint": "^8.46.0",
    "husky": "^8.0.3",
    "keyv": "4.5.3",
    "lcov-result-merger": "^3.3.0",
    "publish-packed": "^4.1.1",
    "rimraf": "^3.0.2",
    "shx": "^0.3.4",
    "syncpack": "^8.5.14"
    "shx": "^0.3.4"
  }
}

This issue does not seem to exist anymore. Closing.


proper clean up

Issue and Steps to Reproduce
Shouldn't be oidc.removeEventSubscription(subscriptionId); called in useEffect cleanup function here?
Versions.
"@axa-fr/react-oidc-context": "^4.5.8"

Thank you for your issue @pinggi, you are right this is an issue thank you very much.

@guillaume-chervet that's great you fixed it. I think you could have deleted also this code:
        if(subscriptionId){
            oidc.removeEventSubscription(subscriptionId);
        }

when a component is mounted, useEffect function is called and the cleanup function returned from it registered
when the useEffect dependencies are changed (in this case configuration or configurationName), the cleanup function is called first and then the useEffect function

I also think the if(isMounted) { check is also useless since it's not an async code that would run when the component is unmounted - it's indented like it would be part of the oidc.subscriveEvents function but in fact it isn't.


chore(deps): update terraform terraform-ibm-modules/event-notifications/ibm to v1.10.21

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/event-notifications/ibm (source)
module
patch
1.10.20 -> 1.10.21

Release Notes

terraform-ibm-modules/terraform-ibm-event-notifications (terraform-ibm-modules/event-notifications/ibm)
v1.10.21
Compare Source
Bug Fixes

do not create auth policy if existing_cos_instance_crn is null (#304) (1ec6f34)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 1.8.11 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ERC4626 Support

[EDIT]: only withdraw and redeem methods in ERC-4626 use the underlying ERC-20 approval, so I edited the below to reflect that. This also simplifies the permit2 integration.
Similar to https://github.com/Uniswap/universal-router/issues/176, I believe ERC-4626 integration directly into permit2 is highly beneficial. The main reasons are:

natural integration - ERC-4626 leverages the native ERC-20 approval system (and by extension permit and permit2) for mutable share exit methods (withdraw and redeem), so naturally benefits greatly
gas savings - Integrations like universalRouter can save gas by directly plugging into ERC-4626 mutable methods rather than transferring shares around (which can be costly if the vault includes custom transfer logic, such as in Compound). New vaults could even default approve permit2 for further user gas savings.
contract size reduction - New ERC-4626 contracts need not include EIP-2612 logic.

Given that permit2 is intended to be deployed only once to save user approvals on gas, I'd lobby for ERC-4626 inclusion in the canonical deployment, whereas on universal router it can be added in a later deployment.
The simplest integration would be to only replicate the single-use transferFrom logic in AllowanceTransfer.sol for both of the withdraw and redeem functions. Batch methods and SignatureTransfer could also be added for each, although I'm not sure if the extra complexity is warranted
Can I get a sense for the intended freeze date for the permit2 contract? If the timelines are possible, I would be able to have a tested PR ready before the end of next week.

Hey @Joeysantoro love the flag here for ERC-4626. We designed permit2 to support other token types (ERC721/ERC1155) as well but each token standard would have their own canonical permit2. So this permit2 contract is specifically for ERC20 tokens and we plan to launch separate permit2 instances for 721/1155. I'm not as familiar with this EIP so I'll need to look more into it, but I guess the question would be, how different is the approve/transfer methods for ERC-4626? Maybe we can also write a canonical ERC-4626 specific permit2 (similar to ERC721/ERC1155 pattern we've designed) or do you see natural benefit in supporting it at the ERC20 permit2 contract layer?

@snreynolds thanks for the quick reply! It makes most sense directly in the ERC20 permit2 contract specifically because the ERC-4626 contract directly uses the ERC-20 approval system (all ERC-4626 contracts are themselves ERC-20). Per (EIP-4626](https://eips.ethereum.org/EIPS/eip-4626):

All EIP-4626 tokenized Vaults MUST implement EIP-20 to represent shares.

Its easiest to imagine ERC-4626 as a cToken. Take cDAI for example, the contract has all basic ERC-20 functionality as well as the ability to deposit and withdraw. The ERC-4626 standard allows for approved addresses of a given cDAI holder to withdraw or redeem the cDAI on behalf of the holder. Taking Universal Router where a user holds cDAI and wishes to swap into ETH as an example:

User approves permit2 once for cDAI
User signs message permitting universalRouter to redeem
User redeems cDAI to DAI to UniversalRouter via permit2 integration with ERC-4626 specifically for "withdraw" method
UniversalRouter swaps DAI to ETH on UniV3

Without the permit2 integration for the withdraw method, step 3 would become:
3a. transfer cDAI to the UniversalRouter
3b. redeem cDAI directly on the router (without using the approval)
You can see how the withdrawal logic shares the ERC-20 approval system in the solmate example ERC-4626 base: https://github.com/transmissions11/solmate/blob/main/src/mixins/ERC4626.sol#L73-L84

+1 Supporting ERC4626 not only provides gas savings but can also provides better price execution. A use could simply unwrap and wrap two different 4626 tokens with the same underlying asset without even having to make a trade incurring slippage and swap fees.
Since all ERC4626 must be ERC20 compliant and they are fully-backed wrappers around ERC20 tokens as well I think it makes sense to include in canonical Permit2 contract as well. 4626 already has a lot of support and there will be a ton of volume between ERC20 and 4626 tokens.

hey @Joeysantoro looping back in here. I think we'd be down to support ERC4626 in this repo but in a different contract similar to how we are doing ERC721 and ERC1155 support which will be implemented this week. If you want to give it a go, essentially created a new Permit2_4626 contract that inherits new AllowanceTransfer and SignatureTransfer contracts that support ERC4626 I am happy to review and get it merged into this repo. We can then discuss integrating those contracts into Universal Router as per the discussion here.
cc @hensha256
Also if you want to chat more feel free to shoot me a TG @saraareynolds

@snreynolds great! Seeing the 721 or 1155 implementation will be helpful as well.
If you all are committed that the contracts should be separate, then either the 4626 contract should contain all ERC-20 methods as well, or users of each user of a 4626 contract should be expected to approve the ERC-20 permit2 instance for ERC-20 functionality and ERC-4626 instance for 4626 functionality
Both are somewhat clunky for integrating contracts such as the router. The former would have better UX (user only needs to approve in one place, but contracts will need to duplicate all ERC-20 permit2 functions across the ERC20 and ERC4626 instances). The latter would have better devX because the ERC-20 functionality is all in the same contract so it is less duplication.
Let me know if this makes sense and what your thoughts are


error message "this app cannot run on your pc "

Hello.
Sorry i don't speach english.
Sometimes, when i run youtube-dl, i have this error message : "this app cannot run on your pc "
Not all time, just sometimes. When i restart my PC, its work, but sometimes :)
I dont no how make.
Its just youtube-dl. Nothing else.
I have windows 10 pro 64 bits.
Youtube-dl version last (youtube-dl is up-to-date (2021.06.06))
Can you help me ?
Thank

This reddit comment might help.

Windows 10 can display this message if your youtube-dl executable is incomplete. Please double check your download of youtube-dl is completing successfully by verifying the correct size and signature.


cannot open the portal after fresh install

I cannot open the portal after clean install on a rpi2
i didnt enable advance feature for the portal
log here
2016-04-10 20:12:48: (mod_fastcgi.c.2695) FastCGI-stderr:   thrown in /var/www/html/classes/common.class.php on line 56
2016-04-10 20:12:48: (mod_fastcgi.c.2695) FastCGI-stderr: PHP Fatal error:  Uncaught exception 'PDOException' with message 'could not find driver' in /var/www/html/classes/common.class.php:56
2016-04-10 20:12:48: (mod_fastcgi.c.2695) FastCGI-stderr: Stack trace:
2016-04-10 20:12:48: (mod_fastcgi.c.2695) FastCGI-stderr: #0 /var/www/html/classes/common.class.php(56): PDO->__construct('mysql:host=loca...', 'adsbuser', 'password')
2016-04-10 20:12:48: (mod_fastcgi.c.2695) FastCGI-stderr: #1 /var/www/html/classes/common.class.php(82): common->pdoOpen()
2016-04-10 20:12:48: (mod_fastcgi.c.2695) FastCGI-stderr: #2 /var/www/html/classes/template.class.php(68): common->getSetting('template')
2016-04-10 20:12:48: (mod_fastcgi.c.2695) FastCGI-stderr: #3 /var/www/html/classes/template.class.php(42): template->readTemplate('master.tpl')
2016-04-10 20:12:48: (mod_fastcgi.c.2695) FastCGI-stderr: #4 /var/www/html/dump1090.php(46): template->display(Array)
2016-04-10 20:12:48: (mod_fastcgi.c.2695) FastCGI-stderr: #5 {main}
2016-04-10 20:12:48: (mod_fastcgi.c.2695) FastCGI-stderr:   thrown in /var/www/html/classes/common.class.php on line 56

Looks as though you ran the portal install page and set things up to use MySQL. Rerun http://[YOUR_IP_ADDRESSS]/admin/install.php and select XML as where data should be stored.

@jprochazka it works
Thanks
Maybe need to add those into readme.md so other people would know
Thanks,


SearchQueryOperator in organizations: serde rename values are rejected by API

// src/b2b/organizations.rs#855-862
pub enum SearchQueryOperator {
    #[serde(rename = "or")]
    #[default]
    OR,
    #[serde(rename = "and")]
    AND,
}

With the renames, the API is rejecting them saying that they must be either "AND" or "OR". If I remove the renames so they are "AND" and "OR", then it works.
I believe the same is true for M2MSearchQueryOperator but not 100% sure

Hi @graysonarts thanks for flagging this! We just rolled out https://github.com/stytchauth/stytch-rust/commit/382372e273036a31c44a512e3b1622570931581e which should fix this issue. We had an issue with our codegen and have since fixed the issue.

Ollie


Vivaldi support

It would be great with support for Vivaldi. I think it should be easy to implement, since the folder seems to be identical to Chrome. I.e ~/Library/Application Support/Vivaldi/Default/bookmarks.
I tried myself, but have never tried Go before so didn't get it to work.

@Donkeyfish87 It is easy to do:

Fork the repo
Edit browser.yml and append the following lines:

- id: "bac1400d-5487-42a9-8571-95131e4dd98b"
  name: "vivaldi"
  fullName: "com.vivaldi.Vivaldi"
  description: "Vivaldi"
  path: "~/Library/Application Support/Vivaldi"
  iconUrl: "img/vivaldi.png"
  profileName: "Default"

Add workflow/img/vivaldi.png

Run make workflow, open build/ folder and find chrome-bookmarks.alfredworkflow here.

@Donkeyfish87 @peng051410 https://github.com/mdreizin/chrome-bookmarks-alfred-workflow#how-to-add-missing-chromium-based-browser


Provide proper SF symbol support

Right now in cacao all symbols are hardcoded and I personally believe there's no real reason for this, and if a symbol doesn't exist; you could just do it like how SwiftUI does it where it's just a blank image and it warns you that it doesn't exist if the OS doesn't support it

Because of this, this is currently how I am having to do SF symbols (if anyone does use this, please note that this isn't safe and will crash if the OS doesn't support it, in my app this doesn't really matter to me though)
pub fn symbol_from(system_name: &str) -> Image {
    Image(unsafe {
        let icon = NSString::new(system_name);
        let desc = NSString::new("");
        msg_send![class!(NSImage), imageWithSystemSymbolName:&*icon
                    accessibilityDescription:&*desc]
    })
}

Which works nicely, I can use symbols from Sequoia now in my Cacao app:

I am requesting that something similar to my function could maybe be included in Cacao to prevent any confusion; I'm not really sure what the best and cleanest way of integrating this would be but I think it could be discussed a little here; I personally like how SwiftUI does it again but ofcourse this isn't very rust-y
Image(systemName: "house.fill")

I personally believe there's no real reason for this

I mean, this is a bit personal - I'll take an enum variant over a string littering the codebase any day. ;)
That said...

I am requesting that something similar to my function could maybe be included in Cacao to prevent any confusion

I think your desired functionality is entirely appropriate and I'd welcome a PR. It might sit for a small bit since I've got a number of things to merge here - and I'm currently on a boat in Antarctica - but I'll get to it eventually. :)
Re: functionality, it could either be a method on SFSymbol or Image for now (e.g SFSymbol::image_from("symbol")) or Image could be rearchitected to be more of a builder type. I'm open to ideas as well.

I came up with Image::with_system_symbol_name("icon.name"). I guess this is a little subjective but i went ahead and PR'd this, can be discussed more there if needed
#144


Loads of very useful ModbusTCP parameters, like a dream

By chance I dedected a bunch of new parameters in the ModbusTCP description for the IDM heatpump. This should give us a lot more possibilities:
You can check the values and addresses yourself by activating the "Service-Ebene", and then
Service - Gebäudeleittechnik - GLT Monitor
In here you see following new, (not yet described) parameters, even some of them for writing:

ID 76 Leistung E-Heizstab, RW reading and writing, not sure what is that for
ID 82 Hausverbrauch RW, that may activate us the statistics in the IDM GUI, worth to try
ID 84 Batterieentladung RW
ID 86 Batterie Füllstand RW ,,, All this values may give us the activation of IDM statistics, I want to try...

and now the very important parameters:

ID 4108 Leistungsbegrenzung Wärmepumpe RW, o my God, this is the desired value; I need this to write. I lack of the unit, but guess kW, because all other power values are in kW.
ID 4112 Leistungsbegrenzung Kaskade RW, no I do not have a "Kaskade" but may be useful for somebody.
ID 4124 elektrische Gesamtleistung, no idea why it is there again, worth to check. will observe and report. Probably only if you have a Kaskade
ID 4126 thermische Leistung no idea why this is there again, the original ID was 1790

I think the ID 4108 Leistungsbegrenzung and ID 82, 84 and 86 are gamechangers...

Interesting, I will take a look as soon as I find some time to work on the integration.

I could not wait and activiated my old modbus writing script.
It actually works and you get a lot more details in the IDM GUI. In the previous ModbusTCP integration you just had seen the overproduction and based on that, the heatpump activation more activities.
With the additional parameters, you got a full Photovoltaik-Integration in IDM. You see the battery, if you have one and your house power. Actually writing these values had been a little bit tricky, since I expected the usage differently.
So here my findings:
ID 76 Leistung E-Heizstab, RW reading and writing, not sure what is that for
... not tried, no idea what it is good for. Anybody an idea?
ID 82 Hausverbrauch RW,
Yes this is a float value, will be written like all other kW values for the PV production. Unit is kW.
This is actually the consuption including the heatpump. What the IDM GUI does, it substracts its own power consuption from that value and assigns the rest to the house. Be caseful to set this value including the consuption of the heatpump, since otherwise it reduces the power from its own consuption and the house consuption is 0.
ID 84 Batterieentladung RW
Is just used for the GUI and nice to see. It is not part of any IDM calculation, just used for statistics. Value is again a float in in kW
ID 86 Batterie Füllstand RW
This is an integer value (I just passed the value without converting to float). I think -1 is interpreted not to have a battery, and it will be removed from display and statistics. Otherwise it is the SOC (state of charge) in %. So valid parameters are from 0 to 100.
The very great thing is, once writing these values, there appear a lot more settings in the IDM GUI. You have now the possiblity to specify a minimum battery level for setting PV mode in IDM. Not exactly understood, how this works, but these values activates a lot more statistics and values in the IDM GUI. Really nice.
Have fun and I look forward to have it integrated here...

ID 76 Leistung E-Heizstab, RW reading and writing, not sure what is that for

Most (if not all) heat pumps have a resistive heating element to assist the actual heat pump in very cold weather (e.g. -10°C). So I guess reading the value tells you whether the heating element is in use. It is in kW and not a simple on/off value, since some IDM models (including mine) have multiple heating elements with different resistances (and therefore wattage), so the heat output can be regulated better.
Not sure what writing would do. Maybe you can manually enable the heating element, maybe it just limits the power, who knows. Without documentation, I don't think I will support writing this value in the integration.

ID 82 Hausverbrauch RW,
ID 84 Batterieentladung RW
ID 86 Batterie Füllstand RW

I added them to #50

yes exactly, I use the parameter of the battery with the excess of solar power. If I have surplus and the SOC is at least 30% I run an additional domestic hot water load for example.

Interesting, I will take a look as soon as I find some time to work on the integration.

By calling id86 I can correctly see the INT value of the battery charge. Great and trying it works

Great work. I will delete my modbus sensors to see soc percentage. Thanks

I will try, but I assume no change. Maybe that is linked to a special contract for your grid provider? Maybe a smart grid contract needs to be active? I am just guessing...

Just checked my previous tries. I already did what you proposed. So I have written 1200.0 to that register, without any success. Still ignored and nothing shown in the GLT Monitor. Further ideas?

Further ideas?

Nope sorry, you'll have to ask IDM how that parameter works.

Correct and thanks for support. I asked my iDM support today, to ask iDM the question about that parameters. My hope is very low they give an reasonable answer, as everything I asked till now, including detailed bug reports had been ignored. iDM no longer is touchable for customers, .... Let's see maybe we got surprised ;-)

I got a response, but the expected one. These parameters are only for iDM themselves. Nothing a user should know about.
Of course, thanks iDM for not helping us, like nowadays always....

Ah this issue can be closed, nothing else to do here...

New upgrade for IDM navigator
Release Notes_Nav_2.0_en.pdf

Cool Krasto82, where you got this document from? Do you have a reasonable contact to iDM, or you found it on internet?

@Krasto82 Very interested in the source.

Cool Krasto82, where you got this document from? Do you have a reasonable contact to iDM, or you found it on internet?

The pdf was shared with me by my installer. He told me that idm always sends a changelog to them when they release a new firmware

Did you got more files, or just the description file? The company name of the installer is Nothaft or a different company?

Only this file, the installer updated the firmware via OTA. Company Name of the installer is Valsolar located in Italy.

Ah finally a different contact. Will they forward requests to iDM? How helpful are they? I struggly a little bit with my contacts...

Ah finally a different contact. Will they forward requests to iDM? How helpful are they? I struggly a little bit with my contacts...

they are a local company and are in contact with idm italia.
If you want I'll leave you their website
https://valsolar.eu/


Enhance UI and Typography

I would like to work on enhancing the user interface design and typography. Kindly assign me this issue.

Its not required currently in website.


Roll Flutter Engine from ac6cb9f24494 to be1ea9e9d67c (1 revision)

https://github.com/flutter/engine/compare/ac6cb9f24494...be1ea9e9d67c
2022-09-22<EMAIL_ADDRESS>Roll Skia from de61ea34de5d to 276e0a2dcc63 (1 revision) (flutter/engine#36345)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Improve mapping between enum value and designation ECFLOW-1931

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (7d4b289) 80.17% compared to head (88e9f3e) 83.53%.

Files
Patch %
Lines

ACore/src/ecflow/core/Enumerate.hpp
85.71%
2 Missing :warning:

Base/src/ecflow/base/cts/user/AlterCmd.cpp
91.66%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #69      +/-   ##
===========================================
+ Coverage    80.17%   83.53%   +3.35%
===========================================
  Files          625      590      -35
  Lines        57957    52438    -5519
===========================================
- Hits         46466    43803    -2663
+ Misses       11491     8635    -2856

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Error when connection lost to the server during edit

From @redswede on December 26, 2014 13:37
Hi. During the editing of a post. Connection get lost. I click submit and nothing happens. I would like a javascript warning to appear (and additionally I would like to be able to disable this warning-message).
Copied from original issue: friendica/red#764

@redswede This issue is closed because Redmatrix evolved into Hubzilla. When this is still an issue in the most recent version of Hubzilla, please recreate this issue over there. We're sorry for any inconvenience this will maybe give you.


⚠ Bofrost.be has degraded performance

In b4ea923, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1090 ms

Resolved: Bofrost.be performance has improved in 94338fc after 7 minutes.


Slow Invocation of Output Binding

In what area(s)?

/area runtime

What version of Dapr?

1.11.x

Expected Behavior
When invoking an azure service bus queue binding the message is sent immediately
Actual Behavior
'Randomly' when invoking the output binding, the request to dapr is sent, however the message isn't put on the queue for around 25 seconds, In normal behaviour, the request takes around 25ms to be processed and the output message is sent.
We are using the queue output binding - https://v1-11.docs.dapr.io/reference/components-reference/supported-bindings/servicebusqueues/.
The structure of our component is an input queue binding, some logic, and then an output binding is invoked
I can't see any pattern for when this happens.
with logging set to debug we can see the following

time="2023-10-24T02:53:12.303977164Z" level=info msg="gRPC API Called" app_id=cache-manager instance=ca-cache-manager-prod--bmgl30d-546b87bbb9-vh7bb method=/dapr.proto.runtime.v1.Dapr/InvokeBinding scope=dapr.runtime.grpc.api-info type=log useragent="grpc-dotnet/2.57.0 (.NET 7.0.12; CLR 7.0.12; net7.0; linux; x64) dapr-sdk-dotnet/v1.11.0+787cba6f4f4d28bd8ec09fcef9c854ef64d4361b" ver=1.11.2-msft-3
time="2023-10-24T02:53:17.65087391Z" level=debug msg="Renewing message locks for queue sbq-cacherequests-prod" app_id=cache-manager component="sbq-cacherequests (bindings.azure.servicebusqueues/v1)" instance=ca-cache-manager-prod--bmgl30d-546b87bbb9-vh7bb scope=dapr.contrib type=log ver=1.11.2-msft-3
time="2023-10-24T02:53:17.655956126Z" level=debug msg="Renewed message locks for queue sbq-cacherequests-prod for 1 messages" app_id=cache-manager component="sbq-cacherequests (bindings.azure.servicebusqueues/v1)" instance=ca-cache-manager-prod--bmgl30d-546b87bbb9-vh7bb scope=dapr.contrib type=log ver=1.11.2-msft-3
time="2023-10-24T02:53:37.650224778Z" level=debug msg="Renewing message locks for queue sbq-cacherequests-prod" app_id=cache-manager component="sbq-cacherequests (bindings.azure.servicebusqueues/v1)" instance=ca-cache-manager-prod--bmgl30d-546b87bbb9-vh7bb scope=dapr.contrib type=log ver=1.11.2-msft-3
time="2023-10-24T02:53:37.652841539Z" level=debug msg="Renewed message locks for queue sbq-cacherequests-prod for 1 messages" app_id=cache-manager component="sbq-cacherequests (bindings.azure.servicebusqueues/v1)" instance=ca-cache-manager-prod--bmgl30d-546b87bbb9-vh7bb scope=dapr.contrib type=log ver=1.11.2-msft-3
[02:53:38 INF] HTTP POST /dapr.proto.runtime.v1.AppCallback/OnBindingEvent responded 200 in 26584.9522 ms
time="2023-10-24T02:53:38.816501044Z" level=debug msg="Completing message 06626fcf-295a-41f7-baa2-e27ebbc4ce9b on queue sbq-cacherequests-prod" app_id=cache-manager component="sbq-cacherequests (bindings.azure.servicebusqueues/v1)" instance=ca-cache-manager-prod--bmgl30d-546b87bbb9-vh7bb scope=dapr.contrib type=log ver=1.11.2-msft-3

You can see here that between trying to invoke the output binding and getting a response, the input binding tries to renew all the locks - I don't know if this is related. Once the output binding is invoked, you can see the input message is then completed
I have configured the direction on the input and output bindings (but no other customisation).
We are running this component in an azure container app (https://learn.microsoft.com/en-us/azure/container-apps/dapr-overview)
Steps to Reproduce the Problem
I can't - this process is ok for 90% of requests - but every now and again we get this issue
Release Note
RELEASE NOTE: FIX Correct output bindings when invoking azure service bus binding

After upgrade into 1.12.0 in QA AKS cluster, I still got same problem


Infinite scroll never dismisses

Short description of the problem:
After scrolling ion-list to the bottom item, the spinner appear, the callback is triggered, the code calls infiniteScroll.complete(), but the spinner never is removed.
What behavior are you expecting?
After the calling of infiniteScroll.complete(), the spinner should be removed from the screen.
Steps to reproduce:
1 include the code bellow in the page controller
public doInfinite(infiniteScroll: InfiniteScroll) {
     console.log('Begin async operation');
     setTimeout(() => {
       console.log('Async operation has ended');
       infiniteScroll.complete();
     }, 500);
   }

2 include the code bellow in the html
 <ion-infinite-scroll (infinite)="doInfinite($event)">
   <ion-infinite-scroll-content></ion-infinite-scroll-content>
 </ion-infinite-scroll>

3 run the code.
Ionic Version:  2.x
Browser & Operating System:
tested on Android
Run ionic info from terminal/cmd prompt:
ionic info

Your system information:

Cordova CLI: 6.0.0
Gulp version:  Requiring external module ts-node/register
Gulp local:   CLI version 3.9.0
Ionic Version: 2.0.0-beta.1
Ionic CLI Version: 2.0.0-beta.17
Ionic App Lib Version: 2.0.0-beta.8
OS: Distributor ID:     Ubuntu Description:     Ubuntu 15.10
Node Version: v5.6.0

Infinite scroll was added in beta.2 and you are running beta.1. I don't think it should be working at all in beta.1, but I can't remember if it was there and just not fully working. Can you try updating to beta.2?

@brandyscarney , I did migrated to beta2! the ionic-cli is showing a wrong info!

@BugsBunnyBR Hmm maybe update the CLI to beta.19 to see if that will print the proper output (for the ionic info being wrong): npm install -g ionic@beta
If you're using beta.2, you should call complete() after the user scrolls to the bottom of the page (if you aren't passing a threshold value it will be within 15% of the bottom of the page. Once you are done letting the user scroll you will have to disable the infinite scroll by calling infiniteScroll.enable(false);.
It looks like to me you are triggering the infinite event and complete event over and over since there is no new data being loaded for the user to scroll down to. In one of our examples we have the doInfinite defined like this:
doInfinite(infiniteScroll: InfiniteScroll) {
  console.log('Begin async operation');

  getAsyncData().then(newData => {
    for (var i = 0; i < newData.length; i++) {
      this.items.push( this.items.length );
    }

    console.log('Finished receiving data, async operation complete');
    infiniteScroll.complete();

    if (this.items.length > 90) {
      infiniteScroll.enable(false);
    }
  });
}

So once the number of items exceeds 90 we stop triggering the infinite scroll. Does this help?

@brandyscarney
I tried to update to the latest ionic-cli, but the update process shown some errors, even though, the ionic info worked and shown the right version of ionic (beta.2). I reverted to ionic-cli 17(sudo npm install -g cordova ionic@2.0.0-beta.17).
I included the call to enable(false) and the infinite-scroll dismissed. Thanks.
How can I enable the infinite scroll again when the user hits the refresher ? I mean, how to get a reference to the InfiniteScroll object?

I think this is a bug with the CSS. When the state changes to just enabled to css should hide spinner. Normally you don't see this because the list is usually tall enough it below the fold.

People, I am using infinite scroll with refresher, if the user reaches the last page, I set the infinite scroll do enable(false), but I need to be able to enable it again when the user hits the refresher (because I only load the first page in the refresher and the user may again go to end of the list).
How to enable the infinite scroll again?
Or even better, if infinite scroll can listen the ion-list scroll event it would know if the user scrolled down and only show the spinner again if it happened.

I have the problem Adam says. When the list is sort and doesn't need scroll the spinner and loading text is visible. Thank you guys, you're doing a really good job.

Thanks for the help everyone!


Run dependabot at 0530

Run dependabot at 0530 so it's ready for first-thing-in-the-morning reviews.

Codecov Report

Merging #342 (0d703b4) into main (c8c57d1) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #342   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           16        16
  Lines         1128      1128
=========================================
  Hits          1128      1128

Flag
Coverage Δ

linux
100.00% <ø> (ø)

macos
?

windows
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


The answer is empty

Hello:
I am sending a GET sentence using Postman (as you can see on the
attached file), I received  the reply, but the list is empty.
The sentence is a GET:
https://trackerexpress.softwarexpress.pe/api-v2/task/route/list?hash=6acc59b139eba0bb9daaca70a0682035
The same answer is received when I use:
https://trackerexpress.softwarexpress.pe/api-v2/task/route/list?hash=fcfecdf3cb72827bd783f0809dad4794

Hi!
You have no created task routes. So what response you expect?
P.S. You should immediately change your password to invalidate sessions due to its compromentation. Session key aka hash it's a secret information.

Please, can you tell me the name of the task and forms API? Do I am calling them properly?


Add example to use with NotificationCenter.

How do one able to use NotificationCenter with Signals?
For example observing UIApplicationWillResignActiveNotification.

Singals can't do this. It's meant to replace custom notifications fired on NSNotificationCenter.


keep track storage bytes

Fix: AccountInfoOf was never inserted

get_storage in AccountDb trait is changed to include account_id and trie_id. This is for performance reason, the accound_id is enough and directAccoundDb could lookup in AccountInfoOf to get the actual trie.
To be honest I don't find it super beautiful and thought at first about making OverlayAccountDb a trait with a TrieId associated type, and this associated type would be an enum Overlayed or Underlying. But it brought complexity and find the current solution ok.

implement tracking of storage byte

TODO: rename current_mem_stored ¿

grumbled addressed + node impl version bumped

related to https://github.com/paritytech/substrate/pull/2154

will need another bump + rebuild.

Can we have a test for this?

done

Please open another PR for the test; merged primarily to increase chances to get this logic in 1.0.


Lambda to Firehose: Part 3

to: @austinbyers
cc: @airbnb/streamalert-maintainers
size: medium
references: #5, #280, #320
Background
Part 3 completes the Lambda and CLI development necessary to send logs directly to Firehose.  At a high level, it accomplishes the following:

Strip special characters from record keys: This enables records to be compliant with loading data into Hive.
Payload size checks: Ensure that the max payload size for Firehose is not surpassed.
Disable log types: For whatever reason, if a user does not want to send a top level log type (cloudwatch for example), it can be added to the config.  This will also remove the Firehose Delivery Stream to keep costs down

Testing
Still in progress, but I have been gradually testing on every commit by deploying to a test AWS account.  More on this soon.

Coverage increased (+0.1%) to 94.303% when pulling 6d10ecaa41bb69098e4e5f47ced6c165bfcb2732 on jacknaglieri-lambda-to-firehose-pt-3 into e9892ba83d1de07e9459b7de59c59c1f9536a6b6 on master.

Coverage increased (+0.1%) to 94.303% when pulling 7b009248d5bc3e89b481f73721f5c0d810a46121 on jacknaglieri-lambda-to-firehose-pt-3 into e9892ba83d1de07e9459b7de59c59c1f9536a6b6 on master.

Coverage increased (+0.1%) to 94.303% when pulling 78a4100bd41629fa0f33c6c6dd5526f3d542c17a on jacknaglieri-lambda-to-firehose-pt-3 into e9892ba83d1de07e9459b7de59c59c1f9536a6b6 on master.

Coverage increased (+0.1%) to 94.182% when pulling 39cd72f63decea5ab25f978b8d21eeddd985a3e5 on jacknaglieri-lambda-to-firehose-pt-3 into 073ad519873c65e29da02a63625dad9fdd5a2d73 on master.

Coverage increased (+0.09%) to 94.167% when pulling 397b577cccca03f52ba03f27e3402f4d336b1763 on jacknaglieri-lambda-to-firehose-pt-3 into 073ad519873c65e29da02a63625dad9fdd5a2d73 on master.

Coverage increased (+0.1%) to 94.201% when pulling 7ab82925df249d05afe90d3a23fbf0d20cb52858 on jacknaglieri-lambda-to-firehose-pt-3 into 2430664e1b46cbd8dce578494fbd1475b46781f3 on master.