Tags or auto sync optional

Hi,
When I'm importing new games I'm using a dedicated saved filter to see games that have no tags, categories and completion statuses - this is a 'new' game in my Playnite (i dont sync any of these when doing metadata download, i use my own tags etc.)
With this plugin, games are automatically synced and tagged, which messes up with my 'new' saved filter.
Is it possible to add a configuration option to skip adding tags? Or, if tags are used internally, can there be an option to not sync automatically?

Yes to both.
On my planned list of features.

Should be solved with v2.1.0. Now:

tags are optional
automatic syncing of newly added games is optional
Let me know, if it works for you :)

Yea, working great for me, thank :)


Add prettier configuration

I've added prettier support (and a script to package.json) which formats the files in the project.
This works mostly fine, but it treats the files as md files and doesn't understand some of the vuepress specific stuff (e.g. tabs).
However, I didn't find any vue specific feature which would be broken by the formatting (you mostly need to add newlines somewhere).
Even if there was some section that should not be formatted, you can exclude it from prettier: https://prettier.io/docs/en/ignore.html
I didn't format anything just yet not to create any conflicts for you. We can reformat the stuff gradually. (This also means you shouldn't run the format script, because that tries to format all files in the project).

We should take time to go over this and decide how to use it.


Explicit SSH Port Forwarding

For the Oracle Linux Vagrantfile, SSH port forwarding isn't necessary as this is done automatically and dynamically by Vagrant. Also, Vagrant boxes for Oracle Linux have SSH password authentication turned off by default. The only way to SSH into the VM would be using the command vagrant ssh. It might also be worthwhile updating the README.

Thanks @fuzziebrain! @scoter-oracle, does that sound good for you, removing the ssh port forwarding?

Yep, I agree. I've commented it out. Thanks

Thanks guys! The README might require updating too. The only way to SSH into the VM out-of-the-box is to use vagrant ssh.

Thank you again @scoter-oracle !


Druid console: ArithmeticException in Datasources tab

Underlying query fails with ArithmeticException when opening Datasources tab in Druid console.
Affected Version
0.20.0
Description
The error message looks like this:

The underlying query looks like this:
SELECT
  datasource,
  COUNT(*) FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1) AS num_segments,
  COUNT(*) FILTER (WHERE is_available = 1 AND ((is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1)) AS num_available_segments,
  COUNT(*) FILTER (WHERE is_published = 1 AND is_overshadowed = 0 AND is_available = 0) AS num_segments_to_load,
  COUNT(*) FILTER (WHERE is_available = 1 AND NOT ((is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1)) AS num_segments_to_drop,
  SUM("size") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0)) AS total_data_size,
  SUM("size" * "num_replicas") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0)) AS replicated_size,
  MIN("size") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0)) AS min_segment_size,
  (
    SUM("size") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0)) /
    COUNT(*) FILTER (WHERE (is_published = 1 AND is_overshadowed = 0))
  ) AS avg_segment_size,
  MAX("size") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0)) AS max_segment_size,
  SUM("num_rows") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0) OR is_realtime = 1) AS total_rows,
  (
    SUM("size") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0)) /
    SUM("num_rows") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0))
  ) AS avg_row_size
FROM sys.segments
GROUP BY 1

and it fails because a divider (in my case SUM("num_rows") FILTER (WHERE (is_published = 1 AND is_overshadowed = 0))) is 0.
Here is what I see when I remove 2 problematic metrics from the query:

Fixed by  #10496 . See https://github.com/apache/druid/issues/10560#issuecomment-739101611


Add Zig.gitignore

This .gitignore template is taken from https://github.com/ziglang/zig/blob/master/.gitignore.

Link to application or project’s homepage: https://ziglang.org/

Sorry, just saw that there was an open PR about this.


docs: showcase updated use cases in example workflows

Summary
This PR updates the example workflows included with this action, and focuses these on common cases 📚
Closes: #158, #270, #349, #361
Preview
Changes in progress: https://github.com/slackapi/slack-github-action/tree/zimeg-docs-examples/example-workflows
Reviewers
All of these workflows have been tested in real GitHub Actions workflows! Including the release announcements!
Please feel free to review these, but I'd like to request specific reviews for the README.md documents included with techniques 🙏 ✨
Requirements

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

@lukegalbraithrussell Thanks for both a review and great suggestions 🙏 ✨
Uploading this to tools.slack.dev sounds so helpful for organizing these pages! I await this with excitement 👾
I reran the included examples and am feeling like we can merge these! I'm open to following up with changes of fixes of course, but in this moment this seems to work well 🚀


Ignore .directory files on Linux

VSCode Version: 1.22.2
OS Version: openSUSE Tumbleweed

Steps to Reproduce:

Use Dolphin file manager to open a folder on Linux with KDE desktop.
Click Preview button in file manager.
Open that folder in VS Code.
You will see a .directory file.

.directory files contains folder information for Dolphin file manager. It is usually useless for any other applications and editors. So it should be excluded in settings by default:
{
  "files.exclude": {
    "**/.git": true,
    "**/.svn": true,
    "**/.hg": true,
    "**/CVS": true,
    "**/.DS_Store": true,
    "**/.directory": true
  }
}

Does this issue occur when all extensions are disabled?: Yes

This is configurable so you can easily add it to your user configuration.
However this hshould not be the default, since Dolphin is not so widely used by our users to my knowledge.
Closing as designed


Add phpcs check and correct style issues

@mosampaio Please take a look at this PR, I run the code style checking tool

LGTM


WIP: limit TInt to the maximum value on any arch

For entertainment purposes only as the network magic used by mainnet is larger than 2^29-1.

Implements #2186

We don't to support 32bit archs right now.


unknown=INCLUDE is currently broken

Hello,
Looks like specifying unknown fields to be included (per marshmallow convention) is not compatible with marshmallow-objects.
Steps to reproduce:
>>> from marshmallow_objects import Model, INCLUDE, fields
>>> class T(Model):
...     class Meta:
...         unknown = INCLUDE
...
...     test = fields.Str()
...
>>> t = T(**{"test": "test string", "not_modeled": 1})
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
<ipython-input-4-f034153db14f> in <module>
----> 1 t = T(**{"test": "test string", "not_modeled": 1})

...marshmallow_objects/models.py in __call__(cls, *args, **kwargs)
     75             for name, value in kwargs.items():
     76                 setattr(obj, name, value)
---> 77                 missing_fields.remove(name)
     78             obj.__missing_fields__ = missing_fields
     79             obj.__setattr_func__ = obj.__setattr_missing_fields__

KeyError: 'not_modeled'

Looks like the issue is in the missing_fields handling logic, it does not account for the Meta.unknown attribute.
This is not an ideal fix, but rather as a proof of concept I made the following change to line 77 in models.py:
   77                 try:
   78                     missing_fields.remove(name)
   79                 except KeyError as e:
   80                     if not obj.__schema__.Meta.unknown == 'include':
   81                         raise e

Then it works as expected:
>>> from marshmallow_objects import Model, INCLUDE, fields
>>> class T(Model):
...     class Meta:
...         unknown = INCLUDE
...
...     test = fields.Str()
...
>>> t = T(**{"test": "test string", "not_modeled": 1})
>>> t
T(**{'test': 'test string'})
>>> t.not_modeled
1

Thanks, will see

Should be fixed by #68
Released in v2.0.0: https://pypi.org/project/marshmallow-objects/2.0.0/

Hey, this issue seems like still exists, above example still fails if we use unknown = INCLUDE.  Is there any plan to fix this?


[Manual] chore(deps): bump actions/github-script from 6 to 7

Identical to #749.

This was a troubleshooting proof-of-concept. See the GH docs here for why the PR was failing (dependabot doesn't have access to repo secrets).


dubbo_proxy: Add the metadata definition

Description: Add the metadata definition for the Dubbo, this metadata information will be used for future Dubbo routing.
Risk Level: low
Testing: unit test
Docs Changes: N/A
Release Notes: N/A
[Optional Fixes #Issue]
[Optional Deprecated:]

/retest

@lizan I have implemented the routing function of Dubbo, but it involves a lot of code, so I split up a lot of PR. This is only the first one. the metadata information is used for routing, filtering, and decoding

@gengleilei
You can submit your complete code to a separate branch, which helps to review this code.

@zyfjeff @lizan DubboProxy full realization code address: https://github.com/gengleilei/envoy/tree/feature-dubbo-router-develop/source/extensions/filters/network/dubbo_proxy

/RETEST

link to #3998


Mention pip's "--ignore-installed" option?

pip's "--ignore-installed" option should let you forcibly downgrade an existing environment (although making a new env is still going to be more reliable)

Ah that's great to know, we should definitely mention that here, thanks for the feedback! 👍


Use Node's native spawnSync for execSync to improve performance

Hello,
I tried to rewrite execSync method with usage of Node's native spawnSync as it should improve performance of sync version of shell.exec method rapidly.
According to my own test script the improvement is from original 12s to 1.6s after change.
What do you think?
I had to remove one test and I hope it works the same way as before. I don't have a Windows machine so it would be great to confirm it works the same also under Windows.
Btw. it could also fix following issues:

#440, #108

Codecov Report

Merging #758 into master will decrease coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #758      +/-   ##
==========================================
- Coverage   97.38%   97.33%   -0.06%
==========================================
  Files          33       33
  Lines        1222     1199      -23
==========================================
- Hits         1190     1167      -23
  Misses         32       32

Impacted Files
Coverage Δ

src/exec.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

I haven't reviewed this carefully, but this will almost certainly break behavior (which is why I was going to write an alternative to exec in #524).
Potential issues I can think of off the top of my head:

Can it still execute commands like echo first command ; echo second command?
Can it still expand globs like git add *.txt?

@nfischer I tested both cases and they seems to be working correctly. I also added test for the first case.
I checked your PR with cmd and it looks something I could use too :-) Next time I need to check other PRs first :-)

@nfischer Oh, sorry, I was too eager… The newly added test failed on Node v8 (I ran node 6.11 locally)…
Hmm, ok, not sure about my "fix" then…

I overlooked that appveyor runs tests on Windows. It probably should? The test wasn't here before so I'm not sure about the previous version of execSync…

I tested both cases and they seem to be working correctly. I also added test for the first case.

Thanks for checking. It looks like we see different behavior for Windows and Unix for spawnSync().
Also, how does this handle output? shell.exec('echo foo ; sleep 5 ; echo bar') should write to the console in real time, not dump all output at the end. Do we have the correct behavior for this patch?

@nfischer Unfortunately real time output is not implemented. I've been playing with it for a while but I cannot figure out any solution. Surprisingly it leads to the original solution of execSync :-) At least I understand the original solution now. My solution cannot be used in case the API has to remain the same and real time output is required too.
I think now… the bottleneck of the original solution, IMO, is that for every command there is spawned node process and several files are created. What if this node process and files are created only once for the first execSync call? We could then use IPC to send the command and options. What do you think? Does it make sense?

My solution cannot be used in case the API has to remain the same and real time output is required too.

