JDX ADDING

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

i want to change my name symbol and logo

Mergefy has been remove

@Mergifyio refresh

@Mergifyio rebase

I can help stop wasting your time refer this: #12970

Go my friend but i got nothing for you

@zakimzf  Go my friend but i got nothing for you

@zakimzf   unsupported file modified: .mergify.yml

The configuration has been changed, queue and merge actions are ignored.

@Mergifyio update

@Mergifyio update

@Mergifyio rebase

@Mergifyio refresh

@Mergifyio refresh


update Readme.md

The preferred way to install react-native packages now is through react-native.  rnpm is deprecated.

Thanks, this change has already been made in another commit.


Media from https://d.atry.it/i/ not loading anymore due to expired certificate

What's the issue?
Images such as this one which appear in message embeds of commands like !glomp aren't loading and shown anymore.
Why is it happening?
The links used in the message embeds load images from https://d.atry.it. Since the certificate used on that domain has expired on 28th Mai 2019 requests using HTTPS are being denied and can't offer any images anymore.
How can this be solved?
Simply renew the certificate using any available software e.g. Certbot. Afterwards Discord should continue loading the images again after a few hours.

Fixed


Clearly define security properties

So far we have specified what the protocol should do, but we should also take the time to explicitly define what it should not do or allow.
This could take the form of a document or section entitled "security properties" which outlines the invariants we need to maintain.
Creating this content has the benefits of:

Guiding test code towards more "sad path" test cases.
Enabling auditors and verification engineers to better focus their efforts.
Helping us to write better, more defensive code and better understand what it means for the system to be secure.

This section of the SAFE protocol spec by Magmo is a good example. I can dig up some others I've seen in my travels.

Closing in favor of some more specific Linear tasks.


When repo card is visited, during hover action internal folders are not visible.

is it during hover action or onclick ?

sorry, during click


WatchOS Travis tests

I will merge this myself, it's only a step for the Travis build for the WatchOS scheme


Changed some copy

I din't think too much @saleiva. I like your proposal. What do you think?

+1

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)


Setup prettier on staging code in local

Run prettier on *.js & *.md files whenever user runs git add
Replace yarn with package-lock since yarn caches precommit state
entries in .gitignore are simplified
Sorted package.json via sort-package-json

Coverage remained the same at 99.531% when pulling ad05f7ebd2f0822235a267a4ca299f73711f168f on sivakumar-kailasam:prettier-setup into d53802a98db2234dcf1a9cdd1e435a67fd86264f on cowboyd:master.

Coverage remained the same at 99.531% when pulling 60c5b9b745b63d637a3ab458b1b0f6ac0cf545ec on sivakumar-kailasam:prettier-setup into d53802a98db2234dcf1a9cdd1e435a67fd86264f on cowboyd:master.

👍


Fix eserver

Fixes for uploading files into eserver: sha256 on fly, check for file existence.
Argument --image-size for eden setup to create large images of EVE

File size is definitely not better (I'd say worse) than sha

Yes, it is. We calculate sha in eserver. In this PR calculation done in the same time with uploading.


Implement backtrack for thick elements and fringes

Backtrack implemented for Bend (expanded model) and quadrupole. Other cases raise an error.
This is sufficient for LHC matching purposes that was the main use case for this feature.
A more complete implementation could be added in the future if needed


Change: replace 'Unlicense/MIT' with 'Unlicense OR MIT' to make the license SPDX compliant

Issue #15

@BurntSushi Can you review this change? The license field being non-SPDX compliant is also flagging in tools for me

It seems like an odd thing to warn about given that Unlicense/MIT was at one point valid. I don't really get why it matters. But anyway, I've published winapi-util 0.1.8 to crates.io with this fixed.


Viro360image and Viro360Video low quality on ViroPortal

When i try to use my own 360 images or videos on ViroPortal the quality is a bit low, it is a problem with the ViroPortal or do i need higher resolution images and videos?

We have not seen any issue with resolution on the ViroPortal component. We are not doing any processing of the 360 photo. Try a higher resolution if you can. Many site recommend 4k or 8k for 360 photo viewing.


added float support in num2words

Adds support for float values
❯ dig 0.1.words @localhost -p 5354

;; QUESTION SECTION:
;0.1.words.                     IN      A

;; ANSWER SECTION:
0.1.                    1       IN      TXT     "0.1 = zero Point one"

153.100.words  -> one hundred fifty three Point one
153.000.words -> one hundred fifty three Point zero
153.0001.words -> one hundred fifty three Point zero zero zero one

closes #45

one issue is still there
in ans section where it shows number and corresponding string, the number when input is 153.000 is showed as 153 (correct should be 153.0)
153.000.                1       IN      TXT     "153 = one hundred fifty three Point zero"


Add an example that uses extra build inputs

I don't think any of the examples does that and I am honestly struggling to do that. What I'm really after is to see some makeOverride usage in an example.

Whats missing from overrides.nix? This is documented in the readme as the place to look.
the pattern in i.e this override is a pretty good example of an extra buildinput

for an absolutely concrete example, how about the following,. This assumes you've a workspace, with all your crates in the ./crates subfolder.
default.nix
{pkgs}:

let
  rustPkgs = pkgs.rustBuilder.makePackageSet' rec {
    packageFun = import ./Cargo.nix;
    packageOverrides = pkgs:
      pkgs.rustBuilder.overrides.all
      ++ (import ./sys-crates.nix { inherit pkgs; });
  };

  crates = pkgs.lib.mapAttrsToList (n: _: n)
    (pkgs.lib.filterAttrs (n: v: v == "directory") (builtins.readDir ./crates));

in pkgs.lib.genAttrs crates (n: rustPkgs.workspace."${n}" { });

sys-crates.nix
{pkgs}:
let
   overrides = {
     # declare your set of overrides here
      foo = makeOverride {
        name = "foo";
        overrideAttrs = drv: {
          propogatedNativeBuildInputs = drv.propagatedNativeBuildInputs or [ ] ++ [ pkgs.openssl.dev pkgs.zlib.dev ];
        };
      };

      bar = makeOverride {
        name = "bar";
        overrideAttrs = drv: {
          propogatedNativeBuildInputs = drv.propagatedNativeBuildInputs or [ ] ++ [ pkgs.openssl.dev pkgs.zlib.dev ];
        };
      };

   };
in (map (key: pkgs.lib.getAttr key overrides) (builtins.attrNames overrides))

Hey, thanks for getting back to me. I believe I did a lousy job reading the README.md - everything that I need to know is alsoexplained in Common issues. Thanks for the example.


Add yarn parser

This change adds fully compatible yarn v1 parser. It only parses yarn.lock file at the moment it doesn't fetch license data

@mokhan can you take a look at this one please ? I don't know why some of the tests are failing


port pcmark to wa3

I've already created a PCMark workload that works on WA3. I will add a PR for it soon.

Cool. I currently have a partially working version that has the limitation of expecting PCMark to already be installed.

Added with https://github.com/ARM-software/workload-automation/pull/744


Proposal: GitHub username identities with the GitHub IdP

Summarizing a thread on Slack:
Currently, when users sign with Sigstore via the GitHub IdP (i.e., federated through Dex), they get an identity token that's bound to the primary email identity for their GitHub account. This is fine in many circumstances, but not fine in others:

Many GitHub users have multiple emails on their accounts, with the "primary" email not reflecting the identity they intend to sign with. For example: work emails, project-specific emails, etc. This can result in surprising scenarios for users; cf. https://github.com/sigstore/sigstore-python/issues/600#issuecomment-1638624381
Not all GitHub users mark their email address as public, and instead prefer to use one of GitHub's "noreply" aliases. However, signing with Sigstore through the GitHub IdP leaks their primary email; this poses anonymity and privacy concerns.
Conceptually, email identities coming from GitHub's IdP are a little "wonky": GitHub isn't itself an email service provider, so these identities reflect the result of GitHub's trust relationship with an email identity (which, in turn, is probably tied to a conventional email verification flow).

This raises the question: what Sigstore used GitHub usernames from the GitHub IdP, rather than emails? This presents some advantages, as well as some challenges:

Advantage: (1) and (2) are "resolved" in the sense that there are no emails anymore: users would instead be signing with an identity that looks like woodruffw!github.com.
Advantage: In many contexts, signing with a GitHub username makes more sense than with the user's underlying primary email: GitHub identities more closely track developer identities than emails do, since emails can be reused by companies, etc.
Challenge: How do we communicate this? User identities are a fuzzier concept than emails, and this would be the first instance of Sigstore actually using them (AFAIK).
Challenge: How do we prevent client breakage? Not all clients necessarily support user identities yet.

As noted by @bobcallaway, this would require some changes to Dex (configuration only, possibly): we'd need to switch to useLoginAsID: https://dexidp.io/docs/connectors/github/#configuration
cc @sethmlarson @wlynch @haydentherapper additionally as Slack convo members 🙂

@wlynch suggested another format which would be using GHs noreply format used for private emails<EMAIL_ADDRESS>This might be the best of both worlds - We get username, we get userid for immutability, the format is an email so no clients would need to be updated.
We'd need to patch our instance of Dex to include https://github.com/dexidp/dex/pull/2618, and would likely still need to configure Dex to add both username and user ID.

That solution is nice, although perhaps it should be all or nothing: having it be the user's primary email in some cases rather than the noreply email means that we lose the consistency of having a GitHub "username" identity that we can mangle out of the email identity.
Perhaps it's too much work, but IMO the ideal solution here is:

Adopt https://github.com/dexidp/dex/pull/2618 for the current GitHub IdP connector, plugging the privacy concern noted above;
Add another GitHub IdP connector that supports username identities. Visually separate this one in the OAuth flow to help users avoid confusing the two.

