some quesion about detection

Hello author, I am honored to read your paper. After completing leather data set training, I tried to detect, but encountered some problems:
Here is my args file:
args19.json
{
"img_size": [256,256],
"Batch_Size": 1,
"EPOCHS": 1000,
"T": 1000,
"base_channels": 128,
"beta_schedule": "linear",
"channel_mults": "",
"loss-type": "l2",
"loss_weight": "none",
"train_start": true,
"lr": 1e-4,
"random_slice": true,
"sample_distance": 600,
"weight_decay": 0.0,
"save_imgs":true,
"save_vids":true,
"dropout":0,
"attention_resolutions":"16,8",
"num_heads":2,
"num_head_channels":-1,
"noise_fn":"simplex",
"dataset":"leather",
"channels": 1
}
After the training:

Error is reported in evaluation.testing() immediately after train:

File "diffusion_training.py", line 153, in train
evaluation.testing(testing_dataset_loader, diffusion, ema=ema, args=args, model=model)
File "/home/dell/local/HK/AnoDDPM-master/evaluation.py", line 124, in testing
x = x.to(device)
NameError: name 'device' is not defined

When I tried detection, I entered the following command:
python3 detection.py args19.json
Here, args19.json is the configuration file I trained with
But the following error is reported:

This is the shape of the layer for the row where the error occurred
Conv2d(1, 128, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1）
Traceback (most recent call last):
File "detection.py", line 959, in
anomalous_metric_calculation()
File "detection.py", line 226, in anomalous_metric_calculation
t_distance=200, denoise_fn=args["noise_fn"]
File "/home/dell/local/HK/AnoDDPM-master/GaussianDiffusion.py", line 354, in forward_backward
out = self.sample_p(model, x, t_batch, denoise_fn)
File "/home/dell/local/HK/AnoDDPM-master/GaussianDiffusion.py", line 299, in sample_p
out = self.p_mean_variance(model, x_t, t)
File "/home/dell/local/HK/AnoDDPM-master/GaussianDiffusion.py", line 279, in p_mean_variance
estimate_noise = model(x_t, t)
File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/module.py", line 889, in _call_impl
result = self.forward(*input, **kwargs)
File "/home/dell/local/HK/AnoDDPM-master/UNet.py", line 400, in forward
h = module(h, time_embed)
File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/module.py", line 889, in _call_impl
result = self.forward(*input, **kwargs)
File "/home/dell/local/HK/AnoDDPM-master/UNet.py", line 35, in forward
x = layer(x)
File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/module.py", line 889, in _call_impl
result = self.forward(*input, **kwargs)
File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 399, in forward
return self._conv_forward(input, self.weight, self.bias)
File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 396, in _conv_forward
self.padding, self.dilation, self.groups)
RuntimeError: Given groups=1, weight of size [128, 1, 3, 3], expected input[1, 3, 256, 256] to have 1 channels, but got 3 channels instead
I wonder if there is something wrong with my detection call (I did not change your code) :

Maybe my question is relatively simple, but it has really caused me a lot of trouble. I would like to express my deep thanks again for troubling you.

Like you, I also made an error in the test after the training, but the channel of my args is 3. The problem I encountered during the test is that args["img_size"] is of float type, resulting in the following error.

Since image dtype is floating point, you must specify '
'the data_range parameter. Please read the documentation '
'carefully (including the note). It is recommended that '
'you always specify the data_range anyway.

https://github.com/scikit-image/scikit-image/blob/6569fbfeb2891a7498418e2d5b8af9e851ca2102/skimage/metrics/_structural_similarity.py#LL194C30-L194C30
The outermost layer of the faulting method stack is the following line of code
https://github.com/Julian-Wyatt/AnoDDPM/blob/3052f0441a472af55d6e8b1028f5d3156f3d6ed3/detection.py#L243
https://github.com/Julian-Wyatt/AnoDDPM/blob/3052f0441a472af55d6e8b1028f5d3156f3d6ed3/evaluation.py#L48
My training args are as follows
{
  "img_size": [
    256,
    256
  ],
  "Batch_Size": 1,
  "EPOCHS": 3000,
  "T": 1000,
  "base_channels": 128,
  "beta_schedule": "linear",
  "channel_mults": "",
  "loss-type": "l2",
  "loss_weight": "none",
  "train_start": true,
  "lr": 1e-4,
  "random_slice": true,
  "sample_distance": 800,
  "weight_decay": 0.0,
  "save_imgs": true,
  "save_vids": false,
  "dropout": 0,
  "attention_resolutions": "16,8",
  "num_heads": 2,
  "num_head_channels": -1,
  "noise_fn": "simplex",
  "channels": 3,
  "dataset": "leather"
}

谢谢你的回复，冒昧问一下你是中国人吗，我看到你的ip显示武汉，也许我们可以微信进行交流一下，我的微信是18839672866，如果有冒犯，跟您说声抱歉！

Like you, I also made an error in the test after the training, but the channel of my args is 3. The problem I encountered during the test is that args["img_size"] is of float type, resulting in the following error.

Since image dtype is floating point, you must specify '
'the data_range parameter. Please read the documentation '
'carefully (including the note). It is recommended that '
'you always specify the data_range anyway.

https://github.com/scikit-image/scikit-image/blob/6569fbfeb2891a7498418e2d5b8af9e851ca2102/skimage/metrics/_structural_similarity.py#LL194C30-L194C30
The outermost layer of the faulting method stack is the following line of code
https://github.com/Julian-Wyatt/AnoDDPM/blob/3052f0441a472af55d6e8b1028f5d3156f3d6ed3/detection.py#L243
https://github.com/Julian-Wyatt/AnoDDPM/blob/3052f0441a472af55d6e8b1028f5d3156f3d6ed3/evaluation.py#L48
My training args are as follows
{
  "img_size": [
    256,
    256
  ],
  "Batch_Size": 1,
  "EPOCHS": 3000,
  "T": 1000,
  "base_channels": 128,
  "beta_schedule": "linear",
  "channel_mults": "",
  "loss-type": "l2",
  "loss_weight": "none",
  "train_start": true,
  "lr": 1e-4,
  "random_slice": true,
  "sample_distance": 800,
  "weight_decay": 0.0,
  "save_imgs": true,
  "save_vids": false,
  "dropout": 0,
  "attention_resolutions": "16,8",
  "num_heads": 2,
  "num_head_channels": -1,
  "noise_fn": "simplex",
  "channels": 3,
  "dataset": "leather"
}

