Add event when max value is reached

Is there a way to know when the max value is limited? I don't see in the doc and I'm trying to find a work around but I'm struggling a little bit because the list of option changes when I select a new one.

Added @max event in 2.5.8.


Create detailed analysis of data flow through MDEW

mod-data-export-worker uses a framework called Spring Batch, with the following stereotypes:

These are predominantly defined in the class org.folio.dew.batch.bursarfeefines.BursarExportJobConfig (the "Job" stereotype above).
Within this job, there are three steps (defined as beans in the same class):

exportChargeFeefinesStep (exports data for charges/debits)
exportRefundFeefinesStep (exports data for refunds/credits)
transferFeefinesStep (marks as transferred)

We will be replacing the first two with our own logic and heavily modifying the last one to ensure we transfer the necessary accounts (with the new specification/logic developed).

Bit better diagram of how processor/reader/etc work

This is also a pretty good resource for spring batch terminology: https://docs.spring.io/spring-batch/docs/current/reference/html/domain.html#domainLanguageOfBatch

I've rearranged some stuff in the worker; here's a really badly drawn diagram of the job hierarchy and the main steps/interfaces the data flows through.  This fits the step stereotypes outlined above.

Looks good, so why the change?

On Mar 8, 2023, at 6:13 PM, Noah Overcash @.***> wrote:
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F8005215%2F223881113-4839269a-4fc2-4435-a0dc-06797b66377b.png&data=05|01|steven.j.turner%40ua.edu|5f3a09d6b4e2486ed5e508db20332603|2a00728ef0d040b4a4e8ce433f3fbca7|0|0|638139176282664012|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=czZMPSoRYp%2FyThue0jdf1K1Ky0KVvEAaIVdaTzEEEvM%3D&reserved=0
I've rearranged some stuff in the worker; here's a really badly drawn diagram of the job hierarchy and the main steps/interfaces the data flows through. This fits the step stereotypes outlined above.
—
Reply to this email directly, view it on GitHub https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fualibweb%2Fmod-data-export-worker%2Fissues%2F10%23issuecomment-1461066544&data=05|01|steven.j.turner%40ua.edu|5f3a09d6b4e2486ed5e508db20332603|2a00728ef0d040b4a4e8ce433f3fbca7|0|0|638139176282664012|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=54%2FEQZLjLaAdN1zrACriyQ1DAhNJ4UrBJLZNKsh15Ys%3D&reserved=0, or unsubscribe https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADDW6CDE5KGQEW3YVOAGXVTW3EODTANCNFSM6AAAAAAVCF2GTQ&data=05|01|steven.j.turner%40ua.edu|5f3a09d6b4e2486ed5e508db20332603|2a00728ef0d040b4a4e8ce433f3fbca7|0|0|638139176282664012|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=x93O34P5BieH%2FjOSl7bdYlZL3dXp6TNaHKIwqSV3qBw%3D&reserved=0.
You are receiving this because you were mentioned.

@sjturner-ua I modified some things to the diagram above; most of it was renaming some parts to have clearer names


ARM64 Build

Hey,
Can you guys release ARM64 builds? I mainly run Kubernetes on ~50-100 ARM clusters and wanted to test Falco out. I did try compiling myself, but gRPC was throwing an odd error during compilation at me (I fixed the errors thrown by protobuf by providing my own compiled installation of protobuf on the system). I assume I could maybe compile newer grpc and provide it on the system... but I'd rather not hack together a build process to get a build for a single release.
Many thanks in advance.
What would you like to be added:
Deb, RPM and pre-built packages for ARM7 and ARM8.
Why is this needed:
We want to have packages for other platforms than X86_64 - ARM7 and ARM7 both seem platforms on which many users want to try Falco.
Related issues

#586

@nfrush I edited this issue to match the new template we have, there's a whole set of improvements to the build system from @leodido in #719 - after that we will be able to include more architectures in an easier way.

I'm closing this since right now ARM support is not in the core maintainers team priorities and since February no one seemed to want to pickup this one.
@nfrush feel free to reopen if you think you can submit a PR to make this happen and feel free to continue the discussion even after the close.

/assign

I am working on this now - I believe the problem is with the lua dependencies
I will report back once I get it building

So I have Falco working on aarch64. The issue was that the version of luajit that was being bundled into the build isn't compatible with arm aarch64 architecture.
There is more detail on the official luajit website, but I tried to compile with both lua 5.1.5 and luajit 2.0.0 as the documentation for ARM suggests.
Both of those failed. Lua failed the check for being a valid lua interpretor, and 2.0.0 and 2.0.5 both returned the error documented here
==== Building LuaJIT 2.0.3 ====
lj_arch.h:55:2: error: #error "No support for this architecture (yet)"
 #error "No support for this architecture (yet)"
  ^
lj_arch.h:279:2: error: #error "No target architecture defined"
 #error "No target architecture defined"
  ^
lj_arch.h:55:2: error: #error "No support for this architecture (yet)"
 #error "No support for this architecture (yet)"

@kris-nova why is this closed? Am I missing something?

Ah sorry I must have hit the wrong button. I have Falco compiling on armv7 and am going to have a blog out by the end of the week and demo on setns this Friday.
Stay tuned for links and documentation on how to do this.

@kris-nova I'm also a user trying to get ARM(64) falco working, you were going to release a blog of sorts, did you ever manage to release it? OR document somewhere your process to follow?

Bump - Can we label this so it doesn't go stale?

Looking forward to this update. Homelab consists of mix of arm & amd.

@fntlnz encouraged me to try building Falco on my RPi4. I used Ubuntu 20 and the 64-bit image. The device has 4GB of RAM.
I'm getting the following error.
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/filtered_re2.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/mimics_pcre.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/nfa.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/onepass.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/parse.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/perl_groups.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/prefilter.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/prefilter_tree.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/prog.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/re2.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/regexp.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/set.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/simplify.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/stringpiece.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/tostring.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/unicode_casefold.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/re2/unicode_groups.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/util/pcre.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/util/rune.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/re2/util/strutil.o
[C]       Compiling third_party/upb/upb/decode.c
[C]       Compiling third_party/upb/upb/encode.c
[C]       Compiling third_party/upb/upb/msg.c
[C]       Compiling third_party/upb/upb/port.c
[C]       Compiling third_party/upb/upb/table.c
[C]       Compiling third_party/upb/upb/upb.c
[AR]      Creating /home/ubuntu/falco/grpc-prefix/src/grpc/libs/opt/libupb.a
ar: creating /home/ubuntu/falco/grpc-prefix/src/grpc/libs/opt/libupb.a
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/upb/upb/decode.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/upb/upb/encode.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/upb/upb/msg.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/upb/upb/port.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/upb/upb/table.o
a - /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/upb/upb/upb.o
[CXX]     Compiling third_party/abseil-cpp/absl/base/dynamic_annotations.cc
c++: error: unrecognized command line option ‘-maes’
c++: error: unrecognized command line option ‘-msse4’
make[4]: *** [Makefile:2968: /home/ubuntu/falco/grpc-prefix/src/grpc/objs/opt/third_party/abseil-cpp/absl/base/dynamic_annotations.o] Error 1
make[3]: *** [CMakeFiles/grpc.dir/build.make:113: grpc-prefix/src/grpc-stamp/grpc-build] Error 2
make[2]: *** [CMakeFiles/Makefile2:848: CMakeFiles/grpc.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:6062: userspace/falco/CMakeFiles/falco.dir/rule] Error 2
make: *** [Makefile:2643: falco] Error 2
ubuntu@ubuntu:~/falco$ uname -a
Linux ubuntu 5.4.0-1015-raspi #15-Ubuntu SMP Fri Jul 10 05:34:24 UTC 2020 aarch64 aarch64 aarch64 GNU/Linux
ubuntu@ubuntu:~/falco$
[1] 0:bash*

Also pinging @kris-nova for follow-up in the week.


Can Timestamp in metadata be overwritten?

By calling this SDK to send EMF logs, it seems we cannot define the timestamp of the metrics.
The timestamp is generated at https://github.com/awslabs/aws-embedded-metrics-dotnet/blob/fec85c5859450d8061e5e515f6ca229d656eb7c9/src/Amazon.CloudWatch.EMF/Model/MetaData.cs#L27-L29, and I don't find the way to overwrite it.
However, we could assign Timestamp as wanted, if we construct the EMF log by our own, see: https://docs.aws.amazon.com/AmazonCloudWatch/latest/monitoring/CloudWatch_Embedded_Metric_Format_Generation_PutLogEvents.html.

@markkuhn We would like to have this feature. Could someone fix it soon? If not, I will try to propose a PR, but it may take some time for me. Thanks!

Hi @zhuliice, Support to add custom timestamp is now supported. Pull request: (https://github.com/awslabs/aws-embedded-metrics-dotnet/pull/55). I am marking it as closed.


youtube-dl Throwing 410 Errors

[youtube] OTLUzIB4QHw: Downloading webpage
[youtube] OTLUzIB4QHw: Downloading video info webpage
WARNING: unable to download video info webpage: HTTP Error 410: Gone
WARNING: unable to download video info webpage: HTTP Error 410: Gone
WARNING: unable to download video info webpage: HTTP Error 410: Gone
WARNING: unable to download video info webpage: HTTP Error 410: Gone
WARNING: unable to download video info webpage: HTTP Error 410: Gone
Traceback (most recent call last):
File "/usr/bin/youtube-dl", line 11, in
load_entry_point('youtube-dl==2019.1.17', 'console_scripts', 'youtube-dl')()
File "/usr/lib/python3/dist-packages/youtube_dl/init.py", line 477, in main
_real_main(argv)
File "/usr/lib/python3/dist-packages/youtube_dl/init.py", line 467, in _real_main
retcode = ydl.download(all_urls)
File "/usr/lib/python3/dist-packages/youtube_dl/YoutubeDL.py", line 2002, in download
url, force_generic_extractor=self.params.get('force_generic_extractor', False))
File "/usr/lib/python3/dist-packages/youtube_dl/YoutubeDL.py", line 793, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/common.py", line 508, in extract
ie_result = self._real_extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/youtube.py", line 1672, in _real_extract
token = video_info.get('token') or video_info.get('account_playback_token')
AttributeError: 'NoneType' object has no attribute 'get'
Trying to download YT videos but i am greeted with this ^
Sorry if this has been posted somewhere else i could not find it
Please help

This was after running:
youtube-dl -f bestvideo+bestaudio --merge-output-format mkv --all-subs --cookies youtube_com.txt https://www.youtube.com/watch?v=OTLUzIB4QHw

load_entry_point('youtube-dl==2019.1.17', 'console_scripts', 'youtube-dl')()

Too old. Update to the latest version first.