OTOH, having two separate GitHub IdP connectors might cause a lot of user confusion, even if visually separated.

Just adding that if we do start using this -- add a test to conformance so clients can begin to handle it.


Add parallelization

Parallel download
Parallel mixture creation

What's still slow is the metadata writing part (add_to_metrics_metadata or the for md_df in md_dic loop), haven't investigated further here.
I tested this once and it seems to work well.
Haven't given much time to properly formatting the code :-)

Hello, Thanks for the PR.
It looks good to me too.
I will run a quick test to double check and then I will merge.
Thanks again ! 🚀

I tried to run the recipe directly from your fork but it broke during the augmentation process.
I haven't look much into it yet but if you have any idea I take it :)

You should update tqdm :-)

You should update tqdm :-)

HI! How much the mix will require with this edit? Because I'm running it on colab and looks like infinite, after 24hours still working and is at 500GB of files


Dragging files into a new window is not possible

I was trying to replicate the following issue (https://github.com/atom/atom/issues/5590) when I stumbled upon this one.
Dragging a file from window to another was not possible unless a previous file was already open. I had to open another (even if Untitled) file to be able to drag the tab onto the other window. The dragging symbol appeared, instead of the crossed circle for invalid, but after releasing the mouse button nothing changed.
I don't know if this might be caused by the Theme I currently (and forever, it's beautiful) use. If it is I'll refer it to the proper channels.
Win 7 64bit
Atom -v181
Seti-UI (theme)

This has been fixed at some point and seems to work just fine now. Please re-open if it doesn't for you on a newer build of Atom.


Debug option returns strange data

Hello. I'm trying use guzzle 6.3 with debug=true
        $fp = fopen('php://temp', 'r+');

        /** @var \GuzzleHttp\Message\ResponseInterface $response */
        $response = (new HttpClient())->get('http://<IP_ADDRESS>', [
            'debug' => $fp,
        ]);

        fseek($fp, 0);
        var_dump(stream_get_contents($fp));

but I see only
<GET http://<IP_ADDRESS>> [CONNECT]
<GET http://<IP_ADDRESS>> [FILE_SIZE_IS] message: "Content-Length: 46" bytes_max: "46"
<GET http://<IP_ADDRESS>> [MIME_TYPE_IS] message: "text/html"
<GET http://<IP_ADDRESS>> [PROGRESS] bytes_max: "46"

I'm doing something wrong or this is a bug?

Seems OK. What do you expect?
BTW, it also seems that you don't have cURL extension installed, so Guzzle uses PHP's stream wrappers as the handler. In case you expect cURL style output...

BTW, it also seems that you don't have cURL extension installed,

Yes, you are right. I've installed it and see the info now. Thank you.
As I know the streams also allow receive response headers and body, I think will be great dump this data in debug mode (IMHO it will be more useful than [PROGRESS] bytes_transferred: "8192" bytes_max: "92984").

Well, cURL doesn't print the body in debug either.
But agree, more details would be better, of course. You can add them for the PHP's streams handler and send a PR ;)


Adding card schema file

@jasonjoh,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

files/actionable-messages/MessageCardSchema.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Configure persistent cache scope in v5

Feature request
As I test the effect of persistent cache in webpack v5, it involves some issue when using DefinePlugin to inject process.env
I tried a entry js like:
import 'lodash';

console.log(process.env.TEST_VALUE);

Production mode build cause 240ms the first time, and 50ms when cache is enabled, however when process.env.TEST_VALUE is changed the output still remains the same.
TEST_VALUE=123 webpack # cuases 240ms
TEST_VALUE=456 webpack # causes 50ms but outputs 123

I can generate cache.version depending on process.env but this cache version is way too dynamic, also I have something like git revision or timestamp into javascript via DefinePlugin so the persistent cache can never hit.
On the other side, I can definitely tell that only src javascript are  affected by those dynamic context, all third party node modules can be cached stably, but I can't find a way to configure persistent cache only on my src scripts.

What is the expected behavior?
I'd like to be able to specify only some scripts being persistent cached depending on their path and content.
What is motivation or use case for adding/changing the behavior?
This can boost build performance even some of the scripts is depending on dynamic context like process environments, build revisions, timestamps, feature flags, etc....
How should this be implemented in your opinion?
Try to add a Condition to cache configuration like:
{
  cache: {
    test(path, content) {
      return path.includes('node_modules/') || !content.includes('process.env.');
    }
  }
}

Both path and content are required, content should be the raw content without any loader or plugin involved.
Are you willing to work on this yourself?
possibly not, I'm not familiar with webpack

Thanks for issue and feedback

Hmm I see. Thanks for trying the alpha version.
For values coming from outside the config file you have multiple options:

cache.name: Create a separate cache for each value
cache.version: Invalidates the cache when value changes
If the value is from another module: cache.buildDependencies
And these additional options when it's used with the DefinePlugin

If the value is from another file: DefinePlugin.runtimeValue(() => ..., [ "/path/to/file" ])
DefinePlugin.runtimeValue(() => ..., true): Opt-out from caching for modules using this value.

For your cases:
TEST_VALUE=123
If there are multiple constant options for this value: cache.name.
If the value changes infrequently: cache.version
If the value is always different or changes very often: DefinePlugin.runtimeValue(() => ..., true)
git revision
DefinePlugin.runtimeValue(() => getRev(), [path.resolve(__dirname, ".git/HEAD"), path.resolve(__dirname, ".git/info/refs")])
timestamp
DefinePlugin.runtimeValue(() => Date.now(), true)
feature flags
cache.name

The complete syntax is:
new DefinePlugin({
  TEST_VALUE: DefinePlugin.runtimeValue(() => JSON.stringify(process.env.TEST_VALUE), true)
})

It probably make sense to allow separate cache name or versions per module to make it easier to add dependencies for modules.
I could image the following additions to make it easier to use the persistent cache:
module.rules: [
  {
    test: ...,
    cache: {
      // This would allow to handle loader options that depend on addition external values
      version: "...",
      name: "..."
    }
  }
]

DefinePlugin.dynamicValue(() => ..., {
  cacheVersion: "...",
  cacheName: "...",
  fileDependencies: [...], // <- that's what's currently provided as second argument
  contextDependencies: [...]
})

This would allow config like this:
const TEST_VALUE = process.env.TEST_VALUE;

module.exports = {
  module: {
    rules: [
      {
        test: /\.js$/,
        loader: "babel-loader",
        options: {
          envName: TEST_VALUE
        },
        cache: {
          name: TEST_VALUE
        }
      }
    ]
  },
  plugins: {
    new webpack.DefinePlugin({
      TEST_VALUE: DefinePlugin.dynamicValue(() => TEST_VALUE, {
        cacheName: TEST_VALUE
      })
    })
  }
}

just my two cents, additional cache setup on loaders looks very complex setup, maybe good manual helps better understand this to new developers


Difference between backbones

Hi @themattinthehatt,
Could you please explain main difference between these backbones: resnet50_human_jhmdb | resnet50_human_res_rle | resnet50_human_top_res ?
Additionally, please share the path for these backbones models.
Many Thanks in advance,
Cheers, Reet :)

Hi, Thank you for the question.
The team is currently on vacation and will be back next week.

@ReetKaur15 you can find more detailed descriptions of the different backbones here: https://lightning-pose.readthedocs.io/en/latest/source/user_guide/config_file.html#model-training-parameters
and you can find the hard-coded paths to the network weights here: https://github.com/paninski-lab/lightning-pose/blob/47ee289110fb2ef2519091b49a5658fab07b4bf4/lightning_pose/models/backbones/torchvision.py#L18
please let me know if you have any further questions!

Hi @themattinthehatt , Thanks for the response.
No information about "resnet50_human_top_res" is given. Could you please share more details about it?

Ah yes I see I missed that one, apologies.
resnet50_human_res_rle: a regression-based ResNet-50 pretrained on MPii dataset (Andriluka et al 2014, 2D Human Pose Estimation: New Benchmark and State of the Art Analysis)
resnet50_human_top_res: a heatmap-based ResNet-50 pretrained on MPii dataset (Xiao et al 2018, Simple Baselines for Human Pose Estimation and Tracking)
Both will be compatible with all model types (supervised and context, with or without unsupervised losses) but I have not tested either of them so cannot say more. If you end up testing both out I'd be curious to hear how they compare!

@themattinthehatt, Thank you for quick response.
As per our previous discussion, I trained the vision transformer with "vit_b" architecture by passing the same parameters and hyperparameters as set in your code., given below. Unfortunately, the results are not much good as compared to the ResNet-152 backbone.
elif "vit_b_sam" in backbone_arch:
ckpt_url = "https://dl.fbaipublicfiles.com/segment_anything/sam_vit_b_01ec64.pth"
state_dict = torch.hub.load_state_dict_from_url(ckpt_url)
new_state_dict = {}
for key in state_dict:
new_key = key.replace('image_encoder.', '')
new_key = new_key.replace('mask_decoder.', '')
new_state_dict[new_key] = state_dict[key]
encoder_embed_dim = 768
encoder_depth = 12
encoder_num_heads = 12
encoder_global_attn_indexes = (2, 5, 8, 11)
prompt_embed_dim = 256
finetune_image_size = image_size
image_size = 1024
vit_patch_size = 16
base = ImageEncoderViT_FT(
depth=encoder_depth,
embed_dim=encoder_embed_dim,
img_size=image_size,
finetune_img_size=finetune_image_size,
mlp_ratio=4,
norm_layer=partial(torch.nn.LayerNorm, eps=1e-6),
num_heads=encoder_num_heads,
patch_size=vit_patch_size,
qkv_bias=True,
use_rel_pos=False,
global_attn_indexes=encoder_global_attn_indexes,
window_size=14,
out_chans=prompt_embed_dim,
)
I am using 8300 frames from 83 videos (100 frames/video) and testing on unseen baby videos, training with 1569 epochs (around 700,000 iterations), batch_size=16.
To improve the results, I tried to change the vit_patch_size = 8 and window_size = 7. It pops up an error that I cannot adjust these hyperparameters because the pre-trained model was trained with patch size 16.
What hyperparameters or parameters can be adjusted to enhance the results? Alternatively, could the results be limited due to insufficient training data?