The most likely answer here is that the code just needs a check if 3 channels are given with an edge case (there could be a nicer way but I'm not super familiar off the top of my head with the SSIM method from sklearn) - When writing my original codebase I naïvely hard coded the channels in some areas leading to issues above.

Hello author, I am honored to read your paper. After completing leather data set training, I tried to detect, but encountered some problems: Here is my args file:
args19.json { "img_size": [256,256], "Batch_Size": 1, "EPOCHS": 1000, "T": 1000, "base_channels": 128, "beta_schedule": "linear", "channel_mults": "", "loss-type": "l2", "loss_weight": "none", "train_start": true, "lr": 1e-4, "random_slice": true, "sample_distance": 600, "weight_decay": 0.0, "save_imgs":true, "save_vids":true, "dropout":0, "attention_resolutions":"16,8", "num_heads":2, "num_head_channels":-1, "noise_fn":"simplex", "dataset":"leather", "channels": 1 }
After the training:

Error is reported in evaluation.testing() immediately after train:

File "diffusion_training.py", line 153, in train evaluation.testing(testing_dataset_loader, diffusion, ema=ema, args=args, model=model) File "/home/dell/local/HK/AnoDDPM-master/evaluation.py", line 124, in testing x = x.to(device) NameError: name 'device' is not defined

When I tried detection, I entered the following command:
python3 detection.py args19.json
Here, args19.json is the configuration file I trained with
But the following error is reported:

This is the shape of the layer for the row where the error occurred Conv2d(1, 128, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1）
Traceback (most recent call last): File "detection.py", line 959, in anomalous_metric_calculation() File "detection.py", line 226, in anomalous_metric_calculation t_distance=200, denoise_fn=args["noise_fn"] File "/home/dell/local/HK/AnoDDPM-master/GaussianDiffusion.py", line 354, in forward_backward out = self.sample_p(model, x, t_batch, denoise_fn) File "/home/dell/local/HK/AnoDDPM-master/GaussianDiffusion.py", line 299, in sample_p out = self.p_mean_variance(model, x_t, t) File "/home/dell/local/HK/AnoDDPM-master/GaussianDiffusion.py", line 279, in p_mean_variance estimate_noise = model(x_t, t) File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/module.py", line 889, in _call_impl result = self.forward(*input, **kwargs) File "/home/dell/local/HK/AnoDDPM-master/UNet.py", line 400, in forward h = module(h, time_embed) File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/module.py", line 889, in _call_impl result = self.forward(*input, **kwargs) File "/home/dell/local/HK/AnoDDPM-master/UNet.py", line 35, in forward x = layer(x) File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/module.py", line 889, in _call_impl result = self.forward(*input, **kwargs) File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 399, in forward return self._conv_forward(input, self.weight, self.bias) File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 396, in _conv_forward self.padding, self.dilation, self.groups) RuntimeError: Given groups=1, weight of size [128, 1, 3, 3], expected input[1, 3, 256, 256] to have 1 channels, but got 3 channels instead
I wonder if there is something wrong with my detection call (I did not change your code) :
Maybe my question is relatively simple, but it has really caused me a lot of trouble. I would like to express my deep thanks again for troubling you.

The direct test methods had a couple of bugs towards the end of my development, so I apologise. Clearly, a device error here should be quick and easy to fix and rerun?
Regarding the second issue.
It is hard to say what the specific issue is here with the model being so large. However, it is evidently a channel issue; so I would ensure that the resulting channels are as expected throughout the Simplex calculations, and when passing the image into the model.
Any further issues, please feel free to reach out.

Hello author, I am honored to read your paper. After completing leather data set training, I tried to detect, but encountered some problems: Here is my args file:
args19.json { "img_size": [256,256], "Batch_Size": 1, "EPOCHS": 1000, "T": 1000, "base_channels": 128, "beta_schedule": "linear", "channel_mults": "", "loss-type": "l2", "loss_weight": "none", "train_start": true, "lr": 1e-4, "random_slice": true, "sample_distance": 600, "weight_decay": 0.0, "save_imgs":true, "save_vids":true, "dropout":0, "attention_resolutions":"16,8", "num_heads":2, "num_head_channels":-1, "noise_fn":"simplex", "dataset":"leather", "channels": 1 }
After the training:

Error is reported in evaluation.testing() immediately after train:

File "diffusion_training.py", line 153, in train evaluation.testing(testing_dataset_loader, diffusion, ema=ema, args=args, model=model) File "/home/dell/local/HK/AnoDDPM-master/evaluation.py", line 124, in testing x = x.to(device) NameError: name 'device' is not defined

When I tried detection, I entered the following command:
python3 detection.py args19.json
Here, args19.json is the configuration file I trained with
But the following error is reported:

This is the shape of the layer for the row where the error occurred Conv2d(1, 128, kernel_size=(3, 3), stride=(1, 1), padding=(1, 1）
Traceback (most recent call last): File "detection.py", line 959, in anomalous_metric_calculation() File "detection.py", line 226, in anomalous_metric_calculation t_distance=200, denoise_fn=args["noise_fn"] File "/home/dell/local/HK/AnoDDPM-master/GaussianDiffusion.py", line 354, in forward_backward out = self.sample_p(model, x, t_batch, denoise_fn) File "/home/dell/local/HK/AnoDDPM-master/GaussianDiffusion.py", line 299, in sample_p out = self.p_mean_variance(model, x_t, t) File "/home/dell/local/HK/AnoDDPM-master/GaussianDiffusion.py", line 279, in p_mean_variance estimate_noise = model(x_t, t) File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/module.py", line 889, in _call_impl result = self.forward(*input, **kwargs) File "/home/dell/local/HK/AnoDDPM-master/UNet.py", line 400, in forward h = module(h, time_embed) File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/module.py", line 889, in _call_impl result = self.forward(*input, **kwargs) File "/home/dell/local/HK/AnoDDPM-master/UNet.py", line 35, in forward x = layer(x) File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/module.py", line 889, in _call_impl result = self.forward(*input, **kwargs) File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 399, in forward return self._conv_forward(input, self.weight, self.bias) File "/home/dell/anaconda3/envs/pytorch181/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 396, in _conv_forward self.padding, self.dilation, self.groups) RuntimeError: Given groups=1, weight of size [128, 1, 3, 3], expected input[1, 3, 256, 256] to have 1 channels, but got 3 channels instead
I wonder if there is something wrong with my detection call (I did not change your code) :
Maybe my question is relatively simple, but it has really caused me a lot of trouble. I would like to express my deep thanks again for troubling you.

The direct test methods had a couple of bugs towards the end of my development, so I apologise. Clearly, a device error here should be quick and easy to fix and rerun?
Regarding the second issue. It is hard to say what the specific issue is here with the model being so large. However, it is evidently a channel issue; so I would ensure that the resulting channels are as expected throughout the Simplex calculations, and when passing the image into the model. Any further issues, please feel free to reach out.

Hello, Professor, your work is very wonderful, but I have some questions when I reproduce your paper, that is, why can the batchsize only be 1? When I change it to a different multiple, the model will report an error during the diffusion process


Multiple versions does not work

I tried to build this on a mac but instead of installing torch-2.0.0 it installs torch-2.0.0+cpu. It makes sense because the plugin forces everything with torch to use that source but there must be a way to change the behavior to work with this case
torch = [
            { version = "2.0.0+cpu", source = "torchcpu", markers = "sys_platform != 'darwin'"},
            { version = "2.0.0", markers = "sys_platform == 'darwin'"}
        ]
[tool.poetry-private-repo-plugin.enforce-source]
"torch-*" = "torchcpu"

[[tool.poetry.source]]
name = "torchcpu"
url = "https://download.pytorch.org/whl/cpu"
default = false
secondary = true

Hi @HadiSDev, have you tried this?

Define two secondary sources, like torchcpu and torchgpu (pointing to two different urls).
Assign explicitly the correct source to each of torch
Remove the tool.poetry-private-repo-plugin.enforce-source setting

Closing as you can now better handle it in Poetry 1.5 and you can we'll dismiss this plugin. Check the conversations in https://github.com/python-poetry/poetry/issues/6409


Add DevCycle NodeJS Provider

This PR

adds DevCycle NodeJS Provider to the docs

Related Issues
Fixes #108

@jonathannorris Looks great to me! Thanks again.
The CNCF requires a signoff. I can squash+signoff if you'd like, or you can and then force push. Let me know.

@toddbaert should be good to squash and merge now.


Made Kazoo.CommitOffset() more efficient by removing parent path creation

Made Kazoo.CommitOffset more efficient by removing parent path creation, instead making it purely an update operation.  The 'make parent' logic is already being handled in Kazoo.create().
Prior to this change we found that our ZK logs were growing by Gigabytes a day with a high volume of "KeeperException: NodeExists" entries for our high-volume consumers.  These log entries are mitigated by the changes included in this PR.

Nice improvement :+1:


Add query for fetching station_group codes and names (and optionally …

…alias_name)

Codecov Report
Base: 91.59% // Head: 91.32% // Decreases project coverage by -0.26% :warning:

Coverage data is based on head (f28104b) compared to base (df4bd37).
Patch coverage: 75.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
- Coverage   91.59%   91.32%   -0.27%
==========================================
  Files           8        8
  Lines         726      738      +12
  Branches       44       44
==========================================
+ Hits          665      674       +9
- Misses         54       57       +3
  Partials        7        7

Impacted Files
Coverage Δ

cimsparql/model.py
81.04% <57.14%> (-0.70%)
:arrow_down:

cimsparql/data_models.py
100.00% <100.00%> (ø)

cimsparql/templates.py
100.00% <100.00%> (ø)

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

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


Add variables support for cache policy

I'm not familiar with TS, but I guess this should fix problem.

@firecow
Can you take a look on the changes please?

@firecow
Can you take a look on the changes please?


Don't delay lookups to runtime

This discussion thread might be very interesting to you :-)

@asterite, oh, thanks, this thread is really useful for me. Looking what can I take from it ;)


Fix memory unsafe behavior in TextDecoder decode()

Previous fix was incorrect: the coercion check for .decode() first argument didn't read "len" so that output buffer allocated was always zero length. This caused memory unsafe behavior for almost any input (but wasn't caught by non-valgrind tests).
Fix by looking up the length once to size the output buffer. Then, when looking up the input data pointer recheck that the length is still the same. If it isn't, fail: it's not safe continue. If the input became longer, the output buffer wouldn't (always) be large enough.
No release entry because bug hasn't been released.

Hm, at first I was thinking you could just defer allocating the output buffer until after the input buffer pointer was retrieved, but that won't actually work because duk_push_fixed_buffer() may indirectly cause side effects up to and including finalizer calls.

Yep - and on the other hand you need the input length to allocate the output buffer.

This is one place where a chunked algorithm would actually be a bit easier: you'd process input in fixed chunks and append it to an output buffer resized on each step. The output reserve would be a fixed amount rather then depending on the input length. At each step the input pointer and length would be rechecked.
This would be memory safe but would be subject to side effects modifying the input buffer so that the output might reflect two "versions" of the input. This is not really a problem as such, because it's still memory safe, and you'd have to quite purposefully modify the input.


Quasar integration

Hi, I need some help here.
Can someone help me to integrate Quasar in this template?
quasar has this plugin for vite: @quasar/vite-plugin, but I don't know how to integrate it in this template.
Thank you

just install quasar like how it stated in the documentation.
i will close this issue since this is not related issue.


search language filter stopped working

Describe the bug
non-english posts are showing up in searches with "lang:en"
To Reproduce
Search for a proper noun and see that it returns results in multiple languages. then search it again with "lang:en" and see that a lot of non-english results are still there.
I'm using the latest version of web.
Screenshots

Unfortunately that's correct, the posts are being tagged with the English language code

Unfortunately the search is correct, it's the posts are being tagged with the English language code

How does the tagging work? Does it just take  the system's language and tags the post the same?

Correct, default post language is system locale.
We do use an NLP model to guess what the post is written in and show a banner recommending to switch language if the confidence is high, but it's not very reliable unfortunately.


add RTL support

by adding dir="rtl" attribute to heml tag all elements should be RTL.

Thanks for bringing this up! I haven't had time to dig into the expected behavior with this. Should this cascade down to all text elements? Should it also affect non-layout tables?

IMO this should affect all styles (e.g. margin, 'text-align', etc) and attributes (e.g. 'direction', 'align', etc), I'm not sure about non-layout tables.
side note, this project is very helpful and awesome, we are very appreciate it. thank you!

I'd like to bump this. Is there any progress on adding rtl-support? I'm not even able to hack around it. It wont get applied properly to the elements after heml has compiled the document.


migration of mprans to xtensor

Based on #1054 to trigger the CI. Both PRs will be merged in the end.

@JohanMabille the main source of the long compile times is the c++ compile of our template instances at high optimization levels.  On the hashdist side we turn off optimization by default in the setup file as per below. I think this was commented out for conda because we weren't carefully replacing the "-Wl,-O*" options, but I've fixed that below and tested that conda compiles proteus without problems (and quickly) on my laptop.  I think generally we want the conda binary on conda forge to use optimization but turn it off and enable debugging of our travis builds.
    if type(value) == str:
        cfg_vars[key] = cfg_vars[key].replace("-Wstrict-prototypes", "")
        cfg_vars[key] = cfg_vars[key].replace("-Wall", "-w")
        cfg_vars[key] = cfg_vars[key].replace("-Wl,-O3","")
        cfg_vars[key] = cfg_vars[key].replace("-Wl,-O2","")
        cfg_vars[key] = cfg_vars[key].replace("-O3", "")
        cfg_vars[key] = cfg_vars[key].replace("-O2", "")
        cfg_vars[key] = cfg_vars[key].replace("-DNDEBUG", "-UNDEBUG")
        cfg_vars[key] = cfg_vars[key] + " -g"

@cekees the last commit before I reverted the change for RANS3PF was building fine with conda, the failure was with hdist.
I will try to disable optimizations and see what it says,I agree that it will be more helpful when migrating to the dict of numpy arrays.

@JohanMabille note that you can't just uncomment the old setup modifications. You need to use what I posted above, particularly  cfg_vars[key] = cfg_vars[key].replace("-Wl,-O3","")         cfg_vars[key] = cfg_vars[key].replace("-Wl,-O2","")
Otherwise the conda build will likely fail.

@tridelat @zhang-alvin @ejtovar This PR swaps out cython wrapping for pybind11 and xtensor arrays for double*. As @JohanMabille mentioned, it doesn't yet reduce the size of the call signatures by directly passing the q, ebqe, etc. dict structures into C++, which is the objective. I do think the pybind11 and xtensor array approach is a signficant improvement as long as you guys can verify that we're not suffering a performance hit on some real problems.  In this interim approach, to use a new array in C++ you'll still need to modify the .py, .h, and .cpp file of a model (the latter takes the place of the pyx file in cython), roughly the equivalent amount of your time as currently modifying .py, .h, and .pyx file. However, you can use multi-dimensional array indexing in c++ and other nice features of xtensor arrays. If are happy with this approach and can verify that you're ready to start using it moving forward, then I agree we should go ahead and merge this.
On the other hand, going the next step of replacing the multiple xtensor arrays with some kind of dictionary/dataframe would compete the full task but make the differences a bit larger.  @SylvainCorlay could you update us on the status of xframe or similar (xrecord?) which we discussed early on? Our q, ebqe, etc., dictionaries are essentially named dataframes with the last (inner) dimension being variable (e.g. scalar, vector, tensor). I don't think that works with a pandas dataframe at present, though perhaps we could define our on vector and tensor types.  Just directly using the existing dictionaries via pybind11 objects seems the simplest path forward.

This is now ready to go!

Apologies for not looking into this sooner. I've recently completed the repository cleanup so all of the commits on the feature branch need to be rebased on-top of the revised master branch.

