Filter out empty actions

Actions may be empty when a job is in progress. Make sure to filter those out as the label would still be there.
Signed-off-by: Joe Grund<EMAIL_ADDRESS>

Codecov Report

Merging #1042 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1042   +/-   ##
=======================================
  Coverage   95.42%   95.42%
=======================================
  Files           2        2
  Lines         153      153
=======================================
  Hits          146      146
  Misses          7        7

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 546ec02...0a74fd7. Read the comment docs.


Support combining batched prediction results on DatasetType

Context
A bug was introduced in https://github.com/iterative/mlem/pull/221, where approach to combining prediction results incorrectly assumed type to be numpy array. This PR utilizes DatasetAnalyzer to first determine the type of the results prior to combining them, since the same model can output different data types.
Modifications

mlem/api/commands.py - Interpret data type prior to combining prediction results
mlem/contrib/numpy.py - Add support for combining batch reading prediction results for nd.array

Which issue(s) this PR fixes:
Fixes https://github.com/iterative/mlem/issues/257

I've updated the branch, now auth issues are gone. But there are some others I didn't investigate. @terryyylim do you mind taking a look?


Upgraded pandas_ta to new version, now values changing (without proof).

Upgraded pandas_ta without changing any features, and the backtests now show different results.
I was using version: pandas-ta-0.1.66b0 before, and upgraded to the most recent one. Features of pandas_ta I were using were: hma, log_return, atr, rsi,bbands, pvt, vwma, sma. Was there any change in how these calculations were done from previous version to upgraded version?
Particularly, I think there is a discrepancy in applying SMA to a series of volume, or how the bbands were calculated.
For now, I have rolled back to 0.1.66b0.
I expected the same results after update. Why did that happen? Is this library not accurate?
Thanks

Hello @ryohno,
Thanks for using Pandas TA!
This Issue I assume is in regards to your Neural Net modules? Why did you disregard the Bug Report questions that I am once again having to ask?

Upgraded pandas_ta without changing any features, and the backtests now show different results.
...
Particularly, I think there is a discrepancy in applying SMA to a series of volume, or how the bbands were calculated.

I'm sorry but anecdotal evidence like this and "I think" is insufficient to qualify as a bug. In the future when your projects begin getting Issues, is this level of detail sufficient for you to help resolve their problem?
What backtests? What backtester are you using? Show me outputs/screenshots from Pandas TA v0.1.66b0 and the current version for the same asset and date range. Is it one asset that seems off or all of them? Include a few of those as well so I know it's not an isolated asset. Where is your proof?

I expected the same results after update. Why did that happen?

🤷♂  Do you have any correlation tests pinpointing the inaccuracy of the mentioned indicators? Because none of the indicators are failing their unit tests on this end and are passing their correlation tests with the de facto TA Lib.

Is this library not accurate?

Quite the opposite. It is one of the few libraries that is accurate. Have you seen other Open Source Python libraries that have passing correlation tests with the de facto TA Lib? Because I have not.
Furthermore, I value my limited free time seriously so I can make Pandas TA an Open Source TA Library that is easy to use, accurate and feature rich. Are you implying that I purposefully choose to dedicate my free time to make inaccurate indicators?
Lastly major changes are listed on the README. Here is the relevant section:

Regards,
KJ

Hello @ryohno,
I will be closing out this issue in a few days unless you can convince me with proof that Pandas TA is inaccurate despite unit tests to the contrary.
Regards,
KJ


[stable/redis] Support for istio

Is your feature request related to a problem? Please describe.
Right now the Egress rules if you set networkPolicy = true don't work for Istio.
Describe the solution you'd like
I'm not 100% sure what the egress ports needed for istio are, but once I figure them out I can submit another PR. I'm also not sure if we would also maybe want to use a namespace selector for kube-system here? Or potentially even blanket allow egress to the kube-system egress
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi @glentakahashi
I guess it could be related with Redis chart services not following the Istio specifications for port names on services.
Check for instance the changes we did on PostgreSQL to comply with Istio requirements:

https://github.com/helm/charts/pull/19249

I got it working with Istio https://github.com/istio/istio/issues/8374#issuecomment-586269097 , but... I miserably stumbled on the cluster-announce-ip issue:  there is no way to let SENTINEL announce the istio-ingressgateway's IPs https://github.com/antirez/redis/issues/2527
even if you hack the redis configuration in a pod... you won't manage to get all the ingressgateways listed, as it is designed for only one IP at a time... so, the only chance would be to have a LB in front

p.s. might be that giving an IP/fqdn of a loadbalancer in the configuration, will cause sentinel's designed mechanisms of monitoring and fail over to go wild, @antirez?

for everyone else... SENTINEL, if asked about the instances... it won't answer with the IPs the client shall connect to from outside the cluster, but each POD's private IP... which would not be reachable from an application outside the cluster... the logic will fail
dst-knode04:26379> SENTINEL slaves mymaster
1)  1) "name"
    2) "<IP_ADDRESS>:6379"
    3) "ip"
    4) "<IP_ADDRESS>"
    5) "port"
    6) "6379"

thanks @szottE for guiding me through this ;)

@glentakahashi I'm not 100% (re-reading ur ticket) if I tackled your same problem, since you are talking of Egress, instead I have in inbound connectivity/logic issue of the application (redis/sentinel) itself

RTFM https://redis.io/topics/sentinel#sentinel-docker-nat-and-possible-issues

I would like to mention that this chart has been deprecated and moved to https://github.com/bitnami/charts/tree/master/bitnami/redis
Find more information at https://github.com/helm/charts/tree/master/stable/redis#this-helm-chart-is-deprecated


feat: Option to disable schema validation in targets

The transferwise variant of target-snowflake has a validate_records setting that is defaulted to false. In the SDK we automatically do validation and don't provide a way to turn it off. We've seen some people struggle to transition to the new meltanolabs target-snowflake because validation is failing, its likely correctly failing but the solution for users is to fix the tap which becomes too big of a barrier for them when switching.
(Default: False) Validate every single record message to the corresponding JSON schema.
This option is disabled by default and invalid RECORD messages will fail only at load time by Snowflake.
Enabling this option will detect invalid records earlier but could cause performance degradation.

cc @tayloramurphy @edgarrmondragon

This was made a lot easier to implement by https://github.com/meltano/sdk/issues/2136 (currently only available in prerelease v0.35.0a2).
One could override Sink.validate_schema to read from config:
class MySink(Sink):
  @property
  def validate_schema(self) -> bool:
    return config.get("validate_records", False)


DON'T MERGE: Feature/telemetry final tweaks