exec() has several problems with it. At this point, I don't see a reason to change its behavior--I think it's best to revise completely (which is the approach I took in #524). In other words, I'd like to avoid breaking changes to exec() since we already have plans to deprecate & replace it.

What if this node process and files are created only once for the first execSync call? We could then use IPC to send the command and options. What do you think? Does it make sense?

There are some potential wins here. I think (but we should verify) that process.on('exit', ...) is a reliable signal to wait for cleanup (for both processes and files).
Process creation should happen lazily on the first exec() call.
Without lots of effort, we may be able to remove the exit code file and just use the process return status (which I think child_process.execSync() provides).
We can rearchitect things a bit like in #524. Right now we write an entire script to a temp file, but we can move this to a new source file and pass parameters in via process.argv. This reduces the amount we have to write to disk, which should speed things up.
Is there a way to do IPC without writing to the file system? That would be ideal, but I've yet to find a way.

@nfischer After quick research I found that IPC uses sockets, it can work with unix sockets, TCP, UDP etc.
But I think this idea won't work because IPC is completely async.
I think we can close this PR without merge as I don't know how to fix it. And if you say exec will be deprecated when you finish #524 then it doesn't make sense to improve it.
Anyway, thanks for your support and time on this.

I won't continue on this PR => closed.


Group Type values

See https://linked.fsdf.org.au/dataset/electrical-infrastructure/collections/TL/items/101:
Type: Feature & Overhead High Voltage Power Line are not grouped

Will be solved by property ordering: https://github.com/RDFLib/prez-ui/issues/2


Create templated repository

It would be nice to have the project configured in a copier, cookie-cutter, etc. configuration so the package, etc. can easily be configured.

This is something I need to work on. To be honest, this repo is still a mix of the "template" and some of the random things I'm playing with at the moment. There are a couple of things I need to smooth out in the DX to make it a proper template.
Copier looks nice!


Error: No data to import

Hello,
i get an error when exporting my skeleton with animations from blender to unreal.

The two Animations have NLA strips to them:

This is the export folder:

Scale is applied to everything.
It only sends the skeletal mesh with the skeleton and a material, not the animations.
Versions:
Blender 4.2.4
Unreal 5.5.0
send2ue 2.6.0

Hmm interesting. Could you please attach the blend so I can try and see what's going on

PlayerArms.zip

Apologies for the delay in response. The issue is you have two armature modifiers being applied to the same mesh. If you remove the small incorrect armature + modifier then it should be good to go.

Ok it worked now! Thanks or the response. Also thanks for keeping this plugin alive!


Add method 'bisect_right' and test

This is a method that I needed for my own usage of fenwick trees.
Adding it here as a proposed additional method too, along with a little test for it :)

Thanks @tommyod.
I made the following changes:

Renamed the function from bisect_right to find_stop
The returned value is one higher than you had, for consistency with the other functions, like prefix_sum.
Used a less strict criteria, but it can be modified with the strict argument
Your PR had j = 2 ** len(self). That starts the search outside the bounds of the array, thus resulting in unnecessary computation (successive bit shifts). For example, with a length of 100, j would be set to 2 ** 100 =<PHONE_NUMBER>228229401496703205376.
The function now returns -1 for results that are out-of-range.

I've pushed the latest release to PyPI; it should be available now through pip.


Fix Lycée Connecté (Nouvelle-Aquitaine) ENT

The URL for Lycée Connecté was wrong, the right URL is https://mon.lyceeconnecte.fr/

You're right the URL must have changed since I implemented this ENT.

Can you format your change with the command black pronotepy in the folder of your repository please

Can you format your change with the command black pronotepy in the folder of your repository please

Sure!

It's done!


Add dual-stack support in Canal

Signed-off-by: Manuel Buil<EMAIL_ADDRESS>Description

This PR provides dual-stack support for canal. It requires three changes in the canal config/manifest:
Change 1
The ipam section of the cni_network_config for dual-stack must be defined like this:
          "ipam": {
              "type": "host-local",
              "ranges": [
                  [
                      {
                          "subnet": "usePodCidr"
                      }
                  ],
                  [
                      {
                          "subnet": "usePodCidrIPv6"
                      }
                  ]
             ]
          },

The effective changes are:
a) the  "subnet": "usePodCidr" moves inside the ranges slice
b) we define a new subnet called usePodCidrIPv6
This PR does not break the current way of deploying canal for ipv4 single-stack. If you want dual-stack, you'll need to apply this config change. If not, you are fine with the current config.
Change 2
The flannel image of the kube-flannel container, must move to, at least, version 0.15 which support dual-stack. For example:
image: quay.io/coreos/flannel:v0.15.1
Change 3
We must activate the dual-stack feature in flannel. To do so, in the flannel net-conf.json, we must add the following fields:
"IPv6Network": "2001:dada:42:0::/56",
"EnableIPv6": true,

The IPv6Network can be anything. We will anyway read the CIDR assigned to the node
In the code, the essential change is that we are now asking for node.Spec.PodCIDRs instead of node.Spec.PodCIDR. As a consequence, we will receive a slice of strings, which will have length=1 in the single-stack case and length=2 in the dual-stack case. There is a new function getIPsByFamily that reads the podCidrs and returns the ipv4 and the ipv6 cidr. The former substitutes "subnet": "usePodCidr" in the host-local ipam config. The latter substitutes "subnet": "usePodCidrIPv6" in the host-local ipam config.
Moreover, we use a dummy ipv4 and ipv6 address to release the host-local IP and a test is included with usePodCidrIPv6
I did not change anything in UpdateHostLocalIPAMDataForWindows because dual-stack is not supported in k8s for vxlan scenarios in Windows: https://kubernetes.io/docs/setup/production-environment/windows/intro-windows-in-kubernetes/#ipv6-networking (added a comment about this)
Related issues/PRs

Todos

[X] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

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

/sem-approve

This looks lovely, thank you @manuelbuil!
Fixes https://github.com/projectcalico/calico/issues/5188

Looks like a static check failure:
internal/pkg/utils/utils.go:358:3: ineffectual assignment to `subnet` (ineffassign)04:30
		subnet = ipv6Cidr04:30
		^04:30
../lib.Makefile:531: recipe for target 'golangci-lint' failed

Looks like a static check failure:
internal/pkg/utils/utils.go:358:3: ineffectual assignment to `subnet` (ineffassign)04:30
		subnet = ipv6Cidr04:30
		^04:30
../lib.Makefile:531: recipe for target 'golangci-lint' failed

Thanks, I see why

Is it normal for the CI to take more than 5 hours to report the result?

/sem-approve

Is it normal for the CI to take more than 5 hours to report the result?

CI only runs once a team member approves the PR to run.

Is it normal for the CI to take more than 5 hours to report the result?

CI only runs once a team member approves the PR to run.

Ah! Thanks for the info

I need some help to fix the unit tests because I don't know what is going on :(. 5 tests are failing with:
Unexpected error:
      <*types.Error | 0xc000c56420>: {
          Code: 999,
          Msg: "nodes \"localhost.localdomain\" not found",
          Details: "",
      }
      nodes "localhost.localdomain" not found
  occurred^

The 6th test that fails is the one I added. It fails because there is no dual-stack k8s cluster and thus there is no ipv6 cidr to replace "subnet": "usePodCidrIPv6". I tried adding an ipv6 cidr in --cluster-cidr= in https://github.com/projectcalico/calico/blob/master/cni-plugin/Makefile#L242 but I still get the error

After spending some hours digging the first error, I still did not find what is going on. I found that the error appears here:
https://github.com/projectcalico/calico/blob/master/cni-plugin/internal/pkg/testutils/utils_linux.go#L241, i.e.
r, err = invoke.ExecPluginWithResult(context.Background(), pluginPath, []byte(netconf), args, nil)

And the error is &types.Error{Code:0x3e7, Msg:"nodes \"localhost.localdomain\" not found", Details:""}
pluginPath is /go/src/github.com/projectcalico/calico/cni-plugin/bin/amd64/calico
netconf is:
"\n\t\t{\n\t\t  \"cniVersion\": \"0.3.1\",\n\t\t  \"name\": \"net1\",\n\t\t  \"type\": \"calico\",\n\t\t  \"etcd_endpoints\": \"http://<IP_ADDRESS>:2379\",\n\t\t  \"datastore_type\": \"etcdv3\",\n\t\t  \"ipam\": {\n\t\t    \"type\": \"host-local\",\n\t\t    \"subnet\": \"<IP_ADDRESS>/8\"\n\t\t  },\n\t\t  \"kubernetes\": {\n\t\t    \"kubeconfig\": \"/home/user/certs/kubeconfig\"\n\t\t  },\n\t\t  \"policy\": {\"type\": \"k8s\"},\n\t\t  \"nodename_file_optional\": true,\n\t\t  \"log_level\":\"info\"\n\t\t}"

args is:
&testutils.cniArgs{Env:[]string{"CNI_COMMAND=ADD", "CNI_IFNAME=eth0", "CNI_PATH=/go/src/github.com/projectcalico/calico/cni-plugin/bin/amd64", "CNI_CONTAINERID=cnitest-88", "CNI_NETNS=/var/run/netns/cnitest-88b68c82-2037-92e6-2887-1859ec3651d7", "CNI_ARGS=K8S_POD_NAME=run1570682536;K8S_POD_NAMESPACE=test;K8S_POD_INFRA_CONTAINER_ID=whatever"}}

I added a log in func cmdAdd(args *skel.CmdArgs) but I don't see it being executed. So somewhere in between fails. I lost track of the error in github.com/containernetworking/cni/pkg/invoke

@manuelbuil I'll take a look and let you know what I find!

@manuelbuil I think I figured out what is going on - stuck some comments on the PR in the relevant places. Let me know if you have any questions!

/sem-approve

@manuelbuil I think I figured out what is going on - stuck some comments on the PR in the relevant places. Let me know if you have any questions!

Tests are finally running locally but in CI it fails with:

Summarizing 1 Failure:

[Fail] Kubernetes CNI tests using host-local IPAM when the same hostVeth exists [It] successfully networks the namespace
/go/src/github.com/projectcalico/calico/cni-plugin/tests/calico_cni_k8s_test.go:507

Ran 76 of 76 Specs in 48.948 seconds
FAIL! -- 75 Passed | 1 Failed | 0 Pending | 0 Skipped
--- FAIL: TestCalicoCni (48.96s)

Any idea?

/sem-approve

@manuelbuil I think I figured out what is going on - stuck some comments on the PR in the relevant places. Let me know if you have any questions!

Tests are finally running locally but in CI it fails with:
  �[91mUnexpected error:
      <*types.Error | 0xc002195740>: {
          Code: 999,
          Msg: "failed to allocate for range 0: <IP_ADDRESS> has been allocated to cnitest-51, duplicate allocation is not allowed",
          Details: "",
      }
      failed to allocate for range 0: <IP_ADDRESS> has been allocated to cnitest-51, duplicate allocation is not allowed
  occurred�[0m
�[91m�[1mSummarizing 1 Failure:�[0m

Any idea? Maybe I should change the range in the second set of tests? Let me try that

It did not help. Same error again:
  Unexpected error:
      <*types.Error | 0xc000733d40>: {
          Code: 999,
          Msg: "failed to allocate for range 0: <IP_ADDRESS> has been allocated to cnitest-86, duplicate allocation is not allowed",
          Details: "",
      }
      failed to allocate for range 0: <IP_ADDRESS> has been allocated to cnitest-86, duplicate allocation is not allowed
  occurred

Any idea? Thanks!

@manuelbuil I'll take a look a this today - sorry for the delay, have been on vacation! Happy new year :)

@manuelbuil I think this might be a result of a flake in the gRPC test that is failing that has been fixed in master - could you try rebasing this PR against master and we'll see if the test passes?

@manuelbuil I'll take a look a this today - sorry for the delay, have been on vacation! Happy new year :)

No worries, I was on vacation as well :). Happy new year to you too!

/sem-approve

/sem-approve

Looks like the tests are happy now, hooray!


[WIP] [Feature] Compression Context Pool

Signed-off-by: xyz<EMAIL_ADDRESS>What type of PR is this：

[ ] bug
[x] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #6870
Problem Summary(Required) ：

TestWith kBenchmarkCompressionTimes=300

Single Thread
Before Optimization
Compression Context Pool
compression_buffer

LZ4F_benchmark_single_slice_compression
889ms
899ms

LZ4F_benchmark_compression
6789ms
6899ms
8711ms

LZ4F_benchmark_compression_decompression
16282ms
18060ms
x

ZSTD_benchmark_single_slice_compression
20430ms
20634ms

ZSTD_benchmark_compression
143334ms
141113ms
140319ms

ZSTD_benchmark_compression_decompression
160073ms
158080 ms

Multi Thread(64 threads)
Before Optimization
Compression Context Pool
compression_buffer

LZ4F_benchmark_compression
42027ms
43084ms
41761 ms

LZ4F_benchmark_compression_decompression
132718ms
108182ms
x

ZSTD_benchmark_compression
347115ms
278532ms
249375ms

ZSTD_benchmark_compression_decompression
539742ms
328513ms

TestWith kBenchmarkCompressionTimes=1000

Single Thread
Before Optimization
Compression Context Pool
compression_buffer

LZ4F_benchmark_single_slice_compression
2953ms
2882ms

LZ4F_benchmark_compression
22750ms
24314ms
28733ms

LZ4F_benchmark_compression_decompression
53431ms
57211ms
x

ZSTD_benchmark_single_slice_compression
65761ms
76152ms

ZSTD_benchmark_compression
458918ms
476458ms
455437ms

ZSTD_benchmark_compression_decompression
536313ms
529809 ms

Multi Thread(64 threads)
Before Optimization
Compression Context Pool
compression_buffer

LZ4F_benchmark_compression
168211ms
150786ms
145729 ms

LZ4F_benchmark_compression_decompression
424416ms
394776ms
x

ZSTD_benchmark_compression
926777ms
894420ms
833702ms

ZSTD_benchmark_compression_decompression
1038836ms
328513ms

Load speed improve under high contection:Test Envrionment:Table has 1008 rows, Three concurrent stream load, each batch with 2W rows, jsonStart 16 flush thread

Before OptimizationWrite IO
After OptimizationWrite IO (with compress buffer)
After OptimizationWrite IO (without compress buffer)

22MB/s
35MB/s
28MB/s

It shows that if there are no a lot of competition between threads, the performance of compression context pool is a little worse than origin one. However, under highly contection circumstances. the performance of compression context pool is better than before.
Also, for load, besides compression context, memtable and page builder also allocate from the central cache of tcmalloc(Because their size is larger than one page). Use compression context pool can reduce the contection. As we see in the load test.

Could you please write some description about it?

Could you please write some description about it?

I have some other things to do next week and i will take next week off. I will add some descriptions when i am back.

run all


Query resources when calling REST API

As a Developer
I need to add querying capabilities to the REST API
So that the Inventory Manager can search retrieve inventory records from the Inventory depending on the filters passed by them
Details and Assumptions
Details:

Add querying capabilities to the "list_inventory_records" method to filter the inventory record by  name and condition separately.

The REST API should also filter the records based on a combination of filters that are passed. Therefore add query string support to the "list_inventory_records"

GET /inventory?name="Xyz"&condition="new"&quantity=10

Acceptance Criteria
Given the list_inventory_record has been correctly implemented
When we make a call to the REST API with filters on the query strings
Then the results would return all the records that satisfy the filtered query strings

This story queries the database by:

Name
Condition
Active (whether the inventory record is available)
Quantity

The databased can be queried based on multiple filters together.
Additionally, we can also query the database on quantity with operator > , < , >= , <= and =
Therefore querying the databased with quantity>=10 will return all records that have a quantity >= 10.


Unable to render italics in Markdown

I admit that I am not sure if this is an issue with my terminal (rxvt-unicode now and Gnome Terminal earlier) or the theme but while I am able to see bold as well as bold italic, italic never renders, appearing instead as regular text.
I know the terminals are capable of rendering italics as I checked that by running echo -e "\e[3mfoo\e[23m" and the text did render in italics.
Could you offer some insight into the matter? Please let me know if you require any information.

Does it work if you enable the theme configuration for italic comments?
Also is it reproducible with other color schemes?
I've shortly checked the theme and noted that the markdownItalic and markdownBold are indeed not set. Can you try to add the following to the theme and check if it fixes italics?
hi! link markdownItalic Italic
hi! link markdownBold Bold

I tried enabling italic comments and that too doesn't work.
I've never used a theme before that actually rendered markdown in italics. The base16 ones I used simply used colors and one of the things I really appreciate this theme is that it actually formats instead of coloring.
I tried adding what you suggested but that too didn't fix italics :(

Could you please consider highlighting italics, bold and bold italics using colors instead of formatting?
Maybe offer formatting as an option the way that you do with comments. I would love to see it formatted but seeing how you warn about italics for comments with respect to terminal support, what do you think of offering colored highlighting as the default when it comes to Markdown?

Tested it with no color scheme at all and italics works there.

This theme uses italics. Read the README to see how to enable italics.
It seems like you're having another problem, or you're not following the README, but I can't tell because you're just asking to support italics (it does), but you're not clearly stating the exact problem.

This theme uses italics if you enable italics via a global flag before using colorscheme nord.

Thanks a lot for this. I tested this earlier by adding this line after setting the color scheme and that didn't have any effect. Adding it before does show comments in italics. It also fixes italics in Markdown files.
@arcticicestudio So yes, that flag does fix things but I would like to have italics only for italics in Markdown files, not for comments. Is there anything I can do to make things that way?

seems you're right. Thanks for clearly stating the problem.
@arcticicestudio ... in markdown files, *hello* is hl group markDownItalic =[linked to]=> htmlItalic =[linked to]=> Italic.

verbose hi Italic yields:

Italic         xxx cterm=italic gui=italic
	Last set from ~/vimfiles/plugged/nord-vim/colors/nord.vim

I've been thinking about adding a configuration to allow users to enable italics.The implementation can be made using a variable which gets initialized with italic when the configuration is enabled, otherwise the variable will be a empty string.

Is there anything one can do presently to have italics in markdown files but not for comments?

augroup MyNordChanges
    autocmd!
    autocmd ColorScheme nord highlight Comment cterm=NONE gui=NONE
    autocmd ColorScheme nord highlight Italic cterm=italic gui=italic
augroup end

Thanks a lot! Correct me if I'm wrong but just adding this much, before declaring the color scheme seems to do exactly what I want:
autocmd ColorScheme nord highlight Italic cterm=italic gui=italic
I no longer have let g:nord_italic_comments = 1 though.
Anyway you could make this the default behaviour of the color scheme, with a PR? Provided @arcticicestudio approves.

Your autocmd does do what you desire. I was just more explicit and handled some edge cases.

You wouldn't want to make italics available by default, but I think, arcticice is considering making italics easier to deal with when one might want italics (for example in markdown, with comments, etc). A lot of terminals (and some fonts) do not support italics.
I might have a go at a small refactor and submit a PR if and see what @arcticicestudio thinks. But I also know how tidy and organi{s,z}ed arcticice is, so I feel like I might just be trampling on toes. Nonetheless, I agree that italic support should be improved.

I've submitted #89 which adds a new option to explicitly enable italics. The nord_italic_comments option now requires the new option to enabled in order to take effect. I'll add the missing groups for the Markdown syntax to fix the problem with italic syntax styles.


color-flipper  page styling

Related Issue

color-flipper  page styling
Closes: #258

Describe the changes you've made
I have add the styling to color flipper project
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Checklist:

[x ] My code follows the style guidelines of this project.
[x ] I have performed a self-review of my own code.
[x ] I have commented my code, particularly in hard-to-understand areas.
[x ] I have made corresponding changes to the documentation.
[x ] New and existing unit tests pass locally with my changes.

Screenshots
Put any screenshot(s) of the project here.
https://user-images.githubusercontent.com/67833077/114207936-8a488380-997a-11eb-8709-816066ed3a36.mp4

@ridsuteri @Vishal-raj-1  @urvashi-code1255  please review my PR .

Thank You @Vishal-raj-1  @ridsuteri


How to get flags inside setupMiddlewares() or setupGlobalMiddleware()

Problem statement
I want to print a version number with the flag --version. Ideally the flag is checked before the server starts running. But I'm not sure how I can fetch these flags in setupMiddlewares() or setupGlobalMiddleware().
If I take the normal route, setting the flag --version will "fail" with: 2018/12/06 12:21:27 the required flags '--tls-certificate' and '--tls-key' were not specified.

you can make your own main file and generate with --exclude main after the first generation.
Then you can inject the logic that recognizes the --version flag around there:
https://github.com/go-swagger/go-swagger/blob/master/examples/composed-auth/cmd/multi-auth-example-server/main.go#L51


Adding snapshot_convertor functionality

What is this PR trying to achieve?
Add the necessary conversion functionality.
This application should stream files elasticdump (line separated JSON objects) format files into the same format where each line is transformed into a DisplayWork and then serialised as in the API.
Who is this change for?
🎲 ⌨
Have the following been considered/are they needed?

[ ] Deployed new versions
[ ] Run terraform apply.

I've also had a quick skim and I agree with it being a bit obscure. The actual core conversion functionality seems to be squashed into one class although it is made by multiple steps with different responsibilities. It makes hard to read for me and, I guess, hard to test as well, cause the conversion can only be tested end to end rather than step by step.
Happy to pair on doing that if I could be of help with akka streams

To be honest, it took such a herculean effort just to get the code in the current shape and massage the integration tests to pass, I think we should leave the suggestions above for future refactoring.
I'm still not sure why the convertor tests are passing on my local macine, but failing in Travis. Any ideas?

I would take the opposite view. By breaking it up a bit, it should become easier to debug!
😄

Rebased against current master.

I’ve made some changes to use the pieces merged in #1742 and #1746, slicing a tidy 50 lines off the diff.

Another rebased against current master, and 60 more lines trimmed from the diff.

All the code in this PR has been merged or superseded by other PRs. The last remaining piece is in #1748, so I’m closing this PR.


Document workaround for cross-shard view agg

Fixes #227

@begriffs This review primarily talks about a workaround for an unsupported view. However, #227 is about documenting our existing view support. This PR doesn't address all the use-cases Murat listed in #227. Is there an issue which talks about adding this use-case as a specific workaround? (i.e. did it come from SE?).
If not, I'd rather close this PR, and then document our existing view support based on Murat's examples.


Does not detect currently playing song

The application doesn't seem to be able to detect the currently playing song on Spotify. I'm running Arch Linux 4.15.10-1, npm 5.7.1, and node 9.9.0. Would love to get this to work. I'd love to give more information if you tell me what you need and how I can get it.

Hmm... Unfortunately, I can't test this on Linux at all. spotify-lyrics depends on node-spotify-webhelper for communicating with Spotify. Make sure the SpotifyWebHelper process is running.
Some time ago, I received a PR from @sudhackar which fixed some stuff and if I remember correctly, they were using Linux. @sudhackar, maybe you can help?

I looked at that issue and checked whether it was using the port mentioned, and it was. But now that I look at the other diff line, it changes from HTTPS to HTTP. But I doubt that that wasn't changed as well. I'd love to help fix the issue.

Closing since out of date.


Creating GitHub Actions workflow to run tests.

Addressing #2.

Thoughts on first experience with GitHub Actions:

Nice and easy to setup.
tkinter not importable preventing tests from even running -- even though tests use a fake_tkinter object. :-(

PROGRESS:

Installing python3-tk system package had no effect.
Python is probably not built with Tcl/Tk support. :-(

NEXT:

Maybe we could install a trustable 3rd party CPython build with Tcl/Tk support?
Grab a container and use that?

PROGRESS:

Simpler idea of creating a dummy tkinter.py file was enough.

Happy with hack. Merging.


Complete method self param declarations

Does this issue consider completing self like here:
impl Foo {
    fn bar(<|>) {
    }
}

It'd be cool if this completed self, &self, &mut self, mut self and self: Arc<Self>
Originally posted by @jhgg in https://github.com/rust-analyzer/rust-analyzer/issues/9311#issuecomment-875249115
A simple implementation has been done in #9548 but this only works for empty parameterlists which is not too useful as typing & already doesn't offer any completions.

I'd propose we complete self, &self, mut self and &mut self. I don't know about special-casing Arc<Self>.

I'd propose we complete self, &self, mut self and &mut self. I don't know about special-casing Arc<Self>.

Fine with me. It would be interesting if we could offer Arc<Self> as a completion in this position though, as it's a fairly common pattern to do... 🤔
impl Foo {
   fn bar(self: <|>) {
   }
}

Is this issue resolved with #9548 merged?

It isn't, that PR only added a very basic form of this. Basically it only works for functions with no parameters in them, see the comments after the merge in https://github.com/rust-analyzer/rust-analyzer/pull/9548#issuecomment-877725649 as well as https://github.com/rust-analyzer/rust-analyzer/pull/9575


Investigate preact

Describe the enhancement you'd like
We should try out preact to assess what needs to be done to improve the speed of the app.
Describe why this will benefit the LibrePhotos
Preact is a smaller alternative to react. It's 95% smaller and could increase the loading time significantly. It also claims to be faster.

We have to remove react-dimensions with react-cool-dimensions.
It would also make sense to create a new setup with preact-cli.

Not worth it.


Take down package fish_serializable

It was uploaded by mistake. The owner was confirmed, and it appears to have no usage yet.

$ dart bin/tools/remove_package.dart list fish_serializable
Package "fish_serializable" has the following versions:
  * 0.0.2
  * 0.0.2-dev
  * 0.0.3-dev

$ dart bin/tools/remove_package.dart remove fish_serializable
Removing GCS objects ...
Committing changes to DB ...
Package "fish_serializable" got successfully removed.


Error when passing data to options using useFetch + typescript Nuxt3

Discussed in https://github.com/nuxt/nuxt/discussions/20466

Originally posted by Roman-00 April 24, 2023
I'm trying to pass options to useFetch, but I get an error

Would you provide a reproduction? 🙏

Would you provide a reproduction? 🙏

Repository for accurate investigation of the cause.
https://github.com/Roman-00/iteca.events/tree/testing
"useFetch" is executed on the path pages/index.vue

mode should be no-cors and not no-corse. Apart from that, it seems to work fine.

mode should be no-cors and not no-corse. Apart from that, it seems to work fine.

Note that body should be an object and you do not need to stringify it.

that's right, it should be no-corse.
Please tell me what version of typescript you have in the project?
Everything is absolutely highlighted in red for me :(

My editor is PhpShtorm

Yes indeed this is a typescript problem in phpstorm downgrading the version solved the problem :)
Thank you very much! I'm sorry for wasting your time, but I'll be more vigilant.


Make the this_mu plot function can select num_dlas to plot

Make the plotting function easier to plot an arbitrary number of DLAs. making plots for cosmology-from-home.

Examples:


update vcxproj files for VS 2017

@paulmon,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot


Move md functions from atomutils

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

401 of 500 (80.2%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 83.199%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

simulationworkflowschema/molecular_dynamics.py
401
500
80.2%

Totals

Change from base Build<PHONE_NUMBER>6:
-0.7%

Covered Lines:
2367

Relevant Lines:
2845

💛  - Coveralls

@JFRudzinski can I merge this?

@JFRudzinski can I merge this?

Yes, I think it should be fine :+1:


Accept hash in channel name

Description

Resolves #68

Checklist

[x] Add tests for these changes
[ ] Test coverage is not decreased
[ ] Entire test suite passes

Codecov Report

Merging #69 into master will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #69      +/-   ##
==========================================
- Coverage   97.58%   97.55%   -0.03%
==========================================
  Files          18       18
  Lines         992      980      -12
  Branches      162      163       +1
==========================================
- Hits          968      956      -12
  Misses         16       16
  Partials        8        8

Impacted Files
Coverage Δ

src/spellbot/__init__.py
96.38% <100.00%> (-0.06%)
:arrow_down:

src/spellbot/data.py
99.16% <0.00%> (-0.03%)
:arrow_down:

Continue to review full report at Codecov.

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


Information from UniProt should be parsed via REST-API

When initializing a ProteinSequence via the from_ncbi method from an u
UniProt ID, information is incomplete. For instance, the coding sequence reference is missing.
UniProt offers a REST-API for protein entries, including information on protein activity, references to alphafoldBD, and GenBank.
I think the data model should be extended to capture these information and all references to other databases.

solved in #56


feat(electron-esbuild): enable cross origin for all proxy in esbuild dev

Related to this comment https://github.com/Kiyozz/electron-esbuild/issues/13#issuecomment-847099528, this change enables the access for other type of compiled files such as workers.
Questions though, why doesn't this Cross-Origin-Embedder-Policy issue happen to the path to files below?
/index.css
/index.js

I'm not sure what exactly this option does. Can you clarify for me? And why this fixe static files?

By static files, I means files that are available on "/". If you want to add any images, or untranspiled js, css, etc. Like a server public path.
Your PR doesn't seem to fix the linked issue (#13), but another problem that is multiple esbuild renderer entrypoints. Maybe you could create a new issue more detailed?

Alright I'll reference a new issue soon.

@Kiyozz I have updated the PR description and this actually fixes #12 issue.
About COEP, the default value is unsafe-none.

Hi, is there any other concern for removing COEP headers in development mode?

Hey josteph, thanks for your changes. I forgot to answer, I want to make a few tests before complete the pr but I did not find the time.
I think their is no problem!

Thanks you for your work.

No, thank you for the great package!


[PDF]: View Page 3: Fields not populating

From Stacey:

Driver's license # and state are not populating.
Race & Ethnicity are not populating.

-Add race & ethnicity
-Made all text fields unique fields--were duplicates before
-Added driver's license # & state fields

Verified fields- #closeissue


Remove partition-topic redirection as broker sends partition-metadata…

… without redirection
Motivation
After introducing global-zk in DiscoveryService, it returns partitionMetadata without redirection. So, Broker and DiscoveryService returns PartitionMetadata without redirection. So, cpp-client doesn't have to handle redirection on PartitionMetadataResponse.
Modifications
Directly read number of partitions from response.
Result
No functional change.

👍


What configuration can be modified to modify the length of the output video

What configuration can be modified to modify the length of the output video

What configuration can be modified to modify the length of the output video

You can modify the fps parameter in the save video function to change the length of the saved video (https://github.com/Vchitect/LaVie/blob/9fcf2f6b787c6a59eda6aadfbac878576c7cb4f2/base/pipelines/sample.py#L79).


add keyword to mark Deptrac as a dev dependency

Adding the static analysis keywords makes Composer prompt the user to suggest re-running composer require with --dev automatically if the option was forgotten by the user (see https://getcomposer.org/doc/04-schema.md#keywords).

@xabbuh could you be so kind and reopen the PR here https://github.com/qossmic/deptrac-src/blob/2.0.x/build/template/composer.json.
The deptrac repo ist just for distribution purposes. The active development should happen in qossmic/deptrac-src.
And it seems that I did something similar in the composer.json in deptrac-src, but forgot the template composer.json.

see qossmic/deptrac-src#91


How can I automate case creation using email?

Hi team,
I want to automate my outlook shared folder emails as "cases" in TheHive. Please help me for the same.

You need to write a script that analyses outlook content and creates a data input that can be passed to thehive4py to push to backend as a case. thehive4py does not provide a solution to read your outlook emails.
However you can easily read outlook email from the powershell level. I've dont this in the past: https://github.com/mnmnc/p0r/blob/master/src/p0r.ps1
$outlook = New-Object -comobject outlook.application
$namespace = $outlook.GetNameSpace("MAPI")

# GETTING PST FILE THAT WAS SPECIFIED BY THE PSTPATH VARIABLE
$pst = $namespace.Stores | ?{$_.FilePath -eq $pstPath}

# ROOT FOLDER
$pstRoot = $pst.GetRootFolder()

# SUBFOLDERS
$pstFolders = $pstRoot.Folders

# PERSONAL SUBFOLDER
$personal = $pstFolders.Item("Personal")

# INBOX FOLDER
$DefaultFolder = $namespace.GetDefaultFolder(6)

# INBOX SUBFOLDERS
$InboxFolders = $DefaultFolder.Folders

# DELETED ITEMS
$DeletedItems = $namespace.GetDefaultFolder(3)

# EMAIL ITEMS
$Emails = $DefaultFolder.Items

# PROCESSING EMAILS
Foreach ($Email in $Emails) {
    # Do stuff here to get email data.
    # Output data to a file, then parse it withing a python script that uses thehive4py
}

Fetching the emails is not the responsibility of TheHive4py, so you need a script that listens to an mailbox, extracts the details and user create_case function to create the case.
You can also check the following opensource tool written by @xme: https://github.com/xme/dockers/tree/master/imap2thehive


Fix trimming of spaces

Removing all occurrences of 2 spaces is dangerous. For example look at the following definition used in https://github.com/polymerelements/paper-styles:
--shadow-elevation-16dp: {
  box-shadow:
    0 16px 24px 2px rgba(0, 0, 0, 0.14),
    0  6px 30px 5px rgba(0, 0, 0, 0.12),
    0  8px 10px -5px rgba(0, 0, 0, 0.4);
};

For indentation reasons the authors have used two spaces between 0 and 6px/8px. Replacing two spaces or more with nothing would result in 06px / 08px.
This pull requests solves this by reducing two or more spaces to one (two or more will always have the same meaning as one) and by removing leading and trailing spaces afterwards.

Oops, just noticed that there was already an issue for this, #8. Well anyways, I think this is a really good shot at solving that issue...

LGTM

@rubenstolk I know there are no tests yet.... but it would be SO cool if you added a simple unit test for this :)

@samccone I guess I would leave that to @azakus or someone else to set it up, I don't mind writing the test itself but not very sure what framework and all we're looking at...

Seems like @azakus is OK with mocha :coffee:
https://github.com/PolymerLabs/crisper/tree/master/test

We won't make you setup tests for the repo in this PR :) I'll let @azakus have the final LGTM and merge b/c I can't update the npm module.

:+1: :+1: just a thought :cloud: obviously not required at all :wink: --- thanks again for this @rubenstolk

Yeah, I've been bad by not adding tests :(. LGTM!

Awesome!

Would you make a 1.2.1 on npm?

Done!

Cheers and beers on me!

I believe this fix is breaking cases where css selectors contain ::content preceded by a space. For example, paper-drawer-panel has a selector like this:
iron-selector:not(.narrow-layout) #main ::content [paper-drawer-toggle]

The new code is removing the space between #main and ::content which is significant:
iron-selector:not(.narrow-layout) #main::content [paper-drawer-toggle]

Hmm right! I guess we might want to remove the : from the regex...

Or look for :[^:] patterns, I'll check it out tomorrow.

Sent a new PR #10


Change message

Is there a way to change the message to other than please select a date?

Is there a way to change the message to other than please select a date?

Hello, Can I have an example?

Sure here:

when useing the command calenadr the message that displayes is please select a date. Is it possible to change it?

const TOKEN = process.env.TELEGRAM_TOKEN || 'YOUR_TELEGRAM_BOT_TOKEN';
const {Telegraf} = require('telegraf');
const NavCalendar = require('telegram-inline-calendar')
const bot = new Telegraf(TOKEN);
const calendar = new NavCalendar(bot, {
    date_format: 'DD-MM-YYYY',
    language: 'en',
    bot_api: 'telegraf'
});
bot.start((ctx) => calendar.startNavCalendar(ctx.message));
bot.on("callback_query", (ctx) => {
    if (ctx.callbackQuery.message.message_id == calendar.chats.get(ctx.callbackQuery.message.chat.id)) {
        res = calendar.clickButtonCalendar(ctx.callbackQuery);
        if (res !== -1) {
            bot.telegram.sendMessage(ctx.callbackQuery.message.chat.id, `Your custom message with date(${res})`);
        }
    }
});
bot.launch();
process.once('SIGINT', () => bot.stop('SIGINT'));
process.once('SIGTERM', () => bot.stop('SIGTERM'));

The clickButtonCalendar method itself returns the date in the format you choose, and can be further processed as you wish.
Examples:
bot.on("callback_query", (ctx) => {
    if (ctx.callbackQuery.message.message_id == calendar.chats.get(ctx.callbackQuery.message.chat.id)) {
        res = calendar.clickButtonCalendar(ctx.callbackQuery);
        if (res !== -1) {
            bot.telegram.sendMessage(ctx.callbackQuery.message.chat.id, `Your custom message with date(${res})`);
        }
    }
});

bot.on("callback_query", (ctx) => {
    if (ctx.callbackQuery.message.message_id == calendar.chats.get(ctx.callbackQuery.message.chat.id)) {
        res = calendar.clickButtonCalendar(ctx.callbackQuery);
        if (res !== -1) {
            console.log(`Your custom message with date(${res})`);
        }
    }
});

@AguChaves92,
I was able to help you?

I don't mean the answer after picking the date, but before.
As  I've understood it, the command tha launches the bot is this one right?:
bot.start((ctx) => calendar.startNavCalendar(ctx.message));
the triggers a response with a message and the calendar from the bot. I want to change that message.

I don't mean the answer after picking the date, but before. As I've understood it, the command tha launches the bot is this one right?: bot.start((ctx) => calendar.startNavCalendar(ctx.message));
the triggers a response with a message and the calendar from the bot. I want to change that message.

I'm sorry, I misunderstood you.
Tomorrow I will release an update with this functionality.

thank you very much

@AguChaves92 Released an update, please check.

I see the custom message now thank you

@AguChaves92, Can the issue be closed?

@AguChaves92  ?

sry, yes


Add git as a "bin" dependency

Since the code of this module relies on calling git as an executable, it's not usable without this command being available. It makes sense to make this dependency transparent.

Thanks!


bundle.generate returns a Promise

In my development environment bundle.generate returns a Promise. Because of this my karma preprocessing fails.
A short fix this can be seen here:
https://github.com/pikobytes/karma-rollup-preprocessor/blob/master/lib/index.js
If you want a i could provide a pull request.
I am using<EMAIL_ADDRESS>

@jacmendt See #21


Unable to delete files / folders in atom tree view in Windows 7

Prerequisites

[x] Can you reproduce the problem in safe mode?
[x] Are you running the latest version of Atom? (1.5.4)
[x] Did you check the debugging guide?
[x] Did you check the FAQs on Discuss?
[x] Are you reporting to the correct repository?
[x] Did you perform a cursory search to see if your bug or enhancement is already reported?

For more information on how to write a good bug report or enhancement request, see the CONTRIBUTING guide.
Description
Unable to delete files / folders in atom tree view in Windows 7
Steps to Reproduce

Right click file / folder
Select Delete option
Select Move to Trash

Expected behavior: File / folder to be deleted
Actual behavior: Nothing
Versions
1.5.4

My company has some rather annoying security software installed, so if no one else can reproduce this, I am ready to declare that at fault instead of Atom.

This is a duplicate of atom/tree-view#666 and will be fixed in Atom 1.7.0.  Sorry for how long this bug has been present in Atom :(.

Works @ home, so most likely an issue with the "security" software we have at work.


[BUG] Website does not have proper meta tags

Description
Describe the bug:
The website does not have meta tags like open graph, twitter, etc. This means that the website is less likely visible in search results, and users are less likely to click on the website's links.
Expected behavior:
The website should contain important meta tags. This will help the website rank higher in search results and attract more visitors.

Please assign me this issue so that I can work on this issue under GSSOC'23.

We will look after the SEO, don't worry


🏄 prepare_spectra(): Improving speed while maintaining reasonable memory consumption

addresses part of #52

before:
after (this PR)
r$> library("opusreader2")
    file <- opus_file()

r$> bnch <- bench::mark(
      data <- read_opus(dsn = file)
    )
Warning message:
Some expressions had a GC in every iteration; so filtering is disabled.

r$> bnch
# A tibble: 1 × 13
  expression                         min   median `itr/sec` mem_alloc `gc/sec` n_itr  n_gc total_time result            memory               time            gc
  <bch:expr>                    <bch:tm> <bch:tm>     <dbl> <bch:byt>    <dbl> <int> <dbl>   <bch:tm> <list>            <list>               <list>          <list>
1 data <- read_opus(dsn = file)   19.4ms     20ms      46.7    2.07MB     52.6    24    27      514ms <named list [20]> <Rprofmem [577 × 3]> <bench_tm [24]> <tibble [24 × 3]>

@ThomasKnecht currently we iterate through all element, independently of what data type they are.
https://github.com/spectral-cockpit/opusreader2/blob/c3e4c2d38594574f38de48283e6e181ddfda8dfb/R/parse_opus.R#L83-L86
What about doing a selective element update in a loop?

closing because this PR with append has no advantages.


Alternative to @v_args()

Suggestion

Use explicitly named decorators: @wrap_every (class level only), @inline_children (method level only)
Let Transformer decide which arguments to pass based on method signature:

If method has single argument, pass children regardless of its name (this will preserve backwards compatibility)
If args are subset of {meta, tree, children} — pass respective values.

v_args is left unchanged for backwards compatibility.

This may make API more explicit and sound. The second approach is used in e.g. Pydantic (see validators in v.1).
I'm not aware of all possible use cases for v_args, but at least some of them are covered.
Describe alternatives you've considered
@v_args is the only alternative.
Additional context
Usage examples:
Class level only @wrap_every
@wrap_every(inline_children)
class Demo(Transformer):
    def rule(self, *items): ...

Method level only @inline_children
class Demo(Transformer):
    def rule1(self, children): ...
    @inline_children
    def rule2(self, *items): ...

Method signature auto detection
class Demo(Transformer):
    def rule1(self, items): ...  # children passed
    def rule2(self, tree): ...  # tree passed
    def rule3(self, meta, children): ...  # pass meta, children
    def rule4(self, meta, items): ...  # to be decided: raise error or pass children to items
    @inline_children
    def rule5(self, tree, meta, *items): ...

If this change sounds reasonable, I'm ready to work on PR.

Use explicitly named decorators: @wrap_every (class level only), @inline_children (method level only)

Can you explain your rationale?

First of all, let me say that Lark is an awesome project and already saved me days of work. I appreciate hard work, dedication and talent of Lark's maintainers. I don't understand much in parsers and lexers, but want to dedicate some of my time to the field where I touch Lark the most as a developer: its API.
The main reasons for change proposed are

Separation of features applicable in different contexts,
Readability, better naming, less code (lower cognitive load).

For v_args(meta=, tree=):

This syntax can be eliminated, args can be deduced from method signature (methods are not called manually, Lark can do this itself).

For v_args(wrapper=):

This feature does not make sense when applied to individual method, is applicable only at class level.

For v_args(inline):

Not immediately obvious what is inlined, inline_children seems better to me. Also this will be more readable when used as a standalone decorator on methods.


Using it with webdriver.io from a test

Hi, I'm unsure on how to add the functionality from inside a test.
dragMock.extendWebdriver(webdriver); and dragMock.loadLibrary(webdriver); assumes I have a webdriver, but afaik I only have a browser?
const dragMock = require('drag-mock');

describe('Something', function() {
  it('should drag n drop', function() {
    // How do I extend and load it into the webdriver here?
    dragMock.extendWebdriver(browser);
    dragMock.loadLibrary(browser);

    browser.dragStart('.Something');
    // ^-causes a "browser.dragStart is not a function" error
  });
});

Do I add it to the webdriver in one of the lifecycle hooks in wdio.conf.js? I can't find the right method or argument there, unfortunately.

Are you using webdriver.io? Then I suggest using the before() hook in your test file. And you also have the webdriver object: http://webdriver.io/guide.html
If you are not using webdriver, you shouldn't have to care about extendWebdriver() and loadLibrary().
Hope that helps!

Thanks, I am using webdriver.io, and that guide (scroll to bottom) is what prompted me to use The browser object which seems to be the webdriver instance.
However, running the following code:
dragMock.extendWebdriver(browser);
dragMock.loadLibrary(browser);

..and subsequently:
browser.dragStart('.Something');

Causes a browser.dragStart is not a function crash in the test. Using the before() hook or not, it seems if I only have one test, and it crashes like that, something else is amiss, don't you think?

Ahhh, now we are on the same page.
I fear that more recent webdriver versions have a different API than one and a half years ago when I wrote this... :-/ Maybe it's a minor issue, but currently I haven't got the time to look into that, sorry.
If you are able to make it work, please share it and we can share the knowledge here.

Could you solve that issue? I have exact the same problem...

Sorry, not yet.

Added a line to the README to at least warn users: https://github.com/andywer/drag-mock#webdriverio-integration


Wont start in Ubuntu 24.04 LTS

I installed wireguard and wireguard-cli but trying to launch wireguard-cli fails, both in user and administrator modes...
paul@ENVY:~$ wireguard-gui
current: Err(Os { code: 2, kind: NotFound, message: "No such file or directory" })
payload: IpPayload { origin: "<IP_ADDRESS>" }
app inited
app state: AppSt(Mutex { data: <locked> })
Gtk-Message: 02:52:08.854: Not loading module "atk-bridge": The functionality is provided by GTK natively. Please try to not load it.

(wireguard-gui:19106): Gtk-WARNING **: 02:52:08.935: GTK+ module /snap/wireguard-gui/5/gnome-platform/usr/lib/gtk-2.0/modules/libcanberra-gtk-module.so cannot be loaded.
GTK+ 2.x symbols detected. Using GTK+ 2.x and GTK+ 3 in the same process is not supported.
Gtk-Message: 02:52:08.935: Failed to load module "canberra-gtk-module"

(wireguard-gui:19106): Gtk-WARNING **: 02:52:08.936: GTK+ module /snap/wireguard-gui/5/gnome-platform/usr/lib/gtk-2.0/modules/libcanberra-gtk-module.so cannot be loaded.
GTK+ 2.x symbols detected. Using GTK+ 2.x and GTK+ 3 in the same process is not supported.
Gtk-Message: 02:52:08.936: Failed to load module "canberra-gtk-module"
libEGL fatal: DRI driver not from this Mesa build ('23.2.1-1ubuntu3.1~22.04.2' vs '23.0.4-0ubuntu1~22.04.1')

paul@ENVY:~$
paul@ENVY:~$
paul@ENVY:~$
paul@ENVY:~$ sudo wireguard-gui
mkdir: cannot create directory ‘/run/user/0’: Permission denied
current: Err(Os { code: 2, kind: NotFound, message: "No such file or directory" })
payload: IpPayload { origin: "<IP_ADDRESS>" }
app inited
app state: AppSt(Mutex { data: <locked> })
Authorization required, but no authorization protocol specified

(wireguard-gui:19222): Gtk-WARNING **: 02:52:20.063: cannot open display: :0

Am i missing some prerequisite? I dont want to develop or build, just run it

Note that snap install wireguard-gui installs 0.1.1
paul@ENVY:~$ snap install wireguard-gui
wireguard-gui 0.1.1 from leone (leonev) installed

Hey thanks for the report, the snap version doesn't work properly currently.
The missing dependencies such as libcanberra-gtk-module libjavascriptcoregtk-4.0-18 libicu70 libwebkit2gtk-4.0-37 is weird i didn't had to install them on a fresh install, i'll add them in the prerequise in the readme

I have updated the readme, closing this issue.

@paulhiggs so the deb file is broken for 24.04 you say? Since I don't have these issues either. However I installed the deb on Linux mint 21.3, which is actually based on a older Ubuntu: Ubuntu 22.04.

This is the error I see when trying to install the deb:

I guess extending the dependencies will fix that i'll add it in 0.1.4 release

0.1.4 is release and should fix the problem

Well not this deb file installation error:

So under Ubuntu 24.04 this package is now called: libwebkit2gtk-4.1-0.  https://packages.ubuntu.com/noble/libwebkit2gtk-4.1-0

So in the package depends for Ubuntu you need to use an "OR" syntax basically like : libwebkit2gtk-4.0-37 | libwebkit2gtk-4.1-0. So on older Ubuntu releases it might need to install and find the 4.0-37 version, while on newer ubuntu releases it's able to find and install 4.1.0 of this package dependency.
I hope that is possible to do that in Nextauri, I use CMake and with CMake I can use this | pipe symbol this purpose.. Here a real-life example of my WineGUI project: https://gitlab.melroy.org/melroy/winegui/-/blob/main/cmake/packaging.cmake?ref_type=heads#L40

I doubt this is possible i'll see what i can do

See also: https://serverfault.com/questions/1055356/how-to-add-multiple-dependencies-to-debian-package

Lubuntu 24.04.01 v0.1.5. also has unsatisfiable dependencies.
instead of libjavascriptcoregtk-4.0-18 needs 4.1.0
instead of libicu70, needs 74
instead of libwebkit2gtk-4.0-37 needs 4.1-0

On Lubuntu 24.04.01 v0.1.5 .deb also has unsatisfiable dependencies.

Yes well. It's the same issue, since Lubuntu 24.04 is based on Ubuntu 24.04 (Noble Numbat).

I'll create a build workflow that run on ubuntu 24 to fix this issue


prune Cloudsql

@KnVerey is there anything that's going to blow up if we remove cloudsql from the prune list?  We've had a few issues due to dangling resources.

blech.

cc @mkobetic


I cannot log in to Google Chat (Workspace)

I reached the login screen of Google Chat and entered my ID/Password, but after a while I get an error like the attached image.
It happens every time. I tried to use a custom app and enter [https://chat.google.com/] directly, but I get a message saying [Your current browser is not compatible with Google Chat.].
Is there something wrong?

Same here with hangouts/gchat. In our company we have an own authenticator enabled for that. It opens, I can enter my credentials and get forwarded to the chat, where I get an annoying "Your current browser is not compatible with Google Chat." Was working with 2.7.0 and a custom app, only. Broke with 2.7.1 or 2.7.2 in the normal and the custom app.


Add more options for writing tables out

I would like to have a Util.print method that takes a Writer or PrintWriter.  Also a simple way just to render a String seems reasonable.

Greetings!
I've replaced the PrintStream in the main output method with PrintWriter and added a few helper methods to adapt Writer, and OutputStream.  I've also added asString which utilizes a StringWriter to render a String.
Please have a look and let me know your thoughts.
Thank you for your interest in my project !


Flush close on destroy, CMake and buff_size

Added CMakeLists.txt and included buff_size in the constructor of the i/ofstream classes
Including a close on the underlying I/ofstream on destruction

@graemenail @sandsmark as you are the most active users, could you test if this PR runs well on your side, too, before I merge ?

This version f7d3bdb2da93 works in my environments using both cmake 3.10.2 (with an extra include_directories in the parent CMakeLists.txt) and 3.18 without it.

Thanks so much! It looks great. Could you maybe add in the Readme a How-to-include section (for each CMake version)?


Update com-server-associations.md

removing duplicate "COMServerAssociation" for Drives and "Unknown Files" (guessing it was a copy/paste mistake)

Oh sorry, guess I could've commited to patch-1 and not make a second request. Next time.


Announce mermaid.js article

Provide the actual tweet content between the commented colons
Announce https://github.com/kubernetes/contributor-site/pull/255
This should be tweeted on 2021-12-01

New on the Kubernetes contributor blog: Improve your documentation with Mermaid.js diagrams https://k8s.dev/blog/2021/12/01/improve-your-documentation-with-mermaid.js-diagrams/

I see no PR, GitHub Actions.

https://github.com/kubernetes-sigs/contributor-tweets/actions/runs/1518146419 implies the workflow failed.

I see no PR, GitHub Actions.

Did you put the tweet content within the space asked for?

It's like this:
<!-- Input your tweet content exactly between the commented colons -->
<!--::-->
New on the Kubernetes contributor blog: Improve your documentation with Mermaid.js diagrams
https://k8s.dev/blog/2021/12/01/improve-your-documentation-with-mermaid.js-diagrams/
<!--::-->

/validate

@sftim So sometimes, there's this weird issue whenever there's a line break before the link this bot fails to parse it. I would suggest to edit and remove the line break before the link.

/validate

There's some issue with the bot, I tried to test it here, and seems like it's not creating the PR
cc @gkarthiks @rajula96reddy

/validate

Our issue to PR automation seems to be broken after https://github.com/kubernetes-sigs/contributor-tweets/pull/104
@sftim  Sorry for this. If this tweet is important, I would recommend creating a PR directly to get the tweet out.
@Debanitrkl Thanks for validating and checking. I will create a new issue and we can follow it up there

I opened https://github.com/kubernetes-sigs/contributor-tweets/pull/121

There's some issue with the bot, I tried to test it here, and seems like it's not creating the PR cc @gkarthiks @rajula96reddy

I was out on medical emg. Back on work. Let me look it up in the coming weeks.

Bit late now
/close


Access the localhost application from another machine using IP

We are able to run this successfully. Backend is started using this command. App is running in local. but my collegue wants to access the application using my ip:port but it doesnt work. please help
.\backend_env\Scripts\python -m quart --app main:app run --port "50505" --host "<IP_ADDRESS>" --reload

You would need to expose the port on your local machine in a way that your colleague can access it. That's not something that I've done myself, but there should be instructions online for how to do that generally, I don't think that's particular to this app.


auto_width of columns does not seem to work

auto width for columns does not seem to work. I see there were a couple of tickets opened for this back in 2017, but I didn't quite get the status of them. Pls help!

Any update on this?


Add support for HTML

"command not configured for HTML!
Add an issue on GitHub or send your own pull Request."
I get this message.
I want to know how to deal with it.
A little more detailed explanation, please.

How you want to "run" html? it is not a programming language. just open a html file in your browser

I don't understand your explanation because I am a beginner.
Thank you very much.
What is the best way to learn autom?
Thank you for your reply.
2016-12-02 19:49 GMT+09:00 Nikita Gryzlov<EMAIL_ADDRESS>
How you want to "run" html? it is not a programming language. just open a
html file in your browser
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rgbkrk/atom-script/issues/1170#issuecomment-264427038,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXMA7SAwd-ovF4csNlFQqfoU4_vLiyyrks5rD_e2gaJpZM4LCa5H
.

--
천병화

@sabora23 Maybe you need this package to open HTML file in your browser. You can install it with apm.
If you want to learn more about Atom, read Atom Flight Manual.

HTML is a markup language and therefore cannot be compiled. Don't forget to check out existing issues (open and closed) before posting one yourself.
Closing as this is a repeat of #330.


spruce up content and link to new full instructions

Spruced up the content and added various links, namely links to instructions for downloads/setup/info. for the various browser/headset/OS platform combinations (which are currently/temporarily being hosted here, until I/@caseyyee embeds/rejigs/finds a new home/format for them).

for a lighter page weight and more readable experience, I also made these changes:

removed the the background-images and Chivo web font (using system fonts now)
switched up the colours
made layout responsive for tablet/mobile widths
did a little something small with the header
tweaked the typography (for contrast and legibility)

live demo temporarily pushed to https://cvan.io/webvr.info/

This is all sorts of great! Thank you! (And sorry for the slow merge. Excuse: Baby)

no worries at all. congratulations very, very much! 👶 🎉


Leaderboard page not redirecting to home

When clicking on the home button/link from the leaderboard page, the redirection does not occur as expected. Instead, it directs to a error 404 page without navigating back to the homepage. This issue affects user experience, as users are unable to return to the main page directly from the leaderboard.

You can start with pr, U just have to edit the href ig?

Yes @B1N4RY-P4R45173  and one more thing I would Like to add is creating a resume button in the leaderboard page which will redirect the page to the Level the person has left from.
and can ou also add gssoc label?

Sure do it

Shall I open another issue and make the pr separately?

Hey @B1N4RY-P4R45173 ! I have made a PR fixing the redirection issue . Please review it .
https://github.com/Vinay-Khanagavi/Cyberbear.github.io/pull/97

@B1N4RY-P4R45173 Shall i start working on this feature?

please assign this issue


Smart AI Robber Logic and Andrew's Additions to Smart AI

When a smart AI rolls the robber, it targets the player with the most victory points
@alang21 Added logic to other gameplay aspects; however, currently building does not work, so this requires further testing
(Refer to Issue #285 )

Correction:
User working on this was @AndrewLang98


Allow more than two dots in template file names.

This should fix #58.

Would you mind resubmitting this to the 2.0 branch?

Done.


remove PassThrough and use cross-spawn stream directly. resolves #16

Per our discussion in #15, here's a change to remove the PassThrough stream and use the cross-spawn stream directly.
x-ref #16

Thanks @evancohen 🙌
I'll go ahead and merge this into the master branch and cut a new release soon.

Published in 0.1.8


[Bridging PCH] Only emit deps for CompileJobActions rdar://31056789

Swift 3.1 branch variant of #8097 , original text follows:
Undo a bug (accidentally introduced in #7944 / fae468f) wherein we generate .d and .swiftdeps files for bridging PCH steps. The presence of these breaks an invariant that only CompileJobActions are supposed to have such depfiles, which in turn causes an attempted cast to go wrong. This bad cast traps in an assertions-build. In non-assertions builds it's a UMR that may cause a miscompile.
rdar://31056789

@swift-ci please test

@swift-ci test


Can't compile for react-native-windows

I'm working on a react-native-windows UWP application, I'm trying to add the library but It seems like I can't compile on windows platform.

This project references NuGet package(s) that are missing on this computer. Use NuGet Package Restore to download them.  For more information, see http://go.microsoft.com/fwlink/?LinkID=322105. The missing file is C:\RN\testapp\windows\packages\Microsoft.UI.Xaml.2.7.0\build\native\Microsoft.UI.Xaml.targets. | RNSVG | C:\RN\testapp\node_modules\react-native-svg\windows\RNSVG\RNSVG.vcxproj | 237 |

Is there something I am missing?
react-native-svg: 13.1.0
react-native: 0.69.0

Same issue
"react": "18.1.0",
"react-native": "^0.70.0",
"react-native-windows": "0.70.0"
"react-native-svg": "^13.1.0",

hi @lnisi @exotexot have you found any solution to this?
I have stumbled upon same issue.

Using NuGet restore should be enough to solve this.
Then it will probably warn you about missing Win2D package, which you can import by:

opening the project on Visual Studio
Tools menu -> NuGet package manager -> Manage NuGet packages for this solution
install Win2D.uwp 1.25

Now it should build correctly. The real problem is that SVG components will not show correctly in the app or not show at all. And I couldn't find any way to make it work, at least with<EMAIL_ADDRESS>

Using NuGet restore should be enough to solve this.
Then, and also according to this issue, it will probably warn you about missing Win2D package, which you can import by:

opening the project on Visual Studio
Tools menu -> NuGet package manager -> Manage NuGet packages for this solution
install Win2D.uwp 1.25

Now it should build correctly. The real problem is that SVG components will not show correctly in the app or not show at all. And I couldn't find any way to make it work, at least with<EMAIL_ADDRESS>
Yeah, actually I did the same and been able to compile but can't render any svg at all, as you mentioned.
I thought there was something wrong with the NuGet package used so I reverted everything and asked here for help.
By now I totally bypassed react-native-svg, but I hope someone can tell us more about it.

@mattia-sanfilippo @lnisi - the rendering issues should be fixed by #1928. Let me know if you're still having issues once that merges.


Remove problematic depthai_pipeline_graph submodule

Exactly the same problem as here: https://github.com/luxonis/depthai/pull/1006
Appeared in latest develop.
Installing previous develop commit works:
pip install git+https://github.com/luxonis/depthai@41d4da7f1b3aab619404612ffe1928953b53a019#subdirectory=depthai_sdk

Installing current, doesn't:
pip install git+https://github.com/luxonis/depthai@develop#subdirectory=depthai_sdk

Trace:
(venv) filipjeretina@filipjeretina-MS-7C02:~/Documents/dev/depthai-clone$ pip install git+https://github.com/luxonis/depthai@develop#subdirectory=depthai_sdk
Collecting git+https://github.com/luxonis/depthai@develop#subdirectory=depthai_sdk
  Cloning https://github.com/luxonis/depthai (to revision develop) to /tmp/pip-req-build-0rjtf322
  Running command git clone --filter=blob:none --quiet https://github.com/luxonis/depthai /tmp/pip-req-build-0rjtf322
  Running command git checkout -b develop --track origin/develop
  Switched to a new branch 'develop'
  Branch 'develop' set up to track remote branch 'develop' from 'origin'.
  Resolved https://github.com/luxonis/depthai to commit b74d54ad9fb999a4f8c716db1ee7b53d8116d866
  Running command git submodule update --init --recursive -q
  fatal: No url found for submodule path 'depthai_sdk/src/depthai_sdk/components/integrations/depthai_pipeline_graph' in .gitmodules
  error: subprocess-exited-with-error

  × git submodule update --init --recursive -q did not run successfully.
  │ exit code: 128
  ╰─> See above for output.

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× git submodule update --init --recursive -q did not run successfully.
│ exit code: 128
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

Best to remove this submodule all together and rather publish pipeline_graph to pypi, and have it as a requirement for SDK

Done, depthai-pipeline-graph is now available on pypi and the sdk uses it as a dependency!


Add support for Timeplus streaming database

Timeplus is a simple, powerful, and cost-efficient stream processing platform. This PR added support for both Timeplus Proton (https://github.com/timeplus-io/proton) and Timeplus Enterprise (https://timeplus.com/product).
To test it:
Start a docker instance of Timeplus Enterprise via
docker run --name tpe2.5.11 -p 8000:8000 -p 8463:8463  -p 7587:7587 timeplus/timeplus-enterprise:2.5.11

Then you can access http://localhost:8000 and create the first user, say admin, with password changeme.
Download the JDBC jar from https://github.com/timeplus-io/timeplus-native-jdbc/releases/tag/v2.0.7
You can create a connection for localhost and 7587 port (we also support streaming SQL using 8463 port, but the SQL will be long running and DBeaver will consider it hang)

PS. the code is largely based on the ClickHouse database support, with necessary SQL keywords or data type names changes.

Thanks for the contribution! We will review your PR and let you know if some changes are required


chore(ui5-application-writer): mapped manifest to ui5

TBI - Retrieve maintained list of mapped manifest-to-ui5 versions #839

Thank you @devinea and @Klaus-Keller


Option for no contract needed

The software assumes that a contract is required, and while I do like that there's a monthly price and intro price selection, even Cox states that though the price goes up after 12 months it does not require a contract. Attached is the very blurry label from their support page.

I see now that you can just clear the contract length, and that that's above in the notice, but way above. I suggest a little fa-info-circle icon next to the contract length that one can hover over and get a notice.


WMS server provides incorrect figure in the case of  dataset update

In the calculations for scientific flight planning, it may happen, that a datafile is updated, having the identical filename, but more timesteps. It seems that the server does not check for the update and returns false figures by not correctly interpolating within the data. After re-starting the user interface and clearing the cache, a correct figure is shown.
It is not clear, whether this is a problem of the user interface or of the server.
wrong figure:

correct figure:

In the shown case, the server was not re-started.

Please retry with emptied cache and afterwards with Cache disabled from the UI.

Reproduction of the error as follows:
There are two data sets, 20210501_00_clamsice_forecast.pl.nc.short (5 timesteps) and
20210501_00_clamsice_forecast.pl.nc.long (11 timesteps) including those of the short data set.
The data sets are copied to icg1805:/private.scratch/icg122/mssdata
On the server, also the data set from previous day is present.

copy short data set (5 timesteps) to the Server
start mss user interface
untick use cache
retrieve one figure (cloud_ice_mixing_ratio, fixed log scale, init 05-01 0:00 UT, valid, 05-01 0:00UT, 300hPa)
copy large data set to the server (overwriting the short data set)
switch to level 250 hPa and retrieve  -> wrong figure
start new top view panel in same session
retrieve identical figure -> correct figure

At the time the server gets a new file replacing the old one the client has to refresh getCapabilities. Thats in the example in 4. and 8.
We could think about to request on each retrieve a verification that the "getCapabilities" data of the client is outdated compared to the servers version.
When it is outdated retrieve happens after the update of GetCapabilities.

This described behaviour is by design.
An optional parameter updatesequence of the WMS standard can be used to enhance this behaviour.
updatesequence
Description: (Optional) The server may include an UpdateSequence value in its service metadata. If present, this value should be increased when changes are made to the Capabilities (e.g. when new maps are added to the service). The client may include this parameter in its GetCapabilities request.

What about the second TopView? Was the get_capabilities button pressed before retrieving the map?
The server only checks for changed files upon a get_capabilities call. This is typically sufficient for our "normal" use case of only "adding" and "removing" files. "Changing" was, so far, not high on our agenda.
This is a design decision to reduce the load on the server.
We could, probably, add a check that an accessed file has not been modified. The default access class cannot do that (because it is stateless and has no notion of its administrated files except their existance), but the cached access class could check for modification time...
We could use the CachedAccess class as default as well. We use it rather successfully operationally since three years now...


Timed actions

Would it be possible to include actions with a timer or pauze in between? For example. When I close a door, I'd like to light to go off after 5 minutes.

Look at this example
trigger:
  platform: state
  state: OFF
  entity: My Door
  for: 300


[nl/nodejs] date range updates for bard

updated date parser to handle "x years ago" or "decade ago" as a single date
when there is a single date, update timeline to handle that by highlighting that data point on the chart by showing a single dot there

thanks for the review!

Will take a look in a few mins.  Also please hold off one merge, since prod push is being triaged and might need cherrypick.


Add markdown-it plugins config docs

Following Issue #212 in the main lume repo, this adds an example of how to pass config options to markdown-it plugins.
I found it quite hard to express this in a clear way, because of the way we need to talk about plugins in two senses (the markdown-it plugin for lume and e.g. the anchor plugin for markdown-it) and the separate ways of setting options for these two kinds of plugin. I moved the markdown-it options example up to the general 'config' section, which I'm hoping might help make the distinction clear. But I'm not confident that I've found a clear and concise way of spelling out the example, so I'm happy to revise this PR if you don't think it's clear.
I also made a couple of other small changes to separate parts of the page, to fix a couple of typos. I hope you don't mind!

Great work. Thanks!


Revert "[FLINK-28046][connectors] Mark SourceFunction interface as @deprecated"

What is the purpose of the change
This PR reverts commit 07bf511a32b525f94d258daec347cf777b31bebb, which marks SourceFunction interface as @deprecated.
We need to resolve all subtasks in FLINK-28045 and then deprecated SourceFunction interface.
Brief change log
reverts commit 07bf511a32b525f94d258daec347cf777b31bebb
Verifying this change
This change is a revert without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no )

Documentation

Does this pull request introduce a new feature? ( no)
If yes, how is the feature documented? (not applicable)

We need to resolve all subtasks in FLINK-28045 and then deprecated SourceFunction interface.

The vote on deprecating the SourceFunction explicitly stated, that we can deprecate it without having resolved all the subtasks. [1]
[1] https://lists.apache.org/thread/kv9rj3w2rmkb8jtss5bqffhw57or7v8v


507117bc-5264-44f8-a4f5-e0f320efec75

this is for automated testing

007e2366-ff71-4ae1-828e-62af9635b6a1


Flexible size thread pool

Thread is created on demand, not when enter main function, and a monitor thread will recycle redundant idle threads.
I modified the worker thread routine,  use a thread local working flag to stop worker thread.
The working flag is like this:
static thread_local bool working;

If monitor thread detects redundant idle thread, it will enqueue a task item like this:
tasks.emplace([this]() { working = false; });

Then a idle thread will take it , and stop its loop.

"Thread is created on demand" means, when enqueue a task item, if there is no idle thread, a thread will be created to fetch this task item;
The variable maxIdle indicates thread pool size, default to std::thread::hardware_concurrency().  When threads are busy working,  thread pool size may be bigger than maxIdle, and after working, monitor thread will recycle the redundant threads, but pool size will never be smaller than maxIdle.


Reconnect after a restart mid-call

Closes #220.

Perhaps we can increment a counter on every connection so that we don't have any collisions on reconnect


Hang when typing special characters with non US keyboard

I'm running into an issue where the CodeEditorView hangs if you type some special characters. I'm using a Portuguese keyboard and I can always reproduce the issue in the preview by typing "dead keys", such as these characters: ~ and ´ which you get from the ' and ] keys on US keyboards, respectively. I'm using it on macOS, haven't tested iOS. Any ideas why, or any suggestions for workarounds?
EDIT: I've narrowed it down to the layoutManager.setRenderingAttributes calls in CodeStorage running in a thread. It seems bad things happen when you call AppKit on a background thread.
MacOS: 14.1
Xcode: 15.1

Thanks for the report. I will have a look.

Should be fine now. Please let me know if there are still problems.

Thanks for the quick fix! I don't see it hanging anymore. Really appreciate the effort.


[CSRanking] Fix a bug in archetype vs. concrete type ranking

Code completion related changes introduced a bug which increased
score2 regardless which type was an archetype, which surfaced
as a source compatibility regression in ReactiveKit.

@swift-ci please smoke test

@swift-ci please test source compatibility

cub failed with error: /Users/ec2-user/jenkins/workspace/swift-PR-source-compat-suite-debug-macos/swift-source-compat-suite/project_cache/cub/build/Build/Intermediates.noindex/Cub.build/Release-iphoneos/Cub iOS.build/Objects-normal/arm64/Cub-Swift.h: No such file or directory (in target 'Cub iOS' from project 'Cub')

swift-protobuf-plugin still has sandbox related issue.

Waiting for Relase build to finish.


🔏 Trust criteria for a third-party GitHub Action

User Need
As an Engineer
I want simple guidelines on how to decide whether to trust a third-party GitHub action for use
so that I am following the same criteria as the rest of the team.
Value
This ensures that we, as a team, apply the same criteria when assessing whether to use a third-party GitHub action
Functional Requirements:

Investigate suitable criteria for determining whether a third-party GitHub Action is trustworthy.
Record these in WOE 😢

Non-Functional Requirements:

Discuss criteria with team

Acceptance Criteria:

[ ] Criteria agreed with team
[ ] Criteria recorded in WOE 😢

Notes:

No longer required.


Subninjas with duplicate rules

Subninjas with duplicate rule names create conflicts.
This is an issue, since sometimes our project relies on another project that also supplies a build.ninja that we like to subninja in order to harness its dependency graph.
Is there a way to remedy this? It'd be nice if rules were isolated to their build.ninja file.

Nevermind; thought I had the latest but it was checked out at the last stable release. Duplicate of #921. My apologies; should have searched a bit better.


Dashboard Token Listing: USD+

Discord Handle of Submitter: maximlafe#8401
Token Symbol: USD+
Network: Polygon
Token Address: 0x236eeC6359fb44CCe8f97E99387aa7F8cd5cdE1f
Network Explorer Link: https://polygonscan.com/token/0x236eeC6359fb44CCe8f97E99387aa7F8cd5cdE1f
Is a Super Token? true
SVG Logo Link: https://imgur.com/a/FpeXn24
Coingecko or Hardcoded Price?: Hardcoded
Coingecko ID:
Harcoded Price (if Coingecko ID is unavailable): $1
Token HEX Color: 001845

CG is available as well https://www.coingecko.com/en/coins/usdplus

Is this actually a Super Token or just a plain ERC20? It appears to be the latter from your submission. What are the benefits have making this token render on the Dashboard?

Hey @sunnyjaycer, thanks for getting back to me.
We support 'ERC20 wrapper super token' https://github.com/ovnstable/ovnstable-app/blob/master/pkg/core/contracts/StaticUsdPlusToken.sol.
USD+ is a USDC pegged stablecoin that yields daily profit to its holders just for holding it! It works by taking all the leg work out for the holder and automatically reinvesting itself into other reward-yielding stablecoin strategies.
The user can freeze his money in USD+ at Superfluid at the begging of the month and he'll get a daily profit before he'll make payments at the end of the month.

@sunnyjaycer do you have updates by any chance?

Hey @maximlafe, could you please verify the contract at https://polygonscan.com/address/0xD81d9D5e1378593A63f979E3f6A9A43041e91eEf#code? This way we can know that it is in fact a streamable super token

@maximlafe Super Token wrappers don't play well with rebasing tokens at the moment, so I wouldn't advise continuing to pursue a listing with this. Feel free to join our Discord and start a deeper discussion if we can support you working out how to handle it!

Closing this for now, please feel free to reopen to continue the conversation or move to discord @maximlafe


CREATE  Error Handling and Dead Letter Queues

Hi, I am trying to do error handling and dead message queue handling, as shown in this confluent example: https://www.confluent.io/blog/kafka-connect-deep-dive-error-handling-dead-letter-queues/, My question is if I can do something similar with kc file pulse, is there any way to be able to send the records that do not comply with a specific structure or that come with errors to an error topic?, On the other hand, how can I throw an error when a record does not meet a valid structure, I am trying to throw an error when the example3 field does not exist:

"filters.ParseLine.columns":"example1:string;example2:string;example3:string",
"filters.SetKey.field": "$key",
"filters.SetKey.value": "$value.example1",
"filters.ValorToNull.type": "io.streamthoughts.kafka.connect.filepulse.filter.FailFilter",
"filters.ValorToNull.if": "{{ exists($value, 'example3') }}",
"filters.ValorToNull.invert": true,
"filters.ValorToNull.message": "Invalid row, example3 is empty : {{ $value }}"

I do not know if I am misusing the "exists" soon, I would appreciate your help too.
This is the data I'm reading:

12313112-453543-45334
23313112-153233-13324

and this is my config.

{
"connector.class": "io.streamthoughts.kafka.connect.filepulse.source.FilePulseSourceConnector",
"fs.scan.directory.path": "/tmp/kafka-connect/examples/",
"fs.scan.interval.ms": "10000",
"fs.scan.filters": "io.streamthoughts.kafka.connect.filepulse.scanner.local.filter.RegexFileListFilter",
"file.filter.regex.pattern": ".*\.txt$",
"task.reader.class": "io.streamthoughts.kafka.connect.filepulse.reader.RowFileInputReader",
"offset.strategy": "name",
"topic": "example-topic",
"internal.kafka.reporter.bootstrap.servers": "kafka1-service:9092",
"internal.kafka.reporter.topic": "streaming.kc-file-pulse-price.connect-status",
"tasks.max": 1,
"filters": "ParseLine,SetKey,ValorToNull",
"filters.ParseLine.trimColumn": "true",
"filters.ParseLine.separator": "-",
"value.converter": "org.apache.kafka.connect.json.JsonConverter",
"value.converter.schemas.enable": "false",
"filters.ParseLine.type": "io.streamthoughts.kafka.connect.filepulse.filter.DelimitedRowFilter",
"fs.cleanup.policy.class": "io.streamthoughts.kafka.connect.filepulse.clean.MoveCleanupPolicy",
"cleaner.output.succeed.path": "/tmp/kafka-connect/examples/succes",
"cleaner.output.failed.path": "/tmp/kafka-connect/examples/failed",
"filters.ParseLine.autoGenerateColumnNames": false,
"filters.ParseLine.columns":"example1:string;example2:string;example3:string",
"filters.SetKey.field": "$key",
"filters.SetKey.value": "$value.example1",
"filters.ValorToNull.type": "io.streamthoughts.kafka.connect.filepulse.filter.FailFilter",
"filters.ValorToNull.if": "{{ exists($value, 'example3') }}",
"filters.ValorToNull.invert": true,
"filters.ValorToNull.message": "Invalid row, example3 is empty : {{ $value }}"
}

Hello, is it possible to implement something like this? Or am I looking for other alternatives? Thank you

Hi @juancamilosalazar, the Dead Letter Queue is a mechanism which is part of Kafka Connect, but only supported for Sink Connectors only, i.e., you cannot use it with Connect File Pulse.
But, YES it should be possible to implement something similar.
The io.streamthoughts.kafka.connect.filepulse.filter.FailFilter will throw an exception and the file will be marked as FAILED. So, I think this is not the right Filter to use. Instead of that try to use the  AppendFilter.
You should be able to do something like this :
"filters":" CheckMissingField, SetDLQ"
"filters.CheckMissingField.type": "io.streamthoughts.kafka.connect.filepulse.filter.AppendFilter",
"filters.CheckMissingField.if": "{{ exists($value, 'example3') }}",
"filters.CheckMissingField.invert": true,
"filters.CheckMissingField.field": "error",
"filters.CheckMissingField.value": "Invalid row, example3 is empty : {{ $value }}",
"filters.SetDLQ.type": "io.streamthoughts.kafka.connect.filepulse.filter.AppendFilter",
"filters.SetDLQ.if":"{{ exists($value, 'error') }}",
"filters.SetDLQ.field":"$topic",
"filters.SetDLQ.value":"dql-topic"

Oh thank you very much, you just saved my life, I had not thought of using the AppendFilter, really thank you very much, greetings.


edited samples/charts READMEs

Literary review and edit of README files in the samples/charts directory.

Looks good to me.