Yeah unfortunately some of those ViT parameters are fixed given the pretrained weights we're using.
We have also found the ViT doesn't clearly outperform resnets on the handful of datasets we've tried it on.
Part of the issue might be in the training - the learning rate and learning rate scheduler are somewhat optimized for resnets, and might need to be adjusted for the ViT. We played around with this a bit (changed the learning rate, tried cosine annealing) but again weren't able to get any easy wins.
Insufficient training data could certainly be a problem, given the large number of parameters in the transformer. How does the validation loss look during training, does it continue to decrease even after 1k+ epochs? There might be some overfitting going on.


this platform does not support websocket protocol

For win7, this platform does not support websocket protocol. Is there a solution for this package?

Sorry for the late reply. This repository was created during my graduate program and was not actively maintained after.  I no longer have an environment or up-to-date ROS/.NET knowledge to properly review PRs, add new features or fix bugs so please fork the repository if you would like to add new features or fix bugs.
To avoid future confusion, I will add a note to the readme and archive the repository.


Bad broadcasting when multiplying sparse and dense tensors

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 64bit
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: NA
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): b'v1.12.0-rc2-3-ga6d8ffae09' 1.12.0
Python version: 3.6.7 Anaconda 64bit
Bazel version (if compiling from source): NA
GCC/Compiler version (if compiling from source): NA
CUDA/cuDNN version: NA
GPU model and memory: NA

Describe the current behavior
When multiplying (* operator) a sparse tensor S with rank N with a dense tensor D with rank M, where N > M and S and D have different but broadcast-compatible shapes, then TensorFlow attempts to perform the multiplication producing and incorrect sparse tensor as a result with the shape of S.
Describe the expected behavior
TensorFlow should refuse to perform a sparse-dense multiplication that requires broadcasting. This is the case when both tensors have the same rank or the rank of the sparse tensor is less than the rank of the dense tensor, but not when the rank of the sparse tensor is greater.
Code to reproduce the issue
import tensorflow as tf

with tf.Graph().as_default(), tf.Session() as sess:
    a = tf.reshape(tf.range(12), [3, 4, 1])
    b = tf.SparseTensor([[0, 0, 1, 0], [0, 0, 3, 0]], [10, 20], [1, 1, 4, 2])
    c = a * b
    print(c.shape)
    # (1, 1, 4, 2)
    print(sess.run(tf.sparse.to_dense(c)))
    # [[[[ 0  0]
    #    [10  0]
    #    [ 0  0]
    #    [60  0]]]]

Other info / logs
NA

@javidcf I added a PR #24098 for the fix. Please take a look.

@yongtang Thanks, that seems to do a better job at validating the input, although to be honest I'm not familiar enough with the source to know if any other changes are needed. Only one thing, where the comment says all dims in lhs is smaller or equal, I think it should be all dims in lhs is greater or equal? Or maybe I'm misunderstanding something...
Besides that, I just noticed that these errors are only raised on evaluation. I suppose it would be convenient to raise an error on graph construction when detectable too... but that is a different issue really, for binary sparse ops in general - and also, if sparse broadcasting is planned to be supported at some point in the future, maybe it's not worth the effort.

@javidcf Thanks. You are right and I have updated the PR for the comment. To raise the error on graph construction might requires more and sometimes the dims are not known at the construction time. I will take a look and create a separate PR if possible.


Prevent postinst from starting the daemon unless it is already running…

… Consolidate init branching.

LGTM

LGTM


docs: Resource documentation databricks.Permissions showws HCL code

Describe what happened
When reading the documentation for databricks.Permissions, the page shows broken HCL code.
Sample program
n/a
Log output
n/a
Affected Resource(s)
n/a
Output of pulumi about
n/a
Additional context
n/a
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Thanks for reporting this - we will take a look at fixing this ASAP.


Headers send twice in some cases

When my model returns nothing for model.getAccessToken because the token doesn't exist in the db, I get two responses. The offending lines are here: https://github.com/oauthjs/express-oauth-server/blob/master/index.js#L49-L52
    .catch(function(e) {
        return handleError(e, req, res); // sends an error response
    })
    .finally(next); // executes after .catch and sends another response

I believe it should be the following, but I am not 100% certain since I'm not very experienced with the flow of promises.
    .tap(next)
    .catch(function(e) {
        return handleError(e, req, res);
    });

Pretty sure this is fixed by getting rid of the .finally(next) statements


Change topic /camera/rgb/image_raw/compressed  to /camera/rgb/image_raw

Hi Sir,
ROS sharp release 1.3
It worked well before when I subscribed topic /camera/rgb/image_raw/compressed
Purpose: change the image on image receiver to a colour one
Topics to be subscribed by Unity from ROS using ROS#:
Change this topic /camera/rgb/image_raw/compressed (sensor_msgs/CompressedImage) to /camera/rgb/image_raw(sensor_msgs/Image)
I've noticed that both sensor_msgs/CompressedImage and sensor_msgs/Image are built in message type. So I don't think I need to create my own type of message.

What I did: change all SensorCompressedImage to SensorImage in the ImageReceiver C# file as shown below in order to receive topic /camera/rgb/image_raw
using System;
using UnityEngine;

namespace RosSharp.RosBridgeClient
{
    [RequireComponent(typeof(MeshRenderer))]
    public class ImageReceiver : MessageReceiver
    {
        public override Type MessageType { get { return (typeof(SensorImage)); } }

        private byte[] imageData;
        private bool isMessageReceived;

        private MeshRenderer meshRenderer;
        private Texture2D texture2D;

        private void Awake()
        {
            MessageReception += ReceiveMessage;
        }
        private void Start()
        {
            texture2D = new Texture2D(1,1);
            meshRenderer = GetComponent<MeshRenderer>();
            meshRenderer.material = new Material(Shader.Find("Standard"));
        }
        private void Update()
        {
            if (isMessageReceived)
                ProcessMessage();
        }
        private void ReceiveMessage(object sender, MessageEventArgs e)
        {
            imageData = ((SensorImage)e.Message).data;
            isMessageReceived = true;
        }