Edited the config files environments to use a 300 second interval and post to the telemetry endpoint, except for the janusgraph_docker environment (it's used for acceptance tests).

@bobvanluijt could you verify whether the telemetry endpoint is up, or if I'm using the correct url? I just tested running a local weaviate, but it was storing its logs in etcd because it couldn't post to http://logs.semi.technology.

Changed name to prevent unintended merging; #829 is still an issue


bump libcurl and openssl version

when building homestore with iomgr 11， the above Incompatible requirements occurs. this PR change the version of openssl and libcurl to align with Iomgr11

Codecov Report

Merging #200 (0f28994) into master (2db98d1) will increase coverage by 0.04%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #200      +/-   ##
==========================================
+ Coverage   63.75%   63.80%   +0.04%
==========================================
  Files          71       71
  Lines        4354     4354
  Branches      550      550
==========================================
+ Hits         2776     2778       +2
+ Misses       1331     1329       -2
  Partials      247      247

Components
Coverage Δ

AuthManager
77.77% <ø> (ø)

Cache
25.04% <ø> (ø)

FDS
70.93% <ø> (+0.16%)
:arrow_up:

FileWatcher
55.90% <ø> (ø)

Flip
65.67% <ø> (ø)

gRPC
77.69% <ø> (ø)

Logging
29.93% <ø> (ø)

Metrics
80.54% <ø> (ø)

Options
100.00% <ø> (ø)

Setting
56.79% <ø> (ø)

StatusObject
73.83% <ø> (ø)

Utility
83.66% <ø> (ø)

Version
95.83% <ø> (ø)


traefik: 3.1.4 -> 3.2.1

https://github.com/traefik/traefik/releases/tag/v3.2.0
https://github.com/traefik/traefik/releases/tag/v3.2.1

@ofborg test traefik


Remove the comparisontool URL includes

In order to let Wagtail take over serving the paying-for-college
landing page, we need to remove the URL entry that sent PFC requests
to comparisontool.
We need to retain three downstream comparisontool pages for now, so
this patch adds explicit URL entries for those pages.
Testing
After checking out the branch, make sure these pages render properly:

http://localhost:8000/paying-for-college/ should show the new wagtail landing page:

These three pages should be unchanged, and show a header with pencils:

http://localhost:8000/paying-for-college/repay-student-debt/
http://localhost:8000/paying-for-college/choose-a-student-loan/
http://localhost:8000/paying-for-college/manage-your-college-money/

changing branch name


error of "Incorrect shape" in example/cpp/image-classification-predict.cc

prediction using image-classification-predict.cc return error below for my input 224x224 color image, which complies with size setting in image-classification-predict.cc. Please kindly advise.
[src/c_api/c_predict_api.cc]MXPredCreatePartialOut(177):CHECK
[00:17:33] /N/software/mxnet/mxnet-master/dmlc-core/include/dmlc/logging.h:235: [00:17:33] src/operator/./concat-inl.h:152: Check failed: (dshape[j]) == (tmp[j]) Incorrect shape[2]: (1,320,15,15). (first input shape: (1,576,14,14)
Thanks!

I met the error too.You can change you data_shape value to solve it.  What is your data_shape's value?

Hi @Stardust, thanks for your support.
data_shape's equivalence, input_shape_data[4] = {1, 3, 224,224} as set below
  //image-classification-predict.cc
   // Image size and channels
   int width  = 224;
   int height = 224;
   int channels = 3;

   const mx_uint input_shape_indptr[2] = { 0, 4 };
   // ( trained_width, trained_height, channel, num)
   const mx_uint input_shape_data[4] = { 1,
                                       static_cast<mx_uint>(channels),
                                       static_cast<mx_uint>(width),
                                       static_cast<mx_uint>(height) };
   PredictorHandle out = 0;  // alias for void *

   //-- Create Predictor
   MXPredCreate((const char*)json_data.GetBuffer(),
                (const char*)param_data.GetBuffer(),
                static_cast<size_t>(param_data.GetLength()),
                dev_type,
                dev_id,
                num_input_nodes,
                input_keys,
                input_shape_indptr,
                input_shape_data,
                &out);

I think this is caused by the pooling behavior with 3x3/2 structures. This affects the CaffeNet, GoogleNet, Inception-BN models, but you can try VGG16, or ResNet50. They work good.
I did not notice this issue, and opened another similar issue #2718 , I think the reasons are the same. Hope that people will quickly fix this soon.
Or you can revert to the version tagged 20160531, it works good.

@taoari, thanks. Will try and report soon.
On Jul 16, 2016 1:02 PM, "taoari"<EMAIL_ADDRESS>wrote:

I think this is caused by the pooling behavior with 3x3/2 structures. This
affects the CaffeNet, GoogleNet, Inception-BN models, but you can try
VGG16, or ResNet50. They work good.
I did not notice this issue, and opened another similar issue #2718
https://github.com/dmlc/mxnet/issues/2718 , I think the reasons are the
same. Hope that people will quickly fix this soon.
Or you can revert to the version tagged 20160531, it works good.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/2709#issuecomment-233108515, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQyyHLZfvA3rPntuGfTL9n5QnspLvDbfks5qWGXUgaJpZM4JNAcK
.

Hi @taoari, for same error at inception_bn at mxnet-20160531 + 20160713's sub-modules (dmlc-core, ps-lite, ms-shadow). Will check with VGG16, or ResNet50

@weishengchong In my case, it works. See #2718 for my description.
It is this PR #2585 that changes the behavior. When I revert to one commit before that PR, it also works well.
I am not using the C++ file image-classification-predict.cc, you can use the notebook predict-with-pretrained-model.ipynb for a quick check.

@taoari

This affects the CaffeNet, GoogleNet, Inception-BN models, but you can try VGG16, or ResNet50. They work good.

I confirmed image-classification-predict.cc + inception_v3 works good using model below
https://github.com/dmlc/mxnet-model-gallery/blob/master/imagenet-1k-inception-v3.md

@weishengchong Maybe, but Inception_BN does not work. I did not try Inception-V3 yet.
3x3/2 pooling behavior is changed, it does not mean that models using 3x3/2 will not work.
For example, ResNet50 will behave like 112->55->28, while in He's paper it should work like 112->56->28. But ResNet50 still runs, and the final results might slightly different. I am not sure to what degree it will hurts the final accuracy.

What about using the 3x3/2 pooling with pad=1? Pre-trained models may still degrade in accuracy though. There seems an offset of one pixel between this implementation and the previous.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Thread two HTML Boms together

Could I create a bom in which I have two boms next to each other. Side by side in the view?

Not sure what you are asking. Just 2 bom tables from different boards, no renders? Generate 2 separate boms and then copy paste the table into excel or google sheets.
If you are asking for something else, add more details.

I mean can I merge two html files together so one ibom will have two renders in the screen just next to each other in one html file.

So you want one common bom table for 2 different boards? No, there is no functionality like that but you can merge your boards in your cad and then generate a bom.

okay! Thanks!


createAzureCommunicationCallAdapter not acting async in @azure/communication-react

Describe the bug; what happened?
What are the steps to reproduce the issue?
You can just use the same code in https://azure.github.io/communication-ui-library/?path=/docs/composites-call--join-existing-call. You will see that the page only renders after the createAzureCommunicationCallAdapter call is finished. Therefore I felt like that inside the function there is something that is NOT async.
What behavior did you expect?
Page render first, and the adapter loads on the side.

OS & Device: Windows on  PC
Browser: Google Chrome
Version  92.0.4515.131

Is there any additional information?
Please reach out to me --<EMAIL_ADDRESS>

Hi @harrychy8
Thanks for reaching out to us.
If you look closely at the code, the page is configured to render only after the adapter state variable has been populated.

This adapter variable is being set only after the createAzureCommunicationCallAdapter  is finished.
This is an intended behavior. The features on the page depend upon this adapter and thus we render the page only after we know that the adapter has been initialized.

Thank you @anjulgarg for the quick response. The issue is that return <>Initializing...</>; is not even rendered when createAzureCommunicationCallAdapter is not finished

That's strange. Could you share your code?
Since return <>Initializing...</>; is outside the conditional blocks it should run.

Closing this issue as this is being caused due to a next.js dynamic import of the component which is out of the scope of this library


[Question] Not showing Intellisense for Object function

PHP version:  7.1.5
XDebug version:  2.3.4?
Adapter version:  1.3.0 ?
I've a PHP class name Db and it contains many function and init on the first include, I called the include file on other file, so it works well, but I wanted the php intellisense to show or peek the function that Db class have, and says no definition found. Is that a bug or not?

Because the other function that normal not object oriented are working well without problem even on different files.

Thanks before for look into this question.

This issue was moved to felixfbecker/php-language-server#411


Update kubernetai plugin to support transfers via the transfers plugin.

The kubernetes plugin was updated to support a new transfer plugin for managing zone transfer requests:
see:-
https://github.com/coredns/coredns/issues/2585 - plugin/etcd: support zone transfers #2585
https://github.com/coredns/coredns/issues/3121 - plugin/transfer: New plugin for handling zone transfers #3121
https://github.com/coredns/coredns/pull/3223 - plugin/transfer: Zone transfer plugin #3223
When this work was performed, the internal implementation of AXFR requests was removed from the kubernetes plugin and instead it supports the transfer.Transferer interface.  kubernetai doesn't support the transfer.Transferer interface and so building CoreDNS v1.11.3 with this plugin results in AXFR requests failing to respond with the zones.
This PR adds a Transfer method to the Kubernetesai object so that it can broker AXFR requests to its embedded Kubernetes plugins.
In order to write tests, I have abstracted the Kubernetes plugin to a embeddedKubernetesPluginInterface.  Which allows me to swap out the underlying Kubernetes plugin.  This is neater than the podHandlerItf approach and allows for us to generate different responses from the embedded plugins needed to check the integration through kubernetes plugins.
This PR is also supported by integration tests in coredns/ci - https://github.com/coredns/ci/pull/158
The kubernetai tests fail against master:
=== RUN   TestAXFR
=== RUN   TestAXFR/matches_stanza_1/test-4.svc.cluster.local._AXFR
    axfr_test.go:187: expected 1 query attempt, observed 0
    axfr_test.go:196: coredns log: .:53
        CoreDNS-1.11.3
        linux/arm64, go1.23.1, ce45312
        [INFO] <IP_ADDRESS>:37727 - 13103 "AXFR IN test-4.svc.cluster.local. tcp 65 false 65535" - - 0 0.000046625s
=== RUN   TestAXFR/matches_stanza_2/test-5.svc.cluster.local._AXFR
    axfr_test.go:187: expected 1 query attempt, observed 0
    axfr_test.go:196: coredns log: .:53
        CoreDNS-1.11.3
        linux/arm64, go1.23.1, ce45312
        [INFO] <IP_ADDRESS>:36319 - 63827 "AXFR IN test-5.svc.cluster.local. tcp 65 false 65535" - - 0 0.0000605s
=== RUN   TestAXFR/matches_first_stanza/cluster.local._AXFR
    axfr_test.go:187: expected 1 query attempt, observed 0
    axfr_test.go:196: coredns log: .:53
        CoreDNS-1.11.3
        linux/arm64, go1.23.1, ce45312
        [INFO] <IP_ADDRESS>:57837 - 14074 "AXFR IN cluster.local. tcp 54 false 65535" - - 0 0.00003625s
--- FAIL: TestAXFR (9.17s)
    --- FAIL: TestAXFR/matches_stanza_1/test-4.svc.cluster.local._AXFR (3.09s)
    --- FAIL: TestAXFR/matches_stanza_2/test-5.svc.cluster.local._AXFR (3.11s)
    --- FAIL: TestAXFR/matches_first_stanza/cluster.local._AXFR (2.97s)
FAIL
FAIL	github.com/coredns/ci/test/kubernetai	24.938s
FAIL

Manual testing against this PR produces:
cd $GOPATH/src/github.com/coredns/ci && go mod tidy && go test -v ./test/kubernetai/... --run=TestAXFR
=== RUN   TestAXFR
=== RUN   TestAXFR/matches_stanza_1/test-4.svc.cluster.local._AXFR
=== RUN   TestAXFR/matches_stanza_2/test-5.svc.cluster.local._AXFR
=== RUN   TestAXFR/matches_first_stanza/cluster.local._AXFR
--- PASS: TestAXFR (8.59s)
    --- PASS: TestAXFR/matches_stanza_1/test-4.svc.cluster.local._AXFR (2.50s)
    --- PASS: TestAXFR/matches_stanza_2/test-5.svc.cluster.local._AXFR (3.01s)
    --- PASS: TestAXFR/matches_first_stanza/cluster.local._AXFR (3.08s)
PASS
ok      github.com/coredns/ci/test/kubernetai    9.297s

The test has passed because the AXFR test has yet to be merged - https://github.com/coredns/ci/pull/158

If this PR is merged before https://github.com/coredns/kubernetai/pull/66 we should see the existing kubernetai plugin fail these tests because it doesn't support the transfer plugin.


SQLite: Index handling is completely broken

Phinx is currently absolutely hopeless when finding, listing, re-creating, or dropping indices for an SQLite database. The root cause is a particular line which overwrites an array key in a loop instead of pushing to the end of the array. Thus, whatever last index SQLite reports internally is all Phinx ends up knowing about. It also doesn't distinguish between incidental indices created by constraints (which are an implementation detail) and actual indices which can be dropped  and need to be re-created when altering a table.

Can you send a pull request to fix this please?

Working on one presently. Code is basically done, but tests will take a little longer.

Question: What is the fundamental purpose of hasIndex()? Is it supposed to answer the question "Does an index with these and exactly these columns exist?", or is supposed to instead answer the question "Is there a usable index covering these columns?"

It is supposed to answer Does an index with these and exactly these columns exist?


Merge rest of branch release-2.6 into main

Signed-off-by: Jonathan Marcantonio<EMAIL_ADDRESS>Merging the remaining changes from the go 1.17 upgrade from the release-2.6 branch

@mikeshng   Could you double check if this PR is still necessary. As we discussed before, the applifecycle-e2e-backend repo will be mostly used in canary test. Since all of the e2e tests are drafted as bash script, I am wondering if we don't have to import these api structures locally. In case we need these api structs, we can import it from go.mod.

The problem I ran into is the sub and channel has this outdated
k8s.io/client-go v12.0.0+incompatible  dependency.
So I think to properly fix this. We should update the channel and appsub repos to use the proper client-go library instead of having to replace like this:
k8s.io/client-go => k8s.io/client-go v0.23.3
That way this e2e repo can be updated to using go 1.17 with appsub and channel as one of the dependency.


Button: v.2. Added support for href and segmented Flow type

Changes documented here: TDD

Added support to href to Button.
Segmented Flow for each Button type to make Flow Type stricter in Button.
Upgraded Buttons Docs.
Added test.

Test Plan

@AlbertCarreras seeing a double scrollbar on the proptable now. Could you investigate?
FYI we do need the overflow auto so the table becomes scrollable with a horizontal scrollbar.

@AlbertCarreras seeing a double scrollbar on the proptable now. Could you investigate?
FYI we do need the overflow auto so the table becomes scrollable with a horizontal scrollbar.

Looks like it's caused by buildDescription

DONE

@AlbertCarreras feel free to self merge after:

Renaming _disabled on Link to disabled
Adding a codemod to remove name from all <Button />s


README.md misprint

Using on s/Dcoker/Docker/
https://github.com/yujiod/wiki#using-on-dcoker

Thank you for your report, fixed. (I forget issue number in commit log...)


Connect everyone's email to GitHub Projects

Update: Everyone except for about 3 members are connected...

Invites have been sent out to everyone

Everyone is now connected!


Run Docker Selfie github action modified so it runs only on original …

…repository.

@nfejzic Thanks!

@ckirsch You're welcome!


Auto-adding for function fails when anonymous function is regarded as a parameter

When anonymous function is passed as a parameter, the auto-added documentation is messed up. I'm not sure is this an issue, or shouldn't I use auto-add documentation for function call like this:
/**
 * [request description]
 * @method request
 * @param  {[type]} url             [description]
 * @param  {[type]} function(error, response,     content [description]
 * @return {[type]}                 [description]
 */
request(url, function(error, response, content) {
  if (error || response.statusCode !== 200) {
    callback(error || new Error(response.statusMessage));
  } else {
    callback(null, content);
  }
});

Is this still a thing? I can't reproduce this.

I'm pretty sure this is not reproducible anymore, otherwise please reopen.


Project created using dotnet new trydotnet-tutorial has compiler warnings

Describe the bug
A new tutorial project has warnings when run.
Repro steps:

Create a project using  dotnet new trydotnet-tutorial.
Run it using dotnet run.

> dotnet run
Program.cs(6,21): warning CS1572: XML comment has a param tag for 'region', but there is no parameter by that name [C:\temp\trydotnet-template-test\trydotnet-template-test.csproj]
Program.cs(7,21): warning CS1572: XML comment has a param tag for 'session', but there is no parameter by that name [C:\temp\trydotnet-template-test\trydotnet-template-test.csproj]
Program.cs(8,21): warning CS1572: XML comment has a param tag for 'package', but there is no parameter by that name [C:\temp\trydotnet-template-test\trydotnet-template-test.csproj]
Program.cs(9,21): warning CS1572: XML comment has a param tag for 'project', but there is no parameter by that name [C:\temp\trydotnet-template-test\trydotnet-template-test.csproj]
Program.cs(10,21): warning CS1572: XML comment has a param tag for 'args', but there is no parameter by that name [C:\temp\trydotnet-template-test\trydotnet-template-test.csproj]
Program.cs(28,27): warning CS1591: Missing XML comment for publicly visible type or member 'Program.HelloWorld()' [C:\temp\trydotnet-template-test\trydotnet-template-test.csproj]
Program.cs(36,27): warning CS1591: Missing XML comment for publicly visible type or member 'Program.DateTime()' [C:\temp\trydotnet-template-test\trydotnet-template-test.csproj]
Program.cs(44,27): warning CS1591: Missing XML comment for publicly visible type or member 'Program.EmptyRegion()' [C:\temp\trydotnet-template-test\trydotnet-template-test.csproj]

Did this error occur while using dotnet try or online?

[x] dotnet-try
[ ] online

What kind of error was it?

[ ] User Interface (UI):  For example the output never displayed
[ ] Service Error: For example "The service is temporarily unavailable. We are working on it"
[x] Other: dotnet new trydotnet-tutorial

Screenshots
If applicable, add screenshots to help explain your problem.
Please complete the following:

OS

[ ] Windows 10
[ ] macOS
[ ] Linux (Please specify distro)
[ ] iOS
[ ] Android

Browser

[ ] Chrome
[ ] Edge
[ ] Safari

Fixed by https://github.com/dotnet/try/pull/251


[BUG][WEBJS] - Still stuck at session initializing

I'm still having issues with session stuck at "STARTING" even after updating the docker image to the new name.

A clear and concise description of what the bug is. Feel free to remove
sections that you don't feel to make the text shorter!

Version
{
  "version": "2024.7.4",
  "engine": "WEBJS",
  "tier": "PLUS",
  "browser": "/usr/bin/chromium"
}

The issue related to being stuck at whatsapp version getting updated got fixed, but the session still doesn't move on because now the QRcode is never generated.

@pablomorato1 hi!
just to check that everything is good, could you try to run a completely new container using different folder? like .sessions2
docker run -v /home/waha/.sessions2:/app/.sessions -p 3000:3000/tcp devlikeapro/waha-plus

if that doesn't help - what country you're in? Where's your VPS is hosted?
another solution would be to try using chrome if you're using default image
docker run -v /home/waha/.sessions2:/app/.sessions -p 3000:3000/tcp devlikeapro/waha-plus:chrome

waiting for your feedback!

any updates?


Add support for Debian Trixie

🗣 Description
Add support for Debian Trixie.  Also switch to using geerlingguy/docker-debian12-ansible now that it exists.
💭 Motivation and context
Debian Bookworm dropped in June, so we need to start testing on Trixie.
🧪 Testing
All automated tests pass.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All new and existing tests pass.

@jsf9k https://github.com/ansible/ansible-lint/pull/3841 has been merged and once a new version is cut (and our pre-commit configuration updated) we should be able to finish this pull request.

@jsf9k ansible/ansible-lint#3841 has been merged and once a new version is cut (and our pre-commit configuration updated) we should be able to finish this pull request.

A new version of ansible-lint has been released.  I have uncommented Trixie in the metadata file and force pushed.

Well it worked on my local machine, but it didn't work here even after I blew away the cache.  I guess I will try again in a day or so. :cry:

Well it worked on my local machine, but it didn't work here even after I blew away the cache. I guess I will try again in a day or so. 😢

@jsf9k The pre-commit hook version has to be bumped and I assume you're not doing that in this PR as it should trickle down from cisagov/skeleton-generic. Please see https://github.com/cisagov/skeleton-ansible-role/actions/runs/6565455910 for an Action run with the new hook version in effect however.


Contribute to special characters in .stmx and .xmile files

I'm trying to ingest the models here, but I've run into parsing issues. After digging in to the problems (with a debugger) I think that the translate_xmile() function is turning stock, flow and aux names into python identifiers, but is not applying the same string operations to variables that appear in equations.
I'd like to contribute and try to get this fixed, but I saw that @enekomartinmartinez is working on an overhaul of parsing. I was planning on restricting my efforts to the translate_xmile() function and getting the (currently skipped) unit test integration_test_xmile_pathway.test_special_characters() to pass, and then doing anything else needed to parse my model.
Let me know if you have any suggestions

After some more digging, I can see that the issue was not what I thought originally. I'm pretty sure that my model can't be parsed because it has special characters in the stock names (newline chars) and it has subscripted variables. It looks like those two pieces have not been implemented for .stmx or .xmile.
I'm interested in contributing working on these two features. I can see that subscripting is a big feature, but I am still willing to put some work in, especially if I have some guidance. I'm interested to know if there is a way to adapt vensim subscript handling pipelines to .xmile files.

HI @djpasseyjr
Thank you for opening this issue. There is currently a more developed version of PySD in the branch split-parsing-building (see #312 ), this will become the PySD 3.0.0 version with several new changes.
In this branch/PR there is already support for Xmile subscripts. I was able to run the model Loans_SFD_v7_Arrays.stmx when replacing the newline chars in the variables names (this is the bug you detected also notified in #203) and when adding 0 to those numbers defined only with decimal values (e.g. .5 -> 0.5). Therefore, two new implementations need to be done:

When reading the variable names with Xmile parser we can replace the new lines chars with whitespaces.
Include the numbers with only decimal values in the parsing grammar.

If you want to contribute solving these two bugs it would be really nice, I would like to ask you to please work with the PySD branch I pointed you so we can implement the new features in the next release without merging conflicts.
We are as well missing a lot of tests for Xmile models, it would be also really helpful if you can help with that.
If you have aby question don't hesitate on asking me.

Thanks for the responses! I actually just discovered BPTK_Py which is built around Stella files. I can't tell if it solves my problem yet, so I'm going to look into their code. If it doesn't line up with what I need I'll be back! Expect to hear more on this soon!

It looks like BPTK_Py does what I need! My problem is solved. Thanks for having this conversation with me!

Glad you found what you needed!


[2201.3.0-stage] Refactor workspace service to gracefully access documents of generated folder with didOpen event

Purpose
$subject
Fixes #38234
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@malinthar we should send this to master as well right?


[Enhancement] Bump harvester dynamiclistener to newer version

In harvester/webhook, bumped rancher/dynamiclistener to v0.3.5 to improve performance.
eg: harvester-network-webhook, before bumping dynamiclistener v0.3.5
➜  ~ k top pod harvester-network-webhook-7bbdc8f4dc-jcg4f
NAME                                         CPU(cores)   MEMORY(bytes)
harvester-network-webhook-7bbdc8f4dc-jcg4f   201m         167Mi
➜  ~ k top pod harvester-network-webhook-7bbdc8f4dc-jcg4f
NAME                                         CPU(cores)   MEMORY(bytes)
harvester-network-webhook-7bbdc8f4dc-jcg4f   201m         167Mi

After bumping dynamiclistener v0.3.5
k top pod harvester-network-webhook-64b48b496c-hgchv
NAME                                         CPU(cores)   MEMORY(bytes)
harvester-network-webhook-64b48b496c-hgchv   74m          170Mi
➜  ~ k top pod harvester-network-webhook-64b48b496c-hgchv
NAME                                         CPU(cores)   MEMORY(bytes)
harvester-network-webhook-64b48b496c-hgchv   74m          170Mi

Close as we can not verify golang's module version, but this commit be included days ago and daily testing still work well.


Release Google.Cloud.Bigtable.Admin.V2 version 3.2.0

Changes in this release:
New features

Publish new fields (commit 8622047)

Created release for Google.Cloud.Bigtable.Admin.V2-3.2.0


Different ruby versions need different time formats

Author Name: Carlos Martín (Carlos Martín)
Original Redmine Issue: 3037, https://dev.opennebula.org/issues/3037
Original Date: 2014-07-11
Original Assignee: Javi Fontan

Reported in the mailing list:
http://lists.opennebula.org/pipermail/users-opennebula.org/2014-July/045409.html
$ ruby -v
ruby 1.8.7 (2013-06-27 patchlevel 374) [x86_64-linux]

$ oneacct -s '06/01/2014 00:00:00' -e '06/30/2014 23:59:59'
Showing active history records from Sun Jun 01 00:00:00 +0200 2014 to Mon Jun 30 23:59:59 +0200 2014

$ ruby -v
ruby 1.9.3p194 (2012-04-20 revision 35410) [x86_64-linux]

$ oneacct -s '06/01/2014 00:00:00' -e '06/30/2014 23:59:59'
invalid argument: -e 06/30/2014 23:59:59

$ oneacct -s '01/06/2014 00:00:00' -e '30/06/2014 23:59:59'
Showing active history records from 2014-06-01 00:00:00 +0200 to 2014-06-30 23:59:59 +0200

$ ruby -v
ruby 2.1.0p0 (2013-12-25 revision 44422) [x86_64-linux]

$ oneacct -s '06/01/2014 00:00:00' -e '06/30/2014 23:59:59'
invalid argument: -e 06/30/2014 23:59:59

$ oneacct -s '01/06/2014 00:00:00' -e '30/06/2014 23:59:59'
Showing active history records from 2014-06-01 00:00:00 +0200 to 2014-06-30 23:59:59 +0200

I have checked, and from ruby 1.9.3 which is the Ruby version we support, the correct way to do this is the same for all version, which is the following:
oneacct -s '01/06/2014 00:00:00' -e '30/06/2014 23:59:59'
Showing active history records from 2014-06-01 00:00:00 +0200 to 2014-06-30 23:59:59 +0200

So the format is DD/MM/YYYY.


Graphql interface to redis to query data

filters added https://github.com/cyberfly-io/cyberfly-node/commit/5ff4b1e011c4997552b49ca07854e7b5fb6a326d


Questionable use of concatMap in unit tests

Some (not all) unit tests declare concatMap as a functional symbol, which is questionable.

On some unit test, makeEvaluateTermsToPredicate returns a different OrCondition if we set concatMapSymbol to be function. The difference is this extra condition:
\and{_PREDICATE{}}(
    /* term: */
    /* D Sfa */ \top{_PREDICATE{}}(),
\and{_PREDICATE{}}(
    /* predicate: */
    \and{_PREDICATE{}}(
        /* Sfa */
        \not{_PREDICATE{}}(
            /* Created: Kore.Internal.Conditional.makeAndPredicate */
            /* Sfa */
            \and{_PREDICATE{}}(
                /* Sfa */
                \ceil{testSort{}, _PREDICATE{}}(
                    /* Created: Test.Kore.Step.Simplification.Equals.f */
                    /* Fn Sfa */
                    f{}(/* Fl Fn D Sfa Cl */ a{}())
                ),
                /* Sfa */
                \ceil{testSort{}, _PREDICATE{}}(
                    /* Created: Test.Kore.Step.Simplification.Equals.f */
                    /* Fn Sfa */
                    f{}(/* Fl Fn D Sfa Cl */ b{}())
                )
            )
        ),
        /* Sfa */
        \not{_PREDICATE{}}(
            /* Sfa */
            \ceil{mapSort{}, _PREDICATE{}}(
                /* Created: Test.Kore.Step.Simplification.Equals.concatMap */
                /* Fn Sfa */
                concatMap{}(
                    /* Fl Fn D Sfa */
                    /* builtin: */
                    /* concrete element: */ elementMap{}(
                        /* Fl Fn D Sfa Cl */ a{}(),
                        /* Fl Fn D Sfa */ x:testSort{}
                    ),
                    /* Fl Fn D Sfa */ m:mapSort{}
                )
            )
        )
    ),
    /* substitution: */
    \top{_PREDICATE{}}()
))

I think its description might be relevant:
It does not attempt to fully simplify the terms (the not-ceil parts used to
catch the bottom=bottom case and everything above it), but, if the patterns are
total, this should not be needed anyway.


Is there a way to add onto the current database using 'dexie-export-import' module?

I read the docs of dexie-export-import thoroughly, however I didn't find a way how to add onto the current database when importing the data. It either completely replaces the whole table or merges and overwrites any row with the same id.
What I'd like to do is to import the data with new id's, while keeping the old data in the table intact.

The option overwriteValues implies that the import uses bulkPut() instead of bulkAdd() - which means that new ids will be added and existing ones will be replaced.

The docs of the addon could really be improved. Any help with that is of course always appreciated. Just use the edit button at https://dexie.org/docs/ExportImport/dexie-export-import and it will generate a github PR for the docs repo.

Yeah, but neither of these do what I'm looking for.
Let's say that I have a table that has fields with ids 1 - 25 and I want to import a json file, that has fields with ids 1 - 15. After importing, I want to have a table with ids 1 - 40 (old and imported fields combined). How can I do that?

If there was an option for dexie-export-import module to ignore imported ids when using bulkAdd or bulkPut(), that would achieve what I'm looking for...

Not sure how id 1-15 would be come 26-40? Are you looking for a feature that would generate new ids for the imported rows?

Yes. Exactly that. :)
I'm working on a site using your awesome Dexie.js module, and I will need a feature that could do that.
I've noticed, that if I remove all the ids from the JSON file, it will generate new ids on its own.

Aha! That's a thing I havent't thought of, but natural according to how IDB put and add works. I still don't see what you are after then? Do you want to merge the fields of the objects in case the import file has id that corresponds to existing id, but generate new ids? Or do you want dexie-export-import to go through your JSON file and remove each id property?

Since I'm not a native English speaker and it's rather difficult for me to describe what I mean, so I will explain it visually:
https://i.imgur.com/atPDlW5.png
What I'm looking for is only the "requested feature" - to be able to combine to add to the current database when importing.
Optional features is something, that could possibly be implemented at the same time, with the main feature - to be able to select which fields to import, and which to ignore..

I see what you're after. However, my direct response would be that this feature is a bit too niche for this general addon, I think. It seems you could equally well achieve all requested features by just altering the json before calling db.import() / importInto().

Hmm, I guess that's possible.. However wouldn't the JSON alteration have to happen entirely in the memory? What if the JSON file is too big?

If that's a real scenario in your use case, that could be an issue, yes. What I could accept would be a generic ImportOption to pass a function that could alter a row on inport - similar to the filter option but return an altered row instead of a boolean.
Pull request would be welcome!

Yeah, that would be a perfect solution!


[Frontend]: React + Effector + Typescript

Good time, at the moment I'm finishing an example on React + Effector.
repo
Todo:

[x] example app

[x] styles

[x] unit tests

[x] readme

@StephenFluin @toddwest @icebob @anishkny, Greetings to you guys. I made an app with Effector + React. Please tell me what else is needed in order for you to publish it at home?

@Alonski, @sandeesh, @Cameron-C-Chapman, @apai4 Hello guys. If any of you have enough time, please add my application. For me and the entire Effector community, this really matters.

Closing the issue as the implementation has been added on the website.
Thanks for your contribution !


[Lua] Files with local and global scoped functions not displaying as expected

Describe the bug
I've noticed that when I have a lua file which has a mixture of local and global functions, Aerial only displays the local ones. Yet when I have a file of just global functions, it displays them as expected.
System information

OS: mac
Neovim version: 0.6.1
AerialInfo:

Filetype: lua
Configured backends:
  lsp (not supported) [LSP client not attached (did you call aerial.on_attach?)]
  treesitter (supported) (attached)
  markdown (not supported) [Filetype is not markdown]
Show symbols: Class, Constructor, Enum, Function, Interface, Module, Method, Struct

Aerial config:

aerial.setup({
  close_on_select = true,
  default_direction = "prefer_right",
  min_width = 40
})

To Reproduce
Steps to reproduce the behavior:

Load functions.lua in Neovim, open Aerial and observe all functions working as expected
Load utils.lua in Neovim, open Aerial and observe that only local functions are displayed

Screenshots
functions.lua

utils.lua

Should be fixed now. Thanks for the report!

Should be fixed now. Thanks for the report!

I'm using the same configuration and latest version, it seems this problem still appears...

@tristone13th could you provide a code snippet of a function that does not appear?

For example ,the "symbols" and "lastplace" function didn't show up in the list.

Should be fixed

Should be fixed

Thanks for your instant fix, keep on your great work!


Blocks can't be mined.

Blocks can't be mined with pickaxe (missing minecraft:mineable/pickaxe tag). Even if i add manually they have no loot tables.

what minecraft and thermal extra version?

Minecraft version: 1.18.2
Mod version: ThermalExtra 2.0.2


Characters not being properly escaped in Python plugins

I have a Wox plugin in development. I'm running into a critical issue - if a "", set of quotation marks, or theoretically any symbol with significant meaning is typed, an exception is raised.
I've attempted to fix this myself, without success, by doing the following:
def query(self, key):
    key.replace("\\", "\\\\")
    return self.argparser(key)

Is there another way I can correct this as a plugin developer? At this point, it seems like an issue with how Wox parses input.

Thx for reporting this bug, I will look into this after Chinese Spring Festival


Support language-specific API generation tools

We should have the option to specify the language your project is written in, and then we will build an API reference for your language appropriately.
Implementation
I imagine having a Programming Language option on project import, similar to the spoken language option already there. Then we would run the specific API tooling for that language, like Javadoc, JSDoc, GoDoc, PHPDoc, etc.
Hopefully we would add some RTD-skinning, and integration between the generated API doc, and the hand-written prose documentation.

I would like to see something like EnvironmentBase extended to support different build environments similar to how Base supports different build tools.
In my particular case I'm extending ReadTheDocs to support building documentation for Node-based projects, and I need to install Node and npm within the build directories, and add the binaries to the build user's PATH. Once that's complete I can use a custom builder that builds documentation using tools like JSDoc.
The work-around I'm putting together will involve hooking into setup_environment to run a Heroku buildpack for setting up a Node environment, in place of the Python and pip steps. Once that's complete, the build process should continue on to using the custom builder.

Part of this work is happening now, at https://github.com/rtfd/sphinx-autoapi
The second half of this work will be implementing the necessary bits that integrate with Read the Docs -- any sort of configuration that needs to pass through, etc.

Closing this, as it will be an AutoAPI feature, not something inside RTD itself.


RFC for renaming app.toml

rendered
Let the bikeshed commence!

Is it possible that project.toml (with the same name and schema) becomes the app descriptor that was roughly described in #3? If yes, we should consider that when deciding on the name (even though it may not change anything).

@jkutner I'm fine with that. I'm not strongly opinionated on the filename itself, but we should move away from app.toml in the spec since there are no plans to make that change.

@nebhale Note that we voted in the WG meeting to revert the spec commit changing launch.toml to app.toml, given that there was never an RFC for it and it was never implemented in the lifecycle. This RFC is now about renaming launch.toml to something else.
@hone changing the title accordingly, hope you don't mind 😄

@sclevine yeah, no worries.
@nebhale I'm fine with that. Did you have new work in mind?
Should we just close/merge this in?


bottle_spec: reduce code duplication in tests

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

This PR reduces some code duplication in test/dev-cmd/bottle_spec.rb, making them a bit easier to read

Review period will end on 2020-12-25 at 20:00:18 UTC.

Review period ended.


Export types needed to implement custom serializers.

Serializer and option types are not exported so you can't use the types or implement a serializer with typing

Thanks @niemyjski!

Can we please get a new release

Oh, I think the deploy didn't go automatically because it was missing semantic commit message. Will be in the next release


initial pr

Initial contribution ip check ticket is here: https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/8594
Will not be merged until ticket is solved positive


Corected spelling of Vespene and gunicorn

Corrected a few misspelled words.

 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 - this will appear the next time I push the docs.  One of the magical powers of the gnuicorn is it is impossible to spell correctly.


Event Hubs API?

Is this included in the spec?
https://msdn.microsoft.com/en-us/library/azure/mt639489.aspx

@sazeesha can you provide a timeline for this?

We will have a EventHub swagger spec out in around two weeks.

Here it is : https://github.com/Azure/azure-rest-api-specs/tree/master/arm-eventhub/2015-08-01/swagger


Clean up current tests and workflows

Tox is not necessary for running the tests so consider removing it and just use standard pytest commands to run the tests. This would also require changes to the GitHub Action workflows and coverage report.

We will need to update the follow documentation for tests to match our updates: https://github.com/deephyper/deephyper/blob/develop/tests/README.rst


Sample app: Possible unhandled promise rejection (id:1)

Description
I just installed the react-native scaffold as described on the website, using the CLI approach (I'm familiar with Xcode development). The app spawns up, but as soon as I click on one of the documentation links, I get an error message in a popup box:
"Possible unhandled promise rejection (id: 1)"
If I look into the log, I can see "TypeError: Netowrk request failed".
I tested with Xcode 15.1 (and 15.2 beta) on macOS Sonoma and iOS 17.1.2, iPhone 12 Pro Max.
Steps to reproduce

Install the "AwesomeProject" sample scaffolding application as described here:
https://reactnative.dev/docs/environment-setup?guide=native
Open the project in Xcode
Set the development team, so automatic signing works
Run the app it on an iOS device.
Tap on one opf the links under "Learn more"

React Native Version
0.73.2
Affected Platforms
Runtime - iOS
Output of npx react-native info
info Fetching system and libraries information...
System:
  OS: macOS 14.2.1
  CPU: (16) arm64 Apple M3 Max
  Memory: 87.41 MB / 48.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 18.18.2
    path: ~/.nvm/versions/node/v18.18.2/bin/node
  Yarn:
    version: 1.22.19
    path: /opt/homebrew/bin/yarn
  npm:
    version: 10.2.4
    path: ~/.nvm/versions/node/v18.18.2/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.14.3
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK: Not Found
IDEs:
  Android Studio: Not Found
  Xcode:
    version: 15.2/15C5500c
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 11.0.21
    path: /usr/bin/javac
  Ruby:
    version: 3.3.0
    path: /opt/homebrew/opt/ruby/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react: Not Found
  react-native: Not Found
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

Stacktrace or Logs
Could not copy it from the app. Please find a screenshot below.

Reproducer
https://github.com/facebook/react-native
Screenshots and Videos
I tapped on "The Basics"

Ok, found it out. I did not start the Metro server.


missing gulp dependency

gulp is only required in devDependencies but used in index.js what makes this module throw an exception when used as a dependency of another module.
This is because gulp doesn't end up in node_modules if one doesn't explicitly do npm install inside this modules's folder (as it is a devDependency).

That's not how it is designed.  You install gulp in your project, and run-sequence uses your copy of gulp.  If it had it's own copy, it wouldn't work.
If you want to use run-sequence as a dependency of a another module, then that module needs to depend on gulp.  run-sequence only depends on gulp for the devDependencies because it needs it for testing.

module.exports = runSequence.bind(null, require('gulp'));  will always throw an exception if gulp is not present in run-sequence/node_modules.
As npm install won't install the dev dependencies of run-sequence when run-sequence is a dependency of another module, gulp will not be available for run-sequence although the requiring module has gulp for itself.
Your tests require gulp so you must have installed the dev dependencies before they won't be able to test this special case.
You are right that it doesn't make any sense for run-sequence to have its own gulp so you shouldn't do require('gulp') in your index.js as it will throw exception.
The usual way of injecting gulp is to export a function that gets gulp as an argument. Check these 18.000+ examples:
https://github.com/search?utf8=✓&q=module.exports+%3D+function(gulp)&type=Code&ref=searchresults
It is true that it is a breaking change, but hey, we've got semver :)

There are literally thousands of users using run-sequence without error.  If gulp is installed by the module that is using run-sequence, it works.  If you are using it in some weird way that doesn't have gulp installed, then I'm sorry, but I'm not interested in fixing that.
I already export a method (in use by others) that allows you to pass in a different instance of gulp.  It's not worth complicating the usage of a fairly simple and widely used library, for this one case.

Ok, I see your position. We just forked.
for the record it breaks when used in combination with node link because of this issue: https://github.com/joyent/node/issues/4757
It is a typical use case when developing a module required by a working project to avoid having to commit, push, pull to test if things work.
For anybody else having this problem, the solution is just to remove the require('gulp') or surround it in a try catch clause as in https://github.com/antwerpes/run-sequence.
Thanks for your work


SOLR-13243: Correct initial capacity of ops

https://issues.apache.org/jira/browse/SOLR-SOLR-13243
Quick improvement for initial ops capacity

I think it's more readable this way, but it's still a preference.

I think @madrob's proposal fits better, a List.of() looks cleaner to me too.

Thanks @heythm


Problem when trying to download (PMIDs): 9029852, 19092482,11382209

This happened to all the three articles i've tried to download, what could be happening?

Thanks

What journal is it published in, and could you download it normally from the computer you were on?

I had the same problem, but was able to download all journals manually.

Same problem re: **fetching of reprint failed

There seems to be a problem with Mechanize; it won't search any of the sites.  I will work on this when I have time, but it wont be for a little bit

Hi @billgreenwald, thanks for taking the time to make this. I'm having the same problem as everyone else for every string of PMIDs I've tried.
Here's an example from when I ran ruby pubmedid2pdf.rb 28238146,28238140,28237713,28236573,28236485,28235723,28235591,28234628,28234487,28234430,28233996,28233941,28233218,28233053,28233000,28232590,28232293,28232288

Hi everyone,
I was able to fix this issue by using rvm to install ruby 2.2.7, then installing rubygems for 2.2.7 and redownloading the gems for the program.  At this point in time, the default gem versions worked, but for reference, they are as follows:
socksify-1.7.1
camping-2.1.532
mechanize-2.7.5.
If anyone needs help with the above steps, let me know.  If someone can try this and see if it works on their system too, that would be great.  I have tested a few pmids from above and was able to fetch them properly now.

This is a great tool! I'm running ruby 2.2.7 and the gems you specified but when I enter ~50 PMIDs it will return only 6 PDFs and the rest fail. Do you know why this is?
Thanks in advance! If I can get the % of PDF hits up this will be amazing!

Could you list some of the PMIDs that did not work?
Thanks!

Sorry for the delayed response! I have been really busy lately, but will look into it as soon as I have time.  It seems that some of the crawling methods are not being called properly.
Could you give me a PMID that worked for you as well?

Hey Bill,
No problem at all, I am not in rush at this point.  25325179 is one PMID that worked for me.  Thanks again for doing this.
Colby Wilkinson
MSc Candidate, Biostatistics
Harvard School of Public Heath
Cell:<PHONE_NUMBER>

On Feb 1, 2018, at 2:11 PM, Bill Greenwald<EMAIL_ADDRESS>wrote:
Sorry for the delayed response! I have been really busy lately, but will look into it as soon as I have time. It seems that some of the crawling methods are not being called properly.
Could you give me a PMID that worked for you as well?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/billgreenwald/Pubmed-Batch-Download/issues/2#issuecomment-362370291, or mute the thread https://github.com/notifications/unsubscribe-auth/APpMd21Der01pkZPZ24Bn-s-6ACA0-kAks5tQgxHgaJpZM4LNgxO.

your email signature posted with that last message from you, including your cell.  Not sure if that was intentional, just wanted to let you know its visible in plaintext

Thanks for the heads up!

It looks like the issue is a known problem with Mechanize.  I will keep looking into it and see if I can figure out a way to fix it, or if rolling back to an earlier version of ruby may help.  The error is apparently well known:
too many connection resets (due to end of file reached - EOFError) after 0 requests on 26040640

Hi everyone,
sorry to revive an old thread; mechanize was proving difficult and many people online had similar problems dating 2 years now, so i ported the code to python and cleaned it up to work with how current websites handle their metadata for scientific articles.  I have uploaded and changed the github accordingly.  I have tested the pdfs in this forum, and have gotten most to work, but have some errors that it would be helpful if you could reproduce, assuming you still care about using the program.
@colbyw5 could you test pmid 25778747.  I got an econnreset error, which I can't tell is on my end or their end. The others worked.
@zakimolvi 28233000 can't be obtianed due to the SSL on the host site.
If you have any questions, let me know.


File not found

fixed the file not found error when the script doesn't exist.

@briancohan
Is this an expected output? Looks like it isn't showing the help page when no args are passed anymore.
wtpython on  file-not-found via wtpython via 🐍 3.9.5
➜ wtpython
usage: wtpython [-h] [-n] [-c] args [args ...]
wtpython: error: the following arguments are required: args

The expected behaviour for command line applications is to show the minified usage and error message if arguments are incorrect. This is infact also the default behaviour of parser.error(...). I have fixed it to show a more appropriate message, but I don't think we should show the full help message unless explicitly asked for (it is fairly verbose)

I seem to have messed up the merge commit (ugh). My git skills are not too hot - can someone please fix it? Just need to add the extra argument for clearing the cache. @briancohan


Add CrowdSec in README as a new source

Hi @hslatman, is this repository still active ?

It is! Not on a daily basis, though; I tend to merge in batches because of day job and other responsibilities. I think I can soon do another round.

I am glad to hear from you, I actually was wondering since the last commit was about 6months ago. I understand, everyone has duties :-)

Thank you, @mazzma12! 😄


USDT transfer

I want to transfer trc20-usdt from one address to another one
my sender account looses 10 trx but transfer not taking place. Can you please explain, I activated the account by 1 trx.

it was due to low energy


KAFKA-3721: Put UpdateMetadataRequest V2 in 0.10.0-IV1

Do we need a documentation update?

We should also start a branchbuilder to make sure the upgrade tests didn't break

From a brief look, it seems fine apart from one comment (and one question for Jun). It would be good for @junrao to take a look too. And we definitely need to run the system tests.

Thanks for the patch. A couple of comments.

We actually made another inter-broker protocol change related to SaslHandshake after the rack-awareness change. The following is how we are using it in KafkaConfig.

val saslInterBrokerHandshakeRequestEnable = interBrokerProtocolVersion >= KAFKA_0_10_0_IV0
So, if we want to do this right, we will need to add 0.10.0-IV2 for SaslHandshake change and use 0.10.0-IV2 in KafkaConfig.

It's a bit hard to see which internal version is for which inter-broker protocol change. Could you add a comment for all the 0.10.0-IV* to describe the corresponding protocol change?

@junrao since we are doing this at the wrong time, is there any benefit to adding two versions? People could be at any point in trunk at this point after all.

Upgrade tests running: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/452/console

@ijuma : Yes, you are right. It's a bit too late to fix this completely at this moment. What we can probably do is to just change the reference to KAFKA_0_10_0_IV0 in KafkaConfig to KAFKA_0_10_0_IV1. This means that KAFKA_0_10_0_IV1 will be corresponding to both the rack-awareness and the SaslHandshake change.

Sounds good to me.

Thanks @junrao . I updated the patch to point SASL handshake to 0.10.0-IV1 as well.
@gwenshap I just submitted a new branch builder test. Should we stop the current branch builder test?

Thanks @becketqin - I stopped mine and let yours run.

killing the system test was a REALLY bad idea.
Looks like the test nodes are now in a bad state and won't run. I'm hoping @granders is around and can help.

Retrying, just in case.... https://jenkins.confluent.io/job/system-test-kafka-branch-builder/454/console

actually... https://jenkins.confluent.io/job/system-test-kafka-branch-builder/455/console

looking better now :)

Thanks, @gwenshap

Thanks for the patch. LGTM


Use custom build comand for editable install of the project

Problem description
We have a python c++ project and there are other commands that need to be run before running python -m pip install -e . , setting environment variables(CMAKE_ARGS) , preparing necessary data etc.
For this purpose we have created an install.py which works cross platform.  It would awesome if a custom command could be specified for the pyproject.toml editable dependency that has been implemented in https://github.com/prefix-dev/pixi/pull/1084
example
[pypi-dependencies]
minimal-project = { path = ".", editable = true, build_cmd=["python install.py develop"]}

@vigneshmanick what about using a build-backend allowing you to do this, and use a pyproject.toml manifest

am already using a build backend but still some variables/steps need to be executed so that the build backened understands it.
Example prod and development build
development build
curl <test file> &&   CMAKE_ARGS="-DEXPERIMENTAL_FEATURE=on"  pip install -e .
prod build
curl <prod file>  &&   CMAKE_ARGS="-DEXPERIMENTAL_FEATURE=off"  pip install -e .
Generally builds are not as simple as pip install .  there is always some customization that needs to be done both at the build backened and preparation stage.
Currenly i have removed the pypi-dependencies section and instead use an install task to achieve the editable install of the package.

Maybe you could do this in an activation script of the environment? Wdyt @ruben-arts

activation script means writing one for each platform which is an overhead and also it seperates a build step out of the actual build part. These are not variables that are acutally required in the environment but only for building.
Let me give more detail for the scenario that i have

we have different config files for different build configurations which result in different cmake args
the build step can be changed by specifying which config file to use/ and also the build mode (e.g python install.py prod, python install.py dev). The config files are read by configparser module and the necessary variables are passed to cmakelists.txt
The build wheel also requires data files that are not part of the repository but are downloaded from multiple other repositories
All of the above are handled via an install.py which handles these tasks before build starts and then builds the project (editable for development, wheel for prod which is then used as artifact for further pipelines)

Allowing a custom install command will solve all of the above instead of having to resort to splitting the processes. This also is more maintainable since the entire build logic is in one place and not split.

I believe some build-backends would allow you to do this. For instance, hatchling has a plug-in concept, that allows you to modify a fair bit of the build process

I am using both scikit-build-core and scikit-build . The above steps still apply.

Maybe you could do this in an activation script of the environment? Wdyt @ruben-arts

@vigneshmanick This pr could possibly help you in the short term, would you like it? I'm holding it back because if the issue described in the PR but it might already help you. https://github.com/prefix-dev/pixi/pull/1156
I'm not sure if custom build commands would open a can of worms for us, but I can see the use-case. We should still take a look at the "build environment" for pypi packages so the environment variables would most-likely also be a part of that.
https://github.com/prefix-dev/pixi/issues/1130 https://github.com/prefix-dev/pixi/issues/1110

Thanks for the response, i checked the  #1156 and is definitely very helpful for tasks within the environment but am afraid won't change much to the situation in this issue.
A custom build environment as mentioned in #1110 would definitely make the process more flexible since this step isoloated.
For my understanding, the dependencies (pypi /conda) need to be solved for all the environments/features/platforms  so that you can ensure that it's valid and this is why you are hesitant to add a custom command?
From my point of view, if i as a user have resorted to a custom command, then the entire responsiblity of ensuring that the build works lies on me. In this case, i would expect that pixi just ignores solving this custom dependency and solves the others and once complete the custom command is run at the end as is.

Also to add, i am a bit hesitant to split the build process since the current process works the same if i have a venv , conda or within pixi. Splitting the process will result in more maintenance since the build has to work also when no pixi is available.

I'm a bit hesitant to add it because it would partially circumvent the entire build-backend in the pyproject.toml and I feel that build backends like meson or anything in use by the numerical python world would have a use-case similar to yours. I mean you can do anything you want with a setup.py e.g. But it could be that I'm missing something :)

E.g. would something like this not be an option for the custom steps? https://github.com/rmorshea/hatch-build-scripts

hatch cannot be used for my use case
Yes CMAKE_ARGS can be defined but i want to modify the CMAKE_ARGS before building dynamically
in addition, customizing the build is one step but i need both build customization and src customization (i need to gather extra data files from other sources which are also part of the wheel based on other parameters) which is not possible(most probably never will) in any of the current build backends.
Currently my build works if i remove
[pypi-dependencies]
minimal-project = { path = ".", editable = true,}

and just add
[tasks]
install = "python install.py develop"

pixi install && pixi run install
If you feel that this is not a use case for pypi-dependencies/hesitant to introduce , that's perfectly fine. I just want to bring to your attentiont that such a feature would improve the QOL /extend the reach in using pixi since all the plumbing is already there :) . As you had already mentioned, this is not an uncommon scenario. Users will always find a way to stretch any tool to it's limits (maybe even repurpose it to cases for which it was not foreseen)

What's the biggest benefit of this instead of doing the work in a setup.py and using setuptools. I know the SDL backend on pypi is doing something like this to download the .so files.

Biggest benefit is that i have a install script that works cross platform / across python platforms (pixi, conda , venv) and also maintain a seperation between what modification is required for the project (this is within pyproject.toml ) and what's required for the build clearly seperated.
setup.py is outdated and the build systems are moving /already moved to pyproject.toml only based build, some build systems will not recognize the setup.py and expect the project data to be in pyprojec.toml / env variables.
In my case the entire setup.py contents has been removed replaced with pyproject.toml + install script.

Yeah, so thats sounds awesome :) But maybe we could find to support it in some other way, some people were mentioning a postinstall or something along those lines.