I have the same problem, and I have the same line that says
load_entry_point('youtube-dl==2019.1.17', 'console_scripts', 'youtube-dl')()
but when I run
sudo youtube-dl -U
it says I'm up to date.


Upgrade to v0.9.34

[x] Upgrade DEX pallet to v0.9.34
[ ] Consult Polkadot v0.9.34 release report for important changes.
[x] Upgrade https://github.com/paritytech/pallet-contracts-xcm to polkadot-v0.9.34
[ ] Upgrade Trappist runtime & node to v0.9.34

Depends on #98

We decided to update current trappist code(0.9.33) to 0.9.36 directly Because Cumulus repo does not have any branch related to 0.9.34 and 0.9.35.


Fix websocket ping-pong handling in components/tcp_transport

The components/esp_websocket_client sends PING frame to server every 10 seconds by default, this usually triggers the server replying with a PONG frame, which could trigger a read error.
This pull request fixed this issue, and adding the following minor changes:

Reply with PONG frame when a PING frame from server is received. This prevent timeout if server has a keep-alive mechanism.
Rephrasing debug message in components/esp_websocket_client prevent misunderstanding.

 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.Johnny Wan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


CMD+S does not work to save a notebook

Splitting this off from #7993 as we need to fix this part at least.
CTRL+S works on windows, but CMD+S does not work on mac

Hey Don. Cmd+S does save now, but it looks like the fix on my review comment to not allow ctrl+s to save didn't work. They both save still. Looks like the commit added only checks OSX for the cmd case, the ctrl+s is still let through for both Windows and Mac.

Validated.


Tbi/app gen/local storage connector 23289

https://github.com/SAP/open-ux-tools/issues/1158
Flexibility colleagues adviced that the SessionStorageConnector is the wrong connector. We should use LocalStorageConnector. This PR replaces SessionStorageConnector with LocalStorageConnector

Why the 2 changesets?

@zdravko-georgiev @tobiasqueck I removed the second changeset. Leftover from previous PR, apologies.

Thank you all 👍


Simplify Cookie Class

Since we use php min version 7.3, why not passing all cookie options to the constructor itself directly.
Removed Cookie::create while new Cookie do the same thing directly.
Renamed Cookie::getExpiresTimestamp to Cookie::getExpires to be short and refer to it's property,
Renamed Cookie::withExpiresAt to Cookie::withExpires to fit it's property and also it may return expired cookie.
I think this is last change to new Cookie class should be done before the upcoming release
@paulbalandan what do you think about this changes? is it keeps the interface "fluent"? Thanks in advance

Cookie::getExpiresString is still here and has not been removed.

cannot set property via array access on class constructor this case syntax error new Cookie('cookie', 'monster')['expires'] = 7200;
syntax error, unexpected token "["

I assume this is yet another PR that will need to be resolved before release. Please prioritize this.

Thank you, @mostafakhudair

@paulbalandan Many thanks to you for you'r professional review and your patience

Thanks all around! I think we are now finally cleared for release.


List subscriptions on the RunEngine

Would it be possible to list subscriptions on the RunEngine?
It seems that the only way to unsubscribe something is to know it's token. However, unless this is actively saved into memory somewhere this is currently unknowable.
Expected Behavior
>>> print(RE.list_subs())
{0:' db.insert,'
1: 'confirm_saved',
2: 'ZMQ'
...
}

If this is already possible, it isn't documented in the Live Visualization section with the other subscription things.

There is no way to provide a view quite as simple as that which is also correct in the general case. I do not think we will exposure a public way to do this.
You can get RunEngine.dispatcher.cb_registry.callbacks but because of considerations related to garbage collection, there is a layer of indirection that makes the reprs less than informative:
In [3]: RE = RunEngine()

In [4]: def f(name, doc):
   ...:     return None
   ...:

In [5]: RE.subscribe(f)
Out[5]: 0

In [6]: RE.dispatcher.cb_registry
Out[6]: <bluesky.utils.CallbackRegistry at 0x11967a898>

In [7]: RE.dispatcher.cb_registry.callbacks
Out[7]:
{<DocumentNames.stop: 'stop'>: {1: <bluesky.utils._BoundMethodProxy at 0x119698358>},
 <DocumentNames.start: 'start'>: {2: <bluesky.utils._BoundMethodProxy at 0x119698630>},
 <DocumentNames.descriptor: 'descriptor'>: {3: <bluesky.utils._BoundMethodProxy at 0x119698828>},
 <DocumentNames.event: 'event'>: {4: <bluesky.utils._BoundMethodProxy at 0x1196989e8>},
 <DocumentNames.bulk_events: 'bulk_events'>: {5: <bluesky.utils._BoundMethodProxy at 0x119698940>}}

I am :-1: on adding public API for this.  I also would not suggest digging into the dispatcher like that.
The reason we return the token is so that you can remove it later.

Yes but many times the token is not put anywhere so it is lost. Eg it is in the ipython profile and not placed into a variable.

The BEC and db.insert registrations are intentionally hard to remove.  There is an edge between library development (bluesky) and application development (what is effectively being done in the profiles).
If you just want to turn the BEC off, use the methods on it to disable plotting / printing.

Actually I was looking to turn off db.insert to not flood the databroker with zeros (since we will be testing out live analysis without beam initially). But I guess I could just comment them out.

:+1: to just commenting them out for testing (or setting using a temporary databroker)

Cool.


Error when loading scene with custom AABB on a GeometryInstance3D

Tested versions
Reproducable in v4.2.stable.mono
System information
Godot v4.2.stable.mono - Windows 10.0.22621 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3070 (NVIDIA; <IP_ADDRESS>17) - AMD Ryzen 5 5600X 6-Core Processor (12 Threads)
Issue description
When you edit the custom bounding box for the GeometryInstance3D in a MeshInstance3D, subsequently opening up the scene containing the node, it throws a cpp error.
servers/rendering/renderer_scene_cull.cpp:1019 - Condition "!is_geometry_instance(instance->base_type)" is true.
This also causes the custom AABB to be ignored and revert to default behaviour.

Steps to reproduce
Create a MeshInstance3D.
Edit 'Custom AABB' in the GeometryInstance3D section.
Save the scene and re-open it.
Minimal reproduction project (MRP)
AABB_test.zip

Would like to mention that this is still happening in 4.3.

This is also printed whenever Custom AABB is printed if no mesh is currently assigned. This likely occurs because Custom AABB is being set before the mesh is actually assigned.
Ideally, we should make it possible to properly assign a custom AABB even if no mesh has been assigned (it can be handy for run-time loading and avoids some pitfalls).


Expose get_smarts to JS

This small PR exposes get_smarts() functionality to JS. This is very hand, for example, to convert MDL queries to SMARTS strings.

nice!

Could you please add get_cxsmarts() too? That’d make it symmetric w.r.t. SMILES generation

@greglandrum Good point; done.


Create "json" field  on PostgreSQL by migration Yii <IP_ADDRESS>

Can not find proper way to create json field by migration mechanism.
Database PostgreSQL 10
What steps will reproduce the problem?
Create migration  for
$this->createTable('table_name', [
            "id" => $this->bigPrimaryKey(),
            "body" => "json",   // warning "json" generates jsonb type
        ]);

What is the expected result?
Created table with field body type "json"
What do you get instead?
Created table with field body type "jsonb"
Additional info
According to postges doc
"In general, most applications should prefer to store JSON data as jsonb, unless there are quite specialized needs, such as legacy assumptions about ordering of object keys."
So some application need create json instead jsonb.
Jsonb just little slower for store and read and it changes keys ordering.
Please add possibilty to create json not only jsonb
Prev  issue

Q
A

Yii version
<IP_ADDRESS>

PHP version    7.1

Database - PostgreSQL 10

Operating system  Linux Debian Jessie

As I;ve already stated in previous issue,  this flag should be added to \yii\db\SchemaBuilderTrait::json() as parameter, if we call it in migration like this
$this->addColumn('public.user', 'kyc_info', $this->json()->null());

as far as I see, this should be changed to json if you want jsonb you need to explicitly type the name:
https://github.com/yiisoft/yii2/blob/7b0fb14c3c9a995cebfb42e5c5f586e9f6edce72/framework/db/pgsql/QueryBuilder.php#L77

it will not work as it will be replaced on the lower level. Even if SchemaBuilder generates a string like json it will be replaced by QueryBuilder by jsonb again.

In case you need to create JSON column instead of JSONB I would suggest not to use schema builder and define type manually. PostgreSQL is the only engine (as far as I know) that supports two different types of JSON and considering JSONB is more flexible, we use it by default for QueryBuilder::json().

So how it would be possible to create json type by migration?
only manually sql command?

Yes.

finnaly i find other way to solve this
$this->createTable('report', [
            "id" => $this->bigPrimaryKey(),
            "body"        =>  " json", // <- this will generate json field not jsonb
        ]);

or for better
public function json($length = null) {
     return $this->getDb()->getSchema()->createColumnSchemaBuilder(" ".Schema::TYPE_JSON, $length);
 }
...............
 $this->createTable('report', [
         "id" => $this->bigPrimaryKey(),
         "body"        =>  $this->json(),
     ]);

(" ".\yii\db\pgsql\Schema::TYPE_JSON) is still a working workaround nowadays.


All example in "Please input one of the following" validation messages need to be built from the "options" lists at the top

When adding a new option to the metadata, I'm always having to double-check if other messages also need to be updated. Shouldn't have to do this.

Duplicate of https://github.com/IfyAniefuna/experiment_metadata/issues/53


helm template ignore empty yaml objects