        private void ProcessMessage()
        {
            texture2D.LoadImage(imageData);
            texture2D.Apply();
            meshRenderer.material.SetTexture("_MainTex", texture2D);
            isMessageReceived = false;

Then I rebuilt the image receiver game object in Unity by following the video tutorial. In rqt_graph it shows ROS is publishing /camera/rgb/image_raw through ROSbridge successfully. However, I don't get any image on the plane. (It worked before when I subscribed topic /camera/rgb/image_raw/compressed )

Can you please kindly walk me through? I've tried a lot of time and can't figure it out. Thanks a lot

Most likely the method Texture2D.LoadImage() does not work with the data contained in sensor_msgs/Image.
sensor_msgs/Imagecontains bitmap data whereas LoadImage() works with png and jpg.
You will have to find a way to import bitmap files into Unity.

Hi @stevensu1838 ! Any update on the problem you were having?
Please give a short update and close the issue is your problem is solved.


No subtitles found for video started via "Torrent History"

Starting a movie through the "Torrent history" category causes the movie name "History" to be passed on the the Kodi subtitle search instead of the filename/hash. This applies to any subtitle provider i have tested (Open subtitles and Subscene).
I assume this is a bug?

@hejman When you open something from torrent history - you open a torrent file, that has no information about it's content, only filenames.
I don't know of any method of instructing Kodi about name for subtitles search.

@elgatito maybe add an option to link an existing torrent in the torrent history to a movie or TV show episode.
For example, episode of a random TV show - right click on the episode - in the menu click attach torrent from history - show a list of all the torrent history - choose a torrent - play the torrent with the right details.
Hope that make sense,
Thanks.

When you open a stream with Elementum Kodi treats it as a file, and when you open Browse dialog - it just tried to get root of this file store, which is a main page of Elementum.
I have no ideas how to change it.

Sorry but I don't really understand what you mean.
The file of the stream can't be modified to contain the relevant information about the episode or movie you attach?
Then how when you choose a stream from Burst, it contains all of the information?
I just throw things out there and hope it works out for you.. I don't have the experience in kodi development.

Then how when you choose a stream from Burst, it contains all of the information?

Because when you search something - you know what you search.
If you run something from torrent history - you have only torrent, you don't know who added it.

Yes I understood that.
Why can't the user choose a specific movie from the catalog and attach it to a torrent from the torrent history?
It's the user responsibility to choose the right movie but it is a solution.
That way you have the information about the movie and you have the torrent from the history.

Why can't the user choose a specific movie from the catalog and attach it to a torrent from the torrent history?

Because it is not implemented. Source code is open and you can submit a Pull Request to fill that gap.

I was asking this hypotheticaly.
As I said, I don't have the experience to do something like this.
It's just a suggestion for you.

should be available in next elementum release.
also please note how it works: https://github.com/elgatito/context.elementum/pull/13#issue-615408894


Abstract methods must be declared or implement

Hi @flugger,
Thanks for a fantastic package!
Me using it with Laravel 5.4 and in my base test case, I'm pulling your helper trait MakesApiRequests. However, that gives me the error:
[Symfony\Component\Debug\Exception\FatalErrorException]
  Class AgentsCreateTest contains 2 abstract methods and must therefore be declared abstract or implement the remaining methods (TestCase::seeJsonSubset, T
  estCase::decodeResponseJson)

Is that something I should implement myself?
Thanks,
Evgenii

Hi!
Thanks for the good words. Laravel 5.4 changed the testing API slightly. They released a package to keep backwards compatibility: https://github.com/laravel/browser-kit-testing
I'm working on a v2 which will include some improvements to this, but in the mean time you will need to bring in the package above for the trait to work.

Hi @flugger,
Thanks for your reply, I'll give it a try!


PT-1540 | fix: remove preview button from enrolment approval/declining

Description :sparkles:
GraphQL schemas were updated using:

REACT_APP_API_URI=https://kultus.api.test.hel.ninja/graphql <-- TESTING BACKEND
REACT_APP_CMS_URI=https://kultus.hkih.stage.geniem.io/graphql <-- STAGING CMS

fix: remove non-working preview from (Approve|Decline)EnrolmentModal
also:

remove the now unused props related to preview functionality
update test snapshots

refs PT-1540
refactor: remove email preview as dead code, update GraphQL schema
also:

use cross-env with "yarn codegen" to make it work also on Windows
fix "yarn build" [1] after GraphQL schema update by taking care of
organisation.persons being possibly null or undefined in getPersons
function

[1]:
Failed to compile.

TS18049: 'organisation.persons' is possibly 'null' or 'undefined'.
	 7 |   organisation: OrganisationNodeFieldsFragment | null | unde...
	 8 | ): PersonFieldsFragment[] =>
      >  9 |   organisation?.persons.edges.map(
	   |   ^^^^^^^^^^^^^^^^^^^^^
	10 |     (edge) => edge?.node as PersonFieldsFragment
	11 |   ) || [];
	12 |

refs PT-1540
Issues :bug:
Closes :no_good_woman:
PT-1540
Related :handshake:
Testing :alembic:
Has not been tested except for the updated automated tests,
looking at the snapshots the preview buttons were removed from them,
which looks like what should happen
Automated tests :gear:
Manual testing :construction_worker_man:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

kultus-admin-ui branch is deployed to platta: https://kultus-admin-ui-pr321.dev.hel.ninja :rocket::rocket::rocket:

kultus-admin-ui branch is deployed to platta: https://kultus-admin-ui-pr321.dev.hel.ninja :rocket::rocket::rocket:


Can't define a table from a dict (in dev version)

Trying:
ds.Table({"A" : range(0,10)})

Throws error:
AssertionError                            Traceback (most recent call last)
<ipython-input-71-01d080f4ca95> in <module>()
      1 ds.__version__
----> 2 ds.Table({"A" : range(0,10)})

/opt/conda/lib/python3.4/site-packages/datascience/tables.py in __init__(self, columns, labels, formatter)
    236         columns = columns if columns is not None else []
    237         labels = labels if labels is not None else []
--> 238         assert len(labels) == len(columns), 'label/column number mismatch'
    239
    240         self._num_rows = 0 if len(columns) is 0 else len(columns[0])

AssertionError: label/column number mismatch

I'm on:
ds.__version__
'0.3.dev21'

@choldgraf confirms that this command throws no errors on v0.3.14

Ah I think I see the problem - when I wrote the guide I was using an old version of datascience. It has since been updated and the proper way to construct a table using a dictionary of col: values is with the from_columns_dict constructor. E.g.:
ds.Table.from_columns_dict({"A" : range(0,10)})

@choldgraf Thanks!  Not sure why the constructor isn't selected by the argument class (any longer), but good to know that works.

Yeah, unfortunately we made this breaking change... (I wonder who
merged in that
pull request https://github.com/dsten/datascience/pull/117)
On Wed, Dec 16, 2015 at 1:35 PM Carl Boettiger<EMAIL_ADDRESS>wrote:

@choldgraf https://github.com/choldgraf Thanks! Not sure why the
constructor isn't selected by the argument class (any longer), but good to
know that works.
—
Reply to this email directly or view it on GitHub
https://github.com/dsten/datascience/issues/161#issuecomment-165247297.

;-) No worries, don't think anyone expects things to be stable while development is on-going, just trying to grok the logic.
I do see a lot of the pandas documentation use this dict style initialization, which is kind of nice as it is easier to visually match up the labels to the columns than when using the [values], [labels] format; so that's why I was hoping to use it here (likewise it parallels the R construction of colname = values).


Error on remove imports with TSX

Hey there, i see some recent activity so i post my issue here :

My guess is a fail on typescript or jsx.
File :
import { PlusCircleOutlined } from "@ant-design/icons";
import { useQuery } from "@apollo/client";
import { Card } from "antd";
import type { NextPage } from "next";
import { useTranslation } from "react-i18next";

import { Title } from "@/_layout/Title";
import { ButtonLink } from "@/ButtonLink";
import { LoadingOrError } from "@/LoadingOrError";
import { NoDataFound } from "@/NoDataFound";
import { VideoCard } from "@/VideoCard";
import { Query, QueryFindFirstUserArgs } from "~/@types/generated/graphqlTypes";
import { whereIdIs } from "~/front/gql/helpers/graphql.helpers";
import { ONE_USER_QUERY } from "~/front/gql/queries";
import { useSession } from "~/front/hooks";

const colProps = {
  md: { span: 12 },
  span: 24,
  xl: { span: 6 },
};

const VideosPage: NextPage = () => {
  const { user } = useSession();
  const { t } = useTranslation();

  const { loading, data, error } = useQuery<Query, QueryFindFirstUserArgs>(
    ONE_USER_QUERY,
    whereIdIs(user?.id),
  );

  if (loading || error) {
    return <LoadingOrError error={error} loading={loading} />;
  }
  const userData = data?.findFirstUser;
  if (!userData) {
    return <NoDataFound dataName={String(t("pages.videos.no-datas"))} />;
  }

  return (
    <>

    </>
  );
};

export default VideosPage;

2022-02-05 12:26:33.168] [exthost] [error] SyntaxError: 'Const declarations' require an initialization value. (27:73)
    at Object._raise (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:119:2828)
    at Object.raiseWithData (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:119:2769)
    at Object.raise (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:119:2237)
    at Object.parseVar (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:26148)
    at Object.parseVarStatement (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:22659)
    at Object.parseStatementContent (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:15356)
    at Object.parseStatement (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:14313)
    at Object.parseStatement (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:121:40542)
    at Object.parseBlockOrModuleBlockBody (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:24575)
    at Object.parseBlockBody (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:24421)
    at Object.parseBlock (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:24135)
    at Object.parseFunctionBody (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:5866)
    at /Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:121:39846
    at Object.forwardNoArrowParamsConversionAt (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:121:43861)
    at Object.parseFunctionBody (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:121:39801)
    at Object.parseArrowExpression (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:5266)
    at Object.parseParenAndDistinguishExpression (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:88794)
    at Object.parseParenAndDistinguishExpression (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:121:55371)
    at Object.parseExprAtom (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:82133)
    at Object.parseExprAtom (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:5294)
    at Object.parseExprSubscripts (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:76077)
    at Object.parseUpdate (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:75665)
    at Object.parseMaybeUnary (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:75261)
    at Object.parseExprOps (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:72396)
    at Object.parseMaybeConditional (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:71963)
    at Object.parseMaybeAssign (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:71230)
    at Object.parseMaybeAssign (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:121:54310)
    at /Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:70785
    at Object.allowInAnd (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:11333)
    at Object.parseMaybeAssignAllowIn (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:123:70765)
    at Object.parseVar (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:26054)
    at Object.parseVarStatement (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:22659)
    at Object.parseStatementContent (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:15356)
    at Object.parseStatement (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:14313)
    at Object.parseStatement (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:121:40542)
    at Object.parseBlockOrModuleBlockBody (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:24575)
    at Object.parseBlockBody (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:24421)
    at Object.parseProgram (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:13187)
    at Object.parseTopLevel (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:12945)
    at Object.parseTopLevel (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:121:57785)
    at Object.parse (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:43635)
    at Object.ioe [as parse] (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:124:44217)
    at Object.Oge [as parse] (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:2304:6728)
    at hL (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:17:889)
    at M_ (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:2304:7360)
    at q_ (/Users/davidbabel/.vscode/extensions/kuscamara.remove-unused-imports-1.0.4/dist/extension.js:2304:7610)
    at r._executeContributedCommand (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:95:31080)
    at r.$executeContributedCommand (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:95:31622)
    at t._doInvokeHandler (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:99:13802)
    at t._invokeHandler (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:99:13486)
    at t._receiveRequest (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:99:12148)
    at t._receiveOneMessage (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:99:10826)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:99:8922
    at u.fire (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:66:1712)
    at r.fire (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:74:18942)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:114:34341
    at u.fire (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:66:1712)
    at r.fire (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:74:18942)
    at n._receiveMessage (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:74:23523)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:74:21057
    at u.fire (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:66:1712)
    at w.acceptChunk (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:74:15770)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:74:14900
    at Socket.T (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:114:13813)
    at Socket.emit (events.js:315:20)
    at Socket.EventEmitter.emit (domain.js:467:12)
    at addChunk (internal/streams/readable.js:309:12)
    at readableAddChunk (internal/streams/readable.js:284:9)
    at Socket.Readable.push (internal/streams/readable.js:223:10)
    at Pipe.onStreamRead (internal/stream_base_commons.js:188:23) remove-unused-imports.main {"value":"kuscamara.remove-unused-imports","_lower":"kuscamara.remove-unused-imports"}

Hi! Thank you for reporting the issue. I'll try to look at it next week. In the meantime, could you share a repo where the error can be reproduced?

Hi @DavidBabel
It is fixed with the last version (1.0.5).
Thanks for using it and reporting the issue :)


