[dependencies] Put dependabot config in vcs

Helps keeping everybody in the loop about which dependency can't be updated and why. I'm not sure how this works with the "ignore" commands so if you use them please also update this file.
This will also solve packages in monorepos blocking other updates. It should be sufficient to group these dependencies into a single PR. If this becomes an issue (e.g. rarely pass CI and need manual adjustments for each individual packages) we can improve.

Looks like it's being picked up :+1:


Improving task868_cfq_mcd1_explanation_to_sql.json

This task seems to be quite difficult. Wondering if there is a way for us to improve it.

@ujjwalaananth, can you have a look at this?

Difficult in terms of the explanation not being clear, or difficult as in the expected answers are too complicated? In the latter case, maybe removing { } and escape sequences like \n would make it much clearer, as they are only there for human readability I believe and the resulting query would still be correct, but more straightforward to learn.

For example, for the input "Did M0 's art director , director , producer , and writer executive produce , write , produce , and direct M1"
current output is "SELECT count(*) WHERE {\n?x0 ns:film.director.film M0 .\n?x0 ns:film.director.film M1 .\n?x0 ns:film.film_art_director.films_art_directed M0 .\n?x0 ns:film.producer.films_executive_produced M1 .\n?x0 ns:film.producer.film|ns:film.production_company.films M0 .\n?x0 ns:film.producer.film|ns:film.production_company.films M1 .\n?x0 ns:film.writer.film M0 .\n?x0 ns:film.writer.film M1\n}"
but maybe this could be simplified to   "SELECT count(*) WHERE x0 ns:film.director.film M0 . x0 ns:film.director.film M1 . x0 ns:film.film_art_director.films_art_directed M0 . x0 ns:film.producer.films_executive_produced M1 . x0 ns:film.producer.film|ns:film.production_company.films M0 . x0 ns:film.producer.film|ns:film.production_company.films M1 . x0 ns:film.writer.film M0 . x0 ns:film.writer.film M1"
(\n and braces removed)
(however I am not entirely sure this query is correct.)

Right, I am actually not sure. For now, let's keep the issue open, in case we have good ideas for solving it.


Create index.tsx automatically with a task?

Any suggestions on how to integrate the index export files automatically?
Its should be fairly trival but tried several index generators and they all s*
My idea was to add a generate-index pnpm task so i can automate this and ser incorrect function declarations more easy while porting everything over to my mino repro setup

Hi, sorry I didn't really try these kinds of tools.
In my opinion, when you are making a package, be it internal or bundled for npm, the goal is also to hide the internals. Then have a clean user-facing API with a few selected exports that you pick manually in your index.ts.