Apologies for not looking into this sooner. I've recently completed the repository cleanup so all of the commits on the feature branch need to be rebased on-top of the revised master branch.

It is unfortunate that we need to rebase so many files with the repository cleanup! Hopefully this will remain a mechanical update.
@cekees this PR already removes a lot of code by avoiding the Cython signature duplication. As we did not want to change everything at once, we figured moving to xtensor structures would be a good intermediary step.
Regarding xframe, the choice to use xframe directly or another "dict of tensors" approach may depend on whether you want to use axes and named dimensions in proteus. We should definitely discuss this further.

@JohanMabille @cekees I think this looks very nice with xtensor and pybind11, directly exposing C++ code to python
I ran a quick check on the test case of a falling sphere with IBM in series on master vs xtensor (hashdist vs conda, both before the repo cleaning, which should not affect the speed), and it looks like xtensor is approximately 2.5 times slower as it is right now with the optimizations disabled. Are we planning to merge this branch with optimizations disabled?
Figure: wallclock time for completing a time step over simulation time (blue: master, orange: xtensor)

The other option is to revert the changes in RANS3PF and keep the Cython until we replace the list of arguments with dictionnary of pyarray.

Since the build is now failing on Conda, let's do that.

This is ready to go now.
@tridelat @zhang-alvin @cekees

@JohanMabille @cekees I think this looks very nice with xtensor and pybind11, directly exposing C++ code to python
I ran a quick check on the test case of a falling sphere with IBM in series on master vs xtensor (hashdist vs conda, both before the repo cleaning, which should not affect the speed), and it looks like xtensor is approximately 2.5 times slower as it is right now with the optimizations disabled. Are we planning to merge this branch with optimizations disabled?
Figure: wallclock time for completing a time step over simulation time (blue: master, orange: xtensor)

Just an update on that, it looks like it might a problem of hashdist vs conda rather than master vs xtensor. I just tested this branch with an hashdist install and it runs as fast as master.
I still get a poorer performance if I run a case with my conda (dev) install vs my hashdist install. Not sure if it is a problem local to my machine or a global problem.

@JohanMabille, @tridelat in the interests of time (and a pending second pass at cleanup) if this doesn't take more than an hour to fix, I suggest punting the removal of the import * statements in several of the changed files to an issue. It doesn't appear to be a big problem to resolve after this is merged in.

Thanks for checking @tridelat . Some of the other hashdist people have mentioned the better performance relative to conda packages, so I'm not surprised. A build from source locally with platform-specific optimizations is usually going to be faster.   I'm happy with this going in if that's the source of the performance difference.

I'm going to ahead and merge this, so we can move forward.

Thanks @zhang-alvin ! Out of curiosity, why merging the master branch into the PR's branch (in genral) when there is no conflict instead of directly merging the PR into master? I think this (merging master into the PR) makes the history more complex and makes the rebasing of other PRs more difficult.
If there are conflicts, I think rebasing the PR is a better option since it keeps the history clean.

Thanks @zhang-alvin ! Out of curiosity, why merging the master branch into the PR's branch (in genral) when there is no conflict instead of directly merging the PR into master? I think this (merging master into the PR) makes the history more complex and makes the rebasing of other PRs more difficult.
If there are conflicts, I think rebasing the PR is a better option since it keeps the history clean.

I had just merged #1075 before merging this PR which introduced a conflict, which was resolved through the merge with master.
As for why not rebasing in this case, I felt like it would be another unnecessary detour to get this merged: you'd need to be alerted of the conflict, then you'd rebase, then the tests run, then we can merge. With the time zone differences, it appears to be a lot of effort when the PR will just be squash merged anyway. This probably merits a proper discussion among the developers.


How to keep my changes to index.html after running jekyll serve

I know that this issue is not necessarily specific to the lanyon, but since I could not find my answer I'm asking this here. Issue is simple, I want to modify the index.html file. I know that if I make the changes to the index.html in _site folder, after running the jekyll serve command the folder content will be overwritten and everything in _site backs to the default. What should I do to keep my changes even after running the jekyll serve command then? I read somewhere that I need to put my file into the _layout folder (anywhere other than _site) which I did but that does not work either.

Don’t modify the content in _site—you need to modify the source files. Find the index.html in the root instead.


Add OS check for removing X button (NEEDS TESTING)

I need someone on windows to test if the X button gets removed.

FIxed running the server on Linux, like I said though, needs testing on WIndows.

Works


Optimize (re-)build time via dependency caching

Highlights of proposed changes within this Pull Request;

Adds matrix for testing and building for multiple environments

We can now target MacOS (+EULA) and/or nightly Rust branch!

Instead of defining SKIP_WASM_BUILD before every command we can
leverage the shared env definitions

Instead of manually installing Rust toolchain we can utilize;

      - name: Install Rust ${{ matrix.rust-branch }}
        uses<EMAIL_ADDRESS>        with:
          toolchain: ${{ matrix.rust-branch }}
          profile: minimal

Persistent caching of dependencies between similar builds should
speed-up subsequent builds

      - name: Utilize Rust shared cached
        uses<EMAIL_ADDRESS>        with:
          key: ${{ matrix.os }}-${{ env.RUST_BIN_DIR }}

Note; as of last tests the above shaves a whole seven minutes when
rebuild conditions are triggered... what a deal!

In the future it may also be a good idea to utilize Action inputs
and outputs, in combination with sccache, and further separate
certain nameed steps into dedicated files

@S0AndS0 you will want to use nightly toolchain nightly-2023-01-18 as per #115

@S0AndS0 you will want to use nightly toolchain nightly-2023-01-18 as per #115

Thanks. I'll get that sorted soon(:tm:)!  Other than that anything else objectionable blocking this merge?


issue 25, codeSet name=SessionRejectReasonCodeSet

Fix for issue 25

Closing as this project is not where the issue can be fixed.


feat: add filesystem based storage

Closes #129
Closes #102

Enforce at least one Content-Digest to be set on each PUT
Compute Content-Digest on PUT client-side #102
Add filesystem-based storage
No deduplication is being done, that will work in a backwards-compatible way by replacing content by a symlink into <repo>/trees

This is what the store structure looks like at the moment:

└── user
    ├── foo
    │   ├── content
    │   ├── meta.json
    │   └── tags
    │       └── 0.1.0
    │           ├── content
    │           ├── meta.json
    │           └── tree
    │               ├── content
    │               └── meta.json
    └── test
        └── bar
            ├── content
            ├── meta.json
            └── tags

Changed from len-sha256 to sha256 for simplicity. I also renamed files from .bson to .bincode because I don't use bson anymore for encoding.

Also using the upstream cap-async-std as my change got merged in.

Feel free to dismiss my review if all PR comments are addressed and we have to move forward. Otherwise, I'd prefer to take a better look on Monday.

@rvolosatovs Yeah I would prefer everyone is okay with this PR before we merge. I would rather wait till monday so you can read stuff over

I'll be out on Monday so feel free to merge at Roman's desire, things left unaddressed can be left for future development.

Please make sure that your commits follow https://www.conventionalcommits.org/en/v1.0.0/
There also probably should not be more than a few (up to 5) commits here. Even 1 is fine IMO, as long as the message and purpose are clear (e.g. feat: implement on-disk storage and a following feat: default to on-disk storage)

@rvolosatovs lgtm, can't self review


API Keys Doc page has outdated UIs

Description:
https://apim.docs.wso2.com/en/next/learn/api-security/api-authentication/secure-apis-using-api-keys/ needs to be updated with the latest screenshots of the UI.
Affected Product Version:
APIM 3.2.0

Fixed with https://github.com/wso2/docs-apim/pull/1748


Support postgresql

from couchbase import LOCKMODE_WAIT
from couchbase.bucket import Bucket
from fastapi import FastAPI

I can use this lib when i'm working with PostgreSQL?

couchbase is a nosql database and postgres is sql.
if you want to use postgres use an orm like sqlalchemy or https://github.com/encode/orm if you need async support

Thanks for the help @SebastianLuebke! And thanks for closing the issue @yezz123! 🍰

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


Problems of building workspace.

Hello everyone. When I was following the instructions, I got a problem.
I am using Ubuntu 16.04 and 64bits.
When I did the last step "catkin build" :
Errors     << bebop_driver:make /home/parrotbebop2/bebop_ws/logs/bebop_driver/build.make.000.log
/usr/bin/ld: /usr/local/lib/libavcodec.a(allcodecs.o): relocation R_X86_64_32 against `ff_h263_vaapi_hwaccel' can not be used when making a shared object; recompile with -fPIC
/usr/local/lib/libavcodec.a: error adding symbols: Bad value
collect2: error: ld returned 1 exit status
make[2]: *** [/home/parrotbebop2/bebop_ws/devel/.private/bebop_driver/lib/libbebop.so] Error 1
make[1]: *** [CMakeFiles/bebop.dir/all] Error 2
make: *** [all] Error 2
cd /home/parrotbebop2/bebop_ws/build/bebop_driver; catkin build --get-env bebop_driver | catkin env -si  /usr/bin/make --jobserver-fds=6,7 -j; cd -
............................................................................................................................
Failed     << bebop_driver:make                    [ Exited with code 2 ]
Failed    <<< bebop_driver                         [ 11.7 seconds ]
Abandoned <<< bebop_tools                          [ Unrelated job failed ]
[build] Summary: 4 of 6 packages succeeded.
[build]   Ignored:   1 packages were skipped or are blacklisted.
[build]   Warnings:  2 packages succeeded with warnings.
[build]   Abandoned: 1 packages were abandoned.
[build]   Failed:    1 packages failed.
[build] Runtime: 2 minutes and 33.2 seconds total.
[build] Note: Workspace packages have changed, please re-source setup files to use them.
Anyone can help? Thanks for your answers :)

Hello everyone. When I was following the instructions, I got a problem.
I am using Ubuntu 16.04 and 64bits.
When I did the last step "catkin build" :
Errors << bebop_driver:make /home/parrotbebop2/bebop_ws/logs/bebop_driver/build.make.000.log
/usr/bin/ld: /usr/local/lib/libavcodec.a(allcodecs.o): relocation R_X86_64_32 against `ff_h263_vaapi_hwaccel' can not be used when making a shared object; recompile with -fPIC
/usr/local/lib/libavcodec.a: error adding symbols: Bad value
collect2: error: ld returned 1 exit status
make[2]: *** [/home/parrotbebop2/bebop_ws/devel/.private/bebop_driver/lib/libbebop.so] Error 1
make[1]: *** [CMakeFiles/bebop.dir/all] Error 2
make: *** [all] Error 2
cd /home/parrotbebop2/bebop_ws/build/bebop_driver; catkin build --get-env bebop_driver | catkin env -si /usr/bin/make --jobserver-fds=6,7 -j; cd -
............................................................................................................................
Failed << bebop_driver:make [ Exited with code 2 ]
Failed <<< bebop_driver [ 11.7 seconds ]
Abandoned <<< bebop_tools [ Unrelated job failed ]
[build] Summary: 4 of 6 packages succeeded.
[build] Ignored: 1 packages were skipped or are blacklisted.
[build] Warnings: 2 packages succeeded with warnings.
[build] Abandoned: 1 packages were abandoned.
[build] Failed: 1 packages failed.
[build] Runtime: 2 minutes and 33.2 seconds total.
[build] Note: Workspace packages have changed, please re-source setup files to use them.
Anyone can help? Thanks for your answers :)