How to apply Recipe when controlling BlendShape with an avatar created with DAZ?

Describe the bug
A clear and concise description of what the bug is.
Environment (please complete the following information):

Unity Version: 2020.3.25
UMA Version: 2.12
Rendering Pipeline: URP
Platform Build (if not editor): editor

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

After watching Youtube DAZ to UMA, I am slowly customizing my avatar.
After watching the tutorial and making almost everything, I made and put on clothes and tried changing the Buff value of BlendShape, but the BlendShape value is not applied to the clothes I wear.
Is there a simple way to solve this problem?
Expected behavior
A clear and concise description of what you expected to happen.
When adjusting the BlendShape Buff value, it is hoped that the buff will be applied to the worn clothes.
The BlendShape for the worn clothes also has a BlendShape value of the same name
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

This is done using Blendshape DNA. Create a Blendshape DNA, and then modify the DNA will affect all Blendshapes with that DNA in the mesh.  If one of the submeshes does not change, then that means the model for that submesh that was imported did not have blendshapes imported. Tested working in RC7.


Fix fungibles unbalanced

Closes: #821
https://github.com/paritytech/substrate/blob/696bdc67af5fd83aca318263dbd405593b043ba3/frame/support/src/traits/tokens/fungibles/balanced.rs#L182-L239

Codecov Report

Merging #823 (3c9f1d1) into master (2c48b62) will increase coverage by 0.13%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #823      +/-   ##
==========================================
+ Coverage   78.24%   78.37%   +0.13%
==========================================
  Files         102      102
  Lines       10129    10191      +62
==========================================
+ Hits         7925     7987      +62
  Misses       2204     2204

Impacted Files
Coverage Δ

tokens/src/tests_events.rs
100.00% <ø> (ø)

tokens/src/lib.rs
84.18% <100.00%> (+0.03%)
:arrow_up:

tokens/src/tests_fungibles.rs
98.94% <100.00%> (+0.48%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

See my comments on the related changes in the Substrate PR


Error: Return form initializer without initializing all stored properties (Is this only me :(

Property status and msg should have initial value.

Thanks @devxoul

Oh thank you so much. 👍 I dint even thought abt that..


FIX Retain value that failed validation on client side

Issue https://github.com/silverstripe/silverstripe-elemental/issues/764
Requires this PR https://github.com/silverstripe/silverstripe-admin/pull/1246 to get the ValidationResult client-side
Video

https://www.youtube.com/watch?v=NvWJHFSHijo
custom "required" validator on content block title
RequiredFields validator on a custom page field
Page has multiple tabs that show invalid icons

Todo:

behat test

yep

is technically possible, though its implementation is honestly quite hackish.  Also I think it's wrong to put field errors inside the general form error.  Also it will be confusing for users that have 2 or more elemental areas, I think warnings above the relevant elemental area would be better.

can do

that's doable

highlighting blocks with some sort of visual indication which have error

Unfortunately not possible without very hacky code, because it would be jquery updating a rendered react component

"I think warnings above the relevant elemental area would be better."

If there wasn't another competing error at the top of the page which is kinda repeating information, and also close in proximity perhaps. Watching the video it's very in your face when you have multiple errors showing with multiple error notifications as well, and then there are instances where the field shows an error too. Just think this might be doing more harm than good having so many errors present.

I understand there might be technical limitations which mean the experience is sub-optimal especially with #4 not being possible.
I think we absolutely need an error to be sticky near the top incase people might miss the fading toast notifications. An idea (might not be the right thing?), we might have the possibility to use sticky toasts with a dismiss (x). Then the top alert info could be transferred into the toast. I don't think we've used the Toast dismiss yet but this could be an okay scenario to use it.

Also I think it's wrong to put field errors inside the general form error.

This was actually seen as best practice years ago when lots of errors can present (pretty sure it still would be), means you can assess all the issues in one place to get a sense of what needs to be done. Helps to prevent people correcting one field and then try to save the form again but only to get the following issue for each repeated save.
Keen to hear your thoughts on moving the top alert info to a sticky toast @emteknetnz


update to git annex 8.20211231

Checklist

[X] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[X] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge/core After latest re-render, getting strange build errors early in the build complaining about issues with the python package being outdated -- but python isn't a dependency of this recipe, so not sure why it needs python at all (let alone the python 2.0 it seems to be trying to install).   Any suggestions on how to debug this?  Thanks a lot!

Try turning off mambabuild by setting the key in the conda-forge.yaml to false.
@wolfv when you are back online can you look at this solve?

the problem seems to come from dbus requiring an unconstrained glib. That glib in turn requires an unconstrained python.
It ends up there because openssh hasn't been built with openssl 3.* and the older python versions don't require openssh at all.
It resolves fine with libffi=3.4 (not 3.3) so maybe that would be the best way to fix this (migration seems to be open).

The libffi 3.4 migration PR had conflicts, I asked the bot to open https://github.com/conda-forge/git-annex-feedstock/pull/135 for you. Basically that one should be merged first and then this one.


Not able to collect docker logs for OTG-HW

This is continuation of this PR  https://github.com/open-traffic-generator/fp-testbed-nokia/pull/17 though I feel that fix will regress current intention of this script which is to pull logs for all three dockes.
So, I am proposing this fix.

close this PR as open it in another repo https://github.com/open-traffic-generator/fp-testbed-template/pull/8


Port out of range error when used in host string

[x] I have the latest version of sake
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] other

Shell

[x] Bash
[ ] Zsh
[ ] Fish
[ ] Powershell
[ ] other

Version: 0.12.0

Problem / Steps to reproduce

~ sake.yaml ~
servers:
  myserver1:  ## not working (error: strconv.ParseInt: parsing "55555": value out of range)
    host: user@<IP_ADDRESS>:55555
  myserver2:  ## working
    host<EMAIL_ADDRESS>    port: 55555

sake check

Acknowledged, will push fix today.

Fixed with v0.12.1


Cannot find any license information

I suggest to either use the Github WebUI or to add a LICENSE.txt/.md and clearify the licensing situation.
Whether you use a viral GPL or a less limiting BSD/MIT/Apache will decide about future developments, re-use of this gem.

@PixelPartner @jhc-at-home  ... Just added a BSD 3 license, I think using the same license as the elm-compiler should be ok ;) ... I did not add a license so far, since I am absolutely not an expert on this topic, but LiaScript was always open-source, but without a license ... But you can suggest me license ;) ... Actually I would prefer the "NoEvil" license, I did some stuff in robotics and ROS, but now I am shocked that the military-sector is using all open-source algorithms from ROS for their purposes (of course not my buggy code)... Thus, it should not be too open, just kidding ...

@andre-dietrich This bug can be closed now I think? My personal preference would've been for a GPL copyleft license but BSD-3 clause is also perfectly fine for our purposes.

@jhc-at-home If you need some help on implementing a template for any specific javascript library, just create an issue... one of the next steps, is to create a documentation on how to define custom liascript extensions ...


AUI-2037

@peterborkuti, please retest this. If all is good, you can send this to @jonmak08.
There was a few issues with your CSS. I also removed some source formatting changes, to keep it in line for 2.0.x.

Hi @blzaugg,
This is a screenshot from my backport:

And this is after thesting your branch:

I think, there are three issues here (which were fixed in my backport) see the yellow circles:

unaligned checkbox
not red bordered not-unique value fileds
alert message is not red

May I missing something? Did I create a bad build or did I used the wrong branch?

I was using $ grunt all to build. I didn't have these CSS issues. I will test it again.
One note, when I built yours, it did not look like the first image.

 I'm guessing you're missing some other CSS as well.
@jonmak08 can you test this?

Hey guys,
This is what I see:
Chrome:

Firefox:

Safari:

Hey guys,
This is what I see from @peterborkuti's original pull request (https://github.com/blzaugg/alloy-ui/pull/63) to @blzaugg.
Chrome:

Firefox:

Safari:

Hi, thanks for testing.
There should be something bad here in my PC when I build Alloy, probably with compass.
I am checking this.
Because of the time limit, could you left me out from the process and could @jonmak08 take care of this instead of me?
Thank you in advance
Péter

I forwarded it to Jon:
https://github.com/jonmak08/alloy-ui/pull/331


Docker Compose setup simplification

I understand what bar assistant is, and what salt rim is. I do not understand what meilisearch or redis are for. Also, I have nginx running without docker compose and do not know how to make all this work given I plan on not including the webserver nginx portion of the example compose file.
Lastly, I think adding vars to env is unnecessarily messy compared to just setting the URLs and meili key directly in the compose yaml; I am currently hosting eight sites all set up in docker compose and managed by nginx and not a single one has the URL set via docker compose! All docker compose knows is ports, and nginx handles forwarding those ports to <sitename>.<mydomain>.com.

meili/redis: can these be skipped? If not, maybe they should be packaged inside bar-assistant?
how to set up for docker compose while nginx is already running?
creating vars for the env seems unnecessary and complicated, the ports should be forwarded to URLs through nginx and not variables in yaml

EDIT: Apparently .env files are not global, making # 3 above much less relevant

Is there a discord/reddit/lemmy/forum (email address, PO box... worm hole?) to post questions? As it is, this is the only place I know of so here goes:
I believe I have somewhat properly set up everything (redis meili salt barass) minus the webserver/nginx part, as both bar assistant and salt rim have a UI. Bar assistant's URL gives me "This is your Bar Assistant instance. Checkout /docs to see documentation." and salt rim gives me a login screen. But, no register button, nor a login button if I enter anything into the email and password fields, and BA"s url/bar is not available.