Output of helm version:
Client: &version.Version{SemVer:"v2.8.2", GitCommit:"a80231648a1473929271764b920a8e346f6de844", GitTreeState:"clean"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.6", GitCommit:"9f8ebd171479bec0ada837d7ee641dec2f8c6dd1", GitTreeState:"clean", BuildDate:"2018-03-21T15:21:50Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.6", GitCommit:"9f8ebd171479bec0ada837d7ee641dec2f8c6dd1", GitTreeState:"clean", BuildDate:"2018-03-21T15:13:31Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Hi there,
I've been using helm template to generate YAML files for deployment. I would like to alter which YAMLs are output based on an env value in my values.yaml file so that I may generate production, dev, staging, etc deployments. For example, I only want to output a tls secret in a production environment:
values.yaml:
# This is a YAML-formatted file.
# Declare variables to be passed into your templates.

env: production

namespace:
  foo: baz
.
.
.

tls.yaml:
{{- if eq .Values.env "production" }}
apiVersion: v1
kind: Secret
metadata:
  name: tls
  namespace: {{ .Values.namespace.foo }}
  labels:
    app: bar
type: kubernetes.io/tls
data:
  tls.crt: {{ .Values.tls.cert }}
  tls.key: {{ .Values.tls.key }}
{{- end }}

Currently, tls.yaml is output as
---
# Source: mychart/templates/tls.yaml

If I try to apply this YAML object, kubectl outputs error: no objects passed to apply. Ideally, if a YAML object is empty after executing the go template, it should just not appear in the output.
I'd like to know two things:

Can I or some existing contributor add a condition and/or runtime argument flag to tell helm template to skip empty objects in its output? I know exactly where to implement this change, and I considered forking, implementing, and opening a PR, but I wanted to hear whether this functionality has been intentionally left out thanks to some better work flow that achieves the same goal.
If this functionality has been intentionally excluded, why? What is the better flow for achieving this goal?

Thanks in advance!

I think you can use helm template --output-dir, and it should omit the empty yaml files automatically

Closing due to inactivity. Please open a new ticket against Helm 3 if this issue persists. Thanks!


Bulk Add Tags?

Hey,
Is there any way to add tags in bulk? I found out how to do it one by one (somewhat unclear by the way, perhaps having that more visible and documented somewhere would be better) but I haven't found a way to apply it to say, a whole folder.
Thanks

Same answer,

Yep, it can be done with custom automatic tagging rules


fix: fix missing RBAC permission

so that we can get tekton pipelines

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: vbehar
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [vbehar]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Payment Channels

Payment channels, as described on the Contracts wiki and implemented in jgarzik/mcp, should be available in picocoin.
Adding here for tracking and discussion.

Concept ACK - idling out this issue

Great to get the ACK, @jgarzik!  Would you care to share some thoughts about the future of picocoin (maybe a 12 month roadmap)?  I remain extremely interested in getting payment channels working in that timeframe!

The focus is on a solid C library for bitcoin (libccoin), with the wallet piece remaining unloved (picocoin).
Implementing payment channels in the lib would be great.
Short term focus in simply keeping up with current BIPs, and removing OpenSSL.

Thanks, Jeff!  I wonder if @chjj would be interested in lending a hand.


Support SQLITE's ON CONFLICT clause

Sqlite supports a non-standard ON CONFLICT clause (https://www.sqlite.org/lang_conflict.html) that determines how inserts/updates/create tables work, taking into account the entire row. Notably, this is distinct from sqlite's implementation of ON CONFLICT for use in its upsert clause - https://www.sqlite.org/lang_UPSERT.html.
An example usecase of this feature is a simplified upsert, that just replaces the underlying with the new row if there's a primary key or unique constraint violation of any sort, instead of having to declare every conflict resolution separate from the main insert body:
(Note the use of OR REPLACE instead of ON CONFLICT REPLACE, which is part of the syntax to be less verbose in INSERT/UPDATE statements)
INSERT OR REPLACE INTO subscriptions (
  id, -- The primary key, if this is conflicted, the row gets deleted and replaced with this new one
  many,
  keys,
  that,
  will,
  always,
  be,
  upserted
) VALUES ( ... )

You can get the same result like this https://kyse.link/?p=s&i=wSDqoqqCK5mM9d5z5Ywy. Since this is trivially implemented using the current features, I don't think we should add another nonstandard method.

You can get the same result like this https://kyse.link/?p=s&i=wSDqoqqCK5mM9d5z5Ywy. Since this is trivially implemented using the current features, I don't think we should add another non-standard method.

But here in your example we can not inser arrays right? If we convert the user object to an array we can insert this array but the doUpdateSet fails. Maybe we should wait for #976 to get this functionality


Adeguate minimum SDK for both plugins

Hi creche-studio,
I loved your plugin for unique identification, but I noticed you placed higher SDK than necessary.
Based on what plugin you used on iOS and Android, I lowered the SDK to the minimum necessary.
I tested it and currently using it in my app on production, and it is working fine.
Thanks for the idea :)

@Ottavio97
Hi, Ottavio97.
Thank you for sending PR to this repo.
I merged your PR and published v0.0.2.


fix(#771):Dart Program to Add Two Numbers

Description
Additional Context
Created a simple dart program to add 2 numbers

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Is this execution time the expected one?

It seems a bit slow to me, is this time the usual one? The project itself it's not big at all.
My setup is using gulp as the taskrunner and gulp-scss-lint to wrap the scss-lint gem.

TL;DR This is not unusual in my experience. Try master. And try disabling linters from the list below if you really need it faster.
I've been working on speeding scss-lint up (#577, #579, #580, #584, #622, #623, #624), but I haven't really seen any huge wins. What version of scss-lint are you running? My last 3 patches have not been released yet, so you might try again on the master branch. I wouldn't be surprised if a ~6.7s process now takes ~5s.
If you're interested in boosting performance, I'd recommend tmm1's stackprof, a truly beautiful tool, for Ruby 2.1+. You can call StackProf.run(mode: :cpu, out: 'tmp/stackprof.dump') do in Runner#run or CLI#scan_for_lints.
I think I found that typically, Sass parsing (not under scss-lint's control) seems to take ~50% of the time, so right there, you're not going to make the the scss-lint process 10x faster, ever (on a single file).
We discuss in #576 the idea of parallelizing scss-lint. I've got a patch in the works that seems to reduce time linting by like ~15%. It's not great. It's also a very very ugly looking patch, so I haven't posted it yet. (I also discovered that something somewhere is not thread-safe: I could not parallelize all of the Sass-parsing, and scss-linting activities across all individual linters and across all files. I can't recall the details...)
Honestly when talking about ~7s, I think a lot goes into Ruby's VM startup time.
I also timed the individual linters, linting over a copy of Bootstrap 4's SCSS files. Here's the slowest 10 linters (before the patches in #622, #623, #624). Times in seconds:
           SCSSLint::Linter::FinalNewline: 0.362
              SCSSLint::Linter::UrlFormat: 0.363
             SCSSLint::Linter::NameFormat: 0.389
      SCSSLint::Linter::PropertySortOrder: 0.396
         SCSSLint::Linter::SelectorFormat: 0.397
          SCSSLint::Linter::SelectorDepth: 0.408
     SCSSLint::Linter::SpaceBetweenParens: 0.419
 SCSSLint::Linter::EmptyLineBetweenBlocks: 0.453
SCSSLint::Linter::SpaceAfterPropertyColon: 0.462
            SCSSLint::Linter::Indentation: 1.045

So you might consider disabling one or more of these, if the linting time is really bogging you down.

@srawlins It looks like you've been doing some good profiling. I think it would be great to add some sort of profiling/benchmark script/task or something that would make it easy for anyone who is interested in improving the performance of this project to hit the ground running. Would you be interested in submitting a pull request for such a thing?

Yeah, I'd be into that. Unfortunately it means modifying code to run something like stackprof, so maybe we can have a long-running git branch named performance or something? And maybe a PERFORMANCE.md or similar document in the master and performance branches with steps on how to profile? I'd be interested in doing this.

I'm a little uncomfortable with a long-running git branch because I worry about things getting out of sync. I wonder if we could add a --profile flag or something that would turn on profiling for that run.

Hmm, yeah. Might be able to do that. I'm not big on adding a dependency in the gemspec when there are currently only two dependencies.

You could make it a soft dependency (i.e. leaving it out from the gemspec) and document having to install it manually as part of running --profile (or however we end up running it).

Given the question as originally asked as been answered, I'm closing this issue.
Happy to accept a pull request that adds a benchmark measurement script of some kind, but agree that it's preferable to not include its dependencies in the gemspec. Thanks!


Add ability to fetch replies to a tweet

Hi @Rishikant181, is this feature supported or any similar API? Sorry if I missed something.

Unfortunately, it's not supported as of now. Last time I tried to implement it, it was broken and I wasn't able to figure out how Twitter web app does that.

Thank you for your quick response. And, as I see at issue 444, he's mentioning searching by conversation id, but it seems like it doesn't work.
Anyway, thanks for your hard work!


How to set the height of the particles on the terrain?

Does particleHeight mean how many meters above the terrain?
If not, is there any variable to set the height of the particles on the terrain?

Thanks for you answer~
But I need to set the distance of the particles from the surface of earth :(
I need to control the height of the particles on the terrain.
Is there any way to do this?

Sorry, I made a mistake.
particleHeight is actually the distance from the surface of earth, and you can set it using the GUI panel I provide

But I set the particleHeight  to 1, it is still can be see on terrain with the height greater than 1 :(

I think this is because the depth test is not very accurate, try to move camera closer to the terrain to see if it yields better result.

Thanks for your reply~
I move camera closer to the terrain but it is still can be see on terrain with the height greater than 1 :(
I wonder if it is because of the lev dimension?
This is my lev dimension:
data.lev.array = new Float32Array([1]);
It is right?
Sorry I don't quite understand the meaning of lev dimension:(


concurrent and multithreaded

Also related with https://github.com/georgeyk/loafer/issues/29
We should consider something like this:

the main thread and loop control/receive the signals and manages "worker" threads and loops
every loop own a thread (or threads) or every thread own a loop ?

Check the thread-pool executor for study and reference.
We should be able to configure the number of threads globally or by route ?

Add section um FAQ about honcho usage


Switch UI to new design

Fixes #105
Same as #109
Details
- Able to Create TrainJob
- Display TrainsJobs
- Display Trials
- Display Trail Details

How to test on local machine (may need to run source env.sh)
$ yarn install
$ yarn start

TODO:
- Refractor the code to to manage states/routes/data globally, enabling the ability to create new application page also time-travel with routing
- Datasets Display/Upload

If possible, you should remove/correct all template metadata from the template were you using. E.g. in bower.json, package.json, public/manifest.json

Integrated your changes with the deployment of the complete Rafiki stack. You should now be able to run a local instance of Rafiki together with your new front-end.

Hi @wild-flame, I've merged the latest changes from v0.1.0 into this branch and did some fixing.
Managed to get the web admin running integrated with the whole Rafiki stack when I run bash scripts/start.sh, and the train jobs page, train job detail page (image below) and the trial detail page seem to be working! However, the changes in APIs for creation of train job & new dataset management API might have broken other parts of the front-end (e.g. the create train job UI). Believe you are working on it.

Take note that in my latest commits, the additional app.js and extra dependencies in package.json are for the Docker-based deployment of the web admin (see the web_admin.Dockerfile).

Finish Dataset Uploading, now will alter the create train job page to match the new API

Hi @wild-flame, I've merged the latest changes from v0.1.0, did some fixing, and tested the latest changes in this branch!
Feedback on UI:

Creating of datasets on the UI was working for me! Both from file system & URL
But the instructions on the create dataset page is inaccurate for now:

The dataset format should conform to the current task specification at https://nginyc.github.io/rafiki/docs/latest/docs/src/user/datasets.html#dataset-type-image-files. Although we want to support the dataset format you described (directories of images), let's do that in a separate PR so that there won't be so many changes for this PR (will be adding an issue for this)
Application developers have to separately split their datasets into train & validation, and create 2 datasets separately. Regarding having Rafiki to do this dataset splitting, again, let's do that in a separate PR

But when I've created a dataset, the create dataset page remains. It should navigate away from the page
May want to do some form validation on the create dataset page
Able to view the datasets I have on the new UI (screenshot below)
But when I click on a dataset, the URL changes but not the page. Subsequently, when I click on the "New Dataset" button, the create dataset page doesn't appear. Maybe should disable this?

The rest features are to be implemented in separate PRs related to issue #125 #122 #118

Hey @wild-flame thanks for the updates! I've checked all the listed functionality and it all works. However, I had do some correction of English, and I believe we should touch-up on the UI more down the road e.g. consistent & sufficient whitespace between UI elements. I'll be merging this PR!


Implement Inference Caching in the Backend Instead of Frontend

Issue description
The goal is to enable caching capability in the backend system to store the outcome of an inference request made on an image. In case the users opt for different models for the same image, the inference request must be executed just once. Switching to a model that has already produced an inference on a chosen image should return the outcome that has been saved in the cache.
Steps to Reproduce

Send a request to the /inf route two time
The first time, the system returns the inference result from the pipeline
The second time, the system returns the result stored in the cache

Expected Behavior
If the cache does not contain any results for the chosen image and pipeline, the system should invoke the necessary model(s) to produce the inference result. On the other hand, if the system finds a matching image in the cache that corresponds to the pipeline inference result, it should simply return the cached result.
Current Behavior
The model(s) are called at every inference request since the inference result is not cached.
Possible Solution
This issue is referencing this comment:
I'm not exactly sure how it works on the front end, but there is already a caching functionality that keeps track of the result.
scores: [],
classifications: [],
boxes: [],
annotated: false,
imageDims: [],
overlapping: [],
overlappingIndices: [],

As of now, inferencing would overwrite what is in there. Maybe we could add another parameter that would contain the model that displays the score, classification, boxes, and everything else related?
 model:{
   scores: [],
   ...
}

function loadResultToCache
For the backend
To implement the same idea in the backend, we would need to keep track of the image passing in the inference request, pair it with the pipelines, and keep the result when returned. Then if the image is call again, we send back the result from cache.
---
title: Test
---
flowchart TD

image[Send image to inference request]
check(back end check if image already exists
 in cache and if pipelines name is the same)
inference[image not in cache
 or pipeline not call]
cache[image in cache
 and pipeline call]
inf_res[send back result from inference]
cache_res[send back result stored in cache]

image -- from Front end--> check
check --> inference & cache
inference-->inf_res
cache-->cache_res

Originally posted by @MaxenceGui in https://github.com/ai-cfia/nachet-frontend/issues/96#issuecomment-1933001080
Additional Context

There is already a caching functionality in the frontend to display the result

Tasks

[ ] Implement a caching functionality to store the inference result
[ ] Implement a checking in inference_request to return the cache result if True

There should be some discussion in th spec about the pipeline and the underlying models version. if a model version in the pipeline is updated, it should invalidate previous call. Also perhaps a mechanism to invalidate cache (from the frontend?).
https://martinfowler.com/bliki/TwoHardThings.html


ECONNRESET

I recently started getting the following error:
{ Error: read ECONNRESET
at exports._errnoException (util.js:1007:11)
at TCP.onread (net.js:563:26) code: 'ECONNRESET', errno: 'ECONNRESET', syscall: 'read' }
I've had this process running for a couple of months, and it just randomly started throwing these errors. Any idea why?

Hi, thank you for using node-netcat
are you not having downtimes  or some network issues?
Take care


ARROW-7289 [C#] ListType constructor argument is redundant

ListType constructor is redundant and should be seperated.

sorry, I'll fix the commit history.

I fixed it.


Content size cannot change

How to change content size (width)?

Hi @mBilal5674,
You should be able change bubble's width by setting maxWidth for bubble. There is a property for it in the ToolTipPreferences (preference.drawing.bubble.maxWidth).


how to train with one gpu

my machine has only one gpu. I changed experiment.conf as follows:
 two_local_gpus {
   addresses {
     ps = [localhost:2222]
-    worker = [localhost:2223, localhost:2224]
+    worker = [localhost:2223]
   }
-  gpus = [0, 1]
+  gpus = [0]
 }

when I run python train.py best
it prints 2018-10-23 12:01:57.495403: E tensorflow/stream_executor/cuda/cuda_driver.cc:406] failed call to cuInit: CUDA_ERROR_NO_DEVICE
I checked tensorflow with gpu by:
import tensorflow as tf
with tf.device('/gpu:0'):
    a = tf.constant([1.0, 2.0, 3.0, 4.0, 5.0, 6.0], shape=[2, 3], name='a')
    b = tf.constant([1.0, 2.0, 3.0, 4.0, 5.0, 6.0], shape=[3, 2], name='b')
    c = tf.matmul(a, b)

with tf.Session() as sess:
    print (sess.run(c))

2018-10-23 12:05:53.137069: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:898] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2018-10-23 12:05:53.137468: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1344] Found device 0 with properties:
name: Quadro P3000 major: 6 minor: 1 memoryClockRate(GHz): 1.215
pciBusID: 0000:01:00.0
totalMemory: 5.94GiB freeMemory: 4.95GiB
2018-10-23 12:05:53.137501: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1423] Adding visible gpu devices: 0
2018-10-23 12:05:53.321772: I tensorflow/core/common_runtime/gpu/gpu_device.cc:911] Device interconnect StreamExecutor with strength 1 edge matrix:
2018-10-23 12:05:53.321825: I tensorflow/core/common_runtime/gpu/gpu_device.cc:917]      0
2018-10-23 12:05:53.321834: I tensorflow/core/common_runtime/gpu/gpu_device.cc:930] 0:   N
2018-10-23 12:05:53.322015: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1041] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 4718 MB memory) -> physical GPU (device: 0, name: Quadro P3000, pci bus id: 0000:01:00.0, compute capability: 6.1)
[[22. 28.]
[49. 64.]]
what's wrong with it?