yes that would work too, one of them is from me actually https://github.com/prefix-dev/pixi/issues/1146

Closing in favor of https://github.com/prefix-dev/pixi/issues/1183


versatile-data-kit: vdk-health dockerfile

The VDK project requires some way to monitor the health of the
repository. This change includes a Dockerfile which, when deployed,
instantiates a set of tools which collect data from Github and
present it in a Kibana dashboard.
Testing done: ran locally
Signed-off-by: gageorgiev<EMAIL_ADDRESS>

Add short readme.md - what is this about and how to configure it (e.g get github API token)


Easymail Website needs to have form to join slack channel.

Currently only an alert exists. But there's no form to accept email ID and inform chan owner that someone wants to join.

Thanks for the suggestion. As soon as we release the new version of EasyMail, we will have more time and we will consider the option to add the form on the website.

Hi mate,
I have just reassigned the task to you. May I please ask you to do it and we will review you Pull Request once you are ready.
Thanks!

No action on that suggestion, closing the ticket. Please reopen in case still relevant


Scrollbars not working after support of multiple email accounts

Somehow the horizontal scrollbar is small while no folder is selected, but after selecting a folder, its just expanding to full size.

Can still reproduce this by just opening the application

this seems to be fixed already as we could not reproduce it in the newest version


Fix a couple of more tests to not create files in the source tree