Hello
Meilisearch and Redis can't be removed, if they could be removed they would not be in docker compose.
Here's a simplified diagram how current compose works
flowchart LR
    REQUEST[request] --> NGINX[nginx]
    API[api] <-->|via container network| REDIS[redis]
    API <-->|via container network| SEARCH[meilisearch]
    NGINX -->|/| CLIENT[Salt Rim]
    NGINX -->|/api| API
    NGINX -->|/search| SEARCH

Browser client (SaltRim in this case) needs public access to Meilisearch endpoints and API endpoints to work correctly. This is handled by nginx. Nginx configures [url]/, [url]/search and [url]/api endpoints that are used in salt-rim container via env variables.
If you have your nginx already running, then you need to configure proxies for Meilisearch and Bar Assistant containers, and define them in salt rim via environment variables.
For example, if you have configuration like the following:

search.example.com -> proxy to container meilisearch:7700
bar-api.example.com -> proxy to container bar-assistant:3000
Then you need to update your salt-rim container like so:

  salt-rim:
    image: barassistant/salt-rim:v2
    depends_on:
      - bar-assistant
    environment:
      - API_URL=api.example.com
      - MEILISEARCH_URL=search.example.com
      - BAR_NAME=$BAR_NAME
      - DESCRIPTION=$BAR_DESCRIPTION
      - DEFAULT_LOCALE=en-US
    restart: unless-stopped

The command docker compose exec bar-assistant php artisan bar:open is old and unused since v3.

I'm just another user but I have an example setup on my GitHub.
Everything in the docker compose file is absolutely necessary. Including the webserver. It doesn't matter if you are already running a webserver container. This setup needs it's own webserver container like it has in the compose. Many people I know have tried to fold/merge their current webserver container configuration into this, it just makes things more complicated. Don't do it. Plus, it really isn't that big a deal anyway, these things are all relatively small containers/files with minimal overhead to run. Adjust host side ports as necessary if you have conflicts.
Also the nginx.conf file is necessary to. And adjust paths as necessary in the docker compose for your needs...just remember to reference the nginx.conf file and place it in the right spot.
The Readme on my GitHub is just instructions on how to import my revamped/customized database, if you want it. Not necessary.
https://github.com/zhdenny/bar_assistant_database

Browser client (SaltRim in this case) needs public access to Meilisearch endpoints and API endpoints to work correctly. This is handled by nginx. Nginx configures [url]/, [url]/search and [url]/api endpoints that are used in salt-rim container via env variables.
If you have your nginx already running, then you need to configure proxies for Meilisearch and Bar Assistant containers, and define them in salt rim via environment variables.
For example, if you have configuration like the following:
* `search.example.com` -> proxy to container `meilisearch:7700`

* `bar-api.example.com` -> proxy to container `bar-assistant:3000`
  Then you need to update your salt-rim container like so:

  salt-rim:
    image: barassistant/salt-rim:v2
    depends_on:
      - bar-assistant
    environment:
      - API_URL=api.example.com
      - MEILISEARCH_URL=search.example.com
      - BAR_NAME=$BAR_NAME
      - DESCRIPTION=$BAR_DESCRIPTION
      - DEFAULT_LOCALE=en-US
    restart: unless-stopped

I re-read the above multiple times, but I'm not understanding. I've never messed with NPM's custom locations before. I think my only issues are in Nginx and not my compose yaml or envs. What I have, which I know is not entirely correct, is below. I think either some of this doesn't matter, or is just wrong:
Details:
bar.example.com
http <IP_ADDRESS> 3003
meili.example.com
http <IP_ADDRESS> 7700
saltrim.example.com
http <IP_ADDRESS> 3004
Screenshot (actual domain name changed for example):

(all three URLs work, but of course they are not connected together properly because no sub folder forwarding has been setup)
What I need to do is set something on the "Custom locations" window, but I don't know what to set for each of barassistant, saltrim, and meilisearch.

And then once those are set correctly, if anything needs to change in compose:
  salt-rim:
    environment:
      - API_URL=$API_URL #http://bar.example.com/bar
      - MEILISEARCH_URL=$MEILISEARCH_URL #http://bar.example.com/search

If you have every service proxied with subdomain then you don't need to setup custom locations.
For your example:

Details:
bar.example.com
http <IP_ADDRESS> 3003
meili.example.com
http <IP_ADDRESS> 7700
saltrim.example.com
http <IP_ADDRESS> 3004

You need to update env with the following
  salt-rim:
    environment:
      - API_URL=http://bar.example.com
      - MEILISEARCH_URL=http://meili.example.com

@karlomikus So simple! That did it, it's now up and running.
(Please let me know if I should open a separate issue for this:)
When I make a public link, such as http://saltrim.guntz.top/e/cocktail/01J24CVXCD5G7RGSEHV9E9KC46/monkey-man-1, it loads a broken recipe and gives me an error:

Only issue so far. Imports default recipes, imports recipes by URL, other basic things seem to be working.

Can you press F12 and check for any errors in the console and network tab?
I have a feeling this is maybe an issue with CORS

I'm getting this:

But the link works on my phone, and on another person's phone. But not my computer in any browser, nor the server that is hosting this.

Also it can't seem to access my clipboard. "Copy as ..." doesn't copy to my clipboard at all.

I found this thread and did a similar setup.  I don't want access outside my LAN and was able to only proxy salt-rim in NPM.  My NPM instance is part of the ba network.
version: "3"

services:
  meilisearch:
    image: getmeili/meilisearch:v1.7 #
    container_name: meilisearch
    environment:
      - MEILI_MASTER_KEY=$MEILI_MASTER_KEY
      - MEILI_ENV=production
    restart: unless-stopped
    volumes:
      - /share/ContainerStation/bar-assistant/meilisearch_data:/meili_data:rw
    ports:
      - 7700:7700
    networks:
      ba:
        ipv4_address: <IP_ADDRESS>

  redis:
    image: redis
    container_name: redis
    environment:
      - ALLOW_EMPTY_PASSWORD=yes
    restart: unless-stopped
    networks:
      ba:
        ipv4_address: <IP_ADDRESS>

  bar-assistant:
    image: barassistant/server:v3
    container_name: bar-assistant
    depends_on:
      - meilisearch
      - redis
    environment:
      # - PUID=1000 # Optional
      # - PGID=1000 # Optional
      - APP_URL=http://<IP_ADDRESS>:3030
      - LOG_CHANNEL=stderr #
      - MEILISEARCH_KEY=$MEILI_MASTER_KEY
      - MEILISEARCH_HOST=http://<IP_ADDRESS>:7700 #
      - REDIS_HOST=redis #
      - ALLOW_REGISTRATION=true
    restart: unless-stopped
    volumes:
      - /share/ContainerStation/bar-assistant/bar_data:/var/www/cocktails/storage/bar-assistant:rw
    ports:
      - 3030:3000
    networks:
      ba:
        ipv4_address: <IP_ADDRESS>

  salt-rim:
    image: barassistant/salt-rim:v2
    container_name: salt-rim
    depends_on:
      - bar-assistant
    environment:
      - API_URL=http://<IP_ADDRESS>:3030
      - MEILISEARCH_URL=http://<IP_ADDRESS>:7700
      - BAR_NAME=$BAR_NAME
      - DESCRIPTION=$BAR_DESCRIPTION
      - DEFAULT_LOCALE=en-US
    restart: unless-stopped
    ports:
      - 3031:8080
    networks:
      ba:
        ipv4_address: <IP_ADDRESS>

networks:
  ba:
    name: ba
    driver: bridge
    ipam:
      driver: default
      config:
        - subnet: <IP_ADDRESS>/24
          gateway: <IP_ADDRESS>


d3 responsive zoom behavior on window resizing

I am using D3 to set up my chart area that makes use of the zoomable behavior. The chart includes both x and y axis.
The chart area should be responsive when window is resized. In this case, I need to reset the x and y axis domain and range on window resizing.
The issue happened with window resizing. I've noticed the zoom focus isn't lined up with the mouse anymore after the following steps:

First pan and zoom in the chart area
Then resize the window
Then pan and zoom the chart area again

After that, the above problem happened. See the following jsfiddle which has this issue.
http://jsfiddle.net/maow/qLwneqxu/10/
So what's the right way of handing this?

You’ll need to call zoom.transform if you change the domain of the scales in order to keep the zoom’s transform in-sync.
Please use Stack Overflow tag d3.js to ask for help. Although I make an effort to assist everyone that asks, I am not always available to provide help promptly or directly. Stack Overflow provides a better collaborative forum for self-help: tens of thousands of D3-related questions have already been asked there, and some answered questions may be relevant to you.
If you have a question about D3’s behavior and want to discuss it with other users, also consider the d3-js Google Group or joining the d3-js Slack.
Thank you! 🤗

I know this issue has been closed. But I've searched anywhere in d3-js Google Group or stack overflow, I couldn't find the example on how this can be done in d3 version 4. I've posted a question at stack overflow as well.
http://stackoverflow.com/questions/39735367/d3-zoom-behavior-when-window-is-resized
As you mentioned, if I changed the axis domain on window resize, I need to call zoom.transform. I've tried resetting the translate to the identity transform which seems to fix the mouse jumping problem. But the scale went back to 1, I then I reapply the old scale, then it adds more problems. Is it possible you can post some examples on how this can be done?

You need to set the transform to whatever the new transform should be based on whatever you set the scale’s domains to, not the identity transform. An example is Brush & Zoom II.

So I've checked Brush & Zoom II example. I can see you're resetting the axis domain and then resetting the circles cx and cy based on the modified axis domain in zoom function, but I didn't see where you're doing zoom.transform.

Yes, sorry. That’s the example I meant.

Thanks for you quick response.
So I've tried your suggestion, call zoom.transform on window resize. What I did is:

on window resize, reset the width and height
reset the axis domain based on new width, new height and scale
then call zoom.transform, something like below:

svg.call(zoom.transform, d3.zoomIdentity
.scale(width / (x(currentXDomain[1]) - x(currentXDomain[0])))
.translate(-x(currentXDomain[0]), -y(currentYDomain[0])));
In my case, if I have reset the axis domain and width, the  .scale(width / (x(currentXDomain[1]) - x(currentXDomain[0]))) will always return 1, and translate value also doesn't seem to be correct. Can you please advise? I created a block, see below:
http://blockbuilder.org/jwmao/b20aa8877c871c860e2598c1e8d1602a

Sorry, but I can’t offer further assistance. Please try the other resources I mentioned previously. Good luck.

ok. Thanks for the help.


ENH: spatial: Implement Rotation.split()

Reference issue
Closes https://github.com/scipy/scipy/issues/21351
What does this implement/fix?
This is essentially the opposite of Rotation.concatenate() and mirrors the syntax of numpy.split() https://numpy.org/doc/stable/reference/generated/numpy.split.html
My current use case is making a pandas dataframe column with Rotations, which requires turning it into a list of single Rotations.
This also fixes a bug where empty rotations were allowed to be created from size (0, 4) arrays.

Bump on this

I realized that indexing with a list comprehension gets most of the way there, I'm ambivalent on the new change now. Thoughts from others?
[r for r in rc]
# returns [r1, r2, r3, r4]

Closing for now, may reopen if there's demand.


Flag to disable wait for app to become ready and to disable runtime subscriptions and input bindings check

In what area(s)?

/area runtime

Describe the feature

Projects like https://github.com/dapr-sandbox/dapr-ambient would benefit from this.
Once you enter the paradigm of running the dapr runtime detached from the application this "smart" features that dapr implements become problematic and source of errors.
When an application (detached from the sidecar) runs as a serverless workload and its scaled to 0, the check done by the dapr sidecar to wait for the app to become ready becomes pointless.
Or the other way around, if the detached dapr sidecar is what is scaled to 0 and an the first thing the application does (before even start an http server) is reaching the sidecar the "wait for the app to become ready" gets the app and the sidecar into a dead lock of both ends waiting for each other.
Then closely related to this are the "runtime" subscriptions read from the /dapr/subscribe endpoint. In scenarios like the ones described above configuring subscriptions via this method becomes pointless and error prone, so it's better to have a flag to just disable it.
And finally we have the same problem with input bindings, altough this one is a bit more difficult to solve. Input bindings are subscribed by first testing if the app implements the corresponding endpoint to receive the events. In scenarios like the ones described above this isn't a good fit. IMO We need a flag to disable this smart check on input bindings and additionally to that we need a declarative way of subscribing input bindings to apps.
Release Note

RELEASE NOTE:

A new flag can be easily added to the Configuration CRD but the input binding problem must be solved first.
IMO, the input binding should be solved via a new annotation (daprd param) to skip app check and always have the input binding as enabled. Something like: "dapr.io/input-bindings: inputA,inputB" - so those can be used in conjunction with the flag.
Happy to see other suggestions.

For bindings, I suggest we make it explicit in the specification, ie. bring it to the component level.
Example
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: <NAME>
spec:
  type: bindings.rabbitmq
  version: v1
  metadata:
  - name: queueName
    value: queue1
  - name: direction
    value:  input

Regardless of what we pick to specify the binding direction, I recommend that we either don't deprecate the old way of discovering the binding endpoint or that we deprecate it after 3+ releases.

Closed via https://github.com/dapr/dapr/issues/6449


868MHz frequency does not work

Hi,
I have RFM69CW 868MHz modules. Trying them with unchanged "MinimalInterruptUno" example works fine. But after changing frequency to 868MHz the example stops working.
    rfm.setFrequency((uint32_t) 868*1000*1000); // set the frequency.

Any help is much appreciated.

Hi,
you can try
rfm.setFrequency(868UL*1000*1000); // set the frequency

Hi Pablo,
I tried with no difference.
I also tried setting frequency registers directly (without conversion):
    // rfm.setFrf(0x4EC000);  // 315MHz - works
    // rfm.setFrf(0x6C8000);  // 434MHz - works
    // rfm.setFrf(0xE4C000);  // 915MHz - doesn't work
    // rfm.setFrf(0xd90000);  // 868MHz - doesn't work

Is your CPU little endian?

It is Arduino Pro mini, ATmega328p 16MHz, 3.3v.
The Internet says it is little endian.

Hmm, that is weird - I'm also using ATmega328p, RFM69@868MHz with no problem...

Setting baud rate for RFM69 module other than 4800 (which is default) fixed the problem.
    // rfm.baud4800();    // doesn't work with 868MHz
    // rfm.baud9600();    // works with 868MHz but with high packet loss
    // rfm.baud153600();  // works with 868Mhz with no packet loss
    rfm.baud300000();     // works with 868MHz with no packet loss

153600 and 300000 works reliably with 868MHz modules.

Hmm, I can definitely see setFrequency failing, the signature is:
void setFrequency(uint32_t freq);

So any value about 32 bits will get truncated. I developed this library against the 315 or 434 MHz version... clearly. Does an arduino support uint64_ts? Because if so we should probably just change the type, that should solve the issue? If anyone can test this and file a PR that'd be much appreciated, I don't have access to this hardware anymore.
@Pablo2048 , are you sure you are setting the frequency with setFrequency on your 868 MHz unit? Because the default values for the register should be good on each unit, but setting them should cause problems :)
The setFrf function should always work if the endianness of the cpu is right. It just copies the 3 relevant bytes from the input.

Setting baud rate for RFM69 module other than 4800 (which is default) fixed the problem.

Cool! Could have had something to do with needing a higher Fdev or something at these higher frequencies. The carrier is now probably set by the default value in the register, which should be fine in general.

Ehm, sorry for confusing - I take a look into my code and i'm not calling setFrequency...

Hi,
I've filed #13 , I cannot test this unfortunately. @Pablo2048 , @sheinz if you have time, could you give that branch a spin and see if that allows you to make a call to setFrequency with a high number and end up with working communication?
Thanks!


apt-update before pkg build

High Level Overview of Change
The GCC version Ubuntu's ubuntu-toolchain-r/test repo provides has been updated

Type of Change

[x] Bug fix (non-breaking change which fixes an issue)

handled in https://github.com/XRPLF/rippled/pull/4650


Show timestamps in local time on Change log

Context
TRS stores timestamps in UTC but for display purposes we should show them in GMT/BST.
Changes proposed in this pull request
Amend the Change log to display timestamps in GMT/BST.
Guidance to review
UI change + tests
Checklist

[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Show timestamps in local time on Change log

 Show timestamps in local time on Change log


Timing bug

Because of the asynchronous nature of the neovim-python client, inputting keys while another key is still processing can cause some weird timing bugs (though not always). Currently, the solution is to force v.tio to run slower, which is hardly an idea solution.

This isn't fixed, but as of https://github.com/DJMcMayhem/V/commit/2be99c3688df075792375883102ed199ab5d182c there is now a workaround. Launching V with -s time will cause V to sleep time seconds between each keystroke. This is nice because not all V programs cause this undefined behavior, so when it rarely does you can fix it, but you don't need to put up with a slow interpreter all the time.


Make feedparser more reliable against dc_based malformed feeds (like craigslist)

Context of this is https://github.com/pipes-digital/pipes/issues/43.
The craigslist feed is not to spec, but works in readers like firefox, see https://newyork.craigslist.org/search/ggg?format=rss&query=logo as an example. But feedparser choked on it, I think because the rss module does not create some methods feedparser is expecting.
This PR adds try_catch blocks around optional assignments, checks for existing methods where it seemed more appropriate, and to make the feed output more useful it checks for item.dc_date as a fallback when item.date is missing.

@onli Thanks for adding more checks.
PS: You're more than welcome to join the feedparser org - please let me know and I send you an invitation via github so you can if needed change the code without push request and help move the feedparser forward. Keep it up.  Cheers. Prost.

Thanks for merging that soon. Will you put it in a gem release soon as well? That would give me peace of mind when updating my gems :)
I'd be happy to join the feedparser org!

@onli  About the gem - will try to push a new gem (release) with your changes tomorrow (Thursday). Please, send a reminder if there's no release up on Friday.
Great. Welcome.  Note: You will get an invitation (by email from GitHub) to join the feedparser org as a member. If you accept, you can commit if needed without any need for a pull request - and you will show up on the people page too (if you set your preference to public - default is private; change the setting in the github org people page).


Std of log mel-filterbank will be close to zero in some dimension when nfilt == 80.

import python_speech_features
y, sr = soundfile.read(file, dtype='float32')
log_mfbank = python_speech_features.logfbank(y,
samplerate=16000,
nfilt=80,
winlen=0.025, winstep=0.01,
nfft=512, lowfreq=125, highfreq=7600, preemph=0.97,
winfunc=np.hanning)
std = np.std(features, axis=0)
print(np.min(std))

I have same problem. Have you fixed it?


Incorrect behaviour with Angular and static link

Flatpickr does something to dynamically injected link in angular applications preventing links to be working correctly. See attached example.
Related PR:
https://github.com/flatpickr/flatpickr/pull/2127
It doesn't fix issue completely though. Those lines are causing those issues:
if (window.ontouchstart !== undefined)
      bind(window.document, "touchstart", documentClick);
else bind(window.document, "mousedown", onClick(documentClick));
bind(window.document, "focus", documentClick, { capture: true });

https://stackblitz.com/edit/angular-8-reactive-form-5gztmr?file=src%2Fapp%2Fapp.component.ts
Your Environment

flatpickr version used: 4.6.3

Browser name and version:
Chrome@latest

OS and version:
MacOs@latest