I installed tensorflow and it works


Support for Adonis v4

Any plan for release Adonis v4?

Yes will be soon.

Any progress?
It will be great if you make first release.
Thx

+1

@thetutlage maybe you need a help to the migration?

@thetutlage Can you help me with new implementation of Session?
Check my changes here - https://github.com/adonisjs/adonis-websocket/compare/develop...firsyura:develop

@thetutlage Can you say the approximate date of implementation? I need told to my leadership date of full migrate to v4. Thx

@ntvsx193 Sorry, not sure about the date right now.
Also it's festival session in my country, so kind of stuck into other stuff. Give me a week, I will go through the required changes, make some plan and then let you know.
Ohh man, some should hire me for full time development on AdonisJs now

@firsyura Will look into it soon and let you know.

@ntvsx193 don't want to sound rude or anything, but if you need the feature, why not support @thetutlage instead of asking stuff for free?
Thanks

@firsyura @diego-vieira Just an update.
I have started working on it and will be out soon, also using Redis a way to scale the websockets on multiple servers

@thetutlage Cool. If you need a helping hand feel free to message me

Closing this.
Feel free to continue the discussion here https://github.com/adonisjs/discussion/issues/51


Fix incorrect rendered template

What type of PR is this?

/kind failing-test
What this PR does / why we need it:
Fix CD failure
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm

/lgtm


Disable tooltip on hover

I'm submitting a...

[ x ] Bug report
[ ] Feature request
[ ] Documentation issue or request

Current behavior

The icon name appears in a tooltip when it's on hover. If I've a custom tooltip on the icon, then two tooltips will appear.
Expected behavior

Disable the icon name tooltip
Minimal reproduction of the problem with instructions
We are using icons with the following components:

icon inside ix-event-list-item
ix-icon-button
ix-button with icon
ix-dropdown-button
icon inside ix-input-group
ix-dropdown-item with icon

What is the motivation / use case for changing the behavior?

I believe it's a bug.

We will remove title tags from all SVGs @silviowolf

Closes siemens/ix#903

title attributes removed, pull requested https://github.com/siemens/ix-icons/pull/49


Differentiate software keyboard and touch keyboard

This pull request updates Software keyboard section in Keyboard interactions page to clarify that the touch keyboard is one of software keyboards.
It also removes a note saying "The touch keyboard does not support toggle and most system commands" because the touch keyboard supports toggling some keys such as Ctrl and perform some keyboard shortcuts.

@masaru-iritani : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Karl-Bridge-Microsoft : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Assert that ProductStockIndexer is updating stock or available stock in several situations

1. Why is this change necessary?
The ProductStockIndexer (\Shopware\Core\Content\Product\DataAbstractionLayer\Indexing\ProductStockIndexer) currently cannot adjust the stock and the available stock in all situations where this would be necessary because the relevant events are not generated by the system.
For example:

Updating the quantity of an order line item
Deleting an order line item
Changing the product of an order line item

2. What does this change do, exactly?
This PR just adds unit tests for expected behaviour.
3. Describe each step to reproduce the issue or behaviour.
The unit tests describe the steps to reproduce the behaviour.
4. Please link to the relevant issues (if any).
5. Which documentation changes (if any) need to be made because of this PR?
6. Checklist

[X] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

Hi,
these events were deliberately not implemented because it is not certain whether the connection to the ERP system takes these situations into account and has synchronized the stock updates via the API.
I'm going to discuss this with my colleagues again.

Hi @windaishi,
I imported your pull request and the tests are green except for the testStockIsUpdatedIfQuantityOfOrderLineItemIsChanged.
Line items that are already in a completed order will not be considered by the StockIndexer.

thanks for your contribution @windaishi :+1: :tada: :blue_heart:


Add package-2019-11 cost management in java readme

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

@ChenTanyi , can you have a look at the detected breaking change in python SDK automation?

@raych1 Is your CI error? I haven't changed anything about API, especially for python.

@ChenTanyi , this error is imported from python SDK generation. Is it expected change?
[ChangeLog] **Features**
[ChangeLog]
[ChangeLog] - Added operation QueryOperations.usage
[ChangeLog] - Added operation DimensionsOperations.list
[ChangeLog]
[ChangeLog] **Breaking changes**
[ChangeLog]
[ChangeLog] - Model QueryDataset no longer has parameter sorting
[ChangeLog] - Removed operation QueryOperations.usage_by_scope
[ChangeLog] - Removed operation DimensionsOperations.list_by_subscription

@raych1 You can see the changed files, I have do nothing about python sdk. If there is breaking change, please ask the person who change the API. I don't know anything about the python sdk automation.


adding send intent receiving