Summary:
These were not being flaky, but they're still making the tree dirty.
These tests were using lldbutil.append_to_process_working_directory to
derive the file path so I fix them by modifying the function to return
the build directory for local tests.
Technically, now the path returned by this function does not point to
the process working directory for local tests, but I think it makes
sense to keep the function name, as I think we should move towards
launching the process in the build directory (and I intend to change
this for the handful of inferiors that actually care about their PWD,
for example because they need to create files there).
Reviewers: davide, aprantl
Subscribers: lldb-commits
Differential Revision: https://reviews.llvm.org/D43506
git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@325690 91177308-0d34-0410-b5e6-96231b3b80d8
(cherry picked from commit 6d2b66f2740b2600b5afef8ba704245ce47a4343)

@swift-ci please test


[feat] Add CarmineOptions faction

I would like to add Carmine Options protocol as a faction

Hi @DaveVodrazka please can I work on this issue

@DaveVodrazka I would like to work on this issue, please.
Creating a PR will be very swift

@DaveVodrazka Can i work on this

Hi, I created this PR: https://github.com/keep-starknet-strange/art-peace/pull/256/commits
Feel free to make changes to it!


Don't hide history buttons

Just enable / disable them depending on history length.
https://bugzilla.redhat.com/show_bug.cgi?id=1326220