You can try to initialize flatpickr outside of the Angular zone:
ngOnInit() {
    this.zone.runOutsideAngular(() => {
      flatpickr(this.inputEl.nativeElement, {
        dateFormat: "d M Y",
        onChange: (selectedDates, dateStr, instance) => {}
      });
    });
  }

https://stackblitz.com/edit/angular-8-reactive-form-vvepyp?file=src%2Fapp%2Fapp.component.ts


Actual pagination APIs do not match the Pagination section in the CS specification

From the CS spec:

Each endpoint that uses pagination may use different parameters. However the theme among them is that they take a from and to token, and occasionally a limit and dir. Together, these parameters describe the position in a data set, where from and to are known as "stream tokens" matching the regular expression [a-zA-Z0-9.=_-]+. If supported, the dir defines the direction of events to return: either forwards (f) or backwards (b).

However, there's only actually one endpoint that implements this interface, namely GET /_matrix/client/r0/rooms/{roomId}/messages. Every other paginated endpoint - that is, every endpoint that returns a chunk and some sort of stream token - takes a different set of parameters.
Here's a brief survey of (non-deprecated) paginated CS API endpoints:
9.4.1   GET /_matrix/client/r0/sync
	# Note: Does *not* return a `chunk` property.
	since
		A point in time to continue a sync from.

9.5.4   GET /_matrix/client/r0/rooms/{roomId}/members
	at
		The point in time (pagination token) to return members for in the room. This token can be obtained from a prev_batch token returned for each room by the sync API. Defaults to the current state of the room, as determined by the server.

9.5.6   GET /_matrix/client/r0/rooms/{roomId}/messages
	from
		Required. The token to start returning events from. This token can be obtained from a prev_batch token returned for each room by the sync API, or from a start or end token returned by a previous request to this endpoint.
	to
		The token to stop returning events at. This token can be obtained from a prev_batch token returned for each room by the sync endpoint, or from a start or end token returned by a previous request to this endpoint.
	dir
		Required. The direction to return events from. One of: ["b", "f"]
	limit
		The maximum number of events to return. Default: 10.

10.5.3   GET /_matrix/client/r0/publicRooms
	since
		A pagination token from a previous request, allowing clients to get the next (or previous) batch of rooms. The direction of pagination is specified solely by which token is supplied, rather than via an explicit flag.
	limit
		Limit the number of results returned.

10.5.4   POST /_matrix/client/r0/publicRooms
	since
		A pagination token from a previous request, allowing clients to get the next (or previous) batch of rooms. The direction of pagination is specified solely by which token is supplied, rather than via an explicit flag.
	limit
		Limit the number of results returned.

And likewise, in the Federation API spec (which possibly means that pagination should not be defined specifically in the CS spec at all?):
15.1   GET /_matrix/federation/v1/publicRooms
	since
		A pagination token from a previous call to this endpoint to fetch more rooms.
	limit
		The maximum number of rooms to return. Defaults to 0 (no limit).

I already brought this up in the spec channel, and Matthew remarked that either the "all pagination looks the same" claim should be removed, or the APIs should be changed to actually meet the claim.
As an implementer, I'd be much in favour of the latter - it would allow for a generic "paginated resource" abstraction to deal with these and future paginated endpoints, which would greatly reduce the complexity needed to deal with pagination. That having been said, I don't know whether there are particular backwards compatibility concerns that would make this problematic to change in the spec.

Additionally: GET /_matrix/client/r0/rooms/{roomId}/members is weird, because it doesn't really paginate (the stream token is just used as a reference point in time), but it still returns a chunk, implying that it should be a paginated endpoint.

More: GET /_matrix/client/r0/sync also behaves like a paginated resource in the sense that it accepts a since parameter - but it does not follow the chunk convention in its response. I've added it to the original survey.
Also, I've removed GET /_matrix/client/r0/events from the survey, since that's actually a deprecated API, and I only just noticed that :)

See also https://github.com/matrix-org/matrix-doc/issues/1898#issuecomment-496373762 and related conversation


Update Readme that support Apache Kylin

Codecov Report

Merging #5818 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5818   +/-   ##
=======================================
  Coverage   63.83%   63.83%
=======================================
  Files         364      364
  Lines       23100    23100
  Branches     2587     2587
=======================================
  Hits        14745    14745
  Misses       8340     8340
  Partials       15       15

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 a411516...8982177. Read the comment docs.


How to use it?

nice project!
can you tell me How to use it? is there any document\blog\wiki?
thanks!

You compile it and parse needed parameters.
Of course, if you want to use it you need access to a VoIP network or if just testing a testing environment (emulated) should be more than enough.

@d3v1l401 Thanks for your comment and reply, that is correct
@cn0xroot i will prepare the wiki and documenation that will have all the basic and essential info

@d3v1l401  @SigPloiter  At frist,thanks for the quick reply.
@SigPloiter I look forward to your wiki and documenation

@cn0xroot  i will do my best to finish it over the weekend

@cn0xroot Wiki updated

@SigPloiter cool!

@SigPloiter  i want attack server with ss7 vilnurabity but i dont have ip adress and  port how  can i cet it

You need to find a SS7/ Sigtran provider to give you access and global
title and point code
On Thursday, 2 November 2017, ayarifak<EMAIL_ADDRESS>wrote:

@SigPloiter https://github.com/sigploiter i want attack server with ss7
vilnurabity but i dont have ip adress and port how can i cet it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SigPloiter/SigPloit/issues/9#issuecomment-341352892,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXXuv-XLGOezznQaNv2L44XcDNL4_1RTks5syYAvgaJpZM4N9tvX
.

i now but use all methode but i cant found


[RFC] Add indentprofile

This is just a little gimmick.
In order to enforce coding guidelines (especially to myself) I added an indent-profile; also to make check.
Also you can use the resulting %.indentlog files directly as patches...
@tstruk : I'd guess that you might like this ?
This is meant as an an RFC
The other two patches remove a problem with indent (; on makro definitions) and give an example...

I actually tried to mimic the doc/coding_standard_c.md file as good as I could.
Please go ahead and have a look and tell me what you think is not quite right yet.

@AndreasFuchsSIT it is easy to see by looking at the indent.log.
I think the main difference between the spec implied format and the indent format is the function prototypes, for instance:
-TSS2_RC Tss2_Sys_ClockRateAdjust(

TSS2_SYS_CONTEXT *sysContext,
TPMI_RH_PROVISION auth,
TSS2_SYS_CMD_AUTHS const *cmdAuthsArray,
TPM2_CLOCK_ADJUST rateAdjust,
TSS2_SYS_RSP_AUTHS *rspAuthsArray)
+TSS2_RC
+Tss2_Sys_ClockRateAdjust(TSS2_SYS_CONTEXT * sysContext,

                    TPMI_RH_PROVISION auth,

                    TSS2_SYS_CMD_AUTHS const *cmdAuthsArray,

                    TPM2_CLOCK_ADJUST rateAdjust,

                    TSS2_SYS_RSP_AUTHS * rspAuthsArray)

I'm ok with either, but I would like to  get @flihp's opinion on that.

oh, the github formatting messes up the copy and paste terminal format. Indent log attached.
indent.log

@tstruk you mean the alignment of parameters with the opening ( ?
you can switch this behavior by adding --dont-line-up-parentheses.
However this will not only change it for function declarations but also for function invocations, which I hightly dislike. e.g.:
     rval = Tss2_Sys_GetEncryptParam(sysContext,
-                                    &currEncryptParamSize,
-                                    &currEncryptParamBuffer);
+        &currEncryptParamSize, &currEncryptParamBuffer);

For this and also my personal preference, I prefer lining up function declarations with opening (

Hello all.
I have no problem modifying our documented coding standard to align with what we can get indent to do for us. I liked the previous function prototype / declaration style mostly because it matched the code from the TPM spec. Let me spend a few minutes with this one today so I can get my head around the scope of the change and side effects etc.

@flihp @tstruk One question came up here: Do we really want to go to 80 char per line ?
It's quite outdated and we have quite long function names. So the proposal was to go for 100 char per line.
What do you think ?

@AndreasFuchsSIT My opinion is that we should try to limit ourselves to 80 chars per line. This is what the most guide lines recommend.

Closing this old PR. The idea about using a tools like indent to check code format is great, but it should just report the issue and not try to fix anything automatically.
I'll create am issue for it.


Ensure a link to this repo is present at pypi

Well … Would be cool ;)

I will try to add this soon


Control Plane (xDS Server - Envoy) Auth + Authz

This issue tracks how the the data plane Envoy Proxy authenticates and authorizes with the xDS Server within Envoy Gateway

@LukeShu I assigned this issue to you based on the 5/24/22 community meeting notes. Please unassign yourself if that's not the case.

xref https://github.com/envoyproxy/gateway/issues/97#issuecomment-1150602629 regarding how CP<>DP auth can be supported.

I think the relevant part of that comment is:

In terms of authenticating between the control plane and Envoy, Contour has solved this problem in the simple case, using an on-disk SDS for hot-reload of certificates, an install process that generates one-year validity certs for both Contour and Envoy, and updates them on each upgrade. The update is seamless because the secrets change "on disk", Envoy sees the SDS change, and then control plane connection is rebuilt in less than 30 seconds. Notably, because it uses Kubernetes Secrets as intended and can handle rotations whenever you're ready, it plugs in easily to Vault or other secret-management tools - the Secret admin just has to ensure that the Secrets are updated for both Envoy and Contour around the same time.

I think that Contour's solution here is pretty great (maybe I'm biased, lol), and will work well to allow secure xds-server <-> Envoy comms no matter what provider is in use (although for things that use localhost only, we can probably skip TLS).

Not stale, just backlogged.

FYI @arkodg I'm going to take this work on.