Now possible to send text (URLs, tweets etc) via "share" in other apps to matrix rooms
(dupe of messed up PR #8 )

added a "Share with:" title when coming from an Send-Intent.
I'm not convinced about setIntent(null), as this would make for a confusing UX.
Currently, you can send someone a link, hit back, select the next user and share the link again, as you basicly are still in the sharing-flow (hitting back again, would bring you back to the original app).

Currently, you can send someone a link, hit back, select the next user and share the link again, as you basicly are still in the sharing-flow

Seems reasonable.
Tested this; there's a few issues with this currently:

Sharing with Matrix from cold (app not open) dumps you into an empty home screen. I would expect this to be the splash screen then transfer you over to a non-empty home screen.
The share fork with the "share with" patch ( https://github.com/SkaveRat/matrix-android-sdk/tree/patch-sendintent ) isn't compiling (alpha/debug/layout/activity_home.xml:21: error: Error: No resource found that matches the given name (at 'text' with value '@string/share_title') and sure enough a grep isn't turning up share_title.

Hmm. not sure how to fix the coldboot problem. Looks like we need the new activity for sharing, which can detect a missig session, and log in first

Yeah, it would be a decent amount of work to fix that, but we should be handling it better than that. It may be preferable to just finish() the the Activity if it isn't ready and was launched from the SEND Intent?

Any further progress on this? If we can add in the finish() to avoid being dropped into a broken Home screen, I'm happy to merge this in.

sorry, didn't have much time.
the current code isn't working anymore, the text doesn't pre-fill the inpt anymore. Merging the current master seems to have broken it.
I'll check out the bug in the next days, and rebase the commits. It's a mess, currently

rebased. still doesn't work and can't seem to find the reason why. Text just doesn't get set.

Going to close this for now. I really would like this feature, but there are too many edge cases hanging around due to the SDK's current code structure (tied too heavily to the Activity). The structure of the SDK needs to realistically be fixed first before this can be added.


Error Reporter Does Not Like Line Breaks

I'm using the latest VS Code & latest Windows.
This is what the terminal reports (VS Code - Build task):

This is the reported problem:

Notice the vertical space. When I click on it, VS Code does not find the file.

@hediet what build of windows are you on? Run ver in terminal

I'm sorry, totally forgot about this.
C:\dev>ver

Microsoft Windows [Version 10.0.19043.985]

Are you still seeing this?

Unfortunately, yes!

This is the text from the terminal:
> Executing task: yarn run watch-clientd <

yarn run v1.22.10
$ deemon yarn watch-client$ node --max_old_space_size=4095 ./node_modules/gulp/bin/gulp.js watch-client
[09:25:39][09:25:39] Using gulpfile C:\dev\microsoft\vscode\gulpfile.js
[09:25:39] Starting 'watch-client'...
[09:25:39] Starting clean-out ...
[09:25:40][09:25:40] Finished clean-out after 819 ms[09:25:40] Starting watch-client ...
[09:25:43] Starting compilation...
ch-client'...
[09:25:39] Starting clean-out ...
[09:25:40] Finished clean-out after 819 ms
[09:25:40] Starting watch-client ...
[09:25:43] Starting compilation...
[09:27:13] Finished compilation with 0 errors after 89708 ms
[09:27:13] Starting compilation...
[09:27:14] Error: C:/dev/microsoft/vscode/src/vs/editor/contrib/hover/modesContentHover.ts(56,9): Property 'addAction' in type 'EditorHoverStatusBar' is not assignable to the same property in base type 'IEditorHoverStatusBar'.
  Type '(actionOptions: { labela: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }) => void' is not assignable to type '(actionOptions: { label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }) => void'.
    Types of parameters 'actionOptions' and 'actionOptions' are incompatible.
      Property 'labela' is missing in type '{ label: string; iconClass?: string |
undefined; run: (target: HTMLElement) => void; commandId: string; }' but required
in type '{ labela: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }'.
[09:27:14] Error: C:/dev/microsoft/vscode/src/vs/editor/contrib/hover/modesContentHover.ts(59,57): Argument of type '{ labela: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }' is not assignable to parameter of type '{ label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }'.
  Property 'label' is missing in type '{ labela: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }' but required in type '{ label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }'.
[09:27:14] Error: C:/dev/microsoft/vscode/src/vs/editor/contrib/hover/modesContentHover.ts(544,82): Argument of type 'EditorHoverStatusBar' is not assignable to parameter of type 'IEditorHoverStatusBar'.
  Types of property 'addAction' are incompatible.
    Type '(actionOptions: { labela: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }) => void' is not assignable to type '(actionOptions: { label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }) => void'.
[09:27:14] Finished compilation with 3 errors after 696 ms
[09:27:40] Starting compilation...
[09:27:42] Finished compilation with 0 errors after 1504 ms
[09:27:49] Starting compilation...
[09:27:49] Error: C:/dev/microsoft/vscode/src/vs/editor/contrib/hover/modesContentHover.ts(56,9): Property 'addAction' in type 'EditorHoverStatusBar' is not assignable to the same property in
base type 'IEditorHoverStatusBar'.
  Type '(actionOptions: { label: string; iconClass?: string | undefined; runa: (target: HTMLElement) => void; commandId: string; }) => void' is not assignable to type '(actionOptions: { label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string;
}) => void'.
    Types of parameters 'actionOptions' and 'actionOptions' are incompatible.
      Property 'runa' is missing in type '{ label: string; iconClass?: string | undefined; run:
(target: HTMLElement) => void; commandId: string; }' but required in type '{ label: string; iconClass?: string | undefined; runa: (target: HTMLElement) => void; commandId: string; }'.
ns: { label: string; iddActioatusB?: string | undefinor/contrib/hover/modesContentHover.ts(59,57conClass?: string | un       meter> void; commandIdsting | undefined; runa: (target: HTMLElementdefined; runa: (targetns: { lar'.                     parameter of type '{ label: string; iconCl: HTMLElement) => voids?: strn' ar/vscode/src/vs/edit=> void; commandId: string; }'.; commandId: string; } (targe     er.ts(544,82): Arme iconClass?: string | undefined; runa: (tar) => void' is not assid; commabel:ar' is not assign trequired in type '{ label: string; iconClasgnable to type '(actiooid' is undeHoverStatusBar'.    void; commandId: string; }'.nOptions: { label: stre '(actment)e incompatible.    or/contrib/hover/modesContentHover.ts(544,8ing; iconClass?: strintring; }) => string; iconClass?signable to parameter of type 'IEditorHoverg | undefined; run: (tndefineype 'get: HTMLElement)> arget: HTMLElement) =>lement)ng; i void' is not assbl void; commandId: stritring; ; run label: string; iss: string | undefined; runa: (target: HTMLElng; }) => void'.             id; c: (target: HTMLEl>  assignable to type '(actionOptions: { labe
[09:27:49] Finished compilati.     }) => void'.      et: HTMLElement) => void; commandId: stringmpilation with 3 error 552 mson with 3 errors after 5s after 552 ms                                       52 ms
[09:28:12] Starting compilation...
[09:28:12] Error: C:/dev/microsoft/vscode/src/vs/editor/contrib/hover/modesContentHover.ts(56,9): Property 'addAction' in type 'EditorHoverStatusBar' is not assignable to the same property in base type 'IEditorHoverStatusBar'.
  Type '(actionOptions: { label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandIda: string; })
=> void' is not assignable to type '(actionOptions: { label: string; iconClass?: string
| undefined; run: (target: HTMLElement) => void; commandId: string; }) => void'.
    Types of parameters 'actionOptions' and
'actionOptions' are incompatible.
      Property 'commandIda' is missing in type '{ label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }' but required in type
'{ label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandIda: string; }'.
[09:28:12] Error: C:/dev/microsoft/vscode/src/vs/editor/contrib/hover/modesContentHover.ts(57,77): Property 'commandId' does not exist on type '{ label: string; iconClass?: string | undefined; run:
(target: HTMLElement)
=> void; commandIda: string; }'. Did you mean 'commandIda'?
[09:28:12] Error: C:/dev/microsoft/vscode/src/vs/editor/contrib/hover/modesContentHover.ts(59,57): Argument of type '{ label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandIda: string; }' is not assignable to parameter of type
'{ label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }'.
  Property 'commandId' is missing in type '{ label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandIda: string; }' but required in type '{ label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }'.
[09:28:12] Error: C:/dev/microsoft/vscode/src/vs/editor/contrib/hover/modesContentHover.ts(544,82): Argument of type 'EditorHoverStatusBar' is not assignable to parameter of type 'IEditorHoverStatusBar'.
  Types of property 'addAction' are incompatible.
    Type '(actionOptions: { label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandIda: string; }) => void' is not assignable to type '(actionOptions: { label: string; iconClass?: string | undefined; run: (target: HTMLElement) => void; commandId: string; }) => void'.
[09:28:12] Finished compilation with 4 errors after 578 ms

This is how it is rendered:

The likelyhood of this happening increases when decreasing the width of the terminal.

There's probably not much we can do about this, I believe you're hitting the limits of our line wrapping heuristic where since old conpty doesn't support line wrapping we assume a link is wrapped if the last character in the row is non-whitespace. Reducing the width of the terminal will increase the chance of a line-ending space which results in a failure. @meganrogge has worked on the fix for this but you'll need to wait until Windows build 21376 https://github.com/microsoft/vscode/issues/91898


[enh] add version to json_serialzable

no longer relevant, as json_serializable has been moved to siibra-api


Update amis for new 1.13 (and 1.12, 1.11) k8s default

Update eks amis
Issue:
https://github.com/rancher/rancher/issues/21992

LGTM, AMIs and regions match the list from AMZ

lgtm


ModuleNotFoundError: No module named 'sklearn'

Hi, after Git Pull command or reinstallation I get this error.
Installing additional components with the command “pip install scikit-learn” helped me.
ModuleNotFoundError: No module named 'sklearn'
Traceback:
File "/root/miniconda3/envs/simplify/lib/python3.9/site-packages/streamlit/runtime/scriptrunner/exec_code.py", line 85, in exec_func_with_error_handling
result = func()
File "/root/miniconda3/envs/simplify/lib/python3.9/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 576, in code_to_exec
exec(code, module.dict)
File "/root/simply-simplify-language/_streamlit_app/sprache-vereinfachen.py", line 33, in
from utils_understandability import get_zix, get_cefr
File "/root/simply-simplify-language/_streamlit_app/utils_understandability.py", line 42, in
scaler = load(f)

Thanks for raising this issue.
I added scikit-learn to requirements.txt.
This should solve the issue.


Replace scrollBar adjustments