Unrelated product features test failure, merging.


We don't see how the last pill is eaten

I know why it happens. This is because on every tick we do the game state logic first, and then we do the rendering of the game state. We detect that all the pills are eaten in the state logic function, and if they are we load the next level immediately in the same function before the rendering. That's why we don't see how the last pill is eaten.


Updated language column to use same label as in recent races page

title: Issue #554

Discord Username: @industrializeddark
What type of PR is this? (select all that apply)

[ ] 🍕 Feature
[X] 🐛 Bug Fix
[ ] 🚧 Breaking Change
[ ] 🧑💻 Code Refactor
[ ] 📝 Documentation Update

Description

Updated snippet language column to use snippet labels from config file.
Related Tickets & Documents

Related Issue #554
Closes #554

QA Instructions, Screenshots, Recordings

UI accessibility concerns?

Added/updated tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

[optional] Are there any post deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

LGTM!


Clean up assignment.yml

The current format is not ideal. It has fields that are not entirely intuitive and fields that can be combined, e.g., into a range. Further, we could use a different format, e.g., the cue format seems like a strong contender. Either way, we should declare a cue schema, even if we keep the yml format. More details about cue here.

We can remove the name attribute from the yml file since we can derive the name from the encapsulating folder. We still need the name field in the assignment proto message type, since it is used in the frontend.
We can probably remove the assignmentid field from the proto message type as well since we can use the deadline to order the assignments. Or at least, if the id is not specified we can derive the id from the ordering provided by the deadline.