Error: undefined method `each_with_object' for nil:NilClass

brew doctor output
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed formulae are deprecated or disabled.
You should find replacements for the following formulae:
<EMAIL_ADDRESS>  youtube-dl

Warning: Unbrewed dylibs were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected dylibs:
  /usr/local/lib/mongosh_crypt_v1.dylib

Warning: You are using macOS 10.12.
We (and Apple) do not provide support for this old version.
It is expected behaviour that some formulae will fail to build in this old version.
It is expected behaviour that Homebrew will be buggy and slow.
Do not create any issues about this on Homebrew's GitHub repositories.
Do not create any issues even if you think this message is unrelated.
Any opened issues will be immediately closed without response.
Do not ask for help from Homebrew or its maintainers on social media.
You may ask for help in Homebrew's discussions but are unlikely to receive a response.
Try to figure out the problem yourself and submit a fix as a pull request.
We will review it but may or may not accept it.

Verification

[ ] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula.

brew config output
HOMEBREW_VERSION: 4.2.13-140-g9a1793a
ORIGIN: https://github.com/Homebrew/brew
HEAD: 9a1793a5ad5f400df6c2d53166a507c19bd02f93
Last commit: 12 hours ago
Core tap HEAD: e92c2161a31606d0666255070e33d437355b329e
Core tap last commit: 24 minutes ago
Core tap JSON: 23 Mar 14:05 UTC
Core cask tap HEAD: 7e6b88eb16e6ac9e076415e2b445174e039b764a
Core cask tap last commit: 65 minutes ago
Core cask tap JSON: 23 Mar 14:05 UTC
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_DISPLAY: /private/tmp/com.apple.launchd.9VUfK1OZUS/org.xquartz:0
HOMEBREW_MAKE_JOBS: 8
HOMEBREW_SORBET_RUNTIME: set
Homebrew Ruby: 3.1.4 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.1.4/bin/ruby
CPU: octa-core 64-bit sandybridge
Clang: 9.0.0 build 900
Git: 2.14.3 => /library/Developer/CommandLineTools/usr/bin/git
Curl: 7.54.0 => /usr/bin/curl
macOS: 10.12.6-x86_64
CLT: <IP_ADDRESS>.1.1510905681
Xcode: 9.2 => /library/Developer/CommandLineTools

What were you trying to do (and why)?
Installing requirements for Kivy iOS building.
What happened (include all command output)?
https://gist.github.com/ok-coder1/3c695cb41edd3a3ff60f40988e6b6cfb
What did you expect to happen?
It should install the requirements normally.
Step-by-step reproduction instructions (by running brew commands)
 brew install pkg-config sdl2 sdl2_image sdl2_ttf sdl2_mixer gstreamer autoconf automake

Sorry, we don't provide support for macOS 10.12. From the output of brew doctor:
Warning: You are using macOS 10.12.
We (and Apple) do not provide support for this old version.
It is expected behaviour that some formulae will fail to build in this old version.
It is expected behaviour that Homebrew will be buggy and slow.
Do not create any issues about this on Homebrew's GitHub repositories.
Do not create any issues even if you think this message is unrelated.
Any opened issues will be immediately closed without response.
Do not ask for help from Homebrew or its maintainers on social media.
You may ask for help in Homebrew's discussions but are unlikely to receive a response.
Try to figure out the problem yourself and submit a fix as a pull request.
We will review it but may or may not accept it.

Update: new brew doctor output. Deleted /usr/local/node.

Fixed with issue https://github.com/doxygen/doxygen/issues/10768


Feature: allow module cache-busting in development mode

Feature request
What is the expected behavior?
This feature is related to this issue: https://github.com/module-federation/module-federation-examples/issues/1025
The short of it is that I want to implement hot reloading for Module Federation, but for this to work I need some way to tell webpack to invalidate loaded modules so that the latest version of a Federated Module can be loaded.
A typical setup would look like this:

A host runs in a separate server instance
A remote (federated module) runs in its own webpack-dev-server instance
Changes are made to the remote, which rebuilds
In response to the changes, the host reloads the latest version of the remote, instantly reflecting the changes rather than reloading the page.

I have gotten most of the above to work, however, whenever I load a new version of the remote, Webpack still uses the old code. It seems as if once webpack has loaded a module, it will always fetch it from cache on future invocations, meaning newer versions are never loaded.
What is motivation or use case for adding/changing the behavior?
To have a hot reloading setup for Module Federation.
How should this be implemented in your opinion?
Assuming it's correct that module cache is the problem, there should be some way (e.g. a client-side API) for telling Webpack to invalidate all module caches, so that the next load of a module results in a fresh load of whatever code is available for it.
Are you willing to work on this yourself?
yes

There is a module.hot.invalidate() API, which allows to schedule a HMR update of a module. All invalidated modules will be re-executed on module.hot.apply().
But that might not work for RemoteModules, since they are loaded during chunk loading. Re-evaluating will not lead to loading them from a new remote, but loading them again from the same remote. So if your exposed module is self-accepting, it would work.

Eventually we probably would like to implement cross-container HMR, where the remote would expose a hot export, which bubbles the HMR event from the exposed module to the remote module.

@sokra thanks for the reply! Sorry if I'm misunderstanding, but would this imply that if I exposed my remote as a new container with each build, I could essentially achieve the same effect? I.e. get the latest compiled version every time? Is there a programmatic way to make sure the container is unique between each build for that purpose?

To answer the above - it makes no difference. Seems like the entire container needs to be busted for this to work (new, unique module IDs etc).

bump

bump

Guessing this is closed due to inactivity. Just pointing out to anyone finding it that it has not been resolved.

Is this completed?

Sorry, no


Best practices for streams mode

Hi Ashley,
Thank you so much for this awesome work, I am planning to use this library for my project, we have bunch SQS queues to process the messages from so I am planning to use streams mode to configure each input - output combination as stream.
My questions is how do I choose required CPU and memory, I am planning to run the entire process as docker container.
Is possible to have thread or go routine pool shared across streams?
For high availability, we may be running multiple instances of this setup then what should we consider for avoid duplicate processing. I know most of the work in done at SQS level it self but in cases there are bugs in out setup how do we monitor errors or duplicate processing.
Looking to hear from you soon.
Thanks,
Dilip

Hey @dilipkk-foyernet, selecting CPU and memory will probably need some trial and error as it'll depend on the size of the data you're processing and what exactly you're doing to it. When running multiple pipelines in streams mode they will share a thread pool equalling the number of threads on the machine, and for each individual stream you can configure a number of parallel processing pipelines that suits the task at hand (or set it to zero to match the number of CPUs).
For monitoring errors there are a few options available: https://www.benthos.dev/docs/guides/monitoring, each input, processor and output you configure will automatically emit metrics showing errors, but you can also use the metric processor for emitting custom metrics. For example, you could do a quick check for short term duplicates with something like this:
pipeline:
  processors:
    - cache:
        resource: foocache
        operator: add
        key: '${! json("message.id") }'
        value: "storeme"
    - catch:
      - metric:
          type: counter
          path: duplicates.detected

resources:
  caches:
    foocache:
      memory:
        ttl: 300

@Jeffail Thanks a lot for quick reply, I will try this and reach out to you if I have any further queries.


store profile IP addresses within omgd.cloud.yml file, make all other profiles git-able

move the IP address / hostname storage of each profile to omgd.cloud.yml
omgd:
  profiles:
    staging:
      servers:
        host: <IP_ADDRESS>

[ ] adjust storage of hosts via infra deploy command to write to omgd.cloud.yml
[ ] adjust profile.go to merge individual profile definition, should it exist, into profile, e.g. the above would read as omgd.profiles.staging.servers.host AND omgd.servers.host if the staging.yml file was loaded
[ ] ensure tests pass and builds work w/ example project

also considering dropping this. technically, the profiles don't store any information or serve any purpose except to have host names / ip addresses that the tool refers to. If such information is held in omgd.cloud.yml, other than needing to adjust the tool and profile structures somewhat awkwardly, then there's an open question as to whether the base profile files are even necessary. so maybe just keep this as is.

dropping this for now to move on, considering the above


Try to use pyenchant builtin methods like set_text

This maybe helps increase speed of service

Done, great speedup achieived


[6.0] Update the swift-docc, swift-lmdb, swift-docc-render-artifact, swift-…

…docc-symbolkit, swift-markdown, swift-llvm-bindings, and llvm-project to swiftlang org
(cherry picked from commit 0e6cd5580c610754bf6658a808c0fbd62418ccff)

@swift-ci test


Let multi_xml decide the best parser

Letting multi_xml decide to use the best available parser makes installation in a custom ruby environment (i.e.: Chef) easier.

Thanks for the pull request, @rveznaver. If a different parser is used is it guaranteed to have the same Ruby hash structure? That's really my only concern.

As far as I can tell, the multi_xml rspec tests expect all parsers to work equally (they are all tested against the same conversions). See parser_shared_example.rb.

Tests are passing, too, so we should be safe. Thanks again for the contribution. I'll cut a new version this week.

Sorry for the delay @rveznaver. I just pushed version 1.1.0 with this change.

@rveznaver Are you planning to write this in to a cookbook? I am planning on doing this in the near future.

Hi @dblessing, already done in the job provider for https://github.com/criteo-cookbooks/rundeck-server.

Awesome! When I get ready to work on this for us I will look at your cookbook and contribute more features if necessary. Thanks.

Do you plan to release to supermarket?

Yup, should be soon (maybe next week or so). Btw, we have made a set of three cookbooks:
https://github.com/criteo-cookbooks/rundeck-server
https://github.com/criteo-cookbooks/rundeck-bridge
https://github.com/criteo-cookbooks/rundeck-node
FYI, if you are managing Windows nodes, we have managed to modify the rundeck plugin to support WinRM over HTTPS client certificate so you can only distribute the public key to nodes (it is currently a fork, but we're working on merging it properly upstream).


Unsupported Media Type Error in Public API Server Logs

fixed here https://github.com/CounterpartyXCP/counterparty-core/pull/1811


Change Field select background-color in safari browser.

Bug report
How to change background-color of option inside  tag in safari.
https://jsfiddle.net/lebedevvladimir/vqskkgod
In chrome and firefox, it is working ok. The background of  is changing color inside select. But in safari browser, it has no effect.
I tried like *{background: rgba(0, 0, 0, 0.75)} inside whole select, it has no effect as well. Is it some modal window, I don't know.

This is unrelated to Redux Form. Ask at StackOverflow.


ticdc lag reached more than 12min when run ha_ticdc(owner)_to_ticdcProcessor(all)_network_partition

What did you do?
1、run tpcc with threads 10 and warehouse 1000
2、After 10 minutes, simulates the owner of ticdc is network isolated from all other processor of ticdc
fault start time：2023-06-13 08:00:03
3、After 10 minutes, recovery the fault
fault recover time：2023-06-13 08:10:04
What did you expect to see?
changefeed lag is less than 30s
What did you see instead?
changefeed lag reached more than 12min after inject fault

Versions of the cluster
git hash : 1e2f277f2e3d9b57b15db9a2a9b2c62832c071ca
current status of DM cluster (execute query-status <task-name> in dmctl)
No response

/remove-area dm
/area ticdc

The owner and the processor keep themselves alive by watching keys in etcd. Since the network between TiCDC and PD is normal, the TiCDC node will not restart after a network partition among TiCDC nodes. However, the owner will collect table replication progress through P2P messages. After a network partition, the owner cannot receive the table replication progress, resulting in increased lag.

By design. I suggest we address it in long term.

TiCDC functions as a distributed system. The owner collects table progression data from the processor through direct communication, which is used to advance a table's barrierTs. This signals to the processor that data preceding the barrierTs is ready for downstream flushing. So, if there's a network partition between the owner and any processor, it's not possible to advance a changefeed.


add throwing wrap

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #1575 (b4eb035) into master (c55003f) will decrease coverage by 0.75%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #1575      +/-   ##
============================================
- Coverage     63.72%   62.96%   -0.76%
+ Complexity     2228     2222       -6
============================================
  Files           450      451       +1
  Lines          9400     9434      +34
  Branches        949      961      +12
============================================
- Hits           5990     5940      -50
- Misses         2935     3019      +84
  Partials        475      475

Impacted Files
Coverage Δ

...ava/org/apache/shenyu/common/enums/PluginEnum.java
100.00% <ø> (ø)

...ache/shenyu/common/exception/ThrowingConsumer.java
0.00% <0.00%> (ø)

...ache/shenyu/common/exception/ThrowingFunction.java
0.00% <0.00%> (ø)

...n/java/org/apache/shenyu/plugin/jwt/JwtPlugin.java
43.33% <ø> (ø)

...enyu/client/tars/TarsServiceBeanPostProcessor.java
34.66% <0.00%> (-56.00%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
56.84% <0.00%> (-7.37%)
:arrow_down:

...in/alibaba/dubbo/cache/ApplicationConfigCache.java
75.86% <0.00%> (-4.60%)
:arrow_down:

.../shenyu/admin/service/impl/AppAuthServiceImpl.java
89.79% <0.00%> (-4.26%)
:arrow_down:

.../sync/data/zookeeper/ZookeeperSyncDataService.java
71.29% <0.00%> (-3.96%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
71.84% <0.00%> (-2.92%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


Tabs are cut off after resize

Hi,
I'm using your plugin together with react-swipable-views to create swipable tabs and I like your plugin a lot.  I have encounter problem with resizing window. When I resize the window the position of tabs is not recalculated correctly which leads to cutoff tabs even when there is enough space to show them all (see pic below).

Could you please check it out? Or If you would be interested I could have a look and submit PR (not sure when I will get to this though).
Thanks

Hi Tomas,
Thanks for your email. I'm glad that you liked this plugin. I built it for
one of my projects and I think I noticed this issue.
I will check it out later tonight and push a fix but can you please open an
issue on github just to keep track.
Best,
Kareem
On Wed, Jul 19, 2017 at 9:39 AM, Tomas Pustelnik<EMAIL_ADDRESS>wrote:

Hi,
I'm using your plugin together with react-swipable-views to create
swipable tabs and I like your plugin a lot. I have encounter problem with
resizing window. When I resize the window the position of tabs is not
recalculated correctly which leads to cutoff tabs even when there is enough
space to show them all (see pic below).
[image: screen shot 2017-07-19 at 09 37 07]
https://user-images.githubusercontent.com/10606260/28355804-e03ca492-6c65-11e7-8e1f-89293d3d0796.png
Could you please check it out? Or If you would be interested I could have
a look and submit PR (not sure when I will get to this though).
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bitriddler/react-swipeable-tabs/issues/2, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACUww4wP7rruNP54oyiUGspHdUgp_RzFks5sPbK2gaJpZM4OcV22
.

Ahhh you already made an issue, never mind then, I will check it later
tonight and push a fix.
Thanks,
Kareem
On Wed, Jul 19, 2017 at 11:47 AM, Kareem Mohamed<EMAIL_ADDRESS>wrote:

Hi Tomas,
Thanks for your email. I'm glad that you liked this plugin. I built it for
one of my projects and I think I noticed this issue.
I will check it out later tonight and push a fix but can you please open
an issue on github just to keep track.
Best,
Kareem
On Wed, Jul 19, 2017 at 9:39 AM, Tomas Pustelnik<EMAIL_ADDRESS>
wrote:

Hi,
I'm using your plugin together with react-swipable-views to create
swipable tabs and I like your plugin a lot. I have encounter problem with
resizing window. When I resize the window the position of tabs is not
recalculated correctly which leads to cutoff tabs even when there is enough
space to show them all (see pic below).
[image: screen shot 2017-07-19 at 09 37 07]
https://user-images.githubusercontent.com/10606260/28355804-e03ca492-6c65-11e7-8e1f-89293d3d0796.png
Could you please check it out? Or If you would be interested I could have
a look and submit PR (not sure when I will get to this though).
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bitriddler/react-swipeable-tabs/issues/2, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACUww4wP7rruNP54oyiUGspHdUgp_RzFks5sPbK2gaJpZM4OcV22
.

Thanks for quick reply and for the effort.

I have pushed a fix for this issue
00bea82427a447495b1b483d18fcc1ebd405c035
Please update to version @2.1.0 and let me know if it works so I can close this issue

Will check it out. Thanks.

So I have checked to issue with the new 2.2.0 version and it is still there. Problem seems to be connected with the translate property on wrapper div element (see pic). After resize of the window this should be recalculated but it isn't and the tabs are still acting like they have the previous size.


await menuService.SaveAsync(menu); System.InvalidOperationException:“Operation is not valid due to the current state of the object.”

await menuService.SaveAsync(menu);
Crash here.
System.InvalidOperationException:“Operation is not valid due to the current state of the object.”

Please follow the documentation
https://winuicommunity.github.io/winUICommunityContextMenuExtensions/
Do you use unpackage mode? If yes, this needs package identity and only works in packaged mode.


AAPT2 when linking resources gives error

AGPBI: {"kind":"error","text":"error: resource android:attr/foregroundInsidePadding is private.","sources":[{"file":"C:\Users\MyPC\.gradle\caches\transforms-1\files-1.1\sdp-android-1.0.5.aar\6ace9734fe0036a609208f5ef2a38545\res\values\values.xml","position":{"startLine":2,"startColumn":4,"startOffset":55,"endColumn":42,"endOffset":93}}],"original":"","tool":"AAPT"}
Failed to process resources.

Hi @akhilsreekar do you have steps to reproduce this issue?

I am getting the same issue does anyone got any success.?
Thanks

I just updated My complileSdk and targetSdk to 28
compileSdkVersion 28
targetSdkVersion 28
And it worked for me


🐛 BUG: workers.api.error.zone_not_found for "/"

Which Cloudflare product(s) does this pertain to?
Workers for Platforms, Wrangler core
What version(s) of the tool(s) are you using?
3.9.0
What version of Node are you using?
18 LTS
What operating system are you using?
Linux
Describe the Bug
When publishing a Worker using a wildcard pattern and zone_name, the API returns error 10083:
wrangler.toml
name = "projectdemo"
route = { pattern = "*/*", zone_name = "testing.ceru.dev" }
Console
[ERROR] A request to the Cloudflare API (/accounts/2dcc5a4f486a50625b4167e90d113158/workers/scripts/projectdemo/routes) failed.

  workers.api.error.zone_not_found for "*/*" [code: 10083]

This is the second time this error has been fixed, then broken again, then fixed. Is there no testing for this usecase?
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
No response

There's a separate issue here to improve the workers API error message (workers.api.error.zone_not_found for "*/*" makes no sense when searching by name) - have forwarded it to the relevant team internally.

@AggressivelyMeows I'm not able to reproduce this behaviour when testing on a test zone. It may be that testing.ceru.dev is a subdomain of a zone rather than a zone itself?
Did this work in a previous version of Wrangler?


Add attachment content-type handling and appropriate attachment rendering

Implement content-type specific rendering of the HipChat message for the addAttachmentToCard event.
If the attachment content-type is an image it is render using  tag, otherwise it is render as url.

Thanks @MariyaNizovtsova - am closing this now to keep it out of the inbox.


import statement not working

I'm having this issue here http://stackoverflow.com/q/39909200/1267778 where a simple import statement does not work for d3-request.
import * as d3select from 'd3-selection';
import * as d3request from 'd3-request';

var mysvg = d3select.select("#mysvg"); //works
var request = d3request.xml('assets/emojis/laughing.svg'); //doesnt work

The d3select call gets properly compiled to the exported "select" function. However the resulting code for the second line compiles to something like:
var d3Request$1 = unwrapExports(d3Request);

var d3Request$2 = Object.freeze({
    default: d3Request$1,
    __moduleExports: d3Request
});

And then tries to call  d3Request$2.xml, which doesn't exist.

It looks like whatever bundled you are using is not importing the code from the ES2015 source, per the module (and jsnext:main) entry point defined in the package.json. I am not familiar with the bundler you are using, and you didn’t post that part of your code, but that’s where I would look.

I'm trying to load d3 in an Ionic2 app. They have settled for Rollup as the bundler and here is the default script the doesn't the bundling: https://github.com/driftyco/ionic-app-scripts/blob/master/config/rollup.config.js
I'm new to a lot of concepts here, how can I edit the build script to include the correct entry point for d3?


Use out_dirs for next macro output

We were seeing readonly filesystem errors on Linux when using a single file out. This outputs a directory and out_dirs produces the correct behavior.

 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.

Thanks for catching that. It should have been out_dirs from the start.


Simplify Existing entities

Existing entities do not need to be as complex and thorought as the awaiting ones. As an example, there is no need to have a complete nested Category inside an ExistingChannel, only it's name (and id). Then, existing nested entities can be found using the existing guild.
Challenges

How to build Discord's requests from the commands and the commander (using id's instead of names)
For channels, we currently pass the roles list and categories list, but would probably need to pass the entire existing guild instead

Maybe it's better to pass it as a trait, like a GuildEntityQuerier?

Need to reasses, might not be needed because it actually adds more complexity and checks when updating the existing guild (aka if building complex references already needs valid existing entities, there is no need to recheck the validity of the references).


No special case for Game Tie

When the game is Tie (no user wins the game) there is no options to restart the game. Please assign this task to me and mark the labels.

When the game is Tie (no user wins the game) there is no options to restart the game. Please assign this task to me and mark the labels.

go for it , please assure putting a short video before pr and all changes will be in new branch

Okay and please check my other pr I have raised. I am doing this Thank you.

@iamparas0 I have raised the pr for that and added videos.
##This is before PR
https://github.com/user-attachments/assets/9a51fe1a-b1c4-413b-a0a9-0596001d7900
##This is after PR
https://github.com/user-attachments/assets/9e1425f3-8b86-4a4c-9fbd-5126e1198989


Enumerable Find, better performance

Find is Enumerable and yields {Path, File::Info}
Use push/pop instead of shift/unshift
follow_symlinks: false to avoid infinite loops
Benchmarks
Before
 Find.find double stat   1.03k (970.82µs) (± 3.70%)  53.2kB/op        fastest

After
 Find.find double stat   2.72M (367.65ns) (± 5.59%)   224B/op   4.96× slower
  Find.new double stat  12.63M ( 79.19ns) (± 7.76%)  96.0B/op   1.07× slower
  Find.new single stat  13.50M ( 74.09ns) (± 9.76%)  96.0B/op        fastest

Find.new double stat doesn't perform File.exists explaining the speed boost from Find.find double stat.
All double stat runs are for comparison with the original code.
Benchmark:
require "benchmark"
require "./src/find"

stats = Hash(Symbol, Int32).new { |h, k| h[k] = 0 }

Benchmark.ips do |bm|
  bm.report "Find.find double stat" do
    stats.clear
    Find.find(".") do |path|
      if info = File.info? path, follow_symlinks: false
        if info.file?
          stats[:file] += 1
        elsif info.directory?
          stats[:dir] += 1
        end
      end
    end
  end

  bm.report "Find.new double stat" do
    stats.clear
    Find.new(["."]).each do |path, _|
      if info = File.info? path, follow_symlinks: false
        if info.file?
          stats[:file] += 1
        elsif info.directory?
          stats[:dir] += 1
        end
      end
    end
  end

  bm.report "Find.new single stat" do
    stats.clear
    Find.new(["."]).each do |path, info|
      if info.file?
        stats[:file] += 1
      elsif info.directory?
        stats[:dir] += 1
      end
    end
  end
end

Enumerable uses:
One line du:
p Find.new(ARGV).select { |_, info| info.file? }.map { _, info.size }.sum

Parallel file processing with concurrent.cr
Find.new(...).parallel.select { |path, info|
  info.file? && path.to_s =~ /\.json/
}.run { |path, _| }
  File.open(path) do |rfile|
    File.atomic_rewrite("#{path.basename}.newext" do |wfile|
      ...
    end
  end
}.wait

Pondering this more, I think File.exists? should be removed

It's utility is dubious.  Either a dangling symlink was followed or there is a race condition which can NOT be solved with File.exists?
Performance penalty especially on remote file systems


Building from source for specific target and submitting source code for security scan

How can I select the source and build it for a specific target environment where I am required to deliver source code for open source components? In this case, the target is ASP.NET, and I need to submit corresponding source code for security scanning.
Thanks!

You can just check out the source code and run build.cmd. That will build for all environments and the artefacts will be in th /build/ directory after completion.
Note that building from source will produce binaries with a different public key to the released binaries as I do not publish the signing certificate for release builds.

Thanks, Jeremy!


I'm very interested in this project!

I'm having trouble getting a demo app to compile and run. Would you be willing to create a very simple example app to get a new users for your package up and running?

Could you please provide some more information about this?

Which Xcode version are you running?
Which error(s) are visible in Xcode when trying to build?
What is the platform you are trying to compile your example app for?
How did you link this library using Swift Package Manager? This project is currently not available on Cocoapods and if you have not previously used Swift Package Manager, this might also be the issue here.

As an example, you can have a look at this project of mine that works on macOS and iOS. It uses this library for the VisualizationPage view.

At first I was not importing MapKit, which obviously wasn't finding any of the associated types. So when I tried to import MapKit it would give the compiler a segmentation fault. I'm using the latest Xcode (Version 13.3.1 (13E500a)), with an empty created Project for iOS. Then adding Map with SPM. I loaded your test project ( thank you by the way for sharing ), with the same result. I quit and restarted Xcode and everything is working as expected. My first project worked as expected after the restart. I'm sorry I bothered you with such a trivial task, I should have tried that first... threw me though since I had other projects open that were still able to compile and if I commented out the Map(...) in my original project it compiled.

no worries - glad it worked now 😊


DO NOT MERGE - trobbins/pr build/dr bootstrapping

This branch contains some hacks to get the service up and running in a different env.
I am using the PR to see diffs more clearly on github.

Currently nothing is used in this branch, there are just some problems under the hood in our CI and this branch is used to get a new SHA.  Ideally CI is improved soon.


feat: update tokens

:tada: This PR is included in version 1.94.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore(master): release 1.1.175

:robot: I have created a release beep boop
1.1.175 (2023-09-02)
Miscellaneous

deps: update dependency @types/node to v18.17.14 (6890f79)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/keywind/releases/tag/v1.1.175 :sunflower:


LinkChooserPanel / LinkChooserBlock

It would be very nice to be able to use the link chooser as a panel and a block.
I would be happy to help but I am not exactly sure what would be involved.  It seems like a lot of code is already there, but it is not clear to me how to hook the js for a chooser panel / block.

This should be pretty straight forward, wagtail.wagtaildocs may be a good starting point for both types of panel.


Impossible to create a lib compatible with version 7 and 8

What version of React, ReactDOM/React Native, Redux, and React Redux are you using?

React: 18
ReactDOM: 18
Redux: 4.2
React Redux: 7 or 8

What is the current behavior?
When we want to create a lib with a peer dependency to react-redux we face a problem.
People that use react 18 will provide react-redux 8. People that work with react 17 will provide react-redux 7.
The lib is doing basic things, but it uses the connect with pure option to true.
In v8, pure option is no more necessary but it throws an error to say the option is not necessary.
This exception makes the full app crash.
It is not possible to remove the option to let v7 correctly works.
We have this case in an unmaintained lib react-beautifull-dnd that blocks the migration of apps to react 18.
What is the expected behavior?
A console warning can be displayed but not a blocking error, mostly if this parameter is not used at all.
Which browser and OS are affected by this issue?
All
Did this work in previous versions of React Redux?

[X] Yes

Hmm. My first answer here is that you shouldn't have to be passing in pure: true in the first place even in v7, because that's already the default and has been for years.

The problem is react-beautifull-dnd is unmaintained since 1 year but still really used. This one is blocking our migration to react 18 / react-redux 8.
Changing from error to warning will unlock the problem.
More the error is only thrown in dev mode. So it can be good to only warn not ?

@paztis there's a fork of react-beautiful-dnd with fix in progress, maybe the fork is an more likely alternative

I've take a look to this fork. It only tries to update react 18 dependency.
I don't know how it can work. Only version 8 of react-redux can support react 18 ?

Just published https://github.com/reduxjs/react-redux/releases/tag/v8.0.2 , which should fix this.


process: merge dev branch into master

Hi, 👋
This is a simple question in regards the release process of this repository;

When does 'dev' branch gets merge to 'master'?

Thanks in advance.

This happened.


Cannot compile against nibbles master branch

; compiling file "/home/dx/common-lisp/sharplispers/ironclad/src/digests/whirlpool.lisp" (written 14 OCT 2019 04:10:25 PM):
; compiling (IN-PACKAGE :CRYPTO)
; compiling (DEFTYPE WHIRLPOOL-REGS ...)
; compiling (DEFUN INITIAL-WHIRLPOOL-REGS ...)
; compiling (DEFCONSTANT +WHIRLPOOL-REGS-HASH-OFFSET+ ...)
; compiling (DEFCONSTANT +WHIRLPOOL-REGS-K-OFFSET+ ...)
; compiling (DEFCONSTANT +WHIRLPOOL-REGS-STATE-OFFSET+ ...)
; compiling (DEFCONSTANT +WHIRLPOOL-REGS-L-OFFSET+ ...)
; compiling (DEFCONST +PRISTINE-WHIRLPOOL-REGISTERS+ ...)
; compiling (DEFUN WHIRLPOOLREGS-DIGEST ...)
; compiling (DEFCONSTANT +WHIRLPOOL-ROUNDS+ ...)
; compiling (DECLAIM (TYPE # ...))
;
; caught ERROR:
;   READ error during COMPILE-FILE:
;
;     The function NIBBLES::%UB32REF/BE is undefined.
;
;     (in form starting at line: 135, column: 0, position: 4736)

; compilation aborted after 0:00:00.044

I tried and didn't have any issue (with nibbles at commit fb9d55bc0921099af112186293434cf87321f2e2, Ironclad at commit d76008b10ff02fbae878f93de6c66d152eb23bc7 and SBCL 1.5.7).
What OS and what compiler version are you using?

Nibbles is no longer a required dependency for Ironclad, so Iet's close this issue.


Switch to terraform-plugin-framework

Replace SDKv2
https://github.com/hashicorp/terraform-plugin-framework

Currently no imports or validation

It's worth reevaluating at some point, but closing this for now.


Maybe add dependabot?

I don't know how this works so I'm just going off what other dotnet repos do. Should be good to merge but I don't know if it's enough...

I was relatively certain this was automatically detected anyways. I'll review and get back to you

I was relatively certain this was automatically detected anyways. I'll review and get back to you

Eh can't be bothered with this

Reopening this as I'd quite like to get it in. @HurricanKai did you find out anything about this?
It looks like Dependabot doesn't pick up on a repo (unless there's a security vulnerability in a package) by default, so we have to enable it using a YAML file but if you've found otherwise let me know.


Avoiding code duplicated and directives that break statements.

Hello?

I'm having a hard time to see what this fixes.
Can you elaborate?

please reopen this ticket when you provide more information, thank you.

@jsorg71 , @itamarjp , sorry for delay.
this change don't fix anything, it's  only aesthetic, it's a suggestion to avoid the use of conditional directives that break statements i think that kind of concern improve the code maintainability do you agree?

OK, you are trying to address a very minor style issue here (preprocessor conditional cutting through if block), but you are creating another issue (bad variable name, it should be is_password or something like that, definitely not test_access). In the same time, you are missing bigger issues in that piece of code.
What is ACCESS anyway? It's always defined in xrdp_login_wnd.c and xrdp_mm.c. Why would anybody want to undefine it? If nobody needs to undefine it, let's just remove it.
Why are we using number 255? If that some magic number? Would g_strcmp do the job?
Why is the comparison case sensitive? What if the field is named "Password"? Any reason to treat it differently?
Finally, it would be great to allow internationalization of the login dialog. There should be a way to specify the label and a meaning separately. That's a big task, but I think it's worth the trouble.

Please see #632 for an alternative approach.

Ok, thanks

This issue can be closed now.


Typo "Azure queue"→"Azure Queue"

https://docs.microsoft.com/en-us/azure/storage/queues/storage-python-how-to-use-queue-storage?tabs=python
#PingMSFTDocs

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

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

The revised text appears to refer generically to an instance, not to an Azure service name. We'll close the PR as inapplicable in this case.
For more information, see https://github.com/MicrosoftDocs/azure-docs/pull/74952#issuecomment-834575322.


mosquitto

In the manual for installing the lattice connect self there is an type error:

Stop mosquitto
$ root@: service mostquitto stop

The right service on the lattice is: mosquitto not mosTquitto

In the manual for installing the lattice connect self there is an type error:

$ root@: service mostquitto stop

The right service on the lattice is: mosquitto not mosTquitto


"Custom controls" example not working

I'm trying to follow the "Custom controls" example from the react-quill README as I'd like to customize the toolbar that I'm using:
var MyComponent = React.createClass({
  /* ... */

  render: function() {
    return (
      <ReactQuill>
        <ReactQuill.Toolbar key="toolbar"
                            ref="toolbar"
                            items={ReactQuill.Toolbar.defaultItems} />
        <div key="editor"
             ref="editor"
             className="quill-contents"
             dangerouslySetInnerHTML={{__html:this.getEditorContents()}} />
      </ReactQuill>
    );
  }
});

However, whenever I copy that code, I'm getting the following error:
Uncaught (in promise) TypeError: this.getEditorContents is not a function(…)
I believe the issue is that the this used to set dangerouslySetInnerHTML refers to my outer react component's this, not the this of the ReactQuill component. I tried to set a ref and then replace this with the ref variable, but the ref variable is not defined by the time the dangerouslySetInnerHTML function is run.
Maybe the code example works when react-quill is setup as a Mixin? How would I get it to work in the case where I'm embedding ReactQuill as a component within an outer component?
Thanks for your help! And overall awesome wrapper library!

I figured out my own issue: the getEditorContents() is not intended to be a call to the react-quill component getEditorContents(), but simply a function in your own class that get's whatever the initial state you want for the editor.


fix(combine-api): fixed reading shared configuration

Addresses #3867

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


Update for thermodynamics and Crystal_output

@bcamino
Ciao Bruno,
There are several updates:

2 base objects are defined to read convergence of SCF and optimisation cycles. The user can return the convergence of energies, gradients and displacements with the new 'get_convergence' method
Fix the issues with spin-polarized systems of read_bandgap and read_fermi and make it possible to get the convergence history of both values.
Convert the numerical Hessian matrix output by CRYSTAL into the Phonopy readable 'FORCE_CONSTATS' format.
Bug fix of pymatgen version requirement

All the old functions in Crystal_output, including get_final_energy get_opt_energy etc. are kept to maintain consistentancy. The only drawback is that I removed the support of read_fermi to band and DOS calculation outputs for now (now it works only for SCF calculations). I am planning to add them back in the next PR. I am also open for discussion if you have any concerns.
Kind regards,
Huanyu

@Spica-Vir , sorry for the delay, I missed this.


[MRG] early decode bytes returned by check_output

Decode bytes strings into unicode strings as soon as we get the output of a subprocess command.
Let's wait for travis to check that this is working properly.

Before merging I would like to add another test.

Nice addition !

The new test uses non-ascii UTF-8 encoded chars. If both SLURM and SGE accept it I will document the encoding parameter. If not I will refactor the tests to:

for ASCII encoding
check that non-ASCII job name fail with a ValueError with a meaningful message.

@arjoly I have updated the docstest to have them pass.
The submit function now always return a utf-8 encoded by string. That might be confusing for some users but it makes it always safe to call os.system on that whatever the version of Python.
Alternatevely, I could change that so that the submit function always return a unicode string. This would probably feel more intutive in most cases. If the user put non-ascii chars in the job_name parameter that would be his/her responsability to encode the output of the submit to utf-8 prior to passing it to os.system or subprocess.call. WDYT?

BTW, both SGE and SLURM seem to accept UTF-8 encoded, non-ascii characters in the job name.

The submit function now always return a utf-8 encoded by string. That might be confusing for some users but it makes it always safe to call os.system on that whatever the version of Python.

Why it's confusing?
I like this option. For me, it seems to be the user friendly choice and always gives stable output.
In which extent is the backward compatibility preserved?
(strings have still a part of mystery for me).

Coverage decreased (-0.08%) to 93.52% when pulling 350f0b89dd7cdbc8c7abeaa4c1688633b0f0b119 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.08%) to 93.52% when pulling 350f0b89dd7cdbc8c7abeaa4c1688633b0f0b119 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.08%) to 93.52% when pulling 350f0b89dd7cdbc8c7abeaa4c1688633b0f0b119 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-6.91%) to 86.69% when pulling 586f0892a6ad7205e0acdbd2b4d476763e306daa on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-6.91%) to 86.69% when pulling 586f0892a6ad7205e0acdbd2b4d476763e306daa on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-17.79%) to 75.81% when pulling 6575acbd8e9aaca8947dd876d5b0e9ecf3c8e705 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.45%) to 93.15% when pulling 46bd7b43dff31eb4db1e7a0cca7401c66dc2ba65 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.45%) to 93.15% when pulling 46bd7b43dff31eb4db1e7a0cca7401c66dc2ba65 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.05%) to 93.55% when pulling b40515e14440bf0df5920f9a7cebedcaebab5614 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.05%) to 93.55% when pulling b40515e14440bf0df5920f9a7cebedcaebab5614 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.05%) to 93.55% when pulling b40515e14440bf0df5920f9a7cebedcaebab5614 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Why it's confusing?

Because users might expect to do (unicode) string operations (such as calling print on it) on them that don't work well in Python 3 without first decoding. For instance to print a byte string for visualization you have to decode it first.
In the current state of this branch, that return encoded byte string you get the following behavior with Python 3:
>>> print(submit(command="my_program", job_name="test_job"))
b'echo \'#!/bin/bash\nmy_program\' | qsub -N "test_job" -l h_rt=24:00:00 -l h_vmem=4000M'
>>> print(submit("my_program", job_name="test_job", backend='sge').decode('utf-8'))
echo '#!/bin/bash
my_program' | qsub -N "test_job" -l h_rt=24:00:00 -l h_vmem=4000M

Python 2 does it implicitly for you but it can be problematic if the platform specifc encoding used by Python 2 is not the one that was used to encode the string in the first place.

For me, it seems to be the user friendly choice and always gives stable output.

If we return unicode string (that is Python 2 unicode or Python 3 str), they are still stable.

In which extent is the backward compatibility preserved?

If we make the submit method return the unicode string you this will be backward compatible

I pushed a new commit with that change. If you like the current state. I will add a new comment to add the docstring for the encoding parameter of the queued_or_running_jobs.

Alright I added the doc. I also updated the users guide to give some insight on how to properly encode the command generated by submit when the job_name parameters contains non-ASCII chars. I can squash the commits if you wish.

Coverage decreased (-0.05%) to 93.55% when pulling 8587808e3fcf11b5854c947d45bcadaf659db11d on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.05%) to 93.55% when pulling 8587808e3fcf11b5854c947d45bcadaf659db11d on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Before a final review, do you also prefer this solution over the unicode one?

Before a final review, do you also prefer this solution over the unicode one?

I prefer the current state of this PR.

Can you remove file.txt?

Can you highlight your contribution in the what's new?

Beside the above comment, LGTM!

Coverage increased (+0.36%) to 93.96% when pulling 5a331d66adabf5bf584997a3f2e832afc8db617a on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage increased (+0.36%) to 93.96% when pulling 5a331d66adabf5bf584997a3f2e832afc8db617a on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage increased (+0.36%) to 93.96% when pulling 5a331d66adabf5bf584997a3f2e832afc8db617a on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.1%) to 93.5% when pulling ea7a509fcd0abf9302c60fe635199306970a4cfd on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.1%) to 93.5% when pulling ea7a509fcd0abf9302c60fe635199306970a4cfd on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Comments addressed. This should be good to merge. Sorry for the leftover file.txt file.

Coverage decreased (-0.1%) to 93.5% when pulling a7a361c793ef1b73efaa39e9afee8b5831dc9276 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Coverage decreased (-0.1%) to 93.5% when pulling a7a361c793ef1b73efaa39e9afee8b5831dc9276 on ogrisel:encoding-fix into c9c450ff9c329bfbedd2e232f10bbf8ef4bc6aa2 on clusterlib:master.

Merge by rebase !!!
Thanks @ogrisel for improving clusterlib!!!


Elkarte 2.0 - Template error on RSS

As described here https://www.elkarte.net/community/index.php?topic=6072

Thank you for posting this here 😸

Fixed on my local

don't forget to update elkarte.net 👍


[QUESTION] How to get Cherami to Spread the Load

I'm running  a Cherami cluster using docker containers. I have:

3x Frontend Hosts (0.5GB mem)
3x Controllers (0.5GB mem)
3x Inputhosts (2GB mem)
3x Outputhosts (2GB mem)
5x Storage hosts (4GB mem)

And Cassandra with 5 nodes, and a RF of 3.
However when we have times of heavy load across the system, most of the boxes sit idle with one storage host that climbs to all its memory and then is killed (OOM).
Is there a way to tell Cherami to spread the load to the other storage hosts? It seems all the extents are placed on the one host, and even when that hosts starts slowing down and getting behind, the load is never shared.
Cassandra is aware of all nodes, each node seems to be working
Many thanks!

For reference my current configuration is:
DefaultServiceConfig:
  ListenAddress: "XXXX"
  RingHosts: "frontend-0.cherami:4922,frontend-1.cherami:4922,frontend-2.cherami:4922"
  EnableLimits: true

# ServiceConfig overrides default config with service-specific config such as ports
ServiceConfig:
  cherami-inputhost:
    Port: 4240
    WebsocketPort: 6189
    tchannel:
      port: 4240
      disableHyperbahn: true
      logLevel: warn
      disableLogging: true
  cherami-storehost:
    Port: 4253
    WebsocketPort: 6191
    tchannel:
      port: 4253
      disableHyperbahn: true
      logLevel: warn
      disableLogging: true
  cherami-outputhost:
    Port: 4254
    WebsocketPort: 6190
    tchannel:
      port: 4254
      disableHyperbahn: true
      logLevel: warn
      disableLogging: true
  cherami-frontendhost:
    Port: 4922
    tchannel:
      port: 4922
      disableHyperbahn: true
      logLevel: warn
      disableLogging: false
  cherami-controllerhost:
    StorePlacementConfig:
      MinFreeDiskSpaceBytes: 10000
    Port: 5425
    tchannel:
      port: 5425
      disableHyperbahn: true
      logLevel: warn
      disableLogging: false
  cherami-replicator:
    Port: 6280
    WebsocketPort: 6310
    tchannel:
      port: 6280
      disableHyperbahn: true
      logLevel: warn
      disableLogging: false

DefaultDestinationConfig:
  Replicas: 1

MetadataConfig:
  CassandraHosts: "cassandra"
  Keyspace: "cherami"
  Consistency: "one"
  ClusterName: "test-cluster"
  NumConns: 1

StorageConfig:
  BaseDir: /var/lib/cherami-store/data
  HostUUID: "XXX"

logging:
  level: info
  stdout: true

The environment peaks at maybe 4000 messages a second, averages a couple hundred - We see cherami back up under the load at every peak time, and usually when this happens one of the storehosts will have hit its 4GB cap, while the others will be laying around a 100MB or so. Same with the output host except it hits its 2GB cap while the others sit firmly on 20MB.
Any help from the active devs @datoug / @kobeyang / @kirg would be most appreciated, as this is the final stumbling block for us with Cherami

@danudell-trustnetworks, @djoodle I suspect this is caused by a bug. I sent out a PR for this:
https://github.com/uber/cherami-server/pull/285
Could you try it out? You can use branch 'distance'

Hi @datoug , thanks for getting back to me quickly.
So now no data is flowing through Cherami - however I am wondering if this is due to other updates since I last pulled the image.
time="2017-08-31T11:33:29Z" level=error msg="ListConsumerGroupsByDstID failed" ctrlID=48fd47d2 deploymentName= destID=f337b32e-4db1-4a8a-bdc0-2ccc33fbad51 err="InternalServiceError({Message:consumer_group of type frozen<consumer_group> has no field options})" module=extentMon
I get this error appear on the controllers. I cannot list consumergroups via the cli tool - only destinations.

Actually you can ignore that, seems like the upgrade wasn't very clean. Will run it this evening and let you know how it goes.

yea from the error, it seems your cherami-thrift is not update to date.

Ok, so the resource profile now I've redeployed looks much more sensible, each host is using about the same amount of memory. The system is quiet overnight so the real test will be tomorrow morning when it comes under load - Will let you know then you can close this.
Thanks again for all the help @datoug !

Hi @datoug, sorry for being slow to get back to you - the fix seems to have worked well, the load is far more evenly balanced across the cluster.
Would you guys be open to pull requests around things like example k8s configs or documentation additions?

@datoug is this fix being recreated or dropped? I just noticed it's been reverted out of master?

@danudell-trustnetworks Yes, PRs are welcome.
My patch had some test issues. I'm investigating that and will re-merge it after it's solved.

@datoug Cheers, we will stick on the branch for now until you can sort the test failures.

@danudell-trustnetworks on it now, will update you today or tomorrow.

@danudell-trustnetworks I have landed https://github.com/uber/cherami-server/pull/294, could you try latest master branch?

Yeah that all works - cheers! You can close this on


Improved OperationSelector performance

Improved OperationSelector performance by creating an OperationSelectorBuilder which constructs a thread-safe OperationSelector instance with immutable data structures.
To prevent contention on the Random instance a new instance is created for each Worker in all tests which use OperationSelector.
Should fix #485.

Changed probability values for OperationSelector from 0.0 to 1.0 to percentage (0 to 100).
I think this makes it easier for us to calculate probabilities and to refactor old tests, which use percentage more often than a probability of max 1.0.

:+1:

Missed a line in OperationSelector.addOperationRemainingProbability() to change from subtracting to adding the total probability. Added a test case for this method.

All review issues should be fixed.

PR updated with latest review comments.

Rebased with master and adapted all new tests.

ping

Danny commented that using percentages for probabilities is a very uncommon thing. So should I revert that part of the PR? Should we nevertheless keep the internal array and the max. granularity of 100? It's such a nice and fast internal implementation. WDYT?

Changed the probability back to a double between 0.0 and 1.0, but kept the array for the OperationSelector. I think this is the best compromise between an intuitive API and a good underlying structure for performance. The maximum precision is 3 (so 0.001 as minimal value). I also added some test for missing edge cases and parameters.
![My work here is done]
(http://cdn.meme.am/instances/500x/53368971.jpg)


Update the Configuration.md

The appsettings.json file has changed and the instructions for setting up its contents are not accurate anymore.
There are now undocumented TwitchClientSettings and some of their names have changed, as well:
"TwitchClientSettings": {
"TwitchUsername": "secret",
"TwitchChannelID": "secret",
"TwitchBotUserID": "secret",
"TwitchBotOAuth": "secret",
"TwitchChannelOAuth": "secret",
"TwitchChannel": "secret",
"TwitchClientId": "secret"}
The old Settings were:
"TwitchClientSettings": {
"TwitchUsername": "secret",
"TwitchUserID": "secret",
"TwitchOAuth": "secret",
"TwitchChannel": "secret",
"TwitchClientId": "secret"
}

I'm currently working on that, especially now that I've got my secret manager to work (thanks @benrick ) :)

How's this PR coming along, @asyncawake ?


Files replaced with symlinks under WSL are not being detected as changed by VScode run from WSL

VSCode Version: 1.49.1
OS Version: Windows_NT x64 10.0.19041

Steps to Reproduce:
# steps run from WSL terminal  launched in an empty directory
git init
mkdir subdir
ls > basefile.txt
git add --all
git commit -a -m initial-commit
cp basefile.txt subdir
code .
# Should now have both VScode and the terminal screen open tied to the same directory
## Visual code reports one file modified as expected
git status
## shows the untracked file in `subdir`
git add --all
git commit -a -m 2nd-commit
## the previous two commands run successfully
git status
## shows a clean working tree
## but VScode still shows the outstanding original change
## clicking refresh in VScode shows a clean working tree
ls > 2ndfile.txt
git status
##  shows new untracked file
## VScode shows the new file, but does not list it as new
## if you hit refresh in VScode the file shows up as a new file for git
git add --all
git commit -a -m 3rd-commit
git status
## shows a clean working tree
## but VScode still shows the outstanding original change
## clicking refresh in VScode shows a clean working tree
cd subdir/
ln -s ../2ndfile.txt .
ls -la
git status
## shows the untracked file in `subdir`
## But in VScode the file does not show up in the Explorer
ls > 3rdfile.txt
git status
## now shows two untracked file in `subdir`
## if you hit refresh in VScode both files show up in Explorer and as new files for git
git add --all
git commit -a -m 4th-commit
git status
## shows a clean working tree
## but VScode still shows the outstanding original change
## clicking refresh in VScode shows a clean working tree
rm *.txt
git status
## shows 3 files in `subdir` tagged for deletion
## if you hit refresh in VScode the same changes show up there
ln -s ../*.txt .
git status
## Shows as expected:
###        deleted:    3rdfile.txt
###        typechange: basefile.txt
## Hitting refresh in VScode only shows the deletion of '/3rdfile.txt' and is missing the type change on 'basefile.txt'
## if you commit the change from vscode it shows a clean working directory
git status
## But this still shows
###        typechange: basefile.txt

The bug is the fact that a symlink is not detected as a change, while a new symlink does show up as a change.
It would also be nice if the VSCode explorer and git integration detected the changes to the underlying file structure more dynamically.  VSCode appears to do a much better job of this when using the Native Windows version on top of the native Windows file system.  But when doing cross-platform development where the code base uses symlinks and/or illegal Windows characters in file and directory names (ie: ':' character) this forces you to use the WSL filesystem and VSCode in WSL.

Does this issue occur when all extensions are disabled?: Yes/No  - Did not test

I should have also added it would be super nice to be able to create symlinks directly from the VSCode UI instead of having to step out to a WSL terminal.


[NewTemplate] [Python] JDBCToGCS

Template should support read partitioning (refer java)

This issue is duplicate of #107


 So I can't seem to get the last step working :(

Another option is that we move to use GitVersion.Tool, but that would require us to reorder some of the Tasks, to make sure the .NET SDK specified in global.json is installed before attempting to install, or use GitVersion.Tool..

* Don't run GitVersion locally

* Use static version when running locally

* If you want GitVersion locally, install GitVersion (https://www.nuget.org/packages/GitVersion.Tool/) and run
  `dotnet gitversion /output buildserver` before running build.ps1/build.sh

Should fix #1891
For some reason GitVersion is not outputting env vars when I run locally. So I can't seem to get the last step working :(

My system (Ubuntu 20.04) cannot run this code, the log output shows the errors:
Could not execute because the specified command or file was not found.
Possible reasons for this include:

You misspelled a built-in dotnet command.
You intended to execute a .NET Core program, but dotnet-gitversion.tool does not exist.
You intended to run a global tool, but a dotnet-prefixed executable with this name could not be found on the PATH.

By a miracle, i could run the command "dotnet gitversion /output buildserver",
The log looks like this:
"
123@123~/Downloads/omnisharp-roslyn$ dotnet gitversion /output buildserver
INFO [10/07/20 15:56:58:91] Working directory: /home/sonnguyen9800/Downloads/omnisharp-roslyn
INFO [10/07/20 15:56:58:94] Project root is: /home/sonnguyen9800/Downloads/omnisharp-roslyn/
INFO [10/07/20 15:56:58:94] DotGit directory is: /home/sonnguyen9800/Downloads/omnisharp-roslyn/.git
INFO [10/07/20 15:56:58:96] Begin: Loading version variables from disk cache
INFO [10/07/20 15:56:58:97] Begin: Deserializing version variables from cache file /home/sonnguyen9800/Downloads/omnisharp-roslyn/.git/gitversion_cache/5103F2D91E6EEF8E89015D37DF141357A457C87D.yml
INFO [10/07/20 15:56:59:02] End: Deserializing version variables from cache file /home/sonnguyen9800/Downloads/omnisharp-roslyn/.git/gitversion_cache/5103F2D91E6EEF8E89015D37DF141357A457C87D.yml (Took: 52.26ms)
INFO [10/07/20 15:56:59:02] End: Loading version variables from disk cache (Took: 52.97ms)
INFO [10/07/20 15:56:59:03] Done writing
"
However, when build the program, the same error shown:
"Feeds used:
/home/sonnguyen9800/.nuget/packages/
https://api.nuget.org/v3/index.json
https://dotnetfeed.blob.core.windows.net/dotnet-core/index.json
https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-tools/nuget/v3/index.json
https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet5/nuget/v3/index.json
https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet5-transport/nuget/v3/index.json
https://www.myget.org/F/omnisharp/api/v3/index.json
All packages listed in /home/sonnguyen9800/Downloads/omnisharp-roslyn/tools/packages.config are already installed.
/home/sonnguyen9800/Downloads/omnisharp-roslyn/build.cake(48,20): warning CS0618: 'ITFBuildProvider.IsRunningOnTFS' is obsolete: 'Please use ITFBuildProvider.IsRunningOnAzurePipelines instead.'
Error: One or more errors occurred. (GitVersion: Process returned an error (exit code 1).)
GitVersion: Process returned an error (exit code 1).
"
"

Superseeded by #2030


Add-on for LAARES

Adapt pack-it-forms for implementing an Outpost "add-on", specifically https://github.com/jmkristian/OutpostForLAARES

Thanks for all this work John. I'll try to make time to look things over and make more detailed/targeted comments in the next day or two, but from a quick scan my initial comment is that this appears to require your server-side solution for the forms to work at all. I'd like forms to be able to be authored in a manner that is independent of the server/integration solution. Based on the work you've done so far, do you have thoughts on how to achieve that in your server/integration approach?

Although they're called 'server side includes', you don't really need a server to implement them. The software that creates or opens a message can read the source HTML file, expand the include and var directives, store the result into a file and browse the file. There's free software to do the expansion. I actually tried this, and it worked OK.
I think that covers the changes up to and including 'Pass data to forms via an "environment" server side var'. But the next two are specific to Outpost's 'add-on' system. Outpost expects an addon message to be in a slightly different format than a PACF message, and Outpost doesn't provide a server to which an addon message can be posted. (You may recall an email conversation with Jim Oberhofer about this.) I'm not certain how to handle this.
One idea is to move the add-on-specific code into the integration model, e.g. outpost-pacread or OutpostForLAARES. This code could be injected into the web page using server side includes.
Perhaps there's a better way. I haven't thought about it much.

The last two commits 9357488 and f1a18b0 are small examples of shifting control to the integration module.

I abandon this request.


EKF won't initialise if a GPS or optical flow sensor isn't fitted

When powered up without the GPS sensor connected and EKF2_IMU_MASK = 1 or 0, there is no attitude solution at the HUD and the ESC's keep beeping indicating the filter has not started. Seems to have been introduced by 968cfae632d499e2e5afd0e9a14e01b52953be97

Fixed by https://github.com/PX4/Firmware/issues/5791


sort APIs alphabetically

This will help finding APIs more quickly :)

Good idea but it got eaten by all the new additions so im gonna close this

I'll probably do this once the existing PR's are fully merged, then make it a CONTRIBUTING.md rule or somrthing.


Moving contributing docs into rustdoc

Problem
Capturing task from Zulip thread
"This is an exploration in moving more contributing docs into rustdoc so that intra-doc links can help us detect when the documentation goes stale. The focus of this documentation would be for our deployment of master, rather than for docs.rs, so it can be in private items, modules that only exist for master documentation, etc."
Notes
I may need guidance on which are best places to start, but from looking at the current contributing docs, the Tests section feels like a good candidate.
For my understanding, will we be shifting them to being hosted at https://doc.crates.io/contrib/apidoc/cargo/?
cc. @epage

Probably the lowest hanging fruit is some of the content under Architecture (e.g. Package and Resolution).
Codebase Overview and testing would get into content in other crates which we aren't publishing yet and we'd just need to decide whether their URLs should be at apidoc/cargo or apidoc/<crate>.  I presume the former but unsure what others thoughts are on it (@weihanglo? since you created it)

we'd just need to decide whether their URLs should be at apidoc/cargo or apidoc/<crate>

Personally prefer the former as well.

I'd love to see an exploration about pros and cons of the current doc and moving to rustdoc.
IIUC, the main benefit of moving to rustdoc is "helping us detect when the documentation goes stale". This is really a good point as we do link to source code when explaining the architecture. The other one might be "having a single place for Cargo documentations".
Across the Rust project, we have rustc-dev-guide, std-dev-guide, and Cargo Contributor Guide hosting their own guides in a separate mdbook. We have Rust Analyzer putting their dev guide contents right under its main git repository. It might be easier to discover and promote by putting contents in separate markdown files. People can read markdown files through GitHub web UI or their favourite markdown editors/readers.
We may need to reach a consensus in the Cargo team first. Do we need to coordinate with other projects for the format of dev guides?

Besides links, I also feel like being in rustdoc will make it easier to keep up-to-date generally.  The further away discussions of internals are from the source code, the harder to remember it exists and harder to track when a change will be relevant for it.

Note that rustc-dev-guide have got a dedicated team to maintain the contents, whereas for Cargo we don't have such a human power and probably don't need a new subteam. Also, std-dev-guide is in a quite difference situation that it's more a spin-off starting from existing docs.

We may need to reach a consensus in the Cargo team first. Do we need to coordinate with other projects for the format of dev guides?

Have there been any updates around this? As well, if priorities have shifted, I'm happy to tackle another issue that can help the team out more!

Thank you for still being interested in this! Cargo team had a discussion about this today. The goal here is not moving the entire contributing doc into rustdoc. Instead, we want to focus on leveraging rustdoc to validate links to items and types, keeping docs specific to code up-to-date. The chapter "Architecture Overview" can be the first good candidate for this migration.
We also plan to move the chapter "Tests" into rustdoc of crate cargo-test-support, though we might hold off a while and see what's going on in https://github.com/rust-lang/rust/pull/105036.

Hey @weihanglo , thank you for the response. I took a crack at this, but realize I don't fully understand the nuances of using rustdoc for keeping the docs up to date, and what that looks like
I'll keep my eyes out for other open issues that I can help on that are more within my understanding


Original Lampton code

I saw your comment on Archive.org for the Gardens of Imagination book, and I did manage to get hold of the original code. A guy called Mike actually bought the book and sent me the code, after seeing my repository.
It can be found here, in the Original folder -> https://github.com/jacmoe/retrogardens.

Thanks Jacob!
I guess you mean the comment here:
https://archive.org/details/gardensofimagina00lamp
I think that the same Mike added a link to my github where I also do some
retro programming.
https://github.com/johangardhage/retro-gardens
--gajo
On Wed, Dec 23, 2020 at 2:02 PM Jacob Moen<EMAIL_ADDRESS>wrote:

I saw your comment on Archive.org for the Gardens of Imagination book, and
I did manage to get hold of the original code. A guy called Mike actually
bought the book and sent me the code, after seeing my repository.
It can be found here, in the Original folder ->
https://github.com/jacmoe/retrogardens.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/johangardhage/retro-gardens/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABXRLFMCXGIESOUX5YYXQQ3SWHS5VANCNFSM4VG64CSQ
.

Ah, it was Mike who linked your repository, my bad!
Not the same Mike who managed to get me the code, though. Michael Coburn is his name. And much praise to him, because I've been looking for the code for years and years.
I bought the book in 1997 or 1998, and then (like a Normal Person) got rid of  the floppy disk during the dark ages of the naughties (2000s).
Anyway, the original - and extremely messy 1997-style C++ - code is available, finally :)


ValueError: Invalid endpoint: https://appconfigdata..amazonaws.com

Traceback (most recent call last):

File "/usr/local/lib/python3.7/site-packages/appconfig_helper/appconfig_helper.py", line 64, in init
self._client = session.client("appconfigdata")
File "/usr/local/lib/python3.7/site-packages/boto3/session.py", line 309, in client
config=config,
File "/usr/local/lib/python3.7/site-packages/botocore/session.py", line 959, in create_client
api_version=api_version,
File "/usr/local/lib/python3.7/site-packages/botocore/client.py", line 132, in create_client
endpoint_bridge,
File "/usr/local/lib/python3.7/site-packages/botocore/client.py", line 475, in _get_client_args
endpoint_bridge,
File "/usr/local/lib/python3.7/site-packages/botocore/args.py", line 134, in get_client_args
proxies_config=new_config.proxies_config,
File "/usr/local/lib/python3.7/site-packages/botocore/endpoint.py", line 402, in create_endpoint
raise ValueError("Invalid endpoint: %s" % endpoint_url)
ValueError: Invalid endpoint: https://appconfigdata..amazonaws.com
Python Package Version Details
boto3                       1.24.46
botocore                    1.27.46
sample-helper-aws-appconfig 2.0.3
I have tried with multiple downgraded versions of boto3 and botocore but, this error is constant.

Looks like you're missing the region name in your AWS config somewhere, or it's set to the empty string.
Do you have AWS_DEFAULT_REGION exported in your environment, or region= in your AWS config file (e.g. ~/.aws/config)? If not, you'll need something like that. If you do, check the value is something like eu-west-1 (of course substitute in the region you're actually using :)
That value is used (by boto3, not this library specifically) to build the API endpoint URL, which is supposed to have the region name between "appconfigdata" and "amazonaws.com", for example appconfigdata.eu-west-1.amazonaws.com.
If you want, you can also pass a configured boto3 Session instance to AppConfigHelper() using session= keyword argument. This will override the credentials/region otherwise being used.

Thanks @jamesoff. It working as expected.


exit on escape

Hello,
first of all, thank you for this wonderful software!
It's a simple pull request to add ESC as exit keybinding in addition to CTLR+D.
I'm not sure if you like that idea or there any conflicts with other tools used in conjunction with fzy, it just fits nicely into my workflow and of course I'm just lazy to patch it every time.
I moved to fzy from peco, it exits on escape.
Thanks and good luck!

@gyakovlev thanks for the PR.
Unfortunately, escape is the start of many other sequences, like UP/DOWN/PAGEUP/PAGEDOWN. This would make fzy quit if you pressed any of those keys.
This can be worked around. Things like vim have a timeout and if no escape sequence comes in within a few hundred milliseconds after an escape, it can be assumed to be an escape by itself. I'd guess peco (or termbox, the library it's using) does something similar.
I worry that this workaround can result in somewhat flaky behaviour. At best, it has a noticeable delay before actually quitting after pressing escape. So I'd like to avoid doing it. Though I agree it would be nice if it could be done well, terminals' quirks don't allow it.


any examples using minimal api ?

Could you please provide any examples using minimal api for file upload.

All the provided samples (code pieces, example implementation) should map 1:1 to minimal APIs. Just replace services with builder.Services and endpoints with app.Map....

I tried your examples, to upload file to s3, its throwing service not registered error in .net core 8.0. If you could add some sample code to consume, would be more helpful
added
builder.Services.AddAmazonS3(builder.Configuration.GetSection("AmazonS3"));
services.AddUploadNet()
        .AddAwsS3Provider("public", options =>
        {
            options.BucketName = "public-images";
        });
like

endpoint.MapPost( IFormFile file , StorageManager _storageManager)
await _storageManager.CreateFile("public", "users/1553/profile.png", inputStream);


conmon: 2.0.26 -> 2.0.27

https://github.com/containers/conmon/releases/tag/v2.0.27

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 115447 at 84d931e7 run on aarch64-linux 1

4 packages built:

buildah
conmon
cri-o
podman

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/virtualization/conmon/default.nix:29:3:
   |
29 |   installPhase = ''
   |   ^


nixos/spacenavd: run as user service

Motivation for this change
This fixes the patches to run spacenavd as a user service.
However, it does not adjust the location of the default config (/etc/spnavrc) and the pid file (/var/run/spnavd.pid).
The user service obviously cannot write to these locations, so it is a bit of a problem. For example spnavcfg (as introduced in https://github.com/NixOS/nixpkgs/pull/118327) requires these files in their default location. I could of course also patch spnavcfg accordingly. What do you think? @gebner @flokli
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I've just tried running freecad and it works fine with the system-wide daemon.

Huuh, yeah, this won't work editing files in the nix store, and running as setuid root isn't even required on user services.

@flokli The /etc/spnavrc file is not in the nix store, it's a regular file in /etc and can be modified by root.  Configuring the daemon "works" at the moment, but you need to run sudo spnavcfg.

I could of course also patch spnavcfg accordingly. What do you think?

If we're already patching spacenavd and co., then I think we should really do this.  I don't have a strong opinion about whether the daemon should be a user service or not, but I'd really like to run spnavcfg without sudo.

I could of course also patch spnavcfg accordingly. What do you think?

If we're already patching spacenavd and co., then I think we should really do this. I don't have a strong opinion about whether the daemon should be a user service or not, but I'd really like to run spnavcfg without sudo.

Yeah, then let's also add the same config file location logic to spnavcfg, so it can read and edit user config. We certainly don't want to setuid spnavcfg and have it poke around in /etc.

I'll try to get around patching everything by the end of the week.

It took me a little longer, but I've patched everything. I haven't tested everything in combination yet however. If someone finds the time, feel free to give it a go.

Built and tested.  FreeCAD with patched libspnav works with the upstream daemon.  After switching to the patched spacenavd, FreeCAD works and spnavcfg no longer requires sudo.


Fix differenet memleaks

Fixes for different memory leaks, usually in error paths, but some occur at the main path.

Please move the invalid parameter checks to a different pull request.

I've updated the PR, please re-review.


[Add] Scroll to top button

Details
Scroll to top button is a must for good user experience. I would to add it, kindly assign.
Screenshots
No response
Type of Contribution

[ ] Update to an existing Animated Button
[X] Adding a new Button
[ ] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[X] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

Not accepted.
Check https://github.com/Spyware007/Animating-Buttons/issues/1347


Feedback

I really want your feedback. Do you see the same problem? Does this mitigate it? Thanks!

Moot at this point.


Fix tests after upstream merge

When running the tests with an absence of a ~/.aws/credentials the mock Authenticator would fail because a default region was not set. The tests are written largely around an assumption of a "region" of us-east-1 and a "backend_region" of us-west-2; so this just adds those parameters to the mocked boto sessions.

This PR is against the old code base and is no longer relevant.


about add learner

I read the document that the add_leaner function will block until the node completes synchronization, but the actual time is 500ms. How can I implement the add_leaner function to block for an infinite time until the synchronization is completed?
After I execute add learner (because it blocks up to 500ms), and then immediately change the status to voter, will this affect the data synchronization of the cluster? If I have a lot of nodes, because in fact these nodes do not satisfy the voter, at this time Their data has not been synchronized from the leader

Hi,I try to express my option.

I think the timeout setting of 500ms is to prevent add caused by network that prevent synchronization from completing add_learner function has been blocking. However, if you are confident that synchronization will always complete successfully and want to ensure add_learner function blocks indefinitely, and you can remove the timeout setting or set it to a very large value.

Regarding your second question, changing the state of a node from learner to voter while synchronization is still in progress may result in data loss or inconsistency in the cluster. When a node is added as a learner, it receives updates from the leader but does not participate in the quorum of write operations. If you immediately change the status of a node from learner to voter after adding it, there is a risk that it may not have received updates from all leaders, resulting in data loss or inconsistency.

I read the document that the add_leaner function will block until the node completes synchronization, but the actual time is 500ms. How can I implement the add_leaner function to block for an infinite time until the synchronization is completed?

The function Raft::add_learner() is designed to wait until the newly added learner catches up with the leader, ensuring that the difference in log entries is within the specified threshold of replication lag. The synchronization check required for adding a learner is performed within this function:
https://github.com/datafuselabs/openraft/blob/b0c30acf92e0aab44a7fcb9cd63294febda53fad/openraft/src/raft.rs#L571-L615
If the Raft::add_learner() returns earlier than expected, attach the debug level log(run your application with RUST_LOG=debug) and let me examine what's going on.

After I execute add learner (because it blocks up to 500ms), and then immediately change the status to voter, will this affect the data synchronization of the cluster? If I have a lot of nodes, because in fact these nodes do not satisfy the voter, at this time Their data has not been synchronized from the leader

At any time, change-membership operations(such as upgrade a learner to voter) are safe. The only issue is that if the learner has not yet caught up with the leader, change-membership may block for a while before committing the change.

check_replication_upto_date
https://github.com/datafuselabs/openraft/blob/b0c30acf92e0aab44a7fcb9cd63294febda53fad/openraft/src/raft.rs#L551
Here the argument to the wait function is None, which means that the maximum duration is 500ms，My actual tests are also like this, no matter how much data the leader has, add_leaner will return quickly

check_replication_upto_date
https://github.com/datafuselabs/openraft/blob/b0c30acf92e0aab44a7fcb9cd63294febda53fad/openraft/src/raft.rs#L551
Here the argument to the wait function is None, which means that the maximum duration is 500ms，My actual tests are also like this, no matter how much data the leader has, add_leaner will return quickly

Oh... my mistake... Let me fix it!


Fix the parsing of the user name from https remote uri

Should fix #173

Wrong fix, everything is already in the yaml configuration file.


Feature/getOrCreateStore

Gets or creates a key-value store with the passed name or ID. The key-value store is retrieved or created automatically for each act run. The ID is passed by the user calling the function instead of the APIFY_DEFAULT_KEY_VALUE_STORE_ID environment variable passed by the Actor platform.
It exposes a class with two methods: getValue and setValue.
Example:
const Store = Apify.getOrCreateStore('My-custom-store');
const previousState = await Store.getValue('STATE');
...
const nextState = {...};
await Store.setValue('STATE');

My apologise, I try to merged the unfinished branch


Is default way to select format changed?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.08.19 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
link: https://www.youtube.com/watch?v=SX2nQG5SKVE
So at may I downloaded that video and got ~20 mb file with AVC1 video codec. Sometime later at july I redownloaded that video and got ~14 mb video file with VP9 videocodec. Is default selected quality/format changed or VP9 is better that AVC1?
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vUF', 'https://www.youtube.com/watch?v=SX2nQG5SKVE']
Active code page: 65001
[debug] Encodings: locale cp1251, fs utf-8, pref cp1251, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.08.19 [48c88e0] (win32_exe)
[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.19041-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 5.0-full_build-www.gyan.dev (setts), ffprobe 5.0-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.08.19, Current version: 2022.08.19
yt-dlp is up to date (2022.08.19)
[debug] [youtube] Extracting URL: https://www.youtube.com/watch?v=SX2nQG5SKVE
[youtube] SX2nQG5SKVE: Downloading webpage
[youtube] SX2nQG5SKVE: Downloading android player API JSON
[youtube] SX2nQG5SKVE: Downloading tv embedded player API JSON
[youtube] SX2nQG5SKVE: Downloading web embedded client config
[youtube] SX2nQG5SKVE: Downloading player 009f1d77
[youtube] SX2nQG5SKVE: Downloading web embedded player API JSON
[debug] Loading youtube-nsig.009f1d77 from cache
[debug] [youtube] Decrypted nsig euQigwxY2jvd6pXzyz => PbCe1R1jEj56Cg
[debug] Loading youtube-nsig.009f1d77 from cache
[debug] [youtube] Decrypted nsig YsLG73DubvmFerV5e_ => TTLNUdUBdFKIAw
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
[info] Available formats for SX2nQG5SKVE:
ID  EXT   RESOLUTION FPS CH │   FILESIZE   TBR PROTO │ VCODEC          VBR ACODEC      ABR ASR MORE INFO
────────────────────────────────────────────────────────────────────────────────────────────────────────────────
sb2 mhtml 48x27        1    │                  mhtml │ images                                  storyboard
sb1 mhtml 80x45        1    │                  mhtml │ images                                  storyboard
sb0 mhtml 160x90       1    │                  mhtml │ images                                  storyboard
249 webm  audio only      2 │  582.51KiB   47k https │ audio only          opus        47k 48k low, webm_dash
250 webm  audio only      2 │  769.40KiB   62k https │ audio only          opus        62k 48k low, webm_dash
140 m4a   audio only      2 │    1.58MiB  130k https │ audio only          mp4a.40.2  130k 44k medium, m4a_dash
251 webm  audio only      2 │    1.50MiB  123k https │ audio only          opus       123k 48k medium, webm_dash
394 mp4   256x144     30    │    1.23MiB  101k https │ av01.0.00M.08  101k video only          144p, mp4_dash
160 mp4   256x144     30    │  616.07KiB   49k https │ avc1.4d400c     49k video only          144p, mp4_dash
278 webm  256x144     30    │    1.15MiB   95k https │ vp9             95k video only          144p, webm_dash
395 mp4   426x240     30    │    2.52MiB  207k https │ av01.0.00M.08  207k video only          240p, mp4_dash
133 mp4   426x240     30    │    1.12MiB   92k https │ avc1.4d4015     92k video only          240p, mp4_dash
242 webm  426x240     30    │    2.11MiB  174k https │ vp9            174k video only          240p, webm_dash
396 mp4   640x360     30    │    4.08MiB  336k https │ av01.0.01M.08  336k video only          360p, mp4_dash
134 mp4   640x360     30    │    2.94MiB  242k https │ avc1.4d401e    242k video only          360p, mp4_dash
18  mp4   640x360     30  2 │    6.20MiB  509k https │ avc1.42001E    509k mp4a.40.2    0k 44k 360p
243 webm  640x360     30    │    4.06MiB  334k https │ vp9            334k video only          360p, webm_dash
397 mp4   854x480     30    │    6.95MiB  572k https │ av01.0.04M.08  572k video only          480p, mp4_dash
135 mp4   854x480     30    │    5.86MiB  482k https │ avc1.4d401f    482k video only          480p, mp4_dash
244 webm  854x480     30    │    6.90MiB  568k https │ vp9            568k video only          480p, webm_dash
22  mp4   1280x720    30  2 │ ~ 19.61MiB 1575k https │ avc1.64001F   1575k mp4a.40.2    0k 44k 720p
398 mp4   1280x720    30    │   13.50MiB 1110k https │ av01.0.05M.08 1110k video only          720p, mp4_dash
136 mp4   1280x720    30    │   10.99MiB  904k https │ avc1.4d401f    904k video only          720p, mp4_dash
247 webm  1280x720    30    │   14.13MiB 1162k https │ vp9           1162k video only          720p, webm_dash

Fixed by https://github.com/yt-dlp/yt-dlp/commit/b25cac650f3cbba16f46c64b0f9b0a96a9171fbc

yt-dlp has considered vp9 > avc since the start

It's probably due to https://github.com/yt-dlp/yt-dlp/commit/91e5e839d3017577dabba7e9b142910ec32a495a.