Rename prewarmed containers

Rename prewarmed containers when initialized
Description

Currently, when prewarmed container moves out from prewarmed pool, it keeps its "prewarmed" name, e.g. wsk00_236_prewarm_nodejs6 while if it wouldn't be taken from prewarmed pool it could have name like wsk00_240_guest_hello which is based on namespace_action name.
In this case, when observing currently running containers, one may have number of containers having default "prewarm" in the name, exceeding the number of prewarmed containers specified in the config.
As well, it may present issues when it comes to monitoring, as containers name differ from expected
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[x] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Basically, the idea of renaming pre-warmed Docker action containers when they are inited makes sense. As you pointed out, a simple docker ps -a --filter 'name=wsk*' will give you all action containers. With your renaming proposal, all containers containing prewarm in their name are pre-warmed while all others are warm - ignoring error conditions for the moment.
The downside of renaming is that it takes time. It will add to the latency of pre-warmed invocations, i.e. Node.js invocations will take longer. Did you run any specific performance benchmarks to understand the effects of renaming or did you compare performance test results of Travis builds?
I only had a very short look at your code changes. You are adding a rename() operation to the ContainerProxy workflow - but it seems that you don't test whether renames are properly performed in ContainerProxy tests.

Codecov Report

Merging #4082 into master will decrease coverage by 4.93%.
The diff coverage is 75%.

@@            Coverage Diff            @@
##           master   #4082      +/-   ##
=========================================
- Coverage   86.14%   81.2%   -4.94%
=========================================
  Files         148     148
  Lines        7219    7232      +13
  Branches      438     449      +11
=========================================
- Hits         6219    5873     -346
- Misses       1000    1359     +359

Impacted Files
Coverage Δ

...la/src/main/scala/whisk/core/mesos/MesosTask.scala
83.78% <0%> (-2.33%)
:arrow_down:

...containerpool/kubernetes/KubernetesContainer.scala
90% <0%> (-4.74%)
:arrow_down:

...re/containerpool/kubernetes/KubernetesClient.scala
27.27% <0%> (ø)
:arrow_up:

...ain/scala/whisk/core/containerpool/Container.scala
81.94% <100%> (+0.25%)
:arrow_up:

...whisk/core/containerpool/docker/DockerClient.scala
98% <100%> (+0.04%)
:arrow_up:

...cala/whisk/core/containerpool/ContainerProxy.scala
94.05% <100%> (-2.04%)
:arrow_down:

...sk/core/containerpool/docker/DockerContainer.scala
96.34% <66.66%> (-1.13%)
:arrow_down:

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-95.54%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-92.6%)
:arrow_down:

... and 6 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 8851cab...236e4b1. Read the comment docs.

Motivating use case is when we are using the DockerContainerFactory on Kubernetes, where the container names will be visible to operators (who may in fact be users, in the case of deploying OpenWhisk on self-managed kube clusters).  By changing the name of the container, platform-level metric services that include the container name will give the user/operator useful information about which functions are consuming resources.

Thanks Dave. So this has to happen early on in the container life cycle then?

@dgrove-oss Even understanding the use case, it seems wrong to track state via name (especially given the concurrency/locking concerns Rodric raised despite appearing to be asynchronous). My main concern is that there is nothing actionable an operator can do from seeing a collection of named containers which may indicate some resource consumption state.  At best, this approach seems to be an attempt at a stop-gap measure for "container watchers", but would require deeper analysis via log/monitoring tools to be actionable. Is it worth going down this road and raising the expectations of operators that name will reflect the actual state with fidelity?

is it ok to do the operation late (after run?) vs early (before init)?

Based on @kpavel's comment that the name doesn't matter for suspend/resume since we use the ContainerID, I would be inclined to do it asynchronously eaerly as is being done now, but with some error handling so that if the rename fails for some reason we at least get an error log.

But if it increases load in the daemon then this could be a concern in production - we limit the max concurrent docker operation iirc.