May require more thinking and checking the code...


listWeek view

Hi everyone,
I'm trying to display a listWeek view on the ui-calendar but it's not working, any ideas?
What I want to display: https://fullcalendar.io/js/fullcalendar-3.4.0/demos/list-views.html
And the only ones I manage to display are the ones with the "see example" link on here:
https://fullcalendar.io/docs/views/Available_Views/
Any thoughts would be really helpful!
Thx

+1

The list views were introduced in fullcalendar v3.0.0 which is not currently supported by ui-calendar.
You can follow https://github.com/angular-ui/ui-calendar/issues/454


Proposal - Add visibility Attribute to <podcast:value> Tag

There is no explicit guidance in the <podcast:value> tag around whether Boosts should be public or private.
Since there is no specific mention in the spec of the private nature of Boosts - I don't see why it's a problem for apps to display them publicly.
However I do think it's important that both listeners and podcasters have choice over what happens.

For Listeners - Fountain has a toggle option in the settings to switch boosts to private. Other apps should offer the same if they want to show Boosts publicly.
For Podcasters - We need a way for podcasters to signal what they want.

I propose we add a visibility attribute to the <podcast:value> tag which could have either public or private. Of course defaults matter here but I think the attribute should be optional and if it's not set apps should be able to decide what to do.
Happy to create a PR if we get consensus this is a good idea.