[RF] Update functional derivatives for RC1

Changes proposed in this pull request
Should finish off file name incompatibilities with Derivatives RC1.
https://docs.google.com/document/d/17ebopupQxuRwp7U7TFvS6BH03ALJOgGHufxK8ToAvyI/edit#

Please disregard my last comment. I've just realized that this is only functional derivatives. Ergo, not cached. My apologies.


dbt compile / run is really slow

Please make sure to fill out either the issue template or the feature template and delete the other one!
Issue
when I issue a dbt run --models etc.folders or dbt run ... it takes as much as a minute or three to complete.
Issue description
dbt run --models etc.model or dbt compile --models etc.model
Results
There are about 65 models in that directory and I would expect those to run in a reasonable time (5-15 seconds?)
It actually takes closer to 2-3 minutes.
System information
The output of dbt --version:
installed version: 0.11.1
   latest version: 0.12.1

The operating system you're running on:
OSX 10.13.6 (17G2307)
The python version you're using (probably the output of python --version)
Python 2.7.10
Steps to reproduce
run dbt compile / run in a larger project.
In this case we have about 400 models total in the project, and about 65 in the folder I'm trying to compile.
Ask:
Is this normal execution time and is expected?  It's problematic as we're working with some autogenerated models and I'm trying to exclude the generated files that came out with issues (unrelated problem).  DBT only compiles to the first failure which means I have to wait a couple of minutes for my feedback loop.
Are there any changes  I could make that would expedite my execution?  Thanks!
--edit--
I am aware that v0.12.11 is released and we'll upgrade soon-ish... However our production is still on 0.11.1 and I'd like to be in sync with prod.  However is 0.12 has improvements that will help I'd definitely move now.

hrm, 12.1 isn't available on brew just yet.

Ok, @drewbanin 12.1 is much faster, thanks for the headsup!

@luck02 awesome, glad to hear it! We're making speed one of our priorities going forwards. More to come on that front with https://github.com/fishtown-analytics/dbt/pull/1128 and https://github.com/fishtown-analytics/dbt/pull/1020

I mean, don't get me wrong, it's not super fast either but it's much better...  Still on the slow side.  Thanks for helping me out!

Hi guys, I was wondering if anybody could help me. When running the klaviyo package for dbt and setting the lookback window to 7 days (336 hours),  all dbt models are executed except the last one. Visual Studio Code stops running before all models could be built. Any suggestions?


Add better error logs

Should added some better error logging for third party use cases where extensive testing is not done. IE users extracting and applying their own templates that could potentially have different schemas, primary keys, etc.
Like #46 for instance.

I also get a similar error, there is no translation.
Extracted files Extracted users Extracted roles Extracted permissions Extracted presets  ›   Warning: Error extracting Translations: Extracting template - from **** to .... !     TypeError: first argument must be a string or instance of Error

Adding proper logs in #63


Merge documentation commits from develop

Looks good to me! (OK to merge)

Coverage decreased (-1.5%) to 87.592% when pulling 19bffe366502c2f9fd286e584ee5be557c07a735 on merge-documentation-commits-from-develop into ec83de9fa11bb58d84c2a24a7eefb73270847600 on release/3.2.x.

Coverage decreased (-0.1%) to 88.967% when pulling 19bffe366502c2f9fd286e584ee5be557c07a735 on merge-documentation-commits-from-develop into ec83de9fa11bb58d84c2a24a7eefb73270847600 on release/3.2.x.


MQTT bridge

I am looking to do the following using a single binary with only CLI params,
Broker1 : publishes to /a/b/c
I  want to subscribe to this topic, and republish to Broker2, using the CLI , to topic /x/y/z
What would be the best way to use mochi for this?
eg mqtt --republish --in broker1:1883/a/b/c --out Broker2:1883/x/y/z
Thank you

Isn't this exactly what Benthos (Redpanda connect) was made to solve?

@analystcmyk If you want you can use the internal client and subscribe to the # topic to get a copy of every single message if that works for you.


Fully initializing structs

The test structures in example.c were not fully initialized; this commit forces full initialization of all parts of the structs.  This is not strictly necessary for correct behavior, but helps quiet pedantic warnings.

I've added multiple commits to this branch, you may need to select which commits you want to pull in.


Error on the command scons platform=linux

Tested versions
reproducible in 4.2.stable
System information
Godot v4.2.2.stable - Windows 10.0.19045 - GLES3 (Compatibility) - ANGLE (AMD, AMD Radeon HD 5570 (0x000068D9) Direct3D11 vs_5_0 ps_5_0, D3D11-<IP_ADDRESS>3) () - Intel(R) Pentium(R) CPU G2030 @ 3.00GHz (2 Threads)
Issue description
Hi, I'm working on GDExtension, when I run the command scons platform=linux, I get the error.
I don't know if it is a bug or if I miss something.
PS C:\Users\Dexter\Documents\gdextension_cpp_example> scons platform=linux
scons: Reading SConscript files ...
Auto-detected 2 CPU cores available for build parallelism. Using 2 cores by default. You can override it with the -j argument.
Building for architecture x86_64 on platform linux
scons: done reading SConscript files.
scons: Building targets ...
Linking Static Library godot-cpp\bin\libgodot-cpp.linux.template_debug.x86_64.a ...
scons: C:\Windows\System32\cmd.exe: No such file or directory
scons: *** [godot-cpp\bin\libgodot-cpp.linux.template_debug.x86_64.a] Error 127
scons: building terminated because of errors.

Steps to reproduce

create a GDExtension
run the command scons platform=linux

Minimal reproduction project (MRP)
N/A

If this is only reproducible when building godotengine/godot-cpp you should open an issue there instead

If this is only reproducible when building godotengine/godot-cpp you should open an issue there instead

Sorry my bad


How to redirect to flutter app after sign a message?

It's can redirect app after connect the Metamask.
service.open(context:...)
but how to sign a message and it also can redirect to my app.
Now i sign a message like this.
      var signMsg = await service.web3App!.request(           topic: service.session!.topic,           chainId: 'eip155:1',           request: SessionRequestParams(             method: 'personal_sign',             params: ['Welcome to My app!', service.address]           ));
It can't open the wallet app(like Metamask), but I use url_launcer to open the wallet
launchUrl(Uri.parse('https://metamask.app.link'))
and then perform the sign request. It can jump to the wallet app and show the sign dialog.
but how to redirect to my app after sign the message?

There is a function called launchCurrentWallet in the WalletConnectModalService.
Call that, and it will redirect you as you wish..


Can you change the default transform property before rendering a chart?

Hi, I've been trying to work out how to make a chart be more zoomed-in on render, as some of the sequences I have are very small along a very long genome.
So far, I have succeeded in changing the transform property, but it wont be applied until a user begins to zoom - at this point it jumps very suddenly to the applied transform level.
      featureViewerRef.current.transform = new Transform(2, 0, 0);
      featureViewerRef.current.render({ annotations });

I'm struggling to figure out how to apply the new transform value before the chart is rendered.
Do you have any advice? It would be greatly appreciated.
Thank you!