Hi all,
in a future time we might consider replacing JS hacks for modal to adjust scrollBar behaviour by native CSS https://wicg.github.io/overscroll-behavior/
This is just for the records now, as support isn't widely supported (yet).
more info: https://developers.google.com/web/updates/2017/11/overscroll-behavior
Cheers
midzer

If that's the solution to be adopted by the original library developers, we will implement it too, we don't want to be left outside and frankly it sounds like lesser code for our modal component.

Yeah, true.
Actually, I forgot this property has to be set in CSS in the first place. Nevermind :)


Update authentication.rst, include quotes

There was no issue (number) for this. Only includes documentation.

Thanks! All updates, no matter how trivial they may seem are appreciated.


Build fails on both luarocks and make

Hi, when I try to update the library with luarocks I get this error (output of sudo luarocks install lua-mbedtls):
Installing https://luarocks.org/lua-mbedtls-0.2.0-1.src.rock

lua-mbedtls 0.2.0-1 depends on lua >= 5.1 (5.1-1 provided by VM)
gcc -O2 -fPIC -I/usr/local/include -c src/base64.c -o src/base64.o -I/usr/local/include
gcc -O2 -fPIC -I/usr/local/include -c src/main.c -o src/main.o -I/usr/local/include
gcc -O2 -fPIC -I/usr/local/include -c src/md.c -o src/md.o -I/usr/local/include
gcc -O2 -fPIC -I/usr/local/include -c src/ssl.c -o src/ssl.o -I/usr/local/include
src/ssl.c:31:10: fatal error: mbedtls/certs.h: No such file or directory
   31 | #include <mbedtls/certs.h>
      |          ^~~~~~~~~~~~~~~~~
compilation terminated.

Error: Build error: Failed compiling object src/ssl.o

I also tried to build it with CMake & make but got similar issues. libmbedtls-dev is installed on my system but ls /usr/local/include/mbedtls | grep certs returns nothing.

I assume you are on macOS and have installed mbedtls 3.0 via Homebrew. The problem is that Mbed TLS 3.0 is immature and its API is not backward compatible with 2.x. With time, I'll add support for it
Also, due to this issue - https://github.com/ARMmbed/mbedtls/issues/5184#issuecomment-986429214 - it can't be used with lua-mbedtls as of today.
Your best bet is to downgrade to Mbed TLS 2.x.

I'm actually on Ubuntu 20.04 and installed mbedTLS via building from source. As you've claimed, downgrading mbedTLS to 2.28.0 and rebuilding worked, thanks! Version 3.1.0 seems to have breaking changes for the library.

As a side note, it looks like Ubuntu 20.04 has an official package named libmbedtls-dev_2.16.4. You might want to try to install that instead of building from source.

FYI: dd6c2e6a added support for MbedTLS 3.x


yolox_nano.pth transforms yolox_nano.onnx file reporting an error ：Missing key(s) & Unexpected key(s) in state_dict... (Models trained by my own data sets)

I recently trained the yolox_nano. pth model using my own dataset (only class 1)
The way I trained went like this:
https://blog.csdn.net/ELSA001/article/details/120918082?spm=1001.2014.3001.5501
And I am trying to convert this yolox_nano. pth to yolox_nano.onnx,but many errors were reported:
Convert command (in YOLOX directory) :
python tools/export_onnx.py -n yolox-nano -c weights/yolox_nano.pth --output-name weights/yolox_nano.onnx
Error message:
RuntimeError: Error(s) in loading state_dict for DataParallel:
 Missing key(s) in state_dict: ....
Unexpected key(s) in state_dict...

The error picture is as follows:

I think the reason for the error is that I am converting my own data set, but when I convert coco data set, there is no error and it can be converted normally:

But I've changed num_classes, depth, width, and input_size in yolox_base.py:

Let's report this error
The complete error message is as follows:
(torch_G) E:\YOLOX>python tools/export_onnx.py -n yolox-nano -c weights/yolox_nano.pth --output-name weights/yolox_nano.onnx
2021-10-24 17:15:37.712 | INFO     | __main__:main:59 - args value: Namespace(batch_size=1, ckpt='weights/yolox_nano.pth', dynamic=False, exp_file=None, experiment_name=None, input='images', name='yolox-nano', no_onnxsim=False, opset=11, opts=[], output='output', output_name='weights/yolox_nano.onnx')
2021-10-24 17:15:40.278 | ERROR    | __main__:<module>:119 - An error has been caught in function '<module>', process 'MainProcess' (66764), thread 'MainThread' (59796):
Traceback (most recent call last):
> File "tools\export_onnx.py", line 119, in <module>
    main()
    └ <function main at 0x00000276616C75E0>
  File "tools\export_onnx.py", line 81, in main
    model.load_state_dict(ckpt)
    │     │               └ OrderedDict([('backbone.backbone.stem.conv.conv.weight', tensor([[[[ 1.2789e-02,  1.7050e-02,  2.4743e-02],
    │     │                           [ 6.535...
    │     └ <function Module.load_state_dict at 0x00000276609BEAF0>
    └ DataParallel(
        (module): YOLOX(
          (backbone): YOLOPAFPN(
            (backbone): CSPDarknet(
              (stem): Focus(
                (c...
  File "E:\Anaconda3\envs\torch_G\lib\site-packages\torch\nn\modules\module.py", line 1223, in load_state_dict
    raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for DataParallel:
        Missing key(s) in state_dict: "module.backbone.backbone.stem.conv.conv.weight", "module.backbone.backbone.stem.conv.bn.weight", "module.backbone.backbone.stem.conv.bn.bias"......
        Unexpected key(s) in state_dict: "backbone.backbone.stem.conv.conv.weight", "backbone.backbone.stem.conv.bn.weight", "backbone.backbone.stem.conv.bn.bias"....

I think it's a network problem, some extra or missing parameter, causing the mismatch.
My trained yolox_nano. pth is fine and can be predicted normally (reasoning) :

That feels very strange, I hope there can have this experience of the students can help me answer, thank you.
Original problem address:
https://ask.csdn.net/questions/7543357?spm=1001.2014.3001.5501

Have you solve this problem? I have met a similar one but don't konw how to deal with it.
RuntimeError: Error(s) in loading state_dict for YOLOX:
Missing key(s) in state_dict: "backbone.backbone.stem.conv.conv.weight", "backbone.backbone.stem.conv.bn.weight", "backbone.backbone.stem.conv.bn.bias", "backbone.backbone.stem.conv.bn.running_mean", "backbone.backbone.stem.conv.bn.running_var".
Unexpected key(s) in state_dict: "backbone.backbone.stem.dconv.conv.weight", "backbone.backbone.stem.dconv.bn.weight", "backbone.backbone.stem.dconv.bn.bias", "backbone.backbone.stem.dconv.bn.running_mean", "backbone.backbone.stem.dconv.bn.running_var", "backbone.backbone.stem.dconv.bn.num_batches_tracked", "backbone.backbone.stem.pconv.conv.weight", "backbone.backbone.stem.pconv.bn.weight", "backbone.backbone.stem.pconv.bn.bias", "backbone.backbone.stem.pconv.bn.running_mean", "backbone.backbone.stem.pconv.bn.running_var", "backbone.backbone.stem.pconv.bn.num_batches_tracked".

https://blog.csdn.net/ELSA001/article/details/121032193?spm=1001.2014.3001.5502

https://blog.csdn.net/ELSA001/article/details/121032193?spm=1001.2014.3001.5502
You can look this blog,following the training steps,maybe can solve the problem.

Have you solve this problem? I have met a similar one but don't konw how to deal with it.
RuntimeError: Error(s) in loading state_dict for YOLOX: Missing key(s) in state_dict: "backbone.backbone.stem.conv.conv.weight", "backbone.backbone.stem.conv.bn.weight", "backbone.backbone.stem.conv.bn.bias", "backbone.backbone.stem.conv.bn.running_mean", "backbone.backbone.stem.conv.bn.running_var". Unexpected key(s) in state_dict: "backbone.backbone.stem.dconv.conv.weight", "backbone.backbone.stem.dconv.bn.weight", "backbone.backbone.stem.dconv.bn.bias", "backbone.backbone.stem.dconv.bn.running_mean", "backbone.backbone.stem.dconv.bn.running_var", "backbone.backbone.stem.dconv.bn.num_batches_tracked", "backbone.backbone.stem.pconv.conv.weight", "backbone.backbone.stem.pconv.bn.weight", "backbone.backbone.stem.pconv.bn.bias", "backbone.backbone.stem.pconv.bn.running_mean", "backbone.backbone.stem.pconv.bn.running_var", "backbone.backbone.stem.pconv.bn.num_batches_tracked".

https://blog.csdn.net/ELSA001/article/details/121032193?spm=1001.2014.3001.5502
You can look this blog,following the training steps,maybe can solve the problem.


PR: All None valued variables (or lists containing None) should be displayed by default

Fixes #4615

@andfoy, please add a tests for this (it could be the list present in #4615).


feat(MOPLAT-353): Show availability of artworks

Description

This PR resolves MOPLAT-353
https://user-images.githubusercontent.com/36475005/185917363-24da6347-1021-4b77-85fa-59efd69def07.mov
PR Checklist

[ ] I tested my changes on iOS and Android.
[x] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Need help with something? Have a look at our docs, or get in touch with us.

Good work.
I just added a few non-blocking comments/suggestions.

@rajsam003 thank you for reviewing, I've added your suggestions, feel free to merge if it's ok :)


Compiling from source x86/x64 issue

I usually don't program desktop applications, but I'm motivated to add some features to this code for a project I'm working on. After a lot of struggle, I've gotten to this point. It appears that during build, it is trying to build as x86 which isn't compatible with some packages, any ideas to what I should do to fix the build? My system itself is Windows 10 x64
ERROR: There are invalid packages (packages that cannot exist for this configuration):
cmake/3.29.3: Invalid ID: CMake binaries are only provided for x86_64 and armv8 architectures
msys2/cci.latest: Invalid ID: Only Windows x64 supported
msys2/cci.latest: Invalid ID: Only Windows x64 supported
    opengl/system:5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9 - Download
    openssl/3.2.1:a749de233b8caeb48dce50def9bb26c33c85e93c - Build
    re2/20231101:b8d2459e16d2c60181448ba7a62446c87c96e0d3 - Build
    spdlog/1.14.1:99cce058efbde75e361654cb489434be1fb2c648 - Build
    sqlite3/3.46.0:aa87a14ad1c7a800a99186d569fd58b2e0156222 - Build
    vulkan-headers/<IP_ADDRESS>:5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9 - Download
    vulkan-loader/<IP_ADDRESS>:556aefa3805608dd7c5cd34ba1a20542bbdff93d - Build
    zlib/1.3.1:70711aff723b218b02483e600038031bf91007bc - Build
Build requirements
    b2/4.10.1 from 'conancenter' - Cache
    cmake/3.29.3 from 'conancenter' - Cache
    gperf/3.1 from 'conancenter' - Cache
    meson/1.2.2 from 'conancenter' - Cache
    meson/1.3.2 from 'conancenter' - Cache
    meson/1.4.0 from 'conancenter' - Cache
    msys2/cci.latest from 'conancenter' - Cache
    nasm/2.15.05 from 'conancenter' - Cache
    ninja/1.11.1 from 'conancenter' - Cache
    pkgconf/2.1.0 from 'conancenter' - Cache
    strawberryperl/<IP_ADDRESS> from 'conancenter' - Cache
Build requirements packages
    b2/4.10.1:11e6a84a7894f41df553e7c92534c3bf26896802 - Build
    cmake/3.29.3:INVALID - Invalid
    gperf/3.1:63c5d5dc6eb958dab897e143053e17f583d55879 - Build
    meson/1.2.2:5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9 - Download
    meson/1.3.2:5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9 - Download
    meson/1.4.0:5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9 - Download
    msys2/cci.latest:INVALID - Invalid
    nasm/2.15.05:63c5d5dc6eb958dab897e143053e17f583d55879 - Build
    ninja/1.11.1:63c5d5dc6eb958dab897e143053e17f583d55879 - Build
    pkgconf/2.1.0:63c5d5dc6eb958dab897e143053e17f583d55879 - Build
    strawberryperl/<IP_ADDRESS>:11e6a84a7894f41df553e7c92534c3bf26896802 - Build

Cross-build from 'Windows:x86_64' to 'Windows:x86'
Installing (downloading, building) binaries...
CMake Error at external/cmake-conan/conan.cmake:651 (message):
  Conan install failed='6'
Call Stack (most recent call first):
  CMakeLists.txt:35 (conan_cmake_install)

This actually may be a bug with Conan. It seems to be detecting my system as x86 even though it is 64bit
C:\Users\tdogb\Documents\VSCode\weather\supercell-wx>cmake -B build-release -S .     -DCMAKE_BUILD_TYPE=Release     -DCMAKE_CONFIGURATION_TYPES=Release     -DCMAKE_PREFIX_PATH=C:/Qt/6.7.1/msvc2019_64
-- Conan: Detected VS runtime: MD
-- Conan: checking conan executable
-- Conan: Found program C:/Program Files/Conan/conan/conan.exe
-- Conan: Version found Conan version 1.64.1
-- Conan executing: C:/Program Files/Conan/conan/conan.exe install C:/Users/tdogb/Documents/VSCode/weather/supercell-wx --remote conancenter --build missing --settings arch=x86 --settings build_type=Release --settings compiler=Visual Studio --settings compiler.version=17 --settings compiler.runtime=MD
Configuration:
[settings]
arch=x86
arch_build=x86_64
build_type=Release
compiler=Visual Studio
compiler.runtime=MD
compiler.version=17
os=Windows
os_build=Windows
[options]
[build_requires]
[env]

32-bit is not supported. What version of Visual Studio are you running? I'm assuming you have a 64-bit compiler installed.
You will want to edit your Conan default profile, changing arch from x86 to x86_64. You can edit this in the .conan folder in your home directory, or with the conan profile command.

32-bit is not supported. What version of Visual Studio are you running? I'm assuming you have a 64-bit compiler installed.
You will want to edit your Conan default profile, changing arch from x86 to x86_64. You can edit this in the .conan folder in your home directory, or with the conan profile command.

My conan profile is set up as follows
[settings]
os=Windows
os_build=Windows
arch=x86_64
arch_build=x86_64
compiler=Visual Studio
compiler.version=17
build_type=Release
[options]
[build_requires]
[env]

My system is 64bit Windows 10 with Visual Studio 2022

C:\Users\tdogb\Documents\VSCode\weather\supercell-wx>cmake -B build-release -S .     -DCMAKE_BUILD_TYPE=Release     -DCMAKE_CONFIGURATION_TYPES=Release     -DCMAKE_PREFIX_PATH=C:/Qt/6.7.1/msvc2019_64
-- Conan: Detected VS runtime: MD
-- Conan: checking conan executable
-- Conan: Found program C:/Program Files/Conan/conan/conan.exe
-- Conan: Version found Conan version 1.64.1
-- Conan executing: C:/Program Files/Conan/conan/conan.exe install C:/Users/tdogb/Documents/VSCode/weather/supercell-wx --remote conancenter --build missing --settings arch=x86 --settings build_type=Release --settings compiler=Visual Studio --settings compiler.version=17 --settings compiler.runtime=MD
Configuration:

Note --settings arch=x86.  Something is overriding arch=x86_64 in your profile.  When I run, I get the following:
[proc] Executing command: "C:\Program Files\CMake\bin\cmake.EXE" --build D:/Projects/build-supercell-wx --config Release --target supercell-wx --
[build] CMake is re-running because D:/Projects/build-supercell-wx/CMakeFiles/generate.stamp is out-of-date.
[build]   the file 'C:/Program Files/CMake/share/cmake-3.25/Modules/CMakeCCompiler.cmake.in'
[build]   is newer than 'D:/Projects/build-supercell-wx/CMakeFiles/generate.stamp.depend'
[build]   result='0'
[build] -- Conan: Detected VS runtime: MD
[build] -- Conan: checking conan executable
[build] -- Conan: Found program C:/Users/dpaul/AppData/Local/Programs/Python/Python312/Scripts/conan.exe
[build] -- Conan: Version found Conan version 1.64.1
[build] -- Conan executing: C:/Users/dpaul/AppData/Local/Programs/Python/Python312/Scripts/conan.exe install C:/Users/dpaul/git/supercell-wx --remote conancenter --build missing --settings arch=x86_64 --settings build_type=Release --settings compiler=Visual Studio --settings compiler.version=17 --settings compiler.runtime=MD --settings compiler.cppstd=17

CMake might require a -A x86_64 argument to force the architecture?

C:\Users\tdogb\Documents\VSCode\weather\supercell-wx>cmake -B build-debug -S .     -A x86_64     -DCMAKE_BUILD_TYPE=Debug     -DCMAKE_CONFIGURATION_TYPES=Debug     -DCMAKE_PREFIX_PATH=C:/Qt/6.7.1/msvc2019_64
CMake Error at CMakeLists.txt:3 (project):
  Generator

    NMake Makefiles

  does not support platform specification, but platform

    x86_64

  was specified.

CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!

Spent hours fixing this error before, but don't remember how I did it. Feeling like one of those endless rabbit holes to get this compiled haha

I have not used the NMake Makefiles generator, and not sure if it'd work on Windows.  I'd recommend the Ninja generator since you're using VS Code, otherwise the Visual Studio generator should work OK.
You can also try running the setup-*.bat script to initialize your build directory, and configure VS Code to reference your already configured/generated directory (CMake plugin settings).

Assuming this is resolved.


How to get  loss on validation via tool/test.py?

Can I modify mmdetection/mmdet/apis/test.py
by
result = model(return_loss=False, rescale=not show, **data)
as
result = model(return_loss=True, rescale=not show, **data)

@hellock because I can't validate in non-distribute mode so I use tool/test.py for validating every epoch. I need to plot loss curve on both training and validation set.

The latest version should already supported validating during training for both non-distributed and distributed mode.


Feature request - Is there potential to support bgzip?

This is a clever solution to get the decompression / compression into a separate process, very helpful little tool. This pattern would work well for a couple additional compression tools that my group uses. I'm going to take a look at adding support for these formats, which should be pretty straightforward.
BGZF, or "blocked gzip" is a format that's used pretty widely in bioinformatics, it's basically a lot of gzipped files concatenated together, with some extra info in the headers and an index in a separate file saying where to seek. It's decompressible by normal gzip, so we actually see bgzf files as .gz more often than .bgz. It'd be really great to be able to compress bgz files with xopen as well. The blocked gzip reference implementation is distributed with htslib as a binary called bgzip, and is available both from conda and most linux distros native packages (tabix on Ubuntu, for example).
Also, it'd be great to see this support zstd as well, which is just an excellent general purpose compression tool that I expect to rapidly grow in usage in the next few years.
Edit: To be clear, both of these tools are already usable from Python, there's a bgzip implementation here, and zstd has excellent Python bindings available, but getting the compression into another process like xopen does makes for much better performance.

There’s a draft pull request for zstd support in #91, please also see the discussion over there.
Regarding bgzip support, I think compression support is fine.
Regarding decompression (which I realize you didn’t ask for), I think the support we already have for gzip is sufficient (we have tests that ensure we can decompress concatenated gzip files). I would argue that exposing the seeking capabilities of bgzip wouldn’t be in scope for xopen because the idea is to provide an interface that lets one abstract away the used compression algorithm.

I would like to add to that, that BGZF compression is only advantageous when the input data is aligned properly with the BGZF blocks. In the case that you not care for that alignment, it is faster to compress with normal gzip compression. It is impossible for xopen to know how the input data should be  aligned. (The interface handles pure bytes objects, not HTS records).
If you know beforehand that you do want BGZIP, there is also no need for xopen's dynamic choosing of compression algorithm. So you might be better of using subprocess and bgzip that way.
As regards to:

Python bindings available, but getting the compression into another process like xopen does makes for much better performance.

Pysam has a threads option when opening its files. That will already probably give more performance, since it does not have to manage open processes like xopen does. It also has the advantage of knowing exactly how the output blocks should be aligned, so the resulting compressed files can be indexed.

For our specific usage case that I want this right now, it doesn't seem like Pysam has the interface for what we're wanting to do: bgzip output to create a .vcf.bgz file and associated tabix index, because downstream tools are expecting bgzipped vcfs with indexes.

Pysam has a threads option when opening its files. That will already probably give more performance, since it does not have to manage open processes like xopen does. It also has the advantage of knowing exactly how the output blocks should be aligned, so the resulting compressed files can be indexed.

I'm going to do some quick performance testing to see how using threads inside of the main Python process does vs opening another process and piping uncompressed data to it like xopen does. My gut would actually say that xopen's approach is faster, just because in the 1st approach, the process will either be copying data in from Python or compressing it in the pysam threadpool, but not both at the same time because of the GIL, while xopen allows Python to send data via pipe without blocking unless the pipe fills up.
I'm taking zstd support discussion to that open PR.

Have you tried using PipedCompressionWriter with program_args = ["bgzip", "-i"]. Wouldn't that already solve your problem?

That looks like it would, I'll validate this.
Would the xopen project be willing to make that support native, so that if you open a .bgz for writing, or a .gz and specify a bgzf or bgz format it will call bgzip?

For xz, bz2 and `gz`` xopen can fall back on modules from Python's standard library if command line applications are not present. There is no such thing for bgzip. Which means xopen can't guarantee that it can open bgzip files. Unless we would add a dependency that can read bgzip from python. At that point I think it is already quite a lot of work to get some support going and we can only support the features that bgzip has in common with the other formats. (so indexing can not be supported). The reason we can not do this, is that it would make the API more complicated with some features that only apply to bgzip. That would be a loss for the project, in my opinion.
Also the whole problem xopen tries to solve is that you don't know what output the user expects: stdout? gzip? uncompressed? Xopen infers that for you. If you already know what format you want (indexed bgzip in your case), then it is much less overhead to simply let the program output that format, rather than rely on xopen.
@marcelm What do you think?

@pettyalex Is it correct that you want an indexed bgz file, that is, that you would want to run bgzip with -i?
For me, the question is whether we can add support for bgzip without adding another parameter to the xopen() function that only applies to bgzip. As Ruben said, we shouldn’t make the API more complicated.
The easiest case is that we just never create an index (run bgzip without -i).
Always generating an index is the other option, but I wonder whether we wouldn’t get requests to enable setting the index name (bgzip option -I).
I’m not too worried about bgzip not being available in the standard library. If bgzip isn’t installed, we just print an error and it’s the responsibility of whoever uses xopen to ensure that bgzip is in their list of dependencies.
Whatever we end up doing, wo should at least document the PipedCompressionWriter and PipedCompressionReader classes better.

Is it correct that you want an indexed bgz file, that is, that you would want to run bgzip with -i?

Yes, absolutely. For our workflows, we need indexes everywhere that we'd be using bgzip, as bgzip itself performs worse both speed and compression-wise vs other options. The index is its only advantage. The default index name would also be appropriate in every situation we'd want to use this with.

Yes, absolutely. For our workflows, we need indexes everywhere that we'd be using bgzip, as bgzip itself performs worse both speed and compression-wise vs other options. The index is its only advantage. The default index name would also be appropriate in every situation we'd want to use this with.

Do you use with the threading option? bgzip -@ 4 .... Also if you compile bgzip, you need to make sure you have libdeflate installed as it is much faster than standard zlib.
Some old zlib benchmarks by HTSlib (bgzip people): http://www.htslib.org/benchmarks/zlib.html
BGZF can be used without index. BCF2FASTQ uses it to compress FASTQ files, where multiple chunks can be compressed at the same time in different threads.


Updating Beginner Documentation

Purpose

Hey, the purpose of these changes is to add additional documentation to assist newcomers more easily in launching and interacting with streampipes in the terminal interface. As someone who recently cloned the project, I noticed a few errors as well as key information that could be added to the documentation for easier implementation.
Note
I was struggling to update the local build, so I was unable to view my local changes. Please let me know if I have edited the files correctly and if this pull request will update the website. Additionally, I believe that this pull request does not update the README.md on the main repo. Please let me know if I should create a pull request there to reflect these changes.

thanks a lot for these improvements @lab596 🙏🏼
In case you want to update the README file in the main repo as well, you need to open there a PR as well 🙂


Tracing is not displayed or stored well

There are 3 variants:

The order of the exchanges is wrong
The only displayed component is the Inbound HTTP component (Instance: next, Tenant: Regression Tests, Flow: Velocity, Postman Test: Velocity headers)
There is no tracing at all (for example in case of the Base64 component option unmarshall), no distinction in transport type

Test findings:

Most of the times, but not always, the individual exchanges are visible in the transactions now, except base64 marshall for example. Looks like it happens in case of transport type queues
The order of exchanges is not always chronological
The list of exchanges is getting longer after some tests

Sometimes the tracing is not always in the right order. Norman didn't find a pattern yet, but will look into to get a reproducible thing.

It's not exactly reproducible how this hapens, but the order of exchanges differs per call. You can test in the flow Tracing01 on next (tenant Microscope), by sending a message body to this url: https://next.dovetail.world/test/inbound_http/533057/Tracing01

The order of the transactions' exchanges looks fine now.
What misses is the message body in every individual exchange. Only the last message body is shown.


[Podman] Disable the autoupdate service/zincati for FCOS

By default zincati service is running which responsible for auto update of the FCOS host and it always polling the Cincinnati server, better to disable this service by default.
# rpm-ostree status
State: idle
AutomaticUpdatesDriver: Zincati
  DriverState: active; periodically polling for updates (last checked Wed 2022-01-05 10:47:33 UTC)
Deployments:
[...]

This is part of the bundle.


[Bug]Invalid url error when adding links to all configured sites

Describe the bug
When adding links to all configured websites, an error message "Invalid URI" appears. It seems to be after adding the GOG link, maybe there is a problem with the sites, that don't have an "add" option?
To Reproduce
Steps to reproduce the behavior:

Click on Add links to all configured websites.

Expected behavior
No error message should appear and the rest of the links should be added
please complete the following information:

Windows version: 11
Playnite version: 10.3
Extension version: 0.6

fixed in 0.6.1


v3 Playground: Presentation Mode Plugin

Build as a separate repo, to enforce the separation of concerns and ensure that the plugin infra (which is already in and used for all the current sidebar tabs) works well with external apps.
Need to add a way to declare that you want to use a custom plugin into the options section (and I'd probably add this one in the list, off by default)
Enabling it would add a new sidebar menu: "Present" which offers a bunch of settings. The key one being either a link to a markdown file (or a markdown gist?)
You write a markdown file a bit like like this file: https://gist.github.com/orta/d7dbd4cdb8d1f99c52871fb15db620bc
Which takes a lot of ideas from https://www.deckset.com
For example:

A slide with just an image centers it

A slide with a H2 and some Ps

A slide with H2s and just LIs

Then when the slide is just code, that code is set into monaco, and you're good to go
Slides styles I always use and would need syntax for:
Showinging blocks of info:

Highlighting a screenshot, with a title:

Done: #204


Pose3SLAM benchmark nondeterministic

I noticed that the Pose3SLAM benchmark gives me different final losses every time that I run it. As far as I know, all the computations should be deterministic.
We should get to the bottom of this because it might indicate a bug. For example, the generic factor graph datastructure uses unsafe memory operations, and we might be reading uninitialized memory somewhere.

I agree on that, and I observed that for some difficult datasets, the nonlinear loss sometimes go to nan which indicates possibly a division-by-zero or something. I suspect a impl error in our Pose3 but I am not super sure. Is there a way to raise an exception when we have nan?

Is there a way to raise an exception when we have nan?

I don't think there's an automatic way to do this.
Manually adding checks to the initializers of all our math types might be a good approach though. It seems like this would catch problems close to where they occur. The check should probably use a function that we define that is controlled by an #if flag -- assert isn't suitable because it gets disabled in -O mode and I thing we'll want to be able to debug nans in -O mode.


Create a pool

Description
Fill in later. Create a pool DRAFT
Fixes # (issue)
Type of change

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

How should this be tested?
Please provide instructions so we can test. Please also list any relevant details for your test configuration.

[ ] Test A
[ ] Test B

Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code where relevant, particularly in hard-to-understand areas
[ ] My changes generate no new console warnings
[ ] The base of this PR is master if hotfix, develop if not

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1201247603968393

To learn more about using Asana, visit asana.com to get started today.

The gray's of the circles need to be the same as the gray's of the connecting bars. Also think the circle gray's are too dark.

There's not enough contrast between label colors here and the spacing is too small so it's very difficult to parse.

Make the vertical spacing between the headers and the tokens bigger
Make the vertical spacing between the tokens and the total row bigger
Make the column headers a subtle gray
I think we need to rename the title of this card to something much more concise but I don't have an idea yet

When I got to 'Set initial liquidity' and clicked cancel I ended up here and everything seems messed up.

One for @pkattera maybe. Do we really need both these modules? Are they not giving the user very similar information?

I shouldn't be able to select ETH or the same token twice.

In each step it should be possible to go back to the previous step. Maybe by clicking on the vertical progress points as well as a back button.

We need to find a better way to allow the user to input optimised amounts. At the moment you either have to go with the absolute max proportional or you have to guess, and if you guess there's no way to automatically re-optimise. Not sure what the designs suggest already but maybe @pkattera could help with this.
@1saf we need a function that takes the amount from the input the user types in and returns the proportional values for the other tokens. Similar to how propAmountsGiven works, but it should be simpler to implement without balances. Once we have this function, we can perhaps just add a toggle to the top of the form that, if toggled on, forces inputs to be proportional, i.e. if you update one the others change regardless if you have the balance or not.


(theme) Provide Stock Images

Provides a number of stock images that may be used. Some photos are
already optimized for social sharing.

@designxsteph I think this is a great idea!


bufio: Writer.ReadFrom ignores the stored error

If an error occurs writing to a Writer, no more data will be accepted and all subsequent writes, and Flush, will return the error.

type writer struct{}

func (w writer) Write(p []byte) (n int, err error) {
	return 0, errors.New("bad move")
}

func main() {
	var wr = bufio.NewWriter(writer{})
	wr.WriteString("test1") // 5 <nil>
	wr.Flush()              // bad move
	wr.ReadFrom(strings.NewReader("test2")) // 5 <nil>
	wr.Buffered() // 10
}

https://play.golang.org/p/--WBbDHbIS9

no more data will be accepted

Then I would expect ReadFrom to fail and the Buffered() count not to grow.

all subsequent writes [...] will return the error.

Isn't ReadFrom a write?
This seems unlikely to have bad consequences, but I don't think it's a valid reason for the documentation to be incorrect about corner cases.

Is it so?

This definitely looks like a bug. It also calls the underlying ReadFrom without persisting the error.

Will it be backported to 1.13 and 1.12 ?

cc @bradfitz @bcmills

Will it be backported to 1.13 and 1.12 ?

If it's not a regression from 1.11 and not a security issue, then a backport seems doubtful. (Do we know whether it's a regression from 1.11?)


charts might not discoverable at first

should chart be visible at start so user understand that they can do something

After usability think this is okay.  but may need to open on default


Feature/ted4 10

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

Comparison is base (6c12c06) 94.77% compared to head (7440de7) 94.79%.
Report is 2 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #503      +/-   ##
==========================================
+ Coverage   94.77%   94.79%   +0.02%
==========================================
  Files         115      117       +2
  Lines        6120     6283     +163
==========================================
+ Hits         5800     5956     +156
- Misses        320      327       +7

Files
Coverage Δ

ted_sws/core/model/supra_notice.py
100.00% <100.00%> (ø)

...ager/adapters/daily_notices_metadata_repository.py
100.00% <100.00%> (ø)

ted_sws/data_manager/adapters/repository_abc.py
100.00% <100.00%> (ø)

...anager/services/daily_notices_metadata_services.py
91.02% <91.02%> (ø)

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