Ultimately, any tag is a means for a podcaster to share his content with the world, and dictates what content he wants shared with the world. We assume if he doesn't have a chapters tag, then he doesn't want chapters, and shouldn't open up a means for listeners to create audience created chapters, no matter how cool it is. Even if it would be cool for the audience to share in the creativity of the podcast, ultimately, that's the podcasters choice, so the lack of a tag indicates they don't want the content attached to their feed.
Since we're talking about value though, we have to determine what we think the podcaster assumes with a certain tag, and since most podcasters don't read the specs, we should try to understand what is most intuitive to most podcasters. I argue that most people assume a Boost is similar to a Direct Message, thus a private message to the podcasting team (value block). If this assertion is true (and that's really what we need to determine when deciding the default for visibility), then I think the default should be private, and the podcaster has to make the conscious choice to opt into public.

Two changes fixes the 'lightning comments' for me.

Label them Boostagrams, not Comments. They do not have the properties of comments. The are simply not the same.

Boostagams are being surfaced in many ways, this is an agreement between podcaster and audience. This would surface as a hierarchical list of boostagrams. Not threaded like comments. This flag would signal this agreement has been made and it is ok to display boostagrams publicly.

Brainstorming the problem based on @adamc199 comments.

I agree Boostagrams and Comments aren't the same.
If an app wants Boostagrams to made public, then a url needs to be provided so an app can display all of the Boostagrams.
lightning:comments already provides a url, but per point 1, that tag should return a comments thread.
perhaps a new tag similar but different called lightning:boostagrams which
a) indicates the podcasters wants Boostagrams to be made visible in the app
b) provides the url in which to retrieve the Boostagrams

The podcaster would have to have some sort of service that aggregates the Boostagrams for a particular episode and returns the appropriate data structure for all of the apps to parse, but Fountain has already solved that problem for podcasters using a Fountain wallet.
This also opens up a neat opportunity for the boostagrams.json file to contain other things in it, such as top ten boosters of all time, a running total of value the podcast has received, pledge drive goals/totals, etc.

I argue that most people assume a Boost is similar to a Direct Message, thus a private message to the podcasting team (value block).

@thebells1111 I'm just not sure this is true. I think the tools / services to publicise boosts just haven't been built yet. My assumption was always they would end up being public. Again it's all about choice and if a podcaster wants them public this should be allowed - hence the need for something like the visibility attribute to enable choice and reduce confusion.

Label them Boostagrams, not Comments. They do not have the properties of comments. The are simply not the same.

@adamc199 up until now you're right that boosts have not had the properties of comments. But does that mean that they can't be integrated? Again it should all be about choice - but if a podcaster chooses to let their listeners to reply to each other's Boosts - is that wrong?

This flag would signal this agreement has been made and it is ok to display boostagrams publicly.

@adamc199 agree this is why the flag is needed and Fountain will respect it and not show Boosts publicly if instructed to by the feed.

up until now you're right that boosts have not had the properties of comments. But does that mean that they can't be integrated? Again it should all be about choice - but if a podcaster chooses to let their listeners to reply to each other's Boosts - is that wrong?

I didn't say it is wrong, but they are not comments. I may choose to show boostagrams myself, but I also want comments.
I've stated my case: Boostagrams should be labeled as boostagrams. Surfacing should be at discretion of the feed.

I didn't say it is wrong, but they are not comments. I may choose to show boostagrams myself, but I also want comments.
I've stated my case: Boostagrams should be labeled as boostagrams. Surfacing should be at discretion of the feed.

@adamc199 ok makes sense - if we can get this attribute added I think we'll be good 👍

I'm also not sure it's true. That's my assumption and intuition, but everyone's different, and I could be very wrong on what most people find intuitive and expect the default to be. I hope we can get some more input so we may have a better feeling of what most people expect the default to be, because my only concern with visibility is what the default value would be. I personally don't care either way, I'm just trying to look out for the podcaster as much as possible and make this intuitive for them.

@thebells1111 yep I'm not sure if my assumption is true either! I think if we can add the visibility attribute we can give podcasters the choice - but by making it optional we give apps the freedom to experiment - which in my view will push Podcasting 2.0 forward even if sometimes we get things a bit wrong.

boosts and boostagrams should to be public, else they are just "donation buttons", use a funding tag for that I say.
The real question is rather they'd are anonymous or not, might want an attribute for that.

I feel boostsgrams evolved naturally as the lightning equivalent of the PayPal donation note  but without the middleman and standardized fee of PayPal.
There is no obligation for the podcaster to read a donation note, whether it be PayPal or Boostagram. They certainly are not surfaced to the audience by default, and if they are surfaced to our audiences thus far it has been by explicit choice and has been discussed on our shows. I can think of a few examples:
On No Agenda often people request anonymity in donation notes. Impossible if donation notes were programmatically surfaced by default.
On Angry Tech News producers are thanked by name but no notes or donation levels are read.
I recall a Podcasting 2.0 episode in the early days of the boostagram that mentioned there was a "rude boostagram" on Podland that was not read. On the other hand Podcasting 2.0 have chosen to read all their boostagrams, even the rude ones. Not possible to make that choice if the boostagram is revealed automatically.
Helipad surfaces boostagrams automatically but privately, to the node operator.
The IRC bot and Boost Bot have also been built in a way that the podcaster must explicitly activate these services to surface boosting, and the audience is made aware how and where these surface.
I think certain audiences have come to expect the boostagram to appear in certain places like IRC or fedi, but ultimately the podcaster should be able to direct how this goes down and opt in at the level of surfacing they wish the boostagrams to have.

Agree. And if they are surfaced, they are "Boostsagrams", not " Comments"

I like the points this discussion is raising.
I think as a means of financial support, Boostagrams should be assumed private by default. This protects both the privacy of the listener and the private financials of the podcaster.
In consideration of privacy policies and privacy laws, I think the safest thing is always to build in automatic privacy, allowing people to opt-in to tracking/publicizing. But, of course, I expect the podcaster to receive the information and handle it properly.

As a listener I would enjoy if this functionality were implemented, and in such a way that it could be overridden per boost (I realize apps would need to also surface this).
Some apps surface splits which enables me to exclude nodes/bots that automatically relay things publicly but nothing is really obvious about what that does or means currently; something like this proposal I think works a great deal of the way towards addressing this.


Upgrade deps and Node version

Successor of #943
As I wrote in #943 I'd really recommend to install Renovate to keep all those deps up to date and to early find issues.

Great thanks.

Hello Everyone,
I would like to disable zoom at the last child cluster.
Here is my code:

Leaflet debug page
<link rel="stylesheet"<EMAIL_ADDRESS>integrity="sha512-07I2e+7D8p6he1SIM+1twR5TIrhUQn9+I6yjqD53JQjFiMf8EtC93ty0/5vJTZGF8aAocvHYNEDJajGdNx1IsQ==" crossorigin="" />

<meta name="viewport" content="width=device-width, initial-scale=1.0">
<link rel="stylesheet" href="map3/screen.css" />

<link rel="stylesheet" href="map3/MarkerCluster.css" />
<link rel="stylesheet" href="map3/MarkerCluster.Default.css" />

Mouse over a cluster to see the bounds of its children and click a cluster to zoom to those bounds

on click of last cluster, it should not be reach at max zoom.
How can i do that? Please help
Thanks


Custom tuple parser, types of fields, raw tuple data.

Please consider making the method Tuple::as_buffer(&self) -> Result<Vec<u8>, Error> public.
I would like to parse unknown tuples with RMP crate.
https://github.com/picodata/tarantool-module/blob/e77a6cb51a6d61d2aaf4bece2674ee753c5d0806/src/tuple.rs#L144
example:
let buf = tuple.as_buffer()?;
let value = rmpv::decode::read_value(&mut &buf[..])?;
debug!("{:?}", value);

Or maybe is it possible to return the type of field by the filed index?

Sorry for the late reply. You can use Tuple::as_struct method for that like so:
let value = tuple.as_struct::<rmpv::Value>()?;
dbg!(value);

Don't forget to enable with-serde feature in rmpv crate:
[dependencies]
rmpv = { version = "^1.0", features = ["with-serde"] }


git version 2.7 requirement vs available centos builds of git

Can we make git version requirement optional somehow? The latest available git-sclo-build for Centos seems to be 2.5.5, and 2.9 is broken since forever. Do you really need to require version 2.7+?
I'm stuck with Centos, and can't do anything about it short of compiling git from sources which I would rather avoid. It's bad enough I have to separately install git >2.0 to be able to use GitLens, now it requires 2.7.2.

The Git version notification is a warning not a block, but there are features of Git in version 1.9 or later that are really required, and I think the only thing that requires 2.7.2 is dealing with tags, because prior to that version Git didn't support the --format option.

OK sorry my bad. I was under the impression that this is a requirement blocking it from functioning, it turned out that I simply needed to enable it. I'm using the git version 2.5.5 for now and it seems to be working okay so far.


Data visibility

A module owner should have the option to select in which context the data should be visible:

Private. Only the owner and, if the owner is member of a group, the data administrator of the group.
Group. All group members can see the module data.
Context. The module data will be visible for all members in all groups belonging to the context.
Public. The module data will be visible for any logged in user.

Added option to set this for modules. Remains to implement to show module data accordingly to visibility.


[Android] [Feature request] Delete confirmation / 'sticky' tag

One thing that might save important items from being accidentally deleted is a confirmation dialogue when deleting tagged and pinned items.
Or, maybe add a special tag, e.g. 'sticky', that if attached to a clipped item, throws a confirmation dialogue on deletion.

I mean you can still undo the deleted items, right? Is it necessary?

Yes, that's two taps to delete, and one more within a couple of seconds to undo, otherwise it's gone.
I like the swipe delete but there's no undo for that.
I've seen a good solution on the XDA forums app.
You can swipe across a little which reveals the delete button, and taking your finger off leaves the delete button visible, and if you still want to delete, you tap it. Then if you show the undo option after that, it would be like combining the hold and select delete option, and the swipe to delete option into one solution.

I'll add the undo button for swipe delete.
I have an idea so when you delete an item the undo snackbar will show but it won't hide unless you touch anywhere on the screen except the bar itself. This will it will be persisted indefinitely until the user touches anywhere on the screen.

Great! Thanks!
And, yes! Great idea! I'd appreciate a persistent undo button.

Hey, this feature is added you can test it & give me early feedback before merging into the final update. Here's how you can do that.

Works great for me! :)


AutoBootcamp - breaks when no troops are in polis

Try to send attack when troops are not in polis, keep spamming errors

Should be fixed in the last push


QGB commands UX discussion

          [discusison around planning]

certainly not something we do in this PR, but as we've discussed in the past, we will eventually need to nail down the UX. All the tool that are needed are in these commands, but devs don't really have easy access to them.
If they're looking at these specs, I would feel a bit confused if this were the only api, since these just logs a yes or no.
Originally posted by @evan-forbes in https://github.com/celestiaorg/celestia-app/pull/1911#discussion_r1230217122

no longer needed as we're switching to BlobstreamX


sys/linux: add descriptions for Qualcomm IPC Router

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

You should use your google.com email address for commits, by putting the correct settings in user.name and user.email in ~/.gitconfig or .git/config.  Afterwards you can use git commit --amend --reset-author to fix this for your existing commits.
This also needs to be rebased to resolve the conflict in the CONTRIBUTORS file.

You should use your google.com email address for commits, by putting the correct settings in user.name and user.email in ~/.gitconfig or .git/config. Afterwards you can use git commit --amend --reset-author to fix this for your existing commits.
This also needs to be rebased to resolve the conflict in the CONTRIBUTORS file.

This is still not addressed fully.
Necip, please do git rebase -i and amend all commits as Eric showed.
Also please try to use @google.com email address.

The change itself looks good to me.

The commit author is now changed as suggested.
Thanks for your review!

Perfect! Thanks!

Do we also need to enable CONFIG_QCOM_IPA in syzbot config?

Do we also need to enable CONFIG_QCOM_IPA in syzbot config?

No, only CONFIG_QRTR and CONFIG_QRTR_TUN are needed.

That was fast...  First bug<EMAIL_ADDRESS>Opening /dev/qrtr-tun and writing 0 bytes to it crashes the kernel apparently?
Might be an "easy" bug to fix if anyone is looking for kernel bugs to fix...