We should probably understand what versions of docker are actually being used in production by the various operators these days.  For example, IBM is no longer on 1.12.  Maybe that is a dev-list discussion (to go with a discussion of #4090).

@dgrove-oss IBM is partly still at Docker 1.12 - but no more with production environment.
Even if we docker rename pre-warmed containers asynchronously in the container proxy, there is a chance that Docker operations on the same container are run consecutively by the Docker daemon. And if not consecutively, there are probably per-container or global locks that prevent fully parallel execution.
We definitely need performance tests to evaluate the effects of a docker rename.

Going to close this.  There was not a clear consensus that we wanted to do this, the PR has gotten out of date, and there's a general movement from docker to containerd as the lowlevel engine in Kubernetes.


Work and section pages are cut off on the bottom on mobile

This is likely due to the header sticking to the top when scrolling, meaning you can't scroll down far enough to see the rest.

this no longer happens on my phone, but i'll keep this open in case someone else raises a similar issue (like on Android)

closing this for now. will reopen if someone raises a similar issue


Emojis in checkbox’s text will misalign the text from the checkbox.

Emojis in checkbox’s text will misalign the text from the checkbox. Pure text works fine. (Safari). Baseline issue

Screenshot or didn’t happen 😃

I don’t think there’s anything we can do about this, this is how Safari renders the text. When you apply a custom line-height, it fixes the issue. So I’m assuming emojis have a different default line height than regular text.

Closing as wontfix


Fix formatting in CSE doc table

For the Quick reference to configuring ingest mappings section, the table is not formatted and just shows plain text. https://help.sumologic.com/docs/cse/ingestion/sumo-logic-ingest-mapping/#quick-reference-to-configuring-ingest-mappings
Slack Message

table is fixed: https://help.sumologic.com/docs/cse/ingestion/sumo-logic-ingest-mapping/#quick-reference-to-configuring-ingest-mappings


Removed some formatted messages

Description

Removed some formatted messages
Fixed duplicated glitchtip errors

Type of change

Refactor (refactoring code, removing useless files)

Testing steps
To be checked on CI.
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Patch coverage is 100.00% of modified lines.

Files Changed
Coverage

server/server.go
100.00%

types/types.go
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


compreface is detecting male as female

Describe the bug
compreface is detecting male as female in more than 90% of the cases.
But the female gender has no issues.
To Reproduce
Steps to reproduce the behavior:

I am creating the images using "docker-compose up -d" cmd
then going to localhost:8000 >> create account >> login >> create app and then add detection service
then click on "test" button and upload image to test.
in most cases, female is detected as female correctly but male also gets detected as female.

Expected behavior
male should be male
Screenshots
please see attached examples.
Desktop (please complete the following information):
-not dependent on browser
Logs
Run those commands and attach result to the ticket:
docker ps
docker-compose logs
both attached here.
Additional context
It worked last week but somehow got into this state. I tried wiping off docker s/w, reinstall, tried all sorts of things but still same problem. I have also tried compreface 1.1 as well as 1.2 - both have same problem with all different docker versions.

docker_ps.log
docker-comp-logs_new (2).txt

Can you kindly assign me this open issue?
Thanks

I don't have access to assign the issues, may be someone else/admin can help us.


[SIG-scale] Goroutine count and memory remains high after VMIs are removed

Summary
Create n VMIs, wait until they are all running, delete n VMIs, wait until they are all deleted, sleep 5s, and repeat. Scenarios for 10, 20, 40, 60, 80, 100, 200, 300 VMIs shows an increase in virt-handler mem/cpu usage and go routines, however only cpu usage returns back to the expected level when there are 0 VMIs.
KubeVirt control plane metrics

Metric

Name
Description
Metric

Memory
Amount of memory used
sum(process_resident_memory_bytes{job=~".*kubevirt.*"}) by (instance, container)

CPU usage
Tme spent using the CPU
sum(rate(process_cpu_seconds_total{job=~".*kubevirt.*"}[5m])) by (instance, container)

Goroutines
Amount of goroutines being used
go_goroutines{job=~".*kubevirt.*", cluster="$cluster"}

Network
Amount of network being used
sum(rate(container_network_receive_bytes_total{namespace=~".*kubevirt.*"}[5m])) by (node, pod) + sum(rate(container_network_transmit_bytes_total{namespace=~".*kubevirt.*"}[5m])) by (node, pod)

/kind bug

[ ] https://github.com/kubevirt/kubevirt/pull/6168

Fixed by #6176 and #6168


[Bug] Fail to execute GPTQ conversion script

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[X] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.

Describe the bug
Hi thanks for the library! However the lite's gptq conversion does not seem to work.

Environment
use docker

Error traceback
CUDA extension not installed.
CUDA extension not installed.
Loading calibrate dataset ...
Token indices sequence length is longer than the specified maximum sequence length for this model (1104485 > 131072). Running this sequence through the model will result in indexing errors
/opt/py3/lib/python3.10/site-packages/huggingface_hub/file_download.py:797: FutureWarning: `resume_download` is deprecated and will be removed in version 1.0.0. Downloads always resume when possible. If you want to force a new download, use `force_download=True`.
  warnings.warn(
Loading checkpoint shards: 100%|██████████| 4/4 [00:01<00:00,  2.30it/s]
Traceback (most recent call last):
  File "/opt/py3/bin/lmdeploy", line 33, in <module>
    sys.exit(load_entry_point('lmdeploy', 'console_scripts', 'lmdeploy')())
  File "/opt/lmdeploy/lmdeploy/cli/entrypoint.py", line 42, in run
    args.run(args)
  File "/opt/lmdeploy/lmdeploy/cli/lite.py", line 138, in auto_gptq
    auto_gptq(**kwargs)
  File "/opt/lmdeploy/lmdeploy/lite/apis/gptq.py", line 93, in auto_gptq
    model.quantize(examples, batch_size=batch_size)
  File "/opt/py3/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/opt/py3/lib/python3.10/site-packages/auto_gptq/modeling/_base.py", line 392, in quantize
    self.model(**example)
  File "/opt/py3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/opt/py3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/opt/py3/lib/python3.10/site-packages/transformers/models/qwen2/modeling_qwen2.py", line 1164, in forward
    outputs = self.model(
  File "/opt/py3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/opt/py3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/opt/py3/lib/python3.10/site-packages/transformers/models/qwen2/modeling_qwen2.py", line 871, in forward
    position_embeddings = self.rotary_emb(hidden_states, position_ids)
  File "/opt/py3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/opt/py3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
    return forward_call(*args, **kwargs)
  File "/opt/py3/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/opt/py3/lib/python3.10/site-packages/transformers/models/qwen2/modeling_qwen2.py", line 163, in forward
    freqs = (inv_freq_expanded.float() @ position_ids_expanded.float()).transpose(1, 2)
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cpu and cuda:0! (when checking argument for argument mat2 in method wrapper_CUDA_bmm)

Install autogptq from its master branch instead of stable version solves the problem


Allow tree to implement custom cacheKey

I could be wrong (close if I am), but it seems like the individual tree can no longer specify its own cacheKey. Instead it always calls hashForDep(ctx.baseDir()) (here), but sometimes you need to make a custom cacheKey that is not related to package.json (for example: bower dependencies).

Ya we should restore a hook on the plugin itself


Validate access token for authentication

At the moment only basic authentication is performed. Additionally I would like to validate the access token for authentication if one is present. I have code for this. Let me know if you are interested in a PR.

Hi! Feel free to make a PR


Update Office ASP.NET SSO sample

Q
A

Bug fix?
no

New feature?
no

New sample?
no

Related issues?
---

What's in this Pull Request?

Remove fallback code as it now requires a second app registration which is a bit complex to include in this sample at this time.
Upgrade to use ASP.NET Core.

@AlexJerabek I updated the readme, so should be good now. Thanks!


关于ureplicator 非联邦 模式的性能问题

Hi:
I have an urgent question to enquire.
My question is about the perfomance of ureplicator. Currently, ureplicators are installed on 3 machines, called master01, master02 and master03 separaterly. The controller roles are started on master01 and master02, and all of the three machines start worker roles.
The speed of copying data within kafka is 3, 000, 000/min, while my goal is around 10, 000, 000/min. If another worker role is added, the speed would be up to 4, 000, 000 per minute.
Since the management cost would be increased when adding more workers here, I wonder if there is any way (setting\optimize) can improve the speed of the task without adding any new worker.
Images below are my settings. The code I used is from branch 0.1.

The command to start the role is:
controller

worker

The jvm optimization of the script to start worker role is:

1: what is your TCP round trip latency
2: what is your replication byte rate
Also, you probably need to check the graphite metrics to figure out either consumer or producer is not fast enough

HI:
1.My TCP latency is：

2.what is replication byte rate？ Where can I find it？
3.how check the graphite metrics to figure?
You can write to me personally if it's convenient,Thank you very much.
Email<EMAIL_ADDRESS>WeChat: huml_1005

1/ Please try to change "ling.ms" in producer configuration to 1000
2/ to check the byte rate, you would need to export uReplicator Worker metrics to graphite, but please try 1/ first.

hi:
Thank you very mach for you reply.
1.I try to change "ling.ms"  in producer configuration to 1000,After a period of time, the speed will increase,for example:
After I apply for a topic copy, the replication speed in the first five minutes is slower than that in the next five minutes
2.I want to check the byte rate,but i don't export uReplicator Worker metrics in graphite, Do you have any documentation or configuration or steps?

for 0.1 branch, worker metrics are export through jmx, there is an example in the user https://github.com/uber/uReplicator/wiki/uReplicator-User-Guide and the jmx configuration example: https://github.com/uber/uReplicator/blob/master/config/jmxtrans.xml
The metrics worth checking are
Consumer side: bytes-consumed-rateTotalBlockingQueueSize, ConsumerLag,MirrorMaker-flushLatencyMs,MirrorMaker-commitLatencyMs
Producer side: buffer-available-bytes, requests-in-flight,

hi:
Thank you very mach for you reply.
1.I tried export uReplicator Worker metrics in graphite ,but i encounter a problem

My configuration is :

hi:
Thank you very mach , i try to install graphite,but this cannot be installed offline
so i use master code and federation mode, at present, 3 worker ,3 controller,1 manager TPS arrive  230, 000/second.
i will colse this issue
Thank you very mach


Could not load local index file 'ncbiRefSeq.txt.gz.tbi'

Hi there,
I was running create_report while a warning came out as below,
Could not load local index file 'https://hgdownload.soe.ucsc.edu/goldenPath/hg19/database/ncbiRefSeq.txt.gz.tbi' : No such file or directory
I have tried to find a parameter for this index in create_report, but there is no argument for this file.
Please let me know if I missed something.
Thanks,
Junhui

The warning is normal, you can ignore it.


SDC fixed Instrumented Android unit tests

Addressing issue #149

apologies, after reading https://kotlinlang.org/docs/reference/coding-conventions.html#formatting i realise the kotlin styleguide advocates for 4 whitespaces... i will update my IDE settings.

apologies, after reading https://kotlinlang.org/docs/reference/coding-conventions.html#formatting i realise the kotlin styleguide advocates for 4 whitespaces... i will update my IDE settings.

No problem I'll update it in my IDE too. I'll merge this PR after the checks are complete.

No problem I'll update it in my IDE too. I'll merge this PR after the checks are complete.


fix: make node moving consistant

Fixes: #700

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


Fix: [M3-6612] Grid/Tag views and toggle styling (Linodes Landing)

Description 📝
This code is a mess. There's so much duplication and it needs a serious refactor. Trying to have a minimal footprint here so only addressing styles and some exports.
The PR achieves a couple things:

Add the active/toggled color on the icon
Cleans up the styles and consolidates grid/list styles and layout

Note: default view (no summary view, no tags) should be unchanged
Preview 📷

Before
After

How to test 🧪

Get to Linodes Landing and toggle all the things

@bnussman-akamai you're right! I improved the hover styles to reflect that


What about support NHibernate 4.1?

If i install last version of NHibernate from NuGet. Now it is NHibernate <IP_ADDRESS>00.
Fluent NHibernate 2.0.3 installed also from NuGet, doesn't work. Because it want version  <IP_ADDRESS>00,
although Fluent NHibernate 2.0.3 well work with NHibernate <IP_ADDRESS>00.

I have the same problem

Let me try that and I'll get back to you. Nuget package doesn't specify an upper bound on NHibernate version - so the problem is odd

So I tried 2 scenarios (VS 2013 with .NET 4.5 project, only installed packages without verifying if it works with db):

installing FluentNHibernate to a project with no previous NH reference installs NH <IP_ADDRESS>00 (which is odd, but I guess that's a nuget behaviour)
installing latest NH (<IP_ADDRESS>00 atm) with no previous Fluent reference and then installing Fluent work ok.

Which one of these have you tried?

I have NH <IP_ADDRESS>00 and Fluent 2.0.3 from before, then I updated to NH <IP_ADDRESS>00. Then i had problem with DB

Which problem? What does the exception say?
2017-01-10 13:02 GMT+03:00 elinato<EMAIL_ADDRESS>
I have NH <IP_ADDRESS>00 and Fluent 2.0.3 from before, then I updated to NH
<IP_ADDRESS>00. Then i had problem with DB
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-271534537,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfIviikuE243bD7tPqydltz6Tkve-ks5rQ1cpgaJpZM4LW02A
.

--
Yours faithfully,
Gleb

`Details: System.IO.FileLoadException: Could not load file or assembly 'NHibernate, Version=<IP_ADDRESS>00, Culture=neutral, PublicKeyToken=aa95f207798dfdb4' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
File name: 'NHibernate, Version=<IP_ADDRESS>00, Culture=neutral, PublicKeyToken=aa95f207798dfdb4'`

Can you send me your .csproj file or a part of it that references Fluent
NHibernate and NHibernate dlls? I suspect it's a broken reference - e.g.
you have downloaded <IP_ADDRESS>00, but the project still wants to find
<IP_ADDRESS>00
2017-01-10 13:19 GMT+03:00 elinato<EMAIL_ADDRESS>
`Details: System.IO.FileLoadException: Could not load file or assembly
'NHibernate, Version=<IP_ADDRESS>00, Culture=neutral, PublicKeyToken=aa95f207798dfdb4'
or one of its dependencies. The located assembly's manifest definition does
not match the assembly reference. (Exception from HRESULT: 0x80131040)
File name: 'NHibernate, Version=<IP_ADDRESS>00, Culture=neutral,
PublicKeyToken=aa95f207798dfdb4'`
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-271538057,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfF_GlhNGb5FtBDKigQWEVTT7RcOiks5rQ1sXgaJpZM4LW02A
.

--
Yours faithfully,
Gleb

check your app.config for a  it should like  (not <IP_ADDRESS>-<IP_ADDRESS>00)

That's also true. I don't remember if VS updates assembly redirects when updating packages

Yeah... I have some in app.config. Will correct it and test

I had missing dependent assembly in my web.config and app.config
 <dependentAssembly>         <assemblyIdentity name="NHibernate" publicKeyToken="aa95f207798dfdb4" culture="neutral" />         <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>00" newVersion="<IP_ADDRESS>00" />       </dependentAssembly>
after I added it it seems to solve the problem.
FluentnHibernate looking for the old nHibernate <IP_ADDRESS>00, that's why need this reference in config files

Fluent NHibernate isn't looking for old NH.
Usually people have a redirect if they had older versions of the assembly.
2017-01-12 15:49 GMT+03:00 elinato<EMAIL_ADDRESS>
I had missing dependent assembly in my web.config and app.config

after I added it it seems to solve the problem.
FluentnHibernate looking for the old nHibernate <IP_ADDRESS>00, that's why
need this reference in config files
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-272155040,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfOfRRWrrfQY2pie6Li38Ln8FW4hdks5rRiFGgaJpZM4LW02A
.

--
Yours faithfully,
Gleb

Closing this

I can confirm that an assembly redirect is required if NHibernate 4.1 is used with Fluent NHibernate 2.0.3. Otherwise Fluent NHibernate tries to load the old assembly.
I know that this is default .NET behaviour (and the reason why assembly redirects exist) but it would still be good to get a new Fluent NHibernate version to support it out of the box. This would also have the advantage that new users of Fluent NHibernate automatically get the latest NHibernate version installed from NuGet.

May be that's NuGet behaviour?
Because it's just not practical shipping Fluenth every time NH publishes
it's new version.
2017-01-18 11:09 GMT+03:00 cremor<EMAIL_ADDRESS>
I can confirm that an assembly redirect is required if NHibernate 4.1 is
used with Fluent NHibernate 2.0.3. Otherwise Fluent NHibernate tries to
load the old assembly.
I know that this is default .NET behaviour (and the reason why assembly
redirects exist) but it would still be good to get a new Fluent NHibernate
version to support it out of the box. This would also have the advantage
that new users of Fluent NHibernate automatically get the latest NHibernate
version installed from NuGet.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-273409906,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfFZrDG-i76cHbSFB4C-CrYsV1-Vrks5rTcjRgaJpZM4LW02A
.

--
Yours faithfully,
Gleb

I don't use NuGet in the project that I used for testing this. As far as I know NuGet helps in this situation because it can add the assemlby redirects automatically to the app.config file when a package is updated. But I don't know if this is on by default.
If you want to avoid that your users need an assembly redirect, you don't have to update Fluent NHibernate for each NHibernate release. Only for the releases that change the assembly version, which is only done by NHibernate on Major and Minor versions, not bugfix releases (e.g. 4.2 or 5.0 would need a new release, but not 4.1.1).

Nuget itself is a console app that downloads files from the internet. I
believe redirect is added by Visual Studio.
So if they release 4.1.1 Nuget next month, assembly version will be still
<IP_ADDRESS>?
2017-01-19 12:58 GMT+03:00 cremor<EMAIL_ADDRESS>
I don't use NuGet in the project that I used for testing this. As far as I
know NuGet helps in this situation because it can add the assemlby
redirects automatically to the app.config file when a package is updated.
But I don't know if this is on by default.
If you want to avoid that your users need an assembly redirect, you don't
have to update Fluent NHibernate for each NHibernate release. Only for the
releases that change the assembly version, which is only done by NHibernate
on Major and Minor versions, not bugfix releases (e.g. 4.2 or 5.0 would
need a new release, but not 4.1.1).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-273729862,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfBf-2rm2mU9gRh_cb_4BBUi0z-f3ks5rTzPIgaJpZM4LW02A
.

--
Yours faithfully,
Gleb

Nuget itself is a console app that downloads files from the internet. I believe redirect is added by Visual Studio.

NuGet has a feature that allows it to change your app.config when a package is installed. See here for some documentation. According to this link, this feature is enabled by default since NuGet 1.2
I just tested it and it indeed works fine. Installing the Fluent NHibernate package first and then updating the NHibernate package automatically adds the required binding redirects to the app.config file.
So a new version of Fluent NHibernate isn't strictly required if you expect all your users to use NuGet or know about binding redirects.

So if they release 4.1.1 Nuget next month, assembly version will be still <IP_ADDRESS>?

Yes. This was first introduced in 3.3.2. I can't find the original discussion right now, but you can read the comment here and the discussion here

See the nuget doc:

Transitive restore applies four main rules to resolve dependencies: lowest applicable version, floating versions, nearest-wins, and cousin dependencies.

https://docs.microsoft.com/en-us/nuget/consume-packages/dependency-resolution

If version specified is "1.0" (interpreted as lower bound, no upper bound>= 1.0), 1.0 is downloaded if possible (the lowest)
If version specified is "1.*" (floating version, any 1.X acceptable) the newest (1.999.999.999) is downloaded.

Thus, changing the reference to Fluent-NHibernate to "4.*" would change the behavior of

installing FluentNHibernate to a project with no previous NH reference installs

to always install the newest available version.

Hi, just NHibernate 5.0 is out. Is there a risk to use the latest version?

@MKiCD I'd like to tell you, but I haven't tested it yet. Will try to run
all our unit tests for 5.0 on the weekend.
On Fri, Oct 27, 2017, 17:40 MKiCD<EMAIL_ADDRESS>wrote:

Hi, just NHibernate 5.0 is out. Is there a risk to use the latest version?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-339990894,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfLc8QWX_k9zsvNlj85ObzRNbLW3Vks5swetogaJpZM4LW02A
.

Great, I'm just starting a new project and have to decide using NH or EF and I suppose the Fluent part makes the life a lot easier. Thank you.

@MKiCD the short answer - current Fluent doesn't even compile with NH 5.0.
Will look closer next week

2017-10-27 17:46 GMT+03:00 MKiCD<EMAIL_ADDRESS>
Great, I'm just starting a new project and have to decide using NH or EF
and I suppose the Fluent part makes the life a lot easier. Thank you.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-339992535,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfOOx-1HCMPZwTqDjsufGQrf2dp4Nks5swezCgaJpZM4LW02A
.

--
Yours faithfully,
Gleb

@chester89  thank you for the info. I'll follow you here and do my my tests with NH 4.0. Is this issue worth a new ticket (because this one is closed)?

@MKiCD you mean that it doesn't work with NH 5? Sure, please file a ticket
On Fri, Nov 3, 2017, 10:53 MKiCD<EMAIL_ADDRESS>wrote:

@chester89 https://github.com/chester89 thank you for the info. I'll
follow you here and do my my tests with NH 4.0. Is this issue worth a new
ticket (because this one is closed)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-341639490,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfDdSug6A3zo3WaRPNalWzYRqaLTSks5sysZpgaJpZM4LW02A
.

@chester89 I added #383 for this question. Thank you for your activity here.

@MKiCD thanks

2017-11-03 20:18 GMT+03:00 MKiCD<EMAIL_ADDRESS>
@chester89 https://github.com/chester89 I added #383
https://github.com/jagregory/fluent-nhibernate/issues/383 for this
question. Thank you for your activity here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-341770116,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfPkyke4OAjtdTGBo9akcJYo0Wgblks5sy0rrgaJpZM4LW02A
.

--
Yours faithfully,
Gleb

@MKiCD good news - I updated Fluent to support NHibernate 5.0. All tests
are passing now. Time permitting, I can cut a new NuGet version on Thursday.

2017-11-07 1:04 GMT+03:00 Gleb Chermennov<EMAIL_ADDRESS>
@MKiCD thanks

2017-11-03 20:18 GMT+03:00 MKiCD<EMAIL_ADDRESS>
@chester89 https://github.com/chester89 I added #383
https://github.com/jagregory/fluent-nhibernate/issues/383 for this
question. Thank you for your activity here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-341770116,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfPkyke4OAjtdTGBo9akcJYo0Wgblks5sy0rrgaJpZM4LW02A
.

--
Yours faithfully,
Gleb

--
Yours faithfully,
Gleb

Waiting for this! Thanks for your amazing work @chester89

Hey @chester89... Some news about the new version?

I got a lot on my plate atm, will try to find some time at the end of next
week

2017-11-21 14:13 GMT+03:00 Jorge Rodríguez Galán<EMAIL_ADDRESS>
Hey @chester89 https://github.com/chester89... Some news about the new
version?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/365#issuecomment-345995140,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfDLhEyVqiutvzwg73Ldq-Mwj8b7bks5s4rBIgaJpZM4LW02A
.

--
Yours faithfully,
Gleb

OK! @chester89. Waiting for this! Thanks!

Any news on the new version?


Consider making icons accessible without the mandatory name/URL etc.

I was looking for a package that provides icons for popular platforms, but the https://typst.app/universe/package/fontawesome only support black and white versions, while this package requires a name. It probably has a different usage in mind, but from the package name it feels like it should also give the full control to the user and allow using the icons separately from the link generator function.
As a workaround, I had to grab the package directory and copy the svg subdirectory to my project. Then I added a few variables for the needed icons and after imported them locally.
Here is my use case:
#let logo(file) = box(image(file, height: 0.7em))
#let discord = logo("./svg/Discord_icon_clyde_blurple_RGB.svg")
#let instagram = logo("./svg/Instagram_logo_2022.svg")
#let linkedin = logo("./svg/LinkedIn_icon.svg")
#let mastodon = logo("./svg/Mastodon_logo_purple.svg")
#let twitter = logo("./svg/Logo_of_Twitter.svg")
#let x = logo("./svg/X_logo_2023.svg")
#let youtube = logo("./svg/YouTube_full-color_icon_(2017).svg")

My local copy includes ./icons/svg/*.svg and ./icons/icons.typ. Then I can import all the variables directly:
#import "./icons/icons.typ": *
#mastodon

Or make an icon module:
#import "./icons/icons.typ" as icon
#icon.mastodon

BTW, I used the Mastodon logo from https://joinmastodon.org/logos/logo-purple.svg (https://joinmastodon.org/branding).

I was looking for a package that provides icons for popular platforms, but the https://typst.app/universe/package/fontawesome only support black and white versions, while this package requires a name. It probably has a different usage in mind, but from the package name it feels like it should also give the full control to the user and allow using the icons separately from the link generator function.

To be honest I really didn't had the use-case of using the icons separately in mind when I created this package but I think it's a legitimate suggestion and shouldn't be that complicated to implement. :smiley:


Update Composer dependencies (2024-02-12-00-16)

Visual regression test passed!
View the visual regression test report


Add github workflow

Hi maintainer, would you be open to setup a github workflow including steps to run tests defined here?
Cheers,
David L.

Thanks David!
Please check commit below, workflows added and test cases fixed.
https://github.com/rebootshen/file-management/commit/451550b6d7822bf7c32eec47f4f7a958ac540b6f


Update scope section

With content from slides from last P4 Workshop.
Fixes #23

@chrispsommers somehow there is always one typo that gets through the cracks
PTAL and approve so I can merge
New PR incoming in a few minutes to set the release version


Inline often called and almost-trivial functions

The intent of most of these functions is akin to C macros. Inlining them honors that intent.

Coverage decreased (-0.6%) to 82.207% when pulling 0af9deb6cb11c9e68016e382d564daaec35a336a on inline-always into a6aedef7650a8d12b07358aa5249294e1b8182a0 on master.


CreateTransaction fails while serializing Digital Signature

Description
Digital Signatures fail to serialize when attempting to push any transaction that contains digital signature in one of the action arguments.
Exception Information
System.Exception: checksum doesn't match.
  at EosSharp.Core.Helpers.CryptoHelper.StringToKey (System.String key, System.Int32 size, System.String keyType) [0x0009b] in <1aafbe1be8d2427d8f9e860b61f6dcf3>:0
  at EosSharp.Core.Helpers.CryptoHelper.SignStringToBytes (System.String sign) [0x00014] in <1aafbe1be8d2427d8f9e860b61f6dcf3>:0
  at EosSharp.Core.Providers.AbiSerializationProvider.WriteSignature (System.IO.MemoryStream ms, System.Object value) [0x00007] in <1aafbe1be8d2427d8f9e860b61f6dcf3>:0
  at EosSharp.Core.Providers.AbiSerializationProvider.WriteAbiType (System.IO.MemoryStream ms, System.Object value, System.String type, EosSharp.Core.Api.v1.Abi abi) [0x000fc] in <1aafbe1be8d2427d8f9e860b61f6dcf3>:0
  at EosSharp.Core.Providers.AbiSerializationProvider.WriteAbiStruct (System.IO.MemoryStream ms, System.Object value, EosSharp.Core.Api.v1.AbiStruct abiStruct, EosSharp.Core.Api.v1.Abi abi) [0x000b7] in <1aafbe1be8d2427d8f9e860b61f6dcf3>:0
  at EosSharp.Core.Providers.AbiSerializationProvider.SerializeActionData (EosSharp.Core.Api.v1.Action action, EosSharp.Core.Api.v1.Abi abi) [0x00089] in <1aafbe1be8d2427d8f9e860b61f6dcf3>:0
  at EosSharp.Core.Providers.AbiSerializationProvider.WriteAction (System.IO.MemoryStream ms, EosSharp.Core.Api.v1.Action action, EosSharp.Core.Api.v1.Abi abi) [0x00064] in <1aafbe1be8d2427d8f9e860b61f6dcf3>:0
  at EosSharp.Core.Providers.AbiSerializationProvider+<SerializePackedTransaction>d__15.MoveNext () [0x001c2] in <1aafbe1be8d2427d8f9e860b61f6dcf3>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <d7ac571ca2d04b2f981d0d886fa067cf>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0003e] in <d7ac571ca2d04b2f981d0d886fa067cf>:0
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x00028] in <d7ac571ca2d04b2f981d0d886fa067cf>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x00008] in <d7ac571ca2d04b2f981d0d886fa067cf>:0
  at System.Runtime.CompilerServices.TaskAwaiter`1[TResult].GetResult () [0x00000] in <d7ac571ca2d04b2f981d0d886fa067cf>:0
  at EosSharp.Core.EosBase+<CreateTransaction>d__31.MoveNext () [0x002cf] in <1aafbe1be8d2427d8f9e860b61f6dcf3>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <d7ac571ca2d04b2f981d0d886fa067cf>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0003e] in <d7ac571ca2d04b2f981d0d886fa067cf>:0
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x00028] in <d7ac571ca2d04b2f981d0d886fa067cf>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x00008] in <d7ac571ca2d04b2f981d0d886fa067cf>:0
  at System.Runtime.CompilerServices.TaskAwaiter`1[TResult].GetResult () [0x00000] in <d7ac571ca2d04b2f981d0d886fa067cf>:0
  at TestScatterScript+<PushTransaction>d__5.MoveNext () [0x00320] in /Users/hotmdev4/Desktop/drakos/scatter-sharp/Unity3D/ScatterSharpUnity3D/Assets/Scenes/Scripts/TestScatterScript.cs:101
UnityEngine.MonoBehaviour:print(Object)
<PushTransaction>d__5:MoveNext() (at Assets/Scenes/Scripts/TestScatterScript.cs:128)
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1:SetException(Exception)
EosSharp.Core.<CreateTransaction>d__31:MoveNext()
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1:SetException(Exception)
EosSharp.Core.Providers.<SerializePackedTransaction>d__15:MoveNext()
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1:SetResult(Abi)
EosSharp.Core.Providers.<GetAbi>d__22:MoveNext()
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1:SetResult(GetRawAbiResponse)
EosSharp.Core.Api.v1.<GetRawAbi>d__14:MoveNext()
System.Runtime.CompilerServices.AsyncTaskMethodBuilder`1:SetResult(GetRawAbiResponse)
EosSharp.Unity3D.<PostJsonWithCacheAsync>d__8`1:MoveNext()
System.Runtime.CompilerServices.MoveNextRunner:Run()
EosSharp.Unity3D.UnityWebRequestAwaiter:OnRequestCompleted(AsyncOperation)
UnityEngine.AsyncOperation:InvokeCompletionEvent()

To Reproduce

Setup a contract with a single action, where one of the parameters is signature type.
ACTION test(signature sig);
Write code using scatter-sharp (or eos-sharp) that creates and pushes a transaction that contains the above action.
The CreateTransaction function should fail with the above exception regardless of the value of the sig parameter.

Analysis
The exception is being thrown in CryptoHelper.cs#L178. I took the code from CryptoHelper and created a small script that runs the same code and generated the below output.  I appears that not only do the checksums not match, but they aren't even the same length.
https://gist.github.com/ARoomWithABue/93cf3b365cd9399795a79a09080eaed4
output:
sig: SIG_K1_KZoEShDrNxiAQq8rYafahdudAESBAfHQxU7ihavonMDMND4jNSHhk9q4UVbs7tTLK6RidFmFmSruipEM1chyxFgN46meSF
digest.Length: 20
digest: [ 38, 189, 167, 157, 74, 198, 230, 153, 7, 108, 205, 141, 173, 55, 95, 93, 55, 145, 150, 148 ]
keyBytes.Length: 69
keyBytes: [ 32, 38, 154, 38, 22, 40, 38, 160, 196, 66, 4, 145, 119, 1, 107, 17, 153, 30, 216, 157, 113, 141, 147, 102, 88, 34, 143, 211, 255, 111, 135, 1, 66, 84, 64, 216, 43, 97, 182, 191, 13, 140, 160, 33, 64, 219, 35, 209, 74, 254, 105, 206, 62, 187, 171, 125, 215, 92, 244, 189, 36, 170, 32, 27, 147, 54, 219, 221, 188 ]
keyBytes checksum: [ 147, 54, 219, 221, 188 ]
digest checksum: [ 38, 189, 167, 157 ]

Thanks for the bug report this one was tricky to find. Fixed here: https://github.com/GetScatter/eos-sharp/commit/9ca6f4b7b30898210aa8a6e456cfd0642f9b464a

Excellent! It looks like fixed it. I discovered this error while using Scatter-Sharp. Will the scatter-sharp.unitypackage be updated with this fix? I can build Eos-Sharp for now and replace the dll for now I think.

The package is updated


Swagger

Codecov Report

Merging #137 (3116f3e) into restx (acaf3e8) will increase coverage by 10.48%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##            restx     #137       +/-   ##
===========================================
+ Coverage   84.21%   94.69%   +10.48%
===========================================
  Files           5        5
  Lines         380      377        -3
===========================================
+ Hits          320      357       +37
+ Misses         60       20       -40

Impacted Files
Coverage Δ

service/__init__.py
88.00% <ø> (+3.38%)
:arrow_up:

service/routes.py
94.70% <ø> (+22.19%)
:arrow_up:

service/utils/error_handlers.py
78.37% <0.00%> (+2.06%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Annotation box does not appear in online demo on first visit

When I view the online demo for the first time, I don't see any annotation boxes. Here's what I see in Edge and Chrome when I visit the demo link:

However, when I refresh the page, the annotation box does appear.
Not sure if it's an issue on my end.

Thanks for reporting - I can reproduce it on my end, too. Doesn't happen consistently for me, but does happen. Looks like it's some sort of timing issue where the page & JavaScript is loaded already (and Annotorious starts to init). But the image isn't fully loaded yet.
I'll investigate!

Update (and mostly a note to self): looked into this and the SVG elements are rendered into the DOM properly, but the SVG viewBox element is set to a value of 0 0 0 0. The value is set here, which shows that the image naturalWidth and naturalHeight, at that point in time are still zero.
Q: how to prevent this best. Defer creation of the layer until the image has been confirmed loading? Or handle inside the layer?


Test explorer should use 'dotnet build' instead of 'msbuild /t:Build' to build the test project for test discovery

Describe the bug
When working on the dotnet/fsharp repository in a dev container, the test explorer test discovery fails with a build error.
Reason is

The test explorer uses msbuild /t:Build
To build the compiler, you must use their build scripts, or use dotnet build, or have BUILDING_USING_DOTNET set.
(This is true once https://github.com/dotnet/fsharp/pull/16335 is merged.)

Steps to reproduce
Open the dotnet/fsharp directory in VS Code Dev Containers.
Wait for the (misleading) error message Couldn't build test projects. Make sure you can build projects with 'dotnet build'.
Machine info

OS: Windows 11 with WSL2/docker and VS Code Dev Containers
.NET SDK version: 8.0.100-rc.1.23463.5 in dev container
Ionide version: 7.16.1

Additional context
As dotnet test is used for discovery and running of tests, I assume dotnet build is ok for building. Or is there a specific reason to use 'msbuild' directly? It would be nice to have test discovery work out of the box when opening dotnet/fsharp in a dev container.
In any case, the error message should be changed to say ... Make sure you can build projects with 'msbuild /t:Build'.

cc @farlee2121

The test explorer uses msbuild because

msbuild returns a signal, so we don't have to parse to tell if the build failed. Dotnet build returns command success even if the build fails.
consistency with other Ionide commands, which use msbuild

As for the error message, the vast majority of the time dotnet build is going to boil down to the same result as msbuild
but is much more likely to be in the user's path and thus easy for them to double check.
I don't know for sure, but I'd guess dotnet build is a light wrapper around msbuild.
Considering tests were discovered for the fsharp repo in #1946, there might be something else going on here.

dotnet build returns command success even if the build fails

This is surprising to me - we're getting the exit code from MSBuild here and should be flowing that up. Can you log an issue to dotnet/sdk with a repro?

Maybe I need to revisit this and look more closely at the exit code, but I remember stumbling into this during my work.
My notes say build failure didn't result in an error code and I linked to this existing issue
https://github.com/dotnet/sdk/issues/8481#issuecomment-314175413


WAA-TRD pre-built link is wrong

In WAA-TRD detection preprocess Pre-built flow chapter 2.2, the link of "bin.tar.gz" is wrong.
Can someone provide the correct link to help me build the detection preprocess flow successfully?

Fixed with https://www.xilinx.com/bin/public/openDownload?filename=Vitis-AI.1.3-WAA-TRD.bin.tar.gz

Fixed with https://www.xilinx.com/bin/public/openDownload?filename=Vitis-AI.1.3-WAA-TRD.bin.tar.gz


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 10:54:29 on 2023.07.12
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 1, errors out: 0
Original problem ID: 32128
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=32128

Problem has been resolved in 9m 0s at 11:03:29 on 2023.07.12
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 32128


My Subscriptions page not displaying text.

Starting from around two days back, Youtube Center stopped formatting the My Subscriptions page properly.
Details:
Youtube Center v2.1.7
Safari 7.1.3
Please see attached screenshot and let me know what else I need to provide!

This has been reported before, please update to the latest dev version which can be downloaded here: https://github.com/YePpHa/YouTubeCenter/wiki/Developer-Version

Oh, hey thanks!


Add wiki page docs

Please merge this so that the people will have instant look at the docs.

Thanks for the PR and the effort in editing the WIki, but this adds an additional place to be manually kept up-to-date when there's a version upgrade. e.g. if I update the docs in the repo but you forgot to update the Wiki, people will get confused. Ideally we want everything to be updated in one single place.

I will be dedicated to update the wiki. Please merge this. And if possible make me one of the docs contributor. I loved your framework.

I appreciate your effort, but keeping docs in one place is just simpler and easier. I don't see why we need to replicate the docs in the wiki. Feel free to contribute to the docs by sending in pull requests though.

OK. All right


error while compiling the substrate node

I got this error after trying to install the substrate-contract-node :
1) I download  rustc
rustc --version
rustc 1.75.0 (82e1608df 2023-12-21)

2) I cloned the repo
I followed the instruction to install substrate-contract-node here
git clone https://github.com/substrate-developer-hub/substrate-node-template

3) then I tried to compile the repo by this commands
git switch -c my-wip-branch
cargo build --release

and then I got this long error
failed to run custom build command for `sc-network-light v0.10.0-dev (https://github.com/paritytech/substrate.git?branch=polkadot-v1.0.0#948fbd2f)`

Caused by:
  process didn't exit successfully: `/home/mohamed/substrate-node-template/target/release/build/sc-network-light-0fea2d72bbd5faeb/build-script-build` (exit status: 101)
  --- stderr
  thread 'main' panicked at /home/mohamed/.cargo/registry/src/index.crates.io-6f17d22bba15001f/prost-build-0.11.9/src/lib.rs:1457:10:
  Could not find `protoc` installation and this build crate cannot proceed without
      this knowledge. If `protoc` is installed and this crate had trouble finding
      it, you can set the `PROTOC` environment variable with the specific path to your
      installed `protoc` binary.If you're on debian, try `apt-get install protobuf-compiler` or download it from https://github.com/protocolbuffers/protobuf/releases

  For more information: https://docs.rs/prost-build/#sourcing-protoc

  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...

so how can I fix this issue ?

I followed the instruction to install substrate-contract-node here

You are linking to a different page that is not referencing substrate-contracts-node and your description also refers to something else. Are you sure you want to install the node from this repository?
It seems you're rather looking for https://github.com/substrate-developer-hub/substrate-node-template?

I am sorry I got confused between the two repositories


Jupyter Extension for VS Code does not show tables created by fastai learners

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
This Jupyter Notebook is expected to show tables with epoch, train_loss, valid_loss, accuraccy, time etc. columns when learn.fit_one_cycle() and / or learn.fine_tune() is called, as seen here:

However these tables are not shown when the same Jupyter Notebook is run in Jupyter Extension for VS Code:

VS Code Version
1.78.2
Jupyter Extension Version
v2023.4.1011241018
Coding Language and Runtime Version
Python 3.11.3
Language Extension Version (if applicable)
v2023.8.0
Running Jupyter locally or remotely?
Local

me too

can you provide a sample notebook to repro this scenario.

this seems to work when I open that notebook, unless the issue was from running the cell. In that case, could you post what the output of that cell is when it's not showing up? and if there is a plain text option if you select "change presentation" from the output menu?


fix: onboarding fixes

The type of this PR is: Bugfix

This PR resolves:

Palette v3 issues with onboarding
Onboarding copy

Description

Add missing copy that we agreed on here
https://www.notion.so/artsy/Onboarding-Socail-auth-error-messages-9aeb58a81f8042a185d55effc27cb461

Fix Following UI Issues:
Before

After

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Palette v3 onboarding issues - mounir
Onboarding copy error message - mounir

iOS user-facing changes

Android user-facing changes

Dev changes

This PR contains the following changes:

Cross-platform user-facing changes (Palette v3 onboarding issues - mounir,Onboarding copy error message - mounir)

  Generated by :no_entry_sign: dangerJS against ea4095fe33f5f5eb597797caa675dbc217ee781f

Ready to be merged after fixing the tests (I think it's just the copy that needs to be changed)

#squashongreen


Bug - Accounts with dot shouldn't break routing

Accounts (outlets) with a dot in the account name (/twitter/cancer.gov) shouldn't break routing.
Current behavior:

Add an account with a dot in the account name (e.g. http://twitter.com/cancer.gov)
Visit the account's canonical URL (/accounts/twitter/cancer.gov)
The account isn't found because the route strips .gov as an output format.

Desired behavior:

...
...
The route for /accounts/[:service_id]/[:account] should only strip off .json, .html, and .xml formats.
@huboard:{"order":13.0}

Verified on the live prototype. Closed.


progress reporter clobbers logs

When running tests from https://github.com/angular/angular.js using WebStorm 7 integration, there are the following lines in "Karma Server" tab:
INFO [karma]: Karma v0.10.2 server started at http://localhost:9876/
INFO [launcher]: Starting browser Chrome
INFO [Chrome 28.0.1500 (Linux)]: Connected on socket 0BuEXA7DrI_qhdCkmRPz
WARN [web-server]: 404: /%7B%7BtestUrl%7D%7D
WARN [web-server]: 404: /%7B%7BdataUri%7D%7D
WARN [web-server]: 404: /foo/bar

Running the same tests in the console results in:
INFO [karma]: Karma v0.10.2 server started at http://localhost:9876/
INFO [launcher]: Starting browser Chrome
INFO [Chrome 28.0.1500 (Linux)]: Connected on socket PLMLEAGJ_TBTBbyMmvOO
Chrome 28.0.1500 (Linux): Executed 692 of 2241 SUCCESS (0 secs / 1.432 secs)
Chrome 28.0.1500 (Linux): Executed 695 of 2241 SUCCESS (0 secs / 1.438 secs)
Chrome 28.0.1500 (Linux): Executed 699 of 2241 SUCCESS (0 secs / 1.448 secs)

Diff: no "WARN" messages in the console.
Initially, I thought it's something wrong with WebStrom 7 integration, but it turned out that log.warn('404: ' + path) is executed in both cases (this line is in karma/lib/middleware/common.js).
Nevertheless log.warns are called, warning messages aren't shown in the console.

Closing in favor of #1411


Added support for 2.X release

Added version 2.X support. Ubuntu 14.04 only. Example params:
class {'grafana':
install_method => 'package',
symlink => false,
version => 'latest',
manage_repository => true,
}

Hi @zdenekjanda thank you for submitting this pull request.
I would like to point out, however, that effort has already been undertaken to provide support for Grafana 2.x in the develop branch of this repository. It currently supports docker, package (deb/rpm) and archive (tarball) installation methods, along with the ability to manage Grafana's configuration by supplying a hash of custom parameters.
Please test the module to see if it suits your needs, and I welcome your feedback. I have been waiting for the official release of Grafana 2.0 before merging this branch into master and publishing a new version of the module on the Puppet Forge. I don't intend for the module to be backwards compatible since Grafana 2.x isn't backwards compatible with 1.x either.
Thanks.

Closing this in favor of #27


ruby 2.2.0 fails to build on Ubuntu

I haven't been able to compile Ruby 2.2.0 using ruby-build as a plugin to rbenv.
This is using ruby-build from master, commit 45b75ed on Ubuntu 14.04.1 LTS
The error appears to be: relocation R_X86_64_32S against `.rodata' can not be used when making a shared object; recompile with -fPIC ./libffi-3.2.1/.libs/libffi.a: error adding symbols: Bad value
Here is the log file: https://gist.github.com/ivar/b595ca0dd03314c96b7a
(edit: I'm running ubuntu 14.04.1 in a virtual machine and originally posted this issue as having occured in my host machine as opposed to VM)
> gcc --version
gcc (Ubuntu 4.8.2-19ubuntu1) 4.8.2

It might help: http://stackoverflow.com/questions/27915581/error-with-rbenv-install-2-2-0


JSON Error

I'm receiving this error when I try to use the workflow:
[2016-08-23 17:32:18][ERROR: input.scriptfilter] JSON error: JSON text did not start with array or object and option to allow fragments not set. in JSON:
Warning: usort() expects parameter 1 to be array, null given in Command line code on line 16

Warning: Invalid argument supplied for foreach() in Command line code on line 18
<?xml version="1.0"?>
<items><item uid="atom57bcdd024fd410.97394851" arg="" valid="yes"><arg></arg><title>No projects found matching ''</title><subtitle></subtitle><icon type="fileicon">/Applications/Atom.app</icon></item></items>

When I manually run csonc on the projects.cson file, it returns it in this format:
{
  "ProjectX": {
    "title": "Project X",
    "paths": [
      "/Users/User/path/to/project"
    ]
  },
  "ProjectY": {
    "title": "Project Y",
    "paths": [
      "/Users/User/path/to/project"
      "/Users/User/another/path"
    ]
  },
}

So, this seems odd to my non-programmer mind. I've done some poking and prodding and I think the issue is with this line of the script filter:
$json = json_decode($json, true);

Before that line, the value of $json is a valid string of JSON. However, after that line, running var_dump($json) returns NULL. It all looks like valid PHP to me, so I must admit that I'm not sure what is happening.

The problem lies in it not being valid JSON. Currently there is no error checking using json_error(), I guess that needs to be implemented and shown to the user with a notification.
Your JSON should look like this to be valid:
{
  "ProjectX": {
    "title": "Project X",
    "paths": [
      "/Users/User/path/to/project"
    ]
  },
  "ProjectY": {
    "title": "Project Y",
    "paths": [
      "/Users/User/path/to/project",
      "/Users/User/another/path"
    ]
  }
}

As the project's list JSON is converted from Atom's native format CSON, can you post the CSON file that is used (usually: ~/.atomprojects.cson)? That way we can correct the CSON so it should output correct JSON as well. That would fix your problem I think.

Thanks! After some more prodding, it seems like I'm getting a Unexpected control character found error on the decode.
Here's my full CSON file (uncensored, so I'll pull it offline soon): http://files.grovedesign.co/projects/projects.cson

Strange, I get a valid JSON file when I convert that using season (tool used to convert CSON to JSON).
If that is truly your JSON output then something is going terribly wrong in the conversion process of CSON to JSON.
Offtopic: A switch from PHP to NodeJS is planned some day. This would simplify the conversion of CSON to JSON and possibly speed up things a bit as well. Yet currently I have no time to do so unfortunately.

Welp, there is all sorts of goofiness going on.
I think I figure out what one of the problems is. I created a new projects.cson and only added on project to it for simplicity.  But it is. I was able to remove all of the garbage leading up to the { (and I also had to remove one whitespace character from the end) and it suddenly got over that error.
Now, the problem is that I'm getting another error:
[ERROR: input.scriptfilter] Unable to decode JSON results, "items" array property missing in JSON:
{
  "grovedesign": {
    "title": "Grove Design",
    "paths": [
      "/Users/Caleb/Design Tools/Websites/Grove Design"
    ]
  }
}

Woa, that is messed up. What version of npm package "season" are you using? I'd recommend taking it up with them at: https://github.com/atom/season/issues


Added complete functionality for external links

This PR adds the complete functionality to let Quick-Tray Links be opened in a new window.

Each link gets a toggle switch to open the link in a new window (defaults to true)
Added translations for new functionality in en  (ru and uk are not complete)
Added complete german translation
Works without changes to grav admin
Updated defaults
Updated readme
Updated version
Updated changelog

@rhukster This is my first PR, so I don't know how this normally work.
Will this be reviewed when there is spare time, or do I need to do something special for it to be reviewed?

Looking good! thanks!


Add support for btrfs filesystems

Adds an fsuse expression to label btrfs as supporting SELinux.
Had some troubles on a F24 vm with a BTRFS filesystem, output from what I was seeing:
/sbin/restorecon set context /usr/bin->sys.id:sys.role:cmd.cmd_file:s0 failed:'Operation not supported'
/sbin/restorecon set context /usr/sbin->sys.id:sys.role:cmd.cmd_file:s0 failed:'Operation not supported'
/sbin/restorecon set context /usr/libexec->sys.id:sys.role:cmd.cmd_file:s0 failed:'Operation not supported'
/sbin/restorecon set context /etc->sys.id:sys.role:config.config_file:s0 failed:'Operation not supported'
/sbin/restorecon set context /usr/lib/systemd/system->sys.id:sys.role:sd_unit.unit_file:s0 failed:'Operation not supported'
/sbin/restorecon set context /var/lib->sys.id:sys.role:var_lib.var_lib_file:s0 failed:'Operation not supported'
/sbin/restorecon set context /var/cache->sys.id:sys.role:var_cache.var_cache_file:s0 failed:'Operation not supported'

Thanks


feat: send user-agent header with auth token requests

This commit updates our various request-based authenticators so that the User-Agent header is included with each outbound token request. The value of the User-Agent header will be of the form ibm-python-sdk-core/<authenticator-type>-<core-version> <os-info>.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

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


Fix -resume with fai usage

Description of the bug
Using the fai element in the ch_fasta channel hinder the resume of the pipeline.
When used the cached element aren't used and the pipeline recompute the processes and the succeeding ones.
Possibilities:

Find why
Remove the fai as it is not directly used (not even sure it really is used in the different processes)

Example: in the test profile the pipeline will not resume the variant calling at the preprocessing of glimpse 1
Command used and terminal output
nextflow run nf-core/phaseimpute -profile singularity,test --outdir results -resume
Relevant files
No response
System information
No response

Linked to https://github.com/nextflow-io/nextflow/issues/5439


Unable to resolve service for type Microsoft.AspNetCore.Identity.IUserStore

Thanks for the awesome lib. I'm trying it out for a new project in .Net core 2.0 and run into the below error when adding the ".AddLinqToDBStores(new DefaultConnectionFactory())" statement.
I've pulled the example from the repo and it gives the same error message. Removing the line makes it build.
PS: I've removed the source and added the Nuget package to re-produce the error.

Sorry for so late response... If you do remember can you provide more details, please


Share functionality not working

When we click on share button, images are getting shared only on whatsApp not on telegram, discord or gmail.
Debug the app and try to fix the error.

I want to work on this issue under JWOC


One shot auth

Hi I'm making an app which requires non authorised users to authenticate using a social account (facebook, flickr, twitter etc.) to join an 'event'. This authorisation is then used to access parts of the user's account (wall feed etc) on their behalf (in the background).
Is there a view that asks a non-authorised user (one without credentials to the main site) to authorise to the site via whatever social account type that is required (some events might need facebook, other's twitter) and then dump them back to a page saying 'thanks' and thats the end of the workflow. Therefore not logging them in as such, just creating the User I can attach to the event with their newly associated social authentication tokens etc.
I've thought: once the user is created and returned, could I just make the User 'non-active' to stop login?
Is there a view for each of the social auth types that would allow me to do this?

You can override the complete view and with that determine the login behavior by setting your custom login function.


ARTEMIS-604 - Message Serialization Improvement

I can see the 2 failure in my local env (individual test run will pass, they only show up in batch tests).
I'm investigating now.

The tests individually will pass because you would be using Core over Stomp.
The Stomp over AMQP test will need to use the factories. You would need to change some properties to be able to run it individually. Talk to me offline if you want to do it.. maybe we can add something to the javadocs on how to do it.

(i had to edit my last comment, I confused Stomp with JORAM :P )... Adding this note here in case you are reading the updates through email.

Closing it as we'll have a new one.


make express install more clear

removed foolish 'unused' variables and redirect to /dev/null instead
add startExpressEdgeRouter function
don't emit systemd files by default since those won't work on mac/wsl
reference start functions at the end to show someone how to start/stop ziti locally

output should end up looking like:

nice!


Array index out of bounds in https://github.com/IBM/differential-privacy-library/blob/main/diffprivlib/tools/quantiles.py#L134

#48
However, in that case, len(probabilties)=len(array)-1 .
That still means, if rand becomes greater than all self._probabilities in line 169 of https://github.com/IBM/differential-privacy-library/blob/eeae0d776eb9915732b4a30bdc02448ac26cb578/diffprivlib/mechanisms/exponential.py#L169, i.e, when idx = len(self._probabilities) as of line 174 of the same as per given conditions, line 134 of https://github.com/IBM/differential-privacy-library/blob/main/diffprivlib/tools/quantiles.py#L134 should still give an "array index out of bounds error" due to array[idx+1] reference, right, since idx=len(probabilities) and  len(probabilties)=len(array)-1, which means array[idx+1] = array[len(array)] ?

From the following, there are k+1 probabilities (indexed from 0 through k) in the mechanism:
https://github.com/IBM/differential-privacy-library/blob/eeae0d776eb9915732b4a30bdc02448ac26cb578/diffprivlib/tools/quantiles.py#L131
So the outputs from this exponential mechanism will be in the range [0, k], inclusive. This is okay, since array now has k+2 entries, indexed from 0 through k+1, so array[idx+1] will always be valid:
https://github.com/IBM/differential-privacy-library/blob/eeae0d776eb9915732b4a30bdc02448ac26cb578/diffprivlib/tools/quantiles.py#L134

But https://github.com/IBM/differential-privacy-library/blob/eeae0d776eb9915732b4a30bdc02448ac26cb578/diffprivlib/mechanisms/exponential.py#L174 states that idx is k+1 in the worst case, not k, if I'm not wrong?

Ah yes, good spot! We'll get that fixed. That should be idx = len(self._probabilities) - 1.
Thanks!

#50 Sent a PR for the same!


Use an external role to set up trust store

Removes the trust store specific changes made earlier aa0921b .
14cd6d3 is just an example of how to achieve the same result with
https://github.com/hajaalin/ansible-role-truststore.
Here is the configuration for this example:
(https://github.com/hajaalin/omero-deploy-example/blob/master/inventory/group_vars/all/main.yml)
e439ebf  was necessary to make training-server.yml run on CentOS 7 in (Vagrant box 1611.01 for VirtualBox).

Hi @hajaalin. Except for the training playbook, all of these files have migrated (announcement forth-coming) to individual repositories (e.g. https://github.com/openmicroscopy/ansible-role-omero-server/pull/2).
Since we likely won't be configuring the generic training server for a truststore, shall we point to your repo from our documentation to your repo (github? galaxy?) as a potential add-on?

Hi @joshmoore , you could add a link to the GitHub repo https://github.com/hajaalin/ansible-role-truststore. I just added there a link to the OMERO deploy example. I'll update that example to use the new OMERO roles eventually.

@hajaalin : adding via https://github.com/openmicroscopy/infrastructure/pull/246 (work-in-progress) -- feedback on the updated ansible documentation certainly welcome.

We'll likely start advertising the new READMEs soon. Your role is listed in https://github.com/openmicroscopy/infrastructure/blob/master/ansible/README.md#from-the-community (More welcome!) And your fix in https://github.com/openmicroscopy/ansible-role-omero-server/releases/tag/1.2.1
Closing this since all the associated files have now been migrated to other repositories.


Implement suitable date system to parse between R, CSV, Python and MySQL

Hi @sargentfrancesca
Date layout for ColCheckDate - 08082016
Date layout for database date_created (version model) - Aug_12_2016
Will check for any other hidden date columns now
D

This is solved, right?
Dates in R and the CSV appear as dd/mm/yyyy and are converted to mysql's yyyy-mm-dd like this:
datetime.datetime.strptime(data['publication_date_digitization'], "%d/%m/%Y").strftime("%Y-%m-%d") if data['publication_date_digitization'] else None


CSS Selectors for Themes

As you can see above, there are sections that are grey. I can change the colors using developer tools, but I cannot do so with RemNote's built in CSS tool.
Here are the css selectors that I tried to change but which I could not:
.incremental-everything-element {
background: #171414 !important;
}
button.bg-blue-50.hover:bg-blue-70.text-white.font-bold.py-2.px-4.rounded.incremental-everthing-next-button {
background: #201C1C !important;
color: white !important;
}
.flex.flex-row.justify-center.items-center.gap-6.incremental-everything-answer-buttons {
background: #171414 !important;
}
button.bg-blue-50.hover:bg-blue-70.text-white.font-bold.py-2.px-4.rounded.incremental-everthing-done-button {
background: #201C1C !important;
color: white !important;
}

I think you could try doing something like this to style the plugin elements:
you can use selectors like this to style plugins based on their id:
div[data-plugin-id='demo-tabs'] {
  font-size: 14px;
}


Use correct ar for host/target

Previously we would always use the ar of the target; this is incorrect.
Fixes #350.

@bgamari Thanks, looks good! I'll wait for CI bots to finish and will merge.


Current access while creating CRDs in the operator

Has the operators don't wait for leader election when creating the CRDs, they can fail since both will try to perform the same actions:
level=info msg="CRD (CustomResourceDefinition) is installed and up-to-date" name=CiliumNode/v2 subsys=k8s
level=error msg="Unable to update CRD" error="Operation cannot be fulfilled on customresourcedefinitions.apiextensions.k8s.io \"ciliumendpoints.cilium.io\": the object has been modified; please apply your changes to the latest version and try again" name=CiliumEndpoint/v2 subsys=k8s
level=info msg="CRD (CustomResourceDefinition) is installed and up-to-date" name=CiliumExternalWorkload/v2 subsys=k8s
level=info msg="Throttling request took 1.103611747s, request: GET:https://<IP_ADDRESS>:443/apis/apiextensions.k8s.io/v1/customresourcedefinitions/ciliumnetworkpolicies.cilium.io" subsys=klog
level=error msg="Unable to update CRD" error="Operation cannot be fulfilled on customresourcedefinitions.apiextensions.k8s.io \"ciliumclusterwidenetworkpolicies.cilium.io\": the object has been modified; please apply your changes to the latest version and try again" name=CiliumClusterwideNetworkPolicy/v2 subsys=k8s
level=error msg="Unable to update CRD" error="Operation cannot be fulfilled on customresourcedefinitions.apiextensions.k8s.io \"ciliumnetworkpolicies.cilium.io\": the object has been modified; please apply your changes to the latest version and try again" name=CiliumNetworkPolicy/v2 subsys=k8s
level=fatal msg="Unable to register CRDs" error="Unable to create custom resource definition: Operation cannot be fulfilled on customresourcedefinitions.apiextensions.k8s.io \"ciliumendpoints.cilium.io\": the object has been modified; please apply your changes to the latest version and try again" subsys=cilium-operator-generic

It's fine to not wait for leader election when registering the CRDs however the code should be more resilient.

Do you have repro steps?

Do you have repro steps?

@christarazi it simply happens by deploying the operator in a fresh cluster (without any CRD deployed). I don't think it's necessary repro steps as code analysis could easily show this.


Style/mui header

Description
MUI styling and basic tab. functionality.
Fixes # (issue)
Type of change
Styling
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

[ ] Complete, tested, ready to review and merge
[x] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[ ] Test A
[ ] Test B

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

looks good, the logo can wait till later depending on your aesthetics


Integration Test - Release - 2022/01/19 07:08:53

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign PRMergerTest2

#reassign NonAllowedUser

#please-close

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#unassign PRMergerTest1

#unassign NonAllowedUser


Media meta info

Support meta info for media, such as copyright holders, authors, etc.

@silend I'd like some feedback on if there are any important fields missing from the media editor


Pre fails on transactions

@rultor release, tag is 0.4.0

@rultor release, tag is 0.4.0

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.4.0

@yegor256 Done! FYI, the full log is here (took me 9min)


RT#792601 v2.31—Are the comments available on the Xenium QC page coming from the appropriate group?

The xenium QC comments seem to be pooling from all the comments on the config page, rather than the Xenium QC comments, is this a bug or can we change it so it pulls the dropdown list from the xenium QC comments please?

—https://rt.sanger.ac.uk/SelfService/Display.html?id=792601

@KatyTudor This has been updated on stan uat

@khelwood its working on UAT


Added the file for kadane's algorithm

Added the kadane's algorithm in the file.

With reference to #5


[5.9][move-only] Make test 32 bit friendly.

• Description: Just updates a test to make it 32 bit friendly.
• Risk: This is low risk since it only adds a test.
• Original PR: https://github.com/apple/swift/pull/66438
• Reviewed By: @jckarter
• Testing: Just an updated test
• Resolves: rdar://110059596
rdar://110059596
(cherry picked from commit 2488f1966d274ea7d3a995a57000a0273c19a8a0)

@swift-ci test

@swift-ci test windows platform

@swift-ci test windows platform


Long clips are truncated

Original ticket:
https://sourceforge.net/p/djv/bugs/64/

I've verified that this is fixed in the current code base, I was able to successfully playback a 16 hour movie (though I didn't watch the whole thing).


Components introduced through script tags are marked as subcomponents？

Version
3.0.4
Reproduction link
https://github.com/silkwromzf/report
Steps to reproduce

Install and run the project.
The component object introduced through src in the vue file is recognized as a subcomponent.
In vuejs-devtools it is correctly recognized as the current component.

// ...

uni-navgation

3.navgationWIthTsx  is also recognized as data-v-264afbb3-s.
4.Can't  use the scope attribute if  use tsx?
What is expected?
data-v- => data-v-264afbb3
What is actually happening?
data-v- => data-v-264afbb3-s

Please read https://new-issue.vuejs.org/?repo=vuejs/vue#why-repro and open a new issue with a full reproduction: Your repro seems to be missing the actual code to show the problem and should not contain TS, routing, nor stores


Puppeteer library fails with ExecutionContext error when using the TAP CLI

I'm running a test suite which uses the puppeteer library to execute tests against a headless Chrome.
The thing is that when I import 'tap', compile the code and run it with the node cli everything goes fine, but when I try to execute the code using tap --ts src/index.ts, the puppeteer library starts yelling something like:
> tap --ts src/index.ts

 FAIL  src/index.ts
 ✖ Evaluation failed: ReferenceError: cov_rsz3v8w4c is not defined     at __puppeteer_evaluation_script__:1:11

  test: "Submit form correctly"
  stack: >
    ExecutionContext._evaluateInternal
    (node_modules/puppeteer/src/common/ExecutionContext.ts:273:13)

    ExecutionContext.evaluate (node_modules/puppeteer/src/common/ExecutionContext.ts:138:12)
  at:
    line: 273
    column: 13
    file: node_modules/puppeteer/src/common/ExecutionContext.ts
    function: ExecutionContext._evaluateInternal

I could open an issue on the puppeteer repo but I think the issue here is that TAP is doing something magic behind the scenes with the Node execution context that I would like to know,
thanks!

You need to add /* istanbul ignore next */ before any function which will be executed remotely.  I don't know puppeteer but in psuedo-code:
/* istanbul ignore next */
puppeteer.executeScript(() => window.location = '/newpage.html')

The istanbul ignore next prevents code coverage from being added to the anonymous function which gets stringified then executed in the browser.  As an alternative you could use tap --no-cov to disable coverage entirely.


[Suggestion] Sugar for using declare global [interface|var|...] on actual declarations please

This is more of a convenience feature that may or may not be very simple to do on your end. But as I explore working with the global JSX namespace to use it with native custom elements, I keep running into scenarios where there is repeated declare global {} blocks scattered around different modules.
While I have not encountered the use of declare global in other use cases over the past few years, I wonder if others may have repeated use for it.
Instead of using the block format to declare what I believe is mostly non-emitted code that is only used for the type system, I would like to propose using something like declare global interface ... which would be synonymous to declare global { interface .... }.
TypeScript Version:  2.2.0
Code
Here is an example from my current experiment before sugaring:
class E extends Element {
    public propA: string;
}

declare global {
    interface CustomElementAttributes {
        'my-e': Attributes<E>,
    }
}

const myE = <my-e prop-a='abc'></my-e>

Here is the proposed syntax change for the global interface declaration:
declare global interface CustomElementAttributes {
    'my-e': Attributes<E>,
}

Expected behavior:
I am using the global CustomElementAttributes interface to define the attributes for custom tags and this interface is then consumed in the global JSX's IntrinsicElements.
I was inclined to try simplifying the code as shown in the proposed syntax and of course this is not currently valid syntax, but it might be a welcome addition.
Checklist:
Syntactic: I assume there are no syntactic concerns.
Semantic: I assume anything that is not valid inside a declare global block remains invalid when using the short hand form. Also, since it is a declaration, it cannot be in the form of an assignment statement or RHS.
Emit: While this does not affect js emit, it does factor in declarations, which I would argue does not require any change expect in the tokenization process. When a declare global is not followed by a {, then it should be followed by a declaration that starts with interface|var|type|... and at the end of this block, that declaration should be wrapped into a block to produce declare global { [declaration] }.
Compatibility: I don't see any potential conflicts. I would venture that error checking might introduce some hard-to-predict challenges (especially for someone on the outside like myself) and the same with tooling that is hard-bound to the current syntax.
Other: I think performance is not an issue, as long as you guys figure out how this impacts error checking. I do imagine this will be a breaking change from a tooling standpoint for tools that do AST-based operations and error checking (linting... etc).

Why not declare all your augmentations in a single .d.ts file, call it globals.d.ts?

True... one potential way could be globals.d.ts and whenever we define a custom element then this file must be edited. Of course the advantage would be that there is one central location where tags are associated with custom element classes, and essentially that is good since elements must use unique tags.
So this suggestion is really not a solution to my actual problem, but something that felt more natural form a coding standpoint and I was inclined to think that other people might have a similar need, which is to be make a single inline global declaration that does feels more natural like when using private, public... etc.
Surely declare global interface ... as a single global declaration feels more natural than having a declare global {...} block for a single member.
Here is the longer version of the actual situation as an example why I needed to use a single global declaration:
I'm trying to get TypeScript's JSX typing mechanism to work nicely with authoring native Web Components, or at least non-React components. My challenge was to be able to use TSX to define an HTMLTemplate that can include custom elements using tags. So for a custom element to an encapsulated web component module that handles it's own CusomElementRegistry#define, the basic assumption should be that the tag name is the only reference that can be imported into scope by the consuming HTMLTemplate.
This means that we need to augment the IntrinsicElements interface of the global JSX namespace as a side effect of importing the component, and the most preferable mechanism in my opinion will be to automatically augment the IntrinsicElements interface whenever an element is being defined in the CusomElementRegistry.
At this point, it seems that every component will only need to use one global declaration statement with no exports, contamination or side effects (to be v1 compliant) which currently looks like this:
declare type MyCustomElementAttributes = Partial<{[attributes: string]: any}>;

declare global {
    interface CustomElementAttributes {
        'my-custom-element': MyCustomElementAttributes
    }
}

Then I wondered if instead they could have this:
declare type MyCustomElementAttributes = Partial<{[attributes: string]: any}>;

declare global interface CustomElementAttributes {
    'my-custom-element': MyCustomElementAttributes;
}

I am not sure i understand the original reason why this interface is defined in the global scope in the first place. can you elaborate?

Oh, that's a requirement for TypeScript's JSX type system, when I looked it up on various threads it was always global... but you make a good point, because the handbook does not specify global anywhere! Thanks a lot for pointing that out, I think that helps a lot!

Intrinsic elements are looked up on the special interface JSX.IntrinsicElements. By default, if this interface is not specified, then anything goes and intrinsic elements will not be type checked. However, if interface is present, then the name of the intrinsic element is looked up as a property on the JSX.IntrinsicElements interface.

So, this is different from the title of the issue at this point, I now have some more research work ahead of me...
But I will, have to ask, generally speaking, don't u feel that a statement like declare global type ... is much more intuitive then the block having only 1 statement, just like when declaring a member public or private?

Global side-effects of modules is not a great practice in general. TS did not have support for it, intentionally, for a long time. Now that we have declare global (meant to match declare module btw), we can not take that back and break existing users; I do not think we should add yet another syntactic form to express the same semantic meaning (one is too many already if you ask me).

These kinds of lone declarations aren't really common enough to warrant an additional syntax for writing them.


Can I use simpla with this Elm like library?

Hello,
Escher is an elm like library for Julia  that allows the creation of composable reactive web apps without using HTML/CSS, though it can express the full breadth of the latter.
It can wrap arbitrary web elements... would it be possible to use simpla with this?

I should imagine so, as long as you can spit out Simpla's HTML to the DOM somewhere then it should work.
/cc @bedeoverend

Shouldn't be a problem - I haven't played with Escher myself, but from a quick look at the docs, it looks like it uses WebComponents + Polymer itself, so it should work really well with Simpla.
The only thing to be aware of, you should install (Simpla locally)[https://www.simpla.io/docs#hosting-elements-locally], rather than hotlinking from our CDN so as to avoid dependency conflicts.
Let us know how you go with it!

Thanks!!

Will close this, let us know if any problems arise!


Sample Open Horizon Agent pattern should be multi-arch

The example Open Horizon Agent pattern listed in the setup instructions
is amd64.  It would be better if the user could also register the pattern on an ARM based edge device.
Request to make the example pattern a multi-arch container.
The IBM team has used the Docker buildx experimental feature to build multi-arch containers.

I would be happy to take on this task.  I've built multi-arch containers for other projects.
I can't quite figure out where this container is sourced from.

Proposed changes would be to
https://github.com/edgexfoundry-holding/open-horizon-integration/blob/master/hub/04-configure-anax.md
and the config files in
https://github.com/edgexfoundry-holding/open-horizon-integration/tree/master/hub/configs

@haveroot is refactoring the documentation, most likely in a new branch.  @johnwalicki can you ping him and determine which version of the configure-anax doc you'll be changing?  I'd like to ensure that your work doesn't get lost in the refactoring.

@johnwalicki please go ahead and submit a PR on this issue, and I'll have @mosquito provide you with past samples to ensure continuity/alignment.


Extract Authors fails

I am trying to run the project locally on my computer.
When executing the line "sudo npm run indexer:extractAuthors", which is given in the readme file, I get the following error.
><EMAIL_ADDRESS>indexer:extractAuthors
> bundle exec ruby scripts/indexer/extract_authors.rb

Processing records:
scripts/indexer/extract_authors.rb:21:in `load': Nil is not a valid JSON source. (TypeError)
	from scripts/indexer/extract_authors.rb:21:in `block (3 levels) in <main>'
	from scripts/indexer/extract_authors.rb:19:in `map'
	from scripts/indexer/extract_authors.rb:19:in `block (2 levels) in <main>'
	from scripts/indexer/extract_authors.rb:18:in `each_slice'
	from scripts/indexer/extract_authors.rb:18:in `block in <main>'
	from scripts/indexer/extract_authors.rb:15:in `open'
	from scripts/indexer/extract_authors.rb:15:in `<main>'

All previous steps were successful, and sample_dataset.txt.gz does exist under scripts/data/.
Thank you very much in advance.

It sounds like there might be an empty line in DATA_FILE. Could you try removing it?

@navataksha - probably a bit late but here's what I did to fix the issue:

Unzip sample_dataset.txt.gz
Open sample_dataset.txt, remove the last blank line and save the file
Recompress the file by doing gzip sample_dataset.txt


⚠ Samsung has degraded performance

In f2bd4c0, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2812 ms

Resolved: Samsung performance has improved in 027edc4.


[SPARK-20621][deploy]Delete deprecated config parameter in 'spark-env.sh'

What changes were proposed in this pull request?
Currently, spark.executor.instances is deprecated in spark-env.sh, because we suggest config it in spark-defaults.conf or other config file. And also this parameter is useless even if you set it in spark-env.sh, so remove it in this patch.
How was this patch tested?
Existing tests.
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Hi @vanzin, just as talked, we should remote the spark.executor.instances from spark-env.sh.

Thanks @srowen.

Jenkins, ok to test

Test build #76531 has started for PR 17881 at commit 50b4f2d.

Test build #76531 has finished for PR 17881 at commit 50b4f2d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76531/
Test PASSed.

Merging to master / 2.2.

Thanks @vanzin .


Error compiling module assets

Just started work with a Laravel project that is using Modules and I am trying to compile module assets with npm run dev and I get the errors reproduced below...
Any ideas?
Thanks!
ERROR  Failed to compile with 2 errors
 error  in ./Resources/assets/sass/app.scss

Module build failed (from ./node_modules/css-loader/index.js):
ModuleBuildError: Module build failed (from PROJECT_ROOT_FOLDER/node_modules/sass-loader/dist/cjs.js):
TypeError: this.getOptions is not a function
    at Object.loader (PROJECT_ROOT_FOLDER/node_modules/sass-loader/dist/index.js:25:24)
    at PROJECT_ROOT_FOLDER/Modules/MODULE_NAME/node_modules/webpack/lib/NormalModule.js:316:20
    at PROJECT_ROOT_FOLDER/Modules/MODULE_NAME/node_modules/loader-runner/lib/LoaderRunner.js:367:11
    at PROJECT_ROOT_FOLDER/Modules/MODULE_NAME/node_modules/loader-runner/lib/LoaderRunner.js:233:18
    at processTicksAndRejections (internal/process/task_queues.js:97:5)

 error  in ./Resources/assets/sass/app.scss

Module build failed (from PROJECT_ROOT_FOLDER/node_modules/sass-loader/dist/cjs.js):
TypeError: this.getOptions is not a function
    at Object.loader (PROJECT_ROOT_FOLDERt/node_modules/sass-loader/dist/index.js:25:24)

 @ ./Resources/assets/sass/app.scss 2:14-277

I've tried this on a new module in my case the CSS file is generated without any errors.
Try deleting the node_modules folder and run the command again.

I just created a brand new module 'Reports':-
artisan module:make Reports
changed directory to the module root and ran:-
npm install
then
npm run dev
Same error - below is the debug output log file contents if that helps?
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/bin/node', '/usr/local/bin/npm', 'run', 'dev' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'predev', 'dev', 'postdev' ]
5 info lifecycle @~predev: @
6 info lifecycle @~dev: @
7 verbose lifecycle @~dev: unsafe-perm in lifecycle true
8 verbose lifecycle @~dev: PATH: /usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:PROJECT_ROOT_FOLDER/Modules/Reports/node_modules/.bin:/opt/local/bin:/opt/local/sbin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/opt/X11/bin:/usr/local/mysql/bin:/HOME_DIR/.composer/vendor/bin:/HOME_DIR:./:/usr/local/sbin
9 verbose lifecycle @~dev: CWD: PROJECT_ROOT_FOLDER/Modules/Reports
10 silly lifecycle @~dev: Args: [ '-c', 'npm run development' ]
11 silly lifecycle @~dev: Returned: code: 2  signal: null
12 info lifecycle @~dev: Failed to exec dev script
13 verbose stack Error: @ dev: `npm run development`
13 verbose stack Exit status 2
13 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:332:16)
13 verbose stack     at EventEmitter.emit (events.js:315:20)
13 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
13 verbose stack     at ChildProcess.emit (events.js:315:20)
13 verbose stack     at maybeClose (internal/child_process.js:1021:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:286:5)
14 verbose pkgid @
15 verbose cwd PROJECT_ROOT_FOLDER/Modules/Reports
16 verbose Darwin 16.7.0
17 verbose argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "dev"
18 verbose node v12.18.3
19 verbose npm  v6.14.6
20 error code ELIFECYCLE
21 error errno 2
22 error @ dev: `npm run development`
22 error Exit status 2
23 error Failed at the @ dev script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 2, true ]

Any other ideas?
Thanks


Added DB.Clone() and fixed build break

Coverage increased (+0.05%) to 91.724% when pulling fea702453b3893f246ea6189483d47097f516ac6 on qiangxue-patch-1 into 4a691af007e34d2fde28c5b3f8792ef7fdcd30f9 on master.

Coverage increased (+0.05%) to 91.724% when pulling fea702453b3893f246ea6189483d47097f516ac6 on qiangxue-patch-1 into 4a691af007e34d2fde28c5b3f8792ef7fdcd30f9 on master.

Coverage increased (+0.05%) to 91.724% when pulling fea702453b3893f246ea6189483d47097f516ac6 on qiangxue-patch-1 into 4a691af007e34d2fde28c5b3f8792ef7fdcd30f9 on master.


Proposal for wrap_go.c ARM schema refactoring

Perf:
base branch | make go_19 : tbc
this branch   | make go_19 : tbc


Make druid Null handling behave like SQL standard by default and deprecate older behaviors

With the changes in https://github.com/druid-io/druid/pull/4754 druid can now handle nulls as per SQL standard, this task is to make this default standard behavior and remove legacy code/tests related to old behavior.

SchemalessTestSimpleTest.testFullOnTopN() needs to be re-enabled maybe

Also look into https://github.com/druid-io/druid/issues/5169 before enabling it by default.


CI: Temporarily disable CI on Julia nightly

CI is failing on Julia nightly because DelimitedFiles was removed from the standard library. This PR temporarily disables CI on Julia nightly. We will re-enable CI on Julia nightly once the underlying issue is fixed.

cc: @KristofferC