You'll want some combination of setting a domainConstraint and/or zoomConstraint on the ChartConfig, then providing some start and end values on the RenderParams.
Here's a simple example on codepen, and here's the example code:
let chart = new soda.Chart({
  selector: "div#soda-chart",
  zoomable: true,
  resizable: true,
  // constrains panning; it's a callback
  domainConstraint: () => [0, 100_000],
  // constrains zooming
  zoomConstraint: [0.0, Infinity],
  draw(params) {
    this.addAxis();
    soda.rectangle({
      chart: this,
      annotations: params.annotations
    });
});

fetch("https://sodaviz.org/data/examples/default")
  .then((response) => response.json())
  .then((annotations) =>
    chart.render({
      annotations,
      // set the initial domain
      start: 25_000,
      end: 30_000
    })
  );

Hope that helps!

This works very well, it's a massive help, thank you!

This works very well, it's a massive help, thank you!


File Name Duplicated When File with Same Name Exists on Disc

Describe the Bug

When creating a new file the file name is diagram-1.bpmn, diagram-2.bpmn and so on. If a file with one of those names already exists on disc the file name can be duplicated.

Steps to Reproduce

Create a new file
Save new file on disc using default name
Close modeler
Open modeler
Create new file
New file also has name diagram-1.bpmn

Expected Behavior
No name duplication.
Environment

Camunda Modeler Version: 5.0.0

Is that a bug? I'd rather call this a simple enough, no magic behavior. And how crucial is that? Moving to spring cleaning. If it is easy to fix, why not, if not, lets close it again.

Like @nikku, I'd argue that this is expected default behavior. An application creates a new file with a default name (like "Untitled Document") and this name could already exist at the place where it will be stored. The system save dialog detects the conflict and handles the case.

It's a good hint to rename the file to the real name if the OS mentions that the diagram-1.bpmn already exists.

Can you explain what your expected behavior is, please?
The current behavior is the standard approach, I believe. See recording.
https://user-images.githubusercontent.com/938244/167113153-65dde0ab-77dc-4dff-8b33-e7307814e08e.mov


feat(snippets): add more rounded corners

Adds some changes to round some extra elements.

Seems for some reason it wasn't in there from before, so there was a conflict it seems.
Does this only take effect when the marketplace has an update and isn't fetched automatically?

Seems for some reason it wasn't in there from before, so there was a conflict it seems. Does this only take effect when the marketplace has an update and isn't fetched automatically?

Yeah I think it was because you had an existing fork that was behind upstream. You need to sync your fork so it's up to date or it will be behind and could have merge conflicts.

Seems for some reason it wasn't in there from before, so there was a conflict it seems. Does this only take effect when the marketplace has an update and isn't fetched automatically?

Yeah I think it was because you had an existing fork that was behind upstream. You need to sync your fork so it's up to date or it will be behind and could have merge conflicts.

Yes I tried to do that at the time but it seems it didn't work so I just deleted the local copy to make sure it was all good.
If you don't mind, do you know the answer to my question? Do snippets only get added to the marketplace once a new release of the marketplace occurs? I assume that it is since I don't see it, so it would take maybe a month or so?

If you don't mind, do you know the answer to my question? Do snippets only get added to the marketplace once a new release of the marketplace occurs? I assume that it is since I don't see it, so it would take maybe a month or so?
I think this answers my question though: #435

If I recall correctly, it fetches and caches the snippets from the GitHub repo. So doesn't need a new release, just needs to be merged in. That issue is different. It's proposing we let users put snippets in a manifest file and host them in their own repo, like how the rest of the marketplace works.

If I recall correctly, it fetches and caches the snippets from the GitHub repo. So doesn't need a new release, just needs to be merged in. That issue is different. It's proposing we let users put snippets in a manifest file and host them in their own repo, like how the rest of the marketplace works.

I see. I have tried to do that before and have just now to make sure and I even see the lists populate but I don't see the snippet I added. So you can see it just to clarify? If so i'll have to try a bit harder to make sure I clear everything.

I have it in my snippets, but it looks like it's the version that hasn't been updated. I took another look at the code and it looks like it's actually getting them from the local file (your local copy of the marketplace custom app) and not pulling from github. So you'd have to download the latest build of marketplace to update it. We should probably change it to do it remotely, if you want to make a new issue for that.


chore: add ghcr image mirror

What kind of change does this PR introduce?
Bug fix, feature, docs update, ...
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4838

Relevant Lines:
5564

💛  - Coveralls


[python] Add flake8 and black for code style and integrated to GA

Add describe and configure for code style
Add CI integrated to GitHub action, separator pydolphinscheduler directory in single CI process
Add some documentation about Github action and local run
Change pydolphinscheduler codebase for conform to code style check

Codecov Report

Merging #6679 (0b2db77) into dev (ef9de9d) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #6679      +/-   ##
============================================
- Coverage     38.49%   38.47%   -0.02%
+ Complexity     3227     3224       -3
============================================
  Files           646      646
  Lines         25886    25886
  Branches       2808     2808
============================================
- Hits           9964     9960       -4
- Misses        15008    15013       +5
+ Partials        914      913       -1

Impacted Files
Coverage Δ

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
37.77% <0.00%> (-2.23%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.11% <0.00%> (-1.41%)
:arrow_down:

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 ef9de9d...0b2db77. Read the comment docs.

Thanks

ref: #6407


Adjust the topology of the global trade in the cim scenario.

Description

Linked issue(s)/Pull request(s)

issue_number

Type of Change

[ ] Non-breaking bug fix
[ ] Breaking bug fix
[x] New feature
[ ] Test
[ ] Doc update
[ ] Docker update

Related Component

[x] Simulation toolkit
[ ] RL toolkit
[ ] Distributed toolkit

Has Been Tested

OS:

[x] Windows
[x] Mac OS
[x] Linux

Python version:

[x] 3.6
[x] 3.7

Key information snapshot(s):

Needs Follow Up Actions

[ ] New release package
[ ] New docker image

Checklist

[ ] Add/update the related comments
[ ] Add/update the related tests
[ ] Add/update the related documentations
[ ] Update the dependent downstream modules usage

Update the results of no repositioning and random repositioning in docs/source/scenarios/container_inventory_management.rst together in this PR.

Codecov Report

Merging #169 into v0.2 will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             v0.2     #169   +/-   ##
=======================================
  Coverage   72.45%   72.45%
=======================================
  Files          89       89
  Lines        4139     4139
=======================================
  Hits         2999     2999
  Misses       1140     1140

Flag
Coverage Δ

unittests
72.45% <ø> (ø)

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

Continue to review full report at Codecov.

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

@wesley-stone add a motivation for this pr, which describes why we update the global trade topology.

And if you have an issue with it, please link it. @wesley-stone

@wesley-stone This branch does not work on global scenario, right? I'm trying to integrate the latest data_lib and simulator part of Master branch. The Master's business_engine does register with on_arrival which I think this branch's business_engine doing something similar in step. Can you share your opinion on this. Thanks!

Out of date


Permission to copy your website/code

Your website is beyond awesome.
I want to reuse the code of your website to build my own.
Permission #1
Reuse the code as is and attribute to you with a backlink.
Permission #2
Reuse the static part of the website, change the haskell code to python/javascript. Something more extensible by others.
Haskell is awesome of course but not widely used apart from FP enthusiasts (I count myself as one. Though erlang is my choice)
Permission #3
Make the code of "permission #2" public so others can build on top of it. Under a license you prefer me to add to it for reuse.
Do let me know :)

The code is already under CC-0 (or as applicable for each specific file, some are licensed under other licenses), so no further permission is required. Good luck.


Error: LinkedMap got modified during iteration

Run from sources:

ERR LinkedMap got modified during iteration.: Error: LinkedMap got modified during iteration.
    at LinkedMap.forEach (file:///Users/alex/src/vscode/out/vs/base/common/map.js:627:27)
    at EditorMemento.cleanUp (file:///Users/alex/src/vscode/out/vs/workbench/browser/parts/editor/baseEditor.js:237:19)
    at EditorMemento.saveState (file:///Users/alex/src/vscode/out/vs/workbench/browser/parts/editor/baseEditor.js:229:22)
    at file:///Users/alex/src/vscode/out/vs/workbench/browser/parts/editor/baseEditor.js:103:39
    at Map.forEach (<anonymous>)
    at NativeTextFileEditor.saveState (file:///Users/alex/src/vscode/out/vs/workbench/browser/parts/editor/baseEditor.js:101:44)
    at NativeTextFileEditor.saveState (file:///Users/alex/src/vscode/out/vs/workbench/contrib/files/browser/editors/textFileEditor.js:201:23)
    at file:///Users/alex/src/vscode/out/vs/workbench/common/component.js:17:22
    at Emitter.fire (file:///Users/alex/src/vscode/out/vs/base/common/event.js:457:42)
    at NativeStorageService.flush (file:///Users/alex/src/vscode/out/vs/platform/storage/node/storageService.js:163:39)
    at file:///Users/alex/src/vscode/out/vs/workbench/browser/workbench.js:154:32
    at Emitter.fire (file:///Users/alex/src/vscode/out/vs/base/common/event.js:457:42)
    at file:///Users/alex/src/vscode/out/vs/base/common/event.js:56:106
    at file:///Users/alex/src/vscode/out/vs/base/common/event.js:67:138
    at Emitter.fire (file:///Users/alex/src/vscode/out/vs/base/common/event.js:457:42)
    at file:///Users/alex/src/vscode/out/vs/base/common/event.js:44:93
    at Emitter.fire (file:///Users/alex/src/vscode/out/vs/base/common/event.js:457:42)
    at file:///Users/alex/src/vscode/out/vs/base/common/event.js:56:106
    at Emitter.fire (file:///Users/alex/src/vscode/out/vs/base/common/event.js:457:42)
    at handler (file:///Users/alex/src/vscode/out/vs/base/parts/ipc/common/ipc.js:393:46)
    at ChannelClient.onResponse (file:///Users/alex/src/vscode/out/vs/base/parts/ipc/common/ipc.js:444:17)
    at ChannelClient.onBuffer (file:///Users/alex/src/vscode/out/vs/base/parts/ipc/common/ipc.js:433:33)
    at file:///Users/alex/src/vscode/out/vs/base/parts/ipc/common/ipc.js:302:73
    at Emitter.fire (file:///Users/alex/src/vscode/out/vs/base/common/event.js:457:42)
    at EventEmitter.fn (file:///Users/alex/src/vscode/out/vs/base/common/event.js:257:44)
    at EventEmitter.emit (events.js:203:13)
    at Object.onMessage (electron/js2c/renderer_init.js:2422:16)

cc @dbaeumer

@dbaeumer wanna take this? Looks like a similar issue to what you fixed here:
https://github.com/microsoft/vscode/blob/d3a10a33b3c73f34106bdd7513adb6ed2a431456/src/vs/workbench/browser/parts/editor/baseEditor.ts#L259-L260

@bpasero @dbaeumer
I just hit this in my Selfhost. I noticed since I tried opening the Git Output log and it was simply empty: there wasn't an editor rendered there. The bug was gone after reloading the window, so I can't repro.

Fixed via https://github.com/microsoft/vscode/pull/97603.


Schema version quoting issue?

This error from qbxml looks fishy:
invalid version '7.0' for schema qb, must be one of [\"2.0\", \"CA3.0\", \"3.0\", \"4.0\", \"4.1\", \"5.0\", \"6.0\", \"7.0\", \"8.0\", \"10.0\", \"11.0\", \"12.0\", \"13.0\"]

Are you specifying a number or a string?

This assignment from config/initializers/qbwc.rb is apparently sending a Float into qbxml:
c.min_version = 7.0

Changing it to a string works around and prevents this error, though it still seems that qbxml should be able to handle a Float.

Yeah, I agree.


Remove version: StrawberryMusicPlayer.Strawberry version 1.0.18

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/strawberrymusicplayer/strawberry/releases/download/1.0.18/StrawberrySetup-1.0.18-msvc-x64.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 911)


warning appearing at HA Boot

Describe the bug
No real issue detected
just this warning message appearing in the log
2024-03-18 20:34:52.030 WARNING (MainThread) [homeassistant.util.async_] Detected blocking call to sleep inside the event loop by custom integration 'tuya_local' at custom_components/tuya_local/device.py, line 77: self._api = tinytuya.Device(dev_id, address, local_key), please create a bug report at https://github.com/make-all/tuya-local/issues
To Reproduce
appearing at each reboot
Expected behavior
just this warning nothing seems unworking
Additional context
Global HA Log file with integration in debug mode
home-assistant_tuya_local_2024-03-18T19-44-51.829Z.log
Device Log of humidifier (i believe error comes from it)
tuya_local-694351ebbef2e767e3c3497b10af7285-Dehumidifier - E0 - Cave-c4547f6c7a40464bde4842110fdd92e7.json

Duplicate of #407


ItemSpacing doesn't give the expected result on Android

Platform (please complete the following information):

OS: Android
Device: all tested devices : Samsung A40, Samsung A52s, Huawei P20, with different versions of Android
Xamarin.Forms: <IP_ADDRESS>8 (previous versions not tested)

Describe the bug
ItemSpacing doesn't give the expected result on Android with vertical list (not tested with other layouts)
ItemSpacing = 0 gives a not null spacing
ItemSpacing = 1 gives a bigger spacing than with Xamarin.Forms CollectionView
On iOS behavior is the same as CollectionView.
To Reproduce
Juste play with ItemSpacing values
Screenshots (if applicable)
ItemSpacing=1 with Xamarin.Forms CollectionView

ItemSpacing=1 with Sharpnado CollectionView

Is there something new about this ?

nope probably a won't fix, but any PR will be gladly accepter


Refactor proofs and fix docstrings

@YaelDillies Reverted the inadequate simp only.

Hopefully all necessary changes have been completed.
Thank you @YaelDillies for taking the time to review this PR!

Note that you will have to fix conflict with the master branch

Note that you will have to fix conflict with the master branch

Sure. Thanks.

Note that you will have to fix conflict with the master branch

Done. I just can't make the new "anonymous dot notation" work...


Is Pump_Alarm_Delay_Setpoint  a subclass of Alarm?

Was it intentional? There are several points like that:
INCORRECT: Low_Humidity_Alarm_Setpoint in Alarm
INCORRECT: CRAC_Low_Humidity_Alarm_Setpoint in Alarm
INCORRECT: Pump_Alarm_Delay_Setpoint in Alarm
INCORRECT: Chilled_Water_Pump_Alarm_Delay_Setpoint in Alarm
INCORRECT: CWS_Chilled_Water_Pump_Alarm_Delay_Setpoint in Alarm
INCORRECT: CRAC_High_Humidity_Alarm_Setpoint in Alarm
INCORRECT: High_Humidity_Alarm_Setpoint in Alarm
INCORRECT: CRAC_Low_Temperature_Alarm_Setpoint in Alarm
INCORRECT: Low_Temperature_Alarm_Setpoint in Alarm
INCORRECT: High_Temperature_Alarm_Setpoint in Alarm
INCORRECT: CRAC_High_Temperature_Alarm_Setpoint in Alarm

I will fix this if it wasn't intentional.

Why shouldnt these be alarms? It seems like they all for that description
On Fri, Apr 6, 2018, 7:40 PM Jason Beomkyu Koh<EMAIL_ADDRESS>wrote:

Was it intentional? There are several points like that:
INCORRECT: Low_Humidity_Alarm_Setpoint in Alarm
INCORRECT: CRAC_Low_Humidity_Alarm_Setpoint in Alarm
INCORRECT: Pump_Alarm_Delay_Setpoint in Alarm
INCORRECT: Chilled_Water_Pump_Alarm_Delay_Setpoint in Alarm
INCORRECT: CWS_Chilled_Water_Pump_Alarm_Delay_Setpoint in Alarm
INCORRECT: CRAC_High_Humidity_Alarm_Setpoint in Alarm
INCORRECT: High_Humidity_Alarm_Setpoint in Alarm
INCORRECT: CRAC_Low_Temperature_Alarm_Setpoint in Alarm
INCORRECT: Low_Temperature_Alarm_Setpoint in Alarm
INCORRECT: High_Temperature_Alarm_Setpoint in Alarm
INCORRECT: CRAC_High_Temperature_Alarm_Setpoint in Alarm
I will fix this if it wasn't intentional.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BuildSysUniformMetadata/Brick/issues/40, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AADYipC1m0PWTDiWxf4437-NSmtVsVY_ks5tl_zNgaJpZM4TK1Sn
.

Simply put, Alarm is a read point and Setpoint is a write point.
In my understanding, Low Humidity Alarm Setpoint is a setpoint, of which value indicates whether an alarm should be set or off. I.e., Low Humidity Alarm Setpoint is related to Alarm but it's not a type of Alarm.

Alarm Setpoint is the value at which the alarm triggers, so its a bit of both, but I think I agree that it should be a subclass of Setpoint rather than a subclass of Alarm

resolved by https://github.com/BuildSysUniformMetadata/Brick/pull/42


making _pyo3_get_function pub?

Currently, it seems that wrap_pyfunction will use the private function __pyo3_get_function_foo that #[pyfunction] generated along with the wrapped function foo in a same sub module. This forbids putting #[pyfunction]  in some rust submodules because wrap_pyfunction will not be able to access the private functions.

Since #1280 (i.e. release 0.13.0) they're pub(crate), will this be sufficient for you? This is deliberately undocumented in the hope that one day we'll have a better solution which is easier to use.

Thanks for the help! The reason I could not make it work previously was that I was using pub use submodule::__pyo3_get_function_foo to reexport the companion function. It should be pub(crate) submodule::__pyo3_get_function_foo.


docs: update the blob module specs part 1

Overview
mostly a minor update to the blob module specs, but this PR also adds some validity rules for BlobTxs/PFBs
part of #650
Checklist

[x] New and updated code has appropriate documentation
[x] New and updated code has new and/or updated testing
[x] Required CI checks are passing
[x] Visual proof for any user facing features like CLI or documentation updates
[x] Linked issues closed with keywords

Codecov Report

Merging #1906 (6a52515) into main (0629c75) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1906   +/-   ##
=======================================
  Coverage   21.85%   21.85%
=======================================
  Files         117      117
  Lines       13304    13304
=======================================
  Hits         2907     2907
  Misses      10107    10107
  Partials      290      290

the markdown lint will continue to fail until we merge #1905


Jakub Marszalek

username:
Duffy15

Invitation send. However, this repo was not used for quite some time.


Implement jQuery deferreds action

https://github.com/tbranyen/backbone.layoutmanager/blob/master/backbone.layoutmanager.js#L117
This above location is perfect for adding in some kind of defer-able action instead of doing an awkard done function property binding.

Done.


Add ERW policy

I didn't realize my previous PR was closed with a comment https://github.com/hashgraph/guardian/pull/3480, just checked it now and raised it against the main branch.

@prernaadev01 ^


Fix thread-use causing navigation mesh data corruption

Fixes navigation mesh data corruption caused by thread-use that changed vertices or polygons while the navigation mesh was processed, e.g. by server region sync, navmesh baking or user thread updates.

Adds NavigationMesh  RW locks to make data thread-safe where relevant.
Adds NavigationMesh  C++ functions to set_data() and get_data() in one call to avoid desync.
Moves NavigationMesh property warning checks from server region  sync() to region navigation mesh setter.
Copies vertices and polygon indices for a pending update immediately from resource to server internal.
Removes server region mesh property as it was mostly useless and prevented a practically unused resource from being freed.

Pairs with https://github.com/godotengine/godot/pull/93407
The NavigationMesh has a problematic legacy API where it is possible to set vertices and polygon indices that are not in sync due to both using different functions with no real validation.
When user-threads were manipulating the resource there was opportunity to read the resource while another thread changed the data in the middle. This was very common in large projects when users would reuse the same navigation mesh resource and started the next update as soon as they set the navigation mesh on a region, not knowing that it takes some time for the sync to happen.

Thanks!


New Feature: Only show orders placed in current_store in current_store admin

Love the contribution and can definitely understand the desire for this particular feature.
As currently written, I'm concerned this change will be too restrictive. I know of a couple of stores that run a single Solidus app to power localized versions of their store. In that scenario, it's often valuable for admins to be able to view all orders.
I think that instead of always filtering to the current store, it would be better to provide a way to search for orders belonging to a specific store. It would be easy enough to add that functionality in to core, since core already has some hidden support for multiple stores. Once that search filter is implemented, we could add the ability to link admins to specific stores and limiting their ability to only manage models associated with that store.

I second @adammathys here. It shouldn't be too hard to add another select input to the order table_filter, should it?

Even though core has this functionality hidden, I feel its better placed place with the multi domain gem. I like the approach @mamhoff suggested the addition of a select filter.
Would ability to lock a specific admin user to only seeing the current store's orders sit well within a CanCan permission?

I had a tiny bit of downtime between talks so I tossed together a PR to add searching for orders based on store to Solidus: https://github.com/solidusio/solidus/pull/1140
And I think this sits particularly well within CanCan permissions. At the very least, it makes it easily configurable so that stores can decide how they want to handle things themselves.

Hi @adammathys , Nice work on the PR (solidusio/solidus#1140) during SolidusConf! I'm just not convinced a store select filter should live in core - the functionality is hidden [in core], so should it not remain that way? It also adds a bit of superfluous UI, which could ultimately confuse a non-technical user. Is it not be better to add this filter via a admin deface within this gem?

Hey @SebAshton, while there's not currently any interface built in to Solidus for easily managing multiple stores, the logic for connecting an order to a store is entirely contained within core. There's logic currently used to pick a current store.  All of the logic for assigning an order to a store is in core. (Here and here.)
I can understand the argument for trying to keep as much as possible out of core. But if it's actually possible for a user to create orders belonging to different stores without the use of this extension I believe they should have ability to filter them in the order admin.

This functionality is now available in Solidus: https://github.com/solidusio/solidus/pull/1140. Closing this PR.
Thanks again for prompting the discussion and the initial PR, @SebAshton.


self.bucket_total[index] = value  ValueError: setting an array element with a sequence.

Hello every one
I executed the examples mentioned in adwin.py for jacob but with dataset first, I converted text data to list because an error occurred and the solution was to convert file.csv to list.After that, the following error is appeared
adwin.add_element(data[i])
IndexError: arrays used as indices must be of integer (or boolean) type
I solved the problem by change the instruction to adwin.add_element(data[i.astype('int64')])
finally, I encountered the following error
adwin.add_element(data[i.astype('int64')])
Traceback (most recent call last):
File "", line 2, in
adwin.add_element(data[i.astype('int64')])
File "C:\ProgramData\Anaconda3\lib\site-packages\skmultiflow\drift_detection\adwin.py", line 190, in add_element
self.__insert_element_bucket(0, value, self.list_row_bucket.first)
File "C:\ProgramData\Anaconda3\lib\site-packages\skmultiflow\drift_detection\adwin.py", line 202, in __insert_element_bucket
node.insert_bucket(value, variance)
File "C:\ProgramData\Anaconda3\lib\site-packages\skmultiflow\drift_detection\adwin.py", line 477, in insert_bucket
self.set_total(value, new_item)
File "C:\ProgramData\Anaconda3\lib\site-packages\skmultiflow\drift_detection\adwin.py", line 512, in set_total
self.bucket_total[index] = value
ValueError: setting an array element with a sequence.

help please !!!

Hello every one
I executed the examples mentioned in adwin.py for jacob but with dataset first, I converted text data to list because an error occurred and the solution was to convert file.csv to list.After that, the following error is appeared
adwin.add_element(data[i])
IndexError: arrays used as indices must be of integer (or boolean) type
I solved the problem by change the instruction to  adwin.add_element(data[i.astype('int64')])
finally, I encountered the following error
adwin.add_element(data[i.astype('int64')])
Traceback (most recent call last):
File "", line 2, in
adwin.add_element(data[i.astype('int64')])
File "C:\ProgramData\Anaconda3\lib\site-packages\skmultiflow\drift_detection\adwin.py", line 190, in add_element
self.__insert_element_bucket(0, value, self.list_row_bucket.first)
File "C:\ProgramData\Anaconda3\lib\site-packages\skmultiflow\drift_detection\adwin.py", line 202, in __insert_element_bucket
node.insert_bucket(value, variance)
File "C:\ProgramData\Anaconda3\lib\site-packages\skmultiflow\drift_detection\adwin.py", line 477, in insert_bucket
self.set_total(value, new_item)
File "C:\ProgramData\Anaconda3\lib\site-packages\skmultiflow\drift_detection\adwin.py", line 512, in set_total
self.bucket_total[index] = value
ValueError: setting an array element with a sequence.

help please ?

The add_element method does not take arrays as inputs, only single values (int or float). This is why you are seeing this (expected) error.

Really!!
So,How can I work with real data sets that contain  rows and attributes?
regards,
On Sun, Oct 6, 2019 at 10:50 PM Jacob<EMAIL_ADDRESS>wrote:

The add_element method
https://scikit-multiflow.github.io/scikit-multiflow/_autosummary/skmultiflow.drift_detection.ADWIN.html#skmultiflow.drift_detection.ADWIN.add_element
does not take arrays as inputs, only single values (int or float). This is
why you are seeing this (expected) error.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/scikit-multiflow/scikit-multiflow/issues/146?email_source=notifications&email_token=ALL3HOQ33DOEAR4TEFBKGGDQNI6QHA5CNFSM4I5QSTE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAOSNZY#issuecomment-538781415,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALL3HOTZQRGOIMJ3U47F2YLQNI6QHANCNFSM4I5QSTEQ
.

Closing, no action required.


Live Website Demo

Deploying portfolio website on GitHub Pages
Here is the link to my website.

My Portfolio Website

Hi @IndieCoderMM :smiley: ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights
Descriptive pull request message and title :heavy_check_mark:
Linter error checks validated :heavy_check_mark:
Commit messages are meaningful. :heavy_check_mark:
GitHub flow observed ✔
The website is deployed using GitHub pages ✔
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


SNTP client doesn't handle KOD codes correctly

I found that SNTP client doesn't handle KOD codes correctly when NX_LITTLE_ENDIAN is defined.
Specifically, 'kiss_of_death_handler' is almost always called with NX_SNTP_KISS_OF_DEATH_PACKET instead of the correct codes like NX_SNTP_KOD_DENY, NX_SNTP_KOD_RATE, etc.
This is probably due to the NX_CHANGE_ULONG_ENDIAN macro being applied to the reference_clock_id and the Kiss code being reversed.
https://github.com/azure-rtos/netxduo/blob/185bca73cc6cafbbc612ed94d5866502a020f29c/addons/sntp/nxd_sntp_client.c#L2804-L2807

Hi @wenhui-xie
Is there any update on this issue?
If you need more information, please let me know.

Hi @jkuwaha, I'll take a look at it and get back to you. It will be helpful if you can share me the packet trace. Thanks.

Hi @wenhui-xie, here's a screenshot. Please note that since it's difficult to let the NTP server send KOD packets, I used a script sending crafted packets.

Packet no.2 is the KOD packet I made. As the reference ID is 'DENY' (44-45-4e-59), so it is expected the kiss_of_death_handler is called with NX_SNTP_KOD_DENY, but it is actually called with NX_SNTP_KISS_OF_DEATH_PACKET.
I also found that NetX held the reference ID as 'YNED' (59-4e-45-44). So, it can be said either my crafted KOD packet or NetX's variable is wrong.  However, I think my KOD packet can be ok based on following reasons.
According to https://www.rfc-editor.org/rfc/rfc5905.html#section-7.3, 'reference ID' is interpreted depending on 'stratum';

as a kiss code if stratum is 0
as a reference identifier code, which is a four-octet string, if stratum is 1
as an IPv4 address if stratum is other than above

According to https://www.rfc-editor.org/rfc/rfc5905.html#section-7.4, kiss codes are encoded in four-character ASCII strings.
And also, I found a sample capture containing a reference identifier code at https://wiki.wireshark.org/SampleCaptures.
In 'NTP_sync.pcap', there is a stratum1 packet having 'GPS' (47-50-53-00) as reference ID.  Both kiss codes and reference id codes are string, kiss codes should be stored in the same manner, I think.

@jkuwaha Hi, thanks for your detailed information. The fix will be in the next release.

Close as fixed. The fix is in Azure RTOS 6.1.10.


missing path of hitcount.handler

:bug: Bug Report
The sample code of hitcounter.py line 24
handler='hitcount.handler',
should be:
handler='lambda/hitcount.handler',
otherwise can't locate hitcount module.

Hey @cmrzlg,
Thank you for the report, and sorry for the delay. Actually, the code is correct as is. The folder is referenced in ln 26 code=_lambda.Code.asset('lambda'),. Once the code is located by the cdk you shouldnt need to reference the folder.
Thanks again!
😸 😷


Fix package installation for integration tests

Pre-review checklist

[x] I've confirmed that instructions included in README.md are still correct after my changes in the codebase.
[ ] I've added or updated automated unit tests to verify correctness of my new code.
[ ] I've added or updated integration tests to verify correctness of my new code.
[x] I've confirmed that my changes are working by executing CLI's commands manually on MacOS.
[ ] I've confirmed that my changes are working by executing CLI's commands manually on Windows.
[x] I've confirmed that my changes are up-to-date with the target branch.
[ ] I've described my changes in the release notes.
[x] I've described my changes in the section below.

Changes description
Integration tests broke overnight with no code changes. My investigation seems to point to some global state being cached, perhaps hatch's venvs. This causes an installed version of snowflake-cli and/or snowflake-cli-labs to take precedence over the local code when importing files, causing ModuleNotFoundErrors and tests to act as if the changes being tested aren't there.
The solution I found is to uninstall those two packages since hatch will implicitly do a pip install -e . afterwards, restoring the correct version in the venv.

@sfc-gh-fcampbell pls consider reviewing following PR updating repository name
https://github.com/snowflakedb/snowflake-cli/pull/1690


chore(tslint): add updated tslint.json file (forgot in previous com…

…mit)

Coverage remained the same at 95.745% when pulling 0ea1ee3133cdb40a71d2ad96c15a77b7940bc73f on tinesoft:chore_tslint_file into 4d722a687e70535a80e32df390ec3d10c22faa81 on MurhafSousli:master.


PIE 0.1.0: Could not find release by tag name for mongodb/mongodb-extension:1.20.0

When trying out PIE 0.1.0, I figured out that the @todo above this line actually applies. 😛
The issue was initially reported on the mongodb/mongo-php-driver repo: https://github.com/mongodb/mongo-php-driver/issues/1716
php .\pie.phar install mongodb/mongodb-extension -vvv

Box Requirements Checker
========================

> Using PHP 8.3.12
> PHP is using the following php.ini file:
  C:\Apache24\php\php.ini

> Checking Box requirements:
  ✔ This application requires a PHP version matching "8.1.*||8.2.*||8.3.*||8.4.*".
  ✔ The package "guzzlehttp/guzzle" requires the extension "json".
  ✔ The package "composer/ca-bundle" requires the extension "openssl".
  ✔ The package "composer/ca-bundle" requires the extension "pcre".
  ✔ This application requires the extension "zip".
  ✔ The package "symfony/service-contracts" conflicts with the extension "psr".

Loading config file C:/Users/mv/AppData/Roaming/Composer/config.json
Loading config file C:/Users/mv/AppData/Roaming/Composer/auth.json
Checked CA file C:\Apache24\php\extras\ssl\curl-ca-bundle.crt: valid
Executing command (C:\): git branch -a --no-color --no-abbrev -v
Executing command (C:\): git describe --exact-match --tags
Executing command (CWD): git --version
Executing command (C:\): git log --pretty="%H" -n1 HEAD --no-show-signature
Executing command (C:\): hg branch
Executing command (C:\): fossil branch list
Executing command (C:\): fossil tag list
Executing command (C:\): svn info --xml
Reading C:/Users/mv/AppData/Roaming/Composer/composer.json (C:\Users\mv\AppData\Roaming\Composer\composer.json)
Loading config file C:/Users/mv/AppData/Roaming/Composer/config.json
Loading config file C:/Users/mv/AppData/Roaming/Composer/auth.json
Loading config file C:/Users/mv/AppData/Roaming/Composer/composer.json (C:\Users\mv\AppData\Roaming\Composer\composer.json)
Loading config file C:\Users\mv\AppData\Roaming\Composer/auth.json
Reading C:\Users\mv\AppData\Roaming\Composer/auth.json (C:\Users\mv\AppData\Roaming\Composer\auth.json)
This command may need elevated privileges, and may prompt you for your password.
You are running PHP 8.3.12
Target PHP installation: 8.3.12 ts, vs16, on Windows x86_64 (from C:\Apache24\php\php.exe)
Downloading https://repo.packagist.org/packages.json
[200] https://repo.packagist.org/packages.json
Writing C:/Users/mv/AppData/Local/Composer/repo/https---repo.packagist.org/packages.json into cache
Downloading https://repo.packagist.org/p2/mongodb/mongodb-extension.json
[200] https://repo.packagist.org/p2/mongodb/mongodb-extension.json
Writing C:/Users/mv/AppData/Local/Composer/repo/https---repo.packagist.org/provider-mongodb~mongodb-extension.json into cache
Found package: mongodb/mongodb-extension:1.20.0 which provides ext-mongodb

In CouldNotFindReleaseAsset.php line 28:

  [Php\Pie\Downloading\Exception\CouldNotFindReleaseAsset]
  Could not find release by tag name for mongodb/mongodb-extension:1.20.0

Exception trace:
  at phar://C:/pie.phar/src/Downloading/Exception/CouldNotFindReleaseAsset.php:28
 Php\Pie\Downloading\Exception\CouldNotFindReleaseAsset::forPackageWithMissingTag() at phar://C:/pie.phar/src/Downloading/GithubPackageReleaseAssets.php:100
 Php\Pie\Downloading\GithubPackageReleaseAssets->getReleaseAssetsForPackage() at phar://C:/pie.phar/src/Downloading/GithubPackageReleaseAssets.php:43
 Php\Pie\Downloading\GithubPackageReleaseAssets->findWindowsDownloadUrlForPackage() at phar://C:/pie.phar/src/Downloading/WindowsDownloadAndExtract.php:26
 Php\Pie\Downloading\WindowsDownloadAndExtract->__invoke() at phar://C:/pie.phar/src/Command/CommandHelper.php:183
 Php\Pie\Command\CommandHelper::downloadPackage() at phar://C:/pie.phar/src/Command/InstallCommand.php:52
 Php\Pie\Command\InstallCommand->execute() at phar://C:/pie.phar/vendor/symfony/console/Command/Command.php:326
 Symfony\Component\Console\Command\Command->run() at phar://C:/pie.phar/vendor/symfony/console/Application.php:1078
 Symfony\Component\Console\Application->doRunCommand() at phar://C:/pie.phar/vendor/symfony/console/Application.php:324
 Symfony\Component\Console\Application->doRun() at phar://C:/pie.phar/vendor/symfony/console/Application.php:175
 Symfony\Component\Console\Application->run() at phar://C:/pie.phar/bin/pie:63
 require() at C:\pie.phar:15

install [--with-php-config [WITH-PHP-CONFIG]] [--with-php-path [WITH-PHP-PATH]] [--] <requested-package-and-version>

It would also be nice to see where it is attempting to download the package from when running the install command with the -vvv flag.

I found that issue as well, but thought it wouldn't apply because prebuilt binaries are used on Windows?

I found that issue as well, but thought it wouldn't apply because prebuilt binaries are used on Windows?

Yes - thanks, you're right, I missed that it was for Windows, trying to do too many things at once :)

After the (maybe not so proper) fix of #69, I get the following exception:
Could not find release asset for mongodb/mongodb-extension:1.20.0 named one of "php_mongodb-1.20.0-8.3-ts-vs16-x86_64.zip, php_mongodb-1.20.0-8.3-vs16-ts-x86_64.zip"

This because the release assets of mongodb/mongodb-extension do not contain the 86_, is this a hard requirement or is this something that could be adapted in PIE itself?

The release assets of mongodb/mongodb-extension do not contain the 86_, which PIE expects. Is this a hard requirement, or is this something that could be adapted in PIE itself?

I'll note that the DLLs produced by PECL don't even include that. The architecture suffixes are either "x86" or "x64". That may suggest it's just a PIE fix.

I found out that xdebug has download links with x86_64, so maybe both need to be supported?

We discussed this during the design, and what PIE expects is now the expected format.
This will also match the automatic DLL building github action for PHP extensions that @Shivammathur is working on, and what will be used in xdebug and hopefully all other extensions, so none of them needs to invent their own wheel(s).


client application id is autogenerated during the client application save

in order to improve security we need to have client app id generated on save event instead to be passed by the user

Sometimes you would like to have control over what client_id uses, but sometimes not.
The feature is intended to have an endpoint to generate random id


[FEATURE] Report name of operators and arguments that faile to type check

Currently, type check failures on operators just give the operator signature and the types of the arguments that cause the checking failure. E.g.,
[TwoPhase.tla:68:6-68:21]: No match between operator signature ((a156, a156) => Bool) and arguments Set(Str) and Str E@14:41:05.402
[TwoPhase.tla:63:1-70:14]: Error when computing the type of TPInit E@14:41:05.410

This is for a case when an equality statement is given the wrong arguments. The offending lne is
/\ tmState = "init"

It'd be much more helpful if the error was something like
No match between operator signature for "=" ((a156, a156) => Bool) and arguments Set(Str) and Str
in expression 'tmState = "init"`

Tracking the arguments might be tough, but I think we should at least be able to report the name of the operator?

True. I ran into that several times too!


WIP: Basic integration testing

Current problems:

IP address of clusterhost
Release pipeline sends emails pipeline fails if unable to send.

@codificat ok I think this is ready to go.


Automatic OWASP Test generation

This article in API's You Wont Hate triggered me to think about automatically adding some OWASP variation tests by Portman.
Questions:

Would that be a useful addition to Portman?
Which security test should/could be added automatically?
And how should this be configured? A simple boolean "InjectOWASP" or more fine-grained?)

Looking for some input from the community: @nicklloyd @danielkocot @tillig @savage-alex

Summon @miriamgreis

Great idea. This will make basic API security testing much more accessible if Portman supports this out-of-the-box.

Is it possible to support the top 10? https://owasp.org/www-project-api-security/

OWASP ZAP already handles this and also takes in an OAS doc. It supports a lot of scan rules, some of which don't necessarily apply to REST APIs (but if you run it and specify it's an API it will adjust).
While I think additional tests are interesting, I would personally probably always keep this off and just run the Docker version of OWASP ZAP.
If this does get added, it may be interesting to consider some sort of plugin model for Portman, where it's not "all built in" but instead has "test generation plugins" - one plugin generates contract tests, one plugin generates security tests... and if you want to add your own plugins, you can. That'd potentially be a pretty big lift from an internals perspective, but I think it'd be valuable going forward - now folks want some other kind of test generated? Add a plugin. Need to test the various test generators in isolation? Add/remove plugins.

@tillig
It does make sense to keep the Portman core focussed on the current use-cases and not expand it in all directions.
A plugin model, where it is possible to extend the Portman core with specific extensions, would be a great way to facilitate the specific needs from the community. It would also mean a large rework on the internals, so something we could take into account when modernising Portman.


Json parse issue

root@linux# ./mtr-exporter -bind <IP_ADDRESS>:8080 -mtr /root/mtr-0.92/mtr -- <IP_ADDRESS>
serving /metrics at <IP_ADDRESS>:8080 ...
launching /root/mtr-0.92/mtr -j <IP_ADDRESS>
failed: json: cannot unmarshal string into Go struct field mtrMtr.report.mtr.tos of type int64
If I launch mtr manually, the output is:
{
"report": {
"mtr": {
"src": "influx",
"dst": "<IP_ADDRESS>",
"tos": "0x0",
"psize": "64",
"bitpattern": "0x00",
"tests": "10"
},
"hubs": [{
"count": "1",
"host": "<IP_ADDRESS>",
"Loss%": 0.00,
"Snt": 10,
"Last": 0.57,
"Avg": 0.62,
"Best": 0.53,
"Wrst": 0.74,
"StDev": 0.07
},
...
Any clues?

@diego-tironi which version of mtr do you have installed on your system to produce the output (mtr, not mtr-exporter).
in general, i wonder why mtr would yield both .tos and .tests now as strings (while the content are numbers). maybe they have change the format underneath a bit.

@diego-tironi aaah, it looks like that behaviour was introduced in mtr:0.94 ~ 2020: https://github.com/traviscross/mtr/pull/355 … which led to https://github.com/mgumz/mtr-exporter/commit/15cfd19c6205ad7dc7af7b57761e670550d092e8 in the past.
so, your mtr version smells like the old flavour.

@mgumz bingo! I compiled the last version of mtr (with jansson support) from the git repository and the issue went away.
Thank you very much!

Because some other people also went into this … not so obvious … trap: 39711bfe7298b951abdd682aa1edf4ab83361d80


Change docs target to build _opae before building docs

Yes I think so. Otherwise the python api spec will not get populated properly. Currently it won't get populated because _opae is getting built after the docs are built, this change resolves that issue.


Deployment exception

Hi trying to deploy project in glassfish 4.1.1 i got the below exception.Using Presto 0.145
Payara 4.1|javax.enterprise.system.core|_ThreadID=94;_ThreadName=admin-thread-pool(6);_TimeMillis=1493383149301;_LevelValue=1000;|Exception while deploying the app [hopsworks-api] : The lifecycle method [stop] must not throw a checked exception. Related annotation information: annotation [@javax.annotation.PreDestroy()] on annotated element [public void io.airlift.http.server.HttpServer.stop() throws java.lang.Exception] of type [METHOD]
The lifecycle method [stop] must not throw a checked exception. Related annotation information: annotation [@javax.annotation.PreDestroy()] on annotated element [public void io.airlift.http.server.HttpServer.stop() throws java.lang.Exception] of type [METHOD]
at org.glassfish.apf.impl.AnnotationProcessorImpl.process(AnnotationProcessorImpl.java:375)
at org.glassfish.apf.impl.AnnotationProcessorImpl.process(AnnotationProcessorImpl.java:383)
at org.glassfish.apf.impl.AnnotationProcessorImpl.processAnnotations(AnnotationProcessor
If anyone got this exception, please help me.
Thanks in Advance
Aruna

It seems like glassfish has extra restrictions on @PreDestroy that we do not have.  I don't think this is something we are likely to change.  Maybe you can check if there is a way to disable this annotation scanning in Glassfish.


Mctc-gcp

Ignore until #288 rebased away.
Description

Changelog description

Status

[x] Code base linted
[ ] Ready to go

Ignoring the test failures for now (waiting for a pydantic v0.8.1 release on c-f), I think this is ready for review, @hokru and @awvwgk.

What is the reason to put the new gCP driver in the xtb environment? The more interesting case IMO would be to have both gCP drivers in the same environment, if they can life peacefully together, or together with DFT harness which can actually make use of the gCP correction.

Yes, a dispersion lane for testing would be nice. I've run gcp (intel, iomp5) and mctc-gcp (openblas, gomp) together locally, but psi4 and mctc-gcp can't install together at present. Glad to re-sort in future as variants multiply.


Added isLevelEnabled(string) & isXXXEnabled()

Make logger expose functions to check if a particular level is enabled. This could be useful if either message or parameter creation is an expensive operation, which should be avoided when log level is disabled.
Logger will contain isLevelEnabled(level: string) function and multiple isXXXEnabled() where "XXX" is the capitalized name of the configured log level. E.g. isDebugEnabled(), isInfoEnabled(), etc.
Resolves https://github.com/winstonjs/winston/issues/905

Hello and thanks for this logging library!
This PR is a proposal and requires some cleanup - tests could be improved and jsdocs are missing. Please let me know if this is even a valid approach and useful feature in general.

Thanks for submitting this PR. This was a previously requested feature, I don't see any strong reason why this won't land soon.

@indexzero do you think implementation in this PR is acceptable? does it require any adjustments?

@lutovich looks like I made some changes in another PR that caused git conflicts here. Would you mind resolving them? Then we should be good to go for merging this 😸

@indexzero PR is now rebased :)

Would be awesome to merge and release this :)

In Winston 1.X I remember the logger would do this kind of check up front and therefore not process the second arg (the meta or whatever it's called). We noticed performance decrease in certain cases after we moved to Winston 2.X because this was no longer the case.
Any change this request can go one step future and embed the logic in the logger.log() method? We actually wrapped our Winston logger in a Proxy to do exactly this.

Just merged this – it will go out in the next release, which should hopefully be published very soon.
I think the comment above is describing a separate issue about the behavior of the log function.   If you see any places where we are not correctly optimizing for the hot paths, especially in the log function, on master, please feel free to open a separate issue or PR so we can take a look. Thanks!

@DABH thank you for all you do! And yes, in the off chance I get some spare time I'll try to put together something formal. 🎩

Did this ever get documented anywhere though?
I can only find mention of it in GitHub threads. This is a very useful feature for efficient log practices and should be in the docs I would say!

@peternann If you don’t spot anything relevant in the docs, I’m sure other devs have similar feelings :) A PR is welcomed!


Site not loading over HTTPS

Howdy,
In Gentoo (Linux), on Chrome 40 (detailed below), I found the streamus.com site not loading over HTTPS, with the errors as shown below (namely, the CDN for the scripts not being over HTTPS as well).
Things like this remind me I'm using HTTPS Everywhere, hehe.
-[~:$]- chromium --version
Chromium 40.0.2214.111

Hey,
Uhhh... clkmon.com is adware! I think your computer is infected and
Streamus is preventing itself from loading because it doesn't want to
expose sensitive data. There's no reference to clkmon in my source code.
Thoughts?
On Fri, May 1, 2015 at 9:39 AM, Matt Vandermeulen<EMAIL_ADDRESS>wrote:

Howdy,
In Gentoo (Linux), on Chrome 40 (detailed below), I found the streamus.com
site not loading over HTTPS, with the errors as shown below (namely, the
CDN for the scripts not being over HTTPS as well).
Things like this remind me I'm using HTTPS Everywhere, hehe.
-[~:$]- chromium --version
Chromium 40.0.2214.111
[image: im an image!]
https://camo.githubusercontent.com/d09ed775d8881fa60f236f635fffe2ab1a509558/68747470733a2f2f7265656e69676e652e6e65742f73637265656e73686f74732f313433303439383239382e706e67
—
Reply to this email directly or view it on GitHub
https://github.com/MeoMix/StreamusChromeExtension/issues/499.

Whoop, that was coming from another extension (that's apparently being quite greedy).
I am still hitting that second error though, as seen here, which is leaving me with a blank page.
Expanding the error shows some Backbone traces.  Have you experienced this before?

Oooh, I know that one, actually. Huh. Okay, I'll put some defensive checks in there.
chrome.runtime is not defined... which it should be on all webkit browsers. Using chrome.runtime from a website is how I allow for adding playlists from the website and detecting that Streamus is installed. It's weird that your browser doesn't support it but does identify as WebKit... might be worth filing a bug. Can I get some more details on your exact OS and Browser versions (down to minor vers) and I'll check for any bug reports on Google's side.

Ah, I see what you mean.

And here's a pastie of some eix and uname output for this box.  I'm about to head out to lunch, so I'll actually update chromium, which will take some time to compile :).
My desktop at home, where I threw streamus onto this morning is running 41.0.2272.76 which worked like a charm.  I'll be running 42.0.2311.90 in about an hour... probably.

After finally closing everything out for the day, and trying in the new version of Chromium, I can safely say the site is working as intended, and that chrome.runtime exists in this version.
I'll leave it to you to close when you're ready.  Thanks!

Good to know, thanks